Non-exportable objects: Add support for future.globals.onReference=function(...) { }
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 35/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Stack tecnologico
- r
- Ambito
- distributed-systems
Direzione di ricerca
Inizia dalla gestione esistente di future.globals.onReference e future:::reference_filters(), incluso il filtro ignore_envirs mostrato nell'issue. Definisci il contratto di ritorno del callback e il modo in cui interagisce con i tipi di riferimento accettati; il lavoro è completato quando il comportamento è specificato e supportato in modo coerente.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Currently, R option future.globals.onReference can take values:
"ignore"- don't check for non-exportable objects among globals (fastest)"message"- produce a message when a non-exportable objects is detected"warning"- produce a warning when a non-exportable objects is detected"error"- produce an error when a non-exportable objects is detected
I think it could be useful to provide some further control to savvy users/developers out there. More precisely, support for something like:
options(future.globals.onReference = function(ref) {
# accept / reject 'ref' based on inspect
# Q: What should be returned? Above `"ignore"`, ..., `"error"`, or TRUE/FALSE?
})
Having this in place would simplify enabling future.globals.onReference = "error" by default. For instance, if it turns out there are references that can indeed be exported (yet to be heard of), then we can fall back to above filter.
OTH, there's alteady an internal future:::reference_filters() function that allows us to set, add, drop, reset the list of accepted reference types, e.g. (the following is already set)
future:::reference_filters("append", ignore_envirs = function(ref, typeof, class, ...) {
typeof != "environment"
})
Maybe this is enough for now?
- Lingua principale
- R
- Stelle
- 1k
- Fork
- 93
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di futureverse/future
-
The first code example in the README does not workForse già presa @sean0407 l’ha presa 4 giorni fa. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
futureverse/future#838 · 1 commento · 1 reazione ·
-
bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
futureverse/future#833 ·
-
feature/resources RNG
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
futureverse/future#832 ·
-
RNG
Difficoltà 5/5 Più di una settimana Idoneità per principianti 38/100
futureverse/future#831 · 1 commento ·
-
feature request
Difficoltà 5/5 Più di una settimana Idoneità per principianti 30/100
futureverse/future#822 · 1 reazione ·
Tutte le issue di futureverse/future
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
HelenaLC/spatialdataR#284 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 71/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
mlr-org/mlr3cmprsk#42 ·
I maintainer di solito rispondono entro 1 giorno
-
Hackathon: Tier 1
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
pharmaverse/logrx#309 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno