Cloudron makes it easy to run web apps like WordPress, Nextcloud, GitLab on your server. Find out more or install now.


Skip to content
  • Categories
  • Recent
  • Tags
  • Popular
  • Bookmarks
  • Search
Skins
  • Light
  • Cerulean
  • Cosmo
  • Flatly
  • Journal
  • Litera
  • Lumen
  • Lux
  • Materia
  • Minty
  • Morph
  • Pulse
  • Sandstone
  • Simplex
  • Sketchy
  • Spacelab
  • United
  • Yeti
  • Zephyr
  • Dark
  • Cyborg
  • Darkly
  • Quartz
  • Slate
  • Solar
  • Superhero
  • Vapor

  • Default (No Skin)
  • No Skin
Collapse
Brand Logo

Cloudron Forum

Apps | Demo | Docs | Install
  1. Cloudron Forum
  2. LanguageTool
  3. Languagetool: Download of n-grams ignores path set in env

Languagetool: Download of n-grams ignores path set in env

Scheduled Pinned Locked Moved Solved LanguageTool
9 Posts 3 Posters 1.3k Views 3 Watching
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • necrevistonnezrN Offline
    necrevistonnezrN Offline
    necrevistonnezr
    wrote on last edited by necrevistonnezr
    #1

    My env:

    NGRAMS_DATASET_PATH=/media/Languagetool/
    

    My volume:
    87097514-7faf-40b0-9860-d2555d3b65ec-image.png
    When I download n-grams

    # /app/pkg/install-ngrams.sh /app/data/ngrams en de
    

    they get installed in /app/data/ngrams

    Be sure to set NGRAMS_DATASET_PATH=/app/data/ngrams in /app/data/env and restart the app.
    
    1 Reply Last reply
    0
    • nebulonN Offline
      nebulonN Offline
      nebulon
      Staff
      wrote on last edited by
      #2

      ah the docs are not obvious then. So the install-ngrams.sh first argument is the path where they get installed. Then it also has to be set in the env.

      1 Reply Last reply
      0
      • necrevistonnezrN Offline
        necrevistonnezrN Offline
        necrevistonnezr
        wrote on last edited by necrevistonnezr
        #3

        I see my stupidity... 🙂
        I thought it would adopt the path set in env

        1 Reply Last reply
        0
        • necrevistonnezrN necrevistonnezr has marked this topic as solved on
        • necrevistonnezrN Offline
          necrevistonnezrN Offline
          necrevistonnezr
          wrote on last edited by necrevistonnezr
          #4

          Added a commit to the documentation, https://git.cloudron.io/necrevistonnezr/docs/-/blob/7625f8d9b7dbfa55f83cc18420fb1bd82177484f/knowledgebase/docs/apps/languagetool.md

          1 Reply Last reply
          0
          • nebulonN Offline
            nebulonN Offline
            nebulon
            Staff
            wrote on last edited by
            #5

            thanks, can you rebase your merge request to current docs master so we can merge?

            1 Reply Last reply
            0
            • necrevistonnezrN Offline
              necrevistonnezrN Offline
              necrevistonnezr
              wrote on last edited by
              #6

              Hmm, I thought I did... now?

              1 Reply Last reply
              0
              • nebulonN Offline
                nebulonN Offline
                nebulon
                Staff
                wrote on last edited by nebulon
                #7

                There seems to be one https://git.cloudron.io/cloudron/docs/-/merge_requests/46 which is 120 commits behind and then an empty draft one at https://git.cloudron.io/cloudron/docs/-/merge_requests/47

                If its easier, I can also just copy the text itself.

                1 Reply Last reply
                0
                • necrevistonnezrN Offline
                  necrevistonnezrN Offline
                  necrevistonnezr
                  wrote on last edited by necrevistonnezr
                  #8

                  That's fine, thank you, I'm not particularly familiar with Git systems.

                  1 Reply Last reply
                  1
                  • girishG Offline
                    girishG Offline
                    girish
                    Staff
                    wrote on last edited by
                    #9

                    I have manually copied/merged.

                    1 Reply Last reply
                    1
                    Reply
                    • Reply as topic
                    Log in to reply
                    • Oldest to Newest
                    • Newest to Oldest
                    • Most Votes


                    • Login

                    • Don't have an account? Register

                    • Login or register to search.
                    • First post
                      Last post
                    0
                    • Categories
                    • Recent
                    • Tags
                    • Popular
                    • Bookmarks
                    • Search