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
Cleanup LogicArray interface #3792
Conversation
a1c2f6a
to
a5b36c4
Compare
a5b36c4
to
89f63fc
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3792 +/- ##
==========================================
+ Coverage 73.54% 73.61% +0.07%
==========================================
Files 49 49
Lines 7916 7941 +25
Branches 2195 2215 +20
==========================================
+ Hits 5822 5846 +24
- Misses 1581 1591 +10
+ Partials 513 504 -9 ☔ View full report in Codecov by Sentry. |
89f63fc
to
cdaaaf4
Compare
cdaaaf4
to
2e4cca5
Compare
2e4cca5
to
7e55ffd
Compare
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.
Very nice improvement to the API, thanks for all the doc and test updates as well!
* Deprecate LogicArray.integer, .signed_integer, and .binstr * Deprecate constructing LogicArray from ints. * Introduce LogicArray.from_unsigned, from_signed, to_signed, and to_unsigned to replace what was deprecated.
7e55ffd
to
e7a0242
Compare
92ea450
to
316b801
Compare
Closes #3660. Closes #3418.
LogicArray.integer
,.signed_integer
, and.binstr
LogicArray
from ints.LogicArray.from_unsigned
,.from_signed
,.to_signed
, and.to_unsigned
to replace what was deprecated.If we need other kinds of construction/conversion, we can add them later.