-
Notifications
You must be signed in to change notification settings - Fork 6
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
Checkpointing option for GAP-C programs #163
Merged
Merged
Changes from 83 commits
Commits
Show all changes
84 commits
Select commit
Hold shift + click to select a range
b1faef7
added --checkpoint optional parameter; adjust Makefile generator if o…
fymue 13fd005
automatically add required includes, archive_periodically method and …
fymue 87dff56
automatically generate remaining code for checkpointing; checkpointin…
fymue 9179d9e
don't need that
fymue 6f09f34
prevent activation of checkpointing for non-primitive answer types
fymue 79d15b9
fixed linter errors; added Checkpoint class to Printer namespace to m…
fymue c62e47f
adjust warning
fymue ed43e9e
add GAP-C copyright msg
fymue bb89e71
improved some error messages/warnings; auto delete tables after progr…
fymue f209412
add checkpointing options for user-compiled binaries; checkpoint rout…
fymue 4249599
cleanup checkpoint.hh indentation; add two functions for Log file wri…
fymue 79a712f
create Logfile w/ user input and table archive paths; automatically p…
fymue c6da3eb
switch from std::filesystem to boost::filesystem for MacOS compatibil…
fymue 1fcabe5
adjusted automatic datatype inference to check every non-terminal tab…
fymue bccd26c
check for boost serialization/filesystem availability; change --check…
fymue 0f22135
add updated configure script after adding boost tests; refactor some …
fymue bae8c92
fixed some newline/printing bugs
fymue 82a4ec5
add a simple test which compiles with checkpointing (no test for actu…
sjanssen2 d5ea452
check for input/output path permissions
fymue 2dbbdd9
remove <filesystem> include in generated code; forgot to change -c op…
fymue 967aea0
update Logfile with cpu time and max rss since program start at every…
fymue d6845cc
don't need to return anything here
fymue 2be8361
add tests that launch a checkpointed program, kill it after the first…
fymue cd1c8f1
missing space in print message
fymue 5079bf4
Merge pull request #165 from jlab/checkpointing_test
fymue ad34b84
accidentally removed the return statement from the wrong func in one …
fymue afbd8e8
Merge branch 'checkpointing' of github.com:jlab/gapc into checkpointing
fymue 95e3dab
explicitly set the checkpoint input path for the tests (we might not …
fymue f73a714
use KSH variable instead of using ksh directly
fymue be8c00f
shorten checkpointing tests a bit so test suite doesn't take as long
fymue 88c6326
pass paths to ifstream/ofstream as const char* & use sizeof instead o…
fymue 8656cf4
add percentage of tabulated values per table to Logfile at every chec…
fymue f6d487a
get rid of unnecessary -H option; prepare for GAPC call string insert…
fymue 89b8a70
refactor some stuff to make adding PID to Logfile more efficient
fymue a5d9bd0
add function to find the Logfile in the input path (necessary now sin…
fymue e31fb73
merge master changes (PR #167) into this branch
fymue aa8a1d9
enable checkpointing of algebra products (as long as parts of the pro…
fymue 3f48e67
fix linter errors
fymue da3e43a
fixed types
fymue d31d969
change input opt to require Logfile instead of path; user can add pre…
fymue a9d8986
specify generated Logfile as input for checkpointing tests
fymue 9242cee
Merge branch 'master' of github.com:jlab/gapc into checkpointing
fymue bc66279
merge master
fymue 0e4aab3
(de)serialize String type to enable checkpointing GAPL programs w/ e.…
fymue 1dc97f7
accidentally deleted this
fymue 39bf6f6
add some asserts for debugging; add mutexes to each table class to av…
fymue 6719aa6
mark Lists with unresolvable links as not tabulated so they get recal…
fymue 4f17789
Merge branch 'master' of github.com:jlab/gapc into checkpointing
fymue d9fd62b
clear RUN_CPP_FLAGS variable so the following tests don't falsely rec…
fymue afb30b4
forgot to delete this
fymue fdb61b9
only store content of block instead of entire block array for better …
fymue 0f38bd0
enable checkpointing of algebra (products) with Shape type
fymue 55ece9e
fixed bug/improved recursion to identify broken ListRefs; add checkpo…
fymue ae9d958
forgot the remove this (leftover stuff from 1st String checkpointing …
fymue 10a258a
enable checkpointing for algebra (products) using Rope type
fymue b0569de
add checkpointing support for programs with hash types (Hash::Ref)
fymue 88f6a69
removed some unneeded leftover code; updated some comments
fymue e671422
remove extra #if defined checks (not really needed for anything)
fymue 8962e13
refactored parse_checkpoint_log and create_checkpoint_log functions t…
fymue 91027f3
merge master changes
fymue c6b8da6
accidentally deleted checkpoint option during merge from master
fymue d0862d0
accidentally deleted checkpoint option during merge from master
fymue 88027d6
didnt mean to push this
fymue 924d14b
fix serializing in Hash::Ref/Hash Inspector classes; also archive tab…
fymue 3fb3e40
need to check for cyk here
fymue 55250db
forgot to wrap cp support check in else clause after cyk check; refac…
fymue 9ef4371
Merge branch 'master' of github.com:jlab/gapc into checkpointing
fymue 907a2ff
fix (de)serializing bug for Shape type
fymue a99a968
only delete Logfile/tables of current test; only delete if test was s…
fymue 9c0b9f3
use cpp_base instead of hard-coding binary name
fymue 1f9a1b7
refactor checkpoint opt help msg; use marco variable in checkpoint he…
fymue cf06ad9
fix ident in user-defined type generation; refactored some stuff; add…
fymue 2140f83
fix str concat bug in cp help msg; fix/add a bunch of comments; refac…
fymue 15eab3a
refactor some stuff/add some comments in Checkpoint class; print spec…
fymue a9ae98a
make checkpointing compatible for classified/interleaved products; pr…
fymue f56cde9
fix variables in test func; fix separation of output path and user-de…
fymue d8c5cb8
sort command line args alphabetically before creating arg string so c…
fymue 8014617
fix check for user file permissions; allow serialization of char tabl…
fymue c7c3755
make array_size member a local variable in load method; add new check…
fymue bd2511d
change gapc-test-suite branch
fymue e7d0c23
add --keepArchives option to prevent deletion of archives at program …
fymue 6fd4a91
forgot to change gapc-test-suite branch for MacOS workflow; refactor …
fymue 51b126d
Merge branch 'master' of github.com:jlab/gapc into checkpointing
fymue 0fcc99e
print filled % for every table in info msg
fymue File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't forget to change back once merged