ukaea/parallel-preprocessor

Request for different key name in manifest.json

Closed this issue · 7 comments

The manifest.json looks for the tag filename but I was wondering if this could be changed to stp_filename

No worries if it is better to keep it as filename, just thought I would ask

yes, it is possible. will do soon. thank you

this should likely stay as filename, what if one gave it an igs file? or a brep file?

For our loyalty tester, I added this feature, it will search a key containing "filename" so "stp_filename" is working now

I just have a manifest which references a few different file types, so I have and entry in the manifest file for stl_filename, stp_filename and a few others. Will this containing filename get confused if it picks up a few which match that criteria?

Perhaps another approach is to have a default dictionary key of filename but allow the user to overwrite this and specify the dictionary key they would like to use in the manifest file. Or perhaps this is unnecessarily complex?

@qingfengxia it would be good if you could reference the issue in the PR that you make, im trying to find any changes you've made to figure out the answer to @shimwell's question, but seeing as I cant find an open PR that would be associated with it, I'm at a loss. Note you can use the # symbol and then a list appears of open issues and pr's

Sorry for causing perhaps more trouble than this issues is worth.