FW 3.2.7 breaks custom web certificates

          my internal CA is as follows:

Root CA
Intermediate CA
Router certificate
the router certificate is signed by my intermediate CA.
Certificate:
    Data:
        Version: 3 (0x2)
        Serial Number:
            83:a3:ae:9a:21:e4:00:4c:d2:89:5a:99:f9:41:fb:ad
        Signature Algorithm: ecdsa-with-SHA256
        Issuer: O = PaulGG Intranet, CN = PaulGG Intranet Intermediate CA
        Validity
            Not Before: Dec 14 12:04:01 2023 GMT
            Not After : Dec 15 12:05:01 2023 GMT
        Subject: CN = router.paulgg.int
        Subject Public Key Info:
            Public Key Algorithm: rsaEncryption
                RSA Public-Key: (2048 bit)
                Modulus:
                    00:a2:dc:42:d7:35:b3:dd:5b:96:86:29:4c:64:e7:
                    32:48:1c:0a:bb:e6:5f:43:ad:63:de:19:12:3e:3f:
                    a5:52:f3:20:1e:30:23:c4:ac:56:aa:d8:b8:5d:ae:
                    d8:fe:6b:e8:ce:c5:b6:e6:c8:20:36:e0:98:35:3f:
                    b5:c3:d3:51:cc:91:ca:1d:38:a7:c8:9c:9e:7b:9a:
                    5e:41:e1:5b:38:7e:b8:7d:6e:a3:d7:51:11:b9:c7:
                    1c:be:f7:44:b2:a9:f5:63:1a:f3:43:64:ea:11:f1:
                    97:4b:6b:a6:e4:95:e1:05:98:45:4f:63:8b:ae:ac:
                    8d:aa:7d:9b:a5:d8:b8:1d:61:5e:c1:9b:3a:d8:91:
                    45:57:a0:d6:b7:1c:d4:88:4e:01:e5:c3:61:b4:6d:
                    68:1d:c8:2c:00:9a:e8:fb:0c:76:f1:f0:b8:d2:22:
                    3b:35:c8:9c:b9:80:db:7b:d9:ee:b3:af:b0:0d:fc:
                    70:b8:ef:0b:f6:4c:39:b5:c8:d0:9c:86:b3:71:a1:
                    a5:42:26:f7:3e:82:4c:23:a0:84:67:f6:0a:a9:34:
                    eb:ac:f2:f6:bc:a5:f5:6e:1a:34:e8:2a:a6:7e:42:
                    90:28:b9:db:34:1d:0d:b7:a1:1a:65:d3:61:55:ad:
                    ec:52:0a:fe:f5:11:41:1d:21:12:25:fc:02:e8:4d:
                    9d:cd
                Exponent: 65537 (0x10001)
        X509v3 extensions:
            X509v3 Authority Key Identifier:
                keyid:58:1E:16:42:18:5A:CF:A5:DA:29:2E:37:50:AF:4E:5D:1B:CE:B9:93

            X509v3 Key Usage: critical
                Digital Signature, Key Encipherment
            X509v3 Extended Key Usage:
                TLS Web Server Authentication, TLS Web Client Authentication
            X509v3 Subject Key Identifier:
                98:41:E0:9B:1A:F8:23:F4:8F:6C:D7:53:37:A4:4E:D6:79:D3:A9:57
            X509v3 Subject Alternative Name:
                DNS:router.paulgg.int
            <IP_ADDRESS>.4.1.37476.9000.64.1:
<EMAIL_ADDRESS>    Signature Algorithm: ecdsa-with-SHA256
         30:45:02:21:00:be:81:ea:60:62:19:71:3e:0b:54:48:43:79:
         56:5f:b5:59:23:dc:21:1b:94:16:07:a6:74:f6:5a:24:ff:af:
         43:02:20:25:29:02:8e:64:0f:01:b0:4e:b8:3d:47:8c:c5:b1:
         f1:80:46:86:fd:48:55:df:65:fb:af:fb:2e:24:eb:8f:a9

I just checked /etc/ssl/certs and looks like there's some stuff that has been deleted, including my root certificate from the keystore.

When I have some more time I'll play around with this...
Originally posted by @therealpaulgg in https://github.com/alxwolf/ubios-cert/issues/61#issuecomment-1856221243

@therealpaulgg opened a new issue, your case is different from the RADIUS issue many people seem to see.
You debugged that your root certificate is gone from keystore - that may be another hint in the direction that UniFiOS says "incorrect / incomplete chain -> trash it, put self-signed in".
but as you wrote, root cause for trust being broken in cert store is unclear...
I assume when you copy the certs manually and restart unifi-core, they get culled, too?

So I mentioned in my last comment that I ended up trying to fix my cert store manually by both re-adding my Root CA and intermediate CA (I didnt need this previously)
ran openssl verify and it reported OK
Yeah copied the certs manually and they are gone.

This one is a bit trickier to solve than the RADIUS one. Still not entirely sure what to do.
I did some digging based on what @ouaibe had mentioned in the other issue, and I was only able to find this one entry:
"unifiNetwork": {
   "certificate": {
    "crt": "CERT_HERE"
   },
   "controllerURL": "https://<IP_ADDRESS>:8443",
   "enabled": true,
   "informURL": "http://<IP_ADDRESS>:8080/inform",
   "sitename": "default",
   "uciAllowList": []
  },

this CERT_HERE is also a base64 encoded value but does not decode the same way that the radius certificates do, so I am a bit confused.
Tried searching for other configuration files and had no luck. There is one called /data/udapi-config/udapi-net-cfg.json but that appears to just be a subset of the previously mentioned /data/udapi-config/ubios-udapi-server/ubios-udapi-server.state.
Its like the implementation of RADIUS and the web certs are just completely different, and I'm not able to find where they are configured at.

Looking at it, that certificate is in DER format (binary), not PEM. These are both interchangeable/convertible using the following commands:
DER -> PEM
openssl x509 -inform der -in certificatename.der -out certificatename.pem
PEM->DER
openssl x509 -outform der -in certificatename.pem -out certificatename.der
So in theory, if you already have your local CRT file server.crt  in PEM format, you can do something like this:
jq --arg content "$(openssl x509 -outform der -in server.pem | base64)" '.services.unifiNetwork.certificate.crt = $content' /data/udapi-config/ubios-udapi-server/ubios-udapi-server.state > /data/udapi-config/ubios-udapi-server/ubios-udapi-server.new.state

Then validate the NEW file, the JSON content at .services.unifiNetwork.certificate.crt and if you're satisified with it (having taken a backup) you can replace it.

This makes sense... only concern I have is where is the key? previously for web certs it needed both the cert and the key. You should need both pieces, right?

A CRT for a server can (should?) contain both the cert and the private key. You could try generating a regular CRT file in PEM format, that would contain the two sections:
-----BEGIN CERTIFICATE-----
...
-----END CERTIFICATE-----
-----BEGIN PRIVATE KEY-----
...
-----END PRIVATE KEY-----

And then convert that to DER and see if it works?

After digging a bit more, the unifiNetwork certificate in the JSON config file has the exact same fingerprint than the one in the keystore at /data/unifi/data/keystore.
Compare the output of  openssl x509 -inform der -in unifinetwork.der -noout -fingerprint and the output of keytool -v -list -keystore /data/unifi/data/keystore -storepass aircontrolenterprise
This might means that the keystore is used to back the signature for that part of the certificate management system.
FWIW you can export the existing private key in PEM format using the following commands:
keytool -importkeystore -srckeystore /data/unifi/data/keystore -srcstorepass aircontrolenterprise -destkeystore /tmp/keystore.p12 -deststoretype PKCS12 -deststorepass aircontrolenterprise

openssl pkcs12 -in /tmp/keystore.p12 -nodes -nocerts -out /tmp/private_key.pem

But I'm not sure how that private key would be useful for you, instead you could try adding your own private key/certificate to the existing keystore and see how it behaves with udapi-server.
Good luck!

Unfortunately it seems this certificate is completely different from the one on the web portal. Fingerprints dont match at all and the issuers are totally different (CN = unifi.local for web cert vs. C = US, ST = New York, L = New York, O = Ubiquiti Inc., OU = UniFi, CN = UniFi)
I found a log in /data/unifi-core/logs/http.log saying:
2023-12-18T21:15:19.493Z - info: Self signed certificate needs regenerated, regenerating
2023-12-18T21:15:19.494Z - info: Generating /data/unifi-core/config/unifi-core.crt

wonder if the underlying cert implementations  for unifi-core and the other stuff are just totally different.

Ah, reading your message I had assumed the CRT in the JSON file was the right one, sorry to have mis-interpreted that.
It's very likely they combine 3+ certificate management mechanisms, it seems to be all over the place between freeradius, the keystore (that must be used for something, maybe intra-Unifi comms over HTTPS), some certs for Protect and the unifi-core/Web UI stuff...
One thing you "could" try, maybe to get more output out of udapi-server boot sequence is to chattr +i the unifi-core.crt file. It is very likely they're not checking for that so either it makes it immutable across reboots as a temporary solution for you, or crashes udapi-server and/or you get more info in its journalctl -u because it cannot replace the file...
Either way, if they are re-generating it, the private key and the cert must be somewhere...

One thing you "could" try, maybe to get more output out of udapi-server boot sequence is to chattr +i the unifi-core.crt file. It is very likely they're not checking for that so either it makes it immutable across reboots as a temporary solution for you, or crashes udapi-server and/or you get more info in its journalctl -u because it cannot replace the file...

After being frustrated by 3.2.9's continuation of 3.2.7's bad behavior, I found this thread. I can confirm that marking both the cert and key immutable prevent them from being replaced upon restarting the unifi-core service. It's a kludge, but it'll do until a real fix comes along!

One thing you "could" try, maybe to get more output out of udapi-server boot sequence is to chattr +i the unifi-core.crt file. It is very likely they're not checking for that so either it makes it immutable across reboots as a temporary solution for you, or crashes udapi-server and/or you get more info in its journalctl -u because it cannot replace the file...

After being frustrated by 3.2.9's continuation of 3.2.7's bad behavior, I found this thread. I can confirm that marking both the cert and key immutable prevent them from being replaced upon restarting the unifi-core service. It's a kludge, but it'll do until a real fix comes along!

Absolutely genius workaround...this totally solves my problem. Finally got around to actually implementing it on my Unifi router and its pretty straightforward. It is a little more complicated since I'm using step-ca which renews certs daily, but it was easy enough to modify the renewal service to simply make the file temporarily mutable and then make it immutable once renewed.
Thank you very much, I had pretty much given up on the idea of accessing Unifi from a trusted domain at this point.

Not sure this helps, but I'm on EA release v3.2.12 and all I had to do was place move the cert and key to /data/eus_certificates/unifi-os.crt and /data/eus_certificates/unifi-os.key respectively.

@forrestrae interesting observation, have you ever used Glenn R.'s "UniFi Let's Encrypt" before?
That script creates such directory. if you did not use it before, maybe UI has implemented that scripts mechanisms into firmware.

guys, what about now, anything updates? this issue still

