[frontend] "Send Code" in signup page should be gray and spining immediately after clicking

https://door.casdoor.com/signup
Because this button is very slow, users are very likely to click it for multiple times.
Moreover, we can find out why sending SMS is so slow. The Go code can be improved, like without waiting for SMS sending result.

https://user-images.githubusercontent.com/3787410/215119348-7fed30fc-26e2-4712-a49d-ea4fa5b3997f.mp4

/assign


Yet another PR for testing Eventbrite stuff (correct direction this time)

Build check is failing as if dependency not met

Didn't realise that there was a list of supported plugins

Fixed


RUMM-1594: Drop KitKat support

What does this PR do?
This PR drops support of API 19 (KitKat) by bumping min SDK to 21, removing non-relevant code and also bumping OkHttp version to 4.x (because 4.x cannot be used with min SDK = 19).
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #1500 (16915bc) into nogorodnikov/add-persistence-executor-service-factory-for-tests (f08005b) will increase coverage by 0.03%.
The diff coverage is 92.31%.

@@                                         Coverage Diff                                         @@
##           nogorodnikov/add-persistence-executor-service-factory-for-tests    #1500      +/-   ##
===================================================================================================
+ Coverage                                                            83.13%   83.15%   +0.03%
===================================================================================================
  Files                                                                  423      423
  Lines                                                                14557    14532      -25
  Branches                                                              2181     2169      -12
===================================================================================================
- Hits                                                                 12101    12084      -17
+ Misses                                                                1870     1861       -9
- Partials                                                               586      587       +1

Impacted Files
Coverage Δ

...ay/internal/recorder/mapper/BaseWireframeMapper.kt
80.00% <0.00%> (-2.35%)
:arrow_down:

...com/datadog/android/okhttp/DatadogEventListener.kt
97.47% <50.00%> (ø)

...nal/net/DefaultFirstPartyHostHeaderTypeResolver.kt
89.47% <75.00%> (ø)

...tadog/android/core/internal/net/CurlInterceptor.kt
85.19% <87.50%> (ø)

...datadog/android/okhttp/trace/TracingInterceptor.kt
81.45% <87.50%> (ø)

...n/com/datadog/android/core/internal/CoreFeature.kt
87.74% <100.00%> (-0.23%)
:arrow_down:

...ndroid/core/internal/net/GzipRequestInterceptor.kt
62.07% <100.00%> (-2.45%)
:arrow_down:

...g/android/core/internal/net/RotatingDnsResolver.kt
95.24% <100.00%> (+4.33%)
:arrow_up:

...rnal/system/BroadcastReceiverSystemInfoProvider.kt
97.92% <100.00%> (-0.12%)
:arrow_down:

...core/internal/system/DefaultAndroidInfoProvider.kt
86.96% <100.00%> (+0.16%)
:arrow_up:

... and 9 more

... and 16 files with indirect coverage changes


google/recaptcha Exception

Hi,
We are getting below error while checking the Recaptcha .
Failed to execute 'postMessage' on 'DOMWindow': The target origin provided ('https://www.google.com') does not match the recipient window's origin ('http://localhost').
Can you please suggest any solution.

I'm also having this issue. Is there a way to disable origin checking?

Closing super old issues. Please re-raise if still relevant.

+1


BUG: The login page css is disturbed

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
the login page is not managed properly. I want to reshape this page

Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@Satyam1923, I want to work on this bug.

@Aman91667 do you want to change the design of it also.

yes

i am anable to complite please unassign me


support for netcore-1.1

Thanks, I haven't used .net core, but I'll take your word that these changes are ok.


added (FCDS) , Alexandria University

added the Faculty of Computers and Data Science in Alexandria University (FCDS)

@Ali-Fathy-Abdelghany I am sorry, the domain alexu.edu.eg is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Make HiveMetastoreClient cached in a Thread Local way

to avoid creating too many connections.
This is to fix #39.

If it's the case that the connection breaks, there may be parts of the code that assume that calling getMetastoreClient() will create a new connection. With the ThreadLocal approach, that thread may retry indefinitely. Maybe we need to add a liveness check before returning the connection for this to work?

Looks like there are some checkstyle failures?

@plypaul You have a good point.  After looking at the code in more details, I decided to add the auto-reconnect capability into ThriftHiveMetastoreClient.  It's easy to verify the correctness of the code since all logics are captured in ThriftHiveMetastoreClient.

LGTM - thanks @zshao!


add edit user profile features.

admin can edit user's password / email.
remove avatar
see user's ip
detele users's all posts

thanks

Hello, I would love to be able to quickly assign and edit user's group if possible. Thanks !


Opaque material appears black and glitched (background repeats over it when i move the camera) But transparent works.

I'm having a trouble where a opaque material appears completely black:

And it glitches when i move the camera:

Doing that weird thing where it copies the border/background all over it.
The lite shader works, but i really needed the full one with a normal map. Also, when i set it to transparent it works:

I get no errors in the console, i've tried closing and opening the project, reimporting everything, nothing works.
I have unity 2021.3.8f1 and URP 12.1.7. This is probably something in my end, but i would appreciate some help if you know what could be causing this. Thank you!

In the repo, there is a demo scene called Additional Lights. It also features a rock with normal mapped material.
Could you please try running it with your setup?

Tried to run it using 2021.3.8f1 and URP 12.1.7, works as expected.


Failed to find the AppId.

Got this error, not sure what's the issue
Unhandled exception: MSStore.API.MSStoreException: Failed to find the AppId.
   at MSStore.CLI.Helpers.IStorePackagedAPIExtensions.EnsureAppInitializedAsync(IStorePackagedAPI storePackagedAPI, DevCenterApplication application, FileInfo directoryInfo, IProjectPublisher projectPublisher, CancellationToken ct) in /_/MSStore.CLI/Helpers/IStorePackagedAPIExtensions.cs:line [24](https://github.com/endworks/lovelust/actions/runs/5206050845/jobs/9392187704#step:15:25)0
   at MSStore.CLI.ProjectConfigurators.FileProjectConfigurator.PublishAsync(String pathOrUrl, DevCenterApplication app, DirectoryInfo inputDirectory, IStorePackagedAPI storePackagedAPI, CancellationToken ct) in /_/MSStore.CLI/ProjectConfigurators/FileProjectConfigurator.cs:line 152
   at MSStore.CLI.Commands.PublishCommand.Handler.InvokeAsync(InvocationContext context) in /_/MSStore.CLI/Commands/PublishCommand.cs:line 105
   at System.CommandLine.NamingConventionBinder.CommandHandler.GetExitCodeAsync(Object returnValue, InvocationContext context)
   at System.CommandLine.NamingConventionBinder.ModelBindingCommandHandler.InvokeAsync(InvocationContext context)
   at System.CommandLine.Invocation.InvocationPipeline.<>c__DisplayClass4_0.<<BuildInvocationChain>b__0>d.MoveNext()```

It seems like you have not executed the init command for your project. Just run, locally and only once: msstore init. This will list your apps and let you select which app from the store this app is related to. Select the right one, and commit the file changes in your pubspec.yaml and whatever else is changed. This will let the CLI know which app you are trying to publish when the CI runs the publish command.

Sorry for the delay in my response. I only receive this error:
💥 Could not retrieve your registered applications. Please try again.

Do you get any more details if you run it in verbose mode? Just add "-v" to the failing command.

Doesn't make any sense, everything looks good to me.
09:48:51 info: Microsoft.Hosting.Lifetime[0] Application started. Press Ctrl+C to shut down.
09:48:51 info: Microsoft.Hosting.Lifetime[0] Hosting environment: CLI
09:48:51 info: Microsoft.Hosting.Lifetime[0] Content root path: A:\Documents\Projects\lovelust
09:48:51 info: MSStore.CLI.Program[0] Command is init
09:48:51 info: MSStore.API.StoreAPI[0] Getting DevCenter authorization token
09:48:51 info: System.Net.Http.HttpClient.Default.LogicalHandler[100] => HTTP GET https://manage.devcenter.microsoft.com/v1.0/my/applications?skip=0&top=100 Start processing HTTP request GET https://manage.devcenter.microsoft.com/v1.0/my/applications?skip=0&top=100
09:48:51 info: System.Net.Http.HttpClient.Default.ClientHandler[100] => HTTP GET https://manage.devcenter.microsoft.com/v1.0/my/applications?skip=0&top=100 Sending HTTP request GET https://manage.devcenter.microsoft.com/v1.0/my/applications?skip=0&top=100

⣷ Retrieving all registered applications...
                                           09:48:52 info: System.Net.Http.HttpClient.Default.ClientHandler[101] => HTTP GET https://manage.devcenter.microsoft.com/v1.0/my/applications?skip=0&top=100 Received HTTP response headers after 1092.1803ms - 400
⣯ Retrieving all registered applications...efault.LogicalHandler[101] => HTTP GET https://manage.devcenter.microsoft.com/v1.0/my/applications?skip=0&top=100 End processing HTTP request after 1097.6315ms - 400
                                           09:48:52 fail: MSStore.CLI.Commands.InitCommand.Handler[0] Error while retrieving applications. MSStore.API.MSStoreException: Failed to get the applications - {"code":"Unauthorized","message":"A valid account could not be found with given authorization token. CorrelationId=f34b4528-d558-434e-86de-b6c721b25632"}  ---> MSStore.API.MSStoreException: {"code":"Unauthorized","message":"A valid account could not be found with given authorization token. CorrelationId=f34b4528-d558-434e-86de-b6c721b25632"}    at MSStore.API.SubmissionClient.InvokeAsync[T](HttpMethod httpMethod, String relativeUrl, Object requestContent, CancellationToken ct) in /_/MSStore.API/SubmissionClient.cs:line 255    at MSStore.API.Packaged.StorePackagedAPI.GetDevCenterApplicationsAsync(Int32 skip, Int32 top, Cancellation💥 Could not retrieve your registered applications. Please try again.at MSStore.API.Packaged.StorePackagedAPI.GetApplicationsAsync(CancellationToken ct) in /_/MSStore.API/Packaged/StorePackagedAPI.cs:line 120    --- End of inner exception stack trace ---    at MSStore.API.Package
09:48:53 info: Microsoft.Hosting.Lifetime[0] Application is shutting down...

It seems like your credentials are invalid. I would try regenerating them and updating your secrets.

@ender-null you might have missed this step where you have to "Add Microsoft Entra application"
Account settings -> User management -> Microsoft Entra application then  Add Microsoft Entra application
From there, add the application you created (or the CLI) created from "App registrations" and give it Manager(Windows) role
If you have the setup locally, try running msstore apps list.
It should Retrieve Managed Applications

https://partner.microsoft.com/en-us/dashboard/account/v3/usermanagement#users

Same issue here, already double checked my credential and restarted all over at least twice.
I'm able to receive apps info in the terminal.

but still failing here


[8.x] Incorrect FQCN for withAggregate() on self-relation

This PR provides a failing test case for when methods in the withAggregate()-family are used on an Eloquent "self-relation".
Aggregate queries on self-relations use a generated table alias, but the aggregate column will fail to be qualified to that alias, silently producing incorrect results, or an sql-error when sql_mode=only_full_group_by.
I not sure yet what the solution would be, but hopefully this failing-test PR can be a starting point to discuss a good approach!
Background
Laravel 8.13 included https://github.com/laravel/framework/commit/b0dc0b2432d426349009a70775fa469d16ea1a14 that added $relation->getRelated()->qualifyColumn($column) to qualify the column name in the aggregate subquery. This seems reasonable as that column name used to end up unqualified, and that is of course a risk.
Unfortunately the introduced qualification is done before knowing what the actual table alias will be in the final query.
It will always qualify using the table name from the relation for the aggregate column.
This goes unnoticed for common relationships between two different models, each with their own table...
What goes wrong?
For a relationship that references the same model (e.g. a student tutoring other students), the table in the subquery is aliased (see example in HasOneOrMany – variants are in other relationship types too).
So, the "child"-table of the subquery becomes aliased (a generated hash), while the selected aggregate column in the very same sub-query is still qualified to what at this point is actually the "parent" table. 😞
Example sql
This reflects current sql produced when ->withMin('created_at') is used on a self-relationship:
select `self_related_stubs`.*,

(select
min(`self_related_stubs`.`created_at`) -- This FQCN is incorrect, it should reference the alias set below!
from `self_related_stubs` as `self_alias_hash`
where `self_related_stubs`.`id` = `self_alias_hash`.`parent_id`
) as `child_foos_min_created_at`

from `self_related_stubs`

Sql error looks something like this:
In aggregated query without GROUP BY, expression #1 of SELECT list contains nonaggregated column 'test_db.self_related_stubs.id'; this is incompatible with sql_mode=only_full_group_by
Possible solutions
Can we go back to not qualifying the aggregate column?
Between 8.12 (when the super-useful withAggregate-methods were introduced in https://github.com/laravel/framework/commit/95d0c9a19179dd1ceefb4d51bb04f7aecbd2fdcf, thanks @khalilst!) and 8.13, aggregates on self-relations worked as expected (at least with mysql 8 that I've been running). The column name wasn't qualified at all and there was just a single from-table in the actual subquery part, so I guess it was unambiguous.

Was the unqualified column ambiguous in other sql-dialects than mysql?
Could there be cases where the aggregate subquery has joins or sub-sub-queries?
Perhaps contributor @dbakan has more information or examples as to why the FQCN was introduced?
Is there a policy to always qualify columns throughout illuminate/database?

What can we do if we want to keep the FQCN?

Can the qualification wait and be applied inside getRelationExistenceQueryForSelfRelation, where the final alias is in scope?
Is it possible to create a general "deferred-FQCN" functionality to be used throughout illuminate/database? So that one can pass an object containing an unqualified column name, indicating that builder methods may qualify at the point they're actually used in queries... This may not make sense... 🤔 I just made it up! 😄

Thanks for reading, and thanks for collaborating on this wonderful framework! Please share your thoughts on how we can solve this!

PS. In my app I've started using withMax('siblings', 'created_at') on a HasMany siblings self-relationship, combined with a having clause on the selected aggregate to create a HasOne relationship on the parent model that only includes the latest item 🤯
An equivalent example would be a Post model having many Comment in a comments HasMany relationship, but also a latestComment HasOne relationship, that can be effectively eager loaded etc. The resulting query is complex but the PHP syntax in the models is really readable. And the query performs just fine with proper indices 😄 This is something I've been wanting to do for years, and withAggregate finally made it possible using nothing but standard Eloquent methods!

The reason was described in https://github.com/laravel/framework/pull/35061#issuecomment-720389938
When using the feature on a belongsToMany, the pivot table and the related table might both have a column of the requested name. That's why I added this qualifyColumn wrapper.
I'm sorry, if this introduced another bug. Is there a way to access the aliases for that kind of relation? Will check on that.

@bjuppa Thanks for your full descriptions. I am trying to find the solution.
@dbakan Please let me know if you are working on the solution for this test case.

@dbakan & @khalilst, big thanks to both of you for joining the effort, and thanks for your earlier contributions!
So the qualification was needed to handle aggregates on belongsToMany... that's a good reason! There's no way we can go back and drop the FQCN completely.
My other option was to move the qualification to later, in the getRelationExistenceQuery method. What do you think about that idea, is it even feasible?
The other way around, to somehow "guess" the alias before getRelationExistenceQuery seems very hard to me, as the final alias is accessible within getRelationExistenceQueryForSelfRelation only.

@bjuppa @dbakan
I found the solution. I will create a PR here.

Just create a fresh pr if you have to thanks.

@khalilst Thank you very much!

@dbakan My pleasure.
@driesvints Sorry for bothering. I have to include the test provided here by @bjuppa  in #35392 ?


added default shortcut command

Adds default keyboard shortcut of Alt+Shift+A to open Alby extension's popup
Type of change (Remove other not matching type)

New feature (non-breaking change which adds functionality)

Describe the changes you have made in this PR -
added default shortcut command. super simple, just executes the browser action of opening Alby's popup with the command Alt+Shift+A by default on install.
Screenshots of the changes (If any) -
How Has This Been Tested?
Uninstalled, reinstalled in chrome and firefox, installs with keyboard shortcut by default.
Checklist:

[ x ] My code follows the style guidelines of this project and performed a self-review of my own code
[ x ] New and existing tests pass locally with my changes
[ x ] I checked if I need to make corresponding changes to the documentation (and made those changes if needed)

oh this is cool! works great.
on a mac it is Option + Shift + A


[beta] Phone viewport improperly spaced with respect to code editor

The phone viewport is overlapping with editor in beta site.

I have tested this on other challenges too.
Browser: Google Chrome (latest) on Linux and Windows

ref #9955

Closing this in favor of the original report.


[Bug] Casing not preserved in seeds

Is this a new bug?

[X] I believe this is a new bug
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
dbt appears to successfully create new seeds from case sensitive file names e.g. mySeed but then errors with 'ambiguous relation' later on because the seed created actually follows the default casing of the warehouse (lower on Redshift, UPPER on Snowflake, etc). Having a cased seed in your project essentially results in an unclear error.
Expected Behavior
I would expect one of two things:

dbt throws an error or warning about cased seeds
dbt preserves the casing of the seed

Steps To Reproduce

On Redshift
Create a seed called 'mySeed.csv' with whatever data
dbt seed.
dbt seed again or dbt build, anything that would involve referencing that seed.

Relevant log output
/home/circleci/project
[0m14:35:31  [[33mWARNING[0m]: Deprecated functionality

User config should be moved from the 'config' key in profiles.yml to the 'flags' key in dbt_project.yml.
[0m14:35:31  Running with dbt=1.8.0-b2
[0m14:35:31  Installing ../
[0m14:35:31  Installed from <local @ ../>
[0m14:35:31  Installing dbt-labs/dbt_utils
[0m14:35:31  Installed from version 1.1.1
[0m14:35:31  Up to date!
[0m14:35:33  [[33mWARNING[0m]: Deprecated functionality

User config should be moved from the 'config' key in profiles.yml to the 'flags' key in dbt_project.yml.
[0m14:35:33  Running with dbt=1.8.0-b2
[0m14:35:33  Registered adapter: redshift=1.8.0-b2
[0m14:35:33  Unable to do partial parsing because config vars, config profile, or config target have changed
[0m14:35:33  Unable to do partial parsing because env vars used in profiles.yml have changed
[0m14:35:33  Unable to do partial parsing because a project dependency has been added
[0m14:35:36  Found 6 models, 23 data tests, 4 seeds, 2 sources, 617 macros
[0m14:35:58  [[33mWARNING[0m]: Deprecated functionality

User config should be moved from the 'config' key in profiles.yml to the 'flags' key in dbt_project.yml.
[0m14:35:58  Running with dbt=1.8.0-b2
[0m14:35:59  Registered adapter: redshift=1.8.0-b2
[0m14:35:59  Found 6 models, 23 data tests, 4 seeds, 2 sources, 617 macros
[0m14:35:59
[0m14:36:01  Concurrency: 1 threads (target='redshift')
[0m14:36:01
[0m14:36:01  1 of 4 START seed file codegen_integration_tests_redshift_Raw_Data_Case_Sensitive.data__Case_Sensitive  [RUN]
[0m14:36:01  1 of 4 ERROR loading seed file codegen_integration_tests_redshift_Raw_Data_Case_Sensitive.data__Case_Sensitive  [[31mERROR[0m in 0.03s]
[0m14:36:01  2 of 4 START seed file codegen_integration_tests_redshift_raw_data.data__a_relation  [RUN]
[0m14:36:02  2 of 4 OK loaded seed file codegen_integration_tests_redshift_raw_data.data__a_relation  [[32mCREATE 2[0m in 0.70s]
[0m14:36:02  3 of 4 START seed file codegen_integration_tests_redshift_raw_data.data__b_relation  [RUN]
[0m14:36:02  3 of 4 OK loaded seed file codegen_integration_tests_redshift_raw_data.data__b_relation  [[32mCREATE 2[0m in 0.60s]
[0m14:36:02  4 of 4 START seed file codegen_integration_tests_redshift_raw_data.data__campaign_analytics  [RUN]
[0m14:36:03  4 of 4 OK loaded seed file codegen_integration_tests_redshift_raw_data.data__campaign_analytics  [[32mCREATE 3[0m in 0.63s]
[0m14:36:03
[0m14:36:03  Finished running 4 seeds in 0 hours 0 minutes and 3.81 seconds (3.81s).
[0m14:36:03
[0m14:36:03  [31mCompleted with 1 error and 0 warnings:[0m
[0m14:36:03
[0m14:36:03    Compilation Error in seed data__Case_Sensitive (seeds/data__Case_Sensitive.csv)
  When searching for a relation, dbt found an approximate match. Instead of guessing
  which relation to use, dbt will move on. Please delete "ci"."codegen_integration_tests_redshift_raw_data_case_sensitive"."data__case_sensitive", or rename it to be less ambiguous.
  Searched for: "ci"."codegen_integration_tests_redshift_Raw_Data_Case_Sensitive"."data__Case_Sensitive"
  Found: "ci"."codegen_integration_tests_redshift_raw_data_case_sensitive"."data__case_sensitive"

  > in macro materialization_seed_default (macros/materializations/seeds/seed.sql)
  > called by seed data__Case_Sensitive (seeds/data__Case_Sensitive.csv)
[0m14:36:03
[0m14:36:03  Done. PASS=3 WARN=0 ERROR=1 SKIP=0 TOTAL=4

### Environment

```markdown
- OS: Alpine in Docker
- Python: 3.9.9
- dbt-adapter: redshift=1.8.0-b2

Additional Context
No response

after discussing with @gwenwindflower, I advised that she open the error here because the issue happens on both dbt-snowflake and dbt-redshift
the error is ApproximateMatchError raised by BaseAdapter.matches() (below), but the root cause might be due to the seed materialization and corresponding helper macros not properly quoting.
https://github.com/dbt-labs/dbt-adapters/blob/4774638f334908815ec685bb21877ac10ce200df/dbt/adapters/base/relation.py#L127-L129

Might be silly, maybe it's not implemented for all adapters, or maybe I'm missing something very obvious, but you can specify quoting seed columns in dbt code.

https://docs.getdbt.com/reference/seed-properties
https://docs.getdbt.com/reference/seed-configs
This page even says "Explicitly set this value if using seed files." https://docs.getdbt.com/reference/resource-configs/quote_columns

i'd be curious to know if wrapping the below this.render() within `adapter.quote()
https://github.com/dbt-labs/dbt-adapters/blob/1edeb74fbcfd6526e356f8b99e55dbba8b9990ed/dbt/include/global_project/macros/materializations/seeds/helpers.sql#L108-L110


What is the return value if the container is not found?

It would be nice to know what the return value is when the container is not found.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: abc7d1b9-d40f-3f4e-11ce-8dd040da095e
Version Independent ID: 127f5081-0f4e-01b4-a3e8-07a0de678c93
Content: IItemContainerMapping.IndexFromContainer(DependencyObject) Method (Windows.UI.Xaml.Controls) - UWP app developer
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.ui.xaml.controls.iitemcontainermapping.yml&version=GBlive&_a=contents
Product: windows
Git Login: wdg-dev-content
Microsoft Alias: wdg-dev-content

Hi, sorry for the delayed response to this issue. The docs have been updated with this information. (Returns -1 if container is not found.)


State Change, CharacterOwnership Signals

Uses Extension logger
Adds a CharacterOwnership added and removed signal
Adds a State Change signal
re-enables the debug logging
Signal Sample https://puu.sh/GvdOS/5caf8675a5.png

Looks good


Fixes possible empty manipulation groups while iterating over a ManipulationSequence instance

When iterating over a ManipulationSequence instance (which happens at here for example), one iterates the "raw" groups array of ManipulationSequence. If this array has been merged, which always happens here because Image sets $this->manipulations to a new instance of Manipulations , an empty group is yielded.
This causes GlideConversion::performManipulations to run a manipulation that makes no sense. I guess this was not noticed earlier because the tests nicely use ManipulationSequence::toArray and Glide does not trigger an error when it's called without any manipulation arguments.
This patch ensures no new group is being added if this is not necessary and it returns a new ArrayIterator of the result of ManipulationSequence::toArray.

Nice catch! Thanks!


url vmExtensionScriptLocation

Hi MS Team,
What is this URL "https://raw.githubusercontent.com/Azure/azurestack-quickstart-templates/master/appservice-fileserver-standalone" stand for ?
I found that link in vmExtensionScriptLocation section, but I can't directly access that link from browser. Is that URL reachable or is it broken link?

@chandrakn
This is why:
IT is a parameter set here:
"vmExtensionScriptLocation": {
"type": "string",
"defaultValue": "https://raw.githubusercontent.com/Azure/azurestack-quickstart-templates/master/appservice-fileserver-standalone",
And it is then used here:
"[concat(parameters('vmExtensionScriptLocation'), '/scripts/OnStartAzureVirtualMachineFileServer.ps1')]",
"[concat(parameters('vmExtensionScriptLocation'), '/scripts/fileserver.cr.zip')]"
Same principal applies to BaseURLs

Closing issue. @chandrakn please reopen if the question is not fully answered yet.


Add pre-commit support

Closes #2

Also, I see there was no merge commit created for this PR, and the PR commits were not squashed as one - I am assuming you rebased the commits onto the main branch. Rebase should be good in most cases but it can result in weird behavior. Rebase also re-writes the git history while it places the commit on top of the existing HEAD. Additionally, you might encounter a bit more issues with rebase on GitHub (you'll only have to worry about the first point if you merge or squash and merge) -

You aren't able to automatically rebase and merge on GitHub.com when:

The pull request has merge conflicts.
Rebasing the commits from the base branch into the head branch runs into conflicts.
Rebasing the commits is considered "unsafe," such as when a rebase is possible without merge conflicts but would produce a different result than a merge would.

I'd suggest we use squash and merge if you want to get rid of the merge commits. The only time when we should not use squash and merge is when a file is renamed and the content within that file is modified a bit (these 2 should be carried out in 2 separate commits, not necessary, just good practice).

Yes, I was using rebase and merge to avoid merge commits. Though I would still want some commit history to show up, just to keep track of changes to files over time (since this repository has many content-based files), so I'll switch to using normal merges instead for all further PRs rather than squash and merge


Chatlio hasn't turned on automatically today

title
Chatlio hasn't turned on automatically today at 9:00 am
environment
tested on chrome
expected result
chatlio is supposed to work automatically - 9:00am - 5:15 pm
actual result
chatlio was in the offline mode - asking for the email for further contact
steps to reproduce
n/a
proof
chat received at 11am in the offline mode

Severity/Priority
Minor
number of users already affected (reporting this issue) of total users
1
team members who tested (apart from creator of bug report)
MM & DK

@jprzybylski4 im switching off the presence detection - we will see how things will perform


#1278 Show latest deployment in stage overview

The list of stages now shows the image from the latest sh.keptn.events.deployment-finished on that stage and is linked if deploymentURIPublic is provided.

I like the enhancement.
But I see no need for the information "2 Services" when I see the two tags.

Besides, what does a tag show when just an onboarding but no deployment happened?

I like the enhancement.
But I see no need for the information "2 Services" when I see the two tags.

Besides, what does a tag show when just an onboarding but no deployment happened?

If the service is onboarded, but now deployment happened yet, then there will be no tags, just the information "x Services".
I think that the information "x Services" makes sense, although it is redundant, because it's like a title to the tags. But it could also be omitted, if you think that it's "to much".

While I agree with @ermin-muratovic, I believe the information "x services" is not needed most of the time.
I vote for removing the "x services" information for now. Long term we should have an API endpoint that should return a list of services and whether an image has been deployed or not.

I removed the "x services" title and removed the link to deploymentURIPublic. Clicking on the service-tag now will bring you to the root event and scroll to the specific stage. There you can inspect that deployment and if you want you will find there also the link to deploymentURIPublic with the sh.keptn.events.deployment-finished-Event


[WIP] General heteropolymer

Description
generalizing the heteropolymer build process.
Todos
Notable points that this PR has either accomplished or will accomplish.

Now possible to build very general heteropolymers
Also cleaned the build code, removing backbone/sidechain code paths
Removed some duplicative checks.

Questions

[ ] Not clear how to add new functional forms easily.

Status

[ ] Almost ready to go, is functional, just want to clean up a bit and collect thoughts on organization.

Codecov Report

Merging #37 into master will increase coverage by 0.55%.
The diff coverage is 8.37%.

I ran get_heteropolymer_1_structure.py and it looks like the Martini parameters are still being used instead of defaults we are setting in the input script. The bond lengths set explicitly aren't being used either:
No bond_lengths definition provided for 'asc_abb_bond_length', setting to 0.47 nm No bond_lengths definition provided for 'bbb_abb_bond_length', setting to 0.47 nm No bond_lengths definition provided for 'bbb_bbb_bond_length', setting to 0.47 nm No bond_lengths definition provided for 'bsc_bbb_bond_length', setting to 0.47 nm No bond_lengths definition provided for 'cbb_bbb_bond_length', setting to 0.47 nm No bond_lengths definition provided for 'cbb_cbb_bond_length', setting to 0.47 nm No bond_lengths definition provided for 'csc_cbb_bond_length', setting to 0.47 nm No bond_lengths definition provided for 'abb_cbb_bond_length', setting to 0.47 nm
Defaults we are setting in the input script for bond_force_constant, bond_angle_force_constant, equil_bond_angle are also ignored.
The connectivity of the 12mer_ABC4s model seems correct, but the bonds are definitely all 4.7A measured in VMD.

I ran get_heteropolymer_1_structure.py and it looks like the Martini parameters

Ah, yeah, a couple bugs there. Pulling default parameters from the wrong place, and not catching a typo in the other definitions.  This upcoming commit should fix it.

Looks good now - the defaults are getting set the way they should, and the topology looks correct.
My only issue is that the in the pdb file, we cannot tell which particle type is which. For example, in the second heteropolymer example, we write X1, X2, X3, X4, X5 for residue type A. We could write particle names instead: a, a, a, b, b.

Good point.  Overall what SHOULD the PDB naming convention be for these models?  What would the format look like?

n the pdb file, we cannot tell which particle type is which. For example, in the second heteropolymer example, we write X1, X2, X3, X4, X5 for residue type A. We could write particle names instead: a, a, a, b, b.

OK, I redid the naming for the pdb to try to match the pdb labeling for proteins.  Most of these changes are in cg_openmm, but I reran the build scripts here to match.
 The 4th entry is the first 3 letters of the monomer name, the 5th is the chain number (will always be A for a single residue -this is optional, we could get rid of it).

I agree with the new pdb formatting, this looks good. I think we can leave the 'A' for chain id.


Add PagedFlux and PagedIterable to Files and Queues

/azp run java - storage - ci

/azp run java - storage - ci

@jaschrep-msft The specification states to use PagedFlux in the case of non-paged list returns. While I'll agree that PagedFlux doesn't tightly align with non-paged responses it allows for all headers and response data to be accessible which is another tenant of the Track 2 clients.

@alzimmermsft I was directed to not use this in the blob client. Opening an issue for the blob client to use this for its non-paged listing operations.


DO NOT MERGE

Why are the changes needed?

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Was this patch authored or co-authored using generative AI tooling?

Codecov Report

Merging #5420 (494f256) into master (1b229b6) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #5420   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files         588     588
  Lines       33423   33423
  Branches     4391    4391
======================================
  Misses      33423   33423

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Console output not outputting the correct folder paths

This is a note to self issue of a bug I found.
Gulp file portion
var sassDest = local_theme_folder+'/sass';

function load_sass(folder) {
	var dest = sassDest+'/'+folder;
	return gulp.src(local_theme_folder+'/sass/'+folder+'/**/*.scss')
		.pipe(fileLoader({ preset: 'scss', dest, fileName: '__includes.scss' }))
		.pipe(gulp.dest(dest))
}

var scssFolders = ['variables', 'mixins', 'components', 'pages'];
var scssLoadTasks = scssFolders.map(folderName => 'load:scss:'+folderName);

scssFolders.forEach((folderName, i) => {
	gulp.task(scssLoadTasks[i], function(){
		return load_sass(folderName)
	})
})

gulp.task('load:scss', scssLoadTasks);

console output
[10:03:30] Starting 'load:scss:variables'...
[10:03:30] Starting 'load:scss:mixins'...
[10:03:30] Starting 'load:scss:components'...
[10:03:30] Starting 'load:scss:pages'...
gulp-file-loader: Generated ../govcms/sites/all/themes/ipaustralia/sass/pages/__includes.scss
[10:03:30] Finished 'load:scss:variables' after 49 ms
gulp-file-loader: Generated ../govcms/sites/all/themes/ipaustralia/sass/pages/__includes.scss
gulp-file-loader: Generated ../govcms/sites/all/themes/ipaustralia/sass/pages/__includes.scss
[10:03:30] Finished 'load:scss:pages' after 63 ms
[10:03:30] Finished 'load:scss:mixins' after 67 ms
gulp-file-loader: Generated ../govcms/sites/all/themes/ipaustralia/sass/pages/__includes.scss
[10:03:30] Finished 'load:scss:components' after 69 ms
[10:03:30] Starting 'load:scss'...
[10:03:30] Finished 'load:scss' after 33 μs

This would be because the dest setting is getting mutated and so the output paths are incorrect as a result of this issue.

Fixed in 2.0.2


Multiple administrators issue counting hosts

Checklist

Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image? Yes
Are you sure you're not using someone else's docker image? Yes
If having problems with Lets Encrypt, have you made absolutely sure your site is accessible from outside of your network? Not the issue.

Describe the bug
When logged in with one administrator the counters on the dashboard only count the hosts this administrator added.
You can only see other hosts if you click on it, these are made by other administrators.
Also there was reported a other bug when you delete a administrator the dashboard wouldn't work, think this is related.
Version: V2.4.0
To Reproduce

Add multiple administrators
Add some hosts with different administrators
The counters of hosts only count the hosts added with the current logged on administrator.

Expected behavior
Every administrator should see in the counter the number of total added hosts, doesn't matter by which administrator.
Also when you delete a administrator this shouldn't make a problem with his added hosts.
Screenshots

Operating System
CentOS 8 with docker.
Additional context
Client: Docker Engine - Community
Version:           19.03.12
API version:       1.39
Go version:        go1.13.10
Git commit:        48a66213fe
Built:             Mon Jun 22 15:46:54 2020
OS/Arch:           linux/amd64
Experimental:      false
Server: Docker Engine - Community
Engine:
Version:          18.09.1
API version:      1.39 (minimum version 1.12)
Go version:       go1.10.6
Git commit:       4c52b90
Built:            Wed Jan  9 19:06:30 2019
OS/Arch:          linux/amd64
Experimental:     false
Browser version: Chrome 84.0.4147.125

This issue also exists when the user is not an administrator. Whenever I create a new user and give it full read permissions, the proxy hosts counter still says "0".

Can confirm still an issue as of v2.9.8, have had a few of my admin staff comment on it as well.


CVE fix for all repos

Issue tracker for CVE-2023-44487 hotfix.
Gitlab CPaaS builds of ModelMesh and KServe have been updated to ubi8/go-toolset:1.19.
In general, across all repos:

golang.org/x/net/http2 updated to v0.17.0 in go.mod
google.golang.org/grpc upgraded to v1.56.3 in go.mod
ubi-minimal base image updated to 8.8

More modules were updated based on the SNYK report suggestions. CVE report on SNYK can be seen here.
Note: vulnerabilities in /docs in all repos and /client in OVMS repo can be ignored since they are not in shipped in the product.

GitHub Issues is not allowing addition of more than 10 PRs. Here are the PRs from downstream:
kserve
* 1.32: https://github.com/red-hat-data-services/kserve/pull/26
* 1.33: https://github.com/red-hat-data-services/kserve/pull/27
* 1.34: https://github.com/red-hat-data-services/kserve/pull/28
modelmesh-serving
* 1.32: https://github.com/red-hat-data-services/modelmesh-serving/pull/24
* 1.33: https://github.com/red-hat-data-services/modelmesh-serving/pull/25
* 1.34: https://github.com/red-hat-data-services/modelmesh-serving/pull/26
modelmesh

1.34: https://github.com/red-hat-data-services/modelmesh/pull/12
1.33: https://github.com/red-hat-data-services/modelmesh/pull/13
1.32: https://github.com/red-hat-data-services/modelmesh/pull/14

modelmesh-runtime-adapter
* 1.32:  https://github.com/red-hat-data-services/modelmesh-runtime-adapter/pull/6
* 1.33: https://github.com/red-hat-data-services/modelmesh-runtime-adapter/pull/7
* 1.34: https://github.com/red-hat-data-services/modelmesh-runtime-adapter/pull/8
rest-proxy
*1.32: https://github.com/red-hat-data-services/rest-proxy/pull/2
*1.33: https://github.com/red-hat-data-services/rest-proxy/pull/3
*1.34: https://github.com/red-hat-data-services/rest-proxy/pull/4
odh-model-controller
*1.32: https://github.com/red-hat-data-services/odh-model-controller/pull/22
*1.33: https://github.com/red-hat-data-services/odh-model-controller/pull/21
*1.34: https://github.com/red-hat-data-services/odh-model-controller/pull/20


Migrate restoreExperiment endpoint.

PR related to -> https://github.com/users/jgiannuzzi/projects/2?pane=issue&itemId=72542972

please add the Python store bindings too

@jgiannuzzi probably means adding the correct entry in https://github.com/jgiannuzzi/mlflow-go/blob/main/mlflow_go/store/tracking.py?


upgrade to python 3.10 for github tests

Can you put a comment in pyproject.toml next to the Python version to remind us to keep this in sync?


404 page

Who wants to have fun?

Ideas:

California sinking off the continent
Exploding/shattering geobear
geobear => real bear

@yarynam @NathanLawrence Wanna help? Could use some of your SVG animation skills.

@gordonje  Would like to =)

There's a free t-shirt in it for you!


Support link highlighting on capitalized link scheme

Summary
Support link highlighting on capitalized link scheme. A scheme refer to the first part of a URL, such as https://. Currently, https:// gets highlighted, but Https:// (capital H) does not.
As a fix, ignore capitalization to highlight both versions. (This is useful, since on mobile, most users have auto-capitalize sentence turned on, which capitalizes the first character of a sentence / message.)
Ticket Link
Jira ticket: 29280
Note: this Jira ticket mentioned Android, but I fixed this on iOS. However, the issue seems pretty universal to me.
Checklist

All unrelated.
Device Information
This PR was tested on: iPhone 12, iOS 15.1.1
Screenshots

Release Note
Added support for link highlighting when a capitalized link scheme, such as Https:// instead of https://

Hello @CeesJol,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/20994
https://circleci.com/gh/mattermost/mattermost-mobile/20993

Artifact links:
https://20994-70265724-gh.circle-artifacts.com/0/tmp/artifacts/Mattermost_Beta.ipa
https://20993-70265724-gh.circle-artifacts.com/0/tmp/artifacts/Mattermost_Beta.apk

Added some unit tests @enahum


SB-1404 org.carlspring.strongbox.providers.io.RepositoryPath#toUri simplified

…mplified

@fuss86 can you please provide how deploy time affected with thin changes?
Also can you please clarify how this changes improves the toUri logic?

local deploy time on master:
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 07:16 min
[INFO] Finished at: 2019-07-01T21:42:43+02:00
[INFO] Final Memory: 224M/926M
[INFO] ------------------------------------------------------------------------

local deploy from this branch
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 07:01 min
[INFO] Finished at: 2019-07-01T21:59:02+02:00
[INFO] Final Memory: 223M/813M
[INFO] ------------------------------------------------------------------------

Please also compare build times in Jenkins

this branch

linux
windows

8min 57s
12min 6s

master

linux
windows

9min 24s
12min 26s

Also can you please clarify how this changes improves the toUri logic?

the logic hasn't changed, since I haven't had to change any test, so it's rather a refactoring change

it simplifies this method (it reduces code invoked)

getTarget().toUri() reduced from 2 to 0 occurrences
URI.resolve reduced from 1 to 0 occurrences
it doesn't need to call RepositoryPath.toAbsolutePath if !isAbsolute() is true
it doesn't relativize if the path is already !isAbsolute()

https://jenkins.carlspring.org/job/strongbox/job/builds/job/strongbox-against-strongbox-from-sources/
Differences time in deploy phase does not look so significant, but please compare Building with cold instance...

this branch (build #223)

3min 56s

master (build #222)

5min 1s

@fuss86 thanks for clarifying, can you also have a look at this comment

Rebased to master

@fuss86 : Could you please rebase this? Thanks! :)

@carlspring , done

rebased again :)

Thanks for this fix @fuss86 :)


Create new HTTP interaction interface

Move the interaction-related requests from the primary Client type to a new InteractionClient type. A new Client::interactions method creates this new client by taking an ApplicationId; this removes the need for set_application_id and application_id on the primary client, as well as the interaction error type variant for the application ID not being configured from every request.
Requests now look like client.interaction(application_id).method instead of client.method.

Codecov Report

Merging #1275 (09987c0) into next (006424b) will increase coverage by 0.10%.
The diff coverage is 9.37%.

:exclamation: Current head 09987c0 differs from pull request most recent head 4e85700. Consider uploading reports for the commit 4e85700 to get more accurate results

@@            Coverage Diff             @@
##             next    #1275      +/-   ##
==========================================
+ Coverage   30.91%   31.02%   +0.10%
==========================================
  Files         361      362       +1
  Lines       14291    14215      -76
==========================================
- Hits         4418     4410       -8
+ Misses       9873     9805      -68

Impacted Files
Coverage Δ

http/src/client/builder.rs
51.16% <ø> (+2.18%)
:arrow_up:

...application/interaction/create_followup_message.rs
0.00% <ø> (ø)

...application/interaction/delete_followup_message.rs
75.00% <ø> (ø)

...pplication/interaction/delete_original_response.rs
0.00% <ø> (ø)

...st/application/interaction/get_followup_message.rs
75.00% <ø> (ø)

...t/application/interaction/get_original_response.rs
0.00% <ø> (ø)

...application/interaction/update_followup_message.rs
0.00% <ø> (ø)

...pplication/interaction/update_original_response.rs
0.00% <ø> (ø)

http/src/request/application/mod.rs
6.25% <ø> (+0.36%)
:arrow_up:

http/src/client/interaction.rs
6.45% <6.45%> (ø)

... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 006424b...4e85700. Read the comment docs.

Maybe it would be great to notice that in the Client type documentation. The Client type has many methods and the interaction method will be displayed at the bottom of the list.

This might actually be a use case for separate impl blocks, having all the regular methods in an impl block that appears after this sub-client method would make it more visible

Maybe it would be great to notice that in the Client type documentation. The Client type has many methods and the interaction method will be displayed at the bottom of the list.


Convert a record of functions to a structurally typed value.

See https://github.com/typelevel/shapeless-3/issues/84


refactor(ubi): Removing the hardcoded 4-second wait when creating a UBI job

Waiting for 4 seconds is very dirty. Due to network issues or other factors, it is quite normal for the image pull or job creation to take longer than 4 seconds.

@yejiayu thanks for your contribution, The hard code is just a temporary solution, can you submit the PR  to the branch doubi? once it is tested, we will merge it to the releases
https://github.com/swanchain/go-computing-provider/tree/doubi

@yejiayu thanks for your contribution, The hard code is just a temporary solution, can you submit the PR to the branch doubi? once it is tested, we will merge it to the releases
https://github.com/swanchain/go-computing-provider/tree/doubi

Is the branch name doubi?

@yejiayu thanks for your contribution, The hard code is just a temporary solution, can you submit the PR to the branch doubi? once it is tested, we will merge it to the releases
https://github.com/swanchain/go-computing-provider/tree/doubi

Is the branch name doubi?

yes


Is there paper name with model?

For example which paper corresponding to CSTN?

I get it, https://libcity.ai/#/model


Change slides/.md file

Caption update
Done

branched
made a commit

Todo

[ ] merge branch
[ ] delete branch

PR comment


This looks like a bug.

In line, https://github.com/ankush-me/SynthText/blob/5f2b8583ad49ac3ce2623a5884524025bda06a3a/synthgen.py#L87 condition will always be true, because of the missing bracket.
it should be area[idx]/(w*h) > TextRegions.pArea).
I made this change, and a few bad regions where text was not clearly rendered are getting filtered now.

Thanks --- indeed it is. Fixed in the commit above.


Copy 2

Updating html file by completing About Section.
Updating Css my completing designing About Section and Creating Scrollbar.
Changing typo in html line 19 and 55
Changing typo in css line 5
Adjusting Navigation Bar Fonts.

Updating html file by completing About Section.
Updating Css my completing designing About Section and Creating Scrollbar.
Changing typo in html line 19 and 55
Changing typo in css line 5
Adjusting Navigation Bar Fonts.


chore: bdd-spring-1586694598 to 0.0.1

chore: Promote bdd-spring-1586694598 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Instructions in INSTAll don't work to build current version

The problem after running cmake, at the make stage, is:
/home/richard/git/libobjc2/arc.mm:6:10: fatal error:
'third_party/robin-map/include/tsl/robin_map.h' file not found
#include "third_party/robin-map/include/tsl/robin_map.h"
While anyone really determined will work out, after a while, that this is due to a missing dependency that needs to be found and installed, it really shouldn't be necessary.  The cmake/make process should handle all that seamlessly.  Is there perhaps some way for git to manage it so that the required third party project is always available as part of the libobjc2 project (ie if libobjc2 is cloned or downloaded from github as a zip file, the robin-map code would be part of it)?

You appear not to have checked out the submodules.  Did you clone with git clone --recursive?  If not, then you will need to run the git submodule commands to pull in the submodules.

On 15 Dec 2019, at 11:15, David Chisnall<EMAIL_ADDRESS>wrote:
You appear not to have checked out the submodules. Did you clone with git clone --recursive? If not, then you will need to run the git submodule commands to pull in the submodules.

Well that would be because I didn't know about those commands/options or that I needed to).
The issue is not that I couldn't find what I needed eventually, but that I should't need to fight to be able to build/install things, and that the INSTALL instructions should be complete/accurate.
Basically, if you go to the project on GitHub, and follow the normal instructions to clone or download a zip archive, the INSTALL instructions simply won't work.

You just need to follow the normal steps for cloning a git project that has submodules.  The install instructions cover how to build the code, they assume that you've already successfully acquired it.

On 15 Dec 2019, at 16:52, David Chisnall<EMAIL_ADDRESS>wrote:
You just need to follow the normal steps for cloning a git project that has submodules.

I think the problem is this notion of 'normal' steps(or the notion that people know about submodules).
If you go to the project on GitHub the only instruction for getting the source is the 'Clone or download' menu, which gives you a URL to use or a button to download a zip archive.  So GitHub seems to disagree with you about what's 'normal'.
Of course, we might say to people "Don't use GitHub",  and put a copy on the GNUstep ftp server, if we don't want them to have a hard time finding they've only got part of the required source.

I quite agree with Richard: It is not very well discoverable that libobjc2 now is a "project that has submodules". A simple git checkout of the clone URL displayed on github will not give any indication that you're missing the submodule (at least not with the git configuration on any machine that I own).

I'm happy to take a PR that documents this, but given the prevalence of submodules and the fact that this is a pretty common git workflow, I'm surprised that it's causing problems.


Ignore errors on closing sockets

Previously, when you manually closed a socket and an error fired as you
did so, it would fire the close event twice on the socket, once for the
error, and once for the actual close. This was causing the reconnection
logic to fire twice, the first time realising it was a clean close, and
the second thinking it was an error, therefor reconnecting.

Seems like test have gotten flakey :/ be super awesome if you could take a took at it

They are failing due to the port being used, it is always gonna happen on a pr due to both running

Are you sure? The failures don't look like they were a result of port conflicts, and the PR and push jobs were run on two separate physical machines.

 Here's the log: https://travis-ci.org/WatchBeam/carina/jobs/160723354

Ah, you're looking at just the last failing test. That was almost certainly a result of an error thrown in a previous test preventing the old socket server from being torn down. That's not a result of the tests running together (if it was it'd be a pretty big security vulernability in Travis ;) )

Can we merge this, builds work fine locally.

@connor4312 @UnwrittenFun can we get this resolved.


Create GeoGit backend

GeoGit is a DVCS for geospatial data.   It adds the ability to track provenance, history, and perform diffs on different data sets.
Ultimately it would be ideal to be able to store and track this information for any feature stored in GeoWave - as a first step to this implementing a GeoGit backend in accumulo (leveraging geowave where possible) is desired.
Geogit has a pluggable data store implementation - it looks like there's a concept of a GraphDatabase, an ObjectDatabase, and a StagingDatabase that need to be implemented.   The MongoDB backend provides a good example of implementing all three of these (see [3]).
The GraphDB implementation can be done from scratch, but it looks like there are canned implementations that leverage the BluePrints API.   There's an a project ([4]) that implements a blueprints-api on accumulo which may speed this up (state of this project is currently known (stability, quality, etc.).
Note that further investigation is needed to determine to what extent GeoWave and GeoGit objects can be co-mingled.   It would be ideal to not duplicate any date when not strictly needed.  It might also be desireable to keep historical data (diffs, versions), in a separate table to keep "current state" queries quick.  Working out the appropriate direction here would be done in conjunction with this task.
There are two groups on the geogit dev list working an HBase (may be migrating to something else - Ceph?) object store as well as spatial indexing - currently most of the stuff seems pretty rudimentary, but might be worth keeping an eye on.
[1] GeoGit: https://github.com/boundlessgeo/GeoGit
[2] DevDocs: https://github.com/boundlessgeo/GeoGit/blob/master/doc/technical/source/developers.rst
[3] MongDB implementation: https://github.com/boundlessgeo/GeoGit/tree/master/src/storage/mongo
[4] Accumulo-Blueprints project: https://github.com/mikelieberman/blueprints-accumulo-graph
[5] Group working NoSQL object database (Hbase - now Ceph?): http://geogitobjdb.blogspot.com/
[6] GeoGit spatial index discussion: https://groups.google.com/a/boundlessgeo.com/forum/#!searchin/geogit/spatial$20index/geogit/9yVQAFL4n4I/VZDFrCsh3kgJ
[7] GeoGit discussion group: https://groups.google.com/a/boundlessgeo.com/forum/#!forum/geogit

+1 for geogit / geogig


Issue 178 - Sort Managed Insalls List

I created a function in the munkireport model to sort the associative
array $report[$report_key]. it will sort by display name if available
and then name if it isn’t.

I did a merge from your upstream master - is it normal for all those "Merge remote-tracking branch 'upstream/master' into wip" lines to show up in the pull request?

Also - not entirely sure why
Arjen van Bochoven	More robust integer_toversion js	 2ac7e9c
Arjen van Bochoven	Fix migration	 1bec9b6
Arjen van Bochoven	Merge branch 'bug fix'
is showing up in my pull request either?

Not sure about the merge issues, but you should pull against the wip branch, not against master.
Apart from that, I think it would be better to use dataTables for sorting on the Managed Installs table. That way, you can also search a potentially large table. See the inventory_items_tab

Closing pull request because I have no idea how to fix the merge issues- so redo using dataTables as well.


numba.cuda.cudadrv.driver - INFO

How could I shut down the information like:
2020-08-31 17:22:14,168 - numba.cuda.cudadrv.driver - INFO - add pending dealloc: cuMemFree_v2 283136 bytes
2020-08-31 17:22:14,170 - numba.cuda.cudadrv.driver - INFO - add pending dealloc: cuMemFree_v2 70656 bytes
2020-08-31 17:22:14,171 - numba.cuda.cudadrv.driver - INFO - add pending dealloc: cuMemFree_v2 610512 bytes
2020-08-31 17:22:14,172 - numba.cuda.cudadrv.driver - INFO - add pending dealloc: cuMemFree_v2 3312 bytes
2020-08-31 17:22:14,173 - numba.cuda.cudadrv.driver - INFO - dealloc: cuMemFree_v2 512 bytes
2020-08-31 17:22:14,174 - numba.cuda.cudadrv.driver - INFO - dealloc: cuMemFree_v2 283136 bytes
2020-08-31 17:22:14,175 - numba.cuda.cudadrv.driver - INFO - dealloc: cuMemFree_v2 70656 bytes
2020-08-31 17:22:14,177 - numba.cuda.cudadrv.driver - INFO - dealloc: cuMemFree_v2 150960 bytes
2020-08-31 17:22:14,178 - numba.cuda.cudadrv.driver - INFO - dealloc: cuMemFree_v2 37920 bytes
2020-08-31 17:22:14,179 - numba.cuda.cudadrv.driver - INFO - dealloc: cuMemFree_v2 795056 bytes
2020-08-31 17:22:14,180 - numba.cuda.cudadrv.driver - INFO - dealloc: cuMemFree_v2 3792 bytes
2020-08-31 17:22:14,181 - numba.cuda.cudadrv.driver - INFO - dealloc: cuMemFree_v2 283136 bytes
2020-08-31 17:22:14,182 - numba.cuda.cudadrv.driver - INFO - dealloc: cuMemFree_v2 70656 bytes
2020-08-31 17:22:14,183 - numba.cuda.cudadrv.driver - INFO - dealloc: cuMemFree_v2 610512 bytes
2020-08-31 17:22:14,184 - numba.cuda.cudadrv.driver - INFO - dealloc: cuMemFree_v2 3312 bytes

These are info from logging. Numba follows the logging convention that names loggers by the module's qualifying name.  Therefore, you can customize individual logger by:
import logging
cuda_logger = logging.getLogger('numba.cuda.cudadrv.driver')
cuda_logger.setLevel(logging.ERROR)  # only show error

Or, you can customize logging for all of Numba by:
import logging
nb_logger = logging.getLogger('numba')
nb_logger.setLevel(logging.ERROR)  # only show error

Note, this issue is most likely caused by the use of logging.basicConfig which enables logging for every logger in the program. For applications that uses many packages/modules, it's best to use more fine-grained config methods.: https://docs.python.org/3/library/logging.config.html#logging.config.dictConfig

These are info from logging. Numba follows the logging convention that names loggers by the module's qualifying name. Therefore, you can customize individual logger by:
import logging
cuda_logger = logging.getLogger('numba.cuda.cudadrv.driver')
cuda_logger.setLevel(logging.ERROR)  # only show error

Or, you can customize logging for all of Numba by:
import logging
nb_logger = logging.getLogger('numba')
nb_logger.setLevel(logging.ERROR)  # only show error

Note, this issue is most likely caused by the use of logging.basicConfig which enables logging for every logger in the program. For applications that use many packages/modules, it's best to use more fine-grained config methods: https://docs.python.org/3/library/logging.config.html#logging.config.dictConfig

These are info from logging. Numba follows the logging convention that names loggers by the module's qualifying name. Therefore, you can customize individual logger by:
import logging
cuda_logger = logging.getLogger('numba.cuda.cudadrv.driver')
cuda_logger.setLevel(logging.ERROR)  # only show error

Or, you can customize logging for all of Numba by:
import logging
nb_logger = logging.getLogger('numba')
nb_logger.setLevel(logging.ERROR)  # only show error

Note, this issue is most likely caused by the use of logging.basicConfig which enables logging for every logger in the program. For applications that use many packages/modules, it's best to use more fine-grained config methods: https://docs.python.org/3/library/logging.config.html#logging.config.dictConfig

It works. Many thanks!


style for replacement rings

they're separate from insertion markers afaik

the background path receives filter="url(#blocklyReplacementGlowFilter)" but I don't think any new blocks are generated :/

I mean I'm not even using the mutationobserver anymore anyway


Fix BlurView blurAmount not work And Crash on iOS8

We found v2.0.0 crashed on iOS8, because [super  effectWithStyle:] called in BlurAmount cause ARC destruct fail. So I use category instead of subclass.
This is the stack: image of stack
self.visualEffectView.effect = self.blurEffect would't trigger [UIBlurEffect effectSettings], the blurAmount did't work. So I still use  self.visualEffectView = [[UIVisualEffectView alloc] initWithEffect:self.blurEffect]; just like v2.0.0

Hi, I also noticed that blurAmount was no longer working on iOS (10). It's just using the default blur amount.
@eliperkins - is this fixed in #174? If so, could you please rebase with the latest commits from master, and I'll point to your branch for now? (Because I've recently merged in some stuff to fix Android.)

Hi @haywoodfu and @eliperkins - please take a look at my PR: #183
I've fixed the blurAmount issue, but also fixed a lot of other things and I think I made the code a lot better.

I think #183 is the right direction to go in, preferring using associated objects over swizzling. I'd recommend getting #183 cleaned up and merged in to address these issues.


Convert this R code to RMark Down File

Hi AEMuelber,
I have download the code for "AmazonTreeMortality" and I am going to convert this code into RMark Down file which is very helpful for sharing code.
as i converted i will share the file, and you upload it in this repository.
thank you,
rao

hi ,
i have converted this code to "R mark down " file please see the zip file
AmazonTree.zip
01_Master3.pdf


Remove msinttypes dependency

Google Code has closed
msinttypes only two files, remove this dependency should not be difficult
CMakeLists.txt (361)
IF(MSVC)
	include(external/MSIntTypes)
	LIST(APPEND INCLUDES ${THIRD_PARTY_DIR}/MSIntTypes)

	include(external/MSDirent)
	LIST(APPEND INCLUDES ${MSDIRENT_INCLUDE_DIR})
ENDIF()

@TianShengLi check feature/windows branch for windows build.... currently there are couple of issues that needs fixing. but there this hole story of MSIntTypes is fixed....


ValueError in maisi_train_controlnet_tutorial.ipynb

INFO:notebook:Inference...
2024-09-23 06:17:47,215 - INFO - 'dst' model updated: 158 of 206 variables.

INFO:maisi.controlnet.infer:Number of GPUs: 2
INFO:maisi.controlnet.infer:World_size: 1
WARNING:py.warnings:unclosed file <_io.TextIOWrapper name='./temp_work_dir_controlnet_train_demo/environment_maisi_controlnet_train.json' mode='r' encoding='UTF-8'>

WARNING:py.warnings:unclosed file <_io.TextIOWrapper name='./temp_work_dir_controlnet_train_demo/config_maisi.json' mode='r' encoding='UTF-8'>

WARNING:py.warnings:unclosed file <_io.TextIOWrapper name='./temp_work_dir_controlnet_train_demo/config_maisi_controlnet_train.json' mode='r' encoding='UTF-8'>

INFO:maisi.controlnet.infer:trained autoencoder model is not loaded.
INFO:maisi.controlnet.infer:trained diffusion model is not loaded.
INFO:maisi.controlnet.infer:set scale_factor -> 1.0.
INFO:maisi.controlnet.infer:trained controlnet is not loaded.
Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/workspace/Code/tutorials/generation/maisi/scripts/infer_controlnet.py", line 207, in <module>
    main()
  File "/usr/local/lib/python3.10/dist-packages/torch/utils/_contextlib.py", line 116, in decorate_context
    return func(*args, **kwargs)
  File "/workspace/Code/tutorials/generation/maisi/scripts/infer_controlnet.py", line 159, in main
    check_input(None, None, None, output_size, out_spacing, None)
  File "/workspace/Code/tutorials/generation/maisi/scripts/sample.py", line 378, in check_input
    raise ValueError(
ValueError: The output_size[0] have to be chosen from [256, 384, 512], and output_size[2] have to be chosen from [128, 256, 384, 512, 640, 768], yet got (128, 128, 128).
E0923 06:17:50.348000<PHONE_NUMBER>87136 torch/distributed/elastic/multiprocessing/api.py:863] failed (exitcode: 1) local_rank: 0 (pid: 209184) of binary: [/usr/bin/python](https://vscode-remote+attached-002dcontainer-002b7b22636f6e7461696e65724e616d65223a222f74657374227d-0040ssh-002dremote-002b7b22686f73744e616d65223a2231302e31392e3138332e313930222c2275736572223a2279756e6c6975227d.vscode-resource.vscode-cdn.net/usr/bin/python)
Traceback (most recent call last):
  File "/usr/local/bin/torchrun", line 33, in <module>
    sys.exit(load_entry_point('torch', 'console_scripts', 'torchrun')())
  File "/usr/local/lib/python3.10/dist-packages/torch/distributed/elastic/multiprocessing/errors/__init__.py", line 355, in wrapper
    return f(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/torch/distributed/run.py", line 919, in main
    run(args)
  File "/usr/local/lib/python3.10/dist-packages/torch/distributed/run.py", line 910, in run
    elastic_launch(
  File "/usr/local/lib/python3.10/dist-packages/torch/distributed/launcher/api.py", line 138, in __call__
    return launch_agent(self._config, self._entrypoint, list(args))
  File "/usr/local/lib/python3.10/dist-packages/torch/distributed/launcher/api.py", line 269, in launch_agent
    raise ChildFailedError(
torch.distributed.elastic.multiprocessing.errors.ChildFailedError:
============================================================
scripts.infer_controlnet FAILED
------------------------------------------------------------
Failures:
  <NO_OTHER_FAILURES>
------------------------------------------------------------
Root Cause (first observed failure):
[0]:
  time      : 2024-09-23_06:17:50
  host      : yunliu-MS-7D31
  rank      : 0 (local_rank: 0)
  exitcode  : 1 (pid: 209184)
  error_file: <N/A>
  traceback : To enable traceback see: https://pytorch.org/docs/stable/elastic/errors.html
============================================================

Hi @guopengf, could you please take a look at this issue? Thanks.

INFO:creating training data:Using device cuda:0
WARNING:py.warnings:You are using `torch.load` with `weights_only=False` (the current default value), which uses the default pickle module implicitly. It is possible to construct malicious pickle data which will execute arbitrary code during unpickling (See https://github.com/pytorch/pytorch/blob/main/SECURITY.md#untrusted-models for more details). In a future release, the default value for `weights_only` will be flipped to `True`. This limits the functions that could be executed during unpickling. Arbitrary objects will no longer be allowed to be loaded via this mode unless they are explicitly allowlisted by the user via `torch.serialization.add_safe_globals`. We recommend you start setting `weights_only=True` for any use case where you don't have full control of the loaded file. Please open an issue on GitHub for any issues related to this experimental feature.

**ERROR:creating training data:The trained_autoencoder_path does not exist!**
INFO:creating training data:filenames_raw: ['tr_image_001.nii.gz', 'tr_image_002.nii.gz']
[rank0]:[W923 05:56:48.650222570 ProcessGroupNCCL.cpp:1207] Warning: WARNING: process group has NOT been destroyed before we destruct ProcessGroupNCCL. On normal program exit, the application should call destroy_process_group to ensure that any pending NCCL operations have finished in this process. In rare cases this process can exit before this point and block the progress of another member of the process group. This constraint has always been present,  but this warning has only been added since PyTorch 2.4 (function operator())

In maisi_diff_unet_training_tutorial.ipynb

Hi @KumoLiu, I think this error occurred after this PR https://github.com/Project-MONAI/tutorials/pull/1825. We added an input check function for the controlnet inference script. We can change the toy data in the controlnet tutorial to [256, 256, 128] with spacing [1.5, 1.5, 1.5] to pass this input check.

INFO:creating training data:Using device cuda:0
WARNING:py.warnings:You are using `torch.load` with `weights_only=False` (the current default value), which uses the default pickle module implicitly. It is possible to construct malicious pickle data which will execute arbitrary code during unpickling (See https://github.com/pytorch/pytorch/blob/main/SECURITY.md#untrusted-models for more details). In a future release, the default value for `weights_only` will be flipped to `True`. This limits the functions that could be executed during unpickling. Arbitrary objects will no longer be allowed to be loaded via this mode unless they are explicitly allowlisted by the user via `torch.serialization.add_safe_globals`. We recommend you start setting `weights_only=True` for any use case where you don't have full control of the loaded file. Please open an issue on GitHub for any issues related to this experimental feature.

**ERROR:creating training data:The trained_autoencoder_path does not exist!**
INFO:creating training data:filenames_raw: ['tr_image_001.nii.gz', 'tr_image_002.nii.gz']
[rank0]:[W923 05:56:48.650222570 ProcessGroupNCCL.cpp:1207] Warning: WARNING: process group has NOT been destroyed before we destruct ProcessGroupNCCL. On normal program exit, the application should call destroy_process_group to ensure that any pending NCCL operations have finished in this process. In rare cases this process can exit before this point and block the progress of another member of the process group. This constraint has always been present,  but this warning has only been added since PyTorch 2.4 (function operator())

WARNING:py.warnings:unclosed file <_io.TextIOWrapper name='./temp_work_dir/./embeddings/tr_image_001_emb.nii.gz.json' mode='r' encoding='UTF-8'>

WARNING:py.warnings:unclosed file <_io.TextIOWrapper name='./temp_work_dir/./embeddings/tr_image_002_emb.nii.gz.json' mode='r' encoding='UTF-8'>

[rank0]:[W923 10:10:11.778510110 ProcessGroupNCCL.cpp:1207] Warning: WARNING: process group has NOT been destroyed before we destruct ProcessGroupNCCL. On normal program exit, the application should call destroy_process_group to ensure that any pending NCCL operations have finished in this process. In rare cases this process can exit before this point and block the progress of another member of the process group. This constraint has always been present,  but this warning has only been added since PyTorch 2.4 (function operator())

In maisi_diff_unet_training_tutorial.ipynb

@dongyang0122 Would you like to look into this error for diffusion unet?


240 add netcdf import for user provided netcdf

Adds support to load timeseries, location attributes, and location crosswalks from netcdf format
Also pins the netcdf4 library to v1.6.5 to avoid HDF5 errors when reading a netcdf file with xarray and importing the kerchunk library

@mgdenno This should be good to go, I think we talked about this work so I'm not sure much review is needed but any comments are welcome

@samlamont sounds good.  I took a quick look though it.  I don't have any comments.


RemoteMixtureOfExperts and RemoteSwitchMixtureOfExperts cannot be used on GPU

If you try to use these classes in a differentiable manner (i.e. backward passes during training), and on GPU - they will both fail with errors like the following:
  File "/home/crow/repos/praxis/venv/lib/python3.12/site-packages/torch/autograd/graph.py", line 768, in _engine_run_backward
    return Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/crow/repos/praxis/venv/lib/python3.12/site-packages/torch/autograd/function.py", line 306, in apply
    return user_fn(self, *args)
           ^^^^^^^^^^^^^^^^^^^^
  File "/home/crow/repos/praxis/venv/lib/python3.12/site-packages/torch/autograd/function.py", line 599, in wrapper
    outputs = fn(ctx, *args)
              ^^^^^^^^^^^^^^
  File "/home/crow/repos/praxis/venv/lib/python3.12/site-packages/hivemind/moe/client/moe.py", line 312, in backward
    inputs_per_expert = zip(*(tensor[alive_ii].split(1, dim=0) for tensor in flat_inputs_cpu))
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/crow/repos/praxis/venv/lib/python3.12/site-packages/hivemind/moe/client/moe.py", line 312, in <genexpr>
    inputs_per_expert = zip(*(tensor[alive_ii].split(1, dim=0) for tensor in flat_inputs_cpu))
                              ~~~~~~^^^^^^^^^^
RuntimeError: indices should be either on cpu or on the same device as the indexed tensor (cpu)

If you trace the code to flat_inputs_cpu within Hivemind, you will clearly see that there are many places where CPU-only operations are being used. This explains why these layer types will work on CPU, but fail on GPU.
I don't think there's a great solution to this, short of refactoring the Hivemind code, or replacing these layers entirely.

I submit a PR to Hivemind, though I may not use this code, in the long run. These CPU operations are very slow, and using GPU does not apparently speed them up in any way. This may not matter, though - since remote peers are going to be slow, no matter what we do.


offset-rotate parse, merge and interpolation tests

Please reply to review comments that you've addressed and send out when ready for the next review. It makes it easier to keep track of the changes.

Thank you for assertNoInterpolation.


Request: Make an option to remove the assumption of return "names" in SumnemmyLua

I find myself never, ever needing to assign a "name" on a return value, like I do for parameters - it's just "---@param [type] [description]" for me. An option in the settings to disable the name, if possible, would be much appreciated.

The param must be ---@param name type.
You can use ---@return type # description (or @ or --)

Great, didn't know you could do a comment with just #. Thanks!


Structured your code

I splitted your code into multiple module files to get a better structure. There is still a lot of work to do and a lot of refactoring necessary but it is a starting point :)
Kept the old GUI2.py as old_GUI2.py so that there is no risk for you to merge my changes into your project.

weapons, armors and potions now have their own class files


fix: use av_guess_sample_aspect_ratio when reporting stream SAR

When user requests stream's SAR, use the upstream's heuristic function (av_guess_sample_aspect_ratio()) to report it.
Resolves: https://github.com/PyAV-Org/PyAV/issues/1267

I'd really prefer we don't merge any new code without the corresponding tests.


holoviews.operation.ElementOperation.call overwrites labels and groups for HoloMaps

When passing a HoloMap to an ElementOperation, the group and label are replaced with the one of the first contained Element, e.g. you get "Curve" as your group, if the Curves in the HoloMap has not been assigned a custom group.

A PR to fix this issue would be very much appreciated, this is clearly bad behavior and I see it all the time.

The fix has been merged.


PLEA FOR ACCESS

Dear @gauravgorade
I love your work. This is awesome. You must have done lots of work. Can you please mail me the access details, to<EMAIL_ADDRESS>I want to explore this great application.
Love to hear from you soon.
Love and light.

Additionally, you can comment out the code responsible for converting the password to a hash and checking the access based on the password hash. Instead, you can directly input the password from the backend and simply check for the username and original password string.


python310Packages.pylitterbot: 2022.12.0 -> 2023.1.0

Diff: https://github.com/natekspencer/pylitterbot/compare/refs/tags/2022.12.0...2023.1.0
Changelog: https://github.com/natekspencer/pylitterbot/releases/tag/2023.1.0
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 209854 run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.litterrobot
python310Packages.pylitterbot
python39Packages.pylitterbot


feat: sort the order of bulk-renamed files

Resolve #1766 . Sort bulk-renamed files in a similar way to topological sort.

Thanks for the PR! Could you please add tests to the sort() function?

Sure! I add a very simple unittest for sort.

Thanks for the change! I added more tests and found that the last two tests are failing. Could you please fix them?
Specifically, the second test is producing unstable sorting results. It could be:
&[("3", "4"), ("2", "3"), ("1", "3")]

or
&[("3", "4"), ("1", "3"), ("2", "3")]

This might be related to the HashMap being used, which has an unpredictable order. Switching it to a BTreeMap or IndexMap might solve the issue.
As for the last test, it hits todo!("Consider cycle"). Normally, a->b and b->a shouldn't happen, but since the data comes from user input, it's an uncontrollable facto, so we need to handle this case to avoid crashing the program here.

I also had some concerns while writing the code, so thank you for creating tests for that. I have two questions.

input validation

I think the input of second test case is invalid. 1 -> 3 and 2 -> 3 cannot do in one bulk rename. In the sort function, is it enough to just return the order without validation? I think it will be catched at maybe_exist and not be renamed, so is it ok?

how to handle error

I think it is sub-problem of first question. As you mentioned, we can handle the error when cyclic rename detected. But I'm not sure about how to handle this invalidate input and what is proper behavior of sort function.

So, could I ask you, the author of this project, to clearly define the roles and responsibilities of the sort function? If it doesn't really matter, I'll work on it at my own discretion.

I haven't fully understood the implementation of sort, but I think we should try to keep the user's input order and only prioritize certain items on top of that, like:
a -> b
b -> b
b -> c

In this case, we have a -> b and b -> b appearing twice, which means the data itself is invalid.
However, we don't need to take responsibility for invalid data; that's the user's responsibility, and the program should only handle valid data, as we already know that b -> c can be prioritized, so the sorted result should be:
b -> c
a -> b
b -> b

Invalid data remain unchanged and are passed to the upper layer for handling, which in this case is maybe_exist.

So do you mean that we will rename a valid subset of the user's input even if the user's full input set is invalid (duplicate target name or cyclic rename)? For that purpose, the role of sort function is just prioritize some rename that could be executed without any concerns, right? Please let me know if I have misunderstood anything.

Yes, the sorting function should only elevate those valid renaming items that have no side effects and can be executed early to the front of the execution queue, assisting the user as much as possible (rather than taking over the user's work and responsibilities) in doing all valid operations, while leaving invalid operations for higher-level resolution — here, that means throwing errors to the user for manual intervention.

One thing I would like to suggest is that the sort function returns a separate rename set containing a cycle. And in the above function, if there is a cycle, how about showing the list to the user on stderr to indicate that the cycle exists and just return (or user interaction to do only valid things or not?).
let (valid, cycle) = Self::sort(old, new);
if !cycle.is_empty() {
    writeln!(stderr, "cyclic rename detected");
    for (old, new) in cycle {
        wrintln!(stderr, "{old} -> {new}");
    }
    return Ok(()); // or user interaction to do or not
}

Sorry - no. I don't really see the practical significance of "cycle detection" for users, as it's more of an internal implementation detail. The program just needs to ensure that it safely handles the valid data provided by the user, while leaving the invalid data for the user to handle manually. Introducing this detection seems to discard the entire bulk rename operation, rather than just the invalid parts

I was mistaken. I thought that if there is a cycle, the rename will be performed and all the files will be messed up, but it will not be performed because it is caught in maybe_exist.
I make result consistent and just return cyclic rename entries instead of panic. Thanks for your comment.

Thanks! I tested the latest changes, and it seems the ordering issue still exists. For example, this test will fail:
#[rustfmt::skip]
cmp(
    &[("b", "b_"), ("a", "a_"), ("c", "c_")],
    &[("b", "b_"), ("a", "a_"), ("c", "c_")],
);

I miss reserving user input order. I fix it!

Hey @yw1ee!
I did some refactoring to simplify the code, could you please do a final review? If everything looks good I'll merge it 🚀

Merging now.
Thank you so much for contributing this awesome feature @yw1ee, really appreciate it! 🎉


Using typeof this.#privateField in declaration emit

class StringHolder {
    #data = "hello";

    public getData(): typeof this.#data {
        return this.#data;
    }
}

error TS4055: Return type of public method from exported class has or is using private name 'this'.

8     public getData(): typeof this.#data {
                               ~~~~

Seems strange that we even have this feature if we can't use this in declaration emit.

This is well-precedented in other contexts, including keyword-private
function fn() {
    const q: unique symbol = Symbol();

    return {
        foo(m: typeof q) { }
    };
}

class StringHolder {
    private data = "hello";

    public getData(): typeof this.data {
        return this.data;
    }
}

Return type of public method from exported class has or is using private name 'this'.

Why is it complaining about this rather than the actual thing that’s private?

Would it make sense to enhance the declaration emit to output the resolved type instead of the typeof?
(In this case string instead of `typeof this.#data)
That avoids the need to emit the list of private names.

Hmm, normally we wouldn't do that since this.foo types are supposed to be deferred so they can be specialized in subclasses, but that obviously doesn't apply here. Seems good. @DanielRosenwasser any concerns?

Honestly, before we try to tackle this "problem", I'd like to hear whether anyone is actually running into this.

I'm seeing the same error even for simple cases like this:
class X {
  setValue(): typeof this {
    ...
   return this;
  }
}

It's convenient to return typeof this in this case because it will also have the right type for the subclasses of X.


PromQL: add "involved samples" for query stats

Proposal
Prometheus will record timings stats for every query, such as evalTotalTime , resultSortTime now. It need more stats to evaluate query performance.
What is we need?
Expose stats "involved sample" as a query stats, i.e. currentSamples https://github.com/prometheus/prometheus/blob/0eb34299dad0ab997bc9d9363a06d6ea6eb45dad/promql/engine.go#L785
Why is it important?
The timings stats is an unstable metric, which may be affected by hardware, query queue, etc., especially it's a random number. Sometimes we wanna evaluate whether query was bad or good, using timings stats is not a good idea.
However, currentSamples is stable and reliable which precisely records how many samples a query travelled. It also reflect how much computation resource consumed, such as memory or cpu.

I am taking this up. Thank you.

I think we want: the number of series, the number of samples and the peak number of samples. Thanks.

Expectations:
$ curl <IP_ADDRESS>:9090/api/v1/query?query=up\&stats=true |python -m json.tool
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   354  100   354    0     0   345k      0 --:--:-- --:--:-- --:--:--  345k
{
    "data": {
        "result": [
            {
                "metric": {
                    "__name__": "up",
                    "instance": "<IP_ADDRESS>:9090",
                    "job": "prometheus"
                },
                "value": [
<PHONE_NUMBER>.93,
                    "1"
                ]
            }
        ],
        "resultType": "vector",
        "stats": {
            "samples": {
                "peakSamples": 9000,
                "totalSamples": 12000,
                "totalEvaluators": 2,
              },
            "timings": {
                "evalTotalTime": 5.4748e-05,
                "execQueueTime": 6.421e-06,
                "execTotalTime": 6.4869e-05,
                "innerEvalTime": 1.4978e-05,
                "queryPreparationTime": 3.7253e-05,
                "resultSortTime": 0
            }
        }
    },
    "status": "success"

@roidelapluie not sure about the evaluatorsCount, but rest all looks :+1:

@brian-brazil any views?

At bug-scrub: we believe this was addressed by #10369.


releases: Move v1.14 to "Currently supported"

Release 1.14 has been officially tagged.
Tags v1.14.0 and v1.14.1 (hotfix) were released on Feb 2, 2024. The releases document is updated to reflect this, and the provisional End of Life date is marked as four months in the future, falling on the nearest working day to allow for a last minute release without risk of breaking someone's weekend.
Closes #1416.

Edits by maintainers are allowed. If the EOL needs to be adjusted, feel free to modify the file directly in the fork.

Hey @sigv - really appreciate you raising this! I didn't see this before I merged @wallrj 's PR in #1414 which does this - so I'll close this.
Thanks very much for reaching out!


Feature/billing

Please review my pull request

Please review my pull request

Please review my pull request

Please review my pull request


Overwrite separator-border width?

Hi, im trying to modify the separator-border width but for some reasons, i coulnd't make it work well, is it possible via the style provided in the document?

:root {
  --sash-size: 10px;
  --sash-hover-size: 50px;
}

[class*="sash-module_sash_"]::before {
  background: gray !important;
}

This one works for me.


Renaming resource used in currently open scene will revert your scene

Godot version:
Mono v3.0.2
OS/device including version:
Windows 8.1 Enterprise
Issue description:
When you are editing a scene and have not saved the changes yet you go to the file system and rename a resource (script, texture, anything referenced in the scene).
After rename the scene will revert to the scene on the disk and you will lose any changes made to the scene.
The expected behavior would be to warn the user about unsaved changes and if he wants to save before renaming the resource.
Example of the bug:
https://youtu.be/AMlF6gbAUKI
Steps to reproduce:

Have a texture with a .png loaded from file system in a saved scene
Modify the scene (Add new node, do not save the scene)
In the file system panel rename the texture that you have in your scene
Observe that your modification from step 2 has disappeared.

Example video:
https://youtu.be/AMlF6gbAUKI

@groud Can you check this out? (reassign to me if you can't) There should obviously be a warning here (and I think I added it at some point, but does not seem to work), despite that I don't know what actually causes this behavior.

Interestingly, this doesn't happen when the resource reference isn't saved yet, i.e. when you add a dependency, but don't save the scene. When you rename the resource, the reference gets correctly updated and nothing is reverted (although resource name in inspector isn't changed until you reselect node). When you save the scene, so the dependency is saved inside .tscn, it will get reverted on rename as described in OP.

Fixed in 3.2


font color of echart theme not readable in dark themes

Legend text not readable in dark mode
Also check font color of axis in dark mode. seems to dark
Originally posted by @ChristianButter in https://github.com/siemens/ix/issues/16#issuecomment-1270052690

Hi @danielleroux, could you please double check if the eChart Theme Builder JSONs are applied correctly? The colors in the screeshot/documentation seem to be completely different from the colors in the theme files – in light and dark, Brand and Classic.


Create Merge_Sort.cpp

Dear Contributor,
Kindly Start ⭐ this repo. After that only your PRs will be accepted. Don't avoid it.
Thank You.


It's is not possible to disable animation programatically.

[REQUIRED] Use case description
I would like to be able to disable PlayerView's animation programmatically. It's possible to do this via XML by adding app:animation_enabled="false" but there is no method exposed from the PlayerView itself.
There is a method setAnimationEnabled inside the PlayerControlView but it seems to be unused.
Proposed solution
Expose the PlayerControlView instance so it's accessible in a subclass of the PlayerView.
Alternatives considered
Add a new method setAnimationEnabled that forwards the call to the PlayerControlView's setAnimationEnabled.

The commit above adds PlayerView.setControllerAnimationEnabled(boolean) for this.

Thank you! 🙂


Deleted source files still published to npm

src/tracing.js was deleted from this package, but the prepublish script doesn't seem to guarantee that dist is empty, so it actually still exists in the published 0.6.0 even though the reference to it in index.js is gone.

Thanks, I'll try to fix this in #86.


Drop radius changes

The drop radius gets bigger as a block is pulled away. I'm not sure if this is by design, but I think drop radius should be constant. This happens for all types of blocks.

We did it intentionally, here: https://github.com/LLK/scratch-blocks/commit/fa992e9f863df2c9b25bcfc140e6105cf41e26b0
To fix this bug: https://github.com/LLK/scratch-blocks/issues/290 (video in there).
There might be a way to fix that some other way. I think we need to consider reducing DRAG_RADIUS in general, too... @rachel-fenichel

There might be a way to fix that some other way. I think we need to consider reducing DRAG_RADIUS in general, too... @rachel-fenichel

Yes, I agree with this. I'm fine with the drop radius increasing as long as it's not by that much. Might make it very annoying for greatly nested operators.

Also some way to disable snapping (eg shift-drag) would be helpful with such a large drop radius

@MegaApuTurkUltra We're addressing this in #635 - give it another try sometime, please!


Login screen - Spotify Auth

Task description
Create a simple login screen only with a button to allow Spotify oAuth that gets the permission to user-read-recently-played token and redirects to the user's profile initial page.
Things to do:

[ ]  Create the screen with the button to Spotify auth;
[ ]  Make the request to retrieve the token and store temporarily;

Good! Can i make a pr?

Go there!


Package Fix: makeblockteam.mBlock, fix hash

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Resolves: https://github.com/microsoft/winget-pkgs/issues/153008
Microsoft Reviewers: Open in CodeFlow

I think the exe might be a dynamic hash, so I decided to change the installer to an msi

I have no idea what is wrong with this file:

Automatic Validation ended with:

2024-05-10 06:16:57.628 [CORE] DeliveryOptimization error: 0x80072f76, extended error: 0x00000000
2024-05-10 06:16:57.683 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\DODownloader.cpp(289)\WindowsPackageManager.dll!00007FFACE52A465: (caller: 00007FFACE52AD01) Exception(1) tid(1e9c) 80072F76

(Automated response - build 880.)

Manual Validation ended with:

2024-05-10 08:42:52.448 [CORE] DeliveryOptimization error: 0x80072f76, extended error: 0x00000000
2024-05-10 08:42:52.449 [FAIL] C:__w\1\s\external\pkg\src\AppInstallerCommonCore\DODownloader.cpp(289)\WindowsPackageManager.dll!00007FFE45B4F180: (caller: 00007FFE45B4FACC) Exception(1) tid(a60) 80072F76
2024-05-10 08:42:54.355 [CLI ] Package hash verification failed. SHA256 in manifest [9d5bf8cddc60080fd570bb79bfc5867daf56a40f45ee9a2dace714ebc555c5c5] does not match download [a83bba255d2d6e424f058c38a204f273a4f40bcac38c8283f551a7056b605640]

(Automated response - build 880.)

I think the SHA256 is dynamic, it keeps changing

If the SHA256 keeps changing to a new value, then it might be receiving continuous updates, possibly at the end of a CI/CD pipeline.
If the SHA256 keeps changing and reverting, then it might be getting an error page sometimes.

Either way complicates installing through a package manager.

Don't know what can be done about this


--color option breaks the output in Git Bash on Windows

$ git diff data/slugify.js | git-split-diffs | less

$ git diff data/slugify.js | git-split-diffs --color | less

(Truncated output; there are a few more lines.)

$ git --version
git version 2.31.1.windows.1

$ node -v
v14.13.0

$ npm ls -g --depth=0
C:\Users\mts\AppData\Roaming\npm
├──<EMAIL_ADDRESS>...

Any idea what might be wrong? I'm using Windows Terminal in the screenshots, but the --color option doesn't work either when used directly in Git Bash.
The output seems to be colorized when using CMD, even without the --color option, but I don't want to use CMD. (Btw the output looks gorgeous! 🤩)
From todo.md I can see that there's a completed item called "Test on windows." I wonder what kind of setup was used for the testing? Was CMD or Git Bash used?

From todo.md I can see that there's a completed item called "Test on windows." I wonder what kind of setup was used for the testing? Was CMD or Git Bash used?

I tested on a fresh Windows 10 VM  in VirtualBox, both with CMD and Git Bash:
$ git --version
git version 2.31.1.windows.1

$ node -v
v14.16.1

From the screenshot, it seems that the ANSI color codes are not being interpreted. I'd guess that this has to do with the TERM environment variable. For me it is set to xterm in git bash and unset in CMD and colors work in both. Can you check that?

Thank you!
$ echo $TERM
xterm-256color

I just noticed that if I don't pipe the output to less, everything works perfectly. So the problem is actually in the less command.
From superuser.com/q/117841/111704 I learned that I need to use the -R flag with less. -r seems to work as well.
So now it's working, and it's beautiful!

Hmm, now I noticed that the less commands in the "Install globally" and "Install locally" sections in the readme use the flags -RFX. Maybe the command in the "Use manually" section should also use at least the -R flag?

Oh yes, thanks. I'll update it