-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding Julia extension instructions to template #76
Comments
I propose we edit the following sections of the RequirementsNote: The application requirements and setup instructions outlined below are intended to serve general users. To build the repository as-is, the following applications are required:
You may download the latest versions of each. By default, the Setup instructions below will assume their usage. Note that some of these applications must also be invocable from the command line. See the Command Line Usage section for details on how to set this up. Note that if you wish to run You must set up a personal If you are using MacOS, ensure your terminal is operating in bash rather than the default Once you have met these OS and application requirements, clone a team repository from Adding PackagesNote: These instructions are relevant for users who have installed PythonAdd any required packages to
RAdd any required packages that are available via StataInstall
JuliaFirst, add any required Julia packages to
Command Line UsageFor instructions on how to set up command line usage, refer to the repo wiki. By default, the repository assumes these executable names for the following applications:
Default executable names can be updated in Windows DifferencesThe only addition here should be updating the WindowsOS executable name for |
Additional flags for @jc-cisneros and I to complete, in addition to the proposed
We can either accomplish the above tasks here, or create a new issue. cc @jc-cisneros |
@snairdesai thanks for this! The additions make sense. The only one I would change is
The reason we are keeping track of the integration of Julia packages with Conda is that in the process above we are solving dependencies separately for Python/R packages and Julia packages. This will not solve dependencies or harmonize the paths across Python/R/Julia (this gave us issues when using a matplotlib backend previously). |
Thanks @jc-cisneros! Got it -- makes sense. I'll remove this text from the |
@snairdesai Regarding the steps outlined in #76 (comment), I would split them into separate issues that are handled in parallel to this one. In particular, I would suggest one issue dealing with using |
Summary + DeliverablesIn this issue (#76), we updated the Following a discussion with @jc-cisneros, we determined that the additional action items referenced above should be addressed in separate issues, which we will open and resolve as more bandwidth arises. I will proceed to open a PR with @jc-cisneros as a reviewer. |
The purpose of this issue (#76) is to provide additional user documentation in
README.md
on our Julia extension, which was integrated withingentzkow/template
in #67. I noticed when pulling these changes to ourselective-exposure
project that theREADME.md
file does not currently have instructions on running Julia scripts or initializing the Julia environment withinminiconda
. This process differs slightly from those we use for other softwares, because Julia is not yet fully integrated withinconda-forge
. I will shortly post a proposed revision to theREADME.md
file describing this procedure.cc @jc-cisneros @gentzkow
The text was updated successfully, but these errors were encountered: