Add base decoder for ComposableCoW contract

Summary
As part of #1649, we need to return extra swap information for TWAP decoding, e.g. the start time, number of parts, time between parts and duration of part.
This adds a new ComposableCowDecoder, based on the required elements of the CoW SDK to detect and decode createWithContext calls. The reason for creating our own decoder is to reduce dependency reliance: the SDK has a peer dependency of ethers for example (which we don't use).
Changes

Add ComposableCoW ABI
Create ComposableCowDecoder
Add test coverage

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3967

Relevant Lines:
6452

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3967

Relevant Lines:
6452

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3967

Relevant Lines:
6452

💛  - Coveralls


Occassional lookups using ddclient sometimes returns the webserver IP

I have ddclient setup inside of kubernetes, using the excellent ifconfig.co to resolve my local IP address. The setup has worked wonders but I have over the last couple of weeks run into an issue where it sometimes finds another IP and sets that (always the same IP). Previously I saw it once every couple of days but I've now been seeing it daily and even multiple times per day.
Baffled by this behavior I've been trying to figure out if I have misconfigured something, or who owns that IP to try and understand what's going wrong. It wasn't until today that I actually just http requested that IP and was presented with the ifconfig.co splash page.

This IP is not the same IP as those registered on the domain name (guessing those are cloudflare proxies)
I only have log output from ddclient, not actual results from ifconfig.co. Before I continue troubleshooting (thinking a curl loop every 61sec and waiting for differing results, adhering to the API limitations), I'm just thinking out loud like, is there a scenario where ifconfig.co returns it's own IP instead of the visitors such as some cache/proxy/load-balancing fault or incoming load being too high?

I constructed a bash loop (while sleep 70; do ip=$(curl -s ifconfig.co) ; if [ "$ip" != "7x.x.x.x3" ] ; then echo results: $ip ; else echo "all good" ; fi ; done)
and once in a while am presented with either a completely empty response or:
<title>520 Origin Error</title>

520 Origin Error

cloudflare-nginx

Not yet quite sure how ddclient in these scenarios manages to pick out the ifconfig.co server ip instead of erroring out, but that's something for another repository.


fix the max button that won't allow to send fractions of PAW

Currently the Max button for sending $PAW  only allows to send entire value of PAW. This is handled by this function (in src/app/components/send/send.component.ts) :
setMaxAmount() {
    const walletAccount = this.walletService.wallet.accounts.find(a => a.id === this.fromAccountID);
    if (!walletAccount) {
      return;
    }

    this.amountExtraRaw = walletAccount.balanceRaw;

    const nanoVal = this.util.nano.rawToNano(walletAccount.balance).floor();
    const maxAmount = this.getAmountValueFromBase(this.util.nano.nanoToRaw(nanoVal));
    this.amount = maxAmount.toNumber();
    this.syncFiatPrice();
  }

As you can read, we first get the balance, convert from RAW to PAW (well nano but we all understand it is a lazy fork) and floor it (so only rounded-low value, then convert back in RAW
To get decimal value we could use the following code instead
  setMaxAmount() {
    const walletAccount = this.walletService.wallet.accounts.find(a => a.id === this.fromAccountID);
    if (!walletAccount) {
      return;
    }

    this.amountExtraRaw = walletAccount.balanceRaw;

    const maxAmount = this.getAmountValueFromBase(walletAccount.balance);
    this.amount = maxAmount.toNumber();
    this.syncFiatPrice();
  }

I don't think this PR is correct. The PAW fork of Nault probably shouldn't be merged into the main repo.
Is this a proposed changed you want to include into Nault? Then create a PR with only this change here. Otherwise you can propose a change at the PAW fork repo.

Is this a proposed changed you want to include into Nault? Then create a PR with only this change here.

even then the PR doesn't make sense because we specify additional raw in a separate variable, and it would break that behavior

Well, to be honest it was my first PR ever
Now, that change is intended to be for PAW (Biome) only, but when on GITHUB or Biome and clicking on PR, I end-up on nault...
So not sure how to do it properly for Biome only (even if that change would be beneficial also on nano main branch)
BR
Karl
From: BitDesert @.>
Sent: Sunday, 1 May 2022 18:51
To: Nault/Nault @.>
Cc: kduytsch @.>; Author @.>
Subject: Re: [Nault/Nault] fix the max button that won't allow to send fractions of PAW (PR #540)
I don't think this PR is correct. The PAW fork of Nault probably shouldn't be merged into the main repo.
Is this a proposed changed you want to include into Nault? Then create a PR with only this change here. Otherwise you can propose a change at the PAW fork repo.
—
Reply to this email directly, view it on GitHub https://github.com/Nault/Nault/pull/540#issuecomment-1114282254 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AIYQYQSK2D7ABWDRA4QDHXDVH2Y77ANCNFSM5UZVD3OQ .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AIYQYQVDUOXHJ2OZL23VPWLVH2Y77A5CNFSM5UZVD3O2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOIJVJSDQ.gif Message ID: @.*** @.***> >


Support UPDATE queries

This adds support for UPDATE queries.
Note that for whatever reason, gorm doesn't support joins in Update or Updates calls, but we can do WHERE x IN (SELECT ...) to get around it for now.  In either case, that's not a problem for this plugin.

Greate jobs!! Thank you! @doug-threatmate


Use the concurrency option in GitHub Actions workflows

Context

Summary
GitHub has recently announced a new concurrency key for use at the workflow and job level to ensure only a single workflow run or job is in progress. When used in combination with the cancel-in-progress key, incomplete workflow runs can be cancelled automatically to prevent running workflows unnecessarily.
Code copied from WordPress core commit - https://core.trac.wordpress.org/changeset/50930/trunk/.github/workflows/coding-standards.yml
Relevant Technical Choices

To-do

User-facing changes

Testing Instructions

QA

[ ] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:

UAT

[ ] UAT should use the same steps as above.

This PR can be tested by following these steps:

Reviews
Does this PR have a security-related impact?

Does this PR change what data or activity we track or use?

Does this PR have a legal-related impact?
Use the concurrency option in GitHub Actions workflows. Copied from WP core github action.
Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[x] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[x] This PR contains automated tests (unit, integration, and/or e2e) to verify the code works as intended (docs)
[x] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #7604

@choumx Any chance of a review?


Fix missing dependency when postgres and redis is disabled

Reproduce:
Custom values:
nodeSelector:
  environment: support
postgresql:
  enabled: false
redis:
  enabled: false

Apply
helm upgrade --install  --version 1.1.3 --wait --timeout 300s --debug --dry-run -f values.yaml chatwoot chatwoot/chatwoot

Output:
history.go:56: [debug] getting history for release chatwoot
Release "chatwoot" does not exist. Installing it now.
install.go:214: [debug] Original chart version: "1.1.3"
install.go:231: [debug] CHART PATH: /home/arch/.cache/helm/repository/chatwoot-1.1.3.tgz

Error: template: chatwoot/templates/tests/test-connection.yaml:19:20: executing "chatwoot/templates/tests/test-connection.yaml" at <include "common.tplvalues.render" (dict "value" .Values.nodeSelector "context" $)>: error calling include: template: no template "common.tplvalues.render" associated with template "gotpl"
helm.go:84: [debug] template: chatwoot/templates/tests/test-connection.yaml:19:20: executing "chatwoot/templates/tests/test-connection.yaml" at <include "common.tplvalues.render" (dict "value" .Values.nodeSelector "context" $)>: error calling include: template: no template "common.tplvalues.render" associated with template "gotpl"

@Placidina Thank you for the PR. Could you please create an issue that describes the problem being solved?

@Placidina Thank you for the PR. Could you please create an issue that describes the problem being solved?

Craeted #116

#101


Publishing to Azure not working

I'm sorry but can't get it working. When I publish the cloud project to azure I don't get an error but the database seems empty (no tables created). When I try to run the app I get the following error:
"Unable to update database to match the current model because there are pending changes and automatic migration is disabled. Either write the pending model changes to a code-based migration or enable automatic migration. Set DbMigrationsConfiguration.AutomaticMigrationsEnabled to true to enable automatic migration."
I've tried to copy the seed function to a migration file, but that didn't make a difference.
Any suggestions?

I've copied the code from module 3 and used a dropCreateDatabaseAlways. With that code it works.

Do we need to do a PR to resolve this issue in the b-release branch? Let me know.


Split Vensim views in separate Python files

This pull requests gives the user the option to get a translated model file organised by Vensim views. In the translated model, Vensim views can be seen as Python modules.
This option is controlled by the split_modules argument in the read_vensim function.
By default, the argument is set to False; hence, unless explicitly modified, the user gets the usual behaviour.
This new functionality has been added by adding a new grammar to parse the necessary bits from the Vensim sketch.
The option to split the model in views is particularly interesting for large models with tens of views. Translating those models into a single file may make the resulting Python model difficult to read and maintain.
In a Vensim model with three separate views (e.g. view_1, view_2 and view_3), setting split_modules to True would create the following tree inside the directory where the .mdl model is located:
| main-folder
| ├── modules_many_views_model
| │ ├── _modules.json
| │ ├── view_1.py
| │ ├── view_2.py
| │ └── view_3.py
| ├── _namespace_many_views_model.json
| ├── _subscripts_dict_many_views_model.json
| ├── many_views_model.py
|
|
If macros are present, they will be self-contained in files named as the macro itself. The macro inner variables will be placed inside the module that corresponds with the view in which they were defined.

Hello @rogersamso! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file pysd/py_backend/builder.py:

Line 237:80: E501 line too long (81 > 79 characters)
Line 399:80: E501 line too long (91 > 79 characters)
Line 400:80: E501 line too long (87 > 79 characters)
Line 421:80: E501 line too long (80 > 79 characters)
Line 568:80: E501 line too long (86 > 79 characters)
Line 709:80: E501 line too long (82 > 79 characters)
Line 826:80: E501 line too long (83 > 79 characters)
Line 829:80: E501 line too long (82 > 79 characters)
Line 941:80: E501 line too long (81 > 79 characters)
Line 1014:80: E501 line too long (83 > 79 characters)
Line 1017:80: E501 line too long (82 > 79 characters)
Line 1047:80: E501 line too long (82 > 79 characters)
Line 1145:80: E501 line too long (84 > 79 characters)
Line 1215:80: E501 line too long (83 > 79 characters)
Line 1218:80: E501 line too long (82 > 79 characters)
Line 1247:80: E501 line too long (82 > 79 characters)
Line 1317:80: E501 line too long (83 > 79 characters)
Line 1320:80: E501 line too long (82 > 79 characters)
Line 1390:80: E501 line too long (84 > 79 characters)
Line 1427:80: E501 line too long (84 > 79 characters)
Line 1455:80: E501 line too long (86 > 79 characters)
Line 1638:80: E501 line too long (82 > 79 characters)
Line 1707:80: E501 line too long (85 > 79 characters)
Line 1730:80: E501 line too long (83 > 79 characters)
Line 1736:80: E501 line too long (82 > 79 characters)

In the file pysd/py_backend/utils.py:

Line 747:80: E501 line too long (85 > 79 characters)
Line 750:80: E501 line too long (85 > 79 characters)
Line 753:80: E501 line too long (92 > 79 characters)
Line 754:80: E501 line too long (88 > 79 characters)

In the file pysd/py_backend/vensim/vensim2py.py:

Line 94:32: E131 continuation line unaligned for hanging indent
Line 367:80: E501 line too long (85 > 79 characters)
Line 481:80: E501 line too long (94 > 79 characters)
Line 487:80: E501 line too long (119 > 79 characters)
Line 493:80: E501 line too long (123 > 79 characters)
Line 494:80: E501 line too long (260 > 79 characters)
Line 496:80: E501 line too long (162 > 79 characters)
Line 501:80: E501 line too long (101 > 79 characters)
Line 508:80: E501 line too long (198 > 79 characters)
Line 510:80: E501 line too long (94 > 79 characters)
Line 517:80: E501 line too long (125 > 79 characters)
Line 523:80: E501 line too long (203 > 79 characters)
Line 524:80: E501 line too long (86 > 79 characters)
Line 525:80: E501 line too long (83 > 79 characters)
Line 529:80: E501 line too long (95 > 79 characters)
Line 533:80: E501 line too long (188 > 79 characters)
Line 535:80: E501 line too long (83 > 79 characters)
Line 541:80: E501 line too long (105 > 79 characters)
Line 549:80: E501 line too long (136 > 79 characters)
Line 566:80: E501 line too long (83 > 79 characters)
Line 575:13: E117 over-indented
Line 625:80: E501 line too long (84 > 79 characters)
Line 901:80: E501 line too long (88 > 79 characters)
Line 909:80: E501 line too long (84 > 79 characters)
Line 918:80: E501 line too long (82 > 79 characters)
Line 919:80: E501 line too long (82 > 79 characters)
Line 1012:80: E501 line too long (81 > 79 characters)
Line 1065:9: E122 continuation line missing indentation or outdented
Line 1066:9: E122 continuation line missing indentation or outdented
Line 1067:9: E122 continuation line missing indentation or outdented
Line 1123:80: E501 line too long (83 > 79 characters)
Line 1242:80: E501 line too long (83 > 79 characters)
Line 1243:80: E501 line too long (84 > 79 characters)
Line 1283:80: E501 line too long (84 > 79 characters)
Line 1288:80: E501 line too long (85 > 79 characters)
Line 1294:80: E501 line too long (88 > 79 characters)
Line 1311:80: E501 line too long (86 > 79 characters)
Line 1330:80: E501 line too long (86 > 79 characters)
Line 1342:80: E501 line too long (84 > 79 characters)
Line 1400:80: E501 line too long (87 > 79 characters)
Line 1444:80: E501 line too long (81 > 79 characters)
Line 1490:80: E501 line too long (84 > 79 characters)
Line 1539:80: E501 line too long (84 > 79 characters)
Line 1564:80: E501 line too long (83 > 79 characters)
Line 1569:80: E501 line too long (103 > 79 characters)
Line 1575:80: E501 line too long (89 > 79 characters)
Line 1587:80: E501 line too long (82 > 79 characters)
Line 1627:80: E501 line too long (82 > 79 characters)
Line 1629:80: E501 line too long (82 > 79 characters)
Line 1724:80: E501 line too long (81 > 79 characters)

In the file tests/more-tests/split_model/translation/test_split_model.py:

Line 20:80: E501 line too long (82 > 79 characters)

In the file tests/unit_test_pysd.py:

Line 625:80: E501 line too long (82 > 79 characters)
Line 757:80: E501 line too long (81 > 79 characters)
Line 772:80: E501 line too long (86 > 79 characters)
Line 792:80: E501 line too long (87 > 79 characters)
Line 793:80: E501 line too long (86 > 79 characters)
Line 801:80: E501 line too long (88 > 79 characters)
Line 819:80: E501 line too long (87 > 79 characters)
Line 844:80: E501 line too long (81 > 79 characters)
Line 856:80: E501 line too long (86 > 79 characters)
Line 877:80: E501 line too long (88 > 79 characters)
Line 880:80: E501 line too long (83 > 79 characters)
Line 884:80: E501 line too long (87 > 79 characters)
Line 903:80: E501 line too long (84 > 79 characters)
Line 958:80: E501 line too long (88 > 79 characters)
Line 970:80: E501 line too long (83 > 79 characters)
Line 976:80: E501 line too long (88 > 79 characters)
Line 977:80: E501 line too long (88 > 79 characters)
Line 982:80: E501 line too long (82 > 79 characters)
Line 1037:80: E501 line too long (88 > 79 characters)
Line 1092:80: E501 line too long (88 > 79 characters)
Line 1376:80: E501 line too long (88 > 79 characters)
Line 1471:80: E501 line too long (85 > 79 characters)
Line 1505:80: E501 line too long (85 > 79 characters)
Line 1620:80: E501 line too long (87 > 79 characters)
Line 1628:80: E501 line too long (80 > 79 characters)
Line 1644:80: E501 line too long (82 > 79 characters)
Line 1757:80: E501 line too long (85 > 79 characters)

In the file tests/unit_test_utils.py:

Line 547:1: W293 blank line contains whitespace
Line 548:5: E303 too many blank lines (2)
Line 550:1: W293 blank line contains whitespace
Line 557:1: W293 blank line contains whitespace
Line 559:5: E303 too many blank lines (2)
Line 561:1: W293 blank line contains whitespace
Line 565:1: W293 blank line contains whitespace
Line 567:1: W293 blank line contains whitespace
Line 570:1: W293 blank line contains whitespace
Line 571:20: W292 no newline at end of file

In the file tests/unit_test_vensim2py.py:

Line 26:80: E501 line too long (88 > 79 characters)
Line 33:80: E501 line too long (88 > 79 characters)
Line 35:80: E501 line too long (80 > 79 characters)
Line 36:80: E501 line too long (87 > 79 characters)
Line 48:80: E501 line too long (85 > 79 characters)
Line 49:80: E501 line too long (87 > 79 characters)
Line 67:80: E501 line too long (87 > 79 characters)
Line 213:80: E501 line too long (80 > 79 characters)
Line 371:80: E501 line too long (85 > 79 characters)
Line 392:80: E501 line too long (80 > 79 characters)
Line 579:80: E501 line too long (86 > 79 characters)
Line 592:80: E501 line too long (88 > 79 characters)
Line 608:80: E501 line too long (88 > 79 characters)
Line 688:80: E501 line too long (80 > 79 characters)
Line 717:80: E501 line too long (81 > 79 characters)
Line 721:80: E501 line too long (80 > 79 characters)
Line 751:80: E501 line too long (129 > 79 characters)
Line 754:80: E501 line too long (111 > 79 characters)
Line 789:1: E302 expected 2 blank lines, found 1
Line 802:80: E501 line too long (87 > 79 characters)
Line 806:1: W391 blank line at end of file

Coverage increased (+0.03%) to 98.539% when pulling 62d13c190e03d512d85148fc8c7170733231265d on rogersamso:building_split_model_merge into 86b109fc99669dc1b291f7bd8aeaaae159961e74 on JamesPHoughton:master.

Other changes included in this PR:

LICENSE added to MANIFEST as suggested by conda-forge maintainer
Added conda version badge to README and installation instructions to docs
Splitting option working with CLI as well
Add some tests for subscripts range parsing error

Hello @rogersamso! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
* In the file [`pysd/py_backend/vensim/vensim2py.py`](https://github.com/JamesPHoughton/pysd/blob/cdf4499d16525df731895ff61c86f2af6114baf5/pysd/py_backend/vensim/vensim2py.py):

Line 503:80: E501 line too long (111 > 79 characters)
Line 507:80: E501 line too long (115 > 79 characters)
Line 508:80: E501 line too long (252 > 79 characters)
Line 521:80: E501 line too long (190 > 79 characters)
Line 542:80: E501 line too long (156 > 79 characters)

Comment last updated at 2021-07-05 12:12:28 UTC

Sorry about this. I think a good solution for it would be to move all grammars into a separate file.

Just a few quesetions - looks good otherwise. =) Nice formatting changes, that'll make it a lot easier to read.


djangopackages.com: Row "Types" is empty

On the comparison site, the row "Types" is empty for python-social-auth.
https://www.djangopackages.com/grids/g/authentication/
Would be nice if you could fill in the missing data.
Thank you.

Can't do at the moment due to a bug on their app, will update that once they fix https://github.com/pydanny/djangopackages/issues/392


Update sample-app-github-cd.yml

Motivation and Context

Description

Contribution Checklist

[x] I have built and tested the code locally and in a deployed app
[x] For frontend changes, I have pulled the latest code from main, built the frontend, and committed all static files.
[x] This is a change for all users of this app. No code or asset is specific to my use case or my organization.
[x] I didn't break any existing functionality :smile:

Hi @sugiura-atsushi-brain , can you please give some motivation and context for this change?  Does this solve a particular problem that you've experienced?

Hi @sugiura-atsushi-brain , I'm going to close this PR for now until you have a chance to answer the above question around the particular problem you're trying to fix here.  Feel free to reopen it when you're ready.


Parallelized read-write operations in Hoodie Merge phase

Parallelized read-write operations in Hoodie Merge phase
Make BufferedIterator generic enough to be able to Buffer any type of payload, not just HoodieRecord.

@vinothchandar Refactored, please take a pass.


Temporal alignment between components

In my recent test, the pose solver was using detector timestamps that clearly did not align with its own. It resulted in data being marked as "old" and discarded.
So we need a means of synchronizing the timestamps between different components.
I think the controller seems like the most logical part of the software to measure and adjust timestamps. In this scenario, if multiple clients connect to the same detector/pose solver, then the detector/pose solver does not need to keep different times for each client.
Question 1: How do we measure the temporal offsets?
It seems fairly straightforward to add endpoints to request the local time of the detector/pose solver... but it may be important to account for the duration of several round-trip request-response.
Question 2: How do we account for the temporal offset once it has been measured?
The controller will have to store the offset for each component, and add/subtract the offset whenever receiving/sending data.

Steps discussed
o	API to have time sync requests and appropriate responses
o	Detector to have a “time sync” mode
o	Detector to handle time sync requests and responses
o	Controller to send time sync requests and responses
o	Controller to calculate network latency
o	Controller to calculate time offset


TS-4976: Regularize plugins - protocol_stack

Incremental work on #1114 plus a small documentation fixup.

RAT check failed! https://ci.trafficserver.apache.org/job/RAT-github/110/

FreeBSD11 build failed! https://ci.trafficserver.apache.org/job/freebsd-github/1792/

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/97/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/111/

AU check failed! https://ci.trafficserver.apache.org/job/autest-github/94/

Intel CC build failed! https://ci.trafficserver.apache.org/job/icc-github/223/

FreeBSD11 build failed! https://ci.trafficserver.apache.org/job/freebsd-github/1793/

Linux build failed! https://ci.trafficserver.apache.org/job/linux-github/1685/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/355/

Yeah, I force pushed immediately after setting up the pull request when I noticed a typo and that seems to have bollixed the PR builds.

Please add Makefile.am.

[approve ci]

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/103/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/117/

FreeBSD11 build failed! https://ci.trafficserver.apache.org/job/freebsd-github/1799/

AU check failed! https://ci.trafficserver.apache.org/job/autest-github/100/

Intel CC build failed! https://ci.trafficserver.apache.org/job/icc-github/229/

Linux build failed! https://ci.trafficserver.apache.org/job/linux-github/1691/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/361/

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/111/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/125/

AU check successful! https://ci.trafficserver.apache.org/job/autest-github/108/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/1807/

Linux build successful! https://ci.trafficserver.apache.org/job/linux-github/1698/

Intel CC build successful! https://ci.trafficserver.apache.org/job/icc-github/237/

clang-analyzer build successful! https://ci.trafficserver.apache.org/job/clang-analyzer-github/369/


[Feature Request] Show all child-terms

Is your feature request related to a problem? Please describe.
If you use the (term-related) Search Term field, it only shows search results after typing in 3 letters.
Describe the solution you'd like
It would be nice if one could see all child-terms of the parent term already when clicking on the blank field.
Describe alternatives you've considered
None.
Additional context
None.

Added option to double click into empty term search field. This will now show a scrollable drop down with all child terms.
This will be added in 0.4.0 and will need future refinement according to userfeedback.


Null exception for parameter "Path" in UWP

Using the plugin in UWP (Windows 10 for desktop and a Windows 10 mobile device)
I get the following exception when calling await recorder.StartRecording();
"{System.ArgumentNullException: Path cannot be null.Parameter name: path   at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options)   at Plugin.AudioRecorder.WaveRecorder.d__6.MoveNext()--- End of stack trace from previous location where exception was thrown ---   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)   at Plugin.AudioRecorder.AudioRecorderService.d__43.MoveNext()--- End of stack trace from previous location where exception was thrown ---   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()   at atWork.ViewModels.HomeViewModel.d__40.MoveNext()}"
I have granted microphone and storage permissions, but the devices dont have external SD card.

I also found this blocking issue while using the package on UWP platform, I'm waiting for an official fix on the nuget package.

Should be fixed in #15, though I'm unable to release a package at the moment as my Windows machine needs to be rebuilt. If you need this fix, just build the project and use the DLLs directly until I have time to revisit.


include phy rate into plot

would you think it's worth the effort to include e.g. the MCS (modulation and coding scheme) index into the plot to have a better understanding what the wifi hardware is doing with the link quality?
br

Yes!  I think that would be great!  If we can settle on what we think the best MCS is, I can just use that for all tests moving forward.

Wait... sorry... I got my projects confused.  Is what you say even possible?

at least it's possible to see these values by issuing some shell command:
iw dev wlan0 link
Connected to 68:7f:74:3b:b0:01 (on wlan0)
SSID: tesla-5g-bcm
freq: 5745
RX: 30206 bytes (201 packets)
TX: 4084 bytes (23 packets)
signal: -31 dBm
tx bitrate: 300.0 MBit/s MCS 15 40Mhz short GI
https://wireless.wiki.kernel.org/en/users/documentation/iw
hope to soon find some time to look more into this

That would be really cool :-D

i picked up to work on this. unfortunately I noticed that the MCS value is not always provided. Will try to add a bitrate based graph to the plot.


Forbidden functions are not part of the coding style

Forbidden functions are not part of the coding style but are per-project and thus cannot be detected should not be detected in a general checker.

I think it's better for NormEZ to flag possible forbidden functions in its default behavior. It can really save a project.  However I'm working on an option to disable forbidden functions warnings.

I was just talking about the Single Resposibility Principle - actually it may reveal useful to check for forbidden functions - forbidden functions are just not in the coding style so maybe you'd pick a more broad name and description.

Feature added by this pull request.


'exception_ptr' is not a member of 'std'

I'm using this library for an ARM chip with the SKU ATSAMC21E18A. Whenever I try to compile, I get the error, 'exception_ptr' is not a member of 'std' from the catch.hpp file. I feel like the issue probably has something to do with the compiler I'm using. I am using the latest toolchain for Atmel Studio. I'm using the GNU compiler with the -std=c++0x option. I also tried using -std=c++11 and std=c++14. The Catch2 github says it works with C++11 and exception_ptr is a C++11 feature. What is the compiler that you used when compiling this library? I'm not using an Arduino, but from looking at the code, I think this library is hardware agnostic. Thanks for your help.

Good morning :wave: :coffee: Please be advised that this is an Arduino library which is only supported in context with using the Arduino toolchain. Anything else and you are on your own. That being said I believe you experience this issue because your toolchain is picking up the test code within the extras/test subfolder. Just delete it and you should be good.

Good morning :wave: :coffee: Please be advised that this is an Arduino library which is only supported in context with using the Arduino toolchain. Anything else and you are on your own. That being said I believe you experience this issue because your toolchain is picking up the test code within the extras/test subfolder. Just delete it and you should be good.

Thanks! I appreciate the response!

Thanks! I appreciate the response!


feat: update to 1.5.5 release

What does this PR do?
Upgrade to 1.5.5 release.
Test Plan
Manual
Related PRs and Issues
None
Have you read the Contributing Guidelines on issues?
Yes

Can't test this due to https://github.com/gitpod-io/gitpod/issues/19662


Implement Comment Section  (UI)

@rhedwan . I sent you on Slack but seem you have not read it since yesterday. Pls can this task be reassigned to another person, not being able to come up with viable solution. Thanks


Merge typed role and role group configs

Description
This PR adds a macro Optional that must be used in the FooConfig(s) of our operators.
# use stackable_operator::config::optional::Optional;
const DEFAULT_PORT: u16 = 11111;
// For example, this:
#[derive(Optional)]
struct FooConfig {
    #[optional(default_value = "DEFAULT_PORT")]
    port: u16,
}
// Expands to (roughly) the following:
#[derive(Merge)]
struct OptionalFooConfig {
    port: Option<u16>,
}
impl From<OptionalFooConfig> for FooConfig {
   fn from(c: OptionalFooConfig) -> Self {
       Self {
           port: c.port.unwrap_or(DEFAULT_PORT),
       }
   }
}

The two created Configs are contained in the CommonConfiguration in an enum like:
pub enum Config<O, M>
where
    O: Clone + Default + Merge,
    M: Configuration + From<O>,
{
    #[serde(rename = "config")]
    Optional(O),
    #[serde(skip)]
    Merged(M),
}

The Optional part representing OptionalFooConfig will be exposed in the CRD. After deserilization and merging the operators work with the Merged part containing the original FooConfig. To retrieve the "original" merged FooConfig in the code, a get method must be called. This is breaking as well as the new Role parametrization:
let port = rolegroup.config.config.get().port;

This means we now need to configure our Role in the FooClusterSpec like:
#[derive(Clone, CustomResource, Debug, Default, Deserialize, JsonSchema, PartialEq, Serialize)]
#[kube(
    group = "foo.stackable.tech",
    version = "v1alpha1",
    kind = "FooCluster",
    plural = "fooclusters",
    shortname = "foo",
    namespaced,
    crates(
        kube_core = "stackable_operator::kube::core",
        k8s_openapi = "stackable_operator::k8s_openapi",
        schemars = "stackable_operator::schemars"
    )
)]
#[serde(rename_all = "camelCase")]
pub struct FooClusterSpec {
   #[serde(default, skip_serializing_if = "Option::is_none")]
    pub foo_server: Option<Role<OptionalFooConfig, FooConfig>>,
}

Additionally, the Role and RoleGroup have a custom deserializer that merges the Role and RoleGroups OptionalFooConfig including the defaults. That means after deserialization no other merging has to be done.
FYI: It works with ZooKeeper, im not fully happy with the created macro which still has some flaws (e.g. the derives of the OptionalFooConfig are currently hardcoded). There are still alot of clones in the code which i wasnt able yet to remove. Additionally, the merge is not working yet for complex structs. They are Optional in the OptionalFooConfig and due to the Merge trait need an impl for Atomic (which basically stops the merge and just takes whatever is behind the option).
@teozkr I really need you to have a look at the general idea/structure and especially the macro :)
Operator implementation: https://github.com/stackabletech/zookeeper-operator/pull/508
fixes: https://github.com/stackabletech/operator-rs/issues/284

Review Checklist

[ ] Code contains useful comments
[ ] (Integration-)Test cases added (or not applicable)
[ ] Documentation added (or not applicable)
[ ] Changelog updated (or not applicable)
[ ] Cargo.toml only contains references to git tags (not specific commits or branches)

Once the review is done, comment bors r+ (or bors merge) to merge. Further information

will be handeld with a different approach by @teozkr. Closing for now.


Fix unknown param in grib namespaces

With this PR when "param" or "shortName" in a namespace is "~" in the GRIB header, metadata() returns the value of "paramId" as a str for both these keys in the relevant namespaces. Previously "~" was returned.

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

Please upload report for BASE (main@7849651). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #477   +/-   ##
=======================================
  Coverage        ?   90.57%
=======================================
  Files           ?      123
  Lines           ?     8340
  Branches        ?      702
=======================================
  Hits            ?     7554
  Misses          ?      655
  Partials        ?      131

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


Fix Rails/UniquenessValidationWithoutIndex false positive on explicit disable

Fixes: Rails/UniqueValidationWithoutIndex
Currently validates :column_name, uniqueness: false will report an offence if the schema does not specify a unique index for the corresponding database column. The cop is currently insensitive to the value of the :uniqueness option. Any value (including falsy false and nil) will cause this cop to expect a unique index in the schema.
This pull request fixes this behaviour such that we will now only report offences when the value of the :uniqueness option is truthy.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.
[x] If this is a new cop, consider making a corresponding update to the Rails Style Guide.

This looks good to me. Can you squash your commits into one?

Thanks!


get 'path' fails when path contains invalid UTF-8 characters

What Ruby, Rails and RSpec versions are you using?
Ruby version: ruby 2.3.3p222 (2016-11-21 revision 56859) [x86_64-darwin18]
Rails version: 5.0.6
RSpec version: 3.7

Observed behaviour
This test fails
require 'rails_helper'

describe Rack::Attack do
  include Rack::Test::Methods

  let(:blocked) { Rack::Utils::SYMBOL_TO_STATUS_CODE[:forbidden] }

  def app
    Rails.application
  end

  describe 'blocking' do
    it 'blocks URLs with invalid UTF-8 characters' do
      get "/?q=a\xff"
      expect(last_response.status).to eq(blocked)
    end
  end
end

with the following output
URI::InvalidURIError: URI must be ascii only "/?q=a\xFF"

  0) Rack::Attack vulnerability probes blocks URLs with invalid UTF-8 characters
     Failure/Error: get "/?q=a\xff"

     URI::InvalidURIError:
       URI must be ascii only "/?q=a\xFF"
     # ./spec/initializers/rack_attack_spec.rb:60:in `block (3 levels) in <top (required)>'

Expected behaviour
The URI is not validated (as attackers will not be passing their URIs through validators!) but is accessed, and blocked by Rack::Attack.
Can you provide an example app?
Unfortunately not — rails _5.0.6_ new issue created an app using rails 5.2.3, and changing the version in the Gemfile to 5.0.6 and running bundle update produced this:
└─▷ bundle exec rails g rspec:install
/Users/me/.rbenv/versions/2.3.3/lib/ruby/gems/2.3.0/gems/railties-5.0.6/lib/rails/railtie/configuration.rb:95:in `method_missing': undefined method `load_defaults' for #<Rails::Application::Configuration:0x007f8b67cd50d8> (NoMethodError)
	from /Users/me/issue/config/application.rb:12:in `<class:Application>'
	from /Users/me/issue/config/application.rb:10:in `<module:Issue>'
	from /Users/me/issue/config/application.rb:9:in `<top (required)>'
	from /Users/me/.rbenv/versions/2.3.3/lib/ruby/gems/2.3.0/gems/spring-2.0.2/lib/spring/application.rb:92:in `require'
	from /Users/me/.rbenv/versions/2.3.3/lib/ruby/gems/2.3.0/gems/spring-2.0.2/lib/spring/application.rb:92:in `preload'
	from /Users/me/.rbenv/versions/2.3.3/lib/ruby/gems/2.3.0/gems/spring-2.0.2/lib/spring/application.rb:153:in `serve'
	from /Users/me/.rbenv/versions/2.3.3/lib/ruby/gems/2.3.0/gems/spring-2.0.2/lib/spring/application.rb:141:in `block in run'
	from /Users/me/.rbenv/versions/2.3.3/lib/ruby/gems/2.3.0/gems/spring-2.0.2/lib/spring/application.rb:135:in `loop'
	from /Users/me/.rbenv/versions/2.3.3/lib/ruby/gems/2.3.0/gems/spring-2.0.2/lib/spring/application.rb:135:in `run'
	from /Users/me/.rbenv/versions/2.3.3/lib/ruby/gems/2.3.0/gems/spring-2.0.2/lib/spring/application/boot.rb:19:in `<top (required)>'
	from /Users/me/.rbenv/versions/2.3.3/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require'
	from /Users/me/.rbenv/versions/2.3.3/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:55:in `require'
	from -e:1:in `<main>'

If I try this with Rails 5.2.3, bundle exec rails g rspec:install hangs and never prints any output, nor does it respond to presses of ^C — I need to kill it manually.

Ah, apologies — this looks to be a rack-test issue, not rspec rails.


[Documentation] Fix UnsafeMutablePointer's documentation

Fix incorrect documentation of UnsafeMutablePointer
See https://forums.swift.org/t/67945

@swift-ci please smoke test

Noting here on the PR, for the record, that Steve Canon already confirmed this was an error.

Thank you, @Kyle-Ye !


feat(union) added union function using the new Set method.

@dev-warner Hey Joe, thank you for the PR! Looks good and merged :)


Correct URL getting logged to be the right address

The URL that was getting logged was incorrect in one location and was missing a /
This updates the URL to be correct

Report bugs in Issues
The following are automatically added:

Add reviewers from OWNER file (in the root of the repository) under reviewers section.
Set PR size label.
New issue is created for the PR. (Closed when PR is merged/closed)
Run pre-commit if .pre-commit-config.yaml exists in the repo.

Available user actions:

To mark PR as WIP comment /wip to the PR, To remove it from the PR comment /wip cancel to the PR.
To block merging of PR comment /hold, To un-block merging of PR comment /hold cancel.
To mark PR as verified comment /verified to the PR, to un-verify comment /verified cancel to the PR.
verified label removed on each new commit push.
To cherry pick a merged PR comment /cherry-pick <target branch to cherry-pick to> in the PR.

Multiple target branches can be cherry-picked, separated by spaces. (/cherry-pick branch1 branch2)
Cherry-pick will be started when PR is merged

To build and push container image command /build-and-push-container in the PR (tag will be the PR number).
To add a label by comment use /<label name>, to remove, use /<label name> cancel
To assign reviewers based on OWNERS file use /assign-reviewers
To check if PR can be merged use /check-can-merge

Supported /retest check runs

Supported labels

hold
verified
wip
lgtm

/verified

/check-can-merge


No healthy upstream at step 3. LB external IP pending.

I'm having the same issue at step 3: edge-envoy external IP is stuck in pending status and in my case every time I try to interact with edge-envoy API  I get no healthy upstream.
NAME                              READY   STATUS    RESTARTS   AGE
pod/edge-envoy-765798cf45-sw2q5   1/1     Running   2          6m
pod/postgres-6ccf94dfb7-kpsnp     1/1     Running   1          2h
pod/usersvc-645d6877fb-rx55n      1/1     Running   0          6m
pod/usersvc-sds-f777d75c5-xs7tf   1/1     Running   0          6m
NAME                  TYPE           CLUSTER-IP       EXTERNAL-IP   PORT(S)          AGE
service/edge-envoy    LoadBalancer   <IP_ADDRESS>         8000:32149/TCP   6m
service/kubernetes    ClusterIP      <IP_ADDRESS>                443/TCP          39d
service/postgres      ClusterIP      <IP_ADDRESS>            5432/TCP         2h
service/usersvc       ClusterIP      <IP_ADDRESS>           80/TCP           6m
service/usersvc-sds   NodePort       <IP_ADDRESS>            5000:31085/TCP   6m
I'm using macOS High Sierra (10.13.6)
minikube version: v0.30.0
kubectl version
Client Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.0", GitCommit:"0ed33881dc4355495f623c6f22e7dd0b7632b7c0", GitTreeState:"clean", BuildDate:"2018-09-28T15:20:58Z", GoVersion:"go1.11", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.0", GitCommit:"fc32d2f3698e36b93322a3465f63a14e9f0eaead", GitTreeState:"clean", BuildDate:"2018-03-26T16:44:10Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
docker version
Client: Docker Engine - Community
Version:           18.09.0
API version:       1.35
Go version:        go1.10.4
Git commit:        4d60db4
Built:             Wed Nov  7 00:47:43 2018
OS/Arch:           darwin/amd64
Experimental:      false
Server:
Engine:
Version:          17.12.1-ce
API version:      1.35 (minimum version 1.12)
Go version:       go1.9.4
Git commit:       7390fc6
Built:            Tue Feb 27 22:20:43 2018
OS/Arch:          linux/amd64
Experimental:     false
Thanks
Originally posted by @m1ch3lp3r3z in https://github.com/datawire/envoy-steps/issues/2#issuecomment-440050257

I am also running into this issue, have you (or anyone else) figured out a solution?


Implement opt-in change detection bypass

Objective
Implements #2348
Solution
Implements the solution outlined here

This looks delightfully overengineered. I would be on board, but my sentiment from linked issue remains: do we actually need it?
The two examples so far can be (and best be, if you ask me) solved by splitting resources/components not just by the data they store, but also by the mutation events that happen to them, if that makes sense. I think that's a universal solution, be it inside a specific plugin or between several plugins.

but also by the mutation events that happen to them

@Ratysz Would you mind clarifying exactly what you mean by this?

but also by the mutation events that happen to them

@Ratysz Would you mind clarifying exactly what you mean by this?
(I assume it just mean on when they trigger change detection via DerefMut for example?)

Yes, pretty much. E.g., we could split Assets into AssetStorage and AssetEvents, separating the concern of providing assets from that of processing asset-related events.

While this can sometimes be an option, I don't think it can be applied globally as it requires changing the external API to do something that only bothers the internal impl. For the asset case for example, the external API shouldn't need to care about how the Assets type also accumulates events and sends them back.  This impl allows for building such abstractions.

Ran into a user with a serious need for this earlier today. Effectively, they wanted to make a change detecting that system that responded to all changes other than its own. I spent a while helping them and couldn't come up with a clean workaround.
This now has my 👍🏽 in terms of use case. Lets take a look at the specific design now...

Gave you some docs ;) The impl looks reasonable to me. Two outstanding concerns:

From an API perspective, I would also be interested in supporting Query<Untracked<& mut A>>, which better communicates the intent in the function signature and should allow ordinary iteration in ways that the current design does not.
This is reasonably non-trivial: it should have tests to verify that it acts as promised.

I haven't used Reflect all that much, but is there a reason it doesn't have the other impl macros? (i.e., impl_into_inner, impl_change_detection_circumvention, impl_debug)
https://github.com/bevyengine/bevy/blob/6591742a73e9632bdaebe9d8ac9cda969b0e4ef3/crates/bevy_ecs/src/change_detection.rs#L216

Closing in favor of the upcoming permissions work

I ran into a very compelling use case of this today.
I have two components: Direction and Rotation, that I want to keep synchronized. If one has changed, update the other to match.
However, this creates an infinite ping-pong, as doing so causes the other to be changed, thus triggering another update. The workaround is to check whether the new value is equal to the old value before setting it.

Reopening this: permissions has serious open questions, and this is at least an important stop-gap.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2890

Questions ?
Please refer to the Backport tool documentation


Add copy method to Nodes

Adds a copy method to Nodes.

Codecov Report

Merging #213 into master will decrease coverage by 0.1%.
The diff coverage is 60%.

@@             Coverage Diff              @@
##             master     #213      +/-   ##
============================================
- Coverage     79.67%   79.57%   -0.11%
- Complexity      376      377       +1
============================================
  Files            46       46
  Lines           930      935       +5
  Branches        174      175       +1
============================================
+ Hits            741      744       +3
- Misses          113      114       +1
- Partials         76       77       +1

Impacted Files
Coverage Δ
Complexity Δ

...rc/main/kotlin/org/cafejojo/schaapi/models/Node.kt
57.14% <0%> (-9.53%)
0 <0> (ø)

...aapi/models/libraryusagegraph/jimple/JimpleNode.kt
86.36% <75%> (-2.53%)
21 <1> (+1)

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 63ca927...069fb65. Read the comment docs.


[$1000] Report list section is empty while sliding.

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

Action Performed:

Open app
Tap on a report
Slide to the right to go to home screen

Expected Result:
Screen should not be empty
Actual Result:
Screen is empty
Workaround:
unknown
Platform:

Where is this issue occurring?

Android

Version Number: 1.2.38-1
Reproducible in staging?: y
Reproducible in production?: y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/207218692-e973bd26-5f89-4161-a112-d2f7bdece129.mp4
https://user-images.githubusercontent.com/43996225/207218723-9cde55f5-097e-4a93-b26b-687a9d29ecd7.mp4
Expensify/Expensify Issue URL:
Issue reported by: @mdneyazahmad
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1670861157206459
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~014a4b9a273415b3b1
Upwork Job ID:<PHONE_NUMBER>076718080
Last Price Increase: 2022-12-13

I was able to reproduce this on BrowserStack with the latest version of the app.

@kavimuru it's reported on slack here

Proposal
I think this one did intensionally to optimise re render when sidebar (LHNOptionsList) not visible should not update them. In this case its replace with null by react-freeze to optimise re-render.
diff --git a/src/pages/home/sidebar/SidebarLinks.js b/src/pages/home/sidebar/SidebarLinks.js
index 12a09d5e18..8935e40119 100644
--- a/src/pages/home/sidebar/SidebarLinks.js
+++ b/src/pages/home/sidebar/SidebarLinks.js
@@ -169,25 +169,23 @@ class SidebarLinks extends React.Component {
                         />
                     </TouchableOpacity>
                 </View>
-                <Freeze freeze={this.props.isSmallScreenWidth && !this.props.isDrawerOpen && this.isSidebarLoaded}>
-                    <LHNOptionsList
-                        contentContainerStyles={[
-                            styles.sidebarListContainer,
-                            {paddingBottom: StyleUtils.getSafeAreaMargins(this.props.insets).marginBottom},
-                        ]}
-                        data={optionListItems}
-                        focusedIndex={_.findIndex(optionListItems, (
-                            option => option.toString() === this.props.reportIDFromRoute
-                        ))}
-                        onSelectRow={this.showReportPage}
-                        shouldDisableFocusOptions={this.props.isSmallScreenWidth}
-                        optionMode={this.props.priorityMode === CONST.PRIORITY_MODE.GSD ? 'compact' : 'default'}
-                        onLayout={() => {
-                            App.setSidebarLoaded();
-                            this.isSidebarLoaded = true;
-                        }}
-                    />
-                </Freeze>
+                <LHNOptionsList
+                    contentContainerStyles={[
+                        styles.sidebarListContainer,
+                        {paddingBottom: StyleUtils.getSafeAreaMargins(this.props.insets).marginBottom},
+                    ]}
+                    data={optionListItems}
+                    focusedIndex={_.findIndex(optionListItems, (
+                        option => option.toString() === this.props.reportIDFromRoute
+                    ))}
+                    onSelectRow={this.showReportPage}
+                    shouldDisableFocusOptions={this.props.isSmallScreenWidth}
+                    optionMode={this.props.priorityMode === CONST.PRIORITY_MODE.GSD ? 'compact' : 'default'}
+                    onLayout={() => {
+                        App.setSidebarLoaded();
+                        this.isSidebarLoaded = true;
+                    }}
+                />
             </View>
         );
     }

If we want to show sidebar always we can remove freeze from LHNOptionsList.

Hangggg on here, I'm not entirely sure this issue needs to be fixed, it sounds like to me that this is an optimization that was very much intentional, and removing this would probably be a regression.
The Freeze component was added here. Marc outlines here how "the only downside is that the frozen elements show as hidden when we are swiping". So it's actually expected that the report list is empty while sliding–it's a side-effect of using the Freeze library.
@marcaaron Do you think we should keep this issue open? Is it worth investigating a solution for this? Feels SUPER edge casey to me and also not important to "fix" at all.

I agree with @jasperhuangg's comment above. @marcaaron could you please confirm if this is expected?

Yep, this is not a bug. Maybe a feature request - but the freeze component is intentional.


Target netstandard2.0 TFM

:up::date:

This can go in to unblock downstream repos since it doesn't depend on any upstream repos (testing, jsonpatch, ...)

ping @pranavkm @natemcmaster @kichalla


sync.WaitGroup support timeout

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

$ go version

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?

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

You can use golang.org/x/sync/errgroup to achieve this. Call WithContext and pass in a context that gets cancelled after a timeout.

What @polarina said. I also don't see how we could add support for this without breaking existing users or without adding extra overhead. You could also just use channels and a combination of select with a timer.


Issues with filters

If I use a filter, it still shows cells even when the filer returned no results:

Also scale should just have 1 colour in this example

@jimmyjones2 Sorry for the delay in responding. For the second problem, you can adjust the number of colors for the scales in the legend. The minimum number is 3 colors.
As for the the filters, you could post a screenshot of the results in the spy panel, so I have an understanding of what elasticsearch is returning.

{
  "took": 120,
  "timed_out": false,
  "_shards": {
    "total": 1,
    "successful": 1,
    "failed": 0
  },
  "hits": {
    "total": 29223,
    "max_score": 0,
    "hits": []
  },
  "aggregations": {
    "2": {
      "doc_count_error_upper_bound": 0,
      "sum_other_doc_count": 92,
      "buckets": [
        {
          "3": {
            "buckets": {
              "DestinationIp: <IP_ADDRESS>": {
                "doc_count": 2
              },
              "DestinationIp: <IP_ADDRESS>": {
                "doc_count": 0
              }
            }
          },
          "key": 137,
          "doc_count": 19
        },
        {
          "3": {
            "buckets": {
              "DestinationIp: <IP_ADDRESS>": {
                "doc_count": 0
              },
              "DestinationIp: <IP_ADDRESS>": {
                "doc_count": 0
              }
            }
          },
          "key": 80,
          "doc_count": 8
        },
        {
          "3": {
            "buckets": {
              "DestinationIp: <IP_ADDRESS>": {
                "doc_count": 0
              },
              "DestinationIp: <IP_ADDRESS>": {
                "doc_count": 0
              }
            }
          },
          "key": 5355,
          "doc_count": 8
        },
        {
          "3": {
            "buckets": {
              "DestinationIp: <IP_ADDRESS>": {
                "doc_count": 0
              },
              "DestinationIp: <IP_ADDRESS>": {
                "doc_count": 0
              }
            }
          },
          "key": 67,
          "doc_count": 5
        },
        {
          "3": {
            "buckets": {
              "DestinationIp: <IP_ADDRESS>": {
                "doc_count": 1
              },
              "DestinationIp: <IP_ADDRESS>": {
                "doc_count": 0
              }
            }
          },
          "key": 138,
          "doc_count": 4
        }
      ]
    }
  }
}

@stormpython No worries, thanks for taking a look. Let me know if you need more details.

@jimmyjones2 so by no results here, I am guessing you mean when results come back as zeros. Currently, zeros are included in the bucket count and so they are counted by the visualization and show up as a certain color.
I think this issue is the same as #36, so I will close this one and look into an option which can exclude zeros from the count.


How to enable dark mode in iFrame mode?

I'm starting a new project using iFrame mode, but dark mode is not working like in other projects without iFrame mode.
Is this the expected behavior?

For what I've tested locally, toggling dark mode does not affect the already opened tabs on the IFrame mode, but when you open a new tab, it inherits the dark mode state of the main body tag. Is this the problem you're facing too?
If not, please add more details and screenshots explaining the problem.

Yes. This is my situation.
This way, the user will need to reopen all tabs every time he switches the dark mode.

Ok, I will try to find a solution for this...

Thank you!

@ruanpepe please, try to check the fix on the new version of the package

Everything working well now.
Thank you!


Allow a node to store its different databases on different volumes

Using the AWS current instances and their configuration makes it impossible to guarantee reproducible system-level benchmark metrics, due to the machines using virtualized I/O. This prevents us from running system-level benchmarks for the LMDB backend.
By running the node on instances with small SSDs (eg m5ad.large or x2gd.large) we could solve this problem, however to lower the disk requirements, it would be better if only the ledger DB would live on an SSD volume, while the chain DB would live in a bigger and slower volume.
Furthermore, SPOs would be interested in this feature since using a configuration with a small local SSD + big slower volume will be cheaper in terms of hardware.
Note that it's currently possible to make this using symlinks, but we could make this easier. Also, the system-level benchmarks start without any folder for the chain DB and let the node create it, so symlinking would make the benchmarking setup more involved.
We should add support (in Consensus) for configuring the volumes where the ledger DB and chain DB live.

This is somewhat implemented in the UTXO-HD branch, although not for general use.


Dad Joke

Whenever someone says something like "I am hungry", the bot will respond with "Hi hungry, I am [acm_bot]!", or something similar.
This would need to scan every new message in every channel, as opposed to working with a slash command. I'm not sure how to do do this, but it is probably possible.

Hi! I am very new to oss. Can I work on this as a first project?

@pizzanapoletana yeah sure! Are you in the acm discord? We will talk more about the project and how to get started in the #discord-bot channel so stay tuned

@JaydenPahukula hey there. I am extremely sorry for the late. I was tied up in several university projects and couldn't get back to you. Really, really sorry about it. I am not in the acm discord. And I am not even a student of Colorado School of Mines. If that's fine, I would love to join the discord and work with you. Again, I am extremely sorry.

@pizzanapoletana Hey its all good. This project is mostly for fun, so don't worry if you get too busy or something, there's no hard contribution requirements. Here is the link to the Mines ACM discord, we'll communicate in the discord-bot channel but feel free to hang out and stuff!

@JaydenPahukula hey threre. Discord says the link is invalid. :(

@pizzanapoletana oops try this

Thank you! That worked, and I am in the discord now. :)


CTE support

Hey guys, are there any plans to work on CTE (common table expressions) support?
They are a part of SQL standard, supported by most popular RDBM and their usage is widely considered to be best practice when there's a need for multiple (potentially nested) subquieries.

Thanks for the suggestion @fuyfuy . I think this can be handled by allowing users defining relations based on queries (in some sense like a macro). What do you think @stechu ?

@fuyfuy Thanks for your suggestion. I think we can include CTE in the next version of Cosette.

Thanks a lot for! Cosette is very interesting project, keep up the great work!


Merge-like creationPolicy that sets ownerReference

Describe the solution you'd like
Update the creationPolicy=Merge (or create a new policy) that also sets the ownerReference on the managed secret.
What is the added value?
This would be useful for bootstrapping the Secret used by a SecretStore, and having external-secrets automatically syncronise changes to the secret after the initial secret creation.
This is possible with the Merge strategy already, however my GitOps tooling (argocd) is attempting to self-heal (converge on declared state) and clean up the secret as without the ownerReference metadata it does not understand the Secrets relationship to the ExternalSecret resource.
Give us examples of the outcome
I think this could be achieved by extending the check in the mutationFunc in pkg/controllers/externalsecret/externalsecret_controller.go to also set apply owner fields for the merge creationPolicy.
Observations (Constraints, Context, etc):
The initial secret is being created directly with kubectl create secret generic ....

Hi @dowlingw ! Have you tried using cretionPolicy=Owner? IIRC, it will take ownership of secrets with no ownerReference setup.

Hi @dowlingw ! Have you tried using cretionPolicy=Owner? IIRC, it will take ownership of secrets with no ownerReference setup.

Oh! I hadn't actually tried that - the documentation suggests other behaviour:

Owner (default)
The External Secret Operator creates secret and sets the ownerReference field on the Secret. This secret is subject to garbage collection if the initial ExternalSecret is absent. If a secret with the same name already exists that is not owned by the controller it will result in a conflict. The operator will just error out, not claiming the ownership.

Testing quickly and it appears that you're absolutely right. I'll PR a documentation fix shortly.

I will change the issue name then so people can refer to it in a better way!


Runic, Ender Pearl reset to 0

I tried to have a lot of runic and enderpearl, so i watched your video, everything worked fine, but when i change the value of my runic, i change to 0 ! I don't know, if a update broke that...

You have to turn off the cheat afterwards and close the Cheat Engine.
It will only resets to 0 when you turn off the game with the cheat enabled.

I know this post is a bit old but I have tried to follow this exactly to the letter and as the OP stated,  All that ever happens is the value goes to 0.   Meaning... While following the steps exactly as stated.... the Ingame Value goes to 0 before activating the script and at that point,  its not recoverable.
So regardless of what ever value we put into the fields, and try to do as the steps instruct,  something nukes the ingame value before activation.  I cant explain it but ive attempted to do this about 20 times now and i am either missing completely minor or something is missing in the instructions,  or they patched the ability to do this out?
Can somebody take a look back into this possibly?   The ONLY thing that works for me on this CT is the Pickaxe Zero Cooldown.  Nothing else works.
The Game version i have is  5.9.28 if that helps.

I know this post is a bit old but I have tried to follow this exactly to the letter and as the OP stated,  All that ever happens is the value goes to 0.   Meaning... While following the steps exactly as stated.... the Ingame Value goes to 0 before activating the script and at that point,  its not recoverable.
So regardless of what ever value we put into the fields, and try to do as the steps instruct,  something nukes the ingame value before activation.  I cant explain it but ive attempted to do this about 20 times now and i am either missing completely minor or something is missing in the instructions,  or they patched the ability to do this out?
Can somebody take a look back into this possibly?   The ONLY thing that works for me on this CT is the Pickaxe Zero Cooldown.  Nothing else works.
The Game version i have is  5.9.28 if that helps.

Ok, I just figured out how to manually insert Dust and Runic.
Follow the steps to get to Jit and get the value...  Copy the Value and then look for other methods.

Right click this and Excute/Invoke Method.  In the box that pops up,  put the address in from the earlier step, and then enter a value and hit ok.    That will update your value ingame.
Same principal for all of them.  Find the code as instructed in the script info,  and then find the Award options and then copy/pasta enter your data and then give it a value and hit ok.

Ok, I just figured out how to manually insert Dust and Runic.
Follow the steps to get to Jit and get the value...  Copy the Value and then look for other methods.

Right click this and Excute/Invoke Method.  In the box that pops up,  put the address in from the earlier step, and then enter a value and hit ok.    That will update your value ingame.
Same principal for all of them.  Find the code as instructed in the script info,  and then find the Award options and then copy/pasta enter your data and then give it a value and hit ok.


Republishing Video also causes audio to be unpublished and republished

Version of the agora_rtc_engine
6.3.0
Platforms affected

[ ] Android
[ ] iOS
[ ] macOS
[ ] Windows
[X] Web

Steps to reproduce
My understanding is the only way to unpublish video (and turn off hte camera is to do the following)
engine.enableLocalVideo(false);
But to renable it you have to do:
await engine.enableLocalVideo(true);
await engine.updateChannelMediaOptions(ChannelMediaOptions(
  publishCameraTrack: true,
));

However, when I do this it also turns off and on the audio.
I believe the code that toggles both of them is here:
https://github.com/AgoraIO-Extensions/iris_web/blob/dc46c16fbbe2871a19cdc56453f12f5ff9f03093/packages/rtc/src/helper/ImplHelper.ts#L359
Expected results
It only toggles on the video and does not effect the audio.
Actual results
Both audio and video are unpublished and then republished.
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.19.3, on macOS 14.1.1 23B81 darwin-arm64, locale en-US)
[!] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.
[✗] Xcode - develop for iOS and macOS
    ✗ Xcode installation is incomplete; a full installation is necessary for iOS and macOS development.
      Download at: https://developer.apple.com/xcode/
      Or install Xcode via the App Store.
      Once installed, run:
        sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer
        sudo xcodebuild -runFirstLaunch
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To install see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.
[✓] Chrome - develop for the web
[✓] Android Studio (version 2023.2)
[✓] VS Code (version 1.84.2)
[✓] VS Code (version 1.86.0-insider)
[✓] Connected device (2 available)
[✓] Network resources

I think you can try muteLocalVideoStream to unpublish or re-publish the video stream.


Location of custom.css

It looks like we have to change the location of custom.css since it becomes impossible to access it when using an add-on. There are two ways we could go about it:
a) Change the location and make a relevant note about breaking change
b) Link it as an additional file
@rchl What do you think?

Are you referring to this issue https://github.com/resoai/TileBoard-addon/issues/12 ?
Because I think that can be fixed in the addon without changing the location.

Yes, I'm referring to this one. I'm sorry I missed that, all working as expected!


"tag too large to fit into the unused bits" on aarch64

failures:
---- src/epoch/atomic.rs - epoch::atomic::Owned<T>::with_tag (line 598) stdout ----
	thread 'rustc' panicked at 'test executable failed:
thread 'main' panicked at 'tag too large to fit into the unused bits: 5 > 3', src/epoch/atomic.rs:66:4
note: Run with `RUST_BACKTRACE=1` for a backtrace.
', src/librustdoc/test.rs:315:16
note: Run with `RUST_BACKTRACE=1` for a backtrace.
---- src/epoch/atomic.rs - epoch::atomic::Ptr<'scope, T>::tag (line 833) stdout ----
	thread 'rustc' panicked at 'test executable failed:
thread 'main' panicked at 'tag too large to fit into the unused bits: 5 > 3', src/epoch/atomic.rs:66:4
note: Run with `RUST_BACKTRACE=1` for a backtrace.
', src/librustdoc/test.rs:315:16
---- src/epoch/atomic.rs - epoch::atomic::Ptr<'scope, T>::with_tag (line 851) stdout ----
	thread 'rustc' panicked at 'test executable failed:
thread 'main' panicked at 'tag too large to fit into the unused bits: 5 > 3', src/epoch/atomic.rs:66:4
note: Run with `RUST_BACKTRACE=1` for a backtrace.
', src/librustdoc/test.rs:315:16
failures:
    src/epoch/atomic.rs - epoch::atomic::Owned<T>::with_tag (line 598)
    src/epoch/atomic.rs - epoch::atomic::Ptr<'scope, T>::tag (line 833)
    src/epoch/atomic.rs - epoch::atomic::Ptr<'scope, T>::with_tag (line 851)

Happens only on aarch64, armv7hl/i686 fail due to #17 and s390x/ppc64/ppc64le/x86_64 just work.

It looks like the problem is that mem::align_of::<u64>() equals 4 instead of 8 on aarch64. I'll fix this.
May I ask where you got all these different architectures to test the library on? I'd be interested in running the tests myself. :)

@stjepang that's Fedora Project builders... I'm afraid that you can't easily get access because that would require to becoming package maintainer in Fedora.. But you can anytime ask me to run tests there, I would be happy to do so.

Just FYI, this problem appeared on i686 as well... https://kojipkgs.fedoraproject.org//work/tasks/4654/23044654/build.log
---- src/epoch/atomic.rs - epoch::atomic::Owned<T>::with_tag (line 598) stdout ----
	thread 'rustc' panicked at 'test executable failed:
thread 'main' panicked at 'tag too large to fit into the unused bits: 5 > 3', src/epoch/atomic.rs:66:4
note: Run with `RUST_BACKTRACE=1` for a backtrace.
', src/librustdoc/test.rs:315:16
note: Run with `RUST_BACKTRACE=1` for a backtrace.
---- src/epoch/atomic.rs - epoch::atomic::Ptr<'scope, T>::tag (line 833) stdout ----
	thread 'rustc' panicked at 'test executable failed:
thread 'main' panicked at 'tag too large to fit into the unused bits: 5 > 3', src/epoch/atomic.rs:66:4
note: Run with `RUST_BACKTRACE=1` for a backtrace.
', src/librustdoc/test.rs:315:16
---- src/epoch/atomic.rs - epoch::atomic::Ptr<'scope, T>::with_tag (line 851) stdout ----
	thread 'rustc' panicked at 'test executable failed:
thread 'main' panicked at 'tag too large to fit into the unused bits: 5 > 3', src/epoch/atomic.rs:66:4
note: Run with `RUST_BACKTRACE=1` for a backtrace.
', src/librustdoc/test.rs:315:16
failures:
    src/epoch/atomic.rs - epoch::atomic::Owned<T>::with_tag (line 598)
    src/epoch/atomic.rs - epoch::atomic::Ptr<'scope, T>::tag (line 833)
    src/epoch/atomic.rs - epoch::atomic::Ptr<'scope, T>::with_tag (line 851)

I've just published a new version that should fix the problem.


How to get email address of user?

I integrate this library successfully. now after user authentication now how to get email address of user ? please help.

You can't.
https://twittercommunity.com/t/how-to-get-email-is-using-twitter-api-on-callback-page-using-php/404

You can https://dev.twitter.com/rest/reference/get/account/verify_credentials

Request a User’s Email Address
Requesting a user’s email address requires your application to be whitelisted by Twitter. To request access, please use this form.
Once whitelisted, the “Request email addresses from users” checkbox will be available under your app permissions on apps.twitter.com. Privacy Policy URL and Terms of Service URL fields will also be available under settings which are required for email access. If enabled, users will be informed via the oauth/authorize dialog that your app can access their email address.
Note
Your app will need to regenerate the user access tokens for previously authenticated users to access their email address.

However, I've found a bug. I've tested getAccountVerifyCredentialsWithIncludeEntites and it works only when the include_email query parameter value (the actual value sent to the verify_credentials endpoint) is set to the value: "true", NOT "t" or "1"  See #241


Update java-jwt to 4.0.0

Updates com.auth0:java-jwt from 3.19.2 to 4.0.0.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.auth0", artifactId = "java-jwt" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.auth0", artifactId = "java-jwt" }
}]

labels: library-update, early-semver-major, semver-spec-major, commit-count:1

Superseded by #123.


Add prevval/nextval to loop context

This should solve the problems related to not being able to keep state across loop iterations (e.g. #641) once and for all.

Any better suggestions for the names of these attributes?
My first thought was prev and next, but that could be confusing since there are already first and last which are booleans.

Maybe previtem and nextitem?

Full disclaimer, I'm relatively unfamiliar with Jinja internals.

This should still go in even if we go ahead with the namespace idea too. It's a simpler solution to probably the majority of cases.

Maybe we should move this to 2.10 too... doesn't make much sense to have #684 in 2.10 and this one in 2.9

I'm fine with that. Can we just release 2.10 now?

For an immediate solution, I created two custom Jinja2 template functions to get/set globals on flask.g. It persists variables across loop iterations:
# Getter for Flask.g request global
def get_global(key):
    return flask.g.get(key)
app.jinja_env.globals.update(
    GET_GLOBAL=get_global,
)

# Setter for Flask.g request global
def set_global(key, val):
    setattr(flask.g, key, val)
    return ''  # return empty string to prevent modifying template
app.jinja_env.globals.update(
    SET_GLOBAL=set_global,
)

Use in your template like:
{{ SET_GLOBAL('myvar', 'someval') }}
{{ GET_GLOBAL('myvar') }}

@ThiefMaster can this be released with 2.10? I'm really looking forward to this feature in the stable release. thanks!

Yes, it will be in 2.10


Drag and drop: Safari iOS 15

Right now HTML5 drag and drop is called out as not working on iOS Safari 15: https://caniuse.com/dragndrop
However, I tested on my own iOS device running Safari 15 and it does appear to be working

Where did you test? :)

https://tests.caniuse.com/?feat=dragndrop
On iOS 15 Safari

Ah, you responded while I was writing this comment - thanks! Checked as well, will create the PR now! Thanks! :)

Huh! Never mind, I was asking because https://bestvpn.org/html5demos/drag/ was kinda wonky, but would you look at that:
https://bugs.webkit.org/show_bug.cgi?id=223596
https://tests.caniuse.com/dragndrop =>
https://user-images.githubusercontent.com/2644614/140818697-d1dd9751-87ad-4bd6-9767-f084dc5a61d7.MP4

According to https://bugs.webkit.org/show_bug.cgi?id=223596 the feature has now been implemented. The bug report calls out it was fixed in Safari 14; but I cannot test that. It looks like it was launched with iOS 15 to support it's new cross app drag and drop experiences

This HTML5 wrapper library is also working nicely on iOS 15 Safari: https://lukasoppermann.github.io/html5sortable/index.html

https://user-images.githubusercontent.com/2182637/140850865-7de8e103-84b5-4e56-a9a7-558e71944ac3.MOV


Create React App builds empty document (blank screen) with react-router

Hi!
I started creating an app with create-react-app and using react router and  firebase.
Everything was working fine 2 days ago, and since yesterday it just shows me a blank screen, but I didn't change anything.
I have been looking everywhere on my app and in internet but I don't find the problem.
below is my App file and my package.json file.
return (             <Router basename='/numbr'>                 <Route exact path={ROUTES.SIGN_IN} component={ConnectScreen} />                 <Route path={ROUTES.HOME} component={HomeScreen} />             </Router>         );
Thanks in advance for your help!

do you have a link to the app?

@bugzpodder I just created a github repo, it s called Numbr.
I took of the firebase.js file because I had all my private config in it.
Thanks in advance for your help!

#7486


why can we use Bernoulli distribution in decoder?

hello, thanks for sharing this code. I am confused about a question. I find MNIST data are continuous(pixel value not just 0 and 1), so why can we use Bernoulli distribution in decoder?

Binary-cross entropy measures the distance between an input and target, including as probabilities. The sigmoid activation squashes the image values between 0 -1, which, as far as I know, means they can then be interpreted as probabilities. Hence, cross-entropy can be used between the output and the target values.

@staturecrane thanks for your reply. I know binary cross-entropy can measure the distance. I just confused about the loss implemented in this code(line 65~70) which I think is not the same with the loss in the paper(https://arxiv.org/pdf/1312.6114v10.pdf).
In this code, the loss is
KLD = -0.5 * tf.reduce_sum(1 + logvar_encoder - tf.pow(mu_encoder, 2) - tf.exp(logvar_encoder), reduction_indices=1)

x_hat = tf.matmul(hidden_decoder, W_decoder_hidden_reconstruction) + b_decoder_hidden_reconstruction
BCE = tf.reduce_sum(tf.nn.sigmoid_cross_entropy_with_logits(x_hat, x), reduction_indices=1)

loss = tf.reduce_mean(BCE + KLD)

and in the paper, the loss is

The KLD is the same with the first term of right, and when p(x|z) is Bernoulli distribution, it is true that the BCE is the same with the second term of right. When p(x|z) is Gaussian distribution, the BCE is not the same. However, only when x is a discrete variable, can we use Bernoulli distribution to model p(x|z). So my question is, what exactly the distribution of p(x|z) is in this code when x is a continuous variable, or do I misunderstand the code or the paper?
the Bernoulli MLP in the paper is

So most MNIST pixels are actually 0 or 1, so it is approximately binary. There are fully binarized versions of mnist (by Hugo Larochelle) which indeed given a better score when trained on.
In general this is so close to being binary that it should be ok. It definitely works better than when trained using the continuous loss function.
FYI in the paper also BCE is used for the MNIST experiments

I know I'm late to the party, but this paper just got publish and provide some important conclusions:
The continuous Bernoulli: fixing a pervasive error in variational autoencoders
https://arxiv.org/pdf/1907.06845.pdf

sigmoid activation function make the values by 0 to 1


optimize ci computation for build job

Only run first and last supported python versions.
First run 1 configuration before firing up other machines.

I think this will do for now. I'll improve/simplify the job definition later, as soon as I learn how.


Problems with from future import annotations and relay

The novel relay module is not compatible with  from __future__ import annotations.
I build a simple demo:
https://github.com/devkral/graphene-protector/tree/demo/withresolve_bug (see first section how to reproduce)
Somehow the field cannot be correctly parsed from its string form.

System Information

Strawberry version (if applicable): 0.186.1

I experimented a bit. It didn't work for me :(.
The best way I think would be to overload the StrawberryResolver signatures to resolve automatically the type when accessed.
I tried this but it still failed, something is missing

@devkral does this still happen for you?


IdentityServer4 on docker

Hi,
My application is in .NET core 2.0. I want to create a centralized A&A server that can connect to an Azure LDAP and perform Authentication and also Authorization. I would want this microService to be in .NET core 2.0 in a Docker container
So the ask is

Is there an official docker image provided by the creators for identityServer 4
Do you have an .NET 2.0 standard compliant IdentityServer4
If there are no docker images then can you please provide some guidance for creating the same.

Thanks

Given the nature of IdentityServer, a base image would not make sense. Your base image would be the ASP.NET Core 2.0 image where you "deploy" IdentityServer to.

@leastprivilege,
I'm still not able to understand what you are saying. Could you please clarify what you mean by saying "Given the nature of IdentityServer, a base image would not make sense". My requirement is I need the Identity server to be able to authenticate application that are running out of docker. The identity server should also run as a docker container. So having said this it would be great if you can guide me to sample which does this end to end or some links that will help me do this

I think you need to learn Docker first.
https://docs.microsoft.com/en-us/aspnet/core/publishing/docker

@leastprivilege,
Thanks for the link. In fact I have attend docker training in Weighbee and have deployed an API app and Postgress in two different container and have linked them using compose and put in on swarm. By point is I want to understand that what is that nature of identity server that prevenst it from putting it in docker

Nothing. It is just a library inside a normal ASP.NET Core application.

@leastprivilege,
So that means we need some host process like ASP.NET core to use this library and host that app in container is it? If that is case do we need any special consideration like subnet mask that we need to take care? DO we have some guidance on how to get this working on docker


SAK-49540 LTI add site navigation placement for tools

This could use a little review from Teaching and Learning folks.  I would like to talk about this with Earle, Wilma, and Earle.

This is ready to go.


Writeup: Logging options

Write a summary of logging options for dnb-hugo elements in dogmata/logging.md

This should be done in https://github.com/dnb-org/components/functions and https://github.com/dnb-org/debug


Server crashed when using alloy furnace

Modpack: Sky Factory iiii
Mod Version: NuclearCraft-2.16f-1.12.2
Forge Version: forge-<IP_ADDRESS>31
Crashlog: https://pastebin.com/6dUPumtz
Unknown reason as it happened on a server and I don't know who did it or how. Let me know if you need more details and I'll try to find it.

Looks like you're using 2.16e - update to 2.16f and that'll fix it ;)

you know what... you're right. I looked at the wrong name. My bad


Know issues with installation on windows?

Question

I tested IBM Docling in an Google Colab Notebook and it works great. With the same code on a windows pc i got only text output and not markdown formated output. Maybe there is some issue with other libaries. Could you help?

Interesting question. We are not aware of any difference expected between Linux and Windows.
Can you please get the exact version of Docling on Colab and Windows?
docling --version

@janschachtschabel If you can provide updates on the versions of docling used in either case, we could follow up. I am closing this until further evidence is provided.


chore: upgrade cypress to use retries

SUMMARY
upgrades cypress to v 5.0 in order to use the new retries features. Also added back a known flaky offender to see if there's any improvement.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN
cypress tests pass, with potentially some retries.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

retrigger FOSSA check.

@eschutho I think you just need to rebase your branch to include in the latest fixes. I don't think CI is configured to run with a merge strategy.

explore/AdhocFilters.test.ts failed too, but I disabled that one on master, rebasing should fix that issue.
Personally I'd advise to also disable the whole License Github action as #10925 can re-enable it as it lands.
We've been suffering from flaky test/broken build for too long now. I vote for disabling all that fails and start clean.

rerunning checks

rerun for FOSSA

@mistercrunch I'll continue to keep an eye on any tests that are failing after this and see which need to be fixed or disabled.


Common Layout Containers, and Composite with Sidebar

Description
This PR have the new common layout containers, these serve as a base that other layout containers can be built from.
The LayoutWithSidebar is a composite component that demonstrates the use of the base containers.
Fixes #154 and #110
Type of change
Please delete options that are not relevant

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?
I tested this locally on the Styleguide. There are sections in the styleguide where the components are documented with examples. Where applicable, I have toggle buttons to demonstrate certain dynamic functionality.
Finally, I also wrapped the entire styleguide page content with the new LayoutWithSidebar component, and prevented this sidebar from being closed, as an example of how it can be used in this way.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I updated the CHANGELOG.md and added documentation if necessary
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have completed manual or automated accessibility testing for my changes
[ ] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Hi @andrewebdev I confirmed that we do need a separate slider component that is separate from regular right column content which always shows. I think the regular right column content will be pretty simple, using something like "flex" to just wrap the content below the main content.

@jupiter007 , @fraserclark I've completed the updates can we possibly arrange a review meetup so I can run through the changes and elements, before you do a code review?

Thanks @andrewebdev. I'll take a look first thing Friday.

@jupiter007 , @fraserclark I've completed the updates can we possibly arrange a review meetup so I can run through the changes and elements, before you do a code review?

Hi @andrewebdev. Let's plan on going over it on Tuesday along with the other PRs.

@andrewebdev thanks for the updates for the sidebar and drawer. They look good. I just had a few comments on your PR.

Hi @andrewebdev. Do you have all your updates checked in? If so, I'll take another look before approving. Also, it looks like you need to merge the latest changes from development into your branch.

Hi @andrewebdev. Do you have all your updates checked in? If so, I'll take another look before approving. Also, it looks like you need to merge the latest changes from development into your branch.

Just need to fix that last bug with the weird tab behaviour. I'll ping you as soon as that's checked in.

@andrewebdev thanks for making all the updates. The Drawer appears to be working as expected now. However, I tried to run npm run build and it encountered some errors. It's always a good idea to run that command, because it catches some issues that people might miss.
I'll try and add some comments where I saw some errors.

@andrewebdev I approved your PR. It looks like you addressed all the comments.


Issues with adding patients to a call list from urgent pregnancies

@lwaldsc and @NerdyGirl537 had some issues getting patients to add to the call list properly, especially from the urgent pregnancies form. Here are the specific issues they ran into and steps to troubleshoot from.

Adding pt from call search does not add, only replaces:
URL: dashboard
Thing I don't like: Adding from the urgent cases does not properly populate the call log
Thing I would like better: Adding from either urgent cases or search properly appends to the call log

Call list is not maintained when returning to main screen from PT screen from link at top left:
URL: edit page (to dashboard)
Thing I don't like: When return to call list from PT info, urgent case has fallen off call list
Thing I would like better: Keep urgent calls on call list when returning to main screen

I've been thinking about this and I wonder if this is an issue with the relationship btwn pregnancy and user being has_many : belongs_to? Should this be something like has_many : belongs_to_many ?

Bumping this into sprint 5.

Want to confirm that my guess was right: a patient can only be on one person's call list at a time. If user one has patients a and b on a call list, and user two adds patient b to theirs, patient b disappears from user ones call list.
Won't be an issue for early stage stuff since the mvp will be single user, but we will Def need to resolve. E

Will try this guy


New code to copy booted kernel prior to applying update

Just the implementation, not yet wired into the update procedure itself.

LGTM; hooray new code, but man that's a lot of templates to copy a file ;)

@barakmich yeah, the underlying design of this code is an attempt to port an Objective C app built around a pipeline model. The awkwardness of translating that pipeline verbatim without re-imagining it was bad enough, but then so many holes got punched in the abstraction to deal with status updates and so on it really fell apart.
http://googlemac.blogspot.com/2008/09/revving-software-with-update-engine.html
https://code.google.com/archive/p/update-engine/wikis/ActionProcessor.wiki


Office 2013 does not require app passwords

It says 'Older email applications like Office 2013 need app passwords', but Office 2013 does support Modern Authentication with registry changes (https://docs.microsoft.com/en-us/office365/admin/security-and-compliance/enable-modern-authentication?view=o365-worldwide)
Suggest changing this to 'Older email applications like Office 2010 need app passwords.' and maybe linking to above article referencing how to enable for Office 2013.

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

ID: 3ed95865-eddb-4ad1-d2e2-5bd618151c1b
Version Independent ID: ef32bb26-e78a-caad-5b41-d7e49f6ef9b4
Content: Set up multi-factor authentication for Office 365 users
Content Source: Office365-Admin/security-and-compliance/set-up-multi-factor-authentication.md
Service: o365-administration
GitHub Login: @Sirkkuw
Microsoft Alias: sirkkuw

@officedocsbot assign @mypil

@ChrisAtMAF - Thank you for submitting feedback.
I will get this issue over to the O365ITPro writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@ChrisAtMAF Thank you for your feedback. We have now added a note which has information on how to enabled Modern auth on 2013. The same can be tracked via PR https://github.com/MicrosoftDocs/OfficeDocs-O365ITPro/pull/674
Regards
Sri

@ChrisAtMAF - From our understanding, the issue has been resolved but it may take a few days for the merged content to appear in the article. If you feel it hasn't been resolved, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!

@officedocsbot close