Preload hangs "Reading image information"

resin-cli version:
resin-cli: 7.4.0
node: 8.9.4
Operating system and architecture:
Centos 7 (x86_64)

I am attempting to preload an OS image with a specific app commit but the command just hangs with
"Reading image information" that just keeps spinning.
Below are the commands I am basically running:
Create config.json
resin config generate --application $APP_NAME --network ethernet --appUpdatePollInterval 10 --output ./config.json
Download OS for device type
resin os download $DEVICE_TYPE -o ./resin.img --version default
Register new device
resin device register $APP_NAME --uuid $DEVICE_UUID
Configure OS image for new device
resin os configure ./resin.img --device $DEVICE_UUID --config ./config.json
Preload app version onto image
chmod 664 ./resin.img resin preload ./resin.img --app $APP_ID --commit $APP_COMMIT
^ hangs...
Initialize and flash OS
resin os initialize ./resin.img --type $DEVICE_TYPE --drive /dev/disk2

Hello @adamtec ,
Can you please rerun resin preload with debug enabled and paste the output here ?
DEBUG=1 resin preload ./resin.img --app $APP_ID --commit $APP_COMMIT

Below is a dump of the debug output. It appears preload.py is trying to use aufs for Docker storage device. Checking Docker it doesn't have aufs enabled but instead overlay/overlay2.
`
[vagrant@localhost Downloads]$ DEBUG=1 resin preload resin.img --app $APP_ID --commit $APP_COMMIT
Building Docker preloader image. [===                     ] 12%
Step 1/7 : FROM docker:17.10.0-ce-dind
Building Docker preloader image. [======                  ] 25%
Step 2/7 : RUN apk update && apk add --no-cache python3 parted btrfs-progs docker util-linux sfdisk file coreutils sgdisk
---> Using cache
Building Docker preloader image. [=========               ] 37%
Step 3/7 : COPY ./requirements.txt /tmp/
---> Using cache
Building Docker preloader image. [============            ] 50%
Step 4/7 : RUN pip3 install -r /tmp/requirements.txt
---> Using cache
Building Docker preloader image. [===============         ] 62%
Step 5/7 : COPY ./src /usr/src/app
---> Using cache
Building Docker preloader image. [==================      ] 75%
Step 6/7 : WORKDIR /usr/src/app
---> Using cache
Building Docker preloader image. [=====================   ] 87%
Step 7/7 : CMD /usr/src/app/preload.py
---> Using cache
---> 2fcaf4b82cbb
Successfully built 2fcaf4b82cbb
Building Docker preloader image. [========================] 100%

Starting preloader container

Fetching application 1011056
\ Reading image informationWaiting for Docker to start...

Reading image informationException in thread background thread for pid 110:
Traceback (most recent call last):
File "/usr/lib/python3.6/threading.py", line 916, in _bootstrap_inner
self.run()
File "/usr/lib/python3.6/threading.py", line 864, in run
self._target(*self._args, **self._kwargs)
File "/usr/lib/python3.6/site-packages/sh.py", line 1540, in wrap
fn(*args, **kwargs)
File "/usr/lib/python3.6/site-packages/sh.py", line 2459, in background_thread
handle_exit_code(exit_code)
File "/usr/lib/python3.6/site-packages/sh.py", line 2157, in fn
return self.command.handle_command_exit_code(exit_code)
File "/usr/lib/python3.6/site-packages/sh.py", line 815, in handle_command_exit_code
raise exc
sh.ErrorReturnCode_1:
RAN: /usr/local/bin/dockerd --storage-driver=aufs --data-root=/tmp/tmp667dfl_m/docker --host=tcp://<IP_ADDRESS>:36382
STDOUT:
STDERR:
time="2018-05-10T23:55:50.365406139Z" level=warning msg="[!] DON'T BIND ON ANY IP ADDRESS WITHOUT setting --tlsverify IF YOU DON'T KNOW WHAT YOU'RE DOING [!]"
time="2018-05-10T23:55:50.376842324Z" level=info msg="libcontainerd: new containerd process, pid: 129"
Error starting daemon: error initializing graphdriver: driver not supported

Traceback (most recent call last):
File "/usr/src/app/preload.py", line 805, in
result = method(**data.get("parameters", {}))
File "/usr/src/app/preload.py", line 765, in get_image_info
images, supervisor_version = get_images_and_supervisor_version()
File "/usr/src/app/preload.py", line 653, in get_images_and_supervisor_version
return _get_images_and_supervisor_version()
File "/usr/src/app/preload.py", line 628, in _get_images_and_supervisor_version
\ Reading image information    with docker_context_manager(driver, mountpoint):
File "/usr/lib/python3.6/contextlib.py", line 82, in enter
return next(self.gen)
File "/usr/src/app/preload.py", line 511, in docker_context_manager
running_dockerd = start_docker_daemon(storage_driver, docker_dir)
File "/usr/src/app/preload.py", line 480, in start_docker_daemon
running_dockerd.wait()
File "/usr/lib/python3.6/site-packages/sh.py", line 792, in wait
self.handle_command_exit_code(exit_code)
File "/usr/lib/python3.6/site-packages/sh.py", line 815, in handle_command_exit_code
raise exc
sh.ErrorReturnCode_1:
RAN: /usr/local/bin/dockerd --storage-driver=aufs --data-root=/tmp/tmp667dfl_m/docker --host=tcp://<IP_ADDRESS>:36382
STDOUT:
STDERR:
time="2018-05-10T23:55:50.365406139Z" level=warning msg="[!] DON'T BIND ON ANY IP ADDRESS WITHOUT setting --tlsverify IF YOU DON'T KNOW WHAT YOU'RE DOING [!]"
time="2018-05-10T23:55:50.376842324Z" level=info msg="libcontainerd: new containerd process, pid: 129"
Error starting daemon: error initializing graphdriver: driver not supported
`

@adamtec the storage driver used depends on the device type being preloaded. Most use aufs.
If we preloaded this image using overlay2 as storage driver, the device wouldn't be able to run the preloaded container.
I'll keep this issue open so I can add an error message for this case.

Understood.
Resin OS utilizes balena which uses aufs as its storage driver, correct?
I see flags for preloading using a remote Docker daemon. I guess I can try that route.
What I am looking into is moving towards a production environment.
This requires flashing lots of SD cards pre-configured for a specific app and assigning a unique device UUID. Only flashing the OS means waiting for the target device to boot up and pull roughly 3-4 GB app images which takes a considerable amount of time on a RPI3 which is why I was looking into preload.

I'm seeing the same problem on Windows 10, although the debug option doesn't show an error..
PS C:\Dev\preload> $env:DEBUG=1
PS C:\Dev\preload> resin preload .\resin-.img --app 11111 --splash-image .\resin-logo.p
ng --dockerHost <IP_ADDRESS> --dockerPort 2375
Building Docker preloader image. [===                     ] 12%
Step 1/7 : FROM docker:17.10.0-ce-dind
Building Docker preloader image. [======                  ] 25%
Step 2/7 : RUN apk update && apk add --no-cache python3 parted btrfs-progs docker util-linux sfdisk file coreutils sgdisk
 ---> Using cache
Building Docker preloader image. [=========               ] 37%
Step 3/7 : COPY ./requirements.txt /tmp/
 ---> Using cache
Building Docker preloader image. [============            ] 50%
Step 4/7 : RUN pip3 install -r /tmp/requirements.txt
 ---> Using cache
Building Docker preloader image. [===============         ] 62%
Step 5/7 : COPY ./src /usr/src/app
 ---> Using cache
Building Docker preloader image. [==================      ] 75%
Step 6/7 : WORKDIR /usr/src/app
 ---> Using cache
Building Docker preloader image. [=====================   ] 87%
Step 7/7 : CMD /usr/src/app/preload.py
 ---> Using cache
 ---> 1beabafcf7ff
Successfully built 1beabafcf7ff
Building Docker preloader image. [========================] 100%

/ Creating preloader container
/ Starting preloader container
- Fetching application 11111
| Reading image information
\ Reading image information <this just continues to spin>

Nevermind, seems that I just needed to give it more time.

@jgentes can you try without the .\ before the image name and the splash image name?

Actually, I was wrong, it never completes. I tried without the .\ before the filenames and had the same result. The Pi can download all of the images in about 10 minutes, and that's much slower than my desktop. I've let it spin for at least 20 minutes with no progress.

Same issue here on Windows 10.

Thanks for the report @emilm, we'll take a look at that. It'd be useful to have a little more information though:

Which CLI version are you using?
Are you using the standalone build, or installing via npm?
If npm, which node version are you using?

It'd also be helpful to see the debug output from your machine. Can you run your command with a DEBUG=1 env var set, as in the example below, and paste the output here?
DEBUG=1 resin preload ./resin.img --app $APP_ID --commit $APP_COMMIT

(@zvin can you take a look into this?)

This may have already been in the latest resin-preload versions: 7.0.3 and 6.3.4

Sorry for late response but I have been in talks with resin.io team and they found a problem and fixed it. Also they fixed an issue related to older node versions, so I had to upgrade to 10.
After that it worked!

I'm using node v10.13.0, npm v6.4.1, installed with
npm install balena-cli -g --production --unsafe-perm on ubuntu 18.04. I'm seeing this problem and not sure how to solve it.
Step 1/7 : FROM docker:17.10.0-ce-dind
Building Docker preloader image. [======                  ] 25%
Step 2/7 : RUN apk update && apk add --no-cache python3 parted btrfs-progs docker util-linux sfdisk file coreutils sgdisk
 ---> Using cache
Building Docker preloader image. [=========               ] 37%
Step 3/7 : COPY ./requirements.txt /tmp/
 ---> Using cache
Building Docker preloader image. [============            ] 50%
Step 4/7 : RUN pip3 install -r /tmp/requirements.txt
 ---> Using cache
Building Docker preloader image. [===============         ] 62%
Step 5/7 : COPY ./src /usr/src/app
 ---> Using cache
Building Docker preloader image. [==================      ] 75%
Step 6/7 : WORKDIR /usr/src/app
 ---> Using cache
Building Docker preloader image. [=====================   ] 87%
Step 7/7 : CMD ["python3", "/usr/src/app/preload.py"]
 ---> Using cache
 ---> be8635137249
Successfully built be8635137249
Building Docker preloader image. [========================] 100%

\ Creating preloader container
- Starting preloader container
/ Fetching application 1252660
| Reading image informationTraceback (most recent call last):
  File "/usr/src/app/preload.py", line 821, in <module>
    result = method(**data.get("parameters", {}))
  File "/usr/src/app/preload.py", line 781, in get_image_info
    images, supervisor_version = get_images_and_supervisor_version()
  File "/usr/src/app/preload.py", line 663, in get_images_and_supervisor_version
    inner_image_path = get_inner_image_path(mountpoint)
  File "/usr/src/app/preload.py", line 621, in get_inner_image_path
    device_type = get_device_type()
  File "/usr/src/app/preload.py", line 601, in get_device_type
    result = get_json("flash-boot", "device-type.json", image=image)
  File "/usr/src/app/preload.py", line 590, in get_json
    with part.mount_context_manager() as mountpoint:
  File "/usr/lib/python3.6/contextlib.py", line 81, in __enter__
    return next(self.gen)
  File "/usr/src/app/preload.py", line 103, in mount_context_manager
    with losetup_context_manager(image, offset, size) as device:
  File "/usr/lib/python3.6/contextlib.py", line 81, in __enter__
    return next(self.gen)
  File "/usr/src/app/preload.py", line 87, in losetup_context_manager
    device = losetup(*args).stdout.decode("utf8").strip()
  File "/usr/lib/python3.6/site-packages/sh.py", line 1427, in __call__
    return RunningCommand(cmd, call_args, stdin, stdout, stderr)
  File "/usr/lib/python3.6/site-packages/sh.py", line 774, in __init__
    self.wait()
  File "/usr/lib/python3.6/site-packages/sh.py", line 792, in wait
    self.handle_command_exit_code(exit_code)
  File "/usr/lib/python3.6/site-packages/sh.py", line 815, in handle_command_exit_code
    raise exc
sh.ErrorReturnCode_1:

  RAN: /sbin/losetup -f --show --offset 50331648 --sizelimit 83886080 /img/balena.img

  STDOUT:

  STDERR:
losetup: /img/balena.img: failed to set up loop device: No such file or directory


Set the wallet name reactive (fix #28)

fix #28

fix the conflict and remove useless defined

WalletSelectorPanel is not reactive anymore.
It works for WalletSelectorField but not for the Panel on the left side as shown in below screenshot. On the left side you can see the old name evias-TAWJ and on the top right bar you can see the new name evias-TAWJ-1. Please also make sure that the panel on the left gets updated.

as per comment above

Now it is available.

not to be merged ; fix to happen in #177


Support xBox one Controller

Binding Device="045E0B00"

This is an Elite Controller.  Looks like they have just added a Device ID.

any update on this? using an xbox controller myself!


Refactors to use region enum instead of strings.

Fixes #42.
Still need to double check use of lifetimes and add a test for the enum to AWS string function.

:ship:


[Bug]: Login and Register Page

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug

Both the pages are working fine even when details are not filled, i.e. user navigate further without typing their details.
Expected behavior
The fields for both the pages must indicate required. It shows error when all fields are not filled on either page. Once the details are filled, the user navigates further.
Add ScreenShots
No response
On which device are you experiencing this bug?
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[ ] I'm a VSOC'24 contributor
[X] I have starred the repository

@me-nilakshi u are assigned

Hey apu52
Please delete the unnecessary files in your repo. I'm facing difficulty in cloning the repo. I've troubleshoot from my end. But since the file size is large and unorganised, it showing error.
Kindly look into this.

Hey apu52 Please delete the unnecessary files in your repo. I'm facing difficulty in cloning the repo. I've troubleshoot from my end. But since the file size is large and unorganised, it showing error. Kindly look into this.

Please try to fork & clone this repo again. & I also think this issue has already been solved

i am ready for solve the problem

I can do it better can you assign me


Use custom icon per node type

At the moment the navigation tree uses the same folder icon for all the node types. Ideally we should have one per type (project, test suite, test case, etc).

Book for project, folder for test suite and text file for test case.
http://semantic-ui.com/elements/icon.html


Improve the look and feel of the POC

Is your feature request related to a problem? Please describe.
At the end, the Yes and No button are shown.
Describe the solution you'd like
Disable or hide them.
Describe alternatives you've considered
None.
Additional context
None.

Now the page has a modern design, the assistant asks the user what to do next and a table with the entered symptoms is displayed.

I corrected one test suite which was failing. Now it is clear that the dynamic predicates are local to the areko_internal module.


test: Re-add "php" test to the test configuration.

We temporarily disabled testing the PHP client in Travis because the PHP composer binary is constantly segfaulting and failing the tests. See: https://github.com/youtube/vitess/pull/2081
We want to keep the configuration entry such that we can still run the command manually e.g. via "./test.go -flavor=mariadb php".

Pavel, sorry, we should have told you yesterday that there's this "Manual" field via which you can disable the test as well.


Feature Suggestion: Multi-Window/Tabs and Minimizing

We usually have multiple blocking terminal windows open when developing code that run processes like backend compiler watch, front-end transpiler watch, active testing, http proxy, cloud service proxies, etc., and all we care about is the output from their last commands. I've noticed this is very common for developers but haven't seen any terminal app to optimize the UX for these kind of blocking terminals which all need to be up together for development work. Of cource we can start them individually and hide them behind tabs and go through them one by one to check on their status, but it would be a lot more useful if we could just somehow minimize them into icons or very small windows that could be monitored in a single view, and the ability to save them in a profile/template for easy launch next time.
The specifics of the user experience needs to be discussed but if you know what I'm talking about, this is going to be an extremely useful feature for developers.

We can't save/restore/template them right you but you can use panes to have multiple shells in a single tab.


Madry Pretrained Model dosen't work: cleverhans/examples/madry_lab_challenges/cifar10/

I load the pre-trained model from https://github.com/MadryLab/cifar10_challenge, and set up the environment as ReadMe.md. But  cleverhans/examples/madry_lab_challenges/cifar10/attack_model.py cannot run.
It says:
Caused by op u'ResNet/input/init_conv/DW/read'
FailedPreconditionError (see above for traceback): Attempting to use uninitialized value ResNet/input/init_conv/DW
To Reproduce
Steps to reproduce the behavior:
To run the attack on CIFAR-10(cleverhans/examples/madry_lab_challenges/)

Clone the CIFAR-10 challenge repository from:
https://github.com/MadryLab/cifar10_challenge
Obtain some weights for the Madry Lab model, following the readme
from their project (either download their presupplied weights or train
your own).
Set your CIFAR10_CHALLENGE_DIR environment variable to point to
repository from step 1.
Run
python cifar10/attack_model.py
See error:
Caused by op u'ResNet/input/init_conv/DW/read'
FailedPreconditionError (see above for traceback): Attempting to use uninitialized value ResNet/input/init_conv/DW

Expected behavior
It should run and give attack examples.
System configuration

OS: linux
Python version: 2.7
TensorFlow version: 1.8.0

Hi,
I found the main problem is the scope name is not matching. For instance, the network demands 'ResNet/input/init_conv/DW/read' while the pre-trained model has '/input/init_conv/DW/read'. It is not matched. But I still have a question about how to match the name. Could you help me with it?
Thank you in advance!

Hi, I couldn't run the code either. I guess its something related to fprop? Any suggestions about how to fix it? Thanks a lot!

I couldn't run the code either.

Unfortunately, we are unable to support the code in the examples folder as stated in the documentation. Feel free to submit a PR if you were able to fix the issue.


Adding an event driven backend function

After days of frustration with AWS, I decided to try out Azure Mobile Services. However, I'm not sure what I can do to have a backend function that gets triggered to run a full text search algorithm in the database. When new data is added to the database, I want this function to be invoked and do a search action in the database, then send notifications according to the search result.
What I want to do is like Parse Cloud Code, or AWS Lambda + S3. In this tutorial : "Add Push Notifications to iOS App and JavaScript Backend" (https://azure.microsoft.com/en-us/documentation/articles/mobile-services-javascript-backend-ios-get-started-push/), it says "In TodoItem, click the Script tab and select Insert. This displays the function that is invoked when an insert occurs in the TodoItem table." Does this mean any function can be triggered by adding data to the database?
Please help, I'm desperate, any suggestions would be highly appreciated.

One quick suggestion first. I highly recommend you try out Azure Mobile Apps (its basically the new version of mobile services). You'll be using and learning our latest and greatest stuff that way.
To get started with Mobile Apps go here. Once you complete that tutorial, you will have a todoitem table. You can work with it, including adding scripts, using the Easy tables option in the settings:

From here, hit the Edit Script button and then you will be taken to an online IDE for working with your server code:

As you can see, we generate you a default script that has examples commented out for running code when reading, inserting etc.
The Mobile Apps version of the tutorial you were looking at is here. That tutorial includes an example todoitem.js script that sends a push notification.
Hopefully that will help you get started! Closing this as answered, feel free to reopen if I didnt answer your question properly.


pull request for e2e test at 01/30/2017 03:15:13

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Error: Unity Log NullReferenceException Spammed in console

this is the error being spammed in the console.

Here is the list of mods, i dont see what would be conflicting

fixed with update i guess? its not spamming anymore

Yes indeed, this was fixed in 1.1.4. I noticed it a bit late because this was not happening in all situations, sorry 😅


[WIP] Page Up / Page Down in text fields

This PR implements the page up/down and shift + page up/down keyboard shortcuts:

page up/down move caret
shift page up/down extend the selection by page
traversing text using page up/down keeps vertical text runs
no changes in ScrollAction

The corresponding macOS shortcuts which scroll without moving the caret are implemented in PR #105497.
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@tgucio Looks like the analyzer check found some issues. Can you take a look and fix those up? While you're at it, maybe also fix the merge conflict? Thanks.

@goderbauer sure - resuming to look into this PR now (it also needs adding some tests).

@tgucio Do you still have plans to add tests soon?

@goderbauer just took a first stab at adding tests.
I pushed two commits with another email (w/o CLA) so since I can't force push to correct that I've opened a new PR: https://github.com/flutter/flutter/pull/107600


Some animations from Lottiefiles aint recognized as lottie animation

Good day. While using AnimatedImage component some animations are not recognized as lottie.
This is an example of animation that is not recognized:
https://lottiefiles.com/animations/merry-christmas-rHWirI2aZ2
This one works well:
https://lottiefiles.com/animations/radar-loading-quRoUzmGZN
Hope both links are valid. Tried to create own animation with lottiefiles, but mine one either fails.

Hello @JChudinov, thanks for the report!
It seems that the merry christmas animation uses some body movin features that are not supported by the rlottie library.
You can try the animation yourself with the rlottie online player:
http://rlottie.com/
This Unity plugin uses the rlottie library internally to render the animation to Unity Texture2D.

You can review the supported features here:
https://github.com/Samsung/rlottie?tab=readme-ov-file#supported-after-effects-features


Replace Grafana cloud credentials by standard OTLP credentials in quickstart guides

Addresses comment: https://github.com/grafana/beyla/pull/530#pullrequestreview-1812965354

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

Comparison is base (9bcd95f) 80.06% compared to head (68eab64) 45.19%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #538       +/-   ##
===========================================
- Coverage   80.06%   45.19%   -34.88%
===========================================
  Files          69       67        -2
  Lines        5778     5603      -175
===========================================
- Hits         4626     2532     -2094
- Misses        930     2919     +1989
+ Partials      222      152       -70

Flag
Coverage Δ

integration-test
?

unittests
45.19% <ø> (+0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


How to config to show first table's column in index view

Hi there,
How to config to show first table's column in index view instead of modify index view (like below) when use scaffold.
index.blade.php
...

            ID	// must add this
            Name
            Display Name
            Description
            Action

@foreach($roles as $role)

            {!! $role->id !!}	// must add this
            {!! $role->name !!}
            {!! $role->display_name !!}
            {!! $role->description !!}
...

@tranghaviet I think you can publish templates and do these modifications and that' how you can have this in your index.blade.php.
Let me know if this helps.

Closing this issue. Please re-open it if needed.


How to tell if the service is down from metrics exposed?

We are currently running "ccloud-schema-exporter" as a Kubernetes pod with the metrics end point exposed. We want to setup alerts to check if the service is up and running. I tried killing the pod/deployment and could still find the metrics end point exposing some data.
Is there any metrics value used to tell if the ccloud-schema-exporter is running(Up or down)? We want to determine when ccloud-schema-exporter is down, so team could recover in case of any issues?

Hi @motonisar, the /metric endpoint will keep emitting until the process is killed. Do keep in mind that ccloud-schema-exporter tries to gracefully shut down when given a SIGTERM (and this might take a bit as it finishes the current sync loop). I'd say to know the exporter is running, the availability of the /metrics endpoint is a good indicator. In kubernetes, I'd probably make my liveliness probe query /metrics and as long as 200 is returned things should be up.
Did you wait a bit for graceful shutdown when sending a SIGTERM to the pod?

@abraham-leal Thank you for the quick response. I waited until the pod completed died and no longer existed. The metrics still continues to be up and return values which is not usual as the backend pod no longer runs. I did add the /metrics port 9020 to liveness probe. It continues to return "HTTP/1.1 200 OK" even though the pod no longer exist.

@motonisar I am not able to replicate this scenario. When deploying in k8s with a LB, if the ccloud-schema-exporter is up the metrics will return, otherwise the call hangs. Could it be that you are deleting the pod and k8s is just recreating it right away so you don't notice the pod being gone? If not, please do let me know how to reproduce this behavior.

Hi @motonisar, is your issue resolved? This seems stale. Please let me know, if not, I'll close the issue.

Hi @motonisar, is your issue resolved? This seems stale. Please let me know, if not, I'll close the issue.


Cleanup process pool after initializing ActiveLearning

On machines with many processing cores, it's apparently necessary to terminate the process pool after using it in this case since all of the processes that this __init__ method uses are still hanging around even if you delete the parent object.

This is going to seem super weird but I actually had the same error that appveyor got when I was trying to setup a development version of dedupe in my local virtualenv. I eventually had to basically nuke everything and then install all of the requirements in setup.py using pip and then run setup.py develop

Here's the traceback (in case that appveyor thing goes away) https://gist.github.com/evz/efbc636358814b096bb8

Okay, I'll merge this an open up an issue to figure out why pool is not being garbage collected. When pool is garbage collected, then the terminate method should be automatically called. https://docs.python.org/3.4/library/multiprocessing.html#multiprocessing.pool.Pool.terminate


Http-parser have a bug

https://github.com/gnet-io/gnet-examples/blob/4c982ee21cf2cabe5fd581cfd7652b4f1eb29332/examples/http/http.go#L160
req.path = sdata[s:q]
req.query = req.path[q+1 : i]
this code in http-parser have a bug,  provide this error :
==> panic: runtime error: slice bounds out of range [:30] with length 15
i know you know what is problem ;-)

Just encountered this issue while browsing through the examples, there's a simple fix:
-req.Path = sdata[s:q]
-req.Query = sdata[q+1:i]
+req.Path = sdata[s:s+q]
+req.Query = sdata[s+q+1:i]

@mircodezorzi

Sure :+1:


Please update dependencies

They are really old and also very specific. I had to add many manual resolutions to my bower.json file to be able to use this directive.
Although, seems to work fine.

Feel free to submit a PR


VerifyError - ShadowGenerator

This issue was originally posted by @gorodechnyj in #127:
And in my case this leads to crash on API 15 emulator. The crash happens when I open DrawerLayout. I also use ActionBarDrawerToggle with it:
actionBarDrawerToggle = new android.support.v7.app.ctionBarDrawerToggle(...
drawerLayout.setDrawerListener(actionBarDrawerToggle);
actionBarDrawerToggle.syncState();

09-02 19:09:42.803    9592-9592/com.masani E/AndroidRuntime﹕ FATAL EXCEPTION: main
java.lang.VerifyError: carbon/shadow/ShadowGenerator
at carbon.widget.LinearLayout.getShadow(LinearLayout.java:489)

This issue was solved in #40

No answer for more than a week. I'm closing the issue.


Forbid generating custom ore veins in The End (and the Nether?) below y=0

Currently, ore vein generators can generate ores below y=0. While this is allowed in the overworld, it is forbidden in The End. (I'm not sure about the Nether, I should test that.)
I should ensure that custom ores can only be generated below y=0 when the world supports it.

This will be fixed in KCI 12.5


[Bug] [UI] the page of task group queue is empty

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened

when we change the language to english ,this page miss the data.
What you expected to happen
show the data
How to reproduce
change the language to english
Anything else
No response
Version
dev
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

If there're running tasks in the queue of the task group, it will display those tasks on the page of 'Task group queue'. So I'd like to confirm whether there're running tasks in the queue of the task group. Would you like to give me more details about this issue? Look forward to your feedback.

If there're running tasks in the queue of the task group, it will display those tasks on the page of 'Task group queue'. So I'd like to confirm whether there're running tasks in the queue of the task group. Would you like to give me more details about this issue? Look forward to your feedback.
The list is not empty。

I will fix this issue. Thanks for your feedback.


Add URL and URLSearchParams

More fully implement URL and URLSearchParams
Please feel free to update the module name mappings if necessary.

@glennsl Please let me know if there's anything else I can do. =)


A problem when running sh cmds/20/motif/predcls/sup/train.sh

The program gets stuck while running. The following problems occur, and the GPU has been stuck at 100%

Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance in your application as needed.

Sorry, I did not meet this problem before. But I guess this might be caused by one of the processes in the distributed training crashes or meets some other problems? So, the program will wait for the predictions of the crashed process and just get stuck here.


chore: change all unit test names to comply with Go convention

Signed-off-by: Jennifer Power<EMAIL_ADDRESS>Description
This PR changes unit test function names that do not comply with Go convention for uniformity.
Type of change
Please delete options that are not relevant.

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
N/A
Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

/retest

/lgtm


go make issue

Hello,
I'm having the following issue.
$ go version
go version go1.13.7 linux/amd64
$ pwd
/home/centos/go/src/github.com/newrelic/nri-network-telemetry
$ make

Please tell me how to make.
Thank you

This should now be resolved.


Would be nice to give instructions for the release build

The readme only has info on how to build/serve for DDC. Would be nice to give the documentation for how to release dart2js.
In particular DDC on this app crashes an iPad Pro chrome, and safari. Probably requires too much memory for the device.

FYI, we now have such guidance on the tool > build_runner page of the 5-dev version of the webdev site.
cc @kwalrath

The dart2js build was a bit convoluted for a little while because build_runner didn't support differed loading. I didn't think it was worth documenting the hoops I was jumping through.
In the latest branch, I am updating to build_runner: 0.8.0 that has full support. At that point you can run
pub run build_runner serve web --config release to serve a version compiled with dart2js. I'll update the README before it lands.

SGTM yeah I did the same thing. Was running dart2js by hand on a release output. Pretty nasty stuff.


[irregular-verbs] added decir

added decir conjugation in indicative present and preterite

Thank you for the contribution much appreciated!


[Hotfix]Captcha compatiblity for Validator

(1) fix setOptions() for compatiblity in Captcha\AbstractAdapter
(2) remove '_' prefix in Captcha\Word
only to pass unit test.... (require updating Captcha's according to ZF2's CS )

Reviewed, merged, and pushed to master.


Fix node 20 reexports

Resolves #29

@luxaritas I see this is a draft... Is it ready to go?

Hey @tlhunter - main reason I left it as a draft is because I'm not confident that my approach here is "correct" - particularly how I'm handling path resolution seems hacky. That said I don't have any pending work on it, really just waiting for input, so I'll mark it as ready

@trentm Thanks for the patch - looks reasonable to me so I've applied it. (I've marked you as a commit coauthor using git metadata from a recent GitHub commit of yours, let me know if that should be something else!)

If I'm not mistaken, this is solved by #43.

If I'm not mistaken, this is solved by #43.

Hmm, I I've tried the fix from #43 (released in 1.7.2) and I think it does not cover the case of CJS re-exports.

If I'm not mistaken, this is solved by #43.

Hmm, I I've tried the fix from #43 (released in 1.7.2) and I think it does not cover the case of CJS re-exports.

Drat!

This is just speculation, based on commit bb038e9 of this PR handling a circular imports issue: I wonder if this PR would also have resolved https://github.com/DataDog/import-in-the-middle/issues/31
I haven't yet had a chance to understand what #43 is doing.

Woot. Thanks very much.


2.0.16 changes id attributes in html generated from .rst

Hi,firstly thanks for the 2.0.16 fix!  On 2.0.16, the build runs but, in html generated from an .rst file, where I was previously seeing the following:

References¶

I now see

References¶

I.e. the id has been capitalised.  Installing 2.0.14 and 2.0.16 back and forth does toggle this behaviour so it's definitely nbsphinx that's doing it, even though this is not in an .ipynb file.  Is this something to do with the 2.0.15 version note "Links to subsections are now possible in all source files"?
This might seem trivial but several typical test processes make use of these ids.  In our case it broke some integration tests.
A good resolution for us would be either (a) to reverse it and guarantee that nbsphinx will not make changes in html generated from .rst files, or (b) to know very explicitly what nbsphinx is doing to id attributes (and have any changes to this flagged explicitly in release notes) so that we can feel secure using the ids when writing tests.
Many thanks in advance.

Thanks for the report!
Sorry, I should have noticed that. And I indeed consider it a bug.
As it is now, just by loading the nbsphinx extension, the HTML output of totally unrelated RST files is changed. That wasn't my intention.
And yes, this was introduced with #135, where we started to use a transformation to generate Sphinx labels on all source files instead of only on Jupyter notebooks.
I've tried to fix this in #147, can you please check if that works for you?

This might seem trivial

Not to me.

(a) to reverse it and guarantee that nbsphinx will not make changes in html generated from .rst files

That's what I'm trying to do in #147, but "guarantee" is a very strong word ... if something changes in the future, just let me know!

Excellent -- yes 0.2.17 corrects this for us.  Thanks for the concern and rapid correction.


Fix some grammer errors, clarify capacity and throughput numbers

@tweissms : Thanks for your contribution to the Azure documentation! The author, @vladvino, has been notified to review your proposed change.

First pr for docs and just realized I should have been on the -pr repo instead.


Log to app insights when using console methods

Related Issue or Background Info

Closes #1715

Changes Proposed

Refactor telemetry to use typescript and hooks
Add a withInsights function that takes the console as a argument and adds telemetry to the log, info, warn, and error methods.

Additional Information

One potential concern with this approach:

If app insights itself calls a console method it could resulting in an infinite loop. I tried to head off the one known instance of this if the argument passed to the logger matches the regex /failed to send telemetry/i, but this seems hacky and will fail for any other potential instances where appInsights might end up calling an affected console method.

Screenshots / Demos

console.error reported to the test environment can be viewed in App Insights here
consolee.log reported as a custom event can be viewed in App Insights here

Checklist for Author and Reviewer
UI

[n/a] Any changes to the UI/UX are approved by design
[n/a] Any new or updated content (e.g. error messages) are approved by design

Testing

[x] Includes a summary of what a code reviewer should verify

Changes are Backwards Compatible

[n/a] Database changes are submitted as a separate PR

[n/a] Any new tables that do not contain PII are accompanied by a GRANT SELECT to the no-PHI user
[n/a] Any changes to tables that have custom no-PHI views are accompanied by changes to those views
(including re-granting permission to the no-PHI user if need be)
[n/a] Liquibase rollback has been tested locally using ./gradlew liquibaseRollbackSQL or liquibaseRollback

[n/a] GraphQL schema changes are backward compatible with older version of the front-end

Security

[n/a] Changes with security implications have been approved by a security engineer (changes to  authentication, encryption, handling of PII, etc.)
[n/a] Any dependencies introduced have been vetted and discussed

Cloud

[n/a] DevOps team has been notified if PR requires ops support
[x] If there are changes that cannot be tested locally, this has been deployed to our Azure test environment for verification

If app insights itself calls a console method it could resulting in an infinite loop. I tried to head off the one known instance of this if the argument passed to the logger matches the regex /failed to send telemetry/i, but this seems hacky and will fail for any other potential instances where appInsights might end up calling an affected console method.

This is a real problem. I dug a bit into the AppInsights source, and to my dismay there's a fair amount of console.warning, both through their warnToConsole and throwInternal methods. Regrettably, they're not prefixed in any way nor do they share any easily greppable patterning. Here's a sampling, beginning of course w/ the one you mention:
_self.diagLog().throwInternal(LoggingSeverity.WARNING, _InternalMessageId.OnError, "Failed to send telemetry.", { message: message });
itemCtx.diagLog().warnToConsole("Telemetry processor check returns false");
logger.warnToConsole("Invalid input for dependency data");
logger.warnToConsole("Two extensions have same priority #" + extPriority + " - " + extPriorities[extPriority] + ", " + identifier);
this._logger.warnToConsole("Auto track page visit time failed, metric will not be collected: " + Util.dump(e));
this._logger.warnToConsole("Call to restart failed: " + Util.dump(e));
this._logger.warnToConsole("Call to start failed: " + Util.dump(e));
this._logger.warnToConsole("Stop page visit timer failed: " + Util.dump(e));
itemCtx.diagLog().throwInternal(LoggingSeverity.CRITICAL, _InternalMessageId.InvalidEvent, "Cannot send telemetry without baseData and baseType");
itemCtx.diagLog().throwInternal(LoggingSeverity.CRITICAL, _InternalMessageId.PluginException, "Plugin [" + plugin.identifier + "] failed during processTelemetry - " + error);
itemCtx.diagLog().throwInternal(LoggingSeverity.CRITICAL, _InternalMessageId.SenderNotInitialized, "Sender was not initialized")
this.diagLog().throwInternal(LoggingSeverity.CRITICAL, _InternalMessageId.StartTrackFailed, "startTrackPage failed, page view may not be collected: " + Util.getExceptionName(e), { exception: Util.dump(e) });
this.diagLog().throwInternal(LoggingSeverity.CRITICAL, _InternalMessageId.StopTrackEventFailed, "stopTrackEvent failed, event will not be collected: " + Util.getExceptionName(e), { exception: Util.dump(e) });
this.diagLog().throwInternal(LoggingSeverity.CRITICAL, _InternalMessageId.StopTrackFailed, "stopTrackPage failed, page view will not be collected: " + Util.getExceptionName(e), { exception: Util.dump(e) });
_self.diagLog().throwInternal(LoggingSeverity.CRITICAL, _InternalMessageId.NotificationException, "send request notification failed: " + Util.getExceptionName(e), { exception: Util.dump(e) });
_self.diagLog().throwInternal(LoggingSeverity.CRITICAL, _InternalMessageId.TransmissionFailed, "Telemetry transmission failed, some telemetry will be lost: " + Util.getExceptionName(e), { exception: Util.dump(e) });

I think we should take this up with @timothybest-usds on Tuesday, and consider instead using a logger util and instead just entirely avoiding console by convention.

@nullflux yes, I considered that as well re: a custom logger. That solves the problem but of course it only means we get insights in places we call our custom logger and not if we're hitting an error in a dependency, for example

That's fair, but I would expect that the type of critical dependency failures that we'd need to know about would throw exceptions rather than just complaining to the console – and we should already hear about those via AppInsights because of the top-level error boundary

@nullflux whoops, I think the solution is in the app insights config object:

loggingLevelConsole: Console logging level. All logs with a severity level higher than the configured level will be printed to console. Otherwise they are suppressed. ie Level 2 will print both CRITICAL and WARNING logs to console, level 1 prints only CRITICAL.
Note: Logs sent as telemetry to instrumentation key will also be logged to console if their severity meets the configured loggingConsoleLevel
0: ALL console logging off 1: logs to console: severity >= CRITICAL 2: logs to console: severity >= WARNING


Consumer spec fixes

@alanbrent I did most of this in one large chunk (lots of trial-and-error on spec setup and such) but tried to split it into digestible chunks for illustrative porpoises.
Feel free to do with what you will, and we can chat tomorrow about anything if you want. All the specs pass on my machine:

Thanks! I will merge in now, though fair warning I'm probably going to reconstruct the specs to match the patterns used in the SQS publisher specs.


feat: release for version 2.6.0 of the spec

Refs https://github.com/asyncapi/spec/issues/894
Features
Added Pulsar Bindings and Protocol to the Specification (#882) ((https://github.com/asyncapi/spec/commit/35693fb90e230b48a51df952a743a12408995dd1)) (binding definition) by @VisualBean

@fmvilas @smoya @char0n @dalelane
Please approve, at least 2 of you.
There were conflicts we struggled with solving so the last commit to solve merge conflicts I had to do directly to upstream next-spec. I reviewed all the 23 changes and they are accurate and contain what's expected
@AceTheCreator
PR is ready for release imho

/rtm

@all-contributors please add @AceTheCreator  for eventOrganizing, ideas, maintenance, talk


Add --output json support to ockam identity list

Current behavior
Currently running ockam identity list --output json produces an error like the following:

Desired behavior
However it should provide a list of identity information in JSON format.
Please take a look at this file to get started.

We love helping new contributors! ❤
If you have questions or need help as you explore, please join us on Discord. If you're looking for other issues to contribute to, please checkout our good first issues.

I have a little of free time to tackle another issue and this is similar to the issue I have in review right now so if it's ok I would like to take it.

@kriogenia That's awesome, this is all yours. Please let us know if you have any questions as you explore. You can also ask questions on the contributors discord https://discord.gg/RAbjRr3kds


Grouping timezones by there continent and removing the continent predecessor in the string name should save kilobytes of space.  Although this seems trivial, it can be very helpful on embedded systems.
find_timezone would need to know how to split the continent from the input and target the correct group.
There are some timezones that are not defined by the continent/area approach (UTC and ROK, for example); these would need to be handled as well.

After some initial testing, the memory saved is nearly negated by the updated code required to unpack the continent


Menu and donation page translated into Chinese

Changes:
Menu and donation page translated into Chinese

I'll try to review this PR and the other one in the next few days. I may need to do some editing on my side, mainly for consistency, if you don't mind.

I'll try to review this PR and the other one in the next few days. I may need to do some editing on my side, mainly for consistency, if you don't mind.

I'll try to review this PR and the other one in the next few days. I may need to do some editing on my side, mainly for consistency, if you don't mind.

I just want to look at the issue of consistency, and I will try to keep it as much as possible after you change it. After that, I mainly want to make Chinese culture based on the existing reference.

I'll try to review this PR and the other one in the next few days. I may need to do some editing on my side, mainly for consistency, if you don't mind.

I just want to look at the issue of consistency, and I will try to keep it as much as possible after you change it. After that, I mainly want to make Chinese culture based on the existing reference.

The main part of it is just following the existing glossary that is compiled here: https://docs.google.com/spreadsheets/d/1yowXBskrVbCCrKtCRel3LTta_t3oXJN_RkK4AOchLGM/edit#gid=0, you can refer to that glossary for mostly programming related terms when you are translating another page in the future. Thanks!

The main part of it is just following the existing glossary that is compiled here: https://docs.google.com/spreadsheets/d/1yowXBskrVbCCrKtCRel3LTta_t3oXJN_RkK4AOchLGM/edit#gid=0, you can refer to that glossary for mostly programming related terms when you are translating another page in the future. Thanks!

ok

ok

Merged with https://github.com/processing/p5.js-website/commit/8d500d6ea5029da0fb7cbd476078ff308f9c2310. Thanks!


Autowrap an entire package

Is there some way we could find all modules in a package and wrap them? I initially thought yes but now I'm unsure. Either way, it would be nice.

With D as it is now, no. Packages are just directories, and the compiler can't do anything with directories.
The only way to get close to this is do what the executable configuration of unit-threaded does and generate code with all module names beforehand.


Add nine patch draw helper and core block plugin

Creates a nine patch draw function for any width/height (divisible by eight) out of a texture of at least 3x3 (using 8x8 tiles).
If bigger than 3x3 repeats textures using modulo.
Invisible lines on the side could break this rendering correctly
Made simple core block plugin to test it

Should probably redo nine patch to use 1 array, will do later


DPE-3698 No easy way to check write

Add continuous writes to unit status
Fixes #22

LGTM, curious if it's a good idea to write out a message along with the ActiveStatus if database config exists but continuous writes not enabled

If the writes process PID that we write to data bag (self.unit_peer_data.get(PROC_PID_KEY)) is reliable, it should not happen


Fix @lroStatus not detecting default states correctly when using union

:x: There is undocummented changes. Run chronus add to add a changeset or click here.
The following packages have changes but are not documented.

:x:@azure-tools/typespec-azure-core

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/392/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/392/


Plane workaround

Currently the HoI Ai does not deploy air wings if a nation starts with 0 planes. With this tiny change, each nation gets at least one fighter plane, which fixes the problem.

The puppet state conversion is now in. :-)
Couldn't test it with "Together For Victory" though, since I haven't bought it yet. Should work with and without the DLC.

My usual review (feel free to look at my commits and review too. I'd actually appreciate it):
HOI4Country.cpp, line 72 - ends in a comma instead of a semicolon
HOI4Country.cpp, line 1731 - there's an extraneous 's' at the end of the file
HoI4World::TagToCountry() might be better named as getCountry() or findCountry()
Instead of returning a reference to a country's puppets and editing it, it's probably better to have a function that just adds a new puppet.
And for any file you  change, bump the copyright to the current year (not a huge deal if missed, but it's nice to catch).
Good set of changes, though. And thanks for catching some of those typos!

I can't find the extra s or \s. Other than that I included the improvements.
I can look at your commits, but I can't help with Linux low-level functions I'm afraid.


fix(nan_console): Error when NAN Discovery is started more than once (IDFGH-13923)

Description

Development board: ESP32-WROOM-32
ESP-IDF version: 5.4.0

In nan_console example, when NAN is started more than once, triggers this netif error: esp_netif_lwip: esp_netif_new_api: Failed to configure netif with config=0x3ffc85e0 (config or if_key is NULL or duplicate key). This means that the existence of a created network interface was not checked.
To resolve this problem, three actions were taken:

Declaration of variable as null g_nan_netif;
Before creating the network interface, g_nan_netif is checked;
When NAN is stopped, g_nan_netif gets NULL.

Testing
Error log:

Fixed log:

@entrop1a , thanks for your contribution, with the minor modification mentioned above, will take the PR for internal review

@nachiketkukade, thanks for your review! Modifications have been made.

sha=71d2e16b46daae5818455fef8cd485877efd0cb3


my dashboard: show record status in uploads list

closes: https://github.com/inveniosoftware/invenio-app-rdm/issues/1478

we have now two labels with the same colour (red on Declined and Metadata only). Would it be an option to change metadata-only colour? Red usually indicates negative correlation in the interface, like "Decline" "delete" "remove" while metadata only is more an information. @ntarocco @lnielsen @jennur

I agree the color of metadata-only could be a different color. Another suggestion is to maybe move the status label to the other side of the container to separate it from the metadata-labels.

If we also remove the delete-button:

@jennur I agree to remove the delete button since it is potentially dangerous action (does it even have a modal to confirm?)
I think however the status of the community submission status should stay on the left side as initially indicated in the mockups, since it is quite important information

Hi, I suggest the text treatment of the labels (content type, metadata only, data) should also factor in whether they are actionable or not ...  there is a ticket somewhere for allowing some of them to be clickable and act like facets.  It makes sense to me that "Book" and "Metadata Only" act that way but that date doesn't.


Code catboost_encoder

This is just an issue to track the progress of the entire function (since the milestone instructions state to create an issue for each function, and we only created issues for each docstring in particular).

[x] Docstring
[ ] Code

Code function as required.
Implement both regression and binary classification.

Will modify as we understand what we need to do in the coming weeks.

See https://github.com/UBC-MDS/encodeR/issues/37


Remove newline in PS1

Motivation for this change

The current behaviour is opinionated and it shouldn't be up to distributions to decide this. Distributions should use the "default" which users expect.
It fills the terminal for no reason.
There is no reason to have it.
People often leave it because they don't know how to remove it. https://stackoverflow.com/questions/51560300/unwanted-bash-newline-nixos

This newline has been present as far back as git allowed me to go in PS1 (7c93a71468e7d5) (2008 more than 10 years ago).
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

-1, though it's a purely personal preference so I won't insist if this is widely supported.

there is a reason to have it, which is that commands that don't terminate their output with a newline don't mess up the prompt.

While messing up all other commands.
The thing is that this is non standard. NixOS (from what I know) is the only distro that does this. Also I'm pretty sure that in Ubuntu commands that finish without newlines aren't messed up, how do they do it?

While messing up all other commands.
The thing is that this is non standard. NixOS (from what I know) is the only distro that does this. Also I'm pretty sure that in Ubuntu, commands that finish without newlines aren't messed up, how do they do it?

zsh fixes that on its own, maybe bash also have an option for that?

While messing up all other commands.

It doesn't mess anything up at all, just looks different.

there is a reason to have it, which is that commands that don't terminate their output with a newline don't mess up the prompt.

It doesn't mess anything up at all, just looks different.

Applies to both

A non-newline-terminated command does mess up the prompt if the prompt doesn't start with a newline. To see what I mean, try printf " %.0s" $(seq $((COLUMNS - 5 ))), and compare behaviour afterwards with a newline in the prompt and without.

A non-newline-terminated command does mess up the prompt if the prompt doesn't start with a newline. To see what I mean, try printf " %.0s" $(seq $((COLUMNS - 5 ))), and compare behaviour afterwards with a newline in the prompt and without.

I think i would be nice to do it similarly to Ubuntu (if they actually have a way and I'm not just making things up) but the same applies:

It doesn't mess anything up at all, just looks different.

This is just a choice but I believe it is better to do what is "expected".

Maybe you could mention it on Discourse and ask for people's opinions.
I don't really care either way, since I use fish and it does this:
$ echo "Foo"
Foo
$ echo -n "foo"
foo⏎                                                                                $

However when I use bash on other distros I'm sometimes confused where my PS1 indicator has gone when it's just at the end of a long line without \n.
So I concur with @lheckemann that I don't want my prompt messed up.
As you mentioned it has been like this for >10 years so I wouldn't change it willy nilly.

Indeed it has been this way since the start and I don't see a strong reason to change it. I prefer it this way since I think it makes bash a bit more readable by clearly separating command invocations. Obviously not everybody will agree but that's true for any default.


Pixiv - Letting animated content play/have an indicator for such work

I'm not sure if this has been mentioned already, but I wouldn't doubt it. I know Pixiv seems to have an issue with letting people copy videos/gifs to share them around as-is, so adding support for that in Grabber sounds tricky; I don't know the specifics of it. However, I'm wondering if adding a play icon on the thumbnail of the upload could be possible. The same thing you see with the boorus. At the very least, I'd know I should open it on my browser in order to see everything.

I think the best solution for playing back gif, webm or oder video files is to run a video player for the downloaded file in the cache (or local, if it already exists). That would skip any kind of authentication or HTTP Header shenanigans required for e.g. pixiv and would use native tools like mpv.


Improved encoding and decoding speed of Vec

I noticed this comment:
https://github.com/bincode-org/bincode/blob/a22afa384d02ae0653d27eb0908da6771baecb26/src/de/impls.rs#L449-L452
You can implement the condition soundly as non_static_type_id::<T>() == TypeId::of::<u8>() where:
use std::any::TypeId;
use std::marker::PhantomData;
use std::mem;

fn non_static_type_id<T: ?Sized>() -> TypeId {
    trait NonStaticAny {
        fn get_type_id(&self) -> TypeId where Self: 'static;
    }

    impl<T: ?Sized> NonStaticAny for PhantomData<T> {
        fn get_type_id(&self) -> TypeId where Self: 'static {
            TypeId::of::<T>()
        }
    }

    let phantom_data = PhantomData::<T>;
    NonStaticAny::get_type_id(unsafe {
        mem::transmute::<&dyn NonStaticAny, &(dyn NonStaticAny + 'static)>(&phantom_data)
    })
}

This is sound because all types with the same TypeId as u8 are u8. In contrast, not all types with the same TypeId as &'static str are &'static str.

cc #618
Will implement this, thanks
Just to confirm, this is still sound to call with types with a lifetime, e.g. if I make a function like:
fn type_equal<T1, T2>() -> bool {
    non_static_type_id::<T1>() == non_static_type_id::<T2>()
}

// this is fine:
type_equal::<&'static str, u8>();
type_equal::<u8, &'static str>();

// this could result in unsoundness:
// type_equal::<&'a str, &'b str>();

It is sound to get the TypeId for any T using this method. You only need to be concerned about the soundness of unsafe code that downcasts based on the value of the obtained TypeId, as different types can have the same TypeId. So non_static_type_id::<T>() == TypeId::of::<&'static str>() does not mean that you can downcast to &'static str.

Implemented this in #667. I ended up making a small helper crate for this:

https://github.com/bincode-org/unty
https://docs.rs/unty
https://crates.io/crates/unty


refactor(Table-component): Refactor onChange event handler

Previously, onChange would return an array on selecting all and an object on selecting an individual
row. This was confusing. Now onChange will return an array for both events.
Any components that use the Table and rely on the onChange prop will need to be updated to handle
this change.

@milemilo @KevinMarren

@KevinMarren @milemilo I updated the branch. I should have added another commit but I just amended and force pushed. I have now enabled "merge + squash" so we can keep a cleaner log and still have useful PRs.


hook: use latest var instead of hardcoded latest

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherrypick release-0.8


Allow CI to open PRs?

@L-M-Sherlock would you be willing to enable the option called "Allow GitHub Actions to create and approve pull requests" described here? I think this link should go to the appropriate page: https://github.com/organizations/open-spaced-repetition/settings/actions
I'm trying to get CI to open a PR when it fails to sync with fsrs-rs (which should in turn trigger an email for me and tell me there's a problem.) However, the repo's "Allow GitHub Actions to create and approve pull requests" option is greyed out because the org doesn't have the option enabled.

OK:


Persist NEP5 Token creation

What current issue(s) does this address?, or what feature is it adding?

This PR adds functionality to listen to the SmartContractEvent.CREATE and then determine if that event created an NEP5 compliant token.
If it is determined to be an NEP5, then that event is persisted alongside other notifications in the NotificationDB.
This makes a full list of all NEP5 Compliant tokens queryable via the neo/api/REST/NotificationRestApi

How did you solve this problem?

code

How did you make sure your solution works?

tests

Did you add any tests?

yes

Are there any special changes in the code that we should be aware of?

not really

Did you run make lint and make test?

yes

Are you making a PR to a feature branch or development rather than master?

yes

Coverage increased (+0.2%) to 75.385% when pulling 805fea96fac1ae65a955480f2f810d7b0d72bcd0 on feature-scevents-contract-create into ef170b001426c762e657cc62127cfae8235808f8 on development.


Auto-completion does not work for js/ts

Check for existing issues

[X] Completed

Describe the bug / provide steps to reproduce it
When try to auto complete symbols , no auto completion menu is being popped up . It just shows that downloading javascript language server   after that completion happens . I have also tried this on the preview 0.119.4 , but same problem was arising .
Environment
Zed: v0.118.1 (Zed)
OS: macOS 14.2.1
Memory: 8 GiB
Architecture: aarch64
If applicable, add mockups / screenshots to help explain present your vision of the feature
No response
If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.
If you only need the most recent lines, you can run the zed: open log command palette action to see the last 1000.
No response

No language server file  has been found inside the javascript/typescript language server .


Trakt API lookup errors

When trying to call the flexget API for trakt lookup, these errors occurs.
Some useful info might be that it does not occur on all all the calls (in the UI, when loading 1 page of the movie list, 3/10 do succeed to load)
2016-05-06 10:15 ERROR    flexget.api                   Exception on /trakt/movie/ [GET]
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1475, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1461, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/usr/local/lib/python2.7/dist-packages/flask_restful/__init__.py", line 477, in wrapper
    resp = resource(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flask/views.py", line 84, in view
    return self.dispatch_request(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flask_restplus/resource.py", line 42, in dispatch_request
    resp = meth(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/database.py", line 34, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/api/trakt_lookup.py", line 120, in get
    return jsonify(result.to_dict())
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/api_trakt.py", line 786, in to_dict
    "actors": list_actors(self.actors),
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/api_trakt.py", line 838, in actors
    self._actors[:] = get_db_actors(self.id, 'movie')
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/collections.py", line 1155, in __setslice__
    values = [__set(self, value) for value in values]
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/collections.py", line 1045, in __set
    item = executor.fire_append_event(item, _sa_initiator)
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/collections.py", line 717, in fire_append_event
    item, initiator)
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/attributes.py", line 941, in fire_append_event
    initiator or self._append_token or self._init_append_token())
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/unitofwork.py", line 46, in append
    sess._save_or_update_state(item_state)
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/session.py", line 1607, in _save_or_update_state
    self._save_or_update_impl(st_)
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/session.py", line 1861, in _save_or_update_impl
    self._update_impl(state)
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/session.py", line 1854, in _update_impl
    self.identity_map.add(state)
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/identity.py", line 132, in add
    % (key, ))
AssertionError: A conflicting state is already present in the identity map for key (<class 'flexget.plugins.api_trakt.TraktImages'>, (1,))

I have no idea about the ui. But all my lookups so far have come back without an issue. Need more information to proceed.

Use the ui then, it'll generate a lot of api calls

I've used the UI now. Seems all of the calls for either series or tv shows Throw many many errors. Probably has something to do with the fact that it's sending 500 calls in 1 second. In short. It's a UI error not a plugin error. 1 thing hasn't finished before another is requested.

My suggestion would be to build a request queue for things like this. Allowing the queue to go 1 at a time until completed. Probably has more to do with threads than anything.

Or better yet. Build it into the UI properly to make a call wait for results. Not slam the api with 500 calls on lookups that might not be there already and have to be made.

I'm not sure where you're getting that many calls, since the movies (and series) page are limited to 10 items per page.
I also want to mention that the lookup calls worked before the latest update to the trakt lookup endpoint. And the series page calls the TVDB or Tvmaze (don't know which one it currently is), which seems to handle this very well.
Also I've included the debug log, maybe it holds some more information?
2016-05-07 10:34 DEBUG    movie_list                    retrieving movie lists
2016-05-07 10:34 DEBUG    movie_list                    fetching list with id 1
2016-05-07 10:34 DEBUG    api_trakt                     movie `The Lost` age 7 expires in 37 days
2016-05-07 10:34 DEBUG    api_trakt                     movie `The Social Network` age 6 expires in 32 days
2016-05-07 10:34 DEBUG    api_trakt                     movie `The Croods 2` age 0 expires in 2 days
2016-05-07 10:34 DEBUG    api_trakt                     movie `Rape Zombie: Lust of the Dead 3` age 3 expires in 17 days
2016-05-07 10:34 DEBUG    api_trakt                     Searching with params: tmdb=256522
2016-05-07 10:34 DEBUG    api_trakt                     Searching with params: tmdb=223702
2016-05-07 10:34 DEBUG    utils.requests                 Fetching https://api-v2launch.trakt.tv/search
2016-05-07 10:34 DEBUG    utils.requests                 Fetching https://api-v2launch.trakt.tv/search
2016-05-07 10:34 DEBUG    api_trakt                     Searching with params: tmdb=175168
2016-05-07 10:34 DEBUG    utils.requests                 Fetching https://api-v2launch.trakt.tv/search
2016-05-07 10:34 DEBUG    utils.requests                 Fetching https://api-v2launch.trakt.tv/movies/156459
2016-05-07 10:34 DEBUG    utils.requests                 Fetching https://api-v2launch.trakt.tv/movies/137261
2016-05-07 10:34 DEBUG    api_trakt                     Searching with params: tmdb=325348
2016-05-07 10:34 DEBUG    api_trakt                     movie `Jaws 2` age 38 expires in 192 days
2016-05-07 10:34 DEBUG    utils.requests                 Fetching https://api-v2launch.trakt.tv/search
2016-05-07 10:34 DEBUG    utils.requests                 Fetching https://api-v2launch.trakt.tv/movies/111210
2016-05-07 10:34 DEBUG    utils.requests                 Fetching https://api-v2launch.trakt.tv/movies/156459/people
2016-05-07 10:34 DEBUG    utils.requests                 Fetching https://api-v2launch.trakt.tv/movies/137261/people
2016-05-07 10:34 DEBUG    utils.requests                 Fetching https://api-v2launch.trakt.tv/movies/221309
2016-05-07 10:34 DEBUG    utils.requests                 Fetching https://api-v2launch.trakt.tv/movies/111210/people
2016-05-07 10:35 DEBUG    utils.requests                 Fetching https://api-v2launch.trakt.tv/movies/221309/people
2016-05-07 10:35 DEBUG    api_trakt                     Searching with params: tmdb=284303
2016-05-07 10:35 DEBUG    utils.requests                 Fetching https://api-v2launch.trakt.tv/search
2016-05-07 10:35 DEBUG    utils.requests                 Fetching https://api-v2launch.trakt.tv/movies/181547
2016-05-07 10:35 ERROR    flexget.api                   Exception on /trakt/movie/ [GET]
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1475, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1461, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/usr/local/lib/python2.7/dist-packages/flask_restful/__init__.py", line 477, in wrapper
    resp = resource(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flask/views.py", line 84, in view
    return self.dispatch_request(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flask_restplus/resource.py", line 42, in dispatch_request
    resp = meth(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/database.py", line 34, in wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/api/trakt_lookup.py", line 120, in get
    return jsonify(result.to_dict())
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/api_trakt.py", line 786, in to_dict
    "actors": list_actors(self.actors),
  File "/usr/local/lib/python2.7/dist-packages/flexget/plugins/api_trakt.py", line 838, in actors
    self._actors[:] = get_db_actors(self.id, 'movie')
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/collections.py", line 1155, in _setslice__
    values = [__set(self, value) for value in values]
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/collections.py", line 1045, in __set
    item = executor.fire_append_event(item, _sa_initiator)
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/collections.py", line 717, in fire_append_event
    item, initiator)
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/attributes.py", line 941, in fire_append_event
    initiator or self._append_token or self._init_append_token())
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/unitofwork.py", line 46, in append
    sess._save_or_update_state(item_state)
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/session.py", line 1607, in _save_or_update_state
    self._save_or_update_impl(st_)
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/session.py", line 1861, in _save_or_update_impl
    self._update_impl(state)
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/session.py", line 1854, in _update_impl
    self.identity_map.add(state)
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/identity.py", line 132, in add
    % (key, ))
AssertionError: A conflicting state is already present in the identity map for key (<class 'flexget.plugins.api_trakt.TraktImages'>, (1, ))

This error is probably because the same actor is added to the actor table multiple times. I encountered a similar error on a plugin I'm working on.

So I guess the first thing to try is shorter sessions?
On Sat, May 7, 2016, 12:28 Claus Vium<EMAIL_ADDRESS>wrote:

This error is probably because the same actor is added to the actor table
multiple times. I encountered a similar error on a plugin I'm working on.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/Flexget/Flexget/issues/1143#issuecomment-217622474

If the sessions are overly long, sure. Otherwise I'd probably just make sure that the same actor isn't added to the same movie/series more than once.

Problem isn't the plugin. I do the checks to make sure that nothing matching is re entered. Problem is  calls call1  |   starts call1 |  calls call 2 |  call1 hasn't finished but still start call2 and so on and so forth. This type of usage is why we implemented the task.queue system for the daemon mode. But we don't use the task queue system we call the plugin directly.

well, I changed the default lookup to exclude actors by default which hid this issue under the rug. The question is it still worth to mess around with this.

I guess not :smile:


Remove the trailing slash in the main URL

Silences warning from Addon Manager.

I haven't been active for a while on FreeCAD/FLOSS/programming hobby, and just saw this. Thanks.


More flexible subtraction logic

There are two reasonable things to do when subtracting a count that exceeds the minuend count: saturate at 0, or error. It would be nice to let the user do both.
See https://github.com/jonhoo/hdrsample/pull/34/files#diff-b4aea3e418ccdb71239b96952d9cddb6R558 for more context.

See especially https://github.com/jonhoo/hdrsample/pull/34#discussion_r106214179


Expressive theme – Checkbox

Checkbox
Label

label line height should be 18px. Check $label-01 spec.
~spacing between label and first checkbox item should be 8px~
Update Wonil and I would like to try not changing the padding for the first round of these changes

Checkbox icon

icon should be 20x20px. I thought Carbon was using that, but looks like 16x16 when I inspect it?
~Space between icon and checkbox item should be 8px, I can't understand how they built this, so can you confirm it's correct?~
Update Wonil and I would like to try not changing the padding for the first round of these changes

Checkbox item

How is Carbon's line height 24px?!? That doesn't exist in the type scale

I might need to sit with the developer to figure this part out

Design intent:

~8px spacing between each item~
Update Wonil and I would like to try not changing the padding for the first round of these changes
type size = $body-short-01

For the checkbox item line height is at 24px since they are hardcoding it instead of using the token

Same was done with the label

@joshblack is the line height something we should be overwriting when applying the expressive theme? Looks like the line-height was set for experimental checkbox styles.

Good question, I think this may need to be changed if the drawn checkbox and box are also updating in size, meaning from 16 -> 20px 👍


Transaction Parsing Module

Start pulling parsing logic out into separate package and refine payload structure for easier client ingestion
      const expected: Tx = {
        txid: tx.txid,
        blockHeight: tx.blockHeight,
        blockTime: tx.blockTime,
        blockHash: tx.blockHash,
        address: address,
        caip2: 'ETH',
        value: tx.value,
        fee: {
          value: '78183644000000000',
          caip19: 'ETH',
        },
        transfers: [
          {
            type: TransferType.Send,
            from: '0xb8b19c048296E086DaF69F54d48dE2Da444dB047',
            to: '0x0d4a11d5EEaaC28EC3F61d100daF4d40471f1852',
            caip19: 'USDT',
            totalValue: '45000000000',
            components: [{ value: '45000000000' }],
            token: usdtToken,
          },
          {
            type: TransferType.Receive,
            from: '0xF82d8Ec196Fb0D56c6B82a8B1870F09502A49F88',
            to: '0xb8b19c048296E086DaF69F54d48dE2Da444dB047',
            caip19: 'KISHU',
            totalValue: '9248567698016204727450',
            components: [{ value: '9248567698016204727450' }],
            token: kishuToken,
          },
        ],
        trade: {
          dexName: Dex.Zrx,
          type: TradeType.Trade,
        },
      }
      ```

This is a great start. I especially love the focus on tests to validate the complex logic.
I see a few areas where the architecture could be tweaked to make it more generic and allow other people to write additional parsers.

This is a great start. I especially love the focus on tests to validate the complex logic.
I see a few areas where the architecture could be tweaked to make it more generic and allow other people to write additional parsers.

Thanks and great feedback! I agree there is def room for improvement to make this more ergonomic and like the idea for the ability to inject custom parsers. Down to talk over those ideas more when we get the initial replacement swapped out for chain-adapters/web so we have caip and an easier payload to interact with!


ESP8266 wifi(mySerial);

I'm  using an arduino nano v3 and I got this error
ConnectWiFi:28: error: no matching function for call to ‘ESP8266::ESP8266(SoftwareSerial&)’
/home/ricardo/.AIDE/arduino/libraries/WeeESP8266-master/ESP8266.h:37: note: candidates are: ESP8266::ESP8266(HardwareSerial&)
/home/ricardo/.AIDE/arduino/libraries/WeeESP8266-master/ESP8266.h:29: note:                 ESP8266::ESP8266(const ESP8266&)
If I just write
ESP8266 wifi(Serial);
Apparently my problem has gone but I can't connect the ESP with my AP
Any idea?

Does this happen every time you try? I've found with my esp01 it locks up from time to time. I found that I if I cycle power to the esp it can connect again. I'm experiment with using a transistor inline with the 3v3 power on the esp board and then allowing for cycling the power from my arduino. If/when I figure it out I'll try and add a method into the library to handle this process automatically.

No this happen always, is not ESP problem, is a library problem. why I think this?.
In the readme says if a have an UNO (nano is basically the same), add this
#include <SoftwareSerial.h>
SoftwareSerial mySerial(3, 2); /* RX:D3, TX:D2 */
ESP8266 wifi(mySerial);
If I change this in the original files I have the problem just compiling
The original files come with
ESP8266 wifi(Serial1);//This not work neither
But if I change just this line a write all compile good without need SoftwareSerial library
ESP8266 wifi(Serial);

Hi, you need to modify the ESP8266.h file first.
Using SoftwareSerial
If you want to use SoftwareSerial to communicate with ESP8266, you need to modify the line in file ESP8266.h:
//#define ESP8266_USE_SOFTWARE_SERIAL
After modification, it should be:
#define ESP8266_USE_SOFTWARE_SERIAL
include the <SoftwareSerial.h> file like this:
#include "ESP8266.h"
#include <SoftwareSerial.h>
SoftwareSerial Serial1(3, 2); /* RX:D3, TX:D2 */
ESP8266 wifi(Serial1);
PS: I use level shifters from 5V to 3.3V as my Arduino ProMINI clone is running at 5V, remember that ESP8266 is ONLY 3.3V tolerant !
regards,
..aniston

aniston's answer is correct! Thank you !

aniston. you are the God.


Duplicated DiscreteFormatStyle in tests

We started getting errors about duplicated symbols now that DiscreteFormatStyle has landed in SDK. Workaround this by adding a typealias like other types.

@swift-ci please test


do sherpa use espeak-ng? espeak-ng is GPL, is there any license issue?

I found out that Sherpa tts android include such so "libespeak-ng.so", is it related to espeak-ng repo? that's a GPL, is there any license or legal risk I use sherpa tts for non-opensource since sherpa is Apache 2?

Please have a look at the following issues/dicussions:

https://github.com/espeak-ng/espeak-ng/issues/912
https://github.com/rhasspy/piper/issues/93
https://github.com/espeak-ng/espeak-ng/issues/908

Note that we are using a fork of https://github.com/rhasspy/espeak-ng that can be found at
https://github.com/csukuangfj/espeak-ng and we are using dynamic link for Android.
The owner of espeak-ng says the following:
https://github.com/espeak-ng/espeak-ng/issues/912#issuecomment-817333378

This site of Speak NG project is used for collaboration of eSpeak NG users and developers. It is not meant neither for court nor police. If mentioned software violates license, you can report it in https://gpl-violations.org/ or https://www.gnu.org/licenses/gpl-violation.html, or similar place.

Please have a look at the following issues/dicussions:

GPLv3 license violation espeak-ng/espeak-ng#912
License question rhasspy/piper#93
Licensing question regarding linking espeak-ng/espeak-ng#908

Note that we are using a fork of https://github.com/rhasspy/espeak-ng that can be found at https://github.com/csukuangfj/espeak-ng and we are using dynamic link for Android.
The owner of espeak-ng says the following: espeak-ng/espeak-ng#912 (comment)

This site of Speak NG project is used for collaboration of eSpeak NG users and developers. It is not meant neither for court nor police. If mentioned software violates license, you can report it in https://gpl-violations.org/ or https://www.gnu.org/licenses/gpl-violation.html, or similar place.

Thanks for quick response and you guys awesome project BTW. I got your point based on those links, we don't need involve more legal discussion. But from android area and my pass experience and https://github.com/espeak-ng/espeak-ng/issues/908, maybe the best way of using project related to espeak, such as sherpa, piper might be through IPC and opensource and deliver that part as GPL. Just for later fellow's information.


bug: hcl2json_linux_arm64 is missing the x permission flag

Is there an existing issue for this?

[X] I have searched the existing issues

Bootstrap or Starter Module Deployment? (Required)
bootstrap
ALZ-PowerShell-Module Version (Required)
2.0.3
alz-terraform-accelerator Version (Required)
1.0.6
Starter Module (Required)
basic
Input arguments of the ALZ-PowerShell-Module (Required)
Deploy-Accelerator -i "terraform" -b "alz_local"

Contents of the configuration file used for the complete starter module. (Optional)
No response
Debug Output/Panic Output (Optional)
Deploy-Accelerator -i "terraform" -b "alz_local"
Getting ready to deploy the accelerator with you...

Checking you have the latest version of Terraform installed...

Checking and Downloading the bootstrap module...
The directory for ./bootstrap/v1.0.6 has been created and populated.

Checking and Downloading the starter module...
The directory for ./starter/v2.0.3 has been created and populated.
Please select the starter module you would like to use, you can enter one of the following keys:
- basic
- complete
- complete_vnext
- hubnetworking
: basic
ResourceUnavailable: /usr/local/share/powershell/Modules/ALZ/2.0.3/Private/Config-Helpers/Convert-HCLVariablesToUserInputConfig.ps1:21:9
Line |
  21 |          $terraformVariables = & $hclParserToolPath $targetVariableFil …
     |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Program 'hcl2json_linux_arm64' failed to run: An error occurred trying to start process '/workspaces/dev-container/bootstrap/v1.0.6/hcl2json_linux_arm64' with working directory '/workspaces/dev-container'. Permission deniedAt
     | /usr/local/share/powershell/Modules/ALZ/2.0.3/Private/Config-Helpers/Convert-HCLVariablesToUserInputConfig.ps1:21 char:31 + … terraformVariables = & $hclParserToolPath $targetVariableFile | Conve … +                        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~.
ResourceUnavailable: /usr/local/share/powershell/Modules/ALZ/2.0.3/Private/Config-Helpers/Convert-HCLVariablesToUserInputConfig.ps1:21:9
Line |
  21 |          $terraformVariables = & $hclParserToolPath $targetVariableFil …
     |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Program 'hcl2json_linux_arm64' failed to run: An error occurred trying to start process '/workspaces/dev-container/bootstrap/v1.0.6/hcl2json_linux_arm64' with working directory '/workspaces/dev-container'. Permission deniedAt
     | /usr/local/share/powershell/Modules/ALZ/2.0.3/Private/Config-Helpers/Convert-HCLVariablesToUserInputConfig.ps1:21 char:31 + … terraformVariables = & $hclParserToolPath $targetVariableFile | Conve … +                        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~.
ResourceUnavailable: /usr/local/share/powershell/Modules/ALZ/2.0.3/Private/Config-Helpers/Convert-HCLVariablesToUserInputConfig.ps1:21:9
Line |
  21 |          $terraformVariables = & $hclParserToolPath $targetVariableFil …
     |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | Program 'hcl2json_linux_arm64' failed to run: An error occurred trying to start process '/workspaces/dev-container/bootstrap/v1.0.6/hcl2json_linux_arm64' with working directory '/workspaces/dev-container'. Permission deniedAt
     | /usr/local/share/powershell/Modules/ALZ/2.0.3/Private/Config-Helpers/Convert-HCLVariablesToUserInputConfig.ps1:21 char:31 + … terraformVariables = & $hclParserToolPath $targetVariableFile | Conve … +                        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~.

Expected Behaviour (Required)
The binary hcl2json_linux_arm64 should be configured to be executable.
Actual Behaviour (Required)
The permissions of the binary are missing thx x flag. Therefore it can not be executed.
Workaround:

Start the process once, let it download the binary.
Find the binary
Use chmod +x  hcl2json_linux_arm64
restart the process

Steps to Reproduce (Optional)
I am building a devcontainer to use with alz.

Build this container:

FROM mcr.microsoft.com/devcontainers/base:ubuntu-22.04

ARG PS_VERSION=7.4.2

RUN sudo apt-get update

# Install az cli
RUN curl -sL https://aka.ms/InstallAzureCLIDeb | sudo bash

# Install powershell
RUN sudo mkdir /usr/local/bin/powershell
RUN wget https://github.com/PowerShell/PowerShell/releases/download/v${PS_VERSION}/powershell-7.4.2-linux-$(dpkg --print-architecture).tar.gz -O powershell.tar.gz
RUN sudo tar -xf powershell.tar.gz -C /usr/local/bin/powershell
RUN rm powershell.tar.gz

# Install ALZ Module
USER root
RUN /usr/local/bin/powershell/pwsh -c "Install-Module -Name ALZ -Force -Scope AllUsers"

# Install kubectl
RUN curl -LO "https://dl.k8s.io/release/$(curl -L -s https://dl.k8s.io/release/stable.txt)/bin/linux/$(dpkg --print-architecture)/kubectl"
RUN sudo install -o vscode -g vscode -m 0755 kubectl /usr/local/bin/kubectl
RUN rm kubectl

# Install terraform
RUN wget https://releases.hashicorp.com/terraform/1.8.2/terraform_1.8.2_linux_$(dpkg --print-architecture).zip -O terraform.zip
RUN unzip terraform.zip
RUN sudo install -o vscode -g vscode -m 0755 terraform /usr/local/bin/terraform
RUN rm terraform LICENSE.txt terraform.zip

Run the container
Start powershell
Start a basic bootstrapping

Important Factoids (Optional)
No response
References (Optional)
No response

@arne21a This is interesting as we do set this switch here: https://github.com/Azure/ALZ-PowerShell-Module/blob/cc905c81af9baf502ea729986514dfdc279e25dc/src/ALZ/Private/Tools/Get-HCLParserTool.ps1#L29
Our e2e tests all pass successfully on linux and macos, so wondering what is different for you?

Closing as no response and unable to replicate.


Kadane's Algorithm

Hi, Project Maintainers. I would like to include this important algorithm in the arrays section of the code. The folder in which I'll add code is Algo-Tree/code/java. I would like you to assign this issue to me. Thank you.

I would like to work on this issue in c++,kindly assign it to me.
Thank you

#178 is same


Ensure cron events can be passed as event topics

Please tell Chat GPT how to summarize this PR:
Includes cron + legacy events in allowable topics
Tell code reviewer how and what to test:
Attempt to pass a cron topic in via the -t/--topics flag or FLATFILE_AGENT_TOPICS

Feel like this is a good quick win but can we get the * wildcard topic work in to the Platform asap?

Closed since we account for that already


add an R version (and maybe others) to "testing locally" and CI episodes

should be straightforward to add code tabs also in those episodes

For testing locally this is part of #215


Observer & Proxy Abstraction Layer (#37)

Implements the changes stated in #37.
Breaks:

List Support
setReactStrategy (deprecated anyways)

The changes very fundamental and a  lot of documentation is outdated.
Follow up work:

Create examples for the new classes
Change ProxyCreator implementation
Update documentation

Testing
Some features are broken that should not be
In need of a fix:

ReactiveSubject
Everything that fails EXCEPT LIST SUPPORT

Renaming
Beside the ProxyCreator section all renames should be considered and applied
Style
All code should be formated correctly

Warnings
The number of warnings due to missing generics or unsafe casting should be reduced
Documentation
The new classes lack propper javadoc


Victor S Thela

Please "drag & drop" exercise 1 pdf file below.

Please "drag & drop" exercise 2 pdf file below.

Please "drag & drop" exercise 3 pdf file below.

Please "drag & drop" exercise 4 pdf file below.

Please UPDATE THE TITLE of this "Issue" in the title bar above with your name.

If you haven't done so, please go back to Module 2 and make sure to take the post-training survey. Participating in surveys is required for receiving the "GALUP Certificate" by the University of Florida
Distance to point of interest in Panda.pdf
.
[Uploading
Panda IDUs in proximity to cattle posts.pdf
Soil clay content suitability for Pandamatenga.pdf…]()

Soil clay content suitability for Pandamatenga.pdf
I have re-attached the exercise


Q: ImageDownloader.defatult.requestModifier

Hi
Previous I had this code:
    let downloader = KingfisherManager.shared.downloader

    downloader.requestModifier = {
        (request: NSMutableURLRequest) in

    }

but now its not available, how can I solve this with the latest version of Kingfisher?

It is now an option. Please see the migration guide and our cheat sheet for more.


[Question] MMKV Limitations as Text-based Database

Not sure if I am allowed to ask this in issues although I did not know where else I could ask this and I desperately need some input.
I am planning on using MMKV to store user data for my Expo app. Yes I am aware that it should not be used as a database although I wanted to ask and see if it would be recommended against for my use case.
It is all text, similar to a notes/flashcard app. I doubt the average user will go over 2-3mb of storage (which is like 3,000,000 characters). So let us say absolute maximum 3-5mb in user storage.
The way I am going to do this is by creating a counter. So for example you create one deck then it is called 'deck_1'. Then counter increments because there's already a Deck. Then next deck is 'deck_2'. When reading it searches for all instances of 'deck_{x}' and then renders it.
Then for each word in a deck, we can do 'deck_1_word_1'. Then next word is 'deck_1_word_2' with the incrementation of a counter. This is the best way I think I should do it.
My only fear is if the data grows and becomes more complex, would it be more difficult then to loop through all instances of 'deck_1_word_{x}'?
Would this be recommended for or against? Thank you!

Hi where can I ask questions?


Expeditions needs gamer loot removed

dork and I are bikeshedding a better version that's less of a powergamer trap but in the interim remove the heals and the RPG and shit.

Okay but like what would we replace them with? Guns make up 75% of the epic loot table.

For now just anything station-oriented like instrument crates or whatever.
We're bikeshedding removing the explicit mission structure and giving it more long-term progression.

me when epic mission giving a plushie crate (real)
also im gonna add some money to each tier since actually getting directly paid seems logical

rpg brutekits medkits swat removed can prob close

So, what's the vision for expeditions? Currently, it's a straight combat challenge, on a role not actually allowed real combat equipment (which near-inevitably is issued [sanctioned or illicit] laser pistols by the QM to be able to do their jobs with a lower total-party-kill rate, and hoards anything else combat-relevant they can find, at least when I played while PKAs were removed), and it's going to earn strictly combat-irrelevant and other-job irrelevant fluff items?
I enjoyed when salvagers could take requests from other departments and be on the lookout to recover equipment and components of real value to the station on expeditions, it's sad to hear the dungeons being remapped to more or less static 'junk' scenery of no value.
Perhaps breaking up the 'combat check' and 'swarm of locusts loot everything' phases of expeditions could be done by breaking the current antagonistic relationship between Supply and Security and actually allowing trained, equipped soldiers onto away missions, Star Trek style (with a probably similar rate of fatalities, unfortunately).
Is the intent that more than just Supply goes on away missions? (Security officers as actual military help rather than commissars hunting for the disloyal, possibly medics or engineers for efficient stripping of the dungeon, etc.) Then having a staging area that's attached to, but not directly in the backrooms of Cargo, would be handy. People from other departments (especially Sec) entering Cargo is usually seen as an accusation of disloyalty, from what I've seen playing a lot of CT/Salv, and that tendency could be mitigated somewhat by having Salvage embark from a more 'neutral' area, while also making it easier to invite outsiders in without giving them access to whatever Cargo has lying around.

you are describing gateway


Swap hex in gradients.json for Mauve + Lush gradients

My bad, intention was to render the gradient in this order, had mistakenly reversed them.

You actually only need edit the gradients.json in the root. The rest gets done via the build process.

@Ghosh I was going to commit only gradients.json file, but since you have not added the build files in .gitignore (which we should), so I thought you want to commit the ones which are built to your repo as well.
In this case, we should add build/scripts/site.js + build/styles/site.css, or am I missing something?

I cannot add the build folder into the .gitignore. uigraidents is hosted via github pages and github cannot run a site out of a folder. The deployment script actually extracts all the code out of the build folder and pushes it into the gh-pages branch which then becomes the root of the site.
For adding a gradient you really just need to edit only the gradients.json file in the root. Its actually all there in the contributing guidelines.

@Ghosh Cool, got it, will do the needful.


Enhance Map accessibility

The main PR was closed because we renamed the master branch to main.
For more infos look into: #6248

#6248:
Changes:
Replace anchor tags with buttons.
Add Alt attribute to images.
Add roles to elements.
Make tooltip focusable.

Should this be closed? I think it's doing a bit much for one PR. And it's changing <a> to <button> which is a breaking change that blocks (until v2?) some non-breaking changes that this PR also introduces?
Anyhow, there are unresolved comments (in the original PR https://github.com/Leaflet/Leaflet/pull/6248).

@Malvoz would it be ok for you, when you create new PRs with the useful suggestions from this PR and then close this one?

create new PRs with the useful suggestions from this PR and then close this one?

Most suggestions in this PR are specific to this PR. Though almost everything in here is proposed in separate issues:

Replace anchor tags with buttons

Issue: https://github.com/Leaflet/Leaflet/issues/7821.

Add Alt attribute to images

The new default was added in https://github.com/Leaflet/Leaflet/pull/7895.

Add roles to elements

(This PR only adds 1 new role - tooltip - which I think is an incorrect role suggestion.)
Issue: https://github.com/Leaflet/Leaflet/issues/7968.

Make tooltip focusable

(Again I think popup is confused with tooltip in this PR)
Issue: https://github.com/Leaflet/Leaflet/issues/2199.

I don't see this PR going anywhere since the original author hasn't responded in a long time, so let's close this.