[jest-each]: Add support for keyPaths in test titles

Summary
Closes #6452
Opening this up for review early as I want @SimenB @thymikee @rickhanlonii opinion on how to share the getPath logic that is defined in expect.
Add support for $key.path syntax in tagged template test titles.
Example:
// Actions
const increment = () => ({type: 'increment'});
const decrement = () => ({type: 'decrement'});

// Reducer
const reducer = (state = 0, {type}) => {
  switch (type) {
    case 'increment':
      return state + 1;
    case 'decrement':
      return state - 1;
    default:
      return state;
  }
};

test.each`
  state | action         | nextState
  ${0}  | ${increment()} | ${1}
  ${0}  | ${decrement()} | ${-1}
`(
  'returns $nextState when the state is $state and $action.type is dispatched $nextState',
  ({state, action, nextState}) => {
    expect(reducer(state, action)).toEqual(nextState);
  },
);

Test plan
See unit tests
TODO:

[x] Changelog
[x] Update docs
[x] Some how share the getPath logic with expect

As part of this is fine

@SimenB have you seen an eslint error like this before?
https://circleci.com/gh/facebook/jest/25790?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link
Not sure why it going wrong 😕

Syntax error in the browserbuild, which must be es5

Hmm @SimenB so it looks like expect cannot depend on jest-util then? How else can we share the getPath code?

Codecov Report

Merging #6457 into master will increase coverage by 0.13%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #6457      +/-   ##
==========================================
+ Coverage   63.48%   63.61%   +0.13%
==========================================
  Files         227      228       +1
  Lines        8697     8730      +33
  Branches        4        3       -1
==========================================
+ Hits         5521     5554      +33
  Misses       3175     3175
  Partials        1        1

Impacted Files
Coverage Δ

packages/jest-util/src/get_path.js
100% <100%> (ø)

packages/jest-each/src/bind.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@SimenB I've updated this to inline a much simpler getPath function so no need to add anything to jest-util


Config/4 docker image

Description
Added image metadata and fixed databse connection command not working by using mongoose
Changes
copilot:walkthrough

Changes
copilot:walkthrough


support: Show date for start of next billing cycle for current plan.

Instead of showing the next invoice date for the plan, show the date for the next billing cycle start (e.g. the next plan renewal charge date), except for plans currently on a free trial.
For plans on a free trial, the next plan renewal date will be when the free trial ends, which is stored as the next invoice date on the plan.
These changes will apply to both Cloud and remote support views.
Screenshots and screen captures:

Basic remote realm

Free trial remote realm
Note that the free trial end date is set to 2 months in populate_billing_realms for the dev environment.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @laurynmm!


Extend functionality

Stability: Move puppeteer to flow context to resolve crashes
New functionality:
More launch options
cookie, title, useragent, viewport handling
This pull request in this form is not intended for direct merge. Needs cleanup.
Before cleaning up however I would like to clarify with you possible changes needed for you to accept it.
Hope to hear from you!

Hi @Schischu, very nice PR.
Could you make a change to support executablePath launch option as well please?
Thanks :)


The uploader has not made this video available in your country

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.11.23. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x ] I've verified and I assure that I'm running youtube-dl 2018.11.23

Before submitting an issue make sure you have:

[x ] At least skimmed through the README, most notably the FAQ and BUGS sections
[x ] Searched the bugtracker for similar issues including closed ones
[x ] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
root@vps:~/test# youtube-dl -v s3O1Xro7oAI
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u's3O1Xro7oAI']
WARNING: Assuming --restrict-filenames since file system encoding cannot encode                                                                                all characters. Set the LC_ALL environment variable to fix this.
[debug] Encodings: locale ANSI_X3.4-1968, fs ANSI_X3.4-1968, out ANSI_X3.4-1968,                                                                                pref ANSI_X3.4-1968
[debug] youtube-dl version 2018.11.23
[debug] Python version 2.7.13 (CPython) - Linux-4.9.0-8-amd64-x86_64-with-debian                                                                                -9.6
[debug] exe versions: ffmpeg 3.2.12-1, ffprobe 3.2.12-1
[debug] Proxy map: {}
[youtube] s3O1Xro7oAI: Downloading webpage
[youtube] s3O1Xro7oAI: Downloading video info webpage
ERROR: The uploader has not made this video available in your country.
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 79                                                                                2, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/common.py",                                                                                line 508, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py",                                                                                line 1920, in _real_extract
    raise ExtractorError(error_message, expected=True)
ExtractorError: The uploader has not made this video available in your country.

...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=s3O1Xro7oAI

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Hello !
I have a vps in France (Gravelines) : https://www.ovh.com/world/vps/vps-cloud.xml VPS Cloud 2
I can download a lot of videos. But I don't know why, I can't download some videos like this : https://www.youtube.com/watch?v=s3O1Xro7oAI
I have the error " The uploader has not made this video available in your country"
I really don't know why. My vps location is in France...
My ip :
root@vps:~/test# curl yt-dl.org/ip
<IP_ADDRESS>
I know that I can use a proxy, but the speed is very slow. And I want to know why I'm blocked
I'm on debian 9 so I don't know if it's okay with a browser
Thank you

the error message is self-explanatory.

@remitamine instead of commenting as such , suggest a workaround, yes?

@remitamine instead of commenting as such , suggest a workaround, yes?


Refactor metadata text extraction logic

Summary
This provides some cleaner abstractions for making generic transformations of content trees and updates the logic for grabbing the first paragraph of text for use as metadata descriptions to work better with varying author content.
Specific changes:

adds a new reduce method to ContentNode
adds a new plaintext computed property to ContentNode
adds a new firstParagraph string utility
updates the extractText mixin method to convert the content to plaintext and only use the first paragraph

\cc @dobromir-hristov
Dependencies
https://github.com/apple/swift-docc-render/pull/46

Very interesting solution, thank you very much for doing it! 💯
Can you explain a bit better why plaintext is a computed property instead of method and we would pass content to it?

ContentNode already has a content prop, so there is no need to also pass the data—it already has it. I wrote it as a computed property for that reason since it is just a value computed directly from the tree.

Very interesting solution, thank you very much for doing it! 💯
Can you explain a bit better why plaintext is a computed property instead of method and we would pass content to it?

ContentNode already has a content prop, so there is no need to also pass the data—it already has it. I wrote it as a computed property for that reason since it is just a value computed directly from the tree.

Sounds great. Thanks for the refactor, Marcus!


CloudOps / Azure AKS Hackfest in MTL & Quebec City Fixed & Improvements

Hi Guys,
It's Adrian, we chatted with Kevin Harris on the phone today. Those are the changes that I would like to make to the hackfest for MTL and Quebec City, as if you don't make them, you won't be able to go past challenge 03 due to the "underscore" restriction when you create the AKS cluster.
There are other changes as well, you could use the Kubernetes UI via an RDP session for example. I put commands on how to do that fast using 'jq' and the Azure CLI.
Let me know if you have any questions.
Best regards,
Adrian Todorov
Container Solution Architect @ CloudOps
<EMAIL_ADDRESS>

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


Document using virtuals.

Setting up a virtual currently does nothing to the output  This is currently enabled separately in Mongoose using the following commands:
// Create a virtual field for id.
schema.virtual('id').get(function(){
    return this._id.toHexString();
});
// Ensure virtual fields are serialised.
schema.set('toJSON', {virtuals: true});
schema.set('toObject', {virtuals: true});

@marshallswain Again, I think this is more of a documentation issue. If you are working with the mongoose model inside a hook you do have access to the virtuals. If you want virtuals included in your output I think you should have to specify that explicitly like you did above. That's really part of mongoose.

@marshallswain thanks mate!

Hi,
The documentation about mongoose's virtuals is not more abailable. What is the suggested way to work virtuals?
I've found that specfiyng lean = false and doing the aforementioned it makes this to work

seem like
// Create a virtual field for id. schema.virtual('id').get(function(){     return this._id.toHexString(); }); // Ensure virtual fields are serialised. schema.set('toJSON', {virtuals: true}); schema.set('toObject', {virtuals: true});
still doesn't work. the output contain no id, only _id. any other options i need to set beforehand?

// Create a virtual field for id.
schema.virtual("id").get(function() {
return this._id.toHexString();
});
schema.set("toJSON", { virtuals: true });
schema.set("toObject", { virtuals: true });
front end is ReactJS with Redux
With this setup, I realize when i service().create() from client to server, the id is in the response. but when service().find() the id is not there, only _id
I did most things accordingly, curious what's wrong or am i missing out anything?

// Create a virtual field for id. schema.virtual("id").get(function() { return this._id.toHexString(); });
schema.set("toJSON", { virtuals: true }); schema.set("toObject", { virtuals: true });
front end is ReactJS with Redux
With this setup, I realize when i service().create() from client to server, the id is in the response. but when service().find() the id is not there, only _id
I did most things accordingly, curious what's wrong or am i missing out anything?

Solution ->>>
The problem is the "lean" option in mongoose queries. Feathers.JS has lean = "true" as default (this can be easily seen in the "feathers-mongoose" codebase). More info: https://mongoosejs.com/docs/6.x/docs/api/query.html#query_Query-lean.
solutions:
use a plugin or
in the .service.js of your service set the option -> lean: false in the object


Cannot add custom selector

Subject of the issue
Can one enable custom selectors? The Docs mention three additions : ":blank", ":filled" and ":unchecked", but what if others are required? I need to check if a selected value is not empty, but is not a specific default Value coming from the backend. Therefore, setting the value = "" (like the Docs suggest https://jqueryvalidation.org/required-method/ ) is not an option.
Your environment

jquery-validate 1.18.0

Steps to reproduce

<form class="form" id="aForm" method="get" action="">
    <select id="s1" name="s1" class="form-control" size="1"
       data-msg-required="Selection missing!">
         <option value="000">Please choose</option>
         <option value="001">apples</option>
         <option value="002">pies</option>
     </select>

<input class="submit" type="submit" value="Submit">

</form>

JQuery Selector itself works as expected:
https://jsfiddle.net/M3ssman/xpvt214o/851013/
but, unfortunately , I don't get it along with the jquery-validator
Expected behaviour
Signal missing Selection.
Actual behaviour
Custom selector is not applied. Somewhere down in jquery-validation it get swallowed inside prototype.depend. Looks like it ain't used at this point.

Thanks a lot for filling this issue! We'll triage and take a look at it as soon as possible!

The latest version has been able to make it.
When you configure “required”, you have three ways：
1.boolean
for example,required:true;it means the input element is required,You should enter something。
2.string
for example,required: "#other:checked";it means if this element exists($("#other:checked").length >0), the input element is required.
3.function
for example ,
required: function(element) {
return $("#age").val() < 13;
}
it means whether it is required depends on the return value of this function。
as for your question,i think you can use method 3.


updated init scripts

Looks good to me. Same comment about shell clean-up (esp. shellcheck) and posix compatibility. Ship it.


Plugin API refactor

This PR:

includes a small refactor: Moves the checks if a plugin ref is valid into a separate function
adds a check if the requested plugin is active
fixes the lookup for settings. Currently, we are looking at SETTINGS - which refers to the static definition of settings. A plugin can also define settings at runtime so we need to lookup settings.

Sorry @matmair this one slipped under my radar. Please see my review comment

@matmair fair enough. Please address failing CI and I'll merge

@matmair CI is passing but conflicts in tests.py?

@matmair CI is passing but conflicts in tests.py?

Those are new; will take a look now

@SchrodingersGat fixed the conflict; no real code changes so should go through


The AzureADHybridAuthenticationManagement module appears to not be supported on Server 2022

While the Prerequisites section has a statement about supported domain controllers, there is no mention that the required Azure AD Kerberos PowerShell (AzureADHybridAuthenticationManagement) module appears to not be supported on Server 2022. The module installs and imports without error, however attempting to run the cmdlets yields an exception.
PS C:\> Install-Module -Name AzureADHybridAuthenticationManagement -Scope AllUsers PS C:\> Import-Module AzureADHybridAuthenticationManagement PS C:\> $domain = $env:USERDNSDOMAIN PS C:\> $userPrincipalName =<EMAIL_ADDRESS>PS C:\> PS C:\> Get-AzureADKerberosServer -Domain $domain -UserPrincipalName $userPrincipalName Get-AzureADKerberosServer: Unexpected exception thrown. Action: GetKerberosDomainSyncConfigAsync, Exception: Operation is not supported on this platform.

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

ID: 50b27a01-622e-d980-83f2-1ed27060613b
Version Independent ID: e6472e15-10d7-bd4a-da02-558f2774dbaf
Content: Passwordless security key sign-in to on-premises resources - Microsoft Entra
Content Source: articles/active-directory/authentication/howto-authentication-passwordless-security-key-on-premises.md
Service: active-directory
Sub-service: authentication
GitHub Login: @Justinha
Microsoft Alias: justinha

@tdoan1
Thanks for your feedback! We will investigate and update as appropriate.

@tdoan1
I'm going to assign this to the document author so they can take a look at it accordingly
@Justinha
Can you please check and add your comments on this doc update request as applicable.

Hi @tdoan1
Got the same problem today, issue was due to the Powershell version i used to run the cmdlet.
It seems that the module AzureADHybridAuthenticationManagement is not compatible with PowerShell 7. Ok with PS 5.1.

Thanks for your feedback. We need to close all open issues at this time so we can migrate to a new Entra repository. I created a work item to have an expert review this feedback and update docs accordingly. #please-close


Need clarity on OSM governance and CLA

@phillipgibson  here are few points that OSM documents should cover:-

Page on governance hierarchy like other CNCF sandbox projects
More clarification on the corporate CLA/DCO like other sandbox projects as current one is not so clear

Thanks @mudit-01 I moved this to our current milestone to be completed.

From CNCF Slack:
Chris Aniszczyk  1:20 PM
It's up to the OSM maintainers but they need to choose between the CLA / DCO

This is what we need to update: https://github.com/openservicemesh/osm/blob/main/CONTRIBUTING.md#contributor-license-agreement
https://github.com/openservicemesh/osm-www/blob/main/CONTRIBUTING.md#contributor-license-agreement

TLDR: I think you need to open a ticket with the CNCF to get the URL to set up the CLA under CNCF.

DCO guidance has been added to the contributing.md doc, DCO bot has been added to the org level, and original CLA requirements have been removed from branch rules.

@phillipgibson the governance part is still missing like other projects have a GOVERNANCE.md. Any updates on it?

@mudit-01 the OSM repo does not contain a governance.md file. OSM is a CNCF sandbox project and falls under the CNCF governance for projects. Can you provide more specifics of what you are looking for? Thanks.

@phillipgibson I was searching for the requisites for becoming maintainers usually they are are documented governance.md

Understood. I will re-open this issue and get a draft of maintainers/contributors guidance in a governance.md file. Thanks for clarifying.

At the current state of a CNCF sandbox project we do not need the governance doc completed at this point. We will be working on drafting the governance.md doc in the near future.


✨(frontend) integrate cunningham

Purpose
Add cunningham design system to the apps, load it from lib-common.
Integrate the component DatePicker component to replace the grommet one.
Proposal

[x] ➕(frontend) add cunningham design system
[x] 💄(frontend) replace to cunningham Datepicker

Marsha (1).webm

We just need this PR on the Cunningham side to be released and all good.

I changed the retention datepicker as wel: fixup! 💄(frontend) replace to cunningham Datepicker

In LTI, if the widget is at the dashboard bottom, the date picker open under the uframe and we can't pick the date.

The component doesn't have a position prop, I am not sure how we can change it as it is now. We could eventually open an issue about it.
On the other hand, it is not really blocking because the opening of the datepicker increase the size of the viewport, by scrolling we can have access to the datepicker (see video below).
localhost_8060_development_.webm

@AntoLC I can't scroll, the iframe height is not modified and I can't see the datepicker

@AntoLC I can't scroll, the iframe height is not modified and I can't see the datepicker

From our conversation, I removed the commit about the DatePicker, only stay the Cunningham integration in this PR.


Don't enable any rulesets by default

By default, the et/open and oisf/trafficid rulesets are enabled by default. This was probably due to personal preference when first creating the container, but at most, et/open should be enabled, or nothing enabled by default which would have suricata-update default to et/open anyways.

Can you suggest a workaround on how create a manual override for these defaults?
From what you express, do you say it is not possible to disable oisf/trafficid right now, nor et/open, if one wanted to?

/var/lib/suricata is a volume, so providing your own will result in the default suricata-update behaviour, which is to use et/open if no other rulesets are enabled.  I'll probably make this the default in the git master  tag of the container and let that ripple into the next major version tag.
Also, maybe some environment variables to auto do some things for those that wish to do it that way.


ROE-209 add logging for submitting to api

JIRA link
https://companieshouse.atlassian.net/browse/ROE-209
Change description
Adding logging when the user submits the submission to log out each step, creating transaction, sending submission data to api, closing transaction, to help diagnose issues.
Also changed the createAndLogError to use the ErrorRequest function that will log the request path as well.
Work checklist

[x] Tests added where applicable
[x] UI changes meet accessibility criteria

Merge instructions
We are committed to keeping commit history clean, consistent and linear.  A BREAKING CHANGE can be part of commits of any type,
types other than fix: and feat: are allowed, for example build:, chore:, ci:, docs:, style:, refactor:, perf:, test:, and others,
footers other than BREAKING CHANGE: <description> may be provided.

will revisit this, once PR 77 is merged


tsc lib config to include MS DOM

TypeScript Version:  2.1.5
I wish I could say something like this in my tsconfig.json:
"lib": [ "es2015", "dom", "dom.microsoft"],

and then I could compile my source without error when it looks like this:
let buffer = 'the buffer you wish to copy'
static POS = !!window.clipboardData

// if IE11, copy buffer to clipboard
if (POS) { window.clipboardData.setData('Text', buffer) }
// if other browser, copy buffer to clipboard
else { Clipboard._data = buffer; document.execCommand('copy')}

#13919

Just declare the extras that you need.


Attempt at fixing #62

great. this now works!

Thank you, @aaronspring! Glad to hear that this is working. Let us know if you encounter any other issues.


Previous Rank Column

It would be nice to see the previous rank of a player instead of their peak rank.  For example on new episodes id like to know the difference from a players peak rank than the rank they actually ended with.
The previous rank could be just the previous acts ending rank.

PR here: https://github.com/zayKenyon/VALORANT-rank-yoinker/pull/153


Error in job Setup_Pluggable_Library on upgrade

When performing an upgrade from 0.6.0 to 0.7.0, the job Setup_Pluggable_Library which is exec by Load_Platform ends in failure.
Expected:
The Load_Platform job finish successfully, upgrading ADOP internals (libraries, cartridges) as needed, too.
Observed:
The Setup_Pluggable_Library job ends in failure, with the following stacktrace:

echo 'Extracting Pluggable library to additonal classpath location: /var/jenkins_home/userContent/job_dsl_additional_classpath/'
Extracting Pluggable library to additonal classpath location: /var/jenkins_home/userContent/job_dsl_additional_classpath/
cp -r src/main/groovy/pluggable/ /var/jenkins_home/userContent/job_dsl_additional_classpath/
echo '******************'

echo 'Library contents: '
Library contents:
ls /var/jenkins_home/userContent/job_dsl_additional_classpath/pluggable/scm/
PropertiesSCMProviderDataStore.groovy
SCMProvider.groovy
SCMProviderDataStore.groovy
SCMProviderFactory.groovy
SCMProviderHandler.groovy
SCMProviderInfo.groovy
bitbucket
gerrit
gitlab
helpers
ERROR: Build step failed with exception
org.jenkinsci.plugins.scriptsecurity.sandbox.RejectedAccessException: Scripts not permitted to use new jenkins.plugins.git.GitSCMSource java.lang.String java.lang.String java.lang.String java.lang.String java.lang.String boolean
at org.jenkinsci.plugins.scriptsecurity.sandbox.whitelists.StaticWhitelist.rejectNew(StaticWhitelist.java:184)
at org.jenkinsci.plugins.scriptsecurity.sandbox.groovy.SandboxInterceptor.onNewInstance(SandboxInterceptor.java:148)
at org.kohsuke.groovy.sandbox.impl.Checker$3.call(Checker.java:197)
at org.kohsuke.groovy.sandbox.impl.Checker.checkedConstructor(Checker.java:202)
at org.kohsuke.groovy.sandbox.impl.Checker$checkedConstructor$0.callStatic(Unknown Source)
at org.codehaus.groovy.runtime.callsite.CallSiteArray.defaultCallStatic(CallSiteArray.java:56)
at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callStatic(AbstractCallSite.java:194)
at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callStatic(AbstractCallSite.java:214)
at Script1.run(Script1.groovy:20)
at org.jenkinsci.plugins.scriptsecurity.sandbox.groovy.GroovySandbox.run(GroovySandbox.java:141)
at org.jenkinsci.plugins.scriptsecurity.sandbox.groovy.SecureGroovyScript.evaluate(SecureGroovyScript.java:333)
at hudson.plugins.groovy.SystemGroovy.run(SystemGroovy.java:95)
at hudson.plugins.groovy.SystemGroovy.perform(SystemGroovy.java:59)
at hudson.tasks.BuildStepMonitor$1.perform(BuildStepMonitor.java:20)
at hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:744)
at hudson.model.Build$BuildExecution.build(Build.java:206)
at hudson.model.Build$BuildExecution.doRun(Build.java:163)
at hudson.model.AbstractBuild$AbstractBuildExecution.run(AbstractBuild.java:504)
at hudson.model.Run.execute(Run.java:1727)
at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
at hudson.model.ResourceController.execute(ResourceController.java:97)
at hudson.model.Executor.run(Executor.java:429)
Build step 'Execute system Groovy script' marked build as failure
Finished: FAILURE

Note
Although this error was obtained in the context of ugprading ADOP from 0.6.0 to 0.7.0, it does not seem to be related with the upgrade scenario, but could be applicable to a clean installation, too.

Thanks @deors for raising the issue - this is due to the script security plugin not whitelisting the use of GitSCMSource. However, the ADOP Jenkins image explicitly adds this to script approval (https://github.com/Accenture/adop-jenkins/blob/d35414cb0fe404fd82a469bd56eff74d58bc823e/resources/scriptApproval.xml).
@SachinKSingh28  / @dsingh07 is this a known issue?
On a side note adop-pluggable-scm library requires versioning, similar to the work done with platform management as new adop/c have the Gitlab SCM provider.


What can be the cause for following

Following is the error:
The asynchronous action method '' returns a Task, which cannot be executed synchronously.
Server Side in ServiceEndoint:
[HttpPost]
public async Task SomeAction(SomeRequest request)
{
//
return Json(response, JsonRequestBehavior.AllowGet);
}
In Grid file:
buttons.push({
title: 'SomeAction',
cssClass: 'send-button',
            onClick: () => {
                 ContactsService.SomeAction({
                    Ids: this.rowSelection.getSelectedKeys()
                },
                    response => {
                        if (response == "Success") {
                            this.rowSelection.resetCheckedAndRefresh();
                            Q.information('Added Successfully', () => { Q.resolveUrl('#'); });
                        }
                        else {

                            Q.alert('Error');
                        }
                    },
                    { async: true } //Even tried with false
                );
            }
        });

Why don't you use a normal service endpoint method with retrieveresponse

Actually using a NuGet package which has all methods which execute in async

You may use Task.Run(() => SomeMethod).Result to execute any async method synchronusly

Used NuGet to accomplish the task


update kobalte variants

Updated the variants for kobalte to match those listed within https://github.com/kobaltedev/kobalte/tree/main/packages/tailwindcss.

thanks @Razboy20 :+1:


fix: Check if socket exists before removing

Type: Bug
Hello,
I want to apologize, but while fixing a bug on #373, i've managed to create another bug.
Removing the socket before a listen call will prevent EADDRINUSE, but blindly removing that file will throw an EADDRINUSE  error if the file does not exists.
This definitely fix errors while using unix sockets.
My sincere apologies.
Lucius.

Note:
This won't affect anyone using http|s, just socket users running it for the first time
I'm sorry.

Sure, no worries 👍  . We release another one and problem solved. Thanks for dive in into that bug.


Laboratory Work #1, Karina Melkonyan - 23FPL2

Nothing to review today.
I advise you to start working as soon as possible!
Good luck!


Add '-p, --settings-path' flags to load custom config path

Sometimes it is desirable to have your configuration in a custom location. Not only does that allow you to quickly switch between full complex configs based on need, it’s better to embed the script in other tools, when messing with the user’s home might not be desirable.

Great idea, but I can't merge this yet. With this change, the default configuration is overwritten after the option flags are dealt with and it's supposed to be the other way.
For example, if my config specifies edit="true" and I run imgur-screenshot.sh -e false it will always use edit=true.
And please don't change the version. I'll handle that  :)

Actually, it should be OK as long as --settings-path is always the first option given. But that isn’t a reasonable expectation, I agree.
I’ve been thinking of a way to fix fix, but can’t seem to find a straightforward solution. Everything that’s occurring to me is somewhat complex (but definitely doable). Do you have any idea in mind of how this could work?
Even if you describe a more complex solution, I’d be willing to work on it.

And please don't change the version. I'll handle that :)

Ah, apologies, will leave the version unchanged in the future.

Actually, it should be OK as long as --settings-path is always the first option given

Hmm, isn't load_settings called after all of the options are dealt with?
Perhaps this could be solved by first parsing all the options without doing anything immediately and then handling the options? That way we could define the order in which options are parsed, regardless of the order in which they were passed. An associative array might be helpful:
bash$ declare -A opts
bash$ opts[settings_path]="/foo/bar"
bash$ opts[edit]=true
bash$ echo ${opts[edit]}
# true
bash$ echo ${opts[settings_path]}
# /foo/bar

So you could loop over the passed options and store them in the array and then iterate over option in a certain order and check if they're in the array. Something like this:
declare -A opts
while [ ${#} != 0 ]; do
  case "${1}" in
    -h | --help)
      opts[help]="true"
      shift;;
    -p | --settings-path)
      opts[settingspath]="${2}"
      shift 2;;
     # ...
  esac
done

for option in "help" "settingspath" …; do
  if test -v opts["$option"]; then
    # do something with ${opts[$option]}
  fi
end

Hmm, isn't load_settings called after all of the options are dealt with?

Ah, yes, you’re right. I was thinking back to a previous edit I made, where load_settings would be called directly with -p.
An associative array, though a good solution (and it actually looks better organised) won’t work because it can’t be used in most OS X setups. OS X is stuck with bash 3.2 thanks to the GPL, and that’s a bash 4 feature.
Closing for now, as we can always continue the discussion, but the PR itself can’t be merged. If we find a solution, I can always submit a new one.

OS X is stuck with bash 3.2

Well thankfully there is homebrew :)
GNU bash, version 4.3.42(1)-release (x86_64-apple-darwin15.0.0)

Yes, but that still leaves most users with 3.2. Most OS X users, even if profecient with the command line, won’t necessarily upgrade. Heck, even if you can and know how, you still might chose not to, especially if you use another shell.


No velocity or acceleration on log files

I have been working with carla simulator (version 0.9.8) but have facing some problems when dealing with the log files. When I try to use get_velocity() and get_angular_velocity() from live simulation it works fine, but when I use those two functions during replaying a log file it returns 0 only and I can't get those information from vehicles. Do you have any guess about it? There is any way to overcome this problem?

Facing the same problem for 0.9.9 when enabling autopilot. It seems that once autopilot is enabled, even if you disable it later, get_velocity() still always return (0, 0, 0).

There also seems to be some problem with get_location(). Not sure what kind of problem yet. However, when I enable and disable autopilot mode alternately with some probability the location of the car seems to update only once several frames.

Hey @lucasnmatias. Yes, when replaying a log file, the physics of all the vehicle are disabled, and the don't move, but teleport around, hence why their velocity is zero. @yanlai00, same thing happens for the traffic manager with hybrid mode, when active, all physics are disabled, and the vehicle teleport around.
We recently changed the recorder and its show_recorder_file_info. start_recorder now has a parameter that, if set to true, returns all the information about the simulation, not just the one needed to replay it. There isn't just one commit that adds it so I can't point that one out, but here is the one that adds the velocity.

@glopezdiest Thanks for response! However, even if I set the attribute "role_name" of vehicle to "hero", which should disable hybrid mode on this vehicle, the problem still persists.

That depends, the hybrid mode has a radius around the ego vehicle  (the one with the hero tag). Are you checking the actors that are inside that radius, or those that are outside?

I am checking exactly the ego vehicle.
On Mon, Aug 3, 2020 at 12:38 AM glopezdiest<EMAIL_ADDRESS>wrote:

That depends, the hybrid mode has a radius around the ego vehicle (the one
with the hero tag). Are you checking the actors that are inside that
radius, or those that are outside?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/carla-simulator/carla/issues/3145#issuecomment-667861439,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI6JSZ6IV5VBYT37M2CUB2LR6ZSQDANCNFSM4PQDKAXA
.

Okay, I'll check your code in detail then, and report back

Indeed, that does seem to be the case. Tagging @jackbart94, he will know much more about the hybrid mode. Perhaps there's something we are missing

Hey @lucasnmatias. Yes, when replaying a log file, the physics of all the vehicle are disabled, and the don't move, but teleport around, hence why their velocity is zero. @yanlai00, same thing happens for the traffic manager with hybrid mode, when active, all physics are disabled, and the vehicle teleport around.
We recently changed the recorder and its show_recorder_file_info. start_recorder now has a parameter that, if set to true, returns all the information about the simulation, not just the one needed to replay it. There isn't just one commit that adds it so I can't point that one out, but here is the one that adds the velocity.

I see, another question, is this also supported on Carla 0.9.8?

No, the recent changes aren't part of any release yet, they are just in current master branch

Bump. This still happens with Carla 0.9.11. Position data etc. is available during playback, but velocity is not, even if additional_data is True in start_recorder.

Apparently commit https://github.com/carla-simulator/carla/commit/f39bc702422261865edf69354cc4b3119681e68e was supposed to add the velocity information (and more), but it does not appear to be working on Carla 0.9.11, the tagged release containing the commit.

Hey @raymanfx, it seems to be working fine for me in both 0.9.11 and current dev. I just created a script that:

starts recording: client.start_recorder(path, True)
spawns a vehicle and fixes its speed
waits for some seconds before stops recording
Gets the recorder info and saves it into a file client.show_recorder_file_info(path, True)
With this, I didn't see any problems, so how are you running the recorder?

Hi @glopezdiest, thanks for taking a look.
This one is actually easy to reproduce with the official examples.
I modified PythonAPI/examples/manual_control.py and changed line 403 (in the 0.9.11 release) to client.start_recorder("manual_recording.rec", True) to record the additional metadata.
Using that script, I pressed CTRL+R to start the recording, drove for a bit, pressed CTRL+R to end the recording and finally CTRL+P to start the playback. The speed in the HUD is always 0 km/h, meaning that the velocity data is missing from the recording AFACIT.

Oh no, that's a different issue. The recorder has a somewhat dual functionality.
The first one (and original behavior) allows the user to reenact the same scene over and over again (using client.replay_file(...)).
The other one is used to get information of the scene (using client.show_recorder_file_info()).
The behavior you are seeing is the first one and it doesn't matter whether or not you start the recorder such that all the velocities are saved, because the playback just doesn't use that information. Instead, the way it works is by teleporting the actors through town, not by physically moving them.

Thanks for clearing that up. I was not able to extract that kind of information from the recorder documentation.
Would it be possible to just "replay" the velocity values from the log while teleporting the actors around? It sounds like all the information is there and I think it would enhance the record/playback feature quite a bit.

I was not able to extract that kind of information from the recorder documentation.

Yeah, the teleporting part is definitely not at the Docs. We're gonna have to update that.

It sounds like all the information is there and I think it would enhance the record/playback feature quite a bit.

The playback was the original implementation of the recorder which is why it only uses the bare minimum information. It definitely would be an improvement but it would also need a complete redo to be able to use that information, and it isn't really a priority right now.

Would it be possible to just "replay" the velocity values from the log while teleporting the actors around?

Not sure, but it isn't that simple, mainly because those values are gotten from the server, and I don't think there is a way to "fake" those values. The other alternative would be my previous comment, but again, that's even harder to do.

I see. Sounds like I should just record the metadata when the agent + recorder is running and then sync it with the frames I record off the playback by matching the world snapshot timestamps.
Since this issue is about the information in the log files (which is present as you confirmed), I guess it can be closed.

I can upon this issue looking to find a way how to get a plot the acceleration, velocity, and torque curve of my ego vehicle..

To confess i don't know how to use setters and getters, i am confused do i need to write my own script or do i have to run manul_control.py and then call getters..
Can someone from you guys post their code of setters and getters =  apply.throttle() getters = get.velocity()

Also, i also tried to record and i have the log file but i don't know how to extract data from that test1.log file?


Building scheme "Starscream" failed on macOS, XCode 7.3.1

Peters-MacBook:UniversalAdvisor jps$ carthage update --platform iOS
*** Fetching ios-sdk
*** Fetching Alamofire
*** Checking out Alamofire at "3.5.1"
*** Checking out Starscream at "32fb0aa87e72f0945867201d92215081f57bf35a"
*** Checking out Freddy at "df68c72fc51e6da3d9f83726054087ea113c78f9"
*** Downloading ios-sdk.framework binary at "v0.8.0"
*** xcodebuild output can be found in /var/folders/58/th1b0h3x4_j201w8byxrm6s40000gq/T/carthage-xcodebuild.9lGqrM.log
*** Building scheme "Alamofire iOS" in Alamofire.xcworkspace
*** Building scheme "MobileFreddy" in Freddy.xcodeproj
*** Building scheme "Starscream" in Starscream.xcodeproj
** BUILD FAILED **

The following build commands failed:
	CompileSwift normal arm64
	CompileSwiftSources normal arm64 com.apple.xcode.tools.swift.compiler
(2 failures)
/Users/jps/Projects/UniversalAdvisor/Carthage/Checkouts/Starscream/Source/WebSocket.swift:249:16: error: initializer for conditional binding must have Optional type, not 'String'
/Users/jps/Projects/UniversalAdvisor/Carthage/Checkouts/Starscream/Source/WebSocket.swift:307:12: error: initializer for conditional binding must have Optional type, not 'String'
/Users/jps/Projects/UniversalAdvisor/Carthage/Checkouts/Starscream/Source/SSLSecurity.swift:165:47: error: type 'SecTrustResultType' (aka 'UInt32') has no member 'Unspecified'
/Users/jps/Projects/UniversalAdvisor/Carthage/Checkouts/Starscream/Source/SSLSecurity.swift:167:23: error: type of expression is ambiguous without more context
/Users/jps/Projects/UniversalAdvisor/Carthage/Checkouts/Starscream/Source/SSLSecurity.swift:211:43: error: type 'SecTrustResultType' (aka 'UInt32') has no member 'Unspecified'
A shell task (/usr/bin/xcrun xcodebuild -project /Users/jps/Projects/UniversalAdvisor/Carthage/Checkouts/Starscream/Starscream.xcodeproj -scheme Starscream -configuration Release -sdk iphoneos ONLY_ACTIVE_ARCH=NO BITCODE_GENERATION_MODE=bitcode CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY= CARTHAGE=YES clean build) failed with exit code 65:
** BUILD FAILED **

The following build commands failed:
	CompileSwift normal arm64
	CompileSwiftSources normal arm64 com.apple.xcode.tools.swift.compiler
(2 failures)

Xcode version: 7.3.1

Hi @peterschleinitz, v0.8.0 of the iOS SDK has been updated to use Swift 2.3 and will only work with Xcode 8. If you'd like to continue using Xcode 7, you'll need to specify v0.7.0 of the SDK in your Cartfile. Hopefully this solves your problem!

understand , thx


directory and files c#

[Enter feedback here]

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

ID: 25601a0f-eb3c-bf18-04d1-f371f5dc4341
Version Independent ID: ce7a72d0-6156-f1f8-56ee-9774d93d6d3d
Content: How to iterate through a directory tree - C# Programming Guide
Content Source: docs/csharp/programming-guide/file-system/how-to-iterate-through-a-directory-tree.md
Product: dotnet-csharp
Technology: csharp-fundamentals
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

@vigilvp Can you add details to this issue? I'm not sure what you are looking for from this issue title.

@vigilvp Can you add details to this issue? I'm not sure what you are looking for from this issue title.


Duplicate requests can be generated when one is in flight

Fixed by https://github.com/mikeboharsik/sentry/commit/e0b8e1664527e07fba3a4f041f9280bd40d38e0b#r52370369


Added vscode-insiders path installed via snap

Closes  #137116
Before
VS Code Insiders installed via Snap isn't detected.
After
[✓] VS Code (version 1.84.0-insider)
    • VS Code at /snap/code-insiders/current/usr/share/code-insiders
    • Flutter extension version 3.75.20231002

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@DanTup does this look good to you from the VS Code side?


align with circe-be changes

Align with circe-be offset-date PR
Change that impacts Capr, addition of Date Adjustment field

add DateAdjustment field

Aim to release parallel to circe-be release

Duplicate of #77


client-side navigation with a quote causes internal failure

[Error] Unhandled Promise Rejection: Error<EMAIL_ADDRESS>Failure] You stumbled upon an bug in `vite-plugin-ssr`'s source code (an internal assertion failed). This should definitely not be happening, and you should create a new issue at https://github.com/brillout/vite-plugin-ssr/issues/new that includes this error stack (the error stack is enough to debug this). Or reach out on Discord. A fix will be written promptly.
	(anonymous function) (vite-plugin-ssr_client_router.js:1029)
	asyncFunctionResume
	(anonymous function)
	promiseReactionJobWithoutPromise
	promiseReactionJob

Reproduction is something like:
const searchQuery = '"a"';
navigate(`/?q=${searchQuery}`);

Note changing it to:
navigate(`/?q=${encodeURIComponent(searchQuery)}`);

...works fine, so it may just be a matter of documentation / a better error message.
(this is a minor issue)

I will have a look at it later today.
Curious; what's your dev environment? You have an unexpected strack trace.

I think that's just Safari, here's Chrome:
newError.ts:18 Uncaught (in promise) Error<EMAIL_ADDRESS>Failure] You stumbled upon an bug in `vite-plugin-ssr`'s source code (an internal assertion failed). This should definitely not be happening, and you should create a new issue at https://github.com/brillout/vite-plugin-ssr/issues/new that includes this error stack (the error stack is enough to debug this). Or reach out on Discord. A fix will be written promptly.
    at newError2 (newError.ts:18)
    at assert (assert.ts:18)
    at parseUrl (parseUrl.ts:13)
    at getFileUrl (getFileUrl.ts:19)
    at retrieveContextProps (getContextProps.client.ts:22)
    at getContextProps (getContextProps.client.ts:16)
    at getPageByUrl.client.ts:14
    at getPageByUrl (getPageByUrl.client.ts:14)
    at fetchAndRender (useClientRouter.client.ts:71)
    at navigateFunction (useClientRouter.client.ts:45)
    at navigate (useClientRouter.client.ts:136)
    at onSubmit (Catalog.tsx:34)
    at HTMLUnknownElement.callCallback2 (react-dom.development.js:3945)
    at Object.invokeGuardedCallbackDev (react-dom.development.js:3994)
    at invokeGuardedCallback (react-dom.development.js:4056)
    at invokeGuardedCallbackAndCatchFirstError (react-dom.development.js:4070)
    at executeDispatch (react-dom.development.js:8243)
    at processDispatchQueueItemsInOrder (react-dom.development.js:8275)
    at processDispatchQueue (react-dom.development.js:8288)
    at dispatchEventsForPlugins (react-dom.development.js:8299)
    at react-dom.development.js:8508
    at batchedEventUpdates$1 (react-dom.development.js:22396)
    at batchedEventUpdates (react-dom.development.js:3745)
    at dispatchEventForPluginEventSystem (react-dom.development.js:8507)
    at attemptToDispatchEvent (react-dom.development.js:6005)
    at dispatchEvent (react-dom.development.js:5924)
    at unstable_runWithPriority (scheduler.development.js:468)
    at runWithPriority$1 (react-dom.development.js:11276)
    at discreteUpdates$1 (react-dom.development.js:22413)
    at discreteUpdates (react-dom.development.js:3756)
    at dispatchDiscreteEvent (react-dom.development.js:5889)

I see what's going on, I will fix it tomorrow.

Released in 0.1.0-beta.35.
This required some profound changes; hopefully I didn't break other aspects of URL handling (but I'm confident I didn't :-)). Good news it that the new URL handling structure is fundamentally more robust. Happy to fix any future URL handling edge cases that may come up.
It may sound surprising but, on second thought, testing URL handling is actually mostly useless (happy to elaborate if you are curious) so I'm not going to programmatically test URL handling for the time being.


ci: add release workflow without pypa action

Closes #68 by adding a release workflow. Note that I'm currently suggesting that we stop using the PyPA action as it is pretty much a one-liner to do it manually.
What is done by the action is can be seen in the PyPA action can be seen in this script based on these default input variables using twine with the following --help message for the check and upload commands.
usage: twine upload [-h] [-r REPOSITORY] [--repository-url REPOSITORY_URL]
                    [-s] [--sign-with SIGN_WITH] [-i IDENTITY] [-u USERNAME]
                    [-p PASSWORD] [--non-interactive] [-c COMMENT]
                    [--config-file CONFIG_FILE] [--skip-existing]
                    [--cert path] [--client-cert path] [--verbose]
                    [--disable-progress-bar]
                    dist [dist ...]

positional arguments:
  dist                  The distribution files to upload to the repository
                        (package index). Usually dist/* . May additionally
                        contain a .asc file to include an existing signature
                        with the file upload.

optional arguments:
  -h, --help            show this help message and exit
  -r REPOSITORY, --repository REPOSITORY
                        The repository (package index) to upload the package
                        to. Should be a section in the config file (default:
                        pypi). (Can also be set via TWINE_REPOSITORY
                        environment variable.)
  --repository-url REPOSITORY_URL
                        The repository (package index) URL to upload the
                        package to. This overrides --repository. (Can also be
                        set via TWINE_REPOSITORY_URL environment variable.)
  -s, --sign            Sign files to upload using GPG.
  --sign-with SIGN_WITH
                        GPG program used to sign uploads (default: gpg).
  -i IDENTITY, --identity IDENTITY
                        GPG identity used to sign files.
  -u USERNAME, --username USERNAME
                        The username to authenticate to the repository
                        (package index) as. (Can also be set via
                        TWINE_USERNAME environment variable.)
  -p PASSWORD, --password PASSWORD
                        The password to authenticate to the repository
                        (package index) with. (Can also be set via
                        TWINE_PASSWORD environment variable.)
  --non-interactive     Do not interactively prompt for username/password if
                        the required credentials are missing. (Can also be set
                        via TWINE_NON_INTERACTIVE environment variable.)
  -c COMMENT, --comment COMMENT
                        The comment to include with the distribution file.
  --config-file CONFIG_FILE
                        The .pypirc config file to use.
  --skip-existing       Continue uploading files if one already exists. (Only
                        valid when uploading to PyPI. Other implementations
                        may not support this.)
  --cert path           Path to alternate CA bundle (can also be set via
                        TWINE_CERT environment variable).
  --client-cert path    Path to SSL client certificate, a single file
                        containing the private key and the certificate in PEM
                        format.
  --verbose             Show verbose output.
  --disable-progress-bar
                        Disable the progress bar.

usage: twine check [-h] dist [dist ...]

positional arguments:
  dist        The distribution files to check, usually dist/*

optional arguments:
  -h, --help  show this help message and exit

@minrk, this has gone stale - does it LGTY?

Thanks for the reminder!


Hebrew : multiline text view widgets dont display correctly on several lines

Describe the bug
In Hebrew, multiline text display widgets do not display strings correctly on several lines
The display is correct if the string fit on one line, but not correct on several lines
define GX_DYNAMIC_BIDI_TEXT_SUPPORT and use of _gx_system_bidi_text_enable doesnt seem to affect the draw
The problem seems to come from the _gx_multi_line_text_view_text_draw function.
The cutting of the character chain in several lines does not seem to be done by taking the chain in the right order.
This one should start in the opposite direction in memory
Imagine the string "GFEDCBA" as Hebrew characters. On one line it is display as "GFEDCBA" (store like this in memory)
If the maximum characters per line is 4 :
On two lines it is displayed as (because the text is drawn from left to right)
"GFED
CBA"
but should be displayed as (from right to left)
"DCBA
GFE"
Please also mention any information which could help others to understand
the problem you're facing:

What target device are you using? : RA6M3 from renesas
Which version of Azure RTOS? 6.2.1, Guix Studio <IP_ADDRESS>
What toolchain and environment? GCC, Ide e2studio

To Reproduce
Steps to reproduce the behavior:

Create a page with a multiline text view widget
Assign a Hebrew string on this widget. Long enough to be displayed on several lines

Expected behavior
The string should be displayed correctly.
Impact
multiline text view widget with Hebrew
Logs and console output
None
Additional context
None

@ThibaudBrandt Have you defined a user-define memory allocator and de-allocator using gx_system_memory_allocator_set API? Runtime allocation of memory is required for dynamic bidirectional text drawing.

@ting-ms : Yes I did. We don't have any allocation issue, except the need to increase it when gx_system_bidi_text_enable was called. The text is displayed correctly if there is only one line, but on severals lines the text is not displayed in the correct order. I also tried to use the "gx_system_bidi_text_enable" function, but I dont see any changes except a slowdown of the display and some string that dont appear anymore.
We managed to display it correctly by reversing the 'string' in the '_gx_multi_line_text_view_text_draw' and '_gx_multi_line_text_view_string_total_rows_compute' functions to force the calculation of the string split across multiple lines, displayed from right to left. However, this doesn't seem optimal, and I believe we might have overlooked something.
I can share the modified files if needed

@ThibaudBrandt Could you please share the Hebrew string along with a screenshot of the displayed results? I would like to test it on my end.

הכינו את הקרם יום קודם הפרידו את הבצק לריבועים של 5 ס''מ לפני הבישול. הכינו את רוטב הסירופ. פזרו אבקת סוכר. הגישו קר.
@ting-ms : Here is above the string.
Here is a screen shot with above the string displayed in GUIX studio and bellow the string in the embedded product.

@ThibaudBrandt The result displayed in GUIX Studio should match those in your embedded product. Would you check the language setting in your GUIX Studio project:

If you are using runtime bidi text drawing, ensure that the option Generate Bidi Text in Display Order is unchecked.

@ting-ms : I tried to unchecked the Generate Bidi Text in Display Order.  It works. However, the real-time rendering is considerably slowed down, and the strings sporadically disappear. I think checking the option 'Generate Bidi Text in Display Order' and correctly cutting and displaying strings is the correct way to do it in our case

@ThibaudBrandt The checkbox "Generate Bidi Text in Display Order" instructs GUIX Studio to generate bidirectional text to the output file in its display order. If this option is selected, no runtime processing is required within the GUIX library to properly render bidirectional text. When this option is selected, bidirectional text rendering should NOT be enabled within the GUIX library. This configuration yields the best runtime performance, but does not support rendering of dynamically defined bidirectional text strings.
In general, the bidi text reordering process is paragraph based without line breaking. To render a bidi text in a multiline text view widget with appropriate line breaks, assign the string to the designated multi-line text view widget within GUIX Studio. Subsequently, GUIX Studio will generate a reordered bidi text for the multi-line text view, incorporating the necessary line breaking.

@ting-ms: All our strings are defined in GUIX Studio with an ID but not necessarily assigned to a widget in particular. I tried to assign the string to the widget it's supposed to be displayed on. However, it is not very convenient because I had to invert the language index to assign the number 1 to Hebrew in order to assign the Hebrew string to the widget. I compared the generated string between assigned to widget/not assigned to widget in the 'resources.c,' and there is indeed a small difference.
However I have a lot of strings and I don't know which widget they will be assigned to. And creating X widgets to assign my X strings to these widgets are not a viable solution

@ThibaudBrandt Only strings that need line breaking should be assigned to the target widget, by doing that, GUIX Studio would know the font and the line width needed for line breaking. You may also consider disablign the optionGenerate Bidi Text in Display Order and in the application code,  reorder the BiDi text for displaying by calling API gx_utility_bidi_paragraph_reorder.

@ThibaudBrandt Please feel free to reopen the issue if you encounter any further issues.


Hoverfly templating issue with number greater than<PHONE_NUMBER>740991

Description of the bug
I am trying  to get data from request to the response via templating.While trying it is observed for numbers greater than<PHONE_NUMBER>740991 in the request , the response gets changed .
Example :
If I am passing this as the request body :
"Input": {
"inputNo1": "100",
"inputNo2": "0009007199254740999",
"inputNo3": "001"
}
Output Response:
"Output": {
"outputNo1": "100",
"outputNo2": "0009007199254741000",
"outputNo3": "1"
}
Template used : "{{ Request.Body 'jsonpath' '$.Input.inputNo2' }}"
Steps to reproduce the issue
Use the above template to extract the data from request .
Here the data is send as string and the template is also placed within "".
Observed result
Hoverfly error messages seen (If none, say none)
(paste here)

If possible, add screenshots to help explain your problem
Expected result
Additional relevant information

Hoverfly version:
| hoverctl | master-4270 |
| hoverfly | v1.9.5
Anything that might help us to diagnose the problem: Could it be due to floating-point precision limitations? What is the solution to it .

@akhilasmohan raised PR for the same. @tommysitu Could you please review the same.

++
On Fri, Nov 29, 2024, 16:08 Kapish Malik @.***> wrote:

@akhilasmohan https://github.com/akhilasmohan raised PR for the same.
@tommysitu https://github.com/tommysitu Could you please review the
same.
—
Reply to this email directly, view it on GitHub
https://github.com/SpectoLabs/hoverfly/issues/1155#issuecomment-2507538276,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVBRCWHKTB23NPLTTHYBEQ32DA73VAVCNFSM6AAAAABSWVYHA2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBXGUZTQMRXGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

@akhilasmohan this issue is fixed in new version 1.10.5

Hi Team,
Sure Thank you
Regards,
Akhila
On Sun, Dec 1, 2024, 22:56 Kapish Malik @.***> wrote:

@akhilasmohan https://github.com/akhilasmohan this issue is fixed in
new version 1.10.5
—
Reply to this email directly, view it on GitHub
https://github.com/SpectoLabs/hoverfly/issues/1155#issuecomment-2510091857,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVBRCWE3OCQD2LOAPLLJSMT2DNBDNAVCNFSM6AAAAABSWVYHA2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJQGA4TCOBVG4
.
You are receiving this because you were mentioned.Message ID:
@.***>


Moved react to peer dependencies

Just a quick PR to move React to peer dependencies in package.json so we don't run in to conflicts with multiple copies of react being installed in a parent app.

Hey Collin,
Appreciate the change. Peer dependencies aren't supported in npm version 2, so this would actually break existing projects. I'd recommend upgrading if you're worried about duplicates. You also get some nice freebies, like babel running faster, if you're using it. Happy coding!

I'm actually running npm version 3+.  It is my impression that peerDependencies is a better place for dependencies like React.  When installing react-currency-masked-input on a project that lists React 15 as a dependency, you'll end up with dupes because of react-currency-masked-input, where with React defined as a peerDependency, you do not.
Your thoughts?

I went to check my understanding on peer deps and found https://github.com/feross/standard/issues/255. If you look at their solution, it was to move eslint to dependencies, and eventually remove peer dependencies. That's the same approach I've taken.
If it's still a problem with you using npm 3, I'd be happy to take a look at a project if you can send it along.

I don't think the eslint usage is the best example.  Eslint isn't usually saved as a dependency, but rather a devDependency, but even so, if your build environment ends up with multiple copies of eslint, no big deal, however React is not so friendly, which is why React itself recommends saving React as a peerDependency or devDependency.
I feel you on peerDependency being on it's way out, I've resorted to marking React as a devDependency for my apps and modules.  Kinda weird, but keeps the errors at bay.
For your consideration, I had to fork your module and move react to peerDependencies to not end up with dupes of React.


How to get picking listener direct from Scene Object

There is a way to get direct from the GVRSceneObject if he is picked? like:
   GVRSceneObject object;
   // ...
      object.setPickingListener(new PickingListener(){
       public void onPick(GVREyePointeeHolder e) {
       }
    });

or
public class Mic extends GVRSceneObject{
//...
    @Override
    public void onPick(GVREyePointeeHolder e) {
    }

}

No, there is not. Nor am I sure that this is desirable. One doesn't always care about only what is exactly in  the center of the field of view: Imagine, for example, an app that tracks the position of the user's hands, and wants to use picking to figure out what she might be touching.

Ok so the method can call onCenter(GVREyePointeeHolder e) instead onPick(GVREyePointeeHolder e)?

By this moment I'm controlling 50 objects (and rising) "onCenter" inside a GVRScript for ... this is not looking good.

This kind of picking is not an unusual use case, but it does require deeper thought than just onPick() (which i assume would map to intersection with object and tapping the touchpad?).  Just off-hand, I can think of onEnter, onLeave, onSelected, onDeselected, and you may want to apply a filter based on the type of object.  This type of thing has been thought out before in APIs like Inventor and X3D.  We will need to give the same kind of thought here as well.

In addition to the onPick() vs. onCenter() issue (and the Other APIs issue that @thomasflynn  cites) I'm having some trouble seeing why this is desirable.
First, the current pickScene() and findObjects() APIs sort hits by their distance from the camera. This lets you do things like pay attention to transparency and walk the list until you get to the object the user 'means to' interact with. Or model translucency, and handle hits until the 'beam' is totally attenuated.
Second, I am sure that it's cleaner to just add GVRSceneObject descendants to a GVRScene and let them Do The Right Thing when picked than to walk the list and Do The Right Thing in the loop ... but, honestly, the instanceof and cast overhead in a loop like
    for (GVRPickedObject picked: findObjectsResult) {
      GVRSceneObject object = picked.getHitObject();
      if (object instanceof MyAppObject) {
        ((MyAppObject) object).myOnPick();
        }
      }

is really quite minimal compared to the cost of picking itself, which has to do non-trivial math on every pickable object.

If we avoid using intanceof, and instead get this info directly on GVRSceneObject, we'll make the bottleneck, that is handling every state of eyepicking, smaller.
For instance, we have to control elements when they are selected and when they are not. If this control was held to GVRSceneObject, we wouldn't have to do it checking the eyepointeeholder.
if (eyePointeeHolders.length == 0) {
            for (GVRSceneObject object : mGVRContext.getMainScene().getWholeSceneObjects()) {
                if (object instanceof SphereFlag && object.equals(lastItem)) {
                    ((SphereFlag) object).restoreObject();
                }
             }
} else {
            for (GVREyePointeeHolder eph : eyePointeeHolders) {
                for (GVRSceneObject object : mGVRContext.getMainScene().getWholeSceneObjects()) {
                    if (eph.getOwnerObject().equals(object) && object instanceof SphereFlag) {
                        ((SphereFlag) object).doSomething();
                    }
                }
             }

for (GVRSceneObject object : mGVRContext.getMainScene().getWholeSceneObjects()) {
   if (object instanceof SphereFlag && object.equals(lastItem)) {
     ((SphereFlag) object).restoreObject();

This seems like a lot of overhead, and I'm not sure it's any different that
if (lastItem != null) {
  lastItem.restorteObject();
}

Similarly, it's not all obvious why the second branch enumerates all the eye pointee holders in an outer loop, then enumerates all the objects in the scene graph in an inner loop. Picking should never return an object that's not in the scene graph.


No such file or directory

Hi Linaqruf,
Thank you for the the colab notebooks. I am trying out the Kohya LoRA Dreambooth one but I am getting the " ./train.sh: line 2: sl: No such file or directory " error for some reason. Here are a few more information:

I didn't use Login to Huggingface hub section in 1.3.1 and the colab notebook was still able to download the Stable-Diffusion-v1-5 model in section 2.1 to /content/pre_trained_model/ as Stable-Diffusion-v1-5.ckpt .
I didn't connect Google Drive in 1.3.2 Mount Drive section because it's written to be optional.
I created the train_data directory in /content/dreambooth/ using the file explorer inside the left side panel of colab.
I put /content/dreambooth/train_data/ as parent directory, train_repeats as 10, concept name as <sl> and class name as man.
I clicked run for section 3.1 using the above parameters and the notebook created a new folder called 10_<sl> man so that that folder's path is /content/dreambooth/train_data/10_<sl> man.
I put all the training images (i.e. 1.png, 2.png, etc.) and the corresponding caption text files (i.e. 1.txt, 2.txt, etc.) inside the folder at path /content/dreambooth/train_data/10_<sl> man.
Then in section 5.1 to Define Important Folder, I set the pretrained_model_file_or_path as /content/pre_trained_model/Stable-Diffusion-v1-5.ckpt, train_folder_directory as /content/dreambooth/train_data/10_<sl> man/ and reg_folder_directory as /content/dreambooth/train_data/reg_data/1_man.
After defining specific LoRA training parameters, I start the LoRA Dreambooth training in section 5.3 but it stops and shows me the error message ./train.sh: line 2: sl: No such file or directory as mentioned above.

Here's some more additional info from config file in case these are relevant:
accelerate launch --config_file=/content/kohya-trainer/accelerate_config/config.yaml --num_cpu_threads_per_process=8 train_network.py --network_dim=128 --network_alpha=128 --network_module=networks.lora --learning_rate=0.0001 --text_encoder_lr=5e-05 --training_comment=this_comment_will_be_stored_in_the_metadata --lr_scheduler=constant --pretrained_model_name_or_path=/content/pre_trained_model/Stable-Diffusion-v1-5.ckpt --caption_extension=.txt --train_data_dir=/content/dreambooth/train_data/ --reg_data_dir=/content/dreambooth/train_data/reg_data/ --output_dir=/content/dreambooth/output --prior_loss_weight=1.0 --output_name=simu_liu_<sl> --mixed_precision=fp16 --save_precision=fp16 --save_n_epoch_ratio=3 --save_model_as=safetensors --resolution=512 --enable_bucket --min_bucket_reso=256 --max_bucket_reso=1024 --cache_latents --train_batch_size=6 --max_token_length=225 --use_8bit_adam --max_train_epochs=20 --gradient_accumulation_steps=1 --clip_skip=2 --logging_dir=/content/dreambooth/logs --log_prefix=simu_liu_<sl> --shuffle_caption --xformers
Please help! Thanks so much!

helllo, try to not use <> for token,

Thank you! Not using the <> for token really helped!
By the way, what's the difference between instance_prompt and just prompt in the 6.2. Inference section?

no different, final prompt is
{instance_prompt}, {prompt} --n {negative_prompt}
instance_prompt field added so you can put your trigger word without need to changing prompt composition, cuz people love to copy-paste prompt.
But i think it's not effective for non-danbooru model

I see. Thanks so much for the tip and the explanation!


Functional test: atomicSet updates to embedded documents do not work

Making a change to the embedded document in this test fails to update the MongoDB record.
This test passes if strategy="set" is used.
repro:
phpunit --filter testFailureToUpdateEmbeddedDocumentWithAtomicSet

emits:
PHPUnit 4.1.6-9-gbe3530d by Sebastian Bergmann.

Configuration read from /mongodb-odm/phpunit.xml

F

Time: 1.41 seconds, Memory: 21.75Mb

There was 1 failure:

1) Doctrine\ODM\MongoDB\Tests\Functional\Ticket\GHXXXXTest::testFailureToUpdateEmbeddedDocumentWithAtomicSet
The chapter title failed to update.
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'First chapter A'
+'A'

/mongodb-odm/tests/Doctrine/ODM/MongoDB/Tests/Functional/Ticket/GHXXXXTest.php:33

FAILURES!
Tests: 1, Assertions: 1, Failures: 1.


manual mail recovering

Hi
I got just now the new version 5.724
There are many changes . Why not .
But the new version erased the old possibility of manual load
The only possibility is now in automatic from never to 15 minutes till 24hours
This is not a good idea for those people who has limited connection.
For them for example a pick up all 30 minutes will decrease the communication  potential.
On the other side "all 24 hours" is too long sometimes and some times not . It depends of what is coming
The best was for me the manual pick up.
Could it be back in next version ?
Thanks

You can still manually refresh a folder by pulling down the message list. You can check an account by pulling down the folder list in the side drawer. You can check all accounts by switching to the list of accounts in the side drawer and pulling down that list.
In the future please use the forum for support requests.


Fix Input shifters cannot be edited

fixes #1379

seems to work when I tested briefly. I can scan for input and it assigns a button also from input shifter.


Lmolto defs

A new LMO / LTO life model was added to SSC in https://github.com/NREL/ssc/pull/588. Here, add a BatteryStateful_LMOLTO.json file for updated LMO / LTO defaults that can be both loaded in SAM and used in PySAM. The values are taken from Toshiba SCiB SPCS-LIB010A 10kWh.
In a future development, https://github.com/NREL/SAM/issues/661, the defaults can be better updated to use the new life model, but for now, just copy the existing life cycling matrix.

Thanks @cpaulgilman for the comments.
1/2. The life_model options aren't available in the GUI yet. So in the GUI now, it should life_model should always be 0. When that's added, it will be made explicit.
The confusing part about the defaults here are that they serve two sometimes disjunct purposes. Most defaults are shared by the GUI and by BatteryStateful, but when it comes to the life model, the GUI still uses the cycle_matrix and calendar_choice & other calendar inputs, whereas the BatteryStateful NMC & LMOLTO doesn't use any of those and only the life_model. So while each case uses different sets of inputs for the life modeling, both of them are in the default JSONs. Once #661 is done, we can move the GUI to use the life_model input and get rid of the unnecessary inputs.
Thanks for catching "0/1/2"! Fixing that.


feat: add serialization support for xml error responses

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 14 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
584

Relevant Lines:
584

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

14 of 14 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
584

Relevant Lines:
584

💛  - Coveralls


Targets controller

Coverage increased (+0.9%) to 34.275% when pulling 887403a61a34e938a3c2839956d583ecae99200c on targets-controller into e1fc19cf8d7d05207c624392412ce4dd0f7ca65e on tag-controller.

/lgtm


fix: do not throw error for missing content-type header

According to RFC 2616 a response SHOULD, but not MUST, include a content-type header. There are cases where a content-type header is not required such as when returning an empty response. This is common for some DELETE endpoints that return a 204 No Content response.
I think in this case, returning a blob is appropriate and let the developers handle it how they please

Codecov Report

Merging #167 (af913a0) into main (8debc53) will increase coverage by 0.57%.
The diff coverage is 83.33%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #167      +/-   ##
==========================================
+ Coverage   62.93%   63.50%   +0.57%
==========================================
  Files          11       11
  Lines         286      285       -1
  Branches       71       70       -1
==========================================
+ Hits          180      181       +1
+ Misses         96       94       -2
  Partials       10       10

Impacted Files
Coverage Δ

libs/ts-rest/express/src/lib/ts-rest-express.ts
11.84% <0.00%> (ø)

libs/ts-rest/core/src/lib/client.ts
92.68% <100.00%> (+4.58%)
:arrow_up:

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


Use SVG badges

Much crisper, nicer, and more scalable than the PNG versions. :-)

@olivierlacan Hey, thank you for this PR!
PS @lotus/core-team I'm merging this, even if CI isn't passing, but there is a breaking change with SimpleCov that I'm gonna fix right now in master.


No? The settings dialog shows on the first tap. What browser are you using?


Binlog missing imported files

Steps to reproduce
Command line
mkdir repro
cd repro
dotnet new console
msbuild /bl

Open msbuild.binlog in viewer, go to Files tab
Expected  behavior
All imported files are present
Actual behavior
Only the root project file is present.
Example binlog: msbuild.binlog.zip
Environment data
This appears to be a recent regression.
Both of the following are affected:
Microsoft (R) Build Engine version 16.3.0-preview-19412-01+01797ab10 for .NET Framework
Copyright (C) Microsoft Corporation. All rights reserved.

Microsoft (R) Build Engine version 16.3.0-preview-19409-03+c1f5c6f00 for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.

The following is not affected
Microsoft (R) Build Engine version 16.3.0-preview-19377-01+dd8019d9e for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.

@rainersigwald @dsplaisted @KirillOsenkov

Regression is in this commit range: https://github.com/microsoft/msbuild/compare/dd8019d9e...c1f5c6f00

This regression was introduced by #4485, but I don't know how or why. I just eyeballed the commits in that range and it was the second one on my short list; reverting it seems to have fixed the problem.
@BenVillalobos can you figure out what went wrong and add a test that catches the problem?

The Importing project events are missing from the log. Investigate why these events aren't being raised:
https://source.dot.net/#Microsoft.Build.Framework/ProjectImportedEventArgs.cs,8141396abad6d0eb
You should expect something like this:
Importing project "C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\MSBuild\Current\Bin\Roslyn\Microsoft.Managed.Core.targets" into project "C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\MSBuild\Current\Bin\Roslyn\Microsoft.CSharp.Core.targets" at (4,3).

You can play back the binlog to view the events:
msbuild msbuild.binlog /v:diag | clip and then paste into a text editor.

For those that run into this issue, the workaround is setting the MSBUILDLOGIMPORTS environment variable to 1


feat(/ipfs/connect): Adding REST API endpoint to connect to an IPFS peer

:tada: This PR is included in version 5.1.0 :tada:
Your semantic-release bot :package::rocket:


Colocalization module

Colocalization is a common metric for analysis of imaging MS/spatial metabolomics data.
In a previous publication from Theo (ColocML), they evaluated metrics/processing steps for computing colocalization. I implemented the best-performing metric.
I think it is a useful addition to the package for users to perform this kind of analysis/processing which cannot be done by other downstream analysis packages such as scanpy or squidpy.

Once merged, closes #16 .


[PROPOSAL] Host updated Airflow constraints files in OL repo to solve MWAA constraints problem

Purpose:
The problem with MWAA is that if not specified otherwise, by default, it uses constraint files from Airflow which means that:

for MWAA 2.7.2 (Airflow 2.7.2, Python3.11 ) the OL provider is bound to ==1.1.0
for MWAA 2.8.1 (Airflow 2.8.1, Python3.11 ) the OL provider is bound to ==1.4.0

So whenever you are putting just apache-airflow-providers-openlineage in the requirements.txt for the MWAA - it uses that version specified in the constraints file.
However, if you put a different version f.e. apache-airflow-providers-openlineage==1.5.0 there is a mismatch (two different versions of the same package are requested), an error is raised and no package is installed.
What one can do, is remove the constraint file by putting --constraint /dev/null as the first line of the requirements.txt file. Alternatively, one could modify those standard Airflow constraints (as they are publicly available) by changing the OpenLineage provider version, host it somewhere and then point to that file f.e. by putting this as a first line in requirements.txt:

--constraint "https://somedomain.com/constraints-2.8.1/constraints-3.11.txt"

Proposed implementation
Maybe we could put some step in the CI, that will prepare a copy of all Airflow constraint files and update the OL version to the latest version we know works fine and put that files in OL repo? This could be helpful for the MWAA users willing ot use latest OL features and fixes but not being able to remove the constraints files.
Thoughts

I don’t think there is a way to change the MWAA behaviour, as it’s not some bug or issue but the desired behaviour introduced by them recently:

Beginning with Apache Airflow v2.7.2, your requirements file must include a --constraint statement. If you do not provide a constraint, Amazon MWAA will specify one for you to ensure the packages listed in your requirements are compatible with the version of Apache Airflow you are using.

And even if we could somehow influence their behaviour, we need to provide a valid alternative. If we asked MWAA to remove the constraints, they would say we can simply use --constraint /dev/null and remove them ourselves. If we asked them to use different constraints, with higher version of OpenLineage, somebody would have to prepare and maintain those (also, users are not so eager to use 3rd party provided files, that’s probably why MWAA uses the one from Airflow) and they would say, we can easily do it ourselves as well.

I'm not sure if we can guarantee the sync between our constraints and Airflow ones over time, something to think about.
Just FYI, here are the docs about generating constraints in Airflow

I think it would be good to have an option for MWAA users to override their dependencies.
Not exactly sure we're good place for that - especially if any other dependencies change, and we don't have good ways to test and make sure we're not breaking something else.

@rahul-madaan I updated the description on the PR that fixed the issue, providing with a solution. Now that i mentioned this PR, it should be linked to this issue, and users should be able to easily find it, also in Google. So i think for now we can just provide the users with link to this issue and they will be able to figure out the solution. WDYT?

Looks good! let us close the issue?


How to customized request body "Example Value"？

I using springfox.swagger2.version 2.8.1-SNAPSHOT  today.
@ExampleProperty(mediaType = "application/json", value = "{\n" +""aa": {\n" +
""type": "aa" \n" +" }") })) @NotNull @RequestBody @Valid DesktopSnapshot snapshot)

but it not work？

I hope it shows like

How to do it？thanks.
Please do consider starring this repository if you do like this project and find it useful.

Like this

hi @dilipkrish  ,
I try your way like this ，but it doesn't work.
@Override
@ResponseBody
@PostMapping(value = "/{desktop_id}")
@ApiOperation(value = "A", response = DesktopSnapshotEntity.class)
@ApiResponses(value = { @ApiResponse(code = HttpStatus.SC_ACCEPTED, message = "SUCCESS") })
@ApiImplicitParams({
@ApiImplicitParam(
name = "snapshot",
dataType = "DesktopSnapshot",
examples = @io.swagger.annotations.Example(
value = {
@ExampleProperty(value = "’snapshot‘：{'type': 'AAA'}", mediaType = "application/json")
}))
})
public ResponseEntity<?> snapshot(
@ApiParam(value = "AA", required = true) @PathVariable("desktop_id") @NotBlank String desktopId,
@NotNull @RequestBody @Valid DesktopSnapshot snapshot) {
...}
------------------DesktopSnapshot .java----------
@Data
@AllArgsConstructor
@NoArgsConstructor
@JsonRootName("snapshot")
public class DesktopSnapshot {
private String type;

..
}
----------------config is----------------
return new Docket(SWAGGER_2).securityContexts(asList(securityContext()))
.securitySchemes(asList(new ApiKey(API_KEY, headerTokenKey, HEADER))).enableUrlTemplating(true).select()
.apis(RequestHandlerSelectors.basePackage("com.huawei.cloudpc"))
.apis(RequestHandlerSelectors.withClassAnnotation(Api.class))
.apis(RequestHandlerSelectors.withMethodAnnotation(ApiOperation.class)).paths(PathSelectors.any()).build().pathMapping("/").alternateTypeRules(newRule(
typeResolver.resolve(DeferredResult.class,
typeResolver.resolve(ResponseEntity.class, WildcardType.class)),
typeResolver.resolve(WildcardType.class)))
.useDefaultResponseMessages(false).additionalModels(typeResolver.resolve(DesktopSnapshot .class))

I hope it shows  {”snapshot“：{”type“: ”AAA“}} in example vaule. but it still like

I want the "snapshot" which  in @JsonRootName("snapshot") can show in UI ”example value“ or use @ExampleProperty value  displaying  directly in  UI ”example value“.
Which way can be achieved ？thanks.

It needs to be ... value = "{'snapshot'：{'type': 'AAA'}}"

@dilipkrish
    @Override
@ResponseBody
@PostMapping(value = "/{desktop_id}")
@ApiOperation(value = "", response = DesktopSnapshotEntity.class)
@ApiResponses(value = { @ApiResponse(code = HttpStatus.SC_ACCEPTED, message = "success") })
@ApiImplicitParams({
        @ApiImplicitParam(
            name = "snapshot",
            dataType = "DesktopSnapshot",
            value = "{'snapshot'：{'type': 'AAA'}}",
            examples = @io.swagger.annotations.Example(
                value = {
                    @ExampleProperty(value = "{'snapshot'：{'type': 'AAA'}}", mediaType = "application/json")
                }))
    })
public ResponseEntity<?> snapshot(
		@ApiParam(value = "s", required = true) @PathVariable("desktop_id") @NotBlank String desktopId,
		@NotNull @RequestBody @Valid DesktopSnapshot snapshot) {

It's not work ?   nothing in example value? Please help me to answer it，thanks.

@dilipkrish
hi, i have a question about annotation @Example.
i cann't find answer, i really can't solve this problem.
i have a api like this:
@PostMapping("test")
public void test(@RequestBody @ApiParam(examples = @Example(value = {@ExampleProperty(mediaType = "application/json", value = "{'test':'test'}")}),
            name="name",value="value") Map<String, String> params) {
}

The paramter type is Map.
Can't I customize request body "Example Value"?
how to customize request body "Example Value"?

What do you mean customize the "Example Value"? It should be available in version 2.9.2

What do you mean customize the "Example Value"? It should be available in version 2.9.2

I see no documentation on how to use this. Can you provide assistance?

Can anyone confirm that @Example annotation really works? I have tried to use it but with no success. I literally copied example annotation form FeatureDemonstrationService, and it still not working. My code looks like this:
@ApiOperation(value = "Description")
@ApiResponses(value = {
        @ApiResponse(code = 404, message = "Not Found",
                examples = @Example(
                        value = {
                                @ExampleProperty(
                                        mediaType = "Example json",
                                        value = "{\"invalidField\": \"address\"}"),
                                @ExampleProperty(
                                        mediaType = "Example string",
                                        value = "The first name was invalid")}))})
@GetMapping(value = "/stats/graph", produces = {"application/json"})
public ResponseEntity<StatisticsResponse> getEventGraph(
        @ApiParam(value = "val1")
        @RequestParam(value = "val1", required = false) List<String> val1,
        @ApiParam(value = "val2", example = "60")
        @RequestParam(value = "val2", defaultValue = "60") Integer val2,
        @ApiParam(value = "val3")
        @RequestParam(value = "val3", defaultValue = "30") Integer val3,
        @ApiParam(value = "val4")
        @RequestParam(value = "val4", defaultValue = "false", required = false) boolean val4) { }

I use newest version of springfox
ext {
// project dependency versions
springfoxVersion = '2.9.2'
}
compile("io.springfox:springfox-swagger2:${springfoxVersion}")
compile("io.springfox:springfox-swagger-ui:${springfoxVersion}")

What does not working mean? in the swagger-ui or does it not render the examples?

Does not render the provided examples. The only one example rendered is the default one generated for the method return type.

If your endpoint is returning a model, to set custom example values in the swagger-ui, use the @ApiModelProperty for each field then use example to define the custom values.
@ApiModelProperty(example="....")

I already pointed that I know this method, but I can't use it. This is because model is provided by external source, so it cannot be annotated with @ApiModelProperty.

Ah I apologize, I didn't read the whole thread.

@tgielbutowicz You can implement your own ParameterBuilderPlugin and replace the complexExamples() call with code that generates an example given a type of parameter.

I also was trying all pointed out here but still not working. I opened the following issue: https://github.com/springfox/springfox/issues/3037

And it's still not working. Looks like the project is dead. New issues were created but then automatically closed because author does not fix them.


[website] Translation strings are not downloaded correctly

Motivation
Noticed following warnings in website CI builds.
Warning: Downloaded translations do not match current project configuration. The following files will be omitted:
 - '/website/i18n/ja-JP.json'

The warnings were generated before i18n/en.json was not generated before downloading translations.
Changes
add yarn write-translations before uploading/downloading translations.
additionaly fix <translate> location for download page

@cckellogg ^^

run java8 tests


Exactly once sematics and DLQ confuguration.

Hi,
We are trying to use the kafka connect  with the mongodb sink connector in order to sink the genearated event messages to the mongodb.  Below are few of our concerns and we need your suggestions on the same.

Does the connect  and connector support the idempotence and exactly once  semantics? I tried to pass the producer and consumer config through the sink properties. But the connect starts with it default producer and consumer config. I used consumer.enable.auto.commit=false and so on.
In case when mongodb server is down the connect retries and throws error 'Task is being killed and will not reciver until manually restarted'. Do we have a way to control this with some configuration?  If the mongodb  servers is up and running shouldn't  it reconnect automatically? Or write to DLQ in case failed due to mongo or connector error? I tried errors.deadletterquee.tooic.name=my-connector-errors .

Thanks

Hi @abhisheksingh87 - thanks for reaching out.

exacty-once semantics: in general sink connectors cannot provide exactly-once behaviour for any data. what is possible with e.g. my sink connector is to write against the sink (mongodb collection) with upsert semantics which when combined with any unique attribute found in kafka records give you idempotent write behaviour. so if you make sure that your data in kafka exhibits any unique attribute in the first place you can achieve exactly-once semantics against the sink given a proper configuration (see DocumentIdAdder strategy options in the README)

when it comes to retries there is currently a very simple logic based on the following 2 config options: mongodb.max.num.retries (default 3) & mongodb.retries.defer.timeout (default 5000ms). this means if MongoDB is down for more than 15 secs (roughly the time of the 3 retries) the retries are exhausted in the sink connector and it will be killed and waits for manual intervention. the connector would of course continue its work in case MongoDB come back up during the retries. you can raise both config settings as you see fit. what's currently missing is e.g. a better strategy like exponential backoff. there is an open feature request #61  - feel free to help enhance this :)

currently cannot comment on the general DLQ feature of the connect framework itself since I haven't used it for now. be aware though that it may not be available to you in case you are running a version of kafka where this didn't exist yet.
please let me know if this helps or you need anything else. thx.

Hi @hpgrahsl,
Thanks for your reply.  We have some more clarification regarding the connector as mentioned below.

Does the connector guarantees atleast  once semantics?
Is it possible to filter events based on any given parameter. We have use cases where we need to filter the write operations to mongodb collection based on an attribute in the event message.
3.How is the offset management handled by the connector?

yes of course. at least once semantics you get basically out-of-the-box without taking any special care on sink connector configuration. be advices though that in my experience this is very rarely what you want. as I said you can make sure to have idempotent writes by doing key-based upserts just by configuring things accordingly.

neither the kafka connect framework itself nor the sink connector is supposed to do filtering on records. while you could achieve this by implementing your custom write model for the sink connector I would discourage you from doing that. most likely the better way to go here is to have a stream processor (kstreams or KSQL) which takes care of filtering and then have let sink connector process the already filtered topic.

there is no explicit offset management done by the sink connector. it relies on what is configured on framework level i.e. connect commits the offsets in (configurable) regular intervals.

hope this clarifies your questions! if you plan to keep using my sink connector I'd be happy to learn about your concrete use case. ideally you're willing to share it "publicly" as a user voice/testimonial in the READEM. let me know :)

@abhisheksingh87 getting back to you about this. if anything is clarified I'd be happy if you close the issue. THX!


Formatting configuration is treeshaken away

This configuration is currently tree-shaken away, and it causes the formatting decision maker to ignore editor.defaultFormatter. I see that the file is imported here, and the format.contribution file is imported in our contributions. So I assume that it's tree-shaken away, but I don't understand why. Is it because the class is not exported?

Ah, I'm not 100% sure, but does this code mean that we tree-shake Registry.as out?


Local Badge Notification code changes

Issue - https://github.com/microsoft/WindowsAppSDK/issues/138
API Spec - https://github.com/microsoft/WindowsAppSDK/pull/4823/files#diff-17dae42945b77909c6b45f1e7635b67969b8656882ccca5d20c56fd9ab653d6a
A microsoft employee must use /azp run to validate using the pipelines below.
WARNING:
Comments made by azure-pipelines bot maybe inaccurate.
Please see pipeline link to verify that the build is being ran.
For status checks on the main branch, please use TransportPackage-Foundation-PR
(https://microsoft.visualstudio.com/ProjectReunion/_build?definitionId=81063&_a=summary)
and run the build against your PR branch with the default parameters.

i think if we are bringing in the hierarchy of base notification -> badge notification
We should also add other classes for the notification types that we currently support (Toast, raw notifications), Or will be follow up PR ?

/azp run