Add emulated_roku component

Description:
Adds a component for emulating a Roku API, allowing Roku remotes to control Home Assistant.
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#6928
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

Hi @mindigmarton,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Awesome to finally see this as a PR!
First off; you'll gonna need to write tests, at least for the parts related to config entries.
Second; is it really any benefit to support the old configuration path?

Awesome to finally see this as a PR!
First off; you'll gonna need to write tests, at least for the parts related to config entries.
Second; is it really any benefit to support the old configuration path?

Yeah, no probs,
Is HA moving away from yaml config? I haven't seen a component that didn't offer yaml configuration.

If you want to support it it's fine. It was mainly a question.

Doesn't take much to support both. The current implementation doesn't import the yaml entry to config entries, which might be confusing as it won't appear in the UI, but importing doesn't make sense for a new component.

Well the emphasis on hass is to configure through gui, read the IQS https://developers.home-assistant.io/docs/en/integration_quality_scale_index.html
I'm removing the documentation for my components as to not invite people to use the configuration.yaml. On new components I don't add the support.
At least I think you should import them as config entries, different ways of configuring but one way to have it registered in HASS. Problem is that there is no easy way to disable the config entry if you later remove the configuration from configuration.yaml. You'd still need to go through the integrations page to remove it

For old components, importing makes sense as a migration step that makes the yaml entry obsolete. When configuring a new component, I wouldn't expect the config entry I just wrote to not work after starting HA once. I guess it's better to remove the yaml config support.

Updated the tests and removed the yaml configuration support.
pylint in Travis seem to have failed for some random reason (for other PRs too it looks like)

Could anyone check the changes? I'll update the docs if everything is OK.
Thanks!

I did a check a week ago with the intention to revisit to go into depth. Haven't had time to, but the quick browse looked fine.

@MartinHjelmare I think this is ready, thoughts?

Just retested everything, one of the commits which removed the default value from upnp_bind_multicast broke SSDP discovery so I've pushed a new version of the library to prevent that from happening.
Bumped the version to 0.1.6.
https://gitlab.com/mindig.marton/emulated_roku/commit/e42bb3c7d5ff4cfb94bd943bb781684249a0e9f1

Had a look in the library. There's a mix of async def and use of asyncio.coroutine decorator within the same module, and sometimes even in the same function.
https://gitlab.com/mindig.marton/emulated_roku/blob/master/emulated_roku/__init__.py#L179

I've been refactoring the library, I've fixed couple of other issues too.
It should be noted that the API is without authentication. I've limited access to local networks and only by advertised ip, so DNS rebinding should not work.
An IP whitelist config option could be added for people with static IPs for their Harmony Hubs, or
using a proxy (bind to localhost, advertise the proxy address) with a whitelist could also work and would probably be more flexible / safe. (I'm not sure which is preferred)
Any thoughts?

Sorry, I'm not a network expert.
Make sure the library is compatible with Python 3.5+. This syntax looks like Python 3.6+:
https://gitlab.com/mindig.marton/emulated_roku/blob/e438f913baa8483d94dc5960bfc667ce4f115943/emulated_roku/__init__.py#L146
There's a merge conflict now.

Python syntax versions are starting to get confusing, looks like that's PEP 526, thanks. I'll update the MR when the library is final, I've changed a lot.
I'd also like to keep the config simple, so I rather avoid adding new config options especially related to networking, as it's much easier misconfigure when it's all over the place.

I've finished refactoring the library. Should support reverse proxies now, if anyone wants to use IP whitelists.
Also, I've refactored the stop / start methods so it cleans up properly in case of errors.


added dotenv

In the Gitignore file we specified that the .env file should not be pushed in git...kindly remove that and then try to push your branch once again..., OR send me .env file locally so that I can merge it into main.


Mod API proxies

mostly just to showcase how this works.
Kokoro (the dependency mod) implements IApiProviderManifest and overrides the GetApi method. any public methods available on the object returned by that method will be usable by other mods. GetApi also gets a default implementation of return this, so if you're lazy you can just put public methods on your manifest class: https://github.com/Shockah/Cobalt-Core-Mods/blob/dev/duo-artifacts/Kokoro/ModEntry.cs#L13 https://github.com/Shockah/Cobalt-Core-Mods/blob/dev/duo-artifacts/Kokoro/ModEntry.cs#L37-L38
Kokoro also provides the "API" folder with types that are ready to be copied into a mod that wants to use it as a dependency. this is optional, but makes others' lifes easier. https://github.com/Shockah/Cobalt-Core-Mods/tree/dev/duo-artifacts/Kokoro/API
DuoArtifacts (the dependent mod) copies the parts of the API it's interested in to its own code. this doesn't necessarily have to be everything the dependency mod provides. most importantly, there needs to be the main interface to implement, this will be used in the next step. https://github.com/Shockah/Cobalt-Core-Mods/tree/dev/duo-artifacts/DuoArtifacts/ExternalAPIs
DuoArtifacts implements IModManifest. in the BootMod method, it can call contact.GetApi, passing in the API interface type from the previous point and the unique mod name of the mod that provides the API. the method returns null and logs if it failed for any reason, otherwise it returns an object implementing the API interface. https://github.com/Shockah/Cobalt-Core-Mods/blob/dev/duo-artifacts/DuoArtifacts/ModEntry.cs#L40
DuoArtifacts proceeds to use Kokoro's API. https://github.com/Shockah/Cobalt-Core-Mods/blob/dev/duo-artifacts/DuoArtifacts/Artifacts/IsaacRiggsArtifact.cs


revert-428-changes

Describe what your pull request addresses

[ ]

Test Instructions

[ ]

no longer needed


Welcome to STC Discussions!

Discussed in https://github.com/stclib/STC/discussions/43

Originally posted by tylov January  4, 2023

👋 Welcome!
We’re using Discussions as a place to connect with other members of our community. We hope that you:

Ask questions you’re wondering about.
Share ideas.
Engage with other community members.
Welcome others and are open-minded. Remember that this is a community we
build together 💪.

To get started, comment below with an introduction of yourself and tell us about what you do with this community.

Its really great to see a modern container library for C.
Thank you for your contribution.
I am new to STC (but not C). I have no experience of the STL,
so to find my way around STC I am running tests on the operations
supported by the containers that are of use to me.
The following test (from a suite of tests):
#include <stdio.h>
#include "ctest.h"
#define i_TYPE Ints, int 	// Container type (name, element type)
#define i_eq(a, b) (*a == *b)
#include "stc/list.h"		// "instantiate" the desired container type
CTEST(vectest, insertAtOperation) {
Ints nums10 = c_init(Ints, {0, 1, 2, 3, 4, 5, 6, 7, 8, 9});
Ints_iter iter = Ints_begin(&nums10);
iter = Ints_advance(iter, 3);
iter = Ints_insert_at(&nums10, iter, 99);
ASSERT_EQ(Ints_count(&nums10), 11);
printf("nums10:");
c_foreach (i, Ints, nums10)
    printf(" %d", *i.ref);		// nums10: 0 1 2 99 3 4 5 6 7 8 9
ASSERT_EQ(*Ints_get(&nums10, 2), 2);
ASSERT_EQ(*Ints_get(&nums10, 3), 99);
//ASSERT_EQ(*Ints_get(&nums10, 4), 3);

Ints_drop(&nums10);

}	// isEmptyOperation
produces the error:
TEST 5/6 vectest:insertAtOperation nums10: 0 1 2 99 3 4 5 6 7 8 9[FAIL]
ERR: listtests.c:81  assertion failed, 3 == 99
on the third (and fourth) ASSERT_EQ.
Can you explain what I may have understood?
Thanks, Ken


Fix the Docker build by porting to salesforce-cli

Port the Docker build to use salesforce-cli and update the workflow for multi-arch Docker image builds.

Change the base image in Dockerfile to salesforce/cli:latest-full.

Remove the installation of sfdx-cli and add the installation of python3-pip, gh (GitHub CLI), and CumulusCI.

Add stages for full browser support (ChromeDriver + Playwright) and no browser automation support in Dockerfile.

Rename .github/workflows/build.yml to .github/workflows/docker-publish.yml.

Update the workflow to include multi-arch Docker image build and push steps.

Add a matrix strategy for target and platform.

Add steps for setting up QEMU, Docker Buildx, and logging into GitHub Container Registry.

Add a new workflow file .github/workflows/docker/publish.yml for building and publishing Docker images.

Include multi-arch Docker image build and push steps.

Set the workflow name to "Build Multi-Arch Docker Images".

Add a matrix strategy for target and platform.

Add steps for setting up QEMU, Docker Buildx, and logging into GitHub Container Registry.

Add .github/dependabot.yml for dependabot configuration.

Set update schedule to daily.

Add configuration for GitHub Actions, Docker, and Python updates.

For more details, open the Copilot Workspace session.

Please add the head branch of this pr to the push triggers


Converted xproj and changed package versions for run in VS 2015

I have changed in .xproj output directory, OutputPath and BaseIntermediateOutputPath.
Currently O2 running with VS2015 (debug mode)
Required: RC 2 Tooling for VS2015
https://download.microsoft.com/download/4/6/1/46116DFF-29F9-4FF8-94BF-F9BE05BE263B/DotNetCore.1.0.0.RC2-VS2015Tools.Preview1.exe
and dotnetcli
http://download.microsoft.com/download/2/1/0/2107669A-0DF9-4A91-A275-74735D433045/dotnet-dev-win-x<IP_ADDRESS>-preview1-002702.zip

The links don't work. How did you get the VS Tools preview ?

sorry my mistake, changed.

Thanks, not sure needed anymore :)


Make checklist for each testnet release

We should collect a list of all the places where things need to be changed every time a testnet is released, and ensure we follow this checklist every time we release a new testnet.

There are a few notes in the repo README under the genesis section re: what needs to change in the genesis file for each testnet which can be removed or moved to this checklist when it exists:

To perform genesis for a testnet, edit the genesis.json file stored in $TMHOME/config/ or ~/.tendermint/config/ (see an example in testnets/genesis_tn001.json). You should edit the following fields:

validators key: add the other validators and their voting power,
app_state key: add the generated genesis notes,
chain_id update the chain_id for the testnet.

Where should this checklist exist, just as another Markdown file in the repo?

One other item for the checklist is to git tag 00X-name such that if breaking changes are merged after the testnet is live, testers are not disrupted

[ ] Update hardcoded testnet URL in code
[ ] Create new git tag e.g. 002-euporie
[ ] Create droplet & associated floating IP
[ ] Add droplet to database trusted sources in DO
[ ] Add team member SSH keys to droplet
[ ] Create A record in Namecheap for testnet DNS
[ ] Create database and credential in DO
[ ] Create database config file on droplet (/root/secretz/db_connection_string)
[ ] Configure grafana (/etc/grafana)


Docs link is broken

Hi,
Thank you so much for this library! It's exactly what I wanted.
The link to the docs: https://chantastic.org/react-svg-spinner worked about a week ago, but 404's as of today. While I'm sure it would be brought up soon, what do you think of having a version of docs in the Readme.md of the repo itself?
While it may not have interactive examples, at least the API reference would be available.

thanks so much for letting me know!
and thanks for using the library.
last week I moved my personal site away from Github Pages
and it broke all my project links.
i've added a redirect.
so the same URL should take you to the project page.
thanks again!


No border radius on action labels

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

Version: 1.71.0
Commit: 784b0177c56c607789f9638da7b6bf3230d47a8c
Datum: 2022-09-01T07:25:25.516Z
Electron: 19.0.12
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
Betriebssystem: Darwin arm64 21.6.0
Version 1.71.0 introduced rounded corners on all buttons. I think it would make sense, to also apply this style change to action labels like these:

Was apparently already fixed in https://github.com/microsoft/vscode/pull/159496


[WIP] Test memory leak fix

Description

Testing instructions

Types of changes

Checklist

[ ] Prefix PR title with [WIP] if necessary.
[ ] Add tests to cover changes as needed.
[ ] Update documentation as needed.
[ ] I have read the guidelines for adding a new blockchain
[ ] If there is a related Issue, mention it in the description (e.g. Fixes #<issue_number> ).

Two fixes are reflected in results, two less errors :)


[DOCS] Use a gif to show a demo

Description
Instead of a picture of the website for demo in the readme, would be nice to have a gif instead
Screenshots
No response
Additional information
No response

Hi Dunsin, great idea. Do you want me to assign this to you?

Na, I'm good, someone else can take it

Awesome, will work on it asap

Awesome! I'll assign this issue to you 💙💙

Can u please add the hacktoberfest-approved label? I saw this issue thanks to the hacktoberfest!

Can u please add the hacktoberfest-approved label? I saw this issue thanks to the hacktoberfest!

Done. I added the label after it was merged, so please tell me if it does or does not appear in your hacktoberfest profile.

Yes, it appears! Thx


EPMLABSBRN-159

EPMLABSBRN-159
Description:  task actions flow adjustments

this MR is not for merging yet( we have to decide on endpoint and the format for sending a completed exercise )
it's rather for review to make all adjustments in a row if needed.


use eccodes-base-image

use eccodes-base-image
Addresses #314 #328

I've updated the Dockerfile to use base-image from ghcr.io
I also added an update to speed up the build-process. I added some suggestions in seperate issue to further speed up the GitHub-action but latest result looks good 7m10s :


Explicitly write down the rule of architecture extension test macros

We didn't write the rule explicitly, so people might not expect those marco
to be available if it's not appear on the table.
So let document that, and add more note to say people can expect that
even not list on the table.

I've left a couple of minor phrasing suggestions.
One alternate approach would be to cut down the table and just list a few examples, on the basis that keeping it up to date is extra work that might not have much value for end users.

@asb Seems like those suggestion are missing by some accident? Did you mind send again?

@asb Seems like those suggestion are missing by some accident? Did you mind reply those suggestion again?

Ah, they were still pending and unsubmitted. Fixed now.


Update preview symbol to construction sign emoji

I don't love the sunrise emoji and 🚧 seems nice :)
Requires #7195

While you are at it, maybe also fix the "Hammer and Wrench emoji" to use the actual emoji character. It is currently missing the required variation selector, so it renders as a black and white symbol instead of as a color emoji:
🛠 -> 🛠

Context for emoji variant selectors: https://www.codejam.info/2021/11/emoji-variation-selector.html. Debugging tool: https://qaz.wtf/u/show.cgi. For me (firefox on ubuntu), it doesn't actually display black and white but just a different emoji:

I've checked that hammer (🛠) is in https://www.unicode.org/Public/emoji/5.0/emoji-variation-sequences.txt while construction sign (🚧) isn't.

For me (firefox on ubuntu), it doesn't actually display black and white but just a different emoji:

Thanks for providing all that information. For reference, this is what I see with Firefox on Windows 11:

They're hilariously identical on Firefox / macOS. I can update it though :)

Wdyt of a flask icon (I haven't searched emojis yet) https://fontawesome.com/search?q=experimental&o=r I think it captures the experimental nature better than under construction (because preview features are functional features that need fine tuning, not unfinished work, which is what I associate with the construction emoji)

Wdyt of a flask icon (I haven't searched emojis yet)

It looks like the closest emoji would be the Test Tube: 🧪 or Alembic: ⚗ (the latter would also require a variation selector).

A flask would be nice. I don't like the test tube as much but it might be as close as we can get. I agree they seem better than the construction sign.

it should obviously be an eppendorf tube rather than a test tube :P but i think either will do fine


fal flow run --target TARGET loads non-flag target before switching

Describe the bug
When running fal flow run --target ddb (my duckdb profile) I get the following error:
20:46:15  Error importing adapter: No module named 'dbt.adapters.fal'
Traceback (most recent call last):
  File "/Users/matteo/.pyenv/versions/3.8.13/envs/dbtdd/lib/python3.8/site-packages/dbt/adapters/factory.py", line 56, in load_plugin
    mod: Any = import_module("." + name, "dbt.adapters")
  File "/Users/matteo/.pyenv/versions/3.8.13/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 973, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'dbt.adapters.fal'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/matteo/.pyenv/versions/3.8.13/envs/dbtdd/lib/python3.8/site-packages/dbt/config/profile.py", line 179, in _credentials_from_profile
    cls = load_plugin(typename)
  File "/Users/matteo/.pyenv/versions/3.8.13/envs/dbtdd/lib/python3.8/site-packages/dbt/adapters/factory.py", line 201, in load_plugin
    return FACTORY.load_plugin(name)
  File "/Users/matteo/.pyenv/versions/3.8.13/envs/dbtdd/lib/python3.8/site-packages/dbt/adapters/factory.py", line 62, in load_plugin
    raise RuntimeException(f"Could not find adapter type {name}!")
dbt.exceptions.RuntimeException: Runtime Error
  Could not find adapter type fal!

And the ddb target has type: duckdb
fal_test:
  target: fal_ddb

  outputs:
# relevant section of my profiles.yml
    fal_ddb:
      type: fal
      db_profile: ddb
    ddb:
      type: duckdb
      path: "/Users/user/duck_db_dbt_dump.db"
      s3_region: "us-east"
      s3_access_key_id: aa
      s3_secret_access_key: "xx"

Your environment

OS: mac
Paste the following commands output:

❯ fal --version
fal 0.7.3

❯ dbt --version
Core:
  - installed: 1.3.0
  - latest:    1.4.1 - Update available!

  Your version of dbt-core is out of date!
  You can find instructions for upgrading here:
  https://docs.getdbt.com/docs/installation

Plugins:
  - duckdb: 1.3.1 - Update available!

  At least one plugin is out of date or incompatible with dbt-core.
  You can find instructions for upgrading here:
  https://docs.getdbt.com/docs/installation

How to reproduce
Point to a not-installed profile in your target:  property in profiles.yml, and pass a yes-installed profile.
Expected behavior
Should completely ignore the target:  of profiles.yml and instead use --target setting.
Actual behavior
It will load the profile with the not-installed profile before actually using the correct one, and raise an exception.

Same issue here on mac os user brew version.
❯ dbt --version
Core:
  - installed: 1.5.0
  - latest:    1.5.1 - Update available!

  Your version of dbt-core is out of date!
  You can find instructions for upgrading here:
  https://docs.getdbt.com/docs/installation

Plugins:
  - postgres: 1.5.0 - Update available!

  At least one plugin is out of date or incompatible with dbt-core.
  You can find instructions for upgrading here:
  https://docs.getdbt.com/docs/installation

❯ fal --version
fal 0.9.1

error:
❯ dbt run --target fal_dev
23:36:05  Running with dbt=1.5.0
23:36:05  Error importing adapter: No module named 'dbt.adapters.fal'
23:36:05  Encountered an error:
Runtime Error
  Credentials in profile "flux_analytics", target "fal_dev" invalid: Runtime Error
    Could not find adapter type fal!

Hello, @iddar
From your output it seems you do not have dbt-fal installed. fal and dbt-fal are two separate packages. To use dbt with a fal type adapter, you need to install dbt-fal.
Can you run pip install dbt-fal==1.5.2 and let us know if know it works?


[MIG-284] Update DVM PVC check to include determination whether PVCs are bound and usable

Today GetDestinationPVCs is a no-op phase, we need to update this to include a check of whether the PVC is bound or not. If the PVC is pending, do not continue.
This check should be moved to after the creation of the transfer pods.

Adding Alay because he has some thoughts to share with whoever works this.


Register taxonomies first?

Thanks for this class, very useful!
One note, in order to ensure taxonomy permalinks always work, it's useful to switch lines 168 and 171, so that the taxonomies are registered first. Bizarre but true, see this post: http://clarknikdelpowell.com/blog/the-right-way-to-do-wordpress-custom-taxonomy-rewrites/
In the current configuration, taxonomy slugs that include a path ("books/genre") instead of just a simple slug ("genre") get created correctly, but result in 404's when you surf to them. Swap so taxonomies are registered first, and they work.

Interesting! I'll get to this today/tomorrow when I get 5mins.
Thanks :)


bugs

Bugs found when open and close the application.
no dot on save icon when you change the code.
slow loading feature...
please regards this issue to further or next update.

Bugs found when open and close the application.
no dot on save icon when you change the code.
slow loading feature...
please regards this issue to further or next update.

Can you send a screenshot?

i send the screenshot when i encounter again...my solution is uninstall the
app and reinstall again.The bugs will fix again but i monitor check all
features and wait to encounter again the bugs
thanks for reply.
Godspeed!
On Sat, Apr 25, 2020, 8:40 PM Zen<EMAIL_ADDRESS>wrote:

Bugs found when open and close the application.
no dot on save icon when you change the code.
slow loading feature...
please regards this issue to further or next update.
Can you send a screenshot?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/deadlyjack/code-editor/issues/24#issuecomment-619373618,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO2MBARGOQV6O25C75VQOL3ROLK2HANCNFSM4MQXMKLA
.

Please can you provide more details on the issue.

When apps is minimize for 50 minutes and after i return to open again and preview the code i can't save because of bug.


docs(): rename Angular 2 to Angular

Short description of what this resolves:
I want to eliminate the V2 Tag from Angular like we did with Ionic 2

Thanks!


Chat -"Unavailable workspace" displayed when user navigated to the message notification

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.28-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Kill the app and lock the main testing device
On the secondary device create a workspace from different account
Invite account from main testing device to the workspace
Send message to the announce room from secondary device
Tap on the message notification from the main testing device

Expected Result:
User should be navigated to the announce room and chat should be properly displayed
Actual Result:
"Unavailable workspace" displayed when user navigate to the message notification in announce room and different room avatar displayed
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/3cb65a39-edff-46b6-8ac0-d431fa3253d2
https://github.com/Expensify/App/assets/78819774/6dd05775-7808-42d8-b89d-3a72a89c7bac
View all open jobs on GitHub

I believe this is a known bug and unlikely to be some thing that's due to the staging diff - see reports of similar behavior here, here, here, and here.

Confirmed after chatting with @francoisl that this is a known bug, not a blocker.


Allow admin overrides of all permission scopes

Closes #559

This allows a user who has the admin flag set on their account to operate against the api in admin-mode by including a ?admin=any_value_here parameter in their request. Otherwise the an admin user's api request continue to behave normally.


Deprecate formula

The formula is not maintained and is no longer necessary as the
installation on OSX is just as simple as any other OS now.

After this is merged I will archive the repo.

cc @aarzilli


ContentDialog do not block keyboard input on background controls

Describe the bug
ContentDialog do not block keyboard input on background controls
Step to reproduce

Run FAControlsGallery.Desktop
Go to ContentDialog page sample
Set Default Button to Primary
Click on button Launch ContentDialog
Press Enter key on keyboard

Screenshots
Desktop/Platform (please complete the following information):

OS: independent
FluentAvalonia Version: master 31140cccf95f9af428d90767d9f192ed6c8492f5
Avalonia Version: 11.0-rc1.1

Additional context
Add any other context about the problem here.

IIRC, ContentDialog doesn't automatically focus upon showing in FA. I worked around this by focusing the dialog, but you also have to be careful that ContextMenu popups don't steal focus back if you're launching a dialog from one.
public async Task AlertAsync(string title, string message)
{
    await Task.Yield(); // Yield ensures any ContextMenus are closed so focus isn't stolen

    var titleBlock = new SelectableTextBlock()
    {
        Text = title
    };

    var contentBlock = new SelectableTextBlock()
    {
        Text = message
    };

    var cd = new ContentDialog
    {
        PrimaryButtonText = "Ok",
        Title = titleBlock,
        Content = contentBlock,
        IsPrimaryButtonEnabled = true,
        IsSecondaryButtonEnabled = false,
        DefaultButton = ContentDialogButton.Primary
    };

    cd.AttachedToVisualTree += InitializeWithFocus;
    await cd.ShowAsync();
    cd.AttachedToVisualTree -= InitializeWithFocus;

    void InitializeWithFocus(object? sender, VisualTreeAttachmentEventArgs e)
    {
        cd.Focus();
    }
}

ContentDialog is supposed to focus the dialog upon opening - the code is there to do it, but for some reason its not working. The dialog is attached to the tree and the actual buttons are realized and .Focus is called, but for some reason the focus is either shifting back to whatever was originally focused or it never applies.
In all honesty, the way ContentDialog is setup is quite messy and has been hacked together to deal with some life cycle stuff that was an issue when it was originally written (this is one of the oldest controls in this library) - something that was improved with TaskDialog, and may have improved with 11.0. I'd like to try and refactor that to make it easier and more reliable, but I really want to get my unit tests back up first to ensure nothing breaks in that process, something I'm hoping to do later this summer. For now, I'd recommend manually focusing the dialog

https://github.com/amwx/FluentAvalonia/blob/9349a97fbd14a3e0a930042cfbf7962132d6c872/src/FluentAvalonia/UI/Controls/ContentDialog/ContentDialog.cs#L356
@amwx I think focus should happen on the UI thread and after loading is completed, so I guess wrapping it in Dispather.UIThead.Post(() => btn.Focus, DispatcherPriority.Loaded) should do the trick. At least I do it that way to focus a textbox in a custom ContentDialog.

https://github.com/amwx/FluentAvalonia/blob/9349a97fbd14a3e0a930042cfbf7962132d6c872/src/FluentAvalonia/UI/Controls/ContentDialog/ContentDialog.cs#L356
@amwx I think focus should happen on the UI thread and after loading is completed, so I guess wrapping it in Dispather.UIThead.Post(() => btn.Focus, DispatcherPriority.Loaded) should do the trick. At least I do it that way to focus a textbox in a custom ContentDialog.

We are still on the UI thread when that is called though. Technically, we never leave the UI thread (or else we'd crash with invalid thread) except for the TaskCompletionSource that's used to await user input on the dialog.
I know this used to work, so I need to figure out why it stopped and what the best way to fix it is. Posting to the dispatcher working though does seem to suggest that maybe we're not entirely connected to the tree yet (as I thought). I have some ideas on this though...


Spilt SCM

Currently the SCM is only in one big file, which makes maintaining and upgrading the SCM harder and harder.
Therefore we should split the SCM functionality into multiple parts, each taking specific responsibilities.
For Example:

Download & Install
Local-Script-Manager (Responsable for the JSON)
UI
...
new Features to come

Additionally, there should be an installer within git directly, which enables us to be almost completely Independent from Pastebin.

Additionally, there should be an installer within git directly, which enables us to be almost completely Independent from Pastebin.

"almost" because we would still need a script for the initial install, right? A script that runs the "install script" within the repository. Updates can then be handled via SCM itself.
Sounds good. We should put everything in a proper directory then. I'd still prefer not to use "external dependencies" for the SCM. There are a few functionalities we use in SCM that we also have as a library, I think those should stay separated.


PP-11850 Reverting CSP change that was causing Naxsi issues

Removing the code in the content security policy that added code for report-to
Was causing an Naxsi issue.

Closing - will raise another PR.


fix issue #31: The body of constexpr function not a return-statement

Check issue #31 to get more details

Could you also update the bgfx, bx and bimg libraries so they point at the latest HEADs?

update the bgfx, bx and bimg libraries done


Shortcut and more

Thanks for taking over! I had prepared these changes in a pull request for the original author, in the hope he might update his mod again.  Maybe you want to include my changes. I added a shortcut and a custom input (just moved them from my own mod "Shortcuts for 1.1") , changed the selection tool so that you can only select resource entities and added a proper changelog.

I seem to have un-watched the repo after creating it, somehow. Will review shortly, Thanks!

Yeah, sorry for the mess. The incorrect whitespace is probably due to the editor I used but it wasn’t my top priority either. I had some trouble with inexplicable conflicts between the different branches I created and the local and online versions. I am not that experienced with GitHub.


Namespace names must be RFC 1123 DNS labels

The how-to docs to create a new namespace already specify this, but I think it's worth mentioning in the overview doc that a namespace name must be an RFC 1123 label, as I didn't initially think to look in the how-to doc for limitations.

/retitle Clarify that namespace names must be RFC 1123 DNS labels


Fix incorrect line text computation to stop comment directive searching

Fixes #38289

@typescript-bot cherry-pick this to release-3.9

@typescript-bot cherry-pick this to release-3.9

Reverting  e860d48  since tests/cases/compiler/checkJsFiles_skipDiagnostics fails with that change

@typescript-bot cherry-pick this to release-3.9


added NameInCamelCase, StringsInCamelCase;

closes #141
@h1alexbel take a look, please

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.60 :tada:

Comparison is base (85af184) 83.87% compared to head (be45b71) 84.47%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #148      +/-   ##
============================================
+ Coverage     83.87%   84.47%   +0.60%
- Complexity       51       55       +4
============================================
  Files            21       23       +2
  Lines           155      161       +6
============================================
+ Hits            130      136       +6
  Misses           25       25

Impacted Files
Coverage Δ

.../io/github/eocqrs/kafka/settings/AttrEnvelope.java
100.00% <ø> (ø)

...va/io/github/eocqrs/kafka/xml/NameInCamelCase.java
100.00% <100.00%> (ø)

...io/github/eocqrs/kafka/xml/StringsInCamelCase.java
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

also added formatting that feels right to me

@l3r8yJ after you create puzzles, we create pr into yegor256/cactoos, after pr will be merged we remove this classes.

@h1alexbel take a look, please

@rultor merge

@rultor merge

@h1alexbel OK, I'll try to merge now. You can check the progress of the merge here


Customizable Field Name for Log Message

Description:
Having the ability to customize the key for the default msg field during logger initialization would provide greater flexibility in shaping log output to meet diverse formatting requirements. This enhancement would not only make it easier to adapt Logrus to existing logging standards but also enable teams to ensure consistency across different systems and workflows.
Proposed Solution:
Introduce a new field in the TextFormatter (and potentially other formatters) to allow users to set a custom field name for msg. For example, this could be done with a CustomFieldKeyMsg option.
log.SetFormatter(&log.TextFormatter{
    DisableColors:     true,
    FullTimestamp:     true,
    CustomFieldKeyMsg: "event",
})

The log output would change from this:
{
    "animal": "walrus",
    "level": "info",
    "msg": "A group of walrus emerges from the ocean",
    "size": 10,
    "time": "2014-03-10 19:57:38.562264131 -0400 EDT"
}

To this:
{
    "animal": "walrus",
    "level": "info",
    "event": "A group of walrus emerges from the ocean",
    "size": 10,
    "time": "2014-03-10 19:57:38.562264131 -0400 EDT"
}

Trying to implement this, I realized that this functionality has already been implemented. Apologies for the oversight! Thank you for the great work on Logrus.
I’ll go ahead and close this issue.


haircut syntax change

instead of haircut —mowhawk Adam could we use haircut --style mohawk Adam a la git log --graph

Sure.  I think that makes sense.  Maybe even --who Adam.  It was just a simple, thrown together example for flags.  I think that it was tougher to parse values for a flag.


Implement browsersync proxy on 'watch' gulp task

https://www.browsersync.io/docs/gulp/

I've had a play with this and didn't like it. It's also veering Stalfos away from being a starting point. I think if a dev wanted to use a tool like Browsersync, they could extend the base gulp setup to do just that.


Minimize to Tray acts as Close to Tray

OS:
Windows 10
Issue Descriptions:
When activating the Minimize to Tray option, the minimize button keeps sending the window to the taskbar but when we use the Close button then the window is hidden, removed from taskbar but kept in the tray. This is called "Close to Tray".
Both options should be offered, Close to Tray which defines the Close button action, and Minimize to Tray which affects the Minimize button action.
But the best would be to keep the minimize button to minimize and the close button to close the program, but add another button in between which does minimize to tray.
At the very least, rename the actual "Minimize to tray" setting to "Close to tray" which is what it does.
Steps to Reproduce:
Activate "Minimize to tray" button. Minimize window.

At the very least, rename the actual "Minimize to tray" setting to "Close to tray" which is what it does.

This is the first time I've heard someone query the distinction between the two, as far as I am aware they are used interchangeably for the same concept.  Our "minimize to tray" does exactly that, you click close and it visually minimizes to the tray.  I don't want to change this behavior now as it would mess with lots of people who understand the existing behavior.
Also adding another button that isn't a standard window control would just confuse the heck out people 😄

And to be fair, most of the language regarding this functionality says "minimize to tray on close" or something similar. I think I've only seen one or two applications actually use "close to tray". So really, it's not so much incorrect as it is incomplete, but even then, the wording is probably clear enough.

After returning to use GPMDP I found myself confused by this setting once again. So I'd like to second the suggestion that the name of this setting be rephrased as per the suggestions by @BrianAllred .


The initial bias of the forget gate

I'm still trying to understand how it all works together. A question: how/where is the initial bias of the forget gate set? According to some papers (e.g. http://www.datascienceassn.org/content/empirical-exploration-recurrent-network-architectures), it should be set between 1 and 2 to "encourage learning". In char_rnn it's set to 1. But I can't find it in torch_rnn.

The bias of the forget gate is set to 1 here:
https://github.com/jcjohnson/torch-rnn/blob/master/LSTM.lua#L54

Oh, I missed  self:reset() in layer:__init ... :)
Thanks Justin!


ci: merge staging to master

This is an automatic PR generated by the pipeline CI/CD. This will be automatically fast-forward merged if successful.

Pipeline Attempt on 589459352 for bb8824913917d6d527ca41aa97c676f894869833
https://gitlab.com/MatrixAI/open-source/typescript-demo-lib-native/-/pipelines/589459352

Pipeline Attempt on 589460354 for 61d80ba9995c9db0801ce445b0114865923e4921
https://gitlab.com/MatrixAI/open-source/typescript-demo-lib-native/-/pipelines/589460354

Changes here should be incorporated into js-db. Except of course all the package builds. We don't need test load balancing there yet... so we can keep it simpler than what this demo repo shows.

Pipeline Attempt on 589461181 for ccb57132fcf3aa2dc1adc5f72e933c3033e68938
https://gitlab.com/MatrixAI/open-source/typescript-demo-lib-native/-/pipelines/589461181

Pipeline Attempt on 589463396 for 570addbd8233b0035112adc90eeff5eaa483ccf3
https://gitlab.com/MatrixAI/open-source/typescript-demo-lib-native/-/pipelines/589463396

Pipeline Attempt on 589463567 for 9ff2a499f84aa9daeb7c580a82c1577c1251873e
https://gitlab.com/MatrixAI/open-source/typescript-demo-lib-native/-/pipelines/589463567

Pipeline Attempt on 589464085 for 0455dfbdb51548bf674835f9f9c203e145e4e4e2
https://gitlab.com/MatrixAI/open-source/typescript-demo-lib-native/-/pipelines/589464085

Something strange here. Both on this repo and the TS demo lib, the second shared windows test job always fails. The test execution results in no output. @emmacasolin any ideas?

I tried running jest "--ci" "--coverage" "--shard=2/2" "--maxWorkers=50%" directly on matrix-win-1 and it worked fine. I am not able to replicate the error that's happening on CI/CD. Especially when there's no error message at all. There's no output after running that command, and it just goes into failure state.

The only thing that could be an issue is that the compilation isn't working. That is after compiling the native addon and copying it into the second windows runner, something causes the native module to no longer execute properly.
This is because I can see a GLOBAL SETUP message, and the very next messages are supposed to come from the native addon directly:
keyValueString_size: 6
keyValueString_len: 6

But now I noticed that even on matrix-win-1 the output looks strange:
size: 6eString_
keyValueString_len: 6

Can't replicate it. Something is broken in gitlab...

The only thing I can imagine is something about the native module breaking on the 2nd windows runner.

Would be worth checking this out on js-db later to see if it also applies there.
But there's a problem, the 2nd windows runner is also failing in ts demo lib too and there's no native module there.

Pipeline Attempt on 589518481 for 7a18cf671cafe4c314b564a37e63be8ae3462f3d
https://gitlab.com/MatrixAI/open-source/typescript-demo-lib-native/-/pipelines/589518481

Latest job https://gitlab.com/MatrixAI/open-source/typescript-demo-lib/-/jobs/2730803683 shows that it fails without sharding and no native modules. Something must be breaking in jest and windows...

Pipeline Attempt on 589772008 for 7201d4e0b1e60301153b7dbaf2da3b9e73f86715
https://gitlab.com/MatrixAI/open-source/typescript-demo-lib-native/-/pipelines/589772008

Actually the latest 2 runs have successful windows tests. No idea how that was fixed. Since this commit 7a18cf6, build:windows succeeded.
It's still failing on TypeScript-Demo-Lib, so I don't know why.
The only problem left for this CI/CD is the fact that the new ./prebuild.js seems to have changed how pkg.js is bundling. It is bundling build/Release/native.node instead of bundling the prebuilds before.
This means the integration:build job is incorrectly calling something that isn't calling the proper prebuild script.

I think this is because using prebuildify, it used to move the build/Release/native.node into the prebuilds location, but the current prebuild.js script only copies it. And we use node-gyp-build to find the right paths, and that's why it ends up bundling the build/ location instead of the prebuild location.
We can change this by setting PREBUILDS_ONLY=1 to ensure that it is bundling only prebuilds.

Setting PREBUILDS_ONLY does mean that packages must only have prebuilds for it be found with node-gyp-build... it's not allowed to bundle with only just builds/ directory. This should be sufficient since it's not likely people distribute native addon prebuilds in node_modules under the build/ directory. Any other kind of native modules would need to be setup differently...

Pipeline Attempt on 589791479 for abc6ade9a6a779716b0acaf6bebafe38a1a07513
https://gitlab.com/MatrixAI/open-source/typescript-demo-lib-native/-/pipelines/589791479

Pipeline Succeeded on 589791479 for abc6ade9a6a779716b0acaf6bebafe38a1a07513
https://gitlab.com/MatrixAI/open-source/typescript-demo-lib-native/-/pipelines/589791479


How to convert a model(tf_model.h5) with tokenizer folder to the onnx format

Feature request
I have trained the TensorFlow model using the Transformers library and saved the trained model and tokenizer in a folder named MODEL_WITH_TOKENIZER. The model is stored inside the folder in a  .h5 format - tf_model.h5
Here is the folder structure.

I want to convert the model to .onnx format
Should I convert the entire MODEL_WITH_TOKENIZER folder to .onnx or only the tf_model.h5 file to onnx?
what are the steps
Motivation
Hi, I have trained the TensorFlow model using the Transformers library and saved the trained model and tokenizer in a folder named MODEL_WITH_TOKENIZER. The model is stored in the .h5 format - model.h5
Here is the folder structure.

I want to convert the model to .onnx format
Should I convert the entire MODEL_WITH_TOKENIZER folder to .onnx or only the tf_model.h5 file to onnx?
what are the steps
Your contribution
I have trained the TensorFlow model using the Transformers library and saved the trained model and tokenizer in a folder named MODEL_WITH_TOKENIZER. The model is stored in the .h5 format - tf_model.h5
Here is the folder structure.

I want to convert the model to .onnx format
Should I convert the entire MODEL_WITH_TOKENIZER folder to .onnx or only the tf_model.h5 file to onnx?
what are the steps

@fxmarty any info?

@pradeepdev-1995 Thank you for the request. You can give a try to pip install optimum[exporters-tf] and
optimum-cli export onnx --model /path/to/your/model --framework tf onnx_output/

should (hopefully) work. You may need to specify the --task argument as well.

@fxmarty
I am using an Ubuntu machine and after installing with this pip install optimum[exporters-tf]
found two issues
1 - Error optimum-cli: command not found`` 2 - The model folder in optimum-cli export onnx --model /path/to/your/model --framework tf onnx_output/```
should be the tf_model.h5 file or the folder MODEL_WITH_TOKENIZER (which contains all files model and tokenizer files that I put in the screenshot)

About 1., can you try: pip uninstall optimum && pip install optimum[exporters-tf]. What are the install logs?
About 2., I'll try to reproduce, thank you.

@fxmarty
I am using ubuntu machine with Python 3.8.16 . The pip install optimum[exporters-tf] not installing it showcasing the given error

So installed via
pip install optimum[all]

Hey @pradeepdev-1995
Picking up this thread - is this still an issue for you?

I have not tried after that


Simulate typing the clipboard contents

Description of the new feature / enhancement
It would be nice to have a feature that allows the user to simulate typing the text content from the Clipboard.
For example, by pressing [Win]+[Shift]+[V] the PowerToys will check the clipboard content, and if the content contains text or can be treated as text content it will start a sequence of keypresses equal to that content.
Settings: delay between keypresses, to enable or disable this feature.
Scenario when this would be used?

Useful when pasting is not allowed.
Virtual OS installation process when integration tools haven't yet installed, usually (or, in my cases) you cannot use [Ctrl]+[V] to send any information into [some] virtual OS.
VNC connections where clipboard feature either not working or not implemented (e.g., Synology VM web based VNC connection)
Web pages, where pasting is disabled.

Supporting information
https://keepass.info/help/base/autotype.html

You could try something like AutoIt and create a script. www.autoitscript.com


feat [sc 106247]: Add a caching layer to prevent SDK from sending the same payloads too frequently upstream

What this PR does / why we need it:

Add caching middleware to the send custom metrics endpoint
Prevents identical payloads from being send repeatedly upstream to replicated-app while preserving the client side UX

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Steps to reproduce

Does this PR introduce a user-facing change?

Does this PR require documentation?

Is there a reason we don't apply this to other methods in the API? If we do, this should be a middleware in the same implementation that gin middlewares work.
It's weird to me that we are returning "Served From Cache" when we rate limit in this method. I do like that we are including a custom header to indicate that the request was rate limited, but the served from cache is intended to mean that the API was disconnected. Will reusing this same header create a more difficult to debug scenario where the user can't tell if the API was down or they are sending too frequently? Continuing to make this return the same status code is great, don't 429 and push the issue to the developer.

@divolgin (CC: @marccampbell ) I made a  cached sub-router and added a X-Replicated-Rate-Limited header.  Let me know what you all think.


autentication problem

sudo mono /root/bin/picasawebsync/picasawebsync.exe /mnt/nas/immagini/ -r -addOnly
WARNING: The runtime version supported by this application is unavailable.
Using default runtime: v4.0.30319
[Initializing]
[Connecting to Picasa web service]
Fatal Error Occured: Picasa error - https://developers.google.com/accounts/docs/AuthForInstalledApps

Just added this to the README:

PicasaWebSync uses ClientLogin to authenticate to the Google Photos API. Per Google: "ClientLogin     has been officially deprecated since April 20, 2012 and is now no longer available. Requests to ClientLogin will fail with a HTTP 404 response. We encourage you to migrate to OAuth 2.0 as soon as   possible." Until an update is made to support OAuth 2.0 PicasaWebSync will not work.


Passing --observe to the VM causes massive slowdown of program

The example provided by @lrhn at dartpad.dev/a16b85040 will run massively slower when invoked via --observe.
The reason seems to be that the kernel file given to the VM doesn't have the right metadata attributes which causes it to treat a typed closure call as a dynamic closure call, which can inhibit inlining.

When --observe is passed it seems to cause the VM to tell the kernel-isolate to use use an IncrementalKernelCompiler (see runtime/bin/main_options.cc) instead of a normal compiler.
Doing so might cause the call-site metadata not to be emitted.
There seems to be an untracked todo for adding the metadata, see here:

  _combinePendingDeltas(bool includePlatform) {
    ...
    for (Component delta in _pendingDeltas) {
      ...
    }

    // TODO(vegorov) this needs to merge metadata repositories from deltas.
    return new Component(
        libraries: combined.values.toList(), uriToSource: uriToSource)
      ..setMainMethodAndMode(mainMethod?.reference, true, compilationMode);
  }

/cc @mraleph

/cc @johnniwinther @jensjoha

For reference, the place in the VM where we would like to have the metadata is in here

@jensjoha could you merge the metadata?

I have no insight into the metadata. I think someone on the VM team (with knowledge of metadata) would be a better match for this.

I think you can just union all the metadata repositories (if there is a conflict the newest one wins) and that it.

Looks like another instance of https://github.com/dart-lang/sdk/issues/45421

I don't like metadata (or our implementation of it)... In this instance, merging correctly without potentially causing leaks is virtually impossible.
Metadata is (roughly) a number of maps mapping from Node to the actual metadata. If just blindly copying everything over into the new component when merging we'll (at some point at least) bring over references to otherwise unreferenced stuff (e.g. it should also be merged in the accept call in pkg/vm/lib/incremental_compiler.dart where when one library is replaced with another one metadata from the first will keep the replaced library alive). That's direct leaks, but they can also via parent pointers (or in the case of non-TreeNode Nodes via parent on contained TreeNodes (e.g. InterfaceType)) bring over entire (old) compilations.
Other than combining and then visiting the entire tree to record which nodes we actually know and just keep those (which would be slow) I don't think we can solve this with the current design of metadata.
I initially thought we could break metadata up by mapping from Library to Node to metadata (i.e. an "indirection" on Library) (with maybe some special handling of adding metadata to the actual component), but we can also associate metadata to nodes only used in constants so that doesn't really work.
The only solution coming to (my) mind is letting every node have a (nullable) metadata field. What the performance and memory characteristics is of that I don't know.
If anyone else have other ideas please share.

@jensjoha we could use Expando instead of Map to store metadata, though it is going to be slower. Could you try benchmarking that? We can change things to fallback to Expando only when merging.

We can't iterate the keys of an Expando so I don't think we'd be able to merge anything.

Our main use of metadata is in AOT where we would like to attach more information to the AST nodes than what the Kernel AST  format currently offers. For that incremental compilation is not much of an issue.
Though in JIT the Kernel AST should mostly be sufficient. So maybe the best strategy is to work with CFE to get all information in the Kernel AST that JIT needs.
In this case it actually seems the FunctionInvocationNode does have a kind which we can rely on. Not requiring us needing this metadata in the first place.
I've made cl/203302 which addresses the performance issue.
That being said, I don't know if there are other place where JIT relies on metadata.


fix: #211 UI Redesign - Desktop - Settings

- What I did
Added new desktop background
Added Settings Tab in sidebar
Added Settings Screen
Fixes #211
- How I did it
Designed and Coded the settings screen for desktop and added it to navigation drawer following the convention of the app
- How to verify it
Click on the setting tab in the sidebar
- Description for the changelog
Added new Settings Screen

Hey @tinashe404 This fixes #241

@Chandram-Dutta it looks like you worked on a ticket that wasn't assigned to you. This task is currently assigned to @AdiAr11

@Chandram-Dutta This looks great! Only one suggestion! could you please resize the delete atsign modal so that it's around the same width as the backup keys one?

@tinashe404 will do it...

@Chandram-Dutta it looks like you worked on a ticket that wasn't assigned to you. This task is currently assigned to @AdiAr11

Yes, i did the mobile version and there were no followups on issue... So i used the codebase of the mobile to desktop...

@Chandram-Dutta we accepted and merged your mobile PR for settings


Netlify deploy failing

Launching from this repo directly, I'm getting a build failure. Looks like astro isn't available in deploy sequence?

8:43:32 AM: ────────────────────────────────────────────────────────────────
8:43:32 AM:   1. Build command from Netlify app
8:43:32 AM: ────────────────────────────────────────────────────────────────
8:43:32 AM:
8:43:32 AM: $ npm run build
8:43:32 AM: npm WARN config tmp This setting is no longer used.  npm stores temporary files in a special
8:43:32 AM: npm WARN config location in the cache, and they are managed by
8:43:32 AM: npm WARN config     cacache.
8:43:32 AM: ><EMAIL_ADDRESS>build
8:43:32 AM: > astro build
8:43:32 AM: sh: 1: astro: not found
8:43:32 AM:
8:43:32 AM: ────────────────────────────────────────────────────────────────
8:43:32 AM:   "build.command" failed
8:43:32 AM: ────────────────────────────────────────────────────────────────
8:43:32 AM:
8:43:32 AM:   Error message
8:43:32 AM:   Command failed with exit code 127: npm run build (https://ntl.fyi/exit-code-127)

Launching from this repo directly, I'm getting a build failure. Looks like astro isn't available in deploy sequence?

8:43:32 AM: ────────────────────────────────────────────────────────────────
8:43:32 AM:   1. Build command from Netlify app
8:43:32 AM: ────────────────────────────────────────────────────────────────
8:43:32 AM:
8:43:32 AM: $ npm run build
8:43:32 AM: npm WARN config tmp This setting is no longer used.  npm stores temporary files in a special
8:43:32 AM: npm WARN config location in the cache, and they are managed by
8:43:32 AM: npm WARN config     cacache.
8:43:32 AM: ><EMAIL_ADDRESS>build
8:43:32 AM: > astro build
8:43:32 AM: sh: 1: astro: not found
8:43:32 AM:
8:43:32 AM: ────────────────────────────────────────────────────────────────
8:43:32 AM:   "build.command" failed
8:43:32 AM: ────────────────────────────────────────────────────────────────
8:43:32 AM:
8:43:32 AM:   Error message
8:43:32 AM:   Command failed with exit code 127: npm run build (https://ntl.fyi/exit-code-127)

I have this issue also.
I guess because astro wasn't defined at package.json file in the main directory.
You might need to deploy the theme folder.

@mitrick2 Are you setting the base directory for your Netlify project to the theme/ folder? This would be necessary for it to deploy the actual site's files instead of the monorepo. I might be able to set up some config for this to be automatic but I'll have to look into that.

Doh! Once I set the base directory, it worked as expected. Thank you!


Converted my previous pull request to swift 2.

Thanks for this @harlanhaskins, but I separated out the SwiftyJSON and Swift 2.0 changes per https://github.com/mapbox/MapboxDirections.swift/issues/8. This PR needed updated to master anyway.


Add name 'jolokia' for 8778 port to enable Hawt.io console access for Java apps

For OpenShift to expose the Hawt.io console for Java application containers, the port 8778 MUST have the name 'jolokia' in the DeploymentConfig. Adding that name creates a link on the Pod page in OpenShift which will open the Hawt.io console.

With #243 merged (which is the CI used to run checks on this PR - which passed - see above), this PR can be merged.


Icons are not displayed in Oruga-next version with custom Iconpack [0.3.3]

Overview of the problem
Oruga version: [0.3.3]
Vuejs version: [3.0.0]
OS/Browser:
Description
Oruga uses default instead of custom Iconpack (working in Oruga version : 0.3.2)
A Custom Iconpack (mdi-Icons) is used.
The Icons are registered like that:
app.use(Oruga, {
	iconComponent: 'icon-mdi',
	iconPack: 'mdijs',
	customIconPacks: {
		mdijs: {
			iconPrefix: 'mdi-',
		},
	},
});
app.component('icon-mdi', iconMdi);

in the Vue component the icons are used like this:
<o-icon icon="map-marker"></o-icon>
Steps to reproduce

use Oruga Version 0.3.2
create a "o-icon" Tag
Icons are working
upgrade to Oruga Version: 0.3.3
Icons are not working anymore

Expected behavior
Icons from default are being loaded and not from the custom Iconpack.
Actual behavior
Icons from Iconpack are being laoded.

Thanks for the feedback, we'll investigate about it.
In the meanwhile can you share a codesandbox in order to reproduce it

Close https://github.com/oruga-ui/oruga/commit/dd5f8838db0551acec3779b439199145319cb5ee


HTTPS required for MaxMind downloads

MaxMind will be requiring HTTPS for all database download requests starting in March 2024.
See this release note.

@christophermluna I wonder if it wouldn't even be better to just remove the script contrib/geoip/geoip-lite-update.
Nowadays MaxMind provides a client to update the geoip databases: https://github.com/maxmind/geoipupdate
It's distributed by most Linux distros: https://repology.org/project/geoipupdate/versions
Is there any reason to NOT use the official geoipupdate client at this point?


Fix arguments in DaskLocalCluster

Description
The positional arguments used for constructing distributed.LocalCluster in DaskLocalCluster mismatch distributed >= 2021.02.0 since the name argument has added into the first position in https://github.com/dask/distributed/pull/4426.
This PR fixed this problem by passing all arguments as keyword arguments. It can avoid the API change of distributed package in the future as far as possible.
Status

[x] Ready to go

Hi darelbeida,
Thanks for the patch! If I'm understanding things correctly, this is a backwards-compatible change even though Dask changed their API in a confusing way (a new argument in the first position, displacing all other).
Some (but not all) of the tests will fail because our OpenEye license is not unpacked on forks. Don't worry about those, this is expected behavior.


Optimise field arithmetic in BaseUltraVerifier.sol

Much of the verification field arithmetic in BaseUltraVerifier.sol can be optimised.
There is currently a lot of duplication of MLOAD operations, we did this so that the code is straightforward to read and reason about, but it is more expensive.
We also have redundant uses of addmod. The BN254 field size is <2^254, meaning that we can call add up to four times before we need to reduce modulo p. This could save some more gas.
Overall gas savings would be very small, ~2,000 tops. Not worth prioritising but could be good for a newcomer who knows Yul and wants to make a contribution to the codebase.

Hey I would like to help this issue if it is still available. @zac-williamson

I am currently dealing with this issue. But I have a question regarding to addmod part. How can we be sure that add is not going to overflow before 4 add calls. Most of the addmod calls can be converted to add without error but i couldn't be sure about edge cases. Lets say
let a := 2^255
let b := 2^255
let c := add(a, b)

public inputs are bytes32 so they can be up to 2^256 and proof is bytes so it can be also up to 2^256.

Public inputs although bytes32, are checked to be less than p (254 bits) on line 586
https://github.com/AztecProtocol/aztec-packages/blob/53eb54fb3416711dbc411e6037bc2da18d523cc3/barretenberg/sol/src/ultra/BaseUltraVerifier.sol#L586

Hey thanks for your response. It makes sense.

We also have redundant uses of addmod. The BN254 field size is <2^254, meaning that we can call add up to four times before we need to reduce modulo p. This could save some more gas.

Can't we call add up to five times?

Hey I made some changes about addmod calls. Most of the tests executes correctly. There is approximately ~1300 gas saving.
> sudo forge test --no-match-contract TestBase

Running 2 tests for test/ultra/Add2.t.sol:Add2UltraTest
[PASS] testFuzzProof(uint16,uint16) (runs: 1000, μ: 447002, ~: 447986)
[PASS] testValidProof() (gas: 434301)
Test result: ok. 2 passed; 0 failed; 0 skipped; finished in 94.19ms

Running 2 tests for test/ultra/Blake.t.sol:BlakeUltraTest
[PASS] testFuzzProof(uint256,uint256,uint256,uint256) (runs: 1, μ: 459481, ~: 459481)
[PASS] testValidProof() (gas: 440644)
Test result: ok. 2 passed; 0 failed; 0 skipped; finished in 3.22s
2023-12-13T10:57:26.633033Z ERROR cheatcodes: non-empty stderr input=["./scripts/run_fuzzer.sh", "ultra", "recursive", "0x1668,0x1687,0x2cef"] stderr="checked circuit before add_recursive_proof\n"
2023-12-13T10:57:26.636836Z ERROR cheatcodes: non-empty stderr input=["./scripts/run_fuzzer.sh", "ultra", "recursive", "0x05,0x0a,0x0f"] stderr="checked circuit before add_recursive_proof\n"

Running 2 tests for test/ultra/ECDSA.t.sol:EcdsaUltraTest
[PASS] testFuzzProof() (gas: 456285)
[PASS] testValidProof() (gas: 451425)
Test result: ok. 2 passed; 0 failed; 0 skipped; finished in 16.30s

Running 2 tests for test/ultra/Recursive.t.sol:RecursiveUltraTest
[PASS] testFuzzProof(uint16,uint16) (runs: 1, μ: 472207, ~: 472207)
[PASS] testValidProof() (gas: 458521)
Test result: ok. 2 passed; 0 failed; 0 skipped; finished in 16.30s

As you can see Recursive test is giving 2 errors before running correctly. It also same without any changes.
Is it normal and is there anything i can do about this problem?

ah very nice! These are just debug outputs to std err, and are nothing to worry about!

I opened a PR with necessary changes. After removing duplicate mload calls gas usage have increased. At least according to forge test.

If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag.

Also I couldn't add crypto tag to PR.

I was running tests to make sure everything is working correctly. But i received PROOF_FAILURE error.
Running 1 test for test/ultra/Add2.t.sol:Add2UltraTest
[FAIL. Reason: PROOF_FAILURE(); counterexample: calldata=0x03911ac500000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000001f55 args=[0, 8021]] testFuzzProof(uint16,uint16) (runs: 8607, μ: 446894, ~: 448005)
Test result: FAILED. 0 passed; 1 failed; 0 skipped; finished in 430.98s

How can i reproduce the error with same inputs. I tried to hard code in contract. But it didn't work.
function testFuzzProof(uint16 input1, uint16 input2) public {
    uint256[] memory inputs = new uint256[](3);
    inputs[0] = uint256(0);
    inputs[1] = uint256(8021);
    inputs[2] = inputs[0] + inputs[1];

    bytes memory proofData = fuzzer.with_inputs(inputs).generate_proof();

    (bytes32[] memory publicInputs, bytes memory proof) = splitProof(proofData, PUBLIC_INPUT_COUNT);

    assertTrue(verifier.verify(proof, publicInputs), "The proof is not valid");
}

The failure depends on both the inputs and the proof, so if you are rerunning it, it could simply be the proof that differs from the old one making it pass the case it hit earlier.

How can i also print Proof on error? It hard to encounter with this error on random testing. I should reproduce it.

I fixed the problem and updated the PR.


EEGSynth ?

What is EEGSynth ? Can you explain this in a couple of lines and provide a brief setup guide.
I am really interested in that project and would like to understand the system (from a schematic perspective).
Thank you

Sure ! I'll update the README.md pretty soon ;)


Fix mistakes in comments

Fixed some mistakes in comments

Oh no, embarrassing how many misspellings!  Thanks flash2048!


Use strict equality operator for each sumFibs test

Pre-Submission Checklist

[x] Your pull request targets the staging branch of FreeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue (replace XXXX with an issue no): Closes #11830

Description

It's likely for loopProtect to break out of sumFibs(4000000), but not any other case. Thus for every other case an array of size num is returned and tested by assert.deepEqual—which will freeze the browser for large nums.
This PR piggybacks off of @marzelin's comment, which suggests comparing the expected and actual results manually with the equality operator.

Nice catch. Not sure what the original reason was for using deepEqual when the function is expected to return a number.
Happy Coding


remove unused SubComponent

ViewModelSubComponent is no longer used since ViewModelModule with multibindind is used by GithubViewModelFactory.
Keep this class could be confusing.

Thanks for PR! This class was unused indeed and it was removed in another CL.


QuickChick derivations for aliased types

Can be replicated using search_tree example because it defines types for the key and value of a tree: Definition key := nat.
It looks like automatic derivation for these aliased types fail (Derive Show for key throws an uncaught exception).
This is probably a bug in QuickChick, but there could be a workaround here where we define custom show and arbitrary functions for the aliased type.

For now, a workaround is using Notation instead of Definition.


[#1749] feat(remote merge): Introduce new reader for reading sorted data.

What changes were proposed in this pull request?
Introduce new reader for reading sorted data. Since #1748 already provides methods for merging blocks, we need to provide a method for reading merged block. The record obtained from the getSortedShuffleData method is sorted using the comparatorClassName which is passed by registerShuffle.
Why are the changes needed?
Fix: #1749
Does this PR introduce any user-facing change?
Yes, add doc in a separated issue
How was this patch tested?
unit test, integration test, test real job in cluster.

Codecov Report
Attention: Patch coverage is 37.05722% with 462 lines in your changes missing coverage. Please review.

Project coverage is 52.72%. Comparing base (5ddcc28) to head (e54d40d).
Report is 45 commits behind head on master.

Files
Patch %
Lines

...pache/uniffle/server/ShuffleServerGrpcService.java
0.00%
182 Missing :warning:

.../uniffle/client/record/reader/RMRecordsReader.java
58.50%
101 Missing and 21 partials :warning:

...ffle/client/impl/grpc/ShuffleServerGrpcClient.java
0.00%
75 Missing :warning:

...a/org/apache/uniffle/client/record/RecordBlob.java
64.10%
13 Missing and 1 partial :warning:

...he/uniffle/client/impl/ShuffleWriteClientImpl.java
0.00%
13 Missing :warning:

...ffle/client/request/RssRegisterShuffleRequest.java
0.00%
11 Missing :warning:

.../java/org/apache/uniffle/client/record/Record.java
41.17%
10 Missing :warning:

...client/request/RssGetSortedShuffleDataRequest.java
0.00%
10 Missing :warning:

...iffle/client/request/RssStartSortMergeRequest.java
0.00%
10 Missing :warning:

...ient/response/RssGetSortedShuffleDataResponse.java
0.00%
8 Missing :warning:

... and 5 more

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2034      +/-   ##
============================================
- Coverage     52.77%   52.72%   -0.05%
- Complexity     2498     2667     +169
============================================
  Files           398      423      +25
  Lines         18135    19946    +1811
  Branches       1660     1847     +187
============================================
+ Hits           9570    10516     +946
- Misses         7981     8765     +784
- Partials        584      665      +81

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@zhengchenyu Could you add documents in an another pull request?

@zhengchenyu Could you add documents in an another pull request?

Yes, I will. When the main feature are merged, I will submit a document.


Adding DevQuotes Plugin to the Marketplace

Hey, I just made a Pull Request!
Hey! Something I made for fun! Adds a little bit of flair to your homepage or a component, hopefully inspires people when they see the quotes :)
P.S Is the logo image too big? I am not good with logos! lmk and I will try something else if no good.
Thanks team!
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Not entirely sure why its complaining about prettier :( - lmk if there is something I can do to fix it, I did run prettier but it didn't change anything.

@Parsifal-M try running it from the /microsite folder, probably some white space it's not happy with

@Parsifal-M try running it from the /microsite folder, probably some white space it's not happy with

Haha, it was a whitespace I think, ran the check locally and it no longer complains so lets see! :+1:


Can't send meesage with markdown

I am trying to send message that look like:
Commands:
/command1 - `optional argument`

ParseMode is set to ParseMode.Markdown. Message is not sent. Why is that?

Probably was because of an old version. SendTextMessageAsync does the work. Closed.


Create gh-pages branch via GitHub

Hi @Serya05!
Thanks for wanting to contribute to Bootstrap by sending this pull request!
Unfortunately, your pull request seems to have some problems:

Normal pull requests should never be against the gh-pages branch.

You'll need to fix these mistakes and revise your pull request before we can proceed further.
Once you've fixed these problems, you can either ask the maintainers to re-open this pull request, or you can create a new pull request.
Thanks!
(Please note that this is a fully automated comment.)

+1 Approve

+1 Approve

@Serya05 Please be more careful when using GitHub in the future.


release: v1.0.0a13

Signed-off-by: Jiri Kuncar<EMAIL_ADDRESS>

Coverage remained the same at 82.963% when pulling 4a616b4f2ca5d5c01b2b8605f934993b6a466d40 on jirikuncar:release-v1.0.0a13 into 519984f5b307b9f1d70f1dcb9bb5b2dbc08e2068 on inveniosoftware:master.


ModelCheckpoint misbehaves when no validation

🐛 Bug
ModelCheckpoint misbehaves when there is no validation step. Eg. if you ask to monitor for loss and save every checkpoint with save_top_k=-1 the callback will not do this silently.
To Reproduce
This line of code is a pure indicator of failure without a validation step.
Expected behavior
Callback being called after every epoch
Environment
latest pytorch-lightning
Additional context
I hoped to finetune the model a bit and aggregate the last checkpoints to do SWA. I've fixed the problem using the modified version of ModelCheckpoint
from pytorch_lightning.callbacks.model_checkpoint import ModelCheckpoint as _ModelCheckpoint

class ModelCheckpoint(_ModelCheckpoint):
    def on_epoch_end(self, trainer, pl_module):
        super().on_validation_end(trainer, pl_module)

    def on_validation_end(self, trainer, pl_module):
        pass

I can open a PR if there is no concern about this kind of change

@SeanNaren should I create a PR so this doesn't take your time?

Also, I have issues in just creating checkpoints every epoch no matter what metric is


Cannot use simd_fabs because of incorrect optimization

simd_fabs appears to optimize incorrectly: rust-lang/rust#84268
We currently use xor to mask off the sign bit, but we should still fix this.

Fixed in nightly


[Improve] : Can't drag n drop file into folder

thanks' for your idea, we will add this to our to do


Issue with 'Create a new app' action

hey @freddydk ,
I thought I would give a try to AL-GO. I followed instructions from this site. I created my repostory, but it failed with action 'Create a new app'.
I received the following errors. I presume this is something related to my emperies Github environment. Any advise?
freddydk/al-go-actions/workflowinitialize@main, freddydk/al-go-actions/readsettings@main, freddydk/al-go-actions/createapp@main, and freddydk/al-go-actions/workflowpostprocess@main are not allowed to be used in sede-res/eid-dynamic-layout. Actions in this workflow must be: within a repository that belongs to your Enterprise account, created by GitHub, verified in the GitHub Marketplace, or matching the following: ./.github/devkit-gh-actions/gatsby-preview, ./.github/workflows/branch_name, ./.github/workflows/file_changes, ./.github/workflows/filesadded, ./.github/workflows/fileschanged, ./.github/workflows/filesrenamed, ./.github/workflows/propogate-preview-uri, ./.github/workflows/pull-request-comment-trigger, 8398a7/action-slack@v3, ActionsDesk/github-actions-details-action@*, ActionsDesk/github-actions-allow-list-as-code-action@*, ActionsDesk/report-action-permissions@*, ActionsDesk/report-action-usage@*, ActionsDesk/ghec-invitations-report-action@*, ActionsDesk/ghec-report-reinv...
Best Regards,
Gintautas

You can ignore my request :)
I made a mistake of using your repo freddydk/AL-Go-PTE. Once I changed to microsoft/AL-Go-PTE, it all worked fine


[4.0] UPSTREAM: docker/distribution: 3131: Add S3 virtual hosted styl…

…e paths for custom region endpoints
Signed-off-by: Corey Daley<EMAIL_ADDRESS>

/assign @adambkaplan @dmage

/lgtm


[Feature Request] SAML for SSO

As a company owner, I need to ensure my users sign into Portainer using secured access methods.
We use SAML as a solution for our other applications, and as such wish Portainer would support this.
The oAuth integration is great, and supports SSO like we would want, but our IDP uses SAML and not oAuth.

If relevant -> SAML Jackson

+1
This lack will certainly prevent us from using Portainer as a larger scale in our company

Throwing in a vote for this as well, if that matters; SAML is a must for us.

+1

+1

+1

Throwing in a vote for this. The oauth method is limited and saml gives more flexibility


Protogalaxy verifier circuit

This PR implements the (partial) in-circuit verifier for Protogalaxy. Note that the computation of $\phi$ will be handled by the CycleFold circuit.
Depends on #94. I am also happy to rebase on the current main branch if merging #94 takes time :)

Can we review this @winderica ?

Since #94 has been merged, is this PR now ready to be reviewed @winderica ? ^^

Yes! I rebased this PR yesterday, so it no longer includes the changes in the previous one, and the conflicts should be resolved now 😄

We can merge when you want @winderica
Feel free to go for it!

Thanks @CPerezz! Let's get it merged, and I will move on to #120 then!


Misaligned popover arrows

Seems similar to #23793, although I could not reproduce the issue for tooltips, but indeed for popovers.
Here is a demo: https://codepen.io/simonihmig/pen/WdxXja. The first button triggers a popover, where you can clearly see that the arrow is misaligned.
It has left: 132px applied by popper.js. However the popover itself has a width of 276px, so the center should rather be 137px. And indeed if I set this manually to this value or left: 50% in Chrome Dev Tools, then it is perfectly aligned! So somewhere this 5px offset is causing the misalignment.

Marked as arrow because as you can see here :

Our arrow element is in the right place but due to our CSS it's not correctly centered

Right! Popper positions this correctly, just the pseudo elements (responsible for rendering the actual arrow) should be aligned to the center of .arrow, not the left edge! (or .arrow should have a zero width?)

Possible fix:
[...] .popover.bs-popover-bottom .arrow::before {
    margin-left: calc((.8rem/2 + 1px) * -1);
}


Object .gm-resize-trigger causes white block overlaying scrollable content on IE 11

On IE 11 the inserted  (.gm-resize-trigger) is displayed as a white block overlaying the scrollable content. Adding visibility: hidden removes it, but I'm not sure if that breaks the 's functionality.
Seems to work fine on 9 and 10, but I have not tested anything over IE11.

Thank you @danwhite85 , fixed in v1.4.1.

Is this possible to reopen this issue please, have the same problem in 1.4.3 cf.
Check https://github.com/noeldelgado/gemini-scrollbar/commit/65dc5b679a1955122f9b583d387488a44fb45a37
Any workaround ? Visibiltiy hidden does not do the trick?

Hi @yohannpoli, in which browser are you having the issue?
Sadly, the object cannot be replaced with a div, the object is used to register the resize event and call the update method automatically, a div does not provide a resize event but the object does.
BTW, the resizer object was added on 1.4.0 and the latest patches (up to 1.4.3) has been for this feature specifically, so, how useful is this feature for you? trying 1.3.2 is an option?. If you use 1.3.2 you will have to call the update method manually whenever you add/remove content or when the dimensions are changed, which, IMO it gives you more control over what the lib does but it can be a problem depending on how structured your code is.
Personally, I’m still using 1.3.2 on my projects, also, I’m thinking about reverting the resizer-trigger feature and publish it as v2.0.0. It all depends on how useful this feature can be.

Issue on Win 7 with IE11.
I'm already manually update the scrollbar after adding/removing content, that's fine but i have to update all Gemini scrollbars in document when i resize the window, i don't want to register and manually trigger them.
After checking your code:
var obj = document.createElement('object');
addClass(obj, [CLASSNAMES.resizeTrigger]);
obj.type = 'text/html';
var resizeHandler = this._resizeHandler.bind(this);
obj.onload = function () {
  var win = obj.contentDocument.defaultView;
  win.addEventListener('resize', resizeHandler);
};

, it seems that you attach the resize event in the object content document, why not, but it will do the same on current window, isn't it?
I will give a try to 1.3.2 version.
Thanks for your answer.

I'm already manually update the scrollbar after adding/removing content, that's fine but i have to update all Gemini scrollbars in document when i resize the window, i don't want to register and manually trigger them.

1.3.2 will do that for you, it registers the window.resize event and runs the update method.


add support for "aria-label" attributes

For accessibility, it would be great to be able to set the aria-label attribute on each shape type, presumably through an ariaLabel method.

@rbaheti some more info:

the attribute should be assigned here: https://github.com/d3plus/d3plus-shape/blob/master/src/Shape/Shape.js#L515
the default value should be defined in the constructor: https://github.com/d3plus/d3plus-shape/blob/master/src/Shape/Shape.js#L47
the user-accessible method to change the value will need to be hooked up here: https://github.com/d3plus/d3plus-shape/blob/master/src/Shape/Shape.js#L622


Cannot resize SCM bucket

re #34963
Watch the movie and relax

Do you feel the same when using the explorer open editors view?

?

You should be able to resize it, when there are more than 5 repositories.


python311Packages.stringparser: 0.6 -> 0.7

Diff: https://github.com/hgrecco/stringparser/compare/refs/tags/0.6...0.7
Changelog: https://github.com/hgrecco/stringparser/blob/0.7/CHANGES
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

20 packages built:

python310Packages.pyvisa-sim
python310Packages.pyvisa-sim.dist
python310Packages.qcodes
python310Packages.qcodes-contrib-drivers
python310Packages.qcodes-contrib-drivers.dist
python310Packages.qcodes-loop
python310Packages.qcodes-loop.dist
python310Packages.qcodes.dist
python310Packages.stringparser
python310Packages.stringparser.dist
python311Packages.pyvisa-sim
python311Packages.pyvisa-sim.dist
python311Packages.qcodes
python311Packages.qcodes-contrib-drivers
python311Packages.qcodes-contrib-drivers.dist
python311Packages.qcodes-loop
python311Packages.qcodes-loop.dist
python311Packages.qcodes.dist
python311Packages.stringparser
python311Packages.stringparser.dist


Modal opens another modal

I have 2 different modals in my code, they have different ids. When I open one of them, it automatically opens the other too. Is this a bug in the library?

Here is a video showing it https://streamable.com/9t5238

@DanDev95 I see... this happens when all modals are siblings 🤦♂
I will fix this on next update. Until then, you can put each modal inside a div to isolate them from others: https://play.tailwindcss.com/71B7rdeaX0


Replace tslint with eslint

Is your feature request related to a problem? Please describe.
TSLint has been deprecated in favor of ESLint, so obviously it should aimed at being replaced. I didn't see an issue open or closed for it, so here it is.
I should give the context that for me this is more relevant for developing a custom SFDX plugin, and I just assume that the place to file an issue would be here.
What are you trying to do
N/A
Describe the solution you'd like
N/A
Describe alternatives you've considered
N/A
Additional context
Installing tslint gives a link to this issue:
https://github.com/palantir/tslint/issues/4534

Closing this, both sfdx and sf plugin templates use eslint for linting.


crate not found in sample

where is tracing_subscriber and tracing?

It's defined in Cargo.toml, are you running the example through cargo run --example main?


Where can I find the rest of the code for the Chefbot

Specifically the code to run on the Launchpad? It should be several .c, .h and .cpp files. It seems like the code examples cited in the book are quite incomplete. We are actually trying to build a chefbot and would love a pointer to that code. We found this https://github.com/qboticslabs/Chefbot_ROS_pkg/tree/master/tiva_c_energia_code_final but it seems unauthorized and perhaps incomplete or buggy, I greatly appreciate your book and it's excellent content and would love to try and bring it over the finish line! THanks

Hi @pitosalas The full firmware code for Tiva C Launchpad and Arduio has been updated in chapter 8.

Hi @qboticslabs long time no see! Thanks.


security bits=-1

I have generated a CA and a host certificate, both with sig algo dilithium5. When I start openssl s_server with option -security_debug_verbose I get:
Using default temp DH parameters
Security callback: Certificate chain EE key=, bits=256, security bits=256: yes
Security callback: Certificate chain CA digest=dilithium5, security bits=-1: no
error setting certificate
802BF3F4147F0000:error:0A00018E:SSL routines:SSL_CTX_use_certificate:ca md too weak:ssl/ssl_rsa.c:221

The certs look ok to me. I have tried this with falcon512 and with the branches main and ossl-sigload. I have straced openssl and verified that oqs-provider is being loaded. Probably, I am doing something wrong, but I have no idea what. Thanks!

With near certainty you are using an old OpenSSL Code base. Full signature provider Support only exists in OpenSSL Master. Please let us know if the failure persists if you're using Master.

@cryptobsv Please share the output of openssl version and openssl list -providers to see whether my hunch above is correct. And/or please retry your test in the docker image at openquantumsafe/oqs-ossl3.

Yes, your hunch above was correct. I thought any openssl 3.x would do and picked 3.0.8 to work with a well-defined version. Moving to master solved this problem. Thanks again for the quick and helpful support!


use little endian int

Change Summary
Construct the uuid with a little endian representation of the int.

Related issue number
After a bit of forensic, I saw that uuid-rs does not convert the endianness of the u128.
https://github.com/pydantic/pydantic-core/pull/772
and @davidhewitt pointed out in this PR
https://github.com/uuid-rs/uuid/issues/406
I tried to use the byte attribute from the python object, so endianness would be handled by python itself. This negated the speed-up I tried to introduce in the first place.
This conversion is a no-op on little endian platforms and should fix the s390x build.
Checklist

[ X] Unit tests for the changes exist
[ ] Documentation reflects the changes where applicable
[ ] Pydantic tests pass with this pydantic-core (except for expected changes)
[ ] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Thanks for helping out!


Doesn't seem to handle multi container pods in respect to logs

A single container pod's logs seems to work fine, but it looks like you might need to add a deeper hierarchy for containers. (the equiv. of kubectl logs podname -c containername).
But so far very fun work!

I see the same problem. Need to pass kubectl the -c flag in some cases:
$ cat ~/k8s/kube-system/pod/kube-dns-v11-19scr/logs
$
$ kubectl --namespace=kube-system logs kube-dns-v11-19scr
Error from server: a container name must be specified for pod kube-dns-v11-19scr, choose one of: [etcd kube2sky skydns healthz]
$ kubectl --namespace=kube-system logs -c etcd kube-dns-v11-19scr
2016-06-29 00:29:03.644732 I | etcdserver: start to snapshot (applied: 4810481, lastsnap: 4800480)
2016-06-29 00:29:03.705105 I | etcdserver: saved snapshot at index 4810481
2016-06-29 00:29:03.705411 I | etcdserver: compacted raft log at 4805481
...

Seems like the primary issue in resolving this is deciding on the proper directory hierarchy.

Yes indeed. I like the simplicity of having a single logs file, but maybe we could add extra folders or files when there is more than one container in the pod.

Since the pods need to have a name it may be as simple as naming the logs «log_container».


Review wording of LZ checklist recommendations

Depending on #181 (since we should start with high prio items), we might want to review the wording of the different checks in the LZ checklist, since some of them could be rephrased more clearly.

New guidance should follow: https://github.com/Azure/review-checklists/blob/main/CONTRIBUTING.md#what-to-contribute

Hi @erjosito, since #792 has been merged, do you want to close this item? Regards.


Update after the release of OCaml 4.14.0

The battle of the submodule caches, but this is now green and ready to go, thanks!


--require expect.js doesn't work

hi, so when i include expect.js over the --require parameter, it doesn't work. i also tried putting this option on mocha.opts
--require expect.js
but i still getting "ReferenceError: expect is not defined"
does --require only work with should.js? or what do i missing?

Thank you @schnittstabil :cake:

@schnittstabil where is the inject script called? Does mocha have an option for passing scripts? I thought it was only Mocha.opts. Feel free to tell me to RTFM if it is in there.


Add some tests for the date handling logic

Added some tests for the internal utility functions used to handle dates:

to_tuples()
normalise_dates()

These would have prevented this glitch earlier :)
I've also made some small refactoring after introducing these tests:

to_tuples() just takes the Vec<Data> instead of the whole RegionData struct (as it's only using the data anyway)

this also simplify the test, see current version vs. original iteration where the test had to construct a RegionData even tho the logic only involves the data

to_tuples() uses Rust's Iterators and avoids mut/Vec::push()

This should resolve Issue #14

I know it's kinda cosmetic but maybe it's worth deciding on what to do in regards of the American vs. British spelling before introducing more inconsistencies :)
For example here I've named one of the tests deserialize_power_data_test() with a z 😬

thanks @xoen


Errors in HISTORY.rst

Dates and changes in HISTORY.rst do not match PyPI.org

Do not spend to much time on this, just fix it to match the information on PyPI.


[Event Request]  table 5773 "Registered Whse. Activity Line" - OnAfterSetSourceFilter

Can we please get the following event?
procedure SetSourceFilter(SourceType: Integer; SourceSubType: Option; SourceNo: Code[20]; SourceLineNo: Integer; SourceSubLineNo: Integer; SetKey: Boolean)
begin
    if SetKey then
        SetCurrentKey("Source Type", "Source Subtype", "Source No.", "Source Line No.", "Source Subline No.");
    SetRange("Source Type", SourceType);
    if SourceSubType >= 0 then
        SetRange("Source Subtype", SourceSubType);
    SetRange("Source No.", SourceNo);
    SetRange("Source Line No.", SourceLineNo);
    if SourceSubLineNo >= 0 then
        SetRange("Source Subline No.", SourceSubLineNo);
    //+Change
    OnAfterSetSourceFilter(Rec, SourceType, SourceSubtype, SourceNo, SourceLineNo, SourceSubLineNo, SetKey);
    //-Change
end;

[IntegrationEvent(false, false)]
local procedure OnAfterSetSourceFilter(var RegisteredWhseActivityLine: Record "Registered Whse. Activity Line"; SourceType: Integer; SourceSubtype: Option; SourceNo: Code[20]; SourceLineNo: Integer;  SourceSubLineNo: Integer; SetKey: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10392.


Wrong color for successful completion

It is expected to have a green color for successful completion of the job in logs. However, red color is used.
Check attached screenshot. String in red, should be actually in green.

@DreierF ☝