not content broken

Hi, I'm using Godot 3.5.1 on Windows with godot clyde 3.2.0 from the asset lib.
In the example script it says the way to check for a finished dialogue is this:
if not content:
However the returned content at the end of the dialog is not null but it is some value that in the print function stringifies to "Null". So currently I use this instead:
if not content is Dictionary:
As I see it, either the example is outdated and should use a different check, or the library is wrong to return something non-null when the dialogue is already finished.

Hello @ConnyOnny . Thanks for reporting this. I've been using Godot 3.5.1 and I tested the plugin in a few other computers and I couldn't reproduce this issue. Does it happen for every dialogue you try to run? I'm trying to check if there is any edge case that might trigger this.
There are a few automated tests that rely on the last line being null (i.e. https://github.com/viniciusgerevini/godot-clyde-dialogue/blob/master/test/test_interpreter.gd#L356), so I'm confident that's the value being returned. However, I don't use windows very often, so I can't discard some peculiarity/bug in Godot.

Bit of a necropost, but I was able to repro this initially when using the sample dialogue provided here https://viniciusgerevini.github.io/clyde/
But when I stripped it down to just the first couple normal text lines it worked properly.  I would assume it's something to with parsing of the options lines or some other bit of oddity there.  Hope that helps.

Thank you all. I wasn't able to reproduce it locally, however the newest version does not return null on dialogue end anymore. Instead it returns a { "type": "end" } object. This should avoid any null handling weirdness.


Starting a container as non-root user

It would be nice to allow starting a container with a non-root user.
On the Docker command line this is normally done with the "--user" argument:
--user 1000:1000

The container class should allow parameter like:
.withUser("1000:1000")
.withUid(1000)
.withGid(1000)
.withCurrentUid()
.withCurrentGid()

The last two are convenience functions to simply use the current user's UID and GID (the user that is running the test).

This would be a really useful addition as we want to be able to delete mapped files created by our test containers at the end of our tests. Unfortunately these files are currently left there with root permissions so we need to delete them manually which is a major pain. If we could configure UID and GID this problem would be solved.

See https://www.testcontainers.org/features/advanced_options/#customizing-the-container. It allows customizing the low level CreateContainerCmd that is passed to Docker, including user/uid/gid.

@michael-schnell did you ever manage to make it work in a simple way ?

No, I didn't test it. We switched to starting/stopping the Docker containers with a Maven plugin in the POM before/after integration tests.

just for the record, something like this achieves what we want (as long as environment variables are correctly set). It may help someone one day.
		String groupID = System.getenv("groupID");
		String userID = System.getenv("userID");

		GenericContainer dockerContainer = new GenericContainer(dataGeneratorImageName);

		dockerContainer.withCreateContainerCmdModifier(cmd -> ((CreateContainerCmd) cmd).withUser(userID + ":" + groupID));

		dockerContainer.withFileSystemBind(inputFolder, "/input", BindMode.READ_ONLY)
				.withFileSystemBind(outputFolder, "/output", BindMode.READ_WRITE)
				.withEnv("SCRIPT_PARAMS", command)
				.withPrivilegedMode(false)
				.waitingFor(new LogMessageWaitStrategy().withTimes(1).withRegEx("bla bla"));


kmsContext is bound to current bearer token at time of instantiation

If the bearer token expires before the ecdh key, then KMS will respond with 401s for each message.Can a kmsContext's bearer token be updated after intantiation or do I need to discard my context and do a new ecdh exchange any time I receive a 401? (given the way we do bulk requests, doing a new exchange would be less than ideal as I could potentially receive hundreds of kmsResponse objects that need to be retried)

The 'clientInfo' (and 'serverInfo') are read-write properties today.  You could just update the credential directly, and subsequent requests would use the new value.
Would that be sufficient?

yep, that should work. thanks


Update security.md - Correct UFW firewall rules

Correct UFW Firewall Rules at bottom of page, both to remove non-functional "from anywhere" and to do both udp and tcp

Closed because this was to the 3g docs rather than the 3h docs as intended.


Allow to easily publish wheels

python setup.py sdist bdist_wheel upload will then publish wheel and sdist to PyPI.

I have merged this change as 2630ed29e447da216a6b0119de6eef2f4212506f, and I also made a small change in f287c30d27d6ca65df3abdfa57265e690396f7ad so the setup.py script will work with both distutils and setuptools. I am going to close this for now, but please let me know if you have any problems with the change and re-open this if you feel it's necessary. Thank you for the pull request.

Thanks for merging and thanks a lot for cronex :-)


[FR] View headings of a file, either from the same file or different files.

Please confirm

[x] I am running the latest version of this plugin

Is your feature request related to a problem? Please describe.
Not related to a problem, but from the following discussion here
Describe the solution you'd like
It would be really nice to have a feature to view and jump to a heading either in the current file Or a different file (other than the current buffer)
For the different file headings scenario, the following suggestion was discussed:

The use-case is basically something like -- after opening find_notes to look for a file, instead of pressing enter on the file that is under selection/under the cursor, one can press some other key that opens the headings of that file that will directly take us to that heading in that file.
This scenario is mentioned in more detail in @lambtho12 reply to my question.
https://github.com/renerocksai/telekasten.nvim/discussions/210#discussioncomment-4833943

Another suggestion is to show headings of all notes in one view and then select and jump to it.
Describe alternatives you've considered

Alternative is just that you open the file using find_notes and once opened, then go to the heading manually using -- ctrl+d / ctrl+u / h / j / k / l. For long notes this will be hectic and tiring.
For the same file heading there is "telescope-headings" plugin, which is a nice one, however, it seems that it adds a bit of extra unnecessary code of other plugins that it supports like org, neorg, beancount, etc., which many may not be using.

Additional context
Add any other context or screenshots about the feature request here.

Note for future me:

Get inspiration from telescope-headings for the heading picker
Make a function find_headings(file) that shows the headings of any file
Make an easy to use function that will default to the current buffer so people can use that mapping (suggest <leader>zh)
Add a Telescope actions (C-h) to the relevant pickers that will call find_headings(selected)
Add picker actions to find_headings(file) so we can yank and insert directly the link to the proper heading

I use tagbar-markdown to get an overview of my Telekasten files (and navigate big md files):
"" Display tags ordered by scope in a window
Plug 'preservim/tagbar'
Plug 'lvht/tagbar-markdown'

You can see all the structure of the file, fold the levels and navigate easily in the tagbar or in the file, open a preview, and even choose to sort in alphabetical order :-)

Seems like a neat plugin! Thanks for pointing it out!

Tagbar is a nice plugin and I've used it before, for a python development project. However, I faced some issues, one of them being like:

If I navigate to a start of a function (line from where function begins) using tagbar and move down in the file (in the same function), then after few seconds, my cursor would automatically jump to the function start line (line from where function starts).
and not just this I was getting some other issues as well. It was getting pretty hectic, since then I stopped using it.
@Rom1deTroyes have you faced such issue ? Not sure if they solved it. Maybe I'll check it out again. Thanks.


Update setup-archive-components.md

Need to escape the asterisks so they actually show in the docs as they are required in the powershell

Learn Build status updates of commit 571d3ab:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

power-platform/guidance/coe/setup-archive-components.md
:white_check_mark:Succeeded

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 learn.microsoft.com contributor guidesPost your question in the Learn support channel

Thanks! I made this change locally in our branch for the November release so closing out here.


Avoid duplicate uploads in archiver

If archives that involve network traffic (e.g. S3) are requested to archive the same video segment multiple times, excess network traffic and API calls are used.
Where practicable, efforts should be made to avoid uploading duplicate video segments.

There is not a simple way to do this for S3 without having to maintain a local cache of the contents of each camera segment directory.
As the event IDs are ideally supposed to be spatially relevant (e.g. events with the ID activity_at_front_of_house may by default include a camera covering a drive and a doorbell camera) having multiple uploads of the same segment should not be a common issue.


Deprecated include of L.Mixin.Events: this property will be removed in future releases, please inherit from L.Evented instead. Error

Deprecated include of L.Mixin.Events: this property will be removed in future releases, please inherit from L.Evented instead. Error

this is the leaflet version problem, use  L.Evented in leaflet version 1.0+.


the smallest buttons!

This patch relies on https://github.com/enactjs/enact/pull/2024 being merged.
This adds the smallest rules for Button and Icon to the existing skins.

We took a different approach.


Unit Test for Project

Core Library
MSAL.js v2 (@azure/msal-browser)
Core Library Version
2.19.1
Wrapper Library
MSAL Angular (@azure/msal-angular)
Wrapper Library Version
2.0.5
Description
I am trying to unit test using jest, but I am getting the below errors.  How to resolve?  Also, if I want to bypass MSALGuards for testing, how can you do this?  A previous question links to a github that doesn't exist anymore.
NullInjectorError: StaticInjectorError(DynamicTestModule)[MenuItemsComponent -> InjectionToken MSAL_GUARD_CONFIG]:
StaticInjectorError(Platform: core)[MenuItemsComponent -> InjectionToken MSAL_GUARD_CONFIG]:
NullInjectorError: No provider for InjectionToken MSAL_GUARD_CONFIG!
MSAL Configuration
No response
Relevant Code Snippets
import { TestBed } from '@angular/core/testing';
import { AppComponent } from './app.component';
import {CUSTOM_ELEMENTS_SCHEMA} from "@angular/core";

describe('AppComponent', () => {
  beforeEach(async () => {
    await TestBed.configureTestingModule({
      schemas: [CUSTOM_ELEMENTS_SCHEMA],
      declarations: [
        AppComponent
      ]
    }).compileComponents();
  });

  it('should create the app', () => {
    const fixture = TestBed.createComponent(AppComponent);
    const app = fixture.componentInstance;
    expect(app).toBeTruthy();
  });

  it(`should have as title 'ClientApp'`, () => {
    const fixture = TestBed.createComponent(AppComponent);
    const app = fixture.componentInstance;
    expect(app.title).toEqual('ClientApp');
  });

  it('should render title', () => {
    const fixture = TestBed.createComponent(AppComponent);
    fixture.detectChanges();
    const compiled = fixture.nativeElement as HTMLElement;
    expect(compiled.querySelector('.content span')?.textContent).toContain('ClientApp app is running!');
  });
});

Identity Provider
Azure AD / MSA
Source
External (Customer)

Hi @gmiked , can you please share the relevant app code as well? Thanks

@bmahall I updated the relevant app code section to include the entire app.component.spec.ts.

@gmiked Thanks for uploading the test file. Can you please make sure that the msal and msal guard modules have been declared in the test file you shared and try re-running it?
You can refer to https://github.com/AzureAD/microsoft-authentication-library-for-js/blob/0a8e67b1159ba4d745b11a57bb598c73e2adbf92/lib/msal-angular/src/msal.broadcast.service.spec.ts#L23 and other test files in the msal-angular code.

Sure I can try that.  I would prefer to mock the services similar to the angular 8 app where possible.  Is it acceptable to mock the services similar to this example for MSAL v2 ?
Angular 8 app


add tools to compare resulting plots

@timtreis I would potentially merge this as is and once points and polygons are working than we can start adding more tests to cover ground.

tests are now failing cause the blobs PR is not in main yet.

Looks good, waiting for aggregate PR to be merged to merge this


Add dashhboard page and move all dashboards there

As designed in https://docs.google.com/presentation/d/1kfzD1jyEAIejWFKoKSTJLCetwvmIriLI2UlFbyB-xbg/edit#slide=id.g529b1e8619_0_1

web-ui-components PR https://github.com/kubevirt/web-ui-components/pull/319/
@cloudbehl @gnehapk

needs rebase

rebased


Cleaning for PR on master

Ready to be merged
Please review

I cleaned the code unused
Unified the description
Unified the indent. like the others FW (OS, AWS, GCP)

@dcaro @rguthriemsft

Ready to be merged
Please review

I cleaned the code unused
Unified the description
Unified the indent. like the others FW (OS, AWS, GCP)
Removed the library unused
Updated the Readme.md

@dcaro @rguthriemsft


Can't select or manipulate elements inside a iframe

Steps to reproduce
Tell us about your environment:
Puppeteer version: 1.4.0
Platform / OS version: Windows 10 Pro
URLs (if applicable): https://www.seguroviagem.srv.br/payment/806440/to3ulroe
Node.js version: 8.11.2
What steps will reproduce the problem?
What steps will reproduce the problem?
const puppeteer = require("puppeteer"),
fs = require("fs");
puppeteer
.launch({
headless: false,
args: ['--disable-web-security']
})
.then(async browser => {
// Configurations
const config = {
nome: "Jonata Carlos dos Santos",
url: "https://www.seguroviagem.srv.br/payment/806440/to3ulroe"
};
// Load the page
const page = await browser.newPage();
await page.goto(config.url), {
waitUntil: "domcontentloaded"
};
page.setViewport({
width: 920,
height: 950
});
// Click on the pay button
const pagar = await page.$(
  "#conteudo > div.ng-scope > div > section:nth-child(6) > section > form > input"
);
await pagar.click();

// Select the frame
const frame = page.mainFrame().childFrames()[1];

// Log the current frame
console.log(frame);

// await for the selector inside the iframe appear and then type

await frame.waitForSelector('#pagarme-modal-box-buyer-name');
await frame.type('#pagarme-modal-box-buyer-name', config.nome);

});
What is the expected result?
I was expecting to insert a text in the selector inside the frame
obs: That iframe is locate in a different address so i guess that can be something related with Cross-Origin Resource Sharing - CORS
What happens instead?
i get an error instead: UnhandledPromiseRejectionWarning: Error: waitForFunction failed: frame got detached.

@FelipeFernandesLeandro  you shoud not close this issue, but rather leave it opened, as I have met exactly the same issue you had, but it is no related to CORS.

i get an error instead: UnhandledPromiseRejectionWarning: Error: waitForFunction failed: frame got detached.

@FelipeFernandesLeandro This errors comes since the page detached the iframe you're manipulating. Once iframe detaches, its execution context gets destroyed, so no evaluations can be run inside of it.

@FelipeFernandesLeandro you shoud not close this issue, but rather leave it opened, as I have met exactly the same issue you had, but it is no related to CORS.

@xprudhomme if you get the same error, than this is what it is: the iframe got detached, so you can't operate with it's internals any more.

I had an issue with finding stripe elements.
The reason for that is the following:

You can't access an  with different origin using JavaScript, it would be a huge security flaw if you could do it. For the same-origin policy browsers block scripts trying to access a frame with a different origin. See more detailed answer here

Therefore when I tried to use puppeteer's methods: Page.frames() and Page.mainFrame(). ElementHandle.contentFrame() I did not return any iframe to me. The problem is that it was happening silently and I couldn't figure out why it couldn't find anything.
Adding these arguments to launch options solved the issue:
'--disable-web-security', '--disable-features=IsolateOrigins,site-per-process'

It cannot resolve my problem, still .I want to know why and how it happend, is another way to avoid it happening?


[0] Error while fetching list of available targets

    **Version:**
    1.3.1

    **OS:**
    <!-- Add your hosting platform e.g Microsoft Windows, WSL, MacOs or Linux -->

    **Stack :**
    No available targets to attach.

    **Additional context:**
    <!-- Add any other context or about the problem or screenshots here. -->

Hi, thank you for reporting this issue. Can you please fill out the description above with more information about how to reproduce the issue you are seeing?


WIP: add an 'offline' option to embed the spec in the rendered page

Hello,
First of all thanks for the work you've put in this project, it has been really useful to me.
However, I desperately needed to be able to generate a completely static HTML page that doesn't load the spec externally, and:

Redoc understandably doesn't support loading file:// URLs
starting a local http server would be waaay overkill
redoc-cli can do it, but requires a whole load of various Javascript libraries & isn't integrated into Sphinx

See: Rebilly/ReDoc#149
I figured embedding the JSON spec in a <script> element would be the way to go. I haven't altered the default behavior, just added an offline parameter.
So what is does is, if the offline parameter is set to True, parse the JSON or YAML spec and embed it in the rendered page. Doesn't work for remote URLs, but I could fix that if you're interested.
Could you please consider merging ? Not yet though, I would like to update the docs first, if you're okay with it.
Thanks a lot !

@etene I'm sorry that I was gone soooo long. Your PR makes sense to me, and I'd like accept this. However, I have a bunch of suggestion of what to improve. Would you like to work on them, so we can land it asap? or you would prefer if I simply continue your work?

Hey, thanks for answering, no problem about the delay !
I agree there's room for improvement, so I'd be glad to hear about your
suggestions and implement them.

I think this should be about it, tell me if you have any more comments !

@etene Looks good! Thank you very much!


[BUG] Upgrading Over Minor Fix (0.10.85 -> 0.10.91) Broke Deployment

Hi @tlienart ,
Yesterday, I was attempting a deployment of our JuliaHealth website and for some reason, the deployment failed arbitrarily. I wrote up details in this issue here: https://github.com/JuliaHealth/juliahealth.github.io/issues/114
I was able to track down the issue with some tips from @DilumAluthge and determined the issue was from us upgrading the Franklin version we used in deployment. We migrated recently from 0.10.85 -> 0.10.91 and somewhere, the breakage occurred. I pinned the last version that worked (0.10.85) and sure enough, no deployment issues.
The issue I linked had even more information, but from the GH Action logs, here is what I see as an error in the deployment:
ERROR: LoadError: LoadError: syntax: invalid keyword argument syntax "shortcut"
Stacktrace:
 [1] top-level scope at /home/runner/.julia/packages/Franklin/xhxut/src/eval/codeblock.jl:84
 [2] include(::Module, ::String) at ./Base.jl:377
 [3] include(::String) at /home/runner/.julia/packages/Franklin/xhxut/src/Franklin.jl:1
 [4] top-level scope at /home/runner/.julia/packages/Franklin/xhxut/src/Franklin.jl:195
 [5] include(::Module, ::String) at ./Base.jl:377
 [6] top-level scope at none:2
 [7] eval at ./boot.jl:[331](https://github.com/JuliaHealth/juliahealth.github.io/actions/runs/5941161601/job/16111528130#step:10:332) [inlined]
 [8] eval(::Expr) at ./client.jl:449
 [9] top-level scope at ./none:3
in expression starting at /home/runner/.julia/packages/Franklin/xhxut/src/eval/codeblock.jl:84
in expression starting at /home/runner/.julia/packages/Franklin/xhxut/src/Franklin.jl:195
ERROR: Failed to precompile Franklin [713c75ef-9fc9-4b05-94a9-213340da978e] to /home/runner/.julia/compiled/v1.4/Franklin/04j2g_aDJTS.ji.
Stacktrace:
 [1] error(::String) at ./error.jl:33
 [2] compilecache(::Base.PkgId, ::String) at ./loading.jl:1272
 [3] _require(::Base.PkgId) at ./loading.jl:1029
 [4] require(::Base.PkgId) at ./loading.jl:927
 [5] require(::Module, ::Symbol) at ./loading.jl:922
Error: Process completed with exit code 1.
##[debug]Finishing: Build site with Franklin.jl

Any thoughts about what might be going awry here @tlienart ?

Is there a reason why you guys are using a 1.4 Julia install in that deploy script? if you change that to 1, things will work.
there's definitely a non-ideal change on Franklin's side to require a (I think 1.6) newer syntax where keywords are implicit (I think the issue here is something like abc = "hello"; foo(; abc) instead of foo(; abc=abc) but I'm not certain).
What I am certain of is that if you upgrade your deploy script, the deployment will work because I just did it here: https://github.com/tlienart/juliahealth.github.io/actions/runs/5956300474/job/16156764977 with this deploy script: https://github.com/tlienart/juliahealth.github.io/blob/dev/.github/workflows/deploy.yml (and removing the cap on Franklin)

Yup! That fixed the problem @tlienart ! Thanks for the suggestion!


Do not consider underscore as punctuation

This might not be fully correct in theory, but should be fine in
practice. Punctuation is commonly used to determine word boundaries and
many 'words' in this project will contain underscores as snake_case is
used.
fix #46380 #46456

Cheeseness is definitely in favor of this change according to this interview :wink:

Did this behavior change between 3.2 and master? Ideally

Yes, master is using ICU to determine punctuation characters, which in turn is using Unicode character database to check if the character belongs to one of the P* categories.

Ideally, we'd have a "non-word characters" editor setting like Atom has.

Should be quite easy to add, if there is demand for such a feature.

An alternative would be to introduce GRAPHEME_IS_UNDERSCORE and handle that in addition to GRAPHEME_IS_PUNCTUATION.
This would allow us to split words according to an editor setting. However, currently multiple editor features use the same function to split by word boundaries. We need to distinguish between editing commands (cursors movement, delete word, etc.) and other features like documentation lookup as mentioned in #46456. The latter must never use underscores to break words apart in order to operate correctly.

Superseded by #47604.


How to disable roll?

how i can disable roll?

Hi @Lampadina17 ,
If you want to suppress world rotation caused by user operations, set the Sp3dRenderer argument allowUserWorldRotation to false.
Also, if you want to suppress zooming caused by the user's mouse scroll, etc., also set allowUserWorldZoom to false.

i want keep yaw, i want keep pitch and clamp 180 deg and disable roll

Do you want to rotate an object by specifying an axis?
For now, let's set the coordinate system as (roll, pich, yaw) = (x axis, y axis, z axis) = (right direction, upward direction, front direction).
To rotate 180 degrees around the x-axis:
// Rotate the object around the origin (0,0,0)
Sp3dObj obj = UtilSp3dGeometry.cube(200, 200, 200, 1, 1, 1);
obj.rotate(Sp3dV3D(1, 0, 0).nor(), 180 * Sp3dConstantValues.toRadian);

// Rotate object in place
Sp3dObj obj = UtilSp3dGeometry.cube(200, 200, 200, 1, 1, 1);
obj.rotateInPlace(Sp3dV3D(1, 0, 0).nor(), 180 * Sp3dConstantValues.toRadian);

Please note that to try this code you will need to import the following packages:
import 'package:util_simple_3d/util_simple_3d.dart';

im not sure that roting model solve the issue, my model correctly moves as expected but i only need to disable camera roll

That's a very cool video!
In your case, I think it is best to set the allowUserWorldRotation argument of Sp3dRenderer to false, and then rotate the object using Sp3dObj's rotate method.
If the Sp3dRenderer argument allowUserWorldRotation is false, the object will not be rotated by user interaction (such as mouse dragging or swiping).
Therefore, I think the object will be displayed fixed in the orientation you rotated in the program.

Has the problem been resolved?
For now, I would like to close this issue.
If my answer does not have enough functionality, please consider extending Sp3dCameraRotationController.
Thank you!


Potential fix for #88: Read from a config.json file

I wanted a way to separate configuration from code, so this is an
attempt to do so.
Reads configuration from a config.json file in the bitbar plugin
directory, and loads it into the environment of the script so that it's
available to plugins.
This is probably the most lines of Objective-C I've ever written in one
sitting. Feel free to critique.

Wow, I completely forgot I wrote this code... 😅


[OpenAI] Add support for Batch API

https://platform.openai.com/docs/api-reference/batch/create
TBD: We need to wait for AOAI TypeSpec definition to be ready.

Can I handle this issue?
Based on my research, it seems that this issue needs to be added to https://github.com/Azure/azure-rest-api-specs. Specifically, a model for batch/create should be added in the path /specification/cognitiveservices/OpenAI.Inference.
Additionally, can I freely add this issue to https://github.com/Azure/azure-rest-api-specs?

@imsosleepy Thank you for reaching out to us.
Yes. We are welcome to contribute to the repo. You are free to open PRs and issue to the repo.

@mssfang
I've been analyzing the openAI part of the azure-rest-api-specs for the past few days, and it's a bit complicated and difficult for an Azure open source newbie like me to get started....
According to my analysis, the Files API of openAI should be implemented first to complete the batch API spec, but it seems that the Files API part is not written yet.
So I think I need to work on the Files API, but where do I start? Should I create an issue for the Files API in this repository first?
And I was hoping you could give me some advice on how to write it in the api spec. It seems like I have to do a buzzing to add a new API SPEC, but is there a set date for the buzzing? I ask because it seems like the criteria for creating openAI versions is the 1st and 15th of each month.

Hi @imsosleepy I can share the OpenAI TSP owners (@jpalvarezl @trrwilson) for more information about how to contribute to OpenAI TSP.

@mssfang
I think the changes to the files API are included in this PR: https://github.com/Azure/azure-rest-api-specs/pull/29067
The above issue doesn't seem to be necessary.
Using the information about the files API in the specification/ai path, I think I can write a spec for the Batch API.
The sequence for external contributors to contribute, as I understand it, is as follows

Write the batch API as a TypeSpec in the correct path in the https://github.com/Azure/azure-rest-api-specs repository (this would be cognitiveservices/OpenAI.Inference)
Create an issue in the https://github.com/Azure/azure-rest-api-specs repository.
Auto-generate a Dataplane (I haven't gotten this far yet, so I'm not sure) (https://azure.github.io/typespec-azure/docs/next/howtos/DataPlane Generation - DPG/00howtogen)
Create a PR in the azure-rest-api-specs repository with the work done.
When that PR is merged and a draft is created, auto-generate a PR in azure-java-sdk
Once the PR is merged into azure-java-sdk, deploy on both sides.

I think TSP means Traning Service Provider, so I'll ask a question, here's what I'm curious about.

is the order I have written for outsiders to contribute correct?
the code in the OpenAI.Inference/examples path looks like generated code, how was it created?
within the data-plane, there is code generated in the authoring path, why is it separated?

I think I'm having a bit of a problem with what I've understood and what the team is working on, but you're right that I'm in need of some help 😢.

@mssfang Is there any way I can contribute?

Hi, @imsosleepy. Thank you for your contribution. It is valuable to us for sure. While I usually don't work on azure-rest-api-specs side, I recommend opening a new PR directly in the repo in the future. I've also heard that Azure OpenAI service will start supporting Batch soon in this 2024-07-01 version. Some people will be adding Batch REST APIs along with other enhancements or new features in that release.
After that is merged, I will start to use it to generate the Java OpenAI SDK baseline and look into to support the Batch from both of Azure OpenAI and OpenAI services.

@mssfang It seems that TypeSpec definitions have an internal timeline, and it's not yet clear where I can start contributing, but maybe monitoring the progress of this issue will help me contribute to this repository in the future?

@imsosleepy Thank you for the understanding. Yes. I will keep you posted here when there is a new update.

@imsosleepy Batch API is supported in the new version
https://repo1.maven.org/maven2/com/azure/azure-ai-openai/1.0.0-beta.11/


Remove version: libjpeg-turbo.libjpeg-turbo.VC version 2.1.5

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/libjpeg-turbo/files/2.1.5/libjpeg-turbo-2.1.5-vc.exe/download
Status Code: 200
(Automated message - build 897)


WIP Adding SemVer support

This pr will add SemVer support.
Not tested well so far.

@kenjiuno Any update? Will you be working on this any more?

Currently no update from me about this pr.
If there are any points to be fixed, I can work on it.

@Deadpikle do you think that this PR will be accepted?
I'm managing two channels for beta and stable updates but I would like to have the suffix "-beta" on my installer :)
Thanks

@DanieleSky yes but I may modify things a little bit. At a cursory glance there are now two filter systems and there probably should only be one. Sorry this project hadn't been a huge priority lately. Will see what I can do to integrate what is here by early next week. Keep in mind the app cast generator may or may not have compatibility depending on if I have time to add that in too (unless someone else adds a PR for that).

Thank you @Deadpikle
About the generator is not a problem because I'm using the --file-version  option to force the version in appcast.xml.
I'll try to watch into the generator to make the modify and the PR ;)

Merged with some changes to remove the duplicate filtering process and fix a few typos/add some more tests/that sort of thing. My changes aren't necessarily perfect (namely there is a duplicate delegate and IAppCastFilter definition; may change how things work in larger restructuring later on) but it's now available and rolling out to NuGet preview.
Need to get this into the README somewhere for better docs.

And RE: The app cast generator — I'm not sure that any changes here are necessary, actually. :-)

And a final thank you to @kenjiuno for implementing this — really appreciate it! ^_^

Hi @Deadpikle
when do you think to make the new release with this PR?
I would like to test

Hi @Deadpikle when do you think to make the new release with this PR? I would like to test

This is already available in preview build 3.0.0-preview20240120001 and is not planned to be backported due to the breaking changes it contained.


Support for WorkflowIdConflictPolicy

What was changed
Added WorkflowIdConflictPolicy to ExecuteWorkflow and SignalWithStart.
NOTE: I haven't seen any WorkflowIdReusePolicy validation inside of testsuite.WorkflowTestSuite (except for child workflows), so I didn't add any for the WorkflowIdConflictPolicy either.
Checklist
Closes https://github.com/temporalio/sdk-go/issues/1447

@stephanos - can you also make sure this is added for ScheduleWorkflowAction?

@cretz I might be missing something, but when I looked at the schedule code I saw it's not using StartWorkflowOptions anywhere and therefore doesn't have either WorkflowIdReusePolicy or WorkflowIdConflictPolicy fields?

My mistake, this is not needed, I was confusing with languages that reused common sets of start options.


boolean facts are not treated in some places

Hey,
it's a great job you have done so far with puppetboard development and I've just a tiny feature request.
It would be a great idea to graph some Boolean facts (apt_reboot_required, apt_updates) and to have  it sort-able by "Inventory" ('Reboot required', 'apt_reboot_required').
I've tried to add this to settings.py:
`
GRAPH_FACTS = ['architecture',
'clientversion',
'domain',
'lsbcodename',
'lsbdistcodename',
'lsbdistid',
'lsbdistrelease',
'lsbmajdistrelease',
'netmask',
'osfamily',
'puppetversion',
'apt_reboot_required',
'apt_updates',
'processorcount']
INVENTORY_FACTS = [('Hostname', 'fqdn'),
('IP Address', 'ipaddress'),
('OS', 'lsbdistdescription'),
('Architecture', 'hardwaremodel'),
('Kernel Version', 'kernelrelease'),
('Reboot required', 'apt_reboot_required'),
('Puppet Version', 'puppetversion'), ]
`
The corresponding web page /fact/apt_updates is then empty
and in the /inventory web page the column "Reboot required" isn't sort-able.
Thank you

@ap-16 , Did you solve the problem with apt_reboot_required?

@ap-16, the boolean facts bug was fixed in 484727b62c7891c42fede4ecbdc2cb70aa796519, this issue can be closed :wink:


Animated stickers running at 30FPS don't pass validation

Despite Telegram's requirements for animated stickers stating that  All animations must run at 60 Frames Per Second, 30 FPS animations are still allowed: for instance the pack SteampunkJulia contains such stickers.
Bot's animated sticker validation should be amended to check that the FPS are at most 60.
Also, add a unit test checking that a low FPS animated sticker passes the validation, similarly to the existing noAnimatedStickerConversionNeeded().
In the meanwhile, the bug has been reported to Telegram: https://bugs.telegram.org/c/33112

Hi @rob93c I would like to work on this issue I have raised this PR can you have a look once.
https://github.com/Stickerifier/Stickerify/pull/164

Hi @ritikverma2000 and thank you for working on this issue!
I'll assign it to you and I'll review your changes as soon as possible

Hi @rob93c I had one concern regarding the test case, how would we be getting the FPS inside the test case for an animated sticker?

I think we should approach this test from a different perspective: we know that an animated sticker running at 30FPS would make the conversion fail (prior to your changes) so we can add a sticker with those characteristics to the folder test/resources/.
We now have a file we can use to test if the changes you made are correct, you can create a test to check that the conversion of such file is now successful (in this case, the success means that null is returned as in the previous test).
As to where to find such file to use in the test, you can open the sticker pack SteampunkJulia and download one of them; maybe a good name for that file would be low_fps_animated_sticker.tgs
Feel free to ping me if you have any doubt

Ya, that approach seems correct to me @rob93c . Added the code for the test case pls check this,
PR again --> https://github.com/Stickerifier/Stickerify/pull/164

Hi @rob93c can you pls check as to why this is failing

Thanks @rob93c for guiding me.

It was failing because Github workflows triggered by forks of a repo can't access the main repository's secrets, in this case the ones with the credentials used to access Docker but that's not an issue so I approved and merged the pull request 🔥
I really appreciate your help, thank you for contributing to this project 😄


Schedule capella on mainnet!

https://discord.com/channels/595666850260713488/745077610685661265/1085958951503405116
ref:

https://github.com/ethereum/execution-specs/pull/727

Referencing consensus-spec PR: https://github.com/ethereum/consensus-specs/pull/3300

:tada: This PR is included in v1.7.0 :tada:


Update Swift Version Compatibility

Our compatibility matrix was out of date, and the podspec didn't specify any
swift versions, which was causing a massive headache when trying to publish
new versions.

Nevermind. I'm just going to require 5.0 right now anyway, so I'll do this in another branch.


[FEATURE] Show shipping methods in checkout

Display proper shipping options

Acceptance criteria

[ ] use getShippingMethods from useCheckout in shipping step to show available methods
[ ] use setShippingMethod and shippingMethod from UseSessionContext

it's already done


add color interp

Codecov Report

Merging #75 into master will decrease coverage by 0.14%.
The diff coverage is 88.37%.

@@            Coverage Diff             @@
##           master      #75      +/-   ##
==========================================
- Coverage   91.91%   91.76%   -0.15%
==========================================
  Files          13       13
  Lines        1089     1130      +41
==========================================
+ Hits         1001     1037      +36
- Misses         88       93       +5

Impacted Files
Coverage Δ

aiocogeo/cog.py
93.84% <76.19%> (-1.58%)
:arrow_down:

aiocogeo/constants.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Fix golint warnings

Signed-off-by: Gábor Lipták<EMAIL_ADDRESS>What this PR does / why we need it: Fix golint warnings

@gliptak: Adding do-not-merge/release-note-label-needed because the release note process has not been followed.

One of the following labels is required "release-note", "release-note-action-required", or "release-note-none".
Please see: https://github.com/kubernetes/community/blob/master/contributors/devel/pull-requests.md#write-release-notes-if-needed.
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.

Hi @gliptak. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

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. I understand the commands that are listed here.

/ok-to-test

@gliptak: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/auto-merge_master.sh
f6e3c088bc6c41a21c83668ad9b05a33b00bdd38
link
/test auto-merge_master

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. I understand the commands that are listed here.

/release-note-none


Bug-fix: Add Group creation wait to teams team add command

I've also refactored the code a bit to align it more with our more recent standards. Following changes have been applied:

optionsets instead of validators
make use of async awaits instead of promise then
replaced in line interface by ms-graph types
added some verbose logging

Closes #2021

Thank you for the enhancements @MathijsVerbeeck! We'll try to review it ASAP!

Manually merged


Filters: Improve OrderFilter's capability to have multiple "order by" options

Description
The current  is built to only handle a singular sort by field:

However, in https://github.com/opencollective/opencollective/issues/7356 we're going to introducing the ability to sort by the "effective date" in addition to the default "created at" sorting in the Transactions table.
And in https://github.com/opencollective/opencollective/issues/7342 we'll end up with multiple sort options like balance, alphabetical and approval date in the Hosted Collectives page.
It would be good if the filter handled this in a neat way other than simply listing all permutations after one another, perhaps through grouping or selecting sort by field and sort order independently.

The ability to handle multiple sort options was added recently in the chart of accounts view, needs improvement to the UX though.
https://github.com/opencollective/opencollective-frontend/blob/main/components%2Fdashboard%2Fsections%2Faccounting%2Findex.tsx#L91


Merge Symmetry_updates to master

Merge Symmetry_updates branch to master.

@sunghjung3 Can you elaborate on what effect this change will have?

@JasonGibsonUfl This is a follow-up pull request from #77 . It is just a more concise implementation of generate_triplets(), which should not affect any results.

When tested on the tungsten dataset, this version of generate_triplets() is 1.27 times faster than the current master's version (49m 37.7s vs 63m 4.4s).
And I verified that the 2 dataframes are identical, up to numerical precision (compared using np.allclose(df_features_1, df_features_2, rtol=1e-5, atol=1e-8))


Fixed freeze on search previous

I've just copied a missing condition from search_next() into search_prev().
Searching a previous result was always using the logic for starting on a result from before, and it could cause an infinite loop in some cases.
Here's the correct logic from search_next():
https://github.com/godotengine/godot/blob/7863ea39dbf3c4ac4395f873de28736a64d75f66/editor/code_editor.cpp#L405-L413
Fixes #31328

Thanks!


FormatTime string_view not found when importing vegafusion (this used to work)

Entirely possible this is not a vegafusion issue, and I don't know what library upgrade caused it, for which I apologize profusely. Any pointer would be appreciated.
Traceback (most recent call last):
  File "./nvperf.py", line 15, in <module>
    import vegafusion as vf
  File "/Users/jowens/Library/Python/3.12/lib/python/site-packages/vegafusion/__init__.py", line 1, in <module>
    from .runtime import runtime
  File "/Users/jowens/Library/Python/3.12/lib/python/site-packages/vegafusion/runtime.py", line 3, in <module>
    import pyarrow as pa
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages/pyarrow/__init__.py", line 65, in <module>
    import pyarrow.lib as _lib
ImportError: dlopen(/opt/local/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages/pyarrow/lib.cpython-312-darwin.so, 0x0002): Symbol not found: __ZN4absl12lts_2024011610FormatTimeENS0_11string_viewENS0_4TimeENS0_8TimeZoneE
  Referenced from: <CD063FAB-C5F0-3295-AA51-ECE6EE9CDBA1> /opt/local/lib/libarrow.1300.0.0.dylib
  Expected in:     <42921224-0959-3E0D-BA2B-099ADB608AA7> /opt/local/lib/libabsl_time.2401.0.0.dylib

(MacOS, mix of MacPorts and pip.)
$ pip show vegafusion
Name: vegafusion
Version: 1.6.7
Summary: Core tools for using VegaFusion from Python
Home-page:
Author:
Author-email:
License: BSD-3-Clause
Location: /Users/jowens/Library/Python/3.12/lib/python/site-packages
Requires: altair, pandas, protobuf, psutil, pyarrow
Required-by:
$ python --version
Python 3.12.3
$ port installed | grep arrow
  apache-arrow @13.0.0_9 (active)
  py312-arrow @1.3.0_1 (active)
  py312-pyarrow @13.0.0_9 (active)

Hi @jowens, I think there is some version mismatch happening between different arrow shared libraries. I'm not familiar with installing pyarrow through MacPorts, but the version 1.3.0 of py312-arrow not matching version 13.0.0 of py312-pyarrow might be an issue.
If it's a possibility for you, I'd recommend uninstalling all of the (py)arrow packages from MacPorts and then install just the pyarrow package using pip, as this bundles the Python and native arrow dependencies in one package.

A rebuild of abseil and all its dependencies solved this problem (https://trac.macports.org/ticket/69932). Thanks @jonmmease!


Add utils::locale_to_utf8()

This is a little detour from #1344, but kind of productive as well. Added some tests, plus a little helper function to save on typing in a few places.
Reviews are welcome! If no issues are raised, I'll merge in three days.

Coverage increased (+0.09%) to 58.478% when pulling 7f6dd5890afc484d9f17e7d29d680b2b689df7fd on feature/locale_to_utf8 into 64c2475e5ba7861243c0992d1e219fd6da10a1c2 on master.

Looks good, thanks for the changes

Looks good, thanks for the changes

Thank you for the review!

Thank you for the review!


1239 Logout everywhere

fixes #1239

Would really appreciate a review of this.  Thank you!

Could someone take a look at this PR for a logout everywhere feature?  it was originally requested in #1239 - I have a project that I am basing on this, so I have an interest that my changes get merged upstream.  @sahat @YasharF?   Thank you!


PPO2 Runner dones problem

https://github.com/openai/baselines/blob/9ee399f5b20cd70ac0a871927a6cf043b478193f/baselines/ppo2/runner.py#L26
    for _ in range(self.nsteps):
        # Given observations, get action value and neglopacs
        # We already have self.obs because Runner superclass run self.obs[:] = env.reset() on init
        actions, values, self.states, neglogpacs = self.model.step(self.obs, S=self.states, M=self.dones)
        mb_obs.append(self.obs.copy())
        mb_actions.append(actions)
        mb_values.append(values)
        mb_neglogpacs.append(neglogpacs)
        mb_dones.append(self.dones)

        # Take actions in env and look the results
        # Infos contains a ton of useful informations
        self.obs[:], rewards, self.dones, infos = self.env.step(actions)
        for info in infos:
            maybeepinfo = info.get('episode')
            if maybeepinfo: epinfos.append(maybeepinfo)
        mb_rewards.append(rewards)

Why  mb_dones.append(self.dones) is placed there, and not below self.env.step?
dones is the return value of self.env.step, not self.model.step.
As it is now, each dones is appened to the next step, and the last dones is lost.

Ok, the last dones isn't lost, it will be appended to mb_dones in the next run call.
The runner of A2C is bit different:
https://github.com/openai/baselines/blob/9ee399f5b20cd70ac0a871927a6cf043b478193f/baselines/a2c/runner.py#L46
The last dones is appended to mb_dones outside the loop:
mb_dones.append(self.dones)
Then mb_masks is mb_dones without the last element:
mb_masks = mb_dones[:, :-1]
Then the first element is removed from mb_dones:
mb_dones = mb_dones[:, 1:]
That would be great, if somebody could explain a bit about these masks and dones, in general.


iOS inline player

Hi, my video plays on android inline but on ios play button occurs and video just playing fullscreen but i want it to play with content.

https://github.com/clappr/clappr/issues/563

I think this is just not possible on iOS on mobile (unless they save your site to their home screen).
http://stackoverflow.com/a/21306314/1048589

What about on iOS 10 with webkit-playsinline ?
https://developer.apple.com/library/prerelease/content/releasenotes/General/WhatsNewInSafari/Articles/Safari_10_0.html

What's the best way to get clappr to play inline?

Take a look at the playback playInline parameter.

I tried that but it's still switching to the native player :( - maybe I'm doing something wrong.

Check the this example. I double-checked, it adds the playinline attribute on video element, and therefore "should" play inside browser.

@kslimani that example too does not work in ios. @rgaufman I think this is a clappr bug i switched to hls.js and added playsinline attribute to the video element everything works without any problem. I think you should consider just changing the player

@NikosEfthias what is your device and which iOS version do you have ?

Check this: https://developers.google.com/web/updates/2017/09/autoplay-policy-changes

@alparslanahmed i think the issue beeing discuted is not about autoplay, but playing video "inline"
(ie: the video plays in the browser without opening quicktime player in fullscreen).


chore: Use lurk-lab fork of bls12_381

Closes #21
The bls12_381 fork is now hosted here, with the same commits to the main branch.

FYI for clarity: I forked the repo directly and pushed the 3 commits on top, so there are no CI changes there yet from our template repo


Fix CI

Fix broken sentry

@Saela After nearly descending into the eternal hellfire, I found that github didn't add my environment variable and passed it as empty, so our docker build failed.
1 hour well spent :)


Skeleton to Rest-API

A issue to link the rest of the issues that has this:
#39  - Rest Controller for categories
#38  - Rest Controller for products
Not currently having a issue:

RestController for users

Proper documentation also:
https://www.baeldung.com/spring-rest-openapi-documentation
(if possible, want to change the controller titles in the swagger documentation to the name of the repository it calls)


onDidChangeActiveTextEditor fires multiple time

Hello,
I have a Problem with the onDidChangeActiveTextEditor event at the moment.
I have created an extension which opens a web view in a viewcolumn beside a code file to show a preview (looks like the .md preview).
example:
tab file A  tab file B | tab webView (always visible)

When I first focus tab B then mouseclick inside the webView and after that click on tab A
-> the onDidChangeActiveTextEditor is fired multiple times in random order ex. first for tab A and then for tab B
When I first focus tab B then click the TAB of the webView and after that click on tab A
-> the onDidChangeActiveTextEditor is called correctly once for tab A.

Maybe there is somekind of alternative for onDidChangeActiveTextEditor to get informed when a tab is focused?
Or is it possible to prevent the webView from getting the focus in any case?

is fired multiple times in random order ex. first for tab A and then for tab B

There is no Math.random in this... Can you please note down the sequence of events that you are seeing and the sequence you are expecting.

first for tab A and then for tab B

@bpasero to confirm but this is likely because the click first activates the group of editors and only then reaches the tab to reveal the other editor

Tracing the editor events I see the following things happening:

first: group gets active making the current editor active (B)
second: editor corresponding to the clicked tab gets active (A)

I would not expect the events in the extension to be AB, rather BA if any... I did not check on that.


fixed e2e test re: routeTables

Which issue this PR addresses:
Fixes #985 by adding the route tables to expected resources
What this PR does / why we need it:
It fixes the e2e tests to properly include routeTables in expected resources for the comparison.
Test plan for issue:
Run e2e test.
Is there any documentation that needs to be updated for this PR?
None. Fixing a test. Inline information is enough to understand the change.

/azp run e2e

I guess we're nearly at a point we can merge this @Makdaam?  Please can you get it into shape?

/azp run e2e

Don't know why this fell off the radar.  Rebased, fixed up, pushed.  Let's see if it passes E2E.

/azp run E2E


Replace doseq with reduce

Makes encoding about 15% faster.

This looks like a significant speedup! Great work!
doseq is in general used for side effects (it returns nil), reduce for producing values, run!is a variant ofreducethat is used for side effects. So I was wondering if you could get away with justrun!` here.

Yes run could be used. The accumulator is guaranteed to stay in a register so I like to sneak something (JsonGenerator) in there if it is otherwise unused, but if everything gets inlined (likely, but not guaranteed) it does not matter.


Finding scale space extrema

Hi,
I'm curious about the lines 333-334 and 338-339 in AKAZE.cpp. The addition + .5*(ratio-1.0) to the accepted points are compared with incoming points that do not have that extra term added to them. Is this intentional? I'm curious since I'm working on a GPU version (with Mårten Björkman who has previously posted here, and Alessandro Pieropan). The idea is that the GPU version should produce the same results as the CPU version, which imposes constraints on the extrema detection and filtering which are hard to do in parallel. In particular the lines mentioned above are giving me trouble when performing the filtering in parallel.
The GPU version can of course be made more efficient if a different result compared to the CPU version can be accepted, but I figured it might be worth having an identical version as well.
Niklas

Hi Niklas,
That seems to be a bug in the code although I suspect it will not change much the performance. The distance computation in line 301 should also consider the factor .5*(ratio-1.0).
I would say that if your GPU version produces almost similar performance as the original version with different constraints in the extrema detection that will be good.
Regards

Hi Pablo,
Thanks for the quick reply! Even if not considering the issue, the detected keypoint set will be very similar to the CPU code. I agree that that is enough, and that even getting similar performance is enough. We might provide two versions where one is more aimed at performance and one at getting the same keypoints as the CPU version. I will let you know when the code is ready. We get quite the speedup compared to the CPU version, so I hope people will find it useful.
Niklas


fix(interactive): Support Optional Match in Graph Optimizer

What do these changes do?
as titled.

Related issue number

Fixes

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

Project coverage is 26.17%. Comparing base (0b8e42f) to head (35d0e56).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3687   +/-   ##
=======================================
  Coverage   26.17%   26.17%
=======================================
  Files         188      188
  Lines       17252    17252
=======================================
  Hits         4516     4516
  Misses      12736    12736

Continue to review full report in Codecov by Sentry.

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


#4566: Adds 'Notes' field to Donate form, passes as metadata

What's this jam do?
This jam adds a simple "Notes" field to the Donate form. It passes the field back to Drupal, which adds a metadata array to the Stripe\Charge::create call.
How can I help Matt?

The updated form needs a visual review.
We need to decide whether this is a text field (current) or text area (do people need to write an essay?).

Where can I read more?
The [Trello card](Trello card), or #4566.
How would I test this?
Clearly, test against a Stripe dev account in test mode. Use the test codes for great success.
You'll see payments with Notes come through like this:

You'll see payments without Notes come through like this:

Pulled down to my local and checked it out –– looking good to me!

Or when switched to a textarea:

Looks good to me. Also, there's a cheat to test the Donate page: https://github.com/DoSomething/dosomething/wiki/DoSomething-Payment#testing

just shot this over to Aria/Kenia to see if they have a text area/field preference. hoping to get this shipped to prod today

chatted with Aria, let's go with a text field. and can we update the field label to read "Message (optional)"?

@sbsmith86 any chance you could make this update real quick?

@mikefantini sure!

Can we merge this and I can make the change on a separate branch? I don't have permissions to make changes on this branch.
@mshmsh5000 @mikefantini @aaronschachter @DFurnes

go for it! :+1:


[AMQ-8472] Fix reload4j scope in activemq-partition module

@cshannon here's a proposed fix

Looking at my build the slf4j stuff is the main culprit of the error:
Caused by: java.lang.IllegalStateException: Detected both log4j-over-slf4j.jar AND bound slf4j-reload4j.jar on the class path

Yes I was only talking about reload4j itself, in relation to log4j, that it wasnt a change in behaviour to pass reload4j on as log4j was already being passed on via the 'linkedin'/zookeeper bits before.
As the new JIRA didnt mention it, I didnt realise the point actually centred more around the slf4j bit. I'd agree that as the slf4j-log4j12 equivalent wasnt being passed on before the reload4j equivalent shouldnt be now.

Yes I was only talking about reload4j itself, in relation to log4j, that it wasnt a change in behaviour to pass reload4j on as log4j was already being passed on via the 'linkedin'/zookeeper bits before.
As the new JIRA didnt mention it, I didnt realise the point actually centred more around the slf4j bit. I'd agree that as the slf4j-log4j12 equivalent wasnt being passed on before the slf4j-reload4j equivalent shouldnt be now.

Ok so maybe we just drop the slf4j part and keep reload4j and let people exclude if they want.

@cshannon yes, it makes sense to me. Let me update the PR.

@cshannon @gemmellr I've updated the PR to:

remove reload4j-slf4j dependency
still use reload4j-core in compile scope (allowing people to exclude it if needed)
Thoughts ?

@jbonofre - i think this is fine so i think you can go ahead with this and re-cut the release

Seems like it would have been simplest to put it at test scope the way it was before, but if the tests pass then I guess its no biggie, just means anyone investigating them possibly needing to add it. Though I guess theres probably little chance of anyone bothering in this case, so...meh :)

(The PR title and commit message are wrong though, since the scope of reload4j wasnt changed at all)


New version: Alacritty.Alacritty version 0.13.0

Pull request has been created with Komac v1.11.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree


[PE-D][Tester B] deletetag command on UG should specify that it is case-sensitive

Since the find command is case-insensitive, it would be good to state that deletetag command is case-sensitive in the UG such that to delete a tag, the user inputted tag must match the tag to be deleted exactly.

Labels: severity.VeryLow type.DocumentationBug
original: qinboan/ped#2

This is stated in the UG


Using example is a bad representative

While i understand the point of the using declaration, I think the example is awful.
I mentally and immediately just transformed it to the code below:
Which I would also say is more in line with the actual code of the "new" code.
And, in this case, is it really all that useful?...
// We must declare the variable outside of the using block
// so that it is in scope to be returned.
using (var file = new System.IO.StreamWriter("WriteLines2.txt"))
{
    int skippedLines = 0;
    foreach (string line in lines)
    {
        if (!line.Contains("Second"))
        {
            file.WriteLine(line);
        }
        else
        {
            skippedLines++;
        }
    }
    return skippedLines;
} // file is disposed here

IMO not really, that being said, the "having to declare the variable outside the using" has hit me from time to time and annoyed me, so I can surely see the benefit. (I have not fully made up my mind on how "Clear" it makes to code to me though, readability)...
But there must be better examples that this to show of the usefulness.

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

ID: a94537c5-754e-9dea-94a8-7573011f588c
Version Independent ID: aba09118-22c5-2c6a-954a-b9659a9644ae
Content: What's new in C# 8.0 - C# Guide
Content Source: docs/csharp/whats-new/csharp-8.md
Product: dotnet-csharp
Technology: csharp-whats-new
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for adding your feedback @jeme
This article gives a quick overview of the syntax of new features. As we update other tutorials you'll see more samples that show this feature in more real-world scenarios.
What we've found by examining C# projects is that the majority of using declarations are closed at the end of a scope (either a loop, an if block, or a method). In those cases, the new syntax is more concise for the most common cases.
I'll close this as answered.

I would have thought it was also suppose to "sell" the ideas, so I had hoped to see a better example in this case instead of something that seems completely pointless.
But that is ofc. your call to make, I have voiced my opinion on the matter, which was what was important for me to do.

I agree with @Jeme. This doesn't really "sell" using the new syntax, unless there are other benefits. I honestly think this is almost worse as it means developers who declare multiple usings in a single scope with this syntax will have the life time of those resources far extended pass their actual use since they will not get disposed until they are no longer in scope.


Fixed typo in xregexp.js

Thanks, however I don't see much value in fixing typos in vendor packages.


Fix incorrect usage of KMeans clustering

I've realized that KMeans clustering in sklearn does not have the option to input a distance matrix to the fitting function, which is how I've been using it. In other words, we have a [n_sample x n_sample] matrix, rather than a [n_sample x n_feature] matrix. https://scikit-learn.org/stable/modules/generated/sklearn.cluster.KMeans.html#sklearn.cluster.KMeans. To get a [n_sample x n_feature] matrix, it seems we would need to know beforehand what the reference structure (native structure) should be (single feature is rmsd to that reference), and it wouldn't work for identifying multiple secondary structures.
The reason that the KMeans clustering runs at all and gives an ok solution for medoids, is that it interprets the rsmd to each frame as its own feature. This, I think, is not what we want, so I would avoid using it for rmsd-based clustering to find a native state.
These are the algorithms in sklearn that can take a distance matrix as input:

AffinityPropagation
AgglomerativeClustering (4 different types of linkage to consider)
DBSCAN (requires careful tuning of eps, min_sample parameters)
OPTICS (requires careful tuning of min_sample, xi parameters)
SpectralClustering

Density-based clustering has been tricky to get working (the parameters seem very system-dependent). I will look into the others.

@tlfobe Just want you to be aware of this issue, I know you were planning on using KMeans with the cg_pyrosetta structures.

Playing with a bunch of these methods, I found that none are that robust at dealing with the very noisy dataset we have for weakly cooperative transitions (need to finely tune clustering parameters, etc.).
However, if we first filter out data points with few neighbors within a cutoff radius (based on RMSD distances), we can reliably identify high density regions of conformational space (filtering criteria can be specified by percent, so very generally applicable). I got the idea from here: https://link.springer.com/article/10.1007/s10822-013-9689-8, where DBSCAN is used successfully on the filtered data, where only 1% of the highest density data passes through. I will add this in a PR later today.
As for the KMeans issues, we can instead use this formulation of KMedoids: https://scikit-learn-extra.readthedocs.io/en/latest/generated/sklearn_extra.cluster.KMedoids.html
Though it doesn't seem to give decent results on the unfiltered data.


port PRs from webdriverio master

https://github.com/webdriverio/webdriverio/pull/800

became obsolete


Application Exception: Request timed out

This is an issue created automatically by GrammarNazi when an exception was captured.
Date (UTC): 12/10/2024 00:29:21
Exception:
Telegram.Bot.Exceptions.RequestException: Request timed out
---> System.Threading.Tasks.TaskCanceledException: The request was canceled due to the configured HttpClient.Timeout of 100 seconds elapsing.
---> System.TimeoutException: A task was canceled.
---> System.Threading.Tasks.TaskCanceledException: A task was canceled.
at System.Threading.Tasks.TaskCompletionSourceWithCancellation1.WaitWithCancellationAsync(CancellationToken cancellationToken)    at System.Net.Http.HttpConnectionPool.SendWithVersionDetectionAndRetryAsync(HttpRequestMessage request, Boolean async, Boolean doRequestAuth, CancellationToken cancellationToken)    at System.Net.Http.RedirectHandler.SendAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)    at System.Net.Http.HttpClient.<SendAsync>g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts, CancellationTokenSource pendingRequestsCts, CancellationToken originalCancellationToken)    --- End of inner exception stack trace ---    --- End of inner exception stack trace ---    at System.Net.Http.HttpClient.HandleFailure(Exception e, Boolean telemetryStarted, HttpResponseMessage response, CancellationTokenSource cts, CancellationToken cancellationToken, CancellationTokenSource pendingRequestsCts)    at System.Net.Http.HttpClient.<SendAsync>g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts, CancellationTokenSource pendingRequestsCts, CancellationToken originalCancellationToken)    at Telegram.Bot.TelegramBotClient.SendRequest[TResponse](IRequest1 request, CancellationToken cancellationToken)
--- End of inner exception stack trace ---
at Telegram.Bot.TelegramBotClient.SendRequest[TResponse](IRequest`1 request, CancellationToken cancellationToken)
at Telegram.Bot.Polling.DefaultUpdateReceiver.ReceiveAsync(IUpdateHandler updateHandler, CancellationToken cancellationToken)
Exception caught counter: 13.

Telegram API down for a moment


[Powertoys Run] Ability to override Windows Search.

Description of the new feature / enhancement
When Windows Search is opened close it and open Powertoys Run instead. Some advantages of this:

You only need to press one key (WIN) to open Powertoys Run. (Which is must faster than doing a combination of keys.)
You still have access to the start menu.

Scenario when this would be used?
When Windows Search is opened; either via the Start Menu, WIN + S or the search button.
Supporting information
No response

/dup #3344


Require EULA/TOS acceptance before download starts.

Description of the new feature/enhancement
Some packages require a EULA to be accepted before commencing the download of the product. But I have noticed no such functionality is present. There should be an option to provide a EULA/TOS source url or file that will be presented before the download of the file begins that will be accepted by the end user.
Proposed technical implementation details (optional)
My thoughts would be that we can have something like a "EULAUrl" or "TermsURL" that will point to the products EULA or Terms, that will be shown in some kind of pager inside the terminal that will allow accepting or declining. If you accept, download commences and process continues as normal. If declined, just exit.
Also maybe a flag to auto-accept the EULA for purposes of automation.

Wouldn't this be problematic when running winget as part of a script? You're not guaranteed to have an interactive shell

It would only be problematic if the package you are trying to script against uses the flag, in which case you probably were not scripting that before this anyway no? since it would imply that the package required users to agree to the EULA/Terms before download anyway

I might have accepted the EULA before creating my script, but if I give my script to a friend who has not accepted the EULA, then that might be a problem 😕 .
Another solution might be to add a --accept-licenses flag which auto-accepts the EULAs and prints something like By using the --accept-licenses flag, you agree to the following terms and conditions followed by links to the relevant EULAs. This isn't perfect but would solve the interactivity-in-non-interactive-use-cases issue.

It's kind of what I meant with my last sentence of having a flag for automation.
With proper messaging like krism suggested, it could work well enough. Similar to how Oracle JDK works on Linux even. You have to accept licenses to use it but has a flag for automation to auto accept.

This is something we have been discussing internally. I do not believe I managed to create an issue for it already. I'm interested in the discussion here. So far it's similar to early thoughts the team had.
If there were a flag in the manifest, the user experience could be improved either with some form of "strict" mechanism requiring user interaction, and / or prioritizing the EULA prompts for download on package set (<- current conceptual name for a collection of packages) initiation.  If one were to specify the entire set needed to be 100% unattended, we could fail on manifests that have a EULA required for download.
We have observed EULA prompts prior to download, at install time, and on first launch. Our goals are not to prescribe how software publishers handle their EULAs. We want to focus on providing the best user experience with the client given the various options.

Description of the new feature/enhancement
Some packages require users accept an End User License Agreement (EULA).
Proposed technical implementation details (optional)
When a EULA is included in a manifest, the user must accept the EULA in order to install the package.
A "EULA" key should be added to the manifest schema.
A "EULAUrl" key should be added to the manifest schema.
When a user executes show, install, upgrade, or import, the EULA and EULAUrl fields should be displayed if they are included in the manifest.
If a EULA field is present, the user must accept the EULA to continue with install, upgrade, or import.
The user may pass an argument to accept the EULA to eliminate required user interaction, but the EULA fields must still be displayed.
-a, --accept-eula

The user may pass an argument to accept the EULA to eliminate required user interaction, but the EULA fields must still be displayed.
-a, --accept-agreements

--accept-agreements throws an error:
PS C:\Users\user\Desktop> winget install --id 9N97ZCKPD60Q --source msstore --accept-agreements
Windows Package Manager (Preview) v1.3.431-preview
Copyright (c) Microsoft Corporation. All rights reserved.

Argument name was not recognized for the current command : '--accept-agreements'


[FEATURE] Seeing itemName in tooltip when hovering over the item

Hello everyone. New feature proposal here.
Sometimes icons don't tell enough, do they? So I thought when in compact mode, I could turn this on and see itemName in tooltip when hovering over the menuItem. So, if you could code review and merge, that'd be great. See ya!
Signed by Branislav Mateáš.

Hello, thank you, you are amazing!
How about adding tooltip option to SideMenuItem?
So user can add custom tooltip to their item.

n to SideMenuItem?
So user can add custom too

Do you mean like override default tooltip - meaning if user doesn't pass anything, we will show the name of the menuItem, right?
Signed by Branislav Mateáš.


please update version python to higher

ModuleNotFoundError: No module named 'bot.utils.tg_manager.TGSession'
This just run lower than 3.11 please update your base

You should use python 3.10

You should use python 3.10

🤦


Add Picnic Shelter preset

picnic_shelter is one of the most common values of shelter_type, with over 22,000 uses. Especially now that the pavilion preset has been corrected to its British meaning—see #5325—a Picnic Shelter preset would prove useful.

Path: amenity/shelter/picnic_shelter
Name: Picnic Shelter
Geometry: point, area
Fields:name, shelter_type, building_area, lit, bin
Tags: "amenity"="shelter", "shelter_type"="picnic_shelter"
Reference: "key": "shelter_type", "value": "picnic_shelter"
Terms: pavilion
Icon: maki-shelter

Hi! I'd like to take on this issue 😄


Incorrect class in code samples for throttling doc

The code samples given in documentation of the
InMemoryThrottledSubmissionByIpAddressAndUsernameHandlerInterceptorAdapter
class are invalid, featuring the similar
InMemoryThrottledSubmissionByIpAddressHandlerInterceptorAdapter
class instead

Why is it invalid?

Great thanks!
Note that the development documentation needs to happen in master, but I'll take care of it.


Qrcodegenerator jenilgajjar20 branch

Hi @thinkswell, I have added a project name QRCode Generator. Also added a screenshot in the readme file.

Same reason buddy. please make PR's again, will review them quickly.


Downstream Option Mechanism

Downstream Option Mechanism
During a Slang compilation work may be performed by multiple other stages including downstream compilers and linkers. It isn't possible in general or perhaps even desirable to provide Slang equivalents of every option available at every stage of compilation. It is useful to be able to set options specific to a particular complation stage - to alter code generation, linkage and other options.
Prior art
The gcc toolchain has a somewhat analagous issue around passing options to the downstream linker. The solution used there is
-Xlinker option

The mechanism requires -Xlinker to be used in front of every option. If an option and value are separate it requires two -Xlinker options.
-Xlinker someOption -Xlinker someOptionsValue

It suggests using the = style setting which makes the setting a parameter somewhat less painful.
-Xlinker -someOption=someOptionValue

This is an improvement. Unfortunately Slang command line processing does not currently support = style.
Gcc also has the -Wl option which allows a comma-separated list of tokens to be passed to the linked.

Pass option as an option to the linker. If option contains commas, it is split into multiple options at the commas. You can use this syntax to pass an argument to the option. For example, -Wl,-Map,output.map passes -Map output.map to the linker. When using the GNU linker, you can also get the same effect with -Wl,-Map=output.map.

The option presumably exists to work around the verbosity of having to write every option with the preceeding -Xlinker. A nice property of the -Xlinker style is that it's pretty clear what's going on. It also allows for easy extension around other kinds of downstream tools by having different names other than linker.
(Clang)[https://clang.llvm.org/docs/ClangCommandLineReference.html#linker-flags] is similar but we also have --for-linker.
-X
-Xlinker <arg>, --for-linker <arg>, --for-linker=<arg>

If there are multiple options using this mechanism could get somewhat monotonous. You could imagine an extension such that all of the parameters up to the next '-X' are taken to go to the target so the above option would become
-Xlinker -someOption someOptionValue

That an empty -X could be used to indicate return to normal command line processing. This is somewhat appealing except the behavior deviates from the -X in gcc/clang, so the additional behavior needs to be a slightly different mechanism.
Discussion
In the Slang scenario it could be argued that there were more 'stages' than just linking.

Source generation
Source compilation (typically but not necessarily) via a downstream compiler
Assembly generation
Linking

There is a desire that it be possible to compile to many targets in a single invocation. Even if this wasn't the case it could be argued useful to be able to target specific options for stages of compilation even if those stages are not invoked, because the options then only need to specified once.
This being the case we probably want a mechanism to be able to target options for specific stages. For example
-Xfxc someFxcOption

That the Slang front end will not perform any parsing of these options.
Parsing
Some downstream tools might may have command line parsing, but the way that Slang interfaces with them that command line parsing isn't available - for example GLSLANG doesn't have a command line parsing interface via our shared library interface. There is GLSLANG (command line interface)[https://github.com/KhronosGroup/glslang/blob/master/StandAlone/StandAlone.cpp] implementation though.
There may be work to implement downstream command line option parsing to make them available and targettable through the mechanism. It may be useful to look at some downstream compiler implementations to see if their native command line parsing can be used as is in some way. Looking at GLSLANGs implementation it doesn't seem it's in a form that is directly usable because of the use of global variables.
Overlapping options
Slang may have options that are also available via regular Slang command line parameters to downstream compilers. What should happen if these values supplied to Slang and the downstream option are inconsistent? Is it necessary to implement parsing for a downstream compiler to access an option that is already available through a standard Slang option?
The simplest solution in the short term is to only allow settings via the Slang options to be set that way, and not to put in any effort into repeating functionality into downstream compiler functionality. Doing so is less effort, but also removes an ambiguity around the inconsistency.
A perhaps more subtle scenario is where an option is not explicitly set on the Slang side, but is on the downstream options. Again here we probably have to go with an option not explicitly set is implicitly set, and that implicit option should be the one used.
Some options perhaps shouldn't be considered as a single setting. Take for example macro definitions. They can be set via multiple options on a command line. Additionally it is easy to imagine scenarios where adding some macro definitions for a particular downstream compiler could be quite useful. It could also provide a mechanism to control options in downstream compiler prelude for example.
There is perhaps another scenario - where we want to set some setting for a downstream compiler using the Slang standardized options. This is probably something useful to support and implies that we would potentially allow Slang command line parsing on command lines that were for downstream compilers. The problem now is that it needs to be unambigous to the downstream compiler that that is the case.
Yet another scenario is that some options are 'generated' as part of a compilation. Take for example the shader model for CUDA - on performing a compilation of Slang source, intrinsics indicate what CUDA SM is required. If we are compiling in the pass through mode - then this isn't possible, and the user will need to be able to specify what shader model they want. If this is implemented (currently it's possible that it's not possible to set SM from Slang command line) then there is the potential for a clash. Do we want the slang command line option to override the the one determined by compiling the source? Should the higher value be taken?
We have the capability system as a mechanism to be able to indicate if a specific capability is required - allowing a use of a feature that is only on available on a later shader model. Should this mechanism work for pass through mode?
Other tools
With the render test tool there is a similar issue - that sometimes we want to specify options to the tool and other options to some system (often Slang) that will be used in the test.
Ideally any mechanism we use would work well in this scenario also.
Other issues
Slang does not support a mechanism to document command line options as part of the runtime.
With extra downstream options it would seem to be beneficial to be able to document them - especially if they are somehow different from the downstream tools usage, or there is no equivalent on a downstream tool.
Options could documented via .md files - but as can be seen with the slang command line options documentation this generally means not all options are covered or covered completely.
Multiple parameter syntax
For supporting multiple parameters the syntax, one option might be -X-
An example in use might be...
slang file -X-fxc -O3 -DFXC -X -Iinclude

The idea here is to make it distinct but a kind of an extension to the standard mechanism. That it is still possible to use the -Xfxc for a single arg. The list can be terminated by the next -X. There is perhaps an open question about how to support parsing -X to a downstream compiler (say with -Xlinker). Or perhaps it can only be terminated via -X-. This would allow -Xlinker to be passed appropriately.
Another style might be -Xfxc...
The idea here is to use the -X syntax but use a trailing ellipsis to indicate that multiple arguments after the -X will be passed to the named item.
We could indicate the end with

-X... - the idea is to keep similar syntax, but this seems to indicate an ongoing ellipsis
-Xend - perhaps clearer. Perhaps we allow -Xfxc-start instead of or as well as?
-X. - just the dot is indicate the 'end'. Allows other -X to go through to downstream

Some examples of the options
// Ellipsis style
slang file -Xfxc... -O3 -DFXC -X. -Iinclude -Xdxc... -DDXC -Invapi
// Start end style
slang file -Xfxc-start -O -DFXC -Xend -Iinclude -Xdxc-start -DDXC -Invapi
// Dash style
slang file -X-fxc -O -DFXC -X- -Iinclude  -X-dxc -DDXC -Invapi

Proposal

Support the -Xthing mechanism for sending parameters to downstream tools
** thing can be the names of downstream tools, and we can have some standardized names such as linker, downstream, slang, cpp (say for any downstream C++ compiler) etc.
Make the render-tool support the same mechanism/s (it seems to currently seems to support -xslang)
Support multiple listed arguments (some discussion on style below)
Support a mechanism to use Slang style argument parsing for downstream compilers (where appropriate) - this is probably the case but should be confirmed
We report a warning for any named downstream entity that is unknown. Those parameters will be discarded.
Parameters are ignored without warning if the downstream entity isn't used in a compilation.
If there are params that are passed to downstream compiler in a way that is incompatible with Slang command line parsing, if it is a desired parameter, it will be made available in a way close to the original mechanism but in a compatible way.

In terms of initial work - downstream compilers command line parsing will only be supported when there is a user need. In general we should prefer options that are part of the Slang option processing. Such options must also be honored by the pass though mode.
Future work

Use SourceLoc/SourceManager mechanism
** Provides a mechanism to support other languages
** Provides a standard mechanism to provide parameters to a message
** Provides a mechanism to highlight visibly where the issue is on a command line
Add support for the --param= and -param= styles
Command line parsing toolkit/framework
Runtime documentation
Mechanism to produce offline documentation

IIUC, both glslang and fxc have the problem that users think of them as having specific command-line options, but our current compiler code can't easily emulate the parsing of those options.
Given that we are building glslang from source, it is reasonable to expect that we could eventually update the slang-glslang shared library interface to include additional string options and then parse those using the same logic as the command-line glslangValidator executable would use.
For fxc, we have the harder issue that there isn't (to my knoweldge) any programmatic interface that can be used to parse command-line arguments in a way that matches command-line fxc.exe and then feed them to D3DCompile. I expect that in practice we'd need to implement that options parsing by hand, and that it could be complicated by the way that fxc.exe uses traditional DOS/Windows style option syntax (e.g., /h for help instead of -h or similar).


Socket error

events.js:174
      throw er; // Unhandled 'error' event
      ^

Error: getaddrinfo ENOTFOUND bro-ms-pc565 bro-ms-pc565:8000
    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:56:26)
Emitted 'error' event at:
    at Socket.<anonymous> (C:\projects\photon\node_modules\@dashersw\axon\lib\sockets\sock.js:199:49)
    at Socket.emit (events.js:189:13)
    at Socket.EventEmitter.emit (domain.js:441:20)
    at emitErrorNT (internal/streams/destroy.js:82:8)
    at emitErrorAndCloseNT (internal/streams/destroy.js:50:3)
    at process._tickCallback (internal/process/next_tick.js:63:19)

How to handle socket errors?

For requester, responder, publisher or subscriber you can do this.sock.on('error' cb).

@dashersw What about redis error?
How to handle it?
Thanks

@dreambit it's a little tricky, but you can access it through requester.discovery.broadcast.socket.sub.on('error', err => console.log(err)) where requester is any cote component.


DPSy Assistant

Is there a GPT that can help produce DSPy code? Would be pretty neat.

Maybe something like this?
https://github.com/seanchatmangpt/dspygen


[BACKLOG-21800] Adding support for @InjectionDeep to StepMetaProps

serialization.
This will allow embedding objects in JmsConsumer meta objects for
better reuse.

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -Dsurefire.runOrder=alphabetical -B -fn -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd clean install && mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd site

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/core/injection/bean/BeanInjector.java
  engine/src/main/java/org/pentaho/di/core/util/serialization/README.md
  engine/src/main/java/org/pentaho/di/core/util/serialization/StepMetaProps.java
  engine/src/test/java/org/pentaho/di/core/util/serialization/StepMetaPropsTest.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes: <0.1% (click to expand)
org.pentaho.di.core.injection.bean.BeanInjector

Instruction Change: -1.1% :small_red_triangle_down:
Line Change: -1.4% :small_red_triangle_down:
Method Change: -6.7% :small_red_triangle_down:

org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change: -0.4% :small_red_triangle_down:
Complexity Change: -0.6% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.step.StepMeta

Branch Change: -0.7% :small_red_triangle_down:
Complexity Change: -0.6% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.csvinput.CsvInputMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -3.3% :small_red_triangle_down:
Complexity Change: -3.8% :small_red_triangle_down:
Instruction Change: -0.3% :small_red_triangle_down:
Line Change: -0.9% :small_red_triangle_down:

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.synchronizeaftermerge.SynchronizeAfterMergeMeta

Instruction Change: -0.1% :small_red_triangle_down:

@pentaho/ackbar


NODE-631 db.collection after db.close() > does not handle error

Suggested change base on the issue that was posted.

Weird, tests just silently stop after running "started test [Should correctly connect to server using big connection pool]" on my machine without getting to the error part.

Weird, tests just silently stop after running "started test [Should correctly connect to server using big connection pool]" on my machine without getting to the error part.


Add support for Mercurial repositories?

Proposed change
In mybinder.org, add a new choice to specify a repository just after "Git repository": "Mercurial repository".
Then, the information to give would be very similar than for the case "Git repository":

"Arbitrary hg repository URL (http://hg.example.com/repo)"
hg branch, tag, topic, bookmark or commit SHA

Who would use this feature?
Mercurial users.
One may argue that monopoly in version control is pretty bad for versioning and open-source. Without Git challenger, versioning is not as good as it could be. For example, Mercurial community (people at Facebook) invented the command absorb which is really useful and I guess Git will get something similar someday.
Mercurial is still alive, with good corporate usage and an active community. The Gitlab fork supporting Mercurial (Heptapod, used for example by PyPy) will soon reach its 1.0 version.
(Optional): Suggest a solution
The implementation for MyBinder should be quite simple : tiny changes in the website, one variable about the kind of repo (git or hg), few diff for the hg commands.

The backend of Binderhub is repo2docker which supports multiple content-providers, you can see the current ones here:
https://www.github.com/jupyterhub/repo2docker/tree/master/repo2docker%2Fcontentproviders
I think this is where the bulk of the work has to happen.
If Mercurial is added there then BinderHub can be updated to offer it.
Is this something you're interested in working on?

Is this something you're interested in working on?

Great, it's in Python! I'm going to have a look at it. Thanks for the link.

Mercurial support has now landed in master of repo2docker :slightly_smiling_face:
I guess the next steps would be (i) wait for a new release of repo2docker (?) and (ii) a PR similar to https://github.com/jupyterhub/binderhub/pull/1100 but for Mercurial repositories.
I don't know JS but if it is as simple as that, it seems doable.
Is there something else to do? Any advice / suggestions?

You don't need to wait for a repo2docker release, all updates are pushed to Docker Hub https://hub.docker.com/r/jupyter/repo2docker/tags and you can also install it from the master branch.
I'm not sure of the exact steps to add a provider to BinderHub, but there are some docs here: https://binderhub.readthedocs.io/en/latest/developer/repoproviders.html#adding-a-new-repository-provider


fix: [#8536] any() expression not available in 2.1?

Description
This PR upgrades the version of the adaptive-expressions package in composer/packages/lib/indexers to avoid showing a warning when the pre-built function any() is used as an expression.
Task Item
Fixes #8536
Screenshots
These images show the displayed warning before the changes and the any() expression being properly validated, with no warnings.

Coverage remained the same at 54.648% when pulling 4d2c84d46f184471c567ce7748f59295059abbcf on southworks/fix/any-expression-warning into a27c3e55332ee6b5a63619f375a5b2fd127923f4 on main.

Closing as the adaptive-expressions package was updated in #9486


packit: build against EPEL{8,9}-next

Signed-off-by: Tomas Tomecek<EMAIL_ADDRESS>

/packit build

aha https://github.com/packit/packit-service/pull/1417
that's why nothing's happening

continuing in #2


output: add support for picking table output fields through jmespath query

[x] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines
~- [ ] Each command and parameter has a meaningful description.~
~- [ ] Each new command has a test.~
(see Authoring Command Modules)

Codecov Report

Merging #3581 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3581      +/-   ##
==========================================
+ Coverage   71.52%   71.53%   +0.01%
==========================================
  Files         420      420
  Lines       26138    26138
  Branches     3958     3958
==========================================
+ Hits        18694    18698       +4
+ Misses       6282     6281       -1
+ Partials     1162     1159       -3

Impacted Files
Coverage Δ

src/azure-cli-core/azure/cli/core/_output.py
62.06% <100%> (+0.66%)
:arrow_up:

...re-cli-vm/azure/cli/command_modules/vm/commands.py
90.1% <100%> (+2%)
:arrow_up:

src/azure-cli-core/azure/cli/core/util.py
69.62% <0%> (ø)
:arrow_up:

...dback/azure/cli/command_modules/feedback/custom.py
32.65% <0%> (ø)
:arrow_up:

...nent/azure/cli/command_modules/component/custom.py
16.23% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Cannot exit visual mode by hitting 'v' key

Cannot exit visual mode by hitting v key

I meet same problem , must hitting escape to exit visual mode .

Sorry for this mess, I didn't noticed this mess, just wanted to fix my commit log, sorry I should learn git.


no support for optional settings in GET request

some GET request support additional setting , but in the class no support for this .
GET exemple with additional setting
pvesh get /cluster/resources -type vm
pvesh get /nodes/localhost/storage/local/content -content vztmpl
pvesh get /nodes/localhost/bootlog -limit 3 -start 5
pvesh get /nodes/localhost/rrd -cf average

@CpuID - Want me to tackle this one, or do you have it covered?

My gut feeling is just appending to the ->get() call first parameter makes sense here? I can understand the elegance of having it natively supported for example as a second optional array parameter to function get (), but it feels like it may blow out logic wise...? Chances are the only thing you would do within the get() function is append to action_path before sending to ->action() anyway...?

@danhunsaker - I'll let you tackle it if you like :)

make it as the put function , in my own script , i just used the same logic of put postfields before sending the request .
any way if is implemented it will be a good option .

@naja7host, @CpuID - I've submitted a PR for this (#12) if you would be willing to check it out.  I'll merge it in a few days if there's no feedback suggesting otherwise.

i have tested it , work like a charm  :)

it not working , because the class send the first request $action_path before the the switch case .
so the get parametre is set later , and it should build with the $action_path before it sended .
moving the line to the end of switch case , do the trick .
curl_setopt($prox_ch, CURLOPT_URL, "https://{$this->hostname}:{$this->port}/api2/json{$action_path}");

is still not working , any one tested the optional settings in GET request ?


Relax constraints for Gaussian Mixture update rules

The current implementation of the update rules for Gaussian Mixture is very strict given the mean-field assumption around this node.
For example, the constraint on q_m  can be relaxed from UnivariateNormalDistributionsFamily to Any (same applies to q_p)
@rule NormalMixture{N}(:out, Marginalisation) (
    q_switch::Any,
    q_m::ManyOf{N, UnivariateNormalDistributionsFamily},
    q_p::ManyOf{N, GammaDistributionsFamily}
) where {N} = begin
    πs = probvec(q_switch)
    return NormalMeanPrecision(sum(πs .* mean.(q_m)), sum(πs .* mean.(q_p)))
end

A similar problem occurs for the update rules on other interfaces for GM node.

Great work @wouterwln! Although I expected to apply this "universality" not only for the out interface.

@albertpod I'll look into it tomorrow. I remembered on the way back this wasn't the only rule to update

Merged in #235


Non-standard course

Here's an interesting course: https://www.udemy.com/ancient-art-of-south-arabia-the-himyarite-kingdom-of-yemen/learn/v4/overview
Each slide has a video and 1 or more static images.
Take a look, maybe you can add support for these types of courses.

Thanks for reporting. The course only has 2 sections and both sections contain videos only?


Wikihub logo

I linked to wikihub on Reddit and noticed that the thumbnail is completely generic, so I made a simple logo for the Wikihub org.
400px:

1600px:

Based on this:
https://openclipart.org/detail/37105/lettre

Awesome!

Im going to use wikihub in my next project and needed to give some credit, so I made a SVG version because SVG for the win!

Hope it helps,

Cool thanks :+1:


🌐 Add Chinese translation for Tutorial - Body - Updates

🌐 Add Chinese translation for Tutorial - Body - Updates

It seems this was translated in another PR, so I'll chose this one, but thanks! ☕