Move page and page_size query param validation into serializer

Fixes

Related to #857 and #843
Description

In this PR I move scattered query parameter validation into the query serializer, where it probably belongs. This has two benefits:

It removes code in favour of using built-in Django Rest Framework serializer validation
It removes usages of the bespoke get_api_exception
It reduces instances of a loose pattern of scattered validation where parameter validation happens multiple times and deep into the runtime of a search query, rather than once at the top (where it is expected to happen and easier to find)

None of these changes change the API surface.
The lines changed is also deceptive on this PR as something like 85 of the lines added or changed were in the tests.
Testing Instructions

Ensure CI passes. Checkout the branch, set up the local API following the README instructions and make some smoke test requests to ensure everything works as expected.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Keeping this PR as a draft as it should not be merged until we've fixed the production ES CPU usage.

@sarayourfriend since the ES CPU usage is under control now, should we push to update, undraft and get this PR merged?

Diff before updating to main, preserved for self-reference.


[Improvement] Forward images/media to admin group when banned by meta

The title is basically it.
It's hard to distinguish false positives when a user is banned for the image without text, as there's no image forwarded to admin group. You have to dig into recent actions in the main group and match the time to figure out what's what.

Created a separate issue #198 for the confirmation error.

I don't mind. If anyone want to make a pr (should be not hard to do) I'll consider


Performance regression test cases

We should probably run each case sequentially on the current iModel from the battery of iModels to test:
const testCases = {
  "identity transform": require("./identity-transform");
};

// need to not download iModels that we don't intend to test for local experiments
for (const iModel of iModelsInProject({ filter: m => m.tShirtsize === 'm' } )) {
  describe(`Transforms of ${iModel.name}`, () => {
    for (const [testCaseName, testCaseRun] of Object.entries(testCases) {
      it(testCaseFunction.name, function () {
        //if (!process.env.CI && iModel.tShirtsize !== 'm') this.skip();
        testCaseRun(iModel);
      });
    }
  });
}

Sorted by priority:

[x] identity transformation (source to empty target)
[x] prepare fork

[x] prepare fork all fed guids
[x] prepare fork few fed guids

create elements with "empty" fed-guid (000-0000...), @itwin/core-bentley should have something for that

[x] prepare fork few fed guids but add them to the source ourselves

copy the source in the test, and use the withSqliteStatement API to query the bis_Element.FederationGuid and update it to have a federation guid if it doesn't already (use randomblob(16))

[x] raw no transformer

we can do this one later. Insert target scope element, iterate over the elements and add ExternalSourceAspects to those without fedguids, as the transformer would, do not invoke the transformer.

[ ] synchronize fork (many/few fed guids)
[ ] reverse synchronize fork (many/few fed guids)
[ ] physical model consolidation
[ ] many deletes
[ ] many relationships
[ ] complex schema
[ ] complex geometry
[x] control

[x] raw insert many elements
[x] apply changeset containing same elements
[x] populate empty target from source containing same elements
[x] populate empty target from source containing same elements, also add provenance

closing as close enough


⚠ Bofrost.be has degraded performance

In 47c7e11, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2638 ms

Resolved: Bofrost.be performance has improved in 9f4ee5f.


Add link to trigger

Before

After.  Note git.head_sha_monitor is now a link. It works the same was as the RULES DETAILS link above it.

This is where it links to.


randomBytes is required from randombytes which requires from crypto

I have been having issues with browserified code using this package because it depends on non-browserified code to implement itself.
Basically, this line is the issue:
exports.randomBytes = exports.rng = exports.pseudoRandomBytes = exports.prng = require('randombytes');

When you require crypto-browserify, you're not using the non-browserified crypto package. But if you go to randombytes' implementation, it requires crypto in its code. This causes two issues:

If you are resolving requires to cryptoas crypto-browserify, you get a circular dependency and the randomBytes field becomes undefined.
If you are not, randombytes will just require regular crypto, which defeats the whole purpose of using a browserified package in the first place.

Maybe this is an issue for randombytes instead. But I figured that most packages that depend on randomBytes usually require crypto instead of randombytes.

In a browser, randombytes will use https://github.com/browserify/randombytes/blob/master/browser.js (per https://github.com/browserify/randombytes/blob/master/package.json#L25), and I don't see any use of node crypto in that implementation.

It will use this:

Which will not be implemented.

If that function doesn't exist (checked here then it uses https://github.com/browserify/randombytes/blob/master/browser.js#L12 which throws, which is by design.
I don't see any infinite loop here. Are you sure you're using browserify as a bundler?

I am using browserify module resolution via webpack.
And yeah, depending on how I bundle things, I'm getting that error actually. Sorry for not being clearer. This is happening to me outside of browser usage, even when running via node in CLI. My target use however is to use JavaScript as an embeded language using C# and a library that implements a JS interpreter. I'm assuming this is not an intended use-case. 😅
My concern is rather that there is no actual implementation of the actual algorithm without depending on the VM, which in my opinion defeats the purpose (I understand if you think otherwise).
I just have no options as this is an issue being caused by some other dependency of my code depending on this package, and there are frankly no alternatives to this other than implementing the algorithm itself somewhere. I assumed this would be the place for it to be implemented? I am definitely willing to help with it if that's the case, but I figured it should be discussed first.

Bundling for "not a browser" is always an unintended use case :-)
You are correct that there is no actual implementation of the algorithm that doesn't rely on something native - whether from the browser or from node. If there is a way to implement it correctly in JS, then the place to do that would be the randombytes package, in the browser entrypoint I liked.

Maybe it will be useful for someone: Solving a crypto-polyfill usage error by using the Web Crypto API


Settings: add option to open automatic backup / sync in a new tab

Now that ESGST is exclusively an extension, there's no reason not to move the automatic backup / sync to the extension button in the toolbar. That way, it doesn't freeze the page, and the user can load SG pages without having to wait for the backup / sync.

And ESGST is back as a userscript, so I'll just add an option to open it in a new tab. It's much better than the hell I would have to face to maintain the toolbar menu anyway.


Loading LUT textures into renderer

Give an option like "Load LUT texture" to apply a "filter" to everything in the renderer. I took a look at ssbh_wgpu and it seems that this code can be repurposed for it perhaps.
https://github.com/ScanMountGoat/ssbh_wgpu/blob/62d38c8fb028216f2b96c7380e0d34d0139502f5/ssbh_wgpu/src/texture.rs#L128

This will end up being part of loading stage rendering data files like the lighting nuanmb. Most of the code for loading the files is already there. I just need to add some setter methods to SsbhRenderer. The stage LUT is one of the last things to be applied before the UI is drawn, so it should work fine as a filter.


opentofu/terraform resources for secure bootable images on GCP

What would you like to be added:
We need changes to the upstream terraform provider for GCP to enable secure bootable images that let us bring our own secure boot config and keys.
Why is this needed:
I identified potential blockers to go forward with the openTofu implementation of our secure boot use cases.

shielded_instance_initial_state needs to be passed to the image
shieldedInstanceInitialState
Terraform google compute_image resource

The terrafom google_compute_image resource cannot define shielded_instance_initial_state at the moment.

This could potentially be implemented (assessment needed).
This could be potentially be worked around by patching the image later (assessment needed).

Upstream Issue opened here: https://github.com/hashicorp/terraform-provider-google/issues/20303

Support has been prepared in https://github.com/b1-systems/google-magic-modules/tree/prs/add-support-for-shieldedInstanceInitialState


Add batchfile support

Checking multiple domains from CSV file. Alternative to the Get-AcceptedDomain approach. Independent of being logged into an Exchange environment.
Export report would complement this greatly (so that has priority over this, I guess)

Fixed in v1.3


Fix class_for_platform method

Required for: https://github.com/ManageIQ/manageiq-providers-ibm_cloud/pull/341
The .classify method will, among other things, "un-pluralize". There's
an edge case where the plural form is the proper name - for example
'IbmCloud::PowerVirtualServers'. We used the service's official name,
with the plural ending.
This commit allows the 'platform' to be set to the exact class name
instead of snake case.

So singular vs plural aside I like this better because it gets away from the "try MiqProvision#{platform} or the infra manager or the cloud manager or the container manager etc ... etc..."

Oh yeah, I like the more explicitness of this change as well.  Think we should fix both.


Change make_function to allow for multiple vector input arguments

make_function allows you to do this
make_function(f,x,y,z) where x,y,z are vectors of input vectors. But the runtime generated function returned by make_function expects a single vector argument which is the concatenation of x,y,z.

This would make using the runtime generated functions simpler when there are multiple input vectors.

This would also be nice for vector-jacobian and jacobian-vector products in terms of efficiency.
I ran into this with DifferentiationInterface.jl: whenever I want to call the generated function for a JVP, I have to feed it vcat(x, v), which leads to an allocation

In concrete terms, solving this issue would allow me to remove the allocation due to vcat in these lines:
https://github.com/gdalle/DifferentiationInterface.jl/blob/3bfcd1f2dc005936c8defe7ca335953f20401713/ext/DifferentiationInterfaceFastDifferentiationExt/allocating.jl#L18-L29

@gdalle do you really want x to be a vector of vectors? Or do you want x to be a vector of scalar input scalar values?

My bad, each being a vector of numbers is fine. Matrices would be ideal but I can handle the flattening on my end


AMPI: Remove NOIMPL declarations of MPI_File symbols provided by ROMIO

Provided by:

mpi-io/iread_all.c
mpi-io/iread_atall.c
mpi-io/iwrite_all.c
mpi-io/iwrite_atall.c
mpi-io/register_datarep.c

Can you make this conditional on CMK_AMPI_WITH_ROMIO?

Can you make this conditional on CMK_AMPI_WITH_ROMIO?

From what I can tell, the inclusion of the iread/iwrite symbols in the NOIMPL list is a clerical error, since they duplicate APIs provided by ROMIO. If we wanted to keep these when ROMIO is disabled, all of ROMIO's symbols should be listed, not just this small subset.

Can you make this conditional on CMK_AMPI_WITH_ROMIO?

From what I can tell, the inclusion of the iread/iwrite symbols in the NOIMPL list is a clerical error, since they duplicate APIs provided by ROMIO. If we wanted to keep these when ROMIO is disabled, all of ROMIO's symbols should be listed, not just this small subset.

Yeah, thats a good point.


Healthcheck tweak

healthcheck tweaks:

fixed small bug, due to which some subsections of the Curation Summary were not being included in the healthcheck
couple of minor formatting edits

Hmm, test fail under java 7 ("java.lang.AssertionError: Email.send() blocked for more than 240 ms") ... We're using java 8.

Restarted the tests, now everything finishes ok (few of those tests depend on timing, so there are false alerts from time to time)

Aha, great.


ansible build for master fails

Trying to configure the master through ansible but running into failure at the end. Any help is highly appreciated:
TASK [master : Enable apiserver] ***********************************************
fatal: [cd3labvdapp002]: FAILED! => {"changed": false, "failed": true, "msg": "Job for kube-apiserver.service failed because the control process exited with error code. See \"**systemctl status kube-apiserver.service**\" and \"journalctl -xe\" for details.\n"}

NO MORE HOSTS LEFT *************************************************************
        to retry, use: --limit @cluster.retry

PLAY RECAP *********************************************************************
cd3labvdapp002             : ok=70   changed=1    unreachable=0    failed=1

[rakeshk@cd3labvdapp002 ansible]$ systemctl status kube-apiserver.service
● kube-apiserver.service - Kubernetes API Server
   Loaded: loaded (/usr/lib/systemd/system/kube-apiserver.service; enabled; vendor preset: disabled)
   Active: activating (start) since Sat 2016-04-30 09:31:52 EDT; 18s ago
     Docs: https://github.com/GoogleCloudPlatform/kubernetes
 Main PID: 12035 (kube-apiserver)
   Memory: 7.6M
   CGroup: /system.slice/kube-apiserver.service
           └─12035 /usr/bin/kube-apiserver --logtostderr=true --v=0 --etcd-servers=http://cd3labvdapp003:2379 --insecure-bind-address=<IP_ADDRESS> --secure-port=443 --allow...
[rakeshk@cd3labvdapp002 ansible]$

I guess we'd have to see the output of systemctl status kube-apiserver.service and journalctl -xe when it fails, for details...

whoops, you sent the status, but it seems to have no logs...
so maybe journalctl -u kube-api-server -r

[rakeshk@cd3labvdapp002 ansible]$ sudo journalctl -u kube-api-server -r
-- Logs begin at Wed 2016-04-27 20:10:10 EDT, end at Sat 2016-04-30 10:46:59 EDT. --

It looks like you may have an extra - in there between api and server.
journalctl -u kube-apiserver -r

Apr 27 20:10:25 cd3labvdapp002 systemd[1]: Starting Kubernetes API Server...
Apr 27 20:10:25 cd3labvdapp002 systemd[1]: kube-apiserver.service holdoff time over, scheduling restart.
Apr 27 20:10:25 cd3labvdapp002 systemd[1]: kube-apiserver.service failed.
Apr 27 20:10:25 cd3labvdapp002 systemd[1]: Unit kube-apiserver.service entered failed state.
Apr 27 20:10:25 cd3labvdapp002 systemd[1]: Failed to start Kubernetes API Server.
Apr 27 20:10:25 cd3labvdapp002 systemd[1]: kube-apiserver.service: main process exited, code=exited, status=255/n/a
Apr 27 20:10:25 cd3labvdapp002 kube-apiserver[29513]: F0427 20:10:25.144286   29513 controller.go:80] Unable to perform initial IP allocation check: unable to refresh the service IP block: 501: All the given peers are not reachable (failed to propose on members [http://<IP_ADDRESS>:4001] twice [last error: Get http://<IP_ADDRESS>:4001/v2/keys/registry/ranges/serviceips?quorum=false&recursive=false&sorted=false: dial tcp <IP_ADDRESS>:4001: connection refused]) [0]
Apr 27 20:10:14 cd3labvdapp002 kube-apiserver[29513]: [restful] 2016/04/27 20:10:14 log.go:30: [restful/swagger] https://<IP_ADDRESS>:6443/swaggerui/ is mapped to folder /swagger-ui/
Apr 27 20:10:14 cd3labvdapp002 kube-apiserver[29513]: [restful] 2016/04/27 20:10:14 log.go:30: [restful/swagger] listing is available at https://<IP_ADDRESS>:6443/swaggerapi/
Apr 27 20:10:14 cd3labvdapp002 kube-apiserver[29513]: I0427 20:10:14.849822   29513 master.go:295] Will report <IP_ADDRESS> as public IP address.
Apr 27 20:10:14 cd3labvdapp002 kube-apiserver[29513]: I0427 20:10:14.849458   29513 master.go:273] Node port range unspecified. Defaulting to 30000-32767.
Apr 27 20:10:14 cd3labvdapp002 kube-apiserver[29513]: I0427 20:10:14.849214   29513 plugins.go:69] No cloud provider specified.
Apr 27 20:10:14 cd3labvdapp002 kube-apiserver[29513]: Flag --portal-net has been deprecated, see --service-cluster-ip-range instead.
Apr 27 20:10:14 cd3labvdapp002 systemd[1]: Starting Kubernetes API Server...

is etcd running? That playbook must be completed first...

Just reran the etcd playbook. The etcd playbook completed fine with no error but I see these on console that worries me:
TASK [etcd : Open firewalld port for etcd] *************************************
failed: [cd3labvdapp003] => (item=2379) => {"failed": true, "item": 2379, "msg": "firewalld required for this module"}
failed: [cd3labvdapp003] => (item=2380) => {"failed": true, "item": 2380, "msg": "firewalld required for this module"}
...ignoring

TASK [etcd : Save firewalld port for etcd] *************************************
failed: [cd3labvdapp003] => (item=2379) => {"failed": true, "item": 2379, "msg": "firewalld required for this module"}
failed: [cd3labvdapp003] => (item=2380) => {"failed": true, "item": 2380, "msg": "firewalld required for this module"}
...ignoring

TASK [etcd : Open firewalld port for etcd legacy ports] ************************
skipping: [cd3labvdapp003] => (item=4001)
skipping: [cd3labvdapp003] => (item=7001)

TASK [etcd : Save firewalld port for etcd legacy ports] ************************
skipping: [cd3labvdapp003] => (item=4001)
skipping: [cd3labvdapp003] => (item=7001)

TASK [etcd : include] **********************************************************
skipping: [cd3labvdapp003]

PLAY RECAP *********************************************************************
cd3labvdapp002             : ok=46   changed=0    unreachable=0    failed=0
cd3labvdapp003             : ok=46   changed=0    unreachable=0    failed=0
cd3labvdapp004             : ok=27   changed=0    unreachable=0    failed=0
cd3labvdapp005             : ok=27   changed=0    unreachable=0    failed=0

Thanks for the catch! And with that the master playbook succeeds ;) But any hint about the firewall failure message above?
PLAY RECAP *********************************************************************
cd3labvdapp002             : ok=46   changed=0    unreachable=0    failed=0
cd3labvdapp003             : ok=33   changed=0    unreachable=0    failed=0
cd3labvdapp004             : ok=27   changed=0    unreachable=0    failed=0
cd3labvdapp005             : ok=27   changed=0    unreachable=0    failed=0

If you aren't using firewalld, they are normal.

OK. But the full run fails - here is the log:
TASK [flannel : Open firewalld vxlan port for flanneld] ************************
fatal: [cd3labvdapp004]: FAILED! => {"changed": false, "failed": true, "msg": "firewalld required for this module"}
...ignoring
fatal: [cd3labvdapp005]: FAILED! => {"changed": false, "failed": true, "msg": "firewalld required for this module"}
...ignoring
fatal: [cd3labvdapp003]: FAILED! => {"changed": false, "failed": true, "msg": "firewalld required for this module"}
...ignoring
fatal: [cd3labvdapp002]: FAILED! => {"changed": false, "failed": true, "msg": "firewalld required for this module"}
...ignoring

TASK [flannel : Save firewalld vxlan port for flanneld] ************************
fatal: [cd3labvdapp004]: FAILED! => {"changed": false, "failed": true, "msg": "firewalld required for this module"}
...ignoring
fatal: [cd3labvdapp005]: FAILED! => {"changed": false, "failed": true, "msg": "firewalld required for this module"}
...ignoring
fatal: [cd3labvdapp003]: FAILED! => {"changed": false, "failed": true, "msg": "firewalld required for this module"}
...ignoring
fatal: [cd3labvdapp002]: FAILED! => {"changed": false, "failed": true, "msg": "firewalld required for this module"}
...ignoring

TASK [flannel : Open flanneld subnet traffic] **********************************
fatal: [cd3labvdapp004]: FAILED! => {"changed": true, "cmd": ["/bin/firewall-cmd", "--direct", "--add-rule", "ipv4", "filter", "FORWARD", "1", "-i", "flannel.1", "-o", "docker0", "-j", "ACCEPT", "-m", "comment", "--comment", "flannel subnet"], "delta": "0:00:00.194704", "end": "2016-04-30 11:14:15.830666", "failed": true, "rc": 252, "start": "2016-04-30 11:14:15.635962", "stderr": "", "stdout": "\u001b[91mFirewallD is not running\u001b[00m", "stdout_lines": ["\u001b[91mFirewallD is not running\u001b[00m"], "warnings": []}
...ignoring
fatal: [cd3labvdapp005]: FAILED! => {"changed": true, "cmd": ["/bin/firewall-cmd", "--direct", "--add-rule", "ipv4", "filter", "FORWARD", "1", "-i", "flannel.1", "-o", "docker0", "-j", "ACCEPT", "-m", "comment", "--comment", "flannel subnet"], "delta": "0:00:00.191378", "end": "2016-04-30 11:14:15.893859", "failed": true, "rc": 252, "start": "2016-04-30 11:14:15.702481", "stderr": "", "stdout": "\u001b[91mFirewallD is not running\u001b[00m", "stdout_lines": ["\u001b[91mFirewallD is not running\u001b[00m"], "warnings": []}
...ignoring
fatal: [cd3labvdapp002]: FAILED! => {"changed": true, "cmd": ["/bin/firewall-cmd", "--direct", "--add-rule", "ipv4", "filter", "FORWARD", "1", "-i", "flannel.1", "-o", "docker0", "-j", "ACCEPT", "-m", "comment", "--comment", "flannel subnet"], "delta": "0:00:00.199151", "end": "2016-04-30 11:14:15.921265", "failed": true, "rc": 252, "start": "2016-04-30 11:14:15.722114", "stderr": "", "stdout": "\u001b[91mFirewallD is not running\u001b[00m", "stdout_lines": ["\u001b[91mFirewallD is not running\u001b[00m"], "warnings": []}
...ignoring
fatal: [cd3labvdapp003]: FAILED! => {"changed": true, "cmd": ["/bin/firewall-cmd", "--direct", "--add-rule", "ipv4", "filter", "FORWARD", "1", "-i", "flannel.1", "-o", "docker0", "-j", "ACCEPT", "-m", "comment", "--comment", "flannel subnet"], "delta": "0:00:00.206406", "end": "2016-04-30 11:14:15.939435", "failed": true, "rc": 252, "start": "2016-04-30 11:14:15.733029", "stderr": "", "stdout": "\u001b[91mFirewallD is not running\u001b[00m", "stdout_lines": ["\u001b[91mFirewallD is not running\u001b[00m"], "warnings": []}
...ignoring

TASK [flannel : Save flanneld subnet traffic] **********************************
fatal: [cd3labvdapp004]: FAILED! => {"changed": true, "cmd": ["/bin/firewall-cmd", "--permanent", "--direct", "--add-rule", "ipv4", "filter", "FORWARD", "1", "-i", "flannel.1", "-o", "docker0", "-j", "ACCEPT", "-m", "comment", "--comment", "flannel subnet"], "delta": "0:00:00.189425", "end": "2016-04-30 11:14:16.486271", "failed": true, "rc": 252, "start": "2016-04-30 11:14:16.296846", "stderr": "", "stdout": "\u001b[91mFirewallD is not running\u001b[00m", "stdout_lines": ["\u001b[91mFirewallD is not running\u001b[00m"], "warnings": []}
...ignoring
fatal: [cd3labvdapp003]: FAILED! => {"changed": true, "cmd": ["/bin/firewall-cmd", "--permanent", "--direct", "--add-rule", "ipv4", "filter", "FORWARD", "1", "-i", "flannel.1", "-o", "docker0", "-j", "ACCEPT", "-m", "comment", "--comment", "flannel subnet"], "delta": "0:00:00.186628", "end": "2016-04-30 11:14:16.537787", "failed": true, "rc": 252, "start": "2016-04-30 11:14:16.351159", "stderr": "", "stdout": "\u001b[91mFirewallD is not running\u001b[00m", "stdout_lines": ["\u001b[91mFirewallD is not running\u001b[00m"], "warnings": []}
...ignoring
fatal: [cd3labvdapp005]: FAILED! => {"changed": true, "cmd": ["/bin/firewall-cmd", "--permanent", "--direct", "--add-rule", "ipv4", "filter", "FORWARD", "1", "-i", "flannel.1", "-o", "docker0", "-j", "ACCEPT", "-m", "comment", "--comment", "flannel subnet"], "delta": "0:00:00.186356", "end": "2016-04-30 11:14:16.561786", "failed": true, "rc": 252, "start": "2016-04-30 11:14:16.375430", "stderr": "", "stdout": "\u001b[91mFirewallD is not running\u001b[00m", "stdout_lines": ["\u001b[91mFirewallD is not running\u001b[00m"], "warnings": []}
...ignoring
fatal: [cd3labvdapp002]: FAILED! => {"changed": true, "cmd": ["/bin/firewall-cmd", "--permanent", "--direct", "--add-rule", "ipv4", "filter", "FORWARD", "1", "-i", "flannel.1", "-o", "docker0", "-j", "ACCEPT", "-m", "comment", "--comment", "flannel subnet"], "delta": "0:00:00.201834", "end": "2016-04-30 11:14:16.603458", "failed": true, "rc": 252, "start": "2016-04-30 11:14:16.401624", "stderr": "", "stdout": "\u001b[91mFirewallD is not running\u001b[00m", "stdout_lines": ["\u001b[91mFirewallD is not running\u001b[00m"], "warnings": []}
...ignoring

Finally I get this:

RUNNING HANDLER [flannel : delete docker0] *************************************
fatal: [cd3labvdapp003]: FAILED! => {"changed": false, "cmd": "ip link delete docker0", "failed": true, "msg": "[Errno 2] No such file or directory", "rc": 2}

PLAY RECAP *********************************************************************
cd3labvdapp002             : ok=68   changed=1    unreachable=0    failed=0
cd3labvdapp003             : ok=95   changed=3    unreachable=0    failed=1
cd3labvdapp004             : ok=64   changed=0    unreachable=0    failed=0
cd3labvdapp005             : ok=64   changed=0    unreachable=0    failed=0

But I can go to the failed box and can execute the failed command:
[rakeshk@cd3labvdapp003 ~]$ ifconfig
docker0: flags=4099<UP,BROADCAST,MULTICAST>  mtu 1500
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
        ether 02:42:7a:60:66:25  txqueuelen 0  (Ethernet)
        RX packets 0  bytes 0 (0.0 B)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 0  bytes 0 (0.0 B)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

ens192: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
        inet6 fe80::250:56ff:fe3c:11f  prefixlen 64  scopeid 0x20<link>
        ether 00:50:56:3c:01:1f  txqueuelen 1000  (Ethernet)
        RX packets 17862  bytes 4163904 (3.9 MiB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 21509  bytes 1944574 (1.8 MiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

flannel.1: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1450
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
        inet6 fe80::60b9:9aff:febf:6c46  prefixlen 64  scopeid 0x20<link>
        ether 62:b9:9a:bf:6c:46  txqueuelen 0  (Ethernet)
        RX packets 0  bytes 0 (0.0 B)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 0  bytes 0 (0.0 B)
        TX errors 0  dropped 10 overruns 0  carrier 0  collisions 0

lo: flags=73<UP,LOOPBACK,RUNNING>  mtu 65536
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>
        inet6 ::1  prefixlen 128  scopeid 0x10<host>
        loop  txqueuelen 0  (Local Loopback)
        RX packets 21400  bytes 1084149 (1.0 MiB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 21400  bytes 1084149 (1.0 MiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

[rakeshk@cd3labvdapp003 ~]$
[rakeshk@cd3labvdapp003 ~]$ sudo ip link delete docker0
[rakeshk@cd3labvdapp003 ~]$ ifconfig
ens192: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
        inet6 fe80::250:56ff:fe3c:11f  prefixlen 64  scopeid 0x20<link>
        ether 00:50:56:3c:01:1f  txqueuelen 1000  (Ethernet)
        RX packets 17934  bytes 4170544 (3.9 MiB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 21620  bytes 1955076 (1.8 MiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

flannel.1: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1450
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
        inet6 fe80::60b9:9aff:febf:6c46  prefixlen 64  scopeid 0x20<link>
        ether 62:b9:9a:bf:6c:46  txqueuelen 0  (Ethernet)
        RX packets 0  bytes 0 (0.0 B)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 0  bytes 0 (0.0 B)
        TX errors 0  dropped 10 overruns 0  carrier 0  collisions 0

lo: flags=73<UP,LOOPBACK,RUNNING>  mtu 65536
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>
        inet6 ::1  prefixlen 128  scopeid 0x10<host>
        loop  txqueuelen 0  (Local Loopback)
        RX packets 21556  bytes 1091957 (1.0 MiB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 21556  bytes 1091957 (1.0 MiB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

[ ] The commands ip, service, iptables worked only when I prefixed with the right path which in this case (redhat) is /usr/sbin/. In some places it is /sbin which may be true for some variant. Should be detected earlier and changed accordingly.


CSS Emmet Completion: Cannot read property 'filterText' of undefined

VSCode Version: Code - Insiders 1.16.0-insider (34cc9f2209be766d5be30a6ecdbb41f5e81fd56b, 2017-08-30T05:11:42.435Z)
OS Version: Darwin x64 16.7.0

Steps to Reproduce:

In a css file:

body {
    background: rgba(2, 173, 2, 0.82) !important;
    |
}

Place cursor at | and type m:

bug
shell.ts:419 Cannot read property 'filterText' of undefined: TypeError: Cannot read property 'filterText' of undefined
	at Object.doComplete (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/emmet/node_modules/vscode-emmet-helper/out/emmetHelper.js:102:43)
	at noiseCheckPromise.then.noise (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/emmet/out/defaultCompletionProvider.js:50:48)
	at <anonymous>

I'm not see a different exception with the same repo steps:
workbench.main.js:sourcemap:29 Cannot read property 'replace' of undefined: TypeError: Cannot read property 'replace' of undefined
	at removeTabStops (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/emmet/node_modules/vscode-emmet-helper/out/emmetHelper.js:181:24)
	at Object.doComplete (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/emmet/node_modules/vscode-emmet-helper/out/emmetHelper.js:101:34)
	at noiseCheckPromise.then.noise (/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/extensions/emmet/out/defaultCompletionProvider.js:50:48)
	at <anonymous>

Fixed it for real this time


Added changes to support SERVICE_NAME and VERSION as a environment variable to support in k8s.

…riable to support in k8s.

Coverage increased (+1.2%) to 63.739% when pulling 9c642daf487c8275c13a6990665cec5a8e5ad8da on savitaashture:envSupport into 2435a19758a11c6eb346a62fbcd8925b985e1f57 on ServiceComb:master.


Fix issue #5277: [Bug]: AttributeError: 'EventStreamRuntime' object has no attribute 'sid' if runtime_extra_deps is not None

This pull request fixes #5277.
The issue has been successfully resolved through a logical fix that addresses the root cause of the AttributeError. The original problem occurred because the code was attempting to use self.log() (which requires self.sid) before the base class initialization that sets self.sid.
The fix properly reorders the operations by:

Moving the runtime_extra_deps check and logging after the base class initialization
Ensuring self.sid is properly initialized before any logging occurs

While there are failing tests in the CI environment, these are explicitly noted as being unrelated to this specific fix (they're failing due to Docker not being available in the test environment). The actual solution to the AttributeError is correct and complete.
For a human reviewer, I would summarize:
"This PR fixes the AttributeError by reordering operations in EventStreamRuntime's init method. The runtime_extra_deps check and logging now occurs after the base class initialization, ensuring that self.sid is properly initialized before any logging calls are made. While there are some test failures in CI, these are due to Docker availability issues and not related to this fix."
Automatic fix generated by OpenHands 🙌

To run this PR locally, use the following command:
docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:f44ceca-nikolaik   --name openhands-app-f44ceca   docker.all-hands.dev/all-hands-ai/openhands:f44ceca

@openhands-agent I made a change to pyproject.toml, and now this PR gives an error in CI:

pyproject.toml changed significantly since poetry.lock was last generated. Run poetry lock [--no-update] to fix the lock file.

Run poetry lock --no-update on this PR branch.

New OpenHands update


Camps - refactor create new and edit camp sunctions

In file camps.js (/public/scripts/camps.js) there are two separate functions that handle the edti camp and the new camp.
The data collectaion part is an exact duplicate and should be refactored.
function on line 251 and 328.

checked it, and it is refactored.
closing issue.


increased gain AKA volume for lower pitches

I don’t think you meant to file this pull request on the original repo.


Const fold (x ^ x), (x == x)

Need to be able to determine that two exprs are the same

Done in b67d1de6acae5351458090d1e6329825a711761f


Upgrade to the released dbt-duckdb 1.3.1

@jwills Can you describe this PR and what problem this solves specifically?

I thought the title explained the change pretty well-- is this a process thing y'all have?
"This repo uses an unreleased fork of dbt-duckdb since dbt-duckdb wasn't compatible with dbt-core 1.3.0 at the time the repo was created. Since we have released dbt-duckdb 1.3.1, we should have the example project repo use the released version instead of the unreleased fork."
?


Update links to design.md to design.rst

Only really one place in docs/manual.rst but also comments in two other test files which I updated for completeness.
This follows on from #1101

Codecov Report

Merging #1103 into master will decrease coverage by 3.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1103      +/-   ##
==========================================
- Coverage   53.24%   50.21%   -3.04%
==========================================
  Files         115      118       +3
  Lines        8909     8946      +37
==========================================
- Hits         4744     4492     -252
- Misses       3469     3773     +304
+ Partials      696      681      -15

Impacted Files
Coverage Δ

src/restic/backend/b2/b2.go
0% <0%> (-80.59%)
:arrow_down:

src/restic/backend/swift/swift.go
0% <0%> (-73.69%)
:arrow_down:

src/restic/backend/swift/config.go
36.95% <0%> (-54.35%)
:arrow_down:

src/cmds/restic/background_linux.go
0% <0%> (ø)

src/restic/node_linux.go
73.33% <0%> (ø)

src/restic/progress_unix.go
57.14% <0%> (ø)

src/restic/repository/index.go
67.98% <0%> (+0.94%)
:arrow_up:

src/restic/node.go
36.75% <0%> (+2.7%)
:arrow_up:

src/restic/node_unix.go
78.57% <0%> (+7.14%)
:arrow_up:

src/cmds/restic/cmd_mount.go
59.7% <0%> (+46.26%)
:arrow_up:

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 ba6815d...cb3cd57. Read the comment docs.

Thanks!


Can't get vs 2013 or 2015 to load all the controls

When I try to add the controls from Dotspatial.Controls.dll located in the "bin/Debug" folder, all I get is 6 controls. Am I missing something?

In commit 838edd5d47de some of the ToolboxItems were changed so that they are no longer visible. Do you need those to be visible?

@jany-tenaj , btw do you know why this was done? this commit has no sense

@jany-tenaj I don't need them to be visible but I thought something was wrong. thanks for your reply.

@mogikanin this didn't make sense to me either. But I've got no idea why it was done.


Add OpenSearch-build as a component of a release

Signed-off-by: Peter Nied<EMAIL_ADDRESS>Description
This build repository itself should be considered part of the release process, and should have issues created for it automatically and be tracked along with all other inputs.
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #865 (04a562e) into main (fa2ebcd) will decrease coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #865      +/-   ##
==========================================
- Coverage   91.18%   90.97%   -0.22%
==========================================
  Files          87       87
  Lines        2370     2370
==========================================
- Hits         2161     2156       -5
- Misses        209      214       +5

Impacted Files
Coverage Δ

src/system/temporary_directory.py
82.75% <0.00%> (-13.80%)
:arrow_down:

src/system/os.py
92.30% <0.00%> (-7.70%)
:arrow_down:

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 fa2ebcd...04a562e. Read the comment docs.


WIP: Fist draft of functions using ext

Description
Changes proposed in this pull request:

...
...
...

Form tradeoffs & considerations

No example code and dependencies based on current runtime - we have only nodejsXX code hardcoded - but we could use presets!
Is template (set as data.template) needed, if we'll have presets?

Related issue(s)

Finished in a different PR


Throw error on unknown module character encoding

Closes https://github.com/NativeScript/ios-runtime/issues/539

:+1:


React/Preact preset createContext not exported

Could you please add createContext to react.ts and preact.ts. it does not get exported.

hi, you can config like this
AutoImport({
      imports: [
        'react',
        {
          react: [
            'createContext',
          ],
        },
      ]
})


JF17 - Request the implementation of the +/,/- Switch

It is just below the UFC. It's function is similar to the F16 waypoint switch and is frequently used. It is centered and has a momentary up and a momentary down position.

Version 1.1(5)
The implementation of the +/,/- Switch works. Requesting longer "-" zone so that the user can see it getting activated. At the moment, when you press the "-" area, it so close to the activated switch that the users finger may cover it.

will check how I can enhance this

touchable zone is now longer

Touchable zone seems to have been extended. But there is a small area very close to the edge of the iPad, in between the edge of the iPad and the end of the lower clickable zone. Touching in this odd zone will not make a sound, but releasing will. There is no animation associated. Finger is not moved to or from another location on the iPad.

I can't reproduce this one. Nevertheless, I extended the touch area for the fast select decrement to the edge of the UFC. I hope this will help.

I have noticed that this issue only happens when the Ipad Air 2 is in landscape mode. It works as designed in portrait mode.

I'll need to find a way to test it on an iPad Air 2, whether physical device or simulator.
I'll create a separate GH issue for the touchable zone due for version 1.2 and I'll close this one.


Change test chef_s3 to use Deep Learning AMI

Signed-off-by: Luca Carrogu<EMAIL_ADDRESS>Please See Git Pull Request Instructions
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1756 into integ-tests/chef will not change coverage.
The diff coverage is n/a.

@@                Coverage Diff                @@
##           integ-tests/chef    #1756   +/-   ##
=================================================
  Coverage             52.61%   52.61%
=================================================
  Files                    24       24
  Lines                  3959     3959
=================================================
  Hits                   2083     2083
  Misses                 1876     1876

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 831d519...5c03f14. Read the comment docs.


Find in Files: newline matches the whole line

VSCode Version: Version: 1.26.1 (Commit: 493869ee8e8a846b0855873886fc79d480d342de)
OS Version: Windows 10.0.17134.228

Steps to Reproduce:

on any file, do "Find in Files", check "Use Regular Expression", and search for "\n"

Expected: only the line endings are matched (as is the case when doing the same with "Find")
Actual: the whole line is matched, which makes it impossible to do a replace
Does this issue occur when all extensions are disabled?: Yes

When you search for a newline, you should get a message that says Literal '\n' not supported. Do you not see that? But I do see that we aren't handling these matches that wrap around properly and I can fix that...

No, I don't see that message. Actually, what happens is that the whole line is matched except for the newlines (because when I do a replace, the newlines remain): https://ibb.co/ewqUAK
When I do the same with Find, it's exactly the inverse, and only the newlines are matched, as expected: https://ibb.co/gqzg7z

Verification steps according to what I saw:

Open some file
Search \n with regex turned on
When search is submitted, every line in the editor blinks orange for an instant
Then an error message appears that \n is not supported, and the editor highlights are cleared.

Maybe there was some other case where the highlights are not cleared...

Ok, I've experimented some more and have seen the behavior you're describing as well now. However, I'm still able to reproduce this, as follows: add the following to your settings:

"workbench.startupEditor": "newUntitledFile"

then restart VS Code, just type some random lines in the untitled file, and search \n with regex turned on via Find in Files

Oh, you probably don't have a folder open. In that case I do see what you described. My fix is still the same.


[build] Enable klib api dump

Blocked on https://github.com/Kotlin/binary-compatibility-validator/issues/218


PayPal function CORS

Hi, I am trying to get the example here to work.
https://github.com/firebase/functions-samples/tree/master/paypal
When I make the POST request I get a CORS error though.

has been blocked by CORS policy: Request header field access-control-allow-origin is not allowed by Access-Control-Allow-Headers in preflight response.

If I add this code and install cors it doesn't respond at all. I kept seeing this as a suggestion while I was searching for a solution.
const cors = require('cors')({origin: true});

exports.pay = functions.https.onRequest((req, res) => {
  return cors(req, res, () => {
       //  code
   });
});

If I add this code
res.set('Access-Control-Allow-Origin', '*');
The error changes to

Response to preflight request doesn't pass access control check: It does not have HTTP ok status.

If I add this code
res.set('Access-Control-Allow-Origin', 'https://semester-at-sea.web.app');
The error changes to

net::ERR_ABORTED 500

This is what my fetch request looks like
 fetch('https://us-central1-semester-at-sea.cloudfunctions.net/pay', {
            method: 'POST',
            body: {price: 5},
            headers: {
                'Content-Type': 'application/json',
                'Accept': 'application/json',
            }
        })
            .then(response => {
                return response.json()
            })
            .then(data => {
                console.log(data);
            })
            .catch(message => {
                console.log(message);
            })

I tested the endpoint with Postman and it gets status 200 ok and redirects. Because of that, I am pretty sure this is strictly a CORS issue with the provided pay pal function. Take CORS out of the question using Postman and everything works fine.
When I check the Firebase function logs I see this:
{ Error: Response Status : 400
    at IncomingMessage.<anonymous> (/srv/node_modules/paypal-rest-sdk/lib/client.js:130:23)
    at emitNone (events.js:111:20)
    at IncomingMessage.emit (events.js:208:7)
    at endReadableNT (_stream_readable.js:1064:12)
    at _combinedTickCallback (internal/process/next_tick.js:139:11)
    at process._tickDomainCallback (internal/process/next_tick.js:219:9)
  response:
   { name: 'VALIDATION_ERROR',
     message: 'Invalid request - see details',
     debug_id: '2c2ef59c9994c',
     information_link: 'https://developer.paypal.com/docs/api/payments/#errors',
     details: [ [Object] ],
     httpStatusCode: 400 },
  httpStatusCode: 400 }

Any ideas on how I can fix this?
Thanks in advance!

I followed this samples to make it work, had to use express


[Bug]: Administrator cannot remove other user's Email OTP

Preflight Checklist

[X] I could not find a solution in the documentation, the existing issues or discussions
[X] I have joined the ZITADEL chat

Environment
Self-hosted
Version
2.37.1
Describe the problem caused by this bug
As instance admin I have an option to remove user's OTPs. But Email OTP removal button click do nothing - no errors, no messages, and Email OTP doesn't disappear. On the other hand, TOTP removal works good.
To reproduce
Prerequisites: organization must allow Email OTP 2FA.
Reproduce:

Register new user
Log in with this new user and add Email OTP.
Login with administrator, go to created user profile, open tab 'Password and security', locate user's Email OTP, click 'Remove" button. Nothing happens.

Screenshots

Expected behavior
When I (as administrator) click 'Remove' button on other user's Email OTP, this 2FA method is being removed. Like TOTP do.
Operating System
Windows, Docker compose under WSL
Relevant Configuration
No response
Additional Context
No response

Hei @dimyle
Thanks for reporting this, I could reproduce it.

Quick check looks like if button "Delete" is clicked, that no api request is triggered


Create readme for the project

Resolved in https://github.com/svrooij/msgraph-sdk-dotnet-batching/commit/f68b471826345e3deee94b335e359de751a2fcaa


Allow for SAF to coexist with openshift-monitoring

These changes are related to work done to deploy SAF into an
environment where OpenShift Monitoring has been deployed. The current
way prior to this change would result in overlapping namespaces for
the ClusterRole and ClusterRoleBinding.
Additionally, Prometheus Operator has been setup to be namespace
bound, and will not act on requests for Prometheus objects that
haven't been created in this namespace.
A new parameter has been added (defaulted to existing 20G sizing) of
the PVC request by Prometheus. Allows for using smaller or larger PVC
requests now.
Downstream and upstream versions have been updated to reflect the new
method that allows us to use a different tag name from that of the
version which makes the tagging slightly less confusing.
Removed the static prometheus.yaml since it is now being generated by
the deploy_builder.yaml via Ansible.
Closes #44

Also I tested with the new downstream build and everything is working as it should.


neovide: fix build on darwin

Motivation for this change
Part of fixing the build on aarch64-darwin: https://github.com/NixOS/nixpkgs/issues/145523
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 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

[ ] Fits CONTRIBUTING.md.

Couldn't get it to work sorry.


Verify Integrity of treatments

ie.
treatments/
  /config
    /errors
    /history
    /queue

all exist

documented in: c0f9755

Can use class method in Stateful
then create Stateful and enumerate through configs

If no arg given, check all configs

Cannot enumerate through all configs; some might be windows and others mac

This is possible, but the configs must be divided by OS: so mac / windows

eg:
mac         # on mac MVE_CONFIGS is set here
windows

If no arg given, check all configs

Not doing this,
verify_config_integrity works on one given config name.
it will already fail if the parent config structure does not exist.
to implement this feature i would have to reuse code from config.py
There is also no need for the feature.
Users should check one at a time if configs exist.

If no arg given, check all configs

Not doing this,
verify_config_integrity works on one given config name.
it will already fail if the parent config structure does not exist.
to implement this feature i would have to reuse code from config.py
There is also no need for the feature. Users should check one at a time if configs exist.

Wait no it is possible, it is as simple as doing
for config in $MVE_CONFIGS/*

Implemented partially in 497984d

done in 4972717


Dataturks inlog error...

Hello is their an solution for this docker issue?
On the localhost/127.
error: Request has been terminated
Possible causes: the network is offline, Origin is not allowed by Access-Control-Allow-Origin, the page is being unloaded, etc
on port 3000 http://<IP_ADDRESS>:32770/projects/login
error: Request has been terminated
Possible causes: the network is offline, Origin is not allowed by Access-Control-Allow-Origin, the page is being unloaded, etc
on port 80 : <IP_ADDRESS>:32771  http://<IP_ADDRESS>/onprem.php redirect problem loading page

I'm facing the same issue. I tried to contact the team by email but no response.

I'm using mentioned docker, but still got following error:
Error: Request has been terminated
Possible causes: the network is offline, Origin is not allowed by Access-Control-Allow-Origin, the page is being unloaded, etc.


Update README.md

Remove what appears to be a redundant "!" chars

It's because of typescript. querySelector will return Element | undefined, while render requires Element.  The ! just asserts that it's not nullable.
You can even see it in the default playground setup: https://playground.solidjs.com/
That being said, if it may be confusing to somebody, I don't mind removing it. It doesn't help the docs in any way. I was just copying some ts code and didn't bother to remove it because ts is effectively the standard nowadays.

Thanks.
The experience I had was copying it into a non-TS project and then receiving an error, which I'm sure was not in your intention :)


Allow nz-select to close when the focus on next element

What problem does this feature solve?
The select component kept open when it have lost the focus
What does the proposed API look like?
Allow nz-select to close when the focus on next element

supported in latest version.


Dealing with DefaultValue

As I had the opportunity to say in my recent pull request #32: Thanks for this really nice binding. IMO the Recoil API types are particularly complex and those bindings really helps 🙏

My question is regarding this special type DefaultValue that Recoil has and that appear to be ignored from this binding.
It's used in some some callbacks instead of passing a value.
For example the set callback of a selector can either receive a value or an instance of DefaultValue. DefaultValue is passed to the callback when the action dispatch is a reset instead of setting a new value.
From recoil doc:
function selector<T>({
  key: string,

  get: ({
    get: GetRecoilValue,
    getCallback: GetCallback,
  }) => T | Promise<T> | RecoilValue<T>,

  set?: (
    {
      get: GetRecoilValue,
      set: SetRecoilState,
      reset: ResetRecoilState,
    },
    newValue: T | DefaultValue,
  ) => void,

  dangerouslyAllowMutability?: boolean,

  cachePolicy_UNSTABLE?: CachePolicy,
})

Note that SetRecoilState can also be used to set a value to a DefaultValue instance.
The common pattern to handle this special value in javascript is the following:
import { atom, selector, DefaultValue } from "recoil"

const atom = atom({ key: "Atom", default: 1 })
const plusOne = selector({
  key: "AtomPlusOne",
  get: ({ get }) => get(atom) + 1,
  set: ({ set }, newValue) =>
    newValue instanceof DefaultValue
      ? set(atom, newValue) // forward the reset action
      : set(atom, newValue - 1)
})

Because of this T | DefaultValue type, it seems difficult to create a zero-cost binding that handle the DefaultValue separately using maybe a rescript variant type.
I'm pretty new rescript binding, but I was thinking that if the library would use a type like the one bellow, it would make handling the default value slightly simpler.
module PossiblyDefaultValue = {
  @unboxed
  type t<'value> = PossiblyDefaultValue('value)

  type defaultValue
  @module("recoil") external defaultValue_: defaultValue = "DefaultValue"

  let defaultValue = defaultValue_

  @warning("-27")
  let isDefaultValue: t<'value> => bool = action => %raw("action instanceof defaultValue")
}

Personally, I would probably prefer a "low-cost" binding that would use a proper variant like option<'value>, or a new custom variant type, but at least I think that my suggestion above could provide the building bricks to build my own low-cost variant.
Do you think @bloodyowl that is is worth considering adding such a PossiblyDefaultValue.t<'value> type? I'll be more than happy to make this my second contribution to the bindings. If you decide to keep the types as they are, I would also be happy to just write down a "Dealing with DefaultValue" section to the readme.

Hi 👋 Thanks for the detailed report!
I think that adding an additional type with runtime here might be a bit too cumbersome for the most common use-cases. Here's what I think is the best compromise given the use-case: https://github.com/bloodyowl/rescript-recoil/pull/35/files
It lies a bit about the actual type in that case, but it seems like the solution with the least breaking change & the most acceptable DX.
How does this look to you?

Your proposed solution in #35 looks great as it introduces no breaking changes 👍
As of DX I would personally wrap this so I get a proper variant DefaultValue | Value<'value> but that's probably because my app uses Reset quite a lot.
Thanks for addressing this 🙏


SessionId will change when set "rewrite" in proxy.conf.json

SessionId will change when set "rewrite" in proxy.conf.json

Sorry for my poor english.
I try to do this..But my backend will receive different sessionId of every request.

So sorry for my poor English, I will try my best to say my problem clearly..
I hava found the reason, my backend is java,  which is running at http://localhost:8082/MusicJava...
but my frontend which uses Angular will send a post request at http://localhost:4200/api..
My code:

I try to change the file proxy.conf.json so that I can make my request /api/... forward to /MusicJava/api/..., however, when I send a request I can get a correct result, but cookie without setting path by backend will set path of cookies as 'MusicJava' such as sessionid..

everytime I send a request, I can't take cookie correctly, because current url is http://localhost:4200/, not http://localhost:4200/MusicJava/ where the cookie at..(I think,I don't konw whether it's wrong or correct, sorry.)

I fixed it by change my send url like this.

Then all request will take cookie correctly...
I can get correct result now but I still can't understant why...I thought pathRewrite just change my send url, It seems like I'm wrong..
Could you please tell me why? Apologize again for my poor English.
I have tried google, but I can't get answer...
Thank you!

From what I can see, it is because the cookie is being dropped with a path of MusicJava, which means it has a restriction for the path.
See: https://developer.mozilla.org/en-US/docs/Web/HTTP/Cookies


Update readme to show how to use postcss/processors?

Hi there!
I saw back in 0.4.0 the option to hook in a CSS processor was added. Based on the source code, I assumed it would be as simple as this:
scss({
  sass,
  output: false,
  outputStyle: "compressed",
  processor: async (css) => {
    const result = await postcss([autoprefixer]).process(css, { from: undefined })
    return result.css
  },
})

I expected the final result to be my CSS, e.g.:
var styles = 'html{display:flex}'

But instead I get this:
var styles = {}

I checked that all my packages are being loaded correctly and even result.css being returned is an actual string of the CSS I expect, but still I get the above. 🤷
Could it be that this is an async function and that use-case isn't handled? My only guess.
Any insight is appreciated. Thanks!

Looks like a bug here: https://github.com/thgh/rollup-plugin-scss/blob/a11c0a0f585c4c9cd7a18248aaa28cd2c2782b20/index.es.js#L71
It should be
        return Promise.resolve(compileToCSS(code)).then((css) => ({
          code: 'export default ' + JSON.stringify(css),
          map: { mappings: '' }
        }))

That looks closer to what I'd expect. Happy to open a PR if you'd like.


feat: add interactive sync mode

Adds interactive mode with ability to choose charts to sync
Use jx updatebot sync --interactive

/ok-to-test

@rast1025 If you resolve the conflicts we could get this merged

@msvticket, rast1025 was on my team at my org, he moved onto bigger and better things, so I've merged master into this

@tomhobson If you fix the the test failures I can get it merged


Tutorial: SnowEx Mission Overview

Lead: HP Marshall
Date: 19/08/2024
Start Time: 1115
Duration: 45
Description: All about SnowEx!
Details

Dependencies (things people should know in advance of the tutorial)
Technical Needs (GPUs? Large file storage? Unique libraries?)

WHoops, not sure if I was asked to do the mission overview or data overview.  Happy to do either.


Propagate reqwest TLS features

Hey, thanks for this library! Just a quick PR to make it so that folks can choose native-tls instead of rustls, if they like.

Thank you so much!

Do you need a release?

Yes—a release would be swell!

Done, it's the v0.2.1, thanks for your contribution :rocket:


after build got this error pages/api/auth/signin/google.js net::ERR_ABORTED 404

I got this error after build
pages/api/auth/signin/google.js net::ERR_ABORTED 404
but i dev mode everything works great
anyone know why this happened ?
I'm using
pages/api/auth/[...nextauth].js

Hi, we don't have enough information to be able to help with this request.
Could you re-submit, using the provided issue templates and including any relevant configuration?

PS: If this is your first time using NextAuth.js, you might want to try cloning the example project and configuring that to get started (as simple was we try to make it, it's still unfortunately a little complicated to get started).


Add Gitea + make improvements to examples

Adds Gitea to the examples as well as changes the README of Wordpress
and GitLab to match the other examples.
.yml is also changed to .yaml

Any ideas why I'm getting these errors? Another thing too, it doesn't look like there is actually any output to the error, but if I run it manually, I can see it:
Example:
github.com/kedgeproject/kedge  add-gitea ✔                                                                                24m  ⍉
▶ ./kedge generate -f examples/gitea/gitea.yaml
2018/02/28 13:12:54 The kedgefile is not valid. see errors :
livenessProbe: Additional property livenessProbe is not allowed
readinessProbe: Additional property readinessProbe is not allowed
postgresql-user: Additional property postgresql-user is not allowed
secrets.0.data: Invalid type. Expected: object, given: null

Travis CI errors:
--- FAIL: Test_examples (6.56s)
	cmd_test.go:158: kedge generate failed for - /home/travis/gopath/src/github.com/kedgeproject/kedge/examples/gitea/gitea.yaml
		 Error is -
=== RUN   Test_init

Ping @surajnarwade

@surajnarwade @kadel @containscafeine Mind testing this?

@cdrage , I am not able to run gitea app on minikube,
 $ kubectl get all
NAME                DESIRED   CURRENT   UP-TO-DATE   AVAILABLE   AGE
deploy/gitea        1         1         1            0           2m
deploy/postgresql   1         1         1            0           2m

NAME                       DESIRED   CURRENT   READY     AGE
rs/gitea-7c6b556456        1         1         0         2m
rs/postgresql-66cf8d5c86   1         1         0         2m

NAME                DESIRED   CURRENT   UP-TO-DATE   AVAILABLE   AGE
deploy/gitea        1         1         1            0           2m
deploy/postgresql   1         1         1            0           2m

NAME                       DESIRED   CURRENT   READY     AGE
rs/gitea-7c6b556456        1         1         0         2m
rs/postgresql-66cf8d5c86   1         1         0         2m

NAME                             READY     STATUS                       RESTARTS   AGE
po/gitea-7c6b556456-cb5bv        0/1       Running                      0          2m
po/postgresql-66cf8d5c86-8d4jz   0/1       CreateContainerConfigError   0          2m

NAME             TYPE           CLUSTER-IP      EXTERNAL-IP   PORT(S)                     AGE
svc/gitea        LoadBalancer   <IP_ADDRESS>   <pending>     80:30643/TCP,22:30583/TCP   2m
svc/kubernetes   ClusterIP      <IP_ADDRESS>       <none>        443/TCP                     5m
svc/postgresql   ClusterIP      <IP_ADDRESS>    <none>        5432/TCP                    2m

@surajnarwade works for me, can you post the logs from kubectl describe po/postgresql-66cf8d5c86-8d4jz


The Vue.js does not execute the command despite the correct encoding

Version
2.6.11
Reproduction link
https://danieleg.000webhostapp.com/vue/vue/slug.html
Steps to reproduce
var app = new Vue({
el: '#daniel-app',
data: {
inPut: '',
outPut: '',
buttonText: 'fab fa-telegram-plane',
clarification: 'Convert the title to Slug'
},
methods: {
convertToSlug () {
var theSlug = this.inPut;
// [1] Convert the post title to lowercase
theSlug = theSlug.toLowerCase();
// [2] Replace spaces with a dash
theSlug = theSlug.replace(/\s /g, '-');
// [3] Replace ampersand with a dash
theSlug = theSlug.replace(/&/g, '-');
// [4] Replace 2 dashs with nothing
theSlug = theSlug.replace(/--/g, '');
// [5] Delete a dash in end and first post title
var firstDash = theSlug.charAt(0);
var endDash = theSlug.charAt(theSlug.length - 1);
if (firstDash === '-' && endDash === '-') {
theSlug = theSlug.replace(firstDash, '');
theSlug = theSlug.replace(endDash, '');
} else if (firstDash === '-') {
theSlug = theSlug.replace(firstDash, '');
} else if (endDash === '-') {
theSlug = theSlug.replace(endDash, '');
}
// Add result post title to output
if (this.outPut === theSlug) {
this.outPut = '';
this.buttonText = 'fab fa-telegram-plane';
this.clarification = 'Convert the title to Slug';
} else if (this.buttonText === 'fas fa-times') {
this.outPut = '';
this.buttonText = 'fab fa-telegram-plane';
this.clarification = 'Convert the title to Slug';
} else {
this.outPut = theSlug;
this.buttonText = 'fas fa-times';
this.clarification = 'Delete this slug';
console.log(endDash);
}
}
}
});
Preview: https://danieleg.000webhostapp.com/vue/vue/slug.html
What is expected?
Convert a dash at the beginning of the title to nothing
And turn a dash at the end of the title to nothing
What is actually happening?
When trying to replace a dash at the beginning of the title to nothing, this happens
But when converting a dash at the end of the address to nothing, this does not happen

The exact error here:
// [5] Delete a dash in end and first post title
var firstDash = theSlug.charAt(0);
var endDash = theSlug.charAt(theSlug.length - 1);
if (firstDash === '-' && endDash === '-') {
theSlug = theSlug.replace(firstDash, '');
theSlug = theSlug.replace(endDash, '');
} else if (firstDash === '-') {
theSlug = theSlug.replace(firstDash, '');
} else if (endDash === '-') {
theSlug = theSlug.replace(endDash, '');
}

Hi, thanks for your interest but Github issues are for bug reports and feature requests only. You can ask questions on the forum, the Discord server or StackOverflow.


Create 'Gallery of Forgotten Memes' Section

Let's make our homepage funnier by adding a "Gallery of Forgotten Memes" section into the starting-page. Your job is to design this section and fill it with old, hilarious internet memes. When you hover over them, they should do something cool. After you're done, share your changes on GitHub Pages for everyone to see and enjoy the nostalgia!

Hey, i have created a pull request, creating the "Gallery of forgotten memes" section, hope you like it.

Hey, I have created a pull request, creating the "Gallery of forgotten memes" section, hope you like it.

Great work, @Dushyant-Mahto! The animations are fantastic. If you're interested, you could enhance the gallery even more by adding left and right navigation buttons. This would help save space and allow for a more organized layout, perhaps by arranging the images in two rows. Keep up the great work!


Fix deprecated field

The deprecated field open got changed to state to hold 3 different values (open, dismissed, and fixed)

@gmlewis @wesleimp Gentle bump

@gmlewis @wesleimp gentle bump

@gmlewis Thank you for approving & merging the PR. Can I know when the next release will be created for go-github?

@gmlewis Thank you for approving & merging the PR. Can I know when the next release will be created for go-github?

No problem. I should be able to cut a new release by tomorrow.

awesome! Thank you!

@ganeshkumarsv - v38 has now been released.


Allow to detect image from non-manged memory (IntPtr)

In which case are you using that?

I have a video stream from a network camera.
Decoding the frames and converting to Argb array with FFMpeg.
Then I'd like to detect the objects on that image without copying it.
Or what do you mean?

We have a simular case. But in our case we receive a byte array. How did you move the images from ffmpeg to c#? You are use the ffmpeg binaries or a custom implementation?

I'm usign FFMpeg.Autogen: https://github.com/Ruslan-B/FFmpeg.AutoGen
IntPtr imageBuffer = IntPtr.Zero; // allocated only once, it points to the allocated memory + bmp header size
var dstArray = new byte*[8];
var dstStrideArray = new int[1];
AVFrame* frame = null;
frame = av_frame_alloc();
while (avcodec_receive_frame(videoCodec, frame) >= 0)
{
dstArray[0] = (byte*)imageBuffer.ToPointer();
dstStrideArray[0] = av_image_get_linesize(targetFormat, width, 0);
sws_scale(yuvToRgbCtx, frame->data, frame->linesize, 0, height, dstArray, dstStrideArray);
// now the decompressed image is in imageBuffer
}
I can send you the souce code of the video decoder class to a private address if you need it.

By the way, we are using your package in a .net core 3.0 project... it works, however it would be good to have a netstandard 2.0 vesrion.

Okay great news, i have planned an update to .netstandard 2.0 :)
We have a c++ project with a namedpipe to transfer the vidoes of a rtsp stream to c#. In my case we have a byte array. Yes please send me your solution that i can test your pull. tino.hager📧alturos.com

I have remove the ImageAnalyzer part i have some changes in the new version and i want merge the code the next weeks... Thanks for your help with the unit test

Thank you very much.
ImageAnalyzer: Ok, I don't need that changes, just wanted to make consistent the detect code with the byte[] version. So for me the current version is perfect. Anyway, you can add that check or ignore it, as you wish.
Now (once it is released on nuget) I can use the official package instead of my "hacked" version:)

I have push the new package https://www.nuget.org/packages/Alturos.Yolo/2.6.4

@zgabi An other way to read frames from a stream https://github.com/nager/Nager.VideoStream


Timer for Halloween is updated

How Many Days Until Halloween? PR Request Template
Please note: Has this feature already been added? Sometimes, duplicate pull requests happen. It's worth checking the pull requests and issue page to see if the change you are requesting has already been made.
Descriptive name.
Your pull request should have a descriptive name.
Type of Change was Made?
What type of change does your code introduce? After creating the PR, tick the checkboxes that apply.

[ ] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Description of the Change Being Made.
It's helpful to outline what changes were made to which files so that I have an idea of what will be involved in reviewing the code and—hopefully—merging it into the codebase.
Issue Number
If your pull request is related to a specific issue, please included it in your description and or pull request name. It helps to keep changes linked. Any issues that are referenced in pull requests become part of the discussion history of the issue.
Potential Performance Issues
Does the PR have a potential impact on performance on the codebase? If so, to what degree and why does the PR warrant the performance hit?
Tests/Checks
What tests were conducted to ensure the PR functions have no impact on previous functionalities of the code base?
New Dependencies
Have new dependencies been introduced? Please list them with links to documentation and add installation steps to the README.

Are you working on it?

If you are busy then i can resolve conflicts or do something for you.

Its looks like this.

I'll look into getting this merged this week.


Add app launcher tiles to the navigation bar

The "Learn More" link under Next Steps doesn't have details on how to add app launcher tiles to the navigation bar.

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

ID: 9488caf7-b791-3002-8749-7fdc4b2ec8b3
Version Independent ID: 08e61eb0-7bcd-f449-46ef-17e209138a8e
Content: Add custom tiles to the app launcher - Microsoft 365 admin
Content Source: microsoft-365/admin/manage/customize-the-app-launcher.md
Service: o365-administration
GitHub Login: @twernermsft
Microsoft Alias: twerner

@dimachernov Thank you for your feedback. I don't see the option to add it to the navigation bar exists anymore.  We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/6917
Once the author verifies and approves, the document will be live.
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @dimachernov Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


fix: Add justify and align options

Looks fine. Just missing the migration. Please create the UpdateFromX() migration to migrate the existing content types.


Example on animating array of items rendered

Description
Looking for an example on how to animate an array of items attached to my rendered template. For example below, if the items array gets changed and milk gets moved to the bottom of the array, it renders instantly. What is the best strategy to manage this?
import { LitElement, html } from 'lit-element';

class MyElement extends LitElement {
  static get properties() {
    return {
      items: { type: Array },
    };
  }
  constructor() {
    super();
    this.items = ['milk','apples','cheese','chips','salsa'];
  }
  render() {
    return html`
      <ul>
        ${this.items.map(i => html`<li>${i}</li>`)}
      </ul>
    `;
  }
}

customElements.define('my-todo', MyElement);

Sorry for the delay on this. We don't have good guidance right now for accomplishing this type of animation, but we hope to include some examples in our next major release which we're working on right now.
Generally, you can animate changes like this using the FLIP technique. In lit-element, you can capture the "before" state of the DOM in render and apply changes to the after state in updated. This can also be done with lit-html directives but applying the "after state" must be done in requestAnimationFrame to be able to go after rendering.
Here's a very simple example using a directive: https://stackblitz.com/edit/lit-element-animation-demo?file=src/my-element.js.
To handle elements being removed from the DOM a directive would need to manage a templateResult directly to be able to control the DOM removal.

Thanks, this if very helpful to see your example using a directive. I thought I should be trying to do this at some part of the lifecycle and couldn't figure out which one. I've never written a directive before but it looks like that's the way to go so I'll have to read up on them.


Integrate AMQP

During the refresh, a Queue system based on MongoDB has been implemented because it was easier to integrate in early stages. Other queues implementation need to be provided. AMQP is one of them to support Active MQ.

@aneojgurhem is working on the web console activation. Following messages activities

Review AMQP for AWS Cloud


Name is not aligned with other elements in the "details" tables

Go to "OpenShift Details" or "AWS Details".
Screenshots

Desktop (please complete the following information):

OS: openSUSE 15.0
Browser: Firefox
Version: 65

Thanks @quarckster
I believe it is a duplicate of #517.

@lcouzens ^ is it a dupe?

I don't think so, unless the underlined issues fixes both problems.

I think the new PF4 table is trying to center text vertically. If you look at the other columns, it is centered with the other text.
I'll try to align this with at the top. If not, will have to open a PF4 issue.


kpi-tracker seems to try to insert a duplicate block

[2024-02-13T03:13:58Z INFO  concordium_kpi_tracker] Processed blocks 8e9f885d953f27a3d9d1c2c8e37b9cf75db9b04fda5f75a256764628c21e015c at heights 12846889 with 2 transactions in 4ms
[2024-02-13T03:14:01Z INFO  concordium_kpi_tracker] Processed blocks fdb768f058b7d829ee78bf1a8cb90117e36677f47bdb85d91980abab50014fdd at heights 12846890 with 2 transactions in 3ms
[2024-02-13T03:14:02Z INFO  concordium_kpi_tracker] Processed blocks 03c60aa5c5de8f505e557fbf49c59c2843e5324759b7c7796bd6bb94ab4fd5c1 at heights 12846891 with 5 transactions in 5ms
[2024-02-13T03:14:04Z INFO  concordium_kpi_tracker] Processed blocks 9980e2c857bbd4f6f8841c0f9f5e1d73f8ce9a0fcd239564675cc631b909ae47 at heights 12846892 with 5 transactions in 4ms
[2024-02-13T03:14:06Z INFO  concordium_kpi_tracker] Processed blocks d871af341616a2c8c8d813e73b30e3fcc691908f56e9637d746a19b456c03d9d at heights 12846893 with 3 transactions in 4ms
[2024-02-13T03:14:08Z INFO  concordium_kpi_tracker] Processed blocks c8c197f26aede00133530a6c5ff723eb409e6abe880e7d2eba5dd2d8fab8b252 at heights 12846894 with 4 transactions in 4ms
[2024-02-13T03:14:10Z INFO  concordium_kpi_tracker] Processed blocks 8fa7731e0ffb0b716fa143a0fee1172797e0f1ced7c0d27bede5d9061272462e at heights 12846895 with 3 transactions in 4ms
[2024-02-13T03:14:12Z INFO  concordium_kpi_tracker] Processed blocks b3e6ea7377fc95fe47c9d94cfc1b4d5d7035b06d41682c8408814b03978af2e2 at heights 12846896 with 4 transactions in 4ms
[2024-02-13T03:14:14Z INFO  concordium_kpi_tracker] Processed blocks f9df791f2f464c926f6a7b79b78182dd2fa36f5ff2ea7e5b02683f39bcad297e at heights 12846897 with 6 transactions in 5ms
[2024-02-13T03:14:16Z INFO  concordium_kpi_tracker] Processed blocks b9e48c166cb03122e2b645f113dab626a6bbe769685ee7038141dd9b2df43c0c at heights 12846898 with 4 transactions in 4ms
[2024-02-13T03:14:18Z INFO  concordium_kpi_tracker] Processed blocks 79af1c4a76a7c13927517d5cec6341892ec84dfd68f802b2c094b76c780178c6 at heights 12846899 with 1 transactions in 3ms
[2024-02-13T03:14:20Z INFO  concordium_kpi_tracker] Processed blocks f4d54bd0d8af35490a5fef73c4117354d5985f754171536d2fbfecc21aa17ac0 at heights 12846900 with 7 transactions in 11ms
[2024-02-13T03:14:22Z INFO  concordium_kpi_tracker] Processed blocks 84acbd7a07f0530daafd90bcd1d3eb79c8265b57494fc547106104e421d266f1 at heights 12846901 with 5 transactions in 4ms
[2024-02-13T03:14:24Z INFO  concordium_kpi_tracker] Processed blocks 7554ee15fa80787a2b6cd7b4a1ac28a1e74e11a88d5826c79ddf46c8ddaee15e at heights 12846902 with 1 transactions in 3ms
[2024-02-13T03:14:26Z INFO  concordium_kpi_tracker] Processed blocks d5a109d2a626b6a3adc857bd8820bf7cd5be4ac77e71182922ad27309c342f83 at heights 12846903 with 7 transactions in 4ms
[2024-02-13T03:14:28Z INFO  concordium_kpi_tracker] Processed blocks 6ace56380d6343f43ce86a048cc94a8ffc15d7e3ce482ba434f7ba3c170704a1 at heights 12846904 with 3 transactions in 3ms
[2024-02-13T03:14:30Z INFO  concordium_kpi_tracker] Processed blocks 3789eef445b7bea86d9541e5c9b6b3d307b8737882069a665830729827f67bed at heights 12846905 with 3 transactions in 3ms
[2024-02-13T03:14:43Z WARN  concordium_kpi_tracker] Endpoint http://<IP_ADDRESS>:20000/ failed with error Could not get block info for block: 12846907. Trying next.
[2024-02-13T03:14:43Z INFO  concordium_kpi_tracker] Processing blocks from height 12846907 using node http://<IP_ADDRESS>:20000/
[2024-02-13T03:14:43Z INFO  concordium_kpi_tracker] Processed blocks 3b60652e8fa027fb5bd378dbd415ad29c91f915048565be456990ae2fbd869ca at heights 12846906 with 4 transactions in 4ms
[2024-02-13T03:14:43Z ERROR concordium_kpi_tracker] Database connection lost due to db error: ERROR: duplicate key value violates unique constraint "blocks_hash_key"
    DETAIL: Key (hash)=(\x3b60652e8fa027fb5bd378dbd415ad29c91f915048565be456990ae2fbd869ca) already exists.: ERROR: duplicate key value violates unique constraint "blocks_hash_key"
    DETAIL: Key (hash)=(\x3b60652e8fa027fb5bd378dbd415ad29c91f915048565be456990ae2fbd869ca) already exists.. Will attempt to reconnect in 1000ms.
[2024-02-13T03:14:44Z ERROR concordium_kpi_tracker] Database connection lost due to db error: ERROR: duplicate key value violates unique constraint "blocks_hash_key"
    DETAIL: Key (hash)=(\x3b60652e8fa027fb5bd378dbd415ad29c91f915048565be456990ae2fbd869ca) already exists.: ERROR: duplicate key value violates unique constraint "blocks_hash_key"
    DETAIL: Key (hash)=(\x3b60652e8fa027fb5bd378dbd415ad29c91f915048565be456990ae2fbd869ca) already exists.. Will attempt to reconnect in 2000ms.
[2024-02-13T03:14:46Z ERROR concordium_kpi_tracker] Database connection lost due to db error: ERROR: duplicate key value violates unique constraint "blocks_hash_key"
    DETAIL: Key (hash)=(\x3b60652e8fa027fb5bd378dbd415ad29c91f915048565be456990ae2fbd869ca) already exists.: ERROR: duplicate key value violates unique constraint "blocks_hash_key"
    DETAIL: Key (hash)=(\x3b60652e8fa027fb5bd378dbd415ad29c91f915048565be456990ae2fbd869ca) already exists.. Will attempt to reconnect in 4000ms.
[2024-02-13T03:14:50Z ERROR concordium_kpi_tracker] Database connection lost due to db error: ERROR: duplicate key value violates unique constraint "blocks_hash_key"
    DETAIL: Key (hash)=(\x3b60652e8fa027fb5bd378dbd415ad29c91f915048565be456990ae2fbd869ca) already exists.: ERROR: duplicate key value violates unique constraint "blocks_hash_key"
    DETAIL: Key (hash)=(\x3b60652e8fa027fb5bd378dbd415ad29c91f915048565be456990ae2fbd869ca) already exists.. Will attempt to reconnect in 8000ms.
[2024-02-13T03:14:58Z ERROR concordium_kpi_tracker] Database connection lost due to db error: ERROR: duplicate key value violates unique constraint "blocks_hash_key"
    DETAIL: Key (hash)=(\x3b60652e8fa027fb5bd378dbd415ad29c91f915048565be456990ae2fbd869ca) already exists.: ERROR: duplicate key value violates unique constraint "blocks_hash_key"
    DETAIL: Key (hash)=(\x3b60652e8fa027fb5bd378dbd415ad29c91f915048565be456990ae2fbd869ca) already exists.. Will attempt to reconnect in 16000ms.
[2024-02-13T03:15:14Z ERROR concordium_kpi_tracker] Database connection lost due to db error: ERROR: duplicate key value violates unique constraint "blocks_hash_key"

The block it is trying to insert again is at height 12846906.
Restarting fixed this, so the most likely cause is that the queue between the query thread and db thread somehow maintained the block after it was already inserted.

Can we just add conflict resolution to ignore duplicates at the DB level?

That's not fixing the problem. There is a bug somewhere since we should never be trying to insert the same thing twice. And patching the symptom is not the way.

I believe we should do both.Address the underlying concurrency issue and ignore the duplicates with an error.I can create another task for the ignores duplicates?


Presence counts are slightly wonky

On the map, the Central Hall and Swag Table kept jumping back and forth (there were 12 people in central hall and 1 in swag, but it would sometimes show 12 people in central hall)

Unsure if this is still an issue?

Testing with 2 user accounts, this appears to be fully-functioning as expected except for there being a baseline of "22 people" in the theater when there are theoretically 0, hard to say since we don't get names (if you move to the theater, there are 23, and if you leave it goes back to 22).
Action items here:

Group test during 10/9 meeting
Brief investigation into what would be required to solve the theater ghost issue. It's possible this issue would appear elsewhere when we have people idling in the space during the event itself.

I think this is generally mildly low priority.

It's unclear what is causing people to show up as idle instead of leaving, but for now my "fix" is to manually delete the list of people currently in the theater (del theaterPresence).
Closing this out for now. If we have presence issues during 2022, we'll open a new bug report.


Feature request to generate shell auto completion

Is there an existing issue that is already proposing this?

[X] I have searched the existing issues

Is your feature request related to a problem? Please describe it
I thought it would be cool if by default Nest Commander could generate tab auto-completion for some shells like bash, zsh, fish, and powershell.
Describe the solution you'd like
The solution I imagined is something like Nest Commander provides an option to enable the auto-completion feature when invoking CommandFactory.run(...). And when it is enabled, automatically command like the completion <shell> is available, which can be used like the app completion bash to generate shell script output for adding an auto-completion feature according to the selected shell.
Teachability, documentation, adoption, migration strategy
No response
What is the motivation / use case for changing the behavior?
Since most modern shells support tab auto-completion, so I think this feature can improve the user experience when using apps built with Nest Commander.

I am interested in this feature as well. It could be behind a global config (passed to CommandFactory#run et al).

Sounds lovely 👍
Looking for this one
Does somebody wants to take this into a PR with me?

Pull Requests are more than welcome! I haven't had the time to look into how to start generating this between work and family

I'll try to tackle that 👍

Here is a PR:
https://github.com/jmcdo29/nest-commander/pull/1045
DEMO video:
https://github.com/jmcdo29/nest-commander/assets/77183871/177d3f68-f8ba-4e9a-9574-744ce09f093a

I haven’t tested it out, but it looks awesome! Thanks, @Avivbens! :pray:

This is released as a part of 3.12.0 :tada:


fixed OverflowException when running as x64

When running as x64, ResourceId.IsIntResource threw an OverflowException. (64 bit pointers can't be cast to uint)

Thanks! If you have a repro, a test would be helpful. Otherwise would you be so kind to update CHANGELOG (via --amend please) as well?
https://github.com/dblock/resourcelib/blob/master/CHANGELOG.md

I just noticed this issue in my own app and fixed it. I'll try to create a unit test.
I think --amend isn't possible as I already published my commit. I'd create another commit which would be added to this PR automatically. Is that ok for you?

You can always force push, git push origin <branch> -f. If you don't mind squashing these please? Just following the protocol :)

I first didn't succeed creating a failing test. But then I switched the target framework of ResourceLibUnitTests to .NET 4.5. Then most of the existing tests failed too (I also checked .NET 4.0, but that was fine). My original fix and another change in FontDirectoryResource made the tests green again :grinning:
Therefore I suggest to change the ResourceLibUnitTests to .NET 4.5 permanently. ResourceLib can stay at .NET 2.0. What do you think?
(it's already late in Germany, I'm off for today)

BTW, I guess this will fix #22 (and #25) too, because it happens with .NET 4.5.

@thoemmi Makes total sense, go for it.

I squashed the changes to a single commit:

Fixed x64 issues in ResourceId and FontDirectoryResource
Changed target framework of ResourceLibUnitTests to .NET 4.5 to make the x64 issues reproducable
updated CHANGELOG with fixed typos and punctuation :wink:

Very well, merged.


Wiki request - rewrite and simplify explanation of Array.push()

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/push

Wiki Page now exists for Array.push()
https://github.com/FreeCodeCamp/FreeCodeCamp/wiki/Array.push()

Looks great, thanks again for working on these! Closing this one as completed.


Implement filtering of emails by sender

Could you please add an option to filter forwarded emails by sender?

@Veetaha now you can setup filtering by command /filter_emails. More info how to adjust it you can get in /help

Cool feature, thanks!


Update dynamodb to 2.16.32

Updates software.amazon.awssdk:dynamodb from 2.16.17 to 2.16.32.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk", artifactId = "dynamodb" } ]

labels: library-update, semver-patch

Superseded by #1044.


Proposition to  help debugging the SPT installation  for non supported CUDA versions

What does this PR do?
Superpoint Transformers rely on ( a lot ) of  highly GPU optimized dependency ( ex: Pytorch and Pytorch geometric) . Those dependency installlation can be quite tricky  if your CUDA versions differ from the one  provided in the readme.md.
I mainly met 4 errors when I tried to install SPT on some device:

torch scatter wasn't installed with GPU support
FRNN  failed in installation ( mostly because of pytorch issue)
pgeof isn't installed
I had issue with data.keys() or data.keys depending on the version of pytorch geometrics.
I updated the readme and a created  a folder " setup". The install script has been moved to this folder and I proposed a script " check_dependency.sh" that try to identify each one  of the above errors .

Those tests is not exhaustive but it helped me tremendously to identify when ( and where)  install.sh failed  when  I had to change clusters.
Before submitting

[x ] Did you make sure title is self-explanatory and the description concisely explains the PR?
[x ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ x] Did you list all the breaking changes introduced by this pull request?
[x ] Did you test your PR locally with pytest command?
[ ] Did you run pre-commit hooks with pre-commit run -a command?

Did you have fun?
Make sure you had fun coding 🙃

Thank your for proposing this PR. I see how these sanity check scripts can be useful, but I think part of why these may be needed now is because the installation process is not super streamlined enough.
I would rather go in the direction of making the entire installation process purely relying on pip. And defining a build recipe to make the whole project pip-installable. In doing so, numerous tools could be used to better control the version of torch, PyG, etc. I have not had the time to work on this so far, but I hope I can look into this in the near future.
This may also involve moving away from FRNN, since it causes so many installation issues. But to do so, I would update my benchmark of CPU-based and GPU-based k-NN and radius-NN libraries, to define new engines for all my CPU/GPU neighbor searches. FRNN-GPU was the fastest option I found in 2023, but maybe I can find something equivalent and more convenient today. Wouldn't mind trading a little bit of speed for a better portability.
Finally, once we have an automated build recipe, we could also design automated build tests on more machine configurations and potentially extend the range of supported CUDA versions, and distribute wheels on PyPi.


Fix double migrate in validator

app is not essential for validators. It wasn't in the compose file before, and it could be removed in the future. That's why I kept the one in celery entrypoint.


RAM Only - RAM requirements

Is it possible to lower the RAM requirement for RAM only in the source?
I have 384 GiB

Not just yet, but RAM-only is planned to be lowered to 256G

Not just yet, but RAM-only is planned to be lowered to 256G

OK great. Should I hold off on shopping for more RAM? TBH I'd prefer to just use what I have.

need a mode only RAM for 128GB


Updated non-github links if sites have moved.

Only one or two of these seemed questionable. The rest stated at the top of their pages that they've moved to github.

Thanks @jessemholmes. Appreciated.


CAMEL-8394 Consolidated configurations and moved client state to component

Consolidated user configurations by removing DockerClientProfile from DockerConfiguration class. Removed map of clients from configuration class to component

Thanks we got it, do you mind closing this PR


Refactor/fetch data school page

Features

Add Infos type for SchoolPage
Add validating information shown, if not exists show "--"
Add fetch api data

Refactors

Remove modalidade field, and adjust width of adm field
Change static-dynamic infos using mock HomeData

Dale


AmazonCognitoIdentityConfig throws exception in release build

Describe the bug
This code works in debug mode on my iMac when running my Xamarin forms app.

var   credentials = new CognitoAWSCredentials(
                "ap-southeast-2:etcetcblahdeblah", // Identity pool ID
                RegionEndpoint.APSoutheast2 // Region

But throws a very poor exception when run in release build
"The type initializer for 'Amazon.CognitoIdentity.AmazonCognitoIdentityConfig' threw an exception."
var config = new AmazonCognitoIdentityConfig();   config.RegionEndpoint = RegionEndpoint.APSoutheast2;
throws the same exception.
Expected Behavior
I expect the release build version to work like the Debug version does.
Current Behavior
this exception is thrown
The type initializer for 'Amazon.CognitoIdentity.AmazonCognitoIdentityConfig' threw an exception.
Reproduction Steps
//        var config = new AmazonCognitoIdentityConfig();
  //      config.RegionEndpoint = RegionEndpoint.APSoutheast2;

        System.Diagnostics.Debug.WriteLine("ready to try");
            try
        {
credentials = new CognitoAWSCredentials(
            "ap-southeast-2:extrabigthatisntgoingtobeshared", // Identity pool ID
            RegionEndpoint.APSoutheast2 // Region
);
        }
        catch (Exception ex)
        {

            System.Diagnostics.Debug.WriteLine(ex.Message);

        }

Possible Solution
No response
Additional Information/Context
No response
AWS .NET SDK and/or Package version used
AWSSDK.CognitoIdentity <IP_ADDRESS>
Targeted .NET Platform
Xamarin forms Android
Operating System and version
iMac Monterey

Hi @bryceburrows,
Good morning.
Thanks for opening the issue. Could you please enable verbose logging and shared the detailed logs? Verbose logging could be enabled using the below code:
Amazon.AWSConfigs.LoggingConfig.LogResponses = Amazon.ResponseLoggingOption.Always;
Amazon.AWSConfigs.LoggingConfig.LogTo = Amazon.LoggingOptions.SystemDiagnostics;
Amazon.AWSConfigs.AddTraceListener("Amazon", new System.Diagnostics.ConsoleTraceListener());

Thanks,
Ashish

Hiya ConsoleTraceListener doesn't exist in the Xamarin forms version the app currently targets. I tried pointing a defaulttracerlistener to log to a file but nothing was created :/
Do you have another option?

@bryceburrows You could try using log4net as mentioned in https://aws.amazon.com/blogs/developer/logging-with-the-aws-sdk-for-net/ to log to a file. Quick question, how are you authenticating via Cognito and then using the temporary session credentials in your application?

@bryceburrows Could you please share are you authenticating via Cognito and then using the temporary session credentials in your application?

I found a solution Clean

remove  all AWS sdks
reinstall AWS sdks
In Xamarin Android build options - compile with no SDK linking.
rebuild and run  - all works.

Cle

@bryceburrows Thanks for your reply. Good to hear that it works for you. Closing this issue.

Hi, I am getting the same exception :
{
  "message": "The type initializer for 'Amazon.CognitoIdentity.AmazonCognitoIdentityConfig' threw an exception.",
  "stackTrace": "LogCapture.GetStackTrace ",
  "type": "Exception"
},

I am using ASW SDK for Unity and used s3 package.
I integrated it into Unity Editor, It works perfectly. I can send the file to s3.

When I create an Android build/ iOS build, It throws the above error and hence can't send the file to s3.

I am using IL2CPP , Unity ver 2020.3.48f1 ,  .Net ver : 2.0


dvyukov runtest repro

tools/syz-prog2c: add -repro flag
sys/test/test: add a hanging test
pkg/repro: fix execution of non-repeatig C programs

@avagin that's what I come up with for the gvisor "no output" bug:
https://syzkaller.appspot.com/bug?id=1fbceba05f872c22b58c01d19b8536118cfef229
It wasn't immediately obvious what party is guilty here and what is the right way to resolve it.
Tried to add a test, but it does not seem to be possible to test within the current infrastructure. The added test is useful on its own, but it does not really test this particular issue.


mpu6050 sensor_value_to_double value not printing using printf or uart_poll_out

Mostly following the samples/sensor/mpu6050 on a  stm32f411e_disco.
I am unable to print double values either using printf directly or using uart_poll_out with a char array generated using sprintf.
The printed value always ends up being blank.
For example: printf("Temp: %f \n", sensor_value_to_double(&temperature); always results in the following output: Temp:
However printing values typecast to int seems fine.
Fpr example: printf("Temp: %d \n", (int)sensor_value_to_double(&temperature); results in an output similar to: Temp: 30
This is true for all SENSOR_CHAN_ACCEL_XYZ, SENSOR_CHAN_GYRO_XYZ and SENSOR_CHAN_AMBIENT_TEMP.
I have not tested it with another sensor so I am not sure if its an issue with sensor_value_to_double or specific to the mpu6050
I can print other double and float values just fine.

@ric96
I don't think it is a sensor_value_to_double() issue, because I'm using it extensively for x_nucleo_iks01a3 (and other shields).  It might be an issue related to the mpu6050 sensor, but I do not have it, so I cannot try it out.

@ric96
Can you investigate more about it? I just saw the driver code and it seems fine to me.

It looks like you are using the newlib (CONFIG_NEWLIB_LIBC=y).
Please make sure that CONFIG_NEWLIB_LIBC_FLOAT_PRINTF is set to y.

Closing since an answer has been provided.


Updated logic for trailing ampersand to check if it is a comment line

I had a F77 file where the last character of a comment was an ampersand, and this erroneously made fortls treat it as a free-form file.  This is a minor fix to address this.

Thanks for the contribution! I just had one minor comment/change and then it looks good to go.