guys, what about now, anything updates? this issue still

you can try your luck with this systemd file. It's the best I've got personally and makes use of the immutable certs.
[Unit]
Description=Certificate renewer for %I
After=network-online.target
Documentation=https://smallstep.com/docs/step-ca/certificate-authority-server-production
StartLimitIntervalSec=0
; PartOf=cert-renewer.target

[Service]
Type=oneshot
User=root

; YOU MAY NEED TO UPDATE THESE PATHS YOURSELF
Environment=STEPPATH=/etc/step-ca \
            CERT_LOCATION=/etc/step/certs/%i.crt \
            KEY_LOCATION=/etc/step/certs/%i.key

; Make the certificate mutable before attempting renewal
ExecStartPre=/usr/bin/chattr -i ${CERT_LOCATION}
ExecStartPre=/usr/bin/chattr -i ${KEY_LOCATION}

; ExecCondition checks if the certificate is ready for renewal,
; based on the exit status of the command.
; I put my step binary in my user's folder because if I put it in /usr/local/bin the OS kept wiping it
ExecCondition=/root/step_0.24.3/bin/step certificate needs-renewal ${CERT_LOCATION}

; ExecStart renews the certificate, if ExecStartPre was successful.
ExecStart=/root/step_0.24.3/bin/step ca renew --force ${CERT_LOCATION} ${KEY_LOCATION}

; Make the certificate immutable after renewal
ExecStartPost=/usr/bin/chattr +i ${CERT_LOCATION}
ExecStartPost=/usr/bin/chattr +i ${KEY_LOCATION}

; Try to reload or restart the systemd service that relies on this cert-renewer
; If the relying service doesn't exist, forge ahead.
ExecStartPost=-/usr/bin/env sh -c "! systemctl --quiet is-active %i.service || systemctl try-reload-or-restart %i"

[Install]
WantedBy=multi-user.target

systemctl start<EMAIL_ADDRESS>

guys, what about now, anything updates? this issue still

you can try branch "2024-fix" to see if this fixes this issue. RADIUS has not yet been addressed.

guys, what about now, anything updates? this issue still

you can try your luck with this systemd file. It's the best I've got personally and makes use of the immutable certs.
[Unit]
Description=Certificate renewer for %I
After=network-online.target
Documentation=https://smallstep.com/docs/step-ca/certificate-authority-server-production
StartLimitIntervalSec=0
; PartOf=cert-renewer.target

[Service]
Type=oneshot
User=root

; YOU MAY NEED TO UPDATE THESE PATHS YOURSELF
Environment=STEPPATH=/etc/step-ca \
            CERT_LOCATION=/etc/step/certs/%i.crt \
            KEY_LOCATION=/etc/step/certs/%i.key

; Make the certificate mutable before attempting renewal
ExecStartPre=/usr/bin/chattr -i ${CERT_LOCATION}
ExecStartPre=/usr/bin/chattr -i ${KEY_LOCATION}

; ExecCondition checks if the certificate is ready for renewal,
; based on the exit status of the command.
; I put my step binary in my user's folder because if I put it in /usr/local/bin the OS kept wiping it
ExecCondition=/root/step_0.24.3/bin/step certificate needs-renewal ${CERT_LOCATION}

; ExecStart renews the certificate, if ExecStartPre was successful.
ExecStart=/root/step_0.24.3/bin/step ca renew --force ${CERT_LOCATION} ${KEY_LOCATION}

; Make the certificate immutable after renewal
ExecStartPost=/usr/bin/chattr +i ${CERT_LOCATION}
ExecStartPost=/usr/bin/chattr +i ${KEY_LOCATION}

; Try to reload or restart the systemd service that relies on this cert-renewer
; If the relying service doesn't exist, forge ahead.
ExecStartPost=-/usr/bin/env sh -c "! systemctl --quiet is-active %i.service || systemctl try-reload-or-restart %i"

[Install]
WantedBy=multi-user.target

systemctl start<EMAIL_ADDRESS>

guys, what about now, anything updates? this issue still

@cls1997 you can try branch "2024-fix" to see if this fixes this issue. RADIUS has not yet been addressed.
So far it has survived service restarts and reboots, will see if it persists the night when some more black magic seems to happen.
If it does, I will merge this with main and take on the "network application" part next, then RADIUS.
why does UI have to make it so complicated...

Thanks a lot. I will try the solution mentioned :)

Works for me so now merged into main. Network application („hotspot portal“) still an issue.

Hotspot portal now fixed, too.
please re-open issue if required.


Credentials are missing

Repro steps:

I follow this lesson: https://docs.microsoft.com/en-us/learn/modules/build-cosmos-db-app-with-vscode/2-setup-cosmosdb?source=learn

I've created a new Azure Cosmos DB account in Visual Studio Code;
After creation it shows an error (given below);
I can't follow the next steps as I have the same error. I can't see a place to put in my credentials;

Action: cosmosDB.createDocDBDatabase
Error type: 407
Error Message:

Access Denied (authentication_failed)

Your credentials could not be authenticated: "Credentials are missing.". You will not be permitted access until your credentials can be verified.This is typically caused by an incorrect username and/or password, but could also be caused by network problems.
For assistance, contact your network support team.
Version: 0.11.0
OS: win32
Product: Visual Studio Code
Product Version: 1.40.2
Language: en

Hi @slarionoff
Are you behind a proxy by chance? If so, you may be able to get around this by using this vscode setting:
"http.proxyStrictSSL": false

Hi @nturinski! This helped. I'm really behind proxy, after setting this parameter I was able to pass further. Thanks a lot!


Consolidate bibtex files

We currently have bibtex files all over the place:
./datasets/sources.bib
./phylogenies/bouckaert_et_al2012/source.bib
./phylogenies/bowern_and_atkinson2012/source.bib
./phylogenies/chacon_and_list2015/source.bib
./phylogenies/chang_et_al2015/source.bib
./phylogenies/dunn_et_al2011_utoaztecan/source.bib
./phylogenies/gray_et_al2009/source.bib
./phylogenies/grollemund_et_al2015/source.bib
./phylogenies/honkola_et_al2013/source.bib
./phylogenies/kitchen_et_al2009/source.bib
./phylogenies/lee_and_hasegawa2011/source.bib
./phylogenies/michael_et_al2015/source.bib
./phylogenies/sicoli_and_holton2014/source.bib

i.e. datasets/ has a combined sources.bib, while each phylogeny has a single source.bib.
Should we consolidate these into a single sources.bib, perhaps in its own top level file (./sources.bib)? or keep them as dataset specific resources?

I'd vote for "consolidate on top-level". This way, it would also be simpler to reference these sources in descriptive text on the website: Just lookup the bibtex key in the one bib file and have a template function which turns this into a full reference link.

Yes, and it's DRYer.

Can I just merge these now, or does there need to be some change to pydplace too?

Done in https://github.com/D-PLACE/dplace-data/pull/272

I don't think pydplace looks at these atm:
$ grep -r sources.bib src/
src/pydplace/api.py:        return BibFile(self.path('datasets', 'sources.bib'))

Ah, ok. If you merge datasets/sources.bib, too, it'll need to change.

Yeah, just realised that's why the tests were failing :) What do you think -- a top level sources.bib (so change pydplace) or a bib in datasets/ and phylogenies/?

top-level, I think. It's still fairly small.

Yeah, that makes sense -- shall I make a PR for pydplace?

Oops, sorry -- I think I pushed to master on pydplace last night. Should have gone to bed earlier :/


Dropped timesteps in replay memory lead to empty updates and crashes

In Replay.tf_retrieve_timesteps, lines 55 and 56, terminal timesteps are excluded from a sampled batch:
https://github.com/reinforceio/tensorforce/blob/master/tensorforce/core/memories/replay.py#L56
This can lead to empty update batches, especially in cases such as CartPole with short episode lengths. For instance, running DDPG with batch_size = 4 often leads to crashes because of empty batches (Thanks @devnag_twitter for pointing that out).
As discussed, the solution could be to exclude terminal timesteps from selection to begin with, and/or employ a loop to fill empty spaces.

I've played with DQN and it may has the same problem,
First the problem is in PrioritizedReplay, where
priority_indices = tf.squeeze(tf.where(condition=mask))
may cause priority_indices to be a tensor with no shape
so I change the code to priority_indices = tf.squeeze(tf.where(condition=mask), [1])
then the second problem comes:
InvalidArgumentError : Inputs to operation dqn/observe-timestep/cond/optimization/clipped-step/step/adam/step/loss/loss-per-instance/Select of type Select must have the same size and shape.  Input 0: [100] != input 1: [0]
the error code is in
https://github.com/reinforceio/tensorforce/blob/master/tensorforce/models/q_model.py#L149
which I've debug it because the next_q_value is a tensor with shape [] and it because in q_model.tf_loss_per_instance, the action_taken is a tensor with shape [].

Just quickly interjecting that prioritised replay is still under construction and not meant to be used yet because the priority update is tricky in pure tensorflow (and not implemented yet)

@michaelschaarschmidt  thanks for replying, then I change the memory mode to replay, the second problem still occurs.

Should be fixed via https://github.com/reinforceio/tensorforce/commit/1109dc5df81e2ddeffd83b9843d50710a014d433, introduced a simple resampling loop, dqn test and quickstart working for me, hope this solves it.

Thanks, I'll test it on Monday!

@michaelschaarschmidt  the replay problem is solved, thanks!

great


Configuring Raven GDAL and Rasterio

I hoping to finalize the Raven GIS class methods in the coming weeks but I'm currently blocked on an issue.
In order to more easily perform raster masking using vectors (extracting points within the contour and setting outside pixels to NaN's) most of the literature suggests that Rasterio would be a robust library to perform this. Unfortunately, Rasterio and native GDAL don't work well together; they can't easily be installed alongside each other even with Anaconda.
My question here would be what seems like a more worthwhile goal: adding some spaghetti code methods cobbled together that seem to do an okay job for now, or configuring Rasterio to to work in its own environment (hopefully its own bird) to remove a lot of boilerplate code and leverage their functionalities. Questions of time, resources, and priorities come to mind.
In the mean time, I'll be looking around for other options (system calls to bash libraries that can more or less do this). Hopefully I can find something that isn't too resource-heavy.
Thoughts?

If we go full tilt with rasterio, what are we missing ?

I'll look into it. AFAIK GDAL handles a lot of GeoTIFF data handling/processing and any processes that would be using it would need to use the rasterio version. It's an "all or nothing" sort of situation; if we don't think we want to leverage traditional GDAL, we shouldn't have any problems though.

@Zeitsperre please consult with @crimbeaulima too. He use GDAL quite a bit, maybe he can advise.

Update: Rasterio and GDAL can be used synonymously so long as GDAL is installed via conda alongside Rasteri and so long as specific types of objects opened with one are not passed to the other (e.g. GDAL band objects). The only instances of Rasterio and GDAL needing to be employed at the same time is in the wps_generic_terrain_analysis.py process and even then the GDAL subprocess call bindings are used on file paths with no objects are passed between each other.
Rasterio has a much more pythonic API and I suggest it should be used as the standard whenever reading/writing DEMs over GDAL's python bindings. With #67 merged, this is no longer a relevant discussion.


☕Buy me a coffee not working

Hey man, this tool saved me some time so I wanted to buy you a coffee, but it says that the creator hasn't selected a payment method, so I couldn't.

thanks @thilonel ! fixed now


question about quantization

Using quantization, with the same parameters, the difference of the training accuracy is about 5-10% each time？

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):Linux Ubuntu 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):pip
TensorFlow version (use command below):1.4
Python version:3.4.3
Bazel version (if compiling from source):1.9
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:
Exact command to reproduce:

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with:
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"

Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.

Nagging Assignee @liyunlu0618: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Can you be more specific, like models of interest, etc? Also, what are you trying to compare here? The accuracy of the floating point model vs. that of the quantized one?

Closing due to inactivity.


kctrl to support offline rendering for given AppCr.

Describe the problem/challenge you have
Currently, kctrl have dev subcommand which can be used to deploy an app without having kapp-ctrl on the target Kubernetes cluster.
That's a very powerful feature. However, as a user of kctrl it's not clear as to what the helm template <imgpkg_bundle> would look like unless it's deployed.
Describe the solution you'd like
Once a user has his/her AppCr.yaml ready, he/she should be able to leverage kctrl to render the app without having a cluster in place.
Expectation: kctrl dev -a <app_name> --offline should return rendered resources.

Imgpkg bundle pushed to the registry, creds can be passed through command line args or any other means.
This should not expect a kubeconfig file or a kubernetes cluster.
Should not deploy any app on cluster even if kubeconfig is provided.

Example:
Given AppCr:
apiVersion: kappctrl.k14s.io/v1alpha1
kind: App
...
spec:
  deploy:
  - kapp:
      delete:
        rawOptions:
        - --apply-ignored=true
      rawOptions:
      - --diff-changes=true
  fetch:
  - imgpkgBundle:
      image: kind-registry.local:5000/tcxtest/sheikhmo/es-operator:sheikhmo_v1
      secretRef:
        name: kubelet-pull-secret
  serviceAccountName: cluster-admin-sa
  syncPeriod: 2m0s
  template:
  - helmTemplate:
      path: es-operator
      valuesFrom:
      - path: es-operator/values.yaml
      - configMapRef:
          name: override-es-operator-app-cr-config
  - kbld:
      paths:
      - '-'
      - .imgpkg/images.yml
 ...

Expected to see the rendered output, similar to what I see when deploying an app. Like the following:
| Target cluster 'https://<IP_ADDRESS>:56744/' (nodes: kind-control-plane)
	    | @@ create serviceaccount/es-operator (v1) namespace: default @@
	    |       0 + apiVersion: v1
	    |       1 + kind: ServiceAccount
	    |       2 + metadata:
	    |       3 +   labels:
	    |       4 +     kapp.k14s.io/app: "1664322049183094000"
	    |       5 +     kapp.k14s.io/association: v1.2d46bb26f0d47b1e8a9796066618739b
	    |       6 +   name: es-operator
	    |       7 +   namespace: default
	    |       8 +
	    | @@ create clusterrole/es-operator (rbac.authorization.k8s.io/v1) cluster @@
	    |       0 + apiVersion: rbac.authorization.k8s.io/v1
	    |       1 + kind: ClusterRole
	    |       2 + metadata:
	    |       3 +   labels:
	    |       4 +     kapp.k14s.io/app: "1664322049183094000"
	    |       5 +     kapp.k14s.io/association: v1.cf41f0c7ac

That way as a user I would be able to see what resources going to get deployed and what values would look like.
Anything else you would like to add:
I believe under the covers that's what kctrl does. I.e. pull imgpkg bundle from registry and renders the resources on shell terminal before deploying and app. So, seems like it's possible to achieve this.

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

cc @renuy
my 2c --offline => --template

cc @renuy
my 2c --offline => --template

thanks updated  my post


feat(sdk): make all @Photon.handler calls async in its own pool.

In the past, when we do
@Photon.handler
def foo(self):
    ...

The code runs in the main uvicorn event loop, and blocks the uvicorn server (during which time basic things like "/openapi.json" and "healthz" are not accessible).
This PR adds two functionalities:

All endpoints defined in @Photon.handler, when deployed, now run in a separate thread, so the uvicorn main event loop is not blocked.
In default, the number of endpoint calls being run concurrently is 1, which is the same as before. With handler_max_concurrency, one can control the concurrency so that multiple endpoints can be run at the same time in different thread. NOTE THAT, the photon framework does not guarantee thread safety of user code - lepton's scaffolding code is thread safe, but any user code's thread safety is owned by the author of the code.
The handler_max_concurrency and background_job_max_concurrency do not share the same pool. One is responsibile for the user code that runs handler endpoints and background jobs concurrently.

This does not apply to endpoints that are defined with async def. In other words, if we defined
@Photon.handler
async def foo(self):
    ...

Then this async function still runs in the main uvicorn loop, and one is expected to manage one's own threading and async behaviors.
It is highly unrecommended to use both sync and async handlers - just sayin'.

Changed asyncfy to the proper name asyncfy_with_semaphore, and kept the original asyncfy file. Landing.


Feature: Enhance Website Design for a More Professional and Balanced Look

Description
I'd like to propose a few design enhancements for our website to create a more professional and balanced appearance.The goal is to make the website more visually appealing and user-friendly.
Proposed Solution

In the second of the website ,titled "Connect," I have applied changes to improve the visual appeal and balance of the section.
Specifically, I have swapped the image position to the right side of the text to create a more visually appealing and balanced layout.

Screenshots
Before:
https://github.com/MilanCommunity/Milan/assets/110025628/9b071972-6e81-48ea-9583-6858a026c59e
After:
https://github.com/MilanCommunity/Milan/assets/110025628/3dd25807-d7f4-44d9-ac5b-d8f783224c52
These changes aim to improve the overall user experience and make the website more inviting to visitors.
🥦 Browser
Google Chrome
Checklist ✅

[X] I checked and didn't find similar issue
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (blank for no)

@tamalCodes please assign this issue to me. Thanks! :)

Hey @varundhand @tamalCodes , I am confident that I can do this UI change. It's really simple, and multiple ways to do it:

We can set the flex direction of the parent container as row reverse
Or we can just arrange those elements in the html/jsx code itself.

please assign me this issue.

Hi there sorry for being late, was not well at all. See for now I would not recommend making any design change as we will be refactoring the whole design soon. I have sole major UI changes planning and will launch it up soon.
Till then I'll be closing this, will re-open if needed.

Sure thing @tamalCodes . I'll be happy to incorporate UI changes as well! You can ping me up anytime


Jupyter CUDA Tensorflow: Git client and Elyra broken

The Jupyter CUDA Tensorflow image (repository) needs to be fixed:

The JupyterLab git client is unable to clone git repositories.
Elyra is unable to connect to RHOAI pipeline servers.

I had problem with cloning git repositories with the JupyterLab git client and I solved it by updating pexpect to version 4.9.0 (from  version 4.8.0).


Is setting forceInMemoryLog=true production-ready?

In what area(s)?

/area placement

Ask your question here
I see that forceInMemoryLog defaults to false, and placement requires persistent volumes (PV). If forceInMemoryLog is set to true, then placement does not require PV, which is convenient.
However, I would like to know what the implications of this parameter are? Can it meet production needs? Thank you.

@majian159 yes you can run in production with this setting as many do, especially on OpenShift.


Support - recommended way to read in udp

I am pretty new to react-native and also developing for mobile; I need to listen for UDP messages by connecting to the local internet (nearest wifi router)
I assume my device (Android) will have wifi connectivity
If I listen for udp on a port, I should get data, right?
I am developing for Android - just curious why there is so much less Java than Objective-C for this project?
Objectively speaking, is this the best module to use for the job of listening for UDP messages coming via a local wifi router?
sorry for super naive questions but trying to figure this outt

also, when I ran the commands as listed in the readme.md
https://github.com/tradle/react-native-udp/blob/master/examples/rctsockets/android/app/src/main/java/com/rctsockets/MainApplication.java#L11
these lines were already added to my .java files, that was magic, why did that happen? :) I mean I guess that's good, never seen people modify java source like that!

Ok more practical question:
I have this code in a test script:
const dgram = require('dgram');
const server = dgram.createSocket('udp4');

server.on('error', (err) => {
  console.log(`server error:\n${err.stack}`);
  server.close();
});

server.on('message', (msg, rinfo) => {
  console.log(`server got: ${msg} from ${rinfo.address}:${rinfo.port}`);
});

server.on('listening', () => {
  var address = server.address();
  console.log(`server listening ${address.address}:${address.port}`);
});

server.bind(12345);

setInterval(function(){
   server.send('abc', 0, 3, 12345,'localhost', function(err){
          console.log('message sent.');
   })
}, 3000);

and then in my Android React-Native code, I have:
const Buffer = global.Buffer = require('buffer');
var dgram = require('dgram')
const socket = dgram.createSocket('udp4');

socket.bind(12345);

socket.once('listening', function() {
  console.log('listening!!');
})

socket.on('message', function(msg, rinfo) {
  console.log('message was received', msg);
    socket.send('def', 0, 3, 12345,'localhost', function(err){
            console.log('message sent.');
     });
});

export default socket;

as you can see, both UDP socket servers are bound to the same port, and I don't see any errors - but I am not getting any messages in my Android log files.
Any idea why the following setup would mean that my React Native code would not receive UDP messages?
The good thing is that my test script server is receiving it's own messages (it's sending messages out to any services listening on its own port). Since I do not set exclusive:true, it seems like we can bind more than one service to a port.

Btw I tried having each UDP server listen on a different port, and that didn't make a difference. My React-Native UDP server still does not receive any messages from the test script server.

@ORESoftware thanks for raising this. I tried your setup and reproduced your results. I think it's cause the emulator has its own ip address, or rather the VM that runs it. There's some port forwarding going on as you can see. I got them connected in three steps:

in index.android.js, set the local IP of your machine, in my case <IP_ADDRESS>:

