[Snyk] Upgrade sequelize from 6.37.2 to 6.37.3

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade sequelize from 6.37.2 to 6.37.3.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 21 days ago, on 2024-04-13.

Release notes

Package name: sequelize

6.37.3 - 2024-04-136.37.3 (2024-04-13)
Bug Fixes

postgres: use schema for foreign key constrains of a table (#17099) (6aba382)

6.37.2 - 2024-03-296.37.2 (2024-03-29)
Bug Fixes

add readOnly to the transaction options types and docs (#17226) (7c8972f)

    from sequelize GitHub release notes

Commit messages

Package name: sequelize

6aba382 fix(postgres): use schema for foreign key constrains of a table (#17099)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Let Dependabot do it in the upstream repository


A few minor README changes / gradlew typo

Not draft, ready

This is a restyling of the readme to no longer match the rest of the project, not sure why? https://raw.githubusercontent.com/LuckPerms/wiki/master/Configuration.md as well as the rest of the plugin's markdown uses * instead of - for lists. Redoing the readme to no longer be consistent with that is not a good idea, imo.

ok then


🐛 Bug Report: Gitlab discovery  of multiple nested catalog-info.yaml

📜 Description
I am trying to setup gitlab discovery for this folder structure
|..root
|.....web
|..............docs
|..............catalog-info.yaml
|.....store
|..............docs
|..............catalog-info.yaml

Right now my gitlab discovery section in app-config reads like this
providers:
    gitlab:
      rhdh:
        branch: main
        catalogPath: /**/catalog-info.yaml
        excludeRepos: []
        host: mygithub.com
        projectPattern: '[\s\S]*'
        schedule:
          frequency:
            minutes: 2
          initialDelay:
            seconds: 15
          timeout:
            minutes: 3
        skipForkedRepos: true

👍 Expected behavior
The embedded catalog-info.yamls should also be auto-discovered - with multiple components being discovered
👎 Actual Behavior with Screenshots
With catalogPath: /**/catalog-info.yaml the system  could discover components which have catalog-info.yaml in the root folder.
👟 Reproduction steps

Create a repo with the above described folder structure
Update the app-config.yaml as described above with wildcarded catalogpath
Only components which have catalog-info.yaml in the root folder are discovered and appear in the Components UI

📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
No, but I'm happy to collaborate on a PR with someone else

Thanks for the report. Just to clarify, does this pick up the root catalog-info file but non of the deeper ones?

Yes,
This picks up the root catalog-info file but non of the deeper ones. I see another
Seems similar to this one here
https://github.com/backstage/backstage/issues/15810

For GitLab discovery the configuration key for the path is entityFilename rather than catalogPath, I believe that one is hooked up to allow for wildcard discovery too. Happy to align the naming here btw if you'd like to submit a PR to make it possible to configure this via catalogPath instead.

I ran into this issue and glad to see it's a recent topic for others so it's not just me...
Using the providers.gitlab.provider-name-here.entityFilename set to catalog-info.yaml, it finds one result (coinciding with the providers.gitlab.provider-name-here.projectPattern and providers.gitlab.provider-name-here.groupPattern).
When I switch providers.gitlab.provider-name-here.entityFilename set to *.yaml it finds 0 results.
When I switch providers.gitlab.provider-name-here.entityFilename set to **/*.yaml it finds 0 results.
I looked at the code where this ends up being used and it seems to be making a LocationSpec which assuming is the same target type as here, then wildcards are not supported.

Saw this other issue and maybe is an alternative to use url locations with globs? https://github.com/backstage/backstage/issues/26802#issuecomment-2379093630

Hmm, that should work. Might be that there's a bug somewhere in the search implementation 🤔
https://github.com/backstage/backstage/blob/01fa56bea81dbc3a0a594195c09d3ecfe993ff9c/packages/backend-defaults/src/entrypoints/urlReader/lib/GitlabUrlReader.ts#L242-L264


Doc/update

add PyTwinLogLevel documentation
Update ansys_dt.png
add download_file and load_data functions documentation to the published one

Codecov Report

Merging #62 (002ffc4) into main (f3aa544) will increase coverage by 0.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #62      +/-   ##
==========================================
+ Coverage   86.58%   86.71%   +0.12%
==========================================
  Files           9        9
  Lines        1774     1776       +2
==========================================
+ Hits         1536     1540       +4
+ Misses        238      236       -2

@chrpetre we have an issue with the link added to the API doc. See below:

Doc creation log says:

I'll try to fix it and keep you posted.

@lboucin I have uploaded the missing files - rerunning all the workflows, thanks!


[stable/prometheus-operator] additionalPrometheusRules conflicts with additionalServiceMonitors

if we provide both additionalPrometheusRules as well as additionalServiceMonitors the helm merging of the templates does not work correctly, as it is trying to merge a List object. This now results in the items in the prometheus.additionalServiceMonitors list being replaced by the items in the additionalPrometheusRules list.
Now, this could be considered a Helm bug and I will create an issue there as well, but for now I propose to explicitly create templates/prometheus/list.yaml and iterate over both sections there, removing templates/prometheus/additionalPrometheusRules.yaml and templates/prometheus/additionalPrometheusRules.yaml as these are the only two using kind: List

I don't think this is a helm bug actually, and this chart is making a mistake by using two template files to create the same Kind: List resource. My proposed solution is a necessary fix imo

still an issue

bump

+1


string/numeric mixup

Reported by ansley b manke on 27 Jun 2017 22:20 UTC
See #2234 for a previous example.  Hella Reide reported this. Here is a Ferret session running her example script:
!! note, without the 'can var/all' before a new test, the behavior changes
!! as listings influence the behavior of subsequent listings

message "problem with masked strings -> unexpected values instead of strings:"
problem with masked strings -> unexpected values instead of strings:
  Hit Carriage Return to continue

can var/all
let test = {1,1,0}
let strings = {"a","b","c"}
list if (test) then strings
             VARIABLE : IF (TEST) THEN STRINGS
             SUBSET   : 3 points (X)
 1   / 1:  9.145-316
 2   / 2:  9.145-316
 3   / 3:  9.145-316
message "(repeat the listing, and the values change every time)"
(repeat the listing, and the values change every time)
  Hit Carriage Return to continue

message "works: define a masked variable instead, then list"
works: define a masked variable instead, then list
  Hit Carriage Return to continue

let masked = if (test) then strings
list masked
             VARIABLE : IF (TEST) THEN STRINGS
             SUBSET   : 3 points (X)
 1   / 1:"a"
 2   / 2:"b"
 3   / 3:""

message "alternatively, do the same listing for a float first"
alternatively, do the same listing for a float first
  Hit Carriage Return to continue

let floats = {2,3,4}
list if (test) then floats
             VARIABLE : IF (TEST) THEN FLOATS
             SUBSET   : 3 points (X)
 1   / 1:  2.000
 2   / 2:  3.000
 3   / 3:   ....

message "now, also for strings listing while masking works"
now, also for strings listing while masking works
  Hit Carriage Return to continue

list if (test) then strings
             VARIABLE : IF (TEST) THEN STRINGS
             SUBSET   : 3 points (X)
 1   / 1:"a"
 2   / 2:"b"
 3   / 3:""

message "the next problem comes with xsequence:"
the next problem comes with xsequence:
  Hit Carriage Return to continue

can var/all
let test = {1,1,0}
let strings = {"a","b","c"}
let masked = if (test) then strings
list xsequence(masked)
             VARIABLE : XSEQUENCE(MASKED)
             SUBSET   : 3 points (X)
 1   / 1:  9.145-316
 2   / 2:  9.145-316
 3   / 3:  9.145-316
message "again unexpected numerical values instead of strings"
again unexpected numerical values instead of strings
  Hit Carriage Return to continue

message "but doing a 'faulty' listing first fixes it:"
but doing a 'faulty' listing first fixes it:
  Hit Carriage Return to continue

can var/all
let test = {1,1,0}
let strings = {"a","b","c"}
list if (test) then strings
             VARIABLE : IF (TEST) THEN STRINGS
             SUBSET   : 3 points (X)
 1   / 1:  9.145-316
 2   / 2:  9.145-316
 3   / 3:  9.145-316
let masked = if (test) then strings
list xsequence(masked)
             VARIABLE : XSEQUENCE(MASKED)
             SUBSET   : 3 points (X)
 1   / 1:"a"
 2   / 2:"b"
 3   / 3:""

message "NOT fixed by defining a second masked variable before:"
NOT fixed by defining a second masked variable before:
  Hit Carriage Return to continue

can var/all
let test = {1,1,0}
let strings = {"a","b","c"}
let masked = if (test) then strings
let masked2 = xsequence(masked)
list masked2
             VARIABLE : XSEQUENCE(MASKED)
             SUBSET   : 3 points (X)
 1   / 1:  9.145-316
 2   / 2:  9.145-316
 3   / 3:  9.145-316

Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2547

Comment by ansley.b.manke on 28 Jun 2017 22:23 UTC
This is fixed in is_uvar_grid.F and a new benchmark test added.  The code was setting the result of the expression to the data-type of the mask instead of the string variable in expressions like
IF mask THEN string


No pose is detected when executing on CPU using Rpi3

Description
Hi,
I am trying to run posenet project on aarch64 device (raspberry pi 3) on CPU. I am using gstreamer. But on running the pose_camera.py script, no pose is detected. No line drawings can be seen as well. I have a USB camera connected as /dev/video0. Condifence scores for keypoints remains at 0;
When I switch over to using opencv, I get accurate pose detections with non-zero confidence scores.
My input command is:
$python3 pose_camera.py --model model.tflite --videosrc /dev/video0
my model.tflite is downloaded from https://raw.githubusercontent.com/google-coral/test_data/master/posenet/posenet_mobilenet_v1_075_353_481_16_quant_decoder.tflite
On running, I see the video on the screen, with live fps and NPoses=1, but i don't see any actual pose detected using the drawn lines.
If i use it using opencv, i get pose outputs, but not using gstreamer. I even installed the posenet_decoder.so
How do I run this project on CPU using the CPU based models?
As a follow- up, how can I port it to run on movenets, using gstreamer? I can run movenet using opencv, but would like to use gstreamer without opencv.
thanks
Click to expand!
Issue Type
Support
Operating System
Ubuntu
Coral Device
Dev Board
Other Devices
Raspberry Pi 3
Programming Language
Python 3.9
Relevant Log Output
No response

I get the same issue when i try to run with a movenet model sourced from tfhub: https://tfhub.dev/google/lite-model/movenet/singlepose/lightning/tflite/int8/4
I am using movenet lightning int8


Dbt support for metrics

Hi,
DBT has now pre-released metrics in their main branch. Will this library start using that standard?
https://github.com/dbt-labs/dbt-core/issues/4071
Cheers!

Hey @AndreasTA-AW ! Absolutely - this is supported in Lightdash now and I'll port this over shortly


TypeError: Cannot read property '_name' of undefined

I tried installing this per the readme, and then put the "quick start" javascript in my javascript file, and I get the following error on the server:
W20150506-11:44:08.203(-5)? (STDERR) /Users/x/.meteor/packages/meteor-tool/.<IP_ADDRESS>wysac9++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/server-lib/node_modules/fibers/future.js:245
W20150506-11:44:08.203(-5)? (STDERR) 						throw(ex);
W20150506-11:44:08.203(-5)? (STDERR) 						      ^
W20150506-11:44:08.227(-5)? (STDERR) TypeError: Cannot read property 'name' of undefined
W20150506-11:44:08.227(-5)? (STDERR)     at Restivus.Restivus.addCollection (packages/nimble:restivus/lib/restivus.coffee:96:38)
W20150506-11:44:08.227(-5)? (STDERR)     at app/restivus.js:17:12
W20150506-11:44:08.227(-5)? (STDERR)     at app/restivus.js:72:3
W20150506-11:44:08.227(-5)? (STDERR)     at /Users/x/NetBeansProjects/restivusDemo/.meteor/local/build/programs/server/boot.js:222:10
W20150506-11:44:08.227(-5)? (STDERR)     at Array.forEach (native)
W20150506-11:44:08.227(-5)? (STDERR)     at Function..each._.forEach (/Users/x/.meteor/packages/meteor-tool/.<IP_ADDRESS>wysac9++os.osx.x86_64+web.browser+web.cordova/mt-os.osx.x86_64/dev_bundle/server-lib/node_modules/underscore/underscore.js:79:11)
W20150506-11:44:08.227(-5)? (STDERR)     at /Users/x/NetBeansProjects/restivusDemo/.meteor/local/build/programs/server/boot.js:117:5

Hi there! Thanks for reporting this issue! I did notice a small omission in the quick start example, but it doesn't appear to be the cause of this issue. The error you're getting is caused by passing a null or undefined object as the first parameter to Restivus.addCollection(), which would only happen in the quick start example if you didn't copy in the first line that initializes the collection. Can I see a complete copy of your code?
The omission I detected was that the Posts collection I use in the custom route is never initialized in the example. I'm surprised nobody has yelled at me for that.
I'll go ahead and fix that omission in the Quick Start example, and also handle that error better so that it's clear what's happening there.

oh... ok, I see it's passing Meteor.users... that must be what is undefined in my test project... do I need a certain package for this to be defined properly?  or do I need to be doing authentication / login?

I'm sorry about that. Sometimes I forget that the accounts package is not part of core. I'll add a note for that into the quick start example (or more likely, just use another custom collection instead).
You are correct, though. You would need accounts-password or any of the other accounts packages to access the Meteor.users collection.

No worries!... I put together a very bare bones example at https://github.com/wesyah234/restivusDemo which doesn't use the accounts packages... let me know if you have any feedback.  Other than this small issue, your package has been very easy to get started with.


Package renaming?

The topic was touched by @Azzaare in https://www.youtube.com/watch?v=G4siuvNMj0c (nice talk, by the way!).
I'm not sure that the contents of this package will be merged soon into MOI, a better name (to indicate closeness to MOI, at least) would make finding this package easier for users. For now, it looks a lot like a name for an experiment (which it was in the beginning). Also, it would be good to have a shortcut for the package (Jean-François proposed CPE). Currently, I'm using a lot either CP or COIx for subpackages (to mimick what MOI does).
Maybe something like ConstrOptInterface? It looks a lot like MOI (which is intended) and is roughly pronunciable. It could be shortened to COI (in French, though, this is "quiet; speechless"; I just hope it's not too vulgar in another language).
Or is the current name completely acceptable for the foreseeable future?
@odow @Azzaare @Wikunia

I think the name is good. There isn't a strong reason to change.

at least) would make finding this package easier for users.

Please make PRs to the JuMP, MOI, or jump-dev.github.io repos to advertise!

I think the name is fine at the moment. I am more worried about JuCP as it goes against the naming convention and is not sounding as nice as JuMP.
Furthermore, I suppose that is not that important until a possible registration of the package anyway.

Rename JuCP to ConstraintProgramming.jl?

I'm not really a fan of the name JuCP either. If we go like @odow proposes, maybe ConstraintProgramming.jl for the user-facing package and ConstraintProgrammingInterface.jl for the solver-facing part?

Even though it was mainly about the name of the organization, we had a thread about naming with @wikunia in discourse: https://discourse.julialang.org/t/constraint-programming-organization-search-for-a-name/51923
Maybe it can give you some ideas.
That aside, I like ConstraintProgrammingInterface.jl as it reminds me of MOI (and CPI doesn't sound bad as shortened name).
For the final name, ConstraintProgramming.jl is nice enough (it is not lying about the content!). We can also consider option that could be funnier (cf the post in discourse) or that refer to JuMP in other ways. I think @wikunia and I thought about some funny names at some point such as Leap.jl (so another kind of jump)

If I had to make a choice, I'd rather go for a no-frills name for the interface package, and something more inventive/fun for the modelling part. In a sense, it's sad that using Ju- or Julia- is discouraged for packages, because I liked JuCoP a lot (it also looks and sounds very much like JaCoP).
Shall we have a poll for naming CPE and JuCP? (One of the option would always be to keep the current names.) We've had a few options here and others from Discourse.


Volume-based QC check after segmentation

Description
This adds the existing QC check from quick_qc.py in run_prediction.py to evaluate segmentations, particularly to halt processing of cases that fail the test.
If a single subject is processed, the inference script exits with an error if the segmentation fails the QC check. This is propagated to run_fastsurfer.sh, which will also fail before running recon-surf (unless surf_only).
If multiple subjects are processed, only a warning message is logged for every failing subject, in addition to a count of failures  at the end.
The list of failed subjects is maintained. This can be used to write a qc.txt for every subject, indicating that this case failed this QC check (and possibly others to be added in the future), or alternatively a list of all failed cases.
An open question is where qc.txt could be located.
(@m-reuter)

I discussed with @LeHenschel that probably we should create directories for the outputs and write the logfile there.

We could:

If batch processing (a Freesurfer directory structure is assumed), add $SUBJECT_DIR/scripts/qc.txt.
If single processing (a Freesurfer directory structure is not guaranteed), create $SEG_PATH/scripts/qc.txt OR always output a pre-defined directory structure (for e.g., Freesurfer's) in run_prediction.py and save to a pre-defined location ($SUBJECT_DIR/scripts/qc.txt).

In ether case, the sub-directory could be something other than scripts/.

If I understood it correctly, running single input stops the process anyway, and there is no output written, if the qc-check passes, right? Do we need a separate log-file in this case? We could add the output to the log-file that is anyway generated if the user specifies it or print it to stdout.
For multi-subject, we anyhow have the scripts directory and can either add a log-file there or again add the output to recon-surf/deep-seg.log.

If I understood it correctly, running single input stops the process anyway, and there is no output written, if the qc-check passes, right?

At the moment, for single input, if the seg fails the QC check, the process is not stopped before saving outputs; it just returns an error code once it's done and logs a warning message.

For this I would honestly create a log-file in a user specified location and just return the list with all failed subjects. This seems more useful than having a qc.txt in each failed case which just contains "failed"? Unless there is some more subject-specific information explaining what failed.

I think the idea is that qc.txt will show failed/success for a list of future qc checks.

Implementation looks ok to me. Do you have test cases somewhere to run the code on?

There are some test cases in /groups/ag-reuter/projects/ahmed/test_set_la5c_adni.

i think the multi-processing might benefit from writing the failed cases to a list directly instead of just appending it to a list.

Noted. It would be better to write directly to file for a subject-list-style qc.txt.

Ok, so the current suggestion is:

for single processing: no log file.
for batch processing: i) if we want to always create a file listing QC results from now on, a log file in each $SUBJECT/scripts/; ii) if we don't want that, a single log file just listing failed subjects (location specified through an argument).

Implemented as described above, with a single, optional failed subject-list file in the batch processing case.


[FLINK-21728][core] Ensure cleaner is called at most once when unsafe…

… memory segment is freed from multiple threads.

What is the purpose of the change
(For example: This pull request makes task deployment go through the blob server, rather than through RPC. That way we avoid re-transferring them on each deployment (during recovery).)
Brief change log
(for example:)

The TaskInfo is stored in the blob store on job creation time as a persistent artifact
Deployments RPC transmits only the blob storage reference
TaskManagers retrieve the TaskInfo from the blob cache

Verifying this change
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (100MB)
Extended integration test for recovery after master (JobManager) failure
Added test that validates that TaskInfo is transferred only once across recoveries
Manually verified the change by running a 4 node cluser with 2 JobManagers and 4 TaskManagers, a stateful streaming program, and killing one JobManager and two TaskManagers during the execution, verifying that recovery happens correctly.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Abandon the PR.

FLINK-21728 will be fixed by removing multiple segment frees.
Will open another PR for guarding against concurrent segment frees. FLINK-21800


Adding Entrust G2 CA Cert

Adding Entrust G2 CA Cert to cacerts.pem

I could be mistaken but I'm rather certain that we don't accept additions to the certificate bundle directly. They have to go through certifi first. @Lukasa am I right?

We do not accept changes to the cert bundle in requests in this manner: certainly not one certificate at a time.
Would you like to open an issue that discusses the actual problem you have encountered?

@Lukasa @sigmavirus24 Thanks for clarifying how this works. I'll open an issue. We use the requests library as part of our project and are missing a certain certificate -- this certificate is present in certifi; however, it hasn't been added to requests. https://github.com/certifi/python-certifi/blob/master/certifi/cacert.pem#L5026

To clarify, Certifi was not a drop-in solution for us as it was missing at least one cert that's bundled with Requests.


add default label formats for week

Issue Number
#562

Nice fix @dkarnutsch, is there any value in adding some tests so that we can ensure this crash won't happen again?


How to implement a broadcast server?

gnet version: v2.0.0
My use case is to build a broadcast server. The server will poll or listen from a data source, which is sending a stream of messages. I would like broadcast the messages to all connected clients with the lowest possible latency.
I imagine I can use c.AsyncWrite(), but I'm not sure if that's the best way of doing this because it is not running on the event loop. Another way I can think of is to add a custom broadcast event to the event loop, in that case I can just use c.Write(), but it doesn't seem like I can add a user defined custom event to the gnet event loop.
How would you implement such a broadcast server?

c.AsyncWrite() is thread-safe, so you can store your connections out of the event-loop and call it when you need it.

thanks!


[WIP] DO NOT MERGE! Added IsDebug to CakeRuntime.

Resolves #902

@mholo65 just wanted to touch base on this one again...
Where are we with it?  Thanks!

@gep13 as mentioned above, this is a "chiclen-egg" issue with some registrations in Autofac. Need to think of an solution which doesn't require major refactoring.

@patriksvensson
It's not possible to inject ICakeArguments to CakeRuntime... For that to work, we'll need to register the ArgumentsModule prior to CoreModule. Don't even know if that's possible, haven't dug that deep..
Should we resort to implementation as  presented here

@patriksvensson ok! I'll fix it. In the meantime, you might want to move #902 and #876 to v0.18.0 milestone

@mholo65 said...
you might want to move #902 and #876 to v0.18.0 milestone

I have cleared those issues from having a milestone assigned.  We can assign them once we know when they are going to be ready.

@patriksvensson still want this and #965?
I could rebase and fix, otherwise close.

Closing. No need for this.


Add ssHattrick

Add ssHattrick, another terminal game :) to be played over ssh. You can just try it at ssh frittura.org -p 2020.

No freaking way!


Add linalg.multi_dot API.

Hi, Team!
The PR aims to add cupy.linalg.multi_dot API. It follows #6078.
I would love to learn from your viewpoints.
~TODO: Add tests.~
Thanks!

Closing, feel free to reopen


Image card

Place image on the wall. Resizable or limit to card height/width.

https://github.com/niventc/wally/commit/44c8071a74a1fed8938374d5b416a423b7cae41e
Full size for now, will raise separate issue for resizing


HMA for Multi Tenant Hybrid

Made changes in the doc for multi tenant Hybrid support.
-Mukesh (alias: mukkum)
-PM Hybrid & Exchange Server

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks

@kelleyvice-msft Please merge this PR> Thanks


class App(ttk.Frame) Error

If using the class app(ttk.Frame) method to generate a window application, using sf = ScrollFrame(self) in this class will result in an error. It needs to be changed to class app(tk.Frame) to work properly. Is it possible to make it compatible?

use this package is better
https://github.com/Akascape/CTkXYFrame
you need to install customtkinter

.


[Request] Compression option

Basically 04a1d3710188c6219c18b112113fc201b14af4fc but as a configurable option. Disabling compression can improve performance, such as reducing stuttering issues.

c1be413bc7f57fa472c797caaff90ee11122b0c2


cmd/compile: improve PPC64.rules for loads and stores as found in encoding/binary ByteOrder interface functions

Use PPC64.rules to improve performance of statements that do multiple single byte loads or stores on consecutive bytes, such as is found in the encoding/binary ByteOrder interface. Instead do a single multi-byte load or store where possible, such as lhz, lwz, ld, sth, stw, std.
This improves functions that are found in encoding/binary:
Uint16([]byte) uint16
Uint32([]byte) uint32
Uint64([]byte) uint64
PutUint16([]byte, uint16)
PutUint32([]byte, uint32)
PutUint64([]byte, uint64)
The initial change for this will handle little endian cases.

Marking release-blocker (because it has a CL) and @randall77 as owner for review.


Deploying runs-on into preexisting VPC

We have a quite specific networking setup in AWS with a lot of peering and Transit Gateway connections. And obviously, we would want runs-on runners to be able to access some of the internal resources using that networking setup. Problem is that since runs-on spins up it's own VPC we have to set up separate connections and routes instead of reusing the already existing ones. Would you be open to allow deployments into preexisting VPC? Or maybe, to widen this discussion a little bit, provide other deployment options so we could deploy just the minimal required stack  (AppRuner service + S3 bucket + whatever else) and supply things like VPC ID and subnets.

@yafanasiev I understand the concern, but 've always been against allowing this, because this would trigger a lot of support work on my side to make sure both configs work, and also answer support issues where the underlying cause might be an incorrect networking configuration. It also means a lot of integrated features (e.g. magic caching) could be lost / not working when using an existing VPC configuration.
I understand it is a bit painful to set up yet another route for the RunsOn VPC, but there are actually multiple clients doing this, and I believe it can be automated fairly easily based on the Cloudformation stack outputs (if not, let me know why).
Also AWS just released a new feature, which may or may not be useful in your case: https://aws.amazon.com/about-aws/whats-new/2024/12/access-vpc-resources-aws-privatelink/


Feature Request: Colored Areas

Thanks for this great library, exactly what I was looking for.
I do, however, have a feature request when you feel you have the time for it. :-)
I have an image map where I would like to highlight some areas, even when they are not hovered. For example I have a few places I would like to have a yellow polygon and somewhere some reds.
If it was possible to specify a `className', color etc. from the map options object and wether or not the area should be always be shown - that would be cool!
All the best!

I like that idea, I wanted something similar. When you click in an area, that area stays "hovered". I tried adding a div on top of the image, and it works fine for the top image elements. But the image is inside another div and is scrollable, so just adding another div on top of it wont follow the scroll. @janhartmann if your image isn't scrollable, you could add divs based on coordinates on the "onLoad" function.

Hi again
I was in a bit of a hurry of this feature, that I actually forked your repository and make the changes my self. I am a TypeScript user, so I also converted the code base to TypeScript and make some minor modifications to the original source of yours.
I have added a playground here with my changes - if you need some inspiration! :)
https://codesandbox.io/s/zzxm6mzkzx

I was also able to do what I needed, but since I haven't the knowledge to change the actual code base, I feel like mine is more like a hack.
Here's what I did:

To the ImageMapper component I am also sending the index of the area like:
onClick={(area, index) => this.clicked(area, index)}

At the onClick function I added:

    width = area.coords[2] - area.coords[0] + "px"
    height = area.coords[3] - area.coords[1] + "px"
    marginTop = area.coords[1] + "px"
    marginLeft = area.coords[0] + "px"

    // remove class from previous clicked and add to current clicked
    $('[name="my-map"]').find("area").removeClass( "elementClicked" );
    $('[name="my-map"]').find("area").removeAttr( "style" );
    $('[name="my-map"]').find("area").eq(index).addClass( "elementClicked" );

   // Change the class to be where the area is
    $(".elementClicked").css('width', width)
    $(".elementClicked").css('height', height)
    $(".elementClicked").css('margin-top', marginTop)
    $(".elementClicked").css('margin-left', marginLeft)```

I know is definitely the best way, but is working for now.

I'll take a look at the pull request soon, in the following days.
I apologize for having no time to keep this package up to date lately.

#20 merged


Doc on FSF wiki looks ancient - FYI (not a bug)

Not a bug for here - but as it is the link that cam up first when I looked for libpsl (for use with curl) - I thought I'd mention that the page https://directory.fsf.org/wiki/Libpsl mentions a very old version of libpsl.
Fortunately it also has a link to this github project - however - maybe the page should refer people here - rather than to an ancient debian distribution.
FYI
Feel free to close at your convenience.

I have an account but fail to login.  Have to close this before I get into a rage 😆


SAF docker not able to convert tensorflow model

i tried to follow the openvino instruction from https://software.intel.com/en-us/articles/OpenVINO-Using-TensorFlow#inpage-nav-8 to convert the model with SAF docker but stuck as follow:
. /opt/intel/computer_vision_sdk/deployment_tools/model_optimizer/install_prerequisites/../venv/bin/activate
(venv) # mo_tf.py --input_model=./ssd_inception_v2_coco_2018_01_28/frozen_inference_graph.pb --tensorflow_use_custom_operations_config ./extensions/front/tf/ssd_v2_support.json --tensorflow_object_detection_api_pipeline_config ./ssd_inception_v2_coco_2018_01_28/pipeline.config  --reverse_input_channels
usage: mo_tf.py [-h] [--input_model INPUT_MODEL] [--model_name MODEL_NAME]
[--output_dir OUTPUT_DIR] [--input_shape INPUT_SHAPE]
[--scale SCALE] [--reverse_input_channels]
[--log_level {CRITICAL,ERROR,WARN,WARNING,INFO,DEBUG,NOTSET}]
[--input INPUT] [--output OUTPUT] [--mean_values MEAN_VALUES]
[--scale_values SCALE_VALUES]
[--data_type {FP16,FP32,half,float}] [--disable_fusing]
[--disable_gfusing] [--extensions EXTENSIONS] [--batch BATCH]
[--version] [--input_model_is_text]
[--offload_unsupported_operations_to_tf]
[--tensorflow_subgraph_patterns TENSORFLOW_SUBGRAPH_PATTERNS]
[--tensorflow_operation_patterns TENSORFLOW_OPERATION_PATTERNS]
[--tensorflow_custom_operations_config_update TENSORFLOW_CUSTOM_OPERATIONS_CONFIG_UPDATE]
[--tensorflow_use_custom_operations_config TENSORFLOW_USE_CUSTOM_OPERATIONS_CONFIG]
[--tensorboard_logdir TENSORBOARD_LOGDIR]
mo_tf.py: error: unrecognized arguments: --tensorflow_object_detection_api_pipeline_config ./ssd_inception_v2_coco_2018_01_28/pipeline.config
should we discuss this issue here or Intel computer vision sdk forum?

This looks like API mismatch. SAF Dockerfile uses OpenVINO version 2018.1.265 which is probably different from the version the instructions are assuming. Please install the latest OpenVINO on a host and follow the instructions. At the same time, please send me the models you'd like to convert over email and I will find you the help you need from Intel. This issue is irrelevant to SAF. Please close.

This looks like API mismatch. SAF Dockerfile uses OpenVINO version 2018.1.265 which is probably different from the version the instructions are assuming. Please install the latest OpenVINO on a host and follow the instructions. At the same time, please send me the models you'd like to convert over email and I will find you the help you need from Intel. This issue is irrelevant to SAF. Please close.

Hello, I encountered the similar problem mentioned above when using MobileNetV2+ssdlite which was transformed from openvino to self-training. Could you please send me your email for further communication? My email is<EMAIL_ADDRESS>Thanks so much!!!!


Rule: Print the -s option

ocaml-mdx rule accept the -s option but it does not print it back.
That means it will generate rules that test the whole file instead of the section the rule was generated for.
As a test, I generate the rule for section.md.

This is very related to #177; perhaps we need a better long-term solution.

Also, as mentioned in #163, I feel that testing ocaml-mdx rule only via tests that do other things is not great: I find overloading tests makes them hard to reason about. #178 would be a step towards writing explicit tests of this behaviour.

This is very related to #177; perhaps we need a better long-term solution.

Do you suggest adding a mechanism to pass option directly to ocaml-mdx test ? This is what is already done with prelude, prelude-str, direction. What is the problem with that interface ?
The -s option have to be handled by both rule and test.

I know -s and --syntax are different flags; it just so happens that rule currently supports neither of them when test does.
I think the current state of some options to rule being used to customise the rule generation and some options being implicitly passed through to ocaml-mdx test in the generated rule is confusing and doesn't scale well (what would happen if we added an -s flag to ocaml-mdx rule for instance?). In #177 I suggest having a single option for ocaml-mdx rule whose sole responsibility is to pass options to the internal ocaml-mdx test call.

I agree that not all of the options can be directly passed through to the ocaml-mdx test case: in some cases you really want to change the way rule behaves. However, I think:

if the option doesn't change the structure of the generated rule then it should be exposed via a passthrough option because (a) we don't want to have to extend rule to account for changes to test and (b) it's easier to reason about / makes for cleaner code
if the options does change the rule generation, then this should be properly documented in the Cmdliner.Arg, rather then borrowing options from the Cli module which have documentation intended for use in ocaml-mdx test instead.

Please update the changelog and I'll merge!


[pigmentcss] Rename zero-runtime to pigmentcss

This PR mainly renames all the zero-runtime references to Pigment CSS and package names.
@pigmentcss/react
@pigmentcss/unplugin
@pigmentcss/vite-plugin
@pigmentcss/nextjs-plugin

[x] I have followed (at least) the PR section of the contributing guide.

I wonder if we should change apps/* to pigment-apps/*? I think the folder is really specific to pigment.

Right now, it is. But could be a generic apps directory in future where non pigment apps also live.

We missed the non breaking space in the name in this PR. I opened https://github.com/mui/material-ui/pull/41438 to fix this.


Cross Origin errors in Sandboxed iFrame

Expected Behavior
When executing Pixi.js code inside a sandboxed iframe that does not have the allow-same-origin attribute, it should automatically make cross origin requests for any loaded resources.
Current Behavior
Currently, this does not happen, due to the default behaviour of https://github.com/englercj/resource-loader which fails to check window.origin to see if it is set to null which it will be inside a sandboxed iframe.
Possible Solution
This issue has been reported upstream at https://github.com/englercj/resource-loader/issues/119 and fixed at https://github.com/englercj/resource-loader/issues/120, once this fix has been merged and a new version of resource-loader has been released, upgrading the package should fix this issue in Pixi.js.
Steps to Reproduce

Run any Pixi.js app that uses resources (particularly WebGL resources) from inside a sandboxed iframe:

<iframe sandbox='allow-scripts' src='index.html'></iframe>

Where the index.html is a bootstrap for the Pixi.js app.

Observe the CORS errors in the console, as WebGL complains about reading out data from a 'tainted' canvas (i.e. one that has improperly loaded cross origin images).

Environment

pixi.js version: 4.5.2 (but should still be extant)
Browser: Chrome 70
Device: Ubuntu 16.04 Desktop

Thanks for reporting this and for your detailed explanation. Will make sure we get this in as soon as @englercj updates resource-loader.

Just published v2.2.0 which contains the fix from @rtibbles.

Hi @bigtimebuddy would it help to submit a PR for the dependency update, or is it simpler to do from your side? Some kids in Uganda are going to be very happy when Pixi.js can work in a sandboxed iframe environment!

Putting in a PR is the fastest way to get changes into pixi :)

Sorry, looks like my fix in https://github.com/englercj/resource-loader/pull/120 did not actually work, am submitting an improvement that directly checks window.origin against window.location.origin.

Thanks for bringing in the fix @englercj - still thinking how to add a test or JSFiddle to show the issue this is fixing, but probably won't get to it until the new year.


Adding Kin Markets and Mint Address

Hi @myyellowshoe, Thanks for the PR! Looks like the markets specified have large price tick sizes relative to the current market price of KIN. I went ahead and re-listed these markets with more reasonable parameters. I will merge those markets shortly.

Hi @myyellowshoe, Thanks for the PR! Looks like the markets specified have large price tick sizes relative to the current market price of KIN. I went ahead and re-listed these markets with more reasonable parameters. I will merge those markets shortly.

Additionally, I believe there is a typo in the coin mint for KIN as the specified mint is not valid base58

Additionally, I believe there is a typo in the coin mint for KIN as the specified mint is not valid base58

Additionally I believe there is a typo in the coin mint as the one provided is the incorrect size.

Additionally I believe there is a typo in the coin mint as the one provided is the incorrect size.

closing this in favor of #49

closing this in favor of #49

Thanks for the feedback @nathanielparke. Still learning the ropes of markets and such on serum :)
Thanks for fixing the issues with those markets.
Ugh yeah looks like I had a t on the end for some reason. Appreciate you catching that.

Thanks for the feedback @nathanielparke. Still learning the ropes of markets and such on serum :)
Thanks for fixing the issues with those markets.
Ugh yeah looks like I had a t on the end for some reason. Appreciate you catching that.


Add small .sh for easier report while investigating problem

Ok, but it should be in its own directory as all other utils.


Create hi.tcl, fibonacci.erl

What the program do ?
Example of function use in tcl
It say Hi to the args :D
Fibonacci in Erlang

In what programming language it is written?
It's written in Tool Command Language (TCL).
It's written in Erlang.

Updated pull request

This pull request do not contains files in specific folders. Please fix it and reopen


feat: basic repo page

Description
Implements the basic repo page under /[org]/[repo] route, fetching data via the API.

What type of PR is this? (check all applicable)

[x] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents
Closes #2985

Mobile & Desktop Screenshots/Recordings
https://github.com/open-sauced/app/assets/20603494/70a0dd3f-728b-4471-b242-de88012df4ec

Steps to QA

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[ ] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Before approving -- can we hide the feature behind a posthog feature flag?
I've already setup the feature flag in posthog (I replicated the one we had for workspaces, and set the cohort to just the internal team) --> us.posthog.com/project/13200/feature_flags/38219
And here are the Posthog docs on how to set them up in the code.
@nickytonline has done this before with Workspaces so he'll be able to help :)

Server side you can call getAllFeatureFlags.
https://github.com/open-sauced/app/blob/dc760cc99ceb0c27c4c99f9cef4a3abed5f4f683/lib/utils/server/feature-flags.ts
Here's an example, https://github.com/open-sauced/app/blob/dc760cc99ceb0c27c4c99f9cef4a3abed5f4f683/pages/hub/insights/index.tsx#L31
If you need it to gate UI, you can use https://github.com/open-sauced/app/blob/dc760cc99ceb0c27c4c99f9cef4a3abed5f4f683/components/shared/feature-flagged.ts

Curious if we should show a 404 instead of a 500 for repos not found.
https://deploy-preview-2988--oss-insights.netlify.app/foo/lala

Curious if we should show a 404 instead of a 500 for repos not found.
deploy-preview-2988--oss-insights.netlify.app/foo/lala

I would suggest a 404 as well.

Maybe a bug in the routing: when I go to kubernetes/kubernetes I'm sent to this explore page:

I thought maybe it was a bug in the routing where org === repo name but I'm able to go to other repos where the org is the same as the repo name (like contaienrdcontainerd`:

The only other thing I can think of is the routing hits the "kubernetes" topic?

This is due to kubernetes being a topic for the /explore/topic page, and since it was previously under the catchall /[pageId] route, we are redirecting to /explore if someone goes to the old route. Since the repo page is under a feature flag, I can try to override the redirect if the user is under the feature flag.

Curious if we should show a 404 instead of a 500 for repos not found.
deploy-preview-2988--oss-insights.netlify.app/foo/lala

I would suggest a 404 as well unless there's an actual crash.
https://github.com/open-sauced/app/blob/b69c40852838685d189ee75085f45bfee05c5e71/pages/lists/[listId]/activity.tsx#L56-L60

Weird since I've put in a 404 error explicitly.
https://github.com/open-sauced/app/pull/2988/files#diff-ea0fcfccb65d6d4d6e8cd2bddf5332c47b58e9dd2395c32848503a10ec2b13cdR24-R26
Most likely a 500 error since it's trying to use the user ID to get the feature flag, but is doing so for a logged out user. Will rethink implementation.

I wonder if it makes sense to add repo pages under a specific route and then the org/repo name? It could be
- /kubernetes/kubernetes
+ /repositories/kubernetes/kubernetes

Kind of like we have for usersm e.g.
https://app.opensauced.pizza/user/nickytonline

I wonder if it makes sense to add repo pages under a specific route and then the org/repo name? It could be
- /kubernetes/kubernetes
+ /repositories/kubernetes/kubernetes

Kind of like we have for usersm e.g.
https://app.opensauced.pizza/user/nickytonline

Would have to get @isabensusan to confirm, but I think we're doing /[org]/[repo] to be in line with GitHub's URL structure for repositories. I assume so people can just switch from github.com to app.opensauced.pizza to get the stats for the repository they're already looking at.

I wonder if it makes sense to add repo pages under a specific route and then the org/repo name? It could be
- /kubernetes/kubernetes
+ /repositories/kubernetes/kubernetes

Kind of like we have for usersm e.g.
app.opensauced.pizza/user/nickytonline

Would have to get @isabensusan to confirm, but I think we're doing /[org]/[repo] to be in line with GitHub's URL structure for repositories. I assume so people can just switch from github.com to app.opensauced.pizza to get the stats for the repository they're already looking at.

The only reason I mentioned in case an org name is the same as one of our base routes. For example, not an org but there's a github user https://github.com/user. If we went to one of their repos, it would try to load a potentially non-existent user on the platform and also, no one would ever be able to view the repo. Maybe an extreme edge case, but that's why I mentioned it.

I wonder if it makes sense to add repo pages under a specific route and then the org/repo name? It could be
- /kubernetes/kubernetes
+ /repositories/kubernetes/kubernetes

Kind of like we have for usersm e.g.
app.opensauced.pizza/user/nickytonline

Would have to get @isabensusan to confirm, but I think we're doing /[org]/[repo] to be in line with GitHub's URL structure for repositories. I assume so people can just switch from github.com to app.opensauced.pizza to get the stats for the repository they're already looking at.

The only reason I mentioned in case an org name is the same as one of our base routes. For example, not an org but there's a github user https://github.com/user. If we went to one of their repos, it would try to load a potentially non-existent user on the platform and also, no one would ever be able to view the repo. Maybe an extreme edge case, but that's why I mentioned it.

We can check for edge cases by checking if the '/[org]' matches one of our top routes and if the '/[org]/[repo]' actually exists. Like since '/user/dwm' is an actual repo, it'll show a repo page, then '/user/zeucapua' isn't a valid repo, then we can continue to the user profile page.
What do you think about this possible solution?

Decided that we keep the /[org]/[repo] route and engineer around the edge cases. I'll get the work on it now to ensure none of the existing routes gets taken by the /[org] route parameter.
Slack thread: https://opensaucedworkspace.slack.com/archives/C035RQ8ANBW/p1711051302766729

TIL github.com/u and github.com/user are users

I'm getting the redirect issue for kubernetes as well. I also get a 404 going to a repo page. I'm going to see if my account is part of the feature flag.

Took it for a test run and the minimal page loads as expected and 404s if I'm logged out (no feature flag enabled). I'd still push back on the route and suggest a /repositories, or /repo at the beginning.

I'm 100% fine with app.opensauced.pizza/{org}/{repo}: in fact, I think Brian is right. The discoverability is a +1.
I'm just concerned that we'll create alot of cruft and tech debt in the middleware due to how the explore page redirects are working. From what I understand, there's a catch all that will be captured by the middleware for a not insignificant number of repos: kubernetes/kubernetes, kubernetes/enhancements, kubernetes/org, etc.
Looks like zeu fixed that specifically for Kubernetes org repos. But, in my opinion, this explore page topic middleware catch all needs to be removed in favor of true 404 pages and a better designed explore page.

I'm honestly torn about it.
Having /[org]/[repo] is nice, and it gives us the option of just have /[org] also.
On the other hand, we just unwound a bunch of tech debt with the existing explore page for matching URLs.
[pageId] - /javascript
[pageId]/[filter] - /javascript/dashboard/
[pageId]/[filter]/filter/[filterName] -  /javascript/dashboard/filter/recent
[pageId]/[filter]/filter/[...selectedFilter] - /javascript/dashboard/filter/kubernetes/kubernetes/or/whatever/else
The folder structure we had for that was an incredibly greedy way to capture the URL, which led to many odd behaviors with navigation. I think if we have just 2 dynamic paths in the URL like this, we're putting ourselves right back in the same position to deal with the tech debt we just moved away from. Any URL that someone types in like /one/two is going to match that route and using middleware to decide what's valid doesn't feel like a great solution I would want to deal with long term.
My vote would be to use some sort of prefix before /[org]/[repo] that's not repo or repositories. Using a URL shortener or a separate domain/subdomain that does the redirect for you could also be an option.
pizza.to/kubernetes/kubernetes takes you to app.opensauced.pizza/[something]/kubernetes/kubernetes for  example.

I'm honestly torn about it.
Having /[org]/[repo] is nice, and it gives us the option of just have /[org] also.
On the other hand, we just unwound a bunch of tech debt with the existing explore page for matching URLs.
[pageId] - /javascript [pageId]/[filter] - /javascript/dashboard/ [pageId]/[filter]/filter/[filterName] - /javascript/dashboard/filter/recent [pageId]/[filter]/filter/[...selectedFilter] - /javascript/dashboard/filter/kubernetes/kubernetes/or/whatever/else
The folder structure we had for that was an incredibly greedy way to capture the URL, which led to many odd behaviors with navigation. I think if we have just 2 dynamic paths in the URL like this, we're putting ourselves right back in the same position to deal with the tech debt we just moved away from. Any URL that someone types in like /one/two is going to match that route and using middleware to decide what's valid doesn't feel like a great solution I would want to deal with long term.
My vote would be to use some sort of prefix before /[org]/[repo] that's not repo or repositories. Using a URL shortener or a separate domain/subdomain that does the redirect for you could also be an option.
pizza.to/kubernetes/kubernetes takes you to app.opensauced.pizza/[something]/kubernetes/kubernetes for example.

Great idea with the URL shortener. Gives us the best of both worlds.

My choices for possible prefixes:

/r: for 'repository', /r/open-sauced/app
/rp: for 'repo page', /rp/open-sauced/app
/s: for 'stats' or 'sauce', /s/open-sauced/app

Great idea with the URL shortener. Gives us the best of both worlds.

+1 - great unlock: an ideal solution for sure!

Repo pages will now be under /s/<org>/<repo>!


DBFace using with Android tflite format

Hello,
I have a question about using the db_keras.tflite model in Android. I am not sure the correct parameters for the face detector when generating anchors. If you can provide info about them that would be amazing. Info that I am lacking are feature sizes & anchor sizes with anchor ratios, here I have added some random data from another object detector:
private static final int[] feature_map_sizes = new int[] {33, 17, 9, 5, 3};

private static final float[][] anchor_sizes = new float[][] {{0.04f, 0.056f}, {0.08f, 0.11f}, {0.16f, 0.22f}, {0.32f, 0.45f}, {0.64f, 0.72f}};

private static final float[] anchor_ratios = new float[] {1.0f, 0.62f, 0.42f};`

If u have pre-processing and post-processing for face detector in Android Java code that would help immensely. Thank you.

I don't have any information about the db_keras.tflite you mentioned.
The pre-processing and post-processing depend on the tflite model, so please ask the person who designed that model.

It is dbface model from your repository just didnt write full name dbface_keras_480x640_float32_nhwc.tflite [https://github.com/terryky/android_tflite/tree/master/tflite_dbface/app/src/main/assets/model].

This model (dbface_keras_480x640_float32_nhwc.tflite) doesn't require anchors.
The detected region can be obtained directly from the inference output.
please refer the post processing code as below:
https://github.com/terryky/android_tflite/blob/6a6d889e413d766b0d3cf656035ecb6341276e99/tflite_dbface/app/src/main/cpp/tflite_dbface.cpp#L83-L133


Coordinators cannot navigate to past events.

Recent change to the Coordinator dashboard has removed the ability for a coordinator to see PAST events.  This is inconvenient when coordinators need to work with times.

closed by commit e72476d


accessing-kubernetes.md is blank

It looks like kubo-deployment/docs/user-guide/accessing-kubernetes.md. Should we delete it?

As all docs are moving to docs-kubo, I think we should delete it.


com.facebook.react:react-native Retrofit2.2.0 comflict

We use GitHub Issues for bugs.
If you have a non-bug question, ask on Stack Overflow: http://stackoverflow.com/questions/tagged/react-native
If you have a feature request, post it on Product Pains: https://productpains.com/product/react-native/
--- Please use this template, and delete everything above this line before submitting your issue ---
Description
[FILL THIS OUT: Explain what you did, what you expected to happen, and what actually happens.]
Reproduction
[FILL THIS OUT: Try to reproduce your bug on rnplay.org and provide a link. If you can't reproduce the bug on rnplay.org, provide a sample project.]
Solution
[FILL THIS OUT: What needs to be done to address this issue? Ideally, provide a pull request with a fix.]
Additional Information

React Native version: [FILL THIS OUT: Does the bug reproduce on the latest RN release?]
Platform: [FILL THIS OUT: iOS, Android, or both?]
Operating System: [FILL THIS OUT: MacOS, Linux, or Windows?]

Hey, thanks for reporting this issue!
It looks like your description is missing some necessary information. Can you please add all the details specified in the template? This is necessary for people to be able to understand and reproduce the issue being reported.
I am going to close this, but feel free to open a new issue with the additional information provided. Thanks!


chore: add xhr2 package

This should fix the error described in https://github.com/haveno-dex/haveno/issues/1482.
To use:
npm install
npm pack
and install the packaged file in sample app
npm install haveno-ts-0.0.30.tgz
npm run start
You will not see the XMLHttpRequest is not defined error anymore

Thanks for your submission. In the end, this solution provided everything we were looking for: https://github.com/haveno-dex/haveno-ts/pull/338

Uses node-xmlhttprequest which is currently used in the jest tests
Only imports types to the global object if not present
Exports the types for clients to use without additional imports
Ignores type warnings
Doesn't introduce vulnerabilities (actually fixes the current high severity vulnerability).


Unable to deploy nodejs bot to azure

version 2.1.2
Steps to reproduce
Create a new bot
Select Node Preview
Select empty bot
Press next
Enter a name (UniqueName)
Create a new folder with that name
Runtime type = Azure Web App
Press Create
As soon as the chatbot is created, test it under the tab Web Chat
Press Publish (menu on the left)
Press Publishing Profile and add New
Give it a name and under Publishing target select Publish bot to Azure
Press Next
Select Create New Resources
Press Next
I need to sign in. Using my Azure credentials results in an error "___ isn't in our system. Mak sure you types it correctly" (Problem 1) I was already logged in Azure (it show an option logout)
Press cancel and continue
Select my subscription
Select existing resourcegroup
Operating system Windows (my app serviceplan uses Windows)
Give the resource a unique name
Region and Luis region: Western Europe
Press Next
Remove all option options except for Application Insights
Press Next
Press Create
-> Provisioning Failure (Problem 2)

@HesselWellema,
More information would be helpful. Could you please check the provision.UniqueName2.log error log mentioned in the error message and see what information it contains?

{
"profileName": "UniqueName2",
"jobId": "df2bff91-810f-41c3-a524-f39fb0b276c5",
"projectId": "82131.37744223078",
"time": "2023-02-13T12:11:05.063Z",
"log": [
"Creating Azure resources...",
"Checking the Existence of the Resource Group ...",
"Use the Resource Group:  at westeurope",
"> Creating App Registration ...",
"App create failed, retrying ...",
"App create failed, retrying ...",
"App create failed, retrying ...",
"App create failed, retrying ...",
"App create failed after 3 retries. Error: Request failed with status code 401",
"{"name":"CREATE_APP_REGISTRATION","message":"App create failed! Please file an issue on Github.","stack":"Error\n    at new CustomizeError (D:\\Bot development programma's\\BotFramework-Composer\\Bot Framework Composer\\resources\\app.asar.unpacked\\extensions\\azurePublish\\dist\\extension.js:276418:25)\n    at createCustomizeError (D:\\Bot development programma's\\BotFramework-Composer\\Bot Framework Composer\\resources\\app.asar.unpacked\\extensions\\azurePublish\\dist\\extension.js:276423:10)\n    at BotProjectProvision.createApp (D:\\Bot development programma's\\BotFramework-Composer\\Bot Framework Composer\\resources\\app.asar.unpacked\\extensions\\azurePublish\\dist\\extension.js:277931:17)"}",
"{"name":"CREATE_APP_REGISTRATION","message":"App create failed! Please file an issue on Github.","stack":"Error\n    at new CustomizeError (D:\\Bot development programma's\\BotFramework-Composer\\Bot Framework Composer\\resources\\app.asar.unpacked\\extensions\\azurePublish\\dist\\extension.js:276418:25)\n    at createCustomizeError (D:\\Bot development programma's\\BotFramework-Composer\\Bot Framework Composer\\resources\\app.asar.unpacked\\extensions\\azurePublish\\dist\\extension.js:276423:10)\n    at BotProjectProvision.createApp (D:\\Bot development programma's\\BotFramework-Composer\\Bot Framework Composer\\resources\\app.asar.unpacked\\extensions\\azurePublish\\dist\\extension.js:277931:17)"}. See provision.UniqueName2.log in your bot folder"
]
}

You appear to be experiencing a 401 error. Azure is unable to authenticate you. Please verify that your credentials are correct, and that you have permissions to create the resources that you are attempting to deploy.
You could also try a clean install of Composer.

closing for inactivity, please reopen if you are still having issues


I can't connect

I'm trying to connect.
It spins for 5 seconds, then the error comes out: "Unexpected error"

Hi, host must be without http. Only IP address.
And, I'm not sure about the service port too.
Common port for inels connection server is 8001.
Kind regards

Hi,
Hope you can help me,
I am getting immediatly "Unexpected error"
I have a CU3-01M Protocol version: 00.02.00
and connection server "ver. connection-server-3.367"
i'm assuming connection servers ip address, port 8001, and CU3-01M for central unit name.

It would be great to get this working,
BR,
TLL

Hi ,
thanks for your post.
Please provide full log here.
JH,

Do i need connection server? I have CU3-02M and cannot to connect.

Do i need connection server? I have CU3-02M and cannot to connect.

Hi Peter,
definitely you need a connection server.
This integration is based on XML-RPC protocol, which is provided by connection server.
Kind regards
J.H.

Hi, thank for your answer. Where can i download connection server?
And one more question: Is it possible to control Inels system bits via InelsForHass? Is possible to control SA2-04M connected to CU3-02M via MI3-02M via InelsForHass?
Thank you

Hi, thank for your answer. Where can i download connection server?
And one more question: Is it possible to control Inels system bits via InelsForHass? Is possible to control SA2-04M connected to CU3-02M via MI3-02M via InelsForHass?
Thank you

InelsForHass comunicate only over XML-RPC protocol, so there is no other way to control iNELS Bus, then with connection server.
You can't download it. You need to buy it. Even if you will get just software of connection server, then you need to emulate it. For example as a container in docker ecosystem.
I tried it couple years ago with no success.
There is necessary to have license key which is generated based on physical network interface and software version I guess.

Thank for your answer, i will buy connection server, but i neet to know, if is possible control inels 2 actors connected to inels3 CU in InelsForHass. For example SA2-04M connected to CU3-02M via MI3-02M
thank you

Thank for your answer, i will buy connection server, but i neet to know, if is possible control inels 2 actors connected to inels3 CU in InelsForHass. For example SA2-04M connected to CU3-02M via MI3-02M
thank you

CU unit comunicate with connection server and connection server comunicate with InelsForHass.
So when SA2-04M will appears in connection server then InelsForHass will find them.
Of course, there can be some sort of difference with naming, but it can be solved individually.
I'm still developing this integration in my free time.
In very close future, I'm going go to refactor it because of collision with official iNELS integration which is now developing in cooperation with Elko a.s.
Kind regards
J.H.


Adding the port group entity

This PR add the port group entity to the plugin DB and implements the create, update and delete methods for this entity used by OSC when an SDN controller supports grouping virtual inspected ports.

WIP

@dmitryintel This is now ready for your review.


Measure Activity for Sugarizer

Developed the Measure Activity in sugar for Sugarizer.

#709

57b113c commit message is a bit brief.  You could say what you used, like p5 sound.
I see you added exec = sugar-activity-web.  Will you be porting this to Sugar or does it work already?

@quozl @llaske  I am developing this activity for Sugarizer. Sugar has the measure activity and I took the inspiration from https://help.sugarlabs.org/measure.html .
I have used the p5.js ,p5.sound and p5.dom libraries in this activity.
There is more work that needs to be done like plotting waveform for different octaves,play a sound and show how waveform is varying at different amplitudes.

Good start. Nice.
You should also adapt the UI to Sugarizer. Specifically, settings should be in a palette, not directly on the string. Plus the buttons in the toolbar should be selected/unselected instead of hidden.

@llaske Thanks for the feedback.
By settings do you mean the sliders at the bottom of the page ?
Also can the x-axis scale be shown on the graph or do I need to place it somewhere else?If I have to place it somewhere else,where should it be placed?
Working on suggestions.

Yes I mean the sliders.
Regarding the x-axis scale, may be you could place it at the bottom of the graph like in Measure activity.

Working on the suggestions.

@llaske I am finding difficulties in moving the sliders to the toolbar palette. As the current sliders have been made using p5.js and these sliders are directly linked to the waveforms if I am making any changes to these, everything breaks. Could you provide me some documentation for sliders in sugarizer . I have tried to implement the working of sliders from different activities but it didn't work.

@abhishektanwar sliders in Sugarizer is just HTML input with the type range. You could find a very basic sample here. It's used for example in Speak or Game of Life activities.
Regarding the current sliders related to p5.js code, may be you could just hide them.

@llaske thanks for the help it worked.
I have fixed all the requested changes. Please review.
Activity is complete now.

@ashish0910 @llaske removed console logs. Please review.

Mmmm. The activity does nothing on my side.
At start the screen is totally black:

When the graph button is clicked the activity seems to listen but nothing happen:

Plus will be nice to improve icons/slide bar proportions/alignement in palette:

@llaske @ashish0910 Waveform is plotted from the values of sliders (amplitude and frequency). Initially, the values of amplitude and frequency slider is 0(zero). As the sliders are moved, the corresponding waveform is plotted.

Fixed alignment of icons and sliders in palette.
Please review

So do not start with a value of zero for amplitude and frequency and display the waveform at launch.
Note sure to understand why sliders values change when switching graph?
Plus, will be nice to save values (and current graph selection) in the datastore so the user could reopen the activity to recover its context.

thanks for the feedback. working on it.

@llaske @ashish0910

added datastore functionality
fixed initial zero value of sliders
on initializing an old instance of the activity , activity starts from previous slider settings and last waveform(time or frequency)
saves current graph when activity is stopped

please review

That's nice.
Few remarks:

Somethings goes wrong with the palette, the icon become big when the palette is open
Sound like Amplitude and Frequency settings are related to the graph. That's fine but all settings are not saved in datastore, only the one of the active graph
When clicking on fullscreen/unfullscreen you should not only hide the toolbar, you should also resize the canvas
Look like the activity description in the tutorial repeat two times the same thing "This activity draw a picture..."
Try to force the opening of palette before displaying the slider helper in tutorial (could be done with event on steps)
Title of graphs should be localized too
Try to better indent your code, in activity.js some blocks have a very weird indentation
Change the title of the HTML page - currently "My activity"

Also, is it possible to add two features from the Sugar version of the activity ?

Freeze the graph (need to add a Pause/Play button in the toolbar)
Save the current graph as PNG in the journal

Sound like Amplitude and Frequency settings are related to the graph. That's fine but all settings are not saved in datastore, only the one of the active graph

I did not understand what you are trying to say. Currently, I am saving the active graph(to load the activity from the save graph when an old instance is initiated), the values of all the slider(on both time and frequency graph) are also saved to start the graph from previous slider settings. Could you please explain a bit? At all time when a slider value is changed in any of the graph time or frequency based , that value is stored locally in data object and when activity is stopped all the updated settings of last active graph and sliders is stored in datastore obejct.

Look like the activity description in the tutorial repeat two times the same thing "This activity draw a picture..."

I don't see this problem on my side. Maybe there is a confusion between the first and the eleventh step of tutorial . First dialog box of tutorial gives the description of what the activity is all about(This activity draw a picture) and eleventh dialog box gives a brief information about the graph ("This graph shows the waveform generated").

Try to force the opening of palette before displaying the slider helper in tutorial (could be done with event on steps)

This is a great suggestion. I tried to work it out, could you please help me with some more information or steps to be followed. Any activity that might have a similar kind of working would help.

Freeze the graph (need to add a Pause/Play button in the toolbar)

What I understand from your point of view is to freeze the graph and take the screenshot.
I am not sure about freezing this because the graph is the result of live input through a microphone, I believe we can add the functionality of taking the screenshot of the graph at any time and save it in the journal.

I did not understand what you are trying to say.

See in the video capture below. Amplitude and Frequence are different for each graph. Values are not retrieved for Sound time frequence graph when the activity is reopen.

I don't see this problem on my side

See screen capture below. I've underlined in red the repeated text.

Any activity that might have a similar kind of working would help.

See the popup for the activity palette in SharedNotes activity here.

What I understand from your point of view is to freeze the graph and take the screenshot.

Yes, see buttons 5 and 6 here.
Regarding 5, may be you could set the input to null during the freeze.
Regarding 6, it's possible to convert a Canvas into an image then store it in the datastore. See for example here how it works in Moon activity.

Yes, see buttons 5 and 6 here.
Regarding 5, may be you could set the input to null during the freeze.
Regarding 6, it's possible to convert a Canvas into an image then store it in the datastore. See for example here how it works in Moon activity.

Regarding button 5, mic captures input live and plots a graph for it, I am afraid that a null input will plot a straight horizontal line on the graph(it will not be plotting sinusoidal wave for null input).
I think what you are expecting from freeze is that whatever is the shape of the wave currently only that should be shown on the canvas with no effect of input through the mic. Like the graph here
Rest of the issues I understand and working on them

Hmmm. May be you could just stop drawing when the Pause button is clicked. If the canvas is not refreshed, it should work.
Another thing: could you remove ES6 syntax in your file? I've seen for example lot of use of let keyword. Unfortunately, ES6 syntax is not supported by all browsers.

Somethings goes wrong with the palette, the icon become big when the palette is open

I tried to fix this however the issue is with this particular icon. So I have changed the icon, now

Sound like Amplitude and Frequency settings are related to the graph. That's fine but all settings are not saved in datastore, only the one of the active graph

FIxed

When clicking on fullscreen/unfullscreen you should not only hide the toolbar, you should also resize the canvas

FIxed

Look like the activity description in the tutorial repeat two times the same thing "This activity draw a picture..."

FIxed

Try to force the opening of palette before displaying the slider helper in tutorial (could be done with event on steps)

FIxed

Title of graphs should be localized too

FIxed

Try to better indent your code, in activity.js some blocks have a very weird indentation

FIxed

Change the title of the HTML page - currently "My activity"

FIxed

Also, is it possible to add two features from the Sugar version of the activity ?

Freeze the graph (need to add a Pause/Play button in the toolbar)
Save the current graph as PNG in the journal

I have incorporated the freeze functionality within the save functionality. The user does not have to freeze the graph/waveform and then take a screenshot/convert graph in an image. Users can simply click the save button and the image of the graph will be saved in the journal without the hassle of freeze/play , this makes it easier to operate the activity.

Cool. Good job.
When the graph is paused it's not at the same place than the running graph.
Could you try to set it at the exact same place?

Nice.
I've got an issue when I'm launching from file://
The graph is not displayed and there is an error in console:

I guess it could be the same issue than here and could require to load the file as a blob object.

It is working on my side @llaske

it worked with google-chrome-stable --allow-file-access-from-files also.
I am not sure why this is happening. Maybe if you could show me how you produced this error

Of course you could open the file alone but there is an issue when it's open from Measure activity in file://

@llaske I have fixed the file:// error(this error was only in chrome ) but after fixing this error I encountered  a new error

the link with the error in console : https://goo.gl/7K7WLu
This error occurs in chrome only and not firefox due to google's policy . I have tried to bypass so that as soon as the activity starts graph can be plotted by the input audio but chrome only take audio input after a user gesture .
It works fine in firefox . what we can do is when whenever or wherever user clicks or touches ,graph will start
I have push fix for both the errors . please review

Okay. I've already seen this error.
I suggest to start the activity paused, in this case only (if on Chome and if file://).
BTW your current fix - paste the entire image in base64 in the source file - is ugly: will be better to use and XmlHttpRequest and read the file into a blob like here.
Plus remove commented and unused lines.

I suggest to start the activity paused, in this case only (if on Chome and if file://).

I think to maintain the working of activity similar to for all the browsers , activity can be started with pause state (the button which is being used to pause the graph) and then the user can start the activity by clicking on play button or we can proceed as you suggested earlier by tweaking the pause button for chrome browser.

I'm not agree. The activity should start automatically.
Paused at startup is just a work around.

I'm not agree. The activity should start automatically.
Paused at startup is just a work around.

Okay then activity will start normally for all browsers . But for chrome ,activity will start with paused state . User will have to click play button.
Please let me know if this is fine.

But for chrome ,activity will start with paused state

For Chrome, only when run from file://

The activity no longer work on http:// :

can you layout the steps you followed to run this activity from the beginning . It worked fine on my side @llaske

Just cleaning the browser cache. Then launch a new instance of Measure.

@llaske there was one error . Fixed it . Please review again.

In http mode it works but the title "Sound time base" is displayed under graph:

In file mode the graph is not drawn correctly:

Try to do more testing before sending your PR.

@llaske it is working fine on my side in file mode .
If you are using some other method to testing other than I am using please let me know .
When in chrome and file mode , I am showing the empty graph ,user clicks on play button which also accounts for the policy of chrome that a user gesture is required for accessing microphone.

Do you clear your browser cache?

@llaske Yes, I have cleared all the cached files in the chrome browser.

Sound like it happens randomly. When it happens, there is a message file protocol error in the console:

Another issue, when switching graph in pause mode, a double graph appears:

Sound like it happens randomly. When it happens, there is a message file protocol error in the console:

What can we do with this?What do you recommend?

Another issue, when switching graph in pause mode, a double graph appears:

Yes it can happen because according to previous discussions activity was designed to switch between time and frequency graphs only when the graph is in play mode (not in pause state).

What can we do with this?What do you recommend?
Try to explore in which condition this can happen and fix it.

Yes it can happen
Okay but it should be solve. We can't let it like that.

Sound like it happens randomly. When it happens, there is a message file protocol error in the console:

I have tweaked the activity for chrome a bit . Please review it again . Works fine on mine side(tested after clearing cache)

Another issue, when switching graph in pause mode, a double graph appears:

Fixed

Still few issues:

The graph should start automatically in Chrome when not in file mode. My request was to pause the activity at start in Chrome AND in file mode
When the activity is reopened, the double background issue still appears:

I've got several times the random error described before. Following is some screen capture when it happens. Did you have the issue sometimes on your side?

Remove commented lines and //change comment

Replaced by https://github.com/llaske/sugarizer/issues/709


Replace radios in search G-Cloud services journey

https://trello.com/c/UQtUlH0n/207-2-replace-radios-with-govuk-frontend-radios-component-in-search-g-cloud-services-journey
For the most part, these were straightforward, but the save search form made use of conditional reveal. This has outstanding accessibility issues so I've moved that field to a new page instead.
The code is quite repetitive - I could achieve the same effect by putting conditional logic on the template itself, but I thought it best to keep the urls discrete. Additionally, I've used WTForms as this seemed like the most efficient way to handle submission and validation - thinking about how we replace 'em is probably a future task.
Functional tests: https://github.com/alphagov/digitalmarketplace-functional-tests/pull/813

My review checklist
I've:

[ ] looked at the code
[ ] tested locally
[ ] run functional tests locally


Ensure js as server only

Describe the problem
My worst fear is that some code that was only ment to be serverside suddenly is referenced by a client side script and thereby is included in the clientside code.
At some point I typically Import some stuff only to find that this wasnt such a great idea.
yes - api endpoints are made for this
no - they dont prevent you from making that sweet /utils/mysecrets/businessrules.js only ment for serverside, but accidently added to clientside ...
Describe the proposed solution
I suggest that any js file can be defined as being server side only and that sveltekit should prevent it from being compiled to client side code.
A few ideas
The folder: Any file in /server can only be accessed from serverside code
The filename: Any file with *.server.js (or *.server.ts) can only be accessed from serverside code
Or reusing from "private modules": Any file or folder prefixed with a "_" such as can only be accessed from serverside code
Alternatives considered
No response
Importance
would make my life easier
Additional Information
No response

How would one know what a server side only component is where you are only allowed to import these files?
Also: This sounds like it could also be solved with a custom ESLint rule.

I agree an eslint rule or something along those lines is a better fit for this. You could also do this today by creating two projects - a backend project and frontend project.
I'm going to go ahead and close this because I've never seen anyone actually hit this issue and it seems relatively low priority compared to all the other issues we have.
We have special handling of environment variables between client and server already, which handles the most crucial aspect of this.

Have to respectfully disagree. It's one of those things that will help ppl not to share code that wasnt intented for any reason.

I agree an eslint rule or something along those lines is a better fit for this.

Some (many) wouldnt know how to eslint this (me included)

You could also do this today by creating two projects - a backend project and frontend project.

That might be true but not really a solution. It seems complicated and require two instances.

We have special handling of environment variables between client and server already, which handles the most crucial aspect of this.

True. But not code.


Simplify exception handling

Right now, I have the following implementation in a wordpress plugin, using WebPConvert:
catch (\WebPConvert\Converters\Exceptions\ConverterNotOperationalException $e) {
    // The converter is not operational.
    $failure = 'The converter is not operational';

    // TODO: We should show link to install instructions for the specific converter (WIKI)

    $msg = $e->getMessage();
} catch (\WebPConvert\Converters\Exceptions\ConverterFailedException $e) {
    $failure = 'The converter failed converting, although requirements seemed to be met';
    $msg = $e->getMessage();
} catch (\WebPConvert\Converters\Exceptions\ConversionDeclinedException $e) {
    $failure = 'The converter declined converting';
    $msg = $e->getMessage();
} catch (\WebPConvert\Exceptions\InvalidFileExtensionException $e) {
    $failure = 'The converter does not accept the file extension';
    $msg = $e->getMessage();
} catch (\WebPConvert\Exceptions\TargetNotFoundException $e) {
    $failure = 'The converter could not locate source file';
    $msg = $e->getMessage();
} catch (\WebPConvert\Exceptions\CreateDestinationFolderException $e) {
    $failure = 'The converter could not create destination folder. Check file permisions!';
    $msg = $e->getMessage();
} catch (\WebPConvert\Exceptions\CreateDestinationFileException $e) {
    $failure = 'The converter could not create destination file. Check file permisions!';
    $msg = $e->getMessage();
} catch (\Exception $e) {
    $failure = 'Unexpected failure';
    $msg = $e->getMessage();
}

WebPConvert throws an awful lot of exceptions, and it would be nice to be able to handle them more generally. Also in order to ensure that my module will react on future exceptions

I propose that the exceptions thrown by WebPConvert all extend a common base class.
This will allow me to do the following:
catch (\WebPConvert\Exceptions\ConversionFailedException $e) {
            // catch all exceptions anticipated by WebPConvert
        }
catch (\Exception $e) {
           // catch exceptions not anticipated by WebPConvert
 }

(or:)
            if (is_subclass_of($e, 'WebPConvert\Exceptions\ConversionFailedException')) {
                // ...
            }

In addition, it would be nice if the exceptions had codes to identify them by.
I would then be able to do this:
$failureTexts = ['The converter is not operational', ''The converter failed converting, although requirements seemed to be met', ...];

catch (\WebPConvert\Exceptions\ConversionFailedException $e) {
    // catch all exceptions anticipated by WebPConvert
    $failure = $failureTexts[$e->getCode()];
}

The codes above will however not work with future exceptions.
So better, perhaps, if the exceptions themselves contained the human readable description of themselves. I could then simply do:
catch (\WebPConvert\Exceptions\ConversionFailedException $e) {
    // catch all exceptions anticipated by WebPConvert
    $failure = $e->getFailureDescription();
}

or perhaps simply $e->$description

The exceptions now has a public variable $description, which contains a human readable description of the exception

The base exception is called "\WebPConvert\Exceptions\WebPConvertBaseException".
Usage:
try {
    ConverterHelper::callConvert('imagick', $source, $destination, array(
        'skip-pngs' => true
    ));
} catch (\WebPConvert\Exceptions\WebPConvertBaseException $e) {
    echo '' . $e->description . '
' . $e->getMessage();
} catch (\Exception $e) {
    echo 'unhandled exception' . $e->getMessage();
}


[Sprig App] Sigma Games

Author name
Author: The Sigmas Of Australia
About your game
What is your game about?
Idk
How do you play your game?
Learn

Duplicate of #2588


fixed label position for the ColorPicker

The current position is intended. Prefixes are supposed to move the label over in Material design, try enabling the leading icon here: https://material.io/components/text-fields


Update tachyons version

Pretty simple code janitor-ing here; the most recent version in package.json wasn't reflected in the source (and thus compiled) css files. Updating the text in src/tachyons.css and recompiling fixes it. While cleaning up I also bumped the tachyons-cli version even though it's not strictly necessary (because what's the harm?)

Thanks!


mrt: small Ticks conversion improvements

this PR responds to comments from https://github.com/lpc-rs/lpc8xx-hal/pull/299

Great, thank you @Lotterleben!

Great, thank you @Lotterleben!


values: add a docstring to MultiProcessValue

To document the method as semi-supported.
Adresses #509

Thanks!

By the way, this contained a breaking change in some cases as it wasn't just a docstring update technically. process_identifier used to be named _pidFunc so for anyone passing that as an explicit kwarg will run in to errors.

The Python convention is that names beginning with an underscore are internal, thus changing or removing them is not breaking.


Test Flowzone PR 476

Auto-generated by https://github.com/product-os/flowzone/pull/476
Will be closed automatically once the PR is merged.

Auto-closing pull request


Hello I have installed LightGBM but it does not on the simple example from the package

Environment info
Operating System: Mojave
CPU/GPU model:
C++/Python/R version: R3.5.1
LightGBM version or commit hash:
Error message
[LightGBM] [Info] Total Bins 137
[LightGBM] [Info] Number of data: 6513, number of used features: 116
[LightGBM] [Info] Start training from score 0.482113
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[1]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[2]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[3]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[4]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[5]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[6]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[7]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[8]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[9]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[10]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[11]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[12]:	test's l2:6.44165e-17
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] Stopped training because there are no more leaves that meet the split requirements
[13]:	test's l2:6.44165e-17
Reproducible examples
data(agaricus.train, package = "lightgbm")
train <- agaricus.train
dtrain <- lgb.Dataset(train$data, label = train$label)
data(agaricus.test, package = "lightgbm")
test <- agaricus.test
dtest <- lgb.Dataset.create.valid(dtrain, test$data, label = test$label)
params <- list(objective = "regression", metric = "l2")
valids <- list(test = dtest)
model <- lgb.train(params,
dtrain,
100,
valids,
min_data = 1,
learning_rate = 1,
early_stopping_rounds = 10)
Steps to reproduce

@kawess2 Which problem did you encounter?

@Laurae2 when I run the function lgd.importance it makes R crashes. But I finally fixed it. My other question is it normal that I get -inf as a best gain like it is mention above ("No further splits with positive gain, best gain: -inf") ?
Thank you for your help.

@kawess2 This warning is normal.


Test for Simple Griefing, coverage

Added tests for SimpleGriefing.sol, SimpleGriefing_Factory.sol.
Also, completed coverage for

test/modules
test/agreement

Closing because SimpleGriefing is now One Way


Use fedora 35 as base image

Merge only when Fedora 35 image contains rpki-client 7.5.

Will fix CI (will likely move docker build to github actions) in a different PR.


Scan more than one component at the time

Summary
This PR create an option for add more packages in scanning component feature, inspired by this issue. With this change, developers can use new @ComponetsScan annotation with values parameter where they can add all needed packages. The original @ComponentScan annotations is 100% preserved and now can cooperate with new @ComponetsScan.
All tests still passes like a charm and new test passed without any problem.
Examples
Before

@Module(includes = [DataModule::class])
@ComponentScan("org.koin.sample.androidx.app")
class AppModule

@Module(includes = [CommonModule::class, RepositoryModule::class])
@ComponentScan("org.koin.sample.androidx.data")
internal class DataModule

After 1. variant

@Module(includes = [CommonModule::class, RepositoryModule::class])
@ComponentsScan(values = ["org.koin.sample.androidx.app", "org.koin.sample.androidx.data"])
class AppModule

After 2. variant

@Module
@ComponentsScan(values = [
    "org.koin.sample.androidx.app", "org.koin.sample.androidx.data",
    "org.koin.sample.androidx.repository", "org.koin.sample.android.library",
])
class AppModule

After 1. variant if we want scan actual package

@Module
@ComponentsScan(values = [
    "", "org.koin.sample.androidx.app", "org.koin.sample.androidx.data",
    "org.koin.sample.androidx.repository", "org.koin.sample.android.library",
])
class AppModule

After 2. variant if we want scan actual package

@Module
@ComponentsScan
@ComponentsScan(values = [
    "org.koin.sample.androidx.app", "org.koin.sample.androidx.data",
    "org.koin.sample.androidx.repository", "org.koin.sample.android.library",
])
class AppModule

NOTE:
If dev use @ComponentsScan or @ComponentsScan([]) or @ComponentsScan([""]) this will scan actual package like original @ComponentScan.
If dev add non existing or correct package name to @ComponentsScan values, the package will be ignored and nothing happens.
If dev add multiple package names to @ComponentsScan values, only one will be used and multiple package names error is not shown.

let's go on branch v2 as it's a new feature / improvement proposal

I propose to keep one dedicated annotation, with @ComponentScan. Let's use the values atrtibutes to gather array of package.
No doubling of @ComponentScan needed. If people wants to go with current package, let's keep it "".
Can you update your code? 🙏

@arnaudgiuliani
I have modified the code, including the examples. The user can use @ComponentScan without additional parameters or "" to scan the current package. I also added a trim to each scanned package, so there shouldn't be a problem with, for example "         ".


Build pipeline using older version of node always ? unable to change to new version

Use Node Task output
Starting: Use Node
Task         : Node.js tool installer
Description  : Finds or downloads and caches the specified version spec of Node.js and adds it to the PATH
Version      : 0.192.0
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/tool/node-js
Found tool in cache: node 16.13.1 x64
Prepending PATH environment variable with directory: /opt/hostedtoolcache/node/16.13.1/x64/bin
Finishing: Use Node
Npm run build task output
`Starting: npm build
Task         : npm
Description  : Install and publish npm packages, or run an npm command. Supports npmjs.com and authenticated registries like Azure Artifacts.
Version      : 1.187.0
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/package/npm
/opt/hostedtoolcache/node/6.17.1/x64/bin/npm --version
3.10.10
/opt/hostedtoolcache/node/6.17.1/x64/bin/npm config list
npm info it worked if it ends with ok
npm verb cli [ '/opt/hostedtoolcache/node/6.17.1/x64/bin/node',
npm verb cli   '/opt/hostedtoolcache/node/6.17.1/x64/bin/npm',
npm verb cli   'config',
npm verb cli   'list' ]
npm info using<EMAIL_ADDRESS>npm info using<EMAIL_ADDRESS>npm verb exit [ 0, true ]
npm info ok
; cli configs
user-agent = "npm/3.10.10 node/v6.17.1 linux x64"
; environment configs
loglevel = "verbose"
userconfig = "/home/vsts/work/1/npm/11107362.npmrc"
; userconfig /home/vsts/work/1/npm/11107362.npmrc
registry = "#####removed###"
; node bin location = /opt/hostedtoolcache/node/6.17.1/x64/bin/node
; cwd = /home/vsts/work/1/s
; HOME = /home/vsts
; "npm config ls -l" to show all defaults.
/opt/hostedtoolcache/node/6.17.1/x64/bin/npm run build
npm info it worked if it ends with ok
npm verb cli [ '/opt/hostedtoolcache/node/6.17.1/x64/bin/node',
`

@phil-hodgson @aasim @satbai could you please take a look? Seems to be some possible issue with Npm task.

I find this works for me with node to install a specific version
  - task: NodeTool@0       displayName: 'Install Node .js'       inputs:         versionSpec: '14.17.3'          force32bit: false # Optional         checkLatest: false # Optional
    I also dont use the npm task I find I have more control over npm when I do the task as a command like this:

    `- script: |
    npm install -g<EMAIL_ADDRESS>    npm install
    ng build --prod
  displayName: 'npm install and build' `