offsets and weights don't work in v6.0-dev for glmSLMA (and therefore for the lmerSLMA functions too) #42

Description

@tombisho

In v5.1 offsets and weights work as expected for glmSLMA

In v6.0-dev, the error is returned:

invalid type (closure) for variable '(offset)'

I believe this is related to the modification of line 167 onwards:

	if(!(is.null(offset)))
{
cbindtext.offset <- paste0("cbind(", offset,")")
offset <- eval(parse(text=cbindtext.offset))
}

where in v6.0 we have the addition of env = parent.frame()

I will attempt to figure out what is going on, because I don't have a good understanding of the workings of R environments with DSI. But I guess that the offset vector is in the 'wrong' environment to be accessed by the glm model...

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      , 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all \u003cpre\u003e\u003ccode\u003e blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks"); } } catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); } })(); })();
      Skip to content

      offsets and weights don't work in v6.0-dev for glmSLMA (and therefore for the lmerSLMA functions too) #42

      Description

      @tombisho

      In v5.1 offsets and weights work as expected for glmSLMA

      In v6.0-dev, the error is returned:

      invalid type (closure) for variable '(offset)'

      I believe this is related to the modification of line 167 onwards:

      	if(!(is.null(offset)))
      {
      cbindtext.offset <- paste0("cbind(", offset,")")
      offset <- eval(parse(text=cbindtext.offset))
      }
      

      where in v6.0 we have the addition of env = parent.frame()

      I will attempt to figure out what is going on, because I don't have a good understanding of the workings of R environments with DSI. But I guess that the offset vector is in the 'wrong' environment to be accessed by the glm model...

      Activity

      Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

      Metadata

      Metadata

      Assignees

      No one assigned

        Labels

        No labels
        No labels

        Type

        No type

        Projects

        No projects

          Relationships

          None yet

          Development

          No branches or pull requests

          Issue actions