[Request] Crafting recipes

Make sure you are not opening a duplicate.

[X] I have searched the issue tracker and did not find an issue describing my bug.

Description of your suggestion.
Make it possible to change crafting recipe in the mod file (At the moment only recurve bow and long bow can be edited).

You can edit all, they are just auto generated :)

How can i edit the other?

If I remember correctly the recipe id is the item id.

What if I want to remove a recipe to make the object uncraftable?

Uh good question but the ID should be the item id

To remove the recipe just use the recipe remover mod. It took a little to figure out how to set the config but it works perfectly.


Add a new template

I would like a add a new template which looks like this:

Closing as this has been merged as a PR.


openvpn: Update to 2.4.3

Connected to https://github.com/resin-os/meta-resin/issues/997
This recipe is imported from meta-oe/rocko.
ResinOS needs dualstack support which is included from openvpn 2.4. We add this
recipe in the common meta-resin layer to make sure that all of our supported
boards have this available.
Change-type: patch
Changelog-entry: Update OpenVPN to 2.4.3
Signed-off-by: Andrei Gherzan<EMAIL_ADDRESS>
Contributor checklist

[ ] Changes have been tested
[x] Change-type present on at least one commit
[x] Signed-off-by is present
[x] The PR complies with the Open Embedded Commit Patch Message Guidelines

Reviewer Guidelines

When submitting a review, please pick:

'Approve' if this change would be acceptable in the codebase (even if there are minor or cosmetic tweaks that could be improved).
'Request Changes' if this change would not be acceptable in our codebase (e.g. bugs, changes that will make development harder in future, security/performance issues, etc).
'Comment' if you don't feel you have enough information to decide either way (e.g. if you have major questions, or you don't understand the context of the change sufficiently to fully review yourself, but want to make a comment)

Not tested yet. Will update as soon as that happens. Feel free to review in the meanwhile.

Tested on rpi3.


Removes ugly blue text when script is running

Adding this to all ps1 scripts isn't necessary because not all scripts create the blue text. But, noticed in your live stream that upon first launch of winutil, when it checks/installs winget, the blue texts appears on powershell window. This change is purely aesthetics.

This is about to revamped and a different install method will be used.


Using keras tuner

Hello everybody I am trying to use keras tuner along with segmentation_models. I installed tensorflow 2.1 and keras 2.3.1 to work with this repo and I installed the latest version of keras tuner but when I try to import it, it causes me this error.

I think there are uncompatiblity of versions.
DId someone use keras tuner and worked fine with this repos? if yes can you tell me about the versions of keras, tensorflow and keras tuner you used?. I would appreciate any help, I need to work with it urgently.
Thank you in advance.

It worked for me using tensorflow 2.0.0 and keras tuner 1.0.1

It worked for me using tensorflow 2.0.0 and keras tuner 1.0.1. Closing issue.


using win32 resource in addon on windows

Hi there:
I'm using vs2015 to build a addon, and the resource cannot be loaded. For exampe, win32 API LoadString() is used for get a static string from resource file. So i use GetModuleHandle() to get the instance handle. But is doesn't work(error : 1814, error msg : The specified resource name cannot be found in the image file.)
Is there other way to get the instance handle for win32 APIs?

sorry we haven't been able to provide an answer here @xbqian, I hope you managed to figure it out and move forward. I'm going to close this due to staleness but feel free to open a new issue with more details, perhaps even some code. The lack of details tends to be the reason these general questions go unanswered.


CUFFT_INTERNAL_ERROR on TTS and RVC inference

Environment

OS: Ubuntu 22.04.3 LTS
Python Version: 3.11.5
Conda Environment: Yes
CUDA Version 12.2
Hardware: 4060 8gb VRAM Laptop

Issue Description
Whether it be through the TTS or the model inference I am encountering this error consistently RuntimeError: cuFFT error: CUFFT_INTERNAL_ERROR
###Attempted sollutions

I have updated CUDA drivers and backdated them from versions 11.5 to 12.2
Downgraded and updated pytorch from versions 2.0.0, 2.0.0cu118, 2.0.0cu117, 2.1.1, 2.2.2 and the nightly binaries
reinstalled Applio

  File "/home/d/Desktop/aihub/Applio-RVC-Fork/lib/infer/modules/vc/modules.py", line 534, in vc_single_dont_save
	audio_opt = self.pipeline.pipeline(
  File "/home/d/Desktop/aihub/Applio-RVC-Fork/lib/infer/modules/vc/pipeline.py", line 686, in pipeline
	pitch, pitchf = self.get_f0(
  File "/home/d/Desktop/aihub/Applio-RVC-Fork/lib/infer/modules/vc/pipeline.py", line 424, in get_f0
	f0 = self.model_rmvpe.infer_from_audio(x, thred=0.03)
  File "/home/d/Desktop/aihub/Applio-RVC-Fork/lib/infer/infer_libs/rmvpe.py", line 616, in infer_from_audio
	mel = self.mel_extractor(
  File "/home/d/Desktop/aihub/Applio-RVC-Fork/.venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
	return forward_call(*args, **kwargs)
  File "/home/d/Desktop/aihub/Applio-RVC-Fork/lib/infer/infer_libs/rmvpe.py", line 530, in forward
	fft = torch.stft(
  File "/home/d/Desktop/aihub/Applio-RVC-Fork/.venv/lib/python3.10/site-packages/torch/functional.py", line 641, in stft
	return _VF.stft(input, n_fft, hop_length, win_length, window,  # type: ignore[attr-defined]
RuntimeError: cuFFT error: CUFFT_INTERNAL_ERROR

'NoneType' object has no attribute 'dtype'

applio doesn't work with 3.10 on windows tho

Python Version: 3.11.5

Are you sure?

File "/home/d/Desktop/aihub/Applio-RVC-Fork/.venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl

This doesn't even use conda. applio runs in a venv

Conda Environment: Yes

About this applio have weird behaviors on linux due is clossplatform and the python versions differ. win use 3.9 and unix like systems 3.11 so i hope you can understand the issue.
Check in the discord for better support


Containerize PaSe into a pair of Docker containers

(pase-indexer and pase-server)
to make it easier to build and run locally.

Note that I dropped the "@testing-library/…" dependencies because they were creating dependency problems when building inside a container, and because they were unused. Of course they would need to be added back (and the dependency problems investigated further) if tests were to be written for the frontend.
I took the opportunity to bump the react dependencies to recent versions.


fix(userspace): compute the drop ratio in the right way

Signed-off-by: Andrea Terzolo<EMAIL_ADDRESS>Co-authored-by: Shane Lawrence<EMAIL_ADDRESS>What type of PR is this?
/kind bug
/kind cleanup
Any specific area of the project related to this PR?
/area engine
What this PR does / why we need it:
First of all, I would to thank  @shane-lawrence for spotting this issue in #2111! Great catch!
As I said here the n_evts fields, both in kmod and in bpf should be intended as "events seen by the drivers" so they should contain also the number of drops. This is for example what we do in our bpf probe after this variable increment we will increment also the drops number in case the event cannot be pushed into the buffer for example.
According to it, we need to change the way in which we compute the drop ratio
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
fix: compute the `drop ratio` in the right way

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Andreagit97
To complete the pull request process, please assign fededp after the PR has been reviewed.
You can assign the PR to them by writing /assign @fededp in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 076ebb307859204d82cfe46e038f8554fc8a5e0e

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


remote control is not a plugin

The Remote Control functionality is not a plugin, but exists in core JOSM. Please update the dialog text.

Done


Make syntax highlighting properly use the visitor

There are two interesting methods in the visitor:

acceptVisitor:
visitNode:

From the outside, we should always use visitNode:.

We can see the effect here:

the comment should be highlighted with gray.

#acceptVisitor: gtReferences select: [:each |
	(each selector beginsWith: 'visit') not and: [
		(each methaaodClass name beginsWith: 'Br') or: [
		(each methodClass name beginsWith: 'Gt') ]]]


Port symbol walker to latest

The kind folks over at @angular were wondering if we could port the type/symbol walker we produced for my declaration flattening prototype to our latest codebase and publicly expose it.
This is that.
@evmar @alexeagle
If one of you has a good test corpus for what you want to traverse, I can try to make a test out of it (so we don't break it in the future). You should also verify that it's capable of what you need - otherwise make some suggestions.
@mhegazy
You probably want to verify that we're OK with maintaining this API.
If anyone has any changes you'd like to see, you should probably propose them here.

Our current janky code for poking around in types is here:
https://github.com/angular/tsickle/blob/9c3b5a949a0b4355df854466dcceaa327d9eeb30/src/type-translator.ts#L106
One ominous comment from there:
        // This comes up when generating code for an arrow function as passed
        // to a generic function.  The passed-in type is tagged as anonymous
        // and has no properties so it's hard to figure out what to generate.
        // Just avoid it for now so we don't crash.
        this.warn('anonymous type has no symbol');

However, I can't remember the particular scenario this is avoiding, and I commented it out and attempted to reproduce the mentioned crash and can't seem to find it now.
The other thing we've had trouble with is mapping a type back to a qualified name.  E.g. the compiler infers x to be a foo.Bar but via x we could only get Bar, which isn't enough to recreate the name.  Not sure if it's at all relevant here though.

@evmar Interestingly enough, this chunk at the bottom of visitSymbol is for handling a similar scenario (anonymous types don't trip up this walker since it happily visits both symbols and types):
forEach(symbol.declarations, d => {
  // Type queries are too far resolved when we just visit the symbol's type
  //  (their type resolved directly to the member deeply referenced)
  // So to get the intervening symbols, we need to check if there's a type
  // query node on any of the symbol's declarations and get symbols there
  if ((d as any).type && (d as any).type.kind === SyntaxKind.TypeQuery) {
    const query = (d as any).type as TypeQueryNode;
    const entity = leftmostSymbol(query.exprName);
    visitSymbol(entity);
  }
});

The original intent in our declaration emitter was to find all the symbols you care about with this, and filter on accept by inspecting the symbol's declarations and checking where they are (user code, stdlib, or external library) and what their flags are (public, protected, private), and collect the visible declarations from the visited symbols. Then you just transform/rewrite the declarations you collected (also keeping a list of all those external references you encountered and filtered out so you can write imports for them).

Thanks @weswigham, this seems like it'd help our use case. We could use this to generate a tool that walks all exported symbols in a module, skipping private. We'd still have to collect what modules symbols are from and emit an appropriate import {Symbol} from 'ambient/module/declaration'; and declare module 'ambient/module/declaration' { ... } bits for our flattened .d.ts file, but that seems doable in a separate project.

#17844 replaces this.


european currency / decimal format

It would be handy to be able to switch between european/american number formats in the component builders. For numbers it would be nice if there was a mask (to be able to leave decimals out f.e.) Is the ng2 version as 'complete' as https://github.com/formio/ngFormBuilder?

The issue is resolved


Possible Plugin system vulnerability

Hyprland Version
Hyprland, built from branch  at commit fe7b748eb668136dd0558b7c8279bfcd7ab4d759  (props: bump version to 0.39.1).
Date: Tue Apr 16 16:01:03 2024
Tag: v0.39.1, commits: 4460
flags: (if any)
System Information:
System name: Linux
Node name: ventura
Release: 6.8.7-gentoo-dist-hardened
Version: #1 SMP PREEMPT_DYNAMIC Sun Apr 28 02:05:58 BST 2024
GPU information:
0c:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Navi 32 [Radeon RX 7700 XT / 7800 XT] [1002:747e] (rev c8) (prog-if 00 [VGA controller])
os-release: NAME=Gentoo
ID=gentoo
PRETTY_NAME="Gentoo Linux"
ANSI_COLOR="1;32"
HOME_URL="https://www.gentoo.org/"
SUPPORT_URL="https://www.gentoo.org/support/"
BUG_REPORT_URL="https://bugs.gentoo.org/"
VERSION_ID="2.15"
plugins:
Bug or Regression?
Bug
Description
https://social.treehouse.systems/@thesamesam/112349504465197920
https://www.openwall.com/lists/oss-security/2024/04/28/3
This type of issue should be fixed immediately which is why im posting it here to hopefully quicker the process.
How to reproduce
.
 Nobody even assigned it a CVE. What are you on about?
Besides, it's fixed already: https://github.com/hyprwm/Hyprland/commit/28c85619243e6320e75d7abcfe8244fa99d054dd

Does it actually allow an attacker to do anything that can't be done with hyprctl plugin load malware-plugin.so?

No. Someone needs access to write to /tmp/hypr, so... they have to be running code on your system already, at which point they can also rm -rf /. That's why this CVE is a joke.
The only "issue" is that other users could do this as well, which they now can't after 28c85619243e6320e75d7abcfe8244fa99d054dd. Though, other users can still technically hyprctl --instance x plugin load malware.so, maybe I should restrict --instance to same user.

It's a joke and not a CVE. Nobody even assigned it a CVE. What are you on about?
Anyways, it's fixed already: 28c8561

My apologizes on the wording of the title - changing it to better reflect what the issue was.
Awesome work on Hyprland btw, keep it up.

yeah sorry for the snarky tone but I was irritated by how some people are claiming its a huge deal, where it really, really isnt


Repeat activation for dismissing Find my mouse

Description of the new feature / enhancement
I find myself wanting to repeat the activation to dismiss Find my mouse.  This behavior would be inline with things like start menu.
Currently you can dismiss via esc but that isn't always as easy if your pinky is already at the ctrl key.
Scenario when this would be used?
speed of typing, keeping same line of thought
Supporting information
No response

Yes. We cancel the bubble through, correct?


[2 of 2] Add integration tests for networks

Relies on the changes in #1996 to correctly compare values in the UI to values from the API response.
This PR adds integration tests for the Networks section.
Checklist

[ ] Did you add a JIRA issue in a commit message or as part of the branch name?
[ ] Did you add new unit tests?
[x] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

retest this please

Can one of the admins verify this patch?

@jfurrow - seems some of the networking tests are failing

Ping @MatApple @orlandohohmeier

LGTM, thanks @jfurrow!

@MatApple I rebased and force pushed thus your review got dismissed. I decided to proceed anyways, since I the code didn't change 🙏


analyzeHeadless analyzing random files?

In terminal i run the following in an attempt to use ghidra headless analyzer (w/ bridge):
ghidra_9.0.1/support/analyzeHeadless ghidra_9.0.1/support/ test -import "$(pwd)/" pyrogenesis -scriptPath /ghidra_scripts -postscript ghidra_bridge_server.py
I have specified my project, my file (pyrogenesis), and the script to run after. However, when I run the script, it seems to just analyze a random file.  I am not sure if this is an issue with ghidra headless or ghidra_bridge. Either way,  has anyone had this issue before?

Why are you using 9.0.1?
Why are you using ghidra_9.0.1/support/ for your project_location, that seems wrong
Completely ignoring why you're using pwd like this, "$(pwd)/" pyrogeneis and "$(pwd)"/pygrogenesis are very different, yours seems wrong.
Your ghidra_scripts files is in your / directory, that seems wrong?

@mumbel thanks I will have to try that. I am a not good/new to cli/ghidra/linux/many more so not surprising if it's a trivial error. Also, I dont know why i am using 9.0.1, I installed it just in June, not sure why it installed an out of date one.

@mumbel where should my project location be? This part always confuses me. I dont really know what my 'project' is.

A project is basically just a database that Ghidra uses to store all of your imported/analyzed programs.  I typically just store my programs in a directory at ~/ghidra_projects.  You can store it anywhere, but like @mumbel said, it's probably best to store it outside of the Ghidra installation directory.
As for your ghidra scripts, if you store them in ~/ghidra_scripts, you don't need to specify the -scriptPath parameter, as Ghidra knows to look in ~/ghidra_scripts automatically.
I also suspect your problem is the extra space between "$(pwd)/" and pyrogenesis, like @mumbel  said.


[e2e]: delegator-lowperms create main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/v46.0.27
Run: https://github.com/slsa-framework/example-package/actions/runs/5797349991
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-lowperms.create.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-lowperms.create.main.default.slsa3.yml
Trigger: create
Branch: v46.0.27
Date: Tue Aug  8 13:12:24 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/v46.0.28
Run: https://github.com/slsa-framework/example-package/actions/runs/5799242478
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.delegator-lowperms.create.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.delegator-lowperms.create.main.default.slsa3.yml
Trigger: create
Branch: v46.0.28
Date: Tue Aug  8 16:09:31 UTC 2023
Tests are passing now. Closing this issue.


Restart docker images after computer/docker restart

What is the best way to restart docker images after a computer/docker restart?

For traditional docker images spun up should be able to add the additional policy via $ docker update
So, for example, list your docker images
$ docker ps -a

Then obtain the container ID for the image you want to update  and update its policy
$ docker update --restart unless-stopped contianer_id

Also, make sure you have the setting "Start Docker Desktop when you log in" enabled

Would this make sense to add as a prompt when enabling a service? Ask if the user wants the service restarted with docker/computer and then run this command behind the scenes if it's desired?

Hm, I'm also wondering if it's possible to do that without the prompt. Problem is, if we auto restarted every container every time, some folks might not like that since they don't want all their containers always running, so we would have to keep track at any given moment of which are started and stopped, and that also seems like a pain...

This feels like a configuration that Takeout should not be managing. If a container needs to be started, just list the containers and select the ones you need to start. This will be a much smoother process in the node port, btw.


win-app-driver test fail

Actual behavior
86 tests in win-app-driver is failed

Hi @timpetroclio
make sure you follow the readme https://github.com/microsoft/PowerToys/tree/master/src/tests/win-app-driver to setup the testing environment properly.
The test are working and they all pass.
Thank you.

@enricogior
Thank you so much. I will close this issue


Keyborad Manager settings are not applied in browser

Microsoft PowerToys version
0.85.0
Installation method
GitHub, Microsoft Store
Running as admin
Yes
Area(s) with issue?
General
Steps to reproduce
Change A to B in Keyborad Manager
✔ Expected Behavior
When I type A in the browser, B appears
❌ Actual Behavior
It is replaced without any problem except when inputting in a browser, but it is not replaced in Chrome or Edge.
However, it is very rarely replaced and the behavior is unstable.
Is this a browser issue?
Other Software
No response

sorry. This page will be closed because I have registered multiple times.


monoculture testrun not working?

Hello Nanomotif-Developers
I wanted to test your tool on a monoculture as described in your readme.
I provided the files as suggested by your guide (bed-file was created with modkit using your command, the config file I created accounting to your test-data)
If I provide all the needed files I will get the suggestion on how to use nanomotif:
nanomotif find_motifs ref-fasta/barcode20.fasta barcode20_pileup.bed contig_bin.tsv -t 10 --out OUT
usage: nanomotif [-h] [--version]
                 {find_motifs,check-installation,detect_contamination,include_contigs,MTase-linker} ...

Motif identification and utilisation commands

positional arguments:
  {find_motifs,check-installation,detect_contamination,include_contigs,MTase-linker}
                        Command descriptions
    find_motifs         Identify methylated motifs on and contig and bin level
    detect_contamination
                        Detect contamination in bins
    include_contigs     Include contigs in bins
    MTase-linker        Commands related to MTase-linker

optional arguments:
  -h, --help            show this help message and exit
  --version             show program's version number and exit

pileup.bed: (modkit pileup --only-tabs ${modmapped_bam} ${name}_pileup.bed)
head barcode20_pileup.bed
contig_1        345     346     m       1       +       345     346     255,0,0 1       0.00    0       1       0       000       0
contig_1        348     349     a       1       +       348     349     255,0,0 1       100.00  1       0       0       000       0
contig_1        349     350     a       1       +       349     350     255,0,0 1       100.00  1       0       0       000       0
contig_1        351     352     a       1       +       351     352     255,0,0 1       0.00    0       1       0       000       0
contig_1        352     353     m       1       +       352     353     255,0,0 1       0.00    0       1       0       000       0
contig_1        353     354     m       1       +       353     354     255,0,0 1       100.00  1       0       0       000       0
contig_1        354     355     a       1       +       354     355     255,0,0 1       0.00    0       1       0       000       0
contig_1        357     358     m       1       +       357     358     255,0,0 1       0.00    0       1       0       000       0
contig_1        358     359     m       1       +       358     359     255,0,0 1       0.00    0       1       0       000       0
contig_1        360     361     a       1       +       360     361     255,0,0 1       0.00    0       1       0       000       0

cat contig_bin.tsv:
`cat contig_bin.tsv

contig_1    bar
contig_2    bar
contig_3    bar
contig_4    bar
contig_5    bar
contig_6    bar`

I'm not sure if I'm missing something
Kind regards
Mike

Hmm, can you provide the error message you get when you run the command?

this here was the error msg^^ :
usage: nanomotif [-h] [--version]
                 {find_motifs,check-installation,detect_contamination,include_contigs,MTase-linker} ...

Motif identification and utilisation commands

positional arguments:
  {find_motifs,check-installation,detect_contamination,include_contigs,MTase-linker}
                        Command descriptions
    find_motifs         Identify methylated motifs on and contig and bin level
    detect_contamination
                        Detect contamination in bins
    include_contigs     Include contigs in bins
    MTase-linker        Commands related to MTase-linker

optional arguments:
  -h, --help            show this help message and exit
  --version             show program's version number and exit

I didn't get anything else unfortunately. I copied it directly from the terminal

check_installation command works.. but with --version I get the Help-msg again
I'm using nanomotif in a docker

Could get you to run nanomotif --version? I suspect it stems from a faulty version, that have now been deprecated

0.3.0

Yes, that is the deprecated version, sorry for the incovenience. Yes 0.3.1 is working, the argumentparser might be subject to future reworks, but will come with a version bump 0.3.x -> 0.4.x

Thank you
Now I get proper error-msgs that I can work with :)
Btw. I use nanomotif in a docker.
should I provide you the docker receipt so other users have another option for easy installation?

That would be nice, thank you!

From nanozoo/template:3.8--12e1614
RUN apt-get update && apt install -y libzstd-dev libz-dev gcc procps git gzip tar wget && apt-get clean && rm -rf /var/lib/apt/lists/* /tmp/* /var/tmp/*
RUN python -m venv nanomotif
RUN . nanomotif/bin/activate # . is eaqual command to "source"
RUN pip install --upgrade pip
RUN pip install --upgrade setuptools wheel
RUN pip install nanomotif
RUN nanomotif check-installation


Bootgrid append limiting to 16 entries

Issue Adding Rows with "append"

I'm trying to append a relatively large collection to bootgrid and it appears to be limiting the number of
elements I can append. Is there some sort of setting I need to enable to allow for larger data sets?
I'm calling:
$('#grid-basic').bootgrid("append",arr);
my array has a length of 366.

I see my problem is link some how to the data-identifier attribute, does this constrain uniqueness?

Maybe this will help you: https://github.com/rstaib/jquery-bootgrid/pull/285


[outlook-module] Recipient not filled correctly for Disposition-Notification-To (and others)

When parsing msg with valid Disposition-Notification-To or Return-Receipt-To the Recipient is not filled correctly. I assume the same applies to Return-Path.
Sample header: Disposition-Notification-To: Fname Lname<EMAIL_ADDRESS>Expected: Recipient{name='Fname Lname'<EMAIL_ADDRESS>type=null}
Actual: Recipient{name='null', address='Fname Lname<EMAIL_ADDRESS>type=null}
The problem happens in OutlookEmailConverter.parseHeader(String headerName, String headerValue, EmailPopulatingBuilder builder) where the header value is passed to EmailPopulatingBuilder.withDispositionNotificationTo(String address).

Do you have a message for me to work with?

Yes, sorry. The original mail was confidential. I was able to reproduce it with outlook by selecting 'Request a <Delivery|Read> Receipt'.
sample.zip
When calling EmailConverter.outlookMsgToEmail you can see the incorrect recipients in the output. Return-Receipt-To still contains the < and > in the address and Disposition-Notification-To has the name and email in the address.

Some context to the problem we are facing: we have this problem with msg #40. As a workaround, we convert msg -> Email -> eml -> Email. This way we have access to the attachments which were not available in the original msg -> Email.
When we have a signed email which has one of the headers above, the first step msg -> Email stores the wrong recipient addresses. Later in eml -> Email, it fails with AddressException: Missing '<' in string ``<"Dude"<EMAIL_ADDRESS>because the address was parsed incorrectly in the first step and thus converted wrongly to eml in Email -> eml.
Stacktrace:
org.simplejavamail.converter.internal.mimemessage.MimeMessageParseException: Error parsing [Disposition-Notification-To] address [<"Dude"<EMAIL_ADDRESS>    at org.simplejavamail.converter.internal.mimemessage.MimeMessageParser.createAddress(MimeMessageParser.java:379)
    at org.simplejavamail.converter.internal.mimemessage.MimeMessageParser.parseHeader(MimeMessageParser.java:223)
    at org.simplejavamail.converter.internal.mimemessage.MimeMessageParser.parseMimePartTree(MimeMessageParser.java:160)
    at org.simplejavamail.converter.internal.mimemessage.MimeMessageParser.parseMimeMessage(MimeMessageParser.java:153)
    at org.simplejavamail.converter.EmailConverter.mimeMessageToEmailBuilder(EmailConverter.java:138)
    at org.simplejavamail.converter.EmailConverter.mimeMessageToEmailBuilder(EmailConverter.java:125)
    at org.simplejavamail.converter.EmailConverter.emlToEmailBuilder(EmailConverter.java:398)
    at org.simplejavamail.converter.EmailConverter.emlToEmail(EmailConverter.java:327)
    at org.simplejavamail.converter.EmailConverter.emlToEmail(EmailConverter.java:319)

I found the problem to be more widespread, where the builder api for setting email addresses from strings doesn't call the appropriate parser method, which should separate the (optional) name part from the address part. It was a quick fix as all the logic was in place, but it wasn't called from all the appropriate places.
Thank you for your analyses as well as the example email, it helped me verify the fix real quick.

Fix released in 8.5.0.


Adding missing callback param to _getTriggerSmartContractArgs

As you can see here, the variable callback is not defined:
https://github.com/tronprotocol/tronweb/blob/d2a300aedf1b1ce0e9008f424f77988752d2ecd1/src/lib/transactionBuilder.js#L1351

We removed the usage of callback on this method. Please try using promise.

I was using a promise and was never using a callback, the callback issue
was happening internally to tronweb, please look at where the variable was
missing.
On Thu, Sep 28, 2023 at 8:29 PM start940315 @.***>
wrote:

Closed #425 https://github.com/tronprotocol/tronweb/pull/425.
—
Reply to this email directly, view it on GitHub
https://github.com/tronprotocol/tronweb/pull/425#event-10506066643, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAABZE7HTWL2ZSVGUGFDK3TX4YXCLANCNFSM6AAAAAA5AOPXUA
.
You are receiving this because you authored the thread.Message ID:
@.***>

It seems like you have put some error data in parameter argument. We have fixed the callback problem in v6, thank you for your pr.


Missing Files

Thank you very much for your excellent research, but there was a bit of a problem when I tried to reproduce it.
The following error occurs when I run bash benchmark/benchmark_llava_gqa.sh.
How can I get the corresponding file? Can you directly provide the download of the corresponding file？

Hi! It seems that you are running the benchmark code to evaluate the performance on the gqa dataset. This dataset is very large, so I did not upload it on GitHub.
You can follow the official evaluation guide in llava here, which provides link to download the gqa dataset, along with others (such as VQA, etc).


Lean backend upgrade

What

More flexible setup mechanism (easy to change Lean version, directory, etc.)
Better error messaging
Partial point grading
Add tests
Speed up compile process by adding the right flags to the lean compiler
Full support of the Poller interface (e.g. multiple check entries)

Why
Because the current Lean implementation is lacking all features mentioned in What.
How

Add variables to setup mechanism
Use the error messages returned by the lean compiler and add custom ones for axioms
Invoke the leanchecker for every theorem/lemma found in check.lean.

Testing

Tests can be executed using tests_run.sh
I uploaded some submissions (1, 2, 3), and they seem to work.

Notes

Partial points need to be tested (I was not sure about how to test that)
By default, Lean is installed to /tmp/lean-grader/<version>. Feel free to change that in variables/lean_version
Other TODOs: see TODOs in README.md

the setup works out of the box, nice. also the deployed judge seems to work fine on the test server.
Yet, there are still some problems.
checking submission 82 gives good results but also prints the following errors:
--- Logging error ---
Traceback (most recent call last):
  File "/usr/lib/python3.6/logging/__init__.py", line 995, in emit
    stream.write(msg)
UnicodeEncodeError: 'latin-1' codec can't encode character '\u03a0' in position 47: ordinal not in range(256)
Call stack:
  File "/tmp/lean-grader/3.4.2/grader.py", line 81, in <module>
    logger.info(checker_result.stdout)
Message: 'axiom propext : \u03a0 {a b : Prop}, (a <-> b) -> a = b\ntheorem lemma2 : \u03a0 n : nat, n <= my.sum n\nchecked 523 declarations\n'
Arguments: ()
--- Logging error ---
Traceback (most recent call last):
  File "/usr/lib/python3.6/logging/__init__.py", line 995, in emit
    stream.write(msg)
UnicodeEncodeError: 'latin-1' codec can't encode character '\u03a0' in position 54: ordinal not in range(256)
Call stack:
  File "/tmp/lean-grader/3.4.2/grader.py", line 81, in <module>
    logger.info(checker_result.stdout)
Message: 'axiom i_am_a_cheater : \u03a0 M : nat, Exists (\u03bb N : nat, \u03a0 n : nat, n >= N -> my.sum n >= M)\ntheorem lemma3 : \u03a0 M : nat, Exists (\u03bb N : nat, \u03a0 n : nat, n >= N -> my.sum n >= M)\nchecked 57 declarations\n'
Arguments: ()

do the UnicodeEncodeError s also occur with you? maybe some different python version?
regarding the partial points:
http://vmnipkow14.in.tum.de/competitions/submission/82/
your judgement is the following:
{"submission_is_valid": false,
"messages": [{"where": "submission.lean at line 3, column 0", "what": "WARNING: declaration 'goal1' uses sorry"},
   {"where": "check.lean at line 1, column 0", "what": "WARNING: imported file 'submission.lean' uses sorry"},
   {"where": "<unknown> at line 1, column 1", "what": "ERROR: failed to expand macro"},
   {"where": "lemma3", "what": "WARNING: Unknown axiom 'i_am_a_cheater' used to prove theorem 'lemma3'."}],
 "checks": [{"name": "lemma1", "result": "error"},
   {"name": "lemma2", "result": "ok"},
   {"name": "lemma3", "result": "ok_with_axioms"}], "log": ""}

I'm not an Lean expert, but I guess for this submission, the submission should be valid ("submission_is_valid": true) and the result for lemma1 should be "ok_with_axioms", then the frontend would count the submission as "passed" and assign it 1/3 points.
it seems that using sorry yields an error in your setup, can you change that?

Partial points work now, see http://vmnipkow14.in.tum.de/competitions/submission/82/

I do not get the  UnicodeEncodeError on my machine. This seems to be a problem with locales. cf

https://github.com/jupyterhub/jupyterhub/issues/1406
https://github.com/WordOps/WordOps/issues/11


[DOC] SlidingWindowSplitter

Describe the issue linked to the documentation
Additional docstring here would be great to better understand how to configure this:

class SlidingWindowSplitter(BaseWindowSplitter):
    """Sliding window splitter
    Parameters
    ----------
    fh : int, list or np.array
        Forecasting horizon
    window_length : int
    step_length : int
    initial_window : int
    start_with_window : bool, optional (default=True)
    """

This is related to #477

Closed by #555


[3231] Add update service to allocations

Context
https://trello.com/c/mC7AqJvX/3231-s-11-change-my-allocations-request
Changes proposed in this pull request

Add update service to set number of places appropriately for allocation

Guidance to review
Checklist

[ ] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

 [S] 11. Change my allocations request


Add debouce to example

What:
Adding debounce to Axios example

Why:
It was commented to add it, not necessary.

How:
N/A

Checklist:

[ ] Documentation
[ ] Tests
[x] Ready to be merged
[x] Added myself to contributors table

Codecov Report

Merging #236 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #236   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           4      4
  Lines         277    277
  Branches       63     63
=====================================
  Hits          277    277

Continue to review full report at Codecov.

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

Done, and thank you for looking at it.  Always like contributing to your repos they are very approachable to new devs 👍.


Fix #1422710, 1422715 - Fix use of Recolor tool after use of Editable sh...

...ape.
The issue is that Recolor temporarily draws on the ToolLayer, which it expects to be invisible.  Editable shapes make the ToolLayer visible when activated but never hides it, so the Recolor tool's temporary work is visible.  Fix by hiding the tool layer when an editable tool finalized all its shapes.  Doing it on tool deactivation misses a case where you switch to a new image and then change the tool.

If I do Recolor -> Line/Curve -> Recolor, and then undo 3 times, the last Recolor operation appears again. Maybe the Recolor tool needs to clear out the ToolLayer when it's done with it?

I think any user of the ToolLayer is supposed to manage its own usage of it, which is:

Turning it on and off when activated/deactivated.
If you turn it on, managing what the content is.

So I think the proper way to do this is the Editable shapes tool needs to clear it when it wants to turn it on.  (Or else we will just hit this again if a 3rd party Add-In tool leaves work on the ToolLayer.)

Fixed the edit engine to always clear the ToolLayer before using it.


HAWQ-713 Make lc_numeric guc to have GUC_GPDB_ADDOPT

Without GUC_GPDB_ADOPT, the values for guc 'lc_numeric' is not dispatched
to the QE processes.
This fix is ported from GPDB.

👍
+1

merge done

Thanks for the review and merge


Initial project major features

These tasks are things that need to be done for setting up the project:

[ ] #4
[ ] #5
[ ] Favicon for the website
[ ] Tweet about it to get others on board

Would you consider the name of an application? Or is it mish-friendly-food?


Basic logging for job and task names

This needs more work before considering it for production

This needs more work before considering it for production


Backport: Update reactor error handler

Backport to 2.5.3-SNAPSHOT
#10323

Applied to version 2.5.3


Exceptions while starting example

There are a lot of similar exceptions at start.
Exception thrown: 'System.Collections.Generic.KeyNotFoundException' in mscorlib.dll
Exception thrown: 'WpfMath.FormulaNotFoundException' in WpfMath.dll

Ah, ok, I see. It's internal parsing machinery.
            // TexFormulaParser.cs
            // lines 342-357
            try
            {
                symbolAtom = SymbolAtom.GetAtom(command);
            }
            catch (SymbolNotFoundException)
            {
            }

            try
            {
                predefinedFormula = GetFormula(command);
            }
            catch (FormulaNotFoundException)
            {
            }

I have encountered that, too, and I think that should be fixed. We'll just need to replace these with a calls to Dictionary.TryGetValue(). I don't like that usage of exceptions for normal program execution flow.


feature bundles [sf#93]

Reported by nlw on 2008-03-10 22:18 UTC
the idea that describes a "bundle" as a set of sequence_features.
some examples of uses might include:
a bundle of ESTs that go to support a cDNA
a bundle of ESTs that are temporally expressed
a bundle of traces that support a contig
feature bundle would then have children like:
EST feature bundle
cDNA feature bundle
Trace feature bundle
the example of EST feature bundle would have children ESTs with a has_part relationship
then, we could use these bundles as evidence for another feature.  for example, an mRNA could has_evidence from a bundle of ESTs (est1, est2, est3...)

Updated by nlw on 2008-03-11 18:36 UTC

priority: 5 --> 8

Commented by eilbeck on 2008-03-11 19:23 UTC
Logged In: YES
user_id=742851
Originator: NO
Hi Nicole,
I think this is the same idea as a collection that I discussed a long while ago - before you joined.
It would be a set of features that are related in some way but not necessarily continuous with each other.
Kinds of collection could then be things like EST_collection?
I would rather call this collection than bundle.
I think there is some provision for collections in the BFO too.
I would like to add this to SO. It would not be a feature however as it is not continous. It would have to be another root node.
Is this the same line as you were thinking?
Is there a deadline on your end for this?
--K

Commented by cmungall on 2008-03-19 20:46 UTC
Logged In: YES
user_id=254723
Originator: NO
I agree with Karen.
sequence_feature_collection may also provide a home for SO:0001026 genome.
It is tricky though. collections do not necessarily have overlaps relations between their members. This means software should perhaps treat them slightly different from sequence_features. But Nicole's use case looks like it consists of co-overlapping sets. This doesn't mean collections should not be used, but perhaps there is a simpler solution.
For example, perhaps it is not the collection of ESTs that form evidence but instead their alignment to the genome, i.e. the match object.
Also it's not really ESTs or collections of ESTs that are co-expressed, it's RNA. According to SO the relation between the EST and expression event is a composite relation derives_from * expressed_in. We can name this composite relation if it is useful.

Commented by eilbeck on 2008-03-25 22:45 UTC
Logged In: YES
user_id=742851
Originator: NO
Hi Nicole,
So I think we are making headway on this.
Chris is right, since your features are all overlapping, a feature collection would be overkill.
I think what would be better for this would be an overlapping_feature_set.
There is also a tracker open for topological relations, which would provide the overlap relation needed for this.
Does this make sense? Would there be a case where the 'bundles' contain things that are not overlapping?
Let me know...
--K

Commented by eilbeck on 2008-07-09 15:58 UTC
Logged In: YES
user_id=742851
Originator: NO
overlapping_feature set was added to experimental result region as a subtype. It has the definition:A continuous region of sequence composed of the overlapping of multiple sequence_features, which ultimately provides evidence for another sequence_feature.
Also a child overlapping_EST_set was added.

Updated by eilbeck on 2008-07-09 15:58 UTC

status: open --> closed-accepted


feat: publish tag without patch version

Summary
Closes https://github.com/astral-sh/uv/issues/6678
This change would publish an additional tag that includes only major.minor.
For a release with x.y.z, this would publish the tags:

ghcr.io/astral-sh/uv:latest
ghcr.io/astral-sh/uv:x.y.z
ghcr.io/astral-sh/uv:x.y

Awesome, thank you! Did you test this in a fork?

Yup, here's a run https://github.com/samypr100/uv/actions/runs/10587568664


badarith,[{erlang,'+',[infinity,64000]

On load testing current master tag 3000rps get crashes
=ERROR REPORT==== 30-Oct-2019::09:55:08.485504 ===
Ranch listener my_http_listener had connection process started with cowboy_clear:start_link/4 at <0.29776.42> exit with reason: {badarith,[{erlang,'+',[infinity,64000],[]},{cowboy_http,commands,3,[{file,"/home/alexs/work/psp/_build/default/lib/cowboy/src/cowboy_http.erl"},{line,948}]},{cowboy_http,loop,1,[{file,"/home/alexs/work/psp/_build/default/lib/cowboy/src/cowboy_http.erl"},{line,234}]},{proc_lib,init_p_do_apply,3,[{file,"proc_lib.erl"},{line,249}]}]}

It's missing a clause for when a flow (read body) command arrives after the body was finished reading.

And?
I use
init(Req, Opts) ->  M = cowboy_req:method(Req),  B = cowboy_req:read_urlencoded_body(Req)  ...
read_urlencoded_body crashed
Is here need some patch?

I'm just saying why there's a crash, I'll work on a fix later.

Seeing the same crash as @stofel also under heavy load
Also a very similar one soon there after. Looks like the same issue.
"Ranch listener ApiServicesWeb.Endpoint.HTTP had connection process started with :cowboy_clear:start_link/4 at #PID<0.19133.8> exit with reason: {:badarith, [{:erlang, :-, [:infinity, 1460], []}, {:cowboy_http, :update_flow, 3, [file: '/home/.../deps/cowboy/src/cowboy_http.erl', line: 381]}, {:cowboy_http, :after_parse, 1, [file: '/home/.../deps/cowboy/src/cowboy_http.erl', line: 358]}, {:proc_lib, :init_p_do_apply, 3, [file: 'proc_lib.erl', line: 249]}]}\n",

Feel free to experiment with a fix I don't think I'll get to it this week.

Please try this patch:
diff --git a/src/cowboy_http.erl b/src/cowboy_http.erl
index f128a44..d60f4e0 100644
--- a/src/cowboy_http.erl
+++ b/src/cowboy_http.erl
@@ -938,6 +938,9 @@ commands(State=#state{out_streamid=Current, streams=Streams0}, StreamID, Command
    Streams = lists:keyreplace(StreamID, #stream.id, Streams0,
        Stream#stream{queue=Queue ++ Commands}),
    State#state{streams=Streams};
+%% When we have finished reading the request body, do nothing.
+commands(State=#state{flow=infinity}, StreamID, [{flow, _}|Tail]) ->
+   commands(State, StreamID, Tail);
 %% Read the request body.
 commands(State=#state{socket=Socket, transport=Transport, flow=Flow0}, StreamID,
        [{flow, Size}|Tail]) ->

Tried this patch. Seems to have fixed half of the problem. Still see the issue on line 381
"message": "Ranch listener ApiServicesWeb.Endpoint.HTTP had connection process started with :cowboy_clear:start_link/4 at #PID<0.20703.5> exit with reason: {:badarith, [{:erlang, :-, [:infinity, 1460], []}, {:cowboy_http, :update_flow, 3, [file: '/home/travis/build/ni-kismet/helium-dataservices/deps/cowboy/src/cowboy_http.erl', line: 381]}, {:cowboy_http, :after_parse, 1, [file: '/home/travis/build/ni-kismet/helium-dataservices/deps/cowboy/src/cowboy_http.erl', line: 358]}, {:proc_lib, :init_p_do_apply, 3, [file: 'proc_lib.erl', line: 249]}]}\n"

That's not supposed to happen, if it occurs it must be because of a different bug. The value is set to infinity when the stream was finished reading or its body will be skipped (so the stream is removed from the list). But this error occurs in code where the stream was not removed?
Do you have any other errors or some more information about the request that's causing this?

We are also seeing from occasionally the badarith error on the call to update_flow/3.
We don't have much context on the error. What information would you need to investigate further? Maybe I can patch cowboy to add some more logging and let it run for a while until the error happens again.

The state would be great. Maybe just adding when Flow0 =/= infinity would be enough to have the state in the error report, otherwise logging when Flow0 =:= infinity.

@essen the crash on the update_flow still appears to have been fixed. No crashes there since we deployed the latest cowboy master.
But now we noticed that we have also been having the badarith on cowboy_http:commands/3
Example of the crash:
Process terminated with reason: {{error,badarith},
 [{erlang,'+',[infinity,8000000],[]},
  {cowboy_http,commands,3,[{file,"/build/example/_build/default/lib/cowboy/src/cowboy_http.erl"},{line,979}]},
  {cowboy_http,loop,1,[{file,"/build/example/_build/default/lib/cowboy/src/cowboy_http.erl"},{line,257}]},
  {proc_lib,init_p_do_apply,3,[{file,"proc_lib.erl"},{line,249}]}]}

Do you need some extra information to debug, or is there any patch already that you want us to try?

Did you try with this patch? https://github.com/ninenines/cowboy/issues/1411#issuecomment-549314493
Anyway both have been merged so maybe just use master.

We are using the latest master (752297b1539b4f74c9329c873f78485a52b5b8bd).

I guess it hasn't been merged or pushed or something, I don't see it. So apply the patch linked on top and you should not get the error.

The code changed in the meantime but I imagine this would be the patch with the current master:
diff --git a/src/cowboy_http.erl b/src/cowboy_http.erl
index 9f0b865..ccdf615 100644
--- a/src/cowboy_http.erl
+++ b/src/cowboy_http.erl
@@ -970,6 +970,9 @@ commands(State=#state{out_streamid=Current, streams=Streams0}, StreamID, Command
        Streams = lists:keyreplace(StreamID, #stream.id, Streams0,
                Stream#stream{queue=Queue ++ Commands}),
        State#state{streams=Streams};
+%% When we have finished reading the request body, do nothing.
+commands(State=#state{flow=infinity}, StreamID, [{flow, _}|Tail]) ->
+        commands(State, StreamID, Tail);
 %% Read the request body.
 commands(State0=#state{flow=Flow0}, StreamID, [{flow, Size}|Tail]) ->
        %% We must read *at least* Size of data otherwise functions

Yes.

I've pushed the fix with a test case in 8fc3da2fc31a04281b0ba8dc0cdc1e1650067da5. I think this can be closed now. Thanks everyone for the reports and testing! This will be part of upcoming Cowboy 2.8.


CAMEL-20517 camel-jbang-plugin-k doesn't recognize command

Description
Instead to spawn a new process, this fix downloads the camel-jbang plugin and run it as part of the same camel-jbang OS process.
The change in DefaultClassResolver.java was motivated as when searching for a resource file from the downloaded maven artifact it's not available from the classloader in getApplicationContextClassLoader() , so it has to search in the list of classloaders
I will add some tests, but in the meantime I would like some preliminary review.

Target

[x] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking
https://issues.apache.org/jira/browse/CAMEL-20517

Apache Camel coding standards and style

[x] I checked that each commit in the pull request has a meaningful subject line and body.

[x] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

I would like if you can try make this without changing code in the core modules in that classloader. So instead can you add a custom classloader into camel-jbang and use it there

@davsclaus @christophd ready for review.


Updated UCWA general reference content

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Skype/UCWA/UCWA2_0GeneralReference.md
:white_check_mark:Succeeded

Skype/WebSDK/docs/SkypeWebSDKVersionUpdateNotifications.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Typos

Description
Few typos. In particular, jackknife instead of Jackknife

Hello @gmartinonQM! Thanks for opening this PR.

In the file mapie/estimators.py:

Line 292:13: W503 line break before binary operator
Line 293:13: W503 line break before binary operator

Do see the Hitchhiker's guide to code style


PP-5613 Add filename to logger

Use Winston's child function and override the logger attribute with the name
of the file in each place we're using a logger. This will help us track down errors to the file
that they're being thrown in (like we do with Java apps).
How to test
Check the log output in the tests to check that logger corresponds to the correct file.

Logging lines look good so I'll merge.
{"logger":"/app/common/middleware/check-secure-cookie/check-secure-cookie.js","message":"[] Valid session defined for sdfihsdufh2e123","level":"info","label":"direct-debit-frontend-sl-beta","timestamp":"2019-09-06T13:40:49.752Z"}


Remove stale 'xdm:stateProvinceISO' from examples

Resolves Issue #220

@asteed please take a look at d805fa1


Incorrect(?) SDK Invalidation: SDK assumes all JWKs must conform to SHC rules

Problem
SHC IG specifies requirements for public JWK to specify signature validation against (use: sig; alg: ES256, etc). However, the SHC IG does not specify that all public keys hosted a domain.com/.well-known/jwks.json must be a SHC-conforming JWK: it's possible that orgs might want to host other various JWKs there and only the JWK with the kid matching that in the VC must conform.
Solution
Update to only validate against the JWK matching the kid, not all keys in the key-set
https://github.com/microsoft/health-cards-validation-SDK/blob/fb6a527401e7e49d9bd0433c465ab4bac6498c31/src/shcKeyValidator.ts#L174
cc @jmandel to make sure I'm interpreting the SHC IG correctly

@christianpaquin assuming we are aligned that this is a problem worth solving, I'm happy to help with a patch here.

Yes, @dleve123, JWK sets certainly can have more than one key. The tool currently gives warnings for keys that cannot be used for Health Card issuance. The same key validation code runs when you validate a key set directly or when one is downloaded while validating a health card. We chose to be overly verbose, to allow developers to be warned about the contents of their key set (before putting them online), to make sure they don't include something in there by mistake.
What you propose is certainly what a real-life verifier should do. We could be more lenient when validating a health card, not displaying the warnings if one key in the set is ok and can be used with a card, but being a developer tool, it feels like we should report as much as we can.

What you propose is certainly what a real-life verifier should do. We could be more lenient when validating a health card, not displaying the warnings if one key in the set is ok and can be used with a card, but being a developer tool, it feels like we should report as much as we can.

Gotcha - I appreciate the nuanced thinking between warning and errors! I would learn towards not including warnings due to unrelated keys, but am fine with either decision here. Feel free to close at your discretion.

It might be worth adding a way for developers to explicitly opt out of certain checks (e.g., a suppression file, or a suppression CLI flag). The first time you get a warning, it might come with advice like:

If you don't want to see this warning in the future you can [...]

While I'm exercising my imagination...the suppression could be scoped to types of errors, or even types of errors combined with data (like, "don't warn me about extra keys" vs "don't warn me about this extra key).
(The common version of this is for Node testing frameworks, where you can specify --include or --exclude type flags to filter out the set of tests applied.0

It might be worth adding a way for developers to explicitly opt out of certain checks (e.g., a suppression file, or a suppression CLI flag). The first time you get a warning, it might come with advice like:

Yes, and it should be fairly easy to do since our logger takes in specific error codes which should be easy to filter out.

Ok, so I'll close this, and keep @jmandel's suggestion alive on issue #35.


Tag does not auto destroy when tagging moves to another tag

When I am doing moving tagging to another tag ( using tagging.update(tag: another_tag)), even the current tag has taggings_count to be zero, the current_tag is not deleted.

Why not delete the tagging and create it in another model ?
I think you are bypassing the Activerecord callbacks

Thank @seuros


Adding Dockerfile to lower the barrier for build and packaging m3dbnode

Adding a simple Docker file to help bootstrap developers with kicking the tires of m3dbnode

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


Scoped git services

Attempt to fix #57

Hi @joaomoreno, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Joao Moreno). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


[v1.13] vagrant: Bump Vagrant box versions

These include Go 1.21 which is needed to update the Go toolchain to a supported version in #29817.

/test-backport-1.13

/test-1.21-4.19

/test-1.26-net-next

/test-runtime

/test-runtime

/test-runtime


Code Mirror pushing to Prism stripping out closing HTML brackets, "</" are missing

Information:

Prism version: PrismJS 1.17.1
Environment: Code MIrror, Browser

Does the latest version of Prism from the download page also have this issue?
Yes
Description
I am using code mirror to push written code into a prism box for easy writing and easy viewing. I am getting the innerHTML value and passing it straight to prism for styling. When I write a block of HTML specifically (all other languages work), the output of the final product strips out the closing bracket and slash and puts it into an HTML comment.
The image below shows an example. The left is the output while the right is what the dev tools is showing.

The code input into Code Mirror is the following:

Seems like a double-unescape issue. Could you please post the code which uses Prism to highlight the Code Mirror text and the code which inserts the Prism output into the DOM?

Seems like a double-unescape issue. Could you please post the code which uses Prism to highlight the Code Mirror text and the code which inserts the Prism output into the DOM?

I'm not 100% sure if this is what you're asking for since it's a bit of a customized solution we have, but the DOM is essentially created through cases of data-types, if it is the data-type of a code block editor, it does the following:
    case "codeblockeditor":
      var o = {"Column": colNameInternal, "ID": internalID, "OnChange": CopyCodeMirrorToSource, "OnSave": CopyCodeMirrorToSource};
      var tag="textarea";
      $("._moduleSettings").html("
Place your code below:
<"+tag+" id='CodeMirrorEditor_TextArea' style='width:100%;height:75%;'>" + source.html() + "</"+tag+">");
      $("#CodeMirrorEditor_TextArea").CreateCodeMirrorEditor(o); // Creates an instance of a code mirror editor
      break;

and later:
var CopyCodeMirrorToSource = function() {
  var activeParam = $(".paramName.active");
  var paramValueNode = activeParam.next(".paramValue");
  var fix = paramValueNode.html($.fn.CreateCodeMirrorEditor.ActiveEditor.getValue())[0].innerHTML;
  paramValueNode.html(fix);
  paramValueNode.trigger("change");
  $(".ToolBar ._rtSave").remove();
}

So basically the user is inserting code in a preview window via code mirror, and every change/save it is being pulled into the DOM. The styling/highlighting for the prism syntax is just taken from the default Prism JS and CSS files that were downloaded from the site. Let me know if this isn't helpful.

Unfortunately, that isn't enough for me to pin down the problem. Could you please also provide the code which calls any of Prism highlight functions?
Also, Prism doesn't use the innerHTML property of an element to extract the code to be highlighted; It uses textContent. Maybe that's the cause of the issue?
Also also, have you verified that it's actually Prism causing parts of the user's code to disappear?

Use your browser's debugger. This is the line where you want to set the breakpoint. It's where Prism extracts the text. From what I heard from you, the problem most likely lies somewhere in your customized solution, so I will close this for now.
From this line, you should be able to backtrack to the cause of this issue.
Unfortunately, without a look into your code, I can't help you more than that.


Zones sometimes change on save and load

Sometimes when saving and loading the zones on the map change. This likely has something to do with map RNG being modified at an inopportune time.
Might make snese to look at which zones use map rng in them and see if those are the culprits.

Solved with #170


Split header tooltip sometimes gets stuck

Checklist

[X] I'm reporting a problem with Chatterino
[X] I've verified that I'm running the most recent nightly build or stable release
[X] I've looked for my problem on the wiki
[X] I've searched the issues and pull requests for similar looking reports

Describe your issue
Right-clicking on a split header while the Chatterino window is inactive causes the split header tooltip stay visible until you hover over the split header again. Normally, the tooltip should disappear as soon as you right-click.
To reproduce:

Focus any other window than Chatterino
Right-click on the split header (this might take a few tries)
Split header tooltip stays stuck until the split header is hovered over again

Screenshots
https://github.com/Chatterino/chatterino2/assets/58441/0dc78732-6c5f-4aea-8127-d25a0a283013
OS and Chatterino Version
Chatterino Nightly 2.5.0-beta.1 (commit 25a69fd1) built on 2024-04-07 with Qt 6.5.0, MSVC 193833135 Running on Windows 11 Version 24H2, kernel: 10.0.26090

can replicate


live reload not possible without body or head tag

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ x] Bug report  <!-- Please search GitHub for a similar issue or PR before submitting -->
[ ] Feature request
[ ] Documentation issue or request
[ ] Other: <!-- Please describe: -->

Current behavior

Live reload not possible without html or body tag
Expected behavior
no showing this alert

auto reload page on save
Environment
Browser:

- [x ] Chrome (desktop) version XX
- [ x] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:

- Live Server: <!-- Check which version is installed -->
- 5.7.5
- Platform:  <!-- Mac, Linux, Windows -->
- Visual Studio Code: <!-- `code --version` -->

Others

Make sure your html has proper format. I mean,


[Bug] Fail to load MR image

Describe the Bug

245C095C.zip
Steps to Reproduce
upload attach image to
https://viewer.ohif.org/local
The current behavior
image not displayed
The expected behavior
render the image
OS
Ubuntu 22
Node version
18
Browser
firefox and chrome

Yeah i see it is not working

The issue appears to be the range of voxels which are extremely small , hmmm

related https://discourse.canceridc.dev/t/dicom-window-level-formula-adjustment/655


[EOC-603] rodo information use solution from sandtime - PART 1 (app)

This branch contains only changes for application. For landing, I'll create separate PR.
Privacy-Policy and TermsOfUse are taken from SandTime. I changed links and email addresses and information about things that we don't use for sure: e.g tracking geolocation, store payment data, etc. I changed also list of vendors that we can pass to or get user data from eg. Amazon WS...
I'll send this policy information to statements to Łukasz for consultation.

I changed the license statement but: MIT license applies to the software, code. Of course, users (developers) can do everything with the software (code) and we provide no warranty for this. But here we have deployed and a full-working web application where Sandstream is the owner and administrator. So I think we have to write certain rules that apply to application users (not developers who can use the code any way they like).


Change command-line VM default to Dart 2

@a-siva I believe this should be pushed to Dart2Stable (we should be able to do it end of May, but not in time for the Beta4 milestone).

The current VM does not support dart --no-preview-dart-2. It would be nice if it did for a backwards compatible transition period.

https://dart-review.googlesource.com/c/sdk/+/58240 has landed.


HSEARCH-5045 Raise various default max-connection config properties for the Elasticsearch backend

https://hibernate.atlassian.net/browse/HSEARCH-5045

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.

I wonder whether it would make sense to make the total max connections some high number (e.g. Integer.MAX_VALUE) by default and let the per-route value keep things limited?

I wonder whether it would make sense to make the total max connections some high number (e.g. Integer.MAX_VALUE) by default and let the per-route value keep things limited?

Seems to me this could lead to problems with large Elasticsearch cluster? I.e. with a 20-node cluster, one app instance could end up opening 400 connections...

good point! 😃 thanks!


[20.09] python3Packages.pymetno: 0.5.1 -> 0.8.1 (#100131)

(cherry picked from commit afbf62d179bff8b3191316eab9890255b512bf78)
Home-assistant 0.114.4 does not work with the old version of this
package.
Original PR #100131.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[X] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Since home-assistant seems to be the only direct user in nixpkgs, this is probably fine to backport.
Result of nixpkgs-review pr 108306 run on x86_64-linux 1

2 packages built:

python37Packages.pymetno
python38Packages.pymetno

Since home-assistant seems to be the only direct user in nixpkgs, this is probably fine to backport.
Result of nixpkgs-review pr 108306 run on x86_64-linux 1

2 packages built:

python37Packages.pymetno
python38Packages.pymetno


Weird behavior when debugging Cosmos Db Distributed Cache with Sessions

Describe the bug
I've installed the Microsoft.Extensions.Caching.Cosmos package and implemented it in a (Blazor Webassembly) asp.net core application. Next I access the session object like so:
var session = HttpContext.Session;
if (!session.IsAvailable)
      await session.LoadAsync();

session.Set("test", System.Text.Encoding.UTF8.GetBytes("test"));
session.TryGetValue("test", out byte[] test);

As soon as I try to access the Session object in debug mode, there is some odd behavior. When I put a breakpoint on the var session line, I can see that the Session Id is Empty, and IsAvailable is false. LoadAsync() does not change this. If i execute a Set or a Get after that, my output window shows some connection errors:
DocDBTrace Information: 0 : RequestAsync failed: RID: dbs/<containername>/colls/<databasename>/docs/0693b8bd-20e2-8783-478d-f59023520f70, Resource Type: Document, Op: (operationType: Read, resourceType: Document), Address: rntbd://cdb-ms-prod-westeurope1-fd75.documents.azure.com:14012/apps/d87b8a42-9922-422e-8c91-e5e66bb6b76e/services/fe07a52b-7665-437b-8fa1-a6c27cc946da/partitions/3b498b24-1743-4fd3-9125-c6f14bb9095a/replicas/132711612049640158s/, Exception: System.Threading.ThreadAbortException: System error.
   at Microsoft.Azure.Documents.Rntbd.LoadBalancingPartition.RequestAsync(DocumentServiceRequest request, TransportAddressUri physicalAddress, ResourceOperation resourceOperation, Guid activityId, TransportRequestStats transportRequestStats)
   at System.Runtime.CompilerServices.AsyncMethodBuilderCore.Start[TStateMachine](TStateMachine& stateMachine)
   at Microsoft.Azure.Documents.Rntbd.LoadBalancingPartition.RequestAsync(DocumentServiceRequest request, TransportAddressUri physicalAddress, ResourceOperation resourceOperation, Guid activityId, TransportRequestStats transportRequestStats)
   at Microsoft.Azure.Documents.Rntbd.LoadBalancingChannel.RequestAsync(DocumentServiceRequest request, TransportAddressUri physicalAddress, ResourceOperation resourceOperation, Guid activityId, TransportRequestStats transportRequestStats)
   at Microsoft.Azure.Documents.Rntbd.TransportClient.InvokeStoreAsync(TransportAddressUri physicalAddress, ResourceOperation resourceOperation, DocumentServiceRequest request)
Microsoft.AspNetCore.Session.DistributedSession: Error: Session cache read exception, Key:0693b8bd-20e2-8783-478d-f59023520f70

System.Threading.ThreadAbortException: System error.
   at Microsoft.Extensions.Caching.Cosmos.CosmosCache.GetAsync(String key, CancellationToken token)
   at System.Runtime.CompilerServices.AsyncMethodBuilderCore.Start[TStateMachine](TStateMachine& stateMachine)
   at Microsoft.Extensions.Caching.Cosmos.CosmosCache.GetAsync(String key, CancellationToken token)
   at Microsoft.Extensions.Caching.Cosmos.CosmosCache.Get(String key)
   at Microsoft.AspNetCore.Session.DistributedSession.Load()

However: If i do put my breakpoint after the session.Set() or session.TryGetValue function call. The errors do not occur, and the session Id appears populated.
To Reproduce
Steps to reproduce the behavior:

Create a brandnew Blazor webassembly .net core project (with identity, etc)
Create a CosmosDb instance in azure and add the connection string to the project.
Add the CosmosDb extension Nuget package
Configure the pipeline as instructed in the documentation.
Add a few session calls in a controller and put breakpoints.

Expected behavior
Session Id should be populated, and debugging step by step should not raise these weird issues.
Additional context
I've tried this with a serverless CosmosDb, as well as with a Free Tier CosmosDb instance.

I cannot comment on webassembly debugging, or how it works with session state. Based on the documentation, the interaction with this caching library (or any other extension) happens only on the LoadAsync call (where it initializes) and on the Get/Set calls.
The exception you are sharing is not related to this library either, the error is a threadabort meaning that the runtime (asp.net core or webassembly) decided to kill the running thread. Most likely one or the other has a limit on the time a request can be processed and the fact that you are putting a breakpoint is holding it for longer than it is allowed.
Some pointers though:

The LoadAsync on the documentation says it should be called always, it does not mention anything about checking IsAvailable https://docs.microsoft.com/en-us/aspnet/core/fundamentals/app-state?view=aspnetcore-5.0
Use Async versions of Get/Set.
The Cosmos DB SDK uses TCP connections by default, I don't know how Webassembly supports TCP, you might want to switch to Gateway mode if Webassembly handles http better.

Hi @ealsur,
thanks for your response.
Maybe it’s a bit misleading, but I’m not connecting from the browser clientside of blazor. I’m in fact debugging the server side, which is a web api that comes scaffolded alongside the client.. so it’s not really interesting that I discovered this on blazor persé.
Anyway: I’m sure it’s too easy here to say the exception is unrelated to the cosmos library, as the exception is thrown while executing it. Plus, the normal session implementation does not have issues at all.
Thats my workaround of course: only using cosmos sessions in release builds.
thanks for the tip about ‘IsAvailable’ though: I read some blogs implementing this, but after some more searching, I agree with you this it should be called always.
And thanks for the blog: interesting read!

@martinlingstuyl Thanks for the clarifications!

Thats my workaround of course: only using cosmos sessions in release builds.

I am not sure I understand this, does this mean that on debug builds (no breakpoints) it does not work? Or that the issue is only happening when you put breakpoints?
Given that you mentioned that the Blazor backend is not on the browser (it's a normal Web API), then it might be limited by the Web API timeout to process the request (and generating the ThreadAbortException when the time is exceeded when placing the breakpoint). If you place the breakpoint after the networking interactions, can you confirm you do not get any errors?

Hi @ealsur
As soon as I put breakpoints before cosmos session interaction, the session breaks.
So if I put breakpoints after the first get or set, everything works as it should.
It doesn’t matter how much time you take for  stepping through the breakpoints.
Of course, I do not want to continuously take this behavior into account when debugging, so I add the regular ‘DistributedSessionCache’ on debug builds.

@martinlingstuyl Got it, it seems that threadabort only happens if the breakpoint is put before. Not sure there is much that can be done in this package though, as the threadabort is coming from the OS. There is no source code in this repo or on the V3 SDK repo (https://github.com/Azure/azure-cosmos-dotnet-v3/) that throws these kind of exceptions, but rather they happen whenever there is a system signal of cancelling an executing thread (async network operations are different than in-memory operations that are used with the memory based cache because they can execute in other threads).
If when using the regular DistributedSessionCache (that I assume is the memory based) and putting the breaking before the operation, waiting several seconds (similar amount as with the Cosmos based cache) and continuing does not yield the same error, I'm really not sure what could be causing it. Something is signaling the thread abort.
I tried reproing on a WebAssembly ASP.NET Core application, the base template on VS, adding Cosmos cache and adding breakpoints before the interactions, was able to debug without issues:

Not sure what kind of change can be done in this library for your case. If you can create a small repro project that we can use, we can try to identify something, but doesn't look like there is much.

Closing as cannot repro. Please re-open if further details arise.


How to allow tags only from suggestions or unique tags?

@mabbas007 Can you look at this? This is priority for me. Thanks

how can prevent enter key from keyboard ?
I just want to add tags from the suggestion list not manually entered


[5.5] use $attributes property directly instead of getAttributes() method

This closes #20902 and provides backwards compatibility for code that overrides the getAttributes() method on a model.

Eh, the change is correct. The package is just expecting the wrong output from that method.

Was hesitant about submitting this as the change on the framework side did indeed seem like a move in the right direction. Thanks @taylorotwell


added linting

I think the lint is getting executed now... Below image from running
./node_modules/.bin/eslint --cache --ext .ts,.tsx .

However, running yarn lint from the root looks like it also runs but then exits fails at the end.  I get the same error if I try to run yarn lint without installing dependencies

Failing on lint errors because there are many of them

I'm not sure why it is still throwing a lint error even though I specified that node is a DrupalNode


cursor position not correct when move

Issue Type
Bug
Description
steps:

a multiline doc,  with cursor at  line 1 and col 3(or any >1), like below

     1# ab|cdefg
     2# abcdefg

press key UP, the cursor whill go to col 1, like below

     1# |abcdefg
     2# abcdefg

now press key DOWN, the cursor went to line 2 and col 3, not col 1 as expected

     1# abcdefg
     2# ab|cdefg

VS Code Info
VS Code version: Code 1.20.1 (f88bbf9137d24d36d968ea6b2911786bfe103002, 2018-02-13T15:31:21.019Z)
OS version: Linux x64 4.10.0-43-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6200U CPU @ 2.30GHz (4 x 2700)

Load (avg)
0, 0, 1

Memory (System)
3.66GB (0.39GB free)

Process Argv
/usr/share/code/code --unity-launch

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

markdown-preview-enhanced
shd
0.3.2

Reproduces without extensions

@alexdima what is the expected behavior. It is not clear to me from the steps

Verification steps:

Have something like:

Press up twice. It goes to the first column, but it remembers the horizontal offset, such that when you press down, it returns to the column 5:

But if up is pressed 3 times, as in one more time, then the horizontal offset is reset:


Fix NPE in LuceneAnalyzer#end

When testing the analyzer with an integration test, I discovered an NPE from a part of the code that wasn't tested.
Here is the code:
https://github.com/datastax/cassandra/blob/9cabec504d9f70ddfb345122dc4defed56a40f85/src/java/org/apache/cassandra/index/sai/plan/StorageAttachedIndexSearcher.java#L91-L100
We build the Analyzer to use one of its methods, but not to analyze any text, hence the NPE.
Here is the NPE:
ERROR [Native-Transport-Requests-1] 2023-08-31 14:35:38,695 QueryMessage.java:121 - Unexpected error during query
java.lang.NullPointerException: null
	at org.apache.cassandra.index.sai.analyzer.LuceneAnalyzer.end(LuceneAnalyzer.java:110)
	at org.apache.cassandra.index.sai.plan.StorageAttachedIndexSearcher.filterReplicaFilteringProtection(StorageAttachedIndexSearcher.java:99)
	at org.apache.cassandra.service.reads.DataResolver.lambda$preCountFilterForReplicaFilteringProtection$6(DataResolver.java:300)
	at org.apache.cassandra.service.reads.DataResolver.resolveInternal(DataResolver.java:341)
	at org.apache.cassandra.service.reads.DataResolver.resolveWithReadRepair(DataResolver.java:244)
	at org.apache.cassandra.service.reads.DataResolver.resolveWithReplicaFilteringProtection(DataResolver.java:284)
	at org.apache.cassandra.service.reads.DataResolver.resolve(DataResolver.java:130)
	at org.apache.cassandra.service.reads.range.SingleRangeResponse.waitForResponse(SingleRangeResponse.java:59)
	at org.apache.cassandra.service.reads.range.SingleRangeResponse.computeNext(SingleRangeResponse.java:65)
	at org.apache.cassandra.service.reads.range.SingleRangeResponse.computeNext(SingleRangeResponse.java:31)
	at org.apache.cassandra.utils.AbstractIterator.hasNext(AbstractIterator.java:47)
	at org.apache.cassandra.db.transform.BasePartitions.hasNext(BasePartitions.java:93)
	at org.apache.cassandra.service.reads.range.RangeCommandIterator.computeNext(RangeCommandIterator.java:123)
	at org.apache.cassandra.service.reads.range.RangeCommandIterator.computeNext(RangeCommandIterator.java:45)
	at org.apache.cassandra.utils.AbstractIterator.hasNext(AbstractIterator.java:47)
	at org.apache.cassandra.db.transform.BasePartitions.hasNext(BasePartitions.java:93)
	at org.apache.cassandra.db.partitions.PartitionIterators$1.hasNext(PartitionIterators.java:154)
	at org.apache.cassandra.db.transform.BasePartitions.hasNext(BasePartitions.java:93)
	at org.apache.cassandra.cql3.statements.SelectStatement.process(SelectStatement.java:892)
	at org.apache.cassandra.cql3.statements.SelectStatement.processResults(SelectStatement.java:518)
	at org.apache.cassandra.cql3.statements.SelectStatement.execute(SelectStatement.java:495)
	at org.apache.cassandra.cql3.statements.SelectStatement.execute(SelectStatement.java:341)
	at org.apache.cassandra.cql3.statements.SelectStatement.execute(SelectStatement.java:102)
	at org.apache.cassandra.cql3.QueryProcessor.processStatement(QueryProcessor.java:290)
	at org.apache.cassandra.cql3.QueryProcessor.process(QueryProcessor.java:394)
	at org.apache.cassandra.transport.messages.QueryMessage.execute(QueryMessage.java:108)
	at org.apache.cassandra.transport.Message$Request.execute(Message.java:242)
	at org.apache.cassandra.transport.Dispatcher.processRequest(Dispatcher.java:111)
	at org.apache.cassandra.transport.Dispatcher.processRequest(Dispatcher.java:131)
	at org.apache.cassandra.transport.Dispatcher.lambda$dispatch$0(Dispatcher.java:78)
	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
	at org.apache.cassandra.concurrent.AbstractLocalAwareExecutorService$FutureTask.run(AbstractLocalAwareExecutorService.java:165)
	at org.apache.cassandra.concurrent.AbstractLocalAwareExecutorService$LocalSessionFutureTask.run(AbstractLocalAwareExecutorService.java:137)
	at org.apache.cassandra.concurrent.SEPWorker.run(SEPWorker.java:119)
	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
	at java.base/java.lang.Thread.run(Thread.java:829)

I am still new to query execution, but I am guessing the unit tests skip this filtering code. I added an integration test that fails without the fix.

@JeremiahDJordan - looks like our recent changes to the branch protections my have also introduced a requirement to pass all "required" github actions. Anecdotally, those 25 new test failures are not new to this PR but are new to the vsearch branch when compared to some other branch. Can we remove that requirement, and then take some time to fix tests tomorrow?

Yeah. I made a new rule set without the pr gate listed for vsearch branch. I thought it compared against the branch you were merging to like the CNDB action does. But I guess not.

Thanks, that makes sense.


Feature support for async-std

For my project I would really like to use this library together with async-std.
Are you planning to support it over an cargo feature?

It's not on our roadmap, the initial design decision was to base the driver on tokio. Still, contributions are welcome :)

This is the one thing that would prevent me from using Scylla. We use async-std, not Tokio. Async-std is more aligned with the standard library instead of reinventing everything itself. Could official support be added to the roadmap? It's not uncommon to have support for both in a single crate.

So this may or may not be helpful to anyone but I started on a branch to introduce other executors: https://github.com/scylladb/scylla-rust-driver/compare/main...cholcombe973:scylla-rust-driver:main. I ran into a problem though in that Glommio doesn't support Send and scylla is using Arc all over the crate. I think it would be too much work for me to rip all that up but it might be helpful to someone else if they want to support async-std. I haven't used async-std so I don't know if it supports Send.

I mean at this rate since it's not officially supported I'm not entirely sure I'll switch to Scylla for anything in the future, but I appreciate the work done!


A bit of formatting if/else

rt.reason avgdelta added toFixed
move console.log result
add if/else
add maxSafeBasal into reason

@LorelaiL I like the avgdelta tofixed and adding the maxsafebasal adjustment to reason. Thanks. Unfortunately the log/error changes would break everything. Use of oref0-determine-basal requires that the output be on stdout to go into the suggestedtemp.json. Putting anything else in there (what currently goes to error) or otherwise changing it will cause it to no longer be valid json, and break openaps. The else stuff I also removed because if you return early, that terminates the program, so by removing else clauses I was able to simplify my if/else hierarchy a lot.

Now that you have #36 opened to track the whole refactor, we should also make a new commit with just the substantive changes, such as https://github.com/openaps/oref0/pull/35/files#diff-40d4a549ff39a9078d8421ac054a8fdfR150, https://github.com/openaps/oref0/pull/35/files#diff-40d4a549ff39a9078d8421ac054a8fdfR196, https://github.com/openaps/oref0/pull/35/files#diff-40d4a549ff39a9078d8421ac054a8fdfR213, https://github.com/openaps/oref0/pull/35/files#diff-40d4a549ff39a9078d8421ac054a8fdfR261, https://github.com/openaps/oref0/pull/35/files#diff-40d4a549ff39a9078d8421ac054a8fdfR316, and any others I missed.  Those will be much easier to merge with minimal testing.


No Little Endian accessor for Double or Float

Expected behavior
getFloatLE or getDoubleLE are not available
Actual behavior
Steps to reproduce
Minimal yet complete reproducer code (or URL to code)
Netty version
JVM version (e.g. java -version)
OS version (e.g. uname -a)

It exists in the latest 4.1 release.

I have 4.1.17 and I must be blind because readDoubleLE doesn't exist ?
Dave Cramer
On 16 November 2017 at 12:04, Norman Maurer<EMAIL_ADDRESS>wrote:

It exists in the latest 4.1 release.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/netty/netty/issues/7413#issuecomment-344989880, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAYz9qSXL8kgA8b__U5mL1vHKqLPgyqxks5s3GsogaJpZM4Qg0JN
.

https://github.com/netty/netty/blob/4.1/buffer/src/main/java/io/netty/buffer/ByteBuf.java#L1552 ?

thx.
Apparently 4.1.7 is not the same as 4.1.17
Dave Cramer
On 16 November 2017 at 12:27, Norman Maurer<EMAIL_ADDRESS>wrote:

https://github.com/netty/netty/blob/4.1/buffer/src/
main/java/io/netty/buffer/ByteBuf.java#L1552 ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/netty/netty/issues/7413#issuecomment-344996874, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAYz9pBiuWs_kb224oxoLv3Qbko26Vajks5s3HCdgaJpZM4Qg0JN
.

@davecramer yep ;)


Adding stage to fix the MC error to initial values.  Needed for mod_chi2

Can confirm from testing that the stage works as intended, the errors stay indeed the same after changing parameters in a pipeline.

Great, once the few requested changes are in gonna merge the PR


Install gcc & g++ in weekly gpu ci

Signed-off-by: Jael Gu<EMAIL_ADDRESS>

@binbinlv CI error, please check test scripts:

@binbinlv CI error, please check test scripts:

Fixed，please retest it, thanks.

/lgtm


A bioclomate non-projected global grid for cubes occurrence (EPSG:4326)

Hello,
I have been exploring the current options for creating global occurrence cubes and have run into some problems. Currently, the  Quarter degree grid cell-QDGC  is available for this purpose but when searching for a global QDGC reference grid I could not find it. In the zenodo repository many national QGCD grids are available but not a global one. This could be explained, among other things, by geographical distortions in certain regions. Without a global reference grid it is not possible to "rasterise" the CSV data cubes... I don't know if I'm missing something.
What comes to my mind is the possibility of offering a non-projected global grid (e.g. EPSG 4326) widely used by the modelling community and connected to the climate community. I would like to suggest the 30-second grid (~1 km at the equator) used by both WorldClim and Chelsa. One of the challenges could be the cell coding system, but we could find a good way around this.

@MattBlissett - this is something that has been asked as high priority request in the project executive meeting.

I have generated global grids for Extended Quarter-Degree Grid Cells.  As this is likely to be useful to others, I'll tidy up the code and publish it to GitHub and Zenodo later.  These grids follow a similar format to the African QDGC grids already on Zenodo. (Levels 6 and 7 to follow, they will be very large — I think this is the only reason no-one has uploaded them before.)
Generating a download with an arbitrary/custom grid is already possible, so long as the grid can be derived using SQL expressions. Not tested, but I think this would provide a cube using a 30" grid (i.e. WorldClim):
SELECT
  FLOOR(decimalLatitude) AS degreesLatitude,
  FLOOR((ABS(decimalLatitude)*60)%60) AS minutesLatitude,
  FLOOR(FLOOR((ABS(decimalLatitude)*120.0)%120) / 60) * 30 AS secondsLatitude,
  FLOOR(decimalLongitude) AS degreesLongitude,
  FLOOR((ABS(decimalLongitude)*60)%60) AS minutesLongitude,
  FLOOR(FLOOR((ABS(decimalLongitude)*120.0)%120) / 60) * 30 AS secondsLongitude,
  COUNT(*) AS occurrences
FROM
  occurrence
GROUP BY
  degreesLatitude,
  minutesLatitude,
  secondsLatitude,
  degreesLongitude,
  minutesLongitude,
  secondsLongitude;

To implement it as a function for the SQL downloads we need a cell code.  A straightforward code could be W1795930S895930 to E1795930N895930, i.e. WdddmmmssSddmmss with the seconds being either 00 or 30.
An alternative scheme could use letters like EQDGC for the sub-degree divisions, but with WorldClim's grid sizes not being halvings/doublings of each other I think this is opaque for no benefit. The 10min grid would have identifiers like E012N56X, (X is in the range 0-9A-Z), the 5 minute E012N56XB (B in A-D), 2.5 minute E012N56XBC (C in A-D), E012N56XBCY (Y in A-Y).  The division of the degree is into six, then it's halved, halved, and divided into five.
Are the other WorldClim grid sizes of interest? 10 minutes, 5 minutes and 2.5minutes.


[test]: Add e2e fallback tests

What this PR does / why we need it:
Adds e2e fallback tests
Which issue this PR fixes:
fixes #206

I think for simplifying the main test file (txtdirect_test.go) is to move all the other tests to their own file and only keep redirect tests + the mock zonefile/zonemap inside the txtdirect_test.go
It doesn't make that much of change but at least we can move for example the record parsing code and its tests to another file to keep the main test file a little cleaner.
I still don't have any idea on how to make e2e tests and the zonemap more simple. Need more time to think about it. Also, think it would be good to create another issue for e2e tests refactor and work on it in another PR.


Target div

Hi,
Is it possible to pass a div id and ask term to draw its terminal inside it?

My bad
term.open(document.getElementById('terminal'));


Build and run tests using Bazel

Add all GoogleTest-based unit test to bazel.
Run tests for your piece of functionality (for example accessors) using this command:
bazel test //cpp/oneapi/dal/table:accessor_tests

Test targets are listed in corresponding BUILD files, for example, see cpp/oneapi/dal/table/BUILD

/intelci: run


Allow additional build args to be set with additional-build-args

This allows additional arguments to be passed to docker build such as the SSH socket or other things. The name is a bit unfortunate as we already have build-args which sets --build-arg arguments.

Add to plugin.yml?

I still haven't worked out when/if that gets used?


Unit in HHP threshold chart is wrong

As pointed out by a user, the unit on the Y-axis of the chart 'Hybrid heat pump threshold COP for space heating' should be ct/MJ instead of EUR/MJ.
Note that a screenshot of this chart is included in the documentation and needs to be changed there too https://docs.energytransitionmodel.com/main/heat-pumps/#threshold-cop
t
Assigning @marliekeverweij , reassign as you see fit.

@antw has fixed this issue 👍


Window remembers previous spacing value

Need to blank out variables on window close

should be corrected by implementation of #11


Middleclick in Ubuntu Unity Launcher does not open a new window

VSCode Version: 1.8.1
OS Version: Ubuntu 16.10

Steps to Reproduce:

Middle-click on the Unity Launcher Icon.
The load mouse indicator appears and disappears again. Then nothing happens.

This is breaking Ubuntus default behaviour of opening a new instance of the application upon a middle click. A left click brings the running app instance to the front or launches a new one (works) and a right click opens the context menu (works).
As a workaround I have to spawn a new window from the file menu each time I need a new instance, which feels really broken.

Duplicate https://github.com/Microsoft/vscode/issues/48

@Tyriar Are you familiar with the Unity desktop and sure this is a duplicate? The other issue is about calling vscode from the command line, this one is about invoking from the Unity launcher (the default desktop launcher in Ubuntu). From reading in the post, there should just be a link to define that "code -n" is being called upon a middle click, which is not the case yet.
I was able to solve it myself. I looked into the .desktop files and notieced that Ubuntu manages the instances itself, so by default every application should assume that the default command for the launcher icon is opening a new window. I have removed the "new-window" section and added the --new-window flag as the default command. This fixes the behaviour and now it behaves just like every other app. Also left-clicking the icon brings up the currently running instance to the front, just as expected.
Here is my working code.desktop:
[Desktop Entry]
Name=Visual Studio Code
Comment=Code Editing. Redefined.
GenericName=Text Editor
Exec=/usr/share/code/code --new-window %U
Icon=code
Type=Application
StartupNotify=true
StartupWMClass=Code
Categories=Utility;TextEditor;Development;IDE;
MimeType=text/plain;inode/directory;
Actions=new-window;
Keywords=vscode;

X-Desktop-File-Install-Version=0.23

Icon=code

Fixed it here: https://github.com/Microsoft/vscode/pull/17699

Yeah I suggested something similar in https://github.com/Microsoft/vscode/issues/48#issuecomment-268616502, let's go forward with this and then try get a similar fix for Windows or for the CLI generally pending the discussion in #48 :+1:


Improve platform inference in shell initialisation.

This is the first of multiple PRs to enable support for the new ARM-based candidate versions. This PR is dependent on some serverside work being done first.

I've merged this with CLI, but Github seems a bit confused. Closing this PR manually.


Allow capping spark partitions during Hoodie upsert stage

@vinothchandar @n3nash This is the change to allow capping spark partitions during Hoodie upsert stage.
The change has been tested internally in Uber.

@n3nash @vinothchandar addressed a few review comments and rebased with current master.

@jianxu Do we plan to make changes to handle multiple files in a single partition in MergeHandle soon ?

@n3nash This change is an immediate mitigation.  Long term wise we need to evaluate different options and prioritize accordingly.

@jianxu So should we punt this PR in the OSS for now ?

I'd suggest land this on the different branch, as I've already noted that this is a much needed mitigation.  @vinothchandar @n3nash

different branch in OSS? I think WIP PRs are a much better tracking mechanism if thats what we are looking at..


BUG: Cannot Save .sql/.ddl when generating DDL

I select generating the DDL
-> Select the Data Model correctly
-> Get into the Windows Dialog for Saving
-> Click Saving
-> No Error Message, but nothing saved.
Additional Information:

From within StarUML I don't have problems saving other files (from "Save" or "Export as")
I use the Evaluate Version

Its installed on Windows and I have all write permissions for the respective folders

I have debugged it to some degree. It apparently fails on an uncaught error (check screenshot)

Obviously a problem with the relationships. If it helps I can also send my ERD

Sorry for late response. Could you upload your .mdj file in order to reproduce the problem?


two-dimensional arrays of objects in typescript

C# code such as
[DataContract]
public class Foo {
  [DataMember] public int FooNum {get;set;}
}

[DataContract]
public class Bar {
  [DataMember] public Foo[][] Foos {get;set;}
}

The issue is when creating the init method for Bar, the generated typescript will look like
...
if (data["Foos"] && data["Foos"].constructor === Array) {
  this.foos = [];
  for (let item of data["Foos"])
    this.foos.push(item); // <------- problem here, not calling Foo.fromJS
}
...

For a one-dimensional array, the typescript correctly uses this.foos.push(Foo.fromJS(item)).  This isn't high priority for me since I was able to remove two-dimensional arrays and instead create an intermediate class.  I don't know how much you want to fix here, support many array dimensions?

Must be improved in
https://github.com/RSuter/NJsonSchema/blob/master/src/NJsonSchema.CodeGeneration.TypeScript/Templates/ConvertToClass.Liquid#L8
and
https://github.com/RSuter/NJsonSchema/blob/master/src/NJsonSchema.CodeGeneration.TypeScript/Templates/ConvertToJavaScript.liquid#L8
and maybe other places in these two files :-)