Providing the ability to supply a bit more help text, beyond just the name of the command.

This will make devices that leverage the shell more end-user-friendly.

@brucemack I am a bit concerned about increased RAM usage to support help text.
This proposal increases the size of each SimpleSerialShell::Command by one pointer (about a 33% increase), even when there is no help text.  This is significant for an Arduino UNO, which only has 2k RAM.
I may be the only person who is still using the original platform, but I haven't given up on it yet.  (I understand my obsession may not be healthy, save me money or save me time.)
Is there some way to avoid extra overhead on ALL commands, including legacy commands with no help text?
Possible workarounds:

for the commands that need extra hints, support a -help argument.
When a command detects a syntax error, print out the appropriate help text.  For example the garbled command motor 3storp might return motor: commands are 'motor stop' and 'motor start'
Add global printMoreHelp or explain XYZ commands and put the text there.
Provide a paper "commands cheat sheet". This has zero overhead for the controller, supports graphics etc.  Just remember where you put the sheet.

Good point, let me try something different.

I've got one approach that involves a delimiter.  I'm using : for now.
Would you be open to that?  I don't want to bother you with a PR if this concept is DOA.
Thanks, Bruce

One other related question since you are obviously experienced architecting for small footprints.
I've been told (in other contexts) to stay away from the Arduino String class in places where memory use really matters because of the risk of inefficient heap usage/fragmentation that can occur as a result of the realloc() activity.  You've got places in your code where there are Strings being created/destroyed a lot.  Does this matter?  My implementation removes the Strings, but I want to make sure I am not wasting my time.
Thanks for your help.

Yes, you should be careful how you use String objects (or any objects), as they can break the allocation heap into unusable fragments.  This is a serious problem for code adding/removing objects a lot, if there is no garbage collection.  A good hint there may be a problem is if you use new and delete -- how do you guarantee you will delete the object?  (Uh oh, better check... my usage is still looks OK...)
I use String in three different places in the code.  In each of these cases each String is a local variable (so no new or delete).  The fixed-size String object is allocated on the stack, and the string's arbitrary-size contents are on the heap.  But since these are local variables, when the function returns, the object and the heap contents are guaranteed freed up again.
So long as there is enough free contiguous memory on the heap to allocate the String in the first place, everything should be OK.  If something else has fragments the heap too much... that is trouble and this code may not work right.
So... I may be using Strings safely, but do really I need to use them?
No Maybe not.  Mostly I use them because in general I think they improve readability.  In fact, now that you point it out, they may not do much more than ensure that input character arrays are null-terminated -- that is a significant real-world problem!   (Addendum:  I think part of the reason I'm using Strings is because regular case comparison doesn't like to work directly with __FlashStringHelper pointers. String helps get ready for strncasecmp().)
The other code that uses the heap is the new call in addCommand().  These calls permanently allocate small pieces of the heap for each Command -- there is no associated delete.  I expect Commands to be added at startup, so hopefully they are all clustered at one end of the heap.  The rest of the (free) heap is still contiguous.

I've got one approach that involves a delimiter. I'm using : for now.

This looks promising.  It does not add RAM overhead to support the hints.  Legacy code is still compatible.
Would you need to modify help()?
(The cheat-sheet approach works for me, but that's mostly because I don't have that many commands -- or users.)

Phil: Thanks for taking the time to write that informative reply on the String usage.  What you are saying makes a lot of sense.  strncasecmp_P() addresses your comment about the ability to compare between RAM and PROGMEM storage.  Sadly, EpoxyDuino is missing that function, but I'll PR that with Brian. :-)  I'll need that for another project anyhow.  Meanwhile, I will leave the Strings the way they are.
Yes, I modified help() and Command::compareName() to support my scheme.
I've got nothing against cheat sheets (or yellow stickies!!), but I'm working on some multi-user projects that have several commands.  The picture below illustrates one help page that I'll be very happy to be able to reproduce (if possible) on your excellent shell.
Do you mind if I PR a proposed change?
Thanks again for all of your guidance.
Bruce

Yes, please submit the PR (this PR?).  Your cheat sheet provides a good example of when yellow stickies stop working :smile:.
It's useful to have the help text as close as possible to the associated command -- that makes it easier to remember to keep it current.

Closing this one based on feedback from Phil on excessive memory footprint.  Another will follow.


feat(base_note): ✨ add next and previous getters

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1121

Relevant Lines:
1121

💛  - Coveralls


How to trains a 8 scales network?

Hi:
The pictures I generated using my own data set are not very good. I want to know how to get a best 8 scales weights.
Thanks

This question is a little too vague, please be more specific on what issues you're encountering.

My training set is 1080 resolution pictures, and they are first downsampled and then partially blurred to generate 128 resolution pictures. Then use 128 resolution and 1080 resolution as training set for training. The network architecture used is RRDN, but for training it, the psrn value is always 20 to 21 particularly small, and the chromaticity of the generated picture is distorted.

so it seems you're using only the gan loss. I would recommend to first train with the psnr loss and see the results afterafter 30-40 epochs. If that is going in the right direction you could either continue (with the open souirce dataset I trained until 70-90 epochs with PSNR) or you introduce the gan loss. If I remember correctly you just need to change the loss weights to do this, or don't pass a discriminator network. It's probably safest to simply set all losses to zero other than generator loss and than (maybe gradually) change this.
You could also first try without blurrying the pictures.
Best of luck!

Thank you very much for your guidance, I hope to communicate a lot in the future and make progress together.

Hi, What do these parameters stand for,
loss_weights =
{
'generator': 0.8,
'feature_extractor':0.0833 ,
'discriminator': 0.01
}
and how do you determine their values during training? How were they determined during your training? Also, can you share your specific training details? For example, which one to train first, then how to train, thank you very much!

Hi,
generator is the loss weights for the plain network, having this to 1 and the others to 0 will only do pixelwise MAE with the original image.
feature_extractor is the weight to the MAE loss using VGG extracted features from the upscaled pictres and VGG extracted features from the original image.
Discriminator is the GANs loss.
First use only generator, the use a combination. You have to experiment. I would always use first generator (and maybe some feature) and then 0 generator with feature and GANs

Thank you very much! Under your guidance, I feel that my training is about
to succeed!
Francesco<EMAIL_ADDRESS>周一19:05写道：

Hi,
generator is the loss weights for the plain network, having this to 1 and
the others to 0 will only do pixelwise MAE with the original image.
feature_extractor is the weight to the MAE loss using VGG extracted
features from the upscaled pictres and VGG extracted features from the
original image.
Discriminator is the GANs loss.
First use only generator, the use a combination. You have to experiment. I
would always use first generator (and maybe some feature) and then 0
generator with feature and GANs
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/idealo/image-super-resolution/issues/127#issuecomment-647447021,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APPVMWUYWWTRV64XHSZUQZTRX43F5ANCNFSM4N6FKXRQ
.

When only the mae loss training is performed, to what extent can the
comprehensive loss training be performed? The picture generated by my mae
loss is a little blurry, not very clear. Can the mae loss only achieve the
desired result?
Xun Wsng<EMAIL_ADDRESS>周一19:07写道：

Thank you very much! Under your guidance, I feel that my training is
about to succeed!
Francesco<EMAIL_ADDRESS>周一19:05写道：

Hi,
generator is the loss weights for the plain network, having this to 1 and
the others to 0 will only do pixelwise MAE with the original image.
feature_extractor is the weight to the MAE loss using VGG extracted
features from the upscaled pictres and VGG extracted features from the
original image.
Discriminator is the GANs loss.
First use only generator, the use a combination. You have to experiment.
I would always use first generator (and maybe some feature) and then 0
generator with feature and GANs
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/idealo/image-super-resolution/issues/127#issuecomment-647447021,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APPVMWUYWWTRV64XHSZUQZTRX43F5ANCNFSM4N6FKXRQ
.

Hi, no, only using mae you will get less defined pictures. You can try adding only the feature loss, if training with GANs is too difficult


STREAMS-463: Move every class in all repos underneath org.apache.streams

👍 this should improve checkstyle score considerably


Match the type of Miou_unix.write with Unix.write and Lwt_unix.write

Both Lwt_unix.write and Unix take bytes as input instead of string. I think keeping it this way would make it easier to miou-ify existing projects.
Also, as a side note: one of the example makes it clear that people usually already have a bytes type and not a string type when using this function.

I'm a bit reluctant to make such a change as I've already had quite a few ownerships problems due to the use of bytes (this brings back bad memories of mirage-tcpip).
I'd be more in favour of proposing unsafe_write with good documentation explaining that the buffer should not be modified until the system call is complete. An bad example will be:
let buf = Bytes.create 0x100 in
let prm0 = Miou.call ~give:[ Miou_unix.owner fd ] (fun () -> Miou_unix.read fd buf ~off:0 ~len:(Bytes.length buf)) in
let prm1 = Miou.call ~give:[ Miou_unix.owner fd ] (fun () -> Miou_unix.write fd buf ~off:0 ~len:(Bytes.length buf)) in
Miou.await_all [ prm0; prm1 ]

It should be noted that with the use of Miou.call_cc (concurrency), this code could work as long as a certain order is expected in the execution of the tasks (which should never be the case). The example may seem dumb, but we've already found ourselves in this type of situation when:

we generally want to have just one buffer (and avoid allocations) throughout the process
when you forget about the notion of ownership on other parts (deep in the stack), which is not explained anywhere

I'd definitely prefer an unsafe_write :+1:.

ok that's fair. I'll close this then. Thanks!


[incubator-kie-issues-1146] Add support for transformations in input/output in activities

Closes: https://github.com/apache/incubator-kie-issues/issues/1146

PR job #1 was: UNSTABLE
Possible explanation: This should be test failures

Reproducer
build-chain build full_downstream  -f 'https://raw.githubusercontent.com/${AUTHOR:apache}/incubator-kie-kogito-pipelines/${BRANCH:main}/.ci/buildchain-config-pr-cdb.yaml' -o 'bc' -p apache/incubator-kie-kogito-runtimes -u https://github.com/apache/incubator-kie-kogito-runtimes/pull/3494 --skipParallelCheckout
NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest_jobs/job/kogito-runtimes-pr/job/PR-3494/1/display/redirect
Test results:

PASSED: 3264
FAILED: 4

Those are the test failures:

org.kie.kogito.codegen.process.ProcessGenerationIT.testProcessGeneration(String)[10]
[EventSubProcessNode] nodes with name [Sub Process 1] are not equal

org.kie.kogito.codegen.process.ProcessGenerationIT.testProcessGeneration(String)[123]
[StartNode] nodes with name [StartProcess] are not equal

org.kie.kogito.codegen.process.ProcessGenerationIT.testProcessGeneration(String)[124]
[StartNode] nodes with name [consumers] are not equal

org.kie.kogito.codegen.process.ProcessGenerationIT.testProcessGeneration(String)[125]
[StartNode] nodes with name [StartProcess] are not equal

PR job #3 was: UNSTABLE
Possible explanation: This should be test failures

Reproducer
build-chain build full_downstream  -f 'https://raw.githubusercontent.com/${AUTHOR:apache}/incubator-kie-kogito-pipelines/${BRANCH:main}/.ci/buildchain-config-pr-cdb.yaml' -o 'bc' -p apache/incubator-kie-kogito-runtimes -u https://github.com/apache/incubator-kie-kogito-runtimes/pull/3494 --skipParallelCheckout
NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution

Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest_jobs/job/kogito-runtimes-pr/job/PR-3494/3/display/redirect
Test results:

PASSED: 3264
FAILED: 4

Those are the test failures:

org.kie.kogito.codegen.process.ProcessGenerationIT.testProcessGeneration(String)[6]
[EventSubProcessNode] nodes with name [Sub Process 1] are not equal

org.kie.kogito.codegen.process.ProcessGenerationIT.testProcessGeneration(String)[96]
[StartNode] nodes with name [consumers] are not equal

org.kie.kogito.codegen.process.ProcessGenerationIT.testProcessGeneration(String)[97]
[StartNode] nodes with name [StartProcess] are not equal

org.kie.kogito.codegen.process.ProcessGenerationIT.testProcessGeneration(String)[98]
[StartNode] nodes with name [StartProcess] are not equal


Swift 3

Notes:
This branch solely addresses migration to Swift 3 and changes to the codebase were made to accomodate this migration.
Further changes to the codebase are outside the scope of this pull request.
Changes:

Migrate to Swift 3.
Change Xcode schemes and target names, making them dynamic.

@g-ackt  Thanks for sharing this - would you be able to look at the other Swift 3 update branches and see how your approach differs? Ideally we should get this to Swift 3 for the next point release.

I have picked a working fork I knew with Swift 3 support and merged that in #68 -- thanks for your contribution back in the day! 🙌 If I missed something cool you did, please create a PR for the changes based off of master.

Also, thanks a lot for contributing to SwiftCSV, @grantkemp and @chamander ! I've invited you to join the SwiftCSV GitHub organization – no pressure to accept! If you'd like more information on what that means, check out our contributor guidelines.


doc: distro.tex: fix figure reference

This patch fixes the wrong "??" reference to the first figure of the
document, page 4.

I tested it on my laptop, and it appears the bug comes from the package caption.
If I removes the caption package, I is working. Could you test it?

I tested on Ubuntu 14.04 and it works as well (without dropping the package caption).
But it still does not work on Fedora, I will investigate later why it works when the caption package is dropped.
Thanks


Day of the week in fitness trend #503

This seems like a matter of personal preference. There will be many people that prefer to see the plain date without day of week.
If implemented, it should be set through user preferences.

So we could add it as option in FT config? If yes wait a bit. One of my upcoming feature will introduce strong changes in FT code


778 tests vdataframe machine learning + join union sort + pivot

UT for machine learning, join union sort  and pivot files.

why 300 tests are failing?

why 300 tests are failing?

@oualib - I checked logs and looks like plotting related tests are failing.
@mail4umar - could you pls have a look at plotting test?
verticapy/tests_new/mlops/test_model_versioning.py .....                 [ 75%]
verticapy/tests_new/plotting/highcharts/test_highcharts_acf.py F.F..FF   [ 75%]
verticapy/tests_new/plotting/highcharts/test_highcharts_bar.py F....FFFF [ 75%]
FF....FFFF                                                               [ 75%]
verticapy/tests_new/plotting/highcharts/test_highcharts_barh.py FFF..FFF [ 76%]
FFFFFF..FF                                                               [ 76%]
verticapy/tests_new/plotting/highcharts/test_highcharts_boxplot.py FF..s [ 76%]
F..ssF..sF                                                               [ 76%]
verticapy/tests_new/plotting/highcharts/test_highcharts_candlestick.py . [ 76%]
ssFFFFs                                                                  [ 76%]
verticapy/tests_new/plotting/highcharts/test_highcharts_contour.py F.... [ 77%]
FF                                                                       [ 77%]
verticapy/tests_new/plotting/highcharts/test_highcharts_density.py F.... [ 77%]
FFFF....F....FFF                                                         [ 77%]
verticapy/tests_new/plotting/highcharts/test_highcharts_heatmap.py F.... [ 77%]
FFsssssss                                                                [ 78%]
verticapy/tests_new/plotting/highcharts/test_highcharts_hist.py F...FF.F [ 78%]
...ss.                                                                   [ 78%]
verticapy/tests_new/plotting/highcharts/test_highcharts_line.py F...sFFF [ 78%]
FFF.F..                                                                  [ 78%]
verticapy/tests_new/plotting/highcharts/test_highcharts_outliers.py F... [ 78%]
.FF....                                                                  [ 79%]
verticapy/tests_new/plotting/highcharts/test_highcharts_pie.py FFFFFF    [ 79%]
verticapy/tests_new/plotting/highcharts/test_highcharts_range.py F....FF [ 79%]
F....FF                                                                  [ 79%]
verticapy/tests_new/plotting/highcharts/test_highcharts_scatter.py F.... [ 79%]
FFFFF....FF                                                              [ 79%]
verticapy/tests_new/plotting/highcharts/test_highcharts_spider.py FFF    [ 80%]
verticapy/tests_new/plotting/highcharts/machine_learning/test_highcharts_champion_challenger.py F [ 80%]
[ 80%]

why 300 tests are failing?

@oualib - I checked logs and looks like plotting related tests are failing. @mail4umar - could you pls have a look at plotting test?
verticapy/tests_new/mlops/test_model_versioning.py ..... [ 75%] verticapy/tests_new/plotting/highcharts/test_highcharts_acf.py F.F..FF [ 75%] verticapy/tests_new/plotting/highcharts/test_highcharts_bar.py F....FFFF [ 75%] FF....FFFF [ 75%] verticapy/tests_new/plotting/highcharts/test_highcharts_barh.py FFF..FFF [ 76%] FFFFFF..FF [ 76%] verticapy/tests_new/plotting/highcharts/test_highcharts_boxplot.py FF..s [ 76%] F..ssF..sF [ 76%] verticapy/tests_new/plotting/highcharts/test_highcharts_candlestick.py . [ 76%] ssFFFFs [ 76%] verticapy/tests_new/plotting/highcharts/test_highcharts_contour.py F.... [ 77%] FF [ 77%] verticapy/tests_new/plotting/highcharts/test_highcharts_density.py F.... [ 77%] FFFF....F....FFF [ 77%] verticapy/tests_new/plotting/highcharts/test_highcharts_heatmap.py F.... [ 77%] FFsssssss [ 78%] verticapy/tests_new/plotting/highcharts/test_highcharts_hist.py F...FF.F [ 78%] ...ss. [ 78%] verticapy/tests_new/plotting/highcharts/test_highcharts_line.py F...sFFF [ 78%] FFF.F.. [ 78%] verticapy/tests_new/plotting/highcharts/test_highcharts_outliers.py F... [ 78%] .FF.... [ 79%] verticapy/tests_new/plotting/highcharts/test_highcharts_pie.py FFFFFF [ 79%] verticapy/tests_new/plotting/highcharts/test_highcharts_range.py F....FF [ 79%] F....FF [ 79%] verticapy/tests_new/plotting/highcharts/test_highcharts_scatter.py F.... [ 79%] FFFFF....FF [ 79%] verticapy/tests_new/plotting/highcharts/test_highcharts_spider.py FFF [ 80%] verticapy/tests_new/plotting/highcharts/machine_learning/test_highcharts_champion_challenger.py F [ 80%] [ 80%]

@vikash018  I don't think it has something specifically to do with plotting test but the test structure. I think the plotting library was maybe changed during the tests leading to wrong object creation. Idk how is it possible with your PR. Maybe because of an import. Please avoid the "import *".

why 300 tests are failing?

@oualib - I checked logs and looks like plotting related tests are failing. @mail4umar - could you pls have a look at plotting test?
verticapy/tests_new/mlops/test_model_versioning.py ..... [ 75%] verticapy/tests_new/plotting/highcharts/test_highcharts_acf.py F.F..FF [ 75%] verticapy/tests_new/plotting/highcharts/test_highcharts_bar.py F....FFFF [ 75%] FF....FFFF [ 75%] verticapy/tests_new/plotting/highcharts/test_highcharts_barh.py FFF..FFF [ 76%] FFFFFF..FF [ 76%] verticapy/tests_new/plotting/highcharts/test_highcharts_boxplot.py FF..s [ 76%] F..ssF..sF [ 76%] verticapy/tests_new/plotting/highcharts/test_highcharts_candlestick.py . [ 76%] ssFFFFs [ 76%] verticapy/tests_new/plotting/highcharts/test_highcharts_contour.py F.... [ 77%] FF [ 77%] verticapy/tests_new/plotting/highcharts/test_highcharts_density.py F.... [ 77%] FFFF....F....FFF [ 77%] verticapy/tests_new/plotting/highcharts/test_highcharts_heatmap.py F.... [ 77%] FFsssssss [ 78%] verticapy/tests_new/plotting/highcharts/test_highcharts_hist.py F...FF.F [ 78%] ...ss. [ 78%] verticapy/tests_new/plotting/highcharts/test_highcharts_line.py F...sFFF [ 78%] FFF.F.. [ 78%] verticapy/tests_new/plotting/highcharts/test_highcharts_outliers.py F... [ 78%] .FF.... [ 79%] verticapy/tests_new/plotting/highcharts/test_highcharts_pie.py FFFFFF [ 79%] verticapy/tests_new/plotting/highcharts/test_highcharts_range.py F....FF [ 79%] F....FF [ 79%] verticapy/tests_new/plotting/highcharts/test_highcharts_scatter.py F.... [ 79%] FFFFF....FF [ 79%] verticapy/tests_new/plotting/highcharts/test_highcharts_spider.py FFF [ 80%] verticapy/tests_new/plotting/highcharts/machine_learning/test_highcharts_champion_challenger.py F [ 80%] [ 80%]

@vikash018 I don't think it has something specifically to do with plotting test but the test structure. I think the plotting library was maybe changed during the tests leading to wrong object creation. Idk how is it possible with your PR. Maybe because of an import. Please avoid the "import *".

@oualib - oh i see. I was doing one import in master conftest file. Now, I have moved this conftest import to test_machine_learning.py to avoid import at master conftest level. Hopefully this should resolve issue.


Bug with owner only

Hey can you put owner only back please I was currently using this feature and i don't know why it's disable. it was so useful please put it back, someone change the avatar of my bot with that

The property has just been transformed to a Tag, see how tags should works here.

The property has just been transformed to a Tag, see how tags should works here.

well thanks but do you have new event message though the template ? like v1.1.1 with test bot

The default messages/commands ?


Add file names to model and texture menu

Currently it's hard to tell model apart if the preview image is at an awkward angle. For example if the model is a plane, the model isn't visible in the preview. Adding the file name to the bottom of each of the object buttons would make asset browsing easier.
(I attached a mock-up image, sorry it's a bit silly looking)

Hi Lizband! Thank you for reporting these issues. This feature seems like a really nice idea. Glad you're continuing to use UltraEd and finding more areas where it can be improved. The mock up looks great and I'll start thinking how I'd go about implementing this. Thanks for reporting the other issues as well and I'll take a look at those too. I was trying to figure out how editor's like Unity take care when the model being imported is flat. They have some algorithm that figures out how to orient the model so it's always visible in the preview.

Not a problem. Thank you so much for all of your work on this project!


[Feature Request] Option to reduce Minutes selector to 15 Minute intervals

Just like the title says. It would be great to optionally reduce the minutes selector to 15 minute intervals for quicker time selection.

Sadly this can't currently be done because the picker interface for selecting the date and time is the default browser interface which I cannot modify.


TS-5087: Updates the AL2 license on all .po files

Fixes for 6.2.x branch that were not upstream.

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1269/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/1164/ for details.


Allow customizing of the default theme in theme_picker_ui

Hi,
Thanks for this package, it is useful and fun to work with. I was wondering if it could be possible to allow changing of the default theme on startup for example to allow something like theme_picker_ui(selected = "zephyr")
I have looked at both the code and function help pages but cannot quite see how this is possible.
Thanks,
Nelson

Completed in #32 with follow up in #39


[zh]Translate reference/kubernetes-api/authorization-resources/local-subject-access-review-v1

Hi, I'm interested to translate the "reference/kubernetes-api/authorization-resources/subject-access-review-v1.md" page.
translate en/docs/reference/kubernetes-api/authorization-resources/local-subject-access-review-v1.md into Chinese.
What would you like to be added:
zh/docs/reference/kubernetes-api/authorization-resources/local-subject-access-review-v1.md
/language zh

/assign

https://github.com/kubernetes/website/pull/34034 worked on this and done.
/close


Unexpected token <

When I try to run I only see the word loading in the we browser.
During the process of running this error appears.
ERROR in ./app/client/actions/app.js
Module not found: Error: Cannot resolve module 'react-intl/lib/locale-data' in /Users/carapai/Projects/my-nutrition/app/client/actions
What I am doing wrong?

open the app.js file and adapt the path to 'react-intl/locale-data/'

Can you try with the latest version ? Also, what browser are you using ?

Thank you very much, the first option fixed the problem


Add logging for deserialization failures.

The empty catch block makes deserialization issues hard to debug. We had an issue between older producers and the latest version of spring-cloud-stream-kafka-streams (https://github.com/spring-cloud/spring-cloud-stream-binder-kafka/issues/509) and the exception was completely silent.

@arnaudjardine Thank you for the contribution. Merged the PR.


Update init file to not raise exception

I apologize if this is a duplicate of something you already have documented @j-wags. The main init file in openforcefield looks like it still raises an exception if OpenEye toolkits aren't installed. It also has a few commented out lines at the bottom that seem like they should be uncommented or removed.

Thanks for catching this! It's weird that our tests don't catch this. I'll take a look at this today, and schedule it for the upcoming milestone as I think this is critical.

Sorry to be late to this party. I just checked out @andrrizzi's PR #280 . I agree with him that this exception was getting caught by try/except logic before, but it did result in us double printing the "you didn't install the OpenEye Toolkit" message.
Looking into this also revealed a potential error with our build system (which is I think what @bannanc wanted to chat about tomorrow in Irvine).
Thanks for posting this issue. I didn't even realize that the top-level init had any logic in it before.
Agree that merging #280 should close this.


Decide on C++ standard to compile to and create build system builder (CMake).

[ ] Start top level CMakeLists.txt to specify overall library properties.
[ ] Decide on C++ Standard to build to.
[ ] CMake creates basic build files on Linux
[ ] CMake creates basic build files on Windows

0003-start-top-level-cmakelists

I took a guess that we would want to build to the most recent fully implemented standard.
So I put C++17 in the CMakeLists file.


Added the negative cycle detection in bellman ford

This PR solves the #76 issue.
I have added the code for detecting a Negative cycle in Bellman Ford Shortest Path Algorithm and I have also added a unit test to test the logic in the test directory.
I did an additional pass over the edges and if there is any change detected ,An Invalid Argument exception is thrown.
"A Negative cycle has been detected in the graph".The unit test checks if this exception is thrown for a negative cycle and all the tests are passed too.

Hey, Thanks for the detailed review, I have committed all the above changes. Let me know if any more changes are needed.


comments for basic and basicEmbedded might be mixed up

I am looking at the RunTests.c files in the test\basic and test\basicEmbedded. If you look at the file comments on line 28, it seems that the comments might be mixed up between the two files.
For example, in the test\basic folder it says the following:
/** Test executor for AceUnit unit test "basicEmbedded".

but in test\basicEmbedded it says
/** Test executor for AceUnit unit test "basic".

I am still learning this framework, so I am not sure if anything else in the files is mixed up as well, or if it is just those comments.

I think I was just confused about how something works.

Actually you're right, thanks for pointing that out. Thus, I reopen it.


[BatchMode] Accumulate SILModule counters across batch constituents.

This just fixes FrontendTool counters to be correct vis-a-vis batch mode / generally any time one makes multiple SILModules in the same frontend.

@swift-ci please test and merge

cc @jrose-apple and @davidungar but I think this one is obvious enough not to need review


Move libbeat/common/kubernetes/metadata package

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
      [](https://ci-stats.elastic.co/app/apm/services/beats-ci/transactions/view?rangeFrom=2022-03-09T14:25:54.800Z&rangeTo=2022-03-09T14:45:54.800Z&transactionName=BUILD elastic-agent-autodiscover-mbp/PR-{number}&transactionType=job&latencyAggregationType=avg&traceId=2926bb570ab38eedb0d0e3ed10a2f40b&transactionId=5a82b756e8ad684f)

Expand to view the summary

Build stats

Start Time: 2022-03-09T14:35:54.800+0000

Duration: 17 min 9 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
198

Skipped
0

Total
198

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.


Add support for multiline labels

Hi, It would be nice to support multiple lines in labels. I want to choose a chart using TGPDiscreteSlider but It has a long name e.g. Temperature Chart. I wanted to write name like "Temperature\nChart" but it only shows first line.
I have created a pull request with necessary changes #35

Closed with issue #35


Build fails every now and again

2:21:34 PM: hack/import-flux2-assets.sh
2:21:34 PM: jq: error (at <stdin>:1): Cannot index string with string "tag_name"
2:21:35 PM: ######################################################################## 100.0%curl: (22) The requested URL returned error: 404
2:21:35 PM: Don't know what to do with '' in /tmp/tmp.S4UVy2Mvrb.

Retrying once or twice "fixes" it, but it'd be good to know what fails here and why. Or have a retry logic.

Duplicate of #719.


[Condor-Release]: Migrated contract from v156 with stored session entry point fails to execute

Category
Request for Tech Support
Your email address
<EMAIL_ADDRESS>Project Name(Optional)
MAKE
Casper Network
Devnet
Priority
Medium
Node Functionality
Execution Engine
Description
We have a contract live on mainnet that contains an entry point with type EntryPointType::Session.
We are testing migration from 1.5.6 to Condor. Other entry points seem to work correctly but this one fails with error Invalid context.
On EntryPointType::Caller  description I can read: Previously installed 1.x stored session code should continue to work as the binary value matches. Since other methods are working, the binary doesn't seem to be the problem. Could you give any advice for troubleshooting?
Date Issue Began(optional)
25/07/2024
Attachments (optional)
No response

@davidatwhiletrue
Are you sending any Wasm to add a new version?
We’ve disabled EntryPoint::Session which is why you’re getting that error

I'm not sending any new wasm. The contract is installed on 1.5.6. The entry point can be called normally.
Then, I'm upgrading the network to 2.0.0. This entry point cannot be called successfully anymore.


Less opinionated form of Get?

Is your feature request related to a problem? Please describe.
I'm primarily interested in Get for the easy dialogs and Get.back() navigation in those dialogs. I'm not quite ready for "not using StatefulWidget" and the other things this library provides. Also, I'm unsure of the effects that switching from a MaterialApp to a GetMaterialApp has had on my app. For instance, are exceptions handled differently now? I keep seeing "another exception has occured" instead of a full stack trace. I'm not sure if that's related? Also, I'm unsure how that switch is affecting analytics plugins and other pre-existing routing.
Describe the solution you'd like
Instructions in the documentation on how to pick and choose certain parts of the functionality (like dialogs) without changing the fundamentals of how my application works.
I think this will help a lot with adoption!

1- As explained in the documentation, only the imported functions are compiled, so if you don't use the state manager or anything else, it won't make any difference to you, no line of code from the state manager will be compiled into your application.
2- "are exceptions handled differently now?" Get does not change the structure of Flutter in any way, there is no possibility that Get will interfere with exception handling in any way. GetMaterialApp is just a preconfigured MaterialApp for dynamic routes, if you don't want to use dynamic routes, just add the Get key to your navigatorKey, and you're done. This is also in the documentation:
https://github.com/jonataslaw/get#other-advanced-apis-and-manual-configurations
3- "without changing the fundamentals of how my application works." get offers a range of APIs not supported by Flutter, such as knowing when a snackbar is open, triggering specific actions when a dialog or page is opened (important if you have some kind of analytics in your application to check the most accessed pages). Get does not switch anything from Flutter, at least nothing that is not problematic, like bottomSheets which have a lot of errors, and Get uses its own widget to mitigate this. I don't understand what kind of "fundamentals of how my application works" Get could change. You can look directly at the source code of GetMaterialApp, and you will see that his son is a MaterialApp, and he only manipulates things like the route list to guarantee resources that are not present in Flutter, that's all.

Thanks so much! That answers my questions perfectly.


Clean up branches

Check if rebBTC branch is still relevant https://github.com/iearn-finance/iearn-finance/commits/renBTC. Cherry pick commits or remove branch.

I reviewed the renBTC branch.
This branch removes packages that are already removed, and comments out code that is no longer present.
Removing this branch.


Dark mode

So, there was this bug where onelist would show up in blue, and i found out this was caused by a values-night/colors.xml from a dependency. Then i accidentially implemented a dark mode that applies the os-theme. Currently lacks the ability to toggle/choose, and icons are not really nice since they retain their color

closes #3

Todo:

[ ] Fix settings-textcolor
[ ] Fix Icons not changing color
[ ] change color of the WhatsNew view


Training Does Not Work / Hangs - GPU Version

Hi,
I have just started doing an EPQ for my A levels and have decided to do this on audio technology, and came across spleeter whilst researching ideas on the net.  The idea is to build a system to separate a string quartet.  I can use the musicians as school as part of the project.  I have installed spleeter (see steps below).
I can separate files, so I know spleeter is installed and running.  When I try to run a training session it "hangs".  No output is displayed after it segments the audio in the training folder.  The file
model.ckpt-0.data-00000-of-00001
does not change and indicate it is reaching the steppoints.  When I run the non GPU version, this increments 300 every 85 minutes (so a model would take about 19 days to learn).  I have left the GPU version running for 12+ hours and no change in these files.
The plan was to get the training working on existing files, and then once confirmed it has worked, start the training on the quartet samples.
This is how I have configured my system.

Installed Visual Studio 2017 community version.
Installed CUDA toolkit 10.0.130
Installed cdDNN version 7.3.1. (any higher than this and spleeter would fail with an error)
Installed Anaconda3
from an anaconda prompt run

conda install -c conda-forge spleeter-gpu

In the navigator, ensured the cudatoolkit and cudnn versions match

I have downloaded the musdb18hq dataset from Inria and extracted to a folder c:\trainingdata\musdb18hq

.csv files stored in c:\trainingdata\configs

open anaconda prompt

changed to the c:\trainingdata folder and typed the following command

spleeter train -p configs\musdb_config.json -d musdb18hq
When ever I made a change to the environment i would shut down and restart.
When this runs, it creates the cache folder and the musdb_model folder and it loads the audio in the chunks defined in the json file (i set to 10 initially as a test).  The config file is the one provided by spleeter just changed where the files were located in the config.
In the .csv files I am training on 10 files with 4 files in the validation .csv
After it stops loading the audio it simply stops scrolling and doesn't do anything more.  Under verbose mode no errors are reported.
Has anyone actually got the training working on a Windows 10 system?  If so, what steps have i missed out?
Environment
| ----------------- | ------------------------------- |
| OS                | Windows 10
| Installation type | Conda             |
| RAM available     | 64Gb                            |
| Hardware spec     | GPU   Nvidia GTX1660TI 6GB  CPU Intel i7 Gen 9 2.6 GHz          |

Have you downloaded the training data? Can you share it to me? I send my requests for access, but no response.


DD 1970 Motor Vehicle Utilization Record aka Dispatch

https://www.esd.whs.mil/Portals/54/Documents/DD/forms/dd/dd1970.pdf

This should be done in GCSS-A now.


CalendarPickerController leaks a SwiftUI binding

Where possible, I've been trying to maintain the boundary between UIKit and SwiftUI. For example, CalendarPicker presents as SwiftUI interface, and CalendarPickerController presents the UIKit interface. Right now, CalendarPickerController leads a SwiftUI Binding. We should probably replace this with some 'on change' closure or similar.

Done.


Setting the format flag within a custom lychee.toml config file does not work

Hello,
I'm just trying to see if I can define the format inside my custom lychee.toml config file, but I found a couple of issues:
With the following config
format = "json"

I get the following error message:
   [ERROR] Error while loading config: Cannot load configuration file `./lychee.toml`: Failed to parse configuration file

Caused by:
    TOML parse error at line 38, column 10
       |
    38 | format = "json"
       |          ^^^^^^
    unknown variant `json`, expected one of `Compact`, `Detailed`, `Json`, `Markdown`, `Raw`

So then, I decided to change from json to Json, but it does not work and the output is shown in the default format compact.

Nice catch! It's fixed in master now. Stay tuned for the next release. 😊


Add types to signatures

Thanks a lot Keshav.
@lawliet19189 Can we play with this with GPT-3.5 and see if it work well for typical tests?

Do you still need this now we have TypedChainOfThought and TypedPredictor?


logic.ipynb incomplete

The notebook for logic.py has many things left to cover. I will start with KB section.

Please see PR #427 . I hope the work doesn't clash.

I'll wait until your PR is merged. Resolving conflicts is a tedious task.

#575 should add most of the required content.


spawn ENOENT

Hello
Thx for this great generator !
When i run grunt server prompt command, i get a fatal error "spawn ENOENT" while running php:server task.
Any idea please ?

Could you describe it a bit more? I don't know how to make sure that php is in my PATH, didn't find anything about it in linked documentation.

Really need more info.
Are you on Windows? Try this: http://php.net/manual/en/faq.installation.php#faq.installation.addtopath
If you are on OSX or Linux, then a lot is going to depend on your particular setup (using built in php on osx or MAMP?,  What flavor of Linux?,  etc etc). Setting up basic system Path variables is really beyond the scope of this project.


Add refactoring description for readNumber in IterImplForStreaming.java

Add description of how to refactor function readNumber in order to reduce complexity.

Wooo! GO Joangooo!

Be proud


[FEATURE] Beautification of frontend

Currently, the product works with a makeshift frontend that serves the basic purpose. However, we would appreciate a better frontend, so if anyone can help us out with that, we will be very grateful.
The contributor can change anything as long as the functionality of the product remains unchanged.
Color schemes, Logos, everything can be changed.
So if someone wants to try out exploring UI/UX, this would be the perfect issue.

I want to work on this issue

Sure! Please do ping me or @hs2361 if you need some help. 😄

In the backend, the settings.py seems to be missing and it is not possible to run the app. Can you please share the necessary settings and configurations to run it?

I have added the required file. You should be able to run it now.

Sir, while making a request, the backend gives the error mentioned below. Do I need to change anything in dependencies, or is it anything else?

What is the version of the google package that you are using? It should be 3.0.0

It worked now

Sir, I have made the following modifications. Are there any changes to be made?

This honestly looks wonderful! Great Work!

Create a pull request linking this issue, I'll be more than happy to merge it.

I have created PR #8 for this issue


Change Package name for the NewCfClient function.

Currently to create a new CfClient we have to import the package 'client'.  And call NewCfClient.   This isn't very go idiomatic.
We probably need to support the API at this stage, but should create a package called harness or flags and define the NewCfClient in it.    That way when instantiating the client the user would call
client := flags.NewCfClient(sdkKey string, options ...ConfigOption)

We can have the current NewCfClient call the new one, to maintain backwards compatability i.e.
package client
func NewCfClient(sdkKey string, options ...ConfigOption) (*CfClient, error) {
    return flags.NewCfClient(sdkKey, options)
}

What do you think about this ff := client.New(sdkKey string, options ...ConfigOption) it doesn require any breaking change just add New function


HDDS-10649. Handle auto hsync commit key having directories.

What changes were proposed in this pull request?
When hard limit is expired for hsync keys, it has to be committed through OpenKeyCleanupService. keyName in openFileTable just contains the leaf, which is being passed in auto commit request. But commit request also requires parentId.
When commit request comes from client, in this case keyName contains full path and from this parentId can be fetched.
In this PR, when commit request is sent through OpenKeyCleanupService we can pass parentObjectId as well which is present in openFileTable.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-10649
How was this patch tested?
New integration test

From what I can tell, it's because keyName is set as the leaf name
https://github.com/apache/ozone/blob/c1e9e9180960777ac99e6483515e8b76e585f5d0/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/request/file/OMFileRequest.java#L618
and it is used by OMKeyCommitWithFSO which sets the keyName of OmKeyInfo entry in keyTable as the leaf.
parentObjectId in KeyArgs is used inside OpenKeyCleanupService. It's not used by client.
It feels like a hack but I can live with it because it does fix the issue.
IMO OMKeyCommitWithFSO should use the full path as the keyName.
But if that is changed, we'd likely open a Pandora's box and need to fix many other places where it is assumed to be the leaf name.

I feel the same way as @jojochuang .  The code cause this problem is OMFileRequest#addOpenFileTableCacheEntry, where it set both the KeyName and FileName in OmKeyInfo. I'm wondering if we can keep the KeyName as full key name, and FileName as leaf name here?
if (omFileInfo != null) {
      // New key format for the openFileTable.
      // For example, the user given key path is '/a/b/c/d/e/file1', then in DB
      // keyName field stores only the leaf node name, which is 'file1'.
      omFileInfo.setKeyName(fileName);
      omFileInfo.setFileName(fileName);
      table.addCacheEntry(dbOpenFileName, omFileInfo, trxnLogIndex);
 } else {
      table.addCacheEntry(dbOpenFileName, trxnLogIndex);
 }

I feel the same way as @jojochuang . The code causes this problem is OMFileRequest#addOpenFileTableCacheEntry, where it set both the KeyName and FileName in OmKeyInfo. I'm wondering if we can keep the KeyName as full key name, and FileName as leaf name here?
if (omFileInfo != null) {
      // New key format for the openFileTable.
      // For example, the user given key path is '/a/b/c/d/e/file1', then in DB
      // keyName field stores only the leaf node name, which is 'file1'.
      omFileInfo.setKeyName(fileName);
      omFileInfo.setFileName(fileName);
      table.addCacheEntry(dbOpenFileName, omFileInfo, trxnLogIndex);
 } else {
      table.addCacheEntry(dbOpenFileName, trxnLogIndex);
 }

Yes I also thought the same but comment specifically just above that mentioned it should store only the leaf. Not sure the background of this why openFileTable requires only leaf in keyName.

Can you try the idea? @ashishkumar50

@ChenSammi I have made the change and the tests are running.
One point is for hsync keys keyArgs are formed based on fileTable and not openFileTable in openKeyCleanupService.
So even though we change openFileTable keyName, it is unused. So to handle this either we need to update fileTable keyName as well but that seems more risky as old key and new key format will differ and may cause other issues.
Or we can use openFileTable just for keyName(Again not efficient solution). That i have currently added in PR.

@ChenSammi I have made the change and the tests are running. One point is for hsync keys keyArgs are formed based on fileTable and not openFileTable in openKeyCleanupService. So even though we change openFileTable keyName, it is unused. So to handle this either we need to update fileTable keyName as well but that seems more risky as old key and new key format will differ and may cause other issues. Or we can use openFileTable just for keyName(Again not efficient solution). That i have currently added in PR.

@ashishkumar50 , Yes, change the fileTable keyName will introduce the incompatibility, an upgrade will be required.  Change keyName in openFileTable is less risky.
The current change overall looks good. The thing is missed. After keyName in openFileTable is changed, we need to change it back to leaf name in OMKeyCommitRequest#validateAndUpdateCache before add to cache and return with OMKeyCommitResponse.  Could you add this part, and verify that the final keyInfo has the same leaf name in keyName and fileName field?

@ChenSammi I have made the change and the tests are running. One point is for hsync keys keyArgs are formed based on fileTable and not openFileTable in openKeyCleanupService. So even though we change openFileTable keyName, it is unused. So to handle this either we need to update fileTable keyName as well but that seems more risky as old key and new key format will differ and may cause other issues. Or we can use openFileTable just for keyName(Again not efficient solution). That i have currently added in PR.

@ashishkumar50 , Yes, change the fileTable keyName will introduce the incompatibility, an upgrade will be required. Change keyName in openFileTable is less risky.
The current change overall looks good. The thing is missed. After keyName in openFileTable is changed, we need to change it back to leaf name in OMKeyCommitRequest#validateAndUpdateCache before add to cache and return with OMKeyCommitResponse. Could you add this part, and verify that the final keyInfo has the same leaf name in keyName and fileName field?

In OMKeyCommitRequest after reading keyName from openKeyTable it overwrites with keyName which comes in client request.
I have added a verification to ensure keyName and fileName are same after auto commit in fileTable.


Shuffle remotes if remote-random is present

This is a first attempt at implementing remote-random. A helper function get_remotes is added that checks for remote-random and shuffles the remotes if required.
This is a draft PR as I need to verify that the remotes are not shuffled more than once. As this code is brittle in this way I may reconsider the PR.

looks fine to me :) thanks.

From studying the code a bit closer I think this will not work:

Miragevpn.handle is called with a new event, such as  `Resolved.
Miragevpn.handle in turn calls Miragevpn.resolve_connect_client with the event. resolve_connect_client calls get_remotes which returns a shuffled remotes list (if remote-random is specified).
When the event is  `Resolved the current remote is looked up by the index stored in the client state to get the port and protocol (tcp/udp) - but the remotes list has been shuffled since that index was passed in the client state and may refer to a different remote now.

The client state may need to be extended with a list of remotes.

Closing this in favor of #69. Please reopen if you disagree.


Add Clear() to be able to remove all items

To me, Clear signals removal of temporary state (as in "clear selection"). I'm thinking RemoveAll() or RemoveItems() makes the intent clearer?

True, I guess I've been using Python too much ;)

LGTM! Thank you contributing!


Suggestions for optimizing and supplementing the functions

A big compliment for the successful update!
As I use it in all my projects, the compatibility with Kirby 4 is an enrichment.
Here are my suggestions for optimizing and supplementing the functions:

To ensure compatibility with the original v3 plug-in, it would be an advantage to deactivate/activate "thead" via toggle. Otherwise, in existing projects, the first row is defined as "thead", which already contains values, instead of row headers. It would involve a lot of extra work to adapt all tables for a large project.

Reason: I use tables in multilingual projects in which the values (mainly numbers) are identical in all languages and editing is only possible in the main language (translate: false). Row headings are defined independently of the table and are considered for other languages. This option can, of course, be omitted for monolingual projects.

The function to move rows and columns by drag-and-drop is a great-added value. However, the function harbors the risk that customers consciously (or unconsciously) change the order and do not consider that rows can already be sorted in the frontend using JavaScript. For example, with this: https://datatables.net/

Hi, thanks for the feedback 😃

I knew since the addition that it would be a problem for those who came from the initial plugin. The headers implementation is not ideal but was the simplest way (after many tries) of getting it working and keeping compatibility with the initial version. I've already planned a headers toggle for the next release.

I'm not so familiar with multilingual setups but I see the need for this. I'll look into adding it in the next release.

The latest v2.1.0 release includes toggles for sorting and the index column as well. Check the field properties.

Please check the new version and give me some thoughts. It also supports K3 now!

The missing header function had forced me to use a temporary solution, but it works accurately. Basically, I am interested in optimizing projects and will adapt my template to your new plugin. After all, you don't want to carry over old ballast into new projects.
The problem with multilingual projects is that changes are required in all languages if a table is changed, even in partial areas. If I add a line in the main language, I have to do the same in all other languages. As my tables only contain numbers, I have deactivated all additional languages of the tables with "translate: false". But the problem would be the header. This is now part of the table. Unfortunately, I don't have a solution for how to make the header editable in all languages, but lock the table fields.

Question: If I set "index: false", nothing changes in the .txt file. This specification is only for the output in the template, right?

It's only for the table component inside the panel. It disables the index column but you'll have to set both index and sortable to false in your blueprint.
By index, it refers to the table row index number. Also the sortable needs to be false as well, otherwise, you'll be left with just the column sorting witch it didn't seem right.

Thanks for the explanation!
In general, I'm already happy with your solution. There will always be projects where individual wishes arise. You can't take them into account. Otherwise, a small plugin becomes an oversized monster (we know that from WordPress...)
For my problem with multilingual pages, I could imagine creating 2 tables:
the first table has only 1 line: the "thead"
the 2nd table contains the "tbody"
This way I can leave "thead" dynamic for different languages and make "tbody" editable only in the main language. I'm sure there is a more elegant solution, but I don't have the in-depth programming knowledge for that.

@gbdesign2023, please check the new released v2.2.0
You can disable translation for non-default languages by adding translate: false to the field blueprint.
Unfortunately, the headers row toggle isn't coming with this release. I've been working on it but it required lots of refactoring and I'm yet to find an implementation way that does not clutter the core. I've decided to leave it for a future release as I've turned my focus on adding new features and future-proofing the codebase to the table component. But it's definitely on my priority list and would come eventually somewhere this month.

@bogdancondorachi Many thanks for the successful update and the new, useful features.
I am pleased with the current range of functions and can use it perfectly in my projects.
Of course, I'm also looking forward to more functions.
Which functions I like to use with tables:

Combine or merge cells (colspan and rowspan)
Text alignment of individual cells

Both can also be implemented in the template output, but require a lot of effort and a strategy.
However, I assume that these functions are not possible in the panel.
By the way 1: I think the focus on Kirby 4 makes sense.
By the way 2: It's great that your plug-in is already listed on the Kirby website.
https://getkirby.com/plugins/k4

Thanks @gbdesign2023, regarding your suggestions:

The colpan/rowspan marge is something I thought about, definitely possible but atm I'm far from that 😶🌫
The text alignment is something closer to the state current state of the plugin and would look into that on the next releases

So far I have a short roadmap for the upcoming versions:

Header row toggle
Clear contents option for column/row
Drawer content edit for column/row
WYSIWYG support
Option to hide/show column/row
Ability to set cell field type on column creation
Horizontal scroll
Rework draggable component to allow actions dropdown on click and sort on drag
Overall UI improvements

Primary focus for now would be to improve the component and content creation within the table. 😃

@gbdesign2023 You can now disable the headers like this in the blueprint and then have your own headers function inside snippets.
table:
    type: table
    headers: false

This would disable the ability to set headers inside the field (if you already have the headers set they would return as the first row).
P.S. The field blueprint has to be lowercase to work in your templates.
Now I know this won't solve your migration problem entirely but I'm afraid I can't do more than this. What I would do in your case is to manually address this at some point. You can benefit from the headers: false in order to remove the first row as headers, than add a new first row with your desired headers and remove the headers: false, this would add your first row (actual headers) as headers.
I would close this for now, if you have more discussions around this issue, please open a discussion thread here.

Thank you very much for your great work. The quality of your plugin is rising steeply!
I have found a small, cosmetic problem: With the setting "headers: false", the first line is still displayed as a header in the inline editor. In the dialog editor, the display is correct.

@gbdesign2023 please open a new issue regarding that if you can. Also if you can demonstrate with a screenshot and more info around it. Thanks

@bogdancondorachi  I have now removed my temporary solution for converting the header and I am using your example in the template. Everything works perfectly now!


Fix client-side routing

It seems like this rule isn't taking effect, so let's try adding it via
the _redirects file instead.

Yep, that seems to work just fine.


Error ConnectionRefused when binding UdpSocket to localhost

This line in the UdpSocket implementation suggests that it can be used with localhost, :: or <IP_ADDRESS>.
But if we try to change the binding address from "unspecified" to "localhost" in the udp tests, they all fail with ConnectionRefused error.
If this is the expected behaviour for the socket, that part of documentation can be seen as somewhat misleading.

Which line in the udp tests are you referring to? The majority of those tests are between 2 hosts, which should error if the socket is bound on localhost.

The majority of those tests are between 2 hosts, which should error if the socket is bound on localhost.

I see now where I misunderstood. I thought hosts shared the same network interface and could communicate over localhost.
I appreciate the clarification. Thanks for helping clear things up and for the quick response!


deprecated (?) systemd_cgroup still printed by "containerd config default"

Description
Apparently systemd_cgroup = true in plugins."io.containerd.grpc.v1.cri" is no longer supported by containerd 1.3.7. Enabling it causes containerd to skip loading the cri plugin:
containerd[17742]: time="2020-09-18T10:46:45.630684350Z" level=warning msg="failed to load plugin io.containerd.grpc.v1.cri" error="invalid plugin config: `systemd_cgroup` only works for runtime io.containerd.runtime.v1.linux"

Steps to reproduce the issue:

containerd config default > /etc/containerd/config.toml
sed -i -e 's/systemd_cgroup = false/systemd_cgroup = true/' /etc/containerd/config.toml
start containerd

Describe the results you received:
It starts, but fails to load the cri plugin:
containerd[17742]: time="2020-09-18T10:46:45.630684350Z" level=warning msg="failed to load plugin io.containerd.grpc.v1.cri" error="invalid plugin config: `systemd_cgroup` only works for runtime io.containerd.runtime.v1.linux"

This in turn makes kubeadm unhappy:
error execution phase preflight: [preflight] Some fatal errors occurred:
[ERROR CRI]: container runtime is not running: output: time="2020-09-18T10:47:07Z" level=fatal msg="getting status of runtime failed: rpc error: code = Unimplemented desc = unknown service runtime.v1alpha2.RuntimeService"

Describe the results you expected:
If systemd_cgroup = true is no longer valid, then systemd_cgroup = false should not be printed in the containerd config default output. Showing the field there implies that it can be edited.
I think I understand where this is coming from (straight dump of the in-memory struct), but it's still not nice.
Output of containerd --version:
containerd containerd.io 1.3.7 8fba4e9a7d01810a393d5d25a3621dc101981175

BTW, it is not clear to me what replaces it.
https://github.com/containerd/containerd/releases/tag/v1.3.0 just says "use runtime options instead" - some pointer to documentation for those wouid be nice.

https://github.com/containerd/containerd/issues/4203 provides a documentation snippet in one of the comments, but I still haven't found some actual documentation about this.

systemd_cgroup is for io.containerd.runtime.v1.linux runtime, which is deprecated but still supported.
To enable systemd cgroup for the current default runtime io.containerd.runc.v2, please see https://github.com/containerd/containerd/issues/4203#issuecomment-651532765

BTW, it is not clear to me what replaces it.
https://github.com/containerd/containerd/releases/tag/v1.3.0 just says "use runtime options instead" - some pointer to documentation for those wouid be nice.

https://github.com/containerd/cri/blob/master/docs/config.md

BTW, it is not clear to me what replaces it.
https://github.com/containerd/containerd/releases/tag/v1.3.0 just says "use runtime options instead" - some pointer to documentation for those wouid be nice.

for runc runtimes under cri the docs are here:
https://github.com/containerd/cri/blob/master/docs/config.md
for kata runtime the docs are here:
https://github.com/kata-containers/documentation/blob/master/how-to/containerd-kata.md#kata-containers-as-a-runtimeclass

Agreed this is confusing. Further, setting this in later version actually gives error:
version = 2

[plugins]

  [plugins."io.containerd.grpc.v1.cri"]
    sandbox_image = 'xxxxx'
    systemd_cgroup = true # <--------
    [plugins."io.containerd.grpc.v1.cri".containerd]
      default_runtime_name = 'nvidia'
.....

# crictl info
FATA[0000] getting status of runtime: rpc error: code = Unimplemented desc = unknown service runtime.v1alpha2.RuntimeService

Setting to false has no effect (which is the default I think).
# crictl info | grep -i cgroup
            "SystemdCgroup": true
            "SystemdCgroup": true
    "systemdCgroup": false,
    "disableCgroup": false,

Our version:
# crictl version
Version:  0.1.0
RuntimeName:  containerd
RuntimeVersion:  1.6.9
RuntimeApiVersion:  v1alpha2

If we remove the setting "systemd_cgroup = false" from config file will it still be defaulted to false?


All Godot 3.x exports crash when running on iPad Mini 4 with iPadOS 15.7.1

Godot version
3.x
System information
iPad Mini 4 Wi-Fi with 128 GB of storage, iPadOS 15.7.1
Issue description
Might related to #67949
All games made with Godot 3.x crashes when running on iPad Mini 4th generation with iOS 15.7.1 (running fine with earlier iPadOS 14.x) I cannot inspect any of logs/consoles since I don't have Macintosh to debug and submit error information.
Steps to reproduce

Open up Safari or any of browsers.
Open up websites with Godot games.
Games seem to load up just fine, and immediately crash and web pages repeatedly refresh.

Minimal reproduction project
No response

Duplicate of #67949


[FEATURE] Making 'global shared vision' turn on when party are close together, and turn off if they split up too much

The idea being - if the party are playing as a group (say, within talking distance) then their vision should be shared, as they are passing information around and they can all hear when a creature pops up nearby even if they can't see it etc. Means that noone misses any of the action if they're near each other.
But then if they split up too much, they loose that benefit, and return to standard 'unshared' vision.

I like this feature, I'll see what I can do.

That would be awesome Thankyou. I'm aware it's probably possible using a
world macro but that would be so far beyond me that I can't event begin to
attempt it haha
On Wed, 25 Jan 2023, 16:09 Material Foundry, @.***>
wrote:

I like this feature, I'll see what I can do.
—
Reply to this email directly, view it on GitHub
https://github.com/CDeenen/SharedVision/issues/30#issuecomment-1403857730,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4T22O2DFSCX74WVB64CBTWUFF4NANCNFSM6AAAAAAUFZ6PTM
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'm thinking of approaching it slightly different from how you might imagine (based on what you've said, but I might've misinterpreted you). It'd be independent of the 'global shared vision' setting.
There'd be a configurable radius in the module settings, let's say 30ft. Any tokens within 30ft of your selected or owned tokens will share their vision with you. This means that if your token is on the outskirts of the party, only part of the party might share their vision. This is a bit more flexible, since you might have a split party, each only seeing what the people in their direct surroundings can seen.
Let me know if you agree, otherwise I could consider using 'global shared vision'.

No, what you have described is how I would have ideally wanted it to work!
Sounds great!
On Wed, 25 Jan 2023, 16:57 Material Foundry, @.***>
wrote:

I'm thinking of approaching it slightly different from how you might
imagine (based on what you've said, but I might've misinterpreted you).
It'd be independent of the 'global shared vision' setting.
There'd be a configurable radius in the module settings, let's say 30ft.
Any tokens within 30ft of your selected or owned tokens will share their
vision with you. This means that if your token is on the outskirts of the
party, only part of the party might share their vision. This is a bit more
flexible, since you might have a split party, each only seeing what the
people in their direct surroundings can seen.
Let me know if you agree, otherwise I could consider using 'global shared
vision'.
—
Reply to this email directly, view it on GitHub
https://github.com/CDeenen/SharedVision/issues/30#issuecomment-1403929005,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4T22IMXAFZFBWS2MRYYALWUFLPBANCNFSM6AAAAAAUFZ6PTM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Just following up, curious if you had any success with this?

I'm thinking of approaching it slightly different from how you might imagine (based on what you've said, but I might've misinterpreted you). It'd be independent of the 'global shared vision' setting. There'd be a configurable radius in the module settings, let's say 30ft. Any tokens within 30ft of your selected or owned tokens will share their vision with you. This means that if your token is on the outskirts of the party, only part of the party might share their vision. This is a bit more flexible, since you might have a split party, each only seeing what the people in their direct surroundings can seen. Let me know if you agree, otherwise I could consider using 'global shared vision'.

Can this range be blocked by walls. If would be weird if the group is separated (by walls) but still withing this range making them share their vision. Perhaps also make the sharing disabled when the blinded condition is involved or other vision imparing effects


Why maven swagger plugin isn't scanning external modules?

We have a maven multimodule project which looks approximatively like that:

Parent
    - admin
    - microserviceA
    - microserviceB
    - microserviceC

The microservices A, B and C expose functional specific rest endpoints.
However, the admin module exposes common rest endpoints for resilience, robustness and recovery.
In order to document our APIs, we are using swagger 2.0 with the maven swagger plugin.
The pom.xml that we are using for each microservice looks like:
MicroserviceA:
<build>
			<plugin>
				<groupId>com.github.kongchen</groupId>
				<artifactId>swagger-maven-plugin</artifactId>
				<version>${swagger.maven.plugin}</version>
				<configuration>
					<apiSources>
						<apiSource>
							<springmvc>false</springmvc>
							<locations>
								    <location>com.xxx.a.resource</location>
									<location>com.xxx.admin.resource</location>
							</locations>
							<schemes>http,https</schemes>
.....
</build>

MicroserviceB:
<build>
			<plugin>
				<groupId>com.github.kongchen</groupId>
				<artifactId>swagger-maven-plugin</artifactId>
				<version>${swagger.maven.plugin}</version>
				<configuration>
					<apiSources>
						<apiSource>
							<springmvc>false</springmvc>
							<locations>
								    <location>com.xxx.b.resource</location>
									<location>com.xxx.admin.resource</location>
							</locations>
							<schemes>http,https</schemes>
.....
</build>

MicroserviceC:
<build>
    			<plugin>
    				<groupId>com.github.kongchen</groupId>
    				<artifactId>swagger-maven-plugin</artifactId>
    				<version>${swagger.maven.plugin}</version>
    				<configuration>
    					<apiSources>
    						<apiSource>
    							<springmvc>false</springmvc>
    							<locations>
    								    <location>com.xxx.c.resource</location>
    									<location>com.xxx.admin.resource</location>
    							</locations>
    							<schemes>http,https</schemes>
    .....
    </build>

Problem : The maven plugin isn't scanning the rest endpoints at the admin module level even by specifying explicitly the admin package in the maven swagger plugin <location>.
PS: I've tried several version of maven plugin.
Is there any solution?

Facing the same issue.

@Ankzz, it Looks like it's  knows limitation of the Swagger plugin.

is there any solution for this issue?

@dshishkaryan,  Unfortunately No.


Build fix for mac

Mac have issues building the new runtime due to the inclusion of a new file, google/protobuf/port_def.inc. This issue appears to be specific to mac.
Fix below add the new path in the cmake file.

I add the same build issue on Mac today. I can confirm that the proposed fix solves the build issue on my Mac environment as well.

Jenkins Linux ppc64le Build #2116 [push] Build fix for mac (#1046... started at 20:23

Jenkins Linux amd64 Build #3007 [push] Build fix for mac (#1046... started at 19:19

Jenkins Linux s390x Build #2998 [push] Build fix for mac (#1046... started at 20:19

Jenkins Linux s390x Build #2998 [push] Build fix for mac (#1046... passed after 35 min

Jenkins Linux ppc64le Build #2116 [push] Build fix for mac (#1046... passed after 49 min

Jenkins Linux amd64 Build #3007 [push] Build fix for mac (#1046... passed after 52 min


Fix saga

This branch was largely an update of saga scripts (landscape and model covars) to streamline and remove the unwanted sysdata.rda and external data files.
I added two internal check function to connect to saga (find_saga_path() and check_saga().  These are currently internal, however they might need to be fully documented as the reference within the create_covariates link is showing up as a Warning?
<body>
<!--StartFragment-->
❯ checking Rd cross-references ... WARNING   Missing link or links in documentation object 'create_covariates.Rd':     'find_saga_path'      Missing link or links in documentation object 'create_samplr_covariates.Rd':     'find_saga_path'      See section 'Cross-references' in the 'Writing R Extensions' manual.  0 errors ✔ \| 1 warning ✖ \| 0 notes ✔
--

> | >
>

<!--EndFragment-->
</body>
</html>

This pull request will close #13 .
I added basic test functionality, but interested on your assessment to add a snapshot to test running the entire suite of outputs.
I also suspect the internal point to a local .saga exe is causing the tests to fail, so commenting out for now. Is there a nice way to deal with this?
ie avoid: sp <- "C:/Programs/saga-9.2.0_x64/saga-9.2.0_x64/saga_cmd.exe" within the testing.
Happy to get your views on overall functionality and any improvements .

@ateucher - I think this is ready for review, before I get too far down the track. It is throwing an apple/mac error still but everything else is passing. Happy to chat first if that is easier.
Thanks!

Thanks @gcperk! I've started figuring out how to install on a Mac - notes here: https://github.com/ninoxconsulting/PEMprepr/issues/10#issuecomment-2455955103. I think I'm almost there and can then test this :)
Also in that comment is a link to install it on Linux, so we could do that in GitHub actions and test conditionally there...

thanks @ateucher. I'm not totally clear how much of the setup is required to get the tests to successfully run? Do you need anything from me at the moment?

Great thanks @ateucher, we completely agree they are long!! I will wait til you are finished with the entire review before I make any changes. A major challenge I have is with the testing as it relies on the saga_path so could see that being an issue. Any ideas there? Thanks for the review as always.

Thanks @ateucher for the review. I updated the create_covariates to update default dem to sinksfilled and updates the csv alongside. I incorporated the minor fixes you found (thanks!). I will now merge back to main so that leaves the saga backend fix which you have on a separate branch.

That's perfect, thanks @gcperk - looks good!


how to set sitemap-images.xml

Details
nuxt: 3.6.5
nuxt-simple-sitemap: 3.2.0
I need to make a sitemap-images, similar to
https://developers.google.cn/search/docs/crawling-indexing/sitemaps/image-sitemaps?authuser=0&skip_cache=true
, but I didn’t find the setting guide.
I have configured a normal sitemap and localhost:3000/sitemap.xml works fine.
I tried using the configuration in the guide, but it didn't work.
// nuxt.config.ts
export default defineNuxtConfig({
sitemap: {
sitemaps: {
posts: {
sources: [
'/api/sitemap-urls'
]
},
},
},
})
// /api/siemap-urls.ts
export default defineSitemapEventHandler(() => {
return [
{
loc: '/about-us',
// will end up in the pages sitemap
_sitemap: 'pages',
}
]
})

Hi, you just add the images to the entry. For example:
{
            loc: '/about',
            lastmod: '2023-02-21T08:50:52.000Z',
            images: [
              {
                loc: 'https://example.com/image.jpg',
              },
            ],
}

I'll keep this issue open as a reminder to add docs for it.

Hi,
Is it possible to make it auto import all the images from the page ?
Notably with nuxt-content (for blog posts) ?
Thank you for the module

Hi, you just add the images to the entry. For example:
{
            loc: '/about',
            lastmod: '2023-02-21T08:50:52.000Z',
            images: [
              {
                loc: 'https://example.com/image.jpg',
              },
            ],
}

I'll keep this issue open as a reminder to add docs for it.

I'm sorry, Still doesn't work, I added it to sitemap-url.ts

Can you provide a reproduction? You can start with https://stackblitz.com/edit/nuxt-starter-dyraxc?file=server%2Fapi%2F_sitemap-urls.ts

posts: {
        sources: [
          '/api/sitemap-urls'
        ]
      },

https://stackblitz.com/edit/nuxt-starter-zfejln?file=nuxt.config.ts

Can you provide a reproduction? You can start with https://stackblitz.com/edit/nuxt-starter-dyraxc?file=server%2Fapi%2F_sitemap-urls.ts

https://stackblitz.com/edit/nuxt-starter-zfejln?file=nuxt.config.ts
I don't know if nuxt-simple-sitemap3 is not supported.

Looks correct to me? Am I missing something?
``xml

http://localhost:3000/aaa
2023-02-21T08:50:52

https://example.com/image.jpg

```

Looks correct to me? Am I missing something?
<?xml version="1.0" encoding="UTF-8"?><?xml-stylesheet type="text/xsl" href="/__sitemap__/style.xsl"?>
<urlset xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:video="http://www.google.com/schemas/sitemap-video/1.1" xmlns:xhtml="http://www.w3.org/1999/xhtml" xmlns:image="http://www.google.com/schemas/sitemap-image/1.1" xmlns:news="http://www.google.com/schemas/sitemap-news/0.9" xsi:schemaLocation="http://www.sitemaps.org/schemas/sitemap/0.9 http://www.sitemaps.org/schemas/sitemap/0.9/sitemap.xsd http://www.google.com/schemas/sitemap-image/1.1 http://www.google.com/schemas/sitemap-image/1.1/sitemap-image.xsd" xmlns="http://www.sitemaps.org/schemas/sitemap/0.9">
    <url>
        <loc>http://localhost:3000/aaa</loc>
        <lastmod>2023-02-21T08:50:52</lastmod>
        <image:image>
            <image:loc>https://example.com/image.jpg</image:loc>
        </image:image>
    </url>
</urlset>
<!-- XML Sitemap generated by Nuxt Simple Sitemap v4.2.1 -->

Yes that's correct, I got it I should ctrl+u to view instead of accessing sitemap-images.xml directly.
Thanks a lot for solving my problem!! Now I need to solve the plugin dependency issue.

I've added a hint to the side panel to view the page source to see the raw XML, I've also added a docs page for images https://nuxtseo.com/sitemap/guides/images-videos


Create CODEOWNERS

@mkitez  I can use your help regarding this PR. I haven't merged this because it is not a good solution and I feel stuck on how to proceed.
Github codeowners conditions are not aggregated and only the last condition met is applied. That means if someone edits a sample.json and a README.md file, then whatever condition I list at the end of this file will be executed. This doesn't help.
I see this article that solves a similar problem : https://bionic.fullstory.com/taming-github-codeowners-with-bots/
Do you think we need to do something like this?
In short term, we could just go with 3-4 blanket reviewers for all files for now but we need a better solution.

@anjgola sounds good. Let's go with what you suggested for now, we can look into the approach described in the article moving forward.


Add support for sK1 2.0 palettes (SKP)

Describe the solution you'd like
Add support for SKP pallettes.

sK1 2.0 - https://github.com/sk1project/sk1-wx
Nightly builds - http://downloads.sk1project.net/sk1/nightbuilds/

Describe alternatives you've considered
Add we-love-colors plugin for sK1 2.0; this is mostly task for @sk1project
Additional context
Here is free (Public Domain) palettes collection in SKP and other formats:

Description - https://sk1project.net/palettes.php
Downloads - http://downloads.sk1project.net/palettes/v1.0/

I will look into it and let you know. Until then, pull requests are always appreciated :fox_face:

Well, it seems that sk1 is already able to import color palettes being generated by this project, eg gpl (GIMP). I guess mentioning this would be enough?


I can not log in. (Error Code 415)

SDK Version: Lateast Version
Unity Version: 5.6.5.p4
Device name and OS version: iPad 10.5 /  iOS 11.4

I can not sign in on Twitter since June 13th.
Twitter policy seems to have changed.
How do I sign in with the Unity plugin?
Will I be solved simply by entering the Callback URL on the Twitter development site? I tried various methods but it does not work.
I entered the Callback URL provided by Firebase. Do you recognize only certain Callback URLs?
I was servicing the app, and this is a fatal accident. :(
Please help me.

callback uri
ios : twitterkit-API_KEY://                     [ ex : twitterkit-DKFJkfjwoDKDKjfdjklwofw:// ]
android : twittersdk://


Return an error when solver input contains duplicate identifiers.

This one could be a nightmare to debug!

/retest

/retest


Configure replicator worker count

configure service binding replicator worker count using BindingWorkerCount
configure service instance replicator worker count using InstanceWorkerCount

Pull Request Test Coverage Report for Build 5172

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 89.149%

Totals

Change from base Build 5162:
0.0%

Covered Lines:
9412

Relevant Lines:
10188

💛  - Coveralls

Validation on k8s cluster: n-1 is succeeded.

Validation on k8s cluster: n is succeeded.

Validation on k8s cluster: n-2 is succeeded.

Validation on k8s cluster: n is succeeded.


Updated comments

comments were not reflecting the change from 16 to 15 inputs
removed test case for 16 inputs

I think you pushed commits from #9 also here, so PRs #9 and #10 are overlapping. Since your README changes are only related to the width limitation, I'd be fine with having just one PR. Maybe let's just apply the README change in #9 (since it has an accurate description)

applied this pr in https://github.com/Lightprotocol/light-poseidon/pull/9


Error in Majestic/Alarm tab

In the latest version of the interface there is a problem with displaying settings in the Alarm tab of Majestic page.

This might be an issue if you switch between stable/dev branch, try to update with the option Install even if the same version or clear the overlay.

This might be an issue if you switch between stable/dev branch, try to update with the option Install even if the same version or clear the overlay.

No, I used bundled version.

In the latest version of the interface there is a problem with displaying settings in the Alarm tab of Majestic page.

Please add a couple of screenshots and show what exactly your problem is, and also tell us the type of device.

gk7205v210 (gk7205v200 family), sc223a, 8 MB nor flash, <IP_ADDRESS>-lite
Here's link to screenshot https://ibb.co/552QHKX

It was already mentioned, try to reset the overlay or remove the overlay files:
rm -rf /overlay/root/var

this can be a problem of cached content in browser. try to use ctrl-r or ctlr-f5 to refresh the page bypassing the cache. please let me know if it fixed the issue.

I cleared the browser cache, checked this problem in new browser. All the same!

Please open source view of the broken page and make a screenshot of the last lines of code to determine where it breaks. Or save the page as html file and send to<EMAIL_ADDRESS>I'll see how to fix it. Thanks.

Here's link to source screenshot https://ibb.co/hfnC1sL

Thanx


Bugfix remove data

related to #197

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

10 of 11 (90.91%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 87.556%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

app/utils/background.py
3
4
75.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
1949

Relevant Lines:
2226

💛  - Coveralls


❓ Detect the QR code frame and main square frame

Question
Please, could someone share the example, how to create a CodeScanner with a frame square and blurred background on both iOS and Android? Inside the frame should detect a code with a green frame, as shown in the picture. I know I need to use frameProcessor and corners/bounding values. How to get them?

Screenshot

What I tried
This is my results, but QR code I can scan in any camera position and Frame is with a black background:

Screenshot

packages:
"react-native-vision-camera": "^3.7.1",
"react-native-worklets-core": "^0.2.4"

I tried create a first frame with SVG:
import { Svg, Defs, Mask, Rect } from 'react-native-svg';
const CameraFrame = () => (
  <Svg height='100%' width='100%'>
    <Defs>
      <Mask height='100%' id='mask' width='100%' x='0' y='0'>
        <Rect fill='#fff' height='100%' width='100%' />
        <Rect fill='black' height='250' width='250' x='18%' y='30%' />
      </Mask>
    </Defs>
    <Rect
      fill='rgba(0, 0, 0, 0.8)'
      height='100%'
      mask='url(#mask)'
      width='100%'
    />
    <Rect
      height='250'
      stroke='#fff'
      strokeWidth='5'
      width='250'
      x='18%'
      y='30%'
    />
  </Svg>
);

Then use in the Scanner Component:
import * as React from 'react';
import { FC, useCallback, useEffect, useRef, useState } from 'react';

import { useIsFocused } from '@react-navigation/native';
import { Alert, AlertButton, Linking, StyleSheet, View } from 'react-native';
import Reanimated from 'react-native-reanimated';
import { Svg, Defs, Mask, Rect } from 'react-native-svg';
import {
  Code,
  useCameraDevice,
  useCodeScanner,
  Camera,
  CameraPermissionStatus,
  useFrameProcessor,
} from 'react-native-vision-camera';

import { useIsForeground } from './hooks/useIsForeground';

const ReanimatedCamera = Reanimated.createAnimatedComponent(Camera);
Reanimated.addWhitelistedNativeProps({
  zoom: true,
});

const VisionCodeScanner: FC = (props): React.ReactElement => {
  const camera = useRef<Camera>(null);
  const [cameraPermissionStatus, setCameraPermissionStatus] =
    useState<CameraPermissionStatus>('not-determined');
  // 1. Use a simple default back camera
  const device = useCameraDevice('back');

  // 2. Only activate Camera when the app is focused and this screen is currently opened
  const isFocused = useIsFocused();
  const isForeground = useIsForeground();
  const isActive = isFocused && isForeground;

  // 3. (Optional) enable a torch setting
  const [torch, setTorch] = useState(false);

  // 4. On code scanned, we show an aler to the user
  const isShowingAlert = useRef(false);

  const onCodeScanned = useCallback((codes: Code[]) => {
    console.log(`Scanned ${codes.length} codes:`, JSON.stringify(codes));
  }, []);

  // 5. Initialize the Code Scanner to scan QR codes and Barcodes
  const codeScanner = useCodeScanner({
    codeTypes: ['qr', 'ean-13'],
    onCodeScanned,
  });

  const requestCameraPermission = useCallback(async () => {
    const permissionStatus = await Camera.getCameraPermissionStatus();

    if (permissionStatus !== 'granted') {
      await Camera.requestCameraPermission();
    }

    setCameraPermissionStatus(permissionStatus);
  }, []);

  useEffect(() => {
    if (cameraPermissionStatus !== 'granted') {
      (async () => {
        await requestCameraPermission();
      })();
    }
  }, [cameraPermissionStatus, requestCameraPermission]);

  const frameProcessor = useFrameProcessor((frame) => {
    'worklet';
    console.log('frame', frame);
  }, []);

  return (
    <View style={styles.container}>
      {device != null && (
        <>
          <ReanimatedCamera
            ref={camera}
            {...props}
            audio={false}
            codeScanner={codeScanner}
            device={device}
            enableZoomGesture
            frameProcessor={frameProcessor}
            isActive={isActive}
            style={StyleSheet.absoluteFill}
            torch={torch ? 'on' : 'off'}
          />

          <View
            style={{
              position: 'absolute',
              top: 0,
              left: 0,
              bottom: 0,
              right: 0,
            }}
          >
            <CameraFrame />
          </View>
        </>
      )}
    </View>
  );
};

export default VisionCodeScanner;

const styles = StyleSheet.create({
  container: {
    flexGrow: 1,
    borderRadius: 20,
    overflow: 'hidden',
    backgroundColor: 'gray',
  },
});

VisionCamera Version
3.7.1
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar questions in the issues page as well as in the discussions page and found none.

Hey! I don't think this is a question related to VisionCamera.
Probably better off to ask this in the community discord.
What you're searching for is a good way to style your Camera so that there is a box around the QR code.


build(docker): Bump dependencies

Reduces exposure to CVEs.
Testing
$ make build
docker run --rm -it seldonio/mlserver:1.5.0.dev1 pip list | grep cryptography

Looks like the cryptography package isn't there?
It does have other packages, as expected:
$ docker run --rm -it seldonio/mlserver:1.5.0.dev1 pip list | grep huggingface
huggingface-hub                          0.21.3
mlserver-huggingface                     1.5.0.dev1

$ docker run --rm -it seldonio/mlserver:1.5.0.dev1 pip list | grep catboost
catboost                                 1.2.3
mlserver-catboost                        1.5.0.dev1

@sakoush: I'll run it locally and if needed, make another PR!


[DO NOT MERGE] Auto PR to update dependencies of pilot

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/pilot-presubmit.sh
1d96ef46506c0bb6278d084b2bf936b9a17150fd
link
/test pilot-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Feature Request: Spectral Normalization

A relevant pull request is #4701.
Now that Chainer supports LinkHook which seems very useful to implement Spectral Normalization (SN) that normalizes links' weights before their forward computations.
If OK, I want to work on implementing this after discussing the design.
As a side note,  my casual SN implementation is this→https://github.com/crcrpar/chainer/blob/eb50f9288b82a20028ecd4aaf4bd39e7b5379f82/chainer/link_hooks/spectral_normalization.py#L80
Miytao-san’s original implementations are https://github.com/pfnet-research/sngan_projection/tree/master/source/links and pytorch’s is https://github.com/pytorch/pytorch/blob/master/torch/nn/utils/spectral_norm.py#L9-L142.
Any comment is welcome.

#5742 added SN linkhook.


Feature request: Add support to pydevd

On the old repo was an issue regarding a slow behavior of ptvsd, especially with remote debugging. I think was a pull request pending, by the way.
Can we expect support for other debuggers like pydevd? (Standard debugger on PyCharm, very fast)
Thank you, and congratulations on joining MS VSCode Team.

Closing in favor of https://github.com/Microsoft/vscode-python/issues/106#issuecomment-344087338


MM-41323: Secure Marketplace URL

Disable marketplace URL text box when
plugin uploads are disabled.
https://mattermost.atlassian.net/browse/MM-41323

/release-note-none

/update-branch


Issue with dependencies via docker for Python 3.10/3.11

Hi,
I was able to install LanceDB fine till a couple days ago, but now it's giving me this error when I try to run LanceDB via  Docker.  Where do you think the issue originates?
This is the requirements.txt file:
lancedb>=0.1.0
duckdb>=0.7.1

And this is the Dockerfile:
FROM python:3.10-slim-bullseye

WORKDIR /wine

COPY ./requirements.txt /wine/requirements.txt

RUN pip install --no-cache-dir -U pip wheel setuptools
RUN pip install --no-cache-dir -r /wine/requirements.txt

The error  when building the Dockerfile is as follows:
#0 12.79 INFO: pip is looking at multiple versions of lancedb to determine which version is compatible with other requirements. This could take a while.
#0 12.79 Collecting lancedb>=0.1.0 (from -r /wine/requirements.txt (line 1))
#0 12.82   Downloading lancedb-0.1-py3-none-any.whl (10 kB)
#0 12.82 ERROR: Cannot install -r /wine/requirements.txt (line 1) because these package versions have conflicting dependencies.
#0 12.82
#0 12.82 The conflict is caused by:
#0 12.82     lancedb 0.1.1 depends on pylance>=0.4.4
#0 12.82     lancedb 0.1 depends on pylance>=0.4.3
#0 12.82
#0 12.82 To fix this you could try to:
#0 12.82 1. loosen the range of package versions you've specified
#0 12.82 2. remove package versions to allow pip attempt to solve the dependency conflict
#0 12.82
#0 12.82 ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts#0 12.79 INFO: pip is looking at multiple versions of lancedb to determine which version is compatible with other requirements. This could take a while.
#0 12.79 Collecting lancedb>=0.1.0 (from -r /wine/requirements.txt (line 1))
#0 12.82   Downloading lancedb-0.1-py3-none-any.whl (10 kB)
#0 12.82 ERROR: Cannot install -r /wine/requirements.txt (line 1) because these package versions have conflicting dependencies.
#0 12.82
#0 12.82 The conflict is caused by:
#0 12.82     lancedb 0.1.1 depends on pylance>=0.4.4
#0 12.82     lancedb 0.1 depends on pylance>=0.4.3
#0 12.82
#0 12.82 To fix this you could try to:
#0 12.82 1. loosen the range of package versions you've specified
#0 12.82 2. remove package versions to allow pip attempt to solve the dependency conflict
#0 12.82
#0 12.82 ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts

pip version below:
 ❯❯❯ pip --version
pip 23.1.2 from /~/python3.11/site-packages/pip (python 3.11)

hey @prrao87 thanks for the issue.
I was able to build successfully using the requirements.txt and Dockerfile you provided. I'm wondering if there is a previous version of the image already built with the older lancedb version? Would you be able to try doing a fresh docker build?
Thanks!

Ahh that was it! I think I had the old version. It works, thanks!


AnonCreds Workshop Questions about Traction

This issue is to collect questions and comments about Traction during the AnonCreds Workshop on May 31, 2023.
Add a comment to this issue if you are having problems in carrying out the preparations for the Workshop.

When I am connecting my Traction Tenant from my BC Wallet app on mobile, after scaning the resulting QR code with my initialized mobile wallet. I keep getting stuck with this.

I tired re-installing the app. And restarting my traction tenant. Still I am facing this issue.

@swcurran This is exactly what happened, the status now shows as 'active'. Thank for helping :)

I retried with a multi-use connection and it worked fine

Are you guys using the latest mediator update? Like using askar instead of indy? This seems like an issue regarding the mediator.

Are you guys using the latest mediator update? Like using askar instead of indy? This seems like an issue regarding the mediator.

The mediator we are using is using the latest version of ACA-Py and askar for secure storage.


bug: Lazy setup in plugin/ makes the runtime plugin directory load twice

Did you check docs and existing issues?

[X] I have read all the lazy.nvim docs
[X] I have searched the existing issues of lazy.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.9.5
Operating system/version
MacOS 14.2
Describe the bug
Trying to move lazy's setup call to the plugin directory rather than directly in init.lua, I got the message Re-sourcing your config is not supported with lazy.nvim. This did not happen when that code was in the init.lua directly. I tried putting print messages in other files within that directory, and they were also loaded twice. Investigating further, the double loading only happens when lazy.setup() happens in that runtime directory, not when in the init.lua file. This also happens when lazy is the only plugin in the spec being passed to the setup, so it's not plugin interference.
Steps To Reproduce

Invoke lazy.setup() in a file in the runtime directory
Put a print() in another file in the runtime directory plugin
Start nvim
See the message being printed twice

Expected Behavior
The runtime directory is only loaded once, no matter where lazy is being setup, it's a bit weird otherwise.
Repro
No response

Can you show your file structure?

For example, if you have the lazy setup code in a file /lua/plugins.lua and then you also have a directory /lua/plugins/ where you have the plugins' specs. In that case, if you have in the setup call require("lazy").setup("plugins"), then you're essentially telling lazy.nvim that the plugins' specs will reside in the module plugins. However /lua/plugins.lua is also a module called plugins and will take precedence compared to the directory. So, lazy.nvim tries to essentially reload its code in the module it resides but it has already been loaded.
That's what the error means and lazy.nvim doesn't support this. The only way would be to change your file structure (for example rename the plugins.lua file to something else so it doesn't conflict with the directory).

init.lua
|- lua/emilia/plugins/ (where the spec lives)
|- plugin/
  |- lazy.lua (where the setup call is for lazy)
  |- opts.lua

The problem is that when the require('lazy').setup('emilia.plugins') lives in the plugin directory, that plugin directory gets interpreted twice (Not just lazy.lua, but everything else in there). It doesn't happen when that same call is in the init.lua.

Yes, you can't put it there. See here step no3. First the lazy.nvim will load and then all the other steps. So, in step 3, if you put the lazy.nvim setup call inside plugin/ directory, the file will be sourced. But lazy.nvim was already loaded, so by putting the setup call in there, it will be resourced. That's why you're getting the error.

Ah, that's the part I missed, lazy takes over the entire initialisation, meaning it can't be setup in any of the usual spots.
Well, that's unfortunate, but me being disappointed is not a bug, closing.

(Also no, lazy was not loaded at the start, the real sequence is:

neovim starts doing its init sequence
it reads plugin/
finds the lazy setup
lazy takes over the init sequence
it also reads plugin/
it finds itself again, hence the double sourcing of that entire directory

You're right. I was under opposite assumption. But I put 2 prints, one right after the lazy call setup and one in a vim file in the /plugin/ directory and the /plugin directory printed first. Thanks for pointing this out :smile: