Feature request: Configuration-driven proxy filtering

In most of the scenarios where I've used YARP, I've found it useful to copy in a class I borrowed from another sample created during the Project Tye era that would read in values from IConfiguration to be pumped into the route and cluster endpoint values. In an example by Shayne Boyer, which I've mimicked in the CustomConfigFilter class I created for the Build 2023 cloud native .NET demo code, a configuration-driven filter would give developers a conventional way of mapping environment variables to placeholders in their YARP configuration.
Take the appsettings.json code below, from the same Build 2023 demo repo:
{
  "Logging": {
    "LogLevel": {
      "Default": "Information",
      "Microsoft.AspNetCore": "Warning",
      "System.Net.Http.HttpClient": "Warning",
      "Yarp": "Warning"
    }
  },
  "ORDERS_API": "http://localhost:5004",
  "STORE_UI": "http://localhost:5176",
  "ReverseProxy": {
    "Clusters": {
      "Frontend": {
        "Destinations": {
          "Store": {
            "Address": "{{STORE_UI}}"
          }
        }
      },
      "OrdersApi": {
        "Destinations": {
          "Store": {
            "Address": "{{ORDERS_API}}"
          }
        }
      }
    },
    "Routes": {
      "ClientRoute": {
        "ClusterId": "Frontend",
        "Match": {
          "Path": "{**catch-all}"
        }
      },
      "OrdersApiRoute": {
        "ClusterId": "OrdersApi",
        "Match": {
          "Path": "/api/orders/{**catch-all}"
        },
        "Transforms": [
          { "PathPattern": "/api/orders/{**remainder}" },
          { "PathRemovePrefix": "/api" }
        ]
      }
    }
  }
}

In this code, the ORDERS_API and STORE_UI values are hard-coded in the file as settings, so they'll be read in from here by default or from environment variables. The placeholders {{ORDERS_API}} and {{STORE_UI}} are replaced with the values in these settings or environment variables at runtime, making it easy for developers to use environment variables to route traffic between services. It also gives developers who want to integrate things like Tye with YARP to create new service discovery approaches.
I'll follow this issue up with a pull request to add the filter to the product's code.

It seems like you're doing 1:1 mapping of route patterns to destinations without any of the other features like health checks, load balancing, etc. In that case the new MapForwarder API might be more appropriate.
app.MapForwarder("{**catch-all}", configuration["STORE_UI"]);
app.MapForwarder("/api/orders/{**catch-all}", configuration["ORDERS_API"]);

{
  "Logging": {
    "LogLevel": {
      "Default": "Information",
      "Microsoft.AspNetCore": "Warning",
      "System.Net.Http.HttpClient": "Warning",
      "Yarp": "Warning"
    }
  },
  "ORDERS_API": "http://localhost:5004",
  "STORE_UI": "http://localhost:5176"
}

Are you actually using the transforms in that example? They seem misconfigured.

This feels like it should be more part of Aspire and a bigger question of how to do API gateways in Aspire. TestShop and eshop both have instances of YARP that use the service discovery Aspire component.
This is probably better addressed with #2525 and having a Aspire apphost component for YARP.
Deploying a YARP to ACA to map URLs seems a bit redundant, but c'est la vie

I have a use for the docker image approach as well, but agree it is a bit redundant. I created an API gateway container component for Aspire but because service discovery awareness is an http client feature there is no good way to send traffic to multiple instances of an API (dynamically hardcoding is doable for each host but not ideal). YARP as a sidecar would enable the gateway to use service discovery indirectly.
The other choice as I see it is emitting all the host values for each replica as an env var like services__apiservice__http__ instead of just the 0 indexed value as seems to be the case right now.


Error when forcing someone to finish a quest

I get this error in the console when I do /questsadmin finish   or when I do /questsadmin nextstage  - http://pastebin.com/442c9xe7

This seems to happen on a few quests not all of them.


feat(import): import @tier4/cspell-dicts/cmake/cspell-ext.json

Test result: Only tmp.txt, which is not a CMake file, has errors.


build: Remove unnecessary SbtPlugin settings

What changes are proposed in this pull request?
To remove the enablePlugins(SbtPlugin) in build.sbt for modules that are not themselves SBT plugins.
This is unnecessary and prevents cross-building to Scala 2.13, which is what I'm actually trying to get at.
How is this patch tested?
By running sbt test:compile and checking that compilation still works (it wouldn't if one of the modules was actually an SbtPlugin).
Does this PR change any dependencies?

[x] No. You can skip this section.

Does this PR add a new feature? If so, have you added samples on website?

[x] No. You can skip this section.

/azp run

Codecov Report

Merging #1771 (dec7901) into master (7ab63a1) will decrease coverage by 80.92%.
The diff coverage is n/a.

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

/azp run


Compliling issues

Describe your problem here.
everytime i fix one thing a new problem arises and no body else can help!!

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
fixed teacall to tea

SORRY NVM FIGURED IT OUT

When you figure out your issue, please close your thread.


Settings page 404s

Tapping the gear in the Pebble app brings up a "404 - there isn't a github pages site here" error page. Unfortunately I don't know which github page it's trying to load because it doesn't have the usual browser widgets.

it tries to load this page http://freakified.github.io/TimeStylePebble/ see https://github.com/freakified/TimeStylePebble/blob/master/src/pkjs/index.js you can find the base config URL there

Any plan to fix this, or is there a way to load the page alternatively? I still use this watchface on a daily basis.

As a workaround solution there is my forked version TimeStyleBBPebble for which the setting page is still working : https://plarus.github.io/TimeStyleBBPebble/config_color.html?appversion=11

Weird. There have been no commits to gh-pages since Feb 2017. This seems like an easy fix, @freakified just needs to check the GitHub Pages settings.

@theLMGN I couldn't find anything wrong in the settings, but I just made a 1-line meaningless change to force a gh-pages rebuild, and it appears that this has fixed it -- http://freakified.github.io/TimeStylePebble/config_color.html now works.
That said, these days I mostly just recommend @plarus's fork (or one of the other forks that I think exists).

I tried @plarus's fork, but the "send to watch" button doesn't do anything, it just remains stuck at the default settings forever. So, thanks for the quick fix!


ENH: use model equations for default calibration

This one addresses issue #85 using method 2.2.
@spencerlyon2 : can you check that this works for you ?

Works great for me, thanks.


Diff or Patch of Comparing

Diff or Patch of Comparing
https://github.com/rails/rails/compare/master...4-1-stable.diff
https://github.com/rails/rails/compare/master...4-1-stable.patch
:neckbeard:

What are we doing in the middle of the desert?


Allow viewing github markdown in development

This will allow us to use github flavored markdown and view it during
development.
Just run yard server -r and navigate to localhost:8808 and the
readme should now be in github flavored markdown.

It's specified as development dependency so it won't be installed as part of gem install blueprinter


[Bug]: ZZZ Cache Updates is checking for broken files

Affected Version
1.82.3-Pre
Bug Behavior
Not sure if this was already reported but instead of checking for cache updates, I think it's checking for broken files like how the game repair functions.
Expected Behavior
Should check for cache updates instead of broken files.
Steps to reproduce

Reinstalled

Related Issues
No response
Screenshot(s)

Additional Information
No response

Since ZZZ Cache checking function uses inherited functions from the Repair counterpart and I think we forgor to put different text on cache, but the function should be different in the backend
TL;DR its a UI/UX issue rather than wrong check
We'll try to fix the text assignment

Ohh. I see. Thank you~

Issue has been resolved in latest preview and stable releases. Thanks for your report!


Move the CHIL engine to demos/engines

Moving out of the way, Makefile to be added

Not quite the code health intention of the day, but it does move it out of the way.

Merged. 44e2d45ffdcda4160d7d36a5b0ee702d0ebad8ac


Decide on project branching method

It is generally a best practice not to commit to master branch unless the code is stable.

[ ] Decide on a branching practice for concurrent development.
[ ] Create basic branch(es) to kick off project development
[ ] Add chosen branching strategy to project development documentation

I believe Git Flow is becoming somewhat popular, and propose we adopt Git Flow branching.


pyk4a.depth_image_to_point_cloud Fails

Describe the bug
Calling pyk4a.depth_image_to_point_cloud kills the python kernel.
To Reproduce
Run the following code
import os
os.environ["K4A_DLL_DIR"] = r"C:\Users\jbrownkramer\projects\OrbbecSDK-K4A-Wrapper\build\bin"
import pyk4a

config = pyk4a.Config(depth_mode = pyk4a.DepthMode.WFOV_2X2BINNED, camera_fps = pyk4a.FPS.FPS_15)
k4a = pyk4a.PyK4A(config)
k4a.start()

capture = k4a.get_capture()
pyk4a.depth_image_to_point_cloud(capture.depth,k4a.calibration,thread_safe=False)

Expected behavior
I expect it to produce a pointcloud.  However, instead it takes down the python kernel.
Screenshots

Desktop (please complete the following information):

OS with Version: Windows 11
SDK Version:  ob/1.8.x

Are you using a self-packaged python SDK or an open source one? You can use the azure kinect python version provided by us.
https://github.com/orbbec/pyKinectAzure

Are you using a self-packaged python SDK or an open source one? You can use the azure kinect python version provided by us.
https://github.com/orbbec/pyKinectAzure

I am using pyk4a.  I was not aware of pyKinectAzure, so that's interesting!


[Gally]: master <- dev

Automatically created by Git-Ally

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update sbt-ci-release to 1.7.0

About this PR
📦 Updates com.github.sbt:sbt-ci-release from 1.5.11 to 1.7.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.sbt", artifactId = "sbt-ci-release" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.github.sbt", artifactId = "sbt-ci-release" }
}]

labels: sbt-plugin-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #186.


generateChangeLog not working

Hello everyone,
I'm making first steps in liquibase plugin for neo4j and I'm struggling with generating the initial changeLog file for the existing neo4j database.
I run the following command:
liquibase --changeLogFile=mydatabase_changelog.xml generateChangeLog
The error I get is:
Unexpected error running Liquibase: Don't know how to query for sequences on neo4j @ jdbc:neo4j:neo4j+s://_databaseaddress_
At the same time, I am able to connect to the database and run a simple changelog which creates a node as per quick start guide, so my guess it is not a connectivity issue.
Can anyone adivse, please, if generateChangeLog functionality working with neo4j?
I am using liquibase-neo4j-<IP_ADDRESS> plugin version and neo4j-jdbc-driver-4.0.6 for connection

Hello, thanks for the report, I'll need to estimate how wide the gap is to support that functionality.
I'll let you know ASAP.

Hello, @mgazanayi and I had a look and it appears generateChangeLog will not be easy to support due to the nature of Neo4j.
TL;DR: the best we can do is generate the index/constraint changes to match those detected in the database. That needs to happen after we implement the createIndex/createConstraint changes in the plugin.
Long version ⬇
Relational databases have a strict separation of structure and data.
You must first define structure (table, columns) before inserting data into it. These structures can be reliably retrieved and that's what generateChangeLog does: get the existing structures and generate the corresponding Liquibase change for it.
With Neo4j, there's no strict separation between structure and data. They are tied together.
You create the data directly (with the exception of indices and constraints).
I don't think it would be reasonable implementation to dump the whole data into a Liquibase change set as a result of generateChangeLog. The data set could be huge, leading to memory issues and probably for no added value (unless each node/relationship has its own schema - which is not true, most of the time).
If we exclude data, the only artefacts we have left are indices & constraints.
We could inspect the target Neo4j database and generate the Liquibase changes that would in turn create indices & constraints when run.
Currently, this is done with raw Cypher queries. @mgazanayi and I just started looking into implementing higher-level constructs that support indices/constraints creation & drop in a Neo4j version-agnostic manner. We want to focus on that first.

Hello @fbiville!
Thank you very much for looking into my question, and for such a detailed explanation.
At least I am now sure that this is not implemented, and not me doing something wrong.
I guess I'll just repeat the data modeling steps on a new Neo4j instance from scratch.
Cheers!

If you can share of your manual work here, that would be super insightful for us!
We would love to learn more about the use cases of generateChangelog for Neo4j.


test swift jobs with the queuedAt field intro

TODO Close after upstream merged

Can we add the new tests in swift-jobs here, so we can verify everything works with redis

Will do!

Can we add the new tests in swift-jobs here, so we can verify everything works with redis

There's already a JobTest for Redis here Unlike the Postgres implementation, the queuedAt is not being persisted.
Does it make sense to stream the JobData struct introduce in the Postgres PR and save it as the buffer here?

Can we add the new tests in swift-jobs here, so we can verify everything works with redis

There's already a JobTest for Redis here Unlike the Postgres implementation, the queuedAt is not being persisted.
Does it make sense to stream the JobData struct introduce in the Postgres PR and save it as the buffer here?

Oh god this is quite awkward. Not sure how you should save the queuedAt time. Currently the redis implementation provides the equivalent of a set of key, value pairs. Where the key is the jobID and the value is the ByteBuffer of encoded data. Augmenting that with an additional value is going to be quite awkward or suboptimal

One solution is to do this at the swift-jobs level and include it in the struct we encode. Then the Postgres and redis implementations don't need to know about it at all.

One solution is to do this at the swift-jobs level and include it in the struct we encode. Then the Postgres and redis implementations don't need to know about it at all.
EDIT: I meant to also add. I've looked at how we could do this, but it is a little messy and I want to clean up the surrounding code first.

Makes sense. Have you any idea when the clean up process will be ready?

Makes sense. Have you any idea when the clean up process will be ready?

I'll try to get something for tomorrow. Basically the issue is the encoding and decoding of jobs are done from different types and the types are spread across multiple files. A job to encode only needs the parameters and job id, a decoded job needs the parameters and job definition so it can run the job.


Not able send base 64 image in adaptive dialog

Github issues should be used for bugs and feature requests. Use Stack Overflow for general "how-to" questions.
Version
4.5* latest Preview version
Describe the bug
Give a clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:
want to send a base64 image in Adaptive Dialog.
Tried using  AdaptiveCards.AdaptiveImage but it says Url is to bix in exception.
Expected behavior
Give a clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
[bug]

@sgellock  thanks let me know if there is any work around.

@vidya091209 can you please clarify how you need to/ trying to send base64 image? By in lining the image in a card attachment or do you need to send this directly via the SendActivity step or something else?

var img = new AdaptiveCards.AdaptiveImage(i.message)
{
Type = i.message.Split(";")[0].Substring(5)
};
dialog.Steps.Add(img);
I am trying this, i think we  can not use send activity for image. i am open to any idea.

@v-kydela, please investigate.

@vidya091209 - In order to determine if this is related to adaptive dialogs, can you successfully send the base64 image outside of an adaptive dialog?

@v-kydela  yes i am able to send it outside the adaptive dialog.

@vidya091209 - Adaptive Cards are unrelated to adaptive dialogs, so naturally an adaptive image is not a valid adaptive dialog step. You can see all the types of steps available to you in the steps folder. What specific branch have you pulled in order to use adaptive dialogs?

@v-kydela I simply wanted to send a base64 image as step in adaptive dialog. i am using 4.Future branch.

@v-kydela I'm moving this over to the Adaptive team to address.  Thanks for the initial help, but I'd like them to address the issue.
@tomlm @carlosscastro we should decide if we can get this fixed before the next train leaves.

You need to use a **SendActivity(activity) step.  That takes a Activity object which can have attachments.
If all you are trying to do is send an image as an activity then it would be something like this:
var activity = dc.TurnContext.Activity.CreateReply();
activity.Attachments.Add(new Attachment(contentType:"image/jpg", contentUrl:"base64:..."));
// this uses new CTOR that takes a raw activity, you need latest bits for this
dlg.Steps.Add(new SendActivity(activity));

// alternatively, you use a staticActivityTemplate()
dlg.Steps.Add(new SendActivity() {  Activity = new StaticActivityTemplate(activity)});

If you are truly trying to send an AdaptiveCard, then you need to do the same thing but add an attachment with adaptive card that has an AdaptiveImage inside that.

Thanks a lot it worked.


WA reports a problem with OBX-23.10

Describe the bug
WA DOH reports a general problem with how ReportStream sends an HL7 message. The OBX-23 Ordering facility fields is not correct. It has an extra CLIA component.
Impact
We are passing WA QA. We may not pass other qualifications in the future.
To Reproduce
Steps to reproduce the behavior:

Look at HL7 message generated by ReportStream

Expected behavior
OBX-23.10 doesn't have the extra field.
Screenshots
If applicable, add screenshots to help explain your problem.
Logs
Additional context
Message from WA
Extra component

What we received:
CLIA&2.16.840.1.1138<IP_ADDRESS>&ISO^XX^^^38D0805506^CLIA

Example: CLIA&2.16.840.1.1138<IP_ADDRESS>&ISO^XX^^^38D0805506

This bug is actually reaching further than just the OBX-23. It affects the specimen-id, filler-id, placer-id and patient-id.

What's the status fo this?

In production.


C++: Fix flow out of this by reference

This PR adds a type of post-update node that allows data flow out of a this pointer passed by reference. This was missing because there is no VariableAccess associated with it, and so it didn't fit into DefinitionByReferenceNode.
There's a false negative in one of the tests I'm adding. This will be fixed by #1884.
I tested performance on MySQL, and it was fine.

Yes, there are still gaps in the coverage of this library. But this gap is actually the last one on my list of language-level problems I want to address.


Add Joi support to mongoose schema

Proper validation is always important.
Joi support needs to be added in the mongoose schema to support various validation.
Reference:
https://hapi.dev/module/joi/

I am working on this.


Feature: Disable URL on table column

Sometimes it could be usefull if column doesn`t link to anything, eg: if you want to copy it or attach own action:
TextColumn::make('is_paid')
    ->disableUrl()
    ->extraAttributes(function (Invoice $record) {
        return [
            'wire:click' => 'mountTableAction("settle", ' . $record->getKey() . ')',
            'class' => 'cursor-pointer',
        ];
    })

This change allows to do it 2 ways by: url(false) or disableUrl(). However only the second method is documented as I think it is more transparent.

@studiowizjo Pretty sure calling ->action() or url() on a table will already do this, no?

Also, you've deleted a Blade file?

@ryangjchandler I think using action() it is not possible to call one of defined table actions. At least I couldn`t figure out how to do this.
The second use case is, as I mentioned, to allow easy selecting and copying data from table without accidentaly clicking link.

Also, you've deleted a Blade file?

It shouldnt be part of commit, it is my oversight. I want to withdraw it, but this is probably not how it should be done.


[Bug]: Misleading error message when binding async function in a widget data

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When a user tries to bind an async function, a misleading error is thrown

Instead, users should be warned that async functions can not be binded/used in the data property
Steps To Reproduce

Create an async function in a JS Object
Bind it in a Table data property
You will see a misleading/incorrect error message

Environment
Production
Version
Cloud

will track this in #12179


Adds loading skeleton

Adds a loading skeleton for MyList and Search pages. Fixes #67
Branched from issue-94 so that should be merged first.

Sweet.


Connection pooling via r2d2 crate

The project could get a boost in performance by utilizing connection pools with r2d2 and Postgres. This is already flagged as a feature within the cargo.toml for diesel.
This is something most likely I could support with a PR, but wanted to have a discussion for desire before implementing.

Looking further, looks like r2d2 is being used. Mistake :).

@espnicholas :) Feel free to take some open issues and PR


Add an Osize option

• Explanation: These changes add an Osize option and changes to the optimizer guarded by this option: the inliner is tuned down, speculative devirtualization, function signature specialization and array property specialization is turned off. LLVM’s OptimizeForSize attribute is added to functions and LLVM’s noinline attribute is attached to metadata and witness accessor.
• Scope of Issue: Introduction of a new feature
• Reviewed By: Erik Eckstein
• Testing: tested by the test suite
rdar://34194331

@swift-ci Please test


auto build prombench apps from source or push to a central repo on every change.

at the moment the fake webserver and the python loader are manually build and pushed to a central repo.
It would be better if prow builds these from source or if we use circle to build and auto push these to quay or docker hub.

fake-webserver and python-loadgen are fairly stabilized and need very minimal changes.
I don't see much reason to automate them.

you are right it these start to need frequent changes will revisit.
after the duscussion today it seems that we would have

prombench docker image - to create/delete clusters/nodes pod deployments.
prometheus runner - which will build and run a prometheus from a mounted src folder.
fake-webserver
python-loadgen


Telemetry is being sent even whilst opted-out + includes personal identifiers

Today I noticed requests were being made regularly to https://api.insomnia.rest/notification. These turned out to be POST requests, and they are sending precisely the telemetry that I have explicitly opted out of sharing.
Contrary to the config screen claim that only anonymous data will be sent if one should opt in, several datapoints are included that both the CCPA and GDPR list as personal identifiable information:  deviceId, firstLaunch, IP address, OS version. It is impossible for a property that is referred to by the word "identifier" (deviceId) to be anonymous to any degree, especially when it is combined with other data.
Additionally, Google Analytics appears to happily be collecting events and who knows what else (all the things). You can never legally process this data anyway as it was collected during a period without consent - which, even if it were to be given at a later date, would not apply retroactively. Please consider disabling data collecting completely when the user has not opted in. Close the holes.
--

The configuration setting that I have disabled says:

Data Sharing
Help Kong improve its products by sending anonymous data about features and plugins used, hardware and software configuration, statistics on number of requests, collections, documents, etc.
Please note that this will not include personal data or any sensitive information, such as request data, names, etc.

However the following data is being transmitted:
Common HTTP request data
{
  "firstLaunch":[redacted],
  "launches":[redacted',
  "platform":"darwin",
  "app":"com.insomnia.app",
  "version":"2021.2.2",
  "requests":[redacted],
  "requestGroups":[redacted],
  "environments":[redacted],
  "workspaces":[redacted],
  "updatesNotSupported":false,
  "autoUpdatesDisabled":false,
  "disableUpdateNotification":false,
  "updateChannel":"stable",
  "deviceId":[redacted],
  "createdRequests":[redacted],
  "deletedRequests":[redacted],
  "executedRequests":[redacted[
}

The fact that this data is being sent to such an inconspicuous URL, one containing the word "notifications" no less, left a very bad taste in my mouth.
However this came about, it is a serious breach of trust and confidence. Today it's telemetry, tomorrow it might be sensitive work data.
I kindly request for this to be fixed as soon as possible.

Hi @pcjmfranken -- Thanks for reporting this. I agree with many of your points and one in particular interests me (that GA is collecting data still). We will take a look into this, as this is unintended.

I'll
khyveasna
On Fri, 2 Apr 2021, 02:44 Eric Reynolds, @.***> wrote:

Hi @pcjmfranken https://github.com/pcjmfranken -- Thanks for reporting
this. I agree with many of your points and one in particular interests me
(that GA is collecting data still). We will take a look into this, as this
is unintended.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Kong/insomnia/issues/3255#issuecomment-812130650, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AODFS272SRLHY5CL6UMFOCLTGTEIPANCNFSM42GO5JBQ
.

just wanted to chip in here and say that I know for a fact that every Kong employee involved with insomnia takes privacy and security seriously and we are actually talking about this / looking into it.

Thanks @dimitropoulos, a clear and confident public statement such as that goes a long way, and the quick turnaround time for the resolving PR shows that you mean it.
I'll add a mention of this to the original ticket message for anyone coming across it in the future.
Thank you again for the quick resolution.


Add support for selectAccount and "Update Mode"

Add support for:

Accepting meta parameter in success callback and configuring selectAccount for Plaid client.
Initializing component in update mode for users who change their online banking credentials.

Issue: #5

This looks great @jpongin! Sorry for the delay on this! Cutting a new release now.


App does not run in osx

The app does not run after the osx package is packaged, with the message "You cannot open the application "wow.export" because it may be corrupted or incomplete." My mac permissions are turned on.

I imagine the app uses some libraries that are not compatible with OSX. That would be rad though.


chore: fix axios dependency

Somehow axios had been completely removed from the dependencies in package.json. I guess our tests still worked because @types/axios must have a dependency on axios, and by the transitive power of npm dependencies it was there. But we actually have axios in the package itself. So, users who want to use the axios emitter that we ship will have to know to also install axios without this change.
Signed-off-by: Lance Ball<EMAIL_ADDRESS>

I didn't realize we still had an axios emitter.  I thought we were trying to move away from including any specific protocol libraries.

I didn't realize we still had an axios emitter.

https://github.com/cloudevents/sdk-javascript/blob/cc43f3bd10caffbc47b9ce42df71782177176001/src/transport/http/index.ts#L1-L17

I thought we were trying to move away from including any specific protocol libraries.

Yes we are. But we forgot to actually remove it for 4.0.

The BEFORE functionality still exists until the release of 4.x at which time it will be removed.

https://github.com/cloudevents/sdk-javascript/pull/342
I mean, I guess we could leave it as-is and consider it "removed" - though it is still accessible, and could work fine as long as the user declares the axios dependency themselves.

Closing this for now. We can revisit later if we need to.


Use Web UI from speedport is not working

Thanks for the integration 🚀 🚀 🚀
I found a problem, when I use the integration I can no longer use the "normal" http://speedport.ip/ because I get logged out all the time.
Is that the case with you too?

Sorry my mistake:
Duplicated see here: https://github.com/Andre0512/speedport/issues/2

Thanks for testing! 🙂


Can't export dataset when task has 18k images

My actions before raising this issue

[ ] Read/searched the docs
[ ] Searched past issues
I tried multiple machines to ensure we have enough ram.

We have a task with 18k frames in it. But when I click on Export as a dataset and select TF OD API, it throws 503 (no upstream header error). For 4k frames, we needed 32gb ram. Do you have any suggestions why is it not working for 18k? or any suggestions on how we can improve it?
Thanks
Expected Behaviour

Current Behaviour

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Git hash commit (git log -1):
Docker version docker version (e.g. Docker 17.0.05):
Are you using Docker Swarm or Kubernetes?
Operating System and version (e.g. Linux, Windows, MacOS):
Code example or link to GitHub repo or gist to reproduce problem:
Other diagnostic information / logs:
Logs from `cvat` container

Next steps
You may join our Gitter channel for community support.

Hi, can you give your CVAT version number as described in the issue template?

This might have been fixed with recent updates, please check if this problem persists.


Calculate some value, for an interval of time to determine

Hello,
I have an issue in the calculator of value, for an interval of time.

the problem is in the 01 of each month, always it is calculates with the previous month
see photos attached for example 12/01/202, is calculated with the month of November
Thanks

Hmmm ... i'll check...
A test case would be appreciated though!

Caught on tests & fixed... releasing soon .
Thanks !

thank you very much for this wonderful tool ^^


v1.3

Upgrade to BepInEx 5.0.1
Set lib and output paths to be in the solution
Add HunieCam BepInEx config to solution

Nothing actually changed in the mod's code so a version bump may not have been necessary, but I just wanted to be on the safe side when referencing a new BepInEx assembly.


Redis Plugin Failing Health Check on Grafana Data Source Settings

Summary
I am trying to connect my Redis server to Grafana using the plugin and am getting a plugin health check failure message when I try to save and test my datasource.
Relevant information
My Redis server is running locally on the default localhost and port(<IP_ADDRESS>:6379.) I have installed the plugin and can see it on my list of available plugins on the Grafana dashboard as well as through the grafana-cli command. I have tried many url combinations(such as localhost:6379/0, http://localhost:6379, <IP_ADDRESS>:6379, etc.) and had no luck. Here is an image of the dashboard page that is causing the issue:

Thanks in advance for your help and guidance.

Double posted due to connection issue.


Don't deprecate setting symbols

in addition to symbols being very useful for stashing debug info, they error when you attempt to cast them to string for the deprecation.

Thanks @runspired good catch


Can't delete an imported cluster and then re-import it

Details TBD

Version - master
Imported a GKE cluster.  Deleted cluster, waited a few seconds.  I then re-imported the cluster.  Everything worked fine.
2 things I did see:

If you try to re-import before the rancher is finished terminating you will see errors like:

Error from server (Forbidden): error when creating "STDIN": serviceaccounts "cattle" is forbidden: unable to create new content in namespace cattle-system because it is being terminated
Error from server (Forbidden): error when creating "STDIN": secrets "cattle-credentials-d8f644d" is forbidden: unable to create new content in namespace cattle-system because it is being terminated
Error from server (Forbidden): error when creating "STDIN": deployments.extensions "cattle-cluster-agent" is forbidden: unable to create new content in namespace cattle-system because it is being termina
ted
Error from server (Forbidden): error when creating "STDIN": daemonsets.extensions "cattle-node-agent" is forbidden: unable to create new content in namespace cattle-system because it is being terminated```

If the user is getting these they should wait until STDIN is finished terminating, try again and it will  import.

2.  Any workloads that you created in rancher before the delete, will still be there.  This seems like a good thing?  As the workload is on the cluster and should show up.

Having the 'same' issue; Steps to reproduce:

import existing cluster as dev
delete cluster dev
create new cluster called dev (from existing nodes in our case)
witness provisioning error: error creating service account kontainer-engine: unable to create new content in namespace cattle-system because it is being terminated

@stefanvangastel @cjellick
I Had the 'same' issue.
This happened because the namespace was not finalize, it stuck in "Terminating".

You have to finalize the namespaces, then try add the cluster again
To Finalize, follow this :
https://stackoverflow.com/questions/52369247/namespace-stuck-as-terminating-how-do-i-remove-it/59667608#59667608
If it is a production cluster, i recommend to look why the namespaces are not terminating.


feat: Extract git remote into its own option.

This makes the usage of git remotes more explicit and easier to understand. It also allows to combine gitRemote and manifestPath if the manifest file is not in the root of the git repository.
/cc @brainstorm , @mmalenic

Kudos for those added changes, me and Marko had a few discussions about what to include in the first cut/MVP of this feature and we were a bit torn about i.e caching or naming it different/separate than manifestPath (manifestUrl?).
Your change set makes a ton of sense, thank you!

I just wish Projen and GitHub Actions didn't break this repository so often so contributing here could be more smooth. I'll release these changes as soon as I'm done fixing the release workflow 🤦

@brainstorm @mmalenic Release 0.0.31 should have these changes.


Allow unified Build Plan

Is your feature request related to a problem?
The current solution of having two different build plans for the template and solution is often not needed and error-prone, since the same changes must be done in two different build plans.
Describe the solution you'd like
Add an option (checkbox) to the programming exercise dialog which allows to select whether two different or only one build plan is needed.

I don't understand this feature request. How should we distinguish between template and solution then?
The template build plan is the one that is copied to all students. While copying it, we could theoretically adapt it, however the API of Bamboo is quite limited here and this approach might also be error-prone, because the idea is that instructors can adapt the template build however they want to their own needs.
We could theoretically allow instructors to skip the solution repository and the solution build plan, but then they won't know for sure, if the solution actually builds successful (all tests pass) and the tests cover all cases. So I don't think this is a good solution either.

I get your point. Each repo which should be tested (template/solution/student repos) needs its own Build Plan. What I would like to see are "synchronized build plans". The idea is to create one template build plan and all other build plans are just synchronized copies of this build plan. I know that copying is already done for the student repos. However, changes/bugfixes cannot be deployed.

Ok, I get your point. We could add an option to synchronized the solution build plan from the template build plan. Artemis would then basically delete the solution build plan, copy the template build plan and change the repository from template to solution. This should be doable. We will consider it in the future.


Added page_limit method on an individual request

Added method page_limit(num), which limits the total_pages and total_count methods.
It returns and empty instance if current_page is bigger than page_limit. In views, the max page showed is page_limit (if it's lower than total_pages), and total_count is limited too.
Implemented for Array, ActiveRecord and MongoID.
Usage
Model.page(params[:page]).per(per_page).page_limit(10)

In this case, if params[:page] is greater than 10, it returns an empty instance. If real total_pages is greater than page_limit, then total_pages will be 10 and total_count will be 10 * params[:per_page].

I realized that the max_pages option is quite tricky. It doesn't display any pages that are > max_pages, but when I directly go to a url with a page number that should be out of range, it'll actually display some entries. This may confuse robots from search engines. I'll defer a decision on merging this request until we fix this weird behaviour first.
Speaking of the implementation, calling #current_page is totally unexpected since users may call page_limit(100).per(10) which may change offset_value. #empty_instance also has an undesired behaviour which makes it impossible to chain scopes.


Actually minifies the .min.js file in dist

I found out that my last pull request made a standalone version, but didn't actually minify/uglify the file, so I added that this time.

@rienheuver Is there a change to get this merged and to build a minified dist?

To build this you can just run 'npm build', so that satisfies your needs @akkie?

Published 2.4.2.
Sometimes I need a poke to actually merge things 😀


Stretch Support and Test Branch tests

Hi!
This PR Merges to test branch changes made directly to master branch, which will make the full merge easier in the future and less prone to merging errors.

Added Support for Stretch
Removed a few lines of code that appears to be useless (script worked just fine without them in all tests made).

The following tests were made to the Test Branch reflecting already this commit:
Debian VM 32Bit - TESTED - Install Script OK as expected -  Not able to test VPN Connection
Ubuntu 17.04 VM  - TESTED - Install Script OK as expected - Not able to test VPN Connection
Duvuan i386 VM . TESTED - Install Script as expected - Not Able to test VPN Connection
Duvuan x64 VM . TESTED - Install Script as expected - Not Able to test VPN Connection
AWS EC2 Debian 64Bit - TESTED - Install Script as expected - Able to connect, VPN working as Expected.
AWS EC2 UBUNTU 16.04 - TESTED - Install Script as expected - Able to connect, VPN Working as Expected.
raspbian Stretch - TESTED - Install Script OK as Expected - Able to Connect, VPN working as Expected, able to reach devices inside Home Network.
This solves issue #342 and probably #360
I Think more issues got solved with this but will need to review them all latter!

@redfast00 @0-kaladin take a look at this and tell me if there's anything that needs to be changed. =)

@redfast00 naaah, no worries, I apologize for that! forgo to review that text! =D

Thank you!

Do want to merge this into master?

It's up to you and to @0-kaladin
As I mentioned I tested it a lot in different distros and architectures and the test branch seems to be working on all of them!
So as for me, The test branch is ok to be merged to Master!
Maybe you and @0-kaladin want to make just a final test to make sure it's all fine .. don't know!

Installed from test branch, worked fine :)


Installing tensorflow for CUDA and ROCM within the same Python installation

Hi,
The HPC cluster of our university has both, nodes with NVIDIA GPUs and nodes with AMD GPUs (and pure CPU nodes without any GPU). Since tensorflow packages provided for CUDA and ROCM are different but have the same name, we can only install one version for each Python installation.
Is there any solution to have both versions installed for the same Python installation?
Or is there any option to have a single package that can run on CPU, NVIDIA GPUs and AMD GPUs (by detecting the GPU)? The CUDA version can for instance already detect if an NVIDIA GPU is available and will otherwise fallback to the CPU version. So the package can run either on pure CPU nodes without any GPU and also on nodes with NVIDIA GPUs.
Using venvs is not an option, as a user can only have one venv activated at a time. So if we would provide tensorflow through a venv, users could not create their own venvs any more if they would like to use tensorflow.
I guess we cannot be the only institution with this problem, so I hope there is already a solution for having both, the ROCM and CUDA version side by side or within a single Python package?
Any help is appreciated.
Best regards
Sam

@samfux84 You're right about the challenges. Currently ROCm and CUDA versions of TensorFlow cannot coexist side-by-side within the same Python package. Could you create separate containers for each TensorFlow version (CUDA and ROCM), then select the desired container based on their specific GPU needs.
Thank you!

@sushreebarsa : Thank you for your reply. I somehow expected that currently it is not possible to have both packages available within one Python installation, but hoped that there is a workaround available.
A very simple solution would be if both packages (Tensorflow for CUDA and for ROCM) would have different package names. Then both packages could be installed within the same Python version.
I think that our current solution will be to have separate Python installations for CUDA and ROCM. This is not really desirable, as it duplicates the work of installing Python (and we plan to provide multiple versions of Python on our cluster).
Long term, it would be desirable to have a single tensorflow package that checks for the GPU type and then uses the appropriate code (Nvidia GPU/ AMD GPU/ Intel or AMD CPU), so that a single installation is sufficient.
Please feel free to close this ticket.

@samfux84 Thank you for the response!
Closing this ticket for now. Thank you!


Accepting cancellation token results into a generated code that has build issues

Accepting cancellation token at server side is really important, and some strange things happen If I accept cancellation token in my api controllers.
[HttpGet]
public async Task<SomeDto[]> GetAll(CancellationToken cancellationToken)
{
    throw new NotImplementedException();
}

It generates the followings:

instead of the followings:
public async Task<SomeDto[]?> GetAll(CancellationToken _token = default)
{
    using var result = await _client.GetAsync($"/api/Problesm/GetAll/", cancellationToken: _token);
    return await result.Content?.ReadFromJsonAsync<SomeDto[]>(cancellationToken: _token);
}

Thanks to the web api value providers, the list of strange things that people can accept in their api actions is unlimited!
For example we accept ODataQueryOptions<SomeDto> which gets filled by web api.
I'm not asking you to ignore all of them in your code, but it would be a good idea to accept a list of parameter types in ApiClientGenerator.json file, so parameters with those types will be ignored in generated code.

@ysmoradi the cancellation token isn't a problem - how do you see the json work? If the type is mentioned in there, then when generating the client, we ignore the parameter as if it didn't exist?

Exactly! Yes, I know CancellationToken is ignored explicitly, but there is a long list of other types to be ignored, that's why making that configurable does make sense!

Hi @ysmoradi I've done some work in v2.5.0 - the  https://github.com/surgicalcoder/ApiClientGenerator/blob/master/GoLive.Generator.ApiClientGenerator.Tests.WebApi/ApiClientGenerator.json example gives you an idea - you can ignore by type, by generic type (ie. any List), or anythign that has a certain attribute.
I've also implemented using IModelNameProvider.Name for renaming properties, and while I was at it, you can also set up rules to change types of parameters that get's outputted.
Hopefully this meets your requirements :)


Automatically add scheme to discovery.ec2.endpoint (backport 2.x) (#11512)

Description
Fix the problem that discovery.ec2.endpoint forgets to fill in the scheme
https://docs.aws.amazon.com/general/latest/gr/rande.html#ec2_region
Example
discovery.ec2.endpoint: ec2.us-east-1.amazonaws.com

Related Issues
Resolves #[Issue number to be closed when this PR is merged]

Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[ ] ~Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)~
[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[ ] ~Public documentation issue/PR created~

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

@peternied backport 2.x from #11512

@heyanlong Can you please rebase?

@heyanlong Can you please rebase?

Rebase to 68a415c2fd45e0cc24c6f8851610f37e92049213


feat: use new version pkg

Dependent on https://github.com/go-vela/types/pull/112
Updating to the new go-vela/types/version package to improve the information provided surrounding the application.
When I run make run locally, the plugin will panic. This is because we haven't created any tag on this repository yet.
After the first v0.1.0 tag is created, the output should look like:
{
  "canonical": "v0.1.0",
  "major": 0,
  "minor": 1,
  "patch": 0,
  "metadata": {
    "architecture": "amd64",
    "build_date": "2020-10-19T15:28:16Z",
    "compiler": "gc",
    "git_commit": "381bf67dcb342fe334e8209f27ecc1f068b9fc90",
    "go_version": "go1.15.2",
    "operating_system": "linux"
  }
}
<more logs>

I think this can stay as a draft PR until we create that first tag 👍
NOTE: This also updates the GitHub Actions pipelines to use our make commands for building the binary.

stale


[GLUTEN-3582] [CH] [Part1] [WIP]Add vectorized parquet record reader

What changes were proposed in this pull request?
This is first task to implement VectorizedParquetRecordReader interface like vanilla spark.
(Fixes: #3582)
How was this patch tested?
Using Existed UT

reopen latter


create go module

Signed-off-by: Lokesh Mandvekar<EMAIL_ADDRESS>(Hope I'm doing this right)
@mrunalp @rhatdan @mtrmac PTAL. IIUC, this is needed for https://github.com/containers/image/pull/701 tests to pass.

@vrothberg PTAL

The change LGTM but containers/image#701 fails because it's not vendored into Skopeo. c/image CI is cloning master and injects the code of the c/image PR into Skopeo to run it against Skopeo's test.

Does this need to exist? Go modules can, in general, import non-module packages, and the upstream of the fork is dead enough that I’d prefer not adding unnecessary extra features.
@vrothberg ’s suggestion is the conventional process. (Maybe, eventually, we should look into make test-skopeo using a module-aware integration mechanism, so that dependencies introduced to c/image/go.mod are automatically imported into the skopeo repository the way any Go module would.)


Permit functions that return a value to also serve as a type guard

Search Terms
Linear type, affine type, type guard
Suggestion
It would be very helpful to allow a function to serve as a type guard, but also return an unrelated value.
Use Cases
This can be used to express type changes as a result of mutating operations, covering some of the use cases of e.g. Rust's affine types. (See also #16148.)
Examples
Consider this example, compiled with --strictNullChecks:
type NonEmptyArray<T> = {
  pop(): T;
} & Array<T>;

function isNonEmpty<T>(array: Array<T>): array is NonEmptyArray<T>;
function isNonEmpty(array: Array<unknown>): boolean {
  return array.length > 0;
}

let array: string[] = ['element'];
if (isNonEmpty(array)) {  // Guard gives 'array' type NonEmptyArray<string>.
  const elem1: string = array.pop();  // Works. This is correct.
  const elem2: string = array.pop();  // Also works, but elem2 will be undefined at runtime!
}

We could make this correct if pop() could both return a value and behave as a type guard. This isn't great syntax, but nonetheless consider if this was supported:
type NonEmptyArray<T> = {
  pop(): T && this is Array<T>;
} & Array<T>;

function isNonEmpty<T>(array: Array<T>): array is NonEmptyArray<T>;
function isNonEmpty(array: Array<unknown>): boolean {
  return array.length > 0;
}

let array: string[] = ['element'];
if (isNonEmpty(array)) {  // Guard gives 'array' type NonEmptyArray<string>.
  const elem1: string = array.pop();  // Returns a string and gives 'array' type Array<string>.
  const elem2: string = array.pop();  // Doesn't compile; pop() returns 'string | undefined'!
}

Checklist
My suggestion meets these guidelines:

[X] This wouldn't be a breaking change in existing TypeScript/JavaScript code

This can use a new, previously invalid syntax to avoid affecting any existing program.

[X] This wouldn't change the runtime behavior of existing JavaScript code
[X] This could be implemented without emitting different JS based on the types of the expressions
[X] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)

There's no change in the code that's emitted; this feature would exist purely at the level of the type system.

[X] This feature would agree with the rest of TypeScript's Design Goals.

I believe that it would. It seems to be aligned well, in particular, with "Statically identify constructs that are likely to be errors."

Could this be a solution to https://github.com/microsoft/TypeScript/issues/18781 ?

That one could be solved today, I think; seems like the blocker there is getting consensus that it's a good idea.
The related #13086 could definitely be solved by this, though. In that one, the issue is this:
const map = new Map<string, string>();
if (map.has('key')) {
  const a: string = map.get('key');  // Error: Map<string, string>#get() returns 'string | undefined'.
}

The ideal thing would be to narrow the type of map such that map.get('key') would return string instead of string | undefined. As folks pointed out in #13086, though, the problem is that you could call map.delete('key') between the call to map.has('key') and map.get('key'). The idea in this issue, or some variant of it, could solve that by widening the type of map again when you call map.delete('key'). (Or by narrowing it such that map.get('key') returns the type undefined.)


<EMAIL_ADDRESS>decompress error

Extracting gimp-2.10.38-setup-1.exe ... ERROR Exit code was 1!
Failed to extract files from C:\Users\efels\scoop\apps\gimp\2.10.38-1\gimp-2.10.38-setup-1.exe.
Log file:
~\scoop\apps\gimp\2.10.38-1\innounp.log

Duplicates #14126.


Setting border colors

Is it possible to change the border color of a cell? Or do I need to use PHPExcel native functions?

Do you resolve it?

I made it work by setting style information:
$sheet->setStyle([
    'borders' => [
        'allborders' => [
            'color' => [
                'rgb' => 'AAAAAA'
            ]
        ]
    ]
]);

Later I define where I want to have borders:
$cell->setBorder('none', 'none', 'thin', 'none');

this working for me, edit this files https://gist.github.com/jasson112/bbcd256c42df336dc3ec9b49abecfbf2
and that should works with one parameter like this $sheet->setBorder('B16:T16', 'thin', "D8572C");

this working for me, edit this files https://gist.github.com/jasson112/bbcd256c42df336dc3ec9b49abecfbf2
and that should works with one parameter like this $sheet->setBorder('B16:T16', 'thin', "D8572C");

This applies border color to the outer borders of the range only. Would you happen to know how to do this to every individual cell? Similar to the "All borders" feature of excel.


Brick fireplace's only face north

This could be fixed by getting the players look direction and inverting the direction to get the facing direction but i have 0 clue how to do this as i don't do java.

I just haven't got around to adding the rotation yet but will add this eventually.


Add firebase hosting

I'm ok with this PR but we shouldn't consider it a done deal. We should also look at serving straight from a storage bucket.
I'm curious about:

Differences in cost.
Differences in using a custom domain.
Control over response headers, and the ease of setting them (the current headers are bad).

We also need to make sure this is set up internally so billing goes to the right place.

Definitely just an interim thing. I'd assumed we'd get this set up somewhere more official soon, just wanted a place with proper h2 + http redirects so lighthouse was happy.

My 2c: I like FB Hosting as a target. You have full control over headers (more so than GCS) and it’s pretty frictionless. But I’ll go with whatever.

That was the other killer feature here. We could grab the firebase.json Link header generation bit from preact-cli and get automatic h/2 push.

Gonna be honest: I don’t wanna deal with h2 push and all its problems. I’d rather not bring it into this project.

Even so, the Link header for preload can still provide an advantage over preloading via meta tags, right?

Sorry, I kinda side-tracked this PR with my comment — my comment does not veto this PR at all.
(And yes, I think being able to control HTTP Link headers can be valuable.)

@developit Please rebase & fix once you are back at work :)

🎉


cleaned up READMEs a bit

Don't worry, I only edited the repo's ones, not the engines'.

Excellent proofreading work. Thanks!


null value from ANTLR is not handled, for expressions like "eval x" or "eval x*2"

Start SeedLang.Shell, enter "eval x\n":

The returned node of parser.single_stmt() contains a null field: node.Expr = null. Meanwhile localCollection.Diagnostics.Count is still 0. The outer logic treated this as a normal case and continued the execution. The output from the shell is:
eval

Start SeedLang.Shell, enter "eval x+2\n":

Unhandled exception. System.NullReferenceException: Object reference not set to an instance of an object.
at SeedLang.Ast.BaseValue.op_Addition(BaseValue lhs, BaseValue rhs) in /Users/ygwang/src/SeedLang/csharp/src/SeedLang.Ast/BaseValue.cs:line 25

Fixed by commit #47


Selection loss when dragging mouse over mention nodes in Firefox

Describe the bug
When using the mouse to select a range, as soon as the mouse goes over a mention node, the selection is reset. This only happens in Firefox, Chrome and Safari seems to work OK. Haven't tested on IE/Edge.
To Reproduce
Steps to reproduce the behavior:
0. Use Firefox

Go to the demo page
Start selecting text with the mouse
The selection will be gone as soon as the mouse touches a mention node.

Expected behavior
The selection should not be lost.
Desktop (please complete the following information):

OS: OS X Sonoma 14.1
Browser Firefox
Version 133.0

Additional context
While we are not using lexical-beautiful-mentions we have a node that was heavily inspired by the MentionNode and I thought I'd check out if the issue is also present in this repo and it turns out that it is.
Moreover, I have found a solution for this and am happy to create a PR for it if it's also agreeable to you. We ended up adding the following CSS rules to our node:
user-select: all;
-moz-user-select: none;

I have stumbled on this while reading lexical playground's source and saw this comment. Thought I'd give it a try with user-select and it seems to work just fine.

Thanks for reporting this issue and sharing your solution!. Feel free to open a PR — I’ll be happy to review it.


Prometheus/Grafana with Lagoon

We have a Prometheus/Grafana running for some of our customers. This ticket is to bring the code from these installations into Lagoon:
Things to define:

How do we run the Prometheus Operator? Some clusters maybe already have a prometheus operator running, so we probably should from a Lagoon point of view expect that the Prometheus Operator is already there?
If a cluster has no Prometheus Operator yet, we should write a short documentation on how to get the Operator running (maybe even have a Repo for it)

Lagoon won't be including prometheus/grafana - this is in the scope of the cluster management.


Alternative menu layout: use sub-menu for profile selection to avoid extra click

I would like to suggest an alternative or maybe even additional menu layout.
Instead of clicking on the command and then getting prompted for profile selection, the main menu entry could already contain a nested profile selection sub-menu.
This could either be configurable in the extension settings or co-exist with the main menu action.

Sub-menu aren't possible atm in extensions, the feature request is https://github.com/microsoft/vscode/issues/9827
I'm not that familiar with the context menu API, we should be able to create a list  but I think it might be possible to use a setting to change the layout to inline several profiles though, maybe that would be good enough? I believe they have to be statically declared at the start though, including the title of the menu item.
Currently we have:
Open in Windows Terminal
But a settings like this:
"windowsTerminal.inlineProfile1": "PowerShell",
"windowsTerminal.inlineProfile2": "WSL"

Could enable:
Open in Windows Terminal (Profile 1)
Open in Windows Terminal (Profile 2)

Alternatively we could whitelist the common/default profile names, these titles would need to be hardcoded though, not dynamically assigned in the settings:
eg.
"windowsTerminal.inlinePowershell": true,

Could enable:
Open in Windows Terminal (PowerShell)


fix: accept zero as max_lines parameter for logs

/deploy #persist

You can access the deployment of this PR at https://renku-ci-nb-1336.dev.renku.ch

This change makes sense, but it does't fix the problem that we were having. The backend still responds with this error when we try to make a query with max_lines=0:
{"error": {"code": 1422, "message": "The request was well-formed but was unable to be followed due to semantic errors.", "detail": null}}

This change makes sense, but it does't fix the problem that we were having. The backend still responds with this error when we try to make a query with max_lines=0:
{"error": {"code": 1422, "message": "The request was well-formed but was unable to be followed due to semantic errors.", "detail": null}}

@ciyer there were actually 2 places where this needs to be changed. The first one I chaged is only for documentation purposes in the swagger page. Now it should truly work.

Yes, this fixes the problem, thanks!


Allow user to use Grid overview rather than Datatables

Motivation
Currently, all recipes are listed only in a table view (using Datatables). Community websites normally use grid views (like blog posts).
Proposed Solution
Also allow a grid view, which can be modified through a separate template

TODO:

[ ] Add pref for max. amount of items in grid and table view (two separate prefs?)
[ ] Add NextPrev for grid view

@Jimmi08 The NextPrev could use some testing, if you have some time :)

@Jimmi08 The NextPrev could use some testing, if you have some time :)

it works thanks

@Moc What is default order by grid?
Is possible to use grid like shortcode - similar functionality as news grid menu?

@Jimmi08

I think it is ascending (showing oldest first, latest last).  I'll make it a preference.
Not yet. But good idea!


fix: envoy is restarting if admin is not enabled

Issue being fixed or feature implemented

When metrics are enabled for gateway, the envoy container is restarting until admin is enabled.
What was done?

Fixed if condition to add admin cluster if admin or metrics are enabled

How Has This Been Tested?

None
Breaking Changes

None
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added or updated relevant unit/integration/functional/e2e tests
[x] I have added "!" to the title and described breaking changes in the corresponding section if my code contains any
[x] I have made corresponding changes to the documentation if needed

For repository code-owners and collaborators only

[x] I have assigned this pull request to a milestone

This is wrong ))


[Feature] Allow setting of common values between head and worker pods

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
It would be nice to be able to specify things like containerEnv once and have the values shared between the head and worker pods.
Use case
The use case here it to reduce duplication and possible errors when specifying the values for a ray cluster using the helm chart.
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

fyi: in Flink we implemented a layered podTemplate structure:
https://nightlies.apache.org/flink/flink-kubernetes-operator-docs-release-1.7/docs/custom-resource/pod-template/#pod-template


[Usage] LLaVA-13B-v0 perform badly in July code

When did you clone our code?
I cloned the code base after 5/1/23
Describe the issue
Hi, many thanks for your wonderful work.
We use 500 samples from GQA to test LLaVA models and observe strange results.

Model
June code
July code

LLaVA-13B-v0 (conv-mode: simple/llava_v0)
40.4
25.2

LLaVA-13B-v0 (conv-mode: v0)
--
28.0

LLaVA-7B-v0 (conv-mode: simple/llava_v0)
36.8
41.2

LLaVA-7B-v0 (conv-mode: v0)
--
41.4

Do you have any idea on what changes made the 7B-V0 better and what made 13B-V0 much worse?
Thank you a lot.

depending on how long it takes to do this eval loop, i would recommend using git bisect to trace the issue


I18N effort for dubbo code base - dubbo-test module

is it necessary to create a issue before i pull request?

manually merged pull request #1071


Potential collision and risk from indirect dependence "github.com/gotestyourself/gotestyourself"

Background
Repo cruise-automation/isopod used the old path to import gotestyourself indirectly.
This caused that github.com/gotestyourself/gotestyourself and gotest.tools coexist in this repo：
https://github.com/cruise-automation/isopod/blob/master/go.mod （Line 50 & 104）
github.com/gotestyourself/gotestyourself v2.2.0+incompatible // indirect
gotest.tools v2.2.0+incompatible // indirect

That’s because the gotestyourself has already renamed it’s import path from "github.com/gotestyourself/gotestyourself" to "gotest.tools". When you use the old path "github.com/gotestyourself/gotestyourself" to import the gotestyourself, will reintroduces gotestyourself through the import statements "import gotest.tools" in the go source file of gotestyourself.
https://github.com/gotestyourself/gotest.tools/blob/v2.2.0/fs/example_test.go#L8
package fs_test
import (
	…
	"gotest.tools/assert"
	"gotest.tools/assert/cmp"
	"gotest.tools/fs"
	"gotest.tools/golden"
)

"github.com/gotestyourself/gotestyourself" and "gotest.tools" are the same repos. This will work in isolation, bring about potential risks and problems.
Solution
Add replace statement in the go.mod file:
replace github.com/gotestyourself/gotestyourself => gotest.tools v2.2.0

Then clean the go.mod.

@jonnylangefeld  @cxuu  Could you help me review this issue? Thx :p


Bitmap API should provide async methods to set source

Is your feature request related to a problem? Please describe.
Currently, all bitmaps are loaded synchronously. Aside from obvious problems with UI lag, we never know the source of the Stream. For instance, browser file streams cannot be read synchronously, only async.
Describe the solution you'd like
A way to create bitmap in an async way.
Describe alternatives you've considered
It could be either a full BitmapSource implementation or just a factory method for Bitmap. Or both.
In UWP it is SetSourceAsync https://learn.microsoft.com/en-us/uwp/api/windows.ui.xaml.media.imaging.bitmapsource.setsourceasync?view=winrt-22621
Additional context
Before this is implemented, developers can create immediate memory streams, and pass that instead. It adds some memory overhead though.
             await using var stream = await storageFile.OpenReadAsync();
+            var memoryStream = new MemoryStream();
+            await stream.CopyToAsync(memoryStream);
+            memoryStream.Seek(0, SeekOrigin.Begin);
-            Avatar = new Bitmap(stream);
+            Avatar = new Bitmap(memoryStream);

This way, storageFile stream is read asynchronously, while MemoryStream is always safe to read synchronously.

cc @kekekeks

Additionnaly, it would be nice if we could use a URL to load bitmap directly in XAML, for ImageBrush and Image.

@dbriard you mean, http(s) uri? It would be out of the scope of this issue, but we do need SetSourceAsync for that anyway, sure.


Pointer-events none not working in IE

Found while testing IE8 #406

This was related to the surveys being locked. In IE8, users can go back and view locked surveys, and are still able to click on the answers and it will appear that they change, but they won't actually save on the backend. I'm removing the milestone and bug label, as it's not actually affecting data or a large proportion of users, and will revisit at a later date.

Closing -- no longer supporting IE8.


DOC: Multivariate normal CDF docstring typo

I noticed that the docstring for the multivariate normal _cdf function says
"""Log of the multivariate normal cumulative distribution function."""

However, I believe that this function is actually computing the standard, non-transformed CDF.
Obviously this is extremely minor, but I thought it could be a good "first issue" for someone!

Thanks @timleathart. True, It should have the same docstrings as cdf.

I'll take this on for a first issue - I use the library often and am looking to be more active fixing. Just will take me a minute to read over the PR process to make sure I'm doing it per the book

Thanks for your interest @bobcatCA!


TextureAtlas addRegion API (Texture)

Hi,
Maybe not so much of an issue, but I'm confused as to why a single textureatlas can contain multiple textures, isn't the point of an atlas to have only one texture with different regions, or is the purpose of this function something else?

https://github.com/libgdx/libgdx/wiki/Issue-Tracker


(BKR-239) Remove dependency on puppet-dashboard rake tasks...

... for shallow gravy.

do that

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/856/
Test FAILed.

retest this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/857/
Test FAILed.

Failures unrelated to patch.

@anodelman don't we need to keep this to work with older versions?

We do not currently have any pipelines using anything pre 3.7 (other than beaker, that does a 3.3.2 -> 3.7.2 upgrade test that is passing).  So there is a risk here, but it is low.  I think that we can clean up breakage in the future, but we need this fix right away.

Will this break anything for PE 3.3.x test runs? ( I'm starting to drop 3.1.x and 3.2.x )

I did a successful 3.3.2 install, so there's a high chance of success.  If we just consider this too risky we could wrap the call in a version check?

I'm assuming its causing issues for newer PE versions?
We could do either of 2 things.

wrap it in a version check
do some testing ( acceptance testing ) to see if it breaks testing runs on PE 3.1.x

Wrapped in a version check, backwards compatible.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/860/
Test FAILed.


The number of active users constantly increases when using constantUsersPerSec in Gatling 3.8.1

When running a test with constantUsersPerSec using Gatling 3.8.1, the number of active users reported into console/graphite constantly increases. The resulting html report also contains an increasing trend.
Here is a simple reproducer:
import io.gatling.core.Predef._
import io.gatling.http.Predef._

import scala.concurrent.duration.DurationInt

class TestSimulation extends Simulation {

  setUp(
    scenario("TestScenario")
      .exec(http("Test").get("https://gatling.io/"))
      .inject(constantUsersPerSec(1) during 10.seconds)
  )
    .protocols(http)
}

Gatling 3.7.6:

Gatling 3.8.1:

Great, thanks @slandelle!


update with latest changes from docker-ce-packaging

Cherry-pick a few choice commits for getting s390x man pages built and hash files for static binaries:

docker/docker-ce-packaging#15
docker/docker-ce-packaging#14
docker/docker-ce-packaging#2
docker/docker-ce-packaging#1

@seemethere @tiborvass

I'm almost done with: https://github.com/docker/docker-ce-packaging/pull/13, should we wait until that to merge?

This one first, to get builds going for RC2.


Changing the height and the width of the circle

Signed-off-by: Joao Vitor<EMAIL_ADDRESS>

tnks ibm


Are there any plans for adding binary release?

@Byron This tool is awesome. Thanks for your brillient work!
Currently we have to install the entire set of rust SDK to compile and install it. This is a bit cumbersome for non-rust users. This also prevents us from using it on some server environment.
So would you consider adding support for binary releases?

Anyhow, I just added releases, the MUSL one should work for most linux distributions out there.

@Byron Sorry for the late reply. The MUSL version works for me. Thank you!

Hi @Byron. Awesome tool!
I was wondering if you could provide pre-compiled binaries in the releases in GitHub. This would make dua's utilization as simple as downloading a file and running it, no rust tools and compiling time necessary.
I am not familiar with rust programming automation and deploy tools, but I can see that some Rust projects do that (not sure how):

rg: https://github.com/BurntSushi/ripgrep/releases
bat: https://github.com/sharkdp/bat/releases
xsv: https://github.com/BurntSushi/xsv/releases
fzf: https://github.com/junegunn/fzf-bin/releases (exclusive branch only for binary releases)

Thank you very much!

Thanks for the request :) - just yesterday I created a Rust github action for the very first time. Thus far I dodged it as I felt there were other priorities, but now I realise why people embrace GitHub actions.
So I will use this ticket to track progress, and since I probably 'just' have to do some copy-paste, it should be available soonish.

From now on, there will be binaries for every release.
The latest one just doubles the dua interactive performance for me, in case you want to have a look.


Enhance Dynamic Log Construction for Actions (#1799)

Summary: This PR improves the logging mechanism for actions by:

Dynamically omitting the data field in log messages when it is null or not applicable (e.g., for Click actions).
Including appropriate prepositions (into, on, from) based on the type of action performed.
Ensuring logs dynamically adapt to whether data exists.

Changes:

Updated the report method to conditionally include the data field in logs.
Enhanced the preposition selection logic to adapt to the type of action.
Refactored the performAction method to ensure consistent handling of actions with or without data.

Examples of Log Output:

Issue Reference: Fixes https://github.com/ShaftHQ/SHAFT_ENGINE/issues/1799.

@AhmdZanoon
For your review dear since you've owned logging for a while now.
Let me know if you're not available and I'll step in.

@MohabMohie i will have a look by tomorrow max .

Dear @KyrillosNageh , thanks for your time and effort , In order to avoid duplicating same methods with same functionality , I recommend having a look on the old logic for same function located here first and see if can get use of it or no.
https://github.com/ShaftHQ/SHAFT_ENGINE/blob/50d5fff76c0eaa72d2c8768d10df2a6f5e5624a9/src/main/java/com/shaft/gui/element/internal/ElementActionsHelper.java#L964
Dear @MohabMohie FYI also.

Dear @AhmdZanoon ,
Thank you for your feedback and for pointing out the existing logic in ElementActionsHelper.java. I appreciate your suggestion to review it so that it doesn't duplicate and maintain consistency.
I will review the old logic this weekend and see how it can be utilized or adapted for the current implementation.


add s390x support

I tried to build it on s390x.
It failed for http://emdebian.org/tools/debian/ repo
Any output on this would be useful

That's used for cross-compiling from x64 to ARM etc.
You could hack all the cross-compile stuff out of the build container, or try running the build non-containerised - make BUILD_IN_CONTAINER=false.

Thanks @bboreham I am able to build by running make BUILD_IN_CONTAINER=false.
I made some changes to Makefile for s390x similar to other arch like ppc64le/arm/arm64 and qemu is also present for s390x.
Can we have a s390x release?

@salamani sure, we take a look at your changes - would you like to make a PR?

@bboreham  as the support for emdebian distribution has been ceased and no further updates and future releases are planned we tried various hacks to include cross-build toolchain for s390x. After exploring the latest debian repo i.e sid it has been observed that crossbuild-essential-s390x is included in its repo. So in the upcoming debian release i.e buster the group package for crossbuild w.r.t s390x will be available in the repo. I rebuit all the BASEIMAGE's required to build weave using associated debian:sid version and I could build multi-arch specific weave image with s390x support.
Once debian:buster stable version is released we will raise a PR that updates the BASEIMAGE used in the Dockerfile with associated image build from buster version instead of stretch which is currently being used. Accordingly we can than remove the emdebian repo list addition step as crossbuild-essential-{ARCH} will be available for listed ARCH. Kindly let us know you if ave any suggestions on same.


Silverlight support

Dear all,
I'd like if jQuery File Upload could support chunked uploads also for IE 7/8/9 using SIlverlight.
Is it possible use this feature right now or the near future?
Thanks,
Marco B

I don't have any plans to implement features relying on browser plugins like Silverlight or Flash.
Even Microsoft itself doesn't seem to have long-term plans for Silverlight, as IE10 on Windows 8 won't support any plugins in the default Metro UI.
Please also have a look at this issue post:
https://github.com/blueimp/jQuery-File-Upload/issues/1012


Target profile page rewrite: Create "Chemical Probes" summary widget

Use case(s)

As a user, I want to know if there are any chemical probes that have been developed for a given gene

Summary widget (showing different states depending on data)

Full size version
Key design and interaction notes

Please make the checkmark the check icon from FontAwesome (e.g. )
Please make the X the times icon from FontAwesome (e.g. )
If we have any data for a given gene, please make the widget icon, widget border, and widget text dark grey (#5A5F5F)
If we have specific data for a given gene (e.g. probe data from Chemical Probes Portal), please make the background of the box purple (#7B196A) and the checkmark white (#ffffff) and likewise, if we don't have the specific data, please make the background of the box white and the X icon light grey (#E2DFDF)
If we do not have data for a given gene, please make widget icon, widget border, and widget text light grey (#E2DFDF)

Design assets

Colour codes are available on the OT Core Platform Team Google Drive

@andrewhercules, can we drop the source list beneath the table. It breaks the pattern across widgets, but the information is duplicated in the above design.

I know it's not ideal given that it's a direct duplication of content, but I'd suggest that we keep the source list to promote consistency. In the case of this widget, I've flagged it as one that I'd like to explore a bit further once we have the drug index and updated tractability data - perhaps it will no longer be needed?
@peatroot, that being said, if there's a way to configure the widgets to show/hide the source - and perhaps for our partners whether the source is internal or external - that would be useful for future widgets and future planning. What do you think?

Ok, let's leave the sources in. We can consider adding a toggle, but I'd be in favour of only supporting toggles like that in the configuration if partners have actually asked for it. The current configuration in the webapp is quite extensive, but we don't really know how much of it is used.

Proposed schema:
type TargetSummaryChemicalProbes {
  hasStructuralGenomicsConsortium: Boolean!
  hasChemicalProbesPortal: Boolean!
  hasOpenScienceProbes: Boolean!
  hasProbeMiner: Boolean!
}

I'm re-opening this as we will need to find a way around the content overflow issue. What exactly is the problem caused by smaller screen resolutions? At at what resolution is the issue seen?

@andrewhercules , at 1295px width, the text Structural Genomics Consortium wraps, making the checkbox grid take up the full widget, with reasonable margins. The text Data on potential chemical probes available from might just fit if we can avoid this happening (eg. writing SGC instead), but it looks a bit cramped. I could also reduce the font size a bit?

(taken at 1295px width)

Noted about the responsiveness issue for the text - we can possibly address in a subsequent iteration.
@MichaelaEBI, the amended widget view above, is that okay? The detail view will still provide further information about the individual probes.

It looks great. The "Data on potential chemical probes" is not ideal anyway since the first three should be high quality probes. Only Probe Miner (with a space!) provides information on potential probes.
If it helps we could replace "Structural Genomics Consortium" with "SGC Chemical Probes" since that is what the section on their website is called.

@MichaelaEBI, is it okay if we just use SGC for now, to compensate for the overflow and responsiveness issue identified above?

SGC is fine, but "SGC Chemical Probes" is shorter than "Chemical Probes Portal" so should not cause any problem, right?

True, my only concern is that we'd have:
SGC, SGC Chemical Probes ...
Since we're just listing sources - and not providing any links - I'm not that concerned about the repetition. But just want to make sure it's scientifically correct :)

Chemical Probes Portal, Open Science Probes and Probe Miner (note the space in Probe Miner) are also repeated. I guess it doesn't really matter what we use, we should just be consistent, ie., use the same wording in the table and below. Also, we are using an acronym for the Structural Genomics Consortium but not the other ones.

Okay, so let's use the following for the summary widget table, the summary widget sources, and in the detail view data table:
SGC
Chemical Probes Portal
Open Science Probes
Probe Miner


Brinqa: Initial Commit - First Feature

Please ensure your pull request (PR) adheres to the following guidelines:

Please refer to our contributing documentation for any questions on submitting a pull request, link: Contribution Guide

Pull Request Checklist
Please check if your PR fulfills the following requirements:

[x] Testing of all the changes has been performed (for bug fixes / features)
[x] The readme.html has been reviewed and added / updated if needed (for bug fixes / features)
[x] Use the following format for the PR description: <App Name>: <PR Type> - <PR Description>
[x] Provide release notes as part of the PR submission which describe high level points about the changes for the upcoming GA release.
[ ] Verify all checks are passing.
[x] Do NOT use the next branch of the forked repo. Create separate feature branch for raising the PR.
[x] Do NOT submit updates to dependencies unless it fixes an issue.

Pull Request Type
Please check the type of change your PR introduces:

[x] New App
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation
[ ] Other (please describe):

Security Considerations (REQUIRED)

[ ] If you are exposing any endpoints using a REST handler,
please document them in the readme.html.
[x] If this is a new connector or you are adding new actions

[x] Please document in the readme.html all methods (eg, OAuth) used to authenticate
with the service that the connector is integrating with.
[ ] If any actions are unable to run on SOAR Cloud, please document this in the readme.html.

[ ] Are you introducing any new cryptography modules? If yes, please elaborate their purpose:
[ ] Are you are accessing the file system? If yes, please verify that you are only accessing paths returned through
the Vault API.
[ ] Are you are marking code to be ignored by Semgrep with nosemgrep?
If yes, please provide justification in an additional comment next to the ignored code.

Release Notes (REQUIRED)

The query in this app is a query against the GraphQL console. There are two direct usages, the first is to put in
a singular model name (ie: Asset) to return the attributes that exist under that model. Otherwise, a pluralized version
of the model is used (ie: assets, note case sensitivity) in conjunction with a filter string (ie: name = assetName)
and a list of return values, space delineated (ie: ip id name risk) to return information, in the case of the example,
on an asset with a specific name

What is the current behavior? (OPTIONAL)

Describe the current behavior that you are modifying.

What is the new behavior? (OPTIONAL)

Describe the behavior or changes that are being added by this PR.

Other information (OPTIONAL)

Any other information that is important to this PR such as screenshots of how the component looks before and after the change.

Pay close attention to (OPTIONAL)

Any specific code change or test case points which must be addressed/reviewed at the time of GA release.

Screenshots (if relevant)

Thanks for contributing!

Hi @Jth155
Thanks for your patience. We'll start with the review.

Thank you so much!

Hi @Jth155,
We have made some code changes as per our standard practice.
Moreover, there are below mentioned changes that need to be done from your end.

Our current supported min_phantom_version is "6.0.0". Can you please upgrade the min_phantom_version and test the app on that version?
There are occurrences of "degub_print" in the connector file which is undefined. The expected method for logging is "debug_print". This needs to be changed
The app "description" field in the json file contains just the name of the app. It should contain at least 22 letters.

Can you please make the required code changes? Please let us know if you have any concerns.

Hi @Jth155, The product_name is "GraphQL API" which is generic. Should we change it to "Brinqa GraphQL API" or something? Also, we can update the app description according to that.


add packagist hook

I created an entry on packagist to be able to install this using composer. @mweibel can you add the packagist hook? You'll need an API key, I can supply one, but i'd like to pm that.

I created now an API key myself, but you can also send me yours to<EMAIL_ADDRESS>if you rather have this.


Keyboard is Locked when we press ALTgr + @ on virtual keyboard on Windows

Hello, i have some problem with virtual keyboard on windows. When i'm trying to tap ALTgr on virtual keyboard on windows, the entire keyboard is locked. It's impossible to tap on any other key. (I'm using french keyboard). If i press Ctrl left two times, the keyboard is now unlocked and i can type other key.
The issue 78005 is about the same problem. A pull request has been done to fix this problem, but i think that fix only for real keyboard and not for a virtual keyboard.
Steps to Reproduce

Execute flutter run -d windows on the code sample (on windows)
Open virtual keyboard like on-screen keyboard.
Press @ with ALTgr in the Textfield and try after to press letters.

Expected results:
I can use keyboard normally.
Actual results:
I can no longer use keyboard if i press ALTgr and @.

Code sample (it's the sample of TextField in flutter)

import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  static const String _title = 'Flutter Code Sample';

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      title: _title,
      home: MyStatefulWidget(),
    );
  }
}

class MyStatefulWidget extends StatefulWidget {
  const MyStatefulWidget({Key? key}) : super(key: key);

  @override
  State<MyStatefulWidget> createState() => _MyStatefulWidgetState();
}

class _MyStatefulWidgetState extends State<MyStatefulWidget> {
  late TextEditingController _controller;

  @override
  void initState() {
    super.initState();
    _controller = TextEditingController();
  }

  @override
  void dispose() {
    _controller.dispose();
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: TextField(
          controller: _controller,
          onSubmitted: (String value) async {
            await showDialog<void>(
              context: context,
              builder: (BuildContext context) {
                return AlertDialog(
                  title: const Text('Thanks!'),
                  content: Text(
                      'You typed "$value", which has length ${value.characters.length}.'),
                  actions: <Widget>[
                    TextButton(
                      onPressed: () {
                        Navigator.pop(context);
                      },
                      child: const Text('OK'),
                    ),
                  ],
                );
              },
            );
          },
        ),
      ),
    );
  }
}

Logs

There are no exceptions with flutter run --verbose

Analyzing bug...
No issues found! (ran in 4.0s)

    • Flutter version 2.10.4 at C:\Users\Optyce\Documents\flutter_sdk\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision c860cba910 (3 weeks ago), 2022-03-25 00:23:12 -0500
    • Engine revision 57d3bac3dd
    • Dart version 2.16.2
    • DevTools version 2.9.2

[√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\Optyce\AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.3
    • Java binary at: C:\Program Files\Common Files\Oracle\Java\javapath\java.exe
    • Java version Java(TM) SE Runtime Environment (build 16.0.1+9-24)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.10.4)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.10.31515.178
    • Windows 10 SDK version 10.0.19041.0

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).

[√] VS Code (version 1.66.1)
    • VS Code at C:\Users\Optyce\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.38.1

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19043.1586]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 100.0.4896.88
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 100.0.1185.29

[√] HTTP Host Availability
    • All required HTTP hosts are available

Hi @Alexandre-Burger, thanks for filing the issue. I tried to verify this on a qwerty keyboard and I was not able to reproduce the issue. I believe this issue could be specific to french keyboards. Can you please check if this is reproducible on the master channel? You can switch to the master channel by running
flutter channel master
 flutter upgrade

flutter doctor -v (windows)
[√] Flutter (Channel stable, 2.10.4, on Microsoft Windows [Version 10.0.19044.1586],
    locale en-US)
    • Flutter version 2.10.4 at C:\flutter_sdk\stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision c860cba910 (5 days ago), 2022-03-25 00:23:12 -0500
    • Engine revision 57d3bac3dd
    • Dart version 2.16.2
    • DevTools version 2.9.2

[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\Users\mahesh\AppData\Local\Android\sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.11.4)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.11.31727.386
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 2020.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[√] VS Code (version 1.64.2)
    • VS Code at C:\Users\mahesh\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension can be installed from:
       https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[√] Connected device (3 available)
    • Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version
      10.0.19044.1586]
    • Chrome (web)      • chrome  • web-javascript • Google Chrome 99.0.4844.82
    • Edge (web)        • edge    • web-javascript • Microsoft Edge 99.0.1150.30

[√] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Hi @maheshmnj. Thanks for your anwser. I try on master channel and it works. i can't reproduce the bug on master channel. So i have a question : can we use master channel for production? Is it safe?

Thanks for the update @Alexandre-Burger.

can we use the master channel for production? Is it safe?

Using master channel is not recommended for production apps as it is the tip of the tree all PR's from contributors are merged in the master channel every day, which includes accidental bugs too :)
To understand more about flutter channels and the release cycle please see https://github.com/flutter/flutter/wiki/Flutter-build-release-channels.

I try on master channel and it works.

Closing as fixed based on the above response.


localization update

Please fix the "_locale.stub" files because their html parts are not up-to-date.
e.g. "menu_template.stub" was updated some month ago, but "menu-template_locale.stub" was not. But it is the same for all localized template files I guess.
Thank you

Seems like is also missing app_locale.stub and sidebar_locale.stub.
When I run php artisan infyom.publish:layout --localized, the below error are shown:
copy(~/laravel/vendor/infyomlabs/adminlte-templates/templates/scaffold/layouts/app_locale.stub): Failed to open stream: No such file or directory

  at vendor/infyomlabs/laravel-generator/src/Commands/Publish/PublishBaseCommand.php:20
     16▕         if (file_exists($destinationFile) && !$this->confirmOverwrite($destinationFile)) {
     17▕             return;
     18▕         }
     19▕
  ➜  20▕         copy($sourceFile, $destinationFile);
     21▕
     22▕         $this->comment($fileName.' published');
     23▕         $this->info($destinationFile);
     24▕     }

*** routes added.
   ErrorException
  file_get_contents(~/laravel/vendor/infyomlabs/adminlte-templates/templates/scaffold/layouts/menu_template_locale.stub): Failed to open stream: No such file or directory

  at vendor/infyomlabs/laravel-generator/src/helpers.php:138
    134▕     function get_template($templateName, $templateType)
    135▕     {
    136▕         $path = get_template_file_path($templateName, $templateType);
    137▕
  ➜ 138▕         return file_get_contents($path);
    139▕     }
    140▕ }

The error shows also for infyom:api_scaffold command.
Maybe I should create a new issue.


Plugin does not support help files

Per comments in https://github.com/jenkinsci/localization-zh-cn-plugin/pull/31, this plugin does not currently support help files. The presence of HTML help files for zh_CN resulted in this looking like it'd "just" work, when in fact it does not.

Mostly handled by https://github.com/daniel-beck/localization-support-plugin/commit/492bb166f5212ae8600fd83f2279f3097deab0c5 for Descriptor#doHelp.
Still open:

war/src/main/webapp/ resources (some freestyle job help files)

Undetermined:

Potentially plugin resource files via Plugin#doDynamic

FWIW, neither the currently released localization-zh-cn plugin, nor this one support localizing war/src/main/webapp/ resources, like e.g. http://localhost:8080/help/project-config/custom-workspace.html
So at least this isn't worse than what currently exists.


Autocomplete

This is a great library!
I built an autocomplete / select control for Solid (https://solid-select.com/). It has a core that is ui independent. I'm curious if it might help with the Autocomplete component you have planned? Let me know if you think it's interesting to collaborate on.

Hey, thanks 😁.
Yes i think we can make a good Autocomplete by mixing the solid-select logic with the base accessible foundation i've built for the Select.

Cool! Let me know if you'd like me to try and work up a pr after solidhack or if you'd like to try first.

You can try work on it if you have time, all the ui design is the same as Select. Also i will add you (and others) to allcontributors after the end of solidhack.


Use a base image instead of scratch

This allows glog to work, since /tmp will be available. This also causes
env vars to be set, so $HOME is available for the config file lookup.
Fixes #31
See https://github.com/coreos/flannel/issues/513 for more context.

Many thanks!!


Merge hotfix to repair broken staging links into develop

And resolve merge differences since the Narrative welcome page and login widget have been changed in develop.

Oops, this should have been merged some time ago. These are minor hotfixes that are already in production.


Bump dependencies

Coverage remained the same at 70.678% when pulling 17666af03784af3ba0a7cba1285f958f55e337c5 on shssoichiro:deps into e44bf1cb93ceba16e6a845e27f1a1a9fbac80712 on xiph:master.


Refactor genericclioptions.Context

What type of PR is this?
/kind cleanup
What does this PR do / why we need it:
Which issue(s) this PR fixes:
Fixes #5176
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

[ ] I have read the test guidelines

How to test changes / Special notes to the reviewer:

/test unit

/test psi-kubernetes-integration-e2e
It tends to show old result.

/test v4.9-integration-e2e
Error: Provider produced inconsistent result after apply

/approve


CIF Page component custom head template overrides support for HtmlPag…

…eItems

Added support for HTML page items to CIF page head

Closes  #546

@dplaton, @herzog31,  @mhaack, could you please review?
The failing tests are most probably because I submitted from a fork (and have no access to CircleCI secrets):

Error response from daemon: Get https://docker-adobe-cif-release.dr-uw2.adobeitc.com/v2/circleci-openjdk-azul/manifests/8-stretch-node-browsers: unknown: Authentication is required

Verified by setting some manual com.adobe.cq.wcm.core.components.config.HtmlPageItemsConfig config on our Venia page