> Yes, we should change the handling of mhchem to act like esint and amsmath. 
> Then the user can decide 
> if mhchem is used when a formula contains \ce or not. Can you please report a 
> bug accordingly at
> http://www.lyx.org/trac/wiki/BugTrackerHome
> and CC "uwestoehr" and "vfr"?

Ok, added.

> Besides this, a general problem of your file is that you have about 100! 
> macros with names like "Q" 
> etc. Such short names will bring you bug trouble when you load packages in 
> the preamble. Imagine you 
> have to compile your file on another PC with another LaTeX system and thus 
> perhaps more recent or 
> older packages that already might define the same command \Q as you.
> It is therefore strongly recommended to use well-determinable command names.

Thanks a lot for all your hints. Let me give some comments:
My experience with packages is that they are themselves using
"well-determinable" command-names and almost never such short ones like
\Q. Your package mhchem seems to be an exception ;-)

> When you nevertheless prefer names like \Q, don't use math macros but define 
> it in the preamble 
> using the command \providecommand .

But I love math-macros ...

> Some general comments while looking at your file:
> - You are not using all 100 macros in your file, so better delete the ones 
> you don't use.

Well, I am using all of them, but not in the same file. When I start a
new file, I simply copy my macros. Deleting them is quite an effort and
makes sense for me only when I run into problems.

> - You have a lot of macros without a real advantage like
>    \global\long\def\Grequivornot{\equiv_{(G)}}
>    but typing "\Grequivornot" to a formula is as quick as typing "\equiv_(G)"

First you complain that I use short names like \Q and now that I use
long names ;-)
Well, the main reason for this macro is not that it is shorter to write,
but that I was not sure how to denote it, and I want to be able to
quickly change it at every appearance, by changing the definition of the
macro. 

> - You define things that are already defined by special packages like
>    \newcommand{\bra}[1]{<#1 |}
>    This is not correct because the "|" won't scale when e.g. #1 is a fraction.
>    Better use the LaTeX-package "braket" that is described in LyX's Math 
> manual,
>    sec. 23.3.

Well, this is intentional. I want to have one that is not scaling. I
have defined a second one \Bra, which is scaling. Also I don't like to
include whole packages, if its so simple definitions like this...

> - You are using unnecessary ERT like \vspace{-.1cm}. This can also be inserted
>    via LyX's vertical space dialog. (OK, You might prefer ERT, I only wanted 
> to
>    mention that we support it natively.)

Thanks for the hint! I noticed the feature recently in lyx1.6.4, but the
file is a bit older.

> - I bet a box of beer that Susy doesn't exist and thus won't be discovered at 
> the LHC ;-).

Ha!Perhaps it won't be discovered at LHC, but for sure it exists and you
will never be able to disprove it! :-) (similar to the god-proofs and
disproofs...)
I take the bet ;-)

Best, Sebastian

Reply via email to