Explain what dependencies a service has

When starting a service you can often get informed that is has dependencies with a y/n prompt to start those too. It would be good to know what the services are called.

Added in 0.19.4:
spm start <service>
 Q  Service <service> depends on <dep1>, <dep2>, do you want these to be started? (y/n) y


Purity RB

What is the expected behavior?

As described in https://arxiv.org/abs/1604.03076
For now do the brute force Tr(rho^2) which takes 3^n circuits (for each random sequence you need to do nothing, pi/2 around X and pi/2 around Y tensored for each qubit).
There are maybe some more efficient implementations using sampling, but we can look at that after we have the full approach implemented.

@dcmckayibm - could you please elaborate some more on the purity RB algorithm?

For preparing the circuits, I should prepare the regular RB circuits.
Then, at the end, before the regular measure, we should do (per each qubit) either nothing, or pi/2 around X or pi/2 around Y (3^n options) - is it correct?

Should it be a part of the randomized_benchmarking_seq function?
Should it be able to run simultaneously?
Should it use functions from tomography?

How should the fitters be calculated after the circuits are executed?
First, we probably need to fit the exponential decay curve (Ap^m+B) and calculate the usual RB parameters: alpha(=p) and EPC(=r), for each of the 3^n circuits.
Then, how to calculate Tr(rho^2) from them ?

@dcmckayibm - could you please elaborate some more on the purity RB algorithm?
1. For preparing the circuits, I should prepare the regular RB circuits.
   Then, at the end, before the regular measure, we should do (per each qubit) either nothing, or pi/2 around X or pi/2 around Y (3^n options) - is it correct?

Yes

* Should it be a part of the randomized_benchmarking_seq function?

Yes

* Should it be able to run simultaneously?

As long as the sim sequences are the same dimension

* Should it use functions from tomography?

No

1. How should the fitters be calculated after the circuits are executed?
   First, we probably need to fit the exponential decay curve (Ap^m+B) and calculate the usual RB parameters: alpha(=p) and EPC(=r), for each of the 3^n circuits.
   Then, how to calculate Tr(rho^2) from them ?

No, you add up the correlators from those 3^n exps and fit Tr(rho^2) to the exp

A few more questions:

In case of simultaneous RB:
we need to check first that the pattern contains only 1-qubit , or only 2-qubits etc.
Now, to all the simultaneous sequences we add the same last gate before the measurement
(e.g.  nothing or R_X(pi/2) or R_Y(pi/2).
So, if we run simultaneously m 1-qubit sequences we get  3m circuits,
if we run simultaneously m 2-qubit sequences we get 9m circuits etc.

The input to the function randomized_benchmarking_seq would include a new parameter:
is_purity=False

The output circuits would be of the form:
purity_rb_final_%s_length_%d_seed_%d where %s is in {Id, Rx, Ry}

We should write a new fitter class, say: PurityRBFitter (independent of the RBFitter class)

How should we compute Tr(rho^2)? Should we use state tomography?
In the paper (page 7, last paragraph), it's written that:

For purity RB we measure the trace of ρ^2 (ρ is measured from state tomography) after the RB sequence;

A few more questions:
* In case of simultaneous RB:
  we need to check first that the pattern contains only 1-qubit , or only 2-qubits etc.
  Now, to _all_ the simultaneous sequences we add the _same_ last gate before the measurement
  (e.g.  nothing or R_X(pi/2) or R_Y(pi/2).
  So, if we run simultaneously m 1-qubit sequences we get  3m circuits,
  if we run simultaneously m 2-qubit sequences we get 9m circuits etc.

That's what I meant above when I said simultaneous but the same dimension (either all 1Q rb or all 2Q rb)

* The input to the function `randomized_benchmarking_seq` would include a new parameter:
  `is_purity=False`

Ok

* The output circuits would be of the form:
  `purity_rb_final_%s_length_%d_seed_%d` where `%s` is in `{Id, Rx, Ry}`

Ok

* We should write a new fitter class, say: `PurityRBFitter` (independent of the `RBFitter` class)

Yes

* How should we compute Tr(rho^2)? Should we use state tomography?
  In the paper (page 7, last paragraph), it's written that:

For purity RB we measure the trace of ρ^2 (ρ is measured from state tomography) after the RB sequence;

We can compute it from sum_i ^2 where i is over all n-qubit paulis

Example of circuits names:
1 qubit (3 circuits):
0 rb_purity_Z_length_3_seed_2
1 rb_purity_X_length_3_seed_2
2 rb_purity_Y_length_3_seed_2

2 qubits (9 circuits):
0 rb_purity_ZZ_length_2_seed_1
1 rb_purity_XZ_length_2_seed_1
2 rb_purity_YZ_length_2_seed_1
3 rb_purity_ZX_length_2_seed_1
4 rb_purity_XX_length_2_seed_1
5 rb_purity_YX_length_2_seed_1
6 rb_purity_ZY_length_2_seed_1
7 rb_purity_XY_length_2_seed_1
8 rb_purity_YY_length_2_seed_1

(where Z stands for do nothing, X for Rx(pi/2) and Y for Ry(pi/2) )

When generating the purity RB sequences I verify that:

All the patterns in rb_pattern have same dimension
The length_multiplier is none

The output of rb.randomized_benchmarking_seq is: rb_purity_circs, xdata, npurity
where:
        circuits_purity: list of lists of lists of circuits for purity rb
        (separate list for each seed and each of the 3^n circuits)
        npurity: the number of purity rb circuits (per seed)
        which equals to 3^n, where n is the dimension

Summarizing the calculation of the purity Tr(rho^2):
Assume that we have n=2 qubits, then we made 9=3^n experiments and got 4=2^n expectation values per experiment:
Purity_ZZ: <II> <IZ> <ZI> <ZZ>
Purity_XZ: <II> <IZ> <XI> <XZ>
Purity_YZ: <II> <IZ> <YI> <YZ>
Purity_ZX: <II> <IX> <ZI> <ZX>
Purity_ZY: <II> <IY> <ZI> <ZY>
Purity_XX: <II> <IX> <XI> <XX>
Purity_XY: <II> <IY> <XI> <XY>
Purity_YX: <II> <IX> <YI> <YX>
Purity_YY: <II> <IY> <YI> <YY>
So, the purity is defined as the following sum of 16=4^n squares (divided by 4=2^n):
( 1 + (sum<IZ>/3)^2 + (sum<IY>/3)^2 + (sum<IX>/3)^2 + (sum<ZI>/3)^2 + (sum<XI>/3)^2 + (sum<YI>/3)^2 + <ZZ>^2 + <XZ>^2 + <YZ>^2 + <ZX>^2 + <ZY>^2 + <XX>^2 + <XY>^2 + <YX>^2 + <YY>^2 ) / 4
Since <II> is 1 , and <IZ>,<IY>,<IX>,<ZI>,<XI>,<YI> appear 3 times each.

According to https://arxiv.org/abs/1604.03076 page 8:
Assuming pure depolarizing noise γ, the density matrix after n Cliffords  (starting in the ground state density matrix ρ0) is:
ρ(n) = γ^n*ρ0 + (1 − γ^n)*I/d
Then:
Tr(ρ^2[n]) = (1-1/d)*γ^(2n) + 1/d
Therefore, we fit the data to A*γ^(2n) + B and label the quantity
E = 3/4 * (1 − γ^(2/3)) as the purity error (per gate)

So, after getting alpha from fitting the standard RBFitter A*p^m+B (p=alpha),
we calculate new alpha and its error:
alpha_pur = np.sqrt(alpha)
alpha_pur_err = alpha_err / (2 * np.sqrt(alpha_pur))

And the purity error per clifford (pepc) and its error:
pepc = 3/4 * (1 - np.power(alpha_pur, 2 / 3))
pepc_err = 0.5 * alpha_pur_err / np.power(alpha_pur, 1 / 3)

https://github.com/Qiskit/qiskit-ignis/pull/218#issuecomment-508331555:

A good test of whether the code works is to look at the same circuits with coherent errors and with incoherent errors. In the first case the purity RB will give 0 error, but the error from standard RB will be non-zero. In the later case the purity and standard RB should give the same answer

So, after getting alpha from fitting the standard RBFitter A*p^m+B (p=alpha),
we calculate new alpha and its error:
alpha_pur = np.sqrt(alpha)
alpha_pur_err = alpha_err / (2 * np.sqrt(alpha_pur))

And the purity error per clifford (pepc) and its error:
pepc = (nrb-1)/nrb * (1-alpha_pur)
pepc_err = (nrb-1)/nrb * alpha_pur_err/alpha_pur
nrb = 2 ** len(qubits)

https://github.com/Qiskit/qiskit-ignis/pull/218


[$500] Web - User still can remove invalid member

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

Action Performed:

Create a new workspace or open an existing one
Open the workspace & click on "Members"
Click on 'Invite' button
Search for an invalid phone number (+2523234211)
Select the user > click on "Next" > click on "Invite"
Select the user & click on the "Remove" button

Expected Result:
Disable the checkbox for invalid entry as discuss here: https://github.com/Expensify/App/issues/24666#issuecomment-1701343822
Actual Result:
User still can remove invalid member
Workaround:
Unknown
Platforms:

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

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/94b70403-971d-4e24-a6e0-fcf49a773304
https://github.com/Expensify/App/assets/93399543/1025c00b-0e3b-435f-a4c5-06f78b1fa36d
Expensify/Expensify Issue URL:
Issue reported by: @DylanDylann
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1695099558494749
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01c2f1b5f7d79b96d2
Upwork Job ID:<PHONE_NUMBER>090893312
Last Price Increase: 2023-09-19
Automatic offers:

cubuspl42 | Reviewer | 26897979
DylanDylann | Contributor | 26897980
DylanDylann | Reporter | 26897981

Proposal
Please re-state the problem that we are trying to solve in this issue.
Still can remove invalid member
What is the root cause of that problem?
We don’t disable checkbox if this member is invalid
What changes do you think we should make in order to solve the problem?
We should disable checkbox of faulty users and even checkbox all if there are no other members
https://github.com/Expensify/App/blob/cb73cde0f0fe602b19eb924a4f8b11f759b079e9/src/pages/workspace/WorkspaceMembersPage.js#L311
In 2 places, we should add || !_.isEmpty(policyMember.errors) to disable checkbox button if there is an error
Result
https://github.com/Expensify/App/assets/141406735/f14594bb-6649-4339-98b8-b9031107f3ac

I wan to discuss this detail with you through slack.

@cubuspl42 @sophiepintoraetz This issue is external but there are no C+ contributors assigned

Triggered auto assignment to Contributor-plus team member for initial proposal review - @cubuspl42 (External)

@DylanDylann It looks like it! This issue is in my queue, I'll review the proposal soon.

Indeed, the typical interactions of the element with reduced opacity and an error are likely expected to be disabled, like in the case of a message with an error. The proposal by @DylanDylann makes sense.
C+ reviewed 🎀 👀 🎀

Waiting on confirmation from @jasperhuangg

@jasperhuangg How I can move forward on this issue?

Sorry for the delay here, @DylanDylann's proposal makes sense to me, let's move forward!

@cubuspl42 PR https://github.com/Expensify/App/pull/27991 is ready for review

We requested a review from the internal engineer on 2023-09-27 but It seems jasperhuangg is busy on some days. It caused the timeline to last more than 3 days. So I think we are eligible for timeline bonus

@DylanDylann

So I think we are eligible for timeline bonus

It's always up to the Expensify team to assign a bonus on a case-by-case basis, and formally, the PR should be put on hold while the clock is still running. It's better to just sum up your case and ask if the delay could be classified as a hold, than state that we are eligible 🙂

@cubuspl42 Thanks so much for your reminder.

It's better to just sum up your case and ask if the delay could be classified as a hold

This is exactly what I mean. Updated

FWIW, I agree - the delay came on our end so I think this qualifies for an urgency bonus!

Hey @anmurali - I'm headed OOO for the week so all that needs doing is payments on 12 Oct. This issue is eligible for the urgency bonus so the totals are:
@cubuspl42 requires payment offer (Reviewer) - $750
@DylanDylann requires payment offer (Contributor) - $800 ($50 for the reporting bonus and $750 for the contributor work)

The PR that introduced the bug has been identified. Link to the PR:

It seems that the bug was present in the relevant components since they were created

The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

N/A

A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

No need for additional discussion

Determine if we should create a regression test for this bug.

Up to the QA team

If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Create or open an existing workspace.
Click on the "Members" section within the workspace.
Press the 'Invite' button.
Input an invalid phone number, such as "+2523234211" in the search bar.
Choose the found user, continue by clicking on "Next", and then click on "Invite".
Note that both the checkbox associated with +2523234211 and the 'Select All' checkbox are disabled.

all set here.


Group terraform files into folder

Changes in this PR:
Group terraform files into terraform folder to help navigating the repo more easily.
Have adjusted the reference to main.tf in the readme to point to this folder now.
Before merging:
I imagine we need to point something towards the moved main.tf file? So we should check this first.
UPDATE: Okay so all the builds failed I imagine since the main.tf is no longer being picked up. First time working with TF so do shout if you know what to do - looking at docs in the meantime
Note on terraform lint:
To make the commit where I moved the .tf files, I had to skip verification with --no-verify since terraform lint was complaining about the following:

Are these things we should be worried about? If not, can someone explain why and help me to add them to the ignore list 🙏
Trello Card:
https://trello.com/c/RqyUEeFz/380-reorganise-enrichment-repo-to-help-navigating-it

This would fail unless the CICD is amended to look at the new location. The CICD and invoking terraform go hand in hand.

This would fail unless the CICD is amended to look at the new location. The CICD and invoking terraform go hand in hand.

yeah I realised I need to either cd into terraform, or adjust terraform commands to be terraform -chdir terraform .....

Closing in favour of new PR due to this one becoming outdated: move all terraform files into a terraform folder: https://github.com/nationalarchives/ds-caselaw-data-enrichment-service/pull/189


Update vintage chroma

Fixed an error that prevents app from compiling due to clashing label attribute.

I suspect the problem you were facing was due to your onClick method implementation. You were probably calling removeListener there. To fix this kind of error you need to iterate the list in a reverse order.


[New game]: RETURN 👾

🎮 Game Request
Game logic and basic description

You were one of the select few chosen to trial a new hyperspace jump drive.  Space and time warps around you, suddenly there is a bang and the ship drops out of hyperspace.
'This isn’t the correct destination' you think to yourself, the jump drive must be faulty. Return is a game about finding your way back home after being lost amongst the stars. ('Home' is the blue planet of course, a.k.a Earth).

Point down the features
Game points
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @0xabdulkhalid !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign


IC-1241/ Attendance-no

What does this pull request do?
Jira card URL:
https://dsdmoj.atlassian.net/browse/IC-1241
Changes in this PR:
Update attendance, when its NO to match the latest design

Page counter- caption - Note to devs: counter to change to 1 of 1 WHEN radio button pressed No
second page is the review "check your answers" form

Screenshots of UI changes:

Hmm, this change seems to have snuck in, or did you pair with David on this? I think it's now part of the main branch - would you feel comfortable removing this change from the PR? Let me know if you need a hand.


fix: add default value to get method from config

@assaftal391
@HananyaS
Review is still pending 🤣


GPU support

Feature description
Is there anyway to paralelize computations using cuda on GPU at the moment?
If not, is there a plan to add it in the near future?

hey @prusu01 thanks for reaching out.  Your questions really are about a more deeper level than what we did on the toolkit.  The toolkit is more of a wrap around for the helib library so it can be easily used.  We don't support this much anymore but it's still available for download if someone wanted to work with it.  We have moved on to the next step of our SDK called HElayers. https://ibm.github.io/helayers/.  If you specifically are looking for HELib improvements, I would reach out the helib GitHub maintainers and see what they have plans for on their side.  Or you can check in with the


Expected behavior
Downloading fails, the app should check if internet is avaiable. IF it is not, wait x seconds and retry on that chapter.
I am not sure how it should behave if the download fails and internet is avaiable.
Actual behavior
With no internet the download fails and the app tries to download the next chapter repeating this process endelessly.
LNReader version
1.1.18
Android version
A8
Device
Huawei p8 lite 2017
Other details
https://github.com/LNReader/lnreader/assets/75091899/ecddc1b4-48d9-43d0-805c-c1b5e1ce702a
Also.. probably related to #403
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with a source, I should be opening an issue in the sources repository.
[X] I have updated the app to version 1.1.18.
[X] I will fill out all of the requested information in this form.

I just noticed that if it happens it would normally spam notifications with the related sound for EVERY chapter failed. I turned off this previously so i didn't notice:

https://github.com/LNReader/lnreader/assets/75091899/7b69453a-2f03-448d-b4b4-cf92d819535a

in v2 app, if no internet app would throws errors on notifications, and chapters would not be marked as downloaded.
I think if your connection is slow app would still keep connection until it reaches timeout.
If you turn off internet yourself, cant help.

Tachiyomi download controller doesnt have that flaw.
Just improve the download controller.
Can you re-open it?

I present you "my" wobbly connection:
https://github.com/user-attachments/assets/fe95b535-df8a-4988-a624-592dc2faa53d
The download queue is not linear. These are missed chapters the download controllee skips due to connection problems.
Imagine users with a weak and limited connection downloading chapters "for the winter" and then discover there are undownloaded stuff (holes).
It can be a problem even when making epubs since they can be shared with others.


goPackages: add ide-like packages

Things done:

[X] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
Built on platform(s)

[X] NixOS
[ ] OS X
[X] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

More
This adds the following packages (used by vim-go):

errcheck
gotool
gometalinter
motion
shlex
asmfmt

Please note, that points are not mandatory, but rather desired.

LGTM


Python test generation error

Description
Python test generation error is showing for quite simple python functions, including examples on utbot.org
To Reproduce

Install one of the latest plugin builds from main  in IntelliJ Idea 2022.2.3
Install Python plugin from MarketPlace
Create a Python project
Add the following code to it:

def print_hi(name):
    # Use a breakpoint in the code line below to debug your script.
    print(f'Hi, {name}')  # Press Ctrl+F8 to toggle the breakpoint.

def div(a, b):
    return a / b

# Press the green button in the gutter to run the script.
if __name__ == '__main__':
    print_hi('PyCharm')

Press Alt+Shift+U in code
Select both functions and generate tests
Agree to install suggested requirements
Make 5-6 steps once again

Expected behavior
Tests are supposed to be generated.
Actual behavior
An error dialog is show with the following test:
Cannot create tests for the following functions: print_hi, div
Visual proofs (screenshots, logs, images)

Environment
Windows 10 Pro
IntelliJ IDEA 2022.2.3 (not latest)
Additional context
Same with pytest or increased timeouts or manually create test sources root.
Generating tests... is displayed in inline progress.

OK, I reproduced this bug. It happens only at the first plugin run. If I restart IDEA after creating python project, I can generate tests


Add javadoc and unit tests for SubscriptionInfo

Coverage increased (+0.7%) to 81.959% when pulling d959a3e413ad0da19e83bd64152a6343bd38f866 on mziccard:pubsub-subscriptioninfo-test into 881a078973929c102eaf70a598443a2ef5ad9f59 on GoogleCloudPlatform:pubsub-alpha.


Windows specific io_handle

Commit Message: In preparation for #18305, I split the implementation of io_socket_handle on different platforms to make the code more readable and separated. The new Windows class leverages the common implementation and now is responsible for registering the events.
This will allow us to cleanly implement MSG_PEEK by read in a consequent PR in a self-contained way.
Additional Description:
Risk Level: Low (null-op)
Testing: Unit tests
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: Windows only uses a different class for io_handle
Signed-off-by: Sotiris Nanopoulos<EMAIL_ADDRESS>

/retest

/retest

LGTM, thanks for helping on this again!


Reduce margin

80 characters followed by a newline will result in an extra blank line in an 80 character cmd terminal. Since this is the default width, we should reduce the margin to 79 characters.
#525

This is a massive change, so review carefully :stuck_out_tongue_winking_eye:


RHIDP-1247: Dynamic plugin Helm install and RHIDP-1487: Chapter 5 restructure

Associated Jira tickets:
RHIDP-1247
RHIDP-1487
rh-pre-commit.version: 2.2.0
rh-pre-commit.check-secrets: ENABLED

@linfraze Please resolve conflicts in the PR.

@linfraze Please rebase your branch


Health checks, status, MetaLog

Rainbow Gum needs a way to collect errors for microservices that do health checks.
Logback does this I think with something called Status or Status Manager.
Not sure Log4j2 does but probably similar.
Currently catastrophic failures are just System.err but if we provide resilient outputs we will want to keep track of this.

Allow custom encoder and custom output for MetaLog.


STORM-2386 Fail-back Blob deletion also fails in BlobSynchronizer.syncBlobs

Please refer STORM-2386 for more details.
This is against master branch,

1.x-branch: #1975

+1


⚠ Auth has degraded performance

In 24a8cdc, Auth (https://auth.k2lin.com) experienced degraded performance:

HTTP code: 200
Response time: 8873 ms

Resolved: Auth performance has improved in 35d6cc0.


1.5.2 Detected as malware

More specifically, my antivirus (Bullguard) detects is as a malware: Trojan.Generic.22695465
I can still run version Beta 1.4.2 without problem.

Same here. Got 1.5 working fine, when updating to 1.5.2 nod32 detects it as generik.leeakmq

Windows defender detecting 1.5.2 as Trojan Win32/Bitrep.A and Trojan Win32/Triggre!plock
Reference imgur images:  1 2

same thing

Also being flagged by Norton. Detected as Trojan.Gen (https://www.symantec.com/security_response/writeup.jsp?docid=2010-022501-5526-99)

I'm also seeing this.

https://www.virustotal.com/de/file/8b05963e50917a2179d77bca8696d86a80cfcdc0c7f8cf1684dfcd8562354d62/analysis/1514105605/

I haven't looked through the new code since it was released, but these appear to be potentially false positives. They are all identifying LaunchBuddy as Generic Trojans, not as a specific trojan.
That said, there is likely something going on with the auto download of new updates alongside the crash log analyzer that is causing this to be flagged, and some work could be done to prevent the false positive.

This is quite annoying because windows defender doesn't seem to honor the
request to ignore it and unquarantine it. This results in continual
deletion of the binary. I haven't found a way to tell windows defender to
allow it permanently.
On Dec 28, 2017 8:07 AM, "KairuByte"<EMAIL_ADDRESS>wrote:

I haven't looked through the new code since it was released, but these
appear to be potentially false positives. They are all identifying
LaunchBuddy as Generic Trojans, not as a specific trojan.
That said, there is likely something going on with the auto download of
new updates alongside the crash log analyzer that is causing this to be
flagged, and some work could be done to prevent the false positive.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/TheCheatsrichter/Gw2_Launchbuddy/issues/56#issuecomment-354312032,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA1Y30jYlwykAu0kz49vesHnuFhUqpi8ks5tE7zTgaJpZM4QzWUD
.

I'm assuming you've attempted this: https://support.microsoft.com/en-us/help/4028485/windows-10-add-an-exclusion-to-windows-defender-antivirus
That said, I'll see what can be done. If I can get a decent auto updater in place to keep with the clean approach that is not detected as a trojan, I'll push the code. If I can't, I'll likely roll back to the old update method... which I hated, but hey it didn't break things!

Yea that's what I tried, and it works for a day or two, and then it detects
it as a different "generic" trojan, and deletes it again.
On Thu, Dec 28, 2017 at 4:56 PM, KairuByte<EMAIL_ADDRESS>wrote:

I'm assuming you've attempted this: https://support.microsoft.com/
en-us/help/4028485/windows-10-add-an-exclusion-to-windows-
defender-antivirus
That said, I'll see what can be done. If I can get a decent auto updater
in place to keep with the clean approach that is not detected as a trojan,
I'll push the code. If I can't, I'll likely roll back to the old update
method... which I hated, but hey it didn't break things!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/TheCheatsrichter/Gw2_Launchbuddy/issues/56#issuecomment-354382055,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA1Y3yRDko4Fji2o5WyCEH7r0DwCJ_h6ks5tFDjQgaJpZM4QzWUD
.

It appears the updater alone is not the main problem, it's likely an interaction between different sections of the code causing the false positive.
This release which is based off an admittedly older version (1.4.2) should have some improvements over 1.4.2 but will not have the features of 1.5+. It also registers as Trojan.Win32.Generic on only 2 of 67 scanners.
You can use that version if you are willing to deal with a different (and lesser) feature set than 1.5+ as well as more bugs (the code is incomplete, essentially alpha).
If enough people use it and report any bugs I can look into getting it "complete" and pulling in 1.5 features, checking virustotal along the way.

That one isnt even slightly descriptive, so I have no idea why it would be throwing it. At least it's an obscure scanner and not a widely used one.
As for the main problem, I think we have likely narrowed down the problem, and a future version should handle it without the need to compile it the way it was to fix the problem (debug vs release).

if it helps more, this is what Windows Defender identifies it as: https://go.microsoft.com/fwlink/?linkid=142185&name=<Trojan:Win32/Tilken.B!cl>&threatid=<2147722740>

or a better online scanner:
http://nodistribute.com/result/cyO2UGBgVnqED8Y3NMdlR7aT

Is that scan using 5.2 or the latest release?

both were the latest release

I currently can't run any version of launchbuddy. The .exe gets blocked in build 1.4.3.a1 and 1.5.2 when I try to build them. Antivirus(Bullguard) notes it as Malware:  Gen:Variant.Ursu.80032

Version 1.6 is a merge between Version 1.4.2 and the refactor branch. So far not AV flags

It is only Avira that flags 1.6.0 and 1.6.1. though. The rest says good for it.

The newest version is up to quite a lot of flags again.
1.6.2
23/68

Down to 16/66 now.


computation of base pathname

Fixed computation of base pathname to trim subpath off of the end of the originalUrl, not just removing it everywhere in the originalUrl.  Error was visible in the JSON from http GET of the base.  Also handle case where originalUrl was null, as it is in the README.md example.

Thanks!


MIMIC-IV FHIR Import Crash - Could not initialize class org.apache.commons.text.lookup.StringLookupFactory

Hello,
I'm following the instructions at https://github.com/kind-lab/mimic-fhir/blob/main/tutorial/mimic-fhir-tutorial-pathling.ipynb to import the MIMIC-IV FHIR ndjson into Pathling using the docker image here: https://hub.docker.com/r/aehrc/pathling. When running the following script:
from pathlib import Path
import requests
import json
import ndjson
import pandas as pd

import matplotlib.pyplot as plt
plt.style.use('ggplot')
plt.rcParams.update({'font.size': 20})

from fhirclient.models.parameters import Parameters, ParametersParameter
from py_mimic_fhir.lookup import MIMIC_FHIR_PROFILES

import_folder = 'file:///usr/share/staging'
server = 'http://localhost:8080/fhir'

def generate_import_parameters(import_folder, profile, resource, mode):
    param_resource = Parameters()

    param_resource_type = ParametersParameter()
    param_resource_type.name= 'resourceType'
    param_resource_type.valueCode = resource

    param_url = {}
    param_url['name'] = 'url'
    param_url['valueUrl'] = f'{import_folder}/{profile}.ndjson'

    param_mode = ParametersParameter()
    param_mode.name= 'mode'
    param_mode.valueCode = mode

    param_source = ParametersParameter()
    param_source.name = 'source'
    param_source.part = [param_resource_type, param_url, param_mode]
    param_resource.parameter = [param_source]

    return param_resource.as_json()

def post_import_ndjson(server, param):
    url = f'{server}/$import'

    resp = requests.post(url,  json = param, headers={"Content-Type": "application/fhir+json"} )
    return resp

mode = 'merge' # overwrite for fresh load (but not really since need to merge Observations not overwrite)

for profile, item in MIMIC_FHIR_PROFILES.items():
    resource = item['resource']
    # ObservationChartevents too large and crashing all the observation searches
    if (profile != 'ObservationChartevents'):
        param = generate_import_parameters(import_folder, profile, resource, mode)
        resp = post_import_ndjson(server, param)
        print(f"{profile}: {resp.json()['issue'][0]['diagnostics']}")

I get the error:
mimic-pathling-1  | 20:31:40.214 [Executor task launch worker for task 1.0 in stage 34.0 (TID 456)] [] ERROR org.apache.spark.executor.Executor - Exception in task 1.0 in stage 34.0 (TID 456): Could not initialize class org.apache.commons.text.lookup.StringLookupFactory
mimic-pathling-1  | 20:31:40.214 [Executor task launch worker for task 38.0 in stage 34.0 (TID 493)] [] ERROR o.a.s.s.e.d.FileFormatWriter - Job job_202403182028225607545436317964348_0034 aborted.
mimic-pathling-1  | 20:31:40.215 [Executor task launch worker for task 38.0 in stage 34.0 (TID 493)] [] ERROR org.apache.spark.util.Utils - Uncaught exception in thread Executor task launch worker for task 38.0 in stage 34.0 (TID 493)
mimic-pathling-1  | java.lang.NullPointerException: null
mimic-pathling-1  |     at org.apache.spark.scheduler.Task.$anonfun$run$3(Task.scala:144)
mimic-pathling-1  |     at org.apache.spark.util.Utils$.tryLogNonFatalError(Utils.scala:1509)
mimic-pathling-1  |     at org.apache.spark.scheduler.Task.run(Task.scala:142)
mimic-pathling-1  |     at org.apache.spark.executor.Executor$TaskRunner.$anonfun$run$3(Executor.scala:554)
mimic-pathling-1  |     at org.apache.spark.util.Utils$.tryWithSafeFinally(Utils.scala:1529)
mimic-pathling-1  |     at org.apache.spark.executor.Executor$TaskRunner.run(Executor.scala:557)
mimic-pathling-1  |     at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
mimic-pathling-1  |     at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
mimic-pathling-1  |     at java.base/java.lang.Thread.run(Thread.java:829)
mimic-pathling-1  | 20:31:40.215 [Executor task launch worker for task 38.0 in stage 34.0 (TID 493)] [] ERROR org.apache.spark.executor.Executor - Exception in task 38.0 in stage 34.0 (TID 493): Could not initialize class org.apache.commons.text.lookup.StringLookupFactory

Any help or guidance would be greatly appreciated.

Thanks for sending this through - let me try to reproduce and I will get back to you.

Hi @cyrilzakka,
I've had a go at loading the full MIMIC-IV data set to reproduce your problem.
I was unable to reproduce the exact error you described, but I did come across a couple of data quality issues that would make it impossible to import this data set in its current form.
I have documented them here:

kind-lab/mimic-fhir#91
kind-lab/mimic-fhir#92

I am also hanging out for these issues to be resolved so that I can start playing around with this wonderful data set!
I did fix the former issue using a script (attached to the issue), but the second issue is a bit more involved to fix using post-processing and I have not been able to get to a fully importable data set yet.


Rule prohibiting use of object and array literals in props

By analogy to jsx-no-lambda and for same reasons. Closes #89.

Thanks for your interest in palantir/tslint-react, @smoogly! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

I'm not signing the CLA. You're free to do what you like to this PR.

Closing this out as unfortunately without signing the CLA we are unable to merge this PR. Feel free to reopen it if you change your mind.

Did your policy change in the last two months?
https://github.com/palantir/tslint-react/pull/168#issuecomment-396028111

I did not approve that PR - not sure what the policy was at that point but as far as I am aware this is the policy now. Not sure what your reasoning is for not signing the CLA, but it is no different to many other companies that manage open source software (Microsoft has a CLA on TypeScript itself, as an example).


java.lang.NoSuchMethodError: lmcoursier.definitions with sbt 1.3

Since we've switched to sbt 1.3.2, I'm getting this error. Might be related to https://github.com/sbt/sbt/issues/5040
sbt:almond> compile
[error] stack trace is suppressed; run last protocol / coursierResolutions for the full output
[error] stack trace is suppressed; run last channels / coursierResolutions for the full output
[error] stack trace is suppressed; run last logger / coursierResolutions for the full output
[error] stack trace is suppressed; run last interpreter-api / coursierResolutions for the full output
[error] stack trace is suppressed; run last scala-interpreter / coursierResolutions for the full output
[error] stack trace is suppressed; run last kernel / coursierResolutions for the full output
[error] stack trace is suppressed; run last interpreter / coursierResolutions for the full output
[error] stack trace is suppressed; run last almond-rx / coursierResolutions for the full output
[error] stack trace is suppressed; run last scala-kernel-api / coursierResolutions for the full output
[error] stack trace is suppressed; run last jupyter-api / coursierResolutions for the full output
[error] stack trace is suppressed; run last almond-spark / coursierResolutions for the full output
[error] stack trace is suppressed; run last test / coursierResolutions for the full output
[error] stack trace is suppressed; run last echo / coursierResolutions for the full output
[error] stack trace is suppressed; run last scala-kernel / coursierResolutions for the full output
[error] stack trace is suppressed; run last coursierResolutions for the full output
[error] (protocol / coursierResolutions) java.lang.NoSuchMethodError: lmcoursier.definitions.ToCoursier$.project(Llmcoursier/definitions/Project;)Lcoursier/core/Project;
[error] (channels / coursierResolutions) java.lang.NoSuchMethodError: lmcoursier.definitions.ToCoursier$.project(Llmcoursier/definitions/Project;)Lcoursier/core/Project;
[error] (logger / coursierResolutions) java.lang.NoSuchMethodError: lmcoursier.definitions.ToCoursier$.project(Llmcoursier/definitions/Project;)Lcoursier/core/Project;
[error] (interpreter-api / coursierResolutions) java.lang.NoSuchMethodError: lmcoursier.definitions.ToCoursier$.project(Llmcoursier/definitions/Project;)Lcoursier/core/Project;
[error] (scala-interpreter / coursierResolutions) java.lang.NoSuchMethodError: lmcoursier.definitions.ToCoursier$.project(Llmcoursier/definitions/Project;)Lcoursier/core/Project;
[error] (kernel / coursierResolutions) java.lang.NoSuchMethodError: lmcoursier.definitions.ToCoursier$.project(Llmcoursier/definitions/Project;)Lcoursier/core/Project;
[error] (interpreter / coursierResolutions) java.lang.NoSuchMethodError: lmcoursier.definitions.ToCoursier$.project(Llmcoursier/definitions/Project;)Lcoursier/core/Project;
[error] (almond-rx / coursierResolutions) java.lang.NoSuchMethodError: lmcoursier.definitions.ToCoursier$.project(Llmcoursier/definitions/Project;)Lcoursier/core/Project;
[error] (scala-kernel-api / coursierResolutions) java.lang.NoSuchMethodError: lmcoursier.definitions.ToCoursier$.project(Llmcoursier/definitions/Project;)Lcoursier/core/Project;
[error] (jupyter-api / coursierResolutions) java.lang.NoSuchMethodError: lmcoursier.definitions.ToCoursier$.project(Llmcoursier/definitions/Project;)Lcoursier/core/Project;
[error] (almond-spark / coursierResolutions) java.lang.NoSuchMethodError: lmcoursier.definitions.ToCoursier$.project(Llmcoursier/definitions/Project;)Lcoursier/core/Project;
[error] (test / coursierResolutions) java.lang.NoSuchMethodError: lmcoursier.definitions.ToCoursier$.project(Llmcoursier/definitions/Project;)Lcoursier/core/Project;
[error] (echo / coursierResolutions) java.lang.NoSuchMethodError: lmcoursier.definitions.ToCoursier$.project(Llmcoursier/definitions/Project;)Lcoursier/core/Project;
[error] (scala-kernel / coursierResolutions) java.lang.NoSuchMethodError: lmcoursier.definitions.ToCoursier$.project(Llmcoursier/definitions/Project;)Lcoursier/core/Project;
[error] (coursierResolutions) java.lang.NoSuchMethodError: lmcoursier.definitions.ToCoursier$.project(Llmcoursier/definitions/Project;)Lcoursier/core/Project;

If it can be of any help, a working temporary solution is to change sbt version to 1.2.8.

The coursier-based sbt launcher is required for now. The most straightforward way of using it is by using the custom sbt-extras script. I committed it as sbt at the root of the almond sources.
This is required as the build is relying on some stuff from sbt-coursier, and sbt-coursier conflicts with sbt 1.3 ’s own coursier stuff.
It may be possible to get rid of sbt-coursier in the almond build, that should just require rewriting some stuff in project/Settings.scala.

Ah thanks, with the coursier based launcher it works fine. I suggest to leave this open though until we either get rid of sbt-coursier or have documented that you have to use the custom launcher.

Hello, I would like to investigate almond code base. Simple ide functionality (jump-to-definition, find-usages) is very useful here. But I am unable to open this project in intellij due to this issue. Custom launcher run from command line works fine. I was trying to make intellij use custom runner but without success...
Do you know a way how to open it in intellij? Or what ide do you use? Thanks!

@james64 Switching temporarily to sbt 1.2.8 as suggested above by @lucag should help getting it working in intellij again.
Set sbt.version=1.2.8 in project/build.sbt

I get this on any project using an sbt version higher than 1.2.8. I had to pin all projects to this older version. Is there any other way to use 1.3.x besides using this custom sbt runner?

Fixed by #563


Add Fabamaq and Gaming category

Created new category to include FMQ core business, Game Development.

Thanks @ruicarest! :star:
BTW, I replaced the emoji, sorry. :)


Support capitalized github organizations

Github supports capitalized orgs, docker does not. Lowercasing
everything makes it work.

Awesome! :+1:


fixed navs

Description
Fixed Nav
Why was this work done?

What is the purpose of the feature?

What feature / user story is it for?
(Link if possible)
Any relevant links or images / screenshots
Type of change
Please delete options that are not relevant.

New feature (non-breaking change which adds functionality)

Testing

Has this feature been tested before conflict?
Has this feature been tested after conflict?

Change Status

Completed, ready to review and merge

Checklist

My code follows the style guidelines of this project
I have performed a self-review of my own code
My code has been reviewed by at least one peer
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
My changes generate no new warnings
There are no merge conflicts

Note values are hardcoded for roles. Ideally we want to have them looked up in a future revision.


Add custom environment variables to a Function's YAML

Is this a BUG REPORT or FEATURE REQUEST?: FEATURE REQUEST
What happened: Could not add custom environment variables to a Function's YAML file.
What you expected to happen: To be able to add custom environment variables to Function's YAML file so that functions could be managed through HelmRelease. The function in question requires a private registry to be configured, therefore the deployment requires environment variables. More specifically, I'm attempting to reproduce [1] CLI is not an option, we need reproducible deployments with HelmReleases.
[1] https://github.com/kubeless/kubeless/issues/933#issuecomment-435598915
How to reproduce it (as minimally and precisely as possible):
apiVersion: kubeless.io/v1beta1
kind: Function
metadata:
  name: test
  label:
    created-by: kubeless
    function: test
spec:
  runtime: nodejs9
  handler: handler.test
  deps: |
    {
      "name": "test-function",
      "version": "0.0.1",
      "description": "Test function in Kubeless.",
      "dependencies": {
        "application": "^1.0.5"
      },
      "publishConfig": {
        "registry": "https:/private-registry-host.com/"
      }
    }
  function: |
    module.exports = {
      legist: function (event, context) {
        return "This is just a dummy function."
      }
    }

Anything else we need to know?: I want to add the following environment variables to the function, so that the dependency "application": "^1.0.5" can be resolved and installed.
              - name: NPM_REGISTRY
                value: "https://private-registry-host.com/"
              - name: NPM_SCOPE
                value: "@scope"
              - name: NPM_CONFIG_EXTRA
                value: "..."

Environment:

Kubernetes version (use kubectl version): 1.17.0
Kubeless version (use kubeless version): 2.1.1

apiVersion: helm.fluxcd.io/v1
kind: HelmRelease
metadata:
  name: kubeless
  namespace: kube-system
spec:
  chart:
    repository: https://kubernetes-charts-incubator.storage.googleapis.com/
    name: kubeless
    version: 2.1.1
  releaseName: kubeless
  values:
    rbac:
      create: true
    ui:
      enabled: false
      service:
        type: ClusterIP

Cloud provider or physical cluster: On-premise Intel blade servers

Thanks!

Hi @zzvara, you should be able to add those env vars. Check the NodeJS section here: https://kubeless.io/docs/runtimes/. If you want to get the YAML of the function to deploy you can execute kubeless function deploy ... --dryrun -o yaml and copy the result.
Also, I don't recommend using the Helm chart from the incubator repository, that's deprecated. Use the manifests that you can find in the "releases" page of this repo.

Hi @andresmgot, everything you just said is very useful! I'll report back my findings ASAP.

I was pretty close. This is how it should look like:
spec:
  deployment:
    spec:
      template:
        spec:
          containers:
          - env:
            - name: NPM_REGISTRY
              value: "https://...."
            - name: NPM_SCOPE
              value<EMAIL_ADDRESS>            - name: NPM_CONFIG_EXTRA
              value: "whatever"
            imagePullPolicy: Always
            name: ""


Initial Addition of gevent

closing as the additions i'd need to make would likely unnecessarily bog down this example repo.


Bump Stack to 10.6 and fix template haskell issues

Ran the tests locally, they worked, and it's compiling fine for platform. Any further fixes if needed will be directed here.


Can't read dumped BIOS from memory card

I actually have a memory card that gives me the same issues discussed in this thread on the ngemu forum in 2013. This probably isn't the same card this person is talking about, but it gives me the same symptoms. Format card with my fat PS2, make sure its blank, plug it into slot 1 of the PSone I want to dump the BIOS of. It dumps successfully. I plug it back into the PS2, and... wLaunchElf shows the card is completely empty. The PS2 Browser says its unformatted. Put it back in the PSone without formatting it beforehand with the dumper already booted up, and the dumper also says its unformatted.
The memory card in question is a generic "RGEEK High Speed 1 Mega PS1 Memory Card". I purchased it on Amazon recently.
 I wanted to use this card I got so that I can make use of tonyhax + FreePSXBoot on the other PS1 memory card I have in my possession: another generic, red card with an SJ logo on it that is probably an actual 1Mb-sized memory card (not tested, but it does work for the FreePSXBoot version of tonyhax).
I got the Constructor game and booted it up on my PSone. I start a game, press the select button and save. The game is supposed to use a whole, standard-sized 15-block (1M) memory card. And conveniently enough, the game counts the blocks it is in the progress of saving. What was interesting to me is that with this RGEEK card, it only gets up to 9. It appears to hang a bit, then jumps to verifying the save. Super handy for testing, right? But funnily enough, it claims success. I quit the current game to go back to the main menu so that I can load the save I just made. The game lists the save I just made. I select it, and after a couple of seconds it tells me the card is unformatted.
I repeat these steps over and over on my PSone and slim PS2. Sometimes I reset after saving and check the memory card in the browser or memory card menu. PS2 says it's unformatted, game says it's unformatted on PSone. Fat PS2 says it's unformatted. I cannot boot up Constructor on the fat PS2.
My guess is that this is a card that only really has 8 blocks to work with, and not 15. Either all these RGEEK cards are like this, or I have a lemon. If anyone else has access to this card, can you test it out with Constructor and post your results?

Yes. There have been an influx of fake 64K cards lately. https://psx0.wordpress.com/2022/04/01/fake-ps1-memory-cards-the-great-aliexpress-amazon-ebay-swindle/

Ah, that explains a lot. I have since ordered some legit PS1 cards to use instead of this one, so I should be able to get cracking on successfully dumping that BIOS in the near future. Good to know it seems to actually be 7 workable blocks rather than 8. I am glad I have always backed up my PS1 and PS2 saves up often (and back up the back ups). Caveat emptor.
Would it be worth mentioning that problems like these ones likely mean your card is less than 15 blocks in the Readme? Thanks again for your wonderful tool, ShendoXT. It makes dumping the PS1 BIOS simple and easy.


Subscript 'subscript(_:)' requires that 'Float' conform to 'Value'

Issues are used to track bugs and feature requests.
Need help or have a general question? Ask on Stack Overflow (tag sqlite.swift).

Build Information
SQLite.swift version : 0.15.3
XCode : 16
Swift Package manager
I'm trying to select a field of type Float but I get an error assigning it to a variable.
The field is defined as:
let iAmount = Expression("Amount");
When I try to populate a variable, I this the error:
Subscript 'subscript(_:)' requires that 'Float' conform to 'Value'
Below is the code:
for ri in try db.prepare(dtRecipeIngredients.order(iName).where(iUUIDRecipe == r[rUUID]))
{
let amount : Float = ri[iAmount]
    let newIngredient = Ingredient (
        Name: ri[iName]
        , Qnty: amount
    )

The error occurs at compile time, not run time.  Should I be declaring the expression for the field in a different way for a Float?

Clarification.  iAmount is declared as:
let iAmount = Expression("Amount");
The same thing will work with a Int type, but not Float.
The error occurs on this line:
let amount : Float = ri[iAmount]


windows: stub device_channel todos

#2659 may have a tiny conflict with this since it adds a field to a struct I touched - I expect it'll be easy to resolve


[lc_ctrl/dv] Updated checklist for V2S

Updated checklist for V2S

Signed-off-by: Nigel Scales<EMAIL_ADDRESS>

if all the checklist items are addressed, please also update this to V2S. We could do it in a separate PR.
https://github.com/lowRISC/opentitan/blob/5215920fc4cf7a455328682d3deff5caec1370b1/hw/ip/lc_ctrl/data/lc_ctrl.prj.hjson#L14

if all the checklist items are addressed, please also update this to V2S. We could do it in a separate PR.
https://github.com/lowRISC/opentitan/blob/5215920fc4cf7a455328682d3deff5caec1370b1/hw/ip/lc_ctrl/data/lc_ctrl.prj.hjson#L14

No problem @weicaiyang I'll do that.

awesome, thanks Nigel, let me check nightly regression tonight since we just merged PR #12324. Will come back to merge this once the numbers meet expectation.


Update Rubocop to 0.34.2

Renames Style/Blocks to Style/BlockDelimiters
Renames Style/UnneededPercentX to Style/CommandLiteral
Disables all new cops for Hound and thoughtbot config.
Does not use the new DisabledByDefault config value, which only
enables cops you specify. I left this off because we have all the cops
explicitly defined in the config already.
Updates expected violation messages to reflect new text
Tweaks a couple tests that were no longer seeing the
"Unnecessary spacing detected." message. I'm not sure what this was
even covering, given that other cops in those tests are completely
handling the whitespace (I think).
Style/RegexpLiteral no longer supports a MaxSlashes parameter, so
it's removed.

Will close #903
Potentially solves #725

Thoughts on not disabling new rules? They are fairly uncommon violation rules, and it might give us a reality check about the config changes we are doing -- given that in the future we'll just be rolling with linter defaults, thus each time we'll update the rules will change on our users.

I'm fine using the Rubocop defaults for all the new cops. I just went into this PR with thought of having the least resistance.

Pulled in Rubocop defaults for new inters in a6c8bda29d5c7e5ce432231c0c738f8b5e706d27

Looks good to me. @salbertson Thoughts?

Looks good to me

Looks good to me! :+1:


Why is there an Ord instance?

The documentation for the Ord (HashMap k v) instance says:

The ordering is total and consistent with the Eq instance. However, nothing else about the ordering is specified, and it may change from version to version of either this package or of hashable.

This makes it seem like a pretty useless instance, so I wonder why it exists in the first place?

Lets them be stored in Sets, used as Map keys, etc.

Hmm, I don't find that a very convincing reason, as I can't think of many applications for using a hashmap/hashset in a map/set key...

The fact you don't find it useful is not a reason to remove it.

Fair.


Error: You appear to be using a native ECMAScript module configuration file, which is only supported when running Babel asynchronously

Hey, thanks for your work here!
We recently switched our repo to use ESM only and then stumbled upon this error. Apparently babel only works with ESM if we use babel.transformAsync instead of babel.transformSync.

We are using

"@babel/cli": "7.23.4"
"@babel/core": "7.23.6"
"vite-plugin-babel": "1.1.3"

I'm creating a PR for this, seems like a simple fix.

I merged it and released in 1.2.0. Thx for contribution! 🙌


undefined-field and need-check-nil diagnostics don't get triggered

How are you using the lua-language-server?
NeoVim
Which OS are you using?
Linux
What is the issue affecting?
Diagnostics/Syntax Checking
Expected Behaviour
local numbers = { [1] = 1, [2] = 2, [3] = 3 } print(numbers.fd)
local numbers print(numbers.fd)
the first snippet should trigger an undefined-check-field warning and the second one a need-check-nil one.
Actual Behavior
both of the snippets don't thow any diagnostic what so ever.
Reproduction steps

try to access an undefined property of an undefined or defined object

Additional Notes
currently I have installed v3.5.3 of sumenko and v5.4.4 lua.
I also tried it in vs code with the sumenko plugin v3.5.3
Log File
No response

I only do type checking if you explicitly annotate the type.
local t = {1, 2, 3}
doSomeThing(t)
print(t.fd) --> I can't decide if this needs to provide a warning

I know Lua is very dynamic so it may not make sense in some situations, but when you really want to be careful, it could be vital. Perhaps this should be added to “strict mode” so that anything that is not explicitly defined will raise an error.

I think this feature is needed. Consider the use case where the user changes the index of a table. After this action there is no way to tell where the old indexes are accessed and it can require a lot of tedious debugging.

I think this feature is needed. Consider the use case where the user changes the index of a table. After this action there is no way to tell where the old indexes are accessed and it can require a lot of tedious debugging.

At current you can add ---@type integer[] above the local.

Yes you are right I was not aware of type annotations in lua. Another example that triggers an undefined field warning is the following
---@type { key1: string, key2: boolean }
local t = { key1 = "value1", key2 = true }
print(t.foo)

Thank you for your help!


cluster: etcd pod have owner ref

etcd pod will have owner ref to its single unique managing cluster.
Service and RS needs some refactoring work and will be pulled up later.

Can we add owner to pod, service and replica set in one pr?

@xiang90 Sure. There needs some additional refactoring work for them.

@etcd-bot retest this please

@xiang90
All added. PTAL

Can we move all the ownership attachment into k8sutil? then we do not need to worry about it in other pkgs.

Fix CI?

@etcd-bot retest this plese

@etcd-bot retest this plese

@etcd-bot retest this plese

@etcd-bot retest this please

@etcd-bot retest this please

@xiang90 PTAL

lgtm


Unicode symbols are turning into emojis

Windows Terminal version
1.20.11381.0
Windows build number
Microsoft Windows NT 10.0.22631.0
Other Software
No response
Steps to reproduce
Log unicode characters like: U+2139, U+26A0,
Expected Behavior
No response
Actual Behavior
These characters are turning into emojis:

This works fine in webstorm
Is there a way to fix or disable it?

An option for this was added to the v1.21 preview (see #16871)

That's not really what I want. It just removes colors

Thanks for filing! We're going to mark this as a /dup of #13110.


Collect usage events to support user analytics

Integrate a third party usage analytics service (e.g., Google Analytics). We should be collecting key user events, such as: faceting behavior, browsing behavior, what data is being viewed, what attributes are being used in searches, when the user has to "enable" more facets because the default list wasn't enough, how long they spend on certain views, etc. etc. etc.

We should first come up with the initial list of events we want to capture;
Come up with a plan for how we will integrate this;
Review the plan before executing on it.

Some additional constraints:

We should probably write our own wrapper API around the analytics provider (e.g., GA) so that we can integrate with different analytics vendors;
Account details (e.g., the Google Analytics account where events are sent to) should come from a configuration, it should not be hardcoded to one operators GA account.
Operators/Administrators should be able to disable user event collection entirely for their own deployments.

We will meet when I'm in town on Monday, 2/22 to go over the Google Analytics tracking we already have in place, especially events tracked by Google Tag Manager.
We will also:

Specify the metrics we want
Come up with proof-of-concept
See what resources we have (can a student do this)?

Stakeholders:

program officers: reporting to program officers
us: improving our own code, and for our own research
users: feedback to "users" that produce data or operate data hubs

Let's move this to wishlist and create more actionable issues. (If you disagree, you can always reopen this issue)


Add EngineLighting.netkan

Fixed broken KerbalStuffBot submission

Don't know what's up with Travis. This builds fine on my local netkan.

He's in hibernation but got a swift kick in the buttocks just now so he should start processing things soon'ish

#rebuild


Activity from Telegram group chat contains no mentions

The activities from group chat on Telegram channel contain no mention objects (GetMentions method) although the users are mentioned.
Here is the log from some message activity (command to bot - the way for the bot to receive the message). There are mentions (text_mentions?) in channelData, but not in the collection returned by GetMentions, which strangely returns only the bot instead:
/hi Андрей Матвей
Activity:
{
  "type": "message",
  "id": "IXNy2KlM2WV",
  "timestamp": "2017-04-03T15:13:05.0542577Z",
  "serviceUrl": "https://telegram.botframework.com",
  "channelId": "telegram",
  "from": {
    "id": "354557895"
  },
  "conversation": {
    "isGroup": true,
    "id": "-208483668",
    "name": "bot debug"
  },
  "recipient": {
    "id": "egorngrok1bot",
    "name": "ngroklocal"
  },
  "text": "/hi Андрей Матвей",
  "entities": [
    {
      "type": "mention",
      "mentioned": {
        "id": "egorngrok1bot",
        "name": "ngroklocal"
      },
      "text": "egorngrok1bot"
    }
  ],
  "channelData": {
    "update_id": 324971640,
    "message": {
      "message_id": 32,
      "from": {
        "id": 354557895,
        "first_name": "Sergey",
        "last_name": "Knyazev"
      },
      "chat": {
        "id": -208483668,
        "title": "bot debug",
        "type": "group",
        "all_members_are_administrators": true
      },
      "date":<PHONE_NUMBER>,
      "text": "/hi Андрей Матвей",
      "entities": [
        {
          "type": "bot_command",
          "offset": 0,
          "length": 3
        },
        {
          "type": "text_mention",
          "offset": 4,
          "length": 6,
          "user": {
            "id": 345018769,
            "first_name": "Andrey"
          }
        },
        {
          "type": "text_mention",
          "offset": 11,
          "length": 6,
          "user": {
            "id": 370536472,
            "first_name": "Matvei",
            "last_name": "Stepanov"
          }
        }
      ]
    }
  }
}
``

GetMentions():
`[
  {
    "mentioned": {
      "id": "egorngrok1bot",
      "name": "ngroklocal"
    },
    "text": "egorngrok1bot",
    "type": "mention"
  }
]
`

To compare, here is the log from Skype channel:

**@NGrok Say hi to @Andrey D**

Activity:
`{
  "text": "<at id=\"28:a72df016-6716-4f39-8cf2-08c61e5340a4\">@NGrok</at> Say hi to <at id=\"live:dronishe\">@Andrey D</at>",
  "entities": [
    {
      "type": "mention",
      "mentioned": {
        "id": "28:a72df016-6716-4f39-8cf2-08c61e5340a4"
      },
      "text": "<at id=\"28:a72df016-6716-4f39-8cf2-08c61e5340a4\">@NGrok</at>"
    },
    {
      "type": "mention",
      "mentioned": {
        "id": "live:dronishe"
      },
      "text": "<at id=\"live:dronishe\">@Andrey D</at>"
    },
    {
      "type": "clientInfo",
      "locale": "en"
    }
  ]
}
`

GetMentions():
`
[
  {
    "mentioned": {
      "id": "28:a72df016-6716-4f39-8cf2-08c61e5340a4"
    },
    "text": "<at id=\"28:a72df016-6716-4f39-8cf2-08c61e5340a4\">@NGrok</at>",
    "type": "mention"
  },
  {
    "mentioned": {
      "id": "live:dronishe"
    },
    "text": "<at id=\"live:dronishe\">@Andrey D</at>",
    "type": "mention"
  }
]
`

Is there a way to make GetMentions method return mentions from Telegram channel?

hi @matthewz0584 - can you provide more details about your issue?
Environment

Operating System:

Windows
macOS
Linux

SDK Platform:

C#/.NET
Node.js

SDK version:
Development environment:

localhost
Azure Web App
Azure Bot Service App
other

If you are using the Node.js SDK:

Node.js version:
NPM version:

Issue
What the issue is, in broad strokes. Link to Stack Overflow post.
Example Code
The code snippet or complete bot example (preferred) that demonstrates the issue.
Steps to Reproduce
Please provide the shortest amount of steps to reproduce your issue.
Expected Behavior
What you expected to happen.
Actual Results
What actually happened. Please give examples and support it with screenshots, copied output or error messages.

OS: Windows 7 x64
SDK: C#/.Net 4.6
Dev environment: Azure Web App
Issue
Activity.GetMentions() for activity originated from Telegram channel group chat returns wrong Mentions array: mentioned users are not there, not mentioned bot is there.
Code
    [BotAuthentication]
    public class MessagesController : ApiController
    {
        [ResponseType(typeof(void))]
        public virtual async Task<HttpResponseMessage> Post([FromBody] Activity activity)
        {
            if (activity != null)
            {
                var activityAsJson = JsonConvert.SerializeObject(activity);
                var mentionsAsJson = JsonConvert.SerializeObject(activity.GetMentions());
            }
            return new HttpResponseMessage(HttpStatusCode.Accepted);
        }
    }

Steps to reproduce
In order for the bot to receive the message at all in Telegram we use / symbol which denotes a command to bot. The message we send is (two users are mentioned, and in Telegram mentioned users are in bold font):
/hi Андрей Матвей
Expected Behavior
Given that, I expect Activity.GetMentions to return correctly filled Mention array, i.e array with two mentions for two mentioned users. Which is quite true for the similar fixture on the Skype channel (the dump of such Activity is given in original issue for reference).
Actual results
The bot receives corresponding Activity, its dump is given in original issue, but the GetMentions() method returns only one mention for the bot, while the trully mentioned users could only be found as "text_mentions" in ChannelData property.

@matthewz0584 are you still experiencing the same issue?

Yep, I am. Recent Bot framework update has fixed some issues (Activity.GetMentions throwing StackOverflowException, for example), but, alas, not mine.
Fresh dump with no mentions in Entities property and two text_mention in ChannelData:
"entities": [
{
"type": "mention",
"mentioned": {
"id": "egorbotbot",
"name": "egorkabot"
},
"text": "egorbotbot"
}
],
"channelData": {
...
"entities": [
{
"type": "text_mention",
"offset": 1,
"length": 6,
"user": {
"id": 354557895,
"first_name": "Sergey",
"last_name": "Knyazev"
}
},
{
"type": "text_mention",
"offset": 8,
"length": 6,
"user": {
"id": 345018769,
"first_name": "Andrey"
}
}
]
}
For now, we parse ChannelData, but I have no idea, why it is not parsed by framework.

open a new issue if further questions

Just want to put on record that facing exactly the same issue even in Bot framework 4.0.


Update DE to Camunda BPM 7.7.0

Changed according to the modifications in the en file: https://github.com/camunda/camunda-tasklist-translations/commit/43a649eddc65e68b825454be8ee6418921d91e5d
Issue: https://app.camunda.com/jira/browse/CAM-7921

Hi @PhilippHeuer,
I've merged this PR and fixed a small type (see https://github.com/camunda/camunda-tasklist-translations/commit/d26b875b174df428cf4cdfa2e545d202742dff37). Thanks for your contribution!
Best regards,
Mat


How do I run this code in React?

I am a beginner developer.
How do I run this code in React?

Hi. To put it simply - you do not. It is built by nodeJS, no React included.
To deploy locally, after downloading this repository, all that is needed is to run npm install in the root folder and then run npm run build. No errors should come up, only warnings. After this, all it takes is to open unitgenerator/index.html in (almost) any browser.
Hope this helps 🙂

Hi. To put it simply - you do not. It is built by nodeJS, no React included.
To deploy locally, after downloading this repository, all that is needed is to run npm install in the root folder and then run npm run build. No errors should come up, only warnings. After this, all it takes is to open unitgenerator/index.html in (almost) any browser.
Hope this helps 🙂


Update coronavirus landing page

Update the coronavirus landing page to align it with the government comms campaign.
Notes to the reviewer
The changes are tailored for the landing page only (for now) so the hub pages will remain the same.
The screen capture includes two additional changes:

[x] an update to the action link component in govuk_publishing_components (this needs to be raised, merged, and released released and rebased)
[x] copy updated in govuk-coronavirus-content (pending review)

BeforeAfter

Trello card
:warning: This application is Continuously Deployed: :warning:

Merged changes are automatically deployed to staging and production.

Make sure you follow the guidance for deployments before you merge.

Check your branch is being deployed in the Release app, after merging.

Thank you for the initial review @andysellick! The merge was postponed and we have some additional changes from https://github.com/alphagov/collections/pull/1891/commits/1bd2806cb648cb5c69c7813921559da8c56ef465 onwards – if you have time to cast your eyes on it would be great. It's about adding a video to the page (pending section title for this). The current plan is to have it ready by 3pm.


Add rider cache directory to Unity ignore list

Reasons for making this change:
Many Unity developers work with JetBrains Rider.
As the Unity.gitignore considers VSCode files it should also consider Rider files.
Riders .idea folder contains a mix of cache, user/shared configuration and settings files.
Similar to Visual Studios .vs folder it should be ignored.
There may be some files (e.g shared dictionaries and settings) in this folders that could be shared.
Sadly JetBrains does not separate local from shared files via folder structure.
Thus it is easier to ignore the folder and later on explicitly state exceptions for files one would consciously share.
Links to documentation supporting these rule changes:
This is JetBrains recommendation on which of the files in the .idea folder to ignore:
https://rider-support.jetbrains.com/hc/en-us/articles/207097529-What-is-the-idea-folder-
The comments are already mentioning though that some files are missing from this official recommendation.
This is their recommended gitigonre file
https://github.com/github/gitignore/blob/master/Global/JetBrains.gitignore
It is quite complex though and is not future proof as it may miss Rider generated files added in the future.
As long as Rider does not clearly separate their local files from their shared files I would recommend to ignore this folder completely.

It looks like there are some duplicate PRs with this same change: #3543, #3665, #3681, and #3687.

This is a good change. I do this manually for each project right now.

This would be very useful indeed! I do this manually for every project as well.


⚠ Fortnite has degraded performance

In 3a0d91f, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1642 ms

Resolved: Fortnite performance has improved in 2f94b7d.


v0.12.1 release

Coverage decreased (-0.3%) to 92.473% when pulling d43d06710c138055319265645463c95b495ac33c on release into fd99397e07a174f65592f202ba0074a963060282 on master.


npk3tool.exe crashed every time

Hi Marcus! I found the key by using your tool! Thank you!
I want to extract the cg by the key but the NPK3tool.exe crashed upon start... I cannot even input anything...
The right step is type -KY & the key & cg.npk? But I can't type anything before it crashed...
What should I do?
I am sorry to bother you here. Thank you in advance.

show me how looks like your command line and any comment of the error if the program display one.
It's impossible to me fix without any information.

I just clicked the NPK3tool executable and it showed up, then I tried to type something with it unexpectedly quitting... I didn't see any error displayed but it just quitted without reason...
And when I dragged the npk file into the npk3tool.exe, and typed "-KY YourKeyHere C:\YourGameNpkFileHere.npk", there's no response at all...
Or should I use the visual studio tool? But I have no knowledge of this...
Sorry to bother you again, thank you ><

Isn't to you type the command line after the dragdrop, you must use the CMD to do that. The drag drop only work with games already supported by the tool

Friend, I really recommend you found someone that understand a bit more of computer before try to translate a game, i'm sure this don't will be your last problem... but whatever, don't drag&drop, this is the CMD but is to you type it, open the tool directory in the cmd then execute the command like in the example.

Thank you for explaining! I just want the cg files. >< I'll try it!!!!
Sorry to bother you so many times...
Many thanks!!!!!


fix(DDI-1241): pass variant prop from react

Passing variant prop from React GoATable to the web component.

:tada: This PR is included in version 1.0.0-alpha.160 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.0.0-alpha.120 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0-alpha.15 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-alpha.21 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Added 2 Locations; Refactored AbstractLocation

Changed PlaceDevelopers to abstract method to allow different usage across different locations. Added InvestmentField and TrainingCenter and initialized in GameController. Still need to add Tool Location (was not on trello).

Looks good to me. I made two minor changes: an output message edit and a comment edit. I also resolved the merge conflict.


fix github actions

We need github actions to be green, otherwise it will have the same effect as before where we merge broken stuff into master

closing old issues


All videos in the ActivityNet training set are unavailable on Youtube as of 2018-03-06

All videos are reporting download errors using the downloader script in this repo. We tried accessing some by hand from Youtube and they report the same error as youtube-dl (as they should, sadly).
The error is invariably

This video is unavailable.

Example:

Update: All training videos are unavailable.

I was able to download almost all of the videos over the past couple weeks. Maybe you are using a too old version of youtube-dl.

Hi @Enamex,
Sorry for the delayed response - we were busy with ECCV. We know that around 1k videos of ActivityNet are unavailable on YouTube now, but the other 19k videos are still up. Are you still unable to download the videos?
Possible reasons we collected from previous users for why they cannot download some videos:

Problem: an old version of youtube-dl; Solution: update to the latest version.
Problem: blocked by country; Solution: use VPN to download (if possible VPN into the US for best results)
Problem: video removed by YouTube or by the content creator (currently there is around 1k of those); Solution: email us the YouTube IDs.

Humam

"The other 19k" only covers the val-set, though, right?
As an additional datapoint, I was able to download 232264 videos from the train-set without problems, and 14270 had problems (9020 "unavailable" and 4247 "account terminated", the rest various other causes). The VPN is a good idea, I will give that a try in order to get most (hopefully) remaining videos, as YouTube does block a lot in Germany.

@lucasb-eyer I think you are referring to the Kinetics dataset. The original question (and my answer) are about the ActivityNet dataset.

Oh, you are right, very sorry for this mixup and distracting from the OP. :blush:


New release

Hi, and thank you for working on this library.
I was wondering, since the current release on pypi is 0.1.0 (from 2018), if it was possible to publish even a minor release update in order to get the package on pypi up to date with the last changes. Thank you.

@kislyuk any idea on the timeline for this?

Released v0.2.0, please test.

(The delay in releasing was due to some unfinished updates that I have in progress to match the most recent RFCs. Since I won't have time to finish those until a bit later, I am releasing the contributions made since the last release in v0.2.0 so people can use those.)

Great, thanks @kislyuk ! Appreciate it 🙂


Release v2.1.0-rc.5

Signed-off-by: disksing<EMAIL_ADDRESS>What problem does this PR solve?
Prepare release.
What is changed and how it works?
Add changelog.
Check List
Tests

Unit test

/run-all-tests tidb=release-2.1 tikv=release-2.1 tidb-test=release-2.1

/run-all-tests tidb=release-2.1 tikv=release-2.1 tidb-test=release-2.1

/run-unit-test


Add details in  guickstart

Closes #373

Btw do you want to change the title of this PR? quickstart instead of guick

LGTM!
I am not sure if we need to include the Azul 17 for Mac users, better if we do

I have added in the step


get user or username from class?

hi,
this is a great middleware thanks
I was wondering if it is possible to get the user or username in my code?
let's say i have
'User 1' => [
  '....hashed foo...',
  '....hashed bar...',
],

for foo:bar
is it possible for me to get 'User 1' or 'foo' in my code?
regards
Joe

Yes, if you extend the ShieldMiddleware  you can get the users name.

I saw you can get $request->user and $request->password I think, but i wasn't sure about they actual key used in the config. ('User 1' above)
thanks
j

Did you solve it?


ENH: Add RATS (commented out) and PCNN to dockerfile

Okay, it went past building the docker image - merging.


params name conflict

when a para named length in params , the jsonp request will not work;
Vue.http.jsonp  will send  0=undefined in request, it is error

I have a similar issue. If the params payload for a GET request contains a length property, vue-resource tries to place the (probably nonexistent) numeric properties from 0 to length in the URL. Renaming the length property immediately fixes the issue.


Update type option in readme file

Documented the new option also to the readme file =]

Thank you very much!

No problem , my pleasure =]


number light level overlay get bugged on fabric

when i turned the light level overlay to numbers every time i stop looking under me the numbers turn to glitched block.

Remove optifine to check if it is an optifine issue

Remove optifine to check if it is an optifine issue

i tried and yes it is an optifine issue, thanks.
but will there be a new update to this mod that fixes it?

No, see #5


Imager equivalent of generateTransformsBeforePageLoad native setting

On a page showing several images with each a few images transforms (for several breakpoints + Retina), I end up with a lot of images transforms required.
However, this requires so much memory that these pages doesn't load, and end with a server error (like max_execution_time). Even if these images will not be shown, they block the page rendering. In that way, I found Imager less convenient that the native Craft way, because of this.
Is there any option that disable the images generation before page load? With the native Craft way, there's the generateTransformsBeforePageLoad config. I'm looking for an Imager equivalent.
I know there's now Imager X, but it's a paid plugin and the automatic generation of transforms features are not working the same way.

See my reply here.


Regression features in RegressorClassifierBase

I added dictionaries to the class so that we can track what features have been used to train the model.
This will be particularly handy when the model is saved on disk and used elsewhere.

could you add a unit test as well?


WCAG: Break out accessibility report into Development Tasks

Tasks are generated from the WCAG report to address WCAG compliance issues.

Hey team! Please add your planning poker estimate with ZenHub @JahidAhmed @RyanCoulsonCA @yileifeng


Error Trying to Run Template After Initial Config

After following the README instructions and trying to run the template in Web Storm with pnpm dev and with Planet Scale details in .env.local it's throwing:
  ▲ Next.js 13.5.5-canary.2
  - Local:        http://localhost:3000
  - Environments: .env.local

 ✓ Ready in 3.7s
 ○ Compiling /page ...
 ✓ Compiled /page in 8.8s (1196 modules)
 ⨯ Internal error: TypeError: fetch failed
    at Object.fetch (node:internal/deps/undici/undici:11457:11)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
 ⨯ Internal error: TypeError: fetch failed
    at Object.fetch (node:internal/deps/undici/undici:11457:11)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
digest: "2026204822"
 ✓ Compiled /favicon.ico/route in 2.8s (865 modules)

This seems very similar to https://github.com/vercel/nextjs-planetscale-nextauth-tailwindcss-template/issues/5  however,

I had already manually created the database and test row. I had also populated .env.local and followed all other instructions in the README.

I tried downgrading next to 13.2.4 from 13.5.5. as mentioned in the prior Issue, but it did not resolve the problem

Browser console:

Uncaught Error: fetch failed
5 [react-server-dom-webpack-client.browser.development.js:1679](webpack://_N_E/node_modules/.pnpm/next@13.5.4_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/compiled/react-server-dom-webpack/cjs/react-server-dom-webpack-client.browser.development.js?945e)
    resolveErrorDev react-server-dom-webpack-client.browser.development.js:1679
    processFullRow react-server-dom-webpack-client.browser.development.js:1734
    processBinaryChunk react-server-dom-webpack-client.browser.development.js:1862
    progress react-server-dom-webpack-client.browser.development.js:1941
    (Async: promise callback)
    progress react-server-dom-webpack-client.browser.development.js:1942
    (Async: promise callback)
    progress react-server-dom-webpack-client.browser.development.js:1942
    (Async: promise callback)
    progress react-server-dom-webpack-client.browser.development.js:1942
    (Async: promise callback)
    progress react-server-dom-webpack-client.browser.development.js:1942
    (Async: promise callback)
    progress react-server-dom-webpack-client.browser.development.js:1942
    (Async: promise callback)
    progress react-server-dom-webpack-client.browser.development.js:1942
    (Async: promise callback)
    progress react-server-dom-webpack-client.browser.development.js:1942
    (Async: promise callback)
    progress react-server-dom-webpack-client.browser.development.js:1942
    (Async: promise callback)
    progress react-server-dom-webpack-client.browser.development.js:1942
    (Async: promise callback)
    progress react-server-dom-webpack-client.browser.development.js:1942
    (Async: promise callback)
    startReadingFromStream react-server-dom-webpack-client.browser.development.js:1949
    createFromReadableStream react-server-dom-webpack-client.browser.development.js:1954
    useInitialServerResponse app-index.js:116
    ServerRoot app-index.js:127
    renderWithHooks react-dom.development.js:10730
    mountIndeterminateComponent react-dom.development.js:15768
    beginWork$1 react-dom.development.js:17352
    beginWork react-dom.development.js:25701
    performUnitOfWork react-dom.development.js:24552
    workLoopConcurrent react-dom.development.js:24538
    renderRootConcurrent react-dom.development.js:24494
    performConcurrentWorkOnRoot react-dom.development.js:23358
    workLoop scheduler.development.js:261
    flushWork scheduler.development.js:230
    performWorkUntilDeadline scheduler.development.js:534
    (Async: EventHandlerNonNull)
    <anonymous> scheduler.development.js:569
    <anonymous> scheduler.development.js:630
    NextJS 4
    <anonymous> index.js:6
    NextJS 4
    <anonymous> react-dom.development.js:27
    <anonymous> react-dom.development.js:36891
16:30:11.393
The above error occurred in the <NotFoundErrorBoundary> component:

NotFoundErrorBoundary@webpack-internal:///(app-pages-browser)/./node_modules/.pnpm/next@13.5.4_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/not-found-boundary.js:54:9
NotFoundBoundary@webpack-internal:///(app-pages-browser)/./node_modules/.pnpm/next@13.5.4_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/not-found-boundary.js:62:62
DevRootNotFoundBoundary@webpack-internal:///(app-pages-browser)/./node_modules/.pnpm/next@13.5.4_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/dev-root-not-found-boundary.js:32:24
ReactDevOverlay@webpack-internal:///(app-pages-browser)/./node_modules/.pnpm/next@13.5.4_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/react-dev-overlay/internal/ReactDevOverlay.js:66:9
HotReload@webpack-internal:///(app-pages-browser)/./node_modules/.pnpm/next@13.5.4_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/react-dev-overlay/hot-reloader-client.js:294:37
Router@webpack-internal:///(app-pages-browser)/./node_modules/.pnpm/next@13.5.4_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/app-router.js:157:93
ErrorBoundaryHandler@webpack-internal:///(app-pages-browser)/./node_modules/.pnpm/next@13.5.4_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/error-boundary.js:82:9
ErrorBoundary@webpack-internal:///(app-pages-browser)/./node_modules/.pnpm/next@13.5.4_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/error-boundary.js:110:53
AppRouter@webpack-internal:///(app-pages-browser)/./node_modules/.pnpm/next@13.5.4_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/app-router.js:440:47
ServerRoot@webpack-internal:///(app-pages-browser)/./node_modules/.pnpm/next@13.5.4_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/app-index.js:126:24
RSCComponent
Root@webpack-internal:///(app-pages-browser)/./node_modules/.pnpm/next@13.5.4_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/app-index.js:142:24

React will try to recreate this component tree from scratch using the error boundary you provided, ReactDevOverlay.

I have this issue both locally and on Vercel. I noticed on Vercel some part of the error mentioning SSL versions.
I made my own isolated test version of page.tsx which uses Planet Scale.
Long story short...
 {
  cause: [Error: 8046A44AF87F0000:error:0A00010B:SSL routines:ssl3_get_record:wrong version number:../deps/openssl/openssl/ssl/record/ssl3_record.c:355:
  ] {
    library: 'SSL routines',
    reason: 'wrong version number',
    code: 'ERR_SSL_WRONG_VERSION_NUMBER'
  }
}

Node.js v18.16.0

So... it's SSL related? I tried changing mysql to mysqls in the DATABASE_URL but no luck. However, I can use mysql on a terminal to connect to Planet Scale with my DATABASE_URL.
In that test file if I use mysql2 instead then it works:
const mysql = require('mysql2/promise');

async function testConnection() {
  const connection = await mysql.createConnection({
    host: 'aws.connect.psdb.cloud',
    user: 'myusername',
    database: 'mydb',
    password: 'your-password-here',
    ssl: {
      // Do not do this in production!!!
      // Or if you do, be sure to add your own value to checkServerIdentity to check the server's cert against.
      rejectUnauthorized: false
    }
  });

  try {
    const [rows, fields] = await connection.execute('SELECT * FROM `users`');
    console.log(rows);
  } catch (error) {
    console.error('Error executing query:', error);
  } finally {
    await connection.end();
  }
}

testConnection();

yielding:
[ { id: 1, email<EMAIL_ADDRESS>name: 'Me', username: 'username' } ]

This has since been moved to Postgres. Let us know if you try it again!


Add BMC Fulltext Search as source

This source should find data and software packages.

Added BMC fulltext source.


Repeated link tag?

In public/index.html it looks like line 7 is a repeat of line 5.  Is that necessary?

You are right it's a mistake. Thanks!


Enumeration name fallback for JSON serialization

The use case is I would like to use enumerations in JSON that follow the "human-readable" convention.  In the example Digit pojo I would like the user to define a digit as "One", "Two" ... "Nine" in JSON.  But for legacy reasons and for completeness I would like the user to be able to use the generated names as well.
So "ONE", "TWO", "THREE" ... "NINE" would all be accepted as well.
Example Schema Definition:
{
  "$id": "https://example.com/schemas/math/Digit.schema.json",
  "$schema": "http://json-schema.org/draft-07/schema#",
  "type": "string",
  "javaType": "com.example.api.math.Digit",
  "description": "All the digits.",
  "enum": [
    "One",
    "Two",
    "Three",
    "Four",
    "Five",
    "Six",
    "Seven",
    "Eight",
    "Nine"
  ],
  "required": [],
  "definitions": {},
  "additionalProperties": false
}

Example Generated POJO:

package com.example.api.math;

import java.util.HashMap;
import java.util.Map;
import com.fasterxml.jackson.annotation.JsonCreator;
import com.fasterxml.jackson.annotation.JsonValue;

/**
 * All the digits.
 *
 */
public enum Digit {

    ONE("One"),
    TWO("Two"),
    THREE("Three"),
    FOUR("Four"),
    FIVE("Five"),
    SIX("Six"),
    SEVEN("Seven"),
    EIGHT("Eight"),
    NINE("Nine");
    private final String value;
    private final static Map<String, Digit> CONSTANTS = new HashMap<String, Digit>();
    private final static Map<String, Digit> NAMES = new HashMap<String, Digit>();

    static {
        for (Digit c: values()) {
            CONSTANTS.put(c.value, c);
        }
        for (Digit c: values()) {
            NAMES.put(c.name(), c);
        }
    }

    private Digit(String value) {
        this.value = value;
    }

    @Override
    public String toString() {
        return this.value;
    }

    @JsonValue
    public String value() {
        return this.value;
    }

    @JsonCreator
    public static Digit fromValue(String value) {
        Digit constantValue = CONSTANTS.get(value);
        if (constantValue == null) {
            {
                Digit nameValue = NAMES.get(value);
                if (nameValue == null) {
                    throw new IllegalArgumentException(value);
                } else {
                    return nameValue;
                }
            }
        } else {
            return constantValue;
        }
    }

}

@joelittlejohn Here is a draft implementation of the feature without unit tests or a way to turn on/off the feature.  Do you think this feature would require configuration property to enable it?  If so should it be on or off by default?

@ddcruver I'm afraid I don't think this is something we should add to jsonschema2pojo. You're asking for the types to support data that is invalid according to the schema, and to encourage multiple different representations of the same data in the wild. Also, the difference between the 'human readable' and the constant name is almost zero (they're both human readable).
I don't want to say you don't have a valid reason for wanting this, because I'm sure you're dealing with an awkward situation, it's just not something we should be including in the core and supporting forever more. I think in this case you should just fork this library and use a custom version in the places you have this requirement, or alternatively, create custom RuleFactory and replace the EnumRule (then there is no need to fork).

My term 'human readable' maybe should be more like displayable to end user.  The names (Java Enum Name) generated by the EnumRule are human readable to an programmer but are not natural to an end user.  Many of our types have a getDisplayFriendly() field which was defined by an interface I created that allows us to provide a value that can be displayed to an end user.  With the migration of our types to jsonschema it is hard to provide this except if we just use the name we want to display to user in the definition of the schema.
The problem I am describing to you is based on converting existing types to support the display friendly name and balance the changes to not cause disruption to other developers on our project and existing configurations that are now invalid because of these changes.   The fallback to the name (Java Enum Name) would reduce these cases.
I could put an warning logger in the fromValue method but I don't believe loggers are supported by the jsonschema2pojo generation yet. (SLF4J would be nice).
I do agree with it being invalid but their is a one-to-one mapping between the value and the enum name generated.
We want to avoid forking the repository but I do understand you hesitation with having jsonschema2pojo support invalid json.  I could implement it off by default but will respect your decision to not include it in the project.  I will have to end up writing some custom Jackson serializes to support this use case on our project.


[BUG]: Unable to get activity reports in custom Audit Log Search connector

Hello,
As explained here - https://docs.microsoft.com/en-us/power-platform/guidance/coe/setup-auditlog#option-2-connect-to-the-audit-log-by-using-basic-authentication,  User "MPP Admin 1" ( Power Platform Admin Role + D365 Admin Role ) is trying to configure custom connector, however "GetActivitiesByOperation " is throwing an error:

I assume user "MPP Admin 1" requires Audit Log Search Role assigned from "Exchange Online Admin" - is that correct?
Can you please clarify?
Thanks and Regards,
Dipti

I confirm that user  "MPP Admin 1"  able to view Audit log reports in Office 365 here - https://protection.office.com/unifiedauditlog
Even though, same user is facing connection issue!
Any update?
Thanks,

Hello,
Just to check if there is any update on this bug?
Thanks and Regards,
Dipti

Have you already validated the items from the setup instructions?
Pasting a few potential issues here inline:

Are audit logs enabled, and do you have permission to view the audit logs? Check protection.office.com > Search > Audit Log Search.
If you don't have permissions, see Requirements to search the audit log. -. Have you enabled the audit log very recently? If so, try again in a few minutes, to give the audit log time to activate.

Hi Jenefer,
Absolutely, as mentioned that user "MPP Admin 1" is able to view audit log search report from the M365 Admin center, because that user has required permission to view Audit Log Search Report, however the same user is failed to view activity reports in the CoE connector as mentioned in the question.
Note: Audit Log has been enabled 2 months back.
Thanks and Regards,
Dipti

Can you clarify, which of these two setups are you doing?
Option 1: Connect to the audit log by using an Azure AD app registration
Option 2: Connect to the audit log by using basic authentication
And if option 2, does your organization support basic authentication?

Sure, I think this could be the concern, as I am using Option 2 and organization is using MFA, Thank you for your time and help here!
Best Regards,
Dipti

Wonderful. Thanks for using CoE!


Default value for maxRowsInMemory

Maybe some discussion on how to approach this matter is appropriate, I suspect there will be other changes spawning from this one, or at least more clarification.
The documentation for batch-ingestion claims that default value for maxRowsInMemory is 5 million, which is not true. It actually defaults to 80000, which in turn is derived from the default rowFlushBoundary.

indexing-hadoop/src/main/java/io/druid/indexer/HadoopTuningConfig.java:113:
this.rowFlushBoundary = maxRowsInMemory == null ? maxRowsInMemoryCOMPAT == null ?  DEFAULT_ROW_FLUSH_BOUNDARY : maxRowsInMemoryCOMPAT : maxRowsInMemory;
indexing-hadoop/src/main/java/io/druid/indexer/HadoopTuningConfig.java:44:
private static final int DEFAULT_ROW_FLUSH_BOUNDARY = 80000;

Furthermore the value defaults to 500000 for realtime/index tasks. Is it sane to have 2 different defaults values for the same config item depending on if it's used in realtime or batch contexts?

indexing-service/src/main/java/io/druid/indexing/common/task/IndexTask.java:501:
private static final int DEFAULT_ROW_FLUSH_BOUNDARY = 500000;
server/src/main/java/io/druid/segment/indexing/RealtimeTuningConfig.java:40:
private static final int defaultMaxRowsInMemory = 500000;

Another minor part of this is also that other references to the value contain the info:

Maximum heap memory usage for indexing scales with maxRowsInMemory * (2 + maxPendingPersists).

This is lacking from batch-ingestion.md, but I'm not sure if this applies to batch-ingestion too, should i add it?

:+1:

not merging yet because I think we should talk about a more reasonable default value for maxRowsinMemory

There is also a third default value, which is Tranquility's value of 75000. That takes precedence over the 500000 in RealtimeTuningConfig. It's lower on the grounds that it's better to be somewhat less efficient, and work, than to reach for the stars and possibly OOM.

than to reach for the stars and possiOutOfMemoryError: java heap space

@gianm, do you want to set defaults everywhere to 75k?

It would be nice if tranquility and druid had consistent defaults. 75k sounds reasonable to me

@bjozet do you mind updating this PR to have 75k be the default value across ingestion?

@fjy sure I can give it a go, but this would also include changing the DEFAULT_ROW_FLUSH_BOUNDARY, is that ok, or should this  line be changed instead, and add a specific default for maxRowsInMemory?

@bjozet change DEFAULT_ROW_FLUSH_BOUNDARY

I've updated the occurrences of the defaults which I could find, I hope none slipped past me. An extra set of eyes on this would be nice.

:+1:

:+1:, Added release notes label so that this gets added to release notes.

This is ready to be merged.
@bjozet: can you fix the conflicts ?

@bjozet last comment, can we squash commits? after that I will merge

@fjy squashed and ready to go. I shamelessly stole @gianm's comment from this thread for commit message.


Deprecation warning due to invalid escape sequences in Python 3.7

Expected Behavior
No warnings
Current Behavior
Deprecation warnings are raised due to invalid escape sequences. This can be fixed by using raw strings or escaping the literals.
Steps to Reproduce
find . -iname '*.py' | grep -Ev 'example|utl|samples' | xargs -P 4 -I{} python3.8 -Wall -m py_compile {}
./dlib/external/pybind11/tools/mkdoc.py:101: DeprecationWarning: invalid escape sequence \w
  cpp_group = '([\w:]+)'
./dlib/external/pybind11/tools/mkdoc.py:102: DeprecationWarning: invalid escape sequence \[
  param_group = '([\[\w:\]]+)'
./setup.py:213: DeprecationWarning: invalid escape sequence \(
  major = re.findall("set\(CPACK_PACKAGE_VERSION_MAJOR.*\"(.*)\"", open(cmake_file).read())[0]
./setup.py:214: DeprecationWarning: invalid escape sequence \(
  minor = re.findall("set\(CPACK_PACKAGE_VERSION_MINOR.*\"(.*)\"", open(cmake_file).read())[0]
./setup.py:215: DeprecationWarning: invalid escape sequence \(
  patch = re.findall("set\(CPACK_PACKAGE_VERSION_PATCH.*\"(.*)\"", open(cmake_file).read())[0]

Version: master branch
Where did you get dlib: github repo
Platform: Linux 64 bit

Pybind11 just needs to be updated.  #1806


pre 1.13 blocks over water rendering issue

There is an issue rendering blocks directly over the water in pre-1.13 regions perhaps.
There is a layer of snow at y=63 directly over the water.
Here is the example 1.12.2 region
r.-74.-93.zip
Rendering the 1.12.2 region with latest minutor - rendering incorrectly. 4 Chunks not rendering snow at y=64

Updating the region to 1.19.2 and rendering it with latest minutor - rendering correctly

When I open this Region file in NBTexplorer and look to these Chunks
(e.g. -2349/-2945 versus -2349/-2946)

(I see that there is a Section Y==14 with some glass)
Sections 0 to 4 are present which is expected, as 16 Block layers per Sections is 0..3 for 0..63 and another one for 64
Section Y==4 contains Block data field and Data data filed with all 0 -> AIR

So it would be expected, that there is NOTHING at Y==64
When I open this Region in Minecraft 1.12.2 I can see your snow Blocks . . .
So you are right, the snow is present, but not stored on disk and therefore not rendered.
I have no clue, why this layer is not stored!

What is interesting is, that it works in Chunks that are located more to the north.
It seems, that only these Chunks are affected that have the glass layer at Y==239

Good point
here's a bigger section - one more region to the south. It's pretty strange
1.12.2 region file

updated to 1.19

Your build looks nice, Christmas themed?
Maybe you could add that Region file also? Than I have something to show for the kids ...

Hi Etlam,
Absolutely - here is the world download with level.dat
Emperium Christmas Base 2019-12-25.zip
It's from an anarchy server called 2b2t, so while I've removed some signs, better doublecheck before you show the kids.