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

Use MorphIO as IO layer #16

Open
wizmer opened this issue May 29, 2019 · 2 comments · May be fixed by #71
Open

Use MorphIO as IO layer #16

wizmer opened this issue May 29, 2019 · 2 comments · May be fixed by #71

Comments

@wizmer
Copy link
Contributor

wizmer commented May 29, 2019

No description provided.

@wizmer
Copy link
Contributor Author

wizmer commented May 29, 2019

@adrien-berchet
Copy link
Member

A first implementation was added in 2a86f62f6d4159b2524c2f753bdffb9f866ad7af, and has since been a bit improved.

Now the plan is the following:

  1. use MorphIO by default and use the internal loader when MorphIO fails (and emit a warning in this case).
  2. if possible, improve MorphIO as described in Add a flag to disable some checks MorphIO#427 in order to reduce the probability of using the internal loader.

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 a pull request may close this issue.

2 participants