For Xamarin Forms?

Hi, I am trying to use your library in a Xamarin forms application but it does not allow me to add it to a PCL project.
There is no way to use it in this environment?
Thank you

There shouldn't be any issues with adding it to a PCL. Can you share the errors Xamarin is throwing please?

hi, this error:
Could not install package 'RSG.Promise 1.2.2'. You are trying to install this package into a project that targets '.NETPortable,Version=v4.5,Profile=Profile111', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.

RSG.Promises is built to work with Unity projects, which are limited to .NET 3.5. It won't build into a project that is higher than that. Yours is set to .NET 4.5

The code should work with with .NET 4.5, just not the existing projects. Feel free to recreate the project yourself and then you should be able to use it.

of course it should. What am I talking about? Shouldn't answer questions first thing in the morning after no sleep. Sorry lolohouse.

I have created a PCL library for .net framework 5.5 (profile111) and compiled it perfectly. I just had to change the ApplicationException class by Exception and everything perfect.
You could create a PCL .net 4.5 library and link classes, so it would be compatible with xamarin
Thank you and forgive my English which is not very good

The latest version of the library no longer uses ApplicationException anywhere so the source should be fully compatible now.
If you'd like to set the project up to also build a library compatible with Xamarin forms we'd be happy to merge it and include that in future releases on Nuget, provided that it doesn't intefere with the .NET 3.5 build.


[Ready-To-Review][Add] in-line hints

Other features:

inside line hints
customizable hints style
hints from completer

Todo:

hints from history

Example:

Nice!

Before merge, i want the inside-line hints to be optional

Ya, i'd also like the hints to be dark gray/configurable but we can change that sometime in the future.

Ya, i'd also like the hints to be dark gray/configurable but we can change that sometime in the future.

For the configurable style it's already implemented

Before merge, i want the inside-line hints to be optional

Done.

@mzanrosso - taking a look now.
For the history-based hints, I was testing it by typing something like:
> another test
> a

For that second line, I should see a hint for another test, right?

@mzanrosso - taking a look now.
For the history-based hints, I was testing it by typing something like:
> another test
> a

For that second line, I should see a hint for another test, right?

@jntrnr This is not my implementation fault; this is because the history file is updated only when you exit.

@mzanrosso - if I exit and restart, I still don't get a hint for the history

@mzanrosso - if I exit and restart, I still don't get a hint for the history

@jntrnr Sorry if it wasn't clear but you cannot enable completer and history hints at the same time

@mzanrosso - if I switch to using the history, are there other settings I need to change so that I can complete the hint?  I tried switching, but if I hit tab, it completes the completion but not the history hint.

@mzanrosso - if I switch to using the history, are there other settings I need to change so that I can complete the hint? I tried switching, but if I hit tab, it completes the completion but not the history hint.

@jntrnr I also noticed this when I did my implementation test; this is because I added the history support for the DefaultHinter and not for the DefaultTabHandler; this could be a next feature for the tab handler

With fish or the appropriate zsh plugin it is typically right arrow or control right arrow to complete the hint at the end of the line either fully or just the next token. This would use a separate keybinding and also restrict greyed out history hints to the end of the line during entry. Depends on what we want to achieve.

@sholderbach / @mzanrosso - sounds good, we can add just the hint, and then go back and add additional commands for doing the completions.

@jntrnr I just noticed that if I use .with_history(history.clone())? it doesn't save the history in the file, do you know why this happens ?

In the current setup flushing to disk occurs on dropping the instance.

In the current setup flushing to disk occurs on dropping the instance.

Why don't insert an option to flush every new command ?

We could flush() on every append() but as soon as the file would be at the chosen capacity and you could not use the append function of the file system anymore, you would have to write the whole file on every append(). At least with the current impl.

@sholderbach I just noticed that the problem that block us occur only when I declare two Box::new(FileBackedHistory::with_file(5, "history.txt".into())?)

Maintaining consistent views within our process via the FS doesn't seem right. Why would you want to use multiple history instances?

We could set it up so that you have one History instance that's protected with locks.

Fixed.

@jntrnr , @sholderbach let me know if we are ready to merge or if I have to modify something else; before merge I need 1sec to squash as usual.

We could set it up so that you have one History instance that's protected with locks.

If I didn't loose track of potential consumers we basically would only have to hand out immutable views/ or iterators to the components like history browsing/search/the completion engine. Only upon finishing the read_line function successfully we would need the mutable access to the data structure (Or if we regularly poll for changes from other processes running reedlines) The history browsing and search are currently mutually exclusive thus them being able to be handled with mutable access to a single struct.

@jntrnr , @sholderbach let me know if we are ready to merge or if I have to modify something else; before merge I need 1sec to squash as usual.

We typically merge as squash commits, so you don't have to explicitly squash on your side.

@jntrnr , @sholderbach let me know if we are ready to merge or if I have to modify something else; before merge I need 1sec to squash as usual.

We typically merge as squash commits, so you don't have to explicitly squash on your side.

oh! this feature is new for me, I always used bitbucket and I never seen it there. let me search how It works.

It has it's downsides as git branch --no-merged won't recognize those merges

I reworded some commit messages, nothing important here.

Agreed, let's do this in stages. For the hints themselves, this looks good.


Convert memory swap values

This converts  cgroup v1 memory swap values to cgroup v2.

Codecov Report

Merging #285 (cf47693) into main (c247c90) will increase coverage by 0.10%.
The diff coverage is 93.54%.

@@            Coverage Diff             @@
##             main     #285      +/-   ##
==========================================
+ Coverage   69.62%   69.73%   +0.10%
==========================================
  Files          46       46
  Lines        5660     5687      +27
==========================================
+ Hits         3941     3966      +25
- Misses       1719     1721       +2


[AB#38228] Add subscribe to subscription plan scenario

Pull Request
Prerequisites

[x] My code follows the coding conventions
[x] All tests pass

Description
This PR adds subscribe to subscription plan scenario to FE Samples.

LGTM! - we can merge once the latest lib version is used


Questions regarding config search

We want to include model analyzer as part of our model deployment pipeline. We want to run a search on the model and get the best configuration automatically as part of the rollout. After experimenting with the tool, we would like to raise a few points regarding the config search functionality:

We have noticed that when we try to run a search on a model that has already been configured, the search immediately stops, we only manage to trigger a search when we strip the instance group and the dynamic batching, and set max batch size to 1. Is this by design or is it a bug? It does not throw an error when it happens, model-analyzer simply creates the configurations and says its done running the search. This happens in both quick and brute searches.
For security reasons and because we run the pipeline on k8s, we have to run model analyzer in remote mode. We have noticed that when we run the triton-sdk pod without assigning it a gpu, model-analyzer can only offer configurations that use instance groups of type kind_cpu. I get it for local, docker and c_api modes, but why for remote as well?
We have noticed that the way you configure dynamic batching only allows for usage of default configuration when running a quick search, while in brute it is a bit more advanced,  with the added control of the max_queue_delay_microseconds parameter. Are there any plans to add more advanced configuration for dynamic search in quick search in the future? It could really help us.
We would also like to know about support for the max_queue_size parameter in dynamic batching: as the instance count increases, it makes sense to increase the queue size accordingly. Do you plan to support this parameter in both quick and brute search, or at least calculate it based on the throughput?
We know quick search is faster and the configuration space is smaller in quick search, but is it possible for you to add support for multi model brute force searches? We ask for it because we want the finer control that brute force allows for, and we do not mind waiting for the best results.
We’d really love the product and would appreciate it if you could answer these question:)

I need to investigate further. Is this for quick search? You are saying that if you start with a model configuration with instance_count and/or max_batch_size not equal to 1, that the search doesn't happen?

This sounds like a bug. I will investigate

If you set max_queue_delay_microseconds in the default config, that value should persist through all created configurations. Are you saying that isn't happening? Or are you asking if that value can be 'searched' as well?

We don't currently support it in quick search. You should be able to specify a list of possible values if you wanted to sweep it in brute, although I haven't verified that it works. I've created a ticket to investigate adding support for searching or intelligently setting max_queue_size.

Our observations was that multi-model brute search could take weeks or even months to finish. It is something we could add support for, but we don't want people accidentally using it. Please confirm if this is something you really want.

This happens for both quick and brute force searches. What you have described is exactly whats happens, when max_batch_size is not configured to 1, model analyzer immediately says "done with config search" and continues to profile the model.
Ok, thanks for that.
The max_queue_delay_microseconds parameter works as you have described, I'm asking for the addition of max_queue_delay_microseconds parameter to quick search. From what we have seen, the only thing configured by model analyzer is whether to enable or disable it, but it is not enough a lot of the time. If this configurable in brute force, we would also expect it to be the same in quick search.
We tried defining max_queue_size in our config and we got an error sayin our configuration was invalid. We have also not found a single appearance of the parameter in your code or your documentation, so we are unsure if it is supported at all.
What exactly bloats this number so much? Is it simply the configuration space being too big? From what we understand, the configuration process for multi model deployments comes down to a zero sum game between all the model configs, which boils down to the product of all the underlying models configs. The max amount of models we want to run this on is around 12, is this too much for it to handle? Maybe the way we are going around configuring these tritons is wrong? If the other features listed above can be implemented for quick search, maybe this will become irrelevant.

In the meantime, we found a new weird phenomenon we’d like to discuss: when letting model analyzer go to an k8s ingress route, where two tritons pods are deployed, after a short period model analyzer crashes and says that perf analyzer failed to find the requested model version. But running regular perf analyzer in the same triton results in no error. From this we have deduced that the error is somehow related to the mode switching done by mode analyzer. Do you know what could possibly cause this and what can we do about it? We ask because we want our automation to configure the amount of triton pods running, and we want to profile the results using model analyzer.

I will try to reproduce
We do have a ticket to try to make quick search more customizable instead of the fixed parameters that are currently searched. This won't be a small change however.
There are a ton of parameters inside of triton as well as for various backends. In theory we support them all, although most of them are not explicitly called out in the code base. I will try to come up with an example of a brute search with max_queue_size. Finishing #3 would also potentially allow max_queue_size to be quick searched.
Yes, the space grows exponentially with each model. You want to do brute with 12 different models? Even if you are only trying 5 configs for each model, that would be 5^12 (244 million) different tests to run.  12 models is definitely too many for brute. With a limited search space and only 2 models it is somewhat viable, but beyond that it just grows too fast.

For your latest comment, can you create a new issue? Add any further information that would be helpful in trying to reproduce.

I have not been able to reproduce number 1. Can you provide what a starting config looks like that stops right away for you?  Note that model analyzer keeps a checkpoint, so if you run model analyzer and then run it again, it will look at all of the results it already has and conclude that it doesn't need any new measurements.
As for number 2, there is a PR up that fixes the main issue (although the reporting still has some typos) https://github.com/triton-inference-server/model_analyzer/pull/806

Here is the config yaml file:
triton_launch_mode: remote
output_model_repository_path: /output_models/ourput
export_path: profile_results
override_output_mode_repository: true
triton_metrics_url: {my_url}
triton_http_endpoint: {my_endpoint}
triton_grpc_endpoint: {my_endpoint}
run_config_search_mode: quick
profile_models:

my_model
And here is the config.pbtxt:
platform: “pytorch_libtorch”
version_policy {
specific {
versions[1]
}
}
max_batch_size: 1
input: {
name: “input”
data_type: TYPE_FP32
dims: [3, 112, 112]
}
output: {
name: “output”
data_type: TYPE_FP32
dims: [512]
}
When the max_batch_size is 2, i immediately get skipping all illegal configurations, done with quick search. When its 1, everything works as expected.

Then we'll simply wait for it to be merged back into a stable release.

This is extremely important to us, because we have seen major improvements when tweaking dynamic batching by hand, and because we automate the process, it makes sense for it to be there, especially when brute force supports it.

I did not manage to use it in my models as you have described, if you could provide an example or even add the relevant info to the documentation it would be great.

If that’s what you say, We’ll give up on it for now. We have originally asked for it because of the missing options in quick search, and because we have seen it miss the mark by a lot in quick search so we thought to ask for a more accurate search option. But by how you described it, it does not seam feasible.

I'm still unable to reproduce 1), but your comment about illegal configuration is a good clue. If there is an illegal configuration at the start of quick search then it doesn't know where to go and will stop.  Can you run with -v (model-analyzer -v profile -f config.yml) and paste what is being printed?  Generally an illegal configuration will involve some bad combination of batch sizes.

For 4), anything in the model config can be manually swept via brute search. It just takes some yaml magic, which is hard to explain in the documents.
There are some examples in this documentation
Here is an example that includes max_queue_size:
run_config_search_mode: brute
profile_models:
  my_model:
    model_config_parameters:
      dynamic_batching:
        max_queue_delay_microseconds: [100, 200]
        default_queue_policy:
          max_queue_size: [1000, 2000]

and the output from MA:
[Model Analyzer] Creating model config: my_model_config_0
[Model Analyzer]   Setting dynamic_batching to {'max_queue_delay_microseconds': 100, 'default_queue_policy': {'max_queue_size': 1000}}
[Model Analyzer]
[Model Analyzer] Creating model config: my_model_config_1
[Model Analyzer]   Setting dynamic_batching to {'max_queue_delay_microseconds': 100, 'default_queue_policy': {'max_queue_size': 2000}}
[Model Analyzer]
[Model Analyzer] Creating model config: my_model_config_2
[Model Analyzer]   Setting dynamic_batching to {'max_queue_delay_microseconds': 200, 'default_queue_policy': {'max_queue_size': 1000}}
[Model Analyzer]
[Model Analyzer] Creating model config: my_model_config_3
[Model Analyzer]   Setting dynamic_batching to {'max_queue_delay_microseconds': 200, 'default_queue_policy': {'max_queue_size': 2000}}

Regarding 1, i tried it and it as it turns out, the problem occurs when we enable dynamic batching as well. Without it the search works as expected. We get the following message:
Illegal model run config because maximum of my model's preferred batch size 2 is greater than model max batch size 1. From what i know the preferred batch size is set by the max batch size, so it is still a bit weird, but thanks for the help.
Regarding 4, we tried it and it worked this time, thanks for that. Maybe the docs should make this clearer that all dynamic batching parameters are available.
And i would like to expand my questing regarding your comment that anything is possible to sweep through: do you really mean anything? For example, if we theoretically have different optimizations like graph optimizations and such, or we would like to try different backends/platforms to try, is this theoretically possible in brute search? If so, I didn't get it from the docs.

For 1), this all makes sense now. There was a fix a few weeks ago for this. See bullet 3 of the description here. This won't actually help you if you already have a config with preferred_batch_size specified, but if you start from scratch that value will no longer magically appear and cause you problems on further reruns. The fix will be included in the 24.01 release.
For 4), I'll look into improving the documentation
For sweeping anything - Model Analyzer should be able to sweep over any model config option that can be specified in the config.pbtxt file. I believe this also includes any options that are specific to whatever backend you are using. For example, ONNX and TensorFlow have a bunch of "execution accelerators" that can be specified, and you should be able to brute search through legal values of those.
If you have different versions of the same model across different build optimizations or backends, Model Analyzer supports running each of them one at a time and then finding the best across all of them. This can be achieved by passing multiple models to Model Analyzer, setting --num-top-model-configs to something like 3, and NOT specifying --run-config-profile-models-concurrently-enable. What should happen is that it should pick the top 3 performing configs across all of the models it profiled.
I know you were talking "theoretically", but if you do have something more concrete I can confirm and/or give an example.

Ok, thanks for the help!
Do you have any plans for the quick search improvements we discussed?

I created a ticket to design and implement the ability to quick search custom fields. I do think there is a lot of value in this, beyond just the cases you are asking for. It is something we plan to do, but likely won't get started at least until the product lead gets back from medical leave. Unfortunately I can't imagine this getting done for at least a few months.
I also created a ticket to look into Model Analyzer setting max_queue_size for some cases. This has a higher chance of getting done in the near term since the scope is smaller, but it probably won't be prioritized unless it is a really important feature to you.


feat: Align with the new design in form for inline error display

Description
Align with the new design in form for inline error display
Discussion

Show detail error message.(Now the error message is fixed)
Notification level. (Now only show error message, but notification message has 2 levels in notification page: warninig & error).

Task Item
closes #1623
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code refactor (non-breaking change which improve code quality, clean up, add tests, etc)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Doc update (document update)

Checklist

[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have functionally tested my change

Screenshots

Also, I didn't see anything to handle lint warnings. The spec is that we display an error if present, then a warning.

Note: What this change does not do is surface errors for any arbitrary field in the form. Only LG, LU and expression errors are surfaced right now.


Flutter generates XCode projects using the Legacy Build system (XCode 10)

Steps to Reproduce

From VSCode use the Flutter: New Project command.
Enter myapp for the project name, let Flutter build the project files
Use XCode to open the project
In the toolbar output there is a yellow hammer indicating the use of the legacy build system.

Please provide a mechanism so that the generated XCode projects to use the new build system, if we are using XCode 10. Its simply better in every way.
https://developer.apple.com/documentation/xcode_release_notes/xcode_10_release_notes/build_system_release_notes_for_xcode_10

+1 Are there any expectations for this?
I'm hoping that having flutter use the new build system would severely reduce the build time on CIs such as Travis.

+1 on this. the legacy build is not fast, and how long will it be supported? will it give rise to other issues going forward?

The legacy build system was explicitly adopted with https://github.com/flutter/flutter/pull/21901/commits/231784ecf6a38b47ec1994609da1bbbb3fe5e99adue to conflicting outputs between build phases related to CocoaPods. I will be switching back to the build system with https://github.com/flutter/flutter/issues/20685.

Closing for now. You can track progress in #20685.

I created a Flutter plugin (Flutter 1.22.3) and it was set by default to be using the Legacy Build system. I wanted to include an xcframework and this is not supported with the legacy build system. I had to manually switch to the new build system for the .xcodeproj

it was set by default to be using the Legacy Build system

@fotiDim

You mean in the plugin example project?
How did you fix it (what's the diff, which file actually changed?).
What was the exact error message?  Was it the legacy build system or the legacy build location?

@jmagman

yes
I fixed it by switching to the new build System on .xcodeproj. Then I switched to the beta channel and the problem came back even with the new build System. I am still trying to narrow it down.
Error is: :0: error: unable to load standard library for target
'arm64-apple-ios9.0-simulator'

On 1.22.3:
$ flutter create --platforms=ios -t plugin test_plugin
Creating project test_plugin...
...
Host platform code is in the ios directories under test_plugin.
To edit platform code in an IDE see https://flutter.dev/developing-packages/#edit-plugin-package.
$ open test_plugin/example/ios/Runner.xcworkspace

File > Workspace Settings...

Build system is New, not legacy.
@fotiDim What reproduction steps am I missing here?

@jmagman I believe you are opening the workspace instead of the project.
flutter --version
Flutter 1.22.3 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 8874f21e79 (7 days ago) • 2020-10-29 14:14:35 -0700
Engine • revision a1440ca392
Tools • Dart 2.10.3

flutter create --org com.mycompany --template=plugin --platforms=android,ios myplugin

open myplugin/example/ios/Runner.xcodeproj

Go to File -> Project Settings -> Advanced

@fotiDim That's the build location, not the build system.  You're hitting https://github.com/flutter/flutter/issues/57083.


Built-in “clear” button

As an opt-in feature (or opt-out, can be discussed), provide a built-in suffix element which, when clicked/tapped, sets the value of the field to an empty string.
The element should be an “X” icon and be shown before any other suffix elements (closest to the value).
Question: When the field is marked as required, should the clear button always be hidden?

Question: should Esc activate the clear button?
In some cases, this would probably be helpful (a search field), but in some other cases, it could be more unwanted (an email field).
I suppose this could also be an opt-in helper feature on top of the clear button itself.

Since this feature would be used by combo box, date picker and dropdown menu, it should be possible to opt-in/out this feature from those wrapping elements as well, even per-instance level.

When the field is marked as required, should the clear button always be hidden?

I'd say no: the clear could be useful also on required fields, e.g. just to clear the value and start over.

should Esc activate the clear button?

I'd say yes, if the clear button is visible; otherwise, no. This is the same behavior of the HTML5 <input type="search"> on most browsers: a clear button is showed and Esc triggers it; if type="text" no clear button is present and Esc does not clear the value.

Adding a clear() method could happen at the same time with this so that there’s a convenient way to do the same thing from the API.

Should this rather be a new text field extension, say <vaadin-search-field>, instead of an additional feature to the <vaadin-text-field>

See also https://developer.mozilla.org/en-US/docs/Web/Events/search

I’m struggling to see how this feature relates to search.
The native <input type="search"> does have a built-in clear button, but that doesn’t mean it’s the only use case that benefits from having one.

One of the advantages of the text-field opt-in feature would be using in vaadin-grid-filter.

Starting to implement this in the next sprint as an opt-in feature. The planned API looks somewhat like: <vaadin-text-field clear-button-visible>

Flag working name: clearButtonVisible(to align with password field's revealButtonHidden)
Don't dispatch "change" on click
Same theming API and appearance as that of <vaadin-combo-box>
Use the suffix area of the components (place as the first suffix)

Work on any open design questions ad-hoc with the designers. The remaining design/API issues we'll tackle during UX/DX phase.


Error running run_train_addition_model.sh

When I run sh src/run_train_addition_model.sh, it told me that
python3 npi/add/training_model.py /Users/yjy/WorkSpace/DeepLearning/NPI/src/../data/train.pkl /Users/yjy/WorkSpace/DeepLearning/NPI/src/../data/addition.model Using Theano backend. training f_enc src/run_train_addition_model.sh: line 16: 23143 Illegal instruction: 4  python3 npi/add/training_model.py /Users/yjy/WorkSpace/DeepLearning/NPI/src/../data/train.pkl /Users/yjy/WorkSpace/DeepLearning/NPI/src/../data/addition.model
What's goes wrong and how to fix it? Thanks in advance !

I ran this code in ubuntu virtualenv, and does not got this error. I really confused what's goes wrong. Anyway, it doesn't have error.


Layout keeps repeating itself (see image)

Did something break in the tile generation?

It's a wrap puzzle, so it's supposed to do that.
From the website:

Wrap pipes are a challenging variation of the puzzle. The connections in this variant are allowed to wrap between borders of the board. Imagine copies of the puzzle placed on all sides of it (left, right, top and bottom). This way tiles on the right border can connect to tiles on the left border - or "wrap" around.
Unlike regular pipes puzzles the wrap variant has no convenient outer walls to start the deductions from. Hence its increased difficulty. To help deal with wrap edges multiple copies of the puzzle are shown, tiling the screen. When you rotate a tile all its copies will rotate too.

Im a dummy :)


Bugfix #33, bugfix #27, removal of unused node packages, cleanup of code, formatting of code with eslint

@rusty-sj Can you re-open this PR but make it merge from branch-rashmi to dev instead of main?


no fps below 60

when i make my fps lower than 60 then it doesnt work and stays as 60 fps
this was when i firstly installed openhack
wanted to play with low fps

What if You Enabled Vertical sync in official gd settings?

i did disabled vertical sync but when i make fps LOWER 60 its not working but bigger then 60 works

Works as intended. The only thing I should change is the minimum value of the input box.
FPS Bypass uses in-game settings to make sure it aligns with what RobTops' code does.
Making custom FPS bypass will make the mod incompatible with other mod menus (if you use any), and i plan to keep the mod nonintrusive as much as I can.


IMU sensitivity is too high

I'm trying to integrate imu (MPU9250) into google cartographer. I publish the IMU data I obtain to the /imu topic by passing it through the Madgwick filter. When I check the sensor with ros_imu_plugin, the movement seems to be very smooth and working properly. However, when you set use_imu = true in Google Cartographer, the mapping becomes very stupid and the orientation of the base_link changes very quickly, so the sensitivity seems to be very low. Even a little movement causes a lot of rotation. What is the reason of this ? I had this problem before in ros_imu_plugin, but I fixed it by playing with the parameters in the Madgwick filter, but I don't know how to solve it in google cartograph. I would also like to point out that I did 2D mapping.
include "map_builder.lua"
include "trajectory_builder.lua"
options = {
map_builder = MAP_BUILDER,
trajectory_builder = TRAJECTORY_BUILDER,
map_frame = "map",
tracking_frame = "base_link",
published_frame = "base_link",
odom_frame = "odom",
provide_odom_frame = true,
publish_frame_projected_to_2d = true,
use_odometry = false,
use_nav_sat = false,
use_landmarks = false,
num_laser_scans = 1,
num_multi_echo_laser_scans = 0,
num_subdivisions_per_laser_scan = 1,
num_point_clouds = 0,
lookup_transform_timeout_sec = 0.2,
submap_publish_period_sec = 0.3,
pose_publish_period_sec = 5e-3,
trajectory_publish_period_sec = 30e-3,
rangefinder_sampling_ratio = 1.,
odometry_sampling_ratio = 1.,
fixed_frame_pose_sampling_ratio = 1.,
imu_sampling_ratio = 1.,
landmarks_sampling_ratio = 1.,
}
MAP_BUILDER.use_trajectory_builder_2d = true
TRAJECTORY_BUILDER_2D.min_range = 0.5
TRAJECTORY_BUILDER_2D.max_range = 8.
TRAJECTORY_BUILDER_2D.missing_data_ray_length = 8.5
TRAJECTORY_BUILDER_2D.use_imu_data = false
TRAJECTORY_BUILDER_2D.use_online_correlative_scan_matching = true
TRAJECTORY_BUILDER_2D.real_time_correlative_scan_matcher.linear_search_window = 0.1
TRAJECTORY_BUILDER_2D.real_time_correlative_scan_matcher.translation_delta_cost_weight = 10.
TRAJECTORY_BUILDER_2D.real_time_correlative_scan_matcher.rotation_delta_cost_weight = 1e-1
TRAJECTORY_BUILDER_2D.motion_filter.max_angle_radians = math.rad(0.2)
-- for current lidar only 1 is good value
TRAJECTORY_BUILDER_2D.num_accumulated_range_data = 1
POSE_GRAPH.constraint_builder.min_score = 0.65
POSE_GRAPH.constraint_builder.global_localization_min_score = 0.65
POSE_GRAPH.optimization_problem.huber_scale = 1e2
POSE_GRAPH.optimize_every_n_nodes = 35
return options

您好！我已收到你的邮件！谢谢！          ——曾君

header:
seq: 780885
stamp:
secs:<PHONE_NUMBER>
nsecs: 214104413
frame_id: "base_link"
orientation:
x: -0.8021243250537446
y: -0.021046560940722856
z: 0.5868600885324634
w: -0.10839209342619172
orientation_covariance: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]
angular_velocity:
x: -0.244140625
y: 0.54931640625
z: 0.0
angular_velocity_covariance: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]
linear_acceleration:
x: -0.267578125
y: -0.69921875
z: 1.32373046875
linear_acceleration_covariance: [0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0]
imu data that i publish


Add support for creating pull requests via git push options

Gitea 1.3 added support for push options: https://docs.gitea.io/en-us/push-options/
It would be nice if pull requests could be opened via push options, like it's done for gitlab: https://docs.gitlab.com/ce/user/project/push_options.html

I would use tea or just klick on the link the git push returns ...

I would use tea or just klick on the link the git push returns ...

@tea requires installing another tool, while git push options are supported by git core itself.

The "Agit Setup" page in the documentation shows something very close to this. It will work for me. It allows you to create a PR using the ref.
https://docs.gitea.com/usage/agit-setup

I guess we can close this issue now :)


Write a Python program to convert days to hours

Description
Write a Python program to convert days to hours
1 day = 24 hours
Input  : 5
Output : 120

How to contribute

Save the solution in program/convert-days-to-hours.py file
Add convert-days-to-hours.py file in convert-days-to-hours folder

!assign


:bug: Owned CRDs should have resources specified

Summary
Owned CRDs should have resources specified
Related issue(s)
Fixes #
      "status": {
        "results": [
          {
            "name": "olm-crds-have-resources",
            "log": "Loaded ClusterServiceVersion: multicluster-global-hub-operator-rh.v1.3.0\n",
            "state": "fail",
            "errors": [
              "Owned CRDs do not have resources specified"
            ],
            "creationTimestamp": null
          }
        ]
      }

ref: http://external-ci-coldstorage.datahub.redhat.com/cvp/cvp-redhat-operator-bundle-image-validation-test/multicluster-globalhub-operator-bundle-container-v1.3.0-31/b854a993-d8f2-4b8e-9e82-b3804a40e233/operator-custom-scorecard-tests-bundle-image-output.txt
Tests

[ ] Unit/function tests have been added and incorporated into make unit-tests.
[ ] Integration tests have been added and incorporated into make integration-test.
[ ] E2E tests have been added and incorporated into make e2e-test-all.
[ ] List other manual tests you have done.

/lgtm


Fzf inside tms does not respect global config

Hello, I have a couple of questions regarding fzf config. It looks like on my setup when used from sessionizer fzf does not respect my global config. Like, for example, I manage my fzf installation and config through smth called home manager, which is based on Nix package manager. I have this line in it:
defaultOptions = [ "--bind=ctrl-j:accept" ];

this is the equivalent of setting
export FZF_DEFAULT_OPTS="--bind=ctrl-j:accept"

But when using tms or tms switch, fzf does not respect that setting. Do you have any ideas as to why, and could I circumvent this?

Ah, sorry, after "skimming" through the repository I realized that you use skim instead of fzf 😅

so I guess to solve this I would have to make this configurable. Would you accept a PR?

@axkirillov I would! But I actually want to switch away from using skim at all. I'd prefer to switch to using nucleo. I have an issue open for that as well. (Though I haven't actually even looked through to see what options nucleo supports.)
It would be sad to see a nice pr for skim that loses some effort when I switch to nucleo (at some point).
So preferable would be to help switch to nucleo first but if you made the changes with skim I would definitely want them merged here.

I'm going to close this for now because we've switched to nucleo and a custom tui.
Custom tui config will be added at some point.


python311Packages.python-benedict: 0.30.1 -> 0.30.2

Diff: https://github.com/fabiocaccamo/python-benedict/compare/refs/tags/0.30.1...0.30.2
Changelog: https://github.com/fabiocaccamo/python-benedict/blob/0.30.2/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] 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.11 Release Notes (or backporting 23.05 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

[ ] Fits CONTRIBUTING.md.

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

4 packages built:

python310Packages.python-benedict
python310Packages.python-benedict.dist
python311Packages.python-benedict
python311Packages.python-benedict.dist


documentation: Logger.Level documentation is incomplete

The documentation for Level is incomplete[1]:
	// The logging level the logger should log at. This is typically (and defaults
	// to) `logrus.Info`, which allows Info(), Warn(), Error() and Fatal() to be
	// logged. `logrus.Debug` is useful in

[1] https://github.com/sirupsen/logrus/blob/master/logger.go#L25-L28

@sirupsen do you remember what you wanted to add there?
If not, I'll remove the unfinished sentence.

What a cliffhanger...
Let's remove that unfinished sentence, I don't think it brings value 👍
Thanks @Bo0mer

Thank you for reporting this @Bo0mer


Alter order repository to include order attributes for the backend too

Description
Please describe your pull request:
This pull request is necessary so that backend order queries can use the attributes as well. This is needed e.g. to provide sorting and filtering in the backend order overview based on custom attributes. This does not add this functionality, but merely provides the needed core changes for plugins to be able to use such functionality.

Questions
Answers

BC breaks?
no

Tests pass?
yes

Related tickets?
no

How to test?
see below

You can easily see that this behaviour is broken by using the Backend "Order" controller's getList endpoint. Take for example:
/backend/Order/getList?_dc=1484413981137&page=1&start=0&limit=60&filter=[{%22property%22:%22attribute.type%22,%22value%22:8,%22operator%22:null,%22expression%22:null}]

this URL (assuming disabled CSRF protection for debugging) will throw an Exception in an unaltered Shopware 5 installation, since the query builder which the controller uses internally (see changed files) does not care about the "attribute" relationship.
Adding the changes proposed in this PR makes filtering by attributes work.
This PR does not include tests, since I could not find a relevant test in the current test suite either.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/#/issues/SW-17469.
Please use this issue to track the state of your pull request.


Build tflite_runtime version 2.1.0.post1 for python3.8

Hey all!
I´m working on a aarch64 system and inside a ros:foxy docker container.
Because I´m working with the google coral chip I need to use tflite_runtime version 2.1.0.post1.
According to the edge tpu guy I need to build from commit "d855adfc5a0195788bf5f92c3c7352e638aa1109"  but after building a python 3.8 wheel file and installing that I still get this error:
ValueError: Model provided has model identifier ' TPU', should be 'TFL3'
I´m not sure if this is a edge tpu or tflite issue (see this issue for reference).
I´m also not sure whether i build the correct version because tflite_runtime.version displays 2.1.0.
Here are my two questions:

Which branch/tag do I need to pull for version 2.1.0.post1?
Do you have any idea where this error might come from? (I´m 100% sure I´m loading the correct file)

From the error looks like you're trying to load a model in TFLite interpreter that is not a TFLite flatbuffer. Follow EdgeTPU guide for running your converted model.

I know that's what it looks like but i´m using the provided examples from the "Getting started" tutorial so the models should be fine. And these issues occur with compiled models as well as normal models.
The paths are fine as well, I tried copy pasting the commands as well as writing them using tab completion.
On the other hand these things work just fine so maybe something about the model conversion changed that breaks the EdgeTPU example models?
Does the commit "d855adfc5a0195788bf5f92c3c7352e638aa1109" build version 2.1.0.post1?
If not maybe this version is responsible for this behavior and I just need to know the right commit to build the right version.

@ItsMeTheBee If it is TFLite that you're using please send reproduce steps to the issue. If it is the EdgeTPU part, then please file issue with EdgeTPU so they can help you.
Commit "d855adfc5a0195788bf5f92c3c7352e638aa1109" is available in v2.2 +


Add sort options for inventory, create pack and view pack

Sort by weight or alphabetically. Utility function should be created.

Agree with this request


[proj4]: x64-windows install no longer produces dlls

Something in a9303736fdda58cf5ff3ccc969be917580493c91 has broken the proj4 port for Windows.
It no longer produces dlls for the x64-windows triplet. It seems to always want to build as static (evidenced by the lack of bin and debug/bin folders and way bigger .lib files upon completion)

#7192 should fix this issue.

Hi @jumpinjackie, thanks for reporting this issue.
I've fixed this issue now, please update vcpkg and rebuild proj4.
Thanks.


Json schema and backstage

Add support for our json schema for autocomplete and adds backstage config

Hey @six7  I just wanted you to check that the behaviour with the intellisense of the schema fits with your idea of how it should work :)


genome (Rnor_5.0) availble in AWS genomes but not listed in config

Check Documentation
I have checked the following places for your error:

[X] nf-core website: troubleshooting
[X] nf-core/rnaseq pipeline documentation

Description of the bug
From reading the docs I assumed that any genome stored in AWS-iGenomes would be readily available when calling the pipeline, but at least one, Rnor_5.0, isn't.
https://github.com/nf-core/rnaseq/blob/master/conf/igenomes.config
Steps to reproduce
Genome is pre-downloaded but error also occours without setting --igenomes_base

Command line: nextflow run nf-core/rnaseq -r 3.1 --input "${PRJ_DATA}/raw/bayer_samplesheet.csv" --igenomes_base "/mnt/genomics/references" --genome "Rnor_5.0"  -profile singularity --aligner "star_salmon" -with-dag --max_memory "40.GB"
See error:

=============================================================================
  Genome 'Rnor_5.0' not found in any config files provided to the pipeline.
  Currently, the available genome keys are:
  GRCh37, GRCh38, GRCm38, TAIR10, EB2, UMD3.1, WBcel235, CanFam3.1, GRCz10, BDGP6, EquCab2, EB1, Galgal4, Gm01, Mmul_1, IRGSP-1.0, CHIMP2.1.4, Rnor_6.0, R64-1-1, EF2, Sbi1, Sscrofa10.2, AGPv3, hg38, hg19, mm10, bosTau8, ce10, canFam3, danRer10, dm6, equCab2, galGal4, panTro4, rn6, sacCer3, susScr3
===================================================================================

Expected behaviour
The pipeline should be able to use any iGenomes available in AWS.
Log files
Have you provided the following extra information/files:

[X] The command used to run the pipeline
[ ] The .nextflow.log file

System

Hardware: Linux server
Executor:
OS: Ubuntu
Version 20.04

Nextflow Installation

Version: 21.04.0-edge.5543

Container engine

Engine: Singularity
version:
Image tag: nfcore/rnaseq:3.1.0

Added in https://github.com/nf-core/rnaseq/pull/642


Update index.html to support viewport (for mobile)

This makes the wasm demo to show correct on mobile devices

Thanks a lot @alorma!


[SURVEY][V8] DataTables MAJOR BREAKING CHANGES

Hi Guys,
I will change the namespace of the package from Datatables to DataTables as I feel it is the right name but it will be a MAJOR BREAKING CHANGE for all the users. Datatables was used in the beginning since it was inherited from the original package.
Would you mind sharing your thoughts on the changes I've been working on version 8? All breaking changes are documented on master branch change log but I may be overdoing it without taking the users impression on it.
Feedbacks highly appreciated.
Thanks!

Draft upgrade guide docs added.

I don't see anywhere but i think will be good add this attributes to searching.
data-order attribute
Sometimes we want order some field diffrent like a getting data. Maybe this is possible? I don't saw this possibility.

@AdrianKuriata thanks for the input. You can already implement the same feature now but on a server-side way.
For data-search, you can have both first_name and last_name included on search by adding a hidden column and display the formatted name using addColumn or via mutator. Same goes for order part.
columns: [
  {data: 'name', name: 'last_name'},
  {data: 'first_name', visible: false}
]

Sure, please create a new issue for a new thread.

I created it, #1244 .

@yajra Hi Yajra. I tested up your solution for searching with other fields which is not visible.

When i did it, in print, copy to clipboard and csv file i see this field. When i added printable and exportable on false this not working. In the D section you can see field which is visible on false and printable, exportable on false. Pic.
With this solution i can't use few field to search, let me explain. I have 3 models, Street, Floor, and Flat. I want add this to the search like a connected one field (ex. Wall Street 3/1 where in Street model is Wall Street etc. in Floor is 3 etc. and Flat is 1 etc.). With your solution i can create couple fields with separated information about street. I tested date. When i have date saved like a timestamp in the database i can't search with her using converted date to format like a d.m.Y. This is because name field working with database, i can't set custom values with searching. When i changed it with editColumn this won't work, i can just searching with timestamp format. With data-order and data-search i can put formated date and searching will be working properly. When i don't add name field(name working like that, when you don't add name field, this will extend data from data field, i thought this will work when i convert data in editColumn, but this won't work. I know i can create custom search engine and use filtering, but solution with data-attribute would be really usefull.

I think that the namespace should definitely be DataTables. It's nice to see you working on this. I will contribute more about laravel routing/filtering stuff after I get free from the current projects I have.
Thanks!


Create Complicated-hello-world

What does the program do ?
This program creates an array with the letters "Hello World" then randomizes it and puts it back together to print out "Hello World".
In what programming language it is written?
Java

Thank you for participating  in hacktoberfest ❣ if you liked working on this repo you can share it with friends and star this project for helping others


VSCode Extension

Implement Mentat as a Visual Studio Code extension. Basic functionality:

Add a "Mentat" tab in VSCode which opens a new vscode webview
Page 1: Select files and click a start button
Page 2: A standard chat interface for the cli workflow, with buttons for interrupt and restart (back to Page 1)

See /vscode/README.md for setup instructions.

@waydegg want to help review this? It's waiting on me to finish #74 , but it'd be good to have you eyes on the VS Code extension implementation

I wonder if the LSP portion (vscode/bundled/tool/lsp_server.py) could be general enough to support other editor extensions at some point, like for NeoVim?

@granawkins do you have any references to VSCode plugins made with Svelte that are somewhat popular/high-quality that you like? Asking since I've never used Svelte before and wanted to see how other plugins do it.

Need to find a way to make the UI work well with light themes:

I'm a big fan of what Cursor is doing for their chat UI/UX. Any thoughts on this?

They have some good feedback in their Discord on what people have liked/disliked too.

Is this just blocked on #74? Happy to review but I also don't wanna add a bunch of comments per-maturely if you're still making big additions/edits :) Pretty dope to see this working in VSCode btw!

Is this just blocked on https://github.com/biobootloader/mentat/pull/74? Happy to review but I also don't wanna add a bunch of comments per-maturely if you're still making big additions/edits :)

@waydegg yes #74 needs to be merged first. The big open question is the best way to run both Mentat and the Python side of the LSP defined here - should we set up an async loop that runs both? Run on different processes?

 Any thoughts on this?

Looks nice! My hope for this PR is to get a working Mentat experience in VS Code that we can then iterate on. So higher level comments on the best way to set this up are most useful. We should also track interface feature ideas somewhere, feel free to create issues for individual ideas or maybe we can start a list somewhere.

I wonder if the LSP portion (vscode/bundled/tool/lsp_server.py) could be general enough to support other editor extensions at some point, like for NeoVim?

Absolutely, that's what I was thinking too. That's how Rift does it. Since this is from a template I figure we leave it like this for now, and if/when we add NeoVim we can separate it out.

The big open question is the best way to run both Mentat and the Python side of the LSP defined here - should we set up an async loop that runs both? Run on different processes?

I think we should def be building towards running everything async (electron already eats up enough processing as is haha). pygls, the library the VSCode extension template uses that interfaces w/ language servers, has first-class async support. Most of Mentat latency is network io also.
Would love you hear what you guys are thinking.

I'm not married to Svelte though, it's my first time using it as well. It's really similar to Vue, which I've used a lot, but the bundle is lighter and it builds really fast. Open to suggestions.

I'm not married to any particular framework either (I've been wanting an excuse to learn Svelte anyways haha). I guess my only concern would be ease-of-contributing since most ppl already know React/Vue and there's obv a huge library ecosystem for both frameworks.

At a high level I think this looks good! I'll create a github discussion/issue for feature suggestions we can all collaborate on.

Interesting: rift already included mentat as an agent.
Also @waydegg I don't see a cursor discord. Do you mean their forum?

Also are we sure we want this as a sub-directory in the mentat repo? I guess if the VSCode and NeoVim plugins end up sharing code it'll be simplest. But it will interfere with the standard way of installing vim plugins: cloning a repo into a plugin directory.

@granawkins and I were talking about this - the benefit of a shared repo is it's easier to coordinate versions and changes between "core" Mentat and the VS Code extension. We do want any code that would be shared by VS Code and NeoVim plugins here (and those bits would probably be useful as well for the community if they make extensions for other editors). @jakethekoenig is the way you see it interfering with vim plugin install just that the cloned repo would be bigger?
Overall my vote is we keep it here for now and see how it goes, but that's not a strong opinion

Interesting: rift already included mentat as an agent.

Just announced today! They way they've included it involves inserting callbacks into the code / patching some input / output functions. When we finish getting Mentat setup for multiple interfaces they'll have an easier time connecting to it.

Also @waydegg I don't see a cursor discord. Do you mean their forum?

They have a discord but looks like they disabled invites for it and are switching to the forum

Also are we sure we want this as a sub-directory in the mentat repo? I guess if the VSCode and NeoVim plugins end up sharing code it'll be simplest. But it will interfere with the standard way of installing vim plugins: cloning a repo into a plugin directory.

@granawkins and I were talking about this - the benefit of a shared repo is it's easier to coordinate versions and changes between "core" Mentat and the VS Code extension. We do want any code that would be shared by VS Code and NeoVim plugins here (and those bits would probably be useful as well for the community if they make extensions for other editors). @jakethekoenig is the way you see it interfering with vim plugin install just that the cloned repo would be bigger?

I have two concerns:

Yeah it would make the cloned repo bigger. I guess that's not actually a big concern because storage/bandwidth is cheap and if becomes one we can make instruct the users to use use git clone --filter and git sparse-checkout to just get what they need. Though it's complicated because normally if I wanted to instruct a user to do something non trivial I'd put it in a script for them to run but if they have access to the script they've already cloned the repo.
Vim plugins expect their files in a specific structure so we may have to litter the root of the project with autoload, doc, plugin and syntax and won't be able to bunch them all up under vim. I suppose there's no need to handle vscode and vim in the same way so we could have the vim plugin be a git submodule but the vscode plugin be a subdirectory like it is now.

Sorry if this is getting off topic for this PR.

Closing this b/c of #213


Translation Romanian

Complete romanian translation.

Hi. Thank you for your contribution.
Translation process has moved to weblate. I just added the romanian translation file there. Could you use weblate for the translation?
https://hosted.weblate.org/projects/wtforms/wtforms/ro/

Hi,
I've fixed end of file test error and completed the translation on weblate.

Thank you.
Weblate has pushed your translation on this repository, so I will close this PR.


rate_limit: Add management command to reset auth rate limit.

https://chat.zulip.org/#narrow/stream/31-production-help/topic/Too.20many.20password.20attempts
The auth attempt rate limit is quite low (on purpose), so this can be a
common scenario where a user asks their admin to reset the limit instead
of waiting. We should provide a tool for administrators to handle such
requests without fiddling around with code in manage.py shell.

Merged, thanks @mateuszmandera!  @alexmv feel free to backport to 4.x after thinking a bit about the command name.

clear_auth_rate_limit_history seems pretty good.  If I were to split hairs, I think I'd go with reset_authentication_attempt_count or something; the word "history" implies it's maybe more of a durable log, and not a number, that's being cleared.  And naming it based on wording that the user sees (authentication attempt) vs the underlying mechanism (rate limit) may make it more discoverable and specific.
@mateuszmandera: Any thoughts on the naming?

@alexmv Yeah, the "reset" naming seems much clearer


⚠ Secret Website 3 has degraded performance

In ad53eae, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 1796 ms

Resolved: Secret Website 3 performance has improved in bb2505f.


Remove duplicated/redundant information from README.md file

Description
The Vue Vocabulary README.md file contains a lot of redundant information. The goal is to make it lean and succint like that of Vocabulary.

@dhruvkb I want to take this issue.


Added data API quickstart

https://github.com/microsoft/PlanetaryComputerExamples/pull/207 switched that notebook to using folium, which renders widgets inside of an iframe, which (should) make its way through the doc build intact.

https://github.com/python-visualization/folium/issues/1277 discusses the trust issue a bit.

The generated HTML was actually correct. It seems that the DOMPurify.sanitize was sanitizing the iframe srcdoc and preventing it from loading. I've (temporarily?) commented disabled the sanitization to see if this works.

OK, this is working at https://icy-meadow-0fc35e30f-346.eastus2.azurestaticapps.net/docs/quickstarts/using-the-data-api/.
Will wait for @mmcfarland for thoughts on whether the change to not sanitizing all the docs is OK, or whether we should be a bit more targeted.

That's great you got this working. I think if we add a few (slightly cumbersome) safety mechanisms, this should be a good option:

Add sandbox="allow-scripts" to the iframe element in the notebook.
Add an allow-list of notebook names that are allowed to be unsanitized, to keep the surface area small
Remove any of the embedded CDN resources that aren't required, and add an SRI hash to the script tag for the remaining

We also should figure out a way to avoid using an OSM basemap for compliance reasons. Azure Maps can integrate with leaflet layers, but supplying the API key to the iframe would be a challenge. I'll think on this a bit and can volunteer to do # 2

Azure Maps can integrate with leaflet layers, but supplying the API key to the iframe would be a challenge.

Probably obvious, but I confirmed that an Azure Maps subscription key is embedded in the notebook document, even if it's passed through as an environment variable or using getpass. The subscription key is embedded in the ipyleaflet / folium object.

It occurred to me after leaving that comment that we probably don't have much control over the iframe construction or contents (I was skimming the notebook as a text file to see the html/js, forgetting that it's probably injected by folium). If those suggestions aren't really feasible, I think just the allow list would be sufficient.

Yeah, the iframe is indeed created by folium. We can, with some effort, modify the HTML as part of the doc build process.


Zuul 2.x : Netty sample for performance testing

Hi,
Is there a sample based on Zuul Netty that can be used for a HTTP performance test?
Can we use [1] with a few tweaks? Also, is there any performance turning guidelines for that.
Thanks.
[1] https://github.com/Netflix/zuul/tree/2.x/zuul-netty-sample

@kasun04 sure that can be used as a starting point for a benchmark. What are you comparing against?
There aren't any performance tuning guidelines per se but here is what we tested sometime back:
WSPerfLab
and here is a helloworld sample in RxNetty

We would like to compare Zuul with a raw Netty based reverse proxy kind of a runtime. Also is there any comparison done against other similar solutions.

We will be publishing performance results, etc once we are live with zuul 2 at netflix.