Problems getting Hygieia UI under Tomcat to connect to API

I am hoping someone can me out.  I think I am missing something very basic.
I have Hygieia set up on a system and can run each piece correctly using the gulp
serve.  This means the API comes and connects to mongod and the UI launches under
gulp and I can access the dashboard through the UI interface.   I have been asked to
get the UI up under Tomcat rather than gulp serve.  To do this I set up Tomcat on the
same system, changed the port to 8090 so it did not conflict with the API Tomcat instance
and then moved the UI dist folder under the Tomcat webapps folder (and renamed it to
Hygieia).  Now, if I access Tomcat on that system and tell it specifically to bring up Hygieia .....
localhost:8090/Hygieia
the login page appears but the API Connectivity shows no connection.
I have seen that in the gulpfile.js file, a proxy is set up to point to localhost:8080 where the
instance of the API Tomcat would be.  I thought that is what I need to add to the start up of
my instance of Tomcat but so far everything I have tried (adding proxy elements to both the
catalina.sh and to the JAVA_OPT in the catalina.properties files) does not work.
I have looked through numerous other posts to try and find nuggets that I might be able
to look into to help me crack this nut - one post asked the submitter for the UI log files.  I
have not been able to find these on my system and I am pretty sure they might provide
some insight.
At this point I am thinking I am making this much more difficult than it has to be.  Any help
that anyone might be able to provide would be greatly appreciated.

@amayesthing You can use apache httpd instead of tomcat and refer to   https://github.com/capitalone/Hygieia/issues/895

Thank you @satishc1
I had looked into that because the issue that I cited above referring to the UI log files #1019 also requested details on the connection in the httpd conf.  I appreciate your suggestion and will take it back to the team to see if that would meet everyone's needs.  I would like to keep this open until I get concurrence from the team just in case anyone has a clever way to get it working under Tomcat.  Thanks again.

@amayesthing @satishc1
I got this working by deploying the Hygieia API within the Tomcat container. Basically, the UI and the API are deployed within Tomcat and all the collectors are running as a stand-alone java process. It works good and also provides an opportunity to scale-up & load balance APIs, if needed.
Let me know if you are interested in this deployment, I can document the changes (2 changes) and will send it out to you.

@spyndi
Wow - thank you.  Yes I am interested in seeing how to set that up.  That is very helpful.  Please send it to me when you get the chance.  Thank you!!

@amayesthing
You are welcome! Here it is..

Build the Hygieia API project as a WAR artifact instead of JAR. i.e, Change the packaging to war in API project pom.xml, like below:
<packaging>war</packaging>
and re-build the API project.

Deploy the api*.war file in Tomcat.

Hygieia API requires the "dashboard.properties" file for MongoDB connectivity. To set this up with in Tomcat container, do the following:
- Add a CATALINA_OPTS variable in Catalina.bat (or. sh under Linux) file like below (update the path based on your environment):
set CATALINA_OPTS="-Dspring.config.location=file:C:\DevOps\HygieiaApi\dashboard.properties"
Note : I added the above variable right below this line:
:java_dir_displayed
echo Using CLASSPATH:       "%CLASSPATH%"
set CATALINA_OPTS="-Dspring.config.location=file:C:\DevOps\HygieiaApi\dashboard.properties"

Restart your Tomcat server. You should be able to run the UI that should connect to your API deployed within Tomcat successfully. Try it out!

@spyndi thanks.

Thanks @spyndi - I appreciate your help - I have not tried this yet but it looks straight forward to me and I think it will resolve our problem.  Thank you for closing this issue.

@spyndi why dashboard.properties is required the same mongo's configuration is already present in application.properties

@anuragmishracrest  - both are same, it is just how you name the properties file.


Cant't seem to build if the path of the KMM project has spaces

What went wrong:
A problem was found with the configuration of task ':core:swiftklibNetworkUtilsImplIosArm64' (type 'CompileSwiftTask'). In plugin 'io.github.ttypic.swiftklib' type 'io.github.ttypic.swiftklib.gradle.task.CompileSwiftTask' property 'pathProperty' specifies directory 'xxxxx' which doesn't exist.
If the path of the KMM project has any folder with spaces, like ~/XYZ ABC/path the space of the path isn't skipped and it fails to build because can't read the folder containing the swift file..

@ahmadmssm this should be fixed in https://github.com/ttypic/swift-klib-plugin/releases/tag/v0.6.3

Feel free to reopen if this issue persists


EWSType_Autoloader

Per: https://github.com/jamesiarmes/php-ews/issues/35

This library can now be loaded with composer via the psr-4 branch which will be merged into master shortly.


All tags are serving PHP 8.0.11

It looks like a recent change to this project - using build args for base image selections - has caused all images to use the latest base PHP image.
A few local tests:
docker run -it chialab/php:5.6 php --version
PHP 8.0.11 (cli) (built: Oct 14 2021 21:21:21) ( NTS )
Copyright (c) The PHP Group
Zend Engine v4.0.11, Copyright (c) Zend Technologies
    with Zend OPcache v8.0.11, Copyright (c), by Zend Technologies

docker run -it chialab/php:7.4 php --version
PHP 8.0.11 (cli) (built: Oct 14 2021 21:21:21) ( NTS )
Copyright (c) The PHP Group
Zend Engine v4.0.11, Copyright (c) Zend Technologies
    with Zend OPcache v8.0.11, Copyright (c), by Zend Technologies

Sorry @hexus , my bad. 😞 We're working on this!

I had forgotten to set the build-args argument in the second build… tests ran on correct images, but the second build (multi-arch) built always the latest image.
Should be fixed in 346723f, pipeline is running and correct images should be made available shortly. Sorry for the inconvenience!


Add default service port for NIM as 8000

LGTM. nit: should we just name that field has port instead of OpenAIPort
sounds good, once NIM supports different ports for health checks, we can change them to healthPort and openAIPort.


[master] fly-outs get floated

I have built mater and I have started to see "fly-outs" (things like the auto-complete window in chrome, or the tab completion in Sublime text) get centered in the screen (as if a float rule was being applied to them), then snap back to the position they are supposed to be in.
This didn't happen before I updated today, so one of the recent commits seems to be the culprit. (If I have more time later this week I can step back though the commits to find the offending one).

Seems like a sideeffect of https://github.com/koekeishiya/kwm/issues/510


How to use grid.css to wireframe a SPI?

I would like to create a vertically-oriented single page site. So far I have not been able to get it to work while using Foundation.
I would like to take this code example I made and make it function while using Foundation, but I can not seem to get the containers to bend to my will yet.

Any ideas?

I'm not sure I understand your intent - do you have a mockup / sketch we could see?

I will give it a go. I had a quick grid question. it makes sense to me that nesting your content inside of the container  > row > columns . However when trying to style elements inside the grid, how should you write your rule? For instance .rulename{some css here} or .container.row.columns .rulename {some css here}. I'm just trying to get a sense of best practices?
Cheers and thanks again.

Definitely the latter - unless you explicitly need it you shouldn't have to include container>row>column in your declarations. Just pretend those aren't part of your styling unless you need to change them. Ping us if you have other questions<EMAIL_ADDRESS>
Jonathan Smiley
ZURB | Design Lead
<EMAIL_ADDRESS>408.540.8667
Build for Every Device
http://foundation.zurb.com
On Jan 23, 2012, at 7:01 AM, Denis R. wrote:

I will give it a go. I had a quick grid question. it makes sense to me that nesting your content inside of the container  > row > columns . However when trying to style elements inside the grid, how should you write your rule? For instance .rulename{some css here} or .container.row.columns .rulename {some css here}. I'm just trying to get a sense of best practices?
Cheers and thanks again.

Reply to this email directly or view it on GitHub:
https://github.com/zurb/foundation/issues/249#issuecomment-3615075


Invalid code generation for GFX10+ target when building PyTorch

I'm not sure if this is an issue in PyTorch, HIP or LLVM.
[3680/4568] Building HIPCC object caffe2/CMakeFiles/torch_hip.dir/operators/hip/torch_hip_generated_batch_sparse_to_dense_op.hip.o
FAILED: caffe2/CMakeFiles/torch_hip.dir/operators/hip/torch_hip_generated_batch_sparse_to_dense_op.hip.o
cd /home/erkki/Downloads/rocm2/pytorch/build/caffe2/CMakeFiles/torch_hip.dir/operators/hip && /usr/bin/cmake -E make_directory /home/erkki/Downloads/rocm2/pytorch/build/caffe2/CMakeFiles/torch_hip.dir/operators/hip/. && /usr/bin/cmake -D verbose:BOOL=OFF -D build_configuration:STRING=RELEASE -D generated_file:STRING=/home/erkki/Downloads/rocm2/pytorch/build/caffe2/CMakeFiles/torch_hip.dir/operators/hip/./torch_hip_generated_batch_sparse_to_dense_op.hip.o -P /home/erkki/Downloads/rocm2/pytorch/build/caffe2/CMakeFiles/torch_hip.dir/operators/hip/torch_hip_generated_batch_sparse_to_dense_op.hip.o.cmake
error: Illegal instruction detected: Invalid dpp_ctrl value: broadcasts are not supported on GFX10+
renamable $vgpr23 = V_MOV_B32_dpp killed $vgpr23(tied-def 0), $vgpr13, 322, 15, 15, 0, implicit $exec
error: Illegal instruction detected: Invalid dpp_ctrl value: broadcasts are not supported on GFX10+
renamable $vgpr47 = V_MOV_B32_dpp killed $vgpr47(tied-def 0), $vgpr14, 322, 15, 15, 0, implicit $exec

Full build output here: https://gist.github.com/rigtorp/db4241d25b753c952962e554686735dc

I can't find any use of inline assembler or builtins (__builtin_amdgcn*) in PyTorch relating to this.
Maybe invalid use of __builtin_amdgcn_mov_dpp?
HIP uses this builtin: include/hip/hcc_detail/device_functions.h
305:    return __builtin_amdgcn_mov_dpp(src, dpp_ctrl, row_mask, bank_mask,

See https://github.com/ROCmSoftwarePlatform/pytorch/issues/718

Turns out this is an issue in rocPRIM: https://github.com/ROCmSoftwarePlatform/rocPRIM/issues/180


Fix to Depth of Field Example

Added example/web_gl_postprocessing_dof/dat.gui.min.js, which is required by the depth of field example but was being excluded by the auto-generated Dart Editor .gitignore file.

It indeed fixes it.  As it is just an example, I guess it's okay to add that JS library there.


Documentation clarification

Hi,
I have a question about the "consumer mode partitions".
I have this kind of code :
`func main() {
config := cluster.NewConfig()
config.Group.Mode = cluster.ConsumerModePartitions
    ....
consumer, err := cluster.NewConsumer(brokers, "my-consumer-group", topics, config)
    ...
    // consume errors
go func() {
	for err := range consumer.Errors() {
	    ...
	}
}()

// consume notifications
go func() {
	for ntf := range consumer.Notifications() {
		....
	}
}()
// consume partitions
for {
	part, ok := <-consumer.Partitions()
            ...
}

}`
And it is working as expected. I'm receiving notifications on the notification channel, errors on the error channel and messages for each partitions (in the respective channels).
However here, the comment is saying that in the "ConsumerModePartitions" mode, the error channel and notification channel would not work (and in your README "ConsumerModePartitions" mode example you're not using them)
So my question is  :  in the "ConsumerModePartitions" mode, is it correct to expect errors and notifications on their respective channels ? Or do I have to handle errors on the messages channels ?

Hi, sorry for the delay. The documentation can be found here: https://godoc.org/github.com/bsm/sarama-cluster#Consumer.Partitions. There is an example too, hope that explains it.


Url with DateTime is not binding correctly to Action

I have a sitelet that has an action shape with a DateTime value:
type Action =
     | Test of DateTime

Sitelet.Infer <| function
    | Test dt -> (...)

The URL to this action is generated correctly: http://localhost:5043/Test/2014-12-19T15:38:20.5096124+00:00
The problem is that if I do a GET request on this URL it will not execute the Test branch. It will, however, if I manually edit URL to be http://localhost:5043/Test/2014-12-19.
I've tested UrlFormatter itself and it seems to be working fine (GetFormatFor reads and shows URL correctly). I suspect it might be something connected with OWIN pipeline.

The problem is that the character : is invalid in the path part of a URI. When I try to do the same with the ASP.NET pipeline, I get a yellow error page with the message:
A potentially dangerous Request.Path value was detected from the client (:).

So clearly the way we encode DateTime in URIs is wrong; I'm not sure what would be a good format though.

Personally I would be happy with "yyyyMMddTHHmmss". Some discusson on the topic here: http://stackoverflow.com/questions/2344987/is-there-a-standard-date-time-format-that-can-be-passed-on-a-url

The default is now "yyyy-MM-dd-HH.mm.ss", and you can use DateTimeFormatAttribute to customize it:
type Action =
    | [<DateTimeFormat("date", "yyyy-MM-dd")>] Test of date: DateTime


In 2 pass mode split video by GOP and encode in parallel

could be when scene changes completely.

just a short test using ffmpeg to split up the video, the speed is at/above x265 just with this.

Yeah, I'm not sure if it's better to do this in an external tool or build it into rav1e. We do have a feature that dumps keyframe locations to a file: https://github.com/xiph/rav1e/commit/50269cb681031d6bdd5e7cfa9e55538e288f58ea

I think it would be better if rav1e can do this, at least until proper multi-thread, sticking ivfs back together with ffmpeg kind of worked, but when I tries to get sample frames from it ffmpeg through a lot of error.

I'll try to get something I was working on out soon.

is that segment mutithreading?

I have a WIP tool to do this https://github.com/rust-av/rav1e-by-gop but it currently doesn't support two-pass. It probably could by using rav1e's first pass to generate the keyframe list instead of av-scenechange. At that point I wonder if it would be better to build it in to rav1e.

looking at https://github.com/xiph/rav1e/commit/50269cb681031d6bdd5e7cfa9e55538e288f58ea it seems to use first-pass data seems quite easy


Can redux-saga-routines work with redux compose?

Hello,
I've tried making it work with redux-saga-routines and compose instead of using sagaMiddleware.run. Should that be working?

compose is just an utility function to make function composition, so i'm not sure what you are trying to do.
createRoutine may work without redux-saga, since it's just a creator, but handling promises is not possible without routinePromiseWatcherSaga which is designed to work with redux-saga, so it will not work without it.

@Ctsavas do you still need help with it?

Hello @afitiskin , sorry for the delayed response.
I was trying to use redux-saga-routines with loading routinePromiseWatcherSaga using the injectSaga pattern on the component level (as described here for example https://heygrady.github.io/redux-saga-watch-actions/docs/middleware/injectSaga.html).
I solved my problem by adding routinePromiseWatcherSaga to a top parent component.
My problem with asking for help in the first place was that the suggested docs way of loading the saga saga with sagaMiddleware.run(), which I couldn't use in my case.

If you check the source code of injectSaga you will see that injectSaga uses sagaMiddleware.run, so it will work for you


Add Support for setting shm_size for accessories

Basically, I would like to increase the size of the /dev/shm in the postgres docker container.
This is what I added to my deploy.yml
accessories:
  db:
    image: postgres:16
    host: ...
    restart: always
    env:
      ...
    files:
      - db/production.sql:/docker-entrypoint-initdb.d/setup.sql
    directories:
      - data:/var/lib/postgresql/data
    options:
      network: "private"
      shm_size: 2gb  <----- this here

When running kamal accessory reboot db I get the following errors:
docker stderr: unknown flag: --shm_size

From the command that get's run on the server I see the following: docker run ... --network "private" --shm_size "2gb" postgres:16.
Can someone potentially help me out?

Did you try specyfing it in yaml as shm-size: 2gb?

@SamuelAierizer do you still have the issue with the suggestion above?


Failover for Context Service

Answer service fails over to one of the other AI's (in ChatAppContainer) but ContextService does not.
Need to add endpoint for gpt-4o-mini-2024-07-18 (or maybe can just put gpt-4o-mini - not clear from this model page https://platform.openai.com/docs/models/gp#gpt-4o-mini )
And then add failover logic

Fixed and test here: https://react-answers-p28r3jahg-test-canada.vercel.app/

Fixed.


Fixup/channel slider fix

Fixed:

activating slider for selecting heatmap by channelId
selecting heatmap by chanellId = 0 on init

 Do not require selecting spectrum on simple change of heatmap mass channel


Update EfficientSugiyamaLayoutAlgorithm.HorizontalAssignment.cs to fi…

…x exceptions thrown if no edges exist.

Thanks!
From: Alexander Smirnov<EMAIL_ADDRESS>Sent: Monday, August 31, 2015 17:14
To: panthernet/GraphX<EMAIL_ADDRESS>Cc: bleibold<EMAIL_ADDRESS>Subject: Re: [GraphX] Update EfficientSugiyamaLayoutAlgorithm.HorizontalAssignment.cs to fi… (#35)
Thanks! :)
—
Reply to this email directly or view it on GitHub https://github.com/panthernet/GraphX/pull/35#issuecomment-136524973 .


Unknown Error: Unknown filter category: genre

Started receding this error on or about 3/10 ... upgraded to latest version and still receiving this error.
More info:
Traceback (most recent call last):|
|   File "/modules/config.py", line 461, in update_libraries
|     builder.run_methods(collection_obj, collection_name, rating_key_map, movie_map, show_map)
|   File "/modules/builder.py", line 684, in run_methods
|     items = self.library.Plex.search(**search_terms)
|   File "/usr/local/lib/python3.9/site-packages/plexapi/library.py", line 668, in search
|     args[category] = self._cleanSearchFilter(category, value, libtype)
|   File "/usr/local/lib/python3.9/site-packages/plexapi/library.py", line 717, in _cleanSearchFilter
|     raise BadRequest('Unknown filter category: %s' % category)
| plexapi.exceptions.BadRequest: Unknown filter category: genre
|
Nothing has changed in my Movie.yml ... here is an excerpt:
Action:
plex_search:
genre: Action
collection_order: alpha
Adventure:
plex_search:
genre: Adventure
collection_order: alpha
Animation:
plex_search:
genre: Animation
collection_order: alpha
Biography:
plex_search:
genre: Biography
collection_order: alpha
Comedy:
plex_search:
genre: Comedy
collection_order: alpha
Crime:
plex_search:
genre: Crime
collection_order: alpha
Documentary:
plex_search:
genre: Documentary
collection_order: alpha
Family:
plex_search:
genre: Family
collection_order: alpha
Fantasy:
plex_search:
genre: Fantasy
collection_order: alpha
History:
plex_search:
genre: History
collection_order: alpha
Horror:
plex_search:
genre: Horror
collection_order: alpha
Music:
plex_search:
genre: Musical
collection_order: alpha
Romance:
plex_search:
genre: Romance
collection_order: alpha
Science-Fiction:
plex_search:
genre: Sci-Fi
collection_order: alpha
Sport:
plex_search:
genre: Sport
collection_order: alpha
Thriller:
plex_search:
genre: Thriller
collection_order: alpha
TV Movie:
plex_search:
genre: TV Movie
collection_order: alpha
War:
plex_search:
genre: War
collection_order: alpha
Western:
plex_search:
genre: Western
collection_order: alpha
None of these collections are updating as a result. My IMDB lists are fine.

Never mind ... I see this is related to Plex Search


userEmail field is missing from Log Analytics for Azure Bastion =  microsoftazurebastionauditlogs

[Enter feedback here]
this is really concerning below , but it has no github raise issue
https://docs.microsoft.com/en-us/azure/azure-monitor/reference/tables/microsoftazurebastionauditlogs?source=docs
Seems that userEmail field is not included when sent to Log Analytics as per above documentation, but IS included when sent to a Storage Account . Is that true / why is that ?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d523f27c-74fc-3a85-e89e-111f8bf8d276
Version Independent ID: 50ad9ec6-a57b-72ed-8eb3-5d63737f0fe5
Content: Enable and work with Azure Bastion resource logs
Content Source: articles/bastion/diagnostic-logs.md
Service: bastion
GitHub Login: @charwen
Microsoft Alias: charwen

Thank you for your feedback! We will review and update as appropriate.

Hello @andyjballgit, thank you for your feedback, this is a known issue, and the team is currently working on the fix. The fix is expected to be rolled out in few weeks from now.
Hope this is helpful. We will now proceed to close this issue out, please tag me in your response if you have any additional questions. Thank you!


Add debug tool to get xDS config for a sidecar

Usage:
First, you need to expose pilot gRPC port:

By port-forward existing pilot:

kubectl port-forward $(kubectl get pod -l istio=pilot -o jsonpath={.items[0].metadata.name} -n istio-system) -n istio-system 15010

Or run local pilot using the same k8s config.

pilot-discovery discovery --kubeconfig=${HOME}/.kube/config

Then, to get LDS config, run:
go run pilot_cli.go -pod <POD_NAME> -ip <POD_IP> -type lds

Use cds | eds | rds for other config types.
You can also provide the path to kubeconfig to find pod IP automatically (by default, it will look for the config under ${HOME}/.kube/config)
go run ./pilot/debug/pilot_cli.go -pod <POD_NAME> -kubeconfig ${HOME}/.kube/config

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

Needs approval from an approver in each of these files:

pilot/OWNERS

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

@diemtvu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
de4fac660b8325913822912935097695567563b0
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

This is great @diemtvu !! This makes it the third debug tool for pilot :P.
@liamawhite has a PR that builds on the debug endpoints that @costinm exposed in pilot (/debug/adsz) [ #4803 ], that he was thinking of using with istioctl debug podID or something like that.

@diemtvu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
8de2c2da81ae1b433f1245e6fc51cd9e3d31fba2
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@diemtvu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
e3e9f8d66fb6bb9c17357bcec8e21f627d93d3c3
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: costinm
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [costinm]

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

New changes are detected. LGTM label has been removed.

@diemtvu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
1d99aa543500776e18378e026d53a6678474e058
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@diemtvu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
ab69a3d132d25cd37ba4f8e127ad38298b83bbf4
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@diemtvu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
e01a74609c25f8939a8bb8446ceca3e8e5955171
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@diemtvu: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
e01a74609c25f8939a8bb8446ceca3e8e5955171
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


[Cherrypick 1.1 -> 1.2] Add application port option for gateways (#14059)

Add application ports for gateways

Check all ports for readiness

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

Needs approval from an approver in each of these files:

install/OWNERS [howardjohn]
pilot/OWNERS

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn, nmittler
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [howardjohn]
pilot/OWNERS [nmittler]

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


Wrong range displaying number

When you click anywhere on the range input after sliding it, the label will show you the former value instead of the new clicked one.

I can reproduce it on the live website http://materializecss.com/forms.html

+1. Range element should update its value while being dragged.

Fixed


iOS: Migrate to build with SPM + Bazel

Migrate away from CocoaPods as a package manager, and build through SPM
We will still publish to CocoaPods but we do not need to build and resolve our dependencies through it, as it complicates usage of Bazel

on https://github.com/player-ui/player/tree/bazel-6
bazel build //:PlayerUI_Pod

will build a SPM compatible zip
zip contents still needs to be published to a separate repo so the JS files will be present in the git location

any update for swift support?

Closing because core work is done, issue to document consumption is #495


discovery/file: fix flaky tests

Closes #5901
This change refactors the test code to make it more deterministic. It also implements different tests to check different cases (initial update, update, deletion, ...) rather than having a big function combining everything.

@brian-brazil PTAL

Thanks for the comments @cstyan. I've updated the PR: there's now *testRunner.copyFile(src string) when the destination name is the same as the source file (most of the cases) and *testRunner.copyFileTo(src string, name string) otherwise.


MKL always links sequentially instead of the parallel versions

It has to do with https://github.com/votca/tools/commit/719d51188b01538cd3f7de061c693640999081f9
I do not really know what is happening, because we set the threading layer here
https://github.com/votca/tools/blob/master/CMakeLists.txt

@junghans I still do not fully understand why we have the FindMKL in csg and tools. Can you explain it once more?

we have find_mkl_library(NAME ThreadLayer LIBRARY_NAME mkl_sequential) in all the CMakeLists

Can you change the default of MKL_THREAD_LAYER back?

Yeah sure, but the default should not be used, because I set it explicitly before that. That confuses me.

https://github.com/votca/tools/pull/160


Plugin does not recognise dependabots Build as a valid pr title

Describe the bug
dependabot sometimes creates pr that start with "Build" which is valid according to conventional commit spec but semantic pr always fails for those prs
semantic.yml contents

Sample PR Title(s) (if relevant)
Build(deps-dev): Bump typescript from 5.0.4 to 5.1.3
Sample Commit Message(s) (if relevant)
No response

@DebkanchanSamadder that specific example fails the check because Build is not all lowercase - if you use build instead then the check will pass
If you want to keep that capitalisation then you can set types in semantic.yml (see readme) to be Feat, Fix, Build, etc, or whatever you want 🙂

I added the following to my org's .github repo's semantic.yml but it still doesn't work
types:
  - Build

@DebkanchanSamadder did you put it in the .github directory inside your org's .github repo?
i.e. if you visit https://github.com/<your org name>/.github/blob/master/.github/semantic.yml then can you see the file?
Looking at the logs, assuming the org that you're talking about is the one from your GitHub profile, then I can see 404 errors when the app is searching for the semantic.yml file, for both your org and for specific repos

I forgot to put it inside .github directory. Now it works fine. Thank you very much.


Type mismatch

Everything used to work just fine until a few days ago, used to make a few changes on publisher/subscriber but then i got stuck on this.....

thank you

i have updated all the files due to a recent change by MQL , its all fine,
sorry for this


State machine binding error

Hello! I tried to use state machine in my code but I get bindingError: 'function StateMachineInstance.advance called with 1 arguments, expected 2 args!' when the play is true.
Could you help me why?
  <canvas riv="rocketWithText">     <riv-state-machine name="State Machine 1" play>       <riv-input name="Text" [value]="100"></riv-input>       <riv-input name="Fire" [value]="100"></riv-input>       <riv-input name="Move" [value]="100"></riv-input>     </riv-state-machine>   </canvas>

Which version of rive-canvas are you using ? I saw they recently changed there API, but I didn't updated yet.
ng-rive should be working with<EMAIL_ADDRESS>

I didn't install rive-canvas explicitly, I use your ng-rive plugin which already contains rive-canvas. (0.7.3). By the way the riv-animation works as expected.

Hmmm ok might be because of the ^ in the package.json. In you package-lock.json the version or riev-canvas is 0.7.3 ?
I'll update to the latest version anyway asap but it might help me avoid future issues.

The version is 0.7.5 in the package-lock.json. So this can be the reason. I will check after you update the library.
Thank you for the help.

I updated the version to 0.7.6: https://github.com/dappsnation/ng-rive/pull/21
Please close the issue if it works.

The issue is fixed. Thank you!


RCTBridgeModule.h file not found

happen to what branch, Do you can give me the wrong address and error screenshot

@febobo

@NexusLee
try change RCTWebChat.xcodeproj → Build Settings → Search Paths → Header Search Paths to
$(SRCROOT)/../../react-native/React and
$(SRCROOT)/../../react-native/Libraries
https://github.com/weflex/react-native-wechat/issues/33

It worked. Thx


Druid dashboard import/export.

Same as: https://github.com/airbnb/superset/pull/1811

@mistercrunch  - fixed the build, will merge and monitor master build.


Bug in jruby resolv.rb causes intermittent DNS lookup failures on agent run

The resolver library in jruby (resolv/lib/resolv.rb) has a bug where it ignores the OS ephemeral port range and instead attempts random UDP ports in the range 1024..65535 until it finds one that is unused. This causes puppet agent runs to experience intermittent DNS lookup failures on systems that are subject to strict firewalling either locally or externally to the system. This has also been raised as a ruby issue here
DNS lookup fails during puppet agent run when it is expected to succeed.
Steps to reproduce the behavior:

configure a system such that the ephemeral port range is restricted, and firewall ports outside of this range.
run puppet agent
failure rate depends on the random ports attempted, the more restrictive the range, the more likely failure is.

Tested on AlmaLinux 8.10 where the default /proc/sys/net/ipv4/ip_local_port_range is 32768 .. 60999 but this should apply to most modern distro variants / OSes.
See the ruby bug for additional details - it should be sufficient to remove the dependence on a secure random number generator altogether (which has generated CVEs in the past) and instead push the security to the OS to determine the ephemeral port and use port = 0 for UDP binds.

@dmz-uk could you give us a bit more information, what version of Puppet are you using and what version of JRuby?

The issue was discovered on the older puppet-agent 7.33.0 from the puppetlabs yum repository packaged for el8, running on AlmaLinux 8 (puppet-agent-7.33.0-1.el8.x86_64) - the jruby is contained in the package (/opt/puppetlabs/puppet/lib/ruby/2.7.0/resolv.rb). However, the issue as identified appears to still exist unmodified in the trunk of both puppet and jruby.


Added a --version option for displaying version info

I'd argue that keeping the -chia makes sense for the user to distinguish that this is a chia-maintained fork, and not official versioning.

I would think that chia would be a separate tag from alpha.  Having both would certainly be good sometimes.  But yes, as long as we are making our own adjusted builds we should mark them.  I suspect there's some level of standard for this but I don't know where offhand.


Allow TypeScript imports in BAML files

I'd like to import types I've already defined in my TypeScript project into BAML src files instead of copying and pasting or redefining the types.

this would be interesting -- i'll chat with the team about this


MQTT Out Message Not Accurate When Returning to Previously Selected Mode

When monitoring the MQTT /out topic, if the controller is set to go back to the previously selected mode, the payload sent is still "OK =off".  Only when the newly selected mode is something other than what was previously selected does the topic change correctly.
To reproduce this problem:

Go to the web console.
Select any combination of Colors/brightness and mode= Static
Select Off (this will produce the payload "OK =off" as expected)
Select mode Static again.  (This will also produce the payload "OK =off".  I would expect this to return "OK /0".)

This works for any combination of mode/color and brightness.  Simply selecting off and then returning to the previously selected mode produces the wrong payload.

It also appears that I am unable to get regular MQTT messages when using ENABLE_HOMEASSISTANT.  The only way I can get status updates to be sent to my broker is to use the Aysync method but that results in a rather significant delay between the change taking place and the message being sent to my broker.
I am trying to use the message to detect the state of the controller since I have two possible control points outside of my McLighting controller.  Rather than go through the trouble of colating them before being sent to the controller I figured I could just use the MQTT messages to detect a change initiated by the other system.  But when I subscribe to home/_ha/state/out, I only get messages when using the ASYNC method.  Any assistance would be greatly appreciated.
This is the relevant section of my definitions.h file.
#define ENABLE_OTA 1
#define ENABLE_MQTT 0
//#define ENABLE_MQTT_HOSTNAME_CHIPID
//#define ENABLE_MQTT_INCLUDE_IP
#define ENABLE_HOMEASSISTANT
//#define MQTT_HOME_ASSISTANT_SUPPORT
//#define DISABLE_MQTT_OUT_ON_MQTT_IN
If you could let me know how to fix either of these issues, that would be a huge help.

I will have a look.

First i can not reproduce.

To reproduce this problem:
Go to the web console.
Select any combination of Colors/brightness and mode= Static
Select Off (this will produce the payload "OK =off" as expected)
Select mode Static again. (This will also produce the payload "OK =off". I would expect this to return "OK /0".)
This works for any combination of mode/color and brightness. Simply selecting off and then returning to the previously selected mode produces the wrong payload.

Would you please post http://{ip_of_your_device}/esp_status.
The second not updating homeassistant using PubSubClient i can reproduce.
FYI: There is a 2-3 sec delay if you use AsyncMqtt. This can be reduced to almost 0, but is not recommended.

Sure.  Here's the esp_status:
{"HOSTNAME":"deskMclighting","version":"3.1.0","heap":21160,"sketch_size":558656,"free_sketch_space":2584576,"flash_chip_size":4194304,"flash_chip_real_size":4194304,"flash_chip_speed":40000000,"sdk_version":"2.2.1(cfd48f3)","core_version":"2_4_2","cpu_freq":80,"chip_id":1458415,"animation_lib":"WS2812FX_DMA","ws2812_pin":3,"led_count":50,"rgb_order":"GRB","rgbw_mode":"OFF","button_mode":"ON","button_pin":14,"button_gy33":"OFF","ir_remote":"OFF","mqtt":"AMQTT","home_assistant":"ON","ota":"HTTP","state_save":"SPIFFS"}

There is a 2-3 sec delay if you use AsyncMqtt.

I'm seeing more of a 5-7 second delay.  Could that is be because of my setup?  Do I need to change one of the settings for QoS (i'm still wrapping my head around the MQTT).

The second not updating homeassistant using PubSubClient i can reproduce.

Oh good...so I'm not going crazy(er)  😉

First i can not reproduce.

Here are the logs from Node Red.  You can clearly see, first both the /out and /ha/state/out both report off.  Then I turn it to static mode by clicking on static in the WEB Gui.  (That and the HTTP API are the only two control methods I have tried and confirmed this issue with).  You can see /out goes to "OK /0" and the same is reported to HA.  Then I turn it off by clicking off in the web GUI.  Again, both statuses show off correctly.
Next, I click "Static" again in the web GUI.  You can clearly see the /out reports "OK =OFF" and HA reports "State": ON.  This is where the problem lies.

Should the last report to /out really be "OK =off"?  The strip is on at that point.

Fixed HA not working with PubSubClient

Ahh, now i can reproduce, but i have no solution yet.
Stand by.....

Ahh, now i can reproduce, but i have no solution yet.

I was about to get out my phone and record it for you.   I might be going Covid-crazy but not THAT crazy.  ;-)  Thanks for all your help!

Fixed HA not working with PubSubClient

I'm still not able to get HA MQTT messages if I only enable:
#define ENABLE_HOMEASSISTANT
Maybe I am not understanding the intended functionality.  But I thought that enabling this would get HA communications using MQTT.  And that enabling:
#define MQTT_HOME_ASSISTANT_SUPPORT
would get auto-discovery of HA devices.  But either that is incorrect or it is not working as intended.  Because I get nothing out unless I enable the second option as well.  In fact, if I have my settings like this:
#define ENABLE_MQTT 0
//#define ENABLE_HOMEASSISTANT
//#define MQTT_HOME_ASSISTANT_SUPPORT
#define DELAY_MQTT_HA_MESSAGE 2
I get MQTT messages on /out.  They're wrong, but I get them.  Then if I change the settings to this:
#define ENABLE_MQTT 0
#define ENABLE_HOMEASSISTANT
//#define MQTT_HOME_ASSISTANT_SUPPORT
#define DELAY_MQTT_HA_MESSAGE 2
I then get no MQTT messages at all, either on /out or on home/_ha/state/out.  It is only when I change the settings to:
#define ENABLE_MQTT 0
#define ENABLE_HOMEASSISTANT
#define MQTT_HOME_ASSISTANT_SUPPORT
#define DELAY_MQTT_HA_MESSAGE 2
do I start to get MQTT messages again.  So, am I totally misunderstanding how those setting are supposed to work or is something seriously wrong here?  Because, I thought you had to have AMQTT enabled for MQTT_HOME_ASSISTANT_SUPPORT  but it is working with PUBSUB.
(note: I removed the extra settings that did not change but the other settings are at default)
I have everything I need now that you have the customizable delay option available.   This is purely for my edification that it just doesn't seem right.  But "That's how it works."  would be a completely acceptable answer at this point.  Don't want you to think I'm harping or nagging.  It just didn't look right to me and I know that kinda stuff bugs the hell out of me (pun totally intended) so I thought I would bring it up.  😄

@ryancasler
Not finished yet but please test the development tree, if the behaviour is now as expected!!

YAY!!!  You rock. I checked all the permutations I could think of and everything seems to be working per the docs now. Great work! Thank you!!!


Use sortedByDescending

This is a thing that detekt could check. It have no sense to create a List sorted to after change it's order. We could create it directly in the other that we want.

Warnings

:warning:
It looks like this PR contains functional changes without a corresponding test.

  Generated by :no_entry_sign: dangerJS against f0484fb4448dce3da1d3a7cb3f3f48c2a05d9017


Update 0112-path-sum.java

The new solution added is little more closer to the one explained in the video, the solution present right now has a different logic of reducing the targetSum. But the one explained in the video is to create a new variable curSum and then compare it with targetSum.

File(s) Modified: 0112-path-sum.java
Language(s) Used: Java
Submission URL:  https://leetcode.com/problems/path-sum/submissions/956511674/

Important
Please make sure the file name is lowercase and a duplicate file does not already exist before merging.

Hi @sujal-goswami
I have created a new pull request for the same: https://github.com/neetcode-gh/leetcode/pull/2835
Could you please check and let me know if it works.


please support max vh and vw

such as:
1.mv, then if 1.vw > 1.vh, 1.mv is 1.vw, else 1.mv is 1.vh

Hello, @Silentdoer Thanks for creating this issue.
I have worked on your issue and created .mv unit. It has the functionality of returning max(.vh, .vw) for a given input.
Check out this PR #4. If any assistance or malfunctions occurred on this new unit let me know.
Finally thanks for contributing.

Published the changes. Kindly change the version to 1.0.0 to get .mv unit.
Let me keep this issue open for a couple of days. If any issues arise, share that in this issue.
Once again thanks for contributing.


Determine how package attributes/files might be templated using package metadata

It was pointed out in the review for #173 that we may want to expand various template attributes (e.g. the version) to other parts of the RPM spec file, such as the %description.
This raises another larger point: what attributes of packages should be template-able?  In many cases, it might make sense to subject all of them to template substitution, for example, some filesystem paths are annotated with versions, like those in /usr/share/doc in CentOS.
Another important part of this is figuring out the templates' syntax and how they're to be processed in a cross-platform manner.

Yes. The more the merrier.  The idea in #198 allows us to bring a dict of string->string mappings into any of the rules so that we can expand it in any way we need. Any rpm spec template is a possibility

This is getting old and not specifically actionable. Closing to clean out the backlog.


When checking file/dir existence, manage all errors

It only happened once, when testing wolfi as docker base: the mounting of resources in docker was somehow faulty, and this code:
func fileExists(filename string) bool {
	info, err := os.Stat(filename)
	if os.IsNotExist(err) {
		return false
	}
	return !info.IsDir()
}

(also the similar dirExists right below) would panic with a "pointer dereference", because there was an err not of the "right" type and it wasn't caught, so info was nil. Give a fault and exit if this happen, so that the reason is more evident.

Released in v0.14.2


[Go] Error was not handled

Describe the bug, including details regarding any error messages, version, and platform.
The error thrown during the ReadRecords process was not handled or returned correctly.

HasNext returns false because of an error, then the for loop broke:

But the error isn't being returned:

Component(s)
Go

Issue resolved by pull request 43861
https://github.com/apache/arrow/pull/43861


Add image convert to webp

Ticket: #
Issue

Using images lighter than jpg
Solution

Allows to convert images to webp like tojpg
Impact

None
Usage Changes

Added a simple example in the documentation
Considerations

Testing

Tests are included, and run without problems

Hello! Do I need any changes?
In travis my tests break down. I think this is related to this issue https://github.com/travis-ci/travis-ci/issues/8435
Will this be ok if I check the existence of the imagewebp function in the tests for skip it?

Any updates on this PR?

@mhz-tamb @pascalknecht — thanks for your patience! I just got back last week from a paternity leave and this PR is next on my plate

@mhz-tamb @pascalknecht thanks again for the PR and your patience. As @mhz-tamb points out there is a Travis limitation on testing for webp — that said, things pass for me locally. I want to make sure we account for webp in the future, so here are two possible mitigations:

Using the guidance of #8435 figure out how to enable imagewebp in the PHP Travis build we're using
Skip the tests if imagewebp is not available.

@mhz-tamb — can you pick things back up from here?


Fix ipv6 netmask

Proposed changes

Unreleased project version (optional)

Related issues (optional)
Fixes #933

cc/ @drnybble, @aboch, @JimGalasyn

Looks good to me

LGTM! Did you update the image as well? That's not included in this PR yet.

The image is for the other netmask issue. #825
This is a fix for an already merged PR which had wrong info.

Thanks for the fix, @shubheksha !


Add tags support for templates

Add tags support for templates. Currently, tags are supported only for apt and raw files.

Fixed on master.

Duplicate #359


sap.m.BusyIndicator wobbles

The icon of the sap.m.BusyIndicator seems to be slightly off, since it wobbles when spinning.
For a preview of some wobbling aciton, see here.

Hi verfehlix,
I will talk to our designers and let you know what they say.
(Further infromation: issue is only reproducible in CHrome, not in IE).
Kind regards,
Janina

Hi verfehlix,
the sap.m.BusyIndicator will be redesigned. I cannot give you a timeline, but we will let you know once the new version is available. This issue should then be fixed as well.
Kind regards,
Janina

Hi vefehlix,
you can checkout the new design when you have a look at the 1.32 preview version.
Have fun,
Janina

Thanks! Will do

Closing this incident because of the "frozen" discussion so maybe the bug does not occur anymore :)
Feel free to reopen it if needed.
Thanks and Kind Regards,
Diana, SAPUI5 Dispatcher

Due to the redisgn, there is indeed no more wobbeling (as there is no longer a spinning circle that could wobble). Thanks for the quick response and great work!


Using debian:stable-slim tag instead of debian:bullseye-slim

Would it be possible to use debian:stable-slim tag on the images instead of debian:bullseye-slim?
If there is no desire to move to "moving" tag such as debian:stable-slim, then perhaps debian:bookworm-slim could be used?
I'm making a small tool which uses these images as base and installs (and uses) some libraries on top, as bullseye is already old stable the libraries are quite old. Currently I update the Debian to latest stable, but because Debian is swithing to merged usr directory on bookworm the update is quite a hassle (and a hack) and of course it takes a little bit of time on the first run of the container.
So it's this bullseye->bookworm jump that is causing problems. If the tag was already debian:bookworm-slim then updating to for example trixie (Debian testing branch as the time of writing) would also be easy.

Thank you for using the containers. Upgrading to bookworm is fine by me.
It's convenient that stable-slim exists, but for reproducibility it's better to specify the Debian version explicitly for each Dockerfile. A few containers use even older versions of Debian since those Scheme implementations are so old. We have alpine versions of some containers, too.

If you have an interest in maintaining the containers, I can try to add you to the Docker Hub schemers organization. (scheme was already taken.) We have a free plan; I'm not sure how many users it permits, but I've been inactive.

(See also the work-in-progress generator. It's not used to generate the current Dockerfiles, but would really help with the maintenance burden.)

Thank you for using the containers. Upgrading to bookworm is fine by me.
It's convenient that stable-slim exists, but for reproducibility it's better to specify the Debian version explicitly for each Dockerfile. A few containers use even older versions of Debian since those Scheme implementations are so old. We have alpine versions of some containers, too.

I understand, I would do the same at $dayjob too and choose the debian:bookworm-slim so when change happens is controllable.

If you have an interest in maintaining the containers, I can try to add you to the Docker Hub schemers organization. (scheme was already taken.) We have a free plan; I'm not sure how many users it permits, but I've been inactive.

I think we can manage with pull requests too?
What I would like to do is first make a script that builds all the containers, to easily test things. Then I would locally switch to debian:bookworm-slim on all Dockerfiles and see what happens when they are built. Then I'd like to fix everything I can, switch those that can not be fixed back to debian:bullseye-slim and then make a pull request.
I'm just asking before I do anything so I know I will not do it all for nothing. :)

(See also the work-in-progress generator. It's not used to generate the current Dockerfiles, but would really help with the maintenance burden.)

Could you tell me what the generator is supposed to do and what are the pain points in the maintenance?

Closed this as I have similar but better idea. Will make new issue to discuss it.

The new issue is 50.

The new issue is #50.


[BUG?] Too many datapoints being hovered/in tooltip (mode: 'x')

If you create a chart with hover-mode 'x'/tooltip-mode 'x' and many values in its datasets (try 500), tooltips will be looking like this:

As you can see in the image more than one datapoint per dataset is being 'hovered' and congruously the tooltip shows all of them.
I would expect to see only the one datapoint per dataset, which is closest to the mouse cursor, (preferred method) OR the API to give me a convenient way to limit the number of datapoints per dataset.
Situation can be improved by reducing the hoverRadius. But this also decreases usability. Is the 'filter' callback perhaps an option here? The 'index' mode is not an option in this scenario, because values can be missing from time to time in one dataset or the other.
Thank you for your efforts!

@mwoco do you have a fiddle that reproduces this?
There is a filter on the tooltip that could be used.

https://codepen.io/anon/pen/rJePWV?editors=1111

Anyone working on this? Do you mind if I try to fix it?

@MPierre9 you're welcome to have a look at it. I would look at the code in https://github.com/chartjs/Chart.js/blob/master/src/core/core.interaction.js

Thanks for the tip @etimberg, it really helped!

I realize that there is a PR in the pipeline currently that will allow for the tooltip filter to handle this a bit better, but I needed a solution ASAP and I figured I'd share what I did in case anyone is looking for something quick and dirty until this gets merged #7324 gets merged in.
WARNING: I'm accessing private variables of the chart, so this really only should be used as a temporary measure until the PR is merged in.  If this blows up in production, you were warned.
first, I've created a reference the the Chart instance in a scope that is accessible to the chart configuration.  I added the following tooltip configuration with filter function.  It's probably best to check first if there are any active tooltips before grabbing the index.
tooltips: {           mode: 'x',           intersect: false,           filter(item: Chart.ChartTooltipItem, data: Chart.ChartData) {             const currentIndex = lineChart.tooltip._active[0]._index;             return item.index === currentIndex;           }         },
Again, both _active and _index are private variables so they may change at any time.

So part of this is how mode: 'x' works. It finds all the things that pass a hit-test check. If multiple points overlap, they will all be returned. A point is considered in range if the distance(point.center, mouseEvent) < point.radius + point.hitRadius hence why multiple points are found here.
The filter work was done in #7556 and there is a better mode to use to achieve this behaviour. If the mode is set to 'nearest' and then the axis is set to 'x', the single nearest item along the X axis from the event position will be found. See https://codepen.io/etimberg/pen/bGepzqy
Between the filter and the better mode, I am going to call this resolved.

When I use mode: 'nearest' with axis: 'x', it searches for the nearest item only along the X-axis, as you say. With this setting, it's very hard to select a peek in a small chart with a huge number of data rendered without actual points.

Is there any way to first search for the nearest point along both axes and then display all points on the same X-axis?
mode: 'index' is almost doing it right, but when datasets are not x-aligned by index, it's not behaving how I require.


SSE test failures on x86 32-bit

I was working on running wip/meson on Travis, and when I was testing locally I added -m32 to the cflags.  After a couple tweaks (52bc6bd3e37e95c9f3ea735a3357b1f9243d1ac7) it's running, but a bunch of tests fail;

[ ] mm_cvt_pi2ps
[ ] mm_cvt_ps2pi
[ ] mm_cvtpi16_ps
[ ] mm_cvtpi8_ps
[ ] mm_cvtps_pi16
[ ] mm_cvtps_pi32
[ ] mm_cvtps_pi8
[ ] mm_cvtpu16_ps
[ ] mm_cvtpu8_ps
[ ] mm_cvtt_ps2pi

I haven't looked at all of them, but it seems like the bugs are really in the tests not SIMDe.

These are all working now.


Fix uncofigure container connectivity after restart

This PR fixes removing of config related to container connectivity (tap, arp, route...) after vswitch restart.
TL;DR
Contiv-vswitch uses CompositeKVProtoWatcher where proxied etcd and local client are grouped, as datasource for defaultplugins (see contiv_flavor.go). Each of these sources, etcd and local client, has its own PrevRevision instance. PrevRevision, as the name suggests, is in-memory structure providing previous value when a change is about to be applied. Retrieved previous value is attached to request and forwarded to subscribers.
E.g: LocalClient sends a request to add an interface. Previous config of the interface is looked up in prevRevision, attached to the request and forwarded to defaultplugins (subscriber).
In contiv-vswitch the config is applied using local client - change notifications are forwarded to subscribers. Afterward, the config is persisted into etcd. Change events generated by etcd are filtered out in order to avoid duplicated application.
Usually PrevRevision of local client and etcd are in sync and the described approach works as expected. However, after restart the items that have been configured in previous run are not present in local client's PrevRevisions. In this scenario it is desired to forward change notifications generated by etcd.

Coverage increased (+0.2%) to 75.211% when pulling 57eb2ab5d6668f5dedcca06d6a65c805dea63bfa on lukasmacko:master into b3f00837269a4d95796fa81ebe9e9d2ef26f4a03 on contiv:master.


Incorrect test goal - "Only use jQuery to add these classes to the element."

Challenge Waypoint: Target a Specific Child of an Element Using jQuery
The waypoint works correctly, however one of the test goals is You should only use jQuery to add these classes to the element.

jQuery Playground

#left-well

        <button class="btn btn-default target" id="target1">#target1</button>
        <button class="btn btn-default target" id="target2">#target2</button>
        <button class="btn btn-default target" id="target3">#target3</button>

#right-well

        <button class="btn btn-default target" id="target4">#target4</button>
        <button class="btn btn-default target" id="target5">#target5</button>
        <button class="btn btn-default target" id="target6">#target6</button>

Can you add more details on what the issue is here?

There my code is working correctly, i am using jquery to add the classes into the element, but after testing it's wrong

Thank you for reporting this issue.
This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "Get Help" button on the challenge on freeCodeCamp and choose the "Ask for help" option, which will help you create a question in the right part of the forum. Volunteers on the forum usually respond to questions within a few hours and can help determine if there is an issue with your code or the challenge's tests.
If the forum members determine there is nothing wrong with your code, you can request this issue to be reopened.
Thank you and happy coding.


Include external CSS in SWC lessons

These lessons reference CSS outside of their own directories within the
website, so require spanning across hosts. Thi is limited to the lesson
site to avoid major expansions of the volume of data stored.
This is only done for Software Carpentry because it is not needed for
Data Carpentry and because DC lessons are posted to the organizations
main domain adding -H increases storage by 10x.
Co-authored-by: Virnaliz Cruz<EMAIL_ADDRESS>

@garezana - check that this is working for you as well and then merge and let the other folks know on Slack that this is fixed.

I also just added some improvements to the lesson output so it's not just a giant wall of streaming wget information


stop handling uncaught exceptions

Addresses #3822.
This used to be necessary to work around the lack of robust mutexes, causing a write lock to be held indefinitely. This is no longer useful.
/cc @tgoyne @bdash

👍

👍
I can't speak to whether or not this isn't actually being used for any other purpose..


Command 'chainsaw test' always picks up the file 'chainsaw-test.yaml'

Description
I am playing with your quickstart (nice tool by the way) and noticed that chainsaw test --test-dir . is only picking up  the test file when its name matches chainsaw-test.yaml. Is this expected or am I missing something here?
To Reproduce
$ chainsaw test --test-dir .
No configuration provided but found default file: .chainsaw.yaml
Loading config (.chainsaw.yaml)...
- TestDirs [.]
- SkipDelete false
- FailFast true
- ReportFormat ''
- ReportName 'chainsaw-report'
- Namespace ''
- FullName false
- IncludeTestRegex ''
- ExcludeTestRegex ''
- Parallel 4
- ApplyTimeout 45s
- AssertTimeout 20s
- ErrorTimeout 10s
- DeleteTimeout 25s
- CleanupTimeout 45s
- ExecTimeout 45s
Loading tests...
- quick-start (.)
Running tests...
=== RUN   chainsaw
=== RUN   chainsaw/quick-start
=== PAUSE chainsaw/quick-start
=== CONT  chainsaw/quick-start
    15:28:10 | quick-start | @setup | CREATE | v1/Namespace | chainsaw-magnetic-mastodon | OK
    15:28:10 | quick-start | step-1 | APPLY  | v1/ConfigMap | chainsaw-magnetic-mastodon/chainsaw-quick-start | RUNNING...
    15:28:10 | quick-start | step-1 | CREATE | v1/ConfigMap | chainsaw-magnetic-mastodon/chainsaw-quick-start | OK
    15:28:10 | quick-start | step-1 | APPLY  | v1/ConfigMap | chainsaw-magnetic-mastodon/chainsaw-quick-start | DONE
    15:28:10 | quick-start | step-2 | ASSERT | v1/ConfigMap | chainsaw-magnetic-mastodon/chainsaw-quick-start | RUNNING...
    15:28:10 | quick-start | step-2 | ASSERT | v1/ConfigMap | chainsaw-magnetic-mastodon/chainsaw-quick-start | DONE
    15:28:10 | quick-start | step-1 | DELETE | v1/ConfigMap | chainsaw-magnetic-mastodon/chainsaw-quick-start | RUNNING...
    15:28:11 | quick-start | step-1 | DELETE | v1/ConfigMap | chainsaw-magnetic-mastodon/chainsaw-quick-start | OK
    15:28:11 | quick-start | step-1 | DELETE | v1/ConfigMap | chainsaw-magnetic-mastodon/chainsaw-quick-start | DONE
    15:28:11 | quick-start | @setup | DELETE | v1/Namespace | chainsaw-magnetic-mastodon | RUNNING...
    15:28:11 | quick-start | @setup | DELETE | v1/Namespace | chainsaw-magnetic-mastodon | OK
    15:28:16 | quick-start | @setup | DELETE | v1/Namespace | chainsaw-magnetic-mastodon | DONE
--- PASS: chainsaw (0.00s)
    --- PASS: chainsaw/quick-start (5.26s)
PASS
Tests Summary...
- Passed  tests 1
- Failed  tests 0
- Skipped tests 0
Done.

$ mv chainsaw-test.yaml chainSaw-test.yaml

$ chainsaw test --test-dir .
No configuration provided but found default file: .chainsaw.yaml
Loading config (.chainsaw.yaml)...
- TestDirs [.]
- SkipDelete false
- FailFast true
- ReportFormat ''
- ReportName 'chainsaw-report'
- Namespace ''
- FullName false
- IncludeTestRegex ''
- ExcludeTestRegex ''
- Parallel 4
- ApplyTimeout 45s
- AssertTimeout 20s
- ErrorTimeout 10s
- DeleteTimeout 25s
- CleanupTimeout 45s
- ExecTimeout 45s
Loading tests...
Running tests...
Tests Summary...
- Passed  tests 0
- Failed  tests 0
- Skipped tests 0
Done.

Yes this is expected.
https://github.com/kyverno/chainsaw/blob/main/pkg/commands/test/command.go#L175

Oh, I see, actually hard-coded.
May I suggest adding a short comment about the enforced usage of this specific filename at the beginning of the Test based syntax page until you consider adding a command line flag for this purpose?
In fact, a newcomer who is trying to reproduce you examples in a row, may get confused by a well defined file naming convention procedure in the Manifest based syntax and the TestStep based syntax sections, and then the lack of guidance for the name of the Test based syntax manifest to use.

@fab7
Maybe add a Note

You mean a PR?

If you want to add it would be great 👍

@shubham-cmyk Thx for the hint; I did not initially dare modifying your documentation.
Thanks for reporting and the docs PR.


⚠ Roblox Devforum has degraded performance

In cc99dbe, Roblox Devforum (https://devforum.roblox.com) experienced degraded performance:

HTTP code: 403
Response time: 1599 ms

Resolved: Roblox Devforum performance has improved in d2b4f84 after 19 minutes.


[ZF-11819] Zend_Filter_Input

Fixed case where PHP Notice emitted when validator metacommand value is empty array

Reviewed, merged, and pushed to master.


config.json: Add "functional programming" topic

Added to accumulate and strain, will keep an eye out for more
applicable exercises.

Yeah, wow, didn't realize the opinions on it were so strong!

I'm going to close this without merging & think on it a bit more.


extend registry to register site stats handlers for plugins

Summary
Extend plugin registry to offer plugins the ability to add stats on the system console > site statistics.
New methods has been added to registry:

registerSiteStatisticsHandler: register a fetcherFunc to collect stats from a plugin

The main reason why this change is created is to allow plugins to push some stats but handle the data by themselves.
More details on the tech proposal
Ticket Link
MM-42516 epic
Related Pull Requests
Playbooks https://github.com/mattermost/mattermost-plugin-playbooks/pull/1128

before
after

Release Note
NONE

Hello @trilopin,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

@mickmister Thank you for your comments.
I tried to avoid redux initially because handlers are not serializable and didn't want that complexity. After your comments and once I looked at the rest of the plugin store I realize that it's better to keep all plugin state together and avoid plugin state split.

@trilopin We usually want to have this QA tested with a minimal example in the https://github.com/mattermost/mattermost-plugin-demo repo before merging. This is for general testing, and for documenting how a plugin can implement this API usage. Can you please provide an example in the demo plugin, and ping @DHaussermann on your PR?


Any plan to include non seasonal TV anime?

Examples of anime content that appears to be not indexed into the project's database now:

Anime promotional materials that are not part of full seasonal anime:
Examples: Madoka Magica Concept Movie, Prism Nana Pilot PV, Kemono Friends JRA/Nissin collaboration PV
Short/Flash/Doujin/Unofficial/Official online(niconico/youtube)/blu-ray special/other release channel animation content
Examples: Touhou Musou Kakyou, "Haruhi-chan", "Index-tan", Flash anime version of "Nyaruko", "Kosys", Kataribe Shoujo Honoma
Music Video in form of and other form of anime produced not independently
Examples: Music Video of Love Live and 22/7 and such, Yuuki Yuuna Pachislot special footage, Kemono Friends Animelo Summer Live 2017 Special Movie
Non-Japanese anime
Some animation products from Korea or China are also commonly described as anime despite the accuracy of such description would be up for debate.

Anime promotional materials that are not part of full seasonal anime:
Since most PV usually appears with the anime name somewhere, so that was not included in database, but I do have plans to include PV from youtube
Short/Flash/Doujin/Unofficial/Official online(niconico/youtube)/blu-ray special/other release channel animation content
I think these should also be included, but since the source varies and is hard to collect and organize, it takes time to include all. Currently my priority is TV>Bluray/DVD>SPs>ONAs>PV>others but priority is usually given to popularity.
Music Video in form of and other form of anime produced not independently
I'd prefer to exclude Events / Live from database, since it isn't really useful for most people. But Anime Music MV like "Shelter" would be indexed
Non-Japanese anime
basically it depends on whether AniDB / Anilist / MAL recognized those as anime. As long as there are entries in these database, it'll be indexed to database, sooner or later.


Chat and menu fixes

Fix text input focus/unfocus conditions
Hide message dates within non-fullscreen chat console
Better handle ESC key (use back button in active modal or close, unfocus chat input)

If I press / on the hub it opens the chat but does not input /

@XLegland Thanks, good catch, I fixed it!


Update footer section in the 'Read a shloka' page

The footer section of the page is experiencing a spacing issue, with inadequate spacing between the elements. This affects the overall visual appearance and user experience of the website.
please assign me this issue! i would be able to fix this!

some are working on this


Make Choreograph Header-Only

Consider making the library header-only so user can reliably specify things like the Time type through a macro.
Having any files of Choreograph that are compiled prevents the user guaranteeing that they can specify the time type before its first inclusion.

Started in the header-only branch. Not likely to pursue much further at this point.

An alternative would be something like what Entityx does with its config.h.in file. This would require more of users in terms on configuration, but the burden is still pretty light.


ROI feature not work well on QSV

computer

intel I5-6500
intel media sdk 2018-R2
use hw hevc and hw h264

STATE

encoder
bitrate mode
state

h264
CQP
work well

h264
CBR
not support (-3)

hevc
CQP
support , but roi not work

hevc
CBR
support , but roi not work

@saosipov , @ilapichev , please respond

Hi lion117
As I remember SKL supports ROI for BRC only and user can set Priority for these regions.
Could you please share here your command line along with output log?
-Sergey

I used official demo  sample_encode and modify the code to test the roi featrure .
Here is My Code
 memset(&m_cfgROISetting, 0, sizeof(m_cfgROISetting));
            m_cfgROISetting.Header.BufferId = MFX_EXTBUFF_ENCODER_ROI;;
            m_cfgROISetting.Header.BufferSz = sizeof(m_cfgROISetting);

            m_cfgROISetting.NumROI = 1;
            m_cfgROISetting.ROIMode = MFX_ROI_MODE_PRIORITY;
            m_cfgROISetting.ROI[0].Left = 160;
            m_cfgROISetting.ROI[0].Right = 640;
            m_cfgROISetting.ROI[0].Top = 0;
            m_cfgROISetting.ROI[0].Bottom = 320;
            m_cfgROISetting.ROI[0].DeltaQP = -51;
            //m_cfgROISetting.ROI[0].Priority = 3;
            m_EncExtParams.push_back((mfxExtBuffer *)&m_cfgROISetting);

Here Is the Roi working video shootscreen only on H264CPQ ratecontrol
downlaod

download

Hi lion117
As I remember SKL supports ROI for BRC only and user can set Priority for these regions.
Could you please share here your command line along with output log?
-Sergey

Hi, Do you have any progress?

Hi lion117,
Thank you for using msdk.
There is a conflict in your code: you set MFX_ROI_MODE_PRIORITY together with DeltaQP value -51. Values for priority are limited to [-3,3] and so initialization fails with INVALID_PARAM.
Note, that DeltaQP and Priority fields make union.
Please set MFX_ROI_MODE_QP_DELTA instead.
BTW, "Priority is deprecated mode and is used only for backward compatibility".
All the best,
-lakulako

Hi lion117,
Thank you for using msdk.
There is a conflict in your code: you set MFX_ROI_MODE_PRIORITY together with DeltaQP value -51. Values for priority are limited to [-3,3] and so initialization fails with INVALID_PARAM.
Note, that DeltaQP and Priority fields make union.
Please set MFX_ROI_MODE_QP_DELTA instead.
BTW, "Priority is deprecated mode and is used only for backward compatibility".
All the best,
-lakulako

Hi, I had tried two different cases which both return  invalid para  when I used cbr bitrate control mode .
case 1
memset(&m_cfgROISetting, 0, sizeof(m_cfgROISetting));
            m_cfgROISetting.Header.BufferId = MFX_EXTBUFF_ENCODER_ROI;;
            m_cfgROISetting.Header.BufferSz = sizeof(m_cfgROISetting);

            m_cfgROISetting.NumROI = 1;
            m_cfgROISetting.ROIMode = MFX_ROI_MODE_PRIORITY;
            m_cfgROISetting.ROI[0].Left = 160;
            m_cfgROISetting.ROI[0].Right = 640;
            m_cfgROISetting.ROI[0].Top = 0;
            m_cfgROISetting.ROI[0].Bottom = 320;
            m_cfgROISetting.ROI[0].Priority = 3;
            m_EncExtParams.push_back((mfxExtBuffer *)&m_cfgROISetting);

case 2
memset(&m_cfgROISetting, 0, sizeof(m_cfgROISetting));
            m_cfgROISetting.Header.BufferId = MFX_EXTBUFF_ENCODER_ROI;;
            m_cfgROISetting.Header.BufferSz = sizeof(m_cfgROISetting);

            m_cfgROISetting.NumROI = 1;
            m_cfgROISetting.ROIMode = MFX_ROI_MODE_QP_DELTA ;
            m_cfgROISetting.ROI[0].Left = 160;
            m_cfgROISetting.ROI[0].Right = 640;
            m_cfgROISetting.ROI[0].Top = 0;
            m_cfgROISetting.ROI[0].Bottom = 320;
            m_cfgROISetting.ROI[0].DeltaQP = -51;
            m_EncExtParams.push_back((mfxExtBuffer *)&m_cfgROISetting);

Hi, hard to understand what is going on in your case without information.
Could you please provide command line, logs and binaries' version?

Hi, hard to understand what is going on in your case without information.
Could you please provide command line, logs and binaries' version?

intel I5-6500
intel media sdk 2018-R2 and 2019-R1
use hw hevc and hw h264

I used the Encode_sample code to test ROI feature.  Test video Roi  by elecard streameye demo .

And what is the command line? Logs?
(for sample_encode you can use -dump <dump_file_name> to dump configuration after init.

And what is the command line? Logs?
(for sample_encode you can use -dump <dump_file_name> to dump configuration after init.

I modify Sample_encode code to make roi feature work . not by cmdline . so roi region is fixed.
there were  no Logs or dumps
# h265
sample_encode.exe h265 -i  singer.yuv -o singer_out_265_1200.h265 -w 960 -h 720  -cbr -b 1200  -f 15  -g  30

# h264
sample_encode.exe h264 -i  singer.yuv -o singer_out_264_1200.h264 -w 960 -h 720  -cbr -b 1200  -f 15  -g  30

I can try with sample_encode. It is more informative. Lets continue with it.
Does sample_encode prints anything to screen? It is log that I asked.
Please try adding "-dump fname" and share fname.

@lion117 ,  have you resolved the issue?

No
------------------ 原始邮件 ------------------
发件人: "Leonid<EMAIL_ADDRESS>发送时间: 2020年2月27日(星期四) 晚上8:24
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS><EMAIL_ADDRESS>主题: Re: [Intel-Media-SDK/MediaSDK] ROI feature not work well on QSV (#1904)
@lion117 ,  have you resolved the issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Let's try with sample_encode. It is more informative. Lets continue with it.
Does sample_encode prints anything to screen? It is log that I asked.
Please try adding "-dump fname" and share fname.

Couldn't U Reproduce the ROI Issue?  was everything working very well when U tested ?

ROI works well on my side, but with different application. Sample encode doesn't support ROI. I will not try sample_encode application with your modifications. I could look into logs that you don't share.

I would like to provide  my  roi verion  sample_encode source code for you .

Yes, this is what I asked. Better provide this as patch to msdk sample_encode.

hi, I uploaded the source code into my cloud bucket.
src code
yuv data
roi core code . which u you search in src code
        memset(&m_cfgROISetting, 0, sizeof(m_cfgROISetting));
         m_cfgROISetting.Header.BufferId = MFX_EXTBUFF_ENCODER_ROI;;
        m_cfgROISetting.Header.BufferSz = sizeof(m_cfgROISetting);

        m_cfgROISetting.NumROI = 1;
        m_cfgROISetting.ROIMode = MFX_ROI_MODE_PRIORITY;
        m_cfgROISetting.ROI[0].Left = 160;
        m_cfgROISetting.ROI[0].Right = 640;
        m_cfgROISetting.ROI[0].Top = 160;
        m_cfgROISetting.ROI[0].Bottom = 320;
        m_cfgROISetting.ROI[0].DeltaQP = -51;

Any question about the source code , please let know

hi , are there any I can help ?

Hi @lion117 , I couldn't build your solution, so I will add your changes to our sample_encode and investigate your issue.

Hi @lion117 , I couldn't build your solution, so I will add your changes to our sample_encode and investigate your issue.

HI, I rebuilded  RunIntelDemo.sln which  builded successfully.

I use VS 2017.  i5-6300(SKL)
I added your code:
        std::memset(&m_cfgROISetting, 0, sizeof(m_cfgROISetting));
        m_cfgROISetting.Header.BufferId = MFX_EXTBUFF_ENCODER_ROI;
        m_cfgROISetting.Header.BufferSz = sizeof(m_cfgROISetting);

        m_cfgROISetting.NumROI = 1;
        m_cfgROISetting.ROIMode = MFX_ROI_MODE_QP_DELTA;
        m_cfgROISetting.ROI[0].Left = 160;
        m_cfgROISetting.ROI[0].Right = 640;
        m_cfgROISetting.ROI[0].Top = 160;
        m_cfgROISetting.ROI[0].Bottom = 320;
        m_cfgROISetting.ROI[0].DeltaQP = -51;

        m_EncExtParams.push_back((mfxExtBuffer *)&m_cfgROISetting);

in CEncodingPipeline::InitMfxEncParams sample_decode from MediaSDK.
AVC
it works fine for -cqp and -cbr:
sample_encode.exe h264 -i 2645142_960x720.yuv -o test.h264 -w 960 -h 720  -cqp -qpi 30 -qpb 32 -qpp 32 -f 15 -g 30
sample_encode.exe h264 -i 2645142_960x720.yuv -o test.h264 -w 960 -h 720  -cbr -b 1200 -f 15 -g 30

HEVC
Windows SKL doesn't support ROI for HEVC.
I run this cmd on ICL - and it works.
sample_encode.exe h265 -i 2645142_960x720.yuv -o test.h265 -w 960 -h 720  -cqp -qpi 30 -qpb 32 -qpp 32 -f 15 -g 30
sample_encode.exe h265 -i 2645142_960x720.yuv -o test.h265 -w 960 -h 720  -cbr -b 1200 -f 15 -g 30

For additional information:
Linux SKL supports ROI for both of codecs.
I hope it helps you.

Thanks  , I will take a try for the AVC operation.
By the way , I have a question about how do you run the cmd on ICL.

@lion117, ICL is next generation of CPU , for example I tested on i7-1060G7.
cmd was the same.

I use VS 2017. i5-6300(SKL)
I added your code:
        std::memset(&m_cfgROISetting, 0, sizeof(m_cfgROISetting));
        m_cfgROISetting.Header.BufferId = MFX_EXTBUFF_ENCODER_ROI;
        m_cfgROISetting.Header.BufferSz = sizeof(m_cfgROISetting);

        m_cfgROISetting.NumROI = 1;
        m_cfgROISetting.ROIMode = MFX_ROI_MODE_QP_DELTA;
        m_cfgROISetting.ROI[0].Left = 160;
        m_cfgROISetting.ROI[0].Right = 640;
        m_cfgROISetting.ROI[0].Top = 160;
        m_cfgROISetting.ROI[0].Bottom = 320;
        m_cfgROISetting.ROI[0].DeltaQP = -51;

        m_EncExtParams.push_back((mfxExtBuffer *)&m_cfgROISetting);

in CEncodingPipeline::InitMfxEncParams sample_decode from MediaSDK.
AVC
it works fine for -cqp and -cbr:
sample_encode.exe h264 -i 2645142_960x720.yuv -o test.h264 -w 960 -h 720  -cqp -qpi 30 -qpb 32 -qpp 32 -f 15 -g 30
sample_encode.exe h264 -i 2645142_960x720.yuv -o test.h264 -w 960 -h 720  -cbr -b 1200 -f 15 -g 30

HEVC
Windows SKL doesn't support ROI for HEVC.
I run this cmd on ICL - and it works.
sample_encode.exe h265 -i 2645142_960x720.yuv -o test.h265 -w 960 -h 720  -cqp -qpi 30 -qpb 32 -qpp 32 -f 15 -g 30
sample_encode.exe h265 -i 2645142_960x720.yuv -o test.h265 -w 960 -h 720  -cbr -b 1200 -f 15 -g 30

For additional information:
Linux SKL supports ROI for both of codecs.
I hope it helps you.

HI, I had tried the method which you mentioned. But It still didnot work.
I Add the Roi  code in the sample_encode demo on the CEncodingPipeline::InitMfxEncParams function.
Could you provide the sample code And the sample_encode.exe for me  ?

@lion117, ICL is next generation of CPU , for example I tested on i7-1060G7.
cmd was the same.

Hi, Are you still on call ?

Hi @lion117 Sorry for delay, I will prepare sample code which should works today

Hi @lion117 Sorry for delay, I will prepare sample code which should works today

please send the sample code to my email<EMAIL_ADDRESS>

@lion117 I noticed that source code was changed and my previous code needs to correct. I need more time to understand what was changed and test ROI again.

@lion117 Could please download last version of MediaSDK.
And add next ROI code to pipeline_encode.cpp:
        auto roi = m_mfxEncParams.AddExtBuffer<mfxExtEncoderROI>();
        roi->Header.BufferId = MFX_EXTBUFF_ENCODER_ROI;
        roi->Header.BufferSz = sizeof(roi);

        roi->NumROI = 1;
        roi->ROIMode = MFX_ROI_MODE_QP_DELTA;
        roi->ROI[0].Left = 160;
        roi->ROI[0].Right = 640;
        roi->ROI[0].Top = 160;
        roi->ROI[0].Bottom = 320;
        roi->ROI[0].DeltaQP = -51;

Also add changes to sample_utils.h to struct  mfx_ext_buffer_id:
template<>struct mfx_ext_buffer_id<mfxExtEncoderROI>{
    enum {id = MFX_EXTBUFF_ENCODER_ROI};
};

It works for AVC -cbr

@lion117 Could please download last version of MediaSDK.
And add next ROI code to pipeline_encode.cpp:
        auto roi = m_mfxEncParams.AddExtBuffer<mfxExtEncoderROI>();
        roi->Header.BufferId = MFX_EXTBUFF_ENCODER_ROI;
        roi->Header.BufferSz = sizeof(roi);

        roi->NumROI = 1;
        roi->ROIMode = MFX_ROI_MODE_QP_DELTA;
        roi->ROI[0].Left = 160;
        roi->ROI[0].Right = 640;
        roi->ROI[0].Top = 160;
        roi->ROI[0].Bottom = 320;
        roi->ROI[0].DeltaQP = -51;

Also add changes to sample_utils.h to struct mfx_ext_buffer_id:
template<>struct mfx_ext_buffer_id<mfxExtEncoderROI>{
    enum {id = MFX_EXTBUFF_ENCODER_ROI};
};

It works for AVC -cbr

I had tried IntelMediaSdk 2018 R2 and IntelMediaSdk 2019R1 . This solution was still not work .
Could u provide the sample code or exe which could work?
I think that would be more efficient

Please try this exe file
sample_encode.zip

Please try this exe file
sample_encode.zip

I  had tested  the sample_encode.exe on windows 7  that you provided  which still not worked.
Are there any solution that could make the qsv roi work on  windows without relaying on any specified environment ?

Do you have the latest video driver installed on your machine?

what kind of video driver ?

yes , I had installed the latest graphic driver

I've checked intel.com and found more recent version -> https://downloadcenter.intel.com/download/29475/Intel-Graphics-Driver-for-Windows-15-45-?wapkw=intel hd graphics
Please try to install and report results with new driver.

I think graphic card driver is not  an issue. Even I updgraded it to the latest version . It was still not working .

Maybe you should test your code on a new computer.

I've tested code on new computer and it works too.

@lion117 Could you please install Window 10, 64 bits? And try to run sample_encode on new OS.
Also Microsoft's support for Windows 7 ended on January 14, 2020, I suggest to move to Win 10 anyway.

I will

@lion117 Could you please install Window 10, 64 bits? And try to run sample_encode on new OS.
Also Microsoft's support for Windows 7 ended on January 14, 2020, I suggest to move to Win 10 anyway.

I had tried the sample_encode.exe on win10 which cpu was  intel I7-9700. The result was still  -3 (not supported).

I've tested code on new computer and it works too.

Had the roi feature effected on the video ?
Are there any solution to make the sample_ecode.exe work on more computers?

Hi @lion117 Could please rebuild libmfx_vs2015.lib with lasted changes from MediaSDK repo and send me your caps

@lion117 Could you please install Window 10, 64 bits? And try to run sample_encode on new OS.
Also Microsoft's support for Windows 7 ended on January 14, 2020, I suggest to move to Win 10 anyway.

I had tried the sample_encode.exe on win10 which cpu was intel I7-9700. The result was still -3 (not supported).

Have you updated driver on new machine?

Also could you please put breakpoint to Query, and find line (from lib) which return error. Maybe problem is not in ROI.

Could you please rebuild libmfx_vs2015.lib with lasted changes from MediaSDK repo and send me your caps
I don't think the unsupported error has relationship with libmfx_vs2015.lib , because I used the sample_encode.exe which had link the latest libmfx.lib .

Also could you please put breakpoint to Query, and find line (from lib) which return error. Maybe problem is not in ROI.

I  had tried to prohibit the roi code . then it worked.
if roi  was enabled , when call  query api, it return  -3 error

Please check Query parameters and compare src, dst

Would I  provide the query Api stack info be helpful to analyse the issue?

@lion117 maybe, please provide and we'll look at this.

Sorry for I was late
each time When I initialized encoder, this code would return -3 error

I debuged GetFirstEncoder()->Query(&m_mfxEncParams, &m_mfxEncParams) Api
before run the api , the m_mfxEncParams stack was

after

@SofyaBalandina can you continue the investigation?

Hi @daleksan, @egorovdanil will continue investigation this issue.

Hi, @lion117 , we have a sample that is support of roi - sample_multi_transcode. Maybe try to encode with it ? Will there be an error in the output?


Preset play list buttons

Expected behaviour
The value for number of clip names to reserve is used in the preset buttons

Actual behaviour
Only 5 preset play list buttons are displayed

System
Build version
2.2.0 (2.2.0-56056c0b-3505)
Operating System
Windows 10
Browser
Chrome

'Reserved' clip names is to prevent $NA from appearing on the button when a playlist does not have a particular item #. They are not related to presets. The presets are demonstration/example buttons to better document how some actions may be used.

imho if you have 10 'reserved' then there should be 10 presets as this documents and provides examples to better document hos they can be used.

The only difference between playlist presets is the Clip ID.  Extra presets do not add any useful information and simply take up more memory. You can copy preset 1 and edit the Clip number for any over 5.


Fixes a bug where the nexthop markNextHop method to mark a host down is not called.

In HttpTransact::handle_response_from_parent() the markParentDown() wrapper was not used.  This will mean that the wrong method will be called, ParentSelection::markParentDown() vs the NextHopSelectionStrategy::markNextHop() resulting in a crash when next hop strategies are used.
This should be back ported to ATS 9.x.x branch and releases.

[approve ci RAT]

Cherry-picked to v9.0.x branch.


IndexError: index 3098 is out of bounds for axis 0 with size 3098 on Jupyter

Current Behaviour
I have a huge data frame(900k rows) and since i received IndexErrors multiple times on the 3.1.0 version, I tried 3 things based on all the similar issues

Imputed all nans to median , so there are no nulls or nans now
Created small random sample of the data frame only taking 5% data df.sample(frac=0.05, random_state=3407)
Upgraded to 3.2.0

My pandas version is 1.3.4
This is the error I get when i run profile = ProfileReport(sampleData)(which executes quickly) and then try to view with profile.to_notebook_iframe() (which runs stages and then errors out):
lib/python3.7/site-packages/numpy/lib/histograms.py in histogram(a, bins, range, normed, weights, density)
    854             # The index computation is not guaranteed to give exactly
    855             # consistent results within ~1 ULP of the bin edges.
--> 856             decrement = tmp_a < bin_edges[indices]
    857             indices[decrement] -= 1
    858             # The last bin includes the right edge. The other bins do not.

IndexError: index 3098 is out of bounds for axis 0 with size 3098

Expected Behaviour
profile.to_notebook_iframe() should give me the profile report of the dataframe
Data Description
Dataset has 984289 rows × 118 columns,  mostly float16(20), int16(44), int8(51), object(3) and memory usage: 190.6+ MB
Code that reproduces the bug
No response
pandas-profiling version
3.2.0
Dependencies
pandas==1.3.4
numpy==1.20.3

OS
Mac OS BigSur 11.6.7
Checklist

[X] There is not yet another bug report for this issue in the issue tracker
[X] The problem is reproducible from this bug report. This guide can help to craft a minimal bug report.
[X] The issue has not been resolved by the entries listed under Common Issues.

Update :

It seemed to be failing only on those columns that had their nans imputed. Removing those columns , made the profile report go further and didn't give the index error.
Upon further inspection , I noticed those columns were cast to float16 by a reduce memory utility function.
There is probably an overflow issue here at work which is why the numbers in the IndexError sometimes go crazy(as seen in some other bug reports and issues) and even after reducing the size of the dataframe, it still gives the error.


Use travis container-based infrastructure

See http://docs.travis-ci.com/user/migrating-from-legacy/

Thanks for the pull requests, @fredj.  The team is at SIGGRAPH this week so we may be slow to respond.
Everyone - we have a corporate CLA from Camptocamp so there's no need for another CLA here.
@mramato can you review and merge?

Looks good, thanks for the link and the tweak.


ZipParameters.clone() does not work

ZipParameters has a public clone() method, but it doesn't work as (since commit 4c50a6a5) the class does not implement Cloneable.
  Exception java.lang.CloneNotSupportedException: net.lingala.zip4j.model.ZipParameters
        at Object.clone (Native Method)
        at ZipParameters.clone (ZipParameters.java:208)

ZipParameters does not implement Cloneable anymore because Cloneable is generally considered to be broken and not implemented correctly in Java. Instead ZipParameters now has constructor that does exactly that - it creates a new instance of ZipParameters from the input zipParameters. Just call new ZipParameters(zipParameters) where zipParameters is your existing instance that you want to clone.
References:
https://blog.jcranky.com/2009/08/26/why-not-use-java-clone-method/
https://howtodoinjava.com/java/cloning/cloneable-interface-is-broken-in-java/

OK; the public clone() override should be removed then as it doesn't work and is just misleading.

You are right. I will take it out.

Fixed in v2.11.0 released today


Multiple routers with v4?

Our navigation is so complex that we are planning to use a router for main navigations and another for navigations in side drawer. (just like #353 and many others but with v4)
We are currently using react-native-drawer instead of react-navigation drawer as described in this comment because the former works well with react-native-scrollable-tabview.
We could demonstrate multiple 'routers' with pure react-navigation when connecting multiple AppNavigator to redux, however when using multiple <Router/> of router-flux v4 it failed with error "no route defined for key [other router scene key]" in getScreenForRouteName.
As it seems in router-flux the AppNavigator is a new navigator created by new NavigationStore instance create(), I guess this is still possible to achieve with appropriate extensions?

Current Actions (navigationStore) is singleton for simplicity - i.e. one instance across ap..  It is not difficult to support multiple navigationStore instances by exporting also NavigationStore class from navigationStore.js and don't use Actions but app-defined classes for it. After that it is easy to modify Router.js to accept navigationStore as optional parameter or create new instance of NavigationStore class each time.
PR is welcome. I don't have time to implement it right now and I would need some demo code anyway to work with.

Thanks for your explanation~
We are currently time restricted and may not have time for extension either, also our case may be a bit rare and single Router with react-navigation built-in drawer is sufficient for most of the cases. However we verified the following workaround for us which is faster:

Upgrade to router-flux v4, keep using it for main navigations and track main nav states in redux by createReducer prop;
Use pure react-navigation Navigator and manage drawer nav states in redux.


Add std_normal_log_qf as prob function

Summary
Adds the quantile function of the standard normal density function with log input. This is an alternative pr to #2731 and puts the files in the prob folder. New folders are created for rev specialties of prob functions in stan -> math -> rev -> prob (and in the fwd folder) and their tests.
Release notes
Adds the quantile function of the standard normal density function with log input.
Checklist

[x] Math issue #2697

[x] Copyright holder: Sean Pinkney
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

This is so strange. The test/unit/math/prim/prob/std_normal_log_qf_test.cpp is failing in CI but it passes locally for me. @SteveBronder or @rok-cesnovar do you have some ideas for what may be the issue?

ok, I think the tests should all pass. Requested reviews from @rok-cesnovar or @SteveBronder.

Thanks for the review @andrjohns! I just had that one question above.

@andrjohns I was able to calculate things on the log scale. Please take a look again.


Basic instructions

There we go, I think I got all your suggestions included.

Also added a link to 2ms RPI MQTT monitor for openmower.

Also added a beatiful dashboard example. Cheated a bit with buttons that are not part of the integration yet. Now I'm done editing. More after you get all the other stuff integrated.

thanks! let's see if people will understand that better.