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

PaSh version 0.11 #647

Merged
merged 78 commits into from
Feb 22, 2023
Merged

PaSh version 0.11 #647

merged 78 commits into from
Feb 22, 2023

Conversation

angelhof
Copy link
Member

@angelhof angelhof commented Feb 21, 2023

Pushes the new annotations to PaSh's main. Also includes a lot of refactoring and cleanup of old code.

festutz and others added 30 commits June 27, 2022 13:38
* WIP: first changes for new annotations

* Make parallelization use mapper info from new annotations

* Make parallelization use mapper info from new annotations

* Delete com_mapper field in dfg_node and use the info from new annotations

* Use input info from new annotations

* Use output info from new annotations

* Make dfg_options use the information from new annotations and completely remove use of old aggregator com_aggregator

* Make to_ast work for wf.sh with some hacks, e.g. to_ast for eager to handle special case of intermediate file as last operand and fixed parsing issue for newline

* Remove com_mapper and com_aggregator from DFGNode

* WIP: incorporating remodelled command invocations

* Parsing with new dataflow node model works

* WIP: parallelization

* Rudimentary parallelization with new annotations works

* Add way to specify where to find repository for annotations repository

* Do not require flag `r_split` since we do consecutive chunks for now

* 1st part of changes due to comments for PR

* minor fix

* 2nd part of changes due to comments for PR

Co-authored-by: Felix Stutz <fstutz@mpi-sws.org>
* Add support for eager-nodes

Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>

* Remove eager from TODO-list

Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>

* Addressed comments from PR

Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
* Add reduce trees

Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>

* Remove task in TODO

Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>

* Remove log statements

Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
…mutative commands (#591)

Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
…ptions for input and output directly (#595)

Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
* Refactor parallelizing transformations in separate choose and apply phases

Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
* Fuse cat and subsequent split

Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
* Add a proper installation of the annotation lib

Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>

* Remove unnecessary sys.path

Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>

* Fix bug in setup

Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>

* fix setup script

Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>
Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>
Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
…612)

* Fix bug in parser to switch from flag to operand mode when reading hyphen

Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>

* Adapt shortest_scripts.sh to work with parser

Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>

* Cover more test cases from script_microbenchmarks

Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>

* Parallelize spell-grep as done in `future`, i.e., not RR but CC for `set_diff`

Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>

* Clean up and clarifying comment in parser

Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>

* Simplify control flow in parallelization

Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>
Signed-off-by: Felix Stutz <fstutz@mpi-sws.org>
Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>
Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>
Signed-off-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com>
Integrate New Annotation Library
Delete files and references to old annotations
Remove more obsolete files in compiler, move tests to the tests directory
Refactor all Shell ast related code to its own directory (ideally being mostly self contained)
@github-actions
Copy link

OS:ubuntu-20.04
Wed Feb 22 00:07:04 UTC 2023
intro: 2/2 tests passed.
interface: 39/39 tests passed.
compiler: 54/54 tests passed.
agg: 109/109 tests passed.

@github-actions
Copy link

OS = Debian 10
CPU = Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz
Ram = 15752
Hash = 9e56d53
Kernel= Linux 4.15.0-197-generic x86_64

benchmark tests passed failed untested unresolved unsupported not_in_use other_status
posix 494 375 41 31 6 40 1 0
intro 2 2 0 0 0 0 0 0
interface 39 39 0 0 0 0 0 0
compiler 54 54 0 0 0 0 0 0
aggregator 109 109 0 0 0 0 0 0

@angelhof angelhof merged commit 1c16dad into main Feb 22, 2023
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

Successfully merging this pull request may close these issues.

None yet

2 participants