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

api.py for pytables #506

Merged
merged 10 commits into from
Aug 7, 2024
Merged

api.py for pytables #506

merged 10 commits into from
Aug 7, 2024

Conversation

HajimeKawahara
Copy link
Owner

if choosing pytables as radis engine, columns was empty because vaex is lazy I/O. So I fixed it and now the user can manually set the radis engine in Mdb(Exomol, Hitemp, Hitran).

@HajimeKawahara HajimeKawahara added bug Something isn't working common API compatibility radis common API compatibility issue labels Aug 7, 2024
@HajimeKawahara HajimeKawahara self-assigned this Aug 7, 2024
@HajimeKawahara HajimeKawahara added this to the v1.6 milestone Aug 7, 2024
@HajimeKawahara
Copy link
Owner Author

Check the integration test: exojax/tests/integration/api

Copy link

github-actions bot commented Aug 7, 2024

Unit Test Results

157 tests  ±0   157 ✔️ ±0   32s ⏱️ -1s
    1 suites ±0       0 💤 ±0 
    1 files   ±0       0 ±0 

Results for commit 27f1cb4. ± Comparison against base commit 0f25954.

♻️ This comment has been updated with latest results.

@HajimeKawahara
Copy link
Owner Author

@HajimeKawahara HajimeKawahara merged commit e9bf6f1 into develop Aug 7, 2024
3 checks passed
@HajimeKawahara
Copy link
Owner Author

#502

@HajimeKawahara HajimeKawahara deleted the apitables branch August 7, 2024 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working common API compatibility radis common API compatibility issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant