Liquidity is calculated wrong

I analyzed jetton master addresses on your platform and the amount of ton accumulated there is far more than the trading fees, which are 0.3 ton per sell, even if you consider 0.3 trading fee for buys it is still a lot more, there is just 9000 tons for example in this address:
https://tonviewer.com/EQDD21KQbeC4wIKChGisYQ1_LHRED7qfT6OMMoUESngAd3ne
you can see even after launching on dedust there is 96 tons left on the jetton master address,  the fees are seen when selling which is around 30 times --> 9 tons, even if you consider buys and sells together it would be around 100 txs --> 30 ton
these funds are accumulated because there is certainly a problem in your algorithm calculating the remaining liquidity which you can see on every coin for example:
https://tonviewer.com/EQBWqEJJGyeKBQl3x1VtIJAnrwVvnHFUAhiLoymlEbfNzcKj
you can see 7.5 tons left on the address, 18 sells --> 6 tons --> actuall liquidty=1.5 tons, but in the app it says 0.93 ton liquidity left
I contacted support multiple times and they said these are fees which clearly are not

i the file GasPumpJetton.ts it is clearly stated that there is only a 0.3 fee for sell which you stated most of it will be refunded

Hi!
Thank you for a detailed question with examples.
My initial guess is that some small amount of TON gas is accumulated on contract in addition to commissions. We'll check what really happens and why

Yes if there is a problem it can be critical, please take this in account carefully, I have an excel sheet extracted that I can send you of 10000 tokens like these with total amount of 9000 ton

I missed one thing. When token is launched on DeDust we take fixed 50 TON commission (just like pump.fun) and you can see it on the contract as addition to balance.

most of the tokens i'm talking about are tokens that never made it to dedust and most of them everybody sold, but the difference in the wallet balance is seen there too and it's a lot

maybe a portion of this 50 ton comission you are talking about is applied every time somebody buys and sells, this 50 ton should be taken only after the token launched on dedust.

I sent the excel file to your email

Hi any updates on the matter?

uhm bro they take commitions from those .3 ton

This is not about 0.3 TON, this is about funds taken more than that

i didn't really get it then

Thank so much brother,i'm very support all produk and all project..if me wrong I so very sorry ...please help me so sorry..🧯🙏🙏👍

UQBOwGb9Nr55ev-ewf28h7ctRpJnmenJ7AF05U1MuqJ7FGdi, my ton wallet...please help me comision..because i want to make gas pump meme coin..i don't have coin to make..

get the fuck outta here


Re-enable some tests

I guess now the question is do I backport the C# changes to 3.4...

Unless there are bugfiexs or some pre-3.5 related functionality, I see no reason to backport them.


No BitString support?

Hi!
Are you planning on adding BitString support to the library anytime soon?

No, but a pull request might be accepted. :)


RSpec

Model

[ ] Log
[ ] Message
[ ] Notice
[ ] Score
[x] Sheet
[ ] Static
[x] User

Controller

[ ] Admins
[ ] Logs
[ ] Recommends
[ ] Rivals
[ ] Scores
[ ] Sheets
[ ] Welcomes

Views

[ ] admins
[ ] devise
[ ] logs
[ ] notice_mail
[ ] recommends
[ ] rivals
[ ] scores
[ ] sheets
[ ] welcomes

Library

[ ] IRT
[ ] Scrape::Maneger
[ ] Scrape::Official

Travis

[x] SetUp
[ ] bundle cache

数が大きすぎるので細かくまとめて作りなおす．


Added new section on transform jobs

Added new section to Index Management about transform jobs.
Check List

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

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Hey @keithhc2, if you're happy here, feel free to merge in and iterate over time.


source register operand missed in vid.v instruction

Add missing vs2 in vid.v example
Signed-off-by: Weiwei Wang<EMAIL_ADDRESS>

This commit already exists in pull request of #181, so close it.

There is no vs2 operand to the vid.v instruction, so this change is incorrect.
Unfortunately, this was merged into pull request for different fix in #181.  Will commit the other pull request, then undo this change.


pyperclip 동작 이상

docker 로 실행시 pyperclip 정상 동작 하지 않음.
회피 루틴이 필요.
Traceback (most recent call last):
  File "/usr/bin/translate", line 10, in <module>
    sys.exit(main())
  File "/usr/lib/python3.6/site-packages/translate/translate.py", line 24, in main
    text = [pyperclip.paste()]
  File "/usr/lib/python3.6/site-packages/pyperclip/__init__.py", line 638, in lazy_load_stub_paste
    return paste()
  File "/usr/lib/python3.6/site-packages/pyperclip/__init__.py", line 301, in __call__
    raise PyperclipException(EXCEPT_MSG)
pyperclip.PyperclipException:
    Pyperclip could not find a copy/paste mechanism for your system.
    For more information, please visit https://pyperclip.readthedocs.io/en/latest/introduction.html#not-implemented-error

Any way, pyperclip.PyperclipException This should be an exception for this part.
e.g)
try:
    ...
except pyperclip.PyperclipException:
    ...

Thank you :)

Anyway, This problem is heavy issue to me. I didn't know that pyperclip is not working on docker services. Some of modules I wrote in work, using pyperclip a lot.
I think working not on GUI environment is one of reason? I should check it. thx.

according to the error message, if there is no mechanisms of handling mouse. Working on shell in docker is not able to handle mouse. It's required to install one of the copy/paste mechanisms.
sudo apt-get install xsel to install the xsel utility.
sudo apt-get install xclip to install the xclip utility.
pip install gtk to install the gtk Python module.
pip install PyQt4 to install the PyQt4 Python module.

The goal of this program is to translate. Using clipboard is one of ability in the program.
So, we should test before using clipboard, If user try to use clipboard  we catch pyperclip.PyperclipException and then we just print the notice.

README.md 및 도움말 명령어 출력 변경

There are version issues. It is an issue that only happens on the docker.
$ pip3 install gtk
Collecting gtk
  Could not find a version that satisfies the requirement gtk (from versions: )
No matching distribution found for gtk


lxc rename VM on ceph fails to rename both ceph images  lxd 4.0.4

andriusjurkus@blazar-linux:~$ lxc rename w10-management w10-managements
Error: Failed to run: rbd --id lxd --cluster ceph --pool ceph-hdd map virtual-machine_w10-managements: rbd: sysfs write failed
rbd: map failed: (2) No such file or directory

It leaves like this
virtual-machine_w10-management
virtual-machine_w10-managements.block
I have to manually rename on ceph virtual-machine_w10-management -> virtual-machine_w10-managements

Confirmed, looking into this now.

Confirmed, looking into this now.


Link is not functional

bzz445  wants to merge 1 commit into alexsuter:master from bzz445:master???

Thank you. At the moment I have no time, i will check that within 2 weeks. sorry.

Ok! Pay attention, he did change the name attribute arrTags to tags, but the fix is fully functional. Sorry my poor english =).
Thanks for great work!

@cerqueiranews The merge is finally done.
The API is broken - use tags instead of arrTags now. I will make some improvments these days and will release version 2.0.0!

Version 2.0.1 is released.

Thanks!
Diego Muniz Cerqueira
Skype: cerqueiranews
MSN<EMAIL_ADDRESS>Gtalk<EMAIL_ADDRESS>Celular: 21 7101-6062
2016-01-21 10:35 GMT-02:00 Alex<EMAIL_ADDRESS>
Version 2.0.1 is released.
—
Reply to this email directly or view it on GitHub
https://github.com/alexsuter/yii2-tag-cloud/issues/2#issuecomment-173557948
.


Gradle setup issue Can't generate proxy for class

Hi, I'm getting the following problem at setup:
A problem occurred evaluating root project 'xxxxx'.

Failed to apply plugin [id 'com.devskiller.jpa2ddl']
Could not generate a proxy class for class com.devskiller.jpa2ddl.GeneratePluginExtension.

I'm using a similar setup script from the one in the readme:
jpa2ddl {
    packages = ['some.package.domain', 'somepackage.security.domain']
    outputPath = '${basedir}/src/main/resources/migrations/'
    jpaProperties = [
		property {
			name = 'hibernate.dialect'
			value = 'org.hibernate.dialect.H2Dialect'
		}
    ]
    action = 'UPDATE'
}

Hi,
which Gradle version do you use? Any specific plugins or configuration?

https://pastebin.com/iARPTzzH
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=icon
Virus-free.
www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=link
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
2018-03-14 10:07 GMT-03:00 Jakub Kubryński<EMAIL_ADDRESS>
Hi,
which Gradle version do you use? Any specific plugins or configuration?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Devskiller/jpa2ddl/issues/18#issuecomment-373014188,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUb_S1gPQE0lMBnnjvyhir3-qoBFg3Iks5teRYSgaJpZM4SqYtM
.

--
Lucas Montenegro Carvalhaes
May the sawce be with you

And which Gradle version?

Sorry for the delay:
$ ./gradlew --version

Gradle 3.5.1
Build time:   2017-06-16 14:36:27 UTC
Revision:     d4c3bb4eac74bd0a3c70a0d213709e484193e251
Groovy:       2.4.10
Ant:          Apache Ant(TM) version 1.9.6 compiled on June 29 2015
JVM:          1.8.0_92 (Oracle Corporation 25.92-b14)
OS:           Windows 10 10.0 amd64
2018-03-15 8:16 GMT-03:00 Jakub Kubryński<EMAIL_ADDRESS>
And which Gradle version?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Devskiller/jpa2ddl/issues/18#issuecomment-373342328,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUb_cb0_7GhwlnaYcQ7aLTMocR6tzNPks5tek2egaJpZM4SqYtM
.

--
Lucas Montenegro Carvalhaes
May the sawce be with you

Could you please try with the latest version of Gradle?

I'll try but later. I'll report as soon as I have results.
2018-03-15 9:49 GMT-03:00 Jakub Kubryński<EMAIL_ADDRESS>
Could you please try with the latest version of Gradle?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Devskiller/jpa2ddl/issues/18#issuecomment-373364530,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUb_daDMZWkoN_Y97Y1n1h6vCFf5XD2ks5temM_gaJpZM4SqYtM
.

--
Lucas Montenegro Carvalhaes
May the sawce be with you

Using gradle 4.6 I get:
Caused by: org.codehaus.groovy.runtime.typehandling.GroovyCastException: Cannot cast object '${basedir}/src/main/resources/migrations/' with class 'java.lang.String' to class 'java.io.File'

Changing the output path declaration to:
outputPath = file('${basedir}/src/main/resources/migrations/')
causes another error:

What went wrong:
A problem occurred evaluating root project 'IdeasApps'.

Could not find method property() for arguments [build_f47zftxyidw5z7cptwtrohxyx$_run_closure4$_closure11@6b68695c] on object of type com.devskiller.jpa2ddl.GeneratePluginExtension.

I've just checked with Gradle 4.6 and 4.7 and it works without any errors. Could you please provide me with the sample project to reproduce the issue?

Closing, as there was no action from the reporter. Feel free if it still occurs


mate.mate-applets: 1.28.0 -> 1.28.1

https://github.com/mate-desktop/mate-applets/compare/v1.28.0...v1.28.1
This replaced GtkSourceView with GtkTextView.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg eval


Stream BatchSize property ignored in SAM template

Hi,
it seems that  BatchSize specified in SAM template for DynamoDB streams is ignored.
LambdaFunction:
      ......
      Events:
        DynamoDBNewItemEvent:
          Type: DynamoDB
          Properties:
            Stream:
              "Fn::GetAtt": [ "TableFileImport", "StreamArn" ]
            StartingPosition: "LATEST"
            BatchSize: 10

I have compared the original template with processed one in CloudFormation and there is no BatchSize properties in the latter.
The stream gets default BatchSize=100
The similar issue was posted on AWS Forum but for Kinesis Streams See there

Thanks for reporting this. Yeah, this is a bug with both Kinesis & DynamoDB streams. We will get this fixed soon.

This should be fixed. Can you verify?

thanks for fixing this!
I can confirm that at least for DynamoDB streams it is fixed


Mojo version?

https://github.com/tairov/llama2.mojo
Imagine 250x speed on the original...

Imagine 250x speed on the original...

Probably you've noticed, this original means llama2.py, not llama2.c,
I am interested in Mojo, though it requires minimum 8GiB RAM only for SDK.

The C version is faster when using multi-threading :fire:


Mark function GetTimestamp of ConsensusState interface as deprecated

This function is not used outside of the light client implementations, and therefore it doesn't need to be an interface function.

Mark function GetTimestamp of ConsensusState interface as deprecated.
Signal accordingly in documentation.

Completed in https://github.com/cosmos/ibc-go/pull/5353. Removal of the function to be done in https://github.com/cosmos/ibc-go/issues/5993.


Overlapping sidebar links

Describe the bug
In the current version, the hyperlinks of the sidebar overlap with the center text. Therefore, whenever I try to
To Reproduce

Go to the example webpage on the PC (https://academicpages.github.io/)
Click on the last link of the first paragraph of the center text: "which uses this Github repository"

Expected behavior
Instead of going to the github repository, which is the hyperlink target in the center text, the email hyperlink of the sidebar is activated instead.
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser: Firefox
Version: 124.01 (64-bit)

Check this thread: #842
Go to line 32 in _sass\_sidebar.css where you have:
@include breakpoint($x-large) {
    padding-right: 0;
}

and write a maximum width like this:
@include breakpoint($x-large) {
    max-width: 250px;
    padding-right: 0;
}


DMMF.Model.description trims every lines

Bug description
When user writes a documentation comments on a specific model (or field), and gets the description comments through DMMF.Model.description API, every lines are trimmed (String.trim()).
As I'm developing third party library prisma-markdown and it utilizes the DMMF.Model.description API, some users are often reporting the String.trim() bug to me assuming that such bug is occured by me.
Therefore, I convey those bug reportings to here. Also, hope to be fixed this issue, too.

https://github.com/prisma/prisma/issues/20756

How to reproduce
Prisma Schema File
/// Product composition information handled in the sale snapshot.
///
/// `shopping_sale_snapshot_units` is an entity that embodies the
/// "individual product" information handled in the
/// {@link shopping_sale_snapshots sale snapshot}.
///
/// For reference, the reason why `shopping_sale_snapshot_units` is separated
/// from {@link shopping_sale_snapshots} by an algebraic relationship of
/// 1: N is because there are often cases where multiple products are sold
/// in one listing. This is the case with so-called "bundled products".
///
/// - Bundle from regular product (laptop set)
///   - main body
///   - keyboard
///   - mouse
///   - Apple Care (Free A/S Voucher)
///
/// And again, `shopping_sale_snapshot_units` does not in itself refer to
/// the final {@link shopping_sale_snapshot_unit_stocks stock} that the
/// customer will purchase.
/// The {@link shopping_sale_snapshot_unit_stocks final stock} can be
/// found only after selecting all given
/// {@link shopping_sale_snapshot_unit_options options} and their
/// {@link shopping_sale_snapshot_unit_option_candidates candidate} values.
///
/// For example, even if you buy a laptop, the
/// {@link shopping_sale_snapshot_unit_stocks final stocks} are determined
/// only after selecting all the
/// {@link shopping_sale_snapshot_unit_options options} (CPU / RAM / SSD), etc.
///
/// @namespace Sales
/// @erd Carts
/// @author Samchon
model shopping_sale_snapshot_units {

DMMF.Model.description returns

console.log(model.documentation);

Product composition information handled in the sale snapshot.

`shopping_sale_snapshot_units` is an entity that embodies the
"individual product" information handled in the
{@link shopping_sale_snapshots sale snapshot}.

For reference, the reason why `shopping_sale_snapshot_units` is separated
from {@link shopping_sale_snapshots} by an algebraic relationship of
1: N is because there are often cases where multiple products are sold
in one listing. This is the case with so-called "bundled products".

- Bundle from regular product (laptop set)
- main body
- keyboard
- mouse
- Apple Care (Free A/S Voucher)

And again, `shopping_sale_snapshot_units` does not in itself refer to
the final {@link shopping_sale_snapshot_unit_stocks stock} that the
customer will purchase.
The {@link shopping_sale_snapshot_unit_stocks final stock} can be
found only after selecting all given
{@link shopping_sale_snapshot_unit_options options} and their
{@link shopping_sale_snapshot_unit_option_candidates candidate} values.

For example, even if you buy a laptop, the
{@link shopping_sale_snapshot_unit_stocks final stocks} are determined
only after selecting all the
{@link shopping_sale_snapshot_unit_options options} (CPU / RAM / SSD), etc.

@namespace Sales
@erd Carts
@author Samchon

Expected behavior
Do not trim every lines, but just keep original space characters what user have written please.
Environment & setup

OS: Windows
Database: Postgres
Node.js version: v18.17.1

Prisma Version
prisma                  : 5.4.2
@prisma/client          : 5.4.2
Current platform        : windows
Query Engine (Node-API) : libquery-engine ac9d7041ed77bcc8a8dbd2ab6616b39013829574 (at<EMAIL_ADDRESS>Schema Engine           : schema-engine-cli ac9d7041ed77bcc8a8dbd2ab6616b39013829574 (at<EMAIL_ADDRESS>Schema Wasm             : @prisma/prisma-schema-wasm 5.4.1-2.ac9d7041ed77bcc8a8dbd2ab6616b39013829574
Default Engines Hash    : ac9d7041ed77bcc8a8dbd2ab6616b39013829574
Studio                  : 0.494.0

### Tasks
- [ ] https://github.com/prisma/prisma/issues/18561

Hi @samchon, just to clarify, would you like us to remove just the ///  characters that precede every model comment line, rather than Removing /// + any number of space that precedes an actual comment line? This is what I understood from this smaller diff, where the markdown list is flattened:
/// - Bundle from regular product (laptop set)
///   - main body
///   - keyboard
///   - mouse
///   - Apple Care (Free A/S Voucher)

- Bundle from regular product (laptop set)
- main body
- keyboard
- mouse
- Apple Care (Free A/S Voucher)

What do you want is doing like this?
It occurs prisma compilation error as you know
Prisma schema loaded from schema.prisma
Error: Prisma schema validation - (get-config wasm)
Error code: P1012
error: Error validating: This line is invalid. It does not start with any known Prisma schema keyword.
  -->  schema.prisma:1321
   |
1320 | ///
1321 | - Bundle from regular product (laptop set)
1322 | - main body
   |
error: Error validating: This line is invalid. It does not start with any known Prisma schema keyword.
  -->  schema.prisma:1322
   |
1321 | - Bundle from regular product (laptop set)
1322 | - main body
1323 | - keyboard
   |
error: Error validating: This line is invalid. It does not start with any known Prisma schema keyword.
  -->  schema.prisma:1323
   |
1322 | - main body
1323 | - keyboard
1324 | - mouse
   |
error: Error validating: This line is invalid. It does not start with any known Prisma schema keyword.
  -->  schema.prisma:1324
   |
1323 | - keyboard
1324 | - mouse
1325 | - Apple Care (Free A/S Voucher)
   |
error: Error validating: This line is invalid. It does not start with any known Prisma schema keyword.
  -->  schema.prisma:1325
   |
1324 | - mouse
1325 | - Apple Care (Free A/S Voucher)
1326 | ///
   |

Validation Error Count: 5

@samchon I have updated my previous comment to clarify that the top snippet referred to the schema.prisma, and the bottom one referred to DMMF.model.description, following the same convention set by your issue description :)

Yes, what that's I want is exacly.
Removes only ///  word instead of removing the spaces after that.

Very reasonable suggestion @samchon, we should totally not mess with the white space at all an only remove the ///  and nothing else.


Feature/reset password

Reset a password by mail confirmation.
In the case of a user have forgotten his password, he can reset it by verifying his mail address.
The email verification is done by sending a mail with a  link to a reset-password page in our site, containing a jwt-token with the user's unique credentials, and 15 minutes expiration time. Upon clicking that link, decryption of that token will verify and authenticate the user. Then he will be presented with a reset-password form

Codecov Report

Merging #289 (ad01dff) into develop (8cd3a6e) will increase coverage by 0.04%.
The diff coverage is 99.42%.

@@             Coverage Diff             @@
##           develop     #289      +/-   ##
===========================================
+ Coverage    99.04%   99.09%   +0.04%
===========================================
  Files           52       62      +10
  Lines         2409     2756     +347
===========================================
+ Hits          2386     2731     +345
- Misses          23       25       +2

Impacted Files
Coverage Δ

app/routers/login.py
96.55% <96.55%> (ø)

app/internal/security/ouath2.py
98.46% <98.46%> (ø)

app/database/models.py
96.38% <100.00%> (+0.08%)
:arrow_up:

app/database/schemas.py
100.00% <100.00%> (ø)

app/internal/security/dependancies.py
100.00% <100.00%> (ø)

app/internal/security/reset_password.py
100.00% <100.00%> (ø)

app/internal/security/schema.py
100.00% <100.00%> (ø)

app/internal/user.py
100.00% <100.00%> (ø)

app/main.py
96.96% <100.00%> (+0.30%)
:arrow_up:

app/routers/logout.py
100.00% <100.00%> (ø)

... and 13 more

Continue to review full report at Codecov.

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

WIP! I Forgot to add at the commit name.
Few things will be updated in this PR once feature\login (current_user) is merged.

Still WIP?

Still WIP?

The main logic and functionality is ok. however,  the function check_jwt_token as well as its returned value are in use here, and they both had changed at feature/login PR (current_user), which is supposed to be be merged. So i guess its better to update it after the merging of feature/login, when those changes will be updated on the develop branch.

Looks like great start!
Currently, It's hard to understand where the authentication of the user happens.
Why are we creating a JWT token for someone who doesn't logged in yet?

The jwt token is sent by mail to the user. It's purpose is to verify that the user is using the mail address he registered with.


TS7022 (implicity 'any' type) popping up in a random scenario

🔎 Search Terms
TS7022
TS7022: 'result' implicitly has type 'any' because it does not have a type annotation and is referenced directly or indirectly in its own initializer.
🕗 Version & Regression Information
Using typescript 5.5.4
⏯ Playground Link
TS Playground
💻 Code
export declare class Client {
  send(x: string | undefined): string
  send(x: string | undefined, cb: () => void): void
  // ^-- error goes away if overload is removed
}

const c = {} as Client
let x: string | undefined = "";
// ^-- error goes away if '= ""' is removed
// ^-- error goes away if '| undefined' is removed

while (true) {
  // ^-- error goes away the code isn't in a loop

  const result = c.send(x)
  // ^-- error TS7022: 'result' implicitly has type 'any' because it does not have a type annotation and is referenced directly or indirectly in its own initializer.

  x = result
  // ^-- error goes away if we add another line x = ""
}

🙁 Actual behavior
error TS7022: 'result' implicitly has type 'any' because it does not have a type annotation and is referenced directly or indirectly in its own initializer.
🙂 Expected behavior
result is of type string
Additional information about the issue
No response

Did you mix up actual and expected behavior? It's the first time I read someone actually expecting that error.

Did you mix up actual and expected behavior? It's the first time I read someone actually expecting that error.

yes - fixed

Then it's essentially a duplicate of #50696 and many others.

result really does have a circular dependency on itself

result really does have a circular dependency on itself

Hrmm... Well, I can see how result being fed back into x... which is fed into the function which is called to initialize result could be seen that way
But ...really?
c.send has only a single parameter, which can match only a single function, which definitely returns string.  You'd think it'd be safe to realize "hey, this is a string".
But if there's technical specs/challenges to unravelling it... so be it.  Then I should have made it a "feature request" :)

Resolving the overload needs the current CFG narrowed types of all parameters, ref https://github.com/microsoft/TypeScript/issues/35638#issuecomment-566158599.  Figuring out that the overload can be resolved without the narrowing in this specific case isn't something that can be easily applied in general.

I'm super curious...
Change
let x: string | undefined = "";
to
let x: string | undefined;

And the entire problem goes away.  It can magically resolve the overload, and do whatever other sorcery is required, all of a sudden.

I'm super curious...
Change
let x: string | undefined = "";
to
let x: string | undefined;

And the entire problem goes away. It can magically resolve the overload, and do whatever other sorcery is required, all of a sudden.

In one case the type of x is string, in the other case it's string | undefined.

I'm super curious...
Change
let x: string | undefined = "";
to
let x: string | undefined;

And the entire problem goes away. It can magically resolve the overload, and do whatever other sorcery is required, all of a sudden.

In one case the type of x is string, in the other case it's string | undefined.

Interesting.
let x: string | undefined = "" as string | undefined;
or
x = result as string | undefined
Those both make the problem go away
but
x = (result as string | undefined)?.toString()
makes the problem come back again
...Anyway, it's clearly just something that'll have to be worked around for now.


Feature overlap with name tag renaming w/o an anvil

Right clicking and shift right clicking a nametag opens up two separate interfaces for renaming the nametag, causing redundancy.

Disabling the Easy Anvils setting edit_name_tags_no_anvil should fix this issue.


[Bug] The User property of the v2/time_entries GET request should only have name and not first_name and last_name

Describe the bug
The FirstName and LastName properties in UserSummary are always null in the context of the GET v2/time_entries response.
What is the expected behavior?
The API currently only returns the full name in a single name property.
"user": {
  "id": 123456,
  "name": "Jean-Philippe Levesque"
},

The C# object mapping should reflect the API response.
Other information
I tested the API on the 2024-03-25.

Thanks for raising @jeanplevesque, I will take a look at this and get a fix out.


Adding CMake work-around for Ubuntu 18.04 and CUDA > 9.2.

Description
Avoids users of CUDA 10/10.1 on Ubuntu 18.04 from having to upgrade their system CMake just to build MXNET. CMake change to fix this issue was made in 3.12.3 here: https://gitlab.kitware.com/cmake/cmake/commit/81e73b7240e3b11b454f01f21bab85b4aa95d6af.
Checklist
Essentials

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes) - I would consider this a "tiny change".
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
No additional tests added as many CI and unit tests run with this flag both enabled and disabled.
[x] Code is well-documented
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[x] Adds conditional work-around to CMakeLists.txt to fix incompatibility between CUDA > 9.1 and CMake < 3.12.3.
[x] Tested in Docker with Ubuntu 16.04/CUDA 9.0/CuDNN 7.6.2 and Ubuntu 18.04/CUDA 10.0/CuDNN 7.6.2.

@JWhitleyAStuff Thanks for your contributions! .
@mxnet-label-bot Add [Cmake, Cuda, pr-awaiting-review]

@szha @rahul003 Can you please review this PR? Thanks!

Thanks @JWhitleyWork. We now always require CMake 3.13 or higher. It's straightforward to install up-to-date cmake via pip. Thus I'm closing this PR.


Change /sys bind mount from ro to rw

When the io.balena.features.sysfs label is used to bind mount /sys into the container, currently the mount is created as read-only. It should be created as read-write to permit access to device peripherals without requiring a fully privileged container.
Further discussion here: https://www.flowdock.com/app/rulemotion/balenalabs/threads/vpx5ucMSw_Vcf9ye7BlOY1ANLgU

Changing the docker behavior here has some implications for what "unprivileged" means as full rw access to sysfs gives you quite a lot of access to the host system.
cgroupfs is under /sys/fs/cgroup, /sys/kernel/* exposes kernel info, etc.

The io.balena.features.sysfs label was introduced in balena-superviosr version 10.8.0 and will bind mount /sys into the container, which should give full access to it.


new version of watchdog package

The problem with Raspberry restarts (with HV=kvm) is hidden in the watchdog package. Updating the package to the latest version fixes the problem for me. However, it is necessary to understand if there are any problems of a different kind with 5.16 version.
Signed-off-by: Petr Fedchenkov<EMAIL_ADDRESS>

Hey @giggsoff -- thanks for the patch, but you'll have to tell us way more than "is hidden in the watchdog package" to fully describe the problems you're seeing with the current watchdog package. Please provide a full description of the problems and how you'd expect an updated version of watchdog to fix them.


Sprite unshaded CanvasItemMaterial option only works as a direct child of Node2D.

Godot version 3.1.1 stable.official
Linux Mint 19.1 (Ubuntu) Kernel Linux 4.15.0-65-generic x86_64. Nividia GTX 950
It seems the unshaded light mode option of a CanvasItemMaterial is not working correctly. Sprites with this unshaded material option will only work as a direct child of Node2D. In this image I have the sprite as a a child of Node2D - TileMap4. The sprite is uneffected by the unshaded material option.

In this image the same sprite is now a child of Node2D and the unshaded material option works as expected.

This issue should be very easy to reproduce. It should be noted that I am using 2D lighting in this example.
New to Godot .. but this issue feels bug'ish. ;)

Using a RigidBody2D node as a parent for the sprite rather then TileMap gets the CanvasItemMaterial light mode option unshaded to work as expected as seen in this image. Perhaps this issue only effects the TileMap node.

Can anyone still reproduce this bug in Godot 3.2.3 or any later release?
If yes, please ensure that an up-to-date Minimal Reproduction Project (MRP) is included in this report (a MRP is a zipped Godot project with the minimal elements necessary to reliably trigger the bug). You can upload ZIP files in an issue comment with a drag and drop.


SRC Issue -- Nonmonotonic RC

Some hec-ras models creates nonmonotonic rating curves.

@CarsonPruitt-NOAA
@LauraKeys-NOAA
@JamesColl-NOAA

This issue is tied to the downstream boundary conditions and hopefully will be resolved in ras2fim v2. We will test the updated results for this issue to verify that everything looks good/improved.


fix OBS-studio DTLS failed

We found use OBS-studio WHIP cannot PeerConnected
package DTLS have problem
We found after this commit have this bug
This PR https://github.com/webrtc-rs/webrtc/pull/496 and this commit after https://github.com/webrtc-rs/webrtc/pull/496/commits/935283abc719b2c3a6925cd8bb90ffb3809bccc5
Finally we found the problem. But, I'm not sure this have bug
https://github.com/RustCrypto/utils/blob/master/block-padding/src/lib.rs#L176-L216
Maybe, webrtc-rs is using this library incorrectly
https://github.com/binbat/live777/issues/39
https://github.com/binbat/live777/issues/58

please fix formatting

@rainliu, can you comment on why https://github.com/webrtc-rs/webrtc/commit/935283abc719b2c3a6925cd8bb90ffb3809bccc5 changed the padding in the first place? I'm not sure I understand the reasoning and this was particularly hard to track down. Thank you!

dtls dependent crates upgrade to new version, however, incorrect usage of new APIs of those crates causes incompatible PADDING.
aes = "0.8"
cbc = { version = "0.1", features = [ "block-padding", "alloc"] }
ccm = "0.5"

I see, but how is PKCS7 relevant to DTLS? Does the RFC say somewhere that the padding schemes are equivalent?


fix errors in table deletion

it is possible to have tables deleted in metastore when scheduled jobs are already running and trying to create new versions in metastore, in such case we should not panic and crash the server

host memory manager should ignore deleted tables when reporting memory usage

Codecov Report

Merging #280 into master will decrease coverage by 0.14%.
The diff coverage is 69.51%.

@@            Coverage Diff             @@
##           master     #280      +/-   ##
==========================================
- Coverage      72%   71.85%   -0.15%
==========================================
  Files         166      166
  Lines       22885    22891       +6
==========================================
- Hits        16478    16448      -30
- Misses       5124     5166      +42
+ Partials     1283     1277       -6

Impacted Files
Coverage Δ

memstore/host_memory_manager.go
97.17% <0%> (ø)
:arrow_up:

broker/broker_schema_mutator.go
59.77% <0%> (ø)
:arrow_up:

controller/mutators/etcd/enum_mutator.go
79.01% <0%> (ø)
:arrow_up:

api/schema_handler.go
80.18% <100%> (ø)
:arrow_up:

controller/mutators/etcd/schema_mutator.go
63.58% <50%> (ø)
:arrow_up:

memstore/archiving.go
86.45% <64.28%> (-1.05%)
:arrow_down:

memstore/schema.go
86.66% <66.66%> (ø)
:arrow_up:

metastore/validator.go
83.62% <68%> (ø)
:arrow_up:

memstore/backfill.go
74.24% <76.92%> (-0.62%)
:arrow_down:

metastore/disk_metastore.go
74.41% <93.75%> (ø)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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


Skhd is not working with kitty

Skhd does not read the commands when kitty is focus, I have try somethings to debug and try to know what is really happening.
When I run skhd -V on kitty terminal  this is the output given and the program return is 1.
skhd: secure keyboard entry is enabled by (78685) 'kitty'! abort..
Does someone know what is happening or what could I do?

If I remember correctly, there's a simple checkmark in on of the Menubar items of Kitty. It just says "Secure Keyboard Entry" and removing the checkmark should hopefully fix your issue I think.


[1.4] Correct Binary Deployment for .NET8 Breaking Change Support

.NET8 introduced a conditional breaking change that influences how the runtime host looks for RID-specific assets. In NET8, support for non-portable RIDs (runtime identifiers specific to versions and distributions, such as win10-x86) is discontinued. The folder structure of the Microsoft.WindowsAppSDK NuGet is structured with assets marked using these non-portable RIDs:
The changes made to the CopyFilesToStagingDir.ps1 script along with the project template changes should help support the .NET8 scenario. The selection of the project template's RIDs is conditional and based on the TFM chosen by the user. If NET8 is chosen, the RIDs become portable, enabling the runtime host to locate the RID-specific assets that were previously unattainable.

We should cherry pick this change instead (future proof):
https://github.com/microsoft/WindowsAppSDK/pull/4012

Can you please deploy a package version with this fix ? Neither the last 1.4.231115000 nor 1.5.231202003-experimental1 contains it.
ty in advance

It's on the way - thanks for your patience


Code the buttons to take you to other pages

Allow the Home button to take you to the home page.

The FAQ page to take you to another webpage
etc etc

Populate the web pages with Lorem ipsum text

I'm taking this one

All done


Fix Tea detection

Change 'Tea with Sugar' to 'Tea'

should be fixed with https://github.com/melbaa/summarize_consumes/commit/0692cfcb0ec936157841c48450c6d5102dbf264e


OAIRepository: CFG_OAI_METADATA_FORMATS fix

FIX Fixes default configuration of CFG_OAI_METADATA_FORMATS,
correctly assigning the XSD and XML namespace to the corresponding
metadata prefix.

Signed-off-by: Samuele Kaplun<EMAIL_ADDRESS>

@Dziolas this is what you need to back port on SCOAP³ to fix the issue reported by your user.


Release (GA) timeline for OAuth2 bearer authentication

Hi,
Is there a timeline which you guys have, when using OAuth2 bearer authentication for securing a REST API will be released?

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

ID: 7cbe5bd8-f4fa-cfac-b3f1-ac25ae741dae
Version Independent ID: 7b7a29b9-b779-88a6-b53a-84b4a6cc1f9b
Content: Secure a Restful service in your Azure AD B2C - Azure AD B2C
Content Source: articles/active-directory-b2c/secure-rest-api.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@deeppr
Thanks for your feedback! We will investigate and update as appropriate.

Hi @deeppr,
We don't have that information yet, but you can check the release notes for the latest updates on this and they'll be posted as soon as the information is available.
If you have further questions feel free to post to Microsoft Q&A and we will gladly continue the discussion.


withId not working.

I'm having an issue with the withId model function.
I have just started using this so I may have misinterpreted something.
const getById = createSelector(
  orm,
  state => state.orm,
  (session, id) => {
    const user = session.User.withId(id + '');
    return user ? user.ref : null;
  }
);

const getWhere = createSelector(
  orm,
  state => state.orm,
  (session, constraints) => {
    const user = session.User.filter(constraints).first();
    return user ? user.ref : null;
  },
);

const getAll = createSelector(
  orm,
  state => state.orm,
  (session) => session.User.all().toRefArray(),
);

describe('Select user', () => {
  it('User id configured correctly!', () => {
    expect(ormData.User.idAttribute).toEqual('id');
  });
  it('Select all works!', () => {
    const users = getAll({ orm: ormData.state });
    expect(users.length).toEqual(3);
    expect(users[0].id).toEqual('1');
  });
  it('Select user 1 generally works!', () => {
    const user = getWhere({ orm: ormData.state }, { id: '1' });
    expect(user).not.toBeNull();
    expect(user.id).toEqual('1');
  });
  it('Select user 1 withId works!', () => {
    // Getting
    const user = getById({ orm: ormData.state }, '1');
    expect(user).not.toBeNull();
    expect(user.id).toEqual('1');
  });
});

Everything works until the last part, using the reducer that uses withId.
Anyway is the a bug, or am I doing something wrong? Perhaps it has something to do with the id's being integer strings?

Just as some extra info, if I use integers instead for the id's I search with, the outcome is still the same.

Ok ignore this, the id just isn't getting through :/

For anybody making the same mistake, this is to create a selector much like reselect where it is a based on combining selectors to make arguments for the final selector. So does not except arguments.
However what you can do is have the selector return a function(instead of just the selected state), that you can filter the result set on.
e.g.
const getById = createSelector(
  orm,
  state => state.orm,
  orm =>
    id => {
      const user = orm.User.withId(id);
      return user ? user.ref : null;
    }
);

so if I called the function getById  it would return a function with the resulting orm state in scope.
In my case I can use it as
const getByIdFilter = getById(state);//Get a filter with access to orm
const user = getByIdFilter(1);//Execute the filter with a passed argument

Added to this we can also memoize that function, so that the function doesn't re-execute if the argument is the same. i.e....
const getById = createSelector(
  orm,
  state => state.orm,
  orm => memoize(
    id => {
      const user = orm.User.withId(id);
      return user ? user.ref : null;
    }
  )
);

Further reading here
https://github.com/reduxjs/reselect/blob/master/README.md#q-how-do-i-create-a-selector-that-takes-an-argument

@relaxomatic Selectors are now composable and accept ID arguments with automatic caching based on re-reselect! Please try out v0.14.0-rc.0 :slightly_smiling_face:


[project details] Implement image carousel component

This workstream is related to implementing the image carousel feature on the project details pages.
Preliminary designs here.
Requirements

[ ] Update designs based on feedback for desktop and mobile. Requirements include (1) Map is always the first image (2) Carousel control cannot overlap the map image since the map is interactive (3) Need generic map icon to indicate first image in the image roll
[ ] Backend sanity DB changes to store images
[ ] Frontend dev to implement the design

Other Implementation Considerations

How is the order of the images determined?
Will need a way to indicate which image to use for project cards in the marketplace (NOTE: this is a future requirement that will not be implemented initially but mentioned now in case it impacts the backend design)

Updated design here

@sy-zygy @Atmosfearful see link above for review


Added actuator capability

SmartApps need to know the actuator capability so that they can interact with this device

Agreed.  I already have this change in my local copy.


Does not work within vnet/firewall/ private endpoint

[Enter feedback here]
This does not work within a private vnet. This makes azure cognitive search be completely public in order for this API extension endpoint to work. Please update documentation to make it clear that this only works with public internet access. And will cease to work if use case is needing to be within a vnet firewall situation.

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

ID: 6317af39-eff1-77b4-8680-9fc06f0d1fdd
Version Independent ID: 60284391-262c-807a-fb8f-9d7422b1926d
Content: Using your data with Azure OpenAI Service - Azure OpenAI
Content Source: articles/ai-services/openai/concepts/use-your-data.md
Service: azure-ai-openai
GitHub Login: @aahill
Microsoft Alias: aahi

@heyal
Thanks for your feedback! We will investigate and update as appropriate.

@heyal
Thank you for bringing this to our attention.
I've delegated this to content author @aahill, who will review it and offer their insightful opinions.

Thanks for the feedback, we've updated this article.
#please-close


How to disable cmp completion for a specific programming language?

I don't want to use cmp completion for python language because I'm using Kite engine for autocompletion. So I don't want the LSP complete, but if using cmp it will not display the Kite complete menu. I read your repo but cannot found something like this so I create a new issue. Sorry for my bad english.

Try the following.
au FileType python lua require("cmp").setup.buffer({completion={autocomplete=false}})

@BlueDruddigon
Please try it https://github.com/hrsh7th/nvim-cmp/wiki/Disable-cmp-by-filetype


Exceptions in algolia code break our code

I'm currently having the following exception thrown from algoliasearch-client-php/src/AlgoliaSearch/Client.php:504:
"AlgoliaSearch\AlgoliaException: Operations quota exceeded, change plan to get more Operations."
We are using a trial plan, it's quite possible I have reached my quota.
Though looking at the stack, this Client code was run by AlgoliaSearchDoctrineEventSubscriber.
This means that every time I use my database via doctrine and update an entity that is indexed in algolia, an exception can be raised by any part of this bundle code (AlgoliaException / \Exception ..). This is not acceptable: our server code will not work any more whenever an exception occurs.
Code in AlgoliaSearchDoctrineEventSubscriber should be within try / catch to avoid any exception bubbling up and breaking our code. I suggest you use the logger to write such exceptions in a log file.

@chriwup I am not sure catching the exception and putting it in the log should be directly included in the extension.
It would means that in case of bad credentials, curl not installed, quota exceeded, ..., you won't have any exception raised by the extension which not what we want.
The errors thrown by AlgoliaSearchDoctrineEventSubscriber occurs when you add/delete/update an object that is supposed to be sync by Algolia. I think it would be better to try catch for AlgoliaException in your own code if you want it to be safe for this kind of error.
What do you think ?

Search is not a critical component for most companies, if syncing fails it is sad but it should in no way prevent the rest of the program from working correctly. For instance if the algolia api server is down (as it happens to all websites on this planet), I don't want it to bring down my server too.
If curl is not installed or bad credentials are given, I suppose the developer doing an Algolia integration will find out that things don't work and a little documentation should point them towards their log file.
You might also add an algolia configuration to let the user of the bundle choose the expected behaviour : catch exceptions and log them or leave them bubbling up.

Side note: as we're dealing with a doctrine subscriber here, I see no way of catching exceptions thrown there on our side..

@chriwup Yes, the algolia configuration is a good idea (with error thrown by default).
Can you modify the pull request to handle it ?

I have updated the pull request.

@chriwup Thanks a lot, I am gonna test it and merge it asap

Fixed with #14


Removed trailing whitespace

Thank you. Just........Thank you.

We do try to stick to a unified style but we do not have strict rules on whitespace.
As such, we will not accept requests to conform to a particular style on a whim.
I am closing this, but that does not mean we are closed to suggestions, even on style.
(E.g. see discussion in #155)

Trailing whitespace is basically file noise. Removing it doesn't change the functionality of the scripts at all. One reason for removing it is it takes up unnecessary storage space. Another reason, especially for interpreted languages (which Lua is), is that parsing of unnecessary whitespace may affect the speed of compilation and execution. I understand not accepting this commit, but please do reconsider.
Lots of modern editors actually automatically remove trailing whitespace, as its almost never intentional. Editors that don't have it built in usually have extensions available to do the job:

Visual Studio: https://visualstudiogallery.msdn.microsoft.com/a204e29b-1778-4dae-affd-209bea658a59
Sublime Text: https://github.com/SublimeText/TrailingSpaces
Emacs: http://emacswiki.org/emacs/DeletingWhitespace
Atom: https://atom.io/packages/trailing-spaces

Related:

https://programmers.stackexchange.com/questions/121555/why-is-trailing-whitespace-a-big-deal
http://www.dinduks.com/why-are-trailing-whitespaces-bad/


Video link of WWDC 2015 Session 509 needs to be updated

For the life of me, I cannot find this video anywhere. It's as if it's been scrubbed from the internet. The next best thing I've found are the slides from the talk. https://devstreaming-cdn.apple.com/videos/wwdc/2015/5091mxk00t/509/509_seamless_linking_to_your_app.pdf

I strongly recommend watching the Apple WWDC 2015, session 509 - Seamless Linking to Your App to understand how the Universal Links work (and are setup).

Could someone with a link to the talk please update this?

Apple removes videos that no longer are best practices, so I guess it is not the best source any more.
Having said that you can find it in the Internet Archive - https://archive.org/details/wwdc2015videos


Zhaoyu/gpu type workaround

add GPUType scheduling workaround.

Coverage decreased (-0.01%) to 51.632% when pulling 44bc4821c4b0e4ee6ae13c3f8e628666067af78d on zhaoyu/gpu_type_workaround into 3e0a8bf53d5ac487705a3a6a03e1daf93250b86d on master.


TFFT recipes

TFFT Controller:

TFFT Casing:

TFFT Multi Hatch:

T1(LV):

T2(MV):

T3(HV):

T4(EV):

T5(IV):

T6(LuV):

T7(UV):

T8(UEV):

T9(UMV):

T10(UXV):

Here is the specific data of the storage blocks
https://github.com/GTNewHorizons/KekzTech/blob/master/src/main/java/common/tileentities/GTMTE_TFFT.java#L49-L60

Why totem of undying (or w/e knock off version we have of it)?
Also no more fluid cores?

we can change recipes in a second pr if some need adjustments

Why totem of undying (or w/e knock off version we have of it)?

The cost of t10 is 0, basically just give it a random battery or something, tfft also uses a lot of eio materials, I guess totemic capacitor is interesting.

Adding totem of undying seems kinda tacky to me, especially as it'll just become yet another single-user item™

Just to clarify, we don't have totem of undying, it's a capacitor from eio


Update quickcheck to 1.0

Closes #148
Closes #157

r? @JohnTitor

r? @JohnTitor

Done, although I think dependabot is smart enough to close pending PRs for deps that are not required anymore

Done, although I think dependabot is smart enough to close pending PRs for deps that are not required anymore

Ah, I forgot the bot can detect it.
@bors r+

Ah, I forgot the bot can detect it.
@bors r+

:pushpin: Commit 438623d36196e8cb24107d9fc8122c5b25c80efc has been approved by JohnTitor

:pushpin: Commit 438623d36196e8cb24107d9fc8122c5b25c80efc has been approved by JohnTitor

:hourglass: Testing commit 438623d36196e8cb24107d9fc8122c5b25c80efc with merge a223bf31011546df735b9eff9b3b3379a1a0c368...

:hourglass: Testing commit 438623d36196e8cb24107d9fc8122c5b25c80efc with merge a223bf31011546df735b9eff9b3b3379a1a0c368...

:sunny: Test successful - checks-actions
Approved by: JohnTitor
Pushing a223bf31011546df735b9eff9b3b3379a1a0c368 to master...

:sunny: Test successful - checks-actions
Approved by: JohnTitor
Pushing a223bf31011546df735b9eff9b3b3379a1a0c368 to master...


Update es5.d.ts

Objects.keys() does not take null or undefined
> Object.keys(undefined)
TypeError: Cannot convert undefined or null to object
> Object.keys(null)
TypeError: Cannot convert undefined or null to object

[x] You've signed the CLA
[x] There are new or updated unit tests validating the change

What about object?

What about object?

object doesn't work with true and false

Related issue #17273

I think this looks like a good change, but would like to get some additional input. @sandersn, do you see any issues with this change?

Looks good to me.


springboot testing cannot check pact if valid,what can i do?whether a wrong config with my code?

i change provider one of api field name,then i use junit testing is ok with a wrong result,but with spring boot testing is a right result.
`
@RunWith(SpringRestPactRunner.class)
@Provider("Some Provider")
@PactBroker
@SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.RANDOM_PORT)
public class PactBrokerProviderTest {
@TestTarget
public final Target targetBoot = new SpringBootHttpTarget();

}
`

i change a request entity field named pwd to pwd1,then the junit testing like this ,and it's my expect result.
10:23:59.250 [main] WARN org.springframework.web.servlet.mvc.support.DefaultHandlerExceptionResolver - Resolved [org.springframework.http.converter.HttpMessageNotReadableException: JSON parse error: Unrecognized field "pwd" (class com.lmx.pactdemoprovider.BookController$DemoReq), not marked as ignorable; nested exception is com.fasterxml.jackson.databind.exc.UnrecognizedPropertyException: Unrecognized field "pwd" (class com.lmx.pactdemoprovider.BookController$DemoReq), not marked as ignorable (5 known properties: "name", "pwd1", "loginTime", "expire", "inner"])  at [Source: (PushbackInputStream); line: 1, column: 71] (through reference chain: com.lmx.pactdemoprovider.BookController$DemoReq["pwd"])]
but the springboot testings with giving a right result.so i get confused.

i resovled this problem. Must be set explicitly MappingJackson2HttpMessageConverter to highest priority.By default the HttpStringMessageConvert in the first of messageConverters list.
like this:
`@SpringBootApplication
public class PactDemoProviderApplication {
/**
 * set jsonConvert to highest priority
 *
 * @return
 */
@Bean
public MappingJackson2HttpMessageConverter newJsonConvert() {
    return new MappingJackson2HttpMessageConverter(new ObjectMapper());
}

public static void main(String[] args) {
    SpringApplication.run(PactDemoProviderApplication.class, args);
}

}`

This is not an issue the pact framework, but with the default settings that Spring uses to parse JSON. There are also Jackson settings to ignore additional properties . See https://stackoverflow.com/questions/5455014/ignoring-new-fields-on-json-objects-using-jackson

@uglyog thanks


Release tagging needs to better handle rate limiting

During our last wave of releases we hit a number of rating limiting issues that causes our github tagging and release process to fail. We should make some improvements to those before the next wave. A couple ideas we could do to improve:

We could create another github service account so that the token we are using isn't shared.
Try to use the reset epoch the give in the error to time our retries. Of course if those resets are more than 5 or 10 mins we should probably not wait that long.

@KarishmaGhiya may have bandwidth to help with this.

The work to create the account and put the PAT into KV was @scbedd
I've verified that the tagging/release process works in JS, .NET, and Python release pipelines with the new PAT. Java validation is pending correct artifact creation.

A few other release pipelines (e.g. mgmt plane, docs) were using the old PAT. They have been updated to use the new PAT.


Make Operation class type arguments optional

This PR changes the Google::Gax::Operation result_type and metadata_type arguments to be optional. This is a backwards compatible change.

@jbolinger @quartzmo Thoughts?

Codecov Report

Merging #142 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #142      +/-   ##
==========================================
+ Coverage   99.27%   99.27%   +<.01%
==========================================
  Files          19       19
  Lines        2058     2064       +6
==========================================
+ Hits         2043     2049       +6
  Misses         15       15

Impacted Files
Coverage Δ

lib/google/gax/operation.rb
100% <100%> (ø)
:arrow_up:

spec/google/gax/operation_spec.rb
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Relying on the registry to do the lookup is probably the common case, so adding the nil defaults seems like a good idea to me.


[Epic MM-23811] Applytheme / centerChannelColor consolidated Hacktober PRs

Summary
This pull request consolidates several PRs from last years Hacktoberfest. It is probably best to review the individual commits as that will make it easier to determine what changes are related. PR's for each commit have already had full dev reviews (eg. https://github.com/mattermost/mattermost-webapp/pull/6807), so now we are just looking for final UX and QA reviews.
See the commits tab for a list of all the relevant commits.
Ticket Link
https://mattermost.atlassian.net/browse/MM-23811
Release Note
NONE

@deanwhillier
I checked listed commits as we discussed and have some I was not able to find.
Prefixed the ones verified with pass

Pass #6807 files only deleted, no other changes
#6810 this should be edit post modal, but I wasn’t able to find bg-text-200 anywhere
#6814 svg-text-color channel icon background - not found
Pass #6812 background color of user role pill e.g system admin on user popover  as expected changed to  rgba(var(--center-channel-color-rgb), 0.3)
Pass #6821 4 files deleted, no other changes all 4 with theme.centerChannelColor not found in the app, as expected
Pass #6862 Channel header dropdown icon, chevron using correct color --center-channel-color-rgb
Pass #6917 channel header is using --center-channel-color Note: noticed the difference in font size file has font-size: 14px;, but it’s font-size: 16px; on test server.
#6915 disabled send button - Not sure this is applied correctly I wasn’t able to find that it is using --center-channel-color , I see color: rgb(255, 255, 255) on the test server
Pass #6986 date separator text is using --center-channel-color as expected
Pass #6985 server URL border on rename channel modal has correct color theme --center-channel-color-rgb
#6933 this PR deletes one file, I wasn’t able to verify, not sure how to trigger system notice “The system notices for admins in the bottom right of the app (not the new in-product notices) should still have their drop shadow coloured correctly”
#6993 not sure where this is, I tried channel header popover, but wasn’t able to confirm
#6992 Also not able to find where the change was applied. It should be “Popovers (such as the Member List in the channel header) should be styled correctly” <popover appears styled correctly
Pass #6991 button transparency as expected
#6938 not able to verify, system notice related, unsure how to trigger on the test server
#7018 this one is for channel header plugin dropdown, I wasn’t able to check. I enabled a bunch of plugins, but I’m probably missing a step to make them appear in the header dropdown.
Pass #6990 channel header popover as expected
Pass #6994  popover border bottom color looks good, using center-channel-color-rgb
Pass #6988 user popover is using center-channel-color-rgb as expected
#7248 not sure where to find post link
Pass #7406 post create using center-channel-color
Pass #9155 - emoji picker header border is using center-channel-color-rgb

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @aspleenic

@zefhemel I have no idea if this pr is relevant anymore, I have two vauge conversations with Dean.. first one was he would need to walk me through all the changes.. and then the second most recent conversation I THINK he mentioned that this would all have to change based on the in progress themeing work?? maybe? sooo I think im going to remove myself as a reviewer on this one, and hand it to you to figure out what to do with this :)

Removing myself as a reviewer as well. Please - re-add me when ready for qa. Thanks.

Heads up that as part of our efforts to move to a monorepo, we're closing out a number of older pull requests like this one to help streamline the effort.
If you'd like to preserve these changes -- even if you're not the original author! -- feel free to resubmit the pull request against the monorepo once it's ready. You can subscribe to mattermost-server-issue-22420 for status updates on this effort.


output format is garbaled since v0.3.0-alpha.1

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behaviour:

Run 'cargo spellcheck check'
Observe garbled rendering, ^^^^ are missing and the erronous line is not rendered

Expected behavior
The line to be rendered with ^^^^ underneath the erroneous word.
Screenshots

Pplease complete the following information:

System: Fedora 32
Obtained:cargo
Version: cargo-spellcheck 0.3.0-alpha.1

Related: #25

The output for multiple lines is still not functional and must be improved, possibly as part of #65 or in a separate PR.

Specifically^^^ sign is shifted from time to time as I see
[zhiburt@zhiburt cargo-spellcheck]$ cargo spellcheck
error: spellcheck(Hunspell)
  --> /home/zhiburt/proj/cargo-spellcheck/src/checker/mod.rs:5
   |
 5 |  Checker

 Trait to handle additional trackers.
 Contains also helpers to avoid re-implementing generic
 algorithms again and again, i.e. tokenization.
   |                                                                                ^^^^^^^^^^^^
   | - autoionization or ionization
   |
   |   Possible spelling mistake found.
   |

I'd wanted to created an issue according to it but I've noticed this issue already exists :+1:


New transaction is not allowed because there are other threads running in the session

Hello, I'm trying to run a procedure, and then an update to a table, however the following error occurs: "New transaction is not allowed because there are other threads running in the session", follow the code below:
`
public function update(TestEntity $testEntity) {
    $sql = "EXEC TEST_PROC 1, 1";

    $entityManager = $this->getEntityManager();
    $stmt = $entityManager->getConnection()->prepare($sql);

    if ($stmt->execute ()) {
        $this->getEntityManager()->merge($testEntity);
        $this->getEntityManager()->flush();
    }
}

`
Any suggestions and / or solutions?

New transaction is not allowed because there are other threads running in the session

What's the RDBMS? Nested transactions are quite normal in ORM context.

Googled a bit, and this may be because the same PHP process has a second
open DB connection. Not really a DBAL or ORM issue: seems more like one of
the usual "SQL server being special" issues...
On Tue, 30 Oct 2018, 13:49 Martin Pfeifer<EMAIL_ADDRESS>wrote:

@Ocramius https://github.com/Ocramius RDBMS is SQL server 2012 and
framework Symfony 3.4.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/doctrine/doctrine2/issues/7446#issuecomment-434286753,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJakOHihgSgA6fDDAdQ4ArzQe9UHm4tks5uqErbgaJpZM4YA5uC
.

@Ocramius
I agree with you, but do you have any solutions and / or suggestions for this case?

Look for the second connection: this is not a bug in this tooling.
On Wed, 31 Oct 2018, 01:18 Martin Pfeifer<EMAIL_ADDRESS>wrote:

@Ocramius https://github.com/Ocramius
I agree with you, but do you have any solutions and / or suggestions for
this case?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/doctrine/doctrine2/issues/7446#issuecomment-434517183,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJakOhG8icJQI1L5uRL6SMQlCGb5ve2ks5uqOxNgaJpZM4YA5uC
.

@Ocramius
I solved it as follows, follow the code below:
`public function update(TestEntity $testEntity) {
$sql = "EXEC TEST_PROC 1, 1";
$entityManager = $this->getEntityManager();
$stmt = $entityManager->getConnection()->prepare($sql);
$stmt->execute();
$stmt->closeCursor();

$entityManager->merge($testEntity);
$entityManager->flush();

}`


[MSHARED-1290] Fix PropertyUtils cycle detection results in false positives

This fixes MSHARED-1290 which was caused by https://github.com/apache/maven-shared/pull/17.

[x] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004

Thanks for having a look! I've addressed your review comments in the new commit @elharo.


[Change Request]: Add an story / example for the itemRenderer property of the Select component.

What type of enhancement/feature is this?
Documentation Improvements
Description
Currently there is no direct example that focuses on promoting the renderItem property on the Select component. Notably there is one for renderSelection.
Use Case/Reasoning for enhancement/feature.
Users are not currently aware that the renderItem property exists unless they go through the detailed list of properties.
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I have searched for existing issues and to the best of my knowledge this is not a duplicate
[X] This is a single enhancement/feature

There are two stories that already do illustrate the renderItem usages:

Async Per Item
Loading Slot

@stefan505 Is this sufficient or is there an additional story required?


Unable to download rlog.bz2 files

Device information

dongle_id | e977b6aeb5565611
Segments:
e977b6aeb5565611|2021-06-16--20-07-31
e977b6aeb5565611|2021-06-16--20-03-45
e977b6aeb5565611|2021-06-16--19-25-58
e977b6aeb5565611|2021-06-16--19-09-14

version = "0.8.5-d4ab1f1e2-2021-06-07T22:13:55",
dirty = false,
gitCommit = "145b222f0d45a8087215d5700240f58e654e7393",
gitBranch = "devel-staging",

Context: these are several drives separated by ignition cycles. First ones had openpilot enabled and the rest were done in dashcam mode.
Also, raw log upload was toggle on after the drives were complete to upload them.
Issue: Uploads appear complete, but rlogs are not downloadable.
The first segment from this drive (e977b6aeb5565611|2021-06-16--18-59-08) has rlogs available and CAN data available in Cabana. All others do not.

This is a preserved route, you probably preserved it before it was done uploading (only preserving some of the files).
unpreserve it and you will be able to download the files if it is recent
preserve it after it is done uploading you will also be able to download the files


greetd.tuigreet: add man page

Closes https://github.com/NixOS/nixpkgs/issues/349072
cc @luc65r as maintainer
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 349180

x86_64-linux

:white_check_mark: 1 package built:

greetd.tuigreet

thank you <3


Feat: Instruct users to setup prerequisites before cluster autoscaler addon.

what

Add prerequisites instructions before cluster autoscaler addon setup.

Please fix the conflicts and also update addons/cluster-autoscaler/config/cluster_autoscaler.yaml with default value.yaml comes with helmchart.

Updated default configuration for cluster autoscaler config file as you mentioned @h1manshu98
addons/cluster-autoscaler/config/cluster_autoscaler.yaml


not working on remote-ssh

i install this on remote-ssh, but it does not working

I had the same issue. It is not working on WSL as well.
VS Code - Insiders suggest to install it on remote machine (WSL in my case),

Not really a solution, but version 1 Bracket Pair Colorizer works on WSL (probably also remote-ssh):

Name: Bracket Pair Colorizer
Id: coenraads.bracket-pair-colorizer
Description: A customizable extension for colorizing matching brackets
Version: 1.0.61
Publisher: CoenraadS
VS Marketplace Link: https://marketplace.visualstudio.com/items?itemName=CoenraadS.bracket-pair-colorizer

Not really a solution, but version 1 Bracket Pair Colorizer works on WSL (probably also remote-ssh):

Name: Bracket Pair Colorizer
Id: coenraads.bracket-pair-colorizer
Description: A customizable extension for colorizing matching brackets
Version: 1.0.61
Publisher: CoenraadS
VS Marketplace Link: https://marketplace.visualstudio.com/items?itemName=CoenraadS.bracket-pair-colorizer
install the ext on local is a right way !


Unknown ruleset element_type: ct helper

[ Fedora 38 - firewalld-1.3.4 ]
Traceback (most recent call last):
  File "~/.local/bin/packet-simulator", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "~/.local/lib/python3.11/site-packages/packet_simulator/__main__.py", line 68, in main
    firewall = Firewall(ruleset)
               ^^^^^^^^^^^^^^^^^
  File "~/.local/lib/python3.11/site-packages/packet_simulator/firewall.py", line 19, in __init__
    self.ruleset = self.parse_ruleset(ruleset)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "~/.local/lib/python3.11/site-packages/packet_simulator/firewall.py", line 72, in parse_ruleset
    raise Exception(f"Unknown ruleset element_type: {element_type}")
Exception: Unknown ruleset element_type: ct helper

Hello, thanks for reporting. Would you mind posting the output of the following commands?
# nft -j list ruleset
# ip -j route show table all
# ip -j address

If necessary, feel free to redact any sensitive information like MAC/IP addresses.

The ct helper issue should be resolved, if you do happen to find another issue please report it as well :).


Adding fargate_profiles attempts to create IAM Role

Description
I operate within an organization that has deligated controls over IAM users being created.  We have a custom Terraform provider that effectively wraps aws_iam_role.  For this module we were able to disable IAM role and pass in an externally created role.
I was attempting to add:
...
  aws_auth_fargate_profile_pod_execution_role_arns = [
    replace(mycompany_iam_role.fargate_eks_role.arn, "iam-role-path/", "")
  ]
  create_iam_role = false
  iam_role_arn    = mycompany_iam_role.eks_cluster_role.arn
  iam_role_path   = "/iam-role-path/"
  fargate_profiles = {
    default = {
      name = "fargate-default"
      selectors = [
        {
          namespace = "fargate-default"
        }
      ]
    }
  }
..

This results in:
module.eks.module.fargate_profile["default"].aws_iam_role.this[0]: Creating...
╷
╷
│ Error: failed creating IAM Role (fargate-default-20220502192005405100000001): AccessDenied: User: arn:aws:sts::x:assumed-role/Admin/you is not authorized to perform: iam:CreateRole on resource: arn:aws:iam::x:role/fargate-default-20220502192005405100000001 with an explicit deny in an identity-based policy
│ 	status code: 403, request id: 99add222-295d-4d50-9d94-da092b73406a
│
│   with module.eks.module.fargate_profile["default"].aws_iam_role.this[0],
│   on .terraform/modules/eks/modules/fargate-profile/main.tf line 31, in resource "aws_iam_role" "this":
│   31: resource "aws_iam_role" "this" {
│
╵

If your request is for a new feature, please use the Feature request template.

[ ] ✋ I have searched the open/closed issues and my issue is not listed.

⚠ Note
Before you submit an issue, please perform the following first:

Remove the local .terraform directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!): rm -rf .terraform/
Re-initialize the project root to pull down modules: terraform init
Re-attempt your terraform plan or apply and check if the issue still persists

Versions

Module version [Required]: 18.20.5

Terraform version:
Terraform v1.0.1

Provider version(s):

+ provider registry.terraform.io/cox-automotive/alks v1.5.15
+ provider registry.terraform.io/gavinbunney/kubectl v1.14.0
+ provider registry.terraform.io/hashicorp/aws v4.12.1
+ provider registry.terraform.io/hashicorp/cloudinit v2.2.0
+ provider registry.terraform.io/hashicorp/helm v2.4.1
+ provider registry.terraform.io/hashicorp/kubernetes v2.11.0
+ provider registry.terraform.io/hashicorp/tls v3.3.0

Your version of Terraform is out of date! The latest version
is 1.1.9. You can update by downloading from https://www.terraform.io/downloads.html

Reproduction Code [Required]

module "eks" {
  # https://github.com/terraform-aws-modules/terraform-aws-eks
  source  = "terraform-aws-modules/eks/aws"
  version = ">= 18.20.5, < 19.0.0"
  create  = true

  # Warning -- `vpc-cni` is not OPTIONAL.  Without it pod networking
  # WILL NOT WORK.
  cluster_addons = {
    coredns = {
      resolve_conflicts = "OVERWRITE"
    }
    kube-proxy = {}
    vpc-cni = {
      resolve_conflicts = "OVERWRITE"
    }
  }

  enable_irsa = true

  aws_auth_fargate_profile_pod_execution_role_arns = [
    replace(alks_iamrole.fargate_eks_role.arn, "x/", "")
  ]

  cluster_endpoint_private_access = true
  cluster_endpoint_public_access  = true

  create_cloudwatch_log_group = false

  cluster_version = "1.21"
  cluster_name    = local.eks_full_cluster_name
  vpc_id          = data.aws_vpc.selected.id
  subnet_ids      = data.aws_subnets.target.ids

  # Cox Automotive's ALKS platform requires that we create
  # the required roles ourselves and pass them into this module.
  # Also note that we prefix the role path, and EKS doesn't like
  # this.  Hopefully in the future ALKS removes the role path.
  create_iam_role = false
  iam_role_arn    = x.eks_cluster_role.arn
  iam_role_path   = "/x/"

  cluster_encryption_config = [{
    provider_key_arn = data.aws_kms_key.x-basic-data-kms-key.arn
    resources        = ["secrets"]
  }]

  eks_managed_node_group_defaults = {
    ami_type        = "AL2_x86_64"
    disk_size       = 100
    instance_types  = ["m6i.large"]
    create_iam_role = false

    # kubernetes-sigs/aws-iam-authenticator#153
    # https://github.com/terraform-aws-modules/terraform-aws-eks/pull/894
    #
    # When you run:
    # kubectl get -n kube-system configmap/aws-auth -o yaml
    # You do not want to see the `/acct-managed` in the rolearn or nodes
    # will be unable to join the cluster.
    iam_role_arn = replace(x.eks_node_role.arn, "x/", "")

    # https://github.com/terraform-aws-modules/terraform-aws-eks/issues/1845#issuecomment-1044652902
    block_device_mappings = {
      xvda = {
        device_name = "/dev/xvda"
        ebs = {
          volume_size           = 100
          volume_type           = "gp3"
          iops                  = 3000
          throughput            = 150
          encrypted             = true
          delete_on_termination = true
        }
      }
    }
  }

  eks_managed_node_groups = {
    runners = {
      min_size     = 3
      max_size     = 10
      desired_size = 3
      update_config = {
        max_unavailable_percentage = 50
      }
    }
  }

  node_security_group_additional_rules = {
    ingress_self_all = {
      description = "Node to node all ports/protocols"
      protocol    = "-1"
      from_port   = 0
      to_port     = 0
      type        = "ingress"
      self        = true
    }

    https_ingress = {
      description = "Node to node all ports/protocols"
      protocol    = "-1"
      from_port   = 443
      to_port     = 443
      type        = "ingress"
      # https://github.com/terraform-aws-modules/terraform-aws-eks/issues/1855
      source_cluster_security_group = true

    }
    egress_all = {
      description      = "Node all egress"
      protocol         = "-1"
      from_port        = 0
      to_port          = 0
      type             = "egress"
      cidr_blocks      = ["<IP_ADDRESS>/0"]
      ipv6_cidr_blocks = ["::/0"]
    }
  }

  cluster_security_group_additional_rules = {
    egress_nodes_ephemeral_ports_tcp = {
      description                = "cluster to node all ports/protocols"
      protocol                   = "-1"
      from_port                  = 0
      to_port                    = 0
      type                       = "egress"
      source_node_security_group = true
    }
  }

  fargate_profiles = {
    default = {
      name = "fargate-default"
      selectors = [
        {
          namespace = "fargate-default"
        }
      ]
    }
  }

  tags = {
    "x-id" = "x"
  }
  cluster_security_group_tags = {
    "x-id" = "x"
  }
  node_security_group_tags = {
    "x-id" = "x"
  }
}

Steps to reproduce the behavior:

Expected behavior

Actual behavior

Terminal Output Screenshot(s)

Additional context

You have disabled the creation of the cluster role, but you have not disabled the creation of the Fargate profile task execution role https://github.com/terraform-aws-modules/terraform-aws-eks/blob/d969e94bbea89ed616713bdab849f042ceda87ac/modules/fargate-profile/main.tf#L31
It looks like based on your info provide and the error message, you will need to have an externally created role for the Fargate task execution as well

Closing since no further action necessary


Dockefile: expand ENV and LABLE command to remove items

ENV needs two args(key/value). we can not unset a variable.
With this patch, we can only pass key to ENV, then we can
unset this variable.
Related Issue: #11922
LABLE command is very similar to ENV command.

Hello, please:

Fix your build failures
Adjust the documentation to clarify what you are doing. We have had several proposals for similar functionality and they will need to be reviewed and contrasted with your pull request.

Thanks!
-Erik

For posterity; this PR was closed in favor of https://github.com/docker/docker/pull/8177 which offers "unsetting" properties inherited from parent images, including ENV.


feat: Remove thiserror

📚 $\text{Stack Overview}$
Pulls submitted in this stack:

#735 👈
main

This comment was automatically generated by st.


How to get classification results in Figure.3？

Hello @eeyhsong,  I have some question as below when I have finished all steps according to Readme file.  Now, i want to get the classification results as shown in Figure.3 in the paper, but when i run draw_pic/plot_topk.py ,  appearing this bug:
Traceback (most recent call last):
File "D:\NICE-EEG-main\draw_pic\plot_topk.py", line 9, in
y_pred = np.load('D:/NICE-EEG-main/pic/y_pred/sub%d_pred.npy' % sub_idx)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "E:\Anaconda3\envs\eeg\Lib\site-packages\numpy\lib\npyio.py", line 427, in load
fid = stack.enter_context(open(os_fspath(file), "rb"))
^^^^^^^^^^^^^^^^^^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory: 'D:/NICE-EEG-main/pic/y_pred/sub1_pred.npy'
I do not find the y_pred file and sub1_pred.npy file after training, could you tell me how to get them?

Hello @az-justin, you can use the code below after the test part, saving the indices of _, indices = similarity.topk(5).

np.save('./pic/y_pred/sub' + str(self.nSub) + '_pred.npy', indices.cpu().numpy())

Best~

Thanks soooo much @eeyhsong.


create detailed build instructions

I'm not familiar with the go build process and like to try your tool as an user.
Would you be so kind and provide detailed build instructions maybe as a docker file!
Thx!

A Dockerfile to build the tool is a great idea! Yes, the build instructions definitely need to be expanded.

Hopefully the updated build instructions are ok. I also added a Dockerfile to build docker-slim if it works better for you. Let me know if it doesn't work or something doesn't make sense :)

Sorry for my late reply, I did not find time earlier.
This Dockerfile is a good start. But I dont know which command to invoke to get binaries built.
My suggestions for some details build instruction:
FROM golang:1.4

RUN go get github.com/mitchellh/gox && \
        gox -build-toolchain -os="linux" -os="darwin"

ADD .   /build
WORKDIR /build

build instructions should be:
docker build -t docker-slim .
docker run -ti docker-slim './src.build.sh'

Then you can find the binaries in ...
This what I expect. I'm completely new to your build process.
Do you understand me?
Cheers Lars


fix: fixed the sequence of steps to remove node

What this PR does:
This PR address changes that fixes the order of steps in the process removing node. Node needs to be deleted after resetting kubeadm installed state.
This PR fixes kubernetes/kubeadm#2201

@neolit123  Review please.

/sig cluster-lifecycle
if you also add the following in the PR description, it will close the original issue:
fixes kubernetes/kubeadm#2201

Thanks @neolit123

This change must be verified and done in all other language docs supported here: https://github.com/kubernetes/website/blob/master/content/

@vinu2003 I also edited the PR description to mark it as a fix for https://github.com/kubernetes/kubeadm/issues/2201 in a machine-readable way.

/approve


(rbac) add principal matchers for dynamic downstream IPs

Commit Message: (rbac) add principal matchers for dynamic downstream IPs
Additional Description:
This PR adds two matchers to RBAC (filter_state_ip, dynamic_metadata_ip) which compare an IP from filter state / dynamic metadata to a CIDR block. If the IP is not found where specified in configuration, the matcher fails. The IP is assumed to be a downstream IP, so the matchers are principal matchers.
Risk Level: low
Testing: unit tested
Docs Changes: none
Release Notes: changelog updated
Platform Specific Features: none

/retest

/wait

succeeded by https://github.com/envoyproxy/envoy/pull/37637


Uploads with styles break when font sizes have decimals

Pushes seem to not work if text has a decimal point, for example a font size of 11.5.

i'm having issues with this. All my text styles disappear as soon as content is synced.

@k0mrad
Interesting. A couple Questions:

Are you pulling or pushing content?
Does your google document have styles in it?
Do you have the Sync Content Styles setting turned on? (see below)

Sync Content Styles
There are two ways to use this:
You Want Styles Synced
This requires the google document's content to have the styles you want.
This will copy the styles of the google document into your sketch file. Careful as if the styles aren't in the google document then it will be overwritten.
To use this you need to turn on syncing styles in the project settings:

You want only the text content to sync
In this case, you only want to the content to sync. This will overwrite any layers that have styles on the text (i.e. a single word in a sentence is bolded). For this, you turn off the Sync Content Styles.

totally my miss @dbrody I am coming from the original plugin and didn't notice this amazing new feature

Let us know what you think!

Also, make sure you are on the latest Sketch Content Sync (2.3.0) and ContentSync.app (0.7.7) versions.

both installed, This is very useful.  is it possible to push all the content and then change the key titles within a spreadsheet or do I need to do this first in the sketch file?

@k0mrad Can you clarify a bit more? Not sure I completely understand. Also, could you submit it as a new issue? So others can find it easily for reference.

Closing this issue as it now does not break pushing / pulling. Instead, ContentSync / Google Spreadsheets will round the font size to the nearest decimal.


Build a gallium tool to build and bundle projects

The project has a couple of important requirements:

Running runtime.LockOSThread and gallium.Loop on the main function
Bundling the binary for certain features to work. e.g: the menu example only works when run from a bundle.

Instead of having people create their own main files, it might be helpful to have a CLI to manage both compilation and bundling.
The UX would be something like this:
github.com/alexflint/gallium/examples/menu/menu.go
package menu

import (
	"os"

	"github.com/alexflint/gallium"
)

type app struct {}

func New() gallium.Starter {
	return &app{}
}

func (*app) Start(app *gallium.App) {
	app.SetMenu([]gallium.Menu{
		{
			Title: "menudemo",
			Entries: []gallium.MenuEntry{
				gallium.MenuItem{
					Title:    "Quit",
					Shortcut: "cmd+q",
					OnClick:  func() { os.Exit(0) },
				},
			},
		}
	})
}

Building the application
$ gallium build github.com/alexflint/gallium/examples/menu

As part of the build, it will create a temporary main package with a main.go file from a template similar to this:
// AUTOGENERATED CODE
// NEVER EDIT MANUALLY
package main

import (
	"runtime"

	"github.com/alexflint/gallium"
	p "{{ .TargetPackage }}"
)

func main() {
	runtime.LockOSThread()
	app := p.New()
	gallium.Loop(os.Args, app.Start())
}

Once generated, it will build the binary and bundle it and produce a ready menu.app.
Beyond not having to deal with thread locking and initialising the loop, we can check whether the returned satisfies other gallium interfaces to build more elaborate main files.

DefaultMenus() []gallium.Menu returning the default menu for the app
Config() gallium.Config could return information used to setup your app or your bundle. Would probably have some information like the one contained in cordova's (phonegap) config.xml.

The CLI could also evolve to offer other features:
$ gallium new			# creates a default app ready to be used with `gallium build`
$ gallium generate menu	# generates all the boilerplate to return a new menu
$ gallium sign Menu.app	# signs Menu.app for distribution

It should also have a command to generate all the Go code needed to build and bundle the app with within the same repo. Everything that would be required to build the app without the cli, so it can be committed to the repo.

I like this proposal a lot. I think a unified gallium tool makes a lot of sense, and I think over time we will have a number of different gallium-specific tasks as you point out, and I agree it will be much better to have a unified tool with a unified interface rather than a collection of different tools. I'm not 100% convinced about code generation, though. I worry that if we encourage users to check the generated code into their vcs then it will be difficult to make API changes since we'll have to somehow update generated code to talk to the new APIs, and I worry that if we encourage users not to check the generated code into their vcs then external tools, like ssa and so forth won't work.
I wonder if the solution is to just have what you described as gallium new - ie a boilerplate generator - and then let users modify and manage the code on their own from there. There could be gallium new --menus to generate boilerplate with a menu system, gallium new --statusbar to generate boilerplate with a status bar icon, and so forth. Users who wanted to structure their code in a different way could then simply work from the docs and write everything themselves.
Thanks for writing this up! Are you thinking of working on this?

@alexflint great!
I'm definitely up for working on this :)

I'm not 100% convinced about code generation, though. I worry that if we encourage users to check the generated code into their vcs then it will be difficult to make API changes since we'll have to somehow update generated code to talk to the new APIs,

Checking-in generated code would actually be better for API changes than asking users to code the main package themselves.
We would generate the code with a big comment saying they must never edit it by hand.
The idea is to allow them to build and bundle the app regardless of whether they have the gallium app installed or with version they have installed. If they wanted to upgrade gallium, they would need to re-generate the code.

and I worry that if we encourage users not to check the generated code into their vcs then external tools, like ssa and so forth won't work.

My worry about keeping the generated code outside of the repo is reproducible builds. I would like somebody to checkout my gallium-based app and be able to build it regardless of whether they have the gallium app installed.

I wonder if the solution is to just have what you described as gallium new - ie a boilerplate generator - and then let users modify and manage the code on their own from there. There could be gallium new --menus to generate boilerplate with a menu system, gallium new --statusbar to generate boilerplate with a status bar icon, and so forth.

There's two different things here in my opinion:

bootstrapping an app from a template (similar to Rails Application Templates: rails new blog -m http://example.com/template.rb)
generating boilerplate code (similar to cobra add and rails generate)

Users who wanted to structure their code in a different way could then simply work from the docs and write everything themselves.

That would always be a possibility, yes.

i disagree here.
I feel that codegen is for much later, and there is little point doing it until the actual code is working.
First get the code working, where you write code against the common API.
Then if you do want to do codegen, dont reinvent the wheel. Use goa.
https://github.com/goadesign/goa
Its a proper DSL driven Code Generator.
I have to say though for menus and other stuff, its just easier to write some code against an API.

@joeblew99 generating the main automatically would actually help while working out the APIs, since we can regenerate the templated main as gallium evolves instead of having people amend the main they wrote manually when they have to upgrade.
When talking about codegen here I'm referring to templated code, rather than DSLs. You would write code against an API rather than a DSL.
Regarding goa. It has a very different scope, since it is a DSL for microservices. I'm talking about something different here.

@ernesto-jimenez thanks for the answer.
so can you point me to an example of where we DO or WOULD gen the main ?. I think i generalised in my answer, and now i want to see the exact context. Curious :)

thanks @ernesto-jimenez


📃: Facebook Login Clone with full responsiveness.

:red_circle: Title : Facebook Login Clone
:red_circle: Aim : Add Facebook Login Clone
:red_circle: Tech Stack Used : HTML , TAILWIND CSS.
Screenshots 📷

:white_check_mark: To be Mentioned while taking the issue :

Full name : Bhavy Zala
What is your participant role? Gssoc'24

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@Avdhesh-Varshney Please assign me this task !

@zalabhavy go ahead.


Apply @fjarrett's fixes for post types and notices.

Hello @kylehg, @majelbstoat,
Please review the following commits I made in branch 'jamie-fix-post-types'.
0781ae409795a9888a6992b25f7238de6d42162c (2015-10-09 08:39:58 -0700)
Apply @fjarrett's fixes for post types and notices.
R=@kylehg
R=@majelbstoat
CC=@fjarrett
MANUAL TESTING=manually verified

:+1:


Border Bottom is missing for Modal titles

Summary
Is missing a blue line below the modal title
What is the current bug behavior?
There's no line border bottom on the modal title
What is the expected correct behavior?
A border bottom with the colour #C1D8E7
Relevant logs and/or screenshots
Figma:

Implemented version:

merged PR

@locothedev fixed


Add additional TimeControl: Time Delay

See https://github.com/mliebelt/pgn-spec-commented/issues/12 and Laws of Chess 6.2.2. There is a time delay mode mentioned, where the increment is not added, but the player plays on the increment.
There is no additional format needed, but a variant of the normal increment mode.

Get hands on some documentation e.g. of DGT chess clocks.

DGT clocks name this mode "Bronstein mode", there is an excellent description to it.

Delay. In Delay settings the players are given a certain amount of free time at the start of each turn before their main thinking time starts counting down. In the official FIDE chess clock this Delay time is added to the main thinking time in the display so that the total time available to each player is always visible. At the start of a player’s turn, the clock starts countdown and if the player moves within the delay time, the time in the display will return to what it was before the start of the turn. This is called Bronstein Delay after Grandmaster David Bronstein who first proposed the method. Options 22-25 have Bronstein Delay settings.

Find a notation for it that can be used then.
Find games played on DGT-Boards that should record the time settings somehow ...

If nothing can be found, the original pgn spec could be expanded like

180+2d: Start with 3 minutes, with a time delay of 2 seconds.
40/5200+30:1800+30d: Start with 40 moves in 90 minutes with a normal increment of 30 seconds, later switch to Bronstein delay for the rest of the game with 30 minutes additional.

Sounds strange, but according to the Laws of Chess / DGT Chess Clocks, that could be possible.

Added https://chess.stackexchange.com/questions/37493/bronstein-timing-in-dgt-clocks-and-the-related-pgn to Stackexchange, could not find any information from the DGT producer about Bronstein and PGN. Next step could be to ask the company about it.

Up to now, no answer (and to be fair, I did not expect any answer). I have sent a request to the DGT team, perhaps they have already an answer what PGN notation should be chosen, when Bronstein Delay is selected on their electronics clocks.

The answer by the DGT team was, that they don't have any special annotation for it. So no information from the one producer that implements the Bronstein delay. So if I want to have an annotation for it, I have to invent my own one.
I don't think it is worth the work to do. There should be some need before you change the standard.


Simplify state conversions

This PR simplifies state conversions in the following ways:

state*2oracle has been removed as well as other mentions to oracle. In short, references to the oracle are now unified into references to the proxy.
statebatch2proxy and statetorch2proxy are now unified by states2proxy
statebatch2policy and statetorch2policy are now unified by states2policy
state2proxy has been removed from all environments because it is enough with the implementation in the base environment, which calls states2proxy.
policy2state has been removed.

I have also taken the chance to add richer docstrings. I believe all other changes should be related to the above.

[x] I have run the sanity checks and everything seems fine.
[x] Tests are passed locally.

Note: the PR includes some deletion of files, which come from a different branch because I messed up things somehow... :sweat: Sorry for that!

@carriepl I would like to have your advice on how to proceed with this PR. The situation is the following:

This branch was off main before main included the crystal's updates.
Therefore, this PR does not update the crystal's environments.
Therefore, if it is merged to main, then main will be broken, until the changes updating the crystal's environments are applied.

I am thinking that an auxiliary branch off main would do, but I wonder if there is any other better way.

@carriepl I would like to have your advice on how to proceed with this PR. The situation is the following:

This branch was off main before main included the crystal's updates.
Therefore, this PR does not update the crystal's environments.
Therefore, if it is merged to main, then main will be broken, until the changes updating the crystal's environments are applied.

I am thinking that an auxiliary branch off main would do, but I wonder if there is any other better way.

Is there any obstacle to rebasing this branch (or a copy of this branch, to be safer) over the main and then updating the crystal environments in the rebased version of this branch?

@carriepl I would like to have your advice on how to proceed with this PR. The situation is the following:

This branch was off main before main included the crystal's updates.
Therefore, this PR does not update the crystal's environments.
Therefore, if it is merged to main, then main will be broken, until the changes updating the crystal's environments are applied.

I am thinking that an auxiliary branch off main would do, but I wonder if there is any other better way.

Is there any obstacle to rebasing this branch (or a copy of this branch, to be safer) over the main and then updating the crystal environments in the rebased version of this branch?

That's about what I thought you could say. The only obstacle is that I don't feel super at ease with rebasing yet. Would you mind outlining what the steps should be?

So I am doing this the way it is easiest and safest to me.

Create auxiliary branch off current main (conversions-simplify-merge).
Resolve conflicts and merge this PR into conversions-simplify-merge (copy of main)
Implement rest of changes related to the simplification of state conversions (on crystal environments) in a new branch off conversions-simplify-merge and merge into main in a new PR (to be done).