Skip to content

now automatically makes table given stats data#65

Open
gimlin123 wants to merge 3 commits into
masterfrom
tablemaker
Open

now automatically makes table given stats data#65
gimlin123 wants to merge 3 commits into
masterfrom
tablemaker

Conversation

@gimlin123
Copy link
Copy Markdown

No description provided.

Comment thread app/modules/analysis/manual.ctrl.coffee Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

don't keep console statements in

@GuruM
Copy link
Copy Markdown
Contributor

GuruM commented Aug 12, 2015

Could you post a screenshot of what this looks like?

Comment thread app/modules/analysis/manual.jade Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

hmm I'm not entirely sure that the mergecells parameter is being read correctly, since you didn't define a sortindex or sortorder before it

@GuruM
Copy link
Copy Markdown
Contributor

GuruM commented Aug 12, 2015

@kevinzenghu you'll probably want to double check this

@gimlin123
Copy link
Copy Markdown
Author

image

Comment thread app/modules/analysis/manual.jade Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

we should probably get rid of this output

Comment thread app/modules/analysis/manual.ctrl.coffee Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

forgot to remove this one

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.

2 participants