socket.send('def', 0, 3, 12346, '<IP_ADDRESS>', function(err) {

run the test script and emulator. In the test script, I started getting messages like this:

server got: u� from <IP_ADDRESS>:65523

notice the forwarded port, 65523
3. I then changed the test script as follows:
server.send('abc', 0, 3, 65523, '<IP_ADDRESS>', function(err){

and then there was magic, see below:

@ORESoftware
Q: I am developing for Android - just curious why there is so much less Java than Objective-C for this project?
A: the amount of code authored in this module is about the same for Obj-C and Java, but the Obj-C end imports the GCDAsyncUdpSocket library from CocoaAsyncSocket
Q: Objectively speaking, is this the best module to use for the job of listening for UDP messages coming via a local wifi router?
A: objectively speaking, I only know of this module for using UDP sockets in react-native

thanks for the info, very useful, shared it with my team, thanks again


Add description for tags

Tags ordering can be handled via tags description field [1]

docs = OpenAPIHandler(
    info=Info(title="Example API", version="0.0.1"),
    tags=[
        Tag(name="Users"),
        Tag(name="Favorites"),
        Tag(name="Reactions"),
        Tag(name="Topics"),
        Tag(name="Reviews"),
        Tag(name="Comments"),g
    ],
)
docs.bind_app(app)

1 - https://swagger.io/docs/specification/grouping-operations-with-tags/
Issue: #327

This is very good, thanks you @tyzhnenko. I'm willing to merge your PR, as I wrote in the linked issue. I recommend one more thing, I try to prepare an example later tonight. 😄

@RobertoPrevato I've updated my PR please take a look when you have time to do that.
by the way, it seems the new release of setuptools breaks the project pipelines.

@tyzhnenko I merge your PR. I started looking into the build issues, I fix them later. Thanks again for your help! By the way, I had a wonderful idea on how to improve the development experience with BlackSheep - something I didn´t like myself. I share it soon. :)


What does the defaults option do in belongs_to?

The documentation states:
:defaults - Default values to use when building the association
With no further elaboration, this is not particularly enlightening. I can't find it easily in the source either.
I want to set a default foreign key, but I have no idea how to do that with this option, or indeed if this is the correct option to use.

Imagine Post.has_many :comments, defaults: [public: true]. Then when you say Ecto.build_assoc(post, :comments) that comment will have comment.public == true, regardless of the default in the comment schema. Can you please send a PR amending the docs? Thank you!

PR opened.
Does this mean there is no way to set a default foreign key value?

@samphilipd only for another association than the one being built. Can the default foreign key be set as a default in the schema?

That's exactly what I'm trying to do:
I want something like this:
 belongs_to(
      :buyer_property_interest_status,
      NestDB.InputChoice.BuyerPropertyInterestStatus,
      default: @b01_enquired_id
    )

But the only available option is defaults - default doesn't exist.

Great, that works!


⚠ ETF has degraded performance

In 51f375d, ETF (https://markets.hankyung.com/equity-product/etf) experienced degraded performance:

HTTP code: 200
Response time: 6054 ms

Resolved: 신규상장 performance has improved in 55eb771.


GitHub actions

Remove Travis, which is now paid only and move to Github actions, so that on every push you get the binary files.

@omersiar probably you need to activate the actions for this repo, but the result is like this: https://github.com/FablabTorino/esp-rfid/actions/runs/1794810394 you can download the binaries from the artifacts section at the bottom.


Merging referrals branch into master

Merging referrals branch into master

OPS Build status updates of commit 5671d66:
:clock10: Preparing: average preparing time is 40 sec(s)

OPS Build status updates of commit 5671d66:
:clock10: Incremental building: average incremental building time is 18 sec(s)

OPS Build status updates of commit 5671d66:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

partner-rest/TOC.yml
:white_check_mark:Succeeded
View

partner-rest/develop/api-authentication.md
:white_check_mark:Succeeded
View

partner-rest/develop/create-a-referral.md
:white_check_mark:Succeeded
View

partner-rest/develop/headers.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


V2/mst/log noid

@marcstern ,
I don't understand the title of this PR. Could you please describe the perceived problem and the change being proposed?

Sorry, it's a mistake. Ignore this.


mySIMBL plugins only work with SIP disabled

Edit: this issue was my own user error and can be closed (sorry!). FWIW, I had written a script to delay loading the SIMBLAgent until some other startup processes were complete, as loading it earlier conflicted with an input manager I use...when I upgraded to High Sierra, the script was removed from my startup items, so SIMBLAgent was not running.

No problem. I'll close this.


Issue with SurroundingRectangle not surrounding the text that it is supposed to

I have run into a major issue with SurroundingRectangle when I was animating with manim community edition.
So, I have a piece of code here,
from manim import *

class Test(Scene):
    def construct(self):
        t = MathTex(r"\sqrt", r"[", r"n", r"]{", r"x^4\sqrt{x^{-5}\sqrt{x^6}}",
                    r"}=", r"\left(x^4\sqrt{x^{-5}\sqrt{x^6}}\right)", r"^{1 \over ", r"n", r"}")
        t.move_to(np.array([0, -0.75, 0]))
        fb = SurroundingRectangle(t[4])
        self.add(t, fb)

This is supposed to surround x^4\sqrt{x^{-5}\sqrt{x^6}} with the SurroundingRectangle fb. However, the SurroundingRectangle does not surround it properly.

I tested with t[4] and I observed that it does show the correct part of the equation
from manim import *

class Test(Scene):
    def construct(self):
        t = MathTex(r"\sqrt", r"[", r"n", r"]{", r"x^4\sqrt{x^{-5}\sqrt{x^6}}",
                     r"}=", r"\left(x^4\sqrt{x^{-5}\sqrt{x^6}}\right)", r"^{1 \over ", r"n", r"}")
        t.move_to(np.array([0, -0.75, 0]))
        tt = MathTex(t[4].get_tex_string())
        self.add(tt)

I have come here after consulting this in the discord server and it appears to be a bug. If it is not, please provide me with a solution to this. If it is, I might need an alternative to this.

I would suggest rendering the index_labels for the tex first:
class Test(Scene):
    def construct(self):
        t = MathTex("""
            \\sqrt[n]{x^4\\sqrt{x^{-5}\\sqrt{x^6}}}
            =\\left(x^4\\sqrt{x^{-5}\\sqrt{x^6}}\\right)^{1 \over n}
        """)
        t.move_to(ORIGIN)
        self.add(t, index_labels(t[0]))

And then use the indexing to form a SurroundingRectangle
class Test(Scene):
    def construct(self):
        t = MathTex(
            """
            \\sqrt[n]{x^4\\sqrt{x^{-5}\\sqrt{x^6}}}
            =\\left(x^4\\sqrt{x^{-5}\\sqrt{x^6}}\\right)^{1 \over n}
            """
        )
        t.move_to(ORIGIN)
        self.add(t, SurroundingRectangle(t[0][3:14]))

This way, you can see the individual pieces and make it reliable. But the main problem is still a bug itself. I hope this helps and I hope the partitioning bug is solved soon.

I would suggest rendering the index_labels for the tex first:
class Test(Scene):
    def construct(self):
        t = MathTex("""
            \\sqrt[n]{x^4\\sqrt{x^{-5}\\sqrt{x^6}}}
            =\\left(x^4\\sqrt{x^{-5}\\sqrt{x^6}}\\right)^{1 \over n}
        """)
        t.move_to(ORIGIN)
        self.add(t, index_labels(t[0]))

And then use the indexing to form a SurroundingRectangle
class Test(Scene):
    def construct(self):
        t = MathTex(
            """
            \\sqrt[n]{x^4\\sqrt{x^{-5}\\sqrt{x^6}}}
            =\\left(x^4\\sqrt{x^{-5}\\sqrt{x^6}}\\right)^{1 \over n}
            """
        )
        t.move_to(ORIGIN)
        self.add(t, SurroundingRectangle(t[0][3:14]))

This way, you can see the individual pieces and make it reliable. But the main problem is still a bug itself. I hope this helps and I hope the partitioning bug is solved soon.

Yes, I figured this out on the discord a while ago. But thank you for replying.


Update essential.css

Improve contrast and make cards stand out from background on board view by making card surface white.

@peterk Thanks for the contribution!


Using RNode in async code doesn't work because of Rc

I am writing a project that uses xrust at https://git.sr.ht/~latex/streamlist
I'm trying to use xrust in async code, and it's not working. How would I get this to work?
error: future cannot be sent between threads safely
   --> src/serve.rs:80:9
    |
80  | /         Box::pin(async move {
81  | |             let response: Response = future.await?;
82  | |             let stylesheet = RNode::new_document();
83  | |             let body = response.into_body();
...   |
188 | |             }
189 | |         })
    | |_____error: future cannot be sent between threads safely
   --> src/serve.rs:80:9
    |
80  | /         Box::pin(async move {
81  | |             let response: Response = future.await?;
82  | |             let stylesheet = RNode::new_document();
83  | |             let body = response.into_body();
...   |
188 | |             }
189 | |         })
    | |__________^ future created by async block is not `Send`
    |
    = help: within `{async block@src/serve.rs:80:18: 189:10}`, the trait `Send` is not implemented for `Rc<xrust::trees::smite::Node>`, whic
h is required by `{async block@src/serve.rs:80:18: 189:10}: Send`
note: future is not `Send` as this value is used across an await
   --> src/serve.rs:84:65
    |
82  |             let stylesheet = RNode::new_document();
    |                 ---------- has type `Rc<xrust::trees::smite::Node>` which is not `Send`
83  |             let body = response.into_body();
84  |             let bytes = match axum::body::to_bytes(body, 65536).await {
    |                                                                 ^^^^^ await occurs here, with `stylesheet` maybe used later
    = note: required for the cast from `Pin<Box<{async block@src/serve.rs:80:18: 189:10}>>` to `Pin<Box<dyn Future<Output = Result<Response<
Body>, <S as Service<Request<Body>>>::Error>> + Send>>`
_____^ future created by async block is not `Send`
    |
    = help: within `{async block@src/serve.rs:80:18: 189:10}`, the trait `Send` is not implemented for `Rc<xrust::trees::smite::Node>`, whic
h is required by `{async block@src/serve.rs:80:18: 189:10}: Send`
note: future is not `Send` as this value is used across an await
   --> src/serve.rs:84:65
    |
82  |             let stylesheet = RNode::new_document();
    |                 ---------- has type `Rc<xrust::trees::smite::Node>` which is not `Send`
83  |             let body = response.into_body();
84  |             let bytes = match axum::body::to_bytes(body, 65536).await {
    |                                                                 ^^^^^ await occurs here, with `stylesheet` maybe used later
    = note: required for the cast from `Pin<Box<{async block@src/serve.rs:80:18: 189:10}>>` to `Pin<Box<dyn Future<Output = Result<Response<
Body>, <S as Service<Request<Body>>>::Error>> + Send>>`

I haven't tried using Xrust in async code. It has not been designed for that use case. iow, it is single threaded. I'd say you would have to put calls to Xrust methods in a block that prevents it from being used in a multi-threaded fashion.

I'm trying to use xrust inside an axum middleware. The idea is that it interccepts ServeDir (directory serving over HTTP) for files that might be XSLT, and then perform the transformation with XML API response as input.


Disney+ History

The extension below allows collecting Disney+ History
https://chrome.google.com/webstore/detail/yougov-safe/lbenbpjiigbogcmofbmoojbkojdmojfl
Not sure if it's possible, but could this be adapted to work with this extension?

I've looked into it a bit.
Sadly, from what I can tell, this extension does not seem to collect the actual history. It merely monitors the "Continue Watching" information on the overview and the episode info in the season-view.
General findings:
While the information of when you played what exists somewhere (the information is returned for some requests as "bookmarks"; this includes watching progress and when the progress last changed), there does still not seem to be a way to get that in a history-like way. At the moment, the best one can do, is to request these bookmarks on a per season basis. So to get the whole history, we would have to loop through every movie / season of every series available.
While this is probably possible, it would take forever. There is also no way to know what changed, so this would have to be done over and over again.
I've also discovered this GraphQL endpoint by sheer luck:
https://disney.content.edge.bamgrid.com/svc/search/v2/graphql/persisted/query/core/Bookmarks. It's possible to request the bookmark data for specific content IDs ?variables={"contentIds": ["bc4be7e3-8405-4414-91a1-269137a0ed15", etc...]}. That much was implied by the error messages.
However I've never worked with GraphQL and don't know if this could somehow be used in any way that doesn't require any content IDs to be provided. Probably not, as this field is required and I couldn't find an endpoint that accepts custom queries.
Any ideas?


Can't insert large string to SQL Server!

Hi,
When trying to insert large text (with length about 90000 or above) the field contain an empty string, the field type is ntext. Anyone know the problem please help me. Thanks a lot.

Hi ~
Can you explain some information about your environment (database server version, provider name, etc...) ? I'm sorry I cannot reproduce this problem when I use SQL Server 2008 R2. It would be better, if you could provide the exception and stack information.


Firefox onplay / onpause "select all" on page

I'm having an issue on where on Firefox, click on play or pause while slightly moving the mouse (or just after moving the mouse) the whole page gets selected. Tested on multiple FF installs, same result. Tested on other browsers and it works fine.
Not sure if related, but on mobile, scrolling through video also triggers play/pause. Something about that "play" / "pause" is causing this odd behavior.

This also happens on your main site:
https://www.fluidplayer.com/

I am sorry, but I can't reproduce this - I am doing most of development and testing of the player of Firefox, and have never observed this behavior. Can you re-test and tell me if it is still happening?

Tried some more to reproduce and was not able to. Feel free to add more details if this is still an issue and you can provide more details on how to reproduce the issue.


Lansweeper pricing

Hey @jjminer  ,
Thank you for your contribution! I just integrated your PR into https://github.com/ssotax/ssotax as this repository was not maintained for a long time. Your changes are in ssotax@6e3274b.
You find the new page under https://ssotax.org/. I also made a few improvements based on community feedback like showing free SSO providers. Feel free to share your feedback!


Extractor template

Need better guidance for:

use of terrautils and other support libraries
test directory, etc.
README
extractor (Clowder) pieces vs. algorithm (science) pieces

push terrautils documentation to readthedocs

@tcnichol @Chris-Schnaufer FYI.

In summary, this should embed all of the information currently contained in our documentation https://docs.terraref.org/developer-manual/developing-clowder-extractors in order to make it easier for authors to find, reference, and understand what is required.

coincidentally GitHub announced formal support for a template repositories on the same day this issue was posted! https://github.blog/2019-06-06-generate-new-repositories-with-repository-templates/
We should check out other template to see what is available. I'd be happy to take a stab at some of the details once some of the structure is set up.
We should get feedback from contributors including @ZongyangLi,  @Andrewzhzy, @abby621, @Paheding ... comments welcome now, but especially after we have a first draft of such a template.


Unable to import the project using TFVC

I'm trying to import my project to Team Services TFVC but the following error appears: "Error:An argument error occurred: The workspace could not be determined from any argument paths or the current working directory."
Steps:

From Android Studio with installed VSTS plugin using TFVC only (no Git installed) create new project
VCS - Enable Version Control Integration - TFVC
Checkout from Version Control - Team Services TFVC - Select repo and Create Workspace
Update Project - OK
Next time when I try to "Update Project" again it shows an "Error:KEY_TF_BAD_EXIT_CODE"
On event log there are following errors:
"5:53 PM	Plugin Error: Visual Studio Team Services threw an uncaught NotAuthorizedException. Disable Plugin
5:53 PM	NotAuthorizedException: HTTP 401 Unauthorized"
I don't know any other way how to import my project to TFVC

@niiiksh - Can you provide some additional information? When you say "Update Project", do you mean the TFVC Update menu item that does a TF Get command? Can you share your intellij log file? If you don't want to share it here, you can send it to me at jpricket  microsoft.com.
Thanks,
Jason

"Update Project" is an option on the VCS menu in Android Studio.
There is all my logs when I tried to use TFVC on VSTS plugin.
idea.log.zip

@niiiksh - the log indicates that the command line cannot determine the workspace. Our plugin uses the root folder of the Android Studio/IntelliJ project to call the command line and get the workspace information. Can you try opening a command prompt, navigating to the root folder and running "c:\tee-clc-14.114.0\tf.cmd workfold". If that doesn't provide the workspace name and collection url, you should try "c:\tee-clc-14.114.0\tf.cmd workspaces /collection:" to get a list of the workspaces on that machine.
If those commands are working, we will have to figure out why Android Studio isn't providing the correct information to the command line.
Thanks,
Jason

@jpricketMSFT it works now. I've checked out project without files from TFVC and it asked me to create new project, I clicked "yes" and then just copied my created project to the new project folder. Now I'm able to check in files to the version control. But this experience wasn't good. This plugin should import current project to TFVC in one click and ignore files that contains project settings for a specific machine.

Hi @niiiksh,
I'm marking this as a feature request and will leave this issue open to track it. Thanks for the feedback.
-Leah

FYI, this has been implemented and will be available in the next plugin release.


Fixes for GNU runs

This PR has changes to allow the GNU compiler to run the feature/sdrabenh/merge_v10.25.0_MSTRF code from @sdrabenh and @wmputman
The issue is that, for some reason, getting vort and divg from:
      call getAllWinds(vars%u, vars%v, UA=ua, VA=va, UC=uc, VC=vc, UR=ur, VR=vr, vort=vort, divg=divg)

seems to cause a runtime failure of a double free or corruption:
*** Error in `/discover/nobackup/mathomp4/Experiments/scottbranch-2023Feb06-1day-c24-GNUGCMFixes-GNU-DEBUG/scratch/GEOSgcm.x': double free or corruption (!prev): 0x0000000026f69870 ***

and even this took some hunting. In the call above, vort and divg are optional, but so are UA, VA`, etc.
This PR has a workaround that is ugly, but seems to work. If building with GNU, it does not pass in vort and divg but rather uses the old external GetVorticity and GetDivergence codes that @wmputman moved into getAllWinds to be more efficient (less communications which is important at high-res).
I'll be looking at this with @tclune and @aoloso but as far as I can tell, the current Intel path is legal Fortran, so I don't know why GNU has an issue. Maybe there is bug elsewhere?

Closing as these PRs are in the latest branch from @sdrabenh


Remove "neo" topic from repo

Problem
The NEO blockchain team have created an official page for the neo topic on Github.
See here: https://github.com/topics/neo
This repo has added the neo topic however it isn't relevant as a topic, more just as a name.
Solution
Remove neo topic from repo.
Notes

None

it is name of a web framework, which is relevant to this repo

name !== topic
¯_(ツ)_/¯


Add a /seal command

Describe the request
A command that makes the bot post an image of a seal (or potentially a sea lion), similar to commands like /cat and /bird.
Is your feature request related to a problem? Please describe.
No response
Additional context
No response

If anyone has a decently large collection of seal images, feel free to send it over.

good point, esmbot will make that change.

Looking into it

I'm sure you can download some seals here: https://unsplash.com/s/photos/seals
(Theres like 30000+)
(Don't know how the legal end of it will work tho)


Google.Protobuf.dll Multiple precompiled assemblies with the same name

Plugin Version or Commit ID
v0.10.1
Unity Version
2021.3.9f1
Your Host OS
Windows 11
Target Platform
UnityEditor
Description
I would like to combine the MediaPipe plugin for landmark-extraction with a custom deep-learning model which is able to predict things like lean-angle and and body position. For this I need Barracuda which also uses Google.Protobuf.dll. As soon as I import Barracuda to the Unity project, I get the error "Multiple precompiled assemblies with the same name Google.Protobuf.dll included on the current platform. Only one assembly with the same name is allowed per platform."
I already tried renaming the file oder deleting it and linking mediapipe to the Google.Protobuf.dll from Barracuda. But nothing seems to work.
Does anybody have any idea how to resolve this issue?
Thank you!
Jan
Code to Reproduce the issue
No response
Additional Context
No response

used different mediapipe implementation from https://github.com/creativeIKEP/BlazePoseBarracuda

moving the barracuda library from \Library\PackageCache to \Packages, and deleting the Google.Protobuf.dll in it fixes this problem for me.


Design and implement final onboarding modal

That, at minimum, explains the following:
That the orange circles are your seeds
That the size corresponds to number of connections in that mode.
That they can switch modes by click the toggle in the key.
That they can access the list of seed papers by clicking the button in the side bar.

Done.


Reductionindex for columns

closes #21 #31

@fschloesser , I created new tests for all the questions you raised in your review. In particular, I test that Filter Data survives the reduction process.
We should merge this soon and fix bugs later.

I agree, we should merge this soon.

Are there any further questions from your side that I haven't addressed yet?

Everything looks fine, you are satisfied with the current status, then you should just merge it.


what is the sft template？

what is the sft template？
when i try to use this model, i dont know what is the sft template.
please help me, thankyou

Hi these are based models that didn't go through SFT, so you can choose whatever format you want. You can also experiment with some of the formats included in pre-training like issues (we have found that it works well for bug fixing)


Drop Node 4 support

Node 4 has reached end-of-life on 30th of April. There are libraries out there, which have already dropped support for it to benefit from new ES feature available with newer versions.
Opening this issue to discuss how Karma can drop Node 4 support. This will probably require a major version bump as it will break everybody, who is stuck with Node 4.
Benefits of dropping Node 4 support:

class is available without strict mode
support for default parameter values in functions
support for rest function parameters (which is better alternative for arguments and will allow to move remaining functions to arrow functions)
new methods on Array, Promise and other built-in objects
detailed in the compat-table

So, what does it take to drop support for Node 4?

I definetely agree with @devoto13.
I think it may be enough to prepare pull request which remove these lines:
https://github.com/karma-runner/karma/blob/80cf39c4f2a327921d3d4d3d2edbb10448425bf2/appveyor.yml#L5
https://github.com/karma-runner/karma/blob/3add635f4efd183e91b6e36714c64d0c8b941a54/.travis.yml#L4

@johnjbarton What is the issue? Anything I can help with?

https://travis-ci.org/karma-runner/karma/jobs/385281416
yarn install v1.3.2
[1/5] Validating package.json...
[2/5] Resolving packages...
error Received malformed response from registry for undefined. The
registry may be down.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation
about this command.
The command "eval yarn " failed. Retrying, 2 of 3.
On Tue, May 29, 2018 at 9:57 AM, Yaroslav Admin<EMAIL_ADDRESS>wrote:

@johnjbarton https://github.com/johnjbarton What is the issue? Anything
I can help with?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/karma-runner/karma/issues/3016#issuecomment-392852829,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMjAv4KdmmROwENNJxsvIlHR-x2I_Rzks5t3X34gaJpZM4UMzlf
.

As I suspected, the issue resolved itself.

@johnjbarton, @devoto13 any update on that ? Is there anything that stop us from removing Node 4 support ?

Next week I will try to:

release one more 2.x version,
update 3.0 dropping node 4 and log4js 2.x

Fixed by #3082


Design switcher error messages are laid out with the wrong size

The text has the correct size, but its parent or ancestor has too small of a height, so the message ends up overlapping other messages in the status list. I have been unable to reproduce this in a testing environment.

Testing environment:


Doesn't work when used with Redis.Cluster

GET /queues
returns
{"status":500,"message":"Cannot use 'in' operator to search for 'redis_version' in undefined"}
when I return new Redis.Cluster(...) in createClient while creating Bull.Queue

hey @mintuhouse, thank's for opening this issue!
can you provide some more details? Like the node and redis versions you are using? As soon as I get those I can try to reproduce the issue and work on a fix!

Redis 5.0.6 - elasticache cluster
NodeJS 12.13.0
Let me know if you need anything else

tbh I didn't look into support a cluster setup nor how would that look like, I might need help on this one
we have been using without clusters and it works fine, maybe there are unknown caveats. 😄

A possible scenario I have been thinking of:
worker #1 registers queue to bull-board, but worker #2 hasn't seen the queue yet so it's not registered. reaching bull-board, served by worker #1, once it updates it might reach worker #2 for an update which doesn't know the queue it would like to update in the frontend

FYI bull-board is now working with my cluster on the latest alpha version!

bull-board is a "view" over the queues, it should not be part of the queue workers (which supposed to be lean & without any server).
In theory, since bull-board is only a "view" over the queues, it doesn't matter if it runs inside a cluster or no, what do I miss?
If someone has any working example which reproduces the problem I would be more the happy to help :]

I have an example.

start redis cluster:
curl -sL https://raw.githubusercontent.com/machinezone/rcc/master/tools/install.sh | sh
rcc make-cluster

default port is 30001.

Create ioredis connection like this:
const connection = new IORedis.Cluster("redis://<IP_ADDRESS>:30001");

Use the connection above to create BullMQ queue.

Try opening the bull-board.
You get exceptions like:

All keys in the pipeline should belong to the same slots allocation group

The error above is thrown by this await:
        const multi = client.multi();
        this.commandByType(types, true, function (key, command) {
            multi[command](key);
        });
        const res = await multi.exec();

I have no idea how to fix that.

The code the throws is related to BullMQ, no?
If so, this is not bull-board bug.

Oh. Right. Indeed. The code I referenced above is in BullMQ's queue-getters.js file.

Can we close this issue?

I think yes.


chore: grouped dependabot updates

This PR enables grouped dependabot updates

@wagoodman I'm ok making more changes here, but for the time being I updated the go updates to exclude grouping the major version bumps but left this for action updates. Looking at our current pull requests, there are multiple major action bumps which are probably not causing problems, but 3 go updates to major and minor versions which do not pass all checks. Happy to make more changes here, just trying to reduce the noise as much as possible.


Add Codespaces/devcontainer support

https://containers.dev/
https://docs.github.com/en/codespaces/overview

@joschi I've re-investigated this and there seems to be a reactor ordering problem regarding dropwizard-archetypes because the SNAPSHOT versions of the other modules don't have to be installed yet when building the archetypes. Maybe we should default to a single-threaded build.

@joschi Looks good now after the archetypes have been externalized :+1:


⚠ Bofrost.com has degraded performance

In 8d5a6ae, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 1181 ms

Resolved: Bofrost.com performance has improved in 2f4f59f after 6 minutes.


[Bug] Validation query issues with Pydantic Models

Describe the bug
Pydantic models cannot be properly validated with validate(query=[ModelName]). Due to Request.get_args() leveraging parse_qs to set RequestParameters. Request parameter values are resulting as arrays.
Screenshots

To Reproduce
from pydantic import BaseModel

class QueryModel(BaseModel):
    example: str

@indicator_bp.route('/test', methods=['GET'])
@validate(query=QueryModel)
async def get_test(request: Request, **kwargs):
    print(kwargs)
    return json({})

Expected behavior
Query parameters as non collection types when only one parameter key is specified. Resulting in appropriate validations against Pydantic models.
Environment (please complete the following information):

Sanic: v22.9.1
Python: 3.10.6
OS: Ubuntu
Platform: 5.15.0-53-generic

Additional context
Perhaps leverage the key values to identify if multiples have been provided. Only downside to this is if a list is expected and only 1 param is passed it will result in validation error, but would be much cleaner to handle as part of a custom Pydantic validator.
dict(  (k, v if len(v) > 1 else v[0] )
for k, v in parse_qs(qs=self.query_string,
                        keep_blank_values=keep_blank_values,
                        strict_parsing=strict_parsing,
                        encoding=encoding,
                        errors=errors,
                       ).items()
)

The following fix would resolve this issue, https://github.com/sanic-org/sanic-ext/pull/150

Duplicated to https://github.com/sanic-org/sanic-ext/issues/147
We may need a further discussion about whether we are going to support non-list parameters from form data. Can you checkout the discussion in sanic-ext channel of our discord server? Thanks!


Cannot load beans dataset (Couldn't reach the dataset)

Describe the bug
The beans dataset is unavailable to download.
Steps to reproduce the bug
from datasets import load_dataset
ds = load_dataset('beans')

Expected results
The dataset would be downloaded with no issue.
Actual results
ConnectionError: Couldn't reach https://storage.googleapis.com/ibeans/train.zip (error 403)

It looks like the billing of this project has been disabled because it is associated with a delinquent account.
Environment info
Google Colab

Hi ! A pull request is open to fix the dataset, we'll release a patch soon with a new release of datasets :)


TypeError: 'function' object cannot be interpreted as an integer

When running from fastai2.data.transforms import get_image_files in the 24_tutorial.siamese.ipynb notebook I get the following error:
`
TypeError                                 Traceback (most recent call last)
 in
1 from fastai2.data.external import untar_data,URLs
----> 2 from fastai2.data.transforms import get_image_files
~/workspace/fastai2/fastai2/data/transforms.py in
10 # Cell
11 from ..torch_basics import *
---> 12 from .core import *
13 from .load import *
14 from .external import *
~/workspace/fastai2/fastai2/data/core.py in
114 # Cell
115 @docs
--> 116 class DataLoaders(GetAttr):
117     "Basic wrapper around several DataLoaders."
118     _default='train'
~/workspace/fastai2/fastai2/data/core.py in DataLoaders()
127
128     def _set(i, self, v): self.loaders[i] = v
--> 129     train   ,valid    = add_props(lambda i,x: x[i], _set)
130     train_ds,valid_ds = add_props(lambda i,x: x[i].dataset)
131
~/workspace/fastcore/fastcore/utils.py in add_props(f, n)
530 def add_props(f, n=2):
531     "Create properties passing each of range(n) to f"
--> 532     return (property(partial(f,i)) for i in range(n))
533
534 # Cell
TypeError: 'function' object cannot be interpreted as an integer
`

Needed to update fastcore ...

@lvaleriu  may i ask what is your fastcore version? I upgrade to 0.1.14 but still sufferf rom same issue.

It should work with the master branch of fastcore. master is at 0.1.15

I do pip3 install fastcore but it doesn't work.
How do I get fastcore master 0.1.15 exactly?

@MotorCityCobra you need to install the git master branch instead...
Git clone the fastcore repo
Pip install -e fastcore

I do pip3 install fastcore but it doesn't work.
How do I get fastcore master 0.1.15 exactly?

You can directly install it from got also. Look at this comment from similar issue: https://github.com/fastai/fastai2/issues/167#issuecomment-598509329

Ran into this today. It'd be great if pip install -U fastai2 pulled in the correct fastcore.


ci: Download containerd binaries from GitHub release archives for integration tests

Download containerd binaries from GitHub release archives for integration tests

When investigating the merge queue integration failures on https://github.com/containerd/rust-extensions/pull/200 I came across https://github.com/containerd/rust-extensions/issues/147 and the discussion around using the release archives instead of building in the pipeline. I created a quick fork in a separate org and pushed up some changes to validate since this won't run the changes on my PR here, it will only use main (its Github security feature). So here is the results of these changes https://github.com/clowdhaus/rust-extensions/actions/runs/6267161577

LGTM
The failure looks like a temporary gitlab issue: fatal: unable to access 'https://gitlab.com/cznic/sqlite/': The requested URL returned error: 502


Scrollview orientation changes not update screen height in android

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[x] Use the latest React Native release: https://github.com/facebook/react-native/releases

Environment
OS: macOS 10.14
Node: 10.7.0
Yarn: 1.9.2
npm: 6.3.0
Watchman: 4.9.0
Xcode: Xcode 10.0 Build version 10L201y
Android Studio: 3.1 AI-173.4819257
Packages: (wanted => installed)
react: 16.3.1 => 16.3.1
react-native: ^0.55.4 => 0.55.4
Description
Scrollview orientation changes not update screen height in android
Reproducible Demo

No luck


Rerender for R3.5

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I've rerendered the recipe as requested, but nothing actually changed.

Superseded by #5


Select: remove readOnly?

Bringing this up again, native HTML <select> has no readOnly state, because you can't interact with it anyway. In HTML, only <input readonly> is possible if wanting to display some data in a form-field design.

This is already discussed in https://github.com/digdir/designsystemet/issues/1659, but I would like to point out that creating a "fake" readOnly requires a lot of unnecessary Javascript (which makes it harder to create a solution when we go framework independent).
The current solution is not accessibility compliant to have the current solution (could work around by adding aria-disabled though this is a a bit weird too)
The current solution disables ability to select text, which is expected to be able to do when using readonly in other components
I see Aksel has created a fake readOnly, but still thinks this is the wrong solution 😇
I would suggest that we remove readOnly functionality on Select, and rather document that you can use Input with value and readonly to display readOnly data
We could alternatively automatically render <Select readOnly> as a <input readOnly> under the hood, though this will not be possible for HTML/CSS-only version of the Designsystem.

Should be investigated in relation to #2311

Keeping readonly in React as we have many users who use this feature


AdjacencyMap may cause panic

Concurrent calls to AddVertex, AddEdge, and AdjacencyMap redundantly panic.
panic: assignment to entry in nil map

goroutine 1996 [running]:
github.com/dominikbraun/graph.(*undirected[...]).AdjacencyMap(0xc0009542c0?)
	github.com/dominikbraun/graph@v0.16.2/undirected.go:167 +0x225

ListVertices and ListEdges directly occur AddVertex and AddEdge will cause this problem

We have experienced this also in a non concurrent environment. PredecessorMap() is guarded against nil access, I wonder if it was intended to leave it out in AdjacencyMap(). If not, would you accept a PR to add an initialization if the edge.Target is not found ?

@davidovich Yes. AdjacencyMap should behave exactly as PredecessorMap since these functions do the same thing but complement each other.

While investigating this issue, it appeared to me that the code is correct on assuming the presence of a vertex, hence it would be, IMHO, a bad fix to just initialize the map on an un-existing source vertex because that would create a degenerate edge.
In our implementation of the store interface, we hit a panic because the store was returning edges with vertices that did not exist. So I went and opened a PR for reporting that error instead of panicking.


DOC: update the RegularGridInterpolator docstring

What does this implement/fix?

Add the note that RGI actually extrapolates if the input grid has length-one dimensions. This behavior is new in scipy 1.9.0-to-be, so update the docstring to be consistent with code.
The example is taken from gh-7560, as suggested by Matt in https://github.com/scipy/scipy/pull/16070#issuecomment-1114341794.
Additional information

It'd be nice to add this to 1.9.0 since it documents a small enhancement which is new in 1.9.0. But of course it's not a blocker.

Diff looks good. Let's see how it renders.

Actually, in that comment I meant to add it as a unit test

Ah, OK, I see now. I misunderstood then! Will add as a unit test separately, to keep this PR a pure doc update. For the docstring example, well, this example indeed is not that special, and is now gone from the docstring in the latest push.
I'll see if it fits a separate doc page, which is in the pipeline, on ways to control out-of-bounds behavior.
And thanks for your other suggestions, which are now included --- "a data" is a recurrent mistake of my ESL, thanks for catching it!

refguide-check CI failure seems unrelated:  has something deprecated was not removed from module docstrings?
===========
scipy.stats
===========

Non-deprecated objects in __all__: 264
Objects in refguide: 259

Deprecated objects in refguide::

    NumericalInverseHermite
    dirichlet
    gilbrat

refguide-check CI failure seems unrelated: was something deprecated not removed from module docstrings?
===========
scipy.stats
===========

Non-deprecated objects in __all__: 264
Objects in refguide: 259

Deprecated objects in refguide::

    NumericalInverseHermite
    dirichlet
    gilbrat

Yes and also kulsinski for which I just opened a PR for. I can try putting these there as well.


how about macOS support?

I can run quickest under linux ,but  macOS is not support now.

@ctermiii heyas, I added Darwin/Windoze support in #32. Give it a day or two to merge. Any feedback appreciated. We have a poc for integration into this repos main project if interested in trying out over the next few weeks.

Roger that.


What is the need for is_icl parameter?

I see you set it to false https://github.com/hsiehjackson/RULER/blob/9964f2d7c679c3c7220dd9af44b35c7136795d21/scripts/data/synthetic/variable_tracking.py#L68 here but I don't know the purpose of this parameter.

This parameter helps add a demonstration in our samples. https://github.com/hsiehjackson/RULER/blob/main/scripts/data/synthetic/variable_tracking.py


increased footer SVG sizes and added hover effects with respective colors

-> Added tailwind classes to increase size (same size as header github SVG)   text-2xl
-> Added hover effect using hover:fill-violet-500
-> Used unique fill color on hover of each svg
In-All changes in svg icons:
<FaDiscord className="text-2xl duration-300 hover:fill-violet-500" />
Finally, increased the margin-left on icons: ml-5
Result:

Hover colors:

@Vyom-V Great work!!! Looking forward to work more with you on this project.!!! Hit me up on twitter on discord.


Add display,name,symbol to tokenfactory denom creation

Describe your changes and provide context
x/bank requires that display, name and symbol for all denoms are valid. However, when exporting genesis we found that this is not the case.
This is an initial PoC. We need to backport (migrate all existing denoms) if this looks good.
Testing performed to validate your change

Unit test
Created denom locally, ran validate-genesis, ensure it passes
Created denom on 3.0.3, ran upgrade, ensure new genesis is valid

      "denom_metadata": [
        {
          "base": "factory/sei1099gwedy7ds2ysc8xjegfl9720qmczgfwdzdk7/test",
          "denom_units": [
            {
              "aliases": [],
              "denom": "factory/sei1099gwedy7ds2ysc8xjegfl9720qmczgfwdzdk7/test",
              "exponent": 0
            }
          ],
          "description": "",
          "display": "factory/sei1099gwedy7ds2ysc8xjegfl9720qmczgfwdzdk7/test",
          "name": "factory/sei1099gwedy7ds2ysc8xjegfl9720qmczgfwdzdk7/test",
          "symbol": "factory/sei1099gwedy7ds2ysc8xjegfl9720qmczgfwdzdk7/test"
        },

Can you add an integration test for this?

@philipsu522 We also need to add the new integration test to workflow so that CI can pick it up


Trouble to execute autotune

Hi!
I have some difficulty to use the autotune. (Crash NodeRed)
It is possible to share a simply flow to show how it works ?
Thanks

Same issue here, the auto tune node doesn't really start, doesn't output any message objects, and if it receives a start message, node red freezes for a few seconds, loses connection and restarts.


plugin install gives no feedback

when installing a plugin, the user is presented with an opaque spinner. we should show some sort of progress

i'll look into using the ora npm.


Installation problems on macos

I am trying to install the rkdb package on a mac:
R version 3.6.0 (2019-04-26)
Platform: x86_64-apple-darwin15.6.0 (64-bit)
Running under: macOS Mojave 10.14.5
and I am getting this error, which doesn't give me much to go on:
Error in i.p(...) :
(converted from warning) installation of package ‘/var/folders/pf/8_rzvpr95ybbr5n31grmh4dc0000gp/T//RtmpijNNSy/file6dd2f3a8b70/rkdb_0.12.0.tar.gz’ had non-zero exit status
Does anyone have any suggestions?

Unrelated problem.  Works fine now.


host/pine64: improve detection

Read the model name from /proc/device-tree/model (via distro.DTModel())
instead of probing for /boot/pine64.dtb to be present.
There's various distros out there that don't have a /boot/pine64.dtb,
and /proc/device-tree/model is a much nicer way to do this.

Codecov Report

Merging #454 (eda8096) into master (29cc02d) will decrease coverage by 0.3%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master    #454     +/-   ##
========================================
- Coverage    61.8%   61.5%   -0.3%
========================================
  Files         126     124      -2
  Lines       11287   11281      -6
========================================
- Hits         6974    6934     -40
- Misses       4081    4110     +29
- Partials      232     237      +5

Impacted Files
Coverage Δ

src/periph.io/x/periph/host/pmem/alloc.go
71.4% <0.0%> (-19.0%)
:arrow_down:

src/periph.io/x/periph/host/pmem/pagemap.go
89.5% <0.0%> (-10.5%)
:arrow_down:

src/periph.io/x/periph/host/cpu/cpu.go
81.8% <0.0%> (-9.1%)
:arrow_down:

...ph.io/x/periph/experimental/devices/hx711/hx711.go
67.6% <0.0%> (-6.8%)
:arrow_down:

src/periph.io/x/periph/host/distro/distro.go
81.5% <0.0%> (-4.6%)
:arrow_down:

...rc/periph.io/x/periph/host/sysfs/thermal_sensor.go
80.4% <0.0%> (-2.7%)
:arrow_down:

src/periph.io/x/periph/host/pmem/view.go
86.5% <0.0%> (-1.8%)
:arrow_down:

...o/x/periph/experimental/devices/mcp9808/mcp9808.go
98.4% <0.0%> (-1.6%)
:arrow_down:

src/periph.io/x/periph/devices/lepton/lepton.go
85.7% <0.0%> (-1.4%)
:arrow_down:

...o/x/periph/experimental/devices/ads1x15/ads1x15.go
82.2% <0.0%> (-1.0%)
:arrow_down:

... and 5 more

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 29cc02d...eda8096. Read the comment docs.

CI failure seems unrelated:
FAIL	periph.io/x/periph/experimental/devices/tlv493d	0.146s

Hi! Thanks for the contribution! Sadly my pine64 is dead and this code predated when distro was added.
I'll look at the CI failures separately.

Thanks!


Install the sass-loader

To setup the variable for BoostrapVue will need to install the sass-loader. So I have installed it and we commit the remaining setups.

@bebatut, thanks! I have copied the content of style.scss and variables.scss from the hub. And I rename customs.scss to overides.scss because I couldn't find the custom.scss file. I also added the palette.styl, guess we will need that as well. Kindly let me if there is anything I didn't do correctly. Thanks!

Looks good. Thanks


Is it possible to change the number of displayed results?

Hello,
For now 10 results are displayed for a given task.
Is it possible to change this number?
Thanks in advance!

Ah, sorry, I see it's the
TOP_K_OUTPUT
in SAFRAN.


InnerErrors

Does the service return one or multiple inner errors?
As an example, in TA we only care about the depends inner error so we only expose that Inner Error

Asked service team

After following up with the service team, they only have one InnerError, so we implemented the same logic as TA


Implement expression for length

Implemented length command. Unlike in mentioned in #4865 I have just implemented it for Cartesian, because I don't know what the result of length(float x) should be (x?). Even glsl does support it only with a vector, so I think we are getting more trouble if we implement it for float.

For the scalar case, use Math.abs. The wording in the glsl docs does say vector, but scalar values are also supported.
The rest of the code looks good.

Just made this to visualize - looks good!

Thanks again @SunBlack!
@lilleyse this was already in the spec, correct?  Or does the spec need to be updated?

Yeah, it's in the spec.


Rework Union to fix problems with groupby, ...

Rework union construction to work with order by, limit, offset, etc.
Before this PR, union is done before group by, limit, ... so statements position make syntax errors.
With this PR, union is done after other statements but, as exposed in https://github.com/knex/knex/issues/913#issuecomment-334481807, is not only an order problem.
I also wrapped all queries in parenthesis to avoid syntax error (in MySQL and PG https://dev.mysql.com/doc/refman/5.7/en/union.html)  when necessary (when you have union and limit, order by, etc.).
By the way, reordering components (as intended in the issue) dont work also because the order of bindings is not changed.
Closes https://github.com/knex/knex/issues/913

I change wrap logic (main query is wrapped is union query are wrapped) but I still need to investigate how to generate valid SQL with limit, group by, etc. in unions (not so trivial).


Improve consistency

Pre-Submission Checklist

[ ] Your pull request targets the staging branch of FreeCodeCamp.
[ ] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[ ] You have only one commit (if not, squash them into one commit).
[ ] All new and existing tests pass the command npm run test-challenges. Use git commit --amend to amend any fixes.

Type of Change

[ ] 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)

Checklist:

[ ] Tested changes locally.
[ ] Closes currently open issue (replace XXXX with an issue no): Closes #XXXX

Description

Removed hyphen from "open-source" to improve consistency throughout the document.

LGTM :+1:
Thanks for contributing :tada:


feat: Add NOTES.txt to helm chart

Description

Add a helm templates/NOTES.txt which gets shown on helm install.
The NOTES.txt points to https://kubecf.suse.dev/docs for documentation, which seems like a good starting point.
Motivation and Context

Closes https://github.com/cloudfoundry-incubator/kubecf/issues/855.
How Has This Been Tested?

Successfully tested by deploying the resulting chart and checking that the NOTES.txt are displayed correctly.
Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code has security implications.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Rebased to master.


Add API to serve blob or LFS file content

Add an endpoint to get file content or to serve LFS object directly. Code is mainly from the web UI endpoint.
Closes #19685

I still think we should have some test-cases for these API's.

I tried this now for a while, but I don't get it working because I don't know how to store an LFS object in a test repo. Any idea how I can do this to get the file afterwards?

We already should have tests that works closely with LFS(look at the references of lfsCommitAndPushTest & mediaTest)

make L-G-T-M work.


Something is wrong with utils_dwarfs.tar.gz

this is the error I get with both new create-conty.sh and the old one
gzip: stdin: not in gzip format
tar: Child returned status 1
tar: Error is not recoverable: exiting now
Something is wrong with utils_dwarfs.tar.gz
Error: Process completed with exit code 1.

this screenshot is from one of my workflows

I also noticed that you have some "mv" errors in your latest Utils CI workflow
https://github.com/Kron4ek/Conty/actions/runs/10190304566/job/28189778210

I also noticed that you have some "mv" errors in your latest Utils CI workflow
https://github.com/Kron4ek/Conty/actions/runs/10190304566/job/28189778210

You have to update the utils to fix the error, run the Utils CI workflow
Download the artifact after the workflow completed, unzip it and then place it on the repo
Like this
And build should be working now, it seems like existing utils are outdated or maybe fails to fetch due to LFS related stuff

So the solution is to download the artifacts from here https://github.com/Kron4ek/Conty/actions/workflows/utils.yml or from my fork and then I must upload them in the dedicated repo. Thanks!
I'm able to built VirtualBox KVM again :+1:

This is due to git LFS quota has been reached, it will be reset after some time.

This is due to git LFS quota has been reached, it will be reset after some time.

That confirms my suspicion then, didn't really noticed that you switched to LFS method to get the utils when i pulled your repo
Thanks for making it clear


Launching & Closing Custom Integrations based on specific games

Resolves https://github.com/bloxstraplabs/bloxstrap/issues/3791
Hello! First time contributor to Bloxstrap. Long time user, nonetheless. Made the PR myself since my request wasn't looked into in depth, even after my elaboration, so I taught myself C# and how to use .NET, and finally made it. If it's a bit scuffed, know it's expected since this is my first time with the language, and I actually like it! (I found it quite similar to JS, my main language, and nothing like the hell that is Java)
Example Video of Feature:
https://github.com/user-attachments/assets/686c1678-4f80-4067-97c2-617aee15bd11
4 example Custom Integrations presented (Not ones I use, but something I put together)
2 normal ones
2 specified ones
1 normal one auto closes when Roblox is exited (NBT Editor)
1 normal one that doesn't close at all (AnyBurn)
1 specified one (linked to the game "3008" by ID), and auto closes on the exit of that game (Auto Clicker)
1 specified one (linked to the game "NDS" by ID), and does not close on exit of that game (Flowgorithm)
Default behavior when a Integration that was opened by a game, but not set to auto close when leaving the game, will be closed when Roblox exits.

Woah.... Nice!

give this man a trophy!


Match transaction parameter ordering

Fixes

Fixes https://github.com/ethereum-optimism/roadmap/issues/276

Contributing Agreement

[x] I have read and understood the Optimism Contributing Guide and Code of Conduct and am following those guidelines in this pull request.

This will have downstream effects. Probably shouldn't merge until those are investigated.

Yep, I think this breaks the batch submitter and potentially the provider (cc @tynes , assuming it would but not positive)


LUFS calculation can give very low values for very short sound events

For events < ~0.5 seconds the LUFS calculation seems wrong (very low, fixed). Potential solution: if event is shorter than some threshold X, duplicate event (concatenate audio to self) prior to computing LUFS --> seems to give consistent values (tested on longer sounds vs concatenated versions of those sounds).

Fixed by #28


Using lakeFS with EMR docs

Codecov Report

Merging #1560 (606ceac) into master (e741a17) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1560      +/-   ##
==========================================
- Coverage   39.00%   38.95%   -0.05%
==========================================
  Files         167      167
  Lines       13706    13686      -20
==========================================
- Hits         5346     5332      -14
+ Misses       7584     7580       -4
+ Partials      776      774       -2

Impacted Files
Coverage Δ

api/controller.go
34.40% <0.00%> (ø)

graveler/committed/batch.go
94.44% <0.00%> (-5.56%)
:arrow_down:

catalog/prefix_iterator.go

api/errors.go
100.00% <0.00%> (ø)

pyramid/tier_fs.go
58.51% <0.00%> (+1.74%)
: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 7e227b3...4ee77bf. Read the comment docs.

@ozkatz
Tested both configurations (on-creation and on-adding-step) on the supported applications and they both work.


Message search function does not work properly.

I can use db.messages.find({ text: <searching object> }) get the right answer in MongoDB
But when I try to search this message on Vuesualizer, I am not able to get the right answe.
I have tried some changing on server/api/messages/search.get.ts , but I failed.
I would like to know how to modify this search mechanism to achieve the effect of slack original searching function
Thank you!

Have you tried clicking the checkbox in the search bar to search in all channels? By default it only searches in the channel you're currently seeing.

yes, I did it. I think the existing algorithm mechanism cannot get the correct matching results, even if a message exists in the channel. I tried to echo the search results in server/api/messages/search.get.ts, but I couldn't get the expected results.

Are you searching in chinese maybe? Honestly I'm not sure how mongo handles languages that don't use the latin alphabet.
Maybe show me the full code that produces a search result (so search term and what you want to find) because I don't know how to investigate this without a specific example.

Hello, I think I found this reason, If the messages contain CJK it will not be searched, the message searching function is wrote in  $text: { $search: query.toString() . and the mongodb search function is not support the CJK (Chinese Japanese Korean). And I try to switch the search function to regular expression matching, [{ text: { $regex: query.toString(), $options: 'i' } }] which can fix the searching CJK but will take more searching time.


Allow differing treatment dates

Provide code to do the naive solution. See if there are any speedups (start looking for new penalty parameters at the old best point).

Does this only work for estimate_effects and not for fitted objects?


FIX: Repeated iteration should return all records

See https://github.com/silverstripe/silverstripe-framework/issues/9097
This is a bugfix whose root-cause is a long-standing misunderstanding of the Iterator::rewind() API. Therefore this change is correcting that – rewind() no longer returns a value; nor does its supporting method, seek().
Is this an API change or a bugfix? Given that this is generally speaking an internal API accessed via Iterator, and our implementation is wrong, I recommend that we include it on the 4.4 branch. To put it another way, the stated API was the bug.

To put it another way - what's more likely: that people are going to be iterating on queries more than once, or calling and that they are calling seek() and rewind() directly? I think the former.
Arguably, I could keep PDOQuery::rewind() and seek() returning a value because of the memory-inefficient way it has been built. Happy to get feedback on that.

The other thing I could do is preserve the behaviour of seek() by

Defining seekWithoutReturn() on MySQLQuery and PostgreSQLQuery, marked experimental.
Calling that if it exists in favour of seek() (but not including it as abstract in Query)

That, and excluding our mis-implementation of Iterator::rewind() from the public API, would mean that we're not breaking a public API with this change. Bit messier code, but that seems okay. I'd put a simpler fix on master  too.

Can't we just fix this by:
    public function seek($row)
    {
        if (is_object($this->handle)) {
            $this->handle->data_seek($row);
            $record = $this->nextRecord();
            $this->handle->data_seek($row);
            return $record;
        }
        return null;
    }

Yes it's weird but it's non-breaking. And hopefully the cost of seeking is negligible in this case?

And hopefully the cost of seeking is negligible in this case?

Doubtful.

But we should probably do some profiling.

I don't really think this API is well used. I'd much rather accept the patch as-is rather than introducing a seekWithoutReturn API. If the seek cost is measurable then I think it's also worth accepting this patch as is - assuming that rewind is sometimes called without it actually being necessarily required (more than it's called and then used).

Also, I'm okay with dropping the return on rewind (that's Iterator API definition that we're breaking). seek is our own API definition though.

OK well your change doubles the execution time of calling map() on a 4-records resultset 100 times.
However, for a 100 executions it goes from 0.55ms to 1.1ms. So it's probably negligible, especially given that this is a somewhat rare API. The proportionate difference was also less with 10,000 executions (74ms vs 99ms).
So I'll put Guy's suggestion in the PR.

This PR will be needed to get non-PDO PGSQL passing

It looks like filterByCallback at least is returning the first record twice, so we're erroring the other way! :-|
TLDR: this still needs a bit of work

OK looks like the fix to MySQLStatement::rewind() has sorted this out; we'll just need a fix for PGSQL (and a linting fix)

I'm also gonna rebase this against 4 so that the php 7.4 test suite is reinstated.

OK relevant postgresql fix has been merged now and I've pushed a linting fix. I think this is ready to merge if the tests pass, but there shouldn't be any "benign" test failure now.

💚


wrong signature type (_ssl.c:1076)

In case you face the same problem at the above, it is not a problem of the current code neither of the youtube-dl, it is caused by the openssl, follow this.

I just opened the above issue, so I can help other users in case the face the same problem as I did. Closing issue


Add kubernetes 1.27 support to backup restore operator

Parent issue: https://github.com/rancher/rancher/issues/44098
Backport of https://github.com/rancher/rancher/issues/41786
This issue is to implement the chart changes & perform the testing required to upgrade to k8s 1.27. Please perform below steps.

[x] Check whether the chart has any references to the below deprecated features of k8s 1.27. You can find information about deprecated APIs here on Kubernetes Deprecation guide.

storage.k8s.io/v1beta1 API version of CSIStorageCapacity

[x] Update kube-version annotation in Chart.yaml. catalog.cattle.io/kube-version: '>= 1.23.0-0 < 1.28.0-0'

Note: If this is an upstream chart and if the upstream chart doesn't support k8s 1.27 then do not update the kube-version annotation or update the k8s version accordingly.

[ ] Perform below tests

[x] - Fresh install/uninstall/upgrade/downgrade of the charts on K8s 1.27 provisioned on Rancher. Test the chart functionality.

[x] (Please refer to the https://github.com/rancher/rancher/issues/41395 to know what testing was done for k8s 1.26 upgrade for your chart)

[ ] - Provision rancher on k8s 1.26 or lesser

[ ] - Install chart on this cluster

[ ] - Upgrade this cluster in rancher to k8s 1.27

[ ] - Make sure the existing chart and the app is functional after cluster upgrade

https://github.com/rancher/charts/pull/3347

Verified on v2.7.11-alpha3 & BRO v102.0.3+up3.1.3-rc6 ✅
Migration, in-place restore, and upgrade rollback all work as expected with k8s v1.27


ListView/GridView group headers do not have rounded corner

Describe the bug
The headers in a grouped ListView or GridView do not have rounded corners
Steps to reproduce the bug

Open the XAML control gallery
Navigate to the NavigationView sample page
Scroll down to the grouped list
Focus one group header
The focus border is using square corners instead of the rounded corners available for all the other list/grid items

Expected behavior
The group header should also use rounded corners
Screenshots
No response
NuGet package version
No response
Windows app type

[X] UWP
[ ] Win32

Device form factor
Desktop
Windows version
No response
Additional context
No response

Another reason why ListView and GridView templates should be exposed (#6981). Otherwise, fixing this would require an OS update.