-
Notifications
You must be signed in to change notification settings - Fork 5
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
Document 3 ways of Lasio-to-Wellio via Jupyter #48
Document 3 ways of Lasio-to-Wellio via Jupyter #48
Conversation
This change demostrates 3 ways to reading from Lasio generated LAS/JSON to Wellio.js/Wellioviz.js within a Jupyter notebook.
This looks awesome. I had some problems but I think they were due to trying this on computer that initially didn't have wellio and wellioviz installed via npm. In the notebook, I think we need to add cells for |
It also looks like I have a problem in latest version of wellioviz that I need to fix with an extra } before it will work here. I'll give that a look. |
pixiedust is a little frustrating with the inability to re-associated variables with a name once used once and needing to restart the kernal. Maybe we should note that in the notebook explicitly. |
@kinverarity1 do you have any thoughts on this from the lasio perspective? |
It all looks very impressive! I would stick to option 1 or 2. The main purpose of lasio's JSON output is serialization and interaction with things like wellio so use that if it suffices? |
Commit ac18d9a adds steps to install Wellio.js and Wellioviz.js only if they are not installed. Commit 7039546 adds a note at the top to restart the Jupyter Kernal when needing to re-init pixiedust_node variables. Let me know if this needs some additional changes/improvements to reach approval for merging. Thank you, |
When I try to run it I'm getting errors for anything wellioviz:
|
I fixed the problem in wellioviz with the try catch statement forgetting the err part of catch(err). This is not fixed in version 0.0.22. |
However, when wellioviz handles multiple logs at the same time, it uses well name to tell them apart. This is a problem as It doesn't in this example notebook, https://github.com/kinverarity1/lasio/blob/master/notebooks/exporting%20a%20LAS%20file%20to%20JSON.ipynb but it seems to in the current version of LASIO? For instance, if I work with this example las file that has well name information: https://github.com/JustinGOSSES/wellio.js/blob/master/assets/00-01-01-073-05W5-0.LAS and I load it like: In the output string of JSON, the metadata fields are all blank even though the original LAS file had data in most of them: The code in question in lasio is probably in https://github.com/kinverarity1/lasio/blob/master/lasio/las.py on lines 901-908. |
…ed to push copies of notebooks, sorry for mess
@JustinGOSSES what version of lasio is being imported? I hope this is an old version. I remember us fixing a bug at some point where the output of "print" was to show |
Yep, I was using 0.24.1 which is a bit out of date. I remembered to check javascript libraries for being to date and forgot lasio! |
Let's merge this and then keep working on it to get all the way to wellioviz visualization. I'm unsure whethe pixiedust is the best way to get there or some modification of |
This change demonstrates 3 ways to reading from Lasio generated LAS/JSON
to Wellio.js/Wellioviz.js within a Jupyter notebook.
I included a markdown version of the ipynb file. It is helpful set 'display the rich diff' setting in-order to read the markdown file easily.
After generating a LAS structure in Lasio, output the structure to a LAS 2.0 file with the following code example, then one of the subsequent 3 methods can be used in Jupyter to successfully read the structure into wellio.js format.
Create LAS structure, and set LAS version to 2.0 by writing out 'scratch_v2.las'
Lasio LAS access/translation methods
Summary
These have been tested on a local jupiter instance. They don't require any changes to Wellio.js to work. So for issue #40, this would resolve the issue just by implementing one of these methods in Jupyter notebooks. Could you check these methods work for you. If they do should we close issue 40 ? (We could merge this branch in-order to have these examples available and documented).
Thank you,
DC