Improve wording of diagnostic about escaping closures capturing inout…

What's in this pull request?

Reword a slightly awkward diagnostic regarding capturing inout parameters in escaping closures.

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

… params.
Fixes: rdar://problem/27208711

@swift-ci Please test.

@dduan It looks like you added this diagnostic. I've reworded it a bit, care to review?

LGTM.
The original wording is straight from the proposal, FYI.

Ah, perhaps @jckarter might want to chime in then, too.


Ui hang when running setWallpaper function..

All the things work perfectly, But when running set Wallpaper function app freezed.
Plz can you fix this error.
How ever thanks for package

I run this on seperate isolate , But same. I think it will freeze whole app not only Main ui thread

This issue is not a plugin error, If you use main thread Async task proper way ,this not happen.
I fix this error by modifiying my async task proper way.
Thanks Ganesh for your plugin package..


Create Azure subscriptions from an Azure family provider

What problem are you facing?
I can manage subscriptions using the monolithic provider: https://marketplace.upbound.io/providers/upbound/provider-azure/v1.2.0/resources/azure.upbound.io/Subscription/v1beta1
There is the deprecation notice but I do not find the subscription kind in any of the family providers. Is there a document mapping resources from the monolithic provider to the new one?
How could Official Azure Provider help solve your problem?
provide better documentation/migration guide on how to move from monolithic provider resources to the correct family instance.

It's here available:
https://marketplace.upbound.io/providers/upbound/provider-family-azure/v1.2.0/resources/azure.upbound.io/Subscription/v1beta1
apiVersion: pkg.crossplane.io/v1
kind: Provider
metadata:
  name: provider-family-azure
spec:
  package: xpkg.upbound.io/upbound/provider-family-azure:v1.2.0

Update from Monolith to Family Providers: https://docs.upbound.io/providers/migration/


Test package only supports GET requests

There is the test.Validate function that ensures that only GET methods are allowed.
It means, the following test.Request is considered to be invalid
env, err := test.New(t, test.Request{
			Method: "POST",
			Url:    "...",
			...
		})

Is it a mistake or is there a reason why no other HTTP methods are supported?

I have just found the same question and there is already PR for that
https://github.com/Kong/go-pdk/pull/75/files#r661797816

Looks like unfinished code

YAGNI
or, in other words, if there's a need now; now is the best time to write it, and not before.
as for #75 , probably it didn't align with the priorities at the time. (and doing different unrelated changes in the same one doesn't help).  again, should be easy to add if you feel something like that would be an improvement

Hello @javierguerragiraldez we have already two opened PRs to solve this issue:

https://github.com/Kong/go-pdk/pull/75
https://github.com/Kong/go-pdk/pull/156

Do you know when it can be merged?

Hello @javierguerragiraldez we have already two opened PRs to solve this issue:

add tests for test package #75
[Fix] Support POST,PUT,PATCH & Fix Error when handle kong.response.exit #156

Do you know when it can be merged?

Thanks for reminding us. We will take a look soon.

@156 is merged.


Correction for raw affiliation Univ Rennes, CNRS, CREM – UMR 6211, Rennes, France

Correction needed for raw affiliation Univ Rennes, CNRS, CREM – UMR 6211, Rennes, France
raw_affiliation_name: Univ Rennes, CNRS, CREM – UMR 6211, Rennes, France
new_rors: 02feahw73;015m7wh34;0047rmf80
previous_rors: 02feahw73;015m7wh34
works_examples: W4386742070
contact: c54405566524b8fc3efe38ebce88a353:deec73e6e5dce6f2d5b1c399a10a6dad @ univ-rennes.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


BAEL-3312 Spring Cloud Gateway WebFilter Factories

Proposed Changes

Add WebFilter Factories implementation
Add Unit Testing
Modify the pom to include Circuit Breaker dependencies and upgrade Spring Boot and Spring Cloud versions

Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests
[x] Build locally
[x] Code format / lint
[ ] Docs have been reviewed and added

Pull request type
Please check the type of change your PR introduces:

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

Hi @eric-martin I did not update the README file so right now it is not including the reference to the new article


Bug: category filter when create/edit/delete product

Fixed here:
https://github.com/ingvr/test-store/commit/18dfda9b4518220921b5fa3c7d0ff83564dcd8a1


BUG: Celery task: lookup-recipient-communication-permissions failed

Bug
We are looking up a communication item id that does not appear to exist.
Search cloudwatch for:
"sqlalchemy.orm.exc.NoResultFound: No row was found for one()"
Expected behavior
It should gracefully handle the request.
Additional Information
The code that fails:

Cloudwatch logs:

No longer relevant.


Remove _raw optimization from regular COPY FROM

Relates to https://github.com/crate/crate/issues/14019
Collecting existing code here but without experiments with COALESCE and some other abandoned approaches, cleaner version of https://github.com/crate/crate/pull/14174

retest this please

Hi @mfussenegger, could you have a look into the first part of _raw removal?
RETURN SUMMARY, dynamic mapping updates, benchmarking of validation=true vs false will follow, gathering everything into a feature branch.
Idea:

Stop using _raw as target reference for regular COPY FROM (keep it for SUMMARY for now)

Generated columns validation: abandoned in the prev PR COALESCE approach couldn't differentiate "no-value  at all" vs "provided NULL" and after some internal discussions I went with marker value approach.

Ie we don't unwrap generated references, modify Indexer to handle marker value.

rebased to get Jenkins tests fix via benchmarks removal, java tests are passing

Can you explain again why the value-markers are necessary at all from a high level perspective?
The amount of complexity this introduces is quite high due to the spread through several components. We should be sure we don't have any less invasive alternatives.
There are also lots of tests muted. What's the plan for these?

why the value-markers are necessary at all from a high level perspective?

Problematic scenario is generated columns and their computation/validation:
This PR collects concrete columns as targets. For generated columns we have kind of dilemma:

collect reference == take provided value. Problem: if value is not provided, it's not computed. Collecting columns with no provided values leads to getting NULL and such rows are rejected with "Given value NULL doesn't match with generated...". Concrete example: testCopyFromWithGeneratedColumn.

collect generated expression = take computed value. Problem: when user provides an incorrect value for generated column, it's anyway ignored and not validated.  Concrete example: testCopyFromWithInvalidGivenGeneratedColumn.

Previous approach: we can collect any expression.
I tried to handle generated columns by collecting expression COALESCE(generatedColumn, generatedExpression).  It was basically adaptive collection of generated columns. It passed all tests but apparently, there is a tricky scenario we didn't tested before: provided NULL value. I added such scenario and the COALESCE approach failed by skipping validation of NULL and not-rejecting that row.
Hence, a marker value has been introduced to differentiate between "provided NULL value" and "no value provided".

We should be sure we don't have any less invasive alternatives.

I can imagine only moving into custom expression direction, i.e improving COALESCE approach. But AFAIK, any of IF, CASE are incapable of differentiating between "provided NULL value" and "no value provided". We can introduce some hidden internal scalar like TAKE_FIRST_PROVIDED which still would compare with the marker value but this way we would introduce marker only in COPY FORM specific components: ColumnExtractingLineExpression, ColumnIdent.get but wouldn't touch indexing, streaming and partition columns/index name resolving components. Not sure making some hidden scalar is a good idea though but I don't see other way for now. wdyt?

There are also lots of tests muted. What's the plan for these?

Some of them are unmuted already in this PR (tackled separate issues in separate commits for easier review) and some of them will be addressed in a follow up PR-s. I have some working prototypes for muted tests but I want to agree on base approach first and then iterate on other tests. I've created a feature branch and will collect all parts there, eventually all tests will be unmuted there.

This PR collects concrete columns as targets. For generated columns we have kind of dilemma:

Can't we control this with the presence of the columns, same as everywhere else?
E.g. in a table with x and y, where y is generated:

INSER INTO tbl (x) VALUES (1) -> y is missing, value gets generated.
INSER INTO tbl (x, y) VALUES (1, null) -> y is provided. Validate value

So if COPY tbl (..) FROM lists columns, we could treat it the same.
If columns are omitted, we would have to make it dynamic depending on which columns are provided in the source file.
JSON may be a bit difficult because rows can have different keys, but I think it would be fine to normalize that -> if one row contains a column, treat it as "column is present".

COPY tbl (..) FROM lists columns, we could treat it the same.

Scenario "provided-NULL-vs-no-value" is different even on such simpler scenario ^. It's possible on COPY FROM and not possible on INSERT.
COPY FROM with target (x,y), null is provided in the file  <==>  INSERT INTO tbl (x, y) VALUES (1, null), it works.
COPY FROM with target (x,y), null is not provided in the file  <==> INSERT INTO tbl (x, y) VALUES (1) which is impossible, fails with SQLParseException[Number of target columns (x, y) of insert statement doesn't match number of source columns].

COPY FROM with target (x,y), null is not provided in the file <==> INSERT INTO tbl (x, y) VALUES (1) which is impossible, fails with SQLParseException[Number of target columns (x, y) of insert statement doesn't match number of source columns].

out of curiosity checked this case on  master to see how _raw source based COPY FROM handles that. Such scenario works on the current master because ValidatedRawInsertSource/GeneratedColumns are capable to differentiate between null value/no-value.
If null is provided, it's validated.
If no value provided, column is not validated

Can't we control this with the presence of the columns, same as everywhere else?

Yep we can, thanks for posting it out., I messed up values-source keys. Will open another PR


Updated shop(1).html Issue#275

Solved Issue#275
This pull request addresses an issue where the square logo at the top of the shop page was not displaying with the correct dimensions, leading to a distorted appearance. To ensure visual consistency across the site, I have modified the logo dimensions to 300px height and 300px width. This change aligns the logo size with that of the transactions page, providing a uniform look and feel.

@Anushkabh
I have resolved the conflicts and created a new Pull request closing the previous one.
Please look into it, the issue has been resolved successfully
Thanks

Fixes #275


fix: add sleep to wait for sg creation

Description
https://github.com/terraform-ibm-modules/terraform-ibm-client-to-site-vpn/issues/45
Release required?
Identify the type of release. For information about the changes in a semantic versioning release, see Release versioning.

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content
If a release is required, replace this text with information that users need to know about the release. Write the release notes to help users understand the changes, and include information about how to update from the previous version.
Your notes help the merger write the commit message for the PR that is published in the release notes for the module.
Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

Use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author. The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).
Merge by using "Squash and merge".

/run pipeline

@jor2 Thanks for the PR. Firstly, this PR needs to be merged to unblock pipeline. Secondly, please be aware that in this PR, the landing-zone example is moving directories. I may actually bundle this change into that PR now (maybe you can review it?) and close this PR to avoid conflicts

@jor2 Ive included the workaround in https://github.com/terraform-ibm-modules/terraform-ibm-client-to-site-vpn/pull/49 - can you please review?
Closing this issue


Could not load provider DNNConnect.CKEditorProvider

Fresh Install of Dnn 9.4.4 latest, logs contain error:
DotNetNuke.ComponentModel.ProviderInstaller - System.Configuration.ConfigurationErrorsException: Could not load provider DNNConnect.CKEditorProvider.CKHtmlEditorProvider, DNNConnect.CKEditorProvider

I cannot reproduce, do you have any other pentential information, environment details, logs ?

Maybe step by step of how you installed Dnn, did you get the install package from the release page, did you check the folder permissions (the app pool needs full permissions on the folder).

Did you unblock downloaded Archive (in file properties), prior to unzipping the files contained?

i did not unblocked. I installed in a server 2016 with 4.6 and 4.8 framework only. not 4.7 like mentioned in the comments. Im away from computer this weekend. I can record the process and make it available.

please check file properties of downloaded install package. if there is a notice at the bottom of first property tabs about "this file has been downloaded from the internet", you need to hit the button or check the box, that this file is trusted.

I'm seeing this error four times on a clean installation of a Platform 9.5.0 instance (even with Unblock).
All I do is a standard manual installation, run the wizard, and land on the home page.  The errors are in the log4net file. That's all that's in there.
2020-04-28 13:55:49.327-07:00 [MyComputerName][D:2][T:1][ERROR] DotNetNuke.ComponentModel.ProviderInstaller - System.Configuration.ConfigurationErrorsException: Could not load provider DNNConnect.CKEditorProvider.CKHtmlEditorProvider, DNNConnect.CKEditorProvider
2020-04-28 13:56:08.733-07:00 [MyComputerName][D:3][T:1][ERROR] DotNetNuke.ComponentModel.ProviderInstaller - System.Configuration.ConfigurationErrorsException: Could not load provider DNNConnect.CKEditorProvider.CKHtmlEditorProvider, DNNConnect.CKEditorProvider
2020-04-28 13:56:44.595-07:00 [MyComputerName][D:4][T:1][ERROR] DotNetNuke.ComponentModel.ProviderInstaller - System.Configuration.ConfigurationErrorsException: Could not load provider DNNConnect.CKEditorProvider.CKHtmlEditorProvider, DNNConnect.CKEditorProvider
2020-04-28 13:57:48.520-07:00 [MyComputerName][D:5][T:25][ERROR] DotNetNuke.ComponentModel.ProviderInstaller - System.Configuration.ConfigurationErrorsException: Could not load provider DNNConnect.CKEditorProvider.CKHtmlEditorProvider, DNNConnect.CKEditorProvider

I have also seen this here and there, mostly right after a clean install but it does not come back after using the site. Not sure what causes it...

Is this still a valid issue?

Yes. I've seen it on all of my recent clean install of 9.6.1.


DDQOT verdict expressions DEV

Description
Resolves #4817
This PR adds the ability to use the Direct Database Query Over Time when performing a Pick Through Time on verdict metric variables. It also adds the general ability for the DDQOT route to retrieve mesh coordinates, which can theoretically be used for other types of queries over time as well.
I've also cleaned up some whitespace.
Type of change
New feature.
How Has This Been Tested?
I've added several tests that ensure that using Pick Through Time on verdict metrics works as expected, and I've ensured that these tests are actually using the DDQOT route. I've also run the entire test suite on Pascal.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have updated the release notes
[ ] I have made corresponding changes to the documentation
[x] I have added debugging support to my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have added any new baselines to the repo
[x] I have assigned reviewers (see VisIt's PR procedures for more information).

@brugger1 Do you mind if I hold off on this until after the release? I still need to figure out how to get these changes into an RC branch with my accidental merge+commit, but I need to switch over to my AML project today. I think I can get this RC branch tackled on Thursday.

Yes, that’s fine.

The cherry-pick issues for the RC branch are fixed now. Ready to move forward.


no data in many trials

Many of the RoiResponseSeries trials are all NaNs and have no data. Is that intentional?
The following trials have no data for all of the dFoF objects:
Trial_00
Trial_01
Trial_02
Trial_03
Trial_04
Trial_07
Trial_08
Trial_09
Trial_10
Trial_11
Trial_12
Trial_13
Trial_15
Trial_16
Trial_17
Trial_18
Trial_19
Trial_20
Trial_21
Trial_22
Trial_23
Trial_24
Trial_25
Trial_27
Trial_28
Trial_30
Trial_31
Trial_32
Trial_33
Trial_34
Trial_35
Trial_36
Trial_38
Trial_39
Trial_40
Trial_42
Trial_44
Trial_45
Trial_46
Trial_47
Trial_48
Trial_49
Trial_51
Trial_52
Trial_53
Trial_54
Trial_56
Trial_57
Trial_64
Trial_66
Trial_68
Trial_69
Trial_70
Trial_71
Trial_72
Trial_75
Trial_76
Trial_79
Trial_80
Trial_81
Trial_82
Trial_83

it seems that the dfof for all these trials is itself NaN in the .mat files


Update my company info

Signed-off-by: Filip Petkovski<EMAIL_ADDRESS>

[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes

Verification

There are some more outdated @onprem @squat. Maybe at least delete the old if you don't want to put the new one. 😉


installing viskit

This line of the installation instructions didn't work for me:
pip install -e viskit
Does this just mean that you should clone viskit somewhere else and install it with pip install -e?
e.g.
cd ~
git clone git@github.com:vitchyr/viskit.git
cd viskit
pip install -e .

Yes, that's correct. I've updated the README with the new installation instruction.


General: Charge Transaction Fees (Gas)

The ixo blockchain uses a custom IxoTx (instead of StdTx) and custom AnteHandlers. These currently do not implement the charging of transaction fees.

Closed by #118


ENS Resolved address doesn't show

Hi,
I recently went through the process to register an Ethereum Name [gregjeanmart] using MyEtherWallet. Thanks by the way for making the procedure easy to use.
After I won the auction, I use the contract ENS - Registry to set a resolver (same as the owner). But when I use the ENS main page on MyEtherWallet, the resolved address is 0x while the ENS Registry contract / Resolver returns 0x31b26e43651e9371c88af3d36c14cfd938baf4fd
I also configured the reverse lookup to the same address.
MyEtherWallet: ENS Lookup

MyEtherWallet: ENS Registry contract

I don't know if I missed something or it's a bug. Just wanted to let you know.
Greg

@gjeanmart you need to point the registry at the resolver and the resolver at the address.
See both parts of this FAQ:
https://myetherwallet.github.io/knowledge-base/ens/setting-the-resolver-for-an-ens-name.html

Closing this for inactivity.


Implementation: Allow configuration and enable file path interpolation in emitters for individual emitted files

Implementation issue to support https://github.com/Azure/cadl-azure/issues/2341
Allows configuration of individual file paths for emitted artifacts, using a set of well-known variables
version
version-status
filename
This would enable a relative-path configuration for individually emitted files, starting from the emitter output path:
@cadl-autorest:
  openapi-file: {version-status}/{version}/{fileName}.json

To support:

[ ] allow 'version', 'version-status', and 'fileName' to be included in emitter configuration without warnings

[ ] Add new helper 'interpolateConfiguration' that emitters can use to resollve the configuration.  This would interpolate any known variables in the configuration, and allow the emitter to provide values for 'version', 'version-status', 'fileName' (and potentially, other, future values)
interpolatieConfiguration(configValue: string, values: object) : string

configValue is the emitter configuration
values are the values to be interpolated for a particular file

[ ] (can be deferred) implement OpenAPI output file path configuration using the new feature

est: 5

Could you provide an example of how that would work for service group folder structure?
so for serviceGroup folder structure support, we would have:
azure-resource-provider-folder: {cwd}/..
openapi-file: serviceGroup/{version-status}/{version}/{fileName}.json

thanks

it depends on how it is executed.  It could either be
open-api-file: <service-group>/{version-status}/{version}/{filename}.json
or
open-api-file: <namespace>/<service-group>/{version-status}/{version}/{filename}.json depending on the value for the emitter output directory.


Fix typo — per year, not per month

Fixes #15
Changes proposed in this pull request:

Update subscription plans copy to correctly state pricing per year, not per month

The credentials to view the Netlify deploy is the following:

username: crabigator
password: googlebegone

Looks like this is a dupe of PR #20


ENH: stats.wilcoxon: return z-statistic (as requested)

Reference issue
closes gh-2625
What does this implement/fix?
gh-2625 asked for stats.wilcoxon to include the z-statistic in the result. This wasn't easy to do in a backward-compatible way before _make_tuple_bunch was introduced; now it is easy. This makes the return type of stats.wilcoxon a bunch and includes the z-statistic as attribute zstatistic.
Additional information
There are a few concerns.

Many tests could return a standardized version of a statistic and non-standardized version of a statistic. In some tests (e.g. ttest_ind), statistic is already standardized. So for these tests, it would not make sense to add zstatistic (or something similar) as a separate field. Adding it to wilcoxon only seems ad-hoc.
It doesn't make much sense to calculate and return a z-statistic when mode='exact', so in this case zstatistic is nan.
The Wilcoxon statistic can be calculated according to two conventions related by a simple identity. (The null distribution is symmetric; one version corresponds with the lower side, one with the upper side.) When alternative='two-sided', wilcoxon calculates both versions and returns the minimum of the two. This loses information. Because zstatistic is computed from statistic, it loses information, too. I don't really like adding an attribute that suffers from this problem, but it would be even more confusing if zstatistic were inconsistent with statistic. (mannwhitneyu used to do this, but we were able to change that after a deprecation cycle. Should we try to do that here?)

This will require some changes to the _axis_nan_policy decorator, which works with scalar results, tuple/namedtuple results, and non-iterable results (e.g. classes with fields). Surprisingly, that doesn't cover all cases: _make_tuple_bunch is funny because it produces objects that are iterable but also have additional fields.
The trouble is really in np.apply_along_axis - it thinks the bunch object it just a normal iterable and treats it as such, returning a numerical array that is missing the zstatistic. But in this case we need it to think of the bunch as a non-iterable object and return an object array, from which we'd be able to unpack all the attributes manually.
I'll close in the meantime.

Meh... fixed it. But I'll mark as draft.

@tirthasheshpatel I'm curious about your thoughts on adding zstatistic. If we decide not to add that to Wilcoxon, that's fine; but let's still use this PR to make the relevant changes to _axis_nan_policy, as these will still be useful.

Thanks @tirthasheshpatel. Thanks for addressing the second concern (about returning nan for exact), but I was wondering what your thoughts were on the other concerns expressed in the top post. Any thoughts on how we might resolve these things?

Many tests could return a standardized version of a statistic and non-standardized version of a statistic. In some tests (e.g. ttest_ind), statistic is already standardized. So for these tests, it would not make sense to add zstatistic (or something similar) as a separate field. Adding it to wilcoxon only seems ad-hoc.

I think adding a zstatistic would be useful for tests that compute it differently (than statistic). We can start an issue (like gh-12506) to track which tests need a zstatistic.

It doesn't make much sense to calculate and return a z-statistic when mode='exact', so in this case zstatistic is nan.

:+1:

The Wilcoxon statistic can be calculated according to two conventions related by a simple identity. (The null distribution is symmetric; one version corresponds with the lower side, one with the upper side.) When alternative='two-sided', wilcoxon calculates both versions and returns the minimum of the two. This loses information. Because zstatistic is computed from statistic, it loses information, too. I don't really like adding an attribute that suffers from this problem, but it would be even more confusing if zstatistic were inconsistent with statistic. (mannwhitneyu used to do this, but we were able to change that after a deprecation cycle. Should we try to do that here?)

Adding an attribute might be confusing. We can start a deprecation cycle since it's possible to get min(r_plus, r_minus) if we return one of the r_plus or r_minus but not the other way around.

Another alternative is to return zstatistic=None when exact=True.

Yes, this was the initial idea. This is one thing that the decorator makes a bit challenging. There is a way to do it, of course, but I didn't feel strongly enough about it to implement it.

I think adding a zstatistic would be useful for tests that compute it differently (than statistic).

Yes, but the confusing thing is that some tests already return a normalized/standardized statistic as statistic and one might want the raw statistic. For instance, one ttest_1samp returns the t-statistic as statistic, but one could imagine wanting it to return a statistic that has not been normalized by the standard error. I'm not sure if that's a great example (it's trivial to calculate the mean and subtracting the population mean if desired), but there might be other cases like it.

We can start a deprecation cycle

The question is what to deprecate, exactly. We could add an alternative=None that would do the thing we want two-sided to do, and we could deprecate use of 'two-sided'. (But that is more confusing than what we were able to do with mannwhitneyu - deprecate use of the function without an explicit alternative parameter. It's not as simple as that here, obviously, since people may already be explicitly specifying alternative='two-sided'.)

Yes, this was the initial idea, but the decorator makes a bit challenging because it has control over the output. There is a way to do it, of course, but I didn't feel strongly enough about it to implement it.

Oh yes, I wasn't thinking of the decorator when I wrote that. I too don't feel very strongly towards returning None at the cost of adding another layer of complexity to the decorator, now that I think of it.

Yes, but the confusing thing is that some tests already return a normalized/standardized statistic as statistic and one might want the raw statistic.

We can do one of these:

Deprecate that functionality and add a zstatistic field.
Add a field called raw_statistic which is equal to statistic (a view of statistic if the result is an array) if method='exact' and we can set that field to the raw statistic value otherwise. This might be more confusing.
Add a boolean argument called return_raw_statistic. This would require changes in the decorator like callable n_outputs.
There is another alternative mentioned below.

I think option 3 sounds like a cleaner way to do this (without any deprecation).

The question is what to deprecate, exactly. We could add an alternative=None ... explicitly specifying alternative='two-sided'.)

One way to handle this is to add a legacy=None keyword-only argument (see, for example, gh-15359). If this isn't set, we raise a deprecation warning: "wilcoxon returns min(r_plus, r_minus) as the statistic but this has been deprecated in favor of returning r_plus. If you wish to preserve the old behavior without this warning, pass legacy=True. To switch to the new behavior, pass legacy=False. The default value of legacy will change to False in SciPy 1.11.0.".
We can also do this for the functions returning the normalized statistic values: "ttest_1samp returns a t-statistic but this behavior has been deprecated in favor of returning the unnormalized statistic value. If you wish ...".
Do you think this could work for stats functions or should we go for something simpler like adding a keyword argument that can be set to true to return r_plus or unnormalized statistic like the option 3 above (and no deprecation would be necessary)?

Thanks for your thoughts.
if we think the z-statistic is something we want to return and zstatistic is good standard name for a z-statistic, then, I'm done with it for now. I'm now more convinced that it is sufficiently useful to add, and we can continue to work out (separately) what to do about old behavior we disagree with enough to change.
A final thought has to do with the idea of returning standard error and confidence intervals wherever we can. Playing devil's advocate: suppose we return the standard error and clearly define what statistic (e.g. normalized or I normalized) we returned as statistic; then the other version is as simple as a multiplacation or division by the standard error. In that case, we could add the standard error instead of the z-statistic.
I'm not sure I agree with that. I think we definitely should add the standard error, but I'm not sure we can do just one or the other here unless we return the other terms in the numerator - the mean under the null hypothesis and the continuity correction. Then again, we might want to add both of those sorts of thing in general, too.


Better tutorial + resource guidance

One of the main struggles I see people having with Vulkan is simply trying to find good guidance on how to get started with Vulkan. There are now a number of useful tutorials and resources out there, but they often cater to different markets, or have various inobvious drawbacks (e.g. not being quite correct or performant); and it's hard to know where to turn.
Something that would be good is to work on giving better guidance on which tutorials can teach you what, what a given helper lib gives you, which audiences they cater to, what sort of drawbacks and benefits are you likely to get with each... etc.
This could almost take the form of a series of reviews and summaries of each resource.
Right now we have a number of lists such as http://stephaniehurlburt.com/blog/2017/7/14/beginner-friendly-vulkan-tutorials
and https://github.com/KhronosGroup/Khronosdotorg/blob/master/api/vulkan/resources.md, but these are fundamentally "just" lists - would be good to add some context to them.

From last week's discussion, the ecosystem group is willing to help curate documentation or any other resources that might be useful for Vulkan users.
@TobiasHector would you be willing to help put together an initial list of resources that should be looked at?

From the call today, I can start going through the lists scattered around and lightly summarize what each tutorial/article goes through. I'll try to highlight some Vulkan code-bases out there as well.
Building context and evaluating the quality of the content is a larger task which needs to deferred to later.
I will link a google doc here once I have something substantial.

Here's a list of lots of resources that I found the other day: https://github.com/vinjn/awesome-vulkan

Some folks spent time doing an initial pass on this, and realized that just curating resources requires a lot of time and effort. There is growing concern about spending time on this topic as opposed to applying those resources towards, e.g., improving tools.
That said, this topic remains quite interesting. We're considering instead whether we can find a way to engage and motivate the community to fill this gap while we fix other more pressing issues within the Vulkan platform. We're not dropping this issue, however, as the hope is that we can still get back to this at a later time.

Yeah, curating resources is hard. To be honest, I haven't done enough curating for awesome-vulkan.

@nsubtil @sfricke-samsung Does the creation of the Vulkan Guide satisfy what is needed here (as long as it continues to get curated)?

I think it depends what people are looking for.
For a general "what is Vulkan as this is the first time I have heard of it" I feel the Guide does a good job
How do I get up and going writing Vulkan Code, or How do graphics programming work, the Guide doesn't cross that scope boundary.

This repository is being archived as it has been replaced with the vulkan.org website (https://www.vulkan.org) and is no longer being maintained (i.e., issues posted here are no longer being addressed ).  After reviewing issues posted here, most (if not all) have been resolved or have already been re-opened in Vulkan-Docs (https://github.com/KhronosGroup/Vulkan-Docs) or other repositories for further consideration.  Therefore, all issues in this repository will be closed.  If you believe your issue has not yet been resolved, please re-open in Vulkan-Docs.  Thanks!


Unable to launch ComfyUI in Paperspace, tensorboard link "Service Unavailable"

Hi Ben, the script is loading as normal but the tensor board link appears broken.
I had a similar error here:
https://github.com/TheLastBen/fast-stable-diffusion/issues/2786
Thanks for your help

I updated the repo now and have the exact error as above. At the bottom it says:
"Starting server
To see the GUI go to: http://<IP_ADDRESS>:6006/"

It seems a logging problem between sd() and server.py

It is temporary solution
Run it.
Url will display.
click the url
after To see the GUI go to: http://<IP_ADDRESS>:6006/" displayed.
`localurl="https://tensorboard-"+os.environ.get('PAPERSPACE_FQDN')
print(localurl)
Args="--listen --port 6006 --preview-method auto"
#--------------------
sd()
!python /notebooks/ComfyUI/main.py $Args`

will check it out

fixed

Thanks for the fix! Can confirm it's working. And thanks @doggs696 for that fix if it happens again.


Added AI Trivia Frame

Frame link - ai-trivia-frame

Congratulations on your PR being merged! This means that you are now eligible for the Bounty. If you did not submit through ETH Global, kindly share your Converse address with us so that we can send you your USDC.
Please note that we are still in the process of finalizing the reviews, so please bear with us. Open Frames was a huge success with over 100 frames integrated, and we wanted to thank you for your valuable contributions.

Converse Address & Coinbase Eth
0x2A116d4722B69bc5D1c3108aD974eEE37aA690Ae

hi @fabriguespe, sorry if this comes out as being desperate, I have not received the bounty yet, can you please let me when I would receive it?
by the way here's my address - 0x2A116d4722B69bc5D1c3108aD974eEE37aA690Ae

Check your dms!


Fix patching of is_connection_dropped from urllib3 (issue #468)

Fix patching of is_connection_dropped from urllib3
With old patching method, urllib3 never detected TCP connections that
were closed by the server side. For example, persistent HTTP connection
that were closed by the server (e.g., due to timeout) were not
recognized as closed. Any following requests that attempted to reuse
the same, closed connection caused the following failure:
urllib3.exceptions.ProtocolError: ('Connection aborted.',
   RemoteDisconnected('Remote end closed connection without response'
))

Fixes: https://github.com/kevin1024/vcrpy/issues/468

We also carry this as local patch, would be nice to see the issue fixed.


Update styled-components example README

v2 is no longer in beta.

@timneutkens not sure if you noticed, but I pushed the commit where I updated package.json too…

Oh sorry! 😄

Has been fixed already as far as I can see, not sure where we did 😅


Feat/adp integration

Implemented Analytics Data Platform UI
Customers can

Deploy ADP cluster through UI
manage ADP cluster through UI
Manage ADP services credentials through UI
See activity logs
Check ADP cluster details and progress

@radireddy Is there any internal ticket reference related to this feature?

CDS Report installation#3224.0 ✘

Stage 1

Install ✘

CDS Report installation#3231.0 ✘

Stage 1

Install ✘

CDS Report installation#3237.2 ✘

Stage 1

Install ✘

CDS Report installation#3238.0 ✘

Stage 1

Install ✘

ping @radireddy
There is also a conflict in the yarn.lock file.
Thanks!

All conflicts fixed and the EPIC has been updated in the PR


Add an initial boxen configuration

Can i have this merged please?

Just so you're aware, in future if you're changing files that are only used by you the convention on this repo is to make the commit message begin "[deanwilson]" and then just push it straight to master.


[UI] Cloud Credentials - Add support for showing errors

Version: v2.2.6, master-head (7/31/19)
What kind of request is this:
Bug
Summary:
This bug is intended to capture various issues that all related to Cloud Credentials not showing most errors in the UI.
Issues and Steps:

No error shown if cloud credentials fields are all are empty

Create a new cloud credential such as with EC2. Do not supply a name, region, access key, secret key.

Result: The UI will at least report you must pick a region, but if we do that and leave everything else blank it won't report any issues. We expected the UI to first check if credentials are null and if so first report that you must supply xyz. We then expected a check if the name is empty.
No error shown if create a cloud credential with a name that is not of type dnsLabel

When you create a cloud credential that does not conform to dnsLabel type the UI doesn't report any error or explanation.
You can attempt to create a cloud credential that has a name with spaces in it or underscores in it and no error is provided to the user.

Result: When you click "Create" nothing happens which is confusing.

Conclusion:
Fixing these issues will make user error and confusion less likely. It will make it more clear to the user when something has gone wrong if they make any mistakes.
Additional Information:
This issue was created based on these issues (which are now closed and captured in this issue to make things easier to work on).
https://github.com/rancher/rancher/issues/19759
https://github.com/rancher/rancher/issues/21503
https://github.com/rancher/rancher/issues/21215
https://github.com/rancher/rancher/issues/19677
https://github.com/rancher/rancher/issues/19758

Fixed in master-head

Version: master-head (v2.3) (8/13/19)
I tested each area:
No error shown if cloud credentials fields are all are empty
We now get messages when we don't supply EC2 cloud credential with Access Key and Secret Key

It's expected that the name is not required because it gets automatically generated if not supplied by the user.
No error shown if create a cloud credential with a name that is not of type dnsLabel
The UI now explains when invalid chars are in the name field:


Update docker_manager.py 'Create Bot' silently fails

docker does not allow . as a valid name for a mount point
linked to: https://github.com/hummingbot/docker-manager/issues/6

I think this PR makes sense but @cardosofede should approve since he created this file originally


Introduce "add cluster" command to Liqoctl

Description
This PR introduces

[x] the "add" command to create a new peering with a remote cluster.
[x] the "generate-add-command" peering to generate the command to perform the peering.
[x] the "chart-path" in install command to support different repository from the official one.

/test

/test

/test

/test

/test

/test

/test

/test

/test

/build

/test

/test

/test

/test

/merge


Weapons transformed are taking on the animations of the weapons they change to.

Currently, I am experiencing a problem with the mod taking on both the appearance and the attributes of the items that they are transformed into. I have been testing what I can change the Simply Swords chakram into, and I discovered that the weapon, while in this custom modpack, has been taking on the Better Combat animations of the weapon that it has been transformed into. For instance, I have been able to, previously, Transmog the chakram into a skillet from Farmers Delight. But now, in the pack I am using, it is instead causing the chakram to attack with the same animations and hitboxes as the skillet.
Below is a list of mods installed, with the version and author as well. We are using Fabric.
Towns and Towers
Towns-and-Towers-1.12-Fabric+Forge.jar
Biban_Auriu
Transmog
transmog-fabric-1.2.4+1.20.jar
Hidoni
Tridents 'n' Stuff
tridents_n_stuff-0.1.3+1.20.1.jar
fzzyhmstrs
Trinkets (Fabric)
trinkets-3.7.2.jar
EmilyPloszaj
Valkyrien Skies (Forge/Fabric)
valkyrienskies-120-2.1.3-beta.1.jar
triode_
Wizards (RPG Series)
wizards-1.0.5+1.20.1.jar
daedelus_dev
Xaero's Minimap
Xaeros_Minimap_23.9.7_Fabric_1.20.jar
xaero96
Xaero's World Map
XaerosWorldMap_1.37.8_Fabric_1.20.jar
xaero96
YDM's Red Panda
redpanda-forge-1.20.1-1.0.4.jar
YourDailyModderx
YDM's Weapon Master
weaponmaster-multi-fabric-1.20.1-3.0.5.jar
YourDailyModderx
YUNG's API (Fabric)
YungsApi-1.20-Fabric-4.0.4.jar
YUNGNICKYOUNG
YUNG's Better Nether Fortresses (Fabric)
YungsBetterNetherFortresses-1.20-Fabric-2.0.5.jar
YUNGNICKYOUNG
YUNG's Better Strongholds (Fabric)
YungsBetterStrongholds-1.20-Fabric-4.0.3.jar
YUNGNICKYOUNG
Thank you for your time.

Hey, could you tell me if the bug still occurs on the latest version of the mod?


Add an variable list yaml file output to the setup process

It would be good to have "mop setup" produce a yaml file with the format:

table1:

var1
var2

table2:
-var1
...

Which can then be used to request exactly the same variables.
This would allow a user to run first the configuration with "tables: all", to see what can be produced based on the mapping file used and then choose what they actually want to produce.

This has been added in tables branch. "mop setup" now produces a file called mop_var_selection.yaml including all the variables it matched for each table. And this can be used later as a varlist.yaml selection file.


'CREATE TABLE permission denied in database' despite using 'UseTempDB'

The agent account does not have table permissions. The same code is executed elsewhere, but with different models. The only notable difference is the second model contains a discriminator.
Despite setting the "BulkConfig":
        public void Execute(IList<GroupPupil> Model)//this works
        {
            var now = DateTime.UtcNow;
            Context.BulkInsertOrUpdate(Model.Where(a => a.End == DateTime.MaxValue).ToList(), new BulkConfig { UseTempDB = true });
            Context.BulkDelete(Model.Where(a => a.End < DateTime.MaxValue).ToList(), new BulkConfig { UseTempDB = true });
        }

        public void Execute(IList<Pupil> Model)//this fails
        {
            var now = DateTime.UtcNow;
            Context.BulkInsertOrUpdate(Model, new BulkConfig { UseTempDB = true });
        }

Non-temp table is created and insert fails:
SELECT TOP 0 T.[Id], T.[Discriminator], T.[GivenName], T.[Guid], T.[PreferredName], T.[SourceType], T.[Surname] INTO [Organisation].[PersonTemp280004e2] FROM [Organisation].[Person] AS T LEFT JOIN [Organisation].[Person] AS Source ON 1 = 0;

Using Latest - 2.3.6
Any advice?

There is remark about this in README:

For BulkInsertOrUpdate when UseTempDB is set then BulkOperation has to be inside Transaction.

Thanks, my bad. Great library though.


Is there any way to declare generic key?

TypeScript
interface Foo {
  [s: string]: number;
}

Is there any way to declare a runtype for the same? The following does't work. I assume there is a way that I don't know.
const Foo = Record({
  [s: string]: Number,
})

I believe you just want
import { Dictionary, Number } from 'runtypes';

const Foo = Dictionary(Number);


Bind variable that includes in its name the work 'hashedOldPassword' does not parse correctly

My Environment

ArangoDB Version:        3.3.11 enterprise
Storage Engine:             RocksDB
Deployment Mode:       Cluster
Infrastructure:               own
Operating System:        MacOS 10.14 beta
Used Package:              Docker - official Docker library

Whenever a bind variable is named hashedOldPassword the query does not return correct results.
For example (with the word hashedOldPassword):
Query string:
 LET user = FIRST(
 FOR u IN @@usersCollection
 	FILTER u._id == @userID && u.primaryEmail == @email && u.password == @hashedOldPassword
 	RETURN u)

 RETURN user != null

Execution plan:
 Id   NodeType          Site       Est.   Comment
  1   SingletonNode     COOR          1   * ROOT
  7   SubqueryNode      COOR          1     - LET #1 = ...   /* const subquery */
  2   SingletonNode     DBS           1       * ROOT
 11   IndexNode         DBS           1         - FOR u IN users   /* primary index scan */
 12   CalculationNode   DBS           1           - LET #4 = ((u.`primaryEmail` ==<EMAIL_ADDRESS>&& (u.`password` == "randomHashedPassword"))   /* simple expression */   /* collections used: u : users */
  5   FilterNode        DBS           1           - FILTER #4
 15   RemoteNode        COOR          1           - REMOTE
 16   GatherNode        COOR          1           - GATHER
  6   ReturnNode        COOR          1           - RETURN u
  9   CalculationNode   COOR          1     - LET #6 = (FIRST(#1) != null)   /* simple expression */
 10   ReturnNode        COOR          1     - RETURN #6

Indexes used:
 By   Type      Collection   Unique   Sparse   Selectivity   Fields       Ranges
 11   primary   users        true     false            n/a   [ `_key` ]   (u.`_id` == "users/0b3b564e-0525-595a-804e-d8ecc379ae1a")

Optimization rules applied:
 Id   RuleName
  1   remove-unnecessary-calculations
  2   use-indexes
  3   remove-filter-covered-by-index
  4   scatter-in-cluster
  5   distribute-filtercalc-to-cluster
  6   remove-unnecessary-remote-scatter

Result:
[
  false
]

Whereas without the word hashedOldPassword:
Query string:
 LET user = FIRST(
 FOR u IN @@usersCollection
 	FILTER u._id == @userID && u.primaryEmail == @email && u.password == @oldPassword
 	RETURN u)

 RETURN user != null

Execution plan:
 Id   NodeType          Site       Est.   Comment
  1   SingletonNode     COOR          1   * ROOT
  7   SubqueryNode      COOR          1     - LET #1 = ...   /* const subquery */
  2   SingletonNode     DBS           1       * ROOT
 11   IndexNode         DBS           1         - FOR u IN users   /* primary index scan */
 12   CalculationNode   DBS           1           - LET #4 = ((u.`primaryEmail` ==<EMAIL_ADDRESS>&& (u.`password` == "randomHashedPassword"))   /* simple expression */   /* collections used: u : users */
  5   FilterNode        DBS           1           - FILTER #4
 15   RemoteNode        COOR          1           - REMOTE
 16   GatherNode        COOR          1           - GATHER
  6   ReturnNode        COOR          1           - RETURN u
  9   CalculationNode   COOR          1     - LET #6 = (FIRST(#1) != null)   /* simple expression */
 10   ReturnNode        COOR          1     - RETURN #6

Indexes used:
 By   Type      Collection   Unique   Sparse   Selectivity   Fields       Ranges
 11   primary   users        true     false            n/a   [ `_key` ]   (u.`_id` == "users/0b3b564e-0525-595a-804e-d8ecc379ae1a")

Optimization rules applied:
 Id   RuleName
  1   remove-unnecessary-calculations
  2   use-indexes
  3   remove-filter-covered-by-index
  4   scatter-in-cluster
  5   distribute-filtercalc-to-cluster
  6   remove-unnecessary-remote-scatter

Result:
[
  true
]

I don't think this is related to the naming of the bind variable.
The execution plans for both queries look absolutely identical.
And the string "hashedOldPassword" is not used by ArangoDB itself in any context.
It rather seems this issue could be a side-effect of something else going wrong, e.g. https://github.com/arangodb/arangodb/issues/5727

I am running a different AQL query, using again bind variables. The query and the explanation are the following:
Query string:
 WITH @@usersCollection
 FOR v, e, p
 IN 1..1
 INBOUND @userID
 GRAPH @friendRequestsGraph
 OPTIONS { uniqueVertices: 'path' }

 FILTER p.vertices[1]._id == @friendID

 LET friendEdges = [@edge1, @edge2]
 FOR friendEdge IN friendEdges
 INSERT friendEdge IN @@friendsEdge
 REMOVE p.edges[0] IN @@friendRequestsEdge

 RETURN 1

Execution plan:
 Id   NodeType            Site       Est.   Comment
  1   SingletonNode       COOR          1   * ROOT
  5   CalculationNode     COOR          1     - LET friendEdges = [ "", "" ]   /* json expression */   /* const assignment */
 11   CalculationNode     COOR          1     - LET #15 = 1   /* json expression */   /* const assignment */
  2   TraversalNode       COOR          1     - FOR v  /* vertex */, p  /* paths */ IN 1..1  /* min..maxPathDepth */ INBOUND 'users/42a25854-4f6c-57ee-ada2-4e65073dd42f' /* startnode */  GRAPH 'friendRequestsGraph'
  9   CalculationNode     COOR          1     - LET #13 = p.`edges`[0]   /* simple expression */
  7   EnumerateListNode   COOR          2     - FOR friendEdge IN friendEdges   /* list iteration */
 17   ScatterNode         COOR          2       - SCATTER
 18   RemoteNode          DBS           2       - REMOTE
  8   InsertNode          DBS           0       - INSERT friendEdge IN friendsEdge
 19   RemoteNode          COOR          0       - REMOTE
 20   GatherNode          COOR          0       - GATHER
 13   DistributeNode      COOR          0       - DISTRIBUTE
 14   RemoteNode          DBS           0       - REMOTE
 10   RemoveNode          DBS           0       - REMOVE #13 IN friendRequestsEdge
 15   RemoteNode          COOR          0       - REMOTE
 16   GatherNode          COOR          0       - GATHER
 12   ReturnNode          COOR          0       - RETURN #15

Indexes used:
 By   Type   Collection           Unique   Sparse   Selectivity   Fields      Ranges
  2   edge   friendRequestsEdge   false    false            n/a   [ `_to` ]   base INBOUND

Traversals on graphs:
 Id   Depth   Vertex collections   Edge collections     Options                                   Filter conditions
  2   1..1    users                friendRequestsEdge   uniqueVertices: path, uniqueEdges: path   (p.`vertices`[1].`_id` == "users/ea514435-619e-564c-a420-ff19b001b7e9")

Optimization rules applied:
 Id   RuleName
  1   move-calculations-up
  2   move-filters-up
  3   remove-redundant-calculations
  4   remove-unnecessary-calculations
  5   move-calculations-up-2
  6   move-filters-up-2
  7   remove-data-modification-out-variables
  8   optimize-traversals
  9   remove-filter-covered-by-traversal
 10   remove-unnecessary-calculations-2
 11   distribute-in-cluster
 12   scatter-in-cluster

Write query options:
 Option                   Value
 ignoreErrors             false
 waitForSync              false
 nullMeansRemove          false
 mergeObjects             true
 ignoreDocumentNotFound   false
 readCompleteInput        true
 useIsRestore             false
 consultAqlWriteFilter    false

However I get this error:
AQL: Error message received from shard '' on cluster node 'PRMR-565f1484-128e-4e8b-b67d-633b7a872ce3': document not found (while executing)
After analyzing the query I isolated the issue when the server tries to remove the last edge from friendRequestsEdge
However, when I remove the bind variable and I execute the query using the collection directly in the AQL, the query runs as expected
Could those issues be related somehow?

I don't think the bind variable plays any role here.
I think there is something wrong with the distribution of the documents to the shards in some case, as indicated by #5727. This leads to arbitrary follow-up failures.
So we need to fix #5727. Work on this is under way already.

Hi @tkallioras ,
Did you have a chance to test this on the latest version(3.3.13) ? Does this issue still persists?
Thanks!

It seems to be solved in version 3.3.13


IntelRealsense rs2::wrong_api_call_sequence_error

Required Info

Camera Model
{  D455 }

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
{Win10, Raspbian}

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC/Raspberry Pi

SDK Version
{ legacy / 2.. }

Language
{C++/labview/nodejs/opencv/pcl/python/unity }

Segment
{Robot/Smartphone/VR/AR/others }

Hello! I write code with c++ and librealsense. I build realsense sdk 2.53.1 with flag BUILD_NETWORK_DEVICE on Windows and -DBUILD_NETWORK_DEVICE and -DFORCE_RSUSB_BACKEND on Raspbian. I can view frame in IntelRealsenseViewer but in code i cannot get frame.Here is my code:

If i start with debugger i get this error:

I built older versions of realsense sdk but it didn't change anything

I can find solution


Python: Support grouped exceptions

QL part of https://github.com/github/semmle-code/pull/44008

Closed in favour of https://github.com/github/codeql/pull/11244.


Add bash completion with default handling

Re-adds bash completion with default option as suggested in https://github.com/cloudfoundry/homebrew-tap/issues/28.
In case there is no completion implemented for custom go-flags args or option, it falls back to default file and folder completion. As a small drawback, this also happens where file completion might not be appropriate, as for example with the -d option.
Tested on Mac and on Ubuntu, probably Mac is just fine as this is for homebrew.
Fixes #175

Merge story: #153676868

@tjvman Sure! Thanks.

ping @cppforlife @dpb587-pivotal @tjvman can we pull this in? Any objections to merging this?

didnt we break something last time auto conpletion was merged?
Sent from my iPhone

On Feb 1, 2018, at 4:28 AM, Marco Voelz<EMAIL_ADDRESS>wrote:
ping @cppforlife @dpb587-pivotal @tjvman can we pull this in? Any objections to merging this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

IIRC file completion didn't work anymore. I think this was specifically tested and fixed by this new approach.
From: Dmitriy Kalinin<EMAIL_ADDRESS>Reply-To: cloudfoundry/bosh-cli<EMAIL_ADDRESS>Date: Thursday, 1. February 2018 at 18:35
To: cloudfoundry/bosh-cli<EMAIL_ADDRESS>Cc: "Voelz, Marco"<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [cloudfoundry/bosh-cli] Add bash completion with default handling (#361)
didnt we break something last time auto conpletion was merged?
Sent from my iPhone

On Feb 1, 2018, at 4:28 AM, Marco Voelz<EMAIL_ADDRESS>wrote:
ping @cppforlife @dpb587-pivotal @tjvman can we pull this in? Any objections to merging this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/cloudfoundry/bosh-cli/pull/361#issuecomment-362342829, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACJv95t4b8KxOwQDjy9dpRbP2AAi6yseks5tQfXEgaJpZM4QroA_.

Looks good and works for me.


ERC-721 Tokens Exercise

Implement a solution for the ERC-721 Tokens Exercise

Submitted to Base Goerli: https://goerli.basescan.org/address/0x7AD2189429018429bfb10FDDaB9d06C8ce7d4cF3#code


[Feature] Add check for available firmware

The problem
Can you add a diagnostic update entity that checks for new firmware available?

Hello, @sepodele,
Thanks for your feedback! :+1:
This feature is possible and I will add it to the to-do list. Unfortunately, I cannot yet guarantee the exact timeframe to implement it.

Will be available in 0.11.0 soon


img link should be start from https.

https://github.com/angular/material2/blob/76a6e7beab28a30d6c215600e1122808049ed83c/src/material-examples/card-fancy/card-fancy-example.html#L7

chrome give error that

Mixed Content: The page at 'https://material.angular.io/worker-basic.min.js' was loaded over HTTPS, but requested an insecure resource 'http://material.angular.io/assets/img/examples/shiba2.jpg'. This request has been blocked; the content must be served over HTTPS.

Duplicate of https://github.com/angular/material2/issues/7757; PR #7800

fixed in #7762, should be close this.


Does this work with opencart marketplace?

Hi, I am using opencart marketplace and would like to know if my sellers can get bitcoins with this gateway.

Seeing as opencart marketplace is a paid plugin we do not test compatibility so we'd very grateful if you could test it for us. For the time being, we have no plans to write a plugin for/verify our plugin with opencart marketplace.


FEATURE: validate text field length based on record objects allowed length

Fixes #3681

Just reviewing the UserVoice features that are "started" any progress on this one?

Sorry, I got Christmas'd. Will have a look at this as soon as I get an opportunity next week and start working on the approach outlined by @tractorcow.

Any movement on this? It's listed as Started on UserVoice https://silverstripe.uservoice.com/forums/251266-new-features/suggestions/6786049-validate-text-field-length-based-on-record-objects

Another bug for a rainy day I guess.

Sorry for letting this go stale. I'm going to close this PR as its too far gone down the beaten track.

Thanks @stevie-mayhew I'll update the roadmap stuff to reflect this today.


Part Two: Commit Secrets to the Repository

(adding comment to try to trip alert; the below keys are not valid)
aws_access_key_id=AKIAQYLPMN5HHHFPZAM2
aws_secret_access_key=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


Mention that filesystems are mounted alphabetically

I just switched my root to be on tmpfs. I decided to start with mounting the real root fs to /persistent and bind-mounting everything from there. Except that it wouldn't boot because it tried to mount /etc before mounting /persistent.
I discovered that naming the mount point /apersistent fixed it tho x) Apparently the filesystems are loaded alphabetically. Is that expected or did I miss something? Will this stay that way with future nix versions? If that's expected behavior then I think it deserves to be mentioned in the README.

Yes, you need to mark the persistent filesystem neededForBoot in order for it to be mounted early enough. This is mentioned in the README.org at the end of the NixOS section (on line 61).

I did that, but it was not enough, because /nix is also neededForBoot, but /nix is supposed to be bind-mounted from /persistent.

In the past I think I worked around this by making my persistent fs be /nix and make /nix/state my impermanence dir. You can see the setup here: https://github.com/lovesegfault/nix-config/blob/master/systems/fourier.nix
It's not great, but it works.

I see. Is there a reason to set it up like this, if you don't mind me asking? If you want to use /nix for persistent storage, you can just use a subpath of it, like /nix/persistent.
I also use /persistent for persistent storage, but it's a mounted btrfs subvolume, not a bind mount. Here is my filesystem config: https://github.com/talyz/nixos-config/blob/master/machines/evals/configuration.nix#L82-L110. I also create a fresh btrfs subvolume on boot and use it as root, which saves memory and provides a way to recover in case of crashes, power outages or forgetfulness ;)

My main reason for setting up /persistent the way I do is that I started from an existing normal nixos installation. So /persistent points to the root of a normal installation, with its /nix, /home, /etc and so on. Btw I want to mention that apart from having to name the mountpoint /apersistent, the rest of the transition was extremely smooth!
Fresh btrfs sounds like a pretty cool idea! But so far my non-persisted / takes 20M tops.

Okay, that makes sense. I guess the clean solution would be moving to something a bit more flexible, like btrfs subvolumes or separate partitions for persistent state. I don't think we can do much here - if the bind mount dependency ordering should be applied anywhere, it's in NixOS.
Yeah, most of the time my root is pretty small too, but when I used tmpfs I ran out of memory more than once due to not planning ahead 😅

Yeah, we'd need NixOS to support something like specifying dependencies between mouts. That's why I thought mentioning it in README is the best we can do from here

It is the only thing we can do, but I'm not sure how to mention it or if we should. This really is a last resort setup which should be avoided if it can be; I don't want to suggest that it's somehow common or recommended or even something most people will have to bother with. Do you, or anyone else, have a suggestion? Also, it might be good to open an issue against the NixOS/nixpkgs repo, so this might get fixed and we have something to refer to.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/384

I haven't tested but it should be fixed by https://github.com/nix-community/impermanence/pull/115

This still causes problems in somewhat niche cases such as this:
    home.persistence = {
      "/persist${user.home}".directories = [ ".config/Code" ];
      # Bastard: https://github.com/microsoft/vscode/issues/3884
      "/cache${user.home}".directories = [
        ".config/Code/Cache"
        ".config/Code/CachedConfigurations"
        ".config/Code/CachedData"
        ".config/Code/CachedExtensionVSIXs"
        ".config/Code/CachedExtensions"
        ".config/Code/CachedProfilesData"
        ".config/Code/Code Cache"
        ".config/Code/DawnCache"
        ".config/Code/GPUCache"
        ".config/Code/Service Worker/CacheStorage"
        ".config/Code/Service Worker/ScriptCache"
      ];
    };

I am assuming that setting the name of the persistence locations to something like 0_persist and 9_cache is a workaround, but definitely not a solution.
Should we consider creating a dependency graph similar to #97, sorting by target path, or at least giving a custom order option? If so, I think this issue should be reopened.


Add techstack with logos in readme

I wil like to add all the tech stacks used in this project with their logo this will make the read me more attractive !!

@rohank2502 Sure go for it :)


OAW 7B Expected Hash & Adding TimeServer parameter

@yil578 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@yil578
Merging this. In the future, when you have significant changes to make, would you please work in the private repo? PRs in the private repo run validation and Acrolinx, and the changes can be checked on staging (especially helpful with table additions, as here).
Thanks!


PR restricting shortening to specific domains, implementing validators and env. variables

Summary | Résumé
This PR takes care of the following:

Implements a new table called allowed_domains that contains entries of allowable domains that the url can shorten.
Added a README file in the db_migrations folder
Uses python library validators to "validate" whether a url is valid or not
Implements function is_domain_allowed which checks to see if the domain exists in the allowed_domains table. If not, a link to a GC form is embedded in the error message and if it exists, we can continue with the shortening process.
Implements environment variables

I have made all the changes requested and for the ones that changes were not immediately made, new issues have been submitted and we can tackle them at a later time.


Permission error with the offline manager

Running the scripts/manager_offline_eeg.py results in a permission error that is likely due to the fact the the PhysioNet data are saved in an inaccessible path. Here is the error, @AntoineBellemare has the same on his desktop :
Traceback (most recent call last):
  File "/home/hyruuk/GitHub/goofi-pipe/scripts/manager_offline_eeg.py", line 30, in <module>
    mngr.run()
  File "/home/hyruuk/GitHub/goofi-pipe/goofi/manager.py", line 110, in run
    self.update()
  File "/home/hyruuk/GitHub/goofi-pipe/goofi/manager.py", line 94, in update
    out.update(self.data_in, finished, intermediates)
  File "/home/hyruuk/GitHub/goofi-pipe/goofi/data_out.py", line 239, in update
    self.client.send(bundle.build())
  File "/home/hyruuk/.local/lib/python3.10/site-packages/pythonosc/udp_client.py", line 55, in send
    self._sock.sendto(content.dgram, (self._address, self._port))
PermissionError: [Errno 13] Permission denied
sys:1: ResourceWarning: unclosed <socket.socket fd=12, family=AddressFamily.AF_INET, type=SocketKind.SOCK_DGRAM, proto=0, laddr=('<IP_ADDRESS>', 60962)>

Don't think this is relevant anymore?


Add func to when a paginator is closed either by cancellation or timeout

Description
It would be useful to have a way to pass a func into a LazyPaginatorBuilder and StaticPaginatorBuilder.
How is this gonna be useful?
This would be useful for doing certain actions when a paginator is closed.
My use case for this would be checking if a user already has an open paginator or not.
Example implementation
var paginator = new LazyPaginatorBuilder()
    .AddUser(Context.User)
    .WithPageFactory(GeneratePages)
    .WithMaxPageIndex(10 - 1)

    .AddOption(new Emoji("⏪"), PaginatorAction.SkipToStart) // Use different emojis and option order.
    .AddOption(new Emoji("◀"), PaginatorAction.Backward)
    .AddOption(new Emoji("⏹"), PaginatorAction.Exit)
    .AddOption(new Emoji("▶"), PaginatorAction.Forward)
    .AddOption(new Emoji("⏩"), PaginatorAction.SkipToEnd)

    .WithCacheLoadedPages(true)
    .WithActionOnCancellation(ActionOnStop.DisableInput)
    .WithActionOnTimeout(ActionOnStop.DisableInput)
    // New implementation.
    .WithActionOnStopped(RemoveUserCooldown)
    .Build();

await Interactive.SendPaginatorAsync(paginator, Context.Interaction, TimeSpan.FromSeconds(30), resetTimeoutOnInput: true);

async Task RemoveUserCooldown(StopType type)
{
    ...
    return Task.CompletedTask;
}

StopType here indicates whether the paginator timed out or was cancelled
Alternative ways this can currently be done
InteractionModuleBase has a AfterExecute method that can be overridden, in that method you could then check for that exact command and take appropriate actions. This method is only called after the paginator has stopped.

I feel that this way of handling the result of paginators (via delegates) is somewhat clumsy, a cleaner way to do this would be to simply check the result of SendPaginatorAsync:
var result = await Interactive.SendPaginatorAsync(...);

if (result.IsCanceled)
{
    // paginator stopped because it was canceled
}
else if (result.IsTimeout)
{
    // paginator stopped because of a timeout
}

If SendPaginatorAsync has to be discarded, then wrapping the code inside Task.Run also works:
_ = Task.Run(async () =>
{
    var result = await Interactive.SendPaginatorAsync(...);

    // post-execution logic
});

A way to hide this logic (and use delegates) is by creating an extension method that sends a paginator and accepts a delegate parameter:
public static class InteractiveServiceExtensions
{
    public static async Task SendPaginatorWithDelegateAsync(this InteractiveService interactiveService, Paginator paginator, SocketInteraction interaction,
        Func<InteractiveMessageResult, Task> resultAction, TimeSpan? timeout = null, InteractionResponseType responseType = InteractionResponseType.ChannelMessageWithSource,
        bool ephemeral = false, Action<IUserMessage> messageAction = null, bool resetTimeoutOnInput = false, CancellationToken cancellationToken = default)
    {
        var result = await interactiveService.SendPaginatorAsync(paginator, interaction, timeout, responseType, ephemeral, messageAction, resetTimeoutOnInput, cancellationToken);
        await resultAction(result);
    }
}


Comparing date/time values

Hi, i have code like this..
dts = Timex.parse!( "2019-01-01T00:00:00.000Z", "{ISO:Extended:Z}" )
now = Timex.parse!( "2019-01-01T00:00:00.100Z", "{ISO:Extended:Z}" )

dt = Timex.shift( dts, seconds: 5 )

if ( dt <= now ) do
	raise ( "why? dt already shift seconds... dt[#{ inspect dt }] now[#{ inspect now }]" )
else
	:ok
end

but i thought that such a comparison should be the opposite result…

In Elixir (and Erlang) the comparison operators do not work with structs, such as DateTime. See this section of the Erlang manual for details on how those comparison operators work (the actual operators are slightly different than in Elixir, but the behavior of the operators is the same).
Instead, you need to use the comparison functions provided in the standard library (or the Timex APIs, like compare or before?/after?, etc.).

thank you very much 👍


Getting unsigned repo error when updating via apt on Ubuntu

VSCode Version: 1.21.0-insider
OS Version: Ubuntu 17.10

Steps to Reproduce:

Run 'sudo apt update'

On running step 1 this error is shown:

I've tried removing the existing /etc/apt/sources.list.d/vscode.list, reinstalling via dpkg to create a new vscode.list file and the same error occurs when subsequently running 'apt update'.

Does this issue occur when all extensions are disabled?: Yes/No

ah, my bad, not vscode.  Apologies


Bump @types/jest to 24 and fix the associated typings and tests

Also add a test for mocking classes.
I'm a bit unsure about the changes in ts-jest-transformer.spec.ts - it seems like the typings for configsFor changed and are expecting something quite different than what the mocks are returning.  Having to do "as unknown" isn't good - but the tests pass?
Fixes #991

This is pretty nice. @huafu you did the mocking part, can you take a look to see whether or not this makes sense?

You can use as Partial as T instead

Sure, as Partial<T> as T also works, but that's just a different way to flat out lie to the Typescript type system - neither better nor worse than the way I did it here with as unknown.
The problem in ts-jest-transformer.spec.ts is: typescript infers the return type of the configsFor method of TsJestTransformer to be ConfigSet.  (It's not annotated in the definition, but the body returns a ConfigSet from every code path.)  However, in ts-jest-transformer.spec.ts that function is being mocked to return objects that look nothing like a ConfigSet.  This inconsistency was already there before this PR - but this PR allows the typescript type system to notice it, so the test code began showing the error.  To get around it, I'm using as unknown to suppress the error--and sure, as Partial<ConfigSet> would also do the same trick.  But what if the error being suppressed is actually a real problem?  If so the error should be fixed, rather than suppressed.
I don't have enough understanding of what configsFor is supposed to return to answer this question. Maybe the code has adaptations to what the mocks are returning, and maybe the type of objects being returned from the mocks should be annotated in the return type of configsFor?  Or maybe the test just "happens to work" even though it's doing something the code didn't intend to support?  I don't know - hopefully someone more familiar with this code can answer.  I submitted this with the error suppression figuring at least this wouldn't make anything worse since the type error (if it is really one) was already there to begin with.

I merged with latest master to get rid of the merge conflict - let me know if I should have rebased & force pushed instead to avoid the merge commit.

Pull Request Test Coverage Report for Build 2563

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 91.601%

Totals

Change from base Build 2556:
0.0%

Covered Lines:
915

Relevant Lines:
967

💛  - Coveralls

cc @GeeWee

I think this is good, but only Huafu really knows this part of the code and he's AWOL. Let's merge it in and see what happens.

(I closed this by mistake, will merge it in as soon as CI passes again)


Update to 1.16.2 and 1.16.3

Although installing 1.16.1ver mod for 1.16.2 is possible, yet clicking the button causes crash. Please update the mod to 1.16.2 and 1.16.3.

Please @Draylar

Please update. @Draylar

Feel free to PR the update if you want it now, I am busy at the moment

seemed to just require a rebuild in newer versions, fixed in 1.1.1 jar build on GH/CF

Does not work when editing a sign.
Directly inputing § does not work.
Causes crash when sending messages with a §, and NotEnoughCrashes reports that this crash may be associated with LambDynamicLight and VoxelMap.


Request payload with human readable names

As it is now the request payload when parsing json must contain IDs of the application and environment. It should be possible to replace these IDs with the names instead.

Looks like this is already implemented.


Tool to update packages

Updating packages in Mariner takes a lot of manual steps, and this isn't really good for the velocity we need to address security vulnerabilities in the tight SLAs we're working with.  I'd like a tool that would automate a lot of these things:

Bump the package release automatically.
Create a new changelog entry (using my name/email from the Git configuration), possibly opening in $EDITOR.
Update the manifests for all the subpackages in the spec file.
If you're updating a package to the new version, try to guess the URL automatically by changing the old version in the URL to the new version; if it doesn't work, it can ask the actual URL for the new version. Regardless, it should update the *.signatures.json files.
If it's a package that's flagged to be part of the toolchain, some additional steps might be involved.
It can optionally build the package to see if it works, determining the build-dep packages from the specfile and building them too.
Once everything is said and done, it'll create a new Git commit with the changes; the message can include the specfile changelog entry.

I was thinking about this for the CLI interface; for instance, to update a package to a new upstream version:
$ mariner-up bash --new-upstream
Currently packaged version is 4.9.
Source currently lives in https://some/gnu/mirror/bash-4.9.tar.gz

*** New version? 5.0

Trying to fetch version 5.0 from https://some/gnu/mirror/bash-5.0.tar.gz
Fetch successful, using this URL for the new source tarball.

Bumping package release from 29 to 30.
Creating ChangeLog entry:
    Mon Aug 17 2020 Leandro Pereira<EMAIL_ADDRESS>5.0-30
    - Update to upstream version 5.0.

*** Edit with /usr/bin/joe (y/N)? n

Git commit created.

*** Package has not been built; verify it builds and update manifests (Y/n)? Y

Building package. Follow progress in /tmp/asdfasdasdf-bash.log
Package built successfully in 00:03:37.
Updating manifests and amending Git commit.

Some of the questions that are asked could be passed as command-line arguments (e.g. --build-package to avoid asking if you want to build the package).
@microsoft/cbl-mariner-devs: Questions? Comments? Suggestions?

Thank you for your suggestion.  At this time your suggestion is being tracked internally for future consideration.


feat: [WIP] CaraML SDK

[TBA]

After the a3ff01e commit, test results are
 86 failed, 124 passed, 7 skipped, 23 warnings in 23.69s

All non integration tests passing now.
 python3 -m pytest -n 8 -m "not integration" -W ignore tests/models/
======== test session starts =======
platform darwin -- Python 3.8.5, pytest-8.0.0, pluggy-1.4.0
rootdir: /Users/kuppilpawanmaniteja/Documents/github/caraml-sdk/packages/caraml
plugins: cov-4.1.0, dependency-0.6.0, xdist-3.5.0
8 workers [160 items]
.................[100%]
======= 160 passed in 9.37s =======

Turing wrapper code added to the PR.  One test case failing, on first look, it seems to be an integration test which connects to turing server. Can be checked later.
================================== short test summary info =============================
FAILED tests/routers/ensembler_test.py::test_list_ensemblers[6] - AssertionError: Regex pattern did not match.
=========== 1 failed, 375 passed in 8.12s =======


Conflict between LEAF and MAP after upgrading spring-cloud-config-server

I'm using spring-cloud-config-server 1.2.2..RELEASE.
I need to update it to a newer version (e.g. 1.3.1..RELEASE, but the issue happens in the latest version as well). When I tried to update the version and performed a call to the YAML endpoint of the config server (with test profile requested), I got an error (ClassCastException: java.lang.String cannot be cast to java.util.Map) on some of my properties.
These properties are defined in application.yaml as a LEAF:
logging.level.com.company: DEBUG
But are defined as MAP on my application specific profile YAML (application-test.yaml):
logging.level.com.company.salesforce.editor.service: DEBUG
Previously, I got in the response YAML something like:
logging:
    level:
        com:
            company: DEBUG
            company.salesforce.editor.service: DEBUG

I debugged the spring-cloud-config-server code I found out there was a major change (from 1.2.x to 1.3.x) in EnvironmentController.java\convertToMap(Environment input, boolean resolvePlaceholders)() that probably caused it.
I found this similar issue: ClassCastException with specific logging settings But it wasn't helpful to me.
How can I use the new version, but get the same response as in the old version? If not possible, what is another course of action I can take?

I'm very sorry, didn't mean to spam - I just didn't know what is the best way to contact. I commented back to you on stack overflow.


Handle Cases in Product where Text Does Not Exist

There are cases where returning a Product Entity, the text field does not exist. This catches that issue.
Resolves Issue #61

Error in Travis Build is related to a change that has occurred in evaluating urls with the filter_var, note here https://3v4l.org/NERQ1 up to PHP 5.6.27 this would have been false, but now evaluates as true starting in PHP 5.6.28 and forward. Possibly related this bug fix in PHP https://bugs.php.net/bug.php?id=73192 Here are the notes for PHP 5.6.28 https://secure.php.net/ChangeLog-5.php#5.6.28

Thank you! Will change travis setup.

Thanks @Swader points to you.


Document the deprecated kubernetes.io/psp annotation

Solves #30578 according to that comment https://github.com/kubernetes/website/issues/30578#issuecomment-974691745.
I took the liberty to change a command in the tutorial as well because I added a step to check that annotation.
- kubectl-admin create -f example-psp.yaml
+ kubectl-admin create -f https://k8s.io/examples/policy/example-psp.yaml

/lgtm


en: update error-handling.md

What is changed, added or deleted? (Required)
update error-handling.md
Which DM version(s) do your changes apply to? (Required)

[x] master (the latest development version, including v2.0 changes for now)
[ ] v1.0 (TiDB DM 1.0 versions)

What is the related PR or file link(s)?

This PR is translated from: https://github.com/pingcap/docs-dm/pull/111
Other reference link(s):

@WangXiangUSTC PTAL, thanks~

/merge


Add Star-Trek Style Engines for KSPI from Kerbal Stuff

This pull request was automatically generated by Kerbal Stuff on behalf of ABZB, to add Star-Trek Style Engines for KSPI to CKAN.
Please direct questions about this pull request to ABZB.

Depends on KSPI-Extended, which we don't index due it overwriting files in KSP-Interstellar. If KSPI-E ever gets a full release this can be added.


feat(region): make DeleteSnapshotExpiredRelease applies to postpaid vm

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

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

release/3.7
release/3.6
release/3.5

/area region
/cc @swordqiu @zexi

Codecov Report

Merging #9681 (763086d) into master (9411d7e) will increase coverage by 2.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #9681       +/-   ##
==========================================
+ Coverage    5.99%   8.06%    +2.07%
==========================================
  Files         847     107      -740
  Lines      131609   12977   -118632
==========================================
- Hits         7887    1047     -6840
+ Misses     123021   11810   -111211
+ Partials      701     120      -581

Flag
Coverage Δ

aFlag
8.06% <ø> (+2.07%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/compute/models/snapshotpolicycache.go

pkg/mcclient/options/ansibleplaybooks.go

pkg/monitor/models/commonalert.go

pkg/util/fileutils2/writable_others.go

pkg/util/vmdkutils/vmdkutils.go

pkg/util/rbacutils/rbac.go

pkg/util/samlutils/request.go

pkg/baremetal/tasks/maintenance.go

pkg/cloudcommon/notifyclient/notify.go

pkg/util/qemuimg/consts.go

... and 729 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 9411d7e...763086d. Read the comment docs.


Moved: Zoom.Zoom.EXE 6.0.0(37205)

Microsoft Reviewers: Open in CodeFlow

https://github.com/microsoft/winget-pkgs/pull/150124


Update CQD-Power-BI-query-templates.md

Added entry for new AA & CQ Historical reporting
Created secondary page that is hopefully linked
Uploaded images for secondary page that are also hopefully linked

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

Hi Johan,
For some reason I can’t view your changes to this PR – they’re not coming up like the other ones (no “view changes” button).
Based on what’s below – I approve the correction to my type (thanks for catching that).
Will this be sufficient?
I will continue to look and see if I can approve the changes directly.
Colin
From: JohanFreelancer9<EMAIL_ADDRESS>Sent: October 26, 2020 23:41
To: MicrosoftDocs/OfficeDocs-SkypeForBusiness<EMAIL_ADDRESS>Cc: Colin Longman<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/OfficeDocs-SkypeForBusiness] Update CQD-Power-BI-query-templates.md (#6034)
@JohanFreelancer9 commented on this pull request.

In Teams/CQD-Power-BI-query-templates.mdhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-SkypeForBusiness%2Fpull%2F6034%23discussion_r512447260&data=04|01|Colin.Longman%40microsoft.com|d0691a0ba1ac494a6d5b08d87a434e99|72f988bf86f141af91ab2d7cd011db47|1|0|637393776788378579|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=uRuLoFRh6I0rtTDS5jODWU04N%2BmDtzO4NMH%2FIs%2FDs2k%3D&reserved=0:

@@ -31,6 +31,7 @@ Make sure you have the right [CQD access role](https://docs.microsoft.com/micros

|  |  |
|---------|---------|
+|(New!) CQD Teams Auto Attendant & Call Queue Historical Report.pbit     |  This template provides the following three reports:Auto Attendant – showing analytics for calls coming into your Auto AttendantsCall Queue – showing analytics for calls coming into your Call QueuesAgent Timeline – showing a timeline view of agents being active in Call Queue CallscallsTo learn more, read Use CQD Power BI report to view Auto Attendant & Call Queue Historical Report.        |
⬇ Suggested change
-|(New!) CQD Teams Auto Attendant & Call Queue Historical Report.pbit     |  This template provides the following three reports:Auto Attendant – showing analytics for calls coming into your Auto AttendantsCall Queue – showing analytics for calls coming into your Call QueuesAgent Timeline – showing a timeline view of agents being active in Call Queue CallscallsTo learn more, read Use CQD Power BI report to view Auto Attendant & Call Queue Historical Report.        |
+|(New!) CQD Teams Auto Attendant & Call Queue Historical Report.pbit     |  This template provides the following three reports:Auto Attendant – showing analytics for calls coming into your Auto AttendantsCall Queue – showing analytics for calls coming into your Call QueuesAgent Timeline – showing a timeline view of agents being active in Call Queue callsTo learn more, read Use CQD Power BI report to view Auto Attendant & Call Queue Historical Report.        |
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-SkypeForBusiness%2Fpull%2F6034%23pullrequestreview-517385541&data=04|01|Colin.Longman%40microsoft.com|d0691a0ba1ac494a6d5b08d87a434e99|72f988bf86f141af91ab2d7cd011db47|1|0|637393776788388570|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=fYKry9wH6Nv5VuqOt%2FVtp473HI%2F9EOu04x4k6OvwBPY%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQGHZH2DV6WRGQSCYS3RL53SMZTQZANCNFSM4S3M3HFA&data=04|01|Colin.Longman%40microsoft.com|d0691a0ba1ac494a6d5b08d87a434e99|72f988bf86f141af91ab2d7cd011db47|1|0|637393776788398566|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=5LGPRYRMI6TKMG%2Be02wdUPIZ4DiEoS4kUGWNGR2bG7I%3D&reserved=0.

@SerdarSoysal PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@SerdarSoysal  Hoping you can review and approve this today as documentation is the only thing holding us back from going GA with this.
If there are any questions please let me know.
Thanks!

@CLYVR - Apologies Colin. I have no idea what happened to my previous edits. I have redone them; mainly punctuation and a few inconsistencies. Please have a look at my suggestions, remembering that they are ONLY suggestions, and include as you see fit. Many thanks.

@JohanFreelancer9 THANK YOU for all the edits - greatly appreciated!

@JohanFreelancer9 Hi Johan.  Do you know who this is waiting on for final approval/to be merged into live?  Really need to get this launched and the documentation update is all that is holding things back at this time.  Thanks!

@CLYVR Hi Colin, best would be to follow up with @scanum. Thanks.

@scanum Hi Manuela.  Would appreciate if you would take a look at this PR and approve it or let me know what is needed from me to get this (and 6037) merged into the public docs.  We are trying to get this new report out to our customers and having the documentation updated is the only pending component.  Thanks!


Better solution for a common IRC exception

One area where the clients differ is the need for this in enrobie.clients.irc.client. Otherwise we receive a traceback in the logs every time we signal to exit.
except ConnectionError:
    self.__status('pending')

I think there just needs to be better handling in enconnect.irc.client.

See note about NOCVR on self.__status('pending') in enrobie/clients/irc/client.py.


Support AD/AAD account

raised by MS tech sales team.
Work Break Down:

Backend user management including the end2end experience – @ydye @wangdian  #2531

Web portal authentication process with AAD –  @ydye #2532

Storage Manager support group configuration  – @wangdian  #2530

When implement also consider the relationship or impact on these items: #2078 #2085 #921.
Design Finalize ETA 4/10
Test Case

This is a related doc: https://docs.microsoft.com/en-us/azure/aks/aad-integration

https://github.com/Microsoft/pai/issues/1811

Design goal for storage changed. Please refer to #2530

done after v0.15


OSX Catalina: 'cfn init' terminated by signal SIGABRT (Abort)

I'm not sure if this belongs here, as it's an OSX Catalina / homebrew python issue. I assume people will look here when they run into this error, so maybe it's something to document.
If you run cfn init outside of a virtualernv on OSX Catalina with python3.7 (installed via homebrew), you get this behaviour and error:
~$: cfn init --verbose
Initializing new project
What's the name of your resource type?
(Organization::Service::Resource)
>> MyOrg::Foo::Bar
Select a language for code generation:
[1] python36
[2] python37
(enter an integer):
>> 2
fish: 'cfn init --verbose' terminated by signal SIGABRT (Abort)

Looking at the error report that is generated, this seems to be an issue related to pyhton3 and libcrypto:
Application Specific Information:
/usr/lib/libcrypto.dylib
abort() called
Invalid dylib load. Clients should not load the unversioned libcrypto dylib as it does not have a stable ABI.

I googled for a solution and I there is a clean workaround (besides running cfn in a virtualenv:
# Install openssl, if it's not installed already
brew install openssl
# Use the openssl libraries
DYLD_LIBRARY_PATH=/usr/local/opt/openssl/lib:$DYLD_LIBRARY_PATH cfn init

ouch, good trouble-shooting! doesn't seem like anything we can work around in the code, but definitely worth adding to the README

Could this situation at least be detected and reported, so that the user knows what action to take?

none of us are running Catalina, so it's hard to reproduce and test. loads of people are hitting this though in all kinds of situations:

https://stackoverflow.com/questions/58272830/python-crashing-on-macos-10-15-beta-19a582a-with-usr-lib-libcrypto-dylib
https://forums.developer.apple.com/thread/119429
https://github.com/Homebrew/homebrew-core/issues/44996
https://github.com/saltstack/salt/issues/55084

particularly, the first and last issue seem to indicate upgrading the asn1crypto package helped. problem is, we don't pull that in directly or indirectly. i assume this is why you don't hit this in a virtual environment. and hacking around broken system dependencies is a bit out of scope. it's unlikely we'll address this any further than pinning and keeping the issue open (for now).


Issue in model.pre_train when setting processed=True in model.preprocess_data

Hello,
I have an issue at the step in which the autoencoder is pretrained only when I give a preprocess anndata object (it works if the adata object is not preprocessed beforehand):

Preprocess data step:

# fit in data
model.get_data(adata=data,                   # count or expression matrix, (dense or sparse) numpy array
               labels = data.obs['cluster_label'],       # (optional) labels, which will be converted to string
               gene_names = data.var['features'], # (optional) gene names, which will be converted to string
               cell_names = data.obs['sample_name']    # (optional) cell names, which will be converted to string
              )

# preprocess data
model.preprocess_data(gene_num = 2000,        # (optional) maximum number of influential genes to keep (the default is 2000)
                     data_type = 'Gaussian', # (optional) data_type can be 'UMI', 'non-UMI' or 'Gaussian' (the default is 'UMI')
                      npc = 64,         # (optional) number of PCs to keep if data_type='Gaussian' (the default is 64)
                     processed=True)

Pretrain step:

---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-9-2da55840b803> in <module>
      3                 batch_size=256,              # (Optional) the batch size for pre-training (the default is 32).
      4                 alpha=0.10,                  # (Optional) the value of alpha in [0,1] to encourage covariate adjustment. Not used if there is no covariates.
----> 5                 num_epoch = 300,             # (Optional) the maximum number of epoches (the default is 300).
      6                 )

~/anaconda3/lib/python3.7/site-packages/VITAE/VITAE.py in pre_train(self, stratify, test_size, random_state, learning_rate, batch_size, L, alpha, num_epoch, num_step_per_epoch, early_stopping_patience, early_stopping_tolerance, path_to_weights)
    274                                                 batch_size,
    275                                                 self.X[id_train].astype(tf.keras.backend.floatx()),
--> 276                                                 self.scale_factor[id_train].astype(tf.keras.backend.floatx()))
    277         self.test_dataset = train.warp_dataset(self.X_normalized[id_test],
    278                                                 None if self.c_score is None else self.c_score[id_test].astype(tf.keras.backend.floatx()),

TypeError: 'NoneType' object is not subscriptable

Thank you in advance.
Best regards.

Thanks for reporting the issue, which has been fixed in v1.1.7.


[stable/openebs] Upgrade to openebs release 0.6

What this PR does / why we need it:
This PR includes the following changes:

Update the image tags to 0.6
Add a new deployment introduced in 0.6 for supporting snapshots
Identation fix with templates
Updated README and Chart.yaml

Signed-off-by: kmova<EMAIL_ADDRESS>

/assign @viglesiasce

@unguiculus @mgoodness @seanknox - can you please help reviewing this PR.

/ok-to-test

/lgtm


Fatal Error with Version 1.1.3

I am using SOAPClient to send SOAP request with XML. But I am getting below error.
[Fatal Error] :1:10: DOCTYPE is disallowed when the feature "http://apache.org/xml/features/disallow-doctype-decl" set to true.
Expecting answer/solution soon.

I'm having the same error. Did you find any solution?
Thanks in advance


Remove publishConfig in package.json in published package

Need to fix clean-publish

https://github.com/shashkovdanil/clean-publish/pull/155


Fix initial logical scrollable state

What does the pull request do?
As described in #13064, the initial state for a ILogicalScrollable control isn't correctly synced to the owner ScrollViewer.
This seems to have been introduced in #10803 because at the time that the subscriptions in ScrollContentPresenter are set up, the _owner is still null and so when the changes arrive at OnPropertyChanged they're not written to the owner.
@TomEdwardsEnscape you wrote #13064, is moving the _owner assignment like this going to break anything else do you think? Was it placed after the subscription setup for a reason?
Fixed issues
Fixes #13064

You can test this PR using the following package version. 11.0.999-cibuild0040209-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]


Kustomize Documentation - Typo in the newName parameter in image section

What happened:
newImage parameter is incorrectly statted as nameName parameter in the documentation for kustomize https://kubectl.docs.kubernetes.io/pages/reference/kustomize.html#
What you expected to happen:
Parameter name should be newName
How to reproduce it (as minimally and precisely as possible):

Visit - https://kubectl.docs.kubernetes.io/pages/reference/kustomize.html#
Anything else we need to know?:
Its only a change int he documentation.
Environment:

Kubernetes client and server versions (use kubectl version): N/A
Cloud provider or hardware configuration: N/A
OS (e.g: cat /etc/os-release): N/A

Will be closed by https://github.com/kubernetes/kubernetes/pull/89086
/priority backlog

/close


Enhancement 14 s3 storage type

fixes #14
pr for storage type S3

Coverage decreased (-0.6%) to 88.61% when pulling a1a03c103ea44014e53889025a90b121db16db32 on devhyunjae:enhancement-14-s3-storage-type into 67a3b21f4bc5511f746cb19b35a2ec2d1964ca15 on lisong:master.

Seems like the test fails for other issues..
If u want unit or e2e test, just tell me

it's ok. ignore circleci.

waiting for appveyor pass.


[vcpkg/ninja] VS CMake tools for Windows/Linux overwrite ninja from within VCPKG

Describe the bug
If any CMake tools from VS are installed vcpkg_find_acquire_program will use ninja installed with the VS CMake tools instead of VCPKGs internal version which is currently a newer version.
Environment

OS: Windows
Compiler: VS 16.5.4

To Reproduce
Install CMake tools for Windows or for Linux in the VS Installer
Install any port using ninja and check the configure log.
Expected behavior
Use the correct ninja version (newer or at least the same version as within VCPKG)
Failure logs
Example from one of the configure logs.
"-DCMAKE_MAKE_PROGRAM=C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe"

if(GENERATOR STREQUAL "Ninja" VERSION_LESS "1.10")
NO FATAL ERROR ?