UTF-8 encoding isn't assumed for application/json unless charset=utf-8 is also present

From<EMAIL_ADDRESS>on May 11, 2013 09:19:43
Version of google-http-java-client (e.g. 1.15.0-rc)? 1.14-beta Java environment (e.g. Java 6, Android 2.3, App Engine)? Android 4.0.4 Describe the problem. A Django/Tastypie server returns only "Content-Type: application/json" with no charset parameter at the end. According to the bug report, they believe that this is correct as per RFC4627 (JSON) ( https://github.com/toastdriven/django-tastypie/issues/717 ).
Within google-http-java-client, HttpResponse.getContentCharset() assumes an ISO-8859-1 encoding where there is no charset parameter within the returned Content-Type field. For "application/json" it should in fact assume the UTF-8 encoding that is the default according to http://www.ietf.org/rfc/rfc4627.txt How would you expect it to be fixed? If the mime type and subtype are application and json respectively, then HttpResponse.getContentCharset() should return a UTF-8 charset instead of ISO-8859-1 as per the JSON RFC 4627
Original issue: http://code.google.com/p/google-http-java-client/issues/detail?id=221

Workaround:
MobileApi.Builder builder = new Builder(...);
builder.setHttpRequestInitializer(new HttpRequestInitializer() {
        @Override
        public void initialize(HttpRequest request) throws IOException {
            request.setResponseInterceptor(new HttpResponseInterceptor() {
                @Override
                public void interceptResponse(HttpResponse response) throws IOException {
                    // Decode JSON as UTF-8 if no charset is specified.
                    if (response.getContentType().equals("application/json") &&
                            response.getMediaType().getCharsetParameter() == null) {
                        response.getMediaType().setCharsetParameter(Charset.forName("UTF-8"));
                    }
                }
            });
        }
    });


Commands not rendered in Explorer and Test tabs

Issue Type: Bug
Some commands in the File Explorer tab are not rendered as links or buttons but are displayed with a MarkDown-like syntax.

The same issue happens with the Test tab when using Python files.

VS Code version: Code 1.63.0 (7db1a2b88f7557e0a43fec75b6ba7e50b3e9f77e, 2021-12-07T06:26:56.179Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7200U CPU @ 2.50GHz (4 x 2712)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.89GB (1.54GB free)

Process Argv
--crash-reporter-id 0c28b813-4c69-410e-a836-c06b69be7d13

Screen Reader
no

VM
0%

Extensions (18)

Extension
Author (truncated)
Version

texlab
efo
3.3.1

latex-workshop
Jam
8.22.0

tex-workshop
Jef
8.11.1

rainbow-csv
mec
1.10.1

vscode-docker
ms-
1.18.0

vscode-language-pack-it
MS-
1.63.2

csharp
ms-
1.23.17

python
ms-
2021.12.1559732655

jupyter
ms-
2021.11.1001550889

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.4

remote-containers
ms-
0.209.5

cpptools
ms-
1.7.1

vsliveshare
ms-
1.0.5196

vsliveshare-audio
ms-
0.1.91

vsliveshare-pack
ms-
0.4.0

bib
phr
0.3.0

latex-support
tor
3.10.0

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
pythonrunftest32:30373476
pythonf5test824:30373475
javagetstartedt:30391933
pythonvspyt187:30373474
vsaa593cf:30376535
pythonvs932:30410667
vscop804cf:30404767
vscop453:30404998
vsrem710:30405998

@TylerLeonhardt they seem to be mangling these more than usual, anything we can do here? On the testing version the call has already been commented:
https://github.com/microsoft/vscode/blob/29cc85b316759b9343692489c894aaec6e3f567e/src/vs/workbench/contrib/testing/browser/testing.contribution.ts#L86
But for the git one it comes in via package.nls.json, which doesn't support comments.
also cc @connor4312 and @lszomoru as area owners

I haven't gotten around to hounding everyone about using this special syntax @danyeh has told me about... If you'd like to get a head start take a look at this issue:
https://github.com/microsoft/vscode/issues/137830#issuecomment-999653753


Bank account - Back arrow is missing in the personal info onfido page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Issue found when executing PR https://github.com/Expensify/App/pull/22628
Action Performed:

Launch New Expensify app.
Go to Settings > Workspace > any workspace > Bank account > Step 3: Personal information.

Expected Result:
Back arrow should be present
Actual Result:
No back arrow in the onfido verification page
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] Windows / Chrome
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:

simply use an Image Icon

simply use an Image Icon
<EMAIL_ADDRESS>https://www.upwork.com/freelancers/~01e34ac222e73068fb


image-only: prevent blank image at startup

fixes #406

Coverage increased (+0.2%) to 74.834% when pulling c7075570c7295a9c62fee8c4781c27ba2cdd1e45 on wait-for-stream-406 into 63c44c190aa18e7b4797225e28803ac321d061f1 on master.


Fixed typo

Fixed an error in the global config stuff.

Ah, good catch.  That will teach me to make releases without my release script.
Does this requirement for maintaining previousVersions imply that the versions selection in the Mod (API) Sandbox has been fixed?  It hasn't actually worked for at least 5 years...

Checking the previous versions is probably a good candidate for adding to the automated checks.


[SPARK-29911][SQL] Uncache cached tables when session closed

What changes were proposed in this pull request?
The local temporary view is session-scoped. Its lifetime is the lifetime of the session that created it.  But now cache data is cross-session. Its lifetime is the lifetime of the Spark application. That's will cause the memory leak if cache a local temporary view in memory when the session closed.
In this PR, we uncache the cached data of local temporary view when session closed. This PR doesn't impact the cached data of global temp view and persisted view.
How to reproduce:

create a local temporary view v1
cache it in memory
close session without drop table v1.

The application will hold the memory forever. In a long running thrift server scenario. It's worse.
0: jdbc:hive2://localhost:10000> CACHE TABLE testCacheTable AS SELECT 1;
CACHE TABLE testCacheTable AS SELECT 1;
+---------+--+
| Result  |
+---------+--+
+---------+--+
No rows selected (1.498 seconds)
0: jdbc:hive2://localhost:10000> !close
!close
Closing: 0: jdbc:hive2://localhost:10000
0: jdbc:hive2://localhost:10000 (closed)> !connect 'jdbc:hive2://localhost:10000'
!connect 'jdbc:hive2://localhost:10000'
Connecting to jdbc:hive2://localhost:10000
Enter username for jdbc:hive2://localhost:10000:
lajin
Enter password for jdbc:hive2://localhost:10000:
***
Connected to: Spark SQL (version 3.0.0-SNAPSHOT)
Driver: Hive JDBC (version 1.2.1.spark2)
Transaction isolation: TRANSACTION_REPEATABLE_READ
1: jdbc:hive2://localhost:10000> select * from testCacheTable;
select * from testCacheTable;
Error: Error running query: org.apache.spark.sql.AnalysisException: Table or view not found: testCacheTable; line 1 pos 14;
'Project [*]
+- 'UnresolvedRelation [testCacheTable] (state=,code=0)

Why are the changes needed?
Resolve memory leak for thrift server
Does this PR introduce any user-facing change?
No
How was this patch tested?
Manual test in UI storage tab
And add an UT

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/18724/
Test PASSed.

Test build #113855 has started for PR 26543 at commit e6bb796.

Test build #113855 has finished for PR 26543 at commit e6bb796.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/113855/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/18737/
Test PASSed.

Test build #113870 has started for PR 26543 at commit f3b8923.

Test build #113870 has finished for PR 26543 at commit f3b8923.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/113870/
Test PASSed.

Test build #113959 has started for PR 26543 at commit 417f3aa.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/18826/
Test PASSed.

Test build #113960 has started for PR 26543 at commit 32d0c49.

Thank you for updating, @LantaoJin .

Test build #113959 has finished for PR 26543 at commit 417f3aa.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/113959/
Test PASSed.

Test build #113960 has finished for PR 26543 at commit 32d0c49.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/113960/
Test PASSed.

Also, cc @gatorsmile and @cloud-fan

Test build #114042 has started for PR 26543 at commit fa3e0dc.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/18903/
Test PASSed.

Test build #114042 has finished for PR 26543 at commit fa3e0dc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/114042/
Test PASSed.

Merged to master


Several bugfixes and styling improvements

Fixes for issues #383, #380 and #403. Also made some styling improvements

[X] Change details added to Unreleased section of changelog.md?
[N/A] Added/updated documentation and descriptions in .schema.mof files where appropriate?
[N/A] Examples updated for both the single server and small farm templates in the examples folder?
[X] New/changed code adheres to [Style Guidelines]?(https://github.com/PowerShell/DscResources/blob/master/StyleGuidelines.md)?
[X] Unit and Integration tests created/updated where possible?

This change is

Checking the Appveyor errors.......Pester is annoying me once again ;-)

Looks like you need to check the tests for that .net 4.6 thing sorted. There was a couple of things I picked up in the review that relate to that, but it needs to be looked at.

Reviewed 83 of 83 files at r1.
Review status: all files reviewed at latest revision, 6 unresolved discussions, some commit checks failed.

CHANGELOG.md, line 6 at r1 (raw file):

 * Fixed issue with mounting SPContentDatabase
 * Fixed issue with SPShellAdmin and Content Database method
 * Added .Net 4.6 support check

Can we make these specifically list the resources they impact - like "added .net 4.6 support check to SPInstallPrereqs" so its clear where the bug fix impacts?

Modules/SharePointDsc/DSCResources/MSFT_SPInstall/MSFT_SPInstall.psm1, line 108 at r1 (raw file):

        $svrsetupDll = Join-Path -Path $BinaryDir -ChildPath "updates\svrsetup.dll"
        $checkDotNet = $true
        if (Test-Path -Path $svrsetupDll) {

Bracket needs to open on new line

Modules/SharePointDsc/DSCResources/MSFT_SPInstall/MSFT_SPInstall.psm1, line 109 at r1 (raw file):

        $checkDotNet = $true
        if (Test-Path -Path $svrsetupDll) {
            $svrsetupDllFileInfo = Get-ItemProperty $svrsetupDll

Positional parameter on Get-ItemProperty

Modules/SharePointDsc/DSCResources/MSFT_SPInstallPrereqs/MSFT_SPInstallPrereqs.psm1, line 433 at r1 (raw file):

        $svrsetupDll = Join-Path -Path $BinaryDir -ChildPath "updates\svrsetup.dll"
        $checkDotNet = $true
        if (Test-Path -Path $svrsetupDll) {

Bracket should be on new line

Tests/Unit/SharePointDsc/SharePointDsc.SPInstall.Tests.ps1, line 126 at r1 (raw file):

        Context "SharePoint binaries are installed and should not be" {
            Mock Get-ItemProperty { return @{} } -ParameterFilter { $Path }

The parameter filter stuff here needs be in the formatted like this: $Parameter -eq "Value". So if you are looking to only mock Get-ItemProperty when -Path is specified to be a certain value (or is specified at all) then it should be $null -ne $Path so it's clear what the intent is. Same for the one below.

Tests/Unit/SharePointDsc/SharePointDsc.SPInstallPrereqs.Tests.ps1, line 92 at r1 (raw file):

            Mock Get-ItemProperty -MockWith {
                return @()
            } -ParameterFilter { $Path }

Same comment in this file about formatting parameterfilter

Comments from Reviewable

Review status: 77 of 83 files reviewed at latest revision, 6 unresolved discussions.

CHANGELOG.md, line 6 at r1 (raw file):
Previously, BrianFarnhill (Brian Farnhill) wrote…
> Can we make these specifically list the resources they impact - like "added .net 4.6 support check to SPInstallPrereqs" so its clear where the bug fix impacts?

Done.

Modules/SharePointDsc/DSCResources/MSFT_SPInstall/MSFT_SPInstall.psm1, line 108 at r1 (raw file):
Previously, BrianFarnhill (Brian Farnhill) wrote…
> Bracket needs to open on new line

Done.

Modules/SharePointDsc/DSCResources/MSFT_SPInstall/MSFT_SPInstall.psm1, line 109 at r1 (raw file):
Previously, BrianFarnhill (Brian Farnhill) wrote…
> Positional parameter on Get-ItemProperty

Done.

Modules/SharePointDsc/DSCResources/MSFT_SPInstallPrereqs/MSFT_SPInstallPrereqs.psm1, line 433 at r1 (raw file):
Previously, BrianFarnhill (Brian Farnhill) wrote…
> Bracket should be on new line

Done.

Tests/Unit/SharePointDsc/SharePointDsc.SPInstall.Tests.ps1, line 126 at r1 (raw file):
Previously, BrianFarnhill (Brian Farnhill) wrote…
> The parameter filter stuff here needs be in the formatted like this: $Parameter -eq "Value". So if you are looking to only mock Get-ItemProperty when -Path is specified to be a certain value (or is specified at all) then it should be $null -ne $Path so it's clear what the intent is. Same for the one below.

Done.

Tests/Unit/SharePointDsc/SharePointDsc.SPInstallPrereqs.Tests.ps1, line 92 at r1 (raw file):
Previously, BrianFarnhill (Brian Farnhill) wrote…
> Same comment in this file about formatting parameterfilter

Done.

Comments from Reviewable

Reviewed 6 of 6 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Modules/SharePointDsc/DSCResources/MSFT_SPContentDatabase/MSFT_SPContentDatabase.psm1, line 330 at r2 (raw file):

            else
            {
                Write-Verbose -Message "Could not update Content Database because the reference is null"

Can you expand on this? Why would it be null? What scenario are you catering for with this? I think we could give a more meaningful message to the user when this happens

Comments from Reviewable

Review status: 82 of 84 files reviewed at latest revision, 1 unresolved discussion.

Modules/SharePointDsc/DSCResources/MSFT_SPContentDatabase/MSFT_SPContentDatabase.psm1, line 330 at r2 (raw file):
Previously, BrianFarnhill (Brian Farnhill) wrote…
> Can you expand on this? Why would it be null? What scenario are you catering for with this? I think we could give a more meaningful message to the user when this happens

Done.

Comments from Reviewable

Review status: 82 of 84 files reviewed at latest revision, 1 unresolved discussion.

Modules/SharePointDsc/DSCResources/MSFT_SPContentDatabase/MSFT_SPContentDatabase.psm1, line 330 at r2 (raw file):
Previously, ykuijs (Yorick Kuijs) wrote…
> Done.

For example: if the database server does not exist, Mount-SPContentDatabase does not give an terminating error and the variable end up being NULL.

Comments from Reviewable

Reviewed 2 of 2 files at r3.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Modules/SharePointDsc/DSCResources/MSFT_SPContentDatabase/MSFT_SPContentDatabase.psm1, line 330 at r2 (raw file):
Previously, ykuijs (Yorick Kuijs) wrote…
> For example: if the database server does not exist, Mount-SPContentDatabase does not give an terminating error and the variable end up being NULL.

Do you think we can list things here to check for the failure? So something like  "unable to update content database, please ensure that the database server exists and that the run as account has permissions to it" or something to that extent.

Comments from Reviewable

Reviewable has an issue, so replying the old-fashioned way.
Resolved the issue by adding -ErrorAction to the Mount-SPContentDatabase cmdlet. As it turns out, having a non-existent database is a non-terminating error which is not picked up by try/catch. By adding ErrorAction Stop, it now is being picked up which removes the necessity for this check.

@ykuijs Thanks - yea reviewable are having an issue right now it seems, but I'm happy with that last fix and happy with the PR. Merging in to dev now 👍


KAFKA-7103: Use bulkloading for RocksDBSegmentedBytesStore during init

This PR uses bulk loading for recovering RocksDBWindowStore, same as RocksDBStore. Will perform unit test and integration test.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@guozhangwang @mjsax @abbccdda

@Ishiihara Can you please create a JIRA for this. By default, all changes should have a JIRA. Thx.

@mjsax I updated the title to include the JIRA number.
@bbejeck For this PR, I would like to focus on the correctness and testing. I filed https://issues.apache.org/jira/browse/KAFKA-7105 to track the refactoring.

@mjsax @bbejeck @guozhangwang Would you please take another look? Thanks!
cc @abbccdda

@mjsax @guozhangwang I discovered an edge case that that might not enable enable the bulk loading for segments that are not at the current on the machine.

Could you elaborate a bit more?

@mjsax Do not have any further comments, you can merge it as is :)

@guozhangwang @mjsax I addressed Guozhang's comment. Could you take another look?

retest this please

@mjsax could you give it another look?

@guozhangwang @mjsax Not sure why Jenkins is keep failing. The failure seems not related to my change. Also, please take another look when you have time. Thanks!

retest this please

retest this please

retest this please

@Ishiihara The jenkins failures are due to
/home/jenkins/jenkins-slave/workspace/kafka-pr-jdk10-scala2.12/streams/src/main/java/org/apache/kafka/streams/state/internals/RocksDBSegmentedBytesStore.java:212: error: unreported exception RocksDBException; must be caught or declared to be thrown
18:24:17                     batch.remove(record.key);
18:24:17                                 ^
18:24:17 /home/jenkins/jenkins-slave/workspace/kafka-pr-jdk10-scala2.12/streams/src/main/java/org/apache/kafka/streams/state/internals/RocksDBSegmentedBytesStore.java:214: error: unreported exception RocksDBException; must be caught or declared to be thrown
18:24:17                     batch.put(record.key, record.value);

Made another pass, except the compilation error it lgtm. @Ishiihara please let me know if you can fix the compilation errors, after that I'll go and merge it.

@guozhangwang Updated the PR to catch RocksDB exception. PTAL.

retest this please

Triggered system test https://jenkins.confluent.io/job/system-test-kafka-branch-builder/1854/ before merging.

System test passed.


PMID:19158664

increased number of Rad52 foci during cellular response to hydroxyurea
increased protein localization to chromatin during cellular response to hydroxyurea
normal protein ubiquitination during cellular response to hydroxyurea

increased number of Rad52 foci during cellular response to hydroxyurea FYPO:0007550
increased protein localization to chromatin during cellular response to hydroxyurea FYPO:0007551
normal protein ubiquitination during cellular response to hydroxyurea FYPO:0007552


Additional case where we pass unicode to pysam. Fixes bioinform/metasv#83

Thanks, @chapmanb for the fix!


Fix for issue #112

This is a proposed solution for issue #112

This should fix the issue.
Could you please also delete the lines 26 and 33-35 in the src/impl/metadata/writer/profileWriter.ts class? The full name was used to build the path of newly created profiles. But this was moved in the profileActions File.

Thanks @manisfdcsfdx  can you combine @genoud's comments and add it to the PR


[IntelliJ Plugin] Fix random error when splitting strings

Purpose
Fixes #20477.
Related PRs
#20528.
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[x] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Thanks @Shan1024


Invalid serialization of a document with a JToken property

What is the bug?
A document with a property of type JToken doesn't write properly. Instead of getting a nice JSON, I'm getting a bunch of empty square brackets.
{
  "_index": "test",
  "_id": "xQ5SvocBEGF2UAwoCWfK",
  "_version": 1,
  "_score": 0,
  "_source": {
    "Id": "2b7058e0-44ea-4bbe-a80b-5a253b412470",
    "Name": "Message",
    "Payload": [
      [
        []
      ],
      [
        []
      ],
      [
        []
      ],
      [
        [
          [
            []
          ],
          [
            []
          ]
        ]
      ]
    ]
  }
}

How can one reproduce the bug?

Download this repo
Run docker compose to have a single OpenSearch node and OpenSearch Dashboard
Run dotnet application
Go to the dashboard and look for index test

What is the expected behavior?
The payload property should contain all the data from the JToken
What is your host/environment?
Windows 11
Do you have any additional context?
I also tried setting up JsonNetSerializer, but the result doesn't change. It's available on this branch

@Jyuart this is a result of using the "low-level" OpenSearch.Net client which does not use the configured JsonNetSerializer when serializing request bodies. This serializes correctly when using the "high-level" OpenSearch.Client client like so:
var connPool = new SingleNodeConnectionPool(new Uri("..."));
var settings = new ConnectionSettings(connPool, JsonNetSerializer.Default);
var client = new OpenSearchClient(settings);

await client.BulkAsync(b => b.Index<object>(i => i.Index("test")
    .Document(new
    {
        Id = Guid.NewGuid(),
        Name = "Message",
        Payload = JToken.Parse(
            "{\"name\":\"John\",\"age\":30,\"city\":\"New York\",\"contact\":{\"email\":\"john@example.com\",\"phone\":\"+1<PHONE_NUMBER>\"}}")
    })));

@Xtansia Thank you very much. It worked.
I assume it's possible to reach the same result with the "low-level" client, but that's probably a lot of work
I'll stick to this solution
Thanks a lot!


Feat: Dark Mode functionality

Describe the feature
I want to add dark mode functionality to the website.
Add ScreenShots
Please assign me for this work
Record

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

Issue already exists


Reshmi and Shak to discuss team scaling and organize a meeting to share the plan

duplicate with #575


Readded images

These were removed wrongly

Undoing https://github.com/andrew/ar-drone-gps/commit/9816d6ea49daee27d94a903d4b1ca98945a8ff5b

Thanks @Squarific


Please update docker container for latest version 1.9.1

Describe the feature
Please update docker container for latest version 1.9.1

Thanks for reporting the bug, i checked and the continuous deployment entry was struck. This has been fixed.


Update phrasing for stove/oven problems in kitchen category?

Right now the phrasing could be confusing because it has two very similar options:

No working oven or stove
Stove or oven not working

I assume the first means that there is no oven or stove on the premises, whereas the second means that the oven or stove on the premises is not working.
Even though this is referenced in the same sanitary code section, updating this to more accurately reflect each issue will help users understand which option they should select.

This issue might be with MadeUpToCode, not GetUpToCode.


Simplify checkpoint download logic

Patch Description

Add trailing wildcard support to PathManager.ls() to support changes below
Update checkpoint caching:

Remove the double call to get_local_path here which may have been causing a race condition. Passing force=True should get the intended effect. Add a utility to stress test file locking
Fix load_checkpoint_to_cpu() to support remote checkpoints when DP>1 (see the stacktrace I got here). I think this only worked before because get_local_path() is a no-op for local paths and the other shard files are already nearby. Updated to ensure we cache all shards locally before attempting to load, using the new wildcard support in PathManager.ls()

Testing steps
Multiple eval runs on OPT 125M:

remote path + consolidated
remote path + DP>1
local path + consolidated
local path + DP>1

Running the stress test:
python -m tests.file_io.async_download_test

LGTM

I think a rebase on main should hopefully fix the failing test...


manuskript: fix broken Python dependencies, add me as maintainer

Pin the Python version for the manuskript package to 3.11 to fix broken dependencies. Also adding me as a maintainer.
Description of changes
Manuskript previously used python3Packages, which defaulted to Python 3.12 or higher, causing a crash on startup. Pinning the version to 3.11 (python311Packages) resolves this issue. Manuskript is orphaned and has been for some time, so I'm adding myself as a maintainer.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[X] 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

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

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

[x] Fits CONTRIBUTING.md.

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

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/4304

manuskript builds for me locally on current master without changes.
Please check locally and report back if this isn't the case for you. Also if you would still like to add yourself as maintainer, I'd be happy to merge after you drop the other commits :+1:

manuskript builds for me locally on current master without changes. Please check locally and report back if this isn't the case for you. Also if you would still like to add yourself as maintainer, I'd be happy to merge after you drop the other commits 👍

manuskript builds, but i wasn't able to launch it without pinning to python311Packages:
`📁 nixpkgs 🌿 manuskript-fix 💞
🕒 01:52 PM💖 manuskript -v
Qt: Session management error: None of the authentication protocols specified are supported
INFO> Logging to file: /home/strawbee/.local/share/manuskript/manuskript/logs/2024-07-22_13-52-59_manuskript#520164.log
INFO> Operating System: Linux-6.6.41-x86_64-with-glibc2.39
INFO> Hardware: x86_64 /
INFO> Running from unpackaged source code.
INFO> Manuskript 0.16.1 (Python 3.12.4)
INFO> * PyQt 5.15.9 (compiled against Qt 5.15.14)
INFO>   * Qt 5.15.14 (runtime)
INFO> * lxml.etree <IP_ADDRESS>
INFO>   * libxml   2.12.7 (compiled: 2.12.7)
INFO>   * libxslt  1.1.41 (compiled: 1.1.41)
INFO> * pyEnchant N/A (libenchant: N/A)
INFO> * pySpellChecker N/A
INFO> * Symspellpy N/A
INFO> * Markdown N/A
INFO> Web rendering engine: QTextEdit
INFO> Preferred translation: ['en-US', 'en', 'en-Latn-US'] (based on available ui languages)
INFO> Using the builtin translation. (U.S. English)
Fatal Python error: Segmentation fault
Current thread 0x00007fafff8fb740 (most recent call first):
File "/nix/store/1k5imkl28cjsr5f6zh6nppxy2s18wxc2-manuskript-0.16.1/bin/../manuskript/models/outlineModel.py", line 8 in
File "", line 488 in _call_with_frames_removed
File "", line 995 in exec_module
File "", line 935 in _load_unlocked
File "", line 1331 in _find_and_load_unlocked
File "", line 1360 in _find_and_load
File "/nix/store/1k5imkl28cjsr5f6zh6nppxy2s18wxc2-manuskript-0.16.1/bin/../manuskript/models/init.py", line 5 in
File "", line 488 in _call_with_frames_removed
File "", line 995 in exec_module
File "", line 935 in _load_unlocked
File "", line 1331 in _find_and_load_unlocked
File "", line 1360 in _find_and_load
File "", line 488 in _call_with_frames_removed
File "", line 1310 in _find_and_load_unlocked
File "", line 1360 in _find_and_load
File "/nix/store/1k5imkl28cjsr5f6zh6nppxy2s18wxc2-manuskript-0.16.1/bin/../manuskript/load_save/version_0.py", line 17 in
File "", line 488 in _call_with_frames_removed
File "", line 995 in exec_module
File "", line 935 in _load_unlocked
File "", line 1331 in _find_and_load_unlocked
File "", line 1360 in _find_and_load
File "/nix/store/1k5imkl28cjsr5f6zh6nppxy2s18wxc2-manuskript-0.16.1/bin/../manuskript/loadSave.py", line 9 in
File "", line 488 in _call_with_frames_removed
File "", line 995 in exec_module
File "", line 935 in _load_unlocked
File "", line 1331 in _find_and_load_unlocked
File "", line 1360 in _find_and_load
File "", line 488 in _call_with_frames_removed
File "", line 1415 in _handle_fromlist
File "/nix/store/1k5imkl28cjsr5f6zh6nppxy2s18wxc2-manuskript-0.16.1/bin/../manuskript/mainWindow.py", line 18 in
File "", line 488 in _call_with_frames_removed
File "", line 995 in exec_module
File "", line 935 in _load_unlocked
File "", line 1331 in _find_and_load_unlocked
File "", line 1360 in _find_and_load
File "/nix/store/1k5imkl28cjsr5f6zh6nppxy2s18wxc2-manuskript-0.16.1/bin/../manuskript/main.py", line 177 in prepare
File "/nix/store/1k5imkl28cjsr5f6zh6nppxy2s18wxc2-manuskript-0.16.1/bin/../manuskript/main.py", line 308 in run
File "/nix/store/1k5imkl28cjsr5f6zh6nppxy2s18wxc2-manuskript-0.16.1/bin/.manuskript-wrapped", line 14 in
Extension modules: PyQt5.QtCore, PyQt5.QtGui, PyQt5.QtWidgets, PyQt5.QtXml, PyQt5.QtWebChannel, PyQt5.QtTest, PyQt5.QtSvg, PyQt5.QtSql, PyQt5.QtNetwork, PyQt5.QtQml, PyQt5.QtQuick, PyQt5.QtQuickWidgets, PyQt5.QtPrintSupport, PyQt5.QtOpenGL, PyQt5.QtDBus, lxml._elementpath, lxml.etree (total: 17)
[1]    520164 segmentation fault (core dumped)  manuskript -v
`

interesting. I wasn't able to find any pointers at the upstream project, nor any patches. I guess it is the easiest way to use the older python311Packages.
Please don't use a specific version on manuskript but change it in top-level/all-packages.nix like so:
  manuskript = libsForQt5.callPackage ../applications/editors/manuskript {
    python3Packages = python311Packages;
  };

If you can re-arrange your commit history like so would be awesome:

maintainers: add strawbee
manuskript: override python version
manuskript: add strawbee as maintainer

interesting. I wasn't able to find any pointers at the upstream project, nor any patches. I guess it is the easiest way to use the older python311Packages. Please don't use a specific version on manuskript but change it in top-level/all-packages.nix like so:
  manuskript = libsForQt5.callPackage ../applications/editors/manuskript {
    python3Packages = python311Packages;
  };

If you can re-arrange your commit history like so would be awesome:

maintainers: add strawbee
manuskript: override python version
manuskript: add strawbee as maintainer

I think that should do it. I've not really used git on a project with more than a few members before, so this has very much been a learning experience.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 329075 run on x86_64-linux 1

1 package built:

manuskript


Error importing discovery from the client API

I am hitting the eror
"from oauth2client import util
ImportError: cannot import name util"
when I use the following imports:
from googleapiclient import discovery
from oauth2client import client
from oauth2client import tools
from oauth2client.file import Storage
Apparently this issue was fixed and closed, but I am seeing this in my IDE, so was not sure if this is deprecated and other ways to get this done.
Thanks

What version of oauth2client and google-api-python-client do you have installed?

Using google-api-python-client 1.6.2 with oauth2client 4.1.1

Can you give the full stacktrace?

Traceback (most recent call last):
File "/gmail/gmail-sampler.py", line 5, in
from googleapiclient import discovery
File "/lib/python2.7/site-packages/googleapiclient/discovery.py", line 57, in
from googleapiclient.errors import HttpError
File "**/lib/python2.7/site-packages/googleapiclient/errors.py", line 26, in
from oauth2client import util
ImportError: cannot import name util

So that's expected: https://github.com/google/google-api-python-client/blob/master/googleapiclient/errors.py#L28
I can't see why it wouldn't be catching that importerror and importing the >= v3 name.

and your suggestion to debug this would be??

I really sort of at a loss. Can you compare the file you have for errors.py with the one here? The different in line numbers leads me to think that your app might somehow be picking up a different, older version.

Alrite. let me take a look and get back to you...Thanks


Update README.md

Some things here @tvansteenburgh :

I think we should update the overview also
should it also link to MicroK8s?
I believe we have all the relevant stuff from the readme captured in docs, but feel free to prove me wrong

I think we should update the overview also
Yes, please take a stab at it.

should it also link to MicroK8s?
Yes, great idea.

I believe we have all the relevant stuff from the readme captured in docs, but feel free to prove me wrong
I believe you, but I'll skim over it anyway.

We should do this for https://github.com/charmed-kubernetes/bundle/tree/master/fragments/k8s/core also.

@tvansteenburgh see also https://github.com/charmed-kubernetes/bundle/pull/777


Sorting by size variable does not work

Checklist

[X] I'm reporting a bug unrelated to a specific site
[X] I've verified that I'm running yt-dlp version 2022.07.18 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
I am unable to sort formats by size. Per the documentation:

Eg: +res prefers format with the smallest resolution.

By the same logic, +size must prefer the format with the smallest "size" (Exact filesize if available, otherwise approximate filesize). However, it appears this variable is not being considered at all. Sorting by +res works as expected, but not +size.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
yt-dlp -F -S +size https://www.paramountplus.com/shows/video/kaWvMpPYbGBFzNVASW8sT6JxB226Mwul/ -v
[debug] Command-line config: ['-F', '-S', '+size', 'https://www.paramountplus.com/shows/video/kaWvMpPYbGBFzNVASW8sT6JxB226Mwul/', '-v']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.07.18 [135f05e]
[debug] Python 3.9.13 (CPython 64bit) - macOS-12.5-arm64-arm-64bit
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 5.1 (fdk,setts), ffprobe 5.1, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.06.15, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] [ParamountPlus] Extracting URL: https://www.paramountplus.com/shows/video/kaWvMpPYbGBFzNVASW8sT6JxB226Mwul/
[ParamountPlus] kaWvMpPYbGBFzNVASW8sT6JxB226Mwul: Downloading JSON metadata
[ParamountPlus] kaWvMpPYbGBFzNVASW8sT6JxB226Mwul: Downloading JSON metadata
[ParamountPlus] kaWvMpPYbGBFzNVASW8sT6JxB226Mwul: Downloading DASH_CENC_PRECON SMIL data
[ParamountPlus] kaWvMpPYbGBFzNVASW8sT6JxB226Mwul: Downloading m3u8 information
[debug] Sort order given by user: +size
[debug] Formats sorted by: hasvid, ie_pref, +filesize, +fs_approx, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), acodec, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[info] Available formats for kaWvMpPYbGBFzNVASW8sT6JxB226Mwul:
ID       EXT RESOLUTION │   FILESIZE   TBR PROTO │ VCODEC        VBR ACODEC    ABR
──────────────────────────────────────────────────────────────────────────────────
hls-95   mp4 audio only │ ~ 69.74MiB   95k m3u8  │ audio only        mp4a.40.2 95k
hls-495  mp4 640x360    │ ~363.39MiB  495k m3u8  │ avc1.640028  495k mp4a.40.2  0k
hls-838  mp4 768x432    │ ~615.20MiB  838k m3u8  │ avc1.640028  838k mp4a.40.2  0k
hls-1616 mp4 960x540    │ ~  1.16GiB 1616k m3u8  │ avc1.640028 1616k mp4a.40.2  0k
hls-2214 mp4 1024x576   │ ~  1.59GiB 2214k m3u8  │ avc1.640028 2214k mp4a.40.2  0k
hls-3112 mp4 1280x720   │ ~  2.23GiB 3112k m3u8  │ avc1.640028 3112k mp4a.40.2  0k
hls-4610 mp4 1280x720   │ ~  3.31GiB 4610k m3u8  │ avc1.640028 4610k mp4a.40.2  0k

yt-dlp -F -S +res https://www.paramountplus.com/shows/video/kaWvMpPYbGBFzNVASW8sT6JxB226Mwul/ -v
[debug] Command-line config: ['-F', '-S', '+res', 'https://www.paramountplus.com/shows/video/kaWvMpPYbGBFzNVASW8sT6JxB226Mwul/', '-v']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.07.18 [135f05e]
[debug] Python 3.9.13 (CPython 64bit) - macOS-12.5-arm64-arm-64bit
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 5.1 (fdk,setts), ffprobe 5.1, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.06.15, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] [ParamountPlus] Extracting URL: https://www.paramountplus.com/shows/video/kaWvMpPYbGBFzNVASW8sT6JxB226Mwul/
[ParamountPlus] kaWvMpPYbGBFzNVASW8sT6JxB226Mwul: Downloading JSON metadata
[ParamountPlus] kaWvMpPYbGBFzNVASW8sT6JxB226Mwul: Downloading JSON metadata
[ParamountPlus] kaWvMpPYbGBFzNVASW8sT6JxB226Mwul: Downloading DASH_CENC_PRECON SMIL data
[ParamountPlus] kaWvMpPYbGBFzNVASW8sT6JxB226Mwul: Downloading m3u8 information
[debug] Sort order given by user: +res
[debug] Formats sorted by: hasvid, ie_pref, +res, lang, quality, fps, hdr:12(7), vcodec:vp9.2(10), acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[info] Available formats for kaWvMpPYbGBFzNVASW8sT6JxB226Mwul:
ID       EXT RESOLUTION │   FILESIZE   TBR PROTO │ VCODEC        VBR ACODEC    ABR
──────────────────────────────────────────────────────────────────────────────────
hls-95   mp4 audio only │ ~ 69.74MiB   95k m3u8  │ audio only        mp4a.40.2 95k
hls-3112 mp4 1280x720   │ ~  2.23GiB 3112k m3u8  │ avc1.640028 3112k mp4a.40.2  0k
hls-4610 mp4 1280x720   │ ~  3.31GiB 4610k m3u8  │ avc1.640028 4610k mp4a.40.2  0k
hls-2214 mp4 1024x576   │ ~  1.59GiB 2214k m3u8  │ avc1.640028 2214k mp4a.40.2  0k
hls-1616 mp4 960x540    │ ~  1.16GiB 1616k m3u8  │ avc1.640028 1616k mp4a.40.2  0k
hls-838  mp4 768x432    │ ~615.20MiB  838k m3u8  │ avc1.640028  838k mp4a.40.2  0k
hls-495  mp4 640x360    │ ~363.39MiB  495k m3u8  │ avc1.640028  495k mp4a.40.2  0k

The issue is the filesize_approx=duration*tbr is calculated after sorting.
For now, you can sort by tbr as a workaround. -S +tbr

I cannot sort by tbr as really my ultimate goal is to do something like size:7G, which is a function of duration as well

My workaround will probably be to get all the formats, and then write my own function to pick the best one and then download that one, but that seems too "hacky" when clearly yt-dlp already has the scaffolding to select the best format in one shit

I suspect actually fixing this will be difficult and will take time
Does -f "bv[filesize_approx<7G]" work for the time-being?

No:
yt-dlp -F https://www.espn.com/watch/player/_/id/f9230b4a-2beb-43c5-84f2-a4a09cf73547 --cookies espn.com_cookies.txt
[WatchESPN] f9230b4a-2beb-43c5-84f2-a4a09cf73547: Downloading JSON metadata
[WatchESPN] f9230b4a-2beb-43c5-84f2-a4a09cf73547: Downloading JSON metadata
[WatchESPN] f9230b4a-2beb-43c5-84f2-a4a09cf73547: Downloading JSON metadata
[WatchESPN] f9230b4a-2beb-43c5-84f2-a4a09cf73547: Downloading JSON metadata
[WatchESPN] f9230b4a-2beb-43c5-84f2-a4a09cf73547: Downloading JSON metadata
[WatchESPN] f9230b4a-2beb-43c5-84f2-a4a09cf73547: Downloading JSON metadata
[WatchESPN] f9230b4a-2beb-43c5-84f2-a4a09cf73547: Downloading m3u8 information
[info] Available formats for f9230b4a-2beb-43c5-84f2-a4a09cf73547:
ID       EXT RESOLUTION FPS │   FILESIZE   TBR PROTO │ VCODEC        VBR ACODEC    ABR
──────────────────────────────────────────────────────────────────────────────────────
hls-250  mp4 320x180     30 │ ~341.71MiB  250k m3u8  │ avc1.4d001f  250k mp4a.40.2  0k
hls-550  mp4 384x216     30 │ ~751.75MiB  550k m3u8  │ avc1.4d001f  550k mp4a.40.2  0k
hls-1000 mp4 512x288     30 │ ~  1.33GiB 1000k m3u8  │ avc1.4d001f 1000k mp4a.40.2  0k
hls-1420 mp4 640x360     30 │ ~  1.90GiB 1420k m3u8  │ avc1.4d001f 1420k mp4a.40.2  0k
hls-2150 mp4 896x504     30 │ ~  2.87GiB 2150k m3u8  │ avc1.4d001f 2150k mp4a.40.2  0k
hls-2980 mp4 960x540     30 │ ~  3.98GiB 2980k m3u8  │ avc1.4d0020 2980k mp4a.40.2  0k
hls-4190 mp4 1280x720    30 │ ~  5.59GiB 4190k m3u8  │ avc1.640028 4190k mp4a.40.2  0k
hls-6600 mp4 1280x720    60 │ ~  8.81GiB 6600k m3u8  │ avc1.640028 6600k mp4a.40.2  0k
hls-8250 mp4 1280x720    60 │ ~ 11.01GiB 8250k m3u8  │ avc1.640028 8250k mp4a.40.2  0k

yt-dlp -f "bv[filesize_approx<7G]" https://www.espn.com/watch/player/_/id/f9230b4a-2beb-43c5-84f2-a4a09cf73547 --cookies espn.com_cookies.txt
[WatchESPN] f9230b4a-2beb-43c5-84f2-a4a09cf73547: Downloading JSON metadata
[WatchESPN] f9230b4a-2beb-43c5-84f2-a4a09cf73547: Downloading JSON metadata
[WatchESPN] f9230b4a-2beb-43c5-84f2-a4a09cf73547: Downloading JSON metadata
[WatchESPN] f9230b4a-2beb-43c5-84f2-a4a09cf73547: Downloading JSON metadata
[WatchESPN] f9230b4a-2beb-43c5-84f2-a4a09cf73547: Downloading JSON metadata
[WatchESPN] f9230b4a-2beb-43c5-84f2-a4a09cf73547: Downloading JSON metadata
[WatchESPN] f9230b4a-2beb-43c5-84f2-a4a09cf73547: Downloading m3u8 information
ERROR: [WatchESPN] f9230b4a-2beb-43c5-84f2-a4a09cf73547: Requested format is not available. Use --list-formats for a list of available formats

'fs_approx<7G' or 'size<7G' both return SyntaxError: Invalid filter specification

For your example, it should be b[filesize_approx<7G], not bv since the formats are pre-merged

This works, thank yoU!


Unbind from BTC alphabet

This would applications with non bitcoin alphabets to use it (eg. https://github.com/stellar/js-stellar-base/blob/master/src/vendor/bs58.js) without needing to hard fork and modify.

@dcousens  ?

@sublimator perhaps look at https://github.com/dcousens/base-x, it is the superset of this module but with support for any alphabet and any length.
It uses a base constructor function to ensure performance is identical.
This module is specifically targeting the base58 standard used by bitcoin, it would be misleading to use it with anything else.

Great! Yeah, I figured it was a little naff, but more interested in
pragmatics of using shared code. Pity modules like bs58check don't build
upon this.


Harmonize fg/bg/session check intervals with Android

Currently the defaults for Objc are incorrectly being passed in ms rather than s.  This is resulting in the defaults being 10000 min and 5000 min rather than 10 and 5 min.
https://github.com/snowplow/snowplow-objc-tracker/blob/master/Snowplow/SPTracker.m#L74-L75

Please bring this forwards into a patch release and put a large warning on the existing documentation, advising users to manually override the defaults themselves.


Support for semantic linefeeds?

Thanks for the useful tool. In addition to the obvious benefits of consistency, I think it has the potential to help reduce merge and diff friction in Github when many people edit Markdown.
Have you considered support for Semantic linefeeds? That is, if a flag is enabled, use heuristics to split based on punctuation (probably period comma, and a few less common ones, at least in English).
I know it's a slightly unusual practice, and frowned on in some places (like Wikipedia) but it offers some useful benefits in the context of Markdown in Git, so thought I'd mention it here.
See https://github.com/jlevy/the-art-of-command-line/issues/167 for some discussion. Having a tool for this convention (and perhaps some tunable variations, to allow experimentation on what conventions are best) might be helpful for the heavily-edited documents (like "awesome lists", https://github.com/jlevy/the-art-of-command-line, etc. ) that are becoming increasingly common on Github.

Thanks for the suggestion. As the label implies, this is something I'm thinking about, but I currently don't have actionable plans that are worth executing.
There are some benefits to semantic linefeeds, but I also like the current model that doesn't insert newlines at all, and expects your text editor/viewer to render the text with word wrap on. That way, as you resize your text editor/viewer, all text reflows and there's no need to manually edit newline positions.
One extra factor is that this will require a flag, and I prefer to avoid having configuration.
I just wanted to reply and give you some more insight on my thoughts on this.

Yes, definitely there are pros and cons to both approaches. In general I agree with flowing text for text documents, but when you have GitHub workflow on Markdown, you begin thinking of Markdown as more like source code (with clear semantics and clean merges of commits) rather than flowing the way it will when it's later formatted. Editors often give previews anyway.
Also get that you want to avoid myriad configuration. That said, this whole discussion is one of those perennial problems and it may take some experimentation to find good solutions. It took decades to properly implement the "gofmt" non-negotiable formatting idea. I think having config settings "discouraged but possible" is one way to allow experimental features (e.g. don't add lots of flags, and require a special config file or something like that).
Anyway, thanks for the response! I'll update if I find a better solution to this problem.

That said, this whole discussion is one of those perennial problems and it may take some experimentation to find good solutions.

I'm in full agreement there.
I do think that it's best for the person most interested in a certain experiment to run it themselves, to maximize the chance of it working out well. I'd be very glad to see you fork this project, for example, experiment this, and we can later decide to merge the efforts if it makes sense.
I still don't see a viable way of making semantic newlines work well in the context of markdown files. My main problem is that, when used, it makes editing text anywhere other than "end" more difficult. Imagine you delete or add some text to the first line, it becomes shorter/longer, and all following lines need to be reflowed. Perhaps markdownfmt itself can do that for you, but wouldn't that still cause each line to have a large diff, defeating the purpose?

Anyway, thanks for the response! I'll update if I find a better solution to this problem.

No problem, and please do. I'm also happy to keep discussing it here; the "thinking" label allows me not worry about wanting to close this issue asap. :)

Also, FWIW, here's what a diff when editing a single paragraph can look like. IMO, it can be quite readable, since individual words that are changed are highlighted too:

Sure, thanks — to continue the discussion: I get that you can see the difference sub-line with coloring (and git --color-words supports this too). That's not the real pain point. Rather it is merge conflicts when many people edit one doc. If two people change the same line, the merge is then a conflict under standard merging rules. E.g., when paragraphs are lines 5 sentences long, then non-trivial merges are 5 times more likely than if the sentences were split. (If we all wrote code with 300-char widths and 5 statements to a line, we'd have the same problem there, too.)
In your example, it sounds like you're thinking of regular word wrapping, e.g. on a column width, and yes, there reflows and breaks everything too. What I was suggesting was a variant on the semantic breaks, where you break on something "stable" like sentence-ending periods and comma phrases more than a certain length. The rules can even be a little complex, as long as it's something deterministic and doesn't make the source ugly. Then, say, modifying one word, would only have "local" effect on a (much shorter) line, so conflicts would be less common.
Yes, perhaps I'll experiment with it, too (but I'll have to find time to pick up enough Go I resist the temptation to redo it in Python 😉 ).

For what it's worth, I've finally revisited this idea, and wrote a new plugin for Atom that handles this need. I think the wrapping approach is preferable to #36's fixed line-length wrapping. It's new and I'll be experimenting with it more, so any feedback (and bugs) welcome!

@jlevy
using git diff --word-diff may be a better solution
If you change a word in a paragraph and rewrap (hard wrap, gwap in vim in normal mode) that paragraph - with this option the command will show only that one word changing. (The default diff output would show several lines.)
As for hard-wrapping itself - it's probably better for reading raw text files in my opinion, although not sure yet I can explain why. HTML and markdown usually ignore newlines either way (except for pre/code blocks).

Just an update: As a practical matter, we've been using this approach in flowmark for some time now in the process of publishing about a dozen books, and overall it's worked well.
git diff --word-diff is a nice idea for cases when you can't control the format. But note GitHub's UI and git merges don't work at word level—they operate at line level. So having content in a form that is readable, normalized, and merges cleanly has been helpful for revising complex docs and editorial workflows.


Evaluation Problem

I have downloaded your source code and trained without any change but I can not achieve the score from paper. The AUC score just 0.2966.
I wonder whether there is any tricks when I use your source code to train and to evaluate.

Did the dropout should change to 0.1, I see you noting it as keep prob and finally I get great score on it

Hi, @UsamiLee
I'd like to ask you what the auc score you got finally?


add log1p function

Added log1p function and its test to SpecialFunctions.
The comparison values used in the test refer to the results of julia.

Great, thank you! We can bring this already in for v5.
Btw, do you by chance know whether there is also a more standard short name for that ExponentialMinusOne function we also have (and which is somewhat related to log1p)?

Hi, no idea. But I didn't do that log1 función. I added the method that
returns the points where the cubic spline has derivative 0 (get horizontal
points) and also the get max point which uses the derivative 0 points.
Kind regards
Edd
On Fri, Dec 24, 2021, 7:01 AM Christoph Ruegg @.***>
wrote:

Merged #887 https://github.com/mathnet/mathnet-numerics/pull/887 into
master.
—
Reply to this email directly, view it on GitHub
https://github.com/mathnet/mathnet-numerics/pull/887#event-5813215918,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHI777P5JZGZC622D7M2M43USRAGDANCNFSM5J3DOKZQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thank you for your comments. I am glad.
As far as I know, "Expm1" is used in similar functions in other languages.
For example, Python provides it as "scipy.special.expm1", and c++ provides it as "std::expm1".
scipy.special.expm1
std::expm1

Btw, do you by chance know whether there is also a more standard short name for that ExponentialMinusOne function we also have (and which is somewhat related to log1p)?

Thank you for your comments. I am glad.
As far as I know, "Expm1" is used in similar functions in other languages.
For example, Python provides it as "scipy.special.expm1", and c++ provides it as "std::expm1".
scipy.special.expm1
std::expm1


use elementwise_grad in examlpes/tanh.py for broadcasting

David suggested using elementwise_grad in the tanh example to add support for broadcasting and hence make the functions faster and easier to invoke (no need for map). This PR makes that change to examples/tanh.py, but I didn't change the code in the README.

This PR makes the examples/tanh.py file run almost instantly on my machine (less than a second total wallclock time including matplotlib setup and drawing), while on master it takes 13.4 seconds (!).
The README should probably include a note about using elementwise_grad for cases like this one, though it should still use the grad function as it does now to emphasize that as the primary function of interest.


add microvm support

Citation:

microvm is a machine type inspired by Firecracker and constructed after its machine model.
It's a minimalist machine type without PCI nor ACPI support, designed for short-lived guests. microvm also establishes a baseline for benchmarking and optimizing both QEMU and guest operating systems, since it is optimized for both boot time and footprint.

Conseqeuntly, it is a interesting platform for RustyHermit and should be supported

Currently, not on the TODO list


Unused newlinesAboveComments in reason_pprint_ast.ml

While working on https://github.com/facebook/reason/issues/41, in an episode of severe bikeshedding, I found that newlinesAboveComments doesn't get used anywhere.
You can write makeList ~newlinesAboveComments:1000, but those newlines are never printed.
prependSingleLineComment (https://github.com/facebook/reason/blob/master/src/reason_pprint_ast.ml#L1780) only prints newlines above doc comments.
I noticed it would clash with newLinesAboveItems. Is that the reason it doesn't get used anywhere?
An example:
type x = int;

/* Two newlines above */
let res = 1 + 1;

If you would write makeList ~newLinesAboveItems:1 ~newLinesAboveComments:1, with an implementation that considers newLinesAboveItems. There are two newlines inserted, when in fact I only want to get one newline for https://github.com/facebook/reason/issues/41.
Just putting this up here, no idea if this is intended?

Looks like we don't actually need this.
We changed the implementation of comments interleaving completely a few months back, this could be what's the remnant of previous implementation.

Killed


Go asset builder

Replaces util/rubyassetbuilder with util/goassetbuilder for building installer and dashboard assets, resulting in 2x+ faster build times through parallelization.
This still uses ruby as there are a few erb files, and calls out to nodejs for transforming ES6 modules and SCSS.

I think we can just call the new one util/assetbuilder since everything is Go.

@titanous comments addressed

@titanous comments addressed

@titanous comments addressed

LGTM


API for accessing parsed code?  ast-like?

pandas has hacked together a handful of lint-like scripts for cython code, but it would be much less kludgy if we could access something resembling an ast.  Is exposing something like that feasible?
The other use case I have in mind is generating a dependency graph.  Module X depends on modules A and B, ...  I can hack this together by regexing the pyx files, but cythonize must produce something like this internally, so using that must be more reliable.

This is a duplicate of #1628.


Poker card images being removed after reveal

since latest miro update the poker card images are removed instead of revealed after pressing the reveal button.

Reinstall of the app did not solve, multiple users and teams are affected

@Friandia Thank you very much for the issue. Sorry for the late response.
The problem has been solved with another Miro update.
By the way, we are testing a new version of the Planning Poker App with the Miro team. We hope to release it in a few days.


Sorting now runs in tensorboards image grid.

Some other changes from previous commits were also added to compiled file through gulp vulcanize.
To make gulp vulcanize work I had to do some other changes first, documented here. Should I make another PR with these (and also add some documentation for gulp vulcanize)? Or would that break the internal build?

Can one of the admins verify this patch?

ping for @dsmilkov or @danmane

@dsmilkov @danmane any feedback on this? I'll rebase it if it's worth the effort.

Can one of the admins verify this patch?

Looks good to me :+1: Rebase please

I'll rebase and merge.

I take that back, someone more qualified please do the rebase.

Im on it. The documentation for generating the distribution file is kinda
lacking, i'll try to fix that in another PR.
On Wed, Feb 10, 2016, 03:19 Martin Wicke<EMAIL_ADDRESS>wrote:

I take that back, someone more qualified please do the rebase.
—
Reply to this email directly or view it on GitHub
https://github.com/tensorflow/tensorflow/pull/839#issuecomment-182170203
.

@martinwicke rebased and ran gulp vulcanize again.

merged. Thanks!


Appstudio update test-component-pac-mubc

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-mubc-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
20 seconds
clone-repository

✅ Succeeded
33 seconds
build-container

✅ Succeeded
18 seconds
inspect-image

✅ Succeeded
18 seconds
deprecated-base-image-check

✅ Succeeded
9 seconds
clair-scan

✅ Succeeded
35 seconds
clamav-scan

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
13 seconds
optional-label-check

✅ Succeeded
13 seconds
label-check

✅ Succeeded
7 seconds
show-summary

✅ Succeeded
5 seconds
show-sbom


how to remove a window rounding

how to remove a window rounding?

I added this after my context init() call:
style = bp.get_style()
style.window_rounding = False
bp.set_style(style)


Ability to run CI against multiple SQLite versions

Issue #1276 happened because I didn't run tests against a SQLite version prior to 3.16.0 (released 2017-01-02).
Glitch is a deployment target and runs SQLite 3.11.0 from 2016-02-15.
If CI ran against that version of SQLite this bug could have been avoided.

Might be easier to do this using https://github.com/coleifer/pysqlite3 rather than try to replace the system sqlite3 on the Ubuntu GitHub Actions instances.


Enroll user with picture

Would it be possible to configure the SDK or modify the source code so that the camera can enroll a person just with a picture of the given person? Right now, when I try to enroll myself with a picture of myself, it says Face Not Detected or Spoofing.

I think is not possible, the F455 uses the depth camera to extract the face features to compare with the data base

I'm not with Intel, but I don't think that is in the plan.  What makes this
device special is the biometric security features.
That being said, there is nothing stopping you from triggering a
realsenseID enrollment action based on face recognition using a different
library.
On Fri., Jun. 11, 2021, 10:22 a.m. luca18, @.***> wrote:

Would it be possible to configure the SDK or modify the source code so
that the camera can enroll a person just with a picture of the given
person? Right now, when I try to enroll myself with a picture of myself, it
says Face Not Detected or Spoofing.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/RealSenseID/issues/86, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKTZOGEN7MAYSXJBORYIVHTTSIZ3HANCNFSM46RB3PFQ
.

Hello guys,
The feature is part of our plan and already in development
You will be able to enroll using .png image

@alexk1976
Interesting, I assumed it could not done since there was not a conversion mechanism between different models. https://github.com/IntelRealSense/RealSenseID/issues/77
Does a match with the png trigger an enrolment action, or will the submission of the png be the enrolment action?

@alexk1976
Do you have a timeline for this feature? What would be the approximate release date?

I assume it's be ready end of next week.
The feature will enable you to send us png image for enrollment only.
Authentication still will require real camera only as before and no plan to enable to authenticate using png images.

@luca18 You can try EnrollImage API in RSID v0.21.0
https://github.com/IntelRealSense/RealSenseID/releases/tag/v.21.0

Supported since version 0.21.0.


Issues on running app

error: src/actions/RestActions.js: # could not be cloned.

https://github.com/dimaportenko/magento-react-native/issues/187

https://github.com/dimaportenko/magento-react-native/issues/184

Not working for me

thanks for letting me know.


remove button to show commits between builds if it is the single buil…

…d page

tested on integ teletraan.
before this pr, the build page shows 3 buttons and the builds page also shows 3 buttons.
after this pr, the build. page shows only 2 buttons which is expected and the builds page shows 3 buttons which is also expected:


Add recv_buffer parameter for in_udp

I accidentally failed to update #1787. So I create a new one.

@repeatedly Could you check this again? If it is okay, I'll apply the change to #1788.


Update the game over message.

Update the game over message so people know how to play again!

Merging files


Recommended citation

What is your recommended citation for this package if it is referenced in a manuscript?  Should we refer directly to the GitHub repository?

Hi there, I am pro-GitHub, as such an action a. cites your input, b. promotes open science initiatives and c. generates traffic to your repository


Video with avc1.f4001e/f not supported on Firefox

Describe the bug
Video with avc1.f4001e and avc1.f4001f won't load on Firefox, showing an error & retry screen.
This fails in level-controller::filterAndSortMediaOptions.
To Reproduce

Open Firefox
Peruse this post: https://bsky.app/profile/bossett.social/post/3l3wru4mlxy2j

Expected behavior
I can watch and enjoy the video in full
Screenshots
:(

Details

Platform: Firefox (web)
Platform version: Firefox 130.0 (64-bit)
App version: Build version: 1.91.0; Bundle info: 9912029 (prod); Bundle date: 24091118; Platform: web

Additional context
I went through a bunch of posts from the videos-only feed that Paul had reposted, logging every codec I encountered. Every single video I saw had avc1.6400xx (xx being one of 14, 15, 1e, 1f, 20). Only Bossett's silly creation had the unplayable avc1.f4001e and avc1.f4001f.
This is also related to #5292, though the issues mentioned there are much more varied.

mediainfo for the file is in #5292 (and the file itself) as well - for me I can play in VLC/MPC-HC (not surprising though since I generated the file)

I don't know if this is a helpful data point, but I can play that video fine in Safari on iOS, but not in the iOS app.

Can you try this one - https://bsky.app/profile/bossett.social/post/3l3y5yjp6v22q - it's the same, but in 4:2:0 (i.e. I think it's not the codec)

Can you try this one - https://bsky.app/profile/bossett.social/post/3l3y5yjp6v22q - it's the same, but in 4:2:0 (i.e. I think it's not the codec)

Like others in that thread, this one plays fine for me, in both Safari on iOS and in the iOS app 👍

I also have a video that's playable in Chrome but not in Firefox, not sure if it's the same problem:
https://bsky.app/profile/omoikane.bsky.social/post/3l3ynwhkpxr2d
Attached is the original video that I tried to upload.
https://github.com/user-attachments/assets/f930ab92-d937-4f70-8c92-c2da4b280002
I will note that Twitter complained that it didn't like the aspect ratio, so maybe it's something about the aspect ratio that's causing trouble for both Bluesky and Twitter.  Mastodon handled it just fine, though.


Update memory_tune.md to provide guidelines for wasi-thread

The recommendations about when to -Wl,exporting malloc and free and when not to, need to be extended now that wasi-thread has been introduced, with a different memory model than the one used by WAMR thread.
Also, a question to clarify on the existing document:

It is recommended to export the malloc/free functions and disable app heap in single thread mode, and for multi-threading, as the libc heap isn't thread-safe

Is it still true that the libc heap isn't thread-safe, and if so, in what way?

Also, a question to clarify on the existing document:

It is recommended to export the malloc/free functions and disable app heap in single thread mode, and for multi-threading, as the libc heap isn't thread-safe

Is it still true that the libc heap isn't thread-safe, and if so, in what way?

the libc heap is thread-safe as far as it's built with a proper options.
if you are using wasi-threads with the wasm32-wasi-threads sysroot provided by wasi-sdk, you don't need to worry.


record2: markdown bullets do not show up on the page

Take a look at the url here...
https://dev.rebuildingakidney.org/chaise/record-two/#2/Protocol:Protocol/Title=Generation of kidney organoids from human pluripotent stem cells&Version=1.0
@RFSH what did you do to fix this issue in record?

In record I used the column type to attach the markdown-container class to the container. In here, I think we can also use isHTML flag.

This issue is fixed. This required adding the markdown-pattern class to record.html.


In-Call-Status-Bar View Size Shifts down and broken

Hi
@dzenbot  I am facing an issue when in call staus bar is ON.My view is getting truncated.Anyone have any idea why  my SLKTextViewController view is not getting fit inside window irrespective of In-call status bar toggle ON/OFF.It would be great if you help me out for this issue.
Thanks in advance.

It is coming when in call status bar is on at the time of launching the app.I am using the latest library.Other then this scenario it is working fine.


Bug report for redirects from broken child components.

Version
2.0.0
Test Case
https://github.com/dedan/router-redirect-debugging
Steps to reproduce
Clone the above repo and follow the instructions in the README.
Expected Behavior
The redirect in componentWillMount of the Container will kick in before the BrokenChildComponent gets rendered and therefore the redirect will not only update the URL, but will also correctly display the HappyChildComponent.
Actual Behavior
The URL gets updated to http://localhost:8100/#/happy, because of the redirect in the container. However the HappyChildComponent won't get displayed.

Use onEnter for redirects, not component lifecycle hooks.

Thank you very much for the fast answer.
On Fri, May 20, 2016 at 4:04 PM Jimmy Jia<EMAIL_ADDRESS>wrote:

Closed #3487 https://github.com/reactjs/react-router/issues/3487.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/reactjs/react-router/issues/3487#event-667180852


[TRACKING] metrics framework future refactors / cleanups

Motivation
Tracking pending cleanups, refactors or additional features for the Falco internal metrics framework https://falco.org/docs/metrics/

CC @FedeDP @sgaist @leogr

[ ] make libs_metrics_collector static https://github.com/falcosecurity/falco/pull/3192#discussion_r1599855210

Great discussions happening in https://github.com/falcosecurity/falco/pull/3140, few follow up items

[ ]  Falco's wrapper metrics num_evts is still missing in the Prometheus output since it requires greater code refactors
[ ] The metrics framework should target the primary event source only, as the metrics snapshots can realistically only expose one current view, especially for Prometheus. Plugin metrics should instead be supported via the new plugin metrics support; see  https://github.com/falcosecurity/libs/pull/1828
[ ] A consolidated and proper Falco metrics model is needed given that we now have even more outputs channels for the metrics (e.g. Prometheus)

The metrics framework should target the primary event source only, as the metrics snapshots can realistically only expose one current view, especially for Prometheus. Plugin metrics should instead be supported via the new plugin metrics support; see  new(plugin_api): add plugin metrics support libs#1828
[ ]  A consolidated and proper Falco metrics model is needed given that we now have even more outputs channels for the metrics (e.g. Prometheus)

Hey @incertum could you elaborate more on these two points?

@leogr I rewrote the text https://github.com/falcosecurity/falco/issues/3194#issuecomment-2111009270, is it more clear? happy to add more details.

Much clearer now, thank you!
Just one thought:

Since we can only provide one view of the metrics at a time

Why? I guess this is a current limitation, but we can fix it in the future. Am I wrong?
I believe that in the long run, all data sources should be first-citizen, and it shouldn't be technically impossible to accommodate this.

Much clearer now, thank you!
Just one thought:

Since we can only provide one view of the metrics at a time

Why? I guess this is a current limitation, but we can fix it in the future. Am I wrong? I believe that in the long run, all data sources should be first-citizen, and it shouldn't be technically impossible to accommodate this.

We can emit multiple rules outputs or lines into the output file ( I would not do it though), but for Prometheus there is just one endpoint to scrape at a time ... IMO there should be more separate plugin specific metrics handling, something that was started in libs. Most metrics are syscalls source specific or generic (e.g. CPU and memory usages or rules counters) anyways. In a way right now I can only think of number of events as useful to be plugin / source specific in case you have multiple sources.

From an operational point of view I like to have the falco metrics easily integrated with our metrics platform. So, I would like to thank everyone involved with exposing the falco metrics in a Prometheus compatible way.
I am not familiar with the falco code at all, so consider the following comments more as an outside view of things, not in any way directly mapping to any part of the code.
Falco metrics:

General metrics; unrelated to syscall or any plugin

falco version
start_timestamp
...

Process resource utilization metrics; unrelated to syscall or any plugin, preferably in standard naming conform the default C library for prometheus

num_cpus
cpu_seconds_total
memory_bytes_total
memory_used_bytes
...

Falco rule engine metrics; syscall/plugin event source can be a labeled dimension of the time serie

events_processed_total{event_source="syscall"} or events_processed_total{event_source="k8saudit"}
rulle_matches_total{event_source="syscall"} or rulle_matches_total{event_source="k8saudit"}

syscall specific metrics
custom plugin metrics; provided with the plugin API / SDK, implemented by the plugin

cloudtrail_xxx
github_repositories
gcpaudit_pubsub_errors_total
okta_xxx

Notes:

Process metrics (2) are not Falco specific, any application/process should be able to provide these metrics in a standard way. If you are familiar with these standard metrics, you can easily apply your existing knowledge to any application/process. E.g. for Golang and Java we even have default dashboards for this base set of metrics.
Falco rule engine metrics (3) are dimensioned by event source. An overall total can easily be calculated by the metrics platform, e.g. with PromQL sum without(event_source) (events_processed_total{}) and has not to be explicitly exposed by Falco
I realise syscall is the original falco event source and the plugin framework, and support for other event sources, has been implemented later. As of 0.35 plugins can also output syscall events, so to me 'falco drivers' and 'plugins' are just a way to provide event input to the falco rule engine and syscall is just one of the event sources. Hence the metrics being split into items 3, 4 and 5.
Different plugins can provide the same event source, e.g. the k8saudit, k8saudit_eks and k8saudit_gke plugin all provide the k8s_audit event source. So (5) are plugin specific metrics, not event source specific metrics.

Few more thoughts:

@mrgian is working on exposing (5), see PRs linked to above https://github.com/falcosecurity/falco/issues/3194#issuecomment-2111009270
As we explained earlier most of the current hiccups are because of a very complicated code refactor in the scap module that broke many of the metrics you listed under (2) when running Falco with a plugin only. @mrgian is also working on that, but it's a bit of a larger refactor. Meanwhile, https://github.com/falcosecurity/cncf-green-review-testing/discussions/14#discussioncomment-8610132 CPU and memory usages can be consumed externally, but as said we are working on fixing the Falco native support for that as well.

Any update on this? :thinking:

/assign @incertum

Updated checkboxes in
https://github.com/falcosecurity/falco/issues/3194#issuecomment-2111009270
Still pending a fix / no open PR yet

Falco's wrapper metrics num_evts is still missing in the Prometheus output since it requires greater code refactors.
re the evt.source reporting and plugin  metrics. Perhaps now that @mrgian  did this tremendously awesome refactor of allowing custom plugin metrics, I believe plugin metrics should be exposed over plugins. Also coming back to this now, isn't num_evts the sum of syscalls and plugin events anyways? Maybe evt.source in the metrics should be the primary event source?
@sgaist suggested some more cleanup and consolidation of falco_metrics and stats_writer duplicative code pieces.

In flight:

multi-platform metrics support -> in flight @mrgian  is on it, thanks again!

Lastly I do agree that there are many posts here. I may need to try to re-read everything again.

Compared to Falco 0.38.2 no breaking changes that I am aware of.

Indeed more breaking changes will happen :D See https://github.com/falcosecurity/falco/issues/3309.

I will move to 0.40.0 since we weren't able to finish everything in time for the 0.39 release.
/milestone 0.40.0

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

I guess we are in good shape here. Any updates?
/remove-lifecycle stale


Locksmith didn't run correctly

Microsoft PowerToys version
v0.64.1
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
File Locksmith
Steps to reproduce
Here is a screenshot of my setup

PowerToysReport_2022-12-04-20-05-06.zip
✔ Expected Behavior
The right-click menu displays "What to use this file"
❌ Actual Behavior
The right-click menu does not correctly display "What to use this file"
Other Software
null

Is this issue still relevant in v0.74.1? /needinfo


stop forcing WINIT_UNIX_BACKEND env var to fix wayland

Removes std::env::set_var("WINIT_UNIX_BACKEND", "x11") which breaks support on pure Wayland environments that have no x11 capabilities.
Fixes #161

This didn't blow up the way it used to, so I guess upstream must be working properly now. I'll have this merged shortly. Thanks!


fix(tap): set clone remote name to origin explicitly

Issue reproduction:
# Set the config value:
❯ git config --global clone.defaultRemoteName notorigin
# Add a tap:
❯ brew tap bazelbuild/tap
# Try updating and see the bug:
❯ brew update
# Remove the broken repo:
❯ brew untap bazelbuild/tap
# Apply the changes from this PR:
❯ echo "Apply change from PR"
# Add the tap again:
❯ brew tap bazelbuild/tap
# Try updating again, it works!
❯ brew update
# Clean up defaultRemoteName setting as you probably don't want it sticking around.
❯ git config --global --unset clone.defaultRemoteName

Output:
# Set the config value:
❯ git config --global clone.defaultRemoteName notorigin

# Add a tap:
❯ brew tap bazelbuild/tap
==> Tapping bazelbuild/tap
Cloning into '/usr/local/brew/Library/Taps/bazelbuild/homebrew-tap'...
remote: Enumerating objects: 23, done.
remote: Counting objects: 100% (23/23), done.
remote: Compressing objects: 100% (17/17), done.
remote: Total 314 (delta 7), reused 11 (delta 5), pack-reused 291
Receiving objects: 100% (314/314), 59.32 KiB | 311.00 KiB/s, done.
Resolving deltas: 100% (150/150), done.
Tapped 1 formula (33 files, 110.8KB).
It looks like you tapped a private repository. To avoid entering your
credentials each time you update, you can use git HTTP credential
caching or issue the following command:
  cd /usr/local/brew/Library/Taps/bazelbuild/homebrew-tap
  git remote set-url origin git@github.com:bazelbuild/homebrew-tap.git

# Try updating and see the bug:
❯ brew update
fatal: 'origin' does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
fatal: 'origin' does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
Error: Fetching /usr/local/brew/Library/Taps/bazelbuild/homebrew-tap failed!
fatal: invalid upstream 'origin/master'

# Remove the broken repo:
❯ brew untap bazelbuild/tap
Untapping bazelbuild/tap...
Untapped 1 formula (34 files, 110.9KB).

# Apply the changes from this PR:
❯ echo "Apply change from PR"
Apply change from PR

# Add the tap again:
❯ brew tap bazelbuild/tap
==> Tapping bazelbuild/tap
Cloning into '/usr/local/brew/Library/Taps/bazelbuild/homebrew-tap'...
remote: Enumerating objects: 23, done.
remote: Counting objects: 100% (23/23), done.
remote: Compressing objects: 100% (17/17), done.
remote: Total 314 (delta 7), reused 11 (delta 5), pack-reused 291
Receiving objects: 100% (314/314), 59.32 KiB | 421.00 KiB/s, done.
Resolving deltas: 100% (150/150), done.
Tapped 1 formula (33 files, 110.8KB).

# Try updating again, it works!
❯ brew update
Already up-to-date.

# Clean up defaultRemoteName setting as you probably don't want it sticking around.
❯ git config --global --unset clone.defaultRemoteName

Commits (oldest to newest)
0f48498ad fix(tap): set clone remote name to origin explicitly
Git now allows setting a different default remote name than origin
when you do a fresh clone by running:
git config --global clone.defaultRemoteName notorigin

This causes brew tap <tap_name> && brew update to fail, as it clones
with a different remote name in the tap and then expects the remote to
be origin in the update.
Fix this by explicitly setting the origin remote in the clone command.

Thanks so much for your contribution!  You rock, @gibfahn!


when capture a map, i get a blank background

hi, when i capture a map(amap, using canvas to display in the browser), i get some mark icon only, i can't see  the map background. what should i do to get all? thx , :)

我也遇到了这个问题，请问您解决了吗


LSC: Add overload items to completion list and expand the code

When you type __init__ in the completion list, and you accept it, it generates the whole method for you that delegates to base class:
    def __init__(self, *args, **kwargs):
        return super().__init__(*args, **kwargs)

We are now tracking this task in our internal Pylance Azure DevOps.


B2B3-164 ensured that videoFactory respected the stretchImages config option

changed the video poster image to use imageFactory so that it would
respect the stretchImages configuration option

@americos please review

i changed the poster image for a video item in blueimp to leverage the imageFactory used by the blueimp image items.  the imageFactory will handle setting the image to a background image & the background cover.
https://github.com/blueimp/Gallery/blob/master/js/blueimp-gallery.js#L953
i will need to open a PR to blueimp as well with this fix

@mtyeh411 Looks Good :+1:
@bernabas @csigg @prosanelli Ready for regulation

@asurin :+1:


Error in README for encryption

https://github.com/Mastercard/client-encryption-go/blob/938946b415bbd96beb6a4753f646388e2c3f0b01/README.md?plain=1#L206
Should this line say:
WithEncryptionPath("$", "$").
?

https://github.com/Mastercard/client-encryption-go/blob/938946b415bbd96beb6a4753f646388e2c3f0b01/README.md?plain=1#L206
Should this line say: WithEncryptionPath("$", "$"). ?

Hi @kaischwanke. This library supports field level encryption.
The encryption path can be used to encrypt/decrypt a specific path/element in the payload.
When an encryption/decryption path of "$" is given then the entire payload is encrypted/decrypted.
However you can also pass something like:WithEncryptionPath("$.foo", "$.encryptedFoo").
This will then encrypt the "foo" element to a new "encryptedFoo" element with the encrypted payload.
See here for an example:
https://github.com/Mastercard/client-encryption-go#-configuring-the-jwe-encryption-

Hi @danny-gallagher , thanks for your reply.
I do understand the syntax. However, I am referring to the README about "Encrypting Entire Payloads".
Is says to use "$" operator as encryption path.
But then in the example below, it shows field level encryption. It might be a copy&paste error?

Hi @danny-gallagher , thanks for your reply. I do understand the syntax. However, I am referring to the README about "Encrypting Entire Payloads". Is says to use "$" operator as encryption path. But then in the example below, it shows field level encryption. It might be a copy&paste error?

Apologies @kaischwanke, you're right! We've updated the README to reflect the correct configuration.
Thanks for pointing out this inconsistency!


Request to make markdown in the chat optional

Module Ver. 0.3.2
FVTT Ver. 0.6.5
Using App, but also on Chromium-based browsers
When using markdown editor in conjunction with Combat Utility Belt's enhanced conditions module it refuses to print the link for the journal entry of the condition, I suspect because markdown doesn't like being surrounded by html tags and the chat message being made uses a template.
Picture:

Should be easy to create:

Have both:

Combat Utility Belt 1.1.3
MEME 0.3.2

Make sure at least Output to Chat is checked in the CUBPuter under Enhanced Conditions
Put a condition from enhanced conditions on any token and have it output to chat

I think it would be pretty unreasonable to suggest accounting for every module posting to the chat. My personal suggestion would just be to allow the ability to turn off markdown in chat as a setting, but I'll leave it up to your judgement.
Discord Handle: @Mercedes-Benz#4852
Let me know if I missed something, I can post full module list as well but I tried on a fresh world with just those two and got the same results.

thinking about it its more of a Meme bug than anything else.
Note (to myself) for fixing:
Do i really still have to modify the enrichHTML method? if yes find a way that does not impact chat rendering.
Also chat markdown as option is a decent idea, which shouldn't be to much work to implement.


non-existant record

I have a fresh (version 4.0.6) version of server on Mac/10.12.6
I'm running the quick start tutorial on the deepstream.io website:
https://deepstream.io/tutorials/getting-started/javascript/index.html
When the client connects, I get the following on server:
INCOMING_CONNECTION | from undefined (<IP_ADDRESS>)
AUTH_SUCCESSFUL | open
VERSION_EXISTS | open tried to update record some-name to version 0 but it already was 0

So already on connect, there is a warning.
As soon as I try typing in the HTML input field, I get errors on the server:
RECORD_UPDATE_ERROR | Received update for non-existant record some-name

I tried replacing the server binary with a version running in NodeJS and swapping between client from CDN and NPM.
This example is very simple and pretty coherent with the documentation on records.
What am I missing? Is something broken?

Hello! This was sadly a small regression in a recent release. A fix will be released in the next day or two.

Okay, thank you.
I got it working now with current<EMAIL_ADDRESS>but had to backstep client to 4.0.2.

@thorsby Check this link pls https://deepstream.io/tutorials/upgrade-guides/v4/client/index.html. When i use await client.login() problem was resolved.

Yup! The issue is when you request a record before being logged in with offline disabled. There’s a lot of permutations, trying to find a way to simplify it a bit

Released 4.0.5 which should address this, please let me know if otherwise!

Was a client issue, hence the version number :D

Was a client issue, hence the version number :D
Nice comment, actually clarified 👍 I did not initial realise that client and server was not on same schedule.


A workflow calls a nonexistent make target

In debug_build.yml the workflow attempts to run make docker, but there is no docker target in the root makefile.
The workflow was never run in 'Actions'.

Hi @Vicky-Goldin , thanks for the issue.
I fixed the workflow targets.
The purpose of the workflow  is to generate and push images into the repository. It doesn't execute automatically, but requires a manual start.


Cannot sign in to proton-pass application

Describe the bug
On the date of this post, shortly after the system tray icon fix update made its way to the unstable repo, suddenly I cannot sign in to proton-pass application. It shows an error with the message “your session could not be resumed”.
I've signed out, and attempted to sign back in with no luck. It does accept my password and even my 2FA code, but once past that prompt, it fails to sign in and provides the error I mentioned above.
There was a new version update prompt earlier today in the app when I was able to sign in shortly after the system tray icon fix. This could potentially be the reason for the sign in error. Perhaps an upgrade to the new version is required?
Steps To Reproduce
Steps to reproduce the behavior:

Sign in to proton-pass unstable nixpkg.
Observe the error.

Expected behavior
It should sign in as usual.
Screenshots

Additional context
A new version is available.
Notify maintainers
@luftmensch-luftmensch
@massix
@SebTM

I think I know where the problem is, and I apologize for this. As a temporary workaround, you can start proton-pass using the environment variable ELECTRON_FORCE_IS_PACKAGED set to the value 1, just be aware that the tray icon won't load.
I forgot to embed the TLS certificates inside the ASAR archive (I must have tested using an asar version I built locally instead of the one from the result folder). Will fix it ASAP, I am truly sorry for this!

Contributor

If this indeed is the root cause, why is the overrideAttrs of the latest version from Proton's website still resulting in the same issue?

Because it is not related to the version of Proton Pass.  The way Electron handles ASAR applications right now is kind of a mess and based on some heuristics which may or may not work on NixOS.  I kind of forced the things with the tray icons (putting there directly inside the ASAR), which worked well but broke the handling of SSL certificates.
The good news is that I managed to solve it, I'll open a PR first thing tomorrow morning, just need to clean a couple of things up.
Again, I am really sorry for the issue, I don't know why I thought it was working well for me before, might be that I wasn't testing the right version after all.

If you want to test the fix you can nix run github:massix/nixpkgs/fix/proton-pass-login#proton-pass, you should have both the tray icon working and the ability to log in.

Well, if you want to keep using my version, you have to launch Proton Pass using the nix run command above or add my repository as an input to your flakes and create an overlay.

nix run github:massix/nixpkgs/fix/proton-pass-login#proton-pass (without the package update) works for me, thanks @massix!
It's quite a nice program I can see why you use it over the browser extension.


customize vstest summary

Hi Guys,
We have some personal requests about the summary of VSTest. I know we could modify the source codes to build our own VSTest. But except this, just want to check if VSTest already has the feature : ).
The current of summary is(for example):
Total tests: 518. Passed: 517. Failed: 0. Skipped: 1.
Could we count the total tests to 517(means ignore the number of skip tests)? Or even not generate the summary, we could process the test result to get our own result?
Environment

Windows OS version 1903
VSTest version 15.8.0

I am very appreciate for any input : ).
Regards,

@Coder-Bryan You'd probably have write your own logger in that case.
You can follow https://github.com/microsoft/vstest-docs/blob/master/docs/report.md