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

Discussion: Remove populate #60

Closed
John-Holt-Tessella opened this issue Apr 28, 2020 · 2 comments
Closed

Discussion: Remove populate #60

John-Holt-Tessella opened this issue Apr 28, 2020 · 2 comments

Comments

@John-Holt-Tessella
Copy link
Contributor

As a developer I would recommend getting rid of populate

Its purpose is to create variables which can be used instead of block names so that scan can be written: scan(block, ... instead of scan("block", .... But it comes with caveats that when the user changes config they must remember to rerun populate to get new blocks and it is different to the recommended way of running g.cset i.e. cset("block", value). However I do not know the history and so would like to understand its motivation.

Acceptance Criteria

  1. A decision is made on populate and the decision is recorder.
  2. Any needed tickets are generated to make the changes
@ChrisM-S
Copy link

ChrisM-S commented Apr 28, 2020 via email

@ThomasLohnert
Copy link

Solved by ISISComputingGroup/IBEX#5401, closing issue.

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

No branches or pull requests

3 participants