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

Add feature to build rosetta with custom patches provided thru the web UI #358

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

terrykong
Copy link
Contributor

This should help test changes more quickly since an ephemeral branch is no longer needed. One pain point before this change is you would have to create a branch using a mirror-branch with the staged test, and after the test succeeds, you would need to clean up your ephemeral branch that referenced the mirror-branch.

…b UI

This should help test changes more quickly since an ephemeral branch is
no longer needed. One pain point before this change is you would have to
create a branch using a mirror-branch with the staged test, and after
the test succeeds, you would need to clean up your ephemeral branch that
referenced the mirror-branch.
@terrykong terrykong requested a review from yhtang November 6, 2023 17:59
Copy link
Collaborator

@yhtang yhtang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will this contradict our reproducible build effort, in the sense that if custom patches are provided via runtime arguments then they are hard to track?

@terrykong
Copy link
Contributor Author

In this workflow:

  1. patch needs to get updated
  2. rebase patch in new branch
  3. create jax-toolbox branch referencing new branch
  4. test
  5. delete jax-toolbox branch referencing new branch
  6. force push new branch into original patch branch

this change would simplify (3) and (5) since you no longer need to create a branch just for testing. All patches should still be present in the patchlists in version control.

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.

2 participants