Skip to content

Conversation

@lmontier
Copy link
Collaborator

No description provided.

@lmontier lmontier force-pushed the update-tf-to-2.9.1 branch from 66472f6 to 995f6d9 Compare August 11, 2022 09:19
@lmontier lmontier force-pushed the update-tf-to-2.9.1 branch from 07e54ac to 59a97b1 Compare August 11, 2022 09:39
@lmontier lmontier force-pushed the update-tf-to-2.9.1 branch from 30a54f2 to a543412 Compare August 11, 2022 14:58
@codecov
Copy link

codecov bot commented Aug 11, 2022

Codecov Report

Merging #48 (e5a4739) into master (0f176f8) will increase coverage by 0.03%.
The diff coverage is 97.43%.

❗ Current head e5a4739 differs from pull request most recent head a543412. Consider uploading reports for the commit a543412 to get more accurate results

@@            Coverage Diff             @@
##           master      #48      +/-   ##
==========================================
+ Coverage   66.59%   66.63%   +0.03%     
==========================================
  Files          44       44              
  Lines         982      983       +1     
==========================================
+ Hits          654      655       +1     
  Misses        328      328              
Impacted Files Coverage Δ
keras_fsl/callbacks/binary_statistics.py 0.00% <0.00%> (ø)
keras_fsl/layers/centroids_matrix.py 100.00% <100.00%> (ø)
keras_fsl/layers/classification.py 31.57% <100.00%> (ø)
keras_fsl/layers/slicing.py 75.00% <100.00%> (ø)
keras_fsl/layers/support_layer.py 47.05% <100.00%> (ø)
keras_fsl/losses/gram_matrix_losses.py 100.00% <100.00%> (ø)
keras_fsl/metrics/gram_matrix_metrics.py 56.00% <100.00%> (ø)
keras_fsl/models/encoders/basic_cnn.py 42.85% <100.00%> (ø)
keras_fsl/models/encoders/darknet.py 27.27% <100.00%> (ø)
keras_fsl/models/encoders/koch_net.py 31.57% <100.00%> (ø)
... and 7 more

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

@lmontier lmontier marked this pull request as draft August 12, 2022 07:19
@ClementWalter
Copy link
Owner

hey, should I merge this?

@ClementWalter ClementWalter marked this pull request as ready for review October 18, 2022 06:30
@ClementWalter
Copy link
Owner

what was the point of replacing all tf.keras to keras?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants