Introduce "delimiter" parameter for seeds? Allow non-csv file types

Describe the feature
Use alternative delimiters for seeds besides "," (comma)
Describe alternatives you've considered
Currently, the only option is to load a non-csv seed into database as a temp table and then reference that as a "source".
Additional context
No, not database specific.
Who will this benefit?
Anyone who uses seeds.

@sgoley When you say "non-csv file types," are you talking only about tabular data with non-comma delimiters (e.g. TSVs)? Or are you also thinking about other formats for structured data (JSON, XML, ...)?
If it's just a question of delimiters, could you provide a bit more background on the difficulty you're encountering? Is there a blocker to pre-processing your files, so as to switch from tab/semicolon/other delimiter to commas?
We've got another issue already open for newline-delimited JSON support: https://github.com/dbt-labs/dbt/issues/2365

Yes, in my case I am specifically talking about non-comma delimiters like TSVs, semi-colon delimited (common in some european countries given the numerical comma standard), or "|" ( 'VERTICAL LINE' U+007C ) delimited files (infrequent but used within US financial systems).
I do completely understand that pre-processing is the only current workable solution, just opening the issue here since I searched and no truly similar issue has been raised or closed.
After reading more into the agate csv reader function, it looks like that particular api does not support a "sep" / "delimiter" parameter which is why I assume this was not supported natively?

@sgoley You're right, we just call agate's from_csv method for this:
https://github.com/dbt-labs/dbt/blob/f7680379fca80f653e8e7e2d45d7165a0fd864da/core/dbt/clients/agate_helper.py#L146
The good news: in that method, agate itself is happy to pass through any/all keyword arguments supported by python's built-in reader. It accepts **kwargs and passes them into csv.reader().
So I think this could be as simple as:

adding a delimiter arg to agate_helper.from_csv
adding a delimiter config to seeds (right around here), with default value either , or None (i.e. comma)
adjusting the load_agate_table context method

        column_types = self.model.config.column_types
        delimiter = self.model.config.delimiter
        try:
            table = agate_helper.from_csv(path, text_columns=column_types, delimiter=delimiter)

So, I'm pretty close to tagging this as a good first issue. I have just a few more questions:

What if you need custom quoting, escape characters, etc? Should we seek to add generalized support for all csv.reader() kwargs as configs? Or do we think a configurable delimiter covers 90% of the bases?
Would you expect to be able to define these seed files with other file extensions (e.g. *.tsv)? That change would need to happen in a different part of the codebase.
Would we be better off saying "no" to all of the above, keep seed support very simple (dbt isn't a data loader!), even at the cost of pre-processing?

Hello @jtcohen6
Following this issue along with  JSON related (following #2365 )
As a  dbt community member, If I may answer your questions to @sgoley
I understand the poor use cases outlined for dbt here https://docs.getdbt.com/docs/building-a-dbt-project/seeds - for valid/god use cases seeding small amounts of data as part of a build process (sampling ) - writing tests, documentation etc
What if you need custom quoting, escape characters, etc? Should we seek to add generalized support for all csv.reader() kwargs as configs? Or do we think a configurable delimiter covers 90% of the bases?
-- I believe the community would benefit from adding generalized support for csv - COMMA is a very common character and shows up everywhere. If the data is escaped or quoted, the the data may shift even for simple files.

support for different delimiters, text enclosure and escape character at a minimum

Would you expect to be able to define these seed files with other file extensions (e.g. *.tsv)? That change would need to happen in a different part of the codebase.
-- Another community benefit, sometimes we need to work with small datasets from different formats. I believe support for CSV (if it has the configurable delimiter with text enclosure and escape character; this covers any character delimited file .txt,.tsv, pipe delimited etc - csv.reader() kwargs should handle this beautifully), json , xml, parquet and ORC: these are becoming common.
Would we be better off saying "no" to all of the above, keep seed support very simple (dbt isn't a data loader!), even at the cost of pre-processing? keeping it simple is great but this limits the entry point to dbt as only comma separated files. Going through
---- As the community grows this is going to be a growing need. Now, dbt seed is still not ideal for loading data into a warehouse, as part of the build process for a data project, sampling (maybe find size limitations to avoid functionality abuse) needs to be recommended as part of best practice to keep the functionality light as is. Yes (dbt is not a a data loader), but can it be used to build an ELT pipeline based on small samples of different file formats? Mocking the (EL) process for common data formats, build pipeline.  Do not say "no"
Thank you for considering this. Keep up the great work.

I opened a pull request that adds this feature into dbt-core: https://github.com/dbt-labs/dbt-core/pull/7186

Hello team, when this issue will be fixed? we have both csv and tsv files in the seeds, seems like tsv is not working in seeds - I'm in version dbt-databricks 1.8.0, Thanks

Hello team, when this issue will be fixed? we have both csv and tsv files in the seeds, seems like tsv is not working in seeds - I'm in version dbt-databricks 1.8.0, Thanks

So for my understanding, the issue is that tab separator still does not work as delimiter after my delimiter support contribution last year? Could you provide me with some more context, e.g. the configuration that you currently use? I assume you should configure with a \t as delimiter. Most likely if \t does not work, this has to do with the underlaying library (agate) that is being used and is not supporting this.
See call with delimiter being passed as argument:
https://github.com/dbt-labs/dbt-core/blob/bdf28d7eff13edcbe0d41746e50ffa8edcfa3cf0/core/dbt/context/providers.py#L1055
In that case I think it might be a bit harder to fix, e.g. this needs to be implemented in agate, or a very big change to handling seeds is required.
But let's first see if we can have a reproducable example, to confirm that this is really the case.


[feature request] point to previous item when clicking left arrow key

if you are inside c:/a/b/c,
then you click left arrow key, it can jump to parent folder and point to parent foler (> c:/a/b/c)

It actually already does that?
if you press left arrow it goes to the parent folder?


set container height..?

i want to set container height to canvas height in modal?
i have custom modal. when i create cropper in modal my view like this in added image

i want to set contianer height to canvas height. Becouse my modal height getting long when i load big image in modal ..
and my code is here
    $image.cropper({
        aspectRatio: myAspectRatio
        viewMode: 2,
        autoCropArea: 0.5,
        built: function () {
            croppable = true;
        }
    });

I am sorry for my english;

Just set the follow styles:
img {
  max-width: 100%; /* This rule is very important, please do not ignore this! */
}

It doesn't work in my code

Were you able to solve this issue?
It is very annoying that the height of the container is always equal to the original height of the image even when the image is downsized by setting a max-width on the container.
Setting max-width 100% on the image does not solve this problem, actually it creates it.

Steps to use Cropper:

Check if the original image is displayed correctly in its place?
2.1 Yes!  Regulate the image to fit your expectation. Go to 3.
Apply copper on the image.

This still an issue and very annoying I don't understand how this issue was closed...

The only way that you can solve it adding a parent div with a fixed height.

original image size not working

on class cropper-container cropper-bg  height like on image source

Issues not closed

The only way that you can solve it is adding a parent div with a fixed height or a max height.

i sovle  this problem by your method. thanks

If you are using cropper in a modal after uploading it using form and having this problem, here is the solution:
give image max-height of some fixed pixel like 300px.

Had a similar issue, fixed by adding position: relative to the container:


Fix #14281

@PatrickRyanMS @PirateJC : I would qualify this as low level confidence. It is a bit of a hack. I think it is acceptable if it does not grow more hacky. So let's make sure to test it widely when merged

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14288/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14288/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14288/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14288/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14288/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14288/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14288/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14288/merge#BCU1XR#0


Add feature: Print breakpoint table

Take example5.evcd, breakpoints=[9,11,17] as example, the table is:
time    9       11      17
scope   testbench
scope   pe8
req_I   128     16      4
req_O   zzzzzzzz        zzzzzzzz        zzzzzzzz
en_I    1       1       1
en_O    z       z       z
scope   pe4
req_I   8       4       2
req_O   zzzz    zzzz    zzzz
en_I    1       1       1
en_O    z       z       z

The column width of tables can be fine-adjusted, but to achieve it, sootty needs to recursively see all the wires twice, so performance is lost.

Have you tested if this works when the waveform is being displayed? The svg is written to stdout and piped to the display program, so if the breakpoints are also printed to stdout that could possibly break the image.

Have you tested if this works when the waveform is being displayed? The svg is written to stdout and piped to the display program, so if the breakpoints are also printed to stdout that could possibly break the image.

Yes, I've tested that this works with or without -o, but I'm not quite familiar with subprocess. What if subprocess.run is used instead of subprocess.Popen()?


Login problems

Hi,
I am getting "401Unauthorized" response on login though prioviding proper credentials.
I am testing against latest RocketChat 3.3 and I think its due to a missing Content-Type header on login… (When comparing with manual calls with PostMan)

login is sending the credentials as FORM data, so I observe the same issue as #47


[8.x] Handle X-Forwarded-Prefix HTTP header

Symfony HttpFoundation supports X-Forwarded-Prefix HTTP header since version 5.3.
This header is used by Traefik reverse proxy for example.

This ensures that the links rendered by Laravel application deployed behind LB are valid even if the application is deployed via prefix URL.
Example routing setup:
route /admin/(.*) => Laravel backend /$1
in this case links rendered by Laravel backend must start with /admin/

c.f. https://github.com/symfony/symfony/pull/37734

This change has been made for 9.x (https://github.com/laravel/framework/pull/40014) being released this month since it was deemed a breaking change (https://github.com/laravel/framework/pull/40003).
Although having your test additions on the master branch might be nice since that previous PR didn't touch the test suite.

This is a breaking change, and cannot be made to 8.x.

Okay sorry I didn't see the PR for 9.x.
After this patch (and the other patch for 9.x which is almost the same), URLs generated by the router (RouteUrlGenerator) are still not correctly prefixed with the value provided by X-Forwarded-Prefix.
The prefix provided by X-Forwarded-Prefix is handled in getBaseUrl() function from Symfony HttpFondation (see https://github.com/symfony/http-foundation/blob/ce952af52877eaf3eab5d0c08cc0ea865ed37313/Request.php#L903)
but in the following code: https://github.com/laravel/framework/blob/283971c37c32fae6283e70c3a53fcb1a926d6350/src/Illuminate/Routing/RouteUrlGenerator.php#L103 the base url is removed from generated URI with a preg_replace call. So after this call, the generated URL becomes invalid.
For the moment, the simplest fix I see is to add a little bit more code to https://github.com/laravel/framework/blob/283971c37c32fae6283e70c3a53fcb1a926d6350/src/Illuminate/Routing/RouteUrlGenerator.php#L106 (we could prepend the prefix here).

Okay sorry I didn't see the PR for 9.x.
After this patch (and the other patch for 9.x which is almost the same), URLs generated by the router (RouteUrlGenerator) are still not correctly prefixed with the value provided by X-Forwarded-Prefix.
The prefix provided by X-Forwarded-Prefix is handled in getBaseUrl() function from Symfony HttpFoundation (see https://github.com/symfony/http-foundation/blob/ce952af52877eaf3eab5d0c08cc0ea865ed37313/Request.php#L903)
but in the following code: https://github.com/laravel/framework/blob/283971c37c32fae6283e70c3a53fcb1a926d6350/src/Illuminate/Routing/RouteUrlGenerator.php#L103
The base url is removed from generated URI with a preg_replace call. After this call the generated URI becomes invalid.

Closing this on 8.x since we are attempting to solve this on the master branch (which is just a few weeks from release). @gguichard are you able to fix the other problem you mentioned?


Add node version dependency in README and package.json

would prevent people from running into issue #3

Thanks for the PR


feat(observability): spin up benchmarking against the Spanner emulator

Pushing up more updates in a couple @alkatrivedi @surbhigarg92, but this PR is currently a draft.


Request to Include ext-ctype and ext-xml Extensions in PHP Chainguard Images

Which image/versions are related to this issue/feature request?
php:latest-dev
php:latest-fpm-dev
php:latest-fpm
php:latest
Issue/Feature description
Hello Chainguard Team,
I hope this message finds you well. I'm reaching out to suggest an enhancement to the PHP Chainguard images that I believe would greatly benefit the PHP developer community. Specifically, I propose adding the ext-ctype and ext-xml (not to be confused with ext-simplexml, which serves a different purpose) extensions to the PHP Chainguard images.
Here's a brief context: Running the command docker run --rm cgr.dev/chainguard/php --modules currently lists the following PHP modules:
[PHP Modules]
Core
curl
date
filter
hash
iconv
json
libxml
mbstring
mysqlnd
openssl
pcre
PDO
pdo_mysql
pdo_sqlite
Phar
random
readline
Reflection
session
sodium
SPL
sqlite3
standard
tokenizer
zlib

[Zend Modules]

As evident from this output, ext-ctype and ext-xml are not present. Here are my reasons for suggesting their inclusion:

Essential for Symfony Compatibility
Symfony, a popular PHP framework, relies on ext-ctype and ext-xml. Without these, developers face challenges in running Symfony-based applications, which rely on these extensions for character type checking and XML processing.

Support for a Wide Range of PHP Applications
Beyond Symfony, many PHP applications and libraries depend on these extensions for essential functionalities. Their inclusion would thus enhance the utility of the Chainguard images for a variety of PHP projects.

Backward Compatibility Concerns
The absence of these extensions in the Chainguard images complicates backward compatibility, particularly for projects transitioning to containerized environments. This inclusion would ease the transition and ensure ongoing support for these projects.

Alignment with PHP Docker Standards
Official PHP Docker images include these extensions. Aligning Chainguard images with these standards would ease transitions between environments and maintain consistency for PHP developers.

Low Overhead, High Value
Adding these extensions would incur minimal overhead in terms of image size and maintenance, while significantly enhancing functionality and developer experience.

The inclusion of ext-ctype and ext-xml would make the Chainguard images more versatile and user-friendly, particularly for developers working with legacy projects and frameworks like Symfony.
Thank you for considering this proposal. Your efforts in maintaining these images are highly valued, and this addition would be a substantial benefit for the PHP community.
Best regards,
Marc
@erikaheidi @imjasonh FYI

Deciding which extensions to include has been tricky, but I believe this is a legit request. Both extensions are widely used and it would be beneficial to support Symfony applications. Should I send a PR @imjasonh ?

Sorry I missed this message! Yeah @erikaheidi any changes you think we should make, send a PR and I'll take a look.
If there are more tests we can add to demonstrate what features we do/don't support, that would be helpful too. It'd also guard against accidental regression.
Thanks for raising this!

@erikaheidi
I would like to ensure that the Chainguard PHP image aligns with the official Symfony documentation requirements, particularly in terms of the necessary PHP extensions. As per the Symfony docs (https://symfony.com/doc/current/setup.html), the following PHP extensions are essential and should be active for optimal performance and compatibility. Some
Of them are already compiled eg. activated:

Ctype
iconv
PCRE
Session
SimpleXML
Tokenizer

These extensions are usually installed and enabled by default in most PHP 8 installations. I believe it would be beneficial for us to double-check and confirm their presence and activation in the Chainguard PHP image. This step will ensure that the image is fully compatible with the most frameworks that rely on these extensions.
Could we discuss the possibility of verifying and, if necessary, incorporating these extensions into the Chainguard PHP image?

I think most of these are already available, but we'll double check! I am starting work on this issue.

A quick update on this: I created a demo repository with a Docker Compose setup that runs a basic Symfony app. I wanted to do this to validate which extensions we will need. I did the same with a Laravel app.
Here are the extensions I needed to add:

php-ctype
php-xml
php-dom
php-simplexml
php-pecl-mcrypt
php-fileinfo

These are all already available in Wolfi. I will send a PR to include them by default.

Hello everyone.
A month has already passed. Are there any further tasks or information available here? Is there anything I can do from my side? In the meantime, I am compiling my own CVE-free images for PHP, distroless.
Cheers

There is another library that is needed to run things like Laravel Horizon (1st party Laravel package for queue management), which is pcntl_async_signals(). This is a PHP method that needs to exist on the image. I think this is the only remaining thing that is preventing me from using Chainguard (though I've not tested exhaustively yet).

This issue seems incomplete as there appears to be no clear ownership from the staff team. It doesn’t make much sense to continue pursuing this issue if the primary focus remains solely on financial aspects without addressing the underlying concerns.


Pull-up plugin-management of maven-deploy-plugin to the BOM

Pull-up plugin-management of maven-deploy-plugin to the BOM so that the version of the plugin used to deploy the BOM is the same as that used to deploy the remainder of the project.
why: currently the version of the deploy plugin used to deploy the BOM not controlled by the build.  So it will be under the influence of the local Maven environment.  This could effect build repeatability.

I think moving to Maven 4 is the way to go for supporting BOM files.

I don't thinks this request is anything to do with maven 4 or BOM related.  The BOM here is just the top level of the build.  The suggestion was to just move it up to top level.  I think the word BOM should have been left out and just said to be moving it up to root of the project.  However, this screams many other issues.  The parent is on same level as the modules which is bad design with maven.  And its that parent that wants to pull from the bom.  Its a bit backwards.  This all becomes clear the moment relativePath is being rewritten everywhere needlessly.  I think the bom should be a module that runs last since it depends on all.  While the build works, having a bom with dependency management that doesn't actually exist at that point is odd design.  But if done at the end of the build it would make more sense logically as those modules really would exist by then.
As it stands do I think this change alone is worth the effort, not really.  Do note I'm not a owner or frequent contributor or anything here.  Just saying moving like this kind of avoids the real issues and is a whack a mole situation.  Maybe this repo might give a pointer to help https://github.com/Waffle/waffle/tree/master/Source/JNA.  Do note its down pretty far in source due to having dotnet stuff higher up (needs split).  There is a pom further up used to launch everything but cannot be used to release and it suffers same issues.  It wants to be on same plugins but the plugins are defined in the parent.  But if you ignore that part and just look at that folder structure you will see it has the parent pom driving the thing and the bom is just a module not the parent-parent.

While I agree with @hazendaz in that I don't believe its anything per se to do with a BOM I do however think this minor change would be good to apply. If anyone rebuilds SLF4J and uses altDeploymentRepository this will lead to inconsistent results as Maven changed the format of that between 2.x and 3.x - and as SLF4J uses both versions of the maven-deploy-plugin this can occur.


Dependency Bump

Issue #, if available:
N/A
Description of changes:

Updated main project to NetStandard 2.1.
Update test project to NET6.0 (LTS).

All unit tests are still passing.

Snyk Security vulnerability (CVE-2021-26701)

Microsoft.AspNetCore.DataProtection.Extensions (v2.0.3)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Code analyzers was using the old deprecated one. I am not sure how you are using them (pipeline/scripts/build etc.) I would recommend stripping it out if its not a needed dependency (for the library that is). Otherwise the newer versions are found in the roslyn analyzer repo latest version is 6.x, earliest version 5.x.
I will leave it out of this PR for simplicity.

Bump @normj

Thanks for the friendly reminder and the PR. It is in dev branch and I'll work on getting it released out.

Thanks for the friendly reminder and the PR. It is in dev branch and I'll work on getting it released out.

Awesome, thanks for all your hardworking.

I just pushed out version 3.0.0 with this PR. As we talked about it is a major version bump because of the platform changes.

Thanks for this @houseofcat and @normj


飞桨1组  小组3  作业7_24_1

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add exception checking in TextViewConnectionListener

Summary of the changes

@ToddGrun reported VS crashing when running integration tests due to an exception in Razor's options logic.
This PR catches any exceptions so we no longer crash.

Not sure I'm following what just happened:

It looks like the PR was closed, GH says it's closed, but it also says merged?

Other PRs still show merged if they were merged.

lol, I have no idea what happened, I was trying to switch the base branch and fix the commits, and it just automatically closed the PR. o___o

@allisonchou recommend restarting your PR but reusing the commits that were here: Create a branch against your preferred target and then cherry-pick the commits you had in this PR.
/fyi GitHub closes PRs whenever it notices there are no changes compared to the target branch. Perhaps that happened for a moment w/ this PR❔ Alternatively, the Closed status is just temporary and GitHub will get it right eventually.


Use app. prefix for parameters

I changed the locale setting to put the app. prefix in order to follow best practices.
https://symfony.com/doc/current/best_practices.html#use-short-and-prefixed-parameter-names :

Consider using app. as the prefix of your parameters to avoid collisions with Symfony and third-party bundles/libraries parameters. Then, use just one or two words to describe the purpose of the parameter

Yes, let's follow our own best practices 🙏
Thanks @seb-jean!


Add progress bar

It would be convenient for users to understand the progress of downloading.
maybe we can integrate https://github.com/cheggaaa/pb into this project.

closed by #69


fix(core): assign embedded properties from class objects

This fixes entity assign not merging embedded properties when the source has a prototype.
Closes #1083

The issue here is that Utils.merge only accepts POJO sources, and since #1087 only affects embeddables, I just destructured it when otherwise.
I noticed the util is used in multiple places, so I didn't want to change it and add some weird behavior that might not be accounted for.

Thanks!


However, each time I use the extension, Anki throws an error saying the search isn't valid (though oddly enough, it seems to search correctly anyway, it just throws the error on top of it).
It would be great to add an option not to search at all, too.

Is the error coming from the animebook page or anki? If it's anki, what version of anki are you using? Got a screenshot?
I'm going to add an option not to search at all in #20 by the way.

The error was coming from Anki, I was on ver. 2.1.22, updated to 2.1.40 and the error no longer appears, so I guess this thread is a bit useless now... I still attached a screenshot of the error just in case. Thanks!

Oh, thanks! I did happen to notice a typing mistake in one of the queries I was sending to Anki, and it may have been coincidentally working in the later versions of Anki, but not in 2.1.22. I fixed it, so once I get a new release out, hopefully other people won't have to run into this.


Gaussian mixture modeling for neighborhoods

Is your feature request related to a problem? Please describe.
The current kmeans approach to neighborhood analysis is extremely fast and gives good results. However, the hard assignment to clusters means we don't get a probability vector of neighborhoods. There are lots of cool analyses that can be done looking at transitions between neighborhoods that rely on the type of output we get from spatialLDA, but this is prohibitively slow
Describe the solution you'd like
Play around with the mixture package in scikit to see how it performs. Do benchmarking to evaluate 1) how much slower it is, 2) how similar the output is to kmeans, and 3) how smooth the transition probabilities are.
This would be a good opportunity to use Erin's original TB dataset, as she did extensive neighborhood analysis with kmeans and spatialLDA that we can compare against.

Sad


Add data_tamer source to Humble

related to https://github.com/ros2-gbp/ros2-gbp-github-org/issues/393

I'm not so sure about this, there are so many other data_tamer packages already out there ;)


Fix loading the state from localStorage

Re-implementation of 7052e64 from #1 to avoid conflicts.

Thank you. I will check that.


handle artifacts generations

read all artifacts (files) produced from file generation maven plugin
add artifact-generations store in mongo to store these
add artifact generation apis for fetching these files
Related to issue https://github.com/finos/legend-sdlc/issues/504

Close in favor of https://github.com/finos/legend-depot/pull/101 based on @elopezcastro comments


FindElementsByTagName("Window") returns zero results

I am trying to get a list of all the windows on the desktop with the code below. However, I get zero results... should this work?
var desktopCapabilities = new DesiredCapabilities();
desktopCapabilities.SetCapability("app", "Root");
desktopCapabilities.SetCapability("platformName", "Windows");
desktopCapabilities.SetCapability("deviceName", "WindowsPC");
using (var desktopDriver = new WindowsDriver<WindowsElement>(WindowsDriverUri, desktopCapabilities))
{
    var windows = desktopDriver.FindElementsByTagName("Window"); // returns zero results
}

I tried the following, it didn't return zero. But, it returned the wrong value.
var windows = desktopDriver.FindElementsByClassName("Window");

Hi @mdesousa,
Would you please retry your scenario using WinAppDriver v1.1.1 release?

It appears to work with v1.1.1... thanks! FYI, I noticed in the WinAppDriver console a number of exception warnings... the resultset still looked ok, don't know if those exceptions are concerning. See the output below... the "Exception" section (including the "Ignoring new object and keeping existing one" message) was repeated many times (more than 20?). It appears to display one entry for each email that I had opened in Outlook... I've obfuscated and left a single entry for illustration, but you can probably reproduce if you make this call while Outlook is open.
==========================================
POST /session/C36B34B7-9BEE-436A-9D02-E5E24BA05EEC/elements HTTP/1.1
Accept: application/json, image/png
Content-Length: 37
Content-Type: application/json;charset=utf-8
Host: <IP_ADDRESS>:4723

{"using":"tag name","value":"Window"}

Exception: Encountered element sharing the same runtimeId 42.6490686.3.-2

    new Object {[OBFUSCATED], FromField, 42.6490686.3.-2}
    runtimeId 42.6490686.3.-2
    className FromField
    name [OBFUSCATED]
    NativeWindowHandle 0

    existing Object {[OBFUSCATED], FromField, 42.6490686.3.-2}
    runtimeId 42.6490686.3.-2
    className FromField
    name [OBFUSCATED]
    NativeWindowHandle 0

Ignoring new object and keeping existing one

HTTP/1.1 200 OK
Content-Length: 325
Content-Type: application/json

{"sessionId":"C36B34B7-9BEE-436A-9D02-E5E24BA05EEC","status":0,"value":[{"ELEMENT":"42.9836860"},{"ELEMENT":"42.793194"},{"ELEMENT":"42.12387662"},{"ELEMENT":"42.23466224"},{"ELEMENT":"42.6624114"},{"ELEMENT":"42.15603540"},{"ELEMENT":"42.14948448"},{"ELEMENT":"42.5576022"},{"ELEMENT":"42.18091894"},{"ELEMENT":"42.66734"}]}

==========================================

Hi @mdesousa,
Thanks for the update. I'm glad it works out very well for you. Yes there is an issue with one of the Outlook release where some distinct elements sharing the same runtimeId. They are aware of this and are working on a fix. These exceptions are merely warnings to help application developer identify the elements that share the runtimeId.

Hi,
I am stuck with the same above error. Unable to do anything as error keeps on rolling and winappdriver gets stuck, please help.

Share your XPath

@anshilabhi
What is you application framework? I had such issue when working on Sciter application. This framework have issue with handling invisible elements are not handled properly and they are returned in indefinite loop what cause winappdriver to stackoverflow exeption after all. The workarround is to do your querries in root elements where you are sure there is no duplicated looped elements.


Add canonical textgen samples

This is part of a larger project to convert samples to a standard format across all languages that can be used in devsite.
See internal doc: https://docs.google.com/spreadsheets/d/10X--DI0YPL3QfEMObe387rYg0t4-kuv33yKn2TIal7E/edit?resourcekey=0-kfy1Ry4YAZh6O4DltLjV9w&gid=0#gid=0
This PR addresses the first file ("text_generation") described in that doc which includes 9 examples. The code to be used in docs is delineated with [START] and [END] region tags marked with the canonical name for that sample.
Media files were taken from the live docs site (https://ai.google.dev/gemini-api/docs/vision?lang=node#technical-details-video) when available and from the Python repo when not available (e.g., audio sample): https://github.com/google-gemini/generative-ai-python/tree/main/third_party
I will delete and replace existing samples files in future PRs as the new samples for those cases are written.

Made some more changes after doing live tests:

Removed web/ dir - doesn't seem like it's staying up to date, should be easy enough for a web developer to consume Node examples.
Use a mediaPath path for all media in the samples.
Upload video files for inference instead of trying to pass them inline.


Reorganize mnit submodules

So now Linux clients import mnit::linux instead of mnit_linux and Android clients import mnit::android. Also rename mnit_display and mnit_input to mnit::display and mnit::input respectively.
This may look familiar as it was a part of #1671.

+1

:shipit:


Coverage remained the same at 87.363% when pulling fe3e60c4412145e0735bbc4e930038c8ae4b85ad on tk8817:breakpoint_support into c49e2ae9c0ee70a212b0e6c74eaf322d166583c1 on cheezy:master.


[ROS2] [RQT_PUBLISHER] Traceback when closing/exiting

When using rqt_publisher under dashing, I get a traceback when closing the application on OSX. This might be related to breaking changes in dashing related to passing the context object to rcl_init().
See here: https://github.com/ros2/rcl/blob/master/rcl/include/rcl/init.h#L82 and the related PR: https://github.com/ros2/rcl/pull/336
Traceback (most recent call last):
  File "/Users/karsten/workspace/osrf/ros2_full/build/rqt_gui_py/src/rqt_gui_py/rclpy_spinner.py", line 32, in run
    executor.spin_once(timeout_sec=1.0)
  File "/Users/karsten/workspace/osrf/ros2_full/install/lib/python3.7/site-packages/rclpy/executors.py", line 695, in spin_once
    handler, entity, node = self.wait_for_ready_callbacks(timeout_sec=timeout_sec)
  File "/Users/karsten/workspace/osrf/ros2_full/install/lib/python3.7/site-packages/rclpy/executors.py", line 649, in wait_for_ready_callbacks
    return next(self._cb_iter)
  File "/Users/karsten/workspace/osrf/ros2_full/install/lib/python3.7/site-packages/rclpy/executors.py", line 532, in _wait_for_ready_callbacks
    self._context.handle)
RuntimeError: Failed to initialize wait set: the given context is not valid, either rcl_init() was not called or rcl_shutdown() was called., at /Users/karsten/workspace/osrf/ros2_full/src/ros2/rcl/rcl/src/rcl/wait.c:130

This issue might be one example for branching out to dashing-devel as described here: https://github.com/ros-visualization/rqt_publisher/issues/10

closing this as the source of issue lies in rqt_gui_py, see https://github.com/ros-visualization/rqt/issues/198


Create Target Collection Before Starting Wizard

Suggestion to create the Desktop Analytics target collection before starting the wizard, as you cannot select outside of the wizard once you start it.

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

ID: b58dfdf5-6060-e4fa-a806-456ca196ede7
Version Independent ID: 1f8325fb-518d-412e-f011-7fd62311b087
Content: Connect Configuration Manager - Configuration Manager
Content Source: sccm/desktop-analytics/connect-configmgr.md
Product: configuration-manager
Technology: configmgr-other
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

Hi @sandersap, good feedback. That suggestion would make a great tip here and in the tutorial.

Thanks to @baardhermansen for making this change in the linked pull request. I'm closing this issue. It should publish live tomorrow.


[wp81] fix for CCUserDefaults.Flush corrupting files

This pull requests fixes problem on WP8.1
After multiple calls to CCUserDefaults.Flush file is corrupted.
I experienced this in my game.
Additional information (it's known issue for WP developers): http://stackoverflow.com/questions/29799265/filemode-open-and-filemode-openorcreate-difference-when-file-exists-c-sharp-bug
Would be great after (and if) this pull request is merged, get a fresh Nuget version.
Thank you.

Thanks Alex


support customized labels

support customized labels
Master Issue: #156
Motivation
to add allow user add customized labels
Modifications
Update the deployment template for new labels
Verifying this change

[x] Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.
Documentation
Check the box below.
Need to update docs?

[ ] doc-required
[x] no-need-doc
this change is self-explain
[ ] doc

@mingmcb thx for your contribution! just few suggestions for this PR:

Can we rename to use labels and podLabels? which I think will be more consistent with some existing fields like the podAnnotations.
We should also add these two fields in the https://github.com/streamnative/pulsar-resources-operator/blob/main/charts/pulsar-resources-operator/values.yaml then other users can notice them.


How to assign session "program_name"?

Issue Type: Feature Request
How assign seesion "program_name" by Azure Data Studio?
Like sys.dm_exec_sessions information, we need change it from Azure Data Studio.
Azure Data Studio version: azuredatastudio 1.3.7-insider (5a3087859918b1564d2622b95defa662facdbef6, 2018-12-07T02:33:40.014Z)
OS version: Windows_NT x64 10.0.17134

This is similar to #3991 , closing as duplicate.


htmlDocContentDumpFormatOutput Messing Up HTML Document

The call to htmlDocContentDumpFormatOutput in msc_crypt.c truncates the document in strange ways.  Even a document as simple as:
<!DOCTYPE html>

This is a link
This is another link

has the < on the first line truncated with more sophisticated documents having more issues.  The HTML is generated fine with the hashes (see my path for the other bug) and I verified this by using the following:
FILE *fp;
fp=fopen("/tmp/htmlDocDump.txt", "w");
...
htmlDocDump(fp,(xmlDocPtr) msr->crypto_html_tree);

in inject_hashed_response_body.  Any ideas?

Just to add an example,  the default Apache homepage on Ubuntu was truncated by about 8000 bytes.  Here are the debug logs:
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] Hook insert_filter: Adding output filter (r 7f02fdacc0a0).
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][9] Output filter: Receiving output (f 7f02fdadac00, r 7f02fdacc0a0).
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] Starting phase RESPONSE_HEADERS.
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][9] This phase consists of 0 rule(s).
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][9] Content Injection: Nothing to inject.
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][9] Output filter: Bucket type MMAP contains 11321 bytes.
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][9] Output filter: Bucket type EOS contains 0 bytes.
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] Output filter: Completed receiving response body (buffered full - 11321 bytes).
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] init_response_body_html_parser: assuming ISO-8859-1.
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] init_response_body_html_parser: Successfully html parser generated.
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] Signing data [/docs/2.4/mod/mod_userdir.html]
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] hash_response_body_links: Processed [0] iframe src, [0] hashed.
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] hash_response_body_links: Processed [0] frame src, [0] hashed.
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] hash_response_body_links: Processed [0] form actions, [0] hashed.
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] hash_response_body_links: Processed [9] links, [1] hashed.
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] inject_hashed_response_body: Detected encoding type [ISO-8859-1].
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] inject_hashed_response_body: Using content-type [ISO-8859-1].
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] inject_hashed_response_body: Copying XML tree from CONV to stream buffer [%zu] bytes.
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] inject_hashed_response_body: Setting new content value 3013
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] inject_hashed_response_body: Stream buffer [3013]. Done
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] Hash completed in 7996 usec.
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] Starting phase RESPONSE_BODY.
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][9] This phase consists of 0 rule(s).
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][9] Content Injection: Data reinjected bytes [3013]
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] Output filter: Output forwarding complete.
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] Initialising logging.
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] Starting phase LOGGING.
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][9] This phase consists of 0 rule(s).
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] Recording persistent data took 0 microseconds.
[27/Jan/2017:12:24:59 +0000] [<IP_ADDRESS>/sid#7f02fdc13b78][rid#7f02fdacc0a0][/index.html][4] Audit log: Logging this transaction

Same as #742


Add ValueConversionAttribute

Is it possible to add the ValueConversionAttribute to the converters? AFAIK it is only available in net full framework and netcore so compiler-ifs would be required.
I could send a pull request if you agree.
https://docs.microsoft.com/en-us/dotnet/api/system.windows.data.valueconversionattribute?view=netframework-4.8

Ok. Can you maybe prepare a minimal PR so I can see how you’d add it - and how you’d use it? Is there a concrete use-case for these annotations? You cannot add it to generic classes, right?

No, generic types cannot be used in attributes. This is why I suggest to add the attribute to all child classes instead of base classes.
Use-case?
A. It helps consumers of the lib to understand the input and outcome of the converter.
B. We have developed the idea of cascading converters (https://github.com/thomasgalliker/ValueConverters.NET/issues/12) in a less intrusive way. Meaning that existing converters do not need to have a reference to a 'PreviousConverter'. This implementation relies on the ValueConversionAttribute and is used like this:
<conv:ValueConverterGroup x:Key="Int32ToInvertedVisibility">     <!--  Add any IValueConverter that is decorated with ValueConvertionAttribute  -->     <conv:Int32ToBoolean />     <conv:BoolInverter />     <conv:BooleanToVisibility /> </conv:ValueConverterGroup>
I would provide a PR if you are intrested. Otherwise adding the ValueConvertionAttribute to your converters allows us to place them inside our ValueConverterGroup.


Citation search result link must be citations article link

Related https://github.com/okulbilisim/ojs/issues/1264


question => vscode integration

My vscode setting is this
"terminal.integrated.shell.windows": "E:\\Program Files\\wsl-terminal\\open-wsl.exe",
"terminal.integrated.shellArgs.windows": ["-a"],

I want to display the terminal in vscode,  (ctrl + `)
but keep it open in a new window,
can I fix this?

No.
Note: To be used as an integrated terminal, the shell executable must be a console application so that stdin/stdout/stderr can be redirected.


Bug - Incorrect Multiplication Result for Decimals less than 1

Describe the bug
When multiplying a decimal by another one, and when the result is less than 1, the result is incorrectly represented.
To Reproduce
Steps to reproduce the behavior:

Run the multiply program with arguments 0.5 and 0.5.
The calculation result is 0.025, instead of 0.25.

Expected behavior
Should output 0.25.
OS

All OS’s are affected.

Additional context
Not yet

Fixed in e28c0cd.


rename file to match folder to allow for convention based plugin mana…

…gers (OMZ+) to use

Hi! @Ashtonian I'm maintaining a fork of this project, where the plugin file is named according to the convention, together with other fixes and improvement.
https://github.com/decayofmind/zsh-fast-alias-tips


Table 'performance_schema.global_status' doesn't exist

after importing my2 dashboard on grafana i get this err message on page load and nothing works.
I imported my2.sql with no issue. and also created a new datasource for my2 database with my2 user.

my mysql info: mysql  Ver 15.1 Distrib 10.1.44-MariaDB, for debian-linux-gnu (x86_64) using readline 5.2

Same issue here.

in variable dashboard (MyVar2) replace with :
select concat('MySQL version: ',version(), '   Created at: ', date_format(min(create_time),'%Y-%m-%d %T') ,
'  Started at: ', date_format(date_sub(now(), INTERVAL variable_value second),'%Y-%m-%d %T')) info
from   information_schema.tables,information_schema.global_status
where variable_name='UPTIME'
group by variable_value
Global_status table is contained information_schema and not in performance_schema.
The same problem is in the my2.sql.
Try to use an editor and replace all performance_schema.global_status with information_schema.global_status. Good luck

Global_status table is contained information_schema and not in performance_schema.
The same problem is in the my2.sql.
Try to use an editor and replace all performance_schema.global_status with information_schema.global_status. Good luck

MySQL 5.7 and 8.0 use performance_schema while older MySQL versions and MariaDB use information_schema. my2.sql code tries to use the right schema for the running version but it is not bullet proof. For Grafana dashboards I have 2 different versions... maybe I'll do the same for the my2 collector. Which fork and version are You using?

Grafana dashboards for MySQL and MariaDB are different

I'm adding extra information to deal with the missing table.
First check where the table actually is. It may be at information_schema or performance_schema.
run this query to know where:
select TABLE_SCHEMA , TABLE_NAME from information_schema.TABLES t  where TABLE_NAME ='global_status'
Also you may enable performance schema at my.conf or mysqld.conf
[mysqld]
performance_schema=ON


transpilation broken with the latest release

its not obvious to me what changed with the build, but the latest build appears to no longer transpile to the degree that it used to. it now leaves es module import syntax in the bundle that is set to main in the package.json.
it probably does make sense to include a version that uses es modules like this, but it would need to be defined as module in the package.json rather than main in order for webpack to handle it correctly.
i use rollup in my projects to easily produce a common-js bundle and an es-module bundle that i define as main and module, respectively.

You're right, for not obvious reason the transpilation was broken (just for the module part). The new version (0.2.5) contains a small fix to the .babelrc config file that should fix the problem (or at least works in the test environment I'm using). Can you please confirm that?

looks like that took care of it. thanks so much for the quick resolution!


update script for a configurable editmode

i think i got this right.  could be wrong, have to check it first, the ai i used to help me made the most, i added only at one of the first lines the confifgurable part. Ill put it to a test and report back on my issues, if errors or success occured

found an error, should be removed now, testing again

ok did the final bugfixes now... testing is done, if you change the default edit value to false, it works as expected: so spawned vehicles are non-editable as default, else the vehicles are editable. Bugfixes are done

rejected


Initialize NN weights in a more favourable space

We need to find a way to initialize the NN weights in a more favourable space. This would most likely speed up the training. A linear function with a slight positive incline could be a sensitive way to start.
Maybe @redouanelg would like to try this? This could be an easy first issue, very ML-focused :)

This doesn't seem to be an issue at all with the current training.


Use go env to fetch GOPATH to support Go 1.8

Go 1.8 do not require env GOPATH to be set and make will fail if it isn't set.
@beorn7

Thanks.


add date.css to examples

What does this change?

@ayman-alkom It helps to provide a before/after screenshot comparison.

In fact I think that this PR is not necessary, it is just about setting the widths of the date input select items, in addition to some media query, witch can any one do in his own way.

@dereuromark this the after/before you mentioned


The plugin break the right click on canvas

With the plugin activated, the right-click menus on the canvas don't appear. It's very annoying because a lot of options aren't accessible otherwise.

Thank you for bringing up this issue, Melody!
In the current version of the plugin, the right-click menu is designed to appear when the mouse button is released, not when it’s pressed. I’ve experienced similar behavior during development, and it seems that this functionality may not work properly when Obsidian is not in focus.
It’s important for the right-click gesture feature to remain functional even when Obsidian is not focused, and I’m keen to fix this as soon as possible. Your feedback is very helpful in identifying this issue.
I appreciate your patience and will work on resolving it. Thanks again for reporting!


visual indication of removed port

Loading...

Loaded.

"Ghost" port exists in the initial library, inferred from the edges, but has been deleted from the component. I made it show up at the end of the lists, but we should indicate it isn't a port visually.

Will be fixed by not inferring ports from edges, users of the-graph always have to specify component ports through library.


Feature request - Caching

When you put multiple link preview components to the screen with same link, then it makes multiple fetches to the link. In our use case we could have thousands of previews with same link

Hmm where I would store the cache given the component can be placed anywhere, including lists? The cache makes sense on the app level, but on the library level, not sure.. I will need async storage or something, but this is a native dependency, plus some people use alternative like mmkv. Do you have ideas how this can be implemented?

@demchenkoalex  I know this is late and to be honest a cache can be more trouble then good but what about a InMemoryCache like apollo graphql

will not work on this in the foreseeable future :/

just an idea on implementing this without native dependencies and giving user flexibility: https://docs.pmnd.rs/zustand/integrations/persisting-store-data
zustand essentially allows you to pass in an object that defines the get set and remove cach functions and it handles them accordingly on startup and during use.
To make it simpler for this library you could just have an optional prop instead of a wrapper cacheHandler={{getItem:AsyncStorage.getItem, setItem:AsyncStorage.setItem, removeItem:AsyncStorage.removeItem, key: 'nameOfCache'}} with getItem(key+otherconfig) called on mount. and setItem(key+otherconfig) called onPreviewDataFetched.


Minimal working code example in experiment API docs

Following the template in #1221, add minimal working code examples for all experiments for ease of use.

Hello @coruscating, can I work on this issue?

Hi @coruscating, if my understanding of this issue (and #1221) is correct, I have to add  a ‘ #section: example ‘ and  a sample code for the experiment( for example,  T1) on the source(/library/characterization/t1.py). Even if it would help for user to do the experiment (T1), the information is duplicated because  ‘#section: manual’ cites /manuals/characterization/t1.  The use of :hide-code: means that the choice of the backend  still remains in the source.  I am happy if you have an alternative good idea?

Yes, there will be a bit of duplicate code in the case where experiments have manuals, but most of the existing experiments don't have a separate manual. The purpose of this code example and the manual are also different: the example is meant to be a really quick code snippet that users can copy and paste to get an experiment running, whereas the manual is supposed to cover more advanced usage and detailed technical info about the experiment.
I think keeping the backend in the source and hiding it in the output is fine—we're only using simulators for documentation, so the idea is that users will instantiate their own backend on which to run the experiment. Do you have more concerns/ideas about this?

How many experiments do you want to add the code snippet?

@Naohnakazawa Ideally all experiments, but if that's too much work at once, you can make a PR for a single category of experiments, for example all characterization experiments.

Hello @coruscating, I have accidentally clicked the button on assignees, and unassigned myself.
Can I continue working on this issue?

@Naohnakazawa Yes, of course. Reassigned you, thanks.


Package language server build (vsix) with the tools distribution

Description:
Currently the vsix file is not in the tools distribution. If you have different builds of Ballerina on your machine, you cannot easily change the VSIX in Visual Studio to match the version you are using, because it requires a rebuild.
Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

We do not include vsix file inside platform distribution but for each release we provide vsix file separately from the ballerina.io website.  For older versions users can visit archived versions and download the relevant vsix file.


Run tests refactoring

-- refactor run_tests.py
-- preparation for running run_tests.py with different dockerfiles for each language
-- improve handling of --arch parameter on linux (now autoselects x86 and x64 docker images)
-- move dockerfiles under tools/dockerfile

Green enough.

Rebased, but was green before: https://grpc-testing.appspot.com/view/Pull requests/job/gRPC_pull_requests/5575/

Copyright test failure is unrelated.

Made this mergeable again.

LGTM. Merge on green.

Interop tests are broken for other reasons, everything is green (enough).


Feature request: disable method names

It's not possible to disable method names, is it? For me, filename is usually enough IMO logging without method names makes it a lot cleaner.

Spitting through the code I see it is possible indeed, though the example in the readme does not contain the option. My feature request changes to: request to update readme :-)

@dirkpostma This is already present in readme. Check [Advance Usage] (https://github.com/DaveWoodCom/XCGLogger#advanced-usage).
systemLogDestination.showFunctionName = false


🌟Improvement: Complete UI and Functionality of FeedbackForm!

🎮 feature Request
Hey @ayush-t02
i have a feature enhancement request to make-
Currently the feedback form of the website is not well aligned, not styled well to follow the website's game theme.

The theme of the feedback form is not matching
There Should not be any Vertical Scrollbar for a feedback form as it leads to inconsistency
The Star Rating System is Enough for giving the rating, so there should not be the "Reaction emojis"
The overall alignment of the input tags is not well aligned

As you can see here - ( I pointed out some of the issues with the violet colors)

Please assign me the issue to fully Improve the UI and functionalities to it
Point down the features
.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

@ayush-t02 i want to work on this.

let the reaction emojis be there


Update the Twitter logo to X

To Reproduce -
Go to the footer of the website.
In the current version of the website the logo of Twitter is not updated.

Hey @vinfinity7, the pull request you opened updates the Twiiter logo in our app.crowd.dev app. This issue is for our crowd.dev public website 😄
Nonetheless, left you a comment in the PR since it's not working as expected

I have created a pull request regarding this issue. Please merge it and label it with hacksquad.

I am new as a contributor and I would like to work on this issue. Can you co assign it to me?

Hey @joanagmaia want to work on this issue can you please assign me with this issue. :)


Initial linting pass over all legacy JS

Just fixing the most common errors like:

Indent with 2 spaces
Replace var with let/const
Replace function() expressions with arrow functions
A few other trivial tweaks like spacing

I've made eslint ignore all non-trivial issues for now

Thanks! Will look into that


fix(style): use secondary-text-color for secondary_info to respect themes

Home assistant entity list is using secondary-text-color to set the secondary_info color. This should be respected to be themable and consistent to HA default behaviour

Thnaks!


Adding Productivity compatibility with plates/dust

Bug report:

While you can insert productivity modules in masher/electric furnace , the bonus effect was not having the desired effect.

Changes:

Created file 5dim_resources/data-final-fixes.lua
Added some helper functions and logic to check when 5dim_resources is added in 5dim_resources/data-final-fixes.lua
Added "5d-iron-plate/dust" compatibility with productivity modules
Added "5d-copper-palate/dust" compatibility with productivity modules

Evidence before:

Evidence After:

Thanks you!


[python3]make python3 support unix

/azp run

Just tried this PR and I was able to successfully install Python3 on macOS.


Light2D does not work across or inside CanvasLayers.

Godot version:

b63c506
OS/device including version:

Win10 64-bit GeForce 9600 GT
Issue description:

When putting a Light2D or Node2D into another CanvasLayer, there is no longer a way to use Light2D nodes.  They only operate on the default canvas layer, no matter where they're located.
This currently makes these features impossible to use together.
Here depicts a sprite getting both lights, and the other getting neither.  Though the lights are contained in their respective canvases.

The problem exists in GLES2 and GLES3, in game and in editor.
Steps to reproduce:
Add a CanvasLayer and try out combinations of nodes and lights.  You will notice the lights disregard the CanvasLayer that is their parent.
Minimal reproduction project:

3.1 Light2D CanvasLayer.zip

You need to set the Light2D Min Layer and Max Layer ranges correctly.

Thanks, my mistake.  Misunderstood these, didn't realize they were canvas layers.

You need to set the Light2D Min Layer and Max Layer ranges correctly.

this!


ci: fix GHA CI build matrix

It would appear that requesting Go 1.21 in the build matrix has no effect whatsoever on what go toolchain is actually used to build the project. See https://brandur.org/fragments/go-version-matrix.
Specifying 1.22.0 in the go.mod go directive forces the 1.22 toolchain to be used for building Go source files, ignoring whatever Go version we set in the GHA CI matrix.
You can see the "wrong" toolchain to be used in the screenshot below:

Luckily we can enforce the right setting by setting GOTOOLCHAIN=local which forces Go to use the locally available toolchain instead of downloading the one requested by go.mod.
If we want to be one minor version back compatible we need to drop the go.mod go directive to 1.21.0, too.
As for the image builds, the Docker image we use sets the GOTOOLCHAIN=local by default:
$ docker run -it golang:1.22.5 /bin/bash
root@f0dde6c33f17:/go# go env GOTOOLCHAIN
local

Ah, yes, these are the fun ones. I think this is caused by https://github.com/distribution/distribution/pull/4347, which switched to the SemVer format, which forces it to download the version. I've tried to avoid using that format, but not sure how much longer Go allows the old format to be used.
Honestly, I think that the go-setup-action should also either set GOTOOLCHAIN=local or produce a big fat error when trying to specify a version without it being set.

What's with the new certs in the vendor folder? Is that really an artifact of the go toolchain change?

What's with the new certs in the vendor folder? Is that really an artifact of the go toolchain change?

I am not entirely sure. The certs were added as part of running go mod vendor. I remember looking into that at the time
I had a look at the dependency repo (github.com/google/s2a-go) and saw that the certs (used for testing only) were added in https://github.com/google/s2a-go/pull/103  which was included in https://github.com/google/s2a-go/releases/tag/v0.1.1
Our deps are pinned to v0.1.4 https://github.com/distribution/distribution/blob/0e850894d651bbd4e47e612c387baae82c7d1e9f/go.mod#L63
So it's a bit strange. My theory at the time was there is a difference in how different toolchain versions handle dependency vendoring (i.e. go mod vendor command) 🤷♂
Now, I'm thinking we should bump the Go version in the go.mod to 1.22 anyway, now that 1.23.X has been out for a bit, which will nuke these certs from the vendor

My theory at the time was there is a difference in how different toolchain versions handle dependency vendoring (i.e. go mod vendor command) 🤷♂

Yes, I think newer versions more aggressively pruning files from vendor. It already excluded testdata directories elsewhere, and for these ones already removed the .pem files.  Perhaps the only reason it kept the directory was because it detected the .der files as binary files and kept them before;
https://github.com/google/s2a-go/tree/v0.1.4/internal/v2/remotesigner/testdata

I think we should bump it back to 1.22 🤷♂ WDYT @Jamstah

We can specify the Go version in the go.mod file and split the matrix into two jobs: one that uses the version defined in go.mod and another that uses the local version.
Like: https://github.com/prometheus/prometheus/commit/1e01e97151c9db59178dfbabd5e60f4b3916cb83#diff-b803fcb7f17ed9235f1e5cb1fcd2f5d3b2838429d4368ae4c57ce4436577f03fR44

We can specify the Go version in the go.mod file and split the matrix into two jobs: one that uses the version defined in go.mod and another that uses the local version.
Like: prometheus/prometheus@1e01e97#diff-b803fcb7f17ed9235f1e5cb1fcd2f5d3b2838429d4368ae4c57ce4436577f03fR44

Isnt that what this PR is actually doing? Am I missing something?

I've decided to bump the Go version in go.mod and Dockerfiles

We can specify the Go version in the go.mod file and split the matrix into two jobs: one that uses the version defined in go.mod and another that uses the local version.
Like: prometheus/prometheus@1e01e97#diff-b803fcb7f17ed9235f1e5cb1fcd2f5d3b2838429d4368ae4c57ce4436577f03fR44

Isnt that what this PR is actually doing? Am I missing something?

I mean, we don't use the matrix, and break them into two jobs.

I mean, we don't use the matrix, and break them into two jobs.

I think the matrix works well. No need for unnecessary redundancy IMHO 🤷♂


Fix an issue that /reveal can still redirect users to other domains.

I tried to fix it in #57 but it was incomplete. It's still possible to specify target=.malicious.com to redirect users to google.org.malicious.com.

LGTM

Thanks for the review. Merged.


getting an error while installation for process of mezzanine

python manage.py createdb
Traceback (most recent call last):
File "manage.py", line 14, in
execute_from_command_line(sys.argv)
File "/usr/lib/python2.7/dist-packages/django/core/management/init.py", line 385, in execute_from_command_line
utility.execute()
File "/usr/lib/python2.7/dist-packages/django/core/management/init.py", line 354, in execute
django.setup()
File "/usr/lib/python2.7/dist-packages/django/init.py", line 21, in setup
apps.populate(settings.INSTALLED_APPS)
File "/usr/lib/python2.7/dist-packages/django/apps/registry.py", line 108, in populate
app_config.import_models(all_models)
File "/usr/lib/python2.7/dist-packages/django/apps/config.py", line 202, in import_models
self.models_module = import_module(models_module_name)
File "/usr/lib/python2.7/importlib/init.py", line 37, in import_module
import(name)
File "/usr/local/lib/python2.7/dist-packages/mezzanine/generic/models.py", line 20, in
class ThreadedComment(Comment):
File "/usr/lib/python2.7/dist-packages/django/db/models/base.py", line 168, in new
new_class.add_to_class(obj_name, obj)
File "/usr/lib/python2.7/dist-packages/django/db/models/base.py", line 297, in add_to_class
value.contribute_to_class(cls, name)
File "/usr/local/lib/python2.7/dist-packages/mezzanine/generic/fields.py", line 77, in contribute_to_class
extant_fields = cls._meta._forward_fields_map
AttributeError: 'Options' object has no attribute '_forward_fields_map'

Works fine for me - you might like to try the mailing list for help.


Set systemd [OOMScore Adjust=-999] to prioritize OOM Killer?

I just noticed that CRI-O configures itself to be one of the last things the OOM killer will kill.
https://github.com/cri-o/cri-o/issues/5126
It does this by setting:
/usr/lib/systemd/system/crio.service:OOMScore Adjust=-999

Let's review whether we should be setting this for anything else.

I don't believe we need to do this for anything else that I can think of.


Activation key with space appears as '1'

Unless you have a hidden version this problem #2662 is still there.
The solution is to replace kVK_Space in file NDKeyboardLayout.m with
// SPACE: should be kVK_Space, but 0x31 clashes with the '1' char - they both have the same unichar value
  {0x97, 0x31, {0x23B5, '\0', '\0', '\0'}}, // 0x02FD

The fix for this will be released in version 2.0 in Quicksilver tomorrow (25th March). Sorry for the delay, but stay tuned and make sure your Quicksilver says version 2.0!


Blog post for F8 video + podcast.

We have some good content to share:

(maybe worth an update when there are the links to the MS talks after next week?)


'NetworkConnectionType' does not contain a definition for 'ModularKcp'

Description
getting this error
Library\PackageCache\io.improbable.gdk.core@0.3.5\Config\RuntimeConfig.cs(10,81): error CS0117: 'NetworkConnectionType' does not contain a definition for 'ModularKcp'
Environment
I have setup a project from scratch for v0.3.5 following Set up your project

Windows 10
Unity 2019.3.7f1
UnityGDK 0.3.5
WorkerSDK 14.0.2

Hey @alex-Arc thanks for the report. Could I ask you you paste / attach both:

The Unity Editor logfile after attempting to open the project.
The contents of the Packages/manifest.json in your Unity project

This will help us track down your issue.

Hi, I just solved the same problem in my project by doing the following steps:

Update 'spatial' command-line tool.
Fetch the latest commit from the repository (I used the blank project template)
Double checked the packages in the manifest.json file and removed "io.improbable.worker.sdk.mobile": "14.0.2", as it was not needed in my project
Open the Unity Editor, click on SpatialOS -> Generate code(force)

PS: The problem was gone only after the last step.

Closing this issue due to inactivity


No color for Intl global variable

Intl is a standard built-in object in JavaScript and should be colored the same way as Math object for example.
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl

Thanks. I'll add this at the next release.

Unsure when I'll publish this small change. Use the current GitHub master if needed.


not retrieving data from skip folder

Hi all,
I used poretools to extract fastq sequences from fast5 PASS files with no problem. However, I was unable to extract valid sequences from files archived on the SKIP folder. i.e
$ poretools stats pass_read.fast5
total reads	1
total base pairs	1864
mean	1864.00
median	1864
min	1864
max	1864
N25	1864
N50	1864
N75	1864

$ poretools stats skip_read.fast5
WARNING:poretools:No valid sequences observed.

I have plenty of files in the skip folder (I suspect due to computer resources) and I would like to obtain as much information as possible from the sequencing process.
Attached you will find both files.
Any advice will be appreciated
pass_read.zip
skip_read.zip

Damn of me!  ;)
Thank you for your tool!


Why not merge in bootstrap3-datetimepicker-rails ?

To me it just doesn't make sense to have 2 GH projects with the same bundled assets.
Otherwise, with every release there will be update requests like this: https://github.com/TrevorS/bootstrap3-datetimepicker-rails/issues/23

Sorry for the delay, the gem has been updated with 84e78ecf17588f0f1e381cb758e30b2cdd0adc48.

Thanks. I"ll close this.


Make a repo short list on a organization or user page

This can be just like how the issues and pr short list is (with the “view all” button). This will be better because when I first downloaded GitPoint, I didn’t know that the repo number is a button to show the repos (it made me really confused).

I love this feature! I'll send PR for it 😉

Pretty similar to #46


[SPARK-20887][CORE] support alternative keys in ConfigBuilder

What changes were proposed in this pull request?
ConfigBuilder builds ConfigEntry which can only read value with one key, if we wanna change the config name but still keep the old one, it's hard to do.
This PR introduce ConfigBuilder.withAlternative, to support reading config value with alternative keys. And also rename spark.scheduler.listenerbus.eventqueue.size to spark.scheduler.listenerbus.eventqueue.capacity with this feature, according to https://github.com/apache/spark/pull/14269#discussion_r118432313
How was this patch tested?
a new test

cc @JoshRosen @dhruve

Test build #77372 has started for PR 18110 at commit cc51dd0.

FWIW I didn't actually say that we should rename that key since the cost of
the confusing name isn't that high right now. So while I don't oppose this
mechanism I'm neutral on it given that the only use case so far seems kind
of minor. I was mostly commenting just so that future readers and reviewers
can more easily spot the issue and hopefully pick better names going
forward.
On Thu, May 25, 2017 at 8:57 AM Apache Spark QA<EMAIL_ADDRESS>wrote:

Test build #77372 has started
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/77372/testReport
for PR 18110 at commit cc51dd0
https://github.com/apache/spark/commit/cc51dd09611adb838c1ef15aaffe11d90b0b119c
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/spark/pull/18110#issuecomment-304047602, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AADGPMaYT_X0iqsDi6q5OhpOOxZpnKQzks5r9aT0gaJpZM4Nmjjt
.

@JoshRosen actually there are more use cases. Currently we have 2 ways to define a config: create a config entry in the config package object, or SQLConf object, and read conf value with the config entry in code. Or read conf value with hard-coded conf name directly in the code. Ideally we should always create config entry, but we can't because it doesn't support alternative keys.

@cloud-fan, what about SparkConf's configsWithAlternatives : https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/SparkConf.scala#L596

It's only used in the SparkCong.get(key: String) code path, not SparkConf.get(entry: ConfigEntry[T]) code path. That's why we only support alternative keys if users get conf value by hard-coded conf name.

Ahhh, makes sense. Thanks for the clarification.

Test build #77372 has finished for PR 18110 at commit cc51dd0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77372/
Test PASSed.

Test build #77398 has started for PR 18110 at commit 3f9d202.

Test build #77398 has finished for PR 18110 at commit 3f9d202.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77398/
Test FAILed.

Merged build finished. Test FAILed.

restest please

retest please

Test build #77411 has started for PR 18110 at commit a637e95.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77411/
Test FAILed.

retest this please

Test build #77417 has started for PR 18110 at commit a637e95.

Test build #77417 has finished for PR 18110 at commit a637e95.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77417/
Test PASSed.

thanks for the review, merging to master!


Fix missing RelationshipModelForm class, add @class_deprecated_in_favor_of decorator

Closes: #2135
What's Changed

Add @class_deprecated_in_favor_of decorator that can be added to classes so that they report a DeprecationWarning when subclassed

Since in Python DeprecationWarnings are silenced by default (one easy way to make them visible in your dev environment is to set the environment variable PYTHONWARNINGS=default) I've also opted to have this decorator output a standard logging warning message as well.

Re-add RelationshipModelForm as a deprecated alias of RelationshipModelFormMixin (fixes #2135)
Toward improving our consistency and clarity of mixins, go ahead and rename a bunch of our other nautobot.extras.forms mixin classes and add their existing names as deprecated aliases:

Former Name
New Name

AddRemoveTagsForm
TagsBulkEditFormMixin

CustomFieldBulkCreateForm
CustomFieldModelBulkEditFormMixin

CustomFieldBulkEditForm
CustomFieldModelBulkEditFormMixin

CustomFieldFilterForm
CustomFieldModelFilterFormMixin

CustomFieldModelForm
CustomFieldModelFormMixin

RelationshipModelForm
RelationshipModelFormMixin

StatusBulkEditFormMixin
StatusModelBulkEditFormMixin

StatusFilterFormMixin
StatusModelFilterFormMixin

TODO

[x] Explanation of Change(s)
n/a Attached Screenshots, Payload Example
[x] Unit, Integration Tests
[x] Documentation Updates (when adding/changing features)
n/a Example Plugin Updates (when adding/changing features)
n/a Outline Remaining Work, Constraints from Design

Marking as a draft for the moment as I investigate alternate approaches to the class mixin (i.e. decorators).

Any reason we don't rename TenancyFilterForm?

Any reason we don't rename TenancyFilterForm?

I'd love to, but was trying to keep the scope of this PR from growing any further than it already did. :-)

We should update the style guide doc to reference the new classes CustomFieldModelFormMixin and RelationshipModelFormMixin instead of the deprecated classes CustomFieldModelForm and RelationshipModelForm

We should update the style guide doc to reference the new classes CustomFieldModelFormMixin and RelationshipModelFormMixin instead of the deprecated classes CustomFieldModelForm and RelationshipModelForm

I did in #2152 if that is alright.

We should update the style guide doc to reference the new classes CustomFieldModelFormMixin and RelationshipModelFormMixin instead of the deprecated classes CustomFieldModelForm and RelationshipModelForm

Darn it! How'd I miss that? Good catch.


Align output of docker version again

Signed-off-by: John Howard<EMAIL_ADDRESS>@thaJeztah. Fixes https://github.com/docker/docker/issues/28775.
  Looks like there's still some discussion going on... :smile:

@jhowardmsft code looks good, but please fix test.

Oops, need one more space removing. Updated.

LGTM

LGTM


refactor(cartrade): dockerize discounted-catrade

Move duplicated app for getting etherum balance to common location in tools.
Use cmd-socketio-server module imports instead of relative paths in discounted-cartrade.
Add docker-compose setup to discounted-cartrade sample app, update scripts.
Clean-up indy-testnet, leave only network setup in it.
Move nginx proxy and validator setup to discounted-cartrade sample (it's more related to possible bussiness usage than core validator / indy functionality).
Update README documentation for all affectd indy and discounted-cartrade components.

@izuru0 @takeutak @petermetz Please review (only the last commit - refactor(cartrade): dockerize discounted-catrade)


scootapi/client - change CLI to use normal json instead of thrifty-json

Thrift's JSON files are strange and difficult to read and generate correctly as a person. Since test targets was refactored to use the thrift interface directly, the ingestion of thrift-generated JSON files was no longer used.  This update lets the scootapi CLI ingest jobdefs from JSON files that look like the JobDefinition structure.
This is planned to be used by the scoot/iron compatibility binary.

Current coverage is 61.61% (diff: 100%)

No coverage report found for master at 27df90a.

Powered by Codecov. Last update 27df90a...afa55ec


Dynamic Random Question Fetching

Issue No: #16
Changes Implemented:

Added Async Function for fetching Dynamic

Modified ;random cmd =>
;random => Gives Any Random Question
;random  easy=> Gives Any Random Question of Easy Difficulty
;random medium => Gives Any Random Question of Medium Difficulty
;random  hard=> Gives Any Random Question of Hard Difficulty
New Package Installed

axios
npm i axios

Result Video:
https://github.com/user-attachments/assets/e9aa2195-5ad2-4294-8c89-c509a05bc8c1
Please Add gssoc-ext , level labels on this PR

@jinx-vi-0 Please change label Level-3 to level3 , Thanks!!


"Could not generate artifact name for document" if schemaPath is inside src directory

Describe the bug
I customized the schemaPath to be inside of my project's src folder, and houdini tried to generate artifacts for the generated .graphql schema file (thinking it was part of my Svelte project). It threw the following error and caused the build process to fail:
❌ Encountered error in src/schema.graphql
Could not generate artifact name for document [the entire contents of my schema]

Since my project only uses inline fragments in .svelte files, I'm able to mitigate this by modifying include to not check for .graphql files. Maybe Houdini should exclude the schemaPath file automatically while generating the artifacts (or there should be a warning about this in the docs).
Side note: I think printing the file name instead of the entire document contents in error messages would make for a better debugging experience. The schema I was working with was pretty long so I had to scroll a lot to see what was happening 😅
Severity
serious, but I can work around it
Steps to Reproduce the Bug
Change the schemaPath in houdini.config.js to be inside src:
const config = {
   // ...
   schemaPath: './src/schema.graphql',
}

Build the SvelteKit project - throws error.

Thanks for opening this @darrylyeo! I'll get the fix deployed this weekend

This should now be fixed on 0.16.8 👍

Awesome work, thank you!


[GEP-26] Allow quota scope to reference WorkloadIdentity

How to categorize this PR?

/area security ipcei
/kind enhancement
/ipcei workload-identity
What this PR does / why we need it:
This PR allows quota.scope to reference WorkloadIdentitys. It also combines the logic w.r.t Secrets and WorkloadIdentitys references and treats them in the exact same way as "credentials".
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Part of https://github.com/gardener/gardener/issues/9586
cc @vpnachev @rfranzke
Release note:

`Quota`s can now have scope of type `WorkloadIdentity`.

/ipcei workload-identity
/assign

/retitle [GEP-26] Allow quota scope to reference WorkloadIdentity

/label ipcei/workload-identity

/lgtm
/approve

/retest


Prepare for v0.16.18 release

filed https://github.com/cilium/cilium-cli/issues/2806 for gke failure. i'm shipping this.


The window has crashed (reason: 'oom', code: '-536870904')

I know this issue was closed in the past, but I was recently affected by it. For some days Visual Studio Code crashes showing a notification window with the message: The window has crashed (reason: 'oom', code: '-536870904') and the program closes. This happens very frequently. Has anyone suffered this problem? Is there a solution?

Does the issue happen when you run with --disable-extensions ? Can you attach the crash logs following the steps at https://github.com/microsoft/vscode/wiki/Native-Crash-Issues#creating-and-symbolicating-local-crash-reports

Does the issue happen when you run with --disable-extensions ? Can you attach the crash logs following the steps at https://github.com/microsoft/vscode/wiki/Native-Crash-Issues#creating-and-symbolicating-local-crash-reports

It does not matter if you run with this option or not.

Does the issue happen when you run with --disable-extensions ? Can you attach the crash logs following the steps at https://github.com/microsoft/vscode/wiki/Native-Crash-Issues#creating-and-symbolicating-local-crash-reports

I have not ckecked it with these flags. Anyway I send the dmp file created after the crash:

Sorry. How can I send you the dmp file?

I have the same issue [Visual Studio Code crashed with message: "The window has crashed (reason: 'oom', code: '-536870904')"] on Windows 7 SP1.
This happens frequently on my system (several times per day).
Have attached several dump files:
reports.zip
1
About Visual Studio:
Version: 1.67.1 (user setup)
Commit: da15b6fd3ef856477bf6f4fb29ba1b7af717770d
Date: 2022-05-06T12:37:03.389Z
Electron: 17.4.1
Chromium: 98.0.4758.141
Node.js: 16.13.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 6.1.760

Hey @deepak1556, this issue might need further attention.
@mariza-dev, you can help us out by closing this issue if the problem no longer exists, or adding more information.

It seems the problem was in the computer. Finally it crashed. But until that time the problem persisted.


[BUG] <link gives 404 page not found >

Description
a link destination is not working ,the page is not found in Cybersecurity/DefenciveSecurity
Screenshots
clicking the second link

Any additional information?
No response
What browser are you seeing the problem on?
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)

Hi @Mouh1b
Please find the correct link and let me know if it's available.
I will assign the PR then.

This is the resource we're talking about.

Hey @Anmol-Baranwal I've found the correct link and it is available

Hey @Anmol-Baranwal I've found the correct link and it is available

do you still want to work on this issue?

yes, assign it to me when you can

@Mouh1b
I have assigned you the issue. Please proceed with it, and don't hesitate to tag any of the maintainers if you require any assistance.


Fix typo in IntelliSense for listItemIndent

There was an extra space at the end of the definition for listItemIndent, which means if you used IntelliSense to populate that setting, it would seem like it does not work.
Fixes #15

Thanks for the contributing :tada:


Handling rate limits in hit_ercot_api.

while downloading 30 days of data using the hit_ercot_api() method, I got rate-limited with the following exception:
requests.exceptions.HTTPError: 429 Client Error: Too Many Requests for url: https://api.ercot.com/api/public-reports/np4-183-cd/dam_hourly_lmp?size=100000&deliveryDateFrom=2024-06-20&deliveryDateTo=2024-06-20&page=1
To reproduce, call this following code in a loop, one day at a time for 30 days.  It would raise the error about 7 days in.
import gridstatus.ercot_api.ercot_api as ercot_api

def dam_hourly_lmp_to_df(begin: datetime, end: datetime) -> pd.DataFrame:
    """
    Get the DAM hourly LMP data from the ERCOT API and return it as a DataFrame.

    Parameters:
    begin (datetime): The begin date of the date range for the data.
    end (datetime): Inclusive, the end date of the date range for the data.
    """

    ercotApi = ercot_api.ErcotAPI()
    dam_hourly_lmp_endpoint = "/np4-183-cd/dam_hourly_lmp"
    api_params = {
        "deliveryDateFrom": begin.strftime("%Y-%m-%d"),
        "deliveryDateTo": end.strftime("%Y-%m-%d"),
    }
    return ercotApi.hit_ercot_api(
        endpoint=dam_hourly_lmp_endpoint, verbose=True, **api_params
    )

The error actually happened on this line because make_api_call() didn't handle status 429 properly.
        # The data comes back as a list of lists. We get the columns to
        # create a dataframe after we have all the data
        columns = [f["name"] for f in response["fields"]]

I added the following retries logic to the make_api_call() and was able to download 30 days of data.
    def make_api_call(
        self,
        url,
        method="GET",
        api_params=None,
        data=None,
        parse_json=True,
        verbose=False,
    ):
        log(f"Requesting url: {url} with params: {api_params}", verbose)
        retries = 0
        delay = 3.0
        if method == "GET":
            while retries < 3:
                response = requests.get(url, params=api_params, headers=self.headers())
                retries += 1
                if response.ok:
                    break
                if response.status_code == 429 and retries < 3:
                    print(f"Rate limited, waiting {delay} seconds on retry {retries}")
                    time.sleep(delay)
                    delay = delay * 2
                else:
                    response.raise_for_status()

ErcotAPI class seemed to have implemented retries with gradually increasing delays.  But the retry function needs to be inside the make_api_call().  It would be nice if retry can be done without tying up a thread by time.sleep.

Thanks for the report! You're right that the current retries logic is inadequate. Do you want to open a PR to address the issue?

I'd give it a try.  can you assign the issue to me?  full disclosure - I am new to python.  you'd need to review my changes thoroughly.

@buckfeng Can we close this now that #417 has been merged? Thank you for your contribution!


Revert "style(config): Revert style change"

This reverts commit 59e9816cf3a82fc8b0d821456fb6da31fe0cfde4.

:tada: This PR is included in version 4.9.11 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Bind SkPixelSerializer

SkPixelSerializer can be used to custom-encode images:
https://github.com/google/skia/blob/master/include/core/SkPixelSerializer.h
https://fiddle.skia.org/c/@SkImage_to_PPM_binary
https://fiddle.skia.org/c/@SkImage_to_PPM_ascii

This has been added to the update-m60 branch already, and will be merged soon.


BUG :Icon Overlap Issue

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
We have identified a bug where the Translate and Chat icons are overlapping on the website. This overlap not only affects the visual layout but also hinders user experience, making it difficult for users to interact with these features effectively.

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Hello! I’m excited to work on this issue as a GSSOC'24 contributor. Could you please assign it to me?


Integrate Embedding Models Using Vertex AI with Service Account Authentication

Description
Due to security policies, embedding API keys are not permitted in most companies. We need to use the Vertex AI service account (gemini_credentials.json) for authentication when integrating embedding models. This approach ensures compliance with security standards and leverages the benefits of using service accounts for authentication.  This also simplifyes the config process becasue all that is requred for people using GCP ( most big companies currently ) is the service account / gemini_credentials.json.
Is this a bug or a feature?

[ ] Bug
[x] Feature

Steps to Reproduce

Setup Vertex AI Service Account:

Ensure gemini_credentials.json is correctly configured and accessible.
Verify that the service account has the necessary permissions for accessing embedding models.

Modify Authentication Method:

Update the codebase to use gemini_credentials.json for authentication instead of embedding API keys.
Implement the necessary changes in the configuration files to support service account authentication.

Testing and Validation:

Conduct thorough testing to ensure that the embedding models function correctly with the service account authentication.
Validate that all functionalities are working as expected and that there are no security issues.

Additional context
This change is necessary to comply with company security policies, which prohibit the use of GCP API keys. By using a service account (gemini_credentials.json), we ensure that our authentication method is secure and aligns with industry best practices.

Hey if someone can point me in the right direction I can probably get a pull request together.


Make Virtual ATM

I need some bills...
Build a virtual ATM simulator where users can deposit, withdraw, and check their account balance in a simple CLI-based interface.

Assign me please , I can do that .


Mousewheel zooming doesn't respond most of the time

Zooming works great with a trackpad, but zooming often doesn't occur when scrolling with the mouse.
It appears to be the trackpad two-finger panning logic that's interfering. Commenting out the conditional so only onZoom is called gives correct behavior with the mousewheel, but then two-finger panning no longer works on a trackpad.
https://github.com/rrag/react-stockcharts/blob/master/src/lib/EventCapture.js#L115-L149

2 finger swipe to pan is a good feature, I dont see that going away


Keep GCThrashingPercentagePerPeriod in DataflowPipelineDebugOptions

https://github.com/apache/beam/pull/27850 will cause problems until new containers are built and used.
ERROR 2023-08-07T16:47:48.011Z Uncaught exception in main thread. Exiting with status code 1.
ERROR 2023-08-07T16:47:48.011Z java.lang.NoSuchMethodError: 'java.lang.Double org.apache.beam.runners.dataflow.options.DataflowPipelineDebugOptions.getGCThrashingPercentagePerPeriod()'
ERROR 2023-08-07T16:47:48.012Z at org.apache.beam.runners.dataflow.worker.util.MemoryMonitor.fromOptions(MemoryMonitor.java:243)
ERROR 2023-08-07T16:47:48.012Z at org.apache.beam.runners.dataflow.worker.StreamingDataflowWorker.<init>(StreamingDataflowWorker.java:588)
ERROR 2023-08-07T16:47:48.012Z at org.apache.beam.runners.dataflow.worker.StreamingDataflowWorker.fromDataflowWorkerHarnessOptions(StreamingDataflowWorker.java:556)
ERROR 2023-08-07T16:47:48.012Z at org.apache.beam.runners.dataflow.worker.StreamingDataflowWorker.main(StreamingDataflowWorker.java:273)

Keeping it for now to avoid blocking people using Beam @ HEAD.

The line that will cause problem is here:
https://github.com/bvolpato/beam/blob/9e3d8d3c5719bbe8052b10c19be028d6b93f3eaf/runners/google-cloud-dataflow-java/worker/src/main/java/org/apache/beam/runners/dataflow/worker/util/MemoryMonitor.java#L245
this change alone won't be effective to eliminate the issue when using an outdated workerJar
to me this is a non-issue / documentation issue. We need to bump https://github.com/apache/beam/blob/3b112e8bd8da50d934955d64f2fd1e1396d3fe01/runners/google-cloud-dataflow-java/build.gradle#L56 here

True, good point @Abacn. I'm building a new container. Provided workaround steps for folks having friction -- closing this for now.


Have you come up with framework name yourself ?

I don't know what you want to express. This is just a personal toy project, and there is no progress at the moment.

I've only been doing it for a month and it's not done yet. If you've been investing in this for more than a year, I'm willing to give that name to you. How do I do this?

Unfortunately it is not possible to unpublish package :(. However you can transfer ownership to me through pub.dev admin console by adding me as Admin and removing yourself.

When you open source your summer project, you can contact me. Then I will hand over the ownership to you.

Deal.


No Module named "transition_amr_parser"

Hi,
I was following the instructions on https://github.com/IBM/transition-amr-parser/issues/35, where I cloned and locally installed the repository. My machine outputs "Successfully installed transition_neural_parser-0.5.4", and I cannot import either "transition_amr_parser" or "transition_neural_parser" in my code. Any idea why this could be happening?
Thanks!

To add a follow-up, I initially tried following README and directly did 'pip install transition_neural_parser' and it still won't let me import.


Default data.body.value to {}

In certain situations, data.body.value is undefined when the reporter
runs. Defaulting to {} and allowing to fall through to either
user-defined metadata or NOT_KNOWN.

Yep totally agree. I added this patch for my team and figured I'd put up a PR just in case. Thanks for looking into it.

Hi @dkpalmer
Looking at your logs again, I think this will only happen when you're testing an app, or not?

This is fixed in https://github.com/wswebcreation/wdio-multiple-cucumber-html-reporter/releases/tag/0.4.0
The problem was in the fact that apps were not supported. They are now


Fix internet-up.ably-realtime.com checks

Run them on each fallback host change, and don't ignore the result.
Fixes #952.
Commit acdb52c supersedes #959 and fixes #931.

CI is failing (sometimes):
[22:54:51]: ▸     ✗ Connection__Host_Fallback__applies_when_the_default_realtime_ably_io_endpoint_is_being_used, expected to have Array<URL> with count 2, got 1
[22:54:51]: ▸     ✗ Connection__Host_Fallback__applies_when_an_array_of_ClientOptions_fallbackHosts_is_provided, expected to be true, got <false>
[22:54:51]: ▸     ✗ Connection__Host_Fallback__every_connection_is_first_attempted_to_the_primary_host_realtime_ably_io, Waited more than 10.0 seconds
[22:54:51]: ▸     ✗ Connection__Host_Fallback__every_connection_is_first_attempted_to_the_primary_host_realtime_ably_io, expected to have Array<URL> with count 3, got 2
[22:54:51]: ▸     ✗ Connection__Host_Fallback__every_connection_is_first_attempted_to_the_primary_host_realtime_ably_io, expected to be true, got <false>
[22:54:51]: ▸     ✗ Connection__Host_Fallback__every_connection_is_first_attempted_to_the_primary_host_realtime_ably_io, Test crashed
[22:54:51]: ▸     ✗ Connection__Host_Fallback__should_retry_hosts_in_random_order_after_checkin_if_an_internet_connection_is_available, expected to equal <[d.ably-realtime.com, e.ably-realtime.com, a.ably-realtime.com, c.ably-realtime.com, b.ably-realtime.com]>, got <[d.ably-realtime.com, e.ably-realtime.com, a.ably-realtime.com, c.ably-realtime.com]>

[22:37:02]: ▸     ✗ Connection__Host_Fallback__should_retry_hosts_in_random_order_after_checkin_if_an_internet_connection_is_available, expected to be true, got <false>


Remove the model argument from Lite's optimizer_step via structural typing

What does this PR do?
See title
Does your PR introduce any breaking changes? If yes, please list them.
None
cc @justusschock @awaelchli @rohitgr7 @carmocca

@carmocca I took care of updating the tests

@carmocca I took care of updating the tests

Thanks! I moved the Steppable assertion into the test so we don't need to mock the engine during the "check step called" tests


Fix date range selector does not show data of last day when user timezone is not UTC

Related to #594 and #617.
In following conditions, re:dash does not show the data of last day.

some timestamp value group by day
user in non UTC timezone

Like this:
SELECT
  to_char(created_at, 'YYYY-MM-DD') as day,
  COUNT(*) AS cnt
FROM
  queries
GROUP BY
  day
ORDER BY
  day

I think this is caused dateRange is treated as local timezone, but point.x is treated as UTC.
Following screenshot is a result of console.log(dateRange, point.x); before https://github.com/EverythingMe/redash/blob/master/rd_ui/app/scripts/services/resources.js#L197

This pull request fix this issue, but I'm not sure that this change is OK for timezone handling in re:dash.
@arikfr What do you think?

Are you sure it's due to time zone differences and not because of the time value of the max value? It looks like your fix actually does that: moves the the hour to be of end of day.
What you did looks ok to me, but I will make sure that we indeed create all time values in the same time zone as you mentioned.
Thanks.

@arikfr Thanks for review. I realized that this is not a issue of the time value of the max value, it's only related timezone difference issue. So I change my code to treat dateRange as UTC, it is as same as point.x.
Is this OK for you?

Merged. Thanks!


generate web app manifest

https://developer.mozilla.org/de/docs/Apps/Manifest

apps only

won't fix ... to far off the default