Skip to content
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

Plant Model page does not currently load for some reason #8

Open
cshenry opened this issue Aug 10, 2015 · 12 comments
Open

Plant Model page does not currently load for some reason #8

cshenry opened this issue Aug 10, 2015 · 12 comments

Comments

@cshenry
Copy link

cshenry commented Aug 10, 2015

I clicked on the "View model" link on a plant genome and got here:

http://140.221.67.62/#/model/plantseed/Models/Alyrata-v.1.0

But I was left with an empty page...

@nconrad
Copy link

nconrad commented Aug 10, 2015

yeah, the links use whatever is available at /plantseed/Models (http://140.221.67.62/#/data/plantseed/Models). I think Sam needs to build the models. The only model currently there is the hidden .Athaliana-TAIR10_Model .

@samseaver
Copy link
Contributor

Yes, I was testing/playing with Athaliana, I'll do the others tonight.

@samseaver
Copy link
Contributor

All PlantSEED models reconstructed and gapfilled. Model links appear to be working.

@nconrad
Copy link

nconrad commented Aug 10, 2015

The 'view model' links should be fixed and deployed. We should reduce the size of the plantseed images if we have time.

@cshenry
Copy link
Author

cshenry commented Aug 10, 2015

Hey guys,

On the PlantSEED model viewer, the gene links are broken. For PlantSEED models, the gene links need to go to our feature page� not PATRIC.

Chris

On Aug 10, 2015, at 5:39 PM, nconrad [email protected] wrote:

The 'view model' links should be fixed and deployed. We should reduce the size of the plantseed images if we have time.


Reply to this email directly or view it on GitHub.

@cshenry
Copy link
Author

cshenry commented Aug 10, 2015

Also, I understand that for public models, we can�t offer a �runfba� or �gapfill� button, but can we have a �Copy model� button? This would copy the model to my own workspace? Then I would see the model on the �My models� menu, and I could runfba or gap filling.

On Aug 10, 2015, at 5:42 PM, Christopher Henry [email protected] wrote:

Hey guys,

On the PlantSEED model viewer, the gene links are broken. For PlantSEED models, the gene links need to go to our feature page� not PATRIC.

Chris

On Aug 10, 2015, at 5:39 PM, nconrad [email protected] wrote:

The 'view model' links should be fixed and deployed. We should reduce the size of the plantseed images if we have time.


Reply to this email directly or view it on GitHub.

@cshenry
Copy link
Author

cshenry commented Aug 10, 2015

This might be nice to have on the genomes page too. Like right below the �view model� link, could I have a �copy genome� link, which would copy the genome to my workspace.

We could add �copy_model� and �copy_genome� commands to the API. What do we think?

Chris

On Aug 10, 2015, at 5:45 PM, Christopher Henry [email protected] wrote:

Also, I understand that for public models, we can�t offer a �runfba� or �gapfill� button, but can we have a �Copy model� button? This would copy the model to my own workspace? Then I would see the model on the �My models� menu, and I could runfba or gap filling.

On Aug 10, 2015, at 5:42 PM, Christopher Henry [email protected] wrote:

Hey guys,

On the PlantSEED model viewer, the gene links are broken. For PlantSEED models, the gene links need to go to our feature page� not PATRIC.

Chris

On Aug 10, 2015, at 5:39 PM, nconrad [email protected] wrote:

The 'view model' links should be fixed and deployed. We should reduce the size of the plantseed images if we have time.


Reply to this email directly or view it on GitHub.

@nconrad
Copy link

nconrad commented Aug 10, 2015

Fine with me. Since there are hidden directories involved (right?), the api calls would be good to have.

@nconrad
Copy link

nconrad commented Aug 10, 2015

oh, it'd also be nice if the copy_model can return the same structure as list_models.

@cshenry
Copy link
Author

cshenry commented Aug 11, 2015

Exactly.

On Aug 10, 2015, at 5:55 PM, nconrad [email protected] wrote:

Fine with me. Since there are hidden directories involved (right?), the api calls would be good to have.


Reply to this email directly or view it on GitHub.

@mmundy42
Copy link

Is copy_genome really needed? The genome is already being copied by ModelReconstruction().

@cshenry
Copy link
Author

cshenry commented Aug 11, 2015

It is for PlantSEED, because PlantSEED is going to permit the modification of genome annotations� thus, genomes have to be owned by users�

In the long run� I think we are going to merge the concept of genome and model together in ModelSEED/PlantSEED. For now though, they will be separate objects.

On Aug 11, 2015, at 8:11 AM, Mike Mundy [email protected] wrote:

Is copy_genome really needed? The genome is already being copied by ModelReconstruction().


Reply to this email directly or view it on GitHub.

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

No branches or pull requests

4 participants