-
Notifications
You must be signed in to change notification settings - Fork 27
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
Use core 2.26.0 #2976
Use core 2.26.0 #2976
Conversation
126c202
to
bb205bf
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2976 +/- ##
==========================================
+ Coverage 89.84% 89.99% +0.14%
==========================================
Files 39 39
Lines 4057 4057
==========================================
+ Hits 3645 3651 +6
+ Misses 412 406 -6
Flags with carried forward coverage won't be shown. Click here to find out more.
|
@@ -343,7 +343,9 @@ def run(self): | |||
"scipy", | |||
# Note: the somacore version is in .pre-commit-config.yaml too | |||
"somacore==1.0.14", | |||
"tiledb~=0.31.0", | |||
# TEMP WHILE WE AWAIT WHEELS |
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.
This is temporary while @dudoslav resolves a disk-space issue at PyPI
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.
BTW we only use tiledb-py for unit tests now
@nguyenv @eddelbuettel this has a temporary lack-of-pin on tiledb-py -- which is currently ok since we only use typedb-py for unit-testing now -- and there are no storage-format changes in core 2.26. Let's go ahead & move forward with this for this morning, and I'll update the tiledb-py pin once pypi/support#4744 after which point @dudoslav will be able to publish all PyPI artifacts for tiledb-py 0.32.0. |
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.
If it has to be two-step because of some pypi woes so be it
[sc-53002]
Uses:
Following our established procedure
The core-arifact URLs were updated as always following our procedure via the one-line, error-proof