Skip to content
This repository has been archived by the owner on Sep 8, 2021. It is now read-only.

Make reorder_dag() maintain list order #21

Open
peterbe opened this issue May 16, 2014 · 1 comment
Open

Make reorder_dag() maintain list order #21

peterbe opened this issue May 16, 2014 · 1 comment

Comments

@peterbe
Copy link
Contributor

peterbe commented May 16, 2014

The reorder_dag() function uses a dict to maintain a map of dependencies. This means we lose the primary order of the initial list which might be something we don't want to do.

@peterbe
Copy link
Contributor Author

peterbe commented May 16, 2014

Meh. Maybe not. It's hard to control the order very tightly. Dependencies is king.

We might want to consider using an ordereddict anyway just to avoid the randomishness of a dict for maintaining order amongst those jobs with the same dependency stuff.

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

No branches or pull requests

1 participant