bool() and nonzero() of Response

Do you really like this?
class Response(...):
    def __bool__(self):
        """Returns true if :attr:`status_code` is 'OK'."""
        return self.ok

    def __nonzero__(self):
        """Returns true if :attr:`status_code` is 'OK'."""
        return self.ok

I had this code:
def foo(request, response=None):
    ....
    if response:
        do_something_with_response

It took some time to understand why this did not work like the way I wanted.
I guess you don't want to change the current behaviour since it could break a lot of existing code.
Question: Do you really like this?

Thanks for this report!
In the future, before filing issues against open source projects, please take the time to investigate whether they have seen an issue before. If you had searched "bool" in our issue tracker the first result you'd have seen would have been for #2002, which already tracks this issue. You'll notice that we agree with you that it's bad, and have removed it in our branch for the next major release of Requests. However, it's a breaking change, so we cannot ship it before the 3.0.0 release, which is currently not timetabled.
So, in answer to your question: no, we don't like it. There are many things we don't like, but we have far too many users to change these things without a good run-up.
(While we're here: you should use this experience to get comfortable with using if x is not None: rather than if x:. Because of Python's booleanness rules, when you're checking a Nullable value like this one you should always explicitly look for the None singleton rather than rely on the fact that None is falsy. Most Python linters will flag this behaviour if they see it.)

I went a little bit overboard when I wrote that. It's totally unnecessary when Resoponse.ok exists.
Sent from my iPhone

On Nov 28, 2016, at 4:55 AM, Cory Benfield<EMAIL_ADDRESS>wrote:
Thanks for this report!
In the future, before filing issues against open source projects, please take the time to investigate whether they have seen an issue before. If you had searched "bool" in our issue tracker the first result you'd have seen would have been for #2002, which already tracks this issue. You'll notice that we agree with you that it's bad, and have removed it in our branch for the next major release of Requests. However, it's a breaking change, so we cannot ship it before the 3.0.0 release, which is currently not timetabled.
So, in answer to your question: no, we don't like it. There are many things we don't like, but we have far too many users to change these things without a good run-up.
(While we're here: you should use this experience to get comfortable with using if x is not None: rather than if x:. Because of Python's booleanness rules, when you're checking a Nullable value like this one you should always explicitly look for the None singleton rather than rely on the fact that None is falsy. Most Python linters will flag this behaviour if they see it.)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Composer update issues

while updating i got the below error please check
using => laravel 5.5
Package phpoffice/phpexcel is abandoned, you should avoid using it. Use phpoffice/phpspreadsheet instead.
Generating optimized autoload files

Illuminate\Foundation\ComposerScripts::postAutoloadDump
@php artisan package:discover

In ProviderRepository.php line 208:
Class 'Carbon\Laravel\ServiceProvider' not found
Script @php artisan package:discover handling the post-autoload-dump event returned with error code 1

Please visit a Forum like Laracasts in order to solve such issues.


error on a function with DiffEqOperators

The following code (MWE) results in
LoadError: MethodError: objects of type TypeVar are not callable
Both Win10 and Mac, Julia 1.4.2, freshly updated all packages (SparsityDetection 0.3.2, DiffEqOperators 4.10.0)
using DiffEqOperators
using SparsityDetection, SparseArrays

nknots = 10
h = 1.0/(nknots+1)

const Δ = CenteredDifference(2, 2, h, nknots)
u0 = rand(nknots, nknots)

function st!(du,u,p,t)
   du .= 0.0
   du[1,:] .= Δ*Dirichlet0BC(Float64)*view(u,1,:)
   return nothing
end

outp = similar(u0)
# st!(outp,u0, nothing, 0.0)
sparsity_pattern = jacobian_sparsity(st!,outp,u0, nothing, 0.0)

The quickest way to get the sparsity pattern here would be to just directly make SparseMatrixCSC(Δ*Dirichlet0BC(Float64)) and the sparsity is then clear from there. That should work on any DiffEqOperators problem.

Chris, thank you for the advice. I have in the meanwhile just got the sparsity pattern numerically. It may take a few minutes to calculate, but then I save it onto the disk, so I have to do it only once for every mesh size.


Set headers in request responses

For example, indicate that these responses can be cached by intermediate layers (it's a REST api!)

Well technically GET is for retreiving information and POST is for sending it so symanticly we're not supposed to use get. I think just using the cache server-side is probably enough for now


arco init hello-arco-pro failed with spawnSync npm.cmd EINVAL(.arco-cli/init.js)

[ ] I'm sure this does not appear in the issue list of the repository

Basic Info

Package Name And Version<EMAIL_ADDRESS>Framework version: vue 3.2
Browser: chrome<IP_ADDRESS>
Reproduction Link: https://github.com/arco-design/arco-design-pro-vue/blob/main/.arco-cli/init.js

Extra info
The patch is deployed in 18.20.2, 20.12.2, 21.17.3 and also 22.0.0.  If the node.js version is lower than just mentioned, this bug will not be triggered.
What is expected?
fix it.
Steps to reproduce
You must pass shell: true in the option on Windows environment if you need to spawn or spawnSync program ended with .bat or .cmd extension.
https://github.com/nodejs/node/issues/52681#issuecomment-2076426887

node.js出现漏洞，修正了之后部署了新版本，在windows上调用spawn和spawnSync接口，必须传递shell: true参数。否则就会报错。

@sHow8e @PengJiyuan

所以最后的解决方案是什么 有人帮忙解答一下么

所以最后的解决方案是什么 有人帮忙解答一下么

切换下node.js版本，第一次初始化规避下。使用  18.20.2, 20.12.2, 21.17.3 之前的版本就可以了


monorepo integration

This PR addresses GitHub issue: #1467 .
Briefly describe the changes proposed in this PR:

modules from rdf4j-storage, rdf4j-tools, and rdf4j-testsuite re-integrated in main rdf4j repo, preserving history where possible
all core library modules are in the core directory, all tools/applications are in tools. Testsuite and compliance tests are top-level and are compiled and run after all libraries and tools are compiled (so they are becoming more system/end-to-end tests).
maven version in pom uses patch snapshot version again (3.0.1-SNAPSHOT).
Verified that tests run and succeed, and that SDK and onejar can be built (with javadoc included)

There are likely still minor glitches with this setup, and it will need some further tinkering, but this should be good enough to get us going with development on the master branch again.

This is a huge PR of course, so I'm not expecting a full review, but I'd like it if you could give it a glance, maybe try it out locally, and give me a green tick if you're happy for this to be merged.
note that Jenkins has not yet been reconfigured to deal with this.

mvn clean install -DskipTests worked just fine (with JDK8), so that's already a good sign, nice work
So I'll give it a shot with tests enabled...

"mvn clean install" worked perfectly on Debian Linux with OpenJDK8 (8u222)

Minor point: you (currently) have to include -Pcompliance for it to execute the compliance tests. We can discuss whether this is a good approach or not but I thought I'd give a heads up at least.


Solaredge ToU Function charging Battery, disable EVCC during forced battery charge

When EVCC recognizes exceeded PV energy, it is charging. But when you have activated the Battery Charge from Grid function, Solaredge wants to charge the battery during cheap times when not enough sun is availbe based on further calculation. But then evcc uses "excessed" energy while parallel solaredge is charging the battery from Grid. But EVCC does not recognize this. So maybe there could be an setting when Solaredge is Charging from Grid, EVCC disables the PV Excess so the battery can be charged with the pv energy instead of the Grid?
Hope you know what i mean

You can only have one controlling system.


feat: add integration with react-hook-form

https://github.com/mll-lab/react-components/pull/176/commits/86b1ff2d189aff831ea1f0a63b213047180fe1ec
Can we remove it like committed? In my opinion it doesn't help when using Typescript, but leads to propblems with props definitions in stories

Let's change eslint in https://github.com/mll-lab/eslint-config - please add a pull request there.


Alpha cut off number gets clamped at 2 digets, cutting off value

Get a texture that a value with only one significant digit is needed to see a difference from .5


[flawhud] Add back the customization to hide the health/ammo box

The hud changed, maybe it's because certain fix that fixed the colorized number when cornered mode bug.
it's kinda blocking my view, and it's kinda harder to see the health numbers because it gotten further away from the crosshair.
are there any settings to disable the black box? and maybe revert the distance? i mean yeah i can adapt, but i'll ask anyway
from this:

to this:

(or the other way around, i don't rember which picture because it entered as a text :))

OS: Windows 10, x64
App Version: 3.1
Selected HUD :FlawHud

Colored Box option added back to the editor. Thanks.


Where to install?

After failing to get snapcastr to run I am wondering if I misunderstood the readme.md.
Is this supposed to run as a client on my laptop - or does this have to be installed on the server that runs  snapserver?

i sincerely hope this has resolved itself by now. it's a web-app for (already running) snapserver application.


flux rift crash

versions:
Thaumcraft-1.12.2-6.1.BETA13
forge-1.12.2-<IP_ADDRESS>76
this happened when I summoned a flux rift
https://pastebin.com/Y1EViZ8i
this happened when I tried to log back in the world
https://pastebin.com/9YHBCgSj

One or more of your mods is/are using a custom graphics library. I see you have CTM in there but you might have others I don't recognize.
Can you try again after removing CTM and other mods that change graphics? It's to find which one is incompatible.

The same thing with Flux Rift. I wrote /summon thaumcraft:fluxrift and after that got crash-report.
But there was ArrayIndexOutOfBoundsException.
https://pastebin.com/btEMJT4T
Also happened when I tried to enter the world again.
Thaumcraft-1.12.2-6.1.BETA13
forge-1.12.2-<IP_ADDRESS>55

Like I said, one of your mods is changing the way Minecraft does its graphics and Thaumcraft doesn't support it, so there's an incompatibility.
For now the solution is to find which mod does this and remove it.

@Geethebluesky This lib is used by Thaumcraft itself :)

Duh.  I must be high on life.

Did we ever establish the solution to this? i have the same thing reoccurring in beta16, and would rather nto lose the world i have been working on.
"Duh.  I must be high on life." seems rather rude and entirely unhelpful in solving the problem. What is obvious ot some people may not be so to others

@Orinai That wasn't meant as a general comment, it was in reply to my own comment that wondered about something that was pointed out as obvious after the fact.


Github link wrong

Just went to create a PR to update a page and it took me to the wrong github repo - http://github.com/pivotal-cf/docs-operating-pas/tree/2.7/external-user-management.html.md.erb.  Please update to use this repo.  Thanks

Hi @skibum55! Thanks for reporting this. Which page were you on when you clicked the link?

Hi @skibum55
We rebuild the page links back in November and, based on our tests, the issue seems to have been resolved. If you come across this again or another issue, please let us know. Thanks!


POC: Force JMAP endpoints to check user existence upon each query.

TODO: Implement this behavior using extension-jars instead.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 4 committers have signed the CLA.:white_check_mark: samiulsami:white_check_mark: shn27:white_check_mark: tamalsaha:x: chibenwaYou have signed the CLA already but the status is still pending? Let us recheck it.


Bug: Incompatible with in-source:false configuration

The Vite Plugin has a hardcoded dependency on files being in source. This makes it impossible to use the in-source: false package spec
import { runCli } from "../cli/RescriptRelayRouterCli__Commands.mjs";
import { transformManifest } from "./RescriptRelayVitePlugin__ManifestTransform.mjs";

https://github.com/zth/rescript-relay-router/blob/af5487a81e5143477fe6c1020d13fbdd549ead5b/packages/rescript-relay-router/vite-plugins/RescriptRelayVitePlugin.mjs#L8

this also prevents using any other extension than .mjs. Converting this file to rescript should fix both issues :)

This should be fixed in the latest versions. Reopen if still a problem.


Allow Kubeflow node inputValues to consume outputPath outputs

Fixes #2197
Requires https://github.com/elyra-ai/pipeline-editor/pull/175

[x] Add additional tests for KFP meta typing
[x] Migrate existing test resource pipelines to new version

What changes were proposed in this pull request?
Updating KFP pipeline construction and behavior. Prior to this change any component parameter deemed an inputvalue could only accept either default values provided in the component spec/definition or a value provided by the user. This change adds a third option where it is able to consume a outputpath value provided by a parent component in the pipeline

How was this pull request tested?
Existing unit tests were modified to check the new component property structure.
Additional kfp operator and checks were added for kfp component meta typing

Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

In this scenario (upstream node has no output) it should not be possible to select the parent as source:

Component sources:

https://raw.githubusercontent.com/ptitzler/examples/how-to-byo-cr/component-catalog-connectors/connector-template/todo_catalog_connector/dummy-component.yaml
https://raw.githubusercontent.com/elyra-ai/examples/master/component-catalog-connectors/kfp-example-components-connector/kfp_examples_connector/resources/download_data.yaml

A pipeline configured as follows

triggers this error:
Traceback (most recent call last):
  File "/opt/anaconda3/envs/akchin-value/lib/python3.7/site-packages/tornado/web.py", line 1704, in _execute
    result = await result
  File "/opt/anaconda3/envs/akchin-value/lib/python3.7/site-packages/elyra/pipeline/handlers.py", line 112, in post
    response = await PipelineValidationManager.instance().validate(pipeline=pipeline_definition)
  File "/opt/anaconda3/envs/akchin-value/lib/python3.7/site-packages/elyra/pipeline/validation.py", line 172, in validate
    response=response)
  File "/opt/anaconda3/envs/akchin-value/lib/python3.7/site-packages/elyra/pipeline/validation.py", line 334, in _validate_node_properties
    pipeline_definition=pipeline_definition)
  File "/opt/anaconda3/envs/akchin-value/lib/python3.7/site-packages/elyra/pipeline/validation.py", line 442, in _validate_custom_component_node_properties
    node_param_value = node_param['NestedEnumControl'].get('value')
KeyError: 'NestedEnumControl'

File "/opt/anaconda3/envs/akchin-value/lib/python3.7/site-packages/elyra/pipeline/validation.py", line 442, in _validate_custom_component_node_properties
node_param_value = node_param['NestedEnumControl'].get('value')
KeyError: 'NestedEnumControl'

Edit: it's sufficient to create a one node pipeline and not select an input source to trigger this.

Pushed a fix for this.
Looking into the parsing no output bug. That should be handled by the parsing and UI i think

@ptitzler - rebased and both scenarios should be resolved now.


[BUG] Arrows directions in RTL (Right to Left)

Describe the bug
Arrows used for navigation are always pointing to the same direction as LTR, regardless of the UI direction (LTR or RTL). This can be confusing for users navigating the UI in RTL mode, where the expectation is for the arrows to point to the different direction.

Steps To Reproduce

Set the UI direction to RTL (Right-to-Left) in your Refinejs project.
Navigate to any page containing one or more arrows used for navigation
Observe that the arrows are all pointing to the same direction as LTR.

Expected behavior
In RTL mode, the arrows used for navigation should be flipped so that they point to the opposite direction. This would be consistent with the user's expectation when navigating a right-to-left UI.
Packages
{
  "dependencies": {
    "@refinedev/antd": "^5.38.1",
    "@refinedev/cli": "^2.16.31",
    "@refinedev/core": "^4.49.2",
    "@refinedev/react-router-v6": "^4.5.9",
    "@refinedev/simple-rest": "^5.0.6",
    "@uiw/react-md-editor": "^3.19.5",
    "antd": "5.16.5",
    "react": "^18.0.0",
    "react-dom": "^18.0.0",
    "react-router-dom": "^6.8.1"
  },
  "devDependencies": {
    "@types/node": "^18.16.2",
    "@types/react": "^18.0.0",
    "@types/react-dom": "^18.0.0",
    "@vitejs/plugin-react": "^4.2.1",
    "typescript": "^5.4.2",
    "vite": "^5.1.6"
  }
}

Additional Context
No response

Can I take a look at it?
And if possible can you provide the files in the codebase I can look at to understand this problem better?

@ShatilKhan You just need to run the rtl example provided at
https://github.com/refinedev/refine/tree/master/examples/customization-rtl
npm create refine-app@latest -- --example customization-rtl

@alicanerdurmaz Can you please assign this issue, I think I can fix it!!

@BatuhanW I have raised a PR for this issue https://github.com/refinedev/refine/pull/5984 please review


layer-wise learning rate

In latest keras, is there a implementation for set different learning rate for each layer?
If yes, please give me a example of how to use it. thanks.

I might be mistaken, but I think you will get a faster response over at StackOverflow. The issues here are meant for bugs or specific feature requests.
If you are making such a request; please update your question and state what you propose more clearly =)


fix: updated types in setClientInCurrentClass

:tada: This PR is included in version 3.39.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Provide a way to Export Shapes #53

Export Shapes with a new copy button

@ludovicm67 could you review & merge when ok?

@BenjaminHofstetter can you validate if this is ok on https://spex.zazuko.com? Where would I see that?

Maybe your app didn't update

Jep shift-reload helped, thanks


Don't display Android details in mobile launcher if build support not installed

Description
Only show android section of mobile launcher if android build support is installed.
Tests

[x] windows with android
[x] windows without android
[ ] mac with android
[x] mac without android

Documentation

[x] changelog

Primary reviewers
If your change will take a long time to review, you can name at most two primary reviewers who are ultimately responsible for reviewing this request. @ mention them.

Should we be doing this for iOS as well?


While trying to create a resource for metered Graph API using the az resource create --resource-group cmdlet fails consistently

Hello everyone, everything good?
I need to use the Metered MSteams API and I cannot perform the command in cloud azure powershell.
I opened a ticket on the azure portal 3 days ago and no one helps me 🤔.
(InternalServerError) S2S33101: Failed to get token from tokenEndpoint: 'https://login.windows.net/1fb81c63-5daf-4103-866d-fa9bf6XXXXXX/oauth2/v2.0/token', clientId: 'e3b317b2-ee2c-4eb2-ab20-c166ef7fe246', resource: 'https://graph.microsoft.com//.default', activityId: '0e0650da-bcb5-4bf8-bc7c-022b3aba25b0'.'.
Same issue here:
#929

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

ID: b356f4b8-50b3-7ff6-8c3f-a5210b027c7c
Version Independent ID: b356f4b8-50b3-7ff6-8c3f-a5210b027c7c
Content: Enable metered APIs and services in Microsoft Graph - Microsoft Graph
Content Source: concepts/metered-api-setup.md
Product: non-product-specific
Technology: microsoft-graph
GitHub Login: @spgraph-docs-team
Microsoft Alias: MSGraphDocsVteam

Hi,
Thanks for your feedback. Unfortunately this repo is for keeping track of issues or new contributions in the Microsoft Graph docs and not to provide help about how to use Microsoft Graph. You should rely on the Microsoft Graph support area (https://developer.microsoft.com/en-us/graph/support) for this kind of questions. I'm going to close this item. Thanks for your understanding.


owner example in the README.md

Because the original text was too terse (for some one new to this project).

The example is inspired by #10.

Thanks!


hide sidebar by default

Assalamua'alaikum..
I'm beginner and still learn. it a fantastic theme, i've try it for practice.
how to hide left side bar on start without clicking the toggle navigation first.
I'm sorry for this dumb question. thank you so much..

how to copy your code @almasaeed2010

@vsuryakala If you use AdminLTE v2.3.x then you need only to add sidebar-collapse to body class


Vue3: Fixed navigation active style not applied

Summary
Fixes #11904

In Vue 3, isActive behavior has changed:
https://router.vuejs.org/guide/essentials/active-links#When-are-links-active-
https://v3.router.vuejs.org/api/#active-class
Created computed property that matches old behavior to use instead of router's isActive.
Note: If you navigate to workloads, close group tab and re-open it again, 'Workload' tab is not highlighted. This is unrelated to Vue3 migration and can be reproduced on 2.9. It is not part of this fix and we might want to address it in the future.
Occurred changes and/or fixed issues

Changed behavior to match Vue2
Technical notes summary

Areas or cases that should be tested

Go to "Local" cluster -> Workloads -> Pods
Click on any pod to go into "Details" page
"Pods" should still be selected in side nav

Areas which could experience regressions

Side bar navigation
Screenshot/Video

https://github.com/user-attachments/assets/c16fb751-e5ac-4ddc-9f86-a1b271744800
Checklist

[x] The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
[x] The PR has a Milestone
[x] The PR template has been filled out
[x] The PR has been self reviewed
[x] The PR has a reviewer assigned
[x] The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
[x] The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes

I'm okay with the change but I think it might still need some tweaking. I clicked around and found what appears to be broken highlighting which looking at fleet. Multiple items were being highlighted when I would only expect one.
https://github.com/user-attachments/assets/b4c1ca5a-f081-45db-a554-a21da39e8de7


Adjust Front-End

Get rid of outdated imports/functionalities.
Get rid of unnecessary components, pages, etc.
Adapt the boilerplate Front-end to our Application.

NOTE: Adjusting the Front-end doesn't mean styling it. It just means ensuring our application functions as desired. I'd say ignore Styles for the moment, we can review later when our app's functionality is polished.

completed


Problem when adding Parse, Add and Loadrecordsets

Hi.
I use the library adding data using Parse function, but also with the Add and LoadRecordset. the property name doesn't work as expected. Here is hoy I solved it!
I have notice some problems, that I have solved by defining my own LoadRecordset method.
`
' Load properties from an ADO RecordSet object into an array
' @param rs as ADODB.RecordSet

public sub LoadRecordSet(byref rs) ' Change 3.8.2

	LoadRecordSet2 rs, true

end sub

public sub LoadRecordSet3(byref rs) ' Change 3.8.2

	LoadRecordSet2 rs, false

end sub

public sub LoadRecordSet4(byref rs, prop_name) ' Change 3.8.2

	dim old_i

	old_i = i_defaultPropertyName

	i_defaultPropertyName = prop_name
	LoadRecordSet2 rs, true
	i_defaultPropertyName = old_i
end sub
' Load properties from an ADO RecordSet object into an array
' @param rs as ADODB.RecordSet
public sub LoadRecordSet2(byref rs, lcaseall) ' Change 3.8.2
	dim arr, obj, field
	set arr = new JSONArray
	while not rs.eof
		set obj = new JSONobject
		for each field in rs.fields
			k = field.name: if lcaseall then k = lcase(k)
			obj.Add k, field.value
		next
		arr.Push obj
		rs.movenext
	wend
	set obj = nothing
	add i_defaultPropertyName, arr
end sub`

With this 4 functions, I can get the desire outcome.
first of all, because I can assing a name to the recordset, not loosing the previous name.
I notice you have not add the feature of lowercase properties (here included).

Hi @dtrillo,
That is cool, thanks!
You can create a PR to add that functionality.
Just need to fork the repo, create the change and send the PR. It will get merged with your contribution to the repo.
Also, what about giving more clear names to the methods there?
My suggestions would be:

rename LoadRecordSet2 to LoadRecordSetWithCase
remove the other ones and change places that call them to call the LoadRecordSetWithCase using the proper flags, because that seems to be more clear and we don't introduce methods to wrap a boolean.

What do you think?

I don't use GitHub very ofter because I don't have it available except at home, and this is a feature of my work. I will try to make the PR in the next days.

I have a possible fix for this in development (4b5fef0), if you still wanna try that out.


chore(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/icd-postgresql/ibm (source)
module
minor
3.12.0 -> 3.14.0

terraform-ibm-modules/landing-zone-vpc/ibm (source)
module
patch
7.18.0 -> 7.18.1

Release Notes

terraform-ibm-modules/terraform-ibm-icd-postgresql (terraform-ibm-modules/icd-postgresql/ibm)
v3.14.0
Compare Source
Features

add missing configuration parameters and add validations for plan-time (#424) (a3fc547)

v3.13.1
Compare Source
Bug Fixes

deps: update terraform-module (#363) (dbdfc3a)

v3.13.0
Compare Source
Features

added support to set log_disconnections and log_connections in the configuration input variable object (#421) (57d9068)

terraform-ibm-modules/terraform-ibm-landing-zone-vpc (terraform-ibm-modules/landing-zone-vpc/ibm)
v7.18.1
Compare Source
Bug Fixes

added fix to ensure vpc_data is always output after initial apply (#781) (069d418)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

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

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

The bump to landing-zone-vpc changed how the module output the vpc.name.
The actual error followed the snippet above
│ The "for_each" map includes keys derived from resource attributes that
│ cannot be determined until apply, and so Terraform cannot determine the
│ full set of keys that will identify the instances of this resource.
│
│ When working with unknown values in for_each, it's better to define the map
│ keys statically in your configuration and place apply-time results only in
│ the map values.
│

The vpc_name may now be generated by the module and becomes derived during the apply. The default example was not impacted, because it passes the same name to both the VPC module and the VPE module. This is not the most elegant solution, since the module can prefix/extend the name, so technically it is not the VPC name. However, it works, because we then use the name without prefix for the names of the gateways and reserved IPs. This is still enough to identify them.
The main module is not changing, so there is no risk of regression/change.

/run pipeline

Postgres install failed with:
2024-06-06T15:42:28.8011205Z TestRunDefaultExample 2024-06-06T15:42:28Z retry.go:99: Returning due to fatal error: FatalError{Underlying: error while running command: exit status 1; ╷
2024-06-06T15:42:28.8013652Z │ Error: Internal Server Error
2024-06-06T15:42:28.8016913Z │
2024-06-06T15:42:28.8017699Z │   with module.postgresql_db.ibm_database.postgresql_db,
2024-06-06T15:42:28.8019229Z │   on .terraform/modules/postgresql_db/main.tf line 55, in resource "ibm_database" "postgresql_db":
2024-06-06T15:42:28.8020649Z │   55: resource "ibm_database" "postgresql_db" {
2024-06-06T15:42:28.8021454Z │

Going to retry..

Hmm, upgrsde test failed with:
2024-06-06T17:20:43.5743221Z         	Messages:   	Resource(s) identified to be updated
2024-06-06T17:20:43.5744150Z         	            	Name: security_group_rule
2024-06-06T17:20:43.5745512Z         	            	Address: module.vpe_security_group.ibm_is_security_group_rule.security_group_rule["allow-all-default-sg-inbound"]
2024-06-06T17:20:43.5746660Z         	            	Actions: [update]
2024-06-06T17:20:43.5747185Z         	            	DIFF:
2024-06-06T17:20:43.5747627Z         	            	  Before:
2024-06-06T17:20:43.5748364Z         	            		{"remote":"r006-f2b664ea-7271-40e1-9b41-c5ebc1d4697d"}
2024-06-06T17:20:43.5749049Z         	            	  After:
2024-06-06T17:20:43.5749533Z         	            		{}

@shemau are we expecting this, or know why its happening?

I was not expecting that. 7.18.0 to 7.18.2 should only have been adding a delay and a new data source.
I will dig into the logs, but will probably run the test again.

/run pipeline

/run pipeline

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.

/run pipeline

I have reverted the VPC module update to see if pipeline passes

/run pipeline

/run pipeline

So even after removing the the VPC update from this PR, the same error occurs:
        	            	Address: module.vpe_security_group.ibm_is_security_group_rule.security_group_rule["allow-all-default-sg-inbound"]
        	            	Actions: [update]
        	            	DIFF:
        	            	  Before:
        	            		{"remote":"r006-337b8e0a-2aaf-431e-884e-d1ec3addadec"}
        	            	  After:
        	            		{}

Since its unrelated to the module, and just updates to the examples, I'm going to add back newer vpc version, and skip upgrade test

/run pipeline

:tada: This PR is included in version 4.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Drop zarr and numcodecs dependency

Description
This PR closes #868 using the solution from @tlambert03 provided in https://github.com/napari/napari/issues/868#issuecomment-573361473 with minor modification. I added zarr to our test dependencies so we can still use it there. We'll see how our tests do and if we then need any followups. @tlambert03 feel free to take over the PR and make them directly if you like.
Once we have our CI fixed, we'll want to merge these changes into #867, #869, #871, #872 and then release I think.
Type of change

[x] Bug-fix (non-breaking change which fixes an issue)

Ok, looks like CI is now working again with all our tests on all platforms, with zarr as a test dependency.  We might want to leave this overnight for @jni to look at the new zarr function, and I wouldn’t mind having one more think on it tomorrow morning.  But assuming everyone is 👍 we can merge this and make sure it works with all those others mentioned above

Thanks!! Agreed let’s revisit tomorrow, would be nice if @jni can give a thumbs up and then we can get the 0.2.10 PRs merged and start the release process tomorrow


conflict between FirebaseAdmin 1.17.1 and Google.Cloud.Firestore 2.2.0

there is a conflict arises when I include both FirebaseAdmin 1.17.1 and Google.Cloud.Firestore 2.2.0 packages.
FirebaseAdmin 1.17.1 package has a dependency on Google.Api.Gax 2.7.0.
Google.Cloud.Firestore 2.2.0 package has a dependency on Google.Api.Gax 3.2.0.
When installing both packages side by side in one .Net project, the FirebaseAdmin 1.17.1 automatically grabs the higher version Google.Api.Gax 3.2.0 which causes compile errors. specifically because the use of Microsoft.Bcl.AsyncInterfaces in Google.Api.Gax 3.2.0 while using System.Interactive.Async in Google.Api.Gax 2.7.0
I guess this issue belongs here because FirebaseAdmin is not restricting Google.Api.Gax dependency to version 2.7.0

When installing FirebaseAdmin 1.17.1 solely.

When installing both packages

I think it's the same issue as https://github.com/firebase/firebase-admin-dotnet/issues/207

as per microsoft documentation, adding square brackets around package version number should solve the issue. for example:
<PackageReference Include="Google.Api.Gax" Version="[2.7.0]" />

Our Gax dependency is due for an upgrade (see #251). I think the same PR should address this issue as well.

Also I wish there was a way in .NET to restrict a dependency to specific major version (like NPM does for Node.js). It seems this can be achieved by specifying versions as [2.0.0, 3.0.0) (which means any 2.x.x version but not. 3.x.x).
See https://docs.microsoft.com/en-us/nuget/concepts/package-versioning#version-ranges

Google.Api.Gax.Rest 3.2.0 should solve the issue. And yes, this is exactly what you might go for [2.0.0, 3.0.0). inclusive minimum, and exclusive maximum. I learnt some good stuff today. thanks.


fs/promises isn't supported by Node 12

Coverage decreased (-0.05%) to 70.301% when pulling bdcc2364f574575fea35dc25f9a3d4d77c69f061 on fs/promises-not-supported-by-node-12 into c093e9cea360b83e9297e4735aa7850113f16ccb on main.


feat: port Geometry.Manifold.ContMdiffMap

[x] depends on: #4656

This PR/issue depends on:

leanprover-community/mathlib4#4656
By Dependent Issues (🤖). Happy coding!

This PR/issue depends on:

leanprover-community/mathlib4#4656
By Dependent Issues (🤖). Happy coding!


May be incorrect serialization of UUID object with JacksonCodec

I tried to insert object containing UUID field. This operation work ok but when I checked database the UUID representation is LUUID (v3). So the uuidRepresentation option work incorrectly for insertOne operation. Also when I tried to obtain object from database findOne operation (or may eq) use correct UUID serialization mechanism and represent UUID in v4. The result of code sample bellow is null
data class UUIDContainer(val uuid: UUID)

fun main() {
    val settings = MongoClientSettings.builder()
        .uuidRepresentation(UuidRepresentation.STANDARD)
        .build()

    val client = KMongo.createClient(settings)
    val database = client.getDatabase("db")
    val uuids = database.getCollection<UUIDContainer>()

    uuids.drop()

    val entry = UUIDContainer(UUID.fromString("151e83ca-1f8e-4d22-b042-7fb439402b29"))
    uuids.insertOne(entry)

    val result = uuids.findOne(UUIDContainer::uuid eq "151e83ca-1f8e-4d22-b042-7fb439402b29".toUUID())
    println(result)
}

I also try to add codec for database using withCodecRegistry method but this will have no effect.
data class UUIDContainer(val uuid: UUID)

fun main() {
    val settings = MongoClientSettings.builder()
        .uuidRepresentation(UuidRepresentation.STANDARD)
        .build()

    val client = KMongo.createClient(settings)
    val database = client.getDatabase("db")
    val uuids = database.getCollection<UUIDContainer>()
        .run {
            withCodecRegistry(
                CodecRegistries.fromRegistries(
                    CodecRegistries.fromProviders(UuidCodecProvider(UuidRepresentation.STANDARD)),
                    codecRegistry
                )
            )
        }

    uuids.drop()

    val entry = UUIDContainer(UUID.fromString("151e83ca-1f8e-4d22-b042-7fb439402b29"))
    uuids.insertOne(entry)

    val result = uuids.findOne(UUIDContainer::uuid eq "151e83ca-1f8e-4d22-b042-7fb439402b29".toUUID())
    println(result)
}

I tried to debug internal code and figure out that KMongo use JacksonCodec in my case and I suppose that JacksonCodec knows nothing about specified UUID representation (may be I wrong). I also tried to setup JAVA_LEGACY representation and this result in exception and as I found out there is no way to get around of this exception because of code bellow:
    private fun createRegistry(codecRegistry: CodecRegistry, uuidRepresentation: UuidRepresentation): CodecRegistry =
        if (uuidRepresentation !== UuidRepresentation.JAVA_LEGACY) {
            OverridableUuidRepresentationCodecRegistry(codecRegistry, uuidRepresentation)
        } else {
            throw CodecConfigurationException(
                "Changing the default UuidRepresentation requires a CodecRegistry that also "
                        + "implements the CodecProvider interface"
            )
        }

I have no idea what this check is for and how I can pass it.
But when I created KMongo client only to get default codecRegistry and then configured standard mongo client with this registry and JAVA_LEGACY representation (as shown in code bellow) I can successfully got object after it inserted in database.
data class UUIDContainer(val uuid: UUID)

fun main() {
    val registries = KMongo.createClient().getDatabase("db").codecRegistry

    val settings = MongoClientSettings.builder()
        .uuidRepresentation(UuidRepresentation.JAVA_LEGACY)
        .build()

    val client = MongoClients.create(settings)
    val database = client.getDatabase("db").withCodecRegistry(
        CodecRegistries.fromProviders(
            UuidCodecProvider(UuidRepresentation.JAVA_LEGACY),
            registries
        )
    )

    val uuids = database.getCollection<UUIDContainer>()

    uuids.drop()

    val entry = UUIDContainer(UUID.fromString("151e83ca-1f8e-4d22-b042-7fb439402b29"))
    uuids.insertOne(entry)

    val result = uuids.findOne(UUIDContainer::uuid eq "151e83ca-1f8e-4d22-b042-7fb439402b29".toUUID())
    println(result)
}

What did I do wrong?

Have you tried to use the setUUIDRepresentation function? http://litote.org/kmongo/dokka/kmongo/org.litote.kmongo.util/-k-mongo-jackson-feature/index.html
HTH


New API front and back to support RenderCommands

The current API doesn't make it easy to support RenderCommands in it's current state, making it mandatory to duplicate code/structs
So this new API should allow use on the two fronts. Some things were changed a bit, other were completly replaced.
PS. Text is still not working because we are waiting on what clay does with this

I'm fine with duplicating structs when we have to. What I recommend we do is something like this
#[repr(C)]
struct FFI {
    a: i32,
    b: f64,
    c: u8,
}

#[repr(C)]
struct RustStruct {
    a: i32,
    b: f64,
    c: u8,
}

fn offset_of<T, F>(field: &F, base: &T) -> usize {
    let base_ptr = base as *const T as *const u8;
    let field_ptr = field as *const F as *const u8;
    (field_ptr as usize) - (base_ptr as usize)
}

#[test]
fn verify_field_offsets() {
    let dummy_ffi = FFI {
        a: 0,
        b: 0.0,
        c: 0,
    };
    let dummy_rust = RustStruct {
        a: 0,
        b: 0.0,
        c: 0,
    };

    // Field offsets in FFI
    let ffi_a_offset = offset_of(&dummy_ffi.a, &dummy_ffi);
    let ffi_b_offset = offset_of(&dummy_ffi.b, &dummy_ffi);
    let ffi_c_offset = offset_of(&dummy_ffi.c, &dummy_ffi);

    // Field offsets in RustStruct
    let rust_a_offset = offset_of(&dummy_rust.a, &dummy_rust);
    let rust_b_offset = offset_of(&dummy_rust.b, &dummy_rust);
    let rust_c_offset = offset_of(&dummy_rust.c, &dummy_rust);

    // Assert that offsets match
    assert_eq!(ffi_a_offset, rust_a_offset);
    assert_eq!(ffi_b_offset, rust_b_offset);
    assert_eq!(ffi_c_offset, rust_c_offset);
}

fn main() {
    println!("Run `cargo test` to verify field offsets.");
}

This will verify that the structs always matches so when a new clay.h pop up we won't miss updating something if they have been moved around.

On my side i'm not ok with having two structs that do the same thing just in a different context, the value stored is the same... So why not having only one struct to store it.
Tests like those are not going to work, every fields are not a one to one representation because of how rust bindings work (See BorderRadius for example).
It might work on most struct, but some aren't going to like that, even more for struct that use enums...

Right, I was thinking of the case if we need to duplicate them for some reason. I just don't want us to get into a case where the some structs doesn't match for some reason which can be very annoying to track down.

No i meant like for example you use Clay_RectangleElementConfig to create the rectangle, and it gets passed back on Clay_RenderCommand.config, so why have two struct to store a rectangle config instead of one, since they are pretty much the same thing

And my PR name isn't very good... I didn't change the API front, I switched around some things, but i changed a lot of the API "back-end" if we can call it that


 I think that it should be compressed less and less as the size of the image goes down

Don't need this anymore


ProgressBar_Enhancement

Fixed the repetitive logging during failed assertion
Added a new logger to the log4j2.properties to be able to print the progress bar on the console
Added the progress bar to indicate the time elapsed since start of assertion

@MohabMohie  @MahmoudElSharkawy
could you please help us with this PR?


Move lifetime registration logic to ServiceCollectionExtensions

Changes made

Dispose SempahoreSlim instance in FormProvider
Add methods to register Forms through assembly scanning
Add and pass the preApplicationRunAction parameter to registering methods
Move registration logic from HostBuilderExtensions to ServiceCollectionExtensions to allow use with the HostApplicationBuilder (introduced in .NET 7). This allows for following code:

var builder = Host.CreateApplicationBuilder(args);

builder.Services.AddWindowsFormsLifetime<MainForm>();
builder.Services.AddFormsFromAssemblyContainingType<IAssemblyMarker>();

var host = builder.Build();
await host.RunAsync();

Hello @midare160! Thank you for the additions. I was not aware of Host.CreateApplicationBuilder(). If you don't mind, can you please update the tabs to be spaces?

Hello @midare160! Thank you for the additions. I was not aware of Host.CreateApplicationBuilder(). If you don't mind, can you please update the tabs to be spaces? Also, please star this repo if you like this project! 👍

Done both 😊


Refresh dispatch model without losing entity mapping

What project is affected?
Dispatch model refresh losing all the changes to Intents  and Entities.
What language is this in?
C#
What happens?
I'm understanding the concepts of BOT Framework by following documentation.
Entities and Intents are wiped off when dispatch model is refreshed.
What are the steps to reproduce this issue?
Step 1: Followed steps to create new KB and then cognitiveModels.json is updated with new values and .\Deployment\Scripts\update_cognitive_models.ps1 -RemoteToLocal command need to be executed which will update Dispatch model.
Step 2: For make BOT more user friendly, added few entities and mapped to Intents. Till here everything works fine.
Step 3: Repeated step 1 then entities which were mapped earlier are wiped off.
What were you expecting to happen?
When I add,update,modify QnA KB, entities which already created should stay in.
Please let me know what I'm missing in achieving it.

Thanks @lauren-mills.  @sidmore - Can you throw some light on how to manage Intents and Entities  when deployed via CLI?

@tsuwandy - Please help me with this.

@narulasrinivas, where did you add the intents/entities?  To the dispatch LUIS app via LUIS portal?  Any changes done to the dispatch app on portal won’t be persisted next time dispatch app is refreshed from cli.  Could you clarify your scenarios?  Why do you need to add intents to the dispatch app directly?  Can the intent be added to the sub LUIS app instead?  Also entities in sub LUIS app are not transferred directly to dispatch app, except for this used in patterns.

hi @narulasrinivas, below are some common Dispatch use-cases:

A user already has one or more LUIS, QnA Maker apps/models, or even some flat files of intent-utterances and the user would like to create a meta-app to dispatch bot traffics to those child apps. In this scenario, entities should be (manually) added to the child LUIS apps instead of to the Dispatch parent app, so that the entity definitions won't get wiped out after the Dispatch app/model is refreshed as @lauren-mills and @tsuwandy had indicated.
A user has a LUIS app and he/she can use Dispatch to evaluate the app's intent prediction performance in a 5-fold cross validation process.

As you can see from the scenarios, Dispatch focuses on intent prediction/dispatching and evaluation, so that entity definitions are not persisted. Please maintain your entities in the child apps, not at the Dispatch level.
I am going to close this issue for now. If you have more questions, please feel free to reopen it.


Remove JavaTestRunner

JavaTestRunner and JavatestUtil are similar file.
And most of tests are using JavatestUtil except https://github.com/adoptium/aqa-tests/blob/4a1eca103c3a40e98f36328ca03f2eef02de05d9/jck/jck.mk#L129
https://github.com/search?q=repo%3Aadoptium%2Faqa-tests JCK_CMD_TEMPLATE&type=code
I think if we update these two tests to run via JavatestUtil, we can remove JavaTestRunner.

Yes, that was the next step in the 'evolution' of this automation (stripping away unneeded layers).

@annaibm can you take a look at this one? Thanks


feat(conditional-types): added-types-as-conditional-to-needs

feat(conditional-types): added-types-as-conditional-to-needs

[x] Improved types config with conditional chainning
[x] Refactored: generateOtp() function based on the new updated types
[x] User/Coder experience doesn't change at all. Same way to use it.

🎉

 Improving the types for the package and it’s usage


Enable logging of API request and response details

Is your feature request related to a problem? Please describe:
When troubleshooting HomeKit automations involving this plug-in, its log entries often lack sufficient information regarding to determine what is happening.
Describe the solution you'd like:
I would like to be able to specify a higher level of logging verbosity for this plug-in in the HomeKit config file. When I do so, I would like e.g.:

“Changes may not have successfully propagated to the carrier API“ to be followed by the API request URL, verb, body (if applicable); and the API response HTTP code (e.g. 500) and body.
“Pushing changes to carrier API“ to be followed by which changed value(s) triggered the push

Describe alternatives you've considered:
I would have to employ some kind of packet sniffing solution without this, and then would have to filter out and correlate results. This would not be practical for ongoing maintenance of HomeKit automations involving my home’s air-conditioning systems.
Additional context:
This is in support of HomeKit automations I use to set my upstairs and downstairs AC systems to Away and back to Schedule based on occupancy sensors on the corresponding floors. When I find, for example, that a system is still set to Away even though the corresponding floor is occupied, I am unable to determine exactly what has gone wrong. The ability to log information as specified above would almost certainly allow me to determine the problem in such instances, greatly speeding the process of making automations work as intended more reliably.

This issue and #312 are complimentary. I separated them because they have fairly different levels of effort to implement, and are also independently useful.

It turns out the plugin was silently swallowing all http errors before... Fixed that and generally made error handling better and more detail should show in console, especially if you have your homebridge in debug mode (option in the homebridge settings).
Let me know how the new build here works for you. Thanks for opening this!


graph.updateChildren won't update new parent Node's _cfg.children

Describe the bug
Hi,
I tried to use graph.updateChildren just like the example: https://g6.antv.vision/zh/examples/interaction/treeBehavior#dragSubtree
and found that the new parent._cfg.children was never change, and the Modal(parent._cfg.model.children) did changed.
The question is how to make the array of nodes(parent._cfg.children) keep the same with Model
Your Example Website or App
https://g6.antv.vision/zh/examples/interaction/treeBehavior#dragSubtree
Steps to Reproduce the Bug or Issue
  const move = (item, data) => {
    const id = item.getID()
    graph.removeChild(id)
    setTimeout(function () {
      const newParentData = graph.findDataById(newParent.getID())
      const newChildren = []
     // WRONG ORDER
      console.log('newParentData.children', newParentData.children)
      if (!newBrotherDown) {
        graph.updateChildren([...(newParentData.children || []), data], newParent.getID())
        return
      }
      newParentData.children.forEach((c) => {
        if (newBrotherDown.get('id') === c.id) {
          newChildren.push(data)
        }
        newChildren.push(c)
      })
      graph.updateChildren(newChildren, newParent.getID())
      const temp = graph.findDataById(newParent.getID())
      // RIGHT ORDER
      console.log('temp', temp.get('children'))
    }, 100)
  }

Expected behavior
the array of nodes(parent._cfg.children) keep the same with Model
Screenshots or Videos
No response
Platform

OS: [e.g. Linux]
Browser: [e.g. Chrome]
"@antv/g6": "~4.5.1",

Additional context
No response

I found graph.updateItem can solve my problem.


Push Notifications

infrastructure for notifications/backend + basic 2 notifications: payment received and request for payment received.
https://github.com/transistorsoft/react-native-background-fetch

Currently the wallet subscribes to blockchain events and broadcasts them
Userstorage subscribes to broadcasts from wallet

Todo:

what happens when background task is started? is the web3 wallet connection working? does it start getting blockchain events?
keep a new field under "feed". lastSeenDate UTC. which is updated every time user visits feed or after push notifications sent.
background task should poll for the events (Transfer - to us, PaymentWithdraw - from us)

need to estimate blocknumber to start polling from that matches lastSeenDate
filter to events that happened after lastSeenDate
update lastSeenDate
send local notification "Payment of XG$ was accepted/received by/from NAME"

@LiavGut do we have a design for push notifications?

@nicastelo you might want to break this into smaller tasks, and make this an epic

@LiavGut do we have a design for push notifications?

@sirpy Not yet.