create AliasSegment

Ref #4437

Pull Request Test Coverage Report for Build 1858

0 of 27 (0.0%)  changed or added relevant lines in 6 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.05%) to 58.169%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

shardingsphere-sql-parser/shardingsphere-sql-parser-engine/src/main/java/org/apache/shardingsphere/sql/parser/sql/segment/generic/TableSegment.java
0
2
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-mysql/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/MySQLDMLVisitor.java
0
5
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-oracle/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/OracleDMLVisitor.java
0
5
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-postgresql/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/PostgreSQLDMLVisitor.java
0
5
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-sql92/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/SQL92DMLVisitor.java
0
5
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-sqlserver/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/SQLServerDMLVisitor.java
0
5
0.0%

Files with Coverage Reduction
New Missed Lines
%

shardingsphere-sql-parser/shardingsphere-sql-parser-engine/src/main/java/org/apache/shardingsphere/sql/parser/sql/segment/generic/TableSegment.java
1
20.0%

Totals

Change from base Build 917:
-0.05%

Covered Lines:
10610

Relevant Lines:
18240

💛  - Coveralls


IVY v9.0.0-next.11 fail

yeah, it's because of the old Renderer removal, seems the library here is still using that.
Time to upgrade to Renderer2 I would say ;)

Fixed in version 5.3.0


Add env variable to disable express logging

Hello,
I've been needing exactly your software but with a slight change so I figured I might as well upload it, if someone needs it.
I just added an environment variable to disable express logging if not needed. I set it to true in the dockerfile to let the default behaviour unchanged.
Thanks,

I've redone the feature with a different environment variable, going in the opposite direction, I didn't want any extra variables required in the Dockerfile.
Also I've just removed the dashes completely, I don't think it served a good purpose.
I'll decline this PR, this is now implemented here:  https://github.com/mendhak/docker-http-https-echo/pull/35

OK it's now in tag :23 and works by setting the environment variable, DISABLE_REQUEST_LOGS=true
AAAH I forgot your table!  OK I'll add that another time it's a nice useful summary


Fix 2797 on Kubernetes - Cluster goes down because all IPs become unreachable

PR to fix #2797. Includes changes from #3022.

Not just a location change for the API, but also a functional change. Will need to refactor code to use new API methods.

Any thoughts on when this will be merged into a release? This is an extremely critical piece of functionality when running K8S on a dynamic infrastructure, weave becomes pretty much unusable when an error (#2797) occurs.

@skny5 right now we're trying to clear up all the points and give it another thorough review; hopefully days rather than weeks.
As you may understand it is a rather destructive operation if we get it wrong.

@skny5 Working on it as we speak. Hoping this will pass review and hit mainline this week.

Some notes from my testing:
I fired up a 3-node Kubernetes cluster using the test 840 script.
Checked logs from all three weave containers.
Checked weave status ipam
Deleted a node with kubectl delete node brya-2
Initially the weave network still had three peers, but then the eviction manager kicked in on brya-2 and deleted all pods, killing all containers including Weave.
Next I restarted one of the remaining weave-net pods, so it would notice the node had gone; this all appeared to go to plan:
INFO: 2017/11/08 12:14:09.239399 Added myself to peer list &{[{76:a1:dd:6c:12:51 brya-0} {0e:e4:4e:dd:92:ca brya-2} {ba:23:5a:0c:f3:10 brya-1}]}
DEBU: 2017/11/08 12:14:09.239596 Nodes that have disappeared: map[brya-2:{0e:e4:4e:dd:92:ca brya-2}]
DEBU: 2017/11/08 12:14:09.239643 Preparing to remove disappeared peer {0e:e4:4e:dd:92:ca brya-2}
DEBU: 2017/11/08 12:14:09.239665 Noting I plan to remove  0e:e4:4e:dd:92:ca
DEBU: 2017/11/08 12:14:09.245074 Nodes that have disappeared: map[brya-2:{0e:e4:4e:dd:92:ca brya-2}]
DEBU: 2017/11/08 12:14:09.245149 Preparing to remove disappeared peer {0e:e4:4e:dd:92:ca brya-2}
DEBU: 2017/11/08 12:14:09.245171 Existing annotation 76:a1:dd:6c:12:51
DEBU: 2017/11/08 12:14:09.245191 weave DELETE to http://<IP_ADDRESS>:6784/peer/0e:e4:4e:dd:92:ca with map[]
INFO: 2017/11/08 12:14:09.252523 rmpeer of 0e:e4:4e:dd:92:ca : 393216 IPs taken over from 0e:e4:4e:dd:92:ca

I attempted to rejoin brya-2 using kubeadm reset then kubeadm join [...]
This resulted in the peer picking up the previous /var/lib/weave/weave-netdata.db so now it cannot join the cluster because it has an incompatible update.  This was previously noted
I manually deleted the persistence file then went through kubectl delete node kubeadm reset kubeadm join again.  Weave Net fired up ok.
slightly puzzling ipam state, presumably because it reclaimed the bridge address and nothing further:
# weave status ipam
76:a1:dd:6c:12:51(brya-0)               917503 IPs (87.5% of total) (1 active)
0e:e4:4e:dd:92:ca(brya-2)                    1 IPs (00.0% of total)
ba:23:5a:0c:f3:10(brya-1)               131072 IPs (12.5% of total)

After moving one of the nettest pods onto brya-2 it is more reasonable:
# weave status ipam
76:a1:dd:6c:12:51(brya-0)               655359 IPs (62.5% of total) (1 active)
0e:e4:4e:dd:92:ca(brya-2)               262145 IPs (25.0% of total)
ba:23:5a:0c:f3:10(brya-1)               131072 IPs (12.5% of total)

Next I am going to abruptly shut down brya-1 and see what happens.

I attempted to rejoin brya-2 using kubeadm reset then kubeadm join [...]
This resulted in the peer picking up the previous /var/lib/weave/weave-netdata.db so now it cannot join the cluster because it has an incompatible update. This was previously noted
I manually deleted the persistence file then went through kubectl delete node kubeadm reset kubeadm join again. Weave Net fired up ok.

@bboreham I haven't encountered this. Running kubeadm reset then kubeadm join ... leaves me with a (eventually) clean IPAM table across the cluster. I'm not doing kubectl delete though.
I am getting the following from somewhere when listening to the k8s_weave_weave-net-... container:
INFO: 2017/11/08 16:13:55.261671 Removed unreachable peer e6:78:31:8a:e6:e1(vagr-1)

Although apparently not from the weave-net codebase? (ran a search for "Removed" and "unreachable", and git couldn't find it!)
And I feel that at this point (or after some retries) that we should be removing the IP allocations for that peer?
This is the log I get when removing a peer with kubeadm reset
INFO: 2017/11/08 16:26:14.364090 ->[<IP_ADDRESS>:56777|e6:78:31:8a:e6:e1(vagr-1)]: connection shutting down due to error: read tcp4 <IP_ADDRESS>:6783-><IP_ADDRESS>:56777: read: connection reset by peer
INFO: 2017/11/08 16:26:14.364386 ->[<IP_ADDRESS>:56777|e6:78:31:8a:e6:e1(vagr-1)]: connection deleted
INFO: 2017/11/08 16:26:14.365670 Removed unreachable peer e6:78:31:8a:e6:e1(vagr-1)

But sudo weave status ipam still shows up unreachable addresses.
9a:33:6c:88:6b:5f(vagr-0)               393216 IPs (37.5% of total) (1 active)
2e:eb:78:2c:d6:d8(vagr-2)               131072 IPs (12.5% of total)
e6:78:31:8a:e6:e1(vagr-1)               524288 IPs (50.0% of total) - unreachable!

Pretty sure I'm running your latest version. (I get the [kube-peers] tagged logs), but I never get the Preparing to remove disappeared peer message, and the 860 test is (correctly) failing.

kubeadm reset removes the k8s installation and stopps running processes, but doesn't delete the node from the api-server. So this PR doesn't reclaim those IPs.

Ok, I'll amend to fix.

I think this is in fairly good shape; I am minded to merge it and raise new issues for follow-up.
One area we need to cover off is what happens when you upgrade from a previous release - the current code will delete persisted data which is ok if you do it one node at a time but not idea.
Also we need to update the cloud.weave.works config generator.


(chore) Changed DefaultMode of Job Deletion as Always

Signed-off-by: Rahul M Chheda<EMAIL_ADDRESS>
What this PR does / why we need it:

Ensures the default jobCleanupPolicy is set to "delete"

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Checklist:

[ ] Fixes #
[ ] Labelled this PR & related issue with documentation tag
[ ] PR messages has document related information
[ ] Labelled this PR & related issue with breaking-changes tag
[ ] PR messages has breaking changes related information
[ ] Labelled this PR & related issue with requires-upgrade tag
[ ] PR messages has upgrade related information
[ ] Commit has unit tests
[ ] Commit has integration tests

Can we add ENV here @ispeakc0de https://github.com/litmuschaos/chaos-runner/blob/9599175389f733bb7d35b22e8c021658f0c08a9c/pkg/utils/experimentHelpers.go#L40 ?

@rahulchheda , do you think this change is better done at the operator side?

@rahulchheda , do you think this change is better done at the operator side?

I think if a runner is spawning an experiment pod then better to delete it by runner

@rahulchheda , do you think this change is better done at the operator side?

I think if a runner is spawning an experiment pod then better to delete it by runner

@ksatchit was talking about setting the defaults in chaos engine , like the jobCleanUpPolicy, runner images, and other defaults from operator, and then keep the functionality of the runner as it is.

@rahulchheda , do you think this change is better done at the operator side?

I think if a runner is spawning an experiment pod then better to delete it by runner

The comment is not related to who is deleting, the runner is the one deleting the experiment pod right from the beginning. It was an alternate possibility about the place where we set defaults to items read from spec. There is  a set of engine attributes whose defaults are being placed in the operator - annotationCheck for instance. Likewise for app info. On the same lines, in principle, should the jobCleanupPolicy be initialized there and passed down to runner was the question. However, since the job lifecycle management is essentially a runner activity, it would still be OK to do it here. IMO, the engineSpec has to be enhanced to separate out which set of items in the engine spec are to be interpreted by the operator v/s runner. This can be ironed out in subsequent releases.


Update M04-Unit 6 Create a Traffic Manager profile using the Azure po…

…rtal.md
Module: 00
Lab/Demo: 00
Fixes # .
Changes proposed in this pull request:

Submitted by viggithub on behalf of MOC Labs Review


The "here" doesn't link you the the page to copy your ps vita's id

Are you able to help in anyway? Thank you for your time!!

It does.


Green light randomly off

A lot of people, including me, are having this strange issue where the hotspot does not turn on the green light outside at all, keeps randomly blinking the blue led inside and nothing happens. Bluetooth doesn't work, hotspot diagnostics do not work also because it shows offline, I have tried it on multiple routers and internets, nothing works, I have 1 miner with this issue.
So, I woke up to my router being offline but with the green light on, when I tried to pair it with the bluetoooh it wasn’t connecting, and the green light wasn’t blinking, so I deduced to turn the helium off by disconnecting it, after I connected it again the green light never turn on again.
I’m getting short of options to troubleshoot, any ideas or solutions?.

Contact<EMAIL_ADDRESS>


Updates to kubernetes v1.23 and minikube to v1.25

Changes

Updates minikube k8s to v1.23.4 (latest available version)
Updates kind k8s to v1.23.3 (latest available version of kindest/node image)
Updates recommended minimum version of minikube to v1.25

Release Note

Updates Kubernetes to v1.23 and recommended Minikube to v1.25

/lgtm


System message for payment is different in LHN vs report

LHN incorrectly indicates that the report was paid by contributors, though the actual and correct payment history is featured in the report.

Filing as an internal bug as part of wave3.
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01fa04cf9614f08985
Upwork Job ID:<PHONE_NUMBER>614326272
Last Price Increase: 2023-08-08

Ah so if this is an internal bug - we just need @fedirjh to review your (eventual) PR, @mountiny?

Yeah. Also @luacmartins @mountiny feel free to put an internal label on this if you'd like. Given the payment flow I expected backend sleuthing to be required.

Bump for either @luacmartins or @mountiny to acknowledge whether they are taking charge of this one? 🙏

It seems that @mountiny is already assigned to this, so I'll let him handle it.

Hello 👋
this is coming from here https://github.com/Expensify/App/blob/61ab4bcf100500ce49bf1e2dfaa128f9511ea941/src/libs/ReportUtils.js#L1239 in case of expense reports, we take the policy name.
This is kind of a side effect of having multiple admins now as that was not possible before. To confirm then, the message should say the display name/ email of the admin who actually paid the request? From the payee perspective/ employee its not really relevant and its the company who paid it

To confirm then, the message should say the display name/ email of the admin who actually paid the request? From the payee perspective/ employee its not really relevant and its the company who paid it

We actually want the message to come from the display name/email of the person who paid, yes. That's the intended design because it keeps a very specific audit trail on the request. Like yes the money is coming from the company bank account, but I think that's separate from this consideration.

Going to take this one as part of wave3.

Will look into this this week after completing some other priorities

I actually can't reproduce this one anymore, and it's not appearing on reports that were recently reimbursed. I think we should actually close this one, though I'll re-open if I see again in the next pay run.


Keep Identity doesn't work for InsertOrUpdate

For BulkInsertAndUpdate operation, when KeepIdentity of SqlBulkCopyOptions is set, generated merge statement doesn't include identity columns for case WHEN NOT MATCHED BY TARGET THEN INSERT
I'd expect that merge statement will cover identity columns when KeepIdentity option is selected.
Is there a way to workaround it?

I have implemented a fix for this.
Try NuGet 2.2.7


modified styling on both Permission Groups and Offices

Office, Permission Group styling
Modified the styling for offices and permission groups to better compliment each other

Coverage remained the same at 93.045% when pulling d82ff1f0f15471600e8ba9d20304273709c7fd4e on perm-office-style into 99b8ee389f15b1fcb5b6048ea192a3e66f017599 on dev.


Add help for common IQ# installation issue

This PR adds some troubleshooting steps for a common IQ# installation issue (see https://github.com/microsoft/iqsharp/issues/14).

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/install-guide/pyinstall.md
95

0
link
:white_check_mark:

articles/install-guide/qjupyter.md
88

0
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit 26367a8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/install-guide/pyinstall.md
:white_check_mark:Succeeded
View

articles/install-guide/qjupyter.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/install-guide/pyinstall.md
95

0
link
:white_check_mark:

articles/install-guide/qjupyter.md
88

0
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit cd1a49a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/install-guide/pyinstall.md
:white_check_mark:Succeeded
View

articles/install-guide/qjupyter.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/install-guide/pyinstall.md
95

0
link
:white_check_mark:

articles/install-guide/qjupyter.md
88

0
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/install-guide/pyinstall.md
95

0
link
:white_check_mark:

articles/install-guide/qjupyter.md
88

0
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit d0ef6ec:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/install-guide/pyinstall.md
:white_check_mark:Succeeded
View

articles/install-guide/qjupyter.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 2b30c39:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/install-guide/pyinstall.md
:white_check_mark:Succeeded
View

articles/install-guide/qjupyter.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Honestly, this could even be a pretty reasonable microsoft/jupyter-core feature; perhaps a new --find-path option that tries to guess the path from System.Reflection.Assembly.GetExecutingAssembly().Location?

@cgranade agreed, that would be nice. Opened https://github.com/microsoft/jupyter-core/issues/56.

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/install-guide/pyinstall.md
95

0
link
:white_check_mark:

articles/install-guide/qjupyter.md
88

0
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit 4c20e8f:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

filemap.json
:x:Error

Details

:x:Error

Details

filemap.json

[Error-PublishToDHSFailed] Publishing of document 'filemap.json' to document hosting service failed with asset id: filemap.json locale: en-us, depot name: MSDN.quamtum-api, branch name: pr-en-us-900. Error: A task was canceled.

[Error-BackendEngineExceptionError] An exception occurred during the build process. Please check the raw log and retry building your content again. If the issue persists, please open a ticket in https://SiteHelp and include the URL of publishing report as well as below information: One or more errors occurred..

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit bfe5986:
:clock10: Pending: waiting for processors (6 builds ahead of you)
:warning: Docs Build is busy, currently there are 6 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Acrolinx Scorecards
A minimum Acrolinx score of 80 is required.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology.

Article
Acrolinxscore
Lowestcategoryscore
Spellingissues
Scorecard
Processed

articles/install-guide/pyinstall.md
95

0
link
:white_check_mark:

articles/install-guide/qjupyter.md
88

0
link
:white_check_mark:

More information about Acrolinx

Install Acrolinx locally for VSCode
False positives or issues
Troubleshooting issues with Acrolinx

Docs Build status updates of commit bfe5986:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/install-guide/pyinstall.md
:white_check_mark:Succeeded
View

articles/install-guide/qjupyter.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Input box does not slide down (transition) unless the user clicks on the phone label

Expected behavior: when the user clicks anywhere in the vicinity of the form input, the input should slide down to accommodate the input of text into the form.
Actual behavior:  User input overlaps the form label.  If the user clicks on the input itself, the input border doesn't slide down, but still allows the user to type words.

Whoops. Also, this will always happen if the input is focused before the page finishes loading.

Fixed. Sometimes if you call magicForm.init() twice it will open and then immediately close the input on focus.


Issue connecting to proposed channel partner ZAP Technologies

Description
Hello,
this is not really a Zap-Android bug, but I thought that you might be in touch with the operator of the "Zap Technologies" lnd node.
When I first installed the app I opened a channel to Zap Technologies which was proposed when opening a new channel. The channel opened without a problem(some month ago). Since a couple of days/weeks(can't say for sure) the channel state switched to "offline" (in the app) or when entering lncli listchannels it has the state "active": false,
I am running lnd version 0.12.1-beta
When manually connecting to the node via:
lncli connect 03634bda49c9c42afd876d8288802942c49e58fbec3844ff54b46143bfcb6cdfaf@<IP_ADDRESS>:9735
output of ournalctl -f -u lnd (removed the timestamps)

[INF] SRVR: Established connection to: 03634bda49c9c42afd876d8288802942c49e58fbec3844ff54b46143bfcb6cdfaf@<IP_ADDRESS>:9735
[INF] SRVR: Finalizing connection to 03634bda49c9c42afd876d8288802942c49e58fbec3844ff54b46143bfcb6cdfaf@<IP_ADDRESS>:9735, inbound=false
[INF] PEER: NodeKey(03634bda49c9c42afd876d8288802942c49e58fbec3844ff54b46143bfcb6cdfaf) loading ChannelPoint(ChannelPoint123456789:1)
[INF] HSWC: Removing channel link with ChannelID(ChannelID123456789)
[INF] HSWC: ChannelLink(658114:1158:1): starting
[INF] HSWC: Trimming open circuits for chan_id=658114:1158:1, start_htlc_id=0
[INF] HSWC: Adding live link chan_id=ChannelID123456789, short_chan_id=658114:1158:1
[INF] HSWC: ChannelLink(658114:1158:1): HTLC manager started, bandwidth=12345 mSAT
[INF] HSWC: ChannelLink(658114:1158:1): attempting to re-resynchronize
[INF] CNCT: Attempting to update ContractSignals for ChannelPoint(ChannelPoint123456789:1)
[INF] PEER: Negotiated chan series queries with 03634bda49c9c42afd876d8288802942c49e58fbec3844ff54b46143bfcb6cdfaf
[INF] DISC: Creating new GossipSyncer for peer=03634bda49c9c42afd876d8288802942c49e58fbec3844ff54b46143bfcb6cdfaf
[INF] DISC: GossipSyncer(03634bda49c9c42afd876d8288802942c49e58fbec3844ff54b46143bfcb6cdfaf): requesting new chans from height=0 and 672483 blocks after
[INF] PEER: unable to read message from 03634bda49c9c42afd876d8288802942c49e58fbec3844ff54b46143bfcb6cdfaf@<IP_ADDRESS>:9735: EOF
[INF] PEER: disconnecting 03634bda49c9c42afd876d8288802942c49e58fbec3844ff54b46143bfcb6cdfaf@<IP_ADDRESS>:9735, reason: read handler closed
[INF] DISC: Removing GossipSyncer for peer=03634bda49c9c42afd876d8288802942c49e58fbec3844ff54b46143bfcb6cdfaf
[INF] HSWC: Removing channel link with ChannelID(ChannelID123456789)
[INF] HSWC: ChannelLink(658114:1158:1): stopping
[WRN] HSWC: ChannelLink(658114:1158:1): error when syncing channel states: link shutting down
[INF] HSWC: ChannelLink(658114:1158:1): exited

So I searched a little bit and found this issue(the logs look similar but it is a different node) in the lnd repo which suggest that there maybe a problem when peers are running outdated lnd versions
https://github.com/lightningnetwork/lnd/issues/4941
Do you know which lnd version the Zap Technologies node is running or could you ask the operator if you know him?
Cheers!

@mrfelton
Do you know what is going on or which version the ZAP Technologies node runs?

I have removed the suggested channel partner list from the open channel view, as it is no longer maintained.


Message "*** NOTE: unrepairable repeated axis coords" gives wrong subscript

Reported by ansley b manke on 20 Aug 2013 00:08 UTC
A time axis has two coordinates which are identical.  Ferret tells us,
yes? use latestOb.nc
 *** NOTE: unrepairable repeated axis coords on axis trdim at subscript 615
 *** NOTE: A dummy axis of subscripts will be used

It should say subscript 2.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2088

Comment by ansley.b.manke on 20 Aug 2013 21:10 UTC
This is fixed in cd_get_1_axis.F


[Android Studio] android_instrumentation_test test button not working on Kotlin test targets

Background
IDE Test Run configuration button doesn't work if android instrumentation test is written in kotlin.

The root cause is that KotlinTestContextProvider only handles kotlin tests, not android tests, and AndroidTestContextProvider only handles java android tests, but not kotlin tests. We need a solution to handle both of them at the same time.
Proposed Fix
My proposed fix is https://github.com/bazelbuild/intellij/pull/2319. After the fix, I verified the test button works for  android_instrumentation_test

Hello @ThomasCJY, Could you please reply on this comment we can move this issue and PR in to further good state. Thanks!

Duplicate #2613


Group source files by role in generated Xcode projects

Currently, the Xcode generation adds every source file, including headers and the build scripts, in an ungrouped list as children of the Xcode project.
Instead, I think it would be clearer to group the files by their role within the project - eg Source, Headers, Build, etc.

This is also useful when creating new files from Xcode, as (it looks like) a directory can be associated with a group and new files default to being created in this directory. Right now, creating a new source file default to adding the new file to the project directory, where Gradle won't see it.

I went with a strictly grouping strategy as opposed to trying any automatic grouping based on the directory. A user can add files directly to the different group, chose the on-disk location, and append them to the indexing target.


TagGroup: add maxRows prop to limit initially shown tags, and include 'Show all' button

Closes https://github.com/adobe/react-spectrum/issues/3660
Will need to test keyboard focus behavior after https://github.com/adobe/react-spectrum/pull/3798 gets merged.
✅ Pull Request Checklist:

[ ] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:
Check story at TagGroup -> maxRows: 2. Resize container and confirm behavior.
🧢 Your Project:
RSP

reidbot: https://reactspectrum.blob.core.windows.net/reactspectrum/f6204de4ba86c7132fcce0a400166cea81c6e244/storybook/index.html?path=/story/taggroup--maxrows-2

reidbot: https://reactspectrum.blob.core.windows.net/reactspectrum/740a6b0cb2d9538cb3fd07c771b1ca1bf64b777d/storybook/index.html?path=/story/taggroup--max-rows

@ktabors Good catch with that first edge case, I just fixed this. With the second, I think that's desired behavior. I don't think we ever want to hide all tags and just show the button, and if someone is limiting the height to one row, I don't think they will also be restricting horizontal space.

Reidbot: https://reactspectrum.blob.core.windows.net/reactspectrum/aa360a67f99409bc5ed32ef9ec9cacad2a4424a6/storybook/index.html?path=/story/taggroup--max-rows&providerSwitcher-express=false&providerSwitcher-toastPosition=bottom

I think the button might need some custom styles to shrink its height since we don't have small size action buttons yet.

Reidbot: https://reactspectrum.blob.core.windows.net/reactspectrum/d3a0f99d0112068bc50a42079996b67eaf4b9b9a/storybook/index.html?path=/story/taggroup--max-rows&providerSwitcher-express=false&providerSwitcher-toastPosition=bottom

Seems like when you make the width bigger, it doesn't display more tags now?

@devongovett If you refresh, it behaves correctly. Looking into why it's different on first load....

Hmm I tried refreshing and it still happened for me. Actually when I make the element smaller, it also wraps to show more than 2 lines:

@devongovett Yeah, looks like the new inline styles I introduced broke the overflow logic. Working on it now.

Should be fixed now. One issue was with font loading, and I needed to set the resize observer to watch the new container. Also went ahead and added in translation strings for the button.


VCF soft filters

Pjotr;
I'm enjoying using bio-vcf and the syntax for filtering based on samples is incredibly useful for more complex filters. Is it possible to do soft-filtering with bio-vcf by adding tags to the FILTER column instead of excluding variants? I'd like to plug bio-vcf in for some somatic filters where I need to select based on tumor FORMAT values, but want to soft filter the variants rather than remove them. Thanks again for all your work on this.

bio-vcf has the --rewrite switch with which you can inject data. I used this to inject sample names:
for x in *.Somatic.vcf ; do ~/izip/git/opensource/ruby/bioruby-vcf/bin/bio-vcf --rewrite "rec.info['sample']='$x'[7..13]" < $x > rewrite/$x ; done

in the rewrite you can also add a conditional. If you send me an example I'll try and make it work.

Pjotr;
Thanks much. I'm looking for the equivalent of applying a soft filter to a VCF file like in bcftools:
bcftools filter -m + -s LowDepth -e `INFO/DP < 10`

http://samtools.github.io/bcftools/bcftools.html#filter
This adds a ##FILTER line to the header and updates the FILTER column when variants fail, instead of removing them. This is the standard way we do filtering to avoid throwing away data since folks can re-filter later if needed. Thanks again.

Hi Brad,
Soft-filter support is added with --add-filter in the master for --filter switch. Please test the latest.
https://github.com/pjotrp/bioruby-vcf/commit/37511c21c710871fd2f5f717b6b8dd47b627b827
I'll do the sample filters after.
Pj.

Pjotr;
Thanks much for this. I spotted one small issue in not over-writing the FILTER field if it was previously set as PASS, but otherwise looks great. Support for sample field selections would be great and let us use this for complex tumor/normal where bcftools currently doesn't not have enough expressibility. Thanks again.

The sample filters also support soft filters now.


Regenerated storage, compute, authorization, features, network, resou…

…rces, and subscriptions specs

Hi @stankovski, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (destanko). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


Gatsby/feature/list of categories

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e3f0402). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #131   +/-   ##
=========================================
  Coverage          ?   24.56%
=========================================
  Files             ?       29
  Lines             ?      232
  Branches          ?       84
=========================================
  Hits              ?       57
  Misses            ?      175
  Partials          ?        0

Continue to review full report at Codecov.

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


Update Gradle Wrapper to 8.11-RC1

Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test RfR without PTS

I've triggered the following builds:

Ready for Release (Trigger) (Check)

Build Scan

The merge queue build has failed. Click here to see all failures.

I've triggered the following builds:

Pull Request Feedback (Trigger) (Check)

Build Scan

@bot-gradle test this

I've triggered the following builds:

Pull Request Feedback (Trigger) (Check)

Build Scan

The following builds have failed:

Pull Request Feedback (Trigger) (Check)

Build Scan

The following builds have been cancelled:

Pull Request Feedback (Trigger) (Check)

Build Scan

@bot-gradle test this

@bot-gradle test this

I've triggered the following builds:

Pull Request Feedback (Trigger) (Check)

Build Scan

The following builds have been cancelled:

Pull Request Feedback (Trigger) (Check)

Build Scan

The following builds have failed:

Pull Request Feedback (Trigger) (Check)

Build Scan

@bot-gradle test RfR

I've triggered the following builds:

Ready for Release (Trigger) (Check)

Build Scan

The following builds have passed:

Ready for Release (Trigger) (Check)

Build Scan

@ljacomet RfR with the changes in this PR is green ✅


Redoing Termite Support Pull Request

Doing this again as I merged to master beforehand, I didn't know that was a mistake.

Added termite template
Automatically append user's additional config to it so he can ln -s straight to the cached file
If the maintainer is fine with it, I would also like to add $XDG_CONFIG_HOME to pywal's config to make it more flexible than hardcoding "~/.config", but I did not do so for now in case this is not wanted.

This would be a cool feature. Is there something holding back the merge?

pywal already works in termite and this also requires the user to have a special config that pywal modifies. pywal doesn't touch the user's files whatsoever.

@dylanaraps understood. Are there any special instructions for Termite? Doesn't seem to work for me out of the box (v3.3.0), can't find anything in the docs and most of my findings on forums and reddit involve generating a custom config with a bash script.

Command:
wal -i img.jpg --vte

Inside your .shellrc (.bashrc, .zshrc etc):
(cat ~/.cache/wal/sequences &)

I understand your concerns. Trouble is, that only works when launching programs that source .bashrc first. Running htop from the .desktop file does not work for example. I tried to compromise insofar as files are modified if and only if you have a file named termite-pywal.conf in your .config/termite. This is needed to house non-theme related configs, as termite does not support the standard .conf file include statement.
Again, I understand if this runs in opposition to the project philosophy. I can see both sides of the argument, which is in part why I am currently writing a daemon that would address these kinds of issues universally (for example dealing with multiple yaml configs which also do not support include statements by default.

And, to clarify, the code doesn't write anywhere outside of .cache/wal, it only reads from a different .config folder. You still need to make the choice to symlink your termite config into .cache/wal/colors-termite.conf.

You can handle the modification of your own files by specifying a script (or scripts) to be executed after pywal has completed. This gives you full control over your files and you can very easily implement th behavior inside this PR.
You can also use a user template file which could be your entire termite config with markers for the color variables. You can then symlink this file from ~/.cache/wal to ~/.config/termite. User templates are identical to the builtin pywal templates and they're read from ~/.config/pywal/templates.
Using either of these methods you can accomplish what this pull request implements.

Brilliant! I wasn't aware of the template feature, I guess I'll do that. I'll close the PR then. Thoughts on using $XDG_CONFIG_HOME, though?

Done: https://github.com/dylanaraps/pywal/commit/668f9bbd5bffc6b7d65254da9a6bf6581a6726e9


Enable GIF file read

i have noticed that gif file format already supported in vips library and has a vips foreign loader but it is not enabled because it is not defined as an image type i defined it and it working now :)

Coverage decreased (-0.7%) to 98.148% when pulling<PHONE_NUMBER>e9fbf2530879cb8cd0eae6e385a860 on digital-flowers:patch-3 into f8eab4996235844e8e767f229275abefa9c0707f on lovell:master.


Current instructions to execute the basic demo not working.

npm run demo  is not available, only dev & test.
And at the moment the library is not properly mapped in the client code, so the full path is required:
import { SocioClient } from './node_modules/socio/dist/core-client.js'
in place of './core-client.js'

Actually, the entire code for the basic demo seems to be not aligned with the current code in the repository. sc.query for example is used in place of sc.Query.

Thanks 👍. I didnt realize that the basic demo was that far behind on Socio versions 😅. Fixed this in commit 78e288efd7bcbde7d71490d7824ab9f1159c1fa1 and the demo was updated to Socio v0.7.12. Should be all good now, but i had to do some funky stuff for base64.js to work 🤔


super.method() in extended class in loose mode generates code with infinite recursive loop

Describe the bug
The playground output, when executed, logs "hola" until the call stack size is exceeded. This does not happen if you turn loose mode off. The bug seems to happen with many browser targets, so it's hard to pin down exactly which transform is problematic here.
Input code
class Superclass {
  doStuff() {
  }
}

abstract class Subclass extends Superclass {
  doStuff() {
    console.log('hola');
    return super.doStuff();
  }
}

new Subclass().doStuff()

Config
{
  "jsc": {
    "parser": {
      "syntax": "typescript",
      "tsx": true,
      "importMeta": true
    },
    "transform": {
      "react": {
        "refresh": true
      }
    },
    "target": "es2022",
    "loose": true,
    "minify": {
      "compress": false,
      "mangle": false
    }
  },
  "module": {
    "type": "es6"
  },
  "minify": false,
  "isModule": true,
  "env": {
    "targets": ""
  }
}

Playground link
https://play.swc.rs/?version=1.3.2&code=H4sIAAAAAAAAA4WNMQ6AMAwD97wiG%2B3SD%2FAMXlDaAEPVoCYVSIi%2FI0DAyGbLPhtC8iLY1ZnKLTdAjNxpHQZjL7fDDuB70eKD4gP0t6BVKcefBcTAWTiRSzyaZuLkG9teQSGtJaOcuHup9rnNtLxfxn6FAzJnxKK4AAAA&config=H4sIAAAAAAAAA0WOOw7DMAiG78KcocrQIXfIIZCLU1d%2BCUjVKPLdazuuusH%2F%2BOCElxhYTsjIQtwmOaLiBxbQI5MYdllhApUqKe80gQs5sa6keCmluoxRbOLQAExo9BoskzxHqsWQN6oWkMy3ea5Yn5LQDxxcdPZoTZNCrlWBxaKXZmHcPI21oUJ67E04%2B5sdeYfyZ4yek3UErxMU373TH6l4gFK%2B2VlBQgQBAAA%3D
Expected behavior
I would expect the output code to log hola once and then exit. super.doStuff() should not cause recursion.
Actual behavior
super.doStuff() seems to call itself.
Version
1.3.2
Additional context
No response

So the output code for Subclass.doStuff() is this:
var Subclass = (function (Superclass) {
  // ...
  _proto.doStuff = function doStuff() {
    console.log("hola")
    return _get(Subclass.prototype, "doStuff", this).call(this)
  }
  // ...
})(Superclass)

When I log the result of _get(Subclass.prototype, "doStuff", this), I can see in the logs that it is Subclass.doStuff instead of Superclass.doStuff.

So this might be as simple as changing the logic in _get a bit to be better at finding the correct superclass.

I can confirm that this happens whether I make _get use Reflect.get or the handwritten _get method, both versions cause the same problem:
function _get(target, property, receiver) {
  if (typeof Reflect !== "undefined" && Reflect.get) {
    _get = Reflect.get
  } else {
    _get = function _get(target, property, receiver) {
      var base = _superPropBase(target, property)
      if (!base) return
      var desc = Object.getOwnPropertyDescriptor(base, property)
      if (desc.get) {
        return desc.get.call(receiver)
      }
      return desc.value
    }
  }
  return _get(target, property, receiver || target)
}


Cosine Similarities are not proper

Hello,
I was trying to measure the cosine similarity between vectors with dimensions (1, 128). Here is the query.
 {   "query": {     "function_score": {       "boost_mode": "replace",       "min_score": 0,       "script_score": {         "script": {           "source": "binary_vector_score",           "lang": "painless",           "params": {             "cosine": true,             "field": "embedding_vector",             "vector": [                                 <VECTOR>              ]           }         }       }     }   },   "size": 170 }
But results are meaningless. Eventhough two vectors is not similar, it's score is over 0.9.
Similarity scores decrease gradually.

I am also having the same problem. Scores does not reflect actual cosine similarities.

did you resolve the problem?

did you resolve the problem?

unfortunately.

Hi guys,
I'm sorry for the late response.
Cosine scoring works fine for us.
I added a test for the scores. feel free to change it, test it & submit it to me if it fails.
@osmantamer Note that the query you pasted is wrong. it should state "lang": "knn"
from the readme:
{
  "query": {
    "function_score": {
      "boost_mode": "replace",
      "script_score": {
        "script": {
          "inline": "binary_vector_score",
          "lang": "knn",
          "params": {
            "cosine": false,
            "field": "embedding_vector",
            "vector": [
               -0.09217305481433868, 0.010635560378432274, -0.02878434956073761, 0.06988169997930527, 0.1273992955684662, -0.023723633959889412, 0.05490724742412567, -0.12124507874250412, -0.023694118484854698, 0.014595639891922474, 0.1471538096666336, 0.044936809688806534, -0.02795785665512085, -0.05665992572903633, -0.2441125512123108, 0.2755320072174072, 0.11451690644025803, 0.20242854952812195, -0.1387604922056198, 0.05219579488039017, 0.1145530641078949, 0.09967200458049774, 0.2161576747894287, 0.06157230958342552, 0.10350126028060913, 0.20387393236160278, 0.1367097795009613, 0.02070528082549572, 0.19238869845867157, 0.059613026678562164, 0.014012521132826805, 0.16701748967170715, 0.04985826835036278, -0.10990987718105316, -0.12032567709684372, -0.1450948715209961, 0.13585780560970306, 0.037511035799980164, 0.04251480475068092, 0.10693439096212387, -0.08861573040485382, -0.07457160204648972, 0.0549330934882164, 0.19136285781860352, 0.03346432000398636, -0.03652812913060188, -0.1902569830417633, 0.03250952064990997, -0.3061246871948242, 0.05219300463795662, -0.07879918068647385, 0.1403723508119583, -0.08893408626317978, -0.24330253899097443, -0.07105310261249542, -0.18161986768245697, 0.15501035749912262, -0.216160386800766, -0.06377710402011871, -0.07671763002872467, 0.05360138416290283, -0.052845533937215805, -0.02905619889497757, 0.08279753476381302
             ]
          }
        }
      }
    }
  },
  "size": 100
}

I'm also having trouble with the scoring and scoring does not seem to reflect the distance.
Did any of you manage to solve it?
I'd love to contribute with a failing test case - how can I run the tests? :-)

I'm also having trouble with the scoring and scoring does not seem to reflect the distance.
Did any of you manage to solve it?

I realize that it was my mistake. In python, compare_faces function of the face_recognition library return the euclidean distance between two faces, (smaller value means similar faces), but in this plug-in it returns cosine similarity (higher value means similar faces). That was my mistake.

Glad to hear. Closing this


Scrolling on IOS doesn't work with regions plugin

Bug description
When you have wavesurfer with region from the begin to the end and try scroll element on IOS (Iphone), it works. but when you scroll again it stop working. moreover all "onClick" events stop working on page.
Environment

Browser: Chrome

Minimal code snippet
Expected result
Obtained result
Screenshots

I'm not sure if it's exactly the same thing but I just ran into something very similar.
On mobile Safari if I have a zoomed waveform and have set dragToSeek: true then things act weird. If you just tap on the waveform it will set the cursor position. If you drag the waveform itself it will set the cursor position and drag the waveform, which I think is reasonable. But if you drag the cursor itself it gets into a state where the only thing you can do on the whole is either drag the cursor or the waveform. Regular taps on the waveform stop working, and regular taps on any other element on the page doesn't work. Like the events are just swallowed somewhere.
If I set dragToSeek: false then I can't get it to go into that funky state.

I've also been able to get it to happen when the waveform is not zoomed, but it's much harder to trigger in that case and I don't know exactly what sequence of events triggers the problem. When the waveform is zoomed and dragToSeek is true it's very easy to trigger.

Thanks!
I’ll look into using the native drag events instead of pointer events. Hopefully that should fix a lot of weird behaviors on mobile.


Duplicate

Problem
Sphinx 4.0.0 changed the way the documentation_options script is inserted, such that we’re now inserting an extra copy of it. This results in invalid HTML (duplicate id).
Reproducible Project
This is reproducible with a totally empty project:
$ pip install Sphinx sphinx_rtd_theme
Collecting Sphinx
[…]
Successfully installed Jinja2-3.0.1 MarkupSafe-2.0.1 Pygments-2.9.0 Sphinx-4.0.2 alabaster-0.7.12 babel-2.9.1 certifi-2021.5.30 chardet-4.0.0 docutils-0.17.1 idna-2.10 imagesize-1.2.0 packaging-20.9 pyparsing-2.4.7 pytz-2021.1 requests-2.25.1 snowballstemmer-2.1.0 sphinx-rtd-theme-0.5.2 sphinxcontrib-applehelp-1.0.2 sphinxcontrib-devhelp-1.0.2 sphinxcontrib-htmlhelp-2.0.0 sphinxcontrib-jsmath-1.0.1 sphinxcontrib-qthelp-1.0.3 sphinxcontrib-serializinghtml-1.1.5 urllib3-1.26.5
$ mkdir docs
$ touch docs/index.rst
$ sphinx-build -C -D html_theme=sphinx_rtd_theme docs build
Running Sphinx v4.0.2
[…]
The HTML pages are in build.

Error Logs/Results
$ grep documentation_options build/index.html
      <script type="text/javascript" id="documentation_options" data-url_root="./" src="_static/documentation_options.js"></script>
        <script data-url_root="./" id="documentation_options" src="_static/documentation_options.js"></script>

Expected Results
Only one copy of <script id="documentation_options"> should be present.
Environment Info

Python Version: 3.8.9
Sphinx Version: 4.0.2
RTD Theme Version: 0.5.2

This will be fixed with https://github.com/readthedocs/sphinx_rtd_theme/pull/1123

This has been fixed with 380647d1596cec8bda15dc374aba270f884e6de8


Fix image name, build multi-arc, fix permission

Signed-off-by: Daishan Peng<EMAIL_ADDRESS>

Merging because golangci-lint is broken


Unable to run playback example code on Ubuntu 18.04 LTS.

Hi,
I'm currenlty running some sample codes of azure kinect on Ubuntu 18.04 LTS.
These are the conditions of the environment that I'm working on:
ubuntu 18.04 LTS
Azure Kinect Sensor SDK v1.2.0
Azure Kinect Body Tracking SDK v0.9.4
OpenCV 4.1.0
cmake 3.15.4
ide: qt creator 5.12.5
While running playback example, there is some issue about
case k4a_image_format_t::K4A_IMAGE_FORMAT_COLOR_MJPG:{
                // NOTE: this is slower than other formats.
                std::vector<uint8_t> buffer( src.get_buffer(), src.get_buffer() + src.get_size() );
                mat = cv::imdecode( buffer, cv::IMREAD_ANYCOLOR );
                cv::cvtColor( mat, mat, cv::COLOR_BGR2BGRA );
                break;
            }

in util.h file.
Especially, mat = cv::imdecode( buffer, cv::IMREAD_ANYCOLOR );  is not working (decoding problem).
However, I tried this code on window 10 using visual studio 2019 with Azure Kinect Sensor SDK v1.2.0,  and it worked.
It also worked with Azure Kinect Sensor SDK v1.3.0 on ubuntu 18.04 LTS.
Still, I face the same issue only on ubuntn 18.04 LTS with Azure Kinect Sensor SDK v1.2.0.
Is there any way to solve this issue?
Thanks.

Hmmm. Sorry, I don't know why occur that problem.
Please update Azure Kinect Sensor SDK to v1.3.0 if that solves the problem.


Fixed typo in repo README

chef generate repo skel typo

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@atward

Please sign the CLA here.

This applies under the "obvious fix" rule. A CLA is not required.
:+1:

:+1:


Failure with non-article documentclass

I use the exam documentclass which defines certain commands (e.g. \lhead). Xenops replaces this with the article document class and then fails when it encounters the \lhead command, which appears only in the preamble. Ideally, xenops would work with document classes that define commands that might appear in the preamble, for instance by using the document class of the document.

Have a look at the issue here which describes how you can set the org-format-latex-header variable to help make xenops work with other document classes

Thanks for pointing this out. It does provide a workaround, of sorts, but it's not entirely satisfactory. Changing org-format-latex-header in init.el doesn't really work because I have different documents that use different classes. It does work when setting it as a file local variable (with some edits), but it's pretty awkward to do this for the various different files. Ideally it should just work when specifying a different document class, but it looks like development/maintenance of the package has ended, unfortunately.

no worries, I usually set the org-format-latex-header in my .dir-locals.el file for each project and has worked really well for me. Once I have time I would like to help contribute to this package to make things a little more automatic where possible to help out dandavison :)

but it looks like development/maintenance of the package has ended, unfortunately.

Hi all, I am still reading these at least. But it's true that I'm not currently doing anything where I could use Xenops and I don't currently have time for Xenops development. If anyone would like to try to contribute that would be great; it is quite fun to work on.

Thanks for the great package. I really enjoy using it. Time constraint issues are totally understandable. Hopefully others will be able to step in and contribute to keeping the package going and even improve the rough edges.


Solid Q1 Monster Merge FOAM

Includes:

Refactoring of TreeGraph to use the new tree graph algorithm and the creation of TreeNode rather than making it an inner class
Rounding of corners on the card border
localToGlobalCoordinates method on CView to enable proper clicking for the Tree
Double parser update to deal with nulls.
make csvparser not return null when it can't parse.
A RequestResponseCache axiom and RequestResponseDAO which can be integrated with EasyDAO

@mcarcaso @nanoMichal @kennyqiyenkan @adamvy @kgrgreer

What is "Liquid"? ;)
Might want to keep these code names a little more secret.

What is "Liquid"? ;)
Might want to keep these code names a little more secret.

I just git inverted the name


Typo "Azure storage emulator"→"Azure Storage Emulator"

https://docs.microsoft.com/en-us/azure/azure-monitor/platform/diagnostics-extension-versions

@hyoshioka0128 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Actually lower case is the standard for this name. Example here https://docs.microsoft.com/en-us/azure/storage/common/storage-use-emulator
#please-close


STATIC - confused about instructions - where to learn more

I am finding the ExpressWorks instructions incomprehensible.  Perhaps I am missing some prerequisites.  I’ve written out my questions below to show the level of ignorance I am coming to this learning session with.  If you have suggestions about where to go to learn what I need before I start, please let me know.
For example, Exercise 2 of 8 (static) says:
Please don't use ANY routes like app.get. ONLY static.
I wonder:

What is a route?
How can I learn about app?
Static?  Static as opposed to dynamic.  What does that mean?

Your solution must listen on the port number supplied by process.argv[2].
I wonder:
4. What listens?
5. Is this like the Hello World one where I used this:
var app = express();
...
app.listen(process.argv[2]);
This is how you can call static middleware:
I wonder:
6. What is static middleware?
7. Where can I learn about it?
For this exercise expressworks will pass you the path:
app.use(express.static(process.argv[3] || path.join(__dirname, 'public')))

I wonder:
8. Is app var app = express();?
9. If so, what is express?
10.  Where did the path variable come from?
This what I think is supposed to happen:  process.argv[3] points to an HTML file.  I am supposed get this HTML file and send it back in a response.  I’m supposed to wait for a request from process.argv[2], but I don’t know if I’m supposed to parse that request in any way.

The best place to start is with the command: expressworks help
This will give you a link to the API documentation and another to a book written by azat-co.
API docs: http://expressjs.com/api.html
Express.js Guide: http://expressjsguide.com
The API docs should answer your questions:

routes: http://expressjs.com/guide/routing.html
app: http://expressjs.com/4x/api.html#app
static: http://expressjs.com/starter/static-files.html
etc.

Hope this helps.

Thank you very much the links to expressjs.com is what I was looking for.  It will be a while before I can get back to it, but I'll post how it went when I do.  Keep it open for now.

Evan Hahn's blog post:  http://evanhahn.com/understanding-express/ helped me out.

@TylerMoeller thanks ;)


⚠ Bofrost.fr has degraded performance

In 3ed59db, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1236 ms

Resolved: Bofrost.fr performance has improved in 11eaaba.


CLI: cloud platform use / rm race condition

Running cloud platform use and cloud platform rm parallel can leave another master in local masters (kontena master ls)
$ kontena cloud platform use org/plat1 &
$ kontena cloud platform use org/plat2

same with
$ kontena cloud platform rm --force org/plat1 &
$ kontena cloud platform rm --force org/plat2

(discovered with terraform)

This is true for any config changes and for example refresh tokens, making you lose access to the master.
#2659


Update mobile-developer-setup.rst

Fixed typo.

Thanks @jfperusse-bhvr for the pull request!
Per the CONTRIBUTING.md file displayed when you created this pull request, we need to add you to the list of approved contributors for the Mattermost project.
Please help complete the Mattermost contribution license agreement?
This is a standard procedure for many open source projects. Your form should be processed within 24 hours and reviewers for your pull request will be able to proceed.
Please let us know if you have any questions.
We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

Thanks for the PR @jfperusse-bhvr!
Once you've signed the CLA, we'll go ahead and merge 👍

Hi @jfperusse-bhvr,
Wondering whether you've had a chance to sign the contribution license agreement so we can go ahead and merge your PR?

Thank you for the PR but as we haven't heard from you regarding signing the CLA so we can accept your PR, I'll close this off for now.


Crash: Memory error

The setup is the same as in my previous post: https://github.com/mitmproxy/mitmproxy/issues/3515
The source of the problem should be the same - resource leakage.

-:49420: Traceback (most recent call last):
  File "/opt/mitmproxy/mitmproxy/proxy/server.py", line 121, in handle
    root_layer()
  File "/opt/mitmproxy/mitmproxy/proxy/modes/transparent_proxy.py", line 19, in __call__
    layer()
  File "/opt/mitmproxy/mitmproxy/proxy/protocol/tls.py", line 286, in __call__
    layer()
  File "/opt/mitmproxy/mitmproxy/proxy/protocol/http1.py", line 83, in __call__
    layer()
  File "/opt/mitmproxy/mitmproxy/proxy/protocol/http.py", line 188, in __call__
    if not self._process_flow(flow):
  File "/opt/mitmproxy/mitmproxy/proxy/protocol/http.py", line 392, in _process_flow
    self.read_response_body(f.request, f.response)
MemoryError

Traceback (most recent call last):
  File "/opt/mitmproxy/mitmproxy/proxy/server.py", line 121, in handle
    root_layer()
  File "/opt/mitmproxy/mitmproxy/proxy/modes/transparent_proxy.py", line 19, in __call__
    layer()
  File "/opt/mitmproxy/mitmproxy/proxy/protocol/tls.py", line 286, in __call__
    layer()
  File "/opt/mitmproxy/mitmproxy/proxy/protocol/http1.py", line 83, in __call__
    layer()
  File "/opt/mitmproxy/mitmproxy/proxy/protocol/http.py", line 188, in __call__
    if not self._process_flow(flow):
  File "/opt/mitmproxy/mitmproxy/proxy/protocol/http.py", line 392, in _process_flow
    self.read_response_body(f.request, f.response)
MemoryError

mitmproxy has crashed!
Please lodge a bug report at: https://github.com/mitmproxy/mitmproxy
-:49420: clientdisconnect

If this is still an issue with the latest release, feel free to open a new issue.


Add view password functionality to login form

Resolves #4671
Description
This change implements an "eyeball" icon functionality on the login page, allowing users to toggle password visibility. This feature aims to improve user experience by enabling users to confirm their password entry, reducing login errors due to typos.
This solution balances usability with security, ensuring the password is hidden by default.
No additional dependencies are required for this change.
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?
I implemented unit tests to verify the password toggle functionality:

Confirmed that clicking the "eyeball" icon switches the password field from masked to visible.
Verified the icon changes correctly between eye and eye-slash states.

Functionality looks fine

Also, erb lint is failing - see above.

Awesome, thanks!


Way to require the user to provide a type?

One of the flow-typed maintainers here...
I'm looking for a way to type a function such that the return value must be typed explicitly by the user for it to be correctly typed checked by Flow. Something like this:
type TypeMe = $RequireType

const works: TypeMe<{ a: string }> = getJSON('{"a":"hello"}')

// $ExpectError
const shouldNotWork = getJSON('{"a":"hello"}')

The use case is valuable for libdefs, such as axios, where the the return type from the server can either be effectively mixed or any, both of which have cons.
I've tried doing something like parameterized types (Flow try) but it didn't work the way I hoped.

to be correctly compiled by Flow

Flow does not compile anything...

Fixed the question.

can you use Phantom Types for this https://medium.com/@gcanti/phantom-types-with-flow-828aff73232b?

it would be something like this https://flow.org/try/#0JYOwLgpgTgZghgYwgAgJIgCYQB5wEYA2EAPACoB8yA3gFDLIDaA1hAJ4BcyAJANJsDOZcgF1OXAGpwCAVwiCKNAL40aCAnH79kAEThg4Q5MAC2AByLGI4Leiy5CJCtTrcWHZHBCsA3C64A3KVlOTx8lFQQAexB+MGQAcwgwACkAZQB5ADlkAF5kIQAKbBCvAEpOXX1DHMpsCOjY5AB3SKgmfgq9AyoPTlioUHjkRUo8xJSMzIKAcioAIjg59jmACwgCAki5xWnSmgB6faiYyKIAOk34gpa2-jO4PeP+U4gLyKub9oZpuGnhPdUDTi-BWkWkBAwmUiYAA6q0mLkEkk0lkZvNFss1hstjtHg0Xm8riCwRCobD4fc9kA
this does not work well because this is a problem with flow https://github.com/facebook/flow/issues/1323

