versioned migration throwing syntax error when creating functons in mariadb

DELIMITER $$

CREATE OR REPLACE FUNCTION gen_uuid() RETURNS VARCHAR(22)
BEGIN
    RETURN concat(
        date_format(NOW(6), '%Y%m%d%i%s%f'),
        ROUND(1 + RAND() * (100 - 2))
    );
END;$$

DELIMITER ;

This code throw syntax error at line 4 when running migration from the sql file.
atlas version: v0.6.0-f064262-canary
mariadb version:

Hey @lallenfrancisl and thanks for reporting this issue.
Currently, the way to achieve this is by setting an Atlas pragma on the top of the file to configure the delimiter used for scanning the statements in the file. For example, the following file will be scanned as one statement because the atlas:delimiter is configured but it's not used.
-- atlas:delimiter ATLAS_DELIMITER

DELIMITER $$

CREATE OR REPLACE FUNCTION gen_uuid() RETURNS VARCHAR(22)
BEGIN
    RETURN concat(
        date_format(NOW(6), '%Y%m%d%i%s%f'),
        ROUND(1 + RAND() * (100 - 2))
    );
END;$$

DELIMITER ;

In the next example, we use the atlas:delimiter in the file and the result is 2 statements:
-- atlas:delimiter ATLAS_DELIMITER

DELIMITER $$

CREATE OR REPLACE FUNCTION gen_uuid() RETURNS VARCHAR(22)
BEGIN
    RETURN concat(
        date_format(NOW(6), '%Y%m%d%i%s%f'),
        ROUND(1 + RAND() * (100 - 2))
    );
END;$$

DELIMITER ;

ATLAS_DELIMITER

CALL gen_uuid();

More examples exist in the repository https://github.com/ariga/atlas/tree/master/sql/migrate/testdata/lex.
This behavior is expected to be improved soon as Atlas now supports parsing SQL files of all the supported dialects and I plan to integrate it into the file scanning as well. Please, let me know if you need more help or if I can close this issue 🙏

Thanks @a8m for the reply. Yes I didn't realise that from their documentation that this thing is only for their command line. For other SQL noobs like me I am just putting another solution here too. The delimiter thing is not actually needed in sql and this is enough
CREATE OR REPLACE FUNCTION gen_uuid() RETURNS VARCHAR(22)
    RETURN concat(
        date_format(NOW(6), '%Y%m%d%i%s%f'),
        ROUND(1 + RAND() * (100 - 2))
    );

@a8m the solution you proposed still does not work, it will start to throw error at the place where DELIM is used
-- atlas:delimiter DELIM
CREATE TABLE `Client_m` (
    `ClientID` varchar(22) NOT NULL,
    `DataBackupDays` int NOT NULL,
    `ApiAccessKey` varchar(22) NOT NULL,
    `ClientAccessType` enum('Basic', 'Web') NULL,
    `Status` enum('0', '1', '2') NOT NULL,
    PRIMARY KEY (`ClientID`)
) CHARSET utf8mb4 COLLATE utf8mb4_general_ci; DELIM

Error: Execution had errors:
Error 1064: You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'DELIM

-- set default id value for ClientID
-- CREATE OR REPLACE TRIGGER Clie...' at line 1

Error: sql/migrate: execute: executing statement "DELIM\n\n-- set default id value for ClientID\n-- CREATE OR REPLACE TRIGGER Client_mClientID BEFORE\n-- INSERT ON `Client_m` FOR EACH ROW\n-- IF new.ClientID IS NULL OR LENGTH(new.ClientID) < 1\n--     THEN SET new.ClientID = gen_uuid();\n--     ELSE SET new.ClientID = new.ClientID;\n-- END IF;\n\n-- DELIM\n\n-- create \"Client_User\" table\nCREATE TABLE `Client_User` (\n    `UserID` varchar(22) NOT NULL,\n    `ClientID` varchar(22) NOT NULL,\n    `Status` enum('0', '1', '2') NOT NULL,\n    PRIMARY KEY (`UserID`)\n) CHARSET utf8mb4 COLLATE utf8mb4_general_ci;" from version "20220822222238": Error 1064: You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'DELIM

-- set default id value for ClientID
-- CREATE OR REPLACE TRIGGER Clie...' at line 1
make: *** [migrate-up] Error 1

Seems there was some issue since the pragma wasn't at the top so moved it top but got empty query error now
-- atlas:delimiter DELIM

-- create "Client_m" table
CREATE TABLE `Client_m` (
    `ClientID` varchar(22) NOT NULL,
    PRIMARY KEY (`ClientID`)
) CHARSET utf8mb4 COLLATE utf8mb4_general_ci;
DELIM

-- set default id value for ClientID
DELIMITER $$

CREATE OR REPLACE TRIGGER Client_mClientID BEFORE
INSERT ON `Client_m` FOR EACH ROW
IF new.ClientID IS NULL OR LENGTH(new.ClientID) < 1
    THEN SET new.ClientID = gen_uuid();
    ELSE SET new.ClientID = new.ClientID;
END IF;$$

DELIMITER ;
DELIM

Error: sql/migrate: execute: executing statement "" from version "20220822222238": Error 1065: Query was empty
make: *** [migrate-up] Error 1

Hey @lallenfrancisl, it looks like the Atlas delimiter conflicts with the MySQL DELIMITER keyword.
Please, update to the latest master and checkout the docs on how to use a different delimiter instead (e.g. \n\n\n): https://atlasgo.io/versioned/new#custom-statements-delimiter
I also added a few additional examples to our tests, you can see them here: https://github.com/ariga/atlas/tree/master/sql/migrate/testdata/lex

Another point that I think can be added to the docs is that the DELIMITER keyword is not needed when interacting with the server directly (not the MySQL CLI). See https://dev.mysql.com/doc/refman/8.0/en/stored-programs-defining.html

Another point that I think can be added to the docs is that the DELIMITER keyword is not needed when interacting with the server directly (not the MySQL CLI). See https://dev.mysql.com/doc/refman/8.0/en/stored-programs-defining.html

Yeah but I think the driver needs it, not working without it

Hey @lallenfrancisl 👋
Can you please provide an example to reproduce this? I just ran the following example on the master version and it worked:
Migration file:
-- atlas:delimiter \n\n\n

CREATE PROCEDURE dorepeat(p1 INT)
BEGIN
    SET @x = 0;
    REPEAT SET @x = @x + 1; UNTIL @x > p1 END REPEAT;
END;

CALL dorepeat(1000);

atlas migrate apply output:
Migrating to version<PHONE_NUMBER>5648 (1 migrations in total):

  -- migrating version<PHONE_NUMBER>5648
    -> CREATE PROCEDURE dorepeat(p1 INT)
BEGIN
    SET @x = 0;
    REPEAT SET @x = @x + 1; UNTIL @x > p1 END REPEAT;
END;
    -> CALL dorepeat(1000);
  -- ok (19.935ms)

  -------------------------
  -- 26.544929ms
  -- 1 migrations
  -- 2 sql statements

MySQL status:
mysql> select routine_definition from information_schema.routines where routine_name='dorepeat';
+---------------------------------------------------------------------------------+
| ROUTINE_DEFINITION                                                              |
+---------------------------------------------------------------------------------+
| BEGIN
    SET @x = 0;
    REPEAT SET @x = @x + 1; UNTIL @x > p1 END REPEAT;
END |
+---------------------------------------------------------------------------------+
1 row in set (0.00 sec)

As I mentioned above, I think the DELIMITER is not something the MySQL server understands and is used only by the CLI.

I will give you an example to reproduce this in the evening, sorry should have put it here in the beginning

No worries, and sorry for the inconvenience 🙏🏻

Closing, as this issue was resolved a long time ago. Note that we also added support for the DELIMITER keyword used by MySQL client, see https://atlasgo.io/versioned/new#using-the-delimiter-command-used-by-mysql-client.
Feel free to reopen it if you still have troubles with it, and you're also welcome to join our Discord community.


Mobile compatibility is dogshit

See title.


[Enhancement] sequential run cases that modifies FE configs (#27126)

avoid global config change affects other concurrent case result

(cherry picked from commit fc928ad12e8661bce0b600c4adc08b12af07370d)
Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[X] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[X] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.1
[ ] 3.0
[ ] 2.5
[ ] 2.4

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)


[doc/keras] predict of tf.keras.models.Sequential is rendered incorrectly

Doc Link: https://www.tensorflow.org/api_docs/python/tf/keras/models/Sequential#predict

Describe the documentation issue
The argument list is not rendered correctly.
See it below
Arguments:
x: Input samples. It could be: - A Numpy array (or array-like), or a list of arrays (in case the model has multiple inputs). - A TensorFlow tensor, or a list of tensors (in case the model has multiple inputs). - A tf.data dataset or a dataset iterator. - A generator or keras.utils.Sequence instance. * batch_size: Integer or None. Number of samples per gradient update. If unspecified, batch_size will default to 32. Do not specify the batch_size is your data is in the form of symbolic tensors, dataset, dataset iterators, generators, or keras.utils.Sequence instances (since they generate batches). * verbose: Verbosity mode, 0 or 1. * steps: Total number of steps (batches of samples) before declaring the prediction round finished. Ignored with the default value of None. * max_queue_size: Integer. Used for generator or keras.utils.Sequence input only. Maximum size for the generator queue. If unspecified, max_queue_size will default to 10. * workers: Integer. Used for generator or keras.utils.Sequence input only. Maximum number of processes to spin up when using process-based threading. If unspecified, workers will default to 1. If 0, will execute the generator on the main thread. * use_multiprocessing: Boolean. Used for generator or keras.utils.Sequence input only. If True, use process-based threading. If unspecified, use_multiprocessing will default to False. Note that because this implementation relies on multiprocessing, you should not pass non-picklable arguments to the generator as they can't be passed easily to children processes.

It should be rendered in a list/item format.

PR submitted already #27596

@csukuangfj I am closing this issue as the PR was already merged and I have looked into the source code that show update. TF website will be updated soon and then you can see correct rendering. Thanks!


Dark Mode tutorial causes error in current version of Next.js:  "DOMException: Failed to execute 'add' on 'DOMTokenList':"

I created a new Next.js project today following the shadcn docs' installation instructions, and then attempted to add a dark theme toggle by following the shadcn docs' dark mode instructions.
After adding ThemeProvider to my layout, my browser displayed the following error:
"DOMException: Failed to execute 'add' on 'DOMTokenList': The token provided ('{"breakpoints":{"keys":["xs","sm","md","lg","xl"],"val...
... contains HTML space characters, which are not valid in tokens.
    at eval (webpack-internal:///(app-pages-browser)/./node_modules/next-themes/dist/index.module.js:8:1169)
    at eval (webpack-internal:///(app-pages-browser)/./node_modules/next-themes/dist/index.module.js:8:1922)
    at commitHookEffectListMount (webpack-internal:///(app-pages-browser)/./node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js:20004:23)
    at commitHookPassiveMountEffects (webpack-internal:///(app-pages-browser)/./node_modules/next/dist/compiled/react-dom/cjs/react-dom.development.js:22057:7)
..."

Specifically, adding attribute="class" to the ThemeProvider is what causes the issue, although simply removing this prop does not seem to be a solution.

Hey!
I just spent the whole week trying to build a project using Next13 and next-themes and it doesn't seem to work in anyway :/
Couldn't find a solution, so I would be glad if anyone could help

Hey! I just spent the whole week trying to build a project using Next13 and next-themes and it doesn't seem to work in anyway :/ Couldn't find a solution, so I would be glad if anyone could help

If you're just want to implement a dark/light mode, the following works:
  const toggleDarkMode = () => {
    document.documentElement.classList.toggle('dark')
  }

To make it dark theme by default, you add the "dark" class to the html tag in your layout.

@xossiane have you checked this

@xossiane @AnthonySchneider2000 have you checked this

I hadn't seen that. I've cloned it and next-themes seems to work correctly even after migrating the project to Next.js 14.
However, I will not close the issue as the shadcn docs for using next-themes still cause this issue.

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


Embedded video

On video embeds, when the user scrolls down to see the transcript, the user cannot see the video player.

When the user is viewing the video player, the user can only view two lines of the transcript.

Also, not sure why this video embed has a larger player with no title on the right side like the others:

@mccalluc: to fix this, I would propose just removing the transcript from embedded video iframe and keeping only the player, title and Open Vault description.

@caseyedavis12 : As you are filing bugs it would help immensely to include the URLs of relevant pages so I can see what you're seeing.

@mccalluc: Sorry, the issue can be viewed at /exhibits/advocates

From: mccalluc<EMAIL_ADDRESS>Sent: Monday, February 29, 2016 11:31 AM
To: WGBH/openvault3
Cc: Casey Davis
Subject: Re: [openvault3] Embedded video (#308)
@caseyedavis12https://github.com/caseyedavis12 : As you are filing bugs it would help immensely to include the URLs of relevant pages so I can see what you're seeing.

Reply to this email directly or view it on GitHubhttps://github.com/WGBH/openvault3/issues/308#issuecomment-190278542.

Tried a variety of screen widths, and not able to reproduce the issue seen in the second comment.
Please file each bug separately. This is not a good place for meandering discussions.
Will remove the transcript... hold on.


Add url scope

Related to issue https://github.com/pokey/cursorless-vscode/issues/426

[x] Yes, I have added tests

I wouldn't be surprised if you guys want me to take another approach and not touch the non-whitespace scope. I would be perfectly fine with that. I just wanted you guys to see the idea so we can have the debate before we decide.
I recommend you go through the commits one by one; that will tell the most coherent story.

I'm definitely in favor of this solution. This enables us to add user defined scopes in the future as they can just contribute the regex.

Also looks like I broke the black linting somehow, not sure how I managed to do that
Run psf/black@stable
Run # Exists since using github.action_path + path to main script doesn't work because bash
--- cursorless-talon/src/vendor/jstyleson.py	2022-01-31 13:20:01.512373 +0000
+++ cursorless-talon/src/vendor/jstyleson.py	2022-01-31 13:20:07.380250 +0000
@@ -103,11 +103,11 @@
         normal = True
         for i in range(former_index, len(json_str)):
would reformat cursorless-talon/src/vendor/jstyleson.py
             result_str[i] = ""

     # Show respect to original input if we are in python2
-    return ("" if isinstance(json_str, str) else u"").join(result_str)
+    return ("" if isinstance(json_str, str) else "").join(result_str)

 # There may be performance suffer backtracking the last comma
 def _remove_last_comma(str_list, before_index):
     i = before_index - 1

Oh no! 💥 💔 💥
1 file would be reformatted, 42 files would be left unchanged.

Could that be one of the fixtures?

One other thing I wanted to mention - I think it would be a good idea to write something about how to get set up to contribute, especially now we have a monorepo.
Here's the workflow I did:

Clone the repo
Delete the cursorless-talon directory I had in my talon user directory, and make a symlink to the one in the newly cloned directory.

ln -s .talon/user/cursorless-talon ~/Desktop/cursorless-vscode/cursorless-talon

Open cursorless-vscode with vscode, and use Run Extension and Extension Tests.

Also, if we could find a way to limit the number of tests that run at a given time that would be really helpful. As a hacky solution, I deleted most of the tests while I was working, other than the ones relevant to stuff I was touching.

Looks good to me! For future, we try to keep the test docs minimal, and as @AndreasArvidsson we try to use "clear" instead of take. Not a showstopper tho
And yeah I do like magic regexes hoisted to the top 😊. Not too bad in this case tho because the context makes it super clear, so I wouldn't block the PR on those grounds if you like this flavour better here

[ ] Would be good to add a test where start of selection is in one regex and end is in another

Great stuff tho!

Also looks like I broke the black linting somehow, not sure how I managed to do that

Run psf/black@stable

Run # Exists since using github.action_path + path to main script doesn't work because bash

--- cursorless-talon/src/vendor/jstyleson.py	2022-01-31 13:20:01.512373 +0000

+++ cursorless-talon/src/vendor/jstyleson.py	2022-01-31 13:20:07.380250 +0000

@@ -103,11 +103,11 @@

         normal = True

         for i in range(former_index, len(json_str)):

would reformat cursorless-talon/src/vendor/jstyleson.py

             result_str[i] = ""

     # Show respect to original input if we are in python2

-    return ("" if isinstance(json_str, str) else u"").join(result_str)

+    return ("" if isinstance(json_str, str) else "").join(result_str)

 # There may be performance suffer backtracking the last comma

 def _remove_last_comma(str_list, before_index):

     i = before_index - 1

Oh no! 💥 💔 💥

1 file would be reformatted, 42 files would be left unchanged.

Could that be one of the fixtures?

Maybe black changed their rules around empty Unicode strings? I'd just run black locally and commit 🤷

One other thing I wanted to mention - I think it would be a good idea to write something about how to get set up to contribute, especially now we have a monorepo.
Here's the workflow I did:

Clone the repo

Delete the cursorless-talon directory I had in my talon user directory, and make a symlink to the one in the newly cloned directory.

ln -s .talon/user/cursorless-talon ~/Desktop/cursorless-vscode/cursorless-talon

Open cursorless-vscode with vscode, and use Run Extension and Extension Tests.

Yeah that's my workflow too. Feel free to paste that somewhere 👍

Also, if we could find a way to limit the number of tests that run at a given time that would be really helpful. As a hacky solution, I deleted most of the tests while I was working, other than the ones relevant to stuff I was touching.

https://github.com/pokey/cursorless-vscode/issues/293


[AMD] Fix computeBasePtr type specification

There was a bug in shared memory base pointer calculation which misuses gep build function and pass ptr type as sharedmem's element type. Fixed the bug.

Can we add a lit test for this?

Good point, I'll have a look. Preferably having offsets and different data types.

Unfortunately it was not easy to add various test for this, the function is used in the middle of emitting lots of LLVM ops and can't be separately controlled from IR-level. Just added a basic test.


Make traits for zip files

It's currently very difficult to have a list or struct of ZipArchives with different underlying reader types (eg. some file-backed, some memory-backed). This introduces new traits for the structs that have generic arguments.
Eg, old:
struct MyStructThatHasAZip<T> {
   zip : Box<ZipArchive<T>>;
}

vs new:
struct MyStructThatHasAZip {
   zip : Box<ZipArchiveRead>;
}

This would be a breaking API change since any method use would require importing the relevant trait instead of the archive type. This is a rust limitation/"feature"
Possible improvements:

Make ZipWriter implement ZipArchiveRead to allow reading back data

I understand the idea, but is this not a bit a niche problem? For the small use-cases that do want this functionality, it could be done by creating an own trait.
Feel free to disagree, I would love to hear some more information.

There's also the option of boxing the reader used by zip if you're looking for a concise solution.
struct MyStructThatHasAZip {
   zip : ZipArchive<Box<dyn Read + Seek>>;
}

This PR is quite stale now, and won't get attention in the short term. Feel free to submit a new one if you feel there's still a need for this 😀


Relating sent entities to ones loaded from the bsp

I've been writing a BSP importer / demo viewer in Unity, although I've hit a bit of a wall. It seems like some information for some entities isn't sent in a PacketEntities message. For example, the model / model index for the doors on de_nuke, or the Z position for most of those doors.
Should I be finding that sort of information by matching the networked entity with one loaded from the BSP on my client? If so, how can I determine which entity from the map is associated with one that enters my PVS?
Thanks!

I might have figured this out - I think I need to use the values from the instancebaseline string table when first reading an entity that enters the PVS if that table contains a matching entry for the entity's class ID.


[Theming] Add Catppuccin as a theme

Catppuccin is a well-defined pastel theme, and I feel it could be a nice addition to the Aurora framework, just as a theme option. I appreciate that this deviates from the traditional Java feel of these desktop apps, but it can illustrate a more modern aesthetic for the user interface.

There are many apps that integrate Catppuccin as a theme due to its popularity, such as FreeTube.
If this isn't suitable to add to Aurora however, it would be a big help if you could give some guidance on creating this myself, thanks!

See documentation under the theming section - it talks about the basic building blocks (color schemes, color scheme association kinds, color scheme bundles, painters) and how to define a whole skin.
And you can look at the source code of individual Aurora skins to see how various parts come together.


Using @focus tag makes all tests without the tag fail in cypress with 'This.skip is not a function'

Current behavior
When using any version of the preprocessor above 2.3.1 and using the @focus tag, any test without the tag will:

Run any code within beforeEach(), essentially starting the test.
Fail once it starts the first scenario step with the following error:

Desired behavior
When running tests with the @focus tag, skip the entire scenario, do not run any code for that scenario. do not throw the error.

Versions

Cypress version:
Preprocessor version: 2.4 and above
Node version:12.14.1

Quite same here.
Using TypeScript 4.0.2, cypress 5.2.0, cypress-cucumber-preprocessor 2.5.5.

I'm also experienced this issue since version 2.5.5 and later.
Please fix it @focus is nice feature to make sure scenario can running well

Due to personal reasons, the previous maintainers of this package are stepping down and handing the reigns over to me, a long-time contributor to the project and a user of it myself. This is a responsibility I'm very excited about. Furthermore, I'd like to thank @lgandecki ++ for all the work that they've done so far.
Read more about the transfer of ownership here.
The repository has however moved and all outstanding issues are being closed. This is not a reflection of the perceived importance of your reported issue. However, if after upgrading to the new version, you still find there to be an issue, feel free to open up another ticket or comment below. Please make sure to read CONTRIBUTING.md before doing so.


abnormal cell cycle arrest with condensed chromosomes

is a synonym of
abnormal cell cycle arrest in mitotic M phase
shoudln't it be
abnormal cell cycle arrest in mitotic metaphase

I don't think so - chromosome condensation starts in prophase, and decondensation doesn't start until telophase.


Remove netty server query support from presto-pinot-driver to remove pinot-core and pinot-segment-local dependencies

Remove netty server query support from presto-pinot-driver to remove pinot-core and pinot-segment-local dependencies

Once this is in place, we can keep the JDK8 comparability for pinot-spi/pinot-segment-spi/pinot-common and move on all other components to JDK 11+

Codecov Report

Merging #9455 (e7e535d) into master (89711a5) will increase coverage by 0.11%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #9455      +/-   ##
============================================
+ Coverage     68.42%   68.53%   +0.11%
+ Complexity     5102     4784     -318
============================================
  Files          1902     1904       +2
  Lines        101572   101661      +89
  Branches      15421    15433      +12
============================================
+ Hits          69496    69676     +180
+ Misses        27154    26986     -168
- Partials       4922     4999      +77

Flag
Coverage Δ

integration1
25.94% <ø> (?)

integration2
?

unittests1
67.10% <ø> (-0.02%)
:arrow_down:

unittests2
15.46% <ø> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...t/core/plan/StreamingInstanceResponsePlanNode.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ore/operator/streaming/StreamingResponseUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...server/starter/helix/SegmentReloadStatusValue.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../operator/blocks/results/MetadataResultsBlock.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ager/realtime/PeerSchemeSplitSegmentCommitter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...urces/ServerReloadControllerJobStatusResponse.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...he/pinot/common/utils/grpc/GrpcRequestBuilder.java
0.00% <0.00%> (-90.91%)
:arrow_down:

...ator/streaming/StreamingSelectionOnlyOperator.java
0.00% <0.00%> (-90.00%)
:arrow_down:

...he/pinot/core/plan/StreamingSelectionPlanNode.java
0.00% <0.00%> (-88.89%)
:arrow_down:

...che/pinot/common/auth/StaticTokenAuthProvider.java
0.00% <0.00%> (-87.50%)
:arrow_down:

... and 247 more

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


Distributions: interactive HTML portion

This is the first step of updating the HTML to support allowing users to directly select the distributions we produce with P2RA.
It displays the intended interface and reacts appropriately when people click on things, but the simulation part is not coded yet and it doesn't work end to end.
You can play with this live at https://data.securebio.org/simulator/index-ww.html
The goal here is to review the interface we present to users before putting in the work to make it all functional.
@dp-rice for review I think it's fine to look only at the description of what is supposed to be happening and the user interface, and ignore the JS that makes it interactive.

It's not obvious that when you select a pathogen radio button that you're selecting the RA as opposed to just the genome length

I think this is now fixed in 992abb4

When you click on one of the swabs or custom it expands a relative abundance box. When you go from a wastewater sample to custom, this box is populated by a string that presumably is the key to get the P2RA distribution for that study/virus. It feels wrong to expose this to the user without explaining what it is (or how to customize it).

Good point.  These aren't customizable, so made e20e97a to clear the box when selecting Custom.

@dp-rice ready for re-review


Cannot destroy namespace with Terraform CR as ServiceAccount is removed before runner created

Hello, I have a potentially odd use case but it seems like it should work, digging into the details we understand why, but would like input on how we can do this before we dive into creating our own work around.
We would like to host the Terraform CR in a namespace with the deployment it is tied to.  We started trying out tf-controller recently and found that while the Terraform CRs have a finalizer on them to ensure the cloud resources are destroyed, there is no mechanism for the ServiceAccount and k8s happily deletes it before the runner pod even has a chance to spin up.
I'm not sure this is technically the problem of tf-controller but it would be nice if there was a way to get tf-controller to keep the requisite service account around until it isn't needed anymore. I see there is code for adding finalizers to dependent Terraform resources, perhaps we could hijack that to add a finalizer to the ServiceAccount?

Sorry to be annoying, but what does the team/wild-watermelon label mean on this issue? Is there a way to follow this further or see this as part of a team's backlog?

Hi @rattboi, that's our internal team label.
We now have an umbrella issue to address the resource-deletion process of the Terraform Controller as a whole here: https://github.com/weaveworks/tf-controller/issues/755

We don't support deleting the namespace right now, and I don't see how can we do that in the near future as the ServiceAccount is created during a helm release based on allowedNamespaces. The controller does not create ServiceAccounts (and I think it shouldn't have permissions to create ServiceAccounts) therefore it can't recreate the deleted resource.
@chanwit: I don't think we can resolve this with the current runner logic (independent from BLOB cache), do you see any way to handle it properly and still in a secure way?


Create new PhotonStructDecoder

Closes https://github.com/Mechanical-Advantage/AdvantageScope/pull/181
Still TODO:

[ ] Verify schema hashes
[ ] Actually plumb this into the rest of the code
[x] Allow PhotonStructDecoder to use normal Struct types as nested members
[ ] Guarantee that float64 x; and double x; hash to the same thing

It's not pretty, but for an MVP it's not bad

https://github.com/Mechanical-Advantage/AdvantageScope/pull/181

I can't drag and drop Translation/Transform3ds onto the field, but it seems like all other decoding works. the bytes consumed member also needs to be trimmed off.
https://github.com/user-attachments/assets/5d2ac990-479b-4545-9616-df84dd5b814d

This should be like 90% of the way to feature complete, but I haven't had a chance to go through and clean-up code yet.

I updated to the latest commit on main, and did a little bit of cleanup. Is there anything major still missing? If not, I'm planning an AdvantageScope release later today and it would be nice to include this feature given the PhotonVision release yesterday.

I don't think there are any blocking features missing. I did want to add schema hash checking, but if I'm getting the schema UUID straight from the source already I don't see a reason to double verify it. I'd like to test once more with photon beta 1 before merging out of paranoia then I'm good to go (I'll be out at work for the next 10 hours fya)
Thank you for cleaning this up!

No problem, I'm not planning to release anything until later tonight anyway.

I did a quick test with the PhotonVision beta running in test mode, and it seems to look OK? I'll probably go ahead and merge unless you have an objection.

Works for me in test mode here. Seeeeennnnnndddd it


[Bug]: Images not distributed correctly across columns if portion of content added below visible area of screen.

Description
Images not distributed correctly across columns if more content added below the currently visible area of screen.
Reproduction
I am displaying a gallery of images where script adds another 25 images if user has scrolled to the very last loaded image.
Initial 2 portions load fantastic and they got distributed across multiple columns as masonry wall should be.
Problem starts if script triggers loading another 25 images and majority of them are not displayed on visible portion of screen (e.g. user has to scroll down more to see them). Images are no more distributed correctly across the columns. Some columns advance by 3, 4 or even 5 images, while others remain empty gaps resulting in exageratedly uneven bottom.
Initially I blamed the way how i added the images, i blamed Quasar framework for its components etc. However, I switched to simple  html tags to display images and simple load more button at the end of currently loaded portion. Problem starts to happen exactly at point when biggest portion of newly loaded images appear below the currently visible screen area. Then i tested it by super zooming out the screen (making everything super small  so i have bigger screen area). And guess what - portions that were problematic before - now do render correctly, but .. again, as soon as i go out of the visible screen area - problems start with those portions.
I have also tried to set my loading script to add only +1 image instead of 25 in my portion load script. In this case everything goes smoothly.
So these two things together => loading images by portions + loading a portion so user has to scroll down to see it, those two things together produce this weird behavior.
I have also attached a video showing it in action (had to blur the pics though). And here i set 10 pics per fetch so it takes several fetches to fill the initial visible screen. Then extra fetch happens when i scroll down and reach the end.
https://www.youtube.com/watch?v=DpfZDot0NXA
Additional context
No response
Preferences

[ ] I want to be assigned to and work on this issue myself

And finally, running all these tests for more than an hour - several times the problem appeared in first few portion fetches, so at the end it might even not be caused by rendering additional portion off-scrreen.
Would be interesting to see your online demo outputing packs of N images rather than adding a single div at a time.

Thank you so much for adding 10 item option.
Looked again at your html output - each div has a known height you set in your div's style.
So i wonder, is it supposed to work with images where each element is <img style="width:100%".. > without knowing images height or width. It should just fill the 100% of column horizontally, and height should turn out without losing proportions?
If possible, could you try this scenario on your demo code? I believe it's specific to images, how fast they load and how the size is getting calculated.. and current demo may not even reveal this problem since height is pretty much hardcoded in each div.
Thanks

Alright, the magic moment has arrived. After nearly a month of going back and forth through different ideas and trying to find out how it could work with images - finally I found a solution. Yes, that involves knowing your pic dimensions!
The magic ingredient to style that should be applied to img is not just width: 100% (that will lead to the problem described above), but this:
style="width:100%; aspect-ratio: sizex/sizey"
where sizex and size y is your actual image width and height. And once the system knows the aspect ratio, seems it is not hard for it to calculate actual dimensions on responsive output.
I really hope the author will include this extra finding in the docs so other users trying to make it work with images wont have to waste precious time finding this truth by themselves.
Cheers!

Thank you very much for the investigation!
I added a section about the limitations concerning dynamically changing item height to the README.

:tada: This issue has been resolved in version 3.0.38 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Plugin health check failed since upgrade to Grafana 10

Hello,
it seems the plugin is not compatible with Grafana 10.
Since the upgrade, I have the following error  when I save the datasource :
"Plugin health check failed"
Is  there a way to fix without downgrade grafana version ?
Thanks for your help.

@jelmokri, what plugin version do you use? v3.1.0 is compatible with v10 https://github.com/grafana/clickhouse-datasource/blob/main/CHANGELOG.md

Hello, I use the 3.1.0 version of the plugin with grafana 10.0.1
When I save the datasource , I have the error above and so all the panels depending of this datasource not work anymore

mark,me too

Hi @jelmokri, @grafana/observability-metrics
I've tested 3.1.0 against Grafana 10 today and I can successfully connect to a ClickHouse datasource. Do you have any more details on the errors you're seeing?

@aangelisc I think we (@grafana/observability-metrics) tagged by mistake? Do you need anything specific from our side?

@itsmylife I'm not sure why you were tagged but yep that's a mistake! Don't worry nothing needed from you 😊

You can close the issue.
I have fixed it by using full FQDN in Server address  of the datasource configuration

I have fixed it by using full FQDN in Server address of the datasource configuration

@jelmokri what server address and port did you use before? We can implement a proper validation

before : server:port
after:    FQDN:port


Implement e2e application reuse

Which issue this PR addresses:
If it works, fixes ARO-5822
What this PR does / why we need it:

for e2e, attempts to utilize a pool of existing EntraID applications as cluster service principals. As of now, this PR is just testing an approach.

Test plan for issue:

[ ] E2E works

Is there any documentation that needs to be updated for this PR?

Yes, E2E documentation (todo)

How do you know this will function as expected in production?

We'll need to make equivalent changes to our release E2E pipeline

Additional notes

I added a new MS Graph endpoint to the generated SDK. This change is isolated to the first commit on this branch

/azp run e2e


Allowing --std=c++11 with GCC ARM tools.

Is there any chance there could be an option to allow --std=c++11 while compiling with the GCC_ARM tools?

@sg- @theotherjimmy @geky @bogdanm

@theotherjimmy Your PR here would allow any arbitrary flags to be used, correct? https://github.com/ARMmbed/mbed-os/pull/2431

I personally would be very excited to adopt C++11. However, IAR does not support C++11 and is stuck at C++03 (here). We simply can't adopt C++11 in mbed and related ecosystem without losing support for IAR.
However, applications are a different story. If an application writer intends for the application to only be used on specific toolchains, adopting C++11 in that application would be acceptable. Unfortunately, that would mean the application is not longer compatible with the rest of the mbed ecosystem or it would come with the false promise of support for IAR.
For this reason, we unfortunately can't encourage C++11. I do think we should allow it through @theotherjimmy's patch for the functionality, because in @theotherjimmy's patch it is clear you are working out of the mbed ecosystem.

I'd be willing to assume the risk of it not working with IAR, at least at the application level.

Unfortunately there is currently no route to enabling C++11 short of modifying the tools (this line, if you were wondering).
@theotherjimmy has been working on exposing compiler flags in a less hacky manner:
https://github.com/ARMmbed/mbed-os/pull/2431

mbed OS will not adopt C++11 as a default in the foreseeable future. When you export a makefile or into another IDE you're free to modify the build flags however you'd like (no compatibility guarantees). We're currently supporting C99 and C++03.


Bazel plugin for IntelliJ IDEA 2020.2.1 (Ultimate Edition)

Build #IU-202.6948.69, built on August 24, 2020
I do not find the compatibles in the source to compile it from. Also, Jetbrains plugins repository is incompatible. Please advise.

You will need to downgrade to 2020.1 at this time.

@sseveran Is there any notes anywhere on what needs to be done to achieve 2020.2+ compatibility?

Duplicate of #1998

@ve6yeq Not that I know of. I believe we are just waiting for Google's internal tools team to update the plugin and they tend to lag a few months.

Closing this as a duplicate of #1998.


Stop emitting unredacted thrift responses by default

Over in dbt-databricks we want to wire-in pysql logs but have found that the complete content of every thrift response is included in the outputted logs. These are tough to redact with a regex because thrift doesn't implement a deterministic kind of serialisation. Instead, the raw bytes are interpreted as utf-8 strings. Rather than fix this in dbt-databricks, we should modify pysql to separate the unredacted logs under a separate logger that most tools can simply ignore.

PR incoming with this change.


Event cleanup

Remove unused code, consolidate event enrichment code (less variables -> just fill the Event type attributes, less duplicated implementations), add option to filter watched events.

:+1: :shipit: :octocat: :dango:

With my limited knowledge of the codebase, :+1: looks good. TestEventFilter could use a few more test cases though.


Add hover animation to scale book images

The book images are very small.
I would like to use a motion library like , framer motion to scale the image to be larger on hover.
I would like to work on this and maybe use the same library to add animations to other parts of the page as well.

This issue has been resolved #55


Enable Cloudflare proxying for apex production domain

Ensure re-direct rules for https + www work correctly

https://foxglove.dev/blog/how-to-configure-vercel-with-cloudflare

Rule deployed at 13:30 BST

certain issues seen with cloudflare optimizations so rolled back


abcmidi: 2018.02.22 -> 2018.03.08

Semi-automatic update generated by https://github.com/ryantm/nix-update tools. These checks were done:

built on NixOS
ran /nix/store/3r7nwg28yy402gl01m9b14h5j865365g-abcMIDI-2018.03.08/bin/abc2abc -h got 0 exit code
ran /nix/store/3r7nwg28yy402gl01m9b14h5j865365g-abcMIDI-2018.03.08/bin/abcmatch -h got 0 exit code
ran /nix/store/3r7nwg28yy402gl01m9b14h5j865365g-abcMIDI-2018.03.08/bin/mftext -h got 0 exit code
ran /nix/store/3r7nwg28yy402gl01m9b14h5j865365g-abcMIDI-2018.03.08/bin/mftext --help got 0 exit code
ran /nix/store/3r7nwg28yy402gl01m9b14h5j865365g-abcMIDI-2018.03.08/bin/mftext help got 0 exit code
ran /nix/store/3r7nwg28yy402gl01m9b14h5j865365g-abcMIDI-2018.03.08/bin/mftext -V and found version 2018.03.08
ran /nix/store/3r7nwg28yy402gl01m9b14h5j865365g-abcMIDI-2018.03.08/bin/mftext -v and found version 2018.03.08
ran /nix/store/3r7nwg28yy402gl01m9b14h5j865365g-abcMIDI-2018.03.08/bin/mftext --version and found version 2018.03.08
ran /nix/store/3r7nwg28yy402gl01m9b14h5j865365g-abcMIDI-2018.03.08/bin/mftext -h and found version 2018.03.08
ran /nix/store/3r7nwg28yy402gl01m9b14h5j865365g-abcMIDI-2018.03.08/bin/mftext --help and found version 2018.03.08
ran /nix/store/3r7nwg28yy402gl01m9b14h5j865365g-abcMIDI-2018.03.08/bin/yaps -h got 0 exit code
ran /nix/store/3r7nwg28yy402gl01m9b14h5j865365g-abcMIDI-2018.03.08/bin/abc2midi -h got 0 exit code
ran /nix/store/3r7nwg28yy402gl01m9b14h5j865365g-abcMIDI-2018.03.08/bin/midi2abc -h got 0 exit code
ran /nix/store/3r7nwg28yy402gl01m9b14h5j865365g-abcMIDI-2018.03.08/bin/midi2abc --help got 0 exit code

Success on x86_64-linux (full log)
Attempted: abcmidi
Partial log (click to expand)
shrinking /nix/store/4vi7rxbzxx6csfk2ghimwiykp758kww3-abcMIDI-2018.03.08/bin/abc2midi
shrinking /nix/store/4vi7rxbzxx6csfk2ghimwiykp758kww3-abcMIDI-2018.03.08/bin/midi2abc
shrinking /nix/store/4vi7rxbzxx6csfk2ghimwiykp758kww3-abcMIDI-2018.03.08/bin/midicopy
shrinking /nix/store/4vi7rxbzxx6csfk2ghimwiykp758kww3-abcMIDI-2018.03.08/bin/yaps
gzipping man pages under /nix/store/4vi7rxbzxx6csfk2ghimwiykp758kww3-abcMIDI-2018.03.08/share/man/
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/4vi7rxbzxx6csfk2ghimwiykp758kww3-abcMIDI-2018.03.08/bin
patching script interpreter paths in /nix/store/4vi7rxbzxx6csfk2ghimwiykp758kww3-abcMIDI-2018.03.08
checking for references to /tmp/nix-build-abcMIDI-2018.03.08.drv-0 in /nix/store/4vi7rxbzxx6csfk2ghimwiykp758kww3-abcMIDI-2018.03.08...
/nix/store/4vi7rxbzxx6csfk2ghimwiykp758kww3-abcMIDI-2018.03.08

Success on aarch64-linux (full log)
Attempted: abcmidi
Partial log (click to expand)
shrinking /nix/store/qs6d9qfpp7nv6alcq0ihf0k3f5zqp3xd-abcMIDI-2018.03.08/bin/mftext
shrinking /nix/store/qs6d9qfpp7nv6alcq0ihf0k3f5zqp3xd-abcMIDI-2018.03.08/bin/abc2abc
shrinking /nix/store/qs6d9qfpp7nv6alcq0ihf0k3f5zqp3xd-abcMIDI-2018.03.08/bin/midi2abc
shrinking /nix/store/qs6d9qfpp7nv6alcq0ihf0k3f5zqp3xd-abcMIDI-2018.03.08/bin/abc2midi
gzipping man pages under /nix/store/qs6d9qfpp7nv6alcq0ihf0k3f5zqp3xd-abcMIDI-2018.03.08/share/man/
strip is /nix/store/3zq400fri5dv7d30lpxlqm2v9y1iis6j-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/qs6d9qfpp7nv6alcq0ihf0k3f5zqp3xd-abcMIDI-2018.03.08/bin
patching script interpreter paths in /nix/store/qs6d9qfpp7nv6alcq0ihf0k3f5zqp3xd-abcMIDI-2018.03.08
checking for references to /build in /nix/store/qs6d9qfpp7nv6alcq0ihf0k3f5zqp3xd-abcMIDI-2018.03.08...
/nix/store/qs6d9qfpp7nv6alcq0ihf0k3f5zqp3xd-abcMIDI-2018.03.08

@matthewbauer @ryantm Why am I not mentioned as maintainer?


Bug: In desktop after click on changing theme it is close automatically

What happened?
In desktop after click on changing theme it is close automatically
Add screenshots

What browsers are you seeing the problem on?
Chrome
Record

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I agree to follow this project's Code of Conduct
[X] I want to work on this issue

Hello @Saurav-Pant please assign this issue to me under gssoc 24

can you please explain this issue in detail @kapilG0

can you please explain this issue in detail @kapilG0

Hello @Saurav-Pant yes sure . As you can in footer there is three icon one is GitHub, second twitter and third is for changing background. In mobile all three working fine but in desktop mode changing background icon opens one pop up for selecting mode but it is closing automatically even user did not select any mode

okay got it, Go ahead @kapilG0


Logger doesn't log to STDERR in Rails console if 'active_record/railties' not required

To fix the bug, add this code to Railtie:
    console do
      unless ActiveSupport::Logger.logger_outputs_to?(Rails.logger, STDERR, STDOUT)
        console = ActiveSupport::Logger.new(STDERR)
        Rails.logger.extend ActiveSupport::Logger.broadcast(console)
      end
    end

Which file exactly?

lib/sequel_rails/railtie.rb

Thanks. Shouldn't this be a PR?

@ludalex
I'm lazy.

@JonathanTron is currently on vacations, he'll probably take care about it when he comes back.

(I'm lazy too)


About the purpose of the overlay

Could you please tell me what "chain3xh_overlay.xml" is for?
They seem to have nothing to do with physical interaction, and I still don't understand what they are used for.

I think same too.. all of the same value is at 'chain3xh.xml' !
Did you found the reason ?


Fix config update

Run all initialization after start of manager, so the webhook is up (needed for validation of updated configs).
This way we can also use the caching client everywhere instead of a reader.
Depends on #147 do not merge before that!
Until then, only last commit relevant.
ECOPROJECT-1084

/hold
wait for #147

other PR merged, rebased, ready for review
/hold cancel

/lgtm


Add left panel related components

TODO:

[x] Add all border radius variables in css
[x] AvatarPile
[x] AvatarOutline
[x] AvatarBadgeWrapper
[x] StatusBadge
[x] RoomTile
[x] RoomTileTitle
[x] RoomTileDesc
[x] RoomTilePlaying
[x] CategoryHeader

@netlify /storybook

Looks great! Also I added a basic Storybook implementation at /storybook. It should only get built in development and in deploy previews, not on production.


[BUG] Left/right margin for code blocks

Describe the bug 描述你遇到的错误
A code block displayed on a page without line numbers has no left/right margin. I don't know if this is a feature or a bug. If it's a feature, where can I configure its behaviors?

I want it displayed as in the demo site:

I find that the code lines with margin consist of  and :

But in the problematic case, code lines are  and :

I searched the source code of the Doit theme repo but could not find anything related.
Expected behavior 期待的行为
Code blocks displayed without line numbers should have left/right margins
Screenshots 屏幕截图
No response
Build Environment 构建环境

OS: MacOS
Theme Version: commit 9b6202d74f52199780ac6903ac6695912394230a, branch main
Hugo version: hugo v0.140.1+extended+withdeploy darwin/amd64 BuildDate=2024-12-23T16:26:35Z

Preview Environment 预览环境

OS: MacOS
Browser: Chrome 131.0.6778.70

Additional Information 补充信息
No response

[markup]
  # Syntax Highlighting (https://gohugo.io/content-management/syntax-highlighting)
  [markup.highlight]
    # false is a necessary configuration (https://github.com/dillonzq/LoveIt/issues/158)
    noClasses = false

I find this missed in my configuration.


wdanalysis.h documentation and refactoring

Add Documentation and Refactor Member Functions Of wdanalysis.h
All of the functions within wdanalysis.h lack documentation and could probably be rewritten to use a buffer instead.
Currently, each function uses a mashup of fprintf() and fputc(), sometimes within a loop. A buffer should be added to collate the desired data such that printing takes only one function call.
I'm undecided on how large the buffer should be and whether or not it will be a static array within each functions' body. Perhaps I can add a function or macro that translates to the actual size of the wingdings array(s).
I'm also undecided on whether or not adding a buffer matters at all, as these functions are generally used for (surprise!) analysis and debugging anyway and probably won't have any place in the actual translator, since they're meant to be used once to spit out details and that's it.

The refactoring of the analysis functions was accomplished during work on issue #3. The title and initial description have been changed accordingly.


tachyon master worker threads condition check

tachyon.master.max.worker.threads can not less than tachyon.master.worker.threads

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2814/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2815/
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/2816/
Test PASSed.


Avoid calling Math.pow if possible.

Related to #9581.
When loading a high-resolution image, I have found that the most time-consuming part is converting sRGB to RGB. Especially calling Math.pow is expensive. The following profile of Chrome is the one when loading the PDF file of #9581.

In this PR, we avoid calling Math.pow if the value of color is close enough to 1. The threshold is determined to assure that the result is larger than 254.5.
> ((1 + 0.055) * 0.99554525**(1/2.4) - 0.055)*255
<PHONE_NUMBER>3134699

The benchmark result for the PDF file of #9581 is significant.
$  node stats/statcmp.js stats/results/baseline.json stats/results/current.json
-- Grouped By browser, stat --
browser | stat         | Count | Baseline(ms) | Current(ms) |    +/- |     %  | Result(P<.05)
------- | ------------ | ----- | ------------ | ----------- | ------ | ------ | -------------
chrome  | Overall      |     3 |        26821 |       11648 | -15172 | -56.57 |        faster
chrome  | Page Request |     3 |            7 |           7 |      0 |   0.00 |
chrome  | Rendering    |     3 |        26814 |       11641 | -15173 | -56.59 |        faster
firefox | Overall      |     3 |        14270 |        9383 |  -4887 | -34.25 |        faster
firefox | Page Request |     3 |            6 |           5 |     -1 | -16.67 |
firefox | Rendering    |     3 |        14264 |        9378 |  -4886 | -34.26 |        faster

One of other possible solutions is to avoid converting sRGB to RGB if not necessary. I think it is not necessary when we just display images without transparency operations.

Thank you for your responses. I have edited as requested.

/botio test

/botio-linux preview

Nice work; thank you for figuring this out!


Extended HTTP metadata

Had a question about how to add tags to Wolverine endpoints. That one's easy, use [Tags("foo", "bar"), but what about others?
Add helpers directly to HttpChain so you can do this w/ IHttpPolicy. Maybe support EndpointBuilder so all the extension methods work?
Ideas

public static void Configure(HttpChain chain)
Add to Wolverine**** attributes? Tags, GroupName (EndpointGroupNameAttribute), DisplayName, RequireHost, Name

Exclude from OpenAPI description

ExcludeFromDescriptionAttribute in metadata

Produces

Add ProducesResponseTypeMetadata to metadata
Also contains the status code! Need for 201 et al?
Can be combinatorial. So add the "application/problem+json" as an example

This as well:
            if (string.IsNullOrEmpty(contentType))
            {
                contentType = "application/problem+json";
            }

            return Produces<HttpValidationProblemDetails>(builder, statusCode, contentType);

Accepts

Add (new AcceptsMetadata(requestType, isOptional, GetAllContentTypes(contentType, additionalContentTypes))

Host

new HostAttribute(hosts)

DisplayName

Directly set the display name on the chain. Think it's covered by WolverineVerb attributes

Name

Add metadata new EndpointNameMetadata(endpointName), new RouteNameMetadata(endpointName)

Tags

Use [TagsAttribute]

Group Name

new EndpointGroupNameAttribute(endpointGroupName), so attribute too

See this thing, tons of extension methods hang off of it:
    public interface IEndpointConventionBuilder
    {
        /// <summary>
        /// Adds the specified convention to the builder. Conventions are used to customize <see cref="EndpointBuilder"/> instances.
        /// </summary>
        /// <param name="convention">The convention to add to the builder.</param>
        void Add(Action<EndpointBuilder> convention);
    }

Needs to be doc'd, but everything is possible now


question

hello, does the plugin support version 1.20.6 and 1.21?

Hello,
AkariLevel does not currently support versions 1.20.6 and 1.20.1.
This is because the compatibility of these versions depends on the Taboolib framework.
We need to wait for its version update to ensure compatibility with 1.20.6 and 1.20.1.
Here is a conversation from the Taboolib community discussing this issue:

"皓: Does Taboolib support 1.21 now?"
"枫溪: Still waiting."
"枫溪: The underlying work has been completed."

Additionally, you might want to check out the AkariLevel 4.0.4 update.
This version introduces a new team experience sharing feature.
We are currently using the DungeonPlus plugin for team integration.
If you are using a different team plugin, please let me know, and I can look into adding support for it.
Thank you for your understanding and support.

ok, thank you very much, it's time to wait, and at the moment I'm not using any, because I'm currently configuring the skyblock of my server, when I get with the survival, if I use any I'll tell you.

@Zarkness
After testing, it seems that the 4.0.6 version already supports 1.20.6 and 1.21.


Add Factor mode to CodeMirror

Second try

Thank you! Merged as 63155b7.
(You can, by the way, simply push to the branch used for a pull request to change the patches in the request. That way, you don't have to create a new PR, but simply ask for another review on the old one.)

Thanks!! I was trying to do that, but didn't find how. ;]


Numerate outputs if there are more than 1

Numerate outputs if there are more than 1

Did this one @dgruano


fix release YML to use fuseki build image version 4.7.0

Description
fix release YML to use fuseki build image version 4.7.0

LGTM


download never completes?

i think the download function is waiting for a command

flutter: [2024-05-17 17:03:35.753097] <IP_ADDRESS>:60859> SIZE /Anya/Desktop/Screenshot 2022-05-21 190952.png
flutter: [2024-05-17 17:03:36.057139] <IP_ADDRESS>:60859< 550 No directory traversal allowed in SIZE param
flutter: [2024-05-17 17:03:36.058355] <IP_ADDRESS>:60859> PASV
flutter: [2024-05-17 17:03:36.361158] <IP_ADDRESS>:60859< 227 Entering Passive Mode (192,168,1,5,181,75).
flutter: [2024-05-17 17:03:36.397860] <IP_ADDRESS>:60859> RETR /Anya/Desktop/Screenshot 2022-05-21 190952.png
flutter: [2024-05-17 17:03:36.700702] <IP_ADDRESS>:60859< 150 Sending file
226 Transmission finished
flutter: downloaded: 215402 of 215402, 100.0% calculated 100.0%, speed: Infinity MB/second
flutter: Downloaded 215402 of 215402 bytes
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: FtpException: Timeout reached for Receiving response!
#0      FtpSocket.read.<anonymous closure> (package:pure_ftp/src/ftp/ftp_socket.dart:163:7)
#1      _RootZone.run (dart:async/zone.dart:1655:54)
#2      Future.timeout.<anonymous closure> (dart:async/future_impl.dart:943:34)
#3      Timer._createTimer.<anonymous closure> (dart:async-patch/timer_patch.dart:18:15)
#4      _Timer._runTimers (dart:isolate-patch/timer_impl.dart:398:19)
#5      _Timer._handleMessage (dart:isolate-patch/timer_impl.dart:429:5)
#6      _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:184:12)

but if that command is 226 Transmission finished it got sent, but looks like it got sent before the file could finish copying
anyway to know what is the problem?

tested using another ftp server, that the message gets sent after finishing the copying

it works correctly there.

oh, i'm found problem

i will try fix it in few moments

@abdelaziz-mahdy  try this https://github.com/crifurch/pure_ftp/tree/fix/dowload_wait_response_error, i tested on large file? i think it is not necessary to wait some response after download

now it should works as expected

the flush is needed, i will test again, since the files gets downloaded but corrupted

pls try test in now, remove pubspec.lock before pub get

btw flutter upgrade should get the updated without the need to delete the lock, but will try it

and don't forget change to binary transfer if you are download some not text filles

changing to binary fixed the corrupted problem, should that be the default? most of the ftp downloads are not text files if i am correct
client.socket.setTransferType(FtpTransferType.binary);

changing to binary fixed the corrupted problem, should that be the default? most of the ftp downloads are not text files if i am correct client.socket.setTransferType(FtpTransferType.binary);

i think yes but in many cases ftp clients use auto mode as default

you can use
FtpClient(
    socketInitOptions: FtpSocketInitOptions(
      host: config['host'],
      port: config['port'],
      transferType: FtpTransferType.binary,
    ),
    authOptions: FtpAuthOptions(
      username: config['username'],
      password: config['password'],
      account: config['account'],
    ),
    logCallback: print,
  );

okay thank you very much, the download completes using the mentioned branch

ok, i will update package now


make multimap Whitelist work

If it says "reject", the multimap whitelist is ignored and whitelisted domains are still getting blocked.
If you give it a Score of 1000 blocking still works.
Whitelisted senders are getting -1000 so that they can pass,

@mimugmail cross-post question: what about rsamd 3.x? ;)

Indeed, doesn't makes sense. Do you want to bump the version in Makefile and add a changelog to pkg-desc?

You mean like the last 2 Pull requests?
Sorry if i did it wrong. Never done that before

Created a new one because i destroyed this. Idk how that happend....
Sorry
New one
https://github.com/opnsense/plugins/pull/2822

@mimugmail cross-post question: what about rsamd 3.x? ;)

PR highjacking!!!11elf
Yes, lets update, maybe give me a hint when ports are synced so I can build a pkg for final testing

Ok, I’ll grab it during tomorrow’s sync.

@fichtner rspamd 3.1 works fine here :)

@mimugmail perfect, thanks for testing ❤


Coin Control Tree Mode

Tree mode shows the interest figure as a header rather than the total amount.

on it...


HTTP Error 404 - After a successful "New Azure OpenAI" deployment to Azure using "Deploy to Azure" button

I ran the "Deploying with a new Azure OpenAI instance"  Azure Portal command from here:
https://github.com/microsoft/semantic-kernel/blob/main/samples/apps/copilot-chat-app/webapi/DeploymentTemplates/README.md
The deployment was successful in the Azure Resource Group deployment templates.
When I test the web app with  the   https://app-skweb.azurewebsites.net/probe
I receive "Semantic Kernel Service up and running"
But when I try to access the chat web app for the first time I receive an HTTP Error 404 - page can’t be found.
Accessing this link gets 404 - https://app-skweb.azurewebsites.net
I'm running this directly from the main github repo so everything is recent.
Any Idea what I'm missing?
Thanks in advance!

The templates only deploy backend web api (i.e., CopilotChatWebApi). For the end-to-end experience you'll need to run the frontend web app (i.e., samples\apps\copilot-chat-app\webapp) and configure it to use your instance of the backend webapi. I think the issue here is that we're missing a section in the READMEs on connecting these two.

Get your API key from your deployed instance (https://github.com/microsoft/semantic-kernel/blob/main/samples/apps/copilot-chat-app/webapi/DeploymentTemplates/README.md#authorization)
Follow the instructions to run the frontend web app locally and configure it to use your deployed backend: https://github.com/microsoft/semantic-kernel/blob/main/samples/apps/copilot-chat-app/README.md#start-the-webapp-frontend-application.

We'll take a look at making this process more obvious this week.

Great! Thank you for the quick response!

One last question - for modifying the .env file with the backend URI what should it be? The Azure WebAPI server doesn't appear to allow traffic on port 40443 - I've opened ports on the NSG and still cannot get a response.
I've tried different variations of the URI but can't get my webapp to open up. It just sits and spins..
URIs I've tried:
http://app-semkernel-skweb.azurewebsites.net:40443/
https://app-semkernel-skweb.azurewebsites.net:40443/
https://app-semkernel-skweb.azurewebsites.net

I figured out what URI to use since port 40443 wasn't used:
REACT_APP_BACKEND_URI=https://app-semkernel-skweb.azurewebsites.net/
Everything is now working


add synthstrip

https://surfer.nmr.mgh.harvard.edu/docs/synthstrip/
but it's also included in the new fressurfer release

Already included in NeuroDesk with freesurfer/7.4.1 .
(base) jovyan@neurodesktop-2024-05-25:~$ ml load freesurfer/7.4.1
(base) jovyan@neurodesktop-2024-05-25:~$ mri_synthstrip
Robust, universal skull-stripping for brain images of any type.
usage: mri_synthstrip [-h] -i file [-o file] [-m file] [-g] [-b BORDER] [--no-csf]
                      [--model file]

optional arguments:
  -h, --help            show this help message and exit
  -i file, --image file
                        Input image to skullstrip.
  -o file, --out file   Save stripped image to path.
  -m file, --mask file  Save binary brain mask to path.
  -g, --gpu             Use the GPU.
  -b BORDER, --border BORDER
                        Mask border threshold in mm. Default is 1.
  --no-csf              Exclude CSF from brain border.
  --model file          Alternative model weights.

If you use SynthStrip in your analysis, please cite:
----------------------------------------------------
SynthStrip: Skull-Stripping for Any Brain Image
A Hoopes, JS Mora, AV Dalca, B Fischl, M Hoffmann
NeuroImage 206 (2022), 119474
https://doi.org/10.1016/j.neuroimage.2022.119474

Website: https://w3id.org/synthstrip

yes, but we need it as a separate container to include as baseline for other containers.


Formatted the Repository

I've used gofmt to format the code in the repository. The benefits of using it are documented in this post on the Go blog and the tool is widely used within the community to help maintain a standard style.

Thanks for the fix too. This looks apparently better.


createStorageSignal is null even with a initial value

// The type of value1 is boolean | null but it should be boolean
const [value1] = createStorageSignal("foo_bar", false);
// The type of value2 is correct, here a null is possible
const [value2] = createStorageSignal("foo_bar);

If you agree that the current behavior is wrong I can make a PR.

Unfortunately, you never know if an external process overwrites the value in the storage; if that happens and the events are active, you will still receive a null value, so as a precaution, we left the types this way. If you're 100% sure, you can ! the storage item you read instead of securing yourself against null.

How about taking a factory function that returns a fallback value for cases when storage returns null?


fix(lsp): onAttach not doing anything

lsp.onAttach was not being used, only
lsp.servers.${server}.onAttachExtra was being applied, this commit fixes that.
It also fixes lsp.onAttach documentation to also state that bufnr is provided, to be consistent with onAttachExtra

Thanks, look good to me 👍


Beartype 0.3.1.

This commit bumps both the condaforge-hosted package for beartype to its next stable release and the minimum major version of Python internally required by this package to Python 3.6, reflecting the recent EOL for Python 3.5.

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Eternally maintain flake.nix

flake.nix using fenix and naersk.

Begins with work by @buffet in #124


Run class file command doesn't get executed

First of all, great to see that someone developed Compile and Run extension for Java in Visual Studio Code.
The alt+c shortcut works perfectly and the .class file is also generated. However, the alt+r shortcut generates the command to be executed but somehow, it doesn't run in the console. Note that the bottom bar of VS Code displays the command to be executed.
I have attached the screenshots below that show that the command to run the Java code is not executed.
I hope this issue is solved by you at the earliest. Thanks once again for developing this extension.

Well, thanks for your feedback. I am very busy，recently. So, I will handle this as far as possible. by the   way, I am not good at English either. :smile:

Well, thanks for your feedback.I recently ran into some trouble about work, so, I will handle this as far as possible.
By the way, I am not good at English either. :smile:  Best wishes! @kaivalya97


Fix TaskState after error in Task::start()

Catches errors in constructing Drivers and sete the Task's state and error accordingly.

@mbasmanova
In Task::start() not only the Driver creation can throw, but pretty much any code.
Should we wrap the whole function in the try?


Misc bugfixes and improvements 👷

👋 Greeitings!
I found your library and there was a bit of breakage in it.
Sorry for the fairly hefty PR, but I'm happy to change pretty much whatever. Thanks for creating this library!
Changes:

.idea/ and .pytest_cache/ added to .gitignore - artifacts from PyCharm and pytest

Added a .travis.yml, which should fix #6. Check out https://travis-ci.org/Fitblip/amusement for an example

Removed the following modules, they were all broken and pointing to hosts that don't exist 😢

DisneylandParis
SeaworldOrlando
BuschGardensTampa
SeaworldSanAntonio
SeaworldSanDiego
BuschGardensWilliamsburg

Updated README.md

Fixed an error in the example
Removed references to the removed modules
Added build images from travis and coveralls that should just work once you click some buttons in both

Added timeout=3 to all requests (default is 30s which made writing the tests quite painful :-P)

Re-enabled ssl verification to all requests

Fixed the DisneyPark module (found a valid client_id)

Full python2/3 compatibility

Coverage reports, see an example - https://coveralls.io/github/Fitblip/amusement

Misc formatting changes

This looks absolutely awesome! Thank you so much for doing this. As you can see, this library has been in disrepair for quite some time.

Thanks for the merge @rambleraptor! 👍
Happy to help :)
Also let me know if you'd like any assistance with Travis/Coveralls (if you're interested in setting them up!)


build: upgrade vitest

When upgrading vitest it was observed that there was significant performance degradation. We should explore adding a worskpace configuration.

Closed by https://github.com/FuelLabs/fuels-ts/pull/2247


fix(test): set jacoco excludes for engine-test's unitTest and integrationTest tasks

upgrade to jacoco 0.8.8 (from 0.8.5)
relates to #4985
How to test
Run integrationTest and unitTest
Run jacocoTestReport
check to see if test output contains loads of messages about IllegalClassFormat
It Would Be Nice If
It would be good to have less configuration duplication. The excludes-list is set in several places,
and there are probably places that need it but don't have it yet. (module tests? subsystem tests?)

 jacoco config


add validate refer to wasm spec

add validate code refer to wasm speculation. https://webassembly.github.io/spec/core/appendix/algorithm.html

Codecov Report

Merging #158 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #158   +/-   ##
=======================================
  Coverage   69.54%   69.54%
=======================================
  Files          43       43
  Lines        5007     5007
=======================================
  Hits         3482     3482
  Misses       1231     1231
  Partials      294      294

Impacted Files
Coverage Δ

wasm/types.go
45.83% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

why this commit not merge??

@carltraveler apologies for the belated answer.
also: deepest apologies if merging the PR #160 before consulting here generated some amount of frustration.
in hindsight, I should have brought you into the review process of that #160 PR...
I meant no disrespect.
I felt the other PR was clearer to me (but that's probably I am more used to reviewing Twitch's PRs by now.)
again: apologies for this mishap, it does not reflect how I think of your valuable contributions to wagon.
(let me close this PR as I believe PR #160 addressed it. but feel free to reopen and/or rebase if you think some aspects are better handled here.)


TODOs

findDevice
device namespace
benchmark method
research profiling
read up on optimisation.

gist -> https://gist.github.com/william-taylor/daae07780bdb00c1264a8e95aa23cdec

cuda-utilities -> new filename

https://devblogs.nvidia.com/parallelforall/even-easier-introduction-cuda/


Get Alerts/findings fixes; more tests

Signed-off-by: Petar Dzepina<EMAIL_ADDRESS>Description
Fixed bug when searching alerts and findings by detectorType
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #74 (b55e508) into main (8d82248) will decrease coverage by 0.09%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##               main      #74      +/-   ##
============================================
- Coverage     43.17%   43.08%   -0.10%
  Complexity      838      838
============================================
  Files           162      162
  Lines          5577     5589      +12
  Branches        708      710       +2
============================================
  Hits           2408     2408
- Misses         2942     2954      +12
  Partials        227      227

Impacted Files
Coverage Δ

...h/securityanalytics/action/GetFindingsRequest.java
0.00% <0.00%> (ø)

...rityanalytics/resthandler/RestGetAlertsAction.java
0.00% <0.00%> (ø)

...tyanalytics/resthandler/RestGetFindingsAction.java
0.00% <0.00%> (ø)

...yanalytics/transport/TransportGetAlertsAction.java
0.00% <0.00%> (ø)

...nalytics/transport/TransportGetFindingsAction.java
0.00% <0.00%> (ø)

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


[Feature Request] Run tests and display results in outline view

It would be nice to have a shortcut to run all tests and see results in the outline view, something similar to flutter's vscode test tree.

@baumths this functionality is achievable using:

https://github.com/nvim-neotest/neotest
https://github.com/sidlatau/neotest-dart
Rather than re-inventing the wheel/taking on the huge task of building a testing plugin these already do the job quite well an I recommend them

Thank you for introducing me to neotest, it is really a great solution.
And also, thank you for this amazing plugin, it makes life so much easier to develop dart/flutter apps, it's truly incredible. Great job!


Use SSL for connecting to RDS

Description
After upgrading to v15, Postgres require us to log in using SSL.
Type of change

Configuration update

Testing steps
CI.
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report
Patch and project coverage have no change.

Comparison is base (0370b83) 59.75% compared to head (02cf2b0) 59.75%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #224   +/-   ##
=======================================
  Coverage   59.75%   59.75%
=======================================
  Files           7        7
  Lines         907      907
=======================================
  Hits          542      542
  Misses        326      326
  Partials       39       39

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


Correct es link in dev tools

I think these links should've changed when we upgraded to ES 7
https://www.elastic.co/guide/en/elasticsearch/reference/current/removal-of-types.html

Elasticsearch 7.x
Specifying types in requests is deprecated. For instance, indexing a document no longer requires a document type.
The new index APIs are PUT {index}/_doc/{id} in case of explicit ids and POST {index}/_doc for auto-generated ids. : > Note that in 7.0, _doc is a permanent part of the path, and represents the endpoint name rather than the document type.*

here's where they are, for reference...


[Run] Packaged apps only available via localized name

Environment
Windows build number: 10.0.19041.264
PowerToys version: 0.18.2
PowerToy module for which you are reporting the bug (if applicable): PowerToys Run

Problem
For packaged apps we not indexing the englisch name.
Details / Example
In start I can search for Rechner or calculator to open the calculator app. The app is shown as "Rechner (= German)". In PT Run the app can only be found by its localized name Rechner.
Screenshots

Related to https://github.com/microsoft/PowerToys/issues/3773

@htcfreek does this still happen in 0.19?

Yes.

now for the ultimate question, how do we get that data :)

now for the ultimate question, how do we get that data :)

Yes that's the BIG question.

removing this as this is unknown how to get this data

Should we add the "help-wanted" label?

@htcfreek are you changing your language while PT Run is up and running?  #4147 shows it with it grabbing it.
#3773 could be related here as well.

@htcfreek are you changing your language while PT Run is up and running?  #4147 shows it with it grabbing it.
#3773 could be related here as well.

No. I didn't change my lang. I only search for English and for German name.

@htcfreek are you changing your language while PT Run is up and running?  #4147 shows it with it grabbing it.
#3773 could be related here as well.

The screenshot in #4147 that shows it grabbing "Câmera" for the "camera" query is a mockup, used only to illustrate the expected behavior.

@crutkas
I think we can't fix this. The native api method we have to use for getting the name has no parameter to influence the locale used. Instead it uses the windows display language and we can't manipulate this language property at runtime. (It's only changeable by the shell itself.)
Should we close this issue?

I think that we can parse the resource (.pri) file to read the display name string in a specific language from it.
We can pass @{Microsoft.WindowsTerminal_1.15.3466.0_x64__8wekyb3d8bbwe?ms-resource://Microsoft.WindowsTerminal/Resources/AppStoreName} to SHLoadIndirectString() to get Windows Terminal's display name in current display language. Since SHLoadIndirectString() loads the string from the app package's .pri file, we can also pass @{C:\Program Files\WindowsApps\Microsoft.WindowsTerminal_1.15.3466.0_x64__8wekyb3d8bbwe\resources.pri?ms-resource://Microsoft.WindowsTerminal/Resources/AppStoreName} to it, replacing the package name with the path to the .pri file, which is usually the package install path + "\resources.pri".
Unfortunately, changing the preferred UI language of the current thread/process doesn't affect its behavior, and there's no documentation on how SHLoadIndirectString() loads the string from .pri file. But fortunately, there's a library in Windows App SDK called MRTCore that provides a way to read any given .pri file.
After building the DLL, I used the following C++ code to test it.
MrmManagerHandle hResMgr;
// load the .pri file given its path
hr = MrmCreateResourceManager(LR"(C:\Program Files\WindowsApps\Microsoft.WindowsTerminal_1.15.3466.0_x64__8wekyb3d8bbwe\resources.pri)", &hResMgr);

MrmContextHandle hResCtx;
hr = MrmCreateResourceContext(hResMgr, &hResCtx);
hr = MrmSetQualifier(hResCtx, L"language", L"en");  // the language of the result string

LPWSTR pStr;
// load string given its resource URI
hr = MrmLoadStringResourceFromResourceUri(hResMgr, hResCtx, L"ms-resource://Microsoft.WindowsTerminal/Resources/AppStoreName", &pStr);

// now the string is stored in pStr

// free allocated strings & handles
MrmFreeResource(pStr);
MrmDestroyResourceContext(hResCtx);
MrmDestroyResourceManager(hResMgr);

Then I successfully get the display name of Windows Terminal in different languages by changing the "language" qualifier value to "en" Windows Terminal, "zh-hant" Windows 終端機, "ja" Windows ターミナル, etc. Setting language to an empty string will return the English version (default?), and if the language is not set, it will return the Simplified Chinese version Windows 终端 which is the display language on my system.
So maybe we can use this to replace the call to SHLoadIndirectString().

@gexgd0419
Sounds interesting. Please have in mind that the Program plugin is written in C++. And to be aligned to Win32 programs we need both localized and English name. The goal would be to show in with localized path and name, but can search for English and localized name.
If you want to create a PR this would be awesome. If you have some questions about the plugin code feel free to ask me/us.

Here's a C# version. It requires Microsoft.WindowsAppSDK.
var resMgr = new Microsoft.Windows.ApplicationModel.Resources.ResourceManager(@"C:\Program Files\WindowsApps\Microsoft.WindowsTerminal_1.18.10301.0_x64__8wekyb3d8bbwe\resources.pri");
var ctx = resMgr.CreateResourceContext();
ctx.QualifierValues["Language"] = "en"; // Specify a language. Skip this line to use the user's display language
var candidate = resMgr.MainResourceMap.TryGetValue("Resources/AppStoreName", ctx);
if (candidate != null) // if exists
    Console.WriteLine(candidate.ValueAsString);

@gexgd0419
I think these are two different things:

Finding appx packages with the inglish name too. (Not only with their localized one.)
Switching the keyboard layout used for PT Run.

We should split them into different issues. Otherwise tracking work will be complicated.

I agree that it would be better if PT Run can just get the keyboard layout right in the first place.
However, #3773 suggests that it work both ways:

English typed in Russian layout
Russian typed in English layout

Product Launcher should support keyboard layout mapping for all system keyboard layouts, for example:
US Keyboard ⟶ Russian Keyboard
Russian Keyboard ⟶ US Keyboard
Query "ьшскщыщае еуфьы" should display the results of "ьшскщыщае еуфьы" and "microsoft teams" queries.
Query "rfkmrekznjh" should display the results of "калькулятор" and "rfkmrekznjh" queries.

The author of #3773 said:

Important note: My locale is "English US" and the region is the United States. Preferred languages:

English US
Russian

So English is the display language, and Russian is in the preferred language list.
Should we support searching in preferred languages other than the display language?

Should we support searching in preferred languages other than the display language?

Currently PT Run supports only the display language and English for Win32 programs. For packaged apps should do the same. (Everything else would be a rare case.)


Fix warnings raised by tests

Pull Request doc
Purpose
Fix warnings. There are three kinds of warnings

one raised by pandas regarding the type of array used for indexing a data frame
one raised by bokeh because we attempt to edit an empty legend
one raised by matplotlib because too many figures have been opened
*Closes #374

What the code is doing

Use a list in place of a set to  index data frame in plot_scatter_capacity_vs_curtailment
Use only coal generator to ensure glyphs are rendered on the map and legend is created in test_plot_capacity_map
Close all plots in each test module where plots are generated

Testing
Warnings are not raised
Where to look
Changes are mostly done in the test modules
Usage Example/Visuals
N/A
Time estimate
5min

We can also use a fixture to close the plot automatically after each test (I wasn't sure so tried it out in this branch)

We can also use a fixture to close the plot automatically after each test (I wasn't sure so tried it out in this branch)

That's smart.

We can also use a fixture to close the plot automatically after each test (I wasn't sure so tried it out in this branch)

That's smart.

Much cleaner. I will cherry pick your commit and update the PR

We can also use a fixture to close the plot automatically after each test (I wasn't sure so tried it out in this branch)

That's smart.

Much cleaner. I will cherry pick your commit and update the PR

Done


Crash destroying ServerContext

The details are provided on https://groups.google.com/forum/#!topic/grpc-io/brTaLVvZZ98
I have only been able to reproduce issue once. It is also possible that the shutdown sequence was a red herring. The callstack is
ntdll.dll!00007ffe6f7ebbdf()	Unknown
 	ntdll.dll!00007ffe6f7c4571()	Unknown
 	ntdll.dll!00007ffe6f7c4490()	Unknown
        gpr_mu_lock(gpr_mu * mu) Line 53	C
 	interned_slice_destroy(interned_slice_refcount * s) Line 94	C
 	interned_slice_unref(grpc_exec_ctx * exec_ctx, void * p) Line 112	C
 	grpc_slice_unref_internal(grpc_exec_ctx * exec_ctx, grpc_slice slice) Line 76	C
 	destroy_channel_elem(grpc_exec_ctx * exec_ctx, grpc_channel_element * elem) Line 926	C
 	grpc_channel_stack_destroy(grpc_exec_ctx * exec_ctx, grpc_channel_stack * stack) Line 166	C
 	destroy_channel(grpc_exec_ctx * exec_ctx, void * arg, grpc_error * error) Line 383	C
 	grpc_exec_ctx_flush(grpc_exec_ctx * exec_ctx) Line 85	C
 	grpc_exec_ctx_finish(grpc_exec_ctx * exec_ctx) Line 100	C
 	grpc_call_unref(grpc_call * c) Line 579	C
 	grpc::ServerContext::~ServerContext() Line 157	C++

As an added detail, I believe the locked count of gpr_mu was -6.

Based on the discussion in the google groups thread, I believe that this is a WAI; killing the server and the server CQ to which an RPC is bound is not sane for the RPC or its context.

What is WAI?
If it is unsafe, we can do the discussion here or over the email I had sent you guys last week. Whatever is preferable.
Thanks.

Sorry, WAI == "Works as intended". We really need the context destroyed before the server.


branch

branch
Originally posted by @Justin09111974 in https://github.com/github/docs/pull/29000#discussion_r1351287393

Closed


Cannot look at types / methods in Observatory

I used to look sometimes at methods code in Observatory - both Dart and generated native.
But now the corresponding Web pages are always blank.
For example:

It should have been fixed by https://github.com/dart-lang/sdk/commit/416ffffab5caf6ca985578e9b0053d6c7a3de655

ACK, it works for me now.


Question and bugs on new Wizard

Hello,
I have a few questions about the new Wizard UI on 6.2.4-4. Thank you for developing it.
I do not get how to do the input query based on field values. Let's say I want to throw an alarm when there are 20 logs with field value  "field": "error" in 15minutes. How would you do that?
What is the attribute in "grouped over" / "over" -> "top" for? Setting it does not seem to change the Watcher Query.
Bugs:
-When a wizard field is opened and the user clicks somewhere else or clicks on the next wizard field the opened field should be closed in my opinion
-There are also document fields listed that aren't fields but higher JSON levels. E.g. JSON:
{ "field": {
"subfield": "value } }
Then also "field" is shown as a field
-You could make the wizard fields longer. As of right now some fields for example say: "FOR LAST 3 yea" instead of "FOR LAST 3 years"
-The chart does not always reload and when it does it lags and the page does not react until it has finished loading. Also when I select "months" in "FOR LAST" I always get no data.
You probably know about these bugs but in case you don't this is just a quick notice.

Also the Action section is just appearing when data could be returned in the chart. I want to set up alarms also on data that is not there yet or does not fulfill the given timeframe. That makes the wizard unfortunately pretty unusable for me

You can access actions if there is no data in the chart.
Save the wizard watcher with a big timeframe, when there is data in the
chart. Then click the wizard watcher in the watchers list. Decrease the
timeframe to a level you need. Actions panel should be still there.
On Thu, 23 Aug 2018, 08:52 bienensenf<EMAIL_ADDRESS>wrote:

Also the Action section is just appearing when data could be returned in
the chart. I want to set up alarms also on data that is not there yet or
does not fulfill the given timeframe. That makes the wizard unfortunately
pretty unusable for me
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sirensolutions/sentinl/issues/529#issuecomment-415311080,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFI9sQ0PSoEevfiEzdm4Z1-o8FnOTshrks5uTlE4gaJpZM4WHie_
.

Here is a wizard builder doc
https://docs.google.com/document/d/1Vbi2IBy5TNmCBXpNqmp23Z99AaDZwIero9XTWOvz4kc/edit?usp=drivesdk
It is still under development.
On Thu, 23 Aug 2018, 09:18 bienensenf<EMAIL_ADDRESS>wrote:

Thanks for answering. Yes that's right but I guess Action page should be
shown always. Could you also tell me how it is possible to filter on field
values in the wizard as said in the original post?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sirensolutions/sentinl/issues/529#issuecomment-415316826,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFI9sUbv3Kb2Lx3sK0qX05CuN1gvnjZKks5uTldQgaJpZM4WHie_
.

Still did not find anything about field values. Did I miss something or is it not in there?

I put some comments in the doc above.
On Thu, 23 Aug 2018, 12:46 bienensenf<EMAIL_ADDRESS>wrote:

Still did not find anything about field values. Did I miss something or is
it not in there?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/sirensolutions/sentinl/issues/529#issuecomment-415372648,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFI9sSUatIeCOeYS3RW6lT7B8mGdUlxJks5uTogNgaJpZM4WHie_
.


RFTools farmer and mysticalgriculture

Hey, I seem to be having an issue on my server. I recently updated the server to 1.12.2 and now mystical crops seem to be dropping 2 seeds nearly every time they are harvested by the rftools farmer. this was not an issue in 1.12.1 and I can guarantee that the extra seed chance is set to 0 in the config file.
Is this in issue with MA ignoring the config or the rftools farmer?

RFTools Farmer? Wrong mod? RFTools does not have a farmer

SHIT sorry! The situation had me a little stressed and I guess I mixed up mod names -_-'


Suggestion: Container Tabs

Hello. Would you mind adding these configs to the user.js?
user_pref("privacy.userContext.enabled", true);
user_pref("privacy.userContext.ui.enabled", true)
Thanks.

Would you mind adding these configs to the user.js?

Will ship in next release. Add them to the bottom of your personal user.js for now.

Completed w/ #61


bug

i meet a bug about 'Conv1d' object has no attribute 'input'
can you help me

Can you please provide more detailed information?


feat(performanceTest): Template colasMQ performa

Before submitting a pull request, please read
https://github.com/bancolombia/scaffold-clean-architecture/wiki/Contributing
Description

Category

[x] Feature
[ ] Fix

Checklist

[x ] The pull request is complete according to the guide of contributing
[x ] Automated tests are written
[x ] The documentation is up-to-date
[x ] The pull request has a descriptive title that describes what has changed, and provides enough context for the changelog
[ ] If the pull request has changed structural files, you have implemented an UpgradeAction according to the guide
[ ] If the pull request has a new Gradle Task, you should add Analytics according to the guide

@Kathechicae  thank you for the contribution, can you rename the DT files in english maybe DT_example...?

Addeed RabbitMQ queues template and sample datasets (.csv .txt)
Renamed datasets for Api and MQ Queues


CdmCorpusDefinition | Document not index previsouly. Please when calling fetchObjectAsync specify the 'importsLoadStrategy' to 'load' on the ResolveOptions object.

Hi,
Many Thanks for ( Microsoft CDM Release Version Update: 1.0.12) Update it has resolved most of the issues
I have a small confusion on this issue Can you please help me with this (what this error about ?)
I'm doing this step (abstract to resolved )
  const manifest = await cdmCorpus.fetchObjectAsync<
  CdmManifestDefinition
  >("abstract/digitalTwin/digitalTwin.manifest.cdm.json");
   const resolvedManifest: CdmManifestDefinition = await manifest.createResolvedManifestAsync('resolved', undefined);
   await resolvedManifest.saveAsAsync(resolvedManifest.manifestName + ".manifest.cdm.json", true);

Hello,
Sorry to hear that the new update is causing you trouble.
We've introduced in this release a way for users to choose when the imports of a document are loaded. This option can be specified by passing a resolveOptions to fetchObjectAsync. In the resolveOptions you should be able to set the property importsLoadStrategy which by default is lazyLoad.
What you described looks initially as a bug so I'll do some investigation about it. In the meantime you can set the importsLoadStrategy to load so all the imports are loaded on fetchObjectAsync. After this is fixed you may want to remove this and let the lazy loading do the work for you.
const resOpt = new resolveOptions();
resOpt.importsLoadStrategy = importsLoadStrategy.load;
const manifest = await cdmCorpus.fetchObjectAsync<CdmManifestDefinition>("abstract/digitalTwin/digitalTwin.manifest.cdm.json", null, resOpt);

Hi,
i have add importsLoadStrategy But still error is same ( but seems functionality is working as expected creating ResolvedManifest files )
const resOpt = new resolveOptions();
resOpt.importsLoadStrategy = importsLoadStrategy.load;

[event log kuberbites type_view_accseibility_type_anouncment_type_touch_explortation_type_text_selection_changed
Type_viewed_focus_it_cleared

Hi,
This error was being mistakenly logged and doesn't affect the functionality of the SDK.
It has been already fixed and should be available in our next release. You won't need to specify the imports strategy property anymore.
Thanks for reporting it.

Closing this issue since the new release with the fix is coming in 5 days. Please reactivate if further discussion is needed.