fs/rpmsgfs: return ENOTTY to vfs to do cmd operate

Summary
and Revert "rpmsgfs: cache statfs result in case of deadlock"
Impact
rpmsgfs
Testing
Xiaomi Watch S1 Pro

Please explain a little bit about the issue caused by that cache statfs, why the deadlock was happening?

Please explain a little bit about the issue caused by that cache statfs,

Since statfs will report the free space, caching the result will make report inaccurately in a long run.

why the deadlock was happening?

Here is a callstack:
    rptun: msg from other cpu
    [ap] [ 5] [<0x2c353768>] arm_switchcontext+0xc/0x10
    [ap] [ 5] [<0x2c322266>] sem_wait+0x5a/0xbc
    [ap] [ 5] [<0x2c7400a8>] inode_semtake+0x24/0x5c
    [ap] [ 5] [<0x2c74087a>] inode_release+0x6/0x60
    [ap] [ 5] [<0x2c741284>] file_close+0x18/0x44
    [ap] [ 5] [<0x2c74fb06>] rpmsgfs_close_handler+0x86/0xe4
    [ap] [ 5] [<0x2c75b8de>] rpmsg_virtio_rx_callback+0xba/0x1b4
    [ap] [ 5] [<0x2c75af70>] rproc_virtio_notified+0x44/0x5c
    [ap] [ 5] [<0x2c75a93c>] remoteproc_get_notification+0x1c/0x2c
    [ap] [ 5] [<0x2c335a08>] rptun_thread+0x74/0x208
    [ap] [ 5] [<0x2c323e72>] nxtask_start+0x3a/0x60

    nsh: send msg to other cpu with inode lock
    [ap] [10] [<0x2c353768>] arm_switchcontext+0xc/0x10
    [ap] [10] [<0x2c3221b2>] nxsem_wait_uninterruptible+0x6e/0xc8
    [ap] [10] [<0x2c74e498>] rpmsgfs_send_recv.constprop.0+0x78/0xc0
    [ap] [10] [<0x2c74ebc0>] rpmsgfs_client_statfs+0x50/0x80
    [ap] [10] [<0x2c3221ca>] nxsem_wait_uninterruptible+0x86/0xc8
    [ap] [10] [<0x2c74da78>] rpmsgfs_statfs+0x28/0x50
    [ap] [10] [<0x2c74534e>] mountpoint_filter+0x66/0x90
    [ap] [10] [<0x2c740016>] foreach_inodelevel+0x66/0x88
    [ap] [10] [<0x2c73fffa>] foreach_inodelevel+0x4a/0x88
    [ap] [10] [<0x2c740062>] foreach_inode+0x2a/0x48
    [ap] [10] [<0x2c745388>] foreach_mountpoint+0x10/0x20
    [ap] [10] [<0x2c745606>] mount_read+0x36/0x70
    [ap] [10] [<0x2c742a34>] read+0x30/0x5c
    [ap] [10] [<0x2c371096>] nsh_catfile+0x36/0x140
    [ap] [10] [<0x2c36d01a>] nsh_parse_command+0x7fe/0xc54
    [ap] [10] [<0x2c370f92>] nsh_session+0x66/0x134
    [ap] [10] [<0x2c3721a4>] nsh_consolemain+0x2c/0x44
    [ap] [10] [<0x2c374fde>] nsh_main+0x2a/0x4c
    [ap] [10] [<0x2c33f7f8>] lib_cxx_initialize+0x24/0x48
    [ap] [10] [<0x2c34274a>] nxtask_startup+0x12/0x24
    [ap] [10] [<0x2c323e7e>] nxtask_start+0x46/0x60

Bascally, the global inode lock introduce the dead lock. We need find a better way to fix the dead lock.


Running the tests with pytest 4 leads to en error like the following:
request = <SubRequest 'pdep_manager' for <Function 'test_get_set[JsonDB]'>>

    @pytest.fixture(params=[JsonDB, DbmDB, SqliteDB])
    def pdep_manager(request):
>       return dep_manager(request)
E       _pytest.warning_types.RemovedInPytest4Warning: Fixture "dep_manager" called directly. Fixtures are not meant to be called directly, are created automatically when test functions request them as parameters. See https://docs.pytest.org/en/latest/fixture.html for more information.

tests/test_dependency.py:71: RemovedInPytest4Warning

In this case the tests have been run with pytest==4.0.2.

After backporting the fixes in 08e9c0b, I'm seeing the following fixture related error:
platform freebsd13 -- Python 3.6.8, pytest-4.5.0, py-1.8.0, pluggy-0.11.0 -- /usr/local/bin/python3.6

ERROR collecting tests/test_dependency.py
__________________________________________________________________
tests/test_dependency.py:72: in <module>
    pytest.fixture(params=[JsonDB, DbmDB, SqliteDB])(pdep_manager)
/usr/local/lib/python3.6/site-packages/_pytest/fixtures.py:986: in __call__
    "fixture is being applied more than once to the same function"
E   ValueError: fixture is being applied more than once to the same function

Related issues:
https://github.com/pytest-dev/pytest/issues/3518
https://github.com/kevin1024/pytest-httpbin/issues/49
https://github.com/pytest-dev/pytest/issues/2334


Backup content is not enclosed

Hello,
While making a backup form the database the content is not enclosed by '.
Example:
INSERT INTO tr_acties (id, naamKort, naamLang, kleur, afb) VALUES (1, bellen, Bellen, #6BBE23, telefoon.png);
Instead of
INSERT INTO tr_acties (id, naamKort, naamLang, kleur, afb) VALUES ('1', 'bellen', 'Bellen', '#6BBE23', 'telefoon.png');
This is seen in version 3.1.6 as in the development version of 27-09-2017.
Greetings,
Ubel Jan van Wijhe

What PHP and MySQL versions is that on?

Hello,
The first system is with PHP Version 5.5.9-1ubuntu4.22 and MySQL server versie | 5.5.57-0ubuntu<IP_ADDRESS>
The second system is a Centos system with PHP 5.6.31 and MySQL mysqlnd 5.0.11-dev.
The fields you saw were varchar fields.
Greetings,
Ubel Jan van Wijhe

Thanks. Though ... that turned out to be irrelevant; I found the issue already.

Was caused by the patch for #5260. See referenced commit for the fix.

Hello,
I have create a database backup using codeigniter $this->load->dbutil(); This Work fine and I can download my database backup. But I try to restore this backup and I got below error..
SQL query:
INSERT INTO app_user (id, user_name, email, password, role, status) VALUES (10, Admin.sgh<EMAIL_ADDRESS>202cb962ac59075b964b07152d234b70, 0, 0)
MySQL said: Documentation
#1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near<EMAIL_ADDRESS>202cb962ac59075b964b07152d234b70, 0, 0)' at line 1
Can you help me to resolve this error. I am using codeigniter 3.1.6 Version..

Any one know how to resolve this error?

Any one know how to resolve this error?

@sanjayacloud you can upgrade your CI to the last stable version from this github, then database backup works properly.


docs(migration): add enhancements and improvements

Preview environments are getting posted here, as soon as they are ready!


Create a custom 404 page

Is your feature request related to a problem? Please describe.
When users encounter a broken link or mistyped URL on the Quizard website, they are redirected to a generic 404 error page. This can be confusing and frustrating for users who may have been looking for specific information on the site.
Describe the solution you'd like
I would like to request the implementation of a custom 404 error page for the Quizard website. This page should clearly communicate to users that the page they were looking for cannot be found and provide helpful suggestions for navigating the site and finding the information they need. The page should also include branding elements and design consistent with the rest of the site.
Describe alternatives you've considered
One alternative solution could be to leave the generic 404 error page in place. However, this would not give users the helpful guidance and direction a custom page could offer.
Additional context
A custom 404 error page would not only improve the user experience on the Quizard website, but it could also help to reduce the bounce rate and improve overall site engagement.

The design team might need to work on this first

 Create a 404 error page


Cannot open Actions workflow run (SyntaxError: Unexpected token 'V', "Verify " is not valid JSON)

Description
Dear Developers,
Sincere, thank you very much for the incredibly marvelous... ineffably magnificent project... ✨

Encountered the following issue which currently does not allow viewing any ran workflows. With a page https://gitea.example.com/org/repo/actions/runs/123 opened and a screen (e.g. attached preview Picture 1).
During the page loading, I see in network request logs a GET request to  https://gitea.example.com/org/repo/actions/runs/123/artifacts that receives a response with HTTP status 401 Unauthorized and body:
Verify

Eventually, an exception appears in Console (e.g. attached preview Picture 2):
SyntaxError: Unexpected token 'V', "Verify
" is not valid JSON

I would appreciate a suggestion whether it's an expected outcome or not to dig further.
Best and kind regards
Gitea Version
1.21.10
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
Picture 1

Picture 2

Git Version
2.39.2
Operating System
Docker CLI in Linux x64
How are you running Gitea?
Docker Compose (+ some local networking)
services:

  gitea:
    image: gitea/gitea
    restart: unless-stopped

    ports:
      - 3020:3000
      - 3021:22

    depends_on:
      db:
        condition: service_healthy

    volumes:
      - '/volume1/docker/data/gitea/data/:/data/'

  gitea-actions-runner-1:
    image: 'gitea/act_runner'
    restart: unless-stopped

    environment:
      GITEA_RUNNER_NAME: "$GITEA_RUNNER_1_GITEA_RUNNER_NAME"
      GITEA_RUNNER_REGISTRATION_TOKEN: "$GITEA_RUNNER_1_GITEA_RUNNER_REGISTRATION_TOKEN"
      GITEA_INSTANCE_URL: "$GITEA_RUNNER_1_GITEA_INSTANCE_URL"
      GITEA_RUNNER_LABELS: "$GITEA_RUNNER_1_GITEA_RUNNER_LABELS"
      CONFIG_FILE: '/app/config.yaml'

    volumes:
      - '/volume1/docker/data/gitea/data/actions/runners/runner-1/config/config.yaml:/app/config.yaml'
      - '/volume1/docker/data/gitea/data/actions/runners/runner-1/data/:/data/'
      - '/var/run/docker.sock:/var/run/docker.sock'

    depends_on:
      - gitea

  db:
    image: 'bitnami/postgresql:latest'
    restart: unless-stopped

    environment:
      POSTGRESQL_USERNAME: "$DB_USERNAME"
      POSTGRESQL_PASSWORD: "$DB_PASSWORD"
      POSTGRESQL_DATABASE: "$DB_DATABASE"

    volumes:
      - '/volume1/docker/data/gitea/db/data:/bitnami/postgresql/data'

    healthcheck:
      test: pg_isready -h <IP_ADDRESS> -d "$DB_DATABASE" -U "$DB_USERNAME"
      interval: 10s
      timeout: 5s
      retries: 10
      start_period: 10s

Database
PostgreSQL

Could you clean the web browser cache and try again?

It's coming from here, definitely some kind of bug as other users on Discord have also reported seeing pages with only this "Verify" text:
https://github.com/go-gitea/gitea/blob/0178eaec256a349371c75e582edd7fefca2085d0/routers/web/web.go#L124


RequestContextExporter does not export a request ID.

Originally reported by @andrewoma
We could add a new enum value BuildInProperty.REQ_ID and export it into a req.id map entry.

Oh If no problem, let me handle this!


Formalize some SourceFile parsing outputs

Currently when parsing a SourceFile, the parser gets handed pointers so that it can write the interface hash and collected tokens directly into the file. It can also call setSyntaxRoot at the end of parsing to set the syntax tree.
In preparation for the removal of performParseOnly, this PR formalizes these values as outputs of ParseSourceFileRequest, ensuring that the file gets parsed when the interface hash, collected tokens, or syntax tree is queried.
In addition, it cleans up SourceFile a little by sinking a couple of parsing options into SourceFile::ParsingFlags.

@swift-ci please test

@swift-ci please test source compatibility

Build failed
Swift Test OS X Platform
Git Sha - 7b9ccb76d597b8184c18fe2b8c9ddb3f7717ff54

https://github.com/apple/llvm-project/pull/1307
@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 7b9ccb76d597b8184c18fe2b8c9ddb3f7717ff54

https://github.com/apple/llvm-project/pull/1307
@swift-ci please test macOS

Build failed
Swift Test OS X Platform
Git Sha - 7b9ccb76d597b8184c18fe2b8c9ddb3f7717ff54

https://github.com/apple/llvm-project/pull/1307
@swift-ci please test macOS

Build failed
Swift Test OS X Platform
Git Sha - 7b9ccb76d597b8184c18fe2b8c9ddb3f7717ff54

https://github.com/apple/llvm-project/pull/1307
@swift-ci please clean test macOS

Build failed
Swift Test OS X Platform
Git Sha - 7b9ccb76d597b8184c18fe2b8c9ddb3f7717ff54

https://github.com/apple/llvm-project/pull/1307
@swift-ci please test macOS

Build failed
Swift Test Linux Platform
Git Sha - 7b9ccb76d597b8184c18fe2b8c9ddb3f7717ff54

https://github.com/apple/llvm-project/pull/1307
@swift-ci please test Linux

https://github.com/apple/llvm-project/pull/1307
@swift-ci please test

https://github.com/apple/llvm-project/pull/1307
@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - db9a27e24a9fa54c126a43abdb4a936e4fd30fd5

https://github.com/apple/llvm-project/pull/1307
@swift-ci please test macOS

Merging so I can make further progress – @rintaro I'm more than happy to address any review feedback in a follow-up.


expose various examples for custom container

First, let me simply introduce our examples for custom container.

#assign-reviewer:xanwal

@dem108
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

I realized that I was creating the PR for our public repo. Let me close this PR and open another PR in the private repo.
https://github.com/MicrosoftDocs/azure-docs-pr/pull/215739

#please-close


Transcode videos with AWS MediaConvert

Pre-Flight checklist

[ ] Screenshots and design review for any changes that affect layout or styling

[ ] Desktop screenshots
[ ] Mobile width screenshots

[ ] Migrations

[ ] Migration is backwards-compatible with current production code

[ ] Testing

[ ] Code is tested
[ ] Changes have been manually tested

[ ] Settings

[ ] New settings are documented and present in app.json
[ ] New settings have reasonable development defaults, if applicable
[ ] Opened issue for DevOps regarding necessary configuration changes to deployed environments

What are the relevant tickets?
(Required)
What's this PR do?
(Required)
How should this be manually tested?
(Required)
Where should the reviewer start?
(Optional)
Any background context you want to provide?
(Optional)
Screenshots (if appropriate)
(Optional)
What GIF best describes this PR or how it makes you feel?
(Optional)

Codecov Report

Merging #469 (df46372) into mb/gdrive (5c4df02) will increase coverage by 4.72%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           mb/gdrive     #469      +/-   ##
=============================================
+ Coverage      90.52%   95.24%   +4.72%
=============================================
  Files            145       72      -73
  Lines           4980     1936    -3044
  Branches         737      379     -358
=============================================
- Hits            4508     1844    -2664
+ Misses           422       89     -333
+ Partials          50        3      -47

Impacted Files
Coverage Δ

static/js/util/factories/websites.ts
98.27% <0.00%> (-1.73%)
:arrow_down:

main/sentry.py

websites/constants.py

content_sync/api.py

main/serializers.py

content_sync/decorators.py

websites/apps.py

websites/permissions.py

websites/__init__.py

content_sync/serializers.py

... and 58 more

Continue to review full report at Codecov.

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


Updated example code to match preview

Updated example code to match what is shown in the preview

Thank you


Export XPUB as plain-text

Hi all,
playing with Samourai Sentinel I realized that it only accepts plain-text x/y/zpubs. The closest to this is in SeedSigner is the BlueWallet export, but the fingerprint and derivation part is not understood by Sentinel. How about adding a simple plain-text static QR export?
Cheers!

BTCPay server also had a problem with the [fingerprint/<derivation_path]xpub... format. But I believe that's been resolved. I think all the other coordinators we've tested (Specter, Sparrow, BlueWallet, Nunchuk) all accept this format.
So it might be better to prod the Samourai team to broaden their support to read and discard the prepended info to get them in line with what otherwise seems to be a consensus standard.


ENG-4028 Apply VoiceOver changes for notifying user about the first selected item in the list, message on navigating via the list

ENG-4028
ENG-3891

This looks good to me other than the removed class, it's good if we can have @ryanfs confirm this change is ok https://github.com/Interfolio/ui-select/pull/20/files#diff-eaec1deb563ae9f87f205ac5494465d0cfb5873c89d883ec91be4c9428f9cc57L1
as this is breaking the dropdown,
where as @ryanfs added those classes sometime back -> https://github.com/Interfolio/ui-select/commit/c298eb90b8492f01f23c576d2aa4bc5219012cdf#diff-eaec1deb563ae9f87f205ac5494465d0cfb5873c89d883ec91be4c9428f9cc57


Syntax error on RUN command

    RUN touch /etc/machine-id && \
        chmod 444 /etc/machine-id

And shell syntax error for
    RUN rm -rf /var/cache/* && \
        journalctl --vacuum-size=1K && \
        rm -rf /etc/machine-id && \
        rm -rf /var/lib/dbus/machine-id

I’m afraid I can’t reproduce this one. The problem may come from something else (I’m working on better isolate the errors BTW).
Could you try to reduce your file to a minimum reproducible example?
VERSION 0.8
foo:
    RUN touch /etc/machine-id && \
        chmod 444 /etc/machine-id

same for the second one
VERSION 0.8
foo:
    RUN rm -rf /var/cache/* && \
        journalctl --vacuum-size=1K && \
        rm -rf /etc/machine-id && \
        rm -rf /var/lib/dbus/machine-id

my entire Earthfile is at here

ummm its weird that i am not seeing this issue right now after a complete restart of vscode. I will reopen this issue if I see it again


Not compatible with @types/react:18.2.22

It was working fine till the last update from @types/react till this PR was merged in https://github.com/DefinitelyTyped/DefinitelyTyped/pull/66723
Error is:

TS2322: Type  Element  is not assignable to type  ReactNode
Property  children  is missing in type  Element  but required in type  ReactPortal

Tested with the example in the Readme.md and results with the same error.

Though, it could potentially be fixed with
yarn why @types/react yarn dedupe @types/react


Update slog and other dependencies

It's getting late and I need to do more maintenance work before release, but I'll be releasing slog-term 1.3.1 soon, and this PR will work then.

Done.


openconnect: enable Yubikey support

Description of changes
openconnect supports Yubikey hardware tokens for 2FA. To use them, it must be built with pcsclite and the configure output shows:
BUILD OPTIONS:
[...]
  Yubikey support:        yes

In can be used like below. The optional --token-secret argument specifies the name of the credential which is to be used.
$ openconnect --protocol=anyconnect -u user@name --token-mode=yubioath [--token-secret=name] server

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

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

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

[x] Fits CONTRIBUTING.md.

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

Sadly this broke macOS support:
  CC       libopenconnect_la-yubikey.lo
In file included from yubikey.c:64:
/nix/store/czlylyj49rl0bicqr9sq63qc9rar30fi-pcsclite-2.2.3-dev/include/PCSC/winscard.h:41:10: error: 'pcsclite.h' file not found with <angled> include; use >
#include <pcsclite.h>
         ^
In file included from yubikey.c:64:
In file included from /nix/store/czlylyj49rl0bicqr9sq63qc9rar30fi-pcsclite-2.2.3-dev/include/PCSC/winscard.h:41:
/nix/store/czlylyj49rl0bicqr9sq63qc9rar30fi-pcsclite-2.2.3-dev/include/PCSC/pcsclite.h:45:10: error: 'wintypes.h' file not found with <angled> include; use >
#include <wintypes.h>
         ^
2 errors generated.
make[1]: *** [Makefile:1540: libopenconnect_la-yubikey.lo] Error 1
make[1]: Leaving directory '/private/tmp/nix-build-openconnect-9.12.drv-0/openconnect-9.12'
make: *** [Makefile:922: all] Error 2


After upgrading to 3.19.5, I found a problem with page rebuild.

Steps to reproduce
After upgrading to 3.19.5, after page A jumps to page B, the life cycle of page A will first build and then dispose, which will not happen in version 3.13. The problem comes from:
https://mp.weixin.qq.com/s/h4iDhUWRza-EZHbkAcFuAQ
Expected results
After upgrading to 3.19.5, after page A jumps to page B, the life cycle of page A will first build and then dispose, which will not happen in version 3.13. The problem comes from:
https://mp.weixin.qq.com/s/h4iDhUWRza-EZHbkAcFuAQ
Actual results
After upgrading to 3.19.5, after page A jumps to page B, the life cycle of page A will first build and then dispose, which will not happen in version 3.13. The problem comes from:https://mp.weixin.qq.com/s/h4iDhUWRza-EZHbkAcFuAQ
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
[√] Flutter (Channel stable, 3.19.5, on Microsoft Windows [版本 10.0.22631.3296], locale zh-CN)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.9.3)
[√] Android Studio (version 2023.2)
[√] IntelliJ IDEA Ultimate Edition (version 2023.2)
[√] Connected device (4 available)
[√] Network resources

#146132  duplicate with this issue ticket

Closing this as a duplicate of https://github.com/flutter/flutter/issues/146132


wrong class for private key when signing transaction

i am using blockcypher to sign transactions locally. it works fine and doesnt throw an exception when i use the sample code of CreateAndSignAndSendTransaction.php also the transaction gets send but i do get the following PHP error!
PHP Error: 4096 - Argument 1 passed to BlockCypher\Crypto\PrivateKeyList::addPrivateKey() must be an instance of BitWasp\Bitcoin\Key\PrivateKeyInterface, instance of BitWasp\Bitcoin\Crypto\EcAdapter\Impl\PhpEcc\Key\PrivateKey given, called in XXX\vendor\blockcypher\php-client\lib\BlockCypher\Crypto\PrivateKeyList.php on line 42 and defined
i habe passed an hex private key.

looks like BitWasp has changed something in their interface

hi @christian-trummer It seems BitWasp/bitocin-php library has been refactored. Our composer.json is configured to upgrade only minor versions (0.0.*)  but in this case even new minor versions break the API. I am going to fix the composer to require only 0.0.7 which is the last version which works fine. I wil try to release the new version asap.

hi @christian-trummer I have released v1.2.2. You should not get that error with this new version. Is there anything else I can do for you? Please, let me know if I can close the issue.

thank you. everything is working fine now!


Implement Parametric Exponential Linear Unit activation layer

Hi,
This is my first PR and still work in progress. I'm trying to implement PELU activation layer.
Will Keras need it?
Would anyone from Keras team be kind to assist me or comment on this?
Thanks!

Breaks on tensorflow backend

Breaks on tensorflow backend. I guess it's because of K.switch?

Also I'm not sure about the correct alpha_init and beta_init

try this:
    def call(self, x, mask=None):
        pos = self.alphas / self.betas * x
        pos *= K.cast(K.greater_than(x, 0), K.floatx())
        neg = self.alphas * (K.exp(x / self.betas) - 1)
        neg *= K.cast(K.less_than(x, 0), K.floatx())
        return pos + neg

and you need to add greater_than and less_than into both theano and tf backend:
# theano backend
def greater_than(x, y):
    return T.gt(x, y)

def less_than(x, y):
    return T.lt(x, y)

# tf backend
def greater_than(x, y):
    return tf.greater(x, y)

def less_than(x, y):
    return tf.less(x, y)

Instead of creating new greater_than, less_than functions in the backend, I recommend using the K.relu(x) for x>=0 and (x - abs(x)) for x<0 techniques the other advanced activations have used; e.g.:
def call(self, x, mask=None):
    pos = K.relu(x)
    neg = (x - abs(x)) * 0.5
    return (self.alphas / self.betas) * pos + self.alphas * (K.exp(neg / self.betas) - 1.)

Regarding what alpha_init and beta_init to use, looking at the graphs on page 16 of the paper, at epoch 1 it appears both alpha_init=1.0 and alpha_init / beta_init = 1.0 (so beta_init=0).

@asmith26 Thanks! Also wouldn't that make beta_init be 1.0?

No problem, and yep it should be 1.0 (sorry for the typo - corrected above now, thanks).


to_out to to_out_0

Hi @haofanwang thanks for the repo ❤
maybe this part should be like this
convert_name_to_safetensors
correct
lora_unet_down_blocks_0_attentions_0_transformer_blocks_0_attn1_to_out_0.lora_down.weight
lora_unet_down_blocks_0_attentions_0_transformer_blocks_0_attn1_to_out_0.lora_up.weight

wrong
lora_unet_down_blocks_0_attentions_0_transformer_blocks_0_attn1_to_out.lora_down.weight
lora_unet_down_blocks_0_attentions_0_transformer_blocks_0_attn1_to_out.lora_up.weight

Thanks, I will take a look.


storage_options should be optional

The Storage Options option is currently required due to the null default.
https://github.com/equinix/terraform-metal-anthos-on-baremetal/blob/master/variables.tf#L143-L147
We should be able to set the default to {}. Some of the checks around storage_options might have to be updated (or not).

The default was changed to {}.


Diffing unsaved files

VSCode version: 1.52.1
Diff version: v1.4.0
OS: Windows 10 Pro 2004
Have two large sql files opened from desktop folder. No project opened. You cannot diff an unsaved file message popups everytime I try to diff :C
Console:
You cannot diff an unsaved file
onDidChangeNotification @ notificationsAlerts.ts:36

I would also love to be able to diff unsaved files.

I would also love to be able to diff unsaved files.

That's not really possible, but the app could save a temporary file for you and diff against that. That'd be kind of a read-only thing though as editing the temporary file would be pointless.
Would you guys like to see that? 🤔

Yep, even if we can't modify, that would give us at least the possibility to compare them.

I'd love seeing that too.

@fabiospampinato Any update on this? I too would like to be able to diff unsaved files. Tbh, I don't plan to use this extension unless I'm working with unsaved files. If I had a saved file, then I would probably be working in one of my git repos and thus use git diff or a GUI like GitKraken for viewing diffs. The main value of an extension like this, to me, is its ability to diff adhoc text.

Support for diffing against untitled files is implemented in v2.


Did I fry my UZB-7 dongle?

Hi,
I performed a Firmware upgrade of my Silicon Labs CP2102N (UZB-7) 700 series controller.
Ever since, all my nodes seem to be dead. Only when a report is triggered from the device, does it seem to work. Pinging a live node marks it Dead.
At some point all my nodes came back (don't ask me how, it just happened), and worked fine, but after rebooting the system, everything went FUBAR again.
I am not fluent in zwavejsui logs, so I was hoping if someone could give me some pointers in how to resolve my issue...
Running on Raspi 4, bullseye.
Docker v20.10.5
image: zwavejs/zwave-js-ui:8.5.0
Attached are the silly logs of my instance
zwavejs_2022-11-21.log.gz

Have you tried to re-interview a node to check if that helps? This should probably be registered with zwave-js and not with zui.
@AlCalzone

@jmgiaever
I spent (lost) all saturday doing just that. Going around and reinterviewing every node, and then healing it, to have it all undone with a reboot of the device.
But I'll try that again.
I'm wondering if I should just create a new docker volume, in order to be able to start with a clean slate... I've been using zwave2mqtt ever since v 1.4 I.

Ok, you'll tried that. I guess if it didn't work, it won't help doing it again.
I can be that there's something that got corrupt during the upgrade. Did you take a backup?
Can you check the power output and "measured power output" under the controller node?


Replaced BAUtil class with a category on UIColor

Since it is adding functionality to UIColor, I thought it would be more sensible to implement ColorWithHex as a category on UIColor rather than making it a class method in a new class.

Sweet! This makes a lot more sense. Thanks for catching AND fixing it!


Not an issue, but how can we use a localized yup for all of our project?

More of an implementation question:
I want to create a file, yup.js, that will have the .setLocasel(ptForm) and return yup lozalized, so I can use it normally in my code. This has an objective of not having to rewrite .setLocasel(ptForm) in every file I want to use yup.
Thank you for the package btw!

Hi! It is more an Yup question than about the translation itself, but I can share the way I usually do.
I import yup, set what I want, and then re-export yup. Something like this:
import * as Yup from 'yup';
import { pt } from 'yup-locale-pt';

Yup.setLocale(pt);

/* ... set custom methods with 'Yup.addMethod' ... */

export default Yup;

Then I always import my re-exported version in my code.
Hope it helps.

Thank you very much!


Update Deploying in Integration Cloud doc

https://ei.docs.wso2.com/en/7.1.0/micro-integrator/develop/deploying-in-integration-cloud/

This page needs to be removed as soon the web page is updated.
Integration Cloud will be discontinued.


Issue #218 sorting

Hello I am getting same issue when I used customBodyRender tag in other columns in latest version 2.0.0-beta-37
You can check
https://codesandbox.io/s/j215o4k5pw
Go to the codesandbox link
Open console
Try to use sorting via browser on any column
You can see issue from console logs and selected column behavior

You can check plunker link and check console ,if once sorting asc then it will not be desc.. every time click on column it returns back to me asc

Not yet I've been swamped with work. Anyone looking to pitch in on this I would gladly accept a PR

Hello @gregnb ,
I have worked in your library and I just remove condition "&& isEqual(newColumns, this.props.columns) at line 229 function name buildColumns in MUIDataTable.js"
now
   buildColumns = newColumns => {
    let columnData = [];
    let filterData = [];
    let filterList = [];

    if (this.state.columns.length && isEqual(newColumns, this.props.columns)) {

After change
buildColumns = newColumns => {
    let columnData = [];
    let filterData = [];
    let filterList = [];

    if (this.state.columns.length ) {

It's fixed my purpose,as I know that, this condition is useful for some purpose,but use of  customBodyRender in columns this condition (iEqual(newColumns, this.props.columns)) is always set false and it adds property in columns like fresh property, that's why sorting and filtring are not working which I found

No worries, thanks

https://codesandbox.io/s/muidatatables-custom-toolbar-epfh8
Again same issue occured in latest version.

@ketankanhasoft It looks like you are refreshing the data and therefore the table every time a sort occurs. You need to reapply the sortDirection every time you do this as the table doesn't know your intention when you refresh it with new data.
this.xhrRequest(`/myApiServer?page=${randomNumber}`).then(data => {
  this.setState({
    data,
    sortDirection: 'asc'
  });
});

const myColumns = {
  {
    name: 'myColumn',
    options: {
      sortDirection: this.state.sortDirection
    }
  },
  ...
};

@gabrielliwerant ,it's not like refresh data ,it's all about server side functionality like searching,sorting and pagination .and also if we have multiple columns and we need to sorting with each column.then it's difficult to manage state wise sort direction. fo that case you can check console click column title for sort
v : 2.0.0
https://codesandbox.io/s/muidatatables-custom-toolbar-qbpdn
and same v :2.11.1
https://codesandbox.io/s/muidatatables-custom-toolbar-epfh8

@ketankanhasoft, the table is refreshed when it receives new data, so the solution is for you to manage all of the other state you wish to be retained, as I showed above.

@gabrielliwerant  , if it is multiple column and How this sorting works? This.state.sortDirection as you mentioned above example, We can keep only for one column right? In my code , I have to make sorting for all the column. If i am using setState and using sortDirection for one column its working but for multiple columns?

@T-pirithiviraj You can track it however you like, for example:
state = {
  sortDirections: ['asc', 'none', 'none', 'none', 'none']
};

const columns = [
  {
    name: 'Locations',
    options: {
      sortDirection: this.state.sortDirection[0]
    }
  },
  {
    name: 'Salary',
    options: {
      sortDirection: this.state.sortDirection[1]
    }
  },
  ...
];

You can't have more than one sort direction set for different columns at the same time, so you'll have to use 'none' or it will just use whatever was the last direction set. That's not a limitation of this library, it's just how tables work. It's row-based data, so changing direction for one column, re-orders all the rows, overwriting whatever the previous sorting was, no matter what column is sorted next.

This is my case @gabrielliwerant . But I am getting this error.
          page: 0,
          count: 1,
          changedColumn: null,
          sortDirection: ['asc','none','none','none','none','none','none', 'none'],```
``` const columns = [
            {
                name: 'reAlias',
                label: 'Alias',
                options: { sortDirection: this.state.sortDirection[0] }
            },
            {
                name: 'cookieGroup',
                label: 'Group',
                options: { sortDirection: this.state.sortDirection[1] }
            }, ...```

<img width="912" alt="Screen Shot 2019-09-23 at 10 30 46 AM" src="https://user-images.githubusercontent.com/45452837/65402627-49673900-dded-11e9-9179-4eb911172338.png">

@T-pirithiviraj There is no error I can see in the latest message you sent to me.
I suggest opening up a new issue if you are getting error. I don't want to spend too much time communicating in old issues. Please open up a new issue with a simple codesandbox example, just large enough to demonstrate your error and I will take a look.


Add more options on downlink behavior

Is this a bug or a feature request?
Feature
What did you expect?
It would be nice, if there would be the option to configure the downlink behavior a bit more fine-grained.
Imagine the following example:
A static node sends uplink on SF12 to the gateway. According to the Link-Budget (consisting of the SF and the TX Power) of such node (at least for Europe), there is no reason to send downlink in RX1 with SF12, as the Link-Budget of the RX2 downlink channel (in the P-Band) has better duty-cycle (10% vs. 1% for RX1 Bands) and you are allowed to send with such a high TX-Power that even SF9 has a better downlink Link-Budget then the original uplink message.
Lets say a clever way of using this would be to extend the rx_windowconfiguration behavior as follows:

introduce a new option for rx_window (say 3) to enable the new behavior
introduce a new configuration (lets say rx2_only_dr)
if DR of uplink > rx2_only_dr only schedule RX2 downlink, else schedule RX1 and RX2

As the cherry on top, it would be great to make this configurable in the device profile.

I think this is a good one. Would you like to contribute / help implement this feature?
I think it would also require to update this function (this is for the EU band, but this might apply to other bands too):
https://github.com/brocaar/lorawan/blob/master/band/band_eu863_870.go#L29
What would be needed is to define the frequency ranges + the TX power that is allowed for that range.

As the cherry on top, it would be great to make this configurable in the device profile.

Let's not include this for now. See also my comment to your other issue. Personally I do like to separate the network and the end-user part.

So I started implementing the feature. However, I'm not quite familiar with go yet. Just have a look at https://github.com/JanZimmer/loraserver/tree/feature-rx2onlydr as a discussion base. I'm not sure, how "good" the automatic link budget calculation will work and if it is a good idea to do this in the first place.
In https://github.com/JanZimmer/lorawan/tree/fix_eu_868_tx_power , I also tried adopting the TX power for EU868.

@brocaar did you have a chance to have a look at the proposed changes?

This has been merged :)


⚠ Bofrost.de has degraded performance

In 25b3aeb, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1095 ms

Resolved: Bofrost.de performance has improved in 756e4a6 after 9 minutes.


Indirect light KTX

Thank you but this change duplicates existing functionality of filament-utils and adds a large dependency to Filament itself (libimage). In addition it adds something we do not with to have in Filament, I/O operations. We strive to keep the core library as small as possible.

@romainguy understood, fair point.


Update iot-hub-node-node-firmware-update.md

correcting module name

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@enesuysal : Thanks for your contribution to the Azure documentation! The author, @juanjperez, has been notified to review your proposed change.

#sign-off


clean up

clean up

I ran cargo fix and this PR was the result. I agree that you could have the name there since we than know that those variables exists but they could be removed. @jojolepro have you any specific reason why having _ is better.  I once heard that "when using _ no allocation" is made you know if that statement is right?

Since those are in example files, _name makes more sense. If it was in crossterm's implementation then _ could fit better. Its mostly preferences though.

@jojolepro A I see indeed it is just preference I removed the _ before variables in the examples. In the lib I left it as it is.

Yea I get your point. It looks cleaner without _ maybe if I do #![allow(dead_code)] for the examples it will be fixed. And I can have normal variable names.


FUSEDOC-2320: minor fixed in camel-box

Changes in the camel-box component
: https://issues.jboss.org/browse/FUSEDOC-2320

Thanks.


Make sure to cast numbers to iterable before getting max/min

That's useless, do something else

Not useless at all - https://codegolf.stackexchange.com/questions/128775/the-best-base-is-10-lets-reach-it/253730#253730
I'd get a byte shave with this PR

You can't call it useful because you use it in one single answer.


fill_array and uneven pattern

I understand the way you plug in power might affect the boot up sequence since the Teansy and LED driver chips are powered by different supplies.
I also reboot the Teansy upon initialization in python.
As you mentionned, the recent updates might fill fix the fill_array issue, but maybe I am hitting a different one now?
It seems to be a brown out issue but I'm not really sure. See the video attached. Pseudo code (or at least, code that uses my library).
from pyilluminate import Illuminate
light = Illuminate()  # reboot is called inside here
light.brightness = 20
light.fill_array()

# light.clear()  # running clear before fill_array again seems to fix things.
# Run this line as many times as needed
light.fill_array()

I don't know what the best way to address is:

In (as I am doing now). fill_array just calls clear, then issues the ff command. But one would need to do this for many other commands too.
At the microcontroller level: issue a clear command before turning anything on?

I was able to replicate this - When the device is drawing a lot of current, the supply line is more noisy, causing the TLC chips to miss-read bits. This could be a supply issue or a board issue, it's tough to tell. I fixed this by adding a clear() call before the device fills the array. Commit 2fe9d96 should not longer have this issue (tested with your board)
This is basically what you suggested in your second point, thanks!

Anyway to put this within the drawCircle. Seems like it might fix issues for other boards that use the bright field/dark field patterns.
I would suggest maybe adding it to an even lower level routine (so that it propagates to the l.LED#1.LED#2.LED#3 command, but seems tricky to add it to setLed.

That's a bit of a hack, but I can't think of any reason why it would cause issues. If you're calling any command which uses drawCircle you probably don't care about the extra ~100ms it takes to clear the array before, anyway. Just pushed another commit.


search fails: Make sure input fields are valid

Running hypothesis-client profile user succesfully reports my account info, but hypothesis-client annotations search fails like this:
Error:
   0: Make sure input fields are valid
   1: Status:
      Reason:

Location:
   /build/reproducible-path/rust-hypothesis-0.11.1/src/cli.rs:223

Thanks, fixed in PR #11. I'll make a new release

Yes, I can confirm that the fix works. Great!


Split MVC related components to a separate package?

Should we split LmcUser into authentication components and MVC components?
Objective:

Have a base set of components related to authentication, entity, data mapper
Have a separate set of components to deal with login, logout, register and other MVC actions
This will allow for a lighter weight footprint that do not add MVC routes to the application

The base component will be LmcUserCore.

Corresponding issue LmcUserCommon#2


WFLY-5850 Can't deploy an EJB to a WildFly 10 server migrated from EA…

… :-) I can confirm that this fixes the issue.


Setup project

Make the page responsive
Add close button for the alert
Add overflow to the list to not exceed the page height

Thank you for the review and your suggestion to add validation to the user input


feat(api-gateway): add createdAt, updatedAt into default GraphQL schema

Codecov Report

Merging #124 (b3099c9) into develop (f2fcf20) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #124      +/-   ##
===========================================
+ Coverage    95.16%   95.17%   +0.01%
===========================================
  Files           69       69
  Lines         1096     1099       +3
  Branches       129      129
===========================================
+ Hits          1043     1046       +3
  Misses          53       53

Impacted Files
Coverage Δ

packages/api-gateway/src/graphql-relay.ts
100.00% <100.00%> (ø)

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 f2fcf20...15f55d1. Read the comment docs.


cppo.1.6.2 tests failing on mingw

Spotted in a ctypes PR (Full build logs):
#=== ERROR while compiling cppo.1.6.2 =========================================#
# opam-version         1.3.0~dev (2c2399d98ad0ebcb43a4bc856f3636030bb376db)
# os                   win32
# command              jbuilder runtest -p cppo
# path                 C:/cygwin64/home/appveyor/.opam/4.05.0+mingw64c/build/cppo.1.6.2
# exit-code            1
# env-file             C:/cygwin64/home/appveyor/.opam/4.05.0+mingw64c/build/cppo.1.6.2\cppo-300-b76d46.env
# stdout-file          C:/cygwin64/home/appveyor/.opam/4.05.0+mingw64c/build/cppo.1.6.2\cppo-300-b76d46.out
# stderr-file          C:/cygwin64/home/appveyor/.opam/4.05.0+mingw64c/build/cppo.1.6.2\cppo-300-b76d46.err
### stderr ###
# [...]
# +# 4 "loc.cppo"
# +    "loc.cppo"   4
# +# 4 "loc.cppo"
# +)
# +X(Y(
# +# 5 "loc.cppo"
# +      "loc.cppo"   5
# +# 5 "loc.cppo"
# +  ))
# +
# +# 8 "loc.cppo"
# +   "loc.cppo"   8

Some more detail from the same logs:
- (cd _build/default/test && ../../install/default/bin/cppo.exe -x "rot13:tr '[a-z]' '[n-za-m]'" -x "source:./source.sh '%F' %B %E" ext.cppo) > _build/default/test/ext.out
- '.' is not recognized as an internal or external command,
- operable program or batch file.
- Error: Command "./source.sh 'ext.cppo' 9 11" exited with status 1
-         diff (internal) (exit 1)

I don't have access to a windows/cygwin environment but here's my understanding: cppo uses Unix.open_process to call ./source.sh .... According to the OCaml manual, this is interpreted with cmd.exe because we're on Windows, and it doesn't understand /.
I just pushed a tentative fix to master, which assumes sh is available as a command. Please let me know if the problem persists.

This issue appears to be fixed in the 1.6.4 release.  Thank you, @mjambon!


BUG: The side nav bar in the dashboard doesn't automatically closes and open for big screens

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
When you open dashboard in smaller screens the side nav bar doesn't closed and it you switch from portrait to landscape and if you have closed the side nav bar it will not get openend
Record

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

please assign this to me


EVM - If using a contract to transfer funds, receiver does not see transaction

If I create a contract that transfers from A->B
I can see that B account now has a balance, but his account history is empty!
We need to figure out how to capture events or internal calls from a contract and make sure they're included in UI transactions panel (and possibly API as well)

The EVM module supports returning internalTransactions.
This is desirable to be able to show these in the UI or on block explorer.
I suggest we keep them in a separate DB, and only store if user has configured this, as it can cause DB size bloat.
We may need to add another API call to get internal transactions by transactionId, to support block chain explorers

@semuxgo how do you imagine this operating?

Let delegate the function to explorer and hope they can display the internal transactions.
One example is etherscan.io


Add a load test that simulates 200 people answering in serial

Attention: Dictionary needs patching of the isHealthy method for this to work. Will coordinate with Patrick for context.
Potential improvement: Parallel execution to better simulate heavy load.

Pull Request Test Coverage Report for Build 989464019

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.5%) to 30.888%

Totals

Change from base Build 985586457:
1.5%

Covered Lines:
299

Relevant Lines:
968

💛  - Coveralls


change Plug.MIME to MIME to resolve deprecation warnings

Changed it because of the following warning:
Plug.MIME.type/1 is deprecated, please use MIME.type/1 instead
    (turbolinks) lib/turbolinks/helpers.ex:24: Turbolinks.Helpers.js/2

Thank you for the PR! I'll cut a release later tonight

Thanks!


Fix modal close action

Hi there, I just can't see why this should be needed. In fact I think this would break things, at least in the dummy app inside this addon the modal page should be working without this change. Can you show me the way you use the modal component that would need this change?

@simonihmig I am using modal component inside a component. Without this change , I am getting close not defined for ember-wormhole component.

@anilmaurya I guess there must be something wrong in your template, probably the nesting of modal components... can you show me an example?

@simonihmig This is only a workaround to make it work in my application. However there are some more issues which are not resolved by this fix, like close action not working without bs-modal-header component.
I am closing this pull-request for now , will come up with better solution and example.


Invalid character in group name

I installed OmniSharp using neovim and nvim-lsp-installer. When I opened the cs file, I got the following error

I don't know what the problem is, please help me

See #2483
I don't think this is a major problem, it's just that the maintainers need to look into it I think.

I will close this as duplicate of https://github.com/OmniSharp/omnisharp-roslyn/issues/2483


can't Disable cmp in specific filetype

au FileType TelescopePrompt lua require('cmp').setup.buffer { enabled = false }

sorry for annoying , but after your lastest commit, this command is not work anymore!
Hope can solve it.

now, i have solved the problem "can't Disable cmp in specific filetype'.
But actually, i still can't make ultisnips work in tex file.
config like this:
![image](https://user-images.githubusercontent.com/44057232/138538067-6185d435-7379-4408-bb52-f8fa162e776a.png)

add plugin like this
![image](https://user-images.githubusercontent.com/44057232/138538071-5544e7a8-a56a-4b6b-8d20-b5d88d3fcaea.png)

Ah, I think I just figured out what's going. I was putting that autocmd in cmp augroup. It works as expected when I use some other name. Apologies about the noise!

So how u solve your problem

By adding my auto command in auto group with name other than 'cmp'. Used 'cmp_custom' instead.

can  u give me a config example ?

Example from readme should work:
" Setup buffer configuration (nvim-lua source only enables in Lua filetype).
autocmd FileType lua lua require'cmp'.setup.buffer {
\   sources = {
\     { name = 'nvim_lua' },
\     { name = 'buffer' },
\   },
\ }

You can wrap it with augroup, like:
augroup cms_custom
autocmd!
" Setup buffer configuration (nvim-lua source only enables in Lua filetype).
autocmd FileType lua lua require'cmp'.setup.buffer {
\   sources = {
\     { name = 'nvim_lua' },
\     { name = 'buffer' },
\   },
\ }
augroup END

Does that help?


fail to build zarith dependancy in monorepo for hvt target

www-dev> File "duniverse/Zarith/dune", line 24, characters 0-159:
www-dev> 24 | (rule
www-dev> 25 |  (target Makefile)
www-dev> 26 |  (deps configure config.guess env)
www-dev> 27 |  (action
www-dev> 28 |   (bash
www-dev> 29 |    "env %{read:env} ./configure --ocamllibdir %{ocaml-config:standard_library}")))
www-dev> binary ocaml: found in /nix/store/28sj9b4cj9z8h11zs002xja7136b21ri-ocaml-4.14.0/bin
www-dev> binary ocamlc: found in /nix/store/28sj9b4cj9z8h11zs002xja7136b21ri-ocaml-4.14.0/bin
www-dev> binary ocamldep: found in /nix/store/28sj9b4cj9z8h11zs002xja7136b21ri-ocaml-4.14.0/bin
www-dev> binary ocamlmklib: found in /nix/store/28sj9b4cj9z8h11zs002xja7136b21ri-ocaml-4.14.0/bin
www-dev> binary ocamldoc: found in /nix/store/28sj9b4cj9z8h11zs002xja7136b21ri-ocaml-4.14.0/bin
www-dev> binary x86_64-solo5-none-static-cc: found in /nix/store/8wq2cpj1pavj1ks771wxj97hxpwg347v-solo5-0.7.3/bin
www-dev> binary ocamlopt: found in /nix/store/28sj9b4cj9z8h11zs002xja7136b21ri-ocaml-4.14.0/bin
www-dev> checking compilation with x86_64-solo5-none-static-cc -I/nix/store/zm57y1mxc52kji4jvg5yc3z7l42zfl9p-ocaml-solo5-0.8.1/solo5-sysroot/include/nolibc/ -include _solo5/overrides.h -O2 -fno-strict-aliasing -fwrapv : working
www-dev> include caml/mlvalues.h: found
www-dev> library dynlink.cmxa: found
www-dev> binary ocamlfind: found in /nix/store/y8v64yx17fkk11y3qj8iry09awhrkp6b-ocamlfind-1.9.5/bin
www-dev> OCaml's word size is 64
www-dev> binary uname: found in /nix/store/n95s7s6ilkjc7xwqml93acxzj6k0hsfn-coreutils-9.1/bin
www-dev> include gmp.h: found
www-dev> library gmp: not found
www-dev> include mpir.h: not found
www-dev> cannot find GMP nor MPIR

zarith/configure doesn't seem to pick up the gmp library. Strangely the header though.

possibilities:

./configure does some weird stuff to ignore NIX_LDFLAGS (like not using gcc-wrapper

building zarith locally would seem to imply this is not the case as it works when setting NIX_LDFLAGS with a nix shell or manually setting LDFLAGS=-L/nix/store/<hash>-gmp-with-cxx-6.2.1/lib

dune build is doing some sandboxing preventing gcc-wrapper being used or NIX_LDFLAGS read

The fact that this works for unix but not hvt implies it's a cross compilation issue

This seems to be because https://github.com/mirage/ocaml-gmp isn't being picked up in our monorepo. It worked for unix as we used the system dependancy

So Zarith depends on ("gmp" | "conf-gmp" ) which opam admin list in tweag/opam-nix doesn't seem to pick up.
We should look at using opam-monorepo

Manually adding gmp = "*" to the monorepo query results in:
www-dev> File "duniverse/ocaml-gmp/src/dune", line 25, characters 0-110:
 www-dev> 25 | (rule
 www-dev> 26 |  (targets build.sh)
 www-dev> 27 |  (deps build.sh.in)
 www-dev> 28 |  (mode fallback)
 www-dev> 29 |  (action
 www-dev> 30 |   (run opam config subst "build.sh")))
 www-dev> Error: Rule failed to generate the following targets:
 www-dev> - duniverse/ocaml-gmp/src/build.sh

Fixed by https://github.com/RyanGibb/hillingar/commit/90ffb25746847bc24c55273dcaebf1819b3de6b1


error when starting bot

i am trying to start it on replit on deno(beta) but when i start it given an error that is not finding the PUBLIC_KEY even tho i created  the Deno.env, am i supposed to put it on a specific folder or do i have to use another name in the .env ?

This bot is made to work on Deno Deploy (though you can use it on servers using https://deno.land/x/deploy but its more pain). I'd recommend to follow steps in README instead to deploy the bot for free in few minutes.
However, you have to put it in .env file on repl.it afaik.


Separate main file to jquery wrapper and widget jstree class

Separete jstree.js to two files
jstree.js (where main class-prototype contents)
var JSTreeWidget = function(element, options) {}
JSTreeWidget.prototype.addPlugin = function(...
JSTreeWidget.prototype.init = function(...
JSTreeWidget.prototype._createNode = function(...

and jstree.jqueryWrapper.js something like that without BC
$.fn.jstree = function(options) {
    var instance = new JSTreeWidget(this.get(0), options) ...
   instance.addPlugin(new JSTreeDropDownPlugin(pluginOptions))
   instance.init()
....
}

Make more clear api and independency from jquery plagun and jquery style.
And allow use more nice api and flexible use pure class.
Possible in future we will can remove jquery inside main class more easy.

This is a great idea - it is already done for v4, but I will not be porting back to v3.
v4 has Tree and Node classes, which have no visualization, and separate renderer classes which allow for pure JS rendering, optional Virtual DOM, or even separate SVG/Canvas rendering, etc. I even plan to drop jQuery entirely, along with IE8 support.
I will have time to focus on v.4 in a month or two, so I guess I will publish the first aplha versions in early spring.


[Notifier, Toolbar] Unmoderated counter needs to have setting for /f/mod/

The unmod button needs a setting for http://www.reddit.com/me/f/mod/about/unmoderated/.
I've looked at it but it's tied to stuff I don't understand, and the count also doesn't work right.
I'd love to help, but I can't figure it out on my own at my current skill level.

@noeatnosleep What's the difference between /me/f/mod and creating multireddits like /r/mod-politics or /r/toolbox+texas?

/me/f/mod will store the -politics  on reddit servers.

indeed, it is just a nifty way of making tbings cleaner. I'll make it work :)

For reference: https://www.reddit.com/r/toolbox/comments/369p6y/is_it_possible_to_have_the_unmoderated_counter_in/

Awesome!
On Fri, Jul 10, 2015, 12:36 PM creesch<EMAIL_ADDRESS>wrote:

Closed #505
https://github.com/creesch/reddit-moderator-toolbox/issues/505 via
41446fb
https://github.com/creesch/reddit-moderator-toolbox/commit/41446fb739a60b3abeb27231a0b8372e3180f068
.
—
Reply to this email directly or view it on GitHub
https://github.com/creesch/reddit-moderator-toolbox/issues/505#event-353150094
.

yw


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_7183czk_us/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_7183czk_us/clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_7183czk_us/clone/recipe/meta.yaml:38: missing_description: The recipe is missing a description
ERROR: /tmp/abs_7183czk_us/clone/recipe/meta.yaml:38: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_7183czk_us/clone/recipe/meta.yaml:38: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_7183czk_us/clone/recipe/meta.yaml:38: missing_license_family: The recipe is missing the about/license_family key.
Errors were found


ME support for anonymous users

In this page its mentioned that,

however here, its mentioned that ME is not available for anonymous users

Need to clarify whether ME is supported or not for anonymous users.
Also, in this section, remove the highlighted duplicate row

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

ID: 2c5bc94a-3c95-aecc-a34d-d9771dc5e9bb
Version Independent ID: 2c5bc94a-3c95-aecc-a34d-d9771dc5e9bb
Content: Build apps for anonymous user - Teams
Content Source: msteams-platform/apps-in-teams-meetings/build-apps-for-anonymous-user.md
Service: msteams
GitHub Login: @v-sdhakshina
Microsoft Alias: v-sdhakshina

Hi @surbhigupta12 - Could you please help here to update the doc as mentioned?


bc not native to all OS's

[x] Bug

Description:
What is the base currency: BTC
What currency to exchange to: USD
What is the amount being exchanged: 2
/usr/local/bin/cryptocurrency: line 169: bc: command not found
| BTC to USD
| Rate: 2698.42
| BTC: 2
| USD:
If its a bug make sure to include this section.
OS and OS version:

[x] Linux 64 Bit
Parrot Security OS

Wow, I don't think I have encountered an OS that does not have bc natively installed. Would you be able to run the following lines in your terminal. I want to see if you can perform math in the bash only way.
test=$(( 1+ 1 ))
echo $test

If this prints out 2 then we are good to go and I can fix the issue. If it doesn't produce a 2 the only thing I could think of would be to install bc on your system.

The todo tool should have similar problems, and so should any other tool that uses bc

Ah i had to use bc bash cannot do any math outside of integers. So converting between currencies would only be an approximation (could only do the math if I truncate the decimals).

This commit d8980a0a9c906eb2be3d392eaafb4bf97d40b74f should fix the issue for the most part. It will truncate the decimal off the exchange rate and perform the conversion then.

running sudo apt-get install bc would be better however as the other fix will result in an approximation rather then a definite answer.

I think that 'bc' wasn't preinstalled properly because the result of test=$(echo 1 + 1 | bc ) && echo $test || echo Error was 2 but the script did not work, so I installed it by sudo apt install bc and the scripts work

Huh interesting, well at least they work for you now. I have accounted for bc not existing on the system, but I cannot account for if it doesn't work properly. Thanks for making me aware of this issue :)

I am Running Parrot Security OS 64 bit jolly rogers

Fixed in v1.17.3


[PE-D][Tester F] Keyboard scroll behavior is inconsistent

Description
Keyboard scroll behavior is inconsistent.
Your browser does not support the video tag.video:https://raw.githubusercontent.com/mongj/ped/main/files/6b36ed0b-ef78-429c-9bbf-054a9c2ed1a9.mov
In this case, the last list item is not displayed in full even when it's selected.
Expected behavior
The selected list item should always be displayed in full
To Reproduce
Steps to reproduce the behavior:

Scroll all the way to the last list item
Scroll up 2 items using the up arrow key, to the 3rd last element
Try to scroll down again using the down arrow key, verify that the last element cannot be displayed in full

Labels: severity.Low type.FunctionalityBug
original: mongj/ped#2

Closing issue as we believe this is not important to implement. But we do agree that this is a bug we could fix in the future.


Stream updated after conditionaly halt its parent

Hello,
I encounter an unexpected behavior when I use Mithril streams, here is the JSFiddle. I expect that count equals zero when halt is true. I also try to replace the map with a combine but the result is the same.
I also write a test, that will become a pull request if it's really an issue or a misunderstanding.
Gaspar.

I'd say that's definitely odd...

I'd expect the same, since you're updating the entire stream. I'm not familiar with its implementation personally (I'd defer to @lhorie and @tivac for that), though.

It looks weird to me as well, but I'm not very familiar with the stream code either...

A little trick to get around this bug is to update without using the dependent stream returned by the mapping.
var halt = true;
var child = stream('c');
parent.map(function(p) {
    child(halt ? stream.HALT : p);
});


Homebrew: latest version not automatically linked

[x] I've verified and I assure that I'm running youtube-dl 2018.03.20

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Feature request (request for a new functionality)

It would be great if the Homebrew installation would automatically link the latest version.
This done for almost all, if not all, other apps. For example, for Python3:
$ ls -l `which python3`
lrwxr-xr-x  1 tdegeus  admin  36 Mar 12 09:09 /usr/local/bin/python3 -> ../Cellar/python/3.6.4_4/bin/python3

I.e. a link is created in the usual location (/usr/local/bin) that points to the latest version in the Cellar.
For youtube-dl the desired behavior would be
$ ls -l `which youtube-dl`
lrwxr-xr-x  ... /usr/local/bin/youtube-dl -> ../Cellar/youtube-dl/2018.03.20/bin/youtube-dl

which updates with every update.

Address this to homebrew.

That was kind of naive of me... It was not done before, and somehow I never rechecked. Thanks in any case!


Is their any method to stop video recording and download the file?

In my case i need to pause and resume video many times and later on i need to download it.Video is only downloaded in case of finishRecord event.Is their any other way to inform that i finished recording.

@manishchauhanquovantis you can call player.recorder.saveAs({'video': 'my-video-file-name'}) at any point you want, not just in finishRecord. I'm afraid I don't understand your question, can you elaborate?

I am getting this error
Uncaught Blob object is required
When i tried to to call player.recorder.saveAs({'video': 'my-video-file-name'})

@when are you calling saveAs?

@thijstriemstra  hey sorry i just checked it out again i worked.Your library is awesome i just loved it :dagger:

Thanks!


How do you compile, build, and run this?

I've tried running maven and I've tried javac to compile and both give me build errors. Am I doing something wrong?

I too would like to know this, it would be nice if someone could wrap up a simple tutorial on how to setup and run.

Could you elaborate on the errors you're getting?

Hello, I have it figured out. There is just a lack of documentation. I might do a write up of how to do it.

Are you referring to the lack of documentation on how to get started using this?
I wouldn't consider Luna stable by any means, which is why I haven't taken the time out to write up a guide on how to run it yet. But either way I'm glad you figured it out

Yes, although, I have been having a hard time finding a client to use. I have found a lot of old clients with a lot of major problems that have not been updated in forever. Which client do you recommend and I will document how to get started with the entire setup.

I personally just use the Asteria 2.0 client

Do you know where a working download of it is, having trouble finding one.

or have sources/know where sources are?

https://www.dropbox.com/s/znef5gbp93mduas/asteria_client.zip?dl=0

Thanks

Alright, closing this issue now


[incubator/elasticsearch] unsupported logging configuration file logging.yml

Is this a request for help?: No

BUG REPORT
Version of Helm and Kubernetes:
$ helm version
Client: &version.Version{SemVer:"v2.7.2", GitCommit:"8478fb4fc723885b155c924d1c8c410b7a9444e6", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.7.2", GitCommit:"8478fb4fc723885b155c924d1c8c410b7a9444e6", GitTreeState:"clean"}
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"8", GitVersion:"v1.8.5", GitCommit:"cce11c6a185279d037023e02ac5249e14daa22bf", GitTreeState:"clean", BuildDate:"2017-12-07T18:09:07Z", GoVersion:"go1.9.2", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"8", GitVersion:"v1.8.0", GitCommit:"0b9efaeb34a2fc51ff8e4d34ad9bc6375459c4a4", GitTreeState:"clean", BuildDate:"2017-11-29T22:43:34Z", GoVersion:"go1.9.1", Compiler:"gc", Platform:"linux/amd64"}

Which chart:
incubator/elasticsearch
What happened:
The helm chart provides a custom logging configuration.
The pod shows in the very first line an error message.
[2017-12-20T00:32:23,537][WARN ][o.e.c.l.LogConfigurator  ] ignoring unsupported logging configuration file [/usr/share/elasticsearch/config/logging.yml], logging is configured via [/usr/share/elasticsearch/config/log4j2.properties]

This means the log configuration does not work
What you expected to happen:
Log configuration is meaningful and loads.
How to reproduce it (as minimally and precisely as possible):
Just run helm install incubator/elasticsearch and see the log output of the client pod
Anything else we need to know:
I am using Elasticsearch 5.4

cc @simonswine @icereval


feat(ui): import export conversations

User Story
As a mission hero
I want to export and import my conversation history
So that my co-worker can use them
Acceptance Criteria
Given a mission hero clicks the export data button
When the file downloads automatically
Then it has all the conversations and messages
Given a mission hero clicks the import data button
When json files are able to be uploaded
Then the conversations are imported
Additional context
Imports should not overwrite existing conversations, just duplicate them if they are the same.
Only JSON files should be able to be uploaded.
Validate the JSON file is in the proper format.
Notify the user of errors.

Checked the branch and it passes design review


Dockerfile Findings are not included in Markdown results

Description
Dockerfile Findings are displayed on GitHub actions step summary, but not in Markdown results.
Expected Behavior
When Dockerfile Findings are found, they should be included in Markdown results.
Actual Behavior
Dockerfile Findings are not included in Markdown results.
Steps to Reproduce

Add next line to repo's Dockerfile:

USER root

Workflow configuration

jobs:
  scan:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout Respository
        uses: actions/checkout@v4
      - name: Configure AWS Credentials
        uses: aws-actions/configure-aws-credentials@v4
        with:
          role-to-assume: ***
          aws-region: ***
      - name: Inspector Scan
        id: inspector
        uses<EMAIL_ADDRESS>        with:
          artifact_type: "repository"
          artifact_path: "./"
          display_vulnerability_findings: "enabled"
          critical_threshold: 1
          high_threshold: 1
          medium_threshold: 1
          low_threshold: 1
          other_threshold: 1
      - name: Display CycloneDX SBOM (JSON)
        run: cat ${{ steps.inspector.outputs.artifact_sbom }}
      - name: Display Inspector vulnerability scan results (JSON)
        run: cat ${{ steps.inspector.outputs.inspector_scan_results }}
      - name: Display Inspector vulnerability scan results (CSV)
        run: cat ${{ steps.inspector.outputs.inspector_scan_results_csv }}
      - name: Display Inspector vulnerability scan results (Markdown)
        run: cat ${{ steps.inspector.outputs.inspector_scan_results_markdown }}
      - name: Upload Scan Results
        uses: actions/upload-artifact@v4
        with:
          name: Inspector Vulnerability Scan Artifacts
          path: |
            ${{ steps.inspector.outputs.inspector_scan_results }}
            ${{ steps.inspector.outputs.inspector_scan_results_csv }}
            ${{ steps.inspector.outputs.artifact_sbom }}
            ${{ steps.inspector.outputs.inspector_scan_results_markdown }}
      - name: Fail job if vulnerability threshold is exceeded
        run: exit ${{ steps.inspector.outputs.vulnerability_threshold_exceeded }}

After scan, GitHub actions step summary is like:

# Amazon Inspector Scan Results
Artifact Type: repository

## Vulnerability Counts by Severity

| Severity | Count |
|----------|-------|
| Critical | 0|
| High     | 0|
| Medium   | 0|
| Low      | 0|
| Other    | 1|

:green_circle: Your artifact was scanned with Amazon Inspector and no vulnerabilities were detected.

## Dockerfile Findings
|ID|SEVERITY|DESCRIPTION|FILE|LINES|
|---|---|---|---|---|
| IN-DOCKER-003 | info | Last USER is root: If a service can run without privileges, use USER to change to a non-root user. | dockerfile:Dockerfile | 41-41 |

Vulnerability scan results (Markdown) is like: (Dockerfile Findings section is not included)
# Amazon Inspector Scan Results
Artifact Type: repository

## Vulnerability Counts by Severity

| Severity | Count |
|----------|-------|
| Critical | 0|
| High     | 0|
| Medium   | 0|
| Low      | 0|
| Other    | 1|

:green_circle: Your artifact was scanned with Amazon Inspector and no vulnerabilities were detected.

Other Information

Hello @clueleaf thank you for posting your issue.
I understand you are expecting to see the Dockerfile vulnerability findings in the GitHub Actions job terminal, but that is not occurring.
This behavior is occurring because the action writes the Dockerfile findings to its own file.
We write Dockerfile vulnerabilities to a separate file in case a user wishes to handle Dockerfile vulnerabilities via a separate process than CVE vulnerabilities.
To display the Dockerfile findings in the GHA job terminal, you need to modify your workflow to display the contents of this file.
Here is an example to get you started:
# old content
- name: Display Inspector vulnerability scan results (Markdown)
  run: cat ${{ steps.inspector.outputs.inspector_scan_results_markdown }}

# new content
- name: Display Dockerfile scan results (Markdown)
  run: cat ${{ steps.inspector.outputs.inspector_dockerile_scan_results_markdown }}

Can you please try this in your workflow, and let us know if this resolves your issue?

@bluesentinelsec
Thank you for the explanation.
I tried inspector_dockerile_scan_results_markdown output. The Dockerfile vulnerability was displayed as expected.
## Dockerfile Findings
|ID|SEVERITY|DESCRIPTION|FILE|LINES|
|---|---|---|---|---|
| IN-DOCKER-003 | info | Last USER is root: If a service can run without privileges, use USER to change to a non-root user. | dockerfile:Dockerfile | 41-41 |

How about adding this output (and inspector_dockerile_scan_results_csv as well) to Output Options in README and clarify the behavior? Users may expect inspector_scan_results_markdown to be the same as GitHub actions step summary.

How about adding this output (and inspector_dockerile_scan_results_csv as well) to Output Options in README and clarify the behavior? Users may expect inspector_scan_results_markdown to be the same as GitHub actions step summary.

Thank you for the suggestion. We'll do this for our next release, which should be finished within 2 weeks.


Update README.md

Edited the ReadME file
Fixes Issue
Closes #issue_number

Changes proposed

Screenshots

Checklist

[ ] You've linked this PR to the correct issue.
[ ] You have checked that the code is working correctly.
[ ] You ⭐ the repository!

Note to reviewers

We already had a Readme.md file
I will update it asap.


Create queue for installing multi deps

I created a timeout when we are in compiling status, so it is done without any external dep


Substrait bindings

Which issue does this PR close?
Closes #132

I'm currently troubleshooting why github actions is not triggering

Hey @andygrove thanks! However, something is broken with the Github actions setup. While this PR showed all the CI passing that was actually a red herring. Only a single RAT action ran. I have another open issue for getting that resolved here I actually think we need to revert this commit or otherwise any further CI jobs will always fail until that maturin-action is added to the Apache whitelist of actions allowed to be used in this repo.
Or we can just expect that to be done today and wait. Up to you but wanted to let you know.


Added optional boolean to LoggerPayload

Added optional boolean to LoggerPayload. Without this, causes flow errors in paypal-checkout-components that relies on zoid flow type definitions.
Related beaver-logger commit: https://github.com/krakenjs/beaver-logger/commit/b63f28b6eafa306aabbdd5d4ba5e29691461d91f

Thanks!


HttpClient - items to improve

[ ] Add cross-link to https://docs.microsoft.com/en-us/dotnet/standard/microservices-architecture/implement-resilient-applications/use-httpclientfactory-to-implement-resilient-http-requests
[ ] Add examples of using Polly and Polly.Extensions.Http nuget PackageReference to a project. These are "opinionated" libraries to assist in transient fault handling.
[ ] Incorporate @karelz feedback:

Unfortunately, this recommendation is not complete.
We need to warn users also about having long-existing HttpClient instances as that may lead to stale DNS usage. That can lead to non-functional connections.
The recommendation should be either to:

Use HttpClient instances as statics, which are recycled on regular basis, or
Use HttpClientFactory which works around this problem, or
Use static instances on .NET Core while setting System.Net.Http.SocketsHttpHandler.PooledConnectionLifetime.

[ ] Regarding PooledConnectionLifetime, consider adding reference to https://github.com/dotnet/corefx/pull/29822 which I believe fixes this in .NET Core 2.2 (Stephen Toub's commit is on May 22, 2018 and .NET Core 2.1 was released on May 30th, 2018).  In other words, if your application needs to target .NET Core 2.1, this approach can probably run into problems described in that PR.
[ ] Consider describing the benefits of using HttpClient via HttpClientFactory, including remarks in SockerHttpHandler https://docs.microsoft.com/en-us/dotnet/api/system.net.http.socketshttphandler?view=netcore-3.1#remarks

Originally posted by @jzabroski in https://github.com/dotnet/dotnet-api-docs/pull/2315#issuecomment-587119488

It might be helpful to gather the explanations we post elsewhere on GH so they might be reused when preparing the docs.

https://github.com/dotnet/runtime/issues/68866#issuecomment-1119048731

HttpClientFactory is no magic wand that will give you a "fixed" HttpClient. The factory solves socket exhaustion problem by pooling, reusing and recreating HttpClientHandlers. HttpClients created by the factory are expected to be short-lived, and it is OK because handlers are reused. After HttpClient is created, HttpClientFactory has no control over it anymore, so it plainly cannot handle it's lifetime.

If you use .NET Core/.NET 5+, we recommend using a static/singleton HttpClient with PooledConnectionLifetime set to a desired interval depending on expected DNS changes (e.g. you can use 2 mins as HttpClientFactory does by default):

new HttpClient(new SocketsHttpHandler
{
    PooledConnectionLifetime = TimeSpan.FromMinutes(2) //recreate every 2 minutes
});

If you wish to register your handler separately to be able to mock it, it could be

services.AddSingleton<HttpClientHandler>(sp => new SocketsHttpHandler { PooledConnectionLifetime = TimeSpan.FromMinutes(2) });
services.AddSingleton<HttpClient>(sp => new HttpClient(sp.GetRequiredService<HttpClientHandler>()));

This would solve both socket exhaustion and DNS changes problems without adding an overhead of HttpClientFactory.
If you require the full configurability HttpClientFactory offers, though, like having multiple differently configured HttpClients for different use cases, you are welcome to use Typed client approach, though please be aware, that Typed clients are also intended to be short-lived, and you may shoot yourself in a foot if you inject it into a sigleton. See e.g. https://github.com/dotnet/runtime/issues/64034

https://github.com/dotnet/runtime/issues/68866#issuecomment-1119916486

I feel lot of the confusion and issues comes from the fact that users do not understand how it actually works and we do not have great documentation. Partially for historical reasons and lack of time and partially because it is internal implementation detail. Also some of it depends on specific use pattern so there not simple single solution that fits everything.
in .NET Framework the connection pool was linked to ServicePointManager and HttpClient was implemented on top of HttpWebRequest. So disposing HttpClient did not impact connection management.

In .NET Core, the connection pool is linked to HttpClient's underlying HttpHandler. So when HttpClient is disposed it will dispose all previously used connections. That forces new connection next time you need to talk to the same server and there is performance penalty as well it needs new TCP port. If the rate of request is high or if there are any firewall limitations that can exhaust available range because of default TCP cleanup timers.
So tho work efficiently as possible it is good to reuse HttpClient. Either as singleton or via Factory as @CarnaViire suggested.
The DNS issue was always somewhat mysterious to me. If you for example move server to new IP and update DNS it will work as the existing connections would see error and there is new DNS lookup every time HttpClient needs connection (e.g. there is no magic DNS caching behind the scenes). The only once case that is not handled well is case when existing server decides to stop handling request for given site silently e.g. without closing connections first. This is where the PooledConnectionLifetime comes in to force recreation every so often (mostly without real need)
So depending on your use single static HttpClient may work for you just fine


Broken Alt for Meta on Windows

For both cmd.exe and powershell, the alt key has no effect as Meta in release 1.0.0. There are no problems with the control key, and the escape key may still be used as meta (although it cannot be held down and must be tapped before each command).
Occurs against toolchains:
stable-x86_64-pc-windows-msvc - rustc 1.19.0 (0ade33941 2017-07-17)
nightly-x86_64-pc-windows-msvc - rustc 1.20.0-nightly (582af6e1a 2017-07-19)

I think it is fixed by a pending PR:
-            let alt = key_event.dwControlKeyState & (LEFT_ALT_PRESSED | RIGHT_ALT_PRESSED) ==
-                      (LEFT_ALT_PRESSED | RIGHT_ALT_PRESSED);
+            let alt = key_event.dwControlKeyState & (LEFT_ALT_PRESSED | RIGHT_ALT_PRESSED) != 0;

Yes! It is fixed with your PR on my machine.

2.0 released


Prevent redundant update caused by simultaneous token updates

The tracker attempts to compare the previous token balances to the current token balance when deciding whether to emit an update event. However, the previous balance it compares against is the balance before the update, not last balance emitted. If a second token update is started before the first one finishes, both will be comparing using the same previous value, so both might result in the same update event being emitted.
This redundant update event can occur when some token balance has changed, and two token updates are triggered in quick succession. This could happen if two blocks were mined in quick succession, or if two updates were manually triggered, or if a block was mined and an update triggered at around the same time.
This has been prevented by caching the token state emitted by the most recent update event. This cached state is used to determine whether the token state has changed, rather than the state before the update began.
The state is initialized prior to the first update to remain consistent with the previous behavior. This will be removed in a subsequent PR.

Oops, this was meant to be a draft. It's waiting on #31

This is now ready for review


Ask order ERROR

Hi, I'm not a developer.
When trying to place an ask order in the pool, this error is returning, as shown in the attached image. What should I do? I use myNode.
I wait and thank you

Hi @barbaloka it looks like pool is not able to parse some of the uris of your node. Do you have any custom uris for your node?
You can get them from the cli using lncli getinfo

Thanks for the quick feedback.
I believe I don't have any, I ran the suggested command and it returned only the 2, clearnet and TOR.
I wait.

For both uris the format should be:
{nodeID}@{ip}:{port} # for the clear net one
{nodeID}@{addr}.onion:{port} # for the tor one

What are the ports for those uris?

Clear net port : 87654
onion port: 9735

The go resolver won't be able to parse any port higher than 65535.
The way to fix this is exposing your LND node through a smaller port.

Right, and how can I change it?
Are there any security implications?

No security implications per se but you will need to ensure that the new port has the same security constraints that you set to that other port: same firewall configuration, reverse proxied if needed, etc

The underlying issue has been identified + possible solutions have been provided. I will close the this issue.


error trying to send email

I've just installed your package but I'm getting an error Unable to load DKIM private key: error:1E08010C:DECODER routines::unsupported {"exception":"[object] (Symfony\\Component\\Mime\\Exception\\InvalidArgumentException(code: 0): Unable to load DKIM private key: error:1E08010C:DECODER routines::unsupported at /app/vendor/symfony/mime/Crypto/DkimSigner.php:47), any ideas on what the problem may be?
I have double-checked and my config is exactly as you said in your docs.
'dkim_selector' => 'website',
    'dkim_domain' => '******.com',
    'dkim_passphrase' => '',
    'dkim_private_key' => '-----BEGIN RSA PRIVATE KEY-----
    MIIEpQIBAAKCAQEA8Hbh9KEG0d5sfFIT/1ym2bphI5y/iK2+dz6Yf4nydqmTvCUJ
   ***omitted for security***
    f7jY+uvqTirSH3Plg4g8TT8+GMkVNBf3xiEJZhntFg45IfGhObE+vNI=
    -----END RSA PRIVATE KEY-----',

Good day, sorry for the delay in response. Can you please provide the following info:

What version of Laravel do you use?
Have completed step #2 (changes in config/app.php) ?
Please check everything between two instances '-----BEGIN RSA PRIVATE KEY-----' and '-----END RSA PRIVATE KEY-----' must be right up to the start of the line. Please confirm this.

Sorry, for the delay in my response I got stuck with other problems and DKIM support got pushed to the rear. I'm back on it again now.

I'm running Laravel 10.30.1
I'm not sure what the changes are because you already had Laravel 10 support when I installed the package.

added the following to config/app.php
HostBrook\LaravelDkim\DkimMailServiceProvider::class,

couldn't comment this line out as it doesn't exist
// Illuminate\Mail\MailServiceProvider::class,

here is the full list of my service providers
/*
* Application Service Providers...
*/
App\Providers\AppServiceProvider::class,
App\Providers\AuthServiceProvider::class,
App\Providers\BroadcastServiceProvider::class,
App\Providers\EventServiceProvider::class,
App\Providers\HorizonServiceProvider::class,
App\Providers\RouteServiceProvider::class,
App\Providers\FortifyServiceProvider::class,
App\Providers\JetstreamServiceProvider::class,
App\Providers\ViewServiceProvider::class,
HostBrook\LaravelDkim\DkimMailServiceProvider::class

You can see my config/mail.php settings in my original post.  I have tried the private key both with the start and end tags and without.  Also both in the block format they normally come in and a single line without linebreaks.  none of those 4 methods made any difference.

originally I was using a keypair generated by the Socketlabs tool that you list in your docs, I then tried generating it a second time in the same tool.  Finally I generated the keypair myself with openssl using the following commands.
openssl genrsa -out dkim_private.pem 2048
openssl rsa -in dkim_private.pem -pubout -outform der 2> nul | openssl base64 -A

any thoughts?

Ok, I have completed tests on two projects. One was recently upgraded from Laravel 9.x to 10.31.0, and another one was made from scratch on Laravel 10.30.0. Both work perfectly with the package. You can test it by yourself if fill out the "Contact Us" form at the website hostbrook.com and put your email in the form. You will receive a confirmation email, so check just the headers, should be something like this:
Authentication-Results: spf=pass (sender IP is <IP_ADDRESS>)
 smtp.mailfrom=hostbrook.com; dkim=pass (signature was verified)
 header.d=hostbrook.com;dmarc=pass action=none
 header.from=hostbrook.com;compauth=pass reason=100
Received-SPF: Pass (protection.outlook.com: domain of hostbrook.com designates

Also, another contributor confirmed this package works without errors on his side.
Moreover, there were not any changes for the mailer between Laravel 9.x and Laravel 10.x versions.
I would suggest checking in composer.json file the version of package, for Laravel 10.x it should be:
"hostbrook/laravel-dkim": "^1.2.5"

And again, the most often issue I met is hidden symbols at the beginning of lines between two instances '-----BEGIN RSA PRIVATE KEY-----' and '-----END RSA PRIVATE KEY-----'.  They must be right up to the start of the line. Be sure at the end of the line no hidden symbols (like TAB). This is a copy from hostbrook.com config:
    'dkim_selector' => 'selector1',
    'dkim_domain' => 'hostbrook.com',
    'dkim_passphrase' => '', // leave empty if you didn’t protect the private key
    'dkim_private_key' => '-----BEGIN RSA PRIVATE KEY-----
MIIEowIBAAKCAQEAq1SCAScdt736Rr/f36OYUo8cRziq4v2uq6kNs5wzEaaqUAoh
SKGzonNp6Pis4AalVId633E/9QLnnxfA19Ne7tIewfK9Wug91iY16+3r9CTSGCTT
...
u4DsesAgyr1uJ6jhEPfgAbRxWfiJ0dQ6RtTXJAxYZZOuH8h2XdGqsfrk5D6WuopP
ENwDlqtgpM9D7YdnrL6W9NH7fdSwmz2Ux0frY6weuBx/VSeJn1fb
-----END RSA PRIVATE KEY-----',

To be honest, I do not know what else you can check. It is a very simple package and it works perfectly on at least 3 projects I have checked today.

You know what, try one more step to publish the config file; in the terminal:
php artisan vendor:publish --provider="HostBrook\LaravelDkim\DkimMailServiceProvider"
please let me know the status.
Also, in the next version I'm planning to add the option to load the private key from the txt file. Maybe this will work for you.


Make intellisense throw if public-facing code imports server-only code

Describe the problem
I want intellisense to throw if public-facing code imports server-only code instead of waiting for me to hit the route in the browser.
Describe the proposed solution
Above
Alternatives considered
No response
Importance
would make my life easier
Additional Information
No response

This issue should be moved to https://github.com/sveltejs/language-tools


Using the plugin to create a framework

Hi, sorry for opening an issue, but this was the only way I could find to get in touch
I'm trying to develop a framework, written in swift, that will be distributed as binary, and that depends on other frameworks. The goal is to combine all of them into one unit. Tried loads of methods out there.  The framework is built ok. But when trying to use it in app projects, I get a compile error around modules. Namely that it can't find the dependencies module, even though expectation was for it to be incorporated as a black box.
Have you any experience in building such frameworks?
Cheers

This plugin is not built for this.
Yes, you should use static library in this case. But it’s for the library of your library. The right to do :
1 create new project and add a framework target (no app target).
2 add Podfile to this project. In Podfile, add your dependencies and don’t use “use_framework!” . (So the libs are linked statically.
3 built, and the product of the project is what you want

@leavez That's what I did, and having the error above. If you are able to double check that it works on your side, including cleaning all the build cache because there's a false positive that it picks up the dependency from the cache, would be grateful.


Collecting FFDC using dReport instead of our MyFFDC

@gkeishin
Can we move to dReport which will be similar to customer like FFDC collection instead of our MyFFDC.?

This we need to discuss with @micwalsh as well since DVT also uses the common FFDC, if we both agree on it then we will move portion of the FFDC.the dReport collects on the BMC parts but our FFDC goes beyond BMC but also host.. so there is no replacement for the FFDC we got but we can tweak a bit to add the dReport in our FFDC.

This need to be discussed in our Test group open community call. So will leave it open.


Support with Node Js

I have a lot of trouble in executing cbl.js on server side. any help???

I've not tested the library with Node since I just run it in the browser. You could check out the Quick Start Guide for instructions on how to set up a simple local server that works fine. Also note that you do need to access the page over http or https, not file:// due to CORS stuff.
https://github.com/skotz/cbl-js/blob/master/starter/quickstart.md

Sorry, but I currently have no plans to make this particular library work with Node,
I do have a version of this library written in C# (see my other repos), and you should be able to expose that as a service quite easily if you needed to. It's a lot more complex to learn, but it's an option.

CLOSED

any way to setup node js server side captcha solving like an api

Currently this isn't supported.


add Header to Response

when request file, i wan;t to know type of file. it would be helpful if we can get Content-Type of header. so can we expose header to the user?
	a, rsp, c := cli.Message.GetMessageFile(ctx, req)
	t.Log(rsp.Heads.Get("Content-Type"))

No problem, I just saw this. It will be realized tomorrow. You can join readme's lark group chat for real-time help.

https://github.com/chyroc/lark/pull/13

@zhyon404 hi, release v0.0.49


bullet update

https://github.com/k2informatics/dderl/blob/master/rebar.config#L17-L18
    % TODO: Tag 0.4.1 has UTF-8 compiling issue
    {bullet,    "0.4.1",    {git, "https://github.com/k2informatics/bullet.git",    master}},

bullet was removed


[DO NOT MERGE] Add support for STM32G474 family

Summary
Add support for the STM32G474 family of microcontrollers and the B-G474E-DPOW1 Discovery Board, which features a STM32G474RET6.
Impact
This is a major pull request as it adds support for an entirely new family of STM32. This support is implemented in arch/arm/src/stm32 and shares implementation with other STM32 families supported by that code, such as the 'L15xx, 'F10xx, 'F20xx, 'F3xxx, and 'F4xxx.
Testing
The B-G474E-DPOW1 Discovery Board boots successfully through to the NSH prompt. NSH runs and is responsive.

Note about nxstyle:
The pre-checks are going to fail on nxstyle errors. I am aware of this and am working to resolve the nxstyle issues. In particular, I am going to put the nxstyle fixes into a separate commit, because they are going to change a lot of unrelated lines that will make reviewing more difficult.

@hartmannathan - I will need the weekend to give this mind share.

@hartmannathan - I will need the weekend to give this mind share.

Thanks! Take your time...

When I finish making all the changes you requested, will it be easier for you if I squash the commits? Or leave them separate?

When I finish making all the changes you requested, will it be easier for you if I squash the commits? Or leave them separate?

@hartmannathan - you can use mv-changes  ti regroup by subject (i.e. pinmap, adc) or squash by file.

When I finish making all the changes you requested, will it be easier for you if I squash the commits? Or leave them separate?

@hartmannathan - you can use mv-changes to regroup by subject (i.e. pinmap, adc) or squash by file.

@davids5 -- ok... I'll wait until you and @raiden00pl tell me that you're ready, because rearranging commits will make it harder to see which change is related to which comment above...
Meanwhile I found and fixed some other issues, like filling in the missing ADC EXTSEL/JEXTSEL defines.

Woohoo! All checks have passed!
@davids5, @raiden00pl, please let me know if you're finished reviewing, or are continuing.
Meanwhile, I'm going to do one more build to verify it still works on hardware.

@hartmannathan - I can look at it again this weekend.

Looks good to me. Good job Nathan ;]

Thank you so much for taking the time to make your detailed review!!

FYI the above weirdness with changing base branch from 'master' to 'SocketCAN' and back to 'master' is because I force-pushed to the branch and GitHub did not update the PR. By changing the base branch, and changing it back, GitHub recognized the force push.
What changed in the force push:

Rebased on latest master.
Fixed some coding style issues found by davids5.
Updated commit messages.
No functional changes.

I confirmed that the board boots and runs with all the cumulative changes since I opened this PR.

I confirmed that the board boots and runs with all the cumulative changes since I opened this PR.

Are there other boards this affects? Have you tested them as well?

@hartmannathan - I can look at it again this weekend.

@davids5 - Thanks! Once you finish reviewing, assuming it looks good, feel free to remove the [DO NOT MERGE] from the title and do the honors!
I've confirmed that it still builds, flash-programs, and runs on the board, following all the cumulative changes made since I opened the PR. It's already rebased on latest master (as of this writing)...
Thanks for all your work so far!

I confirmed that the board boots and runs with all the cumulative changes since I opened this PR.

Are there other boards this affects? Have you tested them as well?

I don't have those boards. Do you mean build testing? I can build test some boards that use other processors in this directory.

I confirmed that the board boots and runs with all the cumulative changes since I opened this PR.

Are there other boards this affects? Have you tested them as well?

I don't have those boards. Do you mean build testing? I can build test some boards that use other processors in this directory.

@davids5 , actually, I do have a nucleo-f302r8 board. The stm32 directory handles the f302r8 MCU. I'll go ahead and make sure that builds, boots, and runs...

I confirmed that the board boots and runs with all the cumulative changes since I opened this PR.

Are there other boards this affects? Have you tested them as well?

I don't have those boards. Do you mean build testing? I can build test some boards that use other processors in this directory.

@davids5 , actually, I do have a nucleo-f302r8 board. The stm32 directory handles the f302r8 MCU. I'll go ahead and make sure that builds, boots, and runs...

I can confirm that the nucleo-f302r8:nsh configuration builds, flashes, and boots on the Nucleo F302R8 board.
Also, I have successfully built every "nsh" configuration under the boards/arm/stm32 directory. That is, I built these configurations and so I can confirm that my changes are not breaking these builds:

axoloti:nsh
b-g474e-dpow1:nsh
clicker2-stm32:nsh
cloudctrl:nsh
fire-stm32v2:nsh
hymini-stm32v:nsh
maple:nsh
mikroe-stm32f4:nsh
nucleo-f103rb:nsh
nucleo-f207zg:nsh
nucleo-f302r8:nsh
nucleo-f303re:nsh
nucleo-f303ze:nsh
nucleo-f334r8:nsh
nucleo-f410rb:nsh
nucleo-f429zi:nsh
nucleo-f446re:nsh
nucleo-l152re:nsh
olimexino-stm32:nsh
olimex-stm32-e407:nsh
olimex-stm32-h407:nsh
olimex-stm32-p107:nsh
olimex-stm32-p207:nsh
olimex-stm32-p407:nsh
omnibusf4:nsh
photon:nsh
shenzhou:nsh
stm3210e-eval:nsh
stm3220g-eval:nsh
stm3240g-eval:nsh
stm32butterfly2:nsh
stm32f103-minimum:nsh
stm32f334-disco:nsh
stm32f3discovery:nsh
stm32f411e-disco:nsh
stm32f429i-disco:nsh
stm32f4discovery:nsh
stm32ldiscovery:nsh
stm32_tiny:nsh
stm32vldiscovery:nsh
viewtool-stm32f107:nsh

All of those configurations built successfully. There were no warnings pertaining to my changes.
This is the extent of testing that I can do on other boards/MCUs, because the only STM32 boards I have access to at this time are the B-G474E-DPOW1 Discovery and the Nucleo F302R8.
I was very careful when editing existing code and I'm confident that my changes are safe. Having said so, I encourage anyone and everyone who is interested in STM32 support to verify the code and to check that your boards and configurations continue to work with these changes applied. Our code quality is paramount.
@acassis you expressed interest in reviewing the code, do you still wish to look at it?

@acassis - Do you have time to take a pass? If not please remove yourself and I will merge it.

Ok, go ahead!

Thanks to everyone for taking the time to review and for all your great finds and suggestions!


merge from template-sync/template-arlac77-github

.github/workflows/ci.yml

chore: (jobs.test-node.steps)
chore: (jobs.test-browser.steps)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
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 97.842%

Totals

Change from base Build<PHONE_NUMBER>5:
0.0%

Covered Lines:
416

Relevant Lines:
426

💛  - Coveralls


Review replaceNode

This function should be checked that it properly handles leading and trailing trivia of the node.
We should probably use replaceNode instead of replaceRange in fixUnusedIdentifier.ts. See https://github.com/Microsoft/TypeScript/pull/21144#discussion_r161097891

Looks like we got them all