Skip to content

bumped up to python3.12 & tf 2.16 (some tf code changed were needed) - #44

Open
nethemis wants to merge 2 commits into
mainfrom
Neofytos-dev
Open

bumped up to python3.12 & tf 2.16 (some tf code changed were needed)#44
nethemis wants to merge 2 commits into
mainfrom
Neofytos-dev

Conversation

@nethemis

Copy link
Copy Markdown

Bumping up to newer python/TF versions. Should help run the model marginally faster on latest MacOS CPUs/GPUs.

Summary of changes:

  • bumped up the library versions
  • replaced direct Keras calls with indirect ones through TF.
  • replaced K.mean() with tf.reduce_mean() which is the suggested way.

Validation steps taken:

  • NN training results stay the same
  • Unit tests pass

@nethemis

Copy link
Copy Markdown
Author

Hi @MichaelVDeSanti, @moselhy,

The CI checks are failing for this PR of mine. I do not have access to the logs to understand the reason why, since it requires University access. Could either of you kindly have a look?

Neofytos

@nethemis nethemis self-assigned this Sep 28, 2025
@nethemis nethemis added good first issue Good for newcomers dependencies Pull requests that update a dependency file labels Sep 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file good first issue Good for newcomers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant