Managing multiple pacts for the same consumer provider combination

Hi,
is there any way to store multiple pacts within the broker for same provider and consumer? In my current project I've two systems communicating via messages (async) and interactions (request / response). For that I've two consumer tests - one for each type - generating the consumer-provider.json
For testing the provider I need to retrieve both pacts from the broker. Is there a way to distinguish different types of pacts? I already thought about using versions and tags to make this work but this seems no be the correct way.
Thanks,
Peter

Currently, the only way to do it is to have two different consumers defined (like consumer and consumer-async). For the provider verifier, you can get all pacts for a provider from the broker, so it will pick up both pacts when verifying.

Thanks for the answer. I hab this in mind as well but I think the visualisation will look a bit strange when defining different consumers. I'll setup two pact brokers - one for each pact message type.
But it might be some feature to think of in the future to manage different pact types in the broker?

You could use tags to differentiate between the two. You'll find the code
for how to tag pacts in here.
https://github.com/bethesque/pact_broker/wiki/How-to-ensure-backwards-compatibility-by-tagging-pacts.
I would definitely not set up two pact brokers!
What language is your consumer written in? I really need to add the tagging
code to the pact broker client code, but just haven't had time.
On Thu, Oct 20, 2016 at 2:58 AM, Peter Salchow<EMAIL_ADDRESS>wrote:

Thanks for the answer. I hab this in mind as well but I think the
visualisation will look a bit strange when defining different consumers.
I'll setup two pact brokers - one for each pact message type.
But it might be some feature to think of in the future to manage different
pact types in the broker?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bethesque/pact_broker/issues/65#issuecomment-254857677,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAbPFFOi3P1ap4qgQaD4uqYz5gVxmBeBks5q1j4egaJpZM4KYvua
.

My providers / consumers are written in Java. I use the gradle plugin for uploading pacts to the broker.
I had the tagging in mind as well but then I need to maintain two different "version branches" for each type (e.g. 1.0.1a and 1.0.1s) - This doesn't feel like it was made for it... And what is shown in the network graph in that case?
Why wouldn't you setup two different brokers?

It's less than ideal either way isn't it. The broker is supposed to be a
central point of information so that you can see the interactions between
all the services in a given context and how they interact. Using multiple
brokers means you won't be able to do that. But hey, if it solves your
problem better than any other way, then you gotta do what you gotta do.
On Thu, Oct 20, 2016 at 4:35 PM, Peter Salchow<EMAIL_ADDRESS>wrote:

My providers / consumers are written in Java. I use the gradle plugin for
uploading pacts to the broker.
I had the tagging in mind as well but then I need to maintain two
different "version branches" for each type (e.g. 1.0.1a and 1.0.1s) - This
doesn't feel like it was made for it... And what is shown in the network
graph in that case?
Why wouldn't you setup two different brokers?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bethesque/pact_broker/issues/65#issuecomment-255013363,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAbPFHKZuNlmR_INrlGvUqDf1i5yVvN7ks5q1v25gaJpZM4KYvua
.


Does not escape - and / in regexes

The code compiles correctly on<EMAIL_ADDRESS>The code doesn't compiles correctly on<EMAIL_ADDRESS>str.replace(/[\-\/]/g, "$1")

gives the same error as #84

This is probably the smallest reproducible test case I can find for this: /\/\//

Fixed in v1.41.4.


[SPARK-29215][SQL] current namespace should be tracked in SessionCatalog if the current catalog is session catalog

What changes were proposed in this pull request?
when the current catalog is session catalog, get/set the current namespace from/to the SessionCatalog.
Why are the changes needed?
It's super confusing that we don't have a single source of truth for the current namespace of the session catalog. It can be in CatalogManager or SessionCatalog.
Ideally, we should always track the current catalog/namespace in CatalogManager. However, there are many commands that do not support v2 catalog API. They ignore the current catalog in CatalogManager and blindly go to SessionCatalog. This means, we must keep track of the current namespace of session catalog even if the current catalog is not session catalog.
Thus, we can't use CatalogManager to track the current namespace of session catalog because it changes when the current catalog is changed. To keep single source of truth, we should only track the current namespace of session catalog in SessionCatalog.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Newly added and updated test cases.

@imback82 I think this should help to simplify https://github.com/apache/spark/pull/25771
also cc @rdblue @gengliangwang

Test build #111228 has started for PR 25903 at commit e901b89.

Merged build finished. Test PASSed.

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

Test build #111230 has started for PR 25903 at commit 7c8f315.

Merged build finished. Test PASSed.

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

Test build #111228 has finished for PR 25903 at commit e901b89.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class CatalogManager(

Merged build finished. Test PASSed.

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

Test build #111230 has finished for PR 25903 at commit 7c8f315.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class CatalogManager(

Merged build finished. Test PASSed.

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

Test build #111302 has started for PR 25903 at commit b05cb4b.

Merged build finished. Test PASSed.

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

cc @rdblue

Test build #111302 has finished for PR 25903 at commit b05cb4b.

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

Merged build finished. Test PASSed.

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

Test build #111317 has started for PR 25903 at commit 31db62d.

Merged build finished. Test PASSed.

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

Test build #111317 has finished for PR 25903 at commit 31db62d.

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

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

Merged build finished. Test PASSed.

Thanks, merged to master.


[Impeller] Modify display_list_canvas_unittests to run on GPU and Impeller

The DisplayList mechanism has a number of rendering tests that test the assumptions about various rendering behaviors and run automatically during CI under the display_list_rendertests executable.
Currently these tests all run on the Skia software renderer. As a result, a number of the tests have to be disabled because the software renderer has known bugs in certain situations. These tests should ideally run with the Skia GPU renderer which is the renderer under which we want our assumptions vetted.
(A side benefit of using a hardware renderer would be performance - it currently takes over 2 minutes to run all of these tests on a debug unopt build which makes them painstaking for use as a debugging tool.)
Additionally, we should start running these tests using the Impeller back end to check rendering consistency between Skia and Impeller.

The tests have been modified to optionally run on GPU SkSurfaces using the --enable-impeller and --enable-opengl in https://github.com/flutter/engine/pull/38993
It is still not yet possible to run them on Impeller as a rendering engine.


OpenWeather - excessive calls

I received this message from OpenWeather: "We have noticed that your account is currently using One Call API quite intensively and exceed the account limit of 1000 calls per day." I am only using it for this homebridge plugin. Does that seem accurate? Is there a way to limit the calls? Thanks!

Can u post your plugin config (without api key) please?

Hi. Sure, I have pasted it below. I've uninstalled Weather Plus for now, as I could not resolve it. I tried generating a new API key, in case someone somehow got ahold of my key and was using it for something else, but that did not help. OpenWeather let me know that I went from about 360 calls per day to about 8600 calls per day on Dec. 20. I did not make any changes to my config at that time. Thanks for your help.
    {
        "platform": "WeatherPlus",
        "service": "openweathermap",
        "key": "APIkey",
        "locationGeo": [
            49.25657395,
            -122.89168239
        ],
        "conditionCategory": "detailed",
        "forecast": [
            0
        ]
    },

I am having the same issue and received the same email, nothing in my configuration has changed.
    {
        "units": "us",
        "interval": 5,
        "stations": [
            {
                "nameNow": "Winter Park",
                "service": "openweathermap",
                "key": "xyzzy",
                "locationCity": "Fraser, CO, US",
                "language": "en",
                "compatibility": "home",
                "conditionCategory": "detailed",
                "now": true,
                "extraHumidity": false,
                "tresholdAirPressure": 1050,
                "tresholdCloudCover": 3,
                "tresholdUvIndex": 6,
                "tresholdWindSpeed": 10
            }
        ],
        "platform": "WeatherPlus"
    },

I am having the same issue and received the same email, nothing in my configuration has changed.
    {
        "units": "us",
        "interval": 5,
        "stations": [
            {
                "nameNow": "Winter Park",
                "service": "openweathermap",
                "key": "xyzzy",
                "locationCity": "Fraser, CO, US",
                "language": "en",
                "compatibility": "home",
                "conditionCategory": "detailed",
                "now": true,
                "extraHumidity": false,
                "tresholdAirPressure": 1050,
                "tresholdCloudCover": 3,
                "tresholdUvIndex": 6,
                "tresholdWindSpeed": 10
            }
        ],
        "platform": "WeatherPlus"
    },

I received also the eMail just yesterday at 23:51 that I have exceeded 1000 calls per day 🤷🏼♂

One Call API usage alert
Dear Customer!
Thank you for choosing OpenWeather[https://openweathermap.org/]!
We have noticed that your account is currently using One Call API[https://openweathermap.org/api/one-call-api] quite intensively and exceed the account limit of 1000 calls per day.

Config (see below) is set to call every 5 minutes - means 288 times in 24h
I started to use it on Oct. 18 to Nov. 22, then a break (raspi down) and again from Dec. 13
No issues so far - except the 1000 exceed mail.
Will keep it running / monitoring.
Is there a possibility to get a more detailed log ?
When it was triggered and with wich result?
{
"units": "si",
"interval": 5,
"stations": [
{
"nameNow": "WWWeather",
"service": "openweathermap",
"key": "xxxxxyyyyyzzzzzz123",
"locationCity": "xxxxxyyyyyzzzzz, DE",
"language": "de",
"compatibility": "both",
"conditionCategory": "simple",
"now": true,
"extraHumidity": false,
"hidden": [
"Condition",
"Condition Category",
"Dew Point",
"Forecast Day",
"Observation Station",
"Observation Time",
"Ozone",
"Snow",
"Temperature Min",
"Temperature Apparent",
"UV Index",
"Visibility",
"Wind Speed Max"
]
},
{
"nameNow": "Forecast",
"service": "openweathermap",
"key": "xxxxyyyyyzzzz1234",
"locationCity": "xxxxyyyyzzzz, DE",
"language": "de",
"compatibility": "both",
"forecast": [
"Tomorrow"
],
"nameForecast": "+ ",
"conditionCategory": "simple",
"now": false,
"extraHumidity": false,
"hidden": [
"Condition",
"Condition Category",
"Dew Point",
"Observation Station",
"Observation Time",
"Ozone",
"Snow",
"Temperature Min",
"Temperature Apparent",
"UV Index",
"Visibility",
"Wind Speed Max"
]
}
],
"platform": "WeatherPlus"
},

I received also the eMail just yesterday at 23:51 that I have exceeded 1000 calls per day 🤷🏼♂

One Call API usage alert
Dear Customer!
Thank you for choosing OpenWeather[https://openweathermap.org/]!
We have noticed that your account is currently using One Call API[https://openweathermap.org/api/one-call-api] quite intensively and exceed the account limit of 1000 calls per day.

Config (see below) is set to call every 5 minutes - means 288 times in 24h
I started to use it on Oct. 18 to Nov. 22, then a break (raspi down) and again from Dec. 13
No issues so far - except the 1000 exceed mail.
Will keep it running / monitoring.
Is there a possibility to get a more detailed log ?
When it was triggered and with wich result?
{
"units": "si",
"interval": 5,
"stations": [
{
"nameNow": "WWWeather",
"service": "openweathermap",
"key": "xxxxxyyyyyzzzzzz123",
"locationCity": "xxxxxyyyyyzzzzz, DE",
"language": "de",
"compatibility": "both",
"conditionCategory": "simple",
"now": true,
"extraHumidity": false,
"hidden": [
"Condition",
"Condition Category",
"Dew Point",
"Forecast Day",
"Observation Station",
"Observation Time",
"Ozone",
"Snow",
"Temperature Min",
"Temperature Apparent",
"UV Index",
"Visibility",
"Wind Speed Max"
]
},
{
"nameNow": "Forecast",
"service": "openweathermap",
"key": "xxxxyyyyyzzzz1234",
"locationCity": "xxxxyyyyzzzz, DE",
"language": "de",
"compatibility": "both",
"forecast": [
"Tomorrow"
],
"nameForecast": "+ ",
"conditionCategory": "simple",
"now": false,
"extraHumidity": false,
"hidden": [
"Condition",
"Condition Category",
"Dew Point",
"Observation Station",
"Observation Time",
"Ozone",
"Snow",
"Temperature Min",
"Temperature Apparent",
"UV Index",
"Visibility",
"Wind Speed Max"
]
}
],
"platform": "WeatherPlus"
},

I forgot to mention I am running on 1.3.0 beta42

I forgot to mention I am running on 1.3.0 beta42

#135 Same issue with some more info.
Thanks @loudgazelle for providing info on ticket.
Has someone the possibility to provide code for logging the calls
so that we can get / provide more info from homebridge / weather-plus?

#135 Same issue with some more info.
Thanks @loudgazelle for providing info on ticket.
Has someone the possibility to provide code for logging the calls
so that we can get / provide more info from homebridge / weather-plus?

Update from my side.... it seems not to happen anymore.
Might have been an OpenWeather issue....

Hi,
Thanks for the update. I may give it another try.
Matt
On Mar 3, 2021, 1:01 PM -0800, MAPEZ1<EMAIL_ADDRESS>wrote:

Update from my side.... it seems not to happen anymore.
Might have been an OpenWeather issue....
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Seems like the issue has solved itself.
Closing.


account screen layout issue: Component under ComponentList are not rendered properly.

Huh fascinating -- thanks for catching and diagnosing this!
I'm curious what broke it. Do you know anything about what caused this to break? You use the term "regression" in your PR, which sounds like saying that you do.

Also I get different behavior, even on the parent commit of your PR:

Still broken layout, but the information is all there; just not spaced appropriately, and the "Send private message" button is missing. That's on both Android 8 and 9, on an emulated Pixel 2 and a real Pixel 2 XL respectively. So whatever's causing this is at least a little subtle.


Implement upgradable database table

Proposed Change
Allow the table columns to be annotated also with the (table version) version where they were added.
Upgrade the table when new fields are added and table version modified instead of drop-and-recreate.

I'm not quite sure on how to handle the default valuefor version.

The idea I had is that the default value would be 1,which should be the
first db table version. One could mess up the versioning though. In some
cases I drop the table, for others sqlite will throw an exception
On Sep 6, 2016 23:06, "Heinrich Reimer"<EMAIL_ADDRESS>wrote:

I'm not quite sure on how to handle the default valuefor version.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/HeinrichReimer/android-inquiry/pull/12#issuecomment-245091708,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGePUNW9-hdUsM5g4GhL4KDOo4gtl6xWks5qndXsgaJpZM4J2GKi
.


updated code to work witk kotlin 1.3.10 and android 28 api libaries.

Added support for Gatt Characteristic notification enabled with descriptor value.

Not required as it is already implemented by @LouisCAD

There's still API 28 to do, then update sample to AndroidX.
On Thu, Nov 29, 2018, 3:38 AM aravindsg<EMAIL_ADDRESS>wrote:

Not required as it is already implemented by @LouisCAD
https://github.com/LouisCAD
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Beepiz/BleGattCoroutines/pull/27#issuecomment-442684893,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGpvBZpkTUOEHaDDiodLZXwvdwCIowObks5uz0ilgaJpZM4YfbkB
.


How to implement a custom cell with .xib and variable height?

See https://github.com/venmo/Static/pull/45

thanks a lot)

ok, how to implement function or to change variables inside custom cell such as custom selection?
could you add switch as accessory with selection, which can add or remove sections and rows?

I'm going to close this in favor of #45


Customize each tutorial page

[ ] For each tutorial, we also need a high-level description of how the particular virtual tutorial will be held. This will be useful since there are a lot of variations. Some are entirely on zoom, some have pre-recorded videos + live sessions, while some are entirely pre-recorded. Check the tab July 5 on this spreadsheet for this information.

[x] Customize each page depending on whether tutorials are livestreamed, pre-recorded (and/or) have rocketchat.

Tagging @hao-fang incase he has more suggestions.

I submitted a PR for this: https://github.com/acl-org/acl-2020-virtual-conference/pull/237


Lidar simulator for FAST.FARM

Hello @fengguoFUAS
The current release of LidSim is only capable of scanning turbulent wind evolution for a single wind turbine (OpenFAST).
But FAST.FARM reads a global inflow by ignoring the parameters in the individual turbine InflowWind module.
From your recent paper, https://ieeexplore.ieee.org/stamp/stamp.jsp?tp=&arnumber=9867301, I understand that you used FAST.FARM for scanning the wind turbine wake using Lidar Sim.
Is it possible to access the modified version of FAST.FARM that integrates LidSim? I want to apply the LAC module on a wind farm scale.
Thank you,
Yisehak A.

Hi Yisehak A.
Thanks for connecting. Actually, I only used the wake deficit from
FAST.FARM in the paper you mentioned. I then developed a DWM code in matlab
and a simple lidarsim module in Matlab (only considering Line-of-sight
measurement and probe volume, assuming no blade blockage and no nacelle
motion).
I recently heard from NREL that they are trying to implement Lidarsim in
FAST.farm. But I have no idea when it will be ready.
I will finish my university contract soon and cannot continue updating the
lidar simulation module in the near future. So, I am regretful that I could
not help you with the lidar simulation in FAST.farm.
BR
Feng
yisehakabreham @.***> 于2023年1月30日周一 10:10写道：

Hello @fengguoFUAS https://github.com/fengguoFUAS
The current release of LidSim is only capable of scanning turbulent wind
evolution for a single wind turbine (OpenFAST).
But FAST.FARM reads a global inflow by ignoring the parameters in the
individual turbine InflowWind module.
From your recent paper,
https://ieeexplore.ieee.org/stamp/stamp.jsp?tp=&arnumber=9867301, I
understand that you used FAST.FARM for scanning the wind turbine wake using
Lidar Sim.
Is it possible to access the modified version of FAST.FARM that integrates
LidSim? I want to apply the LAC module on a wind farm scale.
Thank you,
Yisehak A.
—
Reply to this email directly, view it on GitHub
https://github.com/fengguoFUAS/OpenFAST3.0_LidSim/issues/4, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AO5GN5UXSLJMOAKSVTTFBADWU6AP5ANCNFSM6AAAAAAUK4JWIU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@fengguoFUAS
Thank you for providing the information. And good luck in your future endeavor.
Kindly,
Yisehak A.


:wrench: Upgrade earthly action for nodejs deprecations

gets rid of these messages

Codecov Report

Merging #985 (fdaa59d) into master (18fcb14) will not change coverage.
The diff coverage is n/a.

: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           @@
##           master     #985   +/-   ##
=======================================
  Coverage   22.79%   22.79%
=======================================
  Files          22       22
  Lines        1610     1610
=======================================
  Hits          367      367
  Misses       1179     1179
  Partials       64       64

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


[BUG] chocolatey 2.1 not working

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have tested that this issue has not been fixed in the latest (beta or stable) release.
[X] I have checked the FAQ section for solutions
[X] This issue is about a bug (if it is not, please use the correct template)

Describe your issue
I am using they system chocolatey installation and it is not showing anything
In the logs I get:
----Chocolatey
Chocolatey v2.1
Invalid argument --locaI-onIy. This argument has been removed from the list command and cannot be used.

Steps to reproduce the issue
Run update on wingetui 2.0.2 with chocolatey 2.1.0 installed
WingetUI Log
copying to clipboard does not work for me

Package Managers Logs
copying to clipboard does not work for me

Relevant information
No response
Screenshots and videos
No response

Hello,
For the time being please try reinstalling WingetUI. Chocolatey should get downgraded automatically.

This issue will be fixed on the next version


Gatsby training

I rebased. @ajayns why was there no pull-request yet?

All the revisions took place on the branch gatsby-training via the epic PR #342.
I think there are a couple of necessary changes, specifically remove gatsby-training's local conf.py and LICENSE, and defer to the repo's root conf.py and LICENSE. If so, then we can make a couple of commits and push to this PR. Please advise. Thank you!

Also for the Travis-CI failure, it looks like there are some issues with make test, but that's from the testing training.
There are some redirects and broken links from linkcheck, too, that we need to fix in the Gatsby training.

@cekk and I were thinking since @polyester was the only student for the training, he'll check out the reference, do a run through and we could make any changes and improvements as he suggests.

After talking to Philip, I left the conf.py and LICENSE as is. Maybe it's necessary, maybe it is, who knows?
I did a quick linkcheck, fixed one broken localhost link to be excluded, but I cannot figure out where this 404 URL should point to instead:
https://travis-ci.org/plone/training/builds/450874698#L1043
writing output... [ 24%] gatsby/searchtraversal
...
(line   13) broken    https://plonedemo.kitconcept.com/en/@search - 404 Client Error: Not Found for url: https://plonedemo.kitconcept.com/en/@search

There's also this odd bird, that does not redirect when requested by humans, but does when requested by linkcheck. I assume the latter uses some HTTP verb beside GET.
https://travis-ci.org/plone/training/builds/450874698#L994

@ajayns yes, see my commit for an example of how to exclude links from 'make linkcheck' in Sphinx's 'conf.py' at the root of the repo. Add the link as a list item. This will remove build warnings and make Travis-CI happy. Thank you!

@ajayns please check your pushed commit. I got an email notification, but the commit does not appear in this PR. Maybe you need to rebase?

I assume it was some issue with GitHub cause I did rebase and did check if it's been pushed properly. Also, I was hoping that'd do the trick but it still shows CI failed.

i relaunched the job and now is green ;)

Does this automatically go live on training.plone ? Cause I can't see it there right now.
Also, @polyester did you get the time to take a look at this?

I've been talking to the Gatsby community as well, and they should be supporting us in sharing this when it's published and we could get some potential alpha users in.


Hot fixes for release 8

The branch is based off release-8.

All look good to me. Not really sure why I made those mistakes.


Apply T! macro where posible

apply T! macro implemented in #1248

Hm, OTOH, T!['{'] is inline with how various parser generators handle things.
Let's just merge this, we can always revert to L_CURLY later :D
bors r+

I've got the feeling that this increases compile time a lot. Is this known?

Hm, no, I haven't check it, will run some benchmarks, thanks for the report.
If you have already measured the impact of this change on compile time, I'd be interested in seeing the results!
We might also look at the travis build times I guess.

Hm, I've checked locally, and compile time seems the same.
Specififivally, I've check f7112371ee646d8ecc68861e9fe832af9729878e
and 64ab5ab10d32e7e8ec085af818d3d94211aea39b by
repeateadly running
$ find ./target/debug -type f -maxdepth 1 -delete && rm -fr ./target/debug/{deps,.fingerprint}/{*ra_*,*test*,*tools*,*gen_lsp*,*thread_worker*}
$ cargo check

and
$ find ./target/debug -type f -maxdepth 1 -delete && rm -fr ./target/debug/{deps,.fingerprint}/{*ra_*,*test*,*tools*,*gen_lsp*,*thread_worker*}
$ cargo test --no-run

in both cases compile itme was more or less the same for both commits
@jens1o could you give a little more details about where you are observing longer compile times?

I digged a little bit further. At some point yesterday, I just quit compiling. I now let it run for some while, only to notice that ra_ide_api results into overflow evaluating the requirement (something with salsa). Might be like https://github.com/rust-lang/rust/issues/47032? I do not know how to get rid of it, though. :/ When it really overflows, then this might be the reason why it takes sooo long.

Hm, whihc version of rustc are you using?

I think this is the same as https://github.com/rust-analyzer/rust-analyzer/issues/1283: on the latest nightly I also get a very long compile time + overflow at the end.

Ah, yes it is! Thank you very much and pardon me!

Thanks for the report! Tacking that overflowing error is important: it has been my main gripe with rustc since all hands :D


Image not uploading in DIO 4 and above.

New Issue Checklist

[ ] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
flutter

Dio Version
4.0.0

Android Studio
Arctic Fox
2020.3.1 Patch 4

Issue Description and Steps
Greeting everyone, my image is not uploading to my server when i update to DIO 4.0.0 and above.  The same code works for DIO version below 4.0.0 . Below is the upload code. Kindly help me out. Thanks
 Future<dynamic> upload(String url, BuildContext context,
      {List<File> files,
      Map body,
      FileCount count = FileCount.MULTIPLE,
      authorize = true}) async {

    BaseOptions options = await getOptions(validate: authorize);

    dynamic response, serverResponse;

    List<MultipartFile> uploadFiles = [];

    FormData requestData;

    try {
      Dio dio = new Dio(options)..interceptors.add(Logging());

      (dio.httpClientAdapter as DefaultHttpClientAdapter).onHttpClientCreate =
          (client) {
        SecurityContext sc = new SecurityContext();
        HttpClient httpClient = new HttpClient(context: sc);
        //allow all cert
        httpClient.badCertificateCallback =
            (X509Certificate cert, String host, int port) => true;
        return httpClient;
      };

      // add files to uploadFiles array
      if (files.length > 0) {
        for (var index = 0; index < files.length; index++) {
          MultipartFile file = MultipartFile.fromFileSync(files[index].path,
              filename: basename(files[index].path),contentType:MediaType.parse('image/jpg'));
          uploadFiles.add(file);
        }

        print("file the request: $uploadFiles");

        requestData = FormData.fromMap({"file": uploadFiles});
      } else {
        requestData = FormData();
      }

      body.forEach((key, value) {
        MapEntry<String, String> data = new MapEntry(key, '$value');
        requestData.fields.add(data);
      });

      print("sending the request: ${uploadFiles.asMap().values}");

      serverResponse = await dio.post(
        url,
        data: requestData,
        options: new Options(contentType: 'multipart/form-data'),
        onSendProgress: (int sent, int total) {
          print('$sent $total');
        },
      );

      print("Server Replied: $serverResponse}");

      print(serverResponse.data);

      //check for connection errors
      if (serverResponse.statusCode < 200 || serverResponse.statusCode > 400) {
        return _decoder.convert(
            '{"success":false,"message":"${LocalText.of(context).load("error_executing_request")}"}');
      }

      response = _decoder.convert(serverResponse.data);

      print("Response: $response");
    } on DioError catch (e) {
      // The request was made and the server responded with a status code
      // that falls out of the range of 2xx and is also not 304.
      if (e.response != null) {
        print(e.response.data);
        print(e.response.headers);
        // print(e.response.request);
        response = _decoder.convert(
            '{"success":false,"message":"${LocalText.of(context).load("server_error")}"}');
      } else {
        // Something happened in setting up or sending the request that triggered an Error
        print(e.message);
        response =
            _decoder.convert('{"success":false,"message":"' + e.message + '"}');
      }
    } catch (error) {
      return _decoder.convert(
          '{"success":false,"message":"${LocalText.of(context).load("error_executing_request")}"}');
    }
    return response;
  }

Need more information, is there a server response? is the request not starting? some exception?

yes there is a server response @kuhnroyal and the fields are working perfectly except for the image.  Below is the screenshot please.

Try to use an http poxy to figure out if the file is actually being sent to the server or if something wrong is being sent, or nothing at all.
I don't see anything wrong, but it is hard to tell without a working example.

File is not sent to the server when using DIO 4 and above. But is it working with DIO 3.0.10 and below @kuhnroyal

File is not sent to the server when using DIO 4 and above but the fields works though. Both file and fields are working with DIO 3.0.10 and below.

I, kind of, have the same issue at https://github.com/flutterchina/dio/issues/1444


Initial version of parallelized FSCK

The TODO section at the end of validateVolume() in fs/treewalk.go
defines the current view of remaining work.
As for as how this new version matches up to the functionality it
is replacing, only the LinkCount adjustment has not been applied.

This version of FSCK is by no means complete... lots of TODOs remain as to what I'm hoping to do.
I just wanted to get this out there for others to see what's going on in the code base re FSCK.
The key change is there is this "goroutine" strategy to launch at every opportunity.
Each inode is validated in a distinct goroutine. Each dir found during treewalk gets its own goroutine.
To limit parallelism, there is a "gate" / "turnstile" per parallelized function arbitrarily tuned at present.
Please let me know what you think - comments are welcome!

Ok - getting there...
The remaining TODOs (10) are all beyond what the original FSCK performed.
Hence, this PR technically could be merged at any point moving forward.
The benefit is parallelism... and providing the framework for the remaining work.
But I'll just keep working on those TODOs one by one in the meantime.
Review comments very welcome!


fix(baremetal): register baremetal name invalid

这个 PR 实现什么功能/修复什么问题:

是否需要 backport 到之前的 release 分支:

3.6

Codecov Report

Merging #11045 (bd3c6cf) into release/3.6 (b9d30c0) will increase coverage by 10.11%.
The diff coverage is 0.55%.

@@               Coverage Diff                @@
##           release/3.6   #11045       +/-   ##
================================================
+ Coverage         5.86%   15.98%   +10.11%
================================================
  Files              828      125      -703
  Lines           129133    12837   -116296
================================================
- Hits              7577     2052     -5525
+ Misses          120876    10594   -110282
+ Partials           680      191      -489

Flag
Coverage Δ

aFlag
15.98% <0.55%> (+10.11%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/apis/const.go
0.00% <0.00%> (ø)

pkg/apis/input.go
0.00% <ø> (ø)

pkg/apis/list.go
0.00% <ø> (ø)

pkg/baremetal/tasks/baseprepare.go
0.00% <0.00%> (ø)

pkg/baremetal/tasks/bm_register.go
0.00% <0.00%> (ø)

pkg/baremetal/tasks/ipmiprobe.go
0.00% <0.00%> (ø)

pkg/baremetal/tasks/stop.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/quotas/errors.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/quotas/handler.go
0.00% <0.00%> (ø)

pkg/cloudcommon/db/quotas/register.go
11.11% <0.00%> (-2.41%)
:arrow_down:

... and 668 more

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 a150d04...bd3c6cf. Read the comment docs.


fix(deps): update module github.com/lmittmann/tint to v1.0.6

This PR contains the following updates:

Package
Type
Update
Change

github.com/lmittmann/tint
require
patch
v1.0.5 -> v1.0.6

Release Notes

lmittmann/tint (github.com/lmittmann/tint)
v1.0.6
Compare Source
What's Changed

Only convert to any on KindAny values by @database64128 in https://github.com/lmittmann/tint/pull/83
Skip TestReplaceAttr without faketime by @database64128 in https://github.com/lmittmann/tint/pull/84
Avoid allocating tintError twice by @database64128 in https://github.com/lmittmann/tint/pull/82

New Contributors

@database64128 made their first contribution in https://github.com/lmittmann/tint/pull/83

Full Changelog: https://github.com/lmittmann/tint/compare/v1.0.5...v1.0.6

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Successfully tested usage of updated library


Allow removal of peer certificate to reduce RAM usage

Context: This PR belongs to a series of RAM usage improvements targeting X.509, the first two being #2391 and #2411. The PR is based on #2391 (which is based on #2411).
Goal: Save RAM by not permanently storing the peer's CRT chain unless needed by the user.
Summary: The PR adds a new compile-time option MBEDTLS_SSL_KEEP_PEER_CERTIFICATE, enabled by default, controlling the availability of the existing public API mbedtls_ssl_get_peer_cert() which allows the user to inspect the peer's CRT at any time during the connection. If it is disabled, the PR modifies the stack to parse, copy and keep the peer's CRT only to the extended needed to preserve functionality and security (see below).
Background: After parsing, the peer's CRT is mainly used in the following places:

Certificate verification
The underlying public key is always needed at some point to directly or indirectly verify that the peer possesses the corresponding private key. For example, there are signatures in ServerKeyExchange and CertificateVerify to be verified, or a PMS to be generated in RSA-based key exchanges.
Mitigation of the triple handshake attack: The peer's CRT must not change during renegotiation, and to that end, the peer's CRT is currently stored as part of mbedtls_ssl_session.
The user can query it via mbedtls_ssl_get_peer_cert().

Approach: Remove the peer CRT from the session structure while adapting the above uses as follows:

Verification doesn't need the CRT to be part of the long-lived session structure, as it's local to the handling of the incoming Certificate state. We can parse and verify the peer's chain in place when reading it from the input buffer, and free all involved structures afterwards.
Store a handshake-local copy of the peer's public key which is extracted from the peer's CRT after the parsing and verification step, and free it as soon as it's no longer needed. For example, during an (EC)DHE exchange, the server's ServerKeyExchange signature is verified before performing the RAM-expensive (EC)DHE computation, so the peak RAM usage drops if we free the server's public key after handling the ServerKeyExchange.
Instead of storing the entire CRT in the session structure, store a digest only.

Technical steps: The PR proceeds along the following steps, with the removal of the peer CRT field happening at the very end only:

Take the opportunity to restructure and simplify the handling of the incoming certificate state. Several helper functions are introduced for coordination (do we expect a CRT?), no-CRT notification from the client, and verification.
Add fields to mbedtls_ssl_session for storing the digest of the peer's CRT. Compute and store the digest after CRT parsing and verification, and mitigate the triple handshake attack by comparing hashes, not the actual CRTs. This leaves the existing peer CRT field in mbedtls_ssl_session untouched.
Session cache and session tickets need to be adapted to any change of mbedtls_ssl_session. Adapt them to store the new digest field in place of the peer CRT field if MBEDTLS_SSL_KEEP_PEER_CERTFICATE is disabled. The peer CRT field continues to remain in mbedtls_ssl_session, but it's not kept during session resumption. Note that the only use of the peer CRT across session resumption apart from the user calling mbedtls_ssl_get_peer_cert() is to mitigate the triple handshake attack, so while this change removes the peer CRT from ticket and cache, the library's functionality is maintained.
Add a PK context to mbedtls_ssl_handshake_params to hold the peer's public key. Extract it after parsing and verifying the CRT. Go through all functions using the peer's public key, and use the new PK context instead of the PK context embedded in the peer's CRT (which is still there).
Actually remove the peer CRT from mbedtls_ssl_session, and the final adaptations, like modifying mbedtls_ssl_session_{init/free}(), or disabling some tests which grep for CRT info output from ssl_client2 and ssl_server2.

Todo:

[x] Add tests to all.sh
[x] Enable MBEDTLS_SSL_KEEP_PEER_CERTIFICATE by default. It is currently disabled to have the CI test it.
[x] Free peer's public key as soon as it's no longer needed. So far, it's freed at the end of the handshake only.

Travis failed because of wrong compile-time guards around a helper function.

CI succeeded on default config with MBEDTLS_SSL_KEEP_PEER_CERTIFICATE disabled.

Jenkins failed because of timing on FreeBSD only.

@k-stachowiak Thank you very much for your review; I have addressed your comments. Could you please re-review?

@k-stachowiak I believe to have addressed your comment on ssl-opt.sh. Please re-review.

The pre-requisite PRs #2391 and #2411 have been merged, so removing "need: preceding PR".

(Changed base twice in order to refresh github's cache.)

@hanno-arm tests/scripts/check-generated-files.sh failed in Travis. Please investigate and fix.

@mpg Yes, will do - I don't expect it to be significant, to if you don't mind, please don't hold your review for that.

Sure, most probably a simple matter of re-running the appropriate script. Don't worry, I'm not letting that delay my review :)

@mpg Thank you for your review, I believe to have addressed your comments. Could you please re-review?
@k-stachowiak Could you please review the latest commits, too?

Marking this as approved for design as I found no design flaw in my review.

@mpg Thank for you for the re-review, I've addressed your comments.
@mpg @k-stachowiak Please re-review.

@k-stachowiak @mpg I had to rebase this PR on top of development to fix the CI failure. Could you please re-review? The rebase is entirely conflict-free, and I don't see any places where we'd need adaptation to the PSA changes.

@hanno-arm There are CI failures that look serious and need fixing.

tests/scripts/test-ref-configs.pl config-ccm-psk-tls1_2.h fails with:

/var/lib/build/programs/ssl/ssl_client2.c:2150:43: error: use of undeclared identifier 'peer_crt_info'

        memset( peer_crt_info, 0, sizeof( peer_crt_info ) );

                                          ^

/var/lib/build/programs/ssl/ssl_client2.c:2150:17: error: use of undeclared identifier 'peer_crt_info'

        memset( peer_crt_info, 0, sizeof( peer_crt_info ) );

                ^

/var/lib/build/programs/ssl/ssl_client2.c:2223:43: error: use of undeclared identifier 'peer_crt_info'

        memset( peer_crt_info, 0, sizeof( peer_crt_info ) );

                                          ^

/var/lib/build/programs/ssl/ssl_client2.c:2223:17: error: use of undeclared identifier 'peer_crt_info'

        memset( peer_crt_info, 0, sizeof( peer_crt_info ) );

                ^

4 errors generated.

(Found by Jenkins  cmake-asan-clang-debian-9-i386, cmake-asan-clang-debian-9-x64, cmake-full-gcc-debian-9-i386 and Travis Clang.)

A variant was found by the Travis GCC job and Jenkins cmake-full-gcc-debian-9-x64 with tests/scripts/test-ref-configs.pl now failing in config-thread.h (probably just because Perl randomize hashes enumeration so a different failure was encountered first) with:

/home/travis/build/ARMmbed/mbedtls/programs/ssl/ssl_client2.c: In function ‘main’:
/home/travis/build/ARMmbed/mbedtls/programs/ssl/ssl_client2.c:2150:17: error: ‘peer_crt_info’ undeclared (first use in this function)
         memset( peer_crt_info, 0, sizeof( peer_crt_info ) );
                 ^
/home/travis/build/ARMmbed/mbedtls/programs/ssl/ssl_client2.c:2150:17: note: each undeclared identifier is reported only once for each function it appears in

@hanno-arm Looks like the fix is simple enough: this identifier is only declared when MBEDTLS_X509_CRT_PARSE_C is defined, so the calls to zeroize should be guarded the same way. Sorry for not catching that during review.

@mpg Thanks for spotting, and sorry for not noticing this earlier.
@mpg @k-stachowiak Could you please re-review the latest commit?

This is a new feature we want, has two approvals and a passing CI, a ChangeLog entry and new tests in all.sh, so it's ready for merge.

Release job failed because of unused variable warning in client-only build.

The last commit has been approved and passed the CI. This is now ready for merge again.


[Button group]: mobile width

User story
We need to review and update the button group mobile width logic, the original specs were based on older screen sizes and are not aligned to the current screen sizes
Additional information
Current
The current specs cap the width of the button group to 288 or 320px (small breakpoint)
This results in the button and button group not spanning full width on newer mobile devices like iPhone 12 Pro or larger, and leaves a gap on the far right which looks like a mistake.

Sm breakpoint 320px (iPhone 5)
Mid sm breakpoint 390px (iPhone12)

Explorations / tasks

[ ] Audit common screen sizes (use inspect tool to identify pixel sizes)

iPhone
Android (Galaxy, Pixel, etc)

[ ] Test button group width on common screen sizes

Should we cap width at iPhone12 Pro?
Does button group work better side by side for iPhone Pro Max or iPhone Pro Plus?

[ ] Breakpoint problems

Sm – Md would result in a massively wide button if we span all 4 columns
Custom breakpoint?

Does a solution require a custom breakpoint?
Could the solution be a max width of identified cap/common screen size? (iPhone X / Galaxy X / Pixel X)

Acceptance criteria

[ ] Research for other button pairs collected
[ ] Explorations for new solution shared with Carbon design team
[ ] Specs updated or follow on issue created

Reference Card in card figma file for breakpoint ideas

Additional button & icon alignment issue, see Slack

Review – Team Huddle Jun 20
Width

Button group will span full width on small breakpoints

Icon alignment

Look into option for switching lg for 2xl variant when line wraps
Discuss with system squad updating smaller buttons to allow for wrap


Add subtuple helper to remove layers of Concat, fix bug in loop inv motion

Looks like there are more bad substitutions to track down!

Fixed another substitution bug, and also fixed the snapshot tests to have "snapshot" in the name for easier testing


Assignment changes values copied, CPU

Array value assignment changes the copied values when running on CPU using python wrapping:
a=af.randu(2,2,2)
print a[1]
a[0] =a[1]
print a[1]
After the assignment, the values of a[1] changed unexpectedly. This behaviour is not observed running on CUDA backend.
I am using ArrayFire v3.5.0 (CPU, 64-bit Linux, build 251e942) on Intel(R) Xeon(R) CPU E5-2630 0 and the current python wrapper development version.

@pthon this was a bug in upstream arrayfire. It's been fixed and is available in the devel branch.


Fixing instructions to install chart

This chart lives in the stable folder, but the instructions tell you to use the incubator folder.  This fixes that.

@k8s-bot ok to test

Excellent. Thanks for the fix.


Conditional Content

Is the Conditional content feature only available to rea-time emails? Is there a reason why Marketing emails does not have such function?

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

ID: 04be13b8-4247-00d1-88ff-81ed49619c9b
Version Independent ID: 04be13b8-4247-00d1-88ff-81ed49619c9b
Content: How to use conditional content (Dynamics 365 Marketing)
Content Source: ce/marketing/conditional-content.md
Service: dynamics-365-marketing
GitHub Login: @alfergus
Microsoft Alias: alfergus

Hey @jaimieji -- it's only available in real-time marketing emails and, as far as I know, there aren't plans to add it to outbound marketing.
If you'd like to make a product request, I'd make a posts on the Dynamics 365 Marketing forum at https://community.dynamics.com/365/marketing/f/dynamics-365-for-marketing-forum - repost your question there and a developer will be able to help you.

Hi Alex,
Thank you. Though this is not a developer help to solve.
The lack of conditional content function like real time marketing not only stop the product to compete with the likes of spotler, but also make it hard for marketing users to build their marketing emails. We cannot just ask the users to learn html conditions provided and it is not intuitive to do. It introduces way too much work for the same thing to be done for marketing users with many campaigns to run and setup through out the year.
Regards
Jaimie
Sent from Outlook for Androidhttps://aka.ms/AAb9ysg

From: Alex Ferguson @.>
Sent: Friday, March 24, 2023 10:43:06 PM
To: MicrosoftDocs/dynamics-365-customer-engagement @.>
Cc: jaimieji @.>; Mention @.>
Subject: Re: [MicrosoftDocs/dynamics-365-customer-engagement] Conditional Content (Issue #2948)
Hey @jaimiejihttps://github.com/jaimieji -- it's only available in real-time marketing emails and, as far as I know, there aren't plans to add it to outbound marketing.
If you'd like to make a product request, I'd make a post on the Dynamics 365 Marketing forum at https://community.dynamics.com/365/marketing/f/dynamics-365-for-marketing-forum - repost your question there and a developer will be able to help you.
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/dynamics-365-customer-engagement/issues/2948#issuecomment-1483529525, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAVQVWUAOWEXSBSFWBKLFSTW5YPPVANCNFSM6AAAAAAWGR4SF4.
You are receiving this because you were mentioned.Message ID: @.***>


Test image loading

WHAT
WHY
HOW
Checklist :clipboard:

[ ] Add explicit visibility modifier and explicit return types for public declarations
[ ] Run spotless check
[ ] Run tests
[ ] Update metalava's signature text files

Thank you helpful PR, godspeed.


(#361) Validate cluster name in cluster config yaml

#361

Added validation of cluster name in the cluster config yaml
Added unit tests to cover invalid cluster name in config including a period (.)

@vivek-koppuru @jaxesn @ndeksa @abhay-krishna @ptrivedi
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for picking this up! This has been a long standing issue for us that we never got around to.
/ok-to-test

So glad to do it! Thanks for taking the time to review!

/lgtm


How can IMigrationsModelDiffer be injected in SqliteMigrationsSqlGenerator?

For creating table rebuilds to get unsupported SQLite migrations working i need IMigrationsModelDiffer in SqliteMigrationsSqlGenerator
Since version 2.1.0 IMigrationsModelDiffer itself has a lot more dependencies and it seems like it should be ctor injected instead of manually newed up.
Problem is when I do this:
        public SqliteMigrationsSqlGenerator(
            [NotNull] MigrationsSqlGeneratorDependencies dependencies,
            [NotNull] IMigrationsAnnotationProvider migrationsAnnotations,
            [NotNull] IMigrationsModelDiffer modelDiffer)
            : base(dependencies)
            => _migrationsAnnotations = migrationsAnnotations;

without any further code being added, there is an exception during startup.
MyDbContext.Database.Migrate() throws
System.ArgumentNullException: 'Value cannot be null.
Parameter name: provider'

What is the proper way of accessing IMigrationsModelDiffer  in SqliteMigrationsSqlGenerator?
Further technical details
EF Core version: 2.1.0
Database Provider: Microsoft.EntityFrameworkCore.Sqlite
Operating system: win10 1709
IDE: Visual Studio 2017 15.7.3

@bricelam to provide some guidance here.

IMigrationsModelDiffer is a scoped service while IMigrationsSqlGenerator is a singleton, so it can't be injected. Do you really need to diff models? If you just need to go from an IEntityType to a CreateTableOperation, the code should be trivial enough just to copy into SqliteMigrationsSqlGenerator.
When implementing #329, I expect I'll refactor that code into a component shared by both the differ and the Migrations SQL generators.

I tried copying the relevant parts from MigrationsModelDiffer but it is just not feasible, after copying the 10th depending method I ask myself why there is DI in the first place and what service scopes I need to change to make it work again.
In 2.0 this was way simpler, but now in 2.1 it is just not possible for someone without in-depth knowledge without burning away endless hours.
What is a straight forward pragmatic approach to "just" use the IMigrationsModelDiffer via injection? I do not care about scope-correctness or potentially wasteful injected services during runtime.

@ajcvickers Is it possible for applications to change the lifetime of a service?

@bricelam Yes, but it is not recommended since it can break things in quite unexpected ways. To do it, the application needs to take over building the internal service provider and register the service with the scope that they want before calling AddEntityFrameworkXxx. Since we TryAdd everything, if the service is already registered, then EF will leave the registration alone.

@leak Register IMigrationsSqlGenerator and any of its dependencies as Scoped.

Looks like adding those manually gets me straight trapped in dependency hell (dependencies of dependencies of dependencies....)
            services.AddScoped<CommandBatchPreparerDependencies>();
            services.AddScoped<IChangeDetector, ChangeDetector>();
            services.AddScoped<StateManagerDependencies>();
            services.AddScoped<IRelationalTypeMappingSource, SqliteTypeMappingSource>();
            services.AddScoped<TypeMappingSourceDependencies>();
            services.AddScoped<ValueConverterSelectorDependencies>();
            services.AddScoped<IValueConverterSelector, ValueConverterSelector>();
            services.AddScoped<RelationalTypeMappingSourceDependencies>();
            services.AddScoped<MigrationsAnnotationProviderDependencies>();
            services.AddScoped<IMigrationsAnnotationProvider, MigrationsAnnotationProvider>();
            services.AddScoped<IMigrationsModelDiffer, MigrationsModelDiffer>();

This is what I'm currently adding at the end of ConfigureServices() and there are still errors about missing dependencies trying to get the differ out of DI without exceptions...
            var host = CreateWebHostBuilder(args).Build();
            var differ = host.Services.GetService<IMigrationsModelDiffer>();

I guess by the time i have them all listed my list is equally long to what EF would have built on its own...
I was already struggling with this in 2.0. Back then I was getting away with injecting IMigrationsAnnotationProvider into SqliteMigrationsSqlGenerator and then instanciating the differ myself  var modelDiffer = new MigrationsModelDiffer(Dependencies.TypeMapper, _migrationsAnnotations);
Now in 2.1 EF seems fully trapped in dependency hell where individual parts become unusable outside of their intended scope..

@leak It is certainly feasible to replace EF services with similar services that have different behavior. This should not involve changing scope. On the other hand, where an architectural decision has been made about how these services should function and work together as a system, then attempting to do something that goes against that decision is generally not feasible. In some cases this includes changing the scope of services, since that is fundamental to the internal architecture of EF Core.
In cases like this, a better approach would be to:

Attempt to see if what you want to do can fit into the existing architecture/design. In this case, that means not using the differ in SQL generation.
If this leads to an understanding of how the architecture can be improved, then create an issue for this proposed architecture/design change in order to get feedback from the team and community. For example, in this case that might be the refactoring that @bricelam suggested.
Assuming the feedback is positive, then consider making a PR for the change, such that you can then extend EF in the way you want.


Can you provide convenience for developers?

Versions
<!--
Output from: `ng --version`.
If nothing, output from: `node --version` and `npm --version`.
  Windows (7/8/10). Linux (incl. distribution). macOS (El Capitan? Sierra?)
-->

Repro steps

When I run ng serve, I want to use the cell phone to open it(My computer and phone in the same Wifi).
Like reactjs. When you run 'npm start' ,you can see like this:

Angular is this:

Can you provide convenience for developers?
Observed behavior
<!-- Normally this includes a stack trace and some more information. -->

Desired behavior

Mention any other details that might be useful (optional)

Good point. Btw, it's even more convenient to use your computer name instead of your local IP address (because the IP address can change but the name doesn't). Try it.
Assuming your computer name on the network is jiangslaptop try visiting http://jiangslaptop:3000/ on your cell phone.

Btw, I suggest you rename the issue to something like "ng serve should show an URL with the local IP address for convenience"


image embeddings blog post

Blog post structure :

What are embeddings ?

Let's build embeddings from image
2.1 download
2.2 inference

Knn

What can be done with this ?

Other ways to build image embeddings

Announcement of future blogpost about embeddings in general

quite advanced, now that the UI is available, just some finish touches before this is done and the main goal is achieved

done https://medium.com/@rom1504/image-embeddings-ed1b194d113e


Blazor link updates

Addresses #23009

@guardrex, I was already doing a larger scope broken links sweep in #23009, I will continue with that but remove the 6 blazor topics you have here fixed in this PR.  They had moved locations while I had the PR open and so I was having to redo them anyway.  I will remove just these 6.  Thanks!


Add wallet auto connection

:clipboard: Add associated issues, tickets, docs URL here.
Overview
Describe what your Pull Request is about in a few sentences.
Remove the logic of refreshing the window if users switch chains.

Note: window is refreshed only when accounts are changed. This can be improved too, but it depends on how urgent it is.

Changes
Describe your changes and implementation choices. More details make PRs easier to review.

Add disconnection button
Add logic of firing initializeBase and initializeAccount when the chain id is changed

Testing
Describe how to test your new feature/bug fix and if possible, a step by step guide on how to demo this.

Codecov Report
Patch and project coverage have no change.

Comparison is base (fbfd04d) 48.51% compared to head (cf39fa0) 48.51%.

: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
Additional details and impacted files
@@                 Coverage Diff                  @@
##           feat/wallet-connect     #647   +/-   ##
====================================================
  Coverage                48.51%   48.51%
====================================================
  Files                      132      132
  Lines                     5551     5551
  Branches                  1123     1123
====================================================
  Hits                      2693     2693
  Misses                    2858     2858

Flag
Coverage Δ

coverage
48.51% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Pinning custom Collector

Hello
I have trouble with using a custom Collector inside a data structure I'm writing (contrie).
I'm not sure if I'm just reading the docs wrong, or if the API is actually missing. I've also tried to look into crossbeam-skiplist, but that part seems to not be implemented yet.
I would like to put a Collector inside the data structure, for two reasons:

Being able to perform all delayed destructions in the drop, so I can store non-'static types safely.
My iterator type contains a Guard, so if someone holds it for too long or leaks it, there are going to be problems with memory reclamation in the whole application. Using a private one could at least localize the impact.

I can put the Collector into the struct. However, to get a Guard, I first need to register a LocalHandle. I assume it is relatively expensive call, so I don't think it is meant to be called for each pin call, eg in each method of the map. On the other hand, if I create one and want to reuse it, I have nowhere to put it and asking the user to keep it around doesn't look convenient either.
What is the expected way to do this? What is the plan for crossbeam-skiplist (which, according to a TODO) seems to plan to do a similar thing? If the API is missing, is there the design for it somewhere (depending on my free time and complexity of the surrounding code, I might try to implement it)?

Note: I'm just another crossbeam user. It seems like you might want to cache a local handle it some sort of thread local thing which you can then call pin on. Also interested in this since I'm the author of ccl and The Concurrent Map Competition and I'm looking to do something similiar for one of the datastructures I'm currently developing.

Yes, but the usual thread local variable is somewhat globalish thing ‒ If I have two map instances, they don't get a separate thread local variable each. Having a map ThreadId→handle inside map feels somewhat like a chicken-egg problem, doesn't delete the ones for deleted threads and adds another map lookup ☹. So that doesn't sound like a proper solution.
So I simply hope there's some way how it is supposed to be done, maybe just not yet written :-).

Is nobody able to answer? Or, at least, tell me nobody thought about a way yet? :-(


Add PHP 3LO and API key auth support

Current coverage is 81.27% (diff: 100%)

Merging #456 into disco_head will not change coverage

@@           disco_head       #456   diff @@
============================================
  Files             183        183
  Lines            6971       6971
  Methods             0          0
  Messages            0          0
  Branches          909        909
============================================
  Hits             5666       5666
  Misses           1033       1033
  Partials          272        272

Powered by Codecov. Last update 51f8f4f...c38a017

LGTM

Most APIs passed in compilecheck. Some failed, and I didn't scrutinize the error messages, but I believe they were due to type name mismatches.

Were these mismatches caused by the auth stub or by issues with the client libraries trailing Discovery?


Error on 1.8.9 client 1.7.4 server

Latest 5zig mod and latest ClientViaVersion

log: https://paste.ee/p/o6frS
When I click with an empty hand on any block this problem appears

Bed not working

log: https://paste.ee/p/E0epf

villager not working

enchantment table not working

anvil not working


Model could not be loaded. Please try another model.

Windows 10, Python 3.7.8, AvrilAI 1.5, just installed from scratch today.
Followed the instructions for the install, am getting this error message scrolling off the screen on first running "python launch.py":
Model could not be loaded. Please try another model.
I did run pip install -r requirements .txt and did not get any error messages beforehand.
It also says :09/01/2022 08:50:13 - WARNING - CUDA is not available, you are limited to CPU only. I do have a 3090, does this not support running on NVIDIA cards?
Thanks

If you have CUDA installed for your model it should work. Also, make sure to download the model.bin file listed in the readme. Do note, however, that I am now part of the KoboldAI team and much of the tech here has been merged into it. It handles models a bit better, too.


Improved search and store?

[ ] Key word search and dataset filtering

[ ] Shortlist function (may require user registration)

This is a jkan and other implementations issue now, I would remove.


fix fix auto analyze triggering failure

StatisticsCollector only collect when replicas are normal. In cloud mode, this check needs setCloudCluster before execution. Otherwise, check will be a dead-end failure.

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


feat(topology): complete integration of unified topology

This completes the SDAM refactor (sans authentication) for the native driver. You will note that support has been added to the test configuration to determine whether the unified topology is being used, and a number of tests are being skipped for the unified topology: this is temporary in most cases.
NODE-1291
NOTE: This will not pass CI until the mongodb-core PR is merged

@daprahamian @kvwaleker this is all passing, and addresses Dan's comments. I uncovered some further issues tonight and created NODE-1682 and NODE-1683 to track them, but think we should go ahead and merge this after a final pass by y'all.


clean up ZCL_ABAPGIT_OBJECT_CLAS_OLD

ZCL_ABAPGIT_OBJECT_CLAS_OLD has not been in use for a while, delete
note that _NEW inherits from _OLD, move everything to ZCL_ABAPGIT_OBJECT_CLAS

I will give this a shot

I have a problem.
in the constructors,
ZCL_ABAPGIT_OBJECT_CLAS_OLD does this:
mi_object_oriented_object_fct = zcl_abapgit_oo_factory=>make( ms_item-obj_type ). which ends up instantiating zcl_abapgit_oo_class and ZCL_ABAPGIT_OBJECT_CLAS does this: create object mi_object_oriented_object_fct type zcl_abapgit_oo_class_new.
I won't copy that line from _old as seems pointless, but I wonder if we should decommission/replace zcl_abapgit_oo_class

merged, closing


Announce: framework documentation

I drafted some pages to document libui-node usage:
https://github.com/parro-it/libui-node/tree/master/docs
It would be beautiful to get feedback / suggestions / correction by the community...

Looks great! I'd love a doc on the Area Component though :)

Also the Grid page is missing documentation on .append

Thank you Paul! Area Component docs are coming soon.

Great!


Over Quota

Hello,
When I try to increase size for specific user I get below error.
Saving settings has failed. (Over quota)
Any idea how to override it or fix it?
I read that it's a known issue in old version but I have updated version.
Current Version attached.

Have answered this at: https://github.com/afterlogic/aurora-files/issues/61


Unable to connect to Home Assistant in webview

HA Client version: 1.0.0
Home Assistant version: 0.109.4
Device name: Samsung J3 2016
Android version: 5.1.1
Description
Can't load anything in webview.
All other things working perfect.
Screenshots

Hi @LiJu09
Is it still an issue? If yes please tell me:

How do you connect the app? Is it local IP, duckdns or Remote UI?
Are you able to view your web UI in browser using the same url you are using in the app to connect?

I am connecting using local IP.
I can view web UI using browser on mobile and through official app.
Also I tried same using Samsung A3 (2017) - Android 8.0, App version 1.0.0 and it works.

I'll try to fix this in 1.1.0

Possibly fixed. Will be included in 1.1.0


regex for block tags fixed to include h1-h6

Added support for heading tags to be matched as block tags. Headings were matched against regex for open and close tags so content inside was pushed to new line. This isn't a big deal, but for our project we needed it to be placed inside one line.

@predragleka Please remove the .idea addition for .gitignore.

Done. Sorry for that, did it out of habit.

Didn't figured out that regex is used in test. I fixed test regex to reflect new updated in Indenter class


chore: probably dead links

Actually there are other dead links if you want I can fix them

These links are both still valid - both link to working pages:
http://wiremock.org/docs/configuration/
http://wiremock.org/docs/junit-rule/

Wow they didn't work yesterday. This is strange, but yes you are right.


Refactor TestCacheWorkloadOperations

What type of PR is this?

/kind cleanup
What this PR does / why we need it:
Refactored TestCacheWorkloadOperations so that each test case is independent of the others. I also added some missing test cases as well. Thanks for your review!
Which issue(s) this PR fixes:

Fixes https://github.com/kubernetes-sigs/kueue/issues/72
Special notes for your reviewer:

/ok-to-test

@denkensk Thank you for your review! I've updated the PR so would you review it again? 🙏

@denkensk Thank you. Squashed!


not able to use github.issues.addAssigneesToIssue

Hey! I'm not sure if i'm missing something, but i can't assign myself an issue in one of my own repos.
The main difference i see is that, on github's API version 3, to assign someone to an issue, the request does a
PATCH /repos/:owner/:repo/issues/:number
but reading in the documentation, the request goes to
POST /repos/:owner/:repo/issues/:number/assignees.
Any feedback i can use??
Thanks!

The api endpoint matches what's in the github api doc: https://developer.github.com/v3/issues/assignees/#add-assignees-to-an-issue. Do you have a simple example with output I can take a look at?

Are you trying to call the edit issue endpoint? That would be issues.edit.
The assignee field is deprecated but it should work.
https://developer.github.com/v3/issues/#edit-an-issue

im using the add-ssignees-to-an-issue, and in the meta keys shows the 201 status code but when i see the issue on github, it doesn't show any assignees. and when i try to use the edit an issue, it throws an promise exception

You're seeing it do PATCH /repos/:owner/:repo/issues/:number when you call the api endpoint? Are you using the latest version? Also, what's the exact exception you're getting?
Just to verify: https://mikedeboer.github.io/node-github/#api-issues-addAssigneesToIssue

i managed to figure out the exception, and it was my mistake. Between using issue.edit and issue.addAssigneesToIssue i was using other thing so it'd never work jeje
thanks for so quick response, Joe!

Ah good, glad it's resolved!


⚠ Bofrost.fr has degraded performance

In ea3d006, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1139 ms

Resolved: Bofrost.fr performance has improved in 3a80318 after 308 days, 1 hour, 22 minutes.


Add more disposable domains from https://emailfake.com/

Thanks!
So you know in the future, we match against wildcard subdomains so it's not necessary to include all the n.emailfake.ml for example

Oh I see, thanks for clarify.

v1.2.14 is available now from rubygems with these additions

Nice, thanks!


Support url parameter

Steps to reproduce the issue

I try to render this SVG file: "https://upload.wikimedia.org/wikipedia/commons/f/fc/Flag_of_Mexico.svg"
Keep getting Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'Invalid parameter not satisfying: [str hasPrefix:@"#"]'

Please include all relevant code, including all PocketSVG method calls made.
Has my issue been reported?
My issue has been reported in the following closed issues:
Specifications

PocketSVG Version: HEAD
Xcode Version: 9.x
iOS/macOS Version: iOS 11.x

The assertion fails with this tag:  <ellipse fill="url(#rg1)"
Unfortunately, we don't support the url parameter yet.

is there any update for this feature? I got this issue two

We hope to work on this feature request, but unfortunately we cannot at this stage give you a timeframe for when it'll be done. Pull requests are always welcome, of course :-)
In the meantime, a workaround is to rewrite the url parameters in your SVG file.

I also get the same error, is there any solution for this?
@arielelkin

As we said above, we don't support the url parameter yet unfortunately. We hope to work on this feature request, but unfortunately we cannot at this stage give you a timeframe for when it'll be done. Pull requests are always welcome, of course :-)
Would it help if we print an error message instead of throwing an exception ?

that would just make things fail unnoticed. URL loading is not an appropriate feature for a simple (de)serializer like pocketSVG. although url(#rg1) looks like some sort of file-local reference, so that limited support for that might be an acceptable PR. But not something I would spend time on.


cmd/link: error building standard lib using -buildmode=shared Go 1.20rc1

What version of Go are you using (go version)?

$ go version
go version go1.20rc1 linux/amd64

Does this issue reproduce with the latest release?
No, it does not reproduce with the latest 1.19.x release.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/deparker/.cache/go-build"
GOENV="/home/deparker/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/deparker/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/deparker/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/home/deparker/Code/goroot"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/home/deparker/Code/goroot/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.20rc1"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-O2 -g"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-O2 -g"
CGO_FFLAGS="-O2 -g"
CGO_LDFLAGS="-O2 -g"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build1198116115=/tmp/go-build -gno-record-gcc-switches"

What did you do?

pushd src
./make.bash
popd
GOROOT=$(pwd) ./bin/go install -buildmode=shared -v -x std
What did you expect to see?
No error building shared stdlib.
What did you see instead?
https://gist.github.com/derekparker/d391d9bb492f9ee611c8ad136e813ab3

cc @alexsaezm (found this issue while building Go 1.20rc1 for Fedora)
cc @ianlancetaylor @randall77 @heschi

CC @cherrymui @thanm

go install -buildmode=shared -v -x std

This command includes all packages in std, in particular, both runtime/race/internal/amd64v1 and runtime/race/internal/amd64v3. Both of these packages contain the race syso, so both syso's get into the link, causing duplicated symbols. This seem all but the last error.
In normal build, only the runtime/race package is directly imported, and one of the internal packages is actually imported based on the GOAMD64 setting. But this package selection doesn't work for go install -buildmode=shared std.
I wonder if go install -buildmode=shared std could start with all user-importable packages in std and only pull in internal packages on demand...

/usr/bin/ld: /tmp/go-link-2738326382/go.o: relocation R_X86_64_PC32 against undefined symbol `runtime_test.callerStartLine' can not be used when making a shared object; recompile with -fPIC

The last error is due to a package that is used only in test, and calls a function defined in test. Again building std includes this package, but not the function defined in test. We could probably work around this one. Pulling only user-importable part of std would help this as well.

CC @bcmills

For the runtime/race/internal packages, would it suffice to add go:build constraints to the doc.go files so that only one or the other package actually exists? (Or does the package exist because of the syso files even if all of the .go files are constrained away?)

For the test-only package, does it suffice to move that package inside a testdata directory? It will probably still be importable...

would it suffice to add go:build constraints to the doc.go files so that only one or the other package actually exists?

Thanks. This works.
For the other issue I worked around it using a function pointer.


When navigating with keyboard controls focus moves to elements hidden by lesson resource drop downs

Replication steps:

Go to a lesson on the beta
Tab through the page elements
Keyboard focus will get lost within the "Worksheet" dropdown
Keyboard focus will get lost within the "video" dropdown, but only if it has previously been opened and the mux player has loaded

Is this linked to this ticket at all: https://github.com/oaknational/Oak-Web-Application/issues/1367?
Acceptance Criteria:

[ ] Focus should skip content that is currently hidden in drop downs

Currently working on mobile drop down menu on blogs

:tada: This issue has been resolved in version 1.185.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Rename GITHUB_TOKEN env var to PERSONAL_ACCESS_TOKEN

use PERSONAL_ACCESS_TOKEN to ensure the GITHUB_TOKEN env var default value is not used.

I've update the PR with a second change.
If the IAM_USERNAME env var is not set, the IAM user is read from the AWS credentials asking the AWS api for himself.

@kneemaa I see this PR is still open, do you still maintain the repository?

@marcohutzsch1234 apologies i just saw this as i was never assigned the PR. I can take a look at this later this week and test it in my environment before approving and merging.

@marcohutzsch1234
The who_ami_() functionality is great.
Why do we need to "use PERSONAL_ACCESS_TOKEN to ensure the GITHUB_TOKEN env var default value is not used."?

Hi @kneemaa ,
der GITHUB_TOKEN var cannot be set from outside. If you pass a value for the var in an action, the value is not used. Github prevents action runs from overriding any GITHUB_* env var. The existing GITHUB_TOKEN has only access to the repository, where the pipeline lives in. When you want to update the secrets for a couple of repos, it fails with GITHUB_TOKEN because the GITHUB_TOKEN has no access to other repos.
Hth!
Marco

thanks for merging!


[Enhancement] [SPM] Bundle Resource

Prior Swift tool 5.3, install this package via Swift Package Manager results in crashing every time invoking  DateTimePicker, because the resources file was not bundled.
Description
Bundling resources when distributing via Swift Package Manager.
How Has This Been Tested?

Swift Package Manager:

Initiate playground project with the minimum iOS 10.
Install DateTimePicker as a local package.
Run unit tests.
Run UI tests.
Achrive.

Cocoapods:

Initiate playground project with the minimum iOS 10.
Install DateTimePicker as a local package.
Run unit tests.
Run UI tests.
Archive.

Carthage:

Supposed to running well like Cocoapods.
Screenshots (if appropriate):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

LGTM, merging request. Thanks for the contribution!

@itsmeichigo Thanks, to have this done you should probably bump the version on Cocoapods spec if appropriate.
About SPM, if you think it's ready to be used widely, some update for README.md would be great.


Functions as a WASM

Looks like it’s possible to make each example page a WASM . So we could do serverless style serving of each as a WASM.
not looked into how I can do it with data-star fully.
I have his htmx demo where the same WASM on the server is loaded into the browser as a service worker .
So first request hits the server fully but the response embeds an asynchronous loading script to load the same WASM into the browser as a service worker.
the next client request then hits the service worker . That worker knows it’s in the client and calls for data ( if not cached ).
If you’re interested I can send you the demo.

Nothing actionable that I can see for now.  Feel free to reopen


All embedded connections roundtrip to packageable elements.

What type of PR is this?
improvement
Does this PR introduce a user-facing change?
Creates packageable elements out of embedded connections

This PR is stale because it has been open for 30 days with no activity. Please remove stale label or add any comment to keep this open. Otherwise this will be closed in 5 days.

This PR was closed because it has been inactive for 35 days. Please re-open if this PR is still relevant.


System management Agent config - returns only service config found

On a call such as http://localhost:48090/api/v1/config/edgex-support-notifications,edgex-core-data, the SMA would return {Service: name, Config: {}} for those services it finds.  If Notifications was up and Core Data was not, it would return:  {"Service":"edgex-support-notifications","Config":{"Clients":{"Logging":{"Host":"localhost","Port":48061,"Protocol":"http"}},"Databases":{"Primary":{"Host":"localhost","Name":"notifications","Password":"","Port":27017,"Timeout":5000,"Type":"mongodb","Username":""}},"Logging":{"EnableRemote":false,"File":"./logs/edgex-support-notifications.log"},"Registry":{"Host":"localhost","Port":8500,"Type":"consul"},"ResendLimit":2,"Service":{"BootTimeout":30000,"CheckInterval":"10s","ClientMonitor":15000,"Host":"localhost","Port":48060,"Protocol":"http","ReadMaxLimit":1000,"StartupMsg":"This is the Support Notifications Microservice","Timeout":5000},"Smtp":{"Host":"smtp.gmail.com","Password":"mypassword","Port":587,"Sender":"jdoe@gmail.com","Subject":"EdgeX Notification"}}}
It would not indicate that there is no config for edgex-core-data (or even that it was a valid service).  It should return null for those services not available or where the service does not have any config.  It shouldn't leave the service name out of the return collection.

Errors should also be return in the case there is an issue on response from one service

Will address this shortly.

This bug has, in the past 24 hours, actually been addressed and fixed as part of the following PR which was in flight already: PR #672.

@jpwhitemn Let's please sync up sometime to ensure that the needful has been addressed 👍

On further testing, I do notice that if a service is down, for example the edgex-config-seed service, then a response as follows is returned for that service:

"edgex-config-seed": ""

The call was:

http://localhost:48090/api/v1/metrics/edgex-config-seed,edgex-support-logging,edgex-core-metadata,edgex-support-notifications,edgex-core-data,edgex-core-command,edgex-export-client,edgex-export-distro

This issue has been addressed.


Pass PageServerConf as static ref.

It's created once early in server startup, after parsing the
command-line options, and never modified afterwards. To simplify
things, pass it around as static ref, instead of making copies in all
the different structs. We still pass around a reference to it, rather
than putting it in a global variable, to allow unit testing with
different configs in the same process.

One strategy that I have used in other projects works like this:
To access global resources, you need access to a Kit structure that owns all the long-lived data. Different services own an Arc<Kit>, so lifetimes work without memory leaks. Usually acquiring the Arc happens very early in a service/thread/context so the refcounts are pretty much static. Usually lower-level functions get passed &kit or &kit.some_resource.
Things get tricky when different resources inside the kit have dependencies on one another. I have played around with this idea a little bit, resulting in an unreleased crate called magic_kit that would probably need a bit more work to be usable.
I also dislike the ergonomics of lazy_static, so I built a crate called freezebox to make lazy initialization simpler.

Yeah, this might evolve into a kit like that later, if we need more data or mutable global data.
I would like to avoid passing around these global structs and just have them available everywhere, for ergonomic reasons. But at the same time, in unit tests, be able to have different values for different tests. Maybe a thread_local, and pass the parent thread's value down whenever you spawn a new thread.
I'll push this for now...


Unifying indentation to 4 spaces indent

From the previous issues #15 #11 , we discussed about how to deal with indentation inconsistency issue.
We've discussed detailed how-to on #15, I gonna conclude about this issue.
This may be a simple issue, so we all can get to them easily. Therefore I want to divide them as equal as possible.
However I found empfb uses jupyter-notebook, and indented quite well.
Let's divide total works into 12 parts,

pfb (python-for-beginners) : Chapter 2 - 18

[x] pfb 2-4 @jesusleejy
[ ] pfb 5-7 @ChaeyeonSeo
[x] pfb 8-10 @jesusleejy
[x] pfb 11-13 @ChaeyeonSeo
[ ] pfb 14-16
[ ] pfb 17-18

mpfb (more-python-for-beginners) : Chapter 1 - 9

[x] mpfb 1-3 @Dorika0203
[x] mpfb 4-6 @Dorika0203
[ ] mpfb 7-9 @Dorika0203

empfb (even-more-python-for-beginners-...) : Chapter 1 - 15

[x] empfb 1-5 @devquint
[ ] empfb 6-10
[ ] empfb 11-15

If you want to start one job, pick one and mark like below
original:
- pfb
    - [ ] pfb 20-30

after picking:
- pfb
    - [ ] pfb 20-30 @devquint

after done (made PR):
- pfb
    - [x] pfb 20-30 @devquint

Rules for indentation:
For some def, for, while and so on making code block, use 4 spaces indent, not tab
like:
for a in dict:
    if a in another_dict:
        do_something()
    do_other()

For making some long array and so on, let the no argument on first line, and do 4 spaces and make newline for each or reasonable amount of arguments (Actually we can choose other way, but we'd rather unity it to one below, detailed about this)
like:
foo = long_function_name(
    var_one, var_two,
    var_three, var_four
)

foo = long_function_name(
    var_one,
    var_two,
    var_three,
    var_four
)

# add one more level to function parameter to make it distinguishable from function body
def long_function_name(
        var_one, var_two, var_three,
        var_four):
    print(var_one)

airports = pd.Series(
    [
        'Seattle-Tacoma',
        'Dulles',
        'London Heathrow',
        'Schiphol',
        'Changi',
        'Pearson',
        'Narita'
    ]
)

If you start your works, mark in this issue, and make another issue with name like
Unifying indentation: more-python-for-beginners chapter 1-3
and make branch with its issue name under suit branch like

master

indent-unify

indent-pfb

issue-16 (issue-<issue number>)
issue-17

indent-mpfb

...

indent-empfb

After done, make PR like indent-pfb <- issue-17 and let others to double check it indented well.

NOTE:

do not choose more than 4 tasks
do not get task 2 or more at the same time
you may install jupyter notebook to fix empfb
workflow: mark above of this post -> make an issue -> make a branch -> commits -> make PR -> mark above of this post

Anything more should be discussed about this, please comment in this issue.

@Hyuntae-Jeong I said that do not get 2 or more task same time. You didn't even made an issue one of them, so I removed both tagged. Tag one of them again and make issue before you start.

@devquint Okay Got it

All pull requests are made, please review as soon as possible.


dist: provide one-line installer

We all agree we want a 1-line installer. tools/cmd/getgo is a start but has work to do. This is the tracking bug for shipping it (and advertising it on the install page).

I've been talking with @spf13 about the one-line installer. If the idea is to use tools/cmd/getgo, then how about the following?
curl -LO https://get.golang.org/$(uname)/go_installer \
  && chmod +x go_installer \
  && ./go_installer \
  && rm go_installer \
  && source ~/.go_env

The idea would be to set the necessary environment variables -- GOPATH and appending to PATH -- in the .go_env file. For future sessions, getgo could also add a line sourcing .go_env to the shell config file.

if we rename it from go_installer to "gi" it shortens the command a lot :)
It's a fundamentally different approach from the rustup.sh approach. The rustup approach can't set variables in the current shell which is a major stumbling block for users.

@spf13 Here is an attempt doing what we discussed.

Here is an attempt at resolving #21277 as we've discussed.
/cc @spf13

I saw that no progress was made with getgo so I built goup for myself. Blog post: https://owenou.com/goup.

How can be this?(with stabilization)
https://github.com/sadesakaswl/goup

A few questions I have about a potential tool like this:

Is it going to allow multiple Go toolchains to coexist on a system?
If so, is it going to allow setting a default toolchain?
Do environment variables need to be set? I assume they won't, since the go command has sensible defaults, and GOPATH is deprecated.
Will it be module-aware, i.e. if I run go build in a project that expects a different version of the language, would another version be fetched? I assume not, since there would have to be some interaction between the compiler and the installer.

I think this is a great idea, but continuing the comparison with Rust, that has 3 tools:

rustc - the actual compiler
cargo - the package and project manager
rustup - the rust toolchain / target installer and manager

The go binary seems to cover the first 2 cases. What is nice about using rustup, is it’s much more than just a simple install script. You can upgrade your rust install, you can add targets, and you can add tool chains. The rust project nailed it with their tooling and it’s a pleasure to work with.
Perhaps a in the same naming spirit, a go-up tool which can manage versions of go installed and select the active or default version and upgrade it. And it could have a nice 1 line installer with curl piping into sh.

fyi - getgo is deprecated.
The new GOTOOLCHAIN helps updating/switching go toolchain conveniently - assuming there is already a version of go toolchain installed.


GoBgp client cli capability

Add GoBGP client cli capability to Antrea Agent
gobgp image size is 15 MB
#6189

I am not sure I get the point of this PR, please add more details in the PR summary to clarify what you want to implement in this PR.

I am not sure I get the point of this PR, please add more details in the PR summary to clarify what you want to implement in this PR.
Details available in #6189 ( #6541 )

@antoninbas Thanks for checking my opinion.
@rajnkamr Can you update the description to point to the right issue https://github.com/antrea-io/antrea/issues/6541? https://github.com/antrea-io/antrea/issues/6189 barely provides any information.
However, https://github.com/antrea-io/antrea/issues/6541 doesn't have information about how the gobgp CLI can/will be used either, it says "advanced live debugging", but without a single example. Does antrea-agent even open a socket for gobgp CLI to connect? I don't find such socket. Have you validated any command can work?
Besides, is it right to open such socket to introduce another source of truth of bgp routes? Unlike the typical gobgpd and gobgp case that the latter is used to configure the former, we only leverage the gobgp library to implement bgp using the BGPPolicy as the source of truth. Maybe it could be helpful in read-only mode but I guess there is no permission control in such granularity once the socket is there. Who is supposed to use the CLI, users, developers, or both?
Lastly, I remember there was a topic to have other BGP implementations if required, I suppose gobgp CLI won't work for other implementations?

Generally we should avoid adding things to image when there is no strong reason. Despite of the image size, every dependency comes with CVEs and maintenance efforts. If the reason is just maybe helpful for some scenarios (and in the particular case non use case has been shown), maybe let's at least defer to when users/developers have to install gobgp themselves to address their use cases?
Actually there are many useful networking tools I frequently use when debugging: netstat, tcpdump, etc., which are even more common and frequently used than gobgp. You can imagine the impact to image size if all of them are added. At least it's not a problem for me to install tools / use ephemeral containers when there is a need.

We are utilizing the GoBGP library directly within Antrea, rather than deploying the standalone gobgpd binary. As a result, the gRPC-based interface typically exposed by gobgpd is not available. Consequently, external tools or clients, such as the GoBGP CLI (gobgp), which rely on gRPC to connect to gobgpd, will not be able to interface with Antrea's BGP server operating on port 179.
So to manage and configure the BGP server within Antrea, currently interactions will be conducted only via the Antrea API or CLI (antctl). Hence we will have to continuously enhance antctl by incorporating additional commands to support more functionality.
To fully support advanced debugging commands provided with gobgp binary, a comprehensive solution would involve rewriting gobgp to align with our current architecture. It could be long term solution or we could just continue enhancing antctl with more commands as par with gobgp binary.


docs(firestore-bigquery-export): delete query

fixes #444
Provide example of how to delete rows in the changelog table generated by the firestore-bigquery-export table

@kevinthecheung can you take a look at this?

I think the example given in the docs cloud delete entries in the changelog that are part of the current latest view. The recommended query cloud be something like delete all event ids that are not in the current latest view, and that are older than a day.

@mvergarair sounds interesting. Do you have a link for such a query? I'd be happy to update retrospectively.

Codecov Report

Merging #463 into next will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             next     #463   +/-   ##
=======================================
  Coverage   41.58%   41.58%
=======================================
  Files          30       30
  Lines        1128     1128
  Branches      154      154
=======================================
  Hits          469      469
  Misses        653      653
  Partials        6        6

@mvergarair we're going to merge this as-is, but feel free to open a new issue to discuss an alternative cleanup query.


Adds a variable to allow sshd monitor to be customized

It allow customization of the SSH service port.
This fixes #2

LGTM :smile_cat:


Change back button for search icon, onCancel gets not called

Hi,
Just  Android.
I'd like to have searchbar icon and when showed the back icon, on press on it make it not visible.

I was unable to add a view in the icon parameter, just changing the icon properties

I understood that onCancel might get called when click on Back icon, but I didn't find were it gets called.

I think it's a bug on Android.
Thx
"react": "16.0.0",
"react-native": "0.51.0",
"react-native-elements": "^0.19.0",
"react-native-vector-icons": "^4.5.0",

This is not currently possible @adriancuco, feel free to open a PR that implements this behaviour.


WIP: save ith sensor binary file into output/devicei/raw-X.bin

Current binary savers save camera 1 and 2 data into the same binary file (i.e. output/raw-X.bin), so this update split them into 2 binary files (i.e. output/decive/raw-X.bin).

Duplicated https://github.com/fixstars/ion-kit/pull/268; close


[SMALLFIX] Use @Nullable annotation for methods which may return null

Use the @Nullable annotation for all methods which may return null in /alluxio/core/server/master/src/main/java/alluxio/master/file/activesync/ActiveSyncManager.java

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

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

alluxio-bot, merge this please


Update "Images" lesson with information about CSS image-set

https://developer.mozilla.org/en-US/docs/Web/CSS/image/image-set()

Done!