Cannot read property 'addEventListener' of null when render testing component

I'm having some trouble render testing one of my components because of the following error:
Cannot read property 'addEventListener' of null
Happening at at GoogleMap.componentDidMount (node_modules/google-map-react/lib/google_map.js:791:57)
I'm not sure about how to test my component with google-map-react. My test file is really simple:
import React from 'react'
import renderer from 'react-test-renderer'

describe('<PartnerLocalConfirmation />', () => {
  it('should render correctly', () => {
    const component = renderer.create(
        <MockedProvider>
            <PartnerLocalConfirmation {...props}/>
        </MockedProvider>)
    expect(component).toMatchSnapshot()
  })
})

Can someone help me with that?

Your mock must be wrong and most likely it is not returning google maps object. See some tests i have worked on from here.
Google maps react test
You can also see how i use the library

@drdpedroso Let me know if you need further help, closing issue.


Using GitOps with Grafana Operator to create Grafana Dashboard with Flux.

What does this PR do?

🛑 Please open an issue first to discuss any significant work and flesh out details/direction - we would hate for your time to be wasted. Consult the CONTRIBUTING guide for submitting pull-requests.
Motivation

Using GitOps with Grafana Operator to create Grafana Dashboard with Flux.
More

[X] Yes, I have tested the PR using my local account setup  (Provide any test evidence report under Additional Notes)
[X] Yes, I ran pre-commit run -a with this PR
[X] Yes, I have added a new example under examples to support my PR (when applicable)
[X] Yes, I have updated the Pages for this feature

Note: Not all the PRs required examples and docs.
For Moderators

[ ] E2E Test successfully complete before merge?

Additional Notes

This looks good @elamaran11, how do we validate this?

This looks good @elamaran11, how do we validate this?

@bonclay7 Just run it as is. It will work and you can validate a new dashboard getting created.

@elamaran11 To fully validate this, maybe we should drop this in terraform? so we are sure the operator set it up correctly

So I went ahead and did it, I dropped the dashboard provided by Terraform but for some reason the operator can't renew my API key, maybe i'm doing something wrong?
1.6857285045139527e+09	INFO	found matching Grafana instances for datasource	{"controller": "grafanadatasource", "controllerGroup": "grafana.integreatly.org", "controllerKind": "GrafanaDatasource", "GrafanaDatasource": {"name":"grafanadatasource-sample-amp","namespace":"grafana-operator"}, "namespace": "grafana-operator", "name": "grafanadatasource-sample-amp", "reconcileID": "72cfd60c-a255-44a1-bfbd-88b0cbc4f90c", "count": 1}
1.6857285045445938e+09	INFO	found matching Grafana instances for dashboard	{"controller": "grafanadashboard", "controllerGroup": "grafana.integreatly.org", "controllerKind": "GrafanaDashboard", "GrafanaDashboard": {"name":"external-grafanadashboard-url","namespace":"grafana-operator"}, "namespace": "grafana-operator", "name": "external-grafanadashboard-url", "reconcileID": "5a07e7b0-d200-4223-94bc-a2fe6e6009db", "count": 1}
1.6857285045556655e+09	ERROR	error reconciling datasource	{"controller": "grafanadatasource", "controllerGroup": "grafana.integreatly.org", "controllerKind": "GrafanaDatasource", "GrafanaDatasource": {"name":"grafanadatasource-sample-amp","namespace":"grafana-operator"}, "namespace": "grafana-operator", "name": "grafanadatasource-sample-amp", "reconcileID": "72cfd60c-a255-44a1-bfbd-88b0cbc4f90c", "datasource": "grafanadatasource-sample-amp", "grafana": "external-grafana", "error": "status: 401, body: {\"message\":\"Expired API key\"}\n"}
github.com/grafana-operator/grafana-operator/controllers.(*GrafanaDatasourceReconciler).Reconcile
	github.com/grafana-operator/grafana-operator/controllers/datasource_controller.go:209
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile
	sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:121
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler
	sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:320
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
	sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:273
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
	sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:234
1.6857285046102674e+09	ERROR	error reconciling dashboard	{"controller": "grafanadashboard", "controllerGroup": "grafana.integreatly.org", "controllerKind": "GrafanaDashboard", "GrafanaDashboard": {"name":"external-grafanadashboard-url","namespace":"grafana-operator"}, "namespace": "grafana-operator", "name": "external-grafanadashboard-url", "reconcileID": "5a07e7b0-d200-4223-94bc-a2fe6e6009db", "dashboard": "external-grafanadashboard-url", "grafana": "external-grafana", "error": "status: 401, body: {\"message\":\"Expired API key\"}\n"}
github.com/grafana-operator/grafana-operator/controllers.(*GrafanaDashboardReconciler).Reconcile
	github.com/grafana-operator/grafana-operator/controllers/dashboard_controller.go:216
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile
	sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:121
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler
	sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:320
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
	sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:273
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
	sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:234

@elamaran11 To fully validate this, maybe we should drop this in terraform? so we are sure the operator set it up correctly?
And this too?

@bonclay7 Dropping is not required both boards can coexist for now.

So I went ahead and did it, I dropped the dashboard provided by Terraform but for some reason the operator can't renew my API key, maybe i'm doing something wrong?
1.6857285045139527e+09	INFO	found matching Grafana instances for datasource	{"controller": "grafanadatasource", "controllerGroup": "grafana.integreatly.org", "controllerKind": "GrafanaDatasource", "GrafanaDatasource": {"name":"grafanadatasource-sample-amp","namespace":"grafana-operator"}, "namespace": "grafana-operator", "name": "grafanadatasource-sample-amp", "reconcileID": "72cfd60c-a255-44a1-bfbd-88b0cbc4f90c", "count": 1}
1.6857285045445938e+09	INFO	found matching Grafana instances for dashboard	{"controller": "grafanadashboard", "controllerGroup": "grafana.integreatly.org", "controllerKind": "GrafanaDashboard", "GrafanaDashboard": {"name":"external-grafanadashboard-url","namespace":"grafana-operator"}, "namespace": "grafana-operator", "name": "external-grafanadashboard-url", "reconcileID": "5a07e7b0-d200-4223-94bc-a2fe6e6009db", "count": 1}
1.6857285045556655e+09	ERROR	error reconciling datasource	{"controller": "grafanadatasource", "controllerGroup": "grafana.integreatly.org", "controllerKind": "GrafanaDatasource", "GrafanaDatasource": {"name":"grafanadatasource-sample-amp","namespace":"grafana-operator"}, "namespace": "grafana-operator", "name": "grafanadatasource-sample-amp", "reconcileID": "72cfd60c-a255-44a1-bfbd-88b0cbc4f90c", "datasource": "grafanadatasource-sample-amp", "grafana": "external-grafana", "error": "status: 401, body: {\"message\":\"Expired API key\"}\n"}
github.com/grafana-operator/grafana-operator/controllers.(*GrafanaDatasourceReconciler).Reconcile
	github.com/grafana-operator/grafana-operator/controllers/datasource_controller.go:209
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile
	sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:121
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler
	sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:320
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
	sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:273
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
	sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:234
1.6857285046102674e+09	ERROR	error reconciling dashboard	{"controller": "grafanadashboard", "controllerGroup": "grafana.integreatly.org", "controllerKind": "GrafanaDashboard", "GrafanaDashboard": {"name":"external-grafanadashboard-url","namespace":"grafana-operator"}, "namespace": "grafana-operator", "name": "external-grafanadashboard-url", "reconcileID": "5a07e7b0-d200-4223-94bc-a2fe6e6009db", "dashboard": "external-grafanadashboard-url", "grafana": "external-grafana", "error": "status: 401, body: {\"message\":\"Expired API key\"}\n"}
github.com/grafana-operator/grafana-operator/controllers.(*GrafanaDashboardReconciler).Reconcile
	github.com/grafana-operator/grafana-operator/controllers/dashboard_controller.go:216
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile
	sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:121
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler
	sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:320
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
	sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:273
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
	sigs.k8s.io/controller-runtime@v0.13.0/pkg/internal/controller/controller.go:234

@bonclay7 The Grafana Key in the Secrets Manager is expired. The external secrets will take care of refreshing secrets manager key with k8s secret for grafana operator but operator has to take care of refereshing grafana keys with below :
export GO_AMG_API_KEY=$(aws grafana create-workspace-api-key \
  --key-name "grafana-operator-key" \
  --key-role "ADMIN" \
  --seconds-to-live 432000 \
  --workspace-id <YOUR_WORKSPACE_ID> \
  --query key \
  --output text)

export GO_API_KEY_SECRET_NAME=$(aws secretsmanager list-secrets \
    --query 'SecretList[1].Name' | tr -d '"')

aws secretsmanager update-secret \
    --secret-id $GO_API_KEY_SECRET_NAME \
    --secret-string "${GO_AMG_API_KEY}" \
    --region $GO_AWS_REGION

Did I miss these instructions in the docs, or we are waiting for the full move to make it clear?

Dropping is not required both boards can coexist for now

Yes, but for customers using latest, they would be quite confused that we have two data sources and duplicates dashboards. I would suggest that we make it more seamless

Did I miss these instructions in the docs, or we are waiting for the full move to make it clear?

This is same as how you upgrade the Grafana Keys today. We can add these instructions after full move

Dropping is not required both boards can coexist for now

Yes, but for customers using latest, they would be quite confused that we have two data sources and duplicates dashboards. I would suggest that we make it more seamless

Agreed. I commented out the NodeExporter in dashboard.tf but we can move it fully out in the full migration.

I can confirm after updating the secrets, the new DS pops up and Dashboard too.
I think it will be more sleek if we transition all the dashboards / folders at once so we keep the existing DS name aws-observability-accelerator and dashboard folders

That way we have a transparent transition. But cool to see it works! happy to help on that if you need

I can confirm after updating the secrets, the new DS pops up and Dashboard too.
I think it will be more sleek if we transition all the dashboards / folders at once so we keep the existing DS name aws-observability-accelerator and dashboard folders
That way we have a transparent transition. But cool to see it works! happy to help on that if you need

Thankyou for confirming that Grafana Operator works to create Dashboards in AMG via GitOps. Only problem i have now is Folder creation is not working with external grafana. I have raised this issue with grafana-operator GitHub. Once folder works i can completely move it to new solution. I was thinking this as pilot. I will keep you posted on the issue. But let me know if you want to hold this PR or create all boards without folder or just have this board alone without folder. Im fine with either approaches

@bonclay7 Now i have all dashboards in the mix with Grafana Operator and commented out all dashboards in dashboard.tf. We are dependent on the GrafanaFolder feature from the ticket i previously mentioned and also dependent on this PR.

@bonclay7 Grafana Folder works now with Grafana Operator. So i have migrated all the 6 dashboards for EKS Monitoring Module to Grafana Operator and Flux with GitOps. I think we can seamlessly move other pattern dashboards and cleanup as subsequent work in future PRs. Please take this forward and let me know if you have questions.

Added an operational procedure as well under Troubleshooting on how to replace the expired Grafana API Keys in AWS Secrets Manager.


Add Bubble Sort Algorithm to DSA folder. Create a sub folder Sorting in DSA then add your algorithm to it

Please assign this issue to me.


For docker.yml checkout the tag used in pypi_publish

If you haven't already, please read through our contributing guidelines before opening your PR
What is the purpose of this PR?
Closes #947.
How did you implement your changes
During the actions/checkout step, we specify the head_branch used. source
Remaining issues
None ATM.

Woops, i thought just the upload coverage step got skipped, but it was the actual testing run. My bad, looks like we'll need a follow-up PR to address the issue

@ngreenwald @camisowers Looks like there was a new ipywidgets release, 8.0.5 earlier today. Gonna try take a look and see what needs to be fixed.


Add support of subflow env var

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

Proposed changes

This PR add support of SUBFLOW environment variable support described in SUBFLOW Enhancements design note (https://github.com/node-red/node-red/wiki/Design%3A-Subflow-Enhancements).
Checklist

[x] I have read the contribution guidelines
[x] For non-bugfix PRs, I have discussed this change on the mailing list/slack team.
[x] I have run grunt to verify the unit tests pass
[x] I have added suitable unit tests to cover the new/changed functionality

Thanks @HiroyasuNishiyama
I think this is looking good to merge at this point.
I'm not 100% sure about adding the new subflow_spec.js file rather than do those tests in the existing Subflow_spec.js file. Our general test structure is to have each spec file map to an original source file and subflow_spec.js does not correspond directly to a source file.
I have also spotted there is still a bug in there somewhere that setting a value to any false-like value (0 or false) results in a value of an empty string - I'll investigate that myself.


command not found

After installing this extension, I can't run any of the commands. I get this error message.
command 'extension.applescript.compile' not found
I get the same error message for all four of the commands.

How did you install the extension? What's your OS?

I used the quick open and copied the install command. i'm using os x el capitan 10.11.5

I wish I could reproduce this. I'm using the extension on OS X 10.9, 10.12 (before that 10.11), and Windows 10. I just wiped ~/.vsce and ~/.vscode and reinstalled the extension – still works.
Did you try any other extensions that provide commands, not just snippets?

I'm going to upgrade to 10.12 Sierra and try it then get back to you.

As I said, when I released the extension, I was still on El Capitan. I doubt that an upgrade to Sierra will change anything. The error has to be elsewhere. Did you try the obvious, like removing and re-installing? Are there additional messages in the console?

Okay, so I figured out what was going on. In the applescript script editor, you can save your scripts as a binary which gives it a .scpt extension. But these binary files don't open in visual studio code for editing. So, I made the mistake of trying to recreate the script in VSC and just add a .scpt extension to the end when I saved it. That's where I ran into trouble and it wouldn't find the extension commands when I tried to run them.
The solution is to create a text script with an .applescript extension so that it is in text format. I can do this either in apple's script editor and save as the "Text" option instead of the script option or I can do this directly in visual studio code. When I do it like this, the extension commands work perfectly.

Thanks for looking into it. I know the Atom package for AppleScript passes .scpt files to osadecompile. I'll see that I add this functionality in a future version.


[ACM-15664] Added CAPI component to charts config

Description
Onboarding CAPI component into MCE 2.8.
Related Issue
https://issues.redhat.com/browse/ACM-15664
Changes Made
Added CAPI to charts-config.yaml to enable its integration into automation.
Screenshots (if applicable)
Add screenshots or GIFs that demonstrate the changes visually, if relevant.
Checklist

[ ] I have tested the changes locally and they are functioning as expected.
[ ] I have updated the documentation (if necessary) to reflect the changes.
[ ] I have added/updated relevant unit tests (if applicable).
[ ] I have ensured that my code follows the project's coding standards.
[ ] I have checked for any potential security issues and addressed them.
[ ] I have added necessary comments to the code, especially in complex or unclear sections.
[ ] I have rebased my branch on top of the latest main/master branch.

Additional Notes
Add any additional notes, context, or information that might be helpful for reviewers.
Reviewers
Tag the appropriate reviewers who should review this pull request. To add reviewers, please add the following line: /cc @reviewer1 @reviewer2
/cc @cameronmwall
Definition of Done

[ ] Code is reviewed.
[ ] Code is tested.
[ ] Documentation is updated.
[ ] All checks and tests pass.
[ ] Approved by at least one reviewer.
[ ] Merged into the main/master branch.

/override ci/prow/sonar-pre-submit ci/prow/test-unit

/lgtm


“Feedback” visual system variables

e.g. Success, Error, Warning

Great idea. Maybe we can incorporate a sort of "loudness" language around the use of colors?
https://www.viget.com/articles/visual-loudness/


some error

QR Pop.xcodeproj cannot be opened because it is missing its project.pbxproj file.

A added the project.pbxproj file to the repo 👍


[Bug][Worker Server ] When the DS uses the data source to connect to the Spark Thrift server, the connection channel remains connected and will not be closed

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
I found that after the server netstat spark port The worker server does not disconnect from the Sparkhrift after the connection is established I think the SQL (spark data source) task node in each workflow will remain connected after execution.Restores to normal after the DS service is stopped.
spark version:3.2.1
hive version:3.1.2
What you expected to happen
The worker server did not provide clear error information.
tail worker log file:

netstat spark thrift server pord status:

The status and connect open num of the spark thrift port after the DS service is stopped:

How to reproduce
1.create spark datasource

2.Define workflow and create spark data source query statement in workflow

3.run workflow

4.Connection status of the port after execution

Anything else
No response
Version
3.0.x
Are you willing to submit PR?

[x] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Please confirm if the issue exists in 31x and dev.

Please confirm if the issue exists in 31x and dev.

OK, I'll try it tomorrow. thank

The problem still exists after I upgrade DS to version 3.1x

The problem still exists after I upgrade DS to version 3.1x

@yyh2954360585 can you fix it?

I will try to solve this problem when I have time.

The problem still exists after I upgrade DS to version 3.1x

@yyh2954360585 can you fix it?

will try to solve this problem when I have time.

The problem still exists after I upgrade DS to version 3.1x

@yyh2954360585 can you fix it?

I will try to solve this problem when I have time.

has this bug been fixed ?


<EMAIL_ADDRESS>returns element for offline players

I'm not sure if the same is true for all .type tags, but it certainly is for players.
Using an offline player
/ex narrate<EMAIL_ADDRESS>Returns bob
/ex narrate<EMAIL_ADDRESS>Returns element
Event tacking in .as_player still returns element despite the player being valid.

Fixed in dev builds.

All is good here now. Closing.


Systest: fix gke timeout fails

Motivation
Fix the systest GKE Init Timeout
Description
The GKE Systest Cluster didn't wait for the docker image to be build to start the test, and because of that some runs were the cluster provisioning finish before the docker build the test it self could trigger the timeout waiting for the image to start the test POD
Test Plan
Run the systest and observe the GKE waiting for the docker build
TODO

[X] Explain motivation or link existing issue(s)
[X] Test changes and document test plan
[X] Update documentation as needed
[X] Update changelog as needed

unit tests timed out on arm runner
bors merge


Network Monitor: Runtime selection of DHCP/Static IP with fall back set by the board, and a polled mode for HW lacking a PHY interrupt

Summary
Sister to https://github.com/apache/incubator-nuttx/pull/1886 - merge this APP pr second.
netinit:Network Monitor add a polled option
Not all boards have in interrupt line from the phy to the Soc. This allows the phy to be polled for link status.
This may not work on all MAC/PHY combination that have mutually exclusive link management and operating
modes. The STM32F7 and LAN8742AI do not have such a limitation.
To use this option the arch must supply CONFIG_ARCH_PHY_POLLED  and not CONFIG_ARCH_PHY_INTERRUPT.
If the network is not connected the phy init may time out and netinit_net_bringup will return with out completing DHCP or initializing the ntp client.
This change uses the lack of the DHCP lease, to contiune the netinit_net_bringup, once the interface does come up. It also will renew the lease over time, and will re run the dhcp request after a network down/up transition.
netinit Net monitior support getting address from board
Add the use of CONFIG_BOARDCTL_NETCONF to retrieve the address configurations from the board. Allowing setting
static or dhcp addressing at run time.
netinit:Net monitior support fallback to static IP
After CONFIG_NETINIT_FALLBACK dhcp attempts, fall back to  the static IP provided by the board via CONFIG_BOARDCTL_NETCONF.
Impact
None All options have Kconfig knobs to turn them off by default
Testing
FMUV5X with a test vector of all the new Kconfig options.

I've looked at this twice now and nothing stands out. I'll let the others comments get responded to before approving.

The use of boardctl() in this way is unacceptable.  I discuss in the matching PR apache/incubator-nuttx#1886

I've looked at this twice now and nothing stands out. I'll let the others comments get responded to before approving.

The use of boardctl() in this way is unacceptable. I discuss in the matching PR apache/incubator-nuttx#1886

Not sure how I didn't see the nuttx changes PR or realize there would have needed to be one. That ones a little more active, I'll spend some time reviewing it since this change is meaningless until the other one is resolved.

Not sure how I didn't see the nuttx changes PR or realize there would have to be one. That one's a little more active, I'll spend some time reviewing it since this change is meaningless until the other one is resolved.

I want that change to be made in a more standard and compatible way, using existing mechanisms, proper separation of application and OS functionality (DHCP is application functionality) and without using the KLUDGE boardctl() interface.
My concern is with the integrity and roadmap of the OS.  I don't want quick-n-dirty changes when a proper system solution is required.  That degrades the OS.

Not sure how I didn't see the nuttx changes PR or realize there would have to be one. That one's a little more active, I'll spend some time reviewing it since this change is meaningless until the other one is resolved.

I want that change to be made in a more standard and compatible way, using existing mechanisms, proper separation of application and OS functionality (DHCP is application functionality) and without using the KLUDGE boardctl() interface.
My concern is with the integrity and roadmap of the OS. I don't want quick-n-dirty changes when a proper system solution is required. That degrades the OS.

Thank you all for all the feedback.
While this solve the problem on a small system using netinit thread, it is far from what is needed as a general solution So I am closing this.

@davids5 I think the first two patches are reasonable, could you remove the last one?


Publishing generated apps to artifactory

As a developer, I'd like to publish generated apps to artifactory.

generated apps are published to artfiactory through the Jenkins build now.


UserGuide.adoc: Update /order find command title

In this PR,

I updated the /order find command title
Added 1 more example for /order find

View it here

Pull Request Test Coverage Report for Build 278

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.61%

Totals

Change from base Build 274:
0.0%

Covered Lines:
2793

Relevant Lines:
3188

💛  - Coveralls


About results

Thanks for your work,
I noticed that you saved the query of each account on the same txt, I think it would be very good if you could save it in a different folder for each account.
Example:
\Desktop\Stalkiana\user-messi\11.11.23 result.txt
\Desktop\Stalkiana\user-ruipedrodev\10.11.23 result.txt
and I don't know why, but the number of followers and followers is usually a few people less than the normal number, what is the reason for this?

Hi, thanks for bringing up the issue. The suggestion to store logs in different folders is a great idea, and I will implement that.
Regarding the number of followers being slightly less than the normal count, that is not expected behavior. Could you provide an example of an account where this issue occurs?

this is a slightly different example but as you can see below, it's said following count change but in result.txt said no change.

Thanks for reporting the bug, I just finished fixing it, it turns out the way I was fetching the users was not consistent and sometimes I fetched the same user twice, now it should be fixed.

Thanks !!!


Spec Changes and MakeSummaries for Workspace skeleton

[Current coverage][cc-pull] is 79.30% (diff: 100%)

Merging [#235][cc-pull] into [develop][cc-base-branch] will not change coverage

@@            develop       #235   diff @@
==========================================
  Files            24         24
  Lines          3711       3711
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           2943       2943
  Misses          768        768
  Partials          0          0

Powered by Codecov. Last update [d0495eb...f78fbc8][cc-compare]
[cc-base-branch]: https://codecov.io/gh/kbase/data_api/branch/develop?src=pr
[cc-compare]: https://codecov.io/gh/kbase/data_api/compare/d0495ebe8caaac3ff12fc187d0983f31aa11aa80...f78fbc818d4d9d5977401605d74d9b8ca72fe38b?src=pr
[cc-pull]: https://codecov.io/gh/kbase/data_api/pull/235?src=pr


Feature Request: Multi-Analyzer Support

Thread to track enablement of multiple analyzers measuring against a single system under test.
@araghun would you mind pasting the link to the new sheet in the drive once it's created? I think it might be easier to collaborate exploration/design of this feature there and track status here.
Also, can assign this ticket to me.

Living doc: Multi Analyzer to Multi SUT Proposal Scratchpad
Current POC Branch: multi-analyzer-support
Will attempt to line up remaining work-items for POC in the coming weeks.


Votes should expire

A blockchain parameter should be added that enforces a minimal freshness to votes that are counted and handle cases of lost keys or apathetic votes hanging around.
https://bitsharestalk.org/index.php/topic,18109.0.html

This issue was moved to bitshares/bitshares-core#73


Prevent deadlock in TestPersistableChannelQueue

There is a potential deadlock in TestPersistableChannelQueue due to attempting to
shutdown the test queue before it is ready.
Fix #17709
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

Unit test fails

Codecov Report

Merging #17717 (9028d3c) into main (8fdc524) will increase coverage by 0.10%.
The diff coverage is 47.17%.

@@            Coverage Diff             @@
##             main   #17717      +/-   ##
==========================================
+ Coverage   45.35%   45.46%   +0.10%
==========================================
  Files         799      802       +3
  Lines       89062    89233     +171
==========================================
+ Hits        40393    40568     +175
+ Misses      42163    42130      -33
- Partials     6506     6535      +29

Impacted Files
Coverage Δ

cmd/admin.go
0.00% <0.00%> (ø)

cmd/doctor.go
0.00% <ø> (ø)

cmd/hook.go
7.79% <ø> (+0.01%)
:arrow_up:

cmd/mailer.go
0.00% <ø> (ø)

cmd/web.go
0.00% <ø> (ø)

models/consistency.go
8.84% <0.00%> (ø)

models/db/engine.go
32.00% <ø> (ø)

models/error.go
38.81% <ø> (+0.33%)
:arrow_up:

models/issue_list.go
71.80% <ø> (ø)

models/org.go
76.72% <0.00%> (-0.46%)
:arrow_down:

... and 94 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 72b0882...9028d3c. Read the comment docs.


Bump quay.io/helmpack/kubectl-helm-az to 0.1.2

Signed-off-by: Scott Rigby<EMAIL_ADDRESS>

we may as well go with #133 😄


[8.x] Add elseisset directive

Currently we have only @isset and @endisset directives and we need @elseisset directive (that's the goal of this PR)
@isset($foo)
  //
@elseisset($bar)
  //
@endisset

Just use @if and @else

I just thought while @isset() exist we need a @elseisset() so we can use it with it 👀

@taylorotwell I just thought while @isset() exist we need a @elseisset() so we can use it with it eyes

@taylorotwell I just thought while @Isset() exist we need a @elseisset() so we can use it with it 👀

@taylorotwell I just thought while @isset() exist we need a @elseisset() so we can use them together 👀


Example fails

Hello,
The example does not seem to work fine.
Using
var NXT = require('nxt-api');

// start your NRS client first
var API = new NXT.API('http://<server>:<port>');

API.getNextBlockGenerators().then(function (answer) {
    console.log(answer);
});

API.rsConvert({ account: 'x' }).then(function (answer) {
    // should error
}, function (error) {
    console.log(error);
});

does return { errorCode: 4, errorDescription: 'Incorrect "account"' }. That makes sense.
However:
var NXT = require('nxt-api');

// start your NRS client first
var API = new NXT.API('http://<server>:<port>');

API.getNextBlockGenerators().then(function (answer) {
    console.log(answer);
});

API.rsConvert({ account: '<Valid Numeric Account ID>' }).then(function (answer) {
    // should error
}, function (error) {
    console.log(error);
});

returns
Unhandled rejection Error: [object Object]
    at Object.ensureErrorObject (/Users/will/Dropbox/Projects/nxt/api-tests/node_modules/nxt-api/node_modules/bluebird/js/main/util.js:226:20)
    at Promise._rejectCallback (/Users/will/Dropbox/Projects/nxt/api-tests/node_modules/nxt-api/node_modules/bluebird/js/main/promise.js:427:22)
    at /Users/will/Dropbox/Projects/nxt/api-tests/node_modules/nxt-api/node_modules/bluebird/js/main/promise.js:445:17
    at Request._callback (/Users/will/Dropbox/Projects/nxt/api-tests/node_modules/nxt-api/nxt-api.js:83:33)
    at Request.self.callback (/Users/will/Dropbox/Projects/nxt/api-tests/node_modules/nxt-api/node_modules/request/request.js:344:22)
    at Request.emit (events.js:110:17)
    at Request.<anonymous> (/Users/will/Dropbox/Projects/nxt/api-tests/node_modules/nxt-api/node_modules/request/request.js:1239:14)
    at Request.emit (events.js:129:20)
    at IncomingMessage.<anonymous> (/Users/will/Dropbox/Projects/nxt/api-tests/node_modules/nxt-api/node_modules/request/request.js:1187:12)
    at IncomingMessage.emit (events.js:129:20)
    at _stream_readable.js:908:16
    at process._tickCallback (node.js:355:11)

hello, can you dump the object that is giving the error? Since it seems that an exception is being thrown (instead of an standard rejection), it seems to be coming from request library. you can rethrow the error doing:
API.rsConvert({ account: '<Valid Numeric Account ID>' }).done(function (answer) {
    // should error
});

If I look at the variable value in bluebird/util.js, I get:
{"errorCode":1,"errorDescription":"Incorrect request"}
If I print the value of answer however, I do get an object:
{ requestProcessingTime: 1,
  accountRS: 'the address',
  account: 'the numeric id' }

I think I'll have to create a custom Error to differentiate from API errors and request errors (it may return 500 depending on how fast you query NXT)

:+1:
I am not running the client locally but on a RPi. When I connect the the client on the RPi via http, that works fairly fast. I don´t think it can timeout.

ah, then I think it's a problem with CORS setting, try changing it to see if helps

Ah good point. I will check that out and let you know.

Hi,
I am testing in node and not in the browser so I don´t think this is CORS related. I also tested on localhost and I get the same issue.
Overall, I can call the API without any issue except for the getNextBlockGenerators method.
This is the method that fail in the sample.


Changes to remove hardcoded optional annotation

1.0.5 with this change is available now : https://repo1.maven.org/maven2/io/github/dvgaba/easy-rules/1.0.5/


Vertical scrolling broken in Chrome (redmine_lightbox2 v0.3.2)

Hi.
Using commit 2dd50dd4588a1c187e73e856100a61c50600e819 version 0.3.2 it breaks vertical scrolling in /issues/* in Chrome browser, in Firefox works fine, didn't test in other browsers.
Redmine is v3.3.3.
If I disable the plugin scroll works again.
Can you fix this?
Many thanks

What do you mean by "is broken"? When being in lightbox mode, vertical scrolling is disabled in all browsers. in Chrome and in Firefox and others...

Here is how to reproduce:
go to any page which contains more content than can be displayed at once in your browser (e.g. http://www.redmine.org/issues or http://www.redmine.org/issues/25616).
Plugin hooks into /issues/ page and thus something does with the scroll in Chrome. In Firefox everything works just fine.

I'm afraid, but I cannot reproduce this. I'm running Redmine 3.3.3.stable.16539 with the default theme and a bunch of plugins, and also tried another theme, but I can't reproduce. Scroll is working fine, also for "long" issues...
Did you try using another theme?
Or disabling all plugins but the lightbox plugin?
I tested with Chrome 57.0.2987.133 and Firefox 52.0.2 on Windows 10.

I have the same configuration and ... the same behavior with chrome 59, can you do something ?

I'm also have that problem with installed plugins agile

The same problem here ...

I will try to have a look soon...
Did anybody also report this issue to the author of the redmine agile plugin?
If someone could do so, that would be great!

Confirmed. I can reproduce the issue. However, I cannot solve this by myself. I wrote an email to the redmine_agile developers as their plugin throws an exception in the chrome console which may be the root cause of the problem.

Still waiting for an answer of the redmine_agile developers.
In the meantime I pushed some fixes and changes for general Redmine 3.4 support (#38). Go check out the new master branch...

This is going to be fixed by the redmine_agile team. It's an issue with their CSS. So I'm closing this ticket here.
In the meantime, you can use the workaround provided by their team:

Hello Tobias,
To fix the issue with the scrolling please do the next.
/assets/stylesheets/redmine_agile.css
html {
-  overflow-y: inherit !important;
+  /* overflow-y: inherit !important; */
}

Then restart your web-server.
And we will investigate the second issue also and surely fix it.
Regards, Irina


The sensor need ... should be needs - nit

No data.
The sensor need to be in range of a gateway
would be
the sensor needs to

Opened a new one in the correct repo: https://github.com/ruuvi/station.localization/issues/85

I did not find the string in localization repo master, so it seems to be Web UI issue

@ojousima it's in the dev branch

I had no idea this would be anything other than an insignificant effort, sorry.


Different rendering in react-native on IOS compared to web and android

There is difference between react-native on ios and web in rendering, and from what I see, it's rendered correctly on web and not in react-native. On android, it's rendered like on web, so it's ios that's causing the problem.

I've been too busy with work lately, haven't had any time to look at this yet...

I've been too busy with work lately, haven't had any time to look at this yet...

Ok, thx for response

Tried to analyze this a bit, couldn't find the culprit or even any good candidate reason. Everything looks correct code wise. I think it must be something with the CGPathAddArc and CGContextSetLineDash api.
https://github.com/react-native-community/react-native-svg/blob/1427dd6822382f34337fe08b1a21d4e18612e91f/ios/Shapes/RNSVGCircle.m#L41-L49
https://github.com/react-native-community/react-native-svg/blob/1427dd6822382f34337fe08b1a21d4e18612e91f/ios/RNSVGRenderable.m#L383-L391

At least here is a workaround you can use:
import React from 'react';
import {View, StyleSheet} from 'react-native';
import Svg, {Path, Circle} from 'react-native-svg';
// http://xahlee.info/js/svg_circle_arc.html
const cos = Math.cos;
const sin = Math.sin;
const π = Math.PI;
const f_matrix_times = ([[a, b], [c, d]], [x, y]) => [
  a * x + b * y,
  c * x + d * y,
];
const f_rotate_matrix = x => [[cos(x), -sin(x)], [sin(x), cos(x)]];
const f_vec_add = ([a1, a2], [b1, b2]) => [a1 + b1, a2 + b2];

const f_svg_ellipse_arc = (cx, cy, rx, ry, t1, Δ, φ) => {
  /* [
  returns a SVG path element that represent a ellipse.
  cx,cy → center of ellipse
  rx,ry → major minor radius
  t1 → start angle, in radian.
  Δ → angle to sweep, in radian. positive.
  φ → rotation on the whole, in radian
  url: SVG Circle Arc http://xahlee.info/js/svg_circle_arc.html
  Version 2019-06-19
   ] */
  Δ = Δ % (2 * π);
  const rotMatrix = f_rotate_matrix(φ);
  const [sX, sY] = f_vec_add(
    f_matrix_times(rotMatrix, [rx * cos(t1), ry * sin(t1)]),
    [cx, cy],
  );
  const [eX, eY] = f_vec_add(
    f_matrix_times(rotMatrix, [rx * cos(t1 + Δ), ry * sin(t1 + Δ)]),
    [cx, cy],
  );
  const fA = Δ > π ? 1 : 0;
  const fS = Δ > 0 ? 1 : 0;
  return (
    'M ' +
    sX +
    ' ' +
    sY +
    ' A ' +
    [rx, ry, (φ / (2 * π)) * 360, fA, fS, eX, eY].join(' ')
  );
};

const tau = π * 2;
const segments = 4;
const evenly = tau / segments;
const space = evenly * 0.03;
const sweepAngle = evenly - space * 2;

export default () => (
  <View style={styles.container}>
    <Svg
      width="300"
      height="300"
      strokeWidth="7"
      fill="transparent"
      viewBox="0 0 42 42">
      <Circle
        cx="21"
        cy="21"
        fill="#fff"
        strokeWidth="0"
        r="15.91549430918954"
      />
      <Path
        d={f_svg_ellipse_arc(
          21,
          21,
          15.91549430918954,
          15.91549430918954,
          0,
          sweepAngle,
          3 * evenly + space,
        )}
        stroke="red"
      />
      <Path
        d={f_svg_ellipse_arc(
          21,
          21,
          15.91549430918954,
          15.91549430918954,
          0,
          sweepAngle,
          space,
        )}
        stroke="#ffc34d"
      />
      <Path
        d={f_svg_ellipse_arc(
          21,
          21,
          15.91549430918954,
          15.91549430918954,
          0,
          sweepAngle,
          evenly + space,
        )}
        stroke="blue"
      />
      <Path
        d={f_svg_ellipse_arc(
          21,
          21,
          15.91549430918954,
          15.91549430918954,
          0,
          sweepAngle,
          2 * evenly + space,
        )}
        stroke="green"
      />
    </Svg>
  </View>
);
const styles = StyleSheet.create({
  container: {
    backgroundColor: 'black',
    justifyContent: 'center',
    alignItems: 'center',
    flex: 1,
  },
});

More reusable solution:
import React from 'react';
import {View, StyleSheet} from 'react-native';
import Svg, {Path, Circle} from 'react-native-svg';

const {cos, sin, PI} = Math;
const tau = 2 * PI;

const multiply = ([a, b, c, d], x, y) => [a * x + b * y, c * x + d * y];

const rotate = x => [cos(x), -sin(x), sin(x), cos(x)];

const add = ([a1, a2], b1, b2) => [a1 + b1, a2 + b2];

const ellipse = (cx, cy, rx, ry, t1, delta, theta) => {
  /* [
  returns a SVG path element that represent a ellipse.
  cx,cy → center of ellipse
  rx,ry → major minor radius
  t1 → start angle, in radian.
  delta → angle to sweep, in radian. positive.
  theta → rotation on the whole, in radian
  url: SVG Circle Arc http://xahlee.info/js/svg_circle_arc.html
  Version 2019-06-19
   ] */
  delta = delta % tau;
  const rotMatrix = rotate(theta);
  const [sX, sY] = add(multiply(rotMatrix, rx * cos(t1), ry * sin(t1)), cx, cy);
  const [eX, eY] = add(
    multiply(rotMatrix, rx * cos(t1 + delta), ry * sin(t1 + delta)),
    cx,
    cy,
  );
  const fA = delta > PI ? 1 : 0;
  const fS = delta > 0 ? 1 : 0;
  const path = ['M', sX, sY, 'A', rx, ry, (theta / tau) * 360, fA, fS, eX, eY];
  return path.join(' ');
};

function Segments({
  x = 0,
  y = 0,
  r = 1,
  rx = r,
  ry = r,
  colors,
  t1 = 0,
  phase = 0,
  spacing = 0.03,
} = {}) {
  const segments = colors.length;
  const evenly = tau / segments;
  const space = evenly * spacing;
  const offset = space + phase;
  const sweepAngle = evenly - space * 2;
  return colors.map((color, i) => (
    <Path
      key={color}
      stroke={color}
      d={ellipse(x, y, rx, ry, t1, sweepAngle, i * evenly + offset)}
    />
  ));
}

export default () => (
  <View style={styles.container}>
    <Svg
      width={300}
      height={300}
      strokeWidth={7}
      fill="transparent"
      viewBox="0 0 42 42">
      <Circle
        cx={21}
        cy={21}
        fill="#fff"
        strokeWidth={0}
        r={15.91549430918954}
      />
      <Segments
        x={21}
        y={21}
        r={15.91549430918954}
        colors={['#ffc34d', 'blue', 'green', 'red', 'yellow']}
      />
    </Svg>
  </View>
);

const styles = StyleSheet.create({
  container: {
    backgroundColor: 'black',
    justifyContent: 'center',
    alignItems: 'center',
    flex: 1,
  },
});

Tried it, works fine, thx!


Data not showing in the report

Hi,
Files are generated when I ran the Nunit console application but when i open the index file and navigating further not showing any data. it is just showing empty values
Please help me on this.
Regards,
Nagarjuna

Hi @nagarjunagandra !
What version of Chpr.Core and Ghpr.NUnit are you using?
Are there any errors in browser console when you are opening report?
Are there any exception .txt files the report folder?
Did you follow these instructions to view report?

Hi,
I am using <IP_ADDRESS> version for both Ghpr.core and Ghpr.Nunit.
I could not find any errors in browser console and exceptions in report folder.

Ok, can you attach this report here? Or maybe create simple demo tests to reproduce this issue.
Is it possible that tests fail inside SetUp?

Hi ,
I am able to see the report only in Firefox but not on other browsers.
Please help me to how to enable in other browsers as well.

The reason why it doesn work is that you need to enable access to local files from javascript, which is blocked by default in all browsers. you will see some errors inside browser console. You can serach for browser options to enable access to local files (like security.fileuri.strict_origin_policy parameter for Firefox).
Another way is to create local server (like Apache Tomcat) and put this report on server (you can set output folder to '\yourserver\reportfolder')


Feature/selectable image card accessibility

This change: (check at least one)

[x] Adds a new feature
[ ] Fixes a bug
[ ] Improves maintainability
[ ] Improves documentation
[ ] Is a release activity

Is this a breaking change? (check one)

[ ] Yes
[x] No

Is the: (complete all)

[x] Title of this pull request clear, concise, and indicative of the issue number it addresses, if any?
[x] Test suite(s) passing?
[x] Code coverage maximal?
[ ] Changeset added?
[ ] Component status page up to date?

What does this change address?
A clear and concise description or a direct link to an issue [e.g. #15]
If adding a feature, a rationale for its addition goes here.
How does this change work?
A clear, detailed description of how this change addresses the issue.
This could be a bullet list if there are several moving parts.
Additional context
Add any other context here.

I branched off an older branch and things are getting too confusing. Making a new PR based of of the latest develop


Updating the project structure

Currently, the project structure is not clean. There are a lot of folders that have been created for logos, wireframes which I think needs to be placed inside static files so to make our project structure look clean and good.

I am interested in solving this.


drivers: timer: nrf_rtc_timer: SYS_CLOCK_TICKS_PER_SEC too high for when CONFIG_KERNEL_TICKLESS=n

NRF chips set it indiscriminately to 32768. This makes sense for tickless builds, but will cause far too frequent of interrupts when using a non-tickless build.
The Zephyr Kconfig default definition is to use 100 for non-tickless (and 10000 for tickless). I'd suggest switching the default NRF non-tickless rate to 128.

FYI
@nordic-krch
@jciupis
@pkwiek
@ankuns

@ewalt1 thanks for creating the issue, I've opened a PR.


Data / API resources

See website
Plus

list of APIs https://github.com/toddmotto/public-apis

@yochannah would it be useful to add in any data/API resources we find helpful here?

Here's a fun one @trallard shared a while ago...
https://swapi.co/ - star wars api!


fix: properly dedupe react hooks names

Right now react hook's names are not being deduped even if the option dedupeOperationSuffix is enabled. This PR fixes it.

Thank you @anderson-amorim !


Release 0.17 blog post

Blog post for release 0.17
Signed-off-by: Milan Klanjsek<EMAIL_ADDRESS>
Fixes #1989

@GuessWhoSamFoo just realized: do we need to hook up this to home page? Not sure what's the process for that.

The home page automatically shows the three most recent blog posts - see netlify deploy preview.

@GuessWhoSamFoo fixed and squashed all commits.


[QUERY] how to get  scope for creating AzureResourceManager from azure china

Query/Question
config like official domo:
scope
 @Bean
    public AzureResourceManager azureResourceManager(){
        AzureProfile profile = new AzureProfile(AzureEnvironment.AZURE);
        ClientSecretCredential clientSecretCredential = new ClientSecretCredentialBuilder()
                .clientId("XXX")
                .clientSecret("XXX")
                .tenantId("XXX")

                // authority host is optional
//                .authorityHost("<AZURE_AUTHORITY_HOST>")
                .build();
        return AzureResourceManager
                .authenticate(clientSecretCredential, profile)
                .withScope()
                .withDefaultSubscription();
    }

any help will be welcome ,TKS
dependency:
<dependency>
      <groupId>com.azure.resourcemanager</groupId>
      <artifactId>azure-resourcemanager</artifactId>
      <version>2.9.0</version>
  </dependency>
  <dependency>
      <groupId>com.azure</groupId>
      <artifactId>azure-core-http-netty</artifactId>
      <version>1.9.1</version>
  </dependency>
  <dependency>
      <groupId>com.azure</groupId>
      <artifactId>azure-identity</artifactId>
      <version>1.4.0</version>
  </dependency>

Why is this not a Bug or a feature Request?
A clear explanation of why is this not a bug or a feature request?
Setup (please complete the following information if applicable):

OS: [e.g. iOS]
IDE: [e.g. IntelliJ]
Library/Libraries: [e.g. com.azure:azure-core:1.16.0 (groupId:artifactId:version)]

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Query Added
[x] Setup information Added

@Neil-Boyle
You do not need to specify the scope.
When create the AzureProfile, choose AzureEnvironment,AZURE_CHINA, which should have the preconfigured cloud endpoints and scopes.

it works , ths!


Add timestamps to post and comment

Changes
The UserTag now accepts an optional timestamp that can be rendered as a "relative date". This means that we use the most accurate single unit possible. Implementing this on our side would have been a bit of a pain, so I grabbed some utility code from a popular React repository and ported it's component over to React Native.

Issue ticket number and link
#134

addressed :D


News website sorts articles in the wrong order

Newest items are on page 2, navigated to by clicking the "older" link at the bottom. Please figure out how to resort these.
@mirons-google Did you have insight into a fix?

Yea, @red2k18 and I had an information sharing session on where to look to debug the issue.

#2723


[charts/portal] US805864 Removed env related to pssg from p-d and p-e containers

Description of the change
US805864 Removed env related to pssg from p-d and p-e containers
Benefits

Drawbacks

Applicable issues

fixes #

Additional information
Checklist

[X] Chart version bumped in Chart.yaml according to semver.
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [charts/gateway])
[] If the chart contains a values-production.yaml apart from values.yaml, ensure that you implement the changes in both files

https://jenkins-ren-58.ren.broadcom.net/job/portal/job/tests/job/portalnightly/job/RAT_DockerCompose/job/IntegrationTest_main_trigger/723/


Fallback image for missing/corrupted asset

Hi, my app received in the last couple of days placeholder image instead of the real image trhought the API with the text:

This is a fallback image used in the case where there is a missing/corrupted asset on apod.nasa.gov. Image source: https://en.wikipedia.org/wiki/File:Black_Hole_in_the_universe.jpg

Wondering is this just a temporary glitch, is it possible to detect these temporary placeholders somehow in my app, or not return these placeholders by the API at all? Thanks.

This is interesting !!

This is an attempt to fill the gap when the APOD page changes their syntax or doesn't have an image that the API can programmatically grab. The APOD API is working fine here, just the page the photo is being scraped form isn't formatted properly for it to work.
What alternative behavior would you want?
Tagging @evantayloryates

@JustinGOSSES the alternative behavior is simple, either return valid json and image, or just return NotFound 404 status code until the image and json for the given day is available.
Returning placeholder image is really not a good practice, especially when the placeholder response is completely valid. Many apps or websites is caching those responses and they won't ask for another image for given day after receiving the placeholder response.

It seems different people want different things then.
Others have expressed an interest in getting some image back when the APOD site (totally separate from this API and run by totally different people) changes the structure of their HTML enough that an image can't be found by the APOD API.
Unfortunately, the admin's of this API don't have the time to support immediate development of code changes to fix edge cases. Changes often occurs weeks or months later. Public contributions are of course always welcome!
If you want to be able to detect when the API fails to find an image on APOD, you can check the image and text. The image above and associated text will remain the same any time the APOD API can't find them on the APOD site. You could modify your code to look for this response and change behavior accordingly. Would this work for you?

@JustinGOSSES Just a note, returning valid response, that is in fact a placeholder, goes against RESTful API design best practices. If some websites want to show placeholder, they should do it on their side.
Right now I've added checks to my proxy server and to my apps to look for the placeholder string and if it's found in the response, the response is considered invalid. It works, but it's not a nice solution.

Personally I think that when you ask for a date interval and one of the dates within that interval happen to be invalid/missing/whatever, it shouldn't throw a 404 for the entire request. But it also shouldn't return a placeholder. It should just not return that entry. It's up to the application developers to check if a specific date they asked for within that interval exists or not. It should not, like @martinsuchan mentions, be up to the developers to check if the string is a placeholder or not.
When you ask for a specific date (like 20-06-10), it should return a 404. And when you ask for a random APOD and it happens to land on a missing image, it should re-pick and return only existing images instead.
Thoughts?

@LeMorrow yeah, exactly

asking for specified date which has no APOD data available? -> 404
asking for interval of APOD images, some of them not available? -> return json array only with data for available dates
asking for random APOD day? -> return one of existing APOD dates. This request should never fail.

Thanks for laying it out clearly like that @martinsuchan
@evantayloryates do you have an opinion on this?

I think the requested behavior here is good and pretty simple to implement. I will work on a patch for this over the next few days, as time allows, and update the thread when I've pushed to production.

@martinsuchan @LeMorrow @hemangshah @JustinGOSSES I pushed the fix to production. Here are a couple of URLs you can check to test the behavior (2020-06-10 is a known missing APOD date):
http://apodapi-env-test.us-east-1.elasticbeanstalk.com/v1/apod/?date=2020-06-10
https://api.nasa.gov/planetary/apod?api_key=DEMO_KEY&start_date=2020-06-08&end_date=2020-06-12

@martinsuchan @LeMorrow @hemangshah @JustinGOSSES I pushed the fix to production. Here are a couple of URLs you can check to test the behavior (2020-06-10 is a known missing APOD date):
https://api.nasa.gov/planetary/apod?api_key=DEMO_KEY&date=2020-06-10
https://api.nasa.gov/planetary/apod?api_key=DEMO_KEY&start_date=2020-06-08&end_date=2020-06-12

I'm going to close this issue. If there are any other behavior changes you'd like implemented, feel free to open up a new issue


hook in the minePlots plugin so it doesn't teleport players to other plots

Plugin Version:

Request:

How/Why:

Added support in #130


I dont know what its incompatible with but it crashes when i try and boot up my game, i play on Fabric.

You provided no information like Mc version, mod version, crash report or anything.


Install script assumes useradd to be available

I am not sure, whether this constitutes a proper issue, but here we go:
I tried installing Zulip server according to the guide at https://zulip.readthedocs.io/en/stable/production/install.html#step-2-install-zulip. I changed --certbot to --self-signed-cert. This is because I am installing inside a virtual machine to test things out.
./scripts/setup/install --self-signed-cert --email='my-email-is-here' --hostname='zulip.localhost.com'

In addition I changed my /etc/hosts/ file to resolve localhost.com to <IP_ADDRESS>.
My VM has a Debian installed:
$ lsb_release -a
No LSB modules are available.
Distributor ID:	Debian
Description:	Debian GNU/Linux 10 (buster)
Release:	10
Codename:	buster

I downloaded the latest release zulip-server-latest.tar.gz:
$ sha256sum zulip-server-latest.tar.gz
50a13655cb30ba30b0bfaf8e5a1fea46a73b5f26864f503c1cdf204f9f09da52  zulip-server-latest.tar.gz

I get the following error when running the install command as written above:
...
Extracting to /srv/zulip-yarn...
Adding to /usr/bin
Successfully installed Yarn 1.22.4!
+ mkdir -p /etc/zulip
+ id -u zulip
+ useradd -m zulip --home-dir /home/zulip
./scripts/lib/install: line 377: useradd: command not found

Zulip installation failed (exit code 127)!

The install process is designed to be idempotent, so you can retry after resolving whatever issue caused the failure (there should be a traceback above). A log of this installation is available in /var/log/zulip/install.log

This is probably, because on Debian there is no useradd.
I did not see a "compatible with the following distros" in the readme, so this might be an oversight or it might be by design, as Ubuntu or another distro might be assumed.
Perhaps the script can be made to work with Debian as well?
If you need more info, just ask and I will try to provide. Thanks!

@zulipbot claim

@ZelphirKaltstahl
Until thr PR gets reviewed, you can solve this issue by updating the $PATH variable using this command: export PATH=$PATH:/usr/sbin/useradd

I believe useradd is present in /usr/sbin on Debian, as part of the required passwd package.  Maybe you're running the installer from a context where /usr/sbin isn't part of $PATH.  What exact virtualization technology are you using?

I believe useradd is present in /usr/sbin on Debian, as part of the required passwd package. Maybe you're running the installer from a context where /usr/sbin isn't part of $PATH. What exact virtualization technology are you using?

I am running Debian 10 inside Virtualbox version 6.1.16_Ubuntu r140961.

@ZelphirKaltstahl
Until thr PR gets reviewed, you can solve this issue by updating the $PATH variable using this command: export PATH=$PATH:/usr/sbin/useradd

I'll try this.

Running with root privieges:
PATH=$PATH:/usr/sbin; \
    ./scripts/setup/install \
    --self-signed-cert \
    --email='my-email-is-here' \
    --hostname='zulip.localhost.com'

worked and gets me to:
Please visit the following secure single-use link to register your
new Zulip organization:

    https://zulip.localhost.com/new/...

Thanks for your help with this.

Tagging as a release goal, since we've seen this issue more than once and it shouldn't be hard to fix.


[stable/prometheus-operator] fix annotations typo

What this PR does / why we need it:
Sorry, @vsliouniaev I had a typo in the annotations line. The reason I put this PR together is because we need to exclude Spinnaker from versioning this configmap and this is done through annotations.
Which issue this PR fixes
This fixes a typo
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/mychartname])

/ok-to-test
/lgtm


Change users.py to location.py

What were you initially searching for in the docs?

I was searching on the new Split operations with Router
Is this related to an existing part of the documentation? Please share a link
yes,
https://awslabs.github.io/aws-lambda-powertools-python/1.22.0/core/event_handler/appsync/#split-operations-with-router
Describe how we could make it clearer

Let's assume you have app.py as your Lambda function entrypoint and routes in users.py, this is how you'd use the Router feature.

I think you meant to say

Let's assume you have app.py as your Lambda function entrypoint and routes in location.py, this is how you'd use the Router feature.

Because the example given below is from the location.py file
If you have a proposed update, please share it here
It should be

Let's assume you have app.py as your Lambda function entrypoint and routes in location.py, this is how you'd use the Router feature.

Err, my bad, when copying things over before today’s release. Will send an
update shortly — thanks a lot for catching it!!
On Wed, 17 Nov 2021 at 14:25, boring-cyborg[bot] @.***>
wrote:

Thanks for opening your first issue here! We'll come back to you as soon
as we can.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/awslabs/aws-lambda-powertools-python/issues/830#issuecomment-971580462,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZPQBFKS36Q34PAEJDBEILUMOULHANCNFSM5IG6HJ3A
.

My pleasure 😊. Closing now


Binary not installed

Just installed latest version and the command line binary is not installed in path. I think it's because it's not listed in gemspec as executables

Just pushed 1.0.23 which adds it back into executables, not sure why it was removed or not added.  I've also limited the version of slop which it uses due to breaking changes in v4. (also fixes a bug in the report command of the executable)
Any issues feel free to reopen, i've not used the executable for a while myself.


Add test kitchen with test policy, dummy tests and fix version in metadata

Added .kitchen.yml, test/recipes and test/policyfiles
Fixed version in metadata.rb to match CHANGELOG and github versions

Should test/policyfiles/teamcity_node.rb be in this cookbook?

@Brantone actually, I think we're good enough to drop the extra kitchen steps completely (and test them inside the teamcity cookbook anyway); will update the PR to include only generic parts


Weapon and Nano potions disappear when killing an enemy

After purchasing a nano or weapon potion, and then going to fight an enemy, defeating the enemy results in your potions being removed. Buying at least one nano/weapon potion seems to add all the previously bought potions back to your inventory. Just thought this is something that should be looked into
Example of the potion bug: https://youtu.be/aFEnPcr6W1Y

This bug is due to the packet struct sP_FE2CL_REP_REWARD_ITEM being incomplete when sent. Setting m_iBatteryN and m_iBatteryW would solve it.


Address lines not working on the template

An error has been added on address lines inputs id, creting a difference between the inputs and the template.
The error have been removed.
Commit bf9be0896c2f28b5fffe179f4caf7da96f54ad3d


battery icon not showing

Describe the bug
Since a few days the battery icon is not showing, updated to HA2024.2 last week maybe related?

How to reproduce

Expected behavior

YAML configuration

# please paste here your card config (before backticks below; do not remove them)
type: custom:auto-entities
filter:
  include:
    - attributes:
        device_class: '*battery*'
      state: <101
  exclude:
    - entity_id: '*ramses*'
    - entity_id: '*battery_st*'
    - entity_id: '*binary_se*'
    - entity_id: '*nuc0*'
    - entity_id: '*solis*'
show_empty: true
card:
  type: custom:battery-state-card
sort:
  method: state
  reverse: false
  numeric: true
state_filter:
  - operator: <
    value: 100```

***Entity debug data***
<!-- Add "debug: true" in your card configuration and paste below entity data from the debug output -->

```json

Dev console errors

Screenshots

Version
V3.2

Please use the debug: true in the card config and paste here the entity data.
Pay attention to the version shown there too (I suspect that you don't have v3.2 but the old one cached by the browser)

If after adding  debug: true you don't see something like the above then it means you are not on the latest card version.


Java Runtime Sample Crashes In Lambda Class not found: helloworld.App

Description
The sample Java8 Lambda function that sam creates when calling sam init -r java8 works locally but does not work in Lambda. It throws an error:
{
"errorMessage": "Class not found: helloworld.App",
"errorType": "java.lang.ClassNotFoundException"
}
Steps to reproduce
sam init -r java8 -n java-import-ssl-cert
cd java-import-ssl-cert/
sam build
sam package --template-file template.yaml --output-template-file serverless-output.yaml --s3-bucket mredactedw
sam deploy --template-file serverless-output.yaml --stack-name java8-import-ssl-cert
Observed result
sam local invoke works but when ran on Lambda it throws:
{
"errorMessage": "Class not found: helloworld.App",
"errorType": "java.lang.ClassNotFoundException"
}
Expected result
I expect the function to work properly in Lambda. I also expect sam local invoke to always return the same thing as official AWS Lambda.
Additional environment details (Ex: Windows, Mac, Amazon Linux etc)

OS: Mac
sam --version: SAM CLI, version 0.31.1

@michaelgamazon This is due to you passing --template template.yaml in the sam package command. You are telling the command to use that specific template, which as un-built functions. You should instead remove the --template option on sam package so the built template/artifacts can be properly picked up.
Closing as this is working as expected given the template was passed to the sam package command.


proxy.config.http.number_of_redirections with hit_stale

In testing ATS 9, one of our properties overrides proxy.config.http.number_of_redirections. When the object is stale in cache, ATS returns the 302 instead of chasing the redirect. It appears very reproducible for the 1 property that uses it.

I'm trying to look at this, but am a novice in this area, help welcome.

There are a few recent commits around this
#6589
#6561
#4962
#4902
perhaps more too...

I think I've got a simple test that reproduces the problem.  Works in 8.1.x but not on master.
https://github.com/ywkaras/trafficserver/commit/7c1545f99de9d30c11a6ff813a675fdaa0282408

It looks like the change in behavior is due to the added call to HttpTransact::is_cache_hit() is the cause of the change in behavior:  https://github.com/apache/trafficserver/blob/bd089772b735bc374c0f0f4fedccabd1e6a30cda/proxy/http/HttpSM.cc#L8104 .
This call is not present in the 8.1.x code:  https://github.com/apache/trafficserver/blob/502ee35dd85a9f3f9936f886fc4e4d56507b7fbb/proxy/http/HttpSM.cc#L7959 .

This is the PR that introduced the above change:  https://github.com/apache/trafficserver/pull/4289 .  @SolidWallOfCode any insights?


Documentation for Debian / Ubuntu packaging

I'm seeking documentation for Debian and Ubuntu packaging.

Sorry, forgot to reply to this. We're moving our release/packaging process to GoReleaser in the coming weeks which will make our packaging step simpler. I'll keep the ticket open so we can notify you once we've updated the way we package our releases. If I get a chance I'll have a look at our scripts/makefile to see if I can extract the deb packaging step.

Not forgotten about but taking a little bit longer than planned. We've added this #245 issue to our roadmap to tackle next quarter. We've been busy migrating our OHIs first (like you can see on nri-redis latest release) so hopefully we will get to the agent soon.

We don't plan to work on a non-alpine based image in the near term, releasing the agent directly from Github (GHA) should be completed this month


Fix unsafe requests when connecting to website using HTTPS

Right now users have to allow unsafe requests when accessing the website over HTTPS, because some resources (d3?) are always loaded over HTTP.
Either we force all resources over HTTPS or use the more dynamic // (current protocol).

 #30 Fix unsafe requests when connecting to website using HTTPS

I think this is done, waiting for @J-Son89 to confirm.

As far as I know we resolved this issue, I will commit it shortly.


Add lnd v0.12.0-beta compatibility, allow single macaroon

Depends on https://github.com/lightninglabs/pool/pull/202 and the final release of lnd v0.12.0-beta.
Adds compile time and run time compatibility with lnd v0.12.0-beta.
Also pulls in the latest version of lndclient which allows us to cleanly block on startup until lnd is unlocked.
We can also use a single macaroon now instead of needing to specify all subserver macaroons. Either the admin.macaroon is used if --remote.lnd.macaroondir is specified. Or a custom one (must contain all required permissions!) can be specified with --remote.lnd.custommacaroonpath.
Fixes https://github.com/lightninglabs/lightning-terminal/issues/155.

Your latest rebase includes an updated lnd proto which breaks the CI build. The new proto requires an update to the frontend Typescript types. I think it's fine to keep the old proto in this PR as I will update it separately.

Your latest rebase includes an updated lnd proto which breaks the CI build. The new proto requires an update to the frontend Typescript types. I think it's fine to keep the old proto in this PR as I will update it separately.

The new RPC compile check prevents us from using an old version. Perhaps we need a way to override that.
Or I can update the testdata, should be straightforward.
The check will fail until we reference the final version of lnd anyway since it tries to download it from an URL without the rc tag.

The new RPC compile check prevents us from using an old version. Perhaps we need a way to override that.
Or I can update the testdata, should be straightforward.
The check will fail until we reference the final version of lnd anyway since it tries to download it from an URL without the rc tag.

I agree that the RPC compile check will fail, but the build is also failing on the "typescript compile" step because the frontend code is not updated for v0.12 protos. This also breaks my local docker build because the frontend won't build successfully.
Can you either remove the proto update or update the sampleData.ts file to fix the frontend build?

I agree that the RPC compile check will fail, but the build is also failing on the "typescript compile" step because the frontend code is not updated for v0.12 protos. This also breaks my local docker build because the frontend won't build successfully.
Can you either remove the proto update or update the sampleData.ts file to fix the frontend build?

I fixed the sampleData.ts with a temporary commit. Will remove that once we've tagged the final lnd v0.12.0-beta version.

I fixed the sampleData.ts with a temporary commit. Will remove that once we've tagged the final lnd v0.12.0-beta version.


Expose :authority pseudo-header to Obj-C API.

This exposes the :authority pseudo-header value to the Obj-C API. We have a backend load balancer that performs virtual host routing. This means we need to be able to override the host/authority header.
This relates to https://github.com/grpc/grpc/pull/10858

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

Thanks for your pull request. The automated tests will run as soon as one of the admins verifies this change is ok for us to run on our infrastructure.

@jayantkolhe who can review this?

maybe @makdharma or @jcanizales could take a look?

test this please

Thanks for reviewing! I addressed these issues.

Thanks!

Jenkins: test this please

Jenkins: test this please

Failed tests:
Basic Tests Mac - #10656


[FAB-17173] Update doc to reflect chaincode starts after commit

Signed-off-by: NIKHIL E GUPTA<EMAIL_ADDRESS>Type of change

Documentation update

Description
Content in docs reflects older flow where the first invoke of the chaincode. The chaincode lifecycle has been updated to so that the chaincode starts after the chaincode definition has been committed to the channel.

Merged @pamandrejko updates to the numbering to this PR

Going close this PR to cleanup the commit history and the poorly done merge.


Roll pinned nightly toolchain

This PR is on branch roll-nightly.

#2047

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 89.46%. Comparing base (d3157cc) to head (bdd518f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2047   +/-   ##
=======================================
  Coverage   89.46%   89.46%
=======================================
  Files          16       16
  Lines        5838     5838
=======================================
  Hits         5223     5223
  Misses        615      615

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


BINEXP-115, BINEXP-121

refreshing linked_tasks after solving task from article view
sorting linked_tasks by TaskStatus and ordinal

Nice job!


Fix ubuiltins.complex documentation

Our documentation and typing for ubuiltins.complex currently does not match the implementation.
Let's wait for https://github.com/micropython/micropython/pull/7623 and then document based on decision made upstream.

Let's wait for https://github.com/micropython/micropython/pull/7623 and then document based on decision made upstream.

This was updated and merged in https://github.com/micropython/micropython/pull/8799.
Since this is a rare feature, we can just wait for it to get included when we rebase onto the next release, even if that doesn't make it into V3.2.


Remove end tags from HTML elements

There are some HTML elements throughout the theme that have end tags where they shouldn't. For instance, the first two link tags in this code block have /> and the last link tag only has >. While browsers seem to render these tags correctly, my team uses a post-processing script that doesn't work with these incorrectly closed elements. This PR removes the incorrect forward slash (/) from all void elements that do not require them.
Sources:

"Void elements only have a start tag; end tags must not be specified for void elements." - link
Definition of void elements - link
Definition of end tags - link
MDN pages for meta elements, link elements, and br elements

@jorisvandenbossche, I've updated my PR in 6fba2d9 to revert the changes to tags that are handled by the parent theme. If you can kick off CI again for me here, I believe it should pass this time.

passed! :tada:

anything else preventing this from being merged?

Nope - Looks good to me, thanks for the addition! 🎉


Please add hide tray icon setting

Form minimalism

if we hide the tray icon, there would be no way to get to settings? Or am I misunderstanding

Closing as duplicate of https://github.com/microsoft/PowerToys/issues/765


Silence a rustdoc warning

The 'generated' module is not public, but generated::* is publicly
re-exported. This constellation triggers the private_doc_tests lint.
Silence this with #[allow].
Fixes: https://github.com/psychon/x11rb/issues/310
Signed-off-by: Uli Schlachter<EMAIL_ADDRESS>

Would it be a good idea adding cargo doc to CI?

In my repositories I use travis to deploy the result of cargo doc to github pages, which would be helpful for people on master.

Assuming there comes a time without lots of API breakage, I'd expect docs.rs to be good enough. Personally, I always look stuff up in the source code anyway. (I also did that with xcb/cairo while writing the code in #328.)
Do you think setting up GH pages is really worth it? Would you use it that much? Isn't "stop breaking the API" a better goal? ;-)


Tags/Extra context

Allow for tag context and extra context in angular-raven per the raven-js docs here - https://raven-js.readthedocs.org/en/latest/usage.html#passing-additional-data

thanks!

No problem, also my bad here, I was pulling my own versions in our repo before making this pull request, so the bower.json file actually merged in with version 0.5.13, you'll just want to correct it when you re-tag/version.


Connection not available error when trying to override static query() method and add some initial calls to every created query builder

This is the code I prototyped:
require('sqlite3');
const Knex = require('knex');
const {Model} = require('objection');

const knexSqlite = Knex({
  client: 'sqlite',
  connection: ':memory:'
  useNullAsDefault: true
});

class Child extends Model {
  static get tableName() {
    return 'Child';
  }
}

class Parent extends Model {
  static get tableName() {
    return 'Parent';
  }

  static query(connection) {
    // passing .where here causes  no database connection available for a query error
    return super.query(connection).where('id', 0);
    // this works though
    // return super.query(connection);
  }

  static get relationMappings() {
    return {
      child: {
        relation: Model.HasOneRelation,
        modelClass: Child,
        join: {
          from: 'Parent.id',
          to: 'Child.parentId'
        }
      }
    };
  }
}

async function main() {
  await knexSqlite.schema.createTable('Parent', t => {
    t.increments('id').primary();
    t.string('name');
  });

  await knexSqlite.schema.createTable('Child', t => {
    t.increments('id').primary();
    t.string('data');
    t.integer('parentId');
  });

  const results = await Parent.query(knexSqlite).insertGraphAndFetch([
    {name: 'foo parent', child: {data: 'foo'}},
    {name: 'bar parent', child: {data: 'bar'}}
  ]);

  console.log('Res', results);
}

main()
  .catch(e => console.log('Got error', e))
  .then(() => knexSqlite.destroy());

And it throws:
Got error Error: no database connection available for a query. You need to bind the model class or the query to a knex instance.
    at QueryBuilder.knex (/Users/mikaelle/Projects/stuffs/node_modules/objection/lib/queryBuilder/QueryBuilderOperationSupport.js:61:15)
    at convertArgs (/Users/mikaelle/Projects/stuffs/node_modules/objection/lib/queryBuilder/operations/ObjectionToKnexConvertingOperation.js:29:24)
    at KnexOperation.onAdd (/Users/mikaelle/Projects/stuffs/node_modules/objection/lib/queryBuilder/operations/ObjectionToKnexConvertingOperation.js:22:17)
    at QueryBuilder.addOperation (/Users/mikaelle/Projects/stuffs/node_modules/objection/lib/queryBuilder/QueryBuilderOperationSupport.js:147:33)
    at QueryBuilder.where (/Users/mikaelle/Projects/stuffs/node_modules/objection/lib/queryBuilder/QueryBuilderBase.js:181:17)
    at Function.query (/Users/mikaelle/Projects/stuffs/test.js:23:36)
    at tableInsertion (/Users/mikaelle/Projects/stuffs/node_modules/objection/lib/queryBuilder/graphInserter/inserter.js:19:49)
    at Promise.all.Object.keys.map.tableName (/Users/mikaelle/Projects/stuffs/node_modules/objection/lib/queryBuilder/graphInserter/GraphInserter.js:54:16)
    at Array.map (<anonymous>)
    at GraphInserter._executeNextBatch (/Users/mikaelle/Projects/stuffs/node_modules/objection/lib/queryBuilder/graphInserter/GraphInserter.js:48:26)
    at GraphInserter._executeNormalBatches (/Users/mikaelle/Projects/stuffs/node_modules/objection/lib/queryBuilder/graphInserter/GraphInserter.js:35:17)
    at Promise.try (/Users/mikaelle/Projects/stuffs/node_modules/objection/lib/queryBuilder/graphInserter/GraphInserter.js:21:35)
    at tryCatcher (/Users/mikaelle/Projects/stuffs/node_modules/bluebird/js/release/util.js:16:23)
    at Function.Promise.attempt.Promise.try (/Users/mikaelle/Projects/stuffs/node_modules/bluebird/js/release/method.js:39:29)
    at GraphInserter.execute (/Users/mikaelle/Projects/stuffs/node_modules/objection/lib/queryBuilder/graphInserter/GraphInserter.js:21:23)
    at InsertGraphOperation.onAfter1 (/Users/mikaelle/Projects/stuffs/node_modules/objection/lib/queryBuilder/operations/InsertGraphOperation.js:98:26)

Is this a bug or am I just doing something wrong?

I traced the problem here:
https://github.com/Vincit/objection.js/blob/e944679cbce440764c59f4e3bc9b176921fdc3af/lib/queryBuilder/graphInserter/inserter.js#L19
let insertQuery = tableInsertion.modelClass.query().childQueryOf(builder);

builder has knex conenction in context, but it is not passed to the query() which creates new query builder.
Not sure if this even should work like this at all.

What if you pass ...args to the super query call instead of the first argument?

The same error remains after this:
  static query(connection) {
    console.dir('ARGS');
    console.dir(arguments);
    // passing .where here causes  no database connection available for a query error
    return super.query(...arguments).where('id', 0);
    // this works though
    // return super.query(connection);
  }

I passed builder.knex() to  couple of places in code and then example code did seem to execute correctly:
https://github.com/Vincit/objection.js/blob/e944679cbce440764c59f4e3bc9b176921fdc3af/lib/queryBuilder/graphInserter/inserter.js#L19
=>
let insertQuery = tableInsertion.modelClass.query(builder.knex()).childQueryOf(builder);

and
https://github.com/Vincit/objection.js/blob/e944679cbce440764c59f4e3bc9b176921fdc3af/lib/queryBuilder/operations/InsertGraphAndFetchOperation.js#L34
=>

    return modelClass
      .query(builder.knex())
      .childQueryOf(builder)
      .whereIn(builder.fullIdColumnFor(modelClass), ids)
      .eager(eager)
      .then(models => {
        return this.isArray ? models : models[0] || null;
      });

Ofc I have no idea if that introduces other problems... I think I'll check next code in .childQueryOfif I'm able to create some workaround.
Any ideas how to go around this problem?

I was able to implement this using onBuild instead of directly passing stuff to query.

The connection should get passed along with childQueryOf. I'll reopen this as it seems like a bug

This seems to be a corner case bug when using models without globally binding them to knex and overriding query(). The database connection is passed just after calling query() using childQueryOf. onBuild hook is a good solution.


Threadpool isolation for MDC

Related: https://github.com/ShiftLeftSecurity/common/pull/126 https://github.com/ShiftLeftSecurity/codepropertygraph/pull/1527 slack #mdc-propagation https://github.com/ShiftLeftSecurity/product/issues/8624 https://github.com/ShiftLeftSecurity/codescience/pull/5602
Theoretically this wants to be its own minimal repo, but that comes with red tape.

Superseded.


Release 1.13.0

Include swagger1st 0.24.0

👍

:

👍

Current coverage is 68.31% (diff: 100%)

No coverage report found for master at 998faed.

Powered by Codecov. Last update 998faed...34463ec

Available at: http://repo1.maven.org/maven2/org/zalando/stups/friboo/1.13.0/

👍

👍


Fix syntax errors from previous merge

Made edits to troublshooting.rst, fixing a change introduced in PR #307.
"Fixed" a issue with the enumeration in the same document, which also
seemed to be introduced at the same time
Signed-off-by: Katrina Prosise<EMAIL_ADDRESS>

Docs for 0559a5cb02b2207ec4e072e5f9621e3731dd59d2 are browesable at: https://ci.foundries.io/projects/fio-docs/builds/1351/docs/artifacts/html/index.html


Minor API fixes to get fsList disks

See https://github.com/kubevirt/kubevirt/pull/11099.
The issue will hold a blocker label while a 1.2 backport is pending.
/release-blocker release-1.2

/release-blocker release-1.2

Fixed by https://github.com/kubevirt/kubevirt/pull/11349.


⚠ Bofrost.at has degraded performance

In 176a91c, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1682 ms

Resolved: Bofrost.at performance has improved in 08eea47 after 1 hour, 2 minutes.


how could this CircularBarPager be used in Fragment?

Hi,friend.
When I use CircularBarPager in activity like the sample, it is ok. But when it is used in Fragment, the AndroidStduio logcat shows the message: I/System.out: Binary XML file line #9: Binary XML file line #9: Error inflating class xx.xxx.xxx.xxx.CircleBarPagerLib.CircularBarPager
Thanks Firstly!

Hello @caoyl2015
we use it also in Fragment and it's working fine.
Are you sure you're importing the CirucularBarPager fine?
Try to copy paste the whole stacktrace please?
Cheers

Thank your reply! I have been working on it for a night. Finaly, I was wrong. Two things should be told.
Firstly,I use this
"mCircularBarPager = (CircularBarPager) getFragmentManager().findFragmentById(R.id.frag_real).getView().findViewById(R.id.frag_circularBarPager);",
Now ,I change it to
"View view = inflater.inflate(R.layout.fragment_realtimemonitor, container, false);
mCircularBarPager = (CircularBarPager) view.findViewById(R.id.frag_circularBarPager);
" in onCreateView for getView().
Secondly, I use the source of viewpagerindicator, I forgot remove the
"compile 'com.viewpagerindicator:library:2.4.1@aar'"  from the build.gradle of library.
When I fix them, It works well.


S3 - allow choosing the storage type for upload

Summary
Allow s3 integration to choice of the bucket storage settings
s3.StorageClassStandard (STANDARD)
s3.StorageClassIntelligentTiering (INTELLIGENT_TIERING)
s3.StorageClassStandardIa (STANDARD_IA)
s3.StorageClassOnezoneIa (ONEZONE_IA)
s3.StorageClassGlacier (GLACIER)
s3.StorageClassGlacierIr (GLACIER_IR)
s3.StorageClassDeepArchive (DEEP_ARCHIVE)
s3.StorageClassReducedRedundancy (REDUCED_REDUNDANCY)

Use Cases
Upload s3 artifacts using INTELLIGENT_TIERING as default storage class.

Message from the maintainers:
Love this feature request? Give it a 👍. We prioritise the proposals with the most 👍.

sources to change:
https://github.com/argoproj/pkg/blob/c1082084a390704e94b92cd9d91792ffd7226307/s3/s3.go#L1
https://github.com/argoproj/argo-workflows/blob/main/workflow/artifacts/s3/s3.go
https://github.com/argoproj/argo-workflows/blob/main/examples/output-artifact-s3.yaml

@templarfelix raise a PR


Incorrect work of Collection.IsTheSameAs matcher

in case when compared collections have duplicated values the matcher does not consider them

done in 76b14d8302fd


Windows doc fix

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
Fixes  and close #1874
What are the significant changes in functionality due to this change request?
Documentation will build on windows by running C:\projects\raven\doc> ./make_docs.sh
References in manuals are now generated
Plugins manual is now generated

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[x] 1. Review all computer code.
[x] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[x] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[x] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[x] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[x] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[x] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[x] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[x] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

checklist is good.


[FALSE-POSITIVE]  dlvr.it

Domains or links
dlvr.it
More Information
How did you discover your web site or domain was listed here?
Tried to visit a site served via dlvr.it
Have you requested removal from other sources?
This is the only source
Additional context
This is a URl shortener, not a phishing site.
(duplicate of #200 but I could not reopen it there. this was previously fixed.)

Closing as it is already into the whitelist list.


Incorrect results on s390x (-march=zEC12 -mtune=z13, ZARCH_GENERIC)

I maintain the fflas-ffpack package for the Fedora Linux distribution.  There is currently a push in Fedora to migrate from atlas and the reference blas implementation to openblas.  However, the fflas-ffpack test suite failed on s390x when built with openblas.  The issue is tracked here: https://bugzilla.redhat.com/show_bug.cgi?id=1619074.
I found that the openblas test suite itself reported multiple failures when built on s390x, but did not return a nonzero exit code; the issue was therefore overlooked as the openblas build did not fail.  The openblas test failures can be seen here: https://kojipkgs.fedoraproject.org//packages/openblas/0.3.2/3.fc29/data/logs/s390x/build.log.  Here is the first error in the logs:
OPENBLAS_NUM_THREADS=1 OMP_NUM_THREADS=1 ./sblat3 < ./sblat3.dat
 TESTS OF THE REAL             LEVEL 3 BLAS
 THE FOLLOWING PARAMETER VALUES WILL BE USED:
   FOR N                   0     1     2     3     7    31
   FOR ALPHA             0.0   1.0   0.7
   FOR BETA              0.0   1.0   1.3
 ROUTINES PASS COMPUTATIONAL TESTS IF TEST RATIO IS LESS THAN   16.00
 RELATIVE MACHINE PRECISION IS TAKEN TO BE  1.2E-07
 SGEMM  PASSED THE TESTS OF ERROR-EXITS
 SGEMM  PASSED THE COMPUTATIONAL TESTS ( 17496 CALLS)
 SSYMM  PASSED THE TESTS OF ERROR-EXITS
 SSYMM  PASSED THE COMPUTATIONAL TESTS (  1296 CALLS)
 STRMM  PASSED THE TESTS OF ERROR-EXITS
 ******* FATAL ERROR - COMPUTED RESULT IS LESS THAN HALF ACCURATE *******
           EXPECTED RESULT   COMPUTED RESULT
       1      0.186813          0.373626
 ******* STRMM  FAILED ON CALL NUMBER:
    506: STRMM ('L','U','N','U',  1,  1, 1.0, A,  2, B,  2)        .

There are over 100 such failures.  I have only looked at the first dozen or so, but in each case the computed result appears to be exactly double the expected result.  Fedora packages built for s390x are built with gcc -march=zEC12 -mtune=z13.  The openblas package selects TARGET=ZARCH_GENERIC.

Could you, please, attach linked build log before it disappears?
Looks like same TLS problem being approached at #1742

I am not yet convinced that the test failures are caused by the new thread-local storage allocator, but could you try current "develop" branch where I just merged the aforementioned PR #1742 (that reverts to the old allocation code unless OpenBLAS is built with -DUSE_TLS) ?
And do I get it right that this is an emulator, not actual zarch hardware ?

It's from a real hw (z13), but zEC12 gives same error messages. It looks like as a "feature" of the generic kernel. When the z13 kernel is used, there are no such errors (https://koji.fedoraproject.org/koji/buildinfo?buildID=1133326 built the z13 kernel by mistake). Fedora needs to stick to the generic kernel as we support running the distro on zEC12 or newer hw.

Thanks - as the generic kernel is pure C it may be possible to reproduce this on more mundane hardware.

No issue building on x86_64 with the KERNEL.ZARCH_GENERIC in place of its usual KERNEL.generic file, so not that simple unfortunately. Did earlier Fedora builds all use z13, or is this a recent failure with the generic kernel (which would indeed implicate the new memory.c as the most likely recent change) ?

AFAIK this is a long time issue

xTRMM failing was once seen as fallout from #1419 (0.3.0), but that was reverted by #1564 in mid-May, well before 0.3.2. I see nothing suspicious in the build log.

So the issue has indeed been around for a while, this is 0.2.20 build log from January when the builders were still z13.
https://kojipkgs.fedoraproject.org//packages/openblas/0.2.20/4.fc28/data/logs/s390x/build.log

Interesting, thanks. As far as I can tell, much the same setup is used for GEMM/TRMM on ARMV8, but with USE_TRMM=1 defined in kernel/Makefile.L3 (This is also set when CORE is Z13, might be worthwile to add it for ifeq($(CORE), ZARCH_GENERIC) as well.)

with
diff --git a/kernel/Makefile.L3 b/kernel/Makefile.L3
index b37e536e..81ee93c1 100644
--- a/kernel/Makefile.L3
+++ b/kernel/Makefile.L3
@@ -20,6 +20,10 @@ ifeq ($(ARCH), arm64)
 USE_TRMM = 1
 endif

+ifeq ($(ARCH), zarch)
+USE_TRMM = 1
+endif
+
 ifeq ($(TARGET), LOONGSON3B)
 USE_TRMM = 1
 endif
@@ -44,10 +48,6 @@ ifeq ($(CORE), POWER8)
 USE_TRMM = 1
 endif

-ifeq ($(CORE), Z13)
-USE_TRMM = 1
-endif
-

I see no more those "half accurate" errors


Run Detail Page: PD steps known

Overview
This is a nice-to-have addition and can and should be scoped out if necessary.
Because the protocol steps are not just anticipated, we are 100% certain what they will be, instead of displaying "Anticipated Steps," if the protocol has not started yet, display "Protocol Steps" and once the protocol has started, update the copy to "Next Steps"
figma link: https://www.figma.com/file/JCQNOTTK9tTmYPhRqeOfSj/Milestone-0:-Protocol-Upload-Revamp?node-id=5629%3A145096
Acceptance Criteria

[ ] when a JSON protocol is loaded, the commands list label switches between "Protocol Steps" and "Next Steps" instead of just "Anticipated Steps"

Punting on this. We'll rework this section in the unified app/5.1.0