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

fix syntax errors in logging/error messages in summary.py #425

Merged
merged 1 commit into from
Oct 19, 2022
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 10 additions & 10 deletions ecl2df/summary.py
Original file line number Diff line number Diff line change
Expand Up @@ -461,10 +461,10 @@ def _ensure_unique_datetime_index(dframe: pd.DataFrame) -> pd.DataFrame:

if "TIMESTEP" in dframe:
logger.info(
"Dataframe of smry data contained duplicate timestamps due to limited",
" output resolution. Vector TIMESTEP exists, utilizing it to create "
"discrete timestamps",
f"Original duplicates were:{index_duplicate_log_string}",
"Dataframe of summary data contained duplicate timestamps due to "
"limited output resolution. Vector TIMESTEP exists, utilizing it to "
"create discrete timestamps."
f"Original duplicates were:{index_duplicate_log_string}"
)
index_as_list = dframe.index.to_list()

Expand All @@ -480,16 +480,16 @@ def _ensure_unique_datetime_index(dframe: pd.DataFrame) -> pd.DataFrame:
)
else:
raise ValueError(
"Dataframe of smry data contained duplicate timestamps",
"Vector TIMESTEP exists, but unit could not be identified",
"Dataframe of smry data contained duplicate timestamps. "
"Vector TIMESTEP exists, but unit could not be identified."
)
dframe.index = pd.Series(data=index_as_list, name=dframe.index.name)
else:
raise ValueError(
"Dataframe of smry data contained duplicate timestamps due to limited.",
" output resolution. Vector TIMESTEP was not found. Try to add it to ",
"the SUMMARY section of the simulation deck, as it may be utilized to",
" separate duplicate timestamps.",
"Dataframe of summary data contained duplicate timestamps due to "
"limited output resolution. Vector TIMESTEP was not found. Try to add "
"it to the SUMMARY section of the simulation deck, as it may be "
"utilized to separate duplicate timestamps."
)
return dframe

Expand Down