MadGraph5_aMC@NLO

A typo in source code

Bug #2031914 reported by Leyun Gao
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
MadGraph5_aMC@NLO
Fix Committed
Undecided
Unassigned

Bug Description

For MG5_aMC@NLO v3.5.1, line 90 of bin/mg5_aMC reads:

    logging.warning("\033[91mPath does contains spaces. We advise that you change your current path to avoid to have space in the path.\033[0m")

The message should be 'Path does contain spaces...'. In addition, the term 'path' is somehow misleading, as we have the well-known environment variable PATH. The suggested expression in the place of 'path' is 'current working directory', following the semantics of the function `getcwd()` used in line 89.

Tags: typo
Revision history for this message
Olivier Mattelaer (olivier-mattelaer) wrote :

Thanks,

I have put a more clear message here:
https://github.com/mg5amcnlo/mg5amcnlo/commit/7a506593335d2205ed1ac19419ef6cf55e863a4d

Olivier

Changed in mg5amcnlo:
status: New → Fix Committed
Revision history for this message
Leyun Gao (bloud) wrote (last edit ):

Thanks, Oliver!

The info is now kind and nice. An additional typo fixing may further improve this message:

    does contains -> does contain

I also prepared a PR for your convenience if you'd like to address this revision.

    https://github.com/mg5amcnlo/mg5amcnlo/pull/66

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.