← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Board statistics dialog

 

Hi Alexander,

The 6.0.0-rc1 milestone[1] tag is the HEAD of master or at least what
the next release version that master will become once it's ready to release.

Cheers,

Wayne

[1]: https://launchpad.net/kicad/+milestone/6.0.0-rc1

On 7/31/19 6:31 AM, Alexander Shuklin wrote:
> 
> Hi! Thanks for that.
> Currently I cannot reproduce the bug.
> I think it can be connected to KiCad 6.0.0-rc1 tag. I will compile and look.
> Can somebody please explain me about 6.0 version tag? I mean, should I
> look for bugs in that tag or just follow master?
> Sorry if that's stupid question. I really don't understand that )))
> 
>     Понедельник, 29 июля 2019, 21:20 +03:00 от Diego Herranz
>     <diegoherranz@xxxxxxxxxxxxxxxx>:
> 
>     Done: https://bugs.launchpad.net/kicad/+bug/1838325
> 
>     Thanks!
> 
>     On Sun, 28 Jul 2019 at 15:04, Wayne Stambaugh <stambaughw@xxxxxxxxx
>     <//e.mail.ru/compose/?mailto=mailto%3astambaughw@xxxxxxxxx>> wrote:
> 
>         Diego,
> 
>         Please file a bug report for this and include the board file
>         that causes
>         this so it's easier for us to figure out what is going on.
> 
>         Thanks,
> 
>         Wayne
> 
>         On 7/27/19 5:06 AM, Diego Herranz wrote:
>         > I've been testing this dialog and I think it is a nice
>         addition. Thanks!
>         >
>         > There seems to be something wrong with the area calculation,
>         though. See
>         > image below:
>         > area.png
>         >
>         > Thanks,
>         > Diego
>         >
>         > On Tue, 23 Jul 2019 at 11:18, Ian McInerney
>         <Ian.S.McInerney@xxxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3aIan.S.McInerney@xxxxxxxx>
>         > <mailto:Ian.S.McInerney@xxxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3aIan.S.McInerney@xxxxxxxx>>> wrote:
>         >
>         >     Alexander,
>         >
>         >     Instead of declaring the 2 static variables separately, I
>         would
>         >     suggest creating a struct for the settings then store that
>         as the
>         >     static variable. For an example of this see the
>         >     dialog_create_array.cpp file. This way if any new options
>         must be
>         >     added in the future, they can just be added to the struct
>         very easily.
>         >
>         >     -Ian
>         >
>         >     On Mon, Jul 22, 2019 at 9:39 PM Alexander Shuklin
>         <jasuramme@xxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3ajasuramme@xxxxxxx>
>         >     <mailto:jasuramme@xxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3ajasuramme@xxxxxxx>>> wrote:
>         >
>         >         Damn ><,
>         >         don't use last patch, please.
>         >         It doesn't count total vias amount. Use this one.
>         >
>         >
>         >             Понедельник, 22 июля 2019, 22:14 +03:00 от
>         Alexander Shuklin
>         >             <jasuramme@xxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3ajasuramme@xxxxxxx>
>         <mailto:jasuramme@xxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3ajasuramme@xxxxxxx>>>:
>         >
>         >             Hi,
>         >             thanks for sharing experience, as I never used that
>         >             translations or wxWidgets before. And I have no
>         idea where
>         >             else could I get that information. ))
>         >             So, there's the patch with vias information and
>         some tiny
>         >             improvements.
>         >
>         >
>         >                 Понедельник, 22 июля 2019, 13:34 +03:00 от Ian
>         McInerney
>         >                 <ian.s.mcinerney@xxxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3aian.s.mcinerney@xxxxxxxx>
>         >                 <mailto:ian.s.mcinerney@xxxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3aian.s.mcinerney@xxxxxxxx>>>:
>         >
>         >
>         >
>         >                 On Mon, Jul 22, 2019 at 11:03 AM Dino Ghilardi
>         >                 <dino.ghilardi@xxxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3adino.ghilardi@xxxxxxxx>
>         >               
>          <http://e.mail.ru/compose/?mailto=mailto%3adino.ghilardi@xxxxxxxx>>
>         >                 wrote:
>         >
>         >                     Hi Alexander,
>         >
>         >                     One possible solution for the translation
>         could be
>         >                     put the ":" in a
>         >                     different column of the table and
>         right-align the
>         >                     field description text
>         >                     (so all the colons will be aligned). A
>         rapid google
>         >                     search shown that in
>         >                     French and Vietnamese there should be a
>         space before
>         >                     the colon, while in
>         >                     the rest of the world there is not, so
>         having the
>         >                     translation for the
>         >                     ":" word seems to make sense. Also another
>         question
>         >                     arises: Is there
>         >                     some language in which the colon should be
>         another
>         >                     character before the
>         >                     word? (I'm thinking about spanish where
>         the question
>         >                     mark upside-down
>         >                     appears before a question...)? ...conclusion:
>         >                     keeping "Height:" and
>         >                     "Height" as two different words seem to be the
>         >                     solution that gives
>         >                     maximum flexibility to translators.
>         >
>         >
>         >                 This actually doesn't give them as much
>         flexibility.
>         >                 When translations are done, they need to
>         examine the
>         >                 entire string that needs translating, so the ":"
>         >                 character should be included in the string.
>         Separating
>         >                 out the two portions is the equivalent of
>         saying that
>         >                 every lanugage will follow the same
>         compositional rules.
>         >                  
>         >
>         >                     Another possible solution (probably better
>         then the
>         >                     one above since it
>         >                     just removes the problem) is to remove the
>         ":" and
>         >                     have the cell borders
>         >                     in a different color, just like the tables
>         in the
>         >                     "board setup" dialog
>         >                     (so that you can also take a look at that
>         code to
>         >                     solve also the color
>         >                     problem seeing how it was solved there). The
>         >                     advantage of this approach
>         >                     is also having a more consistent "look"
>         through all
>         >                     the dialogs.
>         >
>         >
>         >
>         >                     P.S. (a little bit off-topic):
>         >                     If you move the statistic window and
>         check/uncheck
>         >                     one of the checkboxes
>         >                     ("subctract holes...." or "Exclude
>         components...")
>         >                     the window "jumps" to
>         >                     the center of the screen (its default
>         position on
>         >                     open): do you have
>         >                     also this behaviour or it is just on my
>         debian-linux
>         >                     with gtk3?
>         >
>         >
>         >                     Cheers,
>         >                     Dino.
>         >
>         >                     On 22/07/19 10:13, Alexander Shuklin wrote:
>         >                     > Hi!
>         >                     > I'll have a look to add vias count to
>         dialog.
>         >                     > There's some questions:
>         >                     >
>         >                     > 1)I don't have too much experience with
>         wxdialogs.
>         >                     There was commit on
>         >                     > master, which says:
>         >                     >  >> remove settings for fg/bg color: the
>         result is
>         >                     unpredictable: was
>         >                     > black texts on black background on my
>         computer.
>         >                     > And now I have all tables with data just
>         in white
>         >                     boxes. Is it how it
>         >                     > meant to be, or just some misbehavior on
>         different
>         >                     systems? I use
>         >                     > archlinux x64 OS.
>         >                     > there's screenshot in attachment
>         >                     >
>         >                     > 2) Can we use something like _( "Height"
>         ) + ":"
>         >                     for translation, not _(
>         >                     > "Height:" )? As far as I understand, now
>         we will
>         >                     need to have 2
>         >                     > translations, first for "Height" and
>         second for
>         >                     "Height:" but that's
>         >                     > basically same word.
>         >                     >
>         >                     >     Воскресенье, 21 июля 2019, 23:42
>         +03:00 от
>         >                     Dino Ghilardi
>         >                     >     <dino.ghilardi@xxxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3adino.ghilardi@xxxxxxxx>
>         >                   
>          <http://e.mail.ru/compose/?mailto=mailto%3adino.ghilardi@xxxxxxxx>>:
>         >                     >
>         >                     >     Makes sense.
>         >                     >     Instead of a generic "via count" a more
>         >                     complete table similar to the
>         >                     >     one generated in the drill report
>         file could
>         >                     be useful, but may be it
>         >                     >     can became quite long if a lot of
>         different
>         >                     drill sizes are used (ok,
>         >                     >     scrollbars are made to handle that).
>         >                     >     Also having "vias", "blind vias" and
>         >                     "microvias" count man make sense
>         >                     >     (or at least having something like
>         "microvias
>         >                     used: yes/no"), just to
>         >                     >     have in board statistics the
>         information about
>         >                     the need of an advanced
>         >                     >     pcb manufacturing process.
>         >                     >
>         >                     >
>         >                     >     Cheers,
>         >                     >     Dino.
>         >                     >
>         >                     >
>         >                     >     On 21/07/19 20:54, Mark Roszko wrote:
>         >                     >      > > Since making every hole in the
>         pcb costs
>         >                     time,  manufacturers
>         >                     >      > calculate the price of the PCB
>         using also
>         >                     that number.
>         >                     >      >
>         >                     >      > A long time ago, holes cost alot.
>         These
>         >                     days if your manufacturer is
>         >                     >      > charging alot per hole, you
>         should run far
>         >                     away. Usually
>         >                     >     manufactuers
>         >                     >      > include 20k-40k holes in the base
>         price per
>         >                     panel before they start
>         >                     >      > charging you tiny penny amounts
>         for more in
>         >                     some increments of
>         >                     >      > thousands. The CNC machines these
>         days as
>         >                     blazing fast at making the
>         >                     >      > holes required and they charge
>         more for the
>         >                     drill bit being worn
>         >                     >     down
>         >                     >      > than the time.
>         >                     >      >
>         >                     >      > What does actually cost money is
>         via type
>         >                     and size. Blind vias make
>         >                     >      > layer stackups a pain in the butt
>         and micro
>         >                     vias needing lasers.
>         >                     >      > And also going below some via
>         size can
>         >                     significantly add cost
>         >                     >     even if
>         >                     >      > not at microvia size because it
>         requires
>         >                     different machines with
>         >                     >     more
>         >                     >      > precision and smaller drill bits.
>         >                     >      >
>         >                     >      >
>         >                     >      > If anything I would say to break
>         down the
>         >                     statistics based on via
>         >                     >     type.
>         >                     >      >
>         >                     >      > On Sun, Jul 21, 2019 at 1:55 PM
>         Dino Ghilardi
>         >                     >     <dino.ghilardi@xxxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3adino.ghilardi@xxxxxxxx>
>         >                   
>          <http://e.mail.ru/compose/?mailto=mailto%3adino.ghilardi@xxxxxxxx>
>         >                     <mailto:dino.ghilardi@xxxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3adino.ghilardi@xxxxxxxx>
>         >                   
>          <http://e.mail.ru/compose/?mailto=mailto%3adino.ghilardi@xxxxxxxx>>
>         >                     >      > <mailto:dino.ghilardi@xxxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3adino.ghilardi@xxxxxxxx>
>         >                   
>          <http://e.mail.ru/compose/?mailto=mailto%3adino.ghilardi@xxxxxxxx>>>
>         >                     wrote:
>         >                     >      >
>         >                     >      > I just tried the board statistics
>         dialog
>         >                     and looks good, I like it.
>         >                     >      >
>         >                     >      > A thing I'd like to have to make
>         it better
>         >                     is adding the number
>         >                     >     of vias
>         >                     >      > to the statistics: Since making
>         every hole
>         >                     in the pcb costs time,
>         >                     >      > manufacturers calculate the price
>         of the
>         >                     PCB using also that number.
>         >                     >      >
>         >                     >      > Also the option to save or print
>         a text
>         >                     with the statistics report
>         >                     >      > would
>         >                     >      > be nice.
>         >                     >      >
>         >                     >      >
>         >                     >      > Cheers,
>         >                     >      > Dino.
>         >                     >      >
>         >                     >
>         >                     >   
>          _______________________________________________
>         >                     >     Mailing list:
>         >                     https://launchpad.net/~kicad-developers
>         >                     >     Post to :
>         kicad-developers@xxxxxxxxxxxxxxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3akicad%2ddevelopers@xxxxxxxxxxxxxxxxxxx>
>         >                   
>          <http://e.mail.ru/compose/?mailto=mailto%3akicad%2ddevelopers@xxxxxxxxxxxxxxxxxxx>
>         >                     >   
>          <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3akicad%2ddevelopers@xxxxxxxxxxxxxxxxxxx>
>         >                   
>          <http://e.mail.ru/compose/?mailto=mailto%3akicad%2ddevelopers@xxxxxxxxxxxxxxxxxxx>>
>         >                     >     Unsubscribe :
>         >                     https://launchpad.net/~kicad-developers
>         >                     >     More help :
>         https://help.launchpad.net/ListHelp
>         >                     >
>         >                     >
>         >                     >
>         >                     > --
>         >                     > Alexander Shuklin
>         >
>         >
>         >                   
>          _______________________________________________
>         >                     Mailing list:
>         https://launchpad.net/~kicad-developers
>         >                     Post to     :
>         kicad-developers@xxxxxxxxxxxxxxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3akicad%2ddevelopers@xxxxxxxxxxxxxxxxxxx>
>         >                   
>          <http://e.mail.ru/compose/?mailto=mailto%3akicad%2ddevelopers@xxxxxxxxxxxxxxxxxxx>
>         >                     Unsubscribe :
>         https://launchpad.net/~kicad-developers
>         >                     More help   :
>         https://help.launchpad.net/ListHelp
>         >
>         >
>         >
>         >             --
>         >             Alexander Shuklin
>         >
>         >
>         >
>         >         --
>         >         Alexander Shuklin
>         >
>         >     _______________________________________________
>         >     Mailing list: https://launchpad.net/~kicad-developers
>         >     Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3akicad%2ddevelopers@xxxxxxxxxxxxxxxxxxx>
>         >     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3akicad%2ddevelopers@xxxxxxxxxxxxxxxxxxx>>
>         >     Unsubscribe : https://launchpad.net/~kicad-developers
>         >     More help   : https://help.launchpad.net/ListHelp
>         >
>         >
>         > _______________________________________________
>         > Mailing list: https://launchpad.net/~kicad-developers
>         > Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3akicad%2ddevelopers@xxxxxxxxxxxxxxxxxxx>
>         > Unsubscribe : https://launchpad.net/~kicad-developers
>         > More help   : https://help.launchpad.net/ListHelp
>         >
> 
>         _______________________________________________
>         Mailing list: https://launchpad.net/~kicad-developers
>         Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>         <//e.mail.ru/compose/?mailto=mailto%3akicad%2ddevelopers@xxxxxxxxxxxxxxxxxxx>
>         Unsubscribe : https://launchpad.net/~kicad-developers
>         More help   : https://help.launchpad.net/ListHelp
> 
>     _______________________________________________
>     Mailing list: https://launchpad.net/~kicad-developers
>     Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
>     <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
>     Unsubscribe : https://launchpad.net/~kicad-developers
>     More help : https://help.launchpad.net/ListHelp
> 
> 
> 
> -- 
> Alexander Shuklin


Follow ups

References