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.

Sign up for GitHub

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

Jump to bottom

Refactoring/remove is empty batch #1144

Merged
merged 4 commits into from
Jul 18, 2024
Merged

Refactoring/remove is empty batch #1144

merged 4 commits into from
Jul 18, 2024

Commits on Jul 18, 2024

  1. Remove Batch.is_empty 

     * Keeps the API lean
 * The same functionality can be achieved using len() on dicts
    @dantp-ai @MischaPanch
    dantp-ai authored and MischaPanch committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    9a083a5 View commit details
    Browse the repository at this point in the history

  2. Minor refactor syntax in tutorials

    @dantp-ai @MischaPanch
    dantp-ai authored and MischaPanch committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    d1e2b33 View commit details
    Browse the repository at this point in the history

  3. Forgot to remove use of is_empty

    @dantp-ai @MischaPanch
    dantp-ai authored and MischaPanch committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    330f5cc View commit details
    Browse the repository at this point in the history

  4. Update Changelog

    @dantp-ai @MischaPanch
    dantp-ai authored and MischaPanch committed Jul 18, 2024
    Configuration menu
    Copy the full SHA
    c5dccf3 View commit details
    Browse the repository at this point in the history