Faulty installation of Ubuntu (Waiting for the essential requirement 1 of 5: "ssh"): util.py[WARNING]: Failed to create user admin

Description
Steps to reproduce:
limactl create template://ubuntu --name=ubuntu
or
limactl create template://ubuntu-lts --name=ubuntu
limactl start ubuntu
Output:
INFO[0000] Using the existing instance "ubuntu"
INFO[0000] QEMU binary "/usr/local/bin/qemu-system-x86_64" seems properly signed with the "com.apple.security.hypervisor" entitlement
INFO[0001] [hostagent] hostagent socket created at /Users/admin/.lima/ubuntu/ha.sock
INFO[0001] [hostagent] Starting QEMU (hint: to watch the boot progress, see "/Users/admin/.lima/ubuntu/serial*.log")
INFO[0001] SSH Local Port: 56893
INFO[0001] [hostagent] Waiting for the essential requirement 1 of 5: "ssh"
INFO[0011] [hostagent] Waiting for the essential requirement 1 of 5: "ssh"
...

Obviously, there is a problem with creating the admin user in cloud-init
Snippet from ~/.lima/ubuntu/serial.log:
[   22.661276] cloud-init[586]: 2023-11-28 05:48:29,178 - util.py[WARNING]: Failed to create user admin
...
[   77.605933] cloud-init[2346]: + loginctl terminate-user admin
[   77.616303] cloud-init[2346]: Failed to look up user admin: No such process
...
[   77.864749] cloud-init[2346]: + grep -q # Lima BEGIN /home/admin.linux/.profile
[   77.867782] cloud-init[2346]: grep: /home/admin.linux/.profile: No such file or directory
[   77.872246] cloud-init[2346]: /mnt/lima-cidata/boot/20-rootless-base.sh: 10: cannot create /home/admin.linux/.profile: Directory nonexistent
[   77.878411] cloud-init[2346]: LIMA| WARNING: Failed to execute /mnt/lima-cidata/boot/20-rootless-base.sh
...
[   77.911972] cloud-init[2346]: + id -g admin
[   77.914698] cloud-init[2346]: id: ‘admin’: no such user
[   77.917552] cloud-init[2346]: + gid=
[   77.919394] cloud-init[2346]: LIMA| WARNING: Failed to execute /mnt/lima-cidata/boot/25-guestagent-base.sh
...

Alpine Linux works as expected

Is this issue specific to your username "admin"?

I don't have a clue. If that is, it's really weird 😐

Do you see other errors around the  util.py[WARNING]: Failed to create user admin line?

Just a deprecation notification a few lines before:
[   11.825236] cloud-init[575]: 2023-11-28 06:25:06,549 - log.py[DEPRECATED]: Deprecated cloud-config provided:
[   11.827869] cloud-init[575]: ca-certs:  Deprecated in version 22.3. Use ``ca_certs`` instead.
[   11.830399] cloud-init[575]: 2023-11-28 06:25:06,549 - schema.py[WARNING]: Invalid cloud-config provided: Please run 'sudo cloud-init schema --system' to see the schema errors.
[   12.810965] cloud-init[575]: 2023-11-28 06:25:07,534 - log.py[DEPRECATED]: Key 'ca-certs' is deprecated in 22.1 and scheduled to be removed in 27.1. Use 'ca_certs' instead.
[   12.820379] cloud-init[575]: Updating certificates in /etc/ssl/certs...
[   13.789139] cloud-init[575]: 0 added, 0 removed; done.

Obviously, there is a problem with creating the admin user in cloud-init

Yes, I think the problem is that cloud-init by default creates both a user and a group with the same name, and the admin group already exists on Ubuntu.
This behaviour can be disabled with the no_user_group option in the user definition in user-data:
users:
- name: admin
  no_user_group: true
  ...

I'm not sure why we need to create the user group in Lima; maybe we should disable this for all users. What do you think @AkihiroSuda? I just wonder if this will break existing use-cases, and we would have to make it configurable, which I would like to avoid. I would rather say: "Any instance created by Lima 1.0 or later will not create an additional group for the user".
Otherwise I expect any user names that match existing group names can't be used. We do have logic to use lima as the user name when the host username is invalid on Linux, but we can't know the list of existing group names until the instance is booted, as which time it is too late to switch the username.

Hmm, we also have #1015 to allow the user to specify a different username for the VM. Maybe that is the best way to deal with this because even when we avoid creating the group name, we would still have the potential clash with existing user names in the Linux distro (bin, daemon, sys, etc), which would still throw an error unless #1015 is implemented.

I'm not familiar with Lima, so the question is: what is the reason of falling back to lima username instead of using lima by default (i.g. not using host username at all).

what is the reason of falling back to lima username instead of using lima by default (i.g. not using host username at all).

Lima provides the illusion of seamlessly blending the host and the guest environment. You can mount volumes from the host and use them inside the guest. It makes it look like nerdctl actually runs on the host which technically it can't. So showing the host username as the owner of the mounted filesystems helps to maintain this feeling of an integrated environment.
So I think providing a mechanism for users to override the username should help to deal with the conflicting names that we can't detect automatically (but maybe we should have a hard-coded list of "known" legacy Unix usernames that are almost always unavailable).
Note that you wouldn't have to specify the alternate username all the time; you could create a default.yaml or override.yaml to specify the alternative name once, and then limactl start would automatically use it with all the templates.

@jandubois, could you provide with the snippets of default.yaml and override.yaml? What exactly should I put there and where these file should be placed?

Apparently "admin" is a special group only created in Ubuntu, which is why it doesn't fail Alpine or Fedora.
https://wiki.ubuntu.com/PrecisePangolin/ReleaseNotes/UbuntuDesktop#PrecisePangolin.2FReleaseNotes.2FCommonInfrastructure.Common_Infrastructure

Up until Ubuntu 11.10, administrator access using the sudo tool was granted via the "admin" Unix group. In Ubuntu 12.04, administrator access will be granted via the "sudo" group. This makes Ubuntu more consistent with the upstream implementation and Debian. For compatibility purposes, the "admin" group will continue to provide sudo/administrator access in 12.04.

The current CIDATA is missing entries for group, so some scripts are currently hardcoding group==user assumption.
LIMA_CIDATA_USER=anders
LIMA_CIDATA_UID=1000

Could probably be a good idea to also export the group, in parallel to whatever the resolution is for system names....
LIMA_CIDATA_GROUP=anders
LIMA_CIDATA_GID=1000

some scripts are currently hardcoding group==user assumption

As I mentioned above, this is the default behaviour of cloud-init.
If we don't want to disable this behaviour (via no_user_group: true), then my suggestion would be that we enhance the "valid user name" check to also check against a list of "well-known" default user names and groups, and we will use the fallback username lima if the host user name matches any of them.
This would of course be guarded by the new lima-version mechanism from #2107, so only applies to instances created by 0.20.0 (or 0.21.0, or whenever this gets implemented).

I just meant that we could add another variable (or two), so that users could update (or improve) their scripts
Similar to using $HOME instead of hardcoding /home/$USER, or whatever the best analogy would be here.

I just meant that we could add another variable (or two), so that users could update (or improve) their scripts

I see, but not sure why that is necessary. First we have the guarantee from cloud-init that the group will be the same as the user. But you can also always use id to query the proper value. You can do this even from a system provisioning script, as long as you have the username:
# export LIMA_CIDATA_USER=jan
# su -c 'id -g' $LIMA_CIDATA_USER
1000
# su -c 'id -g -n' $LIMA_CIDATA_USER
jan

But even if these values would not be readily available, we should first turn provisioning scripts into templates instead of making things available as unsupported env variables (see #2055).

my suggestion would be that we enhance the "valid user name" check to also check against a list of "well-known" default user names and groups

My suggestion was supposed to address the issue of users with conflicting user names. We have seen at least 3 instances of people using admin on the host. By using the lima fallback automatically we would make Lima work for them out-of-the-box.


Add a note to the audit log enablement

Added a note that the Audit log is not ready right after it has been turned on.
When you enable it, there is a note "WARNING: The admin audit log configuration change you specified could take up to 60 minutes to take effect." in PowerShell

Docs Build status updates of commit d0c7dd8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

office-365-management-api/troubleshooting-the-office-365-management-activity-api.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@Michal-Ziemba PR merged. Apologies for the delay. I just saw this PR.


Python Debugger is overidding PYTHONPATH

Environment data

VS Code version: 1.27.1
Extension version (available under the Extensions sidebar): 2018.8.0
OS and version: Windows 7
Python version (& distribution if applicable, e.g. Anaconda): Anaconda 5.2

Actual behavior
When I debug a python file in vscode my PYTHONPATH variable is overridden.
here is the relevant line shown on the terminal.
set "PYTHONPATH=c:\Users\username.vscode\extensions\ms-python.python-2018.8.0\pythonFiles\experimental\ptvsd"
Expected  behavior
I would expect the PYTHONPATH to be appended to not overridden.
Steps to reproduce:

Set the PYTHONPATH variable to something
debug a script with the follow code:

import sys
print(sys.path)

You should not see the path that you set in the PYTHONPATH

I can't remember if we supported this in the old debugger (if we did then this is a regression). Otherwise I think of any reason off the top of my head to not append PYTHONPATH as appropriate.

Thank you for the quick reply. Hopefully this can be resolved.
If someone's python script/program relies on the PYTHONPATH being set then the current behavior of the debugger makes it unusable.

@tkuraku
Please could you confirm you have a value in PYTHONPATH?

@DonJayamanne
I do have a value in PYHTONPATH. The reason I noticed this was that when I run my script from the terminal everything works as expected. However, when I ran the debugger my script failed with an import error on one of my modules that is in the PYTHONPATH. (FYI debugger works fine in pycharm).

I'm having the same issue and it used to work a couple of days ago.
(vscode was updated to 1.26.1 in between.)
running my script in either ipython or python in the terminal works as expected. when running the debugger with these settings
        {
            "name": "Python: Current File",
            "type": "python",
            "request": "launch",
            "program": "${file}"
        },

shows that the PYTHONPATH is being overwritten to
"PYTHONPATH=/home/myname/.vscode/extensions/ms-python.python-2018.8.0/pythonFiles/experimental/ptvsd"

adding individual softlinks to /home/myname/.vscode/extensions/ms-python.python-2018.8.0/pythonFiles/experimental/ptvsd to the required modules works for me as a temporary work-around
ln - s /home/path/to/my/module /home/myname/.vscode/extensions/ms-python.python-2018.8.0/pythonFiles/experimental/ptvsd/

@Argysh @tkuraku
Is PYTHONPATH defined in your python environment or is it a global environment variable?

I believe the solution is not to invoke PTVSD as a package, but to run a python file that'll launch PTVSD with the path setup (e.g. sys.path.append.... or similar).

My PYTHONPATH is a global environment variable. I think that once the script has started that anything added with sys.path.append is fine.

@brettcannon
We are appending PTVSD to the exiting PYTHONPATH. However it looks like we're not getting the value inside the extension process. Could be related to #2363, #1403, and others

@DonJayamanne are we running a separate process to get PYTHONPATH instead of just querying the environment that Node has in the extension itself?

are we running a separate process to get PYTHONPATH instead of just querying the

No we aren't running a separate process.
We just query the environment that node has in the extension.

My PYTHONPATH is a global environment variable as well (set in .bashrc)

After the recent upgrade, I have noticed the same issue on Linux (CentOS 7, Python 3.4, VS Code Version 1.12.1)

$PYTHONPATH is defined as a global (bash) environment variable before starting a VS Code from the command line.

VS Code picks it up. PYTHONPATH value can be observed in the VS Code terminal window
sh-4.2$ echo $PYTHONPATH
/host/viz-json-service/src/code:/host/viz-json-service/src/generated:/host/viz-json-service/src/tests

Here is my lunch.json
{
"name": "Python: run_tests.py",
"type": "python",
"request": "launch",
"program": "${workspaceFolder}/src/run_tests.py",
"console": "integratedTerminal",
}

However, when I start debugger (F5), the PYTHONPATH is overwritten:
sh-4.2$ cd /host/viz-json-service ; env "PYTHONIOENCODING=UTF-8" "PYTHONUNBUFFERED=1" "PYTHONPATH=/headless/.vscode/extensions/ms-python.python-2018.8.0/pythonFiles/experimental/ptvsd" /usr/bin/python3 -m ptvsd --host localhost --port 46405 /host/viz-json-service/src/main.py

As a result, a package fails to load
Exception has occurred: ImportError
No module named 'code.viz_json'; 'code' is not a package
File "/host/viz-json-service/src/tests/viz_json/test_util.py", line 6, in

The same code run from a command line works fine:
$ python3 src/run_tests.py
test_py_zgy (test_py_zgy.TestUtils) ... ok

the code structure is as following:
root/
code
init.py
viz_json
init.py
util.py
tests
....

--
...

@dmitriyrepin FYI that the terminal in VS Code is a fresh terminal and shell (i.e. a separate process and such), so the fact the terminal picks up PYTHONPATH does not necessarily correlate that VS Code itself picked it  up.

FYI: I re-installed VS Code 1.26.1 + plugins in my container - same issue.
However, after installing VS Code 1.24.1 everything works as expected (so, the issue is a regression - it used to work with the old debugger):
sh-4.2$ cd /host/viz-json-service ; env "PYTHONIOENCODING=UTF-8" "PYTHONUNBUFFERED=1" /usr/bin/python3 /headless/.vscode/extensions/ms-python.python-2018.6.0/pythonFiles/PythonTools/visualstudio_py_launcher.py /host/viz-json-service 46428 34806ad9-833a-4524-8cd6-18ca4aa74f14 RedirectOutput,RedirectOutput /host/viz-json-service/src/run_tests.py

Yes, I try to roll back to before version, i can start debugging code

I was able to verify that my PYTHONPATH env var shows up in sys.path in a debugged program.


chore: siwe v2 support

Resolution for #762
Changes:

Bumped siwe peer dependency to ^2.1.3
Bumped  iron-session and next-auth to latest
Amended examples

Blockers:

awaiting siwe v2 to leave beta: https://docs.login.xyz/libraries/typescript/v2.0-beta

I'm really looking forward this commit!
I've tried to update my project with rainbowkit-siwe-next-auth to siwe 2.1.x and it doesn't raise the Sign.

@nfts2me Was blocked by the siwe@v2 package being stuck in beta, but it looks like there is movement now. Will revisit! https://github.com/spruceid/siwe/releases/tag/v2.1.4-beta

Can confirm here - you should be good to go with the latest package to update the guide


WebSocket actions improvement

Summary
I was looking to get posts as soon as the WS reconnects and fixing up a bit the reconnect logic, and I ended up modifying a bit of code to make the WebSocket actions a bit better.
Ticket Link
https://mattermost.atlassian.net/browse/MM-12832

Ticket to move the dispatcher here https://mattermost.atlassian.net/browse/MM-13039

@jwilander @hmhealey refactor this a bit more to remove the dispatch from the client

rebased with the latest master that includes the dialogs

I'm not QA but I did test it using the mobile app and it looked good, but yeah, I can submit a PR to the mobile repo and build it so QA can test


Publish Composition Version on NPM

Describe the bug
I can't update the package because the old version is on npm
Expected behavior
Version 1.1.0 on NPM
Versions
Libraries:

Vue: 2.#.#

https://www.npmjs.com/package/vue-typed-emit

@stalker3343, Sorry, I didn't expect you want it to be published so quickly 😅<EMAIL_ADDRESS>is available on NPM.


 Also, I'm not using Fedora myself these days, so I'm not testing and maintaining it. Therefore, I'm removing it until someone adds it again and commits to maintaining it.

Makes sense :+1:


Update package.json to include missing fields

Adds contributors, keywords, directories, and moves mocha to devDependencies. Also adds package-lock.json as recommended by npm.

nice! 👍🏻 +5


Support Ceph

Current situation
The Ceph mount tool is not available and probably something else is missing (e.g., check kernel config).
Impact
No Ceph for network storage, just NFS
Ideal future situation
Ceph is supported
Implementation options
FUSE vs kernel driver
Additional information

I would prefer the kernel option since it seems like the required kernel modules are already in place. I guess just the mount helper tool mount.ceph is missing. I will try to compile it somewhere else and then test it on Flatcar manually.

So it seems like Ceph itself is already supported by the kernel in Flatcar Linux. It is just the helper mount.ceph missing which is not a requirement for mounting according to https://docs.ceph.com/en/latest/cephfs/mount-using-kernel-driver/.
I was able to successfully mount CephFS via IP addresses without the helper. But taking the helper from Ubuntu's ceph-fs-common also worked.


Check if container belongs to a network when using exposeHostPorts

Makes it possible to use reusable container and exposeHostPorts at the same time.
Closes:

#7335
#3457

Doesn't break solutions proposed in:

#3081

Hi @TomDeBacker I've run through this same issue. Do you have any updates from the test containers team? Would be great to get this merged 🙏🏽

Thanks for your contribution!


Apply RBAC to efk and create fluentd.conf

Making fluentd.conf as configmap to change configuration.
Change elasticsearch rc to deployment.

ci check this


Removes defence skill modal

As the defence skill is calculated based on (rank + combat + defence modifier), it didn't make sense to have the skill change modal for defence as it doesn't do anything.
I've removed the initial defence value and defence modal

This had been on my todo list for a while, thanks for the fix.
There are a few rules remaining to nail down properly but it's getting there!


Parser Error: Could not resolve class "NeuralNetwork", because of a parser error.

Anyways I solved it apparantly i forgot to out ":" at end of if statement


cannot launch vscode on my mac

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: system version: Big Sur 11.4
OS Version: no idea, cause it happend so suddently

Steps to Reproduce:

I just updated a plugin which requires to restart vscode

https://github.com/user-attachments/assets/349369d5-4ab3-4a2d-b765-d7b31629e186

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.91.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

#221528

@elin-croft please provide the output of code --version

"zsh: command not found".  Do I need to add some specific folder to my PATH variable or install something?

Yeah there is a helper in the application to perform that action Shell command: Install code command in PATH. You can instead run the following
/Applications/Visual\ Studio\ Code.app/Contents/Resources/app/bin/code --version

Can you re-install the latest 1.91.1 from https://code.visualstudio.com/updates/v1_91 which should resolve the issue. Related to https://github.com/microsoft/vscode/issues/219893

1.91.1 works. Thx 🫶


Report failure cause for some assertion failures

This seems to be a regular failure on the nightlies but the logs don't contain any information as to the cause.

Processing this pull request

Pull request rejected by merge tool. No change entry file found

Processing this pull request

Merged into branches master in e5283f5f109b3869d99457ef700265f57d70f057

Processing #6314.


Basic Authentication for _utils does not work if require_valid_user_except_for_up is set

Description
When setting require_valid_user_except_for_up instead of require_valid_user in the CouchDB configuration, the system does not prompt for username and password when accessing the _utils endpoint. Instead, a 401 is returned. {"error":"unauthorized","reason":"Authentication required."}
When changing the configuration back to require_valid_user, the basic authentication prompt appears as expected.
Steps to Reproduce

Set require_valid_user_except_for_up = true in local.ini
Restart CouchDB to apply the configuration change.
Attempt to access the _utils endpoint (e.g., http://localhost:5984/_utils).

Expected Behaviour
The _utils endpoint should prompt for authentication
Your Environment

CouchDB version used: 3.3 (Docker)
Browser name and version: Firefox
Operating system and version: MacOS

{
  "couchdb": "Welcome",
  "version": "3.3.3",
  "git_sha": "40afbcfc7",
  "uuid": "3a7f2e8d1c9b4f6e0d5a2c8b7f3e1d9a",
  "features": [
    "access-ready",
    "partitioned",
    "pluggable-storage-engines",
    "reshard",
    "scheduler"
  ],
  "vendor": {
    "name": "The Apache Software Foundation"
  }
}

local.ini
[couchdb]
single_node=true
uuid = 3a7f2e8d1c9b4f6e0d5a2c8b7f3e1d9a

[chttpd]
require_valid_user_except_for_up = true
bind_address = any
authentication_handlers = {chttpd_auth, jwt_authentication_handler}, {chttpd_auth, cookie_authentication_handler}, {chttpd_auth, default_authentication_handler}
enable_cors = true

[jwt_keys]
....

[jwt_auth]
roles_claim_path = cognito:groups

[couch_peruser]
enable = true

[admins]
admin = -pbkdf2-...

[cors]
origins = *
headers = accept, authorization, content-type, origin, referer
credentials = true
methods = GET, PUT, POST, HEAD, DELETE

After writing this, I discovered that the issue seems to be resolved when both require_valid_user and require_valid_user_except_for_up are set to true.
If this is the intended behavior, the documentation may be misleading.

Thanks for posting the resolution. It was not obvious to me that both were required.
I'm adding a link to https://github.com/apache/couchdb/issues/1305 because that was the recommended solution for CouchDB 2.x, but the option no longer exists in 3.x


Header column is empty should tell me which column

Be nice to be told which header column is empty ...
+----------------+---------------+-------------+-------------------------------------------------------+
| result_name    | result_id     |   row_index | result_message                                        |
+================+===============+=============+=======================================================+
| Missing Header | structure_001 |           0 | Headers column is empty.                              |

👍

@georgiana-b can you have a look at this?

goodtables.next
[-,3] [blank-header] Header in column 3 is completely blank

Fixed in https://github.com/frictionlessdata/goodtables-py/tree/next
next will be tested for several weeks now before we merge onto master.


SCM history: tree find (Ctrl+F) doesn't highlight matches

Type: Bug
Without filter mode selected, down-arrow from widget correctly navigates between matching nodes in the tree but the matches are not highlighted.

VS Code version: Code - Insiders 1.87.0-insider (00124e9e5830e3efc897db71c781899f8a676295, 2024-02-16T16:33:57.085Z)
OS version: Windows_NT x64 10.0.22621
Modes:

Fixed in the latest VS Code Insiders release.

@gjsjohnmurray how were you triggering the find in that view? cmd+f doesn't work for me

@meganrogge, it looks like the default keybinding was changed to cmd + option + f.

@meganrogge, it looks like the default keybinding was changed to cmd + option + f.

https://github.com/microsoft/vscode/pull/210634#issuecomment-2063570698


[usecase] Add permissions in manifest.json

Add permissions:
CAMERA
MODIFY_AUDIO_SETTINGS
RECORD_AUDIO
ACCESS_FINE_LOCATION
VIBRATE
CHANGE_WIFI_STATE
CHANGE_NETWORK_STATE
BILLING
READ_EXTERNAL_STORAGE
WRITE_EXTERNAL_STORAGE
READ_CONTACTS
WRITE_CONTACTS
Update xwalk_permissions to xwalk_android_permissions
Update xwalk_version to xwalk_app_version
Failure Analysis:
Not find permissions for Notifications and Midi SysEx.

Impacted tests(approved): new 0, update 10, delete 0
Unit test platform: Crosswalk Project for Android 18.46.453.0
Unit test result summary: pass 9, fail 1, block 0
https://crosswalk-project.org/jira/browse/XWALK-6092

LGTM, @jennycao, please also review.

LGTM


Phoenix.ConnTest.get/3 breaks on queries with compiled documents discovery

I am currently testing compiled queries in Absinthe with Phoenix.ConnTest.post/3. When no query document is sent, Plug.Adapters.Test.Conn will add --plug_conn_test-- to the request body (with a header of {"content-type", "multipart/mixed; boundary=plug_conn_test"})
In my test case, I have:
params = %{variables: %{"id" => thing.uuid}, action: "name_of_compiled_query"}

response =
  conn
  |> post(@graphql_path, params)
  |> json_response(200)

which yields the result:
%{
  "errors" => [
    %{
      "locations" => [%{"column" => 0, "line" => 1}],
      "message" => "illegal: --"
    }
  ]
}

This happens because, when Absinthe.Plug.DocumentProvider.Default comes before the compiled document provider, it pattern matches for any non-nil document. This issue is fixed by putting MyApp.Schema.Documents.Compiled first. If this is expected behavior, it would be very helpful to make that clear in the documentation.
One possible solution - if body is used as the default document when a query parameter is not provided, content-type boundaries should be taken into account. I assume that Absinthe.Plug.DocumentProvider.Default.process/2 was intentionally meant to match non-nil values, so changing that does not seem desireable.
If this is expected behavior, and the default document provider should go last, it would be helpful to explain this in the documentation.
Versions:

absinthe 1.4.14
absinthe_ecto 0.1.3
absinthe_plug 1.4.6

Edit: this issue also may be more appropriate for the plug repo

We use compiled documents and indeed the compiled document provider comes before the Absinthe.Plug.DocumentProvider.Default. This is expected behaviour afaik.
Maybe you could file a PR to make this more clear in the docs.

I added a quick documentation note to address this ordering issue: https://github.com/absinthe-graphql/absinthe_plug/commit/1afbcf245298141c9d27712d993d4146af112736
Thanks!


Feature Request

Is it possible to add the two features below, thank you in advance.

Window Top
Add separator for long numbers, Comma or space

"always on top" button will be available from next version.
Would like to support in the future.

It would be nice to add function suggestions and parentheses auto-completion, just like programming.😊

Auto-completion was reflected in v0.2.8527.34374-beta.


Duplicated item models when using BnB with Geckolib

Minecraft 1.20.1
Fabric loader 0.14.21
Fabric API 0.86.1
Bovines and Buttercups 1.9.0 (latest)
Geckolib 4.2.1 (latest)
Some items, most notably bowls of nectar, have a copy of the item model near them in inventory, in item frames, as dropped items, etc. The copy seems to be shifted by a few pixels to the upper left corner and a bit backwards (it appears to be behind the item frame).
This occurs only when Geckolib is installed. Without it, the items render correctly.
Affected items include: all bowls of nectar and custom flower, custom mushroom, custom mushroom block in the item form (not placed).
A zombie holding the item:

Items in item frames:

Items in invisible item frames:

Items laying on the ground with hitboxes shown:

Other items:

GeckoLib issue.

Fixed in Geckolib 4.2.2


Parse git remote name to determine Heroku app

Several reports have matched #72, which was affecting the restore
command for some users. The cause turned out to be local folder names
not matching the name of the Heroku app. Now that Heroku tracks Git
remotes, we can take advantage of the Git remote to correctly identify
the app name even the local folder name doesn't match.
While this requirement has existed for the entirety of Parity's
existence, it stopped being a problem for most commands once Heroku
introduced remotes. I inadvertently reintroduced this requirement in
ad2c21a42fb85bd00ec58d7326725f0a1d5a7771 when we started automatically
bypassing confirmation arguments for restores to non-production
environments.
Fix #72.

:+1:

@gabebw thank you for the several rounds of thoughtful feedback!


Why is the block log in job manage in the way of snapshot reading?

Why is the block log in job manage in the way of snapshot reading?

Closing this issue because it was created before version 2.3.0 (2022-11-10). Please try the latest version of Flink CDC to see if the issue has been resolved. If the issue is still valid, kindly report it on Apache Jira under project Flink with component tag Flink CDC. Thank you!


ARROW-1296: [Java] Fix allocationSizeInBytes in FixedValueVectors.res…

Fix allocationSizeInBytes in FixedValueVectors.reset

cc @siddharthteotia

I started writing a test for this but realized allocationSizeInBytes is private and there is no getter method.
I am not sure if we want to introduce a package private getter methods for the field for testing. @siddharthteotia @wesm do you know who is a good person to comment on this?

Can we use Reflection for testing?

I am actually against using Reflection for testing in this case, according to https://stackoverflow.com/questions/2811141/is-it-bad-practice-to-use-reflection-in-unit-testing.
I think it's better to add a package private method for this. @siddharthteotia what do you think?

Ping @siddharthteotia, what do you think of the updated change?

1

+1 for me also


python3Packages.opentelemetry-proto: use protobuf4

Fixes #349361.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Upstream has added support for Protobuf 5 but it hasn't made its way into a release yet: https://github.com/open-telemetry/opentelemetry-python/pull/4206

Seems like a number of dependencies on this package have issues building after this change, giving errors like this:
ound duplicated packages in closure for dependency 'protobuf':
  protobuf 5.28.2 (/nix/store/3sgy4wi9zab1j5cpb595aw12d7qjkbkc-python3.12-protobuf-5.28.2)
    dependency chain:
      this derivation: /nix/store/qrqbppkxh0mcz5cbpac303mjvfsrfrp8-python3.12-opentelemetry-exporter-otlp-proto-grpc-1.26.0
      ...depending on: /nix/store/dgsq8gik07pmdxsc51rv145rck0j4z61-python3.12-googleapis-common-protos-1.63.1
      ...depending on: /nix/store/3sgy4wi9zab1j5cpb595aw12d7qjkbkc-python3.12-protobuf-5.28.2
  protobuf 4.25.4 (/nix/store/7q447r8za2233grairjgkrr6al49v0ql-python3.12-protobuf-4.25.4)
    dependency chain:
      this derivation: /nix/store/qrqbppkxh0mcz5cbpac303mjvfsrfrp8-python3.12-opentelemetry-exporter-otlp-proto-grpc-1.26.0
      ...depending on: /nix/store/73scs999mwv61ilb9wczbhy4p3cvnpra-python3.12-opentelemetry-exporter-otlp-proto-common-1.26.0
      ...depending on: /nix/store/nx5z9i75d0sawd5p5d6djm52gr45fxpg-python3.12-opentelemetry-proto-1.26.0
      ...depending on: /nix/store/7q447r8za2233grairjgkrr6al49v0ql-python3.12-protobuf-4.25.4

Looks like this has already been fixed on staging: #348738.


Updated installation after feedback from hackathon

I know the commit history appears long, but it should go away when I squash and merge, shouldn't it?

If you take care to remove all the useless comments than it should collapsed. But why is this happening in the first place?

I addressed the code review comments, PTAL.

But why is this happening in the first place?

 Try git rebase upstream/master.
Also, generally speaking pull = fetch + merge in git so the git fetch upstream isn't necessary unless you want to pull down all branches in addition to master.

@ayj The rebase deletes all my changes, so I will not do it at this time.


Conda package

Adds conda recipe and updates CI jobs to build conda packages for OSX, Linux, and Windows.
Supersedes https://github.com/plotly/orca/pull/108 due to CI permission issues.

@etpinard green here too! Was there anything else you wanted to look over for this one?

@jonmmease have you tested out https://github.com/plotly/orca/pull/113/commits/d6f9c1ecf3ecd51691756da4f8db5fbcbe0d4326 on a linux machine? If not, I can do it no problem

No I didn't, that's good point.  I should also check whether there's an analogous issue on Windows.
I'll test both using my same Python script and let you know what I find.

@etpinard I tested in on Ubuntu and it works just like on OS X.  I tested it on Windows 10 and I had to use some different logic from Python to kill the process, but I was able to so there's nothing that needs to change on the Windows side.
Anything else come to mind?

I tested in on Ubuntu and it works just like on OS X.

Amazing!

ything else come to mind?

As this PR updates our README with new installation instructions, I'll like to wait until the conda channel thing is sorted out (cc @chriddyp ) before merging this thing and quickly release v1.1.0 right after.

I could back-out the README change and put that in a new PR for the 1.1.0 release.  If we bump the version number in that branch we could go ahead and publish the conda packages, and test them from the anaconda channel, before merging it into master.  We could also do the changelog on that branch (I didn't put anything in there for the --graph-only option yet).

I could back-out the README change and put that in a new PR for the 1.1.0 release.

I'd say do whatever is easier for you. If you think having these commits on master now will help your devflow, then revert back the README changes and merge this thing :dancer:

If we bump the version number in that branch we could go ahead and publish the conda packages, and test them from the anaconda channel, before merging it into master. We could also do the changelog on that branch (I didn't put anything in there for the --graph-only option yet).

I usually bump the package version on my local copy of master and push it (along with the git tags) straight to origin master. I update the changelog in a single commit in one go (no need to do during every PR) before the version commit and copy its content to https://github.com/plotly/orca/releases. The complete checklist is here and we essentially use the same one for plotly.js
That said, I'm open to suggestions if you think we could do better.

Thanks for explaining, that makes sense. I'll go ahead and revert the README change and then merge this, that way I can start building the conda packages I need for plotly.py dev off of master.
Do you want to just pull the README off of this branch when you do the release?

Do you want to just pull the README off of this branch when you do the release?

Yep. I'll do that. Thanks!

Once you're happy with the state of master, I'll release v1.1.0.

Brilliant :dancer:


Exploring and Schedule features

I think it would be great to have a section in the home for Groups and Users you're following to see what's upcoming on the front page. Another nice thing to have is some sort of calendar system for Groups and their events, Users and what events they're playing at and at what time, and an overall calendar one for all the groups you're following and one for all the users you're following.

I really like the calendar idea, although it'll take a decent amount of development resources. I'll start fleshing out some concepts soon.
I can probably combine this into a "feed" page, and also have a user's followed events, groups, and users appear on their profile.
I might need to add a "private profile" option for those who don't want the stuff they're following to be public.


Removing redundant topn declaration from Julia implementation

topn is already a global constant so the redefinition is a bit redundant. Using const declared global should be fine for performance, and it is in my local enviroment, but it is worth checking on the virtual machine
edit: also added one(T) and zero(T) instead of 1 and 0, it was already done inside one of the loops where it counts more for performance, here I added it more for consistency, but it should be (almost unnoticeably) faster

That one is actually needed if I remember correctly

Our local testing sometimes miss the CI machines performance characteristics, please stop micro adjusting it for style or something else

For example: https://github.com/jinyus/related_post_gen/pull/182

wow, you are totally right, I rerun the main branch vs. this PR (before last commit) and saw a regression, sorry for that, strangely enough I didn't see it the first time I ran the benchmarks, now I took extra care to benchmark and it should be fine. I think I learnt what you said in the other PR :D


New Crowdin updates

:tada: This PR is included in version 2.4.13 :tada:
The release is available on:

GitHub release
v2.4.13

Your semantic-release bot :package::rocket:


reality count not getting updated on auto reality

I'm playing from https://ivark.github.io/AntimatterDimensions/ (in vivaldi).
I've just bought Nameless one softcap.
Auto reality is set to 1e41 and while it's reached in only a couple seconds, Reality Machines count doesn't go up...
AD Save, Slot 1 #194 (2023-5-19).txt
(the date and save number is a bit bogus - I've been overwriting the file for quite awhile)
The count does properly go up with manual realities.

A combination of the "instantly reality when no glyphs pass the filter" setting being on and your glyph filter settings being set to "Lowest Total Glyph Sacrifice" mode means that your auto-reality will always trigger immediately as soon as you unlock the reality study. This means that it doesn't have any extra time to build up many resources, and it will reality before many of your numbers get big.
In this case your RM is significantly lower than it would be than if you let it run a bit more before auto-realitying. You can see in your "Past Prestige Runs" subtab under statistics that these short realities are only actually giving you about e9 RM, which won't be contributing anything to your current total of e42. It might be a bit obtuse, but this is behaving exactly as intended.

...well, the explanation is sound, the way things work is silly....
Those settings are done while I want to farm RM and don't care about glyphs (cause they're significantly lower level than previously farmed). Having to remember to manually tick off "instant reality" besides just setting to auto-sacrifice is an extra click...
Technically, both are on the same tab, but as you've said, it's a bit obtuse...
Does it ever make sense for these two to apply simultaneously ?

I don't think so - I take a look at making "Lowest sacrifice" mode disable the auto-reality filter setting, adjusting the auto-reality settings to be more comprehensive on the automation tab, and then adding a few more bits of information text as needed.


fails react

because obfuscation happens before the code is transpiled to JS, the process fails.
if it was at the very end then it would work for pretty much anything.

might have to do with issue  #1 and #2
https://github.com/jabuco/parcel-plugin-obfuscate/issues/1#issuecomment-788976185

If this issue still persisted in v2.1.1, I will reopen it.
I tested with react + react-dom which worked fine.


Unable to figure out camera / stream name for telegram photo message

I manually set up the WebRTC component on my HASS installation. I was able to add the webrtc card to lovelace, and it shows my camera stream just fine. In the URL I was using the direct RTSP-url which is provided by the camera, not some form of go2rtc URL.
When I go to the go2rtc UI, I see the following:
https://imgur.com/a/yKninb5
So what would be the pendant to "camera1" given in the telegram example?
I have tried to use "camera1", but this gives me:
Aug 11 11:32:22 raspberrypi hass[1101]: 2023-08-11 11:32:22.230 WARNING (SyncWorker_0) [homeassistant.components.telegram_bot] Status code 404 (retry #1) loading http://<IP_ADDRESS>:1984/api/frame.jpeg?src=camera1
Aug 11 11:32:22 raspberrypi hass[1101]: 2023-08-11 11:32:22.252 WARNING (SyncWorker_0) [homeassistant.components.telegram_bot] Status code 404 (retry #2) loading http://<IP_ADDRESS>:1984/api/frame.jpeg?src=camera1
Aug 11 11:32:22 raspberrypi hass[1101]: 2023-08-11 11:32:22.267 WARNING (SyncWorker_0) [homeassistant.components.telegram_bot] Status code 404 (retry #3) loading http://<IP_ADDRESS>:1984/api/frame.jpeg?src=camera1
Aug 11 11:32:22 raspberrypi hass[1101]: 2023-08-11 11:32:22.282 WARNING (SyncWorker_0) [homeassistant.components.telegram_bot] Status code 404 (retry #4) loading http://<IP_ADDRESS>:1984/api/frame.jpeg?src=camera1
Aug 11 11:32:22 raspberrypi hass[1101]: 2023-08-11 11:32:22.296 WARNING (SyncWorker_0) [homeassistant.components.telegram_bot] Status code 404 (retry #5) loading http://<IP_ADDRESS>:1984/api/frame.jpeg?src=camera1
Aug 11 11:32:22 raspberrypi hass[1101]: 2023-08-11 11:32:22.298 WARNING (SyncWorker_0) [homeassistant.components.telegram_bot] Can't load data in http://<IP_ADDRESS>:1984/api/frame.jpeg?src=camera1 after 5 retries

You need to config go2rtc.yaml

Mind giving an helpful example?

Your example is right. Maybe you have change wrong file or haven't restart go2rtc


Feature Request: News Feed

We want to implement a News Feed feature in the CryptoTracker application that aggregates cryptocurrency news from various sources. This feature will allow users to stay informed about the latest developments in the crypto market.
Please assign me under GSSoC'24 ext and hacktoberfest labels

We want to implement a News Feed feature in the CryptoTracker application that aggregates cryptocurrency news from various sources. This feature will allow users to stay informed about the latest developments in the crypto market.
Please assign me under GSSoC'24 ext and hacktoberfest labels

Oky do it 😀all the best

@vanshchauhan21 can you add level 3 label too?

@vanshchauhan21 done :)


Add transformer for require using Babel

Summary
Fixes #5631

Occurred changes and/or fixed issues

Technical notes summary

Added babel-plugin-transform-require-context library
Converted babel.config.js into a function and added conditionally logic if test

Areas or cases that should be tested
Try to remove comment to the example file and error should change (test is still incomplete 😅)
Areas which could experience regressions

Screenshot/Video

I think this is fine. We should double check our extensions won't need to be updated. I don't think they will.

So far I have never been running unit tests for the extensions, so I don't know even for all the other configurations we have how do they behave.
This change is aimed to the dashboard only, as we use no require in rancher-components.


Support for attribute table restructuring

Use case:
I have added a new column to my geopackage and deleted and existing column.

Note about data types and strings: for geopackage/sqlite the change in the string length is just different value in the metatable, but the actually it both stored same way. Since in reality it is same data type, this metatable difference is ignored by geodiff.  Therefore this is intended behaviour.
robot@gis:~$ ogrinfo gis/point.gpkg -al -so |grep WorkZone
WorkZone: String (10.0)
robot@gis:~$ ogrinfo other_point.gpkg -al -so |grep WorkZ
WorkZone: String (5.0)
robot@gis:~$ geodiff diff --json point.gpkg other_point.gpkg a-to-b.diff
(returns no diff)

On the contrary, if the type changes to int from string, the difference should be caught by geodiff

for string length change: https://github.com/lutraconsulting/geodiff/issues/160


Correlation Id  spring apmq 1.7.x

Hello this code look weird SimpleAmqpHeaderMapper.toHeaders
	@SuppressWarnings("deprecation")
			byte[] correlationId = amqpMessageProperties.getCorrelationId();
			if (correlationId != null && correlationId.length > 0) {
				headers.put(AmqpHeaders.CORRELATION_ID, correlationId);
			}
			String correlationIdString = amqpMessageProperties.getCorrelationIdString();
			if (StringUtils.hasText(correlationIdString)) {
				headers.put(AmqpHeaders.CORRELATION_ID, correlationId);
			}

second why is not there something like
if (StringUtils.hasText(correlationIdString)) {
				headers.put(AmqpHeaders.CORRELATION_ID, correlationIdString);
			}

Well, that's just a bug. Simple copy/paste without testing evil.
Thank you for catching that out!
We will be glad to have a fix from you as a contribution.

Fixed via: https://github.com/spring-projects/spring-amqp/commit/024a32c71bc419ac055d88851d861f9b231094f0
@igieon ,
Thank you very much for the contribution.
Looking forward for more!


One terminal tab hangs, Terminal works until I close the frozen tab - then whole Terminal hangs

Windows Terminal version
1.13.11431.0
Windows build number
10.0.19044.1766
Other Software
wsl (Debian),
docker 20.10.12, build e91ed57
docker-compose 1.29.2, build 5becea4c
Steps to reproduce
Happens randomly, but takes long time, so I cannot find 100% working way to reproduce. Probably not related to docker and compose, I suspect it's more about long output and hibernating.

Start docker-compose in detached mode with many containers producing a lot of logs on a wsl tab
Start docker-compose logs -f
Let it live for several hours
Just in case - open another terminal tabs (I think that's what I always do)
Hibernate computer (S4 power state)
Wake up computer (maybe after a while? maybe a date change is required? I don't know, I usually hibernate it in the evening and wake up in the morning)

Expected Behavior
No response
Actual Behavior
Single tab with wsl and docker-compose command freezes.
Docker containers keep working (expected). Terminal as a whole works (I can switch tabs, use other tabs). When I click [X] to close the frozen tab whole Terminal feezes and I need to kill it.
It started happening around 2 months ago, it was just fine previously, I'm using Terminal for the same things since it was added to the MS Store.

Would you happen to have an NVidia graphics card/? This kinda feel like the repro for #12607, but that only seems to occur on nvidia hardware.

No, I don't have graphicd card at all, just the CPU: 11th Gen Intel(R) Core(TM) i5-1135G7 @ 2.40GHz.
64 GB of RAM, around ~50% of it is used most of the time, but I don't see increased usage when Terminal hangs. Shouldn't be too much for Terminal, but such stuff happens as well, so I'm mentioning it, because I didn't put the specs in the original post.

Butts. When the Terminal is frozen like this, could you grab a dump with Task Manager and send it to us/?

This doesn't look like the Terminal is struggling for resources, this looks like your bog-standard deadlock. Just gotta figure out where.

I will send you a dump next time it happens. Is it safe to send it here, to the public?

It might have PII in it, definitely including whatever output was in the Terminal at the time. You can email it to me (my email address is in my profile), with the usual caveats of security that accompany email. If you're cool with that, ping me here when you send it. Otherwise, I think there's a way to upload files to a Feedback hub report, but I don't know how off the top of my head. That would be secure for sure, but less convenient. Pick your poison 😋

@zadjii-msft

If you're cool with that, ping me here when you send it.

I have the dump. It's over 700MB big, so I guess I can't push them via e-mail, but I can host them somewhere accessible via https and just send you the link via e-mail. I took the dump of frozen process (Not responding in orange), it has different title and icon than your screenshot but I guess this is the one you need.

So, do you want me to send you a specific mail topic or something?

it has different title and icon than your screenshot

Yep that looks like the right wt.exe process!

send you the link via e-mail

That'll work. Just include the issue number or "Terminal" or something in the subject line and ping me here, I should be able to figure it out. Thanks!

Sent 📬
Subject:

Windows Terminal issue #13386 - memory dump

Hmm.
Main thread:
0:000> k
 # Child-SP          RetAddr               Call Site
00 000000f8`5c8feab8 00007ffa`2d561ace     ntdll!ZwWaitForSingleObject+0x14 [minkernel\ntdll\daytona\objfre\amd64\usrstubs.asm @ 211]
01 000000f8`5c8feac0 00007ff9`b3ede979     KERNELBASE!WaitForSingleObjectEx+0x8e [minkernel\kernelbase\synch.c @ 1328]
02 (Inline Function) --------`--------     Microsoft_Terminal_Control!Microsoft::Console::Render::RenderThread::WaitForPaintCompletionAndDisable+0x17 [C:\a\_work\1\s\src\renderer\base\thread.cpp @ 298]
03 000000f8`5c8feb60 00007ff9`b3e84214     Microsoft_Terminal_Control!Microsoft::Console::Render::Renderer::TriggerTeardown+0x41 [C:\a\_work\1\s\src\renderer\base\renderer.cpp @ 332]
04 000000f8`5c8feba0 00007ff9`b3e85fb4     Microsoft_Terminal_Control!winrt::Microsoft::Terminal::Control::implementation::ControlCore::~ControlCore+0x34 [C:\a\_work\1\s\src\cascadia\TerminalControl\ControlCore.cpp @ 206]
05 000000f8`5c8febd0 00007ff9`b3e69173     Microsoft_Terminal_Control!winrt::Microsoft::Terminal::Control::implementation::ControlCore::`scalar deleting destructor'+0x14
06 (Inline Function) --------`--------     Microsoft_Terminal_Control!winrt::impl::root_implements<winrt::Microsoft::Terminal::Control::implementation::ControlCore,winrt::Microsoft::Terminal::Control::ControlCore,winrt::Microsoft::Terminal::Control::ICoreState>::NonDelegatingRelease+0x44682 [C:\a\_work\1\s\src\cascadia\TerminalControl\Generated Files\winrt\base.h @ 7385]
07 (Inline Function) --------`--------     Microsoft_Terminal_Control!winrt::impl::root_implements<winrt::Microsoft::Terminal::Control::implementation::ControlCore,winrt::Microsoft::Terminal::Control::ControlCore,winrt::Microsoft::Terminal::Control::ICoreState>::Release+0x44682 [C:\a\_work\1\s\src\cascadia\TerminalControl\Generated Files\winrt\base.h @ 7266]
08 (Inline Function) --------`--------     Microsoft_Terminal_Control!winrt::implements<winrt::Microsoft::Terminal::Control::implementation::ControlCore,winrt::Microsoft::Terminal::Control::ControlCore,winrt::Microsoft::Terminal::Control::ICoreState>::Release+0x44682 [C:\a\_work\1\s\src\cascadia\TerminalControl\Generated Files\winrt\base.h @ 7859]
09 000000f8`5c8fec00 00007ff9`b3e94620     Microsoft_Terminal_Control!winrt::com_ptr<winrt::Microsoft::Terminal::Control::implementation::ControlCore>::unconditional_release_ref+0x4468f [C:\a\_work\1\s\src\cascadia\TerminalControl\Generated Files\winrt\base.h @ 2544]
0a (Inline Function) --------`--------     Microsoft_Terminal_Control!winrt::com_ptr<winrt::Microsoft::Terminal::Control::implementation::ControlCore>::release_ref+0xb [C:\a\_work\1\s\src\cascadia\TerminalControl\Generated Files\winrt\base.h @ 2538]
0b (Inline Function) --------`--------     Microsoft_Terminal_Control!winrt::com_ptr<winrt::Microsoft::Terminal::Control::implementation::ControlCore>::{dtor}+0xb [C:\a\_work\1\s\src\cascadia\TerminalControl\Generated Files\winrt\base.h @ 2370]

Looks locked in a teardown of a renderer, waiting on _hPaintCompletedEvent. There's 8 renderer threads, lets see...
Looks like in the main thread, the control that's dtoring:
0:000> dx -r1 (*((Microsoft_Terminal_Control!std::unique_ptr<Microsoft::Console::Render::RenderThread,std::default_delete<Microsoft::Console::Render::RenderThread> > *)0x2c7f0b865a0))
(*((Microsoft_Terminal_Control!std::unique_ptr<Microsoft::Console::Render::RenderThread,std::default_delete<Microsoft::Console::Render::RenderThread> > *)0x2c7f0b865a0))                 : unique_ptr {...} [Type: std::unique_ptr<Microsoft::Console::Render::RenderThread,std::default_delete<Microsoft::Console::Render::RenderThread> >]
    [<Raw View>]     [Type: std::unique_ptr<Microsoft::Console::Render::RenderThread,std::default_delete<Microsoft::Console::Render::RenderThread> >]
    [ptr]            : 0x2c7f0a98a10 [Type: Microsoft::Console::Render::RenderThread *]
    [deleter]        : default_delete [Type: std::_Compressed_pair<std::default_delete<Microsoft::Console::Render::RenderThread>,Microsoft::Console::Render::RenderThread *,1>]
    [+0x000] _hThread         : 0xb60 [Type: void *]
    [+0x008] _hEvent          : 0xb84 [Type: void *]
    [+0x010] _hPaintEnabledEvent : 0xb6c [Type: void *]
    [+0x018] _hPaintCompletedEvent : 0xb70 [Type: void *]

Dunno if I can take the handle to the thread and just get the TID straight away. in the debugger
the second, 0x413c is in the middle of a paint frame.
And y'know what, its _hThread is 0xb60 so this is probably the one that's supposed to be tearing down.
...
0:010> k
 # Child-SP          RetAddr               Call Site
00 000000f8`5cdff388 00007ffa`295f3c3a     win32u!ZwGdiDdDDIWaitForSynchronizationObjectFromCpu+0x14 [onecoreuap\windows\core\umode\moderncore\objfre\amd64\usrstubs.asm @ 5022]
01 000000f8`5cdff390 00007ffa`295f3840     d3d11!CallAndLogImpl<long (__cdecl*)(_D3DKMT_WAITFORSYNCHRONIZATIONOBJECTFROMCPU const *),_D3DKMT_WAITFORSYNCHRONIZATIONOBJECTFROMCPU *>+0x12 [onecoreuap\windows\directx\dxg\d3d11\D3DCore\Inc\pch.hpp @ 543]
02 (Inline Function) --------`--------     d3d11!CallAndLog+0x10 [onecoreuap\windows\directx\dxg\d3d11\D3DCore\Inc\pch.hpp @ 591]
03 000000f8`5cdff3f0 00007ffa`21dd5e72     d3d11!NDXGI::CDevice::WaitForSynchronizationObjectFromCpuCB+0x60 [onecoreuap\windows\directx\dxg\d3d11\d3dcore\lowfreq\dxgidevice.cpp @ 9808]
04 000000f8`5cdff480 00007ffa`215aa601     igd10iumd64!GTPin_Init+0x155092
05 000000f8`5cdff4e0 00007ffa`213b0dd6     igd10iumd64!OpenAdapter10_2+0x2f01e1
06 000000f8`5cdff520 00007ffa`213b0312     igd10iumd64!OpenAdapter10_2+0xf69b6
07 000000f8`5cdff560 00007ffa`212bb47a     igd10iumd64!OpenAdapter10_2+0xf5ef2
08 000000f8`5cdff5c0 00007ffa`212bb2d5     igd10iumd64!OpenAdapter10_2+0x105a
09 000000f8`5cdff5f0 00007ffa`21cc18f5     igd10iumd64!OpenAdapter10_2+0xeb5
0a 000000f8`5cdff620 00007ffa`21cc1a5a     igd10iumd64!GTPin_Init+0x40b15
0b 000000f8`5cdff670 00007ffa`29630b79     igd10iumd64!GTPin_Init+0x40c7a
0c 000000f8`5cdff6d0 00007ffa`2b95206e     d3d11!NDXGI::CDevice::WaitForFence+0x89219 [onecoreuap\windows\directx\dxg\d3d11\d3dcore\lowfreq\dxgidevice.cpp @ 4020]
0d 000000f8`5cdff750 00007ffa`296c32d2     dxgi!CDXGISwapChain::AcquireBuffer+0xbe [onecoreuap\windows\directx\dxg\dxgi\dll\swapchan.cpp @ 6100]
0e 000000f8`5cdff790 00007ffa`295b648d     d3d11!CContext::AcquireResource+0x4a [onecoreuap\windows\directx\dxg\d3d11\d3dcore\highfreq\device_hf.cpp @ 15625]
0f 000000f8`5cdff7c0 00007ffa`296d7201     d3d11!CContext::AcquireResourceWithMaskAndGuard+0x49 [onecoreuap\windows\directx\dxg\d3d11\D3DCore\Inc\Device.inl @ 1105]
10 000000f8`5cdff7f0 00007ffa`296ffcbe     d3d11!CResource<ID3D11Resource>::CopyResource<2,6>+0x1f1 [onecoreuap\windows\directx\dxg\d3d11\D3DCore\Inc\Resource.inl @ 3849]
11 000000f8`5cdff840 00007ffa`29754256     d3d11!CContext::TID3D11DeviceContext_CopyResource_<1>+0x3e [onecoreuap\windows\directx\dxg\d3d11\d3dcore\highfreq\device_hf.cpp @ 6700]
12 000000f8`5cdff870 00007ff9`b3e214ab     d3d11!CContext::TID3D11DeviceContext_CopyResource_Amortized<1>+0x86 [onecoreuap\windows\directx\dxg\d3d11\d3dcore\highfreq\deviceamortized_hf.cpp @ 872]
13 000000f8`5cdff8e0 00007ff9`b3e21349     Microsoft_Terminal_Control!Microsoft::Console::Render::DxEngine::_CopyFrontToBack+0xc7 [C:\a\_work\1\s\src\renderer\dx\DxRenderer.cpp @ 1463]
14 000000f8`5cdff940 00007ff9`b3e166b6     Microsoft_Terminal_Control!Microsoft::Console::Render::DxEngine::Present+0x99 [C:\a\_work\1\s\src\renderer\dx\DxRenderer.cpp @ 1590]
15 000000f8`5cdff980 00007ff9`b3e162ba     Microsoft_Terminal_Control!Microsoft::Console::Render::Renderer::_PaintFrameForEngine+0x202 [C:\a\_work\1\s\src\renderer\base\renderer.cpp @ 178]
16 000000f8`5cdffa20 00007ff9`b3e16221     Microsoft_Terminal_Control!Microsoft::Console::Render::Renderer::PaintFrame+0x4a [C:\a\_work\1\s\src\renderer\base\renderer.cpp @ 78]
17 000000f8`5cdffa50 00007ffa`2f887034     Microsoft_Terminal_Control!Microsoft::Console::Render::RenderThread::_ThreadProc+0x71 [C:\a\_work\1\s\src\renderer\base\thread.cpp @ 214]
18 000000f8`5cdffa80 00007ffa`2f9c2651     kernel32!BaseThreadInitThunk+0x14 [clientcore\base\win32\client\thread.c @ 64]
19 000000f8`5cdffab0 00000000`00000000     ntdll!RtlUserThreadStart+0x21 [minkernel\ntdll\rtlstrt.c @ 1153]

That actually kinda reminds me of #12607 - it's like the main thread & render thread got "deadlocked" by waiting for the same event. The main thread waited for the renderer to do something, and in the process of painting, the renderer triggered something to get sent to the main thread to get handled. That's at least the conjecture in the other thread. I really don't know why these OpenAdapter calls seem to call back to the main thread....
https://docs.microsoft.com/en-us/windows-hardware/drivers/display/supporting-multiple-processors:

In particular, the software layer cannot batch calls to functions that return information (for example, CreateResource). When the software layer must call one of these types of driver functions, it flushes all queued commands through the worker thread, and then the software layer calls the driver function on the main application thread.

We may want to try and deal with this along with #12607. I dunno if we can just safely hang on to the renderer in the dtor and do a final_release of that on the background thread.
Heck, do we even need to do a paint in the Renderer::TriggerTeardown in the ControlCore dtor? What does that get us? The thing is already going away! We just need to disable the renderer thread so it will clean itself up.

Terminal crashes for me as well.  Almost every time remote desktop then return to the desktop.  Happens probably %50.  If I should create a new issue lmk.  Attaching

This will most likely be fixed by #14959 because it doesn't call any graphics APIs while the console lock is being held.

Anyone still seeing this on 1.18 Stable & 1.19 Preview/?

Hello,
Personally I can't answer as I'm not a Windows user anymore. Maybe @paulelong (you commented before) or @joelvaneenwyk (you reacted to the issue) can verify it now.


Startup crash with 1.12.0+build.14 "java.lang.NullPointerException: Initializing game"

Using previous version 1.11.8+build.13 works fine, but the latest release crashes every startup with the attached log.
crash-2020-06-09_10.34.14-client.txt

My apologies, I forgot to remove some WIP code I started adding but decided to wait on.

I've fixed this in 1.12.1


[Howto] A few hints on how to integrate pywinauto and Jenkins

Added a "Integration with Jenkins" chapter to quickly sum up the steps required to get pywinauto integrated into a Jenkins pipeline.

Looks like Remote Execution Guide is better place for it. What do you think?
BTW, please amend your commit with correct email that matches to your GitHub account. It's nice to see you in Contributors list.

Now it looks much better. Thanks for your efforts!
I would also mention that PsExec.exe has known issue: it hangs under Java/Jenkins. But there is open-source analogue PAExec which is fully compatible with PsExec by command line options and it also has some extensions. This tool works fine under Jenkins as well.
Sorry for adding more job. If you have no time, I can add it later. Absolutely no problem. And thanks again for raising this!


NIFI-8416 - Remove dl.bintray.com repository dependency (service deprecated).

Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
NIFI-8416 (associated with NIFI-8398).
Depends on PR:

https://github.com/apache/nifi/pull/4989/
Merge this PR after that PR.

Remove dl.bintray.com repository dependency (service deprecated).
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically main)?

[X] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

JDK 11 build failure.  Investigating.


Fix date header parsing

This PR resolves some confusion around how dates are supposed to be used. The signing spec states the following:

A signature MUST have a bounded validity window on the order of minutes. The exact skew limit MAY vary depending upon how closely Clients' and Servers' clocks can be synchronized. Clients MUST include a date header with second precision as defined by [RFC 2616] Section 3.3.1:

HTTP-date := rfc1123-date | rfc850-date | asctime-date

The signature MUST be bound to a canonical representation of the exact date encoded in the signed request's Date header. To ensure consistency across languages and platforms, this MUST be an ASCII encoded UNIX epoch timestamp, to millisecond precision:		 The signature MUST be bound to a canonical representation of the exact date encoded in the signed request's Date header. To ensure consistency across languages and platforms, this MUST be an ASCII encoded UNIX epoch timestamp, to millisecond precision:

date-header := UNIX_Epoch_Seconds<HTTP-date> * 1000

Therefore, all dates used, both to sign the request as well as send in the date header, should be time since the epoch expressed in milliseconds.

This will break existing turnstile clients. Has this been communicated how is the migration strategy? As this involves multiple systems I suggest supporting both cases to provide an upgrade path and at a later stage remove support for non epoch times.

That sounds good. I'll update this PR with support for both and a deprecation notice.

How about logging the iPad address of the request as well, which could help identifying old clients?

I'll create some stories on my side to update the date header. Currently we use RFC_1123 for dates. From what I know, LE team is using turnstile signed requests, as well as Platform Team (they reuse the LE code)

@ilyabiryukov-r7 that should be fine.
You should note that because RFC 1123 doesn't have millisecond precision, when Turnstile parses the date during signature calculation it will effectively floor it to the nearest second.
> d = new Date()
Wed Apr 12 2017 09:42:00 GMT-0400 (EDT)
> d.getTime()
1492004520413
> (new Date(d.toUTCString())).getTime()
1492004520000

That's fine if you're calculating the signature against RFC_1123 dates anyway. Just be aware that using date strings without millisecond precision could give you authorization errors where you wouldn't expect it.

I'm going to merge this but wanted to cc @aguerlain-r7 on this so he's aware of changes he'll need to make eventually.


Unknown layer type: Input

When running python predict.py cityscapes input_image -o output_image have the same error message as reported in caffe repository layer_factory.hpp:81 [...] Unknown layer type: Input. Can you point me to right direction? What does it mean? Version inconsistencies? I'm using caffe-dilation. I also tried to change my make file as described on stackoverflow but it breaks the makefile (this suggestion is from 2015 anyway...)

Hello, I just followed his tutorial, didnt have any problems.


Spike - Application change versioning

What do we need to do soon to make sure that we are able to in the future for us to edit the data in our database resulting from an RFI process prior to sending the application data to ISED.

[x] Look at how CIF has implemented this versioning
[x] Decide how we are going to solve this so that it saves us extra work in the future.

Decided to do the versioning before intake opens, possibly sprint 9


Update schedule

Hi @shinokumura,
I am wondering how frequently this repository is going to be updated. Would it make sense to add a github action to regularly check if updates are needed?

This can be handled automatically in this repo.


Update README.md

Allow a new contributer to identify key features of the application without having to read the full documentation, along with some more intuitive and generic examples to get them started using the program. The current examples only show specific functions.

Thanks for PR but these are not necessary.


Docs: json_normalize appears in docs, but not in the API page.

http://pandas.pydata.org/pandas-docs/dev/api.html
Not here.

http://pandas.pydata.org/pandas-docs/stable/io.html#normalization

@jreback is this all we need?


Update Java patch version

Description
Update Java 11.0.16 -> 11.0.17
Release notes:
Update Java to 11.0.17 in accordance with security advisory: https://openjdk.org/groups/vulnerability/advisories/2022-10-18
Checklist

[x] Added the correct label: < feature | bug | dependencies | infrastructure | ignore-for-release >

Issue(s) this completes
Fixes #3674

Codecov Report

Merging #3761 (eda5160) into main (ef2de76) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #3761   +/-   ##
=========================================
  Coverage     80.54%   80.54%
  Complexity     2441     2441
=========================================
  Files           296      296
  Lines          9653     9653
  Branches        660      660
=========================================
  Hits           7775     7775
  Misses         1624     1624
  Partials        254      254

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[automation] Update go release version 1.18.9

What
Bump go release version with the latest release.
Further details
See changelog for 1.18.9
Automatically generated from https://apm-ci.elastic.co/job/apm-shared/job/bump-go-release-version-pipeline/

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-12-12T03:48:01.446+0000

Duration: 8 min 16 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
749

Skipped
5

Total
754

:robot: GitHub comments
Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.


problem with current test unit

with mysql it composer test gives an error
Error in bootstrap script: Illuminate\Database\QueryException:
SQLSTATE[08006] [7] received invalid response to SSL negotiation: Y (SQL: select tablename from pg_catalog.pg_tables where schemaname in ('public'))

when testing sqllite is set to memory this code
$console->call('migrate:fresh', ['--seed' => true]);
in he bootstrap file does not create the tables and all db tests fail. Kindly help

Hey @Master-maynd that error is a postgres error, I think you haven't changed your local setup properly to MySQL if you are intending to use that.


Add replay attack protection

Implements RFC 6347 Section <IP_ADDRESS>.
Set config.ReplayProtectionWindow to change the size of the protection window.
Default is 64.
Reference issue
Fixes #146

CI failure on WASM will be fixed by #210.

Addressed review feedbacks and rebased.

rebased


CommandLineArgumentProvider documentation eagerly calls get() on provider

Issue type
Wrong or misleading information
Problem description
CommandLineArgumentProvider documentation creates a file provider and then immediately calls get() like layout.buildDirectory.dir('dist').get(). The sample code should instead pass the provider in to the CommandLineArgumentProvider and call .get() when used.
Here is the sample:
https://docs.gradle.org/current/userguide/caching_java_projects.html#dealing_with_file_paths
Context (optional)
No response
Page with the problem
https://docs.gradle.org/current/userguide/caching_java_projects.html#dealing_with_file_paths

Thank you for your interest in Gradle!
This is a valid documentation issue that we will address.


Failure on KITTI Dataset

Dear team members, I appreciate the work done by you guys and i am already a fan of your previous work related to lidar odometry. But the issue i am facing today is regarding lidar_visualizer:
The things is that i was trying to run bag file and it worked fine as long as frames were iterated over and over again but whenever frames ended it raised StopIteration error. Below i shared the snap shot of the error.

I wanted to know is this how it always end? i tried to debug but i couldn't find any mistake with my data. Also this is one case. Another case is when i tried to run my another bag file the visualizer opened but it was blank. (For some reasons i can't share the bag file) but i wanted to know is there specific format in which frames are interpreted for the visualizer?

Are you using the KITTI dataset or bagfiles? :thinking:

@nachovizzo I actually downloaded one from this opensource link inside LIO-SAM repository (which to my understanding is KITTI data converted into bag file) and tried to visualize it. But for my own bag files I didn't see anything although input command line arguments were just fine. The lidar data comes from RSHelios 5515 lidar in my bag files.


Is it possible to display content from a private branch?

I would like to show content of commits but not code itself like a live changelog to a public site but hiding the actual code changes.

@Virtuworks it's already possible to grab content from a private repo.
But I don't get what you mean by "hiding the actual code changes"
Please refer to the github API

Here is what I meant but I went into the plugin code and changed it to not add the URL Link Anchor around a line of code. Would be good if there was an option to enable/disable this. Here is where I am using this plugin. It would be great to be able to also show the Release Label attached to the commit (so users know which build a specific commit was in. Also if we can show the date of the commit. Would also be good if we could hide MERGE rows.
https://mspcontrol.org/changelog/
Is there a way to show more than 25 records or this a GitHub Limit?

@Virtuworks
Hiding the commit link is quite specific I guess
here is a list of what is available:
https://developer.github.com/v3/repos/commits/
I could think about adding options to show/hide nodes
25 is not a limit, we could use last_sha to add more commits

I am not a developer but if some of these options could make it into plugin I would be willing to fund the development of it. If you can contact me to<EMAIL_ADDRESS>if interested we could talk about compensation for it. I am in a big need of showing public people commits from a private branch and letting them look through them (maybe even supporting paging if possible).

The real need here is to create something like a changelog based on commits from GitHub. I love to work agile but it's a lot of work to manage a changelog file when you have 10 developers contributing multiple changesets a day. I am looking for a way to share the commits which I will make my devs be more descriptive with and hopefully if the commit's label can be shown or link the release number it can act as a changelog for end users to understand what has gotten fixed version to version. My whole build process is automated with Team city so a chagelog is additional interaction that I would like to avoid.

Just checking back to see if their is any interest here for us to help cover time to improve this plugin.

@Virtuworks
will check if it's possible to update

@Virtuworks
I added an xperimental custom endpoint function, please download latest version from github test it and give feedback to me.
[github-commits username="yahoo" repository="pure" limit="30" xtended="commit->committer->email,url,commit->committer->name"]
xtended attributes works as a endpoint listing.
it will add a span with a class, with the value.
you can retrieve all the possibilities here :
https://developer.github.com/v3/repos/commits/#list-commits-on-a-repository

Thanks will try later tonight if possible

I've downloaded and updated and it shows nothing just blank.
https://mspcontrol.org/changelog/

@Virtuworks can you paste your shortcode

[github-commits username="Virtuworks" repository="pure" limit="30" xtended="commit->committer->email,url,commit->committer->name"]

I guess you don't have a repository PURE
...

Yup! =) Sorry about that! It seems to work great! Last thing that would be good is ability to hide MERGE PULL REQUESTS is that possible?
https://mspcontrol.org/changelog/
See here I would like to be able to hide all the Merge Commits from the list.

Also the limit of responses is still 30? Anyway to get paged list?

Is it possible to show a label that is associated with the commit? We label the commits with Labels for their Build number. I don't see how to do this.

@Virtuworks download the last release,
pay attention, the format has change to json like:
[github-commits username="Virtuworks" repository="modsecurity-whitelist-apps" limit="30" xtended='{"Date de commit": "commit:committer:date","Email committer": "commit:committer:email","Url de commit": "url"}']

I've updated to latest version of plugin. Did you have a chance to review my other comments? You can see latest output at the same link above.

Please open a new issue for any new concern


Examples are broken

Hello,
The demo are broken with the arrival of Pixi v8

https://pixijs.io/lights/examples/
https://pixijs.io/lights/examples/usage.html

Examples should be working again


gemspec missing licence

License doesnt show up on https://rubygems.org/gems/puppet-blacksmith/versions/4.1.2 and cannot be found by license audit tools like https://github.com/pivotal/LicenseFinder

I've also fixed up the LICENSE file so that github detection works and added a badge to the README.


How to specify compiler options for LLVM

Hello,
I was interested to run the LLVM version of BinTuner but cannot find any related files for compiling via clang.
is there an equivalent python file that compiles using clang similar to examples/gccflags ?
Are there instructions on how to specify constraints for LLVM options and get the correct combination of options for clang?

Hi there,
Thank you very much for your interest in our paper and for being prepared to try out BinTuner!
BinTuner is an auto-tuning framework, which can deliver a near-optimal binary code that reveals much more differences than -Ox settings.
That means you can feel free to add any option to extend or modify the array FLAGS = ['reorder-blocks', 'reorder-functions',  'inline-functions', 'unswitch-loops'] in the main.py program depending on the compiler version. Please note that different compiler versions may support different options, make sure that the choices you add are a match for the compiler you configured (for iterative compilation), otherwise you will get compilation errors.
For clang, please refer to the configuration methods under main_clang.py, for example, LLVM_FLAGS_fno = ['blocks', 'borland-extensions', 'delayed-template-parsing', 'fast-math',], LLVM_FLAGS_mno = ['backchain', 'incremental-linker-compatible', 'global-merge'], LLVM_FLAGS_On = ['gnu-runtime', 'stack-protector-all'], LLVM_FLAGS_Off = ['sanitize-blacklist', 'crash-diagnostics'], there are very detailed configuration methods and use cases here. https://github.com/BinTuner/Dev/blob/main/BinTuner/examples/gccflags/main_clang.py To double check before running to make sure the paths to all program (you will compile) files needed to compile are configured, otherwise you will get compilation errors and failures.
In addition, we use Z3 Prover to analyze all generated optimization option sequences for conflicts and make changes to conflicting options for greater compiling success. For more details, please refer Z3Prover.
Thanks again for your interest and feedback!


GitHub service issue: The social account is already connected to a different account.

Details
I am using this profile to log in. In the past, I would log in with this account, then connect to my foxbunny GH account to retrieve the list of projects. This time, however, RTD seems to have decided that 'foxbunny' is a separate profile which I cannot log into since I never actually signed up to create it in the first place. Disconnecting the connection from the GitHub side did not help.
Expected Result
Expected the outernet profile to be able to connect to foxbunny GH account.
Actual Result
The foxbunny GH account is taken over by a foxbunny profile that cannot be logged into using any of the emails that are connected to the GH account.

I'm having the same issue.

This was a bug that we had for a long time, but has been fixed for a number of months. If you login with GitHub (https://readthedocs.org/accounts/github/login/?process=login) it will bring you to your account connected with that GitHub account, which you can then unlink (https://readthedocs.org/accounts/social/connections/) to fix this.


Request: Magic to use Attribute

Hello,
First off, this system is amazing and my group and I love it.
One thing we enjoy doing is hacking in new classes. Recently, we have been working on a bard class and wanted to make the spell casting use Charisma rather than the standard Wisdom/Intelligence. Unfortunately, this is something we cannot automate in your foundry system due to the hard coded Prayer/Spell options.
Could this be something that is added to allow more flexible addition of non-standard spell types?
Thanks for taking the time to look over this, and I look forward to hearing from you,
Andy

Okay, I'll put a little thought into it and see if I can come up with something. Making no promises and not offering any timelines though.

Implemented for the v0.4.0 release.


Setting up remote connection: Rebuilding codespace...

ADD ISSUE DESCRIPTION HERE
Version: 1.89.1
Commit: dc96b837cf6bb4af9cd736aa3af08cf8279f7685
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Embedder: codespaces

### Tasks
- [ ] Setting up remote connection: Rebuilding codespace

/needsMoreInfo

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

Hi John,
Please find attached the screenshots of the problem. I do not know how to
get my screen to go back to the normal coding screen. Please advise on
this,it is utterly confusing to me.
On Sat, Jun 1, 2024 at 3:03 PM John Murray @.***> wrote:

/needsMoreInfo
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/214052#issuecomment-2143442006,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGYNBPXMLBPXVG2EEUNCN5LZFHBBTAVCNFSM6AAAAABIUBIOR2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTGQ2DEMBQGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Screenshots cannot be received as email attachments. You need to go to the issue on the GitHub website and attach them to a comment.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Issues with v_generate_tbl_ddl.sql

Hello --
Thanks so much for these great scripts.
I've been using them for various Redshift diagnostics. I think I have found an issue with v_generate_tbl_ddl. When I try to run it I get this error message:
SELECT
 schemaname
 ,tablename
 ,seq
 ,ddl
FROM
 (
 SELECT
  schemaname
  ,tablename
  ,seq
  ,ddl
 FROM
  (
  --DROP TABLE
  SELECT
   n.nspname AS s...

[Amazon](500310) Invalid operation: syntax error at or near "SELECT";

I've been trying to troubleshoot this and figure out the query issue but it is a rather large query.
A couple questions to start:

Is there a reason to have the ORDER BY clauses in the subqueries? I don't think these actually do anything. I'm actually a bit surprised Redshift allows this; SQL Server doesn't.
Examples:
Line 81,    ORDER BY a.attnum
Line 93,   ORDER BY seq
Line 141,   ORDER BY abs(a.attsortkeyord))
Line 152,   ORDER BY abs(a.attsortkeyord)
Line 181,     ORDER BY seq
More info on why ORDER BY in subqueries isn't useful and could actually be harmful by having the database do extra unnecessary work :http://stackoverflow.com/questions/5119190/oracle-sql-order-by-in-subquery-problems.
I realize the stackoverflow discussion above is about Oracle, but I think the general point is relevant for any RDBMS.
I have found that sub-parts of the query will work if I remove the excess ORDER BY clauses.

I think there are some extra parentheses?
Examples:
Line 127,   UNION (SELECT
Line 141,   ORDER BY abs(a.attsortkeyord)) -- This time at the end, aside from the ORDER BY
Line 157,   WHERE c.relkind = 'r' )

Does the query as a whole run correctly for you? I have been using SQL Workbench/J and just want to make sure it isn't some sort of query formatting or codepage issue.

Thanks!

The first issue is caused by an issue with how SQL comments are handled within querytext by the Redshift JDBC 1.1.13 driver. You can mitigate this behavior with the 1.1.10 version of the Redshift driver until we release the next version.
The ORDER BY clauses are to ensure a consistent positioning of columns, but testing against the pg_catalog tables shows this consistent ordering is enforced already. These are likely to be redundant - will test and remove if confirmed to be the case.

Thank you, reverting from the Redshift JDBC 1.1.13 driver to the 1.1.10 driver worked.
I'm curious -- I can't see any references now to the 1.1.13 driver anywhere but I know I must have downloaded it from somewhere before. Was the 1.1.13 driver formerly available on Amazon's site and then removed due to some issues found with the driver?

We had previously identified the issue you raised here and removed the driver while we work on a fix.

Is there a reason to have the ORDER BY clauses in the subqueries? ... I think there are some extra parentheses?

We globally sort by seq which is properly generated by incrementing attnum or abs(a.attsortkeyord) to maintain column order, so there is no meaningful value in these redundant ORDER BY clauses in the various subqueries. Benchmarks against a version of the VIEW where these are omitted resulted in a non-measurable performance change. I will remove these as well as the redundant parenthesis.

Thanks, chrisz. I really appreciate the prompt feedback.


Metadata implementation

@tgk still needs some cleanup, but this shows the principle.

@tgk Alright, cleaned it up a bit :)


feat: create personal folder - haksoon

Summary
haksoon folder and the following folders / files were created
How to Review?
Please check foldering rules
Estimated time for review
< 5 min

More info
PR type

[x] Add feature
[ ] Delete feature
[ ] Fix bug
[ ] Update code for dependencies, environments, builds, etc.
[ ] Else

Branches
origin/haksoon
Test results
Every file must work fine with no issues

좋습니다!!


Documentation not fully made

The following we node runners need .

[x] Hardware Requirements:
[ ] Networking Requirements: (Firewall and Ports etc)
[x] Software Requirements: (Windows, Linux , Unbuntu(be specific which versions)
[x] Monitoring and Management: (Where to see logs if it failed etc)
[ ] Language translations if possible for people in other countries .

But I think this is the basic needed for us to run node.

The "SSH" , you should also mention the "SSH" we should use .

Also the "examples" , something should be fixed about that

https://powerloom.network/mint-docs/
Also your team should have add this to the documentation too .

For language use google translate on the browser

Most places requires different requirements.
Like some specific ports to be opened etc
If that can be clarified


Rethink pagination

Pagination could be better:

amount of rows per page looks a bit odd (styling)
refresh is completely voiding the pagination state (both number of rows per page as well as current page)

Would assume that current state should be put in browser link

Fixed


Include all variants from productstream if option is checked

Right now you can't combine "Include variants" and a productstream as product streams will always return the productnumber of the main product.
This PR will use the product id (instead of the variant) if "include variants" is checked.
As a sidenote: the original code used to create a criteria that selects only active variants as well as limiting it to the default shop within the product stream. While this is still very much possible as each ConditionHandler that wants to use the ShopContext has still access to it, the base criteria won't mess around.
This might feel breaking but as this prohibits most products being exported in the first place which IMO makes this PR a bug fix.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/PT-11470
Please use this issue to track the state of your pull request.

thanks for your contribution @pascalheidmann  👍🎉💙


NullPointerException when running any Python script

name: NullPointerException when running any Python script
about: When attempting to run the attached Python script, or indeed any Python script, in Ghidra, a NullPointerException matching the output attached is thrown.
title: 'NullPointerException when running any Python script'
labels: bug
assignees: ''

Describe the bug
When attempting to run the attached Python script, or indeed any Python script, in Ghidra, a NullPointerException matching the output attached is thrown.
To Reproduce
Steps to reproduce the behavior:

Open the Script Manager window.
Select a Python script.
Press the “Run Script” button.
See the error.

Expected behavior
I expected the attached script to import FLIRT signatures for Nintendo 64 SDK functions into Ghidra to aid in decompiling N64 games.
Screenshots
No applicable screenshots.
Attachments
One of the scripts that I was trying to run, and the error message Ghidra throws, are attached.
Environment (please complete the following information):

OS: Sparky Linux Stable 5.10 amd64 (based on Debian 10 “Buster”)
Java Version: AdoptOpenJDK 13.0.2
Ghidra Version: 9.1.1

Additional context
Python 2 and 3 are both installed, as they are each required by varying parts of my system.

I've had this occur a few times but have never been able to consistently reproduce it. Does it occur if you try running the script after restarting ghidra?

Yes, it does; indeed it occurs all of the time for me.

Yes, it does; indeed it occurs all of the time for me.

The attached script ran without a problem for me. May you attach the application.log file? It may be found in $HOME/.ghidra/.ghidra_9.1.1_PUBLIC. Since you can consistently reproduce this it may be helpful to move the existing log, reproduce the issue and then attach the new log since it'll be much shorter.
What happens when opening the Python window in ghidra? Also, the python ghdra is using is Jython and not a system installed python.

Here is application.log.

Looking at the stack trace, it appears as though the real exception is being masked by the Python code.  Specifically,
at org.python.core.Py.dispatchToChecker(Py.java:2490)
is generating a NullPointerException while Ghidra is in the process of reporting a different exception.
Debugging this by putting a breakpoint at the above line will allow you to see the original exception.

I'm afraid to state it here for fear of being yelled at or simply run out of town on a rail, but I don't know how to do that. I am a complete know-nothing when it comes to Java development and little better when it comes to programming in general. Additionally, I am unable to figure out for myself how to set breakpoints within Ghidra itself, and I cannot even find a file named Py.java within either the latest release or the current source code of Ghidra. :(

I'm afraid to admit it for fear of ridicule, but I don't know how to do that.

No worries.  Quite a few of our users are not Java developers.   My previous message was really a note for the community at large.    Naturally, those of us not experiencing the problem cannot debug it.  My hope is that someone else will eventually run into the same issue and then be able to debug it.
As a reference, for those so interested,  Ghidra does integrate well with the Eclipse IDE.   If you run Ghidra from within Eclipse, then you can debug to your heart's content.

While asking an N64 decompilation Discord server that I'm on about how to debug Ghidra itself, I noted that I couldn't find Py.java in the source or Py.class in the binary, and was told that it might be a Gradle dependency, whereupon I realized that Jython might be an external dependency. Naïvely, I ran to my package manager* and installed Jython, which also pulled down Oracle Java 11; not only does Ghidra run, but I'm no longer getting the errors, so I assume it was related to Jython not being present. *shrug*
(* As noted earlier, I was using AdoptOpenJDK 13.0.2 instead of the Oracle Java 11 provided by Debian Buster's package manager; this was out of a (likely mistaken) belief that Ghidra requires Java 12 or newer.)

While asking an N64 decompilation Discord server that I'm on about how to debug Ghidra itself, I noted that I couldn't find Py.java in the source or Py.class in the binary, and was told that it might be a Gradle dependency, whereupon I realized that Jython might be an external dependency. Naïvely, I ran to my package manager* and installed Jython, which also pulled down Oracle Java 11; not only does Ghidra run, but I'm no longer getting the errors, so I assume it was related to Jython not being present. shrug
(* As noted earlier, I was using AdoptOpenJDK 13.0.2 instead of the Oracle Java 11 provided by Debian Buster's package manager; this was out of a (likely mistaken) belief that Ghidra requires Java 12 or newer.)

Check for files in Ghidra/Features/Python/lib path is relative to the ghidra installation directory.

Check for files in Ghidra/Features/Python/lib path is relative to the ghidra installation directory.

total 40M
drwxr-xr-x 2 halian halian 4.0K Feb  3 22:19 ./
drwxr-xr-x 5 halian halian 4.0K Feb  3 22:19 ../
-rwxr-xr-x 1 halian halian  40M Feb  3 22:18 jython-standalone-2.7.1.jar*
-rw-r--r-- 1 halian halian 106K Feb  3 22:18 Python.jar
-rw-r--r-- 1 halian halian  24K Feb  3 22:18 Python-src.zip
 ~/D/g/G/F/P/lib  unzip Python-src.zip                                                                                Wed 05 Feb 2020 05:28:37 PM EST
Archive:  Python-src.zip
   creating: ghidra/
   creating: ghidra/python/
  inflating: ghidra/python/PythonScriptExecutionThread.java
  inflating: ghidra/python/PyDevUtils.java
  inflating: ghidra/python/PythonRun.java
  inflating: ghidra/python/PythonPluginExecutionThread.java
  inflating: ghidra/python/PythonScript.java
  inflating: ghidra/python/GhidraPythonInterpreter.java
  inflating: ghidra/python/PythonPlugin.java
  inflating: ghidra/python/PythonUtils.java
  inflating: ghidra/python/PythonPluginInputThread.java
  inflating: ghidra/python/PythonScriptProvider.java
  inflating: ghidra/python/PythonCodeCompletionFactory.java  ```

Closing, as this seems resolved.  Feel free to post any additional relevant details.


URL Detection

I noticed in my app that the word "picture" is detected as a URL.  After looking into it, I noticed that any phrase starting with "www" or "pic" will get detected as a URL, even though they are not valid URLs.  Looking at the history, we used to use NSDataDetector but switched to a regex. Is there a compelling reason not to use NSDataDetector for this? The only test that fails using NSDataDetector is that "google.com" gets detected as a URL when the test does not want it to be (I would actually prefer it to be).  Right now I am running with my fork that uses NSDataDetector until this can be resolved.

Hi @krjackso, I am sorry for the issue, we should take a look at the regex for the urls. The reason why we switched from NSDataDetector to regex is that data detector is 2x more expensive.
See https://github.com/optonaut/ActiveLabel.swift/pull/40#issuecomment-170157961

Seems like NSDataDetector is slower than a custom Regex (2x the time)... Maybe we should move to a regex for the URLs?

Yeah URL regexes are a real pain :) Perhaps a decent compromise would be opening up the RegexParser so that we can override the different types of detectors if we need them to be different? The other thing I like about using NSDataDetector for URLs is it will be consistent with other things using it to detect links which will be important for my users who are mostly coming from the bundled iOS apps.

Yep, i like the idea, but in order to fix and add tests for it, I would like to know first which text is being detected as url, and it is not. Even if we open the Regex parser, there should be a default regex.

Hi, I am still detecting "picture." highlighted as a URL.

This issue is still not resolved.

Not able to reproduce, closing...


chore: upgrade to yarn 4

Will figure this upgrade out later, unfortunately I can't get yarn >1 to work on github actions with windows builders and I'd rather check for windows incompatibilities in tests that support yarn >1.


implement test Datastore for ciao-controller unit tests

Improve the ciao-controller unit testing by creating a test datastore which can succeed or fail. This will allow us to isolate our unit testing from the datastore and improve coverage.

the test datastore should use channels to communicate when events happen to prevent the need for time.Sleep() in controller_test.

If we add a test datastore won't we then be failing to test the sqlite code? sqlite has a notification ability built in: https://www.sqlite.org/c3ref/update_hook.html but I don't think the golang sqlite binding we're using has this exposed. But some others do: https://gist.github.com/bruth/bfd84fdfc855e79c9e79

Relates to issue #403


Pragma changes

JIRA Issue : https://project-sunbird.atlassian.net/browse/SB-5876
Pragma value need to set based on what plugin will send

There is no unit test-cases again.


feat: add proxy support for amplify cli

fix #459
Issue #459
Description of changes:
Add proxy support for amplify-cli.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@AlessandroVol23 Could you please update your PR based on @UnleashedMind's comments?

PR is updated according to @UnleashedMind comments. I've ran into a problem that graphql was missing after merging the current master. Please check if it's working now with proxy support.

@AlessandroVol23 Could you please rebase this PR? We're ready to merege this.

Sorry, something wen't pretty wrong here. I'll create another branch from the current master with my changes, sorry!

Opened a new PR with new branch. Sorry for that! #1464  I'll close that one


Can you add feature where i can get updated scale value on Double tap, and double finger zoom in/out

🎉 Version 4.2.0 of @likashefqet/react-native-image-zoom has been released with some exciting new features! 🚀
Feel free to update to the latest version to explore these additions. If you have any more suggestions or ideas for improvements, please feel free to share.
Thank you! 🙌


Fix issue with launching visualizer, atleast on OSX

I was getting timeouts when launching visualizer. It was ignoring the PORT Environment variable. Maybe the best way is to pass in the port to publish, i might do it in a future PR. Great little project BTW.

I would drop the PORT variable all together to avoid confusion.

Sounds good will do this and resubmit.

I have updated this pull request. Please review and lmk.


[SP-6600]- Backport of Vulnerable Jackson mapper ASL component changes.

🐸 JFrog Frogbot

:white_check_mark: Build finished in 1h 41m 11s
Build command:
mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl \
plugins/avro-format/assemblies/plugin,plugins/avro-format/core

:ok_hand: All tests passed!
Tests run: 41, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message