Trie as facade for map::Trie

I apologize; this one's a doozy. I've been working happily with trie-rs::map::Trie but I uncovered some bugs depending on what order entries were inserted, they would overwrite the values of preceding entries. This bothered me enough that I decided to try swapping from crate::Trie being the "root" data structure to crate::map::Trie being the root. Each struct remains in their expected place.
Iterators
I got a PR into louds-rs that adds iterators that I wanted to use in trie-rs. I've added variants of the search methods ending in _ref() since they return references instead of clones.
Decloned
I removed the dependency on Clone for Label and Value. Some methods do require Clone but it's now on a method-by-method basis and it's not essential to the library's use.
Mutable Values
Values in map::Trie can be mutated with exact_match_mut().
Breaking changes
The API has been kept such that it should be source compatible with any crate::Trie uses. However, there are breaking changes for crate::map::Trie and the builders now consume self on build().
Incremental Search
I added an incremental search so that a user can service interactive applications as performantly as possible. It's accessed with trie.inc_search(). There are tests and the docs show an example of its usage.
Docs
I updated the readme with an example of mutating values and incremental search.
Performance
I ran the benchmarks with --release and backported a few benchmarks to master to confirm there weren't any regressions. Much of it remains the same, predictive_search() appears to be two times faster. And the *_big_output() tests run with a query that has a huge number of matches, 4220 to be exact but only consumes 100 of them, which is a very contrived way to show off lazy iterator performance.
 I forgot to rebase onto master before submitting PR. Rebased and force pushed to map-as-root branch.

I'm withdrawing this one. I've got to workshop it some more.


The error details are unclear when creating a disk snapshot with whitespaces

Storage Explorer Version: 1.19.0-dev
Build Number: 20210325.3
Branch: main
Platform/OS: Windows 10/ Linux Ubuntu 18.04/ MacOS Catalina
Architecture: ia32/x64
Regression From: Not a regression
Steps to Reproduce

Expand the 'Disks' node -> Open one resource group.
Upload one disk to it -> Click 'Create Snapshot'.
Delete the default snapshot name then type whitespaces as the snapshot name -> Click 'Create'.
Check whether the error details are clear.

Expected Experience
The error details are clear.
Actual Experience
The error details are unclear.

Additional Context

Show the below behavior when trying to type whitespaces as the disk/ disk snapshot name.

This issue also reproduces when uploading one disk then specify whitespaces as the disk name.

Fix merged into main.

@MRayermannMSFT
I added this to the regular test case and marked it as P2.


Add bosh-dns healthy script to match the route_registrar health_check

This change will provide the UAA specific bosh dns healthcheck.

As an aside I would like to propose that the bin/health_check be removed and both bosh_dns and route_registrar use bin/dns/healthy.  So then we are using the SAME script.  but I don't see in the UAA release how that would be enforced in the bosh deployments.  Any suggestions?  Could we work to deprecate the bin/health_check in favour of bin/dns/healthy?

LGTM.
I agree with you, the duplication between bin/dns/healthy and bin/health_check is meh. There are options:

extract a health_check function somewhere and then source that file in both bin/dns/healthy and bin/health_check
make a PR changing updating cf-deployment here https://github.com/cloudfoundry/cf-deployment/blob/master/cf-deployment.yml#L694

The first option is maybe a bit much. The second option requires this PR to merged and a release to be cut first.
Either way, I'm comfortable with the duplication in the short term. I'll merge by EOD unless there are concerns.

Thanks yeah I think this is right.  lets leave them duplicated.  then we can

PR cf-deployment to change
then we remove the old script here.

Will just take a couple of steps and done over a few releases :)
Thanks!

Released as part of UAA v74.11.0.  Thank you for the PR!
https://github.com/cloudfoundry/uaa/releases/tag/v74.11.0


-out switch writes plan file to temp dir

When using the -out switch, eg -out=my_plan, the my_plan file is written to the temp working directory rather than the current dir when the terragrunt command is run.
Of course, a workaround is to use an absolute path, but I think terragrunt should either munge the out file path when passing it to terraform, or copy it from the temp dir to the current working dir.

I think it is a security risk to leave the terraform plan file lying around on the tmp folder.
My thought is to have a parameter to remove the plan file after apply. For instance:
terragrunt apply -rm-plan=$plan_file
Where $plan_file is relative to the working directory.


feat: import LeanSAT's tactic frontends

This is waiting for:

[ ] #4325 to be merged so that we have a builtin solver to use
[x] @bollu to suggest me theorems from their LLVM efforts as integration tests with end users

Apart from that this PR is ready for review. Note that it will currently fail CI due to the missing solver and contains a update-stage0 because it adds builtins.

I picked these files via https://gist.github.com/bollu/e7bfcabd929745efbfce06a2cab547f8, which looks over all files, and searches for occurrences of bitvector operations. We minimize the total number of lines across all files, while still providing a covering of operations of interest by building an ILP problem. Note that we use string matching to look for occurrences of operations so this is best-effort.
We miss some operations, but we hope to provide one or two more files later that fill this gap:
op not found: %
op not found: sle
op not found: extractLsb
op not found: zeroExtend
op not found: srem
op not found: smod
op not found: rotateRight
op not found: rotateLeft
op not found: append
Minimal covering files:
 g2008h07h11hRemAnd_proof.lean
 g2004h11h22hMissedhandhfold_proof.lean
 Popcount.lean
 bitvec_AndOrXor_1683_2.lean
 WillOverflow.lean

Those are unit test covered in the remainder of test cases that I imported so hopefully fine :+1:


Improved Virtual Pinball Spreadsheet Integration

I was wondering if you could add a feature that gives the ability to compare tables I have installed on my vpin to what is available on virtualpinball spreadsheet.  It doesn't need to be the various tables versions that are available for each table just that the game is installed.  This way I can see what games are available that I do not have.  It would help to catalog all installed tables versus all available tables on the spreadsheet.  I think this would be a useful feature.

Mmmh, I'm not sure about this. Right now there are over 1700 tables on VPS. If you want to "collect", yes, that would come in handy. But I usually try to improve the management of a single table. If the VPS guys add some analytics data one one day, it would be intesting to compare if you got all the (e.g.) TOP 100 downloaded tables. Or do you really try to install all tables that are available out there?

I am not trying to install all the tables.  I am just looking for an easier way to see what is available versus what I have installed.  I know that there is a downloadable excel document but it is constantly updated so its obsolete after you download it.  And the virtualpinball spreadsheet is great but not the most user friendly when looking at what is available.  I find it easier to use when I know what table I want.  When you have a lot of tables it can be a little difficult to remember all you have.
And I agree on the Top 100 downloaded tables.  That would be a great feature.

I see. I can add a view which includes all VPS tables, including an "installed" marker. I put it on the list.

That would be great

The first version is coming with 2.11.0.
Feel free to write additional tickets with improvements.

Nice. Thanks
On Sat, Jan 27, 2024, 9:31 AM syd711 @.***> wrote:

The first version is coming with 2.11.0.
Feel free to write additional tickets with improvements.
—
Reply to this email directly, view it on GitHub
https://github.com/syd711/vpin-studio/issues/96#issuecomment-1913173739,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANIYI6EXDRDZALH2L7HY7ILYQUFTXAVCNFSM6AAAAABCDH6Y6GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJTGE3TGNZTHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Spelling fix:  axel -> axle

Fix a spelling error (and try out submitting changes)
I did run "./go build all && ./go publish" successfully, but did not check in all the generated changes (because the clutter up the real change).

Thank you for that @robodan ! I'm a terrible speller, and I've made that mistake so many times.
Excellent work :+1:


Adjust Line thickness and Layout

Issue
At the current stage, the user can draw a color split line that indicates the polarity and adjust its thickness.
However, we do not use the color split anymore and the thickness does not really hold any information.
Desired Behavior
Fixed thickness and monochrome colored line.
Solution
Fix the thickness, remove the levers, and drastically reduce the complexity of drawing the line by switching from a length-wise color split line to a monochrome-colored line.

Call 27.07.2023:
We agreed that the new mask layout would be comprised of a monochrome segmentation mask and two points, one for the post and one for the pre-synaptic synapse.
The points are read from the materialization tables. They should be depicted on their designated slices with a bright color and be indicated on all other slices using a less bright color.
For the manual drawing of the segmentation mask, the user will be enabled to set the two dots on arbitrary slices that do not necessarily have to be the same slice.


Go indexer: Handle (anonymous) function literals. [phab:D1222]

Created by fromberger at 2017-01-05 22:38:26:

shahms accepted this revision at 2017-01-05 22:55:37

fromberger closed via commit 4609b2b36d9b2ea8c90bc3e9bc5455fa2996c1d5 at 2017-01-05 22:57:28


Size of panels changes when margins are passed in.

I cloned the repo and ran the examples.  I changed the static example page elements to size 15 height, and set rowHeight to 20.
When margin is not passed in:

When margin is passed in at [8, 8]:

The panel height should not be affected by margins.
This is affecting us here: https://github.com/elastic/kibana/issues/17114

The larger margins and panelHeight are, the more drastic the difference, because these values are multiplied on line 204 in GridItem.jsx
Doing:
rowHeight: 5,
margin: [0,0],
h: 30

upping margin to 20,20:

Ugh, nevermind. While investigating a "fix", I realize why it's written this way, to make sure the panels always line up.


Bill-debate linking

Something like this to link bills to relevant debates would be cool:

There's already some code done in fill-debate-bill-ids, but it's not very well optimised and wasn't ready for v1.

Additional relevant suggestions: https://www.reddit.com/r/singapore/s/OE9E9XwAgK


Move to qt5-serialport

We should replace our 3th party serial port library with serial port implemention from qt.
This shouldn't be hard, because serial port is used only in leddevices and they have a parent class with serial stuff.
Why?

delete code in hyperion, deleted code is good code :-)
better platform independent
qt serialport is better tested
we can use signal/slot from qt to be more event driven

Serialport migration is nearly done.
Can somebody tell me which sketch is the best for arduino + hyperion + ws2812?

One way: our forum, tut section, ricks post. There is a sketch which usually works

thx it worked, but I would suggest to set baudrate to 115200 as default. The current default was to slow for me and didn't work.

We need a thread for the usage and where are the limits. People should share their informations :)


BUG-58076 removed container option

@akanto pls check

Jenkins build failed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/2493/

Jenkins build failed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/2494/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/2495/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/2497/

Jenkins build finished, all tests passed.
Refer to this link for build results: http://ci.sequenceiq.com/job/cloudbreak-pull-request/2498/


Fix login on homepage

Issue:
location.href doesn't work on Safari, and router.navigate will append to the bottom of the page, and the original page does not remove. The first picture is homepage, and the second page is the homepage after logged-in. The homepage component is supposed to be removed after logged-in.

We use zone wrap around router.navigate as a workaround, and we don't know why.
The user service functions were also refactored by: Zuozhi Wang<EMAIL_ADDRESS>

@aglinxinyuan I suggest we remove demos/screen shots in the description, as there is no UI update to demo. we are fixing an issue and you just need to describe the issue. In this context we know what HomeComponent is referring to.


Missing biber file

INFO - Logfile is 'cache.blg'
read_file '/var/folders/54/hl7tfh_d27s7fzdfb5r70vdr0000gn/T/par-676c656e6e736368756c747a/cache-6bef3f811cde577dd70e44f9f5046ebc66712c46//inc/lib/Biber/LaTeX/recode_data.xml' - sysopen: No such file or directory at /var/folders/54/hl7tfh_d27s7fzdfb5r70vdr0000gn/T/par-676c656e6e736368756c747a/cache-6bef3f811cde577dd70e44f9f5046ebc66712c46/60940e05.pm line 115.
biber fails and when I follow the error to the cache the file is missing - however, I did not delete this file and the last time I complied my document everything was fine.  I often have to reinstall MacTex to fix biber problems.

Which version of biber? This is an old error that should have been fixed a couple of revisions ago.

whichever comes with MacTex2015.  Just fixed it today by reinstalling MacTex2015.  The most recent version for ElCapitain

On Mar 5, 2016, at 10:53 AM, plk<EMAIL_ADDRESS>wrote:
Which version of biber? This is an old error that should have been fixed a couple of revisions ago.
—
Reply to this email directly or view it on GitHub https://github.com/plk/biber/issues/111#issuecomment-192685804.


RESTful API module fore saleor

This is related to #259 as it also requires an API.

Closing as we now have a GraphQL API.


switch for settings and config package from libnuke, improve docs

Updates to<EMAIL_ADDRESS>Resources now define their own DeprecatedAliases and AlternativeResource (aka CloudControl API) resource names.
Removes more code from aws-nuke in favor of using packages from libnuke
Improving documentation around configuration of the tool
Replaces the bulk of the config package for the libnuke/pkg/config and extends it with the extras that aws-nuke needs.
Deprecates the use of feature-flags in favor of the settings.  Documentation has been added on how to migrate. Version 3 is still backwards compatible.
General code cleanups and improvements

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


fix: update schema utils

fixes #63

:tada: This PR is included in version 1.0.8 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Name of "Close Cursors" routine is too long

In SqlIO.tpl the name of the "close cursors" routine should be made shorter, because if causes generated names to frequently exceed the maximum identifier length. Replicator.dbl will need a corresponding change.

Shortened the name of the generated routine that closes cursors. Used to be xxx_CLOSE_CURSORS, now it's xxx_CLOSE.


Fixing Potential regression in kernel build pkg

This commit change the kernel build by removing the find -name '.*' -exec rm {} \; command and replacing it with a command after the kernel headers have been moved to their output directories.
But the replacement command doesn't account for if $FOR_CROSS build is being performed as in this case the include path is no longer guaranteed to be equal to $pkdir/usr/include. In my case see attachment below the actual path for $FOR_CROSS build is just $pkgdir/include as iglunix autobuild doesn't set any $FOR_CROSS_DIR variable.

This can't be placed before coping the files otherwise some important kernel build files are removed. But yes it doesn't handle cross compilation properly.

Okay @Ella-0 What if we instead handle both control paths with their own find statements? Just to allow for cross compilation to work correctly.

This is already handled in iglupkg with the PREFIX variable I think.

That might work on iglunix system but it doesn't seem to be the case for the iglunix-autobuild as it fails to make Linux kernel headers as the command find $pkgdir/usr/include -type f ! -name '*.h' -delete fails as it's looking for a directory that doesn't seem  to exist.
as the cross directory doesn't come out to $pkgdir/usr/include. Atleast on my system it just comes out to $pkgdir/include as the --for-cross-dir option in iglunix-autobuild is left blank, and the PREFIX variable doesn't seem to apply to the autobuild toolchain(or at all as I've not tested on iglunix).

weird I guess it needs to be done in both paths then

This patch should work for the mean time atleast making sure people can build it without issues.

Can ya squash the commits? and potentiall change the commit message to include linux:  as a prefix. Then I can get this merged :3 thank you!


Add nginx rewrite rule to permit vanity URLs for web actions.

Add nginx rewrite rule to permit vanity URLs for web actions:
https://guest.openwhisk.host/public/index.html becomes a proxy for
https://openwhisk.host/api/v1/experimental/web/guest/public/index.html
and for added convenience, and by convention
https://guest.openwhisk.host becomes a proxy for
https://openwhisk.host/api/v1/experimental/web/guest/public/index.html
this of course requires DNS resolution; for local development an entry in /etc/hosts will do.
For example:
> grep guest /etc/hosts
<IP_ADDRESS>  guest.dockerhost

This is a nice idea ! 👍
Is public a name for a package and index a name of an action ?

I'm also cross referencing web-actions docs.

This is a great idea!

pg3/327 🔵

rebased and merged in 71b185279f098bbf4e9a086f1cc4faf10a7e5a3f


Parallelize across thresholds

Metrics are currently computed one threshold value at a time and vectorized across bootstraps. Ideally computation would be vectorized across both, but most systems run into memory issues that prohibit this operation. Instead, we should use multiprocessing to parallelize the for-loop over thresholds across multiple cores (https://www.delftstack.com/howto/python/parallel-for-loops-python/).

Alternatively, use an alternative approach for computing the confusion matrices that more efficiently allows you to vectorize across both. The previous approach had a memory footprint of (data size * num thresholds * num bootstraps), but we can do better by computing the confusion matrices via cumulative sums so that the memory footprint is only (data size * num bootstraps). Then we can vectorize across both or parallelize across bootstraps (?)
Reference: SQL code I made for a Retool dashboard
This computes a confusion matrix for every single data point by looping over the data only once.
SUM("label") OVER (ORDER BY score) AS fn, -- num 1s below threshold
ROW_NUMBER() OVER (ORDER BY score) - SUM("label") OVER (ORDER BY score)  AS tn, -- num 0s below threshold
num_actual_1 - SUM("label") OVER (ORDER BY score) AS tp, -- num 1s above threshold
(num_examples - ROW_NUMBER() OVER (ORDER BY score)) - (num_actual_1 - SUM("label") OVER (ORDER BY score)) AS fp -- num 0s above threshold

==> convert that to numpy

Solved in https://github.com/chrismbryant/classification-curves/pull/18. Computed using cumulative sums, and parallelized across bootstraps


: error G5FE39F1E: Type 'DecoderCallback' not found.

Describe the bug
When trying to build to windows I am getting the following error
Launching lib\main.dart on Windows in debug mode...
windows/flutter/CMakeLists.txt does not use FLUTTER_TARGET_PLATFORM, updating.
CMake Deprecation Warning at CMakeLists.txt:4 (cmake_minimum_required):
Compatibility with CMake < 3.5 will be removed from a future version of
CMake.
Update the VERSION argument  value or use a ... suffix to tell
CMake that the project does not need compatibility with older versions.
CMake Warning (dev) at C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/CMake/share/cmake-3.27/Modules/ExternalProject.cmake:3136 (message):
The DOWNLOAD_EXTRACT_TIMESTAMP option was not given and policy CMP0135 is
not set.  The policy's OLD behavior will be used.  When using a URL
download, the timestamps of extracted files should preferably be that of
the time of extraction, otherwise code that depends on the extracted
contents might not be rebuilt if the URL changes.  The OLD behavior
preserves the timestamps from the archive instead, but this is usually not
what you want.  Update your project to the NEW behavior or specify the
DOWNLOAD_EXTRACT_TIMESTAMP option with a value of true to avoid this
robustness issue.
Call Stack (most recent call first):
C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/CMake/share/cmake-3.27/Modules/ExternalProject.cmake:4345 (_ep_add_download_command)
CMakeLists.txt:9 (ExternalProject_Add)
This warning is for project developers.  Use -Wno-dev to suppress it.
/C:/Users/trust/AppData/Local/Pub/Cache/hosted/pub.dev/pdfx-2.4.0/lib/src/viewer/pdf_page_image_provider.dart(24,55): error G5FE39F1E: Type 'DecoderCallback' not found. [C:\Mobile\digidata\build\windows\x64\flutter\flutter_assemble.vcxproj]
/C:/Users/trust/AppData/Local/Pub/Cache/hosted/pub.dev/pdfx-2.4.0/lib/src/viewer/pdf_page_image_provider.dart(40,7): error G5FE39F1E: Type 'DecoderCallback' not found. [C:\Mobile\digidata\build\windows\x64\flutter\flutter_assemble.vcxproj]
/C:/Users/trust/AppData/Local/Pub/Cache/hosted/pub.dev/pdfx-2.4.0/lib/src/viewer/pdf_page_image_provider.dart(24,55): error G93EAFBA7: 'DecoderCallback' isn't a type. [C:\Mobile\digidata\build\windows\x64\flutter\flutter_assemble.vcxproj]
/C:/Users/trust/AppData/Local/Pub/Cache/hosted/pub.dev/pdfx-2.4.0/lib/src/viewer/pdf_page_image_provider.dart(40,7): error G93EAFBA7: 'DecoderCallback' isn't a type. [C:\Mobile\digidata\build\windows\x64\flutter\flutter_assemble.vcxproj]
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft.CppCommon.targets(254,5): error MSB8066: Custom build for 'C:\Mobile\digidata\build\windows\x64\CMakeFiles\b21556f3db189beffdc5aa2a80c275a0\flutter_windows.dll.rule;C:\Mobile\digidata\build\windows\x64\CMakeFiles\2dcd5d3868b6cede2a3c2396dfe52ea6\flutter_assemble.rule;C:\Mobile\digidata\windows\flutter\CMakeLists.txt' exited with code 1. [C:\Mobile\digidata\build\windows\x64\flutter\flutter_assemble.vcxproj]
Exception: Build process failed.
Exited (1).
To Reproduce
Steps to reproduce the behavior:
flutter build windows
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

I also get the error building to Android
/C:/Users/trust/AppData/Local/Pub/Cache/hosted/pub.dev/pdfx-2.4.0/lib/src/viewer/pdf_page_image_provider.dart:24:55: Error: Type 'DecoderCallback' not found.
ImageStreamCompleter load(PdfPageImageProvider key, DecoderCallback decode) =>
^^^^^^^^^^^^^^^
/C:/Users/trust/AppData/Local/Pub/Cache/hosted/pub.dev/pdfx-2.4.0/lib/src/viewer/pdf_page_image_provider.dart:40:7: Error: Type 'DecoderCallback' not found.
DecoderCallback decode) async {
^^^^^^^^^^^^^^^
/C:/Users/trust/AppData/Local/Pub/Cache/hosted/pub.dev/pdfx-2.4.0/lib/src/viewer/pdf_page_image_provider.dart:24:55: Error: 'DecoderCallback' isn't a type.
ImageStreamCompleter load(PdfPageImageProvider key, DecoderCallback decode) =>
^^^^^^^^^^^^^^^
/C:/Users/trust/AppData/Local/Pub/Cache/hosted/pub.dev/pdfx-2.4.0/lib/src/viewer/pdf_page_image_provider.dart:40:7: Error: 'DecoderCallback' isn't a type.
DecoderCallback decode) async {
^^^^^^^^^^^^^^^
Target kernel_snapshot failed: Exception
FAILURE: Build failed with an exception.

Where:
Script 'C:\Flutter\packages\flutter_tools\gradle\src\main\groovy\flutter.groovy' line: 1350

What went wrong:
Execution failed for task ':app:compileFlutterBuildRelease'.

Process 'command 'C:\Flutter\bin\flutter.bat'' finished with non-zero exit value 1

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.

Please check the new version on PubDev, it has been updated to 2.6.0 and includes the fix for the DecoderCallback issue.


JavaScript Issues That Makes The Project Fail

I just reviewed your code and I spotted some issues that may be affecting the functionality of the code.

Correct the variable name

Try fixing that by naming it links to help the for each looping to work as expected.

Wrong Id names

The ids you used when creating the variable were all since they are different from those in your html file as in the screenshot below:

e.g for name your id in the html file is fname but in the creation of the variable name you have it as name try fixing that and you'll see you code function as intended :grin:

Interpolation of string

To help fix this I hope the documentation will do you good. Here is the link to string interpolation MDN Web Docs.

For your case replace the XXXXX with something like ${suggestedEmail} this will be adding the dynamic rendered data to the variable

Wrong Variable Calling

When creating the variable, you called it suggestedEmail but you called it wrongly in the conditional statement.

Fixing that final issue will help ensure you code work like a Professional's code :sunglasses: :+1:

Thank you, I will take a look on that. Looks like I forgot to take notice on that.


Fix bypass for pull-secret

Like it's described in the README a bypass is needed for the pull-secret. This should be fixed - if it is already the README should be changed.
I've added this issue because I think it is important for the stable release and it has to be documented so that no one will forget about it. The (fix coming soon) comment in the README was added in November last year.

Unfortunately we'd have to ship GA with this workaround ({"auths":{"fake":{"auth": "bar"}}}) due to lack of time to prepare a fix which can be merged upstream

https://github.com/openshift/okd/issues/264 has a better description of the same problem

using this pull secret... {"auths":{"fake":{"auth": "bar"}}} i now have problems with my s2i build config.
Cloning "git@github.com:jkassis/xxxxx.git" ... 	Commit:	f55bf64db1ce4366eb8319f8a507e6cc5f114216 (working on s2i build) 	Author:	Jeremy Kassis<EMAIL_ADDRESS>Date:	Mon Jul 27 17:53:51 2020 -0700 Caching blobs under "/var/cache/blobs". E0728 04:28:56.827076       1 config.go:236] while trying to parse blob "{\"auths\":{\"fake\":{\"auth\":\"bar\"}}}\n": unable to parse auth field Warning: Pull failed, retrying in 5s ... E0728 04:29:02.732065       1 config.go:236] while trying to parse blob "{\"auths\":{\"fake\":{\"auth\":\"bar\"}}}\n": unable to parse auth field Warning: Pull failed, retrying in 5s ... E0728 04:29:08.762960       1 config.go:236] while trying to parse blob "{\"auths\":{\"fake\":{\"auth\":\"bar\"}}}\n": unable to parse auth field Warning: Pull failed, retrying in 5s ... error: build error: After retrying 2 times, Pull image still failed due to error: errors: denied: requested access to the resource is denied unauthorized: authentication required

i changed the content of -n openshift-config secret/pull-secret but this error continues to raise. oc set data secret/pull-secret -n openshift-config --from-file=.dockerconfigjson=/Users/jkassis/.docker/config.json

changing the pull-secret actually did change this message. it just took a while to propagate.


✨ Randomly generate a graph

The user should be able to select the number of nodes they want and click a button and the program will randomly generate a collection of connected nodes on the plane and center the camera appropriately.
For example: Obsidian note taker does this with it's graph view. It just generates a graph visually.

As a starting point, this page gives a high level implementation idea:

I would like to use a better sampling distribution than random such as best candidate: discussed here

Delauney Triangulation for generation of edges
A set of triangles made from a discrete set of vertices such that no vertex lies inside the circumcircle (circle which passes through all three points of the triangle) of any triangle in the set. Named after Boris Delauney.
We can compute the triangulation with the Bowyer-Watson algorithm
Nice explanation video
See PDF below for detailed breakdown of algorithm:
07_An-implementation-of-Watsons-algorithm-for-computing-two-dimensional-Delaunay-triangulations.pdf

& is bitwise and and && is logical and.
The expression x && y will return 1 if both x and y is non-zero, and 0 otherwise. Note that if x is zero, then y will not be evaluated at all. This will matter if y is an expression with side effects. This behviour is called short circuiting.

At the moment I have a weird bug which pops up every now and then. When randomly generating it seems that occassionally the incorrect number of triangles is generated. For instance 9 should be generated by instead 11 are. It seems to me that in this case the resulting triangles intersect. I'm not sure if it has to do with the distribution of the points maybe or some other issue in my implementation of the algorithm. I know for sure that if the distribution is too linear it causes issues, so I'm leaning towards that hypothesis, although I would prefer it if it was a problem in my implementation rather.
A case for which more triangles are generated than expected is for the points:
(-88,-7)
(-33,-6)
(17,11)
(-62,98)
I will debug these points to see if I can pick up on what the bug is.

Sometimes triangulation does fail however, for a case like this:

I think probably I need some check on this, such that it does not happen. Perhaps I need to seed my uniform distribution better or something.

Work well for 4 points but if I increase it up to 10 it fails quite regularly. I'm not sure if that's just due to an incorrect assumption on my part, that the number of triangles generated should be: 2N+1. Possibly this is not always the case?
If I remove that assertion then I do see a clear difference between triangulations which match the expectation versus those which do not, so I feel like there is still something which is wrong. Perhaps in the function which removes duplicate lines the possibility of 3 lines could occur in which case my function would fail.

Lines are also still crossed over.

A case for which 25 triangles are generated when only expecting 21 to be.
[0] = (92, 95)
[1] = (-66, 36)
[2] = (-55, 96)
[3] = (61, -46)
[4] = (33, -19)
[5] = (19, -93)
[6] = (-62, -98)
[7] = (-72, 53)
[8] = (0, 33)
[9] = (-40, 1)

Better but still occassionally too many triangles are generated. So there is still a bug. For the ones where too many are generated I can see that lines are crossing over. For example:
Here I expected 41 triangles and got 127. (A particularly bad example)

triangles formed: 23
triangles that should be formed: 21
( -12;-41 )
( 17;-84 )
( 34;63 )
( 7;17 )
( 99;68 )
( 36;99 )
( -45;-84 )
( -50;-54 )
( 32;72 )
( 41;-36 )

Working much better now. I still caught a case which didn't work while using 100 points:
[0]: (-2, -93)
[1]: (-95, -2)
[2]: (-18, 38)
[3]: (3, -35)
[4]: (14, 98)
[5]: (-24, -83)
[6]: (-22, -79)
[7]: (-21, 40)
[8]: (-15, -17)
[9]: (-73, 29)
[10]: (48, 35)
[11]: (61, 21)
[12]: (-62, -31)
[13]: (11, 60)
[14]: (48, -81)
[15]: (26, -77)
[16]: (84, -86)
[17]: (18, -63)
[18]: (-55, 30)
[19]: (-28, 1)
[20]: (-63, 26)
[21]: (88, 94)
[22]: (-85, -9)
[23]: (-47, -37)
[24]: (-78, -29)
[25]: (-9, 74)
[26]: (-16, 10)
[27]: (44, -74)
[28]: (-67, -27)
[29]: (49, 39)
[30]: (-96, -17)
[31]: (2, -45)
[32]: (-34, -45)
[33]: (11, -11)
[34]: (93, -76)
[35]: (-91, -87)
[36]: (9, -93)
[37]: (-37, -2)
[38]: (94, 76)
[39]: (-55, 32)
[40]: (-74, 55)
[41]: (-10, -62)
[42]: (83, 59)
[43]: (-66, 47)
[44]: (-52, -83)
[45]: (66, 54)
[46]: (44, 67)
[47]: (-54, -12)
[48]: (17, 23)
[49]: (69, -54)
[50]: (-66, -66)
[51]: (50, 86)
[52]: (-18, 72)
[53]: (57, 65)
[54]: (-65, 11)
[55]: (-7, 19)
[56]: (51, 98)
[57]: (-16, 76)
[58]: (82, -83)
[59]: (-15, -63)
[60]: (55, -51)
[61]: (-73, -39)
[62]: (-15, 22)
[63]: (-39, 57)
[64]: (-53, 6)
[65]: (96, -98)
[66]: (76, 68)
[67]: (93, -83)
[68]: (50, 78)
[69]: (57, -66)
[70]: (39, 45)
[71]: (-89, 30)
[72]: (-27, 45)
[73]: (-99, -85)
[74]: (31, -61)
[75]: (-43, 11)
[76]: (84, -61)
[77]: (2, 17)
[78]: (-26, -73)
[79]: (75, 83)
[80]: (-72, -19)
[81]: (-73, -38)
[82]: (-32, -33)
[83]: (98, -39)
[84]: (-28, -74)
[85]: (63, -7)
[86]: (-47, -37)
[87]: (35, 99)
[88]: (-89, 55)
[89]: (39, 79)
[90]: (-94, 80)
[91]: (-39, -81)
[92]: (78, 18)
[93]: (44, 77)
[94]: (-94, -84)
[95]: (-69, 95)
[96]: (14, 40)
[97]: (-45, 37)
triangles formed: 203
triangles that should be formed: 201

For N=1000 I still have issues (apart from how awfully slow it is):
n_1000.txt

check the file for duplicate points -> no duplicates
add a check that for each point introduced there are always 2 more triangles added.

for N=1000 things go wrong at point (4,-51) where triangles formed are 1699 but should be 1697 (i.e 4 triangles were made instead of 2)
Inspect polygon for the previous point  (-92,-66 ) -> before are 15 edges and it removes 6 to give 9 as expected.


Handling APE tags

Hi!
Thanks for creating this library and doing it properly (e.g. not "estimating" the length but really measuring it)!
There were some files in my collection which tripped up however, with the error:
Unexpected frame, header:<PHONE_NUMBER>
In hex thats 0x41504554, which can be read as APET. This points to APE-Tags in the file (https://en.wikipedia.org/wiki/APE_tag). Until today I didn't even know that these could be inside of mp3 files, but I would really appreciate it if you could add some code to skip these.

Thank you for the report!
Can you email me one of the files causing issues at<EMAIL_ADDRESS>(or upload it somewhere)?

I already have a patch for this (is ready now), should I send the file to you anyway?

Thanks a lot for going through the effort of writing the fix <3
I posted a couple minor requests on the PR and then we're good to go.

Published on crates.io as v0.1.5

thanks!


how to set the preferredChargeMode ?

Describe the bug
I'm not sure it's really a bug, but I can't find a way to do it.
I managed to get the preferredChargeMode of my car (VW ID.3), and want to set it if I detect it's incorrectly set (which happens randomly, I suppose another bug from VW)
I managed to get up to this point:
>>> chargemode.preferredChargeMode.value
<ChargeModeEnum.PREFERRED_CHARGING_TIMES: 'preferredChargingTimes'>

But I couldn't find the method to change it. I tried to use the valueSetter in this preferredChargeMode object, but with no success…
TypeError: 'NoneType' object is not callable

How can I fix this ?
Thanks a lot

Hi, it is not implemented to be changed. I am not sure if the API allows to change it at all.

BTW can I do anything to help ?


Load Animal sheets with multiple labeled elements and multiple tracers

FEATURE DESCRIPTION
Feature Inspiration
Many experiments are being performed using multiple tracer compounds per animal and multiple isotopes per tracer compound. These type of experiments must be able to be stored in TraceBase.
Feature Description
Implement loading of Animal sheets with multiple labeled elements and multiple tracers. Using the parser implemented in #407, update the loading scripts to be able to load the sample data from #390 and #399 into the updated models (#406 and #408).
Alternatives Considered
We discussed providing for the ability to load by old format tracer data and newly formatted tracer data.
Comment
Note that loading of the AccuCor/IsoCorr mass spec data is handled in a separate issue.

ISSUE OWNER SECTION
Assumptions

List of assumptions made WRT the code
E.g. We will assume input is correct (explaining why there is no validation)

Requirements

List of conditions to be met for the feature
E.g. Every column/row must display a value, i.e. cannot be empty

Limitations

A list of things this work will specifically not do
E.g. This feature will only handle the most frequent use case X

Affected/Changed Components

Files
Environment variables
External executables
Database tables
Cron job settings
Etc.

DESIGN
GUI Change description
Describe changes the user will see.
Code Change Description (Pseudocode optional)
Describe code changes planned for the feature.
Tests
A test should be planned for each requirement (above), where possible.

I inadvertently did this in #467


Changed Markdown images' background color to transparent

What's the issue:
Markdown images have a background-color so transparent images appear with a darker background than the page itself.
What's fixed:
Markdown images' background is now transparent
Screenshots:

Fixes #102

Thank you for the contribution!


Fix global security config

We didn't actually security when merging.

@swift-ci please smoke test


SyntaxError

solium 0.2.2
Anytime I attempt to lint the Numeraire contracts, I get this SyntaxError:
$ solium --file contracts/Numeraire_dependencies.sol
An error occurred while running the linter on contracts/Numeraire_dependencies.sol:
Error: An error occured while parsing the source code:
SyntaxError: Expected "!=", "!==", "%", "%=", "&", "&&", "&=", "(", "*", "**", "*=", "+", "++", "+=", ",", "-", "--", "-=", ".", "/", "/*", "//", "/=", ";", "<", "<<", "<<=", "<=", "=", "==", "===", ">", ">=", ">>", ">>=", ">>>", ">>>=", "?", "[", "^", "^=", "in", "instanceof", "|", "|=", "||", comment, end of line or whitespace but "{" found. Line: 130, Column: 93
    at EventEmitter.lint (/Users/admin/.nvm/versions/node/v6.9.2/lib/node_modules/solium/lib/solium.js:88:10)
    at lintString (/Users/admin/.nvm/versions/node/v6.9.2/lib/node_modules/solium/lib/cli.js:126:24)
    at lintFile (/Users/admin/.nvm/versions/node/v6.9.2/lib/node_modules/solium/lib/cli.js:157:9)
    at filesToLint.map.file (/Users/admin/.nvm/versions/node/v6.9.2/lib/node_modules/solium/lib/cli.js:176:45)
    at Array.map (native)
    at lint (/Users/admin/.nvm/versions/node/v6.9.2/lib/node_modules/solium/lib/cli.js:176:33)
    at Object.execute (/Users/admin/.nvm/versions/node/v6.9.2/lib/node_modules/solium/lib/cli.js:287:19)
    at Object.<anonymous> (/Users/admin/.nvm/versions/node/v6.9.2/lib/node_modules/solium/bin/solium.js:11:5)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
No issues found.

The contracts compile fine with solc 0.4.10.

Thanks for reporting this @xanderdunn

Maybe you can add something like https://github.com/Latrasis/linter-solidity/blob/master/lib/compiler.js ?

Similar issue with solidity-parser was fixed over here.
Also relevant to #58

Solium has now switched to solidity-parser as dependancy instead of solparse. So this won't give you trouble once v0.5.0 is released (which is very soon :) )


Add Badges initiative. Update links to website-redesign and i18n.

Oops, didn't see https://github.com/nodejs/community-committee/pull/243 😅

@amiller-gh do you wanna remove the url change stuff so it doesn't conflict anymore?

@oe resolved 👍 Sorry about that, busy week.

As far as I know, the badges initiative was ratified in this week's meeting, so I'm going to merge this.


How Can I precache and route custom folder, or public folder for example?

Hi,
when online, cached assets or assets in public folder are available
but when offline it's not anymore.
How can I tell the sw.js to precacheAndRoute specific files/folder?

or maybe use another custom sw.js template instead of templates/workbox/sw.js?

@robotpapier I'm gonna add the possibility to use custom workbox template :)


Not Getting Auto Information

When I go to type something it dosen't completes itself automatically nor does it shows the options of what we can write .
Like I press P or any other letter it is not showing me the drop-down list of options
and I have to write everything myself. so, what's the use Visual Studio then I can go to any other editor too but I like Visual Studio Please Fix This Problem of Mine.

/needsMoreInfo

CTRL+SPACE

opens intellisense


Allow Number Features to choose how the tool numbers features

As a user I might want to number buildings by starting at differnt locations:
LL
UR
LR
UL
Expected Behavior

A user would be able to identify how the tool was labeled.
Current Behavior

Possible Solution

The Sort data tool has logic that allows input for spatial_sort_method, which might be something that we can expose to the tool.
Steps to Reproduce (for bugs)

Context

Your Environment

Version used:
Environment name and version (e.g. Chrome 39, node.js 5.4):
Operating System and version (desktop or mobile):
Link to your project:

This could also capture the idea that some users want to use a pattern such as clockwise ,counter clockwise, and spiral out from a primary feature.


Issue with Azure AD

Thanks for making this plug-in, Finalist team!
We are getting user info back from our AzureAD tenant, but this error is stopping us right now:
"Unexpected: OpenID Connect UserInfo does not contain email field. Cannot correlate to Liferay user."
We have set the portal-ext.properties file as follows:
openidconnect.enableOpenIDConnect=true
openidconnect.token-location=https://login.microsoftonline.com/{{Our tenant ID here}}}/oauth2/token
openidconnect.authorization-location=https://login.microsoftonline.com/{{Our tenant ID here}}}/oauth2/authorize
openidconnect.profile-uri=https://login.microsoftonline.com/{{Our tenant ID here}}}/openid/userinfo
openidconnect.issuer=https://sts.windows.net/{{Our tenant ID here}}}/
openidconnect.client-id={{Our client ID here}}}
openidconnect.secret={{Our secret key here}}}
openidconnect.scope=openid profile email

Is there some setting we're missing that could be causing this error?
We did notice this in ProviderFactory.java:
openidconnect.provider
Will setting the value of the provider as follows "openidconnect.provider=AzureAD" help? Or is there another property we need to set that's not included in your README file? Any guidance you can give would be greatly appreciated. Again, thanks for making this!

Hi,
Thanks for trying out this plugin.
Azure AD should work, support for it was added recently.
And you are right in finding the property openidconnect.provider, however the value should not be AzureAD but simply azure.
I will update the documentation with this, we failed to do this immediately.
Please confirm that this works for you, that will probably also help others.
Hope this helps.
Geert

Thanks, Geert! We tested that setting and still get the error. Here's more info. on the error:
18:16:19,153 ERROR [http-nio-8443-exec-3][Liferay70Adapter:70] Unexpected: OpenID Connect UserInfo does not contain email field. Cannot correlate to Liferay user. UserInfo: {sub={{REDACTED}}, ver=1.0, aio={{REDACTED}}=, amr={{REDACTED}}, onprem_sid={{OBFUSCATED}}, oid={{REDACTED}}, uti={{REDACTED}}, given_name=Ryan, tid={{REDACTED}}<EMAIL_ADDRESS><EMAIL_ADDRESS>name=Benedetti, Ryan, ipaddr={{REDACTED}}, family_name=Benedetti}
So ... we are getting info. back from Azure AD but the email field is missing. I thought it might be the scope but the config from our tenant (https://login.microsoftonline.com/{REDACTED}/v2.0/.well-known/openid-configuration) shows that the openid, profile, and email scopes are enabled on Azure AD:

Have you tried looking into the fields that are actually sent over the wire? This can be done by enabling debug logging for the OpenID Connect filter (category nl.finalist.liferay.oidc) in the Liferay control panel (Server administration -> log levels)

Now that I look again at your error message, I'm not sure whether the openidconnect.provider setting is actually working for you. But enabling debug logging (like I proposed in my previous comment) might shed a light on this as well, as instantiation of the provider class will output a message on debug level.

Geert,
Re: "Have you tried looking into the fields that are actually sent over the wire?"
We turned on debugging as you suggested and the following fields are sent back (with my REDACTS added to protect sensitive data):
17:25:41,155 DEBUG [http-nio-8443-exec-2][Liferay70Adapter:60] Setting OpenIDUserInfo object in session: {sub=[REDACTED], ver=1.0, aio=[REDACTED], amr=[REDACTED], onprem_sid=[REDACTED], oid=[REDACTED], uti=[REDACTED] given_name=Adam, tid=[REDACTED]<EMAIL_ADDRESS><EMAIL_ADDRESS>name=Murakami, Adam, ipaddr=[REDACTED], family_name=Murakami}
So we are indeed getting data back, just no email.

We noted that the version built on the Marketplace is listed as 0.2.2, but that the current version with Azure AD additions is 0the .2.3-SNAPSHOT (with the version change being updated on Feb. 27, 2017). We've been using 0.22, but we will download/build the 0.23 version and see if we can get it working in our DEV environment. We're fairly new to OSGI, so if you have any advice on building this in Liferay Dev Studio for DXP, we would greatly appreciate it.
Thanks again for all of your expedient and expert help, Geert!

Geert,
I'm excited to report that after some fiddling with bnd workspace and maven, we got the 0.2.3 version to build, deployed it and we get: "01:26:01,711 TRACE [http-nio-8443-exec-4][Liferay70Adapter:50] Returning credentials for userId
So we now have success with the plug-in on AzureAD!!! Thanks for all of your help with this!!!

Hi,
Great to hear that you got it working in the end. And you are right, the version on the marketplace lags behind a bit, especially the Azure AD feature.
I will release a new version soon and publish it on the marketplace. Let's keep this issue open until this happens, so to reflect this state...

Submitted 0.3.0 for review in the marketplace.

0.3.0 is published on the MP a while ago, will close this now.


vls: policy-routing-balanced failure

Some users reported an issue paying an invoice with the following error:
[2023-09-20T08:17:22Z WARN  lightning_signer::policy] policy failed: policy-routing-balanced validate_payment_balance: incoming_msat + max_to_invoice_msat < outgoing_msat: 0 + 0 < 2498000

This happens when processing the following ValidateCommitmentTx request:
0023000000df020000000191fcb6f06b5b94586b13101b827bb204d78137d87031a346b27501280b239f0d0000000000539ca180044a0100000000000022002096516bcdab4c23f3fa72953bd7214d1fed2c5b537462593b5e5af212d9c96ea34a01000000000000220020d5d031569d1d9b2697f4e6c58064cae906570a0ca246592abb644b1e070fb28ec309000000000000220020a47089358f15df307dc9fbe6105e8bbaed6f5ec9033307992c5b92f83f75b0b18d7f010000000000220020586919ba64a087365ece03fe2fd9648aac3d09e81e324b41593ddad4aada8121c6c7f020000002fd70736274ff0100df020000000191fcb6f06b5b94586b13101b827bb204d78137d87031a346b27501280b239f0d0000000000539ca180044a0100000000000022002096516bcdab4c23f3fa72953bd7214d1fed2c5b537462593b5e5af212d9c96ea34a01000000000000220020d5d031569d1d9b2697f4e6c58064cae906570a0ca246592abb644b1e070fb28ec309000000000000220020a47089358f15df307dc9fbe6105e8bbaed6f5ec9033307992c5b92f83f75b0b18d7f010000000000220020586919ba64a087365ece03fe2fd9648aac3d09e81e324b41593ddad4aada8121c6c7f0200001012b289a01000000000022002087f8eb092754e219e7a66a6b900360a4cd31b927ed4855469bcbd1268e49b69a220203daa3e7827dcda27b240797ff43691f62ef5db21a3c78da57b33b9c99f280592b47304402204964b53cb5c929857c6c1f5712528ff6ad439ff25221b0c47f6531d81d9966fe02206fcf946f7c330aa0eca026ffaf0c655f7fd3d765f21c6eb278a7ff7c6b3a705d010103040100000001054752210293585196fac8ce82ba42cae3c51deda74667739edf287588cbbe69bf3a07a98d2103daa3e7827dcda27b240797ff43691f62ef5db21a3c78da57b33b9c99f280592b52ae22060293585196fac8ce82ba42cae3c51deda74667739edf287588cbbe69bf3a07a98d084330c3f000000000220603daa3e7827dcda27b240797ff43691f62ef5db21a3c78da57b33b9c99f280592b086c959ddf00000000000101282103daa3e7827dcda27b240797ff43691f62ef5db21a3c78da57b33b9c99f280592bac736460b26800010128210293585196fac8ce82ba42cae3c51deda74667739edf287588cbbe69bf3a07a98dac736460b2680001014d6321037104bcdd022a1507ce3a2e9b975e5b77f2bc82a2217424b9279f26d800278e2e67029000b275210246b6a4612a056a84eb45f00bf490f77aaa4219aaae066bb4a24228e510c48f8c68ac000101252103c9e0ad6900853d166362c7c6a15524c40046f8bd201ea140b247d514fd57d618ad51b2000000000000000000001700000cb14964b53cb5c929857c6c1f5712528ff6ad439ff25221b0c47f6531d81d9966fe6fcf946f7c330aa0eca026ffaf0c655f7fd3d765f21c6eb278a7ff7c6b3a705d010000

This indicates that VLS is failing to account for an invoice that was allowlisted before the verifying the commitment. There could be a variety of different causes for this, so we'd want to reproduce this issue using the repro-tester setup similar to what we did with overflow-panic-repro.
To do so we need the signer logs from someone running on main, including the Processing request and Signer state lines and eventually any line indicating that another state was being used. Using those we can create the JSON representation of the repro case and feed it into the test.
That being said, we have bumped the VLS version a number of times since the case was observed, and we can't reproduce it anymore despite trying several dozen payments, hence why we ask for a repro case in the wild :-)

Reproduced here: https://github.com/breez/breez-sdk/issues/481
I also included trace logs for gl_client, lightning_signer and vls_protocol_signer.
Let me know if that's enough info to troubleshoot it, or if there's anything else you need.

It appears like that failure stems from the fact that VLS assigns a fixed 10k msat fee to each HTLC, which doesn't match our fee budgeting, so we'll need to relax that limit a bit.

I will remove that debug statement, and if the issue does not reappear I think we can close this, right @ok300?

Yes, the issue seems to be gone. This can be closed.


fix: cashout to overlay address if no chequebook

the last PR did not handle cashouts correctly if running without a chequebook. this prevents a go panic and uses the nodes overlay eth address as a destination instead.

This change is

This is the code I expected to see in the previous PR!  Great job!

Codecov Report

Merging #2732 (649dce8) into master (90760e9) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #2732   +/-   ##
=======================================
  Coverage   63.68%   63.69%
=======================================
  Files         237      237
  Lines       26799    26800    +1
=======================================
+ Hits        17068    17069    +1
- Misses       8201     8203    +2
+ Partials     1530     1528    -2

Impacted Files
Coverage Δ

pkg/settlement/swap/swap.go
31.79% <100.00%> (+0.28%)
:arrow_up:

pkg/localstore/subscription_pull.go
84.24% <0.00%> (-3.43%)
:arrow_down:

pkg/p2p/libp2p/internal/reacher/reacher.go
90.90% <0.00%> (-1.30%)
:arrow_down:

pkg/localstore/mode_set.go
55.22% <0.00%> (+1.11%)
:arrow_up:

pkg/localstore/mode_put.go
60.05% <0.00%> (+1.14%)
:arrow_up:

Continue to review full report at Codecov.

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


monte carlo localization interface

I searched the issues and found the first one pretty interesting #206
I already started working on it, but before my next pull request gets to complex, I wanted to lay some groundwork.
The algorithm in the 3rd ed book fig 25.9 is pretty complex and not very generic. The interface I propose is a generic "a Monte Carlo Localization is figuring out ones own state through percepts from the environment". This generic definition is closer to what wikipedia say to MCL The whole implementation with noisy sensors etc can be built on top.
If you agree with this interface, I'd like to provide an example. The AIMA3 example is pretty complex too and I am not willing to copy/implement this whole 2D geometry, ray casting etc. My example would be a much simpler continuous environment, if that's ok. If you like, I can provide a "Story" and a general description of the environment and the agent therein later.

I need to think about this some more but am curious where this will lead. As your proposal diverges from the description from the book for now can you work on this in the 'extra' module (i.e. move the two initial interfaces that you are proposing to live under there). Also, as none of the supporting probability code is available in AIMA4e currently I am not sure of how much of an impact this will be on what you are proposing.

ok, will move it to extra


fix: duplicate build descriptor

Jenkins 2.x Fixed an issue where Invoke Flyway would be displayed dupe.
multi Extension anotation remove.

[ ] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your master branch!
[ ] Ensure that the pull request title represents the desired changelog entry
[ ] Please describe what you did
[ ] Link to relevant issues in GitHub or Jira
[ ] Link to relevant pull requests, esp. upstream and downstream changes
[ ] Ensure you have provided tests - that demonstrates feature works or fixes the issue

screen capture.

Hi @pskumar448
The plugin is in a quiet "bad" state and I'm interested to be co-maintainer of it.
Do you agree with it ?
I open a PR (work in progress, not working yet) that includes all open PR and fixes : https://github.com/jenkinsci/flyway-runner-plugin/pull/23
Best regards,


Center search result once selected

Small, but handy feature. When user selects a line from the Grep menu, execute zz on file.
Currently it moves cursor to the correct line, however it's not centred on the screen which makes it hard(er) to quickly find the cursor position.

hi @savchenko, is it relate to this issue: https://github.com/linrongbin16/fzfx.nvim/issues/426

hi @savchenko  thanks for suggestion, it seems a good feature.


Add Attribute class to keep constants in IR.

This was reviewed in #328, which I accidentally merged it into non-main branch.


etcdserver: purge old snap.db files

Lots of garbage db files in #7957. Should purge.

lgtm

@heyitsanthony can this potentially affect recovering db path? suppose there are 5 .snap files already and 1 xxx.snap.db from leader (maxsnap = 5). while etcdserver is applying snapshot and purgefile go routine deletes xxx.snap.db. then xxx.snap.db can't be found and etcdserver panic.

@fanminshi if that's the case, why is it safe to purge the v2 .snap files? Can you reproduce this issue where the server needs a backlog of five snapshots?

@heyitsanthony it seems to me that it is possible that an etcd can potentially have more than one .snap files and one .snap.db file. That's why we purge then if greater than some threshold right? In the case of https://github.com/coreos/etcd/issues/7957, there are already more than N xxx.snap.db files. If the example I gave just have 6 xxx.snap.db files, where the 6th one is from from the leader. while follower is applying the 6th xxx.snap.db file and the snap.db file gets deleted by purge function.  will that be a issue then?

@fanminshi the purge is sorted; etcd will only purge the oldest files. I'm asking if .snap purging is safe, then why wouldn't .snap.db purging also be safe? If .snap purging is not safe, there needs to be a test case around it that breaks using the current purging policy.

@heyitsanthony I see. if purge only deletes oldest one, then purge is safe and my concern doesn't matter.  LGTM


add countries as selection

... basis for polygon supported excerpt (they are just predefined polygons).
While moving them:

Instead of regenerating the country data each time migration is executed, we should provide a fixture instead.

This won't happen the way it is described.It is working fine the way it is implemented right now, and has been move, closing.


The bugmagnet does not send events to inputs

Hi there,
very nice job with this addon. Our testers try to use your plugin and they find it very
comfortable.
But we encountred one problem. We cannot use BugMagnet when our textarea or inputs have attached some event listeners in JavaScript. I am talking especially about events like 'paste', 'keyup', 'keydown'. Events seems not to be triggered at all. It means that we cannot use BugMagnet when we have complex UI.
Have you ever thought about adding feature like triggering events? It would be awesome, if a tester would choose from modes like 'paste' / 'key events'.

it sends the input event. keyup/keydown etc are good for testing individual characters, not whole sentences. plus you can get inputs without keydowns (eg paste). so in short, I don't need that, but if it's useful to you, fork and change it.


onThreadStart/onThreadComplete doesn't always match

When running a snapshot of current master in a Netflix canary, I've found that the number of onThreadStart hook calls and onThreadComplete hook calls don't always line up.  I'll be adding unit tests until I figure out the specific execution flow which invokes onThreadStart but not onThreadComplete

Resolved by #531


[BUG] Cluster created with --network=host flag creates kubeconfig with incorrect server parameter

What did you do?
Created cluster with servicelb and traefik disabled. The generated kubeconfig would not connect to the cluster without modification.

How was the cluster created?
k3d create cluster --k3s-server-arg "--disable=servicelb" --k3s-server-arg "--disable=traefik" --update-kubeconfig --no-lb --network=host

What did you do afterwards?

ubuntu@k3d:~$ k3d get cluster
NAME          MASTERS   WORKERS
k3s-default   1         0
ubuntu@k3d:~$ k3d get node
NAME                       ROLE     CLUSTER
k3d-k3s-default-master-0   master   k3s-default
ubuntu@k3d:~$ docker ps
CONTAINER ID        IMAGE                      COMMAND                  CREATED             STATUS              PORTS               NAMES
8200fb61dc90        rancher/k3s:v1.18.3-k3s1   "/bin/k3s server --d…"   3 minutes ago       Up 3 minutes                            k3d-k3s-default-master-0
ubuntu@k3d:~$ kubectl get node
The connection to the server <IP_ADDRESS>:45557 was refused - did you specify the right host or port?

What did you expect to happen?
Kubectl to connect to cluster. It was fixed by changing the port in kubeconfig to 6443. (See below)
Screenshots or terminal output
ubuntu@k3d:~$ cat ~/.kube/config
apiVersion: v1
clusters:
- cluster:
    certificate-authority-data: LS0tLS1CRUdJTiBDRVJUSUZJQ0FURS0tLS0tCk1JSUJWakNCL3FBREFnRUNBZ0VBTUFvR0NDcUdTTTQ5QkFNQ01DTXhJVEFmQmdOVkJBTU1HR3N6Y3kxelpYSjIKWlhJdFkyRkFNVFU1TWpRd016Y3dOREFlRncweU1EQTJNVGN4TkRJeE5EUmFGdzB6TURBMk1UVXhOREl4TkRSYQpNQ014SVRBZkJnTlZCQU1NR0dzemN5MXpaWEoyWlhJdFkyRkFNVFU1TWpRd016Y3dOREJaTUJNR0J5cUdTTTQ5CkFnRUdDQ3FHU000OUF3RUhBMElBQkdidUNYSEtILzRxOTU4Tk5CeG8rS2YyenQ1bVdiQndxVDFkcms2THhqakwKY0toUHFkSEFCNVNKYmo3S1I2Sm1rWlRYTW56d1c0eER0amV1Q2g3QnN2V2pJekFoTUE0R0ExVWREd0VCL3dRRQpBd0lDcERBUEJnTlZIUk1CQWY4RUJUQURBUUgvTUFvR0NDcUdTTTQ5QkFNQ0EwY0FNRVFDSUMrQ3gwUkZ5YTNuCjhGekZtSkJNT0RZZVZPcmgzSzJTNjJyTHd4cjhEZVFLQWlCZjhTK2N5WVk3dVA1WW1wbzE1UFRvM2F4M3RHUjAKOGNzaStCcUFJckdlU2c9PQotLS0tLUVORCBDRVJUSUZJQ0FURS0tLS0tCg==
    server: https://<IP_ADDRESS>:45557
  name: k3d-k3s-default
contexts:
- context:
    cluster: k3d-k3s-default
    user: admin@k3d-k3s-default
  name: k3d-k3s-default
current-context: k3d-k3s-default
kind: Config
preferences: {}
users:
- name: admin@k3d-k3s-default
  user:
    password: e6ca6474c80c5a0ce15f4e2543821e45
    username: admin
ubuntu@k3d:~$ ss -ntlp
State       Recv-Q       Send-Q              Local Address:Port                Peer Address:Port
LISTEN      0            128                     <IP_ADDRESS>:10248                    <IP_ADDRESS>:*
LISTEN      0            128                     <IP_ADDRESS>:10249                    <IP_ADDRESS>:*
LISTEN      0            128                     <IP_ADDRESS>:6444                     <IP_ADDRESS>:*
LISTEN      0            128                     <IP_ADDRESS>:46127                    <IP_ADDRESS>:*
LISTEN      0            128                     <IP_ADDRESS>:10256                    <IP_ADDRESS>:*
LISTEN      0            128                 <IP_ADDRESS>%lo:53                       <IP_ADDRESS>:*
LISTEN      0            128                       <IP_ADDRESS>:22                       <IP_ADDRESS>:*
LISTEN      0            128                     <IP_ADDRESS>:10010                    <IP_ADDRESS>:*
LISTEN      0            128                             *:10250                          *:*
LISTEN      0            128                             *:10251                          *:*
LISTEN      0            128                             *:6443                           *:*
LISTEN      0            128                             *:10252                          *:*
LISTEN      0            128                          [::]:22                          [::]:*
ubuntu@k3d:~$ sed -i 's/45557/6443/' .kube/config
ubuntu@k3d:~$ kubectl get node
NAME                       STATUS   ROLES    AGE     VERSION
k3d-k3s-default-master-0   Ready    master   6m13s   v1.18.3+k3s1

Which OS & Architecture?
Ubuntu 18.04.4 amd64
Which version of k3d?
k3d version v3.0.0-rc.3
k3s version v1.18.3-k3s1 (default)

Which version of docker?

You're absolutely right, the random port-choosing that's behind this issue has no effect in hostNetwork mode. This is "fixed" now. Please note, that still you can only create one cluster in hostNetwork mode, since we're not changing the backend port on the container.


Popup just a command bar

I hope to use a shortcut to open Popup which just contains a command bar -- like mac alfred or spotlight

My hopeful use flow:

open the command bar
type to search
open the selected tab or a new one
archive the opened tabs

Can you clarify what's the meaning of: archive the opened tabs?

suspend and store it for search in the future

Thanks for your clarification. To suspend and store tabs is not the primary goal for this extension. There are ton of other extensions can do this.

There is an implementation in:https://github.com/xcv58/Tab-Manager-v2/blob/master/packages/extension/CHANGELOG.md#features

There is an implementation in:https://github.com/xcv58/Tab-Manager-v2/blob/master/packages/extension/CHANGELOG.md#features


run success, can not visit by web browser

npm install structor -g and npm install its deps.
structor -p 2222 with no error reported.
Then visit from web browser, get the following content:
Cannot GET /

Need more manual and info for setup a new project. Thanks.

@vanloswang

Beginning from version 2 Structor should be installed as a local npm module into react-boilerplate project.

As far as Structor uses a custom metadata in order to operate the project's components, there is a fork of react-boilerplate project genuinely prepared for Structor.

Please find the instruction of how to install and start working with Structor in compatible fork of the react-boilerplate project here: https://github.com/ipselon/react-boilerplate;

@ipselon I have react-boilerplate example app runs ok. but I did not get informatios about how to access structor ui, so how can I use it?

@vanloswang

npm run structor
go to localhost:2222/structor
compose component
use one of the available generators to create React Component
import and use it in your project code

@ipselon @Dmytro-Medzatiy Run 'npm run structor' failed on both Windows 10 and CentOS 7, The error info on CentOS 7 is as follow:
npm run structor
npm ERR! Linux 3.10.0-327.28.2.el7.x86_64
npm ERR! argv "/root/.nvm/versions/node/v6.5.0/bin/node" "/root/.nvm/versions/node/v6.5.0/bin/npm" "run" "structor"
npm ERR! node v6.5.0
npm ERR! npm  v3.10.3

npm ERR! missing script: structor
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     /home/ws/webdev/react-boilerplate-3.1.0/npm-debug.log

Full informations in npm-debug.log is as following:
cat npm-debug.log
0 info it worked if it ends with ok
1 verbose cli [ '/root/.nvm/versions/node/v6.5.0/bin/node',
1 verbose cli   '/root/.nvm/versions/node/v6.5.0/bin/npm',
1 verbose cli   'run',
1 verbose cli   'structor' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose stack Error: missing script: structor
4 verbose stack     at run (/root/.nvm/versions/node/v6.5.0/lib/node_modules/npm/lib/run-script.js:151:19)
4 verbose stack     at /root/.nvm/versions/node/v6.5.0/lib/node_modules/npm/lib/run-script.js:61:5
4 verbose stack     at /root/.nvm/versions/node/v6.5.0/lib/node_modules/npm/node_modules/read-package-json/read-json.js:356:5
4 verbose stack     at checkBinReferences_ (/root/.nvm/versions/node/v6.5.0/lib/node_modules/npm/node_modules/read-package-json/read-json.js:320:45)
4 verbose stack     at final (/root/.nvm/versions/node/v6.5.0/lib/node_modules/npm/node_modules/read-package-json/read-json.js:354:3)
4 verbose stack     at then (/root/.nvm/versions/node/v6.5.0/lib/node_modules/npm/node_modules/read-package-json/read-json.js:124:5)
4 verbose stack     at /root/.nvm/versions/node/v6.5.0/lib/node_modules/npm/node_modules/read-package-json/read-json.js:311:12
4 verbose stack     at /root/.nvm/versions/node/v6.5.0/lib/node_modules/npm/node_modules/graceful-fs/graceful-fs.js:78:16
4 verbose stack     at tryToString (fs.js:455:3)
4 verbose stack     at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:442:12)
5 verbose cwd /home/ws/webdev/react-boilerplate-3.1.0
6 error Linux 3.10.0-327.28.2.el7.x86_64
7 error argv "/root/.nvm/versions/node/v6.5.0/bin/node" "/root/.nvm/versions/node/v6.5.0/bin/npm" "run" "structor"
8 error node v6.5.0
9 error npm  v3.10.3
10 error missing script: structor
11 error If you need help, you may report this error at:
11 error     <https://github.com/npm/npm/issues>
12 verbose exit [ 1, true ]

structor is installed by npm, Why there is no script: structor?

@vanlosvang are you sure you have cloned a fork of the React Boilerplate? https://github.com/ipselon/react-boilerplate

@ipselon setup react-boilerplate by https://github.com/ipselon/react-boilerplate/archive/v3.1.0.zip

@vanloswang
Actually, there are only releases of main repo (not forks) in the releases list because I did'n make any releases of my fork so far. Consequently, you have to download an archive file with master branch of the fork: https://github.com/ipselon/react-boilerplate/archive/master.zip

@ipselon Thanks,The master branch runs ok.

@ipselon i have cloned master branch but still given error structor script missing


Routing zone error with ngrx usage

From @pnahtanoj on September 11, 2018 16:20

Did you verify this is a real problem by searching the [NativeScript Forum]
Indeed.  I did look for a dupe, as well.
Tell us about the problem
When routing to a previously visited page (with and ngrx subscription), I'm getting a zone error
Which platform(s) does your issue occur on?
iOS
Please provide the following version numbers that your issue occurs with:

CLI: (run tns --version to fetch it)

$ tns --version
4.1.2

Cross-platform modules: (check the 'version' attribute in the
node_modules/tns-core-modules/package.json file in your project)

"tns-core-modules": "~4.2.0",

Runtime(s): (look for the "tns-android" and "tns-ios" properties in the
package.json file of your project)

"tns-ios": {
"version": "4.1.1"
}

Plugin(s): (look for the version number in the package.json file of your
project)

Please tell us how to recreate the issue in as much detail as possible.
I had issues getting a ngrx integrated into playground, so I created a small application here:
https://github.com/pnahtanoj/routing-error-example
You'll see the error if you got to Page1, then click Add One!
Is there code involved? If so, please share the minimal amount of code needed to recreate the problem.
See previous item
Copied from original issue: NativeScript/NativeScript#6257

Is there a reason this moved?

@pnahtanoj the issue is related to Angular routing so this is the appropriate repository to log it.

@pnahtanoj the error is non-blocking and the items are added as expected on my side. It also looks like an error related to the Angular logic
Error: ExpressionChangedAfterItHasBeenCheckedError: Expression has changed after it was checked.

Previous value: 'items: [object Object]'. Current value: 'items: [object Object],[object Object],[object Object],[object Object]

Refer to this blog post for informati9on on how you could debug and find what is causing the issue
Baically in your case, you need to use the delay operator when piping
    this.items$ = this.store.select(selectItems)
      .pipe(
        delay(0), // HERE
        tap(_ => {
          console.log('HOME SELECT: ', _)
        }));

The examples in the post relate to sourcing data in AfterViewInit, which my example is not doing.  The fetch call is in OnInit, which is the ultimate solution in that blog post.
I've used this pattern in a number of vanilla ng apps without having this type of timing issue, so I don't think we can call it an error with Angular logic.

Something else to consider.  I updated the project to use the service directly (bypassing the state selector) for data sourcing here:
https://github.com/pnahtanoj/routing-error-example/commit/dcda9aa92d5d8118fd6d8c0037322d7c97ef68d6
This works without the delay workaround.  Which leads me to believe that in the native context, ngrx selectors are adding enough overhead to create the timing issues, though I don't really understand why that would be the case.

Similary, moving the fetch into the constructor works:
https://github.com/pnahtanoj/routing-error-example/commit/7ec288bcbacc89c923635ff737b983c44773c41a
Which again seems to belie a timing issue.


Dribbble implementation

I've implemented Dribbble web provider.
If you have any suggestions how to improve the code feel free to do that!

I'll love to see it in master


Empty .net Maui deployed as msix package cannot be uninstalled with error 0x80073cfa

Describe the bug
I use a laptop for development:
Device name	DESKTOP-IG7JGPU
Processor	Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz   2.59 GHz
Installed RAM	16,0 GB
System type	64-bit operating system, x64-based processor
Pen and touch	No pen or touch input is available for this display
with Win10 Pro:
Edition	Windows 10 Pro
Version	22H2
Installed on	4/11/2024
OS build	19045.5131
Experience	Windows Feature Experience Pack 1000.19060.1000.0
Visual Studio:
Microsoft Visual Studio Community 2022
Version 17.12.0
VisualStudio.17.Release/17.12.0+35506.116
Microsoft .NET Framework
Version 4.8.09037
Xamarin   <IP_ADDRESS> (d17-12@1d0906d)
Visual Studio extension to enable development for Xamarin.iOS and Xamarin.Android.
Xamarin Designer   <IP_ADDRESS> (remotes/origin/main@0912a005b9)
Visual Studio extension to enable Xamarin Designer tools in Visual Studio.
Xamarin.Android SDK   <IP_ADDRESS> (d17-5/45b0e14)
Xamarin.Android Reference Assemblies and MSBuild support.
Mono: d9a6e87
Java.Interop: xamarin/java.interop/d17-5@149d70fe
SQLite: xamarin/sqlite/3.40.1@68c69d8
Xamarin.Android Tools: xamarin/xamarin-android-tools/d17-5@ca1552d
The first time the MSIX package is deployed, but then it cannot be uninstalled and no other attempt to debug works.
The MauiApp1 exits with code 0x80073cf9
Steps to reproduce the bug

Create a .net Maui Empty project (standard or Blazor hybrid)
make sure that packaged deployment is enabled (in my installation this is the default)
Debug the app on Windows Machine.
Exit the app
Try to debug again.
The app will not run and exits with error 0x80073cf9
If I try to uninstall the previous version from "Add & Remove Apps", I get error 0x80073cfa

Expected behavior
No response
Screenshots
No response
NuGet package version
None
Packaging type
No response
Windows version
Windows 10 version 22H2 (19045, 2022 Update)
IDE
Visual Studio 2022
Additional context
No response

This issue is a duplicate of #4884.

I do not understand how is this duplicate to #4884.
I am stuck with 3 installed apps that do not go away.
The good thing is that I tested the .net9 blazor maui and this works ok.

The duplicate reference should have been to #4881.


Add description to CakePHP and Laravel entries

Related to issue #928

Needs to follow the existing format.

Do you mean that the desciptions needed a hyphen at the beginning? If not, could you explain me what is the existing format?

Just a quick question on how GitHub works. This is the first time I update a PR and I tried to do it following option 1 of this write up: https://github.com/RichardLitt/knowledge/blob/master/amending-a-commit-guide.md. When I pushed upstream I got this messages:
remote: Permission to sindresorhus/awesome.git denied to inventinto.
fatal: unable to access 'https://github.com/sindresorhus/awesome.git/': The requested URL returned error: 403
The commit I made end up in the PR anyway. I'm not really sure why. Why did I get this messages? Is this the correct way to update a PR? I'm also not sure if this is the right place to ask this question  :grimacing:

Do you mean that the desciptions needed a hyphen at the beginning?

Yes

This is the first time I update a PR and I tried to do it following option 1 of this write up: RichardLitt/knowledge:amending-a-commit-guide.md@master. When I pushed upstream I got this messages:

You need to push to the remote branch in your fork, not this repo. You get message since you don't have access to this repo.

You had a merge conflict that I fixed for you.
There are some good guides here: https://guides.github.com/activities/forking/

Thank you!


[ota-provider-app] fix location parsing and BDX sender

Problem
This PR contains necessary changes for completing an OTA BDX transfer with an ota-requestor-app (which will also require changes submitted in another PR).

file I/O for BDX sending was erroring out on EOF instead of sending the last block
code that verified QueryImage parameters was using emberAfStringLength() which is incompatible with CHIP TLV format
The BDX ExchangeContext was closing after every message send
Fixes #9703

Change overview

don't consider EOF as an error condition when reading blocks for BDX transfer
don't keep calling gcount() to get the number of bytes read
add kExpectResponse for all BDX messages except StatusReport in order to keep exchange open
use strlen() to parse location string instead of emberAfStringLength()
fix message for printing location string length

Testing

tested again modified ota-requestor-app (PR will be submitted soon)

@saurabhst @Damian-Nordic @LuDuda ?


curly

Bug Report
feature request
TypeScript code being linted
for(let i = 0; i < 50; i++)
{
dosomething(i);
}
Actual behavior
tslint complains about my curly.  I prefer to code in this way.
Expected behavior
I would like it not to complain about this.;  It wants the curly brace after the ) which to me is bad form.

This report is pretty old and so I'm going to close it. It should be possible to disable the one-line rule which enforces this behaviour.


Not able to fetch data from introspection query with latest version

Prerequisites

[ ] I have checked the existing issues to ensure this is not a duplicate.
[ ] I am running the latest version of Tailcall.

Describe the bug
getting empty data response for introspection query with latest version  v0.114.2.
Steps to reproduce
Started local server and hit the following query via postman
curl --location --request POST 'http://<IP_ADDRESS>:9999/graphql' \
--header 'Content-Type: application/json' \
--data-raw '{"query":"query  _service { _service { sdl } }","variables":{}}'

got following response
{
    "data": {}
}

Expected behavior
Should return response for the introspection query
Actual behavior
A description of what actually happened.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment information:

Operating System: (e.g., Ubuntu 20.04, macOS Big Sur, Windows 10)
Tailcall Version: (e.g., v1.0.2)
Rust Version: (e.g., rustc 1.58.0)
GraphQL Client/version (e.g., curl/8.1.2)

Additional context
Add any other context about the problem here, such as relevant logs, error messages, or stack traces.
Logs
Paste any relevant logs here.

@vibhor-d11 can you try adding the following to the schema and check?
type Greetings @expr(body: {greet: "Hello World!"}) {
  greet: String
}

The reason this doesn't work is because you might not have any entity defined in your schema. We can add a flag to support subgraph capabilities explicitly. Let me know what you think.

what is meant by entity i didn't get? i do have a query in schema which resolves a endpoint with a defined schema like:
schema
  @server(port: 9999, hostname: "<IP_ADDRESS>")
  @link(type: Operation, src: "health.graphql")
  @upstream(
    baseURL: "base-url"
    allowedHeaders: ["Authorization"]
  ) {
  query: Query
}

type Inventory {
  logo: String!
  campaignId: Int!
  viewAllUrl: String!
  brandName: String!
}

type Data {
  data: [Inventory]
}

type HealthCheckResponse {
  status: String!
}

type Query @addField(name: "inventory", path: ["inventory", "data"]) {
  inventory(realEstateName: String!, contextValue: String!): Data
    @http(
      path: "/inventory"
      query: [
        {
          key: "realEstateName"
          value: "{{.args.realEstateName}}"
          skipEmpty: false
        }
        {key: "contextValue", value: "{{.args.contextValue}}", skipEmpty: false}
      ]
    )
    @omit
  health: HealthCheckResponse @expr(body: {status: "Success"})
}

Now this query runs on a subgraph, and a different service query the subgraph for introspection query to register it with its schema.

@vibhor-d11 Thanks for creating an issue.
We fixed the federation introspection call in latest versions. Please update the tailcall and add enableFederation flag to your subgraph in order to be able to call the federation instrospection


Implement input menu acceptance

Now that the menu bas has been created, now we need to make sure the when the user input a letter corresponding to one of the choices the file opens.

Work on issue will resume after a restructuring of the code has been completed.


Update instructions for using design tokens

Duplicate of user-reported issue:
https://github.com/cds-snc/gcds-docs/issues/117
✅ Definition of Done / Outcomes

[x] Done when README.md is updated with instructions on how to use a design token
[ ] Done when a french and english version of the instructions exist

📜 Acceptance criteria
Add here the list of assumed steps that will be needed to complete the task

[x] Instructions for when users already use either our components or utility framework
[x] Instructions for using the design tokens standalone

📄 Google docs version
Sent off for translation through #traduction-gcds on June 27th

Update:
Translations have been received and the README has now been updated to include the translations here: https://github.com/cds-snc/gcds-tokens/pull/279
Still waiting for an example code to include

Added example code and resent the new text for translation


Getting into the admin brought up an error

While triying to get into the admin i got this error
Running the command on the console:
Delayed::Backend::ActiveRecord::Job.possibly_active
NoMethodError: undefined method `possibly_active' for #Class:0x007fe4748a88c8
Seems like the models/delayed_job.rb file is not being loaded for some reason.

Hi chivoX - I'm not sure what would cause that. I haven't seen that in any of our deployments. Again, I'm not sure the details of the environment you are running the code in, but please feel free to email me to talk it over. I'd email you but I don't see an email listed on your GitHub account.
So for now I'll close this because I cannot reproduce it. But if you email me we can talk more about your specific deployment of this codebase


Gobo manager

At the moment, a capability can have an image attribute that is just a string which is valid for QLC+.
It should be possible that a user can upload a new gobo image that will be delivered together with a referencing fixture. The image attribute then only refers to our gobo library and is adapted for each plugin.

Addition: A fixture should not define gobo images in the capabilities but declare the gobowheel globally in the fixture so the capabilities only have to define the gobo index. This makes it reusable if and index is used twice (e.g. shake) and provides more opportunities for specific plugin gobo features.
Sent from my Bq Aquaris X5 Plus using FastHub

We should also consider that there may not be only Gobo Wheels and Color Wheels, but also others (like the Effect Wheel in Robe's ColorSpot 2500E AT).

Suggested syntax:
{
  // fixture root
  "wheels": {
    "Name of a color wheel": {
      "direction": "CW", // default
      "slots": [
        {
          "slotType": "Open"
        },
        {
          "slotType": "Color",
          "name": "Red",
          "colors": ["#ff0000"],
          "colorTemperature": "5600K" // optional
        },
        {
          "slotType": "Color",
          "resource": "lee/129"
          // no name or colors needed, since that is part of the resource
        },
        {
          "slotType": "Color",
          "resource": "rosco/271"
        }
        // ...
      ]
    },
    "Some Gobo Wheel": {
      "direction": "CCW",
      "slots": [
        {
          "slotType": "Gobo",
          "name": "Shooting Stars",
          "replaceable": true,
          "rotatable": true
        },
        {
          "slotType": "Gobo",
          "resource": "robe/1289"
        },
        // ...
      ]
    },
    "A Cool Effect Wheel": {
      "slots": [
        {
          "slotType": "Prism",

          // at least one of the following has to be set
          "name": "3-facet",
          "facets": 3,

          "rotatable": true
        },
        {
          "slotType": "Iris",
          "openPercent": 25
        },
        {
          "slotType": "Frost",
          "frostIntensity": 25
        }
      ]
    }
  },

  // ...

  "availableChannels": {
    "Color Wheel": {
      "capabilities": [
        {
          "dmxRange": [0, 20],
          "type": "WheelSlotIndex",
          "wheel": "Name of a color wheel",
          "index": 0
        },
        {
          "dmxRange": [21, 40],
          "type": "WheelSlotIndex",
          "wheel": "Name of a color wheel",
          "index": 1
        },
        // ...
        {
          "dmxRange": [200, 255],
          "type": "WheelRotation",
          "wheel": "Name of a color wheel",
          "speed": "fast CW" // or angle
        }
      ]
    },
    "Gobo Wheel": {
      "capabilities": [
        {
          "dmxRange": [0, 20],
          "type": "Nothing" // Wheel not moved into light beam
        },
        {
          "dmxRange": [21, 40],
          "type": "WheelSlotIndex",
          "wheel": "Some Gobo Wheel",
          "index": 0
        },
        {
          "dmxRange": [21, 40],
          "type": "WheelSlotIndex",
          "wheel": "Some Gobo Wheel",
          "index": 0,
          "shakeAngle": "wide"
        },
        // ...
      ]
    }

    // Regular Prism / Iris / Frost capabilities can still be proportional
  }
}


Add ability to label samples with metadata (e.g. Pangolin designation)

Actually, country would also be useful here, right?


PipsPager: sometimes internal scrollviewer makes external scrollviewers scroll

Describe the bug

Sometimes when moving from one pip to another (by clicking on it directly or using the navigation arrows), external scroll viewers on the page will scroll as well.
Steps to reproduce the bug

Steps to reproduce the behavior:

Create a Vertical Pips Pager inside a scroll viewer.
Choose the last Pip.
Observe the outside scroll viewer scrolling as well.

Expected behavior

External scroll viewers shouldn't scroll. It happens because the inner scroll was not "complete" and the bring into view event is bubbled up causing the external scroll viewers to scroll as well. The way to fix it is to handle the bringintoview event on the internal scroll viewer.
Screenshots

Version Info

NuGet package version:
[Microsoft.UI.Xaml 2.6 prerelease]

Device form factor
Saw the problem?

Desktop
Yes

@regis fyi


Layout/SpaceBeforeFirstArg is not found until after "some code"

When running rubocop, it seems to miss Layout/SpaceBeforeFirstArg on the first couple of lines. Only after some other code has been analyzed, it will detect any other Layout/SpaceBeforeFirstArg
Expected behavior
There should have been three errors in total. It is missing two more errors:
db/seeds/roles.seeds.rb:3:18: C: Layout/SpaceBeforeFirstArg: Put one space between the method name and the first argument.
db/seeds/roles.seeds.rb:4:18: C: Layout/SpaceBeforeFirstArg: Put one space between the method name and the first argument.

Actual behavior
Rubocop output:
$ rubocop db/seeds
Inspecting 5 files
C....

Offenses:

db/seeds/roles.seeds.rb:8:18: C: Layout/SpaceBeforeFirstArg: Put one space between the method name and the first argument.
Rails.logger.info"User Roles... Completed\n"

Steps to reproduce the problem
Example file: roles.seeds.rb
# frozen_string_literal: true

Rails.logger.info"\nUser Roles..."
Rails.logger.info"--------------\n"

user_roles = %w[admin user]

Rails.logger.info"User Roles... Completed\n"

user_roles.each do |name|
  Role.find_or_create_by(name: name)
end

Rails.logger.info "User Roles... Completed\n"

$ rubocop db/seeds
RuboCop version
$ rubocop -v
0.68.1

Your version of rubocop seems quite old (almost 5 years), can you upgrade any further? I ran your example on the most recent version and am getting the 3 expected offenses so it has already been fixed somewhere inbetween.

This issue has been resolved in new versions. Please upgrade to the latest RuboCop. Thank you.

Thank you!


added default value to pop function

Intended as fix to issue #313

I've had the same issue with Dark (pop on line 57 of that same file). @fanoway maybe you could also change that in this PR?
@blankjul: Do you have any concerns about these changes?


Short explanation of the issue goes here

customSceneBindings not working properly.
Hotkeys are not getting recognized (F7 to F9 wont work for example)

Is the game focused

Okay I have found out that the reason was an invalid camera settings json file.
Bug can be closed


Limit memory and CPU usage

I am working in a sandbox and would like to know if it is possible to restrict the use of RAM and CPU with wasmtime.

I'd recommend exploring the documentation for this module. Store.set_limits for example can limit memory while you can choose either Config.consume_fuel or Config.epoch_interruption for limiting CPU.

Escreva em ingles: Eu fiz, e gostaria de saber porque o uso de CPU fica em 99% ao configurar limite de memory ram e fuel num codigo que tem um sleep.
Is it a bug or a known issue?
print(
    run(
        """
#include <unistd.h>

int main() {
  sleep(30);
  return 0;
}
"""
    )
)

def run(source: str) -> str:
    with TemporaryDirectory() as path:
        os.chdir(path)

        with open("main.cpp", "w+t") as main:
            main.write(source)
            main.flush()

            try:
                result = subprocess.run(
                    [
                        "emcc",
                        "-O3",
                        "-flto",
                        "-s",
                        "ENVIRONMENT=node",
                        "-s",
                        "WASM=1",
                        "-s",
                        "PURE_WASI=1",
                        "main.cpp",
                    ],
                    capture_output=True,
                    text=True,
                    timeout=300,
                )

                if result.returncode != 0:
                    return result.stderr

                with open("a.out.wasm", "rb") as binary:
                    wasi = WasiConfig()
                    wasi.stdout_file = "a.out.stdout"
                    wasi.stderr_file = "a.out.stderr"

                    config = Config()
                    config.consume_fuel = True
                    engine = Engine(config)
                    store = Store(engine)
                    store.set_wasi(wasi)
                    store.set_limits(memory_size=16 * 1024 * 1024)
                    store.set_fuel(10_000_000_000)

                    linker = Linker(engine)
                    linker.define_wasi()
                    module = Module(store.engine, binary.read())
                    instance = linker.instantiate(store, module)
                    start = instance.exports(store)["_start"]
                    assert isinstance(start, Func)

                    try:
                        start(store)
                    except ExitTrap as e:
                        if e.code != 0:
                            with open("a.out.stderr", "rt") as stderr:
                                return stderr.read()

                    with open("a.out.stdout", "rt") as stdout:
                        return stdout.read()
            except subprocess.TimeoutExpired:
                return "⏰😮💨"
            except subprocess.CalledProcessError as e:
                return e.stderr
            except Exception as e:  # noqa
                return str(e)

Oh sleep isn't implemented with a loop, no, but there are limitations to fuel/epochs. Namely host APIs cannot be preempted. Host APIs include sleep which bottoms out in a WASI API that blocks the current thread. That can't be interrupted. To get interruptible WASI that requires using the async support that Wasmtime provides which isn't bound in Python yet.

Ah, I see, that makes sense. And thank you very much for the detailed explanation.


Class String has no Print method

Just trying out this language and i have immediately got this error message
🔴🔡 has no method 😃
Which to me reads as the same as the issue title

Could you please provide your code?

🐇 🐼 🍇 👴 Create class called 🐼
🐇🐖 🏁 ➡ 🚂 🍇
👴 😃 🔤Hello World🔤
🍎 0
🍉
🍉

Is this the code that produces this error message? Line 3 is commented out and compiles correctly for me.

You are using 😃 instead of 😀.

Shows up as :D on mine haha I will change it and check again


DataFusion 37.0.0 upgrade

What is the problem the feature request solves?
Prepare Comet to DataFusion 37.0.0 upgrade
Describe the potential solution
No response
Additional context
Depends on https://github.com/apache/arrow-datafusion/pull/9697

Related to https://github.com/apache/arrow-datafusion-comet/pull/239/files

Yea, as #239 is going to use specified arrow-rs and DataFusion branch to work around a Java Arrow bug, we can hold on upgrading these dependencies.
The fix of the bug is merged into Java Arrow. Once it is released and we upgrade it in Comet, we can change back to normal released arrow-rs and DataFusion releases. Thanks.

Comet is now using 39.0.0-rc1 (and will upgrade to 39.0.0 when it is available, probably early next week) so I think we can close this issue now


Make withCanvas into monadic modifier

Make withCanvas into monadic modifier as it might simplify scripts sometimes... not 100% sure though, it's a consideration

I decided to close this as "Cancelled" because i decided the current system is good enough. Hope i get a good idea later on reworking withcanvas to be less special and needing an @ all the time for activating since it's a modifier and not a function. but it being a modifier is very useful for short examples


Date of all items is off by a month

The date of all the entries is off by a month.  Hopefully an easy fix!

@wildskyf Any thoughts on this?

@Gitoffthelawn I don't get your point. :/
Can you provide some examples or more-detailed description?
Thank you for your patient... I am really busy recently...

If today is 2017/06/08, and the extension saves an entry, it will save it with a date of 2017/5/8.
Every entry is off by exactly one month.

Oh... I got you !!!
 Orz
It will be fixed in the next version. !!!

LOL :)


removed four space indentation

removed four space indentation from all the code blocks

Please let me know if changes needed. I will update them asap.

Please wait for Ivan to confirm
On Sat, 18 May 2019 at 18:36, Himanshu Nailwal<EMAIL_ADDRESS>wrote:

@ryokugyu commented on this pull request.
In static/docs/commands-reference/commit.md
https://github.com/iterative/dvc.org/pull/351#discussion_r285341791:

@@ -142,8 +142,8 @@ make is adjusting a parameter to CountVectorizer in that script. Namely,
adjusting the max_features option in this line changes the resulting model:

-    bag_of_words = CountVectorizer(stop_words='english',
-                max_features=6000, ngram_range=(1, 2))
+bag_of_words = CountVectorizer(stop_words='english',

Then, I can close this pull request, and make a new pull only for DVC
blocks?

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/iterative/dvc.org/pull/351?email_source=notifications&email_token=AF3UDJXXP7DCAOQ4LMYFAXTPV75N5A5CNFSM4HNZWNFKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOBZA55VY#discussion_r285341791>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AF3UDJUZW6ADD6PNTHDEPBTPV75N5ANCNFSM4HNZWNFA>
.

--
[image: photo]
Kurian Benoy
+91<PHONE_NUMBER> <+91+9400125402> |<EMAIL_ADDRESS>https://kurianbenoy.github.io/
http://us.linkedin.com/in/kurian-benoy-75642b120
http://github.com/kurianbenoy http://twitter.com/kurianbenoy2
http://medium.com/@kurianbenoy1998
Create your own email signature
https://www.wisestamp.com/signature-in-email?utm_source=promotion&utm_medium=signature&utm_campaign=create_your_own

@ryokugyu @kurianbenoy thanks guys, taking a look. It's applicable to all code blocks - python, vim, yaml, etc, not dvc only.

please, joint the chat dvc.org/chat and ping me directly @ivan . There is some miscommunication happening. The previous version was almost perfect with a few small issues.


feat: add new SVGs for UXAnaRangel and UXCorpRangel

This pull request includes changes to the src/data/svgs.ts file to add new SVG entries. The most important changes involve adding entries for UXAnaRangel and UXCorpRangel.
New SVG entries added:

Added UXAnaRangel to the SVG list with a route to '/library/uxanarangel.svg' and a URL to 'https://uxanarangel.com/'.
Added UXCorpRangel to the SVG list with routes for both light and dark versions and a URL to 'https://github.com/UXCorpRangel/'.

Thanks 🚀✨


python3Packages.rasterio: fix execution of Python test suite

Description of changes
Remove installCheckPhase which was preventing rasterio Python test suite to run.

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@sikmir , thank you very much for quick review.

Thank you @sikmir