gdtoolkit_4: init at 4.2.2

Description of changes
Rename gdtoolkit to gdtoolkit_3
Init gdtoolkit_4 at v4.2.2
Update gdtoolkit_3 to v3.5.0
Tested and seems to be working fine.
Resolves #267249
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 291040 run on x86_64-linux 1

2 packages failed to build:

gdtoolkit_4
gdtoolkit_4.dist

2 packages built:

gdtoolkit_3
gdtoolkit_3.dist

logs
=============================== warnings summary ===============================
gdtoolkit/parser/parser.py:8
  /build/source/gdtoolkit/parser/parser.py:8: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
    import pkg_resources

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
=========================== short test summary info ============================
FAILED tests/generated/test_expression_parsing.py::test_expression_formatting - gdtoolkit.formatter.exceptions.TreeInvariantViolation: TreeInvariantViolati...
===== 1 failed, 497 passed, 307 skipped, 2 deselected, 1 warning in 44.31s =====

it build the second time :-)
Result of nixpkgs-review pr 291040 run on x86_64-linux 1

4 packages built:

gdtoolkit_3
gdtoolkit_3.dist
gdtoolkit_4
gdtoolkit_4.dist

I just had the same issue when running on GitHub actions, it was fine the second time I ran it? I'll look into it, but it's a bit difficult since it doesn't happen locally or even consistently.
Some quick research makes me think it is something to do with setuptools, maybe it should be somewhere else rather than propagatedBuildInputs? Still very strange that it isn't always an issue.

I've been using the package for the past week or so without running in to the issue. setuptools had been updated in nixpkgs since creating this PR, so I updated the branch. I believe there was an issue with that specific version of setuptools that has since been fixed.
I think this PR is ready to merge!

That's fair, I can get that updated! The whole naming scheme with Godot is a bit odd, godot3 vs godot_4, but it makes sense to match the pnames.

Yea, I got confused looking into the naming too, the Godot packages don't seem consistent, and they changed it for godot 3 but not for godot 4 at some point, idk. A bit messy, so I guess best you can do is not add to the mess?

Result of nixpkgs-review pr 291040 run on x86_64-linux 1

2 packages failed to build:

gdtoolkit_4
gdtoolkit_4.dist

2 packages built:

gdtoolkit_3
gdtoolkit_3.dist

Oh, I only tested by actually running it as part of my system (actually using it). Also tried nixpkgs-review now, and it fails some tests:
❯ nix-shell -p nixpkgs-review --run "nixpkgs-review pr 291040"
[...]
error: builder for '/nix/store/9s9kzlm9vj39inkznyb64i5hhaamj94j-gdtoolkit-4.2.2.drv' failed with exit code 1;
       last 10 log lines:
       > gdtoolkit/parser/parser.py:8
       >   /build/source/gdtoolkit/parser/parser.py:8: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
       >     import pkg_resources
       >
       > -- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
       > =========================== short test summary info ============================
       > FAILED tests/generated/test_expression_parsing.py::test_expression_parsing - hypothesis.errors.InvalidArgument: Undefined terminals ['_DEDENT', '_INDENT...
       > FAILED tests/generated/test_expression_parsing.py::test_expression_formatting - hypothesis.errors.InvalidArgument: Undefined terminals ['_DEDENT', '_INDENT...
       > ===== 2 failed, 496 passed, 307 skipped, 2 deselected, 1 warning in 11.69s =====
       > /nix/store/xfhkjnpqjwlf6hlk1ysmq3aaq80f3bjj-stdenv-linux/setup: line 1579: pop_var_context: head of shell_variables not a function context
       For full logs, run 'nix log /nix/store/9s9kzlm9vj39inkznyb64i5hhaamj94j-gdtoolkit-4.2.2.drv'.
error: 1 dependencies of derivation '/nix/store/7jxp9pnzc5r05b8bizxxa6aqcnx56czv-review-shell.drv' failed to build

Link to currently reviewing PR:
https://github.com/NixOS/nixpkgs/pull/291040

2 packages failed to build:
gdtoolkit_4 gdtoolkit_4.dist

2 packages built:
gdtoolkit_3 gdtoolkit_3.dist

Oh, I only tested by actually running it as part of my system (actually using it). Also tried nixpkgs-review now, and it fails some tests:
❯ nix-shell -p nixpkgs-review --run "nixpkgs-review pr 291040"
[...]
error: builder for '/nix/store/9s9kzlm9vj39inkznyb64i5hhaamj94j-gdtoolkit-4.2.2.drv' failed with exit code 1;
       last 10 log lines:
       > gdtoolkit/parser/parser.py:8
       >   /build/source/gdtoolkit/parser/parser.py:8: DeprecationWarning: pkg_resources is deprecated as an API. See https://setuptools.pypa.io/en/latest/pkg_resources.html
       >     import pkg_resources
       >
       > -- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
       > =========================== short test summary info ============================
       > FAILED tests/generated/test_expression_parsing.py::test_expression_parsing - hypothesis.errors.InvalidArgument: Undefined terminals ['_DEDENT', '_INDENT...
       > FAILED tests/generated/test_expression_parsing.py::test_expression_formatting - hypothesis.errors.InvalidArgument: Undefined terminals ['_DEDENT', '_INDENT...
       > ===== 2 failed, 496 passed, 307 skipped, 2 deselected, 1 warning in 11.69s =====
       > /nix/store/xfhkjnpqjwlf6hlk1ysmq3aaq80f3bjj-stdenv-linux/setup: line 1579: pop_var_context: head of shell_variables not a function context
       For full logs, run 'nix log /nix/store/9s9kzlm9vj39inkznyb64i5hhaamj94j-gdtoolkit-4.2.2.drv'.
error: 1 dependencies of derivation '/nix/store/7jxp9pnzc5r05b8bizxxa6aqcnx56czv-review-shell.drv' failed to build

Link to currently reviewing PR:
https://github.com/NixOS/nixpkgs/pull/291040

2 packages failed to build:
gdtoolkit_4 gdtoolkit_4.dist

2 packages built:
gdtoolkit_3 gdtoolkit_3.dist

I've been using this locally and worked around this by just disabling tests for these.
+  disabledTestPaths = [
+    "tests/generated/test_expression_parsing.py"
+    "tests/gdradon/test_executable.py"
+  ];

What I find odd though is that if I do multiple rebuilds these tests sometimes don't fail and sometimes do, quite odd, also not quite sure why disabledTests isn't disabling the individual tests that seem to be failing.

I've been using this locally

This is what I've been doing as well, and it works well so far, no problems.

worked around this by just disabling tests for these

I did not have to disable any tests. I'm not sure what your setup is, but I added github:squarepear/nixpkgs/gdtoolkit-4 as an input to my nixos config inputs (I'm using flakes), added it as an overlay to my channels, and then included gdtoolkit_4 in my system packages. Didn't have to do anything else like disabling tests.

I've been using this locally and worked around this by just disabling tests for these.
+  disabledTestPaths = [
+    "tests/generated/test_expression_parsing.py"
+    "tests/gdradon/test_executable.py"
+  ];

It seems to be working for me, but the inconsistency makes it difficult to tell. Hopefully everything is good now, but if anyone still has issues, let me know!

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/how-to-introduce-a-breaking-version-bump/46746/2

Hello, this seemed to break my system upgrade, on unstable-small, even though i don't use godot...
it spits out on the upgrade:
         at /nix/store/ab6iajix47qdm2v098dcdaqis6094300-source/pkgs/top-level/aliases.nix:38:20:

           37|   # Make sure that we are not shadowing something from all-packages.nix.
           38|   checkInPkgs = n: alias:
             |                    ^
           39|     if builtins.hasAttr n super

       error: gdtoolkit has been renamed to gdtoolkit_3 to distinguish from version 4

this host is a media server, running jellyfin, radarr, deluge... nothing with godot or games, or game dev... not even a gui.
here is the config: https://github.com/Thiago-Assis-T/.nixos and the hostname is  ThiagoServer, i tried show trace and mentioned the issue on discourse

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/package-name-changed-gdtoolkit-gdtoolkit-3-but-i-dont-use-that-package-directly-and-was-unable-to-identify-what-it-is-a-dep-of/46761/3

Hello, this seemed to break my system upgrade, on unstable-small, even though i don't use godot...
it spits out on the upgrade:
         at /nix/store/ab6iajix47qdm2v098dcdaqis6094300-source/pkgs/top-level/aliases.nix:38:20:

           37|   # Make sure that we are not shadowing something from all-packages.nix.
           38|   checkInPkgs = n: alias:
             |                    ^
           39|     if builtins.hasAttr n super

       error: gdtoolkit has been renamed to gdtoolkit_3 to distinguish from version 4

this host is a media server, running jellyfin, radarr, deluge... nothing with godot or games, or game dev... not even a gui.
here is the config: Thiago-Assis-T/.nixos and the hostname is ThiagoServer, i tried show trace and mentioned the issue on discourse

That's very strange. I have a bit of time this morning, I'll look into it and see if I can find anything!
My first complete guess would be something like nvim or similar using it as a formatter, but it doesn't make much sense that you would have the gdtoolkit formatter in your config.

Wow, guess seems to have been correct! It looks like it is an issue with nixvim
https://github.com/nix-community/nixvim/issues/1660
https://github.com/nix-community/nixvim/pull/1659

AAAAA yess, i'm using nixvim, with all ts grammars... perhaps thats it? i dont have as formatter or as linter though, never messed with godot, so never added it...

Yep 😄

thank you @squarepear

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/package-name-changed-gdtoolkit-gdtoolkit-3-but-i-dont-use-that-package-directly-and-was-unable-to-identify-what-it-is-a-dep-of/46761/4


Key Vault Secrets vs. Keys

This document states the following.

the preferred model is to use symmetric master keys stored as secrets in Key Vault and cached locally

What are the benefits of doing this with a secret in Key Vault versus a key in Key Vault?

@reicher001
Thanks for your feedback! We will investigate and update as appropriate.

@reicher001 please open this issue on the specific document you are referencing so we can better assist with this question. Closing this out because it should be created on the doc itself. Feel free to tag me in the comments here and reference the new issue you create.


Figure out how the dominoes wrapper backend works

We don't really know how to use this so we should start simple and maybe get the backend to return something like local Dominoes locations. Document this in a README
Acceptance criteria:

Successfully use the dominoes backend to make a call to the dominoes api and return something useful

https://github.com/RIAEvangelist/node-dominos-pizza-api
https://github.com/nat-henderson/terraform-provider-dominos
https://pypi.org/project/dominos/
https://github.com/roshanlam/PizzaPy
https://medium.com/hackernoon/how-i-forced-myself-to-code-daily-using-a-pizza-api-8bd1a0a6f66f
^ good one


Unlock connector

Describe your changes Added call to the method, connector_force_unlock() after authorisation is received.
Issue ticket number and link
Checklist before requesting a review

[ x] I have performed a self-review of my code
[ ] I have made corresponding changes to the documentation
[ ] I read the contribution documentation and made sure that my changes meet its requirements

Not required.


Start hidden?

Is there any way to have the quick return layer start out hidden and only come out when the list is scrolled?
I don't see any public methods to manually set the state of the view.
Obrigado!

Not really but that should be pretty easy to implement. I'll do it today and release a new version.

Thanks!!


Difficulty Replicating Antibody Optimization Results

I've been trying to replicate the antibody optimization results detailed in the paper, strictly adhering to the provided instructions and ensuring the activation of OpenMM relaxation. However, my attempts to reproduce the expected outcomes have not been successful. According to the paper, the average ddG calculated by GNN predictor should be approximately -7, but my results consistently show an average ddG of around -4, even when num_residue_changes is set to 0.

Sorry for the delay! I checked the codes and found that I accidentally changed the logic of optimization when I was refactoring the codes for release. I've updated the repo with the corrected codes, and now it should work by directly following the instructions in README:
GPU=0 bash scripts/test/optimize_test.sh checkpoints/cdrh3_opt.ckpt checkpoints/cdrh3_ddg_predictor.ckpt ./all_data/SKEMPI/test.json 0 50

Sorry for my carelessness again!


Bicep Create Private Endpoint+NIC with complete mode deployment

Hello,
Im stuck on complete mode deployment of privateEndpoint, even if I try to deploy it first time, bicep creates it correctly but then trying to remove NIC with no reason and ofc it cant remove it because it's connected to private endpoint.
In privateEndpoint resource Im using 'customNetworkInterfaceName' but with auto generate NIC name result is the same.
resource privateEndpoints 'Microsoft.Network/privateEndpoints@2021-05-01' =[for i in range(0, length(sites)): {
  name: replace(siteResource[i].name, '-api-', '-privend-api-')
  location: location
  properties: {
    customNetworkInterfaceName: nicArr[i]
    privateLinkServiceConnections: [
      {
        name: replace(siteResource[i].name, '-api-', '-privend-api-')
        properties: {
          privateLinkServiceId: siteResource[i].id
          groupIds: [
            'sites'
          ]
        }
      }
    ]
    manualPrivateLinkServiceConnections: []
    subnet: {
      id: resourceId(rgNetworkName,'Microsoft.Network/virtualNetworks/subnets', vnetNameVar, privateEndpointSubnet)
    }

  }
}]

privateEndpoints resource is a part of entire module which Ive got in base template
module appSvcModule  'modules/appSvc.bicep' = if (apiAppDeploy) {
  name: 'appSvcDeploy'
  dependsOn: [
    Network
    RGsModule
  ]
  params:{
   ...
  }
}

'template' events:

Any idea for workaround on that?
Thanks

I think this is a limitation in the design of complete mode, but should be addressed with deployment stacks which is now in private preview. cc @bmoore-msft as FYI

so I understand that it's not possible to maintain IAAC private endpoints with NICs in Complete mode ?

Yes. If there is a NIC getting automatically created on your behalf in the same RG as these other resources, then a complete mode deployment will attempt to delete that NIC on a redeployment because it is not declared in the bicep code. IOW, it is by design. This is one of many limitations of complete mode and why we are investing in Deployment Stacks to duplicate and extend the functionality of Complete mode, without the drawbacks.

@alex-frankel  thanks for your help and informations.

@kowal20x7 - In general I don't think we ever recommend complete mode - there are too many latent behaviors.  TL;DR deploymentStacks will only attempt to clean up resources that are in the template.  So will help with this particular scenario, the flip side is that if the nic is implicitly created (i.e. not explicitly defined in the template) the stack will not clean it up even if you wanted it to... slightly separate issue, point being I'm interested in your scenario if it's related somehow...

@bmoore-msft yes that's exactly what I would need, to 'ignore' the resource (NIC in this case) from deletion while other resources will be deleted/modified if I wanted.
I tried with 'Lock' on Resource but unfortunatelly Complete deployment return error because is trying to remove it anyway. Currently in my case I see that I need to move private endpoints with NICs to separate resource group and never use Complete mode there.

Just ran into this same issue! I feel like Complete mode is much better in terms of keeping the infrastructure clean and without any loose ends, but this is a major drawback. Basically any infra using private endpoints is unusable with 'Complete' deployment mode. Following this

We've run into this issue as well and I've been scratching my head a couple of days.
This doesn't feel like enterprise-grade IaC solution..
At some point I was investigating option to define the whole NIC in Bicep and simply pass that as property to the private endpoint being deployed, but it seems NIC creation is fully automatic and only PE's NIC name can be customized.
Will examine deployment stacks, but the situation isn't really acceptable.

I opened a support ticket for this issues with Azure support. No one there has yet been able to confirm whether this is a problem. There is no documentation of this issue aside from this github issue, so if it is a known issue that will be addresses in future it would be great if this issue could perhaps be highlighted:-

On the bicep "microsoft learn" learning module when discussing deployment modes.
On the documentation for deploying  Private endpoints: https://learn.microsoft.com/en-us/azure/private-link/create-private-endpoint-template

I am facing the same issue with biceps, NIC and Private endpoints with complete mode deployment. The status of this issue is closed, but I couldnt find a solution or a workaround for this issue. Do anyone have idea on this?

@jegadeesan-ponnusamy can you please share your template for enabling the private link/endpoints?
Are you including the privateLinkServiceConnectionState object?
E.g.

https://github.com/brwilkinson/AzureDeploymentFramework/blob/main/ADF/bicep/x.vNetPrivateLink.bicep#L23-L27

@jegadeesan-ponnusamy
The reason it was Closed was because Complete Mode is known to be an incomplete solution for managing lifecycle. i.e. this was closed, with "will not fix" or "not planned". In the meantime, the recommendation would be to not use complete mode, given it's known limitations.
The new Deployment Stacks capability is currently being worked on/developed, which will be a more robust solution.

I will do some testing on the scenario that you are hitting and see if I can come up with any workaround for complete mode that covers this scenario.

@brwilkinson - Thank you for your inputs. I will wait for your findings. Also, I am already handling the scenario mentioned in az#1511.

@jegadeesan-ponnusamy
The reason it was Closed was because Complete Mode is known to be an incomplete solution for managing lifecycle. i.e. this was closed, with "will not fix" or "not planned". In the meantime, the recommendation would be to not use complete mode, given it's known limitations.
The new Deployment Stacks capability is currently being worked on/developed, which will be a more robust solution.

I will do some testing on the scenario that you are hitting and see if I can come up with any workaround for complete mode that might cover this.

It has been confirmed that the new deployment stacks will not solve this issue either. What makes you think deployment stack will fix it?

@dazinator agree. Will have to test.


Add RAG searcher

@bratseth Please review.
This is for external LLMs. From a user's point of view, setup in services.xml is something like this:
    <component id="openai" class="ai.vespa.llm.clients.OpenAI">
        <config name="ai.vespa.llm.llm-client">
            <!-- <apiKeySecretName>...</apiKeySecretName> -->
            <!-- <endpoint>...</endpoint> -->
      </config>
    </component>

    <search>
        <chain id="rag" inherits="vespa">
            <searcher id="ai.vespa.llm.search.RAGSearcher">
                <config name="ai.vespa.llm.llm-searcher">
                    <providerId>openai</providerId>
                    <!-- <propertyPrefix>...</propertyPrefix> -->
                </config>
            </searcher>
        </chain>
    </search>

The client can of course be injected into other stuff as well such as docprocs etc.

Thanks for the reminder. Added Beta annotations and some copyright headers.


[Fix] electron version

npm install failed with<EMAIL_ADDRESS>: https://github.com/electron/electron/issues/44529

Netron app and version: dev version
OS and browser version: Ubuntu 22.04.3 LTS

npm http fetch POST 200 https://mirrors.cloud.tencent.com/npm/-/npm/v1/security/advisories/bulk 1138ms
npm warn deprecated<EMAIL_ADDRESS>This functionality has been moved to @npmcli/fs
npm info run<EMAIL_ADDRESS>postinstall node_modules/electron node install.js
npm info run<EMAIL_ADDRESS>postinstall { code: 1, signal: null }
npm verbose stack Error: command failed
npm verbose stack     at promiseSpawn (/home/shun/.nvm/versions/node/v22.11.0/lib/node_modules/npm/node_modules/@npmcli/promise-spawn/lib/index.js:22:22)
npm verbose stack     at spawnWithShell (/home/shun/.nvm/versions/node/v22.11.0/lib/node_modules/npm/node_modules/@npmcli/promise-spawn/lib/index.js:124:10)
npm verbose stack     at promiseSpawn (/home/shun/.nvm/versions/node/v22.11.0/lib/node_modules/npm/node_modules/@npmcli/promise-spawn/lib/index.js:12:12)
npm verbose stack     at runScriptPkg (/home/shun/.nvm/versions/node/v22.11.0/lib/node_modules/npm/node_modules/@npmcli/run-script/lib/run-script-pkg.js:77:13)
npm verbose stack     at runScript (/home/shun/.nvm/versions/node/v22.11.0/lib/node_modules/npm/node_modules/@npmcli/run-script/lib/run-script.js:9:12)
npm verbose stack     at /home/shun/.nvm/versions/node/v22.11.0/lib/node_modules/npm/node_modules/@npmcli/arborist/lib/arborist/rebuild.js:329:17
npm verbose stack     at run (/home/shun/.nvm/versions/node/v22.11.0/lib/node_modules/npm/node_modules/promise-call-limit/dist/commonjs/index.js:67:22)
npm verbose stack     at /home/shun/.nvm/versions/node/v22.11.0/lib/node_modules/npm/node_modules/promise-call-limit/dist/commonjs/index.js:84:9
npm verbose stack     at new Promise (<anonymous>)
npm verbose stack     at callLimit (/home/shun/.nvm/versions/node/v22.11.0/lib/node_modules/npm/node_modules/promise-call-limit/dist/commonjs/index.js:35:69)
npm verbose pkgid<EMAIL_ADDRESS>npm error code 1
npm error path /home/shun/Documents/Projects/netron/netron/node_modules/electron
npm error command failed
npm error command sh -c node install.js
npm error HTTPError: Response code 404 (Not Found)
npm error     at Request._onResponseBase (/home/shun/Documents/Projects/netron/netron/node_modules/got/dist/source/core/index.js:913:31)
npm error     at Request._onResponse (/home/shun/Documents/Projects/netron/netron/node_modules/got/dist/source/core/index.js:948:24)
npm error     at ClientRequest.<anonymous> (/home/shun/Documents/Projects/netron/netron/node_modules/got/dist/source/core/index.js:962:23)
npm error     at Object.onceWrapper (node:events:633:26)
npm error     at ClientRequest.emit (node:events:530:35)
npm error     at origin.emit (/home/shun/Documents/Projects/netron/netron/node_modules/@szmarczak/http-timer/dist/source/index.js:43:20)
npm error     at HTTPParser.parserOnIncomingClient (node:_http_client:704:27)
npm error     at HTTPParser.parserOnHeadersComplete (node:_http_common:117:17)
npm error     at TLSSocket.socketOnData (node:_http_client:546:22)
npm error     at TLSSocket.emit (node:events:518:28)
npm verbose cwd /home/shun/Documents/Projects/netron/netron
npm verbose os Linux 6.2.0-26-generic
npm verbose node v22.11.0
npm verbose npm  v10.9.0
npm verbose exit 1
npm verbose code 1
npm error A complete log of this run can be found in: /home/shun/.npm/_logs/2024-11-03T12_41_59_205Z-debug-0.log
Command failed: npm install
npm verbose cwd /home/shun/Documents/Projects/netron/netron
npm verbose os Linux 6.2.0-26-generic
npm verbose node v22.11.0
npm verbose npm  v10.9.0
npm verbose exit 1
npm verbose code 1

<EMAIL_ADDRESS>works ~ Can we change the version?

@megemini please investigate why this is happening upstream.

Thanks for your response!!!
https://releases.electronjs.org/release/v33.0.2 some bugs fixed ~
But<EMAIL_ADDRESS>making netron installation failed is really bad and confused ... ...
I just want to report this issue and could keep on developing with<EMAIL_ADDRESS>, or we should downgrade to electron 33.0.1 in case some other developers encountering the same problem ~ 🫡


Fixed typescript function parameter definitions

This fixes TypeScript definitions for parameters that are functions.

Codecov Report

Merging #60 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #60   +/-   ##
=======================================
  Coverage   72.59%   72.59%
=======================================
  Files           7        7
  Lines         135      135
  Branches       18       18
=======================================
  Hits           98       98
  Misses         34       34
  Partials        3        3

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 43d1cf5...f94715b. Read the comment docs.

@jakeboone02 Published: https://github.com/sapientglobalmarkets/react-querybuilder
Thanks Jake!


RabbitMQAsyncCollector.AddAsync only adds to publish the body, not the properties

Hi.
The parameter "message" in method "RabbitMQAsyncCollector.AddAsync" recieves the full BasicDeliverEventArgs object as ReadOnlyMemory, converted from the class PocoToBytesConverter. In the message, we have the BasicProperties, but this method is adding all the content of the message into the body parameter of the publlish method and setting the BasicProperties to null.
This causes the message to be published without the basic properties (although they are in the body section).

@michaeltg17, are you trying to set output binding parameter like below:
public static void UserFunction(
    [QueueTrigger("queue-name")] string message, // Could be any trigger.
    [RabbitMQ(QueueName = "queue")] IAsyncCollector<BasicDeliverEventArgs> outMessages)

RabbitMQ extension does not have a special handling for BasicDeliverEventArgs for output binding so it treats it as a regular POCO class, converts it to JSON string and passes it as body to the output RabbitMQ queue. It does have special handling for type BasicDeliverEventArgs in trigger binding.
I can be wrong, but I think the reason for this is BasicDeliverEventArgs is only used for accepting messages from queue, not for sending messages back. For that user needs to make direct request to BasicPublishBatch.Add method. Let me know if you can find an example that contradicts this.
If your function looks similar to the example above, you can change it to below to get handle of the RabbitMQ client and then call BasicPublishBatch.Add directly on it and pass in properties:
public static void UserFunction(
    [QueueTrigger("queue-name")] string message, // Could be any trigger.
    [RabbitMQ(QueueName = "queue")] IModel client)


Which header to use?

There are a lot of headers in the page, so which one do I use?

I have the same issue, not knowing which header to use. Did you ever resolve this issue?

I have the same issue, not knowing which header to use. Did you ever resolve this issue?

Unfortunately not. I have tried multiple methods, but were unable to get any of them to work. The best bet would be for me to write a python script that uses inspect element to get the images of the textbook and then one can merge them into a PDF, but tbh, I'm too lazy.

Unfortunately not. I have tried multiple methods, but were unable to get any of them to work. The best bet would be for me to write a python script that uses inspect element to get the images of the textbook and then one can merge them into a PDF, but tbh, I'm too lazy.

Yeah, I think a lot of these scripts for Vitalsource got nuked with some of their latest updates. I tried a couple of scripts as well, they all seem to get my web access "unauthorized". :(  I'm lazy too, I will just use their offline app, much easier! :)

Yeah, I think a lot of these scripts for Vitalsource got nuked with some of their latest updates. I tried a couple of scripts as well, they all seem to get my web access "unauthorized". :(  I'm lazy too, I will just use their offline app, much easier! :)

Aight I think I should have the time to write stuff now.  🤔 I can probably just do a script where it auto clicks each page for you. Shouldn't be too hard.

Aight I think I should have the time to write stuff now.  🤔 I can probably just do a script where it auto clicks each page for you. Shouldn't be too hard.

That would be great, please let me know when you have something to try. Good luck! :)

That would be great, please let me know when you have something to try. Good luck! :)

more than a variable, probably it would be more practical to export the cookies from the browser as a file and have the script loading them from the file instead of depending on a string variable.

oh dang its been 2 years and funnily (or sadly?) enough this issue is still open rippp

That would be great, please let me know when you have something to try. Good luck! :)

Anyhow, hey @99bobster99 I know this is extremely late, but I feel like I do need to give a reply for closure. I'll admit I kinda forgot about this issue post here, and what I eventually did was a brute force technique with VBScript. I loaded the vitalsource textbook on the internet such that iirc it was something like
https://blah blah/blah/blah/<page number>, and I used VBScript to go through an inspect element sequence and save the image. Repeat this 1000+ times and you can save a textbook this way.
It was extremely cumbersome and I can't seem to find the vbs file now that it's been two years. But basically, you're just going through a while true loop click (whether its a click on a specific coordinate point on the computer screen or a keypress) cycle with an inner loop to loop through the page numbers. I had to run it manually for each page number increase in digit (so from like 99 -> 100 I had to redo the inner loop iirc)
I think you probably can quite easily replicate this but again, I really dont recommend this this is kind of dumb lol. Regardless, my classes use webassign now so I don't have any access to test with vitalsource anymore. My apologies and hopefully you had better luck than me.

Can you help. I got 5 books that are going to expire in a month.

Can you help. I got 5 books that are going to expire in a month.

just grab the book off of libgen tbh lol

Hey Captain Chicky, thank you so much for remembering to get back to me on my request! I received an email alert on calvinhobbes23's request and then saw your answer to my question. I'm glad you did, since I need to get some more books for both my University kids. You rock sir, thank you so much!!

Unfortunately Libgen doesnt have them. Those are AWS Training student guide material I got when I took AWS training. They are going to expire in a couple months.

@99bobster99 glad to be at least somewhat of help 😅, though if the vbs thing isn't appealing or is cumbersome you can try to give this a try after fixing it.
@calvinhobbes23 you can try that as well^ or maybe try looking for the books on archive.org
If it comes to naught, honestly just spend like a few hours of a day manually saving each page until you have the book.
I currently don't have any classes using vitalsouce so I don't have any way of developing a downloader tool, but if i ever do in the future I'll see what i can do then kek


Add feature image for social media for Docusaurus 1 blogs

📚 Documentation
How do I set a "feature image" for my docusaurus blogs?
Have you read the Contributing Guidelines on issues? Yes
Our docusaurus(v1) site contains several articles, each with its own set of images. How do I set one of those images as a featured image that would appear as a banner whenever we share the article link on social media like Facebook, Twitter or Linkedin? Currently, whenever we share any article from the blog, the same image is chosen as the feature image. Any help would be appreciated.

Is there any update on this?

Closed by #2856 - https://v2.docusaurus.io/docs/next/blog#header-options

We've added it in v2 - https://v2.docusaurus.io/docs/next/blog#header-options
It's unlikely we're going to add features to v1 now, but if someone wants to add support for it we're ok.

The Docusaurus team is actively working on Docusaurus v2.
After a very long alpha, Docusaurus v2 is now in beta.
We believe that the v2 beta is already better than v1, and v2 is ready for maintstream adoption.
Apart from critical issues and security fixes, we don't plan to invest much time on Docusaurus v1 anymore.
We are all closing v1 issues.
Maybe the current issue is already fixed in Docusaurus v2.
Please try to upgrade, and if the issue is still there on Docusaurus v2, open a new issue.


How to get PSNR and loss value from result image at the testing

Hello,
Can you help me to get and display the psnr and loss value of the result image at the testing?
Thank you

Hello,
Can you help me to get and display the psnr and loss value of the result image at the testing?
Thank you

For the PSNR, you can refer to PSNR section in tutorial.ipynb. For the loss, you can refer to nntools.evaluate()(link).


Pass custom ellipsoid to tileset and raster overlays

Follow up to #722.
Passes the custom ellipsoid to TilesetOptions and RasterOverlayOptions. This fixes tilesets with region bounding volumes that use custom ellipsoids. I don't think this has any affect on Cesium Moon Terrain since it uses box, but it's a good fix nonetheless.

Looks good to me.


Validate kubeReserved and systemReserved resources against Node allocatable resources

How to categorize this PR?

/area quality
/kind bug
What this PR does / why we need it:
This PR introduces a validation for kubeReserved and systemReserved resources against the allocatable resources of a  machine. The sum of kubeReserved and systemReserved resources (CPU, memory) for a worker should be less than the allocatable resources for the same. These reserved fields in the shoot yaml are validated against the corresponding machine type in the cloudProfile. This validation is introduced for both creation of new shoots and updation of existing shoots.
Which issue(s) this PR fixes:
Fixes #6106
Special notes for your reviewer:
Release note:

/test all

/test all

@shafeeqes: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:
Test name	Commit	Details	Required	Rerun command
pull-gardener-unit	ec47987 link	true	/test pull-gardener-unit
Full PR test history. Your PR dashboard. Command help for this repository.

That is weird, the unit tests pass locally for me without any issues.


Add callbacks for AccountRequest

Partially covers #64

Coucou @Morendil

Hi @maukoquiroga - sorry for the delay.
I've hooked up this PR's review app to /compte-test-validation on Slack, so we can test. At the moment the relevant Heroku app doesn't have the COMPTE_TOKEN environment variable, so no actions will be executed.

With no parameters: I receive "A la demande de @hussardnum je créée un compte pour" from betaGouvBot, and "Darn – that slash command didn't work (error message: 500_service_error). Manage the command at betaGouvBot." KO
With a single incorrect parameter (titi.toto): I receive "A la demande de @hussardnum je créée un compte pour titi.toto" - should be "Je ne vois pas de qui tu parles" KO
With two incorrect parameter (foo bar): I receive "A la demande de @hussardnum je créée un compte pour foo" - should be "Je ne vois pas de qui tu parles" KO

It dawned on me that this behaviour was the same no matter how many parameters I supplied, so I went over to https://betagouvbot-pr-92.herokuapp.com/compte and got this:

"Zut, il y a une erreur: undefined method `call' for BetaGouvBot::AccountRequest:Class
Did you mean?  caller"

So I think this still needs work - I haven't looked at the text of the commits and, I encourage you to focus on the externally visible behaviour as a priority, even if this means relying primarily on what seems like "manual testing" at the outset; it's really more a matter of exploring how the Slack API works, so that we have confidence that what our tests specify is in fact the right thing to demand of the code.

With no parameters: "Le format du nom doit être prenom.nom" OK
With a single incorrect valid parameter (titi.toto): "L'email doit être présent et être valide" OK
With two incorrect in/valid parameters (foo bar): "L'email doit être présent et être valide" OK
With two incorrect valid parameters (titi.toto foo@b.ar): "Le mot de passe doit être présent"  OK
With three incorrect valid parameters (titi.toto<EMAIL_ADDRESS>asdf): "Je ne vois pas de qui tu veux parler"  OK
With three correct valid parameters (mauko.quiroga<EMAIL_ADDRESS>asdf): "OK, création de compte en cours !"  OK

Note: format validations are triggered before content checking (does titi.toto exist ?). The behaviour has remained unchanged.

We're nearly there - error messages are being returned correctly, but we should also suppress echoing "A la demande de, etc" to the channel if there are any errors

Done.

@Morendil I propose you to open an issue to address that case. There's already a catch that'll log that error, BTW, but not send it back to the user.
Alternatively, just add the length constraint yourself (I'm not in front of a machine right now), shouldn't  take long to do so.
That being said, beware : we're enforcing our own security standards, which is not a bad thing, but we should keep it in mind.


Ensure that dekorate dependencies don't end up in uber-jar

Why don't you added this exclusion for the client then @geoand ?

I thought the client didn't contain dekorate, but I was wrong :)

LGTM. Size of the client uber jar is now 20MB instead of 38Mb. Many thanks

You are welcome!

Just to be on the safe side:
@iocanel removing the dekorate jars from the spring boot uber jar should not create any problems, correct?

@cmoulliard the k8s-dev-e2e-test job seems to be failing, is that expected?

I should mention that it fails with a microk8s error: - Download snap "microk8s" (581) from channel "1.13/stable" (stream error: stream ID 1; PROTOCOL_ERROR)

So Spring Boot also include compile dependencies?

compile is the maven default, no? And maven doesn't have any compile-only unfortunately...


No mention of the SQL language functions and features that are or are not supported via Synapse Link

Good day,
I am trying to find a summary of what can and can't be done via the Synapse Link into a Cosmos Analytics Store and can't seem to find anything.
As an example I have been unable to use a for json path with an outer apply when querying the Analytical Store, which returns:

The query references an object that is not supported in distributed processing mode.

and most frustratingly it appears filters applied within the where clause only work when there are matches for all of them?  For example, lets say I have two documents that:
{"pk": 1, "id": 1, "value": "00000000-0000-0000-0000-000000000001"}
{"pk": 2, "id": 2, "value": "00000000-0000-0000-0000-000000000002"}

When I run a select that filters for either:
where value in('00000000-0000-0000-0000-000000000001','00000000-0000-0000-0000-000000000002')
where value '00000000-0000-0000-0000-000000000001' or value = '00000000-0000-0000-0000-000000000002'
this works as expected and I get both documents back, but if I filter for a value that doesn't exist:
where value in('00000000-0000-0000-0000-000000000001','00000000-0000-0000-0000-000000000003')
I get no results at all instead of just the document that maches (in this case pk=1 id=1).
Is this a limitation within the Analytical Store that could really do with being documented or is this a bug?
Thanks,
Dave

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 4de135e2-fe93-6b4b-c530-bebb64be7c3b
Version Independent ID: 8a0b0b27-ad2b-f372-1199-4ffdc614e91f
Content: Azure Synapse Link for Azure Cosmos DB supported features - Azure Synapse Analytics
Content Source: articles/synapse-analytics/synapse-link/concept-synapse-link-cosmos-db-support.md
Service: synapse-analytics
Sub-service: synapse-link
GitHub Login: @Rodrigossz
Microsoft Alias: rosouz

Having double checked what I wrote above, this has to be a bug in the implementation?
This is what I am seeing at present:
When querying the openrowset based view without a filter:

Filtering for on existing value:

Filtering for two existing values:

Filtering for the same two existing values and a third non-existant value:

This can't be the expected behaviour?

We are checking this. Tks for the feedback.

@helloyesthisisdave
Thanks for your feedback! We will investigate and update as appropriate.

Hello @helloyesthisisdave . We will your help to better understand the situation. Can you email us at<EMAIL_ADDRESS>with the details below?

Subscription Id
Synapse Workspace name
Cosmos DB database account name
Container name
Statement ID of these query execution that has a strange result

I will close this issue since it's more used for documentation problems. Let's address this issue in emails ok?
Thank you!
#please-close

Hi @Rodrigossz,
I have tried to provide this additional info, but it appears that the email you have given is not a valid one?

Ops. We are checking that too. Please email me<EMAIL_ADDRESS>.
Thank you!

@Rodrigossz, this is quite a significant bug if confirmed, please can we keep this issue open or moved to the correct GitHub repo as an issue. Thanks.

We are addressing this case in emails and support tickets. I will keep the issue closed because this is a documentation repo.

@memblin-taurus Following the support ticket I raised, the mitigation that has been given which has resulted in the support agent looking to close the ticket is as follows:
In my instance, this was related to having an openrowset that defined the data types via a with block.  The BoxId column was defined as a uniqueidentifier as that is the data that is being stored in Cosmos. e.g:
create or alter view dbo.MyCosmosView
as
select c.BoxId
      ,<other cols>
from openrowset( ... )
with(BoxId uniqueidentifier '$.BoxId.string'
    ,<other cols>
    ) as c

And to ensure that the data filters and returns correctly, I have been told to return the Cosmos data as a varchar and then convert to a uniqueidentifier within the select:
create or alter view dbo.MyCosmosView
as
select cast(c.BoxId as uniqueidentifier) as BoxId
      ,<other cols>
from openrowset( ... )
with(BoxId varchar(36) '$.BoxId.string'
    ,<other cols>
    ) as c

This is not really an ideal solution and certainly isn't an obvious one given how the product is clearly designed to work and how the documentation explains things.


run-dev: Rewrite development proxy with aiohttp

This allows request cancellation to be propagated to the server.

Merged, thanks @andersk!


Snackbar stays behind AlertDialog

when an AlertDialog is being displayed and I want to show a Snackbar message, Snackbar stays behind the AlertDialog like image below:
I'm using flutter for desktop windows.

@soheilqk
Can you provide flutter doctor -v and a complete minimal reproducible code sample that we can use to replicate this issue ?
Please provide all details in formatted text and avoid posting screenshots.
Thanks.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


What's required to provide query access of views?

I'm not familiar with PostgreSQL, but now have a project that requires its use which is great.  I have very restricted permissions to the PostgreSQL database, with the developers of the database blocking access to the Tables, and exposing access exposing all table data in views.  With the LINQpad PostgreSQL driver installed in LINQPad it's possible to establish a connection to the database, however the only collections in LINQPad are Tables, and Functions.  The Tables collection is empty, while Functions is populated.  What would be required to display the Schemas, and Views collections?

This driver supports PostgreSQL views in LINQPad; https://github.com/linq2db/linq2db.LINQPad/releases


Automatically generated figures in the doc

WIP PR to use doctest to automatically generate figures that sit in our doc.
Left to do:

[ ] transparent background / darkmode compatible figures
[ ] hide line with comment # hide from the resulting doc
[ ] run doctest in the doc building CI

Will be replaced soon, thanks for the good starting point!


Don't process packets sent by dead players

This is a potential fix for #2396
The first fix is for the EntityPlayerMixin to ensure a consistent isDead state, When a player dies the isDead flag is set to true however when the player relogs without respawning this flag isn't updated on login resulting in a player whose health is 0 and isn't marked as dead.
The method has also been renamed from recalculateXpOnLoad -> onReadEntityFromNBT as it's no longer just recalculating xp.
The second fix is for the PacketPhaseUtil as this is where all packets end up getting processed on the Server Thread, The idea is that when a player dies the only thing they can realistic do is Respawn or Disconnect and as such all packets that aren't a Respawn packet are simply ignored.
The KeepAlive packet is processed on the Netty Thread so it will continue to function.
I have put the logic for this before the try-with-resources I'm not sure if this correct? My reasoning is that there is no point in Sponge attempting to track these packets if they are going to be ignored anyway.

To note this partially fixes #2396.
The second issue is that isDead is set to true after the player has ticked, meaning that isDead stays as false while the packets are processing. For EntityLivingBase it takes 20 ticks for the entity to be declared as isDead due to the death timer.

@isokissa3 Thanks for pointing that out, didn't realize the isDead flag was delayed by the deathTime.
I've changed the check from isDead to isEntityAlive() which fixes the death time issue as the method checks isDead and also the players health. I've also tested with a plugin listening to the MoveEntityEvent and from my testing the event doesn't get fired after the entity has "died" (When it's health is at 0).

@LXGaming Does this block mod provided packets sent after the player is dead ?
That's something we should probably avoid.

Tested with JEI and the cheat item functionality still worked, Looks like Forge is injecting its NetworkDispatcher into the netty pipeline so its before the vanilla packet handler, so no this doesn't appear to block mod packets.
Might have to allow the CPacketCustomPayload just in case a mod calls checkThreadAndEnqueue.


Add "how to cite dockstore"

Is your feature request related to a problem? Please describe.
Dockstore receives citations from researchers but might not know how best to cite us. See citations for some examples of how citations look for a) the site as a whole and b) for individual pages

Describe the solution you'd like

A part of the docs that explains how to cite Dockstore and a copy-paste-able citation. Also in the readme.

A link from the ui (perhaps in the footer?)

Nice-to-have: Instructions on how to cite individual pages properly or a "cite this page" link in the UI. Include author information in the citation if they are linking an entry with that info. (DOI if available?)
Describe alternatives you've considered
Just adding docs for this.
Tell people wanting to cite their workflows to generate a DOI using zenodo instead of citing us directly.
┆Issue is synchronized with this Jira Story
┆Fix Versions: Dockstore 1.14.x
┆Issue Number: DOCK-2432
┆Sprint: Backlog
┆Issue Type: Story

Note for implementer, can build off of:
https://github.com/dockstore/dockstore/blob/develop/CITATION or badge at https://github.com/dockstore/dockstore but adding it to the UI makes sense too.
https://docs.dockstore.org/en/stable/faq.html?highlight=citation#how-do-i-cite-dockstore needs updating to the NAR paper

from https://badge.dimensions.ai/?identifier=pub.1137547513#build


[docs] Add instructions for testing operators

Add instructions for end to end testing of operators, starting from their bundle, pushing to Quay and installing their operator.

/cc @SamiSousa @dmesser

/lgtm


The map point tool should match the icon that is being used on other add-ins

We need to have continuity across all of our add-ins. We need to change this icon to the same icon that we are using on the Distance and Direction and the Visibility add-ins.

@topowright Can you add a screenshot of which one should be used? Thnx.

@topowright updated, please verify

Assigning to @topowright to verify


Race oligath

Added race Oligath
Oligath
Level 1

Attribute Increase - Strength +2
Attribute Increase - Constitution +1
Language - You know Common and Giant.
Move - 6
Powerful Build - You count your size as Large when determining your carrying capacity.
Hot-Blooded - You have resistance to cold damage.
Stone's Endurance - When taking damage, you can use your reaction to roll a d12 + Constitution modifier to reduce damage. You can use this power equal to your proficiency bonus, and regain all expended uses after a long rest.

Are we sure we need 2 more interfaces? Can’t we use the existing ones that trigger before damage?

Need it to apply the reaction on damage receive so that the damage amount is already calculated before prompting the reaction.
Also the existing on damage receive only triggers when character is still alive.
The new also is for handling cases that damage kills the character.

@EnderWiggin , can you please review this one?

@EnderWiggin is working on a fix for the damage reduction. Wondering if we would be able to leverage that one setting damage reduction to 0.

Need it to apply the reaction on damage receive so that the damage amount is already calculated before prompting the reaction (similar to kindred spirit share pain).

Why you need damage to already be calculated?

Also the existing on damage receive only triggers when character is still alive.
The new also is for handling cases that damage kills the character.

For one, I'm not sure dead characters should react. And second - existing FeatureDefinitionReduceDamage triggers before damage is applied, so it can be used to reduce damage and prevent death.

Need it to apply the reaction on damage receive so that the damage amount is already calculated before prompting the reaction (similar to kindred spirit share pain).

Why you need damage to already be calculated?
This is similar to how the tabletop works. The damage is rolled before player decides to use reaction to reduce damage. Having it trigger before damage will make the feature less useful.

Also the existing on damage receive only triggers when character is still alive.
The new also is for handling cases that damage kills the character.

For one, I'm not sure dead characters should react. And second - existing FeatureDefinitionReduceDamage triggers before damage is applied, so it can be used to reduce damage and prevent death.

Can try to rework, but see concern above. Also seems it currently not support rolled damage reduction?


Update README.md

Doesn't work to pull the source with protocol.

Thanks!

published gh-pages -> 107b4ea


Disable faulty Windows Clang job

Description
The problem is that the Windows CI image hasn't yet be updated to Clang 16. See this PR for progress on fixing it
https://github.com/actions/runner-images/pull/8134
Running clang-tidy in Debug mode also means we avoid running another Windows Clang Static Release job. In general it's probably good to keep running clang-tidy across debug configurations though so that change can stay.
Here's what CI on master currently looks like: https://github.com/SFML/SFML/actions/runs/5954008177

Even though the fix for this has been merged we still sometimes get failures. The failures are pretty sporadic so I'm erring on the side of caution and disabling all jobs that use Clang on Windows. We will need to keep these workarounds in place until GitHub Actions runners are finally shipping with Clang 16.

https://github.com/SFML/SFML/actions/runs/6053687811
I'm glad I removed all Clang Windows jobs because now all five of them are consistency failing. Previously only a subset of these five were failing.

https://github.com/ChrisThrasher/argon/actions/runs/6104730037/job/16567174833#step:3:15
In another project I finally can see it using Clang 16. Most jobs still use Clang 15 but hopefully this will be fixed shortly and we can restore the Windows Clang jobs.


CRCError for NIfTI in working directory

@effigies and @mgxd Thanks. Now I was able to process one subject fMRI data without any errors and I still didn't get a time to check those results. I also processed two other subjects and got the following two different errors from them.
Traceback (most recent call last):
  File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/pipeline/plugins/multiproc.py", line 67, in run_node
    result["result"] = node.run(updatehash=updatehash)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/pipeline/engine/nodes.py", line 516, in run
    result = self._run_interface(execute=True)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/pipeline/engine/nodes.py", line 635, in _run_interface
    return self._run_command(execute)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/pipeline/engine/nodes.py", line 741, in _run_command
    result = self._interface.run(cwd=outdir)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/interfaces/base/core.py", line 428, in run
    runtime = self._run_interface(runtime)
  File "/usr/local/miniconda/lib/python3.8/site-packages/niworkflows/interfaces/nibabel.py", line 191, in _run_interface
    img_4d = nb.concat_images(nii_list)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nibabel/funcs.py", line 141, in concat_images
    raise ValueError(f'Affine for image {i} does not match affine for first image')
ValueError: Affine for image 3 does not match affine for first image

Traceback (most recent call last):
  File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/pipeline/plugins/multiproc.py", line 67, in run_node
    result["result"] = node.run(updatehash=updatehash)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/pipeline/engine/nodes.py", line 516, in run
    result = self._run_interface(execute=True)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/pipeline/engine/nodes.py", line 635, in _run_interface
    return self._run_command(execute)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/pipeline/engine/nodes.py", line 741, in _run_command
    result = self._interface.run(cwd=outdir)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/interfaces/base/core.py", line 428, in run
    runtime = self._run_interface(runtime)
  File "/usr/local/miniconda/lib/python3.8/site-packages/niworkflows/interfaces/nilearn.py", line 176, in _run_interface
    new_nii = concat_imgs(self.inputs.in_files, dtype=self.inputs.dtype)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nilearn/_utils/niimg_conversions.py", line 471, in concat_niimgs
    niimg = check_niimg(niimg, ensure_ndim=ndim)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nilearn/_utils/niimg_conversions.py", line 283, in check_niimg
    niimg = load_niimg(niimg, dtype=dtype)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nilearn/_utils/niimg.py", line 134, in load_niimg
    dtype = _get_target_dtype(_get_data(niimg).dtype, dtype)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nilearn/_utils/niimg.py", line 24, in _get_data
    data = np.asanyarray(img._dataobj)
  File "/usr/local/miniconda/lib/python3.8/site-packages/numpy/core/_asarray.py", line 171, in asanyarray
    return array(a, dtype, copy=False, order=order, subok=True)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nibabel/arrayproxy.py", line 391, in __array__
    arr = self._get_scaled(dtype=dtype, slicer=())
  File "/usr/local/miniconda/lib/python3.8/site-packages/nibabel/arrayproxy.py", line 358, in _get_scaled
    scaled = apply_read_scaling(self._get_unscaled(slicer=slicer), scl_slope, scl_inter)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nibabel/arrayproxy.py", line 332, in _get_unscaled
    return array_from_file(self._shape,
  File "/usr/local/miniconda/lib/python3.8/site-packages/nibabel/volumeutils.py", line 519, in array_from_file
    infile.seek(offset)
  File "indexed_gzip/indexed_gzip.pyx", line 651, in indexed_gzip.indexed_gzip._IndexedGzipFile.seek
indexed_gzip.indexed_gzip.CrcError: CRC/size validation failed - the GZIP data might be corrupt (file: /gpfs/projects/fmriprep/scR21_outputs_Sep_2021_N/derivatives/scratch/fmriprep_wf/single_subject_50144_wf/func_preproc_ses_1_task_RSFC_run_4_wf/bold_t1_trans_wf/bold_to_t1w_transform/vol0412_xform-00412.nii.gz)

Originally posted by @sameera2004 in https://github.com/nipreps/fmriprep/issues/2534#issuecomment-922476583

@sameera2004 Is there any chance that you've hit a disk quota or run out of space? And if not, can you try opening the file in question (/gpfs/projects/fmriprep/scR21_outputs_Sep_2021_N/derivatives/scratch/fmriprep_wf/single_subject_50144_wf/func_preproc_ses_1_task_RSFC_run_4_wf/bold_t1_trans_wf/bold_to_t1w_transform/vol0412_xform-00412.nii.gz) to verify that it is corrupted?
My next suggestion would be to delete the containing directory (/gpfs/projects/fmriprep/scR21_outputs_Sep_2021_N/derivatives/scratch/fmriprep_wf/single_subject_50144_wf/func_preproc_ses_1_task_RSFC_run_4_wf/bold_t1_trans_wf/bold_to_t1w_transform) and re-run. It seems unlikely to reproduce reliably.

Thanks @effigies No I have not run out of space. No file is not corrupted. I will rerun the analysis after deleting the directory.

@sameera2004 It would be great if you would be able to share one of the images for which the CrcError is occurring - it is probably due to a bug in indexed_gzip, but I have not been able to reproduce it myself.

@effigies and @pauldmccarthy,I am so sorry I have been busy with so many other projects. Therefore, I didn't get a time to check this. Last Friday, I reran the analysis after deleting the directory. However, it was not successful. I will check those error reports and share you an image by tomorrow. Thank you so much.

No worries! I appreciate you taking the time to report bugs and follow up.

After removing "(/gpfs/projects/fmriprep/scR21_outputs_Sep_2021_N/derivatives/scratch/fmriprep_wf/single_subject_50144_wf/func_preproc_ses_1_task_RSFC_run_4_wf/bold_t1_trans_wf/bold_to_t1w_transform" ) and re running the pipeline I don't see the CrcError. However, I am getting the following.
Traceback (most recent call last):
File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/pipeline/plugins/multiproc.py", line 67, in run_node
result["result"] = node.run(updatehash=updatehash)
File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/pipeline/engine/nodes.py", line 516, in run
result = self._run_interface(execute=True)
File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/pipeline/engine/nodes.py", line 635, in _run_interface
return self._run_command(execute)
File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/pipeline/engine/nodes.py", line 741, in _run_command
result = self._interface.run(cwd=outdir)
File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/interfaces/base/core.py", line 428, in run
runtime = self._run_interface(runtime)
File "/usr/local/miniconda/lib/python3.8/site-packages/niworkflows/interfaces/nibabel.py", line 191, in _run_interface
img_4d = nb.concat_images(nii_list)
File "/usr/local/miniconda/lib/python3.8/site-packages/nibabel/funcs.py", line 141, in concat_images
raise ValueError(f'Affine for image {i} does not match affine for first image')
ValueError: Affine for image 3 does not match affine for first image
Today I will run another subjects data and see whether I can reproduce CrcError.

@sameera2004 It would be great if you would be able to share one of the images for which the CrcError is occurring - it is probably due to a bug in indexed_gzip, but I have not been able to reproduce it myself.

@pauldmccarthy Can you please let me know which image you want? Do you need input nifity image?

Yes, I was able to preprocess the 6 subjects data without any issue this time. I didn't see this error again. We will compare the outputs and let you know whether it solved the SDC issue. Thank you.

Dear experts, I got the same error message about
File "indexed_gzip/indexed_gzip.pyx", line 653, in indexed_gzip.indexed_gzip._IndexedGzipFile.seek
indexed_gzip.indexed_gzip.CrcError: CRC/size validation failed - the GZIP data might be corrupt (file: /gpfs/projects/CanliGroup/hcpa_pilot/script/work_fmriprep_v2023/fmriprep_wf/single_subject_9913090_wf/func_preproc_task_facename_dir_PA_wf/bold_std_trans_wf/_std_target_MNI152NLin2009cAsym./bold_to_std_transform/vol0188_xform-00188.nii.gz). Please kindly find the crash text file below.
I tried to delete the error directory: (/gpfs/projects/CanliGroup/hcpa_pilot/script/work_fmriprep_v2023/fmriprep_wf/single_subject_9913090_wf/func_preproc_task_facename_dir_PA_wf/bold_std_trans_wf/_std_target_MNI152NLin2009cAsym./bold_to_std_transform), but I still got the same error (the same error file: vol0188_xform-00188.nii.gz).
What might be causing this error? Is there a way to fix it?
For your reference, I processed 12 subjects and this is the only subject that failed.
o_9913090_crash.txt

Hello, yes. Please kindly find the attached error file:
o_9913090_vol0188_xform-00188.nii.gz
Any input or suggestions would be very helpful. Thank you so much.

@yenwchen If your HPC is managing its own fMRIPrep setup (rather than using a Singularity image), then yes, asking an admin to update the environment to include indexed-gzip 1.6.4 is the correct fix.


DataGrid ScrollIntoView() works only from simple ObservableCollection

Describe the bug
DataGrid.ScrollIntoView() works only with simple ObservableCollection (i.e. contains only simple types like string, int, etc.). Doesn't work, if the ItemsSource is a DataTable, ObservableCollection<string[]>, ObservableCollection contains only a string[], etc.
In the second case the DataGrid itself works well, shows the values correctly, but the ScrollIntoView() does nothing.
To Reproduce
Create a simple project with DataGrid, and try to change the ItemsSource.
public class data
{
    public string date { get; set; }
    public string name { get; set; }
    public string description { get; set; }
}  => ScrollIntoView() works well.

public class data2
{
public string[] sArray { get; set; } = new string[10];
}  => no scrolling
datagrid.ItemsSource = myDataTable.DefaultView; => no scrolling
Expected behavior
No response
Avalonia version
11.1.1
OS
Windows
Additional context
It would be important if you need a dynamic number of columns. In this case you shouldn't use a static type, it's required a dynamic type for example array, DataTable as ItemsSource.

I have the same problem，Because the number of attributes in the data source was uncertain, I used dynamic objects, and when I called the ScrollIntoView method to locate a row, it didn't take effect


HandshakeError: Header HTTP_UPGRADE is not defined

I've run wsgiref example code with this JS:
 var connection = new WebSocket('ws://localhost:9000/');
  connection.onmessage = function(e) {
    console.log('Server: ' + e.data);
  };
  connection.onopen = function() {
    connection.send(JSON.stringify({message: 'hello'}));
  };

I I've got this error but the socket is working, I get {message: 'hello'} in response. I'm using Python 2.7.10.

Hey @jcubic,

I've got this error

What error?

It was year ago, probably the error from the title.

Whoops, didn't see the timestamp :P

I faced the same error when using wsgiref example with apache and nginx:
Traceback (most recent call last):
  File "/usr/lib/python3.5/wsgiref/handlers.py", line 137, in run
    self.result = application(self.environ, self.start_response)
  File "/home/pi/.local/lib/python3.5/site-packages/ws4py/server/wsgiutils.py", line 101, in __call__
    raise HandshakeError('Header %s is not defined' % key)
ws4py.exc.HandshakeError: Header HTTP_UPGRADE is not defined


Feature idea: Rake task to diff Rubocop.yml against solidus master

It'd be nice if our ecosystem followed a consistent style guide. I believe that was the impetus behind including Rubocop in the default build task. However, it's natural that .rubocop.yml will fall out of sync with https://github.com/solidusio/solidus/blob/master/.rubocop.yml overtime. In order to keep the ecosystem consistent, we could add a rake task which diffs the current master version of .rubycop.yml against your local copy.

Ooo, is this something we could throw in a spec? It's not that solidus_cmd sees a ton of movement, but having CI fail is an easy way to throw a warning flag for something that is always a trivial fix.


Cs79 patch 1

Enable codecov in workflow

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@a029643). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   95.71%
=========================================
  Files             ?        4
  Lines             ?      140
  Branches          ?        0
=========================================
  Hits              ?      134
  Misses            ?        6
  Partials          ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a029643...9555672. Read the comment docs.


[feataure] Sort directories first

À la tree --dirsfirst, an option to sort directories above normal files.
In my view, there are two ways to implement this:

et --sort dir, a manually-implemented "sort by directory"
et --sort ... --dirs-first, an option for all sorting methods to sort directories first

I have both implemented, don't know which (if either) to make a pr for.

Probably should have left a link to see how I have them implemented

et --sort dir
`et --sort ... --dirst-first

Hey @bryceberger because you're now officially a contributor feel free to just make PRs. I'll make note of the fact that existing contributors don't need to ask for permission going forward. But this sounds great!

Alright. I prefer the second (switch for all algorithms) implementation, so I'll open a PR for that.


Difference in no-gost and -DOPENSSL_NO_GOST

Hi there
I am reading the compilation and installation wiki: https://wiki.openssl.org/index.php/Compilation_and_Installation and going through the configure options.
If I want to disable a cipher, for example gost, should I use no-gost or -DOPENSSL_NO_GOST as my configure option. Are they both essentially doing the same thing and it doesnt matter which one do I use?
Thanks

no-gost is correct.

Thanks!


Prop idling in ALTCTL and POSCTL

@kd0aij This is good for testing. Hopefully the prop now idles without GPS lock and althold works without GPS as well.

@LorenzMeier I assume this needs testing after your changes?

Yes, and I'm not 100% finished yet. We do not cover the case of no GPS -> GPS yet, and it could induce an uncommitted altitude change (from the perspective of the user).

@tumbili This is now good to go.  If not, the diff your guide is, Luke.

@LorenzMeier This is bench tested, and everything works as expected. I increased the idle to 0.2, think that should be ok. I can flight test this in the afternoon.

cool. Make sure to fly an ALTCTL mission with initially no GPS (if possible) and then GPS lock mid-air. I want to be sure it does the right thing and resets the altitude set point.

@LorenzMeier This works like a charm. But I wasn't able to the gps from getting a fix right away. Is there a way how the put a delay?

http://dash.oznet.ch/view/NXLhWbxtas5ndE2m6z7RYk

@LorenzMeier I'm wondering if we should do a deadzone for the pitch stick in altctl. Often when I fly turns I apply some pitch and the plane looses altitude. I thought it was the controller but the logs showed what actually happened.

Go for it. Makes a lot of sense.

@tumbili Is that last log from your Ninox?

@kd0aij It's the plane of Lorenz, also one from Bormate but not sure how it's called. It's even small than the Ninox. BTW, I didn't tune one parameter :-D

probably the "Q" then.

@kd0aij I just see that the roll setpoint is always zero in ALTCTL

My Ninox seemed to add power and dive when I switched to ALTCTL. Do you think that means trim airspeed is too high?
I guess rollSP isn't published in ALTCTL?

@kd0aij At what value do you have the trim airspeed. Could well be that it tries to pick up speed.

This is strange. Airspeed min/trim/max params are 10/15/20.
I enter ALTCTL at about 18m/sec then thrust jumps from 50 to 70% and the plane climbs 25m as airspeed drops all the way to 5m/sec. It then takes 17 more seconds to reach trim airspeed and starting altitude; about 25 seconds of confusion before it gets back right. Horrible tuning?
I haven't changed anything but L1 period since the POSCTL/heading lock test which behaved correctly.

@LorenzMeier @tumbili Is there a better way than looking at vehicle_status.h to figure out what the values of main_state mean? QGC reports friendly values like "ALTCTL", but the logs just show integer values.

@kd0aij Sometimes if I'm nor sure I have a look at the rc values and together with the RC_MAP params you can figure it out. Can you post the log here?

https://copy.com/XzrHSOwbrChsjdDK

Did you set the trim throttle to 70? If it's too high then the integrator needs to settle first. You need to get throttle trim and airspeed trim to match.

Ah, I didn't know about that one: FW_THR_CRUISE defaulted to 0.7. Looking at the plot, I guess it should be 0.25

I bench tested this today (OK with GPS lock), but wasn't able to get the Ninox in the air. Overheated ESC, I think.


Is Feedbunch dead?

Been having issues for several days, and now I can't export my feeds OPML.... Help?

A server was having memory issues and it had slipped undetected. Should be fixed now. Can you try exporting your subscriptions again? Sorry for the inconvenience.

I was able to export OPML, thank you. Something is still preventing the site from working for me, however. But now I can use a different reader for the feeds, at least.

What error are you experiencing, exactly? What do you mean by "preventing the site from working"? I'd like to try to fix it, but I'm not aware of any current issues.

Sometimes when I try to load the page, it just spins and spins and spins. Other times is crashes entirely on loading. This started about a week ago. Of course now that I want to grab a screenshot of the error, I can't replicate it. 🙄 Maybe it's fixed for me.

I've found that the problems with background jobs, which blocked you from exporting your OPML, were deeper than I thought. Even after fixing the memory issues a Redis database had a bad corruption problem, probably caused by the memory issues in the first place. I've recreated the Redis instance from scratch and background jobs are again running smoothly.
I'm investigating what may cause those 522 errors you're seeing. For the moment I'm unable to reproduce them.

I'm afraid I can't find the cause for those 522 errors. Feedbunch logs show no errors at the time the error in that screenshot happened, the service was up and running fine. I've checked Cloudflare logs as well, but they give too little information to be useful. The only thing I can think of is a temporary network error either in Cloudflare or the datacenter that hosts FeedBunch servers, but I cannot be sure.
In any case it looks like a transient error, I haven't been able to reproduce it. Closing the issue for now, might reopen if it happens again.


[Snyk] Security upgrade snyk from 1.518.0 to 1.667.0

Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.

Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

server/package.json
server/package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

551/1000  Why? Recently disclosed, Has a fix available, CVSS 5.3
Denial of Service (DoS) SNYK-JS-JSZIP-1251497
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.

Commit messages

Package name: snyk
    The new version differs by 250 commits.

8387aed Merge pull request #2111 from snyk/fix/update-jszip
e686299 fix: update plugins package versions
7f0b32e Merge pull request #2108 from snyk/smoke/binary-cdn
e974b9d test: use binary from cdn for smoke tests
64e3fe8 Merge pull request #2107 from snyk/refactor/analytics-tests
0f2a7e3 refactor: call anaytics module the right way
0b833f6 refactor: use await syntax rather than callbacks
fce83d5 Merge pull request #2096 from snyk/feat/iac-tracking
f7f2961 feat: track IaC local execution tests [CC-972]
9e7f79a Merge pull request #2104 from snyk/feat/centralize-colors
3292ffb feat: Centralize icons and err messages by theme
e3c3ac0 feat: central colors theme
2aecd40 Merge pull request #2074 from snyk/docs/update-protect-readme
0203c6c Merge pull request #2100 from snyk/fix/propagate-error-message
7416bc9 docs: update protect readme
42288e0 fix: propagate userMessage to 404 errors
77336a0 Merge pull request #2099 from snyk/test/migrate-trust-policies-jest
e2106f1 test: migrate trust-policies to jest
b227f78 Merge pull request #2095 from snyk/protect/smoke-tests
20b05d0 Merge pull request #2087 from snyk/chore/async-test-with-polling
d7f23e0 Merge pull request #2098 from snyk/fix/bump-docker-plugin
cb2ecf1 fix: bump docker plugin version with fixes
697f81f chore(protect): fix production test caching
d21cff7 chore: init support async test with polling

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Will update dependencies manually


Removed a content

There is no need to create a custom device policy to enable PIN reset recovery. So removing this portion.
Problem: https://github.com/MicrosoftDocs/windows-itpro-docs/issues/8284#issuecomment-707356130

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

@e0i - Ready to go live.

@joinimran : Thank you for the work you provide in pull requests and ticket comments & replies. Just a minor detail here in this PR ("English grammar tip of the day", if you like): the word "content" is usually considered to describe some undefined amount of mass or uncountable number of items. The most common usage is to skip "a", going straight to the word itself, so in this case the PR title would in most cases be "Removed content" (or "Removed outdated content").
Anyway, this is mostly an advice for future use. No need to change the PR title, because the commit name will stay unchanged.

#sign-off

Instead of removing this section, it should be replaced with steps for configuring the policy via the UI linked by CHDAFNI-MSFT

Instead of removing this section, it should be replaced with steps for configuring the policy via the UI linked by CHDAFNI-MSFT

cc @joinimran

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

Thanks, @illfated. This makes more sense.

@JohanFreelancer9 please copy review.

Thanks, @JohanFreelancer9.

Link to rich diff view of the image:

https://github.com/MicrosoftDocs/windows-itpro-docs/pull/8457/files?short_path=d82adde#diff-d82addeadf260315de8dde6cd3b4d68258bc3ab9865e09ca461d779301e141dc

(pick Swipe or Onion Skin to get a big enough image and a small tug handle at the lower left corner to view the text change in the box (in the middle of the image) when moving the handle)

(also ref. #8509 )

#sign-off

Oh. Is the issue that this MacOS image does not belong in this PR, MicrosoftDocs#8457 , which is about the Hello for Business Pin Reset feature?

Oh. Is the issue that this MacOS image does not belong in this PR, MicrosoftDocs#8457 , which is about the Hello for Business Pin Reset feature?

@illfated - That looks like for a different page and different content.


Missing fields for MeetingResponse?

https://docs.microsoft.com/en-us/exchange/client-developer/exchange-web-services/how-to-propose-a-new-meeting-time-by-using-ews-in-exchange mentions the ProposedStart, ProposedEnd and AssociatedCalendarItemId fields on the MeetingResponse item, but the documentation for MeetingResponse does not mention these fields:
https://github.com/MicrosoftDocs/office-developer-exchange-docs/blob/master/docs/web-service-reference/meetingresponse.md
Which of these is correct?

@ElizabethSamuel-MSFT Is this something you can look into or help reassign?

I added @jasonjoh to this request to update documentation.

@ecederstrand the item in the "how to propose" topic is an AcceptItem, not a MeetingResponse. https://github.com/MicrosoftDocs/office-developer-exchange-docs/blob/master/docs/web-service-reference/acceptitem.md


Attachments are not shown in user-message box in Playback feature

Create a chat based on a model which works with attachments (e.g. DIAL RAG application)
Sent several messages with attached documents
Create Playback chat based  -> attached documents are not shown in user-message box -> issue

Expected: we need to show attached documents in user-message box preview in Playback

Verified on staging server successfully


Typo

The intent was to set the selected_key to '', but the code
mistakenly use the selected_key as a function.

I configured emacs to remove the trailing whitespaces on save, this is why there are several diffs about removing some extra spaces.
I did not remove then from the patch, for I guessed we would agree it would be a good thing to remove them anyway. But if you like, I can provide a patch with only the functional diff.

Thanks! Nice catch! Yeah, removing the whitespace is fine.


build-profile: use existing RPM group instead to build rpms

Fixes #91

not to hijack the voting process, i'll change it if another group is chosen :smile:

LGTM


rename trackerdict, analysis1axis

Our workflow uses the trackerdict and analysis1axis even for fixed tilt systems. for multi-object systems, some can be tracked and others fixed.  We should come up with a different name for it all, to avoid confusing people.
Also, deprecate the non-trackerdict workflow in our tutorials and modelchain.

[ ] trackerdict  -> simdict
[ ] analysis1axis
[ ] makescene1axis
[ ] performance1axis
[ ] set1axis  -> setkeys
[ ] gencumsky1axis
[ ] gendaylit1axis

options: dict_analyisis?  sim_makeScene?


disable for large files until performance issues are addressed

There's a huge pr by kil0metres pending, I'll merge this after that

@kil0meters do i merge this before you're done?

Do you want me to merge this for now or are you working on that PR? @tzachar

I merged this, thank you very much!

10x :)
On Thu, Apr 29, 2021 at 5:00 PM Chinmay Dalal @.***>
wrote:

I merged this, thank you very much!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/p00f/nvim-ts-rainbow/pull/27#issuecomment-829262441,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABFXXC24F76I3OCEB5IP6D3TLFRAPANCNFSM42XO5ADQ
.


Status Code

What happened?
Hello,
I am currently trying to integrate the Status of my Growatt SPH-10000 BL-UP. Do anyone knows what the Status numbers mean?
Thank you for your help.

Shine X

Which Inverter are you using?

SPH-10000 BL-UP

Soweit ich weiß:
Status 1 Normalbetrieb
Status 3 Fehler

@CTRNER for SPH Storage Inverters Protocol Version 1.24 should work. As @ChrisHVL stated 1 is normal, 3 is fault and 0 is waiting.

This should go to a discussion page. Currently the BL-UP is not really supported. Feel free to open a pr to add support. Thank you!


Gallery of block images

This has been a request we've heard from several folks writing curriculum.
They would like access to a gallery (or maybe a generated zip file?) of all the core blocks in the Toolbox for a given editor, in a common image format (jpg, png).
This is for creating documents like this:

the description of the block.

We can generate this in an html page on fly and people can print it to pdf.

https://github.com/microsoft/pxt/pull/6776

Needs to be deployed.


Enable Dependabot for GitHub Actions

Summary of the new feature / enhancement
This repository already uses Dependabot but not for GitHub Actions used in workflows. I suggest enabling it to keep Actions updated.
I see some old (major) versions used in various workflows, with CodeQL v1 being probably the most important one because it will be deprecated in December.
https://github.blog/changelog/2022-04-27-code-scanning-deprecation-of-codeql-action-v1/
Proposed technical implementation details (optional)
Adding it is pretty simple, just append dependabot.yml:
  - package-ecosystem: "github-actions"
    directory: "/"
    schedule:
      interval: "daily"

@Molkree Can you please submit a PR for this?

@daxian-dbw Sure


Can librosa not be a strict dependency?

Given different versions of OSes with different versions of librosa, librosa is a huge pain to install on certain machines.
For example, I use lambdalabs GPUs and the librosa is wrong. This old librosa depends upon a particular pinned version of numba that's a huge pain to install:
ImportError: Numba needs NumPy 1.22 or less

Thus, I think librosa should be deprecated.

Hey @turian. Thanks for raising this. I think we add librosa as an optional dependency since its only needed for the mel filterbanks. Is scipy also a problem for you? I think we could move to be optional as well potentially.

If you want to take a look, I have started a branch that moves librosa and scipy to optional install by default.

scipy is generally fine. Just librosa with it's weird numba pin sucks so much.

I am happy the beta test when the branch is ready!

Great, could you give the librosa branch a try? I did some basic testing and things seemed to be working for me.

pip3.8  install --upgrade git+https://github.com/csteinmetz1/auraloss.git@librosa

appears to work. If you submit a PR, I can also code review the diff if you like

Thanks, see #37.

Should be solved with #37 now and merged to main.


Working directory wrong for latest OPC Publisher

For the OPC Publisher it refers to folders called /appdata but in the latest version of OPC Publisher the working directory is /app. This breaks the container persistance if not configured correctly.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f65acbb4-fdf1-9709-09cf-905c262ceebb
Version Independent ID: 34a0c5d1-8c91-4ccd-22f5-00628e555f66
Content: Run OPC Publisher - Azure
Content Source: articles/iot-accelerators/howto-opc-publisher-run.md
Service: industrial-iot
GitHub Login: @dominicbetts
Microsoft Alias: dobett

@nanowireUK Thank you for taking time to report the issue.
This is being assigned to the content author to evaluate and update the documentation as appropriate.

@barnstee - can you confirm that the working directory has changed. Looking at https://github.com/Azure/iot-edge-opc-publisher/blob/master/README.md, it's still showing as /appdata.

#reassign:barnstee

#reassign:marcshier

marcshier is not a valid GitHub ID, or is not a collaborator on this repo.

marschier is not a valid GitHub ID, or is not a collaborator on this repo.


Feature/nfdiv 1128 notify applicant2 apply for co

JIRA link (if applicable)

https://tools.hmcts.net/jira/browse/NFDIV-1128
https://tools.hmcts.net/jira/browse/NFDIV-1129
https://tools.hmcts.net/jira/browse/NFDIV-1130

Change description

send notification to applicant2 upon lapsing of the holding period, that they can apply for conditional order
send reminders to applicants 1 and 2 if 14 days passed and they haven't applied

Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X] No

Have approved rebase onto CO Submitted work for NFDIV-1053


Pystan unsupported

Changed installer.js and install page so that PyStan is no documented in "Other interfaces" section.

I've updated the information.  The troubleshooting section has no content - remove for now?  refer to CmdStan manual?

A few of the above comments from my prior review are still outstanding but otherwise LGTM


Build a new docker image

Hi! Thanks for working on this project.
Recently there's a critical bugfix in Copybara upstream and it'd be great if olivr/copybara image picks it up. It seems the image building action has been paused due to lack of activity.
https://github.com/Olivr/copybara-action/actions/workflows/copybara-docker.yml
Could you manually roll out a new docker build?
Thanks in advance!

@tsawada thanks for noticing! I have just restarted the workflow and the latest image has just been rebuilt.


Sync programmatically

I am wondering if there is a way (or if such feature is on the roadmap) to sync all files for a site programmatically, instead of using the page. This would be especially useful for multisite installations with several sites (like hundreds of sites). Thank you.

@deivamagalhaes
You can use Utility::add_media() method to sync programmatically.


[hotfix] Fixed spelling mistake of type.

Codecov Report

Merging #39 into master will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #39   +/-   ##
=======================================
  Coverage   90.75%   90.75%
=======================================
  Files           6        6
  Lines         346      346
  Branches       55       55
=======================================
  Hits          314      314
  Misses         32       32

Impacted Files
Coverage Δ

src/datastore/ephemeral.ts
84.78% <100.00%> (ø)

src/datastore/postgres.ts
92.30% <100.00%> (ø)

src/types.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9e9bd20...b46b99f. Read the comment docs.


[RFC] Tags support

E.g. I have a telegram bot with commands implementing some CommandInterface. And I have  HelpCommand which must show all available commands.
Now to achieve this I must edit HelpCommand configuration every time I add new command to the bot.
Using tags I can tag command as a telegram-bot-command so It will be automatically loaded by HelpCommand
Same feature in Laravel and Symfony

i.e. Container collects all the tagged classes, forms a list and passed the list as an argument of HelpCommand.

That could be useful but there are possible disadvantages as well: implicitness, unable to know what's passed to HelpCommand by looking the config.

unable to know what's passed to HelpCommand by looking the config

Catch me if I wrong, but in case non-FQCN service names it's almost the same level of knowledge:
return [
    SomeService::class => [
        '__construct()' => [Reference::to('some-dependency')],
    ],
]

and
return [
    HelpCommand::class => [
        '__construct()' => [Reference::tag('telegram-bot-command)],
    ],
]

In the first example you need to find some-dependency definition in all sources.
Of course it would be good to have some sort of debug tool. Something like Symfony's
bin/console debug:container --tag='console.command'

Without tags it will be:
return [
    SomeService::class => [
        '__construct()' => [
            Reference::to(Command1::class),
            Reference::to(Command2::class),
            Reference::to(Command3::class),
        ],
    ],
]

I mean without FQCN services it would be
return [
    SomeService::class => [
        '__construct()' => [
            Reference::to('command'),
            Reference::to('other-command'),
            Reference::to('some-other-command'),
        ],
    ],
]

So you have to use grep to find class names. Almost the same as with tags

We are usually using FQNs.

Yes, everybody does, but sometimes it's impossible.
Anyway, it's hard to implement described example without tags.
The problem is that developers focusing on creating new commands and can forgot / don't know to add it in HelpCommand.

Got it. Need to evaluate if it is possible without introducing compiler pass that I want to avoid.

https://github.com/illuminate/container/blob/master/Container.php#L476

It would be good to define tags with service definition, otherwise it's also easy to forget to update tags

@BoShurik an example would be helpful.

In Laravel's container definition of tags is separated from service definition
$this->app->bind('MemoryReport', function () {
    //
});

$this->app->tag(['SpeedReport', 'MemoryReport'], 'reports');

In that case it's also easy to forget to tag service.
I would prefer something like this:
return [
    SomeCommand::class => [
        '!tag' => 'telegram-command'
    ],
]

I would prefer something like this:
return [
    SomeCommand::class => [
        '!tag' => 'telegram-command'
    ],
]

We're using this approach in this or https://github.com/yiisoft/di/pull/120

https://github.com/yiisoft/di/pull/144


Some achivements dont load well and are blurred

Describe the bug
Some achivements added manually with steam dont load properly, but adding them with exophase load perfectly
To Reproduce
Steps to reproduce the behavior: Add manually a game achivements with steam
Expected behavior
All achivements added manually with steam should load fine
Screenshots
In Alba: A Wildlife Adventure none of the achivements load
In 20 Minutes Till Dawn all the achievements load fine even with steam
Extensions log
extensions.log

Nevermind, i just realised they were hidden achievements

You can configure how hidden achievements are displayed:


feat: add alias to the keywords tracked by move_decls

As reported in #13725.

Thanks!  I'll wait to see if @YaelDillies thinks that the new script works in the other PR: otherwise, in a couple of days I'll prod this along.

bors merge


_ to ignore mdx files consistently in all content plugins

🚀 Feature
We currently prefix pages with _ to avoid creating a page for certain files (ie src/pages/_myPartial.mdx)
We should allow similar things for blog / docs too. A file in docs/_partials/myPartial.mdx should not create any doc, but just be available for usage in other documents.
Also, we should allow to put partials in the src folder too (like src/partials/myPartial.mdx). I don't think we currently have a "global mdx loader" for the src folder

Related to https://github.com/facebook/docusaurus/pull/3196 and requested by @fnune

Fixed by https://github.com/facebook/docusaurus/pull/5173


iOS Compile error - 'search_choices_for_push-Swift.h' file not found

Getting the following error on building for iOS
XCode Version 11.3.1
Tried to compile example project
.../pub-cache/hosted/pub.dartlang.org/search_choices-1.0.12/ios/Classes/SearchChoicesForPushPlugin.m:8:9: fatal error: 'search_choices_for_push-Swift.h' file not found
#import "search_choices_for_push-Swift.h"
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.
note: Using new build system
note: Planning build
note: Constructing build description
flutter doctor (tested on stable and beta channels )
⇒  flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, v1.15.17, on Mac OS X 10.15.3 19D76, locale en-ZA)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 3.6)
[✓] IntelliJ IDEA Ultimate Edition (version 2019.3.3)
[✓] Connected device (3 available)
• No issues found!

Hello @jaspervdbijl ,
Thank you very much for letting me know. Indeed, I have the same issue right now. I'm looking for a solution and will let you know as soon as I learn something new.

Hello again @jaspervdbijl ,
The easy fix for me was to do the following in the plugin folder and the example folder:
flutter clean
rm -rf ios

And then do the following in the plugin folder:
flutter create .

I am not sure why this is happening. I will try to release a new version that fixes this issue.
Thanks again for letting me know.

Some files were missing. Latest version 1.0.13 fixes this issue by adding those files. Could you please let me know whether this is also the case for you or whether we cannot close this issue?

Verified working on 1.0.13
Thanks

Thanks @jaspervdbijl for the confirmation!


Add context manager for ExaConnection and ExaStatement

I'm using pyexasol in some production environment. There is a need to close connections and statements properly to avoid idle ones occupying too much memory on the Exasol cluster.
I added the context manager so that instead of
try:
    conn = pyexasol.connect(...)
    # do something
finally:
    if conn:
        conn.close()

we can write
with pyexasol.connect(...) as conn:
    # do something

That's a good idea. Please check new version 0.8.0.
The code is generally the same, but I've used slightly different arguments for __exit__() and added  an example with context management.
Hope it helps!

looks good. I'll close this PR.


Refresh when obsidian restarts

Fix: Make pinned tab styles persist across restarts
This PR fixes the issue where pinned tab styles don't persist after restarting Obsidian. Key changes:

Replace direct DOM manipulation with CSS stylesheet injection
Add workspace layout change event listener to reapply styles when needed
Make settings changes async for reliable saving
Add proper cleanup on plugin unload

The fix maintains all existing functionality while making the style changes persistent.
Closes #4
NOTE: I had Claude write this so please feel free to ignore if you're not cool with that, I don't know anything about TypeScript, but it work on my machine now!

Thank you @DuBose-Tuller!


DBZ-8298 Use logical name when creating offset partition name

https://issues.redhat.com/browse/DBZ-8298

@Naros Applied, thanks


Cant map port 53

Issue Description:
Cant map port 53
Docker Run Command:
docker run --name steamcache-dns -p 53:53/udp -e STEAMCACHE_IP=$STEAMCACHE_IP steamcache/steamcache-dns

docker run --name steamcache-dns -p 53:53/udp -e STEAMCACHE_IP=$STEAMCACHE_IP steamcache/steamcache-dns
Docker Container Output:
docker: Error response from daemon: driver failed programming external connectivity on endpoint steamcache-dns (4d2f5a9245e16928e952f4a7344fe7de08f70c7172c7cb6440dbc321213fd5b4): Error starting userland proxy: listen udp <IP_ADDRESS>:53: bind: address already in use.
ERRO[0001] error waiting for container: context canceled

run sudo lsof -i :53 to see what process is using it. then research whether you should kill it.
if you're using Ubuntu 18.04 LTS, it has its own DNS running (dnsmasq) already which you'll need to disable.

This is a problem with dnsmasq, if you use the IP in your -p flag, you should be OK:
docker run --name steamcache-dns -p <IP_ADDRESS>:53:53 -e STEAMCACHE_IP=$STEAMCACHE_IP steamcache/steamcache-dns

Without defining an IP on the command line, it will try and listen on all interfaces (including loopback devices)
We have also just published a compose file to set most of this up for you, it will run a full-stack cache and run everything on a single IP using a reverse proxy: https://github.com/steamcache/docker-compose


Fix README.md example

Example is failing
$ node app.js
file:///Users/snwfdhmp/Dev/workspaces/markets/scraping/app.js:15
  const twitterClient = new TwitterApi(
                        ^

TypeError: TwitterApi is not a constructor
    at main (file:///Users/snwfdhmp/Dev/workspaces/markets/scraping/app.js:15:25)
    at file:///Users/snwfdhmp/Dev/workspaces/markets/scraping/app.js:23:1
    at ModuleJob.run (node:internal/modules/esm/module_job:185:25)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:281:24)
    at async loadESM (node:internal/process/esm_loader:88:5)
    at async handleMainPromise (node:internal/modules/run_main:65:12)

Using import { TwitterApi } from "twitter-api-v2"; fixes it.

Hi,
This example works in a non-ECMA module context.
See the basics.

Create a client
Import the default export/TwitterApi variable from twitter-api-v2 module.
Developer note: Default export is a TypeScript-CommonJS-wrapped default export — it isn't a regular ECMA module default export. See example below.

// This will ONLY work with TypeScript on module: "commonjs"
import TwitterApi from 'twitter-api-v2';

// This will work on TypeScript (with commonJS and ECMA)
// AND with Node.js in ECMA mode (.mjs files, type: "module" in package.json)
import { TwitterApi } from 'twitter-api-v2';

// This will work with Node.js on CommonJS mode (TypeScript or not)
const { TwitterApi } = require('twitter-api-v2');