pnnl / chgl

Chapel HyperGraph Library (CHGL) - HPC-class Hypergraphs in Chapel
https://pnnl.github.io/chgl/
MIT License
29 stars 8 forks source link

For DNS Test, should output to individual files #41

Closed LouisJenkinsCS closed 4 years ago

LouisJenkinsCS commented 5 years ago

Can concatenate back into a single file if needed later, but having individual file for each stage and components can be very helpful.

mandysack commented 5 years ago

Hi @LouisJenkinsCS, I am not sure if you are looking at this, but this is something I am working on in the unit/test.chpl file.

If you have any particular requests please let me know.

LouisJenkinsCS commented 5 years ago

Cool! I'd welcome the contribution! I actually planned to overhaul the hypergraph's segmentation to work in distributed/multi-locale contexts, but I won't be touching the DNS test just yet.

LouisJenkinsCS commented 5 years ago

Also be aware of the fact that the entire project structure has been changed so you may want to rebase/pull sooner rather than later.

LouisJenkinsCS commented 5 years ago

@mandysack By the way, kind of off-topic, but since I'll be working full time on this project this summer (starting June 1st to August 27th), let me know if you have any requests towards how I can improve CHGL to make DNS processing easier, or if I can make a better DNS processor for you (I.E make it interactive).

LouisJenkinsCS commented 5 years ago

@mandysack I'm beginning on this and almost finished with it (should be rolled out today). Just to let you know.

LouisJenkinsCS commented 5 years ago

Changed from test/test.chpl to example/activeDNS.chpl. Right now it has a nasty stack-overflow issue in components, getting on that soon!

LouisJenkinsCS commented 5 years ago

Currently untested as right now there is a stack-overflow being produced in connected components (being addressed in https://github.com/pnnl/chgl/tree/components-overhaul branch), but it should now produce its components into a single file. Will close after confirming.

github-actions[bot] commented 4 years ago

This issue is stale and should either be closed or eventually resolved.