Skip to content

Notes from pull request 33: #40

@ondrejkrejci

Description

@ondrejkrejci

pyPPSTM/ReadSTM.py
return eig;

note 1:

204 - def cut_eigenenergies(eig):
206 + def cut_eigenenergies(eig, cut_min, cut_max):

Member
@ondrejkrejci
ondrejkrejci
on Mar 4, 2025
While, I am definitely for this one, I do not see, that the you would remove the global definitions at line 102 and 103. Would it be possible to remove them? If not, please add a comment so they should be removed in the future.

note 2:

ppstm_run.py

81 + if name=='main':
82 + # Get config file from command line
83 + config_file = sys.argv[1]

Member
@ondrejkrejci
ondrejkrejci
on Mar 4, 2025
Could you please add here a helper message, if there is no argv, or more than 1, thank you!

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions