update panel version

[AjayThorve]

fixes #360

[AjayThorve]

waiting for panel 0.12.7 to be uploaded to conda-forge for CI to pass

[jakirkham]

Thanks Ajay! 😀

Should we update the integration repo's panel version constraint as well?

[mmccarty]

You could also pin jinja2=2.11.3, if there are no other issues with that version.

[jakirkham]

Yeah we were planning to update to the new panel package once released. Already have PR ( conda-forge/panel-feedstock#48 ) up to fix that. Just need a review from the panel developers.

[AjayThorve]

You could also pin jinja2=2.11.3, if there are no other issues with that version.

I think there might be, the @rapidsai/docker repo has pinned the jinja2 package at a higher version (https://github.com/rapidsai/docker/blob/branch-22.04/requirements.txt) for CI

[ajschmidt8]

You could also pin jinja2=2.11.3, if there are no other issues with that version.

I think there might be, the @rapidsai/docker repo has pinned the jinja2 package at a higher version (https://github.com/rapidsai/docker/blob/branch-22.04/requirements.txt) for CI

That's safe to ignore. I can adjust that pin if necessary. It doesn't have any effect on any of the other RAPIDS packages.

[AjayThorve]

You could also pin jinja2=2.11.3, if there are no other issues with that version.

I think there might be, the @rapidsai/docker repo has pinned the jinja2 package at a higher version (https://github.com/rapidsai/docker/blob/branch-22.04/requirements.txt) for CI

That's safe to ignore. I can adjust that pin if necessary. It doesn't have any effect on any of the other RAPIDS packages.

Well in that case, pinning jinja2 might resolve panel issues without waiting for a 0.12.7 release, if that works, I'll go ahead and push the pin for that as well!

[AjayThorve]

Thanks Ajay! grinning

Should we update the integration repo's panel version constraint as well?

@jakirkham added the PR to integration repo -> rapidsai/integration#454

[jakirkham]

panel has been updated 😀

[ajschmidt8]

panel has been updated grinning

I think we can unpin the Jinja pkg now then, right?

[AjayThorve]

panel has been updated grinning

I think we can unpin the Jinja pkg now then, right?

yep, just tested locally, newer panel works with all jinja versions, unpinned it.

[ajschmidt8]

rerun tests

[ajschmidt8]

admin merging since all tests have passed except arm due to some machine issues.

[jakirkham]

Thanks all! 😄