You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug fre pp install installs a configured Cylc workflow (e.g. in ~/cylc-src). If an installed workflow by that name already exists, then it checks that the configuration is identical between the two. If not, error. (We can actually update the running workflow, but let's save that for next month).
The problem is that the check fails when the two configurations are actually identical.
Describe the bug
fre pp install
installs a configured Cylc workflow (e.g. in ~/cylc-src). If an installed workflow by that name already exists, then it checks that the configuration is identical between the two. If not, error. (We can actually update the running workflow, but let's save that for next month).The problem is that the check fails when the two configurations are actually identical.
@singhd789 found that the comparison done now (https://github.com/NOAA-GFDL/fre-cli/blob/main/fre/pp/install_script.py#L27) is comparing
bytes
when it needs to comparestrings
.To convert bytes to string, one can use
decode('utf-8')
.To Reproduce
Problem is intermittent.
Expected behavior
String comparison is desired and expected.
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: