NodePath export hint

export(NodePath, Button) var my_node_path
Makes only Button (and its subclasses) selectable in the 'Select a Node' dialog window in the editor

this also works for global classes
#Inventory.gd
extends Node
class_name Inventory

#...

#Player.gd
#...

export(NodePath, Inventory) var inventory_path

#...

I've been using this patch for ages now and it works perfectly.
Is there any reason why this is not getting merged?

It needs review from @vnen or other contributors familiar with the GDScript 3.x implementation.
As there's a lot of backlog and work on 4.0, it's difficult to give priority to all PRs for review. But you commenting that you've been using this in production is definitely valuable, tests help a lot with getting a qualified review to merge a new feature.

I added the tag to be salvageable because people use this in their games. Hopefully, someone can be a hero and update this to Godot Engine 4.


Update LiveTokenManager.js

Now you can log in just by jumping to the page

How do you know this works?

How do you know this works?
I'm Japanese and actually own SWITCH.I found out about this because when I read the QR code, this query came up.

Ok, when I go to
https://microsoft.com/link?otc=T3ZJR8RD
I get redirected to a blank page.
Maybe this only works in some areas?

Ok, when I go to
https://microsoft.com/link?otc=T3ZJR8RD
I get redirected to a blank page.
Maybe this only works in some areas?

Same for me. Im also getting redirected to a blank page.

Ok, when I go to
https://microsoft.com/link?otc=T3ZJR8RD
I get redirected to a blank page.
Maybe this only works in some areas?

This works perfectly fine for me, but I am using a Azure Application App, but I don't know if that makes any difference or not.

The LiveTokenManager.js that you're modifying is for non-Azure apps (only works for official Minecraft client IDs). If you use an Azure app, MsaAuthFlow (using official msal lib) is used instead. The message for MsaAuthFlow is provided by the server, and we don't modify or manually specify it like in LiveTokenManager.
Anyway when I scan your QR code the input box is still empty, though I do get the "?otc" part in the URL. There might be a geographic difference in the login page code.

@extremeheat This only works for valid codes, the codes shared by the others here have likely been used or expired hence why they don't work and redirect to the main page. I've just tested on a fresh code gen and is working fine. Very useful to be honest!

Yeah, I see. It doesn't work on my phone but does work seem to work on my computer. But only for microsoft.com/link, not at https://microsoft.com/devicelogin endpoint (which LiveTokenManager doesn't use). It's also not documented (I don't see anything in microsoft docs), so I'd suggest adding this link at the end of the current prompt, explicitly using https://microsoft.com/link?otc=....
Perhaps like To sign in, use a web browser to open the page $url and use the code $code or visit http://microsoft.com/link?okx=$code

https://github.com/PrismarineJS/prismarine-auth/assets/149259428/44d48aa6-384b-41d9-8fff-8f1728452f14

Yes, I understand. I was just commenting that we should have both the code separate and embedded into the URL. This way we don't have problems if people go to the link and it's not prefilled and it's not obvious the query string parameter is the code to type in. I've updated the text in the PR to do this.

@extremeheat
I would like to see this feature added to prismarine-auth!

Sorry for the delay. LGTM, thanks for the investigation on this :)


Add support aws_sns_topic_subscription

LGTM!
  aws_sns_topic_subscription:
    - arn:aws:sns:us-east-1:141177182257:demo-topic:44ec61b0-8e8c-4f40-b354-f706c6c484a8


enable/disable VRR / instant game response on the TV

Hi there,
I have been using this wonderful tool in combination with Kodi to switch from expert2 to/from game to enable TruMotion when I am not gaming on a LG C9.
Now with a different NVidia card I am also running 120Hz and G-SYnc/VRR/Instant Game Response. However now it seems with instant game response on I cannot use TruMotion anymore as all options are greyed out. I am also unable to DISABLE instant game response on the Windows side of things. Neither disabling G-Sync, nor using 60Hz standard Resolution works to disable VRR. The only thing that helps is when I disable Instant Game Response within the LG Picture Menu and TURN OFF Instant Game Response on the respective HDMI port.
This makes things pretty ugly. Does anyone know of a way to disable Instant Game Response programmatically ?
THANKS!

Did some more digging... I have found this:
https://github.com/chros73/bscpylgtv/blob/master/bscpylgtv/webos_client.py
...
"gameOptimizationHDMI2": "on",

Using ColorControl.exe I was able to find out this is exactly the setting I need to toggle to turn off/on VRR. How would I go on to toggle this setting in aiopylgtv?
I guess I need to do a luna_request? But how?
 uri = "com.webos.settingsservice/setSystemSettings"

params = {"category": "other", "settings": {"gameOptimizationHDMI2": "off"}}

return await self.luna_request(uri, params)

works! Heureka!


Adding movement processing to L.Edit.Poly(with test suit)

I added a process of moving to L.Edit.Poly.
https://tknpow22.github.io/Leaflet.draw/docs/examples/add-poly-move-test.html

src/edit/handler/Edit.Poly.js
add movement processing to L.Edit.Poly.

src/leaflet.draw.css
add background-color style to .leaflet-edit-move.

spec/suites/EditSpec.js
In L.Edit.Poly test, changed L.Polyline constructor parameters.

Please close this request if you do not need it.
I'm a beginner of github and not good at English. I'm very sorry if I'm mistaken.

Feature

Adds a square to grab and move any polygon

Suggestion
All we need here is a way to turn this feature on or off (default off). You up for that @tknpow22 ?


[BUG]: Bookmark Icon not showing in Vlad's fork

Is there an existing issue for this?

[X] I have searched the existing issues

Bug Description
The bookmark icon that's supposed to display Better Styles isn't showing in Vladmandic's fork; https://github.com/vladmandic/automatic
Note that there was a closed issue (https://github.com/eideehi/sd-webui-better-styles/issues/3) that suggested I add const localization = {}; to the beginning of betterStyles.js, I have done so, but to no effect.
Steps to Reproduce
Installed Better Styles via manual install.
Confirmed installation by visible Better Styles side-tab in Settings.
No bookmark icon displayed near the Generate button.
Screenshot (optional)
Confirmed install from console;

Confirmed install in Settings;

Missing icon;

Note: Egg on face time;
Placing my extensions behind a symlink caused a lot of issues. Those issues vanished when I removed the symlink and copied the extension back to the main folder.
Closing.


Catch all route

🐛 Bug Report
When there's a page /[...all].tsx trying to getPage on /sub/[route].tsx has unwanted results, because [...all].tsx is selected on this heuristic, from getPageObject/getRouteInfo.js
  const matchingRouteInfo = pagePaths
        .map((pagePath, index) => {
        const result = pathname.match(pagePathRegexes[index]);
        if (result) {
            return makeRouteInfo_1.makeRouteInfo({
                route,
                pagePath,
                routeRegexCaptureGroups: result.groups,
            });
        }
    })
        .filter((result) => Boolean(result))
        .sort((a, b) => a.paramsNumber - b.paramsNumber);
    // Return the result with less page params
    return matchingRouteInfo[0];

Because paramsNumber for both routes equals 1. The ordering I believe comes from fastGlob, which returns [. above all else,  see getPageObject/getPagePaths.js, a quick check through their package documentation doesn't seem to offer a way to rank paths by specificity.
To Reproduce

On a next js project, create pages/[...all].tsx and pages/sub/[route].tsx.

Try to renderPage with route on /sub/route0.

[...all].tsx is rendered.

Expected behavior
Catch all routes should have less preference.

I've created a small reproduction repo here: https://github.com/osadi/next-page-tester


Do not specify "from charset"

Let's leave it auto? Otherwise, it can result in:
Warning: mb_convert_encoding(): Illegal character encoding specified in CharsetStreamFilter.php on line 47

@zbateson There could be another issue with this. I am doing more checks.

Thanks @tivnet -- I'll wait.

@zbateson Well, it's more complicated than I expected.
I am failing, for example on
Content-Type: text/plain; charset="iso-8859-8-i"
Content-Transfer-Encoding: base64

(cannot pass this to mb_)
and in some other cases....
So, I just commented out the conversion in CharsetStemFilter, leaving the stub $converted = $bucket->data; - and trying to play with the stream in my own code.

@zbateson Zaahid, is there a way to get the raw body part, without accessing the stream? So far, my code looks like this (and I do not mb_convert in the Filter)... Kind of ugly.
Or, is there an easy way to disable the charset filter?
Thank you!
		/** @var ZBateson\MailMimeParser\Header\ParameterHeader $headerContentType */
		$headerContentType = null;

		$hMailInfo['body'] = '';

		if ( $res = $message->getHtmlStream() ) {
			$hMailInfo['body'] = strip_tags( stream_get_contents( $res ) );
			$headerContentType = $message->getHtmlPart()->getHeader( 'content-type' );
		} elseif ( $res = $message->getTextStream() ) {
			$hMailInfo['body'] = (string) stream_get_contents( $res );
			$headerContentType = $message->getTextPart()->getHeader( 'content-type' );
		}
		$hMailInfo['body'] = trim( $hMailInfo['body'] );

		// Convert only if it's UTF
		if ( 'UTF-8' === strtoupper( $headerContentType->getValueFor( 'charset' ) ) ) {
			$hMailInfo['body'] = mb_convert_encoding( $hMailInfo['body'], $targetCharset, 'UTF-8' );
		}

Hi @tivnet
Thanks for looking into this.  With regards to the best way to disable the CharsetStreamFilter, I suppose the best way at the moment would be to extend SimpleDi::registerStreamExtension so it doesn't setup the filter, then extend MailMimeParser to use your SimpleDi.  It too isn't pretty but at least it would allow you to use MMP as a composer dependency without editing its code directly if that's the goal.
Just to make sure I understand the issue, you have a message encoded as ISO-8859-8-I, calling mb_convert_encoding produces a warning.  Does it actually convert the message successfully though?
Perhaps instead the better solution would be to only call mb_convert_encoding if the charset specified in the content type is one returned by mb_list_encodings or if it's blank.  Unfortunately there may be a couple 'supported' that aren't listed looking at the comments in http://php.net/manual/en/mbstring.supported-encodings.php but it may be a better solution.
I'll close this PR, please let me know if you'll look into this, otherwise I'll get around to it in a few days :)

No, it does not convert the message.
I'd say you are right about calling the encoding only if charset is one of the supported, and maybe also have the target encoding as a parameter. Not always UTF.
Not sure about "if it's blank".
Anyway, let's consider this PR premature. I'll remove my patch and will wait for your next steps.
Thank you!


Add an integration test for systemd in a container

We regressed really badly here. Add a test so we don't do it again.

I fully expect this is failing right now. Will merge once master is fixed.

LGTM

LGTM once tests are happy

LGTM, PR to fix the issue here: https://github.com/containers/libpod/pull/3731

Rebased. Hopefully will go green now.

@mheon Could you rebase this and see if we can get it merged.


Increasingly large PR test

While https://github.com/bojanpotocnik/github-workflow-playground/pull/5 tested what happens when the patch file is too large by far, this PR aim to check what exactly is that limit and if patches can be safely retrieved and compared up to that limit.
Note that while opening the PR, GitHub displayed this message:

Patch length: before = 112279963, after = 112279963
Keeping the reviews because the latest push was only a rebase on main

Use these URLs for manual investigation (append '.patch' to download raw patch files):
Before: https://github.com/bojanpotocnik/github-workflow-playground/compare/main...f3bf9340a76d081a72b36f956ac4be786e500c01
After:  https://github.com/bojanpotocnik/github-workflow-playground/compare/main...7b90ab0d441cdf44e4499434d1235070ebb8dda5
Patch length: before = 124933653, after = 124930771
Dismissing reviews<PHONE_NUMBER>
Dismissing review<PHONE_NUMBER>

Patch length: before = 124930771, after = 124950737

Patch length: before = 124950737, after = 125003483

Patch length: before = 125087585, after = 125135043

So, #5 and #6 show that there are 3 cases:

Patch is short enough for comparison to succeed
Patch is very huge but still shorter than the hard limit - either API request or allocation fails, resulting in Server Error.
We are talking about more than 4 million changes, 16000 files and 125 million characters.
Patch is larger than the hard limit and API results in The patch could not be processed because too many files changed.


Document usage

Somebody should take #4 and continue the job, adding instructions for starting OVOS-Core after a pip or a git install. I am not familiar enough with the changes to finish it myself, and we've gotten questions ("I installed it now what?")
Generally speaking, the practice of making changes and then putting up a TODO for somebody to document them needs to stop. It's hard enough to document your own code after a break. Trying to document somebody else's code is a nightmare. If somebody is less than confident in their English, write your docs and then ping me or Daniel to clean them up.

@ChanceNCounter Is this still valid? Quite old, not sure if it's been addressed (the context is old and unclear to me)


Error with mongoose

ERROR:
Unhandled promise rejection: MongooseError: Operation developers.findOne() buffering timed out after 10000ms
at Timeout. (/home/runner/TastyShack-bot/node_modules/mongoose/lib/drivers/node-mongodb-native/collection.js:158:23)
at listOnTimeout (node:internal/timers:559:17)
at processTimers (node:internal/timers:502:7)
Also wanted to go on the discord server but the invite is invalid..

Or
🚨・Unhandled promise rejection
Error
MongooseError: Operation developers.findOne() buffering timed out after 10000ms
Stack error
MongooseError: Operation developers.findOne() buffering timed out after 10000ms
at Timeout. (/home/runner/TastyShack-bot/node_modules/mongoose/lib/drivers/node-mongodb-native/collection.js:158:23)
at listOnTimeout (node:internal/timers:559:17)
at processTimers (node:internal/timers:502:7)

Invalid: issue because of wrong configuration


Implement missing Api-Endpoints/methods

The following Api-Endpoints/methods are not yet implemented in the C#-SDK and should be added:
/api/v1/products/{id}:Delete
/api/v1/products:Post
/api/v1/products/category:Get
/api/v1/customers/{id}/addresses:Post
/api/v1/customers/addresses/{id}:Get
/api/v1/customers/addresses/{id}:Put
/api/v1/customers/addresses/{id}:Patch
/api/v1/customer-addresses:Get
/api/v1/customer-addresses:Post
/api/v1/customer-addresses/{id}:Get
/api/v1/customer-addresses/{id}:Put
/api/v1/enums/paymenttypes:Get
/api/v1/enums/shippingcarriers:Get
/api/v1/enums/shipmenttypes:Get
/api/v1/enums/orderstates:Get
/api/v1/layouts:Get
/api/v1/orders/{id}/parse-placeholders:Post
/api/v1/shipment/shipwithlabel:Post
/api/v1/shipment/shipments:Get

Hey Benjamin,
i have just created a pull request as draft. If you have time, feel free to take a look. The endpoint and integration test were added. There are still a few things to do.  I hope it is helpful!


Cannot read property 'ringactionState' of undefined

After getting a replacement opener from Nuki, the new opener seems to throw an exception:
"Nuki_Bridge_XXXXX: warning: heartbeat error: TypeError: Cannot read property 'ringactionState' of undefined"
I already enabled debug- and web mode on the opener, updated node.js, renewed the subscription and re-installed the whole plugin. However, the issue persists. Newest firmwares are also installed. Any help would be appreciated!

After that, did you restart Homebridge?

If the problem persists, please turn on DEBUG mode in Homebridge, set the Log Level on the Nuki bridge accessory to 2 (using Eve or another HomeKit app), and capture and attach the Homebridge log file.

After that, did you restart Homebridge?

If the problem persists, please turn on DEBUG mode in Homebridge, set the Log Level on the Nuki bridge accessory to 2 (using Eve or another HomeKit app), and capture and attach the Homebridge log file.

Hi,
Thank you for the quick reply. I found the issue accidently by looking at the eve app. It looks like the bridge still thought the old opener exists, even though I deleted it properly. After removing again from the bridge and then power cycling everything, the old opener was gone and with it the error.

Hi,
Thank you for the quick reply. I found the issue accidently by looking at the eve app. It looks like the bridge still thought the old opener exists, even though I deleted it properly. After removing again from the bridge and then power cycling everything, the old opener was gone and with it the error.


30 second page loads on Azure

Hello,
I'm running version 6.0.0 in an Azure webapp with ~ 4 gigs of data in 100 repositories.  Unfortunately Azure page load times are consistent at 25-30 seconds.  I've tested the same code with the same data on our local web server and page load times at 4 seconds.
I'm not finding anything useful in the error log.  Here are the latest entries.
SnapshotHelper::RestoreSnapshotInternal SUCCESS - File.Copy
SnapshotHelper::RestoreSnapshotInternal SUCCESS - process
Native library pre-loader is trying to load native SQLite library "D:\home\site\wwwroot\Bonobo\bin\x86\SQLite.Interop.dll"...
SnapshotHelper::TakeSnapshotTimerCallback
SnapshotHelper::TakeSnapshotInternal - no new files in CodeGen
Bonobo code is the only application in this webapp.
This has been going on for a couple months now.  Even pushes and pulls are taking a brutally long time.
Please advise.
Thanks,
Kirk

Is SnapshotHelper an Azure thing?  I don't think that's part of Bonobo

Hi Will,
It appears that the SnapshotHelper is an Azure thing.
http://stackoverflow.com/questions/32253963/what-is-snapshothelper-on-windows-azure
I'm not seeing those entries on my local installation.
Kirk

Will,
I'm trying to tack down whether or not the SnapshotHelper is causing this issue.  In the mean time I have attached my error log file.  Perhaps you can find something in there that can point us in the right direction.
Thanks,
Kirk
Bonobo.Git.Server.Errors.pdf

@KY4U  I don't think there's anything useful in that log, and it's hard to tell if it's the result of an hour's running or a year's anyway.
I think it might be useful to look at which resources are slow to load - perhaps use something like the network view in Chrome developer tools and see if every item is coming in slowly or just a specific one.

Can you view the /home/diagnostics in Bonobo page (I think we made this local-only, so you probably can't)?
If you're not logged in, does the log-in page appear promptly?
Can you load a static file quickly?  (As suggested further up this thread)
What type of authentication/authorisation setup do you have?

I apologize, I did not see the request to load the static file.  That png image loaded in 537ms.
When not logged in the login page https://admin.coastalscience.noaa.gov/git loads in ~ 15 seconds.  Most of the time it takes ~ 30 seconds.  Once logged in (forms based authentication) the Repository listing page loads in 26 seconds.  See attached image.

If necessary I can provide you with a username and pw for access to the site.

I will try to create a testcase with many big repos and see where we are taking long.
Can you tell us how many user you have and what kind of authentication you are using?

We have 40 users.  Authentication is forms based using the Internal MembershipService.
Thanks again for looking into this.

On every occasion I just loaded the login page, it took just over 10 seconds, which makes me wonder if there's something timing out there each time.
If you change the repo directory to point to somewhere with no repos, and rename/remove the user database, so that there are no users, does the delay change?

OK.  I've got a sandbox location where I have removed all the repositories.  Now the repository listing page show all those repos as "Missing".  Page load times are still the same.  What is the quickest way to remove the users.  I can copy down the db and remove users from the table if that's what you suggest.

Just delete (or move or rename) the bonobo.git.server.db file in App_Data - when the app restarts it will create a new, empty copy.

Starting with no repositories, and removing the db, I reloaded the sandbox home page  in 5 seconds.

@KY4U  Yes, that's much faster here (much less than 5 seconds)
Obviously that's a different webapp instance to the one we were testing earlier.  Are you able to clearly see that "with DB = 10 seconds, without DB = almost instant"?
How big was the .DB file?
What size Azure webapp are you running?

This is a different web app. I had copied the code, db, and repos from the original webapp.  The sandbox response was the same as the original.  Once I removed the db the load was nearly instant.  Removing the repos did not make a difference in page load times.
DB size is 200kb.  Azure webapp is Standard Small in the Sandbox, Premium 1: Medium in the original webapp.

I came across some table differences between the newly created database and my current database.  See attached images.

@willdean and @RedX2501 I wanted to let you know what I have discovered.
In my sandbox webapp, I pointed my repository setting path to an empty folder.  I removed the database and reloaded the login page.  Load time was very quick < 1 second.  Now we have just one user and no repositories.  I then changed the repository setting path to my existing repositories (~100).  Settings saved in < 1 second.  Clicked Repositories, which showed no repositories at this time.  Page load < 1 second.  Clicked Rescan directory which took 27 seconds.  Now all my repositories are listed with "Discovered in file system" as the description.  At this point if I do a search, or click on Settings, Users, Teams, or any other link, page load times are 25-30 seconds.
Please advise.

@KY4U It just sounds like there's something about Sqlite (or the way we're using it) and Azure which doesn't get on well.  Bonobo is not very highly-tuned with regard to database efficiency, and perhaps it's doing something which interacts badly with Azure's rather unusual I/O arrangements.
  Meaning, once the repo data is in the database, does Bonobo check to see if the file(s) for the repo are there?

No, it only checks if the repo is still there (the folder).

It appears that the function GetRepository(Guid id) is called on every page load.  Is that really necessary?

@KY4U Thanks for narrowing this down - a number of the bits of database access are grotesquely inefficient, which doesn't tend to matter with small databases on fast filesystems, but clearly is hit by the Azure I/O system.
There is no reason that getting a single repository by Guid (which is the primary key) to be inefficient at all, but some of the 'get all repository' stuff might be subject to improvement, and it's possible we've ended up with some kind of N^2 problem in the 'get all reposistories I'm allowed to see' logic.
How many repositories do you have, out of interest?

Some time ago i profiled on my local pc and the part that took longest was populating the navigate to repos dropdown.The merging of all i can see with all the I'm admin in took the longest. I had around 1000 repos but no users. It did take about 3 seconds to populate but on first startup what too longest was adding the repos to the db.
Maybe we can rework that part a bit.

@RedX2501 Would you like me to look at this?

@willdean I have ~110 repos.  By removing the droplist, all pages (except repository listing page) load in ~3 seconds, and my repository listing page loads in 9 seconds.  With the droplist, all pages load in ~ 30 seconds.  Until we can figure out what is causing the bottleneck, my suggestion at this point would be to have the droplist displayed only on the repository page.

@KY4U Would you be able to change RepositoryPermissionService.GetAllPermittedRepositories to look like this:
            var sw = Stopwatch.StartNew();
            try
            {
                return Repository.GetAllRepositories().Where(repo => HasPermission(userId, repo.Id, requiredLevel));
            }
            finally
            {
                Trace.WriteLine("GetAllPerm: " + sw.ElapsedMilliseconds);
            }

Then run this on your Azure instance and report the GetAllPerm line from the log file back here.
With 1000 repos I'm only seeing about 12ms here (fast machine, SSD, etc), so before I dive into trying to optimise this, I'd like to be sure that it's this that's slow for you.

@willdean
The log file shows the following results:
GetAllPerm: 151
GetAllPerm: 126
GetAllPerm: 167
GetAllPerm: 126
GetAllPerm: 195
GetAllPerm: 170
GetAllPerm: 117
GetAllPerm: 112
GetAllPerm: 126

@KY4U And that was with a 30s page load time?  So the problem's not there, anyway, it's somewhere else.

@willdean Page load times still 20-30 seconds so I added the timer to the PopulateRepoGoToList function.
PopulateRepoGoToList: 18391
PopulateRepoGoToList: 16075

@KY4U Yup, I see it here - well done.  I'll let you know when it's fixed.

My Profiling showed that it is literally the line with the linq statement that is merging the two lists. I broke it apart and neither sorting nor grouping is the problem.

@RedX2501 It's basically the HasPermission calls - it's confusing to profile because of the IEnumerable deferred execution.   The time is all in GetAllPermittedRepositories, it just didn't show up in my suggested stopwatch code, because the loop hasn't been executed at that point.
Hopefully I'll have it sorted shortly, I'm doing it now.

@KY4U Could you try the changes to RepositoryPermissionService that are in #561, please?
This is a combination of a 'cheat', to hugely enhance the performance for the system administrator, but also a number of other improvements which should be significant even for other users.
I'm going to continue to look at this, because there's more to come here.

@willdean Very nice!  Page load times 1-3 seconds.  Now that is good work.

@KY4U Thanks!  It was me that messed it up in the first place, so it seems right that I should fix it.


Optimize DDL export framework execution time

Description
Previously, the DDL export framework was taking over 20 minutes to run the ddl_all_objects.sql test, while other tests completed within a minute. Upon further investigation, we found that we were not filtering out objects in the 'sys' schema before checking IsSystemObject. This resulted in executing a query for each object present in the 'sys' schema to determine whether it was a system object or not.
This commit adds a schema filter before the IsSystemObject check in the SMO script. By first excluding objects in the 'sys' schema, we avoid unnecessary IsSystemObject checks on system objects, significantly reducing execution time of framework.
Signed-off-by: Sumit Jaiswal<EMAIL_ADDRESS>Issues Resolved
Task: BABEL-5473
Execution time of Python Worflow
Before : ~ 33 minutes
After: ~ 12 minutes
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

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

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
43105

Relevant Lines:
58421

💛  - Coveralls


Incorrect beam in Bach_Ein_festeBurg.mei

In measure 4 of Bach_Ein_festeBurg.mei, there are two beam groups which contain a quarter note:
https://github.com/music-encoding/sample-encodings/blob/5fba3a/MEI 3.0/Music/Complete examples/Bach_Ein_festeBurg.mei#L670-L674
                           <beam>
                              <note xml:id="d1e1269" pname="c" oct="4" dur="8" stem.dir="down"/>
                              <note xml:id="d1e1289" pname="d" oct="4" dur="4" stem.dir="down"/>
                              <note xml:id="d1e1303" pname="c" oct="4" dur="8" stem.dir="down"/>
                           </beam>

and
https://github.com/music-encoding/sample-encodings/blob/5fba3a/MEI 3.0/Music/Complete examples/Bach_Ein_festeBurg.mei#L680-L684
                           <beam>
                              <note xml:id="d1e1908" pname="e" oct="4" dur="8" stem.dir="up"/>
                              <note xml:id="d1e1926" pname="a" oct="3" dur="4" stem.dir="up"/>
                              <note xml:id="d1e1940" pname="g" oct="3" dur="8" stem.dir="up"/>
                           </beam>

The beam grouping should be removed on these notes.
Source PDF for this encoding:
https://github.com/music-encoding/sample-encodings/blob/5fba3a447513e11ed5bbe6545a7b227a9ed0e735/MEI 3.0/Music/Complete examples/Bach_Ein_festeBurg.pdf
This particular measure:

The beam group causes problems for verovio:

The notes look like eighth notes, but they retain their quarter-note durations while being beamed (which seems a reasonable thing to do given the error in the data).

Looks better:
https://github.com/music-encoding/sample-encodings/blob/935382d656b0d79e5b6166663f650dfcb1787394/MEI 3.0/Music/Complete examples/Bach_Ein_festeBurg.mei#L668-L675
                        <layer n="2">
                           <space xml:id="d56e8a1544"/>
                              <note xml:id="d1e1269" pname="c" oct="4" dur="8" stem.dir="down"/>
                              <note xml:id="d1e1289" pname="d" oct="4" dur="4" stem.dir="down"/>
                              <note xml:id="d1e1303" pname="c" oct="4" dur="8" stem.dir="down"/>
                           <note xml:id="d1e1321" pname="d" oct="4" dur="4" stem.dir="down"/>
                        </layer>

Is there a purpose to the <space> at the start of the layer (also present in layer 1 of staff 2 in the same measure)?
This space element confuses verovio, because it does not have a duration; therefore, verovio assigns a breve duration to the space:

Removing the two empty spaces:

Verovio assigns a breve duration to notes that have no duration ( @lpugin can explain why).  An advantage of this is that the rhythm errors are easy to see, such as when the note in measure 1 is missing a rhythm:

The extraneous spaces should be removed, or I can open a new issue for them :-)

I think I killed all them pesky spaces.

Yes, they are all gone.
Here is the current view, which is starting to look good for the direct display of the MEI data in verovio:

The double fermatas are data errors (I have to submit an issue for those yet :-).  There should be only one fermata encoded, unless @sameas is somehow used.
The weird slurs are probably related to verovio confused about their placement.  I haven't tracked down why that is happening yet, although I suspect that verovio cannot handle the slur properly if it is stored in a measure other than the same system where the slur should start.
You are still missing a fermata on the last measure, bottom staff :-)
Here is the round trip view from MEI to Humdrum to MEI again:

