Welly : AttributeError: 'NoneType' object has no attribute 'position' about subsurface HOT 14 CLOSED

@Japhiolite , no update yet.

from subsurface.

Hi @neha845

the created well object w seems to be empty in your example (that's why it's NoneType).
What is your version of welly?
It may also be, that subsurface has to be updated in that regard, as position is now moved to location.position in welly, I think.

from subsurface.

Thanks for the reply @Japhiolite
Currently, I am using welly=0.5.2

But have tried with welly's previous versions also, same or new errors are coming.

from subsurface.

This may indeed be a bug in subsurface.
I'll have a look into it.

from subsurface.

This may indeed be a bug in subsurface. I'll have a look into it.

Sure

from subsurface.

Hey @Japhiolite
Is there any update in the issue?
Kindly help.

from subsurface.

Hi @neha845

I implemented a change in PR #58 .

However, from your screenshot, I noticed that your welly object seems to be empty (not initialized correctly?), as it shows Nonetype.
could you maybe provide the collar, survey, lith dataframes? Then I could check.

from subsurface.

Sure, pls see the screenshots attached.
Img 1

Img 2

Img 3

Img 4

Img 5

Img 6

Data used is same as provided in the T21 tutorial - https://raw.githubusercontent.com/softwareunderground/subsurface/main/tests/data/borehole/kim_ready.csv

from subsurface.

Hi, @Japhiolite
Pls see.

from subsurface.

Hi @neha845

Ah, didn't know it was the T21 tutorial. I've tested the tutorial notebook, and indeed it raises the error with welly version 0.5.2.

With version 0.5.0, the welly team reworked a good chunk of their code, so there may be some substantial changes to be made to subsurface.

If you downgrade welly to version 0.4.8, the T21 tutorial should work. At least it worked for me, when welly was downgraded to a version < 0.5. Hope that helps.

from subsurface.

Hi @Japhiolite
Thank you for your reply.
I tried with welly = 0.4.9, 0.4.8 and 0.4.6

But got this error in all the 3 cases

from subsurface.

I remember this error. from_dict_advanced was a method developed by @Leguark in Striplog. But that method did not make it in the striplog pip-release. It is referenced in his fork from striplog: https://github.com/Leguark/striplog

For making the Transform 2021 notebook work, you'd have to use the striplog version from Leguark. Best would be to create a new pyhton-environment (e.g. cloning your current one) and there use pip install git+https://github.com/Leguark/striplog.git.
Then you would use the changed striplog version, which is, however, outdated compared to the main by Agile*.

from subsurface.

Hi @Japhiolite
It worked like magic.
Thank you for your help. :)

Now, going to try with custom dataset.

from subsurface.

Great to hear :)

from subsurface.