latest version breaks elevation ruler

Hi, having some very strange behavior with the latest DAE and elevation ruler.
As you can see, the elevation ruler behaves normally when you don't go in a region. But as soon as I drag into the region, which is using the default Underwater (Swimming) terrain, the ruler stops and I get that big error message.
If I try to move the token around in the region, every square moved triggers that error and the ruler never shows up.
Modules and other info:
DAE 11.3.64
Elevation Ruler 0.10.11
libwrapper <IP_ADDRESS>
socketlib 1.1.0
Region Active Effects 1.2.0
Terrain Mapper 0.4.6
Foundry Virtual Tabletop: Version 12 Stable, 12.331
Game System: dnd5e, 3.3.1
Client: Chromium/129.0.6668.59
Error message:
https://cdn.discordapp.com/attachments/1287156848931241994/1287156870804672663/CleanShot_2024-09-21_at_16.50.44.png?ex=66f085c0&is=66ef3440&hm=05454628590cd89b8e2877155328c2e0a9f4eb172cea193d22874b9360c37a10&
when entering a region with default water terrain
https://cdn.discordapp.com/attachments/1287156848931241994/1287156871181897828/CleanShot_2024-09-21_at_16.52.01.png?ex=66f085c0&is=66ef3440&hm=267c3095cdb62566472e58550edc6b8bbed28d6a08f32fb197a4b64919fc12e7&
when moving around a region with default water terrain
https://cdn.discordapp.com/attachments/1287156848931241994/1287156871584546847/CleanShot_2024-09-21_at_16.52.16.png?ex=66f085c0&is=66ef3440&hm=774d1192c571d5bd0ea447887f0dcf5de3200ed315491f8e377b110596da8b9e&
Gif of what it looks like in action:
https://cdn.discordapp.com/attachments/1287156848931241994/1287215170191622255/CleanShot_2024-09-21_at_20.52.27.gif?ex=66f0bc0b&is=66ef6a8b&hm=6ce33b8c88b5bea53b30566c1ca4ac51e91f689a9714f9a41e480d7bfa9c34f1&

Yup something went very wrong on my side also recently

Can confirm. JS Exception:
Uncaught (in promise) TypeError: Cannot redefine property: _inferredSource
[Detected 5 packages: system:dnd5e(4.0.4), vision-5e(2.5.3), elevationruler(0.10.14), lib-wrapper(<IP_ADDRESS>), touch-vtt(2.2.10)]
    at Function.defineProperty (<anonymous>)
    at UtilityActivity.prepareFinalData (base-activity.mjs:563:12)
    at FeatData.prepareFinalActivityData (activities.mjs:292:56)
    at FeatData.prepareFinalData (feat.mjs:114:10)
    at Item5e.prepareFinalAttributes (item.mjs:648:33)
    at actor.mjs:184:37
    at EmbeddedCollection.forEach (foundry-esm.js:5031:9)
    at Actor5e.prepareData (actor.mjs:184:16)
    at Actor5e.prepareData (actor.mjs:13:15)
    at MovePenalty.moveSpeedWithinRegions (MovePenalty.js:218:18)
    at MovePenalty.movementCostForGridSpace (MovePenalty.js:369:20)
    at MovePenalty.movementCostForSegment (MovePenalty.js:297:36)
    at Ruler.js:484:32
    at SquareGrid._measurePath (Grid.js:115:23)
    at 🎁foundry.grid.SquareGrid.prototype._measurePath#0 (libWrapper-wrapper.js:191:52)
    at SquareGrid.measurePath (foundry-esm.js:40093:12)
    at Ruler._computeDistance (foundry.js:36272:38)
    at 🎁call_wrapped [as call_wrapped] (libWrapper-wrapper.js:511:22)
    at Ruler._computeDistance (Ruler.js:411:3)
    at 🎁CONFIG.Canvas.rulerClass.prototype._computeDistance#0 (libWrapper-wrapper.js:191:52)
    at Ruler.measure (foundry.js:36103:10)
    at 🎁call_wrapped [as call_wrapped] (libWrapper-wrapper.js:511:22)
    at 🎁Ruler.prototype.measure#touch-vtt (touch-vtt.js:1:57255)
    at 🎁call_wrapper [as call_wrapper] (libWrapper-wrapper.js:639:16)
    at 🎁Ruler.prototype.measure#0 (libWrapper-wrapper.js:193:20)
    at Ruler._onMouseMove (foundry.js:36719:10)
    at 🎁call_wrapped [as call_wrapped] (libWrapper-wrapper.js:511:22)
    at 🎁Ruler.prototype._onMouseMove#touch-vtt (touch-vtt.js:1:57176)
    at 🎁call_wrapper [as call_wrapper] (libWrapper-wrapper.js:639:16)
    at 🎁Ruler.prototype._onMouseMove#0 (libWrapper-wrapper.js:193:20)
    at #onDragLeftMove (foundry.js:25913:67)
    at MouseInteractionManager.callback (foundry.js:30834:17)
    at 🎁call_wrapped [as call_wrapped] (libWrapper-wrapper.js:511:22)
    at 🎁MouseInteractionManager.prototype.callback#touch-vtt (touch-vtt.js:1:56232)
    at 🎁call_wrapper [as call_wrapper] (libWrapper-wrapper.js:639:16)
    at 🎁MouseInteractionManager.prototype.callback#0 (libWrapper-wrapper.js:193:20)
    at #handleDragMove (foundry.js:31213:26)
    at #handlePointerMove (foundry.js:31178:64)
    at vh.notifyListeners (EventBoundary.ts:1464:33)
    at vh.notifyTarget (EventBoundary.ts:658:14)
    at vh.propagate (EventBoundary.ts:310:18)
    at vh.dispatchEvent (EventBoundary.ts:210:14)
    at vh.mapPointerMove (EventBoundary.ts:815:64)
    at vh.mapEvent (EventBoundary.ts:231:28)
    at ra.onPointerMove (EventSystem.ts:396:31)


[BUG]: pr-builder jobs not running in CI

Version
24.06
Which installation method(s) does this occur on?
Source
Describe the bug.
It appears that the job is running, however is mis-named and simply not being reported propperly.
Minimum reproducible example
No response
Relevant log output
Click here to see error details
[Paste the error here, it will be hidden by default]

Full env printout
Click here to see environment details
[Paste the results of print_env.sh here, it will be hidden by default]

Other/Misc.
No response
Code of Conduct

[X] I agree to follow Morpheus' Code of Conduct
[X] I have searched the open bugs and have found no duplicates for this bug report

Fixed in github settings. Thanks to @ajschmidt8  for spotting the error


Dismiss or disable "This configuration file contains list of Composer dependencies" message

Copied from https://youtrack.jetbrains.com/issue/WI-37611.
What steps will reproduce the problem?

Install the "PHP composer.json support" plugin.
Open a composer.json file, presumably one that has been configured for a project.
A message is displayed at the top, "This configuration file contains list of Composer dependencies", with options to Install and Update.

What is the expected result?
I expect to be able to get rid of the message in some way, either with a "Dismiss" button, X button like a notification, or through a configuration option.
What happens instead?
There appears to be no way to disable this dialog. See the comment in this YouTube video (https://www.youtube.com/watch?v=J58z0njPLMg):

Is there a way to disable the "This configuration file contains list of Composer dependencies" popup in composer.json? I don't like it at all, so I would love to find out a way to disable it. Thanks
Response:

There's no such a way. Is it that bad? Feel free to submit a usability issue: https://youtrack.jetbrains.com/newIssue?project=WI

Environment and versions

openSUSE Leap 42.2
PhpStorm 2017.2.1
Build #PS-172.3544.41, built on August 2, 2017
JRE: 1.8.0_152-release-915-b6 amd64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Linux 4.4.74-18.20-default
PHP composer.json support Version: 1.0.21

Hi,
jetbrains support is wrong, this message is not generated by composer.json plugin - Eugene Morozov is wrong. So I am not able to help you.


kopf._core.reactor.o: Not enough permissions to watch for resources

Hello,
Creating an issue here based on the suggestion from the slack channel.
We recently deployed the Cluster Secret Operator v0.0.9 to one of our GKE clusters and I see a couple of issues..
At the start of the pod logs I see:
kopf._core.reactor.o [WARNING ] Not enough permissions to watch for resources: changes (creation/deletion/updates) will not be noticed; the resources are only refreshed on operator restarts.

and later after the sync happens, I see an error:
Patching failed with inconsistencies: (('remove', ('status', 'namespace_watcher'), {'syncedns':['ns1', 'ns2']}, None),)

We used the helm chart for the deployment and default values.
Any help is appreciated.
Best Regards,
Ravi

Same warning here

I was able to fix the warning by changing the following
https://github.com/zakkg3/ClusterSecret/blob/d212216d4263e1f4829817864559af6981ee6819/yaml/00_rbac.yaml#L21
with
 - apiGroups: [apiextensions.k8s.io]

Meanwhile I released a new version for https://patrungel.github.io/helm-charts/ (it is meant to be superceded by the helm chart from this project one day) which addresses this issue.

Thank you very much @axel7083 and @patrungel!

Hi @zakkg3 , any news about when will we have a new release to have this issue solved in the latest release?
I am needing it atm, it would be very helpful 😄

Release comming soon. we are working on some improvements

fixed here https://github.com/zakkg3/ClusterSecret/pull/109


Constructing IP-address from string

There should be a constructor to create an IP4-address from a string.  Very easy fix, just needs doing.

Fixed with https://github.com/hioa-cs/IncludeOS/pull/692


Update developer guide with appendix

Codecov Report

Merging #290 (668be21) into master (5e6e8c1) will not change coverage.
The diff coverage is n/a.

 Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add the candidate requestor to the Agreement

Fixes #770

Codecov Report

Merging #797 (a8080e3) into master (671f161) will increase coverage by 0.27%.
The diff coverage is 57.93%.

@@            Coverage Diff             @@
##           master     #797      +/-   ##
==========================================
+ Coverage   48.42%   48.70%   +0.27%
==========================================
  Files         140      142       +2
  Lines        6914     7014     +100
==========================================
+ Hits         3348     3416      +68
- Misses       2969     2989      +20
- Partials      597      609      +12

Impacted Files
Coverage Δ

pkg/core/consensus/comms.go
0.00% <ø> (ø)

pkg/core/consensus/fixtures.go
0.00% <ø> (ø)

pkg/p2p/wire/topics/topics.go
68.00% <ø> (ø)

pkg/core/consensus/reduction/firststep/step.go
50.42% <11.53%> (-5.24%)
:arrow_down:

pkg/core/consensus/selection/step.go
48.20% <25.00%> (-0.71%)
:arrow_down:

pkg/core/consensus/reduction/secondstep/step.go
73.77% <33.33%> (-1.23%)
:arrow_down:

pkg/core/chain/sequencer.go
66.66% <40.00%> (-33.34%)
:arrow_down:

pkg/p2p/peer/peer.go
37.07% <45.00%> (-0.10%)
:arrow_down:

pkg/core/candidate/validator.go
57.14% <50.00%> (ø)

pkg/util/nativeutils/rpcbus/rpcbus.go
66.66% <50.00%> (-1.42%)
:arrow_down:

... and 11 more

Continue to review full report at Codecov.

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

LGTM apart from a nit on a couple of redundant lines. Feel free to merge directly once that is taken care of


Fix InvalidParameterError in plot_tree

This PR solves
InvalidParameterError: The 'feature_names' parameter of plot_tree must be an instance of 'list' or None. Got ... instead.

For info this problem is solved in https://github.com/scikit-learn/scikit-learn/pull/27292 and will appear in 1.3.1.


Performance improvement

Webpack stats.toJson() is an expensive call. I'm in the process of upgrading to Webpack 4 and noticed when stepping through the code, I'd run out of memory when the stats were being gathered for the client/server. This PR does 2 things.

Updates the getFilename function to only get the assets data since it only needed the chunk names
Allows the consumer to return their own stats JSON for each of the compilers passed to the middleware

An example use of this would be something like:
app.use(webpackHotServerMiddleware(compiler, {
    webpackStatsHandler: (stats) => {
        return stats.toJson({
            all: false,
            chunkGroups: true,
            assets: true
        });
    }
}));

@richardscarrott any thoughts on getting this merged in?

@okcoker I'll aim to look at this shortly -- sorry for the delay

Sooo. yes? no?


Added fetch_contributors.py script.

This script will fetch a list of all code contributors in a repo, and output a list in markdown format for use in a CONTRIBUTORS.md file.

Also, if you want this in a different repo, or folder, let me know.

This is really solid @jminor - thank you!
A few ideas to improve....

I've seen some projects move towards having a CONTRIBUTORS.csv or CONTRIBUTORS.yml instead, since those are more machine readable. Might be useful as an option, but also could be overkill :-)
Bonus points if you could have an accompanying GH action for this - that would be easy then to roll out to all our project repos!

This might not be a good candidate for automation through GitHub Actions. We found that many people don't have full names on the GitHub profiles, and some key contributors did not show up in the list because of the way their changes were landed into our repo. So this script provided a fine starting point, but we still edited by hand.
Can you point me to a specific example of a CONTRIBUTORS.csv or .yml?


Bugfix - @DubboReference parameters do not put into methodParameters correctly!

What is the purpose of the change
@DubboReference parameters can put into methodParameters correctly!
Brief changelog
Rewrite method parameters during  turning registry url to consumer url.
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue. #9597
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[x] Add some description to dubbo-website project if you are requesting to add a feature.
[x] GitHub Actions works fine on your own branch.
[x] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #9599 (7f4454d) into master (8240ea2) will decrease coverage by 0.74%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #9599      +/-   ##
============================================
- Coverage     60.89%   60.15%   -0.75%
+ Complexity      448      445       -3
============================================
  Files          1100     1100
  Lines         44520    44474      -46
  Branches       6478     6446      -32
============================================
- Hits          27111    26752     -359
- Misses        14441    14763     +322
+ Partials       2968     2959       -9

Impacted Files
Coverage Δ

...e/dubbo/registry/integration/DynamicDirectory.java
64.70% <100.00%> (-2.63%)
:arrow_down:

...ookeeper/ZookeeperDynamicConfigurationFactory.java
0.00% <0.00%> (-66.67%)
:arrow_down:

...gcenter/wrapper/CompositeDynamicConfiguration.java
0.00% <0.00%> (-63.34%)
:arrow_down:

...fig/configcenter/TreePathDynamicConfiguration.java
25.71% <0.00%> (-60.01%)
:arrow_down:

...pport/zookeeper/ZookeeperDynamicConfiguration.java
0.00% <0.00%> (-60.00%)
:arrow_down:

.../dubbo/metadata/report/MetadataReportInstance.java
12.50% <0.00%> (-40.63%)
:arrow_down:

.../configcenter/support/zookeeper/CacheListener.java
0.00% <0.00%> (-32.44%)
:arrow_down:

...gistry/client/migration/MigrationRuleListener.java
55.88% <0.00%> (-23.53%)
:arrow_down:

...ng/transport/dispatcher/all/AllChannelHandler.java
68.96% <0.00%> (-20.69%)
:arrow_down:

...mmon/config/configcenter/DynamicConfiguration.java
68.18% <0.00%> (-18.19%)
:arrow_down:

... and 73 more

Continue to review full report at Codecov.

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


Add touch() function to Component

Objective
Make it easy to manually trigger a change for a Component.
Solution
Add a default function to the Component trait. I just adapted @TheRawMeatball's trick  (&mut *component;) into a function and the name is a taken with courtesy from @Ratysz.
I am not sure if this has any memory/performance impacts, or if something like this is even needed, especially because it will be very "visible" as a default function on each Component. So...some discussion needed there is.

I think it's better to add this as a method in the change_detection_impl macro.
I thought there was already set_changed. Might not be user accessible though

What are the use cases when this is needed?
Needing to trigger change detection on something that didn't change means you're using change detection for something else than it's purpose, but maybe I'm missing use cases 🙂

I think it's better to add this as a method in the change_detection_impl macro.
I thought there was already set_changed. Might not be user accessible though

Oh damn am i blind! 🙈🤦♂
This is really embarrassing! 😳 Sorry for the noise...


Should GestureSource.from(gesture) be new GestureSource(gesture)?

Context: http://codereview.cc/D2247#45260

My assumption is that visual closeness of APIs will provide more value across platforms than implementation closeness for L2 authors.

The static method approach might also imply that we can/will add other functionality to the GestureSource type, which would be further deviant from the spec/other platforms.

That argument is sound.
One issue which I consider to be a block is that new GestureSource(gesture) must necessarily return an instance of GestureSource which itself extends from MotionObservable<Gesture>. The problem is in the naming and will definitely cause confusion for callers who expect to get an Observable back.

Is this a concern if source is clearly documented as being an observable? Note that this argument holds for swift and js as well.

If you view this from the POV of a client who's creating an observable stream, I think it's quite unsettling to be getting a GestureSource back. I do think it'll be a problem for swift/js as well.
I think i should mention this: On Java, I can also name my function gestureSource() as a static function, similar to how you do in swift. The reason why I didn't do that is because by default, the IDE suggests GestureSource.gestureSource(). This is unlike swift. The caller can then manually import gestureSource() itself to get rid of the long prefix. If this is what you want me to do, I can do so.
So if the callers do the manual import, then the java code will be 100% visually identical to swift.  That's when I arrived at the GestureSource.from() syntax. This syntax assumes that callers will not manually import from() (which would actually be quite ugly), so it makes the worst case scenario better.

Forking for meta-discussion of Source: https://github.com/material-motion/starmap/issues/108

Let's go with GestureSource.from(gesture) for java. I'll update starmap.

Added java version of example to https://material-motion.github.io/material-motion/starmap/specifications/streams/sources/gestureSource and https://material-motion.github.io/material-motion/starmap/specifications/streams/sources/springSource


Untamed Llama loses AI when "ridable: true"

Timings link
No response
Expected behavior
I wanted the untamed llama to drop me down, or to get tamed when I sit on it
Observed/Actual behavior
When I sat on untamed llama, it stopped moving, it stopped rotating, it stopped doing anything. I have sat on it for an hour and no hearts appeared nor I got kicked down. I also can not open inventory.
Steps/models to reproduce

in config.yml make llama ridable: true
/summon llama
right click it
sit for an hour
observe that llama have done nothing - no tame, no control, no kick down

Purpur version
This server is running Purpur version git-Purpur-1220 (MC: 1.17) (Implementing API version 1.17-R0.1-SNAPSHOT) (Git: cb1775d on HEAD)
You are running the latest version
Previous version: git-Purpur-1212 (MC: 1.17)
Agreements

[X] I am running the latest version of Purpur available from https://purpur.pl3x.net/downloads.
[X] I have searched for and ensured there isn't already an open issue regarding this.
[X] My version of Minecraft is supported by Purpur.

Other
Llamas work as expected when llama ridable: false but strange things happen when You switch it to true.
By the way, I am including this note from discord:
conclusion when llamas ridable: true :
You can not control, tame untamed llama (it loses AI);
You can not control tamed llama while it has no carpet;
You can control tamed llama while it has a carpet.

It is like untamed llama waits for a carpet in order to think about Temper and Tame stuff haha.

actually in purpur.yml not config.yml


Create a Todo List

In this is mini project, you are invited to create a To-do List using either Vanilla Js or React , we leave the design up to your creativity, but the app must contain some features.

[ ] Create a single page todo interface with an input, and the list of todos

[ ] make each todo able to be toggled as finished, each finished todo will be moved into the "finished" category

[ ] a todo should display the task and the date of creation, if finished and moved in "finished" category, it should also display the date when it was finished

[ ] Saved todos in localstorage and load them when page loads

[ ] OPTION: create a filter setting, to filter the todos by certain constraint like date creation ect.

[ ] OPTION: add priorities to todos

[ ] OPTION: add a color picker in creation (pick a color for each todo), if no color is picked choose random one

[ ] OPTION: save todos in a Database

Hi, can I work on this issue?

@chinmaym07 this is the main issue with that previous task (the "reuse-2" will be deleted/changed)

@sarita1124 Yes, you've assigned!


Utils: port over mesh cut code

https://github.com/exokitxr/mc-wasm/blob/3e67da300269d3783db2511f44dbddc67c18b75d/cut.cc#L5
It's needed for some combat special effects.

We should port over that code, including the dependencies, into app-wasm.
We should get the mesh cut APIs working with a test app.
We should add uv tracking and interpolation support to the C++, so we can keep textures when we cut meshes; I've done this before but I can't find the code for the edit. Essentially we can add uvs tracking to the structure: https://github.com/exokitxr/mc-wasm/blob/3e67da300269d3783db2511f44dbddc67c18b75d/xs/src/libslic3r/TriangleMesh.cpp#L45
and then interpolate the uvs during the slice when generating the split triangles: https://github.com/exokitxr/mc-wasm/blob/3e67da300269d3783db2511f44dbddc67c18b75d/xs/src/libslic3r/TriangleMesh.cpp#L752

is done here:
https://github.com/webaverse/app/pull/2117
and here
https://github.com/webaverse/app-wasm/pull/16

is done here:
https://github.com/webaverse/mesh-cut-example

Sphere cut in half
https://user-images.githubusercontent.com/4487857/147933750-265e737b-71b5-4606-8e4e-90553162f6b6.mp4

Here is really great work by @gonnavis synthesizing the solution in JS. https://github.com/gonnavis/annihilate/blob/a538d8cd0dffefb4d6d1a618703b1e3b15311040/src/index.js#L57
It could be ported to C++ but it's probably already good on its own.
We should have a webaverse-toolkit repo to collect these utilities. We can import that module from webaverse core.

New repo for the purpose: https://github.com/webaverse/metaverse-utils
We can make it a package here: https://github.com/webaverse/app/tree/master/packages


Sanitize error message for sensitive requests

This avoids credentials potentially leaking into log messages

Is it possible to have this fix applied to the 0.9.x line?

Is it possible to have this fix applied to the 0.9.x line?

The patch should apply cleanly.
I made a branch https://github.com/apache/arrow-rs/tree/object_store_0.9-dev
If you make a PR targeting that branch I can run the release process to make a 0.9.2 release

Hi Andrew,
Thanks, but I am unable to apply the patch to 0.9 cleanly; it seems to conflict with 0b0d64b6279ac753ec91e32a12b9ee711b6fea29 and 873176078abe54b4503ef15b8d1a2509095331e9.

@alamb / @tustvold is it possible to get a release for this for the 0.9 branch? I don't want to have to spend a week doing a datafusion etc upgrade just to get the fix for this CVE so that our cargo audit builds can once again succeed.
Thanks.

@alamb / @tustvold is it possible to get a release for this for the 0.9 branch? I don't want to have to spend a week doing a datafusion etc upgrade just to get the fix for this CVE so that our cargo audit builds can once again succeed.

I am willing to run the release process for 0.9.0
However I am not sure I will have time to work on the backport of the code to the 0.9.0 branch. If someone else can do that I will handle the logicistics of releasing

@maxburke  -- Note this CVE only applies when using a certain form of authentication with S3. If you aren't using that form the logs won't contain credentials

@alamb We use cargo audit to report on vulnerabilities in our dependency chain. It doesn't know enough to know if the afflicted code paths are used by us or not. The result though is all of our Rust builds are on hold because there's a CVE reported against the versions of object_store on which we and our dependencies (both datafusion and lancedb) are using, and because of that our CI pipeline is failing.
The Cargo version resolution rules do not accept 0.10 as being a compatible update for 0.9; I don't think releasing an incompatible version that fixes the issue is acceptable.
I also do not believe it is onerous to ask the package maintainers to release a compatible patch for something serious enough to warrant a CVE number.

cargo audit has functionality to suppress advisories if you aren't impacted by them - https://github.com/rustsec/rustsec/blob/main/cargo-audit/README.md#ignoring-advisories

I also do not believe it is excessively onerous to ask the package maintainers to release a compatible patch for something serious enough to warrant a CVE number.

If you are willing to help backport this fix, Andrew has kindly offered to cut a release for you. We're all volunteers here, I would ask you understand that our capacity and time is limited.
FWIW a CVE, much like RUSTSEC, is simply an advisory mechanism, in both cases it is expected that downstream projects assess said advisories, determine to what extent they impact them, and act accordingly. In this case a CVE was filed because it was reported via the apache security team, if it had been reported in a different manner it might have been a RUSTSEC advisory instead, there isn't anything special about a CVE that makes it warrant different handling from any other advisory.


Incoming avoidance

Merge incoming avoidance to the master branch
The issues that my reviewers have brought up have been resolved. And it passed all the checks this time so there should be no more conflict. This should now be able to close issue 22
Close #22
:ballot_box_with_check: Definition of Done checklist

[ ] Meaningful title and valuable description in pull request
[ ] Code change tested manually if not covered by automated tests
[ ] Pull request linked to one or more issues/stories
[ ] Acceptance criteria of the linked stories satisfied
[ ] Code change comprehended by those who approved it
[ ] State of target branch maintained or improved after the code change

Something happened need to fix it


reset scenario fixes

Enhanced reset scenario playbook:

fix Get related records Jinja
Added checks for most related modules including attachments
fixed situations where the playbook fails if the created records are deleted manually then the playbook is invoked

Need to raise new PR for release/1.0.2 branch. Closing the PR.


Upgrade to onnxruntime 1.20.0

I've done the steps as usual. Please can you run the tests as my environment isn't great for GPU support.  Also I haven't done anything with the now deprecated training stuff, probably that should be removed now in this release also.

Excellent, and thank you!  I will run the tests (at least the ones that I can) and remove training support before tagging a new version in this repo.  My plan will be to make the training functions return an error, and include a note in the README indicating the correct version to use the training API.  Unfortunate that it was deprecated after @riccardopinosio put all the work into adding the bindings.

Yes pretty painful they killed it off, and so soon after adding it. Riccardo is working on an alternative in Hugot however!

Tested on Windows x86_64, Linux x64_64, and Linux arm64 (all with CPU only).  Currently waiting for my build of onnxruntime 1.20 on aarch64 with CUDA to finish, but I have no reason to expect it not to work.  If it doesn't, I'll just open a new issue I suppose.  Thanks for the help!

For what it's worth, I have now tested it on aarch64 using CUDA and TensorRT as well.  It worked without issue:
goos: linux
goarch: arm64
pkg: github.com/yalue/onnxruntime_go
BenchmarkOpSingleThreaded
BenchmarkOpSingleThreaded-6   	       1<PHONE_NUMBER> ns/op
BenchmarkOpMultiThreaded
BenchmarkOpMultiThreaded-6    	       1<PHONE_NUMBER> ns/op
BenchmarkCUDASession
BenchmarkCUDASession-6        	       2	 766380430 ns/op
BenchmarkTensorRTSession
BenchmarkTensorRTSession-6    	       4	 260602978 ns/op


Publish v0.6.0 to NPM

I was keen to check out the fix for #329 (thank you!), but it looks like v0.6.0 was tagged but not published to NPM yet.
Please could you publish?

@veritem Thank you!

For now this worked for me:
"eslint-plugin-vitest": "github:veritem/eslint-plugin-vitest#semver:^0.6"

Can you please start checking in ./dist to this or another repo while you do not have npm access?
Alternatively, publish it on jsr.io!

Can you please start checking in ./dist to this or another repo while you do not have npm access?

Please don't check in dist. Put it in another repo.

Alternatively, publish it on jsr.io!

That would be nice.

👋 any updates on this?

I haven't heard back from npm/gitHub support since June, and I don't know anyone who works there.  still waiting...

If you didn’t receive any answer in months I don’t think anyone will answer you anyway. In your position I’ll start thinking in creating another account and publish it as a different name.

In your position I’ll start thinking in creating another account and publish it as a different name.

I second this. Last year I try to reach the support to help someone at work, we never received any answer.

oh, just fork, and temporary publish    "eslint-plugin-vitest-2": "^0.6.1" , wait you npm account

Is there anything that can be done? There really isn't a way to use the latest version of this package right now.

Agree, a new version would be very welcome, especially with the TSESLint peer dep fixed.

oh, just fork, and temporary publish "eslint-plugin-vitest-2": "^0.6.1" , wait you npm account

That’s what @veritem did: https://www.npmjs.com/package/eslint-plugin-vitest-2
It works on my side (do not forget to add the -2 to your package.json and your ESLint config file.)

@meduzen that package does not belong to me.

@meduzen It is Yingchun Tian @tianyingchun who is forked the package to https://www.npmjs.com/package/eslint-plugin-vitest-2 on npm. Given @veritem 's issues with their npm account, I think this is a reasonable stop-gap solution and gives us access to the latest version.

I have moved on guys!
npm install eslint-plugin-vitest-latest

open an issue if something breaks

We can a new home guys
npm install @vitest/eslint-plugin --save

Woah, even better, congrats!

My own project has switched to the new version and works fine，@hyperse/eslint-config-hyperse


feature request: add /newissue command

This should add a new github issue. Syntax could be:
/newissue oh man help i am not good with computer

I would love a pull request for this!


Added jsbeautifier to banks_description

when creating/editing the bank_description, it uses jsbeautifier. noticeable when you enter it as one line string.

have you tested it? A screenshot does not tell me if you have tested the full functionality
have you edited some "untidy" JSON and it gets nicely formatted without making any functional changes to the JSON data?

yes, i when i entered it as one string, removing spaces in keys and values, it reformats it to the pretty version


Deduplicate orderer server TLS root CAs

When the orderer TLS root CAs are updated, an aggregation of all root TLS CA certificates over all channels is injected into the PredicateDialer.
Then, upon client TLS handshake, a fresh TLS config object is built (for orthogonal purposes), however the operation entails parsing of all
root CAs all over again.
In case the orderer is part of too many channels, this induces a high and unnecessary processing overhead.
This commit simply performs a deduplication of the bespoken TLS root CA certificates prior to updating the root CAs.
Change-Id: I21b2ed483afc9595c2ccd7fbe9ec0cf475cc5f62
Signed-off-by: yacovm<EMAIL_ADDRESS>

backport?

backport?

You just merged the last one them.


15.0.0-alpha.1 RxSwift Update

Can u update RxSwift to 6.0 in SPM for 15.0.0-alpha.1 version?

Does #2120 answer your question?

Does #2120 answer your question?

Does #2120 answer your question?

it does not work when use cocoapod
what should I do next?

Does #2120 answer your question?

it does not work when use cocoapod
what should I do next?

it does not work when use cocoapod
can u update it both

it does not work when use cocoapod
can u update it both

it does not work when use cocoapod
can u update it both

pod 'Moya/RxSwift', :git => 'https://github.com/peagasilva/Moya.git', :branch => 'rxswift-6'  # https://github.com/Moya/Moya
ummm, it worked

it does not work when use cocoapod
can u update it both

pod 'Moya/RxSwift', :git => 'https://github.com/peagasilva/Moya.git', :branch => 'rxswift-6'  # https://github.com/Moya/Moya
ummm, it worked

s.subspec "Core" do |ss|
    ss.source_files  = "Sources/Moya/", "Sources/Moya/Plugins/"
    `ss.dependency "Alamofire", "~> 5.0"`
    ss.framework  = "Foundation"
  end

  s.subspec "ReactiveSwift" do |ss|
    ss.source_files = "Sources/ReactiveMoya/"
    ss.dependency "Moya/Core"
    ss.dependency "ReactiveSwift", "~> 6.0"
  end

  s.subspec "RxSwift" do |ss|
    ss.source_files = "Sources/RxMoya/"
    ss.dependency "Moya/Core"
   ` ss.dependency "RxSwift", "~> 5.0"`
  end

It seems like to update the dependency lib?

Can u update RxSwift to 6.1.0 in SPM for 15.0.0-alpha.1 version?

+1

When release it? RxSwift 6.0 released about 4 months ago.

still relevant

Closing as per #2120


Aliases

@lmammino I have added a new alias property for passenger.json since it used to be called modrails. How should we handle this? What if someone search for "modrails" or "mod rails". We need the results to show "Phusion Passenger".

Great idea. We should take care to updating the schema file to ensure proper validation


HMMLearn v0.2.0

For some reason, one of the python 3.6 win builds hung, but I don't think that's related.
We can merge this PR if it's ready to go!

Finally!  After adjusting so many different things!


[3.0] p2p invoke support wildcard url match.

What is the purpose of the change
see #3168
Brief changelog
Solution: Extend Properties.class to support regex matching keys.
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[x] Add some description to dubbo-website project if you are requesting to add a feature.
[x] GitHub Actions works fine on your own branch.
[x] If this contribution is large, please follow the Software Donation Guide.

LGTM

Codecov Report

:exclamation: No coverage uploaded for pull request base (3.0@4bfeeb2). Click here to learn what that means.
The diff coverage is 90.90%.

@@          Coverage Diff           @@
##             3.0    #8247   +/-   ##
======================================
  Coverage       ?   63.70%
  Complexity     ?      310
======================================
  Files          ?     1073
  Lines          ?    45183
  Branches       ?     6782
======================================
  Hits           ?    28786
  Misses         ?    13163
  Partials       ?     3234

Impacted Files
Coverage Δ

...org/apache/dubbo/common/utils/RegexProperties.java
90.00% <90.00%> (ø)

...a/org/apache/dubbo/config/ReferenceConfigBase.java
69.49% <100.00%> (ø)

Continue to review full report at Codecov.

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


OpenCL - buffer creation fails with Integrated Graphics

Description
Attempting to run OpenCL code with Intel Integrated graphics gives the error:
[ RUN      ] ProbDistributionsBernoulliCdf.opencl_matches_cpu_small
exception thrown in signature  const std::tuple<std::vector<int, std::allocator<int> >, Eigen::Matrix<stan::math::var_value<double, void>, -1, 1, 0, -1, 1> >&]:
unknown file: Failure
C++ exception with description "initialize_buffer: clCreateBuffer CL_INVALID_HOST_PTR: Unknown error -37" thrown in the test body.

Which has also been seen in this forum post.
I believe this is due to the use of the CL_MEM_USE_HOST_PTR flag when creating matrix_cl objects:
        buffer_cl_
            = cl::Buffer(ctx, CL_MEM_READ_WRITE | CL_MEM_USE_HOST_PTR,
                         sizeof(T) * size(), A);  // this is always synchronous

Sharing the memory pointer to the GPU appears to cause an issue with integrated graphics that have shared memory for the CPU and GPU, and the CL_MEM_COPY_HOST_PTR flag should be used for these devices instead (and fixes the issue for me locally).
Given that this incurs a copy-cost, we wouldn't want to use it for all cases, only for integrated graphics. I think a simple approach here would be to add a new make/local flag: INTEGRATED_OPENCL, which would request the use of this CL_MEM_COPY_HOST_PTR flag:
#ifdef INTEGRATED_OPENCL
        buffer_cl_
            = cl::Buffer(ctx, CL_MEM_READ_WRITE | CL_MEM_COPY_HOST_PTR,
                         sizeof(T) * size(), A);  // this is always synchronous
#else
        buffer_cl_
            = cl::Buffer(ctx, CL_MEM_READ_WRITE | CL_MEM_USE_HOST_PTR,
                         sizeof(T) * size(), A);  // this is always synchronous
#endif

@SteveBronder @rok-cesnovar I'm completely new to the OpenCL space so make sure to let me know if there's a better or more performant option here!
Current Version:
v4.4.0

Thanks for finding this! What if we pulled out the bitfield like
constexpr auto copy_or_share = CL_MEM_COPY_HOST_PTR * INTEGRATED_OPENCL | (CL_MEM_USE_HOST_PTR * !INTEGRATED_OPENCL)

Then I think putting that in the mem flags would work?

(I forget bitfield logic and am on phone atm but it's something like that)

Neat! I'll do some testing locally and open a PR

Your bitfield magic seems to work like a charm. I'll finish out running all of the OpenCL tests locally to make sure that the integrated implementation doesn't have any hidden issues

Is this issue supposed to be fixed?
I am experiencing a similar issue with Cmdstan 2.32.2.

Have you used the fix added in the PR? Adding INTEGRATED_OPENCL=true to make/local

Oh I see... This works now!
So there is no way to automatically detect the Integrated Graphics and switch the flag in this case?

Oh I see... This works now!
So there is no way to automatically detect the Integrated Graphics and switch the flag in this case?

Unfortunately not, this would require a bit more of a dedicated build system (e.g., a configure script) then we currently use.
But I've opened a PR to add this makeflag to the doc: https://github.com/stan-dev/docs/pull/649

I'm sorry, but I don't understand: why cannot we detect this at runtime?

It has to be detected at compile-time, not runtime - since it requires different code to be compiled

Well, it's just a different flag to pass to cl::Buffer. This can be easily changed at runtime.
Basically, this would mean keeping the same code, but replacing INTEGRATED_OPENCL with some global variable which is initialized once and for all.

Note the constexpr type for the flags, this means that it has to be known at compile time

I'm looking at the OpenCL documentation:
https://man.opencl.org/clCreateBuffer.html
It doesn't say that flags are constexpr. It's only copy_or_share, which, for some reason, has been set constexpr in this patch. There is no reason to revert this.

Sorry, I linked to the C OpenCL library. The C++ library is the same: https://github.khronos.org/OpenCL-CLHPP/classcl_1_1_buffer.html#a66ee6c9c837b5f74f02bef8f90459b5c

(Also, another point: is there any reason to avoid the copy cost when we can? It seems like not copying will be costly as well, because the GPU will need to access these data in general RAM.)


3.5. Implementation Details：Network Architecture.

I conducted the test and your work has achieved good results. I have some questions to ask.
As described in the article The feature backbone consists of the first 4 blocks of ResNet-50 [12], which outputs the feature maps of 1024 channels. Why use the ResNet-50 backbone instead of another deeper network? Is there any specific reason to use ResNet-50?
Thank you!

Hi. In this work we use ResNet-50 for fair comparisons with previous works CFOCNet and FamNet.
I have not tested other deeper backbones. Sorry that I really have no idea how backbones like swin and convnext will perform. I tested ResNet-18 and the MAE on validation set increases to around 18. I notice a recent work (link) obtains better performance but using only ResNet-18, unfortunately the code is not released.
Whether better backbone can improve performances on counting tasks is doubtful. According to my experimence on class-specific counting tasks, simple VGG backbone may performs better than ResNet. Maybe you can test other backbones and see the impact of backbones on class-agnostic counting.


Integration requries reconfiguration: Authentication failed while fetching tplink_deco-clients data: Data empty

Problem
Yesterday the integration has started saying "Integration requires reconfiguration" I goto the integration and press reconfigure, the fields are already completed (and haven't changed) I press submit and get a successful message but then the error starts again
TP-Link Deco Version
Current
Router Model
Deco m5
Debugging Information
None

Please read through the instructions in the reconfiguration prompt.

same issue here :(

@amosyuen so in the instructions in the reconfiguration prompt dont mention anything really, it takes you to the standard login credentials etc under the readme file which I have followed well i.e. username is admin and I have made another manager account that I am using on my mobile.
Also I am using the latest version of this integration but still no joy. Can you help please?

Maybe I need to make it more explicit. From https://github.com/amosyuen/ha-tplink-deco#login-credentials:

Also whenever the owner logs in, all other sessions for the owner will be logged out. So if you log in with the owner credentials in the mobile app, it will cause the integration to be logged out, sometimes resulting in 403 errors

Could you confirm whether you are logging in with owner credentials elsewhere?

Maybe I need to make it more explicit. From https://github.com/amosyuen/ha-tplink-deco#login-credentials:

Also whenever the owner logs in, all other sessions for the owner will be logged out. So if you log in with the owner credentials in the mobile app, it will cause the integration to be logged out, sometimes resulting in 403 errors

Could you confirm whether you are logging in with owner credentials elsewhere?

No need to make it more explicit mate its all understandable. However I have followed your instructions when I first started using your integration I.e.
"Recommend that you create a separate manager account with full permissions to manage the router manually in the mobile app and use the owner credentials only for this integration. If you need to use the owner account to do some manual management, recommend disabling this integration temporarily."
Does that make sense?!

Ok thanks for confirming, then please turn on debugging and paste logs from when integration config fails. https://github.com/amosyuen/ha-tplink-deco/blob/master/DEBUGGING.md#general-steps

I too have the same issue, I have rolled back to v3.0.0 which following a single reconfiguration message has remained stable since.  I also tried (never needed before v3.1.0) creating a separate logging for my phone Deco app, this made no difference on 3.1.0, the integration seems to "work" for a time but then stops with the same reconfiguration message over and over.

@amosyuen so in the instructions in the reconfiguration prompt dont mention anything really, it takes you to the standard login credentials etc under the readme file which I have followed well i.e. username is admin and I have made another manager account that I am using on my mobile.
Also I am using the latest version of this integration but still no joy. Can you help please?

Hi
Can you explain me how/where you create a New manager account for use in a mobile DECO app?
Thanks
PS: sorry for OT

@amosyuen so in the instructions in the reconfiguration prompt dont mention anything really, it takes you to the standard login credentials etc under the readme file which I have followed well i.e. username is admin and I have made another manager account that I am using on my mobile.
Also I am using the latest version of this integration but still no joy. Can you help please?

Hi Can you explain me how/where you create a New manager account for use in a mobile DECO app? Thanks
PS: sorry for OT

You need to:

Head to TP-Link online and create a new account (with a different email address)
In the app you add that account as a manager using the "new" email address
You need to log out of the app from your original account
Re-login using the new account you've just created.

Hope this helps. :-)

@amosyuen so in the instructions in the reconfiguration prompt dont mention anything really, it takes you to the standard login credentials etc under the readme file which I have followed well i.e. username is admin and I have made another manager account that I am using on my mobile.
Also I am using the latest version of this integration but still no joy. Can you help please?

Hi Can you explain me how/where you create a New manager account for use in a mobile DECO app? Thanks
PS: sorry for OT

You need to:

Head to TP-Link online and create a new account (with a different email address)
In the app you add that account as a manager using the "new" email address
You need to log out of the app from your original account
Re-login using the new account you've just created.

Hope this helps. :-)

Thanks
I'm going to try it
Regards

Have also the same problem.
Trying to get you logs as per manual, but getting this error:
The system cannot restart because the configuration is not valid: Integration error: logs - Integration 'logs' not found.
Please advice how to handle this

Ok, got following error:
Эта ошибка возникла в кастомной интеграции.
Logger: custom_components.tplink_deco
Source: custom_components/tplink_deco/config_flow.py:91
Integration: TP-Link Deco
First occurred: 1 февраля 2023 г., 23:57:43 (3 occurrences)
Last logged: 00:00:49
Error testing credentials:
However, easily login to web admin panel using password I enter into integration and the IP address of server

OK, so even with a separate account and having rolled back to v3.0.0 I am still getting this error message that I've never had before...  This starts when my deco's reboot at 02:30 in the morning so I know there is no one else logging in to the admin account at that time.  I wonder if there is a limit on the number of retries maybe or the duration retries occur?  I'm kinda clutching at straws here though now as every morning I have to reconfigure the integration just to be able to track devices.
Logger: custom_components.tplink_deco
Source: helpers/update_coordinator.py:168
Integration: TP-Link Deco (documentation, issues)
First occurred: 02:30:47 (4 occurrences)
Last logged: 07:35:05
Error requesting tplink_deco-decos data: Cannot connect to host <IP_ADDRESS>:80 ssl:default [Connect call failed ('<IP_ADDRESS>', 80)]
Error requesting tplink_deco-clients data: Cannot connect to host <IP_ADDRESS>:80 ssl:default [Connect call failed ('<IP_ADDRESS>', 80)]
Authentication failed while fetching tplink_deco-clients data: Data empty. Likely caused by logging in with admin account on another device. See https://github.com/amosyuen/ha-tplink-deco#login-credentials.
Authentication failed while fetching tplink_deco-decos data: Data empty. Likely caused by logging in with admin account on another device. See https://github.com/amosyuen/ha-tplink-deco#login-credentials.

Thanks for the logs @antonyslater, I added a retry for that empty response case in v3.1.1 see if that helps.

Thanks for the logs @antonyslater, I added a retry for that empty response case in v3.1.2 see if that helps.

Hi @amosyuen,
I have left v3.1.2 running since you posted it yesterday, and it did not need reconfiguring beyond the initial one just after I updated it.  However I log on this morning to find that it has failed once again in the night (02:30) when my deco's are force rebooted...  Not sure how things have changed but I never needed to do this daily before v3.1.0 and my deco's have been rebooting nightly for at least 9 months.
I will attach the log entries I have obtained this morning so as to not "spam" this thread...  Some from 02:30 and some from 08:26, hope these help.
02-40-25 (3 occurrences).txt
08-26-50 (9 occurrences).txt
08-26-52 (4 occurrences).txt
02-30-29 (2 occurrences).txt
02-30-29 (18 occurrences).txt
02-30-35 (54 occurrences).txt

I added some code to clear auth if connection failed, which resolves the problem for me when I reboot the decos. Please try https://github.com/amosyuen/ha-tplink-deco/releases/tag/v3.1.4 and see if that helps.

I added some code to clear auth if connection failed, which resolves the problem for me when I reboot the decos. Please try https://github.com/amosyuen/ha-tplink-deco/releases/tag/v3.1.4 and see if that helps.

Updated and left over night to see what happens... seemed OK when I first loaded HA but after about 5 mins I got the usual reconfigure message...  These are my logs form this morning.
02-31-02 (2 occurrences).txt
02-31-35 (3 occurrences).txt
02-31-38 (2 occurrences).txt
02-31-38 (18 occurrences).txt
02-32-05 (32 occurrences).txt

Fixed my issue. Appeared that the integration can't be configured via hass.io . Managed to make the configuration via local network.

Todays logs, same reconfiguration required this morning... One has a different feel to it today though (32 occurrences)... I will stop sending these now unless you ask for them. :-)
02-30-57 (1 occurrences).txt
02-30-57 (2 occurrences).txt
02-31-30 (2 occurrences).txt
02-31-30 (18 occurrences).txt
02-31-36 (32 occurrences).txt

@antonyslater

Cannot connect to host errors are expected when you are rebooting the deco and shouldn't really affect the integration.
Timeout are errors from deco being slow. You can try tweaking timeout config params added in  https://github.com/amosyuen/ha-tplink-deco/releases/tag/v3.3.0
Future exception was never retrieved is a legit error that should be fixed in https://github.com/amosyuen/ha-tplink-deco/releases/tag/v3.3.0

Cheers @amosyuen I have updated now, will see how things go overnight again and let you know.  Thanks again for your efforts, it really is much appreciated. :-)

Is the latest version already pushed out on HACS?

Is the latest version already pushed out on HACS?

I had to restart for it to show as an update available in HACS, although I believe if you go into the integration in HACS you have the option to download the latest version even before it is "reported" as having an update available.

Is the latest version already pushed out on HACS?

I had to restart for it to show as an update available in HACS, although I believe if you go into the integration in HACS you have the option to download the latest version even before it is "reported" as having an update available.

Nice one thank you

@antonyslater

Cannot connect to host errors are expected when you are rebooting the deco and shouldn't really affect the integration.
Timeout are errors from deco being slow. You can try tweaking timeout config params added in  https://github.com/amosyuen/ha-tplink-deco/releases/tag/v3.3.0
Future exception was never retrieved is a legit bug that should be fixed in https://github.com/amosyuen/ha-tplink-deco/releases/tag/v3.3.0

Hi @amosyuen, great news, no configuration error message this morning....  I updated yesterday and set the new settings as follows:
Timeout error retry count: 10
Timeout seconds: 30 (default I believe)
THANK YOU again for all your help with this. :-)

Glad that is seems resolved so far!

Closing issue as resolved


Improve shareability of computations using containers

Outcomes

Reduce software dependency
Reduce average image size
Manage and maintain containers of computations more easily
Increase visibility and understanding of what containerized computations are available

Learning Resources

Introduction to Singularity: A Container Platform for Scientific and High-Performance Computing

Open Source Tools To Consider

Singularity HPC is optimized for managing containers in an HPC environment
naked-singularity: A repository of definition files for building Singularity containers around the software applications, frameworks, and libraries you need to run on high-performance computing systems.
Singularity Registry Server is a server to provide management and storage of Singularity images for an institution or user to deploy locally. It does not manage building but serves endpoints to obtain and save containers.

@dylanmartin and @rssk thoughts on the value of this issue? It seems more like a mega-task or architecture task if antyhing. This came from Vince and Sandeep's roadmap earlier in the year and I'm unsure of its relevance after your latest work.

There are some good thoughts here but I think it needs to be broken down more to take action on it.


Device disconnected, reconnection impossible

Hello,
I use this add-on to connect a Warema Stick from a RPi 3 (USBip server) to my main RPI4 where HA runs.
On the first start of the system (e.g after piwer outage) the add-on starts correctly and the connection stays active until some network error stops the connection.
However, instead of detaching the device cleanly (so it can be reattached after a restart of the add-on), the add-on gets into some kind of error state where the device isn't available on the HA host, but is also not detached, leading to a fault that can only be fixed by restarting the USBip server.
The Log outputs the following message when attempting to reconnect while in this state:
cont-init: info: /etc/cont-init.d/create_devices.sh exited 0
cont-init: info: running /etc/cont-init.d/load_modules.sh
cont-init: info: /etc/cont-init.d/load_modules.sh exited 0
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service legacy-services: starting
services-up: info: copying legacy longrun usbip (no readiness notification)
services-up: info: copying legacy longrun usbipd (no readiness notification)
[19:09:14] INFO: Starting the USBIP Daemon
[19:09:14] INFO: Attaching usbip devices
usbipd: info: starting usbipd (usbip-utils 2.0)
usbipd: info: listening on <IP_ADDRESS>:3240
usbipd: info: listening on :::3240
s6-rc: info: service legacy-services successfully started
++ mount -o remount -t sysfs sysfs /sys
++ /usr/sbin/usbip --debug attach -r <IP_ADDRESS> -b 1-1.2
usbip: debug: usbip.c:129:[run_command] running command: `attach'
usbip: error: tcp connect
[19:09:17] WARNING: usbip crashed, halting add-on
[19:09:17] INFO: usbip stopped, restarting...
s6-rc: info: service legacy-services: stopping
usbipd: info: shutting down usbipd
[19:09:17] INFO: usbipd stopped, restarting...

The add-on should normally detach devices when stopped, but it doesn't do this in this case...
How could I fix this?
Best regards
Aaron Eisele

@Aaroneisele55 The most recent update I made was adding a "shutdown" script https://github.com/irakhlin/hassio-usbip-mounter/blob/main/rootfs/usr/bin/unmount_devices
If you shutdown the addon it will correctly detach from the server and thus the device can be reconnected. Unfortunately if there is some kind of network error or power outage and the script does not run the detach will not happen. From playing around with usbip outside of home assistant I have noticed that this issue will occur regardless. If the client does not correctly detach from the server the server ends up in a weird state. I will have to look into this to see if its something that can easily be solved.

I just thought about it and I think I have found a possible solution:
If the add--on crashes due to not being cleanly detached at some point, couldn't we just run the detachment script (from  #6 ) when usbipd crahes, and then reattach the devices so the problematic state is resolved?

I just thought about it and I think I have found a possible solution:
If the add--on crashes due to not being cleanly detached at some point, couldn't we just run the detachment script (from #6 ) when usbipd crahes, and then reattach the devices so the problematic state is resolved


Fix tests for faraday 2.2

Coverage: 100.0%. Remained the same when pulling 5074d36d85b2997f534510f1fa157bd8a7663a47 on fix_tests_for_faraday_2.2 into 88548edff9dc1326d406ca86d507eb4f27e6d271 on master.


Rewrite tests to use Jasmine insted of QUnit

PR #216 fails probably because of some QUnit problem. So I rewrote all tests to use Jasmine since it's also used by the editor.
I also updated all the dependencies to the latest versions.

Wow lot of work, good job !! :+1:
Are you sure it was related to Qunit? Can't you merge our work from #216 in your local branch and see if it fixes the problem?

Yes, I tried it locally and it looks like it's working.

Great! Let's see how it goes.


Feature/363 time series summary stats

Closes #363
Implement function that saves a csv file with summary statistics for offset50, ndvi and total_precipitation time series.

This is the current structure of the csv output file


Create script to automate start of cyberway with Golos

Implemented:
Script scripts/boot-sequence.py which start Cyberway & Golos in Docker container. This script requires:

execute from keosd container (can be launch from any directory)
start from clean environment (nodeos with empty blocklog & state, keosd doesn't have any wallet)
installed python3 and python3-numpy in container
running nodeos available at http://nodeosd:8888
compiled contracts at /opt/cyberway/bin/data-dir/contracts/ inside of container

PR #189


upgrade to latest api's

fixes to README
move to latest watson api's
fixed lint issues
fixed queries to work with latest watson discovery api's
update links in UI that point to IBM developer sites
fixed jest test cases

fixes #69

great job @rhagarty !


IMP for non-GO

If I observe a mutant mouse has a phenotype P, is my observation backed by an
IMP? Or is this intended purely from phenotype->function?

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 1 May 2015 at 6:06

Hi @cmungall We have been considering your question, but I'm afraid we require a bit more clarification. Can you elaborate a bit, or perhaps give us some examples?
Thanks!
Marcus & Suvvi

Hi @cmungall - do you still have questions regarding this? We are still confused by the question and would appreciate more explanation. Thanks!

We think this is more of annotation policy issue. Closing. Reopen if you feel term adjustments are needed.


[BUG]: [SOL-DEV | when we go for re-issue summon flow and then click on skip&continue button, the below screen appears ]

Describe the bug
when we go for re-issue summon flow and then click on skip&continue button, the below screen appears
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

https://jam.dev/c/5742dc10-2a30-4af6-8dec-d0baab4d5af8
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Tested in dpg-dev, working as expected.
https://jam.dev/c/4c904fb4-45d7-40a2-90cc-6accefc168ca
cc: @Ramu-kandimalla @rajeshcherukumalli


More than 1 swift file for the bidirectional scenario does not build

When adding a swift file  solve.swift here in the CmakeLists.txt  of the library, the generation of the bridge code fails.
FAILED: lib/fibonacci/include/fibonacci/fibonacci-swift.h /Users/geometryfactory/afabri/swift-cmake-examples/3_bidirectional_cxx_interop/build/lib/fibonacci/include/fibonacci/fibonacci-swift.h
cd /Users/geometryfactory/afabri/swift-cmake-examples/3_bidirectional_cxx_interop/lib/fibonacci && /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/swiftc -frontend -typecheck -I/Users/geometryfactory/afabri/swift-cmake-examples/3_bidirectional_cxx_interop/include fibonacci.swift;solve.swift -sdk /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX14.4.sdk -module-name SwiftFibonacci -cxx-interoperability-mode=default -emit-clang-header-path /Users/geometryfactory/afabri/swift-cmake-examples/3_bidirectional_cxx_interop/build/lib/fibonacci/include/fibonacci/fibonacci-swift.h
<unknown>:0: error: unable to load standard library for target 'x86_64-apple-macosx14.0'

This also happens with the code of  PR #9
Ideally I want the bridge code generator only getting applied on one of the swift files, and the other swift files are the backend code doing the real work. I think this might make a nice scenario for your repository.

error: unable to load standard library for target 'x86_64-apple-macosx14.0'

It looks like I missed the COMMAND_EXPAND_LISTS as you pointed out in #9. Adding that to the custom command should fix it.

Ideally I want the bridge code generator only getting applied on one of the swift files, and the other swift files are the backend code doing the real work.

So normally what I do for C interop is have two libraries, one that does the actual work in Swift, and one that exposes the API that I want in C. Not unlike how folks tend to expose C++ libraries through a pImpl'd facsimile API over the top of their C++ library rather than exposing the C++ directly. In think in terms of "public interface", splitting them into separate libraries probably makes sense.
One of the goals was to make that less required with C++ interop so that you could implement parts of an existing codebase in Swift without needing to do a lot of bridging work. So the idea here is that
What eventually ends up in the generated header is influenced by access level. public access will show up in the public interface, both in Swift and in the generated header. Something private won't, so you can adjust things a bit that way too.

This should be fixed as of https://github.com/apple/swift-cmake-examples/pull/9. Please verify.


Fixed wrong comment in code snippet Indices And Ranges

Summary
Fixed wrong code comment introduced by #40607 in one of the snippets for Indices And Ranges.

Sorry @BillWagner, I must have copy/pasted.


remove redundant schema def of epoch

Problem
Epoch def is added to BorshSchema twice.
Summary of Changes
Remove the redundant Epoch def addition to BorshSchema.
Fixes #

Yeah. fix in 2.0 sounds good to me.


After playing an Alarm with Music, the device's volume is set to the Alerts volume (instead of whatever it was)

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:
Setup:
"Alexa set volume 3" (verify level with "Alexa what time is it")
"Alexa set alarm volume 8"
"Alexa set an alarm to music at [some time a minute or two from now]"
-- alarm goes off --
"Alexa cancel alarm"
"Alexa what time is it" (volume level is now 8 instead of 3)
"Alexa volume down" (volume level will drop to 7)
What is the expected behavior?
Alarm volume should not get conflated with system volume.
What behavior are you observing?
Alarm volume for music alarms is getting set onto the system volume.
Provide the steps to reproduce the issue, if applicable:
steps provided in the summary. This happens on current versions of AVS SDK as well as on several versions of the Echo.
Tell us about your environment:

What version of the AVS Device SDK are you using?
  <1.26.0>

Tell us what hardware you're using:

[x] Desktop / Laptop
[ ] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[x] Linux
[ ] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

Hello @brett-lynnes,
Is this issue still experienced on the latest version of the SDK (3.0)?

Yes


Usign cylon-rolling-spider with Meteor

Hi!
Recently I have been starting to develop APPs based on Meteor, and I would like to develop an APP that allow users to drive a rolling spider using your package.
Despite Meteor has not implemented a "require" method to load NPM packages, I was capable of loading packages using meteorhacks:npm and cosmos:browserify (using chalk as an example). However, when I am trying to do the same with your package an error is triggered.
Do I have to set any additional dependence to the package or do any additional step to make it work? Right know I am a bit lost U.U

Hey, I'm having trouble getting it working, did you have any luck figuring it out?

Nope, I am looking for a different solution but right now I see impossible to make it works with Meteor U.U


Fixed index out of range panic when channels==4 in ToImage function

In the ToImage function, if the channels variable was 4 then the function would throw an index out of range error.
Consider the case in which channels == 4, but the increment is x=x+3:

Iteration 0:

y = 0
x = 0
step = 1000 # made up for the example
c.B = 0
c.G = 1
c.R = 2
c.A = 3

Iteration 1:
If x = x+3, then you end up overwriting the B channel from the previous iteration:

y = 0
x = 3
step = 1000
c.B = 3
c.G = 4
c.R = 5
c.A = 6

However, if the increment is x = x+4:
y = 0
x = 4
step = 1000
c.B = 5
c.G = 6
c.R = 7
c.A = 8

This PR ensures that there are not out of range errors by setting the increment to channels.

LGTM. Thanks for the efforts.


Pretrained model weights

Thank you for sharing @lucidrains ! Really fantastic work. I watched Yanniks video on perceiver and this repo really helps to go one or two levels deeper and understand the paper.
However, for applying this new architecture to practical applications, most of us without 100s of TPUs at our disposal, require pretrained models on imagenet, audioset etc. Could anyone that has successfully trained the model, can share weights?

Would be glad to have some weights for audio tasks as well!

Same as #34 you can find some pre-trained weights in the author's repo: https://github.com/deepmind/deepmind-research/tree/master/perceiver#colabs


Why not directly send to axiom endpoint on browser

https://github.com/axiomhq/next-axiom/blob/ba5dedce6a4100bcee7d4ada078a36b4c5a68941/src/platform/generic.ts#L27
Based on this line, borwser logs will send to next server then next server will send to axiom.
This is redundant because:

the token is still exposed
directly send to axiom work fine, no CORS issue
a bunch of requests will be used in this solution, requests means money

I believe this is to proxy request so there are not blocked by in browser ad blocks.

I believe this is to proxy request so there are not blocked by in browser ad blocks.

I don't think this should be such a high priority issue.

It's a feature of the logging library, not an issue.

Then why sdks of sentry or mux they directly send request from client?

Hey, requests are proxied for two reasons:

Requests will get blocked (you can see this with other Web Vitals or analytics libraries that send directly)
The proxy extends the Web Vitals to add geo information.

@Jokcy would you prefer an option to send directly to us?
Feel free to re-open, closing for now.


Completion for chat text is causing significant pauses during typing

Issue
This issue seems to be new with 0.60.0 (perhaps because it offers so many more completion options). I find typing in the chat very painful because I will get ~ 1 second pauses as it tries to figure out the set of completions to present.
I would be completely happy to just have an option to disable this, since I almost never select anything from the completion and even if there is no delay, the white box popping up constantly is somewhat distracting.
Sorry to be asking for some much. I really appreciate this tool.
Version and model info
Aider 0.60.0
Model Sonnet

This looks like a duplicate of #930. Please see the comments there for more information, and feel free to continue the discussion within that issue.
I'm going to close this issue for now. But please let me know if you think this is actually a distinct issue and I will reopen this issue.


Merge official and external third party exporter-listing

This merges the official third party exporters and the external ones into one category structure. the official ones are, as proposed by @juliusv, marked.
The surrounding text was adapted as well to reflect the new structure.

Ah, could you do bold instead of italic for the official ones? Stands out more...

@juliusv @brian-brazil
everything should be implemented, except for @juliusv comment on the jmx-exporter as I'm not absolutely sure what he's referring to.

@juliusv @brian-brazil @sdurrheimer
Anything more to do before this can get merged? It's lying around here for quite a while now without much activity.

@grobie
Currently the sections are more or less sorted like "if I have seen enough exporters of a kind to justify a category, I'll add it".
Sorting alphabetically would be a possibility, but I think that we should have a special case for miscellaneous then, because having the miscellanii in the middle somewhere would be alphabetically correct but I'd find it strange, because miscellaneous is basically a catch-all that gets everything we can't sort anywhere other.

On the other hand, using alphabetical sorting here might be not too useful, as the category-names are even more "chosen" than the exporter-names. So it might not be too helpful regarding the structure, although certainly possible.

I'm fine keeping miscellaneous at the bottom. While it's true that a category name might not be what I had in mind initially, I'll get at least used to them and at that point it becomes easier to scan for an exporter in the list.

@grobie

While it's true that a category name might not be what I had in mind initially

What did you have in mind initially?
I can try to come up with a reordering-scheme if desired, maybe we have something better than alphabetically (like topic-ish or something). Question is then whether we tackle that in this PR or leave it to a separate one to clean up the diff.

I don't have any better suggestions for the current group names. I guess I'd remove "Other" from "Monitoring systems", but it might be clearer that way.
Let's merge and iterate on it.

Agreed.
The merge of the official exporters into the categories is done here, I'd say, so I'd consider this one ready for merge.


Thousands operator wrongly shown for negative value

If the current value is negative and has digits mod 3 == 0 the thousands symbol is shown:

Also: How can an initialValue be defined and automatically be formatted correctly?
This is my source code:
TextFormField(
  initialValue:
      2045.55.toStringAsFixed(2),
  style: const TextStyle(
        fontSize: 30,
      ),
  inputFormatters: [
    CurrencyTextInputFormatter(
      allowNegative: true,
      prefix: '€ ',
    )
  ],
  keyboardType: TextInputType.number,
  //onChanged: (value) => print(value),
)

Fixed in version 1.0.0+6

Thank you. Do you also have an answer to my question regarding formatting an initialValue?

The formatter will not be called when you set initialValue.
You can manually format the value before assigning into initialValue.
var formatter = CurrencyTextInputFormatter(
  allowNegative: true,
  prefix: '€ ',
);

TextFormField(
  initialValue: formatter
      .formatEditUpdate(TextEditingValue.empty, TextEditingValue(text: 2045.55.toStringAsFixed(2)))
      .text,
  style: const TextStyle(
    fontSize: 30,
  ),
  inputFormatters: [
    formatter
  ],
  keyboardType: TextInputType.number,
)


Update php-version to 8.2.12 from 8.1.11

issue: https://github.com/ZOSOpenTools/phpport/issues/8 opened as first problem

issue #9 opened as next problem (sqlite3 library bug I think)

/run tests

with the latest changes, on zoscan2b, php is now working not too bad... 111 failures out of 3000 or so tests. There are 12000 or so tests to run - it explodes at 3000...
but much better than before...

issue: https://github.com/ZOSOpenTools/phpport/issues/10 opened for protection exception running tests

/run tests

glitch in the build and test above, but it actually DID run clean: https://<IP_ADDRESS>:8443//job/Port-Build/1501/console


overridden durations should be specified from epoch

Overridden durations should be specified from epoch https://docs.aws.amazon.com/acm-pca/latest/APIReference/API_Validity.html
Apologies if i'm miss understanding this. It looks like there was an update previously https://github.com/cert-manager/aws-privateca-issuer/pull/70 that would set certificates that specified duration as being in addition to the default i.e 30 days + whatever duration.
I think what people are expecting is that durations are set from issuance.

Hi @karlhungus - my initial impression is that you are correct with this change. I'd like to discuss with the team to verify this and discuss how we want to handle making this change in a way that minimizes impact on those who are relying on the previous validity period calculation for their certificate expiration.

Hi @karlhungus - I discussed with the team, we agree, this change should be merged in. Could you please add unit tests around this function, to ensure future changes don't cause a bug again? Thank you for this change.

@divyansh-gupta unit tests added, LMK if they look ok to you - also welcome feedback on updating defaults to more closely match cert-manager https://github.com/cert-manager/aws-privateca-issuer/pull/111#discussion_r795966244

Seems like the e2e tests failed for a reason completely unrelated to this change. Will try running again.

Hi @karlhungus I have approved your changes. I spoke with the team about your question on defaults: we will leave the default as-is for now. We do not want to cause further disruption for those who are using this plugin. Thank you for this catch, I am merging it now.


Misdirection based on play buttons resulting in customer support issue

Describe the bug
The play button is not going to the latest form of the template.
https://console.aws.amazon.com/cloudformation/home#/stacks/new?stackName=senzing-poc&templateURL=https://s3.amazonaws.com/public-read-access/aws-cloudformation-ecs-poc-simple/cloudformation.yaml
Resolution: Point to the latest with the play buttons or remove them.
I don't see a support ticket for the conversation with the customer that exposed this issue.

This is covered by our new process of updating the S3 CFT when we make a release.  We're looking to automate this process, but for now, I will manually do the updates.


ICEWS18 download link is broken

This link in https://github.com/rusty1s/pytorch_geometric/blob/master/torch_geometric/datasets/icews.py
url = 'https://github.com/INK-USC/RENet/raw/master/data/ICEWS18'
doesn't exist

Thanks for letting me know. Fixed in master.


creating dockerfile to test  sdx-lc on sdx-continuous-development

creating docker file to test  sdx-lc on sdx-continuous-development

The test failed because the link ports expected strings, not a reference to the ports object


Deadline: add OCIO environment to a job

Changelog Description
OCIO environment variable is now added to job submission in case it is used.
Additional info

also included some clean-up of old env var configuration for OPENPYPE_VERSION

Testing notes:

Enable Global colorspace management on your project with config priority on nuke-default (since those are default configuration at Nuke imageio settings)
Open nuke at some task and publish a Render write instance to Deadline.
Open Deadline Monitor and look to submitted job properties to Environment section and find the OCIO key

Dependencies:
including https://github.com/ynput/OpenPype/pull/5302

What's the source issue this is trying to solve?
I feel like this logic currently is invalid due to a few reasons:

Isn't the launching of an application already setting OCIO in the prelaunch hook?

If it is - wouldn't that also override the OCIO that deadline tries to set too?
If it isn't - shouldn't it?

The OCIO env var will be platform specific - we'll need to get the platform-specific path in Deadline, right?

Its polluting the intent of the PR and will be lost if the PR is not merged.

good point! It had been recreated here https://github.com/ynput/OpenPype/pull/5302

I definitely agree with you @BigRoy. This PR needs more thinking. At the moment the process is broken only if multiple platforms are used within production. Single platform should be working as expected since the current config path is saved into workflows.
Here are some current implementation concerns:
Maya and Nuke (at the moment only hosts I am monitoring) are included OCIO path content into workflows but they are platform specific. Those will work only in case Deadline is having path remapper configured.

Isn't the launching of an application already setting OCIO in the prelaunch hook?

No the prelaunch hook is not started at Deadline. We are only resolving host's environment and it is not launched wrapped with openpype.

The OCIO env var will be platform specific - we'll need to get the platform-specific path in Deadline, right?

Adding the OCIO environment variable to the submitted job environment is not a solution here because it only supports a single path. Therefore, it is still platform-specific and no solution here.

If it is - wouldn't that also override the OCIO that deadline tries to set too?
If it isn't - shouldn't it?

We need to lock the job config dependencies to a specific ocio config path to prevent job failures if someone changes the ocio path during ongoing production.
Ideas to fix this:

Introduce a new variable, OPENPYPE_OCIO, which will be appended with the host submitter to job environments.
a. during publishing to farm a process will store the de-root-ified configuration path. This variable will only be added if the OCIO path was configured in the running environment host that was used during submission. This variable will be  supporting multipath so in case the config path will not be de-root-ifiable and some other paths will be found in imageio settings under ocio_config, those will be added here.
b. Our Deadline plugin sets up environment variables during the process. It takes the OPENPYPE_OCIO and resolves root-ified paths to the appropriate platform root from Anatomy. Alternatively, it checks all working paths and sets the first working path as the active config in OCIO.

Set OCIO  with platform specific path to job environment variable and resolve it with OpenPype plugin before host is launched.
a. The process will be similar to the previous idea, but we will now proceed with the prelaunch processing of the Deadline OpenPype plugin.

There are downsides of second idea:
it is too dynamic and paths are resolved form settings during job processing and those can be changed in settings and not correct towards workfile configurations (if used path was not de-root-ifiable, and production is using multiple paths for individual platforms instead of root token)

We need to lock the job config dependencies to a specific ocio config path to prevent job failures if someone changes the ocio path during ongoing production.

Actually, that's an interesting point. Since we now have versioned settings I suppose we could provide an env var to lock the settings to a particular point in time or state of settings? That could fix the OCIO issue but potentially a lot of others too?

No the prelaunch hook is not started at Deadline. We are only resolving host's environment and it is not launched wrapped with openpype.

I'd actually argue this could be an issue. There are some hosts that actually have pre launch hooks that have quite an influence on how the application behaves, e.g. for Fusion it defines how it initializes with settings which could influence e.g. even what plugins are available in Fusion and thus might be required for a correct render.

Actually, that's an interesting point. Since we now have versioned settings I suppose we could provide an env var to lock the settings to a particular point in time or state of settings? That could fix the OCIO issue but potentially a lot of others too?

That is a good idea and I would love to have the feature available already, but not a problem to be addressed in this PR. We are talking about different feature request.

I'd actually argue this could be an issue. There are some hosts that actually have pre launch hooks that have quite an influence on how the application behaves, e.g. for Fusion it defines how it initializes with settings which could influence e.g. even what plugins are available in Fusion and thus might be required for a correct render.

No argument about this but again not within the scope of this PR.

@jakubjezek001 could you resovle the conflicts?

closing this PR in preference to https://github.com/ynput/OpenPype/pull/5379


IOS webView:didFailLoadWithError - -1004

I'm running into an issue with Ionic and logging in via LinkedIn. Inappbrowser opens as expected, but upon logging in via the LinkedIn dialog / page I'm getting the following error:
webView:didFailLoadWithError - -1004: Could not connect to the server.
&
Error -> Problem authenticating
Any thoughts? I'm not doing anything fancy outside of the demo code - here's my payload code:
$scope.loginWithLinkedIn = function(){
var clientId = "ID";
var clientSecret = "SECRET";
var appScope = ['r_basicprofile'];
var state = "dev-09999";
$cordovaOauth.linkedin(clientId, clientSecret, appScope, state).then(function(result) {
console.log("Response Object -> " + JSON.stringify(result));
}, function(error) {
console.log("Error -> " + error);
});

};
I should add that I'm using the latest version of Cordova and cordova-plugin-inappbrowser 1.0.1-dev

Probably related to #32

Unfortunately I'm running on device every time - still encountering the error.

odd. Did you try to to provide an empty array for appScope? (so it gonna match how your app is configured on linkedin side).
Also make sure you have http://localhost/callback as OAuth 2.0 Redirect URLs.
What did you put as 'JavaScript API Domains'?

Callback was set to http://localhost/callback.
Here's my code:
$scope.loginWithLinkedIn = function(){
var clientId = "key";
var clientSecret = "secret";
var appScope = ['r_emailaddress', 'r_basicprofile'];
var state = "DCEeFWf45A53sdfKef424";
$cordovaOauth.linkedin(clientId, clientSecret, appScope, state).then(function(result) {
console.log("Response Object -> " + JSON.stringify(result));
}, function(error) {
console.log("Error -> " + error);
});

};
I left the domains area empty - should I set this to anything e.g. http://*?
Thanks!

Try
var appScope = [];

and for the domains : http://localhost/callback,http://localhost/

Can you guys confirm the following?:

You are not testing via ionic serve or live-reload.
You are not using the Ionic View app.
You are not using a cloud building service such as Phonegap Build.
What version of iOS are you testing with?
Are you using a device or simulator?

Thanks,

Some answers:

We're testing using ionic build ios
We're not using the view app
Not using Phonegap build
IOS Version 8.1.3
Device (iPhone 5S)

As I was typing this, I was also running a build incorporating MyBoon's suggestions - they worked! (partially). I got the initial 1004 error, but then I got the token return!
I'm guessing this is the race condition described above? I can work around that now - thanks for your help guys.

Something else to add - not sure if it has anything to do with things.
I updated the inappbrowser plugin to the latest - 1.0.1-dev, and had to update the plugin check id in ng-cordova-oauth.js to 'cordova-plugin-inappbrowser'

You're not the first person to report this in the past 3 days.
What version is installed when you just run a plain cordova plugin add org.apache.cordova.inappbrowser?  If you're using a nightly or beta version of the plugin, I don't yet plan on updating the library to reflect.  Has to be a stable release.
Thanks,

It installs 0.6.0
Thanks!

Yea, when Apache updates their plugin repository to reflect the latest version, I will integrate the change.  Until then it will have to remain a manual process for anyone who wants to be bleeding edge.
Sorry about that.

No need to apologize! Thanks for creating a great product, and prompt attention to our issue.

So switching back to inappbrowser 1.0.1-dev broke things again. I'm not really sure what I'm doing wrong. Should I try uninstalling the latest and go back to the ngCordova version (0.6.0)?
What's the best way to clear current plugins and ensure that I'm not leaving residual bits of different versions of inappbrowser around? Remove platform then add?

@nraboy Yeah like I said in #32 : "Works like a charm if you build the ionic project and run it on your device." For me, browserify was the problem.

Hey MyBoon, can you please elaborate on this? As far as I can tell, I'm no using browserify - unless that's part of the default Ionic IOS build process. Please forgive my ignorance - I'm not very experienced in Phonegap / Ionic apps in general. Thanks.

@Shaolin23 : browserify is a dev tool that I'm using. It's not part of the standard ionic build process. I was just letting know @nraboy what was the source of the problem for me.

@Shaolin23, I know this is a little further than you probably wanted to go, but have you tried to build it for Android?  If you throw your project on GitHub, I can take a look and see if I get the same results.

Please see the following:
https://github.com/nraboy/ng-cordova-oauth/issues/75#issuecomment-101083345
I got that error, but it seems to have no negative effect on the authentication process.

Hi @MyBoon @nraboy
I'm experiencing a similar
error - upon submitting email and pass through the login dialog I instantly get
"webView:didFailLoadWithError - -1004"
--------------------------------------- CODE EXAMPLE --------------------------------------
Index.html using code:

App.js using code:
document.addEventListener("deviceready", function () {
$cordovaPlugin.someFunction().then(success, error);
}, false);
....
angular.module(ModuleID, ['ionic', 'ngCordova'])
....
Linkedin.js using code:
Linked.CLIENTID = "75eqw61w9zeubd";
Linked.CLIENTSECRET = "WDWHs6D4B9GUpOkW";
$cordovaOauth.linkedin(Linked.CLIENTID,Linked.CLIENTSECRET, ["r_basicprofile","r_emailaddress"]).then(function(r) {
// results
console.log("OK");
console.log(r);
}, function(error) {
console.log("FAIL LIB");
console.log(error);
});
Can you help me this issue? Also, I have test on the ion-lab and serve then it's working fine. Just only not working when build app with Xcode.
Thanks


Disable negative values

This issue was originally open by @melanke (https://github.com/vuejs-tips/v-money/issues/88), and @hamonCordova (https://github.com/vuejs-tips/v-money/issues/97) on v-money repo.

A new boolean property disable-negative was introduced.


RFE: BIG-IP iRulesLX Version of Code

Hey Bill,
Awesome stuff! been using this for a while now.
Currently looking to integrate it with the SSH Proxy to allow for BIG-IP rules to be introduced.
Is there any chance you have done this?
Final Question - I am also looking at transitioning to kerberos authentication as well, any work done in this area?
NK

Right now there are no plans for a public implementation of this. While you could make it work with iRules LX it's pretty complicated and not supported by this project.
I do know that F5 Professional Services have implemented this feature for customers, but it would be a paid engagement.
This would probably make more sense as a separate project, reason being that iRules LX is pretty much locked in at nodejs v6 and we're starting to run out of runway to support that version of nodejs

since BIG-IP continues to be stuck in Node v6, have to abandon this feature request for this project.


Have Hik-Sink start on boot

I'm running Hik-Sink standalone on a Raspberry Pi, is there anyway to have the app start on boot?  I've got this working at the moment via bashrc but then if you ssh in to the Pi the app launches when the Pi is logged in to via SSH and i just hang the session so the app continues to run.  Be good if it could be done via systemd or similar?

@daveo2k, If you still have this issue try adding it to the /etc/rc.local file.
More info here: https://raspberrypi-guide.github.io/programming/run-script-on-boot

Ideally you would use systemd and turn HikSink into a service. There are a few tutorials around online about how to do this, e.g. https://medium.com/@benmorel/creating-a-linux-service-with-systemd-611b5c8b91d6


chore(main): release 0.50.4

:robot: I have created a release beep boop
0.50.4 (2024-08-29)
Bug Fixes

Analytics: use slug for dataset_id properties (#687) (1eb0a2d)

Miscellaneous

localhosting: Add /files to middleware bypass (#685) (98db559)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/BLSQ/openhexa-frontend/releases/tag/0.50.4 :sunflower:


Allow ind inc and fica tax param inputs in CCC

B-Tax integrates with the Tax Calculator.  This PR is to suggest we allow inputs for IIT parameters on CCC.
There are a few options to do this.  e.g.,:

A link to Tax Brain to enter full list of parameters
Allow a limited set of parameters to be entered in CCC itself (e.g. rates on dividends and capital gains, pass-through business income, etc.).
A mix of (1) and (2)

It might be easiest to start with (2) and go from there.

FYI, #295 provides some history of thought.

Back in #295, this comment described the long run goals as:

Some subset of parameters from TC are displayed on the Cost of Capital Calculator (CCC) input page. (@jdebacker will provide this list)

There is unfolding/expansion option on the CCC input page that allows the CCC user to see all of the TC params that are currently available on TB.

On the TB static results page, there is a "Link to Cost of Capital Calculator" button right next to the "Link to Dynamic Simulations".

@jdebacker, what do you think about constraining this issue to step 1. Specifically, this would entail including all parameters from TaxBrain's Regular Tax, AMT, and Other Taxes sections under an individual income tax section on the CCC input page.
I also propose replacing the second goal with a TC parameter file upload feature. We then would need a separate issue for that and for the third goal.

@MattHJensen  - yes, I think constrain to "step 1" makes sense.  This is my suggestion in the first comment here.
I do think that when this is done, it would be nice to group the parameters ways so that the inputs don't seem overwhelming.  This could be done by using indentation for the sections noted in the column on the left of the screen.  E.g.:
Individual Income Tax
    Regular Tax
    AMT
    Other Taxes
Business Taxes
    Business Income Tax Rates
    Depreciation
    Other
Macroeconomic Assumptions

Or (much more complicated) one could have these grouping expand on the main screen (e.g. you scroll down from the top to the Individual Tax Section and you see "+Regular Tax" - if you click the "+" it'll then expand and allow you to adjust the default values.
I like your idea for json upload vs expanding list.  I think that's a suitable substitute for step 2 - especially if we eventually get to step 3.
One complication we might want consistency in macro assumptions.  I think we are using different interest rates between CCC and TB, but the inflation rates should be consistent.  But my suggestion is to just get these two integrated and then we can workout this minor issue later.

This could be done by using indentation for the sections noted in the column on the left of the screen. E.g.:

Individual Income Tax
    Regular Tax
    AMT
    Other Taxes
Business Taxes
    Business Income Tax Rates
    Depreciation
    Other
Macroeconomic Assumptions

Or (much more complicated) one could have these grouping expand on the main screen (e.g. you scroll down from the top to the Individual Tax Section and you see "+Regular Tax" - if you click the "+" it'll then expand and allow you to adjust the default values.

Agree with the primary point and that the expandable version would be even cooler if it isn't too hard.
I suggest putting the IIT params beneath the BT params in the list, though.

@jdebacker, what do you think about prioritizing the reform file upload capability to allow taxcalc reforms on CCC over the field-based gui approach described in this issue?
I envision this being implemented by adding a new section to CCC nav called something like "IIT/FICA Reform File" in between "other" and "Macroeconomic Assumptions". The content of the section would look like this:

Except "Reform File" would be replaced with "IIT/FICA Reform File" and the "See examples here" would come beneath the section title and we'd also include a link to TC/TD documentation.
Here's why I'm making this suggestion:

Detailed reforms such as the Ryan/Brady blueprint and the Trump tax plan are immediately relevant, and these could be specified using the json reform upload capability but not using the field based gui we are envisioning.
I imagine we could get the json reform file upload capability online faster than a nice form-based gui, although @talumbau and @PeterDSteinberg probably have a better sense of whether that is true than I do.

@MattHJensen, Seems ok to me.  Has the file upload been getting lots of use on TB?  I'm not sure how easy this is for most people so I was supposing it was more of an "advanced user" feature.  If you think those users are the most relevant audience in the short run then, yes, I'd agree that allowing this first would be a good move.

I'm not sure how easy this is for most people so I was supposing it was more of an "advanced user" feature.

Let's call it an "advanced user" or "assisted user" feature. Here's what I mean by assisted: the TC team can post the reform file for different high profile proposals at the examples page, and then all someone has to do is upload that file and tinker on the margins at will.
We've found this feature to be helpful for policy-maker users with extremely detailed proposals, and it is helpful when proposals are moving quickly or are time sensitive for other reasons, and we want to get new tax-calculator features onto the webapp quickly.

Earlier I suggested that we pursue the json form upload before the gui approach based largely on the fact that:

Detailed reforms such as the Ryan/Brady blueprint and the Trump tax plan are immediately relevant, and these could be specified using the json reform upload capability but not using the field based GUI we are envisioning.

However, #436 will mean that all important outstanding reforms are available from the TaxBrain GUI, and https://github.com/open-source-economics/Tax-Calculator/pull/1109 will make it so that new reforms can be added quickly to the GUI.
Therefore, my new recommendation for this issue (#357) is to start off by making IIT and FICA parameters available on the CCC GUI rather than starting with the JSON upload.
In order to facilitate this approach, I reorganized the milestones by slimming down b2 and b3 so that #436 can go in b2 and #357 can go in b3 without slowing down progress on #357.

Given that that changes to pass-through business taxation are a large part of the tax reform discussion now, I wanted to re-up this issue.  Even having just some preset IIT reforms to select would be very helpful.
cc @MattHJensen


Can't instantiate ag-grid in Storybook (Angular)

Bug or support request summary
I am trying to show an ag-grid inside Storybook. However, I am getting errors when I try to include its metadata.
If I include the AgGridModule in the metadata, I get the following error:
Unexpected module 'AgGridModule' declared by the module 'DynamicModule'. Please add a @Pipe/@Directive/@Component annotation.

If I include AgGridNg2 (the actual component) in the metadata, then I get the following error:
StaticInjectorError(DynamicModule)[Ng2FrameworkFactory -> BaseComponentFactory]

Steps to reproduce
I have a minimal repo here. The story under question is here.
Please specify which version of Storybook and optionally any affected addons that you're running

@storybook/angular 4.0.0-alpha.14

Affected platforms
All
Screenshots / Screencast / Code Snippets (Optional)
storiesOf('Order List', module)
    .addDecorator(
        moduleMetadata({
            declarations: [AgGridModule]
            // declarations: [AgGridNg2]
        })
    )
    .add('list with orders', () => ({
        component: OrderListComponent,
        props: {
            orders: orders
        }
    }));

Will be able to check this only by the end of the next week.

@nareshbhatia Modules need to be included in imports, not declarations. E.g.
  moduleMetadata({
    imports: [ AgGridModule ]
  })

Thanks so much, @jonrimmer! Works like a charm now.