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

Web Interface Mockup #224

Closed
wants to merge 13 commits into from
Closed

Conversation

mavidser
Copy link

@mavidser mavidser commented Mar 4, 2015

This is a basic mockup of a web-interface for generating YAML config files.

The main purpose is to identify the appropriate input methods for each field, verify the inputs which are conditional (the inputs with the type container-property), and figure out an easy way to gather the user inputs.

A live version of the mockup is available here.

The current version of the web app is available here.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling cc302b6 on mavidser:web-interface into 0931f4c on tardis-sn:master.

@wkerzendorf
Copy link
Member

@mavidser it looks nice, but is not in python. We'd prefer something like this, but in tornado or flask

@wkerzendorf
Copy link
Member

in the end it would be great if it can read the schema file and autogenerate this.

@mavidser
Copy link
Author

mavidser commented Mar 4, 2015

Yes, it's kinda like the expected output for the schema I hope to implement.
I'll try to create a schema in Python to generate a similar interface .

@landscape-bot
Copy link

Code Health
Repository health decreased by 2% when pulling 0daa5d4 on mavidser:web-interface into 0931f4c on tardis-sn:master.

@landscape-bot
Copy link

Code Health
Repository health increased by 0.07% when pulling bd13bf3 on mavidser:web-interface into 0931f4c on tardis-sn:master.

@landscape-bot
Copy link

Code Health
Repository health increased by 0.61% when pulling e11d3fe on mavidser:web-interface into 0931f4c on tardis-sn:master.

@mavidser
Copy link
Author

mavidser commented Mar 6, 2015

Hi.
In the latest commit, I tried to generate the form using the YAML Schema given here.

The form isn't complete yet, nor is it full generalized, and I still have to parse container-property (hence, Model and MonteCarlo hasn't been implemented), but I am facing some problems understanding its properties (see #239).

But the basic interface works, and a form is successfully being generated from the schema.

You can check out the form currently generated here.

@mavidser
Copy link
Author

@wkerzendorf @orbitfold

I think the form works fairly good for now, given a few kinks, which I'll sort out in further updates. (Preview). Please take a look and see if there's something I'm doing wrong, or any suggestions you might have regarding the form.

Also, on a sidenote, how should one proceed after doing the prerequisite PRs? Should we continue on maintaining/improving the code or should we start building our proposals?

@wkerzendorf
Copy link
Member

@mklauser is one of our mentors and so I'm messaging him here as well.

@@ -0,0 +1,92 @@
import json
Copy link
Member

Choose a reason for hiding this comment

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

nice file name 😉

Copy link
Author

Choose a reason for hiding this comment

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

Just following the pattern. 😄

@wkerzendorf
Copy link
Member

Can you rebase onto the newest master. It looks very nice! Please also start writing up the application.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 55.1% when pulling d000c49 on mavidser:web-interface into dca3806 on tardis-sn:master.

@mavidser
Copy link
Author

It's done. The code's now up-to-date with the latest master. I'll start with the application now too. Thanks. 😄

@landscape-bot
Copy link

Code Health
Repository health increased by 0.22% when pulling d000c49 on mavidser:web-interface into dca3806 on tardis-sn:master.

@landscape-bot
Copy link

Code Health
Repository health increased by 0.22% when pulling d000c49 on mavidser:web-interface into dca3806 on tardis-sn:master.

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.

4 participants