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

Drilling Advancement #38

Open
VrMahmoudabadi opened this issue Nov 11, 2022 · 5 comments
Open

Drilling Advancement #38

VrMahmoudabadi opened this issue Nov 11, 2022 · 5 comments

Comments

@VrMahmoudabadi
Copy link

There are 2 issues with the Advancement element in the schema:

1- There is no place to map the "Readings set reference [AGS Standard field]" in the element, unless it goes to gml:description that I don't believe it is a right place to use. It seems additional field needs to be created for this data.

2- What is the difference between "AdvancementRate" and "PenetrationRate"? Both have very similar description and can be used for one purpose which is the penetration rate.

@VrMahmoudabadi
Copy link
Author

Regarding the "energyDensity":

1- what is the physical meaning of this property in the element?
2- How does this apply to MWD?
3- The uom is not defined for this property in version 2.5.a.

@dponti
Copy link
Collaborator

dponti commented Jan 9, 2023

In 2.6-dev, drillAdvancement property in Borehole is now of ArrayProperty type that has a setName attribute, meaning that you can list a series of Advancement objects under one drillAdvancement property tag and assign a name to the set of advancement readings. This covers the use case were intervals of the hole may be redrilled (the use case that AGS uses reading set reference).

@dponti
Copy link
Collaborator

dponti commented Jan 9, 2023

PenetrationRate is deprecated in 2.6-dev. Use AdvancementRate instead. This duplication was not caught in 2.5.a

@dponti
Copy link
Collaborator

dponti commented Jan 9, 2023

Energy density is (I believe) an inappropriate term - it should be specificEnergy or better mechanicalSpecificEnergy. This explains why the UOM is incorrect. in 2.6-dev, mechanical enSpecificEnergy property is added and energyDensity is deprecated.

With respect to MWD, the Advancement object is used to summarize drill advancement information over an interval of teh hole. If real-time MWD information is recorded, this is best handled through a different object that can allow compact encoding of MWS data as a time series.

@sdeaton75
Copy link
Collaborator

I agree that for MWD data, we really need a more compact format - like the CPT data to properly encode this from automated data acquisition otherwise the file will be massive. I would suggest we add this in 2.6-dev. I will compile a list of properties and run it by one of the MWD drill rig manufacturers and get their feedback.

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

No branches or pull requests

3 participants