search

makeabilitylab / makeabilitylabwebsite

The Makeability Lab website
https://makeabilitylab.cs.washington.edu
MIT License
9 stars 65 forks source link

Author order messed up for pubs #578

Closed jonfroehlich closed 4 years ago

jonfroehlich commented 6 years ago

Author order is extremely important for pubs. There is a mistake in the Design of an Augmented Reality Magnification Aid for Low-Vision Users paper as it lists Leah Findlater first. Instead, Lee Stearns should hold that position. I believe this mistake was automatically introduced when I edited the video field for the paper (previously, the author order was showing up correctly). However, I can't change it in the admin interface (perhaps because of the admin CSS problem?).

image

image

But this does bring to mind, how are we currently tracking order position of authors? How did the author order get switched on this paper when I went to update the video link?

aileenzeng commented 6 years ago

Author order depends on the order that we add authors in -- I believe the first author that gets added becomes first author on the paper (may be the other way around, not sure). I'm not sure how the author order got switched when we updated videos though.

aileenzeng commented 6 years ago

We have a few other that are off (check mark = fixed):

Current author orders on these seem to be alphabetical by last name.

jonfroehlich commented 6 years ago

Thanks Aileen. Imo, this is the highest priority bug that needs to be fixed. Do we need to add an explicit index integer to track author order?

@johnsonkuang, can you first investigate why this is happening and how author order is currently tracked before proposing a solution?

jonfroehlich commented 6 years ago

Shouldn't we be using the https://github.com/gregmuellegger/django-sortedm2m type for this? I see that it's already in our requirements.txt file.

@johnsonkuang, can you track down where we use sortedm2m? Seems like this is the perfect place to use it.

Also, I'd like a report on what you found regarding the multi-select widget and the order returned from that on a save...

johnsonkuang commented 6 years ago

PR changed under #585

aileenzeng commented 6 years ago

I'm starting to manually fix some of the pubs, but sometimes run into Server 500 errors:

I looked at the debug.log and am getting messages like this:

wand.exceptions.BlobError: unable to open image `_Real-Time_Human_Activity_Classification_on_Commodity_Mobile_Phones_2008.pdf': No such file or directory @ error/blob.c/OpenBlob/2712
ERROR 2018-09-05 23:45:34,005 basehttp 131 140618571233024 "POST /admin/website/publication/546/change/ HTTP/1.1" 500 27

Maybe this is related to some of the database changes from today? Not sure.

jonfroehlich commented 6 years ago

Can you make a brand new publication entry?

Wand provides some image handling for us (e.g., helps with auto-thumbnail generation from pds)--if you're just editing metadata for a pub (and not adding or changing the pdf), not sure why it would even need to execute.

On Wed, Sep 5, 2018 at 8:51 PM Aileen Zeng notifications@github.com wrote:

I'm starting to manually fix some of the pubs, but sometimes run into Server 500 errors:

  • "TacTILE: A Preliminary Toolchain For Creating Accessible Graphics With 3D-Printed Overlays And Auditory Annotations" (ID: 572)
  • "Recognizing Clothing Colors and Visual Textures Using a Finger-Mounted Camera: An Initial Investigation" (ID: 560)
  • "Designing 3D-Printed Deformation Behaviors Using Spring-Based Structures: An Initial Investigation" (ID: 574)
  • "Exploring Novice Approaches to Smartphone-based Thermographic Energy Auditing: A Field Study" (ID: 553)
  • "iLearn on the iPhone: Real-Time Human Activity Classification on Commodity Mobile Phones" (ID: 546).

I looked at the debug.log and am getting messages like this:

wand.exceptions.BlobError: unable to open image `_Real-Time_Human_Activity_Classification_on_Commodity_Mobile_Phones_2008.pdf': No such file or directory @ error/blob.c/OpenBlob/2712 ERROR 2018-09-05 23:45:34,005 basehttp 131 140618571233024 "POST /admin/website/publication/546/change/ HTTP/1.1" 500 27

Maybe this is related to some of the database changes from today? Not sure.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/jonfroehlich/makeabilitylabwebsite/issues/578#issuecomment-418955901, or mute the thread https://github.com/notifications/unsubscribe-auth/ABi-9YZDu2TBkVNnFRQqUzkYR9oHfqgjks5uYJumgaJpZM4WCUAi .

-- Jon Froehlich Associate Professor Paul G. Allen School of Computer Science & Engineering University of Washington http://www.cs.umd.edu/~jonf/ http://makeabilitylab.io @jonfroehlich https://twitter.com/jonfroehlich - Twitter

aileenzeng commented 6 years ago

@jonfroehlich Looks like the PDFs for the first two don't exist. They're not in the media folder on the server either.

I also tried to add Designing 3D-Printed Deformation Behaviors Using Spring-Based Structures: An Initial Investigation and it returned a Server 500 error (when I tried to add it as a new pub). I also deleted the existing version (before the new one got added) eeeeek :( Probably will hold off on adding new pubs for now + will create a new ticket.

jonfroehlich commented 6 years ago

I was able to fix all but iLearn on the iPhone: Real-Time Human Activity Classification on Commodity Mobile Phones. I think we might need to re-upload the PDF for this...

jonfroehlich commented 4 years ago

Looks fixed now.