Query API WebSocket upgrade

On the one hand (in Creating a WebSocket subscription):

Connection upgrade procedures permitting an existing HTTP query to transition to a websocket are not currently supported, but may be in the future.

On the other hand (one paragraph further on, in Query API Behaviour):

Query APIs may additionally support the HTTP 'Upgrade' header sent by clients to upgrade an HTTP GET request to a Websocket.

We discussed one fix, to remove the first sentence and clarify the second one:
Query APIs may support the HTTP protocol upgrade mechanism on the list resources like /nodes to transition an HTTP GET request to a WebSocket connection, but are not required to do so.
Could link to e.g. https://developer.mozilla.org/en-US/docs/Web/HTTP/Protocol_upgrade_mechanism or https://tools.ietf.org/html/rfc7230#section-6.7 and https://tools.ietf.org/html/rfc6455?
Presumably the created subscription will be non-persistent?

More guidance, with the link to the MDN documentation mentioned above, could be added to a future implementation guide in the style of INFO-002.


normal_ext base material gets wrong lighting

I did remark on this before that normal_ext materials were somewhat dark but only now it occurred to me that there is a mismatch with how the material gets lit. From the looks of it, I want to say it's not receiving all the lights, I think. In the screenshot there is a normal_ext.specular material to contrast it, as the specular extension does get lit as it should as far as diffuse is concerned.
However, I need to question the specular reflection itself here; it's rather limited to one face of the pillar as if it was being calculated per vertex. The stock specular interacts with the polygons much better, I want to say per pixel but my terminology might be misled here. But then, this could also be related to #55 because I am seeing a lot of polygon seams bleeding in this map and they sometimes bleed through a color so bright that it spikes the bloom. It's hard to tell what's going on at this point.

Just so I have a better grasp on what's going on, does this happen with normal_ext.specular.dynamic as the rendertype?

I'm not sure which one you mean? Also I'll take back that second point about the specular, it's the same as the stock specular so it's just what it is. I think the geometry itself would need to be redone better for the specular effect to look more natural.
But no, the color issue doesn't happen on any kind of specular extensions whatsoever. The specular mapped materials are correct, it's the regular normal_ext materials that have gone daffy. I did check if it might be because the normal maps between these materials use a different compression, but it doesn't seem to be the case.

Sorry, I was being hasty earlier and I think I misread what you were saying. (I thought the issue may have been to do with the normal mapping, but I realize now I was way off.)
Okay so the problem actually is (as opposed to whatever nonsense I was thinking it was) that normal_ext materials without specular lighting are darker than they should be yeah?
After checking out the responsible shader it looks like the diffuse texture is getting multiplied over the ambient and static lighting twice. Which in some cases will significantly decrease the amount brightness of the "reflected" light. It'll be an easy fix.
Do you think this could explain the problems you've been seeing?

Yeah that sounds like the root of it.


[#1499] - Deleted Quartz module.

Fixes #1499 Quartz module deletion
The module was present in shiro/support folder
Following this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GitHub issue filed
for the change (usually before you start working on it).  Trivial changes like typos do not
require a GitHub issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Format the pull request title like [#XXX] - Fixes bug in SessionManager,
where you replace #XXX with the appropriate GitHub issue. Best practice
is to use the GitHub issue title in the pull request title and in the first line of the commit message.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] add fixes #XXX if merging the PR should close a related issue.
[ ] Run mvn verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[x] If you have a group of commits related to the same change, please squash your commits into one and force push your branch using git rebase -i.
[x] Committers: Make sure a milestone is set on the PR

Trivial changes like typos do not require a GitHub issue (javadoc, comments...).
In this case, just format the pull request title like [DOC] - Add javadoc in SessionManager.
If this is your first contribution, you have to read the Contribution Guidelines
If your pull request is about ~20 lines of code you don't need to sign an Individual Contributor License Agreement
if you are unsure please ask on the developers list.
To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

[x] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004
[ ] In any other case, please file an Apache Individual Contributor License Agreement.

Note: We need to wait until we cut the 3.x branch before merging this (but we can get it ready to go!).
My pref would be to hold off creating the branch until we have a few PRs lined up that depend on each (as to avoid any additional back porting), but... avoiding backporting might just be a dream 😆
(I'm also not trying to block anyone from cutting the 3.x branch if they are motivated) 😄

Yea, I think not having 3.x branch is starting to block things...

@Himakar Can you fix the failing build please? I think it's just a matter of removing quartz from the coverage module

Also, please squash the commits into one.
Otherwise, LGTM, thank you!

@Himakar Can you please fix the remaining issues as described above please?
Thank you!

@Himakar Any updates please?
We have created 3.x branch and can merge this if you make the necessary changes.
Thank you!

@Himakar Last thing you have to do is to click the CLA checkbox(es) and squash the commits.
Do you know how to do that?

Hi @lprimak, Could you please let me know hint on CLA checkboxes and squash part

Just read the PR description. There are checkboxes there about license agreement. Read and click on the appropriate checkboxes.
See https://stackoverflow.com/questions/5189560/how-do-i-squash-my-last-n-commits-together

No problem! Somehow your squashed changes didn't "take" on this PR. Please don't create new PRs, and use this PR.
Please try squashing / force-push this PR again.

Also, I based this branch of the 3.x branch now, please make a note of that.

@Himakar Any updates? We are so close :)

Hi @lprimak ,
I can see the commit items in the pull request. Am i missing something else?

Your screenshot points to the closed PR. You need to squash commits on this PR. It still shows 7 commits.
Also, now there is a conflict that needs to be fixed

@Himakar I think you need to rebase your PR.

@Himakar you are rebasing from main rather than 3.x branch, this is why you added more recent commit from main.
You need to be sync to 3.x branch and not main.

Hi @fpapon, Thanks for your help. Now i have rebased my branch from 3.x branch and pushed code.
Is this fine now?

The 4 commits from dependabot are from main and not from 3.x.
You can remove them with git rebase -i HEAD~5 and drop them.

Hi @lprimak , @fpapon ,
Thank you for your help. It means a lot to me.
I am learning new concepts because of this task.
Thank you once again

Awesome!  We are happy to help. Thanks and keep it up!

@Himakar you're welcome and don't hesitate to make other proposal!


Model writing omits keywords in reaction rules

The keywords used as rule modifiers, such as DeleteMolecules and TotalRate are not being output to strings. The result is these keywords are deleted upon output by PyBNG, which creates a model with rules missing the keywords that is not equivalent to the model initially read by PyBNG. An example is provided in the attached bngl and python files. The bngl file has a rule with the DeleteMolecules keyword. When the model is read by PyBNG, the output of the rule does not contain the keyword.
test_deleteMolecules.bngl
begin model
begin parameters
  kdl 0.03 #CBL degrades ZAP/SYK
end parameters
begin molecule types
  A(State~UZAP~PZAP~SHP~uSYK~pSYK,CBL)
  Zeta(receptor,ITAM1~U~PP,ITAM2~U~PP,ITAM3~U~PP,ITAM4~U~PP,ITAM5~U~PP,ITAM6~U~PP)
  CBL(site)
  dead()
end molecule types
begin seed species
  A(State~UZAP,CBL) 1
  Zeta(receptor,ITAM1~U,ITAM2~U,ITAM3~U,ITAM4~U,ITAM5~U,ITAM6~U) 1
  CBL(site) 1
end seed species
begin reaction rules
  CBL_degrade_ITAM1:	Zeta(ITAM1!1).A(State!1,CBL!3).CBL(site!3) -> dead()	kdl DeleteMolecules
end reaction rules
end model
writeModel({suffix=>"changed",evaluate_expressions=>1,overwrite=>1})

run_pybng.py
mname="test_deleteMolecules"
model= bionetgen.bngmodel(mname+".bngl")
print(model.rules)

The output of running the above python code is
begin reaction rules
  CBL_degrade_ITAM1: Zeta(ITAM1!1).A(State!1,CBL!2).CBL(site!2) -> dead() kdl
end reaction rules

It turns out that because of differences between BNGL and its translation to XML (which is what PyBNG uses to create objects), the assignment of keywords to rules can be tricky. The omission of the DeleteMolecules keyword above occurs because of the specific logic of the code that parses the xml version of the model (starting at bionetgen/modelapi/xmlparsers.py line 701) requires that only one instance of the Delete operation occur in the rule. If there are multiple instances, as in the example above, the keyword is not assigned to the rule. This is clearly incorrect, because the DeleteMolecules attribute is set in BNGL at the rule level, so all Delete operations in the rule inherit this attribute. Therefore, if there are multiple Delete operations, as above, each of them will have the DeleteMolecules keyword. However, the test at line 705,
if not isinstance(del_op, list):

only allows the keyword to be set if there is only one Delete operation present.
This points to a clear locus in the code to fix, but we are still working at present to make sure we get the logic of the new code correct.
A concrete example is provided by the following rule:
 dZeta:	Zeta() -> dead()	kdl DeleteMolecules

For this rule  the rule operations include only one Delete, and the DeleteMolecules keyword is correctly associated with the rule.

Commit 49625c234428e00f8971698346a5ccd4e4c02b71 fixes the issue described above but I'm going to leave this issue open because I am not convinced the current code will handle rules that use multiple keywords.


fail to build on jupyterlab 3.1.7 for @5.0.0

jupyter-labextension install<EMAIL_ADDRESS>[LabBuildApp] JupyterLab 3.1.7

failing, showing this in log
[LabBuildApp] yarn run v1.21.1
$ webpack --config webpack.prod.config.js
[webpack-cli] ModuleNotFoundError: Module not found: Error: Can't resolve 'ipysheet/style/index.js' in '/home/hafnium/miniconda3/share/jupyter/lab/staging/build'
    at /home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules/webpack/lib/Compilation.js:1773:28
    at /home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules/webpack/lib/NormalModuleFactory.js:811:13
    at eval (eval at create (/home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:10:1)
    at /home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules/webpack/lib/NormalModuleFactory.js:286:22
    at eval (eval at create (/home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:9:1)
    at /home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules/webpack/lib/NormalModuleFactory.js:442:22
    at /home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules/webpack/lib/NormalModuleFactory.js:124:11
    at /home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules/webpack/lib/NormalModuleFactory.js:673:25
    at /home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules/webpack/lib/NormalModuleFactory.js:882:8
    at /home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules/webpack/lib/NormalModuleFactory.js:1002:5
resolve 'ipysheet/style/index.js' in '/home/hafnium/miniconda3/share/jupyter/lab/staging/build'
  Parsed request is a module
  using description file: /home/hafnium/miniconda3/share/jupyter/lab/staging/build/package.json (relative path: .)
    Field 'browser' doesn't contain a valid alias configuration
    resolve as module
      /home/hafnium/miniconda3/share/jupyter/lab/staging/build/node_modules doesn't exist or is not a directory
      looking for modules in /home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules
        existing directory /home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules/ipysheet
          using description file: /home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules/ipysheet/package.json (relative path: .)
            using description file: /home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules/ipysheet/package.json (relative path: ./style/index.js)
              no extension
                Field 'browser' doesn't contain a valid alias configuration
                /home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules/ipysheet/style/index.js doesn't exist
              .js
                Field 'browser' doesn't contain a valid alias configuration
                /home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules/ipysheet/style/index.js.js doesn't exist
              .json
                Field 'browser' doesn't contain a valid alias configuration
                /home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules/ipysheet/style/index.js.json doesn't exist
              .wasm
                Field 'browser' doesn't contain a valid alias configuration
                /home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules/ipysheet/style/index.js.wasm doesn't exist
              as directory
                /home/hafnium/miniconda3/share/jupyter/lab/staging/node_modules/ipysheet/style/index.js doesn't exist
      /home/hafnium/miniconda3/share/jupyter/lab/node_modules doesn't exist or is not a directory
      /home/hafnium/miniconda3/share/jupyter/node_modules doesn't exist or is not a directory
      /home/hafnium/miniconda3/share/node_modules doesn't exist or is not a directory
      /home/hafnium/miniconda3/node_modules doesn't exist or is not a directory
      /home/hafnium/node_modules doesn't exist or is not a directory
      /home/node_modules doesn't exist or is not a directory
      /node_modules doesn't exist or is not a directory
error Command failed with exit code 2.

You don't need to run this command anymore.
Installing ipysheet with pip install ipysheet or conda install -c conda-forge ipysheet is enough with JupyterLab 3.

why do I then see this:

(base) hafnium@ff4327d62088:~$ conda list | grep -e jupyterlab -e ipysheet
ipysheet                  0.5.0              pyhd8ed1ab_0    conda-forge
jupyterlab                3.1.7              pyhd8ed1ab_0    conda-forge
jupyterlab-widget-ketcher 0.1.4                    pypi_0    pypi
jupyterlab_pygments       0.1.2              pyh9f0ad1d_0    conda-forge
jupyterlab_server         2.7.1              pyhd8ed1ab_0    conda-forge
jupyterlab_widgets        1.0.0              pyhd8ed1ab_1    conda-forge

I would recommend starting again from a clean conda environment


Review JOSS paper

Hey, please find my review of your tool below:

Regarding the manuscript I don't have major comments, other than that there are many typos/grammatical errors, especially in the summary.

I found the installation of the package a bit tedious:

there is no description how to install on Mac/Windows. The latter can I guess be excused, but if the package is not easily installable/available on Mac this excludes a significant fraction of the research community,
to install I needed to use Docker and reproduce what you did here and additionally google how to run ./vcpkg/vcpkg install sciplot fmt on an ARM. I think it would be easiest to provide a Dockerfile that does the exact installation with all dependencies in the example folder (and have a GitHub action that tests on Mac M1).

I think providing install instructions using conda would be helpful and also make it straight-forward to use for Mac users. I think all required dependencies are on there.

I haven't done much C++ development in the last year but seeing how much traction Meson is gaining, I find it unusual that it isn't provided as alternative build system. Would that be easy to offer? Also, is the CMake 3.25 dependency really necessary?

I think providing a self-contained example with exact instructions how the visualisations are created is helpful. I have probably overlooked something, but it's not 100% clear how you made them from a quick look.

Create a CONTRIBUTING file with how-to's on contributing new code.

Cheers,
Simon

Quick update: I'm working on a vcpkg package. This might not be as quick as I hope but eventually it might make the install process easier, especially and at least hopefully on MacOS.

Here's the PR for the ctbench package as a follow-up. The LLVM dependency unsurprisingly makes it very long to iterate on as it needs to be built by vcpkg... I was hoping vcpkg would allow defining optional dependencies to let users pull them as binaries from other sources, but apparently it does not, unless I missed something.
https://github.com/microsoft/vcpkg/pull/31716

Great, let me know when you are done with the review. I think only the CONTRIBUTING doc is a requirement. The rest is up to you I guess, but would certainly be nice to have.

@JPenuchot what is the status about that ticket?


Test issue opened by a member

As a lowly member I cannot assign this issue to a project

but I can leave a comment


Inferencing?

I'm curious about using this for inferencing on an existing llama 4bit model. Replacing the lora file, and replacing the others and recompiling I have done already. However when attempting to load the lora. I get (short snippet of the full error)
  File "C:\Users\Arargd\miniconda3\envs\textgen\lib\site-packages\peft\tuners\lora.py", line 191, in _find_and_replace
    self._replace_module(parent, target_name, new_module, target)
UnboundLocalError: local variable 'new_module' referenced before assignment

Which is the same error you get if you try to load onto a 4bit model normally.
Am I missing something, or what do I need to do to get inferencing working on an existing 4bit model?

Which version of files are you using? I've done several updates on them.
And are those 2 parts replaced?

You can also try load the model using load_llama_model_4bit_low_ram.
And also I'll upload an inference example / training example later.

Using your examples I was able to get it to run!


Any way to use css?

Is there any option for using CSS styles? Can't find anything.

any update ?


Update gradle.properties

addressing Request for upgrading netty version in gradle.properties #1215

Might as well upgrade to the latest, which is 4.1.52.Final


[Bug] Prefer ternary operator - Anti-pattern Deepsource

Description
https://deepsource.io/gh/appsmithorg/appsmith/issue/JS-D009/occurrences
Important Details

Version: [Cloud / Self-Hosted vx.x]
OS: [e.g.MacOSX]
Browser [e.g. chrome, safari]
Environment [production, release, deploy preview]

Hello @Rishabh-Rathod , seems like a quick fix. Is it possible to assign this issue to me? Thanks!!


[syn] Add test synthesis scripts for DC

This adds a testsynth.tcl and a sweep.tcl script for testsyntheses. Both are mainly meant as templates for designers to set up and run a simple wire-load model synthesis.
The at-plot.py script is a helper script that parses results generated with sweep.tcl and plots area versus achieved timing. Up to ten result series can be combined in one AT plot, allowing to do simple design space explorations and/or compare designs.
Note that this PR depends on two other PRs with minor scripting fixes, hence only the last commit in this PR is relevant.

@eunchan @tjaychen @sjgitty let me know if I should rather place this in our foundry repo.

@eunchan @tjaychen @sjgitty let me know if I should rather place this in our foundry repo.

i feel like this is a good to have in the open source...that's just my opinion though.

Ok, rebased. Thanks for the feedback, I'll put this up for review.

Thanks for your comments @cdgori, that is certainly useful. The synthesis flow could definitely benefit from some refactoring at some point in order to modularize and encapsulate commonly used sub-steps. I'll probably do another pass when I'm going to update our tech libraries.


Unable to create issue

Run dacbd/create-issue-action@main
with:
owner: patilvishal1683
repo: temp
token: ***
title: CPU Utilization
body: This is a test issue created using the GitHub API and Python requests library.
Error: HttpError: Bad credentials
Error: Request to create new issue failed

@patilvishal1683 I'm going to close this issue, if you need any more help feel free to reach out.


JS Static Analysis Considerations for Tomorrow

Choosing Static Analysis Tools for the Cloverfield Boilerplate Scaffold
Cloverfield aims to create a next generation JavaScript project boilerplate. That means we'll use the tools that coders in-the-know will be using over the next 1 - 3 years. We're starting with the 2016 edition. Read more.
Which static analysis tools will we be using in 2016? What will the requirements even look like?

Are you a Facebook Flow expert?
Are you on the TypeScript team?
Do you live and breathe Google Closure Compiler?
Does Tern get your wheels turning?

Look into the Future

Where are these tools heading over the course of the next 3 years?
Is there a baseline we can use today that will be easily adaptable for projects we create in 2 years?
What features should we avoid in order to avoid breaking changes in the future?
Which features are must-haves, even if we need to run a build step to transform our code for forward compatibility (if we go this route, we should also create the migration path).

Think about broad adoption so that as many users as possible can benefit from this work.
Am I asking the right questions?  Let's kick off the discussion!

I think we should use Facebook Flow so we can use plain JavaScript, and maybe TypeScript as a second option for those who likes the language.

+1 for Flow. It has some issues with ES6 support though. There might be the way to transform first before flow does it's checks.
https://babeljs.io/docs/advanced/transformers/other/flow/

Considering Sebastian is now working for Facebook, we should probably see some awesome Flow support soon.


Too many requests when two vector tile source have minZoom and maxZoom

Describe the bug
When adding two or more vector tile layer source with minZoom and maxZoom, there is a lot of requests while zooming out too much, freezing web browser
Same happpens when applying tileGrid created via createXYZ
Doing so causes to hit 429 - too many requests error
If I don't add it, I get a lot of map tiler errors that requested tile is out of bounds when zooming in/out to much
To Reproduce
Steps to reproduce the behavior:

Create two vector tile layers
Add them to the map
Zoom out
You'll notice web browser freezes

Repo: https://github.com/Herob527/open-layers-tiles-reproduction
Expected behavior
I expected my browser not to freeze zooming out
If there is something else I can do, please let me know
@Edit
I forgot to mention
This issue happens only when there are more than one vector tile layer with minZoom/maxZoom in source

This is natural behavior when you set a minZoom on the source, and zoom out far beyond that minZoom. If tiles are available for lower zooms, set a lower zoom. If not, set an appropriate minZoom on the layer as well, to avoid the layer being shown at zoom levels where too many tiles need to be loaded.

Thing is, layer still needs  to be visible even below their minimal zoom available on mapserver

If you get out of bounds errors, then tiles are not available for your desired zoom levels. So you either have to create your own tile set with more lower zoom levels, or live with the fact that you need to set a minZoom on the layer as well.
Note that the problem only arises for minZoom. For maxZoom, overzooming is not a problem.

I forgot to mention
It happens when I have two or more vector tile layers
It doesn't happen when I have only one vector tile layer
I'll edit the issue

It does not make a difference in terms of the required solution, which I pointed out above. It's not an OpenLayers issue, but a matter of appropriate configuration.

Then why I'm getting huge amount of requests when having more than 1 vector tile layers? (leading to 429 and browser freeze)
And why I'm not getting the same thing with only 1 layer there?
I'm quite confused there

Your countries layer has a minZoom of 1, so there won‘t be much undersampling for that. Only when you add the contours wirh a minZoom of 9, this becomes a problem.

To explain this better: when you’re looking at a map at zoom 9, with a source that provides tiles from zoom 9 and higher, only a few tiles will be loaded. With every zomm level you zoom in, the required number of tiles is 4 times as much as the previous zoom.
So let‘s assume 6 tiles are needed for zoom 9, it‘s 24 at zoom 8, 96 at zoom 7, 384 at zoom 6 and so on.
If another source that provides tiles for zoom 1 and higher, the same map view at zoom 6 will only need 6 tiles, because the source provides zoom 6.

And in zoom 4 I have thousands
So every layer have different characteristics speaking of downloaded tiles, right?
One layer may require less tiles (let's say 100 requests) and other more (thousands) for lower zooms

Thing is
When I set minZoom and maxZoom to be the same, I still experience freezing

So it araises question
Why I don't experience freezing when not using minZoom and maxZoom, letting the 403 errors be there?

Because you're trying to request tiles that are not available on the server. And I suspect because of the CORS configuration of the server, you get 403 instead of 404 in these cases. This is a question to ask the tile provider though.


[Issue 11096][Tests] Attempt to fix flaky test ResourceGroupConfigListenerTest

Fixes #11096
Motivation
ResourceGroupConfigListenerTest is very flaky. See #11096 .
Modifications
Attempt to reduce flakiness by:

using unique resource group name for each test method
using unique tenant name for testResourceGroupAttachToNamespace test

@bharanic-dev please review this PR

@lhotari thank you for making the changes. The changes look good to me in general and I think they are in the right direction. But I am not sure it explains the failure in the test case. The real issue is that with the metadata-store implementation, there is a race condition that can result in ZK the watch events getting missed. I filed the following issue for it, also explaining the cases where the race condition can happen.
https://github.com/apache/pulsar/pull/11157
I discussed this with @merlimat. @merlimat  has a PR to address the issue, but it is not yet merged.
https://github.com/apache/pulsar/pull/11198
Your changes will likely not fix the problem, but change the timing such that it is difficult to get the race condition to happen. I suggest we wait for the above PR to be merged before we merge the changes you did to the test case.

@bharanic-dev Thanks for the providing the details of the work that has been going on behind the scenes to fix the root cause.

the changes don't remove the flakiness. The problem occured in the most recent build:
Error:  Tests run: 24, Failures: 6, Errors: 0, Skipped: 18, Time elapsed: 32.989 s <<< FAILURE! - in org.apache.pulsar.broker.resourcegroup.ResourceGroupConfigListenerTest
Error:  testResourceGroupAttachToNamespace(org.apache.pulsar.broker.resourcegroup.ResourceGroupConfigListenerTest)  Time elapsed: 10.123 s  <<< FAILURE!
org.awaitility.core.ConditionTimeoutException: Assertion condition defined as a lambda expression in org.apache.pulsar.broker.resourcegroup.ResourceGroupConfigListenerTest expected object to not be null within 10 seconds.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:165)
	at org.awaitility.core.AssertionCondition.await(AssertionCondition.java:119)
	at org.awaitility.core.AssertionCondition.await(AssertionCondition.java:31)
	at org.awaitility.core.ConditionFactory.until(ConditionFactory.java:895)
	at org.awaitility.core.ConditionFactory.untilAsserted(ConditionFactory.java:679)
	at org.apache.pulsar.broker.resourcegroup.ResourceGroupConfigListenerTest.createResourceGroup(ResourceGroupConfigListenerTest.java:80)
	at org.apache.pulsar.broker.resourcegroup.ResourceGroupConfigListenerTest.testResourceGroupAttachToNamespace(ResourceGroupConfigListenerTest.java:135)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:566)
	at org.testng.internal.MethodInvocationHelper.invokeMethod(MethodInvocationHelper.java:132)
	at org.testng.internal.InvokeMethodRunnable.runOne(InvokeMethodRunnable.java:45)
	at org.testng.internal.InvokeMethodRunnable.call(InvokeMethodRunnable.java:73)
	at org.testng.internal.InvokeMethodRunnable.call(InvokeMethodRunnable.java:11)
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at java.base/java.lang.Thread.run(Thread.java:829)
Caused by: java.lang.AssertionError: expected object to not be null
	at org.testng.Assert.fail(Assert.java:99)
	at org.testng.Assert.assertNotNull(Assert.java:942)
	at org.testng.Assert.assertNotNull(Assert.java:926)
	at org.apache.pulsar.broker.resourcegroup.ResourceGroupConfigListenerTest.lambda$createResourceGroup$0(ResourceGroupConfigListenerTest.java:83)
	at org.awaitility.core.AssertionCondition.lambda$new$0(AssertionCondition.java:53)
	at org.awaitility.core.ConditionAwaiter$ConditionPoller.call(ConditionAwaiter.java:222)
	at org.awaitility.core.ConditionAwaiter$ConditionPoller.call(ConditionAwaiter.java:209)
	... 4 more

@lhotari @bharanic-dev #11198 should be ready to go

closing this since #11198 should fix the flakiness issue


Initial memory consumption

Dear @Booritas,
Thank you very much for this helpful repository.
I have a question why the initial memory consumption for SVS images ranges from 1MB to over 100MB per image?
Environment:
Python 3.8.16
pip 22.3.1
slideio 2.0.2
Ubuntu 20.04

Code to reproduce the behaviour.
pip install -U memory_profiler
import slideio
from memory_profiler import profile

@profile(precision=4)
def image_a_svs(path):
    image = slideio.open_slide(str(path), driver="SVS")
    return image

@profile(precision=4)
def image_b_svs(path):
    image = slideio.open_slide(str(path), driver="SVS")
    return image

a = image_a_svs("***_a.svs")
b = image_b_svs("***_b.svs")

Results:
Line #    Mem usage    Increment  Occurrences   Line Contents
=============================================================
    69 156.2344 MiB 156.2344 MiB           1   @profile(precision=4)
    70                                         def image_a_svs(path):
    71 255.5977 MiB  ***99.3633 MiB***           1       image = slideio.open_slide(str(path), driver="SVS")
    72 255.5977 MiB   0.0000 MiB           1       return image

Line #    Mem usage    Increment  Occurrences   Line Contents
=============================================================
    74 255.5977 MiB 255.5977 MiB           1   @profile(precision=4)
    75                                         def image_b_svs(path):
    76 258.0898 MiB   ***2.4922 MiB***           1       image = slideio.open_slide(str(path), driver="SVS")
    77 258.0898 MiB   0.0000 MiB           1       return image

Both images have roughly the same size and contain three levels, but image A has a memory footprint of 99.3633 MiB while image B of 2.4922 MiB.
Do you know what causes this and how to avoid loading 100MB into memory?
While 100MB for one image is fine, loading hundreds of them for training deep learning models quickly adds up. The SCN shows a similar behaviour, but the GDAL for tiff images does not.
I really appreciate any help you can provide.
With kind regards,
Christian

Hi Christian,
Thank you for your message. I will investigate the behavior and let you know about the results.
I have a question about your workflow. Do you open 100 different images at the same time? The library should release the memory as soon as an object (slide, scene) is released. What is a normal scenario in your workflow? Do you have 1 slide and multiple scenes, or multiple slides and multiple scenes at the same time? It will help me to understand what problem you encounter.
I will check the initial memory consumption anyway and let you know.
Best regards,
Stanislav

Dear Stanislav,
Thank you very much for your quick reply.
Our workflow is that we keep the WSI objects in a dict.
An oversimplified example would look like the following:
slide_cache = {}

for idx, file in tqdm(enumerate(files)):
   slide_cache[file] = slideio.open_slide(str(file), driver="SCN")

So during training, we can quickly access tiles like:
random_slide = slide_cache.keys()[random_number]
patch = slide_cache[random_slide ].read_region(x,y)

I hope this helps if you have more questions or need some example files.
Please let me know.
With kind regards,
Christian

Dear Christian,
thanks for your clarification.
I did some profiling. The amount of the allocated memory depends on the image. According to the profiling, most of the memory is allocated for tiff structures of the images. Currently, the library opens the file multiple times, it includes the main image and auxiliary images (thumbnails, macro, labels, etc.).  It is done to avoid concurrent access to the same file structures from different scenes.
I'm optimizing the behavior. Postponing of opening of the auxiliary images until they are requested, already reduced memory consumption. I want to do some more investigation and try to reduce the memory footprint.
I expect that I can publish a new version with reduced memory consumption at end of this week or beginning of the next week. Best Regards,
Stanislav

Dear Stanislav,
Thank you very much for your reply and for looking into it. I will check it out tomorrow.
With kind regards,
Christian

Hi Christian,
I just published a new version '2.0.4'. It contains the fix described in the previous message. As I wrote, the memory consumption depends on image. Most of the memory is allocated by libtiff library. SlideIO add just a few kB.
I hope the fix will help.
Please let me know if you have any problem with the library.
If you like the library, please consider giving a star to the repository.
Best regards and thanks for your help!
Stanislav

Dear Stanislav,
Thank you very much for your excellent work. Interesting that the libtiff library has such different results in memory consumption for images that a roughly the same size.
The repository star for your lib is more than deserved.
With kind regards,
Christian


After changing the system language to certain languages, the software will not work properly

After Connecting, the app should jump to the contacts view, but it doesn't do that, it just keeps loading, like this

As shown in the figure, these languages can trigger this problem

Thanks for reporting this issue!
Unfortunately, I don’t have the device to test this issue directly at the moment, but I’ll definitely look into fixing it after the next release. I appreciate your patience!


Validation comparison

Incorporates the code for the validation results comparison in the reader.
Needs:

[ ] Reader update PR
[ ] QA4SM filename convention update

Pull Request Test Coverage Report for Build 740156893

752 of 865 (86.94%)  changed or added relevant lines in 7 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+1.2%) to 83.179%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/qa4sm_reader/handlers.py
112
120
93.33%

src/qa4sm_reader/plot_all.py
0
11
0.0%

src/qa4sm_reader/img.py
150
162
92.59%

src/qa4sm_reader/plot_utils.py
76
102
74.51%

src/qa4sm_reader/plotter.py
179
205
87.32%

src/qa4sm_reader/comparing.py
231
261
88.51%

Files with Coverage Reduction
New Missed Lines
%

src/qa4sm_reader/plot_all.py
2
0%

Totals

Change from base Build 690145437:
1.2%

Covered Lines:
1011

Relevant Lines:
1174

💛  - Coveralls


Index style and script added   angelo

I want to merge this branch so that I can create a new one off of main.  There were no changes made to the files but I need to get the updated things

print('please')


Unable to notify two elements about same splice: Uncaught Error: unexpected key undefined

On my page I have two <template is="dom-bind"> which bound to same model with an array.
When an array item being removed by third party library I notify the templates with _notifySplice method. Everything works fine under Polymer 1.1.x, but Polymer 1.2.x fails to notify second template with the following exception:
Uncaught Error: unexpected key undefined

Here is a fiddle to demonstrate the exception: http://fiddle.jshell.net/417e3ybz/.
Before you ask: I do need two or more dom-bind templates on same page, and yes, they have to be bound to same model.

notifySplices is now a public API, so you should probably avoid using the private version.

I have updated the fiddle with usage of public notifySplices method: http://fiddle.jshell.net/417e3ybz/1/, but this does not eliminate the exception.

I would use an elements for this, but I managed to get it working with nested dom-bind templates. This probably voids your warranties...But it appears to work.
Basically: create two (or more) separate variables that point to the same model object. Bind to those, then use linkPaths to let Polymer know that there's another path that needs to be notified, but it's really the same object so Polymer won't attempt to mutate the underlying collection that it uses to keep track of the array.
http://fiddle.jshell.net/417e3ybz/
Note that you can remove the inner dom-bind templates without affecting the logic:
http://fiddle.jshell.net/r9ansc9r/
You can also do the same thing inside an element.
I defer to @kevinpschaaf on whether this is a good idea.

Thanks @arthurevans for examples and explanations.
Unfortunately, we cannot use them, as in our case model is a JSON, so we cannot add such references.

Also, I have to say I'm curious about the use case that absolutely requires multiple dom-binds on the same page with the same data.

In our case we build a page out of partials (provided by different parties) based on JSON view-model.
So briefly, build dynamic UI entirely from server-driven JSON view-models.

Load master HTML with root dom-bind, for example:

<template is="dom-bind" id="root">

{{title}}

        <template is="imported-template" content$="{{content.html}}" model="{{content.model}}"></template>

    ...
</template>

Fetch JSON view-model (and keep it with sync with PuppetJS - something like firebase-element, but uses JSON Patch standard, OT & other magic, to keep everything in sync), for example:

var viewmodel = {
       title: "Hello Arthur",
       content: {
           html: "/path/to/partial.html",
           model: {
                greeting: "Welcome to JSON driven SPA",
                // ...
           }
       },
       // ...
}

Use data and {{}} bindings to fill the template is="dom-bind",
Where needed, load partial's markup <imported-template> (vanilla) custom element imports partial template (using HTML Imports to cover script's execution, caching, etc. natively), for example:

<template is="dom-bind" id="nested">

{{model.greeting}}

  ...
</template>

then it attaches data-model to it, and stamps it to the main document.
5. If there is any other (nested) partial to imports it as well.
That is how we ends up with HTML

Hello Arthur

    <template is="imported-template" content="/path/to/partial.html">
</template>

Welcome to JSON driven SPA

    ...
    <template is="dom-bind" id="nested">

{{model.greeting}}

        ...
    </template>

...
<template is="dom-bind" id="root">

{{title}}

        <template is="imported-template" content$="{{content.html}}" model="{{content.model}}"></template>

    ...
</template>

where root dom-bind is bound to viewmodel, and nested dom-bind to viewmodel.content.model


Standardise API config commands

This issue has been moved here from https://github.com/tredly/tredly-host/issues/43. Please see the original thread for more details but continue discussion here.

The following commands need to be changed from:
tredly-host config firewall addAPIwhitelist
tredly-host config firewall clearAPIwhitelist
To:
tredly config api whitelist
tredly config api whitelist clear

Complete in v1.0


TypeError: Cannot read property 'WebSocketAdapter' of undefined

I cant get the web package to run, running into this error:
(anonymous function)
src/pages/Room/index.js:40
  37 |   : server
  38 | console.log(wsurl)
  39 | // TODO: move this to a more global configuration
> 40 | const adapter = new turtus.Adapters.WebSocketAdapter(wsurl)
     | ^  41 | const manager = new turtus.P2PManager(opts)
  42 | manager.setAdapter(adapter)
  43 |

▶ 19 stack frames were collapsed.
Module../src/index.js
src/index.js:7
   4 | import App from './App';
   5 | import * as serviceWorker from './serviceWorker';
   6 |
>  7 | ReactDOM.render(<App />, document.getElementById('root'));
   8 |
   9 | // If you want your app to work offline and load faster, you can change
  10 | // unregister() to register() below. Note this comes with some pitfalls.

webpack_require
turtus/services/web/webpack/bootstrap:785
  782 | };
  783 |
  784 | // Execute the module function
> 785 | modules[moduleId].call(module.exports, module, module.exports, hotCreateRequire(moduleId));
      | ^  786 |
  787 | // Flag the module as loaded
  788 | module.l = true;

fn turtus/services/web/webpack/bootstrap:150
  147 | 		);
  148 | 		hotCurrentParents = [];
  149 | 	}
> 150 | 	return __webpack_require__(request);
      | ^  151 | };
  152 | var ObjectFactory = function ObjectFactory(name) {
  153 | 	return {

turtus/services/web/webpack/bootstrap:785
  782 | };
  783 |
  784 | // Execute the module function
> 785 | modules[moduleId].call(module.exports, module, module.exports, hotCreateRequire(moduleId));
      | ^  786 |
  787 | // Flag the module as loaded
  788 | module.l = true;

checkDeferredModules
turtus/services/web/webpack/bootstrap:45
  42 | 	}
  43 | 	if(fulfilled) {
  44 | 		deferredModules.splice(i--, 1);
> 45 | 		result = __webpack_require__(__webpack_require__.s = deferredModule[0]);
     | ^  46 | 	}
  47 | }
  48 |

Array.webpackJsonpCallback [as push]
turtus/services/web/webpack/bootstrap:32
  29 | 	deferredModules.push.apply(deferredModules, executeModules || []);
  30 |
  31 | 	// run deferred modules when all chunks ready
