Yes, I have noticed it. It is fixed in my new commit.
Danek


-----Original Message-----
From: slava77 [mailto:notifications@github.com]
Sent: Tue 3/11/2014 4:03 PM
To: cms-sw/cmssw
Cc: Kotlinski Bohdan
Subject: Re: [cmssw] Alignment LA correction for pixels (#2067)
 
> @@ -20,8 +20,21 @@
>  PixelCPEGenericESProducer::PixelCPEGenericESProducer(const edm::ParameterSet & p) 
>  {
>    std::string myname = p.getParameter<std::string>("ComponentName");
> +  // Use LA-width from DB. If both (upper and this) are false LA-width is calcuated from LA-offset
> +  //useLAWidthFromDB_ = p.getParameter<bool>("useLAWidthFromDB");
> +  useLAWidthFromDB_ = p.existsAs<bool>("useLAWidthFromDB")?
> +    p.getParameter<bool>("useLAWidthFromDB"):false;
> +  // Use Alignment LA-offset 
> +  //useLAAlignmentOffsets_ = p.getParameter<bool>("useLAAlignmentOffsets");

Copy-paste error here? 
useLAWidthFromDB is read twice, while you probably wanted useLAAlignmentOffsets below this line

---
Reply to this email directly or view it on GitHub:
https://github.com/cms-sw/cmssw/pull/2067/files#r10476916