Combination characters not working in Google Docs

Finding this app on the Play Store was a refresheer since I couldn't use the Maltese 47 key layout on Android with my Bluetooth keyboard.
I tested it and I'm happy with it. However, I discovered that on Google Docs I cannot type accented characters with AltGr + vowel. These shortcuts work in Firefox on Android, but not on Docs.
I'm running this on Android 10.

This is clearly an issue with Google Docs. AltGr works in their comment functionality, but not in editing the document proper. The issue also reproduces with Android-provided layouts like German. Please report the issue to Google.
Here is the feedback that I submitted in-app - let me know if I can help by reporting it some other place as well:

Physical keyboard ralt / altgr working in comments, but not in edit mode
To reproduce:

Connect a physical keyboard to an Android device e.g. via otg / bluetooth
From Settings >Language and Input >Physical Keybaord, check  German (Android)
Open any doc in Google Docs
switch to the German layout in case multiple are selected
Try to edit the  document ant type ralt +q. The at-sign @ should appear, but nothing happens. Type ralt +e. An euro sign  € should appera, but nothing happens. Same with ralt +2 that should type a superscript 2. I suppose none of the ralt combinations is working
Try to do the same in a comment. The ralt layer is working as expected

I've also reproduced this with the Romanian Programmers layout provided by the External Physical Keyboards App. In the attached screenshot, I was trying to type "brânză?"- the Romanian word for cheese followed by a question mark- and only got "brnz?". This worked just fine in comments. Another user of the app also reported it to me.
If you need help testing fixes, don't hesitate to contact me.


Add panda theme

Hi,
I added a theme I found a while back and really liked. Do I need to do anything else?

Thank you


Returned manifest contains Error 446

I'm submitting a ...

[X] bug report
[ ] feature request
[ ] support request

General information
Prerequisites

[X] Are you running the latest version?

Description
Trying to play a video displays the loading icon for a few (2 or 3) seconds, then disappear and nothing is being played.
The most notable thing is, in the logs (see below): “Playback not permitted. Reason: 446 (Playback not permitted.)”
Note that I’m not using any proxy or VPN, this account has been created in my country, I’m still in my country, and it works just fine with the official Netflix app on my TV, on the exact same internet connection.
Context (Environment)
Installation

[ ] I installed the plugin via zip from the Releases page
[X] I´m using the Netflix Repo
[ ] I´m using a different source (Please tell which)

Operating System

[ ] Linux (x86/x64)
[ ] OSX (x86/x64)
[ ] Windows (x86/x64)
[X] Linux (ARM)
[ ] Android

Additional informatin on the environment
On Vero+, on OSMC, updated to kodi 18 like this: https://discourse.osmc.tv/t/testing-kodi-18-leia-builds-for-vero-2-4k/23059
Debug log
http://sprunge.us/0mh3Jt

Same thing here.

duplicate to #555


Won't initialize after AJAX call

I am loading the items via ajax and it returns HTML. I then insert that HTML into the container. Then initialize flipster. First call works just fine but the succeeding calls are breaking. It gives me stacked images after.
Anyone out there who can help?

If you are trying to add images after you have initialized flipster, you may experience issues.
I tried something similar, just pushing a new image to the image array, after flipster was already initialized, and the images were placed all weird.
My work around was waiting until all my images were loaded, then initializing flipster. If I ever had to add a new image after the fact, I removed flipster from the DOM and then reinitialized it.

@DaDanny is correct. Currently everything is set up on initialization and won't compensate for added items.
The 1.0 branch is almost ready for launch which includes an 'index' method that will allow items to be added/removed. So with 1.0, something like this should work:
$flipster = $("#myflip").flipster();

// items added...
$flipster.find('ul').append('
New Item
');

$flipster.flipster('index');


Update geo codes for Azure Backup

Overview/Summary
This PR updates the list of geo codes used for Azure Backup private DNS zones
This PR fixes/adds/changes/removes

Update the list of geo codes used for Azure Backup private DNS zones

Breaking Changes
n/a
Testing Evidence
Replace this with any testing evidence to show that your Pull Request works/fixes as described and planned (include screenshots, if appropriate).
As part of this Pull Request I have

[x] Read the Contribution Guide and ensured this PR is compliant with the guide
[ ] Ensured the resource API versions in .bicep file/s I am adding/editing are using the latest API version possible
[x] Checked for duplicate Pull Requests
[x] Associated it with relevant GitHub Issues
[x] (ALZ Bicep Core Team Only) Associated it with relevant ADO Items
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[ ] Performed testing and provided evidence.
[ ] Updated one or more of the following tests (if required)

Unit
Linting
E2E (End-To-End)
ValidateAzCloud (Base validation in Azure Cloud)
ValidateMcCloud (Base validation in Azure China Cloud)

[ ] Updated relevant and associated documentation (e.g. Contribution Guide, Module READMEs, Wiki Docs etc.)


SwiftyButton does not work with Carthage

$ carthage update --platform iOS
*** Fetching SwiftyButton
*** Fetching SwiftyJSON
*** Fetching Alamofire
*** Checking out Alamofire at "4.3.0"
*** Checking out SwiftyButton at "0.6.0"
*** Downloading SwiftyJSON.framework binary at "3.1.4"
*** xcodebuild output can be found in /var/folders/xl/gxvzy0ws6q3bs_p2q2k69mg00000gn/T/carthage-xcodebuild.CKTxod.log
*** Building scheme "Alamofire iOS" in Alamofire.xcworkspace
*** Skipped building SwiftyButton due to the error:
Dependency "SwiftyButton" has no shared framework schemes for any of the platforms: iOS

If you believe this to be an error, please file an issue with the maintainers at Optional("https://github.com/TakeScoop/SwiftyButton/issues/new")

Bump? I kind of need to use this library and I just switched from CocoaPods to Carthage.

@perryprog Thanks for reply.
I changed it from carthage to cocoapods.

@rnitame you should probably leave it open since the problem still exists. Just so TakeScoop knows that people still have a problem with that.

@perryprog sure. I've reopened this issue

@rnitame @perryprog  I've created a blank project and added the following Cartfile:
github "TakeScoop/SwiftyButton" "master"

Then ran
$ carthage update --platform iOS

*** Fetching SwiftyButton
*** Checking out SwiftyButton at "211690b4f9fdf0bbeefa086d6f527654e76e5022"
*** xcodebuild output can be found in /var/folders/ns/__x7qnxd17b_mg_wzntwzm8c0000gn/T/carthage-xcodebuild.zsW6Mp.log
*** Building scheme "SwiftyButton" in SwiftyButton.xcworkspace

I drag-dropped the resulting framework, imported the framework, created a button and compiled the whole project: everything worked fine.
Am I missing something here? I used carthage 0.20.1.

@ldiqual It worked in own environment! XD

Well then. I'll try that, but right now I'm running into a problem with submitting my app to the appnstore. (Incorrect bundle ID or something.) https://github.com/Carthage/Carthage/issues/1834
That's my shameless plug of the day.

Looks like it works now so I'm gonna close the issue. Feel free to ask for a reopen if you still run into issues. Thanks!


Improve overall readblity of code

Hi,
I made some grammar and spelling fixups to improve overall readablity.
thanks

Hi, @gbaned, shall I revert the commits that are causing the issue?
Thanks


feature request: readonly only outside and writable inside

Search Terms

Suggestion
Sometime we need a value readonly from outside, but writable of cause inside. Usually we do it by double properties, but that's superfluous and waste of performance.
Use Cases
export default class Foo {

    private _value1;
    protected get value1 () { return this._value1; }

    private _value2;
    public get value2 () { return this._value2; }

    protected _value3;
    public get value3 () { return this._value3; }

};

We do all things above to make sure the value could only write from inside, otherwise, there will be a TS warn. If we want to save performance (ignore the superfluous problem being worse):
declare class _Foo {
    protected readonly value1;
    readonly value2;
    readonly value3;
}
const Foo = class {
    value1;
    value2;
    value3;
} as typeof _Foo;
type Foo = _Foo;
export { Foo as default };

Examples
One solution (everyone can suggest a better one of cause, I mainly point out the need):
export default class Foo {
    protected private value1;
    public private value2;
    public protected value3;
};

Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

Duplicate of #37487.


adjusting code to fit a multi-label problem

Hi! Thanks for sharing your work :)
I am trying to perform some tests using paws on a multi-label problem. Major changes I've have already implemented: (1) my dataset custom class and its corresponding "Trans" class as the "TransImageNet"; (2) implemented custom _make_data_transforms functions and _make_multicrop_data_transforms.
Now I'm working on adapting the ClassStratifiedSampler, and therefore also labels_matrix. I'm having trouble fully understanding how these two work together: is labels_matrix simply concatenating one-hot labels from the sampler (ClassStratifiedSampler) and smoothing it? Also, do you think it make sense to adapt ClassStratifiedSampler to a multi-label dataset, or should I just use a regular sampler (then I could do as mentioned in https://github.com/facebookresearch/suncet/issues/22#issuecomment-901921422)?
Thanks in advance for any tip!

Hi @amandalucasp!
That sounds like a really interesting extension! All that sounds great; I'll answer the question below:
On ClassStratifiedSampler and labels_matrix:

The class ClassStratifiedSampler is used to sample the support mini-batch in each iteration. It first samples a set of classes, and then samples an equal number of images from each class (i.e., the common class-balanced sampling setup). This is important since a highly unbalanced support set would bias your nearest-neighbour classifier (i.e., you have a higher chance of predicting a certain class if most of the instances in the support are from that class).
labels_matrix just identifies which images in your mini-batch belong to the same class. Imagine you have 1 billion classes, but in each iteration, your support-set sampler only sub-samples 100 classes. Then labels_matrix can be a one-hot matrix with 100 classes, identifying which images in your mini-batch come from the same class. As an additional step, you can apply label-smoothing on this matrix, which is helpful for improving stability with mixed-precision training.

Honestly, ClassStratifiedSampler is a little complex, just because of how the ImageNet data is structured. If your data is not too imbalanced, you could try starting out with just use a regular sampler, and you can set labels_matrix to the concatenation of the one-hot labels form the sampler (I would still use smoothing here to improve stability though). If your labels_matrix is not one-hot though, then you may need to change the loss since a standard cross-entropy (i.e., multinomial loss) would no longer make sense here.

Hi @MidoAssran! Thanks so much for the quick reply.
I had to take a small break from the project due to health issues (that's why I'm kind of replying so late) but will get back to it soon. Your remarks will be key to my advances. Thanks again for the thorough response :)

Hi @amandalucasp hope you're doing ok!
Yes that logic makes sense to me, but happy to take a look at any code if you'd like to verify!
All the ClassStratifiedSampler needs to do its job is the target_indices property in the dataset object passed to it, which, given a target (integer class label), will map to the indices in your dataset belonging to that target! Sounds to me like you've adjusted this already, so that sounds great. However, I would just look at the labels sampled by your custom ClassStratifiedSampler just to make sure it's working correctly.

I'm fine now, thanks =)
Great! If it comes to that, will do :) thank you!
And yes, I made sure target_indices are integers. But for the rest of the code, I believe the labels are treated as one-hot, as in labels_matrix. Right? If I print the labels from
https://github.com/amandalucasp/dino_paws_submarine/blob/4f6872f386e50137334e89f4ec64272ca408f71b/suncet/src/paws_train.py#L320
This is what I get:
labels: tensor([[0.9100, 0.0100, 0.0100, 0.0100, 0.0100, 0.0100, 0.0100, 0.0100, 0.0100,
         0.0100],
        [0.0100, 0.9100, 0.0100, 0.0100, 0.0100, 0.0100, 0.0100, 0.0100, 0.0100,
         0.0100],
        [0.0100, 0.0100, 0.9100, 0.0100, 0.0100, 0.0100, 0.0100, 0.0100, 0.0100,
         0.0100],
        [0.0100, 0.0100, 0.0100, 0.9100, 0.0100, 0.0100, 0.0100, 0.0100, 0.0100,
         0.0100],
(...)

I believe it makes sense as I'm using label smoothing as you recommended.

Hi @amandalucasp , I am trying to extend PAWS to multi-label classification problem and saw here that you are working on the same problem. Have you managed to write a new stratified sampler? If so, It would appreciate if you could please share your code. cheers!

Hi @mahsaep :) I decided to start by simplifying my problem to fit a multi-class paradigm during pre-training stage and multi-label only for fine-tuning and the downstream application. My dataset is highly imbalanced so a custom sampler would be way more complex. Since I'm still not done with these experiments (some are still running), I haven't got back to the idea of adjusting pre-training with paws for a multi-label dataset... but good luck! =)
Oh and btw, you should also keep in mind some remarks made in  https://github.com/facebookresearch/suncet/issues/35#issuecomment-1157005128, that you might need to change the loss that's implemented, because it will no long fit the problem.

Hi @mahsaep :) My dataset is highly imbalanced therefore a custom sampler would be a bit complex, so for now I'm trying a different approach. Since I'm still not done with these experiments (some are still running), I haven't got back to the idea of adjusting pre-training with paws for a multi-label dataset... but good luck! =)
Oh and btw, you should also keep in mind some remarks made in https://github.com/facebookresearch/suncet/issues/35#issuecomment-1157005128, that you might need to change the loss that's implemented.


Use open_spiel namespace prefix for float assertions

This is to allow float assertion macros to be used without using the open_spiel namespace itself. Most of the other macros use the open_spiel prefixes in this way so I think these two were just missed.

Great, thanks! Importing now, should be merged in tomorrow's update.


fix: Expensify card link navigates to wallet

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/49804
PROPOSAL: https://github.com/Expensify/App/issues/49804#issuecomment-2378111577
Tests

Go to ND and log in
Create a workspace
Navigate to the workspace settings > More Features
Enable Expensify card option
Navigate to the Expensify card and click on Issue card
Select Connect online with plaid and add the VBA
Issue the virtual Expensify card to the current user
Navigate to the workspace chat
Click on the Expensify card link
Verify: User is navigated to the virtual Expensify card details

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[ ] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

https://github.com/user-attachments/assets/d7e14f14-e343-46df-9774-6b45004e7383

iOS: Native

https://github.com/user-attachments/assets/a2561444-88b3-4cb4-8ea9-7ca0ce1c1c09

iOS: mWeb Safari

https://github.com/user-attachments/assets/73521fae-ca81-4ddf-bcd1-6e4d701a7cf1

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/26245782-1df7-4099-84ef-add3c5bb43b3

MacOS: Desktop

https://github.com/user-attachments/assets/bfee2989-4165-439a-9cae-4f7eec9a245b

I cannot run android native. Will update the video later.

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

@truph01 Were you able to actually test this? I can't seem to get a card added.

BUG: If the user the card is issued to is not the admin, clicking the link from the workspace chat (as that non-admin user) they are navigated to a Not found page. The route in this case should be settings/wallet/card/${cardID}.
https://github.com/user-attachments/assets/be20d308-19d1-4a21-8618-a8a9b29c3755

BUG: On first sign-in, clicking the link loads the card details but the cards page behind the modal is blank and has a jittery render when dismissing the modal. This is because normally we'd call OpenPolicyExpensifyCardsPage first when this page is opened "normally".
https://github.com/user-attachments/assets/644c758c-2c63-4a30-9a86-a46246a44175

BUG: If the user the card is issued to is not the admin, clicking the link from the workspace chat (as that non-admin user) they are navigated to a Not found page. The route in this case should be settings/wallet/card/${cardID}.

Fixed this bug.

Member:

https://github.com/user-attachments/assets/9c18f8c9-0121-4e8a-b3fc-5d4e7836b7c4

Admin:

https://github.com/user-attachments/assets/19a0de17-920d-4350-959f-9c9d0f3a4576

BUG: On first sign-in, clicking the link loads the card details but the cards page behind the modal is blank and has a jittery render when dismissing the modal. This is because normally we'd call OpenPolicyExpensifyCardsPage first when this page is opened "normally".

When we are on the report page and then click on the card link, it will navigate to the card detail page in RHN. But in background, which screen should be displayed? There are two options:

Currently, in PR, it is:

https://github.com/user-attachments/assets/ae767500-b84d-4c88-9af5-89c87340a5e0

In the background, the report is displayed.

@iwiznia @jjcoffee

When we are on the report page and then click on the card link, it will navigate to the card detail page in RHN. But in background, which screen should be displayed?

I think it probably makes sense to have the report in the background (central pane), unless it adds unnecessary complexity. Let's do that for now, we can revert if @iwiznia feels strongly about it!

@truph01 Could you also add the Android native video when you get a chance? Thanks!

It's probably a bit better to keep the report (or whatever page you were in) in the background, so when you close the RHP you are in the same place you were before

I cannot receive magic code now. Did you encounter the same? @jjcoffee

@truph01 Should be fixed now!

It's probably a bit better to keep the report (or whatever page you were in) in the background, so when you close the RHP you are in the same place you were before

Updated. Below is result:

Admin:
https://github.com/user-attachments/assets/7ccf4232-1515-495c-b79b-68e9c04f0156
Others:
https://github.com/user-attachments/assets/65a2b47a-8c67-4b3a-ae45-981e1395c6da

@iwiznia Also, please help confirm the @jjcoffee's thoughts:

BUG: If the user the card is issued to is not the admin, clicking the link from the workspace chat (as that non-admin user) they are navigated to a Not found page. The route in this case should be settings/wallet/card/${cardID}.

@truph01 Could you also add the Android native video when you get a chance? Thanks!

@jjcoffee I updated android video

@truph01 Testing on Android native, when I tap the link as an admin (the card is assigned to someone else), the not found page shows up. This also happens for the user the card is assigned to. The route in the logs:

Testing on Android native, when I tap the link as an admin (the card is assigned to someone else), the not found page shows up. This also happens for the user the card is assigned to. The route in the logs:

I encountered the same when I tested on Android. I try to call "npm run android" and start metro "npx react-native start :8081" again, then it works properly. Can you try it?

Coming from https://github.com/Expensify/App/issues/50375, we missed handling cases where the card is no longer active/available. More details here.


ftp tests: move them into the fold

Codecov Report

Merging #77 (0fa1f48) into master (d8fc25b) will decrease coverage by 1.78%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #77      +/-   ##
==========================================
- Coverage   93.13%   91.35%   -1.79%
==========================================
  Files           5        5
  Lines         466      451      -15
==========================================
- Hits          434      412      -22
- Misses         32       39       +7

Impacted Files
Coverage Δ

src/Curl/utils.jl
71.42% <0.00%> (-26.08%)
:arrow_down:

src/Downloads.jl
83.87% <0.00%> (-0.18%)
:arrow_down:

src/Curl/Easy.jl
94.68% <0.00%> (-0.03%)
:arrow_down:

src/Curl/Curl.jl
100.00% <0.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 d8fc25b...0fa1f48. Read the comment docs.


Massive player corrupt file issue

I am getting this spammed in the console sometimes, I think it's due to a player's file being corrupted - http://pastebin.com/yW3BHX6e

Is there any way to trace it to quests? Does disabling it fix it?

I know it's Quests, there are random Quests error messages alongside this one and other examples like the BlockBreakEvent to Quests.
It's to do with a player's corrupted file. To fix it, I need to go into their file, remove all current active quests and stages. Then it's fixed.


chore: add Last-Modified header support for CDN configs

#421  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jonathannorris and the rest of your teammates on  Graphite


[WIP] Assistants v2 support

What:

[ ] Bug Fix
[x] New Feature

Description:
This PR introduces the changes made from the V2 assistants API, with the existing responses updated to match the new schema but this has also added support for the new vector store APIs.
This approach to supporting V2 will introduce breaking changes due to V1 API no longer being accessible
Tasks Remaining before moved from draft PR

[ ] Update Readme
[x] Add VectorStoreFiles API
[x] Add VectorStoreFileBatches API
[ ] Test updated endpoints more thoroughly
[ ] Remove old API endpoints (Files)

Related:
#388 #381

Hi @knash94
Thank you very much for this PR.
I will go to review it now.

@knash94 It is nearly impossible for me to review this PR in one go. There I am going to pick from it in a new PR: https://github.com/openai-php/client/pull/405
Of course, giving all the credit you deserve 🙏🏼
I leave this PR open as a reference for now, but do not add more commits to it.

Support for the Assistants API v2 and the Vector Stores API as finally arrived (beta). 🚀
Happy to hear your feedback before making it a stable release.
https://github.com/openai-php/client/releases/tag/v0.10.0-beta.1


Write XAML

Hi Guys! RoslynPad will have in the future a way to write xaml to make GUI? This will make RoslynPad really awesome and complete IMHO.
(RoslynPad actually is the only C# editor that run on UWP, make UWP from an UWP app would be simply perfect!)

Hey @RealAquilus! Thanks for the suggestion, but this is not a direction I currently see RoslynPad going. My current focus is making RP cross-platform (run on Mac/Linux and .NET Core).
BTW, RP is a desktop WPF app, not UWP. It just uses the Desktop Bridge so it could be distributed via the Microsoft Store.


⚠ Bofrost.at has degraded performance

In 0f64e44, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2007 ms

Resolved: Bofrost.at performance has improved in c5845f7 after 6 minutes.


Http Integration Tests: Create tests to support HTTP(9)

This issue involves creating the below test cases for HTTP as well:
Source: integration_tests/commands/async/

object_test.go
select_test.go
set_data_cmd_test.go
toggle_test.go
touch_test.go
ttl_pttl_test.go
type_test.go

Steps:

Create the corresponding tests for these commands in integration_tests/commands/http.
Use set_test.go as a reference, and how to alter the input and output to match JSON structures from RESP.
Run the tests and fix issues with command parsing in HTTP.

If you need to do anything custom to support HTTP, let @pratikpandey21 @lucifercr07 know.

Hi please assign this to me. Thanks,
Apoorv

I don't have the permissions to assign you, but please go ahead and start @apoorvyadav1111

Closing, merged as part of https://github.com/DiceDB/dice/pull/746


Multilingual Archives not Outputting Language Options

I tried adding language info in the archive.html file frontmatter, but I'm still not able to get language info into the tag pages. No language info means no navigation, hreflang, etc.
Any ideas how to get a language tag in the archive files?

Can you explain a bit more about what you are trying to do, maybe with some examples?

I'd like to have tags for my English posts output to _site/tag/tag-name and my Spanish posts output to _site/es/tag/tag-name. I've only been able to get output for the English posts.
I've setup an example repo if you'd like to look at the code.


Show the CSP Organization name in the browser after successful CSP login

What this PR does / why we need it
This PR has changes to show the cloud services organization name in the browser after successful CSP login while creating tanzu context using interactive login.
NOTE: This feature is only available if the OAuth app (client used by CLI) has Organization scope atleast "Organization member" enabled
TODO: Rebase the PR once PR #645 is merged
Which issue(s) this PR fixes

Fixes #
Describe testing done for PR
created a context and the browser shows the cloud services organization name after successful login.
❯ ./bin/tanzu context create prem-ucp-intg-ctx6 --type tanzu --endpoint https://api.tanzu-dev.cloud.vmware.com --staging
[i] Opening the browser window to complete the login
Log in by visiting this link:

    https://console-stg.cloud.vmware.com/csp/gateway/discovery?client_id=tanzu-cli-client-id&code_challenge=tY38d0uwtHvLCnRgHL7252SCzLLEJPwHuJtJvVmevA4&code_challenge_method=S256&redirect_uri=http%3A%2F%2F<IP_ADDRESS>%3A64862%2Fcallback&response_type=code&state=0d9cb632ae13c89fa681a5e8141b2f3a

    Optionally, paste your authorization code: [...]

[ok] Successfully logged into 'One Tanzu Integration' organization and created a tanzu context
[i] Checking for required plugins for context 'prem-ucp-intg-ctx6'...
[i] All required plugins are already installed and up-to-date

Screenshot of the browser with organization id shown after successful login:

Release note

Additional information
Special notes for your reviewer
Please review the top commit as this PR is based on the changes for PR #645, the first commit is from #645. I will rebase once the PR645 is merged.

Small nit on the text displayed. The second sentence suggests a user action, so it might be better to make it more prominent. What do you think about separating the lines?

Sure. I will update the text to have separate lines. Thanks


Potential new evidence: miRNA experiments from Expression Atlas

@ckongEbi commented on Mon Jun 11 2018
With regards to miRNA experiments, Laura identified ~400 studies in ArrayExpress that might be of OT interest.
ArrayExpress-Experiments-180426-121455.txt
They need to process the required studies through atlas for us to get it via their data feed. The iRAP pipeline is used to process the RNA-seq data and needed to be tweak to process miRNA data. I am testing their pipeline and feedback to Nuno for changes to achieve this.

As of January 2019 there is no miRNA-specific pipeline in Expression Atlas. We will follow developments related to miRNAs.


Switched usage to AndroidX libraries.

Updating to androidx libraries

@freerdp-bot test

@nkotixwolf please create pull requests against master branch, we do the backports later on.


1561 - Maximum Number of Coins You Can Get

Description of the Problem
There are 3n piles of coins of varying size, you and your friends will take piles of coins as follows:

In each step, you will choose any 3 piles of coins (not necessarily consecutive).
Of your choice, Alice will pick the pile with the maximum number of coins.
You will pick the next pile with maximum number of coins.
Your friend Bob will pick the last pile.
Repeat until there are no more piles of coins.
Given an array of integers piles where piles[i] is the number of coins in the ith pile.

Return the maximum number of coins which you can have.
Example 1:
Input: piles = [2,4,1,2,7,8]
Output: 9
Explanation: Choose the triplet (2, 7, 8), Alice Pick the pile with 8 coins, you the pile with 7 coins and Bob the last one.
Choose the triplet (1, 2, 4), Alice Pick the pile with 4 coins, you the pile with 2 coins and Bob the last one.
The maximum number of coins which you can have are: 7 + 2 = 9.
On the other hand if we choose this arrangement (1, 2, 8), (2, 4, 7) you only get 2 + 4 = 6 coins which is not optimal.

Example 2:
Input: piles = [2,4,5]
Output: 4

Example 3:
Input: piles = [9,8,7,6,5,1,2,3,4]
Output: 18

Link To The LeetCode Problem
LeetCode

@vJechsmayr may I be assigned this issue?

@Karnak123 It's yours 😄


Fails to build QT ios example

Hello, i'm trying to build this repository with CMake: https://github.com/forexample/qt-ios-examples
I get the following traceback:
Calebs-MBP:build calebmarshall$ cmake .. && make
-- [hunter] Calculating Config-SHA1
-- [hunter] Calculating Toolchain-SHA1
-- [hunter] HUNTER_ROOT: /Users/calebmarshall/.hunter
-- [hunter] [ Hunter-ID: fb52c56 | Config-ID: cb556b4 | Toolchain-ID: b132be4 ]
-- [hunter] QT_ROOT: /Users/calebmarshall/.hunter/_Base/fb52c56/cb556b4/b132be4/Install (ver.: 5.5.1-hunter-p1)
-- [hunter] Building Qt (component: qtbase)
loading initial cache file /Users/calebmarshall/.hunter/_Base/fb52c56/cb556b4/b132be4/cache.cmake
loading initial cache file /Users/calebmarshall/.hunter/_Base/fb52c56/cb556b4/b132be4/Build/Qt/__qtbase/args.cmake
-- The C compiler identification is AppleClang <IP_ADDRESS>10046
-- The CXX compiler identification is AppleClang <IP_ADDRESS>10046
-- Check for working C compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc
-- Check for working C compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc -- works
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++
-- Check for working CXX compiler: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/c++ -- works
-- Detecting CXX compile features
-- Detecting CXX compile features - done

[hunter ** INTERNAL **] qtbase is not the first component
[hunter ** INTERNAL **] [Directory:/Users/calebmarshall/.hunter/_Base/fb52c56/cb556b4/b132be4/Build/Qt/__qtbase]

------------------------------ WIKI -------------------------------
    https://github.com/ruslo/hunter/wiki/error.internal
-------------------------------------------------------------------

CMake Error at /Users/calebmarshall/.hunter/_Base/Download/Hunter/0.12.26/fb52c56/Unpacked/cmake/modules/hunter_wiki.cmake:12 (message):
Call Stack (most recent call first):
  /Users/calebmarshall/.hunter/_Base/Download/Hunter/0.12.26/fb52c56/Unpacked/cmake/modules/hunter_internal_error.cmake:13 (hunter_wiki)
  CMakeLists.txt:297 (hunter_internal_error)

-- Configuring incomplete, errors occurred!
See also "/Users/calebmarshall/.hunter/_Base/fb52c56/cb556b4/b132be4/Build/Qt/__qtbase/Build/CMakeFiles/CMakeOutput.log".

[hunter ** FATAL ERROR **] Configure step failed (dir: /Users/calebmarshall/.hunter/_Base/fb52c56/cb556b4/b132be4/Build/Qt/__qtbase)
[hunter ** FATAL ERROR **] [Directory:/Users/calebmarshall/.hunter/_Base/Download/Hunter/0.12.26/fb52c56/Unpacked/cmake/projects/Qt/qtbase]

------------------------------ WIKI -------------------------------
    https://github.com/ruslo/hunter/wiki/error.external.build.failed
-------------------------------------------------------------------

CMake Error at /Users/calebmarshall/.hunter/_Base/Download/Hunter/0.12.26/fb52c56/Unpacked/cmake/modules/hunter_wiki.cmake:12 (message):
Call Stack (most recent call first):
  /Users/calebmarshall/.hunter/_Base/Download/Hunter/0.12.26/fb52c56/Unpacked/cmake/modules/hunter_fatal_error.cmake:20 (hunter_wiki)
  /Users/calebmarshall/.hunter/_Base/Download/Hunter/0.12.26/fb52c56/Unpacked/cmake/modules/hunter_download.cmake:384 (hunter_fatal_error)
  /Users/calebmarshall/.hunter/_Base/Download/Hunter/0.12.26/fb52c56/Unpacked/cmake/projects/Qt/qtbase/hunter.cmake:86 (hunter_download)
  /Users/calebmarshall/.hunter/_Base/Download/Hunter/0.12.26/fb52c56/Unpacked/cmake/projects/Qt/hunter.cmake:111 (include)
  /Users/calebmarshall/.hunter/_Base/Download/Hunter/0.12.26/fb52c56/Unpacked/cmake/modules/hunter_add_package.cmake:87 (include)
  CMakeLists.txt:30 (hunter_add_package)

-- Configuring incomplete, errors occurred!
See also "/Users/calebmarshall/desktop/qt-ios-examples/build/CMakeFiles/CMakeOutput.log".

cmake .. && make

Examples were designed for iOS. This way, you will build for macOS.
You have to use toolchain:

https://github.com/forexample/qt-ios-examples#usage

Also, as a general note, Qt is pretty much in an abandoned state in Hunter.

https://github.com/ruslo/hunter/blob/v0.23.214/README.rst


Better/more bumping flexibility for initial-development-versions, i.e. < 1.0.0.

It would be great to finally solve https://github.com/conventional-changelog/standard-version/issues/539
TL;DR
In Standard-Version when you have a version < 1.0.0 it always bumps the patch version, no matter what you do.
It would be great to be able to adjust this behavior.

The SemVer 2.0.0 spec days:
How should I deal with revisions in the 0.y.z initial development phase?
The simplest thing to do is start your initial development release at 0.1.0 and then increment the minor version for each subsequent release.

However, this is only the FAQ part and a suggestion (i.e. "The simplest thing to do").
What the spec really defines is:

Major version zero (0.y.z) is for initial development. Anything MAY change at any time. The public API SHOULD NOT be considered stable.

It explicitly allows y (the minor version) to change.
The thread in https://github.com/semver/semver/issues/333#issuecomment-463390138 tackles this issue and I guess this summarized the current state quite good:

There is nothing explicit about what the rules for initial development should be.

As such, IMHO, there should at least be a flag/configuration option in .versionrc to also bump minor in the initial development stage (<1.0.0) just like you would do when it is stable.
Otherwise the version always stays at 0.0.z until 1.0.0 which feels awkward to me and let me to find this issue as it was clearly unexpected/not what I thought standard-version would behave.
Both variants are clearly in line with the spec, so IMHO, standard-version should support it. You could even, and that may be discussed, doing that new behavior by default. Or better introduce it as optional for now and later make it a default, if you want to go that way.
BTW SemVer 2.0.0 also says:
The simplest thing to do is start your initial development release at 0.1.0 and then increment the minor version for each subsequent release.
That's not the patch version (0.0.z), as Standard-Version currently does! Minor would make more, sense IMHO, but still the usual bumping behavior would be better. (0.y.0)

Idea

The default behavior may be changed. Maybe bump the minor version? Or better use the "default behaviour" for bumping as described above.
An entry in .versionrc where you can choose what to bump in pre-release versions like minor or patch (you can set it to patch to keep/restore the old behavior of standard-version) or auto (or maybe called default), where it chooses based on whether it's a feat(ure) or (bug) fix.

I'm going to close this for now, please reopen if you'd like to discuss this more.

I've added some additional discussion here to describe the fork rationale: https://github.com/absolute-version/commit-and-tag-version/discussions/30

are not consumed and relied on like "packages"

Hmm. I don't think semver is for this usecase, though. The spec says:

Software using Semantic Versioning MUST declare a public API. This API could be declared in the code itself or exist strictly in documentation. However it is done, it SHOULD be precise and comprehensive.

Anyway, could you elaborate on what you mean by "properly keep track of new features and bugfixes til then"? You should be able to track those fine, but what will happen before 1.0.0 is just that the minor version will bump for breaking changes, and the patch version will bump for everything else. As discussed above, this is by convention.

I actually expected the preMajor flag to solve that.

I'm not familiar with this flag. Where did you find it documented / defined?

Considering the laid out scenario, any chance to cover those other use-cases as well?

As above, I don't think it's appropriate to directly cover non-semver or unconventional use-cases. However, if you want to override the version number generation, you can use --release-as to release whatever you like.


remote error: tls: handshake failure

Hi, I'm trying to pull images from this registry and it works on certain vms (libvirt domains) but fails on others
Example where it fails:
$ sudo docker pull registry.k8s.io/pause:3.8
3.8: Pulling from pause
9457426d6899: Retrying in 1 second
error pulling image configuration: download failed after attempts=6: remote error: tls: handshake failure

If I run sudo docker pull k8s.gcr.io/pause:3.8 it works.
No difference in the output of openssl s_client -connect registry.k8s.io:443 between a vm where it works and a vm where it fails.
Any idea what might cause this?
Thanks.

There's not a lot to go on here, unfortunately docker pull doesn't have verbose logging, though if debug logs are enabled in the daemon dockerd there will be more logs there.
crane pull --verbose registry.k8s.io/pause:3.8 /dev/null would have enough information to know which API call is failing etc.
https://github.com/google/go-containerregistry/tree/main/cmd/crane
registry.k8s.io's primary endpoint is currently a fairly standard google cloud loadbalancer fronting the application in this repo, TLS termination happens there in the loadbalancer. Most responses are redirects to a varying set of backends, it's possible the TLS handshake issue is with one of the storage backends, but all of those are common storage SaaS.
We're not doing anything unusual here with TLS as far as I know so ... 🤔

It's ok, its our IT departments' dns that we use for external domains that didn't like this domain and inserted their Cisco Umbrella servers' ip instead of the real ip: prod-registry-k8s-io-us-east-1.s3.dualstack.us-east-1.amazonaws.com
We'll get them to green light it and it will probably be fine after that.

@BenTheElder thanks for the help, the crane tool was useful.
End of the output:
2023/02/17 18:38:00 <-- x509: certificate signed by unknown authority GET https://prod-registry-k8s-io-us-east-1.s3.dualstack.us-east-1.amazonaws.com/containers/images/sha256%3A4873874c08efc72e9729683a83
ffbb7502ee729e9a5ac097723806ea7fa13517 (38.852067ms)
Error: saving tarball /dev/null: unable to calculate manifest: Get "https://prod-registry-k8s-io-us-east-1.s3.dualstack.us-east-1.amazonaws.com/containers/images/sha256%3A4873874c08efc72e9729683a83ffbb75
02ee729e9a5ac097723806ea7fa13517": x509: certificate signed by unknown authority

It's ok, its our IT departments' dns servers that we use for external domains that didn't like this domain and inserted their Cisco Umbrella servers' ip instead of the real ips: prod-registry-k8s-io-us-east-1.s3.dualstack.us-east-1.amazonaws.com

Ah, that'll do it.
So, a word of warning: https://registry.k8s.io#stability
TLDR: We use various cloud services to store the actual image content depending on who is providing us with resources and whatever makes sense from a cost perspective at any given time. While the registry API will be at registry.k8s.io, the storage domains / IPs / services used are subject to change anytime, so you might want to mirror images you use somewhere you control that is more predictable. crane can also help with that.

It's ok, its our IT departments' dns servers that we use for external domains that didn't like this domain and inserted their Cisco Umbrella servers' ip instead of the real ips: prod-registry-k8s-io-us-east-1.s3.dualstack.us-east-1.amazonaws.com

Ah, that'll do it.
So, a word of warning: https://registry.k8s.io#stability
TLDR: We use various cloud services to store the actual image content depending on who is providing us with resources and whatever makes sense from a cost perspective at any given time. While the registry API will be at registry.k8s.io, the storage domains / IPs / services used are subject to change anytime, so you might want to mirror images you use somewhere you control that is more predictable. crane can also help with that.

Thanks for the heads up. We'll put it in our road map.


Update oc client to at least 1.4

The oc client referenced in the lab is v1.3, which is sufficient for the current lab materials, but ideally want to at least match it up with the current OCP 3.4 runtime?
https://github.com/openshift/origin/releases/tag/v1.4.1

Already merged by #42.
It supports 1.3, 1.4 and 1.5 using ocp and origin.


Bug: billing address is used for tax calculation for Click&Collect orders

What are you trying to achieve?
Create an order via checkout that uses C&C as the delivery method and has correctly calculated taxes.
Steps to reproduce the problem

Prepare tax configuration: enable flat rates in a channel and add default tax rates for two countries, A and B (for easier reproduction, choose two with different tax rates).
Configure Click & Collect: prepare a warehouse with the "Pickup" option enabled (e.g. set to "Local stock"), add it to the channel, add stock for a product in this warehouse. For the warehouse, configure an address to use one of the two countries, e.g. country A.
Create a checkout with a product and set only the billing address; use country B (Australia in this case). Example:

mutation checkoutCreate {
  checkoutCreate(
    input: {
      email<EMAIL_ADDRESS>      channel: "default-channel",
      lines: [
        { variantId: "UHJvZHVjdFZhcmlhbnQ6Mzg0", quantity: 1 },
      ],
      billingAddress: {
        firstName: "Test",
        lastName: "Test",
        streetAddress1: "Test",
        city: "Test",
        country: AU,
        countryArea: "VIC",
        postalCode: "3189"
      }
    }
  ) {
    checkout {
      id
      availableCollectionPoints {
        id
        name
      }
      totalPrice {
        gross {
          amount
        }
        net {
          amount
        }
      }
    }
  }
}

At this point billing address country is used for tax calculation.

Assign the collection point to a warehouse that is in country A:

mutation checkoutAddShipping {
  checkoutDeliveryMethodUpdate(
    id: "Q2hlY2tvdXQ6YmNmMWI0ODItZTM3MS00M2I3LWFjY2EtYmViZGRlMDczZDgy"
    deliveryMethodId: "V2FyZWhvdXNlOjA0NTAwNTlmLWMzMzQtNGUxMi05YWU5LWQ2YmU3NzQ5OTY5YQ=="
  ) {
    checkout {
      shippingAddress {
        country {
          code
        }
      }
      billingAddress {
        country {
          code
        }
      }
      deliveryMethod {
        ... on Warehouse {
          address {
            country {
              code
            }
          }
        }
      }
      totalPrice {
        net {
          amount
        }
        gross {
          amount
        }
      }
    }
  }
}

Response:
{
  "data": {
    "checkoutDeliveryMethodUpdate": {
      "checkout": {
        "shippingAddress": null,
        "billingAddress": {
          "country": {
            "code": "AU"
          }
        },
        "deliveryMethod": {
          "address": {
            "country": {
              "code": "PL"
            }
          }
        },
        "totalPrice": {
          "net": {
            "amount": 1.99
          },
          "gross": {
            "amount": 2.19
          }
        }
      }
    }
  }
}

Check the total price - tax is 10%, which is the rate for AU, but the pickup point is in PL, where 23% should be applied. The shipping address is not set, and the billing address is used for tax calculation, which is incorrect. Instead, the C&C pickup address should be used.
What did you expect to happen?
To fix the issue, we can do the following:

When checkoutDeliveryMethodUpdate is used to set C&C delivery, override the checkout's shipping address with the warehouse address. Make sure that price recalculation is triggered afterwards, and the shipping address is used for tax calculations. Update the mutation's description to mention that behavior.
In all mutations where the shipping address can be set, add validation to raise an error when the user is trying to override the shipping address if C&C is selected. Update descriptions accordingly.
Update the changelog to describe behavior changes (in PR to the main branch).

Edge cases:

Changing the address of a warehouse that is used as a collection point should update all checkouts where this address was set as shipping address.

Logs
Context: https://saleorcommerce.slack.com/archives/C0373831YBC/p1704467414278619
Environment
Saleor version: 3.14.47

@maarcingebala in terms of edge case with  Warehouse.address update - Checkout.shipping_address will point to the same Address instance in database - is there any action to take? Or would you happen to think of a case where Warehouse gets a new Address instance?
I found one mutation which can update the shipping_address CheckoutShippingAddressUpdate


Disable "Add subscribers" UI when user doesn't have permissions

When a user doesn't have permissions to add others to streams, we should disable the UI for adding users, rather than giving an error after the user attempts to do so.

Where: Stream creation UI and stream subscribers UI
What:

Disable "Add subscribers" input text box
Disable "Add" button
Show tooltip on both: "You do not have permission to add other users to streams in this organization."

The user will still be able to subscribe themselves to a stream via the "Subscribe" button.
Note that  the current error message when a user comes across this restriction when creating a stream is quite unhelpful:

@zulipbot claim

hello, can i be assigned this issues

@zulipbot claim

@alya  hey i have a question since that  part for  stream creation  has this  design of stream settings

should i change the ui to the above design then  add the disable feature

I'm not sure what you mean; your screenshot doesn't show the stream creation UI. This issue requests a specific change, and not any other changes to the stream UI design.

@joanne-cmd  please take a look at the Zulip contributor guide, and keep in mind the following guideline:

Before you claim an issue, you should be confident that you will be able to tackle it effectively.

I will go ahead and unassign this issue, and you should feel free to re-claim it once you have figured out how to approach it (or pick a different one if you prefer).
You're welcome to ask a specific question about how to approach this issue (see https://zulip.readthedocs.io/en/latest/contributing/asking-great-questions.html) in the Zulip development community.
Thanks!

@alya hey,  i am just curious  why did you unassign me the issues

i know how to tackle it  is just  the way  the current ui looks different and i was asking for clarification

@joanne-cmd I unassigned the issue because it seemed that you were uncertain what was being asked. Please feel free to claim it again if you know how to proceed now.

@zulipbot claim

@alya this issue seems to be already rectified by the 24900 so i guess you can close this issue which might help new contributors like me for assigning again

this issue seems to be already rectified by the https://github.com/zulip/zulip/pull/24900

Ah, yeah, that looks right -- thanks!


[Bug]: Overlay issue in new dashboard block

Describe the bug
When you scroll in https://ui.shadcn.com/blocks#dashboard-01 to the bottom of the mobile view, you can see that the Customer - Amount table is going over the header of the page.
Fix: Remove the class relative from the element with the following classes: relative w-full overflow-auto. It's the parent of the table tag element.
Affected component/components
dashboard-01 block mobile view
How to reproduce

Go to https://ui.shadcn.com/blocks#dashboard-01
Activate mobile view.
Scroll to the bottom

Codesandbox/StackBlitz link
No response
Logs
No response
System Info
Google Chrome browser newest version

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

I saw that you added z-10 instead of removing the relative class. Why is the relative class needed there?

I saw that you added z-10 instead of removing the relative class. Why is the relative class needed there?

The relative is being added to the table component because I didn't want to risk some breaking changes. The bug is more about the index of the stick header than the table. This also affects the avatars in the Recent Sales section.

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


Update the README to include info for enabling new Source-related features

This PR updates the README to include:

An updated example that includes a checkout step for git deploys (to enable sending commit metadata)
Documentation for the docker_repository_url parameter.

I wonder if we should spend a little bit experimenting with your idea of a composite GHA to get around this issue.

Agreed. I'll take a few minutes today and see if I can make it work.


Adding deployment requirements

Add generic deployment
Include deployment via HTTPS

Addresses issue #30

Reviewed the changes and at the very least I can say this shouldn't introduce any breaks as it is mostly new code. Will give @hanwenzhang123 the chance to review in case she is able to spot anything I am not.


Multiple issues: Pinned folders & Forced shutdown

Description
Hello,
I noticed that the files app presents the following issues regularly:

The pinned folders disappear completely
When trying to open more than 1 file, such as 2 pdf files, the app crashes

Steps To Reproduce

Pinned folders disappear unexpectedly during app use
App crashes when trying to open more than 1 file, such as 2 pdf files, the app crashes

Requirements

e.g. Fix crash that occurs when...
Fix crash that occurs when trying to open more than 1 file, such as 2 pdf files, the app crashes.
Fix pinned folders disappearance

Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.4169
User ID
files-community
Log File
debug.log

Hi Josh,
It seems after using the work round option, both issues are resolved for
now.
Thank you for your prompt response and help.
Best regards,
Aristotelis
On Wed, Sep 25, 2024 at 12:08 PM Josh65-2201 @.***>
wrote:

Thanks for the report, please open separate issues for different feedback.
Pinned folders disappear unexpectedly during app use
We're tracking this in #15410
https://github.com/files-community/Files/issues/15410. You can check
there or click subscribe in the right sidebar for updates. Please use the
workaround there
App crashes when trying to open more than 1 file, such as 2 pdf files, the
app crashes
Do you have the preview pane open?
—
Reply to this email directly, view it on GitHub
https://github.com/files-community/Files/issues/16258#issuecomment-2373504365,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BLSGAQFDUFIE3M4AHMU64FLZYJ4QXAVCNFSM6AAAAABOZ3XEWGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZTGUYDIMZWGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


feat-add theme toggle button

closes #291
Changes proposed
Added a reusable toggle button component that toggles between light and dark mode.
Check List (Check all the applicable boxes)

[x] My code follows the code style of this project
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

@biratdatta


Use page title instead of site title on homepage

Hey there,
using this option, I can use the site title as homepage title:
'diesdasdigital.meta-knight' => [
    'siteTitleAsHomePageTitle' => false,
],

.. BUT more often than not, the site title will be different from the single title I want to use for my homepage, eg I want to have a nice but short site title, so page title and site title aren't too long when used together, but on home (= index site), I wouldn't want to use this short-by-nature site title, but a nice long custom title.
I therefore suggest using the page title for more customisability instead of site title on homepage.

Ah well, I accidentally pushed another commit, so they get mixed up - hope that's not a problem .. I could however commit them separately, if you insist ..

Hi @S1SYPHOS, thanks for raising this issue and the corresponding PR. I completely agree – the homepage title should be more configurable. I will merge your PR soon. It will be part of version 1.2.

Hey @jonathanmuth,
I thought about adding flexible og:type values, too and fields based on that, eg og:type=book would require og:type=book:author|isbn etc - what do you think?

That is an interesting idea. I wanted to keep the plugin simple in the beginning – but I can see this functionality being useful to some users. Maybe it could be a power user feature enabled via a config option.
Would you implement this using a structure field?

Well, from my experience so far, I would want to use existing fields to provide the data for these - let me think of something ;)
I'll get back to you!


Invalid application ID and session ID when reporting traces

Hello. In our app, we are reporting SQL operations traces to Datadog. While it is showing up in the dashboard, the application id and session id seems to be always invalid?

We also have RUM configured, but I am not able to see the SQL operations in the RUM events as well. I am guessing this is the reason?
Additional context

Android OS version: 12
Device Model: Pixel 3
Datadog SDK version: 1.11.1

Hi @msasikanth , and thanks for reporting this issue. Are you creating your AndroidTracer before or after initializing the DatadogSDK? If the tracer is initialized before, it can't link itself to the RUM session and set those attributes properly so this might be the cause. Otherwise, could you share with us the code snippet you use to

initialize the SDK
initialize the Android Tracer
generate the spans for your SQL calls ?

Oh good your app's open source. That'll make things easier, i'll take a look right now

I did a test run and it seems that on my side the RUM information is correctly set, and they do appear in the related RUM View. I tried this in debug mode, and from your code it seems it should be there. Can you reproduce this issue when running a debug build too?

I have tried it on both debug and release builds. Are you able to see the SQL queries traces in RUM dashboard? 🤔 . I can see UI actions, network responses and other session data. But not the SQL tracing information. Is it possible to see something like that with the SDK currently?
For example, would it be possible to see the SQL instrumentation data in the session list along with other UI and screen information?

You won't see them directly in the RUM Dashboard side, but they should appear in a RUM View, in the traces Tab:

Another option for you might be to track SQL queries as RUM Resources. Those will appear the same way as network requests.
class DatadogSqlPerformanceReportingSink : ReportSink {

  override fun begin(operation: SqlOperation) {

    GlobalRum.get().startResource(
        key = "${operation.daoName}.${operation.methodName}.${operation.startTimeMillis}",
        method = "POST", // maybe have a mapper to tag methods as POST/GET/DELETE,
        url = "sql://${operation.daoName}/${operation.methodName}",
        attributes = mapOf(
            "dao" to operation.daoName,
            "method" to operation.methodName
        )
    )
  }

  override fun end(operation: SqlOperation) {
    GlobalRum.get().stopResource(
        key = "${operation.daoName}.${operation.methodName}.${operation.startTimeMillis}",
        statusCode = 200, // maybe use a 400/500 if the query fails,
        size = null,
        kind = RumResourceKind.OTHER,
        attributes = emptyMap()
    )
  }
}

Got it, thanks a lot. I am gonna try out reporting it as a RUM resource. For some reason, I didn't see traces in sessions when using 1.10.0. I have updated to the latest version, gonna see if it works in prod. We can close this issue, for now.

Thanks, let us know if you have any other question or blocking points.


[Site-Recovery] Release for a2a

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Site-Recovery


Create 0015-3sum.dart

File(s) Modified: 0015-3sum.dart
Language(s) Used: dart
Submission URL: _https://leetcode.com/problems/3sum/submissions/1234403676/_

Important
Please make sure the file name is lowercase and a duplicate file does not already exist before merging.

@jajpa Thank you for your contributions 👍


Player dosen't work on iphone

Hi, So im working on loading songs from firebase and make some modification on pitch and playbackrate .
my code work nicly on desktop but when i open my app on my iphone  Tone.Player doesn't play the song
here is my code

link to my app : https://musicplayer-4070.web.app/
ps: you need to click on the name of the song

thx @dirkk0 , now it's works good on android phone but in ios even if the user click to play a song AudioContext still on suspended mode

iOS can be very picky.
    if (Tone.context.state !== 'running') {
        Tone.context.resume();
        Tone.start()
    }

Try this directly before your .start() method.

i opened it on private mode and still in the same problem

Then it might be a different problem on iOS. What's the output of the console?

thanks @dirkk0 , it's works now after i added your code and restarted the server


Rename Animation TYPE_TRANSFORM to TYPE_TRANSFORM3D

Follow-up to #38430. I noticed this while rebasing another PR on top of #38430's merge commit.
I'm not against having 2D transform animations or anything, but the current logic is hard-coded to work with 3D, ex:
if (adding_track_type == Animation::TYPE_TRANSFORM && !node->is_class("Node3D")) {
	EditorNode::get_singleton()->show_warning(TTR("Transform tracks only apply to 3D-based nodes."));
	return;
}

@fire Which tests, this one? It didn't work out of the box on the current master due to the color constants, but after fixing it, I get the same results with this PR as I do with the current master.

Thanks!


sg.ac.th

@BlueLynxUs I merge the request, but it is a duplicate to #14606. It is already satisfied.


Cost Calculator

Is there a way to know the cost of building a building?

I dont't understand why but I dont't have access to the Objs variable... It's very strange because it should be works

Thx !


Fix unnecessary delay in setting up microk8s

This change prevents the can-i command from waiting until a return code of 1. A success is return code 0, and the current behavior is to retry 12 times with a delay of 10 seconds, wasting 2 minutes of runner time for no reason.

@ghislainbourgeois Oops, didn't notice you also tackled this. The problem I discovered in #59 is that this check is only relevant when RBAC is enabled. When addons were made configurable, this case was introduced.

Closing as #59 is a more complete fix


frontend/coin-control: add address reuse warning

Add a warning to the coin control modal when a user selects one or more UTXOs from reused addresses. The warning: "One or more UTXOs have a reused address. Be aware  the receiver will see all UTXOs associated with those addresses."
The UTXOs now have red badge when their address was re-used.

@Beerosagos @thisconnect

fixed the repeated warning for each UTXO's script type
the warning is shown only when the user selects the UTXO from the reused address
I changed the warning from Status to Message so it is not all red
added a badge to all UTXOs from re-used addresses
CHANGELOG updated

PTAL

Nit: maybe we can vertically center better the badge?

I'll take a look, ideally badge would be nice and inline with text. Not sure yet what causes this.

I'll take a look, ideally badge would be nice and inline with text. Not sure yet what causes this.

tl;dr it was a combination of inconsitent line-heights (parent element) and vertical centering.
branch with fix
https://github.com/thisconnect/bitbox-wallet-app/tree/frontend-fix-badge-centering
diff
https://github.com/BitBoxSwiss/bitbox-wallet-app/compare/master...thisconnect:frontend-fix-badge-centering?expand=1
@strmci @Beerosagos let me know if you want to merge this PR first and if I should open a followup PR or feel free to chery-pick my commit into this change.

@thisconnect
I cherry-picked your Badge improvement, thanks.


Checker and Error Classification

Modified the Checker to check for transition loop.
Check for using Variables only if declared earlier.
The errors are classified into 2 categories: "errors", "warnings".

The checker in this repo should be focused on specific DSL properties and ensuring that the DSL conforms to specification.
All other checks that are JB Specific need to move to the JB Engine repo

I am creating new issue for splitting the checker between Jugalbandi Studio and PwR-NL2DSL.
We can merge this for now


[edpm_deploy_instance]Wait for the instance to become ACTIVE

/override rdoproject.org/github-check


Highlight certain days

How do I add a class to certain dates?

@budlink
Sorry, currently no

@DanielYKPan Can you add this please?

Weekend highlight is a thing i miss in this component. Any chance to introduce it in the next release?

How to highlight certain dates where some events are scheduled, any support for this or workaround please ?
Thanks.

You could highlight certain day, weekday, month or year via your own css file.
Year: each year cell has a class owl-dt-year- + Year Number. Eg for 2020 cell, it has a class owl-dt-year-2020
Month: each month cell has a class owl-dt-month- + Month Number. Month Number is from 0 (January) to 11 (December). Eg for May, it has a class owl-dt-month-4
Weekday: each day cell has a class owl-dt-day- + WeekDay Number. WeekDay Number is from 0 (Sunday) to 6 (Saturday). Eg for Monday, it has a class owl-dt-day-1.
Day: each day has aria-label value. Eg for day May 12, 2018, it has a aria-label value aria-label="May 12, 2018". You could set this day's css via selector .owl-dt-calendar-cell[aria-label="May 12, 2018"]

Hi Daniel,
Thanks a lot for your solution. I will do that.
In case if you have a similar GIT repository for Calendar showing some
highlighted dates in dashboard, please share the link.
Thanks again,
Kishor
On Mon, May 28, 2018, 7:06 PM Yaokun Pan<EMAIL_ADDRESS>wrote:

You could highlight certain day, weekday, month or year via your own css
file.
Year: each year cell has a class owl-dt-year- + Year Number. Eg for 2020
cell, it has a class owl-dt-year-2020
Month: each month cell has a class owl-dt-month- + Month Number. Month
Number is from 0 (January) to 11 (December). Eg for May, it has a class
owl-dt-month-4
Weekday: each day cell has a class owl-dt-day- + WeekDay Number. WeekDay
Number is from 0 (Sunday) to 6 (Saturday). Eg for Monday, it has a class
owl-dt-day-1.
Day: each day has aria-label value. Eg for day May 12, 2018, it has a
aria-label value aria-label="May 12, 2018". You could set this day's css
via selector .owl-dt-calendar-cell[aria-label="May 12, 2018"]
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/DanielYKPan/date-time-picker/issues/83#issuecomment-392528935,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQIaZfCqIjfXyjs9deqFXbjHSuKywzHvks5t2_1kgaJpZM4OU3JF
.


Bug 2044940: Jenkins Fixes for CVE-2022-20617 and CVE-2022-20612

Upgrade core Jenkins and plugins to address some of the vulnerabilities in the 2022-01 Jenkins security advisory:

CVE-2022-20617: Update docker-commons to 1.18 to mitigate a vulnerability with unsanitized image names/tags.
CVE-2022-20612: Upgrade core Jenkins to the current LTS release (2.319.2). This includes a fix for a CSRF vulnerability impacting 2.319.1 and lower.

Related Bugzilla bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=2044942
https://bugzilla.redhat.com/show_bug.cgi?id=2044846

/assign @coreydaley
For backport risk assessment

/label backport-risk-assessed

/retest
Unrelated AWS error

/lgtm

/bugzilla refresh

@jitendar-singh @prietyc123 this is now ready for cherrypick approval.

ART requests:

Jenkins Plugins: https://issues.redhat.com/browse/ART-3718
Jenkins Core: https://issues.redhat.com/browse/ART-3719


Add addin to website to allow for automatic documentation

i.e. get your addin listed here:
http://cakebuild.net/addins
Please let me know if you have any questions about how to do this.

@MihaMarkic just wanted to touch base to see if you had considered doing this?  Thanks!


Morning

Describe the bug
A clear and concise description of what the bug is. If possible, please attach (upload, do not copy/paste) the log file that can be found at %LocalAppData%\OpenXR-Toolkit\logs.
Environment

Application/Game and its version
Make and model of VR device
Version of the OpenXR runtime
Make and model of GPU

To Reproduce
Steps to reproduce the behavior:

Run '....' game
Enable option '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem. Use the Companion app to enable screenshots, then use Ctrl+F12 to capture. The file is placed in %LocalAppData%\OpenXR-Toolkit\screenshots.

Morning I have purchased a HP Reverb G2 headset and when I start the OpenXR Toolkit i cannot find FFR in menu to change my inner and outer ring settings. There is no option in the menus. I have uninstalled and reinstalled your software but still no success. Can you assist? Regards Eddie

The bug template asked you to fill out info such as the model of your GPU.
I'm guessing your GPU just doesn't support it.

OS Name	Microsoft Windows 10 Home
Version	10.0.19044 Build 19044
Other OS Description 	Not Available
OS Manufacturer	Microsoft Corporation
System Name	DESKTOP-1H00LGG
System Manufacturer	To Be Filled By O.E.M.
System Model	To Be Filled By O.E.M.
System Type	x64-based PC
System SKU	To Be Filled By O.E.M.
Processor	Intel(R) Core(TM) i5-7600 CPU @ 3.50GHz, 3501 Mhz, 4 Core(s), 4 Logical Processor(s)
BIOS Version/Date	American Megatrends Inc. P7.30, 28/11/2016
SMBIOS Version	2.8
Embedded Controller Version	255.255
BIOS Mode	UEFI
BaseBoard Manufacturer	ASRock
BaseBoard Product	Z170 Pro4
BaseBoard Version
Platform Role	Desktop
Secure Boot State	Off
PCR7 Configuration	Binding Not Possible
Windows Directory	C:\WINDOWS
System Directory	C:\WINDOWS\system32
Boot Device	\Device\HarddiskVolume2
Locale	United Kingdom
Hardware Abstraction Layer	Version = "10.0.19041.1566"
User Name	DESKTOP-1H00LGG\Eddie
Time Zone	GMT Daylight Time
Installed Physical Memory (RAM)	16.0 GB
Total Physical Memory	15.9 GB
Available Physical Memory	12.4 GB
Total Virtual Memory	25.9 GB
Available Virtual Memory	20.8 GB
Page File Space	10.0 GB
Page File	C:\pagefile.sys
Kernel DMA Protection	Off
Virtualization-based security	Not enabled
Device Encryption Support	Reasons for failed automatic device encryption: TPM is not usable, PCR7 binding is not supported, Hardware Security Test Interface failed and device is not Modern Standby, Un-allowed DMA capable bus/device(s) detected, TPM is not usable
Hyper-V - VM Monitor Mode Extensions	Yes
Hyper-V - Second Level Address Translation Extensions	Yes
Hyper-V - Virtualization Enabled in Firmware	Yes
Hyper-V - Data Execution Protection	Yes

It's the model of your graphics card that we need here.

As stated on our website, any graphics card older than than the GTX 16xx and RTX20xx series do not have the required hardware to do FFR.

Ok thank you for your help really appreciate it. Have a good day.


Implement end game conditions

Story
User story

As a user, I want to be able to win or tie or loose a game and be notified about it.
Tasks

Implement an "End game" / Surrender button #42
Display if a player has won/tied/lost a game according to the board #60
Client determines if the game has reached an end #61
Backend verifies check mate #49
Backend verifies stale mate #73

Acceptance criteria

[ ]  You can surrender and will subsequently looses the game
[ ]  Checkmate wins the game
[ ]  Remis is a remis
[ ]  The participants are notified of their win or loss

Notes

Should we add the 50 move rule? If I understood it correctly, we need another counter that counts moves since the last capture. If it exceeds 100, the game is drawn. Source

Would be more efficient to store the move, where last capture took place.


Build issue

Please add
target_include_directories(${PROJECT_NAME} PRIVATE ${GDAL_INCLUDE_DIR})

to the appropriate place in the cmake file; otherwise <gdal_priv.h> doesn't get found.

Thanks. Done!

Thanks!


Bug 2124565: Deleted datasource reappear in the ui

Signed-off-by: Matan Schatzman<EMAIL_ADDRESS>
📝 Description
When deleting Datasource, dataimportcron is deleted, but after, so it causes the datasource to reappear in the UI.
The change now will delete first the dataimportcron and then the datasource
🎥 Demo

Please add a video or an image of the behavior/changes

/bugzilla refresh


Add a "How to Contribute" section to opentelemetry.io

Desired feature or idea: There's a lot of information in the OTel repos explaining how to contribute to different areas of OpenTelemetry; however, we don't have a centralized location to document this. Adding a "contributing" tab off of opentelemetry.io could serve as a "one stop shop" for folks who want to know of the different ways in which they can contribute to the project.
Additional context: After speaking with @svrnm, he suggested that I create a blog post on the OTel blog based on this Medium article that I published recently, as a stop-gap, until we can put together a such a page.

Not sure if there should be a dedicated "Contributing" tab, or if this should live under "Community", as we have a simple version of that already there: https://opentelemetry.io/community/#develop-and-contribute, but I overall agree that it would be great to have that "one stop shop"
I think there are a few things we need to distinguish:

The "rules" to contribute, which are contained in the CONTRIBUTING.md of each project, by rules I mean things like "use npm this and that way" or "your commits should have this and that structure"
The "formal" set of things how you can be part of the community, which live at https://github.com/open-telemetry/community/
Some "informal" recommendations (like your blog post), on how to get started with contributing, especially aimed towards new comers

I think we can add the most value by enriching the website with more of the "informal" stuff where we provide people with some soft suggestions on how they can get started. So that would be the point for me to begin with.
As an additional reference, I'll leave https://www.kubernetes.dev/ here, as "[t]his site is the home of contributor documentation, and serves as a starting point for your journey as a Kubernetes contributor."

Hi all. Not sure if you consider this a duplicate or simply a related issue of the following issues, but I wanted to mention and cross-link them since this topic isn't new:

#897
#1890


Package.py is not throw error.

Hi there,
I've tried the Pynguin to generated the test cases for Package.py
And there is one test case that mark as expected fail by Pynguin.
import pytest
import package as module_0
import dataclasses as module_1
@pytest.mark.xfail(strict=True)
def test_case_0():
    module_0.Package(123, [], 123)

And after tested with Pytest, it is throw the error that this case was mark as XPASS(Unexpected pass)
So I checked the Package.py function and found out that the there's no raise type error in there, that's why the Pynguin expected the test cases as failed.
from dataclasses import dataclass, field
from typing import List

@dataclass
class Package:
    '''A folder or a python module'''
    name: str
    children: List['Package'] = field(default_factory=list)
    items_number: int = 0
    # def __post_init__(self):
    #     if not isinstance(self.name, str):
    #         raise TypeError("Name must be a string.")
    # suggestion #

Thank you.

I don't understand what you are trying to achieve. I am not going to change some code that works so that the unit test you generated could work as you expect.
If I wanted to enforce type validation of the attributes of an instance of Package, I would have used tools like pydantic to avoid writing all the __post_init__ methods that you suggest.
Can you explain what your intentions were?


Lazy loading for images

Feature request type
Enhance component
Component name
MudImage, MudAvatar
Is your feature request related to a problem?
No response
Describe the solution you'd like
For better performance (for image gallery for instance) html support lazy loading.
Add support for loading attribute for MudImage and MudAvatar components
Have you seen this feature anywhere else?
No response
Describe alternatives you've considered
No response
Pull Request

[X] I would like to do a Pull Request

Code of Conduct

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

Please use attribute splatting to achieve this:
<MudImage Src="images/mony.jpg" Alt="Mony the dog" Elevation="25" Class="rounded-lg" loading="lazy" />


Fix code scanning alert - Sensitive data read from GET request

Tracking issue for:

[x] https://github.com/IntegerAlex/muzix-backend/security/code-scanning/1

resolved issue


execute multiple actions by register in rasa

Rasa Core version:
0.11.6
Python version:
3.5
Operating system (windows, osx, ...):
ubuntu 16.04
Issue:
hi,i have code multi actions class to execute multi command,but i found some solution in website only can input one parameter to register a action.please help me ! thanks
example code in sdk:
def endpoint_app(cors_origins=None,
                 action_package_name=None
                 ):
    app = Flask(__name__)

    if not cors_origins:
        cors_origins = []

    executor = ActionExecutor()
    executor.register_package(action_package_name)

    CORS(app, resources={r"/*": {"origins": cors_origins}})

    @app.route("/health",
               methods=['GET', 'OPTIONS'])
    @cross_origin(origins=cors_origins)
    def health():
        """Ping endpoint to check if the server is running and well."""
        return jsonify({"status": "ok"})

    @app.route("/webhook",
               methods=['POST', 'OPTIONS'])
    @cross_origin()
    def webhook():
        """Webhook to retrieve action calls."""
        action_call = request.json
        response = executor.run(action_call)

        return jsonify(response)

    return app

you specify an action package, meaning this can be a python file which contains as many actions as you want, or a folder containing as many action files as you want.
I'm gonna close this now as it's more of a usage question, if you have further questions, please post them on our forum


Use new releases for CAPA cluster templates

What does this PR do?
Towards https://github.com/giantswarm/roadmap/issues/3466

CAPA only change for new releases: render release version in config instead of cluster-aws version in App resource.

What is the effect of this change to users?
Users must specify --release flag when rendering templates for new CAPA clusters.
What does it look like?
See --release flag and .global.release.version config set, while App .spec.version is not set.
kubectl gs template cluster --provider capa --name nikola --organization giantswarm --release 25.0.0
---
apiVersion: v1
data:
  values: |
    global:
      connectivity:
        availabilityZoneUsageLimit: 3
        network: {}
        topology: {}
      controlPlane: {}
      metadata:
        name: nikola
        organization: giantswarm
        preventDeletion: false
      nodePools:
        nodepool0:
          instanceType: m5.xlarge
          maxSize: 10
          minSize: 3
          rootVolumeSizeGB: 8
      providerSpecific: {}
      release:
        version: 25.0.0
kind: ConfigMap
metadata:
  creationTimestamp: null
  labels:
    giantswarm.io/cluster: nikola
  name: nikola-userconfig
  namespace: org-giantswarm
---
apiVersion: application.giantswarm.io/v1alpha1
kind: App
metadata:
  labels:
    app-operator.giantswarm.io/version: 0.0.0
  name: nikola
  namespace: org-giantswarm
spec:
  catalog: cluster
  config:
    configMap:
      name: ""
      namespace: ""
    secret:
      name: ""
      namespace: ""
  kubeConfig:
    context:
      name: ""
    inCluster: true
    secret:
      name: ""
      namespace: ""
  name: cluster-aws
  namespace: org-giantswarm
  userConfig:
    configMap:
      name: nikola-userconfig
      namespace: org-giantswarm
  version: ""

Any background context you can provide?

https://github.com/giantswarm/roadmap/issues/3466
https://github.com/giantswarm/rfc/pull/96

What is needed from the reviewers?
Check if I missed to change something.
Do the docs need to be updated?
I don't think so.
Should this change be mentioned in the release notes?

[x] CHANGELOG.md has been updated

Is this a breaking change?
Maybe, as --release flag is needed for CAPA clusters now, but's a change for CAPA clusters, and I don't think it's a change for kubectl-gs.

Will fix the tests.


ECO-1016 CLValue fromBytes on all CLValue Data Types

@zie1ony all of the changes are included in #32


Working with vuejs/nuxt

Hello everyone 👋
I am very interested in this plugin for my open-source project SCT.
I want to use it for the frontend, which uses Nuxt.Js 2, but eslint does nothing.
Here is an example using the string "hey guys" as a test:

The file index.vue
The console output using eslint --ext \".js,.vue\" --ignore-path .gitignore .

Hey, sorry for the late reply.
I don't really understand why your plugin doesn't look in strings 🤔.
Does it mean that the plugin is only useful for developers who want more inclusive code?
But it doesn't impact the result for the client at all?

As mentioned before, you can lint for strings by enabling a custom configuration.
And I also mentioned that this plugin validates JavaScript code (because ESlint is a JavaScript linter). For other file types you've to investigate the usage of a custom parser for ESlint, like vue-eslint-parser.

hummm okay, thank you for your precision.
Sorry for the inconvenience


use crosstool-ng & regenerate

@orgua I have been thinking about regenerating the toolchain we've been using to see if some of the errors (here  and there) that seem internal to the compiler will reduce.
I've little experience with crosstool-ng.
Do I try it?
I'm asking before hand because it takes a while to generate the toolchain.
What do you think?

Just for reference: you are talking about the c-code for the PRUs on the beaglebone.
We are currently using two different cross-compilers (TI's CGT and GCC with PRU-Extension) for the PRU-Code. One compiler (cgt) has trouble calculating correct results with u64, the other currently produces non-optimal code and therefore overflows the program memory. Additionally we use a basic makefile, comparable to a bash-script, to feed the compiler. That's our current "chain of tools".
If you know a way to get rid of these constraints go for it. It should be a good exercise to learn more about embedded software development.

ok, closing this. CGT works for now and GCC saw big improvements lateley


disable broken Windows e2e-with-binary

Didn't catch it earlier because it's only run on push
Release Note
NONE

for windows we should setup a powershell step. I will check that out
@dekkagaijin @mattmoor

ah you already did :) https://github.com/sigstore/cosign/pull/1177/files. you are fast 🎉


Is there a way to get old version of multi ogar edited?

Is there a way to get old version of multi ogar edited? if you know comment down Thanks!

i need the version 1.4.0 does anyone have?

@Gigabyte918 Download it here

Thanks Thank you very much love you all now i can able to play on my mobile thank you!

You can Close this issue i am too lazy to click button :v

the server is working but when i edit the gamerserver.ini the server isnt working could you give real not edited version on MultiOgar 1.4.1

@Gigabyte918 You probably changed the server port by mistake, reinstall it and try again.

could you change this code to make player loss mass but doesnt split

GameServer.prototype.autoSplit = function (client, cell1) {
// rec mode
if (!client.rec) {
var maxSize = this.config.playerMaxSize;
} else {
maxSize = this.config.playerMaxSize * 3;
}
// check size limit
var checkSize = !client.mergeOverride || client.cells.length == 1;
if (checkSize && cell1._size > maxSize && cell1.getAge(this.tickCounter) >= 15) {
    if (client.cells.length >= this.config.playerMaxCells) {
        // cannot split => just limit
        cell1.setSize(maxSize);
    } else {
        // split
        var maxSplit = this.config.playerMaxCells - client.cells.length,
        maxMass = maxSize * maxSize,
        count = (cell1._sizeSquared / maxMass) >> 0,
        count = Math.min(count, maxSplit),
        splitSize = cell1._size / Math.sqrt(count + 1),
        splitMass = splitSize * splitSize / 100,
        angle = Math.random() * 2 * Math.PI,
        step = 2 * Math.PI / count;
        for (var k = 0; k < count; k++) {
            this.splitPlayerCell(client, cell1, angle, splitMass);
            angle += step;
        }
    }
}

};

atleast give answer to my Question!

At least use the latest version of the project.

You cant able to understand why i want this!

after getting the code i can able release the mobile private server

Your install was bad for one of the modules, I can't really help with that.

Thanks I got it to working soon i will release the mobile blob server!


nice lib, can I add subtitle files into playlist?

i want to add subtitle to per video, it supported?

Hi @tronghieu60s, thanks for filing an issue :)
I'm not against this in theory but the only information I can find about including subtitles in a playlist file is for VLC and for HLS manifests. VLC only supports looking up local files on your machine, and I'm not interested in supporting HLS manifests with this library.
VLC
#EXTVLCOPT:sub-file=/only/supports/local/paths.srt

HLS manifest
#EXT-X-MEDIA:TYPE=SUBTITLES,GROUP-ID="subs",NAME="English",DEFAULT=NO,AUTOSELECT=YES,FORCED=NO,LANGUAGE="en",CHARACTERISTICS="public.accessibility.transcribes-spoken-dialog",URI="subtitles_en.m3u8"

If you have information of a standard way to do this I could invesitgate adding support, but I can't find anything when searching myself.


After entering a 'Tab' when entering a field, the cursor does not stay in the next field

Conder an Interface with (predominantly) ALPHANUMERIC fields. The idea is that when I enter a value in such a field and hit the 'Tab' key, the cursor will move to the next field. It seems that

this actually happens, but
the cursor does not stay there,
the result being that when I continue to type characters, they go nowhere (where I would have expected them to be filling in the next (ALPHANUMERIC) field)

Duplicate of https://github.com/AmpersandTarski/Ampersand/issues/993. Fixed in v1.10.2 (12 aug 2021)


issue with Run-EXE

Hi,
I am having issues with the example provided for run-exe with a meterpreter exe. I get an error about DEP compatibility and then it closes the powershell window. Am I doing something wrong?
https://github.com/PowerShellMafia/PowerSploit/issues/362 not sure if this is similar

Maybe it's also related to this: https://github.com/EmpireProject/Empire/issues/415
Run-Exe is just a wrapper for Invoke-ReflectivePEInjection after all. We could also expose the -ForceASLR flag, maybe that will help. No promises though.
However, if your goal is to run Meterpreter and your target system has access to web services on your machine, I'd recommend checking out the web_delivery module of Metasploit. Very easy to use, very powerful. I used to use this in combination with PowerHub's Clip-Exec feature before web_delivery caught up with their AMSI bypass.

Actually the flag -ForceASLR is already applied by default.
You don't say exactly, but the error is actually a warning from what I can tell. The exact phrasing is "WARNING: PE is not compatible with DEP, might cause issues", right? Since it's only a warning, I'm not sure this is an issue. The window closing can be a sign that the  anti virus  interfering. Or it could be meterpreter closing the window and the exe us actually run correctly. Either way, it could be lots of things going on.
Since I believe this is an issue of PowerSploit, I'm closing this.