search

scottransom / presto

Open source pulsar search and analysis toolkit
http://www.cv.nrao.edu/~sransom/presto/
GNU General Public License v2.0
240 stars 177 forks source link

Remove the Pusar Miner in FAQ #185

Closed joezhouchenye closed 1 year ago

joezhouchenye commented 1 year ago

Pulsar Miner is using Python 2.7 and Presto is using Python >= 3.7. It's a little confusing as I tried to use Pulsar Miner and installed Python 2.7, but find out it's a total waste of time.

scottransom commented 1 year ago

I certainly don't want to remove Pulsar Miner from the FAQ. Because it is being used by quite a few people and finding pulsars (you can always install an earlier version of PRESTO). But much better would be to help @alex88ridolfi port Pulsar Miner to Python3+! That's not hard to do at all, and would probably be appreciated.

alex88ridolfi commented 1 year ago

Hi @joezhouchenye and @scottransom . PULSAR_MINER has already been ported to Python 3 and is compatible with PRESTO 3+! Just, I haven't released it publicly because I wanted to also include a number of new functionalities (currently under test) with the overhauled version. Given that it's taking much more time than I was expecting (sorry!), I'll release the Python3 version now, and add the new functionalities in the next release. I'll keep you posted!

scottransom commented 1 year ago

Great news! I did 30min of work on PULSAR_MINER this morning and almost have it working in Python3 myself!

alex88ridolfi commented 1 year ago

Hi @joezhouchenye and @scottransom . I have just pushed PULSAR_MINER 2.0-beta with support to Python3 and PRESTO 3+: https://github.com/alex88ridolfi/PULSAR_MINER Even if I call it "beta", I have been using it for months and it should be very stable. However, if you encounter any issues, please point them out to me.

Cheers, Alessandro

joezhouchenye commented 1 year ago

Thanks, that's great!