Back Midas Rome Roody Rootana
  Midas DAQ System, Page 56 of 157  Not logged in ELOG logo
    Reply  23 Sep 2025, Konstantin Olchanski, Info, obsolete mana.c removal 
> Hi, at the LEM Experiment at PSI, we still use mana.c and would like to keep it until end of 2026, where we will enter a long shutdown.

Excellent, good to hear from you! Once we remove ROOT support rmana.o will be gone, only mana.o (no ROOT) will remain. Will this break your builds?
Entry  23 Sep 2025, Konstantin Olchanski, Info, 64-bit time_t 
To record discussion with Stefan regarding 64-bit time_t

To remember:
    Reply  23 Sep 2025, Pavel Murat, Info, switch midas to c++17 
perhaps c++20? - std::format is an immediately useful feature. --regards, Pasha

> Following discussions at the MIDAS workshop, we propose to move MIDAS from c++11 to c++17. There is 
Entry  23 Sep 2025, Konstantin Olchanski, Info, long history variable names 
To record discussion with Stefan about long history variable names.

We have several requests to remove the 32-byte limit on history variable names.
    Reply  23 Sep 2025, Konstantin Olchanski, Info, switch midas to c++17 
> perhaps c++20? - std::format is an immediately useful feature. --regards, Pasha

confirmed. std::format is an improvement over K&R C printf().
    Reply  23 Sep 2025, Pavel Murat, Info, switch midas to c++17 
> > perhaps c++20? - std::format is an immediately useful feature. --regards, Pasha

> confirmed. std::format is an improvement over K&R C printf().
    Reply  24 Sep 2025, Andreas Suter, Info, obsolete mana.c removal 
Sorry, 

I have had now the time to dig deeper in our code and realized that we actually use rmana, i.e. WITH ROOT. If there is an easy way to incorporate the necessary
    Reply  06 Nov 2025, Konstantin Olchanski, Info, switch midas to c++17 
> Following discussions at the MIDAS workshop, we propose to move MIDAS from c++11 to c++17.

There were no objections to this move.
    Reply  06 Nov 2025, Konstantin Olchanski, Info, removal of ROOT support in mlogger 
> we should finally bite the bullet and remove ROOT support from MIDAS ...

there were no objections to removing ROOT support from mlogger.
Entry  06 Nov 2025, Konstantin Olchanski, Info, make clang-tidy 
I added Makefile rules for running MIDAS through the clang-tidy static code 
analyzer. (rules for running cppcheck have gone missing, I hope I find them).
    Reply  10 Nov 2025, Konstantin Olchanski, Info, db_delete_key(TRUE) 
> I would handle this actually like symbolic links are handled under linux. If you delete a symbolic link, the link gets 
> detected and NOT the file the link is pointing to.
    Reply  20 Nov 2025, Konstantin Olchanski, Info, db_delete_key(TRUE) 
> > I would handle this actually like symbolic links are handled under linux. If you delete a symbolic link, the link gets 
> > detected and NOT the file the link is pointing to.
> > 
    Reply  20 Nov 2025, Konstantin Olchanski, Info, switch midas to c++17 
> > Following discussions at the MIDAS workshop, we propose to move MIDAS from c++11 to c++17.
> We shall move forward with this change.
    Reply  20 Nov 2025, Konstantin Olchanski, Info, removal of ROOT support in mlogger 
> > we should finally bite the bullet and remove ROOT support from MIDAS ...

as discussed, HAVE_ROOT and OBSOLETE were removed from mlogger. rmana and ROOT support in manalyzed remain, 
    Reply  20 Nov 2025, Nick Hastings, Info, switch midas to c++17 

> I notice the cmake does not actually pass "-std=c++17" to the c++ compiler, and on U-20, it is likely 
> the default c++14 is used. cmake always does the wrong thing and this will need to be fixed later.
    Reply  21 Nov 2025, Konstantin Olchanski, Info, cppcheck 
> (rules for running cppcheck have gone missing, I hope I find them).

found them. I built cppcheck from sources.
    Reply  24 Nov 2025, Stefan Ritt, Info, switch midas to c++17 
> > > Following discussions at the MIDAS workshop, we propose to move MIDAS from c++11 to c++17.
> > We shall move forward with this change.
    Reply  25 Nov 2025, Konstantin Olchanski, Info, switch midas to c++17 

> > I notice the cmake does not actually pass "-std=c++17" to the c++ compiler, and on U-20, it is likely 
> > the default c++14 is used. cmake always does the wrong thing and this will need to be fixed later.
    Reply  25 Nov 2025, Konstantin Olchanski, Info, switch midas to c++17 
> target_compile_features(<target> PUBLIC cxx_std_17)
> which correctly causes a
> c++ -std=gnu++17 ...
    Reply  26 Nov 2025, Stefan Ritt, Info, switch midas to c++17 
I switched from 

  target_compile_features(<target> PUBLIC css_std_17)
ELOG V3.1.4-2e1708b5