11461 register torque layer dependencies

Fixes #11461
Makes the following important changes:

Adds Map.carto_layers to return layers with kind carto
Map.data_layers now returns carto and torque layers
Removed Map.carto_and_torque_layers

This makes the interface at map level more similar to the one at layer level. There is an overview table at the ticket. There are more changes that could be made (Map.base_layers is a good candidate, since it returns all layers), but this only focuses in the one producing the actual issue.
This fixes the following:

No update table dependencies on torque layers
No auto-indexing on torque layers (when publishing a map)
No layer node style on torque layers presenter (little effect, since the frontend switches torque layers to normal after applying an analysis)
No updated_at date in torque layers
Incorrect calculation of max layers per map (not counting torque layers)

Acceptance

[ ] STR of the ticket
[ ] Create an animated torque layer over a dataset with > 10k rows. A timeline widget should be automatically added. Check Rollbar, you should see an autoindex trigger.

More acceptance:

[ ] In a map with a Torque layer, try adding a new Torque layer. You shouldn't be able to.
[ ] Export a .carto from a map with labels on top, a normal layer and a torque layer. Import it. It should keep layers and order.
[ ] Repeat with a map using a basemap without layers.

Added your acceptance suggestions to the top list. Ready to go?


Dependabot is Dependably Aggravating

What can be done? @sergejparity, any recommendations?

I should note, its PRs require admin merge.

@harrysolovay was the problem with the PR's created by @dependabot like this one https://github.com/paritytech/capi/pull/287? (can't find any problems with that)
Or with created by GHA https://github.com/paritytech/capi/blob/82308eb581ee74f0a7a42e746f7755dd71cc230a/.github/workflows/udd.yml#L25-L35 ?
This one produces unverified commits as I can see from here https://github.com/paritytech/capi/pull/292/commits

@sergejparity you're correct. Just updated my last comment:

these PRs are from UDD, not dependabot. Ie., unrelated.

Nothing to be done for now––still, going to leave this issue open (hoping to eventually simplify dependency updates).


Update kafka-avro-serializer, ... to 5.4.10

Updates

io.confluent:kafka-avro-serializer
io.confluent:monitoring-interceptors

from 5.4.6 to 5.4.10.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.confluent" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "io.confluent" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #299.


Implement the NegotiateEncryptedMessage message in the MS-WSMV protocol document <IP_ADDRESS>.1

Implement the NegotiateEncryptedMessage message in the MS-WSMV protocol document <IP_ADDRESS>.1, which is the method of encrypting an HTTP connection on the default 5985 port, and you can refer to pywinrm's implementation at https://github.com/diyan/pywinrm/blob/master/winrm/encryption.py. The protocol document can be downloaded from https://learn.microsoft.com/en-us/openspecs/windows_protocols/ms-wsmv/055dc36b-db2a-41ae-a47b-82cbfa0b4a92.
Thank you for your code library, it has been a great help to me, but not having the implementation of the message is frustrating, and I look forward to your future updates.

I have a PR in to implement this in this repo but this seems to be not very active
you could use https://github.com/CalypsoSys/bobwinrm
see https://github.com/CalypsoSys/bobwinrmntlm for usage


Update DBAL 3.x version from 2018-04-17 to 2018-11-26

Some BC breaks happened in DBAL 3.0.x-dev in meantime: https://github.com/doctrine/dbal/blob/snapshot/develop/2018-11-26/UPGRADE.md#upgrade-to-30, these have been fíxed on our side, notably:

NULL for $offset in LIMIT
removed dbal:import
relying on Type::__toString())

This is also pre-requisite for dropping doctrine/common.

@morozov Yes to all your comments, I just didn't want to break BC in this PR (later).


w-minify cache cleanup

See a way to cleanup old cache files.

Maybe in v2 ... not crucial for this release.


Webhooks upgrade complete fix

Fixes #1279
Motivation:
Fixes the bug described in #1279
Modifications:
Moved state change in NIOHTTPClientUpgradeHandler.performUpgrade to after firing buffered messages.
Result:
The upgradeState will now only change to upgradeComplete after buffered reads are handeld.

I'll give it a shot! 😄

@swift-nio-bot test this please

@MrLotU the new test is also missing:
11:31:31 => Checking linux tests... missing changes!
11:31:31 diff --git a/Tests/NIOHTTP1Tests/HTTPClientUpgradeTests+XCTest.swift b/Tests/NIOHTTP1Tests/HTTPClientUpgradeTests+XCTest.swift
11:31:31 index 616c9ba7..d5911d90 100644
11:31:31 --- a/Tests/NIOHTTP1Tests/HTTPClientUpgradeTests+XCTest.swift
11:31:31 +++ b/Tests/NIOHTTP1Tests/HTTPClientUpgradeTests+XCTest.swift
11:31:31 @@ -30,6 +30,7 @@ extension HTTPClientUpgradeTestCase {
11:31:31                  ("testSimpleUpgradeSucceeds", testSimpleUpgradeSucceeds),
11:31:31                  ("testUpgradeWithRequiredHeadersShowsInRequest", testUpgradeWithRequiredHeadersShowsInRequest),
11:31:31                  ("testSimpleUpgradeSucceedsWhenMultipleAvailableProtocols", testSimpleUpgradeSucceedsWhenMultipleAvailableProtocols),
11:31:31 +                ("testUpgradeCompleteFlush", testUpgradeCompleteFlush),
11:31:31                  ("testNoUpgradeAsNoServerUpgrade", testNoUpgradeAsNoServerUpgrade),
11:31:31                  ("testFirstResponseReturnsServerError", testFirstResponseReturnsServerError),
11:31:31                  ("testUpgradeResponseMissingAllProtocols", testUpgradeResponseMissingAllProtocols),
11:31:32 Build step 'Execute shell' marked build as failure
11:31:32 $ ssh-agent -k

please run scripts/generate_linux_tests.rb and record the changes it makes

@swift-nio-bot test this please


[QUESTION]Interpretation and comparison of COMET scores across several languages

❓ Questions and Help
Before asking:

Search for similar issues.
Search the docs.

What is your question?

Is there a way to interpret the COMET score other than using it as a ranking system?
For example, we have a French dataset of 500 sentences from TED Talks and a Spanish dataset consisting of 500 sentences from Parliament sessions.  We are comparing the COMET scores for French and Spanish, the scores are denoted as comet_fr and comet_es. If comet_fr > comet_es, does that mean that the machine translation quality of French is better than the machine translation quality of Spanish? Is the COMET score comparable across languages? Or is this comparison invalid because the source data is not the same?

Code

What have you tried?
What's your environment?

OS: [e.g. iOS, Linux, Win]
Packaging [e.g. pip, conda]
Version [e.g. <IP_ADDRESS>]

Hi @clairehua1,
You should avoid comparing scores between languages and even between domains. This is not just for COMET but for any MT Metric.
For example BLEU, even tho is lexical, highly depends on the underlying tokenizer thus the results vary a lot between different languages.
PS: even human annotation has a lot of variability between languages and domains. If we want reliable and comparable results we need to make sure the test conditions are the same (same data, same annotators)
Cheers,
Ricardo

Thanks for the answer Ricardo! Is there a way to interpret the COMET score other than using it as a ranking system?

@clairehua1 for a specific setting (language pair and domain) you could plot the distribution of scores and analyse it by looking at quantiles. The scores usually follow a normal distribution.
To give a bit more context most models are trained to predict a z-normalized direct assessment (a z-score). Z-scores have a mean at 0 and follow a normal distribution which means that ideally a score of 0 should represent an average translation.
In practise the distribution of scores (for the default models wmt20-comet-da) is slightly skewed towards positive scores which means that an average translation is usually assigned a score of 0.5.  I have an explanation here

I the plots above you can see how different is the scores between English-German and English-Hausa. But you can see that 0.5 is typically the peak of the distribution.


Remove a puts call

This probably shouldn't be here as it can potentially leak PII data via logs and isn't configurable (via loggers).
Might be good to test/lint for puts and similar calls.

Thanks, Kristján!


Is anyway to disable the input for display only?

Hi, this is a great directive. I'm using this for displaying an average rating which doesn't allow update.
Is there a "disabled" attribute?

From the source codes, it seems that adding a "readonly" attribute would serve your purpose.

I've updated how to do it in the readme file, See #17  Closing this one...


Update crate root cfg_set from config instead of using default

This change is

crates/cairo-lang-compiler/src/project.rs line 83 at r1 (raw file):

            Some(CrateConfiguration {
                root,
                settings: CrateSettings {

Why not just crates_config.clone()?


Some visualization imports fail with wrong number of arguments

https://rollbar.com/vizzuality/CartoDB/items/18396/?item_page=0&#instances
ArgumentError: wrong number of arguments (1 for 2)
1
File "/home/ubuntu/www/production.cartodb.com/releases/20160511143153/services/minimal-validation/validator.rb", line 32 in store
2
File "/home/ubuntu/www/production.cartodb.com/releases/20160511143153/services/minimal-validation/validator.rb", line 32 in validate_presence_of_with_custom_message
3
File "/home/ubuntu/www/production.cartodb.com/releases/20160511143153/app/models/visualization/member.rb", line 197 in valid?
4
File "/home/ubuntu/www/production.cartodb.com/releases/20160511143153/app/models/visualization/member.rb", line 168 in store
5
File "/home/ubuntu/www/production.cartodb.com/releases/20160511143153/app/services/carto/visualizations_export_persistence_service.rb", line 61 in save_import
6
File "/home/ubuntu/www/production.cartodb.com/releases/20160511143153/app/connectors/importer.rb", line 108 in block in create_visualization
7
File "/home/ubuntu/www/production.cartodb.com/releases/20160511143153/app/connectors/importer.rb", line 107 in each
8
File "/home/ubuntu/www/production.cartodb.com/releases/20160511143153/app/connectors/importer.rb", line 107 in create_visualization
9
File "/home/ubuntu/www/production.cartodb.com/releases/20160511143153/app/connectors/importer.rb", line 60 in run
10
File "/home/ubuntu/www/production.cartodb.com/releases/20160511143153/app/models/data_import.rb", line 669 in new_importer
11
File "/home/ubuntu/www/production.cartodb.com/releases/20160511143153/app/models/data_import.rb", line 398 in dispatch
12
File "/home/ubuntu/www/production.cartodb.com/releases/20160511143153/app/models/data_import.rb", line 176 in run_import!
13
File "/home/ubuntu/www/production.cartodb.com/releases/20160511143153/lib/resque/importer_jobs.rb", line 9 in block in perform
14
File "/home/ubuntu/www/production.cartodb.com/releases/20160511143153/lib/resque/base_job.rb", line 21 in call
15
File "/home/ubuntu/www/production.cartodb.com/releases/20160511143153/lib/resque/base_job.rb", line 21 in run_action
16
File "/home/ubuntu/www/production.cartodb.com/releases/20160511143153/lib/resque/importer_jobs.rb", line 9 in perform

It's caused by...

a bug in the minimal validator.
visualization importing should transform protected into private, since passwords are not exported.


Add custom font provided by .ttf file support

Partially addresses #56
I have to investigate further why different font formats aren't supported (i.e. .otf), but this PR already introduces new API and adds ability to provide custom font in .ttf format

Codecov Report

Merging #62 into master will increase coverage by 2.47%.
The diff coverage is 63.15%.

@@             Coverage Diff              @@
##             master      #62      +/-   ##
============================================
+ Coverage     55.52%   58.00%   +2.47%
- Complexity       75       77       +2
============================================
  Files            12       12
  Lines           362      400      +38
  Branches         63       71       +8
============================================
+ Hits            201      232      +31
- Misses          117      120       +3
- Partials         44       48       +4

Impacted Files
Coverage Δ
Complexity Δ

...ct/starter/easylauncher/filter/ChromeLikeFilter.kt
11.42% <14.28%> (+1.75%)
3.00 <0.00> (ø)

...t/starter/easylauncher/filter/ColorRibbonFilter.kt
61.53% <62.50%> (-0.76%)
10.00 <0.00> (ø)

.../starter/easylauncher/plugin/EasyLauncherConfig.kt
75.21% <71.42%> (+6.24%)
39.00 <1.00> (+2.00)

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 46cdbb5...6e5228a. Read the comment docs.


amplify status shows Update when changing env?

Describe the bug
I have two amplify enviroments: dev and prod.
If my current env is prod, when I do amplify env checkout dev followed by amplify status. I would see all my functions show Update though nothing is being changed. Why is this? I would expect to see No Change. My current workaround is to do amplify env pull --restore every time after I do amplify env checkout xxx to make sure I didn't modify anything.
Amplify CLI Version
4.16.1
To Reproduce
Create an amplify env dev and auth with admin query function enabled, push to cloud.
Duplicate dev to be prod and push prod to cloud.
Change from prod to dev and do amplify status, observe admin query status to be Update.
Desktop (please complete the following information):
Windows

Please take a look at a similar issue out here - https://github.com/aws-amplify/amplify-cli/issues/1850#issuecomment-592435957
These values changing should not prevent pushing resources, and changes to these lines can be safely ignored. When pushing a resource, these templates get updated with correct s3 bucket keys. If you want to keep git history clean, your team could decide not to commit changes these lines, but any changes should not have a negative effect.Having said that, we're looking into ways and tracking as to how we can avoid this altogether.
cc @UnleashedMind


Update clang tools version in CI

Homebrew formulae for MacOS  installs v12 of clang tools by default now, and once again it is not compatible with the previous version.  This installs v12 on our Docker image, so we don't have to jump through more hoops for Mac development.

Homebrew formulae for MacOS installs v12 of clang tools by default now, and once again it is not compatible with the previous version. This installs v12 on our Docker image, so we don't have to jump through more hoops for Mac development.

What about the Windows peeps?

clang-format 11 on Windows isn't even compatible with the same version on other platforms, so I'm not sure how valuable it is to cater to Windows here.  I've had to switch platforms and re-run the tool to fix bad Windows formatting multiple times.

clang-format 11 on Windows isn't even compatible with the same version on other platforms, so I'm not sure how valuable it is to cater to Windows here. I've had to switch platforms and re-run the tool to fix bad Windows formatting multiple times.

I am good with that answer!

Word of warning, the CI won't use this version to validate the current sources in this PR's checks. It is possible merging will break CI if the current sources are not correct to this clang-format or clang-tidy. Be prepared to post a PR to correct the sources too.


column doesn't update with new data if you relax a filter

To reproduce:
TCGA BRCA
Add gene expression (illuminahiseq), TP53
Filter samples to those in gene expression (agilent)
Add gene expression (illuminahiseq), TP53 (same as before)
Now filter samples to those in gene expression (illuminahiseq)
Observe grey data

This is due to the widget data fetch not being re-run after changing samples-from.

Fixed via 625f51fdacb9c9613d04eab51e726caee7be5ade


[WIP] Edit History Screen

What does this PR?
This PR adds edit history feature for  post which is available in ecency web app.
This is Work in Progress.
Issue number
#2257
Screenshots/Video

Update: 10 May 2022
https://user-images.githubusercontent.com/6298342/167702064-2e83a3c1-aa17-4261-a0cf-410b837d2d41.mov

@noumantahir and @aliseyalvi we have highlight package using in Search page to highlight searched text, same can be used for highlighting modifications/diffs, something to consider in future.


Support Avalonia 11.0.0 preview2

Purpose:

Updates AvaPlot and demo to use Avalonia 11.0.0-preview2
Fixes event names breaking changes
Uses IStorageProvider for save dialog

See also:

https://github.com/ScottPlot/ScottPlot/issues/2183
https://github.com/AvaloniaUI/Avalonia/discussions/9004

@swharden
A note about IStorageProvider: The provider favors streams over filenames, probably in an effort to make the API completely agnostic to file path strings. It would be a good idea, in my opinion, to have a SaveFigure overload that took a stream instead of a filename to support this. Currently, I just use Uri to get the path.

I also just ran into a pretty major bug with System.Drawing.Common. It has to do with this breaking change. Microsoft has decided to offload the cross-platform work onto Skia and MAUI implementations, rather than support this "subpar" option.
On my M1 Mac I had to add the recommended runtimeconfig.template.json file with the following contents:
{
   "configProperties": {
      "System.Drawing.EnableUnixSupport": true
   }
}

I added this file to the root of my own project which references ScottPlot. The plot shows up just fine.
I'm looking forward to ScottPlot 5!

For anyone who finds this PR, the error I received was:
System.TypeInitializationException: The type initializer for 'Gdip' threw an exception.
 ---> System.PlatformNotSupportedException: System.Drawing.Common is not supported on non-Windows platforms. See https://aka.ms/systemdrawingnonwindows for more information.

HI @Xerxes004, sorry it took so long for me to properly address this.
We had a quick chat in the Discord #1966 and decided that ScottPlot's controls should only target Avalonia releases (not previews). I'll close this PR for now, but when Avalonia 11 is fully released we can open it up again and pick this up from there 👍

I think another good alternative is merging this into a separate branch, which we can rebase or fix up whenever Avalonia 11 becomes stable.

Hey @swharden @bclehmann no problems, I totally understand. For now I'm going to target released Avalonia in a separate project and just use that separately from our main project.


Point is not hidden when goes outside the visible cartesian space

Bug Report
Issue:
I'm using a point and a crosshair to highlight a specific point in a graph.
The crosshair works as expected but the point is rendered also when it goes outside the axis ranges.
This happens when the user moves the axis around until the point goes outside the axes ranges
(in the example below the point has coordinates (100,0)).

Reproducing:
The point is added when the graph is initialized

and then is just changed its coordinates in the following way (and its .IsVisible property to show/hide the point).

Maybe I'm doing something wrong in how I use a point, but I've followed this example:
https://scottplot.net/faq/mouse-position/#show-value-near-mouse-with-winforms
System Details

ScottPlot Version: 4.1.27
Operating System: Windows 10
Application Type: WPF
.NET Version: NET Framework 4.7.2

I have also ran into this problem, and after some testing and having checked the source code, I believe it to be a problem with the recently added MarkerPlot type. First of all, when testing with other plot types, MarkerPlot (including AddPoint, which was changed to create a MarkerPlot in 4.1.27) is the only plot type to go out of bounds, as seen in the following screenshot:

Second, after checking the source code, I have noticed that when rendering, most of the other Plottable types call the GDI.Graphics(Bitmap bmp, PlotDimensions dims, bool lowQuality = false, bool clipToDataArea = true) overload (see for example here), whereas MarkerPlot only uses the GDI.Graphics(Bitmap bmp, bool lowQuality = false, double scale = 1.0) version, which does not check for bounds (see here). As such, I believe that simply changing MarkerPlot to call the correct overload might be enough to fix this bug, though I'm not familiar enough with the rest of the codebase to be completely sure, there might have been a reason for calling a different overload in the first place (maybe to prevent the marker size from scaling?).

Dear PremekTill,
I can tell that after I added the PlotDimensions to the second overload you are mentioning the plot area is fully respected - so perhaps it was just forgotten. See attachment:
MarkerPlot.zip


Typescript: Intellisense not working with union type

VSCode Version: 1.23.1
OS Version: Pop!_OS 17.10

Steps to Reproduce:

Create a typescript file and try the following. When the type is "unioned" with undefined, the autocomplete cannot advise to use the enum anymore.

This is most annoying with situations like this, with optional parameters given to functions.

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

This issue was moved to Microsoft/TypeScript#24304


 tag not followed when a parent has a click binding

I added a click.trigger event to a  that allows a row to be "selected":

I then had an Actions column that allowed a user to go to the edit screen via an  tag with:

When you click the  tag, the  click event ends up handling it instead and won't let it bubble up.  Thus, the only way around it is to add your own click event on that  tag.  Is there any way around this that allows a normal  tag click to get handled by itself in this scenario?

Some of your code is missing from above, so I'm not entirely sure of the use case. However, you can allow the event to continue to bubble by returning true from your triggered method. If I understand you correctly, that should solve your problem.

Ah, didn't realize you could do that.  Thanks! :)


Bad calculation the first time, good calculation the second time

Hello,
On the jcarousel of my blog (the one under the title "C'est vous qui le dites !" on this page : http://sxjpl.free.fr/biblio/), the last image is not fully displayed the first time you see it (by clicking the next button twice). However, if you click on the previous button, go to the start, and then go to the end of the carousel once more, the last image is then fully displayed.
I assume there's a calculation problem : the width of the carousel is not right the first time you display it, but it's good the second time... Would you have any idea ? Or is it a problem with my CSS code ? Could you just point me to the right direction ?
Thanx very much in advance for any idea you might have,
Best regards,
Dupond

Furthermore, it preloads four images, whereas only three are necessary !

Have you fixed it? Can't reproduce this behaviour.

No answer, so I assume you still don't see it. Well it's not a big problem ;)
Just one thing though : why does it always preload (on both addresses I've given) 4 images instead of 3 ? And when you click on the right button of the carousel, you see there is always one more image preloaded. Is it a CSS problem of my fault ? Or should I see my JS code again ?
Thanx for your help !
Best regards,
Dupond

Hm, to be honest i'm not entirley sure whats the reason for this.
You could try carousel.reload() and the the end of the carousel_comments_itemAddCallbackfunction.

No, unfortunately it doesn't change anything :(
Best regards,
Dupond

Where is it? Can't find it in your code.

Sorry I was on vacation. Now I'm back.
I had added it, but since it didn't had any effect, I had removed it.
I've added it back, as you can see here : http://sxjpl.free.fr/biblio/

It seems to work now. Btw, since you only one xml with all items, i would recommend to load all items at once.

Yes, the partially displayed image seems to work now.
But there still is the following problem : 4 images are loaded, instead of 3 (when you click on the next button, you can see that the fourth image is already loaded).

I thought you get all images in the xml. So if they are all there, you could add them at once.


Updated tutorial link to learn link

Updated link so it points to the correct dbt Learn page instead of the tutorial page (which uses BigQuery and has the repo creation step that was already done).
Also, in this file it says to name your Github Repo dbt Learn - [initialsurname]. However, the Setting Up page says to name it dbt-learn-[initialsurname]

Thank you!


Cluster Explorer - Navigation Changes

https://github.com/rancher/dashboard/issues/4118

apps and marketplace to apps
add icons to top level

related removes folder and global icons for namespaced and non namespaced: https://github.com/rancher/dashboard/pull/4022

@nwmac I'm stumped trying to find how to pull for "starred" and "more resources" - any pointers?

not sure what you mean by "how to pull for" ?!

@lvuch We're close to being able to merge 2.6.2 issues now, if you update merge conflicts I'll take a look

Closing in favour of https://github.com/rancher/dashboard/pull/5161/files - this PR is out of date and has merge markers in the files.


Error: more than one library with package name 'com.google.android.gms'

Hello,
I'm using this plugin with ionic and when I try to build the Android app, I get the following error:
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':processDebugResources'.

Error: more than one library with package name 'com.google.android.gms'

My cordova version is 6.3.1 and my ionic version is 1.5.5
Any ideia why is this happening? why did it start to fail after I updated this plugin.
Thanks,
Cordeiro

I think you probably installed this plugin incorrect way.
Could you tell me what outputs the below command?
$> cordova plugin list

Here is the list it returned:
android.support.v4 21.0.1 "Android Support v4" com.google.playservices 23.0.0 "Google Play Services for Android" com.googlemaps.ios 2.0.1 "Google Maps SDK for iOS" com.ionic.keyboard 1.0.4 "Keyboard" com.teamnemitoff.phonedialer 0.3.0 "PhoneDialer" cordova-plugin-googlemaps 1.3.9 "phonegap-googlemaps-plugin" de.appplant.cordova.plugin.email-composer 0.8.2 "EmailComposer" nl.x-services.plugins.toast 2.0.4 "Toast" org.apache.cordova.device 0.3.0 "Device" org.apache.cordova.inappbrowser 0.6.0 "InAppBrowser" com.teamnemitoff.phonedialer 0.3.0 "PhoneDialer"

Remove com.google.playservices plugin, because this plugin install it automatically.
https://github.com/mapsplugin/cordova-plugin-googlemaps/blob/master/plugin.xml#L48


Language React Component refactor

This targets the refactor of lang.tsx
This PR depends on approval and merging of PR https://github.com/Microsoft/pxt/pull/5480

Please merge this PR into https://github.com/Microsoft/pxt/pull/5480

LGTM otherwise. It will be easier when looking at history to have those 2 changes together.


Is the discord available to the public?

I joined the discord server listed in the readme twice, and both times it had disappeared from my list. Is it an issue on my end or had I been kicked?
Thanks!

It should be available to the public, and I see your message. I'm sitting in the server so it shouldn't go away, and I'm not doing anything to actively kick. I just recently went public with the application, and I'm pretty sure you're the first to join on Discord. I'll look into it and see if I have something misconfigured.

It looks like I created a link that grants temporary membership. Unfortunately, I can't update the existing link. Here's a new one that grants permanent membership:
https://discord.gg/vFMV76brwU
I'll update the readme shortly to use the new link, and I'll roll it into the application on the next update.

The following commit should fix the issue: 7f624e1
If this works for you, I'll close the issue.

Seems to have worked. If any more issues come up I'll open a new issue. Thanks!


Added Russian translation

Hello,
This PR is related to issue #76   I have noticed that Russian translation is missing too in i18n so I have added the Russian translation, hope you like it & find it helpful 🙂

@AndrGab can you review it once :-)

thanks

Thanks 🙂


[FlatButton, FloatingActionButton, IconButton] Added target html attribute.

Fixes #5459.

[x] PR has tests / docs demo, and is linted.
[x] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[x] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

Added target properties on aforementioned components and written a test for FlatButton that validates the rendering with the target property.
I've also updated the docs page of FlatButtonExampleIcon with the link to the github page with a target="_blank". Not sure if that's desirable, see target _blank vulnerability. I'd like to hear your thoughts on this matter.
What I haven't done is implement the fifth feature of the target attribute:
framename - Opens the linked document in a named frame

Then again I'm not sure if this is desirable to be implemented aswell as it can be defined as an enum now :smiley:.

I'm not sure what went wrong with circleci...

I'm not sure what went wrong with circleci...

You can ignore it. It's not stable yet.
I have one more concern with this PR. There is an endless list of property that are going to be apply on the root element. Why documenting target and not the other ones?
What about instead documenting that any other properties are going to be applied on the root element? Actually, that should be true with all our components. It could be a generic note on all our docs pages. Keeping one test would be good too.

If you want to work on it, the propTypes documentation is generated here https://github.com/callemall/material-ui/blob/master/docs/src/app/components/PropTypeDescription.js.

Thanks for raising this issue :+1:.

Glad I could help :smiley:!


Default style when using the CLI

Uses the default Mermaid style when no explicit style is set. Fixes #196
Added a test to prevent regressions and additionally set the width of the default style to "auto".
Honestly I have no idea if setting the width to "auto" would cause any issues, the previous 1200px just generated a weirdly small image on my system.
Before:

After:

one comment, but otherwise :+1:—thanks so much for putting this together, and for writing up tests!

Great!
Thanks for the pull request and thanks for the review. I will merge and apply fardogs suggested changes.


localtunnel is offline.

I get this error when I try to execute the command -
tunnel server offline: connect ECONNREFUSED <IP_ADDRESS>:80, retry 1s

@chrisns I have upgraded the localtunnel server which should help with it being able to handle the load better.

Hey @chrisns, seems like it's down again. Can you please assist?

@yuliwna1 not my project. you want @defunctzombie.
to that end @defunctzombie are you ready yet to take some contributions towards that upgraded server?

@chrisns I suspect there are some bugs in the localtunnel server itself that contribute to downtime - tho larger servers mitigate this. Currently the server is a 1GB DigitalOcean instances.

@defunctzombie happy to have a dig in the code for bugs or at least a write a healthcheck to cycle the instance when it breaks if you can share the detail of how its deployed and what breaks with some logs etc

@chrisns Deployment is setup via the method described here: https://github.com/localtunnel/server#deploy - largely I [think] the issue stems from long lived connections (not closing tunnels) and possibly leading to OOM.
The logs themselves are not always helpful; the latest set contain a lot of:
[error]	server:60 null == true
AssertionError [ERR_ASSERTION]: null == true
    at Socket.socketOnData (_http_client.js:423:3)
    at Socket.emit (events.js:182:13)
    at Socket.EventEmitter.emit (domain.js:442:20)
    at Socket.Readable.read (_stream_readable.js:489:10)
    at Socket.socketCloseListener (_http_client.js:327:10)
    at Socket.emit (events.js:187:15)
    at Socket.EventEmitter.emit (domain.js:442:20)
    at TCP._handle.close [as _onclose] (net.js:598:12)
{ Error: write EPIPE
    at WriteWrap.afterWrite [as oncomplete] (net.js:835:14) errno: 'EPIPE', code: 'EPIPE', syscall: 'write' }

Which probably means there are more instances of error handling that need to be done in the server code.

to clarify, you're deploying with docker?
if so, if I make a PR to the server with a HEALTHCHECK in the dockerfile and you run it with --restart=always
at least then it'll cycle without manual intervention.
ideally if i can give you a docker-compose.yml with a 3.x schema which would allow multiple instances to run, load balancing and more managable memory limits
if you can let me know the docker engine version running, I can tailor to match.
thoughts?

if so, if I make a PR to the localtunnel/server with a HEALTHCHECK in the dockerfile and you run it with --restart=always
at least then it'll cycle without manual intervention.

If docker has this feature now then yes that sounds about right.

ideally if i can give you a docker-compose.yml with a 3.x schema which would allow multiple instances to run, load balancing and more managable memory limits

This won't "just work" due to how the backend is designed. There is no capability for horizontal scaling of the instances.

@defunctzombie https://github.com/localtunnel/server/pull/98 when you time


Checkbox - Text Alignment

I have been studying Tour De Jewel and this example that you posted on Checkboxes
https://royale.apache.org/selecting-options-from-a-group-of-jewel-checkbox-controls/
In both cases, Checkboxes display with text centered vertically on the button.  In my app, the text is being bottom aligned, but I would like it to be center aligned as in your projects.  I haven't made any checkbox changes to my CSS, and I don't see any in yours either.  How can I get my text to align as yours do?  It seems yours should be the default

May have figured out part of this.  I set a height on my checkbox components.  When I remove the height, the text is center aligned vertically.  When height is added, the text is bottom justified.  Not sure why this would make a difference.


Fix encoding problem with presencelight

Microsoft Reviewers: Open in CodeFlow

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


Microsoft.MachineLearningServices associated resource IDs in workspace are not preserving proper casing

REST API URL:
PUT /subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/test-resources/providers/Microsoft.MachineLearningServices/workspaces/test-Workspace?api-version=2019-11-01 HTTP/1.1

Body:
{
    "identity": {
        "type": "SystemAssigned"
    },
    "location": "westus",
    "properties": {
        "keyVault": "/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/test-resources/providers/Microsoft.KeyVault/vaults/testVault",
        "applicationInsights": "/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/test-resources/providers/microsoft.insights/components/test-AppInsights",
        "storageAccount": "/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/test-resources/providers/Microsoft.Storage/storageAccounts/teststorageaccount"
    },
    "sku": {
        "name": "Basic"
    },
    "tags": {}
}

After the creation complete, perform a GET request:
GET /subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/test-resources/providers/Microsoft.MachineLearningServices/workspaces/test-Workspace?api-version=2019-11-01 HTTP/1.1

Response:
{
    "id": "/subscriptions/00000000-0000-0000-0000-000000000000/resourceGroups/test-resources/providers/Microsoft.MachineLearningServices/workspaces/test-Workspace",
    "name": "test-workspace",
    "type": "Microsoft.MachineLearningServices/workspaces",
    "location": "westus",
    "tags": {},
    "etag": null,
    "properties": {
        "friendlyName": "",
        "description": "",
        "creationTime": "2020-02-06T06:54:56.5115492+00:00",
        "storageAccount": "/subscriptions/00000000-0000-0000-0000-000000000000/resourcegroups/test-resources/providers/microsoft.storage/storageaccounts/teststorageaccount",
        "containerRegistry": null,
        "keyVault": "/subscriptions/00000000-0000-0000-0000-000000000000/resourcegroups/test-resources/providers/microsoft.keyvault/vaults/testvault",
        "applicationInsights": "/subscriptions/00000000-0000-0000-0000-000000000000/resourcegroups/test-resources/providers/microsoft.insights/components/test-appinsights",
        "hbiWorkspace": false,
        "tenantId": "72f988bf-86f1-41af-91ab-2d7cd011db47",
        "workspaceId": "d4cfc4fe-1cd7-4881-bc8a-83ca8aaff7cb",
        "subscriptionState": null,
        "subscriptionStatusChangeTimeStampUtc": null,
        "discoveryUrl": "https://westus.experiments.azureml.net/discovery"
    },
    "identity": {
        "type": "SystemAssigned",
        "principalId": "539eb439-c316-49af-85af-e462e2ed0b90",
        "tenantId": "72f988bf-86f1-41af-91ab-2d7cd011db47"
    },
    "sku": {
        "name": "Basic",
        "tier": "Basic"
    }
}

The IDs in the response lost all their casing, for instance:

resourceGroups -> resourcegroup
testVault -> testvault
test-AppInsights -> test-appinsights

Hi @zhaomuzhi @drajput please have a look at this issue.

@ArcturusZhang
I think all resource id in Azure is case insensitive. So there seems not very important to preserve case.

@ArcturusZhang
I think all resource id in Azure is case insensitive. So there seems not very important to preserve case.

Usually, terraform will require the service to preserve cases.

@weidongxu-microsoft  what is a conclusion? should we assign to service team to address?

@ArcturusZhang Thank you for your feedback. This bug has been reported to our engineering team.

Hi @rastala @v-strudm-msft
This is now 15 months old - has there been some progress here? I want to bring machine learning resources up to speed with Terraform, but everything is a bit outdated. And Microsoft starts to also give guarantees that Terraform runs stably and I think keeping up-to-date and continuous versioning are a fundamental concern in software engineering. I do not like to see that things get postponed over months, as like this one will never have feature-completeness (one is always behind!) and never have stability.
FYI: @ArcturusZhang, @favoretti


update react router dom from v 5.2.0 to v 6.13.0, this will allow us …

…to use navigation hook

"react-router-dom version upgrade is incompatible with this version"

"react-router-dom version upgrade is incompatible with this version"

OK


Improve subcube WCS selection

SpectralCube.subcube allows spatial WCS coordinates to be given to slice out a subcube. However, the current code solves for the pixel coordinates of the upper and lower coordinates in each spatial dimension independently, and it uses 1D slices of WCS coords along the first pixel in each dimension. For images that cover a larger region on the sky (>~1 deg), this can lead to measurable deviations between the pixel slices for the subcube and the actual WCS limits requested.
I encountered this issue making subcubes for some of the spectral-cube tutorials. I created a subcube from one cube and used the cube.latitude_extrema to make an equivalent subcube from a separate cube. There ended up being a noticable ~10 arcsec (10 pixel) offset between the WCS coords in the second subcube from the first:
cube2_cutout = cube2.subcube(xlo=cube1_cutout.longitude_extrema[1],
                             xhi=cube1_cutout.longitude_extrema[0],
                             ylo=cube1_cutout.latitude_extrema[0],
                             yhi=cube1_cutout.latitude_extrema[1])

Here are the WCS coords for each spatial pixel in the two subcubes:

To fix this, I've changed SpectralCube.subcube and Projection.subimage to solve for the spatial coordinate of each corner of the subcube together whenever either x or y coordinates are given in WCS units (i.e., the minimum distance between the corner and the spatial WCS coordinates for the whole grid SpectralCube.spatial_coordinate_map)

The offset is now <1 pixel difference, which is what we expect for slicing instead of a full reprojection to match two cubes.
Because it relies on minimizing the spatial distance, this will slow down SpectralCube.subcube for large spatial sizes.

[ ] Clean up unused code
[ ] Add description to docs or docstring

Coverage decreased (-3.2%) to 84.935% when pulling f7e1ae4da91afd36e9636c6cd6e2336a8922326c on e-koch:improve_subcube_wcs_selection into df078f433b775c91dda3e7c7df8496c68244324d on radio-astro-tools:master.

@keflavich -- Here's a brief demo of where that offset was coming from.
Below is a demonstration of where the ~10 arcsec deviation in the above plots is coming from. The plot shows a pixel slice taken along the Dec axis from the cube.spatial_coordinate_map[0] at the centre of the map (5250 / 2):
dec, ra = hi_cube.spatial_coordinate_map

import matplotlib.pyplot as plt
plt.plot(dec[5250//2])
plt.xlabel("Pixel position along Dec slice")
plt.ylabel('Declination')
plt.grid(True)
plt.text(3500, 30.6598, f"Maximum deviation\n{np.ptp(dec[5250//2]).to(u.arcsec):.2f}")

The ~10 arcsec deviation is the worst case in this example because the previous cube.subcube solver used a 1D slice taken along the 0th pixel along each direction and I was slicing out something near to the centre of the cube. So essentially the min and max in the plot.
The deviation is the sky curvature as the M33 HI mosaic I'm creating the subcube from spans 1.5 deg in dec. So it is reasonably large. In this case, the maximum deviation is ~ 1 beam element. This effect will be far more subtle for smaller images but could produce ~few pixel offsets.
By solving for each corner of the subcube together in both spatial dimensions, the subcube slicing assumes that the location of each corner can use a flat-sky approximation. This should alleviate obvious mismatches in spatial dimensions for subcubes in most cases (I think), at least where slicing is appropriate versus reprojecting onto the same WCS grid of another data set.

Following @astrofrog's comments, I removed the full 2D spatial distance way of finding the corners. So we now have 3 cases:

Everything given in pixels. Easy.
Both coordinates for an upper or lower coords are given in WCS. Use WCS.world_to_array_index_values to solve
We have a mix of 1 pixel and 1 WCS for a corner. In this case, we can revert back to the 1D argmin but use the given pixel value to slice out the nearest row in the direction where the WCS limit was given.

This is much better than the 2D distance argmin I had implemented because we don't need the full spatial grid of coordinates in any case.
I've added explicit tests for all 3 cases.
@astrofrog I think this is ready if tests pass. I don't get why coveralls is showing such a large decrease; the lines it's reporting as uncovered are mostly unchanged in this PR.

Is coverage claiming that subcube_from_regions is getting un-covered?
https://coveralls.io/builds/34032114/source?filename=spectral_cube%2Fspectral_cube.py#L1952

Green! Merging!


Feature: use div instead of CSS marker with list marker

Using div could get fully control on list. For example, I want the marker's color and size be same as the first character. I achieve it in Draft.js's fork in my work. But I counter the problem that the text selection will also contains the div marker. Is it possible to do it with no pain in lexcial

I'm not sure what this issue is about - can you please provide more information or examples?

I believe that the point is that the list marker can be styled using css. if I have e.g.

Good day

so I want it to also have li:marker ... font-size:24px
but now it is not possible to achieve or I do not know how. Please advise. Well thank you


fix floating to int128 and negative to int128

fix https://github.com/kuzudb/kuzu/issues/2334

Return cast(-170141183460469231731687303715884105728, "int128") reports error Cast failed.<PHONE_NUMBER>60469231731687303715884105728 is not within INT128 range. while -170141183460469231731687303715884105728 should be the smallest INT128 value.

I think we only allow -170141183460469231731687303715884105727 to<PHONE_NUMBER>60469231731687303715884105727 to avoid possible overflows.

D select cast(-170141183460469231731687303715884105728 as  hugeint);
Error: Conversion Error: Type DOUBLE with value -1.7014118346046923e+38 can't be cast because the value is out of range for the destination type INT128

Looks like duckdb fails in this number. Not sure what is the limit.

D select cast(-170141183460469231731687303715884105728 as  hugeint);
Error: Conversion Error: Type DOUBLE with value -1.7014118346046923e+38 can't be cast because the value is out of range for the destination type INT128

Looks like duckdb fails with this number. Not sure what is the limit.

limit is -170141183460469231731687303715884105727 not -170141183460469231731687303715884105728. otherwise it will fail here for duckdb.

D select cast(-170141183460469231731687303715884105728 as  hugeint);
Error: Conversion Error: Type DOUBLE with value -1.7014118346046923e+38 can't be cast because the value is out of range for the destination type INT128

Looks like duckdb fails with this number. Not sure what is the limit.

limit is -170141183460469231731687303715884105727 not -170141183460469231731687303715884105728. otherwise it will fail here for duckdb.

Ah, okay. So maybe we need to modify the error message?<PHONE_NUMBER>60469231731687303715884105728 is not within INT128 range. or just leave it as it is.


Update to asp core 1.1

I'm getting errors when I try to update the sample project to use it on the recently released stable version of VS2017 with asp core 1.1.

What are the errors and what did you try to do?

I'm modifying the default project that VS2017 creates and I'm getting errors with this line in startup.cs:
services.TryAddSingleton<IUserClaimsPrincipalFactory, UserClaimsPrincipalFactory>();
It says that UserClaimsPrincipalFactory requires 2 arguments. It also can't found AddDefaultTokenProviders a couple of lines after.

I'm modifying the default project that VS2017 creates and I'm getting errors with this line in startup.cs:
services.TryAddSingleton<IUserClaimsPrincipalFactory, UserClaimsPrincipalFactory>();

That probably because you don't have this in: https://github.com/tugberkugurlu/AspNetCore.Identity.MongoDB/blob/c3c91067332852b63fdfad5fd3254dba6ec2800e/samples/IdentitySample.Mvc/Startup.cs#L171-L229

It also can't found AddDefaultTokenProviders a couple of lines after.

Same as before, it comes from https://github.com/tugberkugurlu/AspNetCore.Identity.MongoDB/blob/c3c91067332852b63fdfad5fd3254dba6ec2800e/samples/IdentitySample.Mvc/Startup.cs#L151-L159

Thanks! Sorry, I haven't seen that before. It worked perfectly.


Upgrade Ray version to 0.5.3.

@simon-mo updated. We have not built wheels for Python 3.7 yet.

Thanks!


Lydian #9 scale includes b9 instead of #9

To Reproduce
Steps to reproduce the behavior:
Tonal.Scale.notes("C lydian #9") => ["C", "Db", "E", "F#", "G", "A", "B"]
Expected behavior
Tonal.Scale.notes("C lydian #9") => ["C", "D#", "E", "F#", "G", "A", "B"]

Thanks for reporting!! 👍
Fixed in an upcoming release. Take a look to: https://github.com/danigb/tonal/tree/feat/v3


Having promise catch statement breaks the minification

var i = document.getElementById('videoElement'); console.log(i); // i.load(); var a = i.play(); if (a !== undefined) {     a.then(function() {       console.log('play worked.');     })     .catch(function(error){         console.error('ERROR!!')         console.error(error);     }); }
I am trying to minify the above code with slimit, getting error as "Unexpected token (CATCH, 'catch')"

Exact same issue as #52, #59, #103, #105 due to keywords not being recognised as identifiers.


Question regarding installing on rPi

As a followup to a thread found in https://github.com/bitly/nsq/issues/265 I'd like to ask how to install NSQ on an rPi? I've tried following the gpm install approach but it seems to fail with no error like it would ignore the Godeps file.
As far as I understand the instructions it should work the following way:
sudo apt-get install golang gpm
git clone git@github.com:bitly/nsq.git
cd nsq
gpm install

What happens after gpm install is
$ gpm install nsq
O0o.oops(): [daemon/startup.c(59)]: You should be root to run gpm!
$ sudo gpm install nsq
$

I'm not really sure what could be wrong?
@timkettering Do you maybe remember how you managed to set it up?

It seems that the gpm package in apt-get repositories is something different. After removing it I've reinstalled it and the nsq source with:
$ wget https://raw.githubusercontent.com/pote/gpm/v1.3.2/bin/gpm && chmod +x gpm && sudo mv gpm /usr/local/bin
...
2015-07-06 12:24:39 (4.76 MB/s) - `gpm' saved [3316/3316]

pi@raspberrypi /tmp $ cd nsq/
pi@raspberrypi /tmp/nsq $ sudo gpm install
>> GOPATH is not set
pi@raspberrypi /tmp/nsq $ export GOPATH=/home/pi/go
pi@raspberrypi /tmp/nsq $ gpm install
>> Getting package github.com/BurntSushi/toml
>> Getting package github.com/bitly/go-nsq
>> Getting package github.com/bitly/go-hostpool
>> Getting package github.com/bitly/go-simplejson
>> Getting package github.com/mreiferson/go-options
>> Getting package github.com/bmizerany/perks/quantile
>> Getting package github.com/mreiferson/go-snappystream
>> Getting package github.com/blang/semver
>> Getting package github.com/bitly/timer_metrics
>> Setting github.com/BurntSushi/toml to version 2dff11163ee667d51dcc066660925a92ce138deb
>> Setting github.com/bitly/go-hostpool to version 58b95b10d6ca26723a7f46017b348653b825a8d6
>> Setting github.com/bitly/go-nsq to version 4271cd1529a78175e327570894988cc2cb21228f
>> Setting github.com/bmizerany/perks/quantile to version 6cb9d9d729303ee2628580d9aec5db968da3a607
>> Setting github.com/bitly/go-simplejson to version fc395a5db941cf38922b1ccbc083640cd76fe4bc
>> Setting github.com/mreiferson/go-options to version 2cf7eb1fdd83e2bb3375fef6fdadb04c3ad564da
>> Setting github.com/bitly/timer_metrics to version afad1794bb13e2a094720aeb27c088aa64564895
>> Setting github.com/mreiferson/go-snappystream to version 028eae7ab5c4c9e2d1cb4c4ca1e53259bbe7e504
>> Setting github.com/blang/semver to version 9bf7bff48b0388cb75991e58c6df7d13e982f1f2
>> Building package github.com/BurntSushi/toml
>> Building package github.com/bitly/go-hostpool
>> Building package github.com/bitly/go-nsq
# github.com/mreiferson/go-snappystream
/home/pi/go/src/github.com/mreiferson/go-snappystream/writer.go:56: w.bw.ReadFrom undefined (type *bufio.Writer has no field or method ReadFrom)
pi@raspberrypi /tmp/nsq $ nsq
nsq
pi@raspberrypi /tmp/nsq $ nsq
nsq
pi@raspberrypi /tmp/nsq $ ls
CONTRIBUTING.md  Dockerfile  LICENSE   README.md  bench     build    dist.sh  internal  nsqadmin  nsqlookupd  test.sh
ChangeLog.md     Godeps      Makefile  apps       bench.sh  contrib  fmt.sh   nsq       nsqd      pynsq
pi@raspberrypi /tmp/nsq $ ./test.sh
building and starting apps/nsqlookupd/nsqlookupd --broadcast-address=<IP_ADDRESS>
  logging to /tmp/nsqlookupd.W4jbiAe
# github.com/mreiferson/go-snappystream
/home/pi/go/src/github.com/mreiferson/go-snappystream/writer.go:56: w.bw.ReadFrom undefined (type *bufio.Writer has no field or method ReadFrom)
pi@raspberrypi /tmp/nsq $

My go version is go1.0.2. I'll try to get a newer version installed

@1am you'll need at least go1.2 to compile from source - gonna close this but please re-open if you're still having issues!


Bring back ambiguity filter when we report an implicit not found error

This reverts one part of #20261. When we fail with both an ambiguity on one implicit argument and another error on another argument we prefer the other error. I added a comment why this is needed.
Fixes #20344

I added to a 3.5.0-RC2 milestone since this fixes a regression


Gear activity play button improvement

#1600 @llaske did the changes to #1506

Good idea to factorize UISwitch :+1:
Thanks.


LIS2DW12 Self Test

Adding ability to check individual axis to see which one is broken.

That is not a bad idea. I was wondering how it could be done to avoid API changes.


AzureCLI Variable escaping

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Arguably both
Enter Task Name: AzureCLIV1, AzureCLIV2
Environment

Server - Azure Pipelines (Releases, classic)

If using Azure Pipelines, provide the account name, team project name, build definition name/build number:
UNIUN, UNIUN, Uniun Website, releaseId=1071&environmentId=2584

Agent - Hosted:

If using Hosted agent, provide agent queue name: win-latest

Issue Description
When using the AzureCLIV1 or V2 task in a classic Release pipeline,
If using variables in the "inline script"
The variables aren't escaped according to the scripting environment being used.
See: https://github.com/microsoft/azure-pipelines-tasks/issues/13161 for more detail.
This means, suppose you want to use a variable in the inline script, and it has been entered into devops and it may contain a special character (special as deemed by the scripting language used by the task, e.g batch script, powershell etc) - you have to create a second copy of that variable in devops, that is the "escaped" version of the same variable, and use that version in the inline script of this task. Otherwise the script will break.
This means you have to maintain two copies of such variables, and manually keep them in sync. This is not ideal. Is there a better way to solve this?
Perhaps you could add a "Escape variables?" option to the task, so that when this is checked, any variables uses in the inline script, will be escaped based on the rules of the target scripting language?
Task logs
[Enable debug logging and please provide the zip file containing all the logs for a speedy resolution]
See: https://github.com/microsoft/azure-pipelines-tasks/issues/13161 for more detail.
Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Error logs
See: https://github.com/microsoft/azure-pipelines-tasks/issues/13161 for more detail.

For a concrete example of the particular difficulty I am getting into with variables and special chars:
https://github.com/Azure/azure-cli/issues/14693

@dazinator we haven't heard this issue from multiple folks. I would like to hear more feedback before trying to fix this issue. I am keeping the issue open for sometime for more folks to report this.


Apple badge value setting

@Dalton-Vo
I think this and this comment should answer your question.
Based on it, I am going ahead and closing this issue. You may also ask this question on support channels like StackOverflow.

Can you help me answer my question : https://github.com/firebase/flutterfire/issues/10345#issuecomment-1410357128


Best practice to use read-only routing in always on cluster

If I have a always on sql cluster that use read-only routing. Does it exists any best practice how to configure this together with EFCore?
https://msdn.microsoft.com/en-us/library/hh213417.aspx
https://msdn.microsoft.com/en-us/library/hh710054.aspx?f=255&MSPPError=-2147217396
I can always create two different instances of the db-context with different connectionstrings but it should be good if it was able to have inbuilt logic for this scenarios to automatic select the correct db-connection depend on the operation.

Possibly related to #6297

https://github.com/aspnet/EntityFramework/issues/6525 might be useful for this

Right, this is the same as my need (#6297). Always hope that EF can choose to use the connection information that is written to the database connection information or read the connection information.

It seems like #6525 could be the basis for this, so rather than keeping this issue open we will track any work on that issue, referencing this scenario as needed.


Plasma - Rendering is incorrect for many settings in "System Settings" panel

Bugs
It seems that the system settings are not displaying correctly. I will present here the differences between ArchLinux and NixOS with a vanilla theme (Breeze light).
At a first glance, the main components affected seem to be QML UIs.
ArchLinux and NixOS system informations
NixOS
Operating System: NixOS 20.03pre212208.8130f3c1c2b
KDE Plasma Version: 5.17.5
KDE Frameworks Version: 5.66.0
Qt Version: 5.12.6
Kernel Version: 5.5.2
OS Type: 64-bit
Processors: 8 × AMD Ryzen 7 PRO 3700U w/ Radeon Vega Mobile Gfx
Memory: 21,5 GiB of RAM

ArchLinux
Operating System: Arch Linux
KDE Plasma Version: 5.17.5
KDE Frameworks Version: 5.67.0
Qt Version: 5.14.1
Kernel Version: 5.5.2-arch1-1
OS Type: 64-bit
Processors: 8 × Intel® Core™ i5-8250U CPU @ 1.60GHz
Memory: 15,5 Gio

Global theme duplicated with duplicates not working correctly

The themes are presented multiple times (Breeze and Breeze Dark) but only one of each is working properly.
The UI to manage displays is broken
The UI seems broken, when we "force scroll" we can see some hidden content appearing from overscrolling. The UI to select screen placement is also not working properly, as shown here :
ArchLinux :

NixOS :

Some UI elements have lost their theme, making them difficult to use
Some elements are not drawn correctly and are unusable because of this. For instance, see following screenshots:
ArchLinux :

NixOS :

Those issues degrade the experience a lot when using the Plasma Desktop Environement under NixOS. Let me know if I can provide any more information or help to track down those issues.
To Reproduce
Steps to reproduce the behavior:

Enable plasma with the following default configuration :

services.xserver = {
  displayManager.sddm.enable = true;
  desktopManager.plasma5.enable = true;
}

Expected behavior
It should display as closely as possible as the reference screenshot for ArchLinux.
Metadata

system: "x86_64-linux"
host os: Linux 5.5.2, NixOS, 20.03pre212208.8130f3c1c2b (Markhor)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.2
channels(root): "nixos-20.03pre212208.8130f3c1c2b"
channels(ely): "home-manager"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:
cc @worldofpeace @ttuegel

Duplicate of #79713


xrdp remote access

Thank you for the image!
After flashing and installing xrdp, I connect to my jetson nano B01 and the screen freezes as shown below.
Do you have some hints that can help with this problem please?

@kyrie2to11,
I'm sorry to hear about your issue. More over, because I think there is not a quick fix.
I've googled 'xrdp jetson nano crash' and got 38.300 hits.
Looking at the first ones, it seems a common issue with a variety of solutions.
In your cast, I would suggest to use Teamviewer. see also https://github.com/Qengineering/Jetson-Nano-Ubuntu-20-image/issues/43.

Thank you for your reply. I will attempt to use Teamviewer.
Additionally, I have conducted the xrdp connection testing with Jetpack 4.6.1 and it functions flawlessly with almost no delay.

@kyrie2to11 so for the following did the trick:
sudo apt install xserver-xorg-video-dummy
sudo nano /usr/share/X11/xorg.conf.d/xorg.conf
Input the following in the file:
Section "Device"
Identifier "Configured Video Device"
Driver "dummy"
EndSection

Section "Monitor"
Identifier "Configured Monitor"
HorizSync 31.5-48.5
VertRefresh 50-70
EndSection

Section "Screen"
Identifier "Default Screen"
Monitor "Configured Monitor"
Device "Configured Video Device"
DefaultDepth 24
SubSection "Display"
Depth 24
Modes "1920x1080"
EndSubSection
EndSection

After that reboot.

i found some other ways:
sudo nano /etc/xrdp/startwm.sh
in the file comment out the last two lines:
#test -x /etc/X11/Xsession && exec /etc/X11/Xsession #exec /bin/sh /etc/X11/Xsession
and add one of the following at the end:
1.  gnome-session 2.  startlxde 3.  startxfce4 you need to install this first if you want to use it: sudo apt install xfce4 xfce4-terminal -y
after that you need to restart xrdp:
sudo systemctl restart xrdp.service


Update to latest bgfx and bx.

Update submodules.
Drop ib-compress and forsyth-too 3rdparty libraries.
Add meshoptimizer to geometryc.

Hi, sorry for the delay in pulling these changes in. I like to run the changes myself. I will try to do that tonight.


Doc

Codecov Report

Merging #24 (274c997) into master (49da1d5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #24   +/-   ##
=======================================
  Coverage   99.37%   99.37%
=======================================
  Files           6        6
  Lines         160      160
=======================================
  Hits          159      159
  Misses          1        1

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 49da1d5...274c997. Read the comment docs.


[Bug] [UI Next][V1.0.0-Alpha] Add date-fns-tz dependent

Purpose of the pull request
refer main issue: #8530

PTAL @songjianet


Latest Ubuntu, Brave

I can log in to gab.com fine.
Logging into the plugin Dissenter however gives me:
"Client authentication failed due to unknown client, no client authentication included, or unsupported authentication method."

Me too, windows Dissenter browser, and firefox addon.

I'm also getting this on Pop!_OS 19.04, all browsers.
I'm also connecting from Canada.

Same problem when connecting through Europe.

My answer from another issue here:

Probably the backend is only allowing browsers that pass certain headers to complete authentication. Brave/others are probably detected as unsupported browsers. My guess is if you tried to do the same with Chromium you would get the same error. Have to have a dev go in and whitelist those headers being passed (just a hunch but I think it's right)

EU/DE same still


Kolla pike 5.0.1

What does this PR do?
This PR will add the functionality to checkout kolla package through tag.
Do you have any concerns with this PR?
No
How can the reviewer verify this PR?
A Reviewer can verify this PR by deploying OpenStack using kolla tag paramter mentioed in deployment.yaml file.
Any background context you want to provide?
NA
Screenshots or logs (if appropriate)
NA
Questions:

Have you connected this PR to the issue it resolves?
NA
Does the documentation need an update?
Yes
Does this add new Python dependencies?
No
Have you added unit or functional tests for this PR?
OpenStack deployment needs to be performed.

The ^M clean up is good to go. I'd say merge away at this point. Thanks Chirag Gupta.


Added commented UDF for Tutorial

Purpose
Added commented out code for the tutorial at the end of the file. Plan is to ask people to comment/delete the existing code and uncomment this one.
Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] New sample or new feature within a sample
[ ] Code style update (formatting, naming)
[X] Refactoring (no functional changes)
[ ] Documentation content changes
[ ] Other... Please describe:

Test By
NA
Other Information

Closing this to add this block as a separate file.


Console often displays setState errors

Warning: setState(...): Can only update a mounted or mounting component. This usually means you called setState() on an unmounted component. This is a no-op. Please check the code for the Preload component.

Looks like promise needs to be cancelled in unMount;
https://facebook.github.io/react/blog/2015/12/16/ismounted-antipattern.html

@staff0rd I thought about adding it, but after some investigation I found, that it is just a warning in development mode, and in production this setState just ignored. makeCancellable approach is not real cancellation of HTTP request, it is just skipping setState after component unmounted. Eventually, adding this makeCancellable will increase code base size and complexity without any advantages.
What I decided, is some time later rewrite all fetch(url) statements to some node-based library to fetch data with an ability to cancel requests, not just skip setState.

These errors are actually preventing my plugin from working. They occur everytime I try to use an external component for getPreview for the display function. What exactly is causing them, and how do I get around it? It doesn't have any issues if I use an inline element.

@BrainMaestro can you share your code example?

getPreview Issue
It works normally on the master branch. Thanks for checking


Use Version::ForEachOverlapping to implement Version::Get()

This code has been tested with existing tests and no new tests are written because it justs implements  Version:: Get() with Version:: ForEachOverlapping() which has been fully tested.

I signed it!

I'm really confused, how come tests like memenv_test failed in this build while this commit has nothing to do with memenv?

@googlebot I fixed it.

@ghemawat sorry for delaying for so long!

THANKS !!!!! This means a lot to me!!!


Random Picture View

Is it possible to make a config true false for random picture.
so if i have a lot of pictures i allways see them in time where is send. the most time new pictures or old pictures.
a nice feature is to do this randomly.

There is no such feature at the moment, but feel free to implement it.

Now that you mention it, the static order is annoying me, too. (thank you :imp: )
Fortunately, it's easy to change. I have created a pull request which introduces this feature.


Add yapf job to lint workflow

This PR adds a job to the lint workflow that checks if all Python files adhere to Google style.
The relevant files for this PR are:

setup.py
.github/workflows/lint.yml

Closed in favor of black for formatting.


Allow software types to be registered and declared

First iteration of registering and declaring software types.  This allows Project plugins to declare the software types they expose through a new @SoftwareType annotation that declares the block name and public model type.  These types are then exposed in the declarative DSL such that when a block matches a software type, the plugin is automatically applied and the public model type can be configured.
Note that the registration of software type providing plugins is not addressed in this work.  Settings plugins currently must register these via an internal service.  In follow up work, we'll provide a mechanism for doing this registration in a static way (i.e. via annotations or some other mechanism).
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test this

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build


[Suggestion] Latest transactions suggestions

-Show negative/outgoing transactions red and positive/incoming as green.
Since you already using arrows at the left, make them red or green. And/or you could also make the amount red or green. Btw I don't find the up/down arrows very intuitive. Up arrow communicates to me that my balance would go up. Also my first instinct reaction was that it would give me a dropdown with extra detailed transaction info. Better is to remove the arrow and just color the amount red or green.
-Also show a little description if available. Check out the financial app of your local bank and take all the good stuff. They have years of a head start (don't get me wrong Firefly already looks great).

Other considerations regarding this issue see https://github.com/iotaledger/firefly/issues/664

Update : I noticed the icon before transaction has a third variant: internal transaction.
So I guess we keep the icons but change the up and down into a box (representing a wallet) with a red outgoing arrow (send funds) and a box with a green incoming arrow (incoming funds).

Perhaps for internal transactions it would be better not to show + or - at all (and then leave it white).
In the "Last Transactions" overview, only the transaction for the account that leads the ranking is displayed.
Or both entries should be displayed (+ and -)


using an open source version directly for pbkdf2

Describe your question
Directly use the open source version golang.org/x/crypto/pbkdf2 v0.5.0 at go.mod,
delete open_src/golang.org/x/crypto/pbkdf2.
remove the dir:

Please verify whether the function is compatible after modification.
ref
https://github.com/golang/crypto/blob/master/pbkdf2/pbkdf2.go

I will fix it


[AutoRelease] t2-containerservice-2022-09-13-84137(Do not merge)

https://github.com/Azure/sdk-release-request/issues/3158
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=982

issue link:https://github.com/Azure/sdk-release-request/issues/3158