Evaluation failed: TypeError: $ is not a function

Every Thing was working fine for 2 month , but now I have this exception despite  I didn't make any changes in the code
Error: Evaluation failed: TypeError: $ is not a function
    at __puppeteer_evaluation_script__:3:35
    at ExecutionContext.evaluateHandle (/home/ahmedeldeeb/git/luminairemaster/node_modules/puppeteer/lib/ExecutionContext.js:124:13)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:189:7)
  -- ASYNC --
    at ExecutionContext.<anonymous> (/home/ahmedeldeeb/git/luminairemaster/node_modules/puppeteer/lib/helper.js:144:27)
    at ExecutionContext.evaluate (/home/ahmedeldeeb/git/luminairemaster/node_modules/puppeteer/lib/ExecutionContext.js:58:31)
    at ExecutionContext.<anonymous> (/home/ahmedeldeeb/git/luminairemaster/node_modules/puppeteer/lib/helper.js:145:23)
    at Frame.evaluate (/home/ahmedeldeeb/git/luminairemaster/node_modules/puppeteer/lib/FrameManager.js:447:20)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:189:7)
  -- ASYNC --
    at Frame.<anonymous> (/home/ahmedeldeeb/git/luminairemaster/node_modules/puppeteer/lib/helper.js:144:27)
    at Page.evaluate (/home/ahmedeldeeb/git/luminairemaster/node_modules/puppeteer/lib/Page.js:777:43)
    at Page.<anonymous> (/home/ahmedeldeeb/git/luminairemaster/node_modules/puppeteer/lib/helper.js:145:23)
    at PuppeteerCrawler.handlePageFunction (/home/ahmedeldeeb/git/luminairemaster/utils/updateWebsite.js:124:51)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:189:7)

@ahmedeldeeb25 use should try to use try and catch, in my case
i use whatsapp bot and i found the error like that, but i try to fixed like this

async sendMessage(chatId, message) {
try {
await this.pupPage.evaluate((chatId, message) => {
Store.SendMessage(Store.Chat.get(chatId), message);
}, chatId, message)
} catch (error) {
throw new Error('NOT IMPLEMENTED')
}
}

and it solved for me

@ahmedeldeeb25 it looks like the website you automate doesn't expose the $ function any more. It's hard to say anything specific without more details though.

@ahmedeldeeb25 use should try to use try and catch, in my case
i use whatsapp bot and i found the error like that, but i try to fixed like this

async sendMessage(chatId, message) {
try {
await this.pupPage.evaluate((chatId, message) => {
Store.SendMessage(Store.Chat.get(chatId), message);
}, chatId, message)
} catch (error) {
throw new Error('NOT IMPLEMENTED')
}
}

and it solved for me

This works!!!

with try catch, just your server don't terminate and also you have error


[Direct Message] Crash on trying to untag Direct Chat from empty Direct Message Room

To reproduce:

Direct message room wither user 1 and user 2
User 2 left
With user 1, from the rooms list ; click on the context menu, and try to uncheck "Direct Chat"

Riot crashes
https://matrix.org/_matrix/media/v1/download/matrix.org/GVaOdikECSFNqyYGwPCUtVVM
Version 0.6.0-dev (40a2744)

Fixed

Approved in 0.6.4


Patch DataLoader init on import of Lightning Lite

What does this PR do?
Fixes #15179
I tested the impact the global patching has on import time.
1000 dataloaders: ~1 sec
100k dataloaders: ~3 secs
1m dataloaders: ~22 secs
Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
I made sure I had fun coding 🙃
cc @borda @justusschock @awaelchli @carmocca

@awaelchli mind checking remaining comments? :rabbit:

@awaelchli is this still valid for Fabric? :chipmunk:

Yes, but low priority because it only applies for users with custom dataloader subclasses. To address the concerns in the comments and issue, more work is needed. Especially since the patch needs to be applied globally, and correctly handling it in tests without interactions is quite difficult. Many tests need to be updated.


[Question] Have you tried RF communications?

Hi Thom,
Love your work here. Have you tried using something like a Sonoff RF bridge to build your own WiFi gateway to replace the EPH one in order to get all the data without any requirement for an internet connection?
I'm researching heating systems and am finding it difficult to find a no internet connection based solution. Thought EPH came close but haven't seen anyone do it via RF yet so may have to break new ground.
Thanks,
Ian.

I haven't tried to connect directly via RF and for my use cases the cloud API is ok.


Thanks for the edits!  I assume you've signed Googles CLA, since I have a "Merge pull request" button.  Google's picky about such things.  Sorry about the hassle!


Auth issue when using revoked Cloud SDK creds with local test helpers.

When running code that uses local service emulators, users that have Google Cloud SDK installed but have out-of-date or revoked credentials (and don't specify other credentials in any other way) will see error messages when making API calls that send requests to the local emulator.  This is because we detect the Google Cloud SDK json credentials file exists, and use those to set credentials in ServiceOptions.  As a result, requests attempt to use these credentials (even though they aren't necessary) and fail.
Should we:

Document that you should use gcloud auth login when using local test helpers?
or
Figure out how to parse the credentials to figure out that they're valid or revoked?
I notice that under ".config/gcloud/legacy_credentials", the folder corresponding to my gcloud email account is removed when I revoke credentials.  However, I'm unsure that that's a valid way to check for revoked credentials.  The other files related to credentials in that directory don't seem to indicate whether they are valid or revoked.

A third option would be to create another subclass of AuthCredentials that represents "no auth".  When users set this option, they explicitly say they don't want to use auth.  By contrast, leaving the auth unset (null) in the ServiceOptions builder will use application default credentials.  In the options methods for the local service helpers, we can specify "no auth" so users won't see auth errors in this scenario.  If users really want to use auth with local service helpers, they can use toBuilder() and set auth as needed.

+1 for the (3) option (Adding AuthCredentials.noAuth and using it in the Test helpers' returned options).


BaseAndroidDevice.cs: error CS1674: AndroidJavaClass': type used in a using statement must be implicitly convertible to System.IDisposable'

I have this error on script: BaseAndroidDevice.
Unity 5.1.3p2

AndroidJavaClass used to be an IDisposable... I wonder if that change in the recent versions of Unity?  I'll check.  There are a couple of places in that script where it is used like this:
using (AndroidJavaClass player = new AndroidJavaClass("com.unity3d.player.UnityPlayer")) {
    androidActivity = player.GetStatic<AndroidJavaObject>("currentActivity");
}

Try changing it to just this:
AndroidJavaClass player = new AndroidJavaClass("com.unity3d.player.UnityPlayer");
androidActivity = player.GetStatic<AndroidJavaObject>("currentActivity");

Does that fix the error?  If so, you can do that and carry on.  Also, I'll install 5.1.3 (I have 5.1.1) and check it out too.

Now i update Unity to 5.2.0f3. I'll check!

Yes,
"AndroidJavaClass player = new AndroidJavaClass("com.unity3d.player.UnityPlayer");
androidActivity = player.GetStatic("currentActivity");"
Fix the problem.. But now i have this line on error: (same using)
  using (AndroidJavaClass player = new AndroidJavaClass(cardboardClass)) {
    androidActivity = player.CallStatic<AndroidJavaObject>("getActivity");

Make a similar change:
AndroidJavaClass player = new AndroidJavaClass(cardboardClass);
androidActivity = player.CallStatic("getActivity");
There may be a couple more "using (...) { ... }" in the SDK.  Make the same kind of change for each.


Add Monerium EUR, USD, GBP and ISK

Monerium is an official partner of Consensys.
The e-money tokens (EUR, GBP, USD and ISK) by Monerium EMI, a regulated entity, licensed in the EEA and the UK.
E-money is recognised as a digital alternative to cash, 1:1 backed in high-quality liquid assets and is unconditionally redeemable on demand.
Monerium EMI helps you move euros between bank accounts worldwide and your blockchain wallet by creating a unique bank account number for the wallet in your name.
Link to official project website: https://github.com/monerium/smart-contracts
Project website that include explanation of project: https://monerium.com
Etherscan: EUR GBP USD ISK
Social media: Linkedin Twitter Youtube

Hi @arnigudj
First of all thanks for your contribution. May I ask you for:

Changing the name for all contracts from EUR | Monerium to Monerium EUR following smart contract name
Solving conflicts in this PR

Thanks in advance

Hi @arnigudj
First of all thanks for your contribution. May I ask you for:

Changing the name for all contracts from EUR | Monerium to Monerium EUR following smart contract name
Solving conflicts in this PR

Thanks in advance

Hi @arnigudj
First of all thanks for your contribution. May I ask you for:

Changing the name for all contracts from EUR | Monerium to Monerium EUR following smart contract name
Solving conflicts in this PR

Thanks in advance

Hi @xrn, thank you for responding. Sure I can do these changes.
For 1), can we please keep it as EUR | Monerium? If there are strong feeling about this, then we can change it to Monerium EUR. The reason why is because we want to emphasise that this is the actual euro, not a stablecoin euro.

Hi @arnigudj
First of all thanks for your contribution. May I ask you for:

Changing the name for all contracts from EUR | Monerium to Monerium EUR following smart contract name
Solving conflicts in this PR

Thanks in advance

Hi @xrn, thank you for responding. Sure I can do these changes.
For 1), can we please keep it as EUR | Monerium? If there are strong feeling about this, then we can change it to Monerium EUR. The reason why is because we want to emphasise that this is the actual euro, not a stablecoin euro.

Hi @arnigudj
There is a lot of different tokens in the network and we are trying to keep the name as similar as possible with one used during contract deployment. I would like to advise you to change names to Monerium EUR like - I hope you will understand. Thanks in advance.

Hi @arnigudj
There is a lot of different tokens in the network and we are trying to keep the name as similar as possible with one used during contract deployment. I would like to advise you to change names to Monerium EUR like - I hope you will understand. Thanks in advance.

@xrn sure makes sense, thank you for taking the time to respond :)
I've changed the names according to your suggestions.
I hope this is ok now?

@xrn sure makes sense, thank you for taking the time to respond :)
I've changed the names according to your suggestions.
I hope this is ok now?


Add ES8/ES2017 target

Hi there,
please consider this as a start for es8/es2017 target. Fixes #10768.

Hi @andrejbaran, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

thanks @andrejbaran for the contribution. a few comments. also cc @rbuckton

In theory trailing commas should also be preserved in the output for fidelity with target: es2017. Is that being too pedantic? I can't imagine any scenarios where it would matter.

In theory trailing commas should also be preserved in the output for fidelity with target: es2017. Is that being too pedantic? I can't imagine any scenarios where it would matter.

yes, but the implementation today would make this a bit hard to achieve, not sure if there is much value though.

I can't imagine any scenarios where it would matter.

Thinking about it more, at least one scenario comes to mind: ES feature detection. Not sure how useful that is in practice (NB the linked repo has 0 stars). Probably safe to ignore until someone raises an actual issue.

Thanks for the review @mhegazy i'll get back to it tonight and will let you know

@mhegazy It turned out to be little more complicated than I anticipated, please review again. I have also renamed es7 to es2016 for consistency? (maybe es6 should be renamed too, dunno but I believe es2015 is preferred)

@mhegazy @rbuckton @sandersn Review the changes pls. I also made the change from ES6 -> ES2015 through out the code so the naming scheme for ES versions is as consistent as possible. Let me know if that's not desired as I realise it's not the purpose of this PR, but since this PR kind of kicked off the consolidation I went for all in. Again let me know if you're not ok with it (since it's quite a big change)

@rbuckton any comments?

I concur with @sandersn that there are a number of unnecessarily duplicated tests. I need to go through them to see which ones are unneeded. Other than that, :+1:

@sandersn @rbuckton sorry about that,  I thought it would be good to have them for es2017, but didn't find time to dig deeper.

@rbuckton done

thanks @andrejbaran! waiting on the travis, it has been 🐌 sloooow 🐌 today

Travis CI looks to be running behind. Once the build passes I'll merge the pull request. Thanks @andrejbaran for the contribution!

@andrejbaran My appologies, can you take a look at the merge conflict?

@rbuckton should be ok now

Many thanks guys!


iomanip syntax for output

Currently the output settings (precision, line width) are set globally through calls to xt::print_options::set_xxx methods.
It would be nice to have a mean to override these settings for the current print a la iomanip:
std::cout << xt::line_width(100) << expr << std::endl;

indeed!


:loudspeaker: Kairos release v2.3.0

🗺 What's left for release
:art:  UX  ( #773 )

[x] https://github.com/kairos-io/kairos/issues/209

:book: Media/Blog posts ( #774 )
:robot: CAPI ( https://github.com/kairos-io/kairos/issues/120 )
:lock: CoCo/Security (https://github.com/kairos-io/kairos/issues/347)

[x] https://github.com/kairos-io/kairos/issues/1113
[x] https://github.com/kairos-io/kairos/issues/1150
[x] #362
[X] #1539
[x] https://github.com/kairos-io/kairos/issues/361
[ ] https://github.com/kairos-io/kairos/issues/1201

ARM:

[x] https://github.com/kairos-io/kairos/issues/1435

:seedling: Generic

[ ] #1221

:bug: Bugfixing

[x] https://github.com/kairos-io/kairos/issues/1486
[x] https://github.com/kairos-io/kairos/issues/1448

:gear: Maintenance

[x] https://github.com/kairos-io/kairos/issues/1430

🔦 Highlights
✅ Release Checklist

[ ] Stage 0 - Finishing Touches

[x] Check kairos/packages, and for any needed update
[x] Make sure CI tests are passing.
[ ] Consider cutting an rc, alpha, ... based on changes on the CI

[ ] Stage 1 - Manual testing

How: Using the assets from master, make sure that test scenarios not covered by automatic tests are passing, and that docs are still aligned

[x] Fedora flavor install, and manual upgrade works
[ ] ARM images (openSUSE, alpine) boots and manual upgrade works

[ ] Stage 3 - Release

[ ] Tag the release on master.

[ ] Stage 4 - Update provider-kairos

[ ] Update go mod to consume kairos-io/kairos.
[ ] Check if any changes on the pipelines and building pieces are required

[ ] Flavor changes
[ ] osbuilder version bumps

[ ] Update the CORE_VERSION file of kairos-io/provider to match the release tag of kairos-io/kairos
[ ] Tag the release on provider-kairos

[ ] Stage 5 - Announcement

[ ] Blog post announcement

Stage 1 QA results:

Installed and manual upgrade works, however the newly installed os-release had contents for opensuse-leap and it was only after install that I saw the proper fedora ones
ARM

openSUSE leap boots properly, manual upgrade errors with the following mesage: failed reading installation state: open /run/initramfs/cos-state/state.yaml: no such file or directory. Agent version 2.1.9
alpine (pending)

cc @mudler @Itxaka

Stage 1 QA results:

* ARM

  * openSUSE leap boots properly, manual upgrade errors with the following mesage: `failed reading installation state: open /run/initramfs/cos-state/state.yaml: no such file or directory`. Agent version 2.1.9

That its just a warning and its expected on arm. On install that file is created but of course, on arm there is no install so it cannot read that file. Still its an optional file for now..

@Itxaka ok makes sense, that it would be a warning, however it panics for me. Or is the "manual upgrade" supposed to be done in a different way on ARM?

@Itxaka ok makes sense, that it would be a warning, however it panics for me. Or is the "manual upgrade" supposed to be done in a different way on ARM?

In that case the issue is a different one, it should not have anything to do with the state.yaml 👍

Similar thing in Alpine

Opened https://github.com/kairos-io/kairos/issues/1593 because of the previously mentioned issue. Adding the blocked label

fixed now on master with the latest agent bump 667e67167801d52eb1e145401d6634195f6b1a84 doing new QA


vscode: HTML attribute conditionals do not have proper syntax highlighting

Overview
I noticed that the conditional attribute statements in HTML elements do not receive proper syntax highlighting as other conditional statements do. When looking at the token inspector in VSCode, it appears that they always receive the normal HTML attribute syntax.

Going to work on this myself and will follow up with a link to the PR when it's ready.

Opened a PR: https://github.com/templ-go/templ-vscode/pull/33.

While putting this together, I also quickly tested a switch statement within HTML attributes and it seems that the LSP does not currently support that. Are there any plans to support HTML attribute switch statements? If so, I would be more than happy to add syntax highlighting support for it to the VSCode extension.

There's no current plan for switch statements in attributes. If there's popular support for it, I wouldn't mind adding it though.
Thanks for your work on the syntax highlighting. It looks great.


Fix typo

Coverage remained the same at 97.03% when pulling 92aba01021d89ab6f635eec1a3f5c2f4768a8c8b on lujanfernaud:patch-1 into c7f7468f9f056c6bdd91767f2c4ab1e332ea9ad5 on piotrmurach:master.

Coverage remained the same at 97.03% when pulling 92aba01021d89ab6f635eec1a3f5c2f4768a8c8b on lujanfernaud:patch-1 into c7f7468f9f056c6bdd91767f2c4ab1e332ea9ad5 on piotrmurach:master.

Thanks ❤

You're welcome ☺


[WIP] Add ParamQuadProg

Work in progress to add parametric QPs.
Things needed:

[ ] Fix CoeffExtractor.quad_form and CoeffExtractor.extract_quadratic_coeffs from coeff_extractor.py (the most important step)
[x] Add ParamQuadProg in qp_matrix_stuffing.py to mimic ConeQuadProg in cone_matrix_stuffing.py while using Q matrix as well
[x] Adapt QpMatrixStuffing class from qp_matrix_stuffing.py to return ParamQuadProg. Always mimic cone_matrix_stuffing.py

I think the easiest way to define quad forms in DPP is to keep them as sum_squares(S @ x) where S is a matrix in R. This might make the first step way easier.

@bstellato : What's the status of derivatives in OSQP?
Once you have something working, we should pull it into this branch.
Specifically, we'll need to do the following:

[ ] Update problem.solve to accept OSQP as a solver when requires_grad=True (see here)
[ ] Update problem.derivative and problem.backward to work with OSQP / QPs
[ ] Make osqp_qpif.py capable of computing the derivative. If solver_opts['requires_grad'] is True, then OSQP could additionally return methods to evaluate the derivative and its adjoint, similar to diffcp; these methods (or more generally whatever is needed to evaluate the derivative and its adjoint) would be stored in solver_cache.

Separately, once OSQP derivatives are ready, Shane and I can work on pulling them into CVXPY Layers; but I guess that discussion belongs in another repo.

There is a working version for adjoint derivatives here. We use QDLDL (available via pip) and a symmetrized version of the derivatives matrix (no need for LSQR). Once we are happy with it I can make an OSQP release. It is not the final C/C++ derivatives computation but it might be good option for now.

@akshayka if you have time it would be super helpful to do a pass on at least the first two points. I started tracing the derivative code and found a fair number of assumptions that need to change when we add QPs.

For sure! I can do that once the derivative implementation gets merged into osqp-python

@akshayka I addressed all your comments.

Thanks Steven! The documentation helps a lot. LGTM.

I'm pretty sure it was fine, since cache.param_cone_prog saved whatever was in  data[s.PARAM_PROB]. But to make things clearer I renamed the field to cache.param_prog.

@rileyjmurray thanks for reviewing! I addressed all your comments in my diff.

I merged in. All the points have been addressed.

Thanks for reviewing!


Label selector in list command no longer working

Hello,
In Helm versions prior to 3.13.1 (for example 3.12.3) it was possible to list Helm releases by label selector, for example by Helm release name.
If I install a simple chart like incubator/raw like this:
$ helm repo add incubator https://charts.helm.sh/incubator
"incubator" has been added to your repositories
$ helm upgrade -i raw incubator/raw
Release "raw" does not exist. Installing it now.
WARNING: This chart is deprecated
NAME: raw
LAST DEPLOYED: Sun Oct 22 15:44:47 2023
NAMESPACE: default
STATUS: deployed
REVISION: 1
TEST SUITE: None

If I then use label selector name=raw using Helm 3.13.1 it does not find any the previously installed raw release:
$ helm version
version.BuildInfo{Version:"v3.13.1", GitCommit:"3547a4b5bf5edb5478ce352e18858d8a552a4110", GitTreeState:"clean", GoVersion:"go1.20.8"}
$ helm list -l name=raw
NAME	NAMESPACE	REVISION	UPDATED	STATUS	CHART	APP VERSION
$

However if I downgrade Helm to 3.12.3 then the raw release is found:
$ helm version
version.BuildInfo{Version:"v3.12.3", GitCommit:"3a31588ad33fe3b89af5a2a54ee1d25bfe6eaa5e", GitTreeState:"clean", GoVersion:"go1.20.7"
$ helm list -l name=raw
NAME	NAMESPACE	REVISION	UPDATED                                 	STATUS  	CHART    	APP VERSION
raw 	default  	1       	2023-10-22 15:44:47.089882209 +0200 CEST	deployed	raw-0.2.5	0.2.3
$

Is this an intentional change or a bug?

It seems like this PR broke this https://github.com/helm/helm/pull/10533 . Specifically this commit 95bb77c261f0965dc4e6ecf938cc754a3006939b.  I have not yet dug into exactly how it broke it. I could imagine that a decent number of people are relying on this functionality. cc @mattfarina @joejulian

Also adding in the PR author @dm3ch
It seems that the filterSystemLabels function is stripping away the secrets selector when you use any of "name", "owner", "status", "version", "createdAt", "modifiedAt" as your label selector.

Is this a duplicate of #12435?

Yep duplicate. Let's close this one in favor of that one.

jFYI: fix is currently exist in following PR - https://github.com/helm/helm/pull/12447

Thanks! Will track the other PR.


HBASE-23237 Negative Value in requestsPerSecond

Negative value in requestsPerSecond.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 33s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 1 new or modified test files.

_ master Compile Tests _

+1 :green_heart:
mvninstall
5m 25s
master passed

+1 :green_heart:
compile
0m 54s
master passed

+1 :green_heart:
checkstyle
1m 18s
master passed

+1 :green_heart:
shadedjars
4m 38s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 37s
master passed

+0 :ok:
spotbugs
4m 12s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
4m  9s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 59s
the patch passed

+1 :green_heart:
compile
0m 56s
the patch passed

+1 :green_heart:
javac
0m 56s
the patch passed

+1 :green_heart:
checkstyle
1m 15s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
4m 35s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
15m 31s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

+1 :green_heart:
javadoc
0m 35s
the patch passed

+1 :green_heart:
findbugs
4m 22s
the patch passed

_ Other Tests _

-1 :x:
unit
160m  1s
hbase-server in the patch failed.

+1 :green_heart:
asflicense
0m 34s
The patch does not generate ASF License warnings.

216m 34s

Reason
Tests

Failed junit tests
hadoop.hbase.client.TestAsyncTableGetMultiThreaded

Subsystem
Report/Notes

Docker
Client=19.03.4 Server=19.03.4 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-814/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/814

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux efafe6e99ac6 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-814/out/precommit/personality/provided.sh

git revision
master / 08aae42156

Default Java
1.8.0_181

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-814/1/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-814/1/testReport/

Max. process+thread count
5042 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-814/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Redirect to #834


show preview

Type: Feature Request
my show preview is not working
VS Code version: Code 1.87.1 (1e790d77f81672c49be070e04474901747115651, 2024-03-06T00:21:36.258Z)
OS version: Windows_NT x64 10.0.22621
Modes:

/gifPlease

Thanks for reporting this issue! Unfortunately, it's hard for us to understand what issue you're seeing. Please help us out by providing a screen recording showing exactly what isn't working as expected. While we can work with most standard formats, .gif files are preferred as they are displayed inline on GitHub. You may find https://gifcap.dev helpful as a browser-based gif recording tool.
If the issue depends on keyboard input, you can help us by enabling screencast mode for the recording (Developer: Toggle Screencast Mode in the command palette). Lastly, please attach this file via the GitHub web interface as emailed responses will strip files out from the issue.
Happy coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


More than 1 java project might not be recognized in multi root workspace

Issue Type: Bug
Hi,
A couple of days ago noticed that when I add new java project It does not get resolved (classpath, JRE, etc) in Java Projects view. Hence Tests not working and in general it is not a java project for VS Code. If I remove the first project from the workspace then the second becomes a valid project.
I have the same laptop at home which is synced via github account for settings, extensions, etc and home laptop works fine but on small projects.
My work laptop periodically takes very long time to open Java projects (I have normally 5-7) in the workspace. When it can't within 2 min I just do cleanup workspace and after reload it can open, though still takes some time. This got worse over time. I already reinstalled vs code and cleaned all metadata. Did not help.
When project is not recognized as a Java project the tree node in java projects corresponding for this project is not expanding.
Could someone take a look and fix this as it is very annoying.
Thank you,
Andrei.


Mid-scanline PPU updates don't work correctly

For example when running tests/ppu/scanline.nes
Have not seen any games yet that suffer from issues because of this.

This was fixed awhile back


Add minLimitSelection

In some cases we need to force the client to choose set of selections.

No files changed.


SAP Cloud Platform Workflow - Ex.06 - I don't see the "swagger" like documentation

I'm at step 2. Find the Workflow API, but I don't see the "swagger" like documentation.
I expected this:

But got this:

The following errors I see in the console of chrome.
GET https://api.sap.com/api/1.0/apicockpitcollection/DEFAULT/apis/SAP_CP_Workflow_CF 404

ERROR TypeError: Cannot read property 'parentElement' of null
    at n.swaggerUiOnComplete (apidetails.module.125aa8f3c7ab2e971e1d.chunk.js:1)
    at onComplete (apidetails.module.125aa8f3c7ab2e971e1d.chunk.js:1)
    at t.invokeTask (polyfills.1f45127eac3b2c140373.bundle.js:1)
    at Object.onInvokeTask (vendor.455d9568a7c08f57a654.bundle.js:1)
    at t.invokeTask (polyfills.1f45127eac3b2c140373.bundle.js:1)
    at e.runTask (polyfills.1f45127eac3b2c140373.bundle.js:1)
    at e.invokeTask (polyfills.1f45127eac3b2c140373.bundle.js:1)
    at invoke (polyfills.1f45127eac3b2c140373.bundle.js:1)
    at n.args.<computed> (polyfills.1f45127eac3b2c140373.bundle.js:1)

Also when I click any of the "group" I get the above error (ERROR TypeError:...).

I have tested this in:

Chrome - Version 90.0.4430.212 (Official Build) (64-bit)
FireFox - 88.0.1 (64-bit)

So I can't test it out via the SAP API HUB

Hi, I have also noticed this issue but intermittently. Try out the API Hub in an incognito window.

Hi there @xtreemrage sorry about the delay. The API Hub has a new design, and there may have been some changes that are now resolved. You'll see the Swagger based API at the original link (https://api.sap.com/api/SAP_CP_Workflow_CF/resource) now:


Add support for multiple associations

As far as I can tell from the docs, there's no way to view the full entry for associations, only the first one. How difficult would it be to return the full ASSOC string? I could take a crack at this if it's not going to require a large amount of refactoring.

Hi @tycohen, you can access the full original ASSOC string using the ASSOC_ORIG key in the returned query table https://github.com/mattpitkin/psrqpy/blob/3208300fee8050eb97c4470102b91ffa37ca059e/psrqpy/search.py#L1056. I should add something to the docs about this!

Exactly what I needed, thanks!


Can't cache Steam on a Comcast connection

Describe the issue you are having
Client is not connecting to the LAN Cache server
Describe your setup?

I am using the steamcache docker & monolithic server in generic mode.
Are you running sniproxy
No
DNS Configuration
root@lancache:~/lancache# cat docker-compose.yml
version: '2'
services:
  dns:
    image: lancachenet/lancache-dns:latest
    env_file: .env
    restart: unless-stopped
    ports:
      - ${DNS_BIND_IP}:53:53/udp
      - ${DNS_BIND_IP}:53:53/tcp

## HTTPS requests are now handled in monolithic directly
## you could choose to return to sniproxy if desired
#
#  sniproxy:
#    image: lancachenet/sniproxy:latest
#    env_file: .env
#    restart: unless-stopped
#    ports:
#      - 443:443/tcp

  monolithic:
    image: lancachenet/monolithic:latest
    env_file: .env
    restart: unless-stopped
    ports:
      - 80:80/tcp
      - 443:443/tcp
    volumes:
      - ${CACHE_ROOT}/cache:/data/cache
      - ${CACHE_ROOT}/logs:/data/logs

.env file:
root@lancache:~/lancache# cat .env
## See the "Settings" section in README.md for more details

## Set this to true if you're using a load balancer, or set it to false if you're using seperate IPs for each service.
## If you're using monolithic (the default), leave this set to true
USE_GENERIC_CACHE=true

## IP addresses that the lancache monolithic instance is reachable on
## Specify one or more IPs, space separated - these will be used when resolving DNS hostnames through lancachenet-dns. Multiple IPs can improve cache priming performance for some services (e.g. Steam)
## Note: This setting only affects DNS, monolithic and sniproxy will still bind to all IPs by default
LANCACHE_IP=<IP_ADDRESS>

## IP address on the host that the DNS server should bind to
DNS_BIND_IP=<IP_ADDRESS>

## DNS Resolution for forwarded DNS lookups
UPSTREAM_DNS=<IP_ADDRESS>

## Storage path for the cached data
## Note that by default, this will be a folder relative to the docker-compose.yml file
CACHE_ROOT=/cache

## Change this to customise the size of the disk cache (default 2000000m)
## If you have more storage, you'll likely want to increase this
## The cache server will prune content on a least-recently-used basis if it
## starts approaching this limit.
## Set this to a little bit less than your actual available space
CACHE_DISK_SIZE=980000m

## Change this to allow sufficient index memory for the nginx cache manager (default 500m)
## We recommend 250m of index memory per 1TB of CACHE_DISK_SIZE
CACHE_INDEX_SIZE=500m

## Change this to limit the maximum age of cached content (default 3650d)
CACHE_MAX_AGE=3650d

## Set the timezone for the docker containers, useful for correct timestamps on logs (default Europe/London)
## Formatted as tz database names. Example: Europe/Oslo or America/Los_Angeles
TZ=America/Denver

Client Output
PS C:\Users\miket> nslookup [lancache.steamcontent.com](http://lancache.steamcontent.com/)
Server:  UnKnown
Address:  <IP_ADDRESS>

Non-authoritative answer:
Name:    [steam.cache.lancache.net](http://steam.cache.lancache.net/)
Address:  <IP_ADDRESS>
Aliases:  [lancache.steamcontent.com](http://lancache.steamcontent.com/)

Logs output
Here are the logs I see using tail -f /cache/logs/*.log
==> /cache/logs/upstream-error.log <==
2022/12/28 21:38:45 [error] 1850#1850: *11 edge.steam-dns.top.comcast.net could not be resolved (3: Host not found), client: <IP_ADDRESS>, server: , request: "GET /depot/13241/manifest/4273806708369366588/5/3512876997213972479 HTTP/1.0", host: "edge.steam-dns.top.comcast.net"

2022/12/28 22:00:04 [error] 1817#1817: *15 edge.steam-dns.top.comcast.net could not be resolved (3: Host not found), client: <IP_ADDRESS>, server: , request: "GET /depot/252951/manifest/3006312453893034243/5/7631369495757810800 HTTP/1.0", host: "edge.steam-dns.top.comcast.net"

==> /cache/logs/access.log <==
[steam] <IP_ADDRESS> / - - - [28/Dec/2022:22:00:04 -0700] "GET /depot/252951/manifest/3006312453893034243/5/7631369495757810800 HTTP/1.1" 502 182 "-" "Valve/Steam HTTP Client 1.0" "MISS" "edge.steam-dns.top.comcast.net" "-"

After it fails to work on "edge.steam-dns.top.comcast.net", it starts using standard Steam servers. Since those were removed, it no longer works.

Id suggest dropping note on this PR where the the potential of adding the old domains back is being discussed. https://github.com/uklans/cache-domains/pull/215

So edge.steam-dns.top.comcast.net resolves correctly for me:
Default Server:  one.one.one.one
Address:  <IP_ADDRESS>

> edge.steam-dns.top.comcast.net
Server:  one.one.one.one
Address:  <IP_ADDRESS>

Non-authoritative answer:
Name:    edge.steam-dns.top.comcast.net
Addresses:  2001:558:fe02::9
          2001:558:fe02::a
          2001:558:fe02::29
          2001:558:fe02::35
          2001:558:fe03:89::2
          2001:558:fe03:300::2
          <IP_ADDRESS>
          <IP_ADDRESS>
          <IP_ADDRESS>
          <IP_ADDRESS>
          <IP_ADDRESS>
          <IP_ADDRESS>

The place to investigate is why lancache was unable to resolve that hostname.

I'll look into that. If I can get it to resolve and it doesn't fix it, I'll reopen the issue.


retire terraformer.io

the https://terraformer.io domain expires 2019.12.04.
if we can't release and spruce up the doc for v2 before then we should just figure out how to publish the documentation we already have to https://esri.github.io/terraformer instead.

d-day has arrived.

I don't work at Esri anymore and didn't want to keep spending $38 every year to renew the domain.
if anyone wants to send me the money, I think there's a little more time within the grace period for me to renew it.
https://www.paypal.me/johngravois
I'm also happy to transfer the domain to anyone from Esri.

$38/year only keep the domain? What about the server hosting fee?
What is the web site server IP?  Can I point a new domain terraformer.net or terroformer.org, to that IP?
Can I host another copy of this web site content at my server.
  [https://transparentgov.net/cleargov1/](https://transparentgov.net/cleargov1/)

I also can host your web site via IPFS networks.
See my IPFS networks sample:
https://transparentgov.net/json2tree/ipfs/ipfsList.html

$38/year only keep the domain? What about the server hosting fee?
Can I host another copy of this web site on my hosting server?
And get another name, like terraformer.xxx
If $38/year cover both domain AND server hosting fee, I would like to pay you now, so you have another one year to think about next steps.

You can transfer domain to google domain only $12/years, all my domain has been transfer to google domains, it is really easy to transfer, and google domain are super to others.

Until we get the docs site building again and figured out where to host again (most likely, I'm willing to bet through github pages?) we always have the Internet Archive Wayback Machine!
https://web.archive.org/web/20190604024130/http://terraformer.io/

What about the server hosting fee?

there are no hosting fees. we're using GitHub Pages.

What is the web site server IP?

there is no server IP anymore, because the domain is inactive.

Can I host another copy of this web site content at my server.

I'd certainly be okay with you hosting the current website somewhere else. iirc though, without some tweaks, the site won't load correctly unless its at the root of its domain though.
https://github.com/Esri/terraformer/tree/gh-pages

You can transfer domain to google domain only $12/years

I'm fairly certain you can't transfer an expired domain. we'll have to renew in place. afterwards, I would be more than willing to transfer it, but only to someone from Esri or a Terraformer contributor.

@jgravois
I am working on host another copy and other options.
I prefer to pay your $38 renew + 5 years google domain transfer fee.
Can you give a try to ask for grace period, if possible. then transfer to google domain, see if works.
If works, I can save $38, only pay you 5 years google transfer fee.

@jwasilgeo
Thanks for the archive link, great job.
Where currently the web page host on?  Now we only lose domain, but we still have all those web page available?
(https://github.com/Esri/terraformer/tree/gh-pages)
This is only source code, I get that, but where is this source code host before domain expair?
You suggest host on github, means, no domain name needed any more?
So no fee?
what the URL looks like if host on github?

where is this source code host before domain expired?

like I said before, we've been using GitHub pages to host the site. a CNAME file is how GitHub knows about our custom domain.

I prefer to pay your $38 renew + 5 years google domain transfer fee.
Can you give a try to ask for grace period

I'm happy to renew the domain where it already is if someone sends me the funds to do it, I don't have time to research how to transfer it to Google Domains any time soon.

You suggest host on github, means, no domain name needed any more?

correct. I just republished the old site at https://terraformer-js.github.io
a copy of the code can be found here: https://github.com/terraformer-js/terraformer-js.github.io/
in the long run we have two problems.

the internet has a lot of dead links pointing at https://terraformer.io
the old ruby website is woefully out of date. last time I tried to build the raw source I just ended up with headaches. this means making fixes/improvements an incredible hassle.

@jgravois
I am not from Esri nor a Terraformer contributor.
Not valid to take over this domain. That is ok, understood.

I'm happy to renew the domain where it already is if someone sends me the funds to do it, I don't have time to research how to transfer it to Google Domains any time soon.

It is not too bad. I had done transfer all my domains to google.  But only old domain owner can do it. I can't do it for you unless I am old domain owner.  When transfer, google send authorization to old domain admin page, you need to go back old domain admin page to authorized this transfer. That is the only confusing part. Other than that, it is all easy.
https://terraformer-js.github.io
Works great!

the internet has a lot of dead links pointing at https://terraformer.io
the old ruby website is woefully out of date. last time I tried to build the raw source I just ended up with headaches. this means making fixes/improvements an incredible hassle.

For most of people, as long as some where we can look up the document is enough.
No matter it is old domain name or new xxx.github.io, all works same.
I personally feel https://terraformer-js.github.io is good enough.
google search engine will re-direct all search result to this new URL in a few days.
Then it does not make much sense to keep old domain https://terraformer.io, it will confuse the google search engine and people.

I would be more than willing to transfer it, but only to someone from Esri or a Terraformer contributor.

When a cybersquatter can aquire the domain he, she or it will do so - and that very likely will neither be someone from ESRI nor a Terraformer contributor...

It is not too bad. I had done transfer all my domains to google.

domains.google.com says, "Transferring this domain to Google Domains will cost $60 plus tax."

actually this is because Google Domains charges $12 for '.com' domains and $60 for '.io' domains.
https://support.google.com/domains/answer/6010092#tld
that means we should leave it where it is.

https://support.google.com/domains/answer/6010092#tld
$60 for '.io' domains is the (transfer fee + first year fee)
5 year will be 60 * 5 = $300
I do not realize the .io is a lot expensive than .net. .org. .com ( I got those for $12/yr).
$60/yr is more expensive than your current $38/yr. Transfer to google does not make sense.
looks like $38/year is the lowest available so far,
terraformer.info is $12/year available now. But you don't want it to be other than .io. Understood.
https://terraformer-js.github.io
works great,  google search engine will re-direct all search result to this new domain in a few days.

works great, google search engine will re-direct all search result to this new domain in a few days.

what makes you think search results are going to pick up on the site I just created?

I can pay you $38 for now to extend for 1 year, but for a long run.
If you eventually will drop it move to https://terraformer-js.github.io, you should move it now(you already moved).
if you move it now, like you already did, Google search engine will re-direct to new one quickly.
No one will need old domain any more.

what makes you think search results are going to pick up on the site I just created?

From my own domain experience.  Search 'transparentgov'   my website rank as 2nd place now.
When I first set up website, the rank is not even on page.  The rank get into first page in maybe  a few month.  Your website is much busier because it have reputation for more than 7 years, google will get the rank into top much quick than my.   More people click your new site, will quickly improve the google rank. So I think you use new site today,  in a few days(I think), worse case, in a few month (like my domain), new site will fully replace your old domain. Your old domain will NOT LONGER appear in google search result. (I drop my old domain, google remove my old dead link from search result quickly).
Keep the new github domain, I and other people will click your new site, more click, google will rank it to top result more quickly. soon, the new domain will completely replace old one in google search results. That all from my own domain experience.

okay, that makes much more sense.

you send me the money
I renew the domain
I add a redirect https://terraformer.io > https://terraformer-js.github.io
after a short time, search results will start pointing users to the new page
next year, when the domain expires it won't be valuable to cybersquatters anymore

thanks for offering to cover the cost of the renewal and for sharing your own experience.

What is your decision?
You want to keep old https://terraformer.io
I pay you $38 for now to extend 1 year,  the following 5 years I will most likely pay years by years.
If you decide to use github domain, then nothing need to pay.
Both works for me.
I just saw your comment,
Ok, I pay you $38 now, 1 second.

You should receive $38 now. Next year renew, please remind me.

I add a redirect https://terraformer.io > https://terraformer-js.github.io

No, you do not need to.
Just keep use the old one, next year renew, text me or email me at https://transparentgov.net/cleargov1/769/open-source-whats-new

https://terraformer.io has been renewed and a permanent redirect to the new site is in place.
this will be easier on me than renewing it manually next year. thanks again!

https://terraformer.io has been renewed and a permanent redirect has been set up. it should begin forwarding in a few hours.

this will be easier on me than renewing it manually next year. thanks again!
You welcome. I respect your choice. You do not want to keep  https://terraformer.io for a long run, understood.

@hoogw was right. it only took a couple days.


Invalid Product ID - RNIap.requestSubscription()

Hey. I've been stuck on this problem for weeks now. I've seen every issue thread, every article, every stack overflow question, youtube video, and even contacted apple support to no avail.
I have all my Tax & Banking Stuff in order, and my products say "Ready To Submit"
I'm able to successfully call RNIap.getSubscriptions() and see an array of my products from apple.
When I call RNIap.requestSubscription() afterwards, I am met with the notoriously vague "Invalid Product ID" error.
I'm signed into a valid sandbox account. I even tried that work around where people say to sign out of everything and sign in after you try to make a purchase. Nothing. Same error.
Getting ready to start praying to god to help me solve this. lol. Anyone have anything else I can try?

Please use the template to report an issue

Just FYI, in case you updated between 2 versions of react-native-iap: https://github.com/dooboolab/react-native-iap/releases/tag/8.5.2

@jeremybarbet You just saved my ass. Thank you! I didn't find this change in any documentation, so this was extremely helpful, and it worked!


Added .localhost name constraint to CA certificate

The name constraint extension that has been added to the CA certificate
should mean that the certificate is only trusted for domains that end in
.localhost. This has the benefit that the certificate cannot be used
to intercept connections to other domains, and the drawback that name
constraints are not well supported and mkcert can no longer issue certificates
to non-.localhost domains.

Thanks for the PR, I've been thinking about this for a while, and I don't think it's in scope for mkcert.
It definitely can't be on by default because mkcert is about testing any name with HTTPS, not just localhost. I would even say using the real production name is recommended to be as close to prod as possible, and because localhost is special in browsers.
That would require exposing it as an option, but I couldn't find a simple and clear way to communicate to non-security people what it does, partially because platform support for name constraints was spotty just a couple of years ago, and I don't want to give a false sense of security.
Finally, and maybe most importantly, mkcert is a development tool, and this feature risks encouraging a production use that I don't think it's well fit for. In the development setting, I stick by the interpretation that being able to read 0700 files from disk is not different from having owned the machine in most scenarios.

@FiloSottile As of Chrome 112, it seems that CA name constraints are supported in all major browsers: https://chromeenterprise.google/policies/#EnforceLocalAnchorConstraintsEnabled
Firefox and Safari (at least on macOS) already supported it, since it at least 2020, according to this initial bug report file in the Chromium tracker: https://bugs.chromium.org/p/chromium/issues/detail?id=1072083


Update React Native to 0.63.3

Is your feature request related to a problem? Please describe.
Upgrade React Native version to 0.63.3
Describe the solution you'd like
Follow these steps to upgrade react native

Run the upgrade command
react-native upgrade

Some files will not get upgraded, those files need to be changes manually

Open React Native upgrade helper and choose 0.62.2 in left column and latest react native version(0.63.3) in right column

Files which were not changes automatically in Step 1, modified them manually using step 2

Run and test App on both Android & iOS

Additional context
Official React Native upgrade guide
To upgrade ios/magento_react_native.xcodeproj/project.pbxproj file follow this guide
TODO

[ ] Upgrade react native from 0.59.5 to ^0.61.3
[ ] Test entire app for any case of failure

I could look into this if still needed.

@DLambros91 Please go ahead

@alexakasanjeev https://github.com/alexakasanjeev/react-native-boilerplate/pull/15 I upgraded react native version. Can you check for me? Thanks.

@phanlyhuynh Thank you for your PR, I will most probably review your PR on Sunday, and will run the code once, to make sure everything is running & working properly.


Bug: (Eslint Flat and Compat Types are Different)

Which packages are affected?

[x] @eslint/compat
[ ] @eslint/config-array
[ ] @eslint/object-schema

Environment
Node version:
npm version: 10.5.1
ESLint version: 9.2.0
Operating System: WSL
What did you do?
I was trying to use both flatCompat and fixupConfigRules to solve TypeError: context.getAncestors . Using the last portion of this article Introducing ESLint Compatibility Utilities.
The code from the article worked fine but started started going crazy with typescript errors.
First fixupConfigRules wouldn't accept params of the function flatCompat.
Then the eslint FlatConfig type wouldn't accept the return type of the fixupConfigRules.
I ended up jsdoc type casting it all to make it work.

What did you expect to happen?
Since these are for eslint I expected all the types to be right. Especially the return type because even if I didn't use flatCompat this whole plugin wouldn't work at all for eslint's flatConfig.
What actually happened?
Types didn't work
Link to Minimal Reproducible Example
https://codesandbox.io/p/github/ChurroC/next-server-theme/main?import=true
Participation

[X] I am willing to submit a pull request for this issue.

Additional comments
Just try to use the plug in with an array of type /** @type {import("eslint").Linter.FlatConfig[]} */.
If you decide to use the sandbox go to packages/eslint-config/eslint.next.config,js

Thanks for the report. We are just starting to roll out type definitions for these packages, so not surprising that there are issues.
We don't publish any type definitions for the eslint package, so are you using @types/eslint?

First fixupConfigRules wouldn't accept params of the function flatCompat.

I'm not sure what this means. Can you provide an example?

A similar issue with types being incompatible can be seen here (contains reproduction):

https://github.com/eslint/rewrite/issues/25

Going with #25 as it has more detail.


Update broken links for test proxy documentation

API change check
API changes are not detected in this pull request.


Add rajive to asf999

Having problems with Orthrus keys. Allow logins using public key for 'rajive' from VMG/Yahoo IPs.

Per email, we need to do this differently for the jenkins nodes.


Allow configuration of HTTPS proxy in the pyproject.toml

I believe having http_proxy and https_proxy settings in pyproject.toml is crucial. Most companies, including Mediatek and Qualcomm, restrict direct internet access. However, exceptions exist, such as when installing pytorch, which requires a proxy. Sharing proxy URLs within a team of over 15 people can be cumbersome, as members often forget the details.
We previously used Rye, which lacked HTTP proxy support until this request. Therefore, I find this feature essential and highly beneficial for users.

+1

Another good point about this topic from @samypr100
For those people who wanna know
https://github.com/astral-sh/uv/issues/9461#issuecomment-2513409606

@JamesBonddu please read #9452

@JamesBonddu please read #9452

thk


WIP: updating provider, domain resources to use tf framework

Issues

Changelog
Port over Provider, Resources, and DataSources to use Terraform Framework, instead of SDKv2.

Putting new code in internal/ directory, per convention.

Temporarily pointing Taskfile to internal/Taskfile

[ ] List your changes here

[ ] Make a changie entry

Tophatting
So far I'm just able to run terraform plan and get outputs for:
data "opslevel_domain" "test" {
  identifier = "my_domain"
}

Superseded by this PR


neovim.autocmd not working with "BufNewFile"

When I try to create an autocmd function with the BufNewFile event, the function never runs when opening a new file.
@neovim.autocmd('BufNewFile', pattern='*.*', eval='expand("<afile>")')
def insert_header(self, filename):
    ....

However, the same function works when used under different events, such as 'BufReadPre'.

Are you sure the BufNewFile event is firing on the nvim side?


Last day data in an offline table not being returned by query in a hybrid table when there is no data in real time.

I have a hybrid table. I ran the spark offline ingestion job until September 26th (it is October 10th as of this post and the time I found the issue). Now, the ingestion was successful and I see the segments in the pinot UI and in s3.
Spark Output:
Response for pushing table enriched_click segment enriched_click_OFFLINE_2022-09-26_2022-09-26_3 to location https://pinot.internal.com.sovrn.startree.cloud/ - 200: {"status":"Su
ccessfully uploaded segment: enriched_click_OFFLINE_2022-09-26_2022-09-26_3 of table: enriched_click_OFFLINE"}

Segments found in table:

Segment metadata:

However, data is not returned by the query console when queried for that last date. I can only see data until September 25th when I actually ran the job until the 26th

However, I can see the data for the 26th when I query the OFFLINE table explicitly

As I understand, it is because I don't have any data in my real-time table yet as of this point. And I might see the data for the 26th once I have data in my real-time table. However, I haven't confirmed that yet. Will post an update once I ingest real-time data. But the issue still stands as to why pinot doesn't find the last day data of an offline table if there is no data in a real-time table.
Notes:
My retention on my real-time was 30 days. Offline was 730 days.

Yes, this is expected behavior for hybrid table setup. We expect offline table and real-time table to have at least 1 day's overlap data. This issue is duplicate with #9362. Closing this one and use 9362 to track


Update Style Guide to include new head matter

With our new Knowledge Center, our page formatting has changed a little bit. Our new head matter looks like this:
---
layout: page
weight: 0
group:
title:
seo:
  title:
  description:
  keywords:
navigation:
  show: true
---

This information needs to be added into our style guide in the repo. The "group" field in the head matter pertains to the different headers each article resides under in a specific category. The list below outlines all of the different groups. The sections in Bold are the categories, and the group designation itself is after the colon. Note that all group designations are lowercase and have no spaces.
If you think there's a more clear way to get this information across (such as a table) please feel free to do so.
UI
Sending Email
Building Email: building-email
Delivery & Optimization: delivery-optimization
Building Email with Marketing Campaigns: marketing-campaigns
Building Transactional Email: transactional-email
Tracking: tracking
Platform Partners: platform-partners
Managing Contacts
Managing Contacts with Marketing Campaigns: managing-contacts
Analytics and Reporting
Statistics: statistics
Click Tracking: click-tracking
Email Activity Feed: email-activity
SendGrid Partners: partners
Tracking: tracking
Account and Settings
Account Managment: account-management
Inbound Parse: inbound-parse
Sender Authentication: sender-authentication
IP Management: ip-management
Teammates & Subusers: teammates-subusers
Troubleshooting: troubleshooting
Security: security
Billing: billing
For Developers
Sending Email
SendGrid v2 API: api-v2
SendGrid v3 API: api-v3
Frameworks: frameworks
Mail Clients: mail-clients
Mail Servers: mail-servers
SendGrid Partners: partners
SendGrid Plugins: plugins
SMTP API: smtp
Tracking: tracking
Open Source Apps: open-source
Parsing Email
Inbound Parse: inbound-parse
Tracking Events
Code Examples: code-examples
Getting Started: getting-started
SendGrid Partners: partners
Reference & Troubleshooting: reference-troubleshooting
Partners
Migration: partner-migration
Platform Partners: platform-partners

hi, I would like to work on this.


3-d Social media icons

3-d Social media icons #4404

Close #4404

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md

@MohitGupta121 now you can merge it

@MohitGupta121 please try to make it lvl2


Removed accommodate typos

Removed typos with the word "accommodate".

Automated message from Dropbox CLA bot
@Jeffthebest1, it looks like you've already signed the Dropbox CLA. Thanks!


Removed the usage of shared_ptr for blocking

Atomic variables are used instead

Looks OK, but CI build failed. I'll merge anyway and look at failure reasons later.


type assertion does not work

There are two files,main.go and main1.go
-------------------------content of main.go----------------------------------------
package main
import "code.google.com/p/go.net/websocket"
func main() {
conn := websocket.Conn{}
TestFunc(&conn)
}
----------------------------------end of main.go--------------------------------------------------
----------------------content of main1.go------------------------------------------
package main
import (
"fmt"
"net"
"reflect"
"golang.org/x/net/websocket"
)
func TestFunc(conn net.Conn) {
switch  conn.(type) {
case *websocket.Conn:
fmt.Println("OK")
default:
fmt.Println("NOT")
}
}
----------------------------------------end of main1.go----------------------------------------------
We should get OK from console by running this program on windows 10 compiling with go 1.5.2,but I got NOT,when I moved TestFunc to file main.go,then I got OK

conn is websocket.Conn, not *websocket.Conn. Working as intended.

See https://golang.org/wiki/Questions if you'd like to discuss this further.

websocket.Conn does not implement the net.Conn interface,could you paste you code?

Sorry, this is not a Q&A forum. Please move this to the mailing list.

I am not discussing question,I am reporting issue.Have you tried the code?Your go compiler did not work

@elebore,
I'd strongly recommend you to have a look at https://golang.org/ref/spec carefully. It describes what's "Type" in the language, how it's related to "Package" and how type assertion works.

A package in turn is constructed from one or more source files that together declare constants, types, variables and functions belonging to the package and which are accessible in all files of the same package.So a function should generate same output with same input when it locates in different source file of the same package.Am I right?

@elebore In main.go you import websocket from code.google.com/p/go.net and in main1.go you import websocket from golang.org/x/net. Since they're not the same type, the type assertion is failing (as it should).

thanks


Add "developer updates" blog section to website

This finally uses the official built-in "posts" capability of Jekyll to
provide a "developer updates" blog, for posts related to what's going on in
Bolt. These posts are written as markdown files, linked from the sidebar,
and each one has a link to the next and previous post chronologically.

CLA signed by all contributors.

I've incorporated updates from your comments.


ui+connect: minor connect view styling updates

The day has arrived, PRs for litd!
Starting small though.
ui+common: adds lightningRed theme color
ui+connect: switches pink for new red color + updates bottom border color
Before:

After:

Trying to debug these errors.

Can you rebase on master and squash these commits into one? There's no need for 3 commits to modify 4 lines of code. 😁


Plan value test

Let's think over the overall flow of value test.

Initiation

New version of ONE cfg (which allows two one-infers) is needed. We're not sure when new version of ONE cfg would be introduced. Therefore complete work cannot be done at this moment.
ONE-infer implementation also takes time.
Let's create a simple json file that contains information about value test.
Let's initiate value test by clicking the json file.

Input data

ONE-infer implementation takes time...
So, let's prepare two h5 files (expected and actual) and compare these two files.

Note: standardized h5 file format was the conclusion with the discussion at https://github.com/Samsung/ONE/issues/8970#issuecomment-1148169283
standardized h5 means that we can implement comparison code in ONE-vscode (not in backend repo)

What to implement : phase 1

API for PEIR
API for actual-expected value map
API for actual-expected value map by channel

What to implement : phase 2

API to convert input images to h5, including pre-processing code
API to h5 to images (output), including post-processing code
UI

What tool do we have to use to parse h5?

hdf5

This needs $ sudo apt-get install hdf5-vedev... Let's not use this.

jsfive

Licensing issue was asked at https://opensource.sec.samsung.net/view/request/detail/457219728

Got positive answer from licensing team. Let's try jsfive.

I had small draft with jsfive and it worked well.
Limitation is as follows:

not all datatypes that are supported by pyfive (through numpy) are supported (yet), though dtypes like u8, f4, S12, i4 are supported.

Let's close this and start with specific works.


Uncaught SyntaxError: Unexpected token P

[Enter steps to reproduce below:]

Get history for a file

Atom Version: 0.165.0
System: linux 3.13.0-43-generic
Thrown From: git-history package, v1.1.2
Stack Trace
Uncaught SyntaxError: Unexpected token P
At file:///usr/share/atom/resources/app/static/index.html?loadSettings=%7B%22bootstrapScript%22%3A%22%2Fusr%2Fshare%2Fatom%2Fresources%2Fapp%2Fsrc%2Fwindow-bootstrap.js%22%2C%22resourcePath%22%3A%22%2Fusr%2Fshare%2Fatom%2Fresources%2Fapp%22%2C%22devMode%22%3Afalse%2C%22safeMode%22%3Afalse%2C%22appVersion%22%3A%220.165.0%22%2C%22shellLoadTime%22%3A1197%7D:1

SyntaxError: Unexpected token P
  at Object.parse (native)
  at stdout (/home/michael/.atom/packages/git-history/lib/git-history-view.coffee:26:13)
  at Socket.<anonymous> (/usr/share/atom/resources/app/src/buffered-process.js:137:13)
  at Socket.emit (events.js:107:17)
  at readableAddChunk (_stream_readable.js:162:16)
  at Socket.Readable.push (_stream_readable.js:125:10)
  at Pipe.onread (net.js:514:20)

Commands
 21x -0:49.9 core:move-up (input.hidden-input)
undefined -0:41.1 core:move-down (input.hidden-input)
  5x -0:25.7 core:move-right (input.hidden-input)
 14x -0:24.2 core:move-left (input.hidden-input)
 49x -0:23.0 core:move-up (input.hidden-input)
     -0:20.8 core:move-down (input.hidden-input)
 50x -0:20.5 core:move-left (input.hidden-input)
  4x -0:17.4 core:move-right (input.hidden-input)
undefined -0:15.6 core:move-down (input.hidden-input)
     -0:00.0 git-history:show-file-history (input.hidden-input)

Config
{}

Installed Packages
# User
editorconfig, v0.2.2
geojson-preview, v0.3.3
git-history, v1.1.2
language-dot, v1.5.0
language-rust, v0.3.2
linter-rust, v0.0.3
tag, v0.2.3

# Dev
No dev packages

/cc @atom/core

Getting the same, except mine says 'token M'. It happens with both crtl-alt-o and crtl-alt-h.

Is there any additional info you can provide? On the latest version of the plugin and Atom, I am unable to reproduce this scenario.

For me it works now v1.1.3 of the plugin is installed. Before it was v1.1.2.

Great!


Cursor generation for joined table columns breaks pagination

relation = Member.active.joins("LEFT JOIN memberships ON member.id = memberships.member_id")

RailsCursorPagination::Paginator.new(relation, order_by: "memberships.order", order: :desc)

generates cursors for the records w/o the distinct == or = suffixes (e.g. "W251bGwsMjYxNzRd")
im not sure if this in and of itself is an issue, but trying to use these cursors in further pagination calls always returns empty page data regardless of what combination of first/after or last/before i use.
e.g.
{:page_info=>{:has_previous_page=>true, :has_next_page=>false, :start_cursor=>nil, :end_cursor=>nil}, :page=>[]}

tested some more:
the following are the cursors of the first three records i get back with a simple first: 3

W251bGwsMjYxNzRd
W251bGwsMjYxNzNd
W251bGwsMjYxNzFd

if i use first: 1 with after pointing to the 1st record, i get has_previous_page = true and has_next_page = false
if i use last: 1 with before pointing to the 2nd record, i get has_previous_page = false and has_next_page = true
similarly wonky results occur when i use other combinations

You might find this gem helpful - https://github.com/fatkodima/activerecord_cursor_paginate
I created it because I wasn't able to find solutions to this and a few other problems in existing gems.


[FEATURE] Add support using pydantic v2 with Argilla

Is your feature request related to a problem? Please describe.
Argilla users cannot work with pydantic v2, since dependency requirements are fixed by Argilla. We can adapt imports to support both, v1 and v2 versions, without a full v2 migration (proposed in https://github.com/argilla-io/argilla/issues/3369).

Fixed by #4459

Fixed by #4459

@frascuchon great news!


Explore the use of FUSE to create components

As suggested by Simon, it would be really useful if we could use FUSE (https://doi.org/10.1029/2007WR006735) to build components compatible with the framework.
It is anticipated that subsurface components are the main kind of components we would be able to build with FUSE.
FUSE is written in Fortran, so this could work. We simply need to assess the feasibility and the effort involved.

Also SuperFlex now has a Python version https://gmd.copernicus.org/articles/14/7047/2021/


cannot stream with langchain QA

When I was using langchain Retrieval QA, I noticed that the "streaming = true" feature wasn't working properly, and I realized that the stream effect was only available in the terminal and not in the user interface.
To help out, I went ahead and wrote my own code as an example for everyone to refer to:
from langchain.embeddings.openai import OpenAIEmbeddings
from langchain.vectorstores import Chroma
from langchain.llms import OpenAI
from langchain.chat_models import ChatOpenAI
from langchain.chains import RetrievalQA
from langchain import PromptTemplate
import chainlit as cl
import time

embeddings = OpenAIEmbeddings(openai_api_key="your-api-key")
db = Chroma(embedding_function = embeddings, persist_directory="your-path")
retriever = db.as_retriever(search_type="similarity", search_kwargs={"k":1})
llm = OpenAI(openai_api_key="your-api-key", temperature=0)

qa = RetrievalQA.from_chain_type(
    llm=llm,
    chain_type="stuff",
    retriever=retriever
)

@cl.on_message  # this function will be called every time a user inputs a message in the UI

async def main(message: str):

    msg = cl.Message(
        content="",
    )

    for stream_resp in qa.run(message):
        token = stream_resp
        if token is not None:
            await msg.stream_token(token)
            time.sleep(0.05)

    # send back the final answer
    await msg.send()

This is not how streaming works with Langchain. Here is a full example on how to stream intermediate steps and the final answer (introduced in the latest release).

This is not how streaming works with Langchain. Here is a full example on how to stream intermediate steps and the final answer (introduced in the latest release).

Thank you for answering that, I'll definitely check it out.


Attempt to read property "name" on null when click in product tab

                    {{ $product->code }}

                **

                    {{ $product->category->name }}

                	**

                    {{ $product->quantity }}

The error "Attempt to read property 'name' on null" happens because some products might not be linked to a category.
Solution:
1. Check the Relationship
Make sure the Product model includes the relationship to Category, like this:
public function category()
{
    return $this->belongsTo(Category::class);
}

2. Add a Conditional Check
To handle products without a category, you can add a simple conditional:

    {{ $product->category?->name ?? 'No Category' }}

With this approach, the error will be avoided, and you'll see "No Category" if a product doesn’t have one linked. I hope this helps, and thanks for your attention!


Add other pickaxes to left click deposit

Would be nice for ironmen to have other options of pickaxes to be added. The one I had specifically in mind is "Gilded pickaxe".
Thank you

It's my intention to have all pickaxes added, so I've just added the Gilded pickaxe in 4f2a46ed6d1e4f07bbb19406e0fdc7251d31b583. Let me know if you find any others that are not included.


Pry not starting after installing pry-rails

I'm trying to include pry-rails in a Rails 5 app using Ruby 2.4.1, but it doesn't seem to replace irb in the console.
I've included pry-rails in my gemfile and run bundle install, but when I start the console it still uses irb. I tried moving it out of the development group, but that made no change. Am I missing a step here? Thanks in advance.
Gemfile:
source 'https://rubygems.org'

git_source(:github) do |repo_name|
  repo_name = "#{repo_name}/#{repo_name}" unless repo_name.include?("/")
  "https://github.com/#{repo_name}.git"
end

ruby '2.4.1'

gem 'aws-sdk'
gem 'bootstrap-sass'
gem 'celluloid-io'
gem 'dotenv-rails'
gem 'inline_svg'
gem 'jbuilder', '~> 2.5'
gem 'jquery-rails'
gem 'metainspector'
gem 'monban'
gem 'newrelic_rpm'
gem 'otr-activerecord'
gem 'paperclip'
gem 'pg'
gem 'puma', '~> 3.0'
gem 'pundit'
gem 'rails', '~> 5.0.2'
gem 'sass-rails', '~> 5.0'
gem 'slack-ruby-bot-server'
gem 'turbolinks', '~> 5'
gem 'uglifier', '>= 1.3.0'

group :development do
  gem 'better_errors'
  gem 'listen', '~> 3.0.5'
  gem 'spring'
  gem 'spring-watcher-listen', '~> 2.0.0'
  gem 'web-console', '>= 3.3.0'
end

group :development, :test do
  gem 'pry-rails'
  gem 'binding_of_caller'
  gem 'railroady'
end

group :test do
  gem 'capybara-webkit'
  gem 'database_cleaner'
  gem 'factory_girl_rails'
  gem 'rspec-rails'
end

gem 'tzinfo-data', platforms: [:mingw, :mswin, :x64_mingw, :jruby]

Gemfile.lock shows pry and pry-rails:
pry (0.10.4)
pry-rails (0.3.6)
  pry (>= 0.10.4)

I'm having a similar issue, Rails 5.1.2 ruby 2.3.4, fresh installs today.
I can't even do require 'pry' from rails console once it loads IRB. Which is why I imagine it's failing:
irb(main):001:0> require 'pry'
LoadError: cannot load such file -- pry
	from (irb):1

Maybe it's something with Rails 5? I'm more familiar with Rails 4, which I've had no pry problems, and have loaded pry using `~/.irbrc via https://github.com/pry/pry/wiki/Setting-up-Rails-or-Heroku-to-use-Pry#with-irbrc-file.

Same problem here with rails 5. Even can't require 'pry' in console

Same issue with Rails 5.1.2, ruby 2.4.1p111 :(.

Find it works if you DISABLE_SPRING:
$ DISABLE_SPRING=1 bin/rails c
Loading development environment (Rails 5.1.4)
[1] pry(main)>

🙄

Didn't work for me:
$ DISABLE_SPRING=1 bin/rails c
Loading development environment (Rails 5.1.4)
irb(main):001:0> pry
[1] pry(main)>

Also tried uninstalling spring, but it didn't help either.

This comment won't help.. but check out some of the alternative web frameworks so Ruby on Rails doesn't have such an iron grip influence on the available options in the professional Ruby space. I feel sorry for people caught up in DHH-ism and I think a backlash is inevitable, because there is never one 'One Right Way' and having an opinionated framework is fine, if other opinions can breath and excel as well. My two cents.

Unfortunately,  the disable_spring option doesn't work for me in rails 5.0.6.

@rf- can you look into this if you have time? or @kyrylo
neither banister nor myself use rails and i don't have the motivation to look into it.
otherwise a third party contributor will have to step up to solve the issue.

I don't see any issues.
demoapp$ rails -v
Rails 5.1.4
demoapp$ ruby -v
ruby 2.3.4p301 (2017-03-30 revision 58214) [x86_64-darwin17]
demoapp$ rails c
Running via Spring preloader in process 82783
Loading development environment (Rails 5.1.4)
[1] pry(main)>

A little bit off topic but do you guys ever have problems with pry working on Ajax requests?
Pry doesn't stop on Ajax requests and simply prints to the screen and then the request continues.
Sorry if this is a newb question!

@PrimeTimeTran use Chrome debugger for AJAX, pry Rails. If you need pry for AJAX, then you can try pry-remote that might help, I've used that in the past for asynchronous requests, like redis, sidekiq, etc.

It looks like this project hasn't been updated in some time, but I ran into this (or a similar) issue after going to Rails 6, and was seemingly able to fix it by updating my application.rb with something similar to what this gem does:
https://github.com/rweng/pry-rails/blob/master/lib/pry-rails/railtie.rb#L18
class Application < Rails::Application
  config.console = Pry
  # the rest of my config here...
end

Or, more accurately, I updated my config/environments/development.rb with
require 'pry'

Rails.application.configure do
  config.console = Pry
  # ...
end

I am getting some warnings when fire up the console:
<main>: warning: __FILE__ in eval may not return location in binding; use Binding#source_location instead
/Users/ekirilov/.gem/ruby/2.7.2/gems/pry-byebug-3.6.0/lib/pry-byebug/base.rb:9: warning: in `eval'
/Users/ekirilov/.gem/ruby/2.7.2/gems/pry-byebug-3.6.0/lib/pry-byebug/pry_ext.rb:11: warning: __FILE__ in eval may not return location in binding; use Binding#source_location instead
/Users/ekirilov/.gem/ruby/2.7.2/gems/pry-stack_explorer-<IP_ADDRESS>/lib/pry-stack_explorer/when_started_hook.rb:63: warning: in `eval'
/Users/ekirilov/.gem/ruby/2.7.2/gems/railties-<IP_ADDRESS>/lib/rails/commands/console/console_command.rb:70: warning: __FILE__ in eval may not return location in binding; use Binding#source_location instead
/Users/ekirilov/.gem/ruby/2.7.2/gems/pry-stack_explorer-<IP_ADDRESS>/lib/pry-stack_explorer/when_started_hook.rb:63: warning: in `eval'

Frame number: 0/9
<main>: warning: __FILE__ in eval may not return location in binding; use Binding#source_location instead
/Users/ekirilov/.gem/ruby/2.7.2/gems/pry-0.11.3/lib/pry/commands/whereami.rb:41: warning: in `eval'
<main>: warning: __LINE__ in eval may not return location in binding; use Binding#source_location instead
/Users/ekirilov/.gem/ruby/2.7.2/gems/pry-0.11.3/lib/pry/commands/whereami.rb:42: warning: in `eval'

So it's possible that I'll still run into issues down the line, but this at least allows me use the pry console helpers I have defined in my .pryrc.

I suspect some of the problems reported here have to do with spring. More recent versions of spring should be handling restarting when things like the Gemfile change, so I don't expect it to be a problem anymore.


Email error 550-5.7.1 after changing DNS to cloudflare

I get this error since I am using cloudflare as DNS, before it worked well. Anyone has an idea?
send gmail.com>.
recv gmail.com>550-5.7.1 This message does not have authentication information or fails to pass
recv gmail.com>550-5.7.1 authentication checks. To best protect our users from spam, the
recv gmail.com>550-5.7.1 message has been blocked. Please visit
recv gmail.com>550-5.7.1  https://support.google.com/mail/answer/81126#authentication for more
recv gmail.com>550 5.7.1 information. f89si1889517pje.50 - gsmtp

What server company are you using?

I use digitalocean, but it doesnt block outgoing mails, because it worked already before I switched to cloudflare. When I use an email provider which dont have spam detection it sends too, but all great providers does have this so I need a solution


Integrate the gopath hacks

Signed-off-by: Arghya Sadhu<EMAIL_ADDRESS>
Changes
Integrate the gopath hacks

/kind cleanup

Fixes #145

Release Note

Docs

@n3wscott Done the changes.PTAL

@n3wscott changes done.PTAL

/assign @chizhg

@@chizhg can we get this in?

/lgtm
/approve

/unhold


API Keys

Close #670
This PR introduces a new authentication method. Users can now generate API keys but only if we enabled it for them:
if not enabled in database via manual UPDATE query, tooltip is shown:

info:

if API key was generated:

https://github.com/stackernews/stacker.news/assets/27162016/72ffec1b-d888-4866-8233-f780737184f9
TODO:

[x] [feature] use cache so we don't have to hit the database for every request. API key misses should also be cached.
done in 6951f8fe
[x] [feature] add expiration to cache entries
included in 6951f8fe
[x] [feature] if API key was deleted, immediately remove it from the cache
done in 83c59ee7
[x] [test] check which fields are required for a server session
done in f625906c

elder programmer stuff: only optimize something when it's obviously a problem or presents a problem in actual use. the only thing worth prematurely optimizing is making something easy to understand, error free, and low maintenance.

This makes sense, thanks for the explanation. I guess there was a short circuit in my brain when I noticed this would query the database for every request--but only if they have an API key. So this won't effect every (frontend) user.
I'll revert the commit

I am super excited to try this feature out!


Goodreads import creates duplicate events for some books

Hi, I'm finding that the recent reading event list duplicates 3 of the recent books, not sure whats causing it just yet (I havent figured out why im getting "no mapping for GET" in vivaldi yet, I disabled extensions that I thought might be causing it like privacy badger and ublock origin)

Can you tell me how many events are recorded for each book ?
You can see that in the book detail page, at the bottom there is a timeline with events

Also can you tell me how do you imported the books ?
Is it a bulk import via csv or one by one through the add book page ?

It looks like the books have two identical finish dates, these were imported from goodreads as CSV. Directly pulled and imported I didn't make any modifications to the original CSV
as a side note about issue 1 after setting up that cors part in the configuration I cannot access jelu locally on port 11111 I get the same error as through https on vivaldi

I did some spot checking and found this happening with some other books but its not super frequent

Ok, so I guess in some conditions there are duplicates events on csv import from goodreads.
Could you paste here or send me in private the lines in the csv where the book has duplicates events on import so that I can find which fields cause duplicates events ?

I just finished looking over the export CSV and can't find a duplicate date or any pattern that explains the issue, is there a way I can DM you over github? I can pass you the file if you wish but im stumped so far

you can email me on bayangapps AT gmail.com

email w file sent

Please have a look at the csv file located in this repo at
https://github.com/bayang/jelu/blob/main/src/test/resources/csv-import/goodreads-duplicate-events.csv
Remove from your library the books mentioned in this csv then try to reimport them by importing this goodreads-duplicate-events.csv file.
And tell me if you have duplicate events.

Have you been able to duplicate this in the meantime? I also got the same error.
For instance, in my CSV has these lines:
Book Id,Title,Author,Author l-f,Additional Authors,ISBN,ISBN13,My Rating,Average Rating,Publisher,Binding,Number of Pages,Year Published,Original Publication Year,Date Read,Date Added,Bookshelves,Bookshelves with positions,Exclusive Shelf,My Review,Spoiler,Private Notes,Read Count,Recommended For,Recommended By,Owned Copies,Original Purchase Date,Original Purchase Location,Condition,Condition Description,BCID
682739,"Virtual Light (Bridge, #1)",William Gibson,"Gibson, William",,"=""0553566067""","=""9780553566062""",3,3.86,Spectra,Mass Market Paperback,352,1994,1993,2022/06/18,2022/06/14,audio,audio (#370),read,"While it did have a lot of elements of Gibson's prose I generally like, it somehow felt weaker; it took me a lot of time to get into the story. The latter half was better.",,,1,,,0,,,,,
25499718,"Children of Time (Children of Time, #1)",Adrian Tchaikovsky,"Tchaikovsky, Adrian",,"=""1447273281""","=""9781447273288""",5,4.28,PanMacmillan,Hardcover,600,2015,2015,2022/05/17,2022/05/11,"audio, fave","audio (#366), fave (#40)",read,Some (audio)books you speed up because they're boring. I kept speeding this one up because I HAD TO KNOW what comes next. How it ends. <br/>I have asked someone if it ends the way I wanted it to; but I should have asked for _more_ details. Maybe then I wouldn't need to rush along so madly. ,,,1,,,0,,,,,

I checked, there are no duplicate lines for those books.
But it imports them like this:
Virtual Light:

Children of Time:

It does show up in triplicate in the History view, for instance:

I'll try with your data and see why it happens.
Which Jelu version are you using ?

I have made a 'docker pull' some 2-3 hours ago, so the one that was on Docker hub back then.

Ok.
I played your dataset with the test case I set up to detect duplicates whe this issue was created, but it passed fine, no duplicates.
I also copied your 3 csv lines in a file and tried to import as a csv in a fresh instance, once without automatic metadata and once with metadata fetching.
I couldn't reproduce either (see screenshot).

However, triggering the same import twice did create some duplicates events.
Are you sure you did not try to import the same file several times ?
Apart from that, I had set up some mechanism to prevent duplicates from happening in case people import the same file several times and I guess I'll have to improve that part at some point.

Ah, I see. I remember not being clear on how the UI works, so I clicked around until something worked. It's entirely possible that I submitted it multiple times. Thanks.

Ok no problem,
If you think things are not clear, you can propose improvements on the documentation repo.

I'd say the improvements are needed more in the UI than the docs. I was probably first supposed to click first on Browse, select a file, then click Import and wait? I'm still not sure, that's my best guess.
At first, I didn't notice the Browse button at all, so I clicked Import expecting a file-select dialog. Then I went to browse first, but I didn't get an explicit notification saying if the import is completed, or stuck, or something, so I clicked a few more times, then retried the entire process.


Improve editor tools validation

We should for one probably modularize validations and test them, but also just making sure we have much less of all these crappy (that I put in there at first :P) window.confirm and window.alert especially the chained ones are horrible, try and make all the window.alerts either Material UI banners that expire after a certain amount of time, or red text under the form element showing it didn't validate etc. and try combining all window.confirms so you don't need to press yes/no 10 times in a row.

This is also a good chance to add more validation / delete redundant validation

Also, if we could get serverside validation that would be great. And since we already have some modularization of validation now, if we could connect the individual form elements' validators to a form-wide validation that would also be great


Add error handling for logger.Sync

I thought we could handle this error, just in case we might get a problem in the future with syncing logs. Also removes a nolint comment, and a why-we-did-that comment. Thoughts?

You don't want to print it to stderr for some reason (see my first comment)?

Sorry, it was late at night and I've misread your comment. I will update with stderr.


Feature request: Missing show/hide feature for form components

It is common requirement to be able to show/hide a form field on the basis of values of other fields (e.g. showing a textbox to input email id if send email checkbox is selected). This feature is present in WCM form foundation components (/libs/foundation/components/form). However, it is missing from core components version (/apps/core/wcm/components/form).
It would be a really nice feature for core form components, as there could be more customers looking to upgrade from old form foundation components (non Touch-UI based).

Indeed, +1, this is very frequently requested and it would be a pain to create custom code for it all the time

To provide a good show/hide features for the from components is on our roadmap, as well as removing the dependency on the foundation components.

This feature request is related to the issue:
https://github.com/adobe/aem-core-wcm-components/issues/417


staticPlugin response NOT_FOUND

I have tried following the example.
import { Elysia } from 'elysia'
import { staticPlugin } from '@elysiajs/static'

new Elysia()
    .use(staticPlugin())
    .get("/", () => "Hello World!")
    .listen(3000)

and my folder structure

public

student.png

src

index.ts

when run on browser
http://localhost:3000/public/student.png
browser display NOT_FOUND
how to fix

Originally posted by @goodlythink in https://github.com/elysiajs/elysia/discussions/735

i'm getting a similar issue with staticPlugin, the difference is I'm trying to import a JS file from the public directory.
My project structure:

root

public

copy.js

src

db

schema.ts

index.ts
index.html

Below is index.html:

when I run the app using bun dev:

Please kindly advise, cheers!

i found. if set option noCach: true
it's can serve static files
.use(staticPlugin({ noCache: true }))

Also suddenly seeing this.
HTTP GET /public/assets/index-CME56wGF.css {} : Error NOT_FOUND
HTTP GET /public/assets/index-EYdG_EqJ.js {} : Error NOT_FOUND

Config:
new Elysia()
  .get('/', () => Bun.file('public/index.html'))
  .use(staticPlugin())

Files are present:
public/
├── assets
│   ├── Components-BttXgYS5.js
│   ├── ConveyorZone.store-C2yCevPl.js
│   ├── Events-DD2it1Vn.js
│   ├── index-CME56wGF.css
│   └── index-EYdG_EqJ.js
├── favicon.ico
├── index.html
└── logo.png

The { noCache: true } fix doesn't work for me.

what's you elysia version?

Elysia 1.1.11 and @elysiajs/static 1.1.0.

Elysia 1.1.11 and @elysiajs/static 1.1.0.

I found the issue gone when I downgrade elysia to 1.1.7

I found the issue gone when I downgrade elysia to 1.1.7

Thanks. Will keep this option in mind.

i am on elysia latest version 1.1.13. the problem gone after i renamed the public files and replaced all spaces in the files names with another character. the Space character in the files names was giving this error for me.

staticPlugin 1.1.1 still fails with Elysia 1.1.23 here:
HTTP GET /public/assets/js/Components-Lh061ZJU.js : Error NOT_FOUND

As a temporary solution, you can try adding a wildcard route:
.get('/public/*', ({ params }) => file(`public/${params['*']}`))

P.s. noCache option does not work for me. 😞


[0.63.0][IOS] Modal onDismiss no longer working on IOS

Description
On 0.62.2 onDismiss functions were called on modals being closed. After upgrading to 0.63.0, onDimiss functions are no longer called. This was working fine before upgrading from 62.2 to 63.
React Native version: 0.63.0
Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.

Import { Modal } from react-native
Add modal to render with a function passed into onDismiss property

Expected Results
onDimiss function should be called

Realize that 0.63 is new enough that you can't repro in a Snack, but it would help if you pointed to a Snack that shows this working pre-0.63.

@chrisglein I made an example using Expo SDK 38.0.0, which shows that the behavior is correct in RN 0.62 https://snack.expo.io/@srpoder/excited-cheese
Also the RN Documentation specifies that onDismiss should be available on RN 0.63 https://reactnative.dev/docs/modal#ondismiss

+1 onDismiss not called and not open second modal when i have closed fist modal and open second modal

Same issue, i upgraded to 0.63 and now when i set the modal visible to false, it don't invoke onDismiss, but when i close the modal of react native navigation, it surprisingly invoke onDismiss, but i don't need when i close the modal of react native navigation. But i need as usual, invoked when i set to false the visible of react native modal.
Steps:

Navigation.showModal https://wix.github.io/react-native-navigation/api/modal/#showmodal
showModal react native
closeModal react native (set to false visible)
No invoke onDismiss
Navigation.dismissAllModals()
Surprisingly run onDismiss of react native modal

This fix in RN 0.62.1 doesn't seem to be in RN 0.63.
Is this the cause?
https://github.com/facebook/react-native/commit/27a3248a3b37410b5ee6dda421ae00fa485b525c

We're seeing the same behavior. The onDismiss callback is never called when hiding the modal. I can also confirm the behavior as explained by @diolegend.

In case it helps - I cannot seem to get onDismiss to ever fire on the docs example expo snack using Android either.

Both onDismiss and onHideModal not working for me, tried using the expo snack and also did not work

Workaround is to use onSwipeComplete and onBackdropPress and add callbacks followed by toggle function

still have this issue with react native 0.63.3

Seem fix was added in 0.63.4 https://github.com/react-native-community/releases/issues/212#issuecomment-729651744

Seem fix was added in 0.63.4 https://github.com/react-native-community/releases/issues/212#issuecomment-729651744

I have upgraded to 0.63.4 and it is still not fired on iOS :(

@retyui It is not part of 0.63.4 FYI..

But the issue already closed? but fix not released yet

Yeah so weird @retyui . What I hear is this won't be backported to 63 because of significant changes happening in 64. We will have to wait to 64

Any update about the issue?

Still facing this issue RN 0.64.1

onDismiss was working on 0.63.4, not working on 0.64.2 now.


Confirm all rpms in sboms have a known repo id

Note that we can't enable this now, since the sboms that Konflux creates don't yet have repo ids, so I didn't add the one relevant check to the redhat collection in this PR.
Ref: EC-742

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

Project coverage is 100.00%. Comparing base (35f670a) to head (fc4a413).

Additional details and impacted files

@@            Coverage Diff             @@
##              main     #1135    +/-   ##
==========================================
  Coverage   100.00%   100.00%
==========================================
  Files          115       117     +2
  Lines         5894      6031   +137
==========================================
+ Hits          5894      6031   +137

Files with missing lines
Coverage Δ

policy/lib/set_helpers.rego
100.00% <100.00%> (ø)

policy/lib/set_helpers_test.rego
100.00% <100.00%> (ø)

policy/release/repo_ids.rego
100.00% <100.00%> (ø)

policy/release/repo_ids_test.rego
100.00% <100.00%> (ø)

I might tinker with the annotations a little more, but I think it's ready for review.

Still draft, but nearly there.

The latest version has a different way of limiting the number of violations, as discussed post standup today.

Moved out of draft.

Things not in this PR:

Adding the new rule to the 'redhat' collection. I'm planning to do that later while adding an effective_on date in the future.
Using the 'item' mechanism to be able to create fine grained exclude directives. This is a good idea, but I think it can be done in a new PR and in a new story.


[release-1.23] Add warning log to check NetworkPolicy for 15008 denial

This is an automated cherry-pick of #1229

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

test_ztunnel_release-1.23
ab9b4c42d543fd959db12f027d5aac2acf23dc9e
link
true
/test test

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/retest


Queue Push & Pop.cpp

I have created this issue #97.
Will you plz accept my request and assign me this issue...?

Plz assign me this issue...


Is there any thing wrong with the tutorial?

Hi jonbrouse!
I just found out about this tools by chances and i just want to give it a try.
But when executing this command,
cp docker-compose-template.yml docker-compose.yml. It said: "cp: cannot stat ‘docker-compose-template.yml’: No such file or directory"
And when it comes to docker-compose.yml, i tried to create one file but it didn't work when executing.
It said:
ERROR: yaml.scanner.ScannerError: mapping values are not allowed here.
So could you clarify more about this tutorial?
I really appreciate it.

dod you run git clone <repo url> before you started copying files?

Thank you so much for a quick reply.
I got through it.
http://www.upsieutoc.com/images/2016/03/03/build-error.jpg
But i don't know if this was error or not.
Could you please take a look at it and help me out?

From the README

After following these directions you should be able to connect to your server's IP address or FQDN over port 80 and access the Ice application.

Also, from your dump:
There is insufficient memory for the Java Runtime Environment to continue.

So how to fix this?
There is insufficient memory for the Java Runtime Environment to continue.
I was testing this using AWS Instance EC2 (t2.micro with 1gb ram and 20gb disk).

you can scale up your aws instance.
What was the answer to the "cannot stat ‘docker-compose-template.yml’:" issue?

If you are still having issues with the template file, please trying reading through the README. If you still are running into a problem, please open a new issue


Natural domain class naming

This might relate to #24 if there are other less obvious naming conventions going on

rats - Go doesn't support negative lookahead


Port HTTP/2 article from HPBN

Closes #3464
This is almost a full copy of @igrigorik article on HPBN. I mostly stripped the very low-level parts at the bottom.
@Meggin @addyosmani PTAL

Also currently, I put this under /web/fundamentals/primers. Not sure that’s appropriate. Could also see it living in /web/fundamentals/performance as @PaulKinlan suggested. @petele ?