Skip to content

DM-6500: irsaviewer conversion #102

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

Merged
merged 2 commits into from
Jun 15, 2016
Merged

DM-6500: irsaviewer conversion #102

merged 2 commits into from
Jun 15, 2016

Conversation

loitly
Copy link
Contributor

@loitly loitly commented Jun 14, 2016

for details: https://jira.lsstcorp.org/browse/DM-6500

also made these changes:
remove irsa footer from fireflyviewer
filter by selected rows
auto-correct table’s filter input.
multiple columns sort via sortByCols
fix menu item not showing selected
ife automatically builds firefly.

To test, make sure you pull ife repos as well.
same branch name on ife.

for details: https://jira.lsstcorp.org/browse/DM-6500

also made these changes:
remove irsa footer from fireflyviewer
filter by selected rows
auto-correct table’s filter input.
multiple columns sort via sortByCols
fix menu item not showing selected
ife automatically builds firefly.
@tgoldina
Copy link
Contributor

tgoldina commented Jun 14, 2016

Some issues I've noticed when testing irsaviewer:

  1. We might want to set minimum width to accommodate image and catalog search panels. If the browser window is more narrow than their width, the layout is getting messed: Login link in the menu bar on top gets cut, search panels are hidden on the left, menu bar and image thumbnail are misaligned, etc.
    fixed: set min-width: 1030px
  2. If the first image search fails, there is no feedback, even in the console. A screen stays empty with a tiny square in the middle.
    good catch, the problem is partially fixed. it is correctly showing a plot failed image. error message can be improved, but that goes for most of our error messages.
  3. http://localhost:8080/irsaviewer/images/fftools-logo-16x16.png Failed to load resource: the server responded with a status of 404 (Not Found)
    fixed.. replaced with an IRSA's icon

Not directly related to this ticket (we might need separate tickets for these):

  • 2MASS All-Sky 'Read Me!' link in Catalog Search panel points to http://localhost:8080/irsaviewer/Q'http://hades.ipac.caltech.edu/applications/Gator/GatorAid/irsa/scan.html'E
  • If you search m31 2MASS scan info catalog (4th catalog in the list), then bring back catalog panel, the first catalog is highlighted, but if you search, you'll notice that you are still searching 2MASS scan info.
  • Both image and XY plot should handle sort action. After sorting on a column the highlight does not match the table. (Notice that when you click on an image or xy plot point, the highlight jumps on some other point.)
    fixed .. dispatch TABLE_SORT action. added action to ChartSync and CatagWatcher so image and XY plot will respond correctly
  • The catalog overlay points can appear beyond image limits. Test: search 2mass image for m31 with the default settings, then search for m31 catalog with the default settings. Notice catalog overlay squares on the right of the image.
    I reported this in ticket: https://jira.lsstcorp.org/browse/DM-6473

@ejoliet
Copy link
Contributor

ejoliet commented Jun 14, 2016

The 'Read me!' link is known issue, is a bug in the server when getting
the IRSA catalog table, on how the link of that attirbute get parsed.
Here:
in
edu.caltech.ipac.firefly.server.catquery.CatMasterTableQuery#modifyToFullURL

Please, raise ticket and fix.
E.

On 6/14/16 10:14 AM, Tatiana Goldina wrote:

Some issues I've noticed when testing irsaviewer:

  1. We might want to set minimum width to accommodate image and
    catalog search panels. If the browser window is more narrow than
    their width, the layout is getting messed: Login link in the menu
    bar on top gets cut, search panels are hidden on the left, menu
    bar and image thumbnail are misaligned, etc.
  2. If the first image search fails, there is no feedback, even in the
    console. A screen stays empty with a tiny square in the middle.
  3. http://localhost:8080/irsaviewer/images/fftools-logo-16x16.png
    Failed to load resource: the server responded with a status of 404
    (Not Found)

Not directly related to this ticket (we might need separate tickets
for these):


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#102 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AAvATGJj4PC9s9oujufai_likOG8GR-Dks5qLuFZgaJpZM4I04bC.

Emmanuel Joliet
NASA/IPAC Infrared Science Archive
MS 100-22, Caltech, Pasadena, CA 91125
Office: 170
Email: [email protected]
Phone: (626) 395-1489

@@ -310,7 +304,7 @@ export function dispatchOnAppReady(callback) {
/*---------------------------- EXPORTED FUNTIONS -----------------------------*/
export function isAppReady() {
return get(flux.getState(), [APP_DATA_PATH, 'isReady']) &&
get(flux.getState(), [APP_DATA_PATH, 'gwtLoaded']);
(get(window, 'firefly.noGWT') || get(flux.getState(), [APP_DATA_PATH, 'gwtLoaded']));
Copy link
Contributor

@tgoldina tgoldina Jun 14, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just curious where firefly.noGWT is coming from.
Because we've removed GWT completely in the latest version, I needed a flag to decide whether or not to wait for GWT's init phase.

set min-width
replaced with an IRSA's icon
fix xyplot and image overlay on table sort.
@loitly loitly merged commit 99517d5 into dev Jun 15, 2016
@loitly loitly deleted the DM-6500_irsa_viewer branch June 15, 2016 00:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants