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

Allow printing of parameters in Fortran #1811

Draft
wants to merge 31 commits into
base: devel
Choose a base branch
from
Draft

Conversation

EmilyBourne
Copy link
Member

@EmilyBourne EmilyBourne commented Mar 29, 2024

Allow the printing of constants as parameter in Fortran. Use this improvement to avoid unnecessary sign checks when using negative indices.

@EmilyBourne EmilyBourne added the blocked Cannot be solved/merged until something else is fixed label Mar 29, 2024
@pyccel-bot
Copy link

pyccel-bot bot commented Mar 29, 2024

Hello again! Thank you for this new pull request 🤩.

Please begin by requesting your checklist using the command /bot checklist

@EmilyBourne EmilyBourne linked an issue Apr 2, 2024 that may be closed by this pull request
EmilyBourne added a commit that referenced this pull request May 14, 2024
Print unknown constants using their values. This fixes #1047 but may be
reverted in #1811 to prefer declaring constants with names. However this
PR also adds support for `NaN` in C and an associated test. It also
removes the duplicate `_print_Constant` function.

---------

Co-authored-by: EmilyBourne <louise.bourne@gmail.com>
@EmilyBourne EmilyBourne removed the blocked Cannot be solved/merged until something else is fixed label May 14, 2024
EmilyBourne added a commit to pyccel/pyccel-cuda that referenced this pull request May 14, 2024
Print unknown constants using their values. This fixes pyccel#1047 but may be
reverted in pyccel#1811 to prefer declaring constants with names. However this
PR also adds support for `NaN` in C and an associated test. It also
removes the duplicate `_print_Constant` function.

---------

Co-authored-by: EmilyBourne <louise.bourne@gmail.com>
@EmilyBourne EmilyBourne self-assigned this May 22, 2024
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.

Improve handling of Constant scalars
1 participant