Detect starting ratings with an initial pass through data

Add option --detect-starting-ratings, which does an initial pass through games and sets starting ratings in many cases.
Everything is just in the one-game-at-a-time data, and it's not at all configurable, but it at least allow some hardcoded experiments.

Initial results look not ideal, since all ratings are depressed. Need to at least fine-tune the auto-detection.

Numbers look kind of okay the full 30M data set after 23248a037fba9bbb54ffb86e42e6c72ff4aadf5c (I was tuning for the first 1-2M games, then did a long run with all 30M). There are some anomalies... but I'm not sure how to interpret the results anyway...
I'm not sure this auto-detection experiment is going to tell us what we want to know. These two scenarios seem quite different:

auto-detection (or self-selection of level) starting from now (after 30M games have been played)
auto-detection starting from the beginning of time

For (1), you have the advantage that the existing ratings strongly influence new ratings, so new players have very little effect on the rating pool.
For (2), you can/should recalibrate ratings and the constants in rank_to_rating.
This experiment is (2), but without recalibrating rank_to_rating. I'm not sure if/how we can use this to judge the correct levels for turning on self-selected starting ranks.
(There could be other flaws in the experiment as well.)

I'm not sure this auto-detection experiment is going to tell us what we want to know.

@anoek, let me know if you have specific ideas about the data you're hoping for here, and I can play around more. I'm hesitant to clean it up and make it ready for review until it's more clear what we're trying to learn.

If we want some confirmation that the choices for starting ranks are reasonable for ratings v5, one idea is to only set auto-detected starting ranks for players whose first game is in the last 1-2 years, and see what happens then.
If we want to backfill auto-detected starting ranks for ratings v6, then maybe we can/should be recalibrating rating_to_rank somehow in conjunction.

@dexonsmith  @anoek  I guess the other "thing we need to learn" is "are the chosen self-select starting ranks going to be OK"
Is that something this is shooting for ?

@dexonsmith @anoek I guess the other "thing we need to learn" is "are the chosen self-select starting ranks going to be OK"
Is that something this is shooting for ?

Right; I think @anoek is hoping to learn that, but I don't think this experiment as-is will tell us that information... right now, it's better set up to answer what would happen if we back-filled starting ratings as part of v6 (but I think that requires recalibration of rank_to_rating to get right).
I'm not quite sure how to nudge it toward answering "the select-select ranks in the context of v5" question. One idea is to auto-detect only for new-players from the last 1-2 years of data.


[vcpkg] run both cxx and c compiler id.

fixes: https://discord.com/channels/400588936151433218/687365466422902841/762996065850621953
  if ( DEFINED CMAKE_C_COMPILER_ID AND DEFINED CMAKE_CXX_COMPILER_ID
       AND NOT "x${CMAKE_C_COMPILER_ID}" STREQUAL "x${CMAKE_CXX_COMPILER_ID}")
    message(FATAL_ERROR "The current configuration mixes Clang and MSVC or "
            "some other CL compatible compiler tool. This is not supported. "
            "Use either clang or MSVC as both C and C++ compilers.")
  endif()

CXX run was added in:
https://github.com/microsoft/vcpkg/commit/741c8cbecde77f771f8c05f00f59a436a2b4213b

Thanks!


Error "SslCaCert"  all https site

Error "SslCaCert" all https site
sample site
https://www.google.com/
https://www.whatismyreferer.com/
https://www.facebook.com/

Do this before .Perform() function*(assuming that object name is easy)* -- easy.SslVerifyPeer = false;

Thank you so much BroZeus


[common] Add Netty based P2P lib for blob transfer

Summary

This PR introduces the Netty-based client and server to transfer files among peers.

It's still a bit rough and needs some improvements (bunch of TODOs left) to provide better monitoring and tune the performance.
This PR uses NIO implementation as much as possible to leverage zero-copy. We need to do some performance testing to find out how's that compared to regular I/O implementation.
One side note is zero copy seems require non-SSL so it's likely we don't get every benefit from it if using in production where SSL is always required, but the chance is we can still get something out of it 🤞
Moved some codes from blob to blobtransfer @sebas-inf

The basic workflow of the client and server is described below:

Client connects to the server and sends GET request to server to request blobs.
Server validates the request.
If request is valid, server will be sending files to client in chunks (default is 8192, and we can tune it if necessary).
For the last chunk, server sends a DefaultLastHttpContent and EMPTY_LAST_CONTENT to indicate the end of a file transfer so when client receives DefaultLastHttpContent, it would know it's the end of file and would reset state then prepare for the next file. (One thing unclear to me is how's EMPTY_LAST_CONTENT is handled by some internal Netty stuff because it's not actually passed to P2PFileTransferClientHandler)
After every file has been transferred, the server sends a final response with a special header telling the client the transfer is done
Client completes the future and close the connection

How was this PR tested?

CI
Does this PR introduce any user-facing changes?

[x] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

Overall the functionality part looks good to me! But since it is lib PR so there seems to be no integration test. Maybe in the follow up integration test we can use this to do a real e2e transfer from one DVC to the other.

yes, that's the goal for this Q.


Receiving 502 Bad Gateway Error when trying to access Map Roulette website

When trying to access the Map Roulette website, I receive the following error: "502 Bad Gateway, nginx/1.14.0 (Ubuntu)"
I am unable to access any challenges.

We are investigating this issue currently. Will update as soon as it is resolved. Thanks for reporting.

The site is back up, but we'll continue to monitor it closely. Thanks again for the report @lmscheurer .


LineUp v3.2.1 is breaking CircleCI builds

Release number or git hash: v3.2.1

Steps to reproduce

See build output of https://circleci.com/gh/datavisyn/tdp_uploaded_data/111 (or any other plugin that is build on top of tdp_core, which uses LineUp)

Observed behavior
    ERROR in tdp_uploaded_data.min.js from UglifyJs
    Unexpected token: name (n) [tdp_uploaded_data.min.js:2324,26275]

npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR<EMAIL_ADDRESS>build:web: `webpack --env prod`
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build:web script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!  Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/circleci/.npm/_logs/2019-09-17T09_09_44_785Z-debug.log
Exited with code 2

Expected behavior

The build should succeed

The error is pointing to this line:
Object.defineProperty(t.prototype,"mean",{get:function(){return this.map(function(t,e){let n=0,r=0;if(void 0===e)for(let e of t)null!=e&&(e=+e)>=e&&(++n,r+=e);else{let i=-1;for(let o of t)null!=(o=e(o,++i,t))&&(o=+o)>=o&&(++n,r+=o)}if(n)return r/n}(this.values))},enumerable:!0,configurable:!0}),

More specifically it is breaking at the for ... of loop. The problem: Our build environment uses Webpack 2 with UglifyJS, which does not support ES6 syntax, like the for ... of loop.
Mapping the minimized JS to TypeScript source code points to the following function:
https://github.com/lineupjs/lineupjs/blob/f98fb9bec7be8813ae6e89e5a8df4497256de5b1/src/internal/math.ts#L131-L133
The mean() function is an import of d3-array:
https://github.com/lineupjs/lineupjs/blob/f98fb9bec7be8813ae6e89e5a8df4497256de5b1/src/internal/math.ts#L1
The d3-array dependency was updated with LineUp v3.2.0 from ^1.2.1 to  ^2.3.1 (https://github.com/lineupjs/lineupjs/commit/225ab3a27b6e8c0b816e5bcf15bf53a1bf51648d).
Comparing the changes of the two D3 versions shows the usage of the for ... of loop.
Looking more closely in the d3-array release note of v2.0.0 it is stated:

Note: this release adopts ES2015 language features such as for-of and drops support for older browsers, including IE. If you need to support pre-ES2015 environments, you should stick with d3-array 1.x or use a transpiler.

Reverting to d3-array v1.2.1 should fix this problem.
The same warning goes for d3-scale v3.0.0. Hence, we should also stick to ^v2.1.0 until we updated our build environment to Webpack 4, which supports ES6.


Added examples of doing defaults inline

When using things like SQL Server Data Tools, it's helpful to be able to create named defaults inline.

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

@nycdotnet : Thanks for your contribution! The author, @stevestein, has been notified to review your proposed change.

@nycdotnet Hello Steve, We like your two Transact-SQL code examples that you want to have added to our documentation article. We are having them added. Thanks for submitting them.
However,.....
The precise markdown syntax that we need for adding the examples is not quite present in this PR 1660.
Further, when I looked at the article today, I saw several unrelated pieces of the article that warrant improvement. So I decided to create a PR (9134) in our private repo. PR 9134 adds your T-SQL examples with the triple-backtick syntax that we need, and makes numerous other smaller improvement.
By this coming Friday or Monday, you should be able to view the raw markdown for this article (in repo 'sql-docs'), and see an HTML comment that mentions the code examples were provided by 'nycdotnet'.
Now I am permanently Closing this public PR 1660, in favor of private PR 9134.
Thanks a lot.
CC: @ktoliver

Closing now.

ty


response validation

This is one of 3 PRs that for setting up response validation:
https://github.com/salesforce/refocus-sample-generator-template-utils/pull/26
https://github.com/salesforce/refocus-collector/pull/141
https://github.com/salesforce/refocus-collector-eval/pull/15 (this one)
The tests won't pass in sgt-utils and refocus-collector until the eval PR is merged.
Currently this is using JSON Schema because Joi does not support serialization. Now that everything is in place, it should be relatively straightforward to swap out JSON Schema for something else if we can make it work. I have created a separate story to look into other more user-friendly options.

Coverage increased (+0.05%) to 99.017% when pulling 42f837381b5d651029d2b40640661d67211e9687 on response-validation into 2101a9181e2f159d082398f1578f133505e6b51b on master.

published.


Bug: Icons are no longer shown on the tab control

Requirements

Display the icon of current directory on the tab control

Are you still looking for someone to work on this problem? I'm a student looking to contribute to an open source project and would love to try to figure it out if that's okay?

Yes, still an issue, would you like to work on it?

Yes, I would like to. please let me know if there is any additional information for this bug.

This issue was intruduced when we switched to WinAppSdk so it might be worth looking at those pull requests.

Just sayin' here, it still is an issue.
Just to let you know... :-)

@sthivaios this issue is high on our priority list as it's a regression since porting to WinAppSDK.
For what it's worth, your comments on issues these past few days have been extremely helpful. It's hard for us to check every issue and community participation makes it a lot easier and we really appreciate it.

@sthivaios this issue is high on our priority list as it's a regression since porting to WinAppSDK.
For what it's worth, your comments on issues these past few days have been extremely helpful. It's hard for us to check every issue and community participation makes it a lot easier and we really appreciate it.

Thank you! I just have a lot of free time now, because we don't have a lot of homework at school, so I decided to help in this community.

Interesting, this was working fine for us on Microsoft.UI.Xaml 2.7.1.

Offered a suggestion of fix if it's high on our priority list. It's more for a temporary fix, and if accepted, I'll open another issue to track that we need to get back to the previous display.

When I looked through it, I attempted switching to an iconbox instead and
it caused the application to crash. I was thinking if we could copy the
formatting used for the sidebar it would work but that wasn't running
properly.
On Wed, Oct 12, 2022 at 09:39 QuaintMako @.***> wrote:

Offered a suggestion of fix if it's high on our priority list. It's more
for a temporary fix, and if accepted, I'll open another issue to track that
we need to get back to the previous display.
—
Reply to this email directly, view it on GitHub
https://github.com/files-community/Files/issues/9929#issuecomment-1276190780,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOJTR5AQ3IAPIQM4EYSEVRLWC25RXANCNFSM6AAAAAAQF7GLAI
.
You are receiving this because you commented.Message ID:
@.***>

When I looked through it, I attempted switching to an iconbox instead and it caused the application to crash. I was thinking if we could copy the formatting used for the sidebar it would work but that wasn't running properly.

Tried to look into this direction as well, but it seems there is an insolvable issue in this path. SidebarViewModel does not have any inheritance whereas TabItem needs to answer to the format of TabViewItem defined by Microsoft. Hence, we cannot replicate the process with the BitmapImage.
Still trying to find a solution that being said.

@yaichenbaum had a look on the Microsoft side for the icon issue, and I don't think it's quite up my alley yet. I'm no C++ coder, and I think it'd take quite a while to find the problem and fix it.
So question is: how important is it for us to have the icons displayed on the tabs? If it's high, I'll persist on the Microsoft side, else I'll go focus on more pressing matters.

It's a pretty high priority in the sense this is a regression since migrating to WinAppSdk but the issue itself isn't that bad. Is there an issue on their repo?

It's a pretty high priority in the sense this is a regression since migrating to WinAppSdk but the issue itself isn't that bad. Is there an issue on their repo?

An issue has been opened, as it was mentionned in the comments above. For me, to try and just try to fix it is gonna have a big entry cost since I don't usually do C++. Not even sure I'll find a solution.
It's mostly to known if I focus on this issue or if I spend sometime first on others pressing issues. (Crashes / perfs etc...)

WinUI 3 is closed source so there isn't anything we can do from our end.

... Okay I was about to work on WinUI 2 ! Thanks for pointing it out. Well, in that case I'll go raise an issue to WinUI3.
I'm still not sure how to make a custom controller for that issue though, so I'll pass the towel to someone more experimented that I.
Worst case scenario, we still have the possibility to replace the icons by Symbol Icons as was done in #10188 until WinUI 3 fixes it;

@yaichenbaum We have same issue. This issue seems to be an issue MUX-side. TabView control certainly worked properly in WinUI 2.7 and also although I searched mux repo for this issue, there is no such an issue filed.

@onein528 have you filed an issue on the WinUI repo?

I'll do it.


The 'modify' keyword is not supported in postgres

I don't understand what this code does. Why compare lengths?
https://github.com/sebastienros/yessql/blob/c0771eb555c67bf419859016cb3563a3540fe7c7/src/YesSql.Provider.PostgreSql/PostgreSqlCommandInterpreter.cs#L18-L22
https://github.com/sebastienros/yessql/issues/244#issuecomment-1489671512

         SchemaBuilder.AlterIndexTable<AuditTrailEventIndex>(table => table
                        .AlterColumn(nameof(AuditTrailEventIndex.CorrelationId),
                            column => column.WithType(typeof(string), 255)),
                    collection: AuditTrailEvent.Collection
                );

Error while running migration version 0 for 'EasyOC.AuditTrail'. System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation.
 ---> Npgsql.PostgresException (0x80004005): 42601: syntax error at or near "modify"

POSITION: 42
   at Npgsql.Internal.NpgsqlConnector.<ReadMessage>g__ReadMessageLong|221_0(NpgsqlConnector connector, Boolean async, DataRowLoadingMode dataRowLoadingMode, Boolean readingNotifications, Boolean isReadingPrependedMessage)
   at Npgsql.NpgsqlDataReader.NextResult(Boolean async, Boolean isConsuming, CancellationToken cancellationToken)
   at Npgsql.NpgsqlDataReader.NextResult(Boolean async, Boolean isConsuming, CancellationToken cancellationToken)
   at Npgsql.NpgsqlDataReader.NextResult()
   at Npgsql.NpgsqlCommand.ExecuteReader(CommandBehavior behavior, Boolean async, CancellationToken cancellationToken)
   at Npgsql.NpgsqlCommand.ExecuteReader(CommandBehavior behavior, Boolean async, CancellationToken cancellationToken)
   at Npgsql.NpgsqlCommand.ExecuteNonQuery(Boolean async, CancellationToken cancellationToken)
   at Npgsql.NpgsqlCommand.ExecuteNonQuery()
   at Dapper.SqlMapper.ExecuteCommand(IDbConnection cnn, CommandDefinition& command, Action`2 paramReader) in /_/Dapper/SqlMapper.cs:line 2848
   at Dapper.SqlMapper.ExecuteImpl(IDbConnection cnn, CommandDefinition& command) in /_/Dapper/SqlMapper.cs:line 581
   at YesSql.Sql.SchemaBuilder.Execute(IEnumerable`1 statements)
   at YesSql.Sql.SchemaBuilder.AlterTable(String name, Action`1 table)
   at YesSql.Sql.SchemaBuilder.AlterIndexTable(Type indexType, Action`1 table, String collection)
   at EasyOC.AuditTrail.Migrations.UpdateAuditTrailMigrations.Create() in D:\SourceCodes\JZSoft\EasyOC.OrchardCore\EasyOC\src\Modules\EasyOC.AuditTrail\Migrations\UpdateAuditTrailMigrations.cs:line 30
  Exception data:
    Severity: ERROR
    SqlState: 42601
    MessageText: syntax error at or near "modify"
    Position: 42
    File: scan.l
    Line: 1133
    Routine: scanner_yyerror
   --- End of inner exception stack trace ---
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Span`1& arguments, Signature sig, Boolean constructor, Boolean wrapExceptions)
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at OrchardCore.Data.Migration.DataMigrationManager.UpdateAsync(String featureId)    at System.RuntimeMethodHandle.InvokeMethod(Object target, Span`1& arguments, Signature sig, Boolean constructor, Boolean wrapExceptions)
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at OrchardCore.Data.Migration.DataMigrationManager.UpdateAsync(String featureId)

Use Alter Column instead
https://www.postgresql.org/docs/current/sql-altertable.html

Yes,  This is the code I'm using now

This kind of problem seems very difficult to cover completely, can pass
Could you consider using other components to implement table structure migration?
For example:
https://github.com/fluentmigrator/fluentmigrator

My assumption based on the first comment

@hishamco
So you think modify should be changed to alter
but why are there two cases , alter  and modify  ?
@sebastienros Can you help me look at this problem? thanks!

Could you point to the two cases? Also, modify your PR according to Seb comment

Could you point to the two cases? Also, modify your PR according to Seb comment

Ok, but I've been very busy lately. Lots of concurrent projects ~ My work is already scheduled after May 10th ~
This unit test shouldn't be difficult, it'll just take a little bit of time

If you don't have a time I can create a PR instead

@hishamco Sure, I've closed my PR

Please leave it open until I create my PR or complete yours :)

Please leave it open until I create my PR or complete yours :)

it doesn't matter I only changed one word


Copyright and license

It seems large parts of your code are taken from the Microsoft.Owin.Security libraries with only minor edits, like renaming the namespace and class, e.g. MohammadYounes.Owin.Security.MixedAuth/Provider/MixedAuthApplyRedirectContext.cs is from Microsoft.Owin.Security.Google/Provider/GoogleApplyRedirectContext.cs.  Shouldn't you retain their copyright headers rather than replacing with your own?  Also, if I'm not mistaken the original code is licensed using the Apache license not MIT's.

Following the patterns used in building OWIN providers in Katana project does not mean the code is taken from there! For example: OwinOAuthProviders which is licensed under MIT too.

I'm not a lawyer so maybe I'm misunderstanding the use of copyrights and licenses.  Are you saying the code in the Provider folder is not copyrightable because they are merely a patterns?  I looked at a few of the files in the RockstarLabs project that you linked and don't see that they claimed any copyright on that code.

Well, now I'm not sure if this is considered a copyright infringement or not! As for RockstarLabs, the copyright notice redirects you to license.txt.

Perhaps I shouldn't have used the word "taken" in my initial comment as I'm sure it was inadvertent.  I just wanted to point out that your copyright claims for some of this code might be somewhat questionable.  For example, compare these:
GoogleAuthenticationProvider
MixedAuthProvider
MicrosoftAccountAuthenticationOptions
MixedAuthOptions

I'm aware of the similarities, as I tried to make it as much as possible similar to Katana, which makes it easier for developers to use/debug the MixedAuth middleware.

I don't see a problem with that.  But why the copyright claim?  If there's to be any copyright header on those files, the original ones should be there instead or at least in addition to yours.  The Apache License and Notice files should also be included.

why the copyright claim?

Because I'm the author of these files.

The Apache License and Notice files should also be included.

I don't think it's necessary.
May I ask why you are so concerned about this ?

Because I'm the author of these files.

As far as I understand, "author" means "original creator", which doesn't appear to me to be the case, even the comments are exactly same.  Btw, you forgot to change the options param in MixedAuthApplyRedirectContext.cs.

I don't think it's necessary.
May I ask why you are so concerned about this ?

Since you took the time to add your own copyright and license to these files and resistant to changing them, you obviously see value in copyrights.  So why not respect copyright claims of others?

I'm not resistant to change, and I don't deny using some code from katana project. But you still didn't answer my question!

Redirecting this issue to make it about me and my concerns is unhelpful and off-topic.  But if your curiosity must be satisfied, I'm concerned because I believe OSS is a sort of ecosystem which works best when we respect each other and our contributions.  And, part of that is not claiming authorship of someone else's work.

I appreciate your concern, and I didn't intend to take credit for someone else's work. I'm sorry it looks like that to you.
Will clear this up in the next update.
Thanks.

@acctforrandomstuff Sorry for being late on this, Is this update clear enough ?


Not a shared variable, a safe variable in the thread stack

Context

StringBuffer is not a shared variable, and it‘s a safe variable in the thread stack. So I think you can use sb instead of it to achieve the purpose of not needing a sync block synchronized{} and executing fewer bytecode instructions
dismiss warning: Return value of the method is never used

Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[x] Ensure that tests pass sanity check: ./gradlew sanityCheck
[x] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Gradle Core Team Checklist

[x] Verify design and implementation
[x] Verify test coverage and CI build status
[ ] Verify documentation

Sorry for the late reply.
@SuppressWarnings("UnusedReturnValue") is not meaningful for this case, but the StringBuffer change is good.
Please remove @SuppressWarnings("UnusedReturnValue") from the PR, resolve conflicts and then it can be merged.

This PR appears to be lacking tests. Consider adding tests to cover the new functionality

@ov7a @SuppressWarnings("UnusedReturnValue") has been removed from this PR，thanks

@bodhili Thanks! Could you please sign DCO, squash, and resolve the conflicts, please?

squash

already solved，thans

@bodhili There is still a merge conflict, please rebase to master.

@bodhili Thanks! Merging.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you. Click here to see all failures if there's any.

@bodhili Thanks! Merging.

Thanks.  This PR was submitted in December 2022. It has been almost a year, and I have forgotten about this PR, haha.


try (Playwright playwright = Playwright.create()) { ...       What is this syntax? Playwright can be called multiple times in multiple threads Create()?[Question]

try (Playwright playwright = Playwright.create()) { ...
What is this syntax? Playwright can be called multiple times in multiple threads Create()?

You can have one playwright instance per thread without complicated synchronization, see https://playwright.dev/java/docs/multithreading


AttributeError: module 'torch.backends' has no attribute 'mps'

Did all install steps
web ui dont work at all, getting error when typing python infer-web.py

Please update torch


Do not set fsGroup when its not needed

For issues like https://github.com/istio/istio/issues/26882
On Kubernetes 1.19+ we can avoid this issue. This adds a flag to disable
setting this option, and automatically disables this if we are on
Kubernetes 1.19+
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@FrimIdan I think this can solve your case with first party jwt as well? WDYT?

@FrimIdan I think this can solve your case with first party jwt as well? WDYT?

@howardjohn Does this fix automatically disable the fsGroup injection when running with first party jwt or we need to also disable to set ENABLE_LEGACY_FSGROUP_INJECTION to false?

Does this fix automatically disable the fsGroup injection when running with first party jwt or we need to also disable to set ENABLE_LEGACY_FSGROUP_INJECTION to false?

No, but I suppose we should do that as well? I don't see any risk here, do you?

Does this fix automatically disable the fsGroup injection when running with first party jwt or we need to also disable to set ENABLE_LEGACY_FSGROUP_INJECTION to false?

No, but I suppose we should do that as well? I don't see any risk here, do you?

This was my intention in the first place - If we run first party we definitely don't need the injection - I've tested it

@howardjohn What was changed in K8s 1.19 that the fsGroup injection is not needed?

@FrimIdan https://github.com/kubernetes/kubernetes/commit/3db7275b549559696c42c0b5f51c9a2397e9571d is the fix

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
68a292b566c2178af5cab2139d0e44232a5966fe
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@FrimIdan kubernetes/kubernetes@3db7275 is the fix

Thanks, looks like if the main container will run with runAsUser != 1337 we will have the same problem - only the main container will have permissions to access the projected SA , or I misunderstand the fix?

@FrimIdan I think we hit the "fallback" case listed in https://github.com/kubernetes/enhancements/pull/1598?

@FrimIdan I think we hit the "fallback" case listed in kubernetes/enhancements#1598?

You probably right - thanks for the info and for the update!

@FrimIdan added the logic for 1p jwt as well. Thanks for pointing this out!

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
9a9f5a25c5541f1c07a9d82fed059cf3e4189511
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@esnible

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
2c5fa089c75fc326819e9b8cb58ba74376b887db
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Sidekick Bookmarklet RC

This PR has been auto-generated from https://github.com/adobe/helix-sidekick-extension/.

Changes in this release:

https://github.com/adobe/helix-sidekick-extension/pull/202
https://github.com/adobe/helix-sidekick-extension/pull/207
https://github.com/adobe/helix-sidekick-extension/pull/210
https://github.com/adobe/helix-sidekick-extension/pull/217


replace "representative" edge ID with a random Integer (and clean up the mess...)

[ ] see title
[x] design efficient way to look up edges via endpoints etc -> (re-)use Dupe checker logic
[ ] replace getEdgeByID with getEdgeByNodes/Direction/Weight/Type
[ ] update all pertaining graph tests

[ ] also throw out uuid (v4) -> we want just an Int32, so we might use that within a number array


Debt: remove no more needed EOL notifications for unsupported OS

I am not sure which one is still needed, but we have:
Windows 8/7 warning:
https://github.com/microsoft/vscode/blob/ccb95fd921349023027a0df25ed291b0992b9a18/src/vs/workbench/electron-sandbox/window.ts#L726
macOS warning:
https://github.com/microsoft/vscode/blob/ccb95fd921349023027a0df25ed291b0992b9a18/src/vs/workbench/electron-sandbox/window.ts#L801

Taking a stab at it: https://github.com/microsoft/vscode/pull/191004


Question: why use _alloc?

Can you explain your changes to the libass code like the below:

Is it to do with WINRT?

It was used because its functionally identical. MSVC doesnt support dynamic array sizing (a C99 feature) so it cant compile 'mul[max_subdiv + 1]'. This line of code just allocates a 'max_subdiv + 1' sized section of memory on the stack for the duration of the function. So the changed line of code does exactly the same thing (alloca to allocate stack memory) just using syntax that msvc can actually compile.

Interesting.  In later sources it was changed to:
enum { max_subdiv = 15 };
double mul[max_subdiv + 1];

Apparently the same problem is not present with enum as a constant int, as this compiles in VS. It's not really dynamic if it is hardcoded in a const int, but I guess msvc couldn't tell that.

Yep, enums are compile time constants whereas a const variable is not (this is why c++ added constexpr variables as those are compile time constants).

FYI, using latest sources, and MSVC 2019, I require no changes to the libass source to compile (I am not using WINRT).
The enum solution is simpler, and perhaps while not exactly "functionally equivalent," it should in theory behave identically, so maybe it's a cleaner code change until a new release comes out.  It has the added benefit of being consistent with newer libass sources, fwtw.
Thanks for explaining the change.


Add configuration capabilities to FerrumPdf

This PR introduces the ability to configure FerrumPdf globally using an initializer. The changes include:

Added a configure method to the FerrumPdf module to allow setting global options.
Updated the browser method to use the global configuration when creating a new Ferrum::Browser instance.
Added a reset_configuration! method for testing purposes.
Updated the README to explain how to use the new initializer.
Added tests to ensure the configuration works as expected.

These changes allow users to set default options for FerrumPdf in their Rails applications, improving flexibility and ease of use. Users can still override these global settings on a per-use basis when calling render_pdf or render_screenshot.
To use the new configuration, users can add an initializer file config/initializers/ferrum_pdf.rb in their Rails application:
FerrumPdf.configure do |config|
  config[:headless] = true
  config[:timeout] = 20
  config[:window_size] = [1280, 800]
  # Add any other Ferrum options as needed
end

This PR aims to enhance the flexibility and customization options of FerrumPdf while maintaining backwards compatibility.

@excid3 I've updated the configuration tests to address the timeout issues we were experiencing. I'm hoping this will allow the GitHub workflows to pass successfully now.
If you think we need to adjust the timeout values further or if you have any other suggestions, please let me know. Thanks!

The only reason to disable headless if for debugging, so I think it's best the test suite uses all headless. 👍


Doc PR |Outbreak Response - Check Point Quantum Security Gateways Information Disclosure Attack

Added 4 md files for Outbreak Response SP, README.md Contents.md Usage.md and Setup.md

Approved


Publish new version on crates.io

Please upgrade your crate at crates.io because the cortex-m dependency is not the same as here
(0.6.4 on crates.io and 0.7.2 here)
Thank you

Thanks for letting us know, @papyDoctor! I've added publishing a new version to my task list. Might take a bit until I can get to it though. I recommend depending on the Git repository directly in the meantime.

I've opened #336. Currently waiting on review.


Decision making and branching in java

Proposed title of article
Decision making and branching in java
Introduction paragraph (2-3 paragraphs):
A java program is a set of statements, which are normally executed sequentially in the order in which they appear. This happens when options or repetitions of certain calculations are not necessary. However, in practice, we have a number of situations, where we may have to change the order of the execution of statements based on certain conditions, or repeat a group of statements until certain specified conditions are met. This involves a kind of decision making to see whether a particular condition has occurred or not and then direct the computer to execute certain statements accordingly.
When a program breaks the sequential flow and jumps to another part of the code, it is called branching. When the branching is based on a particular condition, it is known as conditional branching. If branching takes place without without any   decision, it is known as unconditional branching.
Java language possesses such decision making capabilities and supports control or decision making statements.
Key takeaways

If statement
If…else statement
Nested if…else statement
Switch statement

References:
N/A

seems like a helpful topic - lets please be sure it add value beyond what is in the official docs and that it does not overlap with any existing EngEd articles or incoming topic suggestions (if you haven't already). - approved @Georgewamama


fix: updated call to Asset.new

Asset.new requires a target. passed self
What does this PR do?

Information

[x] This PR fixes an issue.
[ ] This PR adds a new feature.
[ ] This PR refactors code.
[ ] This PR has breaking changes.
[ ] This PR won't change the behavior of the code. (e.g. documentation)

Checklist

[ ] I have reviewed the code and it is clean and well documented.
[ ] I have ran bot and it is working as expected.
[ ] I have updated the document.

Related issues

Thanks!


Send data repeatedly

when the socket is disconnet, the program will add the messages from client into mQueuedMessages ,when the socket is reconnected ,it will be resend to servers. but it don not removed from the  mQueuedMessages .when the socket is disconnect again and reconnected again, the messages can be send again.So, the message must be remove from mQueuedMessages when it has be resend successful.

Thanks!
This is https://github.com/delight-im/Android-DDP/issues/61, by the way. I had proposed a solution there, already, but yours may be better.
Before we can merge this, we need two things:

Can you run an example with the old version of this library (without your changes) and see if the messages are really sent multiple times? We need this to be reproduced in practice.
Can you verify that the messages are not sent multiple times anymore after applying your changes? We need this to be tested in practice as well.

Merged in https://github.com/delight-im/Android-DDP/commit/fdf1ddc76302fb3c4d6998ab3c0c718fc6bd9a57.
Thanks again!


add ability to stash audit log

Put an audit log of what the tool did into S3 or somewhere else for analysis and debugging later. Can help with confusing scenarios of "why did my thing go away" or "why did this thing not go away".

Documented in README


train_test_split_edges uses single random source

current implementation of train_test_split_edges uses multiple randomness sources which harden the ability to reproduce experiment results. Given the name "torch_geometric" one would expect having set torch randomness state, that the produced split would be the same -> but this is not the case!
As such I suggest using only torch.randperm as the randomness source for the function.
Ido

I fully agree. The previous sampling technique has the advantage that it does not have to materialize row.size(0) entries, and hence is a little bit more memory-friendly.

Given that row itself already exist, materializing a vector of similar size seem reasonable, but it is your call.
Ido

That sounds good to me :) Thanks!


Add docs

Summary
Add docs introduction, docs template and docs for /auth/discord/redirect
Checklist

[ ] If endpoints were changed then they have been documented and tested.

[x] I have updated the docmentation to reflect the changes.
[ ] I have updated the tests to support the changes.

[ ] This PR fixes an issue.
[x] This PR adds something new (e.g. new endpoint or parameter).
[ ] This PR is a breaking change (e.g. endpoint or parameters removed/renamed)
[x] This PR is not a code change (e.g. documentation, README, ...)

Making the repo public messed up my fork and my PR, i will PR this again from the fixed repository

Making the repo public messed up my fork and my PR, i will PR this again from the fixed repository


Provide a way to see Wikidata and the source on the same page

I think it would be valuable to have a view where the left side of the screen shows Wikidata and the right side of the screen shows the website to which the reference points. At best the website with an annotation that suggests which lines of the website are the reason for StrepHit thinking that it supports the fact.
Making it easier to see both Wikidata and the source at the same page will reduce the amount of time where a reference get's approved without the user looking at the source page.

This could alternatively be handled by the user's browser, if they had a plugin to produce that view.

Most users won't download plugins. Yet there a good chance that the current setup encourages many people do approve false references.

In order to ease us (the Wikidata dev team) to track the issues of the primary sources tool, we have created a Phabricator ticket for this issue: https://phabricator.wikimedia.org/T148140
Moving forward, please drop your comments onto that Phabricator ticket.


[Bug] cNFT Account Says It Does Not Exist

Describe the bug
cNFT Account Says It Does Not Exist. This seemed to just start happening recently.
Screenshots
Missing cNFT Account: https://explorer.solana.com/address/7JjTKfqt3U9ndNYX5GHuAHUCFHG53rfTKZFqML93MksG?cluster=devnet

Existing cNFT Account: https://explorer.solana.com/address/A4HY5HEsymkbqnyCK99v5zWSU1EMTSnabuGWoLhBibni/attributes?cluster=devnet

@ngundotra this issue still appears to be occurring...
https://explorer.solana.com/address/7X1Lxukqi3pt2ky8cvXX6P59EpoqyCRZhGZgHdGe9zfi?cluster=mainnet
I do see it here though...
https://solscan.io/token/7X1Lxukqi3pt2ky8cvXX6P59EpoqyCRZhGZgHdGe9zfi

@philetos thanks for linking this! It's an error with RPCs. I will pass this along & see if we can get this fixed.
Thanks for flagging this!


Fix custom rule/target file copy

[patch]
Remove placeholder dirs as they cause Ansible to ignore local files
dirs. Simplify fileglob by only copying valid filenames.
Signed-off-by: Ben Kochie<EMAIL_ADDRESS>

Fixed up with [patch]. :)


Resolve logging when the request completes

See this PR feedback: https://github.com/aspnet/Hosting/pull/403#discussion_r42696834
There's some things we can't log accurately right now from hosting (after request). Need to continue the discussion about how to improve the accuracy of this logging.

Offline recommendation was to move the logging into the servers. WebListener already has these kinds of logs.

https://github.com/aspnet/WebListener/blob/dev/src/Microsoft.Net.Http.Server/RequestProcessing/Request.cs#L156

This will be resolved in https://github.com/aspnet/Hosting/issues/395 with the new design and reactions that that design in all affected servers.


#243: Convert sharding-kafka to Pekko

incubator-pekko issue : https://github.com/apache/incubator-pekko/issues/243
Method: See migration guide MR https://github.com/apache/incubator-pekko/pull/249
Since the GRPC and Alpakka tooling are required here this required slightly more code changes/ dependency tweaking.
One of key thing is that alpakka dependencies are going from :  akka-stream-kafka -> pekko-connectors-kafka

lgtm. Some more additional things need to be done but it can be done in future PR's.

thanks. Note I can't merge because i don't have write access on this repo.

@sam-byng BTW I just noticed that your gpg/pgp signature for git commits is coming up as not verified in github.

Just looked into this. I think this is because I've been signing with a different email address to my Github account. Will tweak this for future fixes.


unsigned in migration

normally filed like id will not be -500
in migration function like: $this->integer()->notNull(),
add another function: $this->integer()->notNull()->unsigned(),

https://github.com/yiisoft/yii2/issues/9443


Webpack encounters errors without manually installing some modules

I get errors in the pattern below for the following modules - 'redbox-react', 'react-transform-catch-errors', 'react-transform-hmr'. I notice several are in dev dependencies but not the primary dependencies.
ERROR in ./~/react-text-filter/lib/TextFilter.js Module not found: Error: Cannot resolve module 'redbox-react'
Also, once I install all of the above dependencies, I get the following error - Uncaught Error: locals[0] does not appear to be a moduleobject with Hot Module replacement API enabled. You should disable react-transform-hmr in production by usingenv section in Babel configuration. See the example in README: https://github.com/gaearon/react-transform-hmr
Thanks!

Hi, @carbureted
They should not be in main dependencies since are needed only for development.
If you are using library in your project (as require('react-text-filter')) - it has all pre-built files in there. As well as original src in case you wish to build yourself.

Closing for now, reopen please, if you still have an issue


FCREPO-3086 - Separate server and client managed triples

JIRA Ticket: https://jira.duraspace.org/browse/FCREPO-3086
What does this Pull Request do?
This is somewhat exploratory, still trying to get a sense of how we are going to shuttle the pieces of the object back and forth.

Refactors getTriples into two methods
Adds a stubbed out FedoraResourceImpl with triples related methods populated
Adds read methods for binary content, triples, and server managed properties
Refactors workflow for getting an existing resource to start to flesh out how it might work without prepopulating the object.

What's new?
FedoraResourceImpl and some implementation related to it
How should this be tested?
There aren't any tests yet, although if this approach looks reasonable it might be good to add a test for ResourceFactoryImpl to this PR.

Whew, once I see the 23 new interfaces/classes I'm less convinced this is going to be easier to understand then mutating FedoraResource and we don't have any Delete requests here yet. 🤷♂
Do we think some of these could be combined? Do we need specific Create/Update for RDFSource and NonRdfSource objects?

@whikloj Filling out the operation/factory/builder structure isn't much fun, but its mostly a lot of boiler plate. There will likely be a lot of shared code. I wasn't sure if we needed any delete requests or not. I changed the delete method to take an id instead of a resource in the hopes that was all that was needed.
I ended up with so many operations mainly because RDF and Non-RDF sources have different data needs, and I wanted to the persistence layer to be able to easily tell what kind of operation/object it was dealing with. The builder/factory structure attempts to corral clients into not adding invalid data for an object type or operation. Relaxing things some would probably reduce the number of classes, so that's an option if people feel like it'd be better.

@whikloj and @bbpennel :  Strong thumbs up on having separate classes for each operation.   Inheriting from the abstract will limit the amount of boiler plate and I concur that making the pairing of parameters with operation explicit will make writing tests easier.

@bbpennel : One other comment:  I wonder if it would be better to have a DeleteOperation for the sake of consistency.  I can go either way there,  but just thought I would throw it out there.

@dbernstein i don't have a strong opinion about it, I'll go ahead and add a delete operation


[API-604] Add generated SQL encoders, implement query_id and its encoder.

Adds generated sql_close_encode,  sql_execute_encode, and sql_fetch_encode
Adds sql::impl::query_id
Adds ClientMessage::set(sql::impl::query_id)

Related protocol PR: https://github.com/hazelcast/hazelcast-client-protocol/pull/411

Windows test FAILed.

Linux test FAILed.

Linux test FAILed.

Windows test FAILed.

Windows test PASSed.

Linux test PASSed.

Closing in favor of #959


Supporting type annotations and type checking in JavaScript files

Given that there is substantial overlap between Flow and TypeScript syntax for type annotations and support in the Babel transpiler for stripping Flow annotations, it is now possible to write source files which are both valid type-annotated TypeScript and Babel-parsable ES2015 JavaScript, except for the restriction of type annotations and type checking to .ts files.
I think it could lower the perceived barrier to entry for TypeScript significantly if it could be used with existing Babel/JS codebases (at least those using ES2015 imports), as a type-checker (or "super linter") and IntelliSense/refactoring provider, just by dropping in a suitable tsconfig.json file.
Salsa/JSDoc/--allowJs are welcome steps but JSDoc is much more verbose, especially for the common task of defining interfaces. Documentation on how to describe interfaces precisely is also much better with TypeScript, since JSDoc is often written only to the standards of documentation, not as something that a machine can actually use.
As a concrete proposal, this might mean:

Allowing type annotations, including interfaces, in JS files if --allowJs is enabled
Providing a way to enable type checking within JS files, possibly via a //@flow style comment or project-wide via a tsconfig flag.

Could you elaborate on why you want to add annotations/interfaces to JS files?  Isn't that just the different behavior you'd get anyway if you renamed them to .ts files?

No, as far as I know, Typescript files need to be compiled and all the codebase has to be Typescript if one file is Typescript. I think he means something like allowing per-file typechecking like Flow is doing, but with Typescript's type system and definitions instead of reinventing something from scratch.

My perception is that the perceived barrier to trying out TypeScript is higher than it needs to be. TypeScript solves two problems - it serves as a type checker and transpiler. Many projects already have a perfectly working solution for transpiling JS though in the form of Babel and probably a growing additional set of tooling which supports the same syntax (ESLint etc.)
Enabling TypeScript to be used just as a type checker only, without disrupting the existing build process (or tooling like linters) by renaming source files, would enable it to be used initially like a much smarter linter. For a project that is happily using Babel for its transpilation needs and already using ES2015 module syntax, the process could be like this:

Drop in a tsconfig.json file, perhaps initially configured to type-check only a subset of files.
Run tsc to type-check files.
Gradually add type annotations to JS files, using the subset of syntax that is compatible with Babel/Flow, so there is no disruption to the existing build toolchain.
Perhaps eventually switch to TypeScript for the whole pipeline (less tooling to manage) or perhaps not.

You can achieve something like the above today via say, a Gulp task that copies the files to a different directory, renames the extension in the process and pipes the result through TypeScript with the noEmit flag - but it isn't a well documented workflow that someone can easily try.
More broadly, I think having two tools support the same syntax (and preferably in at least strong agreement on semantics) for type annotations in .js files would help promote the value of type annotations and interface documentation across the JavaScript ecosystem as a whole.

all the codebase has to be Typescript if one file is Typescript.

This is untrue, we have the --allowJS flag which allows you to compile your ES6 down to ES5 or ES3 while also including it in your general TypeScript project.

Flow adds things to .js files that need to be removed before native execution even on fully feature compliant runtimes. The same is true when jsx markup is included in a plain .js file. The difference between this and say native es6 code that has not been transpiled is that the latter is actually JavaScript. I think this is a terrible idea

More arguments in favor of adding this feature in the discussion here: https://github.com/Microsoft/TypeScript/issues/7926#issuecomment-228848197

There has been multiple requests around this area. I have files a new issue to consolidate all the requests along with a proposal for a fix, see https://github.com/Microsoft/TypeScript/issues/10939. Let's use #10939 to track this change.
closing in favor of #10939.


Update Travis Credentials

It looks like the Travis Credentials are out of date.

Hey, can you follow these instructions:
https://github.com/google/clasp/tree/master/tests#clasp-login
Should reduce failures from 23 to 9.

CC @campionfellin

I removed the old encrypted keys.


⚠ Bofrost.be has degraded performance

In 296b101, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1435 ms

Resolved: Bofrost.be performance has improved in 2a7dbc3.


Enhance CMake compiler check on Linux

Somehow I missed the specified compiler version in the docs. Maybe I forgot about it by the time I got to the platform-specific requirements. Maybe I just can't read. Not sure what the problem was, but it would be nice for CMake to catch that sort of mistake.
This commit adds a warning for compilers that don't meet the project's minimum requirements. There is already an explicit check for GNU, but this will catch Clang versions that are too low as well as any non-GNU, non-Clang compilers.
The check is Linux-only as I have only built Filament on Linux and I'm not sure if this check would apply to Windows or OSX. The message is warning rather than an error to minimize the impact of false-positives caused by differing compiler names or version numbers. In particular, forks of clang may be compatible but have different names or numbers.
I know the contribution guidelines suggest I should open an issue before submitting a PR, but I figured this was a small enough change that it was probably unnecessary. I'm guessing the process was set up to avoid wasted development effort on rejected designs, but if there's some other reason I should be opening an issue first, I'd be happy to go back and do that.

I've removed the GNU check, applied my checks to all platforms, rebased against master and updated the commit message to match. I have only tested on Linux, so I'm crossing my fingers there's no surprises elsewhere.
I'm not entirely certain that WARNING is the best message level, but I figured it was better to be cautious. If you ever want to make it a hard error, just change the message level to SEND_ERROR.

I believe I've addressed the issue with OSX, though the CI will have to test it for me. I've also changed the message from a warning to an error, though I don't bail out early. I think that's preferable, but will defer to your judgment if you feel strongly about it.

You're right, the return is not useful. In this case I think fatal error makes sense because if we proceed further we might just get meaningless errors due to the use of the wrong compiler. I see this check as a fatal precondition. For other errors, sure send would be useful

Your change runs just fine on macOS by the way, thanks for doing this!


php5-fpm uses realpath (fixes #134)

From https://github.com/roots/bedrock/issues/134#issuecomment-70996256

php5 opcache doesnt resolve symlinks for file paths (APC and others did), so when you deploy a new version opcache still see the same path (/srv/www/mysite/current/web/myfile.php) so it still serve the old cached file.

That was fast. Have you tried it out yet?

Oh, I was wondering why I was having issues with that...

Awesome! Thank you for the speed @QWp6t

Could you guys tell me if there is a command line to update my instance only for particular role in Ansible? (so I don't have to run the whole book, just those updates on my server)

ansible-playbook -i hosts/production site.yml --tags "nginx"

Awesome! @swalkinshaw


SITE-45: Migrate GPU Day website to DotNest Tenants Core

SITE-45

@BenedekFarkas KEKW moment here but it does not let me merge because the expectedly failing workflow is Required hihi

It also says that it is expecting a ref at validate-this-pull-request which is a consumer of validate-submodule-pull-request. I approved the changes so I thought the expected ref is dev now. Also the prior that has problem with ref was never touched in this PR. Maybe that's why it has problems.

That's OK, in that case you just have to re-run the failed validate-gha-refs check, because after approval it will accept the branch references pointing to dev (instead of the issue branch).


Error during package analysis

Describe the bug
When running the analysis over this project we get this error output:
[17:00:53 DBG] Found target frameworks for package Microsoft.CodeAnalysis.VisualBasic.Workspaces.2.10.0: [".NETStandard,Version=v1.3"]
[17:00:53 DBG] Package Microsoft.CodeAnalysis.VisualBasic.Workspaces, Version=2.10.0 will work on ["netstandard2.0"]
[17:00:53 DBG] NuGet package , Version=0.0.0 not found in package cache
[17:00:53 FTL] Package analysis failed (analyzer Target compatibility reference analyzer: Argument cannot be null or empty. (Parameter 'id')
[17:00:53 DBG] Analyzing packages with Upgrade assistant reference analyzer
To Reproduce
Run analysis over the attached project file.
Exceptions (if any)
Further technical details
Microsoft.PracticesChecker.Analyzers.Classifiers.zip
`

Thanks, @bgarlick for the report. Looks like a bug in UA's package analysis step. I suspect that this reference is causing problems:
<PackageReference Update=”NETStandard.Library” PrivateAssets=”all” />

Most likely the tool just needs fixed to correctly handle the case of PackageReference Update.

I do not get this error w/ the attached file in the latest revs of UA - I know this issue has been around for some time, so thought I'd loop back and see if this is still something happening for you, @bgarlick ?

Honestly I have not used UA in several months, so I haven't experienced it. If the desire is to close the issue, perhaps that is fine for now and we could revisit in the future if necessary.
Thanks.
Brendt.
From: Brandon H @.>
Sent: Thursday, August 11, 2022 4:27 PM
To: dotnet/upgrade-assistant @.>
Cc: Brendt Garlick @.>; Mention @.>
Subject: Re: [dotnet/upgrade-assistant] Error during package analysis (Issue #964)
I do not get this error w/ the attached file in the latest revs of UA - I know this issue has been around for some time, so thought I'd loop back and see if this is still something happening for you, @bgarlickhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbgarlick&data=05|01|Brendt.Garlick%40microsoft.com|b3b7b9ce907f479d3bd008da7bd7d2b9|72f988bf86f141af91ab2d7cd011db47|1|0|637958464130962107|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=iywT%2BZIm1RLok5SAOmh2zfTl3awVzHmu%2FXpA4BEddf8%3D&reserved=0 ?

Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fupgrade-assistant%2Fissues%2F964%23issuecomment-1212460768&data=05|01|Brendt.Garlick%40microsoft.com|b3b7b9ce907f479d3bd008da7bd7d2b9|72f988bf86f141af91ab2d7cd011db47|1|0|637958464130962107|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=BzePnAaFi02IWpoKkmOhejin7BR0tz%2Bw9nCBJLGLsic%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAWKF5PNVISOLGTPJMGQQHEDVYVOYVANCNFSM5JFTKUSA&data=05|01|Brendt.Garlick%40microsoft.com|b3b7b9ce907f479d3bd008da7bd7d2b9|72f988bf86f141af91ab2d7cd011db47|1|0|637958464130962107|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=FL4QQQuULPphctmEb4bJtcEamIsaszsvrE27oSxvm0c%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


Trigger raycasting from events

Use events to trigger ray casts.

Raycasts can now be done on the fly with the new Raycast system param.


Incorrect render of DocFx alert

On which page?
https://platform.uno/docs/articles/external/uno.toolkit.ui/doc/getting-started.html
https://github.com/unoplatform/uno.toolkit.ui/blob/31028b21593d885d79873371c3d3271c72fa6482/doc/getting-started.md?plain=1#L10
What's wrong?

Any feedback?
It could be happening because of the colon after [!NOTE].
@kazo0

@kazo0

for alert tag, you need to follow it immediately with a newline:
> [!NOTE]
> Information the user should notice even if skimming.

see: https://dotnet.github.io/docfx/docs/markdown.html?tabs=linux%2Cdotnet#alerts


skaffold: init at 0.16.0

Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>Motivation for this change
Add skaffold package to nixpkgs.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@nlewo updated :angel:

@GrahamcOfBorg build skaffold

Success on aarch64-linux (full log)
Attempted: skaffold
Partial log (click to expand)
github.com/GoogleContainerTools/skaffold/cmd/skaffold
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/5qr9wpzjm2lcc475qkgikx2970byw3z5-skaffold-0.16.0-bin
shrinking /nix/store/5qr9wpzjm2lcc475qkgikx2970byw3z5-skaffold-0.16.0-bin/bin/skaffold
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/5qr9wpzjm2lcc475qkgikx2970byw3z5-skaffold-0.16.0-bin/bin
patching script interpreter paths in /nix/store/5qr9wpzjm2lcc475qkgikx2970byw3z5-skaffold-0.16.0-bin
checking for references to /build in /nix/store/5qr9wpzjm2lcc475qkgikx2970byw3z5-skaffold-0.16.0-bin...
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip

Success on x86_64-darwin (full log)
Attempted: skaffold
Partial log (click to expand)
find: '/nix/store/dvkhf43zgflh665ivradvy3nkl0rw8jw-skaffold-0.16.0': No such file or directory
find: '/nix/store/dvkhf43zgflh665ivradvy3nkl0rw8jw-skaffold-0.16.0': No such file or directory
find: '/nix/store/dvkhf43zgflh665ivradvy3nkl0rw8jw-skaffold-0.16.0': No such file or directory
find: '/nix/store/dvkhf43zgflh665ivradvy3nkl0rw8jw-skaffold-0.16.0': No such file or directory
find: '/nix/store/dvkhf43zgflh665ivradvy3nkl0rw8jw-skaffold-0.16.0': No such file or directory
find: '/nix/store/dvkhf43zgflh665ivradvy3nkl0rw8jw-skaffold-0.16.0': No such file or directory
find: '/nix/store/dvkhf43zgflh665ivradvy3nkl0rw8jw-skaffold-0.16.0': No such file or directory
find: '/nix/store/dvkhf43zgflh665ivradvy3nkl0rw8jw-skaffold-0.16.0': No such file or directory
find: '/nix/store/dvkhf43zgflh665ivradvy3nkl0rw8jw-skaffold-0.16.0': No such file or directory
find: '/nix/store/dvkhf43zgflh665ivradvy3nkl0rw8jw-skaffold-0.16.0': No such file or directory

Success on x86_64-linux (full log)
Attempted: skaffold
Partial log (click to expand)
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/snl7mm7gj0kr7gy8pg4hak0nscakzspm-skaffold-0.16.0-bin
shrinking /nix/store/snl7mm7gj0kr7gy8pg4hak0nscakzspm-skaffold-0.16.0-bin/bin/skaffold
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/snl7mm7gj0kr7gy8pg4hak0nscakzspm-skaffold-0.16.0-bin/bin
patching script interpreter paths in /nix/store/snl7mm7gj0kr7gy8pg4hak0nscakzspm-skaffold-0.16.0-bin
checking for references to /build in /nix/store/snl7mm7gj0kr7gy8pg4hak0nscakzspm-skaffold-0.16.0-bin...
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
/nix/store/snl7mm7gj0kr7gy8pg4hak0nscakzspm-skaffold-0.16.0-bin


[feat] Expose Always on Bottom

Describe the problem
I'm making an underlay application where I want it to be always on the user's desktop.
Describe the solution you'd like
I want the always_on_bottom wry method to be exposed to Tauri.
Alternatives considered
No response
Additional context
No response

(re-opening in case others would like to see this added)

Hey, I would like to work on this issue.
Correct me if im wrong, but basically you want to add a feature to always keep the application on the users desktop?


Split AppWrapper controller into separate dispatcher and runner

Progress towards multi-cluster MCAD by splitting the AppWrapper reconciler into two separate reconcilers that handle disjoint state transitions in the lifecycle of the AppWrapper.

Ready for review.
Made a pass this morning to update comments and to add additional steps to the Running state to allow a place for the synch/unsynch operations to take place in a true multi-cluster deployment.


Initial main.yml workflow.

Resolves #105
Adds a workflow for buildings and deploying the TelemetrySDK and Exporter.

Restores solution
Builds solution
Run unit tests
Can deploy
Saves the build artifacts, though we are still working out what to do with them or if this is the best method of saving them.

Notes:

Builds of master or adding a formatted tag will trigger a release as in the pipelines.

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


Fix race condition in rate limiter, improve WASM compatibility, and support favorite/vote

29d38026956806a9eee3629e684fcea933dccc68 fixes the case where a request takes longer than REQ_COOLDOWN_DURATION and a second request is allowed to start.
1eaf0b68d2a1de3c5b517e98ea612eb33f439d8c adds support for favoriting posts.
03635d176c4d33c017a7ca98e6d54127ada2e971 adds support for voting on posts.
aebb165f4740676a5936d74699c6c2000433bcfb and 0d01b2c37921e3e983863684eac2cd0ff1bcf438 get the crate working inside a browser in WASM. Instead of setting the user-agent header, this uses the _client query parameter as suggested in the e621 docs.

It looks like something is wrong with the json! macro in the tests

Oops, sorry about that!

Thank you very much!


Support deploying on elest.io

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
Provide an elest.io template for me to self-host without setting up my own infrastructure
https://doc.elest.io/books/cicd-pipelines/page/create-your-own-template-elestioyml
Examples 🌈
https://elest.io/open-source/appsmith
Motivation 🔦
No response

belongs to Toolpad v1


information_schema.tiflash_replica shows table info even if the user has no privileges on it

Bug Report
Please answer these questions before submitting your issue. Thanks!
1. Minimal reproduce step (Required)

Startup tiup playground with --tiflash 1 option

tiup playground --tiflash 1

Open another terminal to connect to the tiup playground instance as root.

mysql --comments --host <IP_ADDRESS> --port 4000 -u root

As a root user, create test1 and test2 databases.

create database test1;
create database test2;

As a root user, create user1 and user2 users.
The user1 has all privileges on all of test1 database, Same as user2 has all prvileges on all of test2 database.

create user 'user1'@'localhost';
create user 'user2'@'localhost';
grant all privileges on test1.* to 'user1'@'localhost';
grant all privileges on test2.* to 'user2'@'localhost';

Exit the root connection

Connect to tiup playground instance as user1

mysql --comments --host <IP_ADDRESS> --port 4000 -u user1;

Use test1 database and create t1 table and create a tiflash replica of t1.

use test1;
create table t1 (id int not null auto_increment, primary key(id));
insert into t1 values();
alter table t1 set tiflash replica 1;

Exit the user1 connection

Connect to tiup playground instance as user2

mysql --comments --host <IP_ADDRESS> --port 4000 -u user2;

Try to see the test1.t1 table as user2, which gets ERROR 1142 as expected.

mysql> select * from test1.t1;
ERROR 1142 (42000): SELECT command denied to user 'user2'@'localhost' for table 't1'

Query information_schema.tables as user2, it does not show any record where where table_schema = 'test1' as expected

mysql> select * from information_schema.tables where table_schema = 'test1';
Empty set (0.00 sec)

Query information_schema.tiflash_replica as user2, it somehow shows the record where table_schema= 'test1' and TABLE_NAME = 't1, 'which is unexpected

mysql> select * from information_schema.tiflash_replica where table_schema= 'test1' and TABLE_NAME = 't1';
+--------------+------------+----------+---------------+-----------------+-----------+----------+
| TABLE_SCHEMA | TABLE_NAME | TABLE_ID | REPLICA_COUNT | LOCATION_LABELS | AVAILABLE | PROGRESS |
+--------------+------------+----------+---------------+-----------------+-----------+----------+
| test1        | t1         |       98 |             1 |                 |         1 |        1 |
+--------------+------------+----------+---------------+-----------------+-----------+----------+
1 row in set (0.01 sec)

2. What did you expect to see? (Required)
At step 11, user2 should not see any record from  information_schema.tiflash_replica where table_schema= 'test1' and TABLE_NAME = 't1'. Same as information_schema.tables
3. What did you see instead (Required)
mysql> select * from information_schema.tiflash_replica where table_schema= 'test1' and TABLE_NAME = 't1';
+--------------+------------+----------+---------------+-----------------+-----------+----------+
| TABLE_SCHEMA | TABLE_NAME | TABLE_ID | REPLICA_COUNT | LOCATION_LABELS | AVAILABLE | PROGRESS |
+--------------+------------+----------+---------------+-----------------+-----------+----------+
| test1        | t1         |       98 |             1 |                 |         1 |        1 |
+--------------+------------+----------+---------------+-----------------+-----------+----------+
1 row in set (0.01 sec)

4. What is your TiFlash version? (Required)
mysql> select tidb_version()\G
*************************** 1. row ***************************
tidb_version(): Release Version: v7.2.0
Edition: Community
Git Commit Hash: 9fd5f4a8e4f273a60fbe7d3848f85a1be8f0600b
Git Branch: heads/refs/tags/v7.2.0
UTC Build Time: 2023-06-27 14:57:05
GoVersion: go1.20.5
Race Enabled: false
Check Table Before Drop: false
Store: tikv
1 row in set (0.01 sec)

Can you move this issue to tidb and assign to sql-infra?

close by https://github.com/pingcap/tidb/pull/45319

Thank you for the fix.


[MUGEN] Add Text-to-Video GPT Model Builder

Stack from ghstack:

#282
#281
-> #276
#280

Summary:

Added text_to_video_gpt builder for text to video generation
Added public attributes to VQVAE and MultimodalGPT

num_embeddings: codebook size --> needed to create projections that bridge the tokenizers and the transformer
embedding_dim: size of each codebook vector --> same as above
num_{in/out}_tokens: number of token states (for both the in- and out- modalities) --> needed in #282 to define the generation states

Added option to freeze video_vqvae_mugen after loading MUGEN checkpoints

Test Plan:

Temporarily modified .coveragerc to show reports for examples/

$ python -m pytest --cov=examples/mugen/generation/ examples/mugen/test/generation/test_text_video_gpt.py examples/mugen/test/generation/test_video_vqvae.py -vv
================================================= test session starts ==================================================
platform darwin -- Python 3.8.13, pytest-7.1.2, pluggy-1.0.0 -- /Users/langong/local/miniconda3/envs/t2v/bin/python
cachedir: .pytest_cache
rootdir: /Users/langong/gpt_attention, configfile: pyproject.toml
plugins: mock-3.8.2, cov-3.0.0
collected 26 items

examples/mugen/test/generation/test_text_video_gpt.py::test_encode_text PASSED                                   [  3%]
examples/mugen/test/generation/test_text_video_gpt.py::test_encode_video[8-expected0] PASSED                     [  7%]
examples/mugen/test/generation/test_text_video_gpt.py::test_encode_video[16-expected1] PASSED                    [ 11%]
examples/mugen/test/generation/test_text_video_gpt.py::test_encode_video[32-expected2] PASSED                    [ 15%]
examples/mugen/test/generation/test_text_video_gpt.py::test_decode_video[8-55462.1641] PASSED                    [ 19%]
examples/mugen/test/generation/test_text_video_gpt.py::test_decode_video[16-112028.1719] PASSED                  [ 23%]
examples/mugen/test/generation/test_text_video_gpt.py::test_decode_video[32-225157.7656] PASSED                  [ 26%]
examples/mugen/test/generation/test_text_video_gpt.py::test_decode_video_checkpoint[8-116013.4766] PASSED        [ 30%]
examples/mugen/test/generation/test_text_video_gpt.py::test_decode_video_checkpoint[16-237488.625] PASSED        [ 34%]
examples/mugen/test/generation/test_text_video_gpt.py::test_decode_video_checkpoint[32-536481.4375] PASSED       [ 38%]
examples/mugen/test/generation/test_text_video_gpt.py::test_lookup[in-expected_shape0--53.7916] PASSED           [ 42%]
examples/mugen/test/generation/test_text_video_gpt.py::test_lookup[out-expected_shape1-42.4742] PASSED           [ 46%]
examples/mugen/test/generation/test_text_video_gpt.py::test_forward[8-80.8206] PASSED                            [ 50%]
examples/mugen/test/generation/test_text_video_gpt.py::test_forward[16-9.1838] PASSED                            [ 53%]
examples/mugen/test/generation/test_text_video_gpt.py::test_forward[32-43.5649] PASSED                           [ 57%]
examples/mugen/test/generation/test_text_video_gpt.py::test_forward_checkpoint[8-80.8206] PASSED                 [ 61%]
examples/mugen/test/generation/test_text_video_gpt.py::test_forward_checkpoint[16-9.1838] PASSED                 [ 65%]
examples/mugen/test/generation/test_text_video_gpt.py::test_forward_checkpoint[32-43.5649] PASSED                [ 69%]
examples/mugen/test/generation/test_video_vqvae.py::TestVideoVQVAEMUGEN::test_invalid_pretrained_model_key PASSED [ 73%]
examples/mugen/test/generation/test_video_vqvae.py::TestVideoVQVAEMUGEN::test_freeze_model PASSED                [ 76%]
examples/mugen/test/generation/test_video_vqvae.py::TestVideoVQVAEMUGEN::test_forward[8--16005.1113] PASSED      [ 80%]
examples/mugen/test/generation/test_video_vqvae.py::TestVideoVQVAEMUGEN::test_forward[16-51112.2266] PASSED      [ 84%]
examples/mugen/test/generation/test_video_vqvae.py::TestVideoVQVAEMUGEN::test_forward[32-450732.6875] PASSED     [ 88%]
examples/mugen/test/generation/test_video_vqvae.py::TestVideoVQVAEMUGEN::test_checkpoint[8-132017.28125] PASSED  [ 92%]
examples/mugen/test/generation/test_video_vqvae.py::TestVideoVQVAEMUGEN::test_checkpoint[16--109636.0] PASSED    [ 96%]
examples/mugen/test/generation/test_video_vqvae.py::TestVideoVQVAEMUGEN::test_checkpoint[32-1193122.0] PASSED    [100%]

---------- coverage: platform darwin, python 3.8.13-final-0 ----------
Name                                          Stmts   Miss  Cover
-----------------------------------------------------------------
examples/mugen/generation/text_video_gpt.py      69      8    88%
examples/mugen/generation/video_vqvae.py         24      0   100%
-----------------------------------------------------------------
TOTAL                                            93      8    91%

================================ 26 passed in 46.01s ======================

Differential Revision: D38865165

@langong347 has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.

Codecov Report

:exclamation: No coverage uploaded for pull request base (gh/langong347/33/base@bfa076e). Click here to learn what that means.
The diff coverage is n/a.

@@                   Coverage Diff                    @@
##             gh/langong347/33/base     #276   +/-   ##
========================================================
  Coverage                         ?   92.18%
========================================================
  Files                            ?       53
  Lines                            ?     3201
  Branches                         ?        0
========================================================
  Hits                             ?     2951
  Misses                           ?      250
  Partials                         ?        0

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

@langong347 has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.

@langong347 has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.


Regression: Mypy integration highlights errors from other files when editing a Python file

Environment data

VS Code version: 1.41.1 2019-12-18T14:57:51.166Z
Extension version (available under the Extensions sidebar): 2020.1.57204
OS and version: macOS Mojave 10.14.6
Python version (& distribution if applicable, e.g. Anaconda): Python 3.8.1
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): venv
Relevant/affected Python packages and their versions: mypy 0.750
Jedi or Language Server? (i.e. what is "python.jediEnabled" set to; more info #3977): JEDI

Expected behaviour
The Python code editor only highlights errors in the file in question.
Actual behaviour
The Python code editor wrongly highlights unrelated errors found in other files that Mypy checked.
Steps to reproduce:
This GIF shows the error at work (click on it to see better, Shift+Cmd+R to restart the animation). Things to notice:

the code was initially free of Mypy errors, as I'm showing
making a single type error produces Mypy errors in three files; that's expected and pretty common
opening one of those other files (redblue.py) will show an unrelated error on line 39 (which is actually the error from midi.py, the original file we edited)

To reproduce this yourself, clone aiotone and configure it.
$ git clone https://github.com/ambv/aiotone
$ python3.8 -m venv /tmp/aiotone
$ source /tmp/aiotone/bin/activate
(aiotone)$ poetry install
(aiotone)$ code aiotone.code-workspace

Now, in VScode, you might need to let it know the location of your Python interpreter within your virtualenv. Then, open aiotone/midi.py and make the same mistake I did on line 39. Then, open aiotone/redblue.py and observe the Mypy error from midi.py appear on line 39 in redblue.py, too.
Observations

This regression is not present in the previous release of python-vscode.
When mypy.ini is not present at all, this bug does not manifest.
One commit I found that might be related: https://github.com/microsoft/vscode-python/commit/1b6fbfba8149ea78244cecbb80fa203b10a1f5a5

Hi @ambv 👋 Thank you for reporting this issue and the thorough description, it was super helpful 👍
I can confirm that reverting https://github.com/microsoft/vscode-python/pull/5834 makes this issue disappear, we'll add this fix to a point release (aiming for Monday).
Thanks!

Wow. A two-hour turnaround on issues is 💯
Seriously impressed. Thanks! ✨ 🍰 ✨

I can confirm it doesn't happen anymore in master. The fix should be out in the stable version next Monday.


Gutters

Hi there,
Have you got any gutter system you use with sass-flexbox-grid? Trying to figure out one for myself but thought I'd ask first.

Thanks for getting back to me. That would be amazing if there were a way you could turn the gutters on/off - exactly what I'm after.
Really loving your grid! Have had a look at a few out there but this one's ticking a lot of the boxes. Love how you have two options to control the layout either by classes or mixins too. Would recommend to anyone looking for a solid grid system.

Hey @drewbot How you getting on with the gutters?

I'm also looking for this feature. Any update?


website

Hey there,
I have peaked into website, may I know what do you want to change in it.
As a web designer shall I try to find the issues in your site.


Copy previous ident_func code from Werkzeug

Werkzeug 2.0 is changing how Local is implemented to use a ContextVar instead. This means that the ident_func is no longer relevant (and probably shouldn't have been used externally before either). Currently, it raises a RuntimeError, but soon it will show a DeprecationWarning. Either way, Flask-SQLAlchemy shouldn't try to use Werkzeug's implementation anymore.
Since Flask-SQLAlchemy doesn't have a notion of coroutine-local connections anyway, we can copy Werkzeug's ident_func import block and use it locally instead. https://github.com/pallets/werkzeug/blob/44866cc6419dd1742b86a0b7bccd32ba2eb58f77/src/werkzeug/local.py#L11-L17
Environment:

Python version: >=3.7
Flask-SQLAlchemy version: latest
SQLAlchemy version: 2.0.0b1

Hi ! I can help on this one if you want ?

Hi, how can this issue be overcome in the meantime?

It doesn't need to be, Werkzeug 2.0 isn't released yet.

Maybe this helps anyone:
I allowed prereleases in my Pipfile (for black) so I had Werkzeug 2 automatically installed which made my code not runnable.
I wrote werkzeug="<2.0.0" in my Pipfile to make it work again.


App is crashing while fecthing the images

thread #10, name = 'io.flutter.1.io', stop reason = EXC_RESOURCE RESOURCE_TYPE_MEMORY (limit=2098 MB, unused=0x0)
frame #0: 0x00000002232e1aa8 libsystem_platform.dylib_platform_memmove + 88 libsystem_platform.dylib_platform_memmove:
-> 0x2232e1aa8 <+88>: stnp q0, q1, [x3]
0x2232e1aac <+92>: add x3, x3, #0x20
0x2232e1ab0 <+96>: ldnp q0, q1, [x1]
0x2232e1ab4 <+100>: add x1, x1, #0x20
Target 0: (Runner) stopped.

This is the more details
[✓] Flutter (Channel stable, 3.3.0, on macOS 12.2 21D49 darwin-x64, locale en-IN)
• Flutter version 3.3.0 on channel stable at /Documents/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision ffccd96b62 (7 days ago), 2022-08-29 17:28:57 -0700
• Engine revision 5e9e0e0aa8
• Dart version 2.18.0
• DevTools version 2.15.0
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0-rc1)
• Android SDK at /Library/Android/sdk
• Platform android-33, build-tools 31.0.0-rc1
• Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 13F100
• CocoaPods version 1.11.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2021.2)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
[✓] VS Code (version 1.70.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.46.0
[✓] Connected device (3 available)
• Iphone 11  (mobile) • 00008030-00114920349B402E • ios            • iOS 15.4.1 19E258
• macOS (desktop)     • macos                     • darwin-x64     • macOS 12.2 21D49 darwin-x64
• Chrome (web)        • chrome                    • web-javascript • Google Chrome 105.0.5195.52
[✓] HTTP Host Availability
• All required HTTP hosts are available

Hi @venkateshavenki, please provide a complete minimal reproducible code sample along with the full crash log and steps to reproduce.
Please also provide your pubspec.yaml & pubspec.lock
Also, what platform are you experiencing this crash on?
Thank you

Hi @danagbemava-nc, Thanks for the replay For displaying images I am using the below code. because of CachedNetworkImage app is crashing.
CachedNetworkImage(
fit: isCover! ? BoxFit.cover : BoxFit.fitHeight,
imageUrl:  url! ,
imageBuilder: (context, imageProvider) => Container(
decoration: BoxDecoration(
image: DecorationImage(image: imageProvider, fit: BoxFit.cover),
),
),
placeholder: (context, url) => const Center(
child: CircularProgressIndicator(
strokeWidth: 2.0,
)),
errorWidget: (context, url, error) => const Icon(Icons.error),
fadeOutDuration: const Duration(seconds: 0),
fadeInDuration: const Duration(seconds: 0),
);

Pubspec.yaml
name: Flutter project
description: A new Flutter project.
publish_to: 'none' # Remove this line if you wish to publish to pub.dev
version: 14.0.0+14
environment:
sdk: ">=2.12.0 <3.0.0"
dependencies:
flutter:
sdk: flutter
flutter_localizations:
sdk: flutter
The following adds the Cupertino Icons font to your application.

ref: a14d392
resolved-ref: a14d3927db216d606807e1f176bb6429532120f1
url: "https://github.com/jonataslaw/videoCompress.git"
source: git
version: "3.1.0"
video_player:
dependency: "direct main"
description:
name: video_player
url: "https://pub.dartlang.org"
source: hosted
version: "2.4.6"
video_player_android:
dependency: transitive
description:
name: video_player_android
url: "https://pub.dartlang.org"
source: hosted
version: "2.3.9"
video_player_avfoundation:
dependency: transitive
description:
name: video_player_avfoundation
url: "https://pub.dartlang.org"
source: hosted
version: "2.3.5"
video_player_platform_interface:
dependency: transitive
description:
name: video_player_platform_interface
url: "https://pub.dartlang.org"
source: hosted
version: "5.1.4"
video_player_web:
dependency: transitive
description:
name: video_player_web
url: "https://pub.dartlang.org"
source: hosted
version: "2.0.12"
video_thumbnail:
dependency: "direct main"
description:
name: video_thumbnail
url: "https://pub.dartlang.org"
source: hosted
version: "0.4.6"
visibility_detector:
dependency: "direct main"
description:
name: visibility_detector
url: "https://pub.dartlang.org"
source: hosted
version: "0.2.2"
wakelock:
dependency: transitive
description:
name: wakelock
url: "https://pub.dartlang.org"
source: hosted
version: "0.6.2"
wakelock_macos:
dependency: transitive
description:
name: wakelock_macos
url: "https://pub.dartlang.org"
source: hosted
version: "0.4.0"
wakelock_platform_interface:
dependency: transitive
description:
name: wakelock_platform_interface
url: "https://pub.dartlang.org"
source: hosted
version: "0.3.0"
wakelock_web:
dependency: transitive
description:
name: wakelock_web
url: "https://pub.dartlang.org"
source: hosted
version: "0.4.0"
wakelock_windows:
dependency: transitive
description:
name: wakelock_windows
url: "https://pub.dartlang.org"
source: hosted
version: "0.2.0"
watcher:
dependency: transitive
description:
name: watcher
url: "https://pub.dartlang.org"
source: hosted
version: "1.0.1"
web_socket_channel:
dependency: transitive
description:
name: web_socket_channel
url: "https://pub.dartlang.org"
source: hosted
version: "2.2.0"
source: git
version: "7.3.0"
win32:
dependency: transitive
description:
name: win32
url: "https://pub.dartlang.org"
source: hosted
version: "2.6.1"
xdg_directories:
dependency: transitive
description:
name: xdg_directories
url: "https://pub.dartlang.org"
source: hosted
version: "0.2.0+1"
xml:
dependency: transitive
description:
name: xml
url: "https://pub.dartlang.org"
source: hosted
version: "6.1.0"
yaml:
dependency: transitive
description:
name: yaml
url: "https://pub.dartlang.org"
source: hosted
version: "3.1.1"
sdks:
dart: ">=2.17.1 <3.0.0"
flutter: ">=3.0.0"

Hi @venkateshavenki, I believe this is a duplicate of https://github.com/flutter/flutter/issues/110715.
Kindly follow up in the linked issue for updates.
Please upvote the linked issue as well.
Closing from here as duplicate.
Thank you


[FEATURE]: Add a column.mapWith that applies to every query - $onSelect

Describe what you want
There are cases where a column might always be decoded in a certain way, so instead of adding .mapWith(myMapper) to every query, we can instead add $onSelect on table declaration that essentially adds mapWith to every query implicitly.
function decodeValue(value: number): string {
  // Decode logic here...
  return decoded;
}

const table = pgTable('table', {
  encoded: integer().notNull().$onSelect(decodeValue)
});

const result = await db.select().from(table); // { encoded: string }[]

This functionality can also complement the feature request mentioned in #2870, leading to Drizzle incorporating a "global" custom encoder and decoder for certain columns.

I can see and try to pick this enhancement


"RuntimeError: Attempting to deserialize object on a CUDA device" error in Automatic1111 v1.6, ControlNet v1.1.409 - Apple Silicon OSX

Using the SD1.5 IP-Adaptor models in v1.6.0 Automatic1111 environment (python: 3.10.13, torch: 2.0.1 ) with latest ControlNet on Apple ARM architecture generates a random image and produces console runtime error below. COMMANDLINE_ARGS="--skip-torch-cuda-test --upcast-sampling --no-half-vae --use-cpu interrogate".
2023-09-10 21:04:57,087 - ControlNet - STATUS - preprocessor resolution = 512
*** Error running process: /Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/extensions/sd-webui-controlnet/scripts/controlnet.py
    Traceback (most recent call last):
      File "/Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/modules/scripts.py", line 619, in process
        script.process(p, *script_args)
      File "/Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/extensions/sd-webui-controlnet/scripts/controlnet.py", line 977, in process
        self.controlnet_hack(p)
      File "/Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/extensions/sd-webui-controlnet/scripts/controlnet.py", line 966, in controlnet_hack
        self.controlnet_main_entry(p)
      File "/Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/extensions/sd-webui-controlnet/scripts/controlnet.py", line 808, in controlnet_main_entry
        detected_map, is_image = preprocessor(
      File "/Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/extensions/sd-webui-controlnet/scripts/utils.py", line 75, in decorated_func
        return cached_func(*args, **kwargs)
      File "/Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/extensions/sd-webui-controlnet/scripts/utils.py", line 63, in cached_func
        return func(*args, **kwargs)
      File "/Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/extensions/sd-webui-controlnet/scripts/global_state.py", line 35, in unified_preprocessor
        return preprocessor_modules[preprocessor_name](*args, **kwargs)
      File "/Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/extensions/sd-webui-controlnet/scripts/processor.py", line 350, in clip
        from annotator.clipvision import ClipVisionDetector
      File "/Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/extensions/sd-webui-controlnet/annotator/clipvision/__init__.py", line 81, in <module>
        clip_vision_h_uc = torch.load(clip_vision_h_uc)['uc']
      File "/Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/modules/safe.py", line 108, in load
        return load_with_extra(filename, *args, extra_handler=global_extra_handler, **kwargs)
      File "/Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/modules/safe.py", line 156, in load_with_extra
        return unsafe_torch_load(filename, *args, **kwargs)
      File "/Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/venv/lib/python3.10/site-packages/torch/serialization.py", line 809, in load
        return _load(opened_zipfile, map_location, pickle_module, **pickle_load_args)
      File "/Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/venv/lib/python3.10/site-packages/torch/serialization.py", line 1172, in _load
        result = unpickler.load()
      File<EMAIL_ADDRESS>line 1213, in load
        dispatch[key[0]](self)
      File<EMAIL_ADDRESS>line 1254, in load_binpersid
        self.append(self.persistent_load(pid))
      File "/Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/venv/lib/python3.10/site-packages/torch/serialization.py", line 1142, in persistent_load
        typed_storage = load_tensor(dtype, nbytes, key, _maybe_decode_ascii(location))
      File "/Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/venv/lib/python3.10/site-packages/torch/serialization.py", line 1116, in load_tensor
        wrap_storage=restore_location(storage, location),
      File "/Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/venv/lib/python3.10/site-packages/torch/serialization.py", line 217, in default_restore_location
        result = fn(storage, location)
      File "/Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/venv/lib/python3.10/site-packages/torch/serialization.py", line 182, in _cuda_deserialize
        device = validate_cuda_device(location)
      File "/Users/guestuser/Documents/Projects/StableDiffusion/stable-diffusion-webui/venv/lib/python3.10/site-packages/torch/serialization.py", line 166, in validate_cuda_device
        raise RuntimeError('Attempting to deserialize object on a CUDA '
    RuntimeError: Attempting to deserialize object on a CUDA device but torch.cuda.is_available() is False. If you are running on a CPU-only machine, please use torch.load with map_location=torch.device('cpu') to map your storages to the CPU.```

@raisindetre can you open a issue in https://github.com/Mikubill/sd-webui-controlnet

This is because you can't use CPU preprocessing on a Mac computer. To fix this, you can open the file located at /stable-diffusion-webui/extensions/sd-webui-controlnet/annotator/clipvision/init.py using a text editor or IDE (such as VSCode or PyCharm).
1、Navigate to line 81 and locate the line: clip_vision_h_uc = torch.load(clip_vision_h_uc)['uc'].
2、Modify this line to: clip_vision_h_uc = torch.load(clip_vision_h_uc, map_location=torch.device('cpu'))['uc'].
3、Save your changes and exit the editor.
4、Run your program again. This should prevent any CUDA-related errors.
However, even after this change, you may still encounter errors when generating images.

On Apple Silicon, change cpu to mps seems also works.

anyone solved this bug? I'm still suffering from the same problem.

This is because you can't use CPU preprocessing on a Mac computer. To fix this, you can open the file located at /stable-diffusion-webui/extensions/sd-webui-controlnet/annotator/clipvision/init.py using a text editor or IDE (such as VSCode or PyCharm).
1、Navigate to line 81 and locate the line: clip_vision_h_uc = torch.load(clip_vision_h_uc)['uc']. 2、Modify this line to: clip_vision_h_uc = torch.load(clip_vision_h_uc, map_location=torch.device('cpu'))['uc']. 3、Save your changes and exit the editor. 4、Run your program again. This should prevent any CUDA-related errors. However, even after this change, you may still encounter errors when generating images.

Still doesn't work.
TypeError: Cannot convert a MPS Tensor to float64 dtype as the MPS framework doesn't support float64. Please use float32 instead.

Still doesn't work. TypeError: Cannot convert a MPS Tensor to float64 dtype as the MPS framework doesn't support float64. Please use float32 instead.

Have you set "--no-half" in your start-up flags? Mine works with that set.

This is because you can't use CPU preprocessing on a Mac computer. To fix this, you can open the file located at /stable-diffusion-webui/extensions/sd-webui-controlnet/annotator/clipvision/init.py using a text editor or IDE (such as VSCode or PyCharm).
1、Navigate to line 81 and locate the line: clip_vision_h_uc = torch.load(clip_vision_h_uc)['uc']. 2、Modify this line to: clip_vision_h_uc = torch.load(clip_vision_h_uc, map_location=torch.device('cpu'))['uc']. 3、Save your changes and exit the editor. 4、Run your program again. This should prevent any CUDA-related errors. However, even after this change, you may still encounter errors when generating images.

THANK YOU! Work here after this fix! (rx6650xt)

Still doesn't work. TypeError: Cannot convert a MPS Tensor to float64 dtype as the MPS framework doesn't support float64. Please use float32 instead.

Have you set "--no-half" in your start-up flags? Mine works with that set.

I tried it today. It works！ Thx ；）

This line change "clip_vision_h_uc = torch.load(clip_vision_h_uc, map_location=torch.device('cpu'))['uc']." and "--no-half" fixed the problem for me on Mac Studio M2 Ultra. All ip-adapters functional!
Thanks!

setting --no-half and that line change does fix the issue, HOWEVER performance when setting --no-half drops about 25% (1:50 -> 2:26 for same image), comparing same image with flag set and not set control net not enabled.


How can Ethereum attack to DAO?

Hi
I'm learning to perform the DAO attack on my test blockchain. However, I don't know how to avoid using up the gas so the transaction is always reverted.
Here are the contracts:
pragma solidity ^0.8.9;
contract Bank{
// put money to the bank
function deposit(){}
function withdraw() public {
msg.sender.call.value(10 ether)("");
}
}
contract Attack{
// setter is omitted
Bank public bank;
function () external payable {
attack();
}
function attack() public {
bank.withdraw();
}
}

Hi, I believe you are rolling about different DAO: The DAO (project on Ethereum).
This repository assigned to DAO IPCI, another project not affiliated with Ethereum and the DAO. Just a form of organization makes prefix DAO.


Google Bard Encountered Error

Hi I've seen this issue on this project before but they seemed to be all closed
Google Bard encountered an error: b')]}'\n\n38\n[["wrb.fr",null,null,null,null,[8]]]\n54\n[["di",82],["af.httprm",83,"6657274337850409787",1]]\n25\n[["e",4,null,null,129]]\n'
Thank you very much this project is amazing.

How did you fix it?

i have this issue, how can I fix that?