hackl / TikZ-StructuralAnalysis

TikZ Library for Structural Analysis
Other
123 stars 29 forks source link

stanli.sty: Use stanli's 3dplot defs only as fallback, load tikz-3dplot/3dplot packages if available. #4

Closed agmartin2 closed 6 years ago

agmartin2 commented 6 years ago

Hi, Jürgen,

Thanks for this project. I started using it recently and found a problem when tikz-3dplot is also loaded.

stanli.sty provides a copy of some 3dplot definitions. However, the way it is handled is incompatible with the use of {tikz-,}3dplot in the same document, causing errors

! LaTeX Error: Command \tdplotsinandcos already defined. Or name \end... illegal, see p.192 of the manual.

This patch checks for tikz-3dplot/3dplot existence and requires them if available. Since this part exits stanli.sty on success, stanli's own copy of the definitions will only be used as fallback.

No problem if tikz-3dplot/3dplot package is already loaded, \RequirePackage will not try to load it again.

Current patch is not too versatile, you cannot add new non 3dplot code after the tests, but at least it shows the problem.

agmartin2 commented 6 years ago

Hi, thanks for the merge.

Bad news, just noticed that using current tikz-3dplot commands is causing problems when compiling stanli manual.

LaTeX Warning: Reference `sec:3dsetaxis' on page 36 undefined on input line 326
4.

! Undefined control sequence.
<argument> \raarot 
                   *\raaeul 
l.3276          \daxis{3}{0}[a][c][.4][63.43][18.43];

? 

No idea about why is this happening.

hackl commented 6 years ago

Hi, it seems that there were changes made in the tikz-3dplot.sty file. Unfortunately, I couldn't figure out why the new version is not working properly. I fixed it by relabeling the commands used in stanli, hence there shouldn't be any conflict with using tikz-3dplots any longer.

agmartin2 commented 6 years ago

Hi,

Verified to work both with stanly.tex and my original file. Thanks a lot for your fix.