search

robotology / whole-body-estimators

YARP devices that implement estimators for humanoid robots.
26 stars 12 forks source link

[Actions] Add cache action to cache source-based dependency builds and add release versioning to source-based dependencies #54

Closed prashanthr05 closed 4 years ago

prashanthr05 commented 4 years ago

Inspired and duplicated from

prashanthr05 commented 4 years ago

Is there a way to manually re-run a workflow? I can't find any.

prashanthr05 commented 4 years ago

Is there a way to manually re-run a workflow? I can't find any.

I would like to test the Cache feature. Is there a clean way to do this instead of adding a dummy commit? @traversaro

traversaro commented 4 years ago

Is there a way to manually re-run a workflow? I can't find any.

I would like to test the Cache feature. Is there a clean way to do this instead of adding a dummy commit? @traversaro

Not that I am aware of. For testing, I personally used by fork of the repo, and then when everything was working I tried in the main repo I was targeting. However, I think that in this case as most of the code is similar to the one working in iDynTree, we can also think of merging directly.

prashanthr05 commented 4 years ago

Not that I am aware of

A way was shown in https://github.com/robotology/whole-body-estimators/pull/54#discussion_r397845803 ;D

prashanthr05 commented 4 years ago

Caching was tested. Build took

But I fear the cache is using iDynTree devel. Technically, there shouldn't be a problem.

traversaro commented 4 years ago

But I fear the cache is using iDynTree devel. Technically, there shouldn't be a problem.

No problem, the cache produced in a PR is ignored in the main branch, so as soon as you merge it a new (correct) cache will be generated, see https://help.github.com/en/actions/configuring-and-managing-workflows/caching-dependencies-to-speed-up-workflows#restrictions-for-accessing-a-cache .