damccorm commented on PR #38090:
URL: https://github.com/apache/beam/pull/38090#issuecomment-4215308386
The original failure I linked to was:
```
The conflict is caused by:
The user requested tenacity<9 and >=8.0.0
google-adk 1.28.1 depends on tenacity<10.0.0 and >=9.0.0
To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip to attempt to solve the dependency
conflict
```
This was clearly an adk issue.
The new set of changes seems to be causing
https://github.com/apache/beam/actions/runs/24182541038/job/70578913707 to fail
all dataframes tests:
```
self = <apache_beam.runners.interactive.display.pipeline_graph.PipelineGraph
object at 0x13ef82060>
vertex_dict = defaultdict(<class 'dict'>, {'"Create"': {}, 'pcoll5370':
{'shape': 'circle', 'label': ''}, '"Map(<lambda at
interacti...onymous_pcollection_5363630832': {'shape': 'circle'},
'"PlaceholderExpression[placeholder_DataFrame_5363640192]"': {}})
edge_dict = defaultdict(<class 'dict'>, {('"Create"', 'pcoll5370'): {},
('pcoll5370', '"Map(<lambda at interactive_runner_test.py:...640192]"'): {},
('"PlaceholderExpression[placeholder_DataFrame_5363640192]"',
'anonymous_pcollection_5363630832'): {}})
default_vertex_attrs = {'color': 'blue', 'fontcolor': 'blue', 'shape': 'box'}
default_edge_attrs = None
def _construct_graph(
self, vertex_dict, edge_dict, default_vertex_attrs,
default_edge_attrs):
"""Constructs the pydot.Dot object for the pipeline graph.
Args:
vertex_dict: (Dict[str, Dict[str, str]]) maps vertex names to
attributes
edge_dict: (Dict[(str, str), Dict[str, str]]) maps vertex name pairs
to
attributes
default_vertex_attrs: (Dict[str, str]) a dict of attributes
default_edge_attrs: (Dict[str, str]) a dict of attributes
"""
with self._lock:
try:
pydot.Dot()
except NameError:
> raise RuntimeError(
'pydot is required for pipeline graph generation. '
'Install it with: pip install pydot')
E RuntimeError: pydot is required for pipeline graph generation.
Install it with: pip install pydot
```
We need this to be fixed before we can merge
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
