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

390 fix normalized phi to normalized eo conversion in the pipeline script #396

Conversation

deemp
Copy link
Member

@deemp deemp commented Jun 5, 2024

Closes #390.

Also fixes some tests and improves a few things:

  • Add --wrap-raw-bytes to automatically convert raw bytes (and terminations) in the output. This is a temporary fix, pending the change mentioned in eoc unphi produces untestable programs eo#3213 (comment)
  • Fix builtin normalizer to produce termination in some situations
  • Fix dataization inside application/dispatch
  • Fix encoding for strings to follow UTF-8 (compatibility with EO)
  • Fix bool representation to require one byte (compatibility with EO)
  • Fix integer division to truncate toward zero (compatibility with EO)
  • Improve pretty-printer (use indentation)
  • Update some examples/docs on the site
  • Support up to 3 positional arguments in the builtin normalizer
  • Non-IO support for Lorg_eolang_io_stdout is NOT implemented, pending response to eoc unphi produces untestable programs eo#3213 (comment)

PR-Codex overview

This PR updates dependencies, adds command options, modifies file paths, and refactors code for better readability and functionality.

Detailed summary

  • Updated eolang dependency to version 0.21.0.
  • Added --tests option to the eo unphi command.
  • Modified file paths in the try-unphi.sh script.
  • Updated options in report.md and quick-start.md.
  • Added prepare-pipeline-tests command.
  • Refactored code for better object wrapping and dataization.
  • Improved functions for handling bytes and strings.

The following files were skipped due to too many changes: eo-phi-normalizer/src/Language/EO/Phi/Rules/Fast.hs, package-lock.json, site/docs/src/normalizer/transform.md

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@deemp deemp linked an issue Jun 5, 2024 that may be closed by this pull request
@fizruk fizruk force-pushed the 390-fix-normalized-phi-to-normalized-eo-conversion-in-the-pipeline-script branch from 10fdf47 to cf04b8c Compare June 5, 2024 20:25
@fizruk fizruk merged commit 8347f00 into master Jun 5, 2024
8 checks passed
@fizruk fizruk deleted the 390-fix-normalized-phi-to-normalized-eo-conversion-in-the-pipeline-script branch June 5, 2024 23:35
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.

Fix normalized PHI to normalized EO conversion in the pipeline script
2 participants