You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
not identify the new _pins.yaml file at the root of the board as a pin
allow folks to write a manifest file from Python
The first one is high priority as we plan to not release pins for R until this gets implemented. The second is low priority and we can wait on it until whatever is a good time.
The text was updated successfully, but these errors were encountered:
@juliasilge it appears adding a `pins.txt` file as a manifest breaks `pin_search` on python pins. The reason is that in the python version `pin_list()` returns the filename, and `pin_search` attempts to retrieve metadata for every pin as part of search.
wdyt is a good move here? I can modify pins-python to ignore pins.txt files, but I'm wondering what the right way sequence is for releasing..
edit: how I tested
Using an .env file with credentials filled in from .env.dev.
In python:
fromdotenvimportload_dotenvload_dotenv()
frompinsimportboard_s3frompins.tests.helpersimportBoardBuilderbb=BoardBuilder("s3")
board=bb.create_tmp_board("pins/tests/pins-compat")
# list path to temporary pins bucket, for copying into Rboard.board
Switching to R:
board<- board_s3(<bucket_from_above>, prefix="<prefix_from_above>/")
# note that this also prints a warningboard %>% board_manifest()
In rstudio/pins-r#661 we added support for a manifest file. In pins for Python, we now need to:
_pins.yaml
file at the root of the board as a pinThe first one is high priority as we plan to not release pins for R until this gets implemented. The second is low priority and we can wait on it until whatever is a good time.
The text was updated successfully, but these errors were encountered: