Re: [Brahms-dev-l] brat version bump AGAIN

Hello.  I adds a few more info about these values in our database.

1.  4 columns are added to "Runs" table in "RUNDB" database.
2.  They are:
    MrsQuality (enumerated list ("Good","Bad","Marginal","Other")) 
default "Good"
    MrsQualityComment (text) default ""  (or empty)
    FsQuality (enumerated list ("Good","Bad","Marginal","Other")) 
default "Good"
    FsQualityComment (text)  default "" (or empty)

3. Since default qualities are good, it requires people to change them 
to bad if the runs are really bad. To facilitate this, there is web page 
to change these values in our database.  It is 
http://pii3.brahms.bnl.gov/daq-cgi-bin/ChangeRunQuality.pl (there is a 
link in our daq page.)  When you find a run to be really bad, I suggest 
you to go that page and change its quality to "Bad" and add comments.

4.  If you want to find a list of good runs, there is already web page ( 
http://pii3.brahms.bnl.gov/daq-cgi-bin/FindGoodRuns.pl ).   This only 
lists runs with only "Good" quality.


Hiro


Kris Hagel wrote:

> Hello,
> As per request of FV, I have added the new FS/MRS quality fields of  
> the run DB into the run db framework.  To use them, the average Joe 
> Sixpack casual brat user would do (after all of the setup that 
> everyone knows how to do) if he has a BrRunInfo* object called runInfo:
> Char_t* mrsQuality = runInfo->GetMrsQuality();
> Char_t* mrsQualityComment = runInfo->GetMrsQualityComment();
> Char_t* fsQuality = runInfo->GetFsQuality();
> Char_t* fsQualityComment = runInfo->GetFsQualityComment();
>
> The more advanced user might access directly the db table (ie a 
> BrDbRun object called runDb); in that case one would use:
> Char_t *mrsQuality = runDb->GetMrsQuality();
> etc (there is a symmetry here)
>
> It should be noted that the qualityComment for both mrs and fs seem to 
> be not often filled in.  In that case a NULL pointer is returned.  So 
> the validity of that variable must be tested before using it.  It also 
> appears that the quality field for earlier runs was set to Good.
>
> I also cleaned up some remaining istream stuff according to our gcc3.2 
> convention as well as making the cout<< operator work for BrRunInfo 
> (was useful to test the new features)
>
> The brat version was bumped to BRAT-2-14-4
>
> Happy brat-ing
>
> Kris
>
> _______________________________________________
> Brahms-dev-l mailing list
> Brahms-dev-l@lists.bnl.gov
> http://lists.bnl.gov/mailman/listinfo/brahms-dev-l



_______________________________________________
Brahms-dev-l mailing list
Brahms-dev-l@lists.bnl.gov
http://lists.bnl.gov/mailman/listinfo/brahms-dev-l