I think you can do that on Flow 78
check this
https://flow.org/try/#0C4TwDgpgBAKgPAQQDQGoBCA+KBeKBvAQwC4pkoUAjEtAXwG4AoBgMwFcA7AY2AEsB7dlGaIkmABTFSSKFShoAlCXjJM+BlCgAnCMFabBhaRRoMTDCAA8wfTcCgATCMwKsANneZiADNK-y6UAD0gVAAzgAWfG72UABWrKF2FNAAtjyhoTzsAOZCNlJQ7Hx2aEA

I'm not quite following what this is doing... also I'm noticing that the error only occurs when it's exported?

Wanted to ping on this again. We've been having several conversations in the flow-typed repo where libdefs could be made more powerful if there was a way to require explicit typing by the user (e.g. flow try and issue).

Closing this since Flow now will complain under inference_mode=lti when some type arguments are underconstrained.


Adds basic session implementation

Will (eventually) close #123.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

21 of 52 (40.38%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.1%) to 79.24%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Http/Session/Session.php
16
28
57.14%

src/Http/Session/ArraySessionHandler.php
5
24
20.83%

Totals

Change from base Build<PHONE_NUMBER>:
-1.1%

Covered Lines:
1584

Relevant Lines:
1999

💛  - Coveralls

Just a few thoughts as I work on a basic implementation here:

We should determine what our approach is to attempting to get or set a value when the session isn't started. Do we attempt to start it or fail fast (throw an exception)?
We should figure out more of the API we want to support. Do we want to add support for dot notation, etc.?

We should determine what our approach is to attempting to get or set a value when the session isn't started. Do we attempt to start it or fail fast (throw an exception)?

Definitely start it 👍

We should figure out more of the API we want to support. Do we want to add support for dot notation, etc.?

Could be useful, there's btw already a class ArrayHelper that can unwrap dot notation, I don't think it can handle to other way around yet. I needed it for request validation.

No don't add PSR-20 🫣

@aidan-casey I say we merge this, since it's blocking multiple issues:

#113
#14

We can improve on it later 👍


Parameter is not described

-AllowFederatedParticipantJoinAsSameEnterprise parameter is not described.

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

ID: 2eb630b9-b94d-a647-ca21-821e1d520b21
Version Independent ID: ede8297e-aa49-fc5e-23bb-2d678bc23e6b
Content: Set-CsConferencingPolicy (skype)
Content Source: skype/skype-ps/skype/Set-CsConferencingPolicy.md
Service: skypeforbusiness-powershell
Product: unspecified
GitHub Login: @kenwith
Microsoft Alias: kenwith

All Skype cmdlets are undergoing audit and being tracked internally to make sure every cmdlet has an example and descriptions.


Delegator unbond events with Id handled. Delegated Amount deduction m…

…oved to shareburn event and removed from unstake event

Looks good. But build failing.

Can you also push a tag on this branch ? It'll auto-release.


Project is missing a license

I noticed that this repository is missing a license. Having a license is essential to provide clarity on how others can use and distribute the code.
If there's no license, people can't legally use or share the code. This means they might get in trouble for copyright issues and have to pay money or go to court. Also, it's hard to know what's allowed and what's not when using or sharing the code, since there are no clear rules or guidelines.
Would it be possible to add a license to this repository, such as the MIT License or the GNU General Public License?

Closed in d113622f1.

Closed in d113622f1042fd35cdf1bc6c3dee7fbd0e4875aa.


camel case relation name not working as expected

Only the visibility change on edit/create works. Rules are ignored and the field 'education_id' is never shown on details page.
Relation:
public function userGroup()
{
    return $this->belongsTo(UserGroup::class);
}

And the field:
BelongsTo::make('UserGroup'),

NovaDependencyContainer::make([
    Number::make('Fortbildungsnummer', 'education_id')
        ->rules('required')
])->dependsOn('usergroup.education_id_needed', 1)

@pschocke How does the table structure for UserGroup look like?
Is education_id_needed suppose to be a boolean and only those with a value of true should be displayed with an extra field Fortbildungsnummer?

Exactly this should happen.
This is the migration for UserGroup
Schema::create('user_groups', function (Blueprint $table) {
    $table->bigIncrements('id');
    $table->string('title');
    $table->text('description');
    $table->boolean('education_id_needed')->default(false);
    $table->timestamps();
});


Add target for Wio LTE.

Thanks for the contribution! There are just a few small things that need to be addressed before we can accept the PR

Fixed. Please review.

Thanks, looks good. I just pushed a commit that fixed a few tabs that should be spaces.


fix(extension): disable cmd click for expand button

Checklist

[x] JIRA - LW-10092
[ ] Proper tests implemented
[ ] Screenshots added.

Proposed solution
Expand button is developed as a tag that has a default in browser behaviour to open a new tab on cmd + click.
Changing it to span solves an issue.
Testing
Follow steps described in ticket.
Screenshots
Attach screenshots here if implementation involves some UI changes

Hi @vetalcore check the SonarCloud report please ☝


curl: (35) BoringSSL SSL_connect: SSL_ERROR_SYSCALL

I am seeing this a lot... having no trouble when using vanilla curl... seems the problem is if TLSv1.2 is not supported by the server, only TLSv1.3, but curl-impersonate uses 1.2...

Can you provide some urls? Vanilla curl will never report this kind of error, because it's not built with BoringSSL.


Add support for PBKDF2 for enc command

[x] documentation is updated
[x] CLA is signed

iteration count used only for PBKDF2 as per Matt request ( PR #1860 )

@mattcaswell I'm wondering whether we should also add a "-legacy" option as well. You then must either provide "-legacy" or "-iter" otherwise you get an error.

No, I rather think that the legacy behaviour should remain the default.  Documentation can suggest better options. A warning could be printed to stderr unless an explicit choice is made.

I agree with @vdukhovni (and -legacy is a bad name; what happens when we need "less old legacy" :)

-young_leg ? (younger legacy) :-)
OK, I will submit changes soon (hope next few days). Thanks for your feedback and comments.

@mattcaswell Can we not generate the IV in the PKCS5_PBKDF2_HMAC call above? i.e. instead of generating data that is "key length" long, we generate data that is "key length + iv length" long.
Hope my new commit complies with your request.

I think that when we're doing PBKDF2 we should consider a new output format:

New salt magic number
KDF and cipher parameters saved along with the salt (digest oid, iteration count and cipher id)

That way, if defaults are changed in the future, it will still be possible to correctly decrypt existing data.
Note that iteration counts need to be rather high these days to provide meaningful key stretching, and the typical "1000" used in days gone by is rather inadequate now (just 10 addtional bits).

Any objection to discuss about new magic number and new output format through the dev list ?

Any objection to discuss about new magic number and new output format through the dev list ?

No - that would be a good place for it.

I believe all your comments have been taken into account now.
Compiled and tested under Windows 10.
I will not be offended if someone propose a better formulation for the warning message or the man page update. :-)
I will try to start the thread on the dev list about the new 'magic' output format before sunday.

I believe all your comments have been taken into account now.

If you've answered my latest comments, then maybe you should remember to push ;-)

As regards the output format. One possibility is to have a new magic number and follow that with a password based encryption AlgorithmIdentifier instead of making up new formats for everything. This format is already supported by OpenSSL (PKCS#8 and PKCS#12 uses it) and adds a degree of future compatibility. By that I mean it will decrypt formats which we can't currently generate using "enc" (e.g. scrypt as the algorithm).

new magic number

"Saline__"?

Did this grind to a halt?

oops, looks like I did some mistakes 15 days ago...

Squashed and pushed!!


Update alfred to 3.3_799

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you, but that’s a prerelese version.

Not according to https://www.alfredapp.com/changelog/, unless they changed their naming scheme. But it is not linked from the homepage just yet, which I omitted to check. ;)

But it is not linked from the homepage just yet, which I omitted to check. ;)

I also checked there. And inside Alfred itself, if you change to the stable channel, you’ll see 3.2.1.

I believe you 100% and closing was justified, I was sloppy.

No need to apologise. Thank you for the continued quality contributions.

Changelog was updated today, with 801 now. Seems like the changelog also gets updated with prerelease versions.

It does. From when I started checking (after the last release) up to 799 they added 'Prerelease' after the version, which made me not check, which I should have...


如果您输入多个标签，您想要一个功能，只要存在其中一个标签就可以搜索到。(Translator)

需求 / Requirement
如果您输入多个标签，您想要一个功能，只要存在其中一个标签就可以搜索到。
例如，如果您输入 male:masturbation 标签和 female:masturbation 标签，您想搜索存在其中一个的画廊。
原因 / Reason
我比较喜欢使用这种APP的快速搜索功能。 但是，如果没有这种形态的多重搜索功能，快速搜索的列表就会增加很多。
建议实现 / Suggested implemention
输入A标签和B标签，我想看所有带有A标签的画廊和带有B标签的画廊。
与e-hentai中"~"的标签搜索功能一致。
备注 / Addition details
No response
自查步骤 / Verify steps

[X] 您已仔细查看并知情 常见问题 中的内容 / You have checked Q&A carefully
[X] 您已搜索过 Issue Tracker，没有找到类似内容 / I have searched on Issue Tracker, No duplicate or related open issue has been found
[X] 您确保这个 Issue 只提及一个问题。如果您有多个功能请求，烦请发起多个 Issue / Ensure there is only one feature request in this issue. Please make mutiply issue for mutiply bugs
[X] 您确保已使用最新版本测试，并且该问题在最新版本中仍未实现 / This feature have not supported in latest version

Why not just input ~male:masturbation ~female:masturbation

If I type ~female:masturbation and press the offer below, ~ disappears. If I just type in Input and don't press Offer, the keyword gets very long. Can you make If I type ~ and press Offer, ~ is added together?

The feature will be added to main search bar:

But for dialog i won't support it because this dialog will be removed later, users can't add tags in this dialog any more.

Do you mean that the function such as group:semimogura that is registered below will be removed, and the format of entering the search word in the keyword column will be basic?
In the form of writing a normal search query in e-hentai?
Please let me know if what I understand is wrong.

I understand, thank you for your kind reply.
Thank you for making a good app. I hope you continue to do well in the future, and have a great day!


Strip binary, update to go-1.17.6

With this PR the binary is compiled with go 1.17.6 to close some of the open CVE´s in go 1.17.1.
Furthermore the resulting binary is stripped which reduces the size of the binary by ~10MB.
before:
-rwxr-xr-x    1 root     root      32766008 Jan 13 16:07 /bin/headscale

after:
-rwxr-xr-x    1 root     root      22325232 Jan 14 08:14 /bin/headscale

[x] read the CONTRIBUTING guidelines
[ ] raised a GitHub issue or discussed it on the projects chat beforehand
[ ] added unit tests
[ ] added integration tests
[ ] updated documentation if needed
[ ] updated CHANGELOG.md

The ci jobs also needs to be upgraded and go.mod, would you mind doing that aswell?
Thanks

The ci jobs also needs to be upgraded and go.mod, would you mind doing that aswell?
Thanks

gimmi a second

Why are all integration tests failing ? @kradalby  Have you any clue

This is great! Thanks :+1:

Why got the build workflow changed to 1.17, this will destroy reproducible builds. Rather set it explicitly to 1.17.6

@juanfont @kradalby What is your opinion on that?

Yeah I will do a PR so set that specifically again.
I was also thinking about something like renovate or dependency bot, as we would also use it for the golang packages and the docker containers. I will also take a look at that and try to test it out in a fork.


Registry tests: review resource naming in tests

https://github.com/apigee/registry/pull/656#discussion_r918068634
What internal style or naming conventions might we want in the resource names used in our tests?

Our tests currently use "my-project" as a default project name. This is used here and in other places to translate resource names for testing in hosted projects.


add missing "settings" object to eslint example configuration

It would be great to merge this.  Following the current README.md produces an error.


Fix #1172, Update to use Ut_Stub_CheckDefaultReturnValue API

Describe the contribution
Fix #1172 - updates API name for nasa/osal#737 fix
Testing performed
Build and execute unit tests
Expected behavior changes
None, name change (as long as corresponding nasa/osal#823 in)
System(s) tested on

Hardware: cFS Dev Server
OS: Ubuntu 18.04
Versions: cFS Bundle main + nasa/osal#823 + this commit

Additional context
Depends on osal PR
Third party code
None
Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

CCB 2021-02-24 APPROVED


App Design

UML diagram.

I shall use the MVC architecture (in the classic design-pattern sense)[1]:

[1] MVC Architecture, Chrome Developers


feat!:  Add infer_objects argument to the to_table() function in the DH pandas module

This issue was auto-generated
PR: https://github.com/deephaven/deephaven-core/pull/6024
Author: jmao-denver
Original PR Body
Fixes #6019
BREAKING CHANGE: the default behavior is changed to converting the object-type columns in the given data frame first before creating a DH table from it. While this change would result in much more sensible column types in DH, e.g. String vs. PyObject for object type column with only strings, it however will break user code written to handle the PyObject type columns in the result table.

This issue is closed by https://github.com/deephaven/deephaven.io/pull/4382.


fixed: documentation: corrected spelling errors

corrected a series of spelling errors in the readme file and some of the other task files

Resolve conflicts


Dev Test: Android App

Version 1.1-dev.4 of the Android App is available for testing.  It is available here: https://play.google.com/apps/internaltest/4697762600441754675.  Please me send the email address for any Play Store user that wishes to test the app.
This version of the app should have the new updated layout to match the iOS app v 1.3 and should show titles, descriptions, and simulations in the locale of the device. (App strings have not been added yet).  There are other changes but they are not of a visible nature (using compressed sims, database optimizations).
QA, please do a dev test of the app. Please complete a general test to make sure that all sims are launching, resources are in the correct locale, that the app receives updates correctly, and that all other functions of the app are working properly (sim info dialog, app info dialog, filters, search, etc).
It should be tested (at a minimum) on a phone and on our two Chromebooks (Lovelace and Euler I think?).  @oliver-phet - can you think of any other devices that should be tested?

Specific issues to confirm:
https://github.com/phetsims/phet-android-app/issues/56
https://github.com/phetsims/phet-android-app/issues/55

@ariel-phet for priority

@KatieWoe higher than #483 lower than #479

@KatieWoe - When opening device specific bugs, please note the Android version used on the device. This is normally found in Settings > About {{Device}} > Android Version.

Due to https://github.com/phetsims/phet-android-app/issues/96, putting an end to this test.

We'll create a new test when these bugs are solved.


Improve configuration cache invalidation tests

add a check prior to applying a change to verify that the cache entry is loaded
assert that the user is informed of why the cache entry is not reused

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.


Unable to arm, only disarm, Ness D8x via Home

Hello Greg,
Firstly, thanks for developing this plug in.
Several months back, I noticed that the Away command was not arming my Ness D8x - there is no audible beep and no change to Disarmed status on the Ness D8x Navigator.
I can successfully disarm the panel using the Home app (including after manually entering the arming code on my Ness D8x Navigator touchscreen).
Here is a video I captured interacting with Home app: https://share.icloud.com/photos/055QwlaLM14Qhv2cNU-5SKh_Q
From Verbose Log (accompanying video)
[26/02/2024, 12:28:46] [Ness D8x] request: AWAY_ARM current panel state matches - do nothing
[26/02/2024, 12:28:47] [Ness D8x] EventReceived: {"address":0,"timestamp":null,"requestId":0,"_includedZones":[]}
[26/02/2024, 12:28:47] [Ness D8x] Status Update: ZoneUpdate :{"address":0,"timestamp":null,"requestId":0,"_includedZones":[]}
[26/02/2024, 12:28:48] [Ness D8x] EventReceived: {"address":0,"timestamp":null,"requestId":20,"_status":[]}
[26/02/2024, 12:28:48] [Ness D8x] Status Update: ArmingUpdate :{"address":0,"timestamp":null,"requestId":20,"_status":[]}
[26/02/2024, 12:28:52] [Ness D8x] Request: DISARM Issue command: DISARM
[26/02/2024, 12:28:58] [Ness D8x] request: AWAY_ARM current panel state matches - do nothing
Using iPhone 15 Pro Max, iOS 17.3.1
Thanks,
Jack

Hi Jack,
The core issue is that the plugin believes that the panel state is AWAY_ARM and so when it gets the request AWAY_ARM it “does nothing” as per the log.
Probably need to gather some more log info, especially re Arming State changes.

A good place to start is a reboot/restart of the plugin/Homebridge.

On restart the plugin queries the panel for the current arming state and you will see something like this:

[3/25/2024, 11:49:24 PM] [Ness] Configured: Ness
[3/25/2024, 11:49:24 PM] Homebridge v1.7.0 (HAP v0.11.1) (Ness) is running on port 50218.
[3/25/2024, 11:49:24 PM] [Ness] Interface: Connected: host: <IP_ADDRESS> port: 2401
[3/25/2024, 11:49:24 PM] [Ness] EventReceived: {"address":0,"timestamp":null,"requestId":0,"_includedZones":[]}
[3/25/2024, 11:49:24 PM] [Ness] Status Update: ZoneUpdate :{"address":0,"timestamp":null,"requestId":0,"_includedZones":[]}
[3/25/2024, 11:49:24 PM] [Ness] Zone changed: zone: 1 state: false
[3/25/2024, 11:49:24 PM] [Ness] Zone changed: zone: 2 state: false
[3/25/2024, 11:49:24 PM] [Ness] Zone changed: zone: 3 state: false
[3/25/2024, 11:49:24 PM] [Ness] Zone changed: zone: 4 state: false
[3/25/2024, 11:49:24 PM] [Ness] Zone changed: zone: 5 state: false
[3/25/2024, 11:49:24 PM] [Ness] Zone changed: zone: 6 state: false
[3/25/2024, 11:49:24 PM] [Ness] Zone changed: zone: 7 state: false
[3/25/2024, 11:49:24 PM] [Ness] Zone changed: zone: 8 state: false
[3/25/2024, 11:49:24 PM] [Ness] Zone changed: zone: 9 state: false
[3/25/2024, 11:49:24 PM] [Ness] Zone changed: zone: 10 state: false
[3/25/2024, 11:49:26 PM] [Ness] EventReceived: {"address":0,"timestamp":null,"requestId":20,"_status":[]}
[3/25/2024, 11:49:26 PM] [Ness] Status Update: ArmingUpdate :{"address":0,"timestamp":null,"requestId":20,"_status":[]}
[3/25/2024, 11:49:26 PM] [Ness] Arming state changed: DISARMED
[3/25/2024, 11:49:26 PM] [Ness] armingStateChanged: DISARMED update current hap to: DISARMED and target hap to: DISARM
[3/25/2024, 11:49:33 PM] [Ness] EventReceived: {"address":0,"timestamp":null,"requestId":19,"_includedAlarms":[]}
[3/25/2024, 11:49:33 PM] [Ness] Status Update: MiscellaneousAlarmsUpdate :{"address":0,"timestamp":null,"requestId":19,"_includedAlarms":[]}
[3/25/2024, 11:49:33 PM] [Ness] Battery Status: Normal
[3/25/2024, 11:49:34 PM] [Ness] EventReceived: {"address":0,"timestamp":null,"requestId":23,"_model":0,"_majorVersion":7,"_minorVersion":8}
[3/25/2024, 11:49:34 PM] [Ness] Status Update: PanelVersionUpdate :{"address":0,"timestamp":null,"requestId":23,"_model":0,"_majorVersion":7,"_minorVersion":8}
[3/25/2024, 11:49:34 PM] [Ness] Panel details: Model: D16x Version: 7.8
[3/25/2024, 11:50:24 PM] [Ness] EventReceived: {"address":0,"timestamp":null,"requestId":0,"_includedZones":[]}
[3/25/2024, 11:50:24 PM] [Ness] Status Update: ZoneUpdate :{"address":0,"timestamp":null,"requestId":0,"_includedZones":[]}
[3/25/2024, 11:50:26 PM] [Ness] EventReceived: {"address":0,"timestamp":null,"requestId":20,"_status":[]}
[3/25/2024, 11:50:26 PM] [Ness] Status Update: ArmingUpdate :{"address":0,"timestamp":null,"requestId":20,"_status":[]}
[3/25/2024, 11:50:54 PM] [Ness] Interface: Last event received 28.02 secs ago
(I’ve deleted entries dealing with “Outputs” to reduce the log size)

Make sure that the “state changed:” matches the state of your panel.

Arm/disarm the panel from the panel and watch what the logs say and make sure that the Arming state tracks the panel.

It may be easier to turn off Verbose Log, there will still be more than enough logging to see what’s going on … eg

[3/25/2024, 11:57:02 PM] [Ness] Configured: Ness
[3/25/2024, 11:57:02 PM] Homebridge v1.7.0 (HAP v0.11.1) (Ness) is running on port 50218.
[3/25/2024, 11:57:02 PM] [Ness] Interface: Connected: host: <IP_ADDRESS> port: 2401
[3/25/2024, 11:57:04 PM] [Ness] Arming state changed: DISARMED
[3/25/2024, 11:57:12 PM] [Ness] Panel details: Model: D16x Version: 7.8
Send me the logs when you’ve had a look at them …
Regards,
Greg

On 26 Feb 2024, at 12:58, oxfordsmiths0a @.***> wrote:
Hello Greg,
Firstly, thanks for developing this plug in.
Several months back, I noticed that the Away command was not arming my Ness D8x - there is no audible beep and no change to Disarmed status on the Ness D8x Navigator.
I can successfully disarm the panel using the Home app (including after manually entering the arming code on my Ness D8x Navigator touchscreen).
Here is a video I captured interacting with Home app: https://share.icloud.com/photos/055QwlaLM14Qhv2cNU-5SKh_Q
From Verbose Log (accompanying video)
[26/02/2024, 12:28:46] [Ness D8x] request: AWAY_ARM current panel state matches - do nothing
[26/02/2024, 12:28:47] [Ness D8x] EventReceived: {"address":0,"timestamp":null,"requestId":0,"_includedZones":[]}
[26/02/2024, 12:28:47] [Ness D8x] Status Update: ZoneUpdate :{"address":0,"timestamp":null,"requestId":0,"_includedZones":[]}
[26/02/2024, 12:28:48] [Ness D8x] EventReceived: {"address":0,"timestamp":null,"requestId":20,"_status":[]}
[26/02/2024, 12:28:48] [Ness D8x] Status Update: ArmingUpdate :{"address":0,"timestamp":null,"requestId":20,"_status":[]}
[26/02/2024, 12:28:52] [Ness D8x] Request: DISARM Issue command: DISARM
[26/02/2024, 12:28:58] [Ness D8x] request: AWAY_ARM current panel state matches - do nothing
Using iPhone 15 Pro Max, iOS 17.3.1
Thanks,
Jack
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Hi Greg,
I restarted Homebridge, and it appears to be working again. I was surprised, as I frequently restart Homebridge as part of installing Pi updates and upgrades. I’ve closed the issue.
My only question is: when I arm the Ness using my phone, the Home app only ever says that the panel is “Arming…” (but when I click into the accessory, it is in Away mode. Is this normal behaviour?
Thanks again,
Jack


feat: Visualize complex Hive column types

This change makes it easier to understand complex Hive types, including array<>, map<>, struct<>, and uniontype<>.  It adds a parser that detects these types and converts them into a representative JSON object, which is then visualized using the react-json-view library added in #991.
Currently it only works on these Hive types.  If it's too specific/niche to merge that's fine, we'll maintain it internally on our side.  Just wanted to throw it out there in case it would be useful to anyone else.

Thanks for the feedback, I agree JSON isn't the most clear way to visualize some of these types.
We're using another product that uses a nested column approach for complex types, but it is also confusing when looking at map<> and array<> types.  Based on the feedback from our users, I think they prefer a confusing visualization over just the raw text.
The advantages were that it was relatively easy to implement and reused the same JSON viewer as the results, providing some consistency for users.  It also is collapsible which is nice for extremely nested types.
I can experiment with different approaches and share some additional options.
Re: the type length limit, I noticed this as well and was going to bring it up later.  The parser degrades as well as it can, but ideally we'd increase the max length.  We too have a significant number of columns synced with truncated types.


Update to FuelWings for support of multiple fuel mods

Update to FuelWings to depend on either Modular Fuel Tanks, Interstellar Fuel Switch or Configurable Containers

Thanks for letting us know! Found the author confirming this change:
https://forum.kerbalspaceprogram.com/index.php?/topic/87605-110xfuelwings-v4-fuel-customization-in-stock-b9-b9-proceduralwings-11082020/&do=findComment&comment=3939576

This looks more suited to an any_of clause than a new provides, though.

@HebaruSan Fixed it

Confirmed on SpaceDock that only the latest release is affected, so we don't have to worry about updates for old releases.
Thanks again! :+1:


Make API more inline with scikit-learn

We'd like all idioms used in KeplerMapper to fit seamless with the scikit-learn API.
What needs to be changed? Below are some things that have been brought up before and questions to ask about the current API.

[ ] nr_cubes should be changed to n_cubes
[ ] Need to implement fit method and have fit_transform only run fit and return the result.
[ ] How should map method fit into this design? Should map be fit instead?
[ ] Does the current graph format abide? Does scikit-learn have a graph/simplicial complex format? Should we adopt another library's (networkx or other) format?

Please suggest other changes to the API!

KeplerMapper's API has not evolved since its first iteration. Back then I only clustered on the projection and it was more of a single-function-does-everything. Right now, I dont even know if fit_transform is the right term to use (you are essentially projecting).
Seperating out fit, transform, and fit_transform would allow for mapping on unseen data (calculate in which interval the new data falls, and add to most relevant cluster).
For small changes like nr_cubes -> n_cubes, if not too much added complexity, id like for this to accept both (non-breaking changes for old code, and updated documentation).
Afaik, scikit-learn does not do graphs at all. The closest data structure is with http://scikit-learn.org/stable/modules/generated/sklearn.cluster.AgglomerativeClustering.html but this uses keyed Numpy arrays (probably forced, to be consistent with other structures/data representations/optimizations). I think either dicts or NetworkX objects are more flexible to experimentation (add link strengths, add link functions). With NetworkX I think it also becomes easier to run graph algorithms (shortest or longest paths) and to compare Gromov-Hausdorff distance between graphs.
As for the visualization side, I can accept both dicts and NetworkX objects. Numpy arrays would become a bit more difficult for me.

I dont even know if fit_transform is the right term to use (you are essentially projecting).

I'm still new to the scikit_learn API, but it seems fit/fit_transform is the main work horse of each algorithm. Because of this I think map should be renamed to fit.  We could roll current map functionality into fit_transform and then allow users to provide a custom lens or a projection:
graph = mapper.fit_transform(data, lens, clusterer, cover)

or
graph = mapper.fit_transform(data, projection, clusterer, cover)

For small changes like nr_cubes -> n_cubes, if not too much added complexity, id like for this to accept both (non-breaking changes for old code, and updated documentation).

Maintaining backwards compatibility is great, but we shouldn't try to keep it around forever.  Maybe we should start dev'ing on a new branch and release backwards breaking changes with 1.2 or 2.0?   It would be best to release all API changes at the same time, so letting them sit while we fine-tune them would be ideal.

I can accept both dicts and NetworkX objects. Numpy arrays would become a bit more difficult for me.

Let's stick with dicts and networkx objects then.  Is it currently wired to accept networkx objects? Adding networkx as a requirement seems okay to me.
One thing I'd like to work on soon is adding ability for a custom linker. I'd like to be able to build arbitrary dimensional simplicial complexes, and this could also provide an interface for specifying the type of output. I think this will also be an essential component for implementing multi-mapper (a custom cover scheme and associated linker to handle the filtration).

In my workflow I found it useful to call .fit_transform multiple times. I also like that behind the scenes it calls .fit_transform and that the output is a Numpy array. First impression is to not consolidate .map and .fit_transform. This seems more friendly to custom projections.
It should also be able to feed inverse_X=None and apply clustering on the projection (suffer projection loss), for instance in the case where it is possible to create a lens from the data (final layer -> t-SNE(2-d)), but not possible to cluster on the original data (clustering on raw pixel values fails to provide meaningful clusterings).
But I don't have a strong opinion about this and need to think about this more (and incorporate feedback from using kmapper on many varied data sets).

Let's stick with dicts and networkx objects then. Is it currently wired to accept networkx objects? Adding networkx as a requirement seems okay to me.

I'll write a networkX -> dict convertor which kicks in if you don't pass a dict, but an object. Lets keep NetworkX optional (required only if you set parameter to output a NetworkX object).

Regardless of consolidating projecting and mapping, it would be cool to be able to pipeline lenses.
projected_X = mapper.fit_transform(X, projection=[PCA(2), custom_projection, "knn_distance_5"])

That would be a nice feature, and shouldn't be very difficult to dev.
Is there any way we could incorporate chaining of function calls into the API? This would make the calls look something like:
lens = mapper(data).fit_transform(PCA(2))\
                   .fit_transform(custom_projection_function)\
                   .fit_transform("knn_distance_5")

This would be a bit more far-fetched and would require that lens be its own object. The more I think about it the less realistic it seems.
Either way, the API you showed should work without major changes.

I like how so much can now fit inside a single tweet:
import kmapper as km
from sklearn.datasets import make_circles as c
from sklearn import manifold

inverse_X, y = c(3001)
K = km.KeplerMapper()
projected_X = K.fit_transform(inverse_X, projection=manifold.TSNE(3))
G = K.map(projected_X, inverse_X)
K.visualize(G, custom_tooltips=y)

I liked being able to call .fit_transform to create custom color_functions. I also liked chaining them, or for creating distance matrices.
If we see .fit_transform close to scikit-learn API (model fit to data and transform function) we can build out a .transform for unseen data. Then you know which hypercube the data fell into, and can try to assign it to the hypercube clusters (1-nearest neighbors on cluster means?), then you can use the previously recorded statistical data from those clusters (how many defaulters ended up in that cluster?) as input for a ML algo like random forests (just have to store the cluster means, and the projection transformers).
I see no objection in having another function where you do more things in one go (perhaps more like in a  Pipeline manner or whatever suits your research).

Wanting something like:
import keplermapper as km
from sklearn import linear_model as lm

mapper = km.KeplerMapper()
projected_X = mapper.fit_transform(
  inverse_X=inverse_X,
  y=y,
  projection=lm.Ridge())

Where the projected_X is created by 5-fold out-of-fold guessing (we can write own code for this, or wait for the upcoming StackingEstimator). This gets us lens support for all Scikit-Learn supervised and unsupervised models.

Also, separating out fit_transform would make it easier to pickle?
from kmapper import Projector

projector = Projector(projection="dist_mean")
projected_X_train1 = projector.fit_transform(X_train)
projected_X_test1 = projector.transform(X_test)
joblib.dump(projector, "projector1.p")

projector = Projector(projection="std")
projected_X_train2 = projector.fit_transform(X_train)
projected_X_test2 = projector.transform(X_test)
joblib.dump(projector, "projector2.p")

I like being able to skip importing commonly used sklearn libraries like decomposition and manifold.
I think we can use:
import kmapper as km

mapper = km.KeplerMapper()
projected_X = mapper.fit_transform(X, projection=km.manifold.TSNE())

if we add from sklearn import manifold, decomposition, cluster in the __init__.py. Are there any possible problems with this? Violation of Python zen?

:thought_balloon:
import kmapper as km

mapper = km.Mapper(cover=km.covers.Cubical(n_cubes=[20,30],
                                           overlap_perc=[0.1, 0.2]),
                   nerve=km.nerves.AdaptivePairwise(min_samples=3),
                   clusterer=km.clusterers.HDBSCAN())

graph_train = mapper.fit_transform(projected_X, inverse_X)
graph_test = mapper.transform(projected_X_unseen, inverse_X_unseen)