Conversation
* Fix typo Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> * Add check that compile_ast result is an IR Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> * Add return type hint to compile_node Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> --------- Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu>
* Fix: inform_daemon_exit on assertion exit Otherwise, --assert_compiler_success hangs. --assert_all_regions_parallelizable doesn't fail, but I also copied the change there. The tests using the flag pass. Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> * Fix: correctly parse "is parallelizable" output Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> --------- Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu>
* Fix double-exit bug in parallel pipelines Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> * Assert running_procs is positive Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> * Fix: don't use exit handler The reason why it worked on Ubuntu 20.04 was that the EXIT handler was not called, while on Ubuntu 24.04 it correctly is. Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> * Add comment explanation Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> --------- Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu>
Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu>
Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu>
* starting to integrate bash into pash * [untested] add bash expansion for bash parser * outline for ast_to_ast updates * ast_to_ast updates complete but untested * some testing on ast_to_ast changes * outline for ast_to_ir * some work on ast_to_ir * fixed bash mode not propogating to JIT engine * update requirements for actions tests * minor bug fixes and new bash testing script * make changes to test scripts to support bash * some work on the bash testing * Add bash tests * start work on benchmarking * updates to the bash benchmark routine * bug fixed with echo ast, crazy bash speed ups now * getting ready to run evaluations * bug fix to attain more speedups * add another benchmark script * nlp test suite * bash evaluation script * Fix: python 3.8 compat Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> * Split up big function Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> * Fix tmpdir in tests Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> * Add 'encoding="utf-8"' to open calls for writing The bash tests contain scripts which use UTF-8 only characters, but, by default, Python throws an exception when writing non-ASCII characters to a file. Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> * Lint: black bash changes Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> * Add comments Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> * Update shasta version Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> * Update libbash version Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> * Fix version specifier in requirements * Rerun CI * Run bash and dash tests together Also changes it so that only the bash tests only run in bash mode, which I feel is fair since they test bash only features * Fix typo * Fix accidential merge removal * Add bash server expansion from sh_expand * Add bash evaluation tests to skip * Minor test fix * Remove failing bash evaluation tests Due to the prescence of a '*' character in the sed/grep regex string which is correctly interpreted as a glob conservatively. * Remove test that fails only in CI * Add bash docs for the new `--bash` flag Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> * doc edits Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> * Add bash intro test Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> * Update annotations version * Fix syntax warning --------- Signed-off-by: Bolun Thompson <bolunthompson@ucla.edu> Co-authored-by: sethsabar <sethsabar@gmail.com>
* Improve local annotations setup and dependency handling - Modified `pa.sh` for compatibility with local annotations - Added `local-annotations-library-documentation.md` with setup instructions - Introduced `local-or-pypi.sh` to configure package installation mode for local annotations Signed-off-by: Divik Chotani divikchotani@g.ucla.edu Signed-off-by: Divik Chotani <divikchotani@g.ucla.edu> * Improve local annotations setup and dependency handling - Modified `pa.sh` for compatibility with local annotations - Added `local-annotations-library-documentation.md` with setup instructions - Introduced `setup-local.sh` to configure package installation mode for local annotations Signed-off-by: Divik Chotani <divik.chotani@gmail.com> * Update local-annotations-library-documentation.md minor change to readme * add --local-annotations-dir support for dynamic annotation loading - Implemented `--local-annotations-dir` to allow dynamic switching of annotation. - Updated `pash_init_setup.sh` to adjust `PYTHONPATH` based on user input. - Allowed `cli.py` to recognize and prioritize local annotations when provided. - Ensured clean handling of `requirements.txt`, avoiding unnecessary modifications. Signed-off-by: Divik Chotani divikchotani@g.ucla.edu Signed-off-by: Divik Chotani <divik.chotani@gmail.com> * feat: add --local-annotations-dir support for dynamic annotation loading - Resolved merge conflicts between remote and local Signed-off-by: Divik Chotani divikchotani@g.ucla.edu Signed-off-by: Divik Chotani <divik.chotani@gmail.com> * - Refactored documentation for the local annotations library - Added guidelines for adding new commands - Updated the flag to print commands on separate lines - Initiated the CI script development - Cleaned up the code, all the code regarding the --local annotations-dir flag is now in pash_init_setup.sh Signed-off-by: Divik Chotani <divik.chotani@gmail.com> * Made check to see if at least one optimization was made, useful for the ci check * Completed CI script, when called it will pull the annotations repo and run a test script hello-world.sh Signed-off-by: Divik Chotani <divik.chotani@gmail.com> * everything but the rework of the githib workflow is done, will write a comprehensive message in that commit Signed-off-by: Divik Chotani <divik.chotani@gmail.com> * made one minor change to the pash_init_setup.sh, as well as testing the annotations workflow script Signed-off-by: Divik Chotani <divik.chotani@gmail.com> * Consolidated CI/CD test changes into one commit * Finished the final requested changes from the PR, as well completed the CI script, it uses the --assert_all_regions_parallelizable flag's return value to check wether or not the test succeeded Signed-off-by: Divik Chotani <divik.chotani@gmail.com> * Finished the final requested changes from the PR, as well completed the CI script, it uses the --assert_all_regions_parallelizable flag's return value to check wether or not the test succeeded Signed-off-by: Divik Chotani <divik.chotani@gmail.com> * Finished the final requested changes from the PR, as well completed the CI script, it uses the --assert_all_regions_parallelizable flag's return value to check wether or not the test succeeded Signed-off-by: Divik Chotani <divik.chotani@gmail.com> * Improved ci script to fix bugs with the return value, modified variable names to be more in line with the rest of the codebase, cleared up confusing documentation wording Signed-off-by: Divik Chotani <divik.chotani@gmail.com> * made the final nits, moved the annotation flag next to all the other checking flags and replaced the cp and rm commands in line 67 of scripts/ci/test_local_annotations.sh with mv Signed-off-by: Divik Chotani <divik.chotani@gmail.com> --------- Signed-off-by: Divik Chotani divikchotani@g.ucla.edu Signed-off-by: Divik Chotani <divikchotani@g.ucla.edu> Signed-off-by: Divik Chotani <divik.chotani@gmail.com> Co-authored-by: Konstantinos Kallas <konstantinos.kallas@hotmail.com> Co-authored-by: Divik Chotani <divikchotani@g.ucla.edu>
* fix: add libtool dep for all distros Signed-off-by: Victor Canard-Duchêne <vcanard@Victors-MacBook-Air.local> * nit: put shared lib in pkgs var instead of sep distro vars Signed-off-by: Victor Canard-Duchêne <vcanard@Victors-MacBook-Air.local> --------- Signed-off-by: Victor Canard-Duchêne <vcanard@Victors-MacBook-Air.local>
* Update workflows * Update installation with venv * github workflows
#763) * Change preprocessor * Checkpoint * graphviz * inline cli * cut some unneccessary args * cleanup * cleanup preprocessor * cleanup * fix a bug * shellcheck ignore * Cleanup * Fix argument parsing * Fix preprocessor * Refactor argument parsing
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
Uh oh!
There was an error while loading. Please reload this page.