Design Explorer 2 by MingboPeng · Pull Request #74 · tt-acm/DesignExplorer

MingboPeng · 2016-06-22T04:16:55Z

Added google Link, and full screen charts.
New scatter plots chart.

load data from google sheet and folder

overwrite the rest files that may have changes from DE1

Fix google link input css

user can upload csv file after used google sheet without reload the page.

added acatter matrix source from benjiec

New scatter charts, and set charts full screen.

added button to set parallel chart full size, but there is a bug of missing brush after resize. I need to fix it next step.

It seems has a bug that cannot resize the height of left chart, need to compare with the previous parcoords.js version

All charts full screen basically working now, but still there are a lot of minor bugs left.

Left chart fullscreen

…tching full screen mode 1. fixed: zoomed area coordinates with the parallel chart when it is switching full screen mode 2. fixed: clean the pre-parameters before load new data 3. beautify the general code structures. 4. correct codes' miner typos

the dots size, opacity

in parcoords.js the function scale seems run before autoscale, so that the value will be overwrite by autoscale.. might need to double check later.

fixed thumbnails-btm_container's height when isLeftChartFullScreenToggled

mostaphaRoudsari · 2016-06-29T01:50:02Z

@MingboPeng thank you for the pull request. I know that I asked you to send a pull request to this repository but at the same time I'm not sure who is going to make decisions about the future of DesignExplore at this point. Let me email @bhowes-tt and @JonatanS to see what will be the best approach and I will get back to you.

mostaphaRoudsari · 2016-06-30T20:25:52Z

@MingboPeng, @bhowes-tt will take care of this PR. Also make sure that you guys will have lunch when you start working at TT soon. Thanks again for doing this 💯

bhowes-tt · 2016-06-30T20:30:50Z

@MingboPeng - thank you so much for this. Awesome work! ⛵

I'm reviewing / testing now, and will likely come back with a few small items for you before we merge this pull. I'm excited to meet you! Be sure to come find us when you get started at TT.

More to follow - stay tuned.

bhowes-tt · 2016-07-01T20:50:00Z

@MingboPeng

I was able to get my own example running using your new build, and google drive to host and serve the images and the data ... AMAZING WORK ⛵ 🐙 🍋 !!!

There are a few issues that need your attention before we'll merge the pull, however:

The major issue is the 3D viewer / links to spectacles.json files. I added a 'threeD' column to my spreadsheet, and uploaded .json files exported from Grasshopper using Spectacles. I figured the 3D content would/should work just like the image content, but it looks like your current build is looking for the .json files on the web server, not in the google drive folder. Also, it looks like you are getting a vA3C / Spectacles error:

Please re-enable the 3D viewer, and let Spectacles.json files work just like images do in the google folder/spreadsheet.

Sliders are still there and working when an old sample data set is loaded (like the one that loads by default), but don't show up in my example. Is this intentional or a bug? Please explain... Maybe I'm missing something...
If the parallel coordinates UI is hidden with the little down arrow, it can never be brought back. This is working in our old build. Please fix this if you can (this is by far the lowest priority issue on this list).
Logo / Splash page. Let's keep the name of the project the same - 'Design Explorer', not 'Design Explorer 2' - and revert back to the old logo. I'd also ask that we omit some of the content on the splash view - see the image below. You can certainly keep all of that content on your fork if you'd like to, but I don't think it makes sense as part of a pull request.
Scatterplots. I do prefer the scatterplots over the radar chart... But, if I check anything in the horizontal list, the plots disappear - it looks like it's just using [2 or more] vertical variables, which feels like a bug. Please have a look and either let us specify both horizontal and vertical (which we'd prefer), or just provide a single list. Also, please refer to the library you are using in the .README file.

Please let me know if you have any questions about any of these issues, and thank you so much for the great work. I'm super excited about the pull... very well done.

MingboPeng · 2016-07-10T23:57:37Z

Hi Ben,

It seems cannot reply this by email!

Thanks for testing out.

The google drive folder was built for images originally, but I have found the method to load json from google drive file as well. Working on it.
The slider problem is because you didn’t put “in:” or “out:” in the csv or google. I also have a solution for it.
This is strange, I will fix it.
Sure, the logo and other staffs were used in the one hosted in a MEBD related server (for UPenn student use purpose), and definitely happy to remove some unnecessary content.
This is Horizontal axis only works while checked item is also checked in the Vertical axis. I know this is weird, because of the way used in original library. I think the easiest solution might be remove all horizontal items, and change vertical axis to the single list.

Thanks again.
Mingbo

Fixed bugs: 1. Hide/Show Parallel coordinate 2. slider's img and 3d link problem

Added Scatter-matrix Chart reference

bhowes-tt · 2016-07-11T13:53:54Z

Excellent. I'll give the new build another test today...

bhowes-tt · 2016-07-11T19:28:38Z

@MingboPeng - Looking good! I like the new loading sequence.

