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

Switch to pydot 1.2.x #7448

Merged
merged 1 commit into from
Jul 29, 2017
Merged

Switch to pydot 1.2.x #7448

merged 1 commit into from
Jul 29, 2017

Conversation

ghisvail
Copy link
Contributor

Development of pydot has now converged here, with version 1.2.0 providing the missing Python 3 support. The old pydot and pydot-ng are no longer maintained.

Development of `pydot` has now converged [here](https://github.com/erocarrera/pydot), with version 1.2.0 providing the missing Python 3 support. The old `pydot` and `pydot-ng` are no longer maintained.
@fchollet
Copy link
Member

The old pydot and pydot-ng are no longer maintained.

The repo you linked was last updated 9 months ago... still more up-to-date than the other versions, but I feel like this one is going unmaintained as well.

Our reliance on PyDot sucks. But when we assessed our options 2 years ago, we didn't have any better library for Python graph plotting.

PR is LGTM

@fchollet fchollet merged commit f98b1da into keras-team:master Jul 29, 2017
ahundt added a commit to ahundt/keras that referenced this pull request Aug 1, 2017
…ize_input_data

* commit '01e2148732e4083b4850345e5ce4dd499cb5999e': (87 commits)
  Fix common backend styles (keras-team#7476)
  Allow dynamic shape for repeat_elements (keras-team#7422)
  Fix test style (keras-team#7473)
  Crossentropy backend API consistency cleanup (keras-team#7199)
  Replace the reserved word `input` with `inputs` (keras-team#7474)
  Increase test coverage (keras-team#7264)
  Fix l2_normalize
  Add default value for `l2_normalize`.
  Remove legacy axis handling in TF backend.
  Update save_model function (keras-team#7455)
  Switch to pydot 1.2.x (keras-team#7448)
  Add save, evaluate and predict functionality for example (keras-team#7430)
  Docs fix: `pointwise` instead of `depthwise` for SeparableConv2D (keras-team#7444)
  Fix conv reccurent test
  Style fix in conv recurrent tests.
  Support return_state parameter in ConvRecurrent2D (keras-team#7407)
  Small simplification in ResNet50 architecture
  Update FAQ with info about custom object loading.
  add example for passing in custom objects in load_model (keras-team#7420)
  Update applications.md (keras-team#7428)
  ...

# Conflicts:
#	tests/keras/backend/backend_test.py
ahundt added a commit to ahundt/keras that referenced this pull request Aug 1, 2017
…s_per_epoch

* commit '01e2148732e4083b4850345e5ce4dd499cb5999e': (98 commits)
  Fix common backend styles (keras-team#7476)
  Allow dynamic shape for repeat_elements (keras-team#7422)
  Fix test style (keras-team#7473)
  Crossentropy backend API consistency cleanup (keras-team#7199)
  Replace the reserved word `input` with `inputs` (keras-team#7474)
  Increase test coverage (keras-team#7264)
  Fix l2_normalize
  Add default value for `l2_normalize`.
  Remove legacy axis handling in TF backend.
  Update save_model function (keras-team#7455)
  Switch to pydot 1.2.x (keras-team#7448)
  Add save, evaluate and predict functionality for example (keras-team#7430)
  Docs fix: `pointwise` instead of `depthwise` for SeparableConv2D (keras-team#7444)
  Fix conv reccurent test
  Style fix in conv recurrent tests.
  Support return_state parameter in ConvRecurrent2D (keras-team#7407)
  Small simplification in ResNet50 architecture
  Update FAQ with info about custom object loading.
  add example for passing in custom objects in load_model (keras-team#7420)
  Update applications.md (keras-team#7428)
  ...
ahundt added a commit to ahundt/keras that referenced this pull request Aug 1, 2017
* commit '01e2148732e4083b4850345e5ce4dd499cb5999e':
  Fix common backend styles (keras-team#7476)
  Allow dynamic shape for repeat_elements (keras-team#7422)
  Fix test style (keras-team#7473)
  Crossentropy backend API consistency cleanup (keras-team#7199)
  Replace the reserved word `input` with `inputs` (keras-team#7474)
  Increase test coverage (keras-team#7264)
  Fix l2_normalize
  Add default value for `l2_normalize`.
  Remove legacy axis handling in TF backend.
  Update save_model function (keras-team#7455)
  Switch to pydot 1.2.x (keras-team#7448)
  Add save, evaluate and predict functionality for example (keras-team#7430)
  Docs fix: `pointwise` instead of `depthwise` for SeparableConv2D (keras-team#7444)

# Conflicts:
#	keras/backend/cntk_backend.py
#	keras/backend/theano_backend.py
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