Opened 10 years ago
Last modified 10 years ago
#13 new enhancement
Discontinue MAGIC support in FACT/Mars
Reported by: | dneise | Owned by: | somebody |
---|---|---|---|
Priority: | minor | Milestone: | milestone: |
Component: | component1 | Version: | |
Keywords: | Cc: |
Description
In a private discussion with Thomas Bretz he mentioned, that it might be a good idea to stop supporting MAGIC in our version of Mars. I welcome this decision very much, however I this needs of course to be agreed on by the majority of users/developers of Mars in FACT.
So, I put up this ticket mainly to have a place to discuss this (if a discussion is needed at all). As soon as the majority agrees, we will officially drop MAGIC support. That's it for the moment. No code will be deleted or so.
We just create a snapshot of the trunk, that is known to be the last version of FACT/Mars, where MAGIC support has not been officially dropped. (Of course we know, that it is not clear whether MAGIC data analysis was fully supported before). After this point in time, developers are free to throw away code, that is not needed by FACT anymore.
An important question of course is, are there any MAGIC members in FACT, that do use the FACT/Mars to analyze some MAGIC data? If this is the case please inform us now.
At the same time I would like to drop support for ROOT versions smaller than 5.0. So if anybody thinks we should still support ROOT 3 or 4, please speak up.
I propose to leave the discussion open for a couple of weeks (say 2 weeks) and if nobody is against this, I will advance with this task.
Change History (3)
comment:1 by , 10 years ago
comment:2 by , 10 years ago
Do we still have support for old ROOT versions? With my 5.18 it doesn't compile anymore since a loooong time.
When I said, "dropping support for ROOT versions smaller than 5" I was referring to parts of the FACT/Mars code base, that explicitly contains work arounds for very old ROOT versions. Since there probably aren't many of these places nor is it likely that despite those dedicated work arounds any of these old versions does still work with FACT/Mars, it might not be worth discussing this matter to a large extend.
Anybody can of course still try to use FACT/Mars to analyse any data from any experiment. All I'm saying is, that we as the FACT collaboration should, for the sake of code cleanliness, focus on FACT only. And therefor explicitly allow fellow developers to drop code, that is not FACT related, if they feel the need to.
comment:3 by , 10 years ago
with my comment, I wanted just to mention that with root 5.18 Mars does not compile on my laptop anymore since at least two years
I'm not sure whether this is the right place to discuss this, as there might be people who are not in FACT, but use the software to analyse MAGIC data. I know that for example Stefan R. was still using it, but as I haven't seen him for a while, I'm not sure what his plans are.
Personally, I definitely interested in still analysing MAGIC data, but I doubt that I will find time for it :(
Do we still have support for old ROOT versions? With my 5.18 it doesn't compile anymore since a loooong time.