how to use result of lassosum to plink

Hi,
When I run the example of lassosum, it is difficult for me to get the same result from lassosum and plink. lassosum code is:
library(lassosum)
setwd(system.file("data", package="lassosum"))
ss <- fread("summarystats.txt")
ref.bfile <- "refpanel"
test.bfile <- "testsample"
LDblocks <- "EUR.hg19"
cor <- p2cor(p = ss$P_val, n = 60000, sign=log(ss$OR_A1))
out <- lassosum.pipeline(cor=cor, chr=ss$Chr, pos=ss$Position,
                         A1=ss$A1, A2=ss$A2, # A2 is not required but advised
                         ref.bfile=ref.bfile, test.bfile=test.bfile,
                         LDblocks = LDblocks)
set.seed(20190822)
pheno <- rnorm(nrow.bfile(out$test.bfile))
v <- validate(out, pheno=pheno)
beta <- v$best.beta
idx_eff <- which(beta != 0) ##lasso zero
idx <- out$sumstats$order[idx_eff]
beta_dat <- data.frame(ss[idx, 1], ss[idx, 3], ss[idx, 4], beta[idx_eff])
test_bim <- data.frame(fread("testsample.bim"))
test_snp <- test_bim[match(beta_dat[, 2], test_bim[, 4]), 2]
beta_dat2 <- data.frame(test_snp, beta_dat[, 3], beta_dat[, 4])
write.table(beta_dat2, file = "eff.txt", col.names = F, row.names = F, quote = F)

plink code is:
cd your/lassosum/datapath/
plink-1.9 --bfile testsample --score eff.txt 1 2 3 sum --out pheno_testsample

Result of lassosum:
head(v$beta.pgs)
-0.418 0.46 -0.451 0.165 0.102

Result of plink:
 FID       IID  PHENO    CNT   CNT2 SCORESUM
   0   HG00096     -9   1634    730 -0.307637
   0   HG00097     -9   1634    748 -0.102278
   0   HG00099     -9   1634    761 -0.202595
   0   HG00100     -9   1634    744 -0.0630223
   0   HG00101     -9   1634    734 -0.221991
   0   HG00102     -9   1634    740 -0.139318
   0   HG00103     -9   1634    716 -0.286505
   0   HG00105     -9   1634    722 -0.221223
   0   HG00106     -9   1634    746 -0.237664

Could you tell me the mistake? Thanks!
Best,
Sheng

Following on this thread. If developers can support output the adjusted beta, that would be the best. We can score by plink for support new file format or even dosage format.
Best regards
Wallace

lassosum and plink are not doing the same thing so I don't understand why you would expect the same result from lassosum and plink.

What do you mean by adjusted beta? From your code, you have generated beta by beta <- v$best.beta. That is correct. Furthermore, out$keep.test will give you a logical vector to indicate which SNPs in the test .bed file map to the betas.

lassosum and plink are not doing the same thing so I don't understand why you would expect the same result from lassosum and plink.

We don't expect the plink and lassosum do the same thing. We just want to use plink's scoring function to use lassosum's adjusted betas. Only the last step.
Thanks much for the prompt reply.
Best regards
Wallace

Can you explain what you mean by "adjusted beta" and how it is supposed to differ from "beta"?

Will the lassosum take in the GWAS summary (univariate beta) and LD structure between snps to update the univariate beta to a new beta (adjusted beta) to control LD between SNPs. And then time the new beta (adjusted beta) with effective allele dosage (0/1/2) to have the PRS? Apologize if my understanding is wrong.
Then our purpose is to use the new beta (adjusted beta) to calculate individual PRS by plink other than by lassosum, picking up the best parameter combination by lassosum is OK.
Best regards
Wallace

If X denote the test genotype matrix, then v$best.pgs = X %*% v$best.beta, which is the "adjusted beta" you described. It could be different from what you get from plink because, (1) plink scales the PGS after X %*% v$best.beta (according the number of SNPs used), (2) the handling of missing genotype in plink maybe different. Try specifying --fill-missing-a2 in plink, and it should have a correlation of nearly 1 with v$best.pgs.

What I understand from the question:

v$beta.pgs is already the "adjusted betas" (meaning directly usable on 0/1/2 genotypes?)
use these effects to compute individual scores with PLINK
individual scores computed with PLINK are not the same as v$best.pgs (correlation of 0.573)

@privefl, I observed the same thing a while ago as you did.
Best regards
Wallace

Hi,
Following on the same thread, would it be possible to export the "v$best.beta" along with their corresponding SNP ids from the "testsample.bim" or the "summarystats.txt" files in the form of a data frame? or does the code from @biostat0903 solve this issue?
@wavefancy @biostat0903 Did you manage to use the code that was posted at the beginning of this thread to export the betas and use plink for the downstream analysis?
Best,
Dheeraj.

Hi,
When I use the v0.4.4, the bug is fixed. Both mse and r2 looks good. Thanks!
Best,
Sheng

@dheerajbobbili1988 ,
Yes. If out is the output from the lassosum.pipeline() function, then the out$sumstats object is a data.frame which gives the details of the SNPs used in v$best.beta. You can simply concatenate the two, e.g. with, sumstats <- cbind(out$sumstats, v$best.beta).

Incidentally, if you know the A1,A2 of the bfile is the same as that in test.bfile, you can use pgs(test.bfile, v$best.beta) to generate the PGS also. The pgs function also supports parallel processing using the cluster option.

Yes, it seemed to work. Thanks a lot!!

I'm trying to export the the final pgs using the option: pgs(out$test.bfile, target.res$best.beta)
However, I'm getting the following error. How to resolve this? TIA
Error in .pgs.default(weights = weights, bfile = bfile, ...) :
Number of rows in (or vector length of) weights does not match number of selected columns in bfile


Pull Requests on WAF-A-MoLE are Welcome!

Hi henrique!
I saw you added a lot of cool features in this fork of WAF-A-MoLE!
If you ever want (or have time) to create one, pull requests on the original WAF-A-MoLE repo are more than welcome :smiley:
Cheers!
\cc @zangobot

@AvalZ opened it up today at the original repo: https://github.com/AvalZ/WAF-A-MoLE/pull/6
I'm leaving this issue open in case anyone stumbles upon this repo first.


Embed images

This issue has been split into several separate issues
#100
#101
#102
#103
#104
#105


[DRAFT] Sample improvements

Fix #1444

Add README.md to samples with no readme
Minor sample edits

Pull Request Test Coverage Report for Build 655

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

Totals

Change from base Build 654:
0.0%

Covered Lines:
730

Relevant Lines:
1447

💛  - Coveralls

@corinagum were you planning to address https://github.com/Microsoft/BotFramework-WebChat/issues/1444 in different pull requests, starting with this one?

@cwhitten I was going to do one pull request, but if you prefer I do multiple then I'm happy to do so. :) Now that I think about it, multiple PR's would be more manageable on the approval side.

That sounds good - thanks. I'll update this and #1444 today.


Installed Add-On : not able to start, no rclone.conf file

The problem
Hi
I did install the add-on.  (in fact 3 times already)
I can't start the add-on.
I search for the config file. Didn't find it.
What version of Rclone Backup has the issue?
3.1.0
What version of Home Assistant are you running?
HA 2024.3
What type of installation are you running?
Home Assistant OS
If you're running HA OS/Supervised, what version of the Supervisor are you running?
2024.3.0
Example YAML snippet
No response
Anything in the logs that might be useful for us?
s6-rc: info: service s6rc-oneshot-runner: starting
s6-rc: info: service s6rc-oneshot-runner successfully started
s6-rc: info: service fix-attrs: starting
s6-rc: info: service fix-attrs successfully started
s6-rc: info: service legacy-cont-init: starting
cont-init: info: running /etc/cont-init.d/00-banner.sh

Add-on: Rclone Backup
Automatically sync backup to remote locations using rclone
Add-on version: 3.1.0
You are running the latest version of this add-on.
System: Home Assistant OS 12.1  (aarch64 / qemuarm-64)
Home Assistant Core: 2024.3.0
Home Assistant Supervisor: 2024.03.0
Please, share the above information when looking for help
or support in, e.g., GitHub, forums or the Discord chat.
cont-init: info: /etc/cont-init.d/00-banner.sh exited 0
cont-init: info: running /etc/cont-init.d/01-log-level.sh
cont-init: info: /etc/cont-init.d/01-log-level.sh exited 0
cont-init: info: running /etc/cont-init.d/init.sh
cont-init: info: /etc/cont-init.d/init.sh exited 0
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service legacy-services: starting
services-up: info: copying legacy longrun rclone (no readiness notification)
services-up: info: copying legacy longrun scheduler (no readiness notification)
s6-rc: info: service legacy-services successfully started
[01:14:32] INFO: Starting Rclone Web GUI...
[01:14:32] INFO: Installed rclone version:
2024/03/14 01:14:32 NOTICE: Serving files from "/usr/share/rclone-webui-react"
2024/03/14 01:14:32 NOTICE: Serving remote control on http://[::]:8099/
2024/03/14 01:14:32 NOTICE: Web GUI is not automatically opening browser. Navigate to http://[::]:8099/ to use.
[01:14:33] INFO: rclone v1.65.0

os/version: alpine 3.16.0 (64 bit)
os/kernel: 6.6.20-haos (aarch64)
os/type: linux
os/arch: arm64 (ARMv8 compatible)
go/version: go1.21.4
go/linking: static
go/tags: none
[01:14:33] INFO: Starting Scheduler...
[2024-03-14] [01:14:33] [WARN]: rclone config not found at "/config/rclone.conf"
[2024-03-14] [01:14:33] [INFO]: checking job configs...
[2024-03-14] [01:14:33] [FATAL]: remote 'google:' does not exist; configured remotes are [[]]
[00:14:33] WARNING: Halt add-on with exit code 1
s6-rc: info: service legacy-services: stopping
[00:14:33] WARNING: Rclone Web GUI crashed, halting add-on (code: 143)
s6-rc: info: service legacy-services successfully stopped
s6-rc: info: service legacy-cont-init: stopping
s6-rc: info: service legacy-cont-init successfully stopped
s6-rc: info: service fix-attrs: stopping
s6-rc: info: service fix-attrs successfully stopped
s6-rc: info: service s6rc-oneshot-runner: stopping
s6-rc: info: service s6rc-oneshot-runner successfully stopped

Additional information
No response

Same issue.
It's because the config (go to the addon's Configuration tab) includes a google drive setup by default, even though it doesn't exist.
[2024-03-14] [01:14:33] [FATAL]: remote 'google:' does not exist; configured remotes are [[]]

This causes the addon to stop, even though you need it running to get the UI... to configure the backups in the first place...
I guess we're forced to specify one config manually in YAML before we can use the Addon?

https://github.com/jcwillox/hassio-rclone-backup/issues/28#issuecomment-1407300750
Workaround

thanks
it worked
succeded to launch rclone and create the config with the UI

Thanks, works for me too :)


Add Tests cases for the Implementation

What this PR does

add tests
setup test coverage with codecov
setup travis ci

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@b82d182). Click here to learn what that means.
The diff coverage is 96.62%.

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   96.66%
=========================================
  Files             ?        2
  Lines             ?       90
  Branches          ?       16
=========================================
  Hits              ?       87
  Misses            ?        3
  Partials          ?        0

Impacted Files
Coverage Δ

src/index.ts
100% <100%> (ø)

src/queryLoader.ts
96.59% <96.59%> (ø)

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 b82d182...d5849df. Read the comment docs.


[#163795598] Delete the db parameter group used for manually enabling the pg_stat_statements extension

What
The extension can now be enabled when creating/updating the database through the rds broker. The broker manages its own custom parameter groups.
All databases have been migrated to the new parameter groups so this one can be removed.
❗ The PR can not be merged until all databases have been migrated to the new parameter group. Check https://www.pivotaltracker.com/n/projects/1275640/stories/163795598 for the list of databases to check.
How to review
Code review is enough
Who can review
Not me.

I've confirmed that there are no databases using this parameter group anymore:
$ aws rds describe-db-instances --region eu-west-2 | jq -r '.DBInstances[].DBParameterGroups[].DBParameterGroupName' | sort -u
prod-lon-pg95-bosh
prod-lon-pg95-cdn
prod-lon-pg95-cf
rdsbroker-mysql57-prod-lon
rdsbroker-postgres10-prod-lon
rdsbroker-postgres95-prod-lon
rdsbroker-postgres95-prod-lon-pgstatstatements
$ aws rds describe-db-instances --region eu-west-1 | jq -r '.DBInstances[].DBParameterGroups[].DBParameterGroupName' | sort -u
prod-pg95-bosh
prod-pg95-cdn
prod-pg95-cf
rdsbroker-mysql57-prod
rdsbroker-postgres10-prod
rdsbroker-postgres95-prod
rdsbroker-postgres95-prod-pgstatstatements

... so this can finally be merged.


Support window parenting on macOS, closes #3751

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[x] No

Checklist

[x] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[x] I have added a convincing reason for adding this feature, if necessary

Other information
Closes #3751
Usage:
#[command]
pub fn create_child_window(id: String, app: AppHandle) {
  let main = app.get_window("main").unwrap();

  let child = window::WindowBuilder::new(&app, id, WindowUrl::default())
    .title("Child")
    .inner_size(400.0, 300.0);

  #[cfg(target_os = "macos")]
  let child = child.parent_window(main.ns_window().unwrap());
  #[cfg(target_os = "windows")]
  let child = child.parent_window(main.hwnd().unwrap());

  child.build();
}

Implemented it in the same way as for Windows, but it would be nice if usage was the same for all platforms. Not sure what the way to implement that would be, maybe RawWindowHandle? Would also be a breaking change, unless a different name is used.

The example is failing because the Windows parent_window expects HWND but main.hwnd().unwrap() returns *mut std::ffi::c_void. Is that a bug with the Windows implementation?

@probablykasper I think the best solution is to change parent_window to return a HWND so I pushed that, but now the child window is not visible (it works if I remove the child.parent_window() call.

Hm, no clue why that would be. Any idea if it works in tao atm?


PP-5439 Administrative find charge by gateway transaction id

Stripe payments before 03 May 2019 store no meta data on Stripe regarding GOV.UK Pay charge information, the only link is the gateway_transaction_id on the charge table
expose internal resource to get a charge by this ID, allowing for intermediate payout reconciliation on older Stripe payouts

@mrlumbu you may have been right https://github.com/alphagov/pay-connector/pull/1508/files/91c08a03ad4939b095812c90c4e7c8d22b456ddd..b66d42740875e37e954ded9bab67857d61aa5253


[1.14][identitygc] Bugfix: Pass shared EnableMetrics into GC struct

Please ensure your pull request adheres to the following guidelines:

[x] For first time contributors, read Submitting a pull request
[x] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[x] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[x] Provide a title or release-note blurb suitable for the release notes.
[x] Are you a user of Cilium? Please add yourself to the Users doc
[x] Thanks for contributing!

This is a 1.14 specific bug as on 1.15 and above there is https://github.com/cilium/cilium/pull/28166. 1.14 shipped with https://github.com/cilium/cilium/pull/22892 and it appears this got missed.
Currently when you run the operator with --enable-metrics that arg gets plumbed into SharedCfg but we never passed it into the GC struct. As a result metrics like https://github.com/cilium/cilium/blob/v1.14/operator/identitygc/kvstore_gc.go#L73-L79 aren't getting emitted on 1.14 as they are checking against GC not the SharedCfg. This changes ensures that we pass through the p.SharedCfg.EnableMetrics parameter to allow the conditional logic to work as expected.
I've tested this on a local 1.14 cluster and configured that with this change the metrics (i.e. cilium_operator_identity_gc_entries and cilium_operator_identity_gc_runs_ are once again emitted during identity GC.
Fix: Ensure enabling metrics turns on identity GC metrics

/test-backport-1.14


KAFKA-5364: ensurePartitionAdded does not handle pending partitions in abortable error state

cc @apurvam
@ijuma Note that this only fixes part of the problem from KAFKA-5364. I'd recommend merging it for the fix and improved test coverage, but we should leave the JIRA open.

@apurvam I noticed one other race condition. When we abort a transaction, we clear the partitions in newPartitionsInTransaction and we abort unsent produce requests. The problem is that the latter happens asynchronously, so we could very well see a batch dequeued for a partition which we had already cleared. I've therefore removed the fatal transition in ensurePartitionAdded in the latest patch.

I left the JIRA open as per @hachikuji's earlier comment in the PR. It may be that subsequent commits addressed the remaining issues. In that case, please close the JIRA.


Changing hive_exec dependency to :core

Edit dependency.gradle to pull in org.apache.hive:hive-exec:1.2.1:core for hive_exec

This is for removing the shaded dependencies within Hive_exec jar, and resolve a guava version conflict.


fix: the expect value to have 7 chunks a 8 chars, last two chunks 7 chars and trailing spaces

Seven chunks should be correct.
I am not sure about last two chunks, can the chunk before last one have 7 chars? If so, then this should fix it.
But it is not clear to me reading the instruction of the exercise.

This comes directly from the problem canonical data. Do you have reason to suspect the canonical data is wrong?

@1ethanhansen see the name of the test function? It states seven chunks, but the error is, that there are eight chunks expected.
And I guess if I read it correct if would have to make a slight adjustment.
Also instructions state c >= r
I asume r is row and c is character. Also the in the instruction is showing c=8, r=7 and the return value has also 7 chunks. And not eight. I am sure that is a mistake in the test.
Can you please tell me the matter with canonical data? should it be fixed there as well or you fix it in canonical data and import it in this repository?

Maybe just rename the test? Imho I would not call it chunks.
https://www.cambridge.org/us/cambridgeenglish/news/how-do-you-know-chunk-when-you-see-one/
https://en.wikipedia.org/wiki/Chunk ?!

The original expect value is correct, but the description should say "8 chunks".
As described in the problem specification, c is the number of columns and r is the number of rows. Since the chunks (which is a fairly well established term in programming to describe this operation) are taken from the columns, there are c of them, each r long. In the edit suggested in this PR, r is 8 and c is 7, which conflicts with the requirements.

Not everyone dominates English very well in this world and think about the audience of exercises. I am not sure if chunk is kind of a real word. But anyways. I agree to continue using chunks, but change the number in the name of the test from 7 to 8.


Fix matchMessage builtin middleware for message events with no text

Summary
As outlined in #182, Slack will send apps message events without a text property when the message was composed with Block Kit. In that situation, Bolt was failing to process the events because it assumed the text property contained a string. This change addresses that problem.
Fixes #182
Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

This PR should fail the tests until I go ahead and add the change for the directMention builtin middleware, and its tests, too.


Update hosting provider

Since smileupps is no longer working, I updated it with fly.io and a guide to hosting

Thanks @Aybee5 sorry I didn’t see PR until just now.  Fly.io looks awesome. Hope I get a chance to try it out.


Fix typo in README.md

Hello,
I have fixed typo in README.md.
Best regards

@Abhayparashar31 Word "Contributers" does not exist in English. See https://www.merriam-webster.com/dictionary/contributor
I have made this pull request, with assumption, that the README file is written in English.
If that's not correct, and you have written it in your own dialect, feel free to ignore this message.


Fancy Zones won't merge

I'm trying to create two vertical columns and one of them horizontally divided. But I can't make the default template with three columns to merge two columns.

Nothing happens when I take one column close or overlap to the other, they are supposed to merge.
Powertoys installed version: v0.43.0

@diegoadelgado are you using a cursor to create the layout? If so, you should be able to click down and drag zone 2 into zone 3 and merge them with the merge prompt that displays when you release the click. To add the horizontal divide, simply click at the area of the column you want to make the split.

Does this help?

Deondre, thanks for your answer.
Yes, I'm using a cursor to create the layout. However, when I click down and drag zone 2 into zone 3 nothing happens. I was expecting that once zone 2 was over to zone 3 to get the merge option but it never showed.
For further reference, I'm connecting via Remote Desktop from my Mac to my windows 10 laptop with powertoys installed.
Best,
Diego Delgado

From: Deondre Davis @.>
Sent: Thursday, August 5, 2021 19:54
To: microsoft/PowerToys @.>
Cc: Diego Delgado @.>; Mention @.>
Subject: Re: [microsoft/PowerToys] Fancy Zones won't merge (#12649)
@diegoadelgadohttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdiegoadelgado&data=04|01||47fef1a5a6bf4f9b7d9008d9586c4f2f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637638044468327604|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=QohYDbh2octg0oyMwPf3yh7eYpt%2FNQgkygO8EopdXB4%3D&reserved=0 are you using a cursor to create the layout? If so, you should be able to click down and drag zone 2 into zone 3 and merge them with the merge prompt that displays when you release the click. To add the horizontal divide, simply click at the area of the column you want to make the split.
[PowerToys Zone Merging]https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F61330599%2F128435664-5c9f107d-2799-4c29-a508-af1a7ebe2856.gif&data=04|01||47fef1a5a6bf4f9b7d9008d9586c4f2f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637638044468337563|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=iMi84guhEVkFy7O7zpse0xtlASJPcAa089GzAECQWOo%3D&reserved=0
Does this help?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FPowerToys%2Fissues%2F12649%23issuecomment-893900134&data=04|01||47fef1a5a6bf4f9b7d9008d9586c4f2f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637638044468337563|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Ouu9hkJ6THXmfFuJl9CpT982iUF8rIJ1p5kSLaBSzOg%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACZA7MMCTLODIKSE5Z2WOITT3MQB3ANCNFSM5BUTOPFA&data=04|01||47fef1a5a6bf4f9b7d9008d9586c4f2f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637638044468347519|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=piyYHry3ZP757D5jB740QbejAZ5jWUC3a1Jskn3EjLY%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01||47fef1a5a6bf4f9b7d9008d9586c4f2f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637638044468347519|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=%2BjgJyK85PbB6gtp7%2Fd4MmIJmz8QuWUC9xCY0rBzkQmQ%3D&reserved=0 or Androidhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26utm_campaign%3Dnotification-email&data=04|01||47fef1a5a6bf4f9b7d9008d9586c4f2f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637638044468347519|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=OuP0N7P1BuYN5nnZFSP9%2Bw8y0VcsmB6rBbKvsOcPr70%3D&reserved=0.

I see, thank you for the additional context! @diegoadelgado can you attach a copy of the logs generated by the Report Bug tool in your systray?

@SeraphimaZ is there a known bug with Remote Desktop interactions with the FancyZone editor? I thought there might be, but I can't seem to find a similar issue.

Deondre,
I'm attaching the PowerToys report generated with the Report Bug menu.
Thanks,
Diego

From: Deondre Davis @.>
Sent: Thursday, August 5, 2021 21:10
To: microsoft/PowerToys @.>
Cc: Diego Delgado @.>; Mention @.>
Subject: Re: [microsoft/PowerToys] Fancy Zones won't merge (#12649)
I see, thank you for the additional context! @diegoadelgadohttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdiegoadelgado&data=04|01||e1717fa68c5446dc94d108d95877043f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637638090453778666|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=XE1WsvtlNR1AlLtRRhRFfiqSWs43dYBrPTztMm3oJlU%3D&reserved=0 can you attach a copy of the logs generated by the Report Bug tool in your systray?
[image]https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F61330599%2F128440838-a35f2134-5aa9-4d4b-a978-5afb1af6846b.png&data=04|01||e1717fa68c5446dc94d108d95877043f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637638090453788624|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=q3JFivvoCFFThfpzLU1ICHK9SZQAiJf0RJ3ggCzAS8M%3D&reserved=0
@SeraphimaZhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSeraphimaZ&data=04|01||e1717fa68c5446dc94d108d95877043f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637638090453788624|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=NQdBCazuxzIh64W3B7DhFR%2FfVjUmhqCS7FqcwXe3Yd8%3D&reserved=0 is there a known bug with Remote Desktop interactions with the FancyZone editor? I thought there might be, but I can't seem to find a similar issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FPowerToys%2Fissues%2F12649%23issuecomment-893928847&data=04|01||e1717fa68c5446dc94d108d95877043f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637638090453798585|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=5cCmp7RfM%2FJfzwT925XiWtegI9V9LYuTBjknNfNxcTg%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACZA7MP2F7NO6MRLB6ZTAOLT3MZBJANCNFSM5BUTOPFA&data=04|01||e1717fa68c5446dc94d108d95877043f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637638090453798585|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=VsRZIc%2BhJqXzQ%2F%2B0X0ITP2Azk5fZToNjPRJzqGnrYGA%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01||e1717fa68c5446dc94d108d95877043f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637638090453808536|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=VOKMJBeETEzpl0MKc%2BjlCaA4fCZO9%2F1c7hsfhwSq7do%3D&reserved=0 or Androidhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26utm_campaign%3Dnotification-email&data=04|01||e1717fa68c5446dc94d108d95877043f|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637638090453808536|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=%2BOK0jqoudD0IYiTKvRCNzYtSaWuG9maZv2yLHOyJBXM%3D&reserved=0.

I see you responded via email. The attachment won't go through unless you attach it directly to a new comment in this issue on the Github repository. Please attach it there so our team can review it.

PowerToysReport_2021-08-05-21-26-08.zip
Deondre, here is the file.

@SeraphimaZ is there a known bug with Remote Desktop interactions with the FancyZone editor? I thought there might be, but I can't seem to find a similar issue.

No, it isn't a known bug. I'll investigate it.

I adding my 2 cents because I got the same issue (without using RDP) and it's seems related.
Even the directive seems clear, the natural reflex is to use the zone separator and trying to put it over a second zone. The results is: nothing happen. As shown on the first post, the OP did like me.
We must grab the square area with the number and drag it over another square area. The blue line under the cursor must me ignored.
I saw other issue about how this is confusing. Using the key "Delete" would be so simpler...

I tried @SteeveGL suggestion, grabbing the zone and dragging it into another, and it worked. I got the merge zones option.

Thanks @SteeveGL for the workaround.

I see, thanks @SteeveGL! We can work on making the displayed instructions/documentation more clear. Possibly adding the Del key to remove zones is something we can also look into following our general stability efforts.

I just spent long time trying to delete a zone by dragging the divider over another divider. The current implementation is not intuitive and the instruction text "Click and drag across zones" is easy to understand in multiple ways.
It seems the current implementation works this way: 1) move your mouse over one zone and press mouse button down but don't release it 2) move the mouse over second zone and release the mouse button after both zones are painted blue 3) click "merge zones" button which just appeared.
Much more intuitive way would be: 1) select a zone by click 2) option "delete" would appear 3) select second zone 4) options "merge" and "delete" would appear.  This would require changing the annoying "click to split" as well, it would need a context aware menu to work more logically.

@SeraphimaZ I understand this issue is not really a bug, but something is not intuitive for (new) users. Am I saying it right? Should we add text to the docs or is there a chance that the suggestion to change the methods/interactions can be changed?

Idea: left-click to add a splitter and right-click to temove a splitter seems very simple too.

In v0.51.1 its even more unintuitive.

docs or is there a chance that the suggestion

I agree with this. It's not a bug, but possible improvement either by making it more clear in docs or introducing a delete button. @crutkas Can we add a gif in Fancy Zones web page showing zones merging?
Other than that, this is basically a duplicate of  #14647

https://docs.microsoft.com/en-us/windows/powertoys/fancyzones#creating-a-custom-layout has pretty clear instructions on merging and the screenshot shows a merge.
In some ways the editor is simple, others it is complex.  So i feel the feedback needs to be hyper targetted.  What is confusing and would docs solve it or do we need to rethink UX and customer interaction models.
I can see a enter/del key being useful to merge and esc to cancel.
/needinfo


Hungarian language support on the UI

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Godot 4: No default environment

Godot version:
godot-4-dev
OS/device including version:
Kubuntu, Vulkan. Graphics Card: RX 580X
Issue description:
When running the project, the window only displays objects, lights, etc. but not the environment.
Steps to reproduce:

Make a new project
Run said project

Minimal reproduction project:
GodotTest.zip
If you have any questions, let me know and i will answer them.

This is because the default environment is now editor-only.

oh... is there a way to make an environment that the viewport can see?

Yes, add a WorldEnvironment node to your scene and add a new Environment resource to it.

ok, i'll try that and tell you how it goes.

okay, while trying to get the viewport to display something other then black (which it did) i then tried setting the background to canvas which made the editor crashed. still couldn't make it display the sky, only other colors.

i should also mention that the viewport only see's objects when a light is on it.

finally got the viewport to see the sky! it was a little tricky though, maybe add something in the docs and or in a hint of some kind?

I can't reproduce any of the above stated issues.
Windows 10, RX 5600 xt, built from latest at the time of writing 165d77a

strange, cause i tried twice last time and got the same results, did you use the project i linked? or did you make your own project for testing?

I made a new project. Your instructions say to use an empty project.
What version of the engine are you using?

it says it in the title

Godot 4 changes multiple times per day. Because it hasn't released yet. Please link the commit that you built from.

i don't know the commit that i built from, i'm gonna update my dev build of godot 4 tomorrow and see if the new commit fixed the problem for me.

I feel like they should have announced in BIG letters wherever they made this change. Converting a project from Godot 3.x to 4.x and a black screen coming up, oh man did my heart just drop!

I feel like they should have announced in BIG letters wherever they made this change. Converting a project from Godot 3.x to 4.x and a black screen coming up, oh man did my heart just drop!

The default environment project setting still exists, but it's set to an empty value by default. I'm surprised the project converter isn't bringing its value along with it though.


fix(e2e): stabilise knative/yaks tests

Release Note
NONE

validation error is what I carelessly inserted into main and not related to this. It's addressed at #3491.


Updating GSU (LWPTENSILE-42)

adding SingleBuffer algorithm for HSS/BSS
removing duplicate kernels when GSU=1 but GlobalSplitUAlgorithm is MultipleBuffer.

No, it is not urgent. Shall I close it?

No open is fine as long as CI doesn't get to bogged down. We just won't merge until after 5.4 FC merges are done.

It looks like precheckin test is failing due to this change. Please check the log.

A number of those test failures are also caused by current rocm build on CI, unrelated to changes in this PR. Send me a message and I can try and narrow down test cases that need to be looked at.

Thanks to Alex for his update, the failures are related to the rocm build, and not the PR.


Android Basics: Shared ViewModel

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-shared-viewmodel?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-unit-3-pathway-4%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-shared-viewmodel#2
In which task and step of the codelab can this issue be found?
In 3 stage,"Update title in app bar" this is last task
Describe the problem
Complete all stages and see how the result does NOT match.
Versions
Android Studio version: 2021.2.1 Patch 1 Build #AI-212.5712.43.2112.8609683, built on May 12, 2022
API version of the emulator: 30
Additional information
Include screenshots if they would be useful in clarifying the problem.

I found the problem: ide suggested import android.os.PersistableBundle fun onCreate(savedInstanceState: Bundle?, persistentState: PersistableBundle?). And if I fix on fun onCreate(savedInstanceState: Bundle?) then everything works.

@eclipsn, thank you!


datasette publish appengine

Similar approach to Heroku, discussed in #90
Looks like this could be pretty easy:  https://cloud.google.com/appengine/docs/flexible/python/quickstart

Wontfix, Cloud Run is already implemented and is a better fit for Datasette.

Wontfix, Cloud Run is already implemented and is a better fit for Datasette.


index.md

This tutorial is about Method Overloading in Java

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hi,
Thanks for this pull request (PR).
Unfortunately, we are marking this PR invalid for hacktoberfest. We believe it does not follow the recommended guidelines:

Quality Standards
In line with Hacktoberfest value #2 (Quantity is fun, Quality is key), we have provided examples of the quality standards we encourage. This applies mainly to beginners.

PRs that are automated e.g. scripted opening PRs to remove whitespace / optimize images.
PRs that are disruptive e.g. taking someone else's branch/commits and making a PR.
PRs that are regarded by a project maintainer as a hindrance vs. helping.
Something that's clearly an attempt to simply +1 your PR count for October.

Learn more at https://hacktoberfest.digitalocean.com/details

Hi,
@RandellDawson what went wrong that you mark this pull request invalid ?

Your PR title was not descriptive at all.  It just says "index.md".  Read box #2 which you put a check mark in.

Ok Thanks
I will review again
On Fri, 19 Oct 2018, 11:36 pm Randell Dawson<EMAIL_ADDRESS>wrote:

Your PR title was not descriptive at all. It just says "index.md". Read
box #2 https://github.com/freeCodeCamp/freeCodeCamp/pull/2 which you
put a check mark in.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/pull/22973#issuecomment-431453423,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcByUo4eXPTrHqO-g0Yz1wtVVETCM72Aks5umhqDgaJpZM4Xwi-i
.


[all OSs] Nodejs 12.x will be removed from the images on November, 14

Breaking changes
Nodejs 12.x will be removed from all the images alongside with its docker containers on Ubuntu runners.
Target date
Rollout of the change will start on November, 14 and will take 3-4 days.
The motivation for the changes
As per the software and image support policy only the 3 latest LTS versions are supported. The v18.x branch has become LTS on 2022/10/25.
Possible impact
If your builds depend on the version v12.x they will take longer as this branch will not be pre-cached in the images anymore.
Platforms affected

[X] Azure DevOps
[X] GitHub Actions

Runner images affected

[X] Ubuntu 18.04
[X] Ubuntu 20.04
[X] Ubuntu 22.04
[X] macOS 10.15
[X] macOS 11
[X] macOS 12
[X] Windows Server 2019
[X] Windows Server 2022

Mitigation ways
Nothing is required, if you are using the Node Tool Installer task (for Azure DevOps) and the setup-node action (for GitHub Actions) both will still download a version you specify if it is available in the versions-manifest.json file.

I need to send it to the wallet

If Node.js 12 is removed from the images will this stop actions relying on Node.js 12 from working or will they still have access to Node.js 12?
See https://docs.github.com/en/actions/creating-actions/metadata-syntax-for-github-actions#runsusing which says:
runs.using
Required The runtime used to execute the code specified in main.
Use node12 for Node.js v12.
Use node16 for Node.js v16.

@MikeMcC399 no, both tasks still will download a version you specify if it is listed in the versions-manifest.json file as stated in the mitigation section.

@mikhailkoliada

no, both tasks still will download a version you specify if it is listed in the versions-manifest.json file as stated in the mitigation section.

Thanks for the confirmation! I wasn't sure, since runs.using wasn't explicitly mentioned in "Mitigation ways" in the original post.

Deployed


Create ADNCA a superset of ADPC

See #602 for previous discussion

@jeffreyad and @RealSlimMahdi it looks like @rossfarrugia wants this completed for next release.  Do you all have bandwidth to complete this?  Can you fill out issue if not and maybe we can get others to work on it??

@jeffreyad and @RealSlimMahdi it looks like @rossfarrugia wants this completed for next release. Do you all have bandwidth to complete this? Can you fill out issue if not and maybe we can get others to work on it??

Yes, I have bandwidth.  This should be able to be closed with #1432 this template creates a template program that creates a dataset suitable for both ADNCA and/or ADPC.

@jeffreyad no pressure if anything slips into the next release in June for this one. Good to have us much as possible in March release from Roche perspective but not the end of the world if any of this one comes bit later.

@jeffreyad no pressure if anything slips into the next release in June for this one. Good to have us much as possible in March release from Roche perspective but not the end of the world if any of this one comes bit later.

@rossfarrugia no problem I'd like to see it in March release as well


add dedent

Hi @mjuksel ! Thanks for the new PR!
I like the overall code but I feel like it could follow our standards a lil more, by reusing internal functions that are already available such as trim, replace, slice, split, etc.
Did you find any barrier on using those?

Did you find any barrier on using those?

@gustavoguichard I kind of did yeah.

Trim doesn't take an argument, so it can't remove new lines.
Using Split<data, '\n'> removes the trailing empty line for some reason?
Because of that, join keeps a leading empty line.
Can't filter out leading/trailing empty lines from the split data, as it will remove empty lines everywhere.

So either we should modify the existing types/functions a bit, or use it this way :smile:

Interesting, lemme think about it... maybe we are seeing some weaknesses of the other methods.
cc: @jly36963

Interesting, lemme think about it... maybe we are seeing some weaknesses of the other methods.

Right.
Trim type can just have an argument added, but that means we can't use the native trim method.
That can easily be solved by replacing it with the one I have here, or something similar.
I'm assuming most of them are easy fixes/tweaks!


Unknown options: reportUnusedDisableDirectives

Describe the bug
Error: Invalid Options:

Unknown options: reportUnusedDisableDirectives
'reportUnusedDisableDirectives' has been removed. Please use the 'overrideConfig.linterOptions.reportUnusedDisableDirectives' option instead.

Reproduction
Nothing
System Info
System:
    OS: macOS 14.3
    CPU: (12) x64 Intel(R) Core(TM) i9-8950HK CPU @ 2.90GHz
    Memory: 2.67 GB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.17.1 - ~/.nvm/versions/node/v18.17.1/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v18.17.1/bin/yarn
    npm: 10.1.0 - ~/.nvm/versions/node/v18.17.1/bin/npm
    pnpm: 8.10.5 - ~/.nvm/versions/node/v18.17.1/bin/pnpm
    bun: 1.0.4 - ~/.bun/bin/bun
  Browsers:
    Safari: 17.3

Used Package Manager
yarn
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

Please provide a minimal reproduction. Thanks.

Why reproduction is required


MINOR: Fix a type of variable

requestTimeoutMs should be int.

RequestTime maybe not very long and integer is enough

LGTM

@granthenke Actually, it's a long everywhere it is passed to another method. That's because timeouts are usually long in APIs. In practice we can use ints for all the timeout options since they all should have a non-infinite value and any reasonable value fits into an int.


[backport] Deprecate cupyx.scipy wavelet functions

Backport of #8061 by @asi1024

/test mini,head

Wavelet-related CI failures resolved in head test, remaining looks irrelevnat.


Vitest 1.0.1 breaks mocked constructors due to unmocked getters

Describe the bug
Calling a constructor that has been auto-mocked with vi.mock will access all its getters, which will fail if the getters depend on anything from the constructor (which is not run due to being mocked).
This seems to have been caused by https://github.com/vitest-dev/vitest/pull/4564 since the mocked constructor now makes a copy of all properties, which will access getters. This seems to stem from the fact that vi.mock will not mock getters, which was a problem in previous versions, but now mocked constructors are completely broken unless you manually mock out every getter using spyOn.
Reproduction
I have made a repo to reproduce the issue here: https://github.com/matanui159/vitest-bug
Run yarn test to see this error:
 FAIL  src/bar.spec.js > Bar
TypeError: Cannot read properties of undefined (reading 'length')
 ❯ t.get nameLength [as nameLength] src/foo.js:3:26
      1| export class Foo {
      2|     get nameLength() {
      3|         return this.name.length;
       |                          ^
      4|     }
      5|
 ❯ new Bar src/bar.js:5:20
 ❯ src/bar.spec.js:7:5

System Info
System:
    OS: macOS 14.1.2
    CPU: (10) arm64 Apple M1 Max
    Memory: 14.17 GB / 64.00 GB
    Shell: 3.6.1 - /opt/homebrew/bin/fish
  Binaries:
    Node: 18.16.1 - ~/Library/pnpm/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 9.5.1 - ~/Library/pnpm/npm
    pnpm: 8.6.10 - ~/Library/pnpm/pnpm
  Browsers:
    Chrome: 119.0.6045.199
    Chrome Canary: 121.0.6167.0
    Edge: 119.0.2151.97
    Safari: 17.1.2
  npmPackages:
    vitest: ^1.0.1 => 1.0.1

Used Package Manager
yarn
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Hi, thanks for reporting the issue and providing a repro!
Indeed, this could be a quite bad regression. I made a PR here https://github.com/vitest-dev/vitest/pull/4677 and hopefully this will be fixed soon.


Comments

Description
comments section components for the send comment and a comment
havent yet made the components take in input so no functionality

Changed to fit the new style, so I made it white
Can post comments, not sure where to get the user ID and project ID from so I had those hardcoded, but I’m assuming those can be retrieved from cookies so I can add that functionality later when specified
Need to change how the api works so that one user can post multiple comments, how it is right now is that once a new user posts, it just overwrites that comment


Lobby selection page

As player of the game
I want to view the currently available lobbies to join them or create a new lobby.
NOTES:

Basic Design
You don't have to implement the design 1:1. Dev has some creative liberty in this story
Use shadcn Table component
You can expect lobbies to only have to work with the current config of 2 players, with playerA and playerB
The colored icon displaying the colors of the joined players IS NOT NEEDED FOR THIS STORY!!!

BLOCKERS:

[x] #196

Acceptance Criteria:

[ ] The table contains all of the designed data, even though it may not be styles exactly like the design
[ ] Create Lobby button exists
[ ] Create Lobby button creates a new lobby, instantly joins it and fowards to player selection
[ ] The table lists all available lobbies, including their desired data
[ ] The join button makes you join the lobby and navigates you to player selection
[ ] The search bar at the top filters the lobbies by their id, searches are executed on change in the underlying text field (so you dont explicitly press search, every keypress just triggers a re-filter)
[ ] The code is fully tested or a good explanation given why it should not or cannot be tested

@0815Sailsman you need a playerName to create a lobby therefore you can't create a lobby before redirecting to playerSelection


Login Button No Action

Thanks for U2F Nice implementation. I tried the same in my own nginx,centos 7 using repository . Got stuck up at login screen. I have place u2fap.js in my PDO-Without-AJAX folder, got all dependencies i.e vendor through composer install and have sqlite installed with two tables users & registration. Kindly guide in resolving issue. Att screenshot of login window.
Thanks Again.
Mitz

Hi Mitz. Are you trying to run the example here https://github.com/Samyoul/U2F-php-server-examples ?
Where are you getting stuck? Does your console log give you any feedback?

Thanks for the reply,
I am newbie and implementing this U2F for one of my academic project as
part of masters.
Yes, i am trying to run U2F-PHP-Server-Examples.
I am stuck at login screen, i am able to see three default users i.e
Samyoul,donkey & Shrek. But after entering on of them and clicking login,
it does not
go further for registration.
I intend to do registration of my yubico security key (i have bought two of
them for testing) with username.
I have created database, by going to mysql -u root -p (XXXX) and tables
users and registration were created.
There is some steps,which i am missing, in case more info  required ,do let
me know, will provide for debug.
Kindly give some steps to look into console log, trying to google also.
Thanks again
Mitz
On Mon, Feb 3, 2020 at 7:04 PM Samuel Hawksby-Robinson <
<EMAIL_ADDRESS>wrote:

Hi Mitz. Are you trying to run the example here
https://github.com/Samyoul/U2F-php-server-examples ?
Where are you getting stuck? Does your console log give you any feedback?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Samyoul/U2F-php-server/issues/7?email_source=notifications&email_token=AOLHF52Z234MLMM44BUDAQLRBAMPJA5CNFSM4KO2MBC2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKT3ILI#issuecomment-581415981,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOLHF53U62I6GY72UISTGUDRBAMPJANCNFSM4KO2MBCQ
.

Hey, Ok. It seems like you have a lot to understand. That's ok, I'll show you how to view your console.
If you're using google chrome open your chrome devtools : https://developers.google.com/web/tools/chrome-devtools/open
Then you need to open the console option : https://developers.google.com/web/tools/chrome-devtools/console

Hi samyoul,
Thanks for reply.
Took some time to learn console feature completely and made some way ahead.
Now i am able to login with some default user.
But not able to register my youbico security key.
Checked console.log and got following error
"TypeError: U2F.register_impl requires at least 4 arguments, but only 3 were passed
u2f-registration.php:37:13
Signatures:
Array []
u2f-registration.php:31:10
Registration Request:
Object { version: "U2F_V2", challenge: "nqfaK9Z71cQ6Y-HHl7WJwU7WVy_s6cgkmcRx5FBdR5Y", appId: "https://localhost" }
"
It indicated one more argument for u2f.register_impl. Please find attached screenshot of the Registration page with console.log screen.
Thanks again.
Mitz


"files uploaded by you" create group error

when I click on "create group" from the files uploaded by you section (the green button), I get following error:
NameError at /fileuploads/search/
name 'user_name' is not defined

update on this...I get the same error when I click on "Create New Group" in the Files drop down menu. I believe the only way to currently create a group is to "create_new" from the file upload menu.
One other problem: when I try to rename a group, I get the following error message
NoReverseMatch at /groups/update_group/
Reverse for 'edit_group' with arguments '('',)' and keyword arguments '{}' not found. 1 pattern(s) tried: ['groups/edit_group/(?P<group_id>[0-9]+)$']

This may be fixed now. It works for me. Can you re-test?

It seems works for me too. I close it for now but please let us know if it is an issue.


Support consumed_capacity keyword argument

Functions in engine.py should accept and pass along a consumed_capacity argument to dynamo3

I am actually in the process of updating and rewriting dynamo3 to support all the new dynamo features. Part of that is refactoring the way it handles the returned ConsumedCapacity data. After that's in a better state I'll look adding some of that logic to flywheel.
What's your use case for this?  Are you trying to do some form of rate-limiting?

Great to hear. Initially I'd like to use the consumed capacity data to collect stats for ad-hoc optimisation. Eventually some kind of rate-limiting or different API behavior could fall out of that.
Will keep an eye on the project, thanks for the quick response!

I just released version 0.4.3 of dynamo3 (with the fix for ConsumedCapacity on BatchGet*). I also added support for setting connection.default_return_capacity = True, which will default all of the relevant queries to using INDEXES instead of NONE. You can also get callbacks for certain events now, like when any call returns a ConsumedCapacity, which I think you can use for your purpose.
conn = DynamoDBConnection.connect('us-west-1')
conn.default_return_capacity = True
def print_capacity(conn, command, query_kwargs, response, capacity):
  print command + ' ' + str(capacity)
conn.subscribe('capacity', print_capacity)

I like this solution more than trying to punch through the capacity arguments in the flywheel API, but let me know if there's anything you still need it to do that it can't yet.

That looks great Steve, thanks for the super fast turnaround!


Proposal to remove mina_signature_kind packages and move to mina_compile_config

the mina_signature_kind library provides a kind of compile time configuration that is as brittle as the mlh configuration we are working to remove. The usage of

src/lib/signature_kind/testnet/mina_signature_kind.ml
src/lib/signature_kind/mainnet/mina_signature_kind.ml

unnecessarily duplicates the codebase's executables in several places:

src/app/cli/src/mina.exe splits into src/app/cli/src/mina_{testnet, mainnet}_signatures.exe
src/app/archive/archive.exe splits into src/app/archive/archive_{testnet, mainnet}_signatures.exe
src/app/rosetta/rosetta.exe splits into src/app/rosetta/rosetta_{testnet, mainnet}_signatures.exe
src/app/rosetta/ocaml-signer/signer.exe splits into src/app/rosetta/ocaml-signer/signer_{testnet, mainnet}_signatures.exe

As we move away from compile time configuration, it as also possible to mix runtime configuration from one network with one of these builds in a way that it incoherent. I would propose that we

remove the signature_kind package(s)
remove the relevant sum type to mina_compile_config and make signature_kind a field there
remove all of these duplicated executables  and supply proper configuration for the remaining one

I have a branch off features/even-less-compile-config
https://github.com/MinaProtocol/mina/tree/martin/remove-mina_signature_kind

related to #15538


options: mark clang-args last

before, bindgen -- -I blah would try to open -I as a header file, leading to a very confusing error message.
The "real" use case behind this is described here: https://github.com/NixOS/nixpkgs/pull/45330
I have checked that these ways of passing args still work/error as expected:
bindgen a.h
bindgen a.h --
bindgen a.h -- -Dblah=1
bindgen -- -Dblah=1
bindgen --
bindgen

I think this looks reasonable, thanks!
@bors-servo r+

:pushpin: Commit c8b5406 has been approved by emilio

:hourglass: Testing commit c8b5406f08af82a92bf8faf852c21ba941d9c176 with merge 9a6ea64167fb1bc4976aacc5f37ce65aca7b4d7b...

Blerg, I forgot CI is busted because rust-ap-syntax doesn't build on nightly.

:boom: Test timed out


Upgrade parcel to latest nightly

This parcel release contains a fix for an issue we had gone around with this PR: #538

Do we care now that we have import * as everywhere now for single part components? Wondering if we should import those normally like they were before now?

Do we care that we have import * as everywhere now for single part components? Wondering if we should import those normally like they were before now?

Haha you read my mind, I was wondering the same too, that's why I linked up the PR where we made these changes in case we wanted to revert them. I'm not sure what to do to be honest, but I also feel like it's weird when we have just one part to import as.

I'm also wondering if it even makes sense for these multipart ones to export the Root one…

I had wondered this before too but it would mean consumers would need to do:
import * as CarouselPrimitive from '@radix-ui/react-carousel';
import { Carousel } from '@radix-ui/react-carousel';

The Root is there so the first line is all they need.

Good point on the docs stuff... maybe we should just leave as it is for now for consistency

Yeah let's keep things as is for now I think, at least externally. As for internally for us what do you reckon? Would you rather use named one when we're using single part imports?

I think it's fine for us to be consistent and just import * as FooPrimitive everywhere. What do you reckon?

Yeah I think that's fine too, makes things easier.


Orders should be "My Orders"

Tracking here: https://www.pivotaltracker.com/story/show/144790949

Resolved by PR #736


Merge 261 to book

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.9 out of 10 committers have signed the CLA.:white_check_mark: randomJoe211:white_check_mark: foesa-yang:white_check_mark: abby-cyber:white_check_mark: cooper-lzy:white_check_mark: whitewum:white_check_mark: izhuxiaoqing:white_check_mark: Nicole00:white_check_mark: Sophie-Xie:white_check_mark: HarrisChu:x: even1111112You have signed the CLA already but the status is still pending? Let us recheck it.


React support

Absolutely would love this!

I have not thought about it, but I would totally try this plugin, if it would support react.

Insist on introducing this feature

Hey all, this formatter is based on a html parser which reorders the attributes. I'm also using TSX a lot and would love react support, but I think this would require a lot of effort and time from my side, as I can't reuse most of the code.
@bassem97 if you insist on introducing this feature I'm happy to review any Pull Request 👍

Hey all, this formatter is based on a html parser which reorders the attributes. I'm also using TSX a lot and would love react support, but I think this would require a lot of effort and time from my side, as I can't reuse most of the code.
@bassem97 if you insist on introducing this feature I'm happy to review any Pull Request 👍

@NiklasPor okay i'll see what i can do

Any updates/progress on this feature?

😭😭😭


HDDS-9697. ContainerStateMachine.applyTransaction(..) should not validate token again.

What changes were proposed in this pull request?
In ContainerStateMachine, the startTransaction(..) method has already validated the token. The applyTransaction(..) should not validate the same token again.
A problem is that the token may expire later on. Then, a server may not be able to re-apply the same transaction when replaying the RaftLog after server restart.
What is the link to the Apache JIRA
HDDS-9697
How was this patch tested?
New unit test.

@adoroszlai , thanks a lot for reviewing and merging this!


Visual diagrams needed for database interactions and API responses

Describe the issue
The codebase uses PostgreSQL as well as an ORM (Prisma). There are various files that make use of Next.js SSR and data fetching, many of these files utilize Prisma.
Considering the various occurrences of the Prisma calls, it would be useful to have a visual diagram on how the codebase uses PostgresSQL and Prisma in the logic.
This could be broken up into multiple diagrams (the following is just a suggestion, feel free to make it less/more modular as needed), i.e.:

What data is stored (table name, rows, columns, etc)
How the data is stored in Prisma after the data is fetched

Additional context
A specific example is 'Classroom' table. There is a column called 'fccCertifications'. The data that's stored in this column is captured when a teacher selects certificates from the dropdown menu when creating a class.
The data in fccCertifications then gets used to identify which curriculum data to extract. Take a look at the api_processor.js file to see some of the URLs that data is requested from.
Here is an example diagram:

Here's another diagram example, please Zoom also I'm happy to share the Figma so navigating the image is easier.

I'm currently working on the re-design for the data processing related to the dashboard and made this diagram - sharing in case it may be helpful for this issue.


How to use svt-av1 on macOS?

Any advice on how to use this library on macOS? I can't seem to get it to load no matter what I do. Any advice welcome.

SVT-AV1 is now included in the macos arm64 wheels.

I can confirm it's now present, although I get some strange errors:
Svt[error]: Error Instance 1: Source Width must be even for YUV_420 colorspace
Svt[error]: Error Instance 1: Source Height must be even for YUV_420 colorspace
Svt[warn]: If you are using tiles with the intent of increasing the decoder speed, please also consider using --fast-decode 1, especially if the intended decoder is running with limited multi-threading capabilities.

...my goal was only to test svt and see if the performance was better than libaom. Any guidance on which library I should prefer?

Those errors are due to a limit of SVT-AV1: it will only encode source images whose widths and heights are even numbers.
I typically use rav1e or SVT-AV1, depending on the circumstances. You can pass speed= as a keyword argument to the save method; speed=0 is the slowest but gives the best quality / compression whereas speed=9 is the fastest. I've found rav1e to offer a good trade-off of speed and efficiency with speed=5 or speed=6 (which you might want if you are encoding images after user upload). SVT-AV1 will give you very good compression if you run it with a slow speed, but it's only really usable that way if you're doing the encoding offline or asynchronously. AOM has a realtime mode that is very fast and is fine if quality and compression aren't very important, and SVT-AV1 can also perform decently with a high speed setting.

Interesting. I'm surprised SVT-AV1 has that limitation; is it only with YUV 4:2:0 inputs?
Let me code up something a bit more scientific and get back to you. ffmpeg has good VMAF support, so now that I have svt running, I can test these three encoders and get a decent understanding of their performance, both in terms of runtime and quality.
I'm encoding massive images (think 100k pixels square) that are broken into tiles, so performance/quality tradeoffs are extremely important to me. One surprising finding was going from speed=6 to speed=4 resulted in about a 4x slowdown in encode time for about two VMAF points.

I wrote a small python library for libvmaf bindings. There's no documentation (you'll need to look at the tests to see how to use it) and I haven't recently updated the CI, so the binary wheels don't include arm64 or python 3.11, but in case it's helpful: https://github.com/fdintino/pyvmaf (pip install pyvmaf==1.0.0a1)


Merge pull request #1 from zephyrproject-rtos/master

update the master branch

wrong operation

Codecov Report

Merging #10827 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #10827      +/-   ##
==========================================
- Coverage   53.29%   53.29%   -0.01%
==========================================
  Files         215      215
  Lines       25843    25843
  Branches     5694     5694
==========================================
- Hits        13773    13772       -1
  Misses       9754     9754
- Partials     2316     2317       +1

Impacted Files
Coverage Δ

boards/posix/native_posix/timer_model.c
51.09% <0%> (-0.73%)
:arrow_down:

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 7fd7be0...3ead1ea. Read the comment docs.


[python] Add test for examples

Make sure all file with .py extension and all file
end with "_example"
Make sure all examples have __doc__
Make sure not have duplicate process definition name
Make sure process definition same as filename

close: #7729

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Don't merge until https://github.com/apache/dolphinscheduler/pull/7632 merge and check whether success or not

Codecov Report

Merging #7759 (265e2f2) into dev (68906f1) will decrease coverage by 0.20%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #7759      +/-   ##
============================================
- Coverage     41.09%   40.89%   -0.21%
+ Complexity     3675     3667       -8
============================================
  Files           642      642
  Lines         26662    26706      +44
  Branches       2992     2997       +5
============================================
- Hits          10957    10921      -36
- Misses        14658    14736      +78
- Partials       1047     1049       +2

Impacted Files
Coverage Δ

...inscheduler/server/log/LoggerRequestProcessor.java
20.31% <0.00%> (-39.07%)
:arrow_down:

...e/dolphinscheduler/api/aspect/AccessLogAspect.java
67.24% <0.00%> (-11.34%)
:arrow_down:

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...er/api/service/impl/TaskDefinitionServiceImpl.java
30.87% <0.00%> (-0.97%)
:arrow_down:

...api/service/impl/ProcessDefinitionServiceImpl.java
26.23% <0.00%> (-0.03%)
:arrow_down:

...e/dolphinscheduler/server/master/MasterServer.java
0.00% <0.00%> (ø)

...e/dolphinscheduler/server/worker/WorkerServer.java
0.00% <0.00%> (ø)

...duler/api/controller/TaskDefinitionController.java
3.70% <0.00%> (ø)

...ler/server/worker/processor/TaskKillProcessor.java
0.00% <0.00%> (ø)

...ache/dolphinscheduler/server/log/LoggerServer.java

... and 6 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 68906f1...265e2f2. Read the comment docs.


Add Laravel 6.0 compatibility.

Missing dependency compatibility:

[ ] https://github.com/jenssegers/model/issues/24

Hi. There's a new jenssegers/model release with proper L6.* support.


Updated dependencies, versions and added CoC and Contributors

Preparing an update to address #2.
@jonblum fyi
@rsenk330 Do you have pypi access still?  I'll be in touch either way..

What's your pypi username? I can give you access.


dotnet build task fails for .NET Core 2 project

Hi, I have just started setting up a build for a dotnet Core 2 project. The project builds fine using the command line locally (dotnet build [ProjectName].csproj). But in VSTS the command fails.
My local dotnet version is 2.1.4. NuGet restore and other tasks before the dotnet build command are successful in VSTS.
I have enabled verbose logging, which throws the following error:

2018-03-02T06:22:04.6010833Z ##[error]Error: C:\Program Files\dotnet\dotnet.exe failed. spawn C:\Program Files\dotnet\dotnet.exe ENOENT
2018-03-02T06:22:04.6028662Z ##[debug]Processed: ##vso[task.issue type=error;]Error: C:\Program Files\dotnet\dotnet.exe failed. spawn C:\Program Files\dotnet\dotnet.exe ENOENT
2018-03-02T06:22:04.6029115Z ##[debug]task result: Failed
2018-03-02T06:22:04.6030026Z ##[error]Dotnet command failed with non-zero exit code on the following projects : D:\a\1\s\src\VolvoLive\VolvoLive.csproj
2018-03-02T06:22:04.6030559Z ##[debug]Processed: ##vso[task.issue type=error;]Dotnet command failed with non-zero exit code on the following projects : D:\a\1\s\src\VolvoLive\VolvoLive.csproj
2018-03-02T06:22:04.6031709Z ##[debug]Processed: ##vso[task.complete result=Failed;]Dotnet command failed with non-zero exit code on the following projects : D:\a\1\s\src\VolvoLive\VolvoLive.csproj
2018-03-02T06:22:04.6032340Z ##[debug]rc:-4058
2018-03-02T06:22:04.6032684Z ##[debug]success:false

VSTS account name: vccsp
team project name: VolvoCarsLive
build definition name: VCL Dev Deploy CI
build number: 20180302.2
agent queue name: Hosted VS2017
Logs: logs_1736.zip

Solved it myself

Can you elaborate the resolution? I am too facing similar issue.

I just cleared the "Working directory" field. Before I had set it to the subfolder containing the .csproj file which was incorrect. The working directory should be the root.

Same problem.  Just to elaborate on baldswede solution (which clearing this field worked), the working directory is the VSTS Build field:

Had the same Problem. Clearing the "Working directory" solved the problem.

I had the same issue with a .NET 6 yaml-pipeline, and the problem was, that I tried to use a file matching pattern (**/MyProject) for the workingDirectory attribute. When replacing it with $(Build.SourcesDirectory)/src//MyProject it worked!


Add support for custom filter types

Currently we only allow two kinds of filters: search (which is a "contains"-type filter) and select (which is an equality filter). We could also let the user define custom filter types; Fixtable-Angular appears to already have this functionality.
For now, not high priority because I can't think of a good use case (search & select should cover the vast majority of use cases). Just wanted to document this feature since the Angular version has it.

👍


Shake unmaximize other IDE windows

On MS Windows 10. I have two projects opened, in two different IDE windows (same IDE). Both windows are maximized.
Project 2 has focus. When shaking, Project 1's window is no longer maximized.
And vice versa: if Project 1 has focus, the problem happens on Project 2.
Please see attached screen recording: the left and right windows are maximized. I shake in the right window, then I observe the left window is no longer maximized.
Nota: my IDE windows icons are not combined in the Windows taskbar. I am using 7+ Taskbar Tweaker for that. This is the same single IDE instance.
https://github.com/hsz/shakeoff/assets/857535/b2115b9f-2e6c-4d30-afae-5b6d0d1e90c9

Thanks for reporting! Do you see any window shaking?
This is how it looks on macOS:
https://github.com/hsz/shakeoff/assets/108333/7e372c64-f587-49c7-8697-e994d9e5302c
Maybe this simply doesn't work on Windows and as a side-effect, window shrinks?

Yeah, I confirm the other window is shaking. Also, I observe this is always the first window that shakes, even if it has focus. If I shake in window 1, it is un-maximized, but window 2 is not affected.
Also tried to de-activate 7+ Taskbar Tweaker, just to be sure. This changed nothing.
https://github.com/hsz/shakeoff/assets/857535/ba1eded8-3d15-44b1-9c13-f2cbc688ff6a

Thanks for reporting! Do you see any window shaking? This is how it looks on macOS:
Screen.Recording.2023-08-09.at.9.42.38.PM.mov
Maybe this simply doesn't work on Windows and as a side-effect, window shrinks?

I have exactly the same issue on:

Fedora 38 KDE Spin
PyCharm 2023.2.1 (Professional Edition) - PY-232.9559.58

In short, shaking the mouse(horizontally) on any PyCharm window, dialogue, etc. causes shaking of the window itself


Add Grains exercise

I believe that the task may be set up a bit more interesting for the PHP implementation.
PHP integers greater than 2^31 (32-bit systems) or 2^63 (64-bit) are casted to floats. In some cases it may lead to problems:

http://php.net/manual/ru/language.types.float.php#warn.float-precision

Consider King hates floats and demands solution with precise integer-only calculations. Don't involve any floats. Don't use gmp or any other similar libraries. Try to make the solution for virtually any board size.

Thanks for submitting this exercise!


Image manager adding max file limit and alternative column structure

Adding user-defined maximum file limit.
Adding a 2 column layout needed in the sidebar
Adding responsive for 4 column layout.

2 Column:

3 file max:

What happens with max = 1?

What happens with max = 1?

You'll be able to have 1 file, which is what we want for the editor.

Yum

Is there a difference between columns and maxFiles? Maybe maxFiles needs to be defined as a prop?

Columns you can have either 2 or 4 column layouts.
maxFiles is a prop that's completely separate from the columns. I didn't add it as a knob because it was acting very oddly if dynamically changed, assuming that's what you're referring to?

No, it needs to be defined as a prop on the component that uses it