Allow OPTIONS response on web actions before checking for authentication.

Description
This patch fixes a bug whereby webactions cannot respond to OPTIONS even for default CORS responses because the handler check for authentication first, before checking if the requests if an OPTIONS request.
Actions that opts for the default response are not affected - they are only invoked if the request is not an OPTIONS request as before. Actions which respond to OPTIONS explicitly will be invoked even if not authenticated yet but this should be acceptable since the action cannot respond to OPTIONS otherwise.
Related issue and scope

[x] I opened an issue to propose and discuss this change (#4879)

My changes affect the following components

[ ] API
[x] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[x] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[x] Bug fix (generally a non-breaking change which closes an issue).
[] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[x] I added tests to cover my changes.
[x] My changes require further changes to the documentation.
[x] I updated the documentation where necessary.

Thanks for catching this @rabbah !

Codecov Report

Merging #4882 into master will decrease coverage by 6.14%.
The diff coverage is 88.88%.

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


Show transaction messages as toasts

Hey there!
In this PR, we change the previous transaction messages as alerts to toasts, as present on the following video:

Note: the toasts are still not persistent between page changes.

 Improve transactions alerts with toasts


Unable to import jax v0.4.13 on Windows

Description
Thank you for preparing wheel files for Windows. The installation process is flawless, but I can't import jax.

The specified module could not be found, but xla_extension.pyd is located in the env\Lib\site-packages\jaxlib folder.

What jax/jaxlib version are you using?
jax v0.4.13, jaxlib v0.4.13,
Which accelerator(s) are you using?
CPU
Additional system info
Python 3.11.4, Windows 11
NVIDIA GPU info
No response

I suspect this StackOverflow answer is relevant: https://stackoverflow.com/questions/59330863/cant-import-dll-module-in-python/59333711#59333711
In short, Windows will only let you load DLLs from trusted locations, so you'll have to tell your system that this is a trusted location.

I can't reproduce this, so I can't debug it.
Are you able to successfully import other Python packages, e.g., NumPy?

The problem disappeared after installing Microsoft Visual C++ Redistributable.

I was able to import NumPy and SciPy (installed as jax dependencies in this case). Therefore, it may be a good idea to mention in the installation guide of Jax for Windows that you need to install Microsoft Visual C++ Redistributable in the version appropriate for your version of Python.
See:

Which Microsoft Visual C++ compiler to use with a specific Python version?
Microsoft Visual C++ Redistributable latest supported downloads

https://github.com/google/jax/pull/16683 adds a note to the installation instructions. Thanks for figuring out the problem! I'm sure this will save other people time in the future.


Fix typos: OrderedSet Documentation

This PR contains a small fix for a typo in OrderedSet Documentation

I found this typo while testing the OrdedSet Documentation - Set Operations

After fix:

Checklist

[x] I've read the Contribution Guidelines
[x] My contributions are licensed under the Swift license.
[x] I've followed the coding style of the rest of the project.
[x] I've added tests covering all new code paths my change adds to the project (if appropriate).
[x] I've added benchmarks covering new functionality (if appropriate).
[x] I've verified that my change does not break any existing tests or introduce unexplained benchmark regressions.
[x] I've updated the documentation if necessary.

@kati-kms Would you mind rebasing this PR on the release/1.0 branch?
I think it would make sense to ship it as part of swift-collections 1.0.6.
(If not, we can still land this; I'll just cherry pick it there myself.)

@swift-ci test

@lorentey
I've done rebasing this PR on the release/1.0 branch, so please check it!

Nice! Thank you 👍

@swift-ci test

The macOS failure is sadly expected, it's not related to this PR.


IGNITE-20906 Sql. Fix possibility to change column type according to IEP-108

AlterTableAlterColumnCommandValidationTest and CatalogManagerSelfTest main validation test, seems no need to copy all checks into integration one.
The Review Checklist

[ ] Formal criteria: TC status, codestyle, mandatory documentation. Also make sure to complete the following:
- There is a single JIRA ticket related to the pull request.
- The web-link to the pull request is attached to the JIRA ticket.
- The JIRA ticket has the Patch Available state.
- The description of the JIRA ticket explains WHAT was made, WHY and HOW.
- The pull request title is treated as the final commit message. The following pattern must be used: IGNITE-XXXX Change summary where XXXX - number of JIRA issue.
[ ] Design: new code conforms with the design principles of the components it is added to.
[ ] Patch quality: patch cannot be split into smaller pieces, its size must be reasonable.
[ ] Code quality: code is clean and readable, necessary developer documentation is added if needed.
[ ] Tests code quality: test set covers positive/negative scenarios, happy/edge cases. Tests are effective in terms of execution time and resources.

Notes

Apache Ignite Coding Guidelines

@AMashenkov i fix all your comments.


Ignore writeasync failures when sending to multiple connections

#1555

Should we log when a connection fails?

If it fails inside the state machine, the exception bubbles up to the caller via the WhenAll. We should do the same.

After the build is fixed...

🆙 📅


consider support for querying different geo_types at the same time

e.g., allow to query both the state CA and all its counties at the same time.
possible notation: extent the logic of geo_values or add a new generic geo parameter to support a colon syntax, similar to the signals parameter within the covidcast_meta option.
examples:

state:CA
state:* or just state
county:06002
`county:06* if we wanna support a prefix syntax, see #380
state:CA,county:06*

This is an excellent candidate improvement for after the Flask conversion -- we'll revisit it then.

This is an excellent candidate improvement for after the Flask conversion -- we'll revisit it then.

mostly done beside the cross geo type mappings -> https://github.com/cmu-delphi/delphi-epidata/issues/542


Bugfixes for loading sym files

allow comments in enum
allow big endian multiplexer
allow comment for multiplexer
allow hexadecimal display of variable
don't ignore byte order for variables which are displayed in hexadecimal

Coverage decreased (-3.6%) to 92.889% when pulling 0819eacee14155265d151b1c1f119bf3f0691297 on erzoe:bugfix-sym into f8fd300bd1a2d2d25e5ca4cd70b87c4c4b81f2f9 on eerimoq:master.


l0st n00b

I've never attempted anything like this before. Help?
Installed Docker and Git. Tried both cloning and building.
In OSX Terminal:
docker run -p 8888:8888 -t -i audiogram
gets me to
audiogram@6e5c1a77f2d7:~/audiogram$
What now?
Thanks!

You should run npm start.

<EMAIL_ADDRESS>start /home/audiogram/audiogram
npm run postinstall && bin/server

<EMAIL_ADDRESS>postinstall /home/audiogram/audiogram
mkdir -p editor/js && browserify client/index.js > editor/js/bundle.js

info: Listening on 8888

Now open a web browser and visit http://localhost:8888/.

YES!
Thank you, Noah. I can feel the eye rolls. I got so close!
Appreciate you and your work.

I've taken a couple of hours to try to figure this out, and am coming up short and have been reading threads that seem to be related but are over my head.
I've made some simple changes to themes.json. Different name, color and background image. Image is in the correct folder. Committed in GitHub and synced. This new theme is not showing up in the dropdown menu.
Thank you, again, for your help. I think this is the final thing I will need. I'm not going to be going deep in the code.

Never mind. We got it fixed! Thank you!


PoC functionalities

This closes #8, closes #9 and closes #10

Looks good. Pretty much like I would have liked to have it.
I only dont like the fact that the contract.json is now fixed in here, because if it changes in the other repo we need to manually update it. On the other hand that means that changes not immediately break things. We should probably think of a way of doing this also for the abi in the client lib and if we want to keep it that way or change it.


add ssl/tls support

Looks good so far. I'll check it out sometime next week.

@awillis could you release a new supermarket version with these changes included? Thank you! :+1:

Apologies for the delay, I posted a new release.

Cool, thanks!


Rework usb-hid-and-msc-support.robot

The problem you're addressing (if any)

[ ] USB001.001 USB devices detected in FW only works with PiKVM
[ ] https://github.com/Dasharo/open-source-firmware-validation/issues/87
[ ] https://github.com/Dasharo/open-source-firmware-validation/issues/206
[ ] https://github.com/Dasharo/open-source-firmware-validation/issues/279

Describe the solution you'd like
No response
Where is the value to a user, and who might that user be?
No response
Describe alternatives you've considered
No response
Additional context
No response

USB001.001 should work when this PR will be merged: https://github.com/Dasharo/open-source-firmware-validation/pull/351
Currently:
 robot -v snipeit:no -L TRACE -v rte_ip:<IP_ADDRESS> -v config:protectli-vp2410  dasharo-compatibility/usb-hid-and-msc-support.robot
==============================================================================
Usb-Hid-And-Msc-Support
==============================================================================
USB001.001 USB devices detected in FW :: Check whether USB devices... | FAIL |
Variable '${PIKVM_IP}' not found.
------------------------------------------------------------------------------
USB001.002 USB devices detected by OS (Ubuntu) :: Check whether th... ......
Checking if usbutils is installed...

Package usbutils is installed
USB001.002 USB devices detected by OS (Ubuntu) :: Check whether th... | FAIL |
'      bInterfaceClass         8 Mass Storage
      bInterfaceClass         9 Hub
      bInterfaceClass       224 Wireless
      bInterfaceClass       224 Wireless
      bInterfaceClass       224 Wireless
      bInterfaceClass       224 Wireless
      bInterfaceClass       224 Wireless
      bInterfaceClass       224 Wireless
      bInterfaceClass       224 Wireless
      bInterfaceClass       224 Wireless
      bInterfaceClass         9 Hub' does not contain 'Human Interface Device'
------------------------------------------------------------------------------
USB001.003 USB devices detected by OS (Windows) :: Check whether t... | SKIP |
USB001.003 not supported
------------------------------------------------------------------------------
USB002.001 USB keyboard detected in FW :: Check whether the extern... | PASS |
------------------------------------------------------------------------------
USB002.002 USB keyboard in OS (Ubuntu) :: Check whether the extern... | PASS |
------------------------------------------------------------------------------
USB002.003 USB keyboard in OS (Windows) :: Check whether the exter... | SKIP |
USB002.003 not supported
------------------------------------------------------------------------------
USB003.001 Upload 1GB file on USB storage (Ubuntu) :: Check whethe... | SKIP |
USB003.001 not supported
------------------------------------------------------------------------------
USB003.002 Upload 1GB file on USB storage (Windows) :: Check wheth... | SKIP |
USB003.002 not supported
------------------------------------------------------------------------------
Usb-Hid-And-Msc-Support                                               | FAIL |
8 tests, 2 passed, 2 failed, 4 skipped
==============================================================================

With my PR:
 robot -v snipeit:no -L TRACE -v rte_ip:<IP_ADDRESS> -v config:protectli-vp2410  dasharo-compatibility/usb-hid-and-msc-support.robot
==============================================================================
Usb-Hid-And-Msc-Support
==============================================================================
USB001.001 USB devices detected in FW :: Check whether USB devices... | PASS |
------------------------------------------------------------------------------
USB001.002 USB devices detected by OS (Ubuntu) :: Check whether th... ......
Checking if usbutils is installed...

Package usbutils is installed
USB001.002 USB devices detected by OS (Ubuntu) :: Check whether th... | PASS |
------------------------------------------------------------------------------
USB001.003 USB devices detected by OS (Windows) :: Check whether t... | SKIP |
USB001.003 not supported
------------------------------------------------------------------------------
USB002.001 USB keyboard detected in FW :: Check whether the extern... | SKIP |
USB002.001 not supported
------------------------------------------------------------------------------
USB002.002 USB keyboard in OS (Ubuntu) :: Check whether the extern... | SKIP |
USB002.002 not supported
------------------------------------------------------------------------------
USB002.003 USB keyboard in OS (Windows) :: Check whether the exter... | SKIP |
USB002.003 not supported
------------------------------------------------------------------------------
USB003.001 Upload 1GB file on USB storage (Ubuntu) :: Check whethe... | SKIP |
USB003.001 not supported
------------------------------------------------------------------------------
USB003.002 Upload 1GB file on USB storage (Windows) :: Check wheth... | SKIP |
USB003.002 not supported
------------------------------------------------------------------------------
Usb-Hid-And-Msc-Support                                               | PASS |
8 tests, 2 passed, 0 failed, 6 skipped
==============================================================================

2 failing tests now succeed and 2 succeeding tests are now skipped as on tested machine there was no keyboard.

Both should be now fixed in https://github.com/Dasharo/open-source-firmware-validation/pull/351:

USB001.001 USB devices detected in FW only works with PiKVM
https://github.com/Dasharo/open-source-firmware-validation/issues/87

From what I've tested

https://github.com/Dasharo/open-source-firmware-validation/issues/279

Should also be fixed (after merge)

usb-hid-and-msc-support.robot should still be reworked somehow, because currently I think for tests to complete successfully USB drive should have preferably one FAT32 partition that's a little over 2 GB in size (maybe a little over 1 GB would work but I'm not sure). This partition should also be "bootable" so USB is detected in Boot Menu
In my tests I prepared USB with gdisk, with partition set as EFI (type EF00) and then formatted with mkfs.fat -F 32 /dev/sdc1
ubuntu@3mdeb:~$ lsblk /dev/sdc -o +fstype
NAME   MAJ:MIN RM  SIZE RO TYPE MOUNTPOINTS FSTYPE
sdc      8:32   1 28,7G  0 disk
└─sdc1   8:33   1   10G  0 part             vfat

And then copied EFI directory from random /boot directory (left only Boot directory removed rest)
.
├── EFI
│   └── Boot
│       ├── bootx64.efi
│       ├── fbx64.efi
│       └── mmx64.efi
└── System Volume Information
    ├── IndexerVolumeGuid
    └── WPSettings.dat

3 directories, 5 files

I also had to set disk letter first time manually in Windows.


Faker Deterministic Random across multiple systems

Hi, first of all, thank you for this amazing gem. Unfortunately, I've run into an issue. In my project, I've set Faker::Config.random = Random.new(77). While the randomness is the same every time for me on my computer and my peer's individually, the randomness is not mutually the same.
For instance, the randomness generated by Faker::Number.number(1) on my computer is 4, 5, 6; and on my peer's computer is 1, 2, 3. All I'm trying to achieve is 4, 5, 6 on all the machines using the same seed.
How can I achieve this? If it can't be done using that, I'd really appreciate someone suggesting me a workaround.

My bad, I didn't run pull the faker related changes on the remote machine. Everything works like a charm!


Now all of our configurations are in a single file, utils/src/config.rs，It also contains a lot of parse_xxx, xxx_format, default_xxx methods, It's messy and difficult to manage. We need to organize these configurations into modules, each containing the configuration itself, its parse method, and default values.
In addition, the configuration can only be created through the new method, which is very inflexible, we need to refactor them use builder pattern,  and it should be easy to modify existing configurations

i want to try it

Due to my personal time constraints, I abandon this issue. I am very sorry.

Hey, since caicancai  is not working on this issue, i would like to try it out!

Hey @themanforfree , since caicancai is not working on this issue, i would like to try it out!

This issue has been assigned to you, thank you for your contribution

Hi, I would like to take a look at this issue. Refactoring seems like a good starting to get to understand the codebase.

@qstommyshu ，Harsh1s has initiate a pull request to solve this issue，can you try another good-first-issue? Thanks 😄

/assignme


[Bug] [process instances] can not stop spark task

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
while using the spark plugin，when click the stop button in the process instances list，the yarn application of spark task is still running.
What you expected to happen
when stop the spark process instance，I hope the yarn application of spark task is also be killed.
How to reproduce

Anything else
No response
Version
3.2.x
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Does the task be killed at ds?

yes, the task has been killed at ds.


Sometimes unit tests are dicovered sometimes no

Environment data
VS Code version:   1.19.0
Python Extension version:  Version 0.9.1
Python Version:   3.5
OS and version:  4.10.0-42-generic  6~16.04.1-Ubuntu SMP
Actual behavior
Sometimes tests are dicovered and some no. With the same code
Steps to reproduce:

Click in "Discover unit test"
Apears mesagge: "No tests discovered, please check the configuration settings for the tests."

Configurations:
{
"python.pythonPath": "${workspaceFolder}/venv/bin/python",
// tests options
"python.unitTest.pyTestEnabled": false,
"python.unitTest.unittestEnabled": true,
"python.unitTest.nosetestsEnabled": false
}
Folders:
rootprogram
--  djangoapp
----    tests
------      test.py
Where the file with tests is test.py
I know it's a django test. But if I change class for unittest.TestCase it doesn't discover the test either. Yesterday, it found them. Today when I opened the editor (before doing any change), not.

Please could you provide (upload or provide a link to the repo) a sample project that we could use to replicate this issue.
Is the test.py in the test directory?. Looks like your settings.json isn't configured correctly.

Well, I made a project with just the structure (at least the one for tests) of mine. If you need me to upload more code I can see what can I do. I can't public all the code.
https://github.com/Kurara/unittest_errors

How do you run these tests from the terminal? What's the command line?

python manage.py test --settings=settings.defaults
I didnt't add file manage.py to my project. I add it now. It should give you an error about DATABASE because I didn't configure it, but tests can be launch even if they have erros.
But I can launch them too with:
python -m unittest
if I configure first enviroment variable DJANGO_SETTINGS_MODULE (if not it gaves an error but execute the same)
Yesterday I could launch tests with your plugin, today not.
I would like to help you making a djangotest module for your visual code testing. I clone your(Microsoft) repository and I want to try. I even wrote you an email :P
But menawhile I would like to try with unittest

Closing in favor of #73


missleading err msg "Multiple users with that username returned. Please use 'cf curl' with specific origin instead"

Command
cf delete-user foo@bar
What occurred
I get output:
Error deleting user foo@bar
Multiple users with that username returned. Please use 'cf curl' with specific origin instead

What you expected to occur
I should get something more clear.
and the word origin in the err msg above really confused me.  It suggests something like  cf curl -X DELETE /v2/users/id?origin=ldap to me. but cc api does not support that origin param.
CLI Version
6.25.0
CC API Endpoint Version
(API version: 2.58.0)

Hi @syslxg
I understand the issue.
Have you figured out how to delete the user, and based on that, have a better suggestion for this message?
How about:
Error deleting user foo@bar
Multiple users with that username returned. Please use 'cf curl' to delete the user by guid instead.

I'd prefer not to include the exact UAA APIs here as with a new API version we would likely forget to update the message.

@dkoper , that is fine with me.

@syslxg Can you confirm whether this issue is resolved with our edge binary?
https://github.com/cloudfoundry/cli#edge-binaries

This turns out to be a wrong message to my test. cf delete-user != cf curl -X DELETE. The former will remove the specified user from both ccdb and uaa while the latter is merely an API call to cc. After a cf delete-user the login with that deleted account will report user not exist. But after a cf curl -X DELETE the account will still be able to login.

@wgu Ah yes, I see what you mean.
Any suggestion on what would be a useful message?
Also, why did you get into this situation in the first place? I remember discussing such a case (same username in different origins) with the UAA PM when we added the --origin flag and could not come up with a use case for people to create same-named users.

This actually was a wrong operation which led to the issue. And yes, I think the ultimate issue is we are allowing users from different "origin"s to have same name. Would that be necessary? To me the platform should allow only 1 idp (origin). But in the case of an external idp co-existing with the internal one, would it be acceptable to keep the username unique as well? If one sees a conflict when trying to create a same username, the option could be either change the name or deal with the existing one first. Agreed that I don't see a use case for a same name account to server different actual users.

Today's release (6.26.0) has the message that turns out not sufficient.
Whether users from different origins should be allowed to have the same name, or 1 external idp co-existing with UAA's (and if/how that is enforcible) is a question for the UAA PM to comment on. -> @sreetummidi?

Usernames can be duplicated in two origins. The combination of username and origin and username should be unique and is enforced by UAA

@jbpivotal & I iterated a few times over improving this error message, but couldn't find a simple way of explaining what to do. I felt any reference to uaac or cf curl would require detailed API documentation on how to use them.
What it boils down to though, is that the cf CLI (for now at least) does not have support for user deletion and role management for users that require their origin to be specified to disambiguate from other users with the same username.
We'll change the error message to simply say "The user exists in multiple origins.".

Addressed in cf CLI 6.33.0.


custome template does not apply

I used docker build --build-arg TEMPLATE="custom_nginx.template".but kong is still running with /usr/local/openresty/nginx/sbin/nginx -p /usr/local/kong -c nginx.conf instead with --nginx-conf=/custom_nginx.conf.
KONG_BASE image
kong:latest
build cmd
docker build . --build-arg PLUGINS="kong-plugin-url-rewrite" --tag "$IMAGE_NAME"
--build-arg TEMPLATE="custom_nginx.template"
can somebodys help me on it?

what docker file are you using?

what docker file are you using?

I just add a location block in the template
 location /robots.txt {
            return 200 'User-agent: *\nDisallow: /';
        }

but it did not work.I want to make kong it self as a nginx service,not only proxy server

@zffocussss so were you using https://github.com/Kong/docker-kong/tree/master/customize ?

possibly related: https://github.com/Kong/docker-kong/pull/331

@zffocussss so were you using https://github.com/Kong/docker-kong/tree/master/customize ?

yes

possibly related: #331

Thanks,I ever met this issue.apk update failed permission denied.but it is not related in this caes

My results (adding here as I go):
Configure the test setup:
docker network create kong-net
docker run -d --name kong-database \
               --network=kong-net \
               -p 5432:5432 \
               -e "POSTGRES_USER=kong" \
               -e "POSTGRES_DB=kong" \
               postgres:9.6
docker run --rm \
     --network=kong-net \
     -e "KONG_DATABASE=postgres" \
     -e "KONG_PG_HOST=kong-database" \
     kong:latest kong migrations bootstrap

created a file; templates/nginx150.conf and added a "--" (double dash) somewhere, to make it an invalid file.
Then I did:
docker run --rm \
     --network=kong-net \
     -e "KONG_DATABASE=postgres" \
     -e "KONG_PG_HOST=kong-database" \
     kong:1.5.0-alpine kong migrations reset -y && \
docker run --rm \
     --network=kong-net \
     -e "KONG_DATABASE=postgres" \
     -e "KONG_PG_HOST=kong-database" \
     kong:1.5.0-alpine kong migrations bootstrap && \
docker build . \
    --build-arg KONG_BASE=kong:1.5.0-alpine \
    --build-arg PLUGINS="kong-plugin-url-rewrite" \
    --tag "custom" \
    --build-arg TEMPLATE="templates/nginx150.conf" && \
docker run --rm \
     --network=kong-net \
     -e "KONG_DATABASE=postgres" \
     -e "KONG_PG_HOST=kong-database" \
    custom

The final output is the expected error:
Successfully tagged custom:latest
Error: could not prepare Kong prefix at /usr/local/kong: nginx configuration is invalid (exit code 1):
nginx: [emerg] unknown directive "--}" in /usr/local/kong/nginx.conf:14
nginx: configuration file /usr/local/kong/nginx.conf test failed

It fails on the extra "--" I injected, but this shows that it actually works on 1.5.0, since it was apparently trying to use my corrupted file

created a 2.0 template file; templates/nginx20.conf and added a "--" (double dash) somewhere, to make it an invalid file.
Then I did:
docker run --rm \
     --network=kong-net \
     -e "KONG_DATABASE=postgres" \
     -e "KONG_PG_HOST=kong-database" \
     kong:2.0.1-alpine kong migrations reset -y && \
docker run --rm \
     --network=kong-net \
     -e "KONG_DATABASE=postgres" \
     -e "KONG_PG_HOST=kong-database" \
     kong:2.0.1-alpine kong migrations bootstrap && \
docker build . \
    --build-arg KONG_BASE=kong:2.0.1-alpine \
    --build-arg PLUGINS="kong-plugin-url-rewrite" \
    --tag "custom" \
    --build-arg TEMPLATE="templates/nginx20.conf" && \
docker run --rm \
     --network=kong-net \
     -e "KONG_DATABASE=postgres" \
     -e "KONG_PG_HOST=kong-database" \
    custom

But this time the final result is Kong actually starting;
Successfully tagged custom:latest
2020/02/08 14:54:23 [notice] 1#0: using the "epoll" event method
2020/02/08 14:54:23 [notice] 1#0: openresty/<IP_ADDRESS>
2020/02/08 14:54:23 [notice] 1#0: built by gcc 9.2.0 (Alpine 9.2.0)
2020/02/08 14:54:23 [notice] 1#0: OS: Linux 4.19.76-linuxkit
2020/02/08 14:54:23 [notice] 1#0: getrlimit(RLIMIT_NOFILE): 1048576:1048576
2020/02/08 14:54:23 [notice] 1#0: start worker processes
2020/02/08 14:54:23 [notice] 1#0: start worker process 22
2020/02/08 14:54:23 [notice] 1#0: start worker process 23

inside the running container the files (custom_nginx.conf, and old_entrypoint.sh)are there, so everything seems in place:
/ $ ls /
bin                   etc                   mnt                   root                  sys
custom_nginx.conf     home                  old-entrypoint.sh     run                   tmp
dev                   lib                   opt                   sbin                  usr
docker-entrypoint.sh  media                 proc                  srv                   var
/ $

Checking the old-entrypoint.sh:
/ $ cat old-entrypoint.sh
#!/bin/sh
set -e

export KONG_NGINX_DAEMON=off

if [[ "$1" == "kong" ]]; then
  PREFIX=${KONG_PREFIX:=/usr/local/kong}

  if [[ "$2" == "docker-start" ]]; then
    kong prepare -p "$PREFIX"

    ln -sf /dev/stdout $PREFIX/logs/access.log
    ln -sf /dev/stdout $PREFIX/logs/admin_access.log
    ln -sf /dev/stderr $PREFIX/logs/error.log

    exec /usr/local/openresty/nginx/sbin/nginx \
      -p "$PREFIX" \
      -c nginx.conf
  fi
fi

exec "$@"
/ $

Shows the problem, since the commandline is no longer passed along, because this line lost the final part  "$@".

@hutchic  what is the format for the custom_nginx.template?

What do you mean exactly by "format" ?
The default templates are here: https://github.com/Kong/kong-ee/blob/next/2.1.0.x/kong/templates


How does Remote concept work(pagination, sorting, filter) ?

Describe the bug
A clear and concise description of what the bug is. In addition, please search issues before you open a report to make sure there's no any duplicated report
I have to integrate API in react and perform remote sorting, pagination, filter. But I am not getting any clarity in terms of execution. What steps should I do
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Screenshots
If applicable, add screenshots to help explain your problem.
codesandbox
Please give a simple and minimal example on https://codesandbox.io so that we can reproduce it easily and handle it effectively

I checked documentation but its not much helpful for me. So can you please help me

@anjaligupta1404 remote mode means for pagination, filter or sorting means you have to do the paging, filter and sorting in outside(In your codebase). here is an online example to show u how to use remote mode with sorting, and we also have enough online example for remote paging, filtering, searching etc. Please read it.
Anyway, I don't have too much time to teach u to write a remote mode code and I believe above example and docs should be enough. Please open another issue when u have any bug or issue. thanks


Update to /get-a-divorce tasklist

There have been some content updates to the tasklist that were missed.

@deborahchua non-blocking - might be good to write a brief description of this PR, maybe just 'updating content based on content team feedback' or something.


serve-laravel.sh No such file or directory

Hey everyone, I KNOW that this is not a laravel issue, but I looked up in homestead project and there was not place to post this question.
I am running into a problem that I have not been able to figure out yet. When I try to set a new project into my homestead machine, it says that the serve-laravel.sh is not located into the folder where it used to be.
Error
vagrant@homestead:~$ serve loans.app /home/vagrant/Code/loans
dos2unix: /vagrant/scripts/serve-laravel.sh: No such file or directory
dos2unix: Skipping /vagrant/scripts/serve-laravel.sh, not a regular file.
bash: /vagrant/scripts/serve-laravel.sh: No such file or directory
This is my homestead filesystem
Filesystem                   Type      Size  Used Avail Use% Mounted on
udev                         devtmpfs  988M  4.0K  988M   1% /dev
tmpfs                        tmpfs     201M  4.5M  196M   3% /run
/dev/mapper/vagrant--vg-root ext4       39G  3.4G   33G  10% /
none                         tmpfs     4.0K     0  4.0K   0% /sys/fs/cgroup
none                         tmpfs     5.0M     0  5.0M   0% /run/lock
none                         tmpfs    1001M  4.0K 1001M   1% /run/shm
none                         tmpfs     100M     0  100M   0% /run/user
/dev/sda1                    ext2      236M   46M  179M  21% /boot
none                         vboxsf    699G  378G  322G  54% /vagrant
none                         vboxsf    699G  378G  322G  54% /home/vagrant/Code
any suggestions or guide?
Thanks!

the command is not needed using the new version of homestead


Video thumbnail could not generated

On the blog page content, there is a video added. while creating an pdf using this plugin, the video thumbnail will be not shown on the generated PDF file.

Please provide concrete steps how to replicate this issue.


WIP: Clone projects on start

What does this PR do?
Implements cloning projects for any DevWorkspace by adding an init-container that parses the projects out of a DevWorkspaceTemplateSpec and clones each one.
There are two parts to this PR:

Automatically injecting an init container into the flattened DevWorkspace (via container component, apply command, and prestart event)
A standalone binary that reads a DevWorkspace and clones projects

The cloning binary supports most of the features for projects specified by git references:

Multiple projects
Multiple remotes in one project
Specific checkoutFrom for branch/tag/revision

PR is a WIP since I'm not sure if we want to keep the separate binary within this repo or create another one for it. A built image of the binary is available at quay.io/amisevsk/project-clone:dev, as is used by the updated deployment templates.
Building your own project-clone binary is done by executing
docker build -t <image:tag> -f ./project-clone/Dockerfile .

from the root of this repo.
What issues does this PR fix or reference?
Closes https://github.com/devfile/devworkspace-operator/issues/205
Is it tested? How?
Included is a sample file samples/plain.yaml which is a plain container with multiple projects that tests various features of Devfile 2.0 projects. Once running, you'll need to exec into the container and look at the filesystem to ensure everything is in place.
Note that this PR currently does not work with Theia due to https://github.com/eclipse/che/issues/19489
TODO before removing WIP tag:

[x] Decide on a quay repo to contain the project-clone image -- quay.io/devfile/project-clone
[x] Add building project-clone into default CI jobs -- done
[x] Cover project-clone in PR check (build image) -- done

Testing on OpenShift works via the DW:
kind: DevWorkspace
apiVersion: workspace.devfile.io/v1alpha2
metadata:
  name: theia
spec:
  started: true
  template:
    projects:
      - name: web-nodejs-sample
        git:
          remotes:
            origin: "https://github.com/che-samples/web-nodejs-sample.git"
    components:
      - name: theia
        plugin:
          uri: https://che-plugin-registry-main.surge.sh/v3/plugins/eclipse/che-theia/next/devfile.yaml
          components:
            - name: theia-ide
              container:
                image: quay.io/crw_pr/che-theia:1041
            - name: remote-runtime-injector
              container:
                image: quay.io/crw_pr/che-theia-endpoint-runtime-binary:1041
    commands:
      - id: say-hello
        exec:
          component: theia
          commandLine: echo "Hello from $(pwd)"
          workingDir: ${PROJECTS_ROOT}/project/app

Hello, what is not yet clear to me is when will be executed the logic that will analyze che-theia-plugins.yaml and .vscode/extensions.json file
if project cloning is happening at init container, it means that it's too late to analyse these files as templates are already there or it means we'll have a workspace restart if we update just after the DevWorkspace templates (or it would mean that dashboard would clone as well the files before ?)

Dashboard or any other clients, like chectl, won't do clonning IMHO

if project cloning is happening at init container, it means that it's too late to analyse these files as templates are already there or it means we'll have a workspace restart if we update just after the DevWorkspace templates (or it would mean that dashboard would clone as well the files before ?)

project clonning and resolving devworkspace are pretty different stuff.
Project clonning is pretty generic for DWO level,
but resolving DW according to file system, is really tight to supported editors and their plugins models.
There are multiple ways to go, for example:

Che creates DW with just DevWorkspace resolver component.
Projects are clonned as init container.
Resolver analyzes file system and update DW and related DWT.
DevWorkspace is restarted.
Theia with all plugins are run.

So, it means DW will be restarted, but the first run will be faster.
But not decision is taken at this point. It's just one of the possible ways.

if project cloning is happening at init container, it means that it's too late to analyse these files as templates are already there or it means we'll have a workspace restart if we update just after the DevWorkspace templates (or it would mean that dashboard would clone as well the files before ?)

This way of defining plugins will always require a workspace restart unless whatever creates the devworkspace does a clone and figures out the templates needed. It's not something we can handle in the Operator.

checkoutFrom needs fix but clonning and setting up remotes work fine, tested with theia-next and plain samples.

Commit 623e28214abf7cd08041576e79662ec07a628e5b adds CI functions for testing/pushing the project-clone image.
One thing I haven't done yet (and should probably be a separate PR) is to update the RELATED_IMAGE env var for project clone in releases (i.e. in make-release.sh) to match the released version. This is also not done for RELATED_IMAGE_devworkspace_webhook_server either, though.

/test v7-devworkspaces-operator-e2e

/test v7-devworkspace-happy-path


Add set operator to "Vehicle" for CurrentRPM

Would be great if a set operator for the RPM of a vehicle could be implemented, because then we could easily control vehicle speeds.

Just use Task.DriveTo with the speed you want. You can take a look into the code of my VAutodrive mod here:
https://bitbucket.org/CyronsTeam/vautodrive/src
Mod download here:
https://www.gta5-mods.com/scripts/vautodrive

Its alot deeper than that I'm afraid I tried adding a setter based of the current rpm getter, but it didnt force the car to accelerate etc

Since this seems to be impossible to add (if @zorg93 says it's hard to accomplish, then propably it's impossible :stuck_out_tongue_winking_eye: ), I will close this.

If I put my mind to it I could find out what needs to be set, however I have a feeling its not RPM that needs to be set, instead its accelerator pressure(what happens when you hold W)


Restructure api client with requester class

The requester class is meant to be shared across multiple classes
sharing credentials, impersonation, host, logger and request methods;
get, post, delete with said settings.

This structure should help implementing new request methods like:
vds = VdsApiV2()
vds.rois[1972].update(description='new description')

using the same credentials and impersonation shared across modules


Support table styles for export PDF

Enhancement
While testing the new alpha release for Jupyter books, we noticed unusual formatting issues when exporting tables to PDF. This problem doesn't occur in Sphinx, as it supports table styles for PDF exports.
To make it successful and migrate to v2, we really would love to see table styles coming to MySTMD for PDF exports.
P.S. The documentation said the following indeed: Styles are Only for HTML CSS styles are currently only used for HTML outputs and are not carried through to all export targets (e.g. LaTeX) and are primarily used for web.

Could you clarify what you specifically mean by table styles? i.e. colour, colspan, etc.
Could you also clarify the template that you're using for PDF exports (I'm curious to determine whether it is LaTeX or Typst). Thanks!

Oops, I could have been more specific. What we mostly look for is widths, heights, etc...
In v1, we use the --builder pdflatex. In v2, I guess it is pdf+tex. I also tried the one from the tutorial in: https://next.jupyterbook.org/start/export-pdfs

@agoose77 @Gijsreyn  we experience the same issue. Is this an issue in the underlying library or something in Jupyter-book / Mystmd?

We need more information here! PDF exports are performed using templates that generate typst or latex. Could you point to a reproduce that we can inspect (I.e a github repo)?
Thanks!


stacking multiple .acr for the same trigger no longer possible.

Windows 2.5.2 version
You used to be able to do multiple custom reactions for the same trigger and the bot would give a random response when the trigger was hit.
It still works, but you can't add more than one .acr for a trigger because the trigger activates instead of recognizing the .acr command.
For example:
.acr "@Saya" Leave me alone.

Response:
Leave me alone.

The bot no longer recognizes the second .acr, but instead activates the trigger

you must have .crca enabled on the first acr, disable it, add the new one and then re-enable it again

you problably has .crca enabled on the first acr, disable it, add the new one and then re-enable it again

you problably have .crca enabled on the first acr, disable it, add the new one and then re-enable it again

I do. But why would that cause a problem? It didn't do that in the last version.

Can confirm that that works. Disabling .crca on other reaction allows you to add a new reaction.

because acr will try to trigger first and crca now ignores special characters. wouldnt be a bad thing if nadeko were to check the prefix before triggering acr :smiley:

cool thanks.

Working as intended


Libjxl static build requires -DJPEGXL_ENABLE_SKCMS=false to link

Describe the bug
Unless -DJPEGXL_ENABLE_SKCMS=false is specified in a static build, a dependent application is not able to link due to unresolved skcms symbols, which appear to be in an uninstalled libskcms.a.  This includes when -DJPEGXL_BUNDLE_SKCMS=true.
To Reproduce
Steps to reproduce the behavior:
With libraries (libpng, libjpeg, liblcms2) built static in advance, this build recipe is used:
    cmake \
        -DCMAKE_INSTALL_PREFIX=$WORK \
        -DBUILD_TESTING=off \
        -DBUILD_SHARED_LIBS=false \
        -DCMAKE_BUILD_TYPE=Release \
        -DJPEGXL_ENABLE_BENCHMARK=false \
        -DJPEGXL_ENABLE_EXAMPLES=false \
        -DJPEGXL_ENABLE_FUZZERS=false \
        -DJPEGXL_ENABLE_JPEGLI=false \
        -DJPEGXL_ENABLE_MANPAGES=OFF \
        -DJPEGXL_ENABLE_SJPEG=false \
        -DJPEGXL_ENABLE_TOOLS=false \
        -DJPEGXL_ENABLE_VIEWERS=false \
        -DJPEGXL_ENABLE_JPEGLI=false \
        -DJPEGXL_FORCE_SYSTEM_LCMS2=false \
        -DCMAKE_C_FLAGS="-DHWY_DISABLED_TARGETS=HWY_SSSE3 ${CFLAGS} -fPIC" \
        -DCMAKE_CXX_FLAGS="-DHWY_DISABLED_TARGETS=HWY_SSSE3 ${CXXFLAGS} -fPIC" \
        "${SRC}/libjxl"

With the above, a spew of unresolved symbols are produced when performing a static link with a dependent application (GraphicsMagick) for purpose of oss-fuzz testing.    But this is not possible since the skcms symbols are not included in libjxl.a.
Environment

OS: Linux
Compiler version: clang version 15.0.0
CPU type: x86_64
cjxl/djxl version string: cjxl v0.10.0 733dfda [AVX3,AVX2,SSE4,SSE2]

Additional context
None

For me, it fails to link no matter if it's using lcms or skcms.
It just fails with different missing symbols
...
jxl_cms.cc:(.text+0x33a8): undefined reference to `cmsCloseProfile'
...

and so on.
As it stands right now, statically linking latest libjxl with build-in l/skcms is impossible.

It used to have the option to bundle skcms into libjxl.a.


Fix some typos in few *.md files

I have spoted some typos in few *.md files.

/ok-to-test
/approve

bot, add author to whitelist

Sorry @arkamar This now needs a rebase.

No problem. It was kind of straightforward anyway.

/lgtm


Server disconnected (code: 1006)

Hello,
I only use the GUI via web interface and I noticed today that the GUI shows me the error message above.
Deleting the container didn't work. I am on Debian Stretch. My Docker version is 17.09.0-ce.
JDownloader is still running because I have folderwatch configured and if I put a file in there it works.
docker inspect output
https://pastebin.com/y47mU9LV
docker logs output
https://pastebin.com/D67cStbq
processes
    1 root       0:00 s6-svscan -s -t0 /var/run/s6/services
   31 root       0:00 s6-supervise s6-fdholderd
  312 root       0:00 s6-supervise app
  313 root       0:00 s6-supervise xvfb
  314 root       0:00 s6-supervise x11vnc
  315 root       0:00 s6-supervise openbox
  317 app        0:00 {startapp.sh} /bin/sh /startapp.sh
  318 root       0:01 /usr/bin/Xvfb :0 -screen 0 1920x1080x24
  320 root       0:00 /usr/bin/x11vnc -display :0 -rfbport 5900 -rfbportv6 -1 -no6 -noipv6 -httpportv6 -1 -forever -desktop JDownloader 2 -cursor arrow -shared -httpdir /opt/novnc/ -nopw
  323 app        0:00 /usr/bin/openbox
  694 app        0:15 /opt/jre/bin/java -Dawt.useSystemAAFontSettings=gasp -Djava.awt.headless=false -jar JDownloader.jar
 4555 app        0:00 sleep 1
 4556 root       0:00 ps - A

It seems that there is no mapping for port 5900.  You can either add one (by adding something like -p 5900:5900 to the docker run command) or update your image to the latest version, which doesn't need the port 5900 to be mapped.

Added the port and pulled the newest image, still the same error.

Could you provide the docker command you use to start the container?
  Can you try to clear the cache and/or try another browser?

I feel dump now... thanks clearing the cache deleting cookies worked.

Hey no worries :)  Happy to see that it's now working.


DC MILO 0.2433.1

DC MILO 0.2433.1

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.59%. Comparing base (dd352e4) to head (5345de6).
Report is 15 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #756   +/-   ##
=======================================
  Coverage   90.59%   90.59%
=======================================
  Files          39       39
  Lines        4890     4892    +2
=======================================
+ Hits         4430     4432    +2
  Misses        460      460

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Failures when iterating over results

The "Rust by Example" book contains several examples where iterators are used over result values. Take for instance this code (actual example in that section):
// compile-flags: --edition 2018
// rmc-flags: --cbmc-args --unwind 4 --object-bits 9
#![allow(unused)]
pub fn main() {
    let strings = vec!["tofu", "93", "18"];
    let numbers: Vec<_> = strings
        .into_iter()
        .filter_map(|s| s.parse::<i32>().ok())
        .collect();
    println!("Results: {:?}", numbers);
}

This example does not successfully verify with RMC. Running this command
rmc src/test/dashboard/books/Rust\ by\ Example/Error\ handling/Iterating\ over\ Results/22.rs --cbmc-args --unwind 4 --object-bits 9 | grep FAIL

gets me these results:
[std::ptr::const_ptr::<impl *const T>::offset_from.pointer_primitives.12] line 391 pointer outside object bounds in POINTER_OBJECT(var_11): FAILURE
[std::ptr::const_ptr::<impl *const T>::offset_from.pointer_primitives.16] line 391 pointer outside object bounds in POINTER_OBJECT(var_12): FAILURE
[std::result::Result::<T, E>::ok.assertion.1] line 647 unreachable code: FAILURE
VERIFICATION FAILED

But the example runs fine in Rust, producing this output:
Results: [93, 18]

Kani still reports two spurious failures on this example"
Check 46: _ZN4core3num60_$LT$impl$u20$core..str..traits..FromStr$u20$for$u20$i32$GT$8from_str17h7557cdc4c649c89fE.assertion.1
	 - Status: FAILURE
	 - Description: "assertion false"

Check 64: std::result::Result::<T, E>::ok.assertion.1
	 - Status: FAILURE
	 - Description: "unreachable code"

I think this is a duplicate from #576. These are issues related to the fact that we are not able to link against std functions.

This issue has been resolved by #1785. The issue is covered by the test added in the same PR. https://github.com/model-checking/kani/blob/main/tests/kani/Strings/parse.rs


VMSS operation : How to scale?

Hi,
I'm searching for a mean to scale a VMSS with the sdk.
From az cli I can do this
az vmss scale --resource-group rg-infra-fc --name {vmss} --new-capacity {capacity}
The operation list seems to not have a scale "only" operation, and begin_update does not seems to ba able to change vmss capacity

Hi @lmussier - Thanks for opening an issue! Tagging people who can take a look at this.
In the meantime, would be able to provide the following information?

package and version that you are using

Hi @swathipil,
I'm doing this right now
async def scaleUp(amd64_tpm:bool, amd64:bool, arm64:bool) -> None:
    logging.info("Starting to scaleup")

    # See what we have
    vmss_client = ComputeManagementClient(credential=DefaultAzureCredential(), subscription_id=SUBSCRIPTION_ID)
    vmssList = vmss_client.virtual_machine_scale_sets.list(resource_group_name=VMSS_RESSOURCE_GROUP_NAME)

    new_scale = list()

    # Inspect every vmss to see if it is a target
    for vmss in vmssList:
        name = vmss.name
        tags = vmss.tags
        capacity = vmss.sku.capacity

        logging.info(f'Found : {name} tags [{tags}] current capacity {capacity}')
        if "ghe-label" in tags :
            ghe_label = vmss.tags["ghe-label"]
            logging.debug(f'Found ghe-label {ghe_label}')
            if ghe_label == "amd64" and amd64 :
                logging.debug(f'Ask for +1 on instance count for {name}')
                new_scale.append((name, capacity+1))

    await vmssScale(new_scale)

The scale function is this one
async def vmssScale(scale:list[tuple[str,int]]) -> None :
# https://stackoverflow.com/questions/54291010/python3-how-to-asyncio-gather-a-list-of-partial-functions

    processes = []
    for vmss, capacity in scale:
        logging.info(f'Change capacity of {vmss} to {capacity}')
        processes.append(asyncio.create_subprocess_shell(f'az vmss scale --resource-group rg-infra-fc --name {vmss} --new-capacity {capacity}',
                                                          stdout=asyncio.subprocess.PIPE,
                                                          stderr=asyncio.subprocess.PIPE))

    results = await asyncio.gather(*processes)

    for process in results:
        await process.wait()

So yes I'm using the management package but without specifying an api version.

Hi @lmussier Azure CLI bases on Python SDK so if you are familiar with CLI source code, it is very easy to find related SDK API. Here is the guideline https://github.com/Azure/azure-sdk-for-python/blob/main/doc/dev/find_sdk/find_sdk_in_cli_command.md which may help.

Hi @lmussier Azure CLI bases on Python SDK so if you are familiar with CLI source code, it is very easy to find related SDK API. Here is the guideline https://github.com/Azure/azure-sdk-for-python/blob/main/doc/dev/find_sdk/find_sdk_in_cli_command.md which may help.

This is better than the direct answer to my question!
A big thanks for this!


Arena and tournament

Can you add it to the backend to save points and add it to the gs thanks

its easy i got it

Can you send it dms?


how can i add case and skin in the game ?

thanks for your help

Currently the only way is to manually edit csgo_gc/inventory.txt. For example this is how you'd add the revolution case and its key:
"items"
{
	"3"
	{
		"inventory"		"1"
		"def_index"		"4880" // csgo/scripts/items/items_game.txt: "crate_community_32"
		"quantity"		"1"
		"level"			"1"
		"quality"		"4" // unique
		"flags"			"0"
		"origin"		"0"
		"rarity"		"1" // common
	}

	"4"
	{
		"inventory"		"1"
		"def_index"		"1393" // csgo/scripts/items/items_game.txt: "community_32 Key"
		"quantity"		"1"
		"level"			"1"
		"quality"		"4" // unique
		"flags"			"0"
		"origin"		"0"
		"rarity"		"1" // common
	}
}


Repeating notifications come at once only when Android app is opened

Hello Notifee team. We are facing some weird behaviour of notifications on some Android (possibly 12?) devices.
We are facing these problems.

samsung SM-G970F | Android 12

all repeating notifications come only when the app is opened and come at once

Vodafone  Prime, OnePlus X 1003 | Android 6.0.1

sometimes repeating notification comes (70%), sometimes they do not and sometimes the time is not correct, unfortunately, we didn't find any pattern

Xiaomi Redmi 4A | Android   7.1.2, Xiaomi Redmi Go | Android  8.1.0 , Galaxy A13 | Android 12

all OK

iOS

working like a charm

note: all tested devices has notifications enabled globally and also directly for the app
Maybe we do something wrong in the setting or code? Or do you have any suggestions on what can possibly cause the issue and how to overcome it? The main problem is, that notifications come only when the app is opened. Thank you for any help.
code we use to create notifications:
 scheduleNotification({
        fireDate: fireDate.getTime(),
        alertTitle: notificationTextData.title,
        alertBody: notificationTextData.body,
        userInfo: createNotificationMetadata(
          notification.id + '_' + day,
          notification.id,
          notification.route,
        ),
        repeatInterval: RepeatFrequency.WEEKLY,
      }),

import notifee, {
  AndroidImportance,
  AndroidLaunchActivityFlag,
  AndroidNotificationSetting,
  AndroidStyle,
  AndroidVisibility,
  AuthorizationStatus,
  Notification,
  RepeatFrequency,
  TimestampTrigger,
  TriggerType,
} from '@notifee/react-native';

export type NotificationDataT = {
  fireDate: number;
  alertTitle: string;
  userInfo: NotificationTypeT | NotificationMetadataT; //any additional metadata
  alertBody?: string;
  category?: string;
  repeatInterval?: RepeatFrequency;
  sendAnalytics?: () => void;
};

export const checkPermissions = async () => {
  const settings = await notifee.getNotificationSettings();
  return settings.authorizationStatus == AuthorizationStatus.AUTHORIZED;
};

export async function scheduleNotification(notificationData: NotificationDataT) {
  try {
    const permissions = await checkPermissions();
    if (permissions) {
      const channelId = await createDefaultNotificationChannel();
      const notification = makeNotification(notificationData, channelId);
      const trigger = makeNotificationTrigger(notificationData);
      const id = await notifee.createTriggerNotification(notification, trigger);
      await storeNotificationId(userInfo, id);
    }
  } catch (error) {
    ...
  }
}

function makeNotificationTrigger({
  fireDate,
  repeatInterval,
}: NotificationDataT): TimestampTrigger {
  return {
    alarmManager: {
      allowWhileIdle: true,
    },
    type: TriggerType.TIMESTAMP,
    timestamp: fireDate,
    repeatFrequency: repeatInterval,
  };
}

function makeNotification(
  { alertTitle, alertBody, userInfo }: NotificationDataT,
  channelId: string,
): Notification {
  return {
    title: alertTitle,
    body: alertBody,
    data: userInfo,
    android: {
      channelId,
      importance: AndroidImportance.HIGH,
      pressAction: {
        id: 'default',
        launchActivity: 'default',
        launchActivityFlags: [AndroidLaunchActivityFlag.SINGLE_TOP],
      },
      smallIcon: 'ic_small_icon',
      largeIcon: 'ic_launcher',
      style: alertBody
        ? {
            type: AndroidStyle.BIGTEXT,
            text: alertBody,
          }
        : undefined,
      showTimestamp: true,
      visibility: AndroidVisibility.PUBLIC,
      sound: 'default',
    },
    ios: {
      sound: 'default',
      foregroundPresentationOptions: {
        sound: true,
        alert: true,
        badge: true,
      },
    },
  };
}

function createDefaultNotificationChannel() {
  return notifee.createChannel({
    id: 'cfheroNotifications',
    name: 'cfheroNotifications',
    importance: AndroidImportance.HIGH,
    vibration: true,
    lights: true,
    sound: 'default',
  });
}

versions:
 "@notifee/react-native": "^5.3.0",
 "react-native": "0.64.2",

Hi @cervebar, thank you for the details!
Everything looks correct. Interesting that it's only some devices 🤔  I can double-check with an S21 device to see if I can reproduce the same issue.
It could be related to some battery optimization which varies a lot based on the model and manufacturer.
There's also a suggestion (https://github.com/invertase/notifee/issues/416) to use ELAPSED_REALTIME_WAKEUP when creating the notification, so maybe we could explore that more.

Thanks for the verification of the code (I was going crazy about what I'm doing wrong 🤪). Unfortunately, I have another device that has the issue with both - not coming notification (could there be some silent exception when creating?) and very delayed notifications (11 minutes delay) - Samsung A20 Android 11, the battery state as reported from our tester was about 30% to 50%.

hello @helenaford, may I have a question, I saw on #416 that you are working on the option for allowing ELAPSED_REALTIME_WAKEUP. Is there any possibility to know(be notified) once the new version with this feature will be released?
With Samsung SM-G970F, Android 12, api Level 31 we are facing the issue with notifications, which just come all at once and with a few hours delay.
As it seems we are currently not able to achieve working alarms with Notifee we probably will have to switch libraries 😢. But before such an unwanted step, I would like to at least try to ask, if there is a possibility that your team will release a new version with such feature?
thanks

@cervebar Please, where you able to get this issue for your project? Which library were you considering using to achieve working alarms? I have been stuck on this for 1 week and I'm just about going crazy.
Thanks in advance 😅

Unfortunately we didn't figure this out with Notifee, so we are planning to write own small native Android module (with AlarmManager and only this specific functionality not whole library), do the platform fork and for iOS keep this. 😢

Hi, we too have users who experience this issue, is there an update on this?

I just had same issue two years later, my android will not send notification in locked status, the notifications will all come suddenly when I unlock my phone. My notification is not repeat type but one-time notification.The notification body like this:
{
"notification":
{
"title": "time to take medications",
"id": "userDrug-07:35-20241028",
"body": "open app to take medications",
"android":
{
"importance": 3,
"groupSummary": false,
"colorized": false,
"pressAction":
{
"launchActivity": "default",
"launchActivityFlags":
[
1
],
"id": "default"
},
"lightUpScreen": false,
"loopSound": false,
"visibility": 0,
"circularLargeIcon": false,
"sound": "alarm_clock",
"asForegroundService": false,
"ongoing": false,
"showTimestamp": false,
"badgeIconType": 2,
"onlyAlertOnce": false,
"groupAlertBehavior": 0,
"showChronometer": false,
"channelId": "takeDrugReminderId",
"autoCancel": true,
"largeIcon": "splashscreen_image",
"localOnly": false,
"defaults":
[
-1
],
"chronometerDirection": "up",
"smallIcon": "ic_launcher"
}
},
"trigger":
{
"repeatFrequency": -1,
"timestamp":<PHONE_NUMBER>000,
"type": 0
}
}
My phone is Xiaomi Phone with android 10.


Deleting elements

The Popup-Menu that appears after clicking on a certain element should have the option to delete this element

Zeitaufwand: 2 Stunden
Punkte: 3


Unicode search support

While using my iOS Gurbani keyboard, I was unable to search.
I suspect this is down to the lack of support for the Gurmukhi/Punjabi unicode characters, and will probably be a useful thing to fix.

Perhaps for those on english keyboards, we could hotkey to the unicode characters? That way we can keep our "#" character for searching full words and other modifiers in the future.
Main gripe with unicode was fonts subjectively didn't look nice. Perhaps that's changed, I'll look into that again. This reminds me that sometimes people want to copy the shabad, so we could also add a "copy shabad" button in the controller.

Yeah, I think for English letters should continue to map to the correct Gurmukhi characters as they do now.
Copy is a good shout.
I haven't really looked into other fonts, but might have a quick glance for unicode alternatives

For dev, we've decided on ascii gurbani, unicode everything else. So now we have to go a bit in reverse and either map unicode to ascii in search (so people's unicode keyboards work on mobile) or keep the ascii to unicode conversion and convert sql query to ascii

@Harjot1Singh Be sure to double check if this is "done" after switching db

Fixed in dev. Kept the search box as Unicode input in case someone wanted to copy the input.


Consolidate duplicate search results

These are indistinguishable, flood the results, and will be confusing to users who see no reason to choose one over the other.

I wonder if we should expose Ops at all. If we do, we should improve the Ops results to give types on parameters, eh?

I wonder if we should expose Ops at all.

Ops are going to add a ton of functionality. I think we should definitely be including them.

If we do, we should improve the Ops results to give types on parameters, eh?

I agree, but this change would have to be made in imagej-ops (specifically in OpSearchResult)

I filed imagej/imagej-ops#643 and added it to the napari-imagej project board.

Thanks. This is also a problem in Fiji, so it is great to kill two birds with one stone!


Consensus lock out at 3+ nodes after constant transaction streaming

Describe the bug
Boot 3 nodes.
Attach a VM
Send transactions consistently (1/sec) to VM
After few minutes consensus will stop occurring
To Reproduce
Steps to reproduce the behavior:

Boot 3 nodes.
Attach a VM
Send transactions consistently (1/sec) to VM
After few minutes consensus will stop occurring

Expected behavior
Continue to reach consensus

outdated


Semicolons and GO added to syntax

Fixing syntax

@allanhirt : Thanks for your contribution! The author, @MashaMSFT, has been notified to review your proposed change.

#sign-off

@MikeRayMSFT: I'm sorry - only the author of this article, @MashaMSFT, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Error on installation - Process.php on line 283

Hello all,
I'm trying to install Pagekit on my server and Im getting this error when I change the debugger to true on config.php on the last step of installation as below

Warning:  proc_open(): CreateProcess failed, error code - 5 in E:\HostingSpaces\badjian\armanpaya
.net\wwwroot\pagekit\app\vendor\symfony\process\Process.php on line 283
{"status":"no-tables","message":"Unable to launch a new process."}
I really appreciate if you can help me solve this issue.
Thanks.

Please apply this Patch and report back
https://github.com/symfony/symfony/commit/e76561dbc0e9a93f38455a76603b1fa881edbbbb

The next Pagekit release (and current dev branch) comes with a reworked installer.
Please open a new issue if problems occur.


Harmonize style

Most of the code base already consistently uses the tab + comma-at-the-end style. This converts the rest.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #413   +/-   ##
=======================================
  Coverage   98.94%   98.94%
=======================================
  Files           6        6
  Lines         474      474
  Branches      145      145
=======================================
  Hits          469      469
  Misses          2        2
  Partials        3        3

Impacted Files
Coverage Δ

src/response.js
100% <ø> (ø)
:arrow_up:

src/body.js
100% <ø> (ø)
:arrow_up:

src/index.js
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Maybe add linting to CI to make sure it stays that way?

@gr2m See https://github.com/bitinn/node-fetch/issues/303. Let's do that in a separate step.

I am happy with this.


Enhancement | Autosave while creating an article

Where ?
/write page
What ?

When user is writing an article, the app should autosave the changes every 15 seconds onto the local Storage.
When user redirects to other pages and revisits the write page, they should be able to see the saved content.
When user logs out and logs in again, visits the write page, they should still be able to see the save content. (until the scope of local storage)
If user A's article is autosaved, user A logouts. And user B logins on the same window and visits the write page, they should not be able to see the saved content of user A on the screen.

@aadeshkulkarni Could you assign this to me?

#114 closes this

Optmization note: An alternate approach could have been to perform throttling on the title & content onChangeHandlers with a delay of 5-10s.


30% chance of modifying response for /ask

This PR makes it so there's a 30% chance that the bot's response for the /ask command is modified. This is an important capability that Kyoyobot must have if it is to compete with LLMs.

wondewfuw, thankies vewy much! uwu


about MSCOCO

Nice work!For the past few days, I have read your paper and codes, but I still quite confused about how to apply your method(or uncertainy) to MSCOCO which is image-based instead of vedio, would you be kind to share the details or provide with the code in MSCOCO.My email address is<EMAIL_ADDRESS>

Hi, thanks for your interest and patient waiting. The implementation of Image-text retrieval has been updated. You can find it in PAU/Image-Text-Retrieval.

Hi, thanks for your interest and patient waiting. The implementation of Image-text retrieval has been updated. You can find it in PAU/Image-Text-Retrieval.

Thank you sooooooo much !

😄


Rendered Font does not face in the forward vector

Font is rendered backwards in regard to its transform forward vector direction.
Might be how it's implemented in the shader?

I just looked into this. This might be what we want. For scripts like Billboard and TapToPlace the rotation of the object is based off the rotation of the camera, so when the camera looks at the object it will be looking at it from behind. For the text to be readable we want to face in the opposite direction of the forward vector.

I disagree. Most likely the text will be a child of the GameObjects that have the scripts on them in the first place.  Generally you'd want things to be facing in the forward direction, so if you need to do some maths on said object, you don't need to guess which direction it's facing.

I'm going to go ahead and close this.  I rewrote the shader once and couldn't get a good way of doing it without breaking stuff.


[multiple repos] DPL-911 Correct Study and Project for cell extraction work

User story
As a user of the scRNA Core Cell Extraction pipeline, I would like the correct Study and Project to be used for the initial automated submission, for the cell extraction (Phase 1) work.
Who are the primary contacts for this story
Katy, Lynda, Rich Clark, Liz Huckle, Sameena
Who is the nominated tester for UAT
It's not a very visible change so tricky for users to test - I think we can just test it within PSD.
Acceptance criteria
To be considered successful the solution must allow:

[ ] Study should come from the one specified in the sample manifest - use existing auto-detection functionality
[ ] Project using the Sample Prep team cost code should be hardcoded against the submission template - this Project has been set up in prod already (Project ID: 3834, Project name: S0842)

Additional (technical) context
Currently, you can ask Sequencescape to 'autodetect' which Study and Project to use:
- set flag autodetect_studies_projects to true in purpose config, as done here
- this will be used in multi_stamp_tubes to pass the flag to Sequencescape via the API
Or, you can hardcode the Study and Project against the submission template, as done here
However, you can't do one option for the Study and the other option for the Project - this is what needs to be changed.

Been thinking about possible solutions (MVP and future) around automated submissions and recording cost codes. Recording thoughts here, as somewhere to put them:

Solution
Pros
Cons
Comments

Automated submission using Sample Prep cost code. Faculty cost code included in RT information (maybe manifest ‘cohort’ column).
Not much dev work. Blood can be processed with no delay.
Cross-charging required. SSRs need to generate report and it will be in a non-standard way. Use of ‘cohort’ field not good design.
Do PSD need to generate a custom report to enable cross-charging?

Automated submission, Sample Prep lab staff enters cost on on screen when scanning tubes into plate.
Not much dev work (bit more than above).
Lab staff won’t know the cost code. Could be multiple cost codes among the group of tubes.
Not feasible due to cons listed.

Manual submission by SSRs.
Consistent with normal process. Supports multiple cost codes. Could use faculty cost codes directly if desired - no cross-charging.
Too slow as blood must be processed same-day. Sample arraying feature would need to be re-designed if there was a pre-existing submission.
Not feasible due to delay in sample processing.

Manual submission by BioResource team.
Supports multiple cost codes. Could use faculty cost codes directly if desired - no cross-charging.
Extra admin for BioResource team. Sample arraying feature would need to be re-designed if there was a pre-existing submission.
Not desirable due to cons listed.

Manual submission by BioResource team - new tool combining manifest upload & submission with minimum clicks.
Supports multiple cost codes. Could use faculty cost codes directly if desired - no cross-charging. Could support strategy of moving away from Sequencescape. Enables customer self-service.
Sample arraying feature would need to be re-designed if there was a pre-existing submission. Lots of dev work to make new tool. Doesn’t fit with desired future integration with Titian, for automatic sample import?
Not feasible for MVP due to lots of dev work? Could be future improvement.

Automated sample import and submission - integration with Titian.
Best user experience.
Lots of design and dev work.
Not feasible for MVP due to lots of dev work.

Additional (non-technical) context
This approach follows their normal process as closely as possible. The Sample Prep team normally put their work under their own cost code initially, and then the SSRs and Fran prepare a cross-charging report, which charges the work retrospectively to the relevant faculty team.
The discussion around this story was because a) it's using Limber, which the Sample Prep team don't normally use, and b) it's using an automated submission, meaning there is no way for users to manually enter the cost code.
We decided to use the Sample Prep team cost code for the automated submission. The BioResource team will include the 'real' faculty team cost code in the RT that they are sending anyway to the SSRs (to notify them there are samples ready to go down the pipeline). Only one cost code will be needed in an RT - the whole batch should have the same one. The SSRs can then manually create the cross-charging report as normal. It was raised previously that sample failures need to be taken into account in the report - this is normally managed manually - the Sample Prep team inform the SSRs of any failures.

Deployment:
Deploy outside of working hours as this contains a breaking change to the API - once the SS change is deployed, some automated submissions from Limber will fail until the Limber change is deployed.

Deploy Sequencescape change
Delete the scRNA Core submission template, and call WIP=009_scrna_core_cell_extraction_submission_templates bundle exec rake record_loader:all to get it to re-create it (to test)
Deploy Limber change


Babel JSX Support: File extension question.

I use Babel's JSX feature in my React project, but I prefer to use a .jsx extension on files that are using it to make it clear, at a glance, which files are components (plus I get some nice editor features).
What's the best way to configure babel-brunch / brunch to accept *.jsx files as well as *.js?

When the plugin was first put together, 6to5 didn't provide jsx support. We'll need to update it to allow the extensions it acts upon to be user-configurable.
Not sure whether jsx should be included as a default as it could mess things up if someone has also installed react-brunch or something else that compiles jsx.

I'd vote for keeping .js as the default, but letting you pass in a desired extension/regex if you wish to override it. At this point, everyone's doing different stuff (ex: we're using .es6 for Babel stuff in one of our non-brunch apps), so it might be futile to push everyone in one direction.
I also like the sentiment that anything with a *.js file extension should be able to run in the browser as-is, no transformations. Yes, you might end up with a maelstrom of file extensions, but at least you aren't accidentally breaking stuff.
Thoughts? Happy to throw up a PR and take on this work.

Yes, PR please.
There are other brunch plugins that allow user-configurable extensions in this way that you could look at. jaded-brunch comes to mind.

Resolved by #11. Just need to document the option in the readme and publish 5.1.0

done


[Bug] Doesn't accept files with some special chars

Describe the bug
If I've a folder with an image with the char ã it gives an error and it doesn't open it.
Expected behavior
It should open the folder
Logs
[2021-03-03 00:50:21] default.ALERT: Fatal Error (E_ERROR): Uncaught UnexpectedValueException: The Response content must be a string or object implementing __toString(), "boolean" given. in /home/public_html/forum/upload/vendor/symfony/http-foundation/Response.php:406 Stack trace: #0 /home/public_html/forum/upload/backend/Kernel/Response.php(23): Symfony\Component\HttpFoundation\Response->setContent(false) #1 /home/public_html/forum/upload/backend/Controllers/FileController.php(55): Filegator\Kernel\Response->json(Object(Filegator\Services\Storage\DirectoryCollection)) #2 [internal function]: Filegator\Controllers\FileController->changeDirectory(Object(Filegator\Kernel\Request), Object(Filegator\Kernel\Response)) #3 /home/public_html/forum/upload/vendor/php-di/invoker/src/Invoker.php(82): call_user_func_array(Array, Array) #4 /home/public_html/forum/upload/vendor/php-di/php-di/src/Container.php(276): Invoker\Invoker->call(Array, Array) #5 /home/public_html/forum/upload/backend/Services/Router/Router.php(78): DI\Container {"code":1,"message":"Uncaught UnexpectedValueException: The Response content must be a string or object implementing __toString(), \"boolean\" given. in /home/public_html/forum/upload/vendor/symfony/http-foundation/Response.php:406\nStack trace:\n#0 /home/public_html/forum/upload/backend/Kernel/Response.php(23): Symfony\\Component\\HttpFoundation\\Response->setContent(false)\n#1 /home/public_html/forum/upload/backend/Controllers/FileController.php(55): Filegator\\Kernel\\Response->json(Object(Filegator\\Services\\Storage\\DirectoryCollection))\n#2 [internal function]: Filegator\\Controllers\\FileController->changeDirectory(Object(Filegator\\Kernel\\Request), Object(Filegator\\Kernel\\Response))\n#3 /home/public_html/forum/upload/vendor/php-di/invoker/src/Invoker.php(82): call_user_func_array(Array, Array)\n#4 /home/public_html/forum/upload/vendor/php-di/php-di/src/Container.php(276): Invoker\\Invoker->call(Array, Array)\n#5 /home/public_html/forum/upload/backend/Services/Router/Router.php(78): DI\\Container","file":"/home/public_html/forum/upload/vendor/symfony/http-foundation/Response.php","line":406,"trace":null} []
Environment (please complete the following information):

FileGator Version [7.5.0]
Server: [Apache 2.4 on Centos 7]
PHP Version [PHP 7.2]
Browser [Chrome]

Hi,
Your file names and paths must me encoded in UTF-8. I've just tried with the image named ã.png and it works fine. You can use utility called convmv to check and convert encodings.
To see if everything is UTF-8 encoded, open the console, go to this folder, and run:
convmv -f UTF-8 -t UTF-8 *

You can also try to make it work with different encoding as described here:
https://github.com/php/php-src/blob/PHP-7.1.0beta2/UPGRADING#L321
But from my experience, PHP does not work very well with non UTF-8 encodings.

Thank you, that solved it.


[TW-125] introduce rate limiting

Adds rate-limiting, as a number of the in-flight bytes for a given peer.

@avieth Actually, let me add some tests for the new code before reviewing this. I'll ping you when I have them.

@avieth Ok, it's ready for a review now.

I tried the approach of having the startNode take a Node m -> m (NT.Transport m) parameter, but I still needed to pass some rules for when to take/release a lock, since that has to be triggered each time we receive data from a peer or finish a handler. So I decided to bundle up the QDisc with a Map of MVars for locking/delaying, and a function for modifying those MVars. That's what the RateLimiting is.
However, there are still two problems with that, that you have already identified:

The interface right now can only adjust the rate depending on the number of in-flight bytes, which is rather inflexible. Having the connection statistics available would be preferable. I'd suggest moving the Statistics to a separate module, so that we can use it in RateLimiting and Node.Internal without introducing circular dependencies.

Possible mismatch of QDiscs used by Transport and RateLimiting in node. As you proposed, I'll change node to take a (forall t. IO (QDisc t)) -> NT.Transport m instead of the Transport.

@avieth I tried replacing the argument of type NT.Transport m with a ((forall t. IO (QDisc t)) -> m (NT.Transport m)), so that node itself will create and close the transport in a bracket.
While it is nice to have the resource management inside the node, and to have a guarantee that the QDiscs match, that is problematic for the way the tests are performed at the moment. The tests exploit that the Transport used by a node can be accessed from without. So if we want to go ahead with this change, we\ll have to change the tests and make sure that no client code relies on accessing the Transport directly.

For the RateLimiting examples which actually do rate limiting, the map of delays is global to the QDisc, so delays enforced by any particular node/EndPoint on one transport will affect all other nodes/EndPoints on that transport. This probably won't come up in any real world situation, but it certainly could come up in testing and simulation.

You're right, and this would be prevented if we did have the Transport created and closed within node (and the fact that doing this would require us to change the way the tests work probably means that the tests are affected).

Seems to me this trouble is caused by the fact that a QDisc is created for a transport. If it were created for each EndPoint then I think having Node m -> m (QDisc t) would be possible. I'm really keen to have rate limiting be a particular case of this, rather than having special infrastructure just for it. That's to say, in my opinion we need a general way of making a QDisc local to an EndPoint/node.
If we had startNode create the whole transport, that would work, but as you mentioned it's a problem for the existing tests. And it's quite a problem in general, since creating a transport will probably involve a lot of resource acquisition, and it would make it impossible to use an in-memory transport for a simulation with more than one node! So that's a no go for sure.
And the in-memory transport is doubly relevant: it takes no QDisc (there's really no point) and so having the QDisc factor into the general network-transport newEndPoint API doesn't make sense.
To summarize my main issue, it's not exactly about the implementation/changes in this pull request, but about the infrastructure that this pull request is forced to use: it's not clear whether it's possible to get mutable state for a QDisc local to each instantiation (EndPoint). I need to think about this some more.

I'm fairly certain we can't do the right thing without making a change to network-transport-tcp. My idea is to have the TransportInternals give a special variant of newEndPoint with this type:
QDisc t -> IO (Either (TransportError NewEndPointErrorCode) EndPoint)

and then node/startNode will not take a Transport but instead an acquirer/releaser for an EndPoint with the node itself in scope:
Node m -> (m (EndPoint m), EndPoint m -> m ())

Now to make a node with a rate-limiting QDisc, the TCP's internals must be used, but no other special infrastructure is needed: just derive the QDisc t from the Node m.
The original network-transport newEndPoint will of course still use the same IO (QDisc t) for all EndPoints.

Yes, that sounds good.
Should we proceed such that you implement the necessary changes in network-transport-tcp, while I add facilities for global rate-limiting?

Or, we could go with implementing global rate-limiting first. @dcoutts suspects that we might not need a per-peer rate limiting anyway if we're queueing fairly and implement global rate-limiting.

Should we proceed such that you implement the necessary changes in network-transport-tcp, while I add facilities for global rate-limiting?

I think all necessary changes to time-warp-nt are here https://github.com/serokell/time-warp-nt/pull/35
network-transport-tcp changes are linked in that PR.

Heads up: https://github.com/haskell-distributed/network-transport-tcp/pull/58 was merged to nt-tcp master and it should help with rate limiting.
-- TransportInternals gives us this:
newEndPointInternal :: (forall t . Maybe (QDisc t)) -> IO (Either (TransportError NewEndPointErrorCode) EndPoint)

-- A node requires this type as its first parameter
type MakeNewEndPoint peerData m = (SharedAtomicT m (LL.NodeState peerData m) -> LL.NodeEndPoint m)

So the SharedAtomicT m (LL.NodeState peerData m) can be in scope when the QDisc is defined, which should make things simpler.


Pull out 'PxeImage'  declaration if it exists in a provision request

Purpose or Intent
Provision requests using PXE provisioning include
the PxeImage class declaration in :pxe_image_id value
when it exists in the provision options hash.
rebuild_automate_request was passing this value via the rest client post
causing the provision to fail.  Stripping out that class value and leaving only the
id allows these requests to go through.
Steps for Testing/QA

Build and run a provision request in the UI using PXE provisioning
Copy the id of this provisioning request.
run rails r tools/rebuild_provision_request.rb --request-id=<request_id> --console
The :pxe_image_id value should only be an integer vs a string: e.g. PxeImage::<pxe_image_id>

@miq-bot add_label enhancement, provisioning, bug

You also need to handle WindowsImage: app/models/miq_provision_virt_workflow.rb#L964

@gmcculloug - Wondering if you could take a look at this again.  I pulled out the specific mention of PxeImage:: and pared it down to :: which matches what the app is doing here - https://github.com/ManageIQ/manageiq/blob/master/app/models/miq_provision/pxe.rb#L9 - so should be generic enough for WindowsImage as well.

Yes, you are correct.  Looks good, thanks.


Weird relation: if (n !== n)

Good day!
I was reading through eventsource.js and saw something strange at Line 133.
Line 131:  function getDuration(value, def) {
Line 132:    var n = value;
Line 133:    if (n !== n) {
Line 134:      n = def;
Line 135:    }
Line 136:    return (n < MINIMUM_DURATION ? MINIMUM_DURATION : (n > MAXIMUM_DURATION ? MAXIMUM_DURATION : n));
Line 137:  }

Is there any mistake here? Seems like the if statement will never be false.
It was changed in the https://github.com/Yaffle/EventSource/commit/cae1b45833732da2e1c51a33ed6ae7d64baf71e5 or https://github.com/Yaffle/EventSource/commit/cae1b45833732da2e1c51a33ed6ae7d64baf71e5#diff-e3467803443c0c8c621be962b3a7127b for convenience.
It was previously the following.
Line 131:  function getDuration(value, def) {
Line 132:    var n = Number(value) || def;
Line 133:    return (n < MINIMUM_DURATION ? MINIMUM_DURATION : (n > MAXIMUM_DURATION ? MAXIMUM_DURATION : n));
Line 134:  }

Regards,
William

This is for NaN value.


Figure out why the GitHub Actions stopped

c2fe40e77f9e5de5fabc5c701ae6b00741d7231b was the last commit which triggered Actions 🙃

Not my fault: https://www.githubstatus.com


Better latex rendering in docs

Thank you! I'm not sure why the docs build isn't working but will fix when I'm back.

I can't checkout this branch so can't update with the changes to main, so if you can do that it would be great please!

Also I'll add permissions so you can make branches in this repo

It may be related to #92


chore: release  thymus 0.1.0

:package: Staging a new release
0.1.0 (2024-06-05)
Features

add thymus helm chart (#3) (f5d6b66)

Miscellaneous Chores

add release please and logic for publishing chart (#4) (c1a9abd)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/carbynestack/thymus/releases/tag/thymus-v0.1.0 :sunflower:


add borderless mode to it

I like the fps upgrade but I also want to use my different monitor without taping out the screen or having a big bar up top. Can you add a borderless mode to it?

In options


What is the license for this repository?

Thank you for your great work! Could you tell me the license for this repository?
Thank you in advance.

Thank you for your quick response!
I will close this issue.


Add missing RCPs for RNN-T

In https://github.com/mlcommons/logging/pull/146 only BS=2048 RCPs are updated. We need to update new numbers for reminder configs.

Since we are updating the RCPs anyway, we would like to request relaxing the WER requirement slightly. With our experiments, we are finding it extremely challenging to observe stable convergence to the required WER (although we are able to get extremely close).

You mean you are failing the RCP test?

@emizan76 - we fail to reach the target WER although our implementation seems to be exactly the same as reference.


How to combine multiple tiff files into a single tiff file using nodeJS

What are you trying to achieve?
I want to merge/concatenate a multi-tiff(multiple pages) file into a single tiff file for the nodeJs project.
I didn't find any examples in the API documentation that seem to show this. I am looking for an example of how to process each page of a multi-page TIFF and merge into single file. If there is an example in the documentation that I've missed, please point me in the right direction. If there isn't, can someone please help me figure this out?

This was asked, using almost identical wording, a few minutes ago in #3399. I assume you and @rajeesab know each other so will close this question. Please don't double-post.


Fix frustum corner calculation for cascading shadows (Vulkan's clip space has a [0,1] z range instead of [-1,1])

The article it's based on is regarding OpenGL, so they had a [-1,1] range:
https://johanmedestrom.wordpress.com/2016/03/18/opengl-cascaded-shadow-maps/
I noticed this issue, when trying out cascading shadows with an orthographic projection instead of a perspective one. There, some parts of the scene were missing, which I backtraced to these calculations. But if my understanding is correct, this will also improve the shadow quality for perspective projection. :)
Oh, and thank you so much @SaschaWillems for this great repository and all the resources you provide - They are an immense help getting into Vulkan! Also, let me know if there is anything else I have to do in this PR, as this is my first time opening one in this repo.

Good catch. Thanks for fixing this and providing this PR. Very much appreciated 👍
And always happy to hear if my work is useful :)


Discussion: Investigate the fetching of unneeded byte array per storage item

Description
Previously, we used the FullTrustProcess for all item thumbnails and an optimization was put in place to associate the underlying byte array with the ListedItem. I'm proposing that we remove this trait from ListedItem and ItemViewModel because we could see a small performance gain for the items affected through simply fetching the StorageItemThumbnail and setting it as the BitmapImage source directly.
The only area leveraging this byte array is the Properties dialog. In my opinion, this makes little sense and we should load thumbnails from the filesystem for the Properties experience on-demand.
Steps To Reproduce
Load a large directory.
Expected behavior
As we know, users expect item thumbnails to appear quickly. I think we should do this without the overhead of converting the thumbnail into a byte array only to subsequently form a bitmap.
Files Version
v2.0
Windows Version
Windows 11, 10
Relevant Assets
Open questions:
Is a converted byte array via the extension method still somehow preferrable over  direct use of the thumbnail stream with BitmapSource.SetSourceAsync(fileThumbnail) ?

Related: https://github.com/files-community/Files/issues/5822

I agree with this 👍

@gave92 FYI https://github.com/files-community/Files/pull/6104#issuecomment-917545832


Template source from config not used

The getTemplateConfig method of ConfigService:
    public function getTemplateConfig($framework, $basePath)
    {
        $templates = __DIR__.'/../Templates/'.$framework;

        if ($framework === 'Laravel') {
            $templateDirectory = $basePath.'/resources/crudmaker/crud';
            if (is_dir($templateDirectory)) {
                $templates = app('config')->get('crudmaker.template_source', $templateDirectory);
            }
        }

        return $templates;
    }

First, it defines the path of the default templates of the package. Then we check if the framework is Laravel (btw, no custom templates possible for Lumen?), and check if resources/crudmaker/crud exists only to provide a default for the config.
Although I do have a path set in my config file, because I do not have the resources/crudmaker/crud directory, it uses the package defaults.
Maybe check if there is a config value first. If not, then check if /resources/crudmaker/crud exists and use that one.
Didn't have time to submit a PR, but the fix should be very straightforward.

Thanks, I use the default mostly so sometimes I miss stuff :)

PR? :)

Yeah I couldnt wait on Otwell to move that forward so I just put it directly in :)


The Android Gradle plugin supports only Kotlin Gradle plugin version 1.3.40 and higher

The Android Gradle plugin supports only Kotlin Gradle plugin version 1.3.40 and higher.
The following dependencies do not satisfy the required version:
project ':video_compress' -> org.jetbrains.kotlin:kotlin-gradle-plugin:1.3.31

created PR #129 that solves the issue


Bundled cli-ruby used by default

WHY are these changes introduced?
Fixes https://github.com/Shopify/cli/issues/1306
Some problems have appeared using the embedded version of cli-ruby to run themes commands or dev command with theme-app-extensions
Those problems seem to be related with the method used to install Ruby requirements in the local environment. There are also some problems with theme-app-extensions in windows as well

WHAT is this pull request doing?

Every command related with themes and theme-app-extension will use the bundled version of cli-ruby
Add a the new env variable SHOPIFY_CLI_EMBEDDED_THEME_CLI to run the command using the embedded version.

How to test your changes?

Run any theme command with --verbose flag and you should see an output similar to this:

Running system process:
  · Command: bundle exec shopify theme check /Users/alvarogutierrez/cli-themes/first-theme --output text
  · Working directory: /Users/alvarogutierrez/cli-themes/first-theme

Post-release steps

Measuring impact
How do we know this change was effective? Please choose one:

[ ] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
[ ] Existing analytics will cater for this addition
[ ] PR includes analytics changes to measure impact

Checklist

[ ] I've considered possible cross-platform impacts (Mac, Linux, Windows)
[ ] I've considered possible documentation changes
[ ] I've made sure that any changes to dev or deploy have been reflected in the internal flowchart.

@alvaro-shopify I believe I am having similar issues relating to this, I would appreciate any insight you can give. When attempting to deploy an app to production (hosted on digital ocean if that matters), shopify app deploy is run. This command also bundles and pushes theme extensions. Part of which is running theme check on the extension code. By default the server is running on 2.5 whereas shopify needs 2.7. Upgrading the ruby version server side should fix probably but this seems a bit clunky no?

@Michael-Gibbons we know depending on Ruby is annoying and we are working on that (it will take some time). But Buildpack should work with recent Ruby versions, no?

@Michael-Gibbons we know depending on Ruby is annoying and we are working on that (it will take some time). But Buildpack should work with recent Ruby versions, no?


Roundtrip pan zoom

Added panning and zooming functionality to roundtrip. The zooming and panning is bound to each tree uniquely so trees can be manipulated without interfering with one another. The brushing functionality remains preserved across trees.

Changes included in #389.


Fix UnBundle command

Variable name changed in vundle, g:bundles -> g:vundle#bundles

Already fixed by  #743