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

Add field-container length method #233

Merged
merged 14 commits into from
Aug 6, 2022
Merged

Add field-container length method #233

merged 14 commits into from
Aug 6, 2022

Conversation

adtzlr
Copy link
Owner

@adtzlr adtzlr commented Aug 6, 2022

and also allow fields to be updated without splitting at offsets.

This fixes #231 and fixes #232.

@adtzlr adtzlr added documentation Improvements or additions to documentation enhancement New feature or request labels Aug 6, 2022
@adtzlr adtzlr added this to the 4.0.0 milestone Aug 6, 2022
@adtzlr adtzlr self-assigned this Aug 6, 2022
@codecov-commenter
Copy link

codecov-commenter commented Aug 6, 2022

Codecov Report

Merging #233 (1e053c2) into main (92c01ec) will increase coverage by 0.01%.
The diff coverage is 100.00%.

❗ Current head 1e053c2 differs from pull request most recent head 7b396fa. Consider uploading reports for the commit 7b396fa to get more accurate results

@@            Coverage Diff             @@
##             main     #233      +/-   ##
==========================================
+ Coverage   96.59%   96.61%   +0.01%     
==========================================
  Files          66       66              
  Lines        2760     2774      +14     
==========================================
+ Hits         2666     2680      +14     
  Misses         94       94              
Impacted Files Coverage Δ
felupe/_field/_container.py 100.00% <100.00%> (ø)
felupe/tools/_newton.py 98.93% <100.00%> (-0.05%) ⬇️

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

@adtzlr adtzlr merged commit 0dea8b9 into main Aug 6, 2022
@adtzlr adtzlr deleted the add-fieldcontainer-length branch August 6, 2022 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow result of sparse-solver to be directly added to the field Add len(field) method
2 participants