Skip to content
This repository has been archived by the owner on Jul 24, 2018. It is now read-only.

Manifest dependency cycle does not generate a clear error. #178

Open
jezogwza opened this issue Nov 30, 2017 · 2 comments
Open

Manifest dependency cycle does not generate a clear error. #178

jezogwza opened this issue Nov 30, 2017 · 2 comments
Labels

Comments

@jezogwza
Copy link

Is this a bug report or feature request? (choose one):
BUG

Python Version (output of python --version):

Development or Deployment Environment?:

Release Tag or Master:

Expected Behavior:

What Actually Happened:

How to Reproduce the Issue (as minimally as possible):

Any Additional Comments:

@gardlt
Copy link
Contributor

gardlt commented Nov 30, 2017

could you provide the error you are getting?

@gardlt gardlt added the bug label Nov 30, 2017
@mark-burnett
Copy link
Contributor

This can be easily reproduced by having a chart definition depend on itself. If you don't want to implement your own cycle detection, you may want to consider using https://networkx.github.io/ for that.

This is exposed by buggy configuration referred to in att-comdev/promenade#47

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants