Require alcf->server array to be non-empty before using it.

If for some reason administrator adds auth_ldap directive, but forgets to add
auth_ldap_servers, servers array for the location remains uninitialized. And when
request with Authorization headers field arrives, NULL gets dereferenced.
This patch checks that array is initialized in ngx_http_auth_ldap_handler(), before
other functions are called.

To reproduce crash, start nginx with configuration:
worker_processes  1;
master_process off;
daemon off;
events { worker_connections  1024; }
http {
    server {
        listen       8080;
        auth_ldap    Hello;
        location / { root html; index index.html index.htm; }
    }
}

and send request "GET / HTTP/1.0\nAuthorization: Basic YTph\n\n".


Plaidchat installation via npm leaves extraneous files?

I'm not 100% sure this is actually a plaidchat error, although it is strange that no other globally installed package reports this error. I use two aliases in order to quickly list installed packages.
The first is npmpkgs which runs this (local): npm list --depth=0
The second is npmgpkgs which runs this npm list -g --depth=0
I've been battling NPM and nodejs all day, hence me being unsure it is actually plaidchat, but I thought I'd start here. Strangely, when listing globally installed packages, I get this in the console:
npm ERR! extraneous: buffer-to-vinyl /usr/lib/node_modules/plaidchat/node_modules/buffer-to-vinyl
npm ERR! error in /usr/lib/node_modules/plaidchat/node_modules/buffer-to-vinyl: ENOENT, open '/usr/lib/node_modules/plaidchat/node_modules/buffer-to-vinyl/package.json'
npm ERR! extraneous: decompress-unzip /usr/lib/node_modules/plaidchat/node_modules/decompress-unzip
npm ERR! error in /usr/lib/node_modules/plaidchat/node_modules/decompress-unzip: ENOENT, open '/usr/lib/node_modules/plaidchat/node_modules/decompress-unzip/package.json'
npm ERR! extraneous: download /usr/lib/node_modules/plaidchat/node_modules/download
npm ERR! error in /usr/lib/node_modules/plaidchat/node_modules/download: ENOENT, open '/usr/lib/node_modules/plaidchat/node_modules/download/package.json'
npm ERR! extraneous: download-status /usr/lib/node_modules/plaidchat/node_modules/download-status
npm ERR! error in /usr/lib/node_modules/plaidchat/node_modules/download-status: ENOENT, open '/usr/lib/node_modules/plaidchat/node_modules/download-status/package.json'
npm ERR! extraneous: duplexify /usr/lib/node_modules/plaidchat/node_modules/duplexify
npm ERR! error in /usr/lib/node_modules/plaidchat/node_modules/duplexify: ENOENT, open '/usr/lib/node_modules/plaidchat/node_modules/duplexify/package.json'
npm ERR! extraneous: meow /usr/lib/node_modules/plaidchat/node_modules/meow
npm ERR! error in /usr/lib/node_modules/plaidchat/node_modules/meow: ENOENT, open '/usr/lib/node_modules/plaidchat/node_modules/meow/package.json'
npm ERR! extraneous: merge-stream /usr/lib/node_modules/plaidchat/node_modules/merge-stream
npm ERR! error in /usr/lib/node_modules/plaidchat/node_modules/merge-stream: ENOENT, open '/usr/lib/node_modules/plaidchat/node_modules/merge-stream/package.json'
npm ERR! extraneous: read-all-stream /usr/lib/node_modules/plaidchat/node_modules/read-all-stream
npm ERR! error in /usr/lib/node_modules/plaidchat/node_modules/read-all-stream: ENOENT, open '/usr/lib/node_modules/plaidchat/node_modules/read-all-stream/package.json'
npm ERR! extraneous: vinyl-fs /usr/lib/node_modules/plaidchat/node_modules/vinyl-fs
npm ERR! error in /usr/lib/node_modules/plaidchat/node_modules/vinyl-fs: ENOENT, open '/usr/lib/node_modules/plaidchat/node_modules/vinyl-fs/package.json'

This is really weird... I tried pruning, but that doesn't work (and it shouldn't, since there is no package.json in my home directory, because why would I need one for globally installed packages).
I don't know if NPM is reporting a false positive, or if something more sinister is at hand. Have already cleared NPM cache (global and local) and still get this in response.

I'm a bit confused as to what you are looking for. Can you explain what you are trying to accomplish?

Hey twolfson,
I'm actually trying to make sure that Plaidchat has not installed with a bunch of leftover dependencies (like dev dependecies), and if so, how would I go about removing them? I have tried navigating to to where plaidchat is installed and running npm prune but this didn't work!

devDependencies should never be installed via npm install when a package name is specified (e.g. npm install plaidchat) unless you have overriden a npm config:
https://docs.npmjs.com/misc/config#dev
That being said to use npm prune, you would need to use it as if you were performing development on that repo. For this scenario, it should work if you go to the plaidchat directory and run npm prune there:
cd /usr/lib/node_modules/plaidchat/
npm prune

I tried npm prune in the /usr/lib/node_modules/plaidchat directory - to no avail.
After some further investigation and closer inspection of the output log, I have a feeling this is actually related to how I've installed NPM/Node. Is it possible NPM is traversing too far down the directory tree from my own project directory and seeing the node_modules folder in /usr/lib and for some reason listing dependencies of plaidchat, perhaps?
That might explain why it lists files as missing, but related files as also extraneous. Full log posted below. Not sure why it's only Plaidchat dependencies popping up and not other packages :confused:
/usr/lib
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>npm ERR! extraneous: buffer-to-vinyl /usr/lib/node_modules/plaidchat/node_modules/buffer-to-vinyl
npm ERR! error in /usr/lib/node_modules/plaidchat/node_modules/buffer-to-vinyl: ENOENT, open '/usr/lib/node_modules/plaidchat/node_modules/buffer-to-vinyl/package.json'
npm ERR! extraneous: decompress-unzip /usr/lib/node_modules/plaidchat/node_modules/decompress-unzip
npm ERR! error in /usr/lib/node_modules/plaidchat/node_modules/decompress-unzip: ENOENT, open '/usr/lib/node_modules/plaidchat/node_modules/decompress-unzip/package.json'
npm ERR! extraneous: download /usr/lib/node_modules/plaidchat/node_modules/download
npm ERR! error in /usr/lib/node_modules/plaidchat/node_modules/download: ENOENT, open '/usr/lib/node_modules/plaidchat/node_modules/download/package.json'
npm ERR! extraneous: download-status /usr/lib/node_modules/plaidchat/node_modules/download-status
npm ERR! error in /usr/lib/node_modules/plaidchat/node_modules/download-status: ENOENT, open '/usr/lib/node_modules/plaidchat/node_modules/download-status/package.json'
npm ERR! extraneous: duplexify /usr/lib/node_modules/plaidchat/node_modules/duplexify
npm ERR! error in /usr/lib/node_modules/plaidchat/node_modules/duplexify: ENOENT, open '/usr/lib/node_modules/plaidchat/node_modules/duplexify/package.json'
npm ERR! extraneous: meow /usr/lib/node_modules/plaidchat/node_modules/meow
npm ERR! error in /usr/lib/node_modules/plaidchat/node_modules/meow: ENOENT, open '/usr/lib/node_modules/plaidchat/node_modules/meow/package.json'
npm ERR! extraneous: merge-stream /usr/lib/node_modules/plaidchat/node_modules/merge-stream
npm ERR! error in /usr/lib/node_modules/plaidchat/node_modules/merge-stream: ENOENT, open '/usr/lib/node_modules/plaidchat/node_modules/merge-stream/package.json'
npm ERR! extraneous: read-all-stream /usr/lib/node_modules/plaidchat/node_modules/read-all-stream
npm ERR! error in /usr/lib/node_modules/plaidchat/node_modules/read-all-stream: ENOENT, open '/usr/lib/node_modules/plaidchat/node_modules/read-all-stream/package.json'
npm ERR! extraneous: vinyl-fs /usr/lib/node_modules/plaidchat/node_modules/vinyl-fs
npm ERR! error in /usr/lib/node_modules/plaidchat/node_modules/vinyl-fs: ENOENT, open '/usr/lib/node_modules/plaidchat/node_modules/vinyl-fs/package.json'
npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS><1.1.0-0, required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>

Well that's weird.
I uninstalled plaidchat completely and cleared my cache, and then reinstalled plaidchat globally - and viola! No more error reports when listing globally installed packages.
This is just a hunch - but I think I accidentally installed plaidchat (globally) while I happened to be in my local project directory. This may have caused some strange associations not present before - whatever the case, if anyone else has the problem, I recommend uninstalling plaidchat, clearing your cache, and making sure you are in your $HOME directory before reinstalling plaidchat globally.
Closing this as it's resolved :+1:

Ah, interesting. On a similar note, now that you mention it -- that would match the behavior of npm link which sets up plaidchat from a local directory to your global. As a result, it would expect devDependencies to be installed as well.
Anyway, thanks for the info and glad it's all fixed =)

it looks like this was fixed?
when i do
sudo npm -g update

on node v7.0.0 it no longer shows me extra modules


How should Postgres behave if etcd is unavailable / unpredictable?

Some of the scenarios to consider are:

governor for Postgres primary cannot communicate with etcd, but rest of cluster can
no governors in Postgres cluster can communicate with etcd
etcd crashes and recovers.  after recovery, etcd leader TTLs have expired.
etcd crashes and recovers.  after recovery, the initialization key is empty.  this would cause issues when new members would come online and race to initialize

The first decision to answer is: should Postgres cluster go readonly if etcd fails?  Or, should the Postgres cluster keep the current Primary, but not have automatic failover functionality?

I think it is best to go readonly so replication from the new elected master can continue.

In my opinion, i would prefer to keep the current Primary and do not take any "decision" when you loose the "brain". In general, i try to avoid the situation that the HA software/layer itself is able to bring down the protected application (and the protected application itself haven't any problems at all).
The Postgres cluster can go read-only as an additional safety measure, but it's not a must-have.


feat(x/asset): Add clawback and transfer auth priviledges

@facs95 can you take a look at this please


[Medium][LandingPage+Server] Add enterprise tier

We need an enterprise tier along with our existing ones on https://trypear.ai/pricing
This enterprise tier will be 35$/month per user for monthly subscription, and 30$/month per user for annual subscription.
Tasks

Think about how to display these extra ones on the pricing page? (UI/UX)
Enterprise tier will get a centralized billing option, where only one person (the business) has to buy the plan, then this person can choose to add more people to it. So only this one person will be charged for all users on the plan. This should involve

A separate section on the dashboard that indicates the total number of requests available (sum of # of users max quota in the plan).
A way for the user to add more people to its team through email. The invited user must accept the email to be added to the team officially, upon which their account will also have access to PearAI Pro.

Think about how to display these extra ones on the pricing page? (UI/UX)

maybe like cursor's pricing page? with a monthly and yearly tab and free / pro / enterprise cards.

I'll try to design the pricing page with the enterprise tier @Fryingpannn @Ellahinator


Add ContextMenu from left side

Currently, it's only possible to swipe from right to left to reveal a ContextMenu. There should be a possibility to swipe from left to right too and reveal a ContextMenu on the left side of the cell.

In Right To Left Approach:
Each Row is a Scrollview, in that two views are added.

1st view is Content View
2nd view is Context Menu
Since, you are adding Context Menu as 2nd view, it's hidden always. when touch happens, you are changing the ScrollX position of the Scrollview to reveal/dismiss the context Menu - its working fine.

But, Left to Right Approach:
We have to interchange the view order,

1st View is should be Context Menu.

2nd view is should be Content View.
When SetContextMenu method calls, contextview is creating and it's visible on the left side as default. We have to change the ScrollView's ScrollX value to ContextMenu's width, then only its getting hide.
When user scrolls, list view rows are recreating, context menu is visible as default, we have to scroll the scrollview every-time to hide the context menu. It’s glitching the animation. - is this understanding correct, any solution for this issue?

i think this below is the best approach :
Instead using Scrollview and adding two view inside that.
Inside a frame layout, stack context menu in back side and top of it add Content Menu.When touch happens, change the Content Menu’s Margin position, to reveals/hide the background context menu.
Any suggestions on this.

@TechBlackSparrow this works, I've done this in my private project. Personally, I don't like the visual effect of it though because the context menu won't move like it does now.

@Zure1  Left To Right context menu, you have done in your private project. is it working on Android also without any animation issue?.

No, it did have animation issues. I opened a stackoverflow question here about my problem. But someone linked to this project in the comments and I'm trying to add this feature to this problem right now.

Here is guideline, how to achieve "left menu"
there is nothing difficult - just set Rotation = 180 to ContextMenuView and to menu's main view and context view.

Good idea, but is it possible like this to have ContextMenu from left and right side?

@Zure1 hmm
Not sure, cuz we use scrollview.
But... Hm, but we can try )


add clustername for module Syslog

Signed-off-by: Federico Jose Zito<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area config
/area outputs

/area tests

What this PR does / why we need it:
Add Clustername for module Syslog with this we can recognize from which cluster it comes.
It is very necessary to know from which cluster the information is received.
https://github.com/falcosecurity/falcosidekick/issues/319
"output_fields":{"clustername":"test"}
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

@fjzito: There is not a label identifying the kind of this PR.
Please specify it either using /kind <group> or manually from the side menu.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: fjzito
To complete the pull request process, please assign fjogeleit after the PR has been reviewed.
You can assign the PR to them by writing /assign @fjogeleit in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/kind feature

/assign @fjogeleit

Hi,
falcosidekick already provides a way to add custom fields to all events, it replicates the exact pattern you set up but in a more global way. from my point of view, we should recreate this feature for only one plugin


feat: Add variables auto completion

Description
Add autocomplete for environment variables.
https://github.com/usebruno/bruno/assets/134304/701515eb-9e95-4571-89f5-ef71ec33eb5f
The trigger is {{ that shows all available environment variables. Supports all fields that have collection passed to it, i.e auth, headers etc.
Might need sanity check for all the places where it's currently applied.
Fixes #1141, #606, #412.
Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Please see here for more information.

I am having a little bug:

You can reproduce with the collection under /bruno/packages/bruno-tests/collection
with environment prod
The suggested value "process" shouldn't be here.
Also I don't think, you should display the value of the variable.
It will display secrets on the screen, which is not good in terms of security.

@busy-panda process is an nested object that could be accessed, normally via dot notation. It's only visible if it's actually set.
Regarding hiding values: there are couple existing issues for the hiding secret values and it should be done on the global level rather this particular case.

Nice job, this should be included in the following releases.


Issue with Github Copilot Account

When trying to connect Github Copilot in Accounts section, noting happens and I experience infinite loading.

Please check FAQ Q2 for details

I have also been experiencing this issue for a couple of versions (maybe since upgrading to 11.0? I don't recall the exact version to be honest).  Followed the FAQ, tried reinstalling multiple times from brew and release.  Verified all permission recommendations, no luck.

I have also been experiencing this issue for a couple of versions (maybe since upgrading to 11.0? I don't recall the exact version to be honest). Followed the FAQ, tried reinstalling multiple times from brew and release. Verified all permission recommendations, no luck.

I don't think you two are experiencing the same issue. The XPC service, aka the CopilotForXcodeEditorExtension is already running on your machine.
Based on the screenshot you provided, I noticed some stderr messages appearing. The app will not access those files, so I guess they are in your bashrc or some other bash configurations. These errors could have prevented Node from running. You can try either locate and fix the errors or disable the Run node with interactive logged-in bash option in the host app.

Oops, didn't notice the original question was also stuck Loading the service. You're right however, disabling the "Run node with interactive logged-in bash" setting fixed my issue. Thank you!

I am also having issues with the copilot account always loading. My xpcservice is running fine. I have node installed fine and can issue commands from the comman line to node. My privacy and scurity settings are all enabled. I have checked all the obvious issues the FAQ on questions 2 & 3. What would you need to help figure out why the copilot auth never loads? I would be happy to provide any details needed.

I am also having issues with the copilot account always loading. My xpcservice is running fine. I have node installed fine and can issue commands from the command line to node. My privacy and security settings are all enabled. I have checked all the obvious issues from the FAQ on questions 2 & 3. What would you need to help figure out why the copilot auth never loads? I would be happy to provide any details needed.

If you have already tried changing  the value of "Run node with", the next things you can try are:

Run the language server in terminal to see if it works: node /Applications/Copilot\ for\ Xcode.app/Contents/Applications/CopilotForXcodeExtensionService.app/Contents/Resources/copilot/dist/agent.js --stdio
Read the logs in Console.app check if there's any stderr sending by CopilotForXcodeExtensionService.

Here is the output. I didn't see any stderr output.

@jasonkruit please turn this off so logs won't be buried by the window related warnings

Hello, I have encountered another issue which may be related to this one. I have enabled Github Copilot and was hoping to log in through it, but ran into this problem.

@ErlichLiu Are you still experiencing the issue? The problem seems to be occurring inside the language server. It is likely due to the language server receiving HTML instead of JSON as a response when attempting to send a request. This could be caused by a temporary issue or other network issues. If you are using a proxy, please try disabling it. If not, please try enabling it and use enhanced mode, if possible.

Hi, I still experiencing this issue. I have tried disable the proxy, or use the enhanced mode, but I have not solved this problem. What else can I try to do?”

@ErlichLiu You may attempt to MITM the request sent by Node to *.github.com to troubleshoot the issue. Please note that I attempted this but was unsuccessful.
Alternatively, you may try examining the logs. By default, logging is disabled and there is no toggle to turn it on. So you will have to run the extension service target in Xcode in release mode:

Clone the source code and initialize the submodule copilot.vim. Stay on the main branch.
Go to GitHubCopilotService.swift, update the line localServer.logMessages = false to true.
Run the Extension Service target in release mode.
Open the installed host app, click remove launch agent and reload launch agent. Keep it open. the source extension should now talk to the Xcode-running extension service.

When you click sign-in, you should see logs like this printed to the debug area. Hopefully you can see some errors there.
2023-05-07 16:29:00.460980+0800 CopilotForXcodeExtensionService[92913:1390890] [ProtocolTransport] received: {"jsonrpc":"2.0","id":2,"result":{"status":"PromptUserDeviceFlow","userCode":"","verificationUri":"https://github.com/login/device","expiresIn":899,"interval":5}}

@ErlichLiu You may attempt to MITM the request sent by Node to *.github.com to troubleshoot the issue. Please note that I attempted this but was unsuccessful.
Alternatively, you may try examining the logs. By default, logging is disabled and there is no toggle to turn it on. So you will have to run the extension service target in Xcode in release mode:

Clone the source code and initialize the submodule copilot.vim. Stay on the main branch.
Go to GitHubCopilotService.swift, update the line localServer.logMessages = false to true.
Run the Extension Service target in release mode.
Open the installed host app, click remove launch agent and reload launch agent. Keep it open. the source extension should now talk to the Xcode-running extension service.

When you click sign-in, you should see logs like this printed to the debug area. Hopefully you can see some errors there.
2023-05-07 16:29:00.460980+0800 CopilotForXcodeExtensionService[92913:1390890] [ProtocolTransport] received: {"jsonrpc":"2.0","id":2,"result":{"status":"PromptUserDeviceFlow","userCode":"","verificationUri":"https://github.com/login/device","expiresIn":899,"interval":5}}

Thanks for your help. I couldn't get the biuld success because of some packet dependency issues, and I couldn't continue with the solution. Is there anything else I can try?

@ErlichLiu What exactly is the package resolution issue you're experiencing? It's possible that there could be an issue with your connection to GitHub. Have you made any changes to your hosts file related to GitHub?

@ErlichLiu What exactly is the package resolution issue you're experiencing? It's possible that there could be an issue with your connection to GitHub. Have you made any changes to your hosts file related to GitHub?
@intitni Thank you very much. It was my own issue. I had changed the IP address in my hosts file for GitHub before this happened. However, I have now restored it and everything is working perfectly fine again. Thank you once more.


Functions UI doesn't work with "*" CORS setting

If you create a function and set CORS to allow all sites, the Functions UI displays a persistent error and does not allow you to update any settings.
Error: "CORS is not configured for this function app. Please add https://functions.azure.com to your CORS list."
The Functions UI should allow CORS to contain just a wildcard (*) entry.

fixed in dev. https://github.com/projectkudu/AzureFunctionsPortal/issues/1015


Please include to update Reply URL's in Azure AD portal

[Enter feedback here]
This document is more for developers. When Developers are testing their applications locally they are in need to use localhost addresses in reply URL's.
So developers are thinking that Reply URL's need to update on App Service Authentication Settings like shown below.

Whereas asApp Services Authentication is not allowing local https://localhost or any configuration other than URI format (.com or .net ect.).
This document is misleading developers.
So please kindly include a note in this document to update Rely URL's on the Azure AD Enterprise Applications side, NOT on App Services side.
Please reach me @ napondur in Teams for any additional information.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: cc4c5921-4070-a714-44e6-c4309fe5fd53
Version Independent ID: c52c5c94-bb2e-10cb-9673-98aae4aa37d4
Content: Redirect URI (reply URL) restrictions - Microsoft Entra
Content Source: articles/active-directory/develop/reply-url.md
Service: active-directory
Sub-service: develop
GitHub Login: @madansr7
Microsoft Alias: saumadan

@nagasandeepMSFT
Thanks for your feedback! We will investigate and update as appropriate.

@nagasandeepMSFT
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.
@madansr7
Can you please check and add your comments on this doc update request as applicable.

@nagasandeepMSFT I see the issue but the document highlighted by @SaibabaBalapur-MSFT is probably not the doc that should be updated. The guidance on the app services side needs to be updated to state that if the developer is trying to test locally, they must go to the app reg blade and change the setting.
@SaibabaBalapur-MSFT  this should be re-assigned to someone in app services.

@nagasandeepMSFT
As you are internal  Microsoft Employee
Please use this form to submit your proposal to contribute to docs.microsoft.com (which includes Docs, Learn, Azure Architecture Center, and Cloud Adoption Framework). You'll receive a follow up once submitted to begin the process.
https://forms.office.com/Pages/ResponsePage.aspx?id=v4j5cvGGr0GRqy180BHbR958mqNUZExPsLDhzSGD8aFUNVBKTFpITkFGR0FWV0U2VVhIWU1EQUcxVi4u

@nagasandeepMSFT please use the links provided by @YashikaTyagii to create an internal update request. Closing the issue in lieu of internal follow up.
#please-close


Lots of 404 responses on main page

Lots of 404 responses on main page (https://highlightjs.org/):

Fixed and updated the site. Thanks for reporting it!


[BUG] How to Set http20ProxyFlag for grpc only

Library name and version
Azure.ResourceManager.AppService 1.2.0
Describe the bug
How I can use the SDK to set this configuration https://learn.microsoft.com/en-us/azure/app-service/configure-grpc#3-configure-the-http2-port to enable grpc support on my webapp? I did some research and I know that I need to config http20ProxyFlag but I cannot find in the SDK where and how to set this flag.
Expected behavior
How I can use the SDK to set this configuration https://learn.microsoft.com/en-us/azure/app-service/configure-grpc#3-configure-the-http2-port to enable grpc support on my webapp? I did some research and I know that I need to config http20ProxyFlag but I cannot find in the SDK where and how to set this flag.
Actual behavior
How I can use the SDK to set this configuration https://learn.microsoft.com/en-us/azure/app-service/configure-grpc#3-configure-the-http2-port to enable grpc support on my webapp? I did some research and I know that I need to config http20ProxyFlag but I cannot find in the SDK where and how to set this flag.
Reproduction Steps
How I can use the SDK to set this configuration https://learn.microsoft.com/en-us/azure/app-service/configure-grpc#3-configure-the-http2-port to enable grpc support on my webapp? I did some research and I know that I need to config http20ProxyFlag but I cannot find in the SDK where and how to set this flag.
Environment
.NET SDK:
Version:           8.0.403
Commit:            c64aa40a71
Workload version:  8.0.400-manifests.e99c892e
MSBuild version:   17.11.9+a69bbaaf5
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.22631
OS Platform: Windows
RID:         win-x64
Base Path:   C:\Program Files\dotnet\sdk\8.0.403\
.NET workloads installed:
Configured to use loose manifests when installing new manifests.
[aspire]
Installation Source: VS 17.11.35327.3
Manifest Version:    8.1.0/8.0.100
Manifest Path:       C:\Program Files\dotnet\sdk-manifests\8.0.100\microsoft.net.sdk.aspire\8.1.0\WorkloadManifest.json
Install Type:        FileBased
Host:
Version:      8.0.10
Architecture: x64
Commit:       81cabf2857
.NET SDKs installed:
3.1.426 [C:\Program Files\dotnet\sdk]
5.0.408 [C:\Program Files\dotnet\sdk]
6.0.135 [C:\Program Files\dotnet\sdk]
6.0.321 [C:\Program Files\dotnet\sdk]
6.0.427 [C:\Program Files\dotnet\sdk]
7.0.120 [C:\Program Files\dotnet\sdk]
8.0.306 [C:\Program Files\dotnet\sdk]
8.0.403 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 3.1.32 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 6.0.26 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 6.0.35 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 7.0.20 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 8.0.10 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 3.1.32 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.26 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.35 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.20 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 8.0.10 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.1.32 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 6.0.26 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 6.0.35 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 7.0.20 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 8.0.10 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Other architectures found:
x86   [C:\Program Files (x86)\dotnet]
registered at [HKLM\SOFTWARE\dotnet\Setup\InstalledVersions\x86\InstallLocation]
Environment variables:
Not set
global.json file:
Not found
Learn more:
https://aka.ms/dotnet/info
Download .NET:
https://aka.ms/dotnet/download

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Thank you @kaibocai for your feedback, @HarveyLink please look into this issue.

Hi @kaibocai ,
As the issue mentioned, https://github.com/Azure/azure-rest-api-specs/issues/25013, this property is not missing from the swagger.
Our SDK is auto generated from the service sepc, so we don' have this property either.
You could open an Azure support request to get further help, let the service team add this property to the service spec. We could quickly release a beta version fix for it.


fix(localizations): Fix Hungarian language subpath exports

Description
This is needed so developers can import from @clerk/localizations/hu-HU
Checklist

[x] npm test runs as expected.
[x] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[x] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Nice catch!!
It seems there the "mn-MN"  is missing from the packages/localizations/src/index.ts and the "ca-ES", "nl-BE" from the packages/localizations/src/utils/generate.ts.
I think we should refactor our current set-up and avoid having all the references.
cc: @nikosdouvlis , @anagstef


Create a random mac address

Having all machines have C0:FF:EE:C0:FF:EE as their mac address is very inconvenient, eg they are indistinguishable... Can we generate a random address please...

Perhaps we should allocate in the CF block https://tools.ietf.org/html/rfc5342#section-2.3.2 to avoid collisions and confusion about OUIs?

Why not just generate some random bytes, set the locally administered bit and clear the multicast bit?


Handle timeouts

As noticed in https://github.com/huggingface/datasets/issues/1939, timeouts were not properly handled when loading a dataset.
This caused the connection to hang indefinitely when working in a firewalled environment cc @stas00
I added a default timeout, and included an option to our offline environment for tests to be able to simulate both connection errors and timeout errors (previously it was simulating connection errors only).
Now networks calls don't hang indefinitely.
The default timeout is set to 10sec (we might reduce it).

I never said the calls were hanging indefinitely, what we need is quite different - in the firewalled env with a network, there should be no network calls or the should fail instantly.
To make this work I suppose on top of this PR we need:

DATASETS_OFFLINE env var to force set timeout to 0 globally (or to 0.0001 if 0 has a special meaning of no timeout)
DATASETS_OFFLINE should guard against failing network calls and not fail the program if it has all the data it needs locally.

Bottom line - if the logic wants to check online if the local file matches online dataset name, let it go wild, but it should fail instantly, recover and use the local file - either specified explicitly or cache if there is one. And only if neither was found only then assert.
I hope this makes sense and is doable.
Thank you, @lhoestq

Yes that was the first step to add DATASETS_OFFLINE :)
With this PR, if a request times out (which couldn't happen before because no time out was set), it falls back on the local files with no error.
As you said, setting the timeout to something like 1e-16 makes the requests fail instantly, which is one step forward. One last thing left is to disable request retries and everything will be instant !

Ah, fantastic. Thank you for elucidating that this PR is part of a bigger master plan!

Merging this one, then I'll open a new PR for the DATASETS_OFFLINE env var :)


Set test environment

Summary
We need 2+ chains based cosmos-sdk, and one relayer.
Also we need to connect each chain and relayer, so it is better making auto test environment.

Making script or infrastructure for initiating chain instances, and relayer.
Making configuration for connecting chain and relayer.

Do you consider using the Github actions runner?
It seems like IBC-go already uses actions runner

Yes, I also prefer to use github actions.


#8 add write method overload with input stream

Refs #8

Please solve conflicts @fg-devs

@dmvieira resolved

Thx!!!


The link to the Google Summer of Docs resources on the CNCF Contributor page is broken.

Description:
There’s a broken link on the CNCF Contributor page related to the Google Summer of Docs program. The link is supposed to guide contributors to relevant resources but currently leads to a dead page.
Steps to Reproduce:

Navigate to the CNCF Contributor Page.
Scroll down to the "Contributor Opportunities" section.
Click on the "details" link under the Google Summer of Docs program: Google Summer of Docs Link.
The link currently leads to a 404 error.

Please update the link with the correct URL.

updated


ME Conduits don't work properly with the ME Security Terminal

If I add a security terminal to my network and exclude build from the default permissions, ME Conduits get all screwy. Placing them usually causes adjacent parts of the network to break, and connections do not always work. Example: the left panel is on, but right panel is not:

Info, just in case it's useful:

Ender IO version = <IP_ADDRESS>9
Applied Energistics 2 version = rv2 alpha 16
Forge version = <IP_ADDRESS>5
I have > 100 other mods installed

When using a security terminal, you can put in a blank card with build permissions to fix it. Conduit doesn't have the right security permissions in it yet (my guess).

Fixed for the next version


Blender Cycles and EEVEE SG's

Question.
After playing around with BakingLab i am asking myself if Blender Cycles as Path Tracer for baking LightmapSG s and Blender EEVEE for fully show off would be working?
A fan.

Unfortunately I have no experience with Blender or its path tracer, so I'm not sure how well it would for baking lightmaps. Generally though if you have the ability to write some code or script that receives the resulting radiance for a ray direction and that can write out the final lightmap texel value, then you would be able to encode the lighting as SH/SG/etc.

I did some fast tests. Some pictures for better explanation.
Simple Geo with IBL. Blender Cycles

Blender EEVEE with an precalculated irradiance volume and an reflection probe.

I searched for the already integrated SH Lightprobes in the Blender Source Code and found some of the spherical_harmonics_L2 and hl2_basis code.
https://github.com/sobotka/blender/blob/ef0e06d7644727883c250975133264b046930b70/source/blender/draw/engines/eevee/eevee_lightprobes.c
https://github.com/sobotka/blender/blob/ef0e06d7644727883c250975133264b046930b70/source/blender/draw/engines/eevee/shaders/irradiance_lib.glsl
The SG Probe volume generation could be pretty forward because the implementation for SH L2 is already there.
The SG Lightmap generation like you did in BakeLab could be harder to add.
But it think to have SG Lightmaps and SG Volume plus mix it with Reflection Probes like you did in The Order could be archive really nice results in EEVEE too and would allow to use your methods in an confortable open source project.
For now i am not sure how the SH L2 are calculated in EEVEE . But the comparision with ground truth and generation of SG s would be really fast and flexible with Cycles.
Cycles can run on CPU, CUDA GPU and since today even on Optix RTX path per option.
So pretty fast an flexibel.

I did some fast tests. Some pictures for better explanation. Simple Geo with IBL and Disneys Principled materials. Blender Cycles. Path-traced.

Blender EEVEE (Rasterisation) with an precalculated irradiance volume and an reflection probe.

model from katarn66: [MEDIA=youtube]Q09wNDtAtVE[/MEDIA]
I searched for the already integrated SH Lightprobes in the Blender Source Code and found some of the spherical_harmonics_L2 and hl2_basis code.
https://github.com/sobotka/blender/blob/ef0e06d7644727883c250975133264b046930b70/source/blender/draw/engines/eevee/eevee_lightprobes.c
https://github.com/sobotka/blender/blob/ef0e06d7644727883c250975133264b046930b70/source/blender/draw/engines/eevee/shaders/irradiance_lib.glsl
The SG Probe volume generation could be pretty forward because the implementation for SH L2 is already there.
The SG Lightmap generation like you did in BakeLab could be harder to add.
But it think to have SG Lightmaps and SG Volume plus mix it with Reflection Probes like you did in The Order could be archive really nice results in EEVEE too and would allow to use your methods in an comfortable open source project.
For now i am not sure how the SH L2 are calculated in EEVEE . But the comparision with ground truth and generation of SG s would be really fast and flexible with Cycles. Cycles can run on CPU, CUDA GPU and since today even on Optix RTX path per option. https://code.blender.org/2019/07/accelerating-cycles-using-nvidia-rtx/?utm_source=www-homepage So pretty fast an flexibel.
All my scenes i did over the last years have perfectly LightMap UV's on UV1.
After baking several days all your test scenes in Bake Lab i am really amazed by the results and see I see so much potential for HQ scenarios with framerates 60+.
Also all the comfortable authoring like create Lightmap UV, create Irradiance Volumes, create Reflection Probes , create bake jobs...... is already there in Blender. So hope you can understand why i am so “hyped“.)

hi , do you know how blender  Interpolate light probe volume?


Staging 0.37.0

[!warning]
This is a public repository, ensure not to disclose:

[x] personal data beyond what is necessary for interacting with this pull request, nor
[x] business confidential information, such as customer names.

What kind of PR is this?
Required: Mark one of the following that is applicable:

[ ] kind/feature
[ ] kind/improvement
[ ] kind/deprecation
[ ] kind/documentation
[ ] kind/clean-up
[x] kind/bug
[ ] kind/other

Optional: Mark one or more of the following that are applicable:

[!important]
Breaking changes should be marked kind/admin-change or kind/dev-change depending on type
Critical security fixes should be marked with kind/security

[ ] kind/admin-change
[ ] kind/dev-change
[ ] kind/security
[ ] kind/adr

What does this PR do / why do we need this PR?

Qa for apps v0.37.0 completed.

Information to reviewers

Checklist

[ ] Proper commit message prefix on all commits

Change checks:

[ ] The change is transparent
[ ] The change is disruptive
[ ] The change requires no migration steps
[ ] The change requires migration steps
[ ] The change upgrades CRDs

Metrics checks:

[ ] The metrics are still exposed and present in Grafana after the change
[ ] The metrics names didn't change (Grafana dashboards and Prometheus alerts are not affected)
[ ] The metrics names did change (Grafana dashboards and Prometheus alerts were fixed)

Logs checks:

[ ] The logs do not show any errors after the change

Pod Security Policy checks:

[ ] Any changed pod is covered by Pod Security Admission
[ ] Any changed pod is covered by Gatekeeper Pod Security Policies
[ ] The change does not cause any pods to be blocked by Pod Security Admission or Policies

Network Policy checks:

[ ] Any changed pod is covered by Network Policies
[ ] The change does not cause any dropped packets in the NetworkPolicy Dashboard

Audit checks:

[ ] The change does not cause any unnecessary Kubernetes audit events
[ ] The change requires changes to Kubernetes audit policy

Falco checks:

[ ] The change does not cause any alerts to be generated by Falco

Bug checks:

[ ] The bug fix is covered by regression tests

Config checks:

[ ] The schema was updated

I'm missing the update to the welcome dashboard. Unless that is supposed to come later.

Yes, we are still using the old templates and the update the welcome dashboards step is not present in the release doc.
I updated them now. Please check.

I'm missing the update to the welcome dashboard. Unless that is supposed to come later.

Yes, we are still using the old templates and the update the welcome dashboards step is not present in the release doc. I updated them now. Please check.

I don't see any updates to the welcome dashboards. Did you miss to push them? :thinking:

I'm missing the update to the welcome dashboard. Unless that is supposed to come later.

Yes, we are still using the old templates and the update the welcome dashboards step is not present in the release doc. I updated them now. Please check.

I don't see any updates to the welcome dashboards. Did you miss to push them? 🤔

yeah I did that now


different results from variants/typeahead_results and datatables/variants endpoints

The gene MGMT shows up in the datatables/variants endpoint, but not the variants/typeahead endpoint.
http://<IP_ADDRESS>:3001/api/variants/typeahead_results?limit=12&query=MGMT
vs
http://<IP_ADDRESS>:3001/api/datatables/variants?count=0&filter[entrez_gene]=MGMT&page=1&sorting[variant]=asc
(genome/civic-client#120 depends on this)

Fixed in master!


Add support for Oracle as backend database

Also, a general comment here - we are supporting almost 3 dbs now, will it be a good idea to use migrations now than actual DDL like Flyway.

Also, a general comment here - we are supporting almost 3 dbs now, will it be a good idea to use migrations now than actual DDL like Flyway.

Totally agree .. that could also help with installation/ bootstrap

Also, a general comment here - we are supporting almost 3 dbs now, will it be a good idea to use migrations now than actual DDL like Flyway.

We are using flyway in trino for resource group migrations so could likely reuse some of what we did there. Are we good with doing this in a follow up PR though?

Are we good with doing this in a follow up PR though?
I agree that we should split these up. Either first Oracle, then Flyway, or the other way round.

Fyi @ebyhr  ... any thought of adding an abstraction layer like Flyway or so? Also given you are ripping over from activejdbc to jdbi it would be good to have a coherent plan in place

At minimum a rebase has to be done. I can review more then ..

@ebyhr the problem I'm facing now is that the SQL in QueryHistoryDao is not compatible with Oracle (limit is not supported in Oracle). Didn't have this issue with activejdbc when calling QueryHistory.limit().
Any suggestions on the preferred approach to deal with this in JDBI?

@posulliv We can call different dao methods based on the connection-url prefix jdbc:oracle.

Chatted with @willmostly .. I think this can be closed and restarted when it becomes relevant. Feel free to reopen after rebasing and adjusting


Unable to reproduce the data preprocessing

I tried to modify the dataset_creation.ipynb to reproduce the data preprocessing. I found all of the rest (changemind, isold, iscorrect, etc. can match the dataset you provided). However, the betas_all I obtained is different from the file you provided, i.e. betas_all_subj01_fp32_renorm.hdf5.
Below is the procedure to get betas_all in my script
import time
from subprocess import call
import numpy as np
import matplotlib.pyplot as plt
import h5py
import os
import copy
import pandas as pd

from tqdm import tqdm
from PIL import Image
from sklearn.preprocessing import StandardScaler

import webdataset as wds
import sys

# nsd_access is from this repo: https://github.com/tknapen/nsd_access
# also see https://cvnlab.slite.page/p/dC~rBTjqjb/How-to-get-the-data for how to download the NSD data!
from nsd_access import NSDAccess
nsd_path = '/gpfs/radev/pi/ying_rex/wq44/natural-scenes-dataset'
nsda = NSDAccess(nsd_path)

import nibabel as nib

import torch
device = torch.device('cuda' if torch.cuda.is_available() else 'cpu')
print("device:",device)

shared1000 = np.load("shared1000.npy")
for sub in [0]: #,1,2,3,4,5,6,7]:
    subject=f'subj0{sub+1}'
    subj=subject
    print(subject)

    abs_cnt = -1
    abs_notshared1000_cnt = -1
    abs_shared1000_cnt = -1

    # load coco 73k indices
    indices_path = "./data/COCO_73k_subj_indices.hdf5"
    hdf5_file = h5py.File(indices_path, "r")
    indices = hdf5_file[f"{subj}"][:]

    nsessions_allsubj=np.array([40, 40, 32, 30, 40, 32, 40, 30])
    nsessions=nsessions_allsubj[sub];
    ntrials = 750*nsessions

    file = f"./data/natural-scenes-dataset/nsddata/ppdata/{subject}/func1pt8mm/roi/nsdgeneral.nii.gz"
    nifti = nib.load(file)
    mask = nifti.get_data()
    mask[mask<1] = 0
    nsdgeneral_mask = mask

    for tar in tqdm(range(nsessions)):
        sess=tar+1

        behav = nsda.read_behavior(subject=subject,
                    session_index=sess,
                    trial_index=[])

        # pull single-trial betas and mask them
        betas = nsda.read_betas(subject=subject,
                            session_index=sess,
                            trial_index=[], # empty list as index means get all for this session
                            data_type='betas_fithrf_GLMdenoise_RR', # GLMSingle beta2
                            data_format='func1pt8mm')

        # betas = betas[mask]
        betas = np.moveaxis(betas,-1,0)

        vox_include = copy.deepcopy(nsdgeneral_mask)
        # ncsnr = nib.load(f"/home/wq44/data/natural-scenes-dataset/nsddata_betas/ppdata/{subject}/func1pt8mm/betas_fithrf_GLMdenoise_RR/ncsnr.nii.gz").get_fdata()
        # ncsnr[ncsnr<.15] = np.nan
        # if tar==0: print("voxels left:", len(vox_include[vox_include>0]))
        # vox_include[np.isnan(ncsnr)] -= 1 # keep all nsdgeneral voxels even if they are below the threshold
        # vox_include[vox_include<0] = 0
        # if tar==0: print("voxels left after ncsnr thresholding:", len(vox_include[vox_include>0])) # subj01 = 49329

        betas = betas.reshape(len(betas),-1)
        betas = betas[:,vox_include.flatten().astype(bool)]
        shape = betas.shape

        train_index = []
        for i in range(len(behav)):
            global_trial = i
            coco73 = int(behav.iloc[i]['73KID'])-1
            if shared1000[coco73]:
                continue
            train_index.append(global_trial)
        global_trial = np.array(train_index)
        scalar = StandardScaler(with_mean=True, with_std=True).fit(betas[train_index]) # YOU SHOULD EXCLUDE SHARED1000 FROM THIS (NOT DONE HERE BUT DONE IN ACTUAL MINDEYE2 PAPER)
        betas_mean = scalar.mean_
        betas_std = scalar.scale_
        betas = (betas - betas_mean) / betas_std
        betas = betas.reshape(shape).astype('float16') # (1, 15724)

        globals()[f'betas_ses{sess}'] = betas
        globals()[f'behav_ses{sess}'] = behav
        print(betas.shape)

    for tar in range(nsessions):
        sess=tar+1

        if sess==1:
            betas_all = globals()[f'betas_ses{sess}']
        else:
            betas_all = np.vstack((betas_all,globals()[f'betas_ses{sess}']))

Hi just a follow-up on this issue. Really appreciate it if you can help with that!

Hi, this code wasn't actually the code we used to get the betas files in the huggingface repo -- actually the betas we obtained were created by coauthor Reese Kneeland as described in the MindEye2 paper -- this script was what we were initially using before Reese provided updated betas (which explained the commented out part of the code saying "YOU SHOULD EXCLUDE SHARED1000 FROM THIS", this is the reason why we switched to Reese's betas). Note that both the betas obtained from this script and the ones provided by Reese led to nearly the same results though. I'd recommend you just take the data directly from the Natural Scenes Dataset:

@PaulScotti Actually, I have the same issue. Could you more elaborate the data processing procedure?
I used betas_fithrf_GLMdenoise_RR with func1pt8mm and ensured that the Shared1000 data is not included in the calculation of mean and std of each session. But, I failed to reproduce your provided betas.

Update: I resolved the problem. In the mindeye2, it seems that all the mean and std value are calculated with entire training dataset (not session-wise).

Ah, yes that would explain it. Sorry I forgot that there was this difference!


SqlAzureDacpacDeploymentV1: Quote user supplied strings

Task name: SqlAzureDacpacDeploymentV1
Description: Quote user supplied strings to remove ambiguity
Documentation changes required: N
Added unit tests: N
Attached related issue: N
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected

Hi @Frankkkkk , thanks for contributing.
In order to procced with approvals and to merge this PR, it is necessary to build and test these changes.
So for that, follow the below simple instructions. Build, test and commit changes to this branch. I will check with relevant task owners to approve this.
Local task test
Have code changes ready (checkout to this branch), navigate to task root path ie >C:...\azure-pipelines-tasks
Note: tfx build works only on node 8 and 10, so make sure to use node 8/10 while testing the task
Step 1: run "npm i"
Step 2: Build:
node make.js build --task SqlAzureDacpacDeploymentV1
Step 3: Install tfx:
npm install -g tfx-cli
Step 4: Login:
tfx login
Service URL : [your task test org url]
PAT :
Step 5: Upload:
tfx build tasks upload --task-path C:\AzurePipelineTask\azure-pipelines-tasks \ _build\Tasks\SqlAzureDacpacDeploymentV1
New task version will be uploaded to your org, please test the changes by running the pipeline.
Step 6: Delete: // optional and can be done after all the testing is completed.
tfx build tasks delete --task-id { }

Works fine, thanks!

Hi @v-mohithgc any news before I rebase please ? Thanks!


[Feature Request] Add more options to Deluge (add torrent from link, etc.)

I wanted to start an issue here for adding torrents to deluge. I will be starting work on it today, it should be fairly easy but I wanted a place to track it and ask for advice/help :) I was going to do it on styxit's repo, but he's not much for adding other people's work anymore.
I will work on adding torrents from a link first and if I have any success, I'll try to figure out how to upload a .torrent file (don't hold your breath on this one...)

Thats awesome, let me know if you need any help. Check out https://github.com/Hellowlol/HTPC-Manager/blob/master2/modules/deluge.py#L110 that is what is used to send a link to deluge for torrent search

Thanks for the pointer. I'll definitely look at that. I'm sure I'll need help, it's just a matter of time. Sometimes I have to just sit at stare at it for a few hours before I decide what I need to do haha.

Regarding the add a local torrent take a look on how I did that for transmission and use add_torrent_file(filename, filedump, options)

@coach0742 is this something your still working on?

I've decided I'm terrible at python, and more importantly, I'm not patient enough to troubleshoot my issues. I don't think I'll be working on this anymore.

Ok, need a hand?

@coach0742 Is this something you still want?

This would be awesome! As a fellow user of Deluge, I feel like there's not much support for it in this project. Would mainly like to see the downloads in the dashboard :)

@d2dyno Can you specify what you want to be able to do with the deluge module? Regarding downloads on the dashboard are you thinking about something like we have done for qbittorrent?

I don't use qbittorrent, so I am not sure what has been done for it. The screenshot on the main Git page, has what looks like (I'm supposing qbittorrent) downloads and torrents widgets. It'd be nice to get those for Deluge as well.

Ok. It's pretty simple. It just shows 5 downloads. I can do that, but I have no eta I got some other stuff I want to finish first.

Sweet, sounds good! :)

@Hellowlol Crap, I completely forgot about this. It absolutely something I still want. I know I could move over to another client like qbittorrent or transmission, but Deluge is still the only client that does everything I want it to.
I would like to take another stab at writing it myself, but I'm afraid I'm quite terrible at python and don't really know what I'm doing.

Then we are two :P

Let me know if you need any help. I'll work on deluge on on the dash

@coach0742 I got time to work on this in a couple of weeks, if your aren't done by then I'll add upload torrent, pause/start all and change speed.

@coach0742 @d2dyno Its better late then never. Added add url, magnet and local torrent to deluge, set speed limit up and down and pause resume all. Please give it a spin and close this request if this is what you wanted


Add Maven instructions to README

can you elaborate?

The readme does not explain how to build the code. We need enough instructions to enable someone outside Google to build this easily.

done


FreeRDP#5329 if using OldLicenseBehaviour, don't try to save the Cal …

…since we're not going to try to load it

@freerdp-bot test

@byteboon Thank you for this patch!


Orphaned fork handling in p2p

Related:  #115 #118.
PR #118 only implements deletion of blocks of height lower than LIB.
The architectural intent is that TopologyCache holds unverified claims of peers' topology, while MyTopologyCache holds the topology that has been accepted by the local node.
By definition the local node cannot accept orphaned forks, that is, blocks whose history branched at any point older than LIB. When advancing the topology cache, MyTopologyCache should prune all blocks on an orphaned fork.
For an example of this situation, suppose our head is 100, our LIB is 90, and there were two forked blocks 91, let us call them 91a and 91b. We have five blocks on the b fork, 91b...95b, as well as 91a...100a on the main fork. Then we receive block 101a, which advances LIB to 92a. The current code discards only 90, 91a and 91b. That is, the code implements only the first of these three rules:

(1) Discard all blocks with height less than LIB.height (i.e. 90, 91a and 91b should be discarded)
(2) Discard all blocks of LIB.height except LIB itself (i.e. 92b should be discarded)
(3) Discard all descendants of blocks discarded according to rule (2) (i.e. 93b-95b should be discarded)

Since rule (1) is already implemented, the purpose of this issue is to implement rules (2) and (3).

We should also refactor deletion into its own method.

This is no longer a problem once #166 is merged


Release check list

Produce a document (on the wiki) enumerating everything we need to do for a release.

Julien's document is a good start
Docker image publication needs to be documentated

Need to mention how the NPM (SockJS bridge) is deployed:
cd target/client
npm publish


Updated dead links

This change is

This should be taken care of by PR #2496, but if that gets held up, I will push this one through.


New collection added : made-in-cameroon

Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/main/CONTRIBUTING.md.
[x] I am not the sole author or employee of a company who created either the topic I am modifying/adding or the collection entry I am modifying/adding.

Which change are you proposing?

[ ] Suggesting edits to an existing topic or collection
[x] Curating a new topic or collection
[ ] Something that does not neatly fit into the binary options above

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288, size <= 75 kB)
[ ] Content (and my changes are in index.md)

Please replace this line with an explanation of why you think these changes should be made.

Curating a new topic or collection

[x] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[x] My folder contains a *.png image (if applicable) and index.md
[x] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docs

Please replace this line with an explanation of why you think this topic or collection should be curated.

Something that does not neatly fit into the binary options above

[ ] My suggested edits are not about an existing topic or collection, or at least not a single one
[ ] My suggested edits are not about curating a new topic or collection, or at least not a single one
[ ] My suggested edits conform to the Style Guide and API docs: https://github.com/github/explore/tree/main/docs

Please replace this line with an explanation of your proposed changes.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Super

Hi,
Thanks for starting this pull request. In order to review it for acceptance, we need to make sure that all of the prerequisites are satisfied. I see that you checked this:

I am not the sole author or employee of a company who created either the topic I am modifying/adding or the collection entry I am modifying/adding.

However, your contribution does not meet this requirement. This is required so that we can maintain a high level of independence in this project.
Thanks!

Hello,
I don't see how this addition breaks the rules.
This is a collection for projects developed by Cameroonians, so of course I'm not the only author.
I also drew inspiration from existing collections ( made-in-france, ... )


Add build-wrapper fragments for maven and gradle

This introduces an option to include build wrapper that is enabled by default but can be turned off to prevent issues in air-gapped environments where the builds won't be able to download wrapper files.
There are two separate fragments, one for "maven" and one for "gradle".
For the multi-module accelerators (spring-smptp-gateway and where-for-dinner) the mvn wrapper has been removed from the modules and will only be added to the top level parent directory.

PR - https://github.com/vmware-tanzu/application-accelerator-samples/pull/162

Rebased and squashed


Compile with deprecation warnings & fix deprecations

This builds upon #271 to prevent conflicts, please review #271 first.
This PR here lets the compiler warn on deprecations, and fixes deprecations:

Deprecation of ToxiproxyContainer.ContainerProxy - refactors proxied kafka and postgresql containers, also moving them to commons so that they can be shared between the classic and reactive modules
Replace deprecated AutowireCapableBeanFactory.applyBeanPostProcessorsAfterInitialization with initializeBean
Replace deprecated org.junit.Assert.assertThat with the hamcrest version

Anybody of you up for a review, @hnrkdmsk @danielrehmann?


Tests, File Browsing, Refactoring

Sorry, looks like this is going to be a bunch of unrelated changes in one PR.


When executing a highstate of apiserver make sure that we check the local apiserver instance

When executing the highstate make sure the apiserver we are checking
is the local one, not any master through haproxy.
Partial backport of https://github.com/kubic-project/salt/pull/384
Fixes: bsc#1079460

WIP as I still need to check if it fixes the upgrade issues to 1.8 with multi master. If it does, I will add a Fixes: bsc# to the commit, so I'll drop any reviews at that time.

 [QA] Bug 1079460 - update kubernetes to 1.8.7 fails


Bump up version for unit test report publish report

Feature to Enhance
Current version often gave API rate limit error

Requested Enhancement
Use newer version

Business Need
Avoid secondary api rate limit error

Additional Context
No response

Already finished in PR #926


fix: non-deterministic apply contract

Contract::document_types is HashMap which has RandomHasher by default so it doesn't guarantee order. Eventually, insert_contract creates trees for document types in a different order that leads to different root hash.

Very good catch btw @shumkov


NetBSD support with envsys ioctl and plist

New examples, simple.rs only refreshed the first battery, and this does not help checking for iterator validity for development, all.rs solves the problem. The example oneshot.rs is just for comfort avoiding loop.
NetBSD exposes battery information through one ioctl so for now we have to "waste" resource getting all sensors info for each single battery refresh. We can't assume the library user will want to refresh all bateries at the same time even if 90% of time it will probably be.
The result of the ioctl is a plist (apple format) that needs to be parsed and accessed.
The plist library only uses options, so this is the reason for utils.rs. These are all oneline functions to reduce the lenght of chains in the netbsd platform device.rs file.
NetBSD does not expose battery vendor, model, technology, cycle among other things for now, but these are optional.
Got inspiration from the FreeBSD port regarding having a better understanding on how the library works.
Thanks to unitedbsd.com community for the guidance, and shout out to https://github.com/distatus/battery/blob/master/battery_netbsd.go as well as the NetBSD devs for the great man pages and the clear code allowing to read what happens for undocumented things.
My code has comments for anyone interested to understand what happens.
By lack of hardware (only tested on one laptop), I couldn't test some specific things such as cases where battery use the watt units instead of ampere. I implemented that with the NetBSD man pages and inspiration from the FreeBSD port.
This port was only tested with a battery using the acpibat driver, but this should work with any battery that follow the same standard in the the sysmon_envsys format and fields.

This repository not having ffi anymore, I think the README could be updated.

I might need help with setting the build CI for NetBSD though. The NetBSD vm-action from the same source as FreeBSD seems to lack libexecinfo.

@naguam If this only requires a newer build image, maybe you can point cross to a more recent one via Cross.toml?
Furthermore, you think the plist-helpers you be replaced by instead using a struct with derive(serde::Deserialize)?

@naguam If this only requires a newer build image, maybe you can point cross to a more recent one via Cross.toml?
Furthermore, you think the plist-helpers you be replaced by instead using a struct with derive(serde::Deserialize)?

I've never used Serde directly, It might me possible to use it, but only might. A friend (Actually @ThalusA) told me it could be possible to use it, but on the other hand, I don't think I can rely on the order of the data and format.
This is why these are property lists and not plain C structs exposed throughout the ioctl after all like FreeBSD does in acpi.rs after all.

@davidkna sorry for all the edits in my answers, I just tried to make it better.

I don't think I can rely on the order of the data and format, it could be if we read how they are built in kernel code, but not according to userspace documentation AFAIK.

serde should not care about the ordering in the plist. Nevertheless, I am not sure if the way the library maps to serde would not cause any issues.
To elaborate on how the Cross.toml file could help: Cross.toml can be used to configure the use of different docker base images for each target, as I mentioned a more recent one could solve the compile issues.

I think I don't understand.
Of course cross can specify other ways of building, but the ways of building and versions it recognises depends on the versions of cross as far as i can understand.
The way it is setup here,
https://github.com/starship/rust-battery/blob/main/.github/workflows/ci.yml
is installed by https://github.com/taiki-e/install-action which if we check.
https://github.com/taiki-e/install-action/blob/main/manifests/cross.json

Gets its version from the release page tarball.
https://github.com/cross-rs/cross/releases
Which means that the available target either Dockerfile image or direct script, are not containing the fix.
Maybe I missed something, but I probably need guidance.
Should I push in the repo the even more recent netbsd edge script, and try running it manually.
I don't think it is maintainable. Maybe my first proposal is easier (disabling the backtrace until new release comes)

@naguam Cross.toml configures cross itself and is unrelated to the version that is installed (reference). It might be possible to apply the required fix-ups via this file too via pre-build.

Ok I think I managed to fix the CI by manually implementing the fix.
Also added clippy fixes.
Once the next proper release of the cross-rs will be published this hot fix will have to be removed (especially if NetBSD version is bumped to 10).

By lack of hardware (only tested on one laptop), I couldn't test some specific things such as cases where battery use the watt units instead of ampere. I implemented that with the NetBSD man pages and inspiration from the FreeBSD port.

Virtualbox translates an acpi battery which is in watt unit.
I have tested my code and my assumptions were correct, so it works, did not need to make changes.

For the CI thanks, I did not know I could do that
(it is probably in the documentation of cross-rs though)
However don't merge it too quickly
c.f. https://www.unitedbsd.com/d/1311-battery-data-such-as-vendor-model/22
It works but it is based on descriptions, this need to do a full rework, probably using serde directly.
I don't have the motivation for now.

In that case, perhaps return an error if a different driver is used for now?

In that case, perhaps return an error if a different driver is used for now?

Yes with the new patch only acpibat is supported I I do not rely on descriptions anymore.

Thanks for the contribution @naguam!


Added more Django versions to the build matrix

This package should probably support more Django versions.
1.3 is no longer supported so I'm not sure if we want to keep supporting it ourselves.

Some of the failures are simple deprecations.
The Django 1.7 failure is something that is probably related to the test suite and is easy to fix.
I'd give it a shot later if anyone else won't beat me to it.

:+1:

1.3 is no longer supported so I'm not sure if we want to keep supporting it ourselves.

I don't see any problem removing 1.3 support.

The 1.7 failure is because of https://github.com/fusionbox/django-emailtools/blob/master/emailtools/cbe/mixins.py#L2
The import of Site just needs to be moved into the method call(s) that use it.

I need #6 and this PR merged before I can tackle #9 and #11.

#6 is merged.


Mini Crash Dump Generation Feature

There are times when a crash is being hit by the fuzzer but unable to reproduce it with the generated log file.
If you are facing the same issue, Mini Crash Dumps are of great help for further investigation.
By default it's turned ON and if you want to disable it set $save_minidump to false in config.rb
By default the crash dumps are stored in $crashes_dir as set in config.rb
Further, if you want to change the amount of information contained in Mini Crash Dump, please set appropriate MINIDUMP_TYPE in node/core/debug/debuggerexception.rb

Merged! Thanks the pull request :)


Improve spatial_mean

Generic Issue
While it works well, there are some caveats.

[ ] "Global" mean with xESMF : We need a way to specify we want to average over the whole grid with the xESMF method.
[x] "cos(lat)" weighting : For the non-xESMF methods, it could be interesting to be able to weights the grid cells according to their real size with the easy cos(lat) approximation.
[ ] xESMF inputs : We need to re-implement the possibility to pass a GeoDataframe as a "shape".
[ ] xESMF : Better error message when polygon cleaning is needed., i.e. : when gdf.simplify(tolerance=XX, preserve_topology=True).buffer(0) is needed. Could also be done automatically, but we'd need a way to guess the tolerance (it depends on the polygon's CRS, on the grid's resolution and on the target precision).
[ ] Segmentize polygons if needed before sending to xESMF and remove warning.
[ ] Expose xESMF's out_chunks arg for the regridder (xESMF 0.8)

Currently, a workaround would be to use smaller polygons. For example, this uses 1 rectangle per 1° of longitude and gives the right answer:
eps = 1e-4
p = []
for lon in range(-180, 180):
    p.append(Polygon([(lon, -90 + eps), (lon, 90- eps), (lon + 1, 90- eps), (lon + 1, -90 + eps)]))

savg = xe.SpatialAverager(ds, p)
savg.weights.max()

savg(ds.tg_mean).mean('geom') - 273.15

Another solution is to circumvent SpatialAverager completely and use ESMF (not xESMF) to find the area covered by each pixel:
from ESMF import Field
import xesmf as xe

grid_in, _, crds = xe.frontend.ds_to_ESMFgrid(ds, need_bounds=True)
area_raw = Field(grid_in, name='area')
area_raw.get_area()

area = xr.DataArray(area_raw.data, dims=crds[::-1], coords={c: ds[c] for c in crds})

ds.tg_mean.weighted(dsa.area).mean(['lat', 'lon']) - 273.15

If you find bugs with xESMF, please open issues.

Indeed, I've been talking to Pascal regarding this. We ideally want to fix the issue in xESMF, I just haven't had the time to make the Issue yet.

The xESMF issue has been "fixed" in 0.8. In fact, the fix is simply to use a function from shapely 2.0, so we might be able to close this issue without even pinning xESMF.

Go ahead with what you think is best, you're the more up-to-date person on that issue!


Expose the backoff limit

Depends-On: https://github.com/openstack-k8s-operators/openstack-ansibleee-operator/pull/356

recheck

/test dataplane-operator-build-deploy-kuttl

/test dataplane-operator-build-deploy-kuttl

recheck


inverted error message

I unintentionally switched two error messages in the previous fixes.
Sorry !

Thanks for the fix!


API Review

This is a catch all issue for the API review. Issues related to an API review should be linked here.
The new API might have breaking changes. It will also use async/await, which is kind of part of the Awesome Endeavour: Async Iterators.
Already opened issue should get the label api-review attached.
There's also a sibling issue for IPLD Formats.

There's also API related issues in the interface-ipfs-api repo:

https://github.com/ipfs/interface-ipfs-core/issues/125
https://github.com/ipfs/interface-ipfs-core/issues/81
https://github.com/ipfs/interface-ipfs-core/issues/137
https://github.com/ipfs/interface-ipfs-core/issues/121


Fix Joystream Link

The joystream link points to apple-store instead of https://github.com/Joystream/substrate-runtime-joystream
I tried to open a PR, but have spent like 10 minutes and don't want to spend more time :P

nice catch! haha ok i will correct


Custom/No Programming Language option when creating a new programming exercise

Is your feature request related to a problem?
When creating a new programming exercise, I am met with possible programming languages that also come with their own template repository. For a programming exercise with Node.js, I would need to choose Python and then remove everything from the repository for exercise, test and solution. After that, the students still see Python as the language of the exercise which is not correct.
Describe the solution you'd like
I would like having an option of no programming language which would create a totally empty repository. Similarly, as a next improvement, it would be nice to input a string for a custom language field that also creates an empty repository.
Describe alternatives you've considered
Creating a Node.js language option. However, this would not solve the problem for another language that someone else might want in the future.

The option to select ‘Other’ as programming language exists now.
JavaScript and Typescript exercise templates exist now, too.


Update toc.yml

Added new TOC entry to correspondent with new article:  Export AMR API permission guide.
commit #478aebc
Category

[X ] Content fix (TOC update)

Related issues
references new article in this commit

commit #478aebc

What's in this Pull Request?
update to add new TOC item -- lines 2095-96. for AMR API permissions guide.

closing & opening PR to trigger PR checks again...


Append parallel trees trained separately to existing model

I wonder if there is a mechanism to "append" several trees in a parallel/bagging manner to an existing model:

The starting point will be a boosted bagging (num_parallel_trees = k > 0) model
New k trees will be boosted in parallel and individually on k (fixed) data subsets based on the prediction of the starting model following this: https://xgboost.readthedocs.io/en/latest/python/examples/boost_from_prediction.html
New k trees shall be appended to the existing model, but as parallel trees in 1 round

I currently did this by manually modifying the model json file, but the results does not appear to be correct. So I wonder if it is possible to achieve this, and how it may be done.

Unfortunately no, the gradient booster is a model as a whole and we don't expose the underlying base model(tree) to user interface. Could you please share the motivation of doing so? Why do you want to train trees with fixed dataset instead of sampled dataset.

I see, thanks for the info!
I am trying to simulate a federated learning scenario where client will only share the boosted tree (instead of feature stats/gradients). So each client will boost their own tree based on local data - "fixed subset". And bagging seems to be the most direct way of doing "server model aggregation" under this setting.

share the boosted tree

The tree splits on actual feature values. Wouldn't it be a major leak of data?

Have fun with the hack, let us know if there's anything we can help.


PokePointer and TouchableVolume fix for packed scenes and nested set-ups

Overview
As described in the issue, when touching a volume, a long distance raycast through the entire scene was used for the poke pointer. This long distance raycast was often hitting other colliders in more packed scenes or certain set-ups.
This change introduces proper penetration distance calculation for touched NearInteractionTouchableVolume. This in turn allows for more narrow raycasting, raising the chance of hitting the correct object in the scene.
With the correct distance to the surface of a touched volume, we can also now properly sort touched volumes, resolving intersecting and nested touchable volumes.
Video of the fix in action:
https://user-images.githubusercontent.com/4631548/162210527-acb0a12f-5ec9-4d49-b7ff-d1b3047f1e0e.mp4
Changes

Fixes: #10530

Verification

This optional section is a place where you can detail the specific type of verification
you want from reviewers. For example, if you want reviewers to checkout the PR locally
and validate the functionality of specific scenarios, provide instructions
on the specific scenarios and what you want verified.
If there are specific areas of concern or question feel free to highlight them here so
that reviewers can watch out for those issues.
As a reviewer, it is possible to check out this change locally by using the following
commands (substituting {PR_ID} with the ID of this pull request):
git fetch origin pull/{PR_ID}/head:name_of_local_branch
git checkout name_of_local_branch

/azp run

@RogPodge I do have some spare time to revisit this, so hit me up with suggestions.

Thanks! Can you address the first PR comment I left? I'll add more as I get chances to dig through the PR.

Also a heads up, can you retarget this to https://github.com/microsoft/MixedRealityToolkit-Unity/tree/prerelease/2.8.0, since we're going to be cutting a release soon and would really like to get this in!

Also a heads up, can you retarget this to https://github.com/microsoft/MixedRealityToolkit-Unity/tree/prerelease/2.8.0, since we're going to be cutting a release soon and would really like to get this in!

You mean, retargeting this PR to merge into the 2.8.0 prelease branch, right?

Also a heads up, can you retarget this to https://github.com/microsoft/MixedRealityToolkit-Unity/tree/prerelease/2.8.0, since we're going to be cutting a release soon and would really like to get this in!

Done! Surprisingly no merge conflicts and tests are still fine on my local system.

/azp run


Textarea bug when a line is preceded by new line

I've create a very simple textarea without any special stuff:
VisTextArea textArea = new VisTextArea("");
getContentTable().add(textArea).grow();

But I've a strange bug when I start typing after a new line. As you can see from the GIF, the cursor does not arrives at the end of the line, this is not just a visual bug, but causes also many crashes of that kind during selection or clicks in the textarea.
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: Index 28 out of bounds for length 28
	at com.kotcrab.vis.ui.widget.VisTextField.calculateOffsets(VisTextField.java:259)
	at com.kotcrab.vis.ui.widget.VisTextArea.calculateOffsets(VisTextArea.java:283)
	at com.kotcrab.vis.ui.widget.VisTextField.draw(VisTextField.java:347)
	at com.badlogic.gdx.scenes.scene2d.Group.drawChildren(Group.java:124)
	at com.badlogic.gdx.scenes.scene2d.Group.draw(Group.java:58)
	at com.badlogic.gdx.scenes.scene2d.ui.WidgetGroup.draw(WidgetGroup.java:170)
	at com.badlogic.gdx.scenes.scene2d.ui.Table.draw(Table.java:120)
	at com.badlogic.gdx.scenes.scene2d.Group.drawChildren(Group.java:111)
	at com.badlogic.gdx.scenes.scene2d.ui.Table.draw(Table.java:111)
	at com.badlogic.gdx.scenes.scene2d.ui.Window.draw(Window.java:252)
	at com.kotcrab.vis.ui.widget.VisWindow.draw(VisWindow.java:239)
	at com.badlogic.gdx.scenes.scene2d.Group.drawChildren(Group.java:111)
	at com.badlogic.gdx.scenes.scene2d.Group.draw(Group.java:58)
	at com.badlogic.gdx.scenes.scene2d.Stage.draw(Stage.java:129)

Or this
Caused by: java.lang.IndexOutOfBoundsException: index can't be >= size: 116 >= 114
	at com.badlogic.gdx.utils.FloatArray.get(FloatArray.java:131)
	at com.kotcrab.vis.ui.widget.VisTextArea.moveCursorLine(VisTextArea.java:164)
	at com.kotcrab.vis.ui.widget.VisTextArea$TextAreaListener.keyDown(VisTextArea.java:437)
	at com.badlogic.gdx.scenes.scene2d.InputListener.handle(InputListener.java:53)
	at com.badlogic.gdx.scenes.scene2d.Actor.notify(Actor.java:188)
	... 10 more

I already use a mono spaced font generated from a TTF. Could you help me to identify the cause of that bug? Unfortunately, in test window I connot reproduce it.

I'd recommend first trying it with libgdx's TextArea, maybe some fix wasn't ported.

Yes, I've tried with Scene2D's TextArea and same style I use for VisTextArea and this issue is not present, it works fine

So fixing this requires syncing sources of VisTextField and VisTextArea with libgdx.

I've look at commits in libGDX and I've found that with this two the issue is fixed: https://github.com/libgdx/libgdx/commit/c2d2313717e90081f79eff7a982f601723be3777#diff-81602c9f8b4b94e2ec22c65427fbd35b and https://github.com/libgdx/libgdx/commit/b0d8a2be9a804201366c9bf4b620dba703360639#diff-81602c9f8b4b94e2ec22c65427fbd35b
There are also a lot of other commits that might be good (or not) for VisUI too, however I don't know if they may affects others VisTextField functionalities, so please let me know if you prefer a PR with that two changes or wait for a yours fix, thanks.

Looks like you already figured it out. I'd prefer a PR with those two fixes.


Update OVS tests to support starting rpc_workers and fix ovs_lib tests

Updated ovs tests to start the RPC listener.
Also fixed existing broken ovs_lib tests that were failing.
Partial-bug: DE1977, DE1240
Not-in-upstream: True

:+1:

Looks like the tests all pass:
./run_tests.sh neutron.tests.unit.openvswitch.test_ovs_lib
Ran 44 tests in 12.853s
OK
./run_tests.sh neutron.tests.unit.openvswitch.test_ovs_security_group
Ran 150 tests in 19.309s
OK (skipped=8)
The 8 skipped tests were due to them being IPv6 tests.


Fix the link in Login/Register button

The current link in the Login / Register button does not route to the login-register page. Changing the button tag to a tag destroys the style. Kindly fix it so that the style is not disturbed and the link works properly

gimme it

Good work


zoxide: 0.9.4 -> 0.9.5

Description of changes
Changelog.

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

[x] 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

nix-shell -p nixpkgs-review --run "nixpkgs-review pr https://github.com/NixOS/nixpkgs/pull/341840"
Running nixpkfs-review fails on building a nixpkgs package felix-fm with an unrelated change where the code in felix requires some additional type annotations. Should this be handled first before updating zoxide? The rest of the packages builds sucessfully.

[x] 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.

Result of nixpkgs-review pr 341840 run on x86_64-linux 1

1 package failed to build:

felix-fm

6 packages built:

tmuxPlugins.session-wizard
tmuxPlugins.t-smart-tmux-session-manager
vimPlugins.telescope-zoxide
vimPlugins.zoxide-vim
yazi
zoxide


Fixed #26765 - Removed ETag when Cache-Control: no-store

Removed ETag when Cache-Control contains no-store.
See https://code.djangoproject.com/ticket/26765 for details.

buildbot, test this please

It doesn't look like I was able to get the buildbot to test this, but the tests do pass with sqlite and postgres locally.
I don't think there's a need to document this since no changes should be apparent to the end-user.
I confirmed that reverting the change breaks the test.

👍 from me

The main question I had was how this might interact with https://github.com/django/django/pull/6393. The author of the other PR didn't reply yet.

buildbot, add to whitelist.

Ah sorry, missed that. Since that one already has conflicts to resolve and hasn't been touched in a couple months I don't see any harm in merging this, if you don't hear back soon at least. Your call.
I believe this is still a valid test that should be duplicated in #6393 (?).

@tobiasmcnulty @timgraham I don't see any problem with #6393 but I guess it's better to wait for it to be finished and after this add @ignore_warnings(category=RemovedInDjango20Warning) to test I've added

Looks good to me, pending a decision about #6393.

merged in 20d39325ca1da57a709f3ba38299dc7b0fc4bdfb, thanks!


UTF8 files with BOM produce E999 "unexpected token"

I tried using ruff 0.0.191 (Python 3.9 on Windows 10) to check a file that is encoded in UTF8 with BOM (byte order mark). I used the default options for ruff.
Ruff reported a E999 SyntaxError: Got unexpected token in line 1 column 2 (and then apparently stopped processing further rules except for line length warnings).
When I changed the encoding of the file to UTF8 without BOM, ruff worked as expected and no E999 was reported.
As a workaround I can just change the encoding of the files to make sure no BOM is present, but I think it would be nice if ruff would be able to handle any of the usual source encodings.

Yeah definitely a bug!

I've raised https://github.com/RustPython/RustPython/issues/4353 for this.