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

Subsets of control networks are not written out correctly. #111

Open
jessemapel opened this issue May 29, 2019 · 0 comments
Open

Subsets of control networks are not written out correctly. #111

jessemapel opened this issue May 29, 2019 · 0 comments

Comments

@jessemapel
Copy link
Contributor

Split from #108 and @astrostu

Writing out a reduced control net (just a few of the columns from a full, JIGSAW'd control net, for just a few (11) measures) has revealed more issues. It does not respect "referenceIndex," instead making the first measure the reference. Any True/False field that has any text other than "True" gets treated as "True" -- such as "ignore" set to "False" will still have the measure's Ignore set to "True" ... I have to make the field "" to get it to be "False." I got errors whenever trying to put in any of the first four apriori columns (I didn't try the others). It did not respect measure type nor point type. Then, I thought it might be a version issue, so then I explicitly set "version=5" when calling io_controlnetwork.to_isis(), and I got errors. I had to modify line 287 of io_controlnetwork.py to be "point_spec = cnp5.ControlPointFileEntryV0005()" and then I at least got a network written out, but then CNETBIN2PVL couldn't read it ("ERROR Unable to write control net to PVL file. Invalid control point has no point ID value."). Despite it having a point ID and at least being able to convert to PVL when using version=2. If I added a prefix when calling .to_isis, like "blah", I got a different error when trying to run CNETBIN2PVL on the output network: "I/O ERROR Reading the control network [test_out.net] failed.\nI/O ERROR Failed to convert protobuf version 2 control point at index [1] into a ControlPoint.\nPROGRAMMER ERROR The SerialNumber is not unique. A measure with serial number [MRO/CTX/0848006063:221] already exists for ControlPoint [blahMC18_01_spacing1x_ptA00000015]." (Again, none of those errors were there with v2.)

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

1 participant