🐛 Bug Report: Error 500 Kubernetes Plugin

📜 Description
The backend returns a 500 when listing the Kubernetes resources associated with a Component.
👍 Expected behavior
List the Kubernetes resources associated with a Component.
👎 Actual Behavior with Screenshots

👟 Reproduction steps

Go on a Component
Go to the Kubernetes Tab

📃 Provide the context for the Bug.
When opening the Kubernetes tab, the frontend successfully receives the list of clusters on the api/kubernetes/clusters endpoint. The 500 is returned when the resources are looked up on the service endpoint api/kubernetes/services/hello-world
The frontend shows:
Errors: Request failed with 500 , {"error":"Cannot read properties of undefined (reading 'credentials')"}

On the backend, we can see a bit more details:
"level":"error","message":"action=retrieveObjectsByServiceId service=hello-world, error=TypeError: Cannot read properties of undefined (reading 'credentials')","plugin":"kubernetes","service":"backstage"}

With 1.24 using backend authentification by default, it appears the cluster-locator was updated to allow passing the option.credentials parameters on clusterSupplier.getClusters; I guess that the service-locator should have something similar as it currently uses clusterSupplier.getClusters without passing the Backstage Credential which cause the error seen in the backend logs (options is undefined):

MultiTenantServiceLocator
SingleTenantServiceLocator
CatalogRelationServiceLocator

🖥 Your Environment
OS:   Darwin 23.4.0 - darwin/arm64
node: v18.16.0
yarn: 1.22.21
cli:  0.26.1 (installed)
backstage:  1.24.0
Dependencies:
@backstage/app-defaults                                          1.5.2
@backstage/backend-app-api                                       0.5.14, 0.6.1
@backstage/backend-common                                        0.20.2, 0.21.5
@backstage/backend-defaults                                      0.2.15
@backstage/backend-dev-utils                                     0.1.4
@backstage/backend-openapi-utils                                 0.1.8
@backstage/backend-plugin-api                                    0.6.15
@backstage/backend-tasks                                         0.5.20
@backstage/backend-test-utils                                    0.3.5
@backstage/catalog-client                                        1.6.2
@backstage/catalog-model                                         1.4.5
@backstage/cli-common                                            0.1.13
@backstage/cli-node                                              0.2.4
@backstage/cli                                                   0.25.2, 0.26.1
@backstage/config-loader                                         1.7.0
@backstage/config                                                1.2.0
@backstage/core-app-api                                          1.12.2
@backstage/core-compat-api                                       0.2.2
@backstage/core-components                                       0.14.2
@backstage/core-plugin-api                                       1.9.1
@backstage/errors                                                1.2.4
@backstage/eslint-plugin                                         0.1.6
@backstage/frontend-plugin-api                                   0.6.2
@backstage/integration-aws-node                                  0.1.11
@backstage/integration-react                                     1.1.25
@backstage/integration                                           1.9.1
@backstage/plugin-api-docs                                       0.11.2
@backstage/plugin-app-backend                                    0.3.63
@backstage/plugin-app-node                                       0.1.15
@backstage/plugin-auth-backend-module-atlassian-provider         0.1.7
@backstage/plugin-auth-backend-module-aws-alb-provider           0.1.7
@backstage/plugin-auth-backend-module-gcp-iap-provider           0.2.10
@backstage/plugin-auth-backend-module-github-provider            0.1.12
@backstage/plugin-auth-backend-module-gitlab-provider            0.1.12
@backstage/plugin-auth-backend-module-google-provider            0.1.12
@backstage/plugin-auth-backend-module-microsoft-provider         0.1.10
@backstage/plugin-auth-backend-module-oauth2-provider            0.1.12
@backstage/plugin-auth-backend-module-oauth2-proxy-provider      0.1.8
@backstage/plugin-auth-backend-module-oidc-provider              0.1.6
@backstage/plugin-auth-backend-module-okta-provider              0.0.8
@backstage/plugin-auth-backend                                   0.22.2
@backstage/plugin-auth-node                                      0.4.10
@backstage/plugin-auth-react                                     0.0.2
@backstage/plugin-catalog-backend-module-github                  0.5.6
@backstage/plugin-catalog-backend-module-scaffolder-entity-model 0.1.13
@backstage/plugin-catalog-backend                                1.20.0
@backstage/plugin-catalog-common                                 1.0.22
@backstage/plugin-catalog-graph                                  0.4.2
@backstage/plugin-catalog-import                                 0.10.8
@backstage/plugin-catalog-node                                   1.10.0
@backstage/plugin-catalog-react                                  1.11.1
@backstage/plugin-catalog                                        1.18.1
@backstage/plugin-events-node                                    0.3.1
@backstage/plugin-github-actions                                 0.6.13
@backstage/plugin-home-react                                     0.1.10
@backstage/plugin-home                                           0.6.2
@backstage/plugin-kubernetes-common                              0.7.5
@backstage/plugin-kubernetes-node                                0.1.9
@backstage/plugin-kubernetes-react                               0.3.2
@backstage/plugin-kubernetes                                     0.11.7
@backstage/plugin-org                                            0.6.22
@backstage/plugin-permission-backend-module-allow-all-policy     0.1.12
@backstage/plugin-permission-backend                             0.5.39
@backstage/plugin-permission-common                              0.7.13
@backstage/plugin-permission-node                                0.7.26
@backstage/plugin-permission-react                               0.4.21
@backstage/plugin-proxy-backend                                  0.4.13
@backstage/plugin-scaffolder-backend-module-azure                0.1.7
@backstage/plugin-scaffolder-backend-module-bitbucket-cloud      0.1.5
@backstage/plugin-scaffolder-backend-module-bitbucket-server     0.1.5
@backstage/plugin-scaffolder-backend-module-bitbucket            0.2.5
@backstage/plugin-scaffolder-backend-module-gerrit               0.1.7
@backstage/plugin-scaffolder-backend-module-gitea                0.1.5
@backstage/plugin-scaffolder-backend-module-github               0.2.5
@backstage/plugin-scaffolder-backend-module-gitlab               0.3.1
@backstage/plugin-scaffolder-backend                             1.22.2
@backstage/plugin-scaffolder-common                              1.5.1
@backstage/plugin-scaffolder-node                                0.4.1
@backstage/plugin-scaffolder-react                               1.8.2
@backstage/plugin-scaffolder                                     1.19.1
@backstage/plugin-search-backend-module-catalog                  0.1.20
@backstage/plugin-search-backend-module-pg                       0.5.24
@backstage/plugin-search-backend-module-techdocs                 0.1.20
@backstage/plugin-search-backend-node                            1.2.19
@backstage/plugin-search-backend                                 1.5.5
@backstage/plugin-search-common                                  1.2.11
@backstage/plugin-search-react                                   1.7.8
@backstage/plugin-search                                         1.4.8
@backstage/plugin-tech-radar                                     0.7.1
@backstage/plugin-techdocs-backend                               1.10.2
@backstage/plugin-techdocs-module-addons-contrib                 1.1.7
@backstage/plugin-techdocs-node                                  1.12.1
@backstage/plugin-techdocs-react                                 1.2.1
@backstage/plugin-techdocs                                       1.10.2
@backstage/plugin-user-settings                                  0.8.3
@backstage/release-manifests                                     0.0.11
@backstage/test-utils                                            1.5.2
@backstage/theme                                                 0.5.2
@backstage/types                                                 1.1.1
@backstage/version-bridge                                        1.0.7
👀 Have you spent some time to check if this bug has been raised before?

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

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

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

Got a short-term fix for the error itself in #23933, although it's possible this still needs further fixing if credentials aren't being forwarded properly

Right, that's what I tested locally @Rugvip. I was already relying on the authenticated backend, now with Backstage 1.24, it's the default, so it's going to be blocking for people.
It's quite a big change, that's why I didn't propose to submit a PR, do you have any suggestion on who could have a look?

Gonna close as the provided fix seems to be enough, we have now also released v1.25.1 as a patch release with this fix.

@Rugvip It does not solve the issue, changes the error to :
error action=retrieveObjectsByServiceId service=backstage, error=ResponseError: Request failed with 401 Unauthorized

To help narrow it down
https://github.com/backstage/backstage/blob/9d89aed378b3929ef6c01469825b33471498aff6/plugins/kubernetes-backend/src/cluster-locator/CatalogClusterLocator.ts#L55-L104
The 401 is returned from this function, because options is undefined, the ternary pass undefined to this.catalogClient.getEntities()
https://github.com/backstage/backstage/blob/9d89aed378b3929ef6c01469825b33471498aff6/plugins/kubernetes-backend/src/service-locator/MultiTenantServiceLocator.ts#L35-L40
options is undefined because getClustersByEntity() doesn't pass the parameter to this.clusterSupplier.getClusters()
https://github.com/backstage/backstage/blob/3b3d549a5f9fd563493b8c1470bf1f587358aee4/plugins/kubernetes-backend/src/service/KubernetesFanOutHandler.ts#L248-L251
serviceLocator.getClustersByEntity() gets called from fanOutRequests()
https://github.com/backstage/backstage/blob/3b3d549a5f9fd563493b8c1470bf1f587358aee4/plugins/kubernetes-backend/src/service/KubernetesFanOutHandler.ts#L231-L236
fanOutRequests get called from getKubernetesObjectsByEntity()
https://github.com/backstage/backstage/blob/9b1abac3cca3818fa165dc9795ae4942553a3328/plugins/kubernetes-backend/src/service/KubernetesBuilder.ts#L385-L400
getKubernetesObjectsByEntity() is initially called by POST /services/:serviceId'

Thank you :+1:, will have another look

@benjaminr  I've tested this with 1.26.0-next.1 but still getting a 401 error, the error message is:
Errors: Request failed with 401 Unauthorized, {"error":{"name":"ResponseError","message":"Request failed with 401 Unauthorized","response":{"size":0,"timeout":0},"body":{"error":{"name":"AuthenticationError","message":"Unable to call 'permissions' plugin on behalf of user, because the target plugin does not support on-behalf-of tokens","stack":"AuthenticationError: Unable to call 'permissions' plugin on behalf of user, because the target plugin does not support on-behalf-of tokens\n at DefaultAuthService.getPluginRequestToken (/Users/z0058456/backstage/backstage/node_modules/@backstage/backend-app-api/src/services/implementations/auth/authServiceFactory.ts:242:17)\n at process.processTicksAndRejections (node:internal/process/task_queues:95:5)\n at _ServerPermissionClient.authorizeConditional (/Users/z0058456/backstage/backstage/node_modules/@backstage/plugin-permission-node/src/ServerPermissionClient.ts:97:9)\n at AuthorizedEntitiesCatalog.entities (/Users/z0058456/backstage/backstage/node_modules/@backstage/plugin-catalog-backend/src/service/AuthorizedEntitiesCatalog.ts:55:7)\n at <anonymous> (/Users/z0058456/backstage/backstage/node_modules/@backstage/plugin-catalog-backend/src/service/createRouter.ts:132:40)"},"request":{"method":"GET","url":"/entities?filter=kind%3DResource%2Cspec.type%3Dkubernetes-cluster%2Cmetadata.annotations.kubernetes.io%2Fapi-server%2Cmetadata.annotations.kubernetes.io%2Fapi-server-certificate-authority%2Cmetadata.annotations.kubernetes.io%2Fauth-provider"},"response":{"statusCode":401}},"cause":{"name":"AuthenticationError","message":"Unable to call 'permissions' plugin on behalf of user, because the target plugin does not support on-behalf-of tokens","stack":"AuthenticationError: Unable to call 'permissions' plugin on behalf of user, because the target plugin does not support on-behalf-of tokens\n at DefaultAuthService.getPluginRequestToken (/Users/z0058456/backstage/backstage/node_modules/@backstage/backend-app-api/src/services/implementations/auth/authServiceFactory.ts:242:17)\n at process.processTicksAndRejections (node:internal/process/task_queues:95:5)\n at _ServerPermissionClient.authorizeConditional (/Users/z0058456/backstage/backstage/node_modules/@backstage/plugin-permission-node/src/ServerPermissionClient.ts:97:9)\n at AuthorizedEntitiesCatalog.entities (/Users/z0058456/backstage/backstage/node_modules/@backstage/plugin-catalog-backend/src/service/AuthorizedEntitiesCatalog.ts:55:7)\n at <anonymous> (/Users/z0058456/backstage/backstage/node_modules/@backstage/plugin-catalog-backend/src/service/createRouter.ts:132:40)"},"statusCode":401,"statusText":"Unauthorized","stack":"ResponseError: Request failed with 401 Unauthorized\n at Function.fromResponse (/Users/z0058456/backstage/backstage/node_modules/@backstage/errors/src/errors/ResponseError.ts:76:12)\n at process.processTicksAndRejections (node:internal/process/task_queues:95:5)\n at CatalogClient.requestRequired (/Users/z0058456/backstage/backstage/node_modules/@backstage/catalog-client/src/CatalogClient.ts:449:13)\n at CatalogClient.getEntities (/Users/z0058456/backstage/backstage/node_modules/@backstage/catalog-client/src/CatalogClient.ts:130:22)\n at CatalogClusterLocator.getClusters (/Users/z0058456/backstage/backstage/node_modules/@backstage/plugin-kubernetes-backend/src/cluster-locator/CatalogClusterLocator.ts:70:22)\n at async Promise.all (index 0)\n at CombinedClustersSupplier.getClusters (/Users/z0058456/backstage/backstage/node_modules/@backstage/plugin-kubernetes-backend/src/cluster-locator/index.ts:41:22)\n at KubernetesBuilder.fetchClusterDetails (/Users/z0058456/backstage/backstage/node_modules/@backstage/plugin-kubernetes-backend/src/service/KubernetesBuilder.ts:472:28)\n at <anonymous> (/Users/z0058456/backstage/backstage/node_modules/@backstage/plugin-kubernetes-backend/src/service/KubernetesBuilder.ts:416:30)"},"request":{"method":"GET","url":"/api/kubernetes/clusters"},"response":{"statusCode":401}}

Any thoughts?

thank you @ZF-fredericvanlinthoudt for reporting the issue 🙏
Fix is coming: https://github.com/backstage/backstage/pull/24145


Stop parameter names from wrapping in markdown table

@dagar As requested, this is all that is required to stop the titles wrapping.
BUT, I cannot test this since I can't work out how to invoke the script (it appears to have become more complicated???)
I tried make parameters_metadata but that failed as shown:
D:\Github\forks\px4\Firmware>make parameters_metadata
File not found - "*_sdflight_*"
FIND: Parameter format not correct
Warning: no parameters found
Creating markdown file parameters.md
FIND: Parameter format not correct
Warning: no parameters found

I think you've hit a platform issue with the Makefile. It's working for me on OSX and Linux.

The metadata is also generated in the new Jenkins pipeline. http://ci.px4.io:8080/blue/organizations/jenkins/Firmware/detail/PR-8375-head/1/pipeline/45
Artifacts are stored here. http://ci.px4.io:8080/blue/organizations/jenkins/Firmware/detail/PR-8375-head/1/artifacts

Thanks @dagar
OK, so the first version didn't work  because even though the text no longer wrapped, it would overwrite into the next cell.  This has now been updated to add a scroll bar at the bottom once the table reaches its minimum compression. You can see that option tablescroll here (NOTE, the parent of this option is what we have currently).
Personally I'd prefer what we have currently than the scroll bar. What do you think? Give it a go on a mobile device and see.

@dagar So lorenz pushed this. Note that it obviously won't go live until next time these are regenerated.


move issues from bitbucket

https://bitbucket.org/schettino72/doit/issues?status=new&status=open

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

page not accessible anymore. :facepalm:


Garage Door Integration

Hello,
I have garage door integration with alarm.com. Is it possible that this could be added in? I think I tried the PDA login and I don't see garage doors listed there. So wasn't sure if this was possible.

If there isn't a way to control the garage door from the mobile alarm.com site (m.alarm.com), then this plugin won't be able to control it.  This plugin uses the mobile site to issue commands to alarm.com.
I don't have an integrated garage door so I can't check myself.

Pretty much what @rcaslis said.


adds jruby <IP_ADDRESS>

JRuby <IP_ADDRESS> was released today (https://www.jruby.org/2021/02/24/jruby-9-2-15-0.html). I believe this is the file that would make it available for ruby-build env.

should solve #1730


Add telemetry for snapshot size & number of blobs in writeSummaryTree

We do have telemetry from runtime, but we do not have any in driver.
This means we do not have visibility into actual payload size, as well as effectiveness of blob reuse code.

Who is looking for this telemetry? When do we want to do this work?

@vladsud Can you give us a little more info about this and who needs it?

Whoever is interested in quality of framework, and container authors who want to better understand performance characteristics of their solutions.
In practical terms, it's me, Christian, and ideally - all OCEs.

@vladsud Thanks. I didn’t know there was an open PR.

There wasn't (at the time). Some issues are easier to address right away than keep tracking :)


[CARBONDATA-2275]Query Failed for 0 byte deletedelta file

When delete is failed on write step because of any exception from hdfs . Currently 0 bye deletedelta file is created and not getting cleaned up .
So when any Select Query is triggered , Select Query is failed
Root Cause:-
deletedelta of 0 size was considered for reader and while getting the hostname it has thrown IndexOutofBound
Even Writer is failed while flush /close (exception from hdfs) but Exception is not thrown back to caller which is making delete operation is success.
Solution :-
While reading ignore 0 size deletedelta file
When Writer is failed ,Throw Failure Exception to caller and show error on query client .
deletedelta of 0 byte will be handled in cleanup automatically in next delete/update .
Any interfaces changed?
NA
Any backward compatibility impacted?
NA
Document update required?
NA
Testing done
Please provide details on

Whether new unit test cases have been added or why no new tests are required?
How it is tested? Please attach test report.
Is it a performance related change? Please attach the performance test report.
Any additional information to help reviewers in testing this change.
Testing is Done Manually.
For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
NA

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4206/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3471/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/4698/

any progress on this? ok to be merged?

retest this please

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3797/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5013/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5014/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3798/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/5015/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3799/

ok to be merged?

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/3948/

retest sdv please

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/4540/

LGTM


feat: added phoenix boilerplate + first endpoint

Motivation

The aim is to create the boilerplate for a phoenix server that we will use to implement our Beacon API, furthermore I have added the first endpoint that we will use in phase 1 of our roadmap to query the root of our latest beaconState.
Description

Added phoenix, removed boilerplate code, implemented first endpoint

Closes #130 #131

I think https://github.com/lambdaclass/lambda_ethereum_consensus/issues/165 should probably be part of this task?

@Arkenan was talking to someone about the fact that maybe Phoenix is an overkill and we should just use plug instead. I would check with him before merging this

@Arkenan was talking to someone about the fact that maybe Phoenix is an overkill and we should just use plug instead. I would check with him before merging this

Could definitely be, will wait for your expertise


Move footer outside div

Hey, I forked this class to use for GDI Atlanta, and I noticed the footer looked different on the class0 slides. This commit fixes it so it matches the rest.

Thanks!


Create PULL_REQUEST_TEMPLATE.md

Adds a pull request template to help ensure traceability and appropriate revision history updates.

This would have helped me when reviewing a PR yesterday
I'm minded to merge this and use it.  We can alter the contents later as required
thank you @rhattersley

Thanks @marqh :+1:
To help avoid further problems I've updated my open PRs to include this checklist.


Add fail-fast input to tox.yml and publish.yml

This adds a fail-fast input to the tox and cibuildwheel publish templates. This is passed directly to fail-fast in the job stratagy.
Default values
For the tox template, I have set it to not fail fast by default because people typically want to see the results for all the tests within a single call to the tox workflow. (And  this is what the Azure template does.) However, this goes against GitHub's default which is fail-fast: true.
For the publish template, I have kept the GitHub default option. As this template is more intended for publishing (rather than testing) it would make more sense to give up as soon as possible, and within each job cibuildwheel fails fast anyway.
Does that seem sensible?
Closes #33

@ConorMacBride @Cadair after using this I think it would be nice to have fast fail be false by default for wheels too - it would be good to see all errors when there are issues - it's more that we just don't want to upload if there are any issues.


Updated Achievements page

@smaranjitghose please check this out and let me know the changes. :)

Instead of having cards that shows the number of projects done by particular members, can you please make simple cards that has project name, done by, a small description and link which shall redirect to github

Yes, for sure @smaranjitghose.
Will these cards work, I can change the data on them or should I go for something else?

Ya,these cards are fine
Put instead of horizontal scrolling make it vertical scrolling with more sets of cards below,
Change the data accordingly to what I suggested above and give a PR.
I'll let you know accordingly


High CPU and Memory leaks in C#

Required Info

Camera Model
D400

Firmware Version
<IP_ADDRESS>

Operating System & Version
Win10

Platform
PC

SDK Version
2.13.0

Language
C#

Issue Description
Question 1:
When i open color stream and depth stream with pipeline function, the computer CPU > 90%.
This is a part of codes:
cfg.EnableStream(Stream.Depth, D_W, D_H, Format.Z16, 30);
cfg.EnableStream(Stream.Color, C_W, C_H, Format.Bgr8,30);
pipeline.Start(cfg);

Question 2:
When i use "SpatialFilter" and "TemporalFilter", the computer memory will increase slowly.
I also open this program one day, the memory >700MB.
This is a part of codes:
using (var f = Pipe.WaitForFrames() as FrameSet)
{
        Align align = new Align(Stream.Color);
        FrameSet _frames = align.Process(f);
        //--------------Spatial Filter---------------
        SpatialFilter Spat = new SpatialFilter();
        Spat.Options[Option.FilterMagnitude].Value = 2.0f;
        Spat.Options[Option.FilterSmoothAlpha].Value = 0.5f;
        Spat.Options[Option.FilterSmoothDelta].Value = 20.0f;
        Spat.Options[Option.HolesFill].Value = 2.0f;
        var Spat_Depth = Spat.ApplyFilter(_frames.DepthFrame);
        Spat.Dispose();
        //-------------------------------------------

        //--------------Temporal Filter---------------
        TemporalFilter Tempor = new TemporalFilter();
        Tempor.Options[Option.FilterSmoothAlpha].Value = 0.4f;
        Tempor.Options[Option.FilterSmoothDelta].Value = 20.0f;
        Tempor.Options[Option.HolesFill].Value = 4.0f;
        var Tempor_Depth = Tempor.ApplyFilter(Spat_Depth);
        Tempor.Dispose();
        //-------------------------------------------

        var Colorized_Depth = Colorize.Colorize(Tempor_Depth);
        Tempor_Depth.Dispose();
}

In the release notes for the latest 2.14.0 SDK release, there is a Known issue that was identified about C# memory leaks:
C# query device function doesn't release memory, causing memory leak
Are you using the query device function in your full script in 2.13.0, please?
Also, the 2.14.0 release contains significant optimizations of the Align processing block.  Would it be possible to try your program with the 2.14.0 SDK and see if you get improved performance?

@MartyG-RealSense
Yes, the cpu problem is solved.
But the memory leak problem still alive.

There may be nothing that can be done about the memory leak until it is fixed in a future SDK update, regretfully.  The Librealsense development team are good at finding and fixing memory leaks, though there is no way to apply a fix until it is released as part of a new SDK version.

This problem like #1983 .
I really think about this problem is filter function, when i not to use it.

On that issue #1983  , SmirkingMan posted an update an hour ago before he closed the issue in which he concluded: "There seems to be an issue with memory allocation when creating and destroying filters / video frames.  The issue can be circumvented by pre-allocating everything up front".
A development team member such as Dorodnic or RealSense-Customer-Engineering can probably provide you with a useful answer when they're back here in the coming week.

@MartyG-RealSense
OK, thanks a lot.
I hope this problem can be solve recently.

Hi @HungLearner
You should really avoid creating new Align, SpatialFilter or any other processing block inside the streaming loop. These translate to heavy native objects with a lot of internal state, that should be initialized once per device.


PowerToys should automatically launch at startup

Summary of the new feature/enhancement
After installation of PowerToys, looks like it needs to be explicitly launched upon OS restart. Please enable auto launch by adding it to OS startup.
Thanks

Oops, there it is:


[BUG] Deezer crashes after few seconds for Siri?

Describe the bug
Deezer ipas used so you can try: https://filebin.net/854oc1kcwzv88np9
Crash log:
Use of the class INVocabulary requires the entitlement com.apple.developer.siri. Make sure you have enabled the Siri capability in your Xcode project.
Call stack: (
	0   CoreFoundation                      0x00000001930ad5f8 6A60BE13-E657-3BEC-A9AC-BA239AE29862 + 185848
	1   libobjc.A.dylib                     0x0000000190629244 objc_exception_throw + 88
	2   Intents                             0x000000019d2c2094 4A90D9A1-5312-3EC7-A9D5-A1CC43DB2BF6 + 2764948
	3   Intents                             0x000000019d043990 4A90D9A1-5312-3EC7-A9D5-A1CC43DB2BF6 + 149904
	4   Intents                             0x000000019d04390c 4A90D9A1-5312-3EC7-A9D5-A1CC43DB2BF6 + 149772
	5   libdispatch.dylib                   0x000000019ae33fa8 8CE3AFB9-6D84-3446-8FD4-E5F798D98403 + 16296
	6   libdispatch.dylib                   0x000000019ae357f4 8CE3AFB9-6D84-3446-8FD4-E5F798D98403 + 22516
	7   Intents                             0x000000019d05ac20 4A90D9A1-5312-3EC7-A9D5-A1CC43DB2BF6 + 244768
	8   Deezer                              0x0000000104989038 Deezer + 3641400
	9   Deezer                              0x0000000104988c28 Deezer + 3640360
	10  Deezer                              0x0000000104e2b774 Deezer + 8501108
	11  CoreFoundation                      0x00000001930ce464 6A60BE13-E657-3BEC-A9AC-BA239AE29862 + 320612
	12  CoreFoundation                      0x00000001930ce394 6A60BE13-E657-3BEC-A9AC-BA239AE29862 + 320404
	13  Foundation                          0x0000000191ce070c 7274DDE3-68D6-34A0-8E67-7726E1265E80 + 374540
	14  Foundation                          0x0000000191ccff68 7274DDE3-68D6-34A0-8E67-7726E1265E80 + 307048
	15  Foundation                          0x0000000191ccfe88 7274DDE3-68D6-34A0-8E67-7726E1265E80 + 306824
	16  Foundation                          0x0000000191cce2e0 7274DDE3-68D6-34A0-8E67-7726E1265E80 + 299744
	17  Foundation                          0x0000000191d05750 7274DDE3-68D6-34A0-8E67-7726E1265E80 + 526160
	18  Foundation                          0x0000000191d05380 7274DDE3-68D6-34A0-8E67-7726E1265E80 + 525184
	19  libdispatch.dylib                   0x000000019ae43394 8CE3AFB9-6D84-3446-8FD4-E5F798D98403 + 78740
	20  libdispatch.dylib                   0x000000019ae33fa8 8CE3AFB9-6D84-3446-8FD4-E5F798D98403 + 16296
	21  libdispatch.dylib                   0x000000019ae3745c 8CE3AFB9-6D84-3446-8FD4-E5F798D98403 + 29788
	22  libdispatch.dylib                   0x000000019ae36b20 8CE3AFB9-6D84-3446-8FD4-E5F798D98403 + 27424
	23  libdispatch.dylib                   0x000000019ae45ec0 8CE3AFB9-6D84-3446-8FD4-E5F798D98403 + 89792
	24  libdispatch.dylib                   0x000000019ae466c4 8CE3AFB9-6D84-3446-8FD4-E5F798D98403 + 91844
	25  libsystem_pthread.dylib             0x000000021dcef644 _pthread_wqthread + 228
	26  libsystem_pthread.dylib             0x000000021dced474 start_wqthread + 8
)

Is this an error due to the free cert? If not is it fixable?
Ps: maybe you need to use a vpn/proxy with usa ip to try.

Siri is not available for free developer account. See https://developer.apple.com/help/account/reference/supported-capabilities-ios

Siri is not available for free developer account. See https://developer.apple.com/help/account/reference/supported-capabilities-ios

@hugeBlack I am thinking, is there a way to bypass this limitation? I mean a workaround in LC or similar? To the end i want only to listen music from deezer and i don't need of Siri, etc..

Yes, there's a way, but this includes patching all api calls and generating fake results to the app to make it function properly. You may find a tweak that does this, but it won't be a function for LiveContainer unless someone makes a pr.

Yes, there's a way, but this includes patching all api calls and generating fake results to the app to make it function properly. You may find a tweak that does this, but it won't be a function for LiveContainer unless someone makes a pr.

Too bad. It would be a great feature for LC. Btw thank you!


Make Blackbird & Afterburner be dynamically added to the ObjectMapper

...depending on Java version and existence in classpath
Description
We are using the Dropwizard framework which creates it's own ObjectMappers with this logic:
"If AfterburnerModule" is in the class path, then register it"
... however, Cosmos requires Afterburner to be in the classpath, as it depends on it. This leads to a bit of a conundrum for us.
This changes things such that Afterburner is only registered when it's in the classpath and < Java16, and secondly, Blackbird is loaded instead if > Java11, as it's the preferred way now.
Instead of the CosmosDB having a dependency on Afterburner and Blackbird, it now has an optional, runtime dependency.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

/azp run java - cosmos - tests

/azp run java - cosmos - spark

/azp run java - cosmos - tests

/azp run java - cosmos - spark

Manually triggered pipeline:  https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3698223&view=results

/azp run java - cosmos - spark

Anything I need to do to push this along?

Anything I need to do to push this along?

please merge from the latest main, and once all the tests pass, we can merge this PR. Will be included in next release

Rebased on the latest main.  Added description line under 4.58.0 in CHANGELOG.md

/azp run java - cosmos - spark

Manually triggered test pipeline: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3712868&view=results


Can't change variable prefix

It will be good staf to change prefix by default for variables

So change it. What's wrong?

Well it's suggestion, I will not able to commit some changes

@Hargat Why you don't like default? Is it too difficult for you to specify own?

 Because I need this for team, and we using npm.
When I change prefix by changing "prefix" config, it change only prefix for "include", not for variables!
I this plugin to create JS templates, and using $$ instead of @@ will help me to avoid incpection problems

And in plugin code, in variable.js, there is check of prefix. But somehow it doesn't works on variables

Sorry, looks like it doesn't works with $$, but works with __


"Install using react-native link" section missing in README.md

It is present in the table of contents though.

+1
@danlugo92 are you able to find how to use it using react-native-link?

I think it should be react-native link @react-native-community/datetimepicker, then cd ios && pod install, then cd ..

@waleedarshad:
react-native link @react-native-community/datetimepicker && cd ios && pod install

If you're using it with RN 0.60 or up you won't need it, I think that's why they removed

@gabrielrra I'm running RN 0.60.3, and running only npm install did not work for me. I was getting the same error reported in this issue.
Running @ditoglez command worked perfectly. I have no idea why the link command was removed from the readme (which happened in this commit), but it looks like it should be put back.


Transform into plain widget

[x] transform to plain widget
[x] fix specs
[x] check and fix docs

Implemented on master (v6.0.0).
(Probably only a little bit) Breaking Change
The laxar-details-layer-widget is now using plain as technology. As such AngularJS is no longer a dependency of the widget and any Angular mechanisms (Directives, Angular Watchers and Event Listeners, ... ) were removed.


Support Ubuntu 20.04

CouchDB 2.x is not built for Ubuntu 20.04 so step one for this should be supporting CouchDB 3.x. See Issue #935.
Some testing of media processing will also need to happen. Newer OS versions shipping different multimedia library and tools versions is a frequent cause of issues with Nunaliit's media processing.


Merge branch

Preview Link

Click to demo updates
Description

Motivation and Context

How has this been tested?

Screenshots (if appropriate):
Types of changes

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality or content)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Refactor (code improvement that doesn't affect UI)
Chore (developer productivity fix that doesn't affect the user)

Everything looks amazing!

Closing as this PR has been merged into #1718


Firefox: warning when hovering over the zero axis

Getting a warning from d3 when hovering over the 0 axis with all the default properties intact on the demo page.
Only happens in FF, none of the other browsers generate a warning...

Will fixed in zoom enhancement release


azure file upload error : 405 The resource doesn't support specified Http Verb.

I tried to upload image to Azure using FineUploader plugin.
I got this error as response from server:
Request Method:GET
Status Code:405 The resource doesn't support specified Http Verb.
My Signature Endpoint is :
https://bradsol.blob.core.windows.net/nrl-imagery/nrl-image.jpg?restype=service&comp=properties&sv=2017-04-17&ss=b&srt=sco&sp=rwdlac&se=2017-12-01T00:00:00Z&st=2017-11-01T00:00:00Z&spr=https,http&sig=6xQPKuBohCCm9CJm1cm2uX8kSFM%2BV3CNBTtkxIiadf0%3D
Container Name : nrl-imagery
Blob Name : nrl-image.jpg
My CORS configuration is as follows:
Allowed origins | Allowed methods | Allowed headers | Exposed headers | Max age |

                   | GET, PUT               |       *                    |          *                  |     200    |

Can anyone suggest if I am committing any mistake here ?
Your coopearation is highly appreciable. Thanks in Advance.

How is anyone supposed to help you without seeing any of your code?  I’m also closing this because you completely ignored the template.


Bracket matching not working at all

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Bracket matcher doesn't work at all for me. It is not highlighting any matching brackets.

Steps to Reproduce

Open HTML file
Get error regarding bracket-matcher

Expected behavior: Selecting a bracket will match it to the next bracket of that type.
Actual behavior: Nothing happens when bracket is selected.
Reproduces how often: Every time, reinstalled atom (latest beta), now there is no error but it's not working at expected at all.
Versions 1.19-beta3 and 1.18.0
You can get this information from copy and pasting the output of atom --version and apm --version from the command line. Also, please include the OS and what version of the OS you're running.
Additional Information
Any additional information, configuration or data that might be necessary to reproduce the issue.

Thanks for the report!  When you say:

Get error regarding bracket-matcher

Do you have a stack trace you can copy and paste for us?

Thank you for your submission!
We haven't gotten a response to our questions in our comment https://github.com/atom/atom/issues/14994#issuecomment-315449590. With only the information that is currently in the issue, we don't have enough information to take action. I'm going to close this but don't hesitate to reach out if you have or find the answers we need, we'll be happy to reopen the issue.


Puzzle involving IPython's '??' magic help functionality

See the screenshot about the difference in the EPICS behaviour if called directly or through (a one line) bluesky function..

Ok, we made some progresses.
The control of the desired PV was regained once ipython was restarted from fresh.
Still:

we don't understand why in the previous screenshot we asked for a function definition (shopen) and we got another one (shclose);
the action of the functions (both shopen and shclose) was consistent with the old definition while the printed definitions were "close to" the new ones.
This was very confusing and possible source of major problems  while operating on the beamline..

Right, shopen and shclose are custom functions defined (by Stuart?) in your
IPython profile. We can't do anything to fix them upstream. But that is not
to say we can't help you understand whatever confusing behavior you are
seeing.
On Wed, Nov 11, 2015 at 10:59 AM dchabot<EMAIL_ADDRESS>wrote:

i'm not sure what you're trying to say here, other than the function docs
are misreporting: the shutter is not opening when you called shopen()?
BTW - this is not really a bluesky-related problem, but perhaps belongs
here https://github.com/NSLS-II/Bug-Reports/issues
—
Reply to this email directly or view it on GitHub
https://github.com/NSLS-II/bluesky/issues/262#issuecomment-155826248.

The function was defined by us, it might be of rather general use though.
We were confused on its behaviour partially by the misreported
(crossed?) references.
However, here the main problem is general: we were quite surprised in
calling a function and getting another one shown.
Do you have any idea why?
Thank you.
Il 11/11/2015 17:06, Dan Allan ha scritto:

Right, shopen and shclose are custom functions defined (by Stuart?) in
your
IPython profile. We can't do anything to fix them upstream. But that
is not
to say we can't help you understand whatever confusing behavior you are
seeing.
On Wed, Nov 11, 2015 at 10:59 AM dchabot<EMAIL_ADDRESS>wrote:

i'm not sure what you're trying to say here, other than the function
docs
are misreporting: the shutter is not opening when you called shopen()?
BTW - this is not really a bluesky-related problem, but perhaps belongs
here https://github.com/NSLS-II/Bug-Reports/issues
—
Reply to this email directly or view it on GitHub
https://github.com/NSLS-II/bluesky/issues/262#issuecomment-155826248.

—
Reply to this email directly or view it on GitHub
https://github.com/NSLS-II/bluesky/issues/262#issuecomment-155828583.

I think I know what happened. This is a tricky one that has bitten me before, though it's incredibly bad luck that you managed to hit such a confusing case. As you'll see...
The ?? functionality works by showing the function's definition in the code. Specifically, it displays the lines in the file where the function was originally defined. If that file has been edited since the function was first loaded, the range of lines it shows could be wrong.
I can reproduce your experience with this simple example:
confusing.py:
def a():
    print('hello')

I will run this file in IPython and check the definition of a. IPython sees that a was defined in lines 1-2 of confusing.py, and so it shows those lines.
In [1]: %run -i confusing.py

In [2]: a??
Signature: a()
Source:
def a():
    print("hello")
File:      ~/confusing.py
Type:      function

Now I will edit confusing.py in the following way:
confusing.py:
def b():
    print('goodbye')

def a():
    print('hello')

Now, in the same IPython session, if I check the definition of a again, IPython again prints lines 1-2, but this time they are wrong! The file has changed, and the information that IPython has about a is stale, out-of-date.
In [4]: a??
Signature: a()
Source:
def b():
    print("goodbye")
File:      ~/confusing.py
Type:      function

I can restore sanity by re-running my file:
In [5]: %run -i confusing.py

In [6]: a??
Signature: a()
Source:
def a():
    print("hello")
File:      ~/confusing.py
Type:      function

@cmazzoli Does this adequately answer your question?

Yes, thanks, at least for the moment. If it will reappear in the future we will try to debug accordingly and in case something more/different appears we will post again..


feat(http_proxy_config): Add http_proxy_config

Describe your changes
Adds a HTTP(S) proxy configuration within this module like described in the upstream Terraform Kubernetes Cluster resource.
This feature is currently not implemented yet and it would let us configure a HTTP(s) proxy for our AKS nodes with a customer CA as well.
You can have a look on a fork which enabled it, please see here.
We are working on a PR already and it will be linked to this issue asap.
Thanks for your understanding and time.
Issue number
#434
Checklist before requesting a review

[x] The pr title can be used to describe what this pr did in CHANGELOG.md file
[x] I have executed pre-commit on my machine
[x] I have passed pr-check on my machine

Fixes: https://github.com/Azure/terraform-azurerm-aks/issues/434

The feature is GA:
https://learn.microsoft.com/en-us/azure/aks/http-proxy

@isantospardo please run pre-commit and pr-check steps like described in the README file.
https://github.com/Azure/terraform-azurerm-aks#pre-commit--pr-check--test

@isantospardo please run pre-commit and pr-check steps like described in the README file.
https://github.com/Azure/terraform-azurerm-aks#pre-commit--pr-check--test

Done, thanks for the quick note :)

@jiaweitao001 could you have a look at this PR?

Thanks @isantospardo for the update, almost LGTM but this pr should be considered as a breaking change, so I'd like to defer this pr to our next major version upgrade (I hope we could make it happened in October).

I'm closing this pr since I've opened a new one #492 , I've continued @isantospardo's work there. Thanks everyone.


Prevent duplication of SNAPSHOT source bundles

This fixes the duplication of source jars with SNAPSHOT versions
reported in stempler/bnd-platform#14.
The duplicate had a non OSGi-fied version (e.g. 1.0.0-SNAPSHOT)
which lead to errors in the update site creation.

check if there is a source Jar that can be found via file name
this is added here specifically for project dependencies
source artifact may be overridden later

Ah, so related to the above comment overriding the source artifact does not happen because the ID is different?
If we remove that we could remove a larger section, because it has no effect w/o that part. The comment with regard to the project dependencies is valid though, I think, so maybe we can solve it differently?
Might the solution be to specify a custom ID for the file artifact (which right now uses the file name)?
If I interpret the handling in ResolvedBundleArtifact correctly the ID should be "$group:$name:$version:sources" instead. The FileBundleArtifact supports a constructor parameter for specifying a custom ID.


Fix uppercase file extension

The HTML file refers to .png, so GitHub pages return 404.

Good catch, thank you!


Added quotes feature: display on the agenda view, quotes service, quo…

…te model, save button that saves the quote but right now only console.logs it since we dont have the database model yet

@KajaGrabowska Great job! Will you be adding a POST request for saving quotes to the database on another branch?

I can use another branch for the POST request once Kai or Andrea does the controller.


Use Shriram's env for better upgrade fix.

The reminder of the upgrade fix - RDS routes, plus conditionals on the new node environment that
detects 1.0.
With this a 0.8 sidecar will accept all configs from 1.0 pilot - so prod can be upgraded/rolled without traffic lost.
For now just using the presence of the env - in 1.1 we may refine this and use per-plugin or capabilities.
As a reminder, upgrade is critical, there are users with 0.8 in production and in google (and everywhere else) ability to gradually roll a new version is required.

[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:

OWNERS [costinm]

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

@costinm: PR needs rebase.

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.

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

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
41e60426e6b7036c7d65bbc80572ba3fc514b031
link
/test istio-pilot-e2e-envoyv2-v1alpha3

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/pkg/networking/plugin/mixer/OWNERS [costinm]

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

@kyessenov or @mandar-jog ?

Yes, mixer filter supported per route config since last year.
On Mon, Jul 9, 2018, 7:08 AM Shriram Rajagopalan<EMAIL_ADDRESS>wrote:

@rshriram commented on this pull request.
In pilot/pkg/networking/plugin/mixer/mixer.go
https://github.com/istio/istio/pull/6894#discussion_r201015202:

func modifyOutboundRouteConfig(in *plugin.InputParams, httpRoute route.Route) route.Route {

if !has10Config(in.Node) {

return httpRoute

}

@qiwzhang https://github.com/qiwzhang / @kyessenov
https://github.com/kyessenov ^^^ .. I distinctly remember that the
mixer was the first to get the per-route filter config.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/6894#discussion_r201015202, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJGIxkMYfZHfv4N1ZsURlfkOZwIXxQVTks5uE2PDgaJpZM4VGDKR
.

type.googleapis.com/envoy.api.v2.RouteConfiguration rejected: Unable to parse JSON as proto (INVALID_ARGUMENT:: Cannot find field.): {"forward_attributes":{"attributes":{"destination.service":{"string_value":"istio-ilbgateway.istio-system.svc.cluster.local"},"destination.service.host":{"string_value":"istio-ilbgateway.istio-system.svc.cluster.local"}}},"mixer_attributes":{"attributes":{"destination.service":{"string_value":"istio-ilbgateway.istio-system.svc.cluster.local"},"destination.service.host":{"string_value":"istio-ilbgateway.istio-system.svc.cluster.local"}}},"disable_check_calls":true}

After enabling the config we're discussing I got the error above.
How about someone who is familiar with 080 config generation makes a separate PR to generate whatever attributes are needed - maybe per route works but only some of the attributes are supported.

Forward attributes in per-route service config is the only new field, not
known to 0.8 envoy.
On Mon, Jul 9, 2018 at 12:04 PM Costin Manolache<EMAIL_ADDRESS>wrote:

After enabling the config we're discussing I got the error above.
How about someone who is familiar with 080 config generation makes a
separate PR to generate whatever attributes are needed - maybe per route
works but only some of the attributes are supported.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/6894#issuecomment-403586582, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJGIxjGdAbbfk1aHNh8kbPuMSqIIkbr_ks5uE6kngaJpZM4VGDKR
.

Ok, will add it back in separate PR, so this part can get into today build. I doubt lots of people will test the upgrade in next build or notice the missing metrics.


describe in documentation how someone can run all the tests

if through unit tests, then we need them to include the
factory.test() tests as well
e.g. j.data.nacl.test()

For running all tests we can use the following script:
import pytest
from Jumpscale import j

test_params = []
for _, group in j.core._groups.items():
    for attr in group.__dict__:
        # attrs which start by "__" like "_group_builder__template" needs to be converted to "_template"
        if "__" in attr:
            attr = "__" + attr.split("__")[1]

        test_params.append((group, attr[1:]))

@pytest.mark.parametrize("group, module_name", test_params[:10])
def test_individual_modules(group, module_name):
    module = getattr(group, module_name)
    test_method = getattr(module, "test", None)
    if test_method:
        test_method()
    else:
        pytest.skip("Module: {} has no test method".format(module.__jslocation__))

The issue is that these tests will take a very long time to run, so what are the specific test we need to run?

need to come up with better test strategy


Python: Bump pyproject.toml to 0.9.0.beta1 for beta release. Match notebooks.

Motivation and Context
In preperation for a Python SDK beta release, bumping the pyproject.toml version to 0.9.0.beta1.

Description
In preperation for a Python SDK beta release, bumping the pyproject.toml version to 0.9.0.beta1.

Contribution Checklist

[X] The code builds clean without any errors or warnings
[X] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[X] All unit tests pass, and I have added new tests where possible
[X] I didn't break anyone :smile:

Python Test Coverage Report • FileStmtsMissCoverMissingTOTAL4781105178% report-only-changed-files is enabled. No files were changed during this commit :)
Python Unit Test Overview

Tests
Skipped
Failures
Errors
Time

978
11 :zzz:
0 :x:
0 :fire:
20.718s :stopwatch:

Python Test Coverage Report • FileStmtsMissCoverMissingTOTAL4781105178% report-only-changed-files is enabled. No files were changed during this commit :)
Python Unit Test Overview

Tests
Skipped
Failures
Errors
Time

978
11 :zzz:
0 :x:
0 :fire:
21.832s :stopwatch:


Rebus no 106 'Abominable': answer doesn't match the words

Expected Behavior

There's a -b after the bomb picture
Bull or bison picture should be 'ble'
(too many plain words/letters then and rebus should be reconsidered)

Actual Behavior
The rebus solves as 'Abombinabull'
Steps to Reproduce the Problem

Open rebus 106
Take a look

Specifications

Browser: All

Hello,
I have put in a pull request to get this updated
Regards

Hey guys, its my first time here in github. How can I ask questions if I need help?
I am trying to work on this issue but the bash terminal from vs code says 'npm: command not found'. So I cant run the game in my machine. Can someone help me? Cheers


Fix race condition.

Solves the issue where a directory would be removed prematurely
if multiple files are downloaded from the same key hierarchy on S3,
in multiple concurrent jobs.  This commit also fixes
https://github.com/snakemake/snakemake/issues/404 where S3 exceptions
were silently ignored.

Thanks for spotting! This is fixed now in a slightly more generic way in commit 84c545856613f358ed2e8cca8c7cb11bfd755897.

And the exception issue is fixed in commit 889f2630.

Thanks for spotting! This is fixed now in a slightly more generic way in commit 84c5458.

awesome. thanks!


matlab code

do u have the matlab code for adaptive beamforming using  LMS algorithm in planar array

Sorry, no


Ternary policy

While this started as a bit of a joke on Twitter, I thought about this a while and came up with this recommendation.

 "Learnings"


Icon Request:  Healthcare Imaging Machine (Donut + table)

As the leading vendor of Imaging technology (MR, CT), GE Healthcare digital experiences could greatly benefit from the addition of an Imaging Machine icon much like Aviation/Rail benefit from plane/train icons.
For examples of the real machines, check these out for reference
https://www.google.com/search?q=imaging+machine&rlz=1C1GGRV_enUS753US753&source=lnms&tbm=isch&sa=X&ved=0ahUKEwiyqergscjVAhXDdpoKHfrXChYQ_AUICygC&biw=1366&bih=638
For examples of their iconic representation, reference these
https://www.google.com/search?rlz=1C1GGRV_enUS753US753&biw=1366&bih=638&tbm=isch&sa=1&q=imaging+machine+icons&oq=imaging+machine+icons&gs_l=psy-ab.3...161598.162531.0.1626<IP_ADDRESS>.<IP_ADDRESS>.0..0.0....0...1.1.64.psy-ab..6.0.0.xQKyGpfA_FI
Thanks in advance for the consideration
Doug

Passing this on to our designers

Hey Doug - I've been exploring some options, what do you think? I think I'm leaning toward one of the two in the middle of the top row...

Wow, thanks for the lightning fast turnaround and splendid craftsmanship Randy
I too gravitate to your top middle two, although using an isometric view explored in the 2nd row might have clearer recognition.    I wonder if top #3 given an isometric view (rotated ~25 degrees off to the right) could be the best of both worlds.  This would the equivalent of rotating the bottom #3 a bunch more to the left.
Regarding the patient representation, I prefer an option w/o it since our needs focus singularly on the Asset, and the convention generally in healthcare is not to show a patient inside the machines.  You can see some of the slight rotation/isometric I mentioned above in these images from the gehc website.
http://www3.gehealthcare.com/en/products/categories/magnetic_resonance_imaging?campaignid=us_hp_magnetic_resonance_imaging
Thanks for asking for my input, and I'll be ecstatic w/ anything you create
Doug

Nice, thanks for the industry perspective! We'll go with something simple and isometric for now (we can always update later if necessary):

We have a few more to add in this batch, then I'll push out a new version of the icon set.

Masterful :1st_place_medal:

Added in v2.0.7 of icon-set.


Improve error message

If forced level spans multiple modules, there is an unwrap() panic in get_forced_levels. That should be avoided or there should be a better error message.

Resolved by not unwrapping but ignoring unused forced levels.


Make timeout configurable per service

Right now the timeout is quite high, so the requests to the badger are open for quite some time, allowing a lot of time for the third party services in the backend to reply. This could be a good thing, but puts additional load on the badger (keeping all the connections open) and also is not an optimal user experience.
Therefore it should be possible to configure a timeout for each service (e.g. doaj, o2r) that is queried for badge information in the background, and return the request to the user of the badger API quicker.

closed with #9


Fix a small logging mistake and clean up

Fix a small logging mistake and clean up
clean up

Use a specific error name for each section of the realizer (see note)
merge two realizeError conditionals

fix
logging tracking error instead of realize error when the tracking fails
Note
The number of different actions with different errors in #Reconcile
feels like a code smell. We're already concerned by the complexity
of reconcile so this is not a surprise, just further evidence.

This is not mergable due to realizers getting unified. Will re-implement

implemented here: https://github.com/vmware-tanzu/cartographer/pull/1040


MSDKUI-2144: Handle deprecations & breaking changes

Signed-off-by: Piotr Szmyt<EMAIL_ADDRESS>

Codecov Report

Merging #145 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #145   +/-   ##
=======================================
  Coverage   87.53%   87.53%
=======================================
  Files          59       59
  Lines        3683     3683
=======================================
  Hits         3224     3224
  Misses        459      459

Flag
Coverage Δ

#ios
87.53% <100%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

MSDKUI/Classes/BooleanOptionItem.swift
93.67% <100%> (ø)
:arrow_up:

MSDKUI/Classes/MultipleChoiceOptionItem.swift
87.89% <100%> (ø)
:arrow_up:

MSDKUI/Classes/NumericOptionItem.swift
94.16% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


.local directory structure is missing

For the error free execution of the command
wget https://github.com/JanLahmann/RasQberry-Two/raw/main/RasQ-init.sh -O RasQ-init.sh
. ./RasQ-init.sh
the directories .local .local/config and .local/bin are missing.
After the manual creation of the the mentioned directory structure, the shell shell script above runs without a problem.
After the initial config via raspi-config the on-click installation were also possible.

What error do you get?
The directories should get created in line 69 and 74 of RasQ-init.sh, e.g.
https://github.com/JanLahmann/RasQberry-Two/blob/64e487f966440345a20b3d105be9dacb0dc0dae7/RasQ-init.sh#L69
Maybe something like
[ ! -d ~/.local ] && mkdir ~/.local
is missing or a suitable flag for mkdir.
But I can’t test it myself right now.

Here is the requested error I get:
wget https://github.com/JanLahmann/RasQberry-Two/raw/main/RasQ-init.sh -O RasQ-init.sh
--2024-08-06 08:37:16--  https://github.com/JanLahmann/RasQberry-Two/raw/main/RasQ-init.sh
Resolving github.com (github.com)... <IP_ADDRESS>
Connecting to github.com (github.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 302 Found
Location: https://raw.githubusercontent.com/JanLahmann/RasQberry-Two/main/RasQ-init.sh [following]
--2024-08-06 08:37:16--  https://raw.githubusercontent.com/JanLahmann/RasQberry-Two/main/RasQ-init.sh
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... 2606:50c0:8001::154, 2606:50c0:8002::154, 2606:50c0:8003::154, ...
Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|2606:50c0:8001::154|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 3718 (3.6K) [text/plain]
Saving to: ‘RasQ-init.sh’
RasQ-init.sh                    100%[=====================================================>]   3.63K  --.-KB/s    in 0s
2024-08-06 08:37:17 (51.6 MB/s) - ‘RasQ-init.sh’ saved [3718/3718]
rasqberry@rasqberry-two:~ $ . ./RasQ-init.sh
This script (RasQ-init.sh) downloads a modified raspi-config tool,
which can be used to install and configure Qiskit and
several Quantum Computing Demos on the Raspberry Pi
This script is available at https://github.com/JanLahmann/RasQberry-Two/
Before running this script, pls setup the SD card using the
Raspberry Pi Imager (with 'Raspberry Pi OS 64-bit with Desktop'),
then login to the RPi via ssh and run the following commands:
wget https://github.com/JanLahmann/RasQberry-Two/raw/main/RasQ-init.sh -O RasQ-init.sh
. ./RasQ-init.sh
Also use the previous command to update the RasQberry tooling.
The script be run with parameters . ./RasQ-init.sh branch gituser devoption
The -branch parameter is used to specify the branch you want to use
The -gituser parameter is used to specify the github user to use for the rasqberry repository
The -devoption parameter is used to install the development version of the rasqberry repository (production=0, dev=1)
default values are: branch=main gituser=JanLahmann devoption=0
example: . ./RasQ-init.sh main JanLahmann 1
See https://rasqberry.org for an in-depth description
of RasQberry, which combines Raspberry Pi and Qiskit,
IBM's open source Quantum Computing software framework.
using Branch  main and from github-user  JanLahmann with DevOption  0
cloning  RasQberry-Two
using production mode
-bash: git: command not found
replacing raspi-config with the modified version
--2024-08-06 08:37:29--  https://raw.githubusercontent.com/JanLahmann/RasQberry-raspi-config/bookworm/raspi-config
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... 2606:50c0:8001::154, 2606:50c0:8002::154, 2606:50c0:8003::154, ...
Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|2606:50c0:8001::154|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 110415 (108K) [text/plain]
Saving to: ‘raspi-config’
raspi-config                    100%[=====================================================>] 107.83K  --.-KB/s    in 0.04s
2024-08-06 08:37:29 (2.52 MB/s) - ‘raspi-config’ saved [110415/110415]
copy all binaries from the GH repo to ~/.local/bin
mkdir: cannot create directory ‘/home/rasqberry/.local/bin’: No such file or directory
move all config files from the GH repo to ~/.local/config
mkdir: cannot create directory ‘/home/rasqberry/.local/config’: No such file or directory
ln: failed to create symbolic link '/home/rasqberry/.local/bin': No such file or directory
RasQ-init.sh finished
You may now run    sudo raspi-config
=====
ls -la
total 136
drwx------ 2 rasqberry rasqberry   4096 Aug  6 08:37 .
drwxr-xr-x 3 root      root        4096 Aug  3 09:02 ..
-rw-r--r-- 1 rasqberry rasqberry    220 Aug  3 09:02 .bash_logout
-rw-r--r-- 1 rasqberry rasqberry   3523 Aug  3 09:02 .bashrc
-rw-r--r-- 1 rasqberry rasqberry    807 Aug  3 09:02 .profile
-rw-r--r-- 1 rasqberry rasqberry 110415 Aug  6 08:37 raspi-config
-rw-r--r-- 1 rasqberry rasqberry   3718 Aug  6 08:37 RasQ-init.sh
-rw-r--r-- 1 rasqberry rasqberry      0 Aug  6 08:37 .sudo_as_admin_successful
-rw-r--r-- 1 rasqberry rasqberry    215 Aug  6 08:37 .wget-hsts

I've always tried with the user "pi", not any other user, e.g. rasqberry.
Yours was an important test.
I think I've fixed the issue; adding a -p flag to mkdir. Could you please try again?

This time it worked:
rasqberry@rasqberry-two:~ $ wget https://github.com/JanLahmann/RasQberry-Two/raw/main/RasQ-init.sh -O RasQ-init.sh
--2024-08-06 10:18:39--  https://github.com/JanLahmann/RasQberry-Two/raw/main/RasQ-init.sh
Resolving github.com (github.com)... <IP_ADDRESS>
Connecting to github.com (github.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 302 Found
Location: https://raw.githubusercontent.com/JanLahmann/RasQberry-Two/main/RasQ-init.sh [following]
--2024-08-06 10:18:39--  https://raw.githubusercontent.com/JanLahmann/RasQberry-Two/main/RasQ-init.sh
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... 2606:50c0:8001::154, 2606:50c0:8000::154, 2606:50c0:8003::154, ...
Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|2606:50c0:8001::154|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 3724 (3.6K) [text/plain]
Saving to: ‘RasQ-init.sh’
RasQ-init.sh        100%[=================>]   3.64K  --.-KB/s    in 0s
2024-08-06 10:18:40 (28.8 MB/s) - ‘RasQ-init.sh’ saved [3724/3724]
rasqberry@rasqberry-two:~ $ . ./RasQ-init.sh
This script (RasQ-init.sh) downloads a modified raspi-config tool,
which can be used to install and configure Qiskit and
several Quantum Computing Demos on the Raspberry Pi
This script is available at https://github.com/JanLahmann/RasQberry-Two/
Before running this script, pls setup the SD card using the
Raspberry Pi Imager (with 'Raspberry Pi OS 64-bit with Desktop'),
then login to the RPi via ssh and run the following commands:
wget https://github.com/JanLahmann/RasQberry-Two/raw/main/RasQ-init.sh -O RasQ-init.sh
. ./RasQ-init.sh
Also use the previous command to update the RasQberry tooling.
The script be run with parameters . ./RasQ-init.sh branch gituser devoption
The -branch parameter is used to specify the branch you want to use
The -gituser parameter is used to specify the github user to use for the rasqberry repository
The -devoption parameter is used to install the development version of the rasqberry repository (production=0, dev=1)
default values are: branch=main gituser=JanLahmann devoption=0
example: . ./RasQ-init.sh main JanLahmann 1
See https://rasqberry.org for an in-depth description
of RasQberry, which combines Raspberry Pi and Qiskit,
IBM's open source Quantum Computing software framework.
using Branch  main and from github-user  JanLahmann with DevOption  0
cloning  RasQberry-Two
using production mode
-bash: git: command not found
replacing raspi-config with the modified version
--2024-08-06 10:18:48--  https://raw.githubusercontent.com/JanLahmann/RasQberry-raspi-config/bookworm/raspi-config
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... 2606:50c0:8001::154, 2606:50c0:8000::154, 2606:50c0:8003::154, ...
Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|2606:50c0:8001::154|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 110415 (108K) [text/plain]
Saving to: ‘raspi-config’
raspi-config        100%[=================>] 107.83K  --.-KB/s    in 0.04s
2024-08-06 10:18:49 (2.51 MB/s) - ‘raspi-config’ saved [110415/110415]
copy all binaries from the GH repo to ~/.local/bin
move all config files from the GH repo to ~/.local/config
RasQ-init.sh finished
You may now run    sudo raspi-config


Add documentation / templates for allowing the apache to blacklist in cases of unintentional DoS

In some cases we may need to shut off certain IP addresses / hosts that are causing temporary problems.
This ticket is closed with either documentation or or other means of allowing this to happen in an AmiGO production environment.

@stuartmiyasato I thought you might find this interesting for your own stuff, though you may do it differently.
I've tried this out and it works great. As apache seems to pick up the list hot, it would be very easy to make a real-time add/back-off system.


Feature Request: wild card mime types for file fields

It would be cool to allow image/* to allow all image types like the browser does
A valid MIME type string, with no extensions.
The string audio/* meaning "any audio file".
The string video/* meaning "any video file".
The string image/* meaning "any image file".
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/file#unique_file_type_specifiers

For now there are no plans for this as it will complicate unnecessary the UI and the validator checks. If you want to insert multiple mimetypes at once you can select one of the "presets":


Fixed #30827 -- Fixed batch_size param overriding to max_batch_size in bulk_create

Fixes the issue described in https://code.djangoproject.com/ticket/30827
This bug causes failure in bulk_create if passed batch_size is greater than calculated max_batch_size.
Note that this already works as expected in bulk_update: https://github.com/django/django/blob/stable/2.2.x/django/db/models/query.py#L504

@felixxm What would it take to get this merged?

@georgi-yanchev this is still missing tests and thus isn't showing in the review queue.
Given this won't be part of 3.0 this won't be available until 3.1 though.

Exactly, test is missing.

@felixxm Sorry, I did not have time to look into tests.
Thanks for adding tests and merging them.


lib/domains/alu/com/ceacfp.txt

CEAC (Centro de Estudios de Aparejadores por Correspondencia)

https://www.ceac.es/fp-informatica-comunicaciones

@Thamihb I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host ceacfp.com.alu not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


PRIORITY: CanCOGeN template Age Bin info gets overwritten when null values entered into Host Age field

When information about age is programmatically provided in the DH, the age bin info gets overwritten when null values are provided in the Host Age field i.e. they are providing the bins but not the actual ages. Is there a way to fix this?
Data providers are having difficulty providing harmonized contextual data in their submission to the national db until this is fixed, so is a high priority for troubleshooting.
Thanks!

I am having difficulty replicating this in v0.13.32.  I can enter values into age bin, and then null values into Host Age and these don't affect age bin values.

This was resolved with an apparent workaround at one user's end. Awaiting any further details to see if there is anything to revise at DH end.


Update README

Issue Template
Description
The README file is currently very long, making it difficult to quickly reach specific sections. Adding a table of contents would improve navigation.
Steps to Reproduce
Please provide step-by-step instructions on how to reproduce the issue:

Go to CODE Section
Scroll down to README Section
See that there is no table of contents available

please assign me this issue

Thank you for bringing this up. However, we believe the README file should be detailed and more comprehensive to help contributors understand the guidelines and contribution steps clearly.


Blacklist for email domains

As a...
service administrator
I want to...
be able to enable end configure a blacklist to filter email domains that can be used to signup
So that...
fake signup attempts relying on temporary email services (e.g. 10minutemail.com) could be drastically reduced

FYI, https://block-temporary-email.com
$ curl -s https://block-temporary-email.com/check/domain/kiabws.com | jq
{
  "status": 200,
  "temporary": true,
  "dns": true
}

$ curl -s https://block-temporary-email.com/check/domain/gmail.com | jq
{
  "status": 200,
  "temporary": false,
  "dns": true
}

Is this maintained? I quickly searched, there are many similar services, but often time the lists are small or not maintained. This one seems to have a very big list, but I'm unsure if/how they keep it up to date.
I'm a bit concerned about introducing an external api call during signup. And maintaining a list seems a painful/never ending job.
Maybe we can add this feature later as an additional app? We could have different apps from diff providers. Thoughts?

Is this maintained? I quickly searched, there are many similar services, but often time the lists are small or not maintained. This one seems to have a very big list, but I'm unsure if/how they keep it up to date.

That service is just an example. It's the result of a rapid query on google. I think the list is populated by scraping well known temporary services.

I'm a bit concerned about introducing an external api call during signup. And maintaining a list seems a painful/never ending job.
Maybe we can add this feature later as an additional app? We could have different apps from diff providers. Thoughts?

Yep! The provider-specific app/plugin seems to me also a valid way. My original idea was to have this feature highly configurable by the end user. The configuration could include

a switch to enable/disable the feature (e.g. verify_mail_domain: bool)
a parameter to select between while/black list (e.g. verify_mail_domain_approach: ['white', 'black'])
a parameter to specify the type of the list (e.g. verify_mail_domain_source_type: ['file', 'remote', 'api', 'db'])

file: /path/to/file.txt
remote: https://www.example.com/file.txt
db: mongodb://mongodb0.example.com:27017
api: https://api.example.com/verify (it could require additional parameters to specify the API key and, for instance, the query template)

(replaced black/white -> block/allow)

https://github.com/reacherhq/check-if-email-exists


Adding ractive syntax highlighting to vscode

Adding ractive (https://ractive.js.org/) syntax highlighting to html documents

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

See the comment here: https://github.com/wraytw/vscode/blob/261862d96533c7cf15b0ce86238be388384399b8/extensions/html/syntaxes/html.json#L2


feat(customJobs): document preconfigured jobs

@dorbin Going to go ahead and merge this. I can refactor the links in the stages reference if you think they should be later.

:+1:


Does non-time-varying variable have number of time steps 0 or 1?

The following code indicates that if a variable is not time varying, it will have 0 time steps. I'm not sure if this is a convention across the entire code base (instead of 1)?
https://github.com/NCAR/VAPOR/blob/b30705c006d6e9c094ac1847420af9876849ed22/include/vapor/NetCDFCollection.h#L736

It should be 1


Not able to set currentPage with a variable based on a Router Param.

Angular 2 version: 2.3.1
ng2-pagination version: 2.0.1
Description of issue: I am trying to define the initial page on the pagination, in case someone bookmark or reload the page. So every time that pagination is used, I update Router. Now I want to have router to update the "currentPage". By updating the value below, such as
paginate: { itemsPerPage: 1, currentPage: SearchService.currentPage }
Does not work.
Everything else is working well. And if I set "currentPage:2" it works on the first page load, but then it does not change when pagination is clicked, which is a logical result. I wanted to load that number from a Router Param, though.
Steps to reproduce:
// SearchService.pagesRange = [1,2,3,4,5];
// SearchService.currentPage = this.activeRouter.url['_value'][2];
// console.log('page === ', this.activeRouter.url['_value'][2]); // page === 3

          <pagination-controls
            (pageChange)="changePage($event)"
            maxSize="9"
            screenReaderPageLabel="page"
            screenReaderCurrentLabel="You're on page"
            #api>
          </pagination-controls>

Expected result:
My expectation is that if I set "currentPage: SearchService.currentPage" and SearchService.currentPage is equals to 2, because on Router params, than pagination should mark the page 2 as current.
Actual result:
The current result is that no elements get selected as "current, or in other words, none of the pagination numbers becomes blue.

My mistake: I needed to parseInt the value from router param. The currentPage was receiving a text with a number. Now that I am parsing it, all works fine. Sorry about the confusion.


updated README.md for logo

Uploaded copycat_logo.png previously in public.

I can't see anything in your branch.

Sorry for that! I am new to github.


Migration fails if i18n is added after spree_fancy was added to the app

I added the spree_fancy gem before the i18n and I discovered this error. Moving the AddSlider... migration behind the i18n migrations fixed it for me. Still good to know :)
==  AddSliderTaxonsAndApplyThem: migrating ====================================
rake aborted!
An error has occurred, this and all later migrations canceled:

PG::UndefinedTable: ERROR:  relation "spree_taxonomy_translations" does not exist
LINE 5:                WHERE a.attrelid = '"spree_taxonomy_translati...
                                          ^
:               SELECT a.attname, format_type(a.atttypid, a.atttypmod),
                     pg_get_expr(d.adbin, d.adrelid), a.attnotnull, a.atttypid, a.atttypmod
                FROM pg_attribute a LEFT JOIN pg_attrdef d
                  ON a.attrelid = d.adrelid AND a.attnum = d.adnum
               WHERE a.attrelid = '"spree_taxonomy_translations"'::regclass
                 AND a.attnum > 0 AND NOT a.attisdropped
               ORDER BY a.attnum
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/connection_adapters/postgresql_adapter.rb:768:in `exec'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/connection_adapters/postgresql_adapter.rb:768:in `exec_no_cache'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/connection_adapters/postgresql/database_statements.rb:138:in `block in exec_query'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/connection_adapters/abstract_adapter.rb:425:in `block in log'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activesupport-4.0.0/lib/active_support/notifications/instrumenter.rb:20:in `instrument'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/connection_adapters/abstract_adapter.rb:420:in `log'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/connection_adapters/postgresql/database_statements.rb:137:in `exec_query'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/connection_adapters/postgresql_adapter.rb:915:in `column_definitions'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/connection_adapters/postgresql/schema_statements.rb:174:in `columns'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/connection_adapters/schema_cache.rb:114:in `block in prepare_default_proc'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/connection_adapters/schema_cache.rb:56:in `yield'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/connection_adapters/schema_cache.rb:56:in `columns'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/connection_adapters/schema_cache.rb:118:in `block in prepare_default_proc'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/connection_adapters/schema_cache.rb:67:in `yield'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/connection_adapters/schema_cache.rb:67:in `columns_hash'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/associations/association_scope.rb:25:in `column_for'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/associations/association_scope.rb:37:in `bind'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/associations/association_scope.rb:73:in `block in add_constraints'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/associations/association_scope.rb:44:in `each'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/associations/association_scope.rb:44:in `each_with_index'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/associations/association_scope.rb:44:in `add_constraints'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/associations/association_scope.rb:19:in `scope'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/associations/association.rb:100:in `association_scope'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/associations/association.rb:84:in `scope'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/associations/collection_association.rb:382:in `scope'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/associations/collection_proxy.rb:37:in `initialize'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/relation/delegation.rb:78:in `new'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/associations/collection_association.rb:37:in `reader'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/associations/builder/association.rb:70:in `translations'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/globalize-4.0.0.alpha.2/lib/globalize/active_record/instance_methods.rb:125:in `translation_for'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/globalize-4.0.0.alpha.2/lib/globalize/active_record/adapter.rb:76:in `fetch_attribute'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/globalize-4.0.0.alpha.2/lib/globalize/active_record/adapter.rb:25:in `block in fetch'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/globalize-4.0.0.alpha.2/lib/globalize/active_record/adapter.rb:24:in `each'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/globalize-4.0.0.alpha.2/lib/globalize/active_record/adapter.rb:24:in `fetch'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/globalize-4.0.0.alpha.2/lib/globalize/active_record/instance_methods.rb:36:in `write_attribute'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/globalize-4.0.0.alpha.2/lib/globalize/active_record/class_methods.rb:142:in `block in translated_attr_accessor'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/attribute_assignment.rb:42:in `public_send'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/attribute_assignment.rb:42:in `_assign_attribute'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/attribute_assignment.rb:29:in `block in assign_attributes'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/attribute_assignment.rb:23:in `each'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/attribute_assignment.rb:23:in `assign_attributes'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/globalize-4.0.0.alpha.2/lib/globalize/active_record/instance_methods.rb:19:in `block in assign_attributes'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/globalize-4.0.0.alpha.2/lib/globalize/active_record/instance_methods.rb:187:in `with_given_locale'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/globalize-4.0.0.alpha.2/lib/globalize/active_record/instance_methods.rb:19:in `assign_attributes'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/core.rb:192:in `initialize'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/inheritance.rb:27:in `new'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/inheritance.rb:27:in `new'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/persistence.rb:36:in `create'
/Users/FJ/Desktop/rails_projects/travelspree/db/migrate/20131112015835_add_slider_taxons_and_apply_them.spree_fancy.rb:4:in `up'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/migration.rb:574:in `exec_migration'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/migration.rb:555:in `block (2 levels) in migrate'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/migration.rb:554:in `block in migrate'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/connection_adapters/abstract/connection_pool.rb:294:in `with_connection'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/migration.rb:553:in `migrate'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/migration.rb:709:in `migrate'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/migration.rb:959:in `block in execute_migration_in_transaction'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/migration.rb:1005:in `block in ddl_transaction'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/connection_adapters/abstract/database_statements.rb:202:in `block in transaction'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/connection_adapters/abstract/database_statements.rb:210:in `within_new_transaction'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/connection_adapters/abstract/database_statements.rb:202:in `transaction'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/transactions.rb:209:in `transaction'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/migration.rb:1005:in `ddl_transaction'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/migration.rb:958:in `execute_migration_in_transaction'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/migration.rb:920:in `block in migrate'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/migration.rb:916:in `each'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/migration.rb:916:in `migrate'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/migration.rb:764:in `up'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/migration.rb:742:in `migrate'
/Users/FJ/.rvm/gems/ruby-2.0.0-p247/gems/activerecord-4.0.0/lib/active_record/railties/databases.rake:42:in `block (2 levels) in <top (required)>'
Tasks: TOP => db:migrate
(See full trace by running task with --trace)

If you remove the 'spree_i18n' gem it will works...I hope it will works for others in future.


[v3] Some flyout windows are not recognized by kwm

Extracting from #232
Some flyout windows (I think they might be WebViews, not positive though) are not recognized by kwm and allow focus-follows-mouse to trigger the window below the flyout window.
I have observed this behavior with Dropbox and 1Password menu trey flyout windows.
Prior to the rewrite these flyout windows were usable with focus-follows-mouse enabled, and did not raise the window below the flyout window.
Happy to collect any logs that might be useful in diagnosing the window.

This seems like a similar issue to what I'm having here: https://github.com/koekeishiya/kwm/issues/397.

I can confirm this behaviour with the flyout windows of Google Drive, instant translate and tripmode, apps that are launched from the top bar.
As soon as the flyout windows appears on top of a tiled window, I click through it and can't get it to focus.
If the app appears in top of a floating window, it is working as expected.

If you can use the OSX accessiblity application to get some information regarding the windows in question, and see if they have a common pattern, that would be great.

How do you launch it ?

Create a rule for the affected applications to allow the AXSystemDialog role. By default, Kwm will only care about windows with the type `AXWindow:AXStandardWindow

@koekeishiya
do you mean this
kwmc rule owner="Dropbox" properties={role="AXSystemDialog"}
kwmc rule owner="1Password" properties={role="AXSystemDialog"}

if so, that did not work for me

Are you sure this shouldn't be solved from KWMs side, this used to work in v2, so this feels like a regression.

It worked previously because of polling. Anything that isn't reported by the applications themselves through the OSX notifications is pretty much not fixable. I really have no idea how to approach this problem.

I'm not sure if it's the same issue, but sometimes window aren't recognised for me either. Even if I have a specific rule for it, the window is completely ignored. It even happens with Safari windows once in a while. Maybe the window isn't available when notification of app activation is triggered?

Got the same issue too, mainly with my IDE, but as you mentioned this is random. The window appears in the space under all floating windows, the window is not listed when launching kwmc query window list, strange...

I'd like to revive this issue. It seems related to #232 as the symptoms are very similar.
It seems like this issue should be solvable since the accessibility inspector reports these dialogs. Is it possible to not raise if the mouse is over a AXSystemDialog?
This issue is very frustrating as it makes some 1Password and Dropbox extraordinarily difficult to use.
Even "Accessibility Inspector" itself is subject to this issue. How can I make autoraise turn off if I am over a AXSystemDialog. Clearly you are responding to mouse move events. Clearly in response to a mouse move event you look up what windows are below the mouse and raise the window that is below the mouse.
How can we get it  to not raise if the mouse is above a AXSystemDialog?

Try the latest commit. I reimplemented ffm to not do anything if the window that is in fact topmost, for some reason is unknown for Kwm.

@koekeishiya works great!

I reimplemented ffm to not do anything if the window that is in fact topmost, for some reason is unknown for Kwm.

Clever solution!


fix(chore): Minor typographical fixes

Fixes include:-

Typo fixes
Grammatical corrections

What kind of change does this PR introduce?

refactoring
Did you add tests for your changes?
N/A

Does this PR introduce a breaking change?
Nope

What needs to be documented once your changes are merged?
N/A

Thanks


Kinesis Analytics CodeContent property is incorrectly defined

When you use CodeContent property to define a Kinesis Analytics application you get an error when exporting the CloudFormation, because the property is defined incorrectly.

Fixed via merge.


Fixes String#match and RegExp#exec declarations. (#2450)

The current declaration of String#match fails when trying to access the index and input properties of the return value.
Tests are failing because error messages are different due to new line numbers. I didn't want to update these by hand and I'm not sure how to update them otherwise (played around with generate_exp.sh).

Hmm, it seems possible that intersection types don't play nicely with Arrays and Objects. After running tests, the only non-line-number-change diff is:
-overload.js:7
-  7: var x1: number = "".match(0)[0];
-                      ^^^^^^^^^^^^^^ string. This type is incompatible with
-  7: var x1: number = "".match(0)[0];
-             ^^^^^^ number
-

-overload.js:8
-  8: var x2: number = "".match(/pattern/)[0];
-                      ^^^^^^^^^^^^^^^^^^^^^^ string. This type is incompatible with
-  8: var x2: number = "".match(/pattern/)[0];
-             ^^^^^^ number
-

-Found 12 errors
+Found 10 errors

I don't think this will work, see: https://flow.org/try/#0CYUwxgNghgTiAEA3W8AeAueBBGMoE8AeAZwBcYBLAOwHMA+eAMngG8AfeEVcqTMy2vDYBfANwAocQApUAOi49MVAK4BbAEYgYASlFA
Not sure if this is a bug that should be fixed separately in Flow, but this diff can't be merged in as is.

I don't think this will work, see: https://flow.org/try/#0CYUwxgNghgTiAEA3W8AeAueBBGMoE8AeAZwBcYBLAOwHMA+eAMngG8AfeEVcqTMy2vDYBfANwAocQApUAOi49MVAK4BbAEYgYASlFA
Not sure if this is a bug that should be fixed separately in Flow, but this diff can't be merged in as is.

@ avikchaudhuri I just went back and tried that try flow link, and it seems to be erroring as expected now. Perhaps something was fixed in flow in the last 1+ year that now allows this to work? Can we revisit this if possible now?

Looks like the landscape has changed a bit. Using this as inspiration along with the new code in master I've opened: https://github.com/facebook/flow/pull/6790


Exit early in blacklist handler

When the message is deleted it could?? cause a crash if we try to access its attributes later. So exit the loop early in that case.
I'm not sure of a better way to do this that is still obvious what's trying to be accomplished.
Technically using a .every would exit early when you return false but that just feels unintuitive for what I'm trying to do

Upon making that change, I feel like the crash could have been an unawaited DM, which would crash if the user has DMs turned off. Either way, I do appreciate refactoring this handler


[SPIKE] Use TypeScript for the Accordion's defaults immutability

Object.freeze only does a shallow freeze of the object, meaning we have nothing checking our code against something setting one of the keys of i18n. The ReadonlyDeep<T> type from type-fest allows TypeScript to check that even deep properties cannot be updated by our code.
This spike illustrates how we would use the ReadonlyDeep<t> type, using the Accordion component and building on the fact that we define the types for each component's configuration as <COMPONENT_NAME>Config.

[!IMPORTANT]
Tests are expected to fail on this spike to demonstrate that TypeScript does indeed pick up on nested properties of the Accordion's defaults getting modified when they shouldn't

Thoughts
Removing Object.freeze
Because Object.freeze only does a shallow freeze, it provide a false sense of safety that deep keys are immutable if you're not aware of its shallowness. Implementing a function to deep freeze an object recursively would mean extra code shipped to the browser for the sake of preventing users' code from modifying a component's defaults.
Leaving these defaults unprotected by removing Object.freeze doesn't seem a massive risk. If anything it allows an undocumented way to set defaults for all the components of a given class, while allowing multiple calls to createAll or initAll to override those defaults with specific configurations for given pages or parts of pages. And it shaves a little bit of code which won't be sent to the browser.
Keeping things readonly in our code
This doesn't mean we cannot check that our code doesn't inadvertently try to update one of the properties in defaults. TypeScript's Readonly<T> type unfortunately works shallowly, just like Object.freeze. However, the type-fest package provides a ReadonlyDeep<T> type that ensures nested objects/Maps/Sets/Arrays are also readonly (and is heavily tested).
The type-fest package doesn't add to our own dependencies as type checking happens at build time, so it only needs to be a devDependency. However, it'll have impact on the @types/govuk-frontend package, for which it's become a dependency (unless there's a system for vendoring it).

This feels like a sensible approach - I think we definitely want to make sure we're testing for read-onliness internally. I'm less worried about the impact on @types/govuk-frontend, tbh.


MSDKUI-1480: Guidance Dashboard overview test

Guidance Dashboard overview test implementation
Signed-off-by: Aleksandr Dotsenko<EMAIL_ADDRESS>

Codecov Report

Merging #39 into master will increase coverage by 2.99%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master     #39      +/-   ##
===========================================
+ Coverage      70.6%   73.6%   +2.99%
+ Complexity     1538     958     -580
===========================================
  Files           126      68      -58
  Lines          5386    3735    -1651
  Branches        817     395     -422
===========================================
- Hits           3803    2749    -1054
+ Misses          973     759     -214
+ Partials        610     227     -383

Impacted Files
Coverage Δ
Complexity Δ

...e/msdkuiapp/routing/RoutingInstructionsFragment.kt

...java/com/here/msdkuiapp/common/PermissionsUtils.kt

...idance/GuidanceEstimatedArrivalTimeViewFragment.kt

...ere/msdkuiapp/routing/RouteManeuverListFragment.kt

.../here/msdkuiapp/routing/SubOptionPanelPresenter.kt

...m/here/msdkuiapp/routing/SubOptionPanelFragment.kt

...msdkuiapp/guidance/GuidanceNextManeuverFragment.kt

...om/here/msdkuiapp/position/LocationServiceUtils.kt

...ain/java/com/here/msdkuiapp/about/AboutActivity.kt

...e/msdkuiapp/position/LocationPermissionFragment.kt

... and 49 more

Continue to review full report at Codecov.

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