Implement the latest select parsing changes

What problem is this PR intended to solve?
In https://github.com/whatwg/html/pull/10557 there are some changes being made to how <select> tags are parsed.

current spec: https://html.spec.whatwg.org/multipage/parsing.html
proposed spec: https://whatpr.org/html/10557/parsing.html
deltas: https://whatpr.org/html/10557/8c74909...b8ba8e6/parsing.html

Have you included adequate test coverage?
Tests are under development on a branch at https://github.com/html5lib/html5lib-tests/pull/178
Does this change affect the behavior of either the C or the Java implementations?
HTML5 is only supported by the C impl.

@stevecheckoway I need your help with this. I'm pretty sure I've implemented the changes correctly, but could use your eyeballs.
There are lots of failures that I don't understand whether I've implemented the spec incorrectly or if the spec needs work? For example this test from tests19.dat:
#data
<!doctype html><select><option><option>
#errors
(1,39): eof-in-select
#document
| <!DOCTYPE html>
| <html>
|   <head>
|   <body>
|     <select>
|       <option>
|       <option>

Run it with
$ bundle exec rake compile test A=-n/TestHtml5TreeConstructionTests19#test_27__script_off/

It fails with
  1) Failure:
TestHtml5TreeConstructionTests19#test_27__script_off [test/html5/test_tree_construction.rb:35]:
Element <option> has wrong number of children ["option"]
   Input: <!doctype html><select><option><option>
Expected: | <!DOCTYPE html>
          | <html>
          |   <head>
          |   <body>
          |     <select>
          |       <option>
          |       <option>
  Parsed: <option><option></option></option>.
Expected: 0
  Actual: 1

The code in question that I added is:
  if (tag_is(token, kStartTag, GUMBO_TAG_OPTION)) {
    if (has_an_element_in_scope(parser, GUMBO_TAG_SELECT)) {
      generate_implied_end_tags(parser, GUMBO_TAG_OPTGROUP, NULL);
      if (has_an_element_in_scope(parser, GUMBO_TAG_OPTION)) {
        parser_add_parse_error(parser, token);
      }
    } else {
      if (node_html_tag_is(get_current_node(parser), GUMBO_TAG_OPTION)) {
        pop_current_node(parser);
      }
      reconstruct_active_formatting_elements(parser);
    }
    insert_element_from_token(parser, token);
  }

which I think matches the spec:

A start tag whose tag name is "option"

If the stack of open elements has a select element in scope, then:

Generate implied end tags except for optgroup elements.
If the stack of open elements has an option element in scope, then this is a parse error.

Otherwise:

If the current node is an option element, then pop the current node off the stack of open elements.
Reconstruct the active formatting elements, if any.

Insert an HTML element for the token.

Ugh, sorry about that -- PEBKAC -- I just needed to force-recompile and that particular test is passing. There are other ones that I'll look into now.

I see that whatwg/html PR has had a bunch of changes since I last looked at it. I can take a look at this PR in more detail next week.

@stevecheckoway Thanks. Actually, my changes look pretty good! Only two tests are failing for non-error-message reasons, and they're both very similar. (This PR is using a fixed-up version of https://github.com/html5lib/html5lib-tests/pull/178 with appropriate error messages.)
Zooming in on the simpler of the two:
#data
<select><b><option><select><option></b></select>
#errors
#document
| <html>
|   <head>
|   <body>
|     <select>
|       <b>
|         <option>
|     <b>
|     <select>
|       <b>
|         <option>

This pr results in the following tree:
<body>
  <select>
    <b>
      <option>
  <b>
    <select>
      <option>

so I wanted to double check if I've missed something or if these tests are incorrect.

@stevecheckoway Before you dig in on this, please catch up on the chat I'm having upstream in https://github.com/html5lib/html5lib-tests/pull/178, TLDR I think I'm right and the test (and chromium) are wrong.

Updated the gumbo tests.
Note that these changes break rails-html-sanitizer, I'll need to work a bit upstream before these changes are safe to merge and release.


[3.0 Beta] Plugin-SEO has "exports": null in package.json

Link to reproduction
No response
Describe the Bug
Not sure how this happens since exports are present in the branch:
https://github.com/payloadcms/payload/blob/c70dcb6a598911117dd2824a0288376ca5e7502b/packages/plugin-seo/package.json#L49-L54
For some reason, the installed module's package.json has:
"exports": null,
To Reproduce

(p)npm init
(p)npm install<EMAIL_ADDRESS>cat<EMAIL_ADDRESS>| grep exports

Tried with both npm and pnpm to rule out any package manager conflict.
Payload Version
3.0.0-beta.6
Adapters and Plugins
No response

Hey @janus-reith On the latest beta.10 I'm not seeing this occur at all, neither in npm or pnpm
"exports": {
    ".": {
      "import": "./dist/index.js",
      "require": "./dist/index.js",
      "types": "./dist/index.d.ts"
    }
  },

I'm going to assume it's been fixed in one of the versions but will re-open if you see this again on the latest version

I'm going to assume it's been fixed in one of the versions but will re-open if you see this again on the latest version

Indeed, just checked and its was fixed in beta.10:
https://github.com/payloadcms/payload/commit/80496aa94cb2980a84b04dc784a132a1e97a4209


RDK-38587 - GTests for Xcast plugin

RDK-38587 - GTests for Xcast plugin
Reason for change:
Adding Gtests for Xcast plugin
TestProcedure: None
Risks: Low

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rdkservices/3870/rdkcentral/rdkservices

Commit: 93f105713b8f7240098369ad40cbe5012d77a93f

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rdkservices/3870/rdkcentral/rdkservices

Commit: c3b029c5822ffa20a964528820e523f44d48854b

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rdkservices/3870/rdkcentral/rdkservices

Commit: c6adb2be96659399923c6cc6e38a412e11df254c

Report detail: gist

Hi @hgfell683
BlackDuck is complaining about a code match at devicesettings.h lines 1278-1303 which matches quite closely to pxScene/pxCore src/rtRef.h.  If this is the source, it must be credited by inserting a comment above the class definition at l.1268:
Based on pxCore, Copyright 2015-2018 John Robinson
Licensed under the Apache License, Version 2.0
If you have another source, please comment.
Thank you - Martin Hughes, RDK CMF Compliance Team

@hgfell683 can you please not copy paste. What exactly from rtRef do you need for the unit tests

@npoltorapavlo Basically, the rtObject files are all interconnected through a series inheritance, mainly through the overloading of operators, and how, holding objects that are then used for various comparisons. The main function that uses rtRef is rtObjectRef through the use of template functions/classes in rtRef (specifically inherited by rtRef). rtObjectRef uses most of the template function that comes from rtRef for most of its comparisons within the xCast plugin. I've tried changing and modifying how this works to slim down on the added code, but most of the time editing the functionality in rtRef causes some kind of error within the code.

@hgfell683 at a glance, rtRef is a smart pointer class. Can we use std smart pointer instead?

@npoltorapavlo I could try and see if that works, but I don't think rtRef is a smart pointer, as it's a template class. Template classes, and smart pointers do have similar syntax, as they both use <> to define themselves when used.

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rdkservices/3870/rdkcentral/rdkservices

Commit: eee0e08cbeaa05359050be0894e54a3a6242c47c

Report detail: gist

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rdkservices/3870/rdkcentral/rdkservices

Commit: 975b3a8b8a162f91451ace75c39dfc4292c377f0

Report detail: gist

Sorry, @hgfell683 - Please will you make sure that new code files have proper Apache headers?  This won't block but is still required.  Thank you!


When items switches from 0 to more than 1 image, the last one is shown initially, not the first one

i assume, it is due to center() not being called after inputs changed (https://github.com/daelmaak/ngx-gallery/blob/master/libs/gallery/src/lib/components/viewer/viewer.component.ts#L274)
my workaround currently is to call gallery.select(0) after we updated the items but this slides the images once :(

Thanks. I wrote a test and created a repo  to reproduce it but no success. Can you please elaborate or create a reproduction stackblitz?

I reproduced it in loop mode 👍


How was this line declared

I can't understand how did you declare this line?
https://github.com/ShahjalalShohag/code-library/blob/cf3039641b5aa84b1c5b184a95d69bd4091974c9/Data Structures/Treap.cpp#L6

https://www.cplusplus.com/reference/random/mt19937/


how to render closed swipeout after flatlist rerender

In a Flatlist each item is a swipeout component. The button of each swipeout item deletes an array element of the flatlist data property. After deleting, the "deleted" swipeout element is still open but with text of the next array element. I don't know if this an issue or feature of flatlist or swipeout -  in all cases it is irritating users. Is there any way to close the swipeout button without animation?

This is happening for me too


Admin dashboard graphs lag over time

Am seeing an issue where the scrolling dashboard graphs begin to lag behind actual clock time when left open for a few hours.  The net effect of this is that the most recently displayed graph value does not match the summary value that's displayed elsewhere in the value.

Upon further investigation, issue was that the top-most graph grows horizontally when switching tabs. (Under certain window sizes, in Chrome). So switching back and forth between tabs can cause "lag" because the newest portion of the graph is hidden from view.


updates withRouter docs to describe all injected props.

previously implied that only router was injected, but location,
params and routes are also injected.

Cool. Thanks!


Add option to limit subdomain tenant resolver to a part of the subdomain

Simply add max chars as the option.

Dine in #2165


KOGITO-3360: Cucumber Tests: Keep logs only for test failures

JIRA: https://issues.redhat.com/browse/KOGITO-3360
Please make sure that your PR meets the following requirements:

[x] You have read the contributors' guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[x] Your feature/bug fix has a unit test that verifies it
[x] You've tested the new feature/bug fix in an actual OpenShift cluster

Hi @ricardozanini , we're ready to merge this PR. Thanks in advance,


Custom postgresql port

Application Name
postgres-init
I'm using an external psql cluster with a different port than 5432. There is a way con configure postgres-init container to that custom port?
Code of Conduct

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

Not currently no, but that feature could be added. Feel free to submit a PR!


Add storage mode option

Add an option to the add torrent dialog which lets the user switch between sparse and full file allocation.

Would be really nice option, cause by downloading >200GB Torrent, don't really want to allocate all space for nothing.

The current behavior is to use sparse allocation, so your use case is the supported one. This feature request is for adding support to select full file allocation instead. Sorry for the confusion :smile:


Input  : The quick brown fox jumps over the lazy dog
Output : Pangram sentence

How to contribute

Save the solution in check_pangram_string.go file
Add check_pangram_string.go file in check-pangram-string folder

@harshraj8843 I can work on this. Can you assign it to me?

Hey @amoghrajesh
You can comment !assign on any good first issue you want to contribute to, to be auto-assigned
!assign

!assign


Batchsize with IoTHubSendInterval not working as intended

Describe the bug
Batchsize and IoTHubSendInterval set in command line arguments but do not seem to be affecting how often messages are sent to the IoTHub
To Reproduce
Steps to reproduce the behavior:

Go to IoT Hub
Set modules for IoT Edge device, with image uri for OPCPublisher being mcr.microsoft.com/iotedge/opc-publisher:latest. The command line
Set container create options to the following:

{
    "Hostname": "opcpublisher",
    "Cmd": [
        "--PkiRootPath=/mount/pki",
        "--pf=/mount/published_nodes.json",
        "--lf=/mount/publisher.log",
        "--mm=PubSub",
        "--me=Json",
        "--fm=true",
        "--fd=false",
        "--bs=1028",
        "--di=20",
        "--sc=1",
        "--si=60",
        "--aa"
    ],
    "HostConfig": {
        "Mounts": [
            {
                "Type": "bind",
                "Source": "/home/pi/opcpublisher",
                "Target": "/mount"
            }
        ]
    }
}

Configure routes to default endpoints
Create deployment for device

The published_nodes.json looks as follows:
[
    {
        "EndpointUrl": "opc.tcp://{host}:4840",
        "UseSecurity": true,
        "OpcAuthenticationMode": "UsernamePassword",
        "OpcAuthenticationUsername": "{username}",
        "OpcAuthenticationPassword": "{password}",
        "OpcNodes": [
            {
                "Id": "{nodeId}"
            }
            ]
    }
]

Filled in with my server's information and ~185 nodes in the array.
Expected behavior
I am following the documentation here for setting command line arguments, and my goal is to be able to monitor specific nodes at different rates, store data changes in the batch/queue, and submit them in a message at the interval specified with --si. However, even with --si=60 and --bs=1028, the messages to the IoT Hub are still happening every few seconds (~10s/msg).
Am I misunderstanding the documentation above regarding the purpose of batchsize and iothubsendinterval?
Additional context
Add any other context about the problem here.
Edge Device:

OS: Ubuntu 20.04.4
IoT Edge Version: 1.3.0

If you have received more than 1028 notifications a batch message is emitted or if the interval has expired and there are not enough notifications yet to be sent (60 seconds).  Can it be that there are more than 1028 notifications every 10 seconds or so?  It is also possible that messages queue up for sending, which are also sent immediately and not held back.  Best would be to use a deterministic OPC UA node as source and try this e.g., you could use the OPC PLC simulation server (which is another project under Azure-Samples organization).

@marcschier Thank you for the response! I have taken your advice and set the OpcNodes in published_nodes.json to just 1 node. This has allowed me to get to a somewhat desirable state. As of right now, the messages are coming through as follows:
[
    {
      "MessageId": "41",
      "MessageType": "ua-data",
      "PublisherId": "opc.tcp://{ip-address}:{port}_{identifier}",
      "Messages": [
        {
          "DataSetWriterId": "1000",
          "SequenceNumber": 41,
          "MetaDataVersion": {
            "MajorVersion": 1,
            "MinorVersion": 0
          },
          "Timestamp": "2023-02-14T14:52:08.397Z",
          "Payload": {
            "ns=6;s=::AsGlobalPV:OPC.DM2EXT.OEEDataCtr": {
              "Value": 145,
              "SourceTimestamp": "2023-02-14T14:52:07.607Z",
              "ServerTimestamp": "2023-02-14T14:52:07.607Z"
            }
          }
        }
      ]
    },
    {
      "MessageId": "42",
      "MessageType": "ua-data",
      "PublisherId": "opc.tcp://{ip-address}:{port}_{identifier}",
      "Messages": [
        {
          "DataSetWriterId": "1000",
          "SequenceNumber": 42,
          "MetaDataVersion": {
            "MajorVersion": 1,
            "MinorVersion": 0
          },
          "Timestamp": "2023-02-14T14:52:11.397Z",
          "Payload": {
            "ns=6;s=::AsGlobalPV:OPC.DM2EXT.OEEDataCtr": {
              "Value": 146,
              "SourceTimestamp": "2023-02-14T14:52:10.634Z",
              "ServerTimestamp": "2023-02-14T14:52:10.634Z"
            }
          }
        }
      ]
    }
]

This is a very close step to where I would like to be, as it is sending these messages every 60 seconds as I defined in BatchTriggerInterval. However, it is sending multiple MessageIds, whereas I would like it to send one MessageId with multiple Messages in the array. Since each change has a timestamp associated with it, this would not cause an issue because the post-processing can handle the ingress messages. Is this possible?

This will be enabled in 2.9.0 release by supporting strict compliance with part 6 and 14.  A network message will then containore than one dataset in batching mode.  Would that be the desired outcome?

@marcschier Yes! That would be a great feature for my project. Much appreciated and looking forward to the 2.9.0 release!


matplotlib__matplotlib-23476 failed at pre-install

Describe the bug
The conda env installation of matplotlib__matplotlib-23476 timeouted at pre-install stage.
Looks like tzdata set up is asking for some keyboard input which it will never receive.
Mitigation: On my machine I solved it by manually pre-run export DEBIAN_FRONTEND=noninteractive TZ=Etc/UTC, but I think it should be added somewhere in the repo?
Steps/commands/code to Reproduce
python run.py --model_name gpt4o   --instance_filter matplotlib__matplotlib-23476 --split test
Error message/results
INFO     ▶  Beginning task 125

INFO     matplotlib__matplotlib not found in container, cloning...
DEBUG    Falling back to full cloning method due to multiple instances or persistent container
INFO     matplotlib__matplotlib__3.5 conda env not found, creating...
DEBUG    Created conda environment with environment.yml
DEBUG    Installed extra pip dependencies
INFO     Running pre-install commands...
ERROR    Read with timeout failed on input:
         ---
         apt-get -y update && apt-get -y upgrade && apt-get install -y imagemagick ffmpeg texlive texlive-latex-extra
         texlive-fonts-recommended texlive-xetex texlive-luatex cm-super dvipng
         ---
......
Setting up libflac8:amd64 (1.3.3-2ubuntu0.2) ...
Setting up poppler-data (0.4.11-1) ...
Setting up tzdata (2024a-0ubuntu<IP_ADDRESS>) ...
debconf: unable to initialize frontend: Dialog
debconf: (TERM is not set, so the dialog frontend is not usable.)
debconf: falling back to frontend: Readline
Configuring tzdata
------------------

Please select the geographic area in which you live. Subsequent configuration
questions will narrow this down by presenting a list of cities, representing
the time zones in which they are located.

  1. Africa      4. Australia     7. Atlantic Ocean  10. Pacific Ocean
  2. America     5. Arctic Ocean  8. Europe          11. US
  3. Antarctica  6. Asia          9. Indian Ocean    12. None of the above
Geographic area:
WARNING  ❌ Failed on matplotlib__matplotlib-23476: Timeout reached while reading from subprocess.

System Information
Platform: Linux-6.5.0-41-generic-x86_64-with-glibc2.35
System: Linux
Release: 6.5.0-41-generic
Python ver: 3.10.14
swebench: 2.0.12
sweagent: 0.6.1 (editable installed, commit b0ee29251b01)
Checklist

[X] I'm running with the latest docker container/on the latest development version (this means running both git pull and docker pull sweagent/swe-agent:latest)
[X] I have copied the full command/code that I ran (as text, not as screenshot!)
[X] If applicable: I have copied the full log file/error message that was the result (as text, not as screenshot!)
[X] I have enclosed code/log messages in triple backticks (docs) and clicked "Preview" to make sure it's displayed correctly.

Hi @HejiaZ2023 . Thank you very much for the nice report! Let me move this to SWE-bench, as it needs to be fixed there.


Codebase reorganisation

I'd like to:

[ ] Move API tests from frontend/tests/test_api_* to api/tests
[ ] Move fetchers and importers from frontend/management/commands to pipeline/management/commands
[ ] Move measure definitions from frontend/management/commands/measure_definitions to somewhere less nested

@sebbacon Any reason not to do so?

Sounds good to me (especially the first point which I've been meaning to do for, like, 2 years)

Did this get done @inglesp ?


Change source and target compilation levels to Java 8

The changes address the following comments/discussions:

https://github.com/saucelabs/ci-sauce/pull/63/files#r455303208
https://github.com/saucelabs/ci-sauce/pull/63/files#r455290220

@adambabik @maciejurbanskisaucelabs could you please review the changes?

Codecov Report

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

@@            Coverage Diff            @@
##             master      #69   +/-   ##
=========================================
  Coverage     61.17%   61.17%
  Complexity      195      195
=========================================
  Files            11       11
  Lines           935      935
  Branches        144      144
=========================================
  Hits            572      572
  Misses          313      313
  Partials         50       50

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 c647b96...0a399d8. Read the comment docs.


Only call parse on JSON_MODULE if it's defined

This addresses https://github.com/chanks/que/issues/189

Hey, thanks @rwz. @hmac, who pointed out the issue initially, also realized that there'd be a problem with MultiJson. I'm trying to figure out whether it's safe to just always call JSON_MODULE.load(json, create_additions: false)

Yeah, it seems that way to me. I just pushed a fix to master, would you verify that it works for you? If it does I'll cut a new release.

This is very likely to fix the issue for our codebase that has MultiJson in it. Seems like we'd also wanna someone with stdlib JSON to weight in. Or... I guess all the specs are exercising no-MultiJson case, so it's probably a good enough warranty?

Yeah, that's my thinking, I'm not worried about the stdlib case. Is it convenient for you to test this, just to double-check that MultiJson works? If not I suppose I'll just release - it's a pretty simple fix, seems to me.
While we're here, I'm wondering if there's a compelling use-case anymore for Que to use MultiJson at all? Way back when it was introduced I believe the rationale was that stdlib had poor performance, but I'm not sure that that's true these days? I'm kind of inclined to remove support (in a major version bump) for simplicity's sake.

Yeah, our CI's just verified that master works fine.

Alright, released 0.12.3.

Cool, thanks!

Hey @rwz, I just pushed a change to master to drop support for MultiJson, since it doesn't seem to be necessary anymore and removing it simplifies things (see #191). When it's convenient would you check and make sure that the current master works for you?
Thanks!

I can check now. Will report back in a few.

We're green.


PR for #104: enable options cluster and robust

Closes #104

Per call: @Constantino-Carreto-Romero will:

move the solution code into a program apart but inside _eventiv
make more tests to check for cases when the code could break

In https://github.com/JMSLab/xtevent/pull/107/commits/4f1692461b28cf9866310a025dc9076a9450438f I moved part of the solution code to a parser at the end of the program. Thanks to the parser I'm not using regex functions anymore. I ran the test file as well.

@jorpppp  in https://github.com/JMSLab/xtevent/pull/107/commits/b9cacdf08c8726563560493176db41153f47338a I added the loop for https://github.com/JMSLab/xtevent/pull/107#discussion_r981754896
I checked that everything keeps working after this little change.

Thanks @Constantino-Carreto-Romero. Merging now.


Update README to mention second-precision jobs

Updates the README to mention this undocumented (and regularly requested!) feature.

Coverage decreased (-0.1%) to 91.688% when pulling 2f520992a27fd8062159be79757fe91668411ccb on tekin:mention-six-parameter-cronline-in-readme into 25d7bc7a92ef52f675bb4e0a3249572a00467e46 on ondrejbartas:master.

@ondrejbartas coverage has dropped slightly, but the PR is only updating the README. Thoughts?

Thank you for this. Anything blocking the merge?

@gogiel not from my perspective.

This should be closed and deleted because .poll_interval was long gone by v3.4.0
https://github.com/mperham/sidekiq/blob/main/Changes.md#340
which was from 2015.
https://github.com/mperham/sidekiq/commit/149949373fe371a3c2c60a7f7c5984317762d811

@beatak thanks for pointing that out. Rather than delete this pull request I've updated it to mention the new configuration mechanism. I've left the PR open as the main point of the PR (to explain how to schedule second-precision jobs) still stands.

Hi, just stumbling across this now. Any issues with merging this PR so the sidekiq-cron community can benefit?

needs to be rebased and can be merged

We also mention the average_scheduled_poll_interval options here https://github.com/ondrejbartas/sidekiq-cron#under-the-hood, but with different syntax and different defaults 🤔 should we "unify" or DRY these explanations?


Fix Rails 5.2 and 6.x Docker images

There were 2 issues that were breaking the build and tests.

Updating Ruby Gems itself without specifying a version causes it to update to the latest version which requires Ruby 3. Instead, we specify the latest version compatible with Ruby 2.6.

Compatibility versions found here:
https://rubygems.org/api/v1/versions/rubygems-update.json

The version of Alpine Linux shipping with the Ruby 2.6 image is missing a shared library needed to load nokogiri. Adding gcompat corrects this.

More information found here:
https://github.com/github/pages-gem/issues/839

Closing for a more complete fix in pull request #2


Error: [vite]: Rollup failed to resolve import "tslib"

Using vite, I get this error when trying to start my Express server:
Error: [vite]: Rollup failed to resolve import "tslib" from<EMAIL_ADDRESS>
It seems like tslib is a devDependency and in dist/pkg/analytics.js tslib is being imported on line 1:
import { __awaiter, __extends, __generator, __spreadArrays } from "tslib";

I'm guessing this vite/Rollup error is due to the devDependency not being available when it's referenced in my app? Adding tslib to my app's dependencies fixes the issue, but maybe it should be a non dev dependency in analytics-next?

@kamui Thanks for reporting this issue! You're right, tslib should be a direct dependency. Created #347 to address this.


Add support for configuring nvidia GPU access for container

some related issues ：
https://github.com/kata-containers/runtime/issues/2269
https://github.com/kata-containers/agent/issues/709
https://github.com/kata-containers/documentation/issues/517
https://github.com/kata-containers/runtime/issues/1876
There are two questions：

When to load nvidia-kernel module
Nvidia kernel module can't be loaded when there is no device to drive, so , it can't be loaded using kata configuration file  or systemd scripts. We need to distinguish the container  which has gpu vfio device invoked  with '--device' , and load nvidia kernel modules at the right time(after devices added).
How to share the driver files（libs, bins）at kata-vm（container‘s host） to container
Reference nvidia-container-runtime ：“ A modified version of runc adding a custom pre-start hook to all containers.If environment variable NVIDIA_VISIBLE_DEVICES is set in the OCI spec, the hook will configure GPU access for the container by leveraging nvidia-container-cli from project libnvidia-container”.
We need to do the same thing to configure GPU access for the container. Because we can't let user  modify Dockerfile to install driver files every time when a container is started, it's  inefficient and complicated for users.

Solution：

Like nvidia-docker， set env “NVIDIA_VISIBLE_DEVICES”（in device-plugin, Dockerfile, or command args）to distinguish the gpu container, then call function to setup nvidia driver for the container.
setup nvidia driver for container :

exec "nvidia-container-cli --kernel-modules info " to load nvidia kernel modules and get gpus info
exec "nvidia-container-cli list --binaries","nvidia-container-cli list --libraries" list the neccessary driver files
mount the binaries and libraries listed before to the container's rootfs
exec "ldconfig" to configure dynamic linker run-time bindings

nvidia-container-cli and nvidia kernel modules should be instlled to kata-vm when building the rootfs.

Test result
Actually, I have done some tests in the way above，it can achieve the same effect as nvidia-docker.

Docker:

kubernetes(with my own device-plugin)

I have added and tested these code in my own project.  If the solution is OK,  I'd like to submit a PR.

yes please, submit a PR, I'll glad to to review it

Hi @RyanAo. How to install nvidia-container-cli and nvidia driver into rootfs?

"in kata Guest vm,  configure nvidia GPU access for container"  this is really needed


document is not defined

I installed html5sortable through npm:
npm install html5sortable

in my module I set:
require('html5sortable');

the first time I use .sortable() I get:
Uncaught ReferenceError: document is not defined

I know for a fact that in NW.js modules the document object is located at window.document.
Adding:
typeof window.document.createElement('span').dragDrop === 'function'

to this line: https://github.com/voidberg/html5sortable/blob/master/dist/html.sortable.js#L177
fixes this issue.

Should be fixed by #165


Can't open DailyAL app from aniyomi

Steps to reproduce

Set DailyAL to open myanimelist.net links
Open one link from aniyomi

Expected behavior
It should open the anime page
Actual behavior
It stays stuck on this screen

Crash logs
No response
Aniyomi version
Preview r7814
Android version
Android 15
Device
Xiaomi Redmi Note 9S Custom ROM
Other details
Ive seen that it doesnt open the app, but rather it stays on aniyomi

Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extension's repository.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

It seems the issue is from the app


Do not create Ceph LVs

All changes to the disks are made only from the node image and not during deployment.

Hint: You cannot use percentage values in the node-image. curtin does not allow this.

Then we should open an issue for it at Curtin to be able to solve it per Curtin and until then at least create the VG directly in Curtin and then a check on the VG and not on the block device. Otherwise this does not work with different disk layouts. We don't always have NVMe but also sda.

You could still create the LVs via node-image in the user-data second stage.

@osfrickler I would leave it here. Then it is easier to change the number of LVs etc. later.


remove mark as cancelled option from kebab button under purchase order with the status cancelled (in detail page)

issue closed


As a user, I would like to interact in a chat like in messaging service, so I can in the futur have some assistance to create Itinerary

PR #164


added missing argument

as per method signature:
private function tryApplication(InternalRequest $ireq, callable $application, array $filters)

Changing method signatures and forgetting to change them all… ;-)
Thanks for the PR :+1:


Use Vant Component Type is Error

https://github.com/youzan/vant/issues/12351

vue version affect


rebuild: more testing bug fixes (CAS-246)

added a few more testcases and that led to fixing a few bugs
rebuild with childs of different sizes (prior that would fail) including
using children with sizes < metadata
using non multiple of the segment size for completeness
the lookup of rebuild jobs and concurrent rebuild jobs which revealed a
bug
rebuild operations idempotence (currently on the client api) though we
may want to restrict it to RPC specifically
bonus: when the rebuild tests fail (local or CI) the cargo test itself
gets stuck which is caused by panics on a different thread (?) so moved
the failures to the reactor thread and also use different nexus and bdev
names on each test - this produces a cleaner failure log
rebuild code coverage reported by grcov is now ~84% (if you trust it)
todo: cas-287 will test the IO failures
Resolves: CAS-246

LGTM


Support for ED25519 and SECP256K1 curves

Secp256k1 / Ed25519 Support
This PR expands the GoSDK to support new curve types for API keys, including generating, storing, retrieving, and signing with these keys.
Specific updates:

apikey.New() now requires a second parameter specifying the curve. This is a breaking change. Example usage:

key, err := apikey.New("some-uuid", apikey.SchemeSECP256K1)

Keys generated with the SDK now have a non-exported scheme property that informs signing and is encoded into local the storage of the private key.
Private keys are stored with a suffix denoting the curve type, which is read into the ApiKey struct when the key is loaded. For example deadbeef123456:secp256k1. This change is backwards compatible, as keys stored with no suffix will be assumed to use the p256 curve (the only one supported prior to this change).
The apikey.FromTurnkeyPrivateKey() method now requires a scheme parameter. This is a breaking change.
Methods for encoding and decoding keys have been split into ECDSA-specific and ED25519-specific methods. This is a breaking change.
The apikey.Stamp() method will return a curve-specific signature, dependent on the scheme property of the ApiKey passed in. The function signature has not changed.

Notes
This PR slightly relaxes the lint restriction on exhaustive switch statements, allowing for default value to be counted as "exhaustive", as well as the restriction on returning non-exported types from exported functions.

I don't think the breaking change is particularly a problem, since I doubt people are using this beyond our own code.  However, an alternative would be to use the functional parameters ellipsis pattern.
func New(id string, opts ...apikey.OptionFunc)

...

apikey.New("some-uuid", apikey.WithCurve(apikey.SchemeSECP256K1))

I don't think the breaking change is particularly a problem, since I doubt people are using this beyond our own code. However, an alternative would be to use the functional parameters ellipsis pattern.
func New(id string, opts ...apikey.OptionFunc)

...

apikey.New("some-uuid", apikey.WithCurve(apikey.SchemeSECP256K1))

Yeah, good call. I think that's more elegant. Will do.


Add ASP.NET Core 3.0 breaking changes

I know this is a draft but wanted to call your attention to the fact that each breaking change should be in an include file so we can easily create future versions like 2.2-3.1. Maybe you already know but I thought it wouldn't hurt to say.

@mairaw I plan to do that, but thank you for checking. For editing, it's much easier to have everything in a single file.

@mairaw It's ready for another look.

@mairaw Ready for another look.


Change wait to sleep in podmanimage readme

Change wait to sleep 100000 as we've had some reports of problems
with wait:  https://github.com/containers/buildah/issues/1665
Signed-off-by: TomSweeneyRedHat<EMAIL_ADDRESS>

Sure, LGTM

/lgtm

/retest


Update ComprehensiveGuide.md

thanks sam

oh dammit, please don't PR master from now on. I made a dev branch, we're going to merge dev into master on every release


Update to latest assert-plus

Hi,
Can you please update to the latest assert-plus npm (currently 0.2.0)?  It fixes an issue I see in my code when I try to run in strict mode (I'm trying to update my code and its dependencies to work with node 5.0.0 in strict mode to get access to ECMAScript 6 features).  This issue occurs when assert-plus's assert.string() method is called.
 /tmp/errors(master) $ node --use_strict foo.js
/private/tmp/errors/node_modules/assert-plus/assert.js:41
                stackFunc = stackFunc || _assert.caller;
                                                ^

TypeError: 'caller' and 'arguments' are restricted function properties and cannot be accessed in this context.
    at _assert (/private/tmp/errors/node_modules/assert-plus/assert.js:41:49)
    at Object.string (/private/tmp/errors/node_modules/assert-plus/assert.js:160:9)
    at Object.<anonymous> (/private/tmp/errors/foo.js:2:8)

3.1.0 has been published with the new assert-plus.

thanks!


mod_auth_openidc + simplesamlphp Segfaults Apache

Environment

mod_auth_openidc version
2.1.6-1 from ubuntu 16.04LTS packages.

Apache version
2.4.25-3+deb9u3 from ubuntu 16.04LTS packages

SimpleSAMLphp versions
1.14.11-1 from ubuntu 16.04LTS packages
1.14.16 (download)

libapache2-mod-php7.0
7.0.19-1 from Ubuntu 16.04LTS packages

platform/distro (e.g. Ubuntu Xenial or Centos 7)
Ubuntu 16.04LTS

Expected behaviour
Apache server with globally enabled mod_auth_openidc to live together with a vhost serving simpleSAMLphp (without openidc configuration).
Actual behaviour
When mod_auth_openidc is enabled globally (a2enmod auth_openidc), a request to simplesamlphp vhost segfaults apache. The vhost does not have Authtype openid-connect configured.
Disabling mod_auth_openidc globally (a2dismod auth_openidc) and loading the module in the oidc vhost configuration seperately, fixes the problem.
Minimized example
Ubuntu 16.04LTS + mod_auth_openidc enabled + simplesamlphp serving vhost (via alias).
Configuration and Log files
This is what apache logs on simplesamlphp request:
[core:notice] [pid 37038] AH00052: child pid 37043 exit signal Segmentation fault (11)

Hey Martin, can you send me the two config files: the crashing one and the one that doesn't?
I'm inclined to believe that loading the module is a global thing anyway and it doesn't make a difference where the LoadModule statement is. I suspect another interference between the two.

This is complicated, because it's an interaction between multiple conf files.
So the failing setup is: a2enmod auth_openidc and this vhost for simplesamlphp:
<VirtualHost *:80>
        ServerName idp-test.*****.nl

        ServerAdmin webmaster@localhost
        DocumentRoot /var/www/html

        Alias /saml /opt/simplesaml/simplesaml/www

        ErrorLog ${APACHE_LOG_DIR}/error.log
        CustomLog ${APACHE_LOG_DIR}/access.log combined

        <Directory /opt/simplesaml>
            Options FollowSymLinks
            AllowOverride None
            Order allow,deny
            allow from all
            Require all granted
        </Directory>
</VirtualHost>

The working example is a2dismod auth_openidc and the second line below added to the oidc.conf vhost:
<VirtualHost *:80>
        LoadModule auth_openidc_module /usr/lib/apache2/modules/mod_auth_openidc.so
        ServerName oidc-test.******.nl

        ServerAdmin webmaster@localhost
        DocumentRoot /var/www/html

        ErrorLog ${APACHE_LOG_DIR}/error.log
        CustomLog ${APACHE_LOG_DIR}/access.log combined

        OIDCProviderMetadataURL https://proxy.******.nl/.well-known/openid-configuration
        OIDCSSLValidateServer Off
        OIDCClientID oidctest
        OIDCClientSecret test
        OIDCResponseType "code"
        OIDCResponseMode "query"
        OIDCScope "openid"
        #OIDCScope "openid email profile"
        OIDCAuthRequestParams claims={"userinfo":{"foobar":null,"nickname":null,"name":null,"uid":null,"email":null,"sho":null,"family_name":null,"eppn":null}}

        OIDCRedirectURI http://oidc-test.******.nl/redirect_uri
        OIDCCryptoPassphrase randompassword

        <Location />
           AuthType openid-connect
           Require valid-user
        </Location>
</VirtualHost>

The simplesamlphp vhost file isn't changed between te two tests.
Hope you understand the test ;)

I cannot reproduce this with a fresh Docker installation.
Can you perhaps try with the files below?
Remove the .txt extension, add:
<IP_ADDRESS> openidc.zmartzone.eu simplesaml.zmartzone.eu

to your /etc/hosts, run "make" and access the two vhosts above?
simplesaml.conf.txt
openidc.conf.txt
Makefile.txt
Dockerfile.txt

Hmm.... I just discovered that what I thought was Ubuntu 16.04 has actually been deployed from Debian GNU/Linux 9 on the VM. My apologies!
I can't run the docker test because I haven't compiled cgroups in my kernel (atm). Hope you can reproduce on Debian 9.

Nope, not able to reproduce. I'm actually going to assume it is interference with another module that is crashing unless you can provide a full example  in a minimized setup. The Dockefile for stretch is as follows:
FROM debian:stretch

RUN apt-get update && apt-get install -y apt-utils curl
RUN apt-get install -y libapache2-mod-auth-openidc
RUN apt-get update && apt-get install -y apache2

ADD openidc.conf /etc/apache2/sites-available/
ADD simplesaml.conf /etc/apache2/sites-available/
RUN a2enmod auth_openidc
#RUN a2ensite openidc
RUN a2ensite simplesaml

Ok, I was able to reproduce it. It appears mod_php7 is crashing when accessing OpenSSL crypto operations. This only happens with the package that comes with the Stretch distribution. It does not happen with the same version downloaded from the releases page of this project.
Hence I believe packaging was done against an incompatible OpenSSL version which messes up things for PHP7. Unfortunately I do not control this but I'll see if I can nudge the package uploader.
Your solution for now would be to use a version distributed as part of this project.
*** Error in `/usr/sbin/apache2': free(): invalid pointer: 0x000000be32302740 ***
======= Backtrace: =========
/lib/x86_64-linux-gnu/libc.so.6(+0x70bcb)[0x7f0a7915abcb]
/lib/x86_64-linux-gnu/libc.so.6(+0x76f96)[0x7f0a79160f96]
/lib/x86_64-linux-gnu/libc.so.6(+0x777de)[0x7f0a791617de]
/usr/lib/x86_64-linux-gnu/libcrypto.so.1.1(EVP_DigestInit_ex+0x205)[0x7f0a7015f465]
/usr/lib/x86_64-linux-gnu/libcrypto.so.1.1(+0x18c79d)[0x7f0a7019a79d]
/usr/lib/apache2/modules/libphp7.0.so(zif_openssl_random_pseudo_bytes+0xd3)[0x7f0a70dfe893]

Well... my fix (loading mod_auth_openidc.so) in the oidc.conf vhost file works for me (although I don't see why), so I think I'll stick to that. I only raised this issue to bring the problem to your attention in the case it was auth_openidc's problem.

Thanks; that reminds me that it is most probably the module loading order that matters (since that's the only thing that changes when loading elsewhere). I'll have to look into that to see if OpenSSL initialization somehow conflicts with that of mod_php7.

continued in #308


Unable to hide the header while searching/filtering recyclerview

I'm unable to hide the headerview (as it is showing only letter "A" header while searching)
while searching for any item in recyclerview when using editext.addontextchangelistener functionality.
Please help me

set getHeaderId()  return -1,will not display headerview


Update Charlie.

Ports 3 PRs from master – #277 #269 #278

Apologies for force push, I wanted to clean up the commit descriptions - the last one was incorrectly formatted and wasn't displayed properly on GitHub.


Shell Navigation with custom routing does not work

I'm trying to get navigation to work using Shell and when I navigate using Shell.GoToAsync(string) it does nothing...how do I report this issue or get help with this?

You've just reported it :) Please give us time, and we'll give a look.
Thanks
El dom., 5 may. 2019 16:38, MarkTi<EMAIL_ADDRESS>escribió:

I'm trying to get navigation to work using Shell and when I navigate using
Shell.GoToAsync(string) it does nothing...how do I report this issue or get
help with this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/TailwindTraders-Mobile/issues/49, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAD24EP2AA6JDKF3YU4HPJLPT3WPBANCNFSM4HK3MXBA
.

We'll probably use the Shell navigation with routes feature in this project - no ETA though

BTW, It's done in the 'q2-maintenance' branch
https://github.com/microsoft/TailwindTraders-Mobile/commit/cf7a4e398200b58505226ca8ca72a58c4e1a2314
Will be eventually merged into 'master'

Update status: It's on a PR just waiting to merge (#62)

Done on master branch. Thanks for the suggestion!


Bugfix/home page typo

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


window size exceeding monitor maximum resolution with GLFW_SCALE_TO_MONITOR

I have my windows DPI set to 125%
using the window hint GLFW_SCALE_TO_MONITOR
and using an initial resolution of my monitor's max (1920x1080) as parameters to glfwCreateWindow
the resulting window size & frame buffer size is scaled beyond the displayable maximum resolution of my monitor. This is while creating a "borderless" window.
workaround I'm using atm:
`point2D_t window_size;
glfwGetFramebufferSize(glfwwindow, &window_size.x, &window_size.y);
// bugfix, if window created is larger than maximum monitor resolution, disable hint GLFW_SCALE_TO_MONITOR and recreate window!
if (window_size.x > max_resolution.x || window_size.y > max_resolution.y) {
	glfwDestroyWindow(glfwwindow); glfwwindow = nullptr;
	glfwWindowHint(GLFW_SCALE_TO_MONITOR, GLFW_FALSE);

	glfwwindow = glfwCreateWindow(resolution.x, resolution.y, Globals::TITLE, monitor, nullptr);
	// refresh window size
	glfwGetFramebufferSize(glfwwindow, &window_size.x, &window_size.y);
}`

Are you trying to create a window that covers a whole monitor? In that case GLFW_SCALE_TO_MONITOR isn't appropriate. Perhaps a windowed fullscreen window would be better.

I've put in support for fullscreen borderless windows and native fullscreen with a toggle in an ini file. So both are working. I just noticed this happening while working on that implementation for my application.

If I understand this correctly then it is working as intended. You are creating a window intended to cover the whole monitor but are not (the first time) passing in a monitor, so GLFW treats it as a non-fullscreen window and resizes it.

A better way to do it is to create a hidden window, then resize and show it. AFAIK, while glfwCreateWindow is affected by GLFW_SCALE_TO_MONITOR - glfwSetWindowSize is not.

@ghuser404 Yes, that should work.


[Adblock BUG] Please remove this extension

Describe bug
I love Spicetify and it's great that Spotify hasn't banned it in any way, so far.
But if you add an extension to block ads, it's going in the wrong direction.
If there's a big reason for them to block Spicetify, it's because of this.
Please remove this feature, please respect Spotify's business model and please don't put other Spicetify users in danger by keeping this extension online, thank you.
Steps to replicate
Necessary screenshots
Spicetify and Spotify Version
Config file:

counterpoint: piracy is based and im not paying for premium


Plugins should not require removing use_frameworks! from Podfile

Right now the Google Sign In example app has removed use_frameworks! from its Podfile.
If this line is added back, the pod install step fails with the following error.
[!] The 'Pods-Runner' target has transitive dependencies that include static binaries: (/Users/jackson/git/gsi42/example/ios/Pods/Google/Frameworks/GGLCore.framework and /Users/jackson/git/gsi42/example/ios/Pods/Google/Frameworks/GGLSignIn.framework)

See https://github.com/firebase/FirebaseUI-iOS/issues/34 for an example of other devs struggling with this issue.
A related issue is that the FlutterFire plugins are also written assuming use_frameworks! is not in the Podfile. For example, the Firebase Analytics example app fails when use_frameworks! is added back to its Podfile.
    <module-includes>:1:1: warning: umbrella header for module 'Flutter' does not include header 'FlutterAsyncMessageListener.h' [-Wincomplete-umbrella]
    #import "Headers/Flutter.h"
    ^
    <module-includes>:1:1: warning: umbrella header for module 'Flutter' does not include header 'FlutterJSONMessageListener.h' [-Wincomplete-umbrella]
    <module-includes>:1:1: warning: umbrella header for module 'Flutter' does not include header 'FlutterMessageListener.h' [-Wincomplete-umbrella]
    3 warnings generated.
    In file included from /Users/jackson/git/firebase_analytics/ios/Classes/FirebaseAnalyticsPlugin.m:7:
    /Users/jackson/git/firebase_analytics/example/ios/Pods/Headers/Private/Firebase/Firebase.h:1:9: fatal error: 'FirebaseAnalytics/FirebaseAnalytics.h' file not found
    #import <FirebaseAnalytics/FirebaseAnalytics.h>
            ^
    3 warnings and 1 error generated.

We should try to make our plugin system compatible with use_frameworks!, or if that's not possible for Google Sign In, at least make the Firebase plugins able to build with use_frameworks!.
/cc @mravn-google @jakobr-google @szakarias @mit-mit

We have a workaround, it's just not pretty, so I’m probably going to focus on other more pressing issues for the next few weeks. This issue is available if anyone else wants to investigate; it's possible that the answer is just "move to Carthage" in which case we can punt it.

If we're going to move to Carthage we'll want to do that sooner rather than later. :)

Carthage is that it isn't officially supported by Firebase or Google Sign-In, so I think we might be stuck with Cocoapods until that changes.

Plus, Carthage doesn't automatically add(/remove) the dependent frameworks to your app project, which is basically the only thing we're using CocoaPods for (well, that, and letting plugins transitively depend on other frameworks).
Switching to Carthage means having to develop our own code for managing the Xcode projects, so it's not happening right now.

We talked about this in our meeting this morning and it sounds like we want use_frameworks! to be commented out by default in the new project template.

@collinjackson @LarkAscending can we go ahead and remove this from the Firebase codelab then?

YES we can do that! Excellent news....and it's a low-impact change to the codelab.
Removing and republishing now.

Removed from step 6, Configure Google Sign-In on iOS section:
https://codelabs.developers.google.com/codelabs/flutter-firebase/#5

In your podfile where You see
target 'Runner' do
  use_frameworks!

comment user_frameworks!
target 'Runner' do
  # use_frameworks!

and go to path /ios of the project
run
pod install
and cheers.

No, but I have the same issue; you're not alone

@shinayser, @pujitm, I have the same issue. See:
https://github.com/aloisdeniel/flutter_plugin_appcenter/issues/3#issuecomment-474801091
Please let me know if you worked it out!

@peanut-flutter The last time I checked, I still had to remove use_frameworks!

Still having this issue.
As per the instructions, I've done the following:

Added camera: ^0.4.3+2 to my pubspec.yaml,
flutter packages get.
I've added NSCameraUsageDescription and NSMicrophoneUsageDescription to my Info.plistwith the appropriate usage description.
I've commented out use_frameworks! in my Podfile, and ran pod install and pod update at different times.

Still, I'm left with the following error:
☁  tcp_flutter [master] flutter run -d 9B1AED9A-88A5-4A88-8618-10DFEA93FEBD
Launching lib/main.dart on iPhone Xʀ in debug mode...
Running Xcode build...
 ├─Assembling Flutter resources...                           1.4s
 └─Compiling, linking and signing...                         0.3s
Xcode build done.                                            3.7s
Failed to build iOS app
Error output from Xcode build:
↳
    ** BUILD FAILED **

Xcode's output:
↳
    === BUILD TARGET Runner OF PROJECT Runner WITH CONFIGURATION Debug ===
    /Users/davydany/.tcp/tcp_flutter/ios/Runner/GeneratedPluginRegistrant.m:6:9: fatal error: 'camera/CameraPlugin.h' file not found
    #import <camera/CameraPlugin.h>
            ^~~~~~~~~~~~~~~~~~~~~~~
    1 error generated.

Could not build the application for the simulator.
Error launching application on iPhone Xʀ.

Same issue here. Any chance we can reopen this isuse?

This issue is definetly real. Please @eseidel, you are our only hope =(
I simply can' run my project on MacOS since september last year (18). Everytime I need to debug ios is a real problem because my project doesn't compile fine though Intellij but it doest through x-code! This is a problem because when compiling through x-code we lose hot reload and need to recompile every change
I have posted it on a separated issue, here https://github.com/flutter/flutter/issues/32910

Oh my god, I thought I was alone in this, I also can get the project to run through XCode but not with the flutter run nor flutter build command.
Really really regretting choosing Flutter in the past couple days where I have been dealing with users uploading videos and playing them back. That's too cutting edge for official Flutter libraries, and packages which go in that way are plagued of pitfalls as they rely on 3rd party plugins which brings us here.
But hey, we've got GameController support right? What else could be more important than that 🤔
Sorry if I'm coming through rude, I'm just exhausted of spending days surfing through blocking, unresolved, closed and ghost issues, struggling with something as basic and elementary as media playing.

This issue was closed 2 years ago.  I think it would be most helpful if someone would be willing to open a fresh issue so we can make sure to address exactly the troubles being hit today.
@collinjackson or @jmagman may have more context as to the current state of things, but I recommend we start with a fresh issue (which can link to this one if desired).

This issue was closed 2 years ago. I think it would be most helpful if someone would be willing to open a fresh issue so we can make sure to address exactly the troubles being hit today.
@collinjackson or @jmagman may have more context as to the current state of things, but I recommend we start with a fresh issue (which can link to this one if desired).

I have already on #32910. It is almost 4 months old =(

Thank you @shinayser!


Add prod configuration

Still under investigation, the goal of this configuration file is to override the baseurl used in _config.yml and by GitHub pages for the production deployment of this website. /cc @kennethgillen

👍 , but I don't currently have a way to test it without merging and re-running the www-dev playbook somewhere.

Tested during today's testing. All worked as expected.


Attempting to clear more verify-libs-and-licenses reports.

Doing my best to resolve some more of the verify-libs-and-licenses warnings.

Travis seems to fail with this, otherwise I would merge.

Great, merging.


Unable to write some text after exit from menu

After exit from menu: menu.showMenu(false) any text cannot be displayed.
My solution is:
In method void showMenu(boolean val, boolean update = true) add _oled->textMode(BUF_ADD) after _oled->clear();

@goorvy Alexgyver has BUF_REPLACE by default. Now I make this setting the default.


Add CodeQL code analysis

Problem
The project currently does not have any automated vulnerability scanning, which is recommended by Stanford IT.
Solution
Enable CodeQL on this repository for vulnerability scanning.
Additional context
No response
Code of Conduct

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

Added some foundations for this in Spezi (https://github.com/StanfordSpezi/Spezi/pull/70) and in the .github repo: https://github.com/StanfordSpezi/Spezi/pull/70


remove filter from cat

For https://github.com/hangxie/parquet-tools/issues/93.

So confirmed jq can handle JSONL format pretty well in term of memory pressure, and jq has really good features for filtering and more.


Ensure that errors from plugins don't require --debug in order to get logged.

If a plugin panics during start up, lyra logs an error message that the load of the plugin failed but that message contains no information about the root cause of the failure. The user must start lyra with --debug in order to see the real reason for the panic, logged as DEBUG output from the plugin.
Panics that cause termination of plugins should be logged as errors and not require --debug to become visible.

I made some findings while implementing a hclog look-alike in TypeScript that I think is relevant to this ticket.
The go-plugin assumes that all log output from a plugin is written on stderr. A special pipe is established to redirect everything that plugin process writes on stderr to the local logger. Two pieces of code are interesting:
Here, a decision is made whether to treat the stderr output as arbitrary and redirect it to the local debug logger: https://github.com/hashicorp/go-plugin/blob/master/client.go#L947-L967. And here is the actual parseJSON() method: https://github.com/hashicorp/go-plugin/blob/master/log_entry.go#L35-L73.
Incidentally, the JSON tags conform exactly with what's output from the hclog logger when JSONFormat is set to true.

Closed by #217, lyraproj/servicesdk#25,  and lyraproj/terraform-bridge#4


Instructions for downloading operator SDK is outdated.

Description
The description in docs/DEVELOPMENT.md only downloads a empty file. Looking at the operatorSDK home page the instruction seems outdated.

Steps to reproduce the issue:

Follow instructions on page.
Notice the file is 0 bytes.
Profit

Describe the results you received:
0 bytes file. :-(
Describe the results you expected:
many bytes file. :-)

I've added a docs PR: https://github.com/openshift/sandboxed-containers-operator/pull/264


Sub-classes of str appear to just operate as aliases of str

>>>>> class String(str):
.....     def foo(self):
.....         return "foo"
.....
>>>>> String()
''
>>>>> String().foo()
Traceback (most recent call last):
  File <unknown>, line 0, in <module>
AttributeError: RefCell { value: [PyObj class "str"] } object has no attribute foo
>>>>> type(String())
<class 'str'>

@cthulahoops This seems like something you'd have the best insight on.

I fixed this issue for ints yesterday. str_new ignores the cls parameter it is passed and always uses str_type. str_new needs rewriting to look more like str_int.
(I couldn't use arg_check! for int_new because it needs to check issubclass rather than isinstance.)

(There's a deeper issue that I'm aware of but avoiding. It is possible to set instance attributes on subclasses of built-in types, but we don't currently have a dictionary to store these in.)

I think the solution here might be to get rid of the PyObjectKind::Instance and give each pyobject a dict or Option<PyObjectRef> dict member. This way each object can have some extra attributes. In any case, I think Instance pyobjectkind is an obvious kind, since all objects are instances.

Yes, this should be exactly the idea!
Using x.__dict__ will simply return the attributes member as a dict I guess?

Yes, that's what I was thinking, though it might be tricky to make sure:
x.__dict__ is x.__dict__

An issue we already have in this case:
>>>>> bool.__mro__ is bool.__mro__
False


Refactor cmake

I create this issue to document information needed to refactor cmake related material.
Minimum requirements to build cytnx. Here is the incomplete list.
For C++

boost
blas/lapack. Effectively, we only support MKL and OpenBLAS. Maybe we only officially support these two.

For python wrapper

python
pybind11

For GPU support

CUDA
cuTENSOR
cuQuantum

For OpenMP

CMP0167 https://cmake.org/cmake/help/latest/policy/CMP0167.html
The FindBoost module is removed in cmake 3.30.
find_package(Boost CONFIG) finds the upstream package directly, without the find module.

CMP0146 https://cmake.org/cmake/help/latest/policy/CMP0146.html
The FindCUDA module is removed in cmake 3.27.
Examples

https://cliutils.gitlab.io/modern-cmake/chapters/packages/CUDA.html

OpenMP examples

https://cliutils.gitlab.io/modern-cmake/chapters/packages/OpenMP.html

CUDA Quantum and CMake¶

OpenBLAS
If libopenblas is installed in /opt/OpenBLAS/
On Linux, after setting
export C_INCLUDE_PATH=/opt/OpenBLAS/include
export CPLUS_INCLUDE_PATH=/opt/OpenBLAS/include
export LD_LIBRARY_PATH=/opt/OpenBLAS/lib

cmake can find OpenBLAS and one can compile cytnx.
On MacOS (Apple silicon), still cmake cannot find OpenBLAS.
On Linux and MacOS, one can use CMAKE_PREFIX_PATH.
CMAKE_PREFIX_PATH=/opt/OpenBLAS cmake ..

then cmake can find OpenBLAS. But it will fail to compile because it cannot find lapacke.h.
One still have to set the include_path as above. Then it will compile OK.
==> Next thing is to try to use submodules to handle OpenBLAS.


License definition

We should choose an open source license for both components. Helps maybe:

https://choosealicense.com/
https://tldrlegal.com/

Since this code bases on the original source of https://github.com/olinux/zskarte2 which is licensed as LGPL-3.0 by default, the license should be kept.
Also, during the migration of the code, the acknowledgments to the original repository were lost. It would be nice, if they could be reintegrated.


Frequent error reporting:  eof receive going to reconnect for good measure

[2017-09-06 15:33:08] local.DEBUG: eof receive going to reconnect for good measure
[2017-09-06 15:32:43] local.DEBUG: ping sent
[2017-09-06 15:32:43] local.DEBUG: not found something so ping
[2017-09-06 15:32:43] local.DEBUG: ping sent
[2017-09-06 15:32:43] local.DEBUG: Recevid: 13
[2017-09-06 15:32:43] local.DEBUG: Fetching: 0
[2017-09-06 15:32:43] local.DEBUG: Recevid: 13
[2017-09-06 15:32:43] local.DEBUG: Fetching: 0
[2017-09-06 15:32:54] local.DEBUG: not found something so ping
[2017-09-06 15:32:54] local.DEBUG: ping sent
[2017-09-06 15:32:54] local.DEBUG: not found something so ping
[2017-09-06 15:32:54] local.DEBUG: ping sent
[2017-09-06 15:32:54] local.DEBUG: Recevid: 13
[2017-09-06 15:32:54] local.DEBUG: Fetching: 0
[2017-09-06 15:32:54] local.DEBUG: Recevid: 13
[2017-09-06 15:32:54] local.DEBUG: Fetching: 0
[2017-09-06 15:32:59] local.DEBUG: Recevid: 3
[2017-09-06 15:32:59] local.DEBUG: Fetching: 54
[2017-09-06 15:33:06] local.DEBUG: Recevid: 3
[2017-09-06 15:33:06] local.DEBUG: Fetching: 32
[2017-09-06 15:33:08] local.DEBUG: eof receive going to reconnect for good measure
[2017-09-06 15:33:08] local.DEBUG: Connected to Broker
[2017-09-06 15:33:08] local.DEBUG: Recevid: 3

Definitely suggest this gets rolled in. For any retained messages, they are re-received constantly when not needed on each reconnect (an expected operation). For any significantly sized retained messages, this could pose a bandwidth issue with a large number subscribers using this library times the many reconnects they make.

Opened pull request...


Bugfix: Height property to get the height param

Small bugfix to accurately fetch the height parameter from the request instead of the width within the height property.

Look good, thank you @msanterre !


Add statsd

Adds statsd support via github.com/DataDog/datadog-go/statsd
Verified on edge that this works regardless of whether a statsd service is running (metrics appears to simply no-op in the absence of a valid client)
r? @stripe/data-platform (will also reach out to observability to have a look)

cc @stripe/observability

lgtm modulo what the default can do so it works if one doesn't have statsd running


Add support for versioning with tags for npm other then latest.

What are you trying to accomplish?

This PR adds support for a broader range of npm tags to prevent ill-formed requirement errors. The following tags are now supported:

latest
stable
beta
dev
canary
next
rc
alpha
experimental
nightly
release
legacy

Anything you want to highlight for special attention from reviewers?

The tags included are based on common usage in the npm ecosystem to ensure wide compatibility and prevent errors with ill-formed requirements.
How will you know you've accomplished your goal?

The goal is accomplished if the additional tags do not cause any ill-formed requirement errors and are correctly recognized by the system.
Checklist

[x] I have run the complete test suite to ensure all tests and linters pass.
[ ] I have thoroughly tested my code changes to ensure they work as expected, including adding additional tests for new functionality.
[x] I have written clear and descriptive commit messages.
[x] I have provided a detailed description of the changes in the pull request, including the problem it addresses, how it fixes the problem, and any relevant details about the implementation.
[x] I have ensured that the code is well-documented and easy to understand.

I did complete the testing manually in my local and unit testings.


feat(auth): add jwt support in auth middleware

Closes #14972
This PR adds support for JWT authorization to the http auth middleware.
This uses the new jsonweb package authorizer types.
When Token based authorization is used an attempt is first made to deserialize the payload as a JWT. If the token is well-formed, it is assumed that JWT based authorization is being attempted. The token is then validated and if successful the permission claims are passed down via the usual Authorizer on context strategy.
Given the token is not a well-formed JWT, then it is regarded as a lookup token for an associated Authorization in the AuthorizationService and treated as such. The Authorization is looked up using the token payload as the key.
Currently there is no decision as to how to configure a signing key, so this PR just introduces the mechanisms required for that work in the future. Currently an attempt to use a JWT will result in a no key found error. The token parser's KeyStore must be replace with a key store which returns a valid signing key for a given key id if JWT support is to be enabled.
UPDATE:
This also now transforms each of our Authorizer types into the new PermissionsAuthorizer type which is an Authorizer which can returns its set of permissions. This is required to create a token from an Authorizer.

[x] CHANGELOG.md updated with a link to the PR (not the Issue)
[x] Well-formatted commit messages
[x] Rebased/mergeable
[x] Tests pass
[x] http/swagger.yml updated (if modified Go structs or API)
[x] Documentation updated or issue created (provide link to issue/pr)
[x] Signed CLA (if not already signed)

See UPDATE in PR description

@GeorgeMac This seems to be failing IDPE compatibility. you may want top update IDPE as well.

Cheers @docmerlin
As per external discussion we have decided to remove this new PermissionsAuthorizer interface now anyway. As its original intention has now been solved without any additional change being required.

I re-requested review and added @jademcgough to this review as I needed to rebase over some recent changes to the auth middleware and could do with a sanity check.
After discussions with the compute team it was decided that permission based auth fits well with the JWT token. The Authorizer interface exposes both permission checking capabilities (via Allowed()) and an identity (via GetUserID()). Which gets confusing. So I decided to make the JWT Authorizer represent purely permissions and return an invalid ID when asked for a UserID.
The change that just went in just made the authentication middleware dependent on the UserID field of the Authorizer type. Now it has a lookup on the user ID to ensure the associated User is active. However, for the JWT token route we are purely interested in the permissions of the authorization and not the associated identity. They are for internal use and will expire shortly after issuing. This is how revokation ultimately will occur.
TL;DR
I updated my change to lean on the validity of the user ID in order to decided whether or not to make a user lookup for activity.
Given an invalid user ID (for example a JWT authorizer) no lookup is required.


Check ok

[1222] - VUNDRzbh

[1222] - Qglynd4F

[1222] - ut5F8sRU

[1222] - Allure report on Netlify: https://5ed92117f57bef1700b9632d--jdi-dark.netlify.app


Permission issues running on Kubernetes

Describe the bug
Server fails to install on first run due to not being able to write to filesystem.
To Reproduce
Deploy as a Kubernetes pod.
Expected behavior
Server should start up
Container/Host Logs
*****EXECUTING USERMOD*****
usermod: no changes
*****STARTING INSTALL/UPDATE*****
tid(22) burning pthread_key_t == 0 so we never use it
Redirecting stderr to '/home/steam/Steam/logs/stderr.txt'
Logging directory: '/home/steam/Steam/logs'
[  0%] Checking for available updates...
[----] Verifying installation...
[  0%] Downloading update...
[  0%] Checking for available updates...
[----] Download complete.
[----] Extracting package...
[----] Extracting package...
[----] Extracting package...
[----] Extracting package...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Installing update...
[----] Cleaning up...
[----] Update complete, launching Steamcmd...
tid(34) burning pthread_key_t == 0 so we never use it
Redirecting stderr to '/home/steam/Steam/logs/stderr.txt'
Logging directory: '/home/steam/Steam/logs'
[  0%] Checking for available updates...
[----] Verifying installation...
Steam Console Client (c) Valve Corporation - version<PHONE_NUMBER>
-- type 'quit' to exit --
Loading Steam API...OK

Connecting anonymously to Steam Public...OK
Waiting for client config...OK
Waiting for user info...OK
 Update state (0x3) reconfiguring, progress: 0.00 (0 / 0)
 Update state (0x61) downloading, progress: 0.00 (0 /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 2.29 (108384492 /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 5.08 (240183440 /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 7.53 (356275404 /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 10.33 (488396193 /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 13.65 (645404772 /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 16.69 (789113983 /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 19.91 (941427815 /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 23.01<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 25.20<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 28.26<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 31.01<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 33.36<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 35.65<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 37.91<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 42.31<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 45.43<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 49.56<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 53.25<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 54.59<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 56.48<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 58.29<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 59.18<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 60.01<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 60.83<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 61.67<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 62.51<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 63.32<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 64.14<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 64.96<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 65.78<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 66.73<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 67.60<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 68.54<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 71.54<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 72.70<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 73.57<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 74.42<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 75.20<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 76.03<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 76.87<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 77.70<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 78.55<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 79.41<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 80.26<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 81.06<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 82.77<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 84.77<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 85.67<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 86.87<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 88.38<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 89.70<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 90.49<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 91.37<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 92.29<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 93.08<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 94.69<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 95.85<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 96.67<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 97.56<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 98.50<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x61) downloading, progress: 99.30<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x81) verifying update, progress: 4.97 (234924295 /<PHONE_NUMBER>)
 Update state (0x81) verifying update, progress: 16.21 (766559038 /<PHONE_NUMBER>)
 Update state (0x81) verifying update, progress: 26.92<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x81) verifying update, progress: 33.01<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x81) verifying update, progress: 40.64<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x81) verifying update, progress: 51.30<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x81) verifying update, progress: 64.44<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x81) verifying update, progress: 77.44<PHONE_NUMBER> /<PHONE_NUMBER>)
 Update state (0x81) verifying update, progress: 90.23<PHONE_NUMBER> /<PHONE_NUMBER>)
Success! App '2394010' fully installed.
*****CHECKING FOR EXISTING CONFIG*****
*****GENERATING CONFIG*****
timeout: failed to run command './PalServer.sh': Permission denied
cp: cannot stat '/palworld/DefaultPalWorldSettings.ini': No such file or directory
RCON_ENABLED=true
sed: can't read /palworld/Pal/Saved/Config/LinuxServer/PalWorldSettings.ini: No such file or directory
RCON_PORT=25575
sed: can't read /palworld/Pal/Saved/Config/LinuxServer/PalWorldSettings.ini: No such file or directory
*****STARTING SERVER*****
./PalServer.sh -port=8211 -players=16 -servername=****** -serverpassword=****** -adminpassword=*******
sh: 1: ./PalServer.sh: Permission denied

Additional context
My deployment:
apiVersion: apps/v1
kind: Deployment
metadata:
  labels:
    app: palworld
  name: palworld
  namespace: game-servers
spec:
  progressDeadlineSeconds: 600
  replicas: 1
  revisionHistoryLimit: 5
  selector:
    matchLabels:
      app: palworld
  strategy:
    rollingUpdate:
      maxSurge: 1
      maxUnavailable: 1
    type: RollingUpdate
  template:
    metadata:
      creationTimestamp: null
      labels:
        app: palworld
    spec:
      containers:
      - env:
        - name: ServerIP
          valueFrom:
            fieldRef:
              fieldPath: status.podIP
        envFrom:
        - configMapRef:
            name: palworld-config-cm
        image: thijsvanloef/palworld-server-docker:v0.10
        imagePullPolicy: IfNotPresent
        name: palworld
        ports:
        - containerPort: 25575
          name: palworld-rcon
          protocol: TCP
        - containerPort: 8211
          name: palworld-game
          protocol: UDP
        - containerPort: 27015
          name: palworld-query
          protocol: UDP
        volumeMounts:
        - mountPath: /palworld/
          name: palworld-data
      dnsPolicy: ClusterFirst
      restartPolicy: Always
      terminationGracePeriodSeconds: 30
      volumes:
      - name: palworld-data
        persistentVolumeClaim:
          claimName: palworld-pvc

Workaround is to add a initContainer:
      initContainers:
      - name: permissions-update
        image: busybox:1.35.0
        imagePullPolicy: IfNotPresent
        command: ["/bin/sh", "-c", "chown -R 1000:1000 /palworld; chmod -R 770 /palworld"]
        volumeMounts:
          - mountPath: /palworld
            name: palworld-data
      - name: permissions-check
        image: busybox:1.35.0
        imagePullPolicy: IfNotPresent
        command: ["/bin/sh", "-c", "ls -ahl /palworld"]
        volumeMounts:
          - mountPath: /palworld
            name: palworld-data

Add any other context about the problem here.

k8s support has just been merged https://github.com/thijsvanloef/palworld-server-docker/pull/65, closing the issue.


Is There A Plan to Support Version 5 ?

Is There A Plan to Support Version 5 ?

Sorry for the delay in responding.
My current situation probably doesn't allow me to interact with this project until next year for now.
But as I always stated any contribution will always be welcome.

+1 on version 5 request. This explorer's awesome - would it be a big job?

I made a version that works with LiteDB v5 and created a pull request (https://github.com/julianpaulozzi/LiteDbExplorer/pull/52) to address this issue. Fingers crossed that everything goes well with the pull request since it is my first ;)

@phantomtypist The reason me and I guess most people want to use this explorer is the dark theme or better, the nicer design in general.
@jswigart @stigzler @tabzhang Until @julianpaulozzi can take a look at my PR and possibly merge it, here's a build you can use which I'm using currently: Debug.zip. Please note though that I'm only using it to view data, not modify and allthough it seems to be working fine from my limited tests I can't guarantee that it's a 100%.

Awesome sauce - just briefly tested + it can read + modify V5 dbs. Great work, fella. Do you have an associated GH project or is it private atm?

@stigzler I'm happy to hear that it works for you!
This would be the associated fork I created: https://github.com/Memphizzz/LiteDbExplorer/tree/LiteDBv5

Sorry for the long delay in replying. @phantomtypist  is right, since 2020 I haven't had time to look at this project. I thought the situation would change but not.
It took a considerable amount of time to found the project, someone with possibilities to maintain the future would be very good.
The update process is linked to the github releases, in which case the easiest thing would be to actually update in another repository to keep the tool as possible for those who still use version 4.

To maintain this repository I accept to invite collaborators who are interested.

Does not work for LiteDB 5...still.


Adjust repository to the community rules suggested by GitHub

So far we have a README and a Wiki, but this is only half of what GitHub recommends, we still need to develop a Code of Conduct and Contributing document.

This isn't actually necessary


Parsing JSON number to Decimal loose precision

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
I believe PR #6327 introduced Decimal validator regression.
https://github.com/pydantic/pydantic/blob/6ef959fa69d1ab1aedf2778b6c8f452e0fee37b6/pydantic/_internal/_std_types_schema.py#L189-L194
Such a schema union causes that in the first place the JSON literal data is cast to the float type. This in turn causes a loss of accuracy upon subsequent conversion to Decimal.
Attached script results are:
v2.0.0
python -> a=Decimal('0.1234')
json -> a=Decimal('0.12339999999999999580335696691690827719867229461669921875')

This result was reported as #6295 fixed by #6321
v2.0.2
python -> a=Decimal('0.1234')
json -> a=Decimal('0.1234')

I believe this result as a correct behaviour, after #6321 is merged.
v2.0.3
python -> a=Decimal('0.12339999999999999580335696691690827719867229461669921875')
json -> a=Decimal('0.12339999999999999580335696691690827719867229461669921875')

This result is actual behaviour with v2.0.3 installed, after #6327 merged. Validation logic was moved to pydantic-core in order to achieve speedup, but precision lost is introduced as a side effect.
According to Contribution Guide I'm writing this issue, but I think that I'm also able to start PR to fix it. It should be simple as removing single
https://github.com/pydantic/pydantic/blob/6ef959fa69d1ab1aedf2778b6c8f452e0fee37b6/pydantic/_internal/_std_types_schema.py#L191
to avoid conversion to float and result in the same validation logic as in v2.0.2
EDIT: Also str_schema strict param should be set to False.
Example Code
from decimal import Decimal
from pydantic import BaseModel

class TestModel(BaseModel):
    a: Decimal

data = {"a": 0.1234}
data_json = '{"a":0.1234}'

print('python ->',TestModel.model_validate(data))
print('json ->',TestModel.model_validate_json(data_json))

Python, Pydantic & OS Version
pydantic version: 2.0.3
        pydantic-core version: 2.3.0 release build profile
                 install path: /home/kuba/pydanticDecimal/venv/lib/python3.11/site-packages/pydantic
               python version: 3.11.3 (main, Apr  5 2023, 14:15:06) [GCC 9.4.0]
                     platform: Linux-5.15.0-75-generic-x86_64-with-glibc2.31
     optional deps. installed: ['typing-extensions']

Selected Assignee: @Kludex

I think you are right. We need to always convert it to a string even if it is a float because the very act of parsing the json float string into a Rust or Python float causes precision loss. I'll make a PR to fix this.

Thanks for reporting this. The fix will be in the next release, hopefully tomorrow.


Integrate with ESLint

Requirement
Provide an ESLint plugin to easily integrate UI5 linter checks into existing development workflows.
Challenges
Or why this was not our initial design goal

ESLint rules only check one file at a time. There's no built-in way of holding a context of the project, so that imported modules from the same project can be found and used to gather type information

A key requirement of UI5 linter is the ability to resolve indirect usage of deprecated API. This is done by resolving the (TypeScript-) types of dependencies used in a module and checking every access to them for usage of deprecated API
This is very similar to what for example typescript-eslint calls "type-aware linting"
ESLint might provide functionality for "project aware linting" in the future, see also: https://github.com/eslint/eslint/discussions/16819

ESLint creates an AST representation every source file. In order to lint non-JS resources such as XML-views, the UI5 manifest.json or ui5.yaml, we would need to provide a set of custom processors or even a custom parser

See for example the eslint-plugin kennyzhangdev/ui5depchk which makes use of a custom parser for XML resources

Unclear integration with typescript-eslint: UI5 JavaScript files first need to be transpiled (from our proprietary AMD implementation to ESM), before doing the type-aware analysis using the TypeScript compiler. The latter could potentially be realized using a plugin for typescript-eslint (similar to for example gund/eslint-plugin-deprecation, but it's unclear how the first step can be integrated in that process

Additionally, since we are explicitly targeting projects that are not using Typescript, we would need to find a way to configure typescript-eslint for our own plugin. For example we need to provide it with our predefined tsconfig (the project typically doesn't have one), and we need to inject the UI5 type definitions provided by UI5 linter

Otherwise every project that wants to use UI5 linter would first need to properly setup TypeScript, even though it is not used for anything but the linter

UI5 linter currently declares its own dependency to typescript (required for linting and transpilation), while typescript-eslint requires projects to define the dependency themselves
typescript-eslint itself shows the limits of integrating "type-aware" linting into ESLint, especially with regards to performance (see https://typescript-eslint.io/packages/parser#allowautomaticsingleruninference, https://github.com/typescript-eslint/typescript-eslint/discussions/6366)

ESLint requires synchronous implementations of custom processors and parsers (see https://github.com/eslint/eslint/discussions/14733 and https://github.com/eslint/eslint/issues/15475)

Some of the UI5 Tooling API currently used within UI5 linter is asynchronous. Providing synchronous alternatives would require substantial development efforts

Proposal

Monitor development on ESLint and typescript-eslint side
Focus on providing a good out-of-box experience

Installing the UI5 linter CLI and execute ui5lint should just work for the majority of today's UI5 projects. Projects should typically not need to be modified
By requiring users to install additional dependencies and to maintain (eslint) configuration, we expect a much lower level of adoption, making such scenarios a secondary requirement.

Provide Node.js API for individual linting steps to enable integration in third-party projects

This might include ESLint plugins provided by the community
Other possible consumers are existing linting solutions such as https://github.com/kennyzhangdev/ui5depchk, https://github.com/iljapostnovs/ui5plugin-linter or https://github.com/SAP/ui5-language-assistant

Investigate LSP implementation to provide direct IDE integration

Challenges

Pretty much all the problems listed here were solved by other communities. Using a custom parser for specific file formats is not a problem. React, Vue, Ember  and others have been using it for years. At the end of the day, this new tool is another custom parser, right?
Dismissing ESLint with arguments like would require substantial development efforts is a bit weird to justify a new tool built from scratch;
Today, the tool doesn't offer anything that couldn't be achieved with ESLint, on the contrary, it offers fewer features;
Linting is much more than checking deprecated APIs, see the aforementioned plugins, they do it and much more;
Will this tool offer rules for security, code complexity, styling and other advanced use cases? For instance, eslint-plugin-security;
ESLint is already used in multiple UI5 projects by customers, partners and SAP itself. They use a plethora of ESLint plugins including for styling (such as Airbnb). Now, they have to use 2 linting tools... Great!
The same will happen in CAP and Web Components projects. Developers will need to use ESLint for them and another tool for the UI5 bit. No reusable rules or styles. Double great!

Proposal

ESLint (js/ts) has tools and communities more than mature. What this tool will do that they don't?;
Installing ESlint works with all UI5 projects today, no need to modify them, a config file is all one needs;
Anybody who knows what linting is, uses ESLint today. The only ones NOT using ESLint are those who won't use this new tool as well because they have no idea what's it for or don't care. SAP is setting a lowest common denominator community;
Why complicate things? This is using ESLint with extra steps and doesn't make any sense;

PS: I'm sorry if this sounds too harsh, but I've spoken to multiple SAP developers working on real projects, and nobody could understand why or see the value in this new linting tool...


Spelling 'Enocde' instead of Encode

Storage Explorer Version: 1.7.0/1.6.2
Platform/OS: _Linux Ubuntu? macOS High Sierra
Bug description
Spelling mistake when a file name need to be encoded when is downloaded into a windows machine

Thanks for reporting this. Splleing is hard. 😄


[core-http] throw AbortError on request cancellation

Fixes #5659
This puts us in accordance with our guidelines. See the linked issue for more details.

@HarshaNalluru Does storage packages align to use AbortError too?

@daviwil
This also now touches identity (managed credentials)

I'll take a look :)

/azp run js - storage-blob - tests

/azp run js - storage-file - tests

/azp run js - storage-queue - tests

The storage-file failures look like an existing issue. I'm seeing failures since Thursday (after the feature/storage branch was merged into master, which is when we saw the AbortError issue as well)

@HarshaNalluru Does storage packages align to use AbortError too?
yes, @XiaoningLiu .

/azp run js - storage-file - tests

/azp run js - storage-file - tests


[Component] TextField: UIKit

Testing informations |
-- | --
Risk level ? | high (component MUST be tested by QAA)
Does this ticket need to be design reviewed ? | yes
Referent designer : | ?
Spec : | figma
Testing checklist : | for visual quality + classic checklist
UATs : | confluence in progress
configurator : | confluence in progress

This one is validated ✅  I'll move it do RFP


px4fmu-v4pro : Add support for ICM20602

Adding support for ICM-20602 on px4fmu-v4pro. ICM-20608-G is "not recommended for new designs" by Invensense and both ICs are pin-to-pin compatible.
@LorenzMeier I think this should go in V1.6

@PX4TestFlights can you fly this on a Pixracer please?

couple of flights with pixracer; no notable issues
http://logs.px4.io/plot_app?log=1ffd5f10-8787-4620-bbe7-5f5a02373c99
http://logs.px4.io/plot_app?log=799cd5c9-31f2-4475-bc7f-8f551a398d46


Review/Finalize Caltrans project charter

Resolving final questions/clarifications in the project charter.
Update 1/4/2024:
Only remaining question is getting more detail from Caltrans on the datasets they listed as in scope beyond the core PeMS dataset. Need follow-on meetings with them to get more info on how these fit into the overall project.

Assigning this to @jasonlally, next step is Jason will send this for signatures with Docusign.

This is signed and filed in our project folder now - closing


[Enhancement] Messaging toast refactor

Description
This PR is a toast refactor as @easyuxd's UX guidelines in https://github.com/SeedSigner/seedsigner/issues/606. Changes are the following:

Add toast messaging categories
General sizing refactor
Unbind icon color from text color
Dynamically adjust toast height
ToastOverlay is now available to use with no icon
New DIRE_WARNING_COLOR=#FF453A and ERROR_COLOR=#AD2013

Screenshots for some examples:

Dynamic height adjustment:

To do:

[ ] New "info" icon. I can get the seedsigner-icons.otf from #586.
[ ] More tests around the delay increase (0.2s) in the lock mechanism to ensure that a button press won’t interfere with a toast displayed immediately afterward.

This pull request is categorized as a:

[x] New feature
[ ] Bug fix
[x] Code refactor
[ ] Documentation
[ ] Other

Checklist

[ ] I’ve run pytest and made sure all unit tests pass before sumbitting the PR

If you modified or added functionality/workflow, did you add new unit tests?

[ ] No, I’m a fool
[x] Yes
[ ] N/A

I have tested this PR on the following platforms/os:

[x] Raspberry Pi OS Manual Build
[ ] SeedSigner OS on a Pi0/Pi0W board
[ ] Other

Note: Keep your changes limited in scope; if you uncover other issues or improvements along the way, ideally submit those as a separate PR. The more complicated the PR the harder to review, test, and merge.

Beautiful work, @alvroble -- thanks for doing this. The toast categories look great so far, and the dynamic height adjustment is clean!


worldwide image to tiles return a error

Describe the bug
A clear and concise description of what the bug is; Exception text (if there was).
To Reproduce
Describe steps to reproduce the behavior.
If you’re running in IDE, please also specify exact file and string number in it, that throws an Exception.
Input data info
Some info about your input GeoTIFF, causing error, at least projection. Would be better, if you can send gdalinfo screenshot.
Ideally – upload GeoTIFF, that causes the error (you can use e.g. Google Drive to share it, since GitHub has some restrictions to the attachment’s size).
Expected behavior
A clear and concise description of what you expected to happen.
Desktop:

OS: [e.g. Windows 10 x64]
Version [e.g. 1.2.0.]. Please, also specify here, if you’re building from the source and not using the pre-compiled binaries on releases page.

Additional context
Add any other context about the problem here.
I am try map tiler  worldwide image,it is return a error:System.OverflowException:“Arithmetic operation resulted in an overflow.”
the Number big the Int32 max value.

Hello, @zhongshuiyuan.
Thank you for reporting this! Starting from 15 zoom, there's too many tiles to create, so it's count oveflows the Int32 type. I think of two possible options on this:

Use Int64 (max value 9,223,372,036,854,775,807) or UInt64 (18,446,744,073,709,551,615). But reporting progress on such a large output would cost A LOT of performance because of millions and millions of system calls to report the progress to console, so there's other option;
Don't report progress if tiles count was too much (but tell the user about that decision).

Hello, @zhongshuiyuan.
Thank you for reporting this! Starting from 15 zoom, there's too many tiles to create, so it's count oveflows the Int32 type. I think of two possible options on this:

Use Int64 (max value 9,223,372,036,854,775,807) or UInt64 (18,446,744,073,709,551,615). But reporting progress on such a large output would cost A LOT of performance because of millions and millions of system calls to report the progress to console, so there's other option;
Don't report progress if tiles count was too much (but tell the user about that decision).

In the end I've decided that this problem shouldn't be fixed honestly. I thinks it is possible, that creating even 2,147,483,647 (max Int32 value in c#) can break/lead to big problems with lifetime and performance of your HDD, so it's better to throw an exception.

In the end I've decided that this problem shouldn't be fixed honestly. I thinks it is possible, that creating even 2,147,483,647 (max Int32 value in c#) can break/lead to big problems with lifetime and performance of your HDD, so it's better to throw an exception.


IgnoreDuplicates not working as intended

Bug report
On making a call  with ignoreDuplicates no header is added to the request:
 let { data, error, status } = await supabase
      .from('emails_d')
      .upsert(emails, { ignoreDuplicates: true });

Package .json:
"@supabase/supabase-js": "^1.22.6",
It looks like it was added but it's not functional

I upgraded the version and I am starting to see the header going through.


htpasswd-deployment: Increase mem limit to 90 Mi

The pod was OOMKilled with 30 Mi, so increase the limit to 90 Mi.

@projectcontour/maintainers any thoughts on this?

@projectcontour/maintainers any thoughts on this?

Seems fine to me, 🤷♂


Error on taking address of zero sized types

Seems good? I'm not sure how we can define the behavior of this

Fixed in a8e828ce24fc8b7a518454ebd1cde5e2881c75ea


Repository protocol extension

@jmnavarro thanks for contributing this, although I am not sure that this extension really helps.
The intention of the protocol is to let the developer knows that he has to implement this based on his needs (for the project I am using this library the !==nil check is not enough). Probably, some developers will miss this functionality as it is camouflaged now and they will start opening new issues for no reason.
Although, I can see your intention for making this awesome library easier to be integrated in a project :+1:,  with that being said, if everyone agrees that your PR will work for most cases then let's merge it.
Thanks for sharing it ;)

This pull request breaks the demo app code. I still had to implement hasPasscode in order to conform to the PasscodeRepositoryType protocol, so not sure this code change is necessary.

@ziogaschr I understand your point, no problem if you consider to close the PR (I see it merged though)

@jmnavarro I am very happy that you took the time to make this PR and to see that this repo is being used :+1:  . @yankodimitrov did a great job. Although let's consider all together if this PR will help, and if not we can revert it.

guys, let's link some testing engine to our repo
we have to know what's happening with the project after every PR and update tests or refusing PRs

Good point @velikanov. The demo code is broken because of this PR, I will revert it. Let me know if you want it back.