Skip to content

Add a warning about running MLflow in windows#60

Merged
mateiz merged 2 commits intomlflow:masterfrom
andrewmchen:windows-warning
Jun 20, 2018
Merged

Add a warning about running MLflow in windows#60
mateiz merged 2 commits intomlflow:masterfrom
andrewmchen:windows-warning

Conversation

@andrewmchen
Copy link
Contributor

@andrewmchen andrewmchen commented Jun 18, 2018

screen shot 2018-06-18 at 4 37 51 pm

@mateiz
Copy link
Contributor

mateiz commented Jun 18, 2018

I'd say we do not support it yet, not that we haven't tested. People have tested it and it doesn't quite work :).

@codecov-io
Copy link

codecov-io commented Jun 18, 2018

Codecov Report

Merging #60 into master will increase coverage by 0.09%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #60      +/-   ##
==========================================
+ Coverage   77.11%   77.21%   +0.09%     
==========================================
  Files          44       45       +1     
  Lines        2543     2611      +68     
==========================================
+ Hits         1961     2016      +55     
- Misses        582      595      +13
Impacted Files Coverage Δ
mlflow/server/__init__.py 65.21% <0%> (-16.04%) ⬇️
mlflow/tracking/__init__.py 82.22% <0%> (-0.56%) ⬇️
mlflow/store/file_store.py 90.95% <0%> (-0.48%) ⬇️
mlflow/tensorflow.py 98% <0%> (ø)
mlflow/server/handlers.py 27.9% <0%> (+0.02%) ⬆️
mlflow/cli.py 84.5% <0%> (+0.92%) ⬆️

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 2c1f181...d091a48. Read the comment docs.

@mateiz mateiz merged commit 5a22b9f into mlflow:master Jun 20, 2018
jdlesage pushed a commit to jdlesage/mlflow that referenced this pull request Dec 23, 2019
dbczumar referenced this pull request in dbczumar/mlflow May 3, 2022
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.

3 participants