Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

Pin tensorflow to < 2.12.0 #104

Merged
merged 1 commit into from
Mar 24, 2023

Conversation

dstansby
Copy link
Member

See brainglobe/cellfinder#293 - tests are failing with the new version of tensorflow, so until we've sorted it probably easiest to pin the tensorflow version.

@codecov
Copy link

codecov bot commented Mar 24, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (d60fc02) 99.25% compared to head (2a487dd) 99.25%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     brainglobe/cellfinder-core#104   +/-   ##
=======================================
  Coverage   99.25%   99.25%           
=======================================
  Files           9        9           
  Lines         269      269           
=======================================
  Hits          267      267           
  Misses          2        2           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@dstansby dstansby marked this pull request as ready for review March 24, 2023 17:03
@dstansby dstansby requested a review from a team March 24, 2023 17:03
Copy link
Member

@alessandrofelder alessandrofelder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 thanks @dstansby

@alessandrofelder alessandrofelder merged commit 23e1353 into brainglobe:main Mar 24, 2023
@dstansby dstansby deleted the tensorflow-pin branch March 25, 2023 07:37
willGraham01 pushed a commit that referenced this pull request Aug 24, 2023
Drop Python 3.7 tests, add 3.9 tests
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants