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 auto backend #80

Merged
merged 3 commits into from
Jan 25, 2022
Merged

Add auto backend #80

merged 3 commits into from
Jan 25, 2022

Conversation

q-posev
Copy link
Member

@q-posev q-posev commented Jan 25, 2022

This PR adds TREXIO_AUTO back end, which can be used in READONLY mode ('r') to automatically determine the back end needed to open an existing TREXIO file. It does so by inspecting the return codes of trexio_text_inquire and trexio_hdf5_inquire functions. It should be updated when a new back end is implemented.

TREXIO_AUTO only works together with r mode in trexio_open, i.e. the correct call pattern is as follows:

  • trexio_open(file_name, 'r', TREXIO_AUTO, &rc_open)

@scemama
Copy link
Member

scemama commented Jan 25, 2022

Excellent!

@scemama
Copy link
Member

scemama commented Jan 25, 2022

Don't forget to update the change log ;-)

@scemama scemama merged commit 213e51f into master Jan 25, 2022
@scemama scemama deleted the add-auto-backend branch February 23, 2022 15:18
@scemama scemama restored the add-auto-backend branch February 23, 2022 15:18
@q-posev q-posev deleted the add-auto-backend branch May 18, 2022 09:33
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