-
Notifications
You must be signed in to change notification settings - Fork 3
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
make pdf_rendering work #7
Comments
Hello Peter and thanks for your nice function :)
|
Hey, thanks for writing in. I am having a hard time understanding your question. Can you make it more readable (help me help you) - e.g. format code as code, have a look here: https://docs.github.com/en/free-pro-team@latest/github/writing-on-github/basic-writing-and-formatting-syntax? Anyhow, most likely the answer is no. No, pdf rendering is not working that is why this issue still is open and I currently have no time/plans to make it work - sorry. |
Hi, thanks for your quick reply. Sorry for the bad structure; I formatted it as code now. I meant that firstly your methods If pdf rendering will not work, could you hint me what to do with the json code, to get a nicely formatted crossword? Do I need to paste it on a webpage? Unfortunately, I am not familiar with json. |
Yeah, I see now.
Look, the code in ./rendering is all commented out - this is for a reason.
This was a try, an experiment, It did not work. In the last 2 years I did
not find neither time nor motivation to work on the problem. I will not
invest any time on this, nor do I have time to support questions on
abandoned code - consider ./rendering as non-existent as far as I am
concerned. Most likely the code in ./rendering is very much out of sync
with the rest of the package. I think I switched from a more function
oriented to object oriented programming solution - browse the git history
if you care to find out what happened.
I did not find the function you were asking for, most likely it was there
once and got refactored away along the way.
If you find a way to make this work, please do make a pull request - (Fork
the repo, make a new branch, solve the problem, document your code, pass R
CMD check, open pull request).
Private fields and methods are most likely private to not expose data/logic
that should not be tempered with or which I do not like to support.
Therefore, I see no reason to expose them.
Sorry, but this package is pretty much as-is as far as I am concerned. But
since it's open source you can of course fork it and work on it under the
terms of the licence agreement.
Best, Peter
Am Di., 13. Okt. 2020 um 09:47 Uhr schrieb Christian-T <
notifications@github.com>:
… Hi, thanks for your quick reply. Sorry for the bad structure; I formatted
it as code now. I meant that firstly your methods cw_glue_puzzle() and
cw_glue_solution() did not work for me, but with the above mentioned
code, they do work. cw is a R6 object, that has some public and some
private methods, and rows and columns are private, which is why we (at
least I) need to access it via this complicated cw$.enclos_env$private$...-part.
Secondly, your function cw_glue_clues() refers to another function called
cw_words_add_id() which seems to add an id column ;) . However, I could
not find this function in your repo; maybe you have stored it locally
somewhere?
If pdf rendering will not work, could you hint me what to do with the json
code, to get a nicely formatted crossword? Do I need to paste it on a
webpage? Unfortunately, I am not familiar with json.
Thanks in advance!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#7 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAHFJA4H64LTAIFBOGIN5ALSKQAZRANCNFSM4EKLHTVQ>
.
|
Also, please do your homework. The official example from the documentation does already provide the information you are trying to forcefully extract from the crossword object, see |
No description provided.
The text was updated successfully, but these errors were encountered: