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

Sample data generation and table display #15

Closed
wants to merge 2 commits into from

Conversation

IshaGupta18
Copy link
Collaborator

@IshaGupta18 IshaGupta18 commented May 9, 2019

fixes #16
Okay so I have written the functions for extracting the sample data and generating the tables for selection of x-axis column and y-axis columns dynamically.

sample

Next step will be to compile the selected columns and send it to chart.js for plotting.

Let me know how this looks @jywarren @namangupta01

@namangupta01
Copy link
Member

Have opened an issue #14, have a look at it.

@namangupta01
Copy link
Member

This looks good.

@namangupta01
Copy link
Member

Why not x-axis can have multiple selections?

@IshaGupta18
Copy link
Collaborator Author

IshaGupta18 commented May 9, 2019 via email

@namangupta01
Copy link
Member

What about this type of graph?
Screen Shot 2019-05-09 at 8 18 18 PM

@namangupta01
Copy link
Member

But I think this will be more complex to do. And also this will consider another axis.

@namangupta01
Copy link
Member

Because we are not introducing any z-axis as of now.

@IshaGupta18
Copy link
Collaborator Author

Yes it looks more like a 3 D graph right?

@IshaGupta18
Copy link
Collaborator Author

yes, because the kind of data we may have to plot just involve the 2 axes only.

@namangupta01
Copy link
Member

So you are now working on this..so I have started working on selected columns and send it to chart.js for plotting part

@namangupta01
Copy link
Member

namangupta01 commented May 9, 2019 via email

@IshaGupta18
Copy link
Collaborator Author

so let me just give you the collected data in a has kind of form, or will you do that? I can explain you the code a little to help you compile this data to a form that can be easily given to chart.js.

@IshaGupta18
Copy link
Collaborator Author

Also, I can also help you with the chart.js part because I will also like to explore that library a little. We can work on it together, like you can pass the data and I can work on the design part and graph selection

@namangupta01
Copy link
Member

I think I can look, If I needing some help I will let you know. I will also be making modules and dividing the parsing.js into multiple modules.

@IshaGupta18
Copy link
Collaborator Author

Okay, no problem. Let me know once you have passed the data to chart.js, I'll start with remaining work for chart.js

@namangupta01
Copy link
Member

Pass the data? Isn't is it somewhat already done here. Because you are printing the data in the console so I think the passing of data part is done? No?

@namangupta01
Copy link
Member

namangupta01 commented May 9, 2019

We already have the selected data ? Right?

@IshaGupta18
Copy link
Collaborator Author

We have the data but not in the compiled form for chart.js. You'll have to make it in a hash like format and give it to chart.js. See here:
image
image

@IshaGupta18 IshaGupta18 mentioned this pull request May 9, 2019
@namangupta01
Copy link
Member

Okay! I have a doubt in mind. Maybe @jywarren can show some light on it. We can use multiple charts on a single page, right?

@IshaGupta18
Copy link
Collaborator Author

IshaGupta18 commented May 9, 2019 via email

@namangupta01
Copy link
Member

I looked at it. Its multiple charts from the same CSV file. But I was talking about charts from different csv sheets. Because as of now we are making the system very rigid, or not scalable -- multiple graphs from multiple CSV files, and we are making changes in the update_file.html file where we should not do this we should only use this for testing. Library should be such that after including only one script in any of the projects we should call a simple function and all this stuff should happen itself -- this is what happens in the standard libraries as far as I know. User can do additional styling if he wants to do but that is not necessary, all the default styling should come only by calling a function. And also we should divide our code in different module in order to make code modularity.
Let me know your views @jywarren and @IshaGupta18.

@jywarren
Copy link
Member

jywarren commented May 9, 2019 via email

@namangupta01
Copy link
Member

Yes, by default simple input and simple output. Just like other libraries, for example: Chart.js, which provides default output when we pass no additional parameters in functional calling but we have an option to do. It would be better to discuss before, what type of approach and system we want in future.

@IshaGupta18
Copy link
Collaborator Author

Yes it will be a good idea in the future, but we'll have to look for efficient ways to implement it, because the implementation might not be very easy. Right now, my approach is to make the system as flexible as it can be, for this point of time. About integration with plots2, we should look at how the Editor has been integrated, because it works very well and is neat. So, yes, we'll have to discuss it at length, on the best way to do this

@namangupta01
Copy link
Member

One minor doubt, related to this. Asked in #18 (comment)

@namangupta01
Copy link
Member

namangupta01 commented May 10, 2019

Closing this because already doing in #18

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Collection of sample data and table display
3 participants