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

Update preliminary TS map #168

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from
Draft

Conversation

PbU-Jason
Copy link
Contributor

Hi, I'm Jason.
I reviewed my previous code on implementing multi-resolution TS map using HealpixMap.
My notebook and explanatory PDF file have been put to cosipy/cosipy/ts_map/preliminary/.
I was hoping that my pull request would not affect the main branch. However, I can not create a branch on cositools/cosipy/.
Therefore, I'm pulling my local branch directly to the main branch of cositools/cosipy/.
Please let me know if anything is wrong. Thanks a lot!

@israelmcmc
Copy link
Collaborator

Thank you so much, @PbU-Jason! Indeed, I agree with you, we need to be a little more careful with the main branch now that we're starting to make releases. I'll work on this. In the meantime, I'll mark this PR as a draft to prevent an accidental merge.

@israelmcmc
Copy link
Collaborator

Tagging @Yong2Sheng. Hopefully this could be merged with the current code.

@israelmcmc israelmcmc marked this pull request as draft March 20, 2024 13:45
@Yong2Sheng
Copy link
Contributor

Thank you very much @PbU-Jason and @israelmcmc, I will also take a look at this.

@israelmcmc israelmcmc changed the base branch from main to develop March 25, 2024 15:15
@israelmcmc
Copy link
Collaborator

I changed the target branch to "develop". I want to avoid adding new features to the version we're releasing today. I'll explain this further tomorrow during the cosipy meeting.

@israelmcmc israelmcmc added tsmap enhancement New feature or request labels May 9, 2024
@lommler lommler marked this pull request as ready for review June 24, 2024 16:22
@israelmcmc israelmcmc marked this pull request as draft July 19, 2024 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request tsmap
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants