search

cositools / cosipy

The COSI high-level data analysis tools
Apache License 2.0
3 stars 16 forks source link

Update preliminary TS map #168

Open PbU-Jason opened 3 months ago

PbU-Jason commented 3 months ago

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 commented 3 months ago

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 commented 3 months ago

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

Yong2Sheng commented 3 months ago

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

israelmcmc commented 3 months ago

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.