search

CellProfiler / cellprofiler.github.com

CellProfiler Website
2 stars 4 forks source link

TranslocationData file needs redirect #8

Open N3llz opened 8 years ago

N3llz commented 8 years ago

http://cellprofiler.org/linked_files/TranslocationData.zip needs a redirect (to http://d1zymp9ayga15t.cloudfront.net/TranslocationActivity/TranslocationData.zip)

N3llz commented 8 years ago

original convo:

Jeanelle Ackerman 17 minutes ago Regarding the redirect, the only place where the Translocation.zip file seems to exist on the website is on the /outreach page. I fixed that link. Is there a reason/situation where people would be accessing it a different way? In the old website script, this path is wrong (that file isn't in /linked_files) So 1) This makes me wonder how long it's been like that and if that pathway/url is irrelevant now. 2) Do you think it's ok as long as the new link works? You can see it by clicking the .zip in the "Image-based screening for quantifying a translocation assay written exercise" Materials section on the outreach page.

Anne Carpenter 15 minutes ago Indeed, the problem is that the link exists in the outside world. Whenever we make a major change to the website, we want to assess any URLs that existed in the old and create redirects to something sensible in the new site, where possible. Does that make sense?

Anne Carpenter 14 minutes ago I don't actually know how to quickly figure out which URLs used to exist for an old site vs new.

N3llz commented 8 years ago

Unfortunately, we need to use Jekyll before we can use redirects. I'll create an issue.

N3llz commented 8 years ago

@mcquin Anne informed me this still does need to be addressed because although the link wasn't in the old website, it still exists from tutorials, so I reopened the issue

mcquin commented 8 years ago

The only thing we can do about this is update the links in the tutorials. OR move the file off CloudFront and back onto cellprofiler.org.

N3llz commented 8 years ago

@AnneCarpenter will it be sufficient to update the links in the tutorials? See claire's comment above.

AnneCarpenter commented 8 years ago

Sadly, they exist out there in the universe so we don't have any control over it. Sounds like adding the file to cellprofiler.org instead of CloudFront is a reasonable solution unless it's painful in some way @mcquin ?

mcquin commented 8 years ago

GitHub repositories are encouraged to not exceed 1GB in size. The current size of the website is ~0.014GB. The size of the file is ~0.010GB. I think we should be fine to host this.

@AnneCarpenter do you happen to know if there are any other similar files missing?

AnneCarpenter commented 8 years ago

I see. Indeed, it sounds not too harmful to place this file in Github - but you're right to question whether it's one of many before doing so!

I wish I knew if there were any others missing. I'd hoped we'd have a listing of all files that used to reside at cellprofiler.org in order to check for any others. @N3llz is that going to be possible?

N3llz commented 8 years ago

Well, what's odd is that this file wasn't residing in the old website - at least not in that path - so that's where it gets tricky because I wouldn't know where to even search for it, other than the tutorials you've mentioned? I'm not sure how we would even go about searching through tutorials for mysterious links like this one. @AnneCarpenter

AnneCarpenter commented 8 years ago

Ah! Well, that explains how it got missed in your process then! I have a copy, will transfer.

N3llz commented 8 years ago

@AnneCarpenter I have the file, but the path "http://cellprofiler.org/linked_files/TranslocationData.zip" will not work if people are trying to use that from some time ago (must have been even before the last version of the website). And like Claire said, we don't know if there are more like this (that would have also been missed in the transfer to the new site). @mcquin we should be fine to just change the link for this file to direct people to its Github location once we host it correct? is that what you're thinking?

mcquin commented 8 years ago

@AnneCarpenter The link has been broken for a long time (since February 9 2016, or before). Either nobody else has noticed or they're silent or they came back here and got the correct link. I'm not sure if restoring this link is essential, though we can do it.

@N3llz I suggest leaving the CloudFront link for this file as it, so new users are taken to the correct location. We can place the file also at http://cellprofiler.org/linked_files/TranslocationData.zip so that anyone trying to access that link directly can get the file.