> 32 | 	return checkDeferredModules();
     | ^  33 | };
  34 | function checkDeferredModules() {
  35 | 	var result;

(anonymous function)

First I'd like to apologize for not getting back to you for a two whole months...
I pretty much abandoned this project when I got a new job and I didn't notice or expect that this repository would get any attention at all.
The problem here was just that I had a version of the turtus-lib package locally linked to the project that was out of date on github...so that's my bad. I probably should have emphasized how unfinished and experimental all of this was in my readme. Sorry if that caused you any frustration.
I actually only noticed this issue coincidentally when I checked reddit to see if anybody else had made any progress on a rabb.it clone, saw that someone mentioned your Neko repository, and then saw that you had referenced this one hahaha. It looks like you're miles ahead of me now

It looks like you're miles ahead of me now

@Khauri It's the circle of life... Welcome to how I felt when I found your project! :joy:
I also got a new job that got in the way of side project development

It's all good, your project gave me enough of a push to get started on neko :P thanks for publishing what you had


0.2.1 - Critical bug fix, improved promises use, dependency and README updates

Special thanks to @neverfox for finding and reporting #38 & #39.

#38, #39 - I/O getting blocked when publishing at high frequencies (think for/while loops).

Removed one-time failed event handler from publish call
Cache reject callbacks from publish
On publish confirmation, remove reject from deferred array
On exchange connection failure, invoke all rejects in deferred array

#37 - document use of close and closeAll calls
Correct improper use of .then( null, ... ) which was creating additional promises.
Update whistlepunk version
Include biggulp to simplify the gulpfile (yay?)

Pulled and ran tests locally, looks good.


ARTEMIS-2293 addPacket in LargeMessageControllerImpl won't notifyAll …

…for exception

Thanks for the fast fix 👍🏻


set_to_zero circular reference patch

This should make Turing.jl CI pass.

@penelopeysm @mhauru I've checked locally -- this seems to fix the HMC section of the Turing.jl tests. I'll get it merged + tagged asap so that we can get the Turing.jl tests running again.

I'm reasonably confident that coverage is lying about the lack of coverage of IdDict code. Therefore I'm going to merge + tag.


Remove "verification" terminology from nip-05 in apps

Benefit: reduce confusion for nostr newbies
via @dtonon

We should stop pairing NIP-5 with the word "verification". For newcomers it risks being a shitcoin game.
Use NIP-5 for its real functions and embrace WoT for actual contacts check.

https://njump.me/nevent1qqsrpvvhnu5fu8gaudnlnuk7cdp3m5yezl6w88x9t38c3zdhaju52tcpzemhxue69uhhyetvv9ujuurjd9kkzmpwdejhgqguwaehxw309ahx7um5wghxy6t5vdhkjmn9wgh8xmmrd9skcqghwaehxw309ahx7um5wghxvmt59emkj73wvf5h5qg5waehxw309ahx7um5wghrzumpwshx7un8ys9k4d

I think a bunch of us settled on "nostr address" instead of "nip05 verification" a long time ago.

@alltheseas

@dtonon where do you encounter 'verification' terminology?
Check the mentioned thread. It's quite common that users talk about "verification".
I'm not aware of any client that use it, maybe some NIP-05 services.

@alltheseas

Renamed thread to onboarding confusion: name, username, nip-05, npub.

This is a really broad topic; I would only keep this topic for the incorrect terminology of “verification”, it can be useful as reference.
Btw, NIP-05 has been already updated with an additional note:
https://github.com/nostr-protocol/nips/commit/da34c57e99dbf6cadfb3b4176b7c4ff33ac5674c

This is a really broad topic; I would only keep this topic for the incorrect terminology of “verification”, it can be useful as reference.

Sure.
In reviewing new people's observations joining nostr from past week, onboarding remains rough.


Add maxResults=False to Jira searches to ensure all results are found - 160

[Auto generated]
Number: [#160]

firewatch(cHmUP)[PR 160]: Closing issue for PR: Add maxResults=False to Jira searches to ensure all results are found.
PR was closed.


Where can we download the corresponding model for v0.4?

I noticed that you updated the model, but did not find the new version in the code repository or release.

It's accessible for patreons right now, and will be public after Christmas, as stated in this tweet:https://twitter.com/devdef/status/1473668908372709386?s=20

That's right. It will be released here on December, 25.

https://github.com/Sxela/ArcaneGAN/releases/tag/v0.4
As promised :D
Merry Christmas!

Thanks Alex, but I don't find in the Readme the collab link with version 0.4

It's been in the repo's readme, but I've added it to release notes as well, thanks for noticing.

Thanks Alex.

https://github.com/Sxela/ArcaneGAN/releases/tag/v0.4 正像承诺的那样：D 圣诞快乐！

how to open jit file


making >> optional in output

This adds a new option named greaterSigns (defaults to true, so no change of behaviour).
Successfully tested with the command line parameter --no-greaterSigns
fixes #5

@HolgerJeromin Thank you for the PR but I think we should consider another solution to logging going forward. Adding more options to customize each detail of the output will get out of hand.
If you have any ideas feel free to submit a rfc otherwise I'll be revisiting logging in the future and submit one myself.
In the meantime, you can install and create your own grunt-legacy-log and replace grunt.log with it:
var Log = require('grunt-legacy-log').Log;
var log = new Log({
  grunt: grunt,
  // By default it will stream to process.stdout but you can
  // replace it with your own filtering stream instead
  outStream: process.stdout
});
grunt.log = log;

If you want to have a "big" solution you could have a look at the Formatters from tslint.

If i understood this correctly, i would have to copy nearly the whole file from the legacy-log. Or at least provide most of the functions and direct it to the original (excluding .warn() and .ok())


Support for round world borders

Plugin Version:
3.0.3
Request:
Add support for round world borders.
How/Why:
Servers might use a plugin in order to have a round world border, such as WorldBorder or ChunkyBorder, which is our case. However, BetterRTP currently only supports square world borders, rather it picks random coordinates in a square.
It'd be nice if there was an option where you could specify in which shape should be the random coordinates picked; maybe something like Shape with square and circle values?
Additional Context:
None.

I would like for this feature actually, sounds pretty reasonable and probably very easy to implement. I will give you an update on this soon

I would like for this feature actually, sounds pretty reasonable and probably very easy to implement. I will give you an update on this soon

Just added support for this on 3.0.4, please let me know if you find any issues!
https://www.spigotmc.org/resources/betterrtp.36081/updates

Just added support for this on 3.0.4, please let me know if you find any issues!
https://www.spigotmc.org/resources/betterrtp.36081/updates

Thank you for adding this, it will help us a lot!
After some testing, I found only one minor issue – the minimum radius isn't respected. I checked the code and I think you must have forgotten to add the min variable to the final radius here.
To be on the safe side, I generated 20,000 random locations using the code of the generateRound() method between radii of 1,000 and 10,000 blocks, see below.
Current formula
double r = max * Math.sqrt(new Random().nextDouble());

Fixed formula
double r = max * Math.sqrt(new Random().nextDouble()) + min;

Thank you for adding this, it will help us a lot!
After some testing, I found only one minor issue – the minimum radius isn't respected. I checked the code and I think you must have forgotten to add the min variable to the final radius here.
To be on the safe side, I generated 20,000 random locations using the code of the generateRound() method between radii of 1,000 and 10,000 blocks, see below.
Current formula
double r = max * Math.sqrt(new Random().nextDouble());

Fixed formula
double r = max * Math.sqrt(new Random().nextDouble()) + min;

I want to revisit this, I am very certain I counter acted this, but I want to make sure

I want to revisit this, I am very certain I counter acted this, but I want to make sure

nope, you are right, I have to offset the radius, thank you very much for letting me know, I will post a patch later today!

nope, you are right, I have to offset the radius, thank you very much for letting me know, I will post a patch later today!

Fixed in 3.0.4-1 ;)

Fixed in 3.0.4-1 ;)

Awesome, thank you!

Awesome, thank you!


Created a live version of the list

🛠 Fixes Issue
Created a live version of the resources, you can view the demo here: https://suhailkakar.github.io/free-Web3-resources/
closes #17
📄 Note to reviewers
You can use GitHub pages and choose the main branch, it should automatically deploy the website :)

📷  Demo
https://user-images.githubusercontent.com/56798748/162453760-ecb9888d-ccf7-40ce-a60c-63cbc512e6cd.mov
EDIT: I will be adding new plugins such search functionality and etc in the future :)

solves #17


[usecase-webapi] Fix a FingerPrint extension directory typo

[usecase-webapi] Fix a FingerPrint extension directory typo.
FingerPrint example is copied from demo-express, so keep the
extension directory same as manifest.json in demo-express.

@Honry

LGTM


Documentation

Could you please provide basic installation instructions?

There's no installation needed, just download the .zip, unpack and use the .exe

Thanks!


[Serving] Support "n" for parallel generation

This PR brings field n to generation config and thereby supports parallel generation. This parallel generation effectively leverages the "fork" functionality of paged KV cache.
This PR supports specifying the number of parallel generation n in stardard OpenAI ChatCompletion API. This is the last feature towards the OpenAI API feature completeness.

Waiting for #1867 to merge first.

Hi @tqchen, I addressed your comments and updated the code and docstrings with term "request state entry". Would you mind taking another look?


Failed to install

I tried to install this plugin but it fails under 1.14.3 x64, OpenSuSe
The log-file says:
Installing<EMAIL_ADDRESS>failed.Hide output…
/bin/bash: mc: Zeile 1: Syntax Fehler: Unerwartetes Dateiende.
/bin/bash: Fehler beim Importieren der Funktionsdefinition für `mc'.

<EMAIL_ADDRESS>install /tmp/apm-install-dir-11728-5200-15flp94/node_modules/term3/node_modules/ptyw.js
node-gyp rebuild

/tmp/apm-install-dir-11728-5200-15flp94
└── (empty)
npm WARN deprecated<EMAIL_ADDRESS>react-tools is deprecated. For more information, visit https://fb.me/react-tools-deprecated
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon as possible. Use 'npm ls graceful-fs' to find it in the tree.
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
sh: mc: Zeile 1: Syntax Fehler: Unerwartetes Dateiende.
sh: Fehler beim Importieren der Funktionsdefinition für mc'. sh: mc: Zeile 1: Syntax Fehler: Unerwartetes Dateiende. sh: Fehler beim Importieren der Funktionsdefinition für mc'.
/bin/bash: mc: Zeile 1: Syntax Fehler: Unerwartetes Dateiende.
/bin/bash: Fehler beim Importieren der Funktionsdefinition für mc'. /bin/sh: mc: Zeile 1: Syntax Fehler: Unerwartetes Dateiende. /bin/sh: Fehler beim Importieren der Funktionsdefinition für mc'.
gyp: Call to 'node -e "require('nan')"' returned exit status 0 while in binding.gyp. while trying to load binding.gyp
gyp ERR! configure error
gyp ERR! stack Error: gyp failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (/usr/share/atom/resources/app/apm/node_modules/node-gyp/lib/configure.js:305:16)
gyp ERR! stack     at emitTwo (events.js:87:13)
gyp ERR! stack     at ChildProcess.emit (events.js:172:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:200:12)
gyp ERR! System Linux 4.9.9-1-default
gyp ERR! command "/usr/share/atom/resources/app/apm/bin/node" "/usr/share/atom/resources/app/apm/node_modules/.bin/node-gyp" "rebuild"
gyp ERR! cwd /tmp/apm-install-dir-11728-5200-15flp94/node_modules/term3/node_modules/ptyw.js
gyp ERR! node -v v4.4.5
gyp ERR! node-gyp -v v3.4.0
gyp ERR! not ok
npm WARN enoent ENOENT: no such file or directory, open '/tmp/apm-install-dir-11728-5200-15flp94/package.json'
npm WARN apm-install-dir-11728-5200-15flp94 No description
npm WARN apm-install-dir-11728-5200-15flp94 No repository field.
npm WARN apm-install-dir-11728-5200-15flp94 No README data
npm WARN apm-install-dir-11728-5200-15flp94 No license field.
npm ERR! Linux 4.9.9-1-default
npm ERR! argv "/usr/share/atom/resources/app/apm/bin/node" "/usr/share/atom/resources/app/apm/node_modules/npm/bin/npm-cli.js" "--globalconfig" "/home/lw/.atom/.apm/.apmrc" "--userconfig" "/home/lw/.atom/.apmrc" "install" "/tmp/d-11728-5200-p1ede/package.tgz" "--runtime=electron" "--target=1.3.13" "--arch=x64" "--global-style"
npm ERR! node v4.4.5
npm ERR! npm  v3.10.5
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the ptyw.js package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs ptyw.js
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls ptyw.js
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /tmp/apm-install-dir-11728-5200-15flp94/npm-debug.log
npm ERR! code 1

Note: When I type in mc ==> Midnight Commander is launched - which makes no sense within Atom...

Install npm on your linux distro.

Installing npm (and node) fixes this issue for me.

I have a similar bug, but I've installed both npm and node.
atom --version
Atom    : 1.17.2
Electron: 1.3.15
Chrome  : 52.0.2743.82
Node    : 6.5.0
node --version
v6.10.3
npm --version
3.10.10
apm install term3
Installing term3 to /home/kyle/.atom/packages ✗

<EMAIL_ADDRESS>install /tmp/apm-install-dir-11755-9614-m7snyj.xii9zilik9/node_modules/term3/node_modules/ptyw.js
node-gyp rebuild

/tmp/apm-install-dir-11755-9614-m7snyj.xii9zilik9
└── (empty)
npm WARN deprecated<EMAIL_ADDRESS>react-tools is deprecated. For more information, visit https://fb.me/react-tools-deprecated
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon as possible. Use 'npm ls graceful-fs' to find it in the tree.
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
gyp ERR! configure error
gyp ERR! stack Error: Python executable "/usr/share/atom/resources/app/apm/bin/python-interceptor.sh" is v3.6.0, which is not supported by gyp.
gyp ERR! stack You can pass the --python switch to point to Python >= v2.5.0 & < 3.0.0.
gyp ERR! stack     at failPythonVersion (/usr/share/atom/resources/app/apm/node_modules/node-gyp/lib/configure.js:454:14)
gyp ERR! stack     at /usr/share/atom/resources/app/apm/node_modules/node-gyp/lib/configure.js:443:9
gyp ERR! stack     at ChildProcess.exithandler (child_process.js:197:7)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:191:7)
gyp ERR! stack     at maybeClose (internal/child_process.js:877:16)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:226:5)
gyp ERR! System Linux 4.4.0-78-generic
gyp ERR! command "/usr/share/atom/resources/app/apm/bin/node" "/usr/share/atom/resources/app/apm/node_modules/.bin/node-gyp" "rebuild"
gyp ERR! cwd /tmp/apm-install-dir-11755-9614-m7snyj.xii9zilik9/node_modules/term3/node_modules/ptyw.js
gyp ERR! node -v v6.9.5
gyp ERR! node-gyp -v v3.4.0
gyp ERR! not ok
npm WARN enoent ENOENT: no such file or directory, open '/tmp/apm-install-dir-11755-9614-m7snyj.xii9zilik9/package.json'
npm WARN apm-install-dir-11755-9614-m7snyj.xii9zilik9 No description
npm WARN apm-install-dir-11755-9614-m7snyj.xii9zilik9 No repository field.
npm WARN apm-install-dir-11755-9614-m7snyj.xii9zilik9 No README data
npm WARN apm-install-dir-11755-9614-m7snyj.xii9zilik9 No license field.
npm ERR! Linux 4.4.0-78-generic
npm ERR! argv "/usr/share/atom/resources/app/apm/bin/node" "/usr/share/atom/resources/app/apm/node_modules/npm/bin/npm-cli.js" "--globalconfig" "/home/kyle/.atom/.apm/.apmrc" "--userconfig" "/home/kyle/.atom/.apmrc" "install" "/tmp/d-11755-9614-5kuyq8.cvl81if6r/package.tgz" "--runtime=electron" "--target=1.3.15" "--arch=x64" "--global-style"
npm ERR! node v6.9.5
npm ERR! npm  v3.10.5
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the ptyw.js package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs ptyw.js
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls ptyw.js
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /tmp/apm-install-dir-11755-9614-m7snyj.xii9zilik9/npm-debug.log
npm ERR! code 1

This is due to a known issue in apm. This fix worked for me. Namely, I went into /usr/share/atom/resources/app/apm/bin and changed the requisite lines in python-interceptor.sh from python to python2. Then term3 successfully built for me.

I ran into similar problems with Ubuntu 18.04.3 LTS (Bionic Beaver)
Even I tried the fix, still failing
atom --version
Atom    : 1.42.0
Electron: 4.2.7
Chrome  : 69.0.3497.128
Node    : 10.11.0
apm --version
apm  2.4.3
npm  6.2.0
node 10.2.1 x64
atom 1.42.0
python 2.7.17
git 2.17.1


Create   _index.md

Signed-off-by<EMAIL_ADDRESS>

/assign @howieyuen

we have done some work duplicated...
https://github.com/kubernetes/website/pull/33372/files

And this PR is also a duplicate of #33407.
/close


Bad imports in .hooks.sccaff.js fail silently

In the process of building a template for the CDK for Terraform, I ran into these lines swallowing an error that happened while requiring my .hooks.sscaff.js file.
In my case I had a bad relative import that could not be resolved and resulted in an error that was just silently overlooked.
Happy to add a PR checking the existence of the hooks file instead and failing if errors happen when requiring the file.

"a comment"

Contributions are welcome!


bug: where's the 0.2.0 code

Hello.


Wrap at?

In the current design we use cats to generate some output (on standard out) that can be used as an argument to at to set the runtime. All other output goes to standard error. This makes handling out output a bit complicated and cats does not really look like a stand alone scheduler.
One option would be to rebuild the command line interface to cats such that it looks like a scheduler itself and under the hood calls at having done the calculation for the start time. This would probably use processing from the standard library. It could also open the door to letting us ship more than one command line programme.
Anyhow, this came up in #52 and it seems like it needs thinking about.

Love the idea of making other schedulers configurable. And yes - an option to generate json encoded information could be very useful for interoperability. I also imagine an API in python-land would be useful for some. Would we want a single command line programme to do everything or more simpler programmes?

Something I've been wondering, what other schedulers can directly use our current output format without any additional parsing? If we wanted to use Cats inside a cron job as it currently stands, I think I'd probably create a cron script to start at the earliest possible time to run the job and then have that invoke at to delay the start based on the Cats output.

So we don't loose the idea, @abhidg suggested:

A simple way to “schedule” could be by using the —begin flag in sbatch: https://slurm.schedmd.com/sbatch.html#OPT_begin
This of course doesn’t guarantee that the job will start then, as the maximum number of jobs might be running already. On a fairly empty queue, this might be a good approximation.

From a quick look at that link, I think we have all the information we need to enable this using the same approach as we use for at at the moment, but we probably need a different time output format. If I'm correct, we would need a new command line argument for this too.


[BUG] Moneyboost and Expboost not working. v4.16.2

Description of issue:
Apparently on version 4.16.2 of the plugin, the commands for moneyboost and expboost don't work properly. When you run the commands, they do show the message correctly, but they won't really change your earnings at all.
I just confirmed this bug on a localhost with the default config of the plugin.

Additional:
I also wanted to talk about a misbehavior of this moneyboost and expboost thing, but I'm not sure if it keeps happening on the newer versions of Jobs since it used to happened me on an older version that I was using. So I remember that whenever I set a multiplier rate with decimals, it would not work properly or it would be rounded, for example "1.75" as a multiplier rate. But again, I'm not sure if it has been fixed on newer versions and I can't prove it out since now the command doesn't even work well.

Server Type (Spigot/Paperspigot/etc):
Spigot
Server Version (using /ver):
1.8.8
Regards!

/jobs moneyboost jobName number the correct usage for boost.

@montlikadani how does this help the problem as they said they was using the command, I’m having this problem as well on my server, it shows it works and everything, puts it into the global boost but the final boost is 0

thanks for fixing this @montlikadani !


OpenVINO not getting installed via Dockerfile.

It seems that OpenVINO is not getting installed in the container via both the binaries.Dockerfile and Dockerfile provided.
For binaries.Dockerfile specifically, In the following line:
https://github.com/opencv/gst-video-analytics/blob/a1f5efb660d3916d63f2a26d23e246f8d8651634/docker/binaries.Dockerfile#L557
I replaced it with:
RUN wget http://registrationcenter-download.intel.com/akdlm/irc_nas/16612/l_openvino_toolkit_p_${OpenVINO_VERSION}.tgz

Which gave me successful execution, and thereafter a successful container run (plus, clear output for l_openvino_toolkit_p_2020.2.120/install.sh being ran). However, when exploring the file system (and going down to the tutorial for "Acquire models and videos" the following instructions:
source /opt/intel/openvino/bin/setupvars.sh
source ~/gva/gst-video-analytics/scripts/setup_env.sh

# Download models listed in models.lst into ~/intel/dl_streamer/models folder
cd ~/gva/gst-video-analytics/samples
./download_models.sh

Does not work because /opt/intel/openvino is not a path in the container.
root@4aac8f4d2944:~# ls /opt/intel/
dldt  mediasdk

This is the same when building with opensource via the Dockerfile provided:
root@bf6b3402dee1:~# ls /opt/intel/
dldt  mediasdk

Any ideas what this could be? I've been able to create a custom docker file that installs all the required dependencies (gstreamer, dl-streamer, and OpenVINO), but when running gstreamer with the dl-streamer plugins + OpenVino models I receive multiple warnings and the output video is the same as the input. My current assumption is that dl-streamer does not support latest versions of gstreamer+plugins... but I digress. Currently exploring the above route again (using the provided Dockerfiles in the repo), but if anything, I'll open a new issue discussing revamping the Dockerfile in this repo.

In the past the URL varied between the releases plus allowing the user to flexibly use other downloaded variants (e.g. alpha-releases, internal releases, pre-releases, etc.), therefor it wasn't downloaded while building the Docker container image.
In the meantime GVA is already part of the ("native") OpenVINO installation, and is now called "DL-streamer".

hey @synchronizing,

source /opt/intel/openvino/bin/setupvars.sh
source ~/gva/gst-video-analytics/scripts/setup_env.sh

You can skip that part for opensource build. They are only required in local installation or in binaries build. That part of instruction will be fixed soon.
However, by output of ls /opt/intel in both cases it looks like you are in the container based on opensource build from this docker/Dockerfile. Could you verify this?

I rechecked binaries.Dockerfile. Seems like you don't need to run these steps at all for both opensource or binaries build:

source /opt/intel/openvino/bin/setupvars.sh
source ~/gva/gst-video-analytics/scripts/setup_env.sh

Also, output of ls /opt/intel should look the same. Seems like we a bit changed our binaries.Dockerfile and I missed that 😃

In the meantime GVA is already part of the ("native") OpenVINO installation, and is now called "DL-streamer".
@brmarkus

Those were my thoughts at first. The first thing I did was try to use the Dockerfile provided by the OpenVINO account on DockerHub (here). The Dockerfile allows me to install DL Streamer via a .sh file provided, but it will not install gstreamer. The documentations on the OpenVINO website does not make it obvious what to do.

However, by output of ls /opt/intel in both cases it looks like you are in the container based on opensource build from this docker/Dockerfile. Could you verify this?
@umed

I tried with both the opensource, and the binaries installation, but I see you clarified that it shouldn't be needed to source the first variable. Let me give it a try, and I'll come back here some point today.

@synchronizing,
Based on my understanding, the script runs model downloader "$DOWNLOADER" of OpenVINO to download the model files from Open Model Zoo site. You don't have to port from host machine.
Does this answer your question?
Mark

@easychizy
Based on my understanding, the script runs model downloader "$DOWNLOADER" of OpenVINO to download the model files from Open Model Zoo site. You don't have to port from host machine.

Hey Mark. $DOWNLOADER is a reference to a non-existing path:
DOWNLOADER=$INTEL_OPENVINO_DIR/deployment_tools/open_model_zoo/tools/downloader/downloader.py

The $INTEL_OPENVINO_DIR is set by the start up script (if I remember correctly), and that is /opt/intel/openvino, which is missing from the container.

So I tried what I had mentioned earlier:

I'm assuming that we are to port over models from the host machine to the Docker container on the docker run stage.

I downloaded the sample models and built the package version  (as described here) with a /root/files/ folder containing the sample models + sample video:
root@c2e63997bb5a:~# ls files/
emotions-recognition-retail-0003.json  emotions-recognition-retail-0003.xml  face-detection-adas-0001.xml  video.mp4

On running the following command:
gst-launch-1.0 \
filesrc location=/root/files/video.mp4 ! \
decodebin ! \
gvadetect \
  model=/root/files/face-detection-adas-0001.xml ! \
gvaclassify \
   model=/root/files/emotions-recognition-retail-0003.xml \
   model-proc=/root/files/emotions-recognition-retail-0003.json ! \
gvawatermark ! \
filesink location=/root/output.mp4

I receive the following output from gstreamer:
Setting pipeline to PAUSED ...
Pipeline is PREROLLING ...
Redistribute latency...
ERROR: from element /GstPipeline:pipeline0/GstGvaDetect:gvadetect0: base_inference plugin intitialization failed
Additional debug info:
/root/gst-video-analytics/gst/inference_elements/base/inference_singleton.cpp(118): acquire_inference_instance (): /GstPipeline:pipeline0/GstGvaDetect:gvadetect0:

Failed to construct OpenVINOImageInference
 Failed to import pre-compiled model
  [NOT_IMPLEMENTED]

ERROR: pipeline doesn't want to preroll.
Setting pipeline to NULL ...
Freeing pipeline ...

The NOT_IMPLEMENTED makes me assume this is due to the original reason for opening this issue, which is my belief OpenVINO never gets successfully installed. Unsure if this is the case, but any guidance would be wonderful!

Not able to cover the Acquire models and videos after installing via docker method. Can someone guide !!!

Please undo all changes in scripts and Dockerfile first :-) (because using Ubuntu 16 or Ubuntu 18 with GNOME deskopt/window-manager on an Intel Core-i9 is working for me).
I'm using "l_openvino_toolkit_p_2020.2.120.tgz", downloaded separately and copied it to the HOST-machine.
As I'm behind a corporate network proxy I needed to set the proxy ENV variables (http_proxy, https_proxy, HTTP_PROXY, HTTPS_PROXY), plus setting DISPLAY and calling "xhost +local:root".
I just used any MP4 video-container with embedded h.264 elementary stream. (as my computer doesn't have an Intel GPU I don't have HW-accelerated video-decoding via MediaSDK).
The needed models are downloaded successfully on my HOST-machine - and then using Docker "volume-mounting" when calling "docker run":
-v /My/Path/gva/data/models/intel:/root/intel_models:ro \
-v /My/Path/gva/data/models/common:/root/common_models:ro \
-e MODELS_PATH=/root/intel_models:/root/common_models \

Please undo all changes in scripts and Dockerfile first :-) (because using Ubuntu 16 or Ubuntu 18 with GNOME deskopt/window-manager on an Intel Core-i9 is working for me).
I'm using "l_openvino_toolkit_p_2020.2.120.tgz", downloaded separately and copied it to the HOST-machine.
As I'm behind a corporate network proxy I needed to set the proxy ENV variables (http_proxy, https_proxy, HTTP_PROXY, HTTPS_PROXY), plus setting DISPLAY and calling "xhost +local:root".
I just used any MP4 video-container with embedded h.264 elementary stream. (as my computer doesn't have an Intel GPU I don't have HW-accelerated video-decoding via MediaSDK).
The needed models are downloaded successfully on my HOST-machine - and then using Docker "volume-mounting" when calling "docker run":
-v /My/Path/gva/data/models/intel:/root/intel_models:ro \
-v /My/Path/gva/data/models/common:/root/common_models:ro \
-e MODELS_PATH=/root/intel_models:/root/common_models \

and how to download models to running docker?
Download models listed in models.lst into ~/intel/dl_streamer/models folder
cd ~/gva/gst-video-analytics/samples
./download_models.sh
after this Im getting
"Downloading models to folder /root/intel_models:/root/common_models
python3: can't open file '/deployment_tools/open_model_zoo/tools/downloader/downloader.py': [Errno 2] No such file or directory"

I have installed OpenVINO in my HOST machine (i.e. outside of the Docker container).
By calling "source /opt/intel/openvino/bin/setupvars.sh" the OpenVINO env variables and paths are exposed, e.g. INTEL_OPENVINO_DIR.
When downloading the models on HOST (i.e. outside of the Docker container) then set the env variable e.g. "export MODEL_PATH=/My/Path/gva/data/models/", (and "make -p /My/Path/gva/data/models/") the downloaded models then will be in subfolders "intel" and "common".

Let me use this opportunity to ask you as well @umed -- I hope you don't mind.
I've been working on porting over the project to a new Dockerfile for easier deployment, and I'm running into some issues where the video file passes through my CLI gstream+DL Streamer pipeline successfully (though with some warnings with debug level 4), but the expected video output does not contain drawn boxes (facial reconition + emotion detection, as the example in the wiki here). I was wondering if DL Streamer requires specific version dependencies for gstreamer and its good, bad, and ugly gstreamer plugins. A clarification here would be wonderful.

@synchronizing, short answer is our inference elements does not draw boxes, they attaches meta data, but we have gvawatermark (should be placed after inference elements) that draws meta on video frames. If that does not solve your problem please create separate issue and we can continue discussion there.

@brmarkus
I have installed OpenVINO in my HOST machine (i.e. outside of the Docker container).

Would you know if this is a must for it to work? My purpose with the Docker container is to eventually deploy it as an API pipeline, so having OpenVINO installed in the host machine isn't much of an option.
Thank you for taking your time to chime in what worked for you. Honeslty appreciated.

@umed
short answer is our inference elements does not draw boxes, they attaches meta data, but we have gvawatermark (should be placed after inference elements) that draws meta on video frames. If that does not solve your problem please create separate issue and we can continue discussion there.

I'll open a new issue from here then. I'm indeed using gvawatermark (ref), but for whatever reason still getting runtime errors when trying to pipe videos through it. With @brmarkus insights, I think I know why now. Thank you for taking the time to answer, and my apologies for the thread growing so large.
In short, the key thing I believe I've gathered from the discussion above is that OpenVINO must be installed in the host machine, and DL Streamer may be installed via Docker. I'll be trying later today and tomorrow to setup a new Dockerfile to install all dependencies in a single container.

@synchronizing Which openvino docker image did you try?
Did you use the data-dev version: https://hub.docker.com/r/openvino/ubuntu18_data_dev
This should have all dependencies for GStreamer and DL Streamer - Can you confirm?

@nnshah1
@synchronizing Which openvino docker image did you try?
Did you use the data-dev version: https://hub.docker.com/r/openvino/ubuntu18_data_dev
This should have all dependencies for GStreamer and DL Streamer - Can you confirm?

I've been using ubuntu18_dev this entire time. I see on the link you mentioned:
This ubuntu18_data_dev image includes the following data processing components unlike ubuntu18_dev image:

DL Streamer*
GStreamer
Speech Libraries and End-to-End Speech Demos

The docs in OpenVINO does not seem to have details on this, neither does the wiki here. Thank you for pointing this out -- I'll give it a try and report back in.

I have installed OpenVINO in my HOST machine (i.e. outside of the Docker container).
By calling "source /opt/intel/openvino/bin/setupvars.sh" the OpenVINO env variables and paths are exposed, e.g. INTEL_OPENVINO_DIR.
When downloading the models on HOST (i.e. outside of the Docker container) then set the env variable e.g. "export MODEL_PATH=/My/Path/gva/data/models/", (and "make -p /My/Path/gva/data/models/") the downloaded models then will be in subfolders "intel" and "common".

I need ask one more thing, while installing opentoolkit on host machine, im not able to install Intel® Graphics Compute Runtime for OpenCL". And FYI im using ubuntu 16.04 there are no drivers for ubuntu16. if so can tell how to do it!! thank you in advance!!

@brmarkus
I have installed OpenVINO in my HOST machine (i.e. outside of the Docker container).

Would you know if this is a must for it to work? My purpose with the Docker container is to eventually deploy it as an API pipeline, so having OpenVINO installed in the host machine isn't much of an option.
Thank you for taking your time to chime in what worked for you. Honeslty appreciated.

@umed
short answer is our inference elements does not draw boxes, they attaches meta data, but we have gvawatermark (should be placed after inference elements) that draws meta on video frames. If that does not solve your problem please create separate issue and we can continue discussion there.

I'll open a new issue from here then. I'm indeed using gvawatermark (ref), but for whatever reason still getting runtime errors when trying to pipe videos through it, with an empty video output. With @brmarkus insights, I think I know why now. Thank you for taking the time to answer, and my apologies for the thread growing so large.
In short, the key thing I believe I've gathered from the discussion above and other opened issues is the following:

OpenVINO must be installed in the host machine, and DL Streamer may be installed via Docker.
(#56 (comment)) DL Streamer has specific gstreamer dependencies.

I'll be trying later today and tomorrow to setup a new Dockerfile to install all dependencies in a single container. I'll leave the thread open for a day (if that sits well with the maintainers here) until I have a single Dockerfile working with OpenVINO + gstreamer + DL Stream.

@synchronizing  Another project that might be interesting / useful is our VA Serving which allows to easily create a pipeline service using DL Streamer. https://github.com/intel/video-analytics-serving, would be interested in any feedback you have on that as well.

@nnshah1
@synchronizing  Another project that might be interesting / useful is our VA Serving which allows to easily create a pipeline service using DL Streamer. https://github.com/intel/video-analytics-serving, would be interested in any feedback you have on that as well.

Oh man, that looks awesome, and exactly within the realm of tools I'm looking for. I'll definitely be check it out further, thank you.

Closing this. Ultimately, will be using the video-analytics-serving platform sent by @nnshah1. Thank you to everyone here who helped out!


TextBox watermark opacity/color override

It would be great to have an ability to override watermark style. I found out that watermark changing it's opacity to 0.2 on focus using storyboard, not style. I didn't find the way to override this storyboard (any suggestions??). I'm my case old  people are working with software and they barely can read watermarks despite that I've made it larger.

Hi @Waexu ,
as far as I can see there is no "easy" way to change the Storyboard, which comes by WPF intern performance reasons. You need to override the entire ControlTemplate.
Here is the original Template / Style: github.com/MahApps/MahApps.Metro/blob/develop/src/MahApps.Metro/Styles/Controls.TextBox.xaml
And here is the original Storyboard:https://github.com/MahApps/MahApps.Metro/blob/07dcfc57a22bfe3fd684cb54593f27532387b178/src/MahApps.Metro/Styles/Controls.Shared.xaml#L17
Happy coding
Tim

Hi @Waexu ,
as far as I can see there is no "easy" way to change the Storyboard, which comes by WPF intern performance reasons. You need to override the entire ControlTemplate.
Here is the original Template / Style: github.com/MahApps/MahApps.Metro/blob/develop/src/MahApps.Metro/Styles/Controls.TextBox.xaml
And here is the original Storyboard:
https://github.com/MahApps/MahApps.Metro/blob/07dcfc57a22bfe3fd684cb54593f27532387b178/src/MahApps.Metro/Styles/Controls.Shared.xaml#L17
Happy coding
Tim

Thank you for a reply!
Do you have any plans for customizing opacity values? I'm asking because I need to decide what to do: override ControlTemplate or just wait. Also I can implement it by myself however I never did commits to open source projects so if there are any special rules - let me know. Thank you

Hi @Waexu ,
I was not able to get it customized using DynamicResource or Binding, so I think it is tricky to implement customization here (I only found a workaround: https://stackoverflow.com/questions/2186933/wpf-animation-binding-to-the-to-attribute-of-storyboard-animation ).
You are welcome to contribute to MahApps, but I have one point for you. There are Templates for Bugs, Feature Requests and Pull Requests. Next time, please fill them out. see: https://github.com/MahApps/MahApps.Metro/wiki/Contributing
Maybe some might find a solution for it, so you can leave this issue open.


[Feature Request] Create cross-compiled debian package of orocos-kdl via docker

⚡ Feature Request
See comment.

Hi @aentinger
is this really necessary? There would already be a package in the mirror: liborocos-kdl-dev

I missed that :facepalm: . However, the most current version available via package manager seems to be v1.3.1 (released in 4/2016). Latest available version (and used within l3xz is v1.5.1).


Update goharbor/harbor-portal Docker tag to v2.8.2

This PR contains the following updates:

Package
Update
Change

goharbor/harbor-portal
minor
v2.6.1 -> v2.8.2

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
Because you closed this PR without merging, Renovate will ignore this update (v2.8.2). You will get a PR once a newer version is released. To ignore this dependency forever, add it to the ignoreDeps array of your Renovate config.
If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.


Unable to access LLaMA weights to build Vicuna-7B

Dear Team,
Thank you for sharing this great work with the community.
I am trying to set up the inference model for Video-LLaMA but I am having difficulties in downloading the official LLaMA 7b weights as they are not officially available from meta.
The provided links in below instruction do not provide for LLaMA 1 models. We can only download llama 2 models from the below links.
Get the original LLaMA weights in the Hugging Face format by following the instructions here.
Could you upload the llama 7b and llama 13b weights so we can directly utilize them?
Thank you in advance!
Kind regards,
Muhammad Uzair

I was able to download the weights from this repo.


[WIP] MyPy Linting Fixes

This PR contains the linting fixes for MyPy

@djyasin could you recreate this from fork? Having temporary branches upstream is usually noisy so @ core we forbid this.

Opened this off my fork now that it's public: https://github.com/ansible/awx-plugins/pull/26


Converted typedefs in actions submodule to using directives

flyby: fixed MSVC warning in internal allocator

This has been subsumed by #3894 and #3892. Closing.


Several participants added sections not under the TOC

The lines got added at the end of the file. We need to clarify this.

This is IMO solved.


Unhandled Promise Rejection: TypeError: Cannot read property 'endsWith'

Here is my log:
discord-bridge_1    | May-29 18:40:03.264 [Store] info: Starting DB Init
discord-bridge_1    | May-29 18:40:03.268 [Store] info: Filename present in config, using sqlite
discord-bridge_1    | May-29 18:40:03.270 [SQLite3] info: Opening database.db
discord-bridge_1    | May-29 18:40:03.285 [Store] info: Database schema version is 16, latest version is 16
discord-bridge_1    | May-29 18:40:03.290 [Store] info: Starting DB Init
discord-bridge_1    | May-29 18:40:03.292 [Store] info: Database schema version is 4, latest version is 4
discord-bridge_1    | May-29 18:40:03.301 [PuppetBridge] info: Starting application service....
discord-bridge_1    | May-29 18:40:03.320 [PuppetBridge] error: Unhandled Promise Rejection: TypeError: Cannot read property 'endsWith' of undefined
discord-bridge_1    |     at new MatrixClient (/opt/mx-puppet-discord/node_modules/@sorunome/matrix-bot-sdk/lib/MatrixClient.js:79:32)
discord-bridge_1    |     at new Intent (/opt/mx-puppet-discord/node_modules/@sorunome/matrix-bot-sdk/lib/appservice/Intent.js:44:23)
discord-bridge_1    |     at Appservice.getIntentForUserId (/opt/mx-puppet-discord/node_modules/@sorunome/matrix-bot-sdk/lib/appservice/Appservice.js:231:22)
discord-bridge_1    |     at Appservice.get botIntent [as botIntent] (/opt/mx-puppet-discord/node_modules/@sorunome/matrix-bot-sdk/lib/appservice/Appservice.js:132:21)
discord-bridge_1    |     at Appservice.<anonymous> (/opt/mx-puppet-discord/node_modules/@sorunome/matrix-bot-sdk/lib/appservice/Appservice.js:154:32)
discord-bridge_1    |     at Generator.next (<anonymous>)
discord-bridge_1    |     at /opt/mx-puppet-discord/node_modules/@sorunome/matrix-bot-sdk/lib/appservice/Appservice.js:8:71
discord-bridge_1    |     at new Promise (<anonymous>)
discord-bridge_1    |     at __awaiter (/opt/mx-puppet-discord/node_modules/@sorunome/matrix-bot-sdk/lib/appservice/Appservice.js:4:12)
discord-bridge_1    |     at registerBotIntent (/opt/mx-puppet-discord/node_modules/@sorunome/matrix-bot-sdk/lib/appservice/Appservice.js:152:45)
discord-bridge_1    | May-29 18:40:03.333 [PuppetBridge] error: Promise {
discord-bridge_1    |   <rejected> TypeError: Cannot read property 'endsWith' of undefined
discord-bridge_1    |       at new MatrixClient (/opt/mx-puppet-discord/node_modules/@sorunome/matrix-bot-sdk/lib/MatrixClient.js:79:32)
discord-bridge_1    |       at new Intent (/opt/mx-puppet-discord/node_modules/@sorunome/matrix-bot-sdk/lib/appservice/Intent.js:44:23)
discord-bridge_1    |       at Appservice.getIntentForUserId (/opt/mx-puppet-discord/node_modules/@sorunome/matrix-bot-sdk/lib/appservice/Appservice.js:231:22)
discord-bridge_1    |       at Appservice.get botIntent [as botIntent] (/opt/mx-puppet-discord/node_modules/@sorunome/matrix-bot-sdk/lib/appservice/Appservice.js:132:21)
discord-bridge_1    |       at Appservice.<anonymous> (/opt/mx-puppet-discord/node_modules/@sorunome/matrix-bot-sdk/lib/appservice/Appservice.js:154:32)
discord-bridge_1    |       at Generator.next (<anonymous>)
discord-bridge_1    |       at /opt/mx-puppet-discord/node_modules/@sorunome/matrix-bot-sdk/lib/appservice/Appservice.js:8:71
discord-bridge_1    |       at new Promise (<anonymous>)
discord-bridge_1    |       at __awaiter (/opt/mx-puppet-discord/node_modules/@sorunome/matrix-bot-sdk/lib/appservice/Appservice.js:4:12)
discord-bridge_1    |       at registerBotIntent (/opt/mx-puppet-discord/node_modules/@sorunome/matrix-bot-sdk/lib/appservice/Appservice.js:152:45)
discord-bridge_1    | }

Please tell me what else you need. Using the sorunome/mx-puppet-discord Docker Container.

I'm seeing the same thing


Refactors LinkedList implementation to be more Swifty.

List of changes:

Removed almost all assert and force unwrapping.
Property var head: Node? replaced to var first: Node?
Removed unnecessary self.
Default for replaced to forEach

How does this conflict?


[Monitoring V2] windows-exporter should no longer depend on rancher/wins

Currently, Monitoring V2 for Windows has a dependency on the underlying distribution using rancher/wins and supporting the ability to mount the established wins named pipes to execute host-process operations on the host.
https://github.com/rancher/charts/blob/8a4cd82fddc632d059d587a47aa7dd318e0781b7/charts/rancher-windows-exporter/100.0.0/templates/daemonset.yaml#L31-L49
This dependency should be removed to allow Monitoring V2 to be installed onto non-RKE2 clusters.

This issue can now be tested as the charts PR for 2.8 has been merged. Version 103.1.0-rc1+up45.31.1 should be used to test this issue. Basic functionality testing for monitoring windows nodes as well as chart upgrade and downgrade paths should be tested.
Testing of the related changes for 2.9 will be tracked in https://github.com/rancher/windows/issues/234

did some testing; currently blocked due to an issue with exporter not liking the fact that wins is on an RC'd version. Harrison is aware of this issue and will push a fix once the proxy testing is done

tested on v2.8-head (421ce19)

deploy a windows cluster on 1.27 using the currently released chart

upgrade the chart to 103.1.0-rc1+up45.31.1 -- pass

deploy a windows cluster on the latest k8s version available (1.28)

install the chart 103.1.0-rc1+up45.31.1 -- pass