Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve help replies (See issue 1409) #1416

Merged
merged 1 commit into from
Mar 12, 2025

Conversation

MarkusFrankATcernch
Copy link
Contributor

BEGINRELEASENOTES

$ geoDisplay -h
PersistencyIO    INFO  +++ Set Streamer to dd4hep::OpaqueDataBlock
DD4hep           INFO  ++ Using globally Geant4 unit system (mm,ns,MeV)
Info in <TGeoManager>: Changing system of units to Geant4 units (mm, ns, MeV).
Info in <TGeoManager::TGeoManager>: Geometry default, Detector Geometry created
Info in <TGeoNavigator::BuildCache>: --- Maximum geometry depth set to 100
geoPluginRun: No geometry input supplied. No geometry will be loaded.
   ------------------------------------------------------------------
  | Welcome to ROOT 6.35.01                        https://root.cern |
  | (c) 1995-2024, The ROOT Team; conception: R. Brun, F. Rademakers |
  | Built for linuxx8664gcc on Jan 20 2025, 14:50:44                 |
  | From heads/master@v6-31-01-4781-gce5f87d2f4                      |
  | With g++-12 (Ubuntu 12.3.0-17ubuntu1) 12.3.0                     |
  | Try '.help'/'.?', '.demo', '.license', '.credits', '.quit'/'.q'  |
   ------------------------------------------------------------------

Usage: -plugin DD4hep_GeometryDisplay  -arg [-arg]                                

     Invoke the ROOT geometry display using the factory mechanism.                

     -detector <string> Top level DetElement path. Default: '/world'                
     -option   <string> ROOT Draw option.    Default: 'ogl'                         
     -level    <number> Visualization level  [TGeoManager::SetVisLevel]  Default: 4 
     -visopt   <number> Visualization option [TGeoManager::SetVisOption] Default: 1
     -load              Only load the geometry. Do not invoke the display          
     -help              Print this help output  
     Arguments given: -help

  • Unfortunately we were not really following the linux rules concerning options:
    • one letter options: -<letter> <value> or
    • multi letter options: --<options>=<value>
  • maybe we should at some time make an effort to homogenize the behavior.
    ENDRELEASENOTES

@MarkusFrankATcernch MarkusFrankATcernch enabled auto-merge (rebase) March 12, 2025 17:18
@MarkusFrankATcernch MarkusFrankATcernch merged commit db315ad into AIDASoft:master Mar 12, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant