Print

Print


Hi Gabriel,
thanks.
Why you suggest to not commit the EcalHitPlots in the case I modify it 
(I have not done it!)?
I mean, the idea I have is that when you work on a piece of the 
software, at a certain point it will be part of the "hps-software".
How is this managed? Who is responsible to decide when a certain "part" 
will be inserted in the main trunk?
I ask this because I prefer to know this in advance, before starting the 
real work, in order to not have to re-do later one.

Thanks!

Andrea

Il 02/26/2014 01:16 PM, Gabriel CHARLES ha scritto:
> Hi Andrea,
>
>   I think you can do both but respecting different conditions.
>
>   If you modify EcalHitPlots and do not commit it, no one will see it, 
> so you can do whatever you want, but you must be sure you won't commit 
> it by mistake and organize your own backup. It is not what I would 
> recommend.
>
>   If you create a new class and want to commit it, you can create it 
> here: "src.main.java.org.lcsim.hps.users.acelentano", for me it is 
> located in the java trunk, and there is a folder "users". I think this 
> is the prefered solution because you can share it with other if needed.
>
>   If it can help, when you create your own PlotsDriver without any 
> option you can call it from the steering file this way:
>
>         <driver name="YourDriverName"/>
>
>         <driver name="YourDriverName" 
> type="org.lcsim.hps.users.acelentano.PlotsDriver">
>         </driver>
>
> Hope it helps,
> ---
> Gabriel CHARLES
> Institut de Physique Nucléaire d'Orsay
>
> On Wed, 26 Feb 2014 12:22:39 +0100, Andrea Celentano wrote:
>> Dear all,
>> a stupid question about the "proper/standard/hps-defined/.." way to
>> proceed for the Ecal monitoring:
>>
>> - I see there is a class org.lcsim.hps.ecal.monitoring.EcalHitPlots
>> that implements (some) of the plots I'd like to add.
>> - Should I modify it, adding what I need
>> OR
>> -Should I create a new class, modify the steering file accordingly
>> (and eventually copying the pieces of code I need from EcalHitPlots)?
>>
>> Thanks
>>
>> Andrea
>>
>>
>> ########################################################################
>> Use REPLY-ALL to reply to list
>>
>> To unsubscribe from the HPS-SOFTWARE list, click the following link:
>> https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=HPS-SOFTWARE&A=1

########################################################################
Use REPLY-ALL to reply to list

To unsubscribe from the HPS-SOFTWARE list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=HPS-SOFTWARE&A=1