The slurs are correctly displayed since I move them to the place where verovio expects them.
The only problem left is automatically fixing the accidental encoding (which I will do sometime this month).


通过接口获取IPv6失败! 接口地址: https://speed.neu6.edu.cn/getIP.php

Description
异常信息: Get "https://speed.neu6.edu.cn/getIP.php": dial tcp6 [2001:da8:9000::66]:443: connect: network is unreachable
DNS Provider
No response
Did you search for similar issues before submitting this one?
No, I didn't
Operating System
Windows
Architecture
i386
Version
v6.6.8
How are you running ddns-go?
Docker
Any other information
No response

IPv6正常？

原来解析IPV6是不能再docker中运行，需要在系统中运行。


Day 11 Lab - App crashes when 'camera' selected on simulator

Right now, I can select camera on the action sheet, even though I'm running on the simulator. This crashes the app.

This bug is still in your app in Day 13's lab.

Bug Fixed.


Sources are missing for com.google.crypto.tink.subtle package in Maven source jar

I assume this isn't intentional, as the sources for the rest of the packages appear to be published properly (except aead) but this package is missing.
Given the source is freely accessible in Github, I assume this is a bug?

Yes, it looks like a bug.
@chuckx Charles, wonder if you want to take a look? Good chance to learn how we're generating the Maven jars.

Yup, I'll dig into this one.

FYI, I'll be performing a release within the next couple days to update the published Maven packages.

This is fixed in https://github.com/google/tink/releases/tag/v1.2.1.
I've verified that https://repo1.maven.org/maven2/com/google/crypto/tink/tink/1.2.1/tink-1.2.1-sources.jar contains com.google.crypto.tink.subtle package.
Please reopen if this still doesn't work. Thanks!


Issue with s3 URI names.    Doesn't support "." in the URI when doing s3 loads.

Using minio.

works
create external table cities (
    continent varchar  null,
    country varchar  null,
    city  varchar null
  )
  ENGINE=file
PROPERTIES
(
    "path"="s3://albert/cities.parquet",
    "format"="parquet",
    "aws.s3.endpoint" = "http://8.tcp.ngrok.io:11930",
    "aws.s3.enable_ssl" = "false",
    "aws.s3.enable_path_style_access" = "true",
    "aws.s3.access_key" = "iNr3UYLxTCgZRnv2",
    "aws.s3.secret_key" = "lM3BZ76MLkQloUnMU2JEtpecZws5uDEM"
)
  ;

don't work
create external table cities (
    continent varchar  null,
    country varchar  null,
    city  varchar null
  )
  ENGINE=file
PROPERTIES
(
    "path"="s3://albert.wong/cities.parquet",
    "format"="parquet",
    "aws.s3.endpoint" = "http://8.tcp.ngrok.io:11930",
    "aws.s3.enable_ssl" = "false",
    "aws.s3.enable_path_style_access" = "true",
    "aws.s3.access_key" = "iNr3UYLxTCgZRnv2",
    "aws.s3.secret_key" = "lM3BZ76MLkQloUnMU2JEtpecZws5uDEM"
)
  ;

If you put in the wrong format, you'll have a run away process.  It won't allow you to show or delete the table.


Actions/Config cleanup part 1

[x] Move platform specific stuff into the platform specific files. before platform-specific exported calls that actions/config used but this means one half or the other has an empty handler. Instead lets just have a per platform saga it forks and does its own thing
[x] Move showMain into platform specific

App.json / actions/app does very little and its not clear why its not just merged w/ config. doing that next
@keybase/react-hackers

native wasn't exporting its empty saga handler, fixed in part 2 but backported here


Turning off theme toggle breaks collapsible callouts

Not including the theme toggle causes a TypeError on line 501 of webpage.js:
Uncaught TypeError: Cannot set properties of null (setting 'checked')
    at setThemeToggle (webpage.js:501:18)

function setThemeToggle(state, instant = false) {
  let toggle = document.querySelector(".theme-toggle-input");
  toggle.checked = state;  // TypeError is here
  ....

This breaks in particular collapsible callouts (they no longer open or close) but may affect other things as well.  Probably just need to extend the null check if statement in setupThemeToggle() to include the code that sets the initial toggle state based on the body theme class.

Thanks for reporting this, I will try to reproduce it when I have time

This should be fixed in v1.7.0! Let me know if it is not


[Feature][Connector-e2e] add ftp e2e test

Purpose of this pull request
close #4501

Check list

[ ] Code changed are covered with tests, or it does not need tests for reason:
[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

At the beginning, I tried to exclude the commons-net package by using the hadoop dependency in the shade, and executed the flink engine to find an error in the content:
java.lang.LinkageError: loader constraint violation: loader (instance of sun/misc/Launcher$AppClassLoader) previously initiated loading for a different type with name "org/apache/commons/net/ftp/FTPClient"
at org.apache.hadoop.fs.ftp.FTPInputStream.(FTPInputStream.java:44)
The exception is thrown by hadoop.fs.ftp.FTPInputStream, so I think this package cannot be ruled out.
I will remove the commons-net dependency under connector-file-base and load the ftp-related dependency through seatunnel-hadoop3-3.1.4-uber.
After removal, the flink and zeta engines executed successfully, but spark reported the following error:
Exception in thread "main" java.lang.NoClassDefFoundError: org/apache/commons/net/ftp/FTPClient
at org.apache.seatunnel.connectors.seatunnel.file.ftp.system.SeaTunnelFTPFileSystem.connect(SeaTunnelFTPFileSystem.java:131)
at org.apache.seatunnel.connectors.seatunnel.file.ftp.system.SeaTunnelFTPFileSystem.listStatus(SeaTunnelFTPFileSystem.java:389)
at org.apache.seatunnel.connectors.seatunnel.file.source.reader.AbstractReadStrategy.getFileNamesByPath(AbstractReadStrategy.java:123)
at org.apache.seatunnel.connectors.seatunnel.file.ftp.source.FtpFileSource.prepare(FtpFileSource.java:83)
at org.apache.seatunnel.core.starter.spark.execution.SourceExecuteProcessor.initializePlugins(SourceExecuteProcessor.java:104)
at org.apache.seatunnel.core.starter.spark.execution.SparkAbstractPluginExecuteProcessor.(SparkAbstractPluginExecuteProcessor.java:49)
at org.apache.seatunnel.core.starter.spark.execution.SourceExecuteProcessor.(SourceExecuteProcessor.java:51)
at org.apache.seatunnel.core.starter.spark.execution.SparkExecution.(SparkExecution.java:57)
at org.apache.seatunnel.core.starter.spark.command.SparkTaskExecuteCommand.execute(SparkTaskExecuteCommand.java:59)
at org.apache.seatunnel.core.starter.SeaTunnel.run(SeaTunnel.java:40)
at org.apache.seatunnel.core.starter.spark.SeaTunnelSpark.main(SeaTunnelSpark.java:35)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at org.apache.spark.deploy.JavaMainApplication.start(SparkApplication.scala:52)
at org.apache.spark.deploy.SparkSubmit.org$apache$spark$deploy$SparkSubmit$$runMain(SparkSubmit.scala:958)
at org.apache.spark.deploy.SparkSubmit.doRunMain$1(SparkSubmit.scala:180)
at org.apache.spark.deploy.SparkSubmit.submit(SparkSubmit.scala:203)
at org.apache.spark.deploy.SparkSubmit.doSubmit(SparkSubmit.scala:90)
at org.apache.spark.deploy.SparkSubmit$$anon$2.doSubmit(SparkSubmit.scala:1046)
at org.apache.spark.deploy.SparkSubmit$.main(SparkSubmit.scala:1055)
at org.apache.spark.deploy.SparkSubmit.main(SparkSubmit.scala)
Caused by: java.lang.ClassNotFoundException: org.apache.commons.net.ftp.FTPClient
at java.net.URLClassLoader.findClass(URLClassLoader.java:387)
at java.lang.ClassLoader.loadClass(ClassLoader.java:418)
at java.lang.ClassLoader.loadClass(ClassLoader.java:351)
... 23 more
I don't know if there is a better solution for the time being...

@TyrantLucifer  I have turned off #4702 and confirmed that it is a permission problem. After reconfiguring the permissions of ftp, it has been submitted to the current branch.

I don't know if there is a better solution for the time being...

Because spark use their own hadoop depenency. Not seatunnel-hadoop3-3.1.4-uber. So maybe you should put commons-net.jar into spark lib (or jars? I forget :) ) directory. If it work, please add this way in the doc.


Missing date separator bars when sending a message with a new date to an existing thread

Here's a screenshot:

There should be a date separator bar before the new message I just sent.
I suspect this is a regression introduced by 00c7f7d42f5384ac0315429f673ba8c3b4e9deef

This was fixed by 47465ea51a2594e0cdb8dde583d36b1cf72f4717


Update procaire-tutorial.md

Our customers only use Idp-initiated, but they frequenly add this step (set up for sp), which then breaks our idp-init login. We'd like this removed from our tutorial, and will encourage our customers to use idp-init logins.

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

#sign-off


Fix transaction invoice PDF endpoint URL

Get transaction invoice PDF endpoint should be /transactions/{$id}/invoice per https://developer.paddle.com/api-reference/transactions/get-invoice-pdf

@Invincibear thanks for the contribution, I took the branch and merged in #12 👍


Main thread (probably?) of FancyZone crashes randomly

Microsoft PowerToys version
0.51.1
Running as admin

[ ] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
Main thread (probably?) of FancyZone crashes randomly and I currently have no idea why. It can work fine for days and may crash in a few hour. Looking in the logs it just crashed on its own, the stack trace seems not helpful. Rebooting it in PowerToy setting will bring it back.
Bug report logs: PowerToysReport_2022-01-07-22-34-30.zip
By the way 0.53.1 has been running for 2 days and it haven't crashed yet.
✔ Expected Behavior
Main thread does not crash, holding shift activates zones.
❌ Actual Behavior
It crashed, after that holding shift does nothing, but FancyZone's process is still running.
Other Software
No response

Please let us know if this still occurring on release 0.53.1. Thanks!

Crashed just 3 mins ago. :(
PowerToysReport_2022-01-08-17-57-07.zip

Hi friends! This bug is relatively old. In an effort to clear out any stale bugs, could you please let us know if this bug is still showing up on any of the latest builds, 0.74 or up!
/needinfo


Hassio: Mosquitto MQTT server won't start

Home Assistant release (hass --version):
Hassio 0.64.1
Python release (python3 --version):
N/A for Hassio
Component/platform:
Mosquitto broker installed from Hassio add-ons
Description of problem:
MQTT server doesn't start when the start button is clicked on this page:
http://hassio.local:8123/hassio/addon/core_mosquitto
Does not start with the system either.
This worked with <0.64.1 .    No configuration changes prior.
Expected:
MQTT server running
Problem-relevant configuration.yaml entries and steps to reproduce:

Try and start Mosquitto server from the web interface by clicking start
It appears to start but reloading the page shows that the MQTT server is not actually running.

Restarts and uninstalling the addon and reinstalling do not help.
Traceback (if applicable):

2018-02-28 23:15:14 ERROR (MainThread) [aiohttp.server] Error handling request
Traceback (most recent call last):
  File "/usr/lib/python3.6/site-packages/aiohttp/web_protocol.py", line 416, in start
    resp = yield from self._request_handler(request)
  File "/usr/lib/python3.6/site-packages/aiohttp/web.py", line 325, in _handle
    resp = yield from handler(request)
  File "/usr/lib/python3.6/site-packages/aiohttp/web_middlewares.py", line 93, in impl
    return (yield from handler(request))
  File "/usr/lib/python3.6/site-packages/homeassistant/components/http/static.py", line 70, in staticresource_middleware
    return (yield from handler(request))
  File "/usr/lib/python3.6/site-packages/homeassistant/components/http/real_ip.py", line 28, in real_ip_middleware
    return (yield from handler(request))
  File "/usr/lib/python3.6/site-packages/homeassistant/components/http/ban.py", line 70, in ban_middleware
    return (yield from handler(request))
  File "/usr/lib/python3.6/site-packages/homeassistant/components/http/auth.py", line 53, in auth_middleware
    return (yield from handler(request))
  File "/usr/lib/python3.6/site-packages/homeassistant/components/http/__init__.py", line 409, in handle
    result = yield from result
  File "/usr/lib/python3.6/site-packages/homeassistant/components/hassio/http.py", line 66, in _handle
    return _create_response_log(client, data)
  File "/usr/lib/python3.6/site-packages/homeassistant/components/hassio/http.py", line 120, in _create_response_log
    log = re.sub(r"\x1b(\[.*?[@-~]|\].*?(\x07|\x1b\\))", "", data.decode())
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xff in position 138: invalid start byte

Additional info:

Issue remains with Hassio 0.64.3

I resolved this by formatting my sd card and starting over.  It may have been corruption...


Property panel is not displayed for some steps

Describe the Bug
Property panel is not displayed for several steps
Steps to Reproduce the Bug or Issue

add google drive or google mail component or AS2 or google-storage-sink kamelet or Saga processor
Select to open property panel
---> Field not found in schema: "parameters.scopes.$"

Screenshots or Videos

Platform
main branch as using online version

@igarashitm funny enough, for as2 is the same issue as google-drive, except the Java definition looks different:
"javaType": "java.lang.String[]"
    "signedReceiptMicAlgorithms": { "index": 35, "kind": "parameter", "displayName": "Signed Receipt Mic Algorithms", "group": "security", "label": "security", "required": false, "type": "array", "javaType": "java.lang.String[]", "deprecated": false, "autowired": false, "secret": false, "configurationClass": "org.apache.camel.component.as2.AS2Configuration", "configurationField": "configuration", "description": "The list of algorithms, in order of preference, requested to generate a message integrity check (MIC) returned in message dispostion notification (MDN)" },

That being said, would it help to have a small parser like the one for the catalog table and parse that?
In any case, as far as I can see, if there's a string array, we can make it as such, but for other kinds of classes, is very likely that we're gonna do the same for the time being :thinking: , so, would be useful to always tag arrays as string[] in the schemas for the time being?

For google-sheets, it's a similar situation, but with a different type:
"javaType": "java.util.Collection<java.lang.String>"
    "scopes": { "index": 4, "kind": "property", "displayName": "Scopes", "group": "common", "label": "", "required": false, "type": "array", "javaType": "java.util.Collection<java.lang.String>", "deprecated": false, "autowired": false, "secret": false, "configurationClass": "org.apache.camel.component.google.sheets.GoogleSheetsConfiguration", "configurationField": "configuration", "description": "Specifies the level of permissions you want a sheets application to have to a user account. See https:\/\/developers.google.com\/identity\/protocols\/googlescopes for more info." },

@lordrip actually now I'm thinking to parse component catalog in @kaoto-next/camel-catalog and put the propertiesSchema onto the catalog, just like we did for languages/dataformats.

This fixes the for components - https://github.com/KaotoIO/kaoto-next/pull/484
For google-storage-sink, it would need to do similar thing for kamelets as well. Also for saga, it might be better to extract the schema from camelYamlDsl.json. And then attach to the catalog as propertiesSchema just like others.


feat: add DataFrame.rows

What type of PR is this? (check all applicable)

[ ] 💾 Refactor
[x] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 🔧 Optimization
[ ] 📝 Documentation
[ ] ✅ Test
[ ] 🐳 Other

Related issues

Related issue #
Closes #285

Checklist

[x] Code follows style guide (ruff)
[x] Tests added
[x] Documented the changes

If you have comments or can explain your changes, please do so below.
I've matched the API with https://docs.pola.rs/api/python/stable/reference/dataframe/api/polars.DataFrame.rows.html

Thanks for your comments. I'll address them soon.

Just got a question about this functionality - instead of DataFrame.rows, couldn't we just add DataFrame.iter_rows?

@MarcoGorelli I was going to propose something similar (you beat me to it 😄). I was thinking we could expose both rows and iter_rows since the polars API does as well.
One issue with this one is I'm not sure if pandas has a convenient equivalent to this when named=True. Rest of the cases are fine:

polars + named=False = df.iter_rows(named=False)
polars + named=True = df.iter_rows(named=True)
pandas + named=False = df.itertuples(index=False, name=None)
pandas + named=True = ?

Please let me know if you're aware of a pandas API which returns an iterator for iterating over rows as dictionaries.

@Priyansh121096 I tried to play with the into argument of .to_dict() method, sadly with no success.
I guess that 4. can become iter(df.to_dict("records"))?!

How about using https://docs.python.org/3/library/collections.html#collections.somenamedtuple._asdict (which , despite the underscore, is public):
something like (simplified)
def iter_rows(df):
    yield from (row._asdict() for row in df.itertuples(index=False))

I guess that 4. can become iter(df.to_dict("records"))?!

I feel like this defeats the purpose of using iter_rows over rows though.

How about using https://docs.python.org/3/library/collections.html#collections.somenamedtuple._asdict (which , despite the underscore, is public):

Amazing! This should work.

@MarcoGorelli I'll raise another PR for iter_rows soon. Can we merge this one?

thanks - tbh I'm not really sure about DataFrame.rows, I might even suggest deprecating it altogether in Polars itself
could we just repurpose this one for DataFrame.iter_rows please? sorry for not having thought about this straight away

could we just repurpose this one for DataFrame.iter_rows please?

@MarcoGorelli pushed a change for this.

@pre-commit.ci autofix


chore(pre-commit): add pyupgrade hook

I've added a pre-commit hook that runs pyupgrade which automatically checks and updates Python syntax to newer versions of the language, so we can make sure we don't use unnecessarily old syntax which is often less convenient. I've configured pyupgrade to check for and update to Python 3.7 syntax which is Copier's minimum Python version at this point.

Codecov Report

Merging #763 (2eb27dd) into master (046327b) will decrease coverage by 14.84%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master     #763       +/-   ##
===========================================
- Coverage   96.62%   81.77%   -14.85%
===========================================
  Files          41       41
  Lines        2930     2930
===========================================
- Hits         2831     2396      -435
- Misses         99      534      +435

Flag
Coverage Δ

unittests
81.77% <ø> (-14.85%)
:arrow_down:

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

Impacted Files
Coverage Δ

tests/test_prompt.py
13.09% <0.00%> (-76.20%)
:arrow_down:

tests/test_complex_questions.py
30.72% <0.00%> (-69.28%)
:arrow_down:

tests/test_tools.py
64.00% <0.00%> (-28.00%)
:arrow_down:

tests/test_templated_prompt.py
72.13% <0.00%> (-27.87%)
:arrow_down:

copier/user_data.py
69.81% <0.00%> (-23.43%)
:arrow_down:

tests/conftest.py
72.72% <0.00%> (-22.73%)
:arrow_down:

tests/test_updatediff.py
81.27% <0.00%> (-18.73%)
:arrow_down:

tests/test_vcs.py
85.71% <0.00%> (-14.29%)
:arrow_down:

copier/cli.py
85.88% <0.00%> (-11.77%)
:arrow_down:

tests/helpers.py
91.22% <0.00%> (-8.78%)
:arrow_down:

... and 8 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

I'm not sure why Codecov is having some issue in so many of the checks. But I think those failures are unrelated to the change made in this PR.


fix: missing branch in show_signature_annotations

This PR addresses #361 by adding a missing branch, when rendering annotations. (TODO: add a test)

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 88.36%. Comparing base (4739a4e) to head (0463b7e).

Files
Patch %
Lines

quartodoc/renderers/md_renderer.py
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #362      +/-   ##
==========================================
- Coverage   88.46%   88.36%   -0.10%
==========================================
  Files          37       37
  Lines        2931     2932       +1
==========================================
- Hits         2593     2591       -2
- Misses        338      341       +3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add What's New In DevTools (Chrome 84)

What's changed, or what was fixed?

Adds https://developers.google.com/web/updates/2020/05/devtools

Fixes: N/A
Target Live Date: 2020-05-13

[ ] This has been reviewed and approved by @kaycebasques
[ ] I have run npm test locally and all tests pass.
[x] I have added the appropriate type-something label.
[ ] I've staged the site and manually verified that my content displays correctly.

CC: @petele

Whoops!
There were 3 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/updates/2020/05/devtools.md#L126 - Consider adding a "was this page helpful?" widget to your page: https://developers.google.com/web/resources/widgets#helpful
src/content/en/updates/2020/05/devtools.md#L18 - Common typo found: 'notifcation' Should it be 'notification'?
src/content/en/updates/2020/05/devtools.md#L58 - Line must be at most 100 characters

Whoops!
There were 3 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/updates/2020/05/devtools.md#L126 - Consider adding a "was this page helpful?" widget to your page: https://developers.google.com/web/resources/widgets#helpful
src/content/en/updates/2020/05/devtools.md#L18 - Common typo found: 'notifcation' Should it be 'notification'?
src/content/en/updates/2020/05/devtools.md#L58 - Line must be at most 100 characters

Whoops!
There were 2 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/updates/2020/05/devtools.md#L162 - Consider adding a "was this page helpful?" widget to your page: https://developers.google.com/web/resources/widgets#helpful
src/content/en/updates/2020/05/devtools.md#L73 - Line must be at most 100 characters

Whoops!
There were 2 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/updates/2020/05/devtools.md#L164 - Consider adding a "was this page helpful?" widget to your page: https://developers.google.com/web/resources/widgets#helpful
src/content/en/updates/2020/05/devtools.md#L77 - Line must be at most 100 characters

Whoops!
There were 2 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/updates/2020/05/devtools.md#L177 - Consider adding a "was this page helpful?" widget to your page: https://developers.google.com/web/resources/widgets#helpful
src/content/en/updates/2020/05/devtools.md#L65 - Line must be at most 100 characters

Whoops!
There were 2 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/updates/2020/05/devtools.md#L177 - Consider adding a "was this page helpful?" widget to your page: https://developers.google.com/web/resources/widgets#helpful
src/content/en/updates/2020/05/devtools.md#L65 - Line must be at most 100 characters

Whoops!
There were 2 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/updates/2020/05/devtools.md#L179 - Consider adding a "was this page helpful?" widget to your page: https://developers.google.com/web/resources/widgets#helpful
src/content/en/updates/2020/05/devtools.md#L65 - Line must be at most 100 characters


should kit build not build UI by default?

https://discord.com/channels/1186394868336038080/1186421052071477428/1210290377823887370

IMO: no. kit build should, indeed, build UI by default. There is a --no-ui flag for the dev-ui flow.


perf(price-part): add a css part to expose price text styles

Alaska Airlines Pull Request
Adds a part to the span containing the price. Allows a consumer to manipulate the text.
Fixes: #38
Type of change:
Please delete options that are not relevant.

[x] New capability

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

:tada: This PR is included in version 1.2.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


FE-79 Use unix domain sockets using pycurl

this is based on the PR https://github.com/pachyderm/pachyderm/pull/9143
changed the client to use pycurl instead of requests_unixsocket package

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: pankaj-mandal:x: Pankaj MandalPankaj Mandal seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@chainlink could you please review.

Closing as this change was accepted through #9366


Update README.md

Important limitation that would have saved me from 10 minutes of wasted time :(

Hi Egon,
sorry for not communicating that more clearly. Hopefully it's better now after your changes.
Best regards,
-Jurriaan


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/Zefau/ioBroker.nello

[x] [E301] Tests on Travis-ci.org are broken. Please fix.
[x] [E605] No actual year found in copyright. Please add "Copyright (c) 2019 [object Object]" or "Copyright (c) 20xx-2019 [object Object]" at the end of Readme

Thanks,
your automatic adapter checker.
P.S.: There is a community in Github, which supports the maintenance and further development of adapters. There you will find many experienced developers who are always ready to assist anyone. New developers are always welcome there. For more informations visit: https://github.com/iobroker-community-adapters/info

Thanks, that all bugs have been fixed.


Remove supportsRasterOverlays bool

This PR addresses #489
Removes supportRasterOverlays from Tileset.
@nithinp7

Thanks @miatang13! Please mention this as a (technically) breaking change in changelog. I'll merge this after CI completes and the changes are updated.


Cannot detect attributes added via init

I've tried to run Codon on this source code:
class MyClass:
    def __init__(self, value):
        self.value

MyClass(1)

codon build --release --cost-kind=latency --exe ./1.py

But got an error:
1.py:3:9-19: error: 'MyClass' object has no attribute 'value'
╰─ 1.py:5:1-11: error: during the realization of __init__(self: MyClass, value: int)

Adding attributes in __init__ method is one of basics of Python, it should not fail.
It can be fixed by:
class MyClass:
    value: int

    def __init__(self, value):
        self.value

MyClass(1)

But this creates a class-level attribute which is shadowed by an object-level attribute with the same name (created in __init__), which is not great.

Hi @dolfinus
Currently, Codon classes are equivalent to Python's dataclasses. So the second example will work as expected (those are not class attributes).
Limited support for Python-like __init__ can be achieved by doing
@autodeduce
def __init__(self, value):
   self.value = value


Document how to add support for new Database

Based on the experience of building support for a new DB, we should add dev docs on how to do it so it is easier for a new joiner in the team to know the process

How is this progressing @pankajkoti ?

Hi @kaxil. I had planned to work on this today at the start of the week. I ran into a few blockers for the benchmarking script for Redshift db performance and still facing some issues. Utkarsh and Tatiana paired up with me initially but there are still some issues in the script and would need more time on it.
@phanikumv also reminded me to work on this one with priority in this week. So was planning to work on the document after the benchmarking task.
One question I had is where should we publish this document? Github repo ReadMe, Notion doc, Google doc, RTFD or some other place?
I realise and understand that there has been delay on the document from my side. Is it okay if I pick up this after the benchmarking task? If not, I will try if I can work on this earlier.

No worries at all, was just curious.
Yeah benchmarking should be done before this task.
The docs should go in our Contributing.rst file

Also mention that benchmark results should be added whenever we add new database. And test the performance well.

Made good progress on the document, PR will be ready in a while. ETA 2 PM IST today.


KAFKA-9662: Wait for consumer offset reset in throttle test to avoid losing early messages

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Ran throttling_test three times, twice with this commit on 2.5 and once on trunk. Both the throttling tests passed in all three cases. Other system tests are not affected since they use consumers with default from_beginning=True.


feat: Useful extensions

Multiple datetime formats are now supported, cc @YH-15. Follow-up issue on #127.
Syntaxes are simplified. More to be edited on v1.4.

Codecov Report
Base: 65.24% // Head: 66.03% // Increases project coverage by +0.79% :tada:

Coverage data is based on head (b6e8768) compared to base (ac59973).
Patch coverage: 48.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #131      +/-   ##
============================================
+ Coverage     65.24%   66.03%   +0.79%
- Complexity      521      522       +1
============================================
  Files           101      102       +1
  Lines          1715     1702      -13
  Branches        188      184       -4
============================================
+ Hits           1119     1124       +5
+ Misses          539      520      -19
- Partials         57       58       +1

Impacted Files
Coverage Δ

...a/jeryl/fyp/logic/commands/AddDeadlineCommand.java
93.33% <ø> (ø)

...eryl/fyp/logic/commands/DeleteDeadlineCommand.java
0.00% <ø> (ø)

...jeryl/fyp/logic/commands/DeleteStudentCommand.java
100.00% <ø> (ø)

.../java/jeryl/fyp/logic/commands/HelpAddCommand.java
0.00% <ø> (ø)

...ain/java/jeryl/fyp/logic/commands/MarkCommand.java
54.16% <ø> (ø)

...c/main/java/jeryl/fyp/logic/parser/ParserUtil.java
70.17% <0.00%> (+18.22%)
:arrow_up:

...rc/main/java/jeryl/fyp/commons/core/Datetimes.java
50.00% <50.00%> (ø)

...main/java/jeryl/fyp/model/util/SampleDataUtil.java
31.03% <50.00%> (+10.20%)
:arrow_up:

...rc/main/java/jeryl/fyp/logic/parser/CliSyntax.java
90.00% <100.00%> (ø)

...rc/main/java/jeryl/fyp/model/student/Deadline.java
64.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Modal - wrapped "title" text does not line up with widget

Hi all,
I've just noticed that, in the modal pop-up, if the "title" for a columnDef wraps then it does not line up with the widget (see attached image).
Regards
Shando

@Shando I noticed that too. But that appears to be how Bootstrap wraps the label. It automatically wraps and expands the label height when the text gets too long.

I tested this with a jsfiddle and this is the result.
Best,
Vincent

OK, I'll see if there's anything within Bootstrap itself that I can change.


Outline Client does not connect on Mac

This issue tracker is for bug reports only. If you want to request troubleshooting assistance with Outline Manager or Outline Client, please do not submit a bug here. Instead, please contact us using our support website. Developers using Outline SDK, please post your questions on the SDK Discussion board.
Describe the bug
The app appears behind the notch on a MacBook pro, so not accessible. Also, when you click on "Connect" it doesn't do it the very first time. You have to open it and click it several times for it to — maybe — work.
To reproduce
Just install the latest version of the client for Mac.
Expected behavior
I expected all of the above to work. It should connect when you click "Connect"
Screenshots
If applicable, add screenshots to help explain your problem.


Remove or update SwiftyGif submodule.

The SwiftyGif repo used to contain a lot of large snapshot test images which makes it very slow to install Agrume via SPM over a slow internet connection while Xcode clones the Agrume repo.
The submodule serves no purpose if you're using SPM, however if there is a reason for having this please consider bumping the commit that the submodule points at to a more recent version of SwiftyGif as the large snapshot files were removed in this commit https://github.com/kirualex/SwiftyGif/commit/c804e36ea21f25715a55216ff40d18c3941f8ced

Thanks for the report! TBH I wasn't aware that SPM clones the submodule. The submodule really only is there for local development since the project supports CocoaPods and Carthage as well and I didn't want to rely on either for local development but it's probably time to revisit that decision. I'll try to get around to it in the coming days but of course you're welcome to submit a PR as well if it's urgent.

Updated in https://github.com/JanGorman/Agrume/releases/tag/5.8.1

@JanGorman thanks for updating this - just looking at this again and I'm not 100% sure this will fix the problem because I don't think submodules are shallow-cloned by default.
However git does support this by adding "shallow = true" to the gitmodules config, e.g.:
[submodule "Frameworks/SwiftyGif"]
	path = Frameworks/SwiftyGif
	url = https://github.com/kirualex/SwiftyGif
	shallow = true


Fix children type in Link interface

<Link to="/">Home</Link>
Link component don't accept text as child element.

Codecov Report

Merging #54 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #54   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           5      5
  Lines         102    102
  Branches       26     26
=====================================
  Hits          102    102

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 b6cbd0b...0e47803. Read the comment docs.

See #56


Added steps for signing with Azure Code Sign to ELAM driver resource …

…file info. Need to wait for the PCA cert to be included in pkiops

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

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

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

Authoring waiting for more information.

@GrantMeStrength it is still accurate/required to be documented. The product "Azure Code Sign" has been rebranded to "Trusted Signing"

Updated references to "Azure Code Sign"  to "Trusted Signing"


Comments after 25th April meeting

Merge CellValue and CellArray, since CellArray = CellValue{T<:AbstractArray}, and only implement for CellArray those methods in which efficiency is important, and replace CellData by CellValue
Replace range_sizeby a return_size that comes from the Point{D} array in the evaluation method
Create CellFieldFromExtended to represent functions in the reference space
Create CellInterpolator <: CellVector that composes a CellFieldFromExtended and CellValue{D<:DOFBasis} (constant at the beginning)

I have implemented all the previous things.
There is a bug in CellQuadrature constructor that I cannot understand yet.
On the other hand, there is something wrong with IndexCellVector and for the moment I am using IndexCellArray{T,1} instead, it fixes all bugs. TO BE investigated.


Text changes to tagmigration show page

Changes made to the text on the tagmigration show page as part of this Trello card.
Paired with @Davidslv

👏 well done!

@carvil thanks for your comments, I've addressed it by using your suggested 2nd option.

Nice one @klssmith it looks great! As discussed IRL, we should probably write a rake task to migrate existing TagMigrations across to the new format (by populating the new fields).

@carvil as discussed there's only one TagMigration that happens to have a source_description with a nil value. We decided there was no need to create a rake task for such.
Thanks 👍


Only error in ES3 and ES5 when using 'arguments' in an arrow function

Because of the subtle nuances of capturing/renaming arguments, we're not sure if we want to continue on #2764.
Instead, we will just error in ES3 and ES5 modes.
Resolves #2430.

:+1:


MSID: 94438 Version: 1 DOI: 10.1101/2<IP_ADDRESS>3874

MSID: 94438
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>3874
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=94438
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>3874

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/94438v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>3874
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/94438/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

Need to remove the duplicate author info section


Spec of the data storage scheme

I'm confused by this. Why do we need a REST API for interacting with orbit-db? In my mind the orbit-db instances would live in the client library (3box-js). I thought that this story was more about the data model we should use for storing the data. Basically how we store the data in the orbit-db instances and how we deal with encryption etc. That's why I created the 3box-js data structure specification link in the readme.
I understand though that this is a bit unclear. I think we need to have a chat today discussing (me, you and @christianlundkvist)


void disableAllIndicator(); function is missing

Function is present in documentation but there is no implementation.

Sorry but we didn't have time to do another release yet. master is the latest version here on github but not the latest released. I hope we can do a release soon.

Ok I didn't look at this point.  At least there is a typo in README file: functions are called witout final s disableAllIndicator() vs disableAllIndicators()


Repetition of Conda package and Arcmap/Pro Toolbox Folders

While looking at issue https://github.com/Esri/military-tools-desktop-addins/issues/66 the related issue has come up concerning the repetition + duplication of the Military Tools Toolbox to several different locations:

ArcMap / Pro version that opens and works from the folder: https://github.com/Esri/military-tools-geoprocessing-toolbox/tree/dev/Geoprocessing
Conda package version that rearranged this folder structure and only works if conda-installed as a system toolbox https://github.com/Esri/military-tools-geoprocessing-toolbox/tree/dev/pythonpackage
Newly created conda package version added to this repo: https://github.com/Esri/military-tools-desktop-addins/tree/cybertech-dev/tools/src that also does not work unless conda-installed (problem #2 above)

There should be some research into whether these 3 different toolboxes are necessary and if they can be combined and what the changes/limitations would be.
The known limitations are:

For the conda package, the toolbox must be located at + installed to {packagename}\esri\toolboxes
If the military tools system toolbox has been installed through Conda, ArcGIS will not open another  toolbox with the same name from a local folder (the toolbox likely won't even appear in Pro)

Expected Behavior

There be one folder that works for both Conda packaging and when opened from the folder, making it clear where to maintain these tools.
Current Behavior

3 different copies of toolboxes + dependencies
Possible Solution

Your Environment

Version used: Pro 2.2+ / ArcMap

Discussed with @lfunkhouser @dfoll about adding this to sprint as time-boxed research task to assist in :

Resolving https://github.com/Esri/military-tools-desktop-addins/issues/66
Resolve the issue of auto-building the Conda package (can't currently)
Resolve various issues seen with input-schema and  output layer files not working in the Conda package

@saip @dfoll - in case you want to review or test, the updated repo with the merged tool folders is in this branch:  https://github.com/Esri/military-tools-geoprocessing-toolbox/tree/csm/%23389-remove-duplication
Toolbox in this folder:  https://github.com/Esri/military-tools-geoprocessing-toolbox/tree/csm/%23389-remove-duplication/tools/militarytools/esri/toolboxes (@dfoll - this is also the folder you would zip to use in ArcMap)

The only issue I have found testing so far is that "GRG from Point" fails on Pro 2.2 at the labeling step - but this seems like a new issue unrelated to all of the folder reorgs (error also occurs from the previous "Geoprocessing" folder version).

I tested all of the tools in the toolbox from the Conda package install today on ArcGIS Pro 2.3.0-beta.1+build.14516
Two tools from the Visibility toolset had problems:

Linear Line of Sight
Radial Line of Sight
In both cases, the Input Observers feature set control did not expand and turn to the blue point input symbol.
The Targets for LLOS worked correctly.

Strangely, the input observer points worked correctly for the Radial Line of Sight and Range tool.
I switched my conda environment and added the toolbox folder to my project folders, ran the two tools from there and saw the same behavior.
I think this may be Pro 2.3 specific behavior.
@csmoore
@dfoll

Added issue
https://github.com/Esri/military-tools-geoprocessing-toolbox/issues/391

as far as what I can think of, only one small doc change was needed (that didn't really apply to this issue directly) On https://solutions.arcgis.com/defense/help/military-tools/get-started/military-tools-for-arcgis-get-started.htm
Changed Note:  Installing the Military Tools toolbox using the Python Package Manager is only supported for ArcGIS Pro 2.2. to Note:  Installing the Military Tools toolbox using the Python Package Manager is only supported for ArcGIS Pro 2.2 and newer.

Unsure which V labels to use for this


Remove aloe

Fixes #274

Whoops, I just merged #291 into this branch because I forgot to change the base in the PR. It's just going to look a bit odd in the history, but it shouldn't change this PR at all

ok, never mind..


fix clippy linting recommendations (edition 2018)

Figured I'd start with recommendations in the current codebase as is, and that'll make handling https://github.com/tokio-rs/prost/issues/553 a bit easier.

@chetaldrich looks like you may need to run rustfmt, ci is failing.

@LucioFranco I ran rustfmt, so should be good to give CI another run.

It turns out you get a compilation error on Rust 1.51.0 if you use the split_once and rsplit_once methods since they were unstable at that version.
In Rust stable (currently 1.58.1), there are clippy recommendations that
say to use those methods instead of rsplitn and splitn if you're only splitting once.
I figured the best way to handle this since we're targeting 1.51 in addition to stable is to just set specific ignore in that case so the old version still compiles successfully.
Should be good for another CI run @LucioFranco, thanks for being prompt about the last one!


[Unstable] Fail to run "db.version();" with an error "The process exited code 1"

OS: Mac/Win 10
Build Version: 20200522.22
Repro Steps:

Install the mongo shell.
F1 --> User Settings --> define the variable 'mongo.shell.path'.
In the scrapbook, run db.version();.

Expect:
The command returns a version.
Actual:
There is an error: "Error running process: The process exited with code 1".

More info:
Starting executable: "/Users/bys/Desktop/mongodb-macos-x86_64-4.2.7/bin/mongo" --quiet mongodb://elsatest1:Gx8GExnlA3OhbftXbzO6VUCiHIaloVvS9sq03MAkdcr8eGXtAeVeMH9vFu4XxVg0GoSBoqFBDKRfZW0jZ28HvQ%3D%3D@elsatest1.documents.azure.com:10255/d1?ssl=true&replicaSet=globaldb&retrywrites=false
2020-05-25T17:01:21.275+0800 I  NETWORK  [js] Starting new replica set monitor for globaldb/elsatest1.documents.azure.com:10255
2020-05-25T17:01:21.276+0800 I  CONNPOOL [ReplicaSetMonitor-TaskExecutor] Connecting to elsatest1.documents.azure.com:10255
2020-05-25T17:01:23.026+0800 I  NETWORK  [ReplicaSetMonitor-TaskExecutor] Confirmed replica set for globaldb is globaldb/cdb-ms-prod-centralus1-fd1.documents.azure.com:10255
2020-05-25T17:01:23.026+0800 I  CONNPOOL [ReplicaSetMonitor-TaskExecutor] Connecting to cdb-ms-prod-centralus1-fd1.documents.azure.com:10255
2020-05-25T17:01:24.764+0800 I  NETWORK  [ReplicaSetMonitor-TaskExecutor] Confirmed replica set for globaldb is globaldb/cdb-ms-prod-centralus1-fd1.documents.azure.com:10255
2020-05-25T17:01:27.311+0800 E  QUERY    [js] DriverSession.UnsupportedError: Logical Sessions are only supported on server versions 3.6 and greater. :
DriverSession.UnsupportedError: Logical Sessions are only supported on server versions 3.6 and greater.
makeDriverSessionConstructor/driverSessionConstructor.UnsupportedError@src/mongo/shell/session.js:997:52
ServerSession@src/mongo/shell/session.js:514:19
createServerSession@src/mongo/shell/session.js:1008:20
driverSessionConstructor@src/mongo/shell/session.js:860:35
startSession@src/mongo/shell/mongo.js:536:30
@(connect):4:6
2020-05-25T17:01:27.314+0800 F  -        [main] exception: connect failed
2020-05-25T17:01:27.314+0800 E  -        [main] exiting with code 1
Error running process: The process exited with code 1.
5:01:27 PM: Error: The process exited with code 1.
The output window may contain additional information.
More info:
Sometimes creating new database can fix this issue.

I ran into this problem too. Running the same command on a v3.6 account works fine (per this part of the error message: "Logical Sessions are only supported on server versions 3.6 and greater"). Because of this it may be good to consider creating v3.6 accounts by default in the extension


Beta Release

Tracking issue for features required for BETA release.
See corresponding issue uisautomation/university-media-deploy#19.
Design docs:

https://gist.github.com/rjw57/6a72b123bdb00595690fea323f878ced

Features:

[ ] UMP database becomes source of truth for media item metadata
[ ] UMP application manages JWP uploads
[ ] UMP ingests legacy SMS data
[ ] Playlist support

Moved to be a milestone, not an issue


GestureDetector/onScaleUpdate details.scale is always a constant under Linux-Desktop channel

https://api.flutter.dev/flutter/widgets/GestureDetector/onScaleUpdate.html
class MyHomePage extends StatefulWidget {
  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  double _scaleFactor = 1.0;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
        backgroundColor: Colors.white,
        body: Container(
            child: GestureDetector(
          onScaleStart: (details) {},
          onScaleUpdate: (details) {
            print(details.scale);
            /*
            setState(() {
              _scaleFactor += details.scale;
            });
            */
          },
          onScaleEnd: (details) {},
          child: Container(
            height: MediaQuery.of(context).size.height,
            width: MediaQuery.of(context).size.width,
            color: Colors.green,
            child: Center(
              child: Text(
                'Test',
                textScaleFactor: _scaleFactor,
              ),
            ),
          ),
        )));
  }
}

details.scale remains 1.0 all the time in Linux-Desktop channel.
I tried Web-Desktop channel, It's working fine. (details.scale changes from -3 to 3)
I'm using a touchable laptop (yoga 260).

@yingshaoxo
I think this is an issue with macOS desktop too. See this similar issue which was closed in favor of 23604. Does it resemble your issue ?

It's the same.
https://github.com/flutter/flutter/issues/69372


docs(analytics): add Generic HTTP as a community-supported analytics …

…tool  Signed-off-by: Agniva Banerjee<EMAIL_ADDRESS>Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

Thanks! :tada:

Thanks! 🎉

Hey Ben can you review this one, I forgot to add the link https://github.com/backstage/backstage/pull/27137


for disconnected, use Vec instead of HashSet to reduce insert overhead

Objective

Improve Schedule::initialize performance

Solution

replace disconnected's type from HashSet to Vec in check_graph

Bench result
➜ critcmp schedule_base schedule_optimize
group                                         schedule_base                          schedule_optimize
-----                                         -------------                          -----------------
build_schedule/1000_schedule                  1.07       2.3±0.15s        ? ?/sec    1.00       2.1±0.11s        ? ?/sec
build_schedule/1000_schedule_noconstraints    2.99     65.3±0.79ms        ? ?/sec    1.00     21.8±0.37ms        ? ?/sec
build_schedule/100_schedule                   1.06     12.8±0.08ms        ? ?/sec    1.00     12.2±0.07ms        ? ?/sec
build_schedule/100_schedule_noconstraints     1.47  1300.6±15.21µs        ? ?/sec    1.00  887.8±192.92µs        ? ?/sec
build_schedule/500_schedule                   1.04    388.1±5.32ms        ? ?/sec    1.00    372.4±6.56ms        ? ?/sec
build_schedule/500_schedule_noconstraints     2.50     16.1±0.14ms        ? ?/sec    1.00      6.5±0.17ms        ? ?/sec
schedule/base                                 1.00     25.7±0.22µs        ? ?/sec    1.01     26.0±0.46µs        ? ?/sec

bors r+


Test memory fails :/

Both test memory examples throw the following error, incrementing the property number with each message (1/frame?)
Uncaught TypeError: Cannot read property '1' of undefined
    at SphereBufferGeometry.WebGLGeometries.onGeometryDispose (VM253 three.js:10380)
    at SphereBufferGeometry.EventDispatcher.dispatchEvent (VM253 three.js:103)
    at SphereBufferGeometry.exports.BufferGeometry.BufferGeometry.dispose (VM253 three.js:8851)
    at render (webgl_test_memory.html:91)
    at animate (webgl_test_memory.html:69)

https://htmlpreview.github.io/?https://github.com/semleti/three.ts/blob/master/examples/webgl_test_memory.html
https://htmlpreview.github.io/?https://github.com/semleti/three.ts/blob/master/examples/webgl_test_memory2.html

Solved.
There still might be bugs with memory, but the examples work, so 👍


Address MultiIndex problem with LightGBM

Closes #1710
Added context on the issue here: https://github.com/alteryx/evalml/issues/1710#issuecomment-770035322. Not sure if this is the best way to solve this issue so would love any thoughts and suggestions :')

@freddyaboulton Ooo, that's a really interesting idea! I wonder what that would mean for the LightGBM estimators specifically. Right now, we take advantage of the auto feature to determine categorical columns, which requires a pandas DF (and us making sure that the categorical cols as determined by WW are set to category dtype). (https://lightgbm.readthedocs.io/en/latest/pythonapi/lightgbm.LGBMClassifier.html#lightgbm.LGBMClassifier) If we converted to numpy, we'd have to manually determine the mapping from col name to integer index for the numpy array and pass that along (assuming also dtypes don't get funky). Could be an interesting alternative to explore though if we see more problems like this crop up / more edge cases hehe.


answer key containers class

Done for accounting recipe.

Closing since the class will be changed to os-solutions-container in the recipe to be consistent with the rest of the books in Kitchen.


Format Currency Extension

Fixes Issue #159
Screenshots :

Please consider this contribution under GSSoC'22.

@priyanshi1282 nice work!!

Thanks  @whiteHatpro  fir the compliment 😄

@priyanshi1282 you have added the extension twice revert it, also the image is not loading properly in the readme file
Kindly do the required changes suggested

@priyanshi1282 ??

@harshita214  sorry for the delay, I'll b updating it today itself

@priyanshi1282 I have put labels on #206 instead of this


Access to file after removing the Microsoft Account

Storage Explorer Version:
Build Number:
Platform/OS:
Architecture:
Regression From:
Bug Description

Steps to Reproduce

Expected Experience

Actual Experience

Additional Context

@Badrikrish can you give us some more information please?

Used the Microsoft account to login to the storage explorer.
Able to get the files uploaded to the cloud from the storage accounts.
Removed the account from the settings.
Still able to open the files that are available in the storage account associated with this account.

Let me know if you need more inputs and or screen shots.

Can you clarify what type of account you are talking about?
Are you signing in and adding a user account?
Are you attaching a storage account?

Hi Matt,
Find the details inline. Let me know if you have any more questions.
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: Matthew<EMAIL_ADDRESS>Sent: Tuesday, March 3, 2020 12:32 AM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [microsoft/AzureStorageExplorer] Access to file after removing the Microsoft Account (#2656)
Can you clarify what type of account you are talking about?
It is a free subscription and I have created a Blob Account and had a file in it added using the portal.
Are you signing in and adding a user account?
Yes I signed in to the storage explorer using my credentials.
Are you attaching a storage account?
No, I am connecting to the already created account in the storage explorer.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/AzureStorageExplorer/issues/2656?email_source=notifications&email_token=AOTRT6X7PISLUTREQOIG2PDRFP7FZA5CNFSM4KYQJEZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENQQ7FQ#issuecomment-593563542, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOTRT6QSWMVDVWLLBYF2UIDRFP7FZANCNFSM4KYQJEZA.

Ok. Are you continuing to access the data via the explorer on the right hand side? What we see is that if you:

Sign in
Open a container (or in other words, double click on a container)
Sign out

Then the view that was opened in step 2 remain opens. Is that what you are trying to report?

Find the below steps to reproduce
1.Open Microsoft Azure Storage Explorer
2.Connect to Azure using the Azure User name and Password
3.Click apply from the account management blade
4.Expand the Storage Explorer tree from the subscription, Expand Blob Containers, Click on the container. Now I can see the uploaded files on the right pane
5.Double click and open the file, it opens up and I can see the content. Close the file.
6.Click on the Account Management icon to the top left vertical pane and click remove (Microsoft Account)
7.The files are still available on the right pane and I am still able to open. Can we disable or remove access when we remove the associated microsoft account when removed?
From: Matthew<EMAIL_ADDRESS>Sent: Monday, March 9, 2020 11:27 PM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [microsoft/AzureStorageExplorer] Access to file after removing the Microsoft Account (#2656)
Ok. Are you continuing to access the data via the explorer on the right hand side? What we see is that if you:

Sign in
Open a container (or in other words, double click on a container)
Sign out

Then the view that was opened in step 2 remain opens. Is that what you are trying to report?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/AzureStorageExplorer/issues/2656?email_source=notifications&email_token=AOTRT6WWZIYDCJHBGZS54HDRGU3XXA5CNFSM4KYQJEZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOILACQ#issuecomment-596684810, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AOTRT6SIF4AK7GVLSBNEOALRGU3XXANCNFSM4KYQJEZA.

Find the below steps to reproduce
1.Open Microsoft Azure Storage Explorer
2.Connect to Azure using the Azure User name and Password
3.Click apply from the account management blade
4.Expand the Storage Explorer tree from the subscription, Expand Blob Containers, Click on the container. Now I can see the uploaded files on the right pane
5.Double click and open the file, it opens up and I can see the content. Close the file.
6.Click on the Account Management icon to the top left vertical pane and click remove (Microsoft Account)
7.The files are still available on the right pane and I am still able to open. Can we disable or remove access when we remove the associated microsoft account when removed?

Ok, got it. Yes we see this too. We'll look into closing the tabs automatically when you sign out. Also, incase you are also concerned about the downloaded file, we will not look into deleting that for you.

Let's also address https://github.com/microsoft/AzureStorageExplorer/issues/426 at the same time. Or at least try to.


Add 'genotype' column for new entities (automatically identified or manually added by authors)

This applies to:

transgenes
strains (explore this feature)

Add also species field
Use list of species specified in the  species field on first tab to autocomplete the fields for this


Bug of HttpUrl.Builder addQueryParameter

What kind of issue is this?

[x] Bug report. If you’ve found a bug, spend the time to write a failing test. Bugs with tests
get fixed. Here’s an example: https://gist.github.com/marti1125/b15eec84d87ccfe1d6e89baf511be6da

the problem is with the final result of  urlquery.url().getQuery()
%22bin%22=%22411111%22 you can check in https://ostermiller.org/calc/encode.html
but I need bin="411111"
my solution was
String queryParams = (urlquery.url().getQuery() != null) ? "?"+urlquery.url().getQuery().replace("%22","") : "";

Can you express this in a test case? It's not really clear where you think the problem is.

hello @JakeWharton https://gist.github.com/marti1125/dcb458f71b5eb1d229747cc58177b6e8
the output is:
%22limit%22=10
?limit=10
Response{protocol=http/1.1, code=200, message=OK, url=https://api.culqi.com/v2/charges?limit=10}
Process finished with exit code 0
%22 is not necessary I have to replace("%22","")

You’re including a " character in your parameter name. Don’t.

hello @swankjesse but this part I added builder.addQueryParameter(pair.getKey().toString(), pair.getValue().toString()); "param" is a String.
Could you explain  me what is wrong with my code? please

ups I fixed map.put(keyValue[0].replace(""",""), keyValue[1].replace(""",""));
Thanks!!!


chore(deps): update dependency @types/semver to v7.3.3

This PR contains the following updates:

Package
Type
Update
Change

@types/semver
devDependencies
patch
7.3.2 -> 7.3.3

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

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

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

:tada: This PR is included in version 5.1.16 :tada:
The release is available on:

v5.1.16
GitHub release

Your semantic-release bot :package::rocket:


semantic version v3.2.0

In your changelog you said the location of the asset has moved.
But doesn't that make it a major version change to the bower package since I can't just simply update the package and move on with the new features and bugfixes?
Just thinking if the version represents the bower package's version or the bootstrap version.

The Bower package is versioned the same as bootstrap-sass itself, not separately.


[Firefox] New WoSign and StartCom certificates will no longer be accepted (Affecting)

https://www.fxsitecompat.com/en-CA/docs/2016/new-wosign-and-startcom-certificates-will-no-longer-be-accepted/

Closing this out since it sounds like it doesn't affect us @cvrebert. (Going to peep the rest of these, too, and close out what I think doesn't apply fwiw.)