-
Notifications
You must be signed in to change notification settings - Fork 6
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
chore: update deps #247
chore: update deps #247
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #247 +/- ##
==========================================
- Coverage 82.57% 82.54% -0.04%
==========================================
Files 34 34
Lines 4242 4246 +4
Branches 4143 4147 +4
==========================================
+ Hits 3503 3505 +2
- Misses 553 554 +1
- Partials 186 187 +1 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we need a HugrView::get_dataflow_op(Node) -> Box<dyn DataflowOpTrait>
thingemy
@@ -1,6 +1,4 @@ | |||
//! Python bindings for TKET2. | |||
#![warn(missing_docs)] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this inherited now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, that's the
[lints]
workspace = true
in Cargo.toml (since rust 1.74)
num_bits += 1 | ||
} else if ty == &FLOAT64_TYPE { | ||
num_params += 1 | ||
if let Some(sig) = op.dataflow_signature() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not for this PR but we might be able to use op: & impl DataflowOpTrait
in this function instead
This throws warnings in older rust versions due to the unrecognised fields, but works alright.