Skip to content
This repository has been archived by the owner on Oct 21, 2024. It is now read-only.

i_case = 32: Format matching 3d.dat #16

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

nbasse
Copy link

@nbasse nbasse commented Mar 9, 2020

Fixed read format to match the standard format of the output in 3d.dat. (As it is supposed to be according to #11 (comment))

  • Corrected number of lines to skip to match the number of lines of settings in 3d.dat
  • added extra dummy variable to handle text between variables
  • edited FORMAT specifier to match the appropriate numbers.

Fixed read format to match the standard format of the output in 3d.dat. (As it is supposed to be according to LHEEA#11 (comment))
- Corrected number of lines to skip to match the number of lines of settings in 3d.dat
- added extra dummy variable to handle text between variables
- edited FORMAT specifier to match the appropriate numbers.
i_case = 32: Format matching 3d.dat
@nbasse
Copy link
Author

nbasse commented Mar 9, 2020

This is my first ever pull request, so not sure how to follow the guidelines in the wiki with the extra lines. (all changes made through the GUI in the browser)
Change-Id: ?
Signed-off-by: Niclas Bassili, [email protected]

@gducrozet
Copy link
Member

@nbasse : it is probably not possible to add the Change-Id line with the GUI in the browser.
By the way, this is not a big issue since I am usually doing the integration myself, in order to maintain both the GitHub version and our local ECN version (that includes other features).

By the way, thanks for the contribution.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants