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

Update Python API for show_xyplot

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: SUIT
    • Labels:
    • Story Points:
      4
    • Epic Link:
    • Sprint:
      SUIT Sprint 2017-2
    • Team:
      Science User Interface

      Description

      show_xyplot in Firefly python API is outdated. Implement it using chart actions.

      To test, put the attached notebook into firefly/src/firefly/test/python directory and run it.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          tatianag Tatiana Goldina added a comment -

          David, please advice me if somethings can be done more efficiently.

          Pull request: https://github.com/Caltech-IPAC/firefly_client/pull/5

          Show
          tatianag Tatiana Goldina added a comment - David, please advice me if somethings can be done more efficiently. Pull request: https://github.com/Caltech-IPAC/firefly_client/pull/5
          Hide
          Permalink
          shupe David Shupe added a comment -

          I tested show_xyplot and fetch_table and they work as advertised. Very nice!

          Detailed comments are in the pull request. We just want to use **chart_params in the function signature and docstrings, so that users can pass in keyword arguments instead of a dictionary.

          Show
          shupe David Shupe added a comment - I tested show_xyplot and fetch_table and they work as advertised. Very nice! Detailed comments are in the pull request. We just want to use **chart_params in the function signature and docstrings, so that users can pass in keyword arguments instead of a dictionary.

            People

            Assignee:
            tatianag Tatiana Goldina
            Reporter:
            tatianag Tatiana Goldina
            Reviewers:
            David Shupe
            Watchers:
            David Shupe, Tatiana Goldina
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.