Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updating split range from [0, 1] to (0, 1) in keras.utils.split_dataset #18028

Closed
wants to merge 7 commits into from

Commits on Apr 21, 2023

  1. Updating split range from [0, 1] to (0, 1) in tf.keras.utils.split_da…

    …taset to make it more intuitive
    
    In the split_dataset API which is used to split a dataset into left and right parts. But documentation states it splits dataset into left half and right half where the parts need not be equal parts.The split can be anything in range(0, 1).Hence I think to replace the word half with part.
    Also in the argument section and exception section the range for split used as [0, 1] which is not a correct notation as 0 and 1 are actually excluded in the range and raises exception if we use either 0 or 1 for either of left_size or right_size for splitting the dataset. Hence i have replaced [0, 1] with (0,1) to make it more intuitive. Thanks.
    SuryanarayanaY authored Apr 21, 2023
    Configuration menu
    Copy the full SHA
    32c4982 View commit details
    Browse the repository at this point in the history

Commits on Jun 27, 2023

  1. Update dataset_utils.py

    Updated the range notation as requested
    SuryanarayanaY authored Jun 27, 2023
    Configuration menu
    Copy the full SHA
    9bc86d6 View commit details
    Browse the repository at this point in the history

Commits on Jul 19, 2023

  1. Update dataset_utils.py

    Don e the formatting changes as requested
    SuryanarayanaY authored Jul 19, 2023
    Configuration menu
    Copy the full SHA
    0a51eeb View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    ed636c9 View commit details
    Browse the repository at this point in the history

Commits on Aug 1, 2023

  1. Update dataset_utils.py

    Reduced the line lengths to <80 chars as required for black format
    SuryanarayanaY authored Aug 1, 2023
    Configuration menu
    Copy the full SHA
    38aab6f View commit details
    Browse the repository at this point in the history

Commits on Aug 24, 2023

  1. Update dataset_utils.py

    Done the formatting as suggested. Thanks
    SuryanarayanaY authored Aug 24, 2023
    Configuration menu
    Copy the full SHA
    a0b1378 View commit details
    Browse the repository at this point in the history
  2. Update dataset_utils.py

    Removed trailing whitespaces as suggested by lint.
    SuryanarayanaY authored Aug 24, 2023
    Configuration menu
    Copy the full SHA
    9c5d415 View commit details
    Browse the repository at this point in the history