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

Support of TensorFlow 2.13 #76

Merged
merged 4 commits into from
Aug 21, 2023
Merged

Support of TensorFlow 2.13 #76

merged 4 commits into from
Aug 21, 2023

Commits on Jul 26, 2023

  1. Configuration menu
    Copy the full SHA
    b01dfd0 View commit details
    Browse the repository at this point in the history

Commits on Aug 21, 2023

  1. fix: support TF 2.13 for conv_utils module

    conv_utils module has been moved again in TF 2.13, and can be imported as in
    TF <= 2.5. The error raised is ImportError, which was added to the try/except
    clause. ModuleNotFoundError is a subclass of ImportError; it is then removed.
    danibene authored and cofri committed Aug 21, 2023
    Configuration menu
    Copy the full SHA
    ec6714b View commit details
    Browse the repository at this point in the history
  2. fix(test): deserialization of h5 models w/ custom obj doesn't work in…

    … TF2.13
    
    It seems that (de)serialization of h5 models in TF 2.13 has been changed:
    custom objects, even with "register_keras_serializable", cannot be loaded.
    Two solutions are possible:
    - use "with_custom_object_scope()" to load a model with custom objects.
    - save model in Keras format and not h5.
    
    The second option was chosen because this format is now preferred for saving
    models.
    cofri committed Aug 21, 2023
    Configuration menu
    Copy the full SHA
    e743035 View commit details
    Browse the repository at this point in the history
  3. fix(test): random test was failing. Fix better seed.

    The power iteration test for Dense was sometimes failing due to a random
    choice of kernel and initialization vector u. Even if np.random.seed(42)
    was set at the beginning of the file.
    
    To ensure a fully deterministic test (deterministic kernel and
    initialization u), a random generator np.random.default_rng(42) was created
    and used to create both kernel and u.
    The results for Dense test are then deterministic, either for a single test
    file or for the whole unittest. Note that this is the only test modified;
    other tests of the file pass, they have not been changed with deterministic
    initializations.
    cofri committed Aug 21, 2023
    Configuration menu
    Copy the full SHA
    1f91e7f View commit details
    Browse the repository at this point in the history