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

Improve YAML anchor names #30

Open
litghost opened this issue Feb 25, 2021 · 0 comments
Open

Improve YAML anchor names #30

litghost opened this issue Feb 25, 2021 · 0 comments
Labels
enhancement New feature or request

Comments

@litghost
Copy link
Contributor

litghost commented Feb 25, 2021

Currently the RapidYAML text format names anchor sequentially, see https://github.com/SymbiFlow/python-fpga-interchange/blob/3809a19f2f9cf80f014e439f9ae8643543c4519f/fpga_interchange/rapidyaml_support.py#L87

To aid readability of the text formats, the anchor names should incorporate some additional information. For example, rather than id001 for a port anchor, idXXX_port_<name> would be useful. To avoid exploding the file size, names should be truncated. Leaving the counter in the front avoids needing to de-duplicate names.

@litghost litghost added this to To Do in FPGA interchange bootstrapping via automation Feb 25, 2021
@issuelabeler issuelabeler bot added the duplicate This issue or pull request already exists label Feb 25, 2021
@litghost litghost changed the title Improvement YAML anchor names Improve YAML anchor names Feb 25, 2021
@litghost litghost added enhancement New feature or request and removed duplicate This issue or pull request already exists labels Feb 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant