bug: spurious AwsSolutions-SMG4 with aws-cdk-lib >= 2.116.0

What is the problem?
we updated our version of aws-cdk-lib today from 2.115 to 2.120 latest
started getting this nag:

AwsSolutions-SMG4: The secret does not have automatic rotation scheduled. AWS Secrets Manager can be configured to automatically rotate the secret for a secured service or database.

but in our cdk code there is clearly a rotation set up
Reproduction Steps
        rds_instance = rds.DatabaseInstance(
            ...
        )
        rds_instance.add_rotation_single_user(
            automatically_after=Duration.days(30),
            rotate_immediately_on_update=True,
        )

then use aws-cdk-lib >= 2.116.0
What did you expect to happen?
no nag, because we have a rotation
What actually happened?
when upgrading aws-cdk-lib to 2.116.0 or later we start getting the nag
cdk-nag version
2.28.7
Language
Python
Other information
Looking at the changes released in aws-cdk-lib 2.116.0 there were two possibly relevant PRs:

https://github.com/aws/aws-cdk/pull/28379
https://github.com/aws/aws-cdk/pull/28303

I think it may be the latter, because here it changes the rotationRules in the generated schedule to use a scheduleExpression rather than automaticallyAfterDays
and the relevant check in cdk-nag: https://github.com/cdklabs/cdk-nag/blob/7e9aa43b300e71fa01ccebd639ae8de67feaaed7/src/rules/secretsmanager/SecretsManagerRotationEnabled.ts#L120 only looks for automaticallyAfterDays rather than any valid scheduling rule

Related to #1565 and #1566

@anentropic Please try cdk-nag v2.28.13


XRC721 how to using Truffle

This PR is in response the issue #35

I am getting an error on the "import" commands in the XRC721.sol file:

Not sure what steps are to fix this issue.

@WalterBlueu  hope you can find your answer here https://stackoverflow.com/questions/67321111/file-import-callback-not-supported

I have updated the code for the contracts. They would work fine now.

@yash0501 thank you, that solved the problem.  We are now up against the same verification issue as we saw with Remix.  Give me a few hours to see if I can get more information on this issue from BlocksScan.

@yash0501 BlocksScan pointed me to this tutorial on flattening the dependencies from OpenZeppelin: https://www.xdc.dev/ivan_blocksscan/learn-how-to-flatten-a-smart-contract-and-verify-on-blocksscan-56on

I don't know if this tool is what you need for Truffle, but it looks like updating the how-to to use a tool like this to flatten the smart contract will get the job done: https://www.npmjs.com/package/truffle-flattener

Okay, I will check it out and try to flatten the smart contract to verify it on blockscan


⚠ TiendaEnLinea has degraded performance

In 539b82c, TiendaEnLinea (https://www.fahorro.com/) experienced degraded performance:

HTTP code: 200
Response time: 5867 ms

Resolved: TiendaEnLinea performance has improved in 45c51b1.


Add mention of Turkish yogurt in banana bread recipe steps + spelling changes; close #20

Changes:

Add mention of Turkish yogurt in recipe instruction steps.
Spelling: Capitalize Turkish
Spelling: Add missing sentence end period (.) in recipe steps.

Very cool! Thanks!


[Big Rock] Fix tests in eShop repo

[x] Address failures with Catalog.UnitTests (action: delete these)
[x] Address failures with Catalog.FunctionalTests (action: figure out how to use Aspire components/testconstainers in testing)
[x] Address failures with Ordering.FunctionalTests (action: figure out how to use Aspire components/testcontainers in testing)
[x] Address failures with Ordering.UnitTests (action: upgrade to test minimal API)
[x] Address failures with Basket.UnitTests (action: need to be updated to unit test gRPC service)
[ ] Address failures with Basket.FunctionalTests (action: figure out how to use Aspire components/testcontainers in testing)
[ ] Address failures in Application.FunctionalTests (action: figure out if functional test setup used for other tests applies here)

We didn't land functional tests for the basket service or the entire E2E for go-live. I'm incline to close this for now and focus our efforts on increasing code coverage for the areas where we do have test infrastructure setup.


err when parse api.md if got err as below when got double quotation mark in api.md

err as below when got double quotation mark in api.md
Errors Hide Parser error on line 5 missed comma between flow collection entries
api.md file looks like:
"a"
I guess:
var SwaggerInfo = swaggerInfo{
Version:     "0.1",
Host:        "",
BasePath:    "/",
Schemes:     []string{},
Title:       "Cmdb Api",
Description: "something",
}
I think should chnage     Description: "something"  to   Description: something
user `` instead of "" or  escape  ”“

More detailed, please.

More detailed, please.

well，I got an api.md file for the main function with description.markdown comment
And in api.md file only have :    "a"
then  swag init
run main.go   and   open http://<IP_ADDRESS>:8080/swagger/index.html
i got err:
Errors
Hide Parser error on line 5
missed comma between flow collection entries

Got it, It's a bug of swagger ui. I think.

This problem can also be reproduced without any markdown involved. Just type any string with quotation marks in the general API @Description field.
I actually this is a bug stemming from the two step application of templates in  writeGoDoc() and then in ReadDoc(), resulting in the escaping of quotes disappearing. Since the problem only appears when running the generated doc.go, it doesn't show up in swagger.{json,yaml}. I came up with a workaround that seems to work well enough: https://github.com/swaggo/swag/pull/743

This problem can also be reproduced without any markdown involved. Just type any string with quotation marks in the general API @Description field.
I think this is a bug stemming from the two step application of templates in writeGoDoc() and then in ReadDoc(), resulting in the escaping of quotes disappearing. Since the problem only appears when running the generated doc.go, it doesn't show up in swagger.{json,yaml}. I came up with a workaround that seems to work well enough: #743

GOD JOB！

Use quote in doc Description trigger the error, simply change it to single quote would solved this problem.
Add some document for this issue would be nice for impatient users.


Columbia IDs

Due to the renaming of the records, there's something that we need to do for some external links to continue working. I've uploaded the list of records that need to be handled in this was in src/main/resources/columbia-ids.csv, here's what needs to be done:
For each {RID} in the csv list,
when purl.bdrc.io/resource/{RID} is called with the "html" http accept header
compute a new modified RID starting with W instead of I (ex: I00JR625 --> W00JR625
redirect to html as if the initial identifier was the new record, for instance redirect to library.bdrc.io/show/bdr:W00JR625.
There's no change in any other case (ex: RID not in the list, accept header not html, etc.).

Done as of commit b047493

thanks! can you merge master into newbiblio?

this doesn't work anymore... @MarcAgate could you take a quick look? See for instance http://purl.bdrc.io/resource/I23714


The output JSON is not valid JSON,

Describe the bug
The output JSON is not valid JSON, the separator should be a comma, but its no there also its not enclosed by [ ]
{"total":"6.7Gi","used":"1.9Gi","free":"2.4Gi","available":"62Mi"}
{"total":"2.0Gi","used":"0B","free":"2.0Gi"}

To Reproduce
Steps to reproduce the behavior:

Issued pxi command.
free -mhw | pxi '(['','total', 'used','free','available'])=>({total,used,free,available})' --from ssv --to json
Error thrown or unexpected behavior.

{"total":"6.7Gi","used":"1.9Gi","free":"2.4Gi","available":"62Mi"}
{"total":"2.0Gi","used":"0B","free":"2.0Gi"}

Expected behavior
[
{"total":"6.7Gi","used":"1.9Gi","free":"2.4Gi","available":"62Mi"},
{"total":"2.0Gi","used":"0B","free":"2.0Gi"}
]

Desktop (please complete the following information):

OS: Ubuntu 20.04
Version (pxi --version): 0.37.0

Additional context
I think comma-separated JSON output should be the default mode.

I am not able to use JSON.parse, so I have to do this instead.
let's say
dfCommandResponse = `{"Filesystem":"/dev/sdc","Size":"251G","Used":"7.6G","Avail":"231G","UsePercent":"4%","MountedOn":"/"}
{"Filesystem":"none","Size":"3.4G","Used":"34M","Avail":"3.4G","UsePercent":"1%","MountedOn":"/mnt/wslg"}
{"Filesystem":"none","Size":"3.4G","Used":"4.0K","Avail":"3.4G","UsePercent":"1%","MountedOn":"/mnt/wsl"}
{"Filesystem":"tools","Size":"419G","Used":"104G","Avail":"316G","UsePercent":"25%","MountedOn":"/init"}
{"Filesystem":"none","Size":"3.4G","Used":"0","Avail":"3.4G","UsePercent":"0%","MountedOn":"/dev"}
{"Filesystem":"sysfs","Size":"0","Used":"0","Avail":"0","UsePercent":"-","MountedOn":"/sys"}
{"Filesystem":"proc","Size":"0","Used":"0","Avail":"0","UsePercent":"-","MountedOn":"/proc"}
{"Filesystem":"devpts","Size":"0","Used":"0","Avail":"0","UsePercent":"-","MountedOn":"/dev/pts"}
{"Filesystem":"none","Size":"3.4G","Used":"8.0K","Avail":"3.4G","UsePercent":"1%","MountedOn":"/run"}
{"Filesystem":"none","Size":"3.4G","Used":"0","Avail":"3.4G","UsePercent":"0%","MountedOn":"/run/lock"}
{"Filesystem":"none","Size":"3.4G","Used":"0","Avail":"3.4G","UsePercent":"0%","MountedOn":"/run/shm"}
{"Filesystem":"none","Size":"3.4G","Used":"0","Avail":"3.4G","UsePercent":"0%","MountedOn":"/run/user"}
{"Filesystem":"binfmt_misc","Size":"0","Used":"0","Avail":"0","UsePercent":"-","MountedOn":"/proc/sys/fs/binfmt_misc"}
{"Filesystem":"tmpfs","Size":"3.4G","Used":"0","Avail":"3.4G","UsePercent":"0%","MountedOn":"/sys/fs/cgroup"}
{"Filesystem":"cgroup2","Size":"0","Used":"0","Avail":"0","UsePercent":"-","MountedOn":"/sys/fs/cgroup/unified"}
{"Filesystem":"cgroup","Size":"0","Used":"0","Avail":"0","UsePercent":"-","MountedOn":"/sys/fs/cgroup/cpuset"}
{"Filesystem":"cgroup","Size":"0","Used":"0","Avail":"0","UsePercent":"-","MountedOn":"/sys/fs/cgroup/cpu"}
{"Filesystem":"cgroup","Size":"0","Used":"0","Avail":"0","UsePercent":"-","MountedOn":"/sys/fs/cgroup/cpuacct"}
{"Filesystem":"cgroup","Size":"0","Used":"0","Avail":"0","UsePercent":"-","MountedOn":"/sys/fs/cgroup/blkio"}
{"Filesystem":"cgroup","Size":"0","Used":"0","Avail":"0","UsePercent":"-","MountedOn":"/sys/fs/cgroup/memory"}
{"Filesystem":"cgroup","Size":"0","Used":"0","Avail":"0","UsePercent":"-","MountedOn":"/sys/fs/cgroup/devices"}
{"Filesystem":"cgroup","Size":"0","Used":"0","Avail":"0","UsePercent":"-","MountedOn":"/sys/fs/cgroup/freezer"}
{"Filesystem":"cgroup","Size":"0","Used":"0","Avail":"0","UsePercent":"-","MountedOn":"/sys/fs/cgroup/net_cls"}
{"Filesystem":"cgroup","Size":"0","Used":"0","Avail":"0","UsePercent":"-","MountedOn":"/sys/fs/cgroup/perf_event"}
{"Filesystem":"cgroup","Size":"0","Used":"0","Avail":"0","UsePercent":"-","MountedOn":"/sys/fs/cgroup/net_prio"}
{"Filesystem":"cgroup","Size":"0","Used":"0","Avail":"0","UsePercent":"-","MountedOn":"/sys/fs/cgroup/hugetlb"}
{"Filesystem":"cgroup","Size":"0","Used":"0","Avail":"0","UsePercent":"-","MountedOn":"/sys/fs/cgroup/pids"}
{"Filesystem":"cgroup","Size":"0","Used":"0","Avail":"0","UsePercent":"-","MountedOn":"/sys/fs/cgroup/rdma"}
{"Filesystem":"drivers","Size":"419G","Used":"104G","Avail":"316G","UsePercent":"25%","MountedOn":"/usr/lib/wsl/drivers"}
{"Filesystem":"lib","Size":"419G","Used":"104G","Avail":"316G","UsePercent":"25%","MountedOn":"/usr/lib/wsl/lib"}
{"Filesystem":"none","Size":"3.4G","Used":"76K","Avail":"3.4G","UsePercent":"1%","MountedOn":"/mnt/wslg/versions.txt"}
{"Filesystem":"none","Size":"3.4G","Used":"76K","Avail":"3.4G","UsePercent":"1%","MountedOn":"/mnt/wslg/doc"}
{"Filesystem":"drvfs","Size":"419G","Used":"104G","Avail":"316G","UsePercent":"25%","MountedOn":"/mnt/c"}`

const str = `[${dfCommandResponse}]`.split('\n').join(',');
const jsonStr = JSON.parse(`${str.substring(0, str.length - 2)}]`);

Thanks for opening this issue @OptimBro. Indeed, the behaviour you are describing is a design decision taken by pxi.
Basically, pxi converts each chunk from one format into another. The default chunk is a line (--by line). So each line individually is being transformed into valid JSON with --to json. This is also known as JSON lines format.
pxi was initially written to process files that are several hundred GBs in size in a streaming fashion with minimal memory. This is why it uses the JSON lines format that lends itself to stream processing.

Thanks @Yord , for the quick response.  I got it, it makes sense.
So is there any way like extra command line arguments to get JSON with comma else no issues. I'll will with manual formatting 😊
Thanks for creating this magical stuff, I liked the way I can get and format the output of commands like dh, ls etc 👌😍

Thanks for creating this magical stuff, I liked the way I can get and format the output of commands like dh, ls etc 👌😍

🙏 Thank you, I appreciate that.

Maybe you can achieve what you are looking for with the following?
free -mhw |
pxi "([total, used, free, available]) => ({total, used, free, available})" --from ssv |
awk '{print $0 ","}' |
sed '$ s/.$//' |
cat <(echo "[") - <(echo "]")

awk adds commas, sed removes the last comma, cat surrounds the stream with brackets.

Thank you so much ^^, really appreciate it.


Is this really necessary?

I assume this issue will be solved soon: a fix is already committed and the issue is major enough to justify a software update by Apple.
That being the case, is there really a need for this library?
I am concerned that if this gets into a common NPM dependency, it might be quite hard to shift: all it takes is one library to fail to remove it and we will all end up with runtime bloat in our JavaScript applications until the end of time.

No matter how it upgrade, there will always be users in this version.  A business company, They have to consider this part of the user, helpless...

No. We cannot polyfill for every regression present in any release of every major web platform. That way lies madness. What you will end up doing is dragging around a whole polyfill JavaScript runtime 'just in case' a user is stuck on an obscure version.
Worse, you remove an incentive for users to upgrade from buggy and insecure legacy browsers. All web technologies have security flaws - and it's inevitable that iOS 12 JavaScript Core will do too. A user who refuses to upgrade is not only risking their own data, but everyone else who might be subject to botnet or DDOS attacks from a compromised user.
If we really care about users - if we care about security, if we want the web to be the best it can be - we cannot get into the business of resolving user agent bugs at the software layer. It's unsustainable and it removes a powerful incentive for browser vendors to fix their halfbaked releases.

This is a serious issue in iOS 12 and will surely be fixed soon.

it removes a powerful incentive for browser vendors to fix their halfbaked releases.

Do you notice that your first assertion and the last assertion are incompatible? 🤓

This project only provide a technical workaround for the bug. We never say everyone should use it. We never say Apple don't need to fix it. We never say users are ok to keep use the buggy version when Apple provide the updates. In fact, all such things are beyond the control of the author/contributors of this project, so do not attack a strawman.
In real world, many sites/apps can just ignore the bug and this project. But if someone meet the problems due to this bug, they need a solution. Especially, it's not a minor bug, but a really serious bug, which can corrupt the data on the page at all. In many cases, waiting Apple release updates and then waiting your users upgrade their OS/Safari may not be thought as a acceptable solution.
Even no problem yet, we can expect the largest platforms like Facebook or Wechat, and the sites/apps related to money like Amazon, Taobao, or even Bank/Finance/Exchange... may decide to land a workaround code to save the users and the business from the risk. Note, these are totally business decisions, not technical decisions, even we delete this repo won't change their decisions at all.
And every project have its life cycle. We hope this polyfill can be dropped soon (several weeks to months), but it not depends on we developers, because we can't decide when Apple land the updates and when people upgrade their iOS or Safari.

And some polyfill have to have much longer life cycle. For example, I have a iPhone 5 as my backup phone, and there may be many people still use old devices because not everyone can afford $749+. And iPhone 5 can not upgrade to iOS 12. So we don't worry such old device suffer this bug, but they may suffer other bugs, and you will never expect the vendors provide updates anymore. And blaming users not buying new device is very inappropriate. The only viable, moral, responsible solution we developers can afford is trying to provide some technique workaround even they are doomed.

About the script bloat and the dependencies problem, it's a common issue of all polyfills. Ideally, we should use server-side adaption like polyfill.io service. But for this case, the polyfill is very small and we expect it could be removed in months, so I think it's not a big deal. Anyway this is out of the scope of this project.

can we privide a function that called ios12NeedPolyfill  use by
if (!ios12NeedPolyfill()){return}

@zhangenming
The buggy() function in the source is just your ios12NeedPolyfill.
If you need on-demand load or server adaption by UA detection to save a request , see https://github.com/fanmingfei/array-reverse-ios12#ua-sniffing
Client sample code:
...
<script>
if (/Version\/12\.0.*Safari\//.test(navigator.userAgent))
  loadScript('https://cdn.jsdelivr.net/npm/array-reverse-polyfill')
// Assume you have `loadScript()` available to create and insert the script tag.
</script>
...

Server-side adaption sample (in PHP):
...
<? if (preg_match('#Version/12\.0.*Safari/#', $_SERVER['HTTP_USER_AGENT'])): ?>
<script src="https://cdn.jsdelivr.net/npm/array-reverse-polyfill"></script>
<? endif; ?>
...


Create EntityName and add to Entity object

Create EntityName to be a member of Entity object. In addition, remove all context specific name elements that would be rendered redundant by the addition of EntityName.

No please don’t do this!  It is good as it is!!
On Tue, Dec 20, 2022 at 9:31 AM Kevin Kirkley @.***>
wrote:

Create EntityName to be a member of Entity object. In addition, remove all
context specific name elements that would be rendered redundant by the
addition of EntityName.
—
Reply to this email directly, view it on GitHub
https://github.com/Open-Orange-Button/Orange-Button-Taxonomy/issues/167,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEU5DA5CQ7OM73GBOZP4CMDWOICPRANCNFSM6AAAAAATE3FKOI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
[image: Blu Banyan] https://www.blubanyan.com/
[image: O-NetSuite Solution Partner]
Jan Rippingale
CEO
Blu Banyan https://blubanyan.com/
m:  <707.234.5395>707.234.5395
e: @.*** sent after hours or on a holiday/weekend
do not require a response outside business hours.

Will not make change. Taxonomy handles names of Entities through specific naming.


Update ChatterOrgChartControllerTest.cls

When deploying this package in Production Org, Test Class gave Duplicate Username Error. Changed username for the test users inserted in the Test Class. Referenced this blog: https://help.salesforce.com/apex/HTViewSolution?urlname=How-to-resolve-DUPLICATE-USERNAME-error-returned-while-performing-a-deployment&language=en_US

Thanks @rashminirmal for this update and the help article!


Hack to make compilers happy [phab:D1136]

Created by salguarnieri at 2016-11-04 18:32:19:
Certain compiler incantations will report an error if we don't fully
specify the path to the Variance enum. Add the full path to make them
happy.

shahms wrote at 2016-11-04 19:27:35:
better than decltype(*this)::Variance::Contravariant I guess ;-)

shahms accepted this revision at 2016-11-04 19:27:35

salguarnieri closed via commit 3885f4a579e7e9ec9b77f2f6ec41a29c39d7914f at 2016-11-04 19:50:50


s.swift_version = '5.0'

I got this error when installing:
[!] Failed to load 'URITemplate' podspec:
[!] Invalid `URITemplate.podspec` file: undefined method `swift_versions=' for #<Pod::Specification name="URITemplate">
Did you mean?  swift_version=
               swift_version.

@SentulAsia I think the CocoaPods you are using us outdated, can you try again with cocoapods 1.7.x which adds support for swift versions?

@kylef I can’t change my cocoapods version until september... I guess I’ll stay to swift 4.2 then...

@kylef can you do this?, if yes, I'll submit a PR:
if spec.respond_to?(:swift_versions) then
    spec.swift_versions = ["4.2", "5.0"]
  else
    spec.swift_version = "5.0"
  end

@SentulAsia That won't work because the specification will be converted by CocoaPods into JSON upon submission. You can make a copy of a podspec in Ruby and keep it in your project and reference it internally as a workaround.

@kylef Thanks for the follow up, already updated to latest version and that solved the issue. Thanks for the reminder.


[BUG-MAYBE] Password field is not hidden on web ui login

@buzzzo thanks for sharing this feedback with us. We addressed the discussion for that internally to make the proper adjustments. In a couple days it should be solved. Originally it's not a bug, but I understand how it looks like one. We will make the proper adjustments and release this change. If you want to follow this thread, I've related the PR #581 to this.
Again, thanks for raising this 🎉


Annotation app stuck with bad taxon

I have seen this get stuck a couple times where I give the app a valid taxon ID and it says it is not valid:

Seeing this in the dev console. I also don't have dropdowns loaded for the genome name and taxon id.
Can we make this disable attempts at validation if the dropdown data is missing?

Cannot submit any annotations at this point !

I think this is fixed now, can you please try again and see if the interactions are better for you now?  Reopen this issue if it isn't.

I'm seeing exactly the same behavior, and the same "text is not defined" error on the console.

This was with Firefox on mac with caches flushed before a reload of the site.

Hmm, you are testing on beta, right?   When you enter a tax id, does the dropdown show the taxon names alongside the taxon ids?  The text not defined error came from the fact that it wasn't automatically selecting the first item in the list if you didn't specifically select one on your own.  It is working for me in both chrome and firefox.  I know you said you cleared cache, but it doesn't seem like the cache was cleared (btw, I will have a change up soon so that clearing cache won't be necessary)

I’m starting with the genome name; I select contigs file, then type “43255” into the genome name to match the Clostridium strain I’m running. I click on the name that filled out, and I get “43255” in the name, cursor moves to tax id field which is empty and red.
—bob
On Jun 4, 2015, at 9:30 AM, Dustin Machi <notifications<EMAIL_ADDRESS>wrote:
Hmm, you are testing on beta, right? When you enter a tax id, does the dropdown show the taxon names alongside the taxon ids? The text not defined error came from the fact that it wasn't automatically selecting the first item in the list if you didn't specifically select one on your own. It is working for me in both chrome and firefox. I know you said you cleared cache, but it doesn't seem like the cache was cleared (btw, I will have a change up soon so that clearing cache won't be necessary)
—
Reply to this email directly or view it on GitHubhttps://github.com/PATRIC3/patric3_website/issues/193#issuecomment-108915460.

I just ran through this same test you described in firefox and chrome.  I enter 43255 into the organism name, select Peptoclostridium difficile ATCC 43255 from the dropdown, and then it fills in 499175 in the taxid with no errors.  The only box remaining red is the output name at the bottom. Note that I'm not trying to discount what you've said, i'm just not seeing that same behavior on beta currently.
Can you confirm that when you type into the tax_id selector, you see both IDs and names in the dropdown?

And I’m seeing the same behavior in firefox and chrome with caches flushed. See
http://youtu.be/xnEPSSOf9ME
If these things are relying on a large amount of data preloaded into the browser, then that might be a difference between running here at ANL and running locally at PATRIC.
If you have your MCS account, it might be interesting to run a socks proxy over ssh from your place to ANL, and fire up the browser pointing at the proxy to get the timing difference. There is also a tool available for the mac that emulates a slower / higher latency connection.
—bob
On Jun 4, 2015, at 9:39 AM, Dustin Machi <notifications<EMAIL_ADDRESS>wrote:
I just ran through this same test you described in firefox and chrome. I enter 43255 into the organism name, select Peptoclostridium difficile ATCC 43255 from the dropdown, and then it fills in 499175 in the taxid with no errors. The only box remaining red is the output name at the bottom. Note that I'm not trying to discount what you've said, i'm just not seeing that same behavior on beta currently.
Can you confirm that when you type into the tax_id selector, you see both IDs and names in the dropdown?
—
Reply to this email directly or view it on GitHubhttps://github.com/PATRIC3/patric3_website/issues/193#issuecomment-108918819.

I don't get stuff filled in when I type in the tax ID.
https://youtu.be/2IKricnLv-A

Hi bob, can you try this again please?

It’s getting there - it works erratically at first but then (apparently after everything loads) it seems to do the right thing.
There’s no indication however that the completion is not working because the data is not yet loaded.
On Jun 4, 2015, at 12:20 PM, Dustin Machi <notifications<EMAIL_ADDRESS>wrote:
Hi bob, can you try this again please?
—
Reply to this email directly or view it on GitHubhttps://github.com/PATRIC3/patric3_website/issues/193#issuecomment-108980794.

I noticed that there was a problem on one of the api nodes.  It was working when happened to hit one node, but not the other. I've restarted it so you can try again, and I'm working on figuring out why it was stalling in the first place.
Dustin
On Jun 4, 2015, at 2:03 PM, olsonanl wrote:

It�s getting there - it works erratically at first but then (apparently after everything loads) it seems to do the right thing.
There�s no indication however that the completion is not working because the data is not yet loaded.
On Jun 4, 2015, at 12:20 PM, Dustin Machi <notifications<EMAIL_ADDRESS>wrote:
Hi bob, can you try this again please?
�
Reply to this email directly or view it on GitHubhttps://github.com/PATRIC3/patric3_website/issues/193#issuecomment-108980794.
�
Reply to this email directly or view it on GitHub.

Ah - much better.
On Jun 4, 2015, at 1:04 PM, Dustin Machi <notifications<EMAIL_ADDRESS>wrote:
I noticed that there was a problem on one of the api nodes. It was working when happened to hit one node, but not the other. I've restarted it so you can try again, and I'm working on figuring out why it was stalling in the first place.
Dustin
On Jun 4, 2015, at 2:03 PM, olsonanl wrote:

It�s getting there - it works erratically at first but then (apparently after everything loads) it seems to do the right thing.
There�s no indication however that the completion is not working because the data is not yet loaded.
On Jun 4, 2015, at 12:20 PM, Dustin Machi <notifications@github.commailto:notifications<EMAIL_ADDRESS>wrote:
Hi bob, can you try this again please?
�
Reply to this email directly or view it on GitHubhttps://github.com/PATRIC3/patric3_website/issues/193#issuecomment-108980794.
�
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHubhttps://github.com/PATRIC3/patric3_website/issues/193#issuecomment-108993911.

There is still a problem when entering tax ID. If I reload the page to a clean interface, then start typing 499175 it turn the 4 into a 6  and changes the other digits as well.if I type it really quickly it does the right thing.
On Jun 4, 2015, at 1:04 PM, Dustin Machi <notifications<EMAIL_ADDRESS>wrote:
I noticed that there was a problem on one of the api nodes. It was working when happened to hit one node, but not the other. I've restarted it so you can try again, and I'm working on figuring out why it was stalling in the first place.
Dustin
On Jun 4, 2015, at 2:03 PM, olsonanl wrote:

It�s getting there - it works erratically at first but then (apparently after everything loads) it seems to do the right thing.
There�s no indication however that the completion is not working because the data is not yet loaded.
On Jun 4, 2015, at 12:20 PM, Dustin Machi <notifications@github.commailto:notifications<EMAIL_ADDRESS>wrote:
Hi bob, can you try this again please?
�
Reply to this email directly or view it on GitHubhttps://github.com/PATRIC3/patric3_website/issues/193#issuecomment-108980794.
�
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHubhttps://github.com/PATRIC3/patric3_website/issues/193#issuecomment-108993911.

Yah, this is harder one to solve because solr has those IDs as numbers instead of strings.  I was in the middle of IMing Maulik to see if we could have those as strings like the rest of the IDs in at least the taxonomy core.  In the mean time I'll adjust the delay that happens between when you stop typing and it starts searching so that you have more time to get a full id out before it searches.
On Jun 4, 2015, at 2:16 PM, olsonanl wrote:

There is still a problem when entering tax ID. If I reload the page to a clean interface, then start typing 499175 it turn the 4 into a 6 and changes the other digits as well.if I type it really quickly it does the right thing.
On Jun 4, 2015, at 1:04 PM, Dustin Machi <notifications<EMAIL_ADDRESS>wrote:
I noticed that there was a problem on one of the api nodes. It was working when happened to hit one node, but not the other. I've restarted it so you can try again, and I'm working on figuring out why it was stalling in the first place.
Dustin
On Jun 4, 2015, at 2:03 PM, olsonanl wrote:

It�s getting there - it works erratically at first but then (apparently after everything loads) it seems to do the right thing.
There�s no indication however that the completion is not working because the data is not yet loaded.
On Jun 4, 2015, at 12:20 PM, Dustin Machi <notifications@github.commailto:notifications<EMAIL_ADDRESS>wrote:
Hi bob, can you try this again please?
�
Reply to this email directly or view it on GitHubhttps://github.com/PATRIC3/patric3_website/issues/193#issuecomment-108980794.
�
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHubhttps://github.com/PATRIC3/patric3_website/issues/193#issuecomment-108993911.
—
Reply to this email directly or view it on GitHub.

I got rid of the list of other IDs for auto completion, so it only shows things with an exact match.


Emails can be duplicated

when a user registers it was noted that the system does not differentiate between capital letters and lower case letters. For example, an email can be typed in by one user as<EMAIL_ADDRESS>and another use like<EMAIL_ADDRESS>This creates a duplication.

Added code to the user service file to ensure email data is validated when both lower and upper case.
Error code 500 Undocumented | Error: Internal Server ErrorResponse bodyDownload{   "detail": "400: Email already exists" }
shows up as expected.


Plan 2023.02.08: ODK Seminar at Ontology Summit 2023

Summit plan
Current schedule
Remember everyone to leave 3 -5 minutes for questions in each unit
09:00am: Introduction and use cases and workflows
09:15am: Overview of the ODK toolbox
09:30am: Setting up your own repo
10:00am: Advanced customisation, adding a custom SPARQL check
10:15am: Advanced customisation, adding a custom import module with ont.Makefile
Tutors
@anitacaron
@StroemPhi
@gouttegd
@matentzn
Todos

[x] All taught units have associated documentation pages
[X] Schedule is finished

Finished, great job all.


Conditionally supports pre-aggregated dependencies in MySQL

This adds the table zipkin_dependencies, which is read when present
to return pre-aggregated dependencies (from the zipkin-dependencies job)
If there is no data in the zipkin_dependencies table, results will be
aggregated on-demand. This allows backwards compatibility for those who
haven't added the new table, or run the spark job yet.

@jcarres-mdsol @marcingrzejszczak @shakuzen @eirslett need a second opinion on leaving the "compute on demand" around in mysql. Pro is that demos will show the aggregation without running any spark job. Con is that the data gets too big to return quickly.

Current approach is to aggregate on demand only if a zipkin-dependencies job has never been run. This means the both have to be true...

zipkin_dependencies table exists
zipkin_dependencies table has data

Sounds good. I think most serious users will aggregate the data as it gets unusable quite quickly to calculate dependencies realtime.
Maybe as you said still aggregate realtime is good for demos, if maintenance is real high, killing that code and displayed an empty graph if the spark job has not created the graph is not terrible either methinks.

ok this will end up as zipkin 1.5 after I make sure the dependencies job works fine, etc


link: replace getopts with clap

Related to #2126

perfect!
if you have a chance, you could add a test to check that it fails with 1 or 3 arguments


Feature Request: Raised Hand(s)

A list of participants who has the hand raised 🤚.
An event when someone raise or lower the hand.

Hi,
That sounds like an easy to add feature!
Raise hand:
{"evt":4131,"body":{"bOn":true,"id":16778240},"seq":26}

Event sent to everyone:
{"body":{"add":null,"remove":null,"update":[{"bRaiseHand":true,"id":16778240}]},"evt":7937,"seq":105}

@sysdef I've added an example as well here
https://github.com/RealKeyboardWarrior/zoomer/blob/main/examples/raise-hands/main.go


Facilitate Cardano Node release

Some related PRs so far:

https://github.com/IntersectMBO/cardano-node/pull/5853
https://github.com/IntersectMBO/ouroboros-consensus/pull/1128


dev infra: use json output explicitly when needed

What this PR does
In the deployment scripts of the dev infrastructure, it makes sure that az command explicitly uses json output when the result is piped into jq command. This is useful when one reconfigures the default output of az cli.

@mbukatov fyi the CI failure is_running_on_fork is saying that you will need to recreate this PR as a branch on the Azure/ARO-HCP repo (instead of a branch on your fork)

@mbukatov fyi the CI failure is_running_on_fork is saying that you will need to recreate this PR as a branch on the Azure/ARO-HCP repo (instead of a branch on your fork)

Thanks. I missed that. I guess we need to update https://github.com/Azure/ARO-HCP/blob/main/CONTRIBUTING.md#getting-started as well

Closing since opening a PR from a fork is not allowed. Will open it properly again.


uv_loop_fork does not fix received signals

Version: 1.41.0
Platform: Linux 5.10.0-9-amd64 #1 SMP Debian 5.10.70-1 (2021-09-30) x86_64 GNU/Linux

After fork a child process goes in infinite loop using 100% cpu if a signal was received just before fork.
#include <uv.h>
#include <unistd.h>

void nosignal(uv_signal_t* h, int s){}
void nohandle(uv_handle_t* h){}

int main(int argc, char** argv)  {
    uv_loop_t loop;
    uv_loop_init(&loop);

    uv_signal_t sig;
    uv_signal_init(&loop, &sig);
    uv_signal_start(&sig, &nosignal, SIGHUP);

    int pid = getpid();
    kill(pid, SIGHUP);
    if (!fork()) {
        uv_loop_fork(&loop);
        uv_close((uv_handle_t*)&sig, &nohandle);
    }
    uv_run(&loop, UV_RUN_DEFAULT);
    return 0;
}

This code is the simplest way I found to reproduce the bug. In real life code fork was called right inside the signal handler.
Original process does nothing after fork. It runs loop and waits for signal. It is ok.
Child process should exit from loop after dispatching the close event, but it does not do so. It runs loop without any timeout using 100% of cpu.
This happens because libuv cannot close the signal handler. In uv__finish_close (unix/core.c) there is a check if (sh->caught_signals > sh->dispatched_signals) { that leads to uv__make_close_pending of signal. In normal case signal event is dispatched on the next loop iteration but after fork it would never happen. Signal should be read from loop->signal_pipefd that was closed and recreated in uv_loop_fork. So caught_signals is always bigger than dispatched_signals and loop keep running. And closing handles leads to epoll_wait timeout being 0 and it leads to 100% cpu load.
I think that uv_loop_fork should reset caught_signals and dispatched_signals for all signals in all loops. Signals written to loop->signal_pipefd was sent to master process, so child should not receive them. I can make a PR with this fix.

Sounds reasonable. Though I would strongly suggest not using fork, as it is very hard to reset all state correctly, and likely impossible in a program that uses threads or signals.


Support - FAQ

Stub.
Feel free to comment here if you have ideas on what this guide should cover. Link to potential resources too.

There is no specific page for FAQ at the current structure. I'm hoping "how tos" and "concepts" cover this need well enough.


Not able to install

Hi,
I am working since hours on the isntallation of the integration but it I am not able to have it working.
I can find it on '/config/custom_components/solarwatt_energymanager' but I can not found it on Configuration -> Integrations
I am using a RP4 with Home Assistant 2022.3.5
Home Assistant OS
Any idea on what can be wrong?
Thanks in advance
ZobDe

Hi,
What version are you using and how did you install it? The latest is 0.3.2.
I installed the component using HACS.
Mas
On Wed, 16 Mar 2022, 11:41 zobde74, @.***> wrote:

Hi,
I am working since hours on the isntallation of the integration but it I
am not able to have it working.
I can find it on '/config/custom_components/solarwatt_energymanager' but I
can not found it on Configuration -> Integrations
I am using a RP4 with Home Assistant 2022.3.5
Home Assistant OS
Any idea on what can be wrong?
Thanks in advance
ZobDe
—
Reply to this email directly, view it on GitHub
https://github.com/Mas2112/solarwatt-energymanager-homeassistant/issues/2,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABJLA7U2EIUQO2SMZ6EPADVAG3FVANCNFSM5Q3OOQHA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi Mas,
I started using the last available 0.3.2 but I also tested some previous versions but the result is the same.
I am installing it via HACS following the instructions.
I can see it installed on '/config/custom_components/solarwatt_energymanager' but I can not found it on Configuration -> Integrations so not able to use it.
Also on HACS after the installation I can not see it but if I install it again it show it is already installed.
I also tried to remove it and start from the beginning. I rebooted many times HA. I also rebooted completely the Raspberry as test
Thanks
ZobDe

Hi ZobDe,
have you tried adding the device by using the "+ADD INTEGRATION" button? It
should show up in the list.
[image: image.png]
Best regards,
Mas
On Thu, Mar 17, 2022 at 8:57 AM zobde74 @.***> wrote:

Hi Mas,
I started using the last available 0.3.2 but I also tested some previous
versions but the result is the same.
I am installing it via HACS following the instructions.
I can see it installed on
'/config/custom_components/solarwatt_energymanager' but I can not found it
on Configuration -> Integrations so not able to use it.
Also on HACS after the installation I can not see it but if I install it
again it show it is already installed.
I also tried to remove it and start from the beginning. I rebooted many
times HA. I also rebooted completely the Raspberry as test
Thanks
ZobDe
—
Reply to this email directly, view it on GitHub
https://github.com/Mas2112/solarwatt-energymanager-homeassistant/issues/2#issuecomment-1070497704,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABJLA3LNX7ZNOH4BPWWUXLVALQWZANCNFSM5Q3OOQHA
.
You are receiving this because you commented.Message ID:
@.***
com>

Hi Mas, sure , I did it.
I just deleted all and started again from the beginning.
I can find following logs on HA

This error is coming from a custom integration.
Logger: custom_components.hacs
Source: custom_components/hacs/base.py:597
Integration: HACS (documentation, issues)
First occurred: 21:19:51 (4 occurrences)
Last logged: 21:19:51
A timeout of 60! seconds was encountered while downloading https://raw.githubusercontent.com/Mas2112/solarwatt-energymanager-homeassistant/0.3.2/custom_components/solarwatt_energymanager/strings.json, check the network on the host running Home Assistant
A timeout of 60! seconds was encountered while downloading https://raw.githubusercontent.com/Mas2112/solarwatt-energymanager-homeassistant/0.3.2/custom_components/solarwatt_energymanager/manifest.json, check the network on the host running Home Assistant
A timeout of 60! seconds was encountered while downloading https://raw.githubusercontent.com/Mas2112/solarwatt-energymanager-homeassistant/0.3.2/custom_components/solarwatt_energymanager/energy_manager.py, check the network on the host running Home Assistant
A timeout of 60! seconds was encountered while downloading https://raw.githubusercontent.com/Mas2112/solarwatt-energymanager-homeassistant/0.3.2/custom_components/solarwatt_energymanager/sensor.py, check the network on the host running Home Assistant

If I put the 4 urls on my browser I can download the them, but HA is not able ( as I understand from the logs ).
My PC and HA are on the same network so there are no filters on my ISP
Regards
Denis

Hello Mas,
I get it solved.
I manually uploaded the 4 files showed on the logs. In reality I imported all 4 files but after a reboot file sensor.py was again missing. Re-uploaded it again, reboot and now the integration works.
I provided Energy manager IP and now I have 1 device and 17 entities.
I have 3 myreserve installed ( 36Kwh in total ) with 1 Energy manager.
The integration show the details like battery state of charge only for 1 myreserve :-(
Kind Regards
Denis

Hi Denis,
3 myreserve battieries? Cool. I don't have any, so I don't know what the
data looks like. It would be very helpful if you can send me a sample json
file from your energy manager. Then I can add support for multiple
batteries.
Can you send me a sample JSON file?
in your browser, you can browse to the address below, and it will return
json data.
http:///rest/kiwigrid/wizard/devices
Best regards,
Mas
On Mon, Mar 21, 2022 at 10:32 AM zobde74 @.***> wrote:

Hello Mas,
I get it solved.
I manually uploaded the 4 files showed on the logs. In reality I imported
all 4 files but after a reboot file sensor.py was again missing.
Re-uploaded it again, reboot and now the integration works.
I provided Energy manager IP and now I have 1 device and 17 entities.
I have 3 myreserve installed ( 36Kwh in total ) with 1 Energy manager.
The integration show the details like battery state of charge only for 1
myreserve :-(
Kind Regards
Denis
—
Reply to this email directly, view it on GitHub
https://github.com/Mas2112/solarwatt-energymanager-homeassistant/issues/2#issuecomment-1073679311,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABJLAYRSDVYLDXB5PPVE5LVBA63RANCNFSM5Q3OOQHA
.
You are receiving this because you commented.Message ID:
@.***
com>

Hi Mas,
here the json
devices.zip
Thanks
Denis

Hi Denis,
thanks. Is it OK if I use your file for my unit tests? I will modify
the data so all the personal data will be fake.
I will need some time to get a version running with multiple batteries.
Best regards,
Mas
On Mon, Mar 28, 2022 at 8:42 AM zobde74 @.***> wrote:

Hi Mas,
here the json
devices.zip
https://github.com/Mas2112/solarwatt-energymanager-homeassistant/files/8360217/devices.zip
Thanks
Denis
—
Reply to this email directly, view it on GitHub
https://github.com/Mas2112/solarwatt-energymanager-homeassistant/issues/2#issuecomment-1080259879,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABJLA7ICNDGMONU3QNMIITVCFIEHANCNFSM5Q3OOQHA
.
You are receiving this because you commented.Message ID:
@.***
com>

Hi Mas,
sure. I didn't realized personal data was included.
May be you can download the file and we delete the link...
Thanks
Denis

Hi Denis,
I have created release 0.4.3-alpha. Please try it out and see if it works.
I have already tested the power and energy values, which is working for me.
But I can't test the batteries.
Best regards,
Mas
On Mon, Mar 28, 2022 at 11:07 AM zobde74 @.***> wrote:

Hi Mas,
sure. I didn't realized personal data was included.
May be you can download the file and we delete the link...
Thanks
Denis
—
Reply to this email directly, view it on GitHub
https://github.com/Mas2112/solarwatt-energymanager-homeassistant/issues/2#issuecomment-1080387005,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABJLA57J4SV7FXGK2YVFA3VCFZEXANCNFSM5Q3OOQHA
.
You are receiving this because you commented.Message ID:
@.***
com>

Hi Mas,
I completely removed the addon and I was not able to install if following the instructions. I manually uploaded the files like I did last time it was working.
Integration is installed but not working.
Here the logs:
Logger: homeassistant.components.sensor
Source: custom_components/solarwatt_energymanager/sensor_factory.py:293
Integration: Sensore (documentation, issues)
First occurred: 15:21:02 (2 occurrences)
Last logged: 15:21:31
Error while setting up solarwatt_energymanager platform for sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 249, in _async_setup_platform
await asyncio.shield(task)
File "/config/custom_components/solarwatt_energymanager/sensor.py", line 48, in async_setup_entry
entities = create_sensors(coordinator, em_data)
File "/config/custom_components/solarwatt_energymanager/sensor_factory.py", line 75, in create_sensors
create_battery_converter_sensors(
File "/config/custom_components/solarwatt_energymanager/sensor_factory.py", line 293, in create_battery_converter_sensors
em.BatteryConverterDevice.TAG_VOLTAGE_BATTERY_CELL_STRING,
AttributeError: type object 'BatteryConverterDevice' has no attribute 'TAG_VOLTAGE_BATTERY_CELL_STRING'
Thanks
Denis

Hi Denis,
I fixed that bug. Can you try 0.4.4-alpha? My component doesn't rely on the order of the values, so it should be able to handle that change you saw.
Here is an animated gif on how I setup the component via HACS.

What doesn't work in your case?
Mas

HI Mas,
here the error log duirng the installation.
Questo errore ha avuto origine da un'integrazione personalizzata.
Logger: custom_components.hacs
Source: custom_components/hacs/base.py:615
Integration: HACS (documentation, issues)
First occurred: 09:52:06 (8 occurrences)
Last logged: 09:53:26
A timeout of 60! seconds was encountered while downloading https://raw.githubusercontent.com/Mas2112/solarwatt-energymanager-homeassistant/0.3.2/custom_components/solarwatt_energymanager/manifest.json, check the network on the host running Home Assistant. This is not a problem with HACS but how your host communicates with GitHub
A timeout of 60! seconds was encountered while downloading https://raw.githubusercontent.com/zachowj/hass-node-red/v1.0.7/custom_components/nodered/const.py, check the network on the host running Home Assistant. This is not a problem with HACS but how your host communicates with GitHub
A timeout of 60! seconds was encountered while downloading https://raw.githubusercontent.com/zachowj/hass-node-red/v1.0.7/custom_components/nodered/discovery.py, check the network on the host running Home Assistant. This is not a problem with HACS but how your host communicates with GitHub
A timeout of 60! seconds was encountered while downloading https://raw.githubusercontent.com/zachowj/hass-node-red/v1.0.7/custom_components/nodered/sensor.py, check the network on the host running Home Assistant. This is not a problem with HACS but how your host communicates with GitHub
A timeout of 60! seconds was encountered while downloading https://raw.githubusercontent.com/zachowj/hass-node-red/v1.0.7/custom_components/nodered/manifest.json, check the network on the host running Home Assistant. This is not a problem with HACS but how your host communicates with GitHub
I can manually import the 5 files and than the integration works.

Here the integration installed. Well done!!!

Just a curiosity. Valure "EnergyManager PowerProduced " is showing the sum of the 3 strings I have. Are also the single value of the strings available or only the sum?
For your info, I have 3 strings from the roof ( 6Kw+6Kw+7Kw). Each of this string is connected to a battery pack. So every battery is connected to a single string. Than 2 batteries are going to a Fronius Inverter 12Kw ( the 2 strings with 6Kw) and the other battery( string 7Kw) is going to another Fronius inverter 7Kw.
Denis

Hi Denis,
I just realized it's really hard to differentiate which value is for which
battery, and which is from the energy manager. I think I need to improve
that.
I've only parsed out the total in and out data and the battery data.
Perhaps you can get the string data from the fronius integration.
https://www.home-assistant.io/integrations/fronius/
I have a Kostal inverter and pull my string data from there directly.
Best regards,
Mas
On Tue, 5 Apr 2022, 11:06 zobde74, @.***> wrote:

Here the integration installed. Well done!!!
[image: image]
https://user-images.githubusercontent.com/79097946/161718007-05c2d954-edfb-45fe-9898-8e6c6b46f444.png
[image: image]
https://user-images.githubusercontent.com/79097946/161718141-f933441e-824d-488a-a9fb-2ffa3c6bd802.png
[image: image]
https://user-images.githubusercontent.com/79097946/161718215-390ac503-652e-4f62-901b-185393610555.png
[image: image]
https://user-images.githubusercontent.com/79097946/161718297-17d69b51-4021-4a30-b47a-dcc9b2fc3d01.png
Just a curiosity. Valure "EnergyManager PowerProduced " is showing the sum
of the 3 strings I have. Are also the single value of the strings available
or only the sum?
For your info, I have 3 strings from the roof ( 6Kw+6Kw+7Kw). Each of this
string is connected to a battery pack. So every battery is connected to a
single string. Than 2 batteries are going to a Fronius Inverter 12Kw ( the
2 strings with 6Kw) and the other battery( string 7Kw) is going to another
Fronius inverter 7Kw.
Denis
—
Reply to this email directly, view it on GitHub
https://github.com/Mas2112/solarwatt-energymanager-homeassistant/issues/2#issuecomment-1088454638,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABJLA7KXVZFE2PVW6J3RWDVDP7AHANCNFSM5Q3OOQHA
.
You are receiving this because you were assigned.Message ID:
@.***
com>

Ciao Mas,
I am using Fronius intergation but it is useless as till the batteries are not fully charged to the inverters arrive only what the house is using. When the battery are fully charged than the inverters show what arrive from the roof.
Regards
Denis

Hi Denis,
It looks like something is blocking HACS from downloading the files from
raw.githubusercontent.com. perhaps a firewall or something?
Best regards,
Mas
On Tue, 5 Apr 2022, 09:58 zobde74, @.***> wrote:

HI Mas,
here the error log duirng the installation.
Questo errore ha avuto origine da un'integrazione personalizzata.
Logger: custom_components.hacs
Source: custom_components/hacs/base.py:615
Integration: HACS (documentation, issues)
First occurred: 09:52:06 (8 occurrences)
Last logged: 09:53:26
A timeout of 60! seconds was encountered while downloading
https://raw.githubusercontent.com/Mas2112/solarwatt-energymanager-homeassistant/0.3.2/custom_components/solarwatt_energymanager/manifest.json,
check the network on the host running Home Assistant. This is not a problem
with HACS but how your host communicates with GitHub
A timeout of 60! seconds was encountered while downloading
https://raw.githubusercontent.com/zachowj/hass-node-red/v1.0.7/custom_components/nodered/const.py,
check the network on the host running Home Assistant. This is not a problem
with HACS but how your host communicates with GitHub
A timeout of 60! seconds was encountered while downloading
https://raw.githubusercontent.com/zachowj/hass-node-red/v1.0.7/custom_components/nodered/discovery.py,
check the network on the host running Home Assistant. This is not a problem
with HACS but how your host communicates with GitHub
A timeout of 60! seconds was encountered while downloading
https://raw.githubusercontent.com/zachowj/hass-node-red/v1.0.7/custom_components/nodered/sensor.py,
check the network on the host running Home Assistant. This is not a problem
with HACS but how your host communicates with GitHub
A timeout of 60! seconds was encountered while downloading
https://raw.githubusercontent.com/zachowj/hass-node-red/v1.0.7/custom_components/nodered/manifest.json,
check the network on the host running Home Assistant. This is not a problem
with HACS but how your host communicates with GitHub
I can manually import the 5 files and than the integration works.
—
Reply to this email directly, view it on GitHub
https://github.com/Mas2112/solarwatt-energymanager-homeassistant/issues/2#issuecomment-1088385518,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABJLA5ZLW6VN3JS256PB3LVDPXCNANCNFSM5Q3OOQHA
.
You are receiving this because you were assigned.Message ID:
@.***
com>

Hi Mas,
I started using the last available 0.3.2 but I also tested some previous
versions but the result is the same.
I am installing it via HACS following the instructions.
I can see it installed on '/config/custom_components/solarwatt_energymanager'
but I can not found it on Configuration -> Integrations so not able to use
it.
Also on HACS after the installation I can not see it but if I install it
again it show it is already installed.
I also tried to remove it and start from the beginning. I rebooted many
times HA. I also rebooted completely the Raspberry as test
Thanks
ZobDe
Il giorno mer 16 mar 2022 alle ore 17:45 Mas2112 @.***>
ha scritto:

Hi,
What version are you using and how did you install it? The latest is 0.3.2.
I installed the component using HACS.
Mas
On Wed, 16 Mar 2022, 11:41 zobde74, @.***> wrote:

Hi,
I am working since hours on the isntallation of the integration but it I
am not able to have it working.
I can find it on '/config/custom_components/solarwatt_energymanager' but
I
can not found it on Configuration -> Integrations
I am using a RP4 with Home Assistant 2022.3.5
Home Assistant OS
Any idea on what can be wrong?
Thanks in advance
ZobDe
—
Reply to this email directly, view it on GitHub
<
https://github.com/Mas2112/solarwatt-energymanager-homeassistant/issues/2
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AABJLA7U2EIUQO2SMZ6EPADVAG3FVANCNFSM5Q3OOQHA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/Mas2112/solarwatt-energymanager-homeassistant/issues/2#issuecomment-1069341648,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AS3PAWQGSUY7DKKBOKVY6ELVAIF4BANCNFSM5Q3OOQHA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***
com>

Hi ZobDe,
There is a newer pre-release version. Can you try that?
Best regards,
Mas
On Tue, Oct 11, 2022, 10:17 zobde74 @.***> wrote:

Hi Mas,
I started using the last available 0.3.2 but I also tested some previous
versions but the result is the same.
I am installing it via HACS following the instructions.
I can see it installed on
'/config/custom_components/solarwatt_energymanager'
but I can not found it on Configuration -> Integrations so not able to use
it.
Also on HACS after the installation I can not see it but if I install it
again it show it is already installed.
I also tried to remove it and start from the beginning. I rebooted many
times HA. I also rebooted completely the Raspberry as test
Thanks
ZobDe
Il giorno mer 16 mar 2022 alle ore 17:45 Mas2112 @.***>
ha scritto:

Hi,
What version are you using and how did you install it? The latest is
0.3.2.
I installed the component using HACS.
Mas
On Wed, 16 Mar 2022, 11:41 zobde74, @.***> wrote:

Hi,
I am working since hours on the isntallation of the integration but it
I
am not able to have it working.
I can find it on '/config/custom_components/solarwatt_energymanager'
but
I
can not found it on Configuration -> Integrations
I am using a RP4 with Home Assistant 2022.3.5
Home Assistant OS
Any idea on what can be wrong?
Thanks in advance
ZobDe
—
Reply to this email directly, view it on GitHub
<

https://github.com/Mas2112/solarwatt-energymanager-homeassistant/issues/2

,
or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AABJLA7U2EIUQO2SMZ6EPADVAG3FVANCNFSM5Q3OOQHA

.
You are receiving this because you are subscribed to this
thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
<
https://github.com/Mas2112/solarwatt-energymanager-homeassistant/issues/2#issuecomment-1069341648
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AS3PAWQGSUY7DKKBOKVY6ELVAIF4BANCNFSM5Q3OOQHA
.
Triage notifications on the go with GitHub Mobile for iOS
<
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
<
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

—
Reply to this email directly, view it on GitHub
https://github.com/Mas2112/solarwatt-energymanager-homeassistant/issues/2#issuecomment-1274285301,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABJLA4KQZYPVK7VU5ZFBGDWCUPBPANCNFSM5Q3OOQHA
.
You are receiving this because you modified the open/close state.Message
ID: <Mas2112/solarwatt-energymanager-homeassistant/issues/2/1274285301@
github.com>

Hi MAS
it seems to work.
I can see more sensor and they are populated with a value.
Now I need to study and understand what it means the new sensors.
Some suggestions:

as example, already have a average value for the multiple sensors.
As example, I have 3 batteries and you show 3 sensors, one per battery and
this is OK. But I am also calculating the total battery charge because I
can not put ( and it makes no sense in my opinion ) 3 value on the home
assistant dashboard. The same also for other sensors like WorkProduced.
Another interesting value can be the 3 values for the incoming energy
from roof but may be this are now included now (EnergyManager WorkACIn?)
and I didn't realized before.
Last thing, it is not easy to understand the multiple sensor to what it
is referring. As example, I have 3 EnergyManager WorkACIn and not easy to
link the value to the right section of the system. But here no idea about
the data Solarwatt is passing. I provided a name like Battery 1, Battery 2
etc but no idea how this is reported. It seems this is not included on json
file

Denis

Hi Denis,
Im glad this is working for you!
It seems you can calculate averages using other features or integrations of
HA. See link below.
https://community.home-assistant.io/t/how-to-create-sensor-that-will-be-a-average-of-other-sensors/167393/11
As for the problem with multiple sensors, I'll have to see what I can do. I
have only one inverter and no battery, so my setup is very simple.
I think you sent me your JSON file already, I'll take a look again...
Mas
On Tue, Oct 11, 2022, 17:05 zobde74 @.***> wrote:

Hi MAS
it seems to work.
I can see more sensor and they are populated with a value.
Now I need to study and understand what it means the new sensors.
Some suggestions:

as example, already have a average value for the multiple sensors.
As example, I have 3 batteries and you show 3 sensors, one per battery and
this is OK. But I am also calculating the total battery charge because I
can not put ( and it makes no sense in my opinion ) 3 value on the home
assistant dashboard. The same also for other sensors like WorkProduced.
Another interesting value can be the 3 values for the incoming energy
from roof but may be this are now included now (EnergyManager WorkACIn?)
and I didn't realized before.
Last thing, it is not easy to understand the multiple sensor to what it
is referring. As example, I have 3 EnergyManager WorkACIn and not easy to
link the value to the right section of the system. But here no idea about
the data Solarwatt is passing. I provided a name like Battery 1, Battery 2
etc but no idea how this is reported. It seems this is not included on json
file

Denis
—
Reply to this email directly, view it on GitHub
https://github.com/Mas2112/solarwatt-energymanager-homeassistant/issues/2#issuecomment-1274846097,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABJLAYTQQ4QWNEIK5ETTHTWCV62TANCNFSM5Q3OOQHA
.
You are receiving this because you modified the open/close state.Message
ID: <Mas2112/solarwatt-energymanager-homeassistant/issues/2/1274846097@
github.com>


Exclude uninteresting directories from coverage reports

The following should be excluded:
/usr/include
third_party/
test/
(basically, we want src/ and include/ from grpc root)

There is an example in run_node.sh of how extract coverage information for particular directories from the full coverage information.

@nicolasnoble is this still relevant?

It's still relevant, but probably not for GA actually.


Slackroll 'list-transient' does not always notify multiple kernel installations.

The basic scenario is...
run 'slackroll kernel-upgrade'  followed by 'slackroll upgrade' for other package upgrades.
Reboot.
run 'slackroll list-transient'.  No mention of multiple kernels present.

Note: It's been several years since I stopped using Slackware. I guess I should install it in a virtual machine for cases like this one, and maybe reopen the "maintainer needed" issue.
If I remember correctly, a local package with multiple installed versions is not in a transient state, because as long as one of the versions matches a remote one, it's OK.
When the list of local packages is rebuilt at some point, however, slackroll will warn you if it detects packages with multiple local versions. You may see this message after "Rebuilding local package list..." but not anymore until the local package list is rebuilt again. That's intended behavior, as a one-time warning, and the warning will be gone after you run kernel-clean, which should probably be run after successfully rebooting to the new kernel.
Does the explanation above make sense to you?

I can confirm. You will only see the error once until the package list is changed again.
Ref: https://github.com/rg3/slackroll/blob/fcfd43b2d8c9808b583b0f66a06bef4796d03752/slackroll#L553

Note: It's been several years since I stopped using Slackware. I guess I should install it in a virtual machine for cases like this one, and maybe reopen the "maintainer needed" issue.

Indeed, I'm rather happy that you still expend the effort to support this tool despite your having no use for it .  I've mentioned it a few times over in the [https://www.linuxquestions.org/questions/slackware-14/](Linuxquestions Slackware forum) because it's a very nice tool.

If I remember correctly, a local package with multiple installed versions is not in a transient state, because as long as one of the versions matches a remote one, it's OK.
When the list of local packages is rebuilt at some point, however, slackroll will warn you if it detects packages with multiple local versions. You may see this message after "Rebuilding local package list..." but not anymore until the local package list is rebuilt again. That's intended behavior, as a one-time warning, and the warning will be gone after you run kernel-clean, which should probably be run after successfully rebooting to the new kernel.
Does the explanation above make sense to you?

Updating the documentation to say something similar to the above would be the most cost-effective solution to this issue.

Updating the documentation to say something similar to the above would
be the most cost-effective solution to this issue.

Fair enough. Given that rebuilding the local packages database is a step that affects many different operations, adding that detail to the individual documentation of the affected operations would be too complicated. I think a brief mention in the program's main webpage or in the FAQ would be good enough.

I shall assume that someone will do that very thing.  Closing the issue.

Lets keep this one open as a reminder that the documentation changes need to be made.


Deprecation notice and proposal to archive this repo

As discussed on the Aries WG call today, we're proposing that we archive this repo. This PR adds a deprecation notice to the README prior to that archiving.
cc @ryjones @TelegramSam @burdettadam @swcurran @TimoGlastra

@dbluhm @swcurran I've added this to the PR


Add run_if property for existing LUCI try builders

With Cocoon's filtering support: https://github.com/flutter/cocoon/pull/917, run_if in this PR enables to trigger only corresponding builders rather than all.
This will somewhat relieve the execution burden of try pool.
Related issue: https://github.com/flutter/flutter/issues/62841

Is this a noop until we land the code to act on it?

No. Existing cocoon will just ignore the run_if without flutter/cocoon#917.


sqlite3 module version issue

sqlite3 is built into python, but I'm getting a interesting issue. Unsure if it's alpine-linux or what:
docker run -i -t python:3.5.1-alpine /bin/sh
python
import sqlite3
conn = sqlite3.connect(":memory:")
sqlite3.sqlite_version
sqlite3.version
sqlite3.__file__
conn.enable_load_extension(True)

useful output:
>>> sqlite3.sqlite_version
'3.9.2'
>>> sqlite3.version
'2.6.0'
>>> conn.enable_load_extension(True)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: 'sqlite3.Connection' object has no attribute 'enable_load_extension'

Trying to track down the error leads mostly to issues with 2x python. the enable_load_extension wasn't available until 2.7.  It should however be in 3.5.
this is the expected behavior:
>>> import sqlite3
>>> conn = sqlite3.connect(":memory:")
>>> sqlite3.sqlite_version
'3.11.0'
>>> sqlite3.version
'2.6.0'
>>> sqlite3.__file__
'/usr/lib/python3.5/sqlite3/__init__.py'
>>> conn.enable_load_extension(True)

After a good night's rest, the Dockerfile would need a --enable-loadable-sqlite-extensions flag:
./configure --enable-loadable-sqlite-extensions --enable-shared --enable-unicode=ucs4

in the make section. Reviewing the standards, it seems like there's little consistency about turning this on or off. My primary use is in a web framework using Spatialite extensions, and probably a json1 extension.
pull request https://github.com/docker-library/python/pull/107


[Unity][Tracking Issue] Phase Ordering

Per the previous discussion of instituting some explicit phase ordering in Relax to account for the fact that some passes already rely on each other, this is a proposal for adopting some of the measures from the discussion.

[ ] P1. Add a comment in transform.h or a README.md in include/tvm/relax that describes the known pass dependencies.
[ ] P2. Include LambdaLifting as a final step after parsing.
[ ] P3. Implement phase tracking during the build.

[ ] P3.1. Use a module attribute to indicate the phase. The phases will be "ingestion" (0, between parsing and normalization), "optimization" (1, applying high-level optimizations), "lowering" (2, some lower-level optimizations that rely on legalizing operators, eliminating purity tracking, and device-specific properties), and "build" (3, the final lowering of builtins to PackedFuncs and preparing for VM code generation). It will be important to mark transition points accordingly (LambdaLifting as 0->1, LegalizeOps as 1->2, VMBuiltinLower as 2->3).
[ ] P3.2. Include checks for the expected phase in each pass. This will largely be boilerplate.
[ ] P3.3. Modify pass implementations to account for phase invariants. For example, after phase 0, passes do not have to handle inner functions (except for BYOC-specific ones, unless we want to change that).
[ ] P3.4. Enforce certain phase invariants in the well-formed checker, tied to the module attribute for phase.

Inner functions should not be permitted after phase 0 (we can make an exception for those used by BYOC or change the implementation of RunCodegen).
Dataflow blocks should not permitted after phase 1.
All functions must be marked either impure or use the relax.force_pure attribute after phase 2. Additionally, there should be no remaining uses of call_pure_packed or invoke_pure_closure after phase 2. (These are both already implemented by the RemovePurityTracking pass; this would just enforce the invariant.)
There should be no remaining uses of call_tir and call_dps_packed after phase 2.
We could also consider enforcing that all arithmetic operators will have been legalized after phase 2 (by checking only for a whitelist of builtins that are lowered in phase 3).

This list is meant as a starting point for further discussion, as there were still some unresolved disagreements in the earlier thread. By dividing the implementation into steps, this could help make the discussion more concrete by focusing on implementation.
In terms of implementation, the biggest question centers on the BYOC passes, which rely on inner functions. This is the only major case where the current implementation of passes differs from what was proposed (namely, eliminating inner functions early on). We should also be certain of which passes are responsible for updating the phase attribute (those proposed above under P3.1 are all non-optional passes, so I would argue they are reasonable candidates).
We may also consider trying to enforce some finer-grained ordering constraints (see the "megapasses" in the discussion thread), though there did not seem to be wide support for this; it may suffice to note these constraints in the documentation without enforcing them directly.
cc @quic-sanirudh

Note from June 13, 2023, community meeting: Documenting pass dependencies might be best to do on the individual passes' docstrings. This would likely be easiest to maintain in the long term as well.


MAINT: changing github org name

I kept the licence file changes separate in case we want to deal with it differently.
I'm opening the PRs first and then change org, and thus I expect CI to not work properly until the actual name change is done.

I'm going ahead and merge this now, if anyone sees any issue with the content of the PR, we should do a follow-up.


Updates to README.md

Added:

required redis modules
how to change used lnbits instance

Updated:

Readme used legend.lnbits.com while lnbits.thespielplatz.com is used, leading to confusing "Wallet not found" error

I noticed that the constants file should be changed to legend.lnbits.com, not the other way around :)


Private aks cluster instructions bug

Followed the documentation to create a privet aks cluster and private endpoint .
az aks create -n  -g  --load-balancer-sku standard --enable-private-cluster --enable-managed-identity --assign-identity  --private-dns-zone [system|none]
And following steps for connecting with private end point , but instructions are not very clear, stuck at following
First thing private cluster did not created private DNS
And did not find node ResourceGroup as per the instructions on the page
"Go to the node resource group in the Azure portal.
Select the private DNS zone and record:
The name of the private DNS zone, which follows the pattern *.privatelink..azmk8s.io
The name of the A record (excluding the private DNS name)
The time-to-live (TTL)"

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

ID: 0b68f2c4-bb6c-11a2-6c61-8af4057a2438
Version Independent ID: e3498bed-1447-6841-8353-9f1b5d3dc8df
Content: Create a private Azure Kubernetes Service cluster - Azure Kubernetes Service
Content Source: articles/aks/private-clusters.md
Service: container-service
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@VIPULKAM
Thanks for your feedback! We will investigate and update as appropriate.

Hi @VIPULKAM Please refer below document for creating private AKS cluster and AKS Private endpoint. Thanks
https://learn.microsoft.com/en-us/samples/azure/azure-quickstart-templates/private-aks-cluster/
https://learn.microsoft.com/en-us/cli/azure/network/private-endpoint?view=azure-cli-latest

@VIPULKAM
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


Delta format of data storage for lake database

Can you confirm Delta is supported now? Statement in this section https://learn.microsoft.com/en-us/azure/synapse-analytics/database-designer/concepts-lake-database#data-storage suggests so, but the diagram above doesn't mention of Delta. Also, I couldn't see delta as an option in Data format dropdown (Parquet and Delimited Text are only options). Is there a migration needed for existing instances?

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

ID: 3fce5979-39ff-6306-65a8-11727240fa83
Version Independent ID: 4f288f22-593d-5d1d-b302-48f2a21db5fa
Content: Azure Synapse lake database concepts - Azure Synapse Analytics
Content Source: articles/synapse-analytics/database-designer/concepts-lake-database.md
Service: synapse-analytics
Sub-service: database-editor
GitHub Login: @matt1883
Microsoft Alias: mahi

@RPaladi
Thanks for your feedback! We will investigate and update as appropriate.

@RPaladi
Azure Synapse Analytics supports Delta Lake integration via Poly Base as of my knowledge cutoff date of September 2021. Delta Lake files can be saved in the Azure Data Lake Storage Gen2 account associated with your Synapse workspace and accessed in Synapse via Poly Base external tables. However, updates may have been made since then, so it's best to consult the most recent Microsoft documentation to confirm current Delta Lake support in Synapse Analytics.
Regarding your query about the diagram and data format dropdown not mentioning Delta, it's possible that the diagram and documentation haven't been updated to reflect the most recent Delta Lake integration. In terms of migration, if you have existing instances that are not in Delta Lake format, you will need to migrate your data to Delta Lake. The process will vary depending on the specifics of your data and workflow, but Microsoft provides guidance on how to migrate data to Delta Lake in their documentation.
For an issue like this, I'd recommend you create a support ticket since the support team will be able to respond much more quickly and have a conversation with you to figure out what could be going on.

thanks for your response @RamanathanChinnappan-MSFT . Can you check on the correctness of the documentation whether or not Delta as a data format is supported in Lake Database? I'll raise ticket, as suggested, for the migration of existing instance

@RPaladi
Thank you for bringing this to our attention. I've assigned this issue to the author who will investigate and update as appropriate.
For an issue like this, I'd recommend you create a support ticket since the support team will be able to respond much more quickly and have a conversation with you to figure out what could be going on.


Improve overload selector

As noted in https://github.com/numba/numba/pull/1869#issuecomment-216665765, this improves the  overload selector so that the most specific signature is selected irrespective to the order in which signatures are added.  The old system picks the first matching definition, leading to difficulty to provide a generic (fallback) implementation.

Two remarks:

since the OverloadSelector class is becoming complex, we should have unit-tests for it
it's good that this PR discovers an ambiguous overload :-) A way to resolve it would be to define a Sized abstract type class, and let both Sequence and BaseTuple inherit from it. Then the current Sequence implementation of bool() (which simply calls len()) can be adapted for Sized.

[Current coverage][cc-pull] is 87.46%

Merging [#1879][cc-pull] into [master][cc-base-branch] will decrease coverage by -<.01%

File .../targets/optional.py was modified. more

Misses +1
Partials 0
Hits -1

@@             master      #1879   diff @@
==========================================
  Files           292        292
  Lines         51981      52008    +27
  Methods           0          0
  Messages          0          0
  Branches       5368       5375     +7
==========================================
+ Hits          45461      45484    +23
- Misses         5603       5606     +3
- Partials        917        918     +1

Powered by Codecov. Last updated by [098c991...c4da1ed][cc-compare]
[cc-base-branch]: https://codecov.io/gh/numba/numba/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/numba/numba/compare/098c99190ff5b851ef6ad7a2262181d2def1c3d0...c4da1ed19da5fc99f0141fb3d3051e58ceb6c509
[cc-pull]: https://codecov.io/gh/numba/numba/pull/1879?src=pr

Here is a proposed alternate implementation allowing somewhat better properties:
https://gist.github.com/pitrou/5eb9ba8bc894a6696807c61ebdbefb55

@pitrou with your patch, one can have the following situation: https://gist.github.com/sklam/1844c0ea78b3606480645dfb8631f235
I find it weird that the first case is not ambiguous and it picks 1.  It may lead to unintentional bug.  Generalizing this, would it be better to not allow an overload that is simultaneously widening (generalizing) and narrowing (specializing)?

I find it weird that the first case is not ambiguous and it picks 1. It may lead to unintentional bug.

I thought of it as a feature, but you're right it may be confusing.

Generalizing this, would it be better to not allow an overload that is simultaneously widening (generalizing) and narrowing (specializing)?

I'm not sure how to do that, perhaps by simplifying the score metric?

I'm not sure how to do that, perhaps by simplifying the score metric?

I also don't like how the final score is the sum of each args' score.  It is hard to reason about. It will have weird breaking point (point that suddenly becomes ambiguous).
I prefer my scheme that is more restrictive.  For this case (simultaneously widening and narrowing), it will treat the two signatures as disjoint and have equal level of genericity.

I also don't like how the final score is the sum of each args' score. It is hard to reason about. It will have weird breaking point (point that suddenly becomes ambiguous).
I prefer my scheme that is more restrictive. For this case (simultaneously widening and narrowing), it will treat the two signatures as disjoint and have equal level of genericity.

I think you're right.

@pitrou: I think you get to be the reviewer for this PR.  Feel free to merge when it is ready.

OK, looks like the review is complete.  Going to merge this.


New event when all fields in form are registred

Using react redux router, 'push' event  to new location is async, so I need to know when form on next page is ready. Unfortunately HOC of redux-form is mounted before that all fields are ready. So form can get props before it ready. It may be @@redux-form/ALL_FIELDS_REGISTERD or @@redux-form/FORM_IS_READY

Your use case is too shallow. Please extend it to better why we should add something like that.
Also, for the next time, please follow the issue template. It was made for a reason.

Ok, sorry for missing template. I thought this is for issues, not for feture requests.
My use case is very simple. I need to know when redux-from done withdidMount. and is ready. Now I have to dispatch event "FORM_IS_LOADED" from my parent component. Other actions (which changes form values) wait for that event and trigger after that. But I'd like to make my HOC more clean without logic in 'componentDidMount', so I think it will be nice to know an exact momemt when redux-form is ready.

@alex-shamshurin to get more traction on this please provide:

current behaviour (either how the library behaves, or what you're having to do to get it to do what you want) - you've somewhat stated this but some sample code would help.
expected behaviour (how you would actually like it to behave, or a sample implementation after th change).
why this is important
any ideas for implementation if you have them :).

I think all we need is @@redux-form/FORM_IS_READY event

Because this issue saw almost no activity for a few months, I'm closing it.


wr.redshift.copy() keeps files in s3 when exception is raised

Describe the bug
A clear and concise description of what the bug is.
When an error is thrown in the wr.redshift.copy() method, s3 files are not cleaned up and remain there.
To Reproduce
Steps to reproduce the behavior. Also add details about Python and Wrangler's version and how the library was installed.
I encountered this when I didn't specify a varchar length for the dataframe and the column exceeded the default for varchar. My expectation is that this method would respect the param for keep_files to be false, and would dispose of any files when in success or failure.
wr.redshift.copy(
    df=df,
    path=path,
    con=con_redshift,
    schema="public",
    table="sample_data_table",
    mode="overwrite",
#     varchar_lengths={'data': 6500}
)

---------------------------------------------------------------------------
ProgrammingError                          Traceback (most recent call last)
<ipython-input-39-489fc07fb9d7> in <module>
      2
      3
----> 4 wr.redshift.copy(
      5     df=df,
      6     path=path,

~/miniconda3/envs/leadgen/lib/python3.8/site-packages/awswrangler/redshift.py in copy(df, path, con, table, schema, iam_role, aws_access_key_id, aws_secret_access_key, aws_session_token, index, dtype, mode, diststyle, distkey, sortstyle, sortkey, primary_keys, varchar_lengths_default, varchar_lengths, keep_files, use_threads, boto3_session, s3_additional_kwargs, max_rows_by_file)
   1345         max_rows_by_file=max_rows_by_file,
   1346     )
-> 1347     copy_from_files(
   1348         path=path,
   1349         con=con,

~/miniconda3/envs/leadgen/lib/python3.8/site-packages/awswrangler/redshift.py in copy_from_files(path, con, table, schema, iam_role, aws_access_key_id, aws_secret_access_key, aws_session_token, parquet_infer_sampling, mode, diststyle, distkey, sortstyle, sortkey, primary_keys, varchar_lengths_default, varchar_lengths, path_suffix, path_ignore_suffix, use_threads, boto3_session, s3_additional_kwargs)
   1170                 s3_additional_kwargs=s3_additional_kwargs,
   1171             )
-> 1172             _copy(
   1173                 cursor=cursor,
   1174                 path=path,

~/miniconda3/envs/leadgen/lib/python3.8/site-packages/awswrangler/redshift.py in _copy(cursor, path, table, iam_role, aws_access_key_id, aws_secret_access_key, aws_session_token, boto3_session, schema)
    112     sql: str = f"COPY {table_name} FROM '{path}'{auth_str}\nFORMAT AS PARQUET"
    113     _logger.debug("copy query:\n%s", sql)
--> 114     cursor.execute(sql)
    115
    116

~/miniconda3/envs/leadgen/lib/python3.8/site-packages/redshift_connector/cursor.py in execute(self, operation, args, stream, merge_socket_read)
    160                 self._c.execute(self, "begin transaction", None)
    161             self._c.merge_socket_read = merge_socket_read
--> 162             self._c.execute(self, operation, args)
    163         except AttributeError as e:
    164             raise e

~/miniconda3/envs/leadgen/lib/python3.8/site-packages/redshift_connector/core.py in execute(self, cursor, operation, vals)
   1095             self.handle_messages_merge_socket_read(cursor)
   1096         else:
-> 1097             self.handle_messages(cursor)
   1098
   1099     def _send_message(self: "Connection", code: bytes, data: bytes) -> None:

~/miniconda3/envs/leadgen/lib/python3.8/site-packages/redshift_connector/core.py in handle_messages(self, cursor)
   1170
   1171         if self.error is not None:
-> 1172             raise self.error
   1173
   1174     def handle_messages_merge_socket_read(self: "Connection", cursor: Cursor):

ProgrammingError: {'S': 'ERROR', 'C': 'XX000', 'M': 'Spectrum Scan Error', 'D': '\n  -----------------------------------------------\n  error:  Spectrum Scan Error\n  code:      15001\n  context:   The length of the data column data is longer than the length defined in the table. Table: 256, Data: 2737, File name: https://s3.us-west-2.amazonaws.com/{**}/3a13c036d1ac_0.snappy.parquet?&X-Amz-Al\n  query:     69281625\n  location:  dory_util.cpp:945\n  process:   fetchtask_thread [pid=1668]\n  -----------------------------------------------\n', 'F': '/home/ec2-user/padb/src/sys/xen_execute.cpp', 'L': '8948', 'R': 'pg_throw'}

Outside of that, this library is so helpful! Thanks for the work on this.

@njdanielsen Thank you for reporting it!
Released on version 2.3.0 🚀

@njdanielsen Thank you for reporting it!
Released on version 2.3.0 🚀

@igorborgest solid work on this library.

@igorborgest solid work on this library.


[T4A2][T09-B2] Goh Quan Yang Daryl

Accepting this PR without a review because another of your PRs
for this tutorial was reviewed in detail.