Skip to content

editing SVM analysis example #88

Closed
akshay326 wants to merge 5 commits intomasterfrom
fix_ex_typo
Closed

editing SVM analysis example #88
akshay326 wants to merge 5 commits intomasterfrom
fix_ex_typo

Conversation

@akshay326
Copy link
Copy Markdown
Collaborator

using JuMP instead of MOI (less code, more understandable)
fixed a minor typo

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 21, 2021

Codecov Report

Merging #88 (8d4b882) into master (a2ab035) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #88   +/-   ##
=======================================
  Coverage   81.20%   81.20%           
=======================================
  Files           6        6           
  Lines         782      782           
=======================================
  Hits          635      635           
  Misses        147      147           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a2ab035...8d4b882. Read the comment docs.

@akshay326
Copy link
Copy Markdown
Collaborator Author

@joaquimg any clue why is this failing? the script unit_example.jl was working earlier i guess

@matbesancon matbesancon reopened this Apr 22, 2021
@matbesancon
Copy link
Copy Markdown
Collaborator

just re-running to see if the error on nightly is still there

@matbesancon
Copy link
Copy Markdown
Collaborator

matbesancon commented Apr 29, 2021

@be-apt when using julia in quote blocks in documentation, it will just create a colorized snippet. What we want is actually running the code, that's why the blocks are named, like in example 1

@matbesancon matbesancon closed this May 5, 2021
@matbesancon matbesancon reopened this May 5, 2021
@akshay326
Copy link
Copy Markdown
Collaborator Author

i guess we can drop example 1 vs julia thing once #93 is closed

@matbesancon
Copy link
Copy Markdown
Collaborator

@be-apt for now let's keep the example 1 blocks to have the examples built, we can move to Literate later

@matbesancon
Copy link
Copy Markdown
Collaborator

example failing here

@akshay326
Copy link
Copy Markdown
Collaborator Author

closing this. just a duplicate of #95

@akshay326 akshay326 closed this Jul 14, 2021
@akshay326 akshay326 deleted the fix_ex_typo branch July 14, 2021 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants