Manually "bootstraping" after security object is registered

According to http://dev_devtoolkit.openmobilealliance.org/IoT/LWM2M10/doc/TS/index.html#!Documents/bootstrapmodes.htm a client may bootstrap itself by using a boostrap server, a smartcard or using factory bootstrap information.
In the examples this is done by setting up the server address in the security object and calling lwm2m_configure.
If you have to wait for factory bootstrap information to be available for one or another reason,
what is the correct way to provide it to wakaama, after lwm2m_configure has been called?
The AwaLWM2M implementation provides a AwaStaticClient_SetFactoryBootstrapInformation in https://github.com/FlowM2M/AwaLWM2M/blob/master/api/include/awa/static.h#L299 for exactly this case. It that already possible or should wakaama support something similar, too?

LWM2M TS V1_0 20160308, <IP_ADDRESS>
„In this mode, the LWM2M Client has been configured with the necessary Bootstrap Information prior to deployment of the device.“
If the client has been configured prior to deployment, I'm not sure, why "lwm2m_configure" should be called before.
May be the term "factory bootstrap" could be interpreted in different ways. But if you want a clarification on that, just use
the OMA issue page (https://github.com/OpenMobileAlliance/OMA-LwM2M-Public-Review/issues).
Mit freundlichen Grüßen / Best regards
Achim Kraus
Bosch Software Innovations GmbH
Communications (INST/ESY1)
Stuttgarter Straße 130
71332 Waiblingen
GERMANY
www.bosch-si.de
www.blog.bosch-si.com
Registered office: Berlin, Register court: Amtsgericht Charlottenburg, HRB 148411 B
Executives: Dr.-Ing. Rainer Kallenbach; Michael Hahn
Von: David Gräff<EMAIL_ADDRESS>Gesendet: Dienstag, 29. März 2016 12:53
An: eclipse/wakaama
Betreff: [eclipse/wakaama] Manually "bootstraping" after security object is registered (#129)
According to http://dev_devtoolkit.openmobilealliance.org/IoT/LWM2M10/doc/TS/index.html#!Documents/bootstrapmodes.htm a client may bootstrap itself by using a boostrap server, a smartcard or using factory bootstrap information.
In the examples this is done by setting up the server address in the security object and calling lwm2m_configure.
If you have to wait for factory bootstrap information to be available for one or another reason,
what is the correct way to provide it to wakaama, after lwm2m_configure has been called?
The AwaLWM2M implementation provides a AwaStaticClient_SetFactoryBootstrapInformation in https://github.com/FlowM2M/AwaLWM2M/blob/master/api/include/awa/static.h#L299 for exactly this case. It that already possible or should wakaama support something similar, too?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

Keep in mind wakaama is not multi-threaded.
When calling lwm2m_step(), if you do not have any server information (bootstrap or LWM2M server), the internal state machine will stay in STATE_BOOTSTRAP_REQUIRED and lwm2m_step() will return COAP_503_SERVICE_UNAVAILABLE.
When factory bootstrap information are available, wakaama will retrieve them from the Security Object lwm2m_object_t.
In your case, it is only a matter of handling the return code of lwm2m_step() accordingly.
Regards,


Don't convert the quotes

Hello,
I would simply want the quotes not converted when using mistletoe:
Here's what happen
>>> mistletoe.markdown('"test"')
'
"test"
\n'

And I would like the result to simply be '
"test"
\n'
Do you know how I could do that ?
Thank you

@barthelemypousset, there's no easy way how to change this in one place currently, you would have to change this in concrete calls to html.escape() within the HTMLRenderer class.
mistletoe historically escapes quotes (") and since version 0.9.0 we also escape apostrophes ('; see #115 and ef9bd3afc1e69b23b9238b2eeee45adec627adc7). But I can see this doesn't always have to be desired, so I think it would be cool to have the escaping as an option (or options) in a future version of mistletoe. Now 1.0.0 is about to be released, so I would plan this to 1.1.0.
Cc @anderskaplan. :)

@barthelemypousset (and @anderskaplan), something like in the created PR draft?
Regarding making the escaping of both double and single quotes disabled by default (not for attributes though, of course!), I'm not personally against that. I always prefer simpler things if there is no evident con (and "something" is definitely more readable and shorter than "something"). It would just require I hope a simple change to the commonpark.py test file.

... It would just require I hope a simple change to the commonmark.py test file, so that tests still pass.

So I've got this prepared, but just for sure, I will probably disable the double quotes escaping in the next release version after 1.1.0. Here is the diff (which would follow after the linked PR):
diff --git a/test/specification/commonmark.py b/test/specification/commonmark.py
index f3adda2..3f17c03 100644
--- a/test/specification/commonmark.py
+++ b/test/specification/commonmark.py
@@ -1,7 +1,7 @@
 import re
 import sys
 import json
-from mistletoe import markdown
+from mistletoe import Document, HTMLRenderer
 from traceback import print_tb
 from argparse import ArgumentParser

@@ -34,7 +34,8 @@ def run_tests(test_entries, start=None, end=None,
 def run_test(test_entry, quiet=False):
     test_case = test_entry['markdown'].splitlines(keepends=True)
     try:
-        output = markdown(test_case)
+        with HTMLRenderer(html_escape_double_quotes=True) as renderer:
+            output = renderer.render(Document(test_case))
         success = test_entry['html'] == output
         if not success and not quiet:
             print_test_entry(test_entry, output)


Swift 5.0 Compatibility

Codecov Report

Merging #17 into master will decrease coverage by 0.04%.
The diff coverage is 40.38%.

@@            Coverage Diff             @@
##           master      #17      +/-   ##
==========================================
- Coverage   83.74%   83.69%   -0.05%
==========================================
  Files         151      152       +1
  Lines        5776     5803      +27
==========================================
+ Hits         4837     4857      +20
- Misses        939      946       +7

Impacted Files
Coverage Δ

LineSDK/LineSDK/Crypto/Algorithms/ECDSA.swift
57.14% <ø> (ø)
:arrow_up:

...saging/Model/Flex/Component/FlexBoxComponent.swift
37.5% <0%> (-17.05%)
:arrow_down:

...g/Model/Flex/Container/FlexCarouselContainer.swift
25% <0%> (-15%)
:arrow_down:

LineSDK/LineSDK/Crypto/CryptoAlgorithm.swift
10% <0%> (-10%)
:arrow_down:

...ineSDK/LineSDKTests/Networking/PipelineTests.swift
94.35% <100%> (+0.28%)
:arrow_up:

LineSDK/LineSDK/General/LineSDKError.swift
81.08% <100%> (-0.26%)
:arrow_down:

LineSDK/LineSDKTests/Utils/ResultUtils.swift
100% <100%> (ø)

LineSDK/LineSDK/Crypto/JWK/JWA.swift
92.2% <100%> (+0.9%)
:arrow_up:

...orking/Model/CustomizeCoding/CodingExtension.swift
67.56% <100%> (ø)
:arrow_up:

LineSDK/LineSDK/Utils/Result.swift
13.88% <16.98%> (+0.47%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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


Sign conversion may change value in remove_trailing_zeros_traits (implicit conversions)

In include/dragonbox/dragonbox.h, in each of the remove_trailing_zeros_traits structs, on lines 2974 and 3012,  the statement exponent += s; invokes a warning from gcc when -Wconversions and Wsign-conversion are enabled.
I think a reasonable fix is the following:
                assert(s < detail::stdr::numeric_limits<DecimalExponentType>::max());
                auto sAddend = static_cast<DecimalExponentType>(s);
                assert(detail::stdr::numeric_limits<DecimalExponentType>::max() - sAddend > exponent);
                exponent += sAddend;

The suggestion would be better imo if detail::stdr had its own analogue of std::remove_reference, so that decltype could be used properly on exponent.
If s + exponent is expected to be able to wrap, that invokes undefined behavior without the option -fwrapv. If it is intended to wrap, a modification to the algorithm involving an unsigned exponent representation  with memcpy's back and forth may be required.

Thanks for raising the issue! In short, overflow can't happen so there is no UB.
In your suggestion,

The first line is pointless. It's entirely obvious that s cannot be larger than 127.
std::size_t is probably not the best type for s from the beginning: it should be the type allowing the best codegen, whatever that is. If the choice of type doesn't really matter in that regard, then we could just choose s to be DecimalExponentType so there is no warning. Otherwise, static_cast should be enough.
Given the correct static guarantee, the second assert is not necessary too but I like it to be there. I found that I didn't take account of trailing zero removal for static range checking of DecimalExponentType though. I will fix it.

My apologies, I didn't do a more in depth analysis of the code before submitting the issue. Thanks for responding so quickly!

Oh, don't get me wrong. My point was that I will investigate further and come up with a better solution if any. Thanks again!


fix(ci): Use chainloop action on scorecard pipeline

Leverages the use of chainloop action rather the manual steps so it complies with ossf/scorecard constraints.

could it be tested on PR?

We could test that the use of the action is correct but not the production checks. I'll run it.


Cleanup script VMM

I can't find The VMM cleanup Script.

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

ID: d4e25918-0eab-3275-8f0e-3d5d60d1b96a
Version Independent ID: ecdd86b3-e2da-3984-372a-83ede968acd1
Content: Remove servers and disable protection - Azure Site Recovery
Content Source: articles/site-recovery/site-recovery-manage-registration-and-protection.md
Service: site-recovery
GitHub Login: @Sharmistha-Rai
Microsoft Alias: sharrai

@tawfik21 Thanks for your comment. We will review the issue and get back to you shortly.

Hi @tawfik21 - we are aware of this issue and are trying to fix this. I will keep this thread updated with any updates we find.

@tawfik21 - we have updated the documentation and added the script in the same documentation page.

@SwathiDhanwada-MSFT - request you to close this issue.

@tawfik21 As the documentation is updated , we will close the issue for now. If there are further questions , please revert and we will be glad to assist you.


False positive InvalidPackageDeclaration in buildSrc folder

Expected Behavior
Rule is not triggered.
Observed Behavior
Rule is triggered.
Steps to Reproduce
Run on this file: https://github.com/shadowsocks/shadowsocks-android/blob/f5bc88b83ebbd4d2941a65bea2eaa565f2072db4/buildSrc/src/main/kotlin/Helpers.kt#L1
Context
This file should be in root package.
Your Environment

Version of detekt used: v1.17.1
Version of Gradle used (if applicable): N/A
Gradle scan link (add --scan option when running the gradle task):
Operating System and version: Linux Mint 20
Link to your project (if it's a public repository): See above

This is sort of by design. Since detekt has no way of knowing what the source root is, any file that does not contain a package declaration is flagged. I see a few ways forward.

You can exclude files directly in buildSrc/src/main/kotlin from the rule (see below)
We could add a flag ignoreMissingPackageDeclaration to the rule
We could ignore files in src/*/java and src/*/kotlin with a missing package declaration

  InvalidPackageDeclaration:
    excludes: ['**/*.kts', '**/buildSrc/src/main/kotlin/*.kt']

To be honest I prefer option 1. :wink:

As discussed in #3947 this is a platform specific customization that we, as detekt, doesn't want to have by default. There are users that doesn't use gradle.
But we are going to create MissingPackageDeclaration that should help with this: #3957


[Legacy Code]: Remove head patting from the lift-n-shifted PAUCAL

Issue Type
Common codebase
Issue Description
Since PAUCAL was only copied into Hex Casting, certain diabolical features are still present, such as head patting and giving random Minecraft users special powers.

Fixed by d1e8c6d


man should be working in utf8 rather than mapping down to single byte

The code is converting UTF8 files into ISO8859-1 / IBM-1047 and it would be good to keep it in UTF8.
Two examples are groff (which has to map UTF8 angled brackets to < and >) and man --locale=fr vim which forces you to change your terminal setting to western ASCII to properly display characters (rather than a more natural UTF8 default)

Another problem pointed out by @lbdyck can be seen with man -K mount

If I switch my display to 'western (ASCII)' then the characters display correctly, but I still see some odd text on the screen afterward

and then:

fixed in 'cleanuppatches'
Japanese:

Russian:
https://files.slack.com/files-pri/T27SFGS2W-F05A1SV9J2G/image.png
French (in UTF-8):

German:

Italian:

Polish:

Turkish:

Another note - if working with UTF-8 characters and you use iterm2 on a mac (or perhaps other terminal emulators), turn on 'use ligatures'. Now Turkish looks like:


docs: builders dev container & modify layout

Fixes https://github.com/daytonaio/enterprise-docs/issues/161

Looks good to me

@ivan-burazin


[Feature] GUI scale support?

Is it possible to get GUI scale support for modules, or is it already in the mod?

For technical reasons I most likely won't be doing this. Some modules change size based on their settings, and the GUI scale option in the vanilla menu still works. Closing for now.


Change selected date on input + enter

possible fix for #848
I couldn't see any reason why we don't want the calendar to open on every InputKey event or change the preSelection on change of the selected input property.
Did I miss something?

Codecov Report

Merging #857 into master will increase coverage by 0.17%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #857      +/-   ##
==========================================
+ Coverage   85.89%   86.06%   +0.17%
==========================================
  Files          12       12
  Lines         645      646       +1
  Branches      120      120
==========================================
+ Hits          554      556       +2
  Misses          7        7
+ Partials       84       83       -1

Impacted Files
Coverage Δ

src/datepicker.jsx
92.25% <100%> (+0.69%)
:arrow_up:

Continue to review full report at Codecov.

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


Coap plugin started, but can't use it

Environment
emq 2.3.9 in docker, official image from here
Expected behaviour
I can start the coap plugin from the dashboard and connect to it with clients libcoap or coap-cli
Actual behaviour
Started image with:
sudo docker run -tid --name emqtt-coap -p 5683:5683 -p 5684:5684 -p 1883:1883 -p 8083:8083 -p 8883:8883 -p 8084:8084 -p 18083:18083 emqttd-docker-v2.3.9
Open plugin with

sudo docker container exec -it [containerID] /bin/sh
./bin/emqttd_ctl plugins load emq_coap

Result

Dashboard shows up

Port listen

Logs

Connetct through libcoap

I belive the port 5683 should be UDP, not TCP.

Thanks, i have resolved this problem by changing tcp port to udp


Move under TypeStrong

Hey! I've been using this for a while personally and I wanted to reach out and ask if you'd be interested in maintaining the repository under https://github.com/TypeStrong? We've got a few TypeScript projects in there and we might be able to help each other along a little as we go.

+1

The owner does not seem to be active and while I would prefer to get explicit permission, the license does allow cloning and reuploading into the TypeStrong repo. I know this is bad practice but having this great tool stuck on TS1.6 is a real blocker for me.

fork as ts-doc

@blakeembrey As you are a member of TypeStrong, can you do this?

Hi all, I'm very sorry for being so unresponsive lately. Both my professional and private situation has changed a few months ago and since then I've been pretty bad at managing my correspondence with the followers of this project. This should be no excuse, I should have been able to manage to drop you a few lines at least and I want to apologize for my behaviour.
I've been working on this project since early 2014 and tried to keep it updated over the years so I'm really dedicated to this work. I would be more than happy if someone want's to take a spin on this project now I am in a situation where I am not able to invest as much time as it would need to keep the project maintained. TypeStrong for sure would be a perfect place to put this project to so if you are still interested I would be happy to transfer the project over there.

@sebastian-lenz Thanks :smile:
Hope all is well also!

@tht13 Thank you. Nothing to worry about, everything is fine, it's really just missing time right now.

@sebastian-lenz just wanted to thank you for work.  I've been using typedoc for the new angular ui-router docs and love it.  Writing api docs feels almost effortless!
+1 for @sebastian-lenz
+1 for moving to TypeStrong

@sebastian-lenz You should be able to transfer it, I'll try to help maintain the issues and PRs with you :+1:

Okay, TypeDoc has a new home!

Thanks for all your work @sebastian-lenz! If you get the time, it'd be awesome to give some overview for others to get started helping out. I'll review the existing issues and PRs when I get a chance.

Yes, thank you very much for your work, @sebastian-lenz! Typedoc was an important factor in us deciding to use typescript.

Just a big thank you @sebastian-lenz . If you have opportunity to meet in Berlin, just ping me. Typedoc has been an enormous help for me and the company I'm working for right now. Kudos for everything.


Add a custom field to logspout

Hi,
Is it possible to add some data in a custom field ?
I have a 3 swarm cluster, namely : dev, staging and production.
I would like to run each logspout container with the corresponding env value.
Something like :
$ docker run -d --name logspout \   -v /var/run/docker.sock:/var/run/docker.sock:ro \   rtoma/logspout-redis-logstash \   redis://<your-redis-server>?key=bla&cluster_env=production
Thanks !

Sounds like a great feature, but I do not have the time to implement this myself. Feel free to submit a pull request.


MassAssignmentException in Model.php

First of all, I really love Backpack, especially the CRUD package. Great work.
I was under the impression that, when using the generators, the new CRUD would work out of the box. But when I test it, using the example from the Docs, I get an exception:
MassAssignmentException in Model.php line 444:
name

When I look at the Model that was created, I see that the fillable is still commented out.
Is this intended behaviour? Should I edit the Model, the Controller and the Request manually?
Or am I doing something wrong?
Any help would be greatly appreciated

@appdezign it does, however what you're describing is laravel specific.
You couldn't populate the fillable automatically array as we have no idea what you'd want to be put in it.
Even if you had uncommented fillable array, that error would still happen because your using the CRUD to update a model, but you've not setup the permission/fillable on your model.
There's nothing we can really do about that, it's just how laravel works, if you try to do a mass assignment illegally it will error.
The reason it's there is just to give new comers the ability to easily add their own fillable.
I'll close this up now just to keep the area tidy, but if you have any more questions I'm sure somebody can answer them :)


Don't use Win32 native APIs on non-Windows for crypto of secure string over remoting

PR Summary
Remoting relies on Windows native crypto APIs to pass a secure string.  Of course these APIs don't work on non-Windows.  However, the remoting code does not expect this to fail so ends up in a hang waiting for an event that never happens.
Have not figured out how to return an error to the user in the case Get-Credential is called within a PSSession.  Currently it just quietly closes the remote connection.  @PaulHigin can you give a pointer to where this code would exist?  Debugging the current code, it has a bunch of error handlers which just result in the pssession being closed and not sure where throwing is appropriate.
Fix https://github.com/PowerShell/PowerShell/issues/8723
PR Context
Fix is to provide implementations of the Win32 native pinvoke apis that throw an exception.  The reason to do it this way is that there's a bunch of code that calls some of the APIs of the wrapper .Net class to setup structures that eventually get passed to the win32 api.  Throwing in the managed class causes other things to fail early that should work on non-Windows.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

[ ] Add [feature] to your commit messages if the change is significant or affects feature tests

@PaulHigin can you re-review?


[LOGGING-2708] Try to trigger release

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

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


[SPARK-49300][CORE][3.5] Fix Hadoop delegation token leak when tokenRenewalInterval is not set.

Backport from master.
When tokenRenewalInterval is not set, HadoopFSDelegationTokenProvider#getTokenRenewalInterval will fetch some tokens and renew them to get a interval value.
https://github.com/apache/spark/blob/dd259b0b27841e6dd7c07f8ca3cc05d275863dd5/core/src/main/scala/org/apache/spark/deploy/security/HadoopFSDelegationTokenProvider.scala#L60-L64
These tokens do not call cancel(), resulting in a large number of existing tokens on HDFS not being cleared in a timely manner, causing additional pressure on the HDFS server.

@zhangshuyan0 mind filling the PR description please? https://github.com/apache/spark/blob/master/.github/PULL_REQUEST_TEMPLATE

cc @sunchao from

https://github.com/apache/spark/pull/47800

@zhangshuyan0 mind filling the PR description please? https://github.com/apache/spark/blob/master/.github/PULL_REQUEST_TEMPLATE

Already added the description.

Merged to branch-3.5, thanks!


some production enhancement to pg_inspector

some production enhancement to pg_inspector

compatible to current and previous version of miq application name
output blocked connections to separate file
use database.yml
only generate server.yml when not exists
add a ruby wrap for decrypt password and cron to run pg_inspector_server with database.yml information

\cc @jrafanie @yrudman @gtanzillo
@miq-bot add-label wip, tools

locks are still too few, I ran all steps minutely for ~10 hours but can't find any block, so I manually lock miq_servers and get this output, seems fine.

@miq-bot remove-label wip

@ailisp let's review this on Wednesday with @yrudman


Disable TestKubernetesROService e2e test

This test is failing probably due to an error with rate limiting. Let's remove this test until we can get to the bottom of this issue.

LGTM


On keycloak redirect back after succes login app crashes

My keycloak

The error I get after a successful login!

My build gradle
Im using the example aplication and my keycloak its working well! But when it redirects to the app i receive this error!
Could you help me?

My config

Hi, there.
I'm just saying that I'm quite busy this week. So, I'll look at this issue again later. Maybe tomorrow or the day after. I'm so sorry for the inconvenience. But in case you have resolved the issue, maybe you can share your findings here.
My only suggestion for now is maybe you can downgrade the package version to see whether the problem only occurs for the latest version or not. Hope that helps.

OK man, I still not solved the problem! But ill try downgrade and try again, thanks!

Man, one more question. Did you had success redirecting from keycloak right to the app?

Yup, most of the time and my company also using this package in production.
I afraid this issue might only happens on the latest version due to dependecies update. However, please read through the documentation thoroughly so you won't miss any important details.

Which is the version you are using in your Company? Do you know the keycloak version? The problem im having is inside android where the keycloak redirect uses http not https and Android guess its insecure!

If you are using http, you have to allow insecure http connection on your android app. Have you checked out this guide?

The question is that Im not running http, im running https but it is changing to http!

Mmm... I've tried using the example app and it is working fine logging to my keycloak server.
I don't think your issue is a configuration issue, but maybe your keycloak server is behind a firewall that your emulator can't reach. Or you have to double check your input value or something. I'm also not sure, but the last thing I can do to help you is thru a video call.

 Thanks for be so willing to help. Maybe we can make a video call and solve this. How is your schedule?

Add me on Linkedin and we'll discuss there.

I have same problem, on keycloakc redirect back after success login app not receive the code.
i review my keycloack credential it is ok.

click in the first icon the message: login failed

click in the second icon the message: No stored state - unable to handle response

@ecolque
Remove android:taskAffinity="" from your AndroidManifest.xml file.


IndividuatlAccounts TDTM Repetitive method call

Note: FOR INTERNAL USE ONLY. For support questions, please use the EDA Chatter group in The Power of Us Hub.
For feature requests, please use the Ideas tab in The Power of Us Hub.

Do you want to request a feature or report a bug?

Bug

What is the current behavior?

When inserting a new Contact without an Account, ACCT_IndividualAccounts_TDTM auto-creates the Account and links the Contact to it (sets the AccountId) using the handleInsertProcessing method. Immediately afterwards, the handleUpdateProcessing method is called...where it tries to auto create the Account again. Fortunately, the method checks for existing Accounts where Primary Contact is set, so it links the contact to the existing account this time through and updates the contact again. So, this is not creating duplicate accounts, but it's resulting in an extra Contact update call (which then goes through all the other TDTM trigger handlers again).

What is the expected behavior?

Should not take repeated trips through the code

What is the motivation / use case for changing the behavior?

Performance is hindered and bulk unit tests can fail more often

Tracking: W-7991797


For ac-arcus

ac-arcus.eu

@NikichaTV Thank you for your patience while waiting for a review and processing your request. Pull request merged. Thank you!


awjudd/feed-reader 1.2.0 requires simplepie/simplepie dev-master -> no matching package found.

Latest simplepie:
    "simplepie/simplepie": "^1.3",

The simplepie releases haven't been updated in more than 2 years because of that the application uses dev-master.
Once the following issue is closed (https://github.com/simplepie/simplepie/issues/390), then I will be able to finally move it off of dev-master.

The issue is closed and the problem stays.

Thank you for informing me - I will test it all out within the next few days.

@TareqElMasri - This has now been completed.  I have made a 1.3.0 release.


Amend scanner to support astral characters in identifiers when parsing es6+

Fixes #31963
I've made the scanner just code-point aware enough to handle passing astral glyphs into isIdentifierPart, but it still uses charCodeAt anywhere that it's not explicitly going to look for astral plane glyphs (ie, is only looking for CharacterCodes.newLine). In addition, I've added a new set of unicode identifier start/part arrays generated by the script I've added in the script folder.

Are there plans to provide emitter for ES5 and below?
var 𝑚 = 4; in JS output will trigger Invalid character in IE11

We don't downlevel the es5 Unicode set into an es3 compatable thing - we just issue Invalid Character errors, and this es6+ Unicode support is the same way.
Btw, the Unicode table I've added was captured with Unicode v11.0. IIRC 12 is out and 13 is in draft. I can update it to a newer version by using latest node, probably, but we should come up with an update policy for the es6+ Unicode table - we don't want to add a unicode: 12 flag or the like just for this, I don't think.

Would it be possible to replace "invalid" characters with an identifier representing their codepoint? E.g., 𝑚 would become U1D45A to match its codepoint representation: U+1D45A (sans plus).

Would it be possible to replace "invalid" characters with an identifier representing their codepoint?

Technically any number of escaping schemes are possible, but by substituting characters, it could affect the public API using those identifiers. For an internal const a rename is no big deal, but export it and suddenly that required renaming has big (undesirable) semantic meaning.

@weswigham oh, I see it now. Thank you for clarification.

Did ES3 support Unicode escapes in identifiers? I can't remember anymore.
var foo = "Foo";
console.log( \u0066\u006F\u006F); // -> "Foo"

Not that it would be of much help for astral-plane characters, since it's limited to 4-digit codepoints anyway... 😕

ping @rbuckton for a review again ❤

@rbuckton any more feedback on this?

The build script could be simplified and made to not rely on the current Node.js/V8 + Unicode version:
const ID_Start = require('unicode-12.1.0/Binary_Property/ID_Start/code-points.js');
const ID_Continue = require('unicode-12.1.0/Binary_Property/ID_Continue/code-points.js');

// ...then add the other needed characters, and write the two arrays to disk.

I'd also like to propose not including Other_ID_Start and Other_ID_Continue "for backwards compatibility" without evidence that it's necessary. Rationale: AFAIK no modern JavaScript engine supports this, no other popular JS tool supports it either, strongly suggesting that it's just not needed. Links to how other tooling implements this:

Esprima: https://github.com/jquery/esprima/blob/master/tools/generate-identifier-regex.js
@babel/parser: https://github.com/babel/babel/blob/master/packages/babel-parser/scripts/generate-identifier-regex.js
Acorn: https://github.com/acornjs/acorn/blob/master/bin/generate-identifier-regex.js
JSHint: https://github.com/jshint/jshint/blob/master/scripts/generate-identifier-data.js
XRegExp: https://github.com/slevithan/xregexp/blob/master/tools/scripts/utils.js

Huge shoutout to the devs who were patient enough to tend to this. =) Kudos. 👍

I'd also like to propose not including Other_ID_Start and Other_ID_Continue "for backwards compatibility" without evidence that it's necessary.

Also a huge shoutout to IBM and EBCDIC for creating the Next Line character (U+0085), despite mainframe systems already supporting CR and LF at the time, but figuring the world needed a third line terminator anyway. 👍
@mathiasbynens, have you ever encountered a legitimate use of Line and Paragraph Separators?  (LS/PS). I know they're only defined by Unicode for compatibility, but I'm curious if you've ever seen them used for anything practical...
They're like Proxies. Not saying they're useless, only that I still haven't found a practical use for one (that doesn't involve membranes or DOM-related shenanigans…)

I'd also like to propose not including Other_ID_Start and Other_ID_Continue "for backwards compatibility" without evidence that it's necessary. Rationale: AFAIK no modern JavaScript engine supports this, no other popular JS tool supports it either, strongly suggesting that it's just not needed.

Would you champion removing note 3 from this section of the spec which very explicitly calls them out as included, then? It would very much seem to me that identifiers named ゜ (which is in the other start table) are, in fact, supported in chrome, at runtime. (Or at least over at https://jsconsole.com/ , I'm on mobile)
As for not relying on the runtime... I think I prefer to use the table generated from the runtime. We'll get exactly the character set matched by a known v8 version (we don't intend to keep multiple copies around, but will probably periodically update the table when new major node releases occur).

Huge shoutout to the devs who were patient enough to tend to this. =) Kudos. 👍

We actually found bugs in browsers while working on follow-ups to this. Go figure.

Would you champion removing note 3 from this section of the spec which very explicitly calls them out as included, then?

That non-normative note just calls out the fact that ID_Start already includes Other_ID_Start per Unicode. I think it's fine to keep it.
I'm simply recommending the removal of the hardcoded Unicode code points such as U+2118 which are Other_ID_Start on the grounds that it's not necessary, since those are already covered by ID_Start:
/\p{ID_Start}/u.test('\u2118');

The only reason you'd want to hardcode those characters explicitly is if you find an engine whose \p{ID_Start} doesn't match them.

We actually found bugs in browsers while working on follow-ups to this. Go figure.

It's great that you're discovering bugs and filing them <3 Thank you!
This (the fact that engines will always have bugs) is exactly why I would recommend against relying on any particular runtime's behavior. Your above statement seems to conflict with:

As for not relying on the runtime... I think I prefer to use the table generated from the runtime. We'll get exactly the character set matched by a known v8 version (we don't intend to keep multiple copies around, but will probably periodically update the table when new major node releases occur).

IMHO it's better to have a separate implementation (so that you'd spot bugs which you could then file) instead of relying on any one implementation to always be correct.

@mathiasbynens, have you ever encountered a legitimate use of Line and Paragraph Separators? (LS/PS). I know they're only defined by Unicode for compatibility, but I'm curious if you've ever seen them used for anything practical...

In my opinion making LS and PS LineTerminators was a mistake, but unfortunately we cannot go back and change it now. I don't really understand how this is on-topic though, since LS and PS are not ID_Start/ID_Continue characters -- could you elaborate?

I'm simply recommending the removal of the hardcoded Unicode code points such as U+2118 which are Other_ID_Start on the grounds that it's not necessary, since those are already covered by ID_Start

Ah, ok. So it's just that the regex is redundant, alright. I was under the impression that the sub-classification group needed special handling (and was called out in the spec) because it wasn't part of the base ID_X property set. Should that be the case, I can simplify the regex in the script a tad, though it shouldn't change the generated table.

I don't really understand how this is on-topic though, since LS and PS are not ID_Start/ID_Continue characters — could you elaborate?

Oops, sorry, never mind. 😅 I was midway through my reply when I got sidetracked browsing Wikipedia, reading up on the different ways to encode a newline.  Yuck…)


$event.stopPropagation prevents modal form opening.

Hello. This is very strange problem for me. Modal is working, but if I set $event.stopPropagetion(), it can't be opened any more.
Add new folder
Other strage thing is if modal's link parent element has (click)="$event.stopPropagation()" modal can't be opened too:

Add new folder

Any idea how to fix this or some kind of workaround ?

You can use MaterializeActions to force opening the modal. Does that work for you?

Yes, this will do the job, but i prefer not to do it this way.

This is kind of the design of this library - actions are performed once via the template, or if you want to dynamically to things then MaterializeActions. Happy if you propose a new one with a PR :)
I'll close this issue for now, unless you think there's something to be done here.
Thanks.


Setup a shared 1Pass

Thanks for creating an issue! Please fill out this form so we can be sure to have all the information we need, and to minimize back and forth.

What are we trying to do?
Now that 1Password is doing free-for-FOSS, let's join in on that and then gather all our sensitive stuff into one shared place and to reduce any lock out risk.

What feature or behavior is this required for?
Better resiliency!

How could we solve this issue? (Not knowing is okay!)
Kat or Colin follow these steps: https://github.com/1Password/1password-teams-open-source#how-to-apply

Anything else?
Figure out who else to add to the 1Pass after the initial setup.

PR opened with 1Pass: https://github.com/1Password/1password-teams-open-source/pull/355

Completed!


Missing CoC and so...

@danielgruesso kindly add it here also as in https://github.com/coti-io/coti-sdk-typescript-examples/pull/8

@gmesika-coti see https://github.com/coti-io/confidentiality-contracts/pull/16

plz fix the links


postcss-loader shows error while compile this style grid-column: 1/-1;

postcss-loader throws error while compile this line
grid-column: 1/-1;
but it works when we add space between the slash like below.
grid-column: 1 / -1;

Thank you for creating this issue. However, issues need to follow one of our templates so that we can clearly understand your particular circumstances.
Please help us help you by recreating the issue using one of our templates.
Please open an error in plugin where you handle grid, we don't touch your code


Add formatting and header for Google sheet export.

This gets the output closer, though still not identical, to our template. For comparison:
Colorado data
Sample export of CO data
Known limitations:

The program description header doesn't have a link. I'm inclined to either hard-code that or just not support links in the headers. We can definitely have them in the data model sheet.
The header groupings (Rebate Amount Fields and Restrictions / Eligibility Conditions) are hard-coded by column number. I also thought about having the user specify a start and end column by name. I think we can revisit it if we notice this breaking a lot.
links/URLs still don't become clickable by default. I will address this in a separate PR

To make this easier, I separated the previous collectedIncentiveToGoogleSheet method into two methods. One performs the value transformations to get to something that any spreadsheet needs; the other is concerned with exporting and formatting specifically to Google Sheets.
The testing story is a bit inconvenient here. We can absolutely add another test that verifies that the standard header output is expected, but it will be a very large golden file. Even the foo-bar-baz version without the header is pretty large.

Nice. I think there's no need to go to great lengths to have links in headers. Whatever the links are can go in the separate explanatory sheet.
I assume data validations for the enum columns are also on your radar as a followup?

Yes, planning to get to that tomorrow!


Introduce a 'script' key for actions

In order to be able to define actions in some other ways then providing scripts, accept actions to be dictionaries and add support for a script key, with a single or multi-line string value, which is executed as a bash script.
Examples:
actions:
  create-archive:
    script: make archive

actions:
  create-archive:
    script: |
      rm *.tar.gz
      make archive

This change is needed to support customizing source-git update-dist-git (see packit/research#150).

I think this would be a big quality of life improvement. Here is a snippet from an existing Packit config:
actions:
  changelog-entry:
  - bash -c '[ -n "$PACKIT_RESOLVED_BUGS" ] && echo "Update to ${PACKIT_PROJECT_VERSION} (rhbz#${PACKIT_RESOLVED_BUGS})" || echo "Update to ${PACKIT_PROJECT_VERSION}"'
  commit-message:
  - bash -c '[ -n "$PACKIT_RESOLVED_BUGS" ] && echo "Update to ${PACKIT_PROJECT_VERSION} (rhbz#${PACKIT_RESOLVED_BUGS})" || echo "Update to ${PACKIT_PROJECT_VERSION}"'

It could be written like this (just an example, there are more possibilities):
actions:
  changelog-entry:
    script: |
      if [ -n "$PACKIT_RESOLVED_BUGS" ]; then
          echo "Update to ${PACKIT_PROJECT_VERSION} (rhbz#${PACKIT_RESOLVED_BUGS})"
      else
          echo "Update to ${PACKIT_PROJECT_VERSION}"
      fi
  commit-message:
    script: |
      if [ -n "$PACKIT_RESOLVED_BUGS" ]; then
          echo "Update to ${PACKIT_PROJECT_VERSION} (rhbz#${PACKIT_RESOLVED_BUGS})"
      else
          echo "Update to ${PACKIT_PROJECT_VERSION}"
      fi

I also think it would be nice to have e.g. interpreter key to be able to script in different languages (and we could also encourage users to use the specfile library to make spec file changes):
actions:
  fix-spec-file:
    interpreter: python
    script: |
      from specfile import Specfile
      spec = Specfile("package.spec")
      ...


request: search bar at the bottom

search bar at the bottom, its more natural to the finger as they travel less from the keyboard

I personally don't like this option. I've used KISS Launcher (which has this feature) for some time and it's really annoying that the bar constantly changes position to be over the keyboard. It's one of the reasons I stopped using KISS.

but it would be nice to have the option!

also support for landscape mode, please!

I don't have a problem if it's another option, I just don't want it to be the only option.

Seems like a duplicate of #64 .

so?


Scroll to last edited clue in wordedit.html

Steps to reproduce

Open a puzzle
In the across or down clue list, scroll to a clue down near the bottom
Click the clue name and edit the clue.  Click OK

The list starts at the top again.  Need to have it stay on the same page

Harder than I thought.  See https://www.w3schools.com/jsref/event_onscroll.asp, maybe.


DHT connectivity lost/unclear

Had pkdns running for a few hours, then published a record using a fresh key from another location on the internet, but pkdns didn't provide an answer within ~5 minutes. So I restarted pkdns, gave it a minute and then it was able to resolve the query successfully.
I'm running pkdns with -v for verbose mode but don't see any messages on DHT status. It would be great to surface some info like DHT connection health every so often.

#31 adds a logging doc. Running RUST_LOG=pkdns=debug,any_dns=debug,pkarr=debug,mainline=debug pkdns will output the mainline logs.
The mainline DHT uses UDP and not TCP and is therefore connectionless so you will not see any connection info. Nonetheless logs like DEBUG mainline::rpc: Populated the routing table table_size=108 show how many other DHT nodes pkdns knows about. It's a great indication if everything is working.

That should help. Closing for now but will reopen if it still happens. Thanks!


Pub publish docs do not specify --server

Tracking bug.
http://pub.dartlang.org/doc/pub-lish.html
No mention of --server

This issue has been moved to dart-lang/pub#485.


Add Page Segmentation Mode

Add support for page segmentation mode. this will be similar to functionality  found in Tesseract
Page segmentation modes

sparse Find as much text as possible (default).
word  Treat the image as a single word.
line    Treat the image as a single text line.

Partial Page Segmentation Mode has been implemented.
Boxes

Following segmentation modes have been added.
    """ "Page Segmentation Modes"""

    # Treat the image as a single word.
    WORD = "word"
    # Sparse text. Find as much text as possible in no particular order.
    SPARSE = "sparse"
    # Treat the image as a single text line.
    LINE = "line"
    # Raw line. Treat the image as a single text line, NO bounding box detection performed.
    RAW_LINE = "raw_line"
    # Multiline. Treat the image as multiple text lines, NO bounding box detection performed.
    MULTI_LINE = "multiline"


fzf relies on incorrect terminal wrapping behavour

Category

[x] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[ ] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

[ ] Linux
[ ] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[ ] bash
[ ] zsh
[ ] fish

fzf seems to rely on that outputting a character at the last column will not wrap to next line. This is true for modern terminal emulators that basically provides infinitely long lines to be able to do dynamic re wrapping of text but doesn't work on a basic xterm/vt100 emulation (which incidentally is what I am working on).

Is there any way I can reproduce the problem on my macbook?

Well... I don't currently know of any terminal emulators except mine that doesn't treat lines as infinitely long :)
Investigating this further it turns out that for instance zshell does this too in some cases with the line editor and custom prompts, so I guess this is just one of the many ways in which de facto behavior has kind of overridden the standard.
Luckily I came up with a work-around; Allow the cursor to be temporarily outside the right border, and don't wrap until the application actually tries to write text there -- it seems to work in all cases I tried.
I guess I'll close this, I was too hasty in thinking that this was an isolated thing that only fzf did.


Broken tree-sitter-hcl/terraform grammar

Describe the bug
Highlighting for *.tf files in Neovim is broken due to a split of the grammar. This seems to be due to a change in how the grammar is provided.
There is no permanent fix for now as far as I know, but I thought this might help others find the issue and not waste a lot of time until there is a new release in https://github.com/MichaHoffmann/tree-sitter-hcl
Workaround: https://github.com/pecigonzalo/dotFiles/blob/ab718c4b6dbf56057cbe3aa8033b617a21966c6d/nix/overlays/neovim.nix#L16-L22
Steps To Reproduce
Steps to reproduce the behavior:

Use 37b97ae3dd714de9a17923d004a2c5b5543dfa6d nixpkg
Configure Neovim and Treesitter with all grammars
Open a .tf file

Expected behavior
Syntax should be highlighted
Additional context

https://github.com/nvim-treesitter/nvim-treesitter/issues/4088

Notify maintainers
@Profpatsch @oxalica
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
❯ nix-shell -p nix-info --run "nix-info -m"
 - system: `"aarch64-darwin"`
 - host os: `Darwin 22.2.0, macOS 13.1`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.12.0`
 - channels(root): `"nixpkgs"`
 - nixpkgs: `/nix/store/qvkzwk5awk8np49w7yd8alisl316l2dr-source`

I had the same issue and tried the solution in your linked dotfiles - thanks for sharing, I hadn't figured out how to override ts parsers, so I appreciate having an example to copy off in future.
I then came across a better solution to this problem that was merged in late last year
I can see in your dotfiles you have the same line as I did https://github.com/pecigonzalo/dotFiles/blob/ab718c4b6dbf56057cbe3aa8033b617a21966c6d/nix/home/editor.nix#L90
You can change that to the syntax they suggest in the PR, but I prefer the slightly more verbose solution, as it can be extended with more parsers.
# original suggestion in docs
(pkgs.vimPlugins.nvim-treesitter.withPlugins (plugins: pkgs.tree-sitter.allGrammars))

# new suggestion in docs
pkgs.vimPlugins.nvim-treesitter.withAllGrammars

# alternative that can be extended
(pkgs.vimPlugins.nvim-treesitter.withPlugins (_: pkgs.vimPlugins.nvim-treesitter.allGrammars))

Thanks for sharing @jay-aye-see-kay! I might have misunderstood but they are separate issues, right?

Creating an overlay adding the terraform grammar
Using withAllGrammars to fix

Does withAllGrammars add the Terraform grammar somehow without adding the overlay?

Ah no sorry I misunderstood. They are definitely separate issues. I didn't realise what you meant by a split in the grammar.
I think my suggestion may remove the need for some of your hard coded TS overlays (i.e bash and kotlin now work OOTB for me), but is not relevant to this issue.


Detecting the IPFS Companion extension on the user

Is your request for an opportunity related to a problem? Please describe.
I directly use IPFS to deliver content to my users, and I want users to download that content faster if the JS feature checks to see if they have IPFS Companion installed, and therefore can minimize the cost of downloading content by using local content downloads instead of using gateways like ipfs.io, cloudflare-ipfs.com and others. This will also help other sites that use IPFS for fast content uploads.
Describe the solution you would like to get.
I can suggest a superficial solution: Load a specific module into your browser, like the MetaMask crypto wallet does (see. https://ethereum.stackexchange.com/a/51356 or https://stackoverflow.com/a/14399123/16782022).
Describe the alternatives you've considered.
I don't see any other alternatives how to set that IPFS Companion is installed on the user.
Additional context.
No.

I would like to take this one :)

If you have DNSLink set up on your domain name, IPFS Companion will redirect and load your website from a local gateway.
Would you be willing to unpack what you mean by "download that content faster if the JS feature checks to see if they have IPFS Companion installed"?


Cloning of response leads to issues

This line:
https://github.com/sveltejs/kit/blob/8805c6daa6b11a46d2d373c3db890227b27e660d/packages/kit/src/runtime/server/page.js#L128
Causes issues in Node where it dies unexpectedly without any message on server. The Node high water mark limit is much smaller than browser. Can increase high water mark as workaround
There's multiple issues talking about this in node-fetch (e.g. 671 and 151). Their recommendation is to stream and provide back pressure: https://github.com/node-fetch/node-fetch/issues/151#issuecomment-482777213

If someone can create a repro it would help a lot

There was a comment on Discord that they're only reproducing this if the request goes through Nginx: https://discord.com/channels/457912077277855764/819723698415599626/823759556789272597

@Rich-Harris yes, self-locking happens here https://github.com/sveltejs/kit/blob/master/packages/kit/src/runtime/server/page.js#L141
If an HTTP packet contains more than two TCP packages (the first packet is usually the header) and one of these TCP packages for the body did not fit into the write buffer.
Basically, await clone.text() waiting for a response from the stream, but this stream waiting drying (reading) buffer by original response what can't be returned from the function because we waiting await clone.text().
Basically, const clone = response.clone(); can work only if all tcp package for HTTP body fit into write buffer OR JS interpretation/compiler/VM smart enough to detect self-locks what NodeJS not.

Is there any way to interpret this other than 'fetch is fundamentally broken'?

If you talking about sveltekit fetch - then yes.
PS one more link https://github.com/whatwg/streams/issues/506#issuecomment-242073695

The way to fix this problem is to resolve the cloned response in parallel with the original. For example:
const fetch = require('node-fetch');

const response = await fetch('https://example.com');
const r1 = await response.clone();

const results = await Promise.all([response.json(), r1.text()]);

The response in SvelteKit is used in two places:

it's serialized and put into the page for hydration
it's used in the server-side implementation of load as desired by the user

The problem here is that we're trying to do these basically in serial by basically intercepting the response.
I can't think of a way to read the stream in the two places in parallel while maintaining our current API. The issue is that load takes fetch as a parameter, so we can't call load and serialize the response in parallel.
So as much as I hate to say it, I think that SvelteKit's load implementation is fundamentally broken and I don't think there's a way to fix it while keeping our current API. The only thing I can really think at this point is that we probably made the wrong decision in https://github.com/sveltejs/kit/issues/268 and should go back to a Sapper-like implementation where the user can handle the fetch response however they like and then they hand it off to us. At that point we can take the user's response and in parallel render the Svelte components and serialize the data for hydration
This is why we're in beta and not final release :grin:

So as much as I hate to say it, I think that SvelteKit's load implementation is fundamentally broken and I don't think there's a way to fix it while keeping our current API

o ye of little faith #635

Proxies are truly the Spanish Inquisition of js language features.

Could it be that moving to https://undici.nodejs.org/#/?id=undicifetchinput-init-promise can help to fix an issue (and yes undici is not equal to node-fetch)

@nosovk probably can't, because it's very fundamental issue, but current proxys really work


Can't find libicucore.dylib in iOS standard frameworks

The README states:

Some additional iOS standard frameworks must be added.

Select your project in the Xcode file explorer and open the "Build Phases" tab. Expand the "Link Binary With Libraries" section and click the + icon

Add the following frameworks

...

libicucore.dylib

but I don't see libicucore.dylib. Instead I see libicucore.tbd. I am using Xcode 7.1:

Are these interchangeable? And could the README be updated to reflect the actual extension seen if that's the case?
UPDATE: It seems that .tbd is the successor to .dylib according to http://stackoverflow.com/questions/31450690/why-xcode-7-shows-tbd-instead-of-dylib

Hi @rajohns08, thanks for pointing that out. I just tested it and updated the readme appropriately.

25 march 2016, issue did closed
Work without cocoapods according [official Mixpanel guide] (https://mixpanel.com/blog/2014/09/30/integrating-the-mixpanel-ios-sdk-without-cocoapods)
and issue was solve after adding libicucore.tbd


Metric timestamp was being lost due to over-writing by end_time

Hi @WilliamDutton, we have found your signature in our records, but it seems like you have signed with a different e-mail than the one used in yout Git commit. Can you please add both of these e-mails into your Github profile (they can be hidden), so we can match your e-mails to your Github profile?

Hi @WilliamDutton, we have found your signature in our records, but it seems like you have signed with a different e-mail than the one used in yout Git commit. Can you please add both of these e-mails into your Github profile (they can be hidden), so we can match your e-mails to your Github profile?


Request for good first issue

Hi @tiendc ,
I hope you are doing well. I found this project fascinating and would like to contribute more.
Can you raise some potential good first issue so I can start contributing towards them?
Thanks! ☺

@RahulDubey391 thanks a lot for your offer. I don't have a plan for new features as this lib is made based on the requirements of the projects I've been working on. If you have use case and need more functions, feel free to create issue and pull request. Adding new UT for complicated functions can also be an option.


Can't list casks & formulae together

Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you repeatedly fail to use the issue template, we will block you from ever submitting issues to Homebrew again.

[x] ran brew update and can still reproduce the problem?
[x] ran brew doctor, fixed all issues and can still reproduce the problem?
[x] ran brew config and brew doctor and included their output with your issue?

What you were trying to do (and why)
See all installed formulae & casks listed together. I expected this to work because it seems that brew outdated & brew upgrade were recently updated to include both formulae & casks by default, with flags --casks and --formulae to deal with either independently.
What happened (include command output)

Command output

Error: Options --cask and --formula are mutually exclusive.

What you expected to happen
brew ls/brew list would show all casks & formulae by default, unless passed --casks or --formulae to restrict output to one type.
Step-by-step reproduction instructions (by running brew commands)

Run brew ls --casks
Run brew ls
Note that 2. doesn't include the output from 1.
Run brew ls --casks --formulae
Note output is Error: Options --cask and --formula are mutually exclusive.

Output of brew config and brew doctor commands

$ brew doctor
Your system is ready to brew.

$ brew config
HOMEBREW_VERSION: 2.4.12-206-g85c0612
ORIGIN: https://github.com/Homebrew/brew
HEAD: 85c061295d61c2664c1480dd7c45e4fc398a3ef8
Last commit: 88 minutes ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 880c7afb21824f1b3f26d973804ee510881dd7af
Core tap last commit: 60 minutes ago
Core tap branch: master
HOMEBREW_PREFIX: /usr/local
HOMEBREW_CASK_OPTS: []
HOMEBREW_EDITOR: subl -w
HOMEBREW_GITHUB_API_TOKEN: set
HOMEBREW_MAKE_JOBS: 4
CPU: quad-core 64-bit skylake
Homebrew Ruby: 2.6.3 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
Clang: 11.0 build 1103
Git: 2.28.0 => /usr/local/bin/git
Curl: 7.64.1 => /usr/bin/curl
macOS: 10.15.6-x86_64
CLT: 1<IP_ADDRESS>
Xcode: 11.5

This is intentional and cannot be changed as it would break backwards compatibility.

@MikeMcQuaid I don't understand why default behavior for brew outdated & brew upgrade were changed, but this one can't be.
If we can't break backwards compatibility for this command, wouldn't it at least be reasonable to have an option that prints all formulae & casks together with a single command?

I don't understand why default behavior for brew outdated & brew upgrade were changed, but this one can't be.

Their output is not consumed by other tools.

If we can't break backwards compatibility for this command, wouldn't it at least be reasonable to have an option that prints all formulae & casks together with a single command?

Yes. We'd consider a PR that made --cask and --formula not conflict.


Wip/6.0 Verify 'column quoting'

https://trello.com/c/3o4BJuo8/102-verify-support-for-quoted-columns
One bug flushed out regarding the HQL 'setClause' ANTLR grammar.

@dreab8 would appreciate it if you take a look. Seems easy PR with easy bug fix.


Improvements and additional tooling

@SleeplessByte Is there a reason we can't have type:module and main points to ES source?  I don't care so much personally but my tooling is modular and it's not happy unless it's inside a module.

Using type module says that main is module. Is that what you want?

Eh. Ok. Let me answer your question at face value:

There is no reason why we can't have type: "module" and make main point to an ES source.
If you want main (the nodejs standard) to be an es source, then this must be done.
If you want module (the webpack standard) to be an es source and use main as cjs source, then don't add it.
If you want to support as many possible tools, with the one requirement that everything is es modules, add type: "module", and make main and module point to the same .es source. Completely drop the cjs bundle.

I would go for the last option :)

I'm so over this ES vs CJS drama. :)  Is Exercism v3 all ESM stuff?

Like this?

Only thing left I would recommend is to:
rm -rf node_modules
npm install --production
npm link

And then create a new folder somewhere with a package.json and use npm link @exercism/codemirror-lang-wren. You can now use your local package. See if the imports work as expected. Note: for VS Code you want to open the action pain and restart the TS server.

See if the imports work as expected.

I'm not doing it the way you mentioned, but I plan to google that now.  I've done this with file in the past and letting npm symlink in the package to another project.   I presume npm link does similar if not exactly the same.
I'm only trying the ES version, but that's working for me as expected.

npm install --production
That breaks because lezer-generator is a development dependency.  dist does not require the generator...
Am I thinking about this wrong?

Eh, ok. You can use --ignore-scripts.
So:
npm install
# this will install with dev, build the output

rm -rf node_modules
npm install --production --ignore-scripts
npm link


Websocket API

Hey,
I needed access to the Bittrex real-time API for market changes/tickers, etc - I couldn't find any nodejs library that supported it and their API documentation doesn't even mention it, but it does exist. Anyway, I needed to figure out how to get access and I thought you could use my findings.
The code isn't implementable as-is but it shouldn't take much more work for you to integrate it. Hope you can make use of it - if not, that's OK too.
npm install --save signalr-client jsonic

var jsonic = require('jsonic');
var signalR = require('signalr-client');

var client = new signalR.client(
  "wss://socket.bittrex.com/signalr",
  ['CoreHub']
);

/**
 * Example data:
 * {
    "MarketName": "BTC-ANS",
    "High": 0.001525,
    "Low": 0.00075611,
    "Volume": 2296953.44814714,
    "Last": 0.0014202,
    "BaseVolume": 2638.26486332,
    "TimeStamp": "2017-06-17T12:04:08.53",
    "Bid": 0.0014361,
    "Ask": 0.0014369,
    "OpenBuyOrders": 1022,
    "OpenSellOrders": 514,
    "PrevDay": 0.000769,
    "Created": "2016-10-26T01:28:31.96"
  }
 */
function tickerUpdate(pairCode, data) {
  console.log('Ticker Update for '+pairCode, data);
};

/**
 * Example data:
 *  {
      "MarketName": "BTC-ETH",
      "Nounce": 661678,
      "Buys": [
        {
          "Type": 1,
          "Rate": 0.0886264,
          "Quantity": 0
        },
        {
          "Type": 1,
          "Rate": 0.08743581,
          "Quantity": 0
        },
        {
          "Type": 1,
          "Rate": 0.08616772,
          "Quantity": 0
        },
        {
          "Type": 1,
          "Rate": 0.08577644,
          "Quantity": 0
        },
        {
          "Type": 1,
          "Rate": 0.08576411,
          "Quantity": 0
        }
      ],
      "Sells": [
        {
          "Type": 0,
          "Rate": 0.13989973,
          "Quantity": 40
        },
        {
          "Type": 2,
          "Rate": 0.1399704,
          "Quantity": 0.10334389
        }
      ],
      "Fills": []
      }
    }
 */
function marketUpdate(pairCode, data) {
  console.log('Market Update for '+pairCode, data);
}

function subscribeToMarkets() {
  [ 'BTC-ETH', 'BTC-SC' ].forEach(function(market) {
    client.call('CoreHub', 'SubscribeToExchangeDeltas', market).done(function(err, result) {
      if (result === true) {
        console.log('Subscribed to ' + market);
      }
    });
  });
};

client.serviceHandlers = {
  bound: function() {
    console.log("Websocket bound");
  },
  connectFailed: function(error) {
    console.log("Websocket connectFailed: ", error);
  },
  connected: function(connection) {
    subscribeToMarkets();
    console.log("Websocket connected");
  },
  disconnected: function() {
    console.log("Websocket disconnected");
  },
  onerror: function (error) {
    console.log("Websocket onerror: ", error);
  },
  messageReceived: function (message) {
    try {
      var data = jsonic(message.utf8Data);
      /**
       * updateSummaryState example:
       * {
          "C": "...",
          "M": [
            {
              "H": "CoreHub",
              "M": "updateSummaryState",
              "A": [
                {
                  "Nounce": 254683,
                  "Deltas": [
                    {
                      "MarketName": "BTC-ANS",
                      "High": 0.001525,
                      "Low": 0.00075611,
                      "Volume": 2296953.44814714,
                      "Last": 0.0014202,
                      "BaseVolume": 2638.26486332,
                      "TimeStamp": "2017-06-17T12:04:08.53",
                      "Bid": 0.0014361,
                      "Ask": 0.0014369,
                      "OpenBuyOrders": 1022,
                      "OpenSellOrders": 514,
                      "PrevDay": 0.000769,
                      "Created": "2016-10-26T01:28:31.96"
                    }, .. ]
                }
              ]
            }
          ]
       }
       * updateExchangeState example:
       * {
          "C": "...",
          "M": [
            {
              "H": "CoreHub",
              "M": "updateExchangeState",
              "A": [
                {
                  "MarketName": "BTC-ETH",
                  "Nounce": 661678,
                  "Buys": [
                    {
                      "Type": 1,
                      "Rate": 0.0886264,
                      "Quantity": 0
                    },
                    {
                      "Type": 1,
                      "Rate": 0.08743581,
                      "Quantity": 0
                    },
                    {
                      "Type": 1,
                      "Rate": 0.08616772,
                      "Quantity": 0
                    },
                    {
                      "Type": 1,
                      "Rate": 0.08577644,
                      "Quantity": 0
                    },
                    {
                      "Type": 1,
                      "Rate": 0.08576411,
                      "Quantity": 0
                    }
                  ],
                  "Sells": [
                    {
                      "Type": 0,
                      "Rate": 0.13989973,
                      "Quantity": 40
                    },
                    {
                      "Type": 2,
                      "Rate": 0.1399704,
                      "Quantity": 0.10334389
                    }
                  ],
                  "Fills": []
                }
              ]
            }
          ]
        }
       */
      if (data && data.M) {
        data.M.forEach(function(M) {
          switch (M.M) {
            case 'updateSummaryState':
              M.A.forEach(function(data) {
                data.Deltas.forEach(function(marketsDelta) {
                  tickerUpdate(marketsDelta.MarketName, marketsDelta);
                });
              });
              break;
            case 'updateExchangeState':
              M.A.forEach(function(data) {
                marketUpdate(data.MarketName, data);
              });
              break;
          }
        })
      } else {
        console.log('Unhandled data', data);
      }
    } catch (e) {
      console.error(e);
    }
    return false;
  },
  bindingError: function (error) {
    console.log("Websocket bindingError: ", error);
  },
  connectionLost: function (error) {
    console.log("Connection Lost: ", error);
  },
  reconnecting: function (retry) {
    console.log("Websocket Retrying: ", retry);
    // change to false to stop retrying
    return true;
  }
};

great stuff, thanks a lot!
i implemented this right away: https://github.com/n0mad01/node.bittrex.api#websockets
(and also mentioned you as a contributor https://github.com/n0mad01/node.bittrex.api#contributors)

nice! no probs

@dparlevliet Hi, how did you know that Websocket connection is possible with Bittrex? They don't mention it in their docs https://bittrex.com/home/api .

I didn't see it mentioned but I knew they had to update the order books somehow and only an insane team (Liqui.io) wouldn't use websockets for that.

If you're asking specifically how I found it then all I did was open the Developer tools in Chrome and look at the WS tab in the Network section. The harder part was figuring out what type of sockets they were using, but thankfully there were some clues in the connection string.

Oh, I got it. Thank you.
I thought that they may have hidden docs or info was leaked)

I'm thinking of doing such kind of investigation for another exchange. Is there a way to see actual data sent through websocket connection in Developer tools? On WS tab I just see different connections. 'Some connections' -> Frames is empty.

That's definitely the right area - but I can't tell you exactly if what you're seeing is right or wrong.

I'm seeing this.

Uhm, I think the content box is overlapping the frames box because your resolution is so small. You should be able to grab and drag the black line that's going under the "Enter regex... " - pull it down and you should see the frames.

@alexvelickiy please, could you take this questions to a more specific board?
there is plenty material out there to learn how to inspect websocket traffic.
use the issues here only for specific questions, thanks.

@n0mad01 sorry, sure I could.

would be nice to add documentation to the WebSocket options (including orders like buy/sell)

@Amindv1 I guess that 0 means that the order is deleted.
I thought that type 0 = sell and type 1 = buy, but WTF is type=2 !?

Rather, I believe that zero means adding and one eliminates (if you realize the records with zero in the amount comes zero as well), since the categories already say "buy / sell", I think it works just like Binance where they are grouped amounts by price.
Example:
{Type: 0, Rate: 0.07247907, Quantity: 0.01061322}
The item "0.07247907" with 0.01061322 of stock is added.
Y
{Type: 1, Rate: 0.07251045, Quantity: 0},
The item "0.07251045" is deleted since the quantity is zero.
regards

On the contrary, I think zero means adding and one eliminates (if you realize that records with zero in the amount are also zero), since the categories already say "buy / sell", I think it works just like Binance, where amounts are grouped by price
Example:
{Type: 0, Rate: 0.07247907, Quantity: 0.01061322}
The item "0.07247907" is added with 0.01061322 of the shares.
Y
{Type: 1, Rate: 0.07251045, Quantity: 0},
The article "0.07251045" is deleted since the quantity is zero.
regards

Query, does anyone know if Bittrex has a Snapshot to preload the database before connecting to the Websocket?


tag_names is not idempotent

hi,
This is about the SSL monitor, but I guess the implementation would be the same everywhere where there is a tag_ids/tag_names.
https://registry.terraform.io/providers/site24x7/site24x7/latest/docs/resources/ssl_monitor
So I associate tags to this monitor and I apply, they apply correctly.
But on the next run, the tag is detected as new. The same applies to user_group_names.
  ~ resource "site24x7_ssl_monitor" "ssl_monitor" {
        id                        = "396627000019682039"
      ~ tag_ids                   = [
          - "396627000019682015",
        ]
      ~ user_group_ids            = [
          - "396627000019683052",
          - "396627000019675047",
          - "396627000019681043",
        ]```

So this is probably related with the fact that I have specified default values.
This is my setup, I do have an empty tag_ids - that should be ignored for being empty.
  tag_ids                   = []
  tag_names             = ["TAG_A"]

Right?


Notebook's kernelspec metadata not updated when executing cells with a different kernel

create a notebook and save it.
Run a cell with a kernel.
Change the kernel and run another cell
reopen the notebook as plain text and note the kernel spec in the metadata.
metadata should specify the kernel that was most recently used, but it specifies the original.
Seems to repro with both local and remote kernels

impacts #9685 since we're not getting the correct suggestion

yes, the kernelspec updates in stable as expected

This is actually a bug in the new metadata API. Transferring to vscode core.


feature request - modify/alter options for store procedures etc.

SQL Operations Studio Version:
023.6

Steps to Reproduce:
Add option to modify/alter store procedures etc.

this feature is added to the next release


Blog index queries take long enough to trigger Gatsby warnings

Split off from #1727
My hypothesis copied from there:

I think the queries that are warned about for taking too long are taking a while because they're fetching a huge amount of data even when the image transforms are cached.
For example, take an index page has 8 thumbnails with our current setup that gives each thumbnail two sourcesets with 4 images: that means gatsby-transformer-sharp, even with working cache, reads the cache and the FS to provide 64 image paths along with any extra blur-up data for each image. While this certainly isn't long as it takes when the images are actually resized, it's still much more intensive than the average query and enough to make them take long enough to get flagged by Gatsby.
the problem isn't very scary with cache, it just takes comparatively much longer than the average query that grabs some basic page content from a markdown file that's already been read to memory. That's why the warning only occurs in the Heroku containers and not local builds- the difference in computing power is enough to drive the time below Gatsby's warning threshold. The warning is much scarier than the thing it's warning about.

While worth looking at, I believe this takes less priority than the other split issue #1732, as that one actually causes the build to multiply in length when it happens.
I believe, the first step to fixing this would be trying to use one source set in blog index thumbnails instead of two. I haven't tested this yet, though.

@rogermparent , are we good to close this now that we are at Gatsy 4? I'm not seeing any blog warnings anymore 🤔

I think I still see them on Heroku during full rebuilds, but I don't think there's much more we can do here- at least not enough to warrant an Issue. I've already done the thumbnail change earlier, so at least all action items in this ticket have been taken.


RAM issue with AnvilRecipeWrappers.

Hello!
Looks like the AnvilRecipeWrappers registered here consume a plenty (about 800M) of RAM.
To check it out, I've built a custom version of EnderIO with anvil recipes registration removed and then captured heap dumps of Minecraft with standard and custom versions of EnderIO.
Standard EnderIO:

EnderIO without anvil recipes registration:

Affected Versions

EnderIO: 5.0.39
EnderCore: 0.5.44
Minecraft: 1.12.2
Forge: <IP_ADDRESS>96
No SpongeForge
No Optifine
Single Player

see https://github.com/SleepyTrousers/EnderIO/tree/feature/jei_improvements


Update noble-bls to improve security

@iancoleman make sure to build it please

I'm in the process now, but am failing this test:
masterPrv = new Uint8Array([25, 207, 229, 88, 53, 10, 115, 205, 117, 155, 230, 16, 158, 148, 203, 202, 99, 4, 146, 214, 139, 43, 228, 28, 218, 233, 237, 186, 78, 16, 3, 80]);
childPrv = libs.blskeygen.deriveChild(masterPrv, 13);
childPrvLen = childPrv.length; // expect 32, get 31

@iancoleman fixing in 1 sec

@iancoleman fixed in<EMAIL_ADDRESS>

Thanks! I'll update the package.json to the latest version.

 Is this expected?
Looks like this line might be causing grief:
https://github.com/paulmillr/bls12-381-keygen/commit/964c5443ac2c8a1b1d0e826398e58013ff9ac7c2#diff-e727e4bdf3657fd1d798edcd6b099d6e092f8573cba266154583a746bba0f346R27
Specifically I'm testing for the eip2333 test cases, see https://github.com/iancoleman/eip2333-tool/blob/3af02cd16d3f65976a40eedac9280d66aafea489/tests/spec/tests.js#L157-L185

@iancoleman sorry for my nonsense, i've copy-pasted LE function instead of BE. Fixed now.

bls-keygen actually has test vectors, they were failing after that change, i've pushed pkg without testing it first.


Displaying text when bar too small

When the bar is too small (<40) it would be cool to left align the value text into the bar container remaining space.
Example:

I did it by overriding this method (to give an idea, not worth PR, a little bit dirty I think)
- (void)drawRightAlignedLabelInRect:(CGRect)rect {
    if (rect.size.width > 40 || (self.frame.size.width - rect.size.width) > 40) {
        UILabel *label = [[UILabel alloc] initWithFrame:rect];
        label.adjustsFontSizeToFitWidth = YES;
        label.backgroundColor = [UIColor clearColor];
        label.text = self.progressTextOverride ? self.progressTextOverride :
            [NSString<EMAIL_ADDRESS>self.progress*100];
        label.font = [UIFont boldSystemFontOfSize:17-self.progressInset.floatValue*1.75];
        GLfloat posx;
        GLfloat width;
        BOOL baseLabelisBlack;
        if (rect.size.width > 40) {
            posx = rect.origin.x + 6;
            width = rect.size.width - 12;
            baseLabelisBlack = [self.color isLighterColor];
            label.textAlignment = NSTextAlignmentRight;
        } else {
            posx = rect.size.width + 6;
            width = self.frame.size.width - rect.size.width - 12;
            baseLabelisBlack = ![self.color isLighterColor];
            label.textAlignment = NSTextAlignmentLeft;
        }
        UIColor *baseLabelColor = baseLabelisBlack ?
            [UIColor blackColor] : [UIColor whiteColor];
        label.textColor = [baseLabelColor colorWithAlphaComponent:0.6];
        [label drawTextInRect:CGRectMake(posx, rect.origin.y, width, rect.size.height)];
    }
}

To be renamed to something like
- (void)drawAlignedLabelForBar:(CGRect)rect;

Current implementation jumps the label as soon as the progress bar reaches a certain amount (and frankly truncates the text sometimes), I guess a better implementation could be like this:
if progress bar doesn't have enough space for text, put the text somewhere anchored to the left (or right based on alignment) of the progressive, let the progress bar fill it under the text, until it reaches to an amount big enough to contain the text, then the text will latch on to the progress bar's like before and moves with progress bar's end.


Fix to allow compilation with MinGW

MinGW is case sensitive when including files, and it ships the main Windows header as windows.h.
This patch converts every reference of the Windows header to lower case.
I ran tests on both MinGW and MSVC successfully.
I used MinGW version 9.2.0 and MSVC 2017 version 15.9.8, in a Windows 10 virtual machine.
Each compiler generated binaries targeting 32 and 64 bits platforms in Release mode.

Well, I'd say lower-casing that header name is a very unfortunate design choice on the side of the MinGW people. The official MS headers clearly have it upper-cased and changing it to lower-case will likely break builds on MSVC installations where the Windows Kit resides on a case-sensitive (network-)drive.
However, since network drives in Windows are still pretty much completely broken, I guess it's rather safe to assume that nobody that is interested in preserving their own sanity would do that anyway, especially case-sensitive, so I'd be okay with merging that anyway.
What do you think @flobernd? Feel free to immediately merge if you agree.

I'd say lower-casing that header name is a very unfortunate design choice on the side of the MinGW people

This.
Merged anyways. Should not cause any problems on Windows I think


Stop Travis builds with success if no secrets were injected

Travis doesn't inject secrets for outside builds, but this can be confusing. Travis should exit with success for PRs that aren't from people with push access. See https://travis-ci.org/CiscoCloud/mantl/jobs/112976692

This will necessitate a move to a real install script, which @sehqlr is working on as part of a new Docker-based deploy process. Could this change be folded into that branch, Sam?

Yeah, I think that would be good. I would like to port the logic to something not bash. So, I'll close this branch, but I won't delete the branch.


Guide on how to upgrade the CRD of the experimental channel from v1.0 to v1.1

What would you like to be added:
Due to the removal of the v1alpha2 version of backendtlspolicy in CRD version v1.1, this will result in encountering errors during the upgrade.
I hope to add a guide to instruct users on how to solve the problem.
Why this is needed:
(⎈|steep-abyss:default)➜  ~ kubectl apply -f https://github.com/kubernetes-sigs/gateway-api/releases/download/v1.0.0/experimental-install.yaml
customresourcedefinition.apiextensions.k8s.io/backendtlspolicies.gateway.networking.k8s.io created
customresourcedefinition.apiextensions.k8s.io/gatewayclasses.gateway.networking.k8s.io created
customresourcedefinition.apiextensions.k8s.io/gateways.gateway.networking.k8s.io created
customresourcedefinition.apiextensions.k8s.io/grpcroutes.gateway.networking.k8s.io created
customresourcedefinition.apiextensions.k8s.io/httproutes.gateway.networking.k8s.io created
customresourcedefinition.apiextensions.k8s.io/referencegrants.gateway.networking.k8s.io created
customresourcedefinition.apiextensions.k8s.io/tcproutes.gateway.networking.k8s.io created
customresourcedefinition.apiextensions.k8s.io/tlsroutes.gateway.networking.k8s.io created
customresourcedefinition.apiextensions.k8s.io/udproutes.gateway.networking.k8s.io created
(⎈|steep-abyss:default)➜  ~ kubectl apply -f https://github.com/kubernetes-sigs/gateway-api/releases/download/v1.1.0/experimental-install.yaml
customresourcedefinition.apiextensions.k8s.io/backendlbpolicies.gateway.networking.k8s.io created
customresourcedefinition.apiextensions.k8s.io/gatewayclasses.gateway.networking.k8s.io configured
customresourcedefinition.apiextensions.k8s.io/gateways.gateway.networking.k8s.io configured
customresourcedefinition.apiextensions.k8s.io/grpcroutes.gateway.networking.k8s.io configured
customresourcedefinition.apiextensions.k8s.io/httproutes.gateway.networking.k8s.io configured
customresourcedefinition.apiextensions.k8s.io/referencegrants.gateway.networking.k8s.io configured
customresourcedefinition.apiextensions.k8s.io/tcproutes.gateway.networking.k8s.io configured
customresourcedefinition.apiextensions.k8s.io/tlsroutes.gateway.networking.k8s.io configured
customresourcedefinition.apiextensions.k8s.io/udproutes.gateway.networking.k8s.io configured
The CustomResourceDefinition "backendtlspolicies.gateway.networking.k8s.io" is invalid: status.storedVersions[0]: Invalid value: "v1alpha2": must appear in spec.versions

          Thanks for reporting this, I definitely get that this behavior is confusing. This is actually intentional though - we made breaking changes (field renames) to BackendTLSPolicy in v1.1. That resulted in us bumping the version to v1alpha3, and dropping v1alpha2 since we do not want to maintain a conversion webhook. I added more information about this transition in https://github.com/kubernetes-sigs/gateway-api/pull/3084.

Originally posted by @robscott in https://github.com/kubernetes-sigs/gateway-api/issues/3086#issuecomment-2115798435

/assign

Hi @tao12345666333 can you please let me know which one is the process to fix the error?

@ilbarone87 We need to add content for version v1alpha2 manually.  I will submit a PR later.
/remove-lifecycle stale

https://gateway-api.sigs.k8s.io/guides/?h=v1.2#backendtlspolicy
I believe the relevant content has already been described in the current document.
/close


Speed up BinningVectorizer

First, thanks for the great library!
I was playing with methods for the DRIAMS-A part of the data and noticed that BinningVectorizer was using a lot of time. Specifically, I load all years, species and Ciprofloxacin + Penicillin:
explorer = DRIAMSDatasetExplorer(DRIAMS_ROOT)

driams_dataset = load_driams_dataset(
            explorer.root,
            'DRIAMS-A',
            ['2015', '2016', '2017', '2018'],
            '*',
            ['Ciprofloxacin', 'Penicillin'],
            handle_missing_resistance_measurements='remove_if_all_missing',
            id_suffix='strat',
)
len(driams_dataset.X)  # 33415

Using the BinningVectorizer with 200 bins then takes 9 seconds for fitting and over 6 minutes for the transform:
bv = BinningVectorizer(200, min_bin=2000, max_bin=20000)
bv.fit(driams_dataset.X)               # 8.9s
Xold = bv.transform(driams_dataset.X)  # 6m 10.0s

The time also seems to worsen as I increase the bin count.
I tried to optimize the code for fit and transform and I can now get results significantly faster:
nbv = NewBinningVectorizer(200, min_bin=2000, max_bin=20000)
nbv.fit(driams_dataset.X)               # 2.1s
Xnew = nbv.transform(driams_dataset.X)  # 20.8s

The results are also the same (up to small rounding errors):
np.all(mbv.bin_edges_ == bv.bin_edges_)         # True
np.abs(Xnew - Xold).max()                       # abs. error = 4.104762319975808e-15
np.abs(Xnew - Xold).max() / np.abs(Xold).max()  # rel. error = 1.3664372704227028e-14

It also passes the test in tests/vectorization/test_binning_vectorizer.py.
I haven't thoroughly checked the performance for smaller datasets. However, with the smaller example in the README (1520 samples) the new code is still faster.
Hopefully, this is useful :) If anything looks weird or should be changed please let me know! I tried to explain each change in their respective commits.

This looks amazing! Thanks so much for your contribution.


Payden_

Hello # @

Not a valid issue report. Going to close this as a sample of a test message.
#please-close


Stacks with same name, after a rename, merge services in UI link selection

Rancher versions: 1.6
Two problems:

Add Stack forces uniqueness constraint from UI while Edit does not

foo.bar is the name of Stack A.  When I edit Stack B and change its name tofoo.bar, UI displays services across Stack A and Stack B under one grouping foo.bar, in some selection dropdowns, making it feel like the services belong to a single stack.

Notice from the above screenshots, dokuwiki-server and MyTopService are shown under a single stack

@kinarashah we need to fix edit, do not allow renaming to something that already exists. Code needs to be done in StackValidationFilter

@alena1108 @kinarashah
I reckon the fix will ensure uniqueness in stack name in the edit scenario.
But if there are existing duplicates out there then the resolution for the user is to simply rename such stack name(s) to a unique name to address the dropdown issue?

@jambajaar yes, the dropdown issue gets solved with the fix.
For test, I renamed test to test2 -

The UI groups the options by name because it is what is displayed and it's supposed to be unique.  If there is a case where it's actually ambiguous what service you're choosing, you have far bigger real problems like overlapping DNS entries, ambiguous access by name in metadata or the /r proxy, etc.

Not sure how/where can I make additional change for existing duplicates?

@kinarashah You can't (and don't need to)

Tested with latest build from v1.6 branch.
Not allowed to edit stack name to a name that is already associated with existing stack in the same environment. User is presented with - Validation failed in API: name is not unique.

Looks like this fix is available in 1.6.11


Resolve Crash during navigating course units

Description
LEARNER-4792

Remove transcriptlistView GlobalLayoutListener otherwise crash.

Thanks for the pull request, @farhan-arshad-dev! I've created OSPR-2705 to keep track of it in JIRA. JIRA is a place for product owners to prioritize feature reviews by the engineering development teams.
Feel free to add as much of the following information to the ticket:

supporting documentation
edx-code email threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will still be done via the GitHub pull request interface. As a reminder, our process documentation is here.
We can't start reviewing your pull request until you've submitted a     signed contributor agreement     or indicated your institutional affiliation. Please see the CONTRIBUTING     file for more information.

@farhan-arshad-dev Thank you for your contribution. @edx/android-devs Can you give this a look?

@natabene getting 404 error.

@farhan-arshad-dev 🎉 Your pull request was merged!
Please take a moment to answer a two question survey so we can improve your experience in the future.


Correct section title of middleware.md

The same reason as item 2 of PR #4023

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

@wplong11 Thank you for the contribution! Your change will go live within the next 24 hours.


Fairseq error

I encountered another error when using infer_file. Seems to be fairseq issue.
Traceback (most recent call last):
  File "/home/chupz22/Desktop/Desktop_Files/Programs/VSCodiumProjects/ChatBot/test_rvc.py", line 1, in <module>
    from rvc_python.infer import infer_file
  File "/home/chupz22/Desktop/Desktop_Files/Programs/VSCodiumProjects/ChatBot/venv/lib/python3.11/site-packages/rvc_python/__init__.py", line 1, in <module>
    import rvc_python.infer
  File "/home/chupz22/Desktop/Desktop_Files/Programs/VSCodiumProjects/ChatBot/venv/lib/python3.11/site-packages/rvc_python/infer.py", line 2, in <module>
    from rvc_python.modules.vc.modules import VC
  File "/home/chupz22/Desktop/Desktop_Files/Programs/VSCodiumProjects/ChatBot/venv/lib/python3.11/site-packages/rvc_python/modules/vc/modules.py", line 19, in <module>
    from rvc_python.modules.vc.utils import *
  File "/home/chupz22/Desktop/Desktop_Files/Programs/VSCodiumProjects/ChatBot/venv/lib/python3.11/site-packages/rvc_python/modules/vc/utils.py", line 3, in <module>
    from fairseq import checkpoint_utils
  File "/home/chupz22/Desktop/Desktop_Files/Programs/VSCodiumProjects/ChatBot/venv/lib/python3.11/site-packages/fairseq/__init__.py", line 20, in <module>
    from fairseq.distributed import utils as distributed_utils
  File "/home/chupz22/Desktop/Desktop_Files/Programs/VSCodiumProjects/ChatBot/venv/lib/python3.11/site-packages/fairseq/distributed/__init__.py", line 7, in <module>
    from .fully_sharded_data_parallel import (
  File "/home/chupz22/Desktop/Desktop_Files/Programs/VSCodiumProjects/ChatBot/venv/lib/python3.11/site-packages/fairseq/distributed/fully_sharded_data_parallel.py", line 10, in <module>
    from fairseq.dataclass.configs import DistributedTrainingConfig
  File "/home/chupz22/Desktop/Desktop_Files/Programs/VSCodiumProjects/ChatBot/venv/lib/python3.11/site-packages/fairseq/dataclass/__init__.py", line 6, in <module>
    from .configs import FairseqDataclass
  File "/home/chupz22/Desktop/Desktop_Files/Programs/VSCodiumProjects/ChatBot/venv/lib/python3.11/site-packages/fairseq/dataclass/configs.py", line 1104, in <module>
    @dataclass
     ^^^^^^^^^
  File "/usr/lib/python3.11/dataclasses.py", line 1232, in dataclass
    return wrap(cls)
           ^^^^^^^^^
  File "/usr/lib/python3.11/dataclasses.py", line 1222, in wrap
    return _process_class(cls, init, repr, eq, order, unsafe_hash,
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/dataclasses.py", line 958, in _process_class
    cls_fields.append(_get_field(cls, name, type, kw_only))
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/dataclasses.py", line 815, in _get_field
    raise ValueError(f'mutable default {type(f.default)} for field '
ValueError: mutable default <class 'fairseq.dataclass.configs.CommonConfig'> for field common is not allowed: use default_factory

Sorted. Python version should be <3.11. I resintalled python 3.10 and now it is working. 3.11 onwards will not work. Thanks


Support HTTP 409 for iterations with similar name/hierarchy

Similar to https://github.com/fabric8-services/fabric8-wit/pull/1509

if an  iteration with the same name is created in the same hierarchy , return a 409 instead of a 400
add tests at storage & controller level

fixes #1512

@sbose78 snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1513-1

@sbose78 snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1513-3

@sbose78 snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1513-4


Don't know what problem this repo is aiming to solve based on readme?

Dynamically generate SQL query conditions


Add a new function

You cannot add a function where every second that passes will unlock a feat randomly, so it will look more real.

I'm not interested in coding such feature, because I don't care if unlocked achievements look "more real" or not. You are welcome to make a fork of plugin and implement it yourself. Also, if you want, I can create a personal version of plugin just for you for about $200.


Improve coverage for qasm3 converter

Fixes #65
Changes

Add tests for different qasm3 files to bring coverage up to 95%

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

Project coverage is 95.26%. Comparing base (2a5f3da) to head (f532fb3).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #74      +/-   ##
==========================================
+ Coverage   92.47%   95.26%   +2.78%
==========================================
  Files          19       19
  Lines         851      845       -6
==========================================
+ Hits          787      805      +18
+ Misses         64       40      -24

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


Various features (type awareness, linting, etc) stop working randomly

Vue - Official extension or vue-tsc version
Official
VSCode version
1.93.1
Vue version
3.5.5
TypeScript version
5.5.4
System Info
System:
    OS: macOS 14.5
    CPU: (12) arm64 Apple M2 Max
    Memory: 32.53 GB / 96.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.14.0 - ~/.local/state/fnm_multishells/66548_1726586633705/bin/node
    npm: 10.8.3 - ~/.local/state/fnm_multishells/66548_1726586633705/bin/npm
    pnpm: 9.10.0 - ~/.local/state/fnm_multishells/66548_1726586633705/bin/pnpm
    bun: 1.1.22 - ~/.bun/bin/bun
  Browsers:
    Chrome: 128.0.6613.138
    Safari: 17.5

package.json dependencies
{
  "dependencies": {
    "@aws-amplify/geo": "^3.0.47",
    "@aws-sdk/client-lambda": "^3.651.1",
    "@aws-sdk/client-s3": "^3.651.1",
    "@aws-sdk/lib-storage": "^3.651.1",
    "@casl/ability": "^6.7.1",
    "@casl/vue": "^2.2.2",
    "@chargebee/chargebee-js-vue-wrapper": "^0.3.2",
    "@codemirror/lang-html": "^6.4.9",
    "@mdi/font": "^7.4.47",
    "@sentry/tracing": "^7.114.0",
    "@sentry/vue": "^8.30.0",
    "@tanstack/vue-query": "^5.56.2",
    "@tiptap/extension-link": "^2.6.6",
    "@tiptap/extension-text-align": "^2.6.6",
    "@tiptap/extension-underline": "^2.6.6",
    "@tiptap/pm": "^2.6.6",
    "@tiptap/starter-kit": "^2.6.6",
    "@tiptap/vue-3": "^2.6.6",
    "@vuepic/vue-datepicker": "^9.0.3",
    "aws-amplify": "^6.6.0",
    "axios": "^1.7.7",
    "axios-retry": "^4.5.0",
    "braintree-web": "^3.107.1",
    "camelcase-keys": "^9.1.3",
    "canvas-confetti": "^1.9.3",
    "change-case": "^5.4.4",
    "chart.js": "^4.4.4",
    "codemirror": "^6.0.1",
    "colord": "^2.9.3",
    "date-fns": "^3.6.0",
    "deep-object-diff": "^1.1.9",
    "filesize": "^10.1.6",
    "gridstack": "^10.3.1",
    "hls.js": "^1.5.15",
    "immutable": "5.0.0-beta.4",
    "js-base64": "^3.7.7",
    "js-beautify": "^1.15.1",
    "keycode": "^2.2.1",
    "lodash-es": "^4.17.21",
    "mitt": "^3.0.1",
    "node-ray": "^2.1.2",
    "peaks.js": "github:reelcrafter/peaks.js",
    "pinia": "^2.2.2",
    "qr-code-styling": "1.6.0-rc.1",
    "snakecase-keys": "^8.0.1",
    "sortablejs": "^1.15.3",
    "tinycolor2": "^1.6.0",
    "uuid": "^10.0.0",
    "vidstack": "^1.12.10",
    "vue": "^3.5.5",
    "vue-chartjs": "^5.3.1",
    "vue-codemirror6": "^1.3.4",
    "vue-router": "^4.4.5",
    "vue3-otp-input": "^0.5.21",
    "vue3-smooth-dnd": "^0.0.6",
    "vuetify": "^3.7.1",
    "waveform-data": "^4.5.0",
    "wavesurfer.js": "^7.8.6",
    "zxcvbn": "^4.4.2"
  },
  "devDependencies": {
    "@aws-amplify/api-graphql": "^4.3.0",
    "@graphql-codegen/cli": "5.0.2",
    "@graphql-codegen/client-preset": "4.3.3",
    "@pinia/testing": "^0.1.5",
    "@playwright/test": "^1.47.1",
    "@rushstack/eslint-patch": "^1.10.4",
    "@sentry/vite-plugin": "^2.22.4",
    "@tsconfig/node20": "^20.1.4",
    "@types/braintree-web": "^3.96.13",
    "@types/canvas-confetti": "^1.6.4",
    "@types/js-beautify": "^1.14.3",
    "@types/jsdom": "^21.1.7",
    "@types/lodash-es": "^4.17.12",
<EMAIL_ADDRESS>"^4.6.9",
    "@types/node": "^20.14.5",
    "@types/sortablejs": "^1.15.8",
    "@types/uuid": "^10.0.0",
    "@types/validator": "^13.12.1",
    "@types/zxcvbn": "^4.4.5",
    "@vitejs/plugin-vue": "^5.1.3",
    "@vue/eslint-config-prettier": "^9.0.0",
    "@vue/eslint-config-typescript": "^13.0.0",
    "@vue/test-utils": "^2.4.6",
    "@vue/tsconfig": "^0.5.1",
    "dotenv": "^16.4.5",
    "eslint": "^8.57.0",
    "eslint-plugin-vue": "^9.28.0",
    "graphql": "^16.9.0",
    "husky": "^9.1.6",
    "jsdom": "^24.1.3",
    "lint-staged": "^15.2.10",
    "npm-run-all": "^4.1.5",
    "prettier": "^3.3.3",
    "rollup-plugin-visualizer": "^5.12.0",
    "sass": "1.78.0",
    "typescript": "~5.6.2",
    "vite": "^5.4.5",
    "vite-plugin-vuetify": "^2.0.4",
    "vitest": "2.1.1",
    "vue-tsc": "^2.1.6",
    "zen-observable-ts": "^1.1.0"
  }
}

Steps to reproduce
Unknown steps (sporadic misbehavior)
What is expected?
Basic features should work:

IntelliSense/autocomplete
Linting/type-checking
Cmd+Click to jump to definition

What is actually happening?
Randomly, these features stop working. To fix it, one needs to reload VS Code.
Link to minimal reproduction
No response
Any additional comments?
No response

Possibly due to tsserver crash, please check the logs and assign more memory.
If still happening please provide a minimal reproduction.

Also, the Add Line Comment and Remove Line Comment randomly stops responding, and then randomly starts working again.
I just enabled TS Server logs. If I find anything of note, I will report back.