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

Fix warnings when using record-backward-compat #1977

Merged
merged 1 commit into from
Jul 8, 2024

Conversation

laggui
Copy link
Member

@laggui laggui commented Jul 5, 2024

Fix some small omissions in the tensor data refactor #1916 with the record-backward-compat feature flag

  • Unused import
  • Allow deprecated use of DataSerialize
  • Leftover generic on dimension

@laggui laggui requested a review from nathanielsimard July 5, 2024 12:16
@laggui
Copy link
Member Author

laggui commented Jul 5, 2024

Looks like a typo was introduced in #1966 (CI didn't run cuz it only affected markdown files)..

Seems to have been fixed in other PRs so I'll leave it as is.

@laggui laggui force-pushed the fix/data/backward-compat branch from 82938e4 to 95d060b Compare July 5, 2024 14:15
Copy link

codecov bot commented Jul 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.29%. Comparing base (1ad2a63) to head (95d060b).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1977   +/-   ##
=======================================
  Coverage   85.29%   85.29%           
=======================================
  Files         798      798           
  Lines       95512    95512           
=======================================
  Hits        81471    81471           
  Misses      14041    14041           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@laggui laggui merged commit 6f158af into main Jul 8, 2024
14 checks passed
@laggui laggui deleted the fix/data/backward-compat branch July 8, 2024 11:58
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