Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-27847

Look into issues with comparing pex_config objects as equal

    XMLWordPrintable

    Details

    • Type: Story
    • Status: To Do
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: pex_config
    • Labels:

      Description

      This ticket will look into issues with determining if configs can be compared equal. In DM-27784 we found there were lots of edge cases that made this difficult. Somewhere the id of the config objects was getting used which I suspect was a mistake.

      When trying to use the toDict methods to compare for equality we ran into trouble where some of our configs may contain NaN on purpose, however NaN never equals anything including itself, so that caused config to break.

      A more robust way to determine equality should be determined and implemented.

      When this is done, the `_eq_` method on `TaskDef` in `pipe_base` should be updated to consider config equality.

        Attachments

          Issue Links

            Activity

            Hide
            krzys Krzysztof Findeisen added a comment -

            A related problem (I could swear I created an issue, but now I can't find it) is that equality of ConfigurableFields and RegistryFields is not well-behaved (IIRC they always compare unequal). Since nearly all task configs use one of these field types...

            Show
            krzys Krzysztof Findeisen added a comment - A related problem (I could swear I created an issue, but now I can't find it) is that equality of ConfigurableFields and RegistryFields is not well-behaved (IIRC they always compare unequal). Since nearly all task configs use one of these field types...

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              nlust Nate Lust
              Watchers:
              Krzysztof Findeisen, Nate Lust
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:

                  Jenkins

                  No builds found.