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

Census tweaks #3

Open
WPettersson opened this issue Nov 13, 2016 · 2 comments
Open

Census tweaks #3

WPettersson opened this issue Nov 13, 2016 · 2 comments
Assignees
Labels

Comments

@WPettersson
Copy link
Member

WPettersson commented Nov 13, 2016

Outputs more useful information to stdout

Instead of just facepairing.str(), tricensus will now output a comma-separated list for each face pairing graph analysed. The values printed are:

index, graph, running time, count

defined as follows.

  • index is either a line number of the input pairs list, or for generated pairs index tells us when in the sequence the graph was generated
  • graph is the text representation of the pairing graph
  • running time is the running time taken to process said graph
  • count is the number of relevant graphs returned

Only parse every x'th line of input

Note: this feature is only functional if graphs are passed via stdin.

tricensus has two new parameters, -x and -y. When appropriately assigned integers, tricensus will only analyse a graph of index i if i ≅ x mod y. This makes it trivial to parallelise tricensus on clusters without having to pre-plan the graph splits. Note that this definitely won't guarantee equal running time, but any jobs that terminate early will return resources to the cluster queue.

Allow %d in output file name

If the output file name given to tricensus contains %d it will be replaced with the index of the given graph (so a new file is created for each graph). This can be used to replicate the output style used by tricensus-mpi.

@WPettersson WPettersson self-assigned this Nov 13, 2016
@WPettersson
Copy link
Member Author

I've got this in a personal branch forked from a pre-renaming master. I'll re-base it off current master once we finish re-naming and push it then, to make it easier to follow.

@WPettersson
Copy link
Member Author

Should add a flag to enable the CSV output, so as to not break existing code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant