[Bug]: UPS Serial Number is not fully displaying

Describe the bug
The UPS Serial Number is only displaying the first digit via PeaNUT. The Operating System, NUT server and client all resolve the full serial number as evidenced in debug output, but this data is being mangled/ignored by PeaNUT possibly due to alphanumeric values (since it cuts off at the first letter in the serial).
Debug Output
Check USB device information:
~$ lsusb -D /dev/bus/usb/001/003
Device: ID 051d:0002 American Power Conversion Uninterruptible Power Supply
Device Descriptor:
  bLength                18
  bDescriptorType         1
  bcdUSB               2.00
  bDeviceClass            0 [unknown]
  bDeviceSubClass         0 [unknown]
  bDeviceProtocol         0
  bMaxPacketSize0        64
  idVendor           0x051d American Power Conversion
  idProduct          0x0002 Uninterruptible Power Supply
  bcdDevice            0.90
  iManufacturer           1 American Power Conversion
  iProduct                2 Back-UPS XS 1500G FW:866.L9 .D USB FW:L9
  iSerial                 3 4B1707P21318

Check NUT server configuration for this UPS device:
~$ cat /etc/nut/ups.conf
pollinterval = 1
maxretry = 3

[myups]
    driver = usbhid-ups
    port = auto
    desc = "Back-UPS XS 1500G"
    vendorid = 051d
    productid = 0002
    serial = 4B1707P21318

Check NUT client configuration for this UPS device:
~$ upsc myups |grep serial
device.serial: 4B1707P21318
driver.parameter.serial: 4B1707P21318
ups.serial: 4B1707P21318

Run PeaNUT and confirm readiness:
~$ docker run -d --name peanut --network monitoring --restart unless-stopped -p 8888:8080 -e TZ=America/New_York -e NUT_HOST=<IP_ADDRESS> -e NUT_PORT=3493 -e WEB_HOST=<IP_ADDRESS> -e WEB_PORT=8080 brandawg93/peanut~

~$ docker logs peanut
><EMAIL_ADDRESS>start
> HOSTNAME=$WEB_HOST PORT=$WEB_PORT node server.js

  ▲ Next.js 14.2.4
  - Local:        http://localhost:8080
  - Network:      http://<IP_ADDRESS>:8080

 ✓ Starting...
 ✓ Ready in 107ms

Check PeaNUT device details via API will display only the first digit (ie: 4) for all 3 parameters:
~$ curl -X GET http://localhost:8888/api/v1/devices/myups
{"battery.charge":100,"battery.charge.low":"10","battery.charge.warning":50,"battery.date":2001,"battery.mfr.date":"2017/02/18","battery.runtime":15750,"battery.runtime.low":"120","battery.type":"PbAc","battery.voltage":27.2,"battery.voltage.nominal":24,"device.mfr":"American Power Conversion","device.model":"Back-UPS XS 1500G","device.serial":4,"device.type":"ups","driver.debug":"0","driver.flag.allow_killpower":"0","driver.name":"usbhid-ups","driver.parameter.pollfreq":30,"driver.parameter.pollinterval":1,"driver.parameter.port":"auto","driver.parameter.productid":2,"driver.parameter.serial":4,"driver.parameter.synchronous":"auto","driver.parameter.vendorid":51,"driver.state":"updateinfo","driver.version":2.8,"driver.version.data":"APC HID 0.100","driver.version.internal":0.52,"driver.version.usb":"libusb-1.0.27 (API: 0x100010a)","input.sensitivity":"medium","input.transfer.high":"139","input.transfer.low":"88","input.voltage":123,"input.voltage.nominal":120,"ups.beeper.status":"disabled","ups.delay.shutdown":"20","ups.firmware":866,"ups.firmware.aux":"L9","ups.load":"0","ups.mfr":"American Power Conversion","ups.mfr.date":2017,"ups.model":"Back-UPS XS 1500G","ups.productid":2,"ups.realpower.nominal":865,"ups.serial":4,"ups.status":"OL","ups.test.result":"No test initiated","ups.timer.reboot":"0","ups.timer.shutdown":-1,"ups.vendorid":51}

Interestingly, checking the PeaNUT API for single parameters will display the accurate serial number:
~$ curl -X GET http://localhost:8888/api/v1/devices/myups/var/device.serial
"4B1707P21318"

~$ curl -X GET http://localhost:8888/api/v1/devices/myups/var/driver.parameter.serial
"4B1707P21318"

~$ curl -X GET http://localhost:8888/api/v1/devices/myups/var/ups.serial
"4B1707P21318"

Checking PeaNUT via web UI will display only the first digit (ie: 4):

Steps to reproduce

Install NUT and configure Driver
Verify UPS information including Serial Number
Install PeaNUT Docker and confirm functionality
Confirm UPS Serial Number in PeaNUT

Device Type
Ubuntu 24.04.1 LTS and kernel 6.8.0-47-generic on Asus PN62S
UPS Device
APC UPS Back-UPS XS 1500G and usbhid-ups libusb-1.0.27
Last Working Version
No response

Thanks for the report! Could you try my latest version? I may have fixed this already.
https://github.com/Brandawg93/PeaNUT/releases/tag/v3.0.0-beta.0

@Brandawg93 I had trouble finding settings.yml but hacked my around and got the beta container running.
Sorry to report the same exact results as initially documented -- not showing full serial number via all the same methods  except for API single parameters which are successful.
How can I help track down this glitch? Thanks for your help.
wget https://github.com/Brandawg93/PeaNUT/archive/refs/tags/v3.0.0-beta.0.tar.gz
tar xvzf v3.0.0-beta.0.tar.gz
cd PeaNUT-3.0.0-beta.0
docker build -t peanut:custom .
mkdir /data/peanut
echo -e "WEB_HOST:localhost\nWEB_PORT: 3493\nWEB_HOST: <IP_ADDRESS>\nWEB_PORT: 8080\nUSER: admin\nPASSWORD: secret" > /data/peanut/settings.yml
docker run -d --name peanut --network monitoring --restart unless-stopped -p 8080:8080 -v /data/peanut:/config peanut:custom
docker logs peanut

><EMAIL_ADDRESS>start
> HOSTNAME=$WEB_HOST PORT=$WEB_PORT node server.js

  ▲ Next.js 14.2.13
  - Local:        http://localhost:8080
  - Network:      http://<IP_ADDRESS>:8080

 ✓ Starting...
 ✓ Ready in 90ms

Does your UPS require that you manually specify the serial number in the configuration files? I've never seen that before. I tried replicating your setup and mine breaks entirely when adding the serial to the configuration.

@Brandawg93 I just tried removing the serial number from NUT ups.conf and everything still works, so I presume it's not required. That said, the same issue persists - only the first digit shows via the same methods in PeaNUT.

Interesting. That value can be either a string or number so it shouldn’t be a type issue from the peanut side of things. Could you login directly to the nut server and print all variables and see what it gives you? I’m wondering if there is some sort of delimiter in the value e.g. “-“.

@Brandawg93 the parameter type is defined as NUMBER for all instances (perhaps it should be string?):
~$curl -X GET http://localhost:8888/api/v1/devices/myups/var/ups.serial/type
"NUMBER"
~$ curl -X GET http://localhost:8888/api/v1/devices/myups/var/device.serial/type
"NUMBER"

Here is the output for all parameters on NUT:
~$ upsc myups@localhost
Init SSL without certificate database
battery.charge: 100
battery.charge.low: 10
battery.charge.warning: 50
battery.date: 2001/09/25
battery.mfr.date: 2017/02/18
battery.runtime: 15750
battery.runtime.low: 120
battery.type: PbAc
battery.voltage: 27.2
battery.voltage.nominal: 24.0
device.mfr: American Power Conversion
device.model: Back-UPS XS 1500G
device.serial: 4B1707P21318
device.type: ups
driver.debug: 0
driver.flag.allow_killpower: 0
driver.name: usbhid-ups
driver.parameter.pollfreq: 30
driver.parameter.pollinterval: 1
driver.parameter.port: auto
driver.parameter.productid: 0002
driver.parameter.synchronous: auto
driver.parameter.vendorid: 051d
driver.state: quiet
driver.version: 2.8.1
driver.version.data: APC HID 0.100
driver.version.internal: 0.52
driver.version.usb: libusb-1.0.27 (API: 0x100010a)
input.sensitivity: medium
input.transfer.high: 139
input.transfer.low: 88
input.voltage: 122.0
input.voltage.nominal: 120
ups.beeper.status: disabled
ups.delay.shutdown: 20
ups.firmware: 866.L9 .D
ups.firmware.aux: L9
ups.load: 0
ups.mfr: American Power Conversion
ups.mfr.date: 2017/02/18
ups.model: Back-UPS XS 1500G
ups.productid: 0002
ups.realpower.nominal: 865
ups.serial: 4B1707P21318
ups.status: OL
ups.test.result: No test initiated
ups.timer.reboot: 0
ups.timer.shutdown: -1
ups.vendorid: 051d

Ah. That'll do it. I guess I can't trust the driver to always give me accurate information. I've added a check for strings and numbers and made a new test version. Let me know if it works for you!

@Brandawg93 yes sir that works! I look forward to your latest official release.

Awesome! The latest version has just been released. Let me know if you run into any issues!


Merge videoUploading samples together

This PR merges both videoUploading and videoUploadingWebCodecs samples together as there's not much value in having both when only the external texture source changes. Note that I set up a redirect so that existing links to videoUploadingWebCodecs still work properly.
See diff at https://www.diffchecker.com/FUiV227s/

@austinEng Unlike my local environment, https://webgpu.github.io/webgpu-samples/samples/videoUploadingWebCodecs does not redirect to https://webgpu.github.io/webgpu-samples/samples/videoUploading?videoSource=videoFrame. Do you know what is happening?

Could it be because I missed "/webgpu-samples/" in redirection URLs?

@austinEng I've just sent https://github.com/webgpu/webgpu-samples/pull/365/ for review


Pandas is generating a sitecustomize error

From<EMAIL_ADDRESS>on 2015-01-07T10:26:11Z
Spyder Version:  2.3.2
Python Version:  2.7.9
Qt Version    :  4.8.6, PyQt4 (API v2) 4.11.1 on Darwin
IPython >=0.13  :  2.3.1 (OK)
matplotlib >=1.0:  1.4.2 (OK)
pandas >=0.13.1 :  None (OK)
pep8 >=0.6      :  1.5.7 (OK)
psutil >=0.3    :  2.2.0 (OK)
pyflakes >=0.5.0:  0.8.1 (OK)
pygments >=1.6  :  2.0.1 (OK)
pylint >=0.25   :  1.4.0 (OK)
rope >=0.9.2    :  0.10.2 (OK)
sphinx >=0.6.6  :  1.2.3 (OK)
sympy >=0.7.0   :  0.7.6 (OK)
zmq >=2.1.11    :  14.4.1 (OK)
What steps will reproduce the problem?

Normal spyder startup
sitecustomize not executed
runfile functions not available

What is the expected output? What do you see instead?
Running python scripts using the built-in python interpreter.
On startup, the following is shown:
'module' object has no attribute 'packaging'
'import sitecustomize' failed; use -v for traceback
Python 2.7.9 (default, Jan  7 2015, 12:39:27)
[GCC 4.2.1 Compatible Apple LLVM 6.0 (clang-600.0.56)] on darwin
Type "help", "copyright", "credits" or "license" for more information.

Imported NumPy 1.9.1, SciPy 0.14.0, Matplotlib 1.4.2

Type "scientific" for more details.
\>>>

Please provide any additional information below
. I've checked the sitecustomize module, i.e. run inside spyder python terminal:
\>>> from spyderlib.widgets.externalshell import sitecustomize
cannot import name NaT
\>>> cannot import name hashtable
cannot import name hashtable

Doing a google search on these issues revealed the issue to be related to external dependency pandas (pandas==0.15.2).
Completely uninstalling pandas enabled using spyder as before. Installing an older version (pandas==0.13.1; the minimum recommend by my spyder install) also enabled the correct functioning again.
Note that this issue not that crucial to me as I do not really rely on the pandas library anyway.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=2106

From ccordoba12 on 2015-01-09T07:29:21Z
I can't reproduce this error. We do import pandas in our sitecustomize to do a couple of monkey-patches, but that is protected with a try/except structure. Just to be sure, I tried to provoke an error in that part of the code and effectively couldn't :-)
However, I see you have another error before the sitecustomize one:
'module' object has no attribute 'packaging'

which seems to point in another direction. I did a google search and found that that is related to setuptools, so maybe your setuptools is outdated or you have a problem with your installation.
To get the full sitecustomize error, could you run this command in a console?
SPYDER_DEBUG=3 spyder
Summary: Pandas sitecustomize error   (was: pandas sitecustomize issue)

From ccordoba12 on 2015-01-08T10:53:21Z
Thanks for reporting. It probably is change in Pandas between 0.13.1 and 0.15.2. I'll try to fix it for our next release.
Labels: Cat-Console MS-v2.3.3

From ccordoba12 on 2015-01-09T08:02:06Z
Summary: Pandas is generating a sitecustomize error   (was: Pandas sitecustomize error)

From<EMAIL_ADDRESS>on 2015-01-09T07:40:47Z
Outputs: terminal (see terminal.log) and spyder python (spyder.log)
pip output for outdated packages:
piano:~ mhemeryck$ pip list -o
setuptools (Current: 9.1 Latest: 11.3.1)
pip (Current: 6.0.3 Latest: 6.0.6)
pandas (Current: 0.13.1 Latest: 0.15.2)
scipy (Current: 0.14.0 Latest: 0.14.1)

pip and setuptools are indeed outdated, but I left them at this version because otherwise it seemed to conflict with my python (homebrew) install. Sorry if this issue is indeed not related to spyder. The pandas issue does not occur outside of my spyder install though.
Attachment: terminal.log spyder.log

From ccordoba12 on 2015-01-09T08:01:45Z
I only see this possible error in spyder.log
cannot import name hashtable

but doesn't seem serious because no traceback is generated.
I'm going to let this one open in case someone else reports the same problem :-)
Labels: -MS-v2.3.3 MS-v2.3.4

From ccordoba12 on 2015-01-09T09:07:53Z
Great to know that you solved your problem!
Status: Invalid
Labels: -MS-v2.3.4

From<EMAIL_ADDRESS>on 2015-01-09T08:11:05Z
I've just upgraded the outdated packages pip and setuptools and now everything seems to work fine. I've tried the same fix before, but then it didn't work.
Afterwards, I also did a brew untap homebrew/python, i.e. all my system-wide python packages are now managed by pip itself.
I've also updated the outdated pandas package (in short, everything updated to latest version).
Don't know what triggered it, sorry for the inconvenience.


[Test-Proxy] Unable to find a record for the request because of quote encoding

Proxy is not finding a request most likely because of the a URL encoding of the request.
Message:
{"Message":"Unable to find a record for the request PUT https://fakeaccountname.blob.core.windows.net/gocblobstartcopydestempty?restype=container\nUri doesn\u0027t match:\n    request \u003Chttps://fakeaccountname.blob.core.windows.net/gocblobstartcopydestempty?restype=container\u003E\n    record  \u003Chttps://fakeaccount.blob.core.windows.net/gocblobstartcopydestempty?restype=container\u003E\nHeader differences:\nBody differences:\n","Status":"NotFound"}

Proxy Version: 1209124
Steps to reproduce

Check out the branch seankane-msft/azblob-recording
Run go test -v -run TestBlobStartCopyDestEmpty with AZURE_RECORD_MODE="playback"

Note: This only shows up for tests in azblob, tested against aztables and this did not show up

Looks like the host names are different:
"fakeaccountname" vs "fakeaccount"

Good catch @JoshLove-msft , that solved it.


kube-proxy/conntrack-fix container is missing resource requests/limits

How to categorize this issue?

/area auto-scaling networking
/kind bug
What happened:
The conntrack-fix sidecar of the kube-proxy is missing resource requests/limits.
It should be set to allow better scheduling and enable autoscaling (if the Shoot VPA feature is enabled).

cc @DockToFuture @ScheererJ @wyb1 @istvanballok @marwinski

Is this really needed? This container is just doing minimal work and very likely will never be required to auto-scale. 👀

VPA seems to recommend resources and apply them on Pod creation.
However, if VPA is not enabled, there are no resource requests in the Pod spec.
But I don't remember if this caused an actual issue.

Following on the discussion in the Quality focus meeting:

This container itself is minimal and should never need autoscaling.
If VPA is enabled resource requests are attached. Therefore, the pod will be scaled.
Consequently, can we close the issue then?

However, if VPA is not enabled, there are no resource requests in the Pod spec.
But I don't remember if this caused an actual issue.

With this,
/close


Compilation error using GCC with SSM Library.

What platform/OS are you using?
Linux Redhat 7
Which version of the SDK?
3.15.2
What compiler are you using? what version?
gccc 9.1.0
What are your CMake arguments?
-DCPP_STANDARD="17" -DCMAKE_INSTALL_PREFIX:PATH=/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug -DCMAKE_INSTALL_RPATH="/lib /apps/gcc/gcc910/lib /apps/gcc/gcc910/lib64 /apps/openssl/openssl-1.1.1/debug/lib /apps/curl/curl-7.65.3/debug/lib" -DCURL_INCLUDE_DIR=/apps/curl/curl-7.65.3/debug/include -DCURL_LIBRARY=/apps/curl/curl-7.65.3/debug/lib/libcurl.so.4.5.0 -DOPENSSL_ROOT_DIR=/apps/openssl/openssl-1.1.1/debug -DBUILD_ONLY="servicediscovery;ssm;s3" -DCMAKE_BUILD_TYPE=Debug -DTARGET_ARCH=LINUX -DBUILD_SHARED_LIBS=OFF
Can you provide a TRACE level log? (sanitize any sensitive information)
: error: expected identifier before numeric constant
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:30:5: note: in expansion of macro ‘Linux’
30 |     Linux
|     ^~~~~
: error: expected ‘}’ before numeric constant
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:30:5: note: in expansion of macro ‘Linux’
30 |     Linux
|     ^~~~~
In file included from /apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/DocumentDescription.h:26,
from /apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/CreateDocumentResult.h:18,
from /apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/SSMClient.h:30,
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:27:3: note: to match this ‘{’
27 |   {
|   ^
: error: expected unqualified-id before numeric constant
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:30:5: note: in expansion of macro ‘Linux’
30 |     Linux
|     ^~~~~
In file included from /apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/DocumentDescription.h:26,
from /apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/CreateDocumentResult.h:18,
from /apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/SSMClient.h:30,
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:35:13: error: ‘PlatformType’ does not name a type; did you mean ‘PlatformTypeMapper’?
35 | AWS_SSM_API PlatformType GetPlatformTypeForName(const Aws::String& name);
|             ^~~~~~~~~~~~
|             PlatformTypeMapper
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:37:48: error: ‘PlatformType’ was not declared in this scope; did you mean ‘Aws::SSM::Model::PlatformType’?
37 | AWS_SSM_API Aws::String GetNameForPlatformType(PlatformType value);
|                                                ^~~~~~~~~~~~
|                                                Aws::SSM::Model::PlatformType
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:26:14: note: ‘Aws::SSM::Model::PlatformType’ declared here
26 |   enum class PlatformType
|              ^~~~~~~~~~~~
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:41:1: error: expected declaration before ‘}’ token
41 | } // namespace Aws
| ^
: error: expected identifier before numeric constant
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:30:5: note: in expansion of macro ‘Linux’
30 |     Linux
|     ^~~~~
: error: expected ‘}’ before numeric constant
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:30:5: note: in expansion of macro ‘Linux’
30 |     Linux
|     ^~~~~
In file included from /apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/DocumentDescription.h:26,
from /apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/CreateDocumentResult.h:18,
from /apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/SSMClient.h:30,
from utils/AwsParamStore.h:13,
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:27:3: note: to match this ‘{’
27 |   {
|   ^
: error: expected unqualified-id before numeric constant
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:30:5: note: in expansion of macro ‘Linux’
30 |     Linux
|     ^~~~~
In file included from /apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/DocumentDescription.h:26,
from /apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/CreateDocumentResult.h:18,
from /apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/SSMClient.h:30,
from utils/AwsParamStore.h:13,
from FileParser.h:20,
from FileParser.cpp:5:
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:35:13: error: ‘PlatformType’ does not name a type; did you mean ‘PlatformTypeMapper’?
35 | AWS_SSM_API PlatformType GetPlatformTypeForName(const Aws::String& name);
|             ^~~~~~~~~~~~
|             PlatformTypeMapper
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:37:48: error: ‘PlatformType’ was not declared in this scope; did you mean ‘Aws::SSM::Model::PlatformType’?
37 | AWS_SSM_API Aws::String GetNameForPlatformType(PlatformType value);
|                                                ^~~~~~~~~~~~
|                                                Aws::SSM::Model::PlatformType
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:26:14: note: ‘Aws::SSM::Model::PlatformType’ declared here
26 |   enum class PlatformType
|              ^~~~~~~~~~~~
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:41:1: error: expected declaration before ‘}’ token
41 | } // namespace Aws
| ^
: error: expected identifier before numeric constant
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:30:5: note: in expansion of macro ‘Linux’
30 |     Linux
|     ^~~~~
: error: expected ‘}’ before numeric constant
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:30:5: note: in expansion of macro ‘Linux’
30 |     Linux
|     ^~~~~
In file included from /apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/DocumentDescription.h:26,
from /apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/CreateDocumentResult.h:18,
from /apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/SSMClient.h:30,
from utils/AwsParamStore.h:13,
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:27:3: note: to match this ‘{’
27 |   {
|   ^
: error: expected unqualified-id before numeric constant
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:30:5: note: in expansion of macro ‘Linux’
30 |     Linux
|     ^~~~~
In file included from /apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/DocumentDescription.h:26,
from /apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/CreateDocumentResult.h:18,
from /apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/SSMClient.h:30,
from utils/AwsParamStore.h:13,
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:35:13: error: ‘PlatformType’ does not name a type; did you mean ‘PlatformTypeMapper’?
35 | AWS_SSM_API PlatformType GetPlatformTypeForName(const Aws::String& name);
|             ^~~~~~~~~~~~
|             PlatformTypeMapper
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:37:48: error: ‘PlatformType’ was not declared in this scope; did you mean ‘Aws::SSM::Model::PlatformType’?
37 | AWS_SSM_API Aws::String GetNameForPlatformType(PlatformType value);
|                                                ^~~~~~~~~~~~
|                                                Aws::SSM::Model::PlatformType
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:26:14: note: ‘Aws::SSM::Model::PlatformType’ declared here
26 |   enum class PlatformType
|              ^~~~~~~~~~~~
/apps/aws/aws-sdk-cpp-1.7.149/cpp17/debug/include/aws/ssm/model/PlatformType.h:41:1: error: expected declaration before ‘}’ token
41 | } // namespace Aws
| ^
When going into the Platform.h file, I was able to see that Linux is written as an enum, but this is also a default Macro that is able to be used within a Linux environment. So when there is a change made to use Linux_ instead, I am able to see that it compiles without the above errors. The fix should be able to be done fairly quickly. The name of the Linux enumeration value just needs to change along with the changes in the corresponding locations.

Hi @manning0218, thanks for pointing this out to us, I'll take a look into it.
Could you create a Pull request with these changes?

Yeah I will have to add them from my personal computer, but will do that when I finish work today.

I apologize for taking long to do this, I have been tied up at work, but will work to push my changes up this weekend, I just need to test them out and will make a pull request as long as everything is still passing


travis: Add build dimension for Openssl master

The intention of this build dimension is to catch build regressions
before they are "officially" released. I expect this build to be
unstable, so it's added to the "allow_failures" section.

Looks like the openssl master build is failing the sigalg integration tests. Might be due to s_client changes.

Strange looks like they removed 'RSA+SHA224' sigalg from openssl master, but it is available in 1.1.0,1.0.2 releases.

Opened https://github.com/openssl/openssl/issues/3633

This change looks good to me. Do we have a backup plan on what to do if OpenSSL declines to re-enable RSA+SHA224?

Worst comes to worst can disable the sigalg test(s) when building against libcrypto 1.1.1-dev. Definitely need to keep the tests with 1.0.2/1.1.0.
This PR needs a couple of other fixes related to DSA-* sigaglgs. Those were removed(for good reason) from Openssl 1.1.1 and we were using them add some "grease" to our tests.

Fixed in Openssl master by https://github.com/openssl/openssl/pull/3639 . This change should be good after removing the sigalgs that were legitimately removed.

After rebasing on master, the 1.1.x build dimension is green: https://travis-ci.org/awslabs/s2n/jobs/248620700

Rebasing onto latest master, should be good to merge if it passes.


Add ignore files language features

Files like .gitignore, .eslintignore etc have ignore as language id.
There is seem no extension to support this language, probably its worth creating standalone extension for this.
Whats worth to support:

go to definition (e.g. clicking on some row to reveal file, something similar is done in https://github.com/zardoy/vscode-experiments/blob/b22a5945b7857ff99d7a2855a26d44c3217a46cf/src/features/jsonGoToFile.ts#LL27C16-L27C16)
completions (I did similar in https://github.com/zardoy/npm-the-fastest/blob/dbbdc3a2c84b3c13c0870818a251257a1baa09d0/src/packageJsonComplete.ts#L120)

ideally (not sure wether its worth to support):

(command) show files in quickpick matching some glob e.g. src/**/*.vue

If you're not interested in this I might implement it later.

the closest extension seems to be Breadnaught.vscode-ignore


BgenVar.minor_allele_dosage mem leak

Hello,
I stumbled across this package when looking for a lightweight bgen parser and it definitely works nicely. However, when repeatedly calling 'BgenVar.minor_allele_dosage,' I end up with a memory leak. It looks like maybe the array constructed to return the dosage is leaving the scope of the garbage collector, but I am not totally sure. It's been awhile since I have written anything in cython (I am using version 0.29.15 with python 3.7 in case this helps). Happy to provide more details if it helps. Overall, great package and definitely quite fast. Thanks for making it public!
Dave Blair

Oooh, good catch! Yep, I wasn't freeing up some memory, which affected repeated calls. I've fixed this in c132193b54cf3caf42bb48efce8cbf089a307e9c, and pushed a new version (1.1.5) to pypi.
Thanks!

Great! Thanks for the quick fix! I’ll let you know if there are any other issues, but so far, seems to be working great!

On May 21, 2020, at 11:30 AM, Jeremy McRae<EMAIL_ADDRESS>wrote:
Oooh, good catch! Yep, I wasn't freeing up some memory, which affected repeated calls. I've fixed this in c132193 https://github.com/jeremymcrae/bgen/commit/c132193b54cf3caf42bb48efce8cbf089a307e9c, and pushed a new version (1.1.5) to pypi.
Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/jeremymcrae/bgen/issues/1#issuecomment-632269959, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB53J4POPOQQYPTYMPGGEMTRSVXM3ANCNFSM4NG7S25A.


Add CNI plugin configuration

find out what should be done
make a TO DO plan

It would be better to use third-party ansible role to install cni. It could be githubixx.cni


Make the ComputeSystemImage looks proportional

Summary of the pull request
Make the ComputeSystemImage look proportional.
References and relevant issues
Before the fix:

After the fix:

Detailed description of the pull request / Additional comments
The fix allows to maintain the image proportions in case it is not square.
Validation steps performed
PR checklist

[ ] Closes #xxx
[ ] Tests added/passed
[ ] Documentation updated

Hoping we can get this fix in #2660  before this one goes in @SohamDas2021


Use FQDN instead of hostname

Hi,
i don't know if you had this already in mind as I saw your TODO in the code. However it would be nice if it would be possible to use socket.getfqdn() instead of the hostname.

Nice catch. Fixed.

I also added another change, using gen-token --fqdn SOMETHING.ELSE, will override the default getfqdn().


display of "we use cookies" in non-browser environments

display of "we use cookies" in non-browser environments
This may soon be noticed in downstream projects that use pxt as the fundament for encapsulating pxt in apps that do not require internet and the laws of the display for cookies used is not relevant.
The importance of projects that do not require internet to program the micro:bit should not be underestimated.

It may be something that can only be fixed in pxt.

An example of a non-browser environment

I am closing this for now. The pxt-microbit-electron repo is just a prototype and there is no immediate release plan.

ok

@abchatra
Please consider reopening this issue
The reference to the ptx-microbit-electron was because it was a "near" microsoft project, however, other projects can and will be affected by the way Microsoft treats the "pxt community"
/ cc @pelikhan

https://github.com/Microsoft/pxt/issues/433

I concur on all @OwenBrotherwood said.
We are a lot of people around the world organizing non profit workshops for kids in places without Internet.
I guess the comment on how community will be treated has a strong meaning in a place like Github.
OpenSource project does not mean only push the source and forget about it.
Please, be open on at least discussion: I thought Pxt project was something to prove me Microsoft has made a true turn on Open Source.

I suggest a title change and an enhancement label: the issue is then catagorised allowing MS to concentrate on the pxt to ensure the base code is stable and can fullfill mircrobit.org goals in the near future.

@titimoby if the issue is reopened, then I would like the comments after the close to be removed to ensure a clean sheet for this momentary lapse which is undoubtedly a product of delivery of the pxt so that it then becomes non-beta @ microbit.org

opening comment rephrased to be acceptable

@abchatra /cc  @pelikhan
All perceivable negative reactions to the closing of this issue are removed from this issue and slack.
Please consider the reopening of this issue with new title and as an enhancement label

@abchatra /cc @pelikhan
Please consider reopening this issue with label "enhancement"
Note:

that the title and text have been changed
All percievable and otherwise, negative reactions to the closing of this issue are removed from this issue and from https://microbit-community.slack.com/messages/pxt/ by myself and Thierry.

There is still an issue open at microbit.org support about the concerns we have for support for the micro:bi offline.
This comment will be removed after reopening of the issue so as not to cloud the issue.

@owen_brotherwood. Pxt does have a strong desire to address the offline scenario for microbit. Most important aspect of offline scenario is ability to patch security issues and not leave users vulnerable. Electron app is the solution for offline scenario or not depends on how our prototype goes. I will reopen this issue once electron app moves beyond prototype. Keep your feedback coming. Thank you.

Opening the issue after further discussion with @pelikhan

Use of --no-browser, to indicate no need for cookie warning.
NB: do not use electron as this is an implementation og not a type
    pxtMicrobit.pxtCore.mainCli(targetDir, ["serve", "--no-browser", "--electron"]);

Until we have an installer with a proper dialg to accept the terms of use, the notification has to stay.

At @pelikhan 's option, this issue can probably be closed.
As the present encapsulation apps require internet to access documentation, it can be still termed as requiring cookie accept.
Note:
I will be trying to encapsulate the documentation, such that documentation is available locally as I believe the internet documentation is generated from a repro ...(?)

fixed?


ZA2-Jora Catalina_Alexandra

Hello, looked at your reducer and it looks good, you may want to removed node_modules from git to make your work easier to review.
You can start with this solution on stackoverflow https://stackoverflow.com/a/50675909/4642046.

Hi Craques! I deleted node modules. Afterward, I attempted to remove deleted files but I created this chaos.

for your case you would need to run:

git rm -r --cached ./server/node_modules

then commit  and push those changes, this should fix your issue

Hi! When I run: git rm -r --cached ./server/node_modules, I get: fatal: pathspec './server/node_modules' did not match any files


Are bill_actions unique within a bill, and should they be?

I'm working an OpenStates issue that leads me here.
In importing a bill via pupa, the exception pupa.exceptions.DataImportError: duplicate key value violates unique constraint "opencivicdata_voteevent_bill_action_id_key" is being evoked.  I infer that this is caused by the same bill action being on two different votes for the same bill, probably enforced by DDL generated by this line.  If the exception is really something else entirely, read no further, sorry to waste your time.
This is happening on a bill with a loop in its process; it was re-referred to committee after going to the floor.  As a result, some bill actions were duplicated.
I can see two ways to go to fix this, but don't have the use-case experience to recommend one over the other.
One fix is to leave the model as is, and require data generators like OpenStates to synthesize unique bill actions.  For example, in the example linked above, the two occurrences of 2nd Reading Passed might become [2nd Reading Passed, 2nd Reading Passed (2)].
The other fix would be to relax the uniqueness requirement here.

this is being addressed in a PR for  opencivicdata/pupa#307


commit_version_bump action errors with "Could not find option for key :force"

I'm not passing force as an option to the commit_version_bump action, so not sure what to do about this error message.
Fastfile:
# This block is called, only if the executed lane was successful
after_all do |lane|
  commit_version_bump
  add_git_tag
  push_to_git_remote
  slack(
    message: "Successfully built lane: #{lane}",
    success: true,
    channel: 'workshop'
  )
end

Error log:
INFO [2015-04-22 23:20:50.21]: ---------------------------------
INFO [2015-04-22 23:20:50.21]: --- Step: commit_version_bump ---
INFO [2015-04-22 23:20:50.21]: ---------------------------------
INFO [2015-04-22 23:20:50.50]: Variable Dump: FILTERED
FATAL [2015-04-22 23:20:50.50]: Could not find option for key :force. Available keys: message, xcodeproj
FATAL [2015-04-22 23:20:51.25]: fastlane finished with errors
/Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/fastlane_core-0.6.2/lib/fastlane_core/configuration.rb:69:in `fetch': Could not find option for key :force. Available keys: message, xcodeproj (RuntimeError)
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/fastlane-0.12.4/lib/fastlane/actions/commit_version_bump.rb:60:in `run'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/fastlane-0.12.4/lib/fastlane/fast_file.rb:112:in `block (2 levels) in method_missing'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/fastlane-0.12.4/lib/fastlane/actions/actions_helper.rb:35:in `execute_action'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/fastlane-0.12.4/lib/fastlane/fast_file.rb:94:in `block in method_missing'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/fastlane-0.12.4/lib/fastlane/fast_file.rb:93:in `chdir'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/fastlane-0.12.4/lib/fastlane/fast_file.rb:93:in `method_missing'
	from (eval):65:in `block (2 levels) in parse'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/fastlane-0.12.4/lib/fastlane/runner.rb:40:in `call'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/fastlane-0.12.4/lib/fastlane/runner.rb:40:in `block in execute'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/fastlane-0.12.4/lib/fastlane/runner.rb:24:in `chdir'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/fastlane-0.12.4/lib/fastlane/runner.rb:24:in `execute'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/fastlane-0.12.4/lib/fastlane/lane_manager.rb:33:in `cruise_lane'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/fastlane-0.12.4/bin/fastlane:39:in `block (2 levels) in run'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/commander-4.3.3/lib/commander/command.rb:178:in `call'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/commander-4.3.3/lib/commander/command.rb:178:in `call'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/commander-4.3.3/lib/commander/command.rb:153:in `run'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/commander-4.3.3/lib/commander/runner.rb:428:in `run_active_command'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/commander-4.3.3/lib/commander/runner.rb:68:in `run!'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/commander-4.3.3/lib/commander/delegates.rb:15:in `run!'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/fastlane-0.12.4/bin/fastlane:118:in `run'
	from /Users/ryansonnek/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/fastlane-0.12.4/bin/fastlane:124:in `<top (required)>'
	from bin/fastlane:16:in `load'
	from bin/fastlane:16:in `<main>'

Apparently during the transition to the new config format this parameter was forgotten.
However, in your case, the problem is actually another one: since you are running commit_version_bump in the after_all block, your git status will most likely be dirty. Try moving it into before_all or, alternatively, running the clean_build_artifacts action before running commit_version_bump should work as well.

you're correct.  my git repo was dirty and adding clean_build_artifacts beforehand did fix the issue.

Thanks again @milch for providing a pull request resolving this issue. In fact, I really forgot to migrate this option and it seems like there was no test for this available parameter.


bump up to jankins-master 1.5

new jenkins-master is coming with set of improvements into the pipelines and new plugins/versions/slaves

done. setup from scratch using zebrunner branch: develop should have 1.5 jenkins-master and 1.5 pipeline version inside

Verified on zebrunner ce dev. branch. Smoke test is done.


Add flow limiting and support all types of storage extensions

Is your feature request related to a problem? Please describe.

Currently there is limited capacity support for flow limiting and we need to perfect it.

Describe the solution you'd like

Increased base flow limiting capability

Describe alternatives you've considered

refer to https://docs.gofiber.io/api/middleware/limiter/

Additional context

None

I'd like to work on this issue


Assembly (DLL) versioning

According to C# an assembly version number is Major.Minor.Build.Revision. Follow semvar rules for Major and Minor.
The build number is a monotonically increasing integer and is maintained by the CI system.
Revision will typically be "0".  I don't see us producing patches.
When the assembly is built on a local dev machine it will be 0.42.*.  Developer built assemblies should never leave the local machine!
A release build is signified with a build number of 0.

CI changed to follow assembly version number rules and create the DLL artifacts.

CI changed to follow assembly version number rules and create the DLL artifacts.

CI changed to follow assembly version number rules and create the DLL artifacts.


Apply CSS class to inline-editable grid cells

New .xh-cell--editable applied, but with no default styling provided by Hoist.
Changed previous .xh-invalid-cell to .xh-cell--invalid for consistency.
Fixes #2553

Hoist P/R Checklist
Pull request authors: Review and check off the below. Items that do not apply can also be
checked off to indicate they have been considered. If unclear if a step is relevant, please leave
unchecked and note in comments.

[x] Caught up with develop branch as of last change.
[x] Added CHANGELOG entry, or determined not required.
[x] Reviewed for breaking changes, added breaking-change label + CHANGELOG if so.
[x] Updated doc comments / prop-types, or determined not required.
[x] Reviewed and tested on Mobile, or determined not required.
[x] Created Toolbox branch / PR, or determined not required.

If your change is still a WIP, please use the "Create draft pull request" option in the split
button below to indicate it is not ready yet for a final review.

Pull request reviewers: when merging this P/R, please consider using a squash commit to
collapse multiple intermediate commits into a single commit representing the overall feature
change. This helps keep the commit log clean and easy to scan across releases. PRs containing a
single commit should be rebased when possible.

Just pushed Toolbox update https://github.com/xh/toolbox/commit/744d1472eb628e0fe6551cf373ec6a0ac0320bd1 to develop to demo the use of an app-specific style to highlight editable cells - assuming we'll merge this PR or some variation on it shortly.

Reviewed with Lee


belongs_to / one

should have the ability to use belongs_to/one since we have many and many through working.

I feel like I'm doing something wrong, or else perhaps the feature is only partially implemented?
when executing a search like so:
SearchTest.findOne({id:1},{include:{'status':{only:['display_name'],where:{'id':'status_id'}}}},function(err,res){ console.log(res);});
I get an object that has a 'status' on it, but the status is an array of all the statuses from my status table.  The relevant portion of the SearchTest schema is as follows:
SearchTest.one = [{'status':TestStatus, joinOn:'search_tests.status_id'}]
I didn't see any examples of having a 0-1 relationship in the tests, so I was using the many syntax as a guide.  Any thoughts?


Update dependency immutable to v4.0.0-rc.12

This PR contains the following updates:

Package
Type
Update
Change

immutable
devDependencies
patch
4.0.0-rc.9 -> 4.0.0-rc.12

Release Notes

facebook/immutable-js
v4.0.0-rc.12
Compare Source
Fixes:

Update to support Flow v0.85 and makes Record strict typing optional to ease migration from v3 or earlier v4 release candidates (#1636)

v4.0.0-rc.11
Compare Source
Potentially Breaking:

Improve hash speed and avoid collision for common values (#1629)

Causes some hash values to change, which could impact the order of iteration of values in some Maps (which are already advertised as unordered, but highlighting just to be safe)

[TypeScript] Remove Iterable<T> as tuple from Map constructor types (#1626)

Typescript allowed constructing a Map with a list of List instances, assuming each was a key, value pair. While this runtime behavior still works, this type led to more issues than it solved so it has been removed. (Note, this may break previous v4 rcs, but is not a change against v3)

Fixes:

Give Records a displayName (#1625)
Set.map produces valid underlying map (#1606)
Support isPlainObj with constructor key (#1627)

Docs:

Fix missing sub-types in API docs (#1619)
Add docsearch (#1610)
Add styles to doc search (#1628)

v4.0.0-rc.10
Compare Source
It's been a long time since the last release candidate, but quite a bit of work has happened since the last once. One step closer to a final release!
Breaking:

Remove IteratorSequence. Do not attempt to detect iterators in Seq(). (#1589)

Iterables can still be provided to Seq(), and most Iterators are also
Iterables, so this change should not affect the vast majority of uses.
For more information, see PR #1589

Node buffers no longer considered value-equal

This was actually broken as of v4.0.0-rc.1 (2dcf3ef)
but was highlighted as a breaking change by (#1437)

New:

Top level predicate functions (#1600)

New functions are exported from the immutable module:
isSeq(), isList(), isMap(), isOrderedMap(), isStack(), isSet(), isOrderedSet(), and isRecord().

Support Typescript 3 (#1593)
Support latest Flow (#1531)
Add RecordOf<TProps> type alias for TypeScript, matching Flow (#1578)
Improved Flow support for Record subclasses (still not advised) (#1414)
Improve performance of toJS (#1581)

Cursory test is >10% faster than both v3.8.2 and v4.0.0-rc.7,
and corrects the regression since v4.0.0-rc.9.

Added optional notSetValue in first() and last() (#1556)
Enable flow strict (#1580)
Make isArrayLike check more precise to avoid false positives (#1520)
map() for List, Map, and Set returns itself for no-ops (#1455) (5726bd1)
Hash functions as objects, allowing functions as values in collections (#1485)

Fix:

groupBy no longer returns a mutable Map instance (#1602)
Fix issue where refs can recursively collide, corrupting .size (#1598)
Throw error in mergeWith() method if missing the required merger function (#1543)
Update isPlainObj() to workaround Safari bug and allow cross-realm values (#1557)
The mergeDeepWith merger is untypable in TS/Flow. (#1532)
Fix missing "& T" to some methods in RecordInstance (#1464)
Make notSetValue optional for typed Records (#1461) (a1029bb)
Export type of RecordInstance (#1434)
Fix Record size check in merge() (#1521)
Fix Map#concat being not defined (#1402)

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Update successful.


[feature request]copy lstm parameters

I'm using tensorflow to implement DQN network and use LSTM at the bottom layer.
But in DQN, a target network is used(coping the main network and keep it for a while, and at the mean time update the original network), when use the tf.Variable as parameter A, I can use tf.Variable.assign to copy A to A', but when I use LSTM, it seems like there is no way to do this copy operation.
Or maybe there is a way to copy the whole graph include its variable weights?

@ebrevdo Could you take a look at this please?

You can access the underlying variables using tf.get_ variable() or, alternatively, by looking through tf.trainable_variables().  E.g.
print [v.name for v in tf.trainable_variables))]

@ebrevdo It can serve my needs, though I need to see what variables have been created by LSTMCell or other interface and their names. Thank you very much!
cc to @jmchen-g

closed

closed #3256

If i have those ema variables, how can i load them into another lstm cell?


Always pass namespaced key to the subscriber of instrumentation API

To be consistent with other events (e.g. read, write), namespaced keys should be passed to the subscribers of the following events:

fetch_hit
generate
read_multi
fetch_multi

@kaorimatz Any chance you could rebase this?

Hi @petergoldstein, I've just rebased on master. Could you please take a look?

Thanks!

Thanks for merging!


Bundler could not find compatible versions for gem "websocket-driver"

Hi,
I 'm new in rails and I want's to install lxd-common to my project. After add gem 'lxd-common' to my Gemfile and run bundle, I get this error:
Bundler could not find compatible versions for gem "websocket-driver":
In snapshot (Gemfile.lock):
websocket-driver (= 0.6.5)
In Gemfile:
rails (~> 5.1.5) was resolved to 5.1.5, which depends on
actioncable (= 5.1.5) was resolved to 5.1.5, which depends on
websocket-driver (~> 0.6.1)
lxd-common was resolved to 0.9.8, which depends on
  nio4r-websocket (~> 0.6) was resolved to 0.6.2, which depends on
    websocket-driver (~> 0.7)

I installed all dependencies with "gem install ...." but nothing changed.
Ruby version: 2.5.0p0
Rails version: 5.1.5

Good timing!!  Documenting this gem has recently bubbled up very high on my todo list...  I'll try to get some starter documentation up this evening to make your life a bit easier.
In regards to your issue, it appears that either rails, or one of your other dependencies has locked the websocket-driver gem to version 0.6.5 whereas my nio4r-websocket gem specifies ~> 0.7.  I did that for no good reason other than it was the current version at the time that I wrote the gem.
And so I'll attempt to relax that restriction to allow 0.6.5, run it through some tests, and let you know 😃

ok try it again.
Make sure that nio4r-websocket resolves to version 0.6.3 and that should allow websocket-driver 0.6.5 to install
And I'll get started on those docs...  Let me know if you need any help in the meantime ;)

I got about halfway done with the docs...  I'll wrap them up tomorrow


Update s3, sdk-core, sqs to 2.17.78

Updates

software.amazon.awssdk:s3
software.amazon.awssdk:sdk-core
software.amazon.awssdk:sqs

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

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

labels: library-update, early-semver-patch, semver-spec-patch

Superseded by #352.


Issue with k8s.io/docs/reference/command-line-tools-reference/kube-proxy/

I don't know why "masqueradeAll" is usless in kube-proxy, and when clustercidr was removed from configmap kube-proxy, all of these new pods can not be started with error
networkPlugin cni failed to set up pod "nginx-5f6ddfdbd-g9w4p_default" network: Get https://[<IP_ADDRESS>]:443/api/v1/namespaces/default: dial tcp <IP_ADDRESS>:443: i/o timeout
the following is the configmap for kube-proxy.
apiVersion: v1
data:
config.conf: |-
apiVersion: kubeproxy.config.k8s.io/v1alpha1
bindAddress: <IP_ADDRESS>
clientConnection:
acceptContentTypes: ""
burst: 10
contentType: application/vnd.kubernetes.protobuf
kubeconfig: /var/lib/kube-proxy/kubeconfig.conf
qps: 5
configSyncPeriod: 15m0s
conntrack:
maxPerCore: 32768
min: 131072
tcpCloseWaitTimeout: 1h0m0s
tcpEstablishedTimeout: 24h0m0s
enableProfiling: true
healthzBindAddress: <IP_ADDRESS>:10256
hostnameOverride: kaas-m-7ec67
iptables:
masqueradeAll: false
masqueradeBit: 14
minSyncPeriod: 0s
syncPeriod: 30s
ipvs:
excludeCIDRs: []
minSyncPeriod: 0s
scheduler: rr
strictARP: true
syncPeriod: 30s
kind: KubeProxyConfiguration
metricsBindAddress: <IP_ADDRESS>:10249
mode: iptables
nodePortAddresses: []
oomScoreAdj: -999
portRange: ""
udpIdleTimeout: 250ms
winkernel:
enableDSR: false
networkName: ""
sourceVip: ""
kubeconfig.conf: |-
apiVersion: v1
kind: Config
clusters:
- cluster:
certificate-authority: /var/run/secrets/kubernetes.io/serviceaccount/ca.crt
server: https://<IP_ADDRESS>:6443
name: default
contexts:
- context:
cluster: default
namespace: default
user: default
name: default
current-context: default
users:
- name: default
user:
tokenFile: /var/run/secrets/kubernetes.io/serviceaccount/token
kind: ConfigMap
metadata:
creationTimestamp: "2020-11-30T06:31:08Z"
labels:
app: kube-proxy
name: kube-proxy
namespace: kube-system
resourceVersion: "522688"
selfLink: /api/v1/namespaces/kube-system/configmaps/kube-proxy
uid: 959f20a4-7d19-48ad-a52a-f9426bf92110
K8S version:1.19

Hi @zhwhao
I'm not sure if you're reporting a defect with https://k8s.io/docs/reference/command-line-tools-reference/kube-proxy/ or with the kube-proxy tool itself.
Can you confirm which is a closer match for the issue you wanted to report?
/triage needs-information

@sftim ,Thanks for your reply, my problem is for kube-proxy tool itself.
1."masqueradeAll:false" is usless in kube-proxy configmap, all of the external traffic has been SNAT by kube-proxy with this option set to false (in iptable section).
2. Can I remove "clustercidr" option from kube-proxy configmap? I'm not sure this is right, what  I want is all of the traffic can not be SNAT, and pods can get source ip from connections.


Add enable service and plugin for LBaaS v2

Add enable service and plugin into local.conf for LBaaS v2
Fixes: #49

Small fix, let's merge this patch to check result.


WIP

PR Type
enhancement, documentation

Description

Integrated PSI (Pressure Stall Information) monitoring into the memory monitor.
Added configuration option PSI_THRESHOLD_PERCENT to specify PSI threshold.
Implemented PSI monitoring thread and related functions.
Updated configuration validation and logging to include PSI settings.
Enhanced documentation to include PSI configuration instructions.

Changes walkthrough 📝
Relevant filesEnhancement9 files

memory-monitor-handler.shAdjust total size calculation for psi.txt file

pkg/memory-monitor/src/monitor/memory-monitor-handler.sh
Adjusted total size calculation to account for psi.txt file.  Ensured psi.txt size is subtracted if it exists.

+4/-0

config.cAdd and validate PSI threshold configuration

pkg/memory-monitor/src/monitor/config.c
Added PSI_THRESHOLD_PERCENT configuration.  Implemented validation for PSI_THRESHOLD_PERCENT.  Updated logging to include PSI_THRESHOLD_PERCENT.

+19/-2

config.hDefine PSI threshold in configuration structure

pkg/memory-monitor/src/monitor/config.h
Defined CHECK_INTERVAL_SEC.  Added psi_threshold_percent to config_t.

+4/-0

monitor.cIntegrate PSI monitoring into memory monitor

pkg/memory-monitor/src/monitor/monitor.c
Included psi.h for PSI monitoring.  Added PSI monitoring thread management.  Updated thread management for PSI monitoring.

+43/-4

monitor.hDefine PSI monitoring arguments structure

pkg/memory-monitor/src/monitor/monitor.h

Added monitor_psi_args_t structure for PSI monitoring.

+5/-0

psi.cImplement PSI monitoring functionality

pkg/memory-monitor/src/monitor/psi.c
Implemented PSI monitoring functionality.  Added functions to check PSI enablement and read PSI values.

+127/-0

psi.hDeclare PSI monitoring functions

pkg/memory-monitor/src/monitor/psi.h

Declared PSI monitoring functions.

+13/-0

util.cAdd string to float conversion utility

pkg/memory-monitor/src/monitor/util.c

Added strtofdec function for string to float conversion.

+18/-0

util.hDeclare string to float conversion function

pkg/memory-monitor/src/monitor/util.h

Declared strtofdec function.

+1/-0

Miscellaneous1 files

procfs.cRemove redundant interval definition

pkg/memory-monitor/src/monitor/procfs.c

Removed redundant CHECK_INTERVAL_SEC definition.

+0/-2

Configuration changes2 files

MakefileAdd local deploy target for pressure tool

pkg/memory-monitor/Makefile

Added local-deploy-pressure target.

+1/-0

memory-monitor.confAdd PSI threshold to default configuration

pkg/memory-monitor/src/monitor/memory-monitor.conf

Added PSI_THRESHOLD_PERCENT with default value.

+2/-1

Documentation1 files

README.mdDocument PSI threshold and config instructions

pkg/memory-monitor/README.md
Documented PSI_THRESHOLD_PERCENT configuration.  Added instructions for copying default config for editing.

+11/-0

💡 PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

Summary by CodeRabbit

New Features

Added a help command for local-deploy-pressure to enhance usability.
Introduced a new configuration parameter PSI_THRESHOLD_PERCENT for improved monitoring control.
Implemented functionality for monitoring Pressure Stall Information (PSI) to provide better insights on memory pressure.

Documentation

Updated README with new instructions for configuration management and added details about PSI_THRESHOLD_PERCENT.

Bug Fixes

Enhanced memory size calculations by accounting for the size of psi.txt.

Chores

Added error handling for string-to-float conversions in utility functions.

Improve the reliability of the PSI availability check by verifying the content format

Consider using a more robust method to check if PSI is enabled, such as attempting to read and parse the content of the /proc/pressure/memory file, instead of just checking if the directory and file exist.
pkg/memory-monitor/src/monitor/psi.c [27-37]
 bool psi_is_enabled() {
-    // Check if the PSI is enabled by attempting to open the /proc/pressure directory
-    if (opendir("/proc/pressure") == NULL) {
-        return false;
-    }
-    // Check if the memory pressure file exists
     FILE *fp = fopen("/proc/pressure/memory", "r");
     if (fp == NULL) {
         return false;
     }

+    char line[256];
+    bool valid_format = false;
+    while (fgets(line, sizeof(line), fp) != NULL) {
+        if (strstr(line, "some avg10=") != NULL && strstr(line, "avg60=") != NULL &&
+            strstr(line, "avg300=") != NULL && strstr(line, "total=") != NULL) {
+            valid_format = true;
+            break;
+        }
+    }
+
+    fclose(fp);
+    return valid_format;
+}
+```

<input type="checkbox" id="" disabled="" class="task-list-item-checkbox"> Apply this suggestion

Suggestion importance[1-10]: 8

Why: This suggestion enhances the robustness of the PSI check by ensuring that the file not only exists but also contains data in the expected format, which is crucial for accurate monitoring.

But the logic for additional checks lays further...

@CodiumAI-Agent /review
--pr_reviewer.inline_code_comments=true
-i
--pr_reviewer.require_can_be_split_review=true
--pr_reviewer.num_code_suggestions="1"

Incremental Review Skipped
No files were changed since the previous PR Review

@CodiumAI-Agent /review
--pr_reviewer.inline_code_comments=true
-i
--pr_reviewer.num_code_suggestions="1"

Incremental Review Skipped
No files were changed since the previous PR Review

@CodiumAI-Agent /review
--pr_reviewer.inline_code_comments=true

PR Reviewer Guide 🔍

⏱ Estimated effort to review: 4 🔵🔵🔵🔵⚪
🧪 No relevant tests
🔒 No security concerns identified
🔀 Multiple PR themes

Sub-PR theme: Implement PSI (Pressure Stall Information) monitoring

Relevant files:

pkg/memory-monitor/src/monitor/psi.c
pkg/memory-monitor/src/monitor/psi.h
pkg/memory-monitor/src/monitor/config.c
pkg/memory-monitor/src/monitor/config.h

Sub-PR theme: Integrate PSI monitoring into main monitor logic

Relevant files:

pkg/memory-monitor/src/monitor/monitor.c
pkg/memory-monitor/src/monitor/monitor.h

Sub-PR theme: Update documentation and configuration for PSI monitoring

Relevant files:

pkg/memory-monitor/README.md
pkg/memory-monitor/src/monitor/memory-monitor.conf
pkg/memory-monitor/Makefile

⚡ Key issues to review
Potential Race ConditionThe use of atomic_compare_exchange_strong in psi_check_threshold may not fully prevent concurrent executions of the handler
Resource LeakMemory allocated for monitor_psi_args_t is not freed if thread creation fails


test.tests sets up trap using numbers that aren't the same as on z/OS

Need to switch from: 0 1 2 3 6 15 to 0 HUP INT QUIT ABRT TERM

fixed in reducefailures branch

with this change, it still fails printing out a 1 instead of a 0


smhasher analysis

it is now integrated into smhasher, all tests pass.
https://github.com/rurban/smhasher/
it's pretty fast, but there's a big problem: in most cases khashv_hash_vector is not inlined.
and the codesize is pretty big.

I will try and look at it these evening when I get a little more time to figure out if I can figure out why it's not in-lining. If you have any examples or a place to zero in on that would be great.
I will add I notice your calling prep seed in every call to the test function which not reflect normal usage. You could either set the struct members directly or use khashv_prep_seed128() which does nothing else but set the seed struct. The khashv_prep_seed32() and  khashv_prep_seed64() do a little extra work to avoid for instance 0xffffffff and 0x00000000ffffffff being treated the same seed. Mainly using the final mixing function permute the bits.

Yes, prep_seed is unnecessary. Will fix that.

Do you have the binaries for both the builds before the change? I find it interesting that it got slower when looking at the diffs. I am wondering what happened? When looking at these, moving the seeding outside the hot loops should have just decreased the latency.
https://github.com/rurban/smhasher/commit/306a533d023431287d69bc83fe8e7ba9d1638e5b#diff-a19735dd699653b53026705c7618a79584242184dee0f6ea7e21bb5c5b6bf4a0
https://github.com/rurban/smhasher/commit/306a533d023431287d69bc83fe8e7ba9d1638e5b#diff-a593d27a47a069621b3affb539f34c685bd7a2a04a9f787aef1bb0d5d874073a

Closing for now.


HDDS-6013. Fix Documentation button for release pages.

Fix documentation button for release pages. See HDDS-6013 for details.
Tested locally with Hugo on the release pages for 1.2.0, 1.1.0 (post hadoop) and 1.0.0 (still part of hadoop). The button worked on all these pages.

Oh whoops I'll close this one as dup.

I think your problem description in the Jira issue was much better.  Mine was a bit vague, hard to correlate with the broken link you also noticed.


decoder shouldn't create unsupported queries

status queries are optional:
VkQueueFamilyQueryResultStatusPropertiesKHR.queryResultStatusSupport should be tested before creating or using status queries.

Sure, my bad, @airlied! I should have NOT created the m_queryPool for platforms that don't support queryResultStatusSupport. I'll fix that.

Hi @airlied, is the latest version working for you, when it comes to regarding the VkQueueFamilyQueryResultStatusPropertiesKHR.queryResultStatusSupport by the sample?

this works now


Flaky results in uniqueness of modes names from different packages

Describe the bug
Flaky results in maintaining unique model names after https://github.com/swaggo/swag/pull/736
To Reproduce

Have two models (structs) with same name in different packages (same package name but different parent folders).

For example in a project called Awesome, a struct called Message is in two packages, namely foo (Awesome/foo) and foo (Awesome/pkg/foo).

Built swagger files again and again, and it shows flaky results.

Expected behavior
The struct names should be: github.com_Awesome_foo.Message and github.com_Awesome_pkg_foo.Message
Actual behavior
Sometimes the struct names are github.com_Awesome_foo.Message and foo.Message.
Only one conflicted struct is handled
Other times it is github.com_Awesome_pkg_foo.Message and foo.Message.
Your swag version
e.g. 1.6.7
Your go version
e.g. 1.14.4
Desktop (please complete the following information):

OS: macOS 10.15.5
Browser: Safari
Version:

Additional context
I would expect that in case of conflict, ideally all conflicted structs should take up full names.

I think a stable result is enough

Sorry, I have not found a better way to make it stable.

I am not able to test this, as $ go get -u github.com/swaggo/swag/cmd/swag is pulling v1.6.7 tag, not master. How can I pull master code of this repo?

@sdghchj Thanks for the super quick action!
Please let me test this.

@sdghchj I known where the problem is, but have not got a proper idea to fix it.

Okay, please keep the issue open in that case. Sooner or later this will need fixing for sure.


Bug Persist output file to Azure Storage with the Batch service API in Python

Package Name: azure
Operating System: Ubuntu16
Python Version: 3.7.6

Describe the bug
I am trying to push the output files generated from Azure batch to Storage account container, by using the OutputFile module from azure.batch.models
The issue is with the file_pattern. The following is part of my code:
for of in output_files:
           file_options = OutputFileUploadOptions(
               upload_condition=OutputFileUploadCondition.task_completion)

           destination_path = f"{job_id}/{task_id}/{of}"
           container = OutputFileBlobContainerDestination(
               container_url=self.file_output_container,
               path=destination_path)
           output_file_destination = OutputFileDestination(
               container=container)

           az_output_files.append(
               OutputFile(**file_pattern**=of,
                          destination=output_file_destination,
                          upload_options=file_options))
           return az_output_files

When I pass a parameter with value of "../std*.txt", it will throw an error
Also, I tried "..\std*.txt" and still failed with invalid input error.
So, I added a command "echo hello >> stdtest.txt" in the job, and change the file_pattern to be "std*.txt", then it works.
As the working directory is "wd", and the stdout and stderr are at one level up directory, I need the "../" stuff, which has some issues
Would appreciate if you have a check on this and have it fixed, thanks

Hi @lsongcoles thanks for reporting this @lsongcoles, adding the right team to take a look at this: @xgithubtriage

Hi @kaerm
Would you like to route this issue to azure batch?
Thanks for your help!

@lsongcoles Could you clarify the error you are seeing? The following task works perfectly fine for me:
{
  "id": "echo2",
  "commandLine": "/bin/bash -c \"echo hello\"",
  "resourceFiles": [],
  "environmentSettings": [],
  "userIdentity": {
    "autoUser": {
      "scope": "pool",
      "elevationLevel": "nonadmin"
    }
  },
  "constraints": {
    "maxWallClockTime": "P10675199DT2H48M5.4775807S",
    "retentionTime": "P7D",
    "maxTaskRetryCount": 0
  },
  "outputFiles": [{"destination": {"container": {"containerUrl": "xxxxxx"}}, "filePattern": "../stdout.txt", "uploadOptions": {"uploadCondition": "taskcompletion"}}]
}

If you are getting a file upload error also try taking a look and/or attaching the fileupload logs from the VM

Closing the issue due to inactivity


Added example image to featureToH3Set

When I used the method featureToH3Set in my code it wasn't quite clear to me that it only adds hexagons with the center inside the polygon.
I therefore added a example screenshot and highlighted the passage in the docs to make this clearer (for people like me who like visual examples 😆)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Coverage remained the same at 100.0% when pulling f2ec9c20b0935e31b23aa0e14f663c82180b3672 on matthiasfeist:master into b4f8cc9487fdb1d1991e3a5d9bf4a536eb76213f on uber:master.


[Enhancement] Support tokenize function

Why I'm doing:
The different results of tokenization provided by various tokenizers are too vague to users, so we need a tokenize function to allow users to figure it out easily.
What I'm doing:
Support a tokenize function, like tokenize(<tokenizer_name>, <content>)
Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.3
[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

@dujijun007 thank you for the contribution, could you create an issue to describe this new function?
About its interface, input, output and limits?

@dujijun007 thank you for the contribution, could you create an issue to describe this new function? About its interface, input, output and limits?

ok, link it here(#45145)


Run Travis on all open PRs

I'd like us to start making progress, so it would be awesome to know which PRs can be merged. Step one: get travis running our tests.
@luishdez what's outstanding for this?

@josephholsten In order to trigger travis… I think I'll have to update all PRs because those exists before the travis setup. No sure if there is another way to trigger travis in old PRs
We should check https://github.com/chef-brigade/mongodb-cookbook/issues/34 before triggering as well

Ok I think we can create a /develop branch to test merges and trigger travis

OK!  I just found out that if you close and open again the PR re-triggers travis :)


Fix 401 Unauthorized errors + empty logstash password check

Pull Request Template
Description
This fixes Issue #39. Additionally I updated the Readme to reference the files which includes the logstash passwords and therefore needs to be updated. A small script automates the password update if someone wants to use it.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How Has This Been Tested?
run: docker-compose up and check the log for 401 Responses from the es01 container
Checklist:

[x] Code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Turns out request which use the credentials in 50-outputs.pfelk need higher privileges or something so I changed the user back to elastic and updated the rest accordingly

@braunbearded - awesome/thanks!  I’ll update in the next day or two as I’m currently traveling.


change the version

issue #3234

@dulcemontemayor The PR updates the version of an associated script file to the most recent version. Please review and merge as the time permits. Thanks.

@DulceMontemayor Following up for updates. Thanks.

@DulceMontemayor Following up for updates.


windows store version unable to switch languages

machine translation:
Dbeaver ce installed in windows store cannot switch language. the prompt file has no permission, and the error is as follows: "Can't switch language to zh"
Reason:
Can't save startup configuration - access denied.
You could try to change national locale manually in 'C:\Program Files\WindowsApps\DBeaverCorp.DBeaverCE_<IP_ADDRESS>_x64__1b7tdvn0p0f9y\ Dbeaver.ini'. refer to readme.txt file for details. "
I think this problem is widespread, and many people in the comment area have given feedback on this problem. You can also go to the windows store to install Dbeaver ce and then enter Window-> Preferences-> User Interface to switch languages to reproduce this problem.

Thank you for the report. As a workaround please open properties of DBeaver label.
-nl -en should иу written there. Delete it, open DBeaver and change the language in Window>Preferences>User interface

谢谢你的报告。作为解决方法，请打开 DBeaver 标签的属性。

nl - en 应该写在那里。删除它，打开 DBeaver 并更改 Window>首选项>用户界面中的语言

C:\Program Files\WindowsApps\DBeaverCorp.DBeaverCE_<IP_ADDRESS>_x64__1b7tdvn0p0f9y\dbeaver.ini

I revised this file to warn me that I have no permission，So is the administrator.
I think I should switch languages on the interface instead of other methods,I don't need to solve it now.
I want you to fix it.
😘

Thank you for the report. As a workaround please open properties of DBeaver label.
-nl -en should иу written there. Delete it, open DBeaver and change the language in Window>Preferences>User interface

C:\Program Files\WindowsApps\DBeaverCorp.DBeaverCE_<IP_ADDRESS>_x64__1b7tdvn0p0f9y\dbeaver.ini

I revised this file to warn me that I have no permission，So is the administrator.
I think I should switch languages on the interface instead of other methods,I don't need to solve it now.
I want you to fix it.
😘

It is hard to use dbeaver for new Windows users, if it is not possible to change some settings, for example the language. And the readme is poor about this. There is a bit about -data option, but this is doesn't work for the language settings.
I tried

"C:\Program Files\DBeaver\dbeaver.exe" -data "C:\Users\ggoer\AppData\Roaming\DBeaverData\workspace6"

"C:\Program Files\DBeaver\dbeaver.exe" -data D:\Users\ggoer

and it looks like different workspaces are used. There are also some folders .metadata created. But whenever I try to change the language I get the same error, no matter in which workspace I am:
Can't save startup configuration - access denied.
You could try to change national locale manually in 'C:\Program Files\DBeaver\dbeaver.ini'. Refer to readme.txt file for details.
  C:\Program Files\DBeaver\dbeaver.ini
  C:\Program Files\DBeaver\dbeaver.ini

-nl en works. OK.
But why not just write this in the readme:

At the moment it is not (or not always) possible to change the language via the settings, as a workaround please use the following parameter.

A normal new user thinks: If I can't change the language setting, then I probably can't change the other settings either. How am I supposed to work with the program then?

@aisbergde I agree that the option to change language has to work correctly and it is going to be fixed in 21.0.1 release.
We gave preference to other tasks for some time, because there was the workaround

Unfortunately it is not possible to modify Store application start parameters. As well as modify any of its configuration files.
At the moment the only way to modify default language is thru app parameter or by changing dbeaver.ini. It is Eclipse platform limitation.

Just run DBeaver as Adminstrator and you can change it. Easiest way my SQL Bros!

DBeaver Version <IP_ADDRESS>303250535
Windows 10 22H2
Can't change the interface language, what I have done:

In the dbeaver.ini added "-nl en" before the line -vmargs;
Ran DBeaver as Adminstrator and tried to change the interface language to "en".

Please help me with it!)

The new version can now switch languages, thank you for your contribution!


Remove Usage of CreateHeapFn in MVI

Chromium doesn't allow usage of std::function, which is what the CreateHeapFn is defined as.
I see two options:

Keep this as a callback and take a dependency on chromium base::OnceCallback.
Get rid of the callback param and inline the code in some manner, probably will add a ResourceAllocator::CreateCommittedResource function and call it.

2 is my preferred option since we don't have any other possible values for CreateHeapFn right now. This will probably mean some code duplication in the future when adding other methods of heap creation (I think this is what's in Dawn).
@egalli
@bbernhar - I'm not sure how much autonomy you're giving me over GPGMM issues while you're on loan. I'm assuming you'll want to have high level input until you tell me otherwise.

Perhaps we could replace CreateHeapFn with a simple c-style call-back instead? A more flexible function type isn't strictly required and the call-back was to guarantee residency operations happen before D3D creation (and w/o D3D12_HEAP_FLAG_CREATE_NOT_RESIDENT).
Happy to give feedback but do as you see fit to unblock.

It looks like we can do this as a C-style function pointer. Issue is we can't use lambda capture if we're assigning it to a function pointer - this means we have pass around a bunch more params and it feels awkward, although it does seem to work.
Whether I do this or just inline everything, both are significant enough deviations from the full GPGMM that my preference would be to just do what's simple - which is just putting CreateCommittedResource inline inside CreateResource.
I'm unsure what the plan is for MVI - is this expected to slowly evolve into full or some subset of GPGMM through Chromium reviews? or is it expected to be deprecated and replaced all at once later on?

Idea is to incrementally replace the MVI with a full GMM (eg. gpgmm). The problem of moving CreateCommittedResource outside of CreateHeap is that CreateHeap wants to prevent D3D from paging out the wrong heaps (implicitly) and needs to be created with the ID3D12Pageable, returned by CreateCommittedResource, it represents. The call-back itself doesn't need to capture any additional state and pass it around so a lambda is overkill.

Fixed with #670


Sudo/GDM login working, not working on screen lock, hibernate, sleep (Manjaro KDE with GDM instead of SDDM)

As the title says, I have Howdy working to log me in at the GDM login screen and for sudo in terminal. However, if I let my laptop sleep/hibernate or manually lock it, then Howdy is not working for that login screen that is part of Manjaro KDE. I cannot figure out what I need to change, or what that login screen would be called since it is not GDM after initial login. I have added 'auth sufficient pam_python.so /lib/security/howdy/pam.py' to the following PAM files trying to get it to work:

sudo
system-local-login
KDE (thought this would fix it)
login (might be unnecessary)
system-login (might be unnecessary)

Thanks in advance for any help!

You might have to try to set more broader permissions for the files in /lib/security/howdy, you can find the exact chmod command in a few other threads

I encountered a similar issue to this just now, using the official Ubuntu PPA. I was unable to use howdy with swaylock - ModuleNotFoundError: No module named 'recorders.video_capture' - because swaylock doesnt run as root.
Is there no way for the Howdy packages to set the executable bits to the correct directories? It would've been nice if howdy worked with non-suid programs out of the box.

@mortie Do you know what user swaylock run as ? If so, you'll need to add this user to a group (for example biologin) and also set howdy files to that group (and /dev/videoXXX). Then you don't need to run as root anymore.

version

Howdy 2.6.1
Linux arch 5.11.6-arch1-1
KDE 5.21.2

pam config

/etc/pam.d/sudo
/etc/pam.d/system-local-login
/etc/pam.d/sddm

described in my bolg
good

sudo
start login

bad
same as u

I'm having an almost-identical issue on Ubuntu 21.10 and Howdy 2.6.1.  I can run all of the howdy commands, but it fails to work as a PAM module.
I did install the OpenCV Python module (Ubuntu: apt install python3-opencv).
user@computer:~$ sudo -s
Attempting face detection
Traceback (most recent call last):
  File "/lib/security/howdy/compare.py", line 23, in <module>
    from recorders.video_capture import VideoCapture
ModuleNotFoundError: No module named 'recorders.video_capture'
Unknown error: 1

I've tried running chmod -R 755 /lib/security/howdy like the wiki mentions, but there was no change afterwards.


[Misc] Code cleanup in GTWplayerblips

I'm not sure why the commit where I updated my fork is in here.

Looks like your fork is ahead of ours currently but I'm just as confused as you are, I suppose we have a conflict in our branches or something but it is however possible to merge and the changes are supposed to be merged anyway, I do not recognize the changes in player-blips however but it looks mostly like formatting changes.
I think it's time to define some formatting guidelines for this project however, as I don't see any point in developers changing to their own preferences back and forth.


Reading capacitance fails on esp32 after sleeping

Hi, loving the sensor. I'm using the I2CSoilMoistureSensor library but I can't get seem to get the sleep() method to work on an esp32. After sleeping all subsequent returned values are 65535 and I get an i2c error: requestFrom(): i2cRead returned Error 263 until I do a hard reset by cutting power to the sensor.
Example code with issue:
Wire.begin(I2C_SDA_PIN, I2C_SCL_PIN);
Wire.setClock(50000);

I2CSoilMoistureSensor moistureSensor;
moistureSensor.begin(true);

while (true) {
    auto moisture = moistureSensor.getCapacitance();
    Serial.println(moisture);

    moistureSensor.sleep();
    delay(1000);
}

Output:
258
[  4099][E][Wire.cpp:513] requestFrom(): i2cRead returned Error 263
65535
[  8122][E][Wire.cpp:513] requestFrom(): i2cRead returned Error 263
65535
[ 12145][E][Wire.cpp:513] requestFrom(): i2cRead returned Error 263
65535
[ 16167][E][Wire.cpp:513] requestFrom(): i2cRead returned Error 263
65535
...

When I remove the moistureSensor.sleep() line and do a hard reset with a power cut it works fine again. Soft rebooting with moistureSensor.resetSensor() doesn't work either.

Hi, thanks for the report,
What version of ESP32 are you using? This might be related to https://github.com/espressif/esp-idf/issues/4173 - 13ms max clock stretching limit on older ESP32 chip versions.

Hi, I think this is indeed the problem because I'm using a regular ESP32 (not ESP32-S2 or S3). I'm going to try working around this with bit banging or a software i2c library (maybe then switching to using hardware i2c once it's awake).


Today, yesterday and monthly views no longer showing

Hey guys,
This extension has been working really well for me the last couple years!
Thanks for keeping it up and running.
Unfortunately, the Today, Yesterday and Monthly grouped views in the mail inbox column are no longer showing. The current appearance is all of the messages in a line going straight downwards, similar to Gmail's current layout.
I think the html coding might have changed in gmail to cause the layout to stop functioning.
Can someone please have a look at this?
Many thanks,
Naughtiboi

@naughtiboi this repo is no longer maintained. It's been moved to
https://github.com/team-inbox/inbox-reborn
It's also approved on the Chrome store as well. There is a bug for this same issue we are working through as well.

Thanks a lot for the heads up. :)


Add tabbed TranslatedContent form

Close #1321; close #1399.
This branch adds a custom template for TranslatedContent models, which makes sure we can add/remove translations inline.
https://github.com/user-attachments/assets/a628756f-9c15-4c13-8d42-eba17e9f7403

NB: I've also experimentally disabled the initCollapsibleInlineForms in experiment_form.js, and instead added the ['wide', 'collapse'] classes to the Inline classes. Let's compare what we like best after merging. I do think the custom solution looks nicer, but perhaps we can achieve this with styling rather than javascript?

Just a quick comment on what I see directly in the video, not code-related per se:

The language is a bit hard to read on a (dark) blue background. Perhaps you can consider making the text white (with a text-shadow)
The tabs' behavior isn't very dynamic. You have to save the experiment before you see the new sorting order or a newly added language. Would it be an idea to add some javascript to show (at least) the newly added language? Having the sorting order or the current language code of the tab in question be dynamic seems of less importance to me.

I implemented the suggested style changes, and also went down deeper in the form to control the behaviour of BlockTranslatedContent: instead of being able to add these independently of the ExperimentTranslatedContent, these will now always be set and prefilled with values from the ExperimentTranslatedContent objects of the Experiment parent.  I decided to leave any dynamic behaviour for later: let's see first if users find the form usable enough without it.
Changing the order dynamically would be, by my estimate, easier than creating an extra tab with a selected language on the fly, which requires dealing with several forms which do not correspond to a database object. Requiring the user to save first before seeing a new ExperimentTranslatedContent object reflected as a tab may then just be the lesser evil, rather than potentially creating many forms on the fly, some of which won't be valid if the user only selected a language, but entered no text for the required Name field. With the tab / hidden forms setup, it would be hard to find back which of the translated content forms throws the validation error.


Determine changeset from build information

This updates knative.dev/pkg to @main to pick up a recent change to make
changeset.Get() determine the built version from information embedded in
the binary, and no longer return an error.
This also lets us remove symlinks for HEAD and refs in kodata, since
they were only there to enable the previous changeset.Get()
implementation.
https://github.com/tektoncd/pipeline/pull/4928 recently bumped our knative/pkg dep to release-1.6 (which was the real hard part, thanks @vdemeester!). Here's the diff between release-1.6 and what we'll import now: https://github.com/knative/pkg/compare/release-1.6...7b8b06028e4faf966147049c807781d8363a7b88
The relevant changes are https://github.com/knative/pkg/pull/2548 and https://github.com/knative/pkg/pull/2549
Similar change to knative/serving to drop HEAD and refs: https://github.com/knative/serving/pull/13130
cc @dprotaso
/kind cleanup
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs included if any changes are user facing
[ ] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[x] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
Version informaiton added to workload labels is determined from information embedded by Go, instead of relying on symlinks to Git information in our build process.

Oh, looks like tests still run with go 1.17:
go version go1.17.11 linux/amd64

We'll need to bump this to 1.18 to get this to work. Since Go only officially supports the last two versions, and 1.19 just came out, we should bump to 1.18 anyway.

/retest

...I forgot that https://github.com/tektoncd/pipeline/pull/5134 hadn't merged yet. d'oh.

/retest

/test check-pr-has-kind-label

/approve

/retest


controller/test: Add simple test framework for controller

We don't have functions for spawning EtcdProxy Controller in tests and therefore we need to repeat the same logic for each test we write.
This PR introduces a simple testing "framework" by adding several functions that can be used to spawn a mocked controller when needed.
Previously this was part of #63, however, as they don't relate to each other at this point and to make it easier to review both PRs, I decided to move this to a new PR.
/cc @sttts @deads2k

minor comment on expressing the idea.  The idea is fine.

/lgtm


hello

Sai can add this function thank you
Server Launch Option which locks the Fountain of Youth to one Golden Age Ruins server:  -AlwaysActiveFountainsOfYouth

Hi jerome,
To activate this on all of your servers, place the following line in the  /etc/atlasmanager/atlasmanager.cfg file.
atlasflag_AlwaysActiveFountainsOfYouth=true

thank you


Fix font-dependent test

closes https://github.com/flutter/flutter/issues/83129
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

(triage): @LongCatIsLooong @yjbanov Do you still have plans for this PR?

CI appears to be failing on this test

I looked at the failure and it looks like I changed the implementation at some point. If the cursor height has to match the glyph it points to, then the cursorHeight setter makes no sense when different InlineSpans can have different font sizes. In general cursor height follows the line height (e.g. the android implementation: https://cs.android.com/android/platform/superproject/+/master:frameworks/base/core/java/android/text/Layout.java;bpv=1;bpt=1;l=1910) but I'm a bit reluctant to make that change right away since getting the line height right now can be a bit expensive.  There are new skparagraph APIs available that would make implementing this much easier so I'll close this until those APIs are exposed.


Move OAuth2 Clients as extension

Description
We replace our current implementation with the extensibility option.
Acceptance Criteria

[ ] Validate with Product Owner if the current implementation is correct
[ ] Replace the current implementation with extensibility
[ ] Deploy to stage and Test all the edge cases on stage
[ ] Deploy to production
[ ] Remove old codebase
[ ] Adjust Tests to use extensions

remember about https://github.com/kyma-project/busola/issues/2090


Errors during "pip3 install -r requirements.txt"

Was walking through the steps and got to this point with the following error:
Could not install packages due to an EnvironmentError: [Errno 13] Permission denied: '/Library/Python/3.7'
Consider using the --user option or check the permissions.
I'm on MacOS.
Note: Sudo would work here and it seems to be a permissions issue but I just want to point out that a lot of people will likely run into this.

@econoar
reinstall pip
curl https://bootstrap.pypa.io/get-pip.py -o get-pip.py
python3 get-pip.py —force-reinstall

install requirements
pip install -r requirements.txt

related? https://github.com/ethereum/eth2.0-deposit-cli/issues/52


Pluto Grid issue

i tried to insert data into plutogrid their was a lot of data but after  filtering out the data their was just 100records so the output of API call was correct but its not displaying the data it says null value cant be implemented into null form and one more was string please can you help me with that for that issue i shifted to data table and its a hell a lot of work in that

@MOULNIA
Can you provide a minimal example code for me to verify your problem?


Title: Add Weather-Forecast intermediate python Desktop application

Is your feature request related to a problem? Please describe.
Currently, the weather application does not provide clear feedback when a user inputs an invalid city name. This can lead to confusion and frustration, as the app either returns an error message that is not user-friendly or fails silently without any response. As a result, users are left uncertain about what went wrong and how to correct their input.
Describe the solution you'd like.
Implement enhanced error handling for invalid city names. This feature should include:
User-Friendly Error Messages: Display a clear and concise error message when an invalid city name is entered, informing the user that the city could not be found.
Suggestions for Common Mistakes: Offer suggestions or alternative city names if a common misspelling is detected.
Retry Option: Provide a simple way for the user to retry their search after receiving an error message, such as clearing the input field and refocusing the cursor.
Describe alternatives you've considered.
While each alternative has its merits, the proposed solution of enhanced error handling with user-friendly messages, suggestions, and a retry option strikes a balance between improving user experience and maintaining simplicity in implementation. This approach addresses the core issue effectively without introducing significant complexity or dependency on external services.
Add any other context or screenshots about the feature request here.

@Kushal997-das plzz assign me to this issue and give appropriate labels


Could not exit program after dying

Hey Chris(@cyberfilth),
While trying Alpha 33 I could not exit the game after I died.
Cheers,
Gus

Ah, the game over screen is still being worked on so the menu text isn't finished yet. 'x' should exit the game over screen though

I've added a hint at the bottom of the Game Over screen now. More will be added later that allow you to view game stats.

Hey Chris(@cyberfilth),
Thanks for that, it work a charm.
Closing!!
Cheers,
Gus