no luck today though, maybe you are still in process, or I'm doing something wrong. I'm getting a http request error after pasting my folder link:

I tried uploading data.csv, and creating and publishing a sheet called data.csv - both produced the same error.

MingboPeng · 2016-07-11T22:44:01Z

@bhowes-tt because google drive now needs different settings than before, I will make a tutorial soon.

Supported Google Drive and server link, and clean the code

MingboPeng · 2016-07-12T14:16:34Z

@bhowes-tt here is a tutorial demo

bhowes-tt · 2016-07-29T03:40:16Z

@MingboPeng - sorry for the long radio silence. I finally got around to testing your latest version, and we are good to go! ⛵ 🐙 ⛵

Thank you again for the really great work. We're going to publish a blog post about this project soon, and I'd like to get some quotes from both you and @mostaphaRoudsari .

MingboPeng · 2016-07-29T15:40:51Z

@bhowes-tt Thanks so much for your testing. Now I am working on a new tutorial of how to prepare the data from Grasshopper for Design Explorer. Trying to make it simple for everyone to use.
Thanks again,
Mingbo

MingboPeng added 15 commits June 2, 2016 23:00

connect Google

2529214

load data from google sheet and folder

overwrite the rest files that may have changes

b79eaeb

overwrite the rest files that may have changes from DE1

Fix google link input css

d8dacc5

Fix google link input css

Fix bugs of uploading csv after used google sheets

eea7a0c

user can upload csv file after used google sheet without reload the page.

added acatter matrix source

45deeb6

added acatter matrix source from benjiec

Added new Scatter Chart on right side

8b6e4fa

New scatter charts, and set charts full screen.

parallel coordinates full size available

b0a3f05

added button to set parallel chart full size, but there is a bug of missing brush after resize. I need to fix it next step.

left chart full screen tested with new parcoords.js

38cfe08

It seems has a bug that cannot resize the height of left chart, need to compare with the previous parcoords.js version

fixed the most major functional bugs when using new parcoords js.

7f63f8f

All charts full screen basically working now, but still there are a lot of minor bugs left.

Merge pull request #1 from MingboPeng/left_chart_fullscreen

51a91a5

Left chart fullscreen

fixed some workflow presenting issues

bee0abf

Improve the scatter chart presentation

ebeefb2

the dots size, opacity

fixed the rating scale problem

0ca49a9

in parcoords.js the function scale seems run before autoscale, so that the value will be overwrite by autoscale.. might need to double check later.

fixed thumbnails-btm_container's height

06c6030

fixed thumbnails-btm_container's height when isLeftChartFullScreenToggled

mostaphaRoudsari assigned bhowes-tt Jun 30, 2016

mostaphaRoudsari added enhancement UI/UX labels Jun 30, 2016

MingboPeng added 2 commits July 10, 2016 23:45

Load Json from Google Drive and fixed bugs

d910c1b

Fixed bugs: 1. Hide/Show Parallel coordinate 2. slider's img and 3d link problem

Added Scatter-matrix Chart reference

0de9bc2

Added Scatter-matrix Chart reference

Supported Google Drive and server link, and clean the code

9da48b0

Supported Google Drive and server link, and clean the code

bhowes-tt merged commit 4b57ede into tt-acm:gh-pages Jul 29, 2016

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Design Explorer 2#74

Design Explorer 2#74
bhowes-tt merged 18 commits intott-acm:gh-pagesfrom
MingboPeng:gh-pages

MingboPeng commented Jun 22, 2016

Uh oh!

mostaphaRoudsari commented Jun 29, 2016

Uh oh!

mostaphaRoudsari commented Jun 30, 2016

Uh oh!

bhowes-tt commented Jun 30, 2016

Uh oh!

bhowes-tt commented Jul 1, 2016

Uh oh!

MingboPeng commented Jul 10, 2016

Uh oh!

bhowes-tt commented Jul 11, 2016

Uh oh!

bhowes-tt commented Jul 11, 2016

Uh oh!

MingboPeng commented Jul 11, 2016

Uh oh!

MingboPeng commented Jul 12, 2016

Uh oh!

bhowes-tt commented Jul 29, 2016

Uh oh!

MingboPeng commented Jul 29, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

MingboPeng commented Jun 22, 2016

Uh oh!

mostaphaRoudsari commented Jun 29, 2016

Uh oh!

mostaphaRoudsari commented Jun 30, 2016

Uh oh!

bhowes-tt commented Jun 30, 2016

Uh oh!

bhowes-tt commented Jul 1, 2016

Uh oh!

MingboPeng commented Jul 10, 2016

Uh oh!

bhowes-tt commented Jul 11, 2016

Uh oh!

bhowes-tt commented Jul 11, 2016

Uh oh!

MingboPeng commented Jul 11, 2016

Uh oh!

MingboPeng commented Jul 12, 2016

Uh oh!

bhowes-tt commented Jul 29, 2016

Uh oh!

MingboPeng commented Jul 29, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants