flutter test command timeout after 3.10

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
We use fastlane to execute our end-to-end test codes on our CI/CD pipelines.
We have an issue on our side, that we need to retry once every time we build for iOS, but that is not the issue I am trying to report here.
The problem is that after upgrading to flutter 3.10, Fastlane seems to not detect that the command flutter test had stopped, which is causing the pipeline to get stuck for 12 minutes before Fastlane decides to continue (and perform the scriped retry).
[14:41:12]: ▸ 02:34 +0: loading /Users/ec2-user/builds/sWhe5qeA/0/ifood/enterprise/app-customer-benefits/ifoodoffice-benefits-flutter/benefits_app/integration_test/e2e_test/e2e_test.dart
[14:41:12]: ▸ Xcode build done.                                           132.7s
[14:41:12]: ▸ Failed to build iOS app
[14:41:12]: ▸ Swift Compiler Error (Xcode): No such module 'IncogniaBR'
[14:41:12]: ▸ /Users/ec2-user/builds/sWhe5qeA/0/ifood/enterprise/app-customer-benefits/ifoodoffice-benefits-flutter/benefits_app/ios/Runner/AppDelegate.swift:5:7
[14:41:12]: ▸ Could not build the application for the simulator.
[14:41:12]: ▸ 02:34 +0 -1: loading /Users/ec2-user/builds/sWhe5qeA/0/ifood/enterprise/app-customer-benefits/ifoodoffice-benefits-flutter/benefits_app/integration_test/e2e_test/e2e_test.dart [E]
[14:41:12]: ▸ TestDeviceException(Unable to start the app on the device.)
[14:41:12]: ▸ package:flutter_tools/src/test/integration_test_device.dart 63:7  IntegrationTestTestDevice.start
[14:50:38]: ▸ TimeoutException after 0:12:00.000000: Test timed out after 12 minutes.

So, my question is: what could have changed on the flutter test command that it is not launching exceptions anymore for Fastlane to catch?
Expected results
flutter test should launch an exception
Actual results
After 3.10, it seems to not be launching anymore.
Code sample
Code sample
[Paste your code here]

Screenshots or Video
No response
Logs
No response
Flutter Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[!] Flutter (Channel stable, 3.10.5, on Microsoft Windows [versÆo 10.0.22621.1992], locale pt-BR)
! Warning: flutter on your path resolves to C:\Users\shina\fvm\versions\3.10.5\bin\flutter, which is not inside your current Flutter SDK checkout at C:\Users\shina\fvm\default. Consider adding C:\Users\shina\fvm\default\bin
to the front of your path.
! Warning: dart on your path resolves to C:\Users\shina\fvm\versions\3.10.5\bin\dart, which is not inside your current Flutter SDK checkout at C:\Users\shina\fvm\default. Consider adding C:\Users\shina\fvm\default\bin to
the front of your path.
[✓] Windows Version (Installed version of Windows is version 10 or higher)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Chrome - develop for the web
[✗] Visual Studio - develop for Windows
✗ Visual Studio not installed; this is necessary for Windows development.
Download at https://visualstudio.microsoft.com/downloads/.
Please install the "Desktop development with C++" workload, including all of its default components
[✓] Android Studio (version 2022.1)
[✓] IntelliJ IDEA Community Edition (version 2022.2)
[✓] IntelliJ IDEA Community Edition (version 2023.1)
[✓] VS Code (version 1.80.2)
[✓] Connected device (3 available)
[✓] Network resources
! Doctor found issues in 2 categories.

I think the most weird question here is: why flutter test kept running even if Xcode build failed?
Failed to build iOS app

Hi @shinayser
Are you using flavor for your test? If it is, there is a known issue: https://github.com/flutter/flutter/issues/129338 (you can check a workaround at https://github.com/flutter/flutter/issues/129338#issuecomment-1613907710 if it can help). Otherwise, can you share a minimal sample code to reproduce this issue locally if any?

Hey @huycozy we are using no flavors. That is the command we are running:
flutter test -d #{device_name} integration_test/e2e_test/e2e_test.dart
Yesterday we were able to fix our pipeline by running a pod install before the tests running, but this change was necessary only after 3.10 while in 3.8 it worked flawlessly.

So, does it mean you can reproduce this locally? Could you please provide a completed and minimal reproducible code sample that doesn’t include 3rd party plugins or complex production code so that we may verify this?


Add installation script for dunner

It finds the best suited in installation method and archive to download from release page of dunner, and then install it.

@ayushjn20 Let's have non-interactive flags also

Done. Please check @agentmilindu


animation.animate_mode() lists the mode number as the frequency

Before submitting the issue

[X] I have checked for Compatibility issues
[X] I have searched among the existing issues
[X] I am using a Python virtual environment

Description of the bug
Using animation.animate_mode(), the exported video shows the mode number but labels it as the frequency.

It would be great to show both mode number and freq
Steps To Reproduce
working_dir = r"D:\Temp"
import os
from ansys.dpf import core as dpf
from ansys.dpf.core import animation
from ansys.dpf.core import examples
model = dpf.Model(examples.download_modal_frame())
disp = model.results.displacement.on_all_time_freqs.eval()
freq_scoping = disp.get_time_scoping()
freq_set = freq_scoping[0]
filename = os.path.join(working_dir,r"mode{}.gif".format(freq_set))
animation.animate_mode(disp ,deform_scale_factor=10.0, mode_number=freq_set, save_as=filename, off_screen=True)
Which Operating System causes the issue?
Windows
Which DPF/Ansys version are you using?
Ansys 2024 R1
Which Python version causes the issue?
3.10
Installed packages
ansys-dpf-core                        0.13.1.dev0

I am running 2024R2, which was not an option in the dropdown menu


Merge (original) Web Services Metadata specification source

Just an FYI...  The original (oracle) Web Services Metadata specification source is now available via this CQ:  https://dev.eclipse.org/ipzilla/show_bug.cgi?id=21464
Please import/modify/merge it when time allows, hopefully in time for Jakarta EE 9.  Thanks!

fixed by #59


Title: Add country code and country name using  drop down menu in Html code

Is your feature request related to a problem? Please describe.
This feature enables the user to select their country name and country code in Phone number section. Its gonna make easier for the user. And it will improvise the code
Describe the solution you'd like.
I would like to use drop down list in order to select the country name and code in Html code. If acceptable  I would like to work on it @agamjotsingh18
Describe alternatives you've considered.
Can use Timezones and Dictionary in javascript for better enhancement of the code.I have done this project in one my internship.So I hope you would consider
Add any other context or screenshots about the feature request here.
No response

pls assign this issue to me , I would do great work.

Wohh ... Excited please hand over me @agamjotsingh18

Please assign this issue to me under GSSOC'23. As i have done it in my recent project and will do it nicely and attractively.

I am GSSoC contributor.Please assign this issue to me.

Sure @20951A0533

I have already completed this task, and I am confident in my skills and efforts. Please assign me this issue so that I can resolve it under GSSOC-23


Button can grab focus when window is out of focus, even if tree is paused

Godot version
3.5 beta3
System information
linux
Issue description
I seem to have related issues:

when paused is true, buttons can grab focus or be hovered
when paused is true and another window has focus, buttons can still grab focus or be hovered
when paused is false and another window has focus, buttons can be pressed and emit signals (and still grab focus or be hovered)

The inputs to change focus or press buttons are made with a controller
Steps to reproduce
Launch the project, Alt-TAB to another window, make sure you can see the buttons below, use dpad to move through the buttons
Minimal reproduction project
test_paused_out_focus.zip

This is done by design: https://github.com/godotengine/godot/issues/16832
Guard your Input.is_action_pressed()/event.is_action_pressed() checks with OS.is_window_focused() if you want to prevent this.

Hi, I understand the design while out of focus, thank you for the link.
I don't understand it while game is paused, though.
By the way, I don't understand how to guard the Inputs, since the movement is handled automatically by godot (like in the example above).


Error when requesting ceritficate...

When requesting a certificate for a domain... I get through all steps successfully, even challenges; however, I get a curl error when requesting the actual certificate:
  + Requesting certificate...
curl: (22) The requested URL returned error: 403 Forbidden
  + Done!

Also curious how to use the staging setup for LetsEncrypt so I can issue test certificates. Changing the CA to the typical "https://acme-staging.api.letsencrypt.org/directory" I use with the official LetsEncrypt client results in 404s.

Yea this still is missing some error handling.
To use the staging environment you should set the CA to "https://acme-staging.api.letsencrypt.org", without the "/directory" at the end, it should work.

Yeah I gave that shot when I was experimenting and that turned the 404 into a 403 Forbidden:
  + Generating private key...
  + Generating signing request...
  + Requesting challenge for verbler.com...
curl: (22) The requested URL returned error: 403 Forbidden

I'm not sure what's going on there, maybe it requires the usage of a different agreement url on registration.

Error after latest commits with https://acme-staging.api.letsencrypt.org as CA
  + Requesting challenge for verbler.com...
  + ERROR: An error occured while sending post-request to https://acme-staging.api.letsencrypt.org/acme/new-authz (curl: (22) The requested URL returned error: 403 Forbidden)

Is there a way for me to receive the full error response body? Would that contain anymore info?

Modified the script a bit and got the real error:

Error: Can't retrieve challenges ({"type":"urn:acme:error:unauthorized","detail":"No registration exists matching provided key","status":403})

Resolved by deleting my privatekey.pem and letting it regenerate.

I am stuck here here as well.

Continually got this error until i ran it with just -c, then I got:
Then i got this:
{"type":"urn:acme:error:malformed","detail":"Provided agreement URL [https://testca.kurz.pw/terms/v1] does not match current agreement URL [https://letsencrypt.org/documents/LE-SA-v1.0.1-July-27-2015.pdf]","status":400}

After setting my LICENSE to that value, which was a complete guess on my part, i then got that same 403 error....

I was able to get around it by just using the -c option and removing my private key which I believe others did as well.  I am moving forward now.

Is there any solution to use my one private key? -Its really annoying -I have to use one of my private keys due to HPKP otherwise I cannot switch my certificate....


Adding support for RBAC resources

Although the feature is in alpha version is fully functional and the RBAC resources (roles, rolebindings, clusterroles and clusterrolebindings) will be kept for future versions.

@brosner i'm back with more contributions, could you please check this PR? Thanks


Add picture to UserProfile

Reference from User Profile doc
Currently UserProfile do not returned with picture.

Don't forget to update the very simple test at UserProfileBridgeTest


simplify Bernoulli function

simplify bernoulli and its derivative function implementations to reduce number of calculations

Improved conditional code for android.


add support for ayaneo2s

@Maclay74 Please take a look. I tried it on my own device (ayaneo2s), and it works pretty googd.

Are you sure you need dedicated file for this device? What's special about those keys?


Confirm NextShardIterator key exist to fetch from existing consumer shard

Hey Hampsterx,
So I was added shards to my kinesis stream and from the discussion I had with AWS engineer is those old shards would still be active for at least during the data retention period of your stream. However in the results from the old shards, they do not contain the NextShardIterator key in the results, even though it's mention it will be null in their docs.
https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/kinesis.html#Kinesis.Client.get_records
This basically makes the consumer move to the next shard if the key NextShardIterator is not found or is string null in results after returning output.

hi @jmcgrath207
Interesting, I assumed at the time something more complicated would be required if the shard count was increased hence the NotImplemented Exception. I wonder if its worth logging a warning or something because currently the consumer does'nt recheck the shards or otherwise have any way of knowing a reshard has occurred. Basically you would have to restard the consumer.
some random googling..
https://docs.aws.amazon.com/streams/latest/dev/kinesis-using-sdk-java-resharding.html

There are two types of resharding operations: shard split and shard merge. In a shard split, you divide a single shard into two shards. In a shard merge, you combine two shards into a single shard. Resharding is always pairwise in the sense that you cannot split into more than two shards in a single operation, and you cannot merge more than two shards in a single operation. The shard or pair of shards that the resharding operation acts on are referred to as parent shards. The shard or pair of shards that result from the resharding operation are referred to as child shards.

https://aws.amazon.com/premiumsupport/knowledge-center/kinesis-data-streams-open-shards/

The shard or pair of shards that the resharding operation acts on are known as parent shards. The shards that are created after the resharding operation are known as child shards. A parent shard also transitions from an OPEN state to a CLOSED state (and eventually, to an EXPIRED state, after the stream’s retention period). This can result in child shards being assigned an OPEN state

After resharding, you should also continue to read data from the CLOSED shards until they are exhausted. This helps to preserve the order of the data read by the consumer applications. After you have exhausted all of the CLOSED shards, you can begin to read data from open child shards. The Amazon Kinesis client library (KCL) is designed to adapt to resharding operations. Any data that existed in the shards before resharding is processed first.

https://tutorialsdojo.com/kinesis-scaling-resharding-and-parallel-processing/

The Kinesis Client Library (KCL) tracks the shards in the stream using an Amazon DynamoDB table, and adapts to changes in the number of shards that result from resharding. When new shards are created as a result of resharding, the KCL discovers the new shards and populates new rows in the table.  The workers automatically discover the new shards and create processors to handle the data from them. The KCL also distributes the shards in the stream across all the available workers and record processors.

Hmm, interesting.
https://github.com/awslabs/amazon-kinesis-client/blob/master/amazon-kinesis-client/src/main/java/software/amazon/kinesis/lifecycle/BlockOnParentShardTask.java

Task to block until processing of all data records in the parent shard(s) is completed.

We check if we have checkpoint(s) for the parent shard(s).
If a checkpoint for a parent shard is found, we poll and wait until the checkpoint value is SHARD_END
(application has checkpointed after processing all records in the shard).
If we don't find a checkpoint for the parent shard(s), we assume they have been trimmed and directly
proceed with processing data from the shard.

Hmm, there is quite a bit of java code (haha) in that library so not clear (after 3m of looking) how it entirely works.
I'm in favour of a simple approach, ie abort if a reshard happens. This is fine as container services will just restart. So long as on restart it then does the right thing of draining the old shards first.
Open to ideas, in your PR, does skipping the shard do the right thing in this case? Guessing probably not? If it does then I am happy to merge as its far simpler than alternative!
So sounds like ideally it would need to (just rambling here..):

detect reshard event? Is this as simple as hit the NextShardIterator missing/null?
pause consuming on all shards
refetch list of shards (somehow know which ones are parent/child?)
start a flag/lock. consume only on "closing" (parent) shards or existing shards (not new)
remove flag/lock, start consuming on all shards

hmm, sounds complicated, would be cool as then can just dynamically reshard up and down and the consumer would be able to keep going like a champ hehe.

hi @jmcgrath207
Interesting, I assumed at the time something more complicated would be required if the shard count was increased hence the NotImplemented Exception. I wonder if its worth logging a warning or something because currently the consumer does'nt recheck the shards or otherwise have any way of knowing a reshard has occurred. Basically you would have to restard the consumer.
some random googling..
https://docs.aws.amazon.com/streams/latest/dev/kinesis-using-sdk-java-resharding.html

There are two types of resharding operations: shard split and shard merge. In a shard split, you divide a single shard into two shards. In a shard merge, you combine two shards into a single shard. Resharding is always pairwise in the sense that you cannot split into more than two shards in a single operation, and you cannot merge more than two shards in a single operation. The shard or pair of shards that the resharding operation acts on are referred to as parent shards. The shard or pair of shards that result from the resharding operation are referred to as child shards.

https://aws.amazon.com/premiumsupport/knowledge-center/kinesis-data-streams-open-shards/

The shard or pair of shards that the resharding operation acts on are known as parent shards. The shards that are created after the resharding operation are known as child shards. A parent shard also transitions from an OPEN state to a CLOSED state (and eventually, to an EXPIRED state, after the stream’s retention period). This can result in child shards being assigned an OPEN state

After resharding, you should also continue to read data from the CLOSED shards until they are exhausted. This helps to preserve the order of the data read by the consumer applications. After you have exhausted all of the CLOSED shards, you can begin to read data from open child shards. The Amazon Kinesis client library (KCL) is designed to adapt to resharding operations. Any data that existed in the shards before resharding is processed first.

https://tutorialsdojo.com/kinesis-scaling-resharding-and-parallel-processing/

The Kinesis Client Library (KCL) tracks the shards in the stream using an Amazon DynamoDB table, and adapts to changes in the number of shards that result from resharding. When new shards are created as a result of resharding, the KCL discovers the new shards and populates new rows in the table.  The workers automatically discover the new shards and create processors to handle the data from them. The KCL also distributes the shards in the stream across all the available workers and record processors.

Hmm, interesting.
https://github.com/awslabs/amazon-kinesis-client/blob/master/amazon-kinesis-client/src/main/java/software/amazon/kinesis/lifecycle/BlockOnParentShardTask.java

Task to block until processing of all data records in the parent shard(s) is completed.

We check if we have checkpoint(s) for the parent shard(s).
If a checkpoint for a parent shard is found, we poll and wait until the checkpoint value is SHARD_END
(application has checkpointed after processing all records in the shard).
If we don't find a checkpoint for the parent shard(s), we assume they have been trimmed and directly
proceed with processing data from the shard.

Hmm, there is quite a bit of java code (haha) in that library so not clear (after 3m of looking) how it entirely works.
I'm in favour of a simple approach, ie abort if a reshard happens. This is fine as container services will just restart. So long as on restart it then does the right thing of draining the old shards first.
Open to ideas, in your PR, does skipping the shard do the right thing in this case? Guessing probably not? If it does then I am happy to merge as its far simpler than alternative!
So sounds like ideally it would need to (just rambling here..):

detect reshard event? Is this as simple as hit the NextShardIterator missing/null?
pause consuming on all shards
refetch list of shards (somehow know which ones are parent/child?)
start a flag/lock. consume only on "closing" (parent) shards or existing shards (not new)
remove flag/lock, start consuming on all shards

hmm, sounds complicated, would be cool as then can just dynamically reshard up and down and the consumer would be able to keep going like a champ hehe.

Bit more..
https://awsdocs.s3.amazonaws.com/kinesis/latest/kinesis-dg.pdf (2013)

After the reshard has occurred and the stream is once again in an ACTIVE state, you could immediately
begin to read data from the child shards. However, the parent shards that remain after the reshard could
still contain data that you haven't read yet that was added to the stream prior to the reshard. If you read
data from the child shards before having read all data from the parent shards, you could read data for a
particular hash key out of the order given by the data records' sequence numbers. Therefore, assuming
that the order of the data is important, you should, after a reshard, always continue to read data from the
parent shards until it is exhausted, and only then begin reading data from the child shards. When
getRecordsResult.getNextShardIterator() returns null, it indicates that you have read all the
data in the parent shard. If you are reading data using the Amazon Kinesis Client Library, the library
ensures that you receive the data in order even if a reshard occurs.

Ahh, so the stream changes state until the reshard is done. Right..
So basically to clarify the current behaviour.
Consumer will keep going, skipping the exausted shards. However it will then only be reading the remaining unaffected shards (ie not the new ones) until a restart. If the consumer failed/restarted before draining old shards then that data would be lost.

Bit more..
https://awsdocs.s3.amazonaws.com/kinesis/latest/kinesis-dg.pdf (2013)

After the reshard has occurred and the stream is once again in an ACTIVE state, you could immediately
begin to read data from the child shards. However, the parent shards that remain after the reshard could
still contain data that you haven't read yet that was added to the stream prior to the reshard. If you read
data from the child shards before having read all data from the parent shards, you could read data for a
particular hash key out of the order given by the data records' sequence numbers. Therefore, assuming
that the order of the data is important, you should, after a reshard, always continue to read data from the
parent shards until it is exhausted, and only then begin reading data from the child shards. When
getRecordsResult.getNextShardIterator() returns null, it indicates that you have read all the
data in the parent shard. If you are reading data using the Amazon Kinesis Client Library, the library
ensures that you receive the data in order even if a reshard occurs.

Ahh, so the stream changes state until the reshard is done. Right..
So basically to clarify the current behaviour.
Consumer will keep going, skipping the exausted shards. However it will then only be reading the remaining unaffected shards (ie not the new ones) until a restart. If the consumer failed/restarted before draining old shards then that data would be lost.

sorry one more comment..
https://github.com/ungikim/kinsumer/blob/master/kinsumer/consumer.py
That implementation uses timer/poll to check the shards every hour. see ShardMonitor
Thats the thing currently, if you add more shards you need to restart the consumer(s) :(
@akursar no wonder the listShards api is 100 ops/se per stream. This allows every consumer to frequently check the stream for shard changes!

sorry one more comment..
https://github.com/ungikim/kinsumer/blob/master/kinsumer/consumer.py
That implementation uses timer/poll to check the shards every hour. see ShardMonitor
Thats the thing currently, if you add more shards you need to restart the consumer(s) :(
@akursar no wonder the listShards api is 100 ops/se per stream. This allows every consumer to frequently check the stream for shard changes!

Hey @hampsterx,
So after spending sometime with this change, it does work by definition however, it's very brute force and I also failed to take into consideration the checkpointer portion, where it would basically blacklist the shard once exhausted.
So in my scenario, I had a stream with 3 shards, I bump it to 4, then 5, in a 24 hour period. This caused the consumer to return 22 shards in total with most being empty.
With this change it would go through all the shards(that are active) to see if it has data or not. Not great but it works.
Saying that I think that part is unavoidable during our first run, however with the check pointer being aware of it we can avoid that once more consumer are added assuming redis checkpointer is being used.
Here is what I am thinking I should change:

keeping the similar logic from this pull request, but push it up and have it deallocate the shard so the checkpointer is aware of it also.
implement a timer logic so it refresh the shard list based on the data retention period(assume we can get that from boto3). So usually 24 hours.

With this we would still get the remaining records from the parent, but once exhausted we tell it too not track again that shard again. This idea falls apart with the memory checkpointer though.
Let me know if I am missing something here.
example line 190 in the consumer.py

            if shard.get("fetch"):
               # timer( monotonic based) logic for refreshing shard list based on data retention time that is returned
                if shard["fetch"].done():
                    result = shard["fetch"].result()

                    if not result:
                        shard["fetch"] = None
                        continue

                    records = result["Records"]

                    if records:
                        log.debug(
                            "Shard {} got {} records".format(
                                shard["ShardId"], len(records)
                            )
                        )

                        total_items = 0
                        for row in result["Records"]:
                            for n, output in enumerate(
                                self.processor.parse(row["Data"])
                            ):
                                await self.queue.put(output)
                            total_items += n + 1

                        if not result.get("NextShardIterator") or result.get("NextShardIterator") == 'null':
                            shard["fetch"] = None
                            self.checkpointer.deallocate(shard["ShardId"])
                            continue

yep I forgot about the checkpointer part as well. The deallocate (for redis) should probably set an expiring key to the duration of the stream retention so if restarted it would know that is an expired shard?
The memory checkpointer I would not be too worried about, I don't really think it makes too much sense not using checkpointing so was mostly there for unit tests, probably it should just error/abort.
Timer logic, was thinking it should be more like every 5m, ie duration in seconds
Still need some logic to say don't start consuming those new shards until the old ones are drained aye
When you get this working i'd like to try and add a unit test that changes the shard count to confirm the behaviour. Seems like its supported in kinesalite but whether it maps correctly to AWS behaviour or not who knows.

@hampsterx any plans to merge it any time soon? with the release of ON_DEMAND mode for kinesis, this PR is a must have

hi @chekan-o this PR needs some more work thats for sure and this was all @jmcgrath207 effort here. This feature is kind of complex hence why very few libraries except KCL implement it. I will see if I can stamina the strength to work on this soonish but no promises, might even dumb it down a bit so it just drains the old shards on reshard event and then dies, would be far simpler.
Are you using the ON_DEMAND mode?

Yes it suites our load profile quite well, and we got quite busy stream with spiky load. On demand works well with lambdas, but when we try it with long running service it would just go down for 24 hours, which is not acceptable in production.
We would definitely help to test it if needed.

Fully supporting shard split/merge logic requires some rework.
This post seems to explain it fairly well:
https://brandur.org/kinesis-by-example
To summarize the above:
In "maintenance" task: List all shards. Update checkpointer metadata based on shard ID at a regular interval.
We'd have to spawn a new task when a shard doesn't exist in checkpointer.
"per shard" task can have the states:

Pending (Do we have a parent/adjacent parent? If so, wait until they are closed before proceeding.)
Active (Set sequenceNumber to StartingSequenceNumber.  Stay here until sequenceNumber matches EndingSequenceNumber)
Closed (SequenceNumber matches EndingSequenceNumber)

If we do it this way, then we can just have each task toss their records onto the pile until they're all closed.
Leaving this comment in case reading the Java code wasn't providing insights.  May attempt a PR later on.

Okay. Realized why this project hasn't been active recently: AWS allowed lambdas to act as consumers, so those wanting to use Python can just use a lambda triggering on KDS events.

This project was built for KDF running in container but I haven't needed to use it on recent projects so its languished a bit, the resharding support would be great though. If you can use Lambda (KDS) all the better just not every project is suitable for it.

Understood. I was confused because most of the activity around packages similar to this one died around late 2019/early 2020.  Then I found out AWS had hooked up lambda support on KDS.  If you'd still find it useful, then I shall attempt to do it when I get a chance. ^^


feat: implement the "file" utility

Good work!
Initial feedback, easy to fix:

"code should look like other posixutils code":   clap use #arg[...] not #clap
"code should look like other posixutils code":   struct Args not FileArgs
PR should merge without conflicts (easy README merge fix)
Code should work out of the box on Linux and MacOS.   So far, it does not work for ELF and zip files on MacOS in my limited testing.  I'm guessing this is a simple file location issue (MacOS has the magic location as /usr/share/file/magic/magic)

@jgarzik

Good work!
Initial feedback, easy to fix:
* "code should look like other posixutils code":   clap use `#arg[...]` not `#clap`

* "code should look like other posixutils code":   `struct Args` not FileArgs

* PR should merge without conflicts (easy README merge fix)

* Code should work out of the box on Linux and MacOS.   So far, it does not work for ELF and zip files on MacOS in my limited testing.  I'm guessing this is a simple file location issue (MacOS has the magic location as `/usr/share/file/magic/magic`)

Fixed!

This fixes --help:
 #[derive(Parser, Debug)]
 #[command(author, version, about, long_about, disable_help_flag = true)]
 struct FileArgs {
+    #[clap(long, action = clap::ArgAction::HelpLong)]
+    help: Option<bool>,
+
     /// Apply default position-sensitive system tests and context-sensitive sys

On MacOS, further testing results.  It is not so simple.  /usr/share/file/magic is a directory containing many individual magic text files.
I tried doing
$ cat /usr/share/file/magic/* > etcmagic.db

and then ran file on itself, and had a crash:
% RUST_BACKTRACE=1 ./target/debug/file --magicfile ./etcmagic.db ./target/debug/file
thread 'main' panicked at file/src/file.rs:396:29:
index out of bounds: the len is 3 but the index is 3
stack backtrace:
   0: rust_begin_unwind
             at /rustc/7cf61ebde7b22796c69757901dd346d0fe70bd97/library/std/src/panicking.rs:647:5
   1: core::panicking::panic_fmt
             at /rustc/7cf61ebde7b22796c69757901dd346d0fe70bd97/library/core/src/panicking.rs:72:14
   2: core::panicking::panic_bounds_check
             at /rustc/7cf61ebde7b22796c69757901dd346d0fe70bd97/library/core/src/panicking.rs:208:5
   3: <usize as core::slice::index::SliceIndex<[T]>>::index
             at /rustc/7cf61ebde7b22796c69757901dd346d0fe70bd97/library/core/src/slice/index.rs:255:10
   4: core::slice::index::<impl core::ops::index::Index<I> for [T]>::index
             at /rustc/7cf61ebde7b22796c69757901dd346d0fe70bd97/library/core/src/slice/index.rs:18:9
   5: <alloc::vec::Vec<T,A> as core::ops::index::Index<I>>::index
             at /rustc/7cf61ebde7b22796c69757901dd346d0fe70bd97/library/alloc/src/vec/mod.rs:2771:9
   6: file::RawMagicFileLine::parse
             at ./file/src/file.rs:396:29
   7: file::parse_magic_file_and_test
             at ./file/src/file.rs:506:42
   8: file::get_type_from_magic_file_dbs::{{closure}}
             at ./file/src/file.rs:529:9
   9: <core::slice::iter::Iter<T> as core::iter::traits::iterator::Iterator>::find_map
             at /rustc/7cf61ebde7b22796c69757901dd346d0fe70bd97/library/core/src/slice/iter/macros.rs:319:38
  10: file::get_type_from_magic_file_dbs
             at ./file/src/file.rs:528:5
  11: file::analyze_file
             at ./file/src/file.rs:618:31
  12: file::main
             at ./file/src/file.rs:644:9
  13: core::ops::function::FnOnce::call_once
             at /rustc/7cf61ebde7b22796c69757901dd346d0fe70bd97/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

I'll have a recursive listing of magic files done in the default directory(if it's a directory) and then have the check done

We definitely need this to work with some very basic tests (platform binaries, ZIP files) on MacOS and Linux.

Sure, i'll have some test run and let you know the results

Using only the zip data definition did not work, so there is some bug somewhere:
% ./target/debug/file --magictext /usr/share/file/magic/zip test.zip
test.zip: data

Here is the patch I made for testing, which also includes some fixes: https://gist.github.com/jgarzik/25832bf69471bab34675443d558b4873

btw i was able to retain the zero dependency goal by not using binread)(it seems if the memory layout of the struct Magic is in C representation, then casting the pointer of an array to the struct can simply do the job)
(the compiled magic parser is still a WIP, will let you know as the progress happens)
(the libmagic source code is quite a time consuming rabbit hole to go through, a fun one though)

@jgarzik, i've removed the WIP for binary magic files(it's ready to be merged now)
the patch you have mentioned(used for testing), i shouldnt be adding that to this PR right?. I mean there's magictext option being used(not specified in the specification), also there's already "-m" and "-M" option(which are used to take in files for position sensitive test i.e magic files)


Bump to latest kernel and musl

The headers_check make option is now deprecated and was removed upstream on version 5.15, causing a build failure on any later kernel.
See https://github.com/torvalds/linux/commit/4fbce819337a6705559788ff7e9583a4703edcb7

Packages were updated, thanks for notifying me!

Fixed in 1ea3ecc.


nrf/gpiote: remove PortInput, move impls to Input/FlexPin.

 This PR moves the wait_for_x functionality to Input directly.
It also adds it to FlexPin for completeness and consistency with Input.
(The reason PortInput exists is a while ago GPIOTE was an owned singleton that you had to initialize, so PortInput::new() would require it to enforce it's been initialized. This doesn't apply anymore now that GPIOTE is "global")

bors r+


Parse server

Hello,
Maybe it will be interesting to integrate with Parse Server
with the target of no firebase dependency.
Thank you!

Thanks @DaBlitzStein


Need property icon rotation in agm-marker

Issue description
Need property icon rotation in agm-marker could be great.
Is possible to have a class for agm-marker
Example : https://stackoverflow.com/questions/46843606/how-to-make-custom-arrow-mark-in-agm-map

yes please anyone have done this because i have same situation need to rotation icon.

currently you can workaround by setting [iconUrl]="{rotation: 40, path: google.maps.SymbolPath.FORWARD_CLOSED_ARROW}"
You are invited to create a PR extending the AgmMarker directive with new properties

@doom777 it doesn't work for custom marker icons

@magnusfernandes , should work. If anything, the example I gave doesn't work because google namespace isn't available at that time.
Can you make an example on stackblitz?

@doom777 can this issue be solved soon? Need it for an app release.

can you create a stackblitz?

@doom777 https://angular-f5wbnz.stackblitz.io

worked for me:

@doom777 can't rotate the custom icon

Same problem here...

@SebastianM @doom777 any update on this, please?

Ok, so I think I got it. MarkerOptions#icon property can either be an Icon or a Symbol. Symbol class has rotation property, while Icon has scaledSize, so you can't have them together. So either convert your icon to an SVG and inline it in Symbol, or actually rotate the image in the icon file manually.
This is a limitation of Google Maps, so we cannot fix it.
Please confirm that this is your problem, and if so, close the issue.

Is someone working on this issue? Otherwise I could try to implement this (need the functionality aswell 👍 ).

stackbiz is not working

Stackblitz stopped working generally.

Rotation is working fine but when i am changing rotation value at run time it not working, Example project https://angular-google-maps-demo-b3cm38.stackblitz.io

You cannot rotate markers using agm-marker and iconUrl property. I recommend using agm-overlay. This way you can set the icon with "img src" and use inline css "style" to rotate the icon. You can check my working example here:
<agm-overlay *ngFor="let marker of markers" [latitude]="marker.latitude" [longitude]="marker.longitude">     <div>       <img *ngIf="marker.type === 'BT'" src='./assets/images/bt.png' style="transform: rotate({{marker.rotation}}deg);         width: 60%;       height: 60%;">       <img *ngIf="marker.type === 'RTMS'" src='./assets/images/test.png' style="transform: rotate({{marker.rotation}}deg);       width: 5%;       height: 5%;">     </div>   </agm-overlay>

Any update on this request?

I created PR to add icon type


[SEMVER MAJOR] Add support for node8 rules environment

This PR makes changes required to support node8.

Shim the jwt magic global to support minor api change between newer jsonwebtoken@7 and previous version
Shim the crypto magic global to support passing string cipheriv keys.
Bump modules to support node@8:

azure-storage from ~0.4.1 to ~2.2.1
couchbase from ~1.2.1 to 1~2.3.5
jsonwebtoken from ~0.4.1 to ~7.4.1
knex from ~0.6.3 to ~0.13.0
mongo-getdb from ~1.2.0 to ^2.0.0
mongodb from ~1.3.15 to ^2.0.0
mysql from ~2.0.0-alpha8 to ^2.0.0
node-cassandra-cql from ^0.4.4 to ^0.5.0
request from ~2.27.0 to ~2.81.0

close in favor of #18


'error[internal]: left behind trailing whitespace' after closure argument list

As the title says, the following code leads to an internal error:
fn main() {
    Some(1).map(|id|
    // Some comment explaining what I am doing
    id + id);
}

Link to playground: https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=0714c35f8e91a1bb365a8059fa532329
The exact error message:

error[internal]: left behind trailing whitespace
--> /playground/src/main.rs:2:2:21
   |
2 |     Some(1).map(|id|
   |                          ^
   |

rustfmt version: 1.8.0-stable
Alternative: This also happens with a linebreak before the second id in the addition
https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=b12719274b196887bbd5691c063a173f
On a cursory skim the only duplicate I found was https://github.com/rust-lang/rustfmt/issues/2896, which is closed and locked without any resolution. Honestly I am a bit taken aback by the fact this bug from 2018 still seems to be around. I also noticed there are quite a few issues for this error caused by a wide variety of (valid) code patterns. Is there any path forward to fixing these? Is there guidance available if I wanted to try and fix this? Maybe it is already known what would have to be changed?

Thanks for the report, but this one is a duplicate #4855. The issue here is that the snippet contains trailing whitespace and rustfmt isn't expecting to find a comment between the closure parameters and the expression in the brace-less closures. In order to not drop the comment rustfmt leaves the closure unfromatted, which means the trailing space is preserved. You can add braces to the closure or manually remove the trailing space.

I also noticed there are quite a few issues for this error caused by a wide variety of (valid) code patterns. Is there any path forward to fixing these?

The error[internal]: left behind trailing whitespace is a generic error and can happen for a number of reasons. The vast majority of cases reported involve code snippets like this one that already contain trailing whitespace that rustfmt has trouble removing.


"Native" support for themes

Idea around this to add "native" support for various frameworks - Bootstrap, Purecss, Angular Material, etc. by customizing the templates to include proper classes for various elements in each framework. This should be a better solution than just emulating the default appearance of a framework, since more often than not sites use their own customization of CSS for each framework.
This is in progress right now, but needs to be completed, and help is welcome. Right now a few frameworks I've added are below, but more are welcomed:

Bootstrap
Angular Material
Pure CSS
Material Design Light

The idea is that current code like this:
<button class="ok" tabindex="1">{{ok}}</button>

becomes this, depending on the theme (bootstrap example below):
<button class='ok btn btn-success' tabindex='1'>{{ok}}</button>

Right now, in 1.0.4 which is going to be published shortly, there's a new public method theme() which makes this to happen:
alertify.theme("bootstrap");

It can be reset with:
alertify.theme("default");

For starters, this only adjusts the button classes, but it should be possible to implement the classes of modal dialogs, etc, too, depending on the framework.
Would love to hear suggestions for which frameworks should be supported, and would welcome any assistance in added more frameworks and completing the ones already started.

Need to update Bootstrap to include the default classes for the input element, too. See issue #52

Can I suggest you use different Bootstrap classes?
Currently you have:
this.dialogs.buttons.ok = "<button class='ok btn btn-success' tabindex='1'>{{ok}}</button>";
this.dialogs.buttons.cancel = "<button class='cancel btn btn-danger' tabindex='2'>{{cancel}}</button>";

So cancel is btn-danger and ok is btn-success. To fit with the examples on the Bootstrap site, you should have btn-default for cancel and btn-primary for success.

Agreed, that makes sense. If you want to submit a PR that would be great, otherwise I'll add it to the to-do list and get to it as soon as I can...

+1 on this. I have a simple dialog and wanted to have a btn-success on it. Got around the issue uglily accessing the "private" object and changing the template directly, but would be way cooler if we could define custom classes - this could get in the way of the lib's lightness, tho.
Sidenote: there are no docs on this. Are you going to include the .theme() method on the docs page, or could I PR it? :) I just found it by accident, while fiddling with the source, trying to find a way to use bootstrap buttons.

@igorsantos07 Go ahead and PR it, you're right it should be on the docs page. Same goes for the second parameter. That sounds like a good idea!

@bradberger Is there a branch that this is being worked on from. I can assist with some of this as well. I would also like to suggest that Semantic UI be added to the list.
Additionally, though maybe not related to this issue, it would be great to be able to disable the injected CSS for alertify.js in another manner. There is no way to disable the injection when being used in React, unless I am missing something. I think it would make more sense to allow people to decide to include the CSS file, where they want to locate it, etc.

@brianespinosa Thanks! The master branch is right now the current production version, that would be the one to fork and PR


update @types/vscode to latest available

Hi there,
I was playing around with making my own custom statusbar extension. I wanted to set the backgroundColor and I was confused why it wouldn't compile. Turned out it was using an older version of the type definitions. Updating fixed the issue.
I wonder if you'd consider trying to automate the samples so that they were updated whenever a new version was released?
Thanks!

@microsoft-github-policy-service agree

Closing, engine also needs to be updated.


Support for mapper 9

Hi,will there be support for Mapper 9 in a future release?
thanks

No, this mapper is blacklisted.
The note on the compatibility sheet says "Impossible to accurately emulate CHR bank switching".
Punch-Out won't be compatible unless someone wants to work with me on patching the game.

Any case that you can create a native nes emulator for snes instead of
merging each time a game with the emulator?
Thanks
Στις Πέμ, 30 Δεκ 2021, 10:58 ο χρήστης Myself086 @.***>
έγραψε:

No, this mapper is blacklisted.
The note on the compatibility sheet says "Impossible to accurately emulate
CHR bank switching".
Punch-Out won't be compatible unless someone wants to work with me on
patching the game.
—
Reply to this email directly, view it on GitHub
https://github.com/Myself086/Project-Nested/issues/44#issuecomment-1002933022,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJCXONKPQHIM56VO5A4B7NTUTQNMNANCNFSM5K5PUF7Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Yes but it's very low priority on my list.
This emulator requires 4 files:

The emulator program
The NES game
Save file for the NES game (sometimes)
Save file for the emulator (feedback to exe)

But the SNES doesn't have a file system so SNES emulators and devices only provide 2 files:

SNES program (ROM)
Save data (SRAM)
Which is why merging is necessary.

If someone wants to update firmwares for Super Everdrive and FXPak Pro then I'd be willing to provide all necessary information to make it possible. However, there's a new feature coming to v1.5 that would be difficult to port to those devices.

Did you ask to Krikzz and Ikari about ?
Maybe that guy here can help
https://github.com/hcs64/neon64v2
He made the native nes emulator for N64

I know how to add support for mapper 9 but it can't be emulated perfectly on SNES due to the reason I posted above.
Games running on mappers 9 and 10 will need to be patched, will be tricky and isn't worth my time.
I am not going to ask around for help on something that I know how to do. If someone wants to come in and help patch the game, I'd be open to that. But until then, mapper 9 is not happening.
N64 is powerful enough to emulate the PPU at a low level, SNES must emulate it at a high level.

Oh i see thanks,is there a list of supported games in titles?

https://docs.google.com/spreadsheets/d/1xKZIyNz1DSI3ZBdMfaTEaa_9b6IEABx-ZPwOb6XqcLQ/edit?usp=sharing
Use this link or click the "Compatibility" button on the bottom left corner of the exe's main window.

Thanks
Is it possible to add scanlines to the emulator by default or just as an option to select before merging a rom with the emulator ??

I'm not sure what you're talking about.

For the games to have scanlines that old Crt tv's had so the games wont show pixels,every emulator has this option at least for pc etc.

This isn't necessary because the resolution on the SNES already matches that of the NES. The image would look very similar to a NES on a real CRT.

I dont mean to add scanlines for crt's tvs i mean to add scanlines for tft's tvs that most people have and nes on this tvs look very bad since they dont have scanlines, .it can always be an option before merging the rom thought.

It can't be an option because the resolution on SNES matches that of the NES. So it's impossible to add black lines between what would be NES lines.

Do you play to implement mappers from pirate games and unlicensed games? Because it would be cool to play those games such as gauntlett 1 & 2,pirated nes supermarioworld,donkeykong country “4” super donkeykong 1 & 2 among other pirated & unlicensed nes games on snes,
and in case you don’t wanna do that you can decide to make this project opensource sothat anybody can help completing this project as soon as possible😁

NES Super Mario World absolutely yes, I made an editor for this game back in 2017.
For other mappers, it mostly depends on how compatible the mapper is to the SNES.

Any recent changes for a native nes emulator without the need of merging??
Thank you

Any recent changes for a native nes emulator without the need of merging??
Thank you

I have no idea how to do it but if you find a way to implement it, feel free to let me know.
Also, please keep issues on topic so I can easily browse through them later.

It will be nice to find a way to do it like that since it will be not necessary anymore to merge hundreds or roms one by one.


Weekly #6: February 8th-16th

Please add in stuff as you see it!

I had to send PR #22 as it looks like I don't have the rights to push to this repo.

Feel free to just cherry-pick the commit into this branch.

@chriscool that was perfect. As well, you can just comment here in markdown and we'll add it in in the future.

Merging with #17.


PARQUET-2201: [parquet-cpp] Add stress test for RecordReader ReadRecords and SkipRecords.

This pull request adds a stress test for testing ReadRecords and SkipRecords for required, optional, and repeated fields. It will create random pages and run a random set of skip and read calls to consume all the pages. It will then compare the expected output with the actual output from the record reader.
These changes are on top of this pull request: https://github.com/apache/arrow/pull/14142

@emkornfield can you take a look?

@emkornfield can you take a look?


"General URL" should not be rendered as a link on repo details page

The 'Repository Details' view renders the Git URL as an href, which encourages people to click on it.  (See #437)
It seems to me that this information is provided for cutting and pasting into clients, and doesn't serve any purpose as an href.
Although I think it probably shouldn't 404 when you go to this URL with a browser, the best place for it to go would probably be to the repository details, so it would still be useless as an href on that page, because it would just link back to itself.
Should this just be displayed as text?

The nice thing about having it as an href is that you can right click and copy target link. But i agree that it can be confusing. Maybe we can check the browser string and redirect to the repo if the git URL is called from the browser.

Perhaps we could have it as plain text but with a 'copy to clipboard' button next to it?

That would be nice.

How does the project feel about using additional external libraries, e.g. https://www.npmjs.com/package/clipboard-js ?

I think it's fine but i don't know anything about licenses and similar
important things.
Am 12.02.2016 11:27 schrieb "Will Dean"<EMAIL_ADDRESS>
How does the project feel about using additional external libraries, e.g.
https://www.npmjs.com/package/clipboard-js ?
—
Reply to this email directly or view it on GitHub
https://github.com/jakubgarfield/Bonobo-Git-Server/issues/449#issuecomment-183263627
.

Copy to clipboard and removal of the link would be nice.Clipboard.js is MIT so no problem with using it if it's a good one.You might also want to have a look at https://github.com/zeroclipboard/zeroclipboard (used by github).

@jakubgarfield @RedX2501  May be rename text before link to "General URL for Git client"?

@padremortius i think that's to too long. Maybe "Git URL"? Because that URL is clearly not a General URL but just not a personalized one.

I was going suggest "Git URL" too

I've put in a PR for this because I'd appreciate a review.  I moved the 'copy' buttons from one end of the URL to the other about 6 times, not really able to decide which looked best.  I used a different JS library, but it's still MIT.  Zeroclipboard seems to be flash-based, which rather put me off.


Fixed: Add glyph.auth and glyph.uri to rend

Addresses result of conversation in #704.
The att.extSym class was broken into two sub-classes so that we could only add rend to one of them, without also pulling in glyph.name and glyph.num. No other elements should be affected by this change.

This looks like a good solution to #704

I like the proposal but wondered whether we should harmonise the attribute names with att.authorized (@auth, @auth.uri, @codedval)?
I think the attributes can be compared as given in this table:

glyph
auth

glyph.auth
auth

glyph.name
? if allowd: label ?

glyph.num
codedval

glyph.uri
auth.uri

Harmonising these could mean any of the following:

change @glyph.name to @glyph.label
change @glyph.num to @glyph.codedval as this would allow for non-numeric identifiers

Or even

introduce something like @auth.name?

More important maybe would be harmonising the attribute definition, e.g. in the case of the .uriattributes

glyph.uri
auth.uri

The web-accessible location of the controlled vocabulary from which the value of glyph.name or glyph.num is taken. Value conforms to data.URI.
A web-accessible location of the controlled vocabulary or other authoritative source of identification or definition for this element or its content. This attribute may contain a complete URI or a partial URI which is completed by the value of the codedval attribute. Value conforms to data.URI.

The definition or documentation of glyph attributes should also indicate wheter I can take the glyph.num form a vocabulary and reference that with glyph.uri but pick a glyph.name deliberately or if that one also hast to come from the same authority.

@bwbohl The URI / name is specifically about the glyphs in the tag. @auth.uri is more about external authorities for other things (people, etc.) So we should stick with the @glyph attributes for now.
If you're suggesting that we also change the names of the @glyph attributes, I think this should be handled in another issue.

I think we can go on with the PR. To not forget about the useful Guidelines comments I also put it in the MEI Guidelines project.

Wait!

I’ve got a paragraph for the guidelines almost ready to go.


[Addition]:

Hotspot b58 Addresses
11Yne5j89zpbb2a7rUupNmD82HLmfKe8CWkSNnpLr6QNVtiHhmi
11kEUKvSSSwDsajNuvtKdFHLvGRdNg4Fd6D1FYPudF7Gic3Yz9n
112u2ZUzdC4cdsyNYjZxV7zuM5o67Wo8GqPhHqSgJovVmJtgtibb
112TyTxWz5vz6cb3YzpdE87fLjzCzPTRhPvDsRx3YRwyjbGmCWZo
112WB8S36xEtwPNSo5a1hMXAcM826wQEuvF6yxnkyh9BReeTKVpy
11iJbz8X1MLoWBKAanz1WMJy73w3ZoiDCFRUFWNyiY4tEQHyXS5
1175ut6U8ApHvwqoGLicBQAu5GF5hPamCsSJPAngixen5yEECPT
112Z45o5eF1eT1ijneqgx4VRj5jz6BJH9bVCcmyDF3UQ243bSm1r
11K9KLNVAneDLVW1MywKxE3w9nfPwjFgswUfPqAH21Aa9ycXUx8
Discord Handle
malder
Reason(s)
This group of hotspots only witness each other, despite there being 313 other online hotspots within 25 KM
They are all Deeper hotspots which fail to interact with non-Deeper (and other Deeper) hotspots in the immediate area.

We have taken into account this addition request in the next iteration of the denylist. Some or all of the Hotspots listed may be added to the denylist. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


s6-overlay-suexec: fatal: can only run as pid 1

Hi,
if i try to start the container it tells me "s6-overlay-suexec: fatal: can only run as pid 1" in the logs.
i'd really like to try you Addon because i hope to get information from my gigaset gateway.

Hey @Dud3z
Thank you for the interest on this.
I have a long overdue upgrade for this. Could you give me until next weekend?

Hi @maovidal,
absolutely! Thank you!

Hey @Dud3z
I have published the 0.1.1 version that should fix that error.
Could you upgrade and test please?


Not working with route consraints in attribute routing

It gives unsupported api version error instead of real error.

Can you provide some additional details? Are you using Web API or Core? Do you have an example of your route constraint? What does your route template look like? What was the real error you were expecting? Do you have a repro?

Since you haven't provided any additional input, I'm not sure how to help. From what I can discern, it seems you are experiencing the same behavior that is being discussed in #105. Feel free to join that conversation.
There is a known bug where 400 is returned instead of 405.  This was reported in #65 and has been fixed, but it has not yet been released. That will go out as part of the official 1.1.0 release. There is a beta version of the packages available now; otherwise, you can use the solution I provided in #105.
Thanks


revert bump of pytest-homeassistant-custom-component

does not fix the HACS load check failure, so cancelling


Update minecraft-ftb-fabric.json

Corrected install command.
 Error: unable to parse filename: linux
--auto - Ask no questions, use defaults.
--path - Directory to install in. Default: current directory
--noscript - Skip creating start script. Default: false
--nojava - Skip downloading a compatible Adoptium JRE. Default: false
--threads - Number of threads to use for downloading. Default: cpucores * 2Do a full integrity check, even on files not changed by the update. integrityupdate assumed. Default: true
--integrityupdate - Whether changed files should be overwritten with fresh copies when updating. Most useful when used with Auto. Default: false
Example: You changed config/test.cfg on your server from default. The modpack updates config/test.cfg - with this flag, it will assume you wish to overwrite with the latest version
--integrity -   --latest -   --verbose - Be a bit noisier on actions taken. Default: false
Install latest, ignoring any version in the file name or arguments. Default: false
--curseforge - Specifies that pack is a Curseforge modpack
--help - This help

Would it be worth setting the --nojava flag as well? Being that Pufferpanel is already downloading java in a previous step.

Would it be worth setting the --nojava flag as well? Being that Pufferpanel is already downloading java in a previous step. If so, the other Fabric templates should have that set too.

Better in other PR so it can be done in all FTB templates and be easier to keep track of

Oh... because they use the filename..... that's why I didn't see the issue on Windows....


Fix stack underflows when yielding twice

Also, refactor GDScriptFunctionState::_signal_callback, removing some excessive repetition.
Fixes #30269.

@bojidar-bg REALLY THANK YOU VERY VERY MUCH !!!
this bug was stopping us from moving further in the Project !!

Thanks!


Updating 6to5 to v3.0.10

Because I want to have these things. There are a number of breaking changes. Not sure what impact they'll have on jsbin. @sebmck?

No API changes other than the transformers being renamed and a few options removed.


failed to install mongodb on WSL

When following instructions in this tutorial for mongodb
"sudo apt-get mongodb" gives error "Package mongodb has no installation candidate"
WSL running with pengwin distro
Enter feedback here]

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

ID: bb282339-e2e5-ae64-4d76-3c999828f4e4
Version Independent ID: f7c88e68-43a2-bc01-73c3-a64b2aa198fd
Content: Get started using Python on Windows with a database
Content Source: hub/python/databases.md
Product: dev-environment
Technology: windows-python
GitHub Login: @mattwojo
Microsoft Alias: mattwoj

Hmm.. thanks @MaxwellWibert. I don't currently have Pengwin but just requested a copy to test. I found this issue in the Pengwin repo: https://github.com/WhitewaterFoundry/Pengwin/issues/469
They suggest trying an update first:
sudo apt update && sudo apt install mongodb
Could you give this a try and let me know if it resolves the problem?
Thanks again,
Matt

@mattwojo I'm afraid that didn't work. same error message after the update finishes

Bummer. I got a copy of Pengwin installed and took a look. I was able to install MongoDB in Pengwin following the instructions for Debian (which Pengwin is based on) from the Mongo site: https://docs.mongodb.com/manual/tutorial/install-mongodb-on-debian/
Basically ran this: echo "deb http://repo.mongodb.org/apt/debian buster/mongodb-org/4.2 main" | sudo tee /etc/apt/sources.list.d/mongodb-org-4.2.list
Then ran sudo apt-get update and sudo apt-get install -y mongodb-org
This seems to have successfully installed. I still had some troubles starting the service though.
The install seems to work fine on WSL in other distros, so I'm closing this issue and recommend filing eiher filing a new issue with the Pengwin repo or adding on to the exitsting issue that I linked above as it looks like an update on their end: https://github.com/WhitewaterFoundry/Pengwin
Thanks for bringing this to our attention. I let the WSL PM know as well.


cannot find module @icons/material/UnfoldMoreHorizontalIcon

After npm installing react-color, the following line of code:
import { CirclePicker } from 'react-color';
seems to break everything. The error message reads:

cannot find module @icons/material/UnfoldMoreHorizontalIcon

During npm install, the following dependencies were skipped:
SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

I don't know if those dependencies are important, and if yes how to install them? I have manually re-installed @icons/material but this did not solve my issue.
Thank you in advance.

What version of the package are you using? The CirclePicker does not use any icons so I am not sure why that would be giving you errors. Maybe try importing the picker directly:
import CirclePicker from 'react-color/lib/Circle'
``

Importing it directly fixed it, and it works great now! Thank you!


Best way to run applier on multiple repos/inventories from one playbook?

I have several git repositories with a single app in each. I want to aggregate the deployment of each of these repos using the applier. What is the best way to accomplish this?
Can the openshift_cluster_content variable be reset between applier runs from a single playbook?
Related to #80

I believe there are a couple of ways you can attempt to do this through the inventory:

use a unique host or group to represent your target repo, then define openshift_cluster_content in each respective host_vars or group_vars. You'd need to create a playbook similar to the one that's already in this repo, but remove the [0] from the target hosts to ensure it utilizes all your targets. IMHO, this is pushing the concept of a "host" a bit, but I don't see why it wouldn't work.

I've not tried this myself, but you should be able to use the combine filter with openshift_cluster_content definitions from multiple inventories - then just specify multiple -i during the execution. Note that this requires that the content is unique as it doesn't know how to "merge" the content in a smart way (I believe it's typically last-one-wins).


[Enhancement] Add the possibility of defining CLICK_ATTR as parameter

Being ngClick the default CLICK_ATTR but being able to use another directive instead.
I have a custom directive for clicking and doing this really helps.

I made the attributes configurable. Maybe you can check it out and tell me if it works for you.

Thank you very much, @johannesjo !! Now I can use it without modifying your directive.


Content changes for UKVI

Clarified when you can come to the UK without a visa, and what documents you need to bring with you. Also made smaller changes for style, and to correct a factual error (amount of savings you need evidence for).

This supersedes #2465

@ikennaokpala: Can this be reviewed given our recent change in workflow?

@chrisroos Thanks for raising this.. I feel best to close the pull request


Fix ie11 issues

This PR fixes two IE11 issues as described in #15 .
The PR improves the user agent check, but it is not future proof. Is setting the xmlns attribute twice really necessary in other browsers?

Regarding setting xmlns twice. I tested on an older Chromium browser and the icons would not show if I disabled the check for userAgent. In the newest version of Chrome it works. Now I understand that it is necessary for backward compatibility.


[AutoRelease] t2-digitaltwins-2023-02-28-98328(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/3837
Live test fail, detailed info is in pipeline log(search keyword FAILED)!!!
https://dev.azure.com/azure-sdk/internal/_build?definitionId=1744
BuildTargetingString
azure-mgmt-digitaltwins
Skip.CreateApiReview
true

issue link:https://github.com/Azure/sdk-release-request/issues/3837


UNB: Attribute sequenceNumber must be between 1 and 99999999

The above error is caused by the attached request
5910_663_Messages_1.zip

@ejudgemeddbase following segment
UNB+UNOC:3+110000381000005:80+500023147100005:80+110216:0934+1000000001++MEDRPT++1'
has interchange sequence number set to<PHONE_NUMBER>
Based on our previous experience in another EDIFACT based project (NHAIS), this value must be between 1 and 99999999

Thanks.


tf.estimator.Estimator.train fails with 3+ keras layer calls and MirroredStrategy distribution

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
linux ubuntu 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
binary (pip install tensorflow-gpu==1.8)
TensorFlow version (use command below):
1.8.0
Python version:
2.7.12
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
9.0.176
GPU model and memory:
GeForce GTX 1080 Ti (11gb memory)

You can collect some of this information using our environment capture script
You can also obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the current behavior
When applying a keras layer 3 or more times using tf.contrib.distribute.MirroredStrategy within the run config, training with a tensorflow estimator hits an exception when applying the layer at graph build time:
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/keras/_impl/keras/engine/base_layer.py", line 314, in __call__
    output = super(Layer, self).__call__(inputs, *args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/layers/base.py", line 671, in __call__
    with scope_context_manager as scope:
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variable_scope.py", line 1962, in __enter__
    self._current_name_scope.__enter__()
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 5974, in __enter__
    % (self._name, self._default_name))
ValueError: At least one of name (None) and default_name (None) must be provided.

additional observations:

runs fine if the layer is applied 2 times (using MirroredStrategy)
runs fine if the layer is applied 3 times (using OneDeviceStrategy)

Describe the expected behavior
estimator.train should succeed with multiple gpus and multiple applications of a keras layer.
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
import numpy as np
import tensorflow as tf

class ConvLayer(tf.keras.layers.Layer):
    def call(self, image):
        shape = (3, 3, 3, 16)
        stddev = 1
        weights = tf.get_variable(
            name='weights',
            initializer=tf.truncated_normal(shape, stddev=stddev),
        )
        conv = tf.nn.conv2d(image, weights, [1, 1, 1, 1], 'SAME')
        return conv

def get_train_op(loss):
    global_step = tf.train.get_global_step()
    optimizer = tf.contrib.optimizer_v2.AdamOptimizer(.001)  # use distribution-aware optimizer
    train_op = optimizer.minimize(loss, global_step=global_step)
    return train_op

def model_fn(features, labels, mode):
    image = features['image']

    # if num_layers >= 3, graph construction will error if more than 1 gpu is used
    num_layers = 3
    layer = ConvLayer()
    values = []
    for _ in xrange(num_layers):
        values.append(layer(image))
    stacked_values = tf.concat(values, axis=0)

    loss = tf.reduce_sum(stacked_values)
    mode = tf.estimator.ModeKeys.TRAIN
    train_op = get_train_op(loss)
    return tf.estimator.EstimatorSpec(mode, loss=loss, train_op=train_op)

def input_fn():

    shape = (100, 100, 3)

    def gen():
        while True:
            image = np.ones(shape, dtype=np.float32)
            yield ({'image': image}, [])

    ds = tf.data.Dataset.from_generator(
        gen,
        ({'image': tf.float32}, tf.float32),
        output_shapes=({'image': tf.TensorShape(shape)}, tf.TensorShape(None)),
    )
    ds = ds.repeat().batch(4)
    return ds

# top-level call
def run():
    gpus = ['/device:GPU:0', '/device:GPU:1']
    # MirroredStrategy fails
    distribution = tf.contrib.distribute.MirroredStrategy(gpus)
    # OneDeviceStrategy works
    # distribution = tf.contrib.distribute.OneDeviceStrategy(gpus[0])

    config = tf.estimator.RunConfig(
        train_distribute=distribution,
        model_dir='/path/to/output'
    )
    estimator = tf.estimator.Estimator(
        model_fn=model_fn,
        config=config,
    )
    estimator.train(
        input_fn=input_fn,
        steps=10,
    )

calling run() after setting num_layers to 1 or 2 works, but fails if num_layers >= 3.
Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/coordinator.py", line 297, in stop_on_exception
    yield
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/contrib/distribute/python/mirrored_strategy.py", line 465, in run
    self.main_result = self.main_fn(*self.main_args, **self.main_kwargs)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/estimator/estimator.py", line 831, in _call_model_fn
    model_fn_results = self._model_fn(features=features, **kwargs)
  File "scott_exp/multigpu.py", line 31, in model_fn
    values.append(layer(image))
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/keras/_impl/keras/engine/base_layer.py", line 314, in __call__
    output = super(Layer, self).__call__(inputs, *args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/layers/base.py", line 671, in __call__
    with scope_context_manager as scope:
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/variable_scope.py", line 1962, in __enter__
    self._current_name_scope.__enter__()
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 5974, in __enter__
    % (self._name, self._default_name))
ValueError: At least one of name (None) and default_name (None) must be provided.

full stacktrace: https://gist.github.com/dogbox/dcceda3d7b0c813c139258ea436e73f1

just noting that this issue isn't pressing - i've been able to work around this by explicitly setting variable scopes instead:
class ConvLayer(object):
    def __call__(self, image):
        with tf.variable_scope(self.name, reuse=tf.AUTO_REUSE):
            shape = (3, 3, 3, 16)
            stddev = 1
            weights = tf.get_variable(
                name='weights',
                initializer=tf.truncated_normal(shape, stddev=stddev),
            )
            conv = tf.nn.conv2d(image, weights, [1, 1, 1, 1], 'SAME')
            return conv

it'd still be nice to use keras layers though, especially since this approach seems to be discouraged [https://github.com/tensorflow/tensorflow/issues/13895#issuecomment-364997554].

@joshl This user request is about specific combination where 3+ layers of Keras model are imported into estimators and a distribution strategy is employed. Can you please review this condition?

You are using a Keras Layer.  Please consider the documentation for writing your own layers, including the provided example: https://keras.io/layers/writing-your-own-keras-layers/.  In particular, the weights need to be define as part of build() rather than call().  I'd try that.  Please re-open if that doesn't resolve your issue.


Conversations screen: re-enable favorites use

User Story android-sp1-6
1- Add the option "épinglage" in the context menu displayed on the selected room.
2- Pinned rooms are marked as favorites, and they are displayed at the top of the rooms lists
3- A badge should be added on the pinned rooms (input required here)
4- The option "quitter la discussion" is renamed "quitter la conversation"

page_1_2018-03-08.zip


Templates for issues

We need to do this.
Maybe we can auto-add a Label like "I didn't search first" and then auto-close any issues that don't have the label removed.
Any other ideas?

That's a great idea. It looks like there is also a REST API for editing issues: https://developer.github.com/v3/issues/#edit-an-issue
Maybe we could create a new bot github user that watches this repo. When a new issue get's posted it'll get an e-mail. The bot would parse the body of the issue looking for the "I didn't search first" tag. If it sees that it will used the issue API to post a standard reply explaining why the issue was closed and then close the issue.
The next question would be where the script would live. Ideally github would have a place to host python scripts, but I haven't found anything like that yet. Barring that I could setup a cheap linux server on AWS to host the script and e-mail server that the bot would listen to.


AlertView does not release from memory

After opening an alert view it is retained in memory and not deinitialized. A simple test is to put a print statement inside of  deinit in SCLAertView and it never gets called. This leads to multiple zombie instances of alertview objects floating around. Most noticeable when interacting with NSNotificationCenter for Keyboard events. Instances start to build up.

Nice catch. Why not fixing this by yourself?

@nbwar feel free to submit pull request. Thanks

I am seeing this issue as well. I don't see any circular references that jump out at me, but will take a further look in a bit.

Any updates about this ?

Is there any update yet?

Any update on this? This is really a tricky issue. In my case (i guess for everyone), because the alertView is not deinitialized also the ViewController showing the alert  is not reinitialised from memory, which could cause huge memory leaks in Apps.
Tried to find the cause, but no luck. Even tried setting the alertView to nil on viewWillDisappear of the calling VC, but that didn't do it either.

Did a bit more digging after I commented and FOUND IT!
It's the actions / target,selectors of the buttons.
Add below code to the completion block of the hideView - AnimateWithDuration function:
for button in self.buttons {
button.action = nil
button.target = nil
button.selector = nil
}

@guidove - show some open-source love and submit a pull req if you have it fixed.

Just submitted. It's my first pull request on github so hope I did it correct.

#125

BOOM! Nice work @guidove - thanks for the quick re: @vikmeup

Nice!, update the Pod please

I think this fix is to blame for issue #130.

error still going on
Xcode 7.2.1
Occurred:  clicking the button more than once


Getting stock data from a particular time of the day using start and end from Ticker.History

Please can I get data of stock for a particular time of the day like so start='2022-10-19 14:42:41' end='2022-10-19 14:52:41'

Use datetime.datetime(...). If you really need the string option then could you implement and submit a PR?

Thanks @ValueRaider  I'll that out


Is this Reshape step redundant?

See this line of code:  https://github.com/philipperemy/keras-attention-mechanism/blob/master/attention_lstm.py#L19
Isnt this redundant?  Because the Permute layer right before it will reshape the Tensor.
Let me know if I'm missing something.  I am trying to understand attention and thus far your writeup is helping

@philipperemy
Also you don't need this line of code:
https://github.com/philipperemy/keras-attention-mechanism/blob/master/attention_lstm.py#L25
you can pass name = 'name' in any layer

@hamelsmu yes the Reshape layer is redundant and does not add any value to the model. Everything is done by the Permute layer.
It's more to enforce the correct shape. The output of the Permute layer is (?, ?) and by adding this Reshape layer, we make it more clear about the real shapes (they are static and known at compilation time).

Thanks for your feedback! Highly appreciated!
a = Dense(TIME_STEPS, activation='softmax', name='attention_vec')(a)
if SINGLE_ATTENTION_VECTOR:
    a = Lambda(lambda x: K.mean(x, axis=1), name='attention_vec')(a)  # this is the attention vector!
    a = RepeatVector(input_dim)(a)

Is this what you meant? Removing the Else clause and adding name='attention_vec' before the If?

Yeah thats right

It does not work lie this because we define different layers with the same names twice.
RuntimeError: The name "attention_vec" is used 2 times in the model. All layer names should be unique.

@philipperemy right.  However I suppose you can say that the attention layer is aprobs because that is the layer that is being multiplied by the inputs.   So you can re-factor to look like this:
    a = Dense(TIME_STEPS, activation='softmax')(a)
    if SINGLE_ATTENTION_VECTOR:
        a = Lambda(lambda x: K.mean(x, axis=1), name='dim_reduction')(a)
        a = RepeatVector(input_dim)(a, name='time_repeat')

    a_probs = Permute((2, 1), name='attention_vec')(a)
    output_attention_mul = merge([inputs, a_probs], name='attention_mul', mode='mul')

Ok seems good for me! The only thing is that attention_vec.shape will change from (1, 2, 20) to (1, 20, 2), where 20 is the number of time steps, and 2 the number of input dims. So we have to change the axis from 1 to 2 (on which we aggregate). Simply because we want to display the vector for the time axis.
attention_vector = np.mean(get_activations(m,
                                                   testing_inputs_1,
                                                   print_shape_only=True,
                                                   layer_name='attention_vec')[0], axis=2).squeeze()

Let me know if it seems good for you:
https://github.com/philipperemy/keras-attention-mechanism/pull/4

Thanks!


Add snapshot tests for ClearResults, DeleteKeys, and RegisterKeys gRPC methods

Motivation
We want snapshot tests for all of the zebra-scan gRPC methods.

I dont think we need snapshot tests for the grpc calls that return empty responses. I do think we should add these to unit tests (i am working on it in another PR as part of  https://github.com/ZcashFoundation/zebra/issues/8244)

https://github.com/ZcashFoundation/zebra/pull/8293


*ignore FIX WIP

I just reworked my changes in apache/cordova-create#8 to work on master, with better test refactoring, as you requested in https://github.com/apache/cordova-discuss/issues/69#issuecomment-392765587. Changes are in 2 commits and I would recommend we keep it that way:

spec (test) updates needed
actual change

It still copies from .npmignore to .gitignore if possible, which I think should be really easy to adapt to the needs discussed in apache/cordova-discuss#69.
This change should be considered WIP, with XXX TODO marked to use fs-extra instead of shelljs once apache/cordova-create#14 is merged. Please feel free to use it for your work on apache/cordova-discuss#69 if you like.
Just finished a long day working on this:)

Closing, I just pushed the updates to apache/cordova-create#8.


no audio no sound,probook 4540s,big Sur

hi Chris
I installed Big Sur(11,2,1) using your guide.
every thing work perfect except aud
EFI.zip
io

![Screen Shot 1400-08-11 at 14 03 39](https://user-images.githubusercontent.com/69670261/139830859-2bd8f33e-0de9-4d0e-9890-b17f5f85fbb0.png

)

hi Chris
I installed Big Sur(11,2,1) using your guide.
every thing work perfect except aud
EFI.zip
io

Solved
I change layout ID from 12 to 13 and it worked.
please change layout id for Probook 4540s to 13 in for guide and package.

And another one:
for Probook 4540s bios version F.68 Rev.A works perfect.


Rawkv gctask metrics

Signed-off-by: ystaticy<EMAIL_ADDRESS>
What is changed and how it works?

Issue Number: Ref https://github.com/tikv/tikv/issues/12557
What's Changed:

RawKV GCTask return handled and wasted count

Check List
Tests

Manual test
If the data is the latest version is deleted or expired and to Compcation,it will trigger GcTask::RawGcKeys
We can get tikv_gc_compaction_filter_mvcc_deletion_handled value is not 0 in grafana

LGTM. Not sure whether using the same metrics as transactional GC is a good idea though.

I think it would be better to separate metrics for different KV modes later.

LGTM. Not sure whether using the same metrics as transactional GC is a good idea though.

I think it would be better to separate metrics for different KV modes later.

Ok, the metrics will be separate later.

/merge


Adding allowed algorithms to verify

See #25

This is a first approach to fix: Critical vulnerabilities for JWT in jws-2.0.js and jws-3.0.js
Based on: https://github.com/davedoesdev/node-jsjws/blob/ac456b5f9bb96695f1dd838acc8f089154b2da04/patches/jsjws.patch
Action items that must be addressed before merging:

[ ]  An automated test suite is required to make sure this work as expected. I've tried it manually but it's not good enough for the size of the change. There might be some cases that I'm missing.
[ ] Remove jws-2.0 and jws-3.0 files create jws-4.0 deprecating old methods.
[ ] Sample 3 should work as before: The fields in the HTML should print the JWT header, payload, etc.

I'm a bit worried about the health of the library. Some suggestions for the future:

[ ] Decide whether \t or   is the prefered identation. Currently, it's mixed in the same file.
[ ] Cleanup of versions: Use git to store library versions. Having all the versions at the same time in the master branch is confusing (ie. jws-2.0 vs. jws-3.0).
[ ] Move samples to a sample folder. Same with tools and minified library versions.
[ ] Split jws into different files.
[ ] Add a .jshintrc to improve maintainability.

Let me know If I can help with anything else. Thanks!

@kjur - Are you still working on jsjws?
@pose - In node-jsjws, I stopped at jws-2.0 because the abstractions in jws-3.0 prevented me getting in and achieving speedups on Node. In that respect, I regard node-jsjws as having forked jsjws but anything I can feed back into jsjws I'm only too happy to do so.

I've removed alg=none support in the verify method. Thank you for your support.


add thrid party login -- GitHub and LinkedIn

add thrid party login -- GitHub and LinkedIn.

Hi @yihong0618,
Thank you for your contribution. I have one idea about the login experience, please let me know your thoughts on this:
We still have only one entry(command) for the login, but when users trigger the command, a drop down list pops up, having the following options:

LeetCode Account
LeetCode Cookie
Third-Party: GitHub
Third-Party: LinkedIn

By doing this, the shortcut in the explorer would be applied to all kinds of login methods. And users won't need to remember different login commands. (Imagine that we might support more 3rd-party login in the future).

@jdneo
You are right I also thinked of that, can I work on it this weekend? And update the code?

Sure, thank you!

@jdneo vscode-leetcode 2.6.20 not found ~

My bad! It should be available right now!

@jdneo
Thank you for your review, the naming problems had changed, and please note that it is not support for leetcode-cn.com for GitHub and LinkedIn login in leetcode-cli 2.6.20 and I raised a commit in this pr to support it.

2.6.21 has been published to npm

2.6.21 has been published to npm

Try again, two-factor seems can not support in our way in vscode-leetcode, because maybe the async to sync way in request is using read fs but in vscode terminal doesn't support. I will try to figure out another if i could. Thank you.

Can you just note users that we are not support two-factor in vscode-leetcode for now, if they want they can use cli to login first.
Thank you very much.

Thanks @yihong0618, let me also take a look first.

Thanks @yihong0618, let me also take a look first.

Thank you, that would be wonderful.
FYI
I am wrong that is zsh problem.

If I comment this line
                if (!pwd) {
                    childProc.kill();
                    return resolve(undefined);
                }
                childProc.stdin.write(`${pwd}\n`);
                // childProc.stdin.end();

vscode side can receive the "two-factor" message from cli side.
But I have problem with how to use the vscode.window.showInputBox

Sorry my laptop broke down last weekend and I have to send it to the store for repairment. I will try to resolve this PR in this week.

@yihong0618
You can write something like this:
childProc.stdout.on("data", async (data: string | Buffer) => {
    data = data.toString();
    leetCodeChannel.append(data);
    if (data.includes("two-factor")) {
        const twoFactor: string | undefined = await vscode.window.showInputBox({
            prompt: "Enter two-factor code.",
            validateInput: (s: string): string | undefined => s && s.trim() ? undefined : "The input must not be empty",
        });
        if (!twoFactor) {
            childProc.kill();
            return resolve(undefined);
        }
        childProc.stdin.write(`${twoFactor}\n`);
    } else {
        const match: RegExpMatchArray | null = data.match(/(?:.*) Successfully (login|cookie login|third party login) as (.*)/i);
        if (match && match[2]) {
            return resolve(match[2]);
        }
    }
});

But I found that there is some problems in the CLI side.

prompt-sync is declared in the devDependencies, which means the module will be missing in the extension side
Why not using prompt but prompt-sync?

@yihong0618
You can write something like this:
childProc.stdout.on("data", async (data: string | Buffer) => {
    data = data.toString();
    leetCodeChannel.append(data);
    if (data.includes("two-factor")) {
        const twoFactor: string | undefined = await vscode.window.showInputBox({
            prompt: "Enter two-factor code.",
            validateInput: (s: string): string | undefined => s && s.trim() ? undefined : "The input must not be empty",
        });
        if (!twoFactor) {
            childProc.kill();
            return resolve(undefined);
        }
        childProc.stdin.write(`${twoFactor}\n`);
    } else {
        const match: RegExpMatchArray | null = data.match(/(?:.*) Successfully (login|cookie login|third party login) as (.*)/i);
        if (match && match[2]) {
            return resolve(match[2]);
        }
    }
});

But I found that there is some problems in the CLI side.

prompt-sync is declared in the devDependencies, which means the module will be missing in the extension side
Why not using prompt but prompt-sync?

Because all request with session are in callback，and prompt must in the sync way. Otherwide it will fall through。
2. I will try tonight thank you
3. I also found that~ will also fix

@jdneo
Sorry, maybe I missed somthing, it didn't work on my win10 with this way.

@yihong0618, not quite get your point. Every async can should be able to change to sync from my understanding.
You can simply change it to:
if (resp.request.uri.href !== urls.github_tf_redirect) {
  return requestLeetcodeAndSave(_request, leetcodeUrl, user, cb);
}
prompt.colors = false;
prompt.message = '';
prompt.start();
prompt.get([
  {
    name:     'code',
    required: true
  }
], function(e, result) {
  if (e) return log.fail(e);
  const authenticityTokenTwoFactor = body.match(/name="authenticity_token" value="(.*?)"/);
  if (authenticityTokenTwoFactor === null) {
    return cb('Get GitHub two-factor token failed');
  }
  const optionsTwoFactor = {
    url:     urls.github_tf_session_request,
    method:  'POST',
    headers: {
      'Content-Type': 'application/x-www-form-urlencoded',
    },
    followAllRedirects: true,
    form:               {
      'otp':                result.code,
      'authenticity_token': authenticityTokenTwoFactor[1],
      'utf8':               encodeURIComponent('✓'),
    },
  };
  _request(optionsTwoFactor, function(e, resp, body) {
    if (resp.request.uri.href === urls.github_tf_session_request) {
      return cb('Invalid two-factor code please check');
    }
    requestLeetcodeAndSave(_request, leetcodeUrl, user, cb);
  });
});

The prompt-sync is too buggy. I'm not in favor of using that module.
By using the above code, then you can update the extension side, with following two callbacks change:
childProc.stdout.on("data", async (data: string | Buffer) => {
    data = data.toString();
    leetCodeChannel.append(data);
    if (data.includes("twoFactorCode")) {
        const twoFactor: string | undefined = await vscode.window.showInputBox({
            prompt: "Enter two-factor code.",
            validateInput: (s: string): string | undefined => s && s.trim() ? undefined : "The input must not be empty",
        });
        if (!twoFactor) {
            childProc.kill();
            return resolve(undefined);
        }
        childProc.stdin.write(`${twoFactor}\n`);
        childProc.stdin.end();
    } else {
        const match: RegExpMatchArray | null = data.match(/(?:.*)Successfully .*login as (.*)/i);
        if (match && match[1]) {
            childProc.stdin.end();
            return resolve(match[1]);
        }
    }
});

&
childProc.on("close", (code: number) => {
    if (code !== 0) {
        reject(new Error("Failed to login.`"));
    }
});

@jdneo
You are right. Because I put the GitHub login without two-factor in the end of the code block, so it would fall through using prompt. And I think you are right. Really learned a lot from your project and the JS(TS), thank you. I will fix these tonight.

@yihong0618 No worry, I should thank you for all of the contributions you have made.
Will take a look at the PR recently.


[FEATURE] GPTQ VectorQuantMatmul Kernel Documentation

Is it possible to include inline comments or docstrings for the quant kernel implementations?
For example, starting from this line in autogptq_cuda_kernel_256.cu of the refactored VectorQuant3MatMul kernel, there seems to be hard-coded numbers for mapping threads to scale / zero groups.  Trying to make sense of how thread idxing changes depending on the quantization bit-width.
Thanks!

I'm not sure who wrote those kernels but it would indeed be nice.


python3Packages.pixelmatch: disable mypy

Motivation for this change
Fix build (https://hydra.nixos.org/build/162948569)
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 153968 run on x86_64-linux 1

2 packages built:

python38Packages.pixelmatch
python39Packages.pixelmatch

@ofborg eval


getActiveWindow Dont work

var processWindows = require("node-process-windows");

processWindows.getActiveWindow((err, processInfo) => {

    if (err) console.log(err)

    console.log("Active window title: " + processInfo.mainWindowTitle);
});

Dont log anything

Reading into the code, I found out this method does nothing, even though it's demoed in the Readme.
function getActiveWindow(callback) {
    callback = callback || noop;

    if (!isWindows) {
        callback("Non-windows platforms are currently not supported");
    }
}

It never executes the subprocess or calls callback(). All of the code to do this is pretty much there, and I think the creator might have just forgotten to commit the final version or something. Anyway, the last update was 2 years ago, so I'm making a fork to fix this.
If you want to fix the module yourself in the mean time, change the function to
function getActiveWindow(callback) {
    executeProcess("--activewindow", callback)
}

And copy windows-console-app.exe from node-process-windows/windows-console-app/windows-console-app/bin/Debug to /bin/Release. This is what makes me think this is a mistake, the fully built binary is in Debug, it just hasn't been copied to Release to implement the --activewindow feature.


Allow material-ui 0.18.0 compatibility

npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^0.16.4 || ^0.17.0 but none was installed.

Thanks @mcbain
I think it's time to update peer dependencies!

fixed in<EMAIL_ADDRESS>


this is unfortunatly not working anymore

Logger: homeassistant.config_entries
Source: custom_components/systemair/init.py:98
Integration: Systemair (documentation, issues)
First occurred: 09:16:02 (2 occurrences)
Last logged: 10:17:06
Error setting up entry Systemair for systemair
Error setting up entry ************* for systemair
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 372, in async_setup
result = await component.async_setup_entry(hass, self)
File "/config/custom_components/systemair/init.py", line 65, in async_setup_entry
sc_devices = await save_connect_device_setup(hass, api)
File "/config/custom_components/systemair/init.py", line 98, in save_connect_device_setup
sc_devices = await api.get_devices(update=True, fetch_device_info=False)
File "/usr/local/lib/python3.10/site-packages/systemair/saveconnect/systemair.py", line 289, in get_devices
devices = await self.graphql.queryGetAccount()
File "/usr/local/lib/python3.10/site-packages/systemair/saveconnect/graphql.py", line 156, in queryGetAccount
response_data = await self.post_request(
File "/usr/local/lib/python3.10/site-packages/systemair/saveconnect/graphql.py", line 206, in post_request
response_data = response.json()["data"]
KeyError: 'data'

Same problem for me :(
Any solution in sight?

I currently have little/no time to fix this issues. It would be awesome if
someone contributed a PR to fix it :)
Per-Arne Andersen, Ph.D
Associate Professor
Artificial Intelligence and Cybersecurity, University of Agder
Center for Artificial Intelligence Research (CAIR)
Centre for Integrated Emergency Management (CIEM )
Adviser for SSHF in Artificial Intelligence
+47 905 315 06 <+47+905+315+06> | @.*** | @.***
https://www. https://www.sysx.no/per-arne.no
Spikkelia 28 4821 Rykene, Norway
http://scholar.google.no/citations?user=oPth46oAAAAJ&hl=no
http://github.com/perara http://www.facebook.com/pereher
http://www.linkedin.com/in/perara http://twitter.com/PerArneAndersen
https://scholar.google.no/citations?user=oPth46oAAAAJ&hl=no
@.?anonymous&ep=signature>
Reserver
et tidspunkt for å møte meg
@.?anonymous&ep=signature>
man. 30. okt. 2023 kl. 13:34 skrev michalkrzempek @.***

:

@perara https://github.com/perara Could you please let us know If you
have plans to fix it?
—
Reply to this email directly, view it on GitHub
https://github.com/perara/systemair-save-connect/issues/3#issuecomment-1785091695,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4ZS7VODVDHUCGGXMAYBSDYB6NEVAVCNFSM6AAAAAAS2E3AFCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBVGA4TCNRZGU
.
You are receiving this because you were mentioned.Message ID:
@.***>


Feature #26: Add Travis CI and CodeCov support

Resolves #26 Travis CI support.
Summary

Adds Travis CI and CodeCov support
Adds targets for iOS and tvOS tests, adds them to appropriate schemes
Adds Fastlane to make running test suite more CI independent
Adds Travis and CodeCov configuration
Updates Readme to add build status and coverage analysis

cc: @lucastrazzullo @madhikarma

Current coverage is 55.79% (diff: 100%)

No coverage report found for develop at e3a6129.

Powered by Codecov. Last update e3a6129...016ceb7