-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add missing reference terms from reference import #120
Comments
Yeah, and there are some items about |
Good work, but I think we need to do a bit more work on this before closing the issue, and there should be a pull request review. Python-specific reference terms are keywords from the Python language. For example, Another example, Another example, While reviewing this issue, see also my comments on recent pull requests: |
Sorry, my mistake. |
Currently I believe that all Python references have been removed from the docs, and R references (including importLibrary) were added. Can this issue be closed? |
Actually, I see |
No, we miss the function |
library is not a valid item in our mode, IMO. 🤔 |
Got it. That will be good for a community contribution.
I don't understand what you mean. We have a whole section on the repo homepage about
...but we don't have |
Oh, my mistake. Yeah we don't have the doc for |
Currently the reference is seeded by importing reference entries, description text, and cross-references from the Python mode.
However, Python mode contains many Python-specific reference entries which are not in the Processing(Java) API, and which will not be in Python.R. For example:
70009bc#diff-67374dee80d03cf169846122755bef45
Some python-specific entries include:
tuple
,dictionary
,string
https://processing-r.github.io/Processing.R-docs/
These need to be removed -- either through an ignore list for the automatic import script, or, if the import was one-time-only, then manually removed (along with any cross-references).
Note however that if there ever is a re-import then we have to clean all over again, so an ignore list (processed by either the import script or a cleanup removal script) might be better approach than a manual cleanup commit.
The text was updated successfully, but these errors were encountered: