Overall Project info page changes.

Page Project
Change ordering of sections to:
The preferred order of categories
Project
Wrangling
Submission steps
Post submission tracking
We feel a need to remove the star system (can discuss with Clay if needed)
Consent to use boolean "Yes" or "No"
Sample type → with enums
Add “library preparation methods”
Add organ, organ_part, disease, cell count to this section
Combine Pubs, people &  pay into one on the page.

sample type enums:

primary tissue
organoid
stem cell derived
cell line

library preparation methods enums

Paris - how are we going to tell if it's frozen or fresh?    Maybe what I'm wanting to capture in this field is spread across many in the spreadsheets.

@JimKent  that would be captured by 'preservation_method', which has the following enums:
"cryopreservation in liquid nitrogen (dead tissue)",
"cryopreservation in dry ice (dead tissue)",
"cryopreservation of live cells in liquid nitrogen",
"cryopreservation, other",
"formalin fixed, unbuffered",
"formalin fixed, buffered",
"formalin fixed and paraffin embedded",
"hypothermic preservation media at 2-8C",
"fresh"

Thanks @pnejad.  I added a preservation_method field hooked up to a vocab in the x table of the same name.  I just populated it with fresh.   I also replaced "fresh sample" in the "sample type" table with "primary tissue."
WHen I release the next version,  possibly later today,  could you edit the sample_type table inside of Tuatara to match your list,   and add the ones (or at least the ones you think we'll see) from the preservation method?   Thanks!

Ok, the dev version has all the stuff from the initial ticket that I agree with and can change.  A few things are reworked enough I'd like you to just have another look, and let me know if you like it.
For the concent I am not using the yes/no.   I feel like we are overloading this field trying to make it serve both "concent from donor" and "concent from researcher".     With this one I am just trying to capture concent from donor.  Also I want to capture the donor concent in a slightly more nuanced way than just yes/no.
The AssayType is now called cDNA library prep.    I have not edited out the SE and PE from the smart-seq2,  and honestly am tempted not to, as  currently the distinction is important,  but apparently both SE and PE smartseq2 is going to be handled soon if not already by the pipeline,  so we could just merge the fields at that point.
In terms of combining a bunch of stuff into the Overall section,  please look at the new arrangement, and let me know if you would still like to.     I did rearrange the Overall section quite a bit.

@JimKent  how often do we anticipate tracking a dataset where the donor has not given consent?
If consent has not been given by the donor, it should not be added to tuatara in the first place. I can only see this being useful in a scenario where the researchers are going back to the donors to request consent.
The consent that wranglers need to worry about is from the researchers (not applicable to GEO wrangling). We can not accept metadata or data files until the data contributor agrees to the terms in this form:  tinyurl.com/HCA-Terms-Conditions

@JimKent

The AssayType is now called cDNA library prep. I have not edited out the SE and PE from the smart-seq2, and honestly am tempted not to, as currently the distinction is important, but apparently both SE and PE smartseq2 is going to be handled soon if not already by the pipeline, so we could just merge the fields at that point.

Where would I see the SE and PE?

It is still right after smart-seq2 in library prep controlled vocab.

In terms of tracking stuff without consent - I do think we'll want to track some projects before we get the T&C.   Historically we have done so with some from Aviv Regev's lab I believe for instance.


feat(kitchensink): add kitchen sink component

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Authorization issue

I have installed Camaleon into my existing project with no problems at all. however when I try to access it I get the following error message
CanCan::AuthorizationNotPerformed in CamaleonCms::Admin::InstallersController#index
This action failed the check_authorization because it does not authorize_resource. Add skip_authorization_check to bypass this check.
Any idea of what I should do?

Put like this in your initializer:
CamaleonCms::Admin::InstallersController.class_eval do
  skip_authorize_resource ....
end

By this way you can add or modify an existent class


Autoroller commit messages do not include all commits in the description.

An example is this attempted roll: https://github.com/flutter/flutter/pull/47283
Having an incorrect list of commits in the description leads to inaccuracies and confusion in diagnosing issues related to rolls.
The description lists the following commits:
2019-12-18<EMAIL_ADDRESS>Manual roll of Dart to revision 8a4cac1ddb141d4a1e395f6e0a7d5bbf2247eb63 (flutter/engine#14542)
2019-12-18<EMAIL_ADDRESS>Calling onConnectionClosed when the input element is blurred (flutter/engine#14484)
2019-12-17<EMAIL_ADDRESS>Roll src/third_party/dart f10fa3df28a8..44da51fc720d (20 commits) (flutter/engine#14533)
2019-12-17<EMAIL_ADDRESS>Roll src/third_party/skia cbeabd969bbe..3794db84e141 (8 commits) (flutter/engine#14532)
2019-12-17<EMAIL_ADDRESS>Rename 'test_vsync_waiters' to 'vsync_waiters_test' for consistency. (flutter/engine#14519)
2019-12-17<EMAIL_ADDRESS>Roll src/third_party/skia fdb2b7d53038..cbeabd969bbe (1 commits) (flutter/engine#14530)

Running the command right there in the commit message manually lists the following however:
44f075779 Manual roll of Dart to revision 8a4cac1ddb141d4a1e395f6e0a7d5bbf2247eb63 (#14542)
07429c1c3 Revert "[fuchsia] Do not Execute paint tasks when there is no vsync (#14298)" (#14541)
fad86658d Roll src/third_party/skia 3794db84e141..190b82d67ce1 (11 commits) (#14538)
c216ba42b Calling onConnectionClosed when the input element is blurred (#14484)
3da1e11fb Roll fuchsia/sdk/core/linux-amd64 from ZHhOi... to WSlsF... (#14534)
ac42640fd Roll src/third_party/dart f10fa3df28a8..44da51fc720d (20 commits) (#14533)
86456e1ee Support timeline can enabled in release mode (#14521)
0f97a1e83 Roll src/third_party/skia cbeabd969bbe..3794db84e141 (8 commits) (#14532)
9b369ad19 Rename 'test_vsync_waiters' to 'vsync_waiters_test' for consistency. (#14519)
ad3bade1d Roll src/third_party/dart 51859cb2056c..f10fa3df28a8 (4 commits) (#14531)
9f2daad64 Fix off-by-one fromRGBO alpha value calculation (#13777)
ee92285ab Roll src/third_party/skia fdb2b7d53038..cbeabd969bbe (1 commits) (#14530)
02825aa02 Roll src/third_party/dart 2d332ee1d3f1..51859cb2056c (2 commits) (#14529)

In this particular case, one of the commits missing from the commit message was the cause of the failure caused by the roll.

Also I found that only the commits that are all green on luci will show up here, the ones with failing jobs are all missing.

Thats a very interesting observation. I only recently found that the commit messages are not generated by using the command listed in the commit message itself.

cc @rmistry, is there something we can do to include all commits in the description?

Will ask borenet@ for advice on this.

Hrm, this has to do with the way the autoroller is structured.  We operate numerous rollers, and there are several types which don't roll git commits at all.  There are some abstractions in place to allow different types of rollers to behave the same way despite these differences.  One of those is Revision, which in this case is a Git commit.  At the lowest level, RepoManager is responsible for determining the current Revision and the set of not-yet-rolled Revisions.  It also knows how to create a roll to a particular Revision.  The rest of the logic is higher up, including the strategy for choosing which Revision to roll, which can be set in the UI.  A lot of this was added in https://skia-review.googlesource.com/c/buildbot/+/255823
So, as you've discovered, the "git log" command is kind of a lie, although we do keep it in sync with the actual command used to generate the list of not-yet-rolled Revisions.  The reason that this particular roller is showing an incomplete list of Revisions is that the RepoManager is configured to filter that list by the presence of corresponding files in GCS: https://skia.googlesource.com/buildbot/+/master/autoroll/config/flutter-engine-flutter.json#39.  That prevents the roller from rolling to a Revision for which those files are not present, which is the desired effect, but it also causes any commits which are missing files in GCS to be dropped from the commit message.
There are other RepoManagers which filter out Revisions, but those do not roll Git commits and thus have no concept of a log.  The best fix is probably to have this RepoManager return the unfiltered list of Revisions but mark those which do not have corresponding files in GCS as invalid roll candidates, and then respect that when choosing a Revision to roll.

The best fix is probably to have this RepoManager return the unfiltered list of Revisions but mark those which do not have corresponding files in GCS as invalid roll candidates, and then respect that when choosing a Revision to roll.

This SGTM. (Eric mentioned he is going to look into doing this)


Updated SwapchainConfig logic

Fixes #2322
Addresses the Metal part of #2291
The PR changes our swapchain configuration to reflect the capabilities better and be constructed from it.
It also removes the earlier hack in Metal backend  w.r.t. applying the hiDPI factor, rewriting the logic of how CAMetalLayer is created and configured. Now, it's logical and physical spaces strongly correspond to the underlying view. It respects the given extents not by changing the layer, but by configuring the images (drawables) produced from it.
Most importantly, the render space matches logical space of the layer. This means that if we have a window of size 1440x900 with hiDPI scale of 2.0, then it will behave as if it's a 1440x900 window and report size and create drawables accordingly. The scaling/stretching is left for the window system to apply. This works for both our examples and Dota. Ultimately, we don't want to be processing more pixels just because our hiDPI is high.
PR checklist:

[x] make succeeds (on *nix)
[x] make reftests succeeds
[x] tested examples with the following backends: metal
[ ] rustfmt run on changed code

Thanks for taking a look! I believe the example doesn't work as intended for me so needs a follow-up fix anyway.
Bors r=msiglreith

On Aug 15, 2018, at 17:38, msiglreith<EMAIL_ADDRESS>wrote:
@msiglreith approved this pull request.
In examples/colour-uniform/main.rs:

@@ -617,7 +610,7 @@ fn create_backend(window_state: &mut WindowState) -> (BackendState<back::Backend
BackendState {
adapter: AdapterState::new(&mut adapters),
surface,

       window,

       //window,

nit: remove?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Oops, didn't realize windows CI isn't happy...

On Aug 15, 2018, at 17:58, bors[bot]<EMAIL_ADDRESS>wrote:
Build failed
continuous-integration/appveyor/branch
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Addition of image collection statistics

I think it would be useful to have some built-in image collection statistics. Not exactly sure what all we would want to include. But things along the lines of (for a given date range)

Number of images taken
Number of images per day
Number of images that pass some specified cuts
etc.

We might even consider adding some interactive plotting capabilities.

On second thought, I don't think this is super useful anymore. Closing.


docs: Add pages for releases

Adding Documentation
Description
Adding Documentation for release notes and a public roadmap
Type of Change

[ ] ✨ feat -- New feature (non-breaking change which adds functionality)
[ ] 🛠 fix -- Bug fix (non-breaking change which fixes an issue)
[ ] ❌ ! -- Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 refactor -- Code refactor
[ ] ✅ ci -- Build configuration change
[ x ] 📝 docs -- Documentation
[ ] 🗑 chore -- Chore

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: kaziwaseef:x: alexdukeinvertaseYou have signed the CLA already but the status is still pending? Let us recheck it.


Some installers failing with Cairo running

I've been using Cairo for a few weeks now, and I've been really enjoying it. However, I recently noticed an issue: Programs relying on Squirrel.Windows for their installer seems to fail on Windows 10 as long as Cairo is open; I tried with Station, Rambox, and MailSpring; each of them failed to install, and left no stack-trace in the SquirrelLog file (located in %APPDATA%/Local/SquirrelTemp). The only solution I've been able to come up with is to close Cairo whenever trying to run one of these installers.
I haven't noticed any other installer frameworks with this issue, nor does it surface with apps that don't use an installer to begin with.

Squirrel installer has some interesting notification area handling code (but I need to spend some time to figure out what's causing this result).
In Cairo Settings > Menu Bar, disabling "Periodically re-establish notification area hook" seems to allow Squirrel installers to run.

Seems like an odd thing to halt an installer, but I can confirm that this allows the Station installer to run properly.

Interestingly, the latest version of the Station installer seems to work now.

This is fixed in the latest release.


WARN  [roslibrust::rosbridge::client] Failed to reconnect: CommFailure(Io(Os { code: 111, kind: ConnectionRefused, message: "Connection refused" }))

Hi, I created my won package using  example  . I am getting the following
[roslibrust::rosbridge::client] Failed to reconnect: CommFailure(Io(Os { code: 111, kind: ConnectionRefused, message: "Connection refused" }))
Code I am using is
use log::*;
use roslibrust::ClientHandle;

roslibrust_codegen_macro::find_and_generate_ros_messages!("assets/ros1_common_interfaces/std_msgs");

/// This example creates a client, and publishes a message to the topic "talker"
/// Running this example at the same time as subscribe_and_log will have the two examples
/// pass messages between each other.
/// To run this example a rosbridge websocket server should be running at the deafult port (9090).
#[tokio::main(flavor = "multi_thread")]
async fn main() -> Result<(), Box<dyn std::error::Error + Send + Sync>> {
    simple_logger::SimpleLogger::new()
        .with_level(log::LevelFilter::Debug)
        .without_timestamps() // required for running wsl2
        .init()
        .unwrap();

    let client = ClientHandle::new("ws://localhost:9080").await?;
    let publisher = client.advertise::<std_msgs::Header>("talker").await?;

    loop {
        let msg = std_msgs::Header::default();
        info!("About to publish");
        publisher.publish(msg).await.unwrap();
        info!("Published msg...");
        tokio::time::sleep(tokio::time::Duration::from_secs(1)).await;
    }
}

Are you running the rosbridge server on the host? You mentioned in the other issue that you were attempting to send a message to a remote machine, but I see the URI here includes localhost which will of course not be a remote machine.

I am trying on the same machine then I will try sending it to remote, I would like to know is there any specific format in which I should enter the ip address of remote computer

I found out the work around where I am using another system without realsense. It is acceptable in my work case.
I figure out that It is essential to run rosbridge on the computer where you want to send data to. After installing rosbridge using the following command sudo apt-get install ros-<rosdistro>-rosbridge-server and launching using the command roslaunch rosbridge_server rosbridge_websocket.launch it was working perfectly. I would like to suggest to take care of the port number as rosbridge usually use 9090 and any different port number in the code could lead to error.
I am closing this issue as it has been solved. If the problem persists, please comment and the issue will be reopened if appropriate.


Allow updating mimeType and filename properties

Fixes https://jira.duraspace.org/browse/FCREPO-1532

Superseded by https://github.com/fcrepo4/fcrepo4/pull/843


Un-generated Client error message could point out that using output might require different import

Problem
It could happen that you set an output in your generator, but you still import { PrismaClient } from '@prisma/client'; in your project.
Under certain circumstances (e.g. when you run npm install in a project that has prisma as devdependency and @prisma/client as dependency, see package.json example below) it can happen that the uninitialized Client is put into the default location in node_modules/.prisma/client`, where the default import will look for a generated Client.
This will lead to an error like this:

@prisma/client did not initialize yet. Please run "prisma generate" and try to import it again.

Right now the error message is not super helpful in connecting these dots and might send a user to hunt a problem (Prisma Client was not generated) that does not even really exist.
Example: https://github.com/prisma/prisma/issues/6644
Suggested solution

The error message could get a second line that points out that if you are using output in generator, you might have forgotten to import your Prisma Client from there and should do that instead.
Optionally it could even read check for an output value and see if this matches the current, own import. If not it could give some helpful output.

Alternatives
Do not forget that you are using an output in your generator
Additional information
package.json:
{
    "name": "test",
    "version": "1.0.0",
    "description": "",
    "main": "index.js",
    "devDependencies": {
        "prisma": "^2.20.1"
    },
    "dependencies": {
        "@prisma/client": "^2.20.1"
    }
}

Hi there,
As part of our effort to clean up our GitHub repo, we’re closing this issue to help focus on features and improvements that align with our focus areas and those of our community.
This issue was originally created for internal tracking purposes, but if you feel this is important to you, feel free to reopen the issue and get like-minded users to upvote as we will prioritize fixing issues based on community interest.
For more insight into our priorities and how we’re thinking about the future of Prisma, please see our blog post: https://www.prisma.io/blog/prisma-orm-manifesto.
Thank you for your understanding and support!


Remove throws in noexcept methods.

Both condition_variable_any::notify_one and condition_variable_any::notify_all
are marked noexcept, so they are not allowed to throw. Since a throw() will
automatically call std::terminate(), it seems reasonable to call it directly.
This change fixes GCC warnings from -Wterminate while retaining the same
runtime behaviour.

Merged, thank you!


Feature request: Ability to place a DragAndDropList inside another scrolling container

Hi @Norbert515
Thanks for creating this great and easy to use widget!
One thing I've run into is that putting a DragAndDropList inside another scrolling container gives an unbounded height error.
Maybe you could expose the Shrinkwrap property in ListView.builder?
Btw: Futter's ReorderableListView has a similar limitation:
Github/Flutter/ReorderableListView is too limited to be (always) useful #25789
Thanks!

I overlooked this issue, I'm sorry.
If you haven't already fixed it - does wrapping this widget in an Expanded help? Or did adding the shrinkwrap property fix the issue?


Uncaught Exception: listen EADDRINUSE <IP_ADDRESS>:5672

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.2.4
Time: Tue May 03 2016 15:22:16 GMT 0200 (MitteleuropÃ¤ische Sommerzeit)
Error:
listen EADDRINUSE <IP_ADDRESS>:5672
Error: listen EADDRINUSE <IP_ADDRESS>:5672
    at Object.exports._errnoException (util.js:890:11)
    at exports._exceptionWithHostPort (util.js:913:20)
    at Server._listen2 (net.js:1234:14)
    at listen (net.js:1270:10)
    at net.js:1379:9
    at _combinedTickCallback (internal/process/next_tick.js:77:11)
    at process._tickCallback (internal/process/next_tick.js:98:9)

This issue was created automatically inside the uncaughtException handler

Closing this automatically reported issue as a duplicate of #325


fix var tooltip message (#1001007)

I just glimpsed at code and I think Simon will confirm, that formatting could be made better :). Actually it is really better place to control of tooltip content.

add to whitelist

As already said, the formatting is a bit off. Especially the whitespace needs to be improved. Would you please run the "format selection" action on a selection of your contributed code?

The commit message needs to be improved too. See this section. The ticket needs to be mentioned as Fix #1001007 in the body and not in the title.

Superseded by #1005


[Issue] OpenAI & Search account not listed in created resources

I'm provisioning open ai account, but it doesn't appear in list

param name string
param location string = resourceGroup().location
param tags object = {}

param deployments array = []

resource account 'Microsoft.CognitiveServices/accounts@2022-10-01' = {
  name: name
  location: location
  tags: tags
  kind: 'OpenAI'
  properties: {
    customSubDomainName: name
    publicNetworkAccess: 'Enabled'
  }
  sku: {
    name: 'S0'
  }
}

@batchSize(1)
resource deployment 'Microsoft.CognitiveServices/accounts/deployments@2022-10-01' = [for deployment in deployments: {
  parent: account
  name: deployment.name
  properties: {
    model: deployment.model
    scaleSettings: deployment.scaleSettings
  }
}]

output id string = account.id
output endpoint string = account.properties.endpoint

@hemarina - Could you also add the search service? Here's the bicep we are using for this project: https://github.com/Azure-Samples/azure-search-openai-demo/blob/main/infra/main.bicep

Marking this as blocker for 0.8 April 5th release. If not needed, please clarify and remove blocker.
@savannahostrowski @ellismg


Need the English to French, English to Hindi and English To Marathi training model

Need the English to French, English to Hindi and English To Marathi training model
Basically as per link http://indianexpress.com/article/technology/tech-news-technology/googles-neural-machine-translation-for-indian-languages-heres-what-it-means/ we are looking to implement tensorflow NMT model for at least two Indian languages Hindi and Marathi. How do I configure tensorflow to perform these translations. Can a single tensorflow model handle multiple languages translations and if so you could provide any such documentation would be highly appreciated .  As per my knowledge Cloud Auto ML https://cloud.google.com/automl/ and NMT appears to be the same. Was just wondering when would the Cloud Auto ML version be available for Google Translate and would be it be available as an on premise option?

Any leads on this ?

No nothing concrete.


small nomanclature change

I say we change all:
"JATOS installation" (like in local installation/server installation)
to
"JATOS instance"
If you say yes, I make the changes in wiki and paper

Okay.
On 23 Apr 2015 5:54 pm, "elisafilevich"<EMAIL_ADDRESS>wrote:

I say we change all:
"JATOS installation" (like in local installation/server installation)
to
"JATOS instance"
If you say yes, I make the changes in wiki and paper
—
Reply to this email directly or view it on GitHub
https://github.com/JATOS/JATOS/issues/82.

Done


Make a document about type check

The type check system of Chainer is difficult to use. It is something like an interpreter. We need to make a document about it.
I think a detailed specification doesn't help users. Instead, how about best practice and a lot of examples?

I'll make a tutorial document about typecheck.


Fixes radio speech transforms

This should fix radio speech transforms for things like accents, etc.

oops, #10226 exists


Notification : OK/Cancel Dialog

Description
It would be useful if there was not only one option to close notifications.
As in Dashboard 1.0, there is an option to have one button for OK and one for Cancel.
For example, you have a button to delete a file and then a notifitcation where you ask the user "Sure" or "Cancel".
Have you provided an initial effort estimate for this issue?
I can not provide an initial effort estimate

I think this is a duplicate of #193

@colinl I think you're right.


Revert back groups cache and split GeofenceManager.refresh() on 3 functions

Split geofence refresh on 3 functions
Enable back groupsCache

For now groupsCache have unwanted update only if device(group) linked to deleted group.
I'm researching foreign keys, but have some issues (null and not null incompatibility I think)
For demo servers you can null dead links via sql, it can help for a while.

Merged, thanks.


Testimonial Grid Section

Describe the project
Testimonial Grid Section using HTML and CSS
Issue Type:

[x] New Project

Screenshots

Sir, I do not understand the issue. can you describe it? I would be happy to work on that and it would be a pleasure for me to work on that.

Already present in the repo


add a flag to disable computing statistics tendencies

Should we have a flag to disable the computation of statistics_tendencies_* during building a dataset?

Another option could be to allow for the delta's of the additions to be specified in the config file?

Anemoi-datasets can now create datasets incrementally https://anemoi-datasets.readthedocs.io/en/latest/building/incremental.html.
This is the recommended way to build datasets without the statistics tendencies.


mtail fix mapfile for mac and add support for name spaces and multi-container

mapfile does not work on mac so replaced.
namespaces can be passed on as parameters now and as well as container name in case of the multi-container pod.

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.
What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.

ℹ Googlers: Go here for more info.

I signed it!

mtail now lives at https://gitlab.com/grzesuav/kubectl-mtail, please make a PR there.
cc: @grzesuav

@ajitchahal https://gitlab.com/grzesuav/kubectl-mtail/merge_requests/1 provides support for container selection, hopefully I have time next week to finish work with adding it to krew


helpers for setting priors?

e.g. for K, offer a function that uses Raymond et al. 2012 equations to generate a first-cut suggestion for a prior based on your model slope, etc.?

In Bob's words:

Ideally we would use the posterior predicted from a model of Kpred = f(vel, Q, slope) to generate the priors for K in any of our metabolism models.

this would go with #312, #255, all that.


Refactor translateRoute by extracting two functions to process redire…

Please provide a description of this PR:
Extract functions out to reduce its complexity and to achieve better maintainability

@hzxuzhonghu: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
6ce981ffeeccf83d634175dc25a7bcbcb2699ca9
link
true
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

cc @howardjohn minor change

@ramaraochavali

Sorry
I am OOO for next 3 days
Will take a look when I am back if John does not get to it by then
On Mon, 10 Jan 2022, 08:58 Zhonghu Xu, @.***> wrote:

@ramaraochavali https://github.com/ramaraochavali
—
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/36648#issuecomment-1008515848, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEDINKHVBXLNBHHLR6U64CLUVJG5JANCNFSM5K32KVYQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

ping @ramaraochavali, suppose you are back

NP, thank you


(maint) Do not rely on external networking for WSASetup validation

For FACT-1551, we added a test to ensure we had called WSASetup before
trying to resolve Ruby facts. As originally implemented, this test
required public network access and a functional external resource.
Most of this is not necessary to validate that WSA has been setup, and
makes the test more fragile. Instead, we can simply ensure that
creating and closing a socket proceeds without error.

This is targeted at LTS, and should be merged up as we go

Put up another commit on top to comment out the fix related to this test, and let's see if it gets through AppVeyor.

@MikaelSmith Sounds like a good plan. I want to see this go green as-is in appveyor/travis first, since I had some local failures that I think (hope) are related to our tests not being run on Arch all the time.

Hmph, that's not good.

OK, so it looks like despite Microsoft's docs, simply opening a socket isn't enough to fail this stuff. I'm gonna try some more things

Fingers crossed that take three here will do the trick :fearful:

I think it might specifically be the hostname resolution that requires WSAStartup.

@MikaelSmith THAT'S NOT WHAT MICROSOFT'S DOCS SAY, but I would totally believe it, and that makes this more of a pain :(

I'm gonna close this for now - it's not turning out to be the quick fix I hoped it was, and it's not all that important. I'll play with it again another day.


Improve error handling of build in commands

Hello, nice project but it has some rough edges (understandably). I'm very used to cd by itself meaning cd ~, but when I tried that on snakeshell I got IndexError: list index out of range and the shell exited.
Even if cd to home is not the default behavior, missing arguments should not make it crash completely.

Hello!
Thank you very much for the feedback! You’re right — cd without arguments is a very common command, but I somehow overlooked this behavior. I’ll fix the issue and add a couple of tests to cover this case.
I’ll also review and fix other built-in commands to ensure they don’t face similar issues and prevent any crashes!

Thanks again for the feedback!
I’ve fixed the issue. The shell is now more stable, and I’ve added error messages for the built-in commands:

cd now goes to the home directory if no arguments are provided.
Errors for missing or invalid arguments no longer cause the shell to crash.

Let me know if you find any other problems!


chore: fix CI by including _book directory

What did you implement:
The previous build failed because:
https://circleci.com/gh/getndazn/dazn-lambda-powertools/1056
In order to push the contents of the directory to gh-pages branch, the directory must be known to github (See this issue on StackOverflow) and the contents not ignored (otherwise the ignored files are not pushed). This means that we also have to temporarily add and commit (in CI) the new book files, before pushing. Frustratingly, Github also doesn't allow you to commit empty directories, so I've added an empty file to the directory (See this issue on StackOverflow)
After running locally this appears to be working: https://getndazn.github.io/dazn-lambda-powertools/

How did you implement it:
As above.

How can we verify it:
Unsure, I think only with push to master

Todos:

[ ] Write tests
[ ] Write documentation
[ ] Fix linting errors
[ ] Make sure code coverage hasn't dropped
[ ] Provide verification config / projects / resources
[ ] Enable "Allow edits from maintainers" for this PR
[ ] Update the messages below

Is this ready for review?: YES
Is it a breaking change?: NO

Thanks for the review @theburningmonk — I've actually since updated the PR if you wanted to take a look. I really wanted to avoid force pushing, but since we're mangling history it seems there's no true idempotent way to get around the problem. The current implementation (whilst not very elegant) at least seems to work repeatably. You can run on your own machine to verify.

@loujaybee it looks good

@gsingh1 we need to add a white background to the image though, it's hard to see the lines on the docs

@theburningmonk agree'd — I wanted to get the docs up first, now I'll go through and make those additional changes in subsequent PR's, thanks for the quick reviews! ❤


Logic App Standard - Run Http Trigger

Library name and version
Azure.ResourceManager.AppService 1.0.2
Query/Question
I want to run HttpTrigger in Logic App (Standard) but I don't find correct way.
I want to get Workflow URL and than trigger my HTTP Trigger

Environment
dotnet 8

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Hi @andreatosato , Thank you for using Azure SDK for .NET.
Since we don't have much domain on a single resource, I can only assume that your senario is:
    var app = (await resourceGroup.GetLogicWorkflows().GetAsync("name")).Value;
    var trigger = (await app.GetLogicWorkflowTriggerAsync("name")).Value;
    var url = (await trigger.GetCallbackUrlAsync()).Value;
    await trigger.RunAsync();

You could have a try on this.

@HarveyLink work for LogicApp Consumptions, not for LogicApp Standard.

Hi @andreatosato , thank you for the clarification, the API is available from 2022-03-01 version, our current AppService SDK is 2021-02-01 version. You could help to open an Azure support request to push WebSite team bump their SDK version.


Cannot assign requested address exception

Issue Title
"Cannot assign requested address" during communication between two services.
General
I have a set of services (core 2.1.5) which are launched with Docker Swarm. For two of them, I have communication issue after some time after startup.
I don't have issues the others services.
So basically, I have :
Service A -> Service B
At startup, everything is fine.
After some hours, I receive this exception.
If I restart the Service container, it works during few hours...
Exception
{
  "Depth": 0,
  "ClassName": "",
  "Message": "Cannot assign requested address",
  "Source": "System.Net.Http",
  "StackTraceString": "   at System.Net.Http.ConnectHelper.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)
at System.Threading.Tasks.ValueTask`1.get_Result()
at System.Net.Http.HttpConnectionPool.CreateConnectionAsync(HttpRequestMessage request, CancellationToken cancellationToken)
at System.Threading.Tasks.ValueTask`1.get_Result()
at System.Net.Http.HttpConnectionPool.WaitForCreatedConnectionAsync(ValueTask`1 creationTask)
at System.Threading.Tasks.ValueTask`1.get_Result()
at System.Net.Http.HttpConnectionPool.SendWithRetryAsync(HttpRequestMessage request, Boolean doRequestAuth, CancellationToken cancellationToken)
at System.Net.Http.RedirectHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
at System.Net.Http.HttpClient.FinishSendAsyncUnbuffered(Task`1 sendTask, HttpRequestMessage request, CancellationTokenSource cts, Boolean disposeCts)
at ServiceClients.TransportProcessingServiceClient.TransportProcessingServiceClient.PostFileAsync(FileContentViewModel fileContentViewModel, Guid transportConfigurationId, Guid retailerId, Guid contextId, CancellationToken cancellationToken, ILogger logger)
at TriggerManagementService.Infrastructure.Executors.ExportTradeItemImageActionExecutor.DoExecute(ExportTradeItemImageAction action) in /src/TriggerManagementService/Infrastructure/Executors/ExportTradeItemImageActionExecutor.cs:line 92
at TriggerManagementService.Infrastructure.Executors.ActionExecutor`2.Execute(T action, Dictionary`2 parameters) in /src/TriggerManagementService/Infrastructure/Executors/ActionExecutor.cs:line 49",
  "RemoteStackTraceString": "",
  "RemoteStackIndex": -1,
  "HResult": -2147467259,
  "HelpURL": null
},
{
  "Depth": 1,
  "ClassName": "",
  "Message": "Cannot assign requested address",
  "Source": "System.Private.CoreLib",
  "StackTraceString": "   at System.Net.Http.ConnectHelper.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)",
  "RemoteStackTraceString": "",
  "RemoteStackIndex": -1,
  "HResult": -2147467259,
  "HelpURL": null
}

Which OS do you run on?
It seems like there are no available ports on the system. Can you check the OS stats at the point of failure? Can you check your container configuration?
Once it happens (after few hours) - does it happen once / all the time / most of the time?

Sorry my mistake I found the issue. The service sent a stream as a response and I did not close it.


Reek/Goto definition support

I have checked that this feature is not already implemented

[X] This feature does not exist

Use case
I'm currently using a bunch of VSCode extensions to support ruby coding, including Solargraph. I'd love to switch to only have ruby-lsp and it's extension which are great, however I'm missing a couple of key features:

goto definition
reek support (https://github.com/troessner/reek)

Description
I'd love to have reek reporting code smells, in a clean way so that it's more like a suggestion than an error. I'd also love to be able to CMD/Control click a class name/method name to go to it's definition.
Implementation
.

Go to definition is tracked by #199.
Regarding adding Reek support, that would need to be done using the Ruby LSP server extensions systems that we're currently working on. Once we finish putting the mechanism in place, Reek can export a Ruby LSP extension to add extra diagnostics (or a separate ruby-lsp-reek gem can be created).

For anyone following this issue RE: reek support, this add-on now exists: https://github.com/igray/ruby-lsp-reek

@valerii-pohrebniak-rg please move the discussion to ruby-lsp-reek's issue, thanks 🙂


Use Typescript Parameter Properties

See description below, this can allow us to clean-up our code quite a bit, https://www.typescriptlang.org/docs/handbook/2/classes.html#parameter-properties

Parameter Properties
TypeScript offers special syntax for turning a constructor parameter into a class property with the same name and value. These are called parameter properties and are created by prefixing a constructor argument with one of the visibility modifiers public, private, protected, or readonly. The resulting field gets those modifier(s):
class Params {
  constructor(
    public readonly x: number,
    protected y: number,
    private z: number
  ) {
    // No body necessary
  }
}
const a = new Params(1, 2, 3);
console.log(a.x);

I've been using this pattern for a while and it's great, I encourage other maintainers to try it out.

Adoption is well underway in the lodestar repo


Feature: Attributes for excluding/including

Hi,
it would be very useful if one can decorate properties/classes with attributes to exclude or include them in audit trails. The attribute could also be used to define DisplayName for the property or entity.
Regards
Sven

Thank you for this great suggestion.

Hello @sven5 ,
The Exclude/Include attribute is currently under development,
There is still some work to do, but so far, we very happen with this new changes
It should be available on Monday. I will let you know as soon as the new version is ready.
Best Regards,
Jonathan

Hi Jonathan,
That's great news.
Thanks

Hello @sven5 ,
The v1.4.10 is now available.
However, Exclude/Include DataAnnotation are not automatically added since they may be a little bit "time consuming."
I still want to improve this feature, but so far you can achieve the result like this:
Audit Exclude
[AuditExclude]
public class Entity
{
}

audit.Configuration.ExcludeDataAnnotation();

Audit Include
[AuditInclude]
public class Entity
{
}

// Exclude All
audit.Configuration.Exclude(o => true);
audit.Configuration.IncludeDataAnnotation();

Let me know if this solution works for you and if you have a better idea how to name these methods ;)
Best Regards,
Jonathan

Closing: Request Done


Add API and implementation for adding and removing MV worlds without loading/unloading.

Other plugins may wish to add MV worlds which they have loading using a custom world loader to the MV config, and later remove them.
They should be able to do this without causing Mutliverse to attempt to load them.

outdated PR


const-random should be listed as optional.

Bug description
const-random is not required for default/std feature but it is listed as required dependency always.

Would you like to work on a fix? [y/n]
N

Hm I don't think there's a way in Cargo to require the dependency if the std feature is turned off. If it was optional, how do you propose it'll be enabled for no std users? Only option I see is to have another no_std flag that is meant to be mutually exclusive with std.

I think so, or add no_std feature and remove std feature?
Some company (e.g. Microsoft...), will scan all packages in the dependency list, const-random has some dependency which contains encryption algorithm. All such packages are not allowed to use except special approval.

@sgodwincs  I'd like to contribute this fix.
I think removing the std feature and adding a no_std feature feels like a good path forward, as the user must "opt-in" to no_std usage.  If that sounds good to you I can get a PR going.

I can open a PR for this.
Having a std feature seems to be in line with what the Cargo docs recommend.  In that vein, I think the best we can do to avoid breaking existing users of default feature std is to:

make const-random optional
add a new feature const-random that brings in the dependency
add a compile error to the code if std and const-random features are both enabled (cargo doc reference)

It's not perfect, but I think it achieves the goal of eliminating the const-random dependency while not breaking existing users of default features; though no_std users will need to add the new const-random feature.
@sgodwincs thoughts on that?

Sounds reasonable to me. Feel free to send a PR.

Sounds reasonable to me. Feel free to send a PR.

Done! #23


[DO NOT MERGE] Auto PR to update dependencies of istio

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
904a5462c611b25d810ea8d098630a3cca9b21a0
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
904a5462c611b25d810ea8d098630a3cca9b21a0
link
/test istio-unit-tests

prow/istio-presubmit.sh
904a5462c611b25d810ea8d098630a3cca9b21a0
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


First and Second modules completed

@szikszail Could you please review my task? Thank you!

Thank you for your solution!


ACL: allow to add zero price level, so members can switch for free

Now membership level can't be changed by user, allow to create zero price level period by studio operator so users can switch to it if such option created by operator.
Please comment if you have any other suggestion.

@AntonLV I've chosen free Premium level for 7 days, but got it forever:

Fixed here
https://github.com/unaio/una/commit/95867a6e05406d9d03f952d18e7ca6a893663d50


React Native Error: The operation couldn’t be completed. (file not found error -15)

Hello, I am trying to retrieve metadata from video file in react-native app, but I always get this error: Error: The operation couldn’t be completed. (file not found error -15). Could you please advise how I can fix it?

You have to download the video file first. Can only use local files.

getting this too, the file is local


feat: norm estimates for various operators in linear algebra

There is a technical issue: I need to register two local instances to be even able to state the norm estimates. The issue is typeclass inference getting stuck in complicated types of linear maps...

I've created an issue in the core repo about this. Can you reference it in the comments? If it ever gets resolved, we can streamline here.

I've referenced the issue in the comments.

Thanks!
bors delegate+

bors r+


unable to do it can u make the instructions a bit more elaborate

unable to run the program i need the detailed instruction guide
where to place these files
all the required files
the exact procedure
i need to unlock my nova3i ine-lx1r

Hi,
in step 4 is everything you need to do. Just clone the repo, cd into and run python unlock

Hi,
in step 4 is everything you need to do. Just clone the repo, cd into and run python unlock
Can you describe your problem more detailed?

i dont know how to do it can u give me in detail how to do it step by step or some video as reference that is the problem

what and how am i supposed to do

when ever i try to open it ,it just opens fast then loads and closes itself

after fixing the previous error now i get this error
image link for the error below

can you please try again? I fixed the initial json file


_parse_test_output failes

I did a little research into this but I haven't found the issue yet. I'll update this when I learn more.
It seems that the parser can't find tests in the file
neotest-minitest: /home/user/.local/share/nvim/lazy/neotest/lua/nio/init.lua:105: The coroutine failed with this message:
...nvim/lazy/neotest-minitest/lua/neotest-minitest/init.lua:142: table index is nil
stack traceback:
        ...nvim/lazy/neotest-minitest/lua/neotest-minitest/init.lua: in function '_parse_test_output'
        ...nvim/lazy/neotest-minitest/lua/neotest-minitest/init.lua:177: in function 'results'
        ...al/share/nvim/lazy/neotest/lua/neotest/client/runner.lua:131: in function '_run_spec'
        ...al/share/nvim/lazy/neotest/lua/neotest/client/runner.lua:89: in function <...al/share/nvim/lazy/neotest/lua/neotest/client/runner.lua:88>

The test file is:
require "test_helper"

class CandidatureEventsControllerTest < ActionDispatch::IntegrationTest
  setup do
    sign_in(users(:one))
  end

  test "should get index" do
    get candidature_candidature_events_url(candidatures(:one))
    assert_response :success
  end
end

The parser doesn't recognise ActionDispatch::IntegrationTest
If I change it to ActiveSupport::TestCase it doesn't throw the error.
I created a minimal env and I couldn't reproduce it. I'm leaving this here in case someone else runs into this.
I can fix it by adding the treesitter parser to init.lua
    ; rails unit test classes
    ((
        class
        name: (constant) @namespace.name
        (superclass (scope_resolution) @superclass (#match? @superclass "ActionDispatch::IntegrationTest$"))
    )) @namespace.definition

here:
https://github.com/zidhuss/neotest-minitest/blob/main/lua/neotest-minitest/init.lua#L56-L61

And if I change the syntax of the test to
  # test "should get index" do
  def should_get_index_test
    get candidature_candidature_events_url(candidatures(:one))
    assert_response :success
  end

it works....

Yeah the issue was I wasn't parsing integration tests with treesitter. I resolved it in the latest commit. Thanks for your contribution!


add option to automatically link named @user/@team in Slack notifications and turn on full slack message parsing

link_names option is turned on by default as this is what most
people would expect, but can be disabled in configuration in those rare
instances when autolinking causes problems
should close: #586
cc: @juliusv
EDIT: This builds and passes rudimentary checks. I still need to  deploy it in our testing  to check how it behaves in real environment.

It works both in theory and in practice. But since no existing slack notification testing rig is in place adding one proved to a be a bit too time consuming. This is what I have now :)

Sorry for such a long delay. Can you fix your conflicts? This should be an easy merge then.

I think https://github.com/prometheus/alertmanager/pull/912 adds most of what this commit tried to do. Since it's already merged a new PR should be opened to add parse option.


Make network timeout configurable

Currently, the timeout for sending transactions is hardcoded in constants.NETWORK_TIMEOUT_MAP.
When a user sends a transaction but the network fee is set too low, the transaction can take a long time to be mined and the ocean.py function call can timeout.
This issue is to make the timeout value configurable.
Proposed solution:

Add timeout property to Config.
Specify chain-specific timeout values in ExampleConfig.CONFIG_NETWORK_HELPER.
Remove constants.NETWORK_TIMEOUT_MAP.
Add timeout property to Wallet
Update send_transaction to get timeout from from_wallet parameter.

Sounds reasonable to me.
Is there already a related pattern in ocean.js that ocean.py can mirror?
Also: if we do this now then we'll want to make sure it goes into both the V3 and V4 versions of ocean.py. (And it sounds like we do want to do this now, given the specific user's need.)

I realized I was missing a couple of important changes in my proposed solution.

Add OCEAN_TRANSACTION_TIMEOUT envvar
Add ocean_transaction_timeout config.ini option

@trentmc

Is there already a related pattern in ocean.js that ocean.py can mirror?

As far as I can tell, there is not. ocean.js does not have a transaction timeout mechanism the same way ocean.py does.
When ocean.js sends a transaction, it uses Promise and async/await but not Promise.race which would be needed to implement a Promise timeout. Thus the method that sends the transaction won't yield a fulfilled Promise until the transaction is (1) mined successfully, (2) returns an error or (3) aborted by user.
I see two options:

Proceed with my original plan to make the timeout configurable and make a timeout of 0 = "no timeout" so that it can be configured to work as ocean.js does.
Remove the timeout mechanism so that methods that send transactions never timeout and ocean.py always works as ocean.js does.

In both cases, ocean.js-like behavior is possible. (1) is more flexible/complex. (2) is less flexible but more simple.
I can't say I lean towards either option.  To me, it depends on what the users of the library would want.

@DMats towards your Q's, I sent you a Q in slack

@trentmc
My apologies, but my previous assertion was incorrect.  ocean.js does support transaction timeouts because it's built into the underlying web3.js library. When connected via HTTP, ocean.js uses the default transactionPollingTimeout value of 750 seconds (12.5 minutes) for all transactions.
This differs from web3.py which does not offer a built-in transaction timeout mechanism, hence why we implemented it in ocean.py.
In light of this, I lean heavily towards option (1) described in my previous comment.  I'll begin on an implementation shortly.


Corrected typographic errors

Corrected typographic errors: exclamation marks (¡), accented words, misplaced commas. The changes make the document more presentable. No changes were made to the translation.

[x] Have you followed the contributing guidelines?
[x] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Xoá phím tắt


'Active blobs/Active and deleted blobs' are missed on the dropdown of container editor

Storage Explorer Version: 1.9.0
Branch: master
Build: 20190809.2
Platform/OS: Linux Ubuntu 19.04/Windows 10/macOS High Sierra
Architecture: ia32/x64
Regression From: Previous release 1.9.0
Steps to reproduce:

Expand one normal account -> 'Blob Containers'.
Open one blob container -> Check the 'Active blobs/Active and deleted blobs' dropdown on the opened editor.

Expect Experience:
'Active blobs/Active and deleted blobs' shows on the dropdown of the opened editor.
Actual Experience:
'Active blobs/Active and deleted blobs' are missed on the dropdown.

This issue has been fixed on the latest master build 20190813.12. So closed it.


[CS2103T-W16-3] StudyBuddy

The pull request for Team W16-3

Please retitle the PR with the correct format.

Section 3.1.2

Small typo in "list of tasks due soon will b e initialized"

Section 3.2.1

Small typo in "displays a calendar fo users"

Generally clear and understandable, but perhaps having at least one class diagram for your modules such as Calendar or the Due Soon Task feature would enhance the comprehensiveness of your DG :)


addon observability doesn't have an icon

It needs to be defined here.
https://github.com/oam-dev/catalog/blob/master/addons/observability/metadata.yaml


Fix render

Changes made:

Hierarchy of app has changed, parent is now App with vovager/graphiql as children. In the same file, (index.js) it was changed from index.tsx -> index.js

App holds functionality to display voyager

Webpack is now targeting index.js as the entry point

Parent Container will now latch on to div element with id of "root" in html file.

In src/visualizer/index.tsx, added /schema to line 5. Voyager will currently not display if this is removed.

We need to add /schema to line 6 in the demo/server.js.


[Bug]: Issue running actions when unmarshalling response on external authentication with Okta

Preflight Checklist

[X] I could not find a solution in the documentation, the existing issues or discussions
[X] I have joined the ZITADEL chat

Environment
Self-hosted
Version
v2.41.6
Database
PostgreSQL
Database Version
14
Describe the problem caused by this bug
Okta Workforce Identity has been registered as an IDP provider. Automatic user creation was enabled in the IDP provider configuration, so it is expected to get the user created automatically. As some fields cannot be obtained automatically from Okta data provided for zitadel, an action was needed to map the fields accordingly to Okta data (covered in this issue https://github.com/zitadel/zitadel/issues/7038 ).
This flow works fine for one of the Okta users, but for another one, the action execution fails. The action is configured as External Authentication -> Post Authentication. Printing the error with zitadel logger using this:
logger.log("ctx", JSON.stringify(ctx.v1.authError));
Gives us this output:
time="2024-01-22T16:05:50Z" level=info msg="log record emitted" caller="/home/runner/work/zitadel/zitadel/internal/logstore/emitters/stdout/stdout.go:19" record="{\"logDate\":\"2024-01-22T16:05:50.438418241Z\",\"took\":0,\"message\":\"ctx \\\"failed to unmarshal response: oidc: language: subtag \\\\\\\"us\\\\\\\" is well-formed but unknown into *oidc.uiAlias {\\\\\\\"sub\\\\\\\":\\\\\\\"00u1ppl2fDdImw6bQ000\\\\\\\",\\\\\\\"name\\\\\\\":\\\\\\\"John Doe\\\\\\\",\\\\\\\"locale\\\\\\\":\\\\\\\"US\\\\\\\",\\\\\\\"email\\\\\\\":\\\\\\\"hello@okta.com\\\\\\\",\\\\\\\"nickname\\\\\\\":\\\\\\\"\\\\\\\",\\\\\\\"preferred_username\\\\\\\":\\\\\\\"hello@okta.com\\\\\\\",\\\\\\\"given_name\\\\\\\":\\\\\\\"Joe\\\\\\\",\\\\\\\"middle_name\\\\\\\":\\\\\\\"\\\\\\\",\\\\\\\"family_name\\\\\\\":\\\\\\\"Doe\\\\\\\",\\\\\\\"zoneinfo\\\\\\\":\\\\\\\"America/Los_Angeles\\\\\\\",\\\\\\\"updated_at\\\\\\\":1702999806,\\\\\\\"email_verified\\\\\\\":true}\\\"\",\"logLevel\":\"info\",\"instanceId\":\"244777729604452470\"}"

It looks like the raw data coming from Okta contains some information (us) that breaks the unmarshalling of such information into zitadel. The key portion of the message is this:
failed to unmarshal response: oidc: language: subtag "us" is well-formed but unknown into *oidc.uiAlias
UPDATE: Here's another log that shows the line where the error is thrown:
time="2024-01-22T18:20:04Z" level=error msg="both external user authentication and action post authentication failed" caller="/home/runner/work/zitadel/zitadel/internal/api/ui/login/external_provider_handler.go:1082" error="failed to unmarshal response: oidc: language: subtag \"us\" is well-formed but unknown into *oidc.uiAlias {\"sub\":\"00u1ppl2fDdImw6bQ696\",\"name\":\"John Doe\",\"locale\":\"US\",\"email\":\"jdoe@okta.com\",\"nickname\":\"\",\"preferred_username\":\"jdoe@okta.com\",\"given_name\":\"John\",\"middle_name\":\"\",\"family_name\":\"Doe\",\"zoneinfo\":\"America/Los_Angeles\",\"updated_at\":1702999806,\"email_verified\":true}"

https://github.com/zitadel/zitadel/blob/f9f9573e516f495866507b4a614290d7a933b20b/internal/api/ui/login/external_provider_handler.go#L1082
To reproduce

Configure Okta Workforce Identity as an IDP provider
Configure an action that tries to auto create user in External Authentication -> Post Authentication with code like this:

let logger = require("zitadel/log")

function mapOktaOauth(ctx, api) {
  logger.log(ctx.v1.authError);
  api.setFirstName(ctx.v1.providerInfo.name);
  api.setLastName(ctx.v1.providerInfo.name);
  api.setEmailVerified(ctx.v1.providerInfo.email)
  api.setEmail(ctx.v1.providerInfo.email)
  api.setPreferredUsername(ctx.v1.providerInfo.preferred_username)
}

try to login with your Okta account
Not sure about what piece of information makes unmarshalling to fail

Screenshots
No response
Expected behavior
Not getting the error described above
Operating System
No response
Relevant Configuration
No response
Additional Context
No response

Hello, sorry for bugging you, but, are there any updates/findings regarding this? We are relying on keeping one of our clients by making sure Okta authentication through zitadel works fine for all users, and some users are able to login while others experience the issue.

Hey @oscarcalderonc
Sorry for the late response. We've fixes this some time ago, but had to figure out on which versions.
Can you please upgrade to 2.41.14. This should solve the issue.

@livio-a thank you very much, that did the trick.


Draw image/text with current transformation appied

The rotated image looks good, but the rotated text looks very aliased. Can it be improved?

@wsw0108 See my comment above? :)

@fogleman oh, I almost forgot this.
I think this should be done by get the vector path of each glyph of the input text, then draw the vector path instead of image.
For now, package golang.org/x/image/font only provide method that get image of glyph. Although package github.com/golang/freetype can be used to get vector path of glyph, it can only process truetype font.


[WIP] Print log to terminal if option selected or if not much text

The way Watson currently behaves it's quite difficult to edit frames by hash: to get the hash of a frame AND enough context to figure out what that frame is about you have to type in 'watson log'; but this brings up a pager and when you close this pager all of the log information vanishes, making it hard to remember the hash number to do a 'watson edit hash_here'.
It would be much easier if there was an option to print the log to the terminal. So I added this option. I also made it so that if the log text can fit in the terminal window, then it should print to the terminal (not the pager); this is how 'git log' works.
To determine the terminal size I used the 'stty size' command over a popen, as recommended here: https://stackoverflow.com/questions/566746/how-to-get-linux-console-window-width-in-python. Apparently this command is widely supported.

I just realized that len(lines) didn't do what I thought it did. I need to fix this. I will rebase and comment when this is fixed.

Likely not going to have time to work on this for the next while.

Ok then. Do not hesitate to re-open this PR if you have spare time @mwiens91!


Use resizable panes

This looks pretty solid.
One minor issue: Story list (controls) is not scrollable.
Once we fixed it, we could take this in.

Fixed scrolling


Update deployments link helper domain names

Description

Closes #
What does this PR change? Give us a brief description.

Thank you for submitting your Pull Request, the following links will become available for preview shortly:

StudioCMS Landing Page

StudioCMS Documentation

StudioCMS Demo (node-playground) (Also deployed with package changes)

StudioCMS Ui Demo (ui-playground)


WinPower custom installation video

It would be easier for new users to learn how to install, troubleshoot and how to use WinPower.
Can you make a video preferably in English language demonstrating how to install winpower, how to validate the installation how commands can be used?

NO REPONSE FROM THE CONTRIBUTOR. CLOSING THE ISSUE.


Update OSP prod-m01

Dev and Stage already verified.
Bumping prod-m01 in preparation for bumping all prod

/lgtm


Keyboard Input triggers onSelection action

When pikaday input is open and a user uses a keyboard input (up, down, left, right arrow), it triggers the onSelection action even if no date has been selected via enter

confirmed.. happening to me as well

You just need to update to 2.2.4 and pass that parameter:
    {{pikaday-inputless
      options=(hash keyboardInput=false)
    }}

Seeing as there's a solution documented above, I'm going to close this.
Thanks for the fix, @IAmJulianAcosta !


Connection reset by server

Sometimes when you generate test rewards, you get a 200 return with an error of "connection reset by server".
If you wait a little bit, it will work.  So I changed the message the user sees for now.

This doesn't always happen though..

Back end has been totally reworked..
https://github.com/FreeLoaderz/rwd-frontend/pull/85


embed workflow version number

related #8

hey @konoui ... strange, I got a notification the other day about a 0.6.1 version being released, but now I can't see any evidence of it ever existing. Was that me temporarily transporting to a parallel universe 🚀  👽  or was there a problem with that release?
thanks

Hi @luckman212 ,
v0.6.1 was disappeared 👽.
I failed to release the version. I am fixing the release problem.
Next release will be success!!

👽 sounds good

v0.6.1 is just released 👽👽.