Fixes to DS functional tests

Change parameter from file to path
Ensure the stderr stream contains everything instead of just a part of the stream

Codecov Report

Merging #8998 into master will decrease coverage by 0.3%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##           master    #8998      +/-   ##
==========================================
- Coverage   57.63%   57.32%   -0.31%
==========================================
  Files         530      530
  Lines       27657    27330     -327
  Branches     4183     4188       +5
==========================================
- Hits        15939    15666     -273
+ Misses      10824    10770      -54
  Partials      894      894

Impacted Files
Coverage Δ

...rc/client/datascience/jupyter/jupyterConnection.ts
71.55% <71.42%> (+0.24%)
:arrow_up:

...nt/datascience/jupyter/invalidNotebookFileError.ts
0% <0%> (-60%)
:arrow_down:

...jupyter/kernels/jupyterKernelPromiseFailedError.ts
0% <0%> (-33.34%)
:arrow_down:

...ent/datascience/jupyter/jupyterWaitForIdleError.ts
0% <0%> (-33.34%)
:arrow_down:

src/client/formatters/serviceRegistry.ts
50% <0%> (-25%)
:arrow_down:

.../client/datascience/jupyter/jupyterConnectError.ts
50% <0%> (-25%)
:arrow_down:

src/client/language/characters.ts
14.7% <0%> (-19.58%)
:arrow_down:

...cience/jupyter/jupyterDebuggerNotInstalledError.ts
33.33% <0%> (-16.67%)
:arrow_down:

...ience/jupyter/jupyterDebuggerRemoteNotSupported.ts
50% <0%> (-10%)
:arrow_down:

src/client/common/utils/icons.ts
75% <0%> (-8.34%)
:arrow_down:

... and 28 more

Continue to review full report at Codecov.

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


[stable/drupal] Set dependencies to the latest version

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Logs:
Welcome to the Bitnami drupal container
Subscribe to project updates by watching https://github.com/bitnami/bitnami-docker-drupal
Submit issues and feature requests at https://github.com/bitnami/bitnami-docker-drupal/issues

WARN  ==> You set the environment variable ALLOW_EMPTY_PASSWORD=yes. For safety reasons, do not use this flag in a production environment.
nami    INFO  Initializing apache
apache  INFO  ==> Patching httpoxy...
nami    INFO  apache successfully initialized
nami    INFO  Initializing php
nami    INFO  php successfully initialized
nami    INFO  Initializing mysql-client
nami    INFO  mysql-client successfully initialized
nami    INFO  Initializing drupal
mysql-c INFO  Trying to connect to MySQL server
mysql-c INFO  Found MySQL server listening at sad-yak-mariadb:3306
mysql-c INFO  MySQL server listening and working at sad-yak-mariadb:3306
drupal  INFO  Installing Drupal with drush...
drupal  INFO  Cleaning the cache..

/ok-to-test

/ok-to-test
/lgtm


Replace dropbox-js with new Dropbox JS SDK

The dropbox-js fork uses the deprecated v1 API and is unmaintained. The Dropbox team have released an SDK for the v2 API which is actively maintained: https://github.com/dropbox/dropbox-sdk-js

This is done


Small dispense_pip Docstring correction

is this true? the firmware docs say otherwise

To my knowledge yet; I can see no physical reason cLLD should not work and there are loads of videos surface dispensations into liquids which would not be possible if cLLD is not possible +dispensing_mode: (Type of dispensing mode 0 = Partial volume in jet mode 1 = Blow out in jet mode 2 = Partial volume at surface 3 = Blow out at surface 4 = Empty tip at fix position) indicates this capability exists.
I can check it empirically on Monday as well.

Here is an example of surface_dispense (though I could only find it for the Zeus quickly) min2:10

Interesting. I'll just wait for you to empirically verify this :)

I can definitively say that Hamilton machines are capable of cLLD during dispensation.
This very feature might be a key factor to their robustness.

Thanks!


Fix #120

Fixes #127
Fixes #120
Needs to be merged after https://github.com/starlinglab/integrity-preprocessor/pull/50

Still needs testing, maybe @YurkoWasHere can help with


cli: exit with proper status code

Why do this?
Utilities invoked at the shell are expected to. clj-kondo and cljfmt
check do, for example. So does the cognitect test-runner.  The really
good reason to is for tools like make, which stop after encountering the
first non-ignore error. Plus, it enables CI builds to indicate a failed
status correctly.
On the implementation
For reasons I cannot fathom, -1 has been chosen as the special "error"
status, so I just keep the absolute value.  Thankfully, 0 is already
considered success. I also clamp to 255 as the maximum value, for
safety.
I stuck with the existing style of not wrapping functions in (fn [] ,,,)
when not necessary, preferring (comp) and (partial).

Somehow this PR fell off my radar…and then I got bitten by the same issue. :-) Thanks for doing this. Merging now.


Adding Recommendation Updater Service [KRUIZE-VPA Integration]

Description
This PR introduces a service for updating deployments for auto or recreate mode experiments.
Type of change

[ ] Bug fix
[x] New feature
[ ] Docs update
[ ] Breaking change (What changes might users need to make in their application due to this PR?)
[ ] Requires DB changes

How has this been tested?
Tested on both the ResourceHub cluster.
Test Configuration
Kubernetes clusters tested on: ResourceHub (OpenShift)
Checklist :dart:

[x] Followed coding guidelines
[x] Comments added
[ ] Dependent changes merged
[ ] Documentation updated
[ ] Tests added or updated

@shekhar316 Can you please add documentation for the "auto" and "recreate" modes in the design folder. Currently we have an docs for Monitoring and Experiment modes and we can create a common doc in UpdateModes.md


ova image: conda does not have write access

$ conda config --add channels r
$ conda config --add channels bioconda
$ conda install seqtk
Fetching package metadata: ........
Solving package specifications: .........

Package plan for installation in environment /opt/miniconda:

The following packages will be downloaded:

    package                    |            build
    ---------------------------|-----------------
    seqtk-1.2                  |                0          74 KB
    conda-env-2.5.0            |           py27_0          28 KB
    ruamel_yaml-0.11.7         |           py27_0         352 KB
    conda-4.1.2                |           py27_0         198 KB
    ------------------------------------------------------------
                                           Total:         652 KB

The following NEW packages will be INSTALLED:

    ruamel_yaml: 0.11.7-py27_0 (soft-link)
    seqtk:       1.2-0         (soft-link)

The following packages will be UPDATED:

    conda:       4.0.5-py27_0 --> 4.1.2-py27_0  (soft-link)
    conda-env:   2.4.5-py27_0 --> 2.5.0-py27_0  (soft-link)

Error: Missing write permissions in: /opt/miniconda
#
# You don't appear to have the necessary permissions to install packages
# into the install area '/opt/miniconda'.
# However you can clone this environment into your home directory and
# then make changes to it.
# This may be done using the command:
#
# $ conda create -n my_root --clone=/opt/miniconda

This recipe does reference conda as far as I can tell - @davebx did you add this manually after the fact?

I've opened an untested PR that should add miniconda in a fashion local to the ubuntu user. I think this is appropriate for the tutorial and for development purposes.

seems fixed in http://images.galaxyproject.org/planemo/20160623.ova


AddJsonClient doesn't respect clientName parameter

AddJsonClient extension method provided by the library has clientName argument to create named clients but actually never use it.
Actually it is not a bug, it was never implemented. We should fix it.

взял в работу

Closed in #35. Thanks to @NYMEZIDE


Bug: ZT Key Vault deployment

Steps to reproduce:
Select ZT option during deployment.
Results in 2 key vaults created - 2nd key vault has zt in the name with nic and PE resources.

@jensheerin thanks for your feedback.
Current code creates 2 keyvaults when ZT is enabled, the keyvaults are:

Workload keyvault:

Name: kv----<4CharRandomString>
Purpose: store deployment credentials (domain join and local admin) as secrets

ZT keyvault:

Name: kv-zt----<4CharRandomString>
Purpose: store disk encryption key

Please let us know if above wasn't the case for your deployment, happy to further check and assist.
@jamasten Jen brings fair concerns about the additional cost of another keyvault and private endpint, let's discuss options/impact of consolidate in just one keyvault for credentials plus ZT.

@danycontre @jensheerin
Hey team, Jen and I discussed this before I implemented the code.  The reason we are using 2 key vaults was based on ZT principles.  The documentation states that RBAC isolation is not enough for Azure resources.  Having a separate key vault with a private endpoint allows network isolation on top of the separation of duties. Private endpoints are charged based on egress traffic right? How is there a cost difference between having 1 or 2 key vaults if the network egress is the same? I'm happy to dig into this more to find a happy, cost effective solution for our customers.

Also, from a gov perspective, typically a COMSEC custodian manages the keys and should not have access to anything but the keys in a key vault.

Let's discuss https://learn.microsoft.com/en-us/azure/key-vault/general/best-practices#use-separate-key-vaults

Could we improve the naming convention of the key vault to reflect the purpose e.g. secrets and keys.

@swathibhat1 let's created this item in the features backlog to time the work on it.


Last code snippet in Step 12 has 3 duplicate import lines

import android.content.Context
import androidx.test.core.app.ApplicationProvider
import androidx.work.ListenableWorker.Result
import android.content.Context
import androidx.test.core.app.ApplicationProvider
import androidx.work.ListenableWorker.Result
...

Thank you, I've corrected the text in the internal codelab source document and it will be visible soon, once we publish a new version on the website.

Thank you, I've corrected the text in the internal codelab source document and it will be visible soon, once we publish a new version on the website.


Automated Contributors List with Scroller Added to README 📜✨

This PR integrates the All Contributors Bot for dynamic contributor updates and implements a horizontal scroller to display contributors in a compact format. These changes streamline maintenance and enhance readability in the README.
fixes: #1953
@ajay-dhangar

@ajay-dhangar DONE WITH THE CHANGES


pie_path

hi sir,  I`m so sorry to bother you.
when i input "python train_test.py 1", it will occur :
Using TensorFlow backend.
Traceback (most recent call last):
File "train_test.py", line 182, in
main(train_test=train_test)
File "train_test.py", line 175, in main
intent_model_path = train_intent(train_test=train_test)
File "train_test.py", line 135, in train_intent
imdb = PIE(data_path=os.environ.copy()['PIE_PATH'])
File "/home/guofengpeng/my/pie/PIEPredict/pie_data.py", line 57, in init
'pie path does not exist: {}'.format(self._pie_path)
AssertionError: pie path does not exist: /path/to/PIE/data/root
but i cannot find where  write "/path/to/PIE/data/root".

The path/to/PIE/data/root is a placeholder, it should be changed to the directory where the PIE dataset is stored on your computer.

thanks，my brothor.


Extended KWS notebook and build for network with py_speech pre-processing

Hi @maltanar,
here it finally is ^^
The demo, which classifies audio files. I had to kind of hack together the input quantization, since onnx is not available on the PYNQ-Z1.
Otherwise the demo is hopefully pretty straightforward. I tested it on one of the PYNQ-Z1 boards and it seems to work fine, but please give it another try. I also added some description of where the dataset comes from and how the input feature extraction works.

Looking great, thanks @HenniOVP !


Last Week in Pony - February 28, 2021

Audio from the February 23, 2021 Pony development sync call is available.

Version 0.39.0 of ponylang/ponyc has been released.
See the release notes for more details.

Version 0.5.0 of ponylang/corral has been released.
See the release notes for more details.

Audio from the March 02, 2021 Pony development sync call is available.


getting " Error 400 (Bad Request): all shards failed [type=search_phase_execution_exception]"

Hi there. Probably i do something wrong, but i cannot spot my mistake.
I used the sample code to create the "twitter" index, write the user "olivere". It all works.
But when it comes to the term query for user "olivere" i get:
Error 400 (Bad Request): all shards failed [type=search_phase_execution_exception]
Even if i create  client and then only try the term query, the error gets thrown.
Code:
	termQuery := elastic.NewTermQuery("user", "olivere")
	searchResult, err := client.Search().
		Index("twitter").   // search in index "twitter"
		Query(termQuery).   // specify the query
		Sort("user", true). // sort by "user" field, ascending
		From(0).Size(10).   // take documents 0-9
		Pretty(true).       // pretty print request and response JSON
		Do(ctx)             // execute
	if err != nil {
		panic(err)
	}

When i issue the same term query with my rest client, it works perfectly:
POST /_search
{
  "query": {
    "term" : { "user" : "olivere" }
  }
}

everything green in the health output, index and all shards created, no log output from ES
(using the official ES docker container)
Which version of Elastic are you using?
[X ] elastic.v5 (for Elasticsearch 5.x)
Please describe the expected behavior
search should return a result
Please describe the actual behavior
error message as described above
Any steps to reproduce the behavior?
Just run the example test code.
Any ideas?

I guess i found out myself. Problem appears to be the missing mapping during index creation. Unfortunately, ES sends the wrong exception so i needed to enable debug logging to find the cause:
Fielddata is disabled on text fields by default. Set fielddata=true on [user] in order to load fielddata in memory by uninverting the inverted index. Note that this can however use significant memory. Alternatively use a keyword field instead.

Yes, this is the root cause. Elastic is only a client. Please consult the Elasticsearch documentation about how to correctly do your mappings. There's an example of a correct mapping on the Elastic homepage that you should be able to use as a starter.

@olivere - You should remove the bad code from README.md

@JAlexoid The code in the README reflects a snippet that doesn't work right away. It has a link to a complete example. That code should work fine in Elastic 3.x (for ES 2.x).
The home page has a fully working example for v5.
I agree that the README for v5 should have a link to a fully working example for v5. You're free to send a PR.

@JAlexoid Ha! And you did... 👍

Was in the same boat. Would be great if the error returned can include RootCause Reasons as well... it helps debugging so much. Thanks!

@taiyangc If Elasticsearch returns a structured error, you should be able to handle it like this. It returns an Error structure that should include the error response from ES.

@olivere ah yes sorry should have read the docs first! thanks!


Applink 17030 atf does not validate values in expect notification with one occurence

Fixed handling of one expectation in connecttest.EXPECT_NOTIFICATION.
mobile_session.ExpectNotification changed to allow validate notifications from another mobile sessions.
Related to: APPLINK-17030
@AByzhynar @VVeremjova @malirod please review.

Is it possible to update unit-test for this fix?

Reviewed

@pestOO There are not existed test for connecttest.lua or mobile_session.lua which I can find.

Reviewed

Reviewed

Reviewed

reviewed. @AGaliuzov  please merge


Documentation and examples for earley_forest

Here is API reference for members of earley_forest.py.
Advanced examples have also been added to demonstrate the usage of ForestVisitor and TreeForestTransformer.

Awesome! Thanks for the PR!


Changes Not Saved in Database with CustomBinaryView and Load Options

Under certain conditions, changes to a BinaryView would fail to be saved to a bndb. This could be triggered specifically using the Parse interface, or loading with options.

Fixed in 2.02199-dev.


ci: increase timeout for e2e java-binding test

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Very high timeout rate. PTAL if it's expected or there are some regressions.

Checklist

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

Documentation

[ ] My PR contains user-facing changes.

Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note

May not need this PR after https://github.com/risingwavelabs/risingwave/pull/10552


Add a processor for Sentry to report failing URLs

Sentry is reporting some exceptions from requests getting a status code 404, but isn't reporting on exactly which URLs are being requested. This processor should add that information. There's even a test!
@nedbat or @sarina, can one of you review this? The raven documentation is here. (raven is the name of the client software that reports clients to Sentry.)

@smarnach FYI - i think this is related to what you were seeing

in that, yes you were seeing some errors, but we don't even know what they were. now we'll have better reporting.

I think this is good :+1:
We'll look into adding validation to the process_pr function as well.

@sarina Good to see this change!  (And good to know where the code for the webhoook is – I'll read it on the next train journey, so I know what this thing is doing.)

Yeah Sven we've made a few PRs this morning to explicitly restrict the
process_pr task to repos the bot has permission on.
On Tue, Nov 3, 2015 at 11:24 AM, Sven Marnach<EMAIL_ADDRESS>wrote:

@sarina https://github.com/sarina Good to see this change! (And good to
know where the code for the webhoook is – I'll read it on the next train
journey, so I know what this thing is doing.)
—
Reply to this email directly or view it on GitHub
https://github.com/edx/openedx-webhooks/pull/54#issuecomment-153404623.

@sarina That's even better than improving the error messages as I suggested.  If it isn't doing anything in the first place, there is little room for things going wrong. :)


Error: Cannot find module 'fs-extra'

After "npm install fs-extra" command error is same.

@tsogtbayar-mn
Do you have any more details? Following the template helps us to help you:

what were you doing?
what were you expecting to happen?
is this something you came across when using caliper?
...

closing due to lack of activity


Unexpected source state when querying: 2

I've seen this error appear transiently in my logs when using eventsByPersistenceId:
Unexpected source state when querying: 2

I think it tracks down to https://github.com/safety-data/akka-persistence-redis/blob/17563b5f236ee88e57b8cc70b896270fa538c044/src/main/scala/akka/persistence/query/journal/redis/EventsByPersistenceIdSource.scala#L203-L205
which seems like it can get hit by https://github.com/safety-data/akka-persistence-redis/blob/17563b5f236ee88e57b8cc70b896270fa538c044/src/main/scala/akka/persistence/query/journal/redis/EventsByPersistenceIdSource.scala#L105-L107
Am I following this code path correctly?

Hi,
you are right, the state should be set to Idle before calling query. I will fix it an publish a new snapshot version tomorrow.
Thanks for reporting.

I believe this is now fixed in the latest snapshot version. Can you congirm?


Support disk.sector-size hardware requirement for mrack

Pull Request Checklist

[x] implement the feature
[ ] write the documentation
[x] extend the test coverage
[x] update the specification
[ ] adjust plugin docstring
[ ] modify the json schema
[ ] mention the version
[ ] include a release note

Updated^^

Rebased,and updated, this one is ready to be merged^^

Unrelated failures, merging.


Embedder lints not being configured.

For example, this flutter/lib/_embedder.yaml does not enable camel_case_types (as expected).
analyzer:
  strong-mode: true
  errors:
    strong_mode_static_type_error: false
    unused_local_variable: false
  language:
    enableSuperMixins: true
linter:
  rules:
    - camel_case_types

Blocks: https://github.com/flutter/flutter/issues/624.

:metal:
Fixed w/ https://github.com/dart-lang/sdk/commit/1e4cea355f6eddf5b39609f5351d63ba21bf6e9b.


feat: Support storing KubeArmor policies to OCI registries

Part of https://github.com/kubearmor/KubeArmor/issues/1130
Support Karmor to interacting with OCI registry for pushing, pulling KubeArmor policies.
Tasks

[x] Complete Push

[x] Validate policy before pushing
[x] Authenticate with OCI registry

[x] Complete Pull

[x] Validate policy after pulling
[x] Authenticate with OCI registry

[x] Refactor
[x] Testing
[ ] Logging

It would be great to document this. Can you do a small writeup on:

What was the problem statement for doing this?
How would users benefit?
How would users use this?
Any refs etc.


No error occurs when creating a table named 'Table' if there is already a table named 'table'

Storage Explorer Version: 1.22.0-dev
Build Number: 20211118.3
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Monterey 12.0.1
Architecture: ia32/x64
How Found: From running test cases
Regression From: Previous release (1.21.3)
Steps to Reproduce

Expand one storage account -> Tables.
Create a new table using the name 'table'.
Create another table using the name 'Table'.
Check an error occurs.

Expected Experience
An error occurs.

Actual Experience
No error occurs and succeed to create a table with name 'Table'.

Additional Context
The table 'Table' disappears after manually refreshing.

Verified this issue on the build main/20211121.3. Fixed


[AIRFLOW-1207] Enable utils.helpers unit tests

Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following

AIRFLOW-1207 Enable utils.helpers unit tests

Description

[x] Currently tests.utils.helpers unit tests are not running. This PR make them to be run.

Tests

[x] Helpers tests enabled.

Codecov Report

Merging #2300 into master will increase coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2300      +/-   ##
==========================================
+ Coverage   69.11%   69.21%   +0.09%
==========================================
  Files         144      144
  Lines       11023    11023
==========================================
+ Hits         7619     7630      +11
+ Misses       3404     3393      -11

Impacted Files
Coverage Δ

airflow/models.py
87.1% <0%> (+0.04%)
:arrow_up:

airflow/utils/helpers.py
56.06% <0%> (+5.78%)
:arrow_up:

Continue to review full report at Codecov.

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


Align text on buttons to center

Please align text+icon to the center of buttons "change", "next", "add all", "show QR", "share address"  etc...

checked


Fault injection for NMR APIs: NmrDeregisterProvider, NmrDeregisterClient

Describe the bug
This issue is raised (as a follow up from #2101) to analyze the need to add Fault Injection for the following NMR APIs:

NmrDeregisterProvider
NmrWaitForProviderDeregisterComplete
NmrDeregisterClient
NmrWaitForClientDeregisterComplete

If needed, the fix can be added in this item.
OS information
Windows 2019, 2022
Steps taken to reproduce bug
Fault injection runs for

unit_tests.exe
netebpfext_unit.exe

Expected behavior
The test cases should pass with the added fault injection for the specified NMR APIs.
Actual outcome
With added fault injection, there are observed failures.
Additional details
No response

Pasting some of the failures observed:
netebpfext_unit.exe
0.006 s: sock_ops_context
sock_addr_context
C:\ebpf-2023\ebpf-for-windows\tests\netebpfext_unit\netebpfext_unit.cpp(676)
...............................................................................
C:\ebpf-2023\ebpf-for-windows\tests\netebpfext_unit\netebpfext_unit.cpp(719): FAILED:
REQUIRE( sock_addr_program_data->context_create( nullptr, 0, (const uint8_t*)&input_context, sizeof(input_context), (void**)&sock_addr_context) == 0 )
with expansion:
12 == 0
0.000 s: sock_addr_context

unit_tests.exe
show verification bpf.o
C:\ebpf-2023\ebpf-for-windows\tests\end_to_end\netsh_test.cpp(284)
...............................................................................
C:\ebpf-2023\ebpf-for-windows\tests\end_to_end\netsh_test.cpp(284): FAILED:
{Unknown expression after the reported line}
due to unexpected exception with message:
ebpf_provider_load failed
0.000 s: show verification bpf.o
C:\ebpf-2023\ebpf-for-windows\x64\Debug>
0.000 s: bpf_object__open_file with .dll
bpf_object__load with .o
C:\ebpf-2023\ebpf-for-windows\tests\unit\libbpf_test.cpp(2372)
...............................................................................
C:\ebpf-2023\ebpf-for-windows\tests\unit\libbpf_test.cpp(2372): FAILED:
{Unknown expression after the reported line}
due to a fatal error condition:
SIGSEGV - Segmentation violation signal
0.000 s: bpf_object__load with .o

bindmonitor-tailcall-native
C:\ebpf-2023\ebpf-for-windows\tests\end_to_end\end_to_end.cpp(984)
...............................................................................
C:\ebpf-2023\ebpf-for-windows\tests\end_to_end\end_to_end.cpp(984): FAILED:
{Unknown expression after the reported line}
due to a fatal error condition:
SIGSEGV - Segmentation violation signal
0.000 s: bindmonitor-tailcall-native
test cases:   71 |   50 passed | 21 failed
assertions: 3397 | 3376 passed | 21 failed
C:\ebpf-2023\ebpf-for-windows\x64\Debug>

map-interpret
C:\ebpf-2023\ebpf-for-windows\tests\end_to_end\end_to_end.cpp(987)
...............................................................................
C:\ebpf-2023\ebpf-for-windows\tests\end_to_end\end_to_end.cpp(987): FAILED:
{Unknown expression after the reported line}
due to a fatal error condition:
SIGSEGV - Segmentation violation signal
0.000 s: map-interpret

0.000 s: bindmonitor-ringbuf-interpret
0.319 s: utility-helpers-jit
utility-helpers-native
C:\ebpf-2023\ebpf-for-windows\tests\end_to_end\end_to_end.cpp(986)
...............................................................................
C:\ebpf-2023\ebpf-for-windows\tests\end_to_end\end_to_end.cpp(986): FAILED:
{Unknown expression after the reported line}
due to a fatal error condition:
SIGSEGV - Segmentation violation signal
0.000 s: utility-helpers-native

map_reuse_2-jit
C:\ebpf-2023\ebpf-for-windows\tests\end_to_end\end_to_end.cpp(2217)
...............................................................................
C:\ebpf-2023\ebpf-for-windows\tests\end_to_end\end_to_end.cpp(2217): FAILED:
{Unknown expression after the reported line}
due to a fatal error condition:
SIGSEGV - Segmentation violation signal
0.000 s: map_reuse_2-jit

0.520 s: xdp-encap-reflect-v6-jit
xdp-encap-reflect-v4-interpret
C:\ebpf-2023\ebpf-for-windows\tests\end_to_end\end_to_end.cpp(1762)
...............................................................................
C:\ebpf-2023\ebpf-for-windows\tests\end_to_end\end_to_end.cpp(1762): FAILED:
{Unknown expression after the reported line}
due to a fatal error condition:
SIGSEGV - Segmentation violation signal
0.000 s: xdp-encap-reflect-v4-interpret

0.520 s: xdp-encap-reflect-v6-jit
xdp-encap-reflect-v4-interpret
C:\ebpf-2023\ebpf-for-windows\tests\end_to_end\end_to_end.cpp(1762)
...............................................................................
C:\ebpf-2023\ebpf-for-windows\tests\end_to_end\end_to_end.cpp(1762): FAILED:
{Unknown expression after the reported line}
due to a fatal error condition:
SIGSEGV - Segmentation violation signal
0.000 s: xdp-encap-reflect-v4-interpret

Closing as 'Won't fix'.
The team discussed that the NmrDeregisterProvider and the other APIs raised in this issue are not intended for fault injection return code. If injected, there can be cleanups not handled in ebpf design code, resulting in Heap-use-after-free.

The exception/crash caught in this analysis below was a known issue and fixed in  #2420
_test_helper_end_to_end::~_test_helper_end_to_end()
{
...
// Verify that all maps were successfully removed.
uint32_t id;
if (!ebpf_fuzzing_enabled) {
REQUIRE(bpf_map_get_next_id(0, &id) < 0);  <<<<
REQUIRE(errno == ENOENT);
}
...
}


chore: add storybook app

Description
As per title

Codecov Report

Merging #33 (22c534e) into main (3de4c01) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #33   +/-   ##
=======================================
  Coverage   95.22%   95.22%
=======================================
  Files          43       43
  Lines         482      482
  Branches      212      212
=======================================
  Hits          459      459
  Misses         22       22
  Partials        1        1


cmd/compile: mark shape type dupok

What version of Go are you using (go version)?

$ go version
go version devel go1.19-fd6c556dc8 Sat Apr 30 04:04:40 2022 +0000 linux/amd64

Does this issue reproduce with the latest release?
No, it marks all type description dupok in 1.18.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/root/.cache/go-build"
GOENV="/root/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/mnt/d/01.golang/01.GOPATH/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/mnt/d/01.golang/02.playground/01.demo/playground/dwarftype/gopath/"
GOPRIVATE=""
GOPROXY="https://goproxy.cn,direct"
GOROOT="/mnt/d/01.golang/03.godev/go"
GOSUMDB="off"
GOTMPDIR=""
GOTOOLDIR="/mnt/d/01.golang/03.godev/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="devel go1.19-fd6c556dc8 Sat Apr 30 04:04:40 2022 +0000"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/mnt/d/01.golang/02.playground/01.demo/playground/dwarftype/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build3478615306=/tmp/go-build -gno-record-gcc-switches"

What did you do?
go install -buildmode=shared -linkshared std

What did you expect to see?
Success.
What did you see instead?

# /tmp/go-build415098603/b236/libstd.so
2022/04/30 22:37:52 duplicated definition of symbol type.go.shape.*uint8_0, from crypto/tls and crypto/elliptic

Marked it "needs investigation" to get it off the dashboard, but it is a bug, repros just fine for me, the fix looks sensible, and I pinged the likely reviewers.

@gopherbot please open a backport issue for 1.18.


Without external access

Is it possible to use this without external access? I use a local cluster and a local LDAP server, so it should be possible?

Hi @anton-johansson,
What do you mean by "local" and "external" ?
If you talk about local network, yes it works (ex: your company ldap on the local network, yes it works)
Regards,

I can't even remember what I meant. I got everything working with company local domains (login.svc.example.local and dex.svc.example.local). Sorry for taking your time!


Builds breaking between expo-cli v0.58 and v0.59

Appologies for using a blank issue. I just wanted to stop bumping in discord. Also hoping this attracts other people with the same issue.
When running a build (local and remote) I get:

Error: [ios.infoPlist]: withIosInfoPlistBaseMod: ENOENT: no such file or directory, open '/Users/spiderclam/projects/mobile/projectname/GoogleService-Info.plist'
Code: ENOENT

This makes sense, because the file doesn't exist until after eas-build-pre-install (That's where I place the correct GoogleService files based on the profile used).
So, for some reason it's running a config plugin before it should be running a plugin.
I've narrowed it down to somewhere between v0.58 and v0.59, since v0.59 (and later releases, too) is where the issue starts popping up.
Additional remarks:

I've tried removing all plugins from app.json and it's still giving me the same error.
It's looking for the GoogleService file in my project root instead of the build root (which is usually some tmp dir). That's weird, because so far I've noticed eas-build-pre-install is run relative to the build root and not the project root.
I noticed Expo-config was bumped a major version between v0.58 and v0.59. Probably not related but who knows.

Can you share a link to the failing build?

@dsokal No, because it never gets uploaded. This error shows up locally.
This runs before eas-build-pre-install, which itself shouldn't run until the build starts afaik. This error shows up immediately after running build. I'll add the full output below (so you can see the order of things which might be helpful?):
Preparation:
$ npm i -g<EMAIL_ADDRESS>// Boring output

Note: Anything above v0.59 (tested up to v1.1.1 has the exact same behaviour._
Local:
$ eas build --profile production --local
★<EMAIL_ADDRESS>is now available.
To upgrade, run npm install -g eas-cli.
Proceeding with outdated version.

✔ Select platform › iOS
✔ Linked to project @myorg/MyProject
    Error: [ios.infoPlist]: withIosInfoPlistBaseMod: ENOENT: no such file or directory, open '/Users/spiderclam/projects/mobile/projectname/GoogleService-Info.plist'
    Code: ENOENT

EAS:
$ eas build --profile production
★<EMAIL_ADDRESS>is now available.
To upgrade, run npm install -g eas-cli.
Proceeding with outdated version.

✔ Select platform › iOS
✔ Linked to project @myorg/MyProject
    Error: [ios.infoPlist]: withIosInfoPlistBaseMod: ENOENT: no such file or directory, open '/Users/spiderclam/projects/mobile/projectname/GoogleService-Info.plist'
    Code: ENOENT

When running a non-local build, the eas-build-pre-install script is only executed in the cloud. See the docs https://docs.expo.dev/build-reference/ios-builds/#remote-steps

The problem is, as it seems, that you don't have GoogleService-Info.plist locally.
You most probably have a reference to the file in app.json/app.config.js.
When starting a build, EAS CLI tries to read the app configuration (locally).

Probable solution:

Keep GoogleService-Info.plist in the project dir locally.
If you don't want to have this file uploaded to EAS Build, add it to .gitignore.

@dsokal

This is a breaking change.
eas-build-pre-install is being called locally (on 0.58 at least) so what you're saying isn't true.
The issue isn't just with local. With the risk of sounding rude (not meant rude), I don't feel like you've read everything I wrote.

Proof of point 2 (output when running using --local):
[PRE_INSTALL_HOOK] Script 'eas-build-pre-install' is present in package.json, running it...

But I want to repeat, this isn't just about --local. This is about any eas build

This is a breaking change.

What is a breaking change?

eas-build-pre-install is being called locally (on 0.58 at least) so what you're saying isn't true.

It's called locally only if you run local builds.

I don't feel like you've read everything I wrote.

Vice versa. Please read my comment again https://github.com/expo/eas-cli/issues/1317#issuecomment-1228204303

What is a breaking change?
It works on 0.58. It doesn't on 0.59. Something broke for me that's why I call it a breaking change. I don't know the change is that broke things for me though, that's what I want to find out 😄

It's called locally only if you run local builds.
Vice versa. Please read my comment again
My bad, I read your comment and you're right, I missed thje "non-" part. But that's still not the actual issue.

The problem is not eas-build-pre-install. The problem is that it's looking for a file (from a config plugin I think?) before even starting the build.
So like you said:

When running a non-local build, the eas-build-pre-install script is only executed in the cloud.

This sounds perfectly logical to me. But before it gets to running a build, something is already looking for a file. It shouldn't do that and before v0.59 it didn't.

I see, I can see we upgraded a bunch of packages in 0.59. You could be right. Can you share your app.json/app.config.js?

Also, I'm aware this could be a breaking change but what do you think about the solution I propose above?

Keep GoogleService-Info.plist in the project dir locally.
If you don't want to have this file uploaded to EAS Build, add it to .gitignore.

It's not the "fix" but it should unblock you.

@dsokal Sure thing.
I do want to note that I already tried removing the entire "plugins" key from the config with the same result. The reason I mention this is because I have a custom plugin in that list, so that's the first thing I tried.
{
  "expo": {
    "backgroundColor": "#ffffff",
    "owner": "ltodev",
    "name": "LTO Dichtbij",
    "slug": "LTO",
    "version": "1.0.0",
    "orientation": "portrait",
    "icon": "./assets/app-icons/icon.png",
    "splash": {
      "image": "./assets/app-icons/splash.png",
      "resizeMode": "contain",
      "backgroundColor": "#ffffff"
    },
    "updates": {
      "enabled": false,
      "fallbackToCacheTimeout": 0
    },
    "assetBundlePatterns": [
      "**/*"
    ],
    "ios": {
      "config": {
        "usesNonExemptEncryption": false
      },
      "associatedDomains": [
        "applinks:ltodichtbij.nl",
        "applinks:landing.test.ltodichtbij.nl"
      ],
      "bundleIdentifier": "nl.ltodichtbij.app",
      "googleServicesFile": "./GoogleService-Info.plist",
      "supportsTablet": false,
      "buildNumber": "57"
    },
    "android": {
      "intentFilters": [
        {
          "action": "VIEW",
          "data": [
            {
              "scheme": "https",
              "host": "*.ltodichtbij.nl",
              "pathPrefix": "/link"
            }
          ],
          "category": [
            "BROWSABLE",
            "DEFAULT"
          ]
        }
      ],
      "package": "nl.ltodichtbij.app",
      "googleServicesFile": "./google-services.json",
      "adaptiveIcon": {
        "foregroundImage": "./assets/app-icons/adaptive-icon.png",
        "backgroundColor": "#FFFFFF"
      },
      "versionCode": 23
    },
    "web": {
      "favicon": "./assets/favicon.png"
    },
    "plugins": [
      "@react-native-firebase/app",
      "@react-native-firebase/crashlytics",
      "./plugins/analytics/disable-ad-id.js"
    ],
    "extra": {
      "eas": {
        "projectId": "b6b896b3-0cd2-47cd-b64a-b4aaa8a33f4c"
      }
    }
  }
}

It's not the "fix" but it should unblock you.

I've thought about that, but I use eas.json to pick the right file based on the profile.  I could move the copy to a bunch of npm scripts instead as a workaround for now?

It's definitely worth trying.

I'll try that. Any way I can find the source of this as well?

What do you mean?

I mean, the source of what is trying to find that file. Which package has the withIosInfoPlistBaseMod that is doing this.
I want to help find the source of the issue so I can be a good open source citizen 😄

I have a similar problem.
I am running a local build on GitHub Actions.
I had to use eas-build-pre-install to run a script to get GoogleService-Info.plist etc from GCP Secret Manager (setup.sh).
In Android's ease build, setup.sh is executed in ease-build-pre-install after fetching JKS and other files.
However, in the iOS easy build, the GoogleService-Info.plist check is performed first, and I get Error: [ios.infoPlist]: withIosInfoPlistBaseMod: ENOENT: no such file or directory.
After several attempts, the versions of eas-cli and expo-cli do not seem to matter.
However, I have found workaround.
It seems to work if I put GoogleService-Info.plist in the directory before running npx eas build --platform ios--local --profile preview --non-interactive.
In my case.

Run setup.sh
Run npx eas build ...

When I did this workaround, eas-build-pre-install was also executed when building iOS.
For Android, it seems that eas-build-pre-install is executed without this workaround.
I have not investigated the code to the bug, but I hope this helps. 🙏

I have the same issue ... it only works if I include a non-empty GoogleService-Info.plist file in the repository / before the eas-build-pre-install

Facing this too on v3.3.2 for iOS only and my setup is very similar to OP's:
My pre-install hook creates the GoogleService-Info.plist file from an environment variable. I first came across this when setting up Expo Github Action since the CI environment doesn't have a GoogleService-Info.plist file present like my machine does. Might be worth noting that the file is in my .gitignore and .easignore.
However, when uploading the file as a secret file to EAS , the build starts (and successfully builds my app) just fine, so I'll be using this as my workaround for now.

This also fails for me on GitHub Actions.

This just started failing for me on Github Actions also

Same issue on Github Actions also

I don't know if it's the same anymore, I don't even remember this issue. So I'm muting notifications and leave it up to you + maintainers to deal with this :D Good luck all <3

I have the same issue ... it only works if I include a non-empty GoogleService-Info.plist file in the repository / before the eas-build-pre-install

To add further context, I see the same thing but working around the issue breaks it on the other platform...
To fix iOS, I add a step to run the eas_pre_install script myself before the build step.
    - name: 📂 Setup Google Services Files
      env:
        GOOGLE_SERVICES_JSON_BASE64: ${{ secrets.GOOGLE_SERVICES_JSON_BASE64 }}
        GOOGLE_SERVICES_PLIST_BASE64: ${{ secrets.GOOGLE_SERVICES_PLIST_BASE64 }}
      run: ./scripts/eas_pre_install

    - name: 👷 Build app
      env:
        GOOGLE_SERVICES_JSON: ${{ vars.GOOGLE_SERVICES_JSON }}
        GOOGLE_SERVICES_PLIST: ${{ vars.GOOGLE_SERVICES_PLIST }}
        EXPO_PUBLIC_FIREBASE_WEB_CLIENT_ID: ${{ secrets.EXPO_PUBLIC_FIREBASE_WEB_CLIENT_ID }}
      run: |
        eas build --local \
          --non-interactive \
          --output=./app-build \
          --platform=${{ github.event.inputs.platform }} \
          --profile=${{ github.event.inputs.profile }}

This works around the issue on iOS, but now breaks for Android...
I now get:
[PREBUILD] [Error: ENOENT: no such file or directory, open '/tmp/runner/eas-build-local-nodejs/7a56da89-d12d-4d48-b594-1d3730b0b21d/build/googleServices/google-services.json'] {
[PREBUILD] Error: [android.dangerous]: withAndroidDangerousBaseMod: Cannot copy google-services.json from /tmp/runner/eas-build-local-nodejs/7a56da89-d12d-4d48-b594-1d3730b0b21d/build/googleServices/google-services.json to /tmp/runner/eas-build-local-nodejs/7a56da89-d12d-4d48-b594-1d3730b0b21d/build/android/app/google-services.json. Please make sure the source and destination paths exist.
✖ Config sync failed

I think there's a discrepancy between where each respective build looks at for the local files.
On Android, the config looks at /tmp/runner/eas-build-local-nodejs/7a56da89-d12d-4d48-b594-1d3730b0b21d/build/googleServices whereas for iOS, it works with /home/runner/work/MyApp/MyApp/googleServices


Cannot connect to (local) server on Chrome, Win10

http://agar.io/?ip=<IP_ADDRESS>:443
I'm opening a local server on my computer. When I try to play it (with above url) on Chrome, it kept showing "Connecting". And there's no error message in console.

So I had to play it on Edge, which worked just fine.

Anyone  have any clues?

On chrome, it works completely fine.  Verify that nothing is blocking the socket connections, or try using incognito mode.

Make sure you're not logged into Google+ or Facebook

Duplicate.


MultilineScript and begin

How do I resolve this:
MultilineScript: Script with trailing operator `begin` should be merged with the script on the following line

In this context? You can't move the haml .col-md-12 to the begin line...
    - begin
      .col-md-12
        = render partial: "path/to/partial"
    - rescue ActionView::MissingTemplate

Hey @crearc,
You shouldn't use begin/rescue in your HAML templates—that kind of logic is meant to go in a helper.
Having said that, this is certainly a bug. I've fixed it in d4cfbbc824f8dcf222769064b28280b3a1789399. Thanks for the report!

That makes sense. Thanks for the bug fix.


Last Affected commits are not rendered as links

https://oss-vdb-test.wl.r.appspot.com/vulnerability/CVE-2022-40373 as an example.

Note to future self: https://github.com/google/osv.dev/blob/ac1e00f9928257d8db85557e92d7efe44f5a828f/gcp/appengine/frontend3/src/templates/vulnerability.html#L76

Hey @andrewpollock ,
I would love to work on this good-first-issue, could you just give some background so I can get started?

@aroramrinaal Thanks! I can provide some background:
The desired behavior is something similar to this: https://oss-vdb-test.wl.r.appspot.com/vulnerability/OSV-2023-415, where the commit links to the commit in the github repo. Currently this is only enabled on the "Introduced" and "Fixed" commits, not the Last Affected commits.
In-fact you can see it attempting to apply the link to the introduced commit in https://oss-vdb-test.wl.r.appspot.com/vulnerability/CVE-2022-40373, but the introduced being 0 the link is actually incorrect.
Heres the link to the relevant template file that renders to that page. osv.dev/gcp/appengine/frontend3/src/templates/vulnerability.html


⚠ Bofrost.at has degraded performance

In 766c297, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2227 ms

Resolved: Bofrost.at performance has improved in 6012e6a.


Remove Offsets

@obartra If you agree that we should do this, I'd like to add this to our production ready milestones.
If you think we should keep offset, then we can just close this ticket.

I think you are right, let's get rid off offset

closed via https://github.com/obartra/reflex/pull/151


REST entity endpoints: Available count is wrong when results filtered by lack of privileges

E,g.,
http://insturl/api/course?q=&length=30 will fall back to LIST_(ENTITY) privilege.  If user does not have this privilege then no results are returned, but the available count will be the total number of entities in the database.

I was aware of this and was tempted to change it from 'available' to something else as there's nothing we can really do to get an accurate count without storing it in lucene or something. I propose we simply advise in the documentation that the available is just a guess.


[HOLD for payment 2024-05-06] [HOLD for payment 2024-05-03] CRITICAL: [Actionable Whispers] [$500] Workspace list doesn't show up after creating a workspace for categorize it

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

Version Number: 1.4.62-13
Reproducible in staging?: Yes
Reproducible in production?: No, new feature
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Navigate to staging.new.expensify.com
Create a new account
Go to your space
Create a manual track expense
Click on Categorize it
Click on Learn more and create a new workspace
Go to your space and click on Categorize it again

Expected Result:
The newly created workspace should show up
Actual Result:
No workspace found shows up
Workaround:
Unknown
Platforms:

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

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/53149281-37f9-481f-90dd-760d98149cc0

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~012e058847a79bee4c
Upwork Job ID:<PHONE_NUMBER>081294336
Last Price Increase: 2024-04-19
Automatic offers:

c3024 | Reviewer | 0
tienifr | Contributor | 0

@puneetlath I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

@shubham1206agra I don't totally follow. Are you saying this is expected and not a deploy blocker?

I don't think this needs to block deploy, it is a new feature - I think this only happens with new workspaces and is only sometimes reproducible. We can probably make this external

I can also take this over since it's related to track - but feel free to re-assign if you're interested

RCA suggested by @tienifr here makes sense. The suggested solution looks good and works well.
🎀 👀 🎀 C+ Reviewed

Great!  @c3024 can you give an ETA on this?

@tienifr
Could you raise the PR today? This is a critical issue. ^^

@tienifr can you give an ETA for opening the PR?

@thienlnam
We have here maxRecentReportsToShow as 5
https://github.com/Expensify/App/blob/690609052072c89c52f24b21d95ea02b2819a769/src/libs/OptionsListUtils.ts#L2011
I can change this if needed.

Yup please do, there should be no limit

However, why didn't the report show up during search then?

That is what I am trying to find as well.

It looks like this here
https://github.com/Expensify/App/blob/690609052072c89c52f24b21d95ea02b2819a769/src/pages/workspace/WorkspacesListPage.tsx#L379
creates a draft but does not write to backend.
Search after Logging out and logging in works fine even with maxRecentReportsToShow as 5.

@c3024 bump on the checklist.
@tienifr has been paid.

Issue for regression test: https://github.com/Expensify/Expensify/issues/393986
Everyone has been paid. Thanks!

Ah ok. So there were two PRs for this issue? @thienlnam can you confirm who all needs to be paid out here?

This PR fixes what was described in the issue description
https://github.com/Expensify/App/pull/40730
I thought this wasn't solved when I was testing it here, but turns out there was a different problem with the limit of workspaces shown in the selector which was fixed https://github.com/Expensify/App/pull/41009

@c3024 worked on both PRs, so we could give them another payment for this PR, and have a payment to @ishpaul777 for being the C+ on that PR as well

Ok so it's:

$500 for @tienifr for https://github.com/Expensify/App/pull/40730 (already paid)
$500 for @c3024 for https://github.com/Expensify/App/pull/40730 (already paid)
$500 for @c3024 for https://github.com/Expensify/App/pull/41009
$500 for @ishpaul777 for https://github.com/Expensify/App/pull/41009

Is that right?

Yup that's correct

Ok @tienifr and @c3024 are now fully paid.
@ishpaul777 sent you an offer: https://www.upwork.com/nx/wm/offer/102304232
Ping me here when you've accepted please.


Update requirements.txt to use PyAML

pip isnt finding yaml. PyAML should do the same.

Thanks.
This looks like a typo, indeed.
I wonder why our tests / checks in this repo didn't catch this.

Merged, thanks.


Can the source code be classified and placed in different folders according to the functionality of the components. Sort like the menu on the Demo site.

Can the source code be classified and placed in different folders according to the functionality of the components. Sort like the menu on the Demo site.

Hey @TimChen44,
We don’t plan such changes.


Csb 396

I think we are using camel-community.version for the camel-test-infra-* artifacts instead of camel-version

thanks @johnpoth ! I'm afraid I'll never get used to that version replace


Petite: inject beans with interface name if only one implementation is registered

If we have Foo interface and the only one FooImpl, allow to annotate the impl and since it is the only implementation; allow to inject it as "foo"

Done. Added alternative bean names for each bean.


Create: 1472-Design-Browser-History.py

File(s) Modified: 1472-Design-Browser-History.py
Language(s) Used: python
Submission URL: https://leetcode.com/submissions/detail/816668592/

Thanks for contributing!


Python39 11x

As requested in https://github.com/pallets/markupsafe/pull/159#issuecomment-733452290
I am putting a separate PR (to be sure nothing forgotten/broken) with Python 3.9 support against 1.1.x branch.
Description:

Adds Python 3.9 to GitHub actions and tox
Replaces 3.8 with 3.9 for windows, mac, docs and linting builds

Wrong way. :)


Fixed the Wijkanders import.

Wijkanders replaced their  with  and suddenly we didn't parse
anything. This should be fixed in this commit which changes the
algorithm to process content until next date or a last phrase is
encountered. I guess this will work until Wijkanders changes something
again. We'll see, we'll see...

This commit should reference #75.

If you squashed/fixupped your "fix hlint" - commit with the main commit I'd be even happier.

It is done. Thank you for the thorough review @Jassob


Adding hint about full screen mode

Summary of the Pull Request
Just adding an addition tip to the usage document.

References
None

PR Checklist

[ ] Closes #xxx
[x] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments
Just updating documentation. No tests or open issues involved.

Validation Steps Performed
Manual steps. Open the Windows Terminal. Press the key combination of Alt+Enter to go to full screen, and then again to exit.

(Er, I should have just closed this. derp)


Add docs for HostEmulator

so clearly after #315, the e2e on http_body became flaky... could you take a look? @anuraaga

maybe simply increase the timeoue on require.Eventually would be fine

Thanks @mathetake - on my fork I could reproduce the failure after one retry, and so far after one retry with raised timeout it is passing. Let's see how it goes

Actually - curious, should we only have the checkMessage in eventually and move the rest out? I think the other parts are supposed to be deterministic

LOL failed...

No go on the test still btw, will keep on trying

https://github.com/tetratelabs/proxy-wasm-go-sdk/compare/testaaaa?expand=1 will give us more context

ok somewhat deterministially Test_http_body/request/append failing

I couldn't reproduce on my local ubuntu...

I hope this fixes https://github.com/tetratelabs/proxy-wasm-go-sdk/pull/320 🤞


Add the option for a custom window icon

This is a follow up from bryphe/reason-glfw#97.
I implemented it in revery by adding an icon: option(string) field to the windowCreateOptions in Window.re. The default is None. An example of this in practice can be found in the examples project, i.e. esy x Examples.

Thanks @zbaylin for your work on this - awesome!
so cool to have the icon now for the example app:


Adding a new line at the end of a file is ignored

Describe the bug
When adding a new line to a file it is ignored
To Reproduce
Steps to reproduce the behavior:

Using the following test (kotlin)

val beforeContent = "rootProject.name = \"sample-repo\""
val afterContent = "rootProject.name = \"sample-repo\"\n"
val patch = "diff --git a/settings.gradle b/settings.gradle\n" +
        "index ef3b8e2..ab30124 100644\n" +
        "--- a/settings.gradle\n" +
        "+++ b/settings.gradle\n" +
        "@@ -1 +1 @@\n" +
        "-rootProject.name = \"sample-repo\"\n" +
        "\\ No newline at end of file\n" +
        "+rootProject.name = \"sample-repo\"\n"

val unifiedDiff = UnifiedDiffReader.parseUnifiedDiff(patch.byteInputStream())
val unifiedAfterContent = unifiedDiff.files[0].patch.applyTo(beforeContent.split("\n")).joinToString("\n")
assertEquals(afterContent, unifiedAfterContent)

Expected behavior
The test should have equal content after applying the patch

You are right. I investigated so much time implementing this \ No newline at ... and forgot to use it for the patch itself. :)


Added automatic app version detection

Added support to fetch the latest published app version from the Apple App Store

This is a nice one, but I prefer to cache the file version from apple like we are doing with the token. Lets say we send in a file location for appversion instead of auto, if we got a file location we lookup the version and store it, the lookup is done once a day (looking at file date). If the file ins't provided (may use a default?) we use the constant.
May you do that instead? I don't like pulling data from third party more than necessary.

The data is only pulled on initialize, how often do you expect the library to be reinitialized?
If the version should be cached to disk, the token cache needs to be updated as well to combine both values in one file.
Where I use this, initialization is called less than once per week (Home Assistant Component).

In my case I use the library in a cron job that is executed every 30 minutes.
So my script would ask Apple API 384 times per week.
The best would be to use Apple service (and cache the result) only if the Panasonic server responds with the error:
pcomfortcloud.session.ResponseError: Invalid response, status code: 401 - Data: {"message":"New version app has been published","code":4106}

@LudovicRousseau waiting for the error is not a good option, it looks like they have a 3-week grace period after releasing a new version before blocking the old version.

In my case I use the library in a cron job that is executed every 30 minutes.
So my script would ask Apple API 384 times per week.
The best would be to use Apple service (and cache the result) only if the Panasonic server responds with the error:
pcomfortcloud.session.ResponseError: Invalid response, status code: 401 - Data: {"message":"New version app has been published","code":4106}

Thumbs up, it's better doing the version check in the login routine instead of in the init,and better yet, only check it when login returns status 40X

The data is only pulled on initialize, how often do you expect the library to be reinitialized?
If the version should be cached to disk, the token cache needs to be updated as well to combine both values in one file.
Where I use this, initialization is called less than once per week (Home Assistant Component).

Hmm, I don't know how it's used, but when using it command line it would do this for every command issued

The data is only pulled on initialize, how often do you expect the library to be reinitialized?
If the version should be cached to disk, the token cache needs to be updated as well to combine both values in one file.
Where I use this, initialization is called less than once per week (Home Assistant Component).

Hmm, I don't know how it's used, but when using it command line it would do this for every command issued

If it's used in a py file called in say a cron job it will be initialized for each execution, so for that scenario I can see the point of caching it.
Though I doubt it would make much of a difference, a cold request takes ~250ms and once they cached it, it's 5ms.

I've refactored the token storage and consolidated it with version information into a settings class that stores the information as json.
Version information has a TTL of 5 days.
TODO maybe add exception handling to the panasonic requests and call _updateAppVersion() if we get the Invalid response, status code: 401 - Data: {"message":"New version app has been published","code":4106} response.

Just a thought - how about fetching the new version when the we actually get rejected by PCC (eg on http error)? That'd result in a lot less requests, and only on a need to basis


Genomics UD RNA-Seq: txt files do not need link to JBrowse

im not aware of others among these datasets bw and rnaseq)


为什么开不出LXC小鸡？

 Done
Building dependency tree... Done
Reading state information... Done
All packages are up to date.
W: Failed to fetch http://deb.debian.org/debian/dists/bullseye/InRelease  Temporary failure resolving 'deb.debian.org'
W: Failed to fetch http://deb.debian.org/debian/dists/bullseye-updates/InRelease  Temporary failure resolving 'deb.debian.org'
W: Failed to fetch http://deb.debian.org/debian-security/dists/bullseye-security/InRelease  Temporary failure resolving 'deb.debian.org'
W: Some index files failed to download. They have been ignored, or old ones used instead.
sudo: unable to resolve host distrobuilder-aa0a95d3-28a7-4de5-8c82-5eb0df946d48: Temporary failure in name resolution
sudo: unable to resolve host distrobuilder-aa0a95d3-28a7-4de5-8c82-5eb0df946d48: Temporary failure in name resolution
Err:1 http://deb.debian.org/debian bullseye InRelease
Temporary failure resolving 'deb.debian.org'
Err:2 http://deb.debian.org/debian bullseye-updates InRelease
Temporary failure resolving 'deb.debian.org'
Err:3 http://deb.debian.org/debian-security bullseye-security InRelease
Temporary failure resolving 'deb.debian.org'
Reading package lists...  They have been ignored, or old ones used instead.
sudo: unable to resolve host distrobuilder-aa0a95d3-28a7-4de5-8c82-5eb0df946d48: Temporary failure in name resolution
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
The following additional packages will be installed:
ca-certificates libbrotli1 libcurl4 libldap-2.4-2 libldap-common libnghttp2-14 libpsl5 librtmp1 libsasl2-2 libsasl2-modules libsasl2-modules-db libssh2-1 openssl publicsuffix
Suggested packages:
libsasl2-modules-gssapi-mit | libsasl2-modules-gssapi-heimdal libsasl2-modules-ldap libsasl2-modules-otp libsasl2-modules-sql
The following NEW packages will be installed:
ca-certificates curl dos2unix libbrotli1 libcurl4 libldap-2.4-2 libldap-common libnghttp2-14 libpsl5 librtmp1 libsasl2-2 libsasl2-modules libsasl2-modules-db libssh2-1
openssl publicsuffix
0 upgraded, 16 newly installed, 0 to remove and 0 not upgraded.
Need to get 3385 kB of archives.
After this operation, 7608 kB of additional disk space will be used.

Error: Command not found
sudo: unable to resolve host distrobuilder-aa0a95d3-28a7-4de5-8c82-5eb0df946d48: Temporary failure in name resolution
sudo: unable to execute ./ssh.sh: No such file or directory
sudo: unable to resolve host distrobuilder-aa0a95d3-28a7-4de5-8c82-5eb0df946d48: Temporary failure in name resolution
Package: zmap nmap masscan medusa
Pin: release *
Pin-Priority: -1
Err:1 http://deb.debian.org/debian bullseye InRelease
Temporary failure resolving 'deb.debian.org'
Err:2 http://deb.debian.org/debian bullseye-updates InRelease
Temporary failure resolving 'deb.debian.org'
Err:3 http://deb.debian.org/debian-security bullseye-security InRelease
Temporary failure resolving 'deb.debian.org'
Reading package lists...  They have been ignored, or old ones used instead.
Error: Command not found
Device ssh-port added to niubi
niubi 60066 4275f6b72

和
https://github.com/spiritLHLS/lxc/issues/17
一个问题，稍后修复，别急

尝试修复了一下，你可以再试试


Possibility to Load Configs

It would be cool if the user could load a config file or paste the contents of a config file into the editor in the web gui. That would make doing changes to an existing configuration much easier.

It's hard to prepare the original parser. Do you have a good idea?


page inspection: support compositions

To make the page inspection feature more useful, it would be a boon if compositions had better support:

by default, composition instances should be collapsed into a representative vertex
it should be possible to drill-down into a selected composition instance using double-click (and maybe the enter key)
when viewing composition contents, there should be an "anchor vertex" representing the connection to the external interface of the composition
the current "flat" view should still be supported as an alternative.

Implemented on master (v2.0.0).
Upgrade Information
The page inspector of the developer tools now visualizes compositions: by default, each composition instance is represented as its own graph node. Drilldown is supported through double-click on a composition node. Alternatively, compositions can be flattened recursively to inspect how their contents are interconnected.


ci: add GH action to test compilation on Zephyr

This PR is to add a GH action to test compilation of the product-mini "simple" example on Zephyr against the STM32 board.

Close the PR since it is in draft and hasn't been updated for a long time.


fixed issue #195 Crash on Android 5.0!

I will fix it ASAP.

I wait this changes in gradle dependency, thanks.

@lapism I've just checked another possible way how to fix this. Is overriding <dimen name="design_appbar_elevation">0dp</dimen>  in the v21\dimens. But this approach shows warning

Overriding @dimen/design_appbar_elevation which is marked as private in com.android.support:design.


small optimization for "130. surrounded regions" in neetcode 150

no need to make a 'borders' vector when you can run the code along the border like this

File(s) Modified: surrounded_regions.cpp
Language(s) Used: C++
Submission URL: _https://leetcode.com/submissions/detail/763100099/_

Thanks, @aadinoir!


[Feat] - Java version

Hi, is there a java version of the "Calling crop directly" code pls?

Yes, please check the FAQ
Or the link for a Java explanation on the README
Is the first links


Add check when adding BrewPiDevices for uniqueness of device_name

Device names currently are required to be unique by the database, but this uniqueness isn't checked when the form is cleaned. This needs to be added, so the form can be properly invalidated (and a valid error message presented to the user)
IntegrityError: UNIQUE constraint failed: app_brewpidevice.device_name
  File "django/db/backends/utils.py", line 64, in execute
    return self.cursor.execute(sql, params)
  File "django/db/backends/sqlite3/base.py", line 328, in execute
    return Database.Cursor.execute(self, query, params)

IntegrityError: UNIQUE constraint failed: app_brewpidevice.device_name
(15 additional frame(s) were not displayed)
...
  File "django/db/backends/utils.py", line 64, in execute
    return self.cursor.execute(sql, params)
  File "django/db/utils.py", line 94, in __exit__
    six.reraise(dj_exc_type, dj_exc_value, traceback)
  File "django/utils/six.py", line 685, in reraise
    raise value.with_traceback(tb)
  File "django/db/backends/utils.py", line 64, in execute
    return self.cursor.execute(sql, params)
  File "django/db/backends/sqlite3/base.py", line 328, in execute
    return Database.Cursor.execute(self, query, params)

IntegrityError: UNIQUE constraint failed: app_brewpidevice.device_name

This issue seams to be fixed (at least for the advanced add)

This issue is partially fixed now, but a complete fix is coming with the fix for #355 . Closing this one now - the other issue will get closed when I merge the fix in.


ClusterTest.testListenersWhenClusterDown fails at linux when using 32-bit Release type library

The fix is related to issue https://github.com/hazelcast/hazelcast-cpp-client/issues/258 It may not fix the issue completely but it will behave more closely with the Java client on invocation logic during cluster disconnects.
Changed the responsibility of owner connection ensure from connection manager to invocation service as it is done for Java client. Also, do not wait for owner connection if it does not exist but just throw an exception as performed in java and rely on invocation retry mechanism.

Test FAILed.

Test PASSed.


TAHub v1.4 not Updating Edited/Deleted Students in Consultations

Currently, Consultations has Students from the Student List.
However, it is possible to modify the Student List to edit or delete the Student.
Yet, the changes to Students are not updated and reflected in the Consultations accordingly.
For Example, a Deleted Student will still remain in a Consultation if the Student was previously added to the Consultation.
The Same applies to an Edited Student.
As the Commands continue to change. We recommend doing a final check on ModelManager.java before the end of v1.5.

This is part of a bigger problem that could happen to Lessons as well. There is an obvious brute force solution where we iterate through all consultations and lessons every time a student is deleted. Hopefully people don't start having 10000 students and 100 classes or the program is going to freeze.


Fix PPP formatting

Codecov Report
Base: 54.63% // Head: 54.63% // No change to project coverage :thumbsup:

Coverage data is based on head (7188c59) compared to base (5cff4f6).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #255   +/-   ##
=========================================
  Coverage     54.63%   54.63%
  Complexity      610      610
=========================================
  Files           130      130
  Lines          2482     2482
  Branches        331      331
=========================================
  Hits           1356     1356
  Misses         1009     1009
  Partials        117      117

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


3rd generation Blink Outdoor live view support?

I am unable to get live view working with my 3rd gen outdoor cameras.  I can view clips and update the preview image, but live video gives me an infinite loading circle.  If 3rd gen blinks are supported, what am I doing wrong?  Thanks for any help.

This is what my log is showing if that helps at all.

I'm sorry new generations use a different syncmodule and communication for liveview has changed. All my devices are first generation so I cannot give support to new ones.


Invalid Checksum Warning Not Displayed

First off, love your tool!
Ok so I haven't completely dove into the code yet, but I have noticed that some invalid checksums are allowed with the tool, without a warning/error appearing mentioning the checksum is invalid.
Let's assume a 12 word passphrase. If we divide the 2048 wordlist into groups of 16 ... exactly 1 word out of a 16 word "block" will be shown as a  mnemonic/checksum for the 11 words selected.  Why is this?  How is the checksum being validated?
See this example:
Entropy (128 bits): 11010011 01100100 00000010 01011110 01010011 11101100 01010011 01101110 01101010 01111000 11010010 11011000 10111010 00100011 11101100 11110010
SHA-256 Hash of entropy = 14 c5 8b c9 05 11 5e 08 27 49 61 1e 48 d6 04 c0 2a 70 8c 39 ad 6c dc 0c 91 2f 70 62 c3 24 71 23
First 4 bits of SHA-256 Hash = 1 (hex) Binary = 0001
Generated Recovery Phrase: square cactus nurse pond share rescue prepare bottom suffer speed will tomorrow
another valid phrase (with same entropy but different checksum): square cactus nurse pond share rescue prepare bottom suffer speed will account
Account = 13 (on the BIPS39 wordlist) Subtract 1 since word index starts at 0 = 12 12 to binary = 1100 Hex = C
C (hex) != 1 (hex)
another valid phrase (with same entropy but different checksum): square cactus nurse pond share rescue prepare bottom suffer speed will acoustic
Acoustic = 17 (on the BIPS39 wordlist) Subtract 1 since word index starts at 0 = 16 16 to Binary: 10000 Hex = 10
10 (hex) != 1 (hex)
Thanks Ian!

You seem to be confused ... a word out of a list of 2048 words means 11 bits (0-2047), so every words represents 11 bits. If you use 12 words you therefor get 132 bits to use, the generator generates a 128bit number, then calculates a 4 bit checksum which determines PART of the 11 bits of the 12th words (last 4 bits).
Same is true for eg. 24 words (24 times 11 = 264, we only use 256bit number so 8 bits of the last word are checksum bits)

Thanks for this. I'm going to close it in favour of the last task in #132.
See https://github.com/iancoleman/bip39/issues/132#issuecomment-348085868 for more details.

I would suggest the following tiny additions be made that could help users
Reveal the resulting checksum in binary format

Thanks @stiVibk for the clarification. Many people seem to be confused about the checksum so having some further clarity in this tool will help all round.
@nekrozon - when the word is changed to acoustic in the last example, the first 7 bits of acoustic are part of the entropy, the last 4 bits are the checksum. The entropy is different due to these last 7 bits, so the hash (and thus checksum) will also be different.

Man I’m kicking myself for that mistake!  Thanks @stiVibk and @iancoleman
Is there a mathematical reason, assuming a 12 word phrase, that 1 out of every 16 words is a valid checksum (if you divide the wordlist into blocks of 16, 1 of them will be a valid checksum)?


Add delete all buttons in mitmweb interface for flow modification

This is simple enhancement proposal I'd be thankful if implemented. I use mitmproxy in debugging my application, and it would be useful being able to clear all historic flow data in one button.
Right now I'm even unable to select multiple flow items and batch delete them.
System information
Mitmproxy version: 2.0.2 (release version)
Python version: 3.6.2
Platform: Linux-4.12.6-1-ARCH-x86_64-with-arch
SSL version: OpenSSL 1.1.0f  25 May 2017
Linux distro: arch

Thanks for the feedback. You can currently click mitmproxy -> New. Is that what you are looking for? If so, we should probably look at making the interface more clear.

Yes, I can, thank you. That's exactly I'm looking for; I just kinda assumed there would be some big red button clear all or something like it.

Thanks for this issue. I spammed the "d" key for waaay too long :)


⚠ Contemporanea Group has degraded performance

In a963aea, Contemporanea Group ($SITE_CNTMG) experienced degraded performance:

HTTP code: 200
Response time: 13857 ms

Resolved: Contemporanea Group performance has improved in 5d29521.


Adds compatibility matrix structure

Hi team!
Adds compatibility matrix with information about current status. This is the first compatibility matrix we deploy so, review it carefully and suggestion are always welcomed!
Current status are recovered from drone pipelines!
Thanks!

Shouldn't fury-kubernetes-monitoring be compatible also with v1.14.x?
Is there anything failing in CI for v1.14.x?

Shouldn't fury-kubernetes-monitoring be compatible also with v1.14.x?
Is there anything failing in CI for v1.14.x?
Hi @lnovara

We did not test it in the pipeline against 1.14 version. This is something that i want to change in upcoming commits/PRs.
So, the 1.14 column does not have information because we did not have automated tests. We can consider mark it as stable if someone has installed it in a 1.14 cluster. But with the data i have, it's considered "untested" in 1.14 and tested in 1.15 ;)

@angelbarrera92 understood, we do have v1.14.x cluster running fury-kubernetes-monitoring but I agree with you that the compatibility matrix needs to be CI-driven.

Just out of curiosity, by "Current status are recovered from drone pipelines!" do you mean automatically?

Just out of curiosity, by "Current status are recovered from drone pipelines!" do you mean automatically?

Not now...
By the moment, i was digging in drone pipelines looking for the correct pipeline and setting the result in the markdown table.
So, first step, make it working in multiple k8s versions. Then, automate as much as possible to get the markdown table written as pipeline output.


[feature] Export Connector which is used to connect to the wallet from @walletconnect library

useWallet hook give as access to some of the functionalities from @walletconnet, for example connect and disconnect functions.
We would like to have whole Connector class exported from @walletconnect library.
class Connector implements IConnector
We need that for example to use
signMessage(params: any[]): Promise<any>;

@Mskalba Sorry, the sign feature is not yet implemented for Terra Station Mobile and @terra-money/wallet-provider.
I am currently developing another service, so it will take time to implement the sign function. I will let you know again when the function is implemented.

@iamssen thank you for quick update, I am working with @Mskalba we have also tried some workarounds but without results as you might see here: https://github.com/terra-money/terra.js/issues/95, could you maybe provide some ETA for this feature?

@tkowalczyk I'm sorry. I have a lot of things to develop, so it will be difficult to start development for this quickly.
I will discuss the development schedule for this part with the team.

fully understand, would be great if you will provide later some ETA, good luck!

Hi!
As you have a lot of things in your pipeline, @iamssen, I would like to ask if you could utilize any help with this feature.
If you could point out which libraries require changes so I could focus on extending them to provide such functionality.
As far as I understand signMessage is not exported by useWallet, because it lacks implementation in Wallets (Terra Station Mobile only?)
Please let me know if you could give me some hints. What is missing in current version and where to look for it so I could have easier start with working on that ;)

Started adding sign() feature. You can refer to this PR(https://github.com/terra-money/wallet-provider/pull/7).
We need to add features to the Mobile App, so it will take some time for the full release.
For now, you can test it based on Chrome Extension.

We discussed and decided not to implement the sign() function in wallet-provider.
In the case of Extension, the sign() function is not a big threat because it is only within the user device, but sending a signed key through the walletconnect bridge server is considered a security concern.

Oh, this is definitively a bad news.
Could I ask what dangers are introduced when we send signed transaction over walletconnect bridge? And how far it is different from creating a signed transaction via walletconnect and broadcasting it to the network.
I might be asking basic questions but I'm new to terra ecosystem and would love to gain as much knowledge as possible ;)


#988 Separate decodeCborHexToBytes and remove textEnvelopeBytes in favor of plutusScriptVxFromEnvelope

Not sure if that is what was meant in the issue.
Pre-review checklist

[x] All code has been formatted using our config (make format)
[x] Any new API features or modification of existing behavior is covered as defined in our test plan
[x] The changelog has been updated under the ## Unreleased header, using the appropriate sub-headings (### Added, ### Removed, ### Fixed), and the links to the appropriate issues/PRs have been included

@klntsky
Not sure I understand task correctly.
I used signature String -> Either TextEnvelopeDecodeError PlutusScript cuz using Aeson/Maybe leads to long boilerplate code like this:
alwaysFailsScript :: Contract () Validator
alwaysFailsScript = do
  let aesonOrLeft = parseJsonStringToAeson alwaysFails
  let aesonError = error "Cannot decode JSON for alwaysFailScript"
  let fromEnvelopeError = error "Cannot decode envelope for alwaysFailScript"
  aeson <- liftEither $ lmap (const aesonError) aesonOrLeft
  liftMaybe fromEnvelopeError $ Validator <$> plutusScriptV1FromEnvelope aeson

Also, I did this: https://github.com/Plutonomicon/cardano-transaction-lib/pull/1145/files#diff-bca190054ba134e1b1cd4ac4fda3466cddc5091bbc6798d2a08c8b20d70d6f66R136
Not sure what this it the best way to access PlutusScript.bytes, but I found no other.

@klntsky

All FFI code should be hot in the tests. Please add a test - just a fixture would be fine.

Is not it already covered by running examples in Plutip tests?

FromBytes for PlutusScript somehow breaks code, so I converted bytes without it like it was in the code before.


[window-app] Set valid Type for Window Covering cluster

Problem
If Feature Map of Window Covering cluster has both LF and TL bits set, Type should be either 6 or 8.
Change overview
Initialize the attribute to 8 in window-app and all-clusters-app so that they can pass certification tests.
Fixes #19163
Testing
Tested manually using nRF Connect all-clusters-app

@Damian-Nordic needs:
./scripts/tools/zap/generate.py src/controller/data_model/controller-clusters.zap -t examples/darwin-framework-tool/templates/tests/templates.json -o zzz_generated/darwin-framework-tool/zap-generated


Super Galaxy Rumble skin

the Super Galaxy Rumble skin is bugged, the fire (q skill) is going sideways, please fix the bug

#1


Add documentation for adding a provider

We have documentation on User and Policy Management, but we don't have documentation on managing Providers.  Currently, we have:

Enroll a new provider
Difference between OAuth/PAT and GitHub Actions provider
How to create a new project vs add a provider to an existing project

We have https://minder-docs.stacklok.dev/understand/providers
That doesn't cover:

How to create a new project vs add a provider to an existing project, but this doc does: https://minder-docs.stacklok.dev/how-to/create_project


ec2-room-validation

places an instance in a subnet
calculates the available ips

Codecov Report

Merging #80 into master will increase coverage by 0.03%.
The diff coverage is 20%.

@@            Coverage Diff             @@
##           master      #80      +/-   ##
==========================================
+ Coverage   20.14%   20.17%   +0.03%
==========================================
  Files          49       49
  Lines        2264     2280      +16
  Branches      239      246       +7
==========================================
+ Hits          456      460       +4
- Misses       1786     1798      +12
  Partials       22       22

Impacted Files
Coverage Δ

lib/python/treadmill_aws/cli/admin/aws/instance.py
0% <0%> (ø)
:arrow_up:

lib/python/treadmill_aws/aws.py
41.17% <100%> (+18.09%)
:arrow_up:

Continue to review full report at Codecov.

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

example:
treadmill --dns-domain ipa.poc.cloud.ms.com admin aws instance create --image ms-generic --image-account<PHONE_NUMBER>31 --secgroup sg-49bf3839 --key SymphonyPOC --size t2.large --subnet subnet-4f3b3b07 --subnet VPC101SNCSR2D --subnet VPC101SNCSR2D --count 15
Explanation:
The cli takes care of subnet duplication.
The availability of the 2 subnets in total is equal to 14 hosts.
In cli I specified 15 hosts, so the cli will spin up the 14 hosts and will raise an exception that in above mentioned subnets has left no space for 1 instance.


Ran the reasoning for hours without stopping

when i run infer_top_match.py , Running for a few hours has not ended, how to control the length of generation?

请问怎么结束生成？运行几个小时依然没有结束生成，没有产生结果

遇到了同样的问题，请问你解决了吗


Zip File TODO

TODO: zip allows for relative path filenames, so we need to sanitize those, otherwise this might allow overwrite of arbitrary files
I read this as we just need to check if there are any other zip files that already exist with the same name and have some kind of response if there is already a file. Is that correct?

Added validate_zip function to execute this function. I roughly followed the pseudo code. If you could double check and make sure it is working how you expected it to that would be great.

@pauchy1: It looks good to me. At some point we should come up with a set of "naughty" zipflle tests to make sure testing catches any regressions.


Cannot import data containing < or >

I have a couple of fields that contain the < > symbols. As an example;
"origin mark":"",
It seems these are not imported and fail. Is there anyway to get around this? Unfortunately they cannot be removed from the original data.

@terryupton Can you provide a bit more information? What sorts of fields are you trying to import this into? Can you provide a short snippet of example feed (XML/JSON)?

Hi @engram-design - I was trying to import this into a plain text field within Craft.
Ultimately, I have got around this by changing this to a > and <
Is there a way I can share the original data sample without posting it publicly?

@terryupton Sure, best to catch me on Slack sometime - handle @crawf


Model fails to sample

import nutpie
import pymc as pm

with pm.Model() as m:
    proposed_pay = pm.MutableData("proposed_pay", np.array([50_000, 200_000], dtype="float64"))
    accepted = pm.MutableData("accepted", np.array([0, 1], dtype="int64"))

    mean = pm.Normal("mu", mu=100_000, sigma=25_000)
    std = pm.Gamma("std", mu=25_000, sigma=5_000)

    p_accept = 1 - pm.logcdf(pm.Normal.dist(mean, std), proposed_pay).exp()
    p_accept = pm.Deterministic("p_accept", p_accept)

    llike = pm.Bernoulli("llike", p=p_accept, observed=accepted)

cm = nutpie.compile_pymc_model(m)
nutpie.sample(cm)

I get these weird warnings from pytensor/numba
/tmp/tmps3yx0tdg:1: NumbaWarning: Cannot cache compiled function "numba_funcified_fgraph" as it uses dynamic globals (such as ctypes pointers and large global arrays)
  def numba_funcified_fgraph(scalar_variable, scalar_variable_1, scalar_variable_7, scalar_variable_11, scalar_variable_3, scalar_variable_5, scalar_variable_15, scalar_variable_19):
site-packages/nutpie/compile_pymc.py:362: NumbaWarning: Cannot cache compiled function "numba_funcified_fgraph" as it uses dynamic globals (such as ctypes pointers and large global arrays)
  return inner(x, *_shared_tuple)

And then a ValueError
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Input In [43], in <cell line: 17>()
     14     llike = pm.Bernoulli("llike", p=p_accept, observed=accepted)
     16 cm = nutpie.compile_pymc_model(m)
---> 17 nutpie.sample(cm)

site-packages/nutpie/sample.py:101, in sample(compiled_model, draws, tune, chains, seed, num_try_init, save_warmup, store_divergences, progress_bar, init_mean, store_unconstrained, **kwargs)
     98 if init_mean is None:
     99     init_mean = np.zeros(compiled_model.n_dim)
--> 101 sampler = lib.PyParallelSampler(
    102     compiled_model.logp_func_maker,
    103     init_mean,
    104     settings,
    105     n_chains=chains,
    106     n_draws=draws,
    107     seed=seed,
    108     n_try_init=num_try_init,
    109 )
    111 expand_draw = compiled_model.expand_draw_fn
    113 def do_sample():

ValueError: Logp function returned error Initialization of first point failed

I think this is a consequence of the initial point choice in nutpie. It currently tries to initialize all parameters in the transformed space in (-1, 1). But in that range we just can't find any valid points. The problem goes away if we change the scaling of the variables:
with pm.Model() as m:
    proposed_pay = pm.MutableData("proposed_pay", np.array([5.0, 20.0], dtype="float64"))
    accepted = pm.MutableData("accepted", np.array([0, 1], dtype="int64"))

    mean = pm.Normal("mu", mu=10.0, sigma=2.5)
    std = pm.Gamma("std", mu=2.5, sigma=.5)

    p_accept = 1 - pm.logcdf(pm.Normal.dist(mean, std), proposed_pay).exp()
    p_accept = pm.Deterministic("p_accept", p_accept)

    llike = pm.Bernoulli("llike", p=p_accept, observed=accepted)

cm = nutpie.compile_pymc_model(m)
tr = nutpie.sample(cm)

It would be better to initialize at draws from the prior though...

So the difference with PyMC is there there we do transformed(moment) +-1 and here we do 0 +- 1?
Could we use the PyMC initial point logic if it's more stable?

I think so. There already is an option to set a mean, we'd only have to plug in the moment there.

A workaround at the PyMC level can be as short as 4 lines, but the latest nutpie release was before the init_mean kwarg was added.
To fix this at the nutpie level, the init_means could be added as another attribute on the compiled_model object.
Today I'll try to workaround with local hotfixes, but I should be able to make a PR for this by the end of the week.

This is the workaround for the PyMC level, however it didn't help for my model 🤔
compiled_model = nutpie.compile_pymc_model(model)
# Pass transformed, concatenated initial values until nutpie does it itself
initial_point = model.initial_point()
initial_means = np.concatenate([initial_point[model.rvs_to_values[var].name].flatten() for var in model.free_RVs])
idata = nutpie.sample(
    compiled_model,
    draws=draws,
    tune=tune,
    chains=chains,
    target_accept=target_accept,
    init_mean=initial_means,
    seed=_get_seeds_per_chain(random_seed, 1)[0],
    progress_bar=progressbar,
    **nuts_sampler_kwargs,
)

The point must be on the unconstrained space, but I think this will instead use values on the constrained space?

The point must be on the unconstrained space, but I think this will instead use values on the constrained space?

The pmodel.initial_point() dictionary has only the unconstrained ones, e.g. noise_log__

Can confirm that this fix does not work, in general. Also tried with random jitter around initial point. Perhaps this is not the issue?

Here's a rather minimal example to reproduce the issue:
import io

import numpy as np
import pandas as pd
import pymc as pm
import pytensor.tensor as pt

def build_model(df_data, *, hsgp: bool):
    with pm.Model(
        coords={
            "records": np.arange(len(df_data)),
        }
    ) as pmodel:
        # Store data
        X = pm.ConstantData("X", df_data.x.to_numpy(), dims="records")
        Y = pm.ConstantData("Y", df_data.y.to_numpy(), dims="records")
        Y_std = pm.ConstantData("Y_std", pt.std(Y).eval())
        Y_mean = pm.ConstantData("Y_mean", pt.mean(Y).eval())

        # Model the (normalized) latent trend
        ls = pm.LogNormal("ls", mu=np.log(0.5), sigma=0.2)
        noise = pm.HalfNormal("noise", sigma=0.05)
        cov = noise**2 * pm.gp.cov.ExpQuad(1, ls=ls)
        mean = pm.gp.mean.Constant(Y_mean)

        if hsgp:
            gp = pm.gp.HSGP(m=[30], c=4.0, cov_func=cov, mean_func=mean)
        else:
            gp = pm.gp.Latent(cov_func=cov, mean_func=mean)
        ylatent = gp.prior("ylatent", X[:, None], dims="records")

        # Connect to observations
        pm.Normal("L", mu=ylatent, sigma=Y_std / 3, observed=Y, dims="records")

    # Keep a handle on the GP
    pmodel.gp = gp
    return pmodel

def analyze(df_data, *, build_kwargs, sample_kwargs):
    pmodel = build_model(df_data, **build_kwargs)
    with pmodel:
        idata = pm.sample(
            chains=4, tune=2000,
            target_accept=0.9, random_seed=1234,
            **sample_kwargs,
        )
    return idata

df_data = pd.read_csv(io.StringIO("""
,x,y
0,6.5,0.03847670954287112
1,7.0,0.040795546149772384
2,7.5,0.04005530626829538
3,6.5,0.03800804967481005
4,7.0,0.042606645754122346
5,7.5,0.03962986979767001
6,6.5,0.03975987684954445
7,7.0,0.042854077804484525
8,7.5,0.0427959406500711
9,6.5,0.0376618863654496
10,7.0,0.043800640042141875
11,7.5,0.04280278855102723
"""), index_col=0)

analyze(
    df_data,
    build_kwargs=dict(hsgp=True),
    sample_kwargs=dict(nuts_sampler="nutpie"),
)

@michaelosthege This does run on my machine, maybe this was fixed with https://github.com/pymc-devs/pytensor/issues/343?
I do see some divergences, but that happens with nutpie and the default sampler.

Confirmed that a new env with PyMC 5.5.0 and PyTensor 2.12.3 fixed this MRE on my machine too.
I'll re-run my benchmarking notebooks next week to see if it fixed all instances I had run into.
Shall we close this and re-open if needed?

I guess I'll close this issue then, but feel free to reopen (or open a new one) if this comes up again...


README

README
User Story
As a bootcamp student
I want the prework notes to be structured on a webpage
So that i can easily find and read the information
Acceptance Criteria
GIVEN a Prework project repository
WHEN I view the README for the repository
THEN I can see a professional README that provides information on the project and how i use it.

Completed


(internal) release_patch: build v5.4.3500

Test plan: automated release PR, CI will perform additional checks

:green_circle: Internal release is ready for promotion!
:warning: Do not close/merge the pull request or delete the associated branch if you intend to promote it.


Installing adonis-mail-provider in v3 dev?

Hi, when I try to install adonis-mail-provider in an Adonis v3 dev app, I get this error:
UNMET PEER DEPENDENCY<EMAIL_ADDRESS>Is it possible to install Mail yet or should I wait until version  3's stable release?
Thanks for any help (great framework)

Yeah, Mail provider is dependent on fold2.x. I am yet to add new drivers to it, like Mailgun.
I will release the new version with correct dependency on fold soon.

Many thanks!!!! Loving Adonis.

As a note, Validator also has a peer dependency on fold 2, but it seems to work regardless...

The peer dependency has been fixed inside the new release.


Add TheTVDB link

While TheTVDB already contains clickable links to all populated sites, it would be awesome to have an icon for TheTVDB on the other sites (at least TMDB) as well.
Official brand colors and logo: https://support.thetvdb.com/kb/faq.php?id=79

This project primarily focuses on linking IMDb and TMDb pages. I've implemented Letterboxd integration since it doesn't require any API calls or web scraping. It simply involves straightforward page redirection. However, integrating TVDB is beyond the scope of this project, as obtaining TVDB IDs necessitates making API calls.