-
Notifications
You must be signed in to change notification settings - Fork 18
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
max_cache human writing #334
Conversation
```max_cache``` in bits is fine, but not very human friendly. This is a downstream implementation of the YAXArraysToolbox package to get ```max_cache``` as e.g. ```max_cache="100MB"``` that can be useful for users of the YAXArrays package.
Update DAT.jl
Codecov ReportAttention:
📢 Thoughts on this report? Let us know!. |
Now this feature is non-breaking previous code.
Thanks a lot for the PR. Something seems to be strange since I can see a series of commits, but it looks like there are no changes to the files. Don't know if the error is on my side @felixcremer do you see proposed changes? |
ca40fa8
to
01123f2
Compare
Thanks Fabian, you're right, somehow I reverted the changes on the branch. Now you should be able to see the changes. |
Pull Request Test Coverage Report for Build 6301293873
💛 - Coveralls |
@dpabon please add some tests, if you can. |
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.
Thanks a lot
max_cache
in bits is fine, but not very human friendly. This is a downstream implementation of the YAXArraysToolbox package to getmax_cache
as e.g.max_cache="100MB"
that can be useful for users of the YAXArrays package.