Update CHANGELOG.md for mgmt plane Azure Loadtesting

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.LoadTestService

CC @venkatr21

hd*
I'm
On October 25, 2022 2:01:42 PM GMT+08:00, Azure SDK Bot @.***> wrote:

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.LoadTestService
--
Reply to this email directly or view it on GitHub:
https://github.com/Azure/azure-sdk-for-net/pull/31990#issuecomment-1290028479
You are receiving this because you are subscribed to this thread.
Message ID: @.***>


Implement Outlines.

For creating a user TOC/navigatable index. See http://www.perlmonks.org/?node_id=1191298

Underway with https://github.com/p6-pdf/PDF-API6/commit/6aec560cb912b88e03251e99662e610d13c0ea5a


Bugfixes

Not yet done - going to add/fix later.

I removed the proposed change to add panics in for now since it will block this patch for some time. Assuming no major concerns, this is ok to merge in now.


Drag and Drop Image p5 web editor example for week 2

A note to remake or find this example. See discussion in #15.

There's also this one https://github.com/ml5js/ml5-homepage-demo!

https://github.com/ml5js/ml5-homepage-demo

Here's the example!
https://editor.p5js.org/ima_ml/sketches/C5Ywejvuo
@joeyklee we can add this to ml5 if that seems useful .. or just leave it here as an extra example for this course.


Polling / Status Request

First of all: great plugin, I love it :D
Is your feature request related to a problem? Please describe:
Right now, whenever someone turns the TV off using its remote, the switch in Homebridge doesn't get refreshed.
Describe the solution you'd like:
It would be great if the plugin could poll to see if the tv is turned on. This would guarantee that Homebridge only shows the device as turned on when it actually is (and vice versa).
Describe alternatives you've considered:
You could add a third URL for polling. A request to this url could return True/False, 1/0, Status Code 200 / 400, etc to signify the status of the device

Hey there!
Thanks a lot, took me a while to figure out all the TV HomeKit stuff which isn't very straightforward lol 😂
At the moment I don't think I really have a use case for such an addition, since the point of this was to get rid of my remotes haha
But you clearly do so I'll see when I can find some time to have a look at adding such a feature. I took inspiration from homebridge-http-switch anyway and they support exactly what you're looking for so it shouldn't be too hard to add...
If you're really keen and need it fast I'd obviously also be happy to merge any pull requests you might come up with haha

Hey there!
Thanks a lot, took me a while to figure out all the TV HomeKit stuff which isn't very straightforward lol 😂
At the moment I don't think I really have a use case for such an addition, since the point of this was to get rid of my remotes haha
But you clearly do so I'll see when I can find some time to have a look at adding such a feature. I took inspiration from homebridge-http-switch anyway and they support exactly what you're looking for so it shouldn't be too hard to add...
If you're really keen and need it fast I'd obviously also be happy to merge any pull requests you might come up with haha

Is it possible for you to make it work? I also would love to ping ip adress with this plugin :) thanks


self.jslocation  -> class

look for all instances of
self.jslocation =
needs to become like
BEFORE
from Jumpscale import j

class PacketNetFactory(j.application.JSFactoryBaseClass):

    def __init__(self):
        self.__jslocation__ = "j.clients.packetnet"
        self.__imports__ = "packet"
        self.connections = {}
        JSConfigFactory.__init__(self, PacketNet)

    def interactive_test(self):
        from IPython import embed
        embed(colors='Linux')

NEEDS TO BECOME
from Jumpscale import j
from .PacketNet import PacketNet

class PacketNetFactory(j.application.JSFactoryBaseClass):
    __jslocation__ = "j.clients.packetnet"
    _CHILDCLASS = PacketNet

    def __init(self):
        self.connections = {}

notice: _init

clients that need changing:

[ ] blockchain
[ ] btc_alpha
[ ] coredns
[ ] currencylayer
[ ] etcd
[ ] gdrive
[ ] git
[ ] gitea
[ ] github
[ ] google_compute
[ ] grafana
[ ] graphite
[ ] http
[ ] influxdb
[ ] intercom
[ ] ipmi
[ ] itsyouonline
[ ] kraken
[ ] kubernetes
[ ] mail
[ ] mongodbclient
[ ] mysql
[ ] oauth
[ ] openvcloud
[ ] ovh
[ ] packetnet
[ ] peewee
[ ] portal
[ ] postgresql
[ ] racktivity
[ ] redis
[ ] redisconfig
[ ] rogerthat
[ ] s3
[ ] sqlalchemy
[ ] ssh
[ ] sshagent
[ ] sshkey
[ ] syncthing
[ ] tarantool
[ ] telegram_bot
[ ] threefold_directory
[ ] traefik
[ ] trello
[ ] virtualbox
[ ] webgateway
[ ] zdb
[ ] zero_boot
[ ] zero_hub
[ ] zero_hub_direct
[ ] zero_os_protocol
[ ] zero_stor
[ ] zerotier

suggest to just do a search on self.jslocation
easy to find & fix


[BUG] Publisher Pipeline Validation Failure

Release version
v4.1.2
Describe the bug
Hi, I'm having validation failure when creating my publisher pipeline, I have tried different troubleshooting steps and ideas. still not working for me.
Expected behavior
Pipeline run successful.
Actual behavior
/tools/azdo_pipelines/run-publisher-with-env.yaml (Line: 34, Col: 1): Unexpected value 'variables' /tools/azdo_pipelines/run-publisher-with-env.yaml (Line: 39, Col: 1): Unexpected value 'stages'
Reproduction Steps
comment added
trigger:
branches:
include:
- main
paths:
exclude:
- tools/*
parameters:

name: API_MANAGEMENT_SERVICE_OUTPUT_FOLDER_PATH
type: string
displayName: Folder where the artifacts reside
default: "artifacts"
name: COMMIT_ID
type: string
displayName: Choose "publish-all-artifacts-in-repo" only when you want to force republishing all artifacts (e.g. after build failure). Otherwise stick with the default behavior of "publish-artifacts-in-last-commit"
default: publish-artifacts-in-last-commit
values:

publish-artifacts-in-last-commit
publish-all-artifacts-in-repo

variables:

group: apim-automation-publish
name: System.Debug
value: true

stages:

stage: push_changes_to_Dev_APIM
displayName: Push changes to Dev APIM
jobs:

job: push_changes_to_Dev_APIM
displayName: Push changes to Dev APIM
pool:
vmImage: ubuntu-latest
steps:

template: run-publisher-with-env.yaml
parameters:
API_MANAGEMENT_SERVICE_OUTPUT_FOLDER_PATH: ${{ parameters.API_MANAGEMENT_SERVICE_OUTPUT_FOLDER_PATH }}
RESOURCE_GROUP_NAME : $(RESOURCE_GROUP_NAME)
API_MANAGEMENT_SERVICE_NAME : $(APIM_NAME)
ENVIRONMENT: "Dev"
COMMIT_ID: ${{ parameters.COMMIT_ID }}

stage: push_changes_to_Prod_APIM
displayName: Push changes to Prod APIM
jobs:

deployment: push_changes_to_Prod_APIM
displayName: Push changes to Prod APIM
variables:
#setting the testSecretValue to the prod resource group name as an example
testSecretValue: $(RESOURCE_GROUP_NAME_Prod)
pool:
vmImage: ubuntu-latest
creates an environment if it doesn't exist
environment: 'Prod'
strategy:
default deployment strategy, more coming...
runOnce:
deploy:
steps:
- template: run-publisher-with-env.yaml
parameters:
API_MANAGEMENT_SERVICE_OUTPUT_FOLDER_PATH: ${{ parameters.API_MANAGEMENT_SERVICE_OUTPUT_FOLDER_PATH }}
RESOURCE_GROUP_NAME : $(RESOURCE_GROUP_NAME_Prod)
CONFIGURATION_YAML_PATH:  $(Build.SourcesDirectory)/configuration.prod.yaml
ENVIRONMENT: "Prod"
COMMIT_ID: ${{ parameters.COMMIT_ID }}

This seems to be an issue with your pipeline. Doesn't seem to be related to the tool itself.

I was able to resolve this issue. Did a fresh re-deployment and it worked.


Canvas reference in CanvasView object

Hi @farfromrefug, thanks for you work here. I want to know why you aren't exposing the Canvas as a property of the CanvasView.
I'm using Svelte-Native, which closely follows Nativescript-Vue, but it's not yet finished. The problem is that the "draw" callback is not passed/called.
When I add a listener: cnvObj._nativeElement.on("draw", canvasLoaded); I get an event with the eventName and an object, but this last is the CanvasView object. Therefore, the code to draw can't work as it can't get a reference to the Canvas in this event.
Can you expose the Canvas object as a property of the CanvasView object? Or what's the reason behind not doing it?
Thanks

@SntsDev I cant really tell how you use it. But i can tell you it works perfectly with svelte.
Here is working example:
https://github.com/farfromrefug/ns-forecastie/blob/cf088d167cd8753eb2c964baad818131b89849bc/app/WeatherCollectionItem.svelte#L115
I will close that issue for now. If you think there is still an issue let me know.

Thanks @farfromrefug, I don't know why I missed to add "on" to on:draw={...}. You're totally right. Thank you!


InfluxDB Cloud CSV upload seems to be broken

Steps to reproduce:

Log into InfluxDB Cloud account
Create a bucket
Click "+Add Data" button
Select CSV Upload option and try to upload a CSV file
It will tell: "Please make sure that CSV was in Annotated Format and try again"

I copy&pasted the content of my csv from here, so i am pretty sure that it is in annotated CSV format.
Expected behavior:
I expected the above method to import annotated CSV files without any error.
Actual behavior:
The above method fails to import annotated CSV files.
Environment info:
InfluxDB Cloud deployment in Frankfurt on AWS.
CSV file written with Notepad on a Win10 x64 machine
Using Chrome V89.0.4389.90

@idea--list thanks for the issue. The csv upload tool in the UI only works with a fully annotated csv output from flux. it's useful for downloading the results from a flux query and loading them to another bucket, or sending them to another user.
the docs link you referenced is for loading data via the influx CLI tooling. that tool is flexible enough to load any csv data format imaginable, but requires much more configuration not available in the UI.

@russorat
By fully annotated csv output from flux you mean Extended annotated CSV documented here?

Now fixed.


implement popup menu

Also a few other changes:

Use glamorous for hover, and make it that icon is black_40 if row is hovered, and black_60 if icon itself is hovered (as in design).
Hide macOS special files (._-prefixed, .DS_Store, .darwin).
Add date/time/author information for file rows.
Fixed sort settings popup on mobile to go from bottom.

Issues:

Long writer name doesn't get cut off or ...-ed properly on mobile.
lastWriter is not available yet, so that's faked to Jared Dunn for now.

Desktop:

iOS:

@chrisnojima mind taking another look please?

CI error seems to be a flake:
Could not download bcpkix-jdk15on.jar (org.bouncycastle:bcpkix-jdk15on:1.48)

Merging!


Migration of the RSS-Bridge project

Hello everyone,
By lack of better means to get in touch with as many as possible people involved in the development of RSS-Bridge, I'm making this an "issue" :)
Having thought about this for some time and having contacted @sebsauvage about it, I created a Github organization, named "RSS Bridge Community" (name can be changed in the future if we have more community projects than just RSS-Bridge).
Continuing the project under such community allows us better members/contributors management, documentation features, team management, and more.
However, to avoid confusion between repositories, I would need to take action on this repository. Since the issue history can't be imported into the new repo, I'd rather not have Sebsauvage delete the repo right away. I'm rather in favor of wiping the repo content and putting an information notice in the README, at the risk of shutting down existing rss-bridge instances where sysadmins pull the repo without checking, but outlining that since it's alpha software, breakage is expected anyway.
TL;DR → closing down here, and get it running at https://github.com/RSS-Bridge/rss-bridge
Any suggestions welcome :)

update: repo is pending transfer to @RSS-Bridge

update: repo has been transferred :)


Bimodal analysis

For #35
Impl plan

[x] Attach methods to source managers for analysis
[x] Reconfigure adapter to make it injectable-ish for tests
[x] Pull in stdlib package list from tip. (so, just ignore core go version for now)
[ ] Incorporate stdlib list in lock input hashing (they amount to ignores, and COULD change)
[ ] Rewrite SourceManager internals to be saner about bootstrapping disk contents on demand, then...
[x] Add funcs to suss out the repo from a package name
[x] Add logic for adding, w/open constraint, things in analysis that aren't in manifest
[x] Skip special rules for now, just ignore things in manifest but not analysis
[x] Change selection structures to operate on both projects and packages
[x] Implement new testing harness specifically for exercising bimodal analysis
[x] Create new selection path for when we're just adding packages to an already-selected project
[x] Testing fixtures for all the reasonable basic permutations of inputs, exercising {0,1,N} behaviors, transitivity, backtracking, multi-stage dep integration, etc.
[x] Modify unselected comparator to always have package-only additions float to the top - those are always more incremental changes
[x] Revisit how fail() analysis chooses what to mark
[ ] Circle back on whether we need to look at all imports, or just the current atom's, in solver.getImportsAndConstraintsOf()
[ ] Add satisfiability check that, if a project/version is already selected, all pkgs targeted within it exist. Also add the reverse, that if a project is being checked and deps already exist, it has the packages already depped upon

Finally got things together to the point where I'm integrating the analysis bits into the core of the algorithm. There are a couple classes of things to handle. First is moving from package names to repos - which, for now, we take to be the same as projects.

We have to exclude stdlib packages, or other things that don't have, or appear to have, a network-addressable identity
We have to translate the package names (those not excluded by the preceding) back down to their repo roots, if we can. Obnoxiously, this may entail a remote HTTP op

Second, we have the issues described originally - discrepancies between the manifest and the analysis.

We could have some things found in the reach that aren't in the manifest. This is easy - we take them, with an open constraint.
We could have some things in the manifest that aren't in the reach, in which case we have two choices:

Ignore some or all of the extras
Incorporate some or all of the extras with special rules - select all their packages

so, so, so close now. the basic system should now be sound; i think there are now just odds and ends to cover.

bleh bleh turns out the static analysis needs work, and tests. doing that, then i think i can finally merge.

WE HAVE FINALLY ARRIVED


Change string/concatenate to support mulitple inputs

String concatenate was only concatenating 2 strings. Now we are able to
concatenate up to 32 strings using a single node. The approach taken is
the same approach from int opeartions that have multiple inputs
Signed-off-by: Otavio Pontes<EMAIL_ADDRESS>

lol. the idea in the title of the commit message is the oposite. Correct:
strings: Rename bytes option to chars in concatenate and compare
Unicode strings are counting the number of characters and not the number
of bytes.

FAILED running string-concat.fbp: wrong exit code, got 1 but expected 0 Output: ==14433== WARNING: AddressSanitizer failed to allocate 0xc00800018078 bytes
check the bot's output. Enable the sanitizers on your machine.

@otaviobp wasn't the conclusion of our discussion on IRC yesterday with @glima to completely remove this counting option?
So we would get rid of this logic inside this node. It's specially true imo if you're going to fix the size only of the final string. It can be easily achieved using string/slice later, right? Something like:
concat(string/concat) OUT -> IN _(string/slice:end=CHARS) OUT -> IN _(console)
instead of
concat(string/concat:chars=CHARS) OUT -> IN _(console)
@glima, what do you think about it?

For me it is ok to remove it. I had understood that I was suppose to keep it, but limiting the size of the full string.

I was ok with the semantics changed to the full size, but removing it altogether is cool as well. I vote for removing code :)


Multiple insert using array notation

Can't we have an array of objects with key:value pair like single insert and not using arr of arrays to insert??

Hi and thanks for the suggestion. It is not clear to me what the ask is, but I suggest perhaps you can contribute it as a pull request?


Preparing for Heroku Deployment

Updates:

Added Procfile (Tells Heroku to run gunicorn to host flask app)
Added requirements.txt (Tells Heroku to install the dependencies for the app)

Heroku


The commit behavior of the PropertyPane editor needs some modifications

Describe the bug
When a commit is triggered from the UI, the component will propagate the change event even if the 'backing' value is exactly the same. This results in a transaction on the undo stack when a user simply enters edit mode and then clicks off (changes nothing). Also the ESC key seems to commit, which it should not.
To Reproduce
No response
Expected Behavior
The ESC key should be an easy one. If ESC is hit, the formatted original value should be restored to the widget, and I think edit mode of the property should be ended.
Fixing the other issue brings up a few more situations to worry about. For example, a backing value of 17.0026 could be displayed as 17.0. But it is possible that the user actually wants to edit the value and set it to exactly 17. So I think we do need to support that case also, but probably only via an explicit Enter key. Here is what I think the Booster team would like to see for behavior once in edit mode:
if (loseFocus) {
if (edit string != originalFormattedString)
tryCommit()
} else if (EnterKey)
tryCommit()
}
tryCommit() {
newValue = parse(editText)
if (isValid(newValue) && newValue != originalValue)
commitForReal()
}
Screenshots
No response
Desktop (please complete the applicable information)
iTwin.js 4.7.2
AppUI 4.12.0
Additional context
No response

The fix released with<EMAIL_ADDRESS>


Reuse existing pull request in increment plugin workflow

Description
Not all plugins can immediately merge changes from this workflow and need to add changes before the generated pull request can be merged.  This update prevents that existing branch from being overridden if the job is re-run while we wait for health builds.
Aspiring issue: https://github.com/opensearch-project/security/pull/2210#issuecomment-1308987770
Issues Resolved

Related https://github.com/opensearch-project/security/pull/2210

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

FYI @DarshitChanpura

Codecov Report

Merging #2874 (80c8b5b) into main (970646d) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #2874      +/-   ##
============================================
- Coverage     93.80%   93.72%   -0.08%
============================================
  Files           213      158      -55
  Lines          4389     4401      +12
  Branches         10        0      -10
============================================
+ Hits           4117     4125       +8
- Misses          266      276      +10
+ Partials          6        0       -6

Impacted Files
Coverage Δ

src/manifests/test_manifest.py
85.71% <0.00%> (-11.66%)
:arrow_down:

src/test_workflow/integ_test/integ_test_suite.py
85.71% <0.00%> (-8.41%)
:arrow_down:

src/test_workflow/test_cluster.py
85.33% <0.00%> (-3.24%)
:arrow_down:

src/system/os.py
100.00% <0.00%> (ø)

src/run_bwc_test.py
87.50% <0.00%> (ø)

src/git/git_commit.py
100.00% <0.00%> (ø)

src/run_integ_test.py
87.50% <0.00%> (ø)

src/system/console.py
100.00% <0.00%> (ø)

src/system/execute.py
100.00% <0.00%> (ø)

src/system/process.py
93.54% <0.00%> (ø)

... and 187 more

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

@prudhvigodithi Anything else you'd like to see in this pull request?

Hey @peternied thanks, just got pulled out of this, one qq did you test with this change in your local fork, that it creates a PR with version increment and then does not override the changes (made while the a PR is open) upon re-push (with workflow re run) ?

I did not test it, is there documentation for how to do that?

I did not test it, is there documentation for how to do that?

Hey @peternied
I mean just the GH workflow test on your fork repo, example something like this https://github.com/prudhvigodithi/job-scheduler/pull/2

I sense that this scenario will only come into play when an auto-cut PR branch for version increment is deleted and the bot attempts to looks for it. Am I correct?

I sense that this scenario will only come into play when an auto-cut PR branch for version increment is deleted and the bot attempts to looks for it. Am I correct?

Hey @DarshitChanpura this is moreover related to retain the changes pushed to the version increment branch, so if a PR is open for a version increment and a user pushes some new changes, the AUTO version increment should retain those changes and not override the changes pushed by the user. @peternied please add if i'm missing anything.
@gaiksaya @peterzhuamazon @bbarani @zelinh
Thank you

Hey @DarshitChanpura this is moreover related to retain the changes pushed to the version increment branch, so if a PR is open for a version increment and a user pushes some new changes, the AUTO version increment should retain those changes and not override the changes pushed by the user. @peternied please add if i'm missing anything. @gaiksaya @peterzhuamazon @bbarani @zelinh Thank you

Ahh.. got it.. thank you @prudhvigodithi !

I did not test it, is there documentation for how to do that?

Hey @peternied I mean just the GH workflow test on your fork repo, example something like this prudhvigodithi/job-scheduler#2

@prudhvigodithi I'm not sure how to setup the secrets in my forked repository, can you help me figure out how to generate the APP_ID?

Hey @peternied thanks for waiting just getting back on this, you can generate your own app and use that and should work as you are testing on your forked repo, another way is to the API Token, this should be easy for testing with only change on the line as follows (token: ${{ secrets.PAT }})
        with:
          token: ${{ secrets.PAT }}
          committer: opensearch-ci-bot<EMAIL_ADDRESS>          author: opensearch-ci-bot<EMAIL_ADDRESS>          commit-message: |
            Increment version to ${{ env.OPENSEARCH_VERSION }}
            Signed-off-by: opensearch-ci-bot<EMAIL_ADDRESS>          delete-branch: true
          branch: create-pull-request/${{ env.OPENSEARCH_VERSION }}
          title: '[AUTO] Increment version to ${{ env.OPENSEARCH_VERSION }}'
          body: |
            - Incremented version to **${{ env.OPENSEARCH_VERSION }}**.

@prudhvigodithi After playing around with this workflow I tested it, and it didn't work because that isn't how the create PR workflow is built.  I'm going to abandon this, maybe someone else can look into it at a future point in time


emphasize su zulip in install instructions

If you try to run ./manage.py shell as described in https://zulip.readthedocs.io/en/latest/prod-auth-first-login.html, it won't work if you're "root."  You need to su zulip.  We should make this less error prone, and probably talk about the zulip userid.

I think we'll planning to remove that particular block of instructions soon, in favor of manage.py generate_realm_invite_link, but that documentation refactoring should still document using manage.py shell via the su zulip somewhere, and probably said documentation will come out of the existing piece, so fixing this won't be wasted work.
An alternative approach would be to make manage.py automatically su zulip if needed.


Stepper fix

Fixes:

Using os.path.join instead of / in file paths,
Handling location files opening with try

LGTM

@douglascamata you up Lmao.

@solderzzc lol, I'm excited to work.... LET'S WWWWOOOORK!

DLGTMA (doesn't look good to my anymore)

@douglascamata,why doesn't?

@Phil9l because you have a merge conflict with dev branch. You need to git pull origin dev and fix the conflicts.

@douglascamata, updated. No conflicts now.

@solderzzc please review again, now it definitely LGTM.


total oops, did not check target repo for the PR

I didn't end up actually removing any rules - I figure if a job posting is unrelated than the more tech-related rules just won't trigger.
I tried to generalize the text of rules to make them more broadly applicable.

@antislice no worries, looks interesting! Let me know if I can help in any way, or if any part of the code-base trips you up


Automate the addition of variable groups

Right now we are asked to manually add a variable group in AzDO. Seems we can automate this (along with the create of the Azure Storage account via AZ cli)
https://docs.microsoft.com/en-us/rest/api/azure/devops/distributedtask/variablegroups/add?view=azure-devops-rest-5.1

This is already available in spk


server patch modularization

The main server patch zirkonium_server.pd should be modularized by:

[x] setting clone numbers via creation arguments, ie. [zirkonium_server 16 16 32] could create a server with 32 IDs, 16 inputs, 16 outputs, etc
[ ] moving OSC IO into a wrapper patch
[x] adding a help file for documentation & testing

This is more or less done in the modularization branch.
A new zirk_server patch takes creation args:

$1 # inputs
$2 # outputs (& speakers)
$3 # ids
$4 # sourcefiles
$5 # channels per sourcefile (2, 8, 16, or 32)

Example: [zirk_server 2 8 16 16 8'
The zirkonium_server patch is now a wrapper for [zirk_server 64 64 64 64 8] and the #zirk-in and #zirk-out send/receive pair.


Segfault (SIGSEV) on Arch Linux

Hi there,
I'm trying to run this through the .jar file on Arch Linux, but it now crashes immediately with the following error every time I launch it, a few seconds after the GUI appears:
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00007ff463de411b, pid=5840, tid=0x00007ff471c56700
#
# JRE version: OpenJDK Runtime Environment (8.0_144-b01) (build 1.8.0_144-b01)
# Java VM: OpenJDK 64-Bit Server VM (25.144-b01 mixed mode linux-amd64 compressed oops)
# Problematic frame:
# C  [libjfxwebkit.so+0x172511b]
#
# Core dump written. Default location: /home/xxx/builds/core or core.5840
#
# An error report file with more information is saved as:
# /home/xxx/builds/hs_err_pid5840.log
#
# If you would like to submit a bug report, please visit:
#   http://bugreport.java.com/bugreport/crash.jsp
# The crash happened outside the Java Virtual Machine in native code.
# See problematic frame for where to report the bug.
#
[1]    5840 abort (core dumped)  java -jar Lightmod-1.0.5.jar

To repo: I went to the chat app template, clicked around a bit, then tried to open the server.clj file. That's when the first crash occurred.

The webview may be unstable on linux, though I haven't experienced this on fedora. Is it possible for you to install JDK 9? I believe it has an updated webview.

Installed jdk9-openjdk (https://www.archlinux.org/packages/extra/x86_64/jdk9-openjdk/), and now it does not launch at all, with the following command-line error:
Error: Could not find or load main class lightmod.core
Caused by: java.lang.NoClassDefFoundError: javafx/application/Application

I'm guessing this is no longer an issue -- Lightmod now has a .AppImage with the JDK built in, and a jar version that requires JDK 11 -- so hopefully the webview is more stable on Arch now. If anyone still notices this crash please let me know.


Integrate job form

ℹ   This branch depends on features that will be introduced with #396, #420, #421, #422, and #423 thus it will fail until the respective branches get merged.

Integrate job form and add job form schemas to provide ans easy to use form alongside the json editor.

Reviewer: @MatApple

Code looks good to me.

Code makes sense! Nice. :)
Some nitpicks though, that could be prevented with a proper configured IDE I guess?

Wainting on the other branches to be merged.

mesosphere-ci retest this please

mesosphere-ci retest this please


Unable to start using npm start

Error when running npm start after checking out code on MAC OSX 10.10:
Running "less:layouts" (less) task
>> NameError: variable @dl-horizontal-breakpoint is undefined in public/vendor/bootstrap/less/type.less on line 214, column 22:
>> 213
>> 214   @media (min-width: @dl-horizontal-breakpoint) {
>> 215     dt {
Warning: Error compiling public/less/bootstrap-build.less Use --force to continue.

Aborted due to warnings.

npm version
{ Drywall: '0.16.4',
  npm: '3.3.12',
  ares: '1.10.1-DEV',
  http_parser: '2.6.0',
  icu: '56.1',
  modules: '47',
  node: '5.1.1',
  openssl: '1.0.2e',
  uv: '1.7.5',
  v8: '<IP_ADDRESS>',
  zlib: '1.2.8' }

grunt -v fails, grunt copy -v passes.

@abhishekb21
I ran into the same issue today.
The reason I found is because the variables.less file is not loaded during the less build process, so it cannot find the dl-horizontal-breakpoint and result in the problem.
I updated /public/less/bootstrap-build.less and inserted @import "../vendor/bootstrap/less/variables.less"; at line 10, which solve the problem. Hope this may help you as well:)

Thanks for opening an issue. This happens sometimes when Bootstrap adds new variables. Those new variables just need to be defined in /public/less/bootstrap-vars.less.

@yeelan0319
Thanks. I got it working with your fix.


Is it possible to make navigation bar translucent (iOS)?

Hi,
i'm trying to achieve translucent navigation bar on iOS, however currently i have no luck with available attributes.
Currently i'm configuring my StackNavigator like this:
const ExploreNavigator = StackNavigator({
  Home: {
    screen: ExploreScene,
    navigationOptions: {
      header: ({ state, setParams }) => ({
          style: {
            translucent: true
          }
        }),
      },
  }
})

However, navigation bar is still just gray and nontransparent.

Hi,
I am also looking for the solution to modify navigation bar color for Android.
my code :
static navigationOptions = {     title: 'Home',     header: {       style: {         backgroundColor : '#454545'       }     }   }
Is there any way to modify navigation bar background color ?

Also experiencing header styling issues. i cant seem to find a way to style a header component from within navigationOptions.
static navigationOptions = {
        title: 'Home',
        header: ({ state, setParams }, defaultHeader) => ({
            ...defaultHeader,
            style: HeaderStyle,
            right: (<Icon
                name="menu"
                style={{ fontSize: 20, right: 15 }}
            />)
        })
    };

...where HeaderStyle is...
{
    backgroundColor: 'red'
}

Having header styling issue as well. "visible" option is not working for IOS, but it's working for Android.
`class HomeScreen extends React.Component {
static navigationOptions = {
title: 'Welcome',
header: ({ state, setParams }) => ({
visible: false
})
};
render() {
return <View style={{backgroundColor:'#ff5976', flex: 1}}>
Hello, Chat App!
<Button
onPress={() => this.props.navigation.navigate('Chat', { user: 'Lucy' })}
title="Chat with Lucy"
/>
;
}
}`

@yapjinkun i had a similar issue, try changing the headerMode to 'screen' in the navigator. Unfortunately, this will not work with headerMode as 'float' (IOS default)

@si-harps much thanks, it's works now to hide the header bar. This will be workaround for me to insert custom header if react-navigation header styling is not working :(

Guys, I manage to style the nav bar using react-navigation that is on master right now. Installed with this command:
npm install --save https://github.com/react-community/react-navigation.git

@plraphael  not working for me :( . Are you sure this issue is fixed?

Thanks @jinxac , that worked for me :)

It looks as though style is applied to all header elements. For example, i have set a border which is being applied to the container as well as each of the buttons. ie. left and right

@si-harps That's a known bug: #189.

Closing in favour of #11

I'm not really sure that #11 is the same ticket. On native iOS, the entire 44px navigation bar is not a solid color, and instead is a mostly-tinted white that blurs the content that is below it. This is completely separate from the Android-only StatusBar.setTranslucent() and light-content styles on the StatusBar component built-in to ReactNative. I think the original poster of this ticket is referring to getting support for the translucent functionality of the navigation bar similar to native iOS, rather than the status bar (an Android only thing).

@grabbou Yes, this has nothing to do with #11. This issue here regards options to get the DrawerNavigator to look like with native iOS apps, see here (1 = translucent, 2 = solid):

https://github.com/react-navigation/react-navigation/blob/master/examples/NavigationPlayground/app/StackWithTranslucentHeader.tsx

Thanks @brentvatne, you got to be the most helpful guy on Github ;-)
I already saw the old example and was wondering if the inset has to be added to every screen manually.
Btw when trying to run the new react-navigation examples via sending a link to my iPhone, i get the following error:


R24.03 Managed Identity Wallets - Release Checks

Release Info
Please provide information on what you want to be included in the Eclipse Tractus-X release.
If you are not owner of this issue, please provide the information as comment to the issue.
Version to be included in Eclipse Tractus-X release: version placeholder
Leading product repository: repository link
Compliance Verifications
This issue tracks all compliance related checks, that need to be performed for a product release in Eclipse Tractus-X.

[x] Gaia-X compliance confirmed
[x] GDPR compliance confirmed (personal data, data protection + privacy DPP)
[x] Interoperability checks performed
[ ] Data Sovereignty checks performed
[x] Compliant with relevant published CX Standards (see the Catena-X standard library)

Documentation

[ ] Arc24 documentation up-to-date
[ ] Administrators Guide up-to-date
[ ] End-User manual up-to-date
[ ] Interface documentation up-to-date

Security Checks

[x] Thread Modelling Analysis passed
[x] Static Application Security Testing (SAST) scans passed
[x] Dynamic Application Security Testing (DAST) tests passed
[x] Secret Scans passed
[x] Software Composition Analysis (SCA) passed
[x] Container Scans passed
[x] Infrastructure as Code (IaC) scans passed

General Checks

[ ] Tractus-X Release Guidelines fulfilled
[x] Compliant with the Style Guide

Test Results

[x] E2E Integration Test passed
[x] User Journey approved

Helpful Links

Eclipse Tractus-X openAPI specs on SwaggerHub
Catena-X standard library

@OSchlienz do we now release it? Or is the issue only the 24.05 release?

@OSchlienz do we now release it? Or is the issue only the 24.05 release?

Hi @Siegfriedk yes we are part of R24.03

No fundamental changes in current release in terms to interoperability.
Will review in 24.05 release.
@kelaja Approved for the time being.

@kelaja All relevant published CX-Standards were considered as well as foreseen standard-requests were reviewed (as good as possible).

Documentation for @vialkoje

Interface documentation
Architecture Documents Arc42
Administrators Guide
End-User manual

@jjeroch could you please confirm compliance with style guide (as MIW has no own UI)

Gaia-X compliance
not applicable: the respective operators of the MIW have to provide a self description of their serviceoffering and for themself as participants.
@wjost could you please confirm?

Hereby confirmed.

Nö changes regarding interoperability since last release. We will recalibrate with 24.05.
@kelaja approved.

@kelaja please update the information above
Version to be included in Eclipse Tractus-X release: 0.3.0
Leading product repository: https://github.com/eclipse-tractusx/managed-identity-wallet

SAST: Approved
SCA: Approved
DAST: Approved
Infrastructure as Code: Approved
Secret scans: Approved

@kelaja as there were no significant changes related to GDPR, the declaration of R23.09 can be used. See Jira-ticket https://jira.catena-x.net/browse/CXRM-1883

Threat Modeling (Security Assessment Process) - Approved
The assessment was last updated by Pablo at 28th November 2023.
For the upcoming release there is no open critical & high findings.
Some security related work based on currently existing recommendations are planned for the release in May and will be evaluated in the following assessment that have to be updated before the release 24.05.

Overview E2E Test Result: https://confluence.catena-x.net/display/PL/2024-02-13+Daily+E2E+Test+-+Release+24.03

INT test performed/documented.
E2E test performed/documented.
No high defect.
TM approved
@kelaja : please approve for "E2E Integration Test passed"

Data Sovereignty requirements unchanged for 24.03; @vialkoje could you please approve?

MIW is part of E2E journey Portal/MIW/EDC. For R24.03 no additional business logic was added in regarding the summary credential, certificates and data sovereignty (use case framework credentials),
BO approval done.

DataSov & Docu pending expert approval
Container Scans
TRG in work
QG approval postponed

Discussed with @borisrizov-zf
Container Scans: Approved

MIW version 0.3.0 has already been Q-Gated in release 23.12. Won't check again for release 24.03.

Documentation for @vialkoje

Interface documentation
Architecture Documents Arc42
Administrators Guide
End-User manual

@vialkoje could you please give your expert approval regarding Data Sovereignty and Documentation? Thanks!

Documentation links perfectly provided. Documentation is existing and looking consistent. Expert Approval granted for documentation.
Data sovereignty requirements unchanged for 24.03 - Expert Approval granted
Please consider the Quality Gate requirements for the 24.05 release. Thank you !

all pre-conditions fulfilled
QG-approval granted
Congrats

final conclusion: TRG 5.07 violated (postgresql DB version 14.x) but successfully tested.
this is covered in overall release note 24.03
QG closed with these conditions.


connect: Strip leading whitespaces in test data.

README and test show test data as such for readability, test data should
should be passed in like the README indicates, without the spaces added
for readability.
Fixes #810 outstanding issue by striping out leading spaces in constructed board array.

How about also adding the re-generated test suite?

Thanks for doing this @tryantwit

If everything looks good, I can squash all this into 1 commit before merge if you want.

Yep looks good.
If you want to squash it, make it 2 commits, one for the generator change and one for the new test case file.

Cleaned up the commits, should be good to go.

Awesome, thanks a lot @tryantwit :heart:


Which database is used in tinyMediaManager?

Can third-party clients browser and edit it's contents?

no, we use the mvstore from H2 along with JSON to create our own document database.
every other approach would be either system dependend (sqlite, mysql, postresql) or huge (h2)


Trailing closing braces next to / cause a compiler error

A trailing curly-brace in a javascript expression next to a self-closing tag's / appears to cause a compiler error in tag definitions. For instance, supplying a conditional binary argument to  as suggested in the riot tag-syntax docs fails when that input tag is self-closing and the curly-braced expression comes last.  For instance:
<input checked={ undefined }/>

causes this compiler error:
[Error] InvalidCharacterError: DOM Exception 5: An invalid or illegal character was specified, such as in an XML name.  setAttribute (riot@2.0(riot.min.js+compiler.min.js), line 3)

Removing the /, or adding a space between the closing brace and the /, prevent the compiler error and are suitable workarounds.  I know it is customary to precede self-closing tag slashes with whitespace, and that  is an open tag, but the riot self-closing tag examples don't follow the space-before-self-closing-tag-slash convention, all parsers I'm aware of permit making  self-closing, and users may be confused.
Thanks for riot, I'm enjoying what I see so far!

I don't seem to get that error locally. I'm compiling this:
<test>
  <input checked={ undefined }/>
</test>

to this
riot.tag('test', '<input __checked="{undefined}">', function(opts) {

});

Can you make a jsfiddle that would show the error?
Thanks!

In making the fiddle I see that the error shows up only when update() is called.  (I've been doing that after loading data from an AJAX request).  Here's a minimal case:
http://jsfiddle.net/07mrwbs5/
Thanks for looking at it!  My apologies in advance if I'm bringing this on myself by using update() inappropriately.
Code from that fiddle:
<!DOCTYPE html>
<html lang="en">
<body>
<test-component/>

<script type="riot/tag">
  <test-component>
    <input checked={ done }/>
    this.items = []
    setTimeout(this.update,0)
  </test-component>
</script>

<script src="//cdn.jsdelivr.net/g/riot@2.0(riot.min.js+compiler.min.js)"></script>
<script>
  riot.mount('*')
</script>

</body>
</html>

I reran this fiddle this morning and am, without any change I'm aware of, no longer getting the error, after getting it consistently time and time again a few hours ago.  I ran the local codebase where I first noticed it and the error is gone there as well. (!!)  I will dig deeper to try to get it to be consistent.

This seems to work on the latest version:
cdn.jsdelivr.net/g/riot@2.0.10(riot.min.js+compiler.min.js)
The jsfiddle used minor version (2.0) and my guess is that earlier you were using an older buggy version which was fixed on 2.0.10.
Anyway, closing this one. Thanks!


Help links in documentation point to wrong locations

On this page: https://reactnavigation.org/help/
The link to "post an issue" points to feature request creation on Canny
The link to "create a feature request on Canny" points to #react-navigation on Reactiflux Discord.

thanks! fixed in here https://github.com/react-navigation/react-navigation.github.io/commit/ae90b40fac4a57f473a41b9436a5bcea16ee9981


Weekly fork

Hello, I saw you made a weekly version of the plugin but I was wondering if it's possible to use it on a aternos server since only the main plugin can be found there.

I didn't make a weekly version of the plugin

hmm then what is this? (genuinely speaking, i really thought that was a weekly version!)
https://github.com/J0J0HA/PlayerTimeLimitWeekly/releases/tag/v1.1.0

It may be, mut it isn't mine. Everyone can make a fork on github.

ohh i see, tysm for the clarification! and is there a way to change it so it resets weekly instead? it'd be super great ^^


Criteo Adapter: Add Price floor support (#8815)

🏷 Type of documentation

[ ] new bid adapter
[ ] update bid adapter
[x] new feature
[ ] text edit only (wording, typos)
[ ] bugfix (code examples)
[ ] new examples

📋 Checklist

[x] Related pull requests in prebid.js or server are linked
[ ] For new adapters check submitting your adapter docs

https://github.com/prebid/Prebid.js/pull/8815


feat(ci): add llvm-cov to template ci

This adds the cut/paste llvm-cov job from the docs.
I tested this by creating a project with the template and put it here: https://github.com/joshrotenberg/rust-template-llvm-cov-test
I removed the call to upload the coverage file in the workflow because it requires the token in secrets, which means the workflow fails by default, which seems bad, but I'm open to suggestions.
Fixes #44

Thanks for this!
How is the user supposed to use the lcov.info file?
It would be nice to show the coverage percentage in the readme. This repo used coveralls for this.

Thanks for this!

👍

How is the user supposed to use the lcov.info file?

Good question. I just added the coveralls action with just the file argument which seems to work out of the box and has a lot of configuration options: https://github.com/marketplace/actions/coveralls-github-action
The output for the run is here: https://github.com/joshrotenberg/rust-template-llvm-cov-test/actions/runs/8530312705/job/23367853431

It would be nice to show the coverage percentage in the readme. This repo used coveralls for this.

Where is this? I don't see it.


Switch MNAIO image manifest URL from DFW to IAD

Currently DFW is undergoing a data centre migration, resulting
in artifact uploads not synchronising quickly enough to provide
the download in time, and therefore producing inconsistencies
when downloading objects - especially large objects.
Testing has proven that IAD is much more reliable, so in this
PR we switch all artifact downloads to use IAD instead.
Issue: RE-2099

Thank you for submitting a pull request to this project!
Once all check jobs have passed and the pull request has received a
sufficient number of reviews, the repository's gate jobs can be triggered by
adding a comment of :shipit: to this pull request.
When the gate jobs have completed successfully, this pull request will get
merged automatically.

The manifest.json file is present on all of the lastest builds on the IAD CDN. This will prevent failure in the running CI jobs. Please note that the manifest.json file is not present in the older builds on the CDN.


Diff no longer shown when using isort --check-only --diff as of v4.2.9

In our CI we use isort --check-only --diff --quiet, so that if there are any errors the diff is shown so developers know what to fix without having to run isort again themselves locally.
This used to work as of v4.2.8, eg:
https://travis-ci.org/mozilla/treeherder/jobs/238235705#L240
However does not in v4.2.9, eg:
https://travis-ci.org/mozilla/treeherder/jobs/238401565#L240
It appears to have been regressed by the return added in:
https://github.com/timothycrosley/isort/commit/6bf21fc1261ef46dd5b66561e5aae8a0ce76666b

(Have edited the OP to correct the commit link)

Thanks for reporting!
This has been fixed in a 4.2.13 hotfix release
~Timothy

Many thanks! :-)


Delete Button Needs a Confirm Step/State

This delete button just goes ahead and nukes the listing with no confirmation. Probably not a great approach unless we can back out of it somehow.

@morebrownies I'll add this to the backlog as a design task as well as a UI task. We should probably have a general purpose confirm state for things, either a two-step button or a tooltip like we have in the transaction designs.

I'm pretty sure this was designed already and it's essentially the same as is already in place when trying to delete a server config:

So.. i imagine some code could possible be re-used.
Also... I have a feeling there may already be a big for this.

The zeplin project for editing listings is closed, can you find the design and paste a screenshot here?

Yeah it is the same thing, as @rmisio pointed out. I can re-open the zeplin project.

Cool. Thanks.


CLI alias for onchain operations

Component
Cast
Describe the feature you would like
@gakonst talked about this a bit here. The use case is clear, it can make it easier and help save time for a lot of common on-chain operations.
The challenge is to make it a plugin-like system, such that users can easily add new commands.
Additional context
Have started looking into this, specifically at the method described here. I will try to learn more about clap, and see if this approach makes sense.
Suggestions for alternative methods are welcome!

oh yeah, the plugin system should probably be able to resolve that. Though the tasks described there seem much more complicated than what is described here, think this can very well act as a starting ground.

Let's consolidate maybe the discussion in the plugins issue?


nvcc fatal   : redefinition of argument 'std'

It proves remove the faiss from src/CMakeLists.txt cannot even solve the problem, said

It turns out:

by the way, I was building without faiss using the -DNO_FAISS option for cmake.
AND If I comment out CMAKE_CUDA_FLAGS line setting -std=c++11. I got a new error at the end of make :

Thanks in advance!

Could you use make VERBOSE=1 to show what the other std argument is?
The second one is desired, since pybind11 heavily relies on C++11 features.


Query deployment status

After a new version is published, the old version is still running for some minutes.
My application needs to know, if it is the old version, so it can stop making requests to other services.

Hi! I do not understand well your need. Could you give me more insights ?


Turbo Stream MIME type doesn't match Turbo Handbook

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary
I recently updated my version of Turbo, and that caused my turbo streams to fail with an error

Error: Form responses must redirect to another location
    requestSucceededWithResponse turbo.es2017-esm.js:561
    receive turbo.es2017-esm.js:352
    perform turbo.es2017-esm.js:335
    start turbo.es2017-esm.js:528
    submitForm turbo.es2017-esm.js:1800
    formSubmitted turbo.es2017-esm.js:2335
    submitBubbled turbo.es2017-esm.js:1576
    submitCaptured turbo.es2017-esm.js:1566
    start turbo.es2017-esm.js:1585
    start turbo.es2017-esm.js:2246
    start turbo.es2017-esm.js:2837
    node_modules bundle.js:2932
    Webpack 4
turbo.es2017-esm.js:1846
    formSubmissionErrored turbo.es2017-esm.js:1846
    requestSucceededWithResponse turbo.es2017-esm.js:562
    receive turbo.es2017-esm.js:352
    perform turbo.es2017-esm.js:335
    start turbo.es2017-esm.js:528
    submitForm turbo.es2017-esm.js:1800
    formSubmitted turbo.es2017-esm.js:2335
    submitBubbled turbo.es2017-esm.js:1576
    (Async: EventListener.handleEvent)
    submitCaptured turbo.es2017-esm.js:1566
    (Async: EventListener.handleEvent)
    start turbo.es2017-esm.js:1585
    start turbo.es2017-esm.js:2246
    start turbo.es2017-esm.js:2837
    js turbo.es2017-esm.js:2875
    Webpack 4

The error message sounds like turbo is not seeing the requirement of HTTP code 303 as described in Redirecting After a Form Submission in the Turbo handbook. This however doesn't match what I am doing which is Streaming After a Form Submission. In that section, MIME type is mentioned.
I looked at the Content-Type that express-hotwire was sending, and that was text/html; turbo-stream; charset=utf-8. I think that might be an outdated mimetype for turbo.
https://github.com/deriegle/express-hotwire/blob/6c82d13bfef1f34818f2897b67231f5de0b74463/src/lib/hotwire-middleware.ts#L138
I think that mimetype works for certain versions of Turbo. I see that mimetype in turbo-7.0.0-beta.1 but not in turbo-7.0.0-beta.7.
I think the Content-Type sent by res.turboStream should now be text/vnd.turbo-stream.html

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)

I circumvented res.turboStream to test if this other mimetype solved the problem, and it did.
Broken
                res.turboStream.append('chats', {
                    partial: 'chat/partials/chat',
                    locals: {
                        chat
                    }
                })

Working
                res.set('Content-Type', 'text/vnd.turbo-stream.html');
                res.render('chat/partials/chat', { chat });

Of course this is a workaround just to see if I had correctly identified the problem. I appreciate this library so I forked and came up with a patch. I'm not sure if it's right, because the older mimetype has 'utf-8' in there and I don't know if that's supposed to be in there on the new one. It definitely works without the utf-8, though.
Also I'm unsure about git cz mentioned in the Contributing guidelines. is that supposed to be git commit -cz? And is that only needed if a git commit was created without a message?
Ohh... It's an actual program! https://github.com/commitizen/cz-cli
I'm going to have to take some time to learn cz but for now here's a personal fork which implements the updated mimetype and solved the issue I was seeing.
https://github.com/deriegle/express-hotwire/compare/main...insanity54:main#diff-b7ad7cfc2914bb92aa8b9e7c0892dcc53e9900e7940a31e77b0518118ee92860

@insanity54 Thank you so much for such a detailed issue description and for forking the repository and adding your fix. I didn't notice this issue until now, but I have resolved this issue in the new version 0.3.1 of this package.
<EMAIL_ADDRESS>


Higher resolution labelmap?

The cornerstone v.4.0 looks really good, thank you for your great work.
Just one question, I think right now, the resolution of the label map is align with the original dicom dimension. (Am I right?)  The contours are very smooth when drawing, but the contour become very zigzag after release the mouse. I think maybe that's because the label map has a small resolution?
I just wondering how can we config the label map to the sub-pixel accuracy? For instance, the original dicom slices are 200x200, the label maps are 600x600 and then scale the dicom slice as the same size of the label map.
Thanks in advance.

I don't believe this is currently supported. Labelmaps are the same size as the slice/stack. Is there value in sub-pixel accuracy? I'm not a radiologist, so I would appreciate any info you have on real-world use cases. I'm trying to understand when you would need to be more granular with your measurements than the modality. I'm unsure of what you would be measuring, as anything sub pixel would be an interpolated value.

For example, if I have a square of pixels/intensities:
5, 10, 7
0, 10, 5
5, 10, 3
Mapped to indexes:
0, 1, 2
3, 4, 5
6, 7, 8
If I want include the range from 0.25 to 1.23 what am I actually including? How would you calculate average intensity for the area? Does that math change when we shift to volumes? What does this level of granularity buy us?

The resolution of the label map is align with the original dicom dimension.

You are correct.  For now we have binary masks per voxel.
We currently do this because:
a) You can write reasonable implementations on the CPU.
b) Its the simplest implementation and is better than we had before.
In general this isn't a simple issue. If what you want is simply a nice visual representation, then you may be better off using contours (e.g. the FreehandRoiTool + FreehandRoiSculpterTool), depending on your use case. However I understand the alure of the cornerstone labelmaps, as they are certainly more developed and user friendly at the current time.

I just wondering how can we config the label map to the sub-pixel accuracy? For instance, the original dicom slices are 200x200, the label maps are 600x600 and then scale the dicom slice as the same size of the label map.

You have two options here:

Supersample your images before they are given to cornerstone, and you can just use the current architecture for free (e.g. a 600x600 image will have a 600x600 labelmap per slice).
Modifiy the labelmap allocation, brush & segmentation tools to multiply the pixel density to a certain desired resolution. This will be a hardfork of the the codebase, however.

The issues @dannyrb raised aren't a problem if you super sample so your grid still aligns (i.e. 3x the resolution, as each voxel in 3x3 block would be the same value as the original voxel.
However, you would have to worry about counting partial volumes if you upscaled by some arbitrary factor.

I urge caution however, that as the labelmaps get too large you do start to see a performance drop, especially with the outline renderer. An ideal solution would keep the same underlying data and just display them differently.
I know Slicer has the option to resample labelmaps to a smooth isosurface which you can display as a nice smooth contour, but that may be challenging to do well fast in the web on the CPU. If you want to spike a webgl implementation, I'd be very happy to look at it. It may eventually be easier to do this sort of thing in vtkjs.

This is definitely something that would be good to have in the future - it's very common to want to mix and match segmentations and images with different geometries (resolution, but also origin and orientation should be free variables).  As James says, this is how Slicer does it and it's important for several clinical scenarios (and for making more faithful representations of anatomy).  Even though it's not the most common case, we should be keeping this in mind when implementing CPU and GPU segmentation code.
Note that the DICOM SEG object supports up to 8 bits per segment which can be used to represent fractional coverage for "soft" segmentation, but in my experience you are much better off supersampling a labelmap by 2 in each dimension, so those same 8 bits can encode more information about the intra-voxel shape, while also preventing overlap and being compatible with existing labelmap algorithms.

If I want include the range from 0.25 to 1.75 what am I actually including? How would you calculate average intensity for the area? Does that math change when we shift to volumes? What does this level of granularity buy us?
I think linear interpolation would be the case. For the current case, what the radiologist want is to display the contours more smoothly. However, to my knowledge, it is useful when contouring the plaques in CTA images. Even though there isn't much information for the sub-pixel grid, the radiologists still could draw a more precise sub-pixel contour when they refer to some other gold standard higher resolution modality images.

You have two options here:
Supersample your images before they are given to cornerstone, and you can just use the current architecture for free (e.g. a 600x600 image will have a 600x600 labelmap per slice).
That's my current solution, but as you said, upsampling the DICOM image means increase transfer bandwidth and proceeding time. I think maybe we could consider using float coordinates rather than binary mask in such case.

@JamesAPetts

Supersample your images before they are given to cornerstone

Do you know how to intercept the moment of passing image to cornerstone? I would like to do the same for segmentations and I found that segmentation image is rendered into the cornerstone canvas right after creating https://github.com/cornerstonejs/cornerstoneTools/blob/e70b894b51dba25e9329fca249f92fb57463bb49/src/eventListeners/internals/renderSegmentationFill.js#L127


Add the initial Community Building committee members.

Thanks for participating in Jupyter governance ✨
Below are a few guidelines for making your proposed change as efficient and productive
as possible.
Questions to answer ❓
Please answer the following questions along with your proposed change:
Background or context to help others understand the change.
We recently approved the Community Building committee on #105, with the initial members from the steering council being Ana, myself, and Sylvain.
A brief summary of the change.
This updates our people file. (It also corrects some typos of names in people.md).
What is the reason for this change?
Transparency of who is on this committee currently.
Alternatives to making this change and other considerations.
The process ❗
The process for changing the governance pages is as follows:

Open a pull request in draft state. This triggers a discussion and iteration phase
for your proposed changes.
When you believe enough discussion has happened,
move the pull request to an active state. This triggers a vote.
During the voting phase, no substantive changes may be made to the pull request.
The Steering Council will vote, and at the end of voting the pull request is merged or closed.

The discussion phase is meant to gather input and multiple perspectives from the community.
Make sure that the community has had an opportunity to weigh in on
the change before calling a vote. A good rule of thumb is to ask several steering council
members if they believe that it is time for a vote, and to let at least one person review
the pull request for structural quality and typos.

I'll leave this in draft stage for one week, and if discussion has settled, move it to the voting stage.

I'll leave this in draft stage for one week, and if discussion has settled, move it to the voting stage.

This PR has been in draft stage for almost two weeks. Marking it now as "ready for review".
For reference, the creation of the committee was approved in https://github.com/jupyter/governance/pull/105.

Per the committee charter, the initial committee is formed by 3 steering council members. This PR updates the docs to reflect the three steering council members who volunteered for this committee.


brew upgrade error:

After the recent update, brew upgrade now fails complaining about a new line in file /Library/Homebrew/cmd/install.rb on the line 198:
$brew upgrade
Error: [linuxbrew]/Library/Homebrew/cmd/install.rb:198: syntax error, unexpected '\n'

This line is:
198: interpreter = glibc && glibc.installed? ? glibc.lib/"ld-linux-x86-64.so.2"
199:    : "/lib64/ld-linux-x86-64.so.2"

If the line 199 is moved to 198, then error goes away. I guess this is a ruby-version issue.
$ brew --config
HOMEBREW_VERSION: 0.9.5
ORIGIN: https://github.com/Homebrew/linuxbrew.git
HEAD: c5f14b65b0ae32a4f8382d74bfd3104cae4650c4
Last commit: 41 minutes ago
HOMEBREW_PREFIX: [linuxbrew]
HOMEBREW_REPOSITORY: [linuxbrew]
HOMEBREW_CELLAR: [linuxbrew]/Cellar
HOMEBREW_BOTTLE_DOMAIN: https://linuxbrew.bintray.com
CPU: 16-core 64-bit 0x63e
Kernel: Linux 2.6.32-504.3.3.el6.x86_64 x86_64 GNU/Linux
OS: Scientific Linux release 6.7 (Carbon)
Codename: Carbon
OS glibc: 2.12
OS gcc: 4.4.7
Linuxbrew glibc: 2.19
Linuxbrew gcc: 5.3.0
Clang: N/A
X11: 2.7.8 => /usr
System Ruby: 1.8.7-p374 => /usr/bin/ruby
Perl: /usr/bin/perl
Python: /usr/bin/python
Ruby: /usr/bin/ruby
Java: 1.7.0_95

Fixed. Thanks for the bug report, Maxim.


Add SALT encoding and decoding

Insert it here:
https://github.com/BurnySc2/sc2-planner/blob/f806961a366e7b1a71702a5a48e043a5513a2b87/src/constants/helper.js#L182-L188
Use file /constants/salt_constants.js
References:
https://drive.google.com/file/d/0Bzrw_bC8iBjfSzFRRGlWWnNWNDg/view
https://github.com/Veritasimo/sc2-scrapbook/blob/68028fdbc7ce67a394ba139746e82485ebbf4312/SALT.cs#L150
https://www.reddit.com/r/starcraft/comments/25losh/build_order_tool_salt_update/chin15d/
Edit: Alternatively it could be done parsing the text of the spawningtool build order output by https://github.com/StoicLoofah/spawningtool

Closing in favor of issue https://github.com/BurnySc2/sc2-planner/issues/9
SALT encoding does contain less information than sc2-planner build orders - technically it could be exported as SALT encoding, but that seems a lot of work for now.
Importing from SALT encoding will lack information, so that would be fairly pointless.
Importing from replays would provide complete information.


Fix property constraints against resolved variables (#724)

Fixed an issue where an already resolved variable, when used for
a property constraint, errored out. See #724 for more details.
This also fixed a regression test that was erroring out, but was
overlooked.
Adjusted and added additional regression tests.
Co-authored-by: Dehowe Feng<EMAIL_ADDRESS>

John and I worked closely on this issue to resolve the limitations of the match clause in reflexive pattern matching. If anyone wants to take a look and review what was done, it would be much appreciated

Hi @dehowef @jrgemignani,
I was wondering why the query mentioned in issue still errors out.
SELECT * from cypher('my_graph_name', $$
  MATCH (a {prop: “val”})-[]- (a {prop2: “val2”})
$$) as (a agtype);

I tried this query in regression tests
SELECT * FROM cypher('cypher_match', $$
	MATCH (a {i: 1})-[]-(a {j: 2}) RETURN a $$) as (a agtype);
ERROR:  variable a already exists
LINE 2:  MATCH (a {i: 1})-[]-(a {j: 2}) RETURN a $$) as (a agtype);

@MuhammadTahaNaveed Actually, that's a good point and a bit of an oversight on my part. My PR fixes the clause to clause previously resolved variables, not within the same clause. I will try to add that in tomorrow and update this PR.

@MuhammadTahaNaveed The issues with MATCH occur in several sections of the code-- we have chosen to address them incrementally for maximum transparency and visibility for the project.
I feel like this method as several advantages, being that each commit will be itemized so that in the event that something is updated, we can go back to the commit where a certain change was implemented to easily view and address it. Thanks!

@dehowef @MuhammadTahaNaveed I'm reviewing my patch to make sure it covers all the cases adequately.

@dehowef @MuhammadTahaNaveed I have updated the PR to point to the correct issue this PR resolves. If you can verify that it works against that issue? Sorry, for the confusion.


 By using cache we'll improve eval speed greatly.

BTW, this code may change. If you want to ensure that there is no regression on the performance, I suggest adding a benchmark for your particular case.

Is there any benchmark suits that I could add to? I suggest to run the benchmark test every time we do a release making sure the performance is acceptable.


error: block in materia lize': Could not find ffi-1.9.10 in any of the sources (Bundler::GemNotFound)

From a fresh clone, when I run jekyll serve I get this:
➜  brume git:(master) jekyll serve
/Users/sindhus/.rvm/gems/ruby-2.2.1/gems/bundler-1.12.5/lib/bundler/spec_set.rb:95:in `block in materia
lize': Could not find ffi-1.9.10 in any of the sources (Bundler::GemNotFound)
        from /Users/sindhus/.rvm/gems/ruby-2.2.1/gems/bundler-1.12.5/lib/bundler/spec_set.rb:88:in `map
!'
        from /Users/sindhus/.rvm/gems/ruby-2.2.1/gems/bundler-1.12.5/lib/bundler/spec_set.rb:88:in `mat
erialize'
        from /Users/sindhus/.rvm/gems/ruby-2.2.1/gems/bundler-1.12.5/lib/bundler/definition.rb:140:in `
specs'
        from /Users/sindhus/.rvm/gems/ruby-2.2.1/gems/bundler-1.12.5/lib/bundler/definition.rb:185:in `
specs_for'

        from /Users/sindhus/.rvm/gems/ruby-2.2.1/gems/bundler-1.12.5/lib/bundler/definition.rb:174:in `
requested_specs'
        from /Users/sindhus/.rvm/gems/ruby-2.2.1/gems/bundler-1.12.5/lib/bundler/environment.rb:19:in `
requested_specs'
        from /Users/sindhus/.rvm/gems/ruby-2.2.1/gems/bundler-1.12.5/lib/bundler/runtime.rb:14:in `setu
p'
        from /Users/sindhus/.rvm/gems/ruby-2.2.1/gems/bundler-1.12.5/lib/bundler.rb:95:in `setup'
        from /Users/sindhus/.rvm/gems/ruby-2.2.1/gems/jekyll-3.0.1/lib/jekyll/plugin_manager.rb:38:in `
require_from_bundler'
        from /Users/sindhus/.rvm/gems/ruby-2.2.1/gems/jekyll-3.0.1/bin/jekyll:13:in `<top (required)>'
        from /Users/sindhus/.rvm/gems/ruby-2.2.1/bin/jekyll:23:in `load'
        from /Users/sindhus/.rvm/gems/ruby-2.2.1/bin/jekyll:23:in `<main>'
        from /Users/sindhus/.rvm/gems/ruby-2.2.1/bin/ruby_executable_hooks:15:in `eval'
        from /Users/sindhus/.rvm/gems/ruby-2.2.1/bin/ruby_executable_hooks:15:in `<main>'

How do I fix this?
Thanks!

Sorry for the late response. This error was caused by Gemfile.lock file which I forgot to add to .gitignore and it had outdated gem dependencies. It should be fixed in cbe91e8da2e8674f2f2a8f44921594101ffcdc12.

Thanks! am no longer using jekyll (switched to nikola), do you have the same theme for nikola?

I haven't used nikola, but it should be easy to port this theme to any other static site generator.


Cuda Stream Support

Thank you for creating and maintaining this library, it's great being able to experiment with machine learning in Rust. I am looking for potential performance improvements of some code that uses it.
Based on the CUDA streams section at https://pytorch.org/docs/stable/notes/cuda.html, it is my understanding that using streams is necessary to allow multiple operations to execute concurrently on a single GPU (one workaround could be to run operations from different processes, but that also has overhead).
Here are some sources I found that describe how streams can be used with the C++ API:

https://discuss.pytorch.org/t/cuda-streams-in-c-lib/38217
https://github.com/pytorch/pytorch/issues/16614#issuecomment-461160875

Would it be possible to add support for CUDA streams to tch?

Related C++ struct: at::cuda::CUDAStreamGuard.

Right now I'm loading your great rust-bert crate to run a model. I'd like to have a bunch of Rust threads do operations on the same model but it seems that I gradually run out of memory. I could wrap the model (and therefor all of torch) in a mutex, but then I can't saturate the GPU's throughput. I suspect that if I put every rust thread in its own Stream it would work. Do you agree with that?
If so, what are the steps for getting this feature merged? I could start by actually testing it!


Obsoletion notice: GO:0052812 phosphatidylinositol-3,4-bisphosphate 5-kinase activity

Dear all,
The proposal has been made to obsolete 'GO:0052812 phosphatidylinositol-3,4-bisphosphate 5-kinase activity'. The reason for obsoletion is that there is no evidence that this activity exists.
There are no more manual annotations to this term (1 incorrect EXP removed in https://github.com/geneontology/go-annotation/issues/4569); this term was incorrectly mapped to EC:<IP_ADDRESS>, which actually corresponds to GO:0046934 'phosphatidylinositol-4,5-bisphosphate 3-kinase activity'. This term is not present in any subsets.
You can comment on the ticket: https://github.com/geneontology/go-ontology/issues/25355
Thanks, Pascale

Actually there is one Reactome reaction - https://reactome.org/content/detail/R-HSA-1675773
Need to check references

This Reactome reaction may correspond to a difference substrate of https://enzyme.expasy.org/EC/<IP_ADDRESS>
@sjm41 @deustp01 any thoughts on this?

This Reactome reaction may correspond to a difference substrate of https://enzyme.expasy.org/EC/<IP_ADDRESS>

Yes, that is how I read this part of the entry for <IP_ADDRESS> -

Comment(s)
This enzyme can also phosphorylate PtdIns3P in the 4-position,     and PtdIns, PtdIns3P and PtdIns(3,4)P2 in the 5-position in vitro, but to a lesser extent. The last of these reactions occurs in vivo and is physiologically relevant.

The "last of these reactions" looks like the one that Mark Williams annotated for Reactome  https://reactome.org/content/detail/R-HSA-1675773. You'll see that, in agreement with the EC entry, Mark was not sure about the physiological role of this reaction.
@sjm41 @pgaudet  So practically, what do we do? Keep the current GO term? Keep the current Reactome reaction and its enzyme but use a different GO term? Either fix seems OK to me and there may be others I'm overlooking.
Thanks

So <IP_ADDRESS> corresponds to GO:0016308 "1-phosphatidylinositol-4-phosphate 5-kinase activity", but the GO def (and the RHEA:14425 ref) are specific to the main reaction (1-phosphatidyl-1D-myo-inositol 4-phosphate -> 1-phosphatidyl-1D-myo-inositol 4,5-bisphosphate).
I don't see a specific entry for the "1-phosphatidyl-1D-myo-inositol 3,4-bisphosphate -> 1-phosphatidyl-1D-myo-inositol 3,4,5-trisphosphate" reaction in either EC or RHEA. Given this, and the fact the Reactome entry says "This is a minor reaction, however, and its physiological role is uncertain.", I'd suggest we go ahead with the obsoletion and Reactome reannotate with the parent term "GO:0052742 phosphatidylinositol kinase activity".

I'd suggest we go ahead with the obsoletion and Reactome reannotate with the parent term "GO:0052742 phosphatidylinositol kinase activity".

Re-annotation will be done today and will become publicly visible with our June release.


feat: virtual machine groups - add resource

Adds the Terraform resource for managing Virtual Machine Groups in Katapult.
This is just the first part to fully supporting groups, as we will need to add support to use it as a data source and to add them to virtual machines.

To expand further on testing, the convention I've gone with is have a _minimal test where I only specify required attributes and validate what everything gets set to. This could also double as the test for when segregate is not specified. Then I'd suggest a test for when it's set to true, and one for when it's set to false, in addition to a update test which changes the name, and segregate value back and fourth between true/false.


But how can i start a Washing Program with Options from HomeAssistant side ?

Hey, you have a configuration section where you can configure with which settings you want to start the program

and then you have the control section here

For washing machines, you have to set the hOn program on the machine, analog to the app. Then the controls become available in home assistant and you can start it 🙂


refactor: testing matcher and logging

Refactors the error matcher to accommodate the fact that we know the shape of the things we are testing. This is only a change to our own matchers and is not publicly exposed. This was tested by intentionally breaking several tests and checking outcomes. The debugging information provided by the assertion is also improved.
I can't believe I didn't spend some time doing this ages ago. There's still room for improvement here, but this does make things easier to debug.

Rethinking this change. As most people will be using a deep equals algorithm that is less forgiving with notifications structures. (i.e. matching { kind: 'N', value: 1 } with { kind: 'N', value: 1, error: undefined }.

fyi I made a custom matcher once:

https://github.com/kwonoj/rx-sandbox/blob/master/src/assert/marbleAssert.ts


Fuzzilli options to generate js from grammar file and linter

I'm learning fuzzilli now, but I don't know my grammar correct or not. I think by adding options to generate js given grammar file, we can check resulting js valid or not.
fuzzilli generate example_grammar
generate js from example_grammar
fuzzilli generate -n 5 example_grammar
generate 5 js combinations from example_grammar
Adding linter I think will be helpful too
fuzzilli lint example_grammar
a linter to check correctness of a grammar

Fuzzilli is not a grammar-based fuzzer, instead it is based on mutations on a custom IL: https://github.com/googleprojectzero/fuzzilli#concept
For a grammar-based fuzzer, take a look at domato for example: https://github.com/googleprojectzero/domato :)


Extend description for date parameters

For parameters which expect a date a reference to derive_vars_dt() should be added.

Hi Stefan, here is the steps that I am planning to do:

find all the functions containing a parameter expecting a date object.
In the header, For "@param [date]" ...,  add  "Refer to derive_var_dt() to impute and derive a date from a date character vector to a date object".

Please confirm if this will address the issue? Thanks

I would add the sentence rather to the description of the parameter than the "Details" section.

If would be repetitive if we add the sentence to each of the parameter description. For example, compute_duration() has 2 date parameter (eg. start_date, end_date).  How about adding the sentence to the "Description" section as a general guidance to the user?

I would repeat the sentence if a function has more than one date parameter because then it is more likely that the users find the information.

That makes sense. I will update as suggested.

Here is the steps:

Find all .R files that contains a date parameter using grep "@param" *.R| grep "date".
Go through the functions and add the sentence below to the date parameter expecting a date object.

"Refer to derive_var_dt() to impute and derive a date from a date character vector to a date object."


Get captcha to image

Hi it's me again, thanks for your help last time. This time I have a kind of annoying captcha, I tried to do the decoding but still failed. This captcha is not the same as normal, it requires converting letters into numbers. Hope to receive your help!
Here are some examples of this captcha code type

You'll need to do something a lot more custom here. It always helps to start by noticing the flaws in the system: it's always 5 characters, there's a legend mapping the characters to one of only 5 digits, etc. If this is a game then it's also possible that there are other ways to obtain the data, like maybe memory editing (search for an actual solution in memory and save the address to look up other solutions in the future) or brute force (if it doesn't lock you out after a few invalid attempts then there's only 5^5=3125 combinations per puzzle to solve, so just try them all until one works). Hope that helps.


Ignore patterns specified in excludes in docfx.json

@IEvangelist I'm going to close for now in favor of your suggested simpler approach: https://github.com/dotnet/docs-actions/pull/25.


Realm remove pk password (WIP)

WIP, do NOT merge yet.

@xiaoluSF Oops. Don't you see WIP and do NOT merge?


[#626] chore(ci): skip build operator if no code changes

What changes were proposed in this pull request?
skip build operator if no related code change
Why are the changes needed?
Speed up and reduce power usage for CI.
Fix: #626
Does this PR introduce any user-facing change?
No.
How was this patch tested?
github actions.

Codecov Report

Merging #628 (5112cd7) into master (7814cb9) will increase coverage by 3.55%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #628      +/-   ##
============================================
+ Coverage     60.90%   64.46%   +3.55%
+ Complexity     1799     1666     -133
============================================
  Files           214      187      -27
  Lines         12381     9134    -3247
  Branches       1042      920     -122
============================================
- Hits           7541     5888    -1653
+ Misses         4437     2933    -1504
+ Partials        403      313      -90

Impacted Files
Coverage Δ

.../java/org/apache/uniffle/server/ShuffleServer.java
60.95% <0.00%> (-3.43%)
:arrow_down:

...storage/handler/impl/DataSkippableReadHandler.java
83.78% <0.00%> (-2.71%)
:arrow_down:

...java/org/apache/hadoop/mapred/SortWriteBuffer.java

.../java/org/apache/hadoop/mapreduce/RssMRConfig.java

...oy/kubernetes/operator/pkg/controller/util/util.go

deploy/kubernetes/operator/pkg/utils/util.go

...n/java/org/apache/hadoop/mapreduce/RssMRUtils.java

...preduce/task/reduce/RssRemoteMergeManagerImpl.java

...pkg/controller/sync/shuffleserver/shuffleserver.go

deploy/kubernetes/operator/pkg/utils/rss.go

... and 19 more

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

I have an idea, run selected CI on PR, run full CI on master.

Anyway, do you have more comments on this PR? I prefer the improvement to this PR should be added in later prs.

I still believe we should run full CI on master for safety and consistency, especially when some CI are skipped in PR.
Cc @jerqi @zuston @xianjingfeng WDYT?

Anyway, do you have more comments on this PR? I prefer the improvement to this PR should be added in later prs.

I still believe we should run full CI on master for safety and consistency, especially when some CI are skipped in PR.
Cc @jerqi @zuston @xianjingfeng WDYT?

+1

We may still need to review how CI should be ran to reduce power usage later.

Another improvement is to turn off CI on draft PRs. And trigger CI when a draft PR becomes ready to review.


Very slow filling an input with Mink/Selenium/PhantomJS

Hey,
I'm very new to Behat/Mink/PhantomJS and overall BDD. I just got it all working with PhantomJS to test my websites with javascript.
Everything is working fine so far, but its painly SLOW. My test only fetches the login-page and fills the username and password fields, and does nothing else. This test already takes 40 seconds to finish.
Does anyone know, what the issue could be? Don't tell me this is the normal testing time? :)
I'm running the phantomjs webdriver (phantomjs --webdriver=8643) and am using it with Behat/Mink/Selenium. Please tell me of you need any other information (I'm quite confused right now).
Thanks!

Tools you use (phantomjs/selenium/other similar) ARE SLOW, and will always be. That's their nature. It's not something anyone can fix. This is a Behat repo by the way. Your issue is more related to Mink.
If you want quick tests do not go through the user interface, but exercise your code directly (then, obviously have few integration or smoke tests going through the whole thing).

CLosing because this is indeed not a Behat issue. Behat itself is not slow (the WebDriver stack for Mink is because of how WebDriver works)

You could probably make it a bit faster, but not by much.
As Jakub said, going through UI in your tests is your personal choice, not something BDD or Behat dictates. And going through UI will always be slow, because instead of simply exercising code directly you go through the entire app + phantomjs stack.
I'm closing this issue as it's completely unrelated to Behat.

And going through the UI with WebDriver is even slower than using an headless driver for Mink (BrowserKit, Goutte or Zombie)

Alright, I'm going to ask at the MInk-repo then.
@stof As i said, I'm still quite confused with all of this. Isn't PhantomJS also headless? What am I not getting in your last comment?
I want to go through the UI to make it as "real" as possible. And i need Javascript for this. Unfortunately Zombie didn't work on my machine, so I switched to PhantomJS.


OpenCvSharp on Raspberry Pi 4 Model B install

Summary of your issue
I want to use OpenCVSharp fore some motion detection projects.
Environment
Raspberry Pi 4 Model B
What did you intend to be?
I wanted to cross compile an arm version of OpenCVSharp on my Win 10 desktop pc.
Has someone done this and have an instructions, step-by-step for me?

Hi,
i have successfully compiled OpenCV and OpenCVSharp on my Raspberry Pi. Version 4.3.0. I used this tutorial: https://www.engineer2you.com/2017/11/install-opencv-raspberry-pi.html
Also i have build a program in VS 2019 for linux-arm, and it runs on RPi. Build on Windows in VS 2019: dotnet build --runtime linux-arm
On windows i have referenced OpenCVSharp nuget pkg for Windows and then i have replace OpenCvSharp (i think not needed) and additional i have copy libOpenCvSharp in executable folder on RPi to run a programm.

It was not so hard as i thought at the beginning.

Hi @Jerac102 ,
Did you copy OpenCVSharp.dll and libOpenCvSharp.so from your build repository. Do you had any problems with libOpenCvSharpExtern.so?
Do you think it is possible to do this without compilation of  OpenCV and OpenCVSharp?
There is no way to run this using only nuget packages and/or ready released dlls?

Hi @wsad4ryba,
what I am doing now is i just build it with dotnet and runtime arg: --configuration $(BuildConfiguration) -r linux-arm and it works.
Also I have included nuget package:   (from csproj file).
I don't cop anything (any more) from build repository, so I don't have any problems with libOpenCvSharpExtern.so.
As you see, no complation of OpenCV[Sharp] is needed.

Hi @jerkohren,
Thank you for replay.
I use VSC with publish -r linux-arm -o bin/linux-arm/publish\ and so far it has always worked.
Before asking you for help, I always included two references in the .csproj file.
I tested both publishing on windows and copying files to raspberry, and creating a project directly on raspberry with SDK installed.
so my .csproj files contained either:
   <ItemGroup>
     <PackageReference Include = "OpenCvSharp4" Version = "<IP_ADDRESS>10817" />
     <PackageReference Include = "OpenCvSharp4.Windows" Version = "<IP_ADDRESS>10817" />
   </ItemGroup>

or:
   <ItemGroup>
     <PackageReference Include = "OpenCvSharp4" Version = "<IP_ADDRESS>10817" />
     <PackageReference Include = "OpenCvSharp4.runtime.linux-arm" Version = "<IP_ADDRESS>10817" />
   </ItemGroup>

Running the program on windows it worked without any problems, but on raspberry, no matter if publishing or running directly on raspberry via dotnet run, I always got an error "Unable to load shared library 'OpenCvSharpExtern' or one of its dependencies". I tried all of this with <TargetFramework> net5.0 </TargetFramework>
and <TargetFramework> netcoreapp3.1 </TargetFramework>.
Now I checked your solution, so only one reference to <PackageReference Include = "OpenCvSharp4.Windows" Version = "<IP_ADDRESS>10817" />
The same effect, running on windows, works, not on raspberry.
Do you have any idea what I'm doing wrong?

Here is my publish profile and dotnet --info from RPi. Maybe it help you to figure out what is wrong.
<?xml version="1.0" encoding="utf-8"?>
<!--
https://go.microsoft.com/fwlink/?LinkID=208121.
-->
<Project ToolsVersion="4.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
  <PropertyGroup>
    <Configuration>Debug</Configuration>
    <Platform>Any CPU</Platform>
    <PublishDir>bin\Debug\net5.0\publish\</PublishDir>
    <PublishProtocol>FileSystem</PublishProtocol>
    <TargetFramework>net5.0</TargetFramework>
    <RuntimeIdentifier>linux-arm</RuntimeIdentifier>
    <SelfContained>true</SelfContained>
    <PublishSingleFile>False</PublishSingleFile>
    <PublishTrimmed>False</PublishTrimmed>
  </PropertyGroup>
</Project>

dotnet --info:
dotnet --info
.NET SDK (gemäß "global.json"):
Version:   6.0.100-preview.3.21168.18
Commit:    823c1dfceb
Laufzeitumgebung:
OS Name:     raspbian
OS Version:  10
OS Platform: Linux
RID:         linux-arm
Base Path:   /usr/share/dotnet/sdk/6.0.100-preview.3.21168.18/
Host (useful for support):
Version: 6.0.0-preview.3.21167.1
Commit:  0f64b267ac
.NET SDKs installed:
6.0.100-preview.3.21168.18 [/usr/share/dotnet/sdk]
.NET runtimes installed:
Microsoft.AspNetCore.App 6.0.0-preview.3.21167.12 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.NETCore.App 6.0.0-preview.3.21167.1 [/usr/share/dotnet/shared/Microsoft.NETCore.App]
To install additional .NET runtimes or SDKs:
https://aka.ms/dotnet-download

I am still getting issues.
My project file:
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>net6.0</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <Compile Include="Program.fs" />
  </ItemGroup>

  <ItemGroup>
     <PackageReference Include="OpenCvSharp4" Version="<IP_ADDRESS>11207" />
     <PackageReference Include="OpenCvSharp4.runtime.linux-arm" Version="<IP_ADDRESS>11207" />
   </ItemGroup>

</Project>

I run below command directly in Raspberry pi 4:
dotnet publish -r linux-arm --no-self-contained
Then I run:
dotnet OpenCVDemo.dll
After I execute I got below error:
Unhandled exception. System.TypeInitializationException: The type initializer for 'OpenCvSharp.Internal.NativeMethods' threw an exception.
---> System.DllNotFoundException: Unable to load shared library 'OpenCvSharpExtern' or one of its dependencies. In order to help diagnose loading problems, consider setting the LD_DEBUG environment variable: libOpenCvSharpExtern: cannot open shared object file: No such file or directory
at OpenCvSharp.Internal.NativeMethods.redirectError(CvErrorCallback errCallback, IntPtr userdata, IntPtr& prevUserdata)
at OpenCvSharp.Internal.ExceptionHandler.RegisterExceptionCallback()
at OpenCvSharp.Internal.NativeMethods.LoadLibraries(IEnumerable`1 additionalPaths)
at OpenCvSharp.Internal.NativeMethods..cctor()
--- End of inner exception stack trace ---
at OpenCvSharp.Internal.NativeMethods.videoio_VideoCapture_new1(IntPtr& returnValue)
at OpenCvSharp.VideoCapture..ctor()

Hi @jerkohren,
The link you gave for the installation of opencvsharp4 on raspi 4 is not working. Any other link that  you can suggest? Thank you.

hi @ffrankozz ,
i don't have other links  and I don't need to build OpenCvSharp4 on RPi. Today I am using nuget package (OpenCvSharp4.Windows) and my app is deployed to RPi trough Azure DevOps CD/CI pipeline, and architecture is ARM.
Sorry that I can't help.

@jerkohren Thank you for your reply.   if I am not using Azure Devops, I can build the linux docker image of my app on my Windows 10  m/c and run it on Raspberry pi, provided that the raspi has docker installed, I suppose.

@ffrankozz i suppose you can, because for testing i am publishing ARM from my windows visual studio to the RPi


@conda-forge-admin, please add bot automerge

Additional comment:
@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/pygmailsorter-feedstock#2.


fog gem required but not installed

Its quite possible I have a borked system, but every time I tried to run dzt, I got errors, such as
Brians-Mac-mini:~ brian$ dzt
/Users/brian/.rvm/rubies/ruby-2.1.1/lib/ruby/2.1.0/rubygems/core_ext/kernel_require.rb:55:in `require': cannot load such file -- fog (LoadError)
	from /Users/brian/.rvm/rubies/ruby-2.1.1/lib/ruby/2.1.0/rubygems/core_ext/kernel_require.rb:55:in `require'
	from /Users/brian/.rvm/gems/ruby-2.1.1/gems/dzt-0.2.1/bin/dzt:4:in `<top (required)>'
	from /Users/brian/.rvm/gems/ruby-2.1.1/bin/dzt:23:in `load'
	from /Users/brian/.rvm/gems/ruby-2.1.1/bin/dzt:23:in `<main>'
	from /Users/brian/.rvm/gems/ruby-2.1.1/bin/ruby_executable_hooks:15:in `eval'
	from /Users/brian/.rvm/gems/ruby-2.1.1/bin/ruby_executable_hooks:15:in `<main>'
Brians-Mac-mini:~ brian$ subl /Users/brian/.rvm/gems/ruby-2.1.1/gems/dzt-0.2.1/bin/dzt

After trying a bunch of stuff, and going in circles, I tried gem install fog, and then the dzt command worked. Is it possible its not setup correctly in the gem dependencies? Is this a rvm issue?
Anyway, it seems to be working now, but thought I would mention it in case it helped anyone else.

Fixed in https://github.com/dblock/dzt/commit/bb0a311cb34d4c548ee9f4f36dd16fc8113efd83.

Released in 0.2.2.


Can someone tell me which source file is the consensus algorithm?

Are you serious, that you don't even know this basic stuff about the Achain-blockchain? What kind of project is this?


[Bug] blockView is not exported from @milkdown/plugin-block

Initial checklist

[X] I agree to follow the code of conduct
[X] I searched issues and discussions and couldn’t find anything (or linked relevant results below)

Affected packages and versions
7.3.3
Link to runnable example
https://codesandbox.io/p/sandbox/confident-ritchie-d9lv98?layout=%257B%2522sidebarPanel%2522%253A%2522EXPLORER%2522%252C%2522rootPanelGroup%2522%253A%257B%2522direction%2522%253A%2522horizontal%2522%252C%2522contentType%2522%253A%2522UNKNOWN%2522%252C%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522id%2522%253A%2522ROOT_LAYOUT%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522UNKNOWN%2522%252C%2522direction%2522%253A%2522vertical%2522%252C%2522id%2522%253A%2522clrgqtm0t00062869o1p8elza%2522%252C%2522sizes%2522%253A%255B70%252C30%255D%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522EDITOR%2522%252C%2522direction%2522%253A%2522horizontal%2522%252C%2522id%2522%253A%2522EDITOR%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522EDITOR%2522%252C%2522id%2522%253A%2522clrgqtm0s00022869zokvcg6g%2522%257D%255D%257D%252C%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522SHELLS%2522%252C%2522direction%2522%253A%2522horizontal%2522%252C%2522id%2522%253A%2522SHELLS%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522SHELLS%2522%252C%2522id%2522%253A%2522clrgqtm0s00032869tqcjeznk%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%255D%257D%252C%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522DEVTOOLS%2522%252C%2522direction%2522%253A%2522vertical%2522%252C%2522id%2522%253A%2522DEVTOOLS%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522DEVTOOLS%2522%252C%2522id%2522%253A%2522clrgqtm0t00052869k6ipeal1%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%255D%252C%2522sizes%2522%253A%255B50%252C50%255D%257D%252C%2522tabbedPanels%2522%253A%257B%2522clrgqtm0s00022869zokvcg6g%2522%253A%257B%2522tabs%2522%253A%255B%257B%2522id%2522%253A%2522clrgqtm0s00012869enx5bqfj%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522FILE%2522%252C%2522filepath%2522%253A%2522%252Findex.html%2522%252C%2522state%2522%253A%2522IDLE%2522%257D%252C%257B%2522id%2522%253A%2522clrgqvcre003l2869jjtl3kyt%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522FILE%2522%252C%2522initialSelections%2522%253A%255B%257B%2522startLineNumber%2522%253A4%252C%2522startColumn%2522%253A48%252C%2522endLineNumber%2522%253A4%252C%2522endColumn%2522%253A48%257D%255D%252C%2522filepath%2522%253A%2522%252Fsrc%252Findex.ts%2522%252C%2522state%2522%253A%2522IDLE%2522%257D%255D%252C%2522id%2522%253A%2522clrgqtm0s00022869zokvcg6g%2522%252C%2522activeTabId%2522%253A%2522clrgqvcre003l2869jjtl3kyt%2522%257D%252C%2522clrgqtm0t00052869k6ipeal1%2522%253A%257B%2522tabs%2522%253A%255B%257B%2522id%2522%253A%2522clrgqtm0s00042869ko3zmcxb%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522UNASSIGNED_PORT%2522%252C%2522port%2522%253A0%252C%2522path%2522%253A%2522%252F%2522%257D%255D%252C%2522id%2522%253A%2522clrgqtm0t00052869k6ipeal1%2522%252C%2522activeTabId%2522%253A%2522clrgqtm0s00042869ko3zmcxb%2522%257D%252C%2522clrgqtm0s00032869tqcjeznk%2522%253A%257B%2522tabs%2522%253A%255B%255D%252C%2522id%2522%253A%2522clrgqtm0s00032869tqcjeznk%2522%257D%257D%252C%2522showDevtools%2522%253Atrue%252C%2522showShells%2522%253Atrue%252C%2522showSidebar%2522%253Atrue%252C%2522sidebarPanelSize%2522%253A15%257D
Steps to reproduce
Importing the blockView field from @milkdown/plugin-block is undefined, opposed to the instructions.
Expected behavior
I would assume there to be some kind of object there. I grep'd the plugin source code and couldn't find anything for blockView.
Actual behavior
Build fails.
Runtime
Chrome, Firefox
OS
Linux
Build and bundle tools
Rollup, Vite

The documentation is out dated. There's no blockView in block now. Checkout https://github.com/Milkdown/milkdown/blob/main/packages/crepe/src/feature/block-edit/handle/index.ts#L75


Fail to load front-end page after “azd up” for sample "AspireShop"

Describe the issue:
Run the azd up command successed but clicking on the front-end page reports the following error:

Repro Steps:

Clone code (preview.5 branch).
cd\samples\AspireShop
azd init
azd up
click on the front link.

Environment:

Azd version: azd version 1.8.0-beta.1-daily.3669344 (commit 9dc2a31392349f383afce69d94b563c4d58afe4d)
Aspire version: 8.0.0-preview.5.24201.12/8.0.100
Sample: AspireShop
Branch: preview.5

Expected behavior:
The front-end page can load normally.
@rajeshkamal5050, @vhvb1989 for notification.

Can you show the logs?

Ok, figure below:

I meant from the azure portal. It should show the exception.

Ok, figure below:

It won't work because the PostgreSQL container is set to use a volume which currently doesn't work in ACA (specifically, the PostgreSQL container attempts to do a chmod during its initialization and that fails when using volumes in ACA). If you remove the call to WithDataVolume() for the PostgreSQL and Redis resources it should work.

I just verified this myself using the changes in the preview.6 branch and latest VS/azd bits. If you update the Program.cs of the AppHost project so that no password parameter is set and the WithDataVolume() method is not called, it deploys and runs successfully on ACA:
var catalogDb = builder.AddPostgres("catalog")
    .AddDatabase("catalogdb");

var basketCache = builder.AddRedis("basketcache")
    .WithRedisCommander();


Update Leptos to use latest wasm-bindgen version

wasm-bindgen got another update, so I took the liberty of bumping the version used by Leptos.

We'll need to keep this in sync with the version in cargo-leptos, we should be able to merge this for the upcoming 0.7 dot release
On Fri, Nov 29, 2024, at 12:50 PM, SleeplessOne1917 wrote:

wasm-bindgen got another update, so I took the liberty of bumping the version used by Leptos.
You can view, comment on, or merge this pull request online at:
https://github.com/leptos-rs/leptos/pull/3304
Commit Summary
• 1e36883 https://github.com/leptos-rs/leptos/pull/3304/commits/1e36883f15ff16ea75aabfaf437b7a9da1da8bc8 Update Leptos to use latest wasm-bindgen version
File Changes
(9 files https://github.com/leptos-rs/leptos/pull/3304/files)
• M Cargo.lock https://github.com/leptos-rs/leptos/pull/3304/files#diff-13ee4b2252c9e516a0547f2891aa2105c3ca71c6d7a1e682c69be97998dfc87e (33)
• M any_spawner/Cargo.toml https://github.com/leptos-rs/leptos/pull/3304/files#diff-404970d4396703e7cd94b08fa19c6019477d548917aa64216cdcf60d3f62d417 (2)
• M leptos/Cargo.toml https://github.com/leptos-rs/leptos/pull/3304/files#diff-4010553df1571a02882686c4fe4c7b071878d911b94652f61acef8b2f8294eba (4)
• M leptos_dom/Cargo.toml https://github.com/leptos-rs/leptos/pull/3304/files#diff-96828be5bdc0692c8b65092116a29e55201094168eb1eea981f3d59acbf90cae (6)
• M meta/Cargo.toml https://github.com/leptos-rs/leptos/pull/3304/files#diff-4058a55b27a12bb422d295d30b24286c081e4a77eecd6ccbfc258436e9ddb559 (4)
• M reactive_graph/Cargo.toml https://github.com/leptos-rs/leptos/pull/3304/files#diff-aa453c736c7c19e682c93e1505fcffb9744f69111da69a6cf4ad656584edd18a (2)
• M router/Cargo.toml https://github.com/leptos-rs/leptos/pull/3304/files#diff-5b4ec6c8005e6b142ac974571bb8dc557a9bb7ab7c3d0b09554dfc9cc50dcf0b (6)
• M server_fn/Cargo.toml https://github.com/leptos-rs/leptos/pull/3304/files#diff-0e08e6740eee1441699c4026db1a9971033352034a4a33ff13a3f83ad7f84693 (8)
• M tachys/Cargo.toml https://github.com/leptos-rs/leptos/pull/3304/files#diff-9584b323f10190f2f470a09d6b7ecbd05db7e4491b482ee9c49b718b7a1dc504 (6)
Patch Links:
• https://github.com/leptos-rs/leptos/pull/3304.patch
• https://github.com/leptos-rs/leptos/pull/3304.diff
—
Reply to this email directly, view it on GitHub https://github.com/leptos-rs/leptos/pull/3304, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABVBTCLU5CVPNDMKLQ33TQT2DDHTHAVCNFSM6AAAAABSXXSLT6VHI2DSMVQWIX3LMV43ASLTON2WKOZSG4YDMMJTGYZTKOI.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Would it also be possible to backport the wasm-bindgen dependency update to 0.6?

These dependencies do not need to be updated; cargo will automatically resolve them to the highest patch version in your application. The only one that needs an update is cargo-leptos, and that update will apply to both 0.6 and 0.7 as cargo-leptos is agnostic about leptos version.

I can delete this if you don't think it's necessary.