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

Merge gain-calibration branch #106

Open
piyanatk opened this issue Oct 1, 2024 · 0 comments
Open

Merge gain-calibration branch #106

piyanatk opened this issue Oct 1, 2024 · 0 comments
Assignees
Labels
code enhancement New feature or request help wanted Extra attention is needed important priority-medium

Comments

@piyanatk
Copy link
Collaborator

piyanatk commented Oct 1, 2024

We need to start the process of merging the gain-calibration branch to main, so that we can identify parts of the codes that still need updating and refinement, as well as the unit tests that still need to be written.

  • @wkhu-astro will tidy up the current code in the gain-calibration branch and submit a PR to merge the branch into main. Tag @piyanatk as a reviewer for the PR.
    • New functionality that will be added should be developed in a new development branch by branching out from gain-calibration. If the gain-calibration PR is merged before the new development branch is ready, it should be submitted as a new PR to be merged into main. Otherwise, the development branch can be merged back to gain-calibration. New commits will automatically show up in the PR.
  • @piyanatk will start reviewing the PR to identify what still need to be updated and added. A list of tasks for more code updates and unit tests will be made and delegated.
@piyanatk piyanatk added code help wanted Extra attention is needed labels Oct 1, 2024
@kfornaz kfornaz self-assigned this Oct 1, 2024
@piyanatk piyanatk self-assigned this Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code enhancement New feature or request help wanted Extra attention is needed important priority-medium
Projects
None yet
Development

No branches or pull requests

3 participants