[Feature Request] Configurable time for mouse click event

Hello,
First of all - thanks for the great tool!
During usage of the software I've realized that the circles printed in Mouse Click Task could be slightly longer.
In my settings (20fps) only 2 frames contains the circles, and it seems isn't enough that users of my gifs could somehow see those clicks easily.
So, the idea is to add new configurable parameter (along with colors and size) about the time to have this circle in the gif.
Best regards!

I've realized that the time of the circle visibility depends on the recorder.
Apologize :)


[T6A1][T16-B1]Alwinson Au-yong ex2

Ready for Review

@alwinsonauyong
Some comments added. Please ack & close the PR after reading comments.


Request routing issue from browser to application service in istio 1.6.x to 1.7.x

The requests arent getting routed from browser to application service.  as the issue  #23851 got closed opening this one.
In issue #8807 its mentioned as below
@krancour thanks for clearing that up. I will note that this is still not 100% fixed, we recently realized with wildcard hostnames its not quite right. Support in envoy is needed, see envoyproxy/envoy#12647
even though the there is "*" in place of port when compared to port 80 in earlier versions.  the requests arent passed from browser and we keep getting connection refused error.
Below is the error from curl command

About to connect() to <IP_ADDRESS> port 30603 (#0)
Trying <IP_ADDRESS>...
Connection refused
Failed connect to <IP_ADDRESS>:30603; Connection refused
Closing connection 0
curl: (7) Failed connect to <IP_ADDRESS>:30603; Connection refused

if we edit the ingress-gateway svc and modify as below

name: http2
nodePort: 30603
port: 30603
protocol: TCP
targetPort: 30603

below is the curl output post modification

About to connect() to <IP_ADDRESS> port 30603 (#0)
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 30603 (#0)

Hi @nataraj24 ,w need more info about the config you have, and the failing request. its helpfuil to get a config dump as well: https://github.com/istio/istio/wiki/Troubleshooting-Istio
The current state:

1.7.3 and older: any wildcard (*.example.com") with a port in the host header will fail
1.7.4+: any wildcard with a port in the host header that does NOT match the service will fail
Ideal future, if envoy issue linked is fixed: everything works

Hi @howardjohn
please find attached config dump.  below dump is of 1.6.12 for the application pod
infra_dump.zip

this config only applies to gateway, the dump is from a sidecar.
Also need to know what request is sent that fails

Hi @howardjohn
Let me know which configuration dump you are looking for I will generate the same and share it with you.
If you send http request from browser or using curl command both fails.  It won’t reach application service.
If  you edit the ingress gateway svc port and give same value then it will go through

We need the config dump from the ingress gateway pod and the exact curl command that is failing

Hi @howardjohn
Below is the curl command
curl http://k8s.ad.test.com:31128/infra/ui/infra.jsp
Below is the config dump of the ingress gateway pod.
config_dump.zip

@nataraj24 everything looks fine here. Can you try curl http://k8s.ad.test.com:31128/infra/ui/infra.jsp -H "Host: k8s.ad.test.com" as well to be sure?
In the original issue you mentioned two different error cases, which do you hit?
Can you attach the Service? I suspect the request is not even getting to the pod

Hi @howardjohn
sorry i had given browser url.  Below is the exact curl url and its output
curl -v http://<IP_ADDRESS>:31128/infra/ui/infra.jsp

About to connect() to <IP_ADDRESS> port 31128 (#0)
Trying <IP_ADDRESS>...
Connection refused
Failed connect to <IP_ADDRESS>:31128; Connection refused
Closing connection 0
curl: (7) Failed connect to <IP_ADDRESS>:31128; Connection refused

Below is the  service definition
apiVersion: v1
kind: Service
metadata:
name: infra-svc
labels:
app: infra
spec:
type: "ClusterIP"
ports:

name: http
port: 8080
protocol: TCP
targetPort: 8080
selector:
app: infra
sessionAffinity: ClientIP

as mentioned earlier the call get routed properly after specifying port as below for ingress gateway svc
name: http2
nodePort: 31128
port: 31128
protocol: TCP
targetPort: 31128

When you are calling the nodePort address you shouldn't call the Service IP, you need to call the node IP. SEe https://istio.io/latest/docs/tasks/traffic-management/ingress/ingress-control/#determining-the-ingress-ip-and-ports for more information

Hi @howardjohn
I am using nodeip only in curl command Not using service ip

Hi @nataraj24 . It seems this issue is about issues with your Kubernetes setup or configuration. From Istio's side, everything is configured correctly but your request is not reaching the proxy, which is outside of Istio's ownership. https://kubernetes.io/docs/concepts/services-networking/service/#nodeport may help diagnose nodeport issues in Kubernetes. Thanks.


SVGJS 2.3.2 hangs in chrome while rendering offscreen

A

B

This code will make chrome browser unresponsive.
If we take version 2.3.1, everything will work.
If we place line 2 right after line 1, everything will work.

Simple answer: Dont render offscreen. Its not faster because svg.js has to work around this by placing the elements in the dom anyway to get its bbox (in this case y() will call bbox).
Actually 2.3.2 was released to fix exactly that bug. No idea why its there again. Have to look into that...

For some reason SVG.Text had its own clone() method which leaded to his error.
Fixed in 2.3.3


Fix Test for Update Application

test_update_application is failing in the pytest github action. Identify the issue and fix it.

Fixed in commit https://github.com/adi-kiran/career-flow/commit/25984657eed206d949557111aa3ea62a7273db13

Done


KeyError 'page_id' When Parsing Documents with Custom Endpoints on Mindee API

Summary:
My API project is facing a KeyError 'page_id' when using new custom endpoints with the Mindee API. This issue is not present with older endpoints, suggesting a potential problem with the newer custom endpoint configuration or the library's response handling.
Background:
Our application leverages Mindee's OCR capabilities to extract data from PDFs. While older endpoints operate as expected with library version 3.13, the integration with newly created custom endpoints leads to a KeyError, even after updating to version 4.0.1.
Issue Description:
Upon invoking client.parse with product.CustomV1 for new custom endpoints, a KeyError is thrown, indicating the absence of 'page_id' in the parsed results. This exception is traced back to the construction of ListFieldV1 within the Mindee library.
Logs:
Traceback (most recent call last):   File "C:\Users\PC\PycharmProjects\multiCotizadorAPI\utils\mindee\mindee.py", line 30, in analyze_pdf_with_mindee     result = mindee_client.parse(product.CustomV1, input_doc, endpoint=custom_endpoint)              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\PC\PycharmProjects\multiCotizadorAPI\venv\Lib\site-packages\mindee\client.py", line 114, in parse     return self._make_request(            ^^^^^^^^^^^^^^^^^^^   File "C:\Users\PC\PycharmProjects\multiCotizadorAPI\venv\Lib\site-packages\mindee\client.py", line 339, in _make_request     return PredictResponse(product_class, dict_response)            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\PC\PycharmProjects\multiCotizadorAPI\venv\Lib\site-packages\mindee\parsing\common\predict_response.py", line 28, in __init__     self.document = Document(prediction_type, raw_response["document"])                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\PC\PycharmProjects\multiCotizadorAPI\venv\Lib\site-packages\mindee\parsing\common\document.py", line 47, in __init__     self.inference = prediction_type(raw_response["inference"])                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\PC\PycharmProjects\multiCotizadorAPI\venv\Lib\site-packages\mindee\product\custom\custom_v1.py", line 27, in __init__     self.prediction = CustomV1Document(raw_prediction["prediction"])                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\PC\PycharmProjects\multiCotizadorAPI\venv\Lib\site-packages\mindee\product\custom\custom_v1_document.py", line 29, in __init__     self.fields[field_name] = ListFieldV1(field_contents)                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\PC\PycharmProjects\multiCotizadorAPI\venv\Lib\site-packages\mindee\parsing\custom\list.py", line 45, in __init__     self.page_id = raw_prediction["page_id"]                    ~~~~~~~~~~~~~~^^^^^^^^^^^ KeyError: 'page_id'
Troubleshooting Steps:

Confirmed operational status of older endpoints.
Encountered the error upon integrating new custom endpoints.
Library updated from 3.13 to 4.0.1, with no resolution.
Adherence to the updated Mindee documentation confirmed.
Codebase reviewed for discrepancies with custom endpoint usage.

Current Findings:

The issue is isolated to new custom endpoints.
The potential misalignment between the library's expectations and the custom endpoint's response.
The persistent problem post-library update suggests the issue might be beyond the client-side code.

-We request the Mindee team's input on this matter. Could you advise on any known issues with custom endpoint integration or suggest further steps we might take to debug this problem? Any assistance would be greatly appreciated to facilitate a swift resolution.
Thank you for your attention to this matter.

Fixed in #194


Loading multiple namespaces at the same time causes all but the last resolved to be cleared.

So, I have narrowed my problem down to LocaleProvider
On initialization, If the namespace changes, then only the last successfully loaded namespace locale will be saved.
Same issue if the locale changes once, and namespace changes multiple times.
componentWillReceiveProps calls translate.setLocale when namespace changes, but locale hasn't.

Serializing the load of these items would solve this issue. (although slow down the rendering of the page)

this is probably better solved in translate-maker

This was resolved.


Update DETELE endpoint

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
Cherry-pick changes from #734
Why
Known limitations
N/A

Reference Server Preview is available here:https://anchor-ref-pr735.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr735.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr735.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr735.previews.kube001.services.stellar-ops.com/


Cleanup client-side exceptions

Switches to using specific error classes for client errors instead of
raw Exception.
Also fixes an issue where errors during cluster startup were being conflated
with connect long polling timeouts, resulting in poor exceptions
client-side. A test has been added for this code path.

cc @ericdill, this should fix the json decode exception you were seeing while blocking on new_cluster.

I can confirm that this fixes the stack trace I was getting. Clusters are successfully starting and I can run simple dask code against it


Support running commands outside of the docker-compose context

There's been a couple of situations where I'd really, really, really would have loved being able to run a couple of quick commands before the actual docker-compose started running. I've ended up creating very simple homemade plugins just to run a single line of bash, and I think this could be easier. I'd like to explain them so that you can understand these use cases and consider this possibility for a future release of this wonderful plugin.
First situation: decrypting a file. We run some visual tests with percy.io, for which we need to have a PERCY_TOKEN available. We prefer not to have secret tokens like these committed in plain sight in our repos, even private ones. We use https://github.com/mozilla/sops in all our projects for this. Our agents include the sops binary and would be able to decrypt the files. All I'd need is to be able to run sops --decrypt secrets.env  or similar.
I could add this token to the agents themselves, but it's a unique token per project, so we'd need multiple ones. And we'd really prefer to have these tokens on the project repos. I could also somehow... decrypt the file from inside the container that needs the secret, but I'd need to add sops to that container, pass along the necessary things to be able to decrypt, etc.
Second situation: this is a bit convoluted, but I'd be interested in having available the modulo of the current build number. Something like echo "NETWORK_ID=$(($BUILDKITE_BUILD_NUMBER % 128))" >> .env. This would let me interpolate that $NETWORK_ID in the docker-compose.yml.
I'm guessing this could also be achieved by generating the pipeline dynamically.
As it is now, I can't easily run multiple steps in the same agent, and the only way to run anything outside of docker-compose is via plugins.
I hope you find this all reasonable. Would you accept PR's for this?

I'm going to close this as it hasn't gotten any activity in two years. We're achieving this using https://github.com/improbable-eng/metahook-buildkite-plugin


Update to latest binutils-mips-ps2-decompals

this needs to be 0.3 now right?

Yes

ready to merge


Use ToDetailString for arguments error message

Previously many types weren't represented when conversion to a string failed.
This pull request changes the error message to use the ToDetailString string from the v8 value.
Previous messages for booleans, numbers, and Promises, etc.:

Error processing argument at index %d, conversion failure from

New messages for booleans, numbers, and Promises, etc.:

Error processing argument at index 0, conversion failure from #<Promise>
Error processing argument at index 0, conversion failure from 100
Error processing argument at index 0, conversion failure from false

👍


Update PPM to HHG converter to create a domestic short haul shipment

Description
Currently, the PPM to HHG converter only adds Domestic Linehaul shipment. As the A-team moves through pricing service items we will need other types of shipments. This PR modifies the converter to create a linehaul and shorthaul shipment when converting from PPM to HHG. The Shorthaul shipment has it's origin and destination hard-coded for now to select two locations that are less than 50 miles apart.
Reviewer Notes
This is one part of making it so we can price Domestic short haul shipments in staging. See MB-3157 for the other fix before DSH pricing can be checked in staging.
Setup
Prep the db with the dev e2e data and start both the server and the client.
make db_dev_e2e_populate
make server_run
make client_run

Create new Service member

go to http://milmovelocal:3000/
Create a new milmove user and Sign in the local sign in feature
Create a new PPM move request

Use the fake names spreadsheet for names and addresses
Go through the whole thing and sign the form to submit the move

Submit the move

Sign in as TOO role

Sign into http://officelocal:3000/ as<EMAIL_ADDRESS>(PPM office) via the local sign in feature
See that the move is listed in the queue
Click Customer Details Page Skeleton link
See that an HHG_LONGHAUL_DOMESTIC and HHG_SHORTHAUL_DOMESTIC shipments are added
Approve each and see that the appropriate service items are added for each. (See this spreadsheet to confirm what those service items should be)

NOTE MTO Service items are tied to a shipment but that is not reflected in the skeleton page view.
Code Review Verification Steps

[ ] Code follows the guidelines for Logging
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

MB-3155 for this change
this slack thread explains more about the issue.

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against d6aed80645336503087568aa4383a21d8eca8c1c

@gidjin Do all moves receive Counseling and Shipment Management services? I see that those two services aren't in the linked spreadsheet (https://docs.google.com/spreadsheets/d/1zeS5J2LVd_cyB2TW2rQPu1hHqKSXT-UY7RpUzfvTi0k/edit#gid=0).
Other than that, all of the HHG Long and Shorthaul service items are on the Customer Details page that I see.

@gidjin Do all moves receive Counseling and Shipment Management services? I see that those two services aren't in the linked spreadsheet (https://docs.google.com/spreadsheets/d/1zeS5J2LVd_cyB2TW2rQPu1hHqKSXT-UY7RpUzfvTi0k/edit#gid=0) but they are on the details page.
Other than that, all of the HHG Long and Shorthaul service items are on the Customer Details page that I see.

Yes, Counseling and Shipment Management services are special. They apply to the MTO and are not shipment specific like the other service items. However, they are services that the prime will get paid for so they are listed automatically when the MTO is created.

@garrettqmartin8 I saw that error too, it happens without my changes so I also assumed it to be unrelated.


updated xquartz (2.7.9)

Changes to a cask
Editing an existing cask

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Merged as b69a1422a73cd3886dd143915da5fdd46f223489.


Html bug into section 'Fonts in Xamarin.Forms'

Issue moved from MicrosoftDocs/feedback#2816

Please respond to @Nihilem.

From @Nihilem on Wednesday, June 3, 2020 11:47:18 AM
All of the page is clickable, there are some problems with  tags in  headlines text.
to Reproduce
Steps to reproduce the behaviour:
Go to  https://docs.microsoft.com/it-it/xamarin/xamarin-forms/user-interface/text/fonts
Desktop (please complete the following information):
OS: Windows 10
Browser Chrome
Version 83.0.4103.61

Issue moved from MicrosoftDocs/feedback#2816

Please respond to @welcome[bot].

From @welcome[bot] on Wednesday, June 3, 2020 11:47:20 AM
Thank you for creating the issue! One of our team members will get back to you shortly with additional information. If this is a product issue, please close this and contact the particular product's support instead (see https://support.microsoft.com/allproducts for the list of support websites).

Hi @Nihilem
Thanks for opening the issue. I fixed it early last week, so if you look at https://docs.microsoft.com/it-it/xamarin/xamarin-forms/user-interface/text/fonts you'll see that the page now renders correctly.


Streaming mode does sent "Accept" header "application/json"

Issue
When Aider does sent a chat-completion request, it sends the Accept header with the value application/json, no matter if stream is set to false or true. It should actually be text/event-stream for stream=true.
Version and model info
Aider: 0.62.1
Model: any
Prompt/Request: any

Thanks for trying aider and filing this issue.
Sorry, what makes you concerned about this? Are you experiencing some specific problem?

I am currently trying Aider with an custom LLM proxy, which does not support sending a stream with Content-Type: application/json. When using streaming, it requires/sends Content-Type: text/event-steam.

This sounds like something you may need to work out with your custom proxy.
Aider uses litellm to connect to almost any LLM api. You can also check their docs.

Can you verify that LiteLLM is causing the error or does Aider manipulate the request in any way?

I'm labeling this issue as stale because it has been open for 2 weeks with no activity. If there are no additional comments, I will close it in 7 days.Note: A bot script made these updates to the issue.


synchronize ecs-logging spec

What
ECS logging specs automatic sync
Why
Changeset

https://github.com/elastic/ecs-logging/commit/d3ffcf1 Adding service.environment to the spec (https://github.com/elastic/ecs-logging/pull/62)

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-05-16T12:01:29.969+0000

Duration: 15 min 55 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
80

Skipped
0

Total
80

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)


servustv is broken for some urls

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2020.09.20
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

[debug] System config: []
[debug] User config: ['-f', 'bestvideo[height<=?1080][fps<=?30]+bestaudio/best', '--no-mtime']
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.servustv.com/videos/aa-24h32f1411w12/']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.09.20
[debug] Python version 3.7.8 (CPython)
[debug] exe versions: ffmpeg 4.3.1, ffprobe 4.3.1, rtmpdump 2.4
[debug] Proxy map: {}
[Servus] AA-24H32F1411W12: Downloading webpage
[Servus] AA-24H32F1411W12: Downloading m3u8 information
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/lib/python3.7/site-packages/youtube_dl/YoutubeDL.py", line 797, in extract_info
    ie_result = ie.extract(url)
  File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/common.py", line 532, in extract
    ie_result = self._real_extract(url)
  File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/servus.py", line 61, in _real_extract
    self._sort_formats(formats)
  File "/usr/lib/python3.7/site-packages/youtube_dl/extractor/common.py", line 1359, in _sort_formats
    raise ExtractorError('No video formats found')
youtube_dl.utils.ExtractorError: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

Most urls still work fine, but https://www.servustv.com/videos/aa-24h32f1411w12/ is broken

seems the video is located at:
https://cs.liiift.io/v1/STV/pd/37/13/B8/30/FO-25GUA77KDBH11/master.m3u8

same problem with
https://www.servustv.com/videos/aa-2575dep351w12

Same problem here:
https://www.servustv.com/videos/aa-1yqgz6v452112/
https://www.servustv.com/videos/aa-24dsn9qvn1w12/
Works clueless with jDownloader2 but youtube-dl_v2020.09.20 reports:

ERROR: No video formats found; please report this issue on https://yt-dl.org/bug

Going back to older versions of youtube-dl didn't solve the issue :(
@puchu

seems the video is located at:
https://cs.liiift.io/v1/STV/pd/37/13/B8/30/FO-25GUA77KDBH11/master.m3u8

How did you find that m3u8-link without "youtube-dl -g"?


AttnBlock3D 'bug' fix

What does this PR do?

Fixes # (issue)
Adds # (feature)
Before submitting

[x] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you read the contributor guideline?
[ ] Did you make sure to update the documentation with your changes? E.g. record bug fixes or new features in What's New. Here are the
documentation guidelines
[x] Did you build and run the code without any errors?
[ ] Did you report the running environment (NPU type/MS version) and performance in the doc? (better record it for data loading, model inference, or training tasks)
[x] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.
@xxx

这样改相当于重新引入原v1.0的bug (时间轴T未正确permute)，we should align to the right thing instead of the format.
关于这个bug，原仓已讨论且已合入  https://github.com/PKU-YuanGroup/Open-Sora-Plan/issues/171
且1）即使用加载原权重，bug fix后，重建视频正确。2）如果v1.0用新数据重训，bug fix后，效果会更好。


Unexpected spacing with Huggingface wav2vec library

Hi, when using huggingface and FB wav2vec, I'm getting some missing spaces when using various language models which I have created - including a simple LM with just a few phrases. Please can you assist with what is wrong?
from transformers import Wav2Vec2Processor, Wav2Vec2ForCTC, Wav2Vec2FeatureExtractor
# from datasets import load_dataset
import soundfile as sf
import torch

processor = Wav2Vec2Processor.from_pretrained("facebook/wav2vec2-large-robust-ft-swbd-300h")
model = Wav2Vec2ForCTC.from_pretrained("facebook/wav2vec2-large-robust-ft-swbd-300h")

# !wget https://dldata-public.s3.us-east-2.amazonaws.com/1919-142785-0028.wav

filename = "1919-142785-0028.wav"

audio, sampling_rate = sf.read(filename)

input_values = processor(audio, return_tensors="pt", padding="longest", sampling_rate=sampling_rate).input_values  # Batch size 1

# retrieve logits
logits2 = model(input_values).logits.cpu().detach().numpy()[0]

from pyctcdecode import Alphabet, BeamSearchDecoderCTC, LanguageModel

kenlm_model = kenlm.Model('/root/DeepSpeech/data/lm/lm.binary')
lm=LanguageModel(kenlm_model,alpha =0.169, beta = 0.055)
# make alphabet
vocab_list = list(processor.tokenizer.get_vocab().keys())
# convert ctc blank character representation
vocab_list[0] = ""
# replace special characters
vocab_list[1] = "⁇"
vocab_list[2] = "⁇"
vocab_list[3] = "⁇"
# convert space character representation
vocab_list[4] = " "
# specify ctc blank char index, since conventionally it is the last entry of the logit matrix
alphabet = Alphabet.build_alphabet(vocab_list, ctc_token_idx=0)

hotwords = [ "ringing", "up"]

# build the decoder and decode the logits
decoder = BeamSearchDecoderCTC(alphabet,lm)
decoder.decode(logits2)

result:
No LM:
["OH HELLO IT'S PAKER ID HER I'M BRINGING UP THIS PAPER OL ODAY HELLO MY SISTER'S BEEN BUSY"]
With LM:
"OH HELLO IT'S PAKERIDHER I'MSBRINGINGUPTHIS PAPERAALTODAY HELLOMY SISTER'SBEEN BUSY"
No LM:
"WANTED CHIEF JUSTICE OF THE MASSACHUSETTS SUPREME COURT IN APRIL THE S J C 'S CURRENT LEADER EDWARD HENNISE REACHES THE MANDATORY RETIREMENT AGE OF SEVENTY AND THE SUCCESSOR IS EXP"
With LM:
"WANTED CHIEF JUSTICE OF THE MASSACHUSETTS SUPREME COURT IN APRIL THE S C'S CURRENT LEADER "EDWARDHENNISE" REACHES THE MANDATORY RETIREMENT AGE OF SEVENTY AND THE SUCCESSOR IS EXP"
No LM:
'BOIL THEM BEFORE THEYARE PUT INTO THE SOUP OR OTHER DISH THEY MAYBE INTENDED FOR'
With LM:
'BOIL THEM BEFORE THEY ARE PUT INTO THE SOUP OR OTHER DISH THEY MAY BE INTENDED FOR'

hey, two quick thoughts on things to try:

case information: looks like the vocabulary is all uppercase. is the language model also all upper case or maybe lower case?
unigrams are missing: can you explicitly pass all known unigrams into the language model? that way the decoder can build a trie under the hood to efficiently decode partial words

Ahh great suggestions, thanks! Will revert back

closing for now, unless issues are persisting

Thanks your suggestions about case helped, much appreciated. Not tried the second option.


Conditional SvgXml rendering causes overlapped Svgs.

Bug
I try to render SvgXml conditionally with switch-case like under code.
switch(type){
  case('broken_egg') return <SvgXml xml={broken_egg} width={24} height={24} />
  case('confetti_egg') return <SvgXml xml={confetti_egg} width={24} height={24} />
}

But, when type is confetti_egg, broken_egg is not disappear and it comes in front of confetti_egg like behind image.

This is confetti_egg (ignore the number inside Svg).

Can you give me some advice or way to fix it?

this issue is not related react-native-svg.
the reason about this issue is from LayoutEffectAnimation.
so i close it


Add mm-sustain load test

Changes

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #324

PR #314

PR #310

PR #311

PR #312

PR #313

PR #315

PR #316


Numeric Slider Support

models.py updated the attributes for optional editor object which can include future visualization options for attributes
Added in the color option for the attribute timeline/filtering tools
Added selection between input box/slider for numeric attributes

https://user-images.githubusercontent.com/61746913/184219479-56bc1e5d-d2ad-4821-9b73-8820a0cf98d3.mp4

When changing a combobox with an existing value to a slider, the value is lost. Is that behavior we want or would we want the slider positioned at that previous value?

I think that's okay because I can't guarantee that the new range contains the old value.  We auto set a range when they swap so that is why the value changes.  Ideally this should be set up in the configuration file once and then values are undefined/0 until the user sets them on a specific track/detection attribute.


Request: control over cipher suites

We had a small discussion about the ciphers that Rclone supports. https://forum.rclone.org/t/webdav-preferred-cipher-suite-list/5002
It would be nice to be able to customise the cipher list, to:

disable ciphers that are considered weak (like the ones with 3DES, or even all ciphers without Forward Secrecy),
control the cipher order so you can steer servers that don't have their own preference into selecting a strong one.

Cheers,
Onno

I just checked again with this version:
M0415012:~ onno$ Downloads/rclone-v1.43-030-g37b2754f-beta-osx-amd64/rclone --version
rclone v1.43-030-g37b2754f-beta
- os/arch: darwin/amd64
- go version: go1.11

Wireshark lists this "client hello" cipher list:
Cipher Suites (16 suites)
Cipher Suite: TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256 (0xc02f)
Cipher Suite: TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (0xc030)
Cipher Suite: TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256 (0xc02b)
Cipher Suite: TLS_ECDHE_ECDSA_WITH_AES_256_GCM_SHA384 (0xc02c)
Cipher Suite: TLS_ECDHE_RSA_WITH_CHACHA20_POLY1305_SHA256 (0xcca8)
Cipher Suite: TLS_ECDHE_ECDSA_WITH_CHACHA20_POLY1305_SHA256 (0xcca9)
Cipher Suite: TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA (0xc013)
Cipher Suite: TLS_ECDHE_ECDSA_WITH_AES_128_CBC_SHA (0xc009)
Cipher Suite: TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA (0xc014)
Cipher Suite: TLS_ECDHE_ECDSA_WITH_AES_256_CBC_SHA (0xc00a)
Cipher Suite: TLS_RSA_WITH_AES_128_GCM_SHA256 (0x009c)
Cipher Suite: TLS_RSA_WITH_AES_256_GCM_SHA384 (0x009d)
Cipher Suite: TLS_RSA_WITH_AES_128_CBC_SHA (0x002f)
Cipher Suite: TLS_RSA_WITH_AES_256_CBC_SHA (0x0035)
Cipher Suite: TLS_ECDHE_RSA_WITH_3DES_EDE_CBC_SHA (0xc012)
Cipher Suite: TLS_RSA_WITH_3DES_EDE_CBC_SHA (0x000a)

For clarity these are the http client ciphers, rather than the http server ciphers.
Yes it would be possible to configure these reasonably easily.
Would you like to work on that?


GH-38626: [Python] Fix segfault when PyArrow is imported at shutdown

Rationale for this change
Some C++ destructors may be called after the Python interpreter has ceased to exist.
If such a destructor tries to call back in the Python interpreter, for example by calling Py_DECREF, we get a crash.
What changes are included in this PR?
Protect OwnedRef and OwneRefNoGIL destructors against decref'ing a Python object after Python finalization.
Are these changes tested?
Yes.
Are there any user-facing changes?
No.

Closes: #38626

@github-actions crossbow submit -g python -g wheel


Request: add bountysource.com to the filter

Evidence: https://app.bountysource.com/teams/ghacksuserjs/issues?tracker_ids=55128897
Original: https://app.bountysource.com

you do realize that both are the same domain?

Thanks a lot for your contribution.
I'm not sure this website is a useless copycat of GitHub. It seems it allows to have bounties for resolving issues on open-source projects. It sounds interesting, but I don't know how "real" it is.

afaik bountysource is a legitimate service, at least they've been there for a good number of years and i've seen a few projects using them before.

Thanks for the information @Nothing4You. I'm closing the issue: bountysource seems legitimate to me.


Update package nutanix tests

Issue #, if available:
Description of changes: Splitting package nutanix tests by K8 versions since the current infrastructure is not designed to handle multiple test cases together.
Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/retest

/approve

/cherrypick release-0.16

/cherry-pick release-0.16


[SPARKTA-352] Refactor rpm/deb generation

Made changes according https://stratio.atlassian.net/wiki/display/PRODUCT/Linux+packaging
Fixed log bug. It seems we exclude log4j library and Sparkta was looking for logback.xml

+1

:shipit:

Comment accepted. Will treat it ASAP

Squashing and checks in progress.


Gutter icons in VS Code are displayed in the wrong line

Please provide the environment you discovered this bug in.
Clone minimal reproducible example:
https://github.com/kevinkasper/vitest-vscode-gutter-icons-mre

yarn install
Open src/app/app.component.spec.ts file
Run all tests within the Test Explorer

Which area/package is the issue in?
vite-plugin-angular
Description
The gutter icons in VS Code are displayed in the wrong line if the @analogjs/vite-plugin-angular plugin is used.
Below is an example with activated plugin in the vite.config.ts and empty lines in the spec file:

Below is an example without activated plugin in the vite.config.ts and empty lines in the spec file:

In the second screenshot, the tests fail, but the gutter icons are in the right place.

This problem has already been reported in the vitest-dev/vscode repository. In the following issue it was confirmed that it is a problem with @analogjs/vite-plugin-angular:
https://github.com/vitest-dev/vscode/issues/400
Many thanks in advance!
Please provide the exception or error you saw
No response
Other information
No response
I would be willing to submit a PR to fix this issue

[ ] Yes
[X] No

Same issue here, which prevents us from evaluating Vitest. Would be great to get that resolved.

Here too. The gutter icons move all over the place when adding comments to tests.

@brandonroberts has this fix made it to one of the releases?

Any news on this? It really sends both our breakpoints and our gutter icons all over the place. Makes it nearly impossible to use the vitest extension in vscode nor debugging tests on the IDE.

Still looking into this one. The sourcemaps are fine when served in the browser, so trying to see what the difference is when using Vitest

@brandonroberts thank you very much! Forgive my ignorance, but what exactly does the vite-plugin-angular do?

@rafagsiqueira no problem. The plugin does the Angular compilation step so your external styles,  templates, and constructor DI works during testing.
If you use all inline styles and templates, and the inject function for DI, then you don't need the plugin. I have found this to be rare though.

Sourcemaps fix for VSCode/IDEs has landed in the 1.9.0-beta.12 release.  I've done some testing but please validate the fix on your end also


Fix: Remove redundant LICENSE/README names from files

npm already includes all of the files (even the ones for Closure Compiler and esprima). On the other hand, release-ops will no longer normalize line endings for the LICENSE files before publishing, but I assume those files' line endings are not as critical.
This should allow the project to be released, in theory.

Coverage remained the same at 99.249% when pulling a211977ab26f66230e847e2c386535dedd2aeaf4 on fix-package-json into aaa37294df8b6d85c39acdae12f2a4a974d50459 on master.

I tested this by running npm pack on the package. The license and readme files were still present in the tarball.

unrelated: it's weird that we didn't release 2.0.1, but a tag was there. need to delete?

Release consists of two parts, github release and npm release. Unfortunately the error we ran into was after github release was already done, but npm release hasn't started yet. At this point we need to manually push the release to NPM.


Fix read env execution

Explain your changes
Fix read env execution
Checklist

[x] I have read the “Pull requests” section in the contributing guidelines.
[x] I agree to the terms within the code of conduct.

🛟 If you need help, consider asking for advice over in the Kinde community.

Trung has tried to setup on the client app and it works fine now, you can view the recording here: https://app.screencast.com/ZD447ffKZHEr3?tab=Details&conversation=yoVlCpHcmFEi2RfWdvTV4O
Trung has also tested with an APK, it runs with no issue, please see recording here:
https://app.screencast.com/JtswJLP56morS?tab=Details&conversation=WFU1k2DU9yUSa4xX0YOaGn


DNN.Media module

Doing searches for the module takes you do the codeplex website; http://dnnmedia.codeplex.com
Since this is going away, it links here.  But I haven't seen anything here which allows me to download a release of the latest version.
Is this module pretty much dead?

I don't think this module is dead, but a release of it does appear to be missing.  Perhaps @hismightiness can help?

It's not dead.  I just moved it to its own repo in the DNN Community org.  Please feel free to post any issues you wish there.
https://github.com/DNNCommunity/DNN.Media


duplicate labels

distinct-values for labels with same label-type and content (merge/ legacy-name)
when date exists: do not delete

example Wiesbaden:
https://pmb.acdh.oeaw.ac.at/apis/entities/entity/place/883/detail


deleted apps should have an icon that shows they are deleated

when apps are deleted they will show up with a / and will be unclickable under listing management all listings

Is this still an issue?

Passed on Latest.  Icons in Listing Management display properly.  Deleted Apps are removed from Center Homepage.

Passed on Staging.


add interface for INetworkServer

Code is not compiling

Code is not compiling

Just the 'normal' random test failure :/. Its passing (for) now.


Added -r (regex) and -i (inverse regex) params for filtering repos

This PR addresses issue #290

Well, that was an excellent contribution :) Thanks!

ImportError: cannot import name 'gen' from 'tornado' (/Users/runner/Library/Caches/pypoetry/virtualenvs/tsrc-B3Sg_Vcl-py3.8/lib/python3.8/site-packages/tornado/init.py)

Don't worry about that, installing deps on macOS is flaky lately

all good, happy to contribute!


Follow-up for [Extensibility Request] Reminder Report

Hello,
Can you please let us know when will this request https://github.com/microsoft/ALAppExtensions/issues/16639 be implemented?
Thank you.

It is implemented in version 21.


Error when running meshing.workflow.InitializeWorkflow(WorkflowType="Watertight Geometry")

🔍 Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

🐞 Description of the bug
'Session' object has no attribute 'workflow'
📝 Steps to reproduce
I have tried to run and i am getting the issue
💻 Which operating system are you using?
Windows
🐍 Which Python version are you using?
3.10
📦 Installed packages
pip install pyfluent

@pavankumarkonchada , Could you please provide the steps to reproduce?

import ansys.fluent.core as pyfluent
from ansys.fluent.core import examples
import_filename = examples.download_file("mixing_elbow.pmdb", "pyfluent/mixing_elbow")
meshing = pyfluent.launch_fluent(precision="double", processor_count=10, meshing_mode=True,show_gui=True)
meshing.workflow.InitializeWorkflow(WorkflowType="Watertight Geometry")
when I am trying to run the above commands it is throwing Session object has no attribute worflow
AttributeError                            Traceback (most recent call last)
Input In [2], in <cell line: 1>()
----> 1 meshing.workflow.InitializeWorkflow(WorkflowType="Watertight Geometry")
AttributeError: 'Session' object has no attribute 'workflow'

@pavankumarkonchada, Actually we have a new launcher preference in place now. So, either use any one of the following:

Session = pyfluent.launch_fluent(precision="double", processor_count=10, meshing_mode=True, show_gui=True)
Session.meshing.workflow.InitializeWorkflow(WorkflowType="Watertight Geometry")

or, 2. meshing = pyfluent.launch_fluent(precision="double", processor_count=10, mode="meshing", show_gui=True)
meshing.workflow.InitializeWorkflow(WorkflowType="Watertight Geometry")
Both the above techniques will work. However, second approach is preferred as the first approach has been deprecated.
Thank you

@pavankumarkonchada, Actually we have a new launcher preference in place now. So, either use any one of the following:

Session = pyfluent.launch_fluent(precision="double", processor_count=10, meshing_mode=True, show_gui=True)
Session.meshing.workflow.InitializeWorkflow(WorkflowType="Watertight Geometry")

or, 2. meshing = pyfluent.launch_fluent(precision="double", processor_count=10, mode="meshing", show_gui=True) meshing.workflow.InitializeWorkflow(WorkflowType="Watertight Geometry")
Both the above techniques will work. However, second approach is preferred as the first approach has been deprecated. Please let us know if the issue still persist. Thank you

@prmukherj that's correct. In the example from @pavankumarkonchada, it would have been necessary to call meshing.meshing.workflow.InitializeWorkflow(WorkflowType="Watertight Geometry")

Using the second method I am getting this error
TypeError                                 Traceback (most recent call last)
Input In [4], in <cell line: 3>()
1 import ansys.fluent.core as pyfluent
2 from ansys.fluent.core import examples
----> 3 meshing = pyfluent.launch_fluent(precision="double", processor_count=10, mode="meshing", show_gui=True)
4 meshing.workflow.InitializeWorkflow(WorkflowType="Watertight Geometry")
TypeError: launch_fluent() got an unexpected keyword argument 'mode'

@pavankumarkonchada, do you have the latest version of the code? Could you please confirm that? Thank you

I am following as mentioned in the below blog
https://www.ansys.com/en-in/blog/open-source-access-to-fluent-with-pyfluent
And after that when I have opened in launcher.py I see the arguments passed to launcher definition is as below
def launch_fluent(
version: str = None,
precision: str = None,
processor_count: int = None,
journal_filename: str = None,
meshing_mode: bool = None,
start_timeout: int = 3000,
additional_arguments: str = "",
env: Dict[str, Any] = None,
start_instance: bool = None,
ip: str = None,
port: int = None,
cleanup_on_exit: bool = True,
start_transcript: bool = True,
show_gui: bool = None,
) -> Session:

I am following as mentioned in the below blog https://www.ansys.com/en-in/blog/open-source-access-to-fluent-with-pyfluent And after that when I have opened in launcher.py I see the arguments passed to launcher definition is as below def launch_fluent( version: str = None, precision: str = None, processor_count: int = None, journal_filename: str = None, meshing_mode: bool = None, start_timeout: int = 3000, additional_arguments: str = "", env: Dict[str, Any] = None, start_instance: bool = None, ip: str = None, port: int = None, cleanup_on_exit: bool = True, start_transcript: bool = True, show_gui: bool = None, ) -> Session:

The referenced blog doesn't use meshing.


add test templates step to push ci

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 28.186%

Totals

Change from base Build<PHONE_NUMBER>0:
0.0%

Covered Lines:
1307

Relevant Lines:
4637

💛  - Coveralls


Fix #936

Fixes #936. I've also added some new tests, a couple of which fail without the change.
The fix breaks binary compatibility. It would be possible to fix the issue in a way that doesn't, but I noticed that master already contains a number of changes that aren't compatible with 2.3.3.

LGTM. Many thanks! Merging ...


x/net/http2: a canceled request context can deadlock other requests on the same connection

What version of Go are you using (go version)?

$ go version
go version go1.17.6 darwin/amd64

Does this issue reproduce with the latest release?
Yes, this issue reproduces with golang.org/x/net v0.0.0-20220425223048-2871e0cb64e4 (see reproduction below)
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE="auto"
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/milesbryant/Library/Caches/go-build"
GOENV="/Users/milesbryant/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/milesbryant/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/milesbryant"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
<EMAIL_ADDRESS>GOSUMDB="sum.golang.org"
GOTMPDIR=""
<EMAIL_ADDRESS>GOVCS=""
GOVERSION="go1.17.6"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"

What did you do?
We enabled HTTP/2 for an internal proxy written in Go, and noticed that after 10 minutes, every connection into the proxy seemed to be hanging. A goroutine dump revealed that many goroutines were blocked on acquiring the ClientConn mutex (e.g. see https://github.com/golang/net/blob/2871e0cb64e47e47ba7466fad6e11562caf99563/http2/transport.go#L790).
One goroutine, however, had acquired the mutex and was blocked in abortStreamLocked trying to close the request body: https://github.com/golang/net/blob/2871e0cb64e47e47ba7466fad6e11562caf99563/http2/transport.go#L379. In our case this seemed to be indefinitely blocked trying to read from a network socket. We haven't figured out why yet, but that's a separate issue.
I've knocked up a very crude and simple reproduction here - I'll try and come back and tidy this up a bit later: https://github.com/milesbxf/go-http2-request-close-deadlock/blob/main/deadlock_test.go
For this to occur:

One request/stream has its context cancelled, causing the abortStream
The same request/stream also blocks on requestBody.Close
Other requests on the same connection try to acquire the mutex

Is this actually expected/intended behaviour? I.e. do we require that closing the request body should always be non-blocking? If so, I guess we could mitigate by increasing the concurrent connection count, but it feels like we could still hit the deadlock across multiple connections
What did you expect to see?
Other requests be able to make progress whilst the first request is blocked
What did you see instead?
All requests on the same client connection were blocked.
Other things
I'm very happy to attempt a fix, but I'm a bit of a novice in this part of the codebase!

@neild What is the current status here?  This issue is currently in the 1.19 milestone.  Should it move to 1.20?  To Backlog?  Thanks.

We currently assume that Request.Body.Close will complete reasonably quickly. As demonstrated here, there are circumstances where we call Close with connection or stream mutexes held, which can cause general badness if Close hangs.
We should probably avoid holding locks while calling Request.Body.Close. I think it's okay if a slow Close blocks some operations (for example, ClientConn.Close will block until all in-flight request bodies are closed, and that's probably okay), but one slow request body shouldn't interfere with other requests on a connection.

/assign

do you know if the fix will be backported to Go 1.19 in a point release?

You can use http2.ConfigureTransports to configure a Transport to use the golang.org/x/net/http2 HTTP/2 implementation rather than the bundled one in net/http, which provides a mechanism to pick up this fix in older Go versions.


Dockerfile enhancements

hi 👋🏼
I am going to be trying to update the dockerfile to make it more lightweight and support multiarch builds. I'll update the title when I am all set :)
Improvements:

Adds tini -  see What is advantage of Tini?
Add tzdata which allows use of switch timezone via the TZ envar
Use a multi-stage build to reduce Docker image size - maybe some more improvements to be made here. Ill leave that up for developers to figure out what's needed and what's not :)
Allows use of multiple architectures

That would be awesome - thanks so much! :)

I have build locally and verify these changes still work, if you ever choose to implement multi-arch builds this single dockerfile should work for all architectures!


Additional Configuration Options

Good work on the GUI binary, works as currently advertised.
It would be nice to be able to include additional configurations (e.g. PostUp & PostDown) in the conf files generated. Either Manually or via the GUI would be fine i believe, however, currently any change made manually to the conf file is overwritten by the GUI.
Any workaround? Or can this feature be added?
Thanks.

PostUp and PostDown would be great for configuring NAT

@zachberger Yes it is, have 2 scripts for these now. But as an enhancement would be a nice thing to have :)

PostUp and PostDown would be great for configuring NAT

@nigerninja @zachberger do try https://github.com/ngoduykhanh/wireguard-ui/pull/28 out if you have the opportunity to and let me know if it's good.
I have an exported docker image downloadable here too: https://drive.google.com/file/d/1annHkDeyXGUwYMN5hl2RisUOBpFhQIre/view?usp=sharing
You can import the above image into docker by using:
docker load -i "/path/to/wgui.tar"

@ngoduykhanh are you accepting pull requests?

If PostUp and PostDown already exist--or are added manunally--to the configuration file, will they be stripped out by wireguard-ui? If not, this is not a biggie.

I just tested it, it doesn't read an existing wg0.conf at all. That's no... good.

If PostUp and PostDown already exist--or are added manunally--to the configuration file, will they be stripped out by wireguard-ui? If not, this is not a biggie.

Yes, to my understanding it will be overwritten according to the codes. May be wrong though, never tested that.

I just tested it, it doesn't read an existing wg0.conf at all. That's no... good.

This issue is only for additional config options, parsing wg0.conf will probably require more work, maybe it's down the pipeline.

I merged the PR from @jarylc for PostUp and PostDown.
For now, the Wireguard-UI will simply override/generate a new config file, depends on the path you set in the setting. Reading from an existing config file will need more work to handle several cases that I am not going to include it now in the project.

@jarylc Thanks! Would update & try it out in a couple of days, in the middle of a project right now. :+1:


Long left click triggers context menu

Version
2.0.0
Test Case
https://vkbansal.github.io/react-contextmenu/#/simple-menu
Steps to reproduce
Left click inside the well for an extended amount of time
Expected Behavior
Nothing
Actual Behavior
Context menu appears

Read holdToDisplay description here

My fault.. when I went through it the first time I stopped reading after "This is applicable only for touch screens."


Documented log level option missing from config (IDFGH-5625)

Environment

Development Kit: ESP32-PICO-Kit
Kit version (for WroverKit/PicoKit/DevKitC): v4.1
Module or chip used: ESP32-PICO-D4
IDF version (run git describe --tags to find it): v4.3
Build System: [Make|CMake|idf.py]
Compiler version (run xtensa-esp32-elf-gcc --version to find it): 8.4.0
Operating System: Windows
(Windows only) environment type: PowerShell
Using an IDE?: Yes (Eclipse with ESP-IDF plugin)
Power Supply: USB, or External with DC-DC regulator to 5V
Wifi Router: Apple Airport model A1408 (~10 years old)

Problem Description
According to the documentation (https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/system/log.html), there should be two menuconfig parameters for managing log level:

At compile time: in menuconfig, set the verbosity level using the option CONFIG_LOG_DEFAULT_LEVEL.

Optionally, also in menuconfig, set the maximum verbosity level using the option CONFIG_LOG_MAXIMUM_LEVEL. By default this is the same as the default level, but it can be set higher in order to compile more optional logs into the firmware.

According to this document (https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/kconfig.html), both of these CONFIG_LOG options should be available in menuconfig under: Component config > Log output.
However, Component config > Log output only shows the "Default log verbosity". It does not show maximum log level. Further, the "Default log verbosity" appears to act as both options simultaneously: it sets the default level and it defines the maximum level that the program can set, not allowing the program to set it any higher using esp_set_log_level.

Expected Behavior
It was expected that both the default and maximum log level settings would be available per the documentation so that each of these can be individually controlled/configured.
Actual Behavior
Only one of the two documented options appears to be implemented in menuconfig, and the one implemented does not completely behave as described in the documentation.
Steps to reproduce

step1
...

// If possible, attach a picture of your setup/wiring here.
Code to reproduce this issue
// the code should be wrapped in the ```cpp tag so that it will be displayed better.
#include "esp_log.h"

void app_main()
{

}

// If your code is longer than 30 lines, GIST is preferred.
Debug Logs
Debug log goes here, should contain the backtrace, as well as the reset source if it is a crash.
Please copy the plain text here for us to search the error log. Or attach the complete logs but leave the main part here if the log is *too* long.

Other items if possible

[ ] sdkconfig file (attach the sdkconfig file from your project folder)
[ ] elf file in the build folder (note this may contain all the code details and symbols of your project.)
[ ] coredump (This provides stacks of tasks.)

Hi,
It looks like you are using IDF v4.2. The feature you are talking about was added recently and doesnt exist in v4.2. On the latest master branch this option is available in menuconfig:

If you are using an earlier version of IDF we recommend using the docs that corresponds to that version, for release/v4.2 this would be https://docs.espressif.com/projects/esp-idf/en/release-v4.2/esp32/

@ESP-Marius thank you very much.
When I execute idf.py --version I get ESP-IDF v4.3-dirty. How can I make sure I have a complete 4.3 version?

Hi,
It looks like you are using IDF v4.2. The feature you are talking about was added recently and doesnt exist in v4.2. On the latest master branch this option is available in menuconfig:

v4.3 does not support  CONFIG_LOG_MAXIMUM_LEVEL, it's only available in master (v4.4).

v4.3 does not support CONFIG_LOG_MAXIMUM_LEVEL, it's only available in master (v4.4).

@AxelLin  Thank you! This was my mistake as I thought I was using the latest ESP-IDF.

@mbratch That's good! I'm closing the issue then.


RadarViewer does not allow changing columns in SpiderChart

https://github.com/datagrok-ai/public/assets/6126004/146a76e9-e8c0-4ca2-8f05-1ce74f87c23e

Hello, @tcarolan!
Thanks for contacting us! Once the problem is resolved, we will send you the updated package version with the necessary fixes.

Thank you Lyak. Please check the selecting rows changes the view of the RadarViewer. I noticed this didn't work today. Do I need to make a different ticket?
https://github.com/datagrok-ai/public/assets/6126004/8876b18d-59b4-40f2-b206-997a5c7cd83c

Sure, we'll look at it too, no need in creating the ticket. Thanks!


[HIGH] Clean up Hoverable component

Coming from here
Problem
Hoverable definition contains mixed logic of a few states (disabled , scrolling, hasHoverSupport), which makes it harder to understand and maintain.
Solution
Here is a list of issues with potential improvements that could help with that:

Logic responsible for internal updates is interleaved with logic responsible for disabled and hoveredEnabled states of the component. This may lead to hard-to-spot bugs.

Split the component into two distinct paths, InactiveHoverable and ActiveHoverable (see linked PR). This way we don't care about handling the disabled logic inside actual Hoverable

Component children definition is not strict enough causing complicated logic for child retrieval

Component should only accept single ReactElement child or callback that returns such single child - if more than one element is needed to have a hovered mechanism, then wrap them into a mutual parent

Component subscribes to unnecessary events, such as onBlur - if Hoverable should be responsible for focus effects as well, it should be taken into account explicitly

Let’s discuss if we could safely remove the subscription to onBlur

There are two pairs of callbacks onHover* and onMouse* which distinction and role are not clearly visible (they are not the same but I am not sure if we need the distinction, e.g. onMouse* event can be taken directly from children passed to Hoverable)

Discuss if we could simplify it, so only one pair is needed (see linked PR)

There are a lot of Hoverable components on the page (lists’ items etc.) and each of them subscribes to scrolling event

Is there an alternative way for what we currently have?

cc @kacper-mikolajczak can you please comment here

Hi @mountiny , I would like to work on this issue

PR is in a review

Logic responsible for internal updates is interleaved with logic responsible for disabled and hoveredEnabled states of the component. This may lead to hard-to-spot bugs.

Split the component into two distinct paths, InactiveHoverable and ActiveHoverable (see linked PR). This way we don't care about handling the disabled logic inside actual Hoverable

✅

Component children definition is not strict enough causing complicated logic for child retrieval

@kacper-mikolajczak Would you say a word or two about how this requirement was met?

Component should only accept single ReactElement child or callback that returns such single child - if more than one element is needed to have a hovered mechanism, then wrap them into a mutual parent

✅

Component subscribes to unnecessary events, such as onBlur - if Hoverable should be responsible for focus effects as well, it should be taken into account explicitly

Let’s discuss if we could safely remove the subscription to onBlur

@kacper-mikolajczak Analogically

There are two pairs of callbacks onHover* and onMouse* which distinction and role are not clearly visible (they are not the same but I am not sure if we need the distinction, e.g. onMouse* event can be taken directly from children passed to Hoverable)

Discuss if we could simplify it, so only one pair is needed (see linked PR)

@kacper-mikolajczak Analogically

There are a lot of Hoverable components on the page (lists’ items etc.) and each of them subscribes to scrolling event

Is there an alternative way for what we currently have?

@kacper-mikolajczak Analogically

Component children definition is not strict enough causing complicated logic for child retrieval
@kacper-mikolajczak Would you say a word or two about how this requirement was met?

The requirement has been already met after refactoring the component to TS. In this PR we simply removed unnecessary mapChildren function which indicated that the component can receive multiple children. Also, the TS typings for child was corrected.

Analogically. Could we safely remove the subscription to onBlur?
After the discussion, we decided not to remove the onBlur handler as it might revive old issues.

Analogically. Is there an alternative way for what we currently have?
Unfortunately, no findings on this one.

Let me know if that answered your concerns 👍

Great, thanks!

The PR was reverted, we'll need to regroup 🙁

Hi @cubuspl42, definitely! I am going to work on the solution as soon as I am done with current stuff 👍

💡 keep the solution suggested in this PR and refactor all the components so we always have single child passed into Hoverable. That way user knows what element Hoverable hooks into. On the flip side, we would not need to make any workarounds but it would require deep understanding of current behaviour and testing against regression.

Sounds clean, but I'm aware of the risk. How many uses are we talking about?

Thanks a lot for the insight @bernhardoj! 👍
In this case, if we want to keep the single child approach but also have possibility to provide single child wrapped in the array (e.g. OfflineWithFeedback example), then we would need to add a mapper for that.
I've added this functionality here: 76a200fa020e162fa6cb9da8acd0cfe6e2dd8fe4, but it was not enough to solve the issue.
While debugging, I came to the point where I tracked down what is causing that and it seems that the problem lies in this change: 87d8979687fd19a16eb685a342231ca60a006728. After I reverted the changes to BaseTooltip (it did clone the React element to pass the mouse event handler), it looked okay.
Question is, if we want to go back to previous approach, where we can pass onMouse* events through Hoverable props (thus avoiding cloning there) or shall we find different approach.
I am open to suggestions ^^

I'm not sure. I liked the idea with ensuring a single child is passed to Hoverable.

I've had a second more on this and it turned out that what was crashing the app is React.cloneElement inside BaseTooltip to acquire array as a children which resulted in an error.
I've posted the fix that we can use here .
It basically parse children in order to extract single child. Let me know what you guys think @cubuspl42 @bernhardoj

I left a comment in that PR and stand by what I said earlier.

Kacper is out of office but pr review continues

@kacper-mikolajczak has been out the PR is moving along though, whats your ETA?

Hi @mountiny, I am going to complete the PR today and hand it off to the reviewer 👍

@roryabraham Assigning you here too since you have helped a lot with the issue and reviews

@500 to @cubuspl42 I believe

Automation didn't seem to create an Upwork offer - Created one here for you @cubuspl42 : https://www.upwork.com/nx/wm/offer/100533857

Accepted 👍

Paid and contract ended, thanks @cubuspl42


Update from bump-version

This is an automated pull request to update from branch bump-version

📝Changed routes:

/

(view full size)

Commit 3ed24210988a4cc6a772c3f382203e51debd0c18 (https://bckstudio-on-zeitnow-x0usiib7j-stryder031.vercel.app).


srange-like manipulation for arbitrarily nested structures

Here is my data structure.
(this has lots (of ?&holes ?that (?are deeply ?nested) ?and stuff ?like (?that)))

Now, by using (walker hole?), I can get all the symbols that start with a question mark. This is useful, but I also want to do something else. If the first symbol that starts with a question mark also has an ampersand (in this case, that condition is true), I want to expand it to two holes, the first of which does not have the ampersand. The final data will look like this
(this has lots (of ?holes ?&holes ?that (?are deeply ?nested) ?and stuff ?like (?that)))

The first "hole" could be anywhere, so walker seems to be a good fit, but it doesn't seem to allow for this type of transformation. srange also seems to be related. Now, I think I could completely reimplement walker to allow for what I'm trying to do, but there is probably a better, and more idiomatic way. What's the right way to do this?

This is a use case that's better handled by zippers. Just keep calling next until you find your "hole", and then call insert-right. The next release of Specter will have zippers integrated into Specter, though it's not currently integrated.

You're lucky that you asked this particular question as I was finishing up the zipper integration :)
Here's sample code from the master branch that handles this use case:
(use 'com.rpl.specter)
(use 'com.rpl.specter.macros)
(use 'com.rpl.specter.zipper)

(declarepath find [predfn])

(providepath find
  (if-path [NODE pred]
    STAY
    [NEXT (params-reset find)]
    ))

(defn number>=10 [n]
  (and (number? n) (>= n 10)))

(transform [VECTOR-ZIP (find number>=10) (collect-one NODE) INNER-RIGHT]
  (fn [v _]
    [(str v "!")])
  [1 [2 10 [4]] 12])

This code finds the first number >= 10 and inserts that number as a string with an exclamation point immediately after it. If it finds no such number, the structure is unchanged. As you can see here, the find navigator is parameterized with the predicate to match by, and so is completely generic.

Haha, that's awesome; thanks a lot. I noticed that you used pred in the providepath but predfn in the declarepath; is that a mistake?

No, that code is correct. This is about as advanced as Specter usage gets, as find is a recursive parameterized path, so it might take a bit to wrap your head around it. Here's a quick explanation:

predfn is just naming one of the parameters to the find path. The name is irrelevant to the execution, all Specter cares about at runtime is how many parameters find requires.
pred is a filter navigator that takes in the filter function as a parameter. As you can see from the definition of find, its definition requires a single parameter total (for pred).
Since it's recursive, find needs to call itself. But to make sense it needs to parameterize its invocation. So params-reset tells it to use the same parameters for the recursive navigation.


Migration Guide from v6 to v7 because of issues with aliases

The documentation website is not extremely helpful if you need to migrate from CRA4 and craco 6 to CRA5 and craco 7.
Wondering if there's any guide or place where I can find more information around it?
Keeping the same config and just bumping versions is causing a lot of errors that I have no clue where they are coming from cause based on the doc my config looks correct but is not working anymore after the update.
If it helps this is my current craco config
const path = require('path')
const { CracoAliasPlugin } = require('react-app-alias')
const SentryPlugin = require('@sentry/webpack-plugin')

module.exports = {
  plugins: [
   {
      plugin: CracoAliasPlugin,
      options: {},
    },
  ],
  babel: {
    plugins: [
      [
        'babel-plugin-styled-components',
        {
          ssr: false,
          displayName: process.env.REACT_APP_NODE_ENV !== 'production',
          pure: true,
        },
      ],
    ],
  },
  jest: {
    configure: {
      setupFilesAfterEnv: ['./src/test/setupTests.ts'],
      moduleDirectories: [
        'node_modules',
        path.join(__dirname, 'src'),
        path.join(__dirname, 'utils'),
        '!src/utils/**/*.{ts,tsx}',
        '!src/env/*.ts',
      ],
      collectCoverage: true,
      coverageDirectory: 'coverage',
      collectCoverageFrom: [
        'src/**/*.{ts,tsx}',

        // Ignore test coverage for the following files:
        '!src/utils/**/*.{ts,tsx}',
        '!src/env/*.ts',
      ],
      resetMocks: true,
      testURL: 'http://localhost/',
    },
  },
  webpack: {
    alias: {
      api: path.resolve(__dirname, 'src/api'),
      app: path.resolve(__dirname, 'src/app'),
      assets: path.resolve(__dirname, 'src/assets'),
      components: path.resolve(__dirname, 'src/components'),
      env: path.resolve(__dirname, 'src/env'),
      services: path.resolve(__dirname, 'src/services'),
      test: path.resolve(__dirname, 'src/test'),
      utils: path.resolve(__dirname, 'src/utils'),
      globalStyle: path.resolve(__dirname, 'src/globalStyle.ts'),
      types: path.resolve(__dirname, 'src/types'),
      logging: path.resolve(__dirname, 'src/utils/logging'),
    },
    plugins: {
      add:
        process.env.REACT_APP_NODE_ENV !== 'local'
          ? [
              new SentryPlugin({
                release: process.env.REACT_APP_VERSION,
                dist: process.env.REACT_APP_VERSION,
                include: './build/static/js',
                ignore: ['node_modules'],
              }),
            ]
          : [],
    },
  },
  eslint: {
    enable: false
  },
}

With this I'm getting some import errors while running dev like
Uncaught TypeError: Cannot read properties of undefined (reading 'emailRegex') at Module.emailRegex (index.ts:1:1)

that is a simple import of a const from an alias.
and it looks like aliases are broken in tests as well
I have also had to add GENERATE_SOURCEMAP=false to fix some other warnings when starting dev
If anyone can give me some advice would be really appreciated!

I'm gonna close this since I think the issue is mainly around webpack 5 and aliases rather than craco.


修复文档中sink.label-prefix 的默认值说明

What type of PR is this：

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[x] Doc
[ ] Tool

Problem Summary(Required) ：
sink.label-prefix 配置默认值描述错误,  代码中是 'flink' 字符串
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)
[ ] I have added documentation for my new feature or new function

 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.


Using GITHUB_TOKEN instead of setting another one

I opened this pull request for taking out the requirement of setting PERSONAL_ACCESS_TOKEN since GitHub Action create a secret called GITHUB_TOKEN automatically. The configuration for projects that use this template would be easier without creating another access token in repository. Please feel free to have a discussion on this PR. Thanks!

Unfortunately, PERSONAL_ACCESS_TOKEN is actually needed for triggering another workflow after version bumping.
As stated in the Using the GITHUB_TOKEN in a workflow section, GITHUB_TOKEN won't be able to trigger another workflow
When you use the repository's GITHUB_TOKEN to perform tasks on behalf of the GitHub Actions app, events triggered by the GITHUB_TOKEN will not create a new workflow run. This prevents you from accidentally creating recursive workflow runs. For example, if a workflow run pushes code using the repository's GITHUB_TOKEN, a new workflow will not run even when the repository contains a workflow configured to run when push events occur.

Anything that would be triggered by versioning bumping. For example, if a new version of commitizen is released through https://github.com/commitizen-tools/commitizen/blob/master/.github/workflows/bumpversion.yml, it should also trigger this one https://github.com/commitizen-tools/commitizen/blob/master/.github/workflows/pythonpublish.yaml#L3.


Orders should be lowercase

Thank you for contributing, we fixed this already :) Hope you enjoyed the workshop!


Can't activate account

Subject: Can't activate account
Description:
Nothing, I clicked on the button sent to me via email and it activated this error page
Results:
Should've activated my acc
Backend version: develop-b3cee755
Frontend version: d5976f07e36d2b578615fff0b4c9b964e1d4352a
User Agent: Mozilla/5.0 (Linux; Android 11; SM-A105FN Build/RP1A.200720.012) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/67.0.3371.0 Mobile Safari/537.36
Page: https://couchers.org/signup?token=qTeGLbswT1sdD-uOX245Ui3_RsnetRKKn0FF94mzYVA=
User:

not enough info or can't reproduce


Make PPP checking more lenient

Only checks that PR#XXX substring is present now.

Codecov Report

Merging #510 (4833c4b) into master (273160a) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 4833c4b differs from pull request most recent head 3b02fc1. Consider uploading reports for the commit 3b02fc1 to get more accurate results

@@            Coverage Diff            @@
##             master     #510   +/-   ##
=========================================
  Coverage     65.53%   65.53%
  Complexity      730      730
=========================================
  Files           147      147
  Lines          2553     2553
  Branches        288      288
=========================================
  Hits           1673     1673
  Misses          742      742
  Partials        138      138

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Mixpanel causes android app when enableProguardInReleaseBuilds=true

My app crashes on android if i set enableProguardInReleaseBuilds=true not sure why build this doesn't happen when debug is true.
Anyone know what could be causing this?

hi @Kazion500 , you're using ProGuard so you need to add the following rule to android/app/proguard-rules.pro:
-keep class com.facebook.jni.** { *; }
please checkout the breaking changes in the RN release notes: https://github.com/react-native-community/releases/blob/master/CHANGELOG.md#breaking


WebHook support

Is it okay to add support to update message from webhook?
I did some research by adding a new method to type Bot
func (b *Bot) WebHook(url string, subscription chan Message) http.HandlerFunc

and seems working fine.
If you think it's acceptable to add such method, I can send a PR for you. Thanks

I don't really think that we do really need this and can't think of a good usecase, can you?

It's impossible to use long polling mode in restricted environment like google app engine.
But you are right. After some refactoring to my code, I found that in webhook mode we need an "API wrapper" not a "bot" instance. So it might be better to create another project forked/rewrote from here to do this.
Thank's for your reply :)

Thaaaat's what I was talking about, alright.


PE Viewer crash (0xC0000005)

PE Viewer crash (0xC0000005)
PE Viewer crashes when opening the library exports table.
Steps to reproduce (optional)
Create a library with the following exported functions (.def file in Visual Studio IDE)
LIBRARY TESTLIB
EXPORTS
  Test @123
  Test2 @456
  Test3 @789
  Test4 @1000
  Test5 @7
  Test6 @666

Compile. When you open the library exports table in PE viewer, you will receive a message invalid access to memory allocation. The library is working and in x64dbg everything is displayed correctly.
Expected behavior (optional)
No response
Actual behavior (optional)
No response
Environment (optional)
No response

Create a library

Static library or DLL?

Create a library

Static library or DLL?

Dll

Unable to reproduce with nightly builds

I suspect this is related to this issue: https://github.com/winsiderss/systeminformer/issues/1907
I am also unable to reproduce. But I intended to close this out when the next build publishes per the change above. If the issue persists after the next build please reopen the issue.

3.0.7459 - Help > Check for updates 👍


Support dynamic query params by reading FHIR definitions

@rcrichton im happy with all these changes and think its ready to be merged into master :+1:


can't get auth key

Had to reset the switchman a few times to get it show up during a switchmate.py scan, but it finally did show up during a scan!
I tried to get the auth key and I got the following error:
$:~/switchmate$ sudo ./switchmate.py xx:xx:xx:xx:xx:xx auth
Traceback (most recent call last):
File "./switchmate.py", line 157, in
device.writeCharacteristic(AUTH_NOTIFY_HANDLE, NOTIFY_VALUE, True)
File "/usr/local/lib/python2.7/dist-packages/bluepy/btle.py", line 459, in writeCharacteristic
return self._getResp('wr')
File "/usr/local/lib/python2.7/dist-packages/bluepy/btle.py", line 334, in _getResp
resp = self._waitResp(wantType + ['ntfy', 'ind'], timeout)
File "/usr/local/lib/python2.7/dist-packages/bluepy/btle.py", line 290, in _waitResp
raise BTLEException(BTLEException.DISCONNECTED, "Device disconnected")
bluepy.btle.BTLEException: Device disconnected
Questions:
-you say: "you cannot use this script and the Switchmate app simultaneously." does that mean I can't have the switch paired with the app at all to use the python script?
-is it required to have a bluetooth 4.0 interface to use this? (currently running this on a older Mac mini running ubuntu server)
Thanks for your work!
-Mike

I'm afraid I won't be able to help you diagnose your particular hardware issues. There are too many variations of bluetooth hardware out there to support. I would suggest trying to find a popular USB bluetooth adapter and use that.

does that mean I can't have the switch paired with the app at all to use the python script?
When the official Switchmate app pairs with a Switchmate switch, it generates a new auth key. You might be able to use the app and this script simultaneously if you were able to use the same auth key but it's not easy to retrieve or intercept that auth key from the app

is it required to have a Bluetooth 4.0 interface to use this?
Yes, as far as I know switchmate devices only support the Bluetooth Low Energy (a.k.a BTLE, BLE) protocol. This protocol was introduced in Bluetooth 4.0


feat: Support pnpm

This adds support for pnpm, which is the third largest node package manager.
vscode‑eslint already supports this (https://github.com/microsoft/vscode-eslint/issues/529, https://github.com/microsoft/vscode-eslint/pull/572)

Thanks!


feat(scrollable-image-ui): upload image to gcs

[x] integration test with https://github.com/twreporter/cloud-functions/pull/24 on staging

Close due to image hosting is not going to be a public service in current plan.


Removal of '20' from the website everywhere

_pages/home.md
I've agreed with some of the OC that this is an ongoing thing hopefully if it is popular so we need to remove the '20' from the website.

Fixed in #169


feat: Add consentmanager

Add consentmanager as provider.
Website: https://www.consentmanager.net/

@PheysX I have some difficulties finding some documentation on how to implement it. Do you have any documentation?


History table only shows placeholders for the head row

When searching for a user, the table displayed doesn't show the correct headings.
Instead of this (as shown on all other pages):

It looks like this:

This issue is also present on the demo version of the website, so I created this issue in an effort to help improve the base code.
I don't know much about PHP, so I was hoping someone more skilled could help out.
Thank you in advance for your help.

I'm just now seeing that the code on Spigot, that links to this repository, looks completely different. Therefore, this issue might not apply here.


admob component

[dcc32 Fatal Error] DW.AdMobBannerAd.pas(14): F1026 File not found: 'DW.GlobalDefines.inc'

Please read the instructions in the readme, regarding Project Configuration: https://github.com/DelphiWorlds/Playground/blob/main/Readme.md

Is your issue solved?

Yes, thanks


hamburger icon not working

hamburger icon appears too soon at lg devices. should display on med devices and below. need to increase its weight, looks to small.

On it Luis will have it working ASAP


missing string formatter for one file in export_system.py

line 287 is missing the string formatter character to replace the parameter in the filename

fixed in r1.1


Expose api and add activation decorator

Fixes #3
Fixes #4

Codecov Report

Merging #5 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master     #5   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines         147    158   +11
  Branches       24     24
=====================================
+ Hits          147    158   +11

Impacted Files
Coverage Δ

responsex/mock.py
100% <100%> (ø)
:arrow_up:

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 d372bd6...31d938e. Read the comment docs.


Rigid and Soft body simulation

integrate a library like Ammo or Cannon.
Simulated objects setup could be done with a few nodes, such as RBDPrepare and RBDSolver. Optional nodes could create low res hulls if needed.

https://socket.dev/npm/package/jolt-physics has rigid and soft body support too.
Was familiar as Jolt Physics is in the Godot Engine channels.

Looks like it's was time for me to clean those outstanding issues, as a few of them should have been closed.
This is the case for this one, as polygonjs now has RBD solver, using rapier ( see examples or source of the main solver now ), and a soft body solver, with its tetrahedralizer, based on 10 min physics  ( no examples yet, but here is the main solver node source ).
And while those 2 solvers are available, they do not interact with one another.
With that in mind, I only heard about Jolt Physics recently, and had not checked it out, so thank you for the link. It is definitely interesting to consider, but changing what I have with rapier now to another solver would need to bring in substantial benefits. For instance, is there a noticeable performance gain?
I'll keep this open for now, as I'm curious to know more about jolt.

I would pick one solver like rapier and be done. The issue was out of date.


Updated docs for k3s

Added info for new --disable-cloud-controller flag for k3s
https://github.com/rancher/k3s/issues/905

LGTM


New docs set on "Contributing to BigchainDB"

It consolidates a bunch of existing docs into one place, and adds some structure and connective tissue.
Audience
New employees, new contributors, existing devs who want a refresher
TODO

How to run a local node with MongoDB and Tendermint??? Including to run all tests. I asked Vanshdeep to write a short doc about how he does it.

The idea is that this will become a third set of contributing docs under this repo, and CONTRIBUTING.md will mostly just point to that set of docs, which will be rendered on ReadTheDocs.
Related Issues

Network Driver - @kansi - #2116
How to set up a standard dev/test node - @muawiakh and @shahbazn

Okay there is still work to do but I will just make a to do list for the future, so we can merge this now.

[ ] Update CONTRIBUTING.md
[ ] Update the GitHub issue template - See "Issue Rules". We only allow bug reports in issues now.
[ ] Update the GitHub pull request template - It could be a BEP or a PR, point to the docs for explanation
[ ] Update README.md

Nice fact:

You can put CONTRIBUTING.md, ISSUE_TEMPLATE.md, and PULL_REQUEST_TEMPLATE.md files in a folder named .github/ and everything will work as expected.

Sorry this one grew so big. The follow up ones can all be small.

Also, there will be a companion page to the page Vanshdeep wrote about running a BigchainDB node "as processes" (MongoDB, Tendermint, BigchainDB) on your local machine:
a page about running a node as containers orchestrated by Docker Compose.
And in the future maybe even other ways to run a node on your local machine! For example, maybe  using Minikube to run a local Kubernetes cluster and then using Kubernetes to hold and connect all the containers together (with storage too).
Stay tuned!

@vrde I will tackle your comments but right now I don't have any more energy. Thanks for your review.

There are still some unresolved comments by @vrde but I need to merge this PR. It has grown out of control. It can be followed by a series of small PRs, like one to resolve issue #2137


[BUG]Web Online Desinger  The barcode displayed in the data list is the same,BarCode type is EAN13

Web Online Desinger  The barcode displayed in the data list is the same，As shown in the figure：

Hello! Thank you for bug-report. Can you send me report template on e-mail<EMAIL_ADDRESS>?

We can solve this problem by selecting barcode 128 as the barcode type, but this problem still exists. You can create a new report at your side, drag a barcode, and then set the barcode type to EAN13 to see this error

It seems to me that this problem appeared due to the fact that the barcode is using the wrong data format, because of the default expression. Therefore, the barcode does not change its type. Also, in the screenshot, I saw that the data is written in the text component (another), and not in the text or expression field of the barcode. Try to put the barcode type EAN-13 and, for example, in its “text” field insert<PHONE_NUMBER>62, or any other in the correct format.


Debian 11 Bullseye compatibility

[Enter feedback here]
Hi, thank you for all the effort in your documentation. I would love to see the compatibility for Debian 11 (Bullseye), specifically in Network/Static IP injection.
I'm trying to use a PowerShell Script for configuring a Static IP of a Debian 11 VM gen 1, but it is not working. When I see the logs in /var/log/syslog I have this error:

KVP: Failed to execute cmd '/usr/libexec/hypervkvpd/hv_set_ifconfig /var/lib/hyperv/ifcfg-eth0'; error: 2 No such file or directory

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

ID: b46eaa07-8134-fb75-5322-d6896a8f3a1b
Version Independent ID: 562af23a-e5ed-33eb-c539-bd2a6a442d17
Content: Supported Debian virtual machines on Hyper-V
Content Source: WindowsServerDocs/virtualization/hyper-v/Supported-Debian-virtual-machines-on-Hyper-V.md
Product: windows-server
Technology: hyper-v
GitHub Login: @BenjaminArmstrong
Microsoft Alias: benarm

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


[router] Add Storybook integration.

Is your feature request related to a problem? Please describe.
It'd be great to have support for the @nuxtjs/storybook module.
Describe the solution you'd like
Create DruxtRouter stories via nuxt.hook('storybook:config', () => {}).
Describe alternatives you've considered
N/A
Additional context

https://storybook.nuxtjs.org/
https://github.com/druxt/druxt-entity/blob/develop/nuxt/druxt-entity.stories.js
https://github.com/druxt/druxt-entity/blob/develop/src/nuxtStorybook.js

This has been done in #376


Dual dummy video support (WebM + MP4)

Prevents audio interruption on both Android and iOS web browsers while also supporting a wider range of browsers (i.e. Chromium on Android that does not play MP4 files).
Based on the idea presented here: https://github.com/richtr/NoSleep.js/issues/55#issuecomment-443984705

Maybe NoSleep could provide the source file in both MP4 and WebM and let the browser play the best fitted one?

I think this is a good idea. Provide two video sources and let the browser pick one. Regarding https://github.com/richtr/NoSleep.js/issues/47 I think the WebM video should not include an audio track.

Consists of:

Fix for https://github.com/richtr/NoSleep.js/issues/55
Fix for https://github.com/richtr/NoSleep.js/issues/47
Deprecates (cherry-picked that proposal on to this PR): https://github.com/richtr/NoSleep.js/pull/45

Merged as part of https://github.com/richtr/NoSleep.js/pull/58. Closing this PR for now.

[ Quote richtr @ CE 2018-12-04 08:01:22 UTC:
https://github.com/richtr/NoSleep.js/pull/58#issue-387151394
    Chromium on Android that does not play MP4 files ]
<^>    MP4 is merely container format.
    What it may not play is probably the H.264 video stream. (try MPEG-2 TS?)


Error Generator

The FFHB site has been completely redone and your tool no longer works.

Indeed, they seems to have changed everything!
I just took a quick look, and it seems that result from API are now encrypted.
I found this part in their code responsible for decrypting API results:
// https://www.ffhandball.fr/wp-content/plugins/smartfire-blocks-project-library/build/static/js/shared/utils.ts
const getBodyDataAttribute = (attributeName, defaultValue = '') => {
    const attribute = document.body.getAttribute(attributeName)
    return attribute ?? defaultValue
}

const xorDecipherBase64 = (strBase64, key) => {
    const str = window.atob(strBase64)
    let result = ''
    let key_len = key.length
    for (let i = 0; i < str.length; i++) {
        result += String.fromCharCode(str.charCodeAt(i) ^ key.charCodeAt(i % key_len))
    }
    return result
}

const xorDecipherAndParse = (str) => {
    const decodeKey = getBodyDataAttribute('data-cfk', 'nokey')
    const decodedStringifyAttributes = xorDecipherBase64(str, decodeKey)
    return JSON.parse(decodedStringifyAttributes)
}

xorDecipherAndParse('DATA FROM API HERE')

I will take a closer look in the next few days.
But maybe it's time for this project to end, as it seems there's a new option on FFHB website "Ajouter les matchs à votre calendrier" that generate a .ICS file.
To be continued 🕵

So,
The ICS file you can download on FFHB website does not have all the informations (score, location, etc), so the project will still be alive.
I just published a new commit with stuff fixed.
Some comments:

For some unknown reasons, referees should be in the API but the fields are always empty (same in the match details page), so for now they are no longer displayed in the generated ICS
Address informations are no longer available in the API, and I can't find another API returning it, so I scraped this data from the match details page
You no longer need to write you team name, as it's already pre selected via the URL field (take a look at the placeholder and help text from the URL input)
As of now, the Handball season as ended, so maybe when the new one is going start, we will find more issues with this fix, wait and see

Let me know if it's working for you 💁♂


Handle case where shim is reaped before the call to the runtime start

This avoid erroring out with a false positive
Signed-off-by: Kenfe-Mickael Laventure<EMAIL_ADDRESS>

LGTM


Establish automated deployment of Mudrod to NSF Infrastructure

Hi @Yongyao, we should establish an automated process which polls the Github master branch, fetches (upon commit to master branch), builds and deploys the Web application and codebase to the Testbed server we have running.
Can you please post here the link to the testbed server.
We can work on this when we talk on Friday.

@Yongyao if this issue is no longer relevant can you please close it off? Thanks


Include all theme sidebars in REST API responses

Developer
Stylesheets

[ ] Any theme or plugin whose stylesheets have changed has had its version
string incremented.

Secrets

[ ] All new secrets have been added to Pantheon tiers
[ ] Relevant secrets have been updated in Github Actions
[ ] All new secrets documented in README

Documentation

[ ] Project documentation has been updated
[ ] No documentation changes are needed

Accessibility

[ ] ANDI or Wave has been run in accordance to
our guide and
all issues introduced by these changes have been resolved or opened as new
issues (link to those issues in the Pull Request details above)

Stakeholder approval

[ ] Stakeholder approval has been confirmed
[ ] Stakeholder approval is not needed

Dependencies
YES | NO dependencies are updated
Code Reviewer

[ ] The commit message is clear and follows our guidelines
(not just this pull request message)
[ ] The changes have been verified
[ ] The documentation has been updated or is unnecessary
[ ] New dependencies are appropriate or there were no changes

Closing this without merging - this change is no longer needed, as the work to crawl the site efficiently has proven to be more achievable.


[Issue] User Management: Update a user via the API

Subtask of user story: User Management https://github.com/usdigitalresponse/cpf-reporter/issues/81
DOD: A createUser graphQL mutation resolver exists which allows authorized users to modify existing users via the API
Blocked by issues:

https://github.com/usdigitalresponse/cpf-reporter/issues/103

Blocks these issues:

https://github.com/usdigitalresponse/cpf-reporter/issues/106
https://github.com/usdigitalresponse/cpf-reporter/issues/107

Implementation Details:

Define an UpdateUserInput input type in users.sdl.ts that implements the following data model, per the User Prisma model:

name: string, optional
agencyId: int, optional
role: enum RoleEnum, one of USDR_ADMIN, ORGANIZATION_ADMIN, ORGANIZATION_STAFF, optional
isActive: boolean, optional

Ensure that the gql schema in users.sdl.ts implements a mutation for updateUser that:

Accepts input conforming to the UpdateUserInput input schema
Uses the @requireAuth directive to restrict access to authenticated users with a role of USDR_ADMIN or ORGANIZATION_ADMIN.

Define an API-side users service with an updateUser mutation resolver which satisfies the following scenarios:

Note: use service validations to validate input and other user creation constraints. Relevant service validations are linked below.
Users may only be updated when the following input conditions are satisfied:

name must not be null
role must not be null and must be a valid UserRole enum
agencyId must refer to an existing agency

Requests made by a user with role USDR_ADMIN should be able to update a user of any role for any organization
Requests made by a user with role ORGANIZATION_ADMIN should be able to create a user of any role in the same organization as the authenticated user, but not for any other organization. Note that this must be determined via the agency relationship of both users, e.g.// Assume `currentUser` has `.agency.organizationId` loaded
validateWith(async () => {
  if (input.agencyId === undefined) { return }
  const targetUserOrgId = (await db.agency.findUniqueOrThrow({
    where: { id: input.agencyId },
    select: { organizationId: true },
  })).organizationId
  if (targetUserOrgId !== currentUser.agency.organizationId) {
    throw new Error("You don't have permission to do that")
  }
})

Changes to agencyId are only allowed if the new/target agency identified in the update input belongs to the same organization as the user's current/previous agency, e.g.validateWith(async () => {
  if (input.agencyId === undefined) { return }
  const currentAgency = (await db.user.findUniqueOrThrow({
    where: { id: id },
    select: { agency: { select: { id: true, organizationId: true }}},
  })).agency
  if (input.agencyId === currentAgency.id) { return }
  try {
    await db.agency.findUniqueOrThrow({
      where: { id: input.agencyId, organizationId: currentAgency.organizationId },
    })
  } catch (err) {
    // Consider also including IDs for user, current agency, and new agency in the structured log data
    log.error({ err }, 'change to user.agencyId is not allowed')
    throw new Error('agencyId is invalid or unavailable to this user')
  }
})

@as1729 Can you confirm if we want the same basic validations on name, email and role for both create as well as update? I noticed createUser has a validation on email (whereas this ticket doesn't specify), and this ticket has a role validation whereas createUser does not. I'd be inclined to make a common method for both w/those three criteria, but not sure if I am missing intended behavior :)

@laurelkorwin good point- the validations should be identical for both create and update 👍

Also, noodling over permissions in my head and wanted to clarify -- it seems like:

USDR_ADMIN should be able to create / update any type of user, for any org, for any role:
ORGANIZATION_ADMIN should be able to create / update users of any role except for USDR_ADMIN, for any agency where the new agency id is part of their organization
ORGANIZATION_STAFF should not be able to create / update users at all.

Yup that summary is correct