Skip to content
This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

Support SEGY files that do not have geometry - consistent support of segy files #87

Open
MJZawacki opened this issue Dec 5, 2019 · 1 comment · Fixed by #199
Open
Labels
Prior: Medium Type: Feature New feature or request

Comments

@MJZawacki
Copy link
Collaborator

The current code uses segyio.tools.cube() to extract a volume of data from segy files that can easily be chuncked up. However, most segy files do not have geometry in them that allows segyio to infer the dimensions of the volume. In the case where geometry is not present. You need to manually infer the dimensions by reading through the trace headers and constructing a volume manually.

@maxkazmsft maxkazmsft added this to the second OSS release milestone Dec 18, 2019
@maxkazmsft
Copy link
Contributor

The current code uses segyio.tools.cube() to extract a volume of data from segy files that can easily be chuncked up. However, most segy files do not have geometry in them that allows segyio to infer the dimensions of the volume. In the case where geometry is not present. You need to manually infer the dimensions by reading through the trace headers and constructing a volume manually.

Thanks! Could you please provide a reference to where you see this in our code?

@maxkazmsft maxkazmsft removed this from the Distributed training with AML milestone Jan 7, 2020
@maxkazmsft maxkazmsft added this to New in sprint triage Jan 24, 2020
This was referenced Feb 10, 2020
@MJZawacki MJZawacki linked a pull request Feb 11, 2020 that will close this issue
@maxkazmsft maxkazmsft changed the title Support SEGY files that do not have geometry Support SEGY files that do not have geometry - consistent support of segy files Mar 6, 2020
@maxkazmsft maxkazmsft added Prior: High Type: Feature New feature or request labels Mar 6, 2020
@maxkazmsft maxkazmsft removed this from New in sprint triage Mar 6, 2020
@ashbhati ashbhati added this to Mn: Backlog in Manganese Mar 16, 2020
@maxkazmsft maxkazmsft removed this from Mn: Backlog in Manganese Apr 3, 2020
@maxkazmsft maxkazmsft modified the milestones: V0.1.3, V0.1.4 [BYOD] Jun 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Prior: Medium Type: Feature New feature or request
Projects
None yet
3 participants