bash script fails with "/bin/bash: bad interpreter: no such file or directory"

Version
Microsoft Windows [Version 10.0.22000.613]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
5.10.16
Distro Version
Ubuntu 20.04
Other Software
No response
Repro Steps
I just installed Ubuntu from the Microsoft Store and then enabled WSL and rebooted
Expected Behavior
I would expect to be able to run this script (i can run it fine in git bash)
Actual Behavior
stevenbrix@poseidon:/mnt/c/dev/browser/core-tools$ ./scripts/core-dev-setup
-bash: ./scripts/core-dev-setup: /bin/bash^M: bad interpreter: No such file or directory
Diagnostic Logs
No response

nvm closing this since the issue is line endings


Add show-hide capacity to the frontend toolkit

Support shown/hidden fields
Adds the ability for a boolean question to show or
hide 'text' or 'textarea' input fields.
Docs for usage are here.
It's worth pointing out that the scope of this is
limited to these particular input types because
these are how we plan to do 'nice-to-have'
questions. It could be expanded to support other
fields if needed, but for now, these changes aren't
really general things we need to do.
Also worth pointing out that the hidden attribute
will create a js-hidden class which will only
truely hide the element if javascript is enabled.
Bumped the govuk_frontend_toolkit version
This is the version that actually contains the
show/hide javascript.  It's a bit pointless updating
this because our toolkit doesn't actually use this
(the frontend apps are responsible for initiating
the javascript), but I've just updated it here for
completeness.
I thought about doing an example of this being used,
but it seems like it doesn't really work the way
oour other examples do (it includes two different
types of input fields), and it's also not really a
feature that we've built.  Instead, the example to
check out should be the one in the govuk frontend
toolkit.

👍
Let's bump the minor version though.

This is so good it deserves to be featured in the documentation. On the selection buttons page perhaps?
Also, @allait, isn't this introducing a feature and so 19.1.0 works?

I've added an example to the docs pages:

This required some hacking in the code that generates the pages but nothing that should break anything. Should be good to merged now.


Error while creating dynamic group using powershell

Guys, I  am trying to create Azure dynamic groups using powershell, but getting an error.
PS C:\WINDOWS\system32> New-AzureADMSGroup -DisplayName "Dynamic Group 01" -Description "Dynamic group created from PS" -MailEnabled $False -MailNickName "group" -SecurityEnabled $True -GroupTypes "DynamicMembership" -MembershipRule "(user.department -eq ""Management"")" -MembershipRuleProcessingState "On"
New-AzureADMSGroup : A parameter cannot be found that matches parameter name 'MembershipRule'.
At line:1 char:197
+ ... Enabled $True -GroupTypes "DynamicMembership" -MembershipRule "(user. ...
+                                                 ~~~~~~~~~~~~~~~
    + CategoryInfo         : InvalidArgument: (:) [New-AzureADMSGroup], ParameterBindingException
    + FullyQualifiedErrorId : NamedParameterNotFound,Microsoft.Open.MSGraphV10.PowerShell.NewMSGroup

I tried installing and importing  AzureADPreview.
But issue still persists.

@Girishvtm Thank you for your feedback! Since this channel is for driving improvements towards MS Docs, could you provide the URL of the specific documentation that your feedback is applicable to? That way, we are able to make sure your feedback is addressed by the right team.

hey @Girishvtm it looks like you're referring to this doc : https://docs.microsoft.com/en-us/powershell/module/azuread/new-azureadmsgroup?view=azureadps-2.0
Per the code :
New-AzureADMSGroup -DisplayName "Dynamic Group 01" -Description "Dynamic group created from PS" -MailEnabled $False -MailNickName "group" -SecurityEnabled $True -GroupTypes "DynamicMembership" -MembershipRule "(user.department -contains ""Marketing"")" -MembershipRuleProcessingState "On"

Id                            : 9126185e-25df-4522-a380-7ab697a7241c
Description                   : Dynamic group created from PS
OnPremisesSyncEnabled         :
DisplayName                   : Dynamic Group 01
OnPremisesLastSyncDateTime    :
Mail                          :
MailEnabled                   : False
MailNickname                  : group
OnPremisesSecurityIdentifier  :
ProxyAddresses                : {}
SecurityEnabled               : True
GroupTypes                    : {}
MembershipRule                : (user.department -eq "Marketing") MembershipRuleProcessingState : Paused

Per the doc here : https://docs.microsoft.com/en-us/powershell/azure/active-directory/install-adv2?view=azureadps-2.0
Please utilize the general availability one as the preview one is not meant to be in full working order. Using the command :
Install-Module AzureAD

To uninstall the preview one. Using the command :
Uninstall-Module AzureADPreview

hey @Girishvtm it looks like you're referring to this doc : https://docs.microsoft.com/en-us/powershell/module/azuread/new-azureadmsgroup?view=azureadps-2.0
Per the code :
New-AzureADMSGroup -DisplayName "Dynamic Group 01" -Description "Dynamic group created from PS" -MailEnabled $False -MailNickName "group" -SecurityEnabled $True -GroupTypes "DynamicMembership" -MembershipRule "(user.department -contains ""Marketing"")" -MembershipRuleProcessingState "On"

Id                            : 9126185e-25df-4522-a380-7ab697a7241c
Description                   : Dynamic group created from PS
OnPremisesSyncEnabled         :
DisplayName                   : Dynamic Group 01
OnPremisesLastSyncDateTime    :
Mail                          :
MailEnabled                   : False
MailNickname                  : group
OnPremisesSecurityIdentifier  :
ProxyAddresses                : {}
SecurityEnabled               : True
GroupTypes                    : {}
MembershipRule                : (user.department -eq "Marketing") MembershipRuleProcessingState : Paused

Per the doc here : https://docs.microsoft.com/en-us/powershell/azure/active-directory/install-adv2?view=azureadps-2.0
Please utilize the general availability one as the preview one is not meant to be in full working order. Using the command :
Install-Module AzureAD

To uninstall the preview one(You'll need to run PS in Admin mode). Using the command :
Uninstall-Module AzureADPreview

hey @Girishvtm it looks like you're referring to this doc : https://docs.microsoft.com/en-us/powershell/module/azuread/new-azureadmsgroup?view=azureadps-2.0
Per the code :
New-AzureADMSGroup -DisplayName "Dynamic Group 01" -Description "Dynamic group created from PS" -MailEnabled $False -MailNickName "group" -SecurityEnabled $True -GroupTypes "DynamicMembership" -MembershipRule "(user.department -contains ""Marketing"")" -MembershipRuleProcessingState "On"

Id                            : 9126185e-25df-4522-a380-7ab697a7241c
Description                   : Dynamic group created from PS
OnPremisesSyncEnabled         :
DisplayName                   : Dynamic Group 01
OnPremisesLastSyncDateTime    :
Mail                          :
MailEnabled                   : False
MailNickname                  : group
OnPremisesSecurityIdentifier  :
ProxyAddresses                : {}
SecurityEnabled               : True
GroupTypes                    : {}
MembershipRule                : (user.department -eq "Marketing") MembershipRuleProcessingState : Paused

Per the doc here : https://docs.microsoft.com/en-us/powershell/azure/active-directory/install-adv2?view=azureadps-2.0
Please utilize the general availability one as the preview one is not meant to be in full working order. Using the command :
Install-Module AzureAD

To uninstall the preview one(You'll need to run PS in Admin mode). Using the command :
Uninstall-Module AzureADPreview

Thanks Frank for the response.
I will uninstall AzureAD Preview. We cannot create dynamic groups in Azure using powershell, am I right?

@Girishvtm Yes, you can create dynamic groups by utilizing the -GroupTypes parameter, please refer to the article made by the identity team for more information on how to to perform this task, https://techcommunity.microsoft.com/t5/Azure-Active-Directory-Identity/New-enhancements-to-the-AzureAD-PowerShell-2-0-preview-Manage/ba-p/245153
Please let us know if you have anymore questions otherwise we will be going ahead and closing this issue out end of business day Monday

Hi Frank,
I tried the same but I am unable to see the the property 'MembershipRule" in the command New-AzureADMSgroup.

@Girishvtm I don't understand, that is expected behavior? Please see :

from https://docs.microsoft.com/en-us/powershell/module/azuread/new-azureadmsgroup?view=azureadps-2.0

@Girishvtm I'm following up on this could you please respond in regards to the last comment?
Thanks!

@FrankHu-MSFT , When I type New-AzureADMSgroup - , I am able to see the properties(like Description, DisplayName, MailEnabled, Mailnickname, SecurityEnabled,GroupsTypes,Visibility,Verbose, Debug) in a list except MembershipRule.

Hey Girish, I sort of meant a picture of what you see. But do you see the AAD Group in the portal? And is it a dynamic group?

Hi Frank, Please find the screen shot of the issue. Please let me know if any additional info.

Thanks for your help.

Oh I understand now! The screenshot clarifies what the problem is. I'll forward this to the correct people accordingly. Thanks for the screenshot!

@Girishvtm
The documentation seems to be wrong here and we are currently engaging to fix this as soon as possible. Thanks for letting us know!
The MembershipRule is not currently supported in the New-AzureADMSGroup cmdlet of the AzureAD module.
This is because the underlying property in the Microsoft Graph (membershipRule) is only available on the "beta" endpoint. So this capability is only available in the AzureADPreview module.
You can install the AzureADPreview module side-by-side with the AzureAD module:
Install-Module AzureADPreview -Scope CurrentUser -AllowClobber
To be sure you're using the cmdlet from the correct module, unload the AzureAD module, and import the AzureADPreview module:
# Unload the AzureAD module (or continue if it's already unloaded)
Remove-Module AzureAD -ErrorAction SilentlyContinue
# Load the AzureADPreview module
Import-Module AzureADPreview

Then you should find you're able to create your dynamic group:

C:\> New-AzureADMSGroup -DisplayName "DisplayName" `
                        -MailNickname "Nickname" `
                        -MailEnabled $false `
                        -SecurityEnabled $true `
                        -GroupTypes @("DynamicMembership") `
                        -MembershipRule 'user.userType eq "Guest"' `
                        -MembershipRuleProcessingState "On"

Id                                   DisplayName        Description
--                                   -----------        -----------
<ID> <Dynamic-Group>

Please let us know if you have anymore questions within the scope of this github issue, otherwise we will be closing this out by end of day @Girishvtm thanks!

Thanks for the updates.

The documentation may still be incorrect here.
 still shows MembershipRule in one of the examples but even with version <IP_ADDRESS> of the AzureAD module the MembershipRule parameter is not found, it's still only in graph preview perhaps?
At least until the feature has been promoted to General from Beta, can we get some sort of banner in the docs file indicating the example will never work except with the AzureADPreview release? Or can the docs be updated accordingly? Quite confusing otherwise!

Hi @FrankHu-MSFT, I was facing the same issue and with installing the AzureADPreview module the object MembershipRule works. Thanks! =]
But it is almost two years later, why is the MembershipRule object not working on the AzureAD module only?

@FrankHu-MSFT @MohitVerma-MSFT this issue is not closed. Both the 2.0 general release and the 2.0 preview documentation list -MembershipRule as a parameter in the example, even though it is only available in the preview release. Please correct this as it is wasting people's time troubleshooting something that isn't available in the general release yet.

@Girishvtm
The documentation seems to be wrong here and we are currently engaging to fix this as soon as possible.

@FrankHu-MSFT FrankHu-MSFT
The lie detector determined that was a lie. 2 years later and neither the documentation nor the general release have been updated.

The lie detector determined that was a lie. 2 years later and neither the documentation nor the general release have been updated.

Hi @Belliqueu
please do this two steps:
Uninstall-Module -Name AzureAD
Install-Module -Name AzureADPreview -RequiredVersion <IP_ADDRESS>
After this steps the Property "-MembershipRule" should be available.


[FEATURE] Add  Option to Exit  the Game

Is your feature request related to a problem? Please describe.
After enter the game there is no option to Exit the game or return back to dashboard. In this time it's possible with the help of browser.
Describe the solution you'd like
I want to add one like a x sign to Exit the game and a homepage sign for directly return to the dashboard.
Describe alternatives you've considered
Apart from this I've no idea in this moment but any extra suggestion or feature welcome.
Additional context
EG;-

Hey @anikdhabal, this is an excellent idea, I think we should have a button named "Exit" or "Back" in the bottom right of the test page. And this button will basically reload the page (if we want the existing form values to be erased) or we can remove the test page elements and put the form element again.
I would like to know what you think about it, also please tell me if you want to work on this issue so that I can assign it to you.
Happy Contributing!

@prakhartiwari0 yes, I want work on this. pls assign me

Sure then @anikdhabal, start coding!

Can i Work on this

Can i Work on this

@ooh-shit You can help in the improvement of the PR made by @anikdhabal #93. I have shared a few suggestions there.

Hi, can you assign this to me, I'd love to work on this under GSSOC'23.

@oshinsaraf We need to wait for a day or two, let's if the assigned person replies, if they don't then we will need to assign the issue to somebody else.

Anybody interested should read the already happened discussion, take a look at the PR #93 and then share a message about how can they solve this issue.


fix: draft preview from panel not working

Draft previews are currently not working from within the panel, since the page() method is not able to find drafts. See https://getkirby.com/docs/reference/templates/helpers/page.
In order to only allow previews for logged in users the isReadable() method is used. See https://getkirby.com/docs/reference/objects/cms/page/is-readable.

@johannschopplich done!

Thanks!


Install error because of test target

Removing the [targets] section allows me to install it, but otherwise I get this error:
julia> Pkg.add(Pkg.PackageSpec(url="https://github.com/mohamed82008/KissThreading.jl"))
Updating git-repo `https://github.com/mohamed82008/KissThreading.jl`
ERROR: Expected value for target `test` to be a list of dependency names.

julia> versioninfo()
Julia Version 1.2.0-DEV.unknown
Commit 60457de* (2019-03-08 17:26 UTC)
Platform Info:
OS: Linux (x86_64-pc-linux-gnu)
CPU: Intel(R) Xeon(R) CPU E3-1220 v5 @ 3.00GHz
WORD_SIZE: 64
LIBM: libopenlibm
LLVM: libLLVM-6.0.1 (ORCJIT, skylake)

Works with me on Julia 1.1.

I can confirm that it works here with 1.1 as well.  But I tried the today's nightly binary and it failed there as well, so I guess there was either a regression or a change in how Project.toml is interpreted.

It's now working for me on current master branch.


TracingDriverConnectionFactory calls the deprecated method AbstractMySQLPlatform::getName()

Environment

PHP 8.2
sentry/sentry-symfony:4.8.0

Steps to Reproduce
AbstractMySQLPlatform::getName() was deprecated in https://github.com/doctrine/dbal/issues/4749 but is still used in Sentry\SentryBundle\Tracing\Doctrine\DBAL\TracingDriverConnectionFactory.
This triggers the following deprecation warning:
AbstractMySQLPlatform::getName() is deprecated. Identify platforms by their class. (AbstractMySQLPlatform.php:1209 called by TracingDriverConnectionF
actory.php:40, https://github.com/doctrine/dbal/issues/4749, package doctrine/dbal)

Doctrine team suggests to use the class name to identify the platform instead.
As far as I can see the only usage of the platform name is in Sentry\SentryBundle\Tracing\Doctrine\DBAL\TracingDriverConnection::getSpanTags().

Is this a duplicated of #579? In case it would be already fixed in #723

No, it isn't. It's a completely different deprecation, which involves the connection class and not the driver class. Actually, I think that the only solution to the problem is to implement a switch that checks the instanceof the platform and returns the expected name for each supported platform, even though it would now be hardcoded and thus less maintainable in the long term.

Would we really need a switch though? We could just use $databasePlatform::class. It just means that the db.system tag would change from mysql to \Doctrine\DBAL\Platforms\MySQL57Platform. Is that a problem?

Although not required, I tried to follow as much as possible the guidelines of OpenTelemetry regarding the Connection-level attributes, Since the specification is the best standard out of here in regards to distributed tracing, I think we should continue on this way.

The list of Sentry's span operations can be found here. It's mandatory to use these, as certain expectations in the product rely on them.

We're not talking about span operations, but about span tags :smiley:

I see. There is a movement away from creating tags automatically in the SDKs as well, I'll clarify internally. Most likely, we'll move this data to span.data.

@cleptric any updates?

We should move the span tag values over to span data. As Spans are not indexed in Sentry, this has no impact on the product.

I still don't understand what this has to do with this issue. Note I'm not arguing about whether it makes sense or not, but just about the fact that it's a completely different topic.

I would be OK to remove $databasePlatform as I don't see it adding this much value.

It was added because I was trying to follow the OpenTelemetry specification which requires this info to be available. Yes, maybe it doesn't add much value, but sometimes respecting some kind of standard is valuable 😉

PR is up https://github.com/getsentry/sentry-symfony/pull/743

@ste93cry we try to follow the otel conventions for span attributes as much as possible in span data. This is documented here.
Our span data conventions are essentially a superset of otel's, because they are lacking some conventions for mobile/browser and also don't have everything documented for the web server use case also.

I assume that your answer is about moving the information from tags to data. As I said before, I don't argue about doing it, to be honest I care little about it, I was just pointing out that the problem being reported in this issue has nothing to do with such change, therefore we should just care about solving the deprecation in the SDK. Any other thing should be done in a different PR.


Feature: Custom image scale?

I actually sort of have this in my fork already, but I think this library is pretty great and can be expanded even further. One small thing would be for someone to define the scale of the image within the white background.
Some images just look better if they were a bit bigger.
for instance, by default we have:
alertViewVector.frame = CGRectMake(alertViewContents.frame.size.width/2 - 15.0f,
                                           -15.0f,
                                           30.0f,
                                           30.0f);

We could maybe do something like:

size = 30.0f * scale;

alertViewVector.frame = CGRectMake(alertViewContents.frame.size.width/2 - size/2,
                                           -size/2,
                                           size,
                                           size);

The scale would ideally be a number that defaults to 1 and goes up to 2, because at 2 it is the size of the entire circle.
Just an idea to think about. I write in Swift pretty exclusively, so the above might be sloppy/invalid Objective-C lol, but you get the point.

Hi there,
Thank you for suggesting this, I can see how it could be useful for some developers.
I've implemented your method in FCAlertView V1.2.6. You can update and enjoy the release. Here's how it works:
alert.customImageScale = 1.5; // or set 1.5 to desired range of scale between 0 and 2

Thanks for using FCAlertView! Also, if you'd like to contribute and update the Swift version of FCAlertView, please let me know.
Cheers,
Nima


Replace TODO URL with release tab for supported versions

This would assume that a quick version table gets added to the release page after attaching the bindings during the release. EX:

OS
Node

Windows x86 & x64
0.10, 0.12, 1, 2, 3, 4, 5

OSX x64
0.10, 0.12, 1, 2, 3, 4, 5

Linux x86 & x64
0.10, 0.12, 1, 2, 3, 4, 5

This probably a decent MVP way to do this.


Can’t change anndata dimensions

From https://scanpy.discourse.group/t/cant-change-anndata-dimensions/519/1:

Hello
I clustered some cells using the top 3000 genes with the highest variance, removed some cell types I am not interested in and now I want to cluster them again using a new set of top variable genes.
My object has 78016 obs and 3000 vars. However, all genes are still there in adata.raw.X. Because I want to select these new genes, I have tried using adata2.X=adata2.raw.X. This returns the message ValueError: Data matrix has wrong shape (78016, 22633), need to be (78016, 3000).
Anyway to overcome this?

I have same question as above: after normalizing, filtering and/or scaling genes, how can I convert back to the original count matrix (assuming no cells have been removed). This is trivial in Seurat switching between “counts” (with all original genes from input matrix) and “scaled”(with only highly variable genes).

Closing - moved to scverse discourse.


erase: INCORRECT_ELEMENT_OF_SET

Traceback (most recent call last):
File "/home/username/mysql_ch_replicator/mysql_ch_replicator/main.py", line 6, in
main()
File "/home/username/mysql_ch_replicator/mysql_ch_replicator/mysql_ch_replicator/main.py", line 152, in main
run_db_replicator(args, config)
File "/home/username/mysql_ch_replicator/mysql_ch_replicator/mysql_ch_replicator/main.py", line 98, in run_db_replicator
db_replicator.run()
File "/home/username/mysql_ch_replicator/mysql_ch_replicator/mysql_ch_replicator/db_replicator.py", line 178, in run
self.run_realtime_replication()
File "/home/username/mysql_ch_replicator/mysql_ch_replicator/mysql_ch_replicator/db_replicator.py", line 379, in run_realtime_replication
self.handle_event(event)
File "/home/username/mysql_ch_replicator/mysql_ch_replicator/mysql_ch_replicator/db_replicator.py", line 407, in handle_event
self.upload_records_if_required(table_name=event.table_name)
File "/home/username/mysql_ch_replicator/mysql_ch_replicator/mysql_ch_replicator/db_replicator.py", line 547, in upload_records_if_required
self.upload_records()
File "/home/username/mysql_ch_replicator/mysql_ch_replicator/mysql_ch_replicator/db_replicator.py", line 571, in upload_records
self.clickhouse_api.erase(
File "/home/username/mysql_ch_replicator/mysql_ch_replicator/mysql_ch_replicator/clickhouse_api.py", line 184, in erase
self.execute_command(query)
File "/home/username/mysql_ch_replicator/mysql_ch_replicator/mysql_ch_replicator/clickhouse_api.py", line 67, in execute_command
self.client.command(query)
File "/home/username/mysql_ch_replicator/mysql_ch_replicator/.venv/lib/python3.11/site-packages/clickhouse_connect/driver/httpclient.py", line 351, in command
response = self._raw_request(payload, params, headers, method, fields=fields, server_wait=False)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/username/mysql_ch_replicator/mysql_ch_replicator/.venv/lib/python3.11/site-packages/clickhouse_connect/driver/httpclient.py", line 462, in _raw_request
self._error_handler(response)
File "/home/username/mysql_ch_replicator/mysql_ch_replicator/.venv/lib/python3.11/site-packages/clickhouse_connect/driver/httpclient.py", line 385, in _error_handler
raise OperationalError(err_str) if retried else DatabaseError(err_str) from None
clickhouse_connect.driver.exceptions.DatabaseError: HTTPDriver for http://localhost:8123 received ClickHouse error code 124
Code: 124. DB::Exception: Incorrect size of tuple in set: 8 instead of 2: While processing _CAST(if((departments, termine) IN (73024, 580, 73024, 579, 73024, 645, 73024, 646), _CAST(0, 'UInt8'), _row_exists), 'UInt8'). (INCORRECT_ELEMENT_OF_SET) (version <IP_ADDRESS>7 (official build))
I fixed it by chanching this:
git diff mysql_ch_replicator/clickhouse_api.py
diff --git a/mysql_ch_replicator/clickhouse_api.py b/mysql_ch_replicator/clickhouse_api.py
index 6dd3e6c..253d26b 100644
--- a/mysql_ch_replicator/clickhouse_api.py
+++ b/mysql_ch_replicator/clickhouse_api.py
@@ -174,7 +174,8 @@ class ClickhouseApi:
    def erase(self, table_name, field_name, field_values):
        field_name = ','.join(field_name)
-        field_values = ', '.join(list(map(str, field_values)))
+        field_values = ', '.join(f'({v})' for v in field_values)
        query = DELETE_QUERY.format(**{
            'db_name': self.database,
            'table_name': table_name,

This is the table in mariadb:
CREATE TABLE Termin_to_Department (
departments int(11) NOT NULL,
termine int(11) NOT NULL,
PRIMARY KEY (departments,termine),
KEY IDX_FC0880B716AEB8D4 (departments),
KEY IDX_FC0880B7168C0A8F (termine),
CONSTRAINT FK_FC0880B7168C0A8F FOREIGN KEY (departments) REFERENCES task_Department (id),
CONSTRAINT FK_FC0880B716AEB8D4 FOREIGN KEY (termine) REFERENCES Termin (id)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb3 COLLATE=utf8mb3_unicode_ci

Fixed in 0.0.43, please check if it helped


Guard all the vaules in the Metrics for undefined.

We've seen a few cases where the WebRTC metrics report contains undefined values when the stream hasn't started yet. JavaScript, when we send to the backend, drops undefined members of the object, for example:
> JSON.stringify({rtc_frames_dropped: undefined})
< '{}'

This will fail the backend validation logic and cause an error to get emitted to the front-end reporting the missing key. I don't think this does anything to the session, but it's an error we can avoid by guarding more of the statistics with a || 0.
Some of the values had this already, this just adds a few more.

This fixes a console error that was found in https://github.com/KittyCAD/modeling-app/issues/867


Palindrome Linked List Solution#42

@rishiparashar02 Please add problem statement add approach in your file and rename as protocol.

@rishiparashar02 Please add problem statement add approach in your file and rename as protocol.

Thank you sir for your remark, I have updated all the things you mentioned. Kindly let me know if i am lagging somewhere else.
Thank you!


Importer sometimes skips duplicates without prompting for a choice

Beets will skip sometimes duplicate albums when a prompt is needed.  I found this to happen quite often with large imports, especially after after the 20-30 mark where more than 50% gets trumped unjustly. The problem is rather annoying and potentially dangerous, as it creates empty folders without bringing in the audiofiles. The only way to know that something went wrong is check the log file and correct manually, which due to the high occurrence is not realistically feasible.
For every release, I always have a mp3 and flac folder. When importing, it should detect the duplicate and then ask whether I want to keep, release or remove, which it often does. But not always. The result is that I get both a folder for the flac and the mp3 version (I have paths setup to separate them), but one of them won't have any audiofiles, although the .cue .log file might be there from copyartifacts. The output on screen is "This album is already in the library! All duplicates are empty, we ignore them". In the logfile, I get a "duplicate-replace". This only happens with duplicates that get accepted automatically (t my knowledge). When I to have manually apply, it will give the duplicate prompt.
I did some basic troubleshooting. When checking the database afterwards with ls, I only get one of the releases, either the flac or the mp3. When manually adding just the missing one, the duplicate question pops up and it gets added without an issue, showing every file twice with ls. When removing the database entry for that release (both mp3 files and flac files), and re-importing both folders manually again, again there is no problem.
I also tried it without being threaded: no issue.
My guess is that when working with a large import copying the files to a new location (which is what I'm doing), a large queue of files to be copied might stack up, especially when there are a lot of highly recommended releases that get processed faster than they can get copied. When such a copy-backlog is present, and 2 duplicate folders (with different formats) come immediately one after the other, the first one gets processed normally and the copy operation is added to the copy-queue. When the second one gets processed, beets detects a duplicate in the database, checks the folder but finds it still to be empty (because the copy operation is still in the backlog) and does a duplicate-replace.
To reproduce the bug is not going be very straight forward, I'm afraid. I suggest preparing a folder with plenty of releases that get recognized easily and automatically get tagged as highly recommended (about 20 full size albums should be sufficient), followed by a few releases for duplicate albums but different formats. Probably the easiest way to do it, is to a run an import on a basic folder with a lot of releases till you have done about 40. Remove all the ones that needed input. Add the duplicate albums/different formats. If necessary, rename the folders of these albums to make them start with a "z" or something at the back of the alphabet, as those will be processed last by the importer. I have the error when performing an import-copy from one HDD to another. SSD might be too fast to easily reproduce the error.

To quote myself on the mailing list:

I see two problems here:

Albums should never be added empty. The insertion of the album structure should happen atomically, in a transaction that prevents any other threads from seeing an empty album. The transaction handling is pretty careful about this, but somehow that's not working.
There is a race from when an album is approved by the user and when it is available for conflict detection. (The UI happens in the user_query thread; the album is actually added in apply_choices.) I don't remember when we split this up, but it was a mistake and we need to revert it.

Do you think you could try putting together a bundle of music that (at least sometimes) triggers the problem for you? That would be an amazing starting point for getting to the bottom of the problem.

I think I could, but I'm afraid it would not suitable for distribution. PM?

Yes, good thinking! My email's on my GitHub profile page.

I wanted to create a folder with as little releases as possible in it, but that was still able to recreate the error so I could forward that. So I've copied the folders which caused the issue to another computer (for easier editing) and gave it another go so I could trim out the releases that needed input for quicker testing: this time all went through without error.:-(
I have no idea what might be causing this difference, except maybe for the fact that the disk on the second pc is faster, although it did manage to gather up a high copy-backlog as well (after asking the last release, it still had 10-15 minutes of copying left). First is Windows 7, the second Windows 10. First is a 2-disk operation, the second a 1 disk.
Went back to the first pc, for a new test after deleting the library (to make sure there was nothing there that might be causing the issue): same issue again. I have to add, on this run the issue started  only after processing about 15GB of releases (about as much mp3 as FLAC), but when it started, it was very frequent.
I'll be copying everything to pc 1 and doing my tagging from there, keeping my eye on the logfile for duplicate-replace.
If you have any suggestions or requests, please let me know. I could create a bundle of music as requested, but it'll be huge to upload and might not reproduce the same issue on every pc, so probably not the best way to go.

Huh, that's certainly frustrating. Thanks for looking into it!
If you can even get this to reproduce consistently on one machine, that could be helpful by itself. If, for example, you can get a verbose log from a run that exhibits the error, we can start there. I'll try applying a fix and ask you to run it again to see if we've actually fixed it.

Good news. After numerous lengthy test runs and a lucky observation, I was able to compile a bundle of music and a tagging procedure to consistently reproduce the error on both pc's, narrowing down the circumstances in which it occurs to some quite specific parameters. Instead of being caused by a copy queue, it seems to be triggered by a queue of releases that need input.
Basically the releases I'm testing in 1 import batch with autotagging, are all pairs of mp3 versus flac duplicate albums that get recognized as highly recommended. When running the import, it hits the first and second release and detects a duplicate. Instead of immediately responding, keep it waiting for a while so that other threads start resolving the other albums as well, creating a queue of releases that need input. When after a while, you finally do select to keep the duplicate, it scrolls really fast through the inputqueue to the last question in the queue, skipping all the ones in between with a skip-replace and the "All duplicates are empty, we ignore them" remark in the output. By doing so, it creates a folder for the duplicate with no audio files but with the artifacts in it if the plugin is activated. If the copyartifacts plugin is not activated, no extra folder is created. But in both instances, beets is skipping the duplicate I want to (have an option to) keep.
Running in verbose only creates a folder for the first hit, then displays an error "state file could not be read: [Errno 2] No such file or directory: u'C:\Users\couli\AppData\Roaming\beets\state.pickle'" , then the rolls through to the last release without errors and then hangs, requiring a KeyboardInterrupt.
FYI:
I tried it on both my personal config as the default: same issue.
I had beets create a new database on my test runs to eliminate possible issues with the database. I also deleted the state.pickle at every run.
I'll send you the music files, together with my log files and config files, so you can have a look yourself. And thank you in advance for doing so! To be honest, with this bug, I can't really use beets. I just have too many duplicates. And although I'm still very new to beets, I did really learn to appreciate all it can do and represents.
Let me know it if there is anything I can do to help.
Ekjaker

Perfect! Thank you so much for doing all this investigation; I'll see what I can find using your replication case.

Thank you again for getting together a replication case! This worked perfectly.
I tracked down the bug. It was an unhappy interaction between two pieces of code that I didn't like much anyway:

An explicit way of keeping track of recently-imported albums for the purpose of duplicate detection without adding them to the library. This was always fragile and, fortunately, unnecessary with the current importer design.
The UI code that decided to skip asking about duplicates when they were all empty. This was erroneously classifying recent conflicts as the all-empty case. (For the record, my original hypothesis was wrong: we were not, in fact, adding empty albums to the database.)

Removing either of these would have fixed the problem, but I dove in and removed them both. :smiley: This should fix the race you were seeing.

Fantastic! I found number 2 as well and was already thinking of going all dirty on the code and just commenting that part out, but this obviously beats that.
 Or is it better to just wait for the new 1.3.6 release? Actually, never mind.  :-) Should be doable with all the info in the wiki. And maybe in the future I just might try some small coding changes myself if I can find the time to invest in Python.
Thank you again for fixing this, and even more, doing it so fast!!


Hide explorer by default

VSCode Version: 1.6.1
OS Version: Windows 10 1607

How do we make explorer not to forcibly show when opening a file? It's wasting space. Can't find anything related with hide explorer, show explorer, hide sidebar, show sidebar with in:title. There is some "explorer.autoReveal" in settings, but setting it to false doesn't seem to do anything.

Are you saying that when you open a new vscode window, the sidebar is always visible, even if it was closed last time? This is probably #13825. You can set "telemetry.experiments.showDefaultViewlet": false as a workaround.


Add benchmarks of IDX variants.

Need to add benchmarks of IDX variants.

worth remembering with these benchmarks that it's not necessary intended to be faster, it's also a copy-less way of performing sparse operations, so would be a lower-memory option


[ios] add feature picker

标准
http://gitlab.alibaba-inc.com/Jerry/api/blob/master/picker.md
测试用例
[picker](单选picker例子

Current coverage is 88.44% (diff: 100%)

Merging #1707 into ios-feature-20161208 will not change coverage

@@           ios-feature-20161208      #1707   diff @@
======================================================
  Files                        41         41
  Lines                      2233       2233
  Methods                     354        354
  Messages                      0          0
  Branches                    500        500
======================================================
  Hits                       1975       1975
  Misses                      258        258
  Partials                      0          0

Powered by Codecov. Last update f8efcae...7ecc264


Add , to json file

Without the comma the json code will fail

@crazy-max Is there a better way to add a custom script without manually modifying the json file?

@usha-mandya scripts is a map so I think the current assumption to add this script is correct.  , "debug": is confusing.

We could also use jq with something like:
jq '.scripts + {"debug": "nodemon --inspect=<IP_ADDRESS>:9229 server.js"}' < package.json

But don't think it's worth it.

@crazy-max yeah, so we were discussing "how can we make this less error-prone"? We can't show the whole package.json because it's generated (and way too large to show) - otherwise we could've shown "this is what your file should look like".
Editing JSON can be error-prone, so I was thinking "perhaps rpm has a command to update it" (something like npm script add --name "debug" --command "the command to run" so that it would just "work"
I guess the best alternative then would be to call out "make sure the JSON is valid after you edit it" (or something along those lines)

because it's generated (and way too large to show)

Perhaps it's still an option to show just the scripts section of the file (if that section always looks the same) to show what it must look like after editing

Yeah with https://github.com/docker/docker.github.io/pull/14423#issuecomment-1078946631 it would look like this in output:
{
  "build": "ncc build src/main.ts --source-map --minify --license licenses.txt",
  "lint": "eslint src/**/*.ts __tests__/**/*.ts",
  "format": "eslint --fix src/**/*.ts __tests__/**/*.ts",
  "test": "jest --coverage",
  "all": "yarn run build && yarn run format && yarn test",
  "debug": "nodemon --inspect=<IP_ADDRESS>:9229 server.js"
}

with https://github.com/docker/login-action/blob/master/package.json

Also if someone has npx installed he could:
$ npx npm-add-script \
  -k "debug" \
  -v "nodemon --inspect=<IP_ADDRESS>:9229 server.js" \
  --force

@thaJeztah @crazy-max Thanks for weighing in on this.
@YgorPerez Thanks for the PR. Discussed this PR internally and decided not to go ahead with adding , before debug as it can cause confusion and scripting this would also mean that we are complicating this. Closing this PR here without accepting your changes for this reason. Feel free to continue the conversation after I close.


Added Help Page

Related Issue
#2480
Fixes:  #(issue no.)
Description
Added the help page with FAQ, Contact and Feedback link
Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)

Checklist:

[x] I have made this change from my own.
[x] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

@huamanraj Please check it


Newest List data is frozen

Step to reproduce:

go to Landing page
Newest List component & Latest Sales show data from 4 days ago
when I click on "show more", I have more recent data in the gallery

https://user-images.githubusercontent.com/9987732/172561459-9b072342-b09f-419a-95d2-a40aaecca6bc.mov

👋

Is there anything update on /rubick/006/ recently ? cc @vikiival

Or maybe nothing happened on the chain?

Or maybe nothing happened on the chain?

Emm, I do not think so. It's been more than a week. 👀

Emm, I do not think so. It's been more than a week.

Then best thing is to verify the queries with 004 vs 006 or subsquid vs subquery

Emm, I do not think so. It's been more than a week.

Then best thing is to verify the queries with 004 vs 006 or subsquid vs subquery

That's a good idea. I tested this query on 004 and it works well, so there must be something wrong with 006.
004 result:

Could this be cache related?

Solved in kodadot/rubick#98

Problem was that kodadot/rubick#87 introduced average as bigint and App was unable to recover.
Try it yourself
const a = '490000.00'
BigInt(a)


[draw]FPS does not render when debug mode is enabled

[draw]FPS does not render when debug mode is enabled. Looks to be caused by ShapeRenderer, however FPS should simply draw on top of that.

Closing as duplicate of #1


Version packages

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and publish to npm yourself or setup this action to publish automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
<EMAIL_ADDRESS>Patch Changes

c628b63: Swap out performance ponyfill

<EMAIL_ADDRESS>Patch Changes

029c42f: Publish new package name for material-ui interactors
Updated dependencies [c628b63]

<EMAIL_ADDRESS>

Blocked by https://github.com/thefrontside/interactors/pull/70


FR: Show in Folder

It would be useful if the extension list had a button to open the source folder of the individual extension, to more conveniently make changes to the extension code and assets.

Very good idea!

Another related idea; Open in editor


Where's the documentation?

Is your feature request related to a problem? Please describe.
Unable to find documentation.
Describe the solution you'd like
A link to a working docs site, or at least a reference to the documentation in the docs-site directory would be useful in the README. We want to know about all the available props, etc. without digging through the source if possible.
Describe alternatives you've considered
I've tried running everything locally, including docs-site, but I get a Package subpath './lib/tokenize' is not defined by "exports" in ... error.
Additional context
Maybe I just can't find the docs site? I know https://reactdatepicker.com/ works, but it doesn't document all the props and I specifically needed to know about the open prop to be able to manually open and close the component. I only found it thanks to ChatGPT4 somehow knowing about it.

@Stan-Stani is this what you're looking for: https://github.com/Hacker0x01/react-datepicker/blob/master/docs/datepicker.md?
It is mentioned in the Configuration section.

@Stan-Stani is this what you're looking for: https://github.com/Hacker0x01/react-datepicker/blob/master/docs/datepicker.md?
It is mentioned in the Configuration section.

Ah, my bad. I guess there comes a time when skimming readmes is not enough.

Took me a while to find it too 😄


h1 and h2 outputs

Just an FYI, the h1 and h2 outputs in the results file appear to be h12 and h22.

Thank you. I will explain it better in the user manual.


agones 1.13.0 support. removed deprecated chart. temporarily removed modular install

Issue #, if available:
Description of changes:
Updated the full install doc and the quick install script to support Agones 1.13.0 and Kubernetes 1.18. Temporarily removed the modular install script. It will be restored when it has also been updated to Agones 1.13.0 and Kubernetes 1.18.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Tested both the full install and the quick install to verify it works. Please open a GitHub issue if there are any deployment concerns.


List Item Type Coalescing

This extends from both #19 and #20. 19 was originally going to be both for this and the Less-String Writing, but these ended up feeling like separate issues, based on a common theme.
On par with making more object data structures accessibly writable, I think it could make sense to also coalesce array values to be a single primitive type, rather than throwing an error for it. Ideally, your data coming into NBTify should be NBT-spec compliant, but if not, I think it's fair for the data to coalesce into being spec-compliant.
This would leave the user to work with their data accordingly, and to fall back on type definitions, or manual error checking for the item types.
In writing that last sentence, the other side of my mind says that it also makes sense to keep the error, as this feels similar to the recursive object data structure error that JSON.stringify() will throw if it encounters an object with that being the case. Maybe the Less-Strict Writing idea (#20) should only apply to objects, and maybe arrays should still be strict.
Gonna experiment with all three of these ideas, and see how they feel together. Never know how you feel about an idea until you see it fleshed out!
Big Note:
Another thing too, is that List tag item type error handling feels the closest to the recursive object handling for the JSON module, and the Object prototype error handling feels like it's trying to be the BigInt error, which is kind of treating Compound tag objects as primitive-ish values, which might be part of what feels weird about it's error handling. Object prototype error handling is checking the object itself, while the List tag item type error handling is checking the structure of the data, just like the recursive object handling does. It's the shape of the data, vs the thing of data itself. I think I may be on to something! :smiley:

Going to go with the more-strict route, and keep the error handling for inconsistently-typed List tag item types. As mentioned in the summary for #20, it's the user's data that NBTify is working with, so it's not the library's place to ensure that the data will comply with the NBT format.


Does not seems to work well with cross domain file-storage-url

Consider following settings:
tolgee.file-storage-url=//translate-static.example.com
tolgee.front-end-url=https://translate.example.com

The Screenshots in Quick Translation is looking for https://translate.example.com/translate-static.example.com/uploaded-images/xxx

After close investigation this looks like a tolgee-platform issue.

Sorry for late response. May I ask what is your use case specifically?

It just seems that it cannot handle the // prefix, which is a common way to automatically switch between http/https using the base address by browser.

Hey, since we never stated // prefix is supported I am closing this, since there seems to be no need to actually add this. If you wish, you can submit PR fixing this and we will accept it.


What's the license of this package

Hi,
I couldn't find the license (if any) under which this package can be used. Can you please clarify?
Thanks,
Assaf

It is Apache 2.0

Hi, I am seeing a compilation error in "LossyCounting.java" on
@Override
add(T item, long incrementCount)

as it is not overriding any existing method. Please advice. Thanks


Update load-balancer-outbound-connections.md

Fixed typo and grammar

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

Note that publishing in this repository is suspended until Tuesday, May 25. Merging will resume Tuesday. Authors can still review and sign off on PRs in the interim. Thanks.

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

Looks good; thanks for the fix. The typo was fixed in another PR, but the grammar adjustment looks good.


Multithreading

Hi there,
I was wondering if it would be possible to multithread the spidr gem? I don't know much about multithreading in ruby, but I believe only Ruby 1.9.x is able to do so?
I had a look through the source but couldn't find where the spidr gem makes its http requests.
Maybe something like Typhoeus can be used?! (http://rubygems.org/gems/typhoeus)
Thanks,
Ryan

What happened with this request?

I don't have the time currently to work on such a large feature.

been a year, any chance you have time to work on such a feature now? :-)

I've written more than a crawler or N in my career and if you didn't make it multi-threaded from the start, it is damn hard to do so in retrospect.  Now, that said, I think the overall goal here is throughput rather than threads.  If the discovered urls can be surfaced to an overall queue (Redis or SQS) then that would change the equation because rather than threads you simply run more instances (or containers) of Spidr and let the queue handle distribution of work across N copies.
Thoughts?

A distributed Spidr is a little out of scope, or at least further down the road.
Multi-threading here is mainly to address blocking I/O when waiting on responses to come back from the HTTP Sessions. Luckily, net-http-persistent is already thread aware. We'd just need to replace the spidering loop with a producer/consumer thread pool. Each thread would have it's own session cache via net-http-persistent, would dequeue URLs, and enqueue the responses/Pages. All additional logic with headers and parsing HTML would still be done in the main thread, to avoid additional Mutex complexity. There's probably other hidden work and locking issues hidden in the details.

+1, a producer/consumer for the requests would be awesome! I really like the interface of your library by the way.

I don't understand "a producer/consumer for the requests"...
==================
David Adams
<EMAIL_ADDRESS>On Sun, Sep 24, 2017 at 9:33 AM, Vincent Wochnik<EMAIL_ADDRESS>wrote:

+1, a producer/consumer for the requests would be awesome! I really like
the interface of your library by the way.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/postmodern/spidr/issues/26#issuecomment-331713971,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAzcSIIeYHPwcLScA05GBD8Qk5UGEPG0ks5slmhIgaJpZM4BF_tL
.

I mean a producer/consumer pattern where one a thread pool of worker threads that do the requesting are connected to the main thread with queues, like a manufacturing band.
The main thread puts all requests that it wants to have resolved in a queue and any worker thread can pick the task from the queue, do the request, and put it inside the finished responses queue which is being read by the main thread. In this way, the main thread does not do any requesting, i.e. blocking activity, itself which will lead to a speedup.


[core] fix isStreamComplete issue

Extract @MaryGao's fix from PR https://github.com/Azure/azure-sdk-for-js/pull/31027
Fix an issue in isStreamComplete where the method never resolves when the stream is not readable.

API change check
API changes are not detected in this pull request.


Delete / Edit an event

Please describe the feature you wish to add.
Delete events we don't want anymore, or maybe edit the event for a typo.
What is the user story associated with this feature?
I accidentally created the wrong event and need to delete / edit it.
Additional context
N/a

Basic functionality added in 9f1f5954c1da8ff1b1ed08f071724afa8a216b45


(maint) Docker DNS reliability improvements

New variable PUPPETSERVER_PORT is added that defaults to 8140

Rename wait_for_host to wait_for_host_name_resolution to better
reflect its purpose.
Change it's behavior to use the host command (which returns proper
exit codes) instead of using ping. ping will not work in a k8s
environment where hosts are not reachable by ping

After succeeding or failing DNS lookup with the host command,
emit the dig output for that lookup for the sake of logs

There appear to be many reported issues against Alpine DNS. This
is an attempt to work around the ones we're experiencing.

In local testing (specifically under LCOW), DNS resolution under
Alpine seems to be very problematic.
nslookup may repeatedly fail to perform a DNS resolution against
another container name like puppet.local repeatedly.
Lookup failures will resemble something like:
/ # nslookup puppet.local
nslookup: can't resolve '(null)': Name does not resolve
nslookup: can't resolve 'puppet.local': Name does not resolve
Even successes have problems with the DNS server
/ # nslookup puppet.local
nslookup: can't resolve '(null)': Name does not resolve
Name:      puppet.local
Address 1: <IP_ADDRESS>

Supposedly the "can't resolve '(null)'" part is innocuous, but it's
unclear if that is the case.  More info at:
nicolaka/netshoot#6
gliderlabs/docker-alpine#476

It seems that just having the bind-tools package installed will
increase the reliability, but after running dig once against the
given host, intermittnet DNS resolution problems seem to go away
/ # nslookup puppet.local
Server:         <IP_ADDRESS>
Address:        <IP_ADDRESS>#53
Non-authoritative answer:
Name:   puppet.local
Address: <IP_ADDRESS>
So the script is changed to query for the postgres hostname

We don't use curl here because we're mostly interested in making
sure a host with a given name should exist.
There are scenarios where host / dig will succeed, but latter
checks with curl may not - and we want to differentiate those
failure modes as much as possible
serverfault.com/questions/335359/how-is-it-possible-that-i-can-do-a-host-lookup-but-not-a-curl

This should remove the need for #2922

This works in the k8s environment as a replacement of #2922.
							IN	A

;; Query time: 1 msec
;; SERVER: <IP_ADDRESS>#53(<IP_ADDRESS>)
;; WHEN: Thu May 02 23:48:52 UTC 2019
;; MSG SIZE  rcvd: 47

Test PASSed

Interesting @jarretlavallee I don't see the ANSWER section from dig in your output! Did you remove it from your output?
			 				IN	A

;; Query time: 1 msec
;; SERVER: <IP_ADDRESS>#53(<IP_ADDRESS>)
;; WHEN: Fri May 03 00:15:44 UTC 2019
;; MSG SIZE  rcvd: 47

$ kubectl exec -it puppetdb-7cfdc5cd4b-7s8kh host -t puppet
puppet.default.svc.cluster.local has address <IP_ADDRESS>
$ kubectl exec -it puppetdb-7cfdc5cd4b-7s8kh nslookup puppet
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#53

Name:	puppet.default.svc.cluster.local
Address: <IP_ADDRESS>

It looks like this is because dig does not search the domains in the resolv.conf by default. The man page says that the option is disabled.

 +[no]search
     Use [do not use] the search list defined by the searchlist or domain directive in resolv.conf (if any). The search list is not used by default.

 IN	A

;; ANSWER SECTION:
puppet.default.svc.cluster.local. 5 IN	A	<IP_ADDRESS>

;; Query time: 0 msec
;; SERVER: <IP_ADDRESS>#53(<IP_ADDRESS>)
;; WHEN: Fri May 03 15:36:37 UTC 2019
;; MSG SIZE  rcvd: 121

Thanks a ton for looking into that @jarretlavallee!
I will put up a PR and add +search in, because the purpose of emitting that info was to know the addresses that PDB is resolving.

Opened up https://github.com/puppetlabs/puppetdb/pull/2937 to improve dig output


Fix #2986 - Make ModelMetadata getters null-safe

/cc @dougbu

Design looks good. Ready for the real PR :smiling_imp:

Added a test :+1:

:shipit:


cargo shuttle ignores Shuttle.toml if run in subdir of crate

The logic to find Cargo.toml works fine, but we don't then use the location of Cargo.toml to search for a Shuttle.toml (and will continue as if it's not there). This means that it will have the wrong project name.
I wrote a test for this in https://github.com/shuttle-hq/shuttle/pull/122 to help when implementing this. See https://github.com/shuttle-hq/shuttle/pull/122#discussion_r866546185_
It's very much an edge case, so might not be worth fixing.

I'll take it!


Grammar issue

PAWs should be used for managing the operating system and applications that provide Directory Synchronization and Identify Federation for cloud ....

Should read

PAWs should be used for managing the operating system and applications that provide Directory Synchronization and Identity Federation for cloud ...

Thanks, Ivan - I'll pass this along to the writer who owns this content.

Fixed in 5529499.


Streams issue unnecessary warning on close when Flutter hot restarts

Describe the bug
Hey, thank you so much for this project.
I ran into a small issue with flutter hot restart when using a static StreamSink. Essentially on every hot restart the app will print out a "Fail to post message to Dart" message.
The reason why this happens is because when StreamSink is dropped, it calls the Drop implementation of the _closer field in it's StreamSinkBase struct.
My guess is that the infrastructure used to pass the close message is "down" or something along those lines, because the send_or_warn consistently fails everytime.
I am making this issue to discuss whether the error from send should simply be ignored in that specific context, since it doesn't make much sense to use send_or_warn (in my opinion).
MRE available here: https://github.com/ehuff700/frb-bug-mre.git
Steps to reproduce
Clone the MRE above and run with flutter run, observe the error message on hot restart. After debugging I tracked it down to the send_or_warn method in that specific Drop implementation.
Logs
N/A

Expected behavior
No response
Generated binding code
No response
OS
No response
Version of flutter_rust_bridge_codegen
flutter_rust_bridge_codegen 2.0.0-dev.31
Flutter info
No response
Version of clang++
No response
Additional context
No response

Hi, I guess one way is to detect this specific scenario and do not warn if it happens (since it is not a problem). However, I may not immediately work on it, since it just prints a warning and does not cause big troubles - but if it causes real bugs feel free to ping me!

Hi @fzyzcjy, like you said this is just a warning and I don't think there are any bugs so this is really a small issue. I could make a PR for it, do you have any pointers on how we would detect a hot reload?

@ehuff700 Hi, thanks for the interest of PR! However, I am not sure whether there is an easy exact way to detect hot restart without any false positives. If there are scenarios when something really goes wrong but we do not realize today and make it non-warning, then we will lose a real warning and thus makes people hard to debug.


Modifying ABC Race to have ひらがな and カタカナ and mixed

I am not quite sure how to customize this game myself mainly how to create a link to the copied repo on my own Github.
The idea is to make a modified version of the abc race with hiragana and katakana for students in special education.
It would be three modes with the basic 46 hiragana (no " or o), the 46 katakana, and a mix of the two.
I looked at the files in the figured that I would need to have the svg files for all of the hiragana and katakana.
This page has a list of all of the hiragana and katakana. It seems like the katakana for n is missing.
https://studio.beatnix.co.jp/kids-it/kids-programming/scratch/scratch-material/hiragana-katakana-minchyo/
The audio files I have found from this website.
https://www.thoughtco.com/how-to-pronounce-hiragana-japanese-hiragana-with-audio-files-4077351
All of these files are also on my repository found here.
https://github.com/pet3rpan/media
I am not quite sure if this is the right place to make a request and it is understandable if you are busy as well. If you have the time and are willing to take a look, it would be much appreciated. Thank you.

It wouldn't take me very long to make a かなレース version.
List of things to be done:

Gather and rename SVGs and MP3s
Replace lowercase and abcde... with katakana and アイウエオ...,
Replace uppercase with hiragana and ABCDE... with あいうえお...
Make new buttons
Make new results screen

If I make SVGs from the UD Digi Kyokasho font, it will be in the same style, since I used the same company's font for the English letter SVGs.

https://altivities.earthiverse.ca/kanarace/
It is complete.

Thank you very much for taking the time to work on it. It's perfect.


Move CommonCrypto and zlib dependencies mapping to custom module map

Details:
https://github.com/daltoniam/Starscream/pull/520

Can you look into why it doesn't build with Swift PM. I get this error:
Compression.swift:30:8: error: no such module 'Starscream.SSCZLib'
import Starscream.SSCZLib

Sure, let me check

Looks like proposed solution cannot be adopted by SPM. It doesn't seem to support explicit modules. I will investigate what can be done here.

I would see for SPM if you could just use Swift 4.2's system targets. They look something like this https://github.com/nuclearace/SwiftDiscord/tree/vapor3/Sources/Sodium and https://github.com/nuclearace/SwiftDiscord/blob/vapor3/Package.swift#L43

@nuclearace Thank you, it looks very relevant.
Also I think that in iOS12/Mojave CommonCrypto has system bindings in swift so if target is the latest SDK I can get rid of Obj-C helper

Maybe this is the option we can aim. You can still use older release for lover Swift version.

@turbulem looks like you're right about CommonCrypto https://stackoverflow.com/a/50690346

In which version of Swift are these libraries included? I think need to explicitly set Swift 4.2 version in Package.swift and podspec file. .swift-version is going to be deprecated and isn't used anymore?

@fassko I updated my solution. It only works with new SDK, which might not be the best option. However it solves the problem and seem to work fine with Pods/Carthage/SPM.


Release v1.1.1

Resolves CVE-2020-7598.

Note: Some of "minimist" dependencies in package-lock.json are still <1.2.2. However, they are only used in devDependencies.


Add MBS 0205 0216 and 0255

What is the context of this PR?
Addition of new MBS questionnaires. Northern Ireland theme not currently available for use with 0216 so will need updating for logo and Introduction page.
How to review
Compare with google slides.

HTML can be removed once #1540 is in as it will need to use content rather than description.

@dwyeradam can we lose the section headings "Exports" etc

@dwyeradam @nealedj mentioned it the other day that hitting back button randomly now you get form submission error. I think we have introduced this bug recently

I have added a card to backlog that when definitions feature goes in we use it for 'significant changes' on all business surveys

@charlotte-hirst I think you're seeing the browser's form resubmission error:

You can reliably recreate this by:

go to a question that requires some validation
enter an invalid value and click continue to post the form
enter a valid value and you should be redirected to the next page
click back

At that point you'll see the resubmission error. This is because the browser is taking you back to the point in its history where you tried to post the form (and failed due to the navigation error). All browsers will either throw an error or ask you to resubmit your data in this case.
It won't have been a recently introduced issue. It's something that will exist in all web forms following the post/redirect/get pattern.

Do not merge until SDX have added the new qcodes.

survey id should be 009

@dwyeradam I am good to approve this, but I'll keep my blocking review to stop merge


Resolve Sprout access issues

Taz Chiles
Alex Monk<EMAIL_ADDRESS>Hi Alex,
I hope you're well - we normally run a manual weekly update for social media stats on the Data Microsite. I've just tried to log in this morning and getting a credential error.
Do you still have access to the Sprout platform or has something changed at your side? Please let me know if log in details have changed.
We'll add a temporary notice to the site so that people are aware we are not showing the most up to date stats.
Many thanks
Tazmin

Follow up sent 4/8

Another follow up sent 31/08 and checked Sprout access - still getting credential error.
Limitations added to the page

Closing #186 as completed, will create a new ticket to review final social media data


Report relative humidity (tested on TFA Airco2ntrol)

This adds relative humidity reporting on the TFA Airco2ntrol, I am not sure if other devices use the same entry point though. I think back in November when I wrote it I saw mentions of other entry points, but hopefully none re-use the same entry point for a different value...

Thanks for the PR. However, my unit (AIRCO2NTROL MINI) always reports humidity of 0%. Is it possible to auto-detect whether a unit supports it? We should either try to auto-detect it or make it a config option.

I can confirm this PR works as-is for me (I don't really know which unit I have).
@patrislav1 Maybe your unit supports it, but is sent under a different value? This article suggests at least some units, it is sent at 0x44 instead of 0x41.

Is it possible to auto-detect

A humidity value of 0% is below the value range of the unit I have (20%-90% ; according to the label). Is a bit ugly, but we could use that to see if it is supported?


Refactoring of the AppController for the flows

Defines interfaces for dependency graph and resource scheduler
Passes those interfaces to resource template classes so that scheduler could be
used as part or resource template deployment. Interfaces make it possible to avoid
circular dependencies between resources and scheduler packages
Modifies dependency graph construction algorithm so that resource factories are
created in top-to-bottom order so that it possible to pass parameters and other context
information along the graph. Also it should become possible to extract flow subgraph
from the dependency graph
Refactors resource template classes so that it will become possible to get wrapped
resource name from its ResourceDefinition
Updates unit tests for the changes made
Replaces custom resource definitions and dependencies mock clients with standard
Fake-based clients

This change is

Only one nit. Rest things will be covered in https://github.com/Mirantis/k8s-AppController/issues/217

Reviewed 43 of 43 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

pkg/scheduler/scheduler.go, line 293 at r1 (raw file):

}

// Create starts the deployment of a DependencyGraph

Func name changed but docstring points to old name.

Comments from Reviewable

Reviewed 43 of 43 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable


items don't have a creative inventory tab

the items in the mod don't show up as a creative inventory tab, and i can't find them in JEI
so every time i want to add another monitor and gpu i have to use commands or craft it manually
pls fix

What minecraft version are you using?

1.20.1


instalation via composer fail

when I try to install the library via composer I recive this:

Installing elfet/php-ssh (v1.1.0)
Downloading https://api.github.com/repos/elfet/php-ssh/zipball/439becf180272542cbf154df4856a49dd3c4bf04
Downloading: connection...

Failed: [Composer\Downloader\TransportException] 404: The "https://api.github.com/repos/elfet/php-ssh/zipball/439becf180272542cbf154df4856a49dd3c4bf04" file could not be downloaded (HTTP/1.1 404 Not Found)
Failed to download elfet/php-ssh from dist: The "https://api.github.com/repos/elfet/php-ssh/zipball/439becf180272542cbf154df4856a49dd3c4bf04" file could not be downloaded (HTTP/1.1 404 Not Found)
Now trying to download from source

Installing elfet/php-ssh (v1.1.0)
Cloning 439becf180272542cbf154df4856a49dd3c4bf04
Executing command (CWD): git clone --no-checkout 'git://github.com/elfet/php-ssh.git' '/srv/www/vhosts/online-gadget.local/vendor/elfet/php-ssh' && cd '/srv/www/vhosts/online-gadget.local/vendor/elfet/php-ssh' && git remote add composer 'git://github.com/elfet/php-ssh.git' && git fetch composer
Executing command (CWD): git clone --no-checkout 'https://github.com/elfet/php-ssh.git' '/srv/www/vhosts/online-gadget.local/vendor/elfet/php-ssh' && cd '/srv/www/vhosts/online-gadget.local/vendor/elfet/php-ssh' && git remote add composer 'https://github.com/elfet/php-ssh.git' && git fetch >composer
Username for 'https://github.com':

and in packagist :

This package is abandoned and no longer maintained. The author suggests using the herzult/php-ssh package instead.

You need to run composer self-update then composer clear-cache. :)

thanks, but the problem it's the same, continue try use elfet/php-ssh and request error:

[RuntimeException]
Failed to clone git@github.com:elfet/php-ssh.git via git, https, ssh protocols, aborting.

git://github.com/elfet/php-ssh.git
Cloning into '/srv/www/vhosts/.../vendor/elfet/php-ssh'...
fatal: remote error:
Repository not found.
https://github.com/elfet/php-ssh.git
Cloning into '/srv/www/vhosts/.../vendor/elfet/php-ssh'...
remote: Invalid username or password.
fatal: Authentication failed for 'https://github.com/elfet/php-ssh.git/'
git@github.com:elfet/php-ssh.git
Cloning into '/srv/www/vhosts/.../vendor/elfet/php-ssh'...
Warning: Permanently added 'github.com,<IP_ADDRESS>' (RSA) to the list of known hosts.
Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

Exception trace:
() at phar:///usr/bin/composer/src/Composer/Util/Git.php:221
Composer\Util\Git->throwException() at phar:///usr/bin/composer/src/Composer/Util/Git.php:80
Composer\Util\Git->runCommand() at phar:///usr/bin/composer/src/Composer/Downloader/GitDownloader.php:54
Composer\Downloader\GitDownloader->doDownload() at phar:///usr/bin/composer/src/Composer/Downloader/VcsDownloader.php:66
Composer\Downloader\VcsDownloader->download() at phar:///usr/bin/composer/src/Composer/Downloader/DownloadManager.php:201
Composer\Downloader\DownloadManager->download() at phar:///usr/bin/composer/src/Composer/Installer/LibraryInstaller.php:156
Composer\Installer\LibraryInstaller->installCode() at phar:///usr/bin/composer/src/Composer/Installer/LibraryInstaller.php:87
Composer\Installer\LibraryInstaller->install() at phar:///usr/bin/composer/src/Composer/Installer/InstallationManager.php:152
Composer\Installer\InstallationManager->install() at phar:///usr/bin/composer/src/Composer/Installer/InstallationManager.php:139
Composer\Installer\InstallationManager->execute() at phar:///usr/bin/composer/src/Composer/Installer.php:580
Composer\Installer->doInstall() at phar:///usr/bin/composer/src/Composer/Installer.php:230
Composer\Installer->run() at phar:///usr/bin/composer/src/Composer/Command/UpdateCommand.php:140
Composer\Command\UpdateCommand->execute() at phar:///usr/bin/composer/vendor/symfony/console/Symfony/Component/Console/Command/Command.php:257
Symfony\Component\Console\Command\Command->run() at phar:///usr/bin/composer/vendor/symfony/console/Symfony/Component/Console/Application.php:874
Symfony\Component\Console\Application->doRunCommand() at phar:///usr/bin/composer/vendor/symfony/console/Symfony/Component/Console/Application.php:195
Symfony\Component\Console\Application->doRun() at phar:///usr/bin/composer/src/Composer/Console/Application.php:146
Composer\Console\Application->doRun() at phar:///usr/bin/composer/vendor/symfony/console/Symfony/Component/Console/Application.php:126
Symfony\Component\Console\Application->run() at phar:///usr/bin/composer/src/Composer/Console/Application.php:83
Composer\Console\Application->run() at phar:///usr/bin/composer/bin/composer:43
require() at /usr/bin/composer:25

Which version of deployer are you trying to install?

~2.0

Try to install 2.0.5. And v3 stable will be very soon.

nice, now works, so thanks very much ! :)

:beer:


Confusing not to see expression level

https://www.pombase.org/reference/PMID:11683390
Confusing not to see expression level of wt here
cig1Δ cdc13+ cig2Δ   (knockdown)
We might already have a ticket by this opened by Antonia
(Nurse lab)

In Canto we format it like: "cdc13+[Knockdown]".
Should we do the same in the web site?

I think I have at least mentioned it before, I seem to have said something similar in  #163 but that ticket isn't relevant any longer because of other changes.
Kim I think that would work

Kim I think that would work

Great, I'll do that then.

yes  I thought that you also mentioned it @Antonialock , but I could not find the old ticket.
agree that would work.
v

I've added the expression, but it makes the phenotype annotation summaries a bit hard to read.  eg.
https://www.pombase.org/gene/SPCC18B5.03
Any suggestions for improving things?

I think we only need to include for  [Overexpression]
(isn't that what we did on the old site?)

On the old site we excluded [Not Assayed]
We included overexpression and knockdown

On the old site we excluded [Not Assayed]
We included overexpression and knockdown

Thanks.  I removed "Not assayed" and that has helped.


[Azure] Add sovereign cloud support for Availability Set module

The Base URI is incorrectly resolved for the Availability Set client, and should be fixed to read from the environment.
Addresses #811

related PR merged https://github.com/gruntwork-io/terratest/pull/845. Can we close it please

Closing!


Cannot use in Nest.js

Hello! Awesome library! But when I try to re-export my customized logger in a Typescript library,
// My library
export const customLogger = adze.withEmoji.timestamp.seal();

But when I tried to import it in Node environment (NestJS), I encountered this:
ReferenceError: navigator is not defined
    at /web-utils/dist/index.cjs:2:2597
    at Module.<anonymous>

So I tried to use adze directly in NestJS.
I tried import, require, dynamic import, and none of them work. I've set tsconfig.json compiler lib to DOM per the documentation.
const adze = require('adze');

Don't know why:
Error [ERR_REQUIRE_ESM]: require() of ES Module /backend/node_modules/adze/index.js from /backend/dist/src/app/app.module.js not supported.
Instead change the require of index.js in /backend/dist/src/app/app.module.js to a dynamic import() which is available in all CommonJS modules.

I can use it in Electron but not in NestJS. Weird.

@boan-anbo , I went ahead and added a FAQ's section on the documentation website regarding a resolution to this issue. Essentially the problem is that Adze is exported as an ES module and NestJS does not currently support ES Module libraries without using some workarounds. There is an Open PR that will hopefully be merged into NestJS in the near future that will resolve the issue, but in the meantime, check the FAQ page to see the workaround options.
https://adzejs.com/faqs.html#how-can-i-make-adze-work-with-nestjs

I have also opened a ticket to explore exporting adze as UMD to make it backwards compatible with commonjs frameworks.
https://github.com/adzejs/adze/issues/176


Change --min-py3-version behavior

Replace --min-py3-version with --min-py-version
Change how the min version is determined; use --min-py-version as the source of truth if supplied, otherwise infer it

Resolves #178

sorry it's been a while -- haven't gotten around to this.  at the very least we'll need to keep the old option and have it be an alias to the new option (perhaps with a warning?)

at the very least we'll need to keep the old option and have it be an alias to the new option (perhaps with a warning?)

Perhaps we split this into multiple releases
Release 1: Add a --min-py-version flag which is used to override every other setting. That flag is exclusive with --min-py3-version
Release 2: Add a warning when --min-py3-version is used
Release 3: Drop --min-py3-version entirely, change the logic to act like what's in this PR
Release 4: Ensure --min-py-version>2, delete py2 code? Too heavy-handed?

I think simpler would be to just remove the py2 code and make --min-py3-version an alias to --min-py-version and disallow anything less than 2 -- and release that as a major version (then later remove the --min-py3-version option)

OK, so in cases where python_requires >= 2.7 - would the tool just promote that to the version in --min-py-version, regardless of whether or not it's set explicitly?

yep!  intentionally removing any support for python 2.x packages

OK. Same for tox.ini envlist & classifiers too?

yep yep

Dropping py2 support and enforcing that --min-py-version takes precedence is kind of complicated so let me split up the PRs


ProduceAvailability should be 1.0 as long as ProduceService does not see exception

@smccauliff can you take a look?


[BUG / UNEXPECTED BEHAVIOR] The error-message does not reflect the cause of the error.

Environment

React-formio version: "^4.2.4"
Formio.js version: "4.11.0"
Frontend framework: React "^16.8.6"
Browser: many browsers.
Browser version: many browser versions.

Steps to Reproduce

Create a form, add the "Text Field" component to it. (The edit window opens immediately after that)
Enter non-Latin characters in the "Label" field.
Try to save.
Pay attention - An area with an error description appears

Expected behavior
It is expected that the user to be notified that non-Latin characters are not allowed.
I have some suggestions:

Add an additional notification to the user that non-Latin characters are forbidden in the "Properties Name" field of the "API" tab.
Disable the ability to automatically fill in the "Property Name" field of the "API" tab when filling in the "Label" field of the "Diaplay" tab.

Observed behavior
Error message:

Please fix the following errors before submitting.
Property Name: The property name must only contain alphanumeric characters, underscores, dots and dashes and should not be ended by dash or dot.

Example:

Hello @vaspaul,
Thanks for reaching out! This error means that the API that you have added to the API key section is not valid, so you will need to change it. You can change the Label back after the initial save and it will not impact the API key.
I hope that this helps.

Hi, @wag110894 ,
Will developers add this information to the error message explicitly in the next release?


Add Techqueria.org to FirstContributions.io site

Make a PR to the repo by adding Techqueria.org to this file
https://github.com/firstcontributions/firstcontributions.github.io/blob/source/src/components/ProjectList/listOfProjects.js

✅ By submitting this issue, I have verified the following

[x] Checked to see if the issue has already been discussed before 🤔
[x] Made sure enough details were provided if proposing a new feature 🔍

Hi @fvcproductions, I wanted to get the requirement clarified. Do you mean you want an entry of  Techqueria.org to be added to the projectList array?
If yes, could you please LMK what would be the entries of the imageSrc, description and tags for your website?
name: ' Techqueria.org',
imageSrc: ,
githubLink: 'https://github.com/techqueria',
description: ,
tags:

@sonali9696 Yes, that's right!
Here are the details
imageSrc: https://avatars1.githubusercontent.com/u/17460806?s=200&v=4
description: "We're a community of Latinx professionals in the tech industry."
tags: "latinx", "latinx in tech", "hugo", "netlify", "jamstack"

On observing the UI of https://firstcontributions.github.io/, I have mixed feelings about putting a full stop in the description. According to https://bootsnipp.com/snippets/92xNm, full stop seems like the way to go but some of the cards in https://firstcontributions.github.io/ do not have full stops and they look good to me too. For now, I am proceeding with the full stop but I would like to know your views too @fvcproductions

The changes have been made and PR has been raised: https://github.com/firstcontributions/firstcontributions.github.io/pull/43. I tried connecting the PR directly to this issue but since it is in a different repo, it was only redirecting to their own issues.

@sonali9696 PR looks great with full stop - thanks so much! 👍

@sonali9696 Let's close this issue for now since the PR has been opened and it's in their ball game now. I made sure to subscribe to the PR for any future changes. 🌮


Implement TS-Chief

https://arxiv.org/abs/1906.10329

I can take this on!

Hi @jordanrmerrick, that would be much appreciated!

@mloning Hi Markus, I might be able to track this issue for the python implementation


How does a two stage detector deals with input H W with different size in a single batch?

I used to use detectron2, in which a FasterRCNN will resize the image in different sizes to a fixed size in one batch in order to form a tensor. And the resizing is implemented as a non-training torch module in model .
Now I'm using mmdet 3 and I wonder how it deals with it. When we use the multiscale training, the size of each image will be different in a batch. What happen after one image is resized to a random size and before it is feed into a FasterRCNN ?

@sipie800 Here https://github.com/open-mmlab/mmdetection/blob/main/mmdet/models/data_preprocessors/data_preprocessor.py#L31


[LOC] Back-Translation "新規-AddressRewriteEntry"→"New-AddressRewriteEntry"

🛑 IMPORTANT: You can get your feedback addressed faster if you use the comment section for the article in which you encountered a problem (if available).

Link to article:
Tell us where exactly a problem occurs.
"https://docs.microsoft.com/ja-jp/powershell/module/exchange/mail-flow/new-addressrewriteentry?view=exchange-ps"
Problem:
What's wrong with the article?
[LOC] Back-Translation "新規-AddressRewriteEntry"→"New-AddressRewriteEntry"

@OfficeGlobal, please uplift this topic to human translation. @Soran-M FYI this is an MT overlocalization issue in PowerShell content.

Working on a fix

@hyoshioka0128 , @TinaMcN the article was HumanTranslated and published live

@OfficeGlobal , @TinaMcN - Fixed.
Closed #2320.
Thanks.


MultiAddSelect components allows duplicate item selection

What package(s) are you using?

[X] Carbon for IBM Products (or Carbon for IBM Cloud & Cognitive)
[ ] Legacy/CDAI
[ ] Legacy/Security

Detailed description

Describe in detail the issue you're having.

Depending on the sequence of user selections, it is possible to have duplicate items in the selection list. In the example below, folder 1 appears twice in the selection list.

Is this issue related to a specific component?

This issue is specific to the MultiAddSelect component.

What did you expect to happen? What happened instead? What would you like to
see changed?

I would expect the component to ignore an item with a duplicated id when updating the state for selected items.

What browser are you working in?

Google Chrome

What version of the @carbon/ibm-products (or @carbon/ibm-cloud-cognitive)
package are you using?

Carbon for IBM Products v1.23.1
Steps to reproduce the issue

Go to https://carbon-for-ibm-products.netlify.app/?path=/story/ibm-products-patterns-add-select-multiaddselect-canary--with-hierarchy and launch the AddSelect component modal.
Select an item within the Business terms list (i.e. folder 1).
Then use the checkbox at the top of the list (i.e. the checkbox next to Business terms) to select all the items under Business terms
Observe that folder 1 now appears in the selected items twice.

@davidmenendez

@mohammed-mian this should be resolved. please pull latest and let me know if you're still experiencing this issue.

Verified that the issue is fixed, thanks.


Create an asset factory to generate FERC1 output tables

draft of asset factories for #3147.
Based on the tutorial in https://dagster.io/blog/python-factory-patterns, creates 3 assets using a factory for: out_ferc1__yearly_purchased_power_and_exchanges_sched326
out_ferc1__yearly_plant_in_service_sched204
out_ferc1__yearly_balance_sheet_assets_sched110
all of which just merge and then organize columns.
a few questions:

Is this the type of solution you were looking for?
double checking: Is it ok to omit the organize_cols parts of those functions?
We are currently omitting the pd.DataFrame input type specification in these assets, is this ok?

We've created an asset factory that appeared to work locally on dagster. When trying to push our code, we ran into some errors getting our codebase up to date that we are currently resolving.

@zaneselvans  For some reason we are failing unit tests when we try to commit that seem unrelated to the code in output/ferc1.py we were working with.
=========================== short test summary info ============================
FAILED test/unit/workspace/datastore_test.py::TestDatapackageDescriptor::test_get_partition_filters - TypeError: Package.metadata_validate() missing 1 required positional argume...
FAILED test/unit/workspace/datastore_test.py::TestDatapackageDescriptor::test_get_resource_path - TypeError: Package.metadata_validate() missing 1 required positional argume...
FAILED test/unit/workspace/datastore_test.py::TestDatapackageDescriptor::test_get_resources_filtering - TypeError: Package.metadata_validate() missing 1 required positional argume...
FAILED test/unit/workspace/datastore_test.py::TestDatapackageDescriptor::test_json_string_representation - TypeError: Package.metadata_validate() missing 1 required positional argume...
FAILED test/unit/workspace/datastore_test.py::TestDatapackageDescriptor::test_modernize_zenodo_legacy_api_url - TypeError: Package.metadata_validate() missing 1 required positional argume...
FAILED test/unit/workspace/datastore_test.py::TestZenodoFetcher::test_doi_format_is_correct - TypeError: Package.metadata_validate() missing 1 required positional argume...
FAILED test/unit/workspace/datastore_test.py::TestZenodoFetcher::test_doi_of_prod_epacems_matches - TypeError: Package.metadata_validate() missing 1 required positional argume...
FAILED test/unit/workspace/datastore_test.py::TestZenodoFetcher::test_get_descriptor_http_calls - TypeError: Package.metadata_validate() missing 1 required positional argume...
FAILED test/unit/workspace/datastore_test.py::TestZenodoFetcher::test_get_known_datasets - TypeError: Package.metadata_validate() missing 1 required positional argume...
FAILED test/unit/workspace/datastore_test.py::TestZenodoFetcher::test_get_resource - TypeError: Package.metadata_validate() missing 1 required positional argume...
FAILED test/unit/workspace/datastore_test.py::TestZenodoFetcher::test_get_resource_with_invalid_checksum - TypeError: Package.metadata_validate() missing 1 required positional argume...
FAILED test/unit/workspace/datastore_test.py::TestZenodoFetcher::test_get_resource_with_nonexistent_resource_fails - TypeError: Package.metadata_validate() missing 1 required positional argume...
FAILED test/unit/workspace/datastore_test.py::TestZenodoFetcher::test_get_unknown_dataset - TypeError: Package.metadata_validate() missing 1 required positional argume...
= 13 failed, 1367 passed, 1 skipped, 3 deselected, 9 xfailed, 44 warnings in 32.77s =
conda-lock...........................................(no files to check)Skipped
We thought it was a package update issue, guessing that we were pulling code from the remote but not updating the packages that accompany them, so we tried:
make conda-clean
make conda-lock.yml
make install-pudl
but we are still receiving these errors. Not sure where to go from here, any help is appreciated!

Those unit test errors look like they result from having an older version of the frictionless package installed in your environment. We recently updated from v4 to v5.
I would remove any local changes to the conda lockfiles, remove your existing pudl-dev environment, make sure your local branch has all the most recent changes from main upstream, and rebuild the conda environment, which should look something like:
git checkout -- environments
mamba deactivate
mamba env remove -n pudl-dev
git pull
make install-pudl
mamba activate pudl-dev

Given the changes you're making, you shouldn't need to run make conda-clean or make conda-lock.yml, which will update/change the locked dependencies.

@zaneselvans our most recent commit from right before our meeting has some updated changes to it as well as fixing the merge conflicts, if you are able to take a look at that.

I think the semester ended for the students who were working on this issue / PR so it's probably not going to get merged unless we can rope someone else into working on it... @seeess1 would you be interested by chance? Mostly it's just some Dagster asset factory stuff.  Once we get your environment back to working!

I think the semester ended for the students who were working on this issue / PR so it's probably not going to get merged unless we can rope someone else into working on it... @seeess1 would you be interested by chance? Mostly it's just some Dagster asset factory stuff. Once we get your environment back to working!

Yep would be happy to take a look. Hoping to get to the point where I can load assets in a notebook. We can try and resolve that first in Slack and then I can take a look here.

Will wrap this up in #3883, closing this PR.


git lfs test

@kratz00 use LFS like this?

In case there are just cab files I would directly add them, the Zip is just overhead (does not save disk space either).
The only exception I currently see is the wireplay Zip file, as most data files from it are needed.

In case there are just cab files I would directly add them, the Zip is just overhead (does not save disk space either). The only exception I currently see is the wireplay Zip file, as most data files from it are needed.

One of the reasons for having .zip files is to make it harder for DMCA scanning bots and similar crap, in case there would be a match.

Understood, I will update my PR and put the cab files into Zip files again.

@twogood I disabled the crashing test for now (see issue #156) to unblock this pull request.

Fixed in #149


fix: Correct plugin installation path for older tf versions.

What does this do / why do we need it?
For older tf versions (< v0.12.31) the location to place the compiled provider is different. This PR fixes that
Check lists

[ ] Test passed
[ ] Coding style (indentation, etc)

Additional Comments (if any)
{Please write here}
Which issue(s) does this PR fix?
fixes #53

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 49 (2.04%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-6.7%) to 60.0%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/app/install.go
1
49
2.04%

Totals

Change from base Build<PHONE_NUMBER>:
-6.7%

Covered Lines:
258

Relevant Lines:
430

💛  - Coveralls


Allow adding a custom NotFound middleware to Vulcan.

WHAT:
1. Inside mux.go is a NotFound http.Handler that is used
as the catch-all handler for requests that do not
match any route.
2. Repeatedly, we face situations where we want to know
such requests and act on them (i.e. either throw a
friendlier 404 Not Found page, or redirect them,
or audit/log them to find missing/erraneous routes
in our system.
3. This is distinctly different from using the CircuitBreaker
that only operates on routes that exist, but fail.
HOW:
1. mux.go already has a NotFound handler, which I renamed to
DefaultNotFound (for clarity should anyone read the code in
the future and try to make sense of where the default
one resides.)
2. I then modified proxy.Options to hold a Middleware that could
be set by the caller (service.go who instantiates mux router)
3. service.go itself obtains this middleware from Registry which
also holds all MiddlewareSpecs.
4. I added methods to the registry to add/get the default middleware.
Wrote some tests to go along with the changes where necessary.
WHY NOT:
1. Why not use etcd/engine? - Two reasons: Major one being time.
I need this change to use vulcand ASAP. However, the major
reason was because I need a NotFound handler when routes are
failing - i.e. waiting for etcd to SET a middleware would
be a point of failure. Given that I'm already compiling vulcan
using vbundle, this seemed like a reliable way to ensure I have
something in there, if all else fails.
2. Why not pass a MiddlewareSpec, instead of Middleware? -
   Mainly because instantiating middlewares requires some
   parameters - which for other middlewares are obtained from
   etcd/engine. However, this middleware must be instantiated
   and ready to go with as little ceremony as possible, in order
   for it to be the catch-all default handler.

FUTURE IMPROVEMENTS:
1. I may add an etcd or engine-based path as well. But we'll
need to discuss a new key/namespace where we can add
"default" or "catch-all" middlewares and write
logic around injecting them.

Is there something I can do to help this move along? I'm using this in production for quite a time, and it appears to be stable so far.
I'm happy to change/modify/discuss any concerns you may raise.

Aside from a couple of minor comments, looks good to me


cd shows wrong answer

Shows "created".

Fixed by https://github.com/minthemiddle/shell-demo/commit/90c967becdecc6de3f19c26014cbb75cc24d736a


Fix ports for kubeadm client and master configs for ha setups

Fixes https://github.com/kubernetes-incubator/kubespray/issues/3071

ci check this

/lgtm
/approve

@mlushpenko  Thanks


Option  servers: [{ url: "v1" }] generate bad request

What happens?
When we add the option  servers: [{ url: "v1" }], the request doesn't add the basepath of url (localhost or https://mysite.com)
In the console, the request is :
http://v1/hello/Peter
What did you expect to happen?
the correct request is :
http://locahost:8000/v1/hello/Peter
How can we reproduce the issue?
in openapi.json
{
  "openapi": "3.1.0",
  "info": {
    "version": "1.0.0",
  },
  "tags": [
    {
      "name": "foo",
      "description": "foo API Endpoints"
    }
  ],
  "servers": [
    {
      "url": "v1"
    }
  ],

Swagger/OpenAPI Example
https://sandbox.scalar.com/e/y1nlJ

@hapaxlife thanks for making this issue!
I think we should check if it's a valid URL, and if it's a path we append it to the window origin, thoughts @hanspagel  @hapaxlife ? :)

Not sure if I should create a new issue, but this change results in issues where the protocol is also parameterized. For example, here is the servers property in our openAPI spec:

This results in Scalar generating broken URLs such as http://<IP_ADDRESS>:3000/https://localhost:8083/management/v2/environments/{envId}/apis.


Home path is missing the drive letter on Windows

What steps will reproduce the problem?
1. cindex any/path (without CSEARCHINDEX set)
2.
3.

What is the expected output?
Using .csearchindex in home dir.

What do you see instead?
Failing due to the home path missing the drive letter. In index/read.go, the
line in File() getting the home dir should probably read 'home =
os.Getenv("HOMEDRIVE") + os.Getenv("HOMEPATH")'

What version of the product are you using? On what operating system?
Windows 7, 64 bit

Please provide any additional information below.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 23 Jan 2012 at 10:39

This was probably fixed by #41  and can be closed.


Missing Documents / Website Issues

Firstly, many thanks for this useful simulator.
Since several days, the UAL Guide has some issues.
The following URLs are not working.

https://tomcl.github.io/visual2.github.io/memory-transfer.html
https://tomcl.github.io/visual2.github.io/multiple-register-transfer.html

Would it be possible to fix it?
Many thanks.

Alas Visual2 is not currently under active dev - since the 1st year module that used it has been changed and now uses
https://tomcl.github.io/issie/


Feature: DropdownMenu & FontAwesomeIcon

This adds the DropdownMenu and a dependency of it, FontAwesomeIcon.
As such, the FontAwesome CSS library was also included as a dependency for Storybook.

Nice work. My only major concern is Font Awesome isn't kept separate from the core gather-ui codebase. I bet we could use FA as an external package - https://github.com/FortAwesome/Font-Awesome

Hah, I did see this repo but I assumed the "FortAwesome" was the newer (and different) version of the FontAwesome we're using. I now see that it seems to contain both, I'll check if they're interchangeable. If they are, I'll make the swap :)

What version do we need?
Here's the npm package if it helps - https://www.npmjs.com/package/font-awesome (again, depending on version)


posenet model support

Hi Dusty, it appears that posenet only supports onnx pose models. Does it support the old trt-pose .pth models or do those have to be converted somehow to onnx format? If so, how does one convert .pth to onnx?
Thanks in advance!

@silent-code jetson-inference uses TensorRT and doesn't directly load any PyTorch .pth model checkpoints, it always loads ONNX (or legacy caffemodels or .uff)
The trt_pose models I converted to ONNX using these scripts:

https://github.com/dusty-nv/trt_pose/blob/6fa5f5d36c9be4bdf314838d840a82dd1f2960d3/onnx_export.py
https://github.com/dusty-nv/trt_pose/blob/6fa5f5d36c9be4bdf314838d840a82dd1f2960d3/onnx_export.sh

Perfect thanks!


Port build configuration over to Bazel

Right now, we use ASDF, and sometimes it is a mess, because of Haskell versions etc. Centralizing on Bazel would allow us to simplify the CI scripts, and avoid version mismatch for different people when regenerating the doc comments.

Christoph and Anton ran into an issue with this today related to node-gyp not working properly.