Comments (2)
I like this a lot.
Is it too magical?
I don't think so. It looks explicit to me.
Some misc thoughts:
- How does
civis_script
handle custom vs container jobs? Should it becivis_container
instead? - How are you planning to handle run ids? Default to latest if it's NULL?
- Maybe out of scope, but would it make sense to implement
write_civis(civis_container(...))
too?
from civis-r.
Yeah, these are all great questions.
- How does
civis_script
handle custom vs container jobs? Should it becivis_container
instead?
I'm using jobs_get
to rerturn something for all job types. Then I can use scripts_list_*_runs_outputs
based on the job type (I just have a lookup table).
The only confusion is that the job type is the same for custom
and container
. However, I can tell if a script is custom by whether it has a non-null fromTemplateId
. So I can fully resolve what to do unambiguously for any job type returned from jobs_get
.
- How are you planning to handle run ids? Default to latest if it's NULL?
Yep!
- Maybe out of scope, but would it make sense to implement
write_civis(civis_container(...))
too?
Yes! I'm planning on thinking about writing to container outputs next.
from civis-r.
Related Issues (20)
- ENH helper function to list models
- ENH helper function to share models
- ENH automatically deploy docs on release
- ENH provide option to override default polling interval HOT 1
- ENH: Automatically retry on all verbs when rate limit is exeeded
- ENH: Point civis_ml script templates to aliases HOT 1
- Add `patch` option to `get_script_fun` HOT 4
- Generalize `write_job_output` to an arbitrary object instead of "File" HOT 2
- Generalized `patch` function (plus a motivating example) HOT 3
- ENH: retry more than 3 times
- Is there any use in writing a file locally first before calling `write_civis_file`? HOT 3
- Adding project/report sharing utility code HOT 3
- BUG: error handling on 429
- Switch to CSVImports for Redshift upload
- DOC: Documentation on CASS/NCOA endpoint not complete HOT 1
- BUG: `fetch_all` throws error on `aliases_list` API function HOT 3
- Argument 'local' of Future() to become defunct [breaks your package] HOT 4
- future: Rename 'state' from `"succeeded" to "finished" HOT 10
- Is this repo and the 'civis' package maintained? HOT 2
- New SQL reports will need API functions
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from civis-r.