Crash when reordering file tabs

Issue Type: Bug
If I drag reorder a tab, Code reports that the window has crashed and reopens
VS Code version: Code 1.28.2 (7f3ce96ff4729c91352ae6def877e59c561f4850, 2018-10-17T00:23:51.859Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770K CPU @ 3.50GHz (8 x 3498)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
31.94GB (14.51GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (58)

Extension
Author (truncated)
Version

html-snippets
abu
0.2.1

project-manager
ale
9.0.1

package-to-readme
ant
0.0.3

cform
aws
0.0.15

vscode-icontheme-nomo-dark
be5
1.3.6

vscode-node-readme
ben
3.0.2

file-templates
brp
1.2.0

better-toml
bun
0.3.2

npm-intellisense
chr
1.3.0

gitignore
cod
0.5.0

vscode-markdownlint
Dav
0.20.0

vscode-eslint
dba
1.6.1

vscode-npm-source
dku
1.0.0

githistory
don
0.4.3

xml
Dot
0.2.0

xml
Dot
2.3.2

vscode-pandoc
Dou
0.0.8

vscode-generate-getter-setter
DSK
0.5.0

vscode-npm-script
eg2
0.3.5

vscode-great-icons
emm
2.1.44

terraform-autocomplete
erd
0.0.8

prettier-vscode
esb
1.6.1

git-project-manager
fel
1.7.1

vscode-npm
fkn
3.3.0

flow-for-vscode
flo
0.8.4

code-runner
for
0.9.4

beautify
Hoo
1.4.6

CreateModule
Hoo
0.0.5

rest-client
hum
0.20.2

open-folder-in-new-instance
jma
0.0.1

docthis
joe
0.7.1

docomment
k--
0.1.2

codeshell
Kip
0.2.2

terraform
mau
1.3.7

terraform-snippets
min
2.1.0

vscode-exec-node
mir
0.5.0

prettify-json
moh
0.0.3

mssql
ms-
1.4.0

python
ms-
2018.9.1

cpptools
ms-
0.19.0

csharp
ms-
1.16.2

Go
luk
0.6.78

node-debug2
ms-
1.29.0

PowerShell
ms-
1.9.0

vs-keybindings
ms-
0.2.0

debugger-for-chrome
msj
4.10.2

node-modules-resolve
nau
1.0.2

vscode-docker
Pet
0.3.1

vscode-versionlens
pfl
0.22.0

material-icon-theme
PKi
3.6.0

view-in-browser
qin
0.0.5

vscode-yaml
red
0.0.17

vscode-icons
rob
7.27.0

yo
sam
0.9.3

code-settings-sync
Sha
3.2.0

gitconfig
sid
2.0.0

tmlanguage
Tog
0.9.0

vscode-react-native
vsm
0.6.18

(3 theme extensions excluded)

Can you please follow the steps in https://github.com/Microsoft/vscode/wiki/Native-Crash-Issues to get at more details around the crash and attach the result here? Thanks!

Hello Benjamin,
I have the DMP and log files. I have attached the log files. Do you want
the DMP as well?
Regards
David
On Mon, Oct 22, 2018 at 11:32 AM Benjamin Pasero<EMAIL_ADDRESS>wrote:

Can you please follow the steps in
https://github.com/Microsoft/vscode/wiki/Native-Crash-Issues to get at
more details around the crash and attach the result here? Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vscode/issues/61433#issuecomment-431869471,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGbe_r6g8co2rMWsZBULyXGJaPxaIU4ks5uneUAgaJpZM4Xygbl
.

@dbuksbaum let's start with the log files. Also:
Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: https://code.visualstudio.com/insiders/


Type system is overcomplicated

I'm wondering if the code is more complicated than it needs to be. There are three types for each universal function, eg BusingerParams, BusingerType, and then Businger.
The "*Type" is superfluous with the *Params types, since each Type is simply related to each Params adding no additional information:
https://github.com/CliMA/SurfaceFluxes.jl/blob/f83dc8642b032184c3b31b2c444ed1a6f43168f1/src/UniversalFunctions.jl#L1113-L1118
I propose refactoring the code to simplify the type structure by combining the concepts of "params" and "type".
I also cannot see the purpose of the types which are currently called Businger, Gryanik, etc. These types are distinct from the "params" and "type" objects by providing a place to store the Monin-Obukhov length. But these objects to not compute this length; it's computed externally. And the length stored there is never actually used outside of the scope where it is computed. In other words, the only time we access the length L is a few lines after it is computed (here, L_MO is an input argument to the function):
https://github.com/CliMA/SurfaceFluxes.jl/blob/f83dc8642b032184c3b31b2c444ed1a6f43168f1/src/SurfaceFluxes.jl#L806-L817
If there is no specific purpose for Businger, Gryanik then they should be eliminated. Note also that these types are all identical, so even if we want to keep them, there only needs to be one type that holds on to L.
In summary, I propose the following changes:

Eliminate the types that are typically notated as uf: Businger, Gryanik, etc
Eliminate the "*Type" constructs: BusingerType, etc
Use the *Params types for dispatch, where Businger and BusingerType are now used
Remove the Params suffix from the existing types to make the lexicon more concise and clear, reflecting the fact that these types both control dispatch and also hold parameter values. Eg BusingerParams should simply be called Businger (or BusingerUniversalFunction to be more verbose).

The first 3 changes are purely internal refactoring, while the last changes the user API.

If we do undertake such a refactor, it's worth discussing whether we want to clean up the nomenclature at the same time. There's significant confusion generated by widespread conflation of ideas and words: the function surface_conditions takes in an object of type "AbstractSurfaceConditions" and spits out SurfaceFluxConditions. Reusing the words "surface" and "conditions" in this way makes the code quite difficult to understand when reading it. It's important to sharply distinguish between the concept of the "state" (or conditions), and the fluxes that are computed as a function of that state.
There seem to be many other instances of names generating confusion. Another one is the function windspeed, which actually returns the magnitude of the difference between the atmosphere velocity and the surface velocity. The "wind speed", on the other hand, cannot be defined as anything other (in a sane world) than the magnitude of the atmosphere velocity.
Given these pop out just from a casual reading of the code, it would seem that a thorough review of this codebase is needed to refactor this code, which is critical to our coupled modeling system, into high quality, maintainable, and understandable software...

@akshaysridhar I'm happy to take this on. If we accept that refactoring is needed then perhaps a place to start is an SDI that proposes a new structure and naming scheme for SurfaceFluxes.jl. I'd also propose splitting the code into more files with more module structure. I find it hard to navigate as is.

@akshaysridhar I'm happy to take this on. If we accept that refactoring is needed then perhaps a place to start is an SDI that proposes a new structure and naming scheme for SurfaceFluxes.jl. I'd also propose splitting the code into more files with more module structure. I find it hard to navigate as is.

@glwagner Thanks for raising this issue - Agreed - we can start a refactor SDI + consistent nomenclature  to make integration into other systems more intuitive.


[Candidate 1v1 Arab Tradegy ZHv2] Unsafe supply piles are spread to far apart of each other

Describe the bug
Unsafe supply piles are spread to far apart of each other.
Expected behavior
Supply piles are placed closer together. Better for GLA and China.

AAAAAAAAAAAAAAAAAAAAAAAAAAAAA
why can't you list all the errors in one place ????
I have already made a new version with corrected lighting and water - but I just noticed this topic. This is VERY inconvenient!

Yeah as of right now you need to look through all bugs on all pages and search for map name to find all bugs for a map. It's not ideal but it's ok. Bugs need to be separate otherwise we cannot close single issues when fixed.

This map has 8 issue reports right now.

Is this a problem?

In my opinion I think it would be best so make them closer together, as soon as GLA or China finishes collecting from 1 or 2 piles, they have to start doing long distance, which I think for a base supply is not good. For USA this is no problem.

Or like this

Issue #21 has not been fixed.
This also applies for the bottom pad.

@Jundiyy please provide image for issue #21 because no one knows what you mean.

I closed #21 as it is covered in #158 and the map is being overhauled by Lyo.

I'm taking over

Fixed with https://github.com/xezon/GeneralsRankedMaps/commit/4638621dc9bead2f455e04457ce248ad8f6526b4


feat: Allow the files in CodeMap to be updated

The algorithm for reusing spans is very naive here. It only does a linear search for a place that fits (first-fit?) after checking that the previous location does not fit. Assuming that the CodeMap won't store that many files though I think this is fine, any update will be followed by a recompilation anyway which will dominate the recompile time.
Since this does not impact normal usage in any way I didn't bother creating another type.

Does CodeMap::update invalidate later files?

Not sure what you mean, but it will only remove the file mentioned with the index. Since CodeMap does not care how large the gaps are in between the files it shouldn't affect any other files.

Seems that there is a bug with the feature gate

I thought it was more that it's on it's way to being stabilized, so the feature gate has been removed. But I might not understand how feature gates work 😛

Derp, you are right it gets stabilized next version. Thought I read something about it becoming stable first with the 2018 edition.
https://github.com/Aaronepower/rust/blob/master/RELEASES.md#version-1260-2018-05-10


ModSecurity: Warning. Matched "Operator Rx' with parameter ^0?$' against variable REQUEST_HEADERS:Content-Length' (Value: 206' )

---BpvUOiJ1---A--
[08/Feb/2023:22:25:57 +0530]<PHONE_NUMBER>55.526256 xx.xx.xx.xx 43638 xx.xx.xx.xx 443
---BpvUOiJ1---B--
GET /Microsoft-Server-ActiveSync?Cmd=Ping&User=user%40domain.com&DeviceId=androidc0000675677&DeviceType=Android HTTP/1.1
Authorization: Basic Zm5ibWFya2V0aW5nQGZhbWlseXdvcmxkLmNvLmluOlJlc2V0QDEyMw==
Host: activesync.domain.com
User-Agent: Android-Mail/2<IP_ADDRESS>1398065.Release
Accept-Encoding: gzip
MS-ASProtocolVersion: 14.0
Content-Type: application/vnd.ms-sync.wbxml
Content-Length: 206
Connection: Keep-Alive

---BpvUOiJ1---D--

---BpvUOiJ1---E--
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">\x0a\x0a    <title>404 — Not Found</title>\x0a    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/>\x0a    <meta name="description" content="Sorry, page not found"/>\x0a    \x0a\x0a\x0a\x0a
activesync.domain.com
\x0a
404
\x0a
Page Not Found
\x0a
\x0a        It seems that the page you were trying to reach does not exist anymore, or maybe it has just moved.\x0a        You can start again from the home or go back to previous page.\x0a
\x0a\x0a\x0a

---BpvUOiJ1---F--
HTTP/1.1 403
Server: jupiter
server: jupiter
Date: Wed, 08 Feb 2023 16:55:57 GMT
Content-Length: 1450
Content-Type: text/html
Last-Modified: Tue, 19 Feb 2019 11:35:21 GMT
Keep-Alive: timeout=60
Connection: keep-alive
X-XSS-Protection: 0; mode=block
ETag: "5c6be9f9-5aa"

---BpvUOiJ1---H--

ModSecurity: Warning. Matched "Operator `Rx' with parameter `^0?$' against variable `REQUEST_HEADERS:Content-Length' (Value: `206' ) [file "/etc/nginx/modsec/rules/REQUEST-920-PROTOCOL-ENFORCEMENT.conf"] [line "162"] [id "920170"] [rev ""] [msg "GET or HEAD Request with Body Content"] [data "GET"] [severity "2"] [ver "OWASP_CRS/3.3.4"] [maturity "0"] [accuracy "0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-protocol"] [tag "paranoia-level/1"] [tag "OWASP_CRS"] [tag "capec/1000/210/272"] [hostname "xx.xx.xx.xx"] [uri "/Microsoft-Server-ActiveSync"] [unique_id "167587535755.526256"] [ref "o0,3v0,3v377,3"]
ModSecurity: Warning. Matched "Operator `Within' with parameter `|application/x-www-form-urlencoded| |multipart/form-data| |multipart/related| |text/xml| |application/xml| |application/soap+xml| |application/json| |application/cloudevents+json| |application/cloudev (16 characters omitted)' against variable `TX:content_type' (Value: `|application/vnd.ms-sync.wbxml|' ) [file "/etc/nginx/modsec/rules/REQUEST-920-PROTOCOL-ENFORCEMENT.conf"] [line "938"] [id "920420"] [rev ""] [msg "Request content type is not allowed by policy"] [data "application/vnd.ms-sync.wbxml"] [severity "2"] [ver "OWASP_CRS/3.3.4"] [maturity "0"] [accuracy "0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-protocol"] [tag "paranoia-level/1"] [tag "OWASP_CRS"] [tag "capec/1000/255/153"] [tag "PCI/12.1"] [hostname "xx.xx.xx.xx"] [uri "/Microsoft-Server-ActiveSync"] [unique_id "167587535755.526256"] [ref "o0,29v331,29t:lowercase"]
ModSecurity: Access denied with code 403 (phase 2). Matched "Operator `Ge' with parameter `5' against variable `TX:ANOMALY_SCORE' (Value: `10' ) [file "/etc/nginx/modsec/rules/REQUEST-949-BLOCKING-EVALUATION.conf"] [line "81"] [id "949110"] [rev ""] [msg "Inbound Anomaly Score Exceeded (Total Score: 10)"] [data ""] [severity "2"] [ver "OWASP_CRS/3.3.4"] [maturity "0"] [accuracy "0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-generic"] [hostname "xx.xx.xx.xx"] [uri "/Microsoft-Server-ActiveSync"] [unique_id "167587535755.526256"] [ref ""]
---BpvUOiJ1---I--

---BpvUOiJ1---J--

---BpvUOiJ1---Z--

We are using Z-Push Opensource ActiveSync solution without mod Secuity its working fine but after enabling the mod security it's not working.
Please help what changes to be done to rules file.

Hi @bsbudhwar,
I'm going to close out this issue now. Feel free to reply and re-open if you have any further questions.


[Q] How to set description and name of run in huggingface trainer?

I am using huggingface trainer to train model and I want to ask how to set name of run and description of run?

As in the image, my run names are random names and description is empty, but I cannot find where to set these variables.

Internal GitHub User commented:
I am using huggingface trainer to train model and I want to ask how to set name of run and description of run?

As in the image, my run names are random names and description is empty, but I cannot find where to set these variables.

Hi @JulioZhao97 ,
Thank you for reaching out. You can check our documentation here. This documentation contains information that might help you.
Regards,
Carlo Argel

Hi @JulioZhao97 ,
Thank you for reaching out. You can check our documentation here. This documentation contains information that might help you.
Regards, Carlo Argel

Thnaks! I fount it.

WandB Internal User commented:
Carlo-Argel commented:
Hi @JulioZhao97 ,
Thank you for reaching out. You can check our documentation here. This documentation contains information that might help you.
Regards,
Carlo Argel

WandB Internal User commented:
JulioZhao97 commented:

Hi @JulioZhao97 ,
Thank you for reaching out. You can check our documentation here. This documentation contains information that might help you.
Regards, Carlo Argel

Thnaks! I fount it.


golang: bump automation with updatecli and GH actions

What does this PR do?
Automate the golang bump within the project itself using updatecli and github actions
Why is it important?
Remove the dependency with Jenkins and the definition of what to do in apm-pipeline-library
Use updatecli that provides the tooling for bumping versions
Issues
Similar to https://github.com/elastic/beats/pull/34458
Test
GIT_USER=*** \
GIT_EMAIL=*** \
updatecli diff --config .ci/bump-golang.yml

does nothing in terms of changing if no a new version.
if a new patch version is available then it bumps all those files as expected. Major and minor version to be queried is defined in the .go-version.
Expand to view

I forced the version 1.19.5 to be used... so the outcome will be

+++++++++++
+ PREPARE +
+++++++++++

Loading Pipeline ".ci/bump-golang.yml"

SCM repository retrieved: 1

++++++++++++++++++
+ AUTO DISCOVERY +
++++++++++++++++++

++++++++++++
+ PIPELINE +
++++++++++++

#########################################
# BUMP GOLANG-VERSION TO LATEST VERSION #
#########################################

SOURCES
=======

minor
-----
The shell 🐚 command "echo 1.19.5" ran successfully with the following output:
----
1.19.5
----

latestGoVersion
---------------
WARNING: ⚠ No GitHub Release found, we fallback to published git tags
Searching for version matching pattern "go1\\.19\\.(\\d*)$"
✔ Github Release version "go1.19.5" found matching pattern "go1\\.19\\.(\\d*)$"

CHANGELOG:
----------
no GitHub Release found for go1.19.5 on "https://github.com/golang/go"

gomod
-----
The shell 🐚 command "echo 1.19.5" ran successfully with the following output:
----
1.19.5
----

CONDITIONS:
===========

goDefaultVersion-check
----------------------
The shell 🐚 command "grep -v -q 1.19.5 .go-version # 1.19.5" ran successfully with the following output:
----
----

dockerTag
---------
✔ The Docker image index.docker.io/library/golang:1.19.5 (amd64) exists and is available.

TARGETS
========

update-gomod
------------

**Dry Run enabled**

⚠ updated the [dry run] content of the file "/var/folders/hn/j6gds1qx1f5bysfn0qp0t03c0000gn/T/updatecli/github/elastic/elastic-agent-libs/go.mod"

--- /var/folders/hn/j6gds1qx1f5bysfn0qp0t03c0000gn/T/updatecli/github/elastic/elastic-agent-libs/go.mod
+++ /var/folders/hn/j6gds1qx1f5bysfn0qp0t03c0000gn/T/updatecli/github/elastic/elastic-agent-libs/go.mod
@@ -1,6 +1,6 @@
 module github.com/elastic/elastic-agent-libs

-go 1.17
+go 1.19

 require (
 	github.com/Microsoft/go-winio v0.5.2

update-golang.ci
----------------

**Dry Run enabled**

⚠ updated the [dry run] content of the file "/var/folders/hn/j6gds1qx1f5bysfn0qp0t03c0000gn/T/updatecli/github/elastic/elastic-agent-libs/.golangci.yml"

--- /var/folders/hn/j6gds1qx1f5bysfn0qp0t03c0000gn/T/updatecli/github/elastic/elastic-agent-libs/.golangci.yml
+++ /var/folders/hn/j6gds1qx1f5bysfn0qp0t03c0000gn/T/updatecli/github/elastic/elastic-agent-libs/.golangci.yml
@@ -83,7 +83,7 @@

   gosimple:
     # Select the Go version to target. The default is '1.13'.
-    go: "1.18.10"
+    go: "1.19.5"

   nakedret:
     # make an issue if func has more lines of code than this setting and it has naked returns; default is 30
@@ -117,19 +117,19 @@

   staticcheck:
     # Select the Go version to target. The default is '1.13'.
-    go: "1.18.10"
+    go: "1.19.5"
     # https://staticcheck.io/docs/options#checks
     checks: ["all"]

   stylecheck:
     # Select the Go version to target. The default is '1.13'.
-    go: "1.18.10"
+    go: "1.19.5"
     # https://staticcheck.io/docs/options#checks
     checks: ["all"]

   unused:
     # Select the Go version to target. The default is '1.13'.
-    go: "1.18.10"
+    go: "1.19.5"

   gosec:
     excludes:

update-go-version
-----------------

**Dry Run enabled**

⚠ updated the [dry run] content of the file "/var/folders/hn/j6gds1qx1f5bysfn0qp0t03c0000gn/T/updatecli/github/elastic/elastic-agent-libs/.go-version"

--- /var/folders/hn/j6gds1qx1f5bysfn0qp0t03c0000gn/T/updatecli/github/elastic/elastic-agent-libs/.go-version
+++ /var/folders/hn/j6gds1qx1f5bysfn0qp0t03c0000gn/T/updatecli/github/elastic/elastic-agent-libs/.go-version
@@ -1 +1 @@
-1.18.10
+1.19.5

ACTIONS
========

WARNING: **Fallback** Please add a title to you configuration using the field 'title: <your pipeline>'
[Dry Run] An action of kind "github/pullrequest" is expected.

=============================

REPORTS:

⚠ Bump golang-version to latest version:
	Source:
		✔ [gomod] Get version in go.mod format (kind: shell)
		✔ [latestGoVersion] Get Latest Go Release (kind: githubrelease)
		✔ [minor] Get minor version in .go-version (kind: shell)
	Condition:
		✔ [dockerTag] Is docker image golang:1.19.5 published (kind: dockerimage)
		✔ [goDefaultVersion-check] Check if defined golang version differs (kind: shell)
	Target:
		⚠ [update-go-version] Update .go-version (kind: file)
		⚠ [update-golang.ci] Update .golangci.yml (kind: file)
		⚠ [update-gomod] Update go.mod (kind: file)

Run Summary
===========
Pipeline(s) run:
  * Changed:	1
  * Failed:	0
  * Skipped:	0
  * Succeeded:	0
  * Total:	1

https://github.com/elastic/beats/pull/34486 was created to test these changes

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2023-02-06T12:35:37.762+0000

Duration: 8 min 6 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
749

Skipped
5

Total
754

:robot: GitHub comments
Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.


Bluetooth Evidence Source attempts to unregister devices that already have bee

Bluetooth ES doesn't remove device notification entries from the dictionary during stop, it just blindly assumes they actually exist

Fixed in 519bf3ff9ae425d848cba5a221fd22337f694f04


Dark theme issues

Description
When we change the theme of the website from white to dark Service section loose its visibility
##Screenshot

I want solve this issue


src/index.html template not being used

Adding an src/index.html file with a modified template does not seem to work. Is there an example of how to use this feature?

I have updated the README. Tell me if you are still having an issue.

Thanks @csharad for your quick response . Works perfectly by following the updated README.

Great! Closing this as the issue is resolved.


Added note to The Length Property.

Pointed people to go to a different section to do the resizing of the array.

Can you improve, please, @itsJuicyBytes .

Added the link and added an explanation to read only.

Thanks!


Sort by raised amount with trace donations counted.

We need to implement a query to count trace donation plus io donations we may need it for TP version.

@mateodaza
You can use this query in staging

query {
  projects(

  orderBy: {
      field: TraceDonations,
      direction: DESC
  }
) {
    totalCount
    projects {
      traceCampaignId
      id
      title
      slug
      qualityScore
      walletAddress
      creationDate
      adminUser{
        name
      }
      users {
        name
      }
      updatedAt
      totalReactions
      totalDonations
      totalTraceDonations
    }
    categories {
      name
    }
  }
}

@divine-comedian can you find a band with and help us on this with service testing:
1- total amount donated for a non-traceable project should be correct and the same with DB
2- total amount donated for a traceable project should be correct: all donations with giveth.io + its campaign donation amount in trace.
All of them in staging.

testing out donations on trace vs. donations on IO
develop.giveth.io
next.giveth.io
right now I can see zero consistency between the two dapps. I tested making a few donations on both via Ropsten
https://develop.giveth.io/campaign/testing-trace-endpoint
https://next.giveth.io/project/testing-trace-endpoint

I made two separate transactions each for 0.003 ETH to each project (trace and IO) so it should show both donations but only shows the one I made on the platform (sorry I used the same amount so it makes it more confusing)

https://develop.giveth.io/campaign/ram-4th-project
https://next.giveth.io/project/ram-4th-project

@divine-comedian @MoeNick Sorry I had missed the last comment of Mitch, I have created a PR for fixing query in featehrs after merging that it would fix this issue

@divine-comedian And the frontend side is not implemented yet, so we need to test it with webservice directly

@mohammadranjbarz  okay - how can I test this issue? should we wait for the frontend implementation?

testing-trace-endpoint

You can call queries directly like this, to see sort by trace donations amount works
query {
  projects(
    take: 10,
    searchTerm: "",
    category: "" ,
    orderBy: {
        field: TraceDonations,
        direction: DESC
         }
  ) {
    totalCount
    projects {
      traceCampaignId
      id
      title
      slug
      qualityScore
      walletAddress
      creationDate
      adminUser{
        name
      }
      users {
          name
      }
      updatedAt
      totalReactions
      totalDonations
      totalTraceDonations
    }
    categories {
      name
    }
  }
}

@mateodaza can you pls make sure it's being used for next and v2 version together.


perf(bitmap): optimize bitmap for all zeros and all ones

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
In our scenarios, it is very likely to have a bitmap with all bits set to 1. For example, the visibility of a compacted chunk and the null bitmap of an array with all values non-null. In this case, the dynamic allocated buffer can be avoided to safe both memory and time. A similar optimization is used in SmallVec.
We already have such an optimization for visibilities (Vis). But this enum is not a perfect abstraction over bitmap and leaks its internal structure (bitmap or len). This leads to our current mixed use of Vis, Bitmap and Option<Bitmap> for visibilities.
This PR includes this optimization as a built-in feature of Bitmap. Since we have maintained the bit count in Bitmap, we can identify whether the bitmap is all-zeros or all-ones by checking count_ones and num_bits only.

if count_ones == 0, the bitmap should be all-zeros
if count_ones == num_bits, the bitmap should be all-ones

in both cases, the bits buffer can be None.
With this optimization, we are free to use Bitmap everywhere without worrying about the extra cost in all-one cases. Next step, we can remove Vis and only use Bitmap for visibilities.
Benchmark results on bitmap of size 1024:

bench
before

after

change

zeros
20.075
ns
480.24
ps
-98%

ones
20.506
ns
480.21
ps
-98%

from_bytes
26.216
ns
28.430
ns
8%

get_from_ones
320.24
ps
320.31
ps
0%

get
318.48
ps
320.31
ps
1%

and
29.640
ns
31.000
ns
5%

or
29.224
ns
31.020
ns
6%

not
23.268
ns
21.869
ns
-6%

eq
317.18
ps
318.11
ps
0%

iter
477.25
ns
479.22
ns
0%

iter_on_ones
485.85
ns
333.99
ns
-31%

iter_ones_on_zeros
6.0423
ns
317.61
ps
-95%

iter_ones_on_ones
790.78
ns
334.26
ns
-58%

iter_ones_on_sparse
92.240
ns
90.978
ns
-1%

iter_ones_on_dense
696.99
ns
698.80
ns
0%

iter_range
333.91
ns
333.42
ns
-0%

zeros_iter
5.0219
ms
3.3781
ms
-33%

ones_iter
5.0097
ms
3.3936
ms
-32%

Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[x] All checks passed in ./risedev check (or alias, ./risedev c)
[x] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR contains user-facing changes.

I will take a look tmr. I suggest running q17 again btw. I remember it is performance sensitive with bitmap, and the last time some optimizations with bitmap actually caused regressions, although micro-bench shows improvement.

the last time some optimizations with bitmap actually caused regressions, although micro-bench shows improvement.

Just out of curiosity, which one is that?

the last time some optimizations with bitmap actually caused regressions, although micro-bench shows improvement.

Just out of curiosity, which one is that?

Sorry it should be "potentially caused regressions", don't have such strong evidence.
https://github.com/risingwavelabs/risingwave/pull/8848 I think it was this one. It was likely just an observation of the performance from me, could be just variance.

I will take a look tmr. I suggest running q17 again btw. I remember it is performance sensitive with bitmap, and the last time some optimizations with bitmap ~actually caused regressions~ had minimal benefit on e2e performance, although micro-bench shows improvement.

Okay. I'm running q17 now.  The actual purpose of this PR is to encourage the use of bitmap and avoid workarounds out of performance concerns (such as Vis). 😄

The performance of Q17 seems not changed significantly.


Set drop/select readonly on some scope condition

Hey Danial!
Is it possible to set ngf-drop, ngf-select disabled on some scope conditions?
Like this

I need this drop areas to be readonly on some point

Added feature in version 5.0.0, as you suggested here.

Great!
Thanks for following up on this.

This feature now move to ngf-select-disabled and ngf-drop-disabled directives.


Family Day in New Brunswick

New Brunswick started observing Family Day beginning on February 19, 2018.

Sorry, this issue has too few details (is this about Canada? which day is it on?).
Please update it with complete information, thx.

My apologies.  I forget that other places are called New Brunswick.
New Brunswick, Canada started observing Family Day in 2018 on the 3rd Monday in February.

Okay, updated on beta. Thank you


PWA-119: Added Portals to more discount badges

Liveshopping widget
ProductCard

Coverage remained the same at 38.07% when pulling 99663d1d637cb5553311fb6d87f3f214adb8666c on PWA-119-add-portals-to-discount-badge-on into 4ac35f208e31054c9aa020a5e60258140389382e on develop.


How do I enable chatbot failback?

Team,
        Thank you so much in advance for working this project, I am very happy to learn that Mycroft has an AIML skill/fallback but I am having trouble with running it. Here are my steps & errors

Platform: Ubuntu server 17.10
Step 1: Install Mycroft in /home - ~/mycroft-core using git clone method
Step 2: Install skill in skills directory & add my own AIML files
Step 3: launch & attempt to start chatbot - no idea how to start the chatbot.

I agree, this doesn't seem to work on my mycroft installation either (unless it falls back to AIML only if there's no Internet connection). I assume it was meant to provide conversational responses in the case that a command doesn't match, but all I get is "Sorry, I didn't catch that. Please rephrase and try again."

Hi,
this was disabled (in a hard way) back in August when we had some memory issues associated with this skill. I've been meaning to re-enable it now when we've gotten that issue resolved but I've not gotten around to it.
I'll make an update to enable this in the skill settings on home.mycroft.ai during the week.

My pull request #4  includes a settingsmeta.json.ignore that could be used to control the usage of an aiml fallback. Have disabled it atm because if the user does not wish to use aiml, he may uninstall the skill...

#7 will fix this

I'm trying to use Mycroft git/dev with fallback-aiml/master and "Enable chatbot fallback" set via skill configuration on Mycroft Home. The fallback doesn't seem to work. Is this a known issue? As I'm new to Mycroft - could you give me a hint, how to debug the problem? I didn't see anything wrong in /var/log/mycroft/skills.log so far.

I am also looking to get this skill working, Any input from the author would be appreciated

Hi @th3r00t sorry for the delay in answering. There was an issue due to a change on how boolean settings were reported which made this skill never activate. This was resolved in #19, the update is still pending in mycroft-skills repo so it won't be upgraded unless you ask mycroft to install the beta version.


Export BITRISE_PROVISIONING_PROFILE_PATH too

When using tools like fastlane resign action (https://github.com/fastlane/fastlane/tree/c60edbf5080453c7d873f50fb02caff9748d9bf4/sigh#resign):
resign(
  ipa: 'path/to/ipa', # can omit if using the `ipa` action
  signing_identity: 'iPhone Distribution: Luka Mirosevic<PHONE_NUMBER>)',
  provisioning_profile: 'path/to/profile', # can omit if using the `sigh` action
)

Is handy to have this step export the provisioning profile path so we don't have to manually
compose and hardcode the path where bitrise stores the provisioning profile.

Awesome! I'll release a new version of the step right away! ;)

1.4.2 released! Thanks again! ;)


Stable .ByCount index sort?

Currently the .ByCount sorter for indices does not order things reliably within a given count.  This means that every time I make a build my index of tags changes order.  It's not a huge deal but it's sortav annoying and adding a secondary sort seems like it would only help.  I could probably draft a patch but I wanted to verify that others are interested first.

Patch is welcomed.

Cool.  I just upgraded to 0.15 and I think this was resolved without any kind of mention in the Changelog.  I'll verify and then close.

@frioux have you checked?

Yeah, sadly the issue does still exist.  I am not sure why it varied from
one system to the next.  I still would like to send a patch but if I don't
get around to it it seems like good low hanging fruit.
On Fri, Mar 11, 2016 at 8:05 AM, Bjørn Erik Pedersen <
<EMAIL_ADDRESS>wrote:

@frioux https://github.com/frioux have you checked?
—
Reply to this email directly or view it on GitHub
https://github.com/spf13/hugo/issues/1930#issuecomment-195429299.

--
fREW Schmidt
http://blog.afoolishmanifesto.com

A patch would be great, but a test case that could be added to hugolib/taxonomy_test.go would be helpful, too.  We clearly need more tests for the taxonomies.


JSON Parsing is Slow

Problem
Parsing JSON is much slower (3.6x) in PBJ than standard Protobuf library and it is much slower than the fastest out there. There is a JMH benchmark attached to this issue to test performance vs the standard Protobuf library that can be checked into the com.hedera.pbj.intergration.jmh package.
Benchmark                                  Mode  Cnt       Score      Error  Units
JsonBench.AccountDetailsBench.parsePbj     avgt    5   48494.176 ±  118.987  ns/op
JsonBench.AccountDetailsBench.parseProtoC  avgt    5   13453.996 ±   42.288  ns/op

Solution
To get a JSON codec out the door quickly while minimizing 3rd party dependencies we use a Antlr 4 generated parser for JSON that tokenizes the input then build a parse tree of context objects. We then extract the data we need from that context object tree and build model record objects. This works fine but is slow and generates huge amounts of garbage in the form of Token and Context objects. Also Antlr starts by buffering the input and converting it into a unicode integer array which is not the fastest and requires a complete read of the stream and large memory allocation.
We should write custom parsing code or use a 3rd party streaming library. Ideally we would not add any 3rd party dependencies unless absolutely necessary. JSON is a very simple format and we should be able to hand write a simple fast parser, like the standard Google protobuf library does.
There is a benchmark for testing all the fastest Java JSON parsers. Ideally we would match the fastest for performance. Most of them are Apache license so we can use them as reference examples for how to write a fast JSON parser.  Because we meed the JSON format to exactly follow the Protobuf spec rules we can not use any of these out of the box I believe.
https://github.com/fabienrenaud/java-json-benchmark

FASTJSON 2.0 seems to be the one to match, beat or use https://github.com/alibaba/fastjson The 3 key things we need to achieve are:


Abandoned project

Hi,
The project https://delphi.readthedocs.io/en/latest/ is connected to a non-existent Github repo: https://github.com/tacsbb/delphi-book/blob/master/index.rst.
I'm mentioning the user @tacsbb here in the hopes that they would be willing to release the delphi project name to use for our actively-developed project named Delphi: https://github.com/ml4ai/delphi.
I didn't see an email address listed on their profile, so I'm trying this method. If this doesn't work, could one of the RTD maintainers contact @tacsbb?

@adarshp I just sent an email to the owner. We will follow our policy and I'll come back in 6 weeks if he/she doesn't respond within this time.

@humitos Thank you!

Hi @adarshp!
The owner of the project didn't answer. The namespace delphi is free to use.
Closing here.

Great, thank you very much!


Cannot find your dataset

How do I access the dataset that you used? It gives 404 Error when I click on the link to your dropbox.

https://research.donorschoose.org/t/download-opendata/33

In the URL above, make sure you use the 'Project' link :
http://s3.amazonaws.com/open_data/opendata_projects000.gz
And keep no more than 20000 lines in this file or your node server will run out of memory and fetching data will take a lot of time
Plus : make sure you add this header line :
_projectid, _teacher_acctid, _schoolid, school_ncesid, school_latitude, school_longitude, school_city, school_state, school_zip, school_metro, school_district, school_county, school_charter, school_magnet, school_year_round, school_nlns, school_kipp, school_charter_ready_promise, teacher_prefix, teacher_teach_for_america, teacher_ny_teaching_fellow, primary_focus_subject, primary_focus_area ,secondary_focus_subject, secondary_focus_area, resource_type, poverty_level, grade_level, vendor_shipping_charges, sales_tax, payment_processing_charges, fulfillment_labor_materials, total_price_excluding_optional_support, total_price_including_optional_support, students_reached, total_donations, num_donors, eligible_double_your_impact_match, eligible_almost_home_match, funding_status, date_posted, date_completed, date_thank_you_packet_mailed, date_expiration
And it appears that the date format may have chenged, I had to use in dashboard.js :
var dateFormat = d3.time.format("%Y-%m-%d %H:%M:%S");

Thanks for your response !


Fixed Brazilian references one more time, and removed spaces from jur…

…isdiction strain names e.g Northern Ireland to NorthernIreland, adjusted a few strain name dupes

This should be the last big (1000 line+)commit, most of it due to removal of spaces in jurisdiction names for strains.

Thank you once again @nodrogluap !!


Docker - MFA token persistant

Sorry if this was covered but is there a directory that i can make persistent so that MFA doesn't need entered at each run/container cycle?
Would be cool if the tool had the ability to also grab the picture/video details like date taken and location.
Great work on this.  Thanks for sharing.

How do you run gimme-iphotos? I might need to add support for cookies directory (at least for docker).
Did you check if this is in EXIF data on photos?

I initially create the container with Portainer and use the console connection to run the first sync.  This way I can plug in the MFA code.  After that I want to edit the container start command to run the sync at startup but in order to add that startup command it basically creates a new container which doesn't have the authentication tokens and needs MFA done again but I cannot directly interacts with that run because it is running as a service.   If I just reboot the original container and use a console connection, I can run the sync without it asking for MFA again because they must be cached somewhere.  Do I just need to make the /app directory persistent?
I apologize, it is under the advanced properties.  I chose a couple older photos that did not have the info on the first look.  I can see the info on all others.
Thanks for all your work on this.  Awesome tool!

I did some research and seems to be possible to replicate the effect of "remember me" checkbox you have when visiting icloud.com in a browser. Will try to implement during this week.
Great, thanks for checking :+1:


Adding dart-sample with gradle

Adding gradle dart sample.
It is the same code as the sample already available with Maven.

I think we should

just add a gradle build to the sample we already have
not include the wrapper folder in the tree : see https://github.com/JakeWharton/butterknife/tree/master/butterknife-sample


[Replace Option with Enum] NL local table 11409 'Elec. Tax Declaration Header', field 41 'Declaration Period'

The list of possible periods for the VAT declaration is not complete. (quarterly declaration, starting in the 2nd or 3rd month of a 'standard' quarter is not possible now)
We expect that this will be fixed by Microsoft, but until that time, making the mentioned field extensible will suffice.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Enforce real name

We have many PRs from users using their GitHub username in the author field.
Is there a way to enforce usage of the real name in the author field? E.g. by checking for a space. This check would have worked for every sign-off with a non-real name I saw until now.

Sorry for filing this two times. I received a 404 on the first submit., but it worked though.


Documentation idea

Making an issue to track an idea for making a new documentation page showing the scores in a table

Closing this issue now that Steph-Chong is working on this, tracked in another issue


Conflict with this repo's purpose and hacktoberfest 2018's github tracker

Overview
Hacktoberfest may be tracking our pull-requests in github indiscriminately. This automates the process by taking away the busy work of reporting and validating contributions.
This repository is created to privately track our work in Hacktoberfest as a class.  Professor also would like us to practice proper use of the tool by using pull request to make changes.
Problem
Due to Hacktoberfest's way of tracking our pull-request activities, us creating a pull request in this repository to make a report may unfortunately be counted towards actual contribution towards Hacktoberfest.
Can this be clarified?
Also, if this is true, what can we do to address the issue?

The participation details for Hacktoberfest say:

Pull requests can be made in any GitHub-hosted repositories/projects. As long as the project is public and GitHub-hosted, your PRs will count towards your participation

(emphasis mine).
This says to me that PRs made to this repo will count towards the Hacktoberfest count.
A simple fix seems to be to make the repository private, which is something @humphd will have to take care of. At that point, Hacktoberfest won't count PRs opened against this repository towards our totals for contribution.

I've emailed them for clarification.  I'll report back here.

I still haven't heard back from the Hacktoberfest organizers, but given @Pooch11's comment above, I'm going to move the reporting requirements for our students into a wiki in this repo.  I'll do that now, and disable pull requests.
I'll report back when it's done.

I've moved things to the wiki, and fixed the repo in f5d31aa.  I'll discuss in class with everyone, but feel free to start adding things there, and making whatever wiki pages you want.


Hard to use!

Sorry, I don’t already know how to fix the documentation to make it easier. Can you describe in a little more detail, or is this a problem of the whole component?
Did you watch this demo? Maybe it will be easier than the sample code in README.
Codepen


Updating 'Define Custom Error Messages' in Documentation

The section makes no mention of the fact that in fact it depends on the specific Error Classes in werkzeug.exceptions. The examples given also violate this rule, therefore I am updating the section to reflect this. The pull request I submitted addresses this issue
https://github.com/flask-restful/flask-restful/pull/597

Please just add a comment to your existing pull request to "bump" it. Creating a new issue with the exact same text just adds noise.


Update README.md

Added Upgrade steps to README file
Removed specific links in the Installation steps
Overall updated format

@cla-bot check


[Feature Request] Clear cache of a single model instance

Right now, it clears all of the model's cache when flushCache is called. Suggestion is to allow cache clearing for a single model instance or a collection of models (e.g. clear the cache of models by query).

Hi @d3radicated , unfortunately there is no way to remove a single model instance from cache due to the way the cache is built. Or are are asking to flush the cache pertaining to a give model class?
Can you explain why you want to use this, so I can better understand your requirements?
Thanks!

Oh, i see. I haven't been able to dabble with how the cache is built.
I just noticed that when a cached model's relation is updated, the cached model is not updated.

Ah, that sounds like a bug. Could you provide some code to replicate the issue? Thanks!

Don't know how to provide it but the gist of it is like this:
A blog (cached) belongs to one author (not cached). Whenever I query for a list of blogs with authors, the collection is cached along with the relation (right?). Now, when I update the author (e.g. disabled the author's account) and run the same query again, the author relation is not updated.

I would need example eloquent queries, so that I can recreate them in a test. Often times the issue lies in how the queries are built, and not necessarily with the package. Providing concrete examples will help me test and eliminate any misunderstanding. :)

Sorry for the late reply. I haven't checked back if the issue still exists since we just clear the blog cache when an author is updated. I'll get back to you when we do some code clean-up. Thanks. :)


mosquitto: fix pkg-config files

Description of changes

Fix prefix in pkg-config files

Before:
$ cat result-dev/lib/pkgconfig/libmosquitto.pc
prefix=/nix/store/h1brpgrnvwfw546882wwvwg46469k6p1-mosquitto-2.0.18
exec_prefix=${prefix}
includedir=/nix/store/j9rrc2sz74c837zgws1in1cc82pgrra5-mosquitto-2.0.18-dev/include
libdir=${exec_prefix}/lib

Name: mosquitto
Description: mosquitto MQTT library (C bindings)
Version: 2.0.18
Cflags: -I${includedir}
Libs: -L${libdir} -lmosquitto

After:
$ cat result-dev/lib/pkgconfig/libmosquitto.pc
prefix=/nix/store/4anskp1303vbqzvzc7923k7yadawsfv1-mosquitto-2.0.18-lib
exec_prefix=${prefix}
includedir=/nix/store/l8hq7nl0cfh3iplnipmy2shbc4zr420q-mosquitto-2.0.18-dev/include
libdir=${exec_prefix}/lib

Name: mosquitto
Description: mosquitto MQTT library (C bindings)
Version: 2.0.18
Cflags: -I${includedir}
Libs: -L${libdir} -lmosquitto

Use uthash from nixpkgs instead of bundled one
No need make -C man, manpages are generated by default
No need -DWITH_THREADING=ON, it's ON by default

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] 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.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

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

7 packages built:

collectd
domoticz
ebusd
mosquitto
mosquitto.dev
mosquitto.lib
owntracks-recorder

Result of nixpkgs-review pr 273652 run on x86_64-darwin 1

2 packages marked as broken and skipped:

collectd
domoticz

3 packages built:

mosquitto
mosquitto.dev
mosquitto.lib

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3087

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1316

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

7 packages built:

collectd
domoticz
ebusd
mosquitto
mosquitto.dev
mosquitto.lib
owntracks-recorder

nixos test also passes, LGTM. Thanks!


Switch some match statements from Bundle -> Record

In dsptools I'm playing around with making DspComplex a Record. This fixes places where tests die.
(Note that this is against localflatten branch, which is needed to build against current chisel3)

@chick please consider merging this -- without it, I can't do any testing on anything with Record (some hetero vec stuff) in the IO.

This also seems obsoleted by https://github.com/ucb-bar/chisel-testers/pull/125

This is obsoleted by #125


Cant get past peer dependency

Thinking the peer dependency<EMAIL_ADDRESS>should be a devdependency - currently version 2.0 cannot be installed by the public because of this.
ERESOLVE overriding peer dependency
npm WARN While resolving<EMAIL_ADDRESS>npm WARN Found: @qlik-trial/sprout@undefined
npm WARN node_modules/@qlik-trial/sprout
npm WARN
npm WARN Could not resolve dependency:
npm WARN peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>npm WARN   @nebula.js/sn-pivot-table@"*" from the root project
npm notice
npm notice New minor version of npm available! 9.5.1 -> 9.7.1
npm notice Changelog: https://github.com/npm/cli/releases/tag/v9.7.1
npm notice Run npm install -g<EMAIL_ADDRESS>to update!
npm notice
npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/@qlik-trial%2Fsprout - Not found
npm ERR! 404
npm ERR! 404  '<EMAIL_ADDRESS>is not in this registry.
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

+1 having the same issue

Has anyone found a solution for this yet?
Judging from the weekly npm downloads I'd have figured installing v2.0.0 would work but I still get the above mentioned error.
Also, it seems as though v1.x.x might still work but would require me to downgrade from<EMAIL_ADDRESS>to v3.x.x which is also really not what I had planned.
Surely this package hasn't been "unusable" since June 19th?
Thanks in advance!

Looking into this 👀

:tada: This issue has been resolved in version 2.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


ERROR bad signature: Unknown signature subpacket: 33 (error 1002)

[write something useful and descriptive here]
my log id: fc46500d51a11c686519731c
$:
You have two options.

(1) Keybase can use GPG commands to sign the installation.

(2) Keybase can export your secret key from GPG and save it to keybase's local encrypted
    keyring. This way, it can be used in 'keybase pgp sign' and 'keybase pgp decrypt'
    going forward.
Which do you prefer?: 1
Enter a public name for this device: archlinux
▶ ERROR bad signature: Unknown signature subpacket: 33 (error 1002)

Dupe
On Wed, Dec 7, 2016 at 10:46 AM Erik Løvland<EMAIL_ADDRESS>wrote:

[write something useful and descriptive here]
my log id: fc46500d51a11c686519731c
$:
You have two options.
(1) Keybase can use GPG commands to sign the installation.
(2) Keybase can export your secret key from GPG and save it to keybase's local encrypted
keyring. This way, it can be used in 'keybase pgp sign' and 'keybase pgp decrypt'

going forward.

Which do you prefer?: 1
Enter a public name for this device: archlinux
▶ ERROR bad signature: Unknown signature subpacket: 33 (error 1002)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/5025, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA05_xVhDKFhAj6o_Q4yTCHjoBuJRhS0ks5rFtTggaJpZM4LGvtt
.

#4947


add qb passer rating calculation

I added from __future__ import division to the top of the file, I did a quick search and didn't see any other division that was relying on the standard python2 convention for dividing ints.

@monstermudder78 This looks great, thank you. Could you please remove tabs and use 4 space indents? :-) Tabs in Python is blasphemy (and mixing spaces & tabs is a big no-no). Once that's done, I'll merge. :-)

So sorry! Changed text editors and didn't fix the settings. I'll get right on it.

Hopefully fixing it through github's editor worked, if not let me know.

@monstermudder78 Looks much better. FYI, you can make changes locally and push to your qbrating branch and they will show up in this PR.
In any case, thank you! I'll pushing out the next release with this change included, 1.2.13, right now. It should be up in a few minutes.


Multi-screen recording failing

Hello! Thanks for this great tool!
I'm having trouble recording videos from multiple screens at once- it will run fine for up to several minutes then spontaneously freeze without throwing any errors. See the following code. After some troubleshooting I've found the script is getting stuck on the frame1 = cam1.get_latest_frame() line. I thought video_mode=True is non-blocking though. Any ideas what's going on?
One note: it's important in my application that accurate timestamps are saved for each frame, so I call get_latest_frame() for each screen before any are written to disk in order to keep the buffer grabs close in time. If there's a better way of doing this please let me know. And thanks for the help.
#create cameras
cam1 = dxcam.create(output_idx=2, output_color="BGR")
cam2 = dxcam.create(output_idx=1, output_color="BGR")
cam3 = dxcam.create(output_idx=3, output_color="BGR")

#input settings
target_fps = 60
output_path = "G:/posevideos/test/screen1"

#start cameras
cam1.start(target_fps=target_fps,video_mode=True)
cam2.start(target_fps=target_fps,video_mode=True)
cam3.start(target_fps=target_fps,video_mode=True)

#outpaths for each video
vidoutpath1 =  os.path.join(output_path, "screencam1.mp4")
vidoutpath2 =  os.path.join(output_path, "screencam2.mp4")
vidoutpath3 =  os.path.join(output_path, "screencam3.mp4")

#writers for each video
writer1 = cv2.VideoWriter(vidoutpath1,cv2.VideoWriter_fourcc(*"mp4v"),target_fps,(1024,600))
writer2 = cv2.VideoWriter(vidoutpath2,cv2.VideoWriter_fourcc(*"mp4v"),target_fps,(1024,600))
writer3 = cv2.VideoWriter(vidoutpath3,cv2.VideoWriter_fourcc(*"mp4v"),target_fps,(1024,600))

#record loop
print("Press q key to end recording.")
while(True):

  #grab a frame from every buffer.
  #timestamp recorded here
  frame1 = cam1.get_latest_frame()
  frame2 = cam2.get_latest_frame()
  frame3 = cam3.get_latest_frame()

  #once all frames are grabbed, save them
  w1 = writer1.write(frame1)
  w2 = writer2.write(frame2)
  w3 = writer3.write(frame3)

  #check for end keypress
  if keyboard.is_pressed('q'):
      print(" ")
      break

#stop and release the resources
cam1.stop()
cam2.stop()
cam3.stop()
writer1.release()
writer2.release()
writer3.release()

One extra piece of information- if I set video_mode=False then the first get_latest_frame() always blocks forever. Not sure if I'm doing something wrong, or...?

same issue for me. any chance for a solution?


installer windows gui

Add NSIS installer for Windows GUI
update changelog, docs, add tests for windows creator

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.73%. Comparing base (d793fad) to head (08dd9a1).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #43      +/-   ##
==========================================
+ Coverage   99.72%   99.73%   +0.01%
==========================================
  Files          24       25       +1
  Lines        1435     1504      +69
==========================================
+ Hits         1431     1500      +69
  Misses          4        4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Fix ..com typo in Safe Links domain

The wrapping domain  nam01.safelinks.protection..com contains .. before com which is not a valid domain.

Additionally, the domain seems to be incorrect, unless Microsoft own protection.com?
protection.com does not seem to have any relationship with Microsoft. Should this domain be safelinks.protection.outlook.com instead?

@denisebmsft

Additionally, the domain seems to be incorrect, unless Microsoft owns protection.com? If they do, the relationship is certainly not obvious from the contents of protection.com
Should this domain be safelinks.protection.outlook.com instead?

As @halmartin mentioned, it doesn't look like Microsoft owns protection.com, based on what's hosted there and whois.

Do you know what the actual URL is? Shouldn't this be nam01.safelinks.protection.outlook.com instead of nam01.safelinks.protection.com (the thing that's merged now)?

I updated the content to match observed reality:

Definitely protection.outlook.com.
The namXX' part doesn't appear to be consistently 'nam01' (it likely depends on your tenant and which datacenter they're in). Unclear if worldwide datacenters could also be involved.

I updated the text to account for knowns and unknowns, which should be accurate either way:

Safe Links scans incoming email for known malicious hyperlinks. Scanned URLs are rewritten or wrapped using the Microsoft standard URL prefix: https://<DataCenterLocation>.safelinks.protection.outlook.com (for example, https://nam01.safelinks.protection.outlook.com). After the link is rewritten, it's analyzed for potentially malicious content.

Feel free to make additional changes


WORLD and LOCAL_WORLD_ALIGNED reference for force / impulse cost functions

It is important to allow users to define various reference for contact force / impulse cost functions as well. This will require tricky conversion of forces --and their derivatives-- from LOCAL to WORLD and LOCAL_WORLD_ALIGNED. Note that our contact dynamics computes everything at the LOCAL level.
This task could be done potentially done inside Pinocchio (i.e. the new contact dynamics). @proyan, @jcarpent  have you though about it? Could you add this in the new contact dynamics of Pinocchio? or do we need to handle it here?
In any case, @jcarpent could you help me with this task?

This is handled in #1123.


[Subtask] Add Catalog Error Handler and related exceptions, test cases in client-python

Describe the subtask
Add Catalog Error Handler and related exceptions, test cases in clinet-python based on client-java
Parent issue
#4225

Hi, may I continue working on this issue?

@jingjia88 Sure, thanks a lot.


Add version directly into the code

Add version directly into code, this is a very classic and handy pattern in classic libs like lodash.

Okay, David. If you insist.


Use pre-commit for continuous integration

This PR integrates pre-commit for continuous integration. Contributors can first activate it locally by pre-commit install, which will then be triggered automatically at git commit to check, fix and reformat the commit.

@CCInc, the advantages are mainly two-fold:

It allows the contributors to perform the check locally before pushing the changes to the repo.
It supports more than just reformatting the code: it also checks some type errors (see https://github.com/mit-han-lab/torchsparse/blob/dev/pre-commit/.pre-commit-config.yaml for more details).

@CCInc @sandeepnmenon, this PR is ready for review. @kentangSJTU, I've tested the correctness of this version on semantic segmentation. It would be great if you could test it on object detection as well.


Identify user testing requirements

Summary
If ethics approval is received - need to pick key parts of the platform to go through user testing
This is a summary following the July 2023 meet up (Google doc) and can be edited as needed
Deadline
September 2023
What needs to be done?
Features to investigate

[ ] Understandability of CoC
[ ] The overall UX and workflow of adding stories
[ ] Viewing stories
[ ] Can you change permission
[ ] Can you share an experience
[ ] Do people understand what sharing their experiences mean
[ ] User testing the copy on the homepage (do they understand the ask) - Not sure what that means

Who can help?
Anyone interested
Updates

I've linked back to #517 which had a few suggestions for the overall planning, and there's a Google Doc Georgia made for planning.
Regarding the last point on "do people understand the ask": What I meant was whether people understand what we would like them to do when engaging with AutSPACEs. A typical user-testing approach might be to give people clear asks of "can you use our product to do XYZ". But, would a new user themselves find out what we would like them to do/what the overarching goal of AutSPACEs is from just looking at the platform or not?

@GeorgiaHCA and I just had a co-working session on the usability testing have come up with a rough outline and the tasks here: https://docs.google.com/document/d/1P6Lw-2vl0CamEYH3G1wbHRQxWX4M4HmHXUd4cp3CSrM/edit

The two rounds of user-testing are now done! 🚀


AllOrdersRequest

AllOrdersRequest miss some params that are mandatory to enable paging.
docs https://github.com/binance/binance-spot-api-docs/blob/master/rest-api.md#all-orders-user_data
startTime | LONG | NO |
-- | -- | -- | --
endTime | LONG | NO
seems missing on api

Don't forget #339


Add ahschools.us

@alucard0134 Pull request merged. Please notice that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age. Thank you!


Uploading new dataset (no version on GCP yet) fails

gcpl._get_metadata_gcp fails if no previous version of dataset exists on gcp
Desired result:

[ ] New datasets can be uploaded
[ ] Real error are still caught
[ ] Add test for new dataset (delete all GCP versions on test env before uploading)

PR #83 fixes the core bug. Test should still be added.
I am not sure what I meant by "real errors should still be caught" - I think it should be "skipping datasets should still occur when relevant".


Resource exhausted: OOM when allocating tensor with shape...

Hi when I run Jonathan K. Kummerfeld's notebook. When run the "Run prediction" section, line:
! GPU=0 python predict.py $CHOSEN_MODEL sample.in.json sample.out.txt
it shows the following:
2020-10-08 16:48:11.126861: W tensorflow/core/common_runtime/bfc_allocator.cc:319] ___________________________________*************************************
2020-10-08 16:48:11.134264: W tensorflow/core/framework/op_kernel.cc:1502] OP_REQUIRES failed at tile_ops.cc:220 : Resource exhausted: OOM when allocating tensor with shape[202893,17445] and type int32 on /job:localhost/replica:0/task:0/device:GPU:0 by allocator GPU_0_bfc
Traceback (most recent call last):
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py", line 1356, in _do_call
return fn(*args)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py", line 1341, in _run_fn
options, feed_dict, fetch_list, target_list, run_metadata)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py", line 1429, in _call_tf_sessionrun
run_metadata)
tensorflow.python.framework.errors_impl.ResourceExhaustedError: 2 root error(s) found.
(0) Resource exhausted: OOM when allocating tensor with shape[202893,17445] and type int32 on /job:localhost/replica:0/task:0/device:GPU:0 by allocator GPU_0_bfc
[[{{node Tile_1}}]]
Hint: If you want to see a list of allocated tensors when OOM happens, add report_tensor_allocations_upon_oom to RunOptions for current allocation info.
 [[TopKV2/_1997]]

Hint: If you want to see a list of allocated tensors when OOM happens, add report_tensor_allocations_upon_oom to RunOptions for current allocation info.
(1) Resource exhausted: OOM when allocating tensor with shape[202893,17445] and type int32 on /job:localhost/replica:0/task:0/device:GPU:0 by allocator GPU_0_bfc
[[{{node Tile_1}}]]
Hint: If you want to see a list of allocated tensors when OOM happens, add report_tensor_allocations_upon_oom to RunOptions for current allocation info.
0 successful operations.
0 derived errors ignored.

There was some software dependency issues that have now been fixed. You may want to try again.


Not able to save file nor rename files in Jupyter Notebook gateway

I installed Dataproc with a simple command:
gcloud beta dataproc clusters create presto \
--optional-components=ANACONDA,JUPYTER,PRESTO \
--zone europe-west3-a --master-machine-type n1-standard-2 --master-boot-disk-size 500 \
--num-workers 2 --worker-machine-type n1-standard-2 --worker-boot-disk-size 500 \
--num-preemptible-workers 1 \
--image-version 1.4-debian9 \
--enable-component-gateway

Once, Dataproc is installed opened Jupyter notebook gateway and tried creating a new notebook and tried renaming it, but I got error:

Then, tried creating new Folder it also gave an error:

Initializing and creating the cluster is just scripts right, why it's behaving differently every damm time?
A lot of problems:
Stuck at interpreter initializing
Unable to save the code, just closing the notebook window will delete [Not saving the code]
Lack of documentation, For example, how can a user import library dependency lets say 'spark-cassandra-connector:2.4.1-s_2.11'. In python, it's very easy just !pip install package and docs of python jupyter notebook was good but dataproc is ...

This is an issue with Component Gateway that we're working on fixing in the next couple weeks. I believe the underlying cause is that our deployment of Knox doesn't allow PATCH requests by default.
Renaming jupyter notebooks still works if you use the old ssh tunnel method (edit and run launch-jupyter-interface.sh).

Seems to have been fixed over a month ago: https://cloud.google.com/dataproc/docs/release-notes#may_6_2019


Uploading ODM files directly to FHIR server fails

Although the server address http://localhost:8090/fhir/metadata can be reached locally, I get following errors when I run the Docker image with the option --fhir.server.url=http://localhost:8090/fhir:
2021-02-19 10:20:42.417  WARN 1 --- [           main] d.d.u.o.f.writer.FHIRBundleServerWriter  : Attempt 1 to send bundle failed with error

ca.uhn.fhir.rest.client.exceptions.FhirClientConnectionException: Failed to retrieve the server metadata statement during client initialization. URL used was http://localhost:8090/fhir/metadata

By default, Docker uses a bridge network for containers and because of that localhost refers basically to the container itself and not to the (Docker) host. To solve your issue you can either specify the IP address instead of localhost or you can make Docker use the host network by adding --network=host as parameter to the run command.


Added social programming sites

As proposed in #60
Sorry for the delay, though!

JSON doesn't like trailing commas.

Uh-oh. Fixed!


Is the project abandoned?

Hi,
I see that the project hasn't had any commits for the last 5 months. Is the project still under active development?

maybe, it is currently WIP

Any updates on this?


Allow for scalar sample_domain

It would be very practical to allow for scalar value for instance when trying to visualize decision foinctions
ypred = clf.predict(X,sample_domain=1)

should assume that all samples in X come from domain 1.

I think it is the same issue as in #143


add test for regression and fix directory exclusion without wildcards

Fixes #488
I've added a test to make sure that both wildcard directory exclusion and the 1.5.1 version of directory exclusion work.

I tested locally, and unfortunately this isn't restoring the exclude behavior that was available on 1.5.1.
1.5.1:
browne-a02:workspace browne$ bandit/.tox/py36/bin/bandit -r column/ -x tests`

[main]	INFO	profile include tests: None
[main]	INFO	profile exclude tests: None
[main]	INFO	cli include tests: None
[main]	INFO	cli exclude tests: None
[main]	INFO	running on Python 3.6.7
Run started:2019-05-10 05:13:12.790455

Test results:
	No issues identified.

Code scanned:
	Total lines of code: 1100
...

1.6.0+PR489
browne-a02:workspace browne$ bandit/.tox/py36/bin/bandit -r column/ -x tests/
[main]	INFO	profile include tests: None
[main]	INFO	profile exclude tests: None
[main]	INFO	cli include tests: None
[main]	INFO	cli exclude tests: None
[main]	INFO	running on Python 3.6.7
Run started:2019-05-10 05:19:46.158749

Test results:
	No issues identified.

Code scanned:
	Total lines of code: 1395
...

However, it I prepend column, then the exclude does work.  But that wasn't required before in 1.5.1
browne-a02:workspace browne$ bandit/.tox/py36/bin/bandit -r column/ -x column/tests
[main]	INFO	profile include tests: None
[main]	INFO	profile exclude tests: None
[main]	INFO	cli include tests: None
[main]	INFO	cli exclude tests: None
[main]	INFO	running on Python 3.6.7
Run started:2019-05-10 05:19:51.807202

Test results:
	No issues identified.

Code scanned:
	Total lines of code: 1100

Thanks for the review, when I get the chance I'll add more tests and fix for this.

@ericwb This PR is ready for a re-review

@mattjegan Any update on the changes requested by @ericwb? It would be awesome to get this polished up and merged to release 1.6.1

@tylerwince I haven't had a chance to take a look yet but might get some time this evening.

@ericwb I have fixed the styling issue however I was unable to reproduce your example when ran against column HEAD (https://github.com/vmware/column/commit/86c72b29e8c1ec58474bd39757f18a3834a446ef). If I'm missing something in my testing please let me know.
(.venv) ➜  eganm bandit --version         bandit 1.5.1
  python version = 3.6.5 (default, Jun 17 2018, 12:13:06) [GCC 4.2.1 Compatible Apple LLVM 9.1.0 (clang-9<IP_ADDRESS>)]
(.venv) ➜  eganm bandit -r column -x tests
[main]  INFO    profile include tests: None
[main]  INFO    profile exclude tests: None
[main]  INFO    cli include tests: None
[main]  INFO    cli exclude tests: None
[main]  INFO    running on Python 3.6.5
Run started:2019-05-16 11:14:14.010747

Test results:
        No issues identified.

Code scanned:
        Total lines of code: 1100
        Total lines skipped (#nosec): 0

Run metrics:
        Total issues (by severity):
                Undefined: 0.0
                Low: 0.0
                Medium: 0.0
                High: 0.0
        Total issues (by confidence):
                Undefined: 0.0
                Low: 0.0
                Medium: 0.0
                High: 0.0
Files skipped (0):
(.venv) ➜  eganm bandit/.tox/py36/bin/bandit --version         bandit 1.6.1
  python version = 3.6.5 (default, Jun 17 2018, 12:13:06) [GCC 4.2.1 Compatible Apple LLVM 9.1.0 (clang-9<IP_ADDRESS>)]
(.venv) ➜  eganm bandit/.tox/py36/bin/bandit -r column/ -x tests
[main]  INFO    profile include tests: None
[main]  INFO    profile exclude tests: None
[main]  INFO    cli include tests: None
[main]  INFO    cli exclude tests: None
[main]  INFO    running on Python 3.6.5
Run started:2019-05-16 11:14:32.925246

Test results:
        No issues identified.

Code scanned:
        Total lines of code: 1100
        Total lines skipped (#nosec): 0

Run metrics:
        Total issues (by severity):
                Undefined: 0.0
                Low: 0.0
                Medium: 0.0
                High: 0.0
        Total issues (by confidence):
                Undefined: 0.0
                Low: 0.0
                Medium: 0.0
                High: 0.0
Files skipped (0):

@mattjegan Sorry for my late reply.  It does look like you're testing the same way as me.  Let me retry again.

I tried this again, in Linux env instead of Mac and it checks out.

An ever-increasing number of projects are needing to add and backport 1.6.0 exclusions until this fix appears in a new release. Any chance of tagging something along the lines of a 1.6.1 hotfix release in the near future?


perf(semantic): reduce AstNodeId to u32

AstNodeId was a usize. This seems excessive.
Parser has a limit on size of a JS file of 4 GiB. While it is possible for a JS file of that size to create an AST with more than 1 << 32 (~4 billion) AST nodes, that would be insanely large.
So make AstNodeId u32 instead.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#4264  👈
#4263
#4262
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on  Graphite

Now we're talking!

@rzvxa When we were chatting a while back, you argued against making it a u32, in order to support very large files. However, I think it's hard to argue against this kind of performance boost. What do you think?

@rzvxa When we were chatting a while back, you argued against making it a u32, in order to support very large files. However, I think it's hard to argue against this kind of performance boost. What do you think?

Yep, We can add a feature later on for large-file-support that would use usize.


Fix unit summaries for listings directory

Unit summaries got changed after I worked on then and path have been changed. This one updates the path.

Tested locally and LGTM.


Why is the link to a class missing in the index of the module page?

Search terms
module index page
Question
This is related to my typedoc plugin which can merge the content of modules and using it on a monorepo project.
Without using the plugin on the monorepo project the main page looks like this:

With using the plugin to merge modules (in this case they equal the projects having the same name) the module page looks like this:

As you can see class "C" is missing in the index but shows up in the navigation on the left. The question is why?
What I think the problem is:
When the plugin is used with TypeDoc's entryPointStrategy set to packages the plugin is listening to the even Application.EVENT_PROJECT_REVIVE and merges the modules when this event is emitted by TypeDoc: https://github.com/krisztianb/typedoc-plugin-merge-modules/blob/master/src/plugin.ts#L63
As I understand the code TypeDoc uses a CategoryPlugin to create the model used for rendering the module index. It seems to me that the index is generated before the plugin is merging the modules. Therefore the index is outdated in the end. Do you think that this is the case? If so how could this be fixed?
How to reproduce
I added automated tests to the plugin repo which can be used to reproduce the problem: https://github.com/krisztianb/typedoc-plugin-merge-modules/tree/master/test/merge-module-monorepo
You can use npm install followed by npm run build and npm test to run the tests and see the result.

Your guess at what the problem is is exactly correct, categorization happens during Converter.EVENT_RESOLVE_END, so it has actually already been done for each individual project. Once projects are merged together, the existing categories are assumed to be correct.
Fixing this is going to be somewhat annoying. @category tags are removed during categorization, and TypeDoc doesn't re-run categorization during project revival. You probably get 90% of the way there by manually checking and merging module.categories (should also check for module.groups to ensure all values of the navigation option work as expected)
... but that still leaves the other 10%. TypeDoc uses the sort option to decide how to sort reflections within modules/groups/categories, and the sorting function isn't exposed (I guess this is an argument that it should be, which means I have to figure out where to put it...) seems kind of weird to directly export getSortFunction...

I haven't a reason for something like this to be included in TypeDoc itself, looking at the use cases in the plugin's readme - generating documentation for applications is something that's never made sense to me... why not just look at the code? Can't merging modules be done already via re-exports, so that the documentation actually matches the library exports?

Thanks for your feedback. I agree that exporting the sort function seems kind of strange. Maybe the functionality of the plugin should be part of TypeDoc itself?
Another solution could be to somehow invoke the sorting of the categories from the plugin. So the plugin can merge the categories and then get them sorted by TypeDoc.

Totally please :) specially in monorepos

I haven't a reason for something like this to be included in TypeDoc itself, looking at the use cases in the plugin's readme - generating documentation for applications is something that's never made sense to me... why not just look at the code? Can't merging modules be done already via re-exports, so that the documentation actually matches the library exports?

I can tell our specific use case.
We have a library in a monorepo divided by contexts, like auth or region.
Inside each context we have different packages, like in hexagonal architecture. This is to take full advantage of NX, tool to handle the monorepo.
We then expose everything through a single entry point but that entry point cannot be used to generate the typings.
Repo
|- auth
|     | - domain
|     |        | - package.json
|     | - application
|              | - package.json
| - entry
| - auth
Some modules are compiled to js due to technical restrictions and in 'entry' there is no access to the typescript source, so I can only generate the doc for each independent package, but in the typedoc output I don't want to separate domain and application, I only want to have auth.

I don't much like it, but that's a pretty good reason for this being in typedoc itself.


Add 'CsWinRTEnableIDynamicInterfaceCastableSupport' feature switch

This PR adds the 'CsWinRTEnableIDynamicInterfaceCastableSupport' feature switch to allow trimming of all IDIC-related code. This would also include fixing the size regression introduced by #1566 (which can't be fixed without making the code non trim-safe).

[!NOTE]
Opening as draft until I get a sizoscope diff.

This would break As<T> right?

Only if it gets to the fallback IDIC cast from IInspectable to the interface type, otherwise it'll still work.

Only if it gets to the fallback IDIC cast from IInspectable to the interface type, otherwise it'll still work

The only case this would be true is if the interface is using built-in COM Interop, which already doesn't support NativeAOT anyway.

Saves 52 KB in the minimal sample, plus more incremental savings as you have more enumerable types 🚀


[ie/RTS] Support new URLs; drop old formats

IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information

Fixes #2779; closes #5275. See #9827 and #7363 if we can do better.
The change in this PR is very aggressive. I've almost completely deleted the very old extraction code.
All URLs have been tested in Switzerland, but still cannot cover all code paths.
Template

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

[x] I am the original author of this code and I am willing to release it under Unlicense
[x] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence) This PR supersedes #5275.

What is the purpose of your pull request?

[x] Fix or improvement to an extractor (Make sure to add/update tests)
[ ] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

We have timestamp with value -40280400 in the item of a playlist test (http://www.rts.ch/archives/tv/divers/3449373-les-enfants-terribles.html). To pass the test on Windows, we need #11902 to be merged.


Slow Block and BlockProposal Handlers

I observe in my log slow handling times being reported from the peer loop. I'm not sure this is a cause for concern; just drawing attention to the issue.
2024-11-26T10:29:36.119793376Z  INFO ThreadId(02) neptune_core::peer_loop: Got new block from peer [2a02:21b4:b885:3f00:ea4c:2826:2dce:6cec]:9798, height 733, mined 2024-11-26T11:29:34.501+01:00
2024-11-26T10:29:36.517401459Z  INFO ThreadId(03) neptune_core::peer_loop: Block with height 733 is valid. mined: 2024-11-26T11:29:34.501+01:00
2024-11-26T10:29:36.517480949Z  INFO ThreadId(03) neptune_core::peer_loop: Updated block info by block from peer. block height 733
2024-11-26T10:29:36.517496989Z  INFO ThreadId(03) neptune_core::peer_loop: Rewarding peer 2a02:21b4:b885:3f00:ea4c:2826:2dce:6cec for ValidBlocks(1)
2024-11-26T10:29:36.517518379Z  WARN ThreadId(03) neptune_core: executed handle_peer_message()::PeerMessage::Block in 0.397752653 secs.  exceeds slow fn threshold of 0.001 secs.  location: /home/alan/neptune-core/src/peer_loop.rs:529:17
2024-11-26T10:29:36.52880389Z  WARN ThreadId(01) neptune_core: executed handle_peer_task_message()::PeerTaskToMain::NewBlocks in 0.011248181 secs.  exceeds slow fn threshold of 0.001 secs.  location: /home/alan/neptune-core/src/main_loop.rs:560:17

2024-11-26T10:31:45.685763553Z  INFO ThreadId(02) neptune_core::peer_loop: Got block proposal from peer.
2024-11-26T10:31:46.056667888Z  INFO ThreadId(02) neptune_core::peer_loop: Rewarding peer 2a02:21b4:b885:3f00:ea4c:2826:2dce:6cec for NewBlockProposal
2024-11-26T10:31:46.056714418Z  WARN ThreadId(02) neptune_core: executed handle_peer_message()::PeerMessage::BlockProposal in 0.370938875 secs.  exceeds slow fn threshold of 0.001 secs.  location: /home/alan/neptune-core/src/peer_loop.rs:
1125:17

It shouldn't be a problem unless you also see a slow-lock warning.  especially a slow write-lock warning.  Fortunately I haven't been seeing these much.
I could remove or alter some of the slow-scope warnings now that are kind of false positives.  It's only really a problem for async code when it calls "slow" blocking code that does not await.
That said, each of these warnings is a reminder that we could speed things up.   For example I know there is still some un-cached tip5 hashing taking place, eg for Utxos.  Probably plenty of other low-hanging fruit once we fire up some profiling tools.


Updating evidence parsers for intogen.

There's a new release for intogen on 31th May. Given how infrequently Intogen releases, we could assume the update comes with a series of schema changes.
Tasks:

[x] Download new data
[x] Sort out existing and new data in the input bucket (gs://otar000-evidence_input/IntOgen/data_files)
[x] Update parser accounting for changes in the schema
[x] Investigate generated evidence file to identify new cancer types that need to be mapped to EFO
[ ] Update disease mapping file
[ ] Update config file
[ ] Run snakemake rule with new config
[ ] Run validation

Updates: the progression can be followed in this draft PR

Stats based on the new data:

Old evidence count: 3333
New evidence count: 4360
Unmapped evidence count: 1968
Unmapped disease count: 45

Questionable mappings:

Plasma Cell Myeloma -> multiple myeloma.
uveal melanoma mapped to HP_0007716 and EFO_1000616 they have the same labels.

All suggestion from Ontoma make sense.
It also makes sense for me that we assume cancers for the less confident ones.
As for the non mapped ones, only one suggestion:

For Gastrointestinal Neuroendocrine Tumors, I'd rather suggest Digestive System Neuroendocrine Tumor

Good. Thank you!

Some more stats highlighting improvements:

old
new

evidence_count
3333
4360

association_count
2080
2630

target_count
568
619

disease_count
66
71

cohort_count
214
254

Interesting the new dataset has lost 660 associations, while gaining 1210. It might worth checking why we are missing associations. The 660 lost association belongs to 54 distinct diseases, 17 of them were not found in the new dataset altogether while 37 have new associations.

The data is prepared, ready to be picked by ETL (gs://otar000-evidence_input/IntOgen/json/intogen-2023-06-02.json.gz).
Code in the evidence datasource parser repo is reviewed and merged.

New significantDriverMethods:
+-----------------------+
|significantDriverMethod|
+-----------------------+
|oncodriveclustl        |
|combination            |
|smregions              |
|mutpanning             |
|cbase                  |
|hotmaps                |
|dndscv                 |
|oncodrivefml           |
+-----------------------+

All terms were already found in the old intogen dataset, so no need to update front-end mappings.


test commit

Current dependencies on/for this PR:

develop

PR #132   👈

This stack of pull requests is managed by Graphite.


Feature Request: Please add ability to open new tasks directly from Toggl Desktop

Currently, It's possible to create new projects, but for tasks - we must go to the website,
Task is created (At least on my usecase) a lot more often - and in lot of the cases, times are recorded on a wrong task just because opening new tasks requires entering the website

Thank you for the request. We try to keep the management part in the Desktop app to a minimum and that's why we have not added many of the things present in the web to the app.
We'll take this request under consideration when planning new features.


Defining the same variant across multiple styles not working

Description
Hello!
I am implementing a stylesheet with 3 styles, each one of them is implementing the same variants as described in the documentation: https://v2.unistyl.es/reference/variants/#defining-the-same-variant-across-multiple-styles
However, the inferred type is not being consolidated, causing problems when trying to extend the type in an interface.
Steps to Reproduce
Code to reproduce:
const stylesheet = createStyleSheet(theme => ({
  root: {
    justifyContent: 'center',
    alignItems: 'center',

    variants: {
      orientation: {
        horizontal: {
          flexDirection: 'row',
          marginVertical: 20,
        },
        vertical: {
          flexDirection: 'column',
          marginHorizontal: 20,
        }
      }
    }
  },
  line: {
    flex: 1,
    backgroundColor: theme.colors.border,

    variants: {
      orientation: {
        horizontal: {
          height: 1
        },
        vertical: {
          width: 1
        }
      }
    }
  },
  text: {
    color: theme.colors.border,

    variants: {
      orientation: {
        horizontal: {
          marginHorizontal: 10,
        },
        vertical: {
          marginVertical: 10,
        }
      }
    }
  }
}));

type DividerVariants = UnistylesVariants<typeof stylesheet>;

Infered type:

Error:

Snack or Repository Link (Optional)
No response
Unistyles Version
2.20.0
React Native Version
0.76.3
Platforms
Android, iOS, Web
Expo
Yes

Hey @fabcstro this is how TypeScript work and we can't do anything about it.
https://www.totaltypescript.com/concepts/an-interface-can-only-extend-an-object-type
To fix it, change interface to type and all the types will be inferred correctly.


upgrade flapdoodle process to 1.50.0

The migration is easy and tests are passing, but I get alot of warnings in the logs.
lets discuss this.

Not sure what warnings you are talking about, but migration without any functional changes yielded me same results.
I also migrated to ExtractedArtifactStore which was a little harder, but then again it gives us caching out of the box, so worth it. see https://github.com/wix/wix-embedded-mysql/commit/c7213d074b4ffdce12dd9a195a622ebf5fe34a7a
Now just have to extract that pesky PackagePaths to property files and bind them to versions:)

@viliusl it keeps complaining it cannot shut down the process, false positive, but still, we need to fix that as well.

@noam-almog - do you see those errors in your project or wix-embedded-mysql builds? sample project please, because I'm not seeing that anywhere right now.


Moved rigetti integration test logic to higher level conftest.py file…

… and included slow marker logic.

@pavoljuhas Created this pull request for issue #6211 . This is my first pull request, if I have made any mistake in the process kindly guide me.

@nafaynajam - thank you for contributing, I will do the review later today.

BTW, I have verified the default test selection and the --rigetti-integration option behave as before.

Partial fix of #6211


matterjs can't support  WebAssembly？

matterjs can't support  WebAssembly？

I'm not sure what you're asking here, can you explain?


static objects don't work on JS

shared void run() {
    print(C.o); // prints null
}

class C {
    shared static object o {}
    shared new () {}
}

@chochos this one is for you I guess...

Must have been fixed along with some other issue, because this now prints C.o@0


DPDK VPP: bugfixes for new versions and SUSE

SLES doesn't have vpp packages available for some versions, so check if there are none after adding the repository and refreshing. Skip if there is not a supported package available.

Fix interface check to look for 'VSC' as well as 'VirtualFunction' since names can differ between versions.

@mcgov please update PR when you have time, thanks!

@mcgov please update PR when you have time, thanks!

@mcgov could you please update this PR?

@mcgov feel free to create new one or reopen it when it is ready, thanks


Beam.pro site still listed in files

the Beam.pro site is still listed in files.  the api call is https://mixer.com/api/v1/ not https://beam.pro/api/v1/

This was last updated from before the rebranding from Beam to Mixer. All you need to do is change beam.pro to mixer.com and should be ok


Update Getting Started With Hubot doc link.

Help link had gone dead; here's an update.

Automated message from Dropbox CLA bot
@brainwane, thanks for the pull request! It looks like you haven't yet signed the Dropbox CLA. Please sign it here.

Merged, thanks @brainwane!

Automated message from Dropbox CLA bot
@brainwane, thanks for signing the CLA!


New Version

Hi, thanks for everything about this mod. When will come new version ? and what changes the new version?

The new version will be 2.00 - it is in the making. There will be lots of changes. When we release it there will be a release post on what is changed,

Closing issue as I plan to release 2.00 on the 1st of May


[Bug]: androidx.credentials.exceptions.NoCredentialException: No credentials available

General Info

[X] I checked for similar bug report
[X] I am using the latest version
[X] I checked the troubleshooting page for similar problems

Version(s)
2.5.2
Kotlin Target(s) and their respective versions
2.0.0
What happened? (include your code)
I upgraded the version to 2.5.2 because i want to use Google Sign In on iOS but after upgrade from 2.5.0 this error appear on Android
when execute composeAuth.rememberSignInWithGoogle().startFlow()

Credential exception
androidx.credentials.exceptions.NoCredentialException: No credentials available
at androidx.credentials.CredentialProviderFrameworkImpl.convertToJetpackGetException$credentials_release(CredentialProviderFrameworkImpl.kt:276)
at androidx.credentials.CredentialProviderFrameworkImpl$onGetCredential$outcome$2.onError(CredentialProviderFrameworkImpl.kt:152)
at androidx.credentials.CredentialProviderFrameworkImpl$onGetCredential$outcome$2.onError(CredentialProviderFrameworkImpl.kt:143)
at android.credentials.CredentialManager$GetCredentialTransport.lambda$onError$2(CredentialManager.java:694)
at android.credentials.CredentialManager$GetCredentialTransport.$r8$lambda$nlbYav9mLBoE6Yh1vFKCvITF3ks(Unknown Source:0)
at android.credentials.CredentialManager$GetCredentialTransport$$ExternalSyntheticLambda2.run(Unknown Source:6)
at androidx.credentials.CredentialManager$$ExternalSyntheticLambda0.execute(Unknown Source:0)
at android.credentials.CredentialManager$GetCredentialTransport.onError(CredentialManager.java:693)
at android.credentials.IGetCredentialCallback$Stub.onTransact(IGetCredentialCallback.java:123)
at android.os.Binder.execTransactInternal(Binder.java:1344)
at android.os.Binder.execTransact(Binder.java:1275)

Steps To Reproduce (optional)
No response
Relevant log output (optional)
No response

@Aaron246 With the old solution, you could just set setFilterByAuthorizedAccounts to false and retry the request. Any idea how to do that with GetSignInWithGoogleOption?

@Aaron246 This issue is probably caused by the Google Sign In change, any ideas?

I don't think the change has anything to do with this.
@Deorigami can you please check all of these for your project:

You have 2 credentials in google cloud console, one web application and one android.

You are using the client ID in web application (the first one in the above screenshot) for

install(ComposeAuth) {
googleNativeLogin(serverClientId = "google-client-id")
}
in your android project and in the supabase console here

3. The sha-1 certificate and package name in the android credential on the google cloud console (the second credential on google cloud console) is same as the app you are testing right now i.e. if you are testing on debug version or the release version make sure the respective sha-1 key is used in the google cloud console here :

If all of these are correct, the issue seems to be with the new Credential Manager API which is being used in ComposeAuth.
Lot of people are facing issues with the API you can see here.

I upgraded the version to 2.5.2 because i want to use Google Sign In on iOS but after upgrade from 2.5.0 this error appear on Android
Just to clarify @Deorigami, does 2.5.0 work?

@jan-tennert Yes 2.5.0 works well on android without this issue, i have to rollback to 2.5.0

Hmm, just tried updating one of my old projects and it seems to work fine. Not sure what we can do on this issue other than referring to Google


Add MesonGS method

MesonGS: Post-training Compression of 3D Gaussians via Efficient Attribute Transformation [ECCV 2024]
https://shuzhaoxie.github.io/mesongs/

Hey @ShuzhaoXie, it would be great to have your method in the comparison of 3DGS compression methods! For it to appear in the table and the plots of the compression survey, please provide your resulting numbers in your own repo in the format documented here:
https://github.com/w-m/3dgs-compression-survey?tab=readme-ov-file#including-your-own-results

MesonGS is listed on the survey.


Portfolio site

Hi, I want to add a portfolio website to this repo, if u find it valuable then please allow me with hacktober tag.

Hi, I want to add my portfolio website, can I?

Hii, I want to Contribute in ur Repository. I want to add my portfolio to this repo. Will u allow???

Guys we have multiple portfolio website requests. So, Lets discuss first by LIVE DEMO.

You can Check Website at https://www.rakshitmehra.me/portfolio2. Hope for the Positive reply :)


In which part of the code does the bsgs mode work?

In which part of the code does the bsgs mode work?

Well the function name is thread_process_bsgs but if you are asking for it, i doubt that it will be useful for you


Add confirmation table when signing with hermit

Similar to hardware wallet signing displays a table of information to verify against hermit.  Scrolling turned off if hermit to keep the scanner window on screen.

Closing due to inactivity.


fix: comment out, instead of skipping, intermittently failing test

skip seems to working only sometimes. It intermittently fails on Windows in CI. The reason for skipping the test in the first place was to prevent intermittent failures of the test itself, so adding skip did not improve the situation, unfortunately.

Sorry for the incorrect suggestion. Also thanks for the explanation why you decided to get rid of it.


[Chip] Update style to match the specification

There is probably still work to do, it's one step closer to the spec.
Also add a disabled prop: Closes #5164

The vertical text alignment is off:

...but prhaps we should remove the text avatar examples, since there are non in the spec?

@mbrookes I was wondering about it. I figure that it still look OK and that it doesn't cost us anything to support it. Would you agree with this conclusion?
I should have solved the alignment issue.

Using a single character is a bit of a fudge, since the primary use-case for characters is to display a user's initials in place of an image.

Google Contacts:

Check mate. 👑

However... the text alignment will still be off in any app that does use two characters, so perhaps it should be fixed.

@mbrookes To be clear, the alignment issue was solved with the introduction of lineHeight: 1 in the style of the Avatar (following the strategy of the Badge). I don't have a strong preference, I have noticed Vuetify using a single character, figured it looks nice. If you think two characters is better, let's revert :).

the alignment issue was solved with the introduction of lineHeight: 1

Sorry, must have skimmed the diff too quickly.

I don't have a strong preference

Neither do I, so I think "what does Google do" wins.


Feature Request - Alphanumeric field

Hello,
Will there be an alphanumeric field in the future?
{{ alphanumeric 10 }}
KLK893KLE0

Yes I've thought about this! I'd like to do something using regex syntax so users can define an alphanumeric pattern, eg: [A-Z]{2}[0-9]{3}[A-Z]{2} but I need to set aside some time to investigate it.
In the meantime I've thrown together a customer helper you can use that'll generate a completely random pattern of characters:
const customHelpers = {
  alphanumeric(count) {
    let string = '';
    for (let i = 0; i < count; i++) {
      string += dummyjson.utils.random() > 0.5
        ? dummyjson.utils.randomChar()
        : dummyjson.utils.randomInt(0, 9)
    }
    return string;
  }
}

const template = `{{alphanumeric 10}}`
const result = dummyjson.parse(template, { helpers: customHelpers });