problem to update ilo pass

when I play the role " updatepass", I receive "Bad Request" error.
I already operated "updaterole" without any issue. But update request to ILO password is not successful.

tasks file for updatepass

name: Update user password
redfish_command:
category: Accounts
command: UpdateUserPassword
baseuri: "<IP_ADDRESS>"
username: "ckirim"
password: "admin123"
account_username: "ckirim"
account_password: "serkan"
~

(oneview) [root@localhost ilo]# ansible-playbook playbook.yml -vvv
[DEPRECATION WARNING]: Ansible will require Python 3.8 or newer on the controller starting with Ansible 2.12. Current version: 3.6.12 (default, Sep 15 2020, 12:49:50) [GCC 4.8.5 20150623 (Red Hat 4.8.5-37)]. This feature will be removed
from ansible-core in version 2.12. Deprecation warnings can be disabled by setting deprecation_warnings=False in ansible.cfg.
ansible-playbook [core 2.11.7]
config file = None
configured module search path = ['/root/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
ansible python module location = /root/oneview/lib64/python3.6/site-packages/ansible
ansible collection location = /root/.ansible/collections:/usr/share/ansible/collections
executable location = /root/oneview/bin/ansible-playbook
python version = 3.6.12 (default, Sep 15 2020, 12:49:50) [GCC 4.8.5 20150623 (Red Hat 4.8.5-37)]
jinja version = 3.0.3
libyaml = True
No config file found; using defaults
host_list declined parsing /etc/ansible/hosts as it did not pass its verify_file() method
Skipping due to inventory source not existing or not being readable by the current user
script declined parsing /etc/ansible/hosts as it did not pass its verify_file() method
auto declined parsing /etc/ansible/hosts as it did not pass its verify_file() method
Skipping due to inventory source not existing or not being readable by the current user
yaml declined parsing /etc/ansible/hosts as it did not pass its verify_file() method
Skipping due to inventory source not existing or not being readable by the current user
ini declined parsing /etc/ansible/hosts as it did not pass its verify_file() method
Skipping due to inventory source not existing or not being readable by the current user
toml declined parsing /etc/ansible/hosts as it did not pass its verify_file() method
[WARNING]: No inventory was parsed, only implicit localhost is available
[WARNING]: provided hosts list is empty, only localhost is available. Note that the implicit localhost does not match 'all'
[WARNING]: running playbook inside collection hpe.ilo
redirecting (type: modules) ansible.builtin.redfish_command to community.general.redfish_command
Skipping callback 'default', as we already have a stdout callback.
Skipping callback 'minimal', as we already have a stdout callback.
Skipping callback 'oneline', as we already have a stdout callback.
PLAYBOOK: playbook.yml **********************************************************************************************************************************************************************************************************************
1 plays in playbook.yml
PLAY [localhost] ****************************************************************************************************************************************************************************************************************************
TASK [Gathering Facts] **********************************************************************************************************************************************************************************************************************
task path: /root/.ansible/collections/ansible_collections/hpe/ilo/playbook.yml:2
<<IP_ADDRESS>> ESTABLISH LOCAL CONNECTION FOR USER: root
<<IP_ADDRESS>> EXEC /bin/sh -c 'echo ~root && sleep 0'
<<IP_ADDRESS>> EXEC /bin/sh -c '( umask 77 && mkdir -p "echo /root/.ansible/tmp"&& mkdir "echo /root/.ansible/tmp/ansible-tmp-1645442563.2000966-129058-278645987446545" && echo ansible-tmp-1645442563.2000966-129058-278645987446545="echo /root/.ansible/tmp/ansible-tmp-1645442563.2000966-129058-278645987446545" ) && sleep 0'
Using module file /root/oneview/lib64/python3.6/site-packages/ansible/modules/setup.py
<<IP_ADDRESS>> PUT /root/.ansible/tmp/ansible-local-129051_vkfwboz/tmpt41t_lb7 TO /root/.ansible/tmp/ansible-tmp-1645442563.2000966-129058-278645987446545/AnsiballZ_setup.py
<<IP_ADDRESS>> EXEC /bin/sh -c 'chmod u+x /root/.ansible/tmp/ansible-tmp-1645442563.2000966-129058-278645987446545/ /root/.ansible/tmp/ansible-tmp-1645442563.2000966-129058-278645987446545/AnsiballZ_setup.py && sleep 0'
<<IP_ADDRESS>> EXEC /bin/sh -c '/root/oneview/bin/python3.6 /root/.ansible/tmp/ansible-tmp-1645442563.2000966-129058-278645987446545/AnsiballZ_setup.py && sleep 0'
<<IP_ADDRESS>> EXEC /bin/sh -c 'rm -f -r /root/.ansible/tmp/ansible-tmp-1645442563.2000966-129058-278645987446545/ > /dev/null 2>&1 && sleep 0'
ok: [localhost]
META: ran handlers
TASK [hpe.ilo.updatepass : Update user password] ********************************************************************************************************************************************************************************************
task path: /root/.ansible/collections/ansible_collections/hpe/ilo/roles/updatepass/tasks/main.yml:3
<<IP_ADDRESS>> ESTABLISH LOCAL CONNECTION FOR USER: root
<<IP_ADDRESS>> EXEC /bin/sh -c 'echo ~root && sleep 0'
<<IP_ADDRESS>> EXEC /bin/sh -c '( umask 77 && mkdir -p "echo /root/.ansible/tmp"&& mkdir "echo /root/.ansible/tmp/ansible-tmp-1645442564.411589-129120-34397279681846" && echo ansible-tmp-1645442564.411589-129120-34397279681846="echo /root/.ansible/tmp/ansible-tmp-1645442564.411589-129120-34397279681846" ) && sleep 0'
redirecting (type: modules) ansible.builtin.redfish_command to community.general.redfish_command
Using module file /root/.ansible/collections/ansible_collections/community/general/plugins/modules/redfish_command.py
<<IP_ADDRESS>> PUT /root/.ansible/tmp/ansible-local-129051_vkfwboz/tmpg7x1c33g TO /root/.ansible/tmp/ansible-tmp-1645442564.411589-129120-34397279681846/AnsiballZ_redfish_command.py
<<IP_ADDRESS>> EXEC /bin/sh -c 'chmod u+x /root/.ansible/tmp/ansible-tmp-1645442564.411589-129120-34397279681846/ /root/.ansible/tmp/ansible-tmp-1645442564.411589-129120-34397279681846/AnsiballZ_redfish_command.py && sleep 0'
<<IP_ADDRESS>> EXEC /bin/sh -c '/root/oneview/bin/python3.6 /root/.ansible/tmp/ansible-tmp-1645442564.411589-129120-34397279681846/AnsiballZ_redfish_command.py && sleep 0'
<<IP_ADDRESS>> EXEC /bin/sh -c 'rm -f -r /root/.ansible/tmp/ansible-tmp-1645442564.411589-129120-34397279681846/ > /dev/null 2>&1 && sleep 0'
[WARNING]: Module did not set no_log for account_password
fatal: [localhost]: FAILED! => {
"changed": false,
"invocation": {
"module_args": {
"account_password": "VALUE_SPECIFIED_IN_NO_LOG_PARAMETER",
"account_properties": {},
"account_username": "ckirim",
"auth_token": null,
"baseuri": "<IP_ADDRESS>",
"boot_next": null,
"boot_override_mode": null,
"bootdevice": null,
"category": "Accounts",
"command": [
"UpdateUserPassword"
],
"id": null,
"new_password": "VALUE_SPECIFIED_IN_NO_LOG_PARAMETER",
"new_username": "ckirim",
"password": "VALUE_SPECIFIED_IN_NO_LOG_PARAMETER",
"resource_id": null,
"roleid": null,
"session_uri": null,
"strip_etag_quotes": false,
"timeout": 10,
"uefi_target": null,
"update_creds": null,
"update_image_uri": null,
"update_protocol": null,
"update_targets": [],
"update_username": null,
"username": "ckirim",
"virtual_media": null
}
},
"msg": "HTTP Error 400 on PATCH request to 'https://<IP_ADDRESS>/redfish/v1/AccountService/Accounts/3', extended message: 'Bad Request'"
}
PLAY RECAP **********************************************************************************************************************************************************************************************************************************
localhost                  : ok=1    changed=0    unreachable=0    failed=1    skipped=0    rescued=0    ignored=0

I asked my internal team to check.

It is working for us. Please check the password length and other password requirements

thanks for the reply. Its working when I give 8 characters password.  :)

Great...Closing it


Search not working for versions 24.x and older [Documentation] [Bug]

Version
23.6.0
Steps to reproduce

Visit https://archive.jestjs.io/docs/en/23.x/getting-started.html
Type in search box
Press Enter

Expected behavior
Search results are shown
Actual behavior
Nothing happens, error logged to console
Additional context
No response
Environment
macOS Big Sur, Brave browser

@slorber is this something we can fix?

v24.x is archived (not easy to update now) and it's using Docusaurus v1: https://archive.jestjs.io/docs/en/24.x/getting-started.html
There has been an Algolia DocSearch migration, see:

https://v1.docusaurus.io/blog/2021/11/05/algolia-migration
https://docusaurus.io/blog/2021/11/21/algolia-docsearch-migration

Normally it should keep working, it's just that Algolia wouldn't update the existing indexes anymore for older deployments that do not migrate.

@shortcuts any idea why there's a 403 response on requests now?
React-Native is in a similar situation and the archived Docusaurus v1 still works fine: https://archive.reactnative.dev/

Can confirm it works now when deployed 👍


Proposal to eliminate warnings

Right now regexp_parser controlled code can generate warnings. On projects that have warnings enabled (such as mutant) and have the warning generating call side in the main loop: This can lead megabytes of warnings in the build logs.
This proposoal is about:

[ ] Getting agreement on removing all warnings from regexp_parser
[ ] Removing all warnings
[ ] Getting agreement on making the build fail in future in the presence of warnings
[ ] Enforce warnings are absent on the build

I'm happy to take 2. and 4. in case @ammar agrees to 1. and 4.

@mbj, I completely agree. Can you please point me to a reproducible case.

Reproducible case:

This build https://circleci.com/gh/mbj/mutant/2662
From this PR: https://github.com/mbj/mutant/pull/598
From this branch: https://github.com/mvz/mutant/tree/fix-timeout-example
From this ref: https://github.com/mvz/mutant/commit/45ff5713b85890be053efbeea709db1b9c44e1eb

So to reproduce:

Checkout mutant at that ref
run bundle install && bundle exec rspec spec/integration/mutant/corpus_spec.rb

As you asked for a reproducible case I gave you ^^. If you wanted I could get you a minimized reproduction that will likely not involve mutant. But for now I saved myself the time.
In theory you just have to enable warnings in your build to get all of them (assuming decent coverage).

Got it. That should be enough. Thanks.

@ammar Does this mean you want to eliminate warnings by yourself (and enforce they do not come back?) or you understand the case and want me to do it (as I offered)?

Found a simple way to reproduce thanks to @mbj hint:
ruby -w -Ilib test/test_all.rb

Not sure those are all the warnings, but that's a start.

@mbj, I'm happy to take it on.

Not sure those are all the warnings, but that's a start.

By default I fix all of them. Just for removing the need for whitelisting.

I'd be interested in integrating something like mutant's WarningFilter for the sake of capturing warnings emitted during tests so we can assert "tests did not emit errors"

@ammar If you plan to PR your fixes, I'd be happy to review them to make sure we get the most out of it.
The more devs look at stuff the better it gets. I do not want to "heckle" you in any way, but maybe we can catch a thing that would result in another iteration we can spare with a review, or more likely I ask a dump question on the code and with your reply learn something on the inner workings of regexp_parser.

@ammar can we invite you to the mutant slack channel? Should make communicating about this stuff a bit easier.

@backus Yes, please do.

@ammar invited (using your email from your github profile)

Fixed with the release of 0.3.6
Thanks @backus


Write Magnetometer Driver

Honeywell HMC5883L-TR

Progress started on branch magnetometer

If someone could look at the datasheet for this
http://www51.honeywell.com/aero/common/documents/myaerospacecatalog-documents/Defense_Brochures-documents/HMC5883L_3-Axis_Digital_Compass_IC.pdf
And tell me where I'm missing how to convert the hex values for the data registers (X, Y, and Z) into decimal values, I'd be really grateful. Because as far as I know, it doesn't specify what exactly the vales of -2048 to 2047 actually correspond to.
So while knowing that we're pointed at, for example,  (1568, 200, -987) is great, I have no idea what to do with that information to convert it into something meaningful, such as a heading.

Well the conversion itself is just converting from base 16 to base 10, no?
And then it looks like the three headings are just vectors of magnetic
field strength in each direction, which you can convert using some trig to
a theta angle for the XY plane, a phi angle for the XY/Z direction, and a
magnitude. Please correct me if I am wrong haha.
--Matan Silver
On Sun, Feb 14, 2016 at 6:38 PM, RamboRambowski<EMAIL_ADDRESS>wrote:

If someone could look at the datasheet for this
http://www51.honeywell.com/aero/common/documents/myaerospacecatalog-documents/Defense_Brochures-documents/HMC5883L_3-Axis_Digital_Compass_IC.pdf
And tell me where I'm missing how to convert the hex values for the data
registers (X, Y, and Z) into decimal values, I'd be really grateful.
Because as far as I know, it doesn't specify what exactly the vales of
-2048 to 2047 actually correspond to.
So while knowing that we're pointed at, for example, (1568, 200, -987) is
great, I have no idea what to do with that information to convert it into
something meaningful, such as a heading.
—
Reply to this email directly or view it on GitHub
https://github.com/ProjectKarman/avionics-firmware/issues/13#issuecomment-184006749
.

Actually the calculations for heading might be a bit more complicated,
since it is the rocket that is moving and the magnetic field that is
stable. So the vector of the sum of the three vectors that they give is the
vector of the magnetic field (North to South I think) relative to the
rocket. So if the magnetic field vector is 90 degrees clockwise of the
rocket, the rocket is 90 degrees counterclockwise of the magnetic field (so
going East to West). This gets a little fiddly in 3 dimensions though, I'm
not sure how it would go from there.
--Matan Silver
On Sun, Feb 14, 2016 at 7:00 PM, Matan Silver<EMAIL_ADDRESS>wrote:

Well the conversion itself is just converting from base 16 to base 10, no?
And then it looks like the three headings are just vectors of magnetic
field strength in each direction, which you can convert using some trig to
a theta angle for the XY plane, a phi angle for the XY/Z direction, and a
magnitude. Please correct me if I am wrong haha.
--Matan Silver
On Sun, Feb 14, 2016 at 6:38 PM, RamboRambowski<EMAIL_ADDRESS>wrote:

If someone could look at the datasheet for this
http://www51.honeywell.com/aero/common/documents/myaerospacecatalog-documents/Defense_Brochures-documents/HMC5883L_3-Axis_Digital_Compass_IC.pdf
And tell me where I'm missing how to convert the hex values for the data
registers (X, Y, and Z) into decimal values, I'd be really grateful.
Because as far as I know, it doesn't specify what exactly the vales of
-2048 to 2047 actually correspond to.
So while knowing that we're pointed at, for example, (1568, 200, -987) is
great, I have no idea what to do with that information to convert it into
something meaningful, such as a heading.
—
Reply to this email directly or view it on GitHub
https://github.com/ProjectKarman/avionics-firmware/issues/13#issuecomment-184006749
.

So the answer is ask someone who knows how magnetometers work how to use them? Calculating theta, phi and a magnitude doesn't sound very hard, the hard part is figuring out what they mean as far as where we are and where we're pointing.
I might throw an email to one of the software engineers at my last co-op and see if he can give me some intuition as far as this stuff is concerned.

Yes that sounds like a good plan, I'm sure they would give a much more detailed answer. Maybe contact the manufacturer too? There might be some details about what those values mean that only they would know.

Basically what we're doing is creating a homebrew Inertial measurement unit, right? Which means we're going to have our magnetometer and 3-axis accelerometer inside a gyroscope setup. And when we combine the measurements from both of these devices we'll be able to get our change in position, in the form of xyz coordinates, and our heading, as either yaw/pitch/roll or quaternions.
So from an architectural standpoint, should calculating our heading be the job of the driver, or the task? It would be really easy to just say okay, if you want to read the magnetometer, here you go have fun with these hex values. But if we wanted it to be on the driver level, we'd have to do those calculations before sending our heading back to the task for it to make decisions with.
Would it make sense to have a task dedicated specifically to the IMU?

I was under the impression we didn't need an IMU because GPS just gives the location--that way we don't have to worry about the inherent inaccuracies of integrating acceleration to get velocity, and velocity to get position (where errors will accumulate very quickly, especially at such high accelerations). From what I understand it really doesn't make sense to try to make our own IMU when we already have two redundant GPS modules. I think the magnetometer driver might only be needed to keep track of heading for the sake of logging data and making sure things are in order, but I might be wrong

I got an email back from the software engineer I messaged, and this is what he said:

Okay I see what you want.  With the magnetometer you can measure the X, Y and Z field strengths.  The X, Y and Z strengths in ECR coordinates can be found by running the world magnetic model, wmm.  Just google them they have free software in C for doing this calculation.   For starters you cannot get the absolute orientation from the magnotometer.  For example you can rotate your magnetometer about the measured field an nothing changes, just like accelerometers  you cannot get yaw because rotations about the z arises doesn't affect the readings.  But some things you can do with the magnotomete are calculate roll.   For example if the rocket is flying mostly straight up, the z axis, the earth magnetic field is mostly parallel to the ground. The the orientation of the field in the X, Y plane, atan2( y, x ) gives the roll of the rocket with respect to magnetic north.  This is often done to compensate for drift in the roll rate sensors. Because magnetometers are very hard to calibrate they are mostly used to track changes in roll and not so much absolute values.

So the World Magnetic Model is here https://www.ngdc.noaa.gov/geomag/WMM/soft.shtml
It would give us the magnetic field strength of the earth's magnetic field based on where we are (i.e lat/lon/alt). So using the magnetometer we could calculate the earth's magnetic field based on our gps coordinates, And do math on the earth's field compared to our readings to give us what our roll is.


[gp712] align with the new otPlatFlash API

above build failded due to
Err:1 http://azure.archive.ubuntu.com/ubuntu bionic/universe amd64 clang-tools-6.0 amd64 1:6.0-1ubuntu2
  Could not connect to azure.archive.ubuntu.com:80 (<IP_ADDRESS>), connection timed out [IP: <IP_ADDRESS> 80]
Err:2 http://azure.archive.ubuntu.com/ubuntu bionic-updates/universe amd64 clang-tools amd64 1:6.0-41~exp5~ubuntu1
  Unable to connect to azure.archive.ubuntu.com:http: [IP: <IP_ADDRESS> 80]
E: Failed to fetch http://azure.archive.ubuntu.com/ubuntu/pool/universe/l/llvm-toolchain-6.0/clang-tools-6.0_6.0-1ubuntu2_amd64.deb  Could not connect to azure.archive.ubuntu.com:80 (<IP_ADDRESS>), connection timed out [IP: <IP_ADDRESS> 80]
E: Failed to fetch http://azure.archive.ubuntu.com/ubuntu/pool/universe/l/llvm-defaults/clang-tools_6.0-41~exp5~ubuntu1_amd64.deb  Unable to connect to azure.archive.ubuntu.com:http: [IP: <IP_ADDRESS> 80]
E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?
##[error]Process completed with exit code 100.

This seems something outside of my control

@thcu-gp, the failure is certainly due to sudo apt install .... I have triggered a re-run of the jobs for you. For future reference, there should be a button in the upper right that allows you to re-run the jobs on the GitHub Actions page.

@jwhui thanks, I must have overlooked this button. Next time I'll look a bit closer


Improve C2751

Fix a few issues:

Missing /c flag
Adding definitions to the std namespace is UB
Superfluous C2955 (use of class template requires template argument list) error

Overall, the example is now slightly simpler.

#sign-off


Notify about a Tip in tip-bot matrix channel

Once a tip is made (either successfully or with error) notify tip-bot chat, so other tip approvers have a visibility what/where is being tipped, and we also monitor how tip bot functions

Related: https://github.com/paritytech/devops/issues/2598

Reopening because I didn't manage to add the new secret in a proper way and staging deployment failed.


Add constructions for Monads and Comonads

This commit adds a few constructions for monads and comonads. In particular, it adds the identity monad and comonad for a category. It also adds the identity and composite morphism between monads on the same category.

FYI: might be a few days before I can properly review this. Very welcome additions, I'm just facing a number of deadlines at the moment.

oops sorry for requesting review @Taneb  that was on accident

CI fails with
 Checking Categories.Monad.Strong (/home/runner/work/agda-categories/agda-categories/src/Categories/Monad/Strong.agda).
/home/runner/work/agda-categories/agda-categories/src/Categories/Monad/Strong.agda:47,43-45
Ambiguous name id. It could refer to any one of
  Categories.Monad.id bound at
    /home/runner/work/agda-categories/agda-categories/src/Categories/Monad.agda:35,3-5
  Categories.Monad.Strong.id bound at
    /home/runner/work/agda-categories/agda-categories/src/Categories/Category/Core.agda:23,5-7
id is in scope as
  * a defined name Categories.Monad._.id brought into scope by
    - the opening of Categories.Monad at /home/runner/work/agda-categories/agda-categories/src/Categories/Monad/Strong.agda:19,13-29
    - its definition at /home/runner/work/agda-categories/agda-categories/src/Categories/Monad.agda:35,3-5
  * a record field Categories.Monad.Strong._.id brought into scope by
    - the application of Category at /home/runner/work/agda-categories/agda-categories/src/Categories/Monad/Strong.agda:28,8-16
    - its definition at /home/runner/work/agda-categories/agda-categories/src/Categories/Category/Core.agda:23,5-7
when scope checking id

I ran the CI locally, it seems like everything should pass now


merge the example data into the website

maybe an issue for the main repo...

https://github.com/wc-guide/wc-guide-data/pull/12


Reduce heap allocations to zero

@lemire @CarlVerret Amazing work!! (both the algorithm and the C# port) I've made a couple of changes to reduce heap allocations (and made it zero-alloc basically) here are the benchmark results:
Was (see allocations in the last column):
|                             Method |      Mean |     Error |    StdDev |       Min | Ratio |     Gen 0 | Gen 1 | Gen 2 |  Allocated |
|----------------------------------- |----------:|----------:|----------:|----------:|------:|----------:|------:|------:|-----------:|
|               FastFloat.PaseDouble |  9.642 ms | 0.1912 ms | 0.2276 ms |  9.390 ms |  0.37 | 6179.6875 |     - |     - | 38803136 B |
|     'FastFloat.PaseDouble w/o <T>' |  9.738 ms | 0.1862 ms | 0.1742 ms |  9.508 ms |  0.38 | 6171.8750 |     - |     - | 38803136 B |
| 'FastFloat.PaseDouble - constants' |  9.851 ms | 0.1934 ms | 0.2070 ms |  9.487 ms |  0.38 | 6171.8750 |     - |     - | 38803136 B |
|                         'PNS only' |  2.708 ms | 0.0128 ms | 0.0107 ms |  2.692 ms |  0.10 |  707.0313 |     - |     - |  4445040 B |
|                     Double.Parse() | 25.843 ms | 0.0570 ms | 0.0476 ms | 25.774 ms |  1.00 |         - |     - |     - |          - |

This PR:
|                             Method |      Mean |     Error |    StdDev |       Min | Ratio | Gen 0 | Gen 1 | Gen 2 | Allocated |
|----------------------------------- |----------:|----------:|----------:|----------:|------:|------:|------:|------:|----------:|
|               FastFloat.PaseDouble |  4.875 ms | 0.0142 ms | 0.0126 ms |  4.854 ms |  0.19 |     - |     - |     - |         - |
|     'FastFloat.PaseDouble w/o <T>' |  4.871 ms | 0.0206 ms | 0.0172 ms |  4.854 ms |  0.19 |     - |     - |     - |         - |
| 'FastFloat.PaseDouble - constants' |  4.930 ms | 0.0422 ms | 0.0395 ms |  4.892 ms |  0.19 |     - |     - |     - |         - |
|                         'PNS only' |  2.610 ms | 0.0054 ms | 0.0042 ms |  2.603 ms |  0.10 |     - |     - |     - |         - |
|                     Double.Parse() | 25.714 ms | 0.2072 ms | 0.1938 ms | 25.498 ms |  1.00 |     - |     - |     - |         - |

Overall the algorithm is insanely fast! (5x faster???) I'll try to run all the tests we have in the dotnet/runtime for floating points this weekend

Note, that you can get slightly better codegen if you change it from IntPtr to nint. You also could switch back to using bool, as the optimization for ROS happens for any one byte type.
Here is a sharplab showing that:
https://sharplab.io/#v2:EYLgxg9gTgpgtADwGwBYA0AXEBDAzgWwB8ABAJgEYBYAKGIAYACY8gOgCUBXAOwwEt8YLAJI8YUCAAcAymIBuvMDFwBuGvSatOPfoIDCEfBN4AbMTKjzFKmmoDMTUg10MA3jQYeGvUVC7ZjGkgMwBAQAby4APq4EtiKkQDuvBgAFpHAAJ4YMJESGFAAFJnZDGAAlO6ebtSetQxsMNgAJgDyXMYZUrFcADzFMAB8DBjYwKYMALwMXDAJwVkwANoAuq6VdXV0aAxbO9u7B/tHewzk22en51cnhzfHt4frG54P928nF69331+/R0/PH7vP5A0Eg8FoAHPCFg4Fw2EIrZQjYw1HwtGw5GbdE4xG4tFY2oY4n40l7QkvMkkvE0pE1QGU2lU5l4ikeakclmvNlM3m/AC+qmoWP6wUmDAAqlxcNgAGaCACCTSaACEFi1ZbLcDAMAVYLKGABZGD4aAZQ3YKC4FL+FgAcR1DXlsC4igKIzGMDK2wKIgwAAV8mVykLAcQAOyS6VyxW4PoLbYhMIDPUwA3AMqhzz8p5PbzZXz+QLBULhKIxOI5JKpdILSJcfNFBalCr0jzVQENZptDpdbC9fpDD3jKYzOb9FZrNsMzkfa4XBdcuE82efJd8yHTwGzndLlfr3cb/cbk+74+Hi8488Hm9M6+n28ErfQx+v1nPlFvy+Yj/Yh98wUbF/UVgHFKUZXlFglVVdVNW1XV9SNE0zQtK0bWMe1HTTMQYFdGB3VGUxvQYAoGx4YNMyxCMowg2N42yRNSxTRCMyzDwc2nPMfD8AJmCCJMy2iWJ4mrNIBPrRtRXKJ4OzqLtWnaTpuj6JjhkImBxTHEswknWTallfxtW2AzjCMhgTLMiyYGMwzrPM2ybNMuyrMcsz8g4Oz3M8qAPO2Ly/J85yHPspzXKC0KQss4KXMi8KotCqEYqS6KUoi5K0uC/zYrCnLsry9L4sKuLitygqSsS1KitKyqSvymrqoyxqqrqprarKhqqoq1qOra+qWua9r+t67qhp6sbsq6ga+sGmbprmkbZoW+apu6ybhpWjb1q28bFs2nblvK59du20bTuO/alsuvazoctaLuu86bquk7Hte+q7qeh6Dvul7vs+37VqOv63uen6wf+8Hzo+kGvtBiH/sA59qPAmMoOVNVsg1LUdVTA1jVNKBzUta1bQdDAnRwvCCM9YjSPzCi2IYDj+SAA

better codegen if you change it from IntPtr to nint

Or by casting to uint first, to get rid of the sign-extenting movsxd. It's basically the same as using nint.

as the optimization for ROS happens for any one byte type.

Just for the record: C# doesn't apply this trick to types which are greater in size than 1, as endianess-concerns arise.

@gfoidl @mburbea One of you guys want to issue a pull request? It seems that it would save a move instruction.

Yeah, I can do (in a couple of hours).

I just knocked it out in #52.
Also, @gfoidl, while totally not supported you can use MemoryMarshal.Cast if you know your target audience is going to be using little endian. See this horrible hack I did here:
https://github.com/mburbea/koar-item-editor/blob/master/KoAR.Core/Hasher.cs#L10-L20


Refused to display 'PDF_URL' in a frame because it set 'X-Frame-Options' to 'sameorigin'.

Hello,
I have a problem with Pdfs.
Basically i need to open the pdf saved on pocketbase inside an iframe to make it viewable on my site.
I build the url of the pdf via an api call to poketbase using the name of the pdf generated by pocketbase and the id that the api returns to me. if I open the url directly from the browser it loads it correctly while if I try to view it inside an iframe it gives me the following error: Refused to display 'PDF_URL' in a frame because it set 'X-Frame-Options' to 'sameorigin'.
What can I do?
Thank you
I'm using PocketBase v0.7.7

This is to protect from clickjacking.
I'll have to think a little more on this. Maybe we can add an exception for the files API only.
For now, if you want to allow iframe embedding in external websites, you'll have to use a reverse proxy like nginx and change the X-Frame-Options response header from there (eg. via the add_header directive).

An exception for the files serving api was added in master. The fix will be included in the next minor bugfix release v0.7.8 (sometime next week).


[test] Run TypeScript module augmentation tests for Joy UI in CI

[x] I have followed (at least) the PR section of the contributing guide.

Module augmentation tests were not running in CI for Joy UI.
After adding the command in CI, there was a failing test so fixed it.

Awesome, thanks!


updated chaincodes for asset-transfer-basic in order to show good example on how achieving determinism over json

Show good praticse in order to make sure that data will be stored in the blockchain in a deterministic way using json
Signed-off-by: fraVlaca<EMAIL_ADDRESS>

Thank you @fraVlaca !  Could you also update the corresponding doc tutorial? I've created https://github.com/hyperledger/fabric/issues/2898 for it.


Added functionality to remove broken references

I've made a mistake and merged the branch direclty, ignoring this PR: Sorry :/


RAG.forward() missing 1 required positional argument: 'question'

Hi,
qa = RAG() pred = qa("xxx?") print(f"Predicted Answer: {pred.answer}")
works with the following, but compiling does not. Can anyone help?
from dspy.datasets import DataLoader dl = DataLoader() JR_qa = dl.from_csv("/content/ai_integration_qa_no_headers.csv") splits = dl.train_test_split(JR_qa, train_size=15) #dataset` is a List of dspy.Example
train_dataset = splits['train']
test_dataset = splits['test']
trainset = [x.with_inputs('question') for x in train_dataset]
devset = [x.with_inputs('question') for x in test_dataset]
class GenerateAnswer(dspy.Signature):
"""Answer questions with short factoid answers."""
context = dspy.InputField(desc="may contain relevant facts")
question = dspy.InputField()
answer = dspy.OutputField(desc="often between 1 and 5 words")

class RAG(dspy.Module):
def init(self, num_passages=3):
super().init()
    self.retrieve = dspy.Retrieve(k=num_passages)
    self.generate_answer = dspy.ChainOfThought(GenerateAnswer)

def forward(self, question):
    context = self.retrieve(question).passages
    prediction = self.generate_answer(context=context, question=question)
    return dspy.Prediction(context=context, answer=prediction.answer)

from dspy.teleprompt import BootstrapFewShot
Validation logic: check that the predicted answer is correct.
Also check that the retrieved context does actually contain that answer.
def validate_context_and_answer(example, pred, trace=None):
answer_EM = dspy.evaluate.answer_exact_match(example, pred)
answer_PM = dspy.evaluate.answer_passage_match(example, pred)
return answer_EM and answer_PM
Set up a basic teleprompter, which will compile our RAG program.
teleprompter = BootstrapFewShot(metric=validate_context_and_answer)
Compile!
compiled_rag = teleprompter.compile(RAG(), trainset=trainset)
`
——>
TypeError                                 Traceback (most recent call last)
 in <cell line: 14>()
12
13 # Compile!
---> 14 compiled_rag = teleprompter.compile(RAG(), trainset=trainset)
4 frames
/usr/local/lib/python3.10/dist-packages/dspy/primitives/program.py in call(self, *args, **kwargs)
24
25     def call(self, *args, **kwargs):
---> 26         return self.forward(*args, **kwargs)
27
28     def named_predictors(self):
TypeError: RAG.forward() missing 1 required positional argument: 'question'

Hi! I think you need to explicitly pass question, i.e.
qa = RAG()
answer = qa(question=“xxx?”).answer

Does this work for your case?

The RAG itself works:
qa = RAG() pred = qa("What is an AI dashboard?") print(f"Predicted Answer: {pred.answer}")
What does not work is compiling the module:
from dspy.teleprompt import BootstrapFewShot def validate_context_and_answer(example, pred, trace=None):     answer_EM = dspy.evaluate.answer_exact_match(example, pred)     answer_PM = dspy.evaluate.answer_passage_match(example, pred)     return answer_EM and answer_PM teleprompter = BootstrapFewShot(metric=validate_context_and_answer) compiled_rag = teleprompter.compile(RAG(), trainset=trainset)
Any idea why?
I am using the signature and module from the documentation (https://dspy-docs.vercel.app/docs/tutorials/rag).
What I changed is using my own article a dataset:
from dspy.datasets import DataLoader dl = DataLoader() qa = dl.from_csv("/content/ai_integration_qa.csv") splits = dl.train_test_split(qa, train_size=15)  train_dataset = splits['train'] test_dataset = splits['test'] trainset = [x.with_inputs('question') for x in train_dataset] devset = [x.with_inputs('question') for x in test_dataset]
I can't figure out what the issue is...

Hey @jeremierostan ! Do pass keyword arguments, like @mlederbauer said.

Passing keyword arguments means:
Do not write pred = qa("What is an AI dashboard?")
Instead, write pred = qa(question="What is an AI dashboard?")
Apply this in every call to a module. I hope this helps.

Hi, I had the same issue and the solution is to specify input_keys during loading the data, e.g.
data = dl.from_csv("data/dl.csv", input_keys=["question"])

Hi, I had the same issue and the solution is to specify input_keys during loading the data, e.g.
data = dl.from_csv("data/dl.csv", input_keys=["question"])


Optimize remote writes

This PR fixes some performance issues with the remote write path (used via cluster writes and hinted handoff).

Switch remote write marshaling to use a faster method to avoid parsing the line protocol format again on the remote side.
Avoid parsing the points multiple times on the receiving node.  The monitoring stats were inadvertently parsing the points again.
Optimize the Cache entry.add a bit to reduce lock contention with adding points.

These changes help remote cluster writes keep up with the coordinating node and avoid queuing them to hinted handoff.

One comment otherwise lgtm.


Add integration with Kafka

Use SASL_PLAINTEXT for auth to Kafka.
And setup trace context propagation

Hi,
I'm new to open source contributing and I'm interested in working on this issue. Is it still open for contributions?
Thanks,

Hi @botelhorui,
yes it's still open. Feel free to contribute.
I've updated description and added some points

Hi @botelhorui
Any updates on this?

Hi @botelhorui
Any updates on this?

Hi @mfvanek
Could you please reassign this task? Due to lack of time.


agent/core: Implement LFC-aware scaling

Part of #872.
This builds on the metrics that will be exposed by neondatabase/neon#8298.
For now, we only look at the working set size metrics over various time windows.
The algorithm is somewhat straightforward to implement (see wss.go), but unfortunately seems to be difficult to understand why it's expected to work.
See also: https://www.notion.so/neondatabase/874ef1cc942a4e6592434dbe9e609350

Notes for review:
Currently just a draft, pending validation that this does behave as expected.
Before merging, I want to:

Add some tests for LFC-based scaling to state_test.go's Test_DesiredResourcesFromMetricsOrRequestedUpscaling.
Add some more tests in wss_test.go
Add some more comments explaining some of the weirder things (the new algorithm, why desiredResourcesFromMetricsOrRequestedUpscaling now needs hasMetrics, etc.)

As a follow-up, I'd like to resurrect #737 to make this cleaner. I have a WIP branch for that locally; figured it'd be easier to get this PR in first, and then tidy up.
Also, for now this PR builds on #929 for consistency with what's deployed on staging.

Rebased onto main & squashed all commits -- merging left a lot of commits in the branch history.


Added Kie Navigator sections

@bbrodt Could you confirm that this section should also exist in the master branch?
Thanks!

@psiroky Please merge? Thanks!


Respect robots.txt

Most sites will also look at robots.txt before parsing meta tags, would be great if unfurl can take this into account too!

Unfurl itself isn't a crawler. I don't think this is in unfurl's remit really.


Create plugin to monitor VM "uptime"

Summary.QuickStats.UptimeSeconds

e.g., too long of an update means that a kernel update didn't install properly (they're usually released monthly)

Not sure if this is based on power state, or guest OS uptime. If the former, this might require setting a lengthy value in order to be useful. For example, the power state "uptime" for some VMs could be many months at a time if there isn't a hard requirement to shut it down. This is with regular maintenance, OS updates and reboots.
If the Summary.QuickStats.UptimeSeconds value is tied to a VM "reboot", then that will do nicely.

If the Summary.QuickStats.UptimeSeconds value is tied to a VM "reboot", then that will do nicely.

Brief testing appears to confirm that this is indeed linked to the power cycle for a VM and not the guest uptime.

Rename issue to better reflect the outcome. Not sure if there is integration via VMware Tools to query the Guest OS uptime, but if there is, that's likely to be a separate plugin.


fix #782

I can't think of any real reason not to do #782. In particular, the submodule complicates work on parallel feature branches that both modify vendor/.

I might be trigger-happy this week, but I'm just going to do this. We can revert it if it doesn't work out.


Change BytePatchSerializationPayload to work with BinaryWriter/Reader

This is an attempt to get rid of all those pesky byte[] allocations - from what I've seen in reference, it does not allocate internally, so we should be set.
One thing I wasn't certain of was keeping both serialization and deserialization in a single class - but man, having those together just feels so natural. I added a mode and a bunch of asserts though.

There, I've split it. The serialization API did get a little bit more complicated, though I have to admit it got a lot more flexible now. As a nice bonus, you still CAN pass the same class to the generic twice, so the old code could still work.


v4-support.jar seems empty

https://github.com/mit-cml/appinventor-sources/blob/master/appinventor/lib/android/support/support-v4-26.1.0.jar
This empty jar got only 1kb, and would eclipse classpath not work with this while some code references the v4 support classes (e.g. ActivityStarter requires FileProvider)

FileProvider is located in support-core-utils. If you are importing into Eclipse, you should include all of the JAR files in that directory to resolve imports.
Once upon a time, all of the support files were in support-v4, but Google decided to split them out into different parts so that different subsets could be included without including everything. I believe now it's just left to include the BuildConfig, which the other dependencies rely on.


visualstudio2017: defend against Chocolatey AutoUninstaller

When uninstalling the first VS 2017 product package installed on the machine, the Chocolatey AutoUninstaller will remove the VS Installer, removing all other VS 2017 products.
Until chocolatey/choco#1257 is resolved, the packages need to work around this behavior.

What are all of the other VS 2017 products?

Just trying to determine why if I uninstall VS 2017 that I would not want to remove everything?

Here is the full list of VS 2017 products:
Build Tools
Community
Enterprise
Feedback Client
Professional
Team Explorer
Test Agent
Test Controller
Test Professional
Each of these products can be installed and uninstalled independently, side-by-side with others. You can even install multiple instances of the same product in different directories (perhaps to test an untrustworthy extension?). None of the products appear in Programs and Features, only the Visual Studio Installer (which manages all instances of all products installed on the machine) does.
This differs radically from the installation model of VS 2015 and earlier, where generally there was only one instance of the given Visual Studio major version on the system (the "integrated shell") and installing various Visual Studio products either added functionality to that instance (for example, augmenting Professional by installing Test Professional) or replaced it (Enterprise).
With VS 2017, the user can, for example:

install VS 2017 Feedback Client        <-- this installs the VS Installer
install VS 2017 Community
install VS 2017 Professional
uninstall VS 2017 Community
install VS 2017 Test Controller
uninstall VS 2017 Feedback Client

Currently, during step 6) the AutoUninstaller will attempt to remove the VS Installer, while it clearly should be left intact (there are other VS products still installed: Professional and Feedback Client).

@heaths I've noticed your comment which seems to imply it is possible to install the VS Installer only, without installing any VS product. Could you please confirm that one could, for instance, download the Feedback Client bootstrapper and invoke it as such:
vs_FeedbackClient.exe --update --quiet

and it will install just the VS Installer (on a machine which does not contain any VS 2017 bits)?
(I don't see the --update switch mentioned in the documentation. There is the update command, but its description only mentions updating an already installed product.)
If there was a way (supported and/or documented) to install just the VS Installer, it would allow me to sidestep the issue with the over-eager Chocolatey AutoUninstaller - the visualstudio2017-installer package would actually install the Installer.

We are updating documentation currently and it should be out soon. The update command actually updates an instance. The bootstrapper passes most command line arguments through to the installer, but some like --update are handled explicitly.

Great! Does it support updating the installer to a specific version, perhaps by providing a specific, local channel manifest file? I'd like to achieve fully repeatable and deterministic installs, if possible.

It will update to the latest version across all channels.

Tested: --quiet --update works (--update --quiet did not - exit code 5005) - it installs the installer on the machine if it wasn't present before. However, it does not register with Programs and Features, so unfortunately it won't help with the AutoUninstaller problem (but it will help with update scenarios).

https://github.com/chocolatey/choco/issues/1257

@heaths I have been trying to update the VS Installer from an offline layout (prepared earlier using --layout). I have established, by trial and error, that the following example command line accomplishes this:
X:\Path\vs_FeedbackClient.exe --quiet --offline X:\Path\vs_installer.opc --update

Can you confirm this is the correct way? (I could not find this parameter documented anywhere.)

Why are you trying to update the OPC from an offline layout? We specifically designed the Visual Studio Installer to be "evergreen" so we can fix any issues necessary blocking updates. In fact, some product updates will require newer installers. You shouldn't try to control the version of the installer yourself.
Also, the --update --quiet ordering issue is known.

I need to do this on machines without Internet connectivity, because --update --quiet fails otherwise (it always tries to download the update from the net). And I would like the installer update to be a separate step from product update/installation, because I've seen cases where updating the installer as part of product update made the process unreliable (#7, #8).

@mikemarlowe, are these issues fixed in more recent builds? I thought they were.

Yes, updating your layout will update the version of the Visual Studio Installer it contains.
When you update your instance of Visual Studio that was installed from this layout, the Visual Studio Install will be updated.
https://docs.microsoft.com/en-us/visualstudio/install/update-a-network-installation-of-visual-studio
has more details and shows examples of the commands to run


Signal readiness to systemd when init is finished

This allows systemd services to set Type=notify so it knows when the proxy is ready to serve requests.
More information on this mechanism is here:
https://www.man7.org/linux/man-pages/man3/sd_notify.3.html

Codecov Report

Merging #2 (f14f55e) into main (fd0b784) will decrease coverage by 0.31%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main       #2      +/-   ##
==========================================
- Coverage   79.84%   79.53%   -0.32%
==========================================
  Files           9        9
  Lines         511      513       +2
==========================================
  Hits          408      408
- Misses         85       87       +2
  Partials       18       18

Impacted Files
Coverage Δ

pkg/runner/runner.go
46.23% <0.00%> (-1.02%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Not sure how to increase the test coverage for this change really.

looks good to me, will merge this.
thank you for your contribution @zonque


feat: compile-time arithmetic for constants

Feature Request
Describe the Feature Request
The Cairo compiler should be able to calculate the compile-time value of constants, to make it easier to write such constants, e.g.:
const some_value = (256 * 1080) & 0xfaff + 1234; // should become 14336
const some_power_of_2: felt252 = 2**128 - 1; // should become<PHONE_NUMBER>20938463463374607431768211455

The only operator not currently in Cairo is **, but I feel an 'integer power' operator is super convenient when doing blockchain development, so it seems like it should be implemented as well.
Describe Preferred Solution
The compiler should just compile that arithmetic expression into its proper value.
If the feature request is approved, would you be willing to submit a PR?
(Help can be provided if you need assistance submitting a PR)

[x] Yes
[ ] No

(I intend to provide a PR for this soon)


Odo Docker tests should run without a Kube config

It seems that the Odo integration tests for Docker run with a kube config (https://github.com/openshift/odo/blob/master/.travis.yml#L129):
   # Docker push target test command does not need a running cluster at all, however few test
   # scenario of docker devfile url testing needs only Kube config file. So the test has been
   # added here just to make sure docker devfile url command test gets a proper kube config file.
   # without creating a separate OpenShift cluster.

This is problematic, and since Odo's docker support does not require a Kube config, the Odo integration tests should not require it either.
Problems have arised in Odo's Docker support when there isn't a Kube config and we need to make sure that's covered:

https://github.com/openshift/odo/issues/3074
https://github.com/openshift/odo/issues/3038

/area devfile
/area local


Fix build errors

Was broken by #100.

This is weird; how did CI pass on #100? I thought I merged it when green

GitHub says "avsm merged commit 9619aaa into ocaml-multicore:main [...] 1 check was pending". I guess CI was being slow that day.


Add missing code block

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #50024

This change adds the missing lines under the solutions header in the Positive and Negative Lookahead README file in Regular Expressions.

Hey @Mervecelikkkk
Thank you for opening this pull request.
Tests do not depend on the sampleWord or result variables, they just use pwRegex.
This is a standard message notifying you that we've reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding.


Rename the in-band HubProtocol "negotiation" to "Handshake"

Right now, we have two things called Negotiation:

In the Sockets layer, the client sends an HTTP POST to [endpoint]/negotiate to establish a connection and get a list of available transports
In the SignalR layer, after that HTTP POST and establishing a connection using a transport, the client then sends a JSON payload { "protocol": "json" } (or messagepack) to set the hub protocol to use

This is confusing, there are two negotiates :). I propose renaming the second of these to "Handshaking".
Possibly incomplete list of things to rename:

[ ] Rename things in the spec
[ ] Rename Microsoft.AspNetCore.SignalR.Internal.Protocol.NegotationProtocol
[ ] Rename Microsoft.AspNetCore.SignalR.Internal.Protocol.NegotiationMessage
[ ] Rename Microsoft.AspNetCore.SignalR.HubConnectionContext.NegotiateAsync

FYI: Other than the spec, these are all mostly internal types

Maybe we should just do this as part of #1548

Closing this, we're handling it as part of #1548 so I don't want to track it twice.


Decompile docs should mention the new decompile command from within vscode

For instance: https://learn.microsoft.com/en-us/azure/azure-resource-manager/bicep/decompile and https://learn.microsoft.com/en-us/azure/azure-resource-manager/bicep/migrate
See https://github.com/Azure/bicep/pull/8804 - it's not merged yet but I expect it to be in the 0.12 release next week.

@mumian - do you have bandwidth for this?

I will take care of it.

PR created - https://github.com/MicrosoftDocs/azure-docs-pr/pull/218065


Storyboard reference bugfix

Ports SwinjectStoryboard #15 to v1.
Fixes #128, #125

Thank you for your great work to fix the issues👍👍👍


[v2int 4.3 port] ci: Pin version downloaded by npx lerna (#15922)

See https://github.com/microsoft/FluidFramework/pull/15922
Description
Several pipelines and tool/scripts in our repo still do npx lerna for some things. The release of Lerna 7.0.0 a few hours ago started causing issues in PR builds because of a breaking change dropping support for a property in our lerna.json files.
The issue might have been addressed by removing the property from that file too, since the new default behavior probably would do the right thing; but pinning package versions we "install dynamically" (npx) is still preferable and avoids any other surprises we might get from newer versions of Lerna.
Lerna ^5.6.2 is the last version specifier https://github.com/microsoft/FluidFramework/pull/15531 so seems like the right call here.

I think if you merge in the changes from #15963 then it'll unblock your CI run.


Locally bind to specific address via new LocalIpAddress property

As promised in discussion in https://github.com/robinrodricks/FluentFTP/issues/713
A PR with only *.cs files
Not sure it will build with all targets (net20 and such, who still uses that anyway ?)

Awesome, I'll check it out!

This does the same stuff [thing.DoSomethingAsync().Result and thing.DoSomething()]
It’s safe to revert
From: Robin Rodricks @.>
Sent: 04 May 2021 09:10
To: robinrodricks/FluentFTP @.>
Cc: David Denis @.>; Author @.>
Subject: Re: [robinrodricks/FluentFTP] Locally bind to specific address via new LocalIpAddress property (#719)
@robinrodricks commented on this pull request.

In FluentFTP/Streams/FtpSocketStream.cshttps://github.com/robinrodricks/FluentFTP/pull/719#discussion_r625554035:

@@ -815,7 +821,7 @@ public class FtpSocketStream : Stream, IDisposable {

           /// <param name="ipVersions">Internet Protocol versions to support during the connection phase</param>

           public void Connect(string host, int port, FtpIpVersion ipVersions) {

#if CORE

                 IPAddress[] addresses = Dns.GetHostAddressesAsync(host).Result;

So can I revert this or is it needed to make your functionality work? If it is needed I can use a flag (option) to let users enable/disable this? Or are you sure it won't break the existing functionality?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/robinrodricks/FluentFTP/pull/719#discussion_r625554035, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMB4IIQ7UPJ5PWNINB2Y7MDTL6MUVANCNFSM434XXJMQ.

Great, thanks.

Thanks for this!

My pleasure 😊
From: Robin Rodricks @.>
Sent: 04 May 2021 15:15
To: robinrodricks/FluentFTP @.>
Cc: David Denis @.>; Author @.>
Subject: Re: [robinrodricks/FluentFTP] Locally bind to specific address via new LocalIpAddress property (#719)
Thanks for this!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/robinrodricks/FluentFTP/pull/719#issuecomment-831932224, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMB4IIUWQWOPEPQMPARGHITTL7XNPANCNFSM434XXJMQ.

It's probable that without the proper DNS resolving for IPv6, people will have to use the IPv6 addresses both for local and remote :) Can you point me to the place where you do DNS resolving in the current code ? I'll try it

These are all I can find:

https://github.com/robinrodricks/FluentFTP/blob/master/FluentFTP/Streams/FtpSocketStream.cs#L824
https://github.com/robinrodricks/FluentFTP/blob/master/FluentFTP/Streams/FtpSocketStream.cs#L939

Added to docs.
https://github.com/robinrodricks/FluentFTP/wiki/FTP-Connection#socket-settings

Thanks a lot Robin 😊 Will give it a try. Keep you posted.
From: Robin Rodricks @.>
Sent: 05 May 2021 16:11
To: robinrodricks/FluentFTP @.>
Cc: David Denis @.>; Author @.>
Subject: Re: [robinrodricks/FluentFTP] Locally bind to specific address via new LocalIpAddress property (#719)
Released. I did some refactoring / cleanup too.
Can you test it and see that nothing is broken in your new functionality? Btw prop is now called SocketLocalIp and others with Socket prefix to keep the property naming structured.
https://www.nuget.org/packages/FluentFTP/34.0.0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/robinrodricks/FluentFTP/pull/719#issuecomment-832719620, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMB4IIR2ONNTK2HZVIEBXCDTMFGWHANCNFSM434XXJMQ.


Getting BadDeviceToken on remote server but works on local machine

I've configured my project for APNS.
The notification goes to the iPhone device with the device registration_id on the local machine, but when I try to do the same thing on remote server it will give this error in the admin panel (using "send test message from admin actions"): Some messages could not be processed: 'BadDeviceToken', when trying to the same on the server shell (./manage.py shell remote machine) I get the following traceback:
In [8]: device.send_message('yellow')
[14/Jul/2017 19:41:11] INFO [hyper.http20.connection:724] Received unhandled event <RemoteSettingsChanged changed_settings:{ChangedSetting(setting=SettingCodes.HEADER_TABLE_SIZE, original_value=4096, new_value=4096), ChangedSetting(setting=SettingCodes.MAX_CONCURRENT_STREAMS, original_value=None, new_value=1000), ChangedSetting(setting=SettingCodes._max_frame_size, original_value=16384, new_value=16384), ChangedSetting(setting=SettingCodes._max_header_list_size, original_value=None, new_value=8000)}>
[14/Jul/2017 19:41:11] INFO [apns2.client:178] Connected to APNs
[14/Jul/2017 19:41:11] INFO [hyper.http20.connection:724] Received unhandled event <SettingsAcknowledged changed_settings:{ChangedSetting(setting=SettingCodes.ENABLE_PUSH, original_value=1, new_value=0)}>
[14/Jul/2017 19:41:11] INFO [hyper.http20.connection:724] Received unhandled event <SettingsAcknowledged changed_settings:{}>
[14/Jul/2017 19:41:11] INFO [hyper.http20.connection:724] Received unhandled event <RemoteSettingsChanged changed_settings:{ChangedSetting(setting=SettingCodes.HEADER_TABLE_SIZE, original_value=4096, new_value=4096), ChangedSetting(setting=SettingCodes.MAX_CONCURRENT_STREAMS, original_value=1000, new_value=1000), ChangedSetting(setting=SettingCodes.INITIAL_WINDOW_SIZE, original_value=65535, new_value=65535), ChangedSetting(setting=SettingCodes._max_frame_size, original_value=16384, new_value=16384), ChangedSetting(setting=SettingCodes._max_header_list_size, original_value=8000, new_value=8000)}>
---------------------------------------------------------------------------
BadDeviceToken                            Traceback (most recent call last)
/app/.heroku/python/lib/python3.6/site-packages/push_notifications/apns.py in apns_send_message(registration_id, alert, application_id, certfile, **kwargs)
    112                         registration_id, alert, application_id=application_id,
--> 113                         certfile=certfile, **kwargs
    114                 )

/app/.heroku/python/lib/python3.6/site-packages/push_notifications/apns.py in _apns_send(registration_id, alert, batch, application_id, certfile, **kwargs)
     93                 get_manager().get_apns_topic(application_id=application_id),
---> 94                 **notification_kwargs
     95         )

/app/.heroku/python/lib/python3.6/site-packages/apns2/client.py in send_notification(self, token_hex, notification, topic, priority, expiration, collapse_id)
     55         if result != 'Success':
---> 56             raise exception_class_for_reason(result)
     57

BadDeviceToken:

During handling of the above exception, another exception occurred:

APNSServerError                           Traceback (most recent call last)
<ipython-input-8-5da7ea2c015f> in <module>()
----> 1 device.send_message('yellow')

/app/.heroku/python/lib/python3.6/site-packages/push_notifications/models.py in send_message(self, message, certfile, **kwargs)
    157                         alert=message,
    158                         application_id=self.application_id, certfile=certfile,
--> 159                         **kwargs
    160                 )
    161

/app/.heroku/python/lib/python3.6/site-packages/push_notifications/apns.py in apns_send_message(registration_id, alert, application_id, certfile, **kwargs)
    118                         device.active = False
    119                         device.save()
--> 120                 raise APNSServerError(status=reason_for_exception_class(apns2_exception.__class__))
    121
    122

APNSServerError: BadDeviceToken

I'm using Heroku.
Versions:

django-push-notifications==1.5.0
Django==1.10.7
Python 3.6.1

Hello @Alir3z4
I have encountered an issue very similar to what you describe when I have DEBUG=True on my local machine with the apple sandbox configured and DEBUG=False on the production server against the production apns api. In my situation we had to ensure the correct APNS cert and TOPIC were set in each environment and the device had the correct token registered.

@Alir3z4 Does your Django project remain DEBUG=True and your app built in release mode?
I was having same error. I fix it by adding APNS_USE_SANDBOX = False in settings.

I had DEBUG=False in both production and staging environment of the app, however it was DEBUG=True in development for sure.
I had the Topic set correctly as well, but that didn't work as well, tried many different settings but I wasn't able to get it working.
At the end we decided to get to Pusher.com for this, however I really wanted to get notifications working with Django Push Notifications.
However I still think I haven't tried all possible fixes for this scenario, at the moment I'm not working on that part of project and cannot confirm any suggested solutions can resolve the problem.
Please feel free to close this issue since.
@matthewh and @michaelluk thank you very much for the help.

@Alir3z4
Is your device token from the debug build or release build? Make sure the environments are synchronized.

I'm having same issue. It works on local machine (Mac OS Sierra) but doesn't work from remote instance on AWS EC2 (Ubuntu). Checked all the ports on EC2 and regenerated the certs multiple times. Still doesn't work.
In both cases DEBUG=True, APNS_USE_SANDBOX=True and topic set to app bundle-ID. Same settings on both systems using the development push certificate.  Not getting any errors. It just silently fails.
On a whim installed Houston CLI and used it to send a push using the very same dev certificate from EC2. It worked. Appears to be isolated to django-push-notifications.

Exactly the same issue here as well! Waiting for help.

raise APNSServerError(status=reason_for_exception_class(apns2_exception.class))
push_notifications.apns.APNSServerError: BadDeviceToken
Exactly the same issue here as well! Anything yet???.

I now seem to be getting this error for some users. I definitely have the topic set correctly, APNS_USE_SANDBOX = False and Debug = False. Is there anything else I can check?

The same error on Python 3.6 + App 1.6.1

Same error, Python 3.6.7 + django-push-notifications 1.6.1
/usr/local/lib/python3.6/site-packages/push_notifications/apns.py in apns_send_message(registration_id, alert, application_id, certfile, **kwargs)
    118                         device.active = False
    119                         device.save()
--> 120                 raise APNSServerError(status=reason_for_exception_class(apns2_exception.__class__))
    121
    122

APNSServerError: BadDeviceToken

Both on STAGING and PROD environment. Regenerated and tested our .pem certificate as per these instructions, made sure the device is posting a fresh token to the server, but still getting this error.
Has anyone figured this out yet?

For me I had the same issue, I was using token based authentication and the error was regarding device tokens. Make sure your application setup is correct. Also you can try with use_sandbox flag as false. Sometimes the apps even on the test env are getting the tokens from the live account

What if you try to send a notification to a registration_id which doesn't exist?

I got the same error, BadDeviceToken, when trying to send push notifications to ios devices (APNS). I thought registration_ids didn't exist or wasn't registered in firebase, but that's not true.
I've tried to send a test message thu Firebase to a workmate's device and the push notification was received.
So, the problem should be configuration's. Which I don't know how to solve...

I had this same issue and in my case, I was something like this:
[
...
"APNS_USE_SANDBOX": os.environ.get('APNS_USE_SANDBOX', False),
...
]

and the problem was that I was setting APNS_USE_SANDBOX to False in Heroku, and os.environ.get('APNS_USE_SANDBOX', False) was returning False, but False as a Boolean I guess.
I'm not a professional developer so I don't know whether django-push-notifications should handle this or it's a pretty standard thing.


WTL should not be installed in wtl subdirectory

I think it is wrong. Even in official samples WTL headers are included as:
#include <atlapp.h>

This issue hasn’t been updated in a year; if it is still an issue, please reopen this issue.


Permission window popup when open the app

Describe the bug
Permission window pops out periodically
To Reproduce
Steps to reproduce the behavior:

Open the app, it may pop out the window to install command line
When open the window again, it may pop out
If it does not pop out, wait for some time, open the window again, it will pop out the window

Expected behavior
For the first time, the window pops out, then no window should be popped out.
Screenshots

Logs
If applicable, attach log files by opening the OrbStack menu and clicking “Show Logs”, or email them for privacy.
Info report

OrbStack info:
  Version: 0.5.2 (50200)
  Commit: 1070de4aca19fab0c2c0b57d7c0bacca7812ca23 (v0.5.2)

System info:
  macOS: 13.2.1 (22D68)
  CPU: arm64, 10 cores
  CPU model: Apple M2 Pro

---------------- [ cut here ] ----------------

Please copy and paste the above information into your bug report.
Open an issue here: https://github.com/orbstack/orbstack/issues/new/choose

Additional context
Add any other context about the problem here.

Fixed for the next version, thanks for reporting.

Fix released in v0.6.0.

Newest version 0.8.1, Popup the same window when my computer restarts. Is it normal?

@kdrag0n

Newest version 0.8.1, Popup the same window when my computer restarts. Is it normal?

@chunyang-wen Yes, that's normal. We'll eventually add a helper tool to fix this.

Every day it asks for password to install a new helper tool. that's annoying.

@emersonbroga It won't ask again if you deny the prompt.


rename resource labels in config output

rather than label everywhere, rename to tableLabel, systemLabel, namespaceLabel
and also add namespaceLabel to each table and system output object

was going to do this for v2.1 release (multiple namespaces) but its quite a deep change and I was able to work around it in a few of the places I need the e.g. namespace label, so may punt on this for a little while
can still add this as a non-breaking thing while leaving label for backwards compat


the Havit components support support rtl?

it there be an rtl support for this component that will be grate

Duplicates #109.


Exception thrown when Cloud Repository not connected

When cloud repository is added to VBR Console but not connected(see image), the powershell script shows an exception, just before PRTG output.

Doesn't affect prtg output directly, but i assume this should be an error in PRTG.

Ausnahme beim Aufrufen von "SyncSpaceInfoToDb" mit 2 Argument(en):  "Failed to establish connect to the service provider"
In C:\Program Files (x86)\PRTG Network Monitor\Custom Sensors\EXEXML\PRTG-VeeamBRStats2.ps1:78 Zeichen:6
+                     [Veeam.Backup.Core.CBackupRepositoryEx]::SyncSpaceInfoToDb($r, $true)
+                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
    + FullyQualifiedErrorId : Exception

Don't know how(didn't find any way to) but perhasp you could check if the repository is connected.
Otherwise, maybe catch the exception and turn the sensor red?

I think the problem is not cloud Repo specific.
All unailable Repositories should be effected (Server disconnected for example ).
Have you tried a simple Try / Catch around '[Veeam.Backup.Core.CBackupRepositoryEx]::SyncSpaceInfoToDb($r, $true)'?
I will try to create a test scenario inside my Lab for that Situationen.

I'm not too familiar with powershell. Just tried to fiddle around to find the issue here.
I can test for you though, if needed.

Might you please check this Version:
https://github.com/mycloudrevolution/Advanced-PRTG-Sensors/pull/21
in Debug Mode you should see the message: "SyncSpaceInfoToDb Failed" but no error should occure.

Works as expected.


maven-compiler-plugin 3.8.1

 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.

Thankyou!

obsolete.


feat: custom error types.

Description
Add custom error types for all errors generated in the library.
Fix #160

@kubewarden/kubewarden-developers THIS IS NOT READY. I would like to refine the error names and refactor the tests to follow our latest standards. ;)

@kubewarden/kubewarden-developers THIS IS NOT READY. I would like to refine the error names and refactor the tests to follow our latest standards. ;)

@kubewarden/kubewarden-developers I've decided to mark this PR as ready for review now. I'll open another one fixing the tests. Just to avoid mixing two different fixes into a single PR

@kubewarden/kubewarden-developers I've decided to mark this PR as ready for review now. I'll open another one fixing the tests. Just to avoid mixing two different fixes in a single PR

From what I can see, only the Windows tests are broken and they are broken because of the changes introduced by this PR. Do not open a new PR, but rather fix the broken windows tests over there.
We can still review the PR in the meantime, just push the windows fixes into a dedicated commit.

@fabriziosestito I didn't know about this approach. That looks way better than the one we use inside of other crates we contribute to!
Sounds like have to do some refactoring elsewhere :)

@flavio we've decide to merge this because during the last daily call you did not seems against to it after discussing your last comment. Please, let us know if you want to change something. I'll be glad to do it.


Deprecate elevation enum in azure-maps-render package

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[v] The pull request does not introduce [breaking changes]
[v] CHANGELOG is updated for new features, bug fixes or other significant changes.
[v] I have read the contribution guidelines.

General Guidelines and Best Practices

[v] Title of the pull request is clear and informative.
[v] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-maps-render

/azp run python - maps - ci

/azp run python - azure-maps-render - ci

@tjprescott  and @annatisch , can you help to review this PR? Thank you so much!


movementX/Y work differently using Tauri

Currently, movementX/Y is being used in the handleMouseMove function. For a temporary fix, see https://stackoverflow.com/questions/7096921/html5-panning-based-on-mouse-movement.

Fixed with https://github.com/FHICT-S-Koen/info-map/commit/b429c01a6ba32bf76377091ae0b7a2e8eb4b14f3


docs: modifying v1 of docs to point to managed experience

Fixes #
Description
This PR adds a base for the readme, to point people towards the managed offering for an easy experience getting things setup and configured.
We still need to solve documentation for self-hosted Helm Chart experience.
Does this change impact docs?

[x] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

Release Note

@tallaxes do we want to include it in both the readme and the website? I guess we can wait for website pr to be merged before removing contribution experience section


Support FOR ALL TABLES in logical replication

RDS supports it https://docs.aws.amazon.com/dms/latest/sbs/chap-manageddatabases.postgresql-rds-postgresql-full-load-publisher.html#chap-manageddatabases.postgresql-rds-postgresql-full-load-publisher-replication and it is handy feature to have.
Postgres allows it only for superusers:
/* FOR ALL TABLES requires superuser */
if (stmt->for_all_tables && !superuser())
    ereport(ERROR,
            (errcode(ERRCODE_INSUFFICIENT_PRIVILEGE),
                errmsg("must be superuser to create FOR ALL TABLES publication")));

there is no specified reason for that, but looking down at the usage of this flag:
/* Associate objects with the publication. */
if (stmt->for_all_tables)
{
    /* Invalidate relcache so that publication info is rebuilt. */
    CacheInvalidateRelcacheAll();
}
else
{
    ObjectsInPublicationToOids(stmt->pubobjects, pstate, &relations,
                                &schemaidlist);

it looks like superuser requirement there because there is no good way to ensure that the current user has access to future tables. That way one can read some future table that they might not have access to.
In our setup it should be okay to change superuser requirement to neon_superuser group membership.

FYA, @vadim2404 @knizhnik @arssher @save-buffer

Should we also eventually move neon_superuser into being a builtin role? Instead of having compute_ctl create it?

created a PR for review for Pg16. Waiting for review

Will be a part of the next release

A member of neon_superuser can't create a publication for all tables w/o switching the role

This should begin working once Migrations are merged in. Migrations enable INHERIT on neon_superuser.

To check it today (migrations were merged)

Migrations have been enabled for everyone, so I will close this issue

It's released


refresh footer, swap logo color, adjust sitemap

Should we change the color of the last section? The dark blue background flows into the dark blue of the footer, and maybe looks like one giant footer? What do you think?

Yes. I believe design has a shade in the hifi designs in figma for the about us page that could work. I was going to add it to the theme and do a pass correcting this type of thing. I also plan to do another pass on the fonts as the hifi designs don't seem to use the figma theme that I used to build the code theme.


Add an option to create a separate Kafka topic per-thread to PepperBoxLoadGenerator

As part of enhancing PepperBoxLoadGenerator to suit some of my need in terms of test tools, I'd like to add a similar configuration option to that implemented in the PepperBox Java Sampler, where the caller can ask that a separate topic is created for each thread.
Partly to enable us to consume the data with the current Consumer code, we'll use the same naming convention where a suffix is added in the form .n e.g. mytopic.3 for the 4th thread (assuming we can obtain contiguous incremental thread numbers); otherwise we may have to rethink the approach.

Implemented in https://github.com/commercetest/pepper-box/commit/e5014d6958c675abd3abe81d480d5457b57785be and several related commits.


Feature: Google Fonts

In phpBB those are already served out of the box via CDN, great idea.

Up!

+1


Generator script not working

Trying to run the generator script and it doesn't work. I am using Azure openai/Ai search for index/code space for testing.
Please help if possible

[ x] bug report -> please search issues before submitting
[ ] feature request
[x ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

### Minimal steps to reproduce
>

### Any log messages given by the failure

>05:06:56 (INFO) scripts: Using Azure OpenAI Service with Azure Developer CLI Credential
05:06:56 (INFO) azure.identity._internal.decorators: AzureDeveloperCliCredential.get_token succeeded
05:06:56 (INFO) scripts: Using Azure Search Service with API Key from AZURE_SEARCH_KEY
05:06:56 (INFO) scripts: Generating 50 questions total, 5 per source, based on search results
05:06:56 (INFO) azure.core.pipeline.policies.http_logging_policy: Request URL: 'https://testservcie3.search.windows.net/indexes('index12')/docs/search.post.search?api-version=REDACTED'
Request method: 'POST'
Request headers:
    'Content-Type': 'application/json'
    'Content-Length': '27'
    'api-key': 'REDACTED'
    'Accept': 'application/json;odata.metadata=none'
    'x-ms-client-request-id': '822ae6a6-e5ae-11ee-8800-000d3acb1738'
    'User-Agent': 'azsdk-python-search-documents/11.4.0 Python/3.11.8 (Linux-6.2.0-1019-azure-x86_64-with-glibc2.31)'
A body is sent with the request
05:06:56 (INFO) azure.core.pipeline.policies.http_logging_policy: Response status: 200
Response headers:
    'Transfer-Encoding': 'chunked'
    'Content-Type': 'application/json; odata.metadata=none; odata.streaming=true; charset=utf-8'
    'Content-Encoding': 'REDACTED'
    'Vary': 'REDACTED'
    'Server': 'Microsoft-IIS/10.0'
    'Strict-Transport-Security': 'REDACTED'
    'Preference-Applied': 'REDACTED'
    'OData-Version': 'REDACTED'
    'request-id': '822ae6a6-e5ae-11ee-8800-000d3acb1738'
    'elapsed-time': 'REDACTED'
    'Date': 'Tue, 19 Mar 2024 05:06:56 GMT'
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/workspaces/ai-rag-chat-evaluator/scripts/__main__.py", line 6, in <module>
    app()
  File "/home/vscode/.local/lib/python3.11/site-packages/typer/main.py", line 328, in __call__
    raise e
  File "/home/vscode/.local/lib/python3.11/site-packages/typer/main.py", line 311, in __call__
    return get_command(self)(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/vscode/.local/lib/python3.11/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/vscode/.local/lib/python3.11/site-packages/typer/core.py", line 778, in main
    return _main(
           ^^^^^^
  File "/home/vscode/.local/lib/python3.11/site-packages/typer/core.py", line 216, in _main
    rv = self.invoke(ctx)
         ^^^^^^^^^^^^^^^^
  File "/home/vscode/.local/lib/python3.11/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/vscode/.local/lib/python3.11/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/vscode/.local/lib/python3.11/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/vscode/.local/lib/python3.11/site-packages/typer/main.py", line 683, in wrapper
    return callback(**use_params)  # type: ignore
           ^^^^^^^^^^^^^^^^^^^^^^
  File "/workspaces/ai-rag-chat-evaluator/scripts/cli.py", line 40, in generate
    generate_test_qa_data(
  File "/workspaces/ai-rag-chat-evaluator/scripts/generate.py", line 37, in generate_test_qa_data
    logger.info("Processing search document %s", doc["sourcepage"])

### Expected/desired behavior
>na

### OS and Version?
windows 10 VM

### Mention any other details that might be useful
na

Hm, it seems like the final error code is cut off, but I'm wondering if your search fields are different than its expecting. What fields are in your index?

Hi Pamela
Thanks for looking into this
So the final error code is : KeyError: 'sourcepage' .Sorry for missing this one.
Attaching the Azure ai search field screenshot. Does the generator script requires particular field values?

Thanks so much Pamela. It works now.

Open PR, to make the citation field name configurable: #96

Closing this issue now, thanks @jakeatmsft !


How to set up the run_in_iframe var

How do you send the allow iframe recording var to fullstory as explained in this tutorial: https://help.fullstory.com/using/recording-iframes ?

was this issue resolved? Im wondering the same thing.


Move to Java 8

From @ryantenney in #837

I keep been second guessing the decision to move to Java 8, but let's do it. We'll still have v3.2 for anyone stuck on Java <=7.
With regard to default methods on interfaces, I'd encourage you to not overuse them. I think that they should be generally avoided unless a) the interface is intended to be used as a lambda function, or b) an existing interface is being expanded and we need to maintain backwards compatibility.

This means we can move forward and make use of lambdas for gauges, and other benefits.

Now I notice the extra word in my post facepalm

Closed by #974


feat: Use aws_iam_role_policies_exclusive

Description

Moved inline_policy in aws_iam_role to aws_iam_role_policy and aws_iam_role_policies_exclusive as force_detach_policies is true.
Motivation and Context

inline_policy block in aws_iam_role  was deprecated since aws provider v5.68.0 as aws_iam_role_policies_exclusive was added.
https://github.com/hashicorp/terraform-provider-aws/pull/39203
https://github.com/hashicorp/terraform-provider-aws/releases/tag/v5.68.0
Breaking Changes

No.
This change doesn't remove existing inline policies in migration process. I checked using TF_LOG=debug and there was no aws api call to remove existing inline policies.
How Has This Been Tested?

[ ] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

Sorry, my local fork was too outdated.


Use macos-13 for x64 and macos-latest for arm64

Pull Request Description
GitHub made arm64 runners generally available and changed macos-latest label to point to them.
The runner architecture is coupled with GH-hosted runners OS version: macos-13 is the last one to run on x64.
Important Notes

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[ ] The documentation has been updated, if necessary.
[ ] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[ ] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[ ] Unit tests have been written where possible.
[ ] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

Deferring for now due to issues with npm install. Going forward with https://github.com/enso-org/enso/pull/9801


Paging And Sorting Error - Ajax Grid

Hi,
Followed your set up instructions for http://mvc6-grid.azurewebsites.net/Grid/Ajax?mainParam=mainValue
Getting a javascript error that is stopping the paging and sorting working.
It is calling back to the IndexGrid Action result but the UI is broken due to JS error.

VS2017
MVC .net core 2.0.3

Is your _IndexGrid partial only rendering grid content?

Thanks for getting back to me.
I had wrapped the Grid code in a simple .  No id's etc.
It is maybe a little fragile with regard finding the grid again?
Anyway, thanks muchly.

Above should say div

Yes, your _IndexGrid partial should only contain a grid, if you want to wrap them, you can do that before using the grid in the main view.
<div class="wrapper">
    @Html.AjaxGrid(Url.Action("IndexGrid"))
</div>


Question: Is it possible to combien Pulid + control net?

Title.
For even more precision.

Yes, it is possible.

Yes, it is possible.

Do we need to retrain the model ? If yes, how can we do that, i can't find the training code.

one more question, i want to modify your loss function and some of your network architecture, is it possible to do that without the training code ?


Develop Co-Sim Tab

This is a meta issue that should be closed once the co-sim tab is mostly working.

Closed in https://github.com/into-cps/intocps-ui/commit/8cba2a883675504228271107f619661379f382ed


Additional multiplayer support

From The_Zhad: "do you think it would be posible to upgrade the voting mod and add the interface to the mutiplayer lobby? As a host of the room you do have access to it as you the one to pick the song but if you join a lobby then you dont."
From Mondanzo: "what would also be cool if you could have it also in the multiplayer lobby maybe on the bottom with the preview or in the endscreen after a song if that's possible"
To do:

Add UI elements to bottom of multiplayer lobby screen as well
#10 handles endscreen voting

Finished with #12


[request] separate glob-cli from glob

I was investigating CommonJS <-> ESM interop issues on yarn v1 and found the CJS dependencies installed by glob > jackspeak > @isaacs/cliui as the cause of this. See this comment too: https://github.com/yargs/cliui/pull/139#issuecomment-1537061717
TBH, I had no idea that glob has its own built-in CLI, and my question is now why? We use glob rather as a library than as an executable.
I would suggest extracting the CLI into a separate package.

Yarn allegedly supports package aliases, but resolves deps incorrectly on update. This is a yarn bug. Use npm. It works. Or, blow away your yarn.lock and do a fresh install if you're really attached to yarn, as that seems to be a workaround people have found.
I'm not going to work around yarn bugs in my libraries when pnpm and npm can handle it just fine, sorry.

Removing yarn.lock and fresh install doesn't work.
Manually editing yarn.lock and adding name: string-width to the package would work, however I do not want to do that since the file is auto-generated.
Resolutions work fine with yarn, one could set jackspeak to resolve to a non-existing local path, or switch to the original cliui:
{
    "resolutions": {
        "jackspeak": "./node_modules/.cache/null",
        //  or:
        "@isaacs/cliui": <EMAIL_ADDRESS>    }
}

This would do the trick, however, since glob is a widely used package, it would be nice if we did not have to work around it, especially since the feature that causes the issue is not really a core feature and brings in a lot of other dependencies :)

I'm sorry to hear that deleting the yarn lock and reinstalling didn't fix the issue. I thought someone else had gotten it to work that way, I'll try to dig up the older issue when I'm at a full keyboard, maybe they have some other steps that they could share.
The added deps are tiny. If they were vendored, or if yarn didn't have this bug, you wouldn't notice the added disk usage.
I'm not going to stop using dependency aliasing when it's useful. As more parts of our ecosystem go esm-only, it's likely that those of us writing hybrid packages will need to use that workaround again in the future. And, I use jackspeak in every cli I write these days. So, it is almost certainly going to bite you again eventually.
In fact, it may have already, in a more insidious way; if the aliased deps in question were not esm-only, thus causing a crash when loaded, it's likely that your programs already just get the wrong versions of aliased deps, which can cause incorrect behavior in ways that are not as easy to detect early.
Your package manager is failing to resolve dependency contracts correctly, and putting the work on you to define resolutions manually. Use a package manager that can manage packages. I'm not trying to be harsh, but having spent over a decade working on npm, it's somewhat shocking to me that such an obvious bug has persisted for so long.


Revamp: Clean up Singer pages

@rwfeather - Does it make sense to split this into two issues?:

Singer content pages.
Singer tap/target detail pages.

The detail pages can be redirects to the main extractor/loader detail pages.
The content pages just need to exist and be moderately presentable from a CSS perspective (hopefully made easier by the Tailwind integration in #783).

Closing this issue to be replaced by more granular:

#792
#793


User should be presented with warning when using invalid names to export

Background
An issue was created (#385) to warn a user when they are using an invalid feature class name, but it only catches some of the cases.
Current behavior
If you enter a special character *,&,!,(,< in any portion of the name, you are presented with a warning.

If you start the name with a number, this is not allowed, you get an error instead of a warning.

Expected behavior
The user should the warning anytime the filename is invalid.
Additional information
This is found on ArcGIS Pro 2.4. beta 1
Distance and Direction build 68

[ ] Distance and Direction October Resources
[ ] \DefenseSolutions\DistanceAndDirection\October2019\Dev\108\
[ ] Military Tools for ArcGIS October Resources
[ ] \DefenseSolutions\MilitaryToolsforArcGIS\October2019\Dev\333\

doc QC'd
@dfoll can you confirm the build you tested on and add the verified and appropriate application version labels. Issue can be closed after that.


Withdraw Nonce fix

Withdraw Nonce fix

Hi I just tried the changes and I there is a extra } below the line
const nonce_s = web3.utils.padLeft(nonce.toString(16), pad)

Maybe you already saw it, but I left it because I got an error when trying the remix haha


Eigh gradients can be wrong

In a lot of cases, eigh doesn't return the correct gradients.
I attached a very simple example, comparing JAX gradients (incorrect ones) with autograd gradients (correct ones),
JAX gradients computation:
from jax import grad
import jax.numpy as jnp

a = jnp.array([[1.93865817, 0.35509264, 0.64405863, 1.3430815 , 0.92342772],
               [0.35509264, 1.30340895, 0.90821576, 0.7199631 , 1.49002679],
               [0.64405863, 0.90821576, 0.50616539, 0.97171981, 0.9621489 ],
               [1.3430815 , 0.7199631 , 0.97171981, 1.27540148, 1.44715998],
               [0.92342772, 1.49002679, 0.9621489 , 1.44715998, 1.75984414]])

def fun(x):
    e,v = jnp.linalg.eigh(x)
    return jnp.sum(jnp.abs(v))

diff_fun = (grad(fun))
print(diff_fun(a))

JAX Output:  (incorrect)
[[-0.33112606  0.32743236 -0.18718082  0.643335   -0.30847853]
[ 0.32743236  0.5745873  -0.07279176 -0.0815831  -0.50257957]
[-0.18718082 -0.07279176 -0.4819119   0.29376295  0.19632888]
[ 0.643335   -0.0815831   0.29376295 -0.4126568  -0.28805062]
[-0.30847853 -0.50257957  0.19632888 -0.28805062  0.65110785]]
Autograd gradients computation script:
import autograd.numpy as npa
from autograd import elementwise_grad as grad

a = npa.array([[1.93865817, 0.35509264, 0.64405863, 1.3430815 , 0.92342772],
               [0.35509264, 1.30340895, 0.90821576, 0.7199631 , 1.49002679],
               [0.64405863, 0.90821576, 0.50616539, 0.97171981, 0.9621489 ],
               [1.3430815 , 0.7199631 , 0.97171981, 1.27540148, 1.44715998],
               [0.92342772, 1.49002679, 0.9621489 , 1.44715998, 1.75984414]])
def fun(x):
    e,v = npa.linalg.eigh(x)
    return npa.sum(npa.abs(v))

diff_fun = (grad(fun))
print(diff_fun(a))

Autograd Output:  (correct)
[[-0.33112716  0.52218473  0.7270792   0.67123525 -1.13012818]
[ 0.13268013  0.57458894  1.47796036  0.09381337 -1.45197577]
[-1.10143499 -1.62353805 -0.48191193  0.63382524  1.86241553]
[ 0.61543807 -0.25697898 -0.04630763 -0.41265955  0.06441824]
[ 0.51316611  0.44681027 -1.46975969 -0.64051274  0.65110971]]
It's worth noting that the mistake maybe related exclusively to the eigenvectors. When using a primitive function that only uses the eigenvalues, JAX returns the correct gradients, for example:
def fun(x):
    e,v = jnp.linalg.eigh(x)
    return jnp.sum(jnp.abs(e))

Notable observation: the V-matrix in the eigenvalue decomposition is not unique because $VΛV^⊤ = (V D) Λ (VD)^⊤$ for any matrix $D=diag(±1)$.
Running your code, we can see that the output for Jax is
array([[ 0.07,  0.11, -0.49, -0.74,  0.44],
       [-0.44, -0.14, -0.57,  0.55,  0.4 ],
       [ 0.59, -0.72,  0.05,  0.11,  0.34],
       [-0.55, -0.21,  0.62, -0.2 ,  0.48],
       [ 0.38,  0.63,  0.23,  0.31,  0.55]], dtype=float32)

whereas for autograd it is
array([[-0.07,  0.11,  0.49, -0.74, -0.44],
       [ 0.44, -0.14,  0.57,  0.55, -0.4 ],
       [-0.59, -0.72, -0.05,  0.11, -0.34],
       [ 0.55, -0.21, -0.62, -0.2 , -0.48],
       [-0.38,  0.63, -0.23,  0.31, -0.55]])

I.e. $D = [-1, -1, -1, -1]$. Of course $‖V‖{1,1}=∑{ij}|V_{ij}|$ and the gradient should be identical either way.

So I guess there might be a sign mistake somewhere in https://github.com/google/jax/blob/d43cb36dae7e2f4cf734de29431cc371a5efeac5/jax/_src/lax/linalg.py#L706

I notice that the gradient output of Jax is just the symmetry part of that of autograd.
grad_jax = jnp.array([[-0.33112606, 0.32743236, -0.18718082, 0.643335, -0.30847853],
[ 0.32743236, 0.5745873, -0.07279176, -0.0815831, -0.50257957],
[-0.18718082, -0.07279176, -0.4819119, 0.29376295, 0.19632888],
[ 0.643335, -0.0815831, 0.29376295, -0.4126568, -0.28805062],
[-0.30847853, -0.50257957, 0.19632888, -0.28805062, 0.65110785]])

grad_autograd = jnp.array([[-0.33112716, 0.52218473, 0.7270792, 0.67123525, -1.13012818],
[ 0.13268013, 0.57458894, 1.47796036, 0.09381337, -1.45197577],
[-1.10143499, -1.62353805, -0.48191193, 0.63382524, 1.86241553],
[ 0.61543807, -0.25697898, -0.04630763, -0.41265955, 0.06441824],
[ 0.51316611, 0.44681027, -1.46975969, -0.64051274, 0.65110971]])

print(grad_jax - 0.5 * (grad_autograd + grad_autograd.T))

The output is
[[ 1.1026859e-06 -5.9604645e-08 -2.9504299e-06 -1.6689301e-06 2.4735928e-06]
 [-5.9604645e-08 -1.6689301e-06 -2.9280782e-06 -2.8312206e-07 3.2186508e-06]
 [-2.9504299e-06 -2.9280782e-06  2.9802322e-08  4.1425228e-06 9.5367432e-07]
 [-1.6689301e-06 -2.8312206e-07  4.1425228e-06  2.7418137e-06 -3.3676624e-06]
 [ 2.4735928e-06  3.2186508e-06  9.5367432e-07 -3.3676624e-06 -1.8477440e-06]]

Since the orginal matrix a to be diagonalized is symmetric, this discrepancy between Jax and autograd results is not really a bug. In practical applications, the symmetry property of a is usually guaranteed by some upstream computations, and whether the adjoint of a itself is symmetric or not would not affect those gradients of real interest.

Sorry, I haven't looked in detail, but could this issue be analogous to #10815? (See my comment there.)


Android

[x] Running on Android
[x] LGPL implications?
[x] Screen sync
[ ] Test Android TV AVD
[ ] In Play Store
[ ] Camera sync
[ ] SSDP detection of PC installs
[ ] Sync settings with PC
[ ] Run scenes, effect, etc. off PC

Please also take AndroidTV (Shield, Sony/Philips TVs) into consideration

Tested on Android TV... BRAVIA 2015 KDL-43W807C 6.0.1
No monitor detected in sync and when clicked start sync crashes.

@JurajNyiri is it connected to a bridge?

What version of Android is installed?
Any chance of getting logcat output when the crash happens? I've never tried this from Android TV, given that I don't have an Android TV device, but it should be like:
https://stackoverflow.com/questions/39834242/sony-bravia-android-tv-how-to-enable-retrieve-adb-logcat-logs
So far there's reports from Sony X900E, Nvidia Shield, and Fire TV working without a crash, and I've tested 5.1 with an emulator.

Yes it was linked ... and it was on 6.0.1 I got a prompt and ran an update so unfortunately I will not be able to provide log...
I updated it to 7.0... issue is gone, first it wasnt working - I started sync, it went to main menu automatically and did nothing else than changing color to blue and sitting on it... Then I let it stay in the app for a while and switched through about, sync and link pages until it started looping through colors under "monitor". Then I started sync and it works - not in YT but it works everywhere else I tried - menus, kodi, which is the main thing.
Awesome work, thank you! :)

@JurajNyiri did you ever find a solution for this?  I just installed this on my Sony Bravia TV and works until it crashes. When it’s working, it’s great!
I see that Philips released their own app today for Mac and Windows.  I may try that out tonight.

Folded into #126