keras-io

keras-io copied to clipboard

Updating Image Captioning.py and Image Captioning.ipynb by adding visualization of the Image Data.

def plot_images_data(filename):
    """
    Loads the image data 
    params: filename: File path that contains the image data
    """
    #define the size of plot window
    plt.figure(figsize=(20,20))
    for view_images in range(9):
        plt.subplot(3,3,view_images+1)
        #resize and plot the image
        img = cv2.imread(f"{filename}/{random.choice(os.listdir(filename))}")
        img = cv2.resize(img,(200,200),interpolation=cv2.INTER_CUBIC)
        plt.axis("off")
        plt.imshow(img)

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

Hey @fchollet,

Should anything else need to be updated here?

@fchollet, any updates?

Hi @lucifertrj, thanks for the PR. I believe @fchollet wants you to follow Google style for doc strings. So this

def plot_images_data(filename):
    """
    Loads the image data 
    params: filename: File path that contains the image data
    """

would look like this

def plot_images_data(filename):
    """Loads the image data.

    Args:
        filename: File path that contains the image data.
    """

Could you 1) fix the merge issue; 2) update the docstring; 3) regenerate the notebook and markdown?

This PR is stale because it has been open for 14 days with no activity. It will be closed if no further activity occurs. Thank you.

This PR was closed because it has been inactive for 28 days. Please reopen if you'd like to work on this further.