Why define Thank words on every message

https://github.com/Dhravya/Spacebot/blob/5a3c545fe8faa0fcfa8b301c93feff95494baccf/src/cogs/backgroundtasks.py#L66
IMO, the overall background tasks are a bit too complex. You might wanna split then up into cogs or smth

Can you please elaborate as to what to split, etc? Thanks for the feedback though, i completely agree

You could move thank_words = ["thanks","thank you", "thank"] outside of the class (somewhere around line 8-9)
Or you could define it as a class attribute. Define it near line 24 outside of the __init__ (single indented) and access it using self.thank_words

You can also use regular expressions to check if the thank words are in the message.
Splitting the content into words and looping through all of them will put load on the machine if there are long messages.
And thank you will never work as it is 2 words.

yes maybe remove 'thank you ' coz "you' will be taken as a member and when it doesnt find it ,it'll throw error

Yeah but you can use regex instead as it will work on multi-worded string

Yes thats right. @MasterBaiter123 I noticed that you forked this repository. You're working on #9  Right? maybe you could also try fixing this issue along with it?

Sure. i'll try

Thank you !!


feat: more rust 1.62

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


CodeMirror-gutter CodeMirror-foldgutter

because of "CodeMirror-gutter CodeMirror-foldgutter" my windows looks like this
http://prnt.sc/cwpify

Thanks for reporting. Should be fixed now.


[ENHANCEMENT]

Describe the solution you'd like
Add a terraform data resource for the dataplane url
Describe alternatives you've considered
Putting it in by hand

HI @phumberdroz. Our terraform provider does not have any support for data plane deployments as of now. Can you describe your use case in more detail?


Build antrea-cni binary and release binaries without cgo

antrea-cni can be installed (by the initContainer) on systems with
incompatible system libraries. To increase portability, we build
antrea-cni without cgo by setting CG_ENABLED to 0.
Same goes for release binaries (e.g. antctl). Note that when
cross-compiling, cgo is disabled by default, so there is no change there
(e.g. antctl binary for macOS). It's only for native Linux amd64 builds
that there is an actual change.
Fixes #2095

Surprisingly, the antrea-cni binary is actually smaller with cgo disabled... I was expecting the opposite...
$ make antrea-cni
GOOS=linux CGO_ENABLED=0 go build -o /home/abas/vmware/antrea/bin  -ldflags ' -X antrea.io/antrea/pkg/version.Version=v1.1.0-dev -X antrea.io/antrea/pkg/version.GitSHA=3134f7ee -X antrea.io/antrea/pkg/version.GitTreeState=dirty -X antrea.io/antrea/pkg/version.ReleaseStatus=unreleased' antrea.io/antrea/cmd/antrea-cni
abas@ubuntu:~/vmware/antrea$ ls -l bin/antrea-cni
-rwxr-xr-x 1 abas abas 11673235 May 18 16:44 bin/antrea-cni

$ make antrea-cni
GOOS=linux CGO_ENABLED=1 go build -o /home/abas/vmware/antrea/bin  -ldflags ' -X antrea.io/antrea/pkg/version.Version=v1.1.0-dev -X antrea.io/antrea/pkg/version.GitSHA=3134f7ee -X antrea.io/antrea/pkg/version.GitTreeState=dirty -X antrea.io/antrea/pkg/version.ReleaseStatus=unreleased' antrea.io/antrea/cmd/antrea-cni
abas@ubuntu:~/vmware/antrea$ ls -l bin/antrea-cni
-rwxr-xr-x 1 abas abas 11741710 May 18 16:45 bin/antrea-cni

Codecov Report

Merging #2189 (f2851b2) into main (3134f7e) will decrease coverage by 0.30%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2189      +/-   ##
==========================================
- Coverage   61.26%   60.95%   -0.31%
==========================================
  Files         270      270
  Lines       20486    20486
==========================================
- Hits        12550    12487      -63
- Misses       6636     6701      +65
+ Partials     1300     1298       -2

Flag
Coverage Δ

kind-e2e-tests
51.66% <ø> (-0.42%)
:arrow_down:

unit-tests
41.33% <ø> (-0.05%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/agent/controller/traceflow/packetin.go
56.94% <0.00%> (-9.73%)
:arrow_down:

pkg/apiserver/certificate/certificate.go
69.86% <0.00%> (-6.85%)
:arrow_down:

pkg/controller/networkpolicy/tier.go
47.50% <0.00%> (-5.00%)
:arrow_down:

pkg/agent/stats/collector.go
94.61% <0.00%> (-3.85%)
:arrow_down:

...agent/controller/traceflow/traceflow_controller.go
69.61% <0.00%> (-3.54%)
:arrow_down:

...gent/controller/networkpolicy/status_controller.go
72.60% <0.00%> (-2.74%)
:arrow_down:

pkg/controller/traceflow/controller.go
69.27% <0.00%> (-2.41%)
:arrow_down:

pkg/controller/networkpolicy/status_controller.go
85.16% <0.00%> (-1.94%)
:arrow_down:

pkg/agent/openflow/pipeline.go
68.82% <0.00%> (-1.71%)
:arrow_down:

pkg/agent/route/route_linux.go
43.08% <0.00%> (-0.55%)
:arrow_down:

... and 4 more

@tnqn oops I forgot to sign my commit, shame on me :P I need another approval...

/test-all


Button Styling Issues

It looks like in the v30 Snapshot we have some unintentional changes to default button styling:

No longer seeing any hover in light theme on standard buttons.
The background color on dark theme is pretty different than what we have.

I am looking at the grid of buttons in Toolbox ->  Other -> Popups, where the problem is apparent.  Have not looked more extensively at other buttons where it might be happening

Apologies for letting this one through :(
Thanks for jumping on this John, changes look great!


Templating output-event-adapters.xml.

Proposed Changes

Template wso2events and email in  output-event-adapters.xml.

Validate the mail.publisher.address using a regex.

Check whether username and password are empty when email notification is enabled and give an error message.

Templated in the component level hence closing the PR.


Add support for raw tracepoints

Taking advantage of https://github.com/iovisor/bcc/pull/1718.
Example usage: https://github.com/cloudflare/ebpf_exporter/pull/38.

@iaguis, @schu can you have another look? Thanks!


Add support for hidden: true on variants

Migrated from GitLab: https://gitlab.com/meltano/hub/-/issues/148
Originally created by @tayloramurphy on 2021-10-20 18:11:15

View 2 previous comments from the original issue on GitLab


Feature request: use of django channels instead of celery?

Is it possible to use colossus with channels, and this way use Django Foundation supported core features, only?

Django Channels and Celery tackle different problems. Celery is currently being used to handle heavy duty tasks outside the request/response cycle. For example sending bulk emails, which can take a few minutes to complete. It's also used to run tasks in the background to clean up the list when there is available resources on the server. Or to update Geolocation data on the subscribers.
Django Channels could be added to handle the notification system for example, but it's not on my roadmap at the moment. It would be just nice to have, but very little benefit.


⚠ Jenkins Plugins Artifactory has degraded performance

In a09a033, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1254 ms

Resolved: Jenkins Plugins Artifactory performance has improved in a5764a9 after 24 minutes.


Add trivy scan workflow for kctrl

Describe the problem/challenge you have
Currently we don't have a workflow to scan for security vulnerabilities in kctrl.
Describe the solution you'd like
We should update the existing trivy scan workflow to support kctrl or add a new one.
Anything else you would like to add:

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

We can use the workflow in carvel-release-scripts once we start supporting kctrl in it (related issue - https://github.com/vmware-tanzu/carvel-release-scripts/issues/23).


Seasons in chests / items from season sprits

This is pretty easy since the way the game gives you seasons by giving you the rod with a different sub-ID. Leaving the normal rod in the game if the rods are randomized like this doesn't make much sense, but that's fine since fool's ore is there to fill the missing slot.

This was implemented in https://github.com/jangler/oos-randomizer/commit/94adadb3a8b6628f315e58d830a959d26418e881.


VersionControl.ps1 "corrupts" version number

File https://github.com/dsccommunity/DscWorkshop/blob/dev/DSC/Build/Tasks/VersionControl.ps1 causes a "corruption" in the version number. It replaces the PATCH version with the release version. This leads to a version number on the server, that is different than the version number in the configuration repository e.g., Repository: 1.1.3, Server 1.1.1.
According to the Semantic Versioning standard - which I assume is used here - the build number should be added as follows: MAJOR.MINOR.PATCH+BUILD.

While writing a patch for it, I noticed that System.Version is used in VersionControl.ps1, which is structured as follows: MAJOR.MINOR.BUILD.REVISION
I think it would be a good idea to explain this in the documentation, so that when Azure DevOps is used, or any of the tools listed below, no changes are applied to the version number that the author didn't expect.
https://github.com/dsccommunity/DscWorkshop/blob/0f25496945d2b83282ad27375cb8c061ebaad3a8/DSC/Build/Tasks/VersionControl.ps1#L3


Flutter Web sourcemaps support

Related to:
https://github.com/flutter/flutter/issues/16883
https://github.com/flutter/flutter/issues/37875

Dart has some extensions for source maps: https://github.com/dart-lang/sdk/blob/master/pkg/compiler/doc/sourcemap_extensions.md
@marandaneto Does this need to be considered in Sentry service?

@ueman probably, if we don't support such optimizations, we'd need to figure it out, I'll double-check that, thanks for the heads up.

For further reference, Dart's own implementation for getting readable stacktraces out Dart2JS compiled apps:
https://pub.dev/packages/source_map_stack_trace

although we can still improve this, it works well at some degree:

@marandaneto are there any resources to troubleshoot why we can't see the lines of code?
This is what I am seeing in our app:

This is just a link to our index.html file

@venkatd I'd open a new issue though, with a bit more context eg how (params) you've uploaded the source maps, if versioning and dist matches (the event itself and the source maps).
btw you can also try this: https://github.com/getsentry/sentry-dart-plugin

@marandaneto thanks I just created an issue at #562
Can move our discussion over there.

@marandaneto are there any resources to troubleshoot why we can't see the lines of code?
This is what I am seeing in our app:
This is just a link to our index.html file
Here's a subset of the logs on our GitHub actions:
logs

Hey Manoel!
I'm not sure how you got to this point, can you please share the exact steps and code?
I followed these steps and got a pretty useless stacktrace -

Created a new project at sentry and selected flutter
Downloaded https://github.com/getsentry/sentry-dart/tree/main/flutter/example
Added my sentry DSN to https://github.com/getsentry/sentry-dart/blob/fce129d973473dc8b7707b0a7da730925f23c3b8/flutter/example/lib/main.dart#L17
Added org and project to https://github.com/getsentry/sentry-dart/blob/fce129d973473dc8b7707b0a7da730925f23c3b8/flutter/example/pubspec.yaml#L43
Ran flutter build web --release
Ran flutter packages pub run sentry_dart_plugin
Opened the app in Chrome (using python3 http.server in build/web)
Pressed the "Dart: try catch" button

@devxpy you've to pass --source-maps to your build command, https://github.com/getsentry/sentry-dart/blob/main/.github/workflows/web-example-ghpages.yml#L20

I've added to the README as well https://github.com/getsentry/sentry-dart-plugin/commit/14a37814b160e9ea819253306362f579775f7947#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5

I've added to the README as well getsentry/sentry-dart-plugin@14a3781#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5

Thank you so much for the prompt reply :)
I got the source code to show up on sentry now!
Any ideas on how I can get it to show the correct exception type too? (It shows minified:kP)

Output
$ flutter build web --release --source-maps
Warning: You are using these overridden dependencies:
! sentry 6.3.0 from path ../../dart
! sentry_dio 6.3.0 from path ../../dio
! sentry_flutter 6.3.0 from path ..
Running "flutter pub get" in example...                          2,600ms

💪 Building with sound null safety 💪

Compiling lib/main.dart for the Web...                             30.9s

$ flutter packages pub run sentry_dart_plugin
☑ reading config values
☑ validating config values
[❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚] 18% uploading debug symbols..
includeNativeSources is disabled, not uploading sources.

stdout: > Found 0 debug information files
> No debug information files found

☑ uploading debug symbols
[❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚❚] 27% uploading source maps..
stdout: Created release<EMAIL_ADDRESS>

stdout: > Found 5 release files
> Analyzing 5 sources
> Rewriting sources
> Adding source map references
> Bundled 5 files for upload
> Uploaded release files to Sentry
> File processing complete
> Organization: dara-network
> Project: dara-app
> Release<EMAIL_ADDRESS>> Dist: None

Source Map Upload Report
  Scripts
    ~/flutter_service_worker.js
  Minified Scripts
    ~/canvaskit/canvaskit.js (sourcemap at ../main.dart.js.map)
    ~/canvaskit/profiling/canvaskit.js (sourcemap at ../../main.dart.js.map)
    ~/main.dart.js (sourcemap at main.dart.js.map)
  Source Maps
    ~/main.dart.js.map

stdout: > Found 3 release files
> Analyzing 3 sources
> Rewriting sources
> Adding source map references
> Bundled 3 files for upload
> Uploaded release files to Sentry
> File processing complete
> Organization: dara-network
> Project: dara-app
> Release<EMAIL_ADDRESS>> Dist: None

Source Map Upload Report
  Scripts
    ~/lib/generated_plugin_registrant.dart
    ~/lib/main.dart
    ~/lib/user_feedback_dialog.dart

stdout: Finalized release<EMAIL_ADDRESS>
☑ uploading source maps

That's a limitation right now, See https://docs.sentry.io/platforms/flutter/troubleshooting/#known-limitations
Look at the Obfuscate Caveat.

Ah, dang! Thanks for linking the above. I guess your example was a debug build without obfuscation?

any updates

So Web source maps should work already if you upload source maps, we're interested in collecting feedback in case there's any edge case that Sentry isn't able to symbolicate, e.g. off by 1 line numbers, please raise a new issue, thanks.

@ahmed-ashraf open a new issue, I'm no longer part of the Sentry staff.

I'm using flutter web source maps with no issues in Sentry today - I know this is an old thread but the issues seem to be resolved


utilize new API uspport for client/group acl granularity

If the server response to ACL GET includes 'clients' and 'users'
fields, use those instead of 'actors' for submitting updates. When
showing acls, exclude 'actors' when 'clients' and 'users' are present,
since the API guarantess it will always be empty in that case.
This is backward compatible with no effect when the chef server does not support the expanded ACL API.

👍 Looks reasonable


Not working with TTNv3

This firmware working fine with TTN v2 but not working with TTNv3? Is there any update required for that?

Did you change the define in gwayConfig.h?
// TTN Version 2:
//#         define _TTNSERVER "router.eu.thethings.network"
// TTN Version 3:
define _TTNSERVER "eu1.cloud.thethings.network"
define _TTNPORT 1700                                                                                // Standard port for TTN
#endif
From: gowthamrajg88 @.>
Sent: Saturday, July 3, 2021 9:54 AM
To: things4u/ESP-1ch-Gateway-v5.0--OLD @.>
Cc: Subscribed @.***>
Subject: [things4u/ESP-1ch-Gateway-v5.0--OLD] Not working with TTNv3 (#101)
This firmware working fine with TTN v2 but not working with TTNv3? Is there any update required for that?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fthings4u%2FESP-1ch-Gateway-v5.0--OLD%2Fissues%2F101&data=04|01||f1f434d495784d7e3fe008d93df7acc7|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637608956220432184|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=d8V6g1YDcviz885GQzaE7R9RRYEHt%2FIGmDb2rP5HTMo%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABMSRHASLC6IG6J6D6ITDVTTV26YJANCNFSM47X4UQXA&data=04|01||f1f434d495784d7e3fe008d93df7acc7|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637608956220432184|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=0srf9HYZIkDAFU9mYP4zfQTV5xjzicIXVZn3D%2FngFKQ%3D&reserved=0.


Revert "CI: fix latest package URL getter tool name"

This reverts commit 9c42f14cc37c4cb3665a51634fdc0f0c5dc6d45d.
Signed-off-by: Karol Gugala<EMAIL_ADDRESS>

I fixed this with https://github.com/SymbiFlow/symbiflow-tools-data-manager/pull/10 instead.


Optional security claims are not handled by convertor

Conversion fails when security is {} or not exists in  securitySchemes

@brightAMasiuk, please update the bug description to include the necessary information such as steps to reproduce, expected results, and actual results.
@BMNTeam @pmstss fyi

Steps:

Start scan
Import the attached schema
Check validation error

Expected Result:
Schema parsed successfully
Actual Result:
Converter error:
Cannot read properties of undefined (reading 'type')

@brightAMasiuk the description is weakly formatted, senseless and useless. Which schema should I import and where? What does "run scan" mean in terms of the har-sdk and especially the oas package?
@BMNTeam @pmstss fyi


Sixcolium nugget crashing game

I am on version 1.07 of the pack
When attempting to create the sixcolium nugget, the game crashes completely
Upon restart, the game gives me back the ingot, and if I try to create the nugget again, it just crashes
I have attached the latest crash report
crash-2021-05-08_22.25.10-client.txt

Update the modpack pls.

Correction, it's version 1.07 of the pack, I made a typo. To my knowledge, this is the latest version that is available on curse forge, unless I'm missing something?

Just delete the Fast Workbench mod, I remembered that I've deleted before, but it still here, so weird.


Artifacts after projection to SWAGAN latent space

Hi!
When trying to perform a projection to a trained SWAGAN's latent space, the generations I receive tend to include some artifacts, thus rendering the overall image generation unrealistic. Do you have any clues about why is this happening?
Thank you in advance!

Generally projection process is not very stable and consistent. Does projection with stylegan2 works well with your samples?

Where did you get the pretrained-SWAGAN weights? Thanx in advance.


Update triggeraction.go

Use background context
Overview
This PR fixes #
Signed Commits

[ ] Yes, I signed my commits.

/improve

/describe


[Enhancement] Resolve backend FQDN to ip when generating plan fragment

To release the pressure on DNS server when multi backends querying it.
Besides, move the NetworkAddress from ExportJob.java to NetworkUtils.java.
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[x] 3.1
[ ] 3.0
[ ] 2.5

[FE Incremental Coverage Report]
:disappointed: fail : 11 / 39 (28.21%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/common/util/FQDNResolveException.java
0
11
00.00%
[21, 22, 25, 26, 29, 30, 33, 38, 42, 43, 45]

:large_blue_circle:
com/starrocks/common/util/NetworkUtils.java
8
24
33.33%
[45, 52, 54, 62, 66, 67, 70, 74, 75, 86, 91, 96, 108, 109, 110, 111]

:large_blue_circle:
com/starrocks/system/ComputeNode.java
2
3
66.67%
[215]

:large_blue_circle:
com/starrocks/common/Config.java
1
1
100.00%
[]

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

@Mergifyio backport branch-3.2

@Mergifyio backport branch-3.1


Display summary of paid reviews per game

Extend Steam review data to include whether a review is paid or not
Header combines meta + review data to compose data
Code is messy though :/

Fixed issues mentioned, and the build passed (locally, the tests pass - build shows the old test assertion for some reason). Looks like this change is good to go.


Some minor accessibility improvements in using git.

Issue Type: Feature Request
cc: @isidorn
When performing an operation in git, the orca, the screen reader I use to access the VSCode, does not inform me about the result of the operation. This happens for example when I commit, pull, push and so on. I never know if the operation ended and what the result was.
I also can't easily identify if VSCode is asking for a login to access the repository.
Maybe the VSCode is providing this information and the orca just isn't reading it.
VS Code version: Code - Insiders 1.58.0-insider (e1161be26874f0d39a5b9cdeb2c5fbfd4f83bb80, 2021-06-14T05:13:48.044Z)
OS version: Linux x64 5.12.10-arch1-1
Restricted Mode: No

@jvesouza thanks for the feedback!
In general vscode does not announce when some command successfully finished. Doing this as a special case for SCM commands would be a bit weird.
I think we should do the opposite approach, be 100% sure that if the command fails that the user is aware of it. The only missing piece would be how to know if the operation is still in progress - and for that we are actually using progress api which should be read out by the screen reader if the progress is long enough. I am not sure if this work well in practice, but in theory it should work (I tried it with Orca).
@jvesouza What do you think about this approach?
As for the login not being read, that sounds like an issue we should fix, do you have steps for this which we could follow and reproduce?

Doing this as a special case for SCM commands would be a bit weird.

There is a long-existed setting git.showPushSuccessNotification, which when set to true, will show a notification after git.push command succeeded. But it doesn't show notification for other SCM operations, even the git.sync command.

@jvesouza What do you think about this approach?

To be honest, I don't really like this approach, but I fully understand that it's a matter of point of view. I always like to be informed of the results of operations.

As for the login not being read, that sounds like an issue we should fix, do you have steps for this which we could follow and reproduce?

As for the login issue, it doesn't seem to be directly related to git but rather a general issue. I found for example that the same problem happens when I try to log in to a remote server using the remote ssh extension. If the connection requires a login / password, the orca doesn't read the prompts unless I press shift+tab and tab.

@jvesouza can you please open a new issue for the login problem and ping me @isidorn on it
As for the git no success notification, can you please try the setting
	"git.showPushSuccessNotification": true,

We can maybe change the default of that setting when a screen reader is detected.
Or introduce a more general setting that would cover all the git operations.

No, I do not think there is anything that should be done that is specific for scm thus let's close this.
In general I think we should introduce some smartness when a screen reader is detected that we have a way to notify users when a command completes. Just that we keep that in mind of the future as a general workbench thing.


Line is blinking if perspective set to false

While above gif seems line randomly blinks because of low frame, actually it blinks periodically and short term.
Below is the spawn of line with set off perspective since default value of perspective is false.
PolylineBundle {
        transform: Transform::from_xyz(0., 3., -10.),
        polyline: polylines.add(Polyline {
            vertices: vec![Vec3::new(2., 2., 0.), Vec3::new(-2., -2., 0.)],
            ..default()
        }),
        material: polyline_materials.add(PolylineMaterial {
            color: Color::WHITE,
            ..default()
        }),
        ..default()
}

Do you see this on the minimal example?

Ok but i'll do it later because I'm on part time job.
Imvironment is Windows 10, gtx 2060

It quite works well. No blinking at all.

Can you please try to make a minimal reproduction? If it doesn't happen in the minimal example, but does happen in the code above, there must be something in that code that is causing this to happen.

I know it sounds very weird, I couldn't reproduce blinking again. I forgot the code.
But after few experiments, I found a noticeable other bug seems very related with blinking.
That is,
If there are many poly lines, their perspective value application highly depends on their spawn order.
All lines that perspective is false spawned after even one perspective true line, they become as if perspective is true.
If all lines that perspective is false are spawned after even one perspective true line, they become as if perspective is true.

If all lines that perspective is false are spawned after even one perspective true line, they become as if perspective is true.


update maskdino init method

The following lines need to be removed from __init__.py  in the maskdino/ folder:
from .data.dataset_mappers.mask_former_instance_dataset_mapper import (     MaskFormerInstanceDatasetMapper, )
 from .data.dataset_mappers.mask_former_panoptic_dataset_mapper import (     MaskFormerPanopticDatasetMapper, )
These modules were removed in the latest commits.

Thanks, I have updated these lines.


kubebuilder project changes: domain, group, and repo location

This is an API breaking change, necessary to make kubebuilder happy.

Domain was changed to arlon.io
Group name is now core
Repo is now github.com/platform9/arlon
Controller image version bumped to 0.2.0, and repo location updated (platform9-incubator --> platform9)

After this change, existing installations will need to install a new CRD and redeploy the arlon controller. Also, the image at ghcr.io/platform9/arlon/arlon-controller is still private and cannot be pulled by Kubernetes. I asked devops team to make it public.

Looks good. @gjayaprakash This should make imports easier for you.


Hardware shopping list

Hi, nice project !
Would it be possible for you to provide in this repo a complete hardware shopping list with links ??
Cheers.

Hi @artivis
Thanks for being interested in this project!
The package now is in beta version.
After final testing, we will release all details including the way to buy hardware.
(by the end of this month~)

Thanks for the quick answer.
That's great ! Looking forward to it.
Cheers.

Hi @artivis
We have released reference of hardware shopping list.
Please find detail in readme file!.


chore: Fix backLen comparison in MFRC522Extended.cpp

The comparison of backLen in MFRC522Extended.cpp was incorrect. When compiled in ESP/IDF, the compiler gave the error : "ordered comparison of pointer with integer zero"

Q
A

Bug fix?
yes

New feature?
no

Doc update?
no

BC breaks?
no

Deprecations?
no

Fixed tickets
none

i just tested this fix with a esp32 nodemcu. it works: cardreader is functional.

Better use nullptr instead of NULL. I had this issue and fixed it with if (backData && (backLen != nullptr))


Echo app cannot listen on 9090

$ go run ./pkg/test/echo/cmd/server --port 9090
Listening HTTP/1.1 on 9090
Listening GRPC on 7070
2020-07-15T19:20:27.797943Z     error   listen tcp :9090: bind: address already in use
exit status 255

From its help:
$ go run  ./pkg/test/echo/cmd/server -h
Echo application for testing Istio E2E

Usage:
  server [flags]

Flags:
<other flags omitted>
      --grpc ints                     GRPC ports (default [7070])
      --port ints                     HTTP/1.1 ports (default [8080])
      --tcp ints                      TCP ports (default [9090])
      --tls ints                      Ports that are using TLS. These must be defined as http/grpc/tcp.
      --uds string                    HTTP server on unix domain socket
      --version string                Version string

I think if user pass --port 9090 to server, he/she also needs to specify --tcp <port other than 9090>.

From its help:
$ go run  ./pkg/test/echo/cmd/server -h
Echo application for testing Istio E2E

Usage:
  server [flags]

Flags:
<other flags omitted>
      --grpc ints                     GRPC ports (default [7070])
      --port ints                     HTTP/1.1 ports (default [8080])
      --tcp ints                      TCP ports (default [9090])
      --tls ints                      Ports that are using TLS. These must be defined as http/grpc/tcp.
      --uds string                    HTTP server on unix domain socket
      --version string                Version string

I think if user pass --port 9090 to server, he/she also needs to specify --tcp <port other than 9090>.

# apple @ liuming-dev in ~/Workspace4go/src/github.com/istio/istio on git:master o [14:52:46] C:1
$ go run ./pkg/test/echo/cmd/server --port 9090 --tcp 9091
Listening HTTP/1.1 on 9090
Listening GRPC on 7070
Listening TCP on 9091
2020-07-23T06:52:49.391627Z     info    ready for TCP endpoint <IP_ADDRESS>:9091
2020-07-23T06:52:49.392189Z     info    HTTP Request:
  Method: GET
  URL: /,
  Host: <IP_ADDRESS>:9090
  Headers: map[Accept-Encoding:[gzip] User-Agent:[Go-http-client/1.1]]
2020-07-23T06:52:49.392204Z     info    HTTP service not ready, returning 503
2020-07-23T06:52:49.392329Z     info    ready for HTTP endpoint http://<IP_ADDRESS>:9090
2020-07-23T06:52:49.392951Z     info    GRPC Request:
  Host: [::]:7070
  Message: hello
  Headers: map[:authority:[[::]:7070] content-type:[application/grpc] user-agent:[grpc-go/1.29.1] x-request-id:[0]]

2020-07-23T06:52:49.393211Z     info    ready for GRPC endpoint [::]:7070
2020-07-23T06:52:49.393225Z     info    Echo server is now ready

@howardjohn it is not a issue, just mistake about --tcp and --port, please close it.


filter action runs

https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#onpushpull_requestpaths

So if I get this right, we can simply skip for example our pytest runners if we filter the weldx directory and nothing changed in it?

yes, I just stumbled upon that and wanted to keep a reminder
a few ideas:

don't run pytest for changes only to CHANGELOG.md, doc/** etc.
only run pytest/build for changes to the python package and tutorials
don't run linters if no code changes

we should always run everything on master and tags though I guess


[AKS] az aks install-cli: Add support for kubelogin darwin/arm64 releases

Description

Currently the az aks install-cli does not work on M1 Macs (darwin/arm64) due to the fact that the wrong release of kubelogin is being used. I have recently merged this PR which adds arm64 builds for kubelogin. The change here  adds a check to make sure the right arch is used on the Darwin platform.
Testing Guide

az aks install-cli on an M1 Mac.
History Notes

Fixes #18993
This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

AKS

@Volatus Could you please help to fix the style issue?

@Volatus Could you please help to fix the style issue?

Done.


Provide Arrow Schema Hint to Parquet Reader - Alternative 2

Which issue does this PR close?
Closes  #5657. Alternative to #5671.
Rationale for this change
The parquet reader automatically uses an embedded arrow schema to hint type inference for decode. In particular if the hinted type is compatible with the underlying parquet type, it performs a cast. However, in situations where the writer was not an arrow writer the schema is not available and the arrow types are inferred from the parquet schema. This is not always desirable.
For example, you may want to cast an INT64 column to a Timestamp column or a Timestamp column to a different timezone. This PR allows a schema to be provided for use with type hinting.
What changes are included in this PR?
This is a draft PR for discussion. There are some API alternatives that can be considered. These are described in the section on user facing changes.
Currently, this PR adds supplied_schema field to ArrowReaderOptions and modifies the try_new method of ArrowReaderMetadata to use the supplied_schema instead of the metadata when one is provided. This option defaults to None so it should be backwards compatible?
The option is provided with a new with_schema method on ArrowReaderOptions.
Adds tests showing a simple success case. Once the approach is finalized, these can be expanded to cover off all the valid cast scenarios using an approach similar to run_single_column_read_tests.
It also adds test showing two failure scenarios:

The supplied_schema must have exactly the same number of columns as the parquet schema or an error is thrown.
The final inferred schema must be contained by the supplied schema.

Are there any user-facing changes?
A with_schema method is added to the ArrowReaderOptions. It is intended to be used as follows:
let file = File::open("file.parquet");
let schema = Arc::new(Schema::new(vec![Fields::new("col", DataType::Int32, false)]));
let options = ArrowReaderOptions::new().with_schema(schema);
// This may fail as described above.
let builder = ParquetRecordBatchReaderBuilder::try_new_with_options(file, options).unwrap();
let reader = builder.build().unwrap();

It need to be provided as an option before the builder is constructed so it can be utilized to provide type hints when the metadata is being read.
There are different approaches that could be taken with the supplied_schema:

Currently, it needs to provide an complete schema for the entire file. It seems like it might be possible for it be treated like a partial schema that would override the the inferred schema or any schema in the metadata. This would be more ergonomic if only wanting to update a subset of fields is needed.
If the supplied type hints can't be utilized the API doesn't need to return an error and just use the type from the Parquet schema. This is what happens with the type hints derived from the metadata, so it might be better to make it consistent.

Thanks @efredine  -- I started the CI on this PR

I expanded the tests to include converting from an String to a Dictionary and also included a nested field. I also applied what I learnt about constructing record batches from an iterator ;-).


Unhandled AttributeError during refresh_cameras() on construction of UnifiVideoAPI

This just started happening this morning. I haven't changed any configuration of the Unifi system.
AttributeError in camera.py's _load_data method (line 199 is the offender). Adding AttributeError to the except clause appears to fix the issue for me.

@twoodrow mind checking which version of unifi_video you're running? This should be fixed in the latest release; 0.1.6.

Closing due to inactivity.


issue-2397 catchThrowableOfType enriched alternatives

Check List:

Fixes 2397
Unit tests : YES
Javadoc with a code example (on API only) : YES
PR meets the contributing guidelines

Thanks for the PR @Spacca. I see you added catch versions for additional exception types, which could be totally fine from my perspective. We may want to align the corresponding assertThat...Exception methods, probably in a separate PR.
What do you think @joel-costigliola?

We do want to align with assertThat...Exception, probably  in a different issue, in the new issue we could also add a test to verify that we have matching assertThat...Exception with catch...Exception.

Thanks for your first contribution, @Spacca!


Initialized is fired too late?

LSP logs initialized after the first textDocument/didOpen goes out.
The spec says it should be sent:

after the client received the result of the initialize request but before the client is sending any other request or notification to the server

It is possible the logging is out-of-order, need to investigate.
LSP: starting ['pyls']
LSP: request 1: initialize
LSP: client registered for window 2 {'pyls': <LSP.main.Client object at 0x106a7a2d0>}
LSP: got json:  {"result":{"capabilities":{"definitionProvider":true,"signatureHelpProvider":{"triggerCharacters":["(",","]},"hoverProvider":true,"documentSymbolProvider":true,"executeCommandProvider":{"commands":[]}
LSP: notify: textDocument/didOpen
LSP: notify: initialized
LSP: got json:  {"method"

Yes, I also noticed didChange notifications to be somewhat out of date with me typing on my keyboard, but I assume this is by design?

didChange is fired either 500ms after you've stopped typing, or immediately when a completion is requested. I hope the logging matches that behaviour :)

On another note, are you going to put more capabilities in the Initialize request?

We must be missing a few since I last looked at it... it looks like applyEdit on workspace, synchronization on textDocument are incomplete.


update transition message

We can be a little more optimistic now that Gratipay 2.0 is out the door.

Ping @rohitpaulk et al.

Self-merging, tiny change and time-sensitive.


Changelog?

Do you have the changes to this package as they relate to version bumps documented anywhere?

Will add a changelog in the next release :)


prisma-client not returning non-required json fields.

type A {
  y: Json
}

y won't be returned by the prisma client.
type B {
  x: Json!
}

x will.

Hi @tiagocpontesp, I cannot reproduce this error. Can you share a reproduction repository?
For reference, for this datamodel:
type User {
  id: ID! @unique
  name: String!
  json: Json
  anotherJson: Json!
}

after this mutation
mutation b {
  createUser(data: {
    name: "Nilan"
    json: "{ \"a\": 1 }"
    anotherJson: "{ \"a\": 1 }"
  }) {
    id
  }
}

this code
const { prisma } = require('./generated/prisma')

const main = async () => {
  const u = await prisma.users()
  console.log(u)
}

main()

will reproduce this expected result:
[ { id: 'cjn1v7vy90acs0b72x3ujkflz',
    name: 'Nilan',
    json: { a: 1 },
    anotherJson: { a: 1 } } ]

I'm on version 1.18.0-beta.22. Please provide the information as per the bug report template.

This seems to be fixed. Please comment again to re-open this issue if it still persists for you using the recent version.

Hi. I apologise @marktani, this got fixed very quickly, and I didn't notice the update.


Spec/implementation mismatch: Connect token address type byte?

While working on my implementation based on the spec at https://github.com/networkprotocol/netcode.io/blob/master/STANDARD.md, I hit a bit of a snag: while the spec says that in the private connect token, for each server address, a byte identifies the address type and is either 0 or 1, I was getting a 1 byte despite the buffer clearly containing an IPv4 address (first four bytes were 127, 0, 0, 1, and the next two bytes together formed the port I had bound my server to). I'm not generating my own token yet, I'm using the server example that comes with the Netcode.IO browser project over at the Redpoint Games github, and that one just calls into native code, so this isn't an issue with an invalid connect token as far as I can tell.
So after digging through the C code I noticed that it defined the following:
#define NETCODE_ADDRESS_NONE 0
#define NETCODE_ADDRESS_IPV4 1
#define NETCODE_ADDRESS_IPV6 2

And, later, it just writes the same values into the buffer. Which would make sense, as it means a value of 1 actually means IPv4, not IPv6, as hinted at by the 127, 0, 0, 1 bytes. If this is true and I haven't missed something (which is entirely possible, it's late and I'm tired), I'd suggest the docs be modified to indicate that valid values are actually 1 and 2, not 0 and 1.

Very nice catch. Thank you! I will update the spec to match the reference code. The value of 0 means "no address".


[1.12] Bump dcos-net

High-level description
Bump dcos-net
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS_OSS-5118 DCOS-DNS always returns A records in the same order

Related tickets (optional)
Other tickets related to this change:
Checklist for all PRs

[x] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change:
[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain here: none
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[x] Test Results: [link to CI job test results for component]
[x] Code Coverage (if available): [link to code coverage report]

https://github.com/dcos/dcos-net/compare/23de0e27b1041a6cc8a40510e02419655443e1d5...5361e1d9b3cc5c4e55271fab666c07ca6c3f823b
https://github.com/dcos/dcos-net/pull/151
https://circleci.com/gh/dcos/dcos-net/1167
https://codecov.io/github/dcos/dcos-net/commit/5361e1d9b3cc5c4e55271fab666c07ca6c3f823b

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and review process
What is the review process and when will my changes land?
All PRs require 2 approvals using GitHub's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

@mesosphere-mergebot label [Ready For Review]


Request terminates in middleware.  How to call next middleware in stack?

Request terminates in middleware. There is however an option "option.onProxyRes" that allows you to further handle the request as you wish in the onProxyRes handler.
However, this handler only takes proxyRes, req, res as args. How may i then call the next middleware in my stack?
Excerpt from readme:
function onProxyRes(proxyRes, req, res) {
  proxyRes.headers['x-added'] = 'foobar'; // add new header to response
  delete proxyRes.headers['x-removed']; // remove header from response
}

Fairly new to express so please excuse me if this is a silly question..
Thanks!

Provided a middleware stack where i want to perform additional operations on the response.
E.g.:
app.use(baseUrl, (req, res, next) => {
      console.log("before");
      return res.end();
    })
     app.use(baseUrl, apiProxy);
     app.use(baseUrl, (req, res, next) => {
      console.log("after");
      return res.end();
    })

Request terminates in proxy middleware. "after" is therefore never logged.
Suppose i could move all middleware after proxy into the proxyResponse handler, but this is not ideal.
Alternatively create a separate express server that only handles the proxying add then wrap this as middleware for use in a outer express instance.

Provided a middleware stack where i want to perform additional operations on the response.
E.g.:
app.use(baseUrl, (req, res, next) => {
      console.log("before");
      return res.end();
    })
     app.use(baseUrl, apiProxy);
     app.use(baseUrl, (req, res, next) => {
      console.log("after");
      return res.end();
    })

Request terminates in proxy middleware. "after" is therefore never logged.
Suppose i could move all middleware after proxy into the proxyResponse handler, but this is not ideal.
Alternatively create a separate express server that only handles the proxying add then wrap this as middleware for use in a outer express instance.

Duplicate of #248

Duplicate of #248


Adding entries using dynamic syntax

Error occurs while trying to add multiple entries one by one using dynamic syntax.
First insert is successful, but after setting values to the second one OData Client inserts values from the first one.
Form I use:
dynamic result = await _client.For(x.Company) .Filter(x.Name == "myTestCompany") .NavigateTo(x.TestWorker) .Set(x.ID = workerDto.Id.ToString() , x.Name = workerDto.Name, x.Age = workerDto.Age) .InsertEntryAsync();

Error occurs while trying to add multiple entries one by one using dynamic syntax.
First insert is successful, but after setting values to the second one OData Client inserts values from the first one.
Form I use:
dynamic result = await _client.For(x.Company) .Filter(x.Name == "myTestCompany") .NavigateTo(x.TestWorker) .Set(x.ID = workerDto.Id.ToString() , x.Name = workerDto.Name, x.Age = workerDto.Age) .InsertEntryAsync();


issue found on android 13

Steps to reproduce
It works fine  on android 12 but when i run into android 13 i am not able to open gallery. Camera option works fine on every android version
Expected results
Not working
Actual results
When i run app on android 13 device. Image select from gallery button not working.
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.19.5, on Microsoft Windows [Version 10.0.19045.4291], locale en-US)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[√] Chrome - develop for the web
[√] Android Studio (version 2023.1)
[√] VS Code (version 1.88.1)
[√] Connected device (3 available)
[√] Network resources

Thanks for the report @Mohammad2464
Do you get any error or exception when you say the gallery doesn't open ?
Can you provide device details (make and model of android 13) on which this behavior occurs ?
Can you also provide a short video showing the behavior ? Also check if there are any settings on that device which prevents opening gallery (ex: permissions)
Also, check using image_picker plugin example on same device and see if you get same behavior or not.


Special characters not supported in delimiter of json_lines codec

According to the documentation of json_lines codec plugin 2 the default value of the delimiter option is newline (\n). However, when \n (or other unicode/special character like \x00) is explicitly defined in the input configuration of Logstash, it is considered to be a string containing of two characters:\ and n.
See this thread:
https://discuss.elastic.co/t/special-characters-not-supported-in-delimiter-of-json-lines-codec/114560
I encountered this around 5.2 and was surprised to see that it is still an issue in 6.4.2.

I would like to use the plugin like this:
output {
    tcp {
	codec => json_lines {
          delimiter => "\0"
        }
...
    }
}

But I end up with:
{...}\0{...}

instead of the actual \0 character, the same is true for any special characters.


main to live merge

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
Public repo URL: https://github.com/MicrosoftDocs/azure-docs
Private repo URL: https://github.com/MicrosoftDocs/azure-docs-pr
We have closed this pull request. Please resubmit the updates through the private repo. For more information, [see the contributor guide].(https://docs.microsoft.com/en-us/contribute/)

Docs Build status updates of commit dcb32f3:
:warning: Validation status: warnings

File
Status
Preview URL
Details

microsoft-community-training/infrastructure-management/install-your-platform-instance/configure-login-social-work-school-account.md
:warning:Warning
View
Details

microsoft-community-training/infrastructure-management/monitor-your-platform-infrastructure/troubleshoot-guide-for-azure-support.md
:warning:Warning
View
Details

microsoft-community-training/infrastructure-management/configure-your-platform-infrastructure/Add-multiple-AAD-to-B2C-as-a-social-account.md
:bulb:Suggestion
View
Details

microsoft-community-training/learning-in-offline-mode/azure-IOT-edge-device-setup.md
:bulb:Suggestion
View
Details

microsoft-community-training/learning-in-offline-mode/azure-IOT-hub-initial-setup.md
:bulb:Suggestion
View
Details

microsoft-community-training/TOC.yml
:white_check_mark:Succeeded
View

microsoft-community-training/analytics/custom-reports/database-schema.md
:white_check_mark:Succeeded
View

microsoft-community-training/infrastructure-management/configure-your-platform-infrastructure/set-up-custom-homepage-for-your-mct-instance.md
:white_check_mark:Succeeded
View

microsoft-community-training/infrastructure-management/install-your-platform-instance/create-teams-app-for-your-training-portal.md
:white_check_mark:Succeeded
View

microsoft-community-training/learning-in-offline-mode/offline-learning-with-microsoft-community-training.md
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity1.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity10.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity11.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity12.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity13.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity14.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity2.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity3.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity4.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity5.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity6.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity7.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity8.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity9.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/MCT Architecture.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/MultipleAAD1.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/MultipleAAD2.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/teamsapipermissions.png
:white_check_mark:Succeeded
View

microsoft-community-training/settings/configurations-on-the-training-platform.md
:white_check_mark:Succeeded
View

microsoft-community-training/infrastructure-management/install-your-platform-instance/configure-login-social-work-school-account.md

Line 164, Column 5: [Warning-duplicate-alt-text] Alt text 'Application Claims' is duplicated. Within an article, alt text must be unique.
Line 209, Column 9: [Warning-duplicate-alt-text] Alt text 'Application Claims' is duplicated. Within an article, alt text must be unique.

microsoft-community-training/infrastructure-management/monitor-your-platform-infrastructure/troubleshoot-guide-for-azure-support.md

Line 13, Column 1: [Warning-insecure-link] Link 'http://communitytraining.microsoft.com/' is insecure. Links to Microsoft sites must use 'https' instead of 'http'.
Line 5, Column 14: [Suggestion-duplicate-descriptions] Attribute 'description' with value 'Your training portal is composed of various resources such as Azure SQL, Azure Media Service, Azure App Service etc.' is duplicated in 'infrastructure-management/monitor-your-platform-infrastructure/monitor-your-infrastructure.md(6,14)', 'infrastructure-management/monitor-your-platform-infrastructure/troubleshoot-guide-for-azure-support.md(5,14)'.
Line 25, Column 63: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/azure/managed-applications/overview' will be broken in isolated environments. Replace with a relative link.

microsoft-community-training/infrastructure-management/configure-your-platform-infrastructure/Add-multiple-AAD-to-B2C-as-a-social-account.md

Line 16, Column 249: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/azure/active-directory/develop/quickstart-register-app' will be broken in isolated environments. Replace with a relative link.
Line 50, Column 86: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/azure/active-directory-b2c/direct-signin?pivots=b2c-user-flow#redirect-sign-in-to-a-social-provider' will be broken in isolated environments. Replace with a relative link.

microsoft-community-training/learning-in-offline-mode/azure-IOT-edge-device-setup.md

Line 18, Column 8: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /azure/iot-edge/how-to-register-device?view=iotedge-2018-06&tabs=azure-portal. Otherwise, remove the view parameter.
Line 22, Column 8: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /azure/iot-edge/how-to-install-iot-edge?view=iotedge-2018-06. Otherwise, remove the view parameter.
Line 43, Column 8: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /azure/iot-edge/how-to-deploy-modules-cli?view=iotedge-2018-06. Otherwise, remove the view parameter.

microsoft-community-training/learning-in-offline-mode/azure-IOT-hub-initial-setup.md

Line 63, Column 155: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /sql/relational-databases/security/password-policy?view=sql-server-ver15. Otherwise, remove the view parameter.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit dcb32f3:
:warning: Validation status: warnings

File
Status
Preview URL
Details

microsoft-community-training/infrastructure-management/install-your-platform-instance/configure-login-social-work-school-account.md
:warning:Warning
View
Details

microsoft-community-training/infrastructure-management/monitor-your-platform-infrastructure/troubleshoot-guide-for-azure-support.md
:warning:Warning
View
Details

microsoft-community-training/infrastructure-management/configure-your-platform-infrastructure/Add-multiple-AAD-to-B2C-as-a-social-account.md
:bulb:Suggestion
View
Details

microsoft-community-training/learning-in-offline-mode/azure-IOT-edge-device-setup.md
:bulb:Suggestion
View
Details

microsoft-community-training/learning-in-offline-mode/azure-IOT-hub-initial-setup.md
:bulb:Suggestion
View
Details

microsoft-community-training/TOC.yml
:white_check_mark:Succeeded
View

microsoft-community-training/analytics/custom-reports/database-schema.md
:white_check_mark:Succeeded
View

microsoft-community-training/infrastructure-management/configure-your-platform-infrastructure/set-up-custom-homepage-for-your-mct-instance.md
:white_check_mark:Succeeded
View

microsoft-community-training/infrastructure-management/install-your-platform-instance/create-teams-app-for-your-training-portal.md
:white_check_mark:Succeeded
View

microsoft-community-training/learning-in-offline-mode/offline-learning-with-microsoft-community-training.md
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity1.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity10.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity11.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity12.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity13.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity14.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity2.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity3.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity4.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity5.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity6.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity7.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity8.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/LoginIdentity9.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/MCT Architecture.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/MultipleAAD1.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/MultipleAAD2.png
:white_check_mark:Succeeded
View

microsoft-community-training/media/teamsapipermissions.png
:white_check_mark:Succeeded
View

microsoft-community-training/settings/configurations-on-the-training-platform.md
:white_check_mark:Succeeded
View

microsoft-community-training/infrastructure-management/install-your-platform-instance/configure-login-social-work-school-account.md

Line 164, Column 5: [Warning-duplicate-alt-text] Alt text 'Application Claims' is duplicated. Within an article, alt text must be unique.
Line 209, Column 9: [Warning-duplicate-alt-text] Alt text 'Application Claims' is duplicated. Within an article, alt text must be unique.

microsoft-community-training/infrastructure-management/monitor-your-platform-infrastructure/troubleshoot-guide-for-azure-support.md

Line 13, Column 1: [Warning-insecure-link] Link 'http://communitytraining.microsoft.com/' is insecure. Links to Microsoft sites must use 'https' instead of 'http'.
Line 5, Column 14: [Suggestion-duplicate-descriptions] Attribute 'description' with value 'Your training portal is composed of various resources such as Azure SQL, Azure Media Service, Azure App Service etc.' is duplicated in 'infrastructure-management/monitor-your-platform-infrastructure/monitor-your-infrastructure.md(6,14)', 'infrastructure-management/monitor-your-platform-infrastructure/troubleshoot-guide-for-azure-support.md(5,14)'.
Line 25, Column 63: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/azure/managed-applications/overview' will be broken in isolated environments. Replace with a relative link.

microsoft-community-training/infrastructure-management/configure-your-platform-infrastructure/Add-multiple-AAD-to-B2C-as-a-social-account.md

Line 16, Column 249: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/azure/active-directory/develop/quickstart-register-app' will be broken in isolated environments. Replace with a relative link.
Line 50, Column 86: [Suggestion-docs-link-absolute] Absolute link 'https://docs.microsoft.com/azure/active-directory-b2c/direct-signin?pivots=b2c-user-flow#redirect-sign-in-to-a-social-provider' will be broken in isolated environments. Replace with a relative link.

microsoft-community-training/learning-in-offline-mode/azure-IOT-edge-device-setup.md

Line 18, Column 8: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /azure/iot-edge/how-to-register-device?view=iotedge-2018-06&tabs=azure-portal. Otherwise, remove the view parameter.
Line 22, Column 8: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /azure/iot-edge/how-to-install-iot-edge?view=iotedge-2018-06. Otherwise, remove the view parameter.
Line 43, Column 8: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /azure/iot-edge/how-to-deploy-modules-cli?view=iotedge-2018-06. Otherwise, remove the view parameter.

microsoft-community-training/learning-in-offline-mode/azure-IOT-hub-initial-setup.md

Line 63, Column 155: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /sql/relational-databases/security/password-policy?view=sql-server-ver15. Otherwise, remove the view parameter.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Fix overlapping plugins/products build database access

This regression was introduced in https://github.com/apple/swift-package-manager/pull/7164/files#diff-7d5950f89d75bc14a591deb8270ebd32308c18248892c7abcb9d0ee3c6b14fd5L201.
We can restore the previous behavior in a hacky way by making sure that build parameters passed to plugin builds and invocations have the database path set in the same way as previously. This somewhat defeats the point of tools/products build parameters isolation, but we're able to isolate it only to a few lines in a single function.
I've added an explicit test case for this in new PluginsBuildPlanTests.swift file, so we can be sure it doesn't regress again.
Resolves rdar://120560817

@swift-ci test

@swift-ci test linux

https://github.com/apple/swift/pull/70996
@swift-ci test linux


Refactor the SHOW/SET documentation; include USE and SET NAMES.

Both SHOW and SET are really single statements which act on the
session (and they are also handled using a single code path in
CockroachDB!), with the exception of SET TRANSACTION which acts on the
current transaction. This patch acknowledges the commonality and
merges the various documentation pages into two more general pages.
Extends #1107, off release-1.1.
Fixes #1502.
Fixes #1503.
Also covers the change implemented in cockroachdb/cockroach#16459.

This change is

Somehow TC is confused. What should I do?

@knz, not sure. I'm going to rerun it and see if it happens again.

@benesch, any idea why teamcity won't run on this PR? It's opened against release-1.0 not master.

Against release-1.1 not release-1.0

Oh, I see now that the release-1.1 branch doesn't include the ci directory. @benesch, should we take care of this even though we'll move to the real versioning solution soonish?

@jseldess @benesch to integrate this change in the new layout I have a question: the PR contains a change to the sidebar, should I convert this by editing the sidebar-*.json directly? or is there some other file that the json is generated from?

Changes rebased, PTAL

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/da675fb82201e7c77c7269e2aa1a4b243223524b/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/e9b9f4ab301bb0d0de55f772e08df506e78aacef/


WIP: Cases page redesign

Description

Checklist:

[ ] Have you documented your changes in the pull request description?
[ ] Does the change present any security considerations?
[ ] Is any gem functionality overloaded? Eg: Devise controller methods being overloaded.
[ ] Has acceptance criteria been tested by a peer?
[ ] Has the CHANGELOG been updated? (If change is worth telling users about.)

General testing (author)

[ ] Test without JavaScript
[ ] Test on small screen

Accessibility testing (author)

[ ] Works keyboard only
[ ] Tested with one screen reader
[ ] Zoom page to 400% - content still visible
[ ] Disable CSS - does content make sense and appear in a logical order?

Codecov Report

Merging #1793 (aacf929) into master (a6a2eae) will decrease coverage by 56.67%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           master    #1793       +/-   ##
===========================================
- Coverage   89.75%   33.07%   -56.68%
===========================================
  Files         399      399
  Lines        8632    10387     +1755
  Branches     1257      789      -468
===========================================
- Hits         7748     3436     -4312
- Misses        703     6837     +6134
+ Partials      181      114       -67

Impacted Files
Coverage Δ

app/controllers/application_controller.rb
44.00% <0.00%> (-56.00%)
:arrow_down:

...ontrollers/investigations/activities_controller.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

...ontrollers/investigations/businesses_controller.rb
0.00% <0.00%> (-92.24%)
:arrow_down:

...pp/controllers/investigations/images_controller.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

...nvestigations/supporting_information_controller.rb
0.00% <0.00%> (-97.83%)
:arrow_down:

app/controllers/investigations_controller.rb
0.00% <0.00%> (-76.79%)
:arrow_down:

app/controllers/searches_controller.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

app/helpers/investigations_helper.rb
15.44% <0.00%> (-80.49%)
:arrow_down:

app/helpers/search_helper.rb
57.14% <0.00%> (-35.72%)
:arrow_down:

app/models/incident.rb
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 270 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 a6a2eae...aacf929. Read the comment docs.


Unique identifier for worker/controller nodes

Right now, all of the workers/controllers created by kube-aws receive the EC2 Name tag <cluster-name>-<nodepool>-(worker|controller). I'm wondering if it's possible to assign them unique names instead, or using a separate tag.
Use case
I'm trying to leverage Datadog to monitor all of the EC2 instances in my K8s clusters. Etcd nodes have a numeric suffixes which makes them easy to identify, but workers and controllers do not. The best unique identifier I can find is host (e.g. host:i-0f8ee52badf787725). This makes it a little difficult to correlate metrics with nodes that are named something like ip-X-X.X.X.ec2.internal.

Feel free to pass --hostname-override to kubelet with  whatever value you seem feet. you can make this edit in cloudformation template

@drywheat Hi! Sounds like what you need is achievable with the redbaron's suggestion.
Do you still need unique values for the Name tags for another use-cases?

Well, unless --hostname-override can be templated with something like <cluster>-<nodepool>-worker-${index} I'm not sure it's exactly what I need.
Basically, I want to build graphs in DD that will overlay metrics from multiple nodes and have some type of unique identifier that I can then use for deeper investigation.

You can have cluster and nodepool name as part of kube-aws template (golang template) and use instance metadata to get unique id. It will not be an ${index} as instances in AutoScalingGroup are not ordered


Problems with shallow clones via dup with custom each validators

@aeberlin can you provide a case that this is covering? It's hard to see what this PR is actually changing.


Find folks to help open projects

We need open project support!

@laurenjerome Ooooo nice! @nohorse

Daily work has ramped up, so I'm putting it up for grabs in case someone has some ideas to run with.

I am going to the event tomorrow


Include Color variables in the switch

Sketch 69 introduced color variables. This is a great way to have themes in separate libraries.
However, when using Camilo, it doesn't replace the color variables.

Would also like to see this implemented 👍

Hi @Pausansol , after installing the v2, the plugin stopped working, I can't toggle on its window anymore. Tried uninstalling the old version and installing a v2 from scratch.

Sketch version and macOS version?
El El jue, 5 nov 2020 a las 14:32, danieliustv<EMAIL_ADDRESS>escribió:

Hi @Pausansol https://github.com/Pausansol , after installing the v2,
the plugin stopped working, I can't toggle on its window anymore. Tried
uninstalling the old version and installing a v2 from scratch.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Pausansol/Camilo/issues/32#issuecomment-722380067,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDMZP376A4MQT743JVYCXDSOKSQRANCNFSM4SNCH5UQ
.

Sketch version and macOS version? El El jue, 5 nov 2020 a las 14:32, danieliustv<EMAIL_ADDRESS>escribió:
…

Sketch 69.2 (107504), Mojave 10.14.6 (18G4032)

I released a new version 2.0.1 with a bug fixed. Can you try the new one?

Thanks, it does work!! Does a v2 supports the color variables?

Not yet, but it will be ready soon

Hi @Pausansol, any news on this? 👍

Hello @Pausansol - amazing plugin!!!  I am also interested if any updates to this colour task.
Just to note it would great for the colour variables be linked to the self contained document itself, currently the layer styles do swap out the colour swatch but they reference directly the library file ( hosted locally / or cloud )

Is this still in progress, or on hold? As I'm very interested in this enhancement it would allow for styling of components/icons ect in designs which would swap style per theme, without having to create all variations in each theme.

+1 on this one, color variables in sketch is the enhancement that would make this plugin even better!

Hi all, thank you for your interest. I'm gonna release a new version with color variables features this week. Stay tunned. Thank you again for your support.

Released :) Please, report any issues related to color vars feature in another thread. Thank u all for your support.


Make log truncation configurable

Currently hard-coded to 50k, should be configurable. There is a command line option but it is ineffective.
@cket has a fix for this in one of his open (but stale) PRs. @cket which one? It might make sense to pull that change out.

Its actually not on a PR yet, I have a local branch where I was messing around with a solution for https://github.com/BD2KGenomics/toil/issues/697. I think they should be solved in the same PR but I can just open the PR and we can discuss from there once you see the code

Sounds good.


IE8 Object expected

IE8 is throwing "Object expected" at this line:
autoform-events.js#L272: getFormValues(...);
// Gather all form values
var formDocs = getFormValues(template, formId, ss);

No longer actively trying to support IE8, but I'll accept a pull request if someone figures out a fix.


[Web App] back and forward issue [Suggest] Back button behavior

hello,
i just try out flutter and found RouteMaster is so convenience with navigator 2.0. but i have some issues regards web app.

in Chrome, back button work as expected, but we cannot go forward.
in Ms Edge, back button go back to root and cannot go forward.

in addition, back button behavior should have option to 1. Reverse-chronological or 2. Upward.
regards,

Hi @ranuth-ruby thanks for the issue - we're discussing back behaviour over in #176, please feel free to join the conversation!
I'm afraid I don't know what the problem is with the forward button - are you able to make a small repo showing the issue?


ENH: quadratic assignment problem

What does this implement/fix?
Building off the great work of @asaadeldin11 in #11862, I have implemented a bare-bones method for approximate solution of the quadratic assignment problem. The method implements Umeyama's eigendecomposition approach. This method is simple, deterministic, and parameter-free.
Additional information
I have tried to get 100% code coverage with the tests. There are no benchmarks included, since they can be added at a later stage.
I have tried to copy the interface of linear_sum_assignment as much as possible. Calculating the score is perhaps a bit unwieldy though. I have left row_ind in the return value so we can implement rectangular matrices at some point.

I'm afraid I can't review this fully right now, but since this was inspired by gh-11862, I think it should be worked into gh-11862 rather than forcing the authors of the original PR to rework their contribution into this. Does that make sense?
Also, if we are going to add this as a separate method, then didn't you want there to be a method argument @pmla? Shouldn't quadratic_assignment just dispatch to another function depending on the value of method?
I worked with the authors of gh-11862 quite a bit on the interface, and while not everything was finished, one decision that was important was that the method should not simply return row_ind, col_ind like linear_sum_assignment because that cannot be extended in the future. It should return an OptimizeResult.

Thanks for your comments Matt.

since this was inspired by gh-11862, I think it should be worked into gh-11862 rather than forcing the authors of the original PR to rework their contribution into this. Does that make sense?

#11862 is very large, and working this into it will make it larger. My intention with this PR was to make a minimal change set which can be built upon with later PR's. If that is no good, we can close this and continue with #11862 as is.

if we are going to add this as a separate method, then didn't you want there to be a method argument @pmla? Shouldn't quadratic_assignment just dispatch to another function depending on the value of method?

I have split the solution logic off into a separate file, _qap_umeyama.py. As more solution methods are added, the logic of _qap.py can be extended to dispatch to them. A method argument is necessary until more methods are added.

I worked with the authors of gh-11862 quite a bit on the interface, and while not everything was finished, one decision that was important was that the method should not simply return row_ind, col_ind like linear_sum_assignment because that cannot be extended in the future. It should return an OptimizeResult.

I have changed the return value to OptimizeResult. I also copied the examples from the docstring in #11862.

I'm also noticing some unusual behavior... the score doesn't match the output matrix, and it's lower than the global optimum.

i think iGraphMatch has an R implementation we could compare to:
https://rdrr.io/github/dpmcsuss/iGraphMatch/man/graph_match_methods.html
On Fri, Aug 7, 2020 at 1:47 AM Matt Haberland<EMAIL_ADDRESS>wrote:

 External Email - Use Caution      *

@mdhaber commented on this pull request.
In scipy/optimize/tests/test_quadratic_assignment.py
https://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fscipy%2Fscipy%2Fpull%2F12161%23discussion_r466833742&data=02|01|jovo%40jhu.edu|2608ccf699284540a0d408d83a955d8c|9fa4f438b1e6473b803f86f8aedf0dec|0|0|637323760479099123&sdata=qmzBslqejomSqGgiAUzt9WRaUtvzDWTKCh%2BcbRKBwTE%3D&reserved=0
:

+def test_directed_weighted_graph_matching():

Example in section IVB of Umeyama:

"An eigendecomposition approach to weighted graph matching"

A = np.array([[0, 3, 4, 2],

             [0, 0, 1, 2],

             [1, 0, 0, 1],

             [0, 0, 1, 0]])

B = np.array([[0, 4, 2, 4],

             [0, 0, 1, 0],

             [0, 2, 0, 2],

             [0, 1, 2, 0]])

result = quadratic_assignment(A, B, maximize=True)
assert_array_equal(result.col_ind, [0, 2, 3, 1])

@asaadeldin11
https://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fasaadeldin11&data=02|01|jovo%40jhu.edu|2608ccf699284540a0d408d83a955d8c|9fa4f438b1e6473b803f86f8aedf0dec|0|0|637323760479109128&sdata=IRiXi7jg3WCg7yPrq5Cdxuh%2BdvDUXUs7FuFwJqR6mBk%3D&reserved=0
@jovo
https://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjovo&data=02|01|jovo%40jhu.edu|2608ccf699284540a0d408d83a955d8c|9fa4f438b1e6473b803f86f8aedf0dec|0|0|637323760479109128&sdata=F8u2EYQIhN1GO5naAUd15FRmcuGI84DcIAXA2ydaxg4%3D&reserved=0
Do you have easy access to an implementation of Umeyama's method? Can you
check the results on this problem?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fscipy%2Fscipy%2Fpull%2F12161%23discussion_r466833742&data=02|01|jovo%40jhu.edu|2608ccf699284540a0d408d83a955d8c|9fa4f438b1e6473b803f86f8aedf0dec|0|0|637323760479119115&sdata=NlBXC1QlbFhXuCRzuDCWIC0AiEY3bXuSw1vaZZjX13c%3D&reserved=0,
or unsubscribe
https://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAAKG4QH6BKVIC2N6GLAKFLR7OIO3ANCNFSM4NEOM5BA&data=02|01|jovo%40jhu.edu|2608ccf699284540a0d408d83a955d8c|9fa4f438b1e6473b803f86f8aedf0dec|0|0|637323760479119115&sdata=1nq9PrPEOBHnkPaWrEMTVIj2E1EAQawTZSWsaTBSv7c%3D&reserved=0
.

--
With gratitude,
Joshua T Vogelstein, PhD
neurodata.io | BME@JHU https://www.bme.jhu.edu/ | dA/dt > 0 > dJ/dt
https://twitter.com/neuro_data/status/1279067902658916352
where A = appreciating, J = judging, and t = time


8385.8 Add Slack master integration

Shippable/pm#8385
Show the Slack master integration in the add-ons section of the UI with a checkbox. Make sure that it is enabled/disabled when the button is clicked.

complete. closing.


Author ordering, completeness, acknowledgements, affiliations

@HeidiSeibold , @jlinx , @jpthiele , @jcohen02 @unode @samumantha , @jngrad , @fer-rum , @mhagdorn , @hvwaldow , @mschwarzmeier , @braunms , @BeastyBlacksmith , @tobyhodges , @knarrff , @dokempf , @ljgarcia , @fiveop
We need to have some consent, over the author ordering.
Also make sure that your acknowledgements are up-to-date, and your affiliations.
And that you are in the author list!!!!!

Actually I like the approach of The Turing Way

The first author is always The Turing Way Community and then authors are listed in alphabetical order by last name.

Likewise we could put de-RSE as first author. But I don't have strong feelings about this either way

the de-RSE part happens when we submit it as position paper, which is a separate process. For the first thing we would need a name....

There are several affiliations that only mention the university, without information about the institute or country: @mschwarzmeier @mhagdorn @fer-rum @unode @jpthiele @jcohen02 @jlinx @CaptainSifff
Alphabetical order would be fine by me, this is also how we did in the de-RSE position paper 001. As it stands, @unode would be the first author.

In order to emphasis different roles and ways of contribution, we could use CRediT.
Default contribution could be "Writing – original draft" and then additional roles can be assigned to each author, e.g "Supervision" for @CaptainSifff
I haven't used CRediT so far and I am not sure how we could include the roles in the authors list. Maybe in the footnotes where the email addresses might be placed usually?

There are several affiliations that only mention the university, without information about the institute or country: @mschwarzmeier @mhagdorn @fer-rum @unode @jpthiele @jcohen02 @jlinx @CaptainSifff

Good point, fixed and thereby checked mine!

There are several affiliations that only mention the university, without information about the institute or country: @mschwarzmeier @mhagdorn @fer-rum @unode @jpthiele @jcohen02 @jlinx @CaptainSifff

[x] fixed my affiliation

Added myself in #188. As my contribution is minimal, I am of course okay with being anywhere in the authors list. I would also support that those people that put a lot of effort in get mentioned more prominently, but that sort of requires a consensus on who that would be.

There are several affiliations that only mention the university, without information about the institute or country: @mschwarzmeier @mhagdorn @fer-rum @unode @jpthiele @jcohen02 @jlinx @CaptainSifff

Likewise, fixed my affiliation (only added city/country so pushed directly to main to avoid opening another PR).

One thing(irrespective of the ordering discussion) I'd like to add to the discussion:
Since this is a community paper, everybody actually profits, the more people and institutions represent and stand behind this effort.

I am fine with any ordering, but also would like it if the "first author" could be the "de-RSE community". Maybe @CaptainSifff as last author as you managed the paper writing. But really I have not been involved enough as of late to have a say. 😬

As someone favored by the English alphabet, I would like it if I don't accidentally end up at a prominent place near the beginning. Some people contributed significantly more, and I would like to see those in the front as corresponding/equal contributors, while the rest of us can follow in, I guess, alphabetical order. I know that this is difficult to decide, but I hope that a benevolent dictator with good overview could make a suggestion.
I don't know how it works with having a community as an author, but it would be very nice to anyway have somewhere the term "community paper". It clarifies a lot.

I reviewed my author information.
As for author order, whichever the community decides is fine by me.
Also agree with having the de-RSE community as author and calling it a community paper.
Regardless, I suggest to be explicit about this somewhere in the text as there are different conventions and assumptions in different communities. Some memorable ones include order of mushroom picking skills :mushroom:.
Perhaps there's a humorous one that can be used here (largest number of lines/characters of text/code deleted?)

Perhaps there's a humorous one that can be used here (largest number of lines/characters of text/code deleted?)

Since we plan on having the first author as "de-RSE", we could use a placeholder like the protagonist of our story (Kay) as a fictitious contributor (see F. D. C. Willard). The number of lines of text is fun too, although it could be hard to measure due to how much text was re-shuffled.

While the idea, of putting de-RSE as first author sounds nice, I don't think it is right for an arxiv upload on friday.
We can change that (The authors, and the ordering do not have to stay fixed for different versions of the paper), once we have published it to the de-RSE ML and gave people a possibility for feedback for some time. And it will get the RSE badge, once it goes through the de-RSE position paper process. But all of this comes later.
I agree with @MakisH that probably the abstract should already include the community aspect.

If we cannot have de-RSE yet, I propose Florian as first author for the arxiv upload, with everyone else listed alphabetically afterwards. That will give us time to create a description of everyone's contributions according to CRediT (as proposed by @braunms above), to use when we are able to adjust the author list to add de-RSE as first author, while recognising @CaptainSifff's extraordinary contribution in the meantime.

One thing(irrespective of the ordering discussion) I'd like to add to the discussion: Since this is a community paper, everybody actually profits, the more people and institutions represent and stand behind this effort.

Still, I feel a bit awkward to just add my name to competencies.md and contributors.yml since I did not make any commit. There are probably others out there who feel the same?!
Thus, this is just to clearify. I just participated in the discussions at deRSE unconf and did some research on resources (#133) and masters programmes (#4).
If that's fine for you, I'd be happy to add my name to the list to strengthen the community (at least quantitatively :smile: )

Any ordering will be fine with me.
Alphabetical upwards/downwards by n-th letter of first / last name, random, 'age' as years in academia, ordering of affiliations from north to south or west to east,...

in reply to @tobyhodges
maybe it's just me feeling that it would be wrong to claim already now that we speak for all of de-RSE....

I've added @dokempf to the author list, assuming that he added himself to the contributors.yml file, but forgot to add himself to competencies.md

I've added @dokempf to the author list, assuming that he added himself to the contributors.yml file, but forgot to add himself to competencies.md

Ups, I had misclicked and added myself to call_for_action.md . I guess I am obliged to contribute to that one now.

maybe it's just me feeling that it would be wrong to claim already now that we speak for all of de-RSE....

No, I agree. I was suggesting that we list you @CaptainSifff as first author for now, and change that to de-RSE when we formally receive the blessing of the e.V.

resolved by e9d93cf96f840a8507968dd030539f7be6a578c0 for this publication. We remember to move this to de-RSE community once we get community blessing.


Configure warnings to be errors and fail build

In our app's front matter, if I forget a required field (in this case pageType) I get a helpful warning in the console when running a build:
Warning: Found 1 problems in 24 documents.
└── Missing required fields for 1 documents. (Skipping documents)
 • "components/button/tabs/history.mdx" is missing the following required fields:
       • pageType: enum

Is there a way to turn this warning into an error? Would like to fail the build in this case.

Yes. You can configure this via the onMissingOrIncompatibleData option in makeSource:


Discretized Model Scalling

I follow how the model dynamics are linearized/discretized to form Ad, Bd, and gd. However, could you provide some more detail about the following scaling step that seems to lift the problem:
% constructing augmented system with state-input scaling
    Ak = [MPC_vars.Tx*Ad*MPC_vars.invTx MPC_vars.Tx*Bd*MPC_vars.invTu; zeros(nu,nx) eye(nu)];
    Bk = [MPC_vars.Tx*Bd*MPC_vars.invTu;eye(nu)];
    gk = [MPC_vars.Tx*gd;zeros(nu,1)];

I don't quite follow why we increase the dimension of x by nu.

https://github.com/alexliniger/MPCC/blob/949909291a0c32c3043017cc4f9caab37f01b0ce/Matlab/getMPCmatrices.m#L35
The idea is that the MPC does not compute the inputs but how much to change the inputs, this allows for rate constraints and rate costs, while still maintaining the structure necessary for hpipm.

Thanks, I missed that note.


qtPreHook Error

While basically everything seemed to work (it built and posted the result), I see an error in the output:
$ git -c fetch.prune=false fetch --no-tags --force https://github.com/NixOS/nixpkgs master:refs/nixpkgs-review/0 pull/151388/head:refs/nixpkgs-review/1
remote: Enumerating objects: 85, done.
remote: Counting objects: 100% (54/54), done.
remote: Total 85 (delta 54), reused 54 (delta 54), pack-reused 31
Unpacking objects: 100% (85/85), 91.40 KiB | 272.00 KiB/s, done.
From https://github.com/NixOS/nixpkgs
   c0b0059e8a4..e7eb62576c8  master                -> refs/nixpkgs-review/0
   eed4bbb918b..8ebce93d1f6  refs/pull/151388/head -> refs/nixpkgs-review/1
$ git worktree add /home/austin/.cache/nixpkgs-review/pr-151388/nixpkgs e7eb62576c8d3bba35136fdafb80ae1aa8e92a05
Preparing worktree (detached HEAD e7eb62576c8)
Updating files: 100% (29330/29330), done.
HEAD is now at e7eb62576c8 Merge pull request #151346 from applePrincess/exploitdb-2021-12-18
$ nix-env --option system x86_64-linux -f /home/austin/.cache/nixpkgs-review/pr-151388/nixpkgs -qaP --xml --out-path --show-trace
$ git merge --no-commit --no-ff 8ebce93d1f6962bbb0c690872c3c64db5e6e0fd3
Automatic merge went well; stopped before committing as requested
$ nix-env --option system x86_64-linux -f /home/austin/.cache/nixpkgs-review/pr-151388/nixpkgs -qaP --xml --out-path --show-trace --meta
1 package updated:
digikam (7.3.0 → 7.4.0)

$ nix --experimental-features nix-command build --no-link --keep-going --option build-use-sandbox relaxed -f /home/austin/.cache/nixpkgs-review/pr-151388/build.nix
error: builder for '/nix/store/kh2h1cj3n6d8binmmg38mmhgiqbsyz91-review-shell.drv' failed with exit code 1;
       last 2 log lines:
       > qtPreHook
       > Error: wrapQtAppsHook is not used, and dontWrapQtApps is not set.
       For full logs, run 'nix log /nix/store/kh2h1cj3n6d8binmmg38mmhgiqbsyz91-review-shell.drv'.

Link to currently reviewing PR:
https://github.com/NixOS/nixpkgs/pull/151388

1 package built:
digikam

Posting result comment on https://github.com/NixOS/nixpkgs/pull/151388
$ nix-shell /home/austin/.cache/nixpkgs-review/pr-151388/shell.nix

Running nix log as suggested:
@nix { "action": "setPhase", "phase": "qtPreHook" }
qtPreHook
Error: wrapQtAppsHook is not used, and dontWrapQtApps is not set.

The error comes from a Qt hook in nixpkgs. So I assume it's complaining about the digikam package. But the output is confusing to me for two reasons:

Is the failure about nixpkgs-review or the package being built?
Should this count as a success as it did?

We just opportunistically add all affected packages to a build.nix. We might need to work around by adding dontWrapQtApps = true to this file: https://github.com/Mic92/nixpkgs-review/blob/335f0265ced9d5074efe19c393a446fc9af3b274/nixpkgs_review/nix.py#L205


ConversationResourceResponse.java json properties case problem

Hello.
When we tried to create new conversation using java Library the function createConversation returned ConversationResourceResponse object without error but the id parameter was always null.  By debugging the code we have found that the MicrosoftBotFramework rest service result contained non null Id parameter.
The problem is that Json annotations in ConversationResourceResponse are by default case sensitive.  So when MBF rest service returns parameters with names Id, ServiceUrl, ActivityId they are not properly deserialized into properties in ConversationResourceResponse class with these annotations.
@JsonProperty(value = "activityId")
@JsonProperty(value = "serviceUrl")
@JsonProperty(value = "id")
We were able to fix this behavior by using changed annotations
@JsonProperty(value = "ActivityId")
@JsonProperty(value = "ServiceUrl")
@JsonProperty(value = "Id")
Please consider finding some solution for example: changing rest-response or changing Json annotations or configure deserializer to be case insensitive. etc...
Thank you.

Hi @rmenkyna , when creating a Conversation the REST API (according to documentation) only returns the ResourceResponse ID, hence you should only able to see the conversation id (created or re-used).
FYI I can get the conversation ID via the Java api without problems:
ConversationResourceResponse conversationResourceResponse =
connector.conversations().createConversation(parameters);
LOGGER.log(Level.INFO, "conversation.id {0}", conversationResourceResponse.id());

Hope it helps.
/Bep

By convention, the Microsoft BotFramework uses camel case when naming JSON properties, so the JSON Annotations should be camel case as well. activityId in the incoming JSON should map to @JsonProperty(value = "activityId") when the framework deserializes the incoming JSON into a ConversationResourceResponse object.
When I run the Java BotFramework Connector Tests, I am not seeing any issues regarding the activityId, and I'm not getting any errors when I run @gcatanese's code above.
@Test
public void CreateConversation() {

    Activity activity = new Activity()
            .withType(ActivityTypes.MESSAGE)
            .withRecipient(user)
            .withFrom(bot)
            .withText("TEST Create Conversation");

    ConversationParameters params = new ConversationParameters()
            .withMembers(Collections.singletonList(user))
            .withBot(bot)
            .withActivity(activity);

    ConversationResourceResponse result = connector.conversations().createConversation(params);
    Assert.assertNotNull(result.activityId());
}

Closing as not repro


run preFlightCheck and postFlightCheck immediately after migration

Closes #1818
This makes it so that the preFlightCheck and postFlightCheck are now run immediately before and immediately after the migration is run. This gives the advantage that one could run stuff against the adapter if they wanted to in the preFlightCheck, and that the postFlightCheck will run before the migration is logged as having run. This is especially beneficial in the latter's case given that by default, it will throw an exception if the user has incomplete table operations.
While this does slightly change the order things and run in, and that the checks are run within the migration timer, it's not really BC breaking enough to warrant necessarily going in the 0.13.x line only.

Sounds reasonable to me.


[Runtime] Allow aborting fetchWithCache through AbortSignal

This is a follow-up for a previous change https://github.com/apache/tvm/pull/17208.
This Pull Request updates function fetchWithCache() in TVM runtime class to accept an optional parameter signal: AbortSignal, so that users could use AbortController to abort the fetch process if needed.
https://developer.mozilla.org/en-US/docs/Web/API/AbortController
Related issues:
https://github.com/mlc-ai/web-llm/issues/484
https://github.com/mlc-ai/web-llm/issues/499

cc. @tqchen