Skip to content

Conversation

PivovarA
Copy link
Contributor

  • Add frameworks list
  • Add algorithms list
  • Add algorithms parameters
  • Add description of configuration file

@PivovarA PivovarA requested a review from KalyanovD June 22, 2020 12:58
@teabagk7
Copy link

Great work!👍

remove Config JSON Schema paragraph
change main description of repository
apply minor text changes
Copy link
Contributor

@outoftardis outoftardis left a comment

Choose a reason for hiding this comment

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

Note: I haven't reviewed everything.

What you have to change:

  • make sure that the first word in each sell in each table starts with a capital letter (this does not apply to the cells that start with parameter names, type names, and similar info)
  • if you have a full sentence in a table sell, make sure it and everything else in the same column ends with the full stop .
  • when you describe a parameter, instead of "Number of" you have to say "The number of", instead of "Path to" -- "The path to"

PivovarA and others added 3 commits July 16, 2020 11:57
Co-authored-by: Ekaterina Mekhnetsova <outoftardis@users.noreply.github.com>
Co-authored-by: Ekaterina Mekhnetsova <outoftardis@users.noreply.github.com>
Apply all suggestions

Co-authored-by: Ekaterina Mekhnetsova <outoftardis@users.noreply.github.com>
@PivovarA PivovarA merged commit 04ddc78 into IntelPython:master Jul 16, 2020
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