Provide an output that can be used in later steps to avoid duplicate runs

The request
As another step towards reducing load on the rate limiting - could you provide an action output that we could use like result=fail. This would be decoupled completely from exit code (for which the behavior would remain the same as it is today, depending on the exit-code flag)
The background
To explain where this would be useful - our typical flow up to now has been to run Trivy twice in a workflow

once to produce the nice readable table view (with exit-code 1) so engineers can see the feedback direct in the workflow if it should fail a scan and we can "break the build"
then immediately again (if: always())to get in SARIF format for upload to GHAS

What would an output solve?
Mainly that we can just start to use convert instead of a second run whilst presenting a workflow output that is easy and obvious to an engineer to see their results in a workflow when it fails.
To do this now in a less obvious format for the consumer of the workflow I have something like:
- name: Trivy Image Scan
  id: scan
  uses<EMAIL_ADDRESS>  with:
      image-ref: some-image
      exit-code: 1
      format: json
      output: "report.json"

- name: Setup trivy for SARIF
  if: always()
  uses<EMAIL_ADDRESS>
- name: Convert to SARIF
  if: always()
  run: |
      trivy convert --format=sarif --output="results.sarif" "results.json"

- name: Upload results to GitHub Security
  if: always()
  uses: github/codeql-action/upload-sarif@v3
  with:
      sarif_file: "results.sarif"

- name: View readable report
  if: steps.scan.outcome == 'failure'
  run: |
      trivy convert --format=table results.json && exit 1

The problems with this are:

there will be two failed steps. One of which is the original scan step with no output because its written to file (confusing to engineer and they may assume it was a runtime error) and then a second step further down the job log to find the actual results in a readable format.
I cannot trigger the readable report without also failing the scan step
Its all a lot of faff :)

What would the solution look like
- name: Trivy Image Scan
  id: scan
  uses<EMAIL_ADDRESS>  with:
      image-ref: some-image
      exit-code: 0                                               <<<------ engineer will not look here
      format: json
      output: "report.json"

- name: Setup trivy for SARIF
  if: always()
  uses<EMAIL_ADDRESS>
- name: Convert to SARIF
  if: always()
  run: |
      trivy convert --format=sarif --output="results.sarif" "results.json"

- name: Upload results to GitHub Security
  if: always()
  uses: github/codeql-action/upload-sarif@v3
  with:
      sarif_file: results.sarif

- name: View results
  if: steps.scan.outputs.result == 'fail'
  run: |
      trivy convert --format=table "results.json && exit 1       <<<----- nice obvious end of job view if there are failures

So now the workflow output makes total sense to a consumer when there are findings, you get less requests for the DB because we arent running it twice.
Alternative suggestions
Adding the convert command to the action which would also be nice - but I suspect much more work than a simple addition of an output?
Allow multiple outputs in a single execution - but you guys already rejected that many times and is the reason you added the excellent convert command :)
Add a upload-sarif flag and you handle the creation of and upload of the sarif directly in the initial execution - also a lot more work but would be nice
You could go bananas with outputs like total-findings, critical-findings, high-findings etc etc

As a consumer of the trivy findings, I endorse this request.

Maybe I'm misunderstanding something here but running trivy twice in the same pipeline job doesn't request a DB twice, does it?

@simar7  It shouldn't (though it has in recentish runs I think because of now-resolved bugs)
I try to be a good citizen and have 2 steps in the same job - I have seen many having parallel jobs (for speed).
But bringing convert as the correct method to approach multiple and dependent formats requires either the output or convert to be better exposed by the action.
I am pretty close to throwing out the action anyway now give we have the setup-trivy action.... its a lot of ugly flag management in yaml-wrapped shell to just use native trivy... but most those suggestions would alleviate the need for that


Create Service Instance Stepper: Bind to app fails for async SI's

when creating a SI we specify accepts_incomplete, which means for async SI creation CF returns SI entity as if it were created
then, if bind to app is specified, we attempt and fail to bind to it
before binding we should check last_operation, and act differently if create is still in progress
we could do one of...

(low bar) show message that we can't bind just yet
poll SI and wait until service is created before binding (what would we do if this takes a looong time?

Can test async SI creation using overview-broker

Presumably this is the cause of the error I get when attempting to bind minibroker services:
Failed to create service instance binding: An operation for service instance mongoblog-db is in progress.. Code: CF-AsyncServiceInstanceOperationInProgress

Linked to #3682. In v3 they're removing sync (accepts_incomplete) operations, to support that we'll need to rewrite all service actions and this will be fixed/obsoleted in the process.


Remove unused code in mlx_scatter_args_slice

Proposed changes
The up is not used anywhere in the code below.
Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING document
[x] I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have updated the necessary documentation (if needed)

This was a bug. I added a fix in #1122. Closing this.


Transaction causes device to spew newlines

For example, I'm working with a electronic scale. When I open it with cutecom and select 1200 baud, 8 bits data, no parity, 1 stop bit I can send the "Get Weight" command which is D05\r\n, then I see in console 123.032g\r\n.
When I try to use go-serial and writing the command after opening it with same mode parameters, setting the read timeout to 1 second I get my buffer filled with newlines as soon as I send the command. Switching to Cutecom and opening the port in this state causes cutecom to crash due to the newlines filling the log. The only way to reset the devices gone-amok state is by disconnecting and connecting the USB cable.
Why would go-serial give different behavior to pySerial and cutecom? What could be making the device to enter this tilted/"stuck" state?

Big upvote for this here. I am implementing a driver for a laboratory balance, facing EXACTLY the same issue as above.

Please provide the source code to reproduce the issue.

@Darkskald The way I ended up solving this issue is by adding a mutex to the balance struct and making sure I write once every 1 second minimum. Writing faster will trigger the amok state (though have not tested at what frequency the bug pops up).
I am not using a read timeout, but am wrapping the serial device with a cereal.NonBlocking to not block on a failed message, which happens often.

It could be that you get partial Reads.
May you try to print the results of all Read operations?
	if err := port.Write(cmd); err != nil {
		fmt.Println("Error writing to port: ", err)
	}
	n, err := port.Read(buffer[:])
	fmt.Printf("READ: n=%d err=%v data:>%s<", n, err, buffer[:n])
	// The content of the buffer[:n] here may be a partial response
	// You may need to read again to get the remainder of the full response

	if err := port.Write(cmd); err != nil {
		fmt.Println("Error writing to port: ", err)
	}
	n, err = port.Read(buffer[:])
	fmt.Printf("READ: n=%d err=%v data:>%s<", n, err, buffer[:n])

	if err := port.Write(cmd); err != nil {
		fmt.Println("Error writing to port: ", err)
	}
	n, err = port.Read(buffer[:])
	fmt.Printf("READ: n=%d err=%v data:>%s<", n, err, buffer[:n])

	if err := port.Write(cmd); err != nil {
		fmt.Println("Error writing to port: ", err)
	}
	n, err := port.Read(buffer[:])
	fmt.Printf("READ: n=%d err=%v data:>%s<", n, err, buffer[:n])
	fmt.Println(buffer[:n])

The partial reads are not material to the issue of newline spewing. The
issue I'm having is that I must be extremely careful when using bugst
compared to cutecom and pyserial since writing too frequently with bugst
will cause the scale to irrecoverably go into a state where all it does is
write newlines until I unplug and plug again.
Normally I'd attribute this to bad firmware or hardware... but I then
observed cutecom and pyserial never run into the issue so I began wondering
if it was a problem in how linux port is handled by bugst.
On Sun, Feb 18, 2024, 13:16 Cristian Maglie @.***>
wrote:

It could be that you get partial Reads.
May you try to print the results of all Read operations?
if err := port.Write(cmd); err != nil {
fmt.Println("Error writing to port: ", err)
}
n, err := port.Read(buffer[:])
fmt.Printf("READ: n=%d err=%v data:>%s<", n, err, buffer[:n])
// The content of the buffer[:n] here may be a partial response
// You may need to read again to get the remainder of the full response
if err := port.Write(cmd); err != nil {
fmt.Println("Error writing to port: ", err)
}
n, err = port.Read(buffer[:])
fmt.Printf("READ: n=%d err=%v data:>%s<", n, err, buffer[:n])
if err := port.Write(cmd); err != nil {
fmt.Println("Error writing to port: ", err)
}
n, err = port.Read(buffer[:])
fmt.Printf("READ: n=%d err=%v data:>%s<", n, err, buffer[:n])
if err := port.Write(cmd); err != nil {
fmt.Println("Error writing to port: ", err)
}
n, err := port.Read(buffer[:])
fmt.Printf("READ: n=%d err=%v data:>%s<", n, err, buffer[:n])
fmt.Println(buffer[:n])
—
Reply to this email directly, view it on GitHub
https://github.com/bugst/go-serial/issues/160#issuecomment-1951374411,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5BK55VAXBZN63YQG4WBWXDYUISOTAVCNFSM6AAAAAAZDQNIX2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJRGM3TINBRGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'll also note that I started work on a pyserial syscall-identical serial implementation in Go to test this out here, still not finished since reading syscall python code is quite hard to follow.

Normally I'd attribute this to bad firmware or hardware... but I then
observed cutecom and pyserial never run into the issue so I began wondering
if it was a problem in how linux port is handled by bugst.

In my opinion, is the firmware that is buggy and goes into this weird loop state if you try to send a command while the scale is still writing the response of the previous command.
In this sense getting a partial reply means that you goes into sending the next command while the previous response is still being written.

To support my hypothesis you may try to use this function instead of the simple Read:
func ReadLine(port serial.Port) (string, error) {
	buf := make([]byte, 100)
	line := ""
	for {
		n, err := port.Read(buf[:])
		if err != nil {
			return "", err
		}
		if n == 0 {
			return line, errors.New("timeout")
		}
		line += string(buf[:n])
		if i := strings.Index(line, "\n\r"); i != -1 {
			return line[:i], nil
		}
	}
}


Define Dart API presentation of invalid native plugin state conditions

As pointed out here, the following comment which is duplicated in a few places needs resolution:

TODO if ably is nil here then an error response, perhaps? or allow Dart side to understand null response?

This is a fairly key architectural decision so should be documented well.

@QuintinWillison this issue is irrelevant now with singleton Ably instance on either platforms


TypeError: Cannot read properties of undefined (reading 'content')

Starting from 0.13.1, while having a conversation with deepseek-coder-v2:16b-lite-instruct-q6_K (maybe with other models too, haven't tested) such an exception gets thrown:
deepseek-coder-v2:16b-lite-instruct-q6_K

Flow to reproduce it on my machine:

Send a big message (Code of razor component, 600+ lines)
Get response from the model
Send another message
Error.

The funny things is that the response can be shown for some time, but then it gets delete and replaced with "Sorry". And I end up with an error message and then the response

This error message is indicative of the context window of the model being exceeded.
We made some changes to the way we interact with Ollama a few months ago (way before 0.13.1) which was causing a similar error message to appear.
Unfortunately I'm traveling right now and it's impossible for me to download such a big model on hotel wifi, but I'll try to take a look at it in the next few weeks.
In the meantime, what version of Ollama are you using?

Thank you for the response! At the time the error appeared, I had been using 0.13.1. Then I updated to the most recent one (0.14.x), but the error did not disappear

@EduardKononov sorry, I was asking about your version of  Ollama.
Right now, the latest is 0.3.11

Oh, sorry, misread. 0.3.11

@EduardKononov I tried this model a bunch, with long conversations and I'm unable to replicate this issue. Is it possible your Ollama server is running out of memory?
Can you provide the specs of the Ollama server environment?
You can run npx envinfo --system --npmPackages svelte,rollup,webpack --binaries --browsers.

I can replicate this on a system with 8GB VRAM + 64GB RAM, so ollama is not running from memory, here is some more info.
I'm using ollama 0.3.12 with CUDA (ollama-cuda) and model is partialy on GPU, partially on CPU (since it does not fit within 8GB of VRAM)
NOTE: I can also reproduce this error while running this model with ollama run, so I don't think that the issue is related to Hollama.
When running with ollama run, followin error occurs after some time of chatting:
Error: an unknown error was encountered while running this model

NAME                    ID              SIZE    PROCESSOR       UNTIL
deepseek-coder-v2:16b   63fb193b3a9b    10 GB   22%/78% CPU/GPU 2 minutes from now

For debugging purposes I have prepared a following conversation that should trigger this (each line is a new prompt in the same session):
Generate me a curl command that makes a POST request to https://moodle.example.com/login/index.php with from data "username" and "password" and saves cookies output to a file

I need to use form data not query params

I also need a logintoken from moodle, how can I obtain it programatically with curl

What does -d parameter do?

Can you use -F instead of -d?

What do -oP flags in grep do?

What does \K mean in regular expression?

Write this using awk and regular expressions capture groups instead of \K


IDC-1215: CCC uni/bidirectional annotations to Cornerstone tool state

Uses  a new dcmjs Cornerstone MeasurementReport hook to allow defining Crowds Cure Canter matching logic to map a measurement group to a Cornerstone tool class instead of relying on CCC random tracking identifiers e.g. "web annotation" which doesn't work for current SR hydration implementation

DCMJS PR (already merged): https://github.com/dcmjs-org/dcmjs/pull/201
Related ticket: https://github.com/OHIF/Viewers/issues/1215
Datasets for uni directional: https://wiki.cancerimagingarchive.net/display/DOI/Crowds+Cure+Cancer%3A+Data+collected+at+the+RSNA+2017+annual+meeting
Datasets for bi direictional: https://wiki.cancerimagingarchive.net/display/DOI/Crowds+Cure+Cancer%3A+Data+collected+at+the+RSNA+2018+annual+meeting

just some questions, since I am not familiar with SR:

why is this PR not pushed to https://github.com/OHIF/Viewers as well?
Is this a IDC customization since CrowdsCureCancer identifiers are IDC specifics? would this mean that this PR will not work with "general"/"standard" uni/bidirectional annotations?
could you please give a brief summary which SR annotations this PR implements, what is already available in the OIHF viewer and what is missing in the context of OHIF#1215 and http://docs.google.com/document/d/1bR6m7foTCzofoZKeIRN5YreBrkjgMcBfNA7r9wXEGR4/?

@Punzo to get you up to speed...
This is IDC specific request. There's an open discussion here and here about a better way to store planar annotations using Clunie's paper. It's not a solid standard but the path we should take, we still need to come up with a solid representation + solid matching that would cover other annotations + including some math checks but this will require more thought and more people.
This is a temporary solution for the current implementation that uses tracking identifiers as the key to mapping from SR to a Cornerstone annotation. The problem is that this tracking identifier is populated with Cornerstone tool info only when the STOWRS by OHIF. Since these CCC annotations were generated somewhere else + are uni/bi-directional annotations only, i just added a simple and straightforward check for long axis or length (okey for this scope).

just some questions, since I am not familiar with SR:

why is this PR not pushed to https://github.com/OHIF/Viewers as well?
Is this a IDC customization since CrowdsCureCancer identifiers are IDC specifics? would this mean that this PR will not work with "general"/"standard" uni/bidirectional annotations?
could you please give a brief summary which SR annotations this PR implements, what is already available in the OIHF viewer and what is missing in the context of OHIF#1215 and http://docs.google.com/document/d/1bR6m7foTCzofoZKeIRN5YreBrkjgMcBfNA7r9wXEGR4/?

@Punzo to get you up to speed...
This is IDC specific request. There's an open discussion here and here about a better way to store planar annotations using Clunie's paper. It's not a solid standard but the path we should take, we still need to come up with a solid representation + solid matching that would cover other annotations + including some math checks but this will require more thought and more people.
This is a temporary solution for the current implementation that uses tracking identifiers as the key to mapping from SR to a Cornerstone annotation. The problem is that this tracking identifier is populated with Cornerstone tool info only for OHIF STOWRS. Using a tracking identifier to do this mapping is not right but it was a valid POC at the time because it doesn't require smart matching SR properties / scoord to a Cornerstone annotaiton. Since these CCC annotations were generated somewhere else + are uni/bi-directional annotations only, i just added a simple and straightforward check for long axis or length (okey for this scope).

OK, thanks for the exaplanation!

@igoroctaviano can you please:

let me know if the sandbox has been updated to include the features you implemented to support CCC annotations
sandbox reports version 4.9.13 - how do I figure out where that is with respect to this PR merged?
what is the procedure to get to see those in-plane annotations (in the likely event I get confused)

@igoroctaviano can you please:

let me know if the sandbox has been updated to include the features you implemented to support CCC annotations
sandbox reports version 4.9.13 - how do I figure out where that is with respect to this PR merged?
what is the procedure to get to see those in-plane annotations (in the likely event I get confused)

Here's the test study with CCC annotations, which does not work for me: https://idc-sandbox-000.firebaseapp.com/projects/idc-tcia/locations/us-central1/datasets/tcia-idc-datareviewcoordination/dicomStores/CrowdsCure/study/<IP_ADDRESS>.4.1.145<IP_ADDRESS>.9203.4004.100545948082587796019777812429

I just asked Bill to check that + ask if we have IDC-specific version files to update to keep track.
The procedure is basically opening the measurement panel and clicking to jump to annotation.


Fix invalid HELM option in usage docs

Description
Fix usage documentation with valid helm template flag --include-crds
https://helm.sh/docs/helm/helm_template/
Fixes: #373
Checklist
Please make sure that your PR fulfills the following requirements:

[ ] Reviewed the guidelines for contributing to this repository
[ ] The commit message follows the Conventional Commits Guidelines.
[ ] Tests for the changes have been updated
[ ] Are you adding dependencies? If so, please run go mod tidy -compat=1.17 to ensure only the minimum is pulled in.
[x] Docs have been added / updated
[ ] Optional. My organization is added to USERS.md.

Type of Change

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[x] Documentation content changes
[ ] Other (please describe)

Other information

 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.

@brudnyhenry Don't forget to sign the CLA so we can review and accept your contributions! 😄

Codecov Report

Merging #374 (96f393b) into main (9c7288a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #374   +/-   ##
=======================================
  Coverage   77.64%   77.64%
=======================================
  Files          22       22
  Lines        1011     1011
=======================================
  Hits          785      785
  Misses        142      142
  Partials       84       84

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@brudnyhenry Can you rebase this PR so we can merge it in? Thanks! 😄


Can't run docker image from microservice tutorial

Problem encountered on https://dotnet.microsoft.com/learn/aspnet/microservice-tutorial/run-docker
Operating System: macos
Provide details about the problem you are experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
MacOS version: 10.14.6
Docker version: 19.03.5 build 633a0ea
dotnet version: 3.1.101
When I created the microservice image and want to run it, the following message appears:
It was not possible to find any installed .NET Core SDKs
Did you mean to run .NET Core SDK commands? Install a .NET Core SDK from:
      https://aka.ms/dotnet-download

When I check the docker images both the aspnet and sdk images are there as version 3.1

Fixed it. Problem was a human error, not a software error


fix: refactored code and added support of environment variables

This pr fixes #56.
Description:

This PR implements a environment variable support & barrel export pattern for the routers in the routes directory.
Previously, each router was imported individually in the index.js file, leading to repetitive import statements and potential maintenance issues.
With this change, all routers are now exported from index.js using a barrel export approach, improving code organization and maintainability.
This change also aligns with best practices for project structure and module organization.

Code Changes

[x] Implemented the support of dynamic environment variables as mentioned in .env.example in the root directory.
[x] Implemented barrel export pattern for routers in the routes directory.
[x] Refactored code to use barrel imports for routers in index.js.
[x] Updated router import statements in other files to use the new barrel export pattern.
[x] Organized & Modularized route handling.

Reviewer Requests:

@Nostromos  Requesting your review and feedback on the changes introduced by this PR.
Particularly interested in feedback on the implementation of the barrel export pattern and its impact on code organization.

Not sure how you found this project but a tip - if a project only has one or two people working on it, you should ask before going straight to a PR. I'd also recommend diving deeper and proposing more substantial changes than just changing up imports and dotenv.
Lastly, if you're looking for more open source involvement, you should focus on bigger projects. First, spend 2-3 weeks really learning how a tool works and is built. Then, when people create issues, look for smaller, simple ones like bugs and ask if the maintainers mind if you try to solve. If they say yes, then you try to solve the problem and only put a PR in when you've done that.

I'm really sorry for the inconvenience I caused. I understand now that I should have asked for permission before making changes, especially in such a small team setting. I'll make sure to communicate better and ask for guidance in the future. I truly value your feedback and will strive to learn from this mistake. Again, I apologize.

No inconvenience and apologies if I came across too harshly! Best of luck in your OSS journey.
BTW, I agree with all the changes you proposed but they're minor and I'm trying to do a few big things before I go back through and clean up.


[feature] GetUserPresence

Hey there!
I'd like to know if there's any reason why the method GetUserPresence is not implemented. If you feel like it's worthy the addition, I can submit a PR.
Thanks in advance.

Welp, somehow I missed it in the users.go here.
Closing.


[Feature] Add new Accordion

Description
A new Accordion with icons
Screenshots
https://github.com/Rakesh9100/Beautiify/assets/112852251/5fdc0d50-65ec-4d2c-9d56-ca1af7a14a37
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[X] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

Every accordion expands on click, what is new in your issue @yusra05

It has different style from others

You have to provide me the name of this accordion @yusra05

Iconic Accordition   [ Accordition with icons ]

Are you giving the name or should I close the issue @yusra05

how about simple accordition.

Okay, go ahead @yusra05


[nice to have] Example of testing component with Link element

Hi,
In the component I need to test, I have an  element and I didn't manage to find any example of this case on your documentation.
I believe this could be an interesting ad to your documentation since it's a common scenario.
Thanks,
Gabriella

Thanks so much


homepage and category page load /page-not-found

Current behavior

When load homepage or category page. It will get a request to /page-not-found. Page will reload twice.
Expected behavior

Don't random load /page-not-found
Steps to reproduce the issue

Repository

my local
Can you handle fixing this bug by yourself?

[ ] NO

Which Release Cycle state this refers to? Info for developer.
developer

[ ] This is a bug report for test version on https://test.storefrontcloud.io - In this case Developer should create branch from develop branch and create Pull Request 2. Feature / Improvement back to develop.
[ ] This is a bug report for current Release Candidate version on https://next.storefrontcloud.io - In this case Developer should create branch from release branch and create Pull Request 3. Stabilisation fix back to release.
[ ] This is a bug report for current Stable version on https://demo.storefrontcloud.io and should be placed in next stable version hotfix - In this case Developer should create branch from hotfix or master branch and create Pull Request 4. Hotfix back to hotfix.

Environment details

Browser: Chrome
OS: MAC-OS Mojave 10.14.1
Node: 10.2.1
Code Version: 1.8.2

Additional information

Could You please make sure that this issue still occurs in the 1.8.4 / 1.9.0.rc-2? Please provide us with more information about the URL's that causes this issue etc

@wh00527 Do you by any chance have defaultStoreCode set?


Python 3 compatibility added

The PR to make Elodie Python 2 and 3 compatible.

@zserg Thanks for this. Looks like the exiftool site is down which is causing the tests to fail. However I get some other failures when I try to run the tests which are not related. Here is what I see.
Some of these are not a big deal and I'm okay to update the flake8 and pep8 commands accordingly (see the .pre-commit file in the root of the repo). But I'd like to get the tests passing before I merge; even if that means updating the syntax checking for some of the issues.
(elodie-py3) ➜  elodie-py3 git:(zserg-py3_1) python -V
Python 3.5.2
(elodie-py3) ➜  elodie-py3 git:(zserg-py3_1) ./.pre-commit
elodie/geolocation.py:6:1: E402 module level import not at top of file
elodie/geolocation.py:8:1: E402 module level import not at top of file
elodie/geolocation.py:9:1: E402 module level import not at top of file
elodie/geolocation.py:10:1: E402 module level import not at top of file
elodie/geolocation.py:10:1: F401 'fractions' imported but unused
elodie/geolocation.py:12:1: E402 module level import not at top of file
elodie/geolocation.py:13:1: E402 module level import not at top of file
elodie/geolocation.py:13:22: E401 multiple imports on one line
elodie/geolocation.py:15:1: E402 module level import not at top of file
elodie/geolocation.py:16:1: E402 module level import not at top of file
elodie/geolocation.py:80:80: E501 line too long (84 > 79 characters)
elodie/geolocation.py:151:43: E231 missing whitespace after ','
elodie/media/base.py:17:7: E111 indentation is not a multiple of four
elodie/media/base.py:19:7: E111 indentation is not a multiple of four
elodie/media/base.py:21:1: E302 expected 2 blank lines, found 1
elodie/media/media.py:12:1: F401 'builtins.object' imported but unused
elodie/media/photo.py:9:1: F401 'builtins.str' imported but unused
elodie/media/video.py:10:1: F401 'builtins.str' imported but unused
elodie/media/video.py:11:1: F401 'builtins.object' imported but unused
elodie/media/video.py:12:1: F401 'past.utils.old_div' imported but unused

@zserg looked at the diff. Great work on this.

@jmathai thanks for comments. All done.

Coverage increased (+0.6%) to 82.575% when pulling ad1cbefb15077844a6f64dca567ea5600477dd52 on zserg:py3_1 into 4f19c0c00d7ea76d7d8884b201992853b1689c4f on jmathai:master.


Support categorization of tags

Is your feature request related to a problem? Please describe.
Currently, there is no way to group tags together. For example: I may want to distinguish tags that indicate quality from tags that classify a data type.
Describe the solution you'd like
Modelling:

Create a tag category concept
Add a property to tags that lets tags be linked to a tag category
Index tag <> tag category relationships in the graph database

UX:

when creating a tag, add an option to add a category to a tag or create a new category
from a tag's page, option to add or change the category of a tag
a tag's badge color is determined by its category
the tag category linked to on a tags entity page
a tag category page that lists all tags in that category

Describe alternatives you've considered

Allow categorization by tagging tags. We considered this because it simplifies the data model. However, it may cause additional confusion when a tag is used as both a tag category and a standard tag.

Additional context
Tags rfc: https://github.com/linkedin/datahub/pull/2112

In my opinion tag categories/groups/types is something that users don't get to control via UX... they only get to assign them to tags in UX a list of categories should be predefined especially since categories are be mapped to styles in UI.

Hi folks,
We are now managing feature requests through our new portal to improve visibility & prioritize work into our public roadmap. If this request is still relevant, please create a new issue so we can gauge current interest from the Community & prioritize work accordingly.
We'll close out this issue in the meantime. Feel free to reach out to me on DataHub Slack with any questions!


Add clippy script.

This change is

.github/workflows/ci.yml line 30 at r1 (raw file):

      - run: |
          cargo +nightly-2022-11-03 fmt --all -- --check
          cargo clippy --all-targets --all-features

it seems that it didn't work.
Code quote:
cargo clippy --all-targets --all-features

.github/workflows/ci.yml line 30 at r1 (raw file):
Previously, ilyalesokhin-starkware wrote…
it seems that it didn't work.

acutally, nvm


High severity vulnerabilities found in npm audit

Hey, when installing tremor at the moment, npm reports high severity vulnerabilities within the package.

hi @adamkona, this is an ongoing issue in recharts (the chart library we are using under the hood). I hope this gets resolved soon!

It seems as though there's been some recent movement:  https://github.com/recharts/recharts/pull/3120

any news about this ?

@pedroresende seems like the PR in the recharts repo is still open, we hope it gets merged soon. @bloomfieldj, thanks for mentioning this!

Thanks @mitrotasios

Just a heads up – it looks like the PR was merged!

thanks @stritt - will adjust asap!

Should be fixed with release v1.6.0

Can confirm it's fixed. Great work! 🚀

awesome thanks 🥳


Date, Timeouts, and Intervals

I plan on adding the ability to fake time. With fake time you will be able to set a starting time and control when or how time ticks forward. When time is faked, Date, setTimeout, clearTimeout, setInterval, clearInterval would all be faked on the window object until time.restore() is called.

https://github.com/udibo/mock/pull/6


Facebook title

Hello, I tried this
<%= facebook_button title: article.title, :turbolinks => true, url: article_url(article) %>
but article title is not picked up in the view.
If i remove title: article.title completely, then it uses the page title.
What I really want is for the share button to have the custom title that I give it.

I solved this by adding meta tags with the help of this https://developers.facebook.com/tools/debug/


update boxplot theme

Adds theme that always angles the x-axis for readability.

This looks great!


Split the instantiation function into two phases

The allocation phase is guaranteed to succeed and just puts enough of the structure together to make things work.
The completion phase does any component metadata lookups that are necessary (for the superclass, fields, etc.) and performs layout; it can fail and require restart.
Next up is to support this in the runtime; then we can start the process of making metadata accessors actually allow incomplete metadata to be fetched.

@swift-ci Please test and merge.


Update hello-hybrid-key-whfb-settings-pki.md

The note regarding NTAuth certificate store is invalid. It states that you must add the DC certificate in the store. It should have stated to add the issuing CA certificate in the store. +Added a link to certutil command

@AlexLafargue : Just in case your content change gets accepted by the document author, would it be OK to ask you to add a MarkDown compatibility space after the quote markers ( > ) ?

This PR has been approved by the doc author, but it needs follow-up from the DM/PM for the final merge process.

@Dansimp The PR that has been approved by document owner can be merged. Thanks.

Fixed in https://github.com/MicrosoftDocs/windows-itpro-docs/pull/5389

@AlexLafargue : The change suggested in this PR is still not part of the documentation.
If you are certain that this content is correct, you will want to open a new PR containing the exact same changes.


error in chrome and also in ionic

Refused to load the image
'data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAAA4AAAAECAQAAAAU2sY8AAAAL0lEQ…/9P///yP9d/9d/ZkAHjybCJScyYIJbE85/OvJp1wQG4gAADBkams/Cpm0AAAAASUVORK5CYII=' because it violates the following Content Security Policy directive: "default-src 'self' http:". Note that 'img-src' was not explicitly set, so 'default-src' is used as a fallback.

have you found a fix?


从v0.22回滚到0.21后无法读取数据库了

Describe the bug
从v0.22回滚到0.21后无法读取数据库了，我的程序自动更新到了0.22版本，后来发现手机端memos无法连接至服务端，于是回滚到v0.21版本，发现一直显示读取数据，重新升级到0.22版本后没有问题
After rolling back from v0.22 to 0.21, I couldn't access the database. My program automatically updated to version 0.22, but later I found that the memos on the mobile end could not connect to the server. So I rolled back to version v0.21 and found that it kept showing "reading data". After upgrading to version 0.22 again, there were no issues.
Steps to reproduce
1、updated to version 0.22
2、open memos
3、Downgrade to v0.21
4、Unable to read data
The version of Memos you're using
v0.22
Screenshots or additional context
No response

Duplicated with https://github.com/usememos/memos/issues/3348


TestServer blocks on HostedService asyncronous background threads

A .NET Core 2.1 TestServer instance which adds a HostedService (with an infinite loop) at startup blocks on that thread instead of running it asynchronously.
To Reproduce
Steps to reproduce the behavior:

Using: ASP.NET Core '2.1', XUnit 2.4, and Microsoft.AspNetCore.Mvc.Testing 2.1.3
In service's Startup.cs:
In ConfigureServices(IServiceCollection services):
services.AddHostedService();
In IHostedService.StartAync() <- infinite loop for background work
In Test project, create instance of TestServer using service's IWebHostBuilder and execute tests against service

Expected behavior
Tests should execute, but instead the tests hang as the service never completes starting up.
I suspect that TestServer is awaiting IHostedService.StartAsync. Have a workaround by refactoring IHostedService's StartAsync to return immediately calling another async method to perform work. Example:
//hangs in TestSever instance
public async Task StartAsync(CancellationToken cancellationToken)
{
//infinite loop
}
//works in TestServer instance
public async Task StartAsync(CancellationToken cancellationToken)
{
DoTheWork(cancellationToken);
}
public async Task DoTheWork(CancellationToken cancellationToken)
{
//infinite loop
}
Additional context
Does not effect Kestrel server, only TestServer

Possibly by design. Return the Task or Execute and return Task.CompletedTask

We periodically close 'discussion' issues that have not been updated in a long period of time.
We apologize if this causes any inconvenience. We ask that if you are still encountering an issue, please log a new issue with updated information and we will investigate.


Backend notes on project members (for project owner view only)

Feature request: building in a multiple-notes-field dashboard view for a project owner to be able to track basic information about a participant's data. This will NOT be shown to the members/participants, but will be used for administrative purposes.
I.e. fields could include "Is the data valid/relevant for this project?" "Did they fill out the survey?" "Data was used in Project X" etc.

(Having this will be key for most easily optimizing https://github.com/OpenHumans/open-humans-api/issues/2 once built, to make it easy to only pull down the already-vettted data sets for further analysis).

Shorter term, if the above is complex to build, wondering about an alternative of making it possible to store copies of WhiteList and ExcludeList and OtherList files on the backend with a project in OH? So project admin's don't have to save them locally and/or externally as a way to share them, but can be tied to the project?


More robust code for cleaning up expired games

Description
More robust and defensive code around the background task that cleans up expired games.
Checklist

[x] Add tests for these changes
[ ] Test coverage is not decreased
[ ] Entire test suite passes

Codecov Report

Merging #240 into master will decrease coverage by 0.23%.
The diff coverage is 77.77%.

@@            Coverage Diff             @@
##           master     #240      +/-   ##
==========================================
- Coverage   97.54%   97.30%   -0.24%
==========================================
  Files          37       37
  Lines        2036     2041       +5
  Branches      282      284       +2
==========================================
  Hits         1986     1986
- Misses         26       29       +3
- Partials       24       26       +2

Impacted Files
Coverage Δ

src/spellbot/__init__.py
95.26% <71.42%> (-0.43%)
:arrow_down:

src/spellbot/data.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


[python3] Fix distutils path

Fixes https://github.com/microsoft/vcpkg/issues/30607
Modified the output of the get_python_inc() function. From the original path: installed\triplet\tools\python3\include to now the output path is \installed\triplet\include\python3.10

[X] Changes comply with the maintainer guide
[ ] SHA512s are updated for each updated download
[ ] The "supports" clause reflects platforms that may be fixed by this new version
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[X] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[X] Only one version is added to each modified port's versions file.

No that is not the solution for https://github.com/microsoft/vcpkg/issues/30607. Don't move python3 internals.
Start a shell open up python3 from vcpkg and do:
'import sys, distutils.sysconfig; print(distutils.sysconfig.get_python_inc())'
disutils is a python module. You dont want to move distutils, you want to fix what the output of distutils is so that you get the right answers in regard to file layout of vcpkg.

disutils is a python module. You dont want to move distutils, you want to fix what the output of distutils is so that you get the right answers in regard to file layout of vcpkg.

I think this is doing that now? Can you confirm that you're happy @Neumann-A ?

Can you confirm that you're happy @Neumann-A ?

Output works now as expected. However I don't currently know if more stuff needs to be fixed if environment variables like PYTHONHOME and PYTHONPATH are set (However it currently seems to behave consistent).
Python on windows has the following problems:

DLLs are not found from the prefix/bin folder and setting PATH doesn't do anything more since python decided not to search there any longer for security reason. (You can patch in os.add_dll_searchpath() but this makes the py files basically unmovable in relation to the DLLs.)
There are no rules for ports where the install location of python wrappers for C/C++ libs should be.

As long as those points are not solved advanced usage of python within vcpkg on windows is basically blocked

As long as those points are not solved advanced usage of python within vcpkg on windows is basically blocked

Python continues to be the mother of all edge cases, not the least of which being that it wants to control the whole universe and needs a bunch of prerequisites set up by the end user in order to be used safely. I'm not sure if there is an answer for Python that doesn't involve a ton of special cases for it.
It sounds like this is at least better than before, so I'm going to try to patchify the diff and merge. Thanks for the review!

From what I understand, based on discussion in discord, there are ports that embed Python incorrectly. Embedding Python on unix-like systems tends to "just work" because everything is in a standard location, so the embedder doesn't have to worry about anything. Python on Windows (both the official installer and vcpkg) use a layout that isn't unix-standard, so embedders have to do legwork to make sure that the Python DLLs, standard library, and any required .pyd python modules are appropriately deployed as part of their application's install process. From what I can tell, the port being discussed would be broken even with the official Python installer for Windows - it doesn't set PYTHONHOME or PYTHONPATH either. I guess the question is: do we want to expend the effort to fix something in the upstream project? Maintaining the python3 port is hard enough as is because of PSF's policy of dumping Windows 7 (which, despite the official EOL, many users continue to cling to) and CPython's objectively bad build system. Python 3.11 has been out since October, and I've really only had a chance to see that roughly half of the patches fail to apply 😭

Embedding Python on unix-like systems tends to "just work" because everything is in a standard location, so the embedder doesn't have to worry about anything.

Except not really, because the version that you get from vcpkg and the version on the system do not necessarily match.

From my own adventure with embedding Python, I learned that the Python library itself has to load standard library Python code as part of the library initialization (a bit of a chicken and egg problem, for sure). From what I can tell, the port(s) being discussed would be broken even with the official Python installer for Windows - it doesn't set PYTHONHOME or PYTHONPATH either.

Right, this is kind of what I meant by whatever happens being entirely specific to Python. Nothing else knows or cares about .pyd files, and there doesn't seem to be a standard location for those, so everyone is on the hook for their own mess. If other ports embedding python are doing so 'wrong', its largely fallout from there not really being an obvious 'correct' way of doing so.

Maintaining the python3 port is hard enough as is because of PSF's policy of dumping Windows 7

Do note that we, the vcpkg team, are not terribly interested Win7 support either: it breaks our 'do not implement features in patches' policy. The patch is pretty reasonable now and we have someone who is interested in maintaining it but we are not.

Except not really, because the version that you get from vcpkg and the version on the system do not necessarily match.

I am speaking about the general case, so vcpkg is not relevant to my statement. Otherwise, we seem to be on largely the same page.

its largely fallout from there not really being an obvious 'correct' way of doing so.

CMake provides a way to get the location of the Python standard library in the FindPython module, and .pyd files are just DLLs with a different extension, FWIW. it's not that it's a particularly hard problem, but it does require some understanding on the part of the build system author to deploy everything correctly. There is an assumption that the embedder has read the documentation and provided a vehicle for loading the aforementioned standard library. Sometimes this can be simply by copying the files into a location in the application install directory or packaging it and providing a loader as specified in PEP 451.
While all of this is not immediately obvious, it is not particularly hard to figure out, especially if you have a modicum of familiarity with the Python ecosystem, which I assume anyone embedding Python in their application should.

My two cents:
Python modules on windows (.pyd files) should be stored with other libs in lib and debug/lib AND copied to tools/python3/DLLs with their dependencies.
Python modules on unix should be installed into lib/pythonXYZ/lib-dynload.
Applications that embed python should set PYTHONHOME by calling Py_SetPythonHome before initialization. On unix that would be the folder where lib/pythonXYZ is located, on windows that would be just application path.
Windows users should copy standard library and required .pyd files to the application directory. Stdlib is found in tools/python3 and can optionally be precompiled and zipped.

@Hoikas , @vicroms @JavierMatosD @dan-shaw @ras0219-msft and I discussed the patching situation today.

We agree with Billy's statement above:

If you are asking for rules that bind other contributors, that the vcpkg maintainers enforce for you, you are asking for policy.

I patchified this PR to give you space to make the policy argument. If you are interested in doing that, please create a PR making the corresponding docs changes to our patching instructions: https://github.com/microsoft/vcpkg-docs/blob/c61c79eeffc5f6c3d975af1243ad2d9c388a042f/vcpkg/examples/patching.md?plain=1#L147
In order for us to make that the policy we use, we need a way to get both 'am' and 'editable-ish' workflows to work (as described by @Neumann-A above https://github.com/microsoft/vcpkg/pull/30822#discussion_r1176094723 )

For example, if there is a trivial constant block that we can add to the top of diffs to make git am accept them that would work

@dan-shaw points out that we would be willing to adopt a policy that applies only to Python3 (or other nominal owner of a port) if we can't agree on an overall policy to apply across vcpkg.
@ras0219-msft says that that still means the nominal owner would be on the hook to write such policy even if it applies only to a port.
@JavierMatosD notes that the policy must be consistent with 'global' policies, even in such port 'fiefdoms'.
@BillyONeal notes that policy must include instructions that other contributors can follow, "it must be patches" is insufficient.
======================
Speaking personally, I am weakly against catering to am/format-patch because that workflow triggers updates to unintended to be edited patches, and the workaround in the other direction is not too onerous.


Implementation for multi-gpu usage

Hi,
First of all, Thanks a lot for sharing this package.
I notice that the current package only a single GPU during model training. Is it possible to distribute the training process to multi-gpu, which will accelerate the training process?
Thanks again.
Best,
Yu.

Thanks for your interest. That's definitely possible but requires the code to be modified. This repo won't be updated as it's purpose is to reproduce the paper.


UIAppearance not supported

Component's architecture biased to a design through Interface Builder or directly in code. There is another way of defining visual style of components which is UIAppearance. I know at least several projects where defying style directly in code and in IB is prohibited by architectural conventions. Supporting all standard ways of customization may potentially increase adoption for a component.

@nikita-leonov Thanks for recommendation.
I never tried befre to adopt my own class to UIAppearance protocol.
Seems that, that I need is implement
static func appearance() -> Self
AND
static func appearanceForTraitCollection(_ trait: UITraitCollection) -> Self

I will consider that proposal and try to adopt Gauge class to this protocol. Thanks!

I'd be more than happy to merge a PR adding this feature, it seems like it would be useful for sure 👍


Python IntelliSense not working

Type: Bug
I have VS Code version 1.90.2 running on Windows 10. Recently I noticed that most if not all of my IntelliSense features are not working in my Python files.

Nothing happens when I hover over types (functions, variables, etc.) in Python files
Ctrl + Click on a type does nothing
Ctrl + Right Click results in the same menu as Ctrl + Left Click

I cannot jump to a type definition, view type definitions, etc. because of this.
I have tried/checked:

Restarting VS Code
Disabling all extensions and re-enabling
Making sure Python + Pylance are installed
Making sure workspace is trusted
Making sure hovering is enabled
Making sure IntelliSense is enabled

Extension version: 2024.14.1
VS Code version: Code 1.90.2
OS version: Windows_NT x64 10.0.19045
UPDATE: It appears that the issue may be that latest version of Pylance (v2024.8.2) is not compatible with VS Code 1.90.2

@rjpost20, it's possible that Pylance crashed. Can you please switch to our latest prerelease build and see if the issue goes away? There was a recent crash related to interactive windows. Perhaps you were hitting that.
If prerelease doesn't fix the issue, can you provide your "Python Language Server" log from the Output pane?

I experience this issue too since I upgraded Python extensions and VSCode. IntelliSense did work after upgrade but it stopped recently. I could not found out what triggers this, nor could I restore the IntelliSense. I use VSCode on MacOS (version 1.93.1) and work via remote tunnel on Ubuntu (code version 1.93.1). The python.languageServer is set to Default.
This is screenshot of what I see now:

If IntelliSense works then the imported packages show green.

@JoostHazelzet can you provide your 'Python Language Server' output?

What does your 'Python' output show? And what's your version of Pylance. 2404.14.2 is not a Pylance version number. That's likely the Python extension.

My mistake, Pylance extension is on version v2024.9.2. Here is the Python output:
python.log

Are you sure your Pylance extension is enabled? And that the 'Language server' option for Python is 'Default' or 'Pylance'?
There should be a line like this in the 'Python' log:
2024-09-26 09:02:02.900 [info] Starting Pylance language server.

I noticed the error messages in the python log referring to a former Mamba environment. I decided for a reinstallation of VSCode and recreation of this Mamba environment which solved the issue for me. Pylance is working like a charm again. Thank you for your support.

Ok if you see the issue again, feel free to reactivate this issue.


Change ZeroTier plugin launch

At the moment when calling ZeroTier plugin in custom firmware based on lite the following error occurs.
ZeroTier plugin is not supported on OpenIPC lite.
However this is custom firmware, ZeroTier is there and works.
I propose to make check linking not by lite/ultimate profile but by actual presence of binary file, for example /usr/sbin/zerotier-cli.
Thank you !

done. test it and see if it works for you now.

please reopen if it is still the case.


Improve sidebar and filters

Hide the main menu in sidebar when not in low resolution

Improve visual separation of the route options/filters

Do not open the sidebar automatically in mobile devices when it is not really necessary

start all route options/filters closed. This will also "hide" advanced options by default

Check if the icon/style of the selected profile component underneath the profiles selector can be improved/changed.

Done


Ruby Advanced Interop Server Side

Implements advanced interop test functionality for Ruby server.
closes #8477

Addressed all comments


customMetadata not work in wdio v7

my pr is https://github.com/wswebcreation/multiple-cucumber-html-reporter/pull/151
Environment (please complete the following information):

multiple-cucumber-html-reporter: 1.18.0
Node.js version: 14.16.1
NPM version: 6.14.12
Platform name and version: Mac
Cucucmber version: 7.1.0

Config of multiple-cucumber-html-reporter
onComplete(exitCode, config, capabilities, results) {
  report.generate({
    jsonDir: `${BASE_DIR}/json`,
    reportPath: `${BASE_DIR}/html`,
    openReportInBrowser: true,
    displayReportTime: true,
    displayDuration: true,
    pageFooter: "Zigbang E2E Test",
    customMetadata: true,
    metadata: [
      {name: 'OS', value: `${platformName} ${platformVersion}` },
      {name: 'Device', value: deviceName },
    ]
  })
}

Describe the bug
customMetadata not work in wdio v7
my pr is
To Reproduce
set customMetadata: true
and set metadata like
metadata: [
  {name: 'OS', value: `${platformName} ${platformVersion}` },
  {name: 'Device', value: deviceName },
]

Expected behavior
it works in wdio v7

@wswebcreation wasn't this issue closed by #166 but automation didn't kick in because the usual GH syntax wasn't used on the PR?
I think we can close this. :thinking:

Correct, thanks!


Tweaks to merge_queue.md

Some small styling changes to the file to make it a little bit more legible.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] I followed the breaking change policy and added Data Driven Fixes where supported.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Duplicate, @Piinks is already aware of this!


Plugin URL

Using OpenAI plugin tool, where am I supposed to find the URL for that specific plugin?

Its the plugin url from chatgpt plugins - https://openai.com/blog/chatgpt-plugins
For example: Klarna - https://www.klarna.com/.well-known/ai-plugin.json


[WFLY-17868] Replace uses of ServiceName SocketBinding.JBOSS_BINDING_…

…NAME with use of the capability in RTS subsystem
Issue: https://issues.redhat.com/browse/WFLY-17868

Updated, thanks.

Thanks @soul2zimate, @pferraro and @tadamski


Payment Request Button dosen`t show

In the safari , I can see apple pay but Pixel 3a Chrome does`t see GPay
By the way , The window10 Chrome can see GPay or Pay now

i am facing the same issue for the google pay that we are trying to implement. i can only view the google play button on chrome browsers only .And in apple it does not show up either

I face as well a similar strange behavior where the button does not always shows up. I am using an android phone, i can only see the GPay button when i am in desktop mode of chrome. see pricutre attached. I used a demo site to be sure it is not myu code. Any thing that can be done to avoid this ? it does nto seems to come from the phone. ( i use Chrome version 97.0.4692.87, the last release)
https://stripe-payments-demo.appspot.com/
Chrome in desktop mode:

Chrome in mobile mode:


Fix Channel form

Fixes https://github.com/Sylius/InvoicingPlugin/issues/105

Thank you, Mateusz! :1st_place_medal:


Passes event instance to listeners

Currently the listeners on the annotations are receiving only the context.
I think it could be helpful to get also the event instance, separately, as additional argument.
A use case where to use the event could be to check if any modifier key is pressed in order to perform own logic only when a key is pressed.

Why not set the event in the context? also I'd pass the ChartEvent, because it contains the canvas coordinates and provides the native event inside.

Why not set the event in the context?

I thought that in the near future the context in the plugin will be "persistent" as in chartjs and in datalabels plugin in order to store own data and to get them in further calls.
In this case it couldn't make sense to add it to the context because you could have a property related to an obsolete object (last managed event). Of course, I can change it and add it to the context. Let me know.

also I'd pass the ChartEvent, because it contains the canvas coordinates and provides the native event inside.

Yesterday I was used the ChartEvent but today I recognized that the types were not correct because I set to Event.
Of course, I'll change in next minutes.

I thought that in the near future the context in the plugin will be "persistent" as in chartjs and in datalabels plugin in order to store own data and to get them in further calls.
In this case it couldn't make sense to add it to the context because you could have a property related to an obsolete object (last managed event). Of course, I can change it and add it to the context. Let me know.

Makes sense


[FR] Add support for channel pressure (monophonic aftertouch)

Aka mono(phonic) aftertouch.
Relevant opcodes:

https://sfzformat.com/opcodes/cutoff_chanaft
https://sfzformat.com/opcodes/amplfo_depthchanaft
https://sfzformat.com/opcodes/amplfo_freqchanaft
https://sfzformat.com/opcodes/fillfo_depthchanaft
https://sfzformat.com/opcodes/fillfo_freqchanaft
https://sfzformat.com/opcodes/pitchlfo_depthchanaft
https://sfzformat.com/opcodes/pitchlfo_freqchanaft

(Does SFZ not allow channel aftertouch to be mapped to filter resonance?)
NB: the sfizz.h header currently lists a function ambiguously named sfizz_send_aftertouch. I suggest naming this sfizz_send_channel_pressure or at least adding this as an alias, so there is no confusion that this is about handling channel pressure (aka monophonic aftertouch), not poly pressure (aka poly aftertouch).

Once support is added to the engine, it can be easily supported in the LV2 plugin with:
diff --git a/lv2/sfizz.c b/lv2/sfizz.c
index 87c0923..df2ccf2 100644
--- a/lv2/sfizz.c
+++ b/lv2/sfizz.c
@@ -434,6 +434,14 @@ sfizz_lv2_process_midi_event(sfizz_plugin_t *self, const LV2_Atom_Event *ev)
                       (int)msg[1],
                       msg[2]);
         break;
+    case LV2_MIDI_CHANNEL_PRESSURE:
+        // lv2_log_note(&self->logger,
+        //              "[process_midi] Received channel pressure %d/%d at time %d\n", msg[0], msg[1], ev->time.frames);
+        sfizz_send_aftertouch(self->synth,
+                      (int)ev->time.frames,
+                      (int)MIDI_CHANNEL(msg[0]) + 1,
+                      (int)msg[1]);
+        break;
     default:
         break;
     }

Thanks! For this particular target I still need LFOs in general, which would better be implemented against the SFZv2 spec as it is more general. It's on the todo list, after which adding modulation targets from aftertouch events would be quite easy.
On the other hand I really have no hardware to test this, nor did I ever see it used; you have a good example lying somewhere?

Tons of keyboards support channel pressure. Keyboards, which generate polyphonic aftertouch, though, are very rare beasts.
But mono aftertouch is often used by keyboard players to control vibrato, so they don't have to use their other hand on the modulation wheel for that and can play something else with it. Or to add expression by controlling the filter cutoff or volume.
For testing you can just convert an arbitrary Control Change into Channel Pressure. For example, the following Lua script for the Moony.lv2 plugin converts Control Change #1 (modulation) messages into Channel Pressure messages on the same channel:
-- Convert specific Control Change events to MIDI Channel Pressure (aka Mono Aftertouch)

-- 0..15 or set to -1 to apply to events on all channels
local filter_chan = -1
-- whether to pass non-polyAT events
local pass_other = true
-- lowest note of affected note range
local source_cc = 1

-- NO NEED TO CHANGE ANYTHING BELOW

function cc2channelpressure(self, frames, forge, chan, cc, value)
  if (filter_chan == -1 or chan == filter_chan) then
    if cc == source_cc then
        forge:time(frames):midi(MIDI.ChannelPressure | chan, value)
    else
        forge:time(frames):midi(MIDI.Controller | chan, cc, value)
    end
  end
end

-- define a MIDIResponder object to handle note-on and note-off events
local midiR = MIDIResponder({
  [MIDI.Controller] = cc2channelpressure,
}, pass_other)

function run(n, control, notify, seq, forge)
  -- iterate over incoming events
  for frames, atom in seq:foreach() do
    -- call responder for event
    local handled = midiR(frames, forge, atom)
  end
end

Great thanks, I'll use this to test. I have to say though that it's slightly longer term than some more immediate items I'd like to add :)

It's also much less important than pitch bend :)

I would be interested in seeing Polyphonic Aftertouch as well. Devices such as the Roli Seaboard support it, as do any devices that support MPE. (The list is growing every day).
I understand that this would be a long term goal.

I would be interested in seeing Polyphonic Aftertouch as well. Devices such as the Roli Seaboard support it, as do any devices that support MPE. (The list is growing every day).

Some Roland electronic drum modules (not sure if others) use it for cymbal choking, so it might be useful also for that (even if maybe with a minimal use of it)

Just a correction:
MPE uses monophonic aftertouch, each MIDI channel represents a note. PolyAft is reserved in MPE.

It's not the scope of this issue, but as the hardware question came up: a relatively inexpensive and recent device that emits Poly Aftertouch messages is the Arturia Microfreak.

Some additional opcodes:

https://sfzformat.com/opcodes/hichanaft.html
https://sfzformat.com/opcodes/lochanaft.html

as well as CC129 apparently? https://sfzformat.com/extensions/midi_ccs.html

The extended MIDI CCs seem to add considerable complexity. Tbh, I'm not sure why one would use them or if they ar worth implementing. Obviously they can not be sent via MIDI, so the would just add another way to trigger event handlers via the API, wouldn't they?
Still hoping to see support for at least filter (LFO) and pitch LFO modulation added soon. Is this still on the roadmap for the next release?

The extended CC129 is supposed to be computed according to MIDI, so are most of extended ones listed on the page. The higher CC numbers are supposed to be used by a user interface for implementing custom graphical controls.
The modulations are worked on, but maybe early for a next release.
I think, we had better have at least a next release dedicated to bug fixes, because there were numerous compatibility fixes of existing instruments from the previous release.

#630 will implement the basic support, although not the XXXlfo_XXXchanaft opcodes which require internal links in the modulation matrix. These will readily come when said links are implemented :)

Should all be in now !


running ASpIRE Chain Model

Hi,
first of all thank you a lot for this project!
It would be just awesome to configure this docker for "state-of-the-art" ASpIRE Chain Model.
I'm trying to configure it but so far it's quite confusing for me.
Any tips for that?
The best wishes,
Aleksei

Hi,
if I'm not wrong, Chain Models use NNET3 framework. So, you could use something like this:
You have to download Librispeech "online nnet2" models in order to use this sample
Run download-librispeech-nnet2.sh in 'test/models' to download them.
use-nnet2: True
decoder:
# All the properties nested here correspond to the kaldinnet2onlinedecoder GStreamer plugin properties.
# Use gst-inspect-1.0 ./libgstkaldionline2.so kaldinnet2onlinedecoder to discover the available properties
nnet-mode: 3
use-threaded-decoder: True
model : /path/to/final.mdl
word-syms : /path/to/words.txt
fst : /path/to/HCLG.fst
mfcc-config : /path/to/mfcc.conf
ivector-extraction-config : /path/to/ivector_extractor.conf
max-active: 10000
beam: 15.0
lattice-beam: 8.0
acoustic-scale: 0.06
do-endpointing : false
##endpoint-silence-phones : "1:2:3:4:5:6:7:8:9:10"
traceback-period-in-secs: 1
chunk-length-in-secs: 1
num-nbest: 10
#Additional functionality that you can play with:
#lm-fst:  /opt/models/nnet_ms_online/G.fst
#big-lm-const-arpa: /opt/models/nnet_ms_online/G.carpa
#phone-syms: /opt/models/nnet_ms_online/phones.txt
#word-boundary-file: /opt/models/nnet_ms_online/word_boundary.int
#do-phone-alignment: true
out-dir: /opt/tmp
use-vad: False
silence-timeout: 10
Just a sample post-processor that appends "." to the hypothesis
post-processor: perl -npe 'BEGIN {use IO::Handle; STDOUT->autoflush(1);} s/(.*)/\1/;'
A sample full post processor that add a confidence score to 1-best hyp and deletes other n-best hyps
full-post-processor: /opt/kaldi-gstreamer-server/sample_full_post_processor.py
logging:
version : 1
disable_existing_loggers: False
formatters:
simpleFormater:
format: '%(asctime)s - %(levelname)7s: %(name)10s: %(message)s'
datefmt: '%Y-%m-%d %H:%M:%S'
handlers:
console:
class: logging.StreamHandler
formatter: simpleFormater
level: DEBUG
root:
level: DEBUG
handlers: [console]
Note that you must say use-nnet2: True and nnet-mode: 3.

Just tried this and it works. you have to set frame-subsamping-factor: 3 and acoustic-scale: 1 though.


Add gallium-main to generate the code for main.go

Since there are some special config we have to do in main.go, it might
be better for gallium to manage that file.
This a proof of concept of what generating that file could be like.
Re: #15

@ernesto-jimenez I've been thinking about this quite a bit while I've been away. Overall I think gallium should not use code generation unless there is a really compelling need for it. The main reason is that  when there is code generation involved, a gallium user has to think about how modifications to their code will change what code gets generated, as well as thinking about how the code will behave when executed. I agree it would be nice to cut down the boilerplate, and I agree that the runtime.LockOSThread thing is unfortunate, but ultimately it's just a small number of lines that the user has to write once per application, and I don't think that's a big enough problem to warrant code generation. I think that instead we should provide a good set of example apps for people to start from.

i can see both points.
But i think that @ernesto-jimenez your trying to make a programming framework that works in tandem with Gallium. Devs will be able to use many different frameworks (web, micro etc) with gallium.
Instead good quality examples are a better start. Maybe if we call them "test harnesses" its easir to understand. Each can test a critical aspect.

process control. this really does vary between win, and osx.
installer & updater

a CLI or Scaffolder to layout the code later can be build. to be clear this is different than a code generator.
Very good examples are in Dart actually.
Flutter create
Dart stagehand.
Polymer also has the Polymer CLI
these tools do just enough and nothing more.
Now a code generator example is of course goa.
https://github.com/goadesign/goa/tree/v2-wip
I am using v2, because it support GRPC as well as JSON. Very nice.
This is really what shoudl be used for galium code gen IMHO.


SetPage not working on IOS on v7.0.0 next

Environment
React native pager view v7.0.0 next
React native tab view v3.5.2
React native 0.69.9
React 18.2
Description
SetPage method of pager view is called but it does not change the page when used in conjunction with react native tab view. React native tab views pages contain pager view and on clicking a tab it should change the page but it does not happen. Only on IOS this happens and on android it works fine. Also i noticed that this happens on next version and on other versions the swiping part is disabled for tab view

SetPageWithoutAnimation works only if I use setTimeout with value of 0

Same here, setPage not work but setPageWithoutAnimation works.

Closing as we are moving away from next experimental implementation.


Sentinel support

Are there plans to add Sentinel support for high availability?
Initial implementation in this issue.

Sentinel support would be nice! Can you summarize what a client needs to do to support Sentinel?
We should also have eredis_cluster in mind. Is there anything special to thing about for sentinel + cluster?

Guidelines for Redis clients with support for Redis Sentinel
The client use sentinel nodes to find the master on first connection and reconnection.
Redis cluster and sentinel have different use cases.
Redis cluster provides sharding and high availability. It requires minimum 6 machines - 3 masters and 3 replicas.
Sentinel provides only high availability. It requires 3 sentinel nodes (for quorum), 1 master node and at least 1 replica. It's possible to have 3 machines each with redis node and sentinel node.

Ah, so if you use cluster, you don't need sentinel? Good to know. I thought it's possible to combine both.
Thanks for the link!
We don't have plans to implement sentinel support (because we use cluster) but if you implement it, we will accept a PR.
Regarding how it will affect eredis, the old PR added

To enable sentinel support for eredis app:
Start eredis_sentinel main process under supervisor with list of all sentinels as argument:
eredis_sentinel:start_link([{"host1.lan", 20367}, {"host2.lan", 20367}]).

When starting eredis clients use string sentinel:master_name instead host:
eredis:start_link("sentinel:mymaster", 0).

I think this way of starting eredis seems a bit strange. Can we instead have {sentinel, SentinelOpts} as one of the options to eredis:start_link/1? The eredis_sentinel can be a helper module but we want the public API in the eredis module.
If the sentinel code is expected to be large, we could also consider having it as a separate repo, if that's possible, just like we have eredis_cluster as a separate repo with dependency to eredis. Then we would just add minimum code to eredis so that it can refer to eredis_sentinel.
WDYT?

Yes wen can have {sentinel, SentinelOpts} and eredis_sentinel module.

If the sentinel option is set, eredis_client will need to get the master host from eredis_sentinel:get_host(SentinelOpts) and only try to connect when it get it.
eredis_sentinel will find the master host from one of the sentinel nodes.
eredis_client will remove the host from the state on disconnection and get a new one from eredis_sentinel in case the master changed.

Looks good.


(PE-26509) Add support for pe_compiler role

This adds support for the new 'pe_compiler' role, which is a compile master with PuppetDB, by including the existing compile_master configuration when pe_compiler is specified.
Should be merged with https://github.com/puppetlabs/beaker-pe/pull/144 and https://github.com/puppetlabs/beaker-pe-large-environments/pull/30.

LGTM

pinging @smcelmurry for review/merge


Auto-recognize embedded links and tag them appropriately

Auto-recognize embedded links and tag them appropriately
Resolves #6: Users should be able to embed links(maybe a gist, medium article or youtube video) in their post
Modifications

Replace ReactEmoji.emojify inside the Post component with a custom function.
{addLinks(post.title)}

Define a function to match URLs and tag them appropriately. All text blocks processed by the function will be ReactEmoji.emojifyed to retain the style.const addLinks = (postText) => {
  ...
  if (block.match(urlRegexp)) {
    return (
      <React.Fragment key={i}>

          {ReactEmoji.emojify(block)}

      </React.Fragment>
    );
  } else {
    return (
      <React.Fragment key={i}>
        {ReactEmoji.emojify(block)}
      </React.Fragment>
    );
  }
  ...
};

Notes

ReactEmoji.emojify doesn't return raw text and had to be used separately after the processing of links.
Future removal of the addLinks from the component and storing and importing it from a separate custom-modules or services folder would be suggested to clean-up the component code.

Demonstration
The post was done on the Heroku server and not locally.

please work on the links

@shelcia Pardon? Please clarify what I need to do. Now that there is a merge conflict, I definitely need to resolve that too.

Please resolve the conflicts

Please resolve the conflicts

Done. Please review the update. I have provided all the modification details within the PR.


Astar v0 implementation

We are in intense chat with Astar and Shiden and once we will see support by SubSquid for Contracts pallet, we can kick of this stuff.

[ ] NFT marketplace written ink!

[ ] Orderbook v0
[ ] Royalties
[ ] Swaps
[ ] Make Offers
[ ] Bundles

NFT standards we want to see implemented

[ ] PSP34
[ ] PSP1155

KodaDot compatibility

[ ] Update existing components using ink! contract
[ ] Gallery item - actions within ink! contracts
[ ] Explorer - browsing
[ ] Basic Filters - for sale, price
[ ] Minter - minting with traits

Indexer

[ ] We can anticipate support for WASM indexing end of June from SubSquid
[ ] Support for Factory contracts

Ref

Partially inspired by Reactive NFTs by Koda
https://github.com/AstarNetwork/AstarBounties/issues/6

For reference

vikiival/mimick#2

Releavant for solidity - impl - https://github.com/kodadot/gibanica/issues/2


test x86 runtime performance

Should see if the refactored Matric::operator= improve performance for larger matrices

dev-lazymatrix implements a simpler std::vector internal, some of the benefits found in this branch are:

Better compiler optimization for clang loop vectorize up to 690 GFLOPs for M2
Allows single standard Matrix implementation, some contentions in reverting back to std::array for smaller matrices for $\mathbb{R}^{M \times N} < \mathbb{R}^{256 \times 256}$ are: (a) stack size varies per hardware per type so it is difficult to design for reliable performance and universal types, (b) C++ standard does not impose stack size requirement for std::thread library so it's difficult to conditionally adapt $M,N$ size per hardware (metadata is not available).

Avenues to explore:

[ ] GCC equivalence of loop vectorized directive and associated compiler optimization for SIMD
[ ] See if it is worth it to specialize for NEON/SSE intrinsic for special types


Release profile for OSS Repository Hosting

@chrismattmann - Please review.
Checkout ->
https://oss.sonatype.org/content/groups/public/edu/usc/ir/lucene-geo-gazetteer/0.2/
How to push in Maven Central?

$ mvn clean deploy -P release
Verify on -> https://oss.sonatype.org/content/groups/staging
$ mvn nexus-staging:release
Verify release on -> https://oss.sonatype.org/content/groups/public

Ref JIRA - https://issues.sonatype.org/browse/OSSRH-19541
Ref Guide - http://central.sonatype.org/pages/ossrh-guide.html
How to use l-g-g from maven central?

Download latest lucene-geo-gazetteer-<ver>-jar-with-dependencies.jar  from https://oss.sonatype.org/content/groups/public/edu/usc/ir/lucene-geo-gazetteer/
alias lucene-geo-gazetteer="java -cp /path/to/lucene-geo-gazetteer-<ver>-jar-with-dependencies.jar edu.usc.ir.geo.gazetteer.GeoNameResolver -i /dir/path/to/geoIndex"
lucene-geo-gazetteer -b /path/to/allCountries.txt
lucene-geo-gazetteer -s Pasadena

@smadha looks good to me! 2 things:

remove the version bump out of this, and please update to 0.2 (instead of 0.3-SNAPSHOT). We'll want to tag a 0.2 release (which I can do after you update this PR and I merge it) that way we have a tag that we can map to the released version on Central.

After we complete 1, you need to file a ticket in OSSRH saying we verified it and to push to Central. No wait I'm wrong about that, I just checked: http://repo1.maven.org/maven2/edu/usc/ir/lucene-geo-gazetteer/0.2/ and it's already pushed. Great!

thanks @smadha merged!


Add range from function

@zhangyunhao116 I have merge the most recent upstream changes and squash the commits. It doesn't have any print functions.


Missing breadcrumb when setting the "root-chunk-override" parameter.

After setting this parameter in the original html5 plugin:

I found that the breadcrumb cannot be generated (I have made sure that the parameter "BREADCRUMBS" has been asigned with the value "yes"). Should I make some changes in the breadcrumb template? I would be very grateful if you can give some suggestions.

Generating HTML using
./dita -i ...   --args.breadcrumbs=yes --root-chunk-override=to-content

Doesn't really make sense, as you have stated that you are merging all *.dita files into one HTML, so there is no other HTML page for the breadcrumb to link to - this is why no breadcrumbs are generated.
The XSLT breadcrumb.xsl is iterating through ancestors and displaying the @href if there is one. Also Line 49 is deliberately ignoring chunked content from the extra processing. If you want breadcrumbs for each of your sub-topics, you are going to have to add another <xsl:when> clause for chunks above line 41.
Instead of --root-chunk-override=to-content You could selectively add chunk="to-content" to parts of your map as shown.
<topicref href="parent1.dita">
      <topicref href="child1.dita" chunk="to-content">
        <topicref href="grandchild1.dita"/>
        <topicref href="grandchild2.dita"/>
      </topicref>
</topicref>


Add docs page on visualizing 3D skeletons

A couple of screenshots from a local run of the page:

@thanushipeiris Could you review this for me? Anything I should add?


[doc] Self-reference register in hjson description

Description
In the hjson files in the hw/ip/*/data/ directories, descriptions sometimes refer to other registers, but they may also reference themselves. Should we remove these or shall we support adding the link?
Example:
https://github.com/lowRISC/opentitan/blob/319d32e45b1d72bb98990bdff83653a90410e9a5/hw/ip/spi_host/data/spi_host.hjson#L419

Updated this issue since it is already working for referencing other registers, but not for self references.


Consider adding a .editorconfig file for external contributors

I contributed for the first time some days ago and I had to adjust some settings manually to match some rules defined in the detekt config to be able to reformat automatically the code (e.g. on IntelliJ by default is with CMD + ALT + L).
Obviously ./gradlew check on the CI detects anyway an incorrect format so currently there isn't any risk by a contributor to use a different code style.
The .editorconfig would be useful in two cases:

first time contributors won't need to adjust settings manually to match the detekt basic configs
if one day the code style of this repo changes (e.g. 2 spaces indent), all the contributors will be automatically aligned without changing their settings again

Consider adding a .editorconfig for external contributors so they won't need to adjust the basic settings for this project manually.
e.g. https://github.com/fondesa/kpermissions/blob/master/.editorconfig

I think detekt has a gradle task that updates your IntelliJ idea settings to match the linter.

Oh, I didn't know it, which is it?
I see the following detekt tasks:
detekt
detektBaseline - Creates a detekt baseline on the given --baseline path.
detektGenerateConfig - Generate a detekt configuration file inside your project.
detektIdeaFormat - Uses an external idea installation to format your code.
detektIdeaInspect - Uses an external idea installation to inspect your code.
detektMain - EXPERIMENTAL & SLOW: Run detekt analysis for main classes with type resolution
detektTest - EXPERIMENTAL & SLOW: Run detekt analysis for test classes with type resolution

I tried them all but none of them had the desired result.

Oh, I thought there was a task for this. It may have been replaced.
https://github.com/detekt/detekt-intellij-plugin should allow you to use the detekt config that's already in the repo.
https://github.com/Flank/flank/blob/master/config/detekt.yml

It still doesn't work, but in case, it's something on detekt side, not on Flank ones. Feel free to close this issue. Thanks.


v_5.0/angular-react-vue cannot compile on linux

Executing xcopy is a bad practice and won't work on Linux or Mac.
Consider using https://docs.microsoft.com/en-us/visualstudio/msbuild/copy-task instead.

@OskarZyg the samples using xcopy are meant for Windows Only.
There is a note in ReadMe pointing that out - https://github.com/chromelyapps/demo-projects#readme

Notes
Only the CrossPlatDemo is ready out-of-the-box for all platforms.
All other demos are Windows-ready by default. You can run them, but may require a bit of knowledge on how those frameworks  work.
For Angular/React/Vue using start url - local://dist/index.html it uses xcopy (on Windows to copy the dist folder to bin/exe folder) since it may be different for other platforms, you may have to copy the dist folder manually or replace xcopy functionality.
For Blazor demos,** xcopy** is also used in PostBuildEvent. This may require replacement too (or manual copy) for Linux and MacOS.


Error using 3.8.4-openjdk-17-slim in Dockerfile.jlink

Environment Details

Helidon Version: Helidon 3.0.1
Helidon MP
JDK version: 17.0.4
OS: Mac OS Version 12.6
Docker version (if applicable): 4.12.0.

I am using:
docker build -t quickstart-mp-jri -f Dockerfile.jlink .
But , I am getting it error:
`#13 23.43 Including 17 Java dependencies: java.base, java.datatransfer, java.desktop, java.instrument, java.logging, java.management, java.naming, java.prefs, java.security.sasl, java.sql, java.transaction.xa, java.xml, jdk.attach, jdk.internal.jvmstat, jdk.jdi, jdk.jdwp.agent, jdk.unsupported
#13 23.43 Including debug support: jdk.jdwp.agent
#13 23.43 Creating base image: target/quickstart-mp-jri
#13 27.12 Installing 111 application jars in target/quickstart-mp-jri/app
#13 27.12   checking index in CDI beans archive quickstart-mp.jar
#13 27.14   creating missing index for CDI beans archive jersey-weld2-se-3.0.5.jar
#13 34.02 [INFO] ------------------------------------------------------------------------
#13 34.02 [INFO] BUILD FAILURE
#13 34.02 [INFO] ------------------------------------------------------------------------
#13 34.02 [INFO] Total time:  33.057 s
#13 34.02 [INFO] Finished at: 2022-10-03T06:19:07Z
#13 34.02 [INFO] ------------------------------------------------------------------------
#13 34.02 [ERROR] Failed to execute goal io.helidon.build-tools:helidon-maven-plugin:3.0.0:jlink-image (jlink-image) on project quickstart-mp: Image generation error: io.helidon.build.common.ProcessMonitor$ProcessFailedException: Creating Class Data Sharing archive for quickstart-mp.jar  failed with exit code 134 -> [Help 1]
#13 34.02 [ERROR]
#13 34.02 [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
#13 34.02 [ERROR] Re-run Maven using the -X switch to enable full debug logging.
#13 34.02 [ERROR]
#13 34.02 [ERROR] For more information about the errors and possible solutions, please read the following articles:
#13 34.02 [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
executor failed running [/bin/sh -c mvn -Ddocker.build=true package -Pjlink-image -DskipTests]: exit code: 1`

To work-around the issue: Update Dockerfile.jlink and update the line that builds the jlink image to include -Djlink.image.addClassDataSharingArchive=false. For example:
RUN mvn -Ddocker.build=true package -Pjlink-image -DskipTests -Djlink.image.addClassDataSharingArchive=false

A work-around was implemented in #5158

I have created issue #5209 to track upgrading the base docker image used in our example Dockerfiles. I'm going to close this issue because we have implemented a work-around implemented in #5158


[Snyk] Fix for 24 vulnerabilities

Snyk has created this PR to fix 24 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

package.json
package-lock.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

SQL Injection SNYK-JS-SEQUELIZE-2932027
  401

Forgeable Public/Private Tokens npm:jws:20160726
  272

Authentication Bypass npm:jsonwebtoken:20150331
  266

Regular Expression Denial of Service (ReDoS) SNYK-JS-SEMVER-3247795
  265

Uninitialized Memory Exposure npm:base64url:20180511
  246

Authorization Bypass SNYK-JS-EXPRESSJWT-575022
  241

SQL Injection SNYK-JS-SEQUELIZE-2959225
  222

Use of a Broken or Risky Cryptographic Algorithm SNYK-JS-JSONWEBTOKEN-3180026
  217

Improper Control of Generation of Code ('Code Injection') SNYK-JS-PUGCODEGEN-7086056
  213

Improper Authentication SNYK-JS-JSONWEBTOKEN-3180022
  197

Improper Filtering of Special Elements SNYK-JS-SEQUELIZE-3324088
  197

Prototype Pollution SNYK-JS-JSON5-3182856
  180

Access of Resource Using Incompatible Type ('Type Confusion') SNYK-JS-SEQUELIZE-3324090
  162

Regular Expression Denial of Service (ReDoS) SNYK-JS-MOMENT-2944238
  160

Prototype Pollution SNYK-JS-MOUT-2342654
  160

Prototype Pollution SNYK-JS-DOTTIE-3332763
  159

Improper Restriction of Security Token Assignment SNYK-JS-JSONWEBTOKEN-3180024
  133

Missing Release of Resource after Effective Lifetime SNYK-JS-INFLIGHT-6095116
  131

Inefficient Regular Expression Complexity SNYK-JS-MICROMATCH-6838728
  124

Prototype Pollution SNYK-JS-UNSETVALUE-2400660
  115

Directory Traversal SNYK-JS-MOMENT-2440688
  114

Information Exposure SNYK-JS-SEQUELIZE-3324089
  75

Prototype Pollution SNYK-JS-MINIMIST-2429795
  60

Cross-site Scripting (XSS) SNYK-JS-COOKIE-8163060
  44

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Cross-site Scripting (XSS)
🦉 Prototype Pollution
🦉 Improper Authentication
🦉 More lessons are available in Snyk Learn

:no_entry: Snyk checks have failed. 1 issues have been found so far.

Icon
Severity
Issues

Critical
0

High
1

Medium
0

Low
0

:no_entry: security/snyk check is complete. 1 issues have been found. (View Details)
:white_check_mark: license/snyk check is complete. No issues have been found. (View Details)


chore: go-demo-6 to 1.0.330

chore: Promote go-demo-6 to version 1.0.330

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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


RuntimeError: Java gateway process exited before sending its port number when deploying Pyspark model to Azure Container Instance

I am trying to deploy a PySpark model trained in Azure Databricks with MLflow to an ACI in Azure Machine Learning.
I am following the steps in this link:
https://docs.microsoft.com/en-us/azure/machine-learning/how-to-deploy-mlflow-models#example-notebooks
but I get this error:
SPARK_HOME not set. Skipping PySpark Initialization.
Initializing logger
2022-02-21 09:29:30,269 | root | INFO | Starting up app insights client
logging socket was found. logging is available.
logging socket was found. logging is available.
2022-02-21 09:29:30,270 | root | INFO | Starting up request id generator
2022-02-21 09:29:30,270 | root | INFO | Starting up app insight hooks
2022-02-21 09:29:30,270 | root | INFO | Invoking user's init function
JAVA_HOME is not set
2022-02-21 09:29:31,267 | root | ERROR | User's init function failed
2022-02-21 09:29:31,268 | root | ERROR | Encountered Exception Traceback (most recent call last):
  File "/var/azureml-server/aml_blueprint.py", line 191, in register
    main.init()
  File "/var/azureml-app/execution_script.py", line 15, in init
    model = load_model(model_path)
  File "/azureml-envs/azureml_5d25bdfadca034daea176336163db1e0/lib/python3.8/site-packages/mlflow/pyfunc/__init__.py", line 667, in load_model
    model_impl = importlib.import_module(conf[MAIN])._load_pyfunc(data_path)
  File "/azureml-envs/azureml_5d25bdfadca034daea176336163db1e0/lib/python3.8/site-packages/mlflow/spark.py", line 703, in _load_pyfunc
    pyspark.sql.SparkSession.builder.config("spark.python.worker.reuse", True)
  File "/azureml-envs/azureml_5d25bdfadca034daea176336163db1e0/lib/python3.8/site-packages/pyspark/sql/session.py", line 228, in getOrCreate
    sc = SparkContext.getOrCreate(sparkConf)
  File "/azureml-envs/azureml_5d25bdfadca034daea176336163db1e0/lib/python3.8/site-packages/pyspark/context.py", line 392, in getOrCreate
    SparkContext(conf=conf or SparkConf())
  File "/azureml-envs/azureml_5d25bdfadca034daea176336163db1e0/lib/python3.8/site-packages/pyspark/context.py", line 144, in __init__
    SparkContext._ensure_initialized(self, gateway=gateway, conf=conf)
  File "/azureml-envs/azureml_5d25bdfadca034daea176336163db1e0/lib/python3.8/site-packages/pyspark/context.py", line 339, in _ensure_initialized
    SparkContext._gateway = gateway or launch_gateway(conf)
  File "/azureml-envs/azureml_5d25bdfadca034daea176336163db1e0/lib/python3.8/site-packages/pyspark/java_gateway.py", line 108, in launch_gateway
    raise RuntimeError("Java gateway process exited before sending its port number")
RuntimeError: Java gateway process exited before sending its port number

My code looks like this:
from mlflow.deployments import get_deploy_client

# set the tracking uri as the deployment client
client = get_deploy_client(mlflow.get_tracking_uri())

# set the model path
model_path = "k_means_model"

# define the model path and the name is the service name
# the model gets registered automatically and a name is autogenerated using the "name" parameter below
client.create_deployment(model_uri='runs:/{}/{}'.format(run_id, model_path), name = 'k-means-model-ml-flow')

While my model settings are:
artifact_path: k_means_model
databricks_runtime: 10.3.x-cpu-ml-scala2.12
flavors:
  python_function:
    data: sparkml
    env: conda.yaml
    loader_module: mlflow.spark
    python_version: 3.8.10
  spark:
    model_data: sparkml
    pyspark_version: 3.2.1

Can someone help please?

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Machine Learning:0.5382375,App Services:0.10650265,Azure.Core:0.030425286'

Tried to deploy as a realtime endpoint but failed with same error...

Thanks @toby-htx!
Sorry for the delayed response - I'm tagging this for the ML team to take a look :)

It is now recommended to use v2 endpoints for this use case. Please consider upgrading. https://learn.microsoft.com/en-us/azure/machine-learning/v1/how-to-deploy-azure-container-instance


hidetitle:: property does not seem to work

Info:

OS: MacOS 12.6.1

Logseq ver: 0.8.17

Plugin ver: 1.13.1

Theme name: Flow

[ ] ⚠ Do i have issue with DISABLED my custom.css?
Yes

Description:
Added the property hidetitle:: true to several pages with icons, but the title is still visible.
Screenshots

This seems fixed in version 1.14.6 that I just updated to :)) awesome, thank you!


Infer panel definitions from field types

To address the problem I ran into here, I propose that only panel types that fall into the Structural section here be required in panels/content_panels definitions.  Basic and Chooser-type panels could still be explicitly specified when the user needs to provide class names or specify alternative widgets, but it seems like, otherwise, the panel type could be inferred from the field definition.  I think that not requiring the specification of a panel type would be more DRY, and less typing would mean faster development.  If this feature request is not implemented, then at the very least it would be nice if some kind of warning would be given when FieldPanel is inadvertently specified rather than StreamFieldPanel for a StreamField.
This feature request relates to, and somewhat expands, this request by @kaedroho.

#7684 does away with most specialized field panels.

Yep, agreed.


Actions from new action set does not bind correctly

I cant use the binding UI in the browser because of #154 so I use the UI in the dashboard.
I have created a new action set and binded the actions inside the dashboard UI. Problem is GetActive returns false even when they are binded in the UI and GetStateDown return false when i click the button etc
Bindings in the default action set works

Guys please fix docs, its very hard to guess all features. Found the ActivateSecondary browsing the code

I was running into this problem as well. I wanted to use my own action set that's not the default. Thanks for pointing out the ActivateSecondary function. Spent 2-3 days trying to figure out what was wrong with my bindings. I ended up just adding SteamVR_ActionSet reference and calling ActivateSecondary on it in Start(). Turns out you can also pass the argument to the function to disable the default action set as well.


Can't add extensions from Chrome Web Store

Describe the bug
Can't install extensions from mobile view

To Reproduce
Add button doesn't appear

Expected behavior
I expected add button to appear

Screenshots

Smartphone (please complete the following information):

Device : Nothing Phone 2a

Android Version : 14

Kiwi Browser Version 124.0.6327.2

Additional context
I don't have any another problems for now! :)

I experience the same issue.
Switching to desktop mode forces the button to appear and installation then works fine.
Maybe the fixes implemented here (a contextual menu to force plugin installation) may be worth considering to be more future proof against chrome webstore display changes? https://github.com/NeverDecaf/chromium-web-store/issues/31


Allow cgmath types in constant buffers

Added

Allow cgmath::{Deg, Rad, Matrix2, Matrix3, Matrix4, Point2, Point3} types to be used in constant buffers, disabled by default under the math feature gate.
Pull in cgmath 0.13 as an optional dependency.

Changed

Convert examples from cgmath version 0.7 to 0.13.
Add CI support for Travis and AppVeyor.

CC @kvark

This PR looks great! Very clean and straightforward positive.
Just one thing that I'd like to be changed is the feature name. "math" is ambiguous. We should have either "cgmath", if possible, or something with "cgmath" in it, e.g. "cgmath-types".

@kvark Thanks for the feedback! I have revised the feature name to "cgmath-types", as requested.

Sweet, thank you!
@homu r+

:pushpin: Commit 91d3394 has been approved by kvark

:hourglass: Testing commit 91d3394 with merge 12de9eb...

:sunny: Test successful - status