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

Msidev #511

Merged
merged 4 commits into from
Jan 12, 2023
Merged

Msidev #511

merged 4 commits into from
Jan 12, 2023

Conversation

antonhibl
Copy link
Contributor

Msi Driver in a working state along with the development notebook, the change in the appveyor file is related to python3.11 build systems, it is commented and shouldn't affect the CI system.

@codecov-commenter
Copy link

Codecov Report

Merging #511 (aabe725) into master (8840637) will decrease coverage by 0.19%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #511      +/-   ##
==========================================
- Coverage   18.26%   18.06%   -0.20%     
==========================================
  Files          49       51       +2     
  Lines        5377     5436      +59     
==========================================
  Hits          982      982              
- Misses       4395     4454      +59     
Impacted Files Coverage Δ
ale/drivers/msi_drivers.py 0.00% <0.00%> (ø)
src/Orientations.cpp 89.07% <0.00%> (-0.76%) ⬇️
ale/__init__.py 0.00% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@antonhibl antonhibl merged commit 004c45d into DOI-USGS:master Jan 12, 2023
@antonhibl antonhibl deleted the msidev branch January 12, 2023 19:14
jrcain-usgs pushed a commit to jrcain-usgs/ale that referenced this pull request May 19, 2023
* transition to new build system with pyproject.toml and setup.cfg

* updated CI

* adding Msi driver, still needs floating point value adjustment tuning.

* cleaning for PR, comment inappveyor is for future python3.11 builds
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants