Fixed Search Function to Notify Users of Empty Input and No Matching Books

Description
A clear and concise description of what the PR does.

This PR does the following:
Fixes the issue where no message was shown when a book search returned no results.
Updates the search feature to display a notification when no book is found in the database.

Related Issues

"Fixes #423 "

Changes
List the detailed changes made in this PR.
Added a pop-up message when the search returns no matching books.
Refactored the search component to handle empty result sets more efficiently.
Testing Instructions
Detailed instructions on how to test the changes. Include any setup needed and specific test cases.

Pull this branch.
Run npm install to install dependencies.
Run npm test to execute the test suite.
Search for a non-existent book, and verify that a pop-up card or message is displayed, notifying the user that no results were found.

Screenshots (if applicable)
https://github.com/user-attachments/assets/c80ca259-dbde-4dce-8d12-d19305314b42
Additional Context
Any additional context or information that reviewers should be aware of.

This PR is based on user feedback regarding the confusion when book search results are empty.

Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I am working on this issue under GSSOC

@Trisha-tech  I wanted to let you know that the search filter bug has been solved. Please take some time to check it out, and let me know if there are any changes or improvements you would like to suggest.

Description
A clear and concise description of what the PR does.

This PR does the following:
Fixes the issue where no message was shown when a book search returned no results.
Updates the search feature to display a notification when no book is found in the database.

Related Issues

"Fixes [Bug]: Search Filter #423 "

Changes
List the detailed changes made in this PR. Added a pop-up message when the search returns no matching books. Refactored the search component to handle empty result sets more efficiently.
Testing Instructions
Detailed instructions on how to test the changes. Include any setup needed and specific test cases.

Pull this branch.
Run npm install to install dependencies.
Run npm test to execute the test suite.
Search for a non-existent book, and verify that a pop-up card or message is displayed, notifying the user that no results were found.

Screenshots (if applicable)
searchbug.1.mp4
Additional Context
Any additional context or information that reviewers should be aware of.

This PR is based on user feedback regarding the confusion when book search results are empty.

Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.

[x]  I have performed a self-review of my code
[x]  I have commented my code, particularly in hard-to-understand areas
[x]  My changes generate no new warnings
[x]  I am working on this issue under GSSOC

Kindly resolve these conflicts @darshinihoney

@Trisha-tech I have resolved the commit conflicts and checked them to ensure everything is in sync. Let me know if further review is needed.

@Trisha-tech I have resolved the commit conflicts and checked them to ensure everything is in sync. Let me know if further review is needed.

Kindly look into your code which changes the features that have already been implemented. Pull the merge commit before your code, add your changes and then make the pull request again. @darshinihoney


Say something when Dr_Dubz posts art

Another simple change that I hope is worth it. I don't really want to single out users but considering how much Dubz has done for Tracker, I felt like this one was maybe worth doing.

I wonder if this could be made a bit more generic and able to be set in the configs? Partially because it does feel weird to have things hard coded for a specific person, but I could see us possibly using this in the future for other shenanigans. Like, maybe a rare line Doctor Smasher calls out wanting to go to Shaktool.

Yeah, of course. Just didn't want to make this overly complicated. I figured we could always easily expand this to be configurable later on if this is received well.


Flaky test: TestHDFS

=== RUN   TestHDFS
    object_storage_test.go:93: PUT failed: create /unit-test/.test.tmp.5516401644572827060: addBlock call failed with ERROR_APPLICATION (java.io.IOException)
--- FAIL: TestHDFS (0.02s)

Fixed by #3031


Reconciled Text Colour not working

The Reconciled Text Colour setting is having no effect for me regardless of what setting I choose. I'll do some investigating with the console and see what's going on.

@Geiseku I think I found the issue.
Since 'use strict' is in the main.js file for this plugin, we're failing on line 17/18 setting an undeclared variable.
I think we might need to look into JSCS rules or maybe even consider using eslint to catch these issues. I can make a PR that fixes this unless you would rather?

Go for it.


Fix snapshot

What is the change?
Finalize database changes for snapshots
Why is the change being made?
#1090 was merged prior to final internal testing. after internal testing some minor changes were needed to snapshots.

Checklist

[x] This PR has only one purpose or idea.
[ ] Tests have been added/updated to verify any new/changed code.

[x] The code style follows good practices.
[x] The commit message(s) follow good practices.

[x] The release notes have been updated if necessary.
[x] The documentation is still up-to-date in the doc folder.
[x] The dependencies are still up-to-date in pyproject.toml.

After offline discussion, I understand that we have an internal integration test that covers this scenario. I am okay without the addition of another unit test to this PR.
Right now the unit tests are failing. I can approve when that is resolved.

The unit tests are currently broke.
And ruff requests some changes.


Add observability cluster permissions

Signed-off-by: Joshua Li<EMAIL_ADDRESS>opensearch-project/security-dashboards-plugin pull request intake form
Please provide as much details as possible to get feedback/acceptance on your PR quickly

Category: New feature

Github Issue # or road-map entry, if available: https://github.com/opensearch-project/trace-analytics/issues/178

Description of changes: Add new cluster permissions for observability plugin

Why these changes are required? Observability plugin will be released in 1.2

What is the old behavior before changes and new behavior after changes? (Please add any example/logs/screen-shot if available)

Testing done: (Please provide details of testing done: Unit testing, integration testing and manual testing)

TO-DOs, if any: (Please describe pending items and provide Github issues# for each of them)

Is it backport from main branch? (If yes, please add backport PR # and commits #)

By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #851 (5832bb3) into main (a30a6d4) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 5832bb3 differs from pull request most recent head e1494a3. Consider uploading reports for the commit e1494a3 to get more accurate results

@@           Coverage Diff           @@
##             main     #851   +/-   ##
=======================================
  Coverage   71.98%   71.98%
=======================================
  Files          87       87
  Lines        1906     1906
  Branches      247      247
=======================================
  Hits         1372     1372
  Misses        480      480
  Partials       54       54

Impacted Files
Coverage Δ

...rds-plugin/public/apps/configuration/constants.tsx
85.71% <0.00%> (ø)

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 a30a6d4...e1494a3. Read the comment docs.


get_token no client_secret in params

while using https://github.com/scrogson/oauth2_example (google auth)  in
def get_token(%{token_method: method} = client, params \\ [], headers \\ [], opts \\ []) do
        {client, url} = token_url(client, params, headers)
        case Request.request(method, client, url, client.params, client.headers, opts) do

client.params =
%{"client_id" => "some_client_id",
  "code" => "5/codestring",
  "grant_type" => "authorization_code",
  "redirect_uri" => "http://localhost:4000/auth/google/callback"}

and this ends up throwing
%OAuth2.Response{body: %{"error" => "invalid_request",
    "error_description" => "client_secret is missing."},

just to test this theory (because I don't the inner workings) replacing the case line with
case Request.request(method, client, url, Map.put_new(client.params, "client_secret", client.client_secret), client.headers, opts) do

and it works.

@version "0.8.0"

Hello @jtzero,
Please see #72 for an explanation of this issue.
The issue you are experiencing is not with this repo, but the implementation on the oauth2_example repo.
You will need to use put_param/3 to the Google module here.
def get_token(client, params, headers) do
  client
  |> put_param(:client_secrect, client.client_secret)
  |> put_header("accept", "application/json")
  |> AuthCode.get_token(params, headers)
end

If you want to contribute a fix over there, feel free!

awesome thanks for pointing me in the right direction!


Installer .msi link installs something other than GYB

New to GYB and following the Wiki to install.  Something weird about the linked msi.
Install GYB Quickly > Windows Users > The Installer can be downloaded here
The displayed URL doesn't resolve:
[https://github.com/GAM-team/got-your-back/releases/Setup.msi]
But the weird part is the underlying URL (https://qsdownloads.com/Setup/) downloaded Setup13.2.msi which seems to be something called NovaPDF, although the .msi has no interface so it's 'troubling'.
Is the Wiki messed up or am I missing something?

Confirmed that the Wiki is publically editabe.  Also confirmed that links have been replaced with links to download  powershell trojans.  DO NOT TRUST the links in the Wiki page.

Sigh, this is why we can't have nice things.
I've disabled public Wiki editing now.
I believe all links are now correctly pointing at GYB GitHub downloads but if you notice anything broken lmk.
Edits to wiki pages will now need to go through me (which means slower, sadly)


[Bug] Quantization for networks using AdaptivePadding, like efficientnet

Describe the bug
When I tried to quantize "efficientnet" with PTQ, the error messages occurred.
Traceback (most recent call last):
File "/opt/conda/envs/midap-mmrazor-torch1.13/lib/python3.9/site-packages/mmengine/registry/build_functions.py", line 121, in build_from_cfg
obj = obj_cls(**args)  # type: ignore
File "/root/workspace/mmrazor/mmrazor/models/algorithms/quantization/mm_architecture.py", line 90, in init
self.qmodels = self._build_qmodels(self.architecture)
File "/root/workspace/mmrazor/mmrazor/models/algorithms/quantization/mm_architecture.py", line 300, in _build_qmodels
observed_module = self.quantizer.prepare(model, concrete_args)
File "/root/workspace/mmrazor/mmrazor/models/quantizers/native_quantizer.py", line 231, in prepare
traced_graph = self.tracer.trace(model, concrete_args=concrete_args)
File "/root/workspace/mmrazor/mmrazor/models/task_modules/tracer/fx/custom_tracer.py", line 430, in trace
'output', (self.create_arg(fn(*args)), ), {},
File "/root/workspace/mmclassification/mmcls/models/classifiers/image.py", line 111, in forward
feats = self.extract_feat(inputs)
File "/root/workspace/mmclassification/mmcls/models/classifiers/image.py", line 196, in extract_feat
x = self.backbone(inputs)
File "/root/workspace/mmrazor/mmrazor/models/task_modules/tracer/fx/custom_tracer.py", line 406, in module_call_wrapper
return self.call_module(mod, forward, args, kwargs)
File "/opt/conda/envs/midap-mmrazor-torch1.13/lib/python3.9/site-packages/torch/ao/quantization/fx/tracer.py", line 103, in call_module
return super().call_module(m, forward, args, kwargs)
File "/opt/conda/envs/midap-mmrazor-torch1.13/lib/python3.9/site-packages/torch/fx/_symbolic_trace.py", line 434, in call_module
return forward(*args, **kwargs)
File "/root/workspace/mmrazor/mmrazor/models/task_modules/tracer/fx/custom_tracer.py", line 400, in forward
return _orig_module_call(mod, *args, **kwargs)
File "/opt/conda/envs/midap-mmrazor-torch1.13/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/root/workspace/mmclassification/mmcls/models/backbones/efficientnet.py", line 411, in forward
x = layer(x)
File "/root/workspace/mmrazor/mmrazor/models/task_modules/tracer/fx/custom_tracer.py", line 406, in module_call_wrapper
return self.call_module(mod, forward, args, kwargs)
File "/opt/conda/envs/midap-mmrazor-torch1.13/lib/python3.9/site-packages/torch/ao/quantization/fx/tracer.py", line 103, in call_module
return super().call_module(m, forward, args, kwargs)
File "/opt/conda/envs/midap-mmrazor-torch1.13/lib/python3.9/site-packages/torch/fx/_symbolic_trace.py", line 434, in call_module
return forward(*args, **kwargs)
File "/root/workspace/mmrazor/mmrazor/models/task_modules/tracer/fx/custom_tracer.py", line 400, in forward
return _orig_module_call(mod, *args, **kwargs)
File "/opt/conda/envs/midap-mmrazor-torch1.13/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/opt/conda/envs/midap-mmrazor-torch1.13/lib/python3.9/site-packages/mmcv/cnn/bricks/conv_module.py", line 207, in forward
x = self.conv(x)
File "/root/workspace/mmrazor/mmrazor/models/task_modules/tracer/fx/custom_tracer.py", line 406, in module_call_wrapper
return self.call_module(mod, forward, args, kwargs)
File "/opt/conda/envs/midap-mmrazor-torch1.13/lib/python3.9/site-packages/torch/ao/quantization/fx/tracer.py", line 103, in call_module
return super().call_module(m, forward, args, kwargs)
File "/opt/conda/envs/midap-mmrazor-torch1.13/lib/python3.9/site-packages/torch/fx/_symbolic_trace.py", line 434, in call_module
return forward(*args, **kwargs)
File "/root/workspace/mmrazor/mmrazor/models/task_modules/tracer/fx/custom_tracer.py", line 400, in forward
return _orig_module_call(mod, *args, **kwargs)
File "/opt/conda/envs/midap-mmrazor-torch1.13/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/opt/conda/envs/midap-mmrazor-torch1.13/lib/python3.9/site-packages/mmcv/cnn/bricks/conv2d_adaptive_padding.py", line 53, in forward
max((output_h - 1) * self.stride[0] +
File "/opt/conda/envs/midap-mmrazor-torch1.13/lib/python3.9/site-packages/torch/fx/proxy.py", line 298, in bool
return self.tracer.to_bool(self)
File "/opt/conda/envs/midap-mmrazor-torch1.13/lib/python3.9/site-packages/torch/fx/proxy.py", line 174, in to_bool
raise TraceError('symbolically traced variables cannot be used as inputs to control flow')
torch.fx.proxy.TraceError: symbolically traced variables cannot be used as inputs to control flow
To Reproduce
Write PTQ configuration for efficientnet, and execute it.
Post related information
I did some debuggings, and I realized that Conv2dAdaptivePadding operator made the error.
Since "max" and "if" control flow operations, whose inputs are the input of the operator, were used in the operator, fx tracer failed to trace the network.
I think it should be fixed for the future work because not only efficientnet, but transformer networks like ViT also use AdaptivePadding.

@choong-park How did you resolve this ? i am working with mmyolo for openvino quantization.


Support for regular expressions in ingressClasses and storageClasses

Describe the feature
Right now for ingress\storage classes you need to pass an array of distinct values. In our env, when user orders a tenant in k8s cluster, he can specify if he wants dedicated ingress nodes and which ingress he wants (e.g. haproxy\nginx) and then we provision separate nodes with specified ingresses. We want to name these ingress classes using following naming convention: {tenantName}-haproxy, {tenantName-nginx}. So for us, it would be much easier to be able to pass regexp in ingressClasses\storageClasses.
What would the new user story look like?
Tenant admin will be able to create tenants with following options (as per discussion with @prometherion):
ingressClasses:
  allowed:
  - class-with-no-regex
  - another-class
  allowedRegex: ^oil-.*$
storageClasses:
  allowed:
  - class-with-no-regex
  - another-class
  allowedRegex: ^oil-.*$

@prometherion, please assign to me :)

You asked for it, @MaxFedotov!


python:free() invalid pointer when set is_kfusion=True

I have ran successful your code with set is_kfusion=false. Now I want to ran your kinect_fusion.cpp with set this flag to true, but I got error

Have you encoutered same error as me? Could you give some suggestions?
In order to avoid pangolin error, I have comment all pangolin code in kinect_fusion.cpp
Supplement:
I also got  python: free(): invalid next size (fast) if run test_kinect_fusion.sh on  native notebook,, and found that code crash down in initMarchingCubesTables() of create_tensors() by std::cout info

Can you give more methods to test kinect_fusion code (like Video_$1.pango dataset in kinect_fusion/run.sh)?
Another try
I have modified kinect_fusion.cpp/main() image-input-interface and use cv2.imread to replace VideoInput as follows

Then direct run main() function by cmd and error shows cuda_error in initMatchingCubes()

It seems that this error is same with running test_kinect_fusion.py. So all this errors caused by cuda? The CUDA version I installed is cuda-8.0

Have you tried it with Python > 3 ?

Hi, @kevinkit , thank you very much , I will try it later.
Have you run DA-RNN successful with kinect_fusion ? Another error I have encoutered is that LD_PRELOAD can not found libtcmalloc.so.4, could you also give some suggestion?

I had this issue, when running under ubuntu 14.04 , are you running it on ubuntu 16.04?

@JackHenry1992 Like @D0nBilb0  said, this was our case, and no we are still stuck on #7 , even on a native machine

@D0nBilb0 , yes , I am running on ubuntu16.04 and encoutered this error.
@yuxng , it will be very grateful to us for your advice.

@JackHenry1992 regarding to your free() problem, have you checked the tensorflow version #2 ?
Can you maybe give all the steps needed to get it to run on ubuntu 14.04 , we tried that - however we came across many things that needed to be changed, I opened another Issue for that: #10

@kevinkit , I just run test_kinect_fusion.py and also got error. And try direct run executable file (build/kinectFusion) also got error , seems that cuda run error.

What is the compute capablity of your GPU? I read that in some cases textures may not work on smaller compute capablities

This is my notebook gpu params

You can access the details, e.g. compute capability under: https://developer.nvidia.com/cuda-gpus. Your GPU (GeForceGTX 960 M) has a compute capability of 5.0 , a good look what this gpu supports is given here: https://en.wikipedia.org/wiki/CUDA . There are some drawbacks reagring textures with this compute capability (Cache working set per multiprocessor for texture memor,...), , that may not happen at a higher compute capability ( @yuxng used a Titan 1080, which has compute capablity 6) - however this may not be the source of error

@JackHenry1992 refering to your problem with LD_PRELOAD, we get the same error but only as a warning. However, when we tried to start the scripts there were other dependencies which needed to be installed, too (opencv, scipy, Pillow, yaml)
pip install scipy
pip install opencv-python
pip install Pillow
pip install pyyaml

Using tcmalloc speeds the tensorflow training. Otherwise, I saw tensorflow slow down after iterations. However, I also see that using tcmalloc in testing crashed Pangolin. So you can disable tcmalloc when you run kinect fusion in testing.

Thank you for your reply, we ran into similiar errors when trying to run the test script, can you tell how to disable tcmalloc when runing your test scripts?

Do NOT issue the command "export LD_PRELOAD=/usr/lib/libtcmalloc.so.4" when you run the scirpt.

So basically, if my LD_PRELOAD is empty, I should be good to go?
If I simply run
./experiments/scripts/rgbd_scene_multi_rgbd_test.sh 0
which does NOT issue the command...
In a new terminal (so LD_PRELOAD was NOT set by anything before), I still get free() invalid pointer error.

I have tried DA-RNN in GeoForce 1050（ubuntu16.04）， which compability>6, and cannot run kinectfusion...

Hi @JackHenry1992 , I am wondering if you have solved free() issue that you mentioned. I am trying to reproduce the framework and encountered the same problem. Any comments are appreciated. .

Hi @JackHenry1992 , I am wondering if you have solved free() issue that you mentioned. I am trying to reproduce the framework and encountered the same problem. Any comments are appreciated. .

I encountered the same problem too. My GPU is RTX2080Ti with  compute capability 7.5


Improved framework for writing parametric functions

@haozhun can you take a first look, and then assign to @martint?

Just general comment. A bit more verbose commit messages, explaining what was changed by the commit and why, would be very appreciated. It is great help for us (external devs) when we read the codebase.

Addressed your comments, added some details to the commit message, and also added a section to the Developer Guide. Let me know how that looks!

Updated to remove ComparableTypeParameter and OrderableTypeParameter. I also added more docs to the developer guide

Updated with comments

Updated to add support for implementations which are instance methods

Refactored to introduce an ImplementationParser to break up the giant parseImplementation method

Updated

Updated

Updated to fix formatting

Looks good to me.

@martint are we good to merge this?

Go for it, but check the travis failures first.


PSA: Heroku service change at 15 July 2015 may reduce uptime by 25%

CORS Anywhere is currently hosted at Heroku using the free plan. Currently, this plan offers an uptime of 24/7 if the application is actively used.
As of 1 July 2015, all existing free plans will be "upgraded" to a new free plan that limits the uptime to 18 hours a day (i.e. fo 6 hours a day, CORS Anywhere will be offline). This change has been announced at https://blog.heroku.com/archives/2015/5/7/new-dyno-types-public-beta.
I suggest to self-host CORS Anywhere if uptime is critical.
(To maintain the 24/7 uptime for the existing deployment at cors-anywhere.herokuapp.com, 7$ per month needs to be paid. 84$/year for something that doesn't give me any revenue is a bit too much.)

There was some confusion about the flag date.
The documentation provides a clarification: https://devcenter.heroku.com/articles/traditional-dynos (emphasis mine)

Applications that use traditional dynos and that do not have paid usage will be migrated to free dynos starting on July 15th, 2015. However, if you have an application that is using traditional dynos and that has paid usage for dynos, a database or add-ons, you can choose to keep your app on traditional pricing through January 31, 2016. You can experiment with the new dynos at any time for these “grandfathered” apps by going to Dashboard and clicking edit on the resources page and choosing the dyno type you want. If you want to switch back to traditional pricing during the grandfathering period you can simply edit the resources for the app.

So, it seems that if I stick to the free plan, the reduced availability becomes effective on 15 July, unless I pay 7$/mo. But if I have any paid feature (e.g. a 2$/mo SMS app), I can still use the existing free "traditional plan" until January 2016.
An issue is that Heroku requires a credit card for payment, and does not accept other methods like Paypal...

Does this mean that, even if I were to host this myself using Heroku's free tier, it would be down at undetermined points during the day? Or is that only the case if it's in constant use?
I've never used Heroku before and don't understand the implications of "sleeping" nor do I know how the "dyno hours" are counted. If it sleeps after 30 minutes of inactivity, is there a delay to being woken up? Does it consume at least 30 minutes of "dyno hours" every time it's woken up?

My public demo appears to still be up 24/7. So this issue is a non-issue.
From https://www.heroku.com/pricing :

Sleeps after 30 mins of inactivity

My demo is constantly being used, so it's not affected. When it was, the first request takes a few seconds longer to process, but after that you won't notice the difference.

Uses an account-based pool
of free dyno hours

This is explained as:

Verified accounts come with a monthly pool of 1000 Free dyno hours; unverified accounts receive 550. When a Free dyno is active, it draws from the pool; your free apps can run as long as there are dyno hours left in your monthly pool.

I guess that my account has been verified, because one month has about 24x30 = 720 hours, and my demo is still up 24/7.


add HuggingGPT Agent

Description: add HuggingGPT Agent, see the paper and the original project.
Notebook: add a notebook showing its use.

@baskaryan Merge from #7864 to resolve conflicts. Please review the code and merge PR .

@hwchase17 Appreciate you reviewing the code. I have fixed some import errors.


[DNM] Adding codeowners file

Adds the following codeowners:

Team 1 for any changes in the repo
Team 3 for any charts changes

@MbolotSuse Team 3 shouldn't be a code owner in this repo or the webhook chart.


Test ci

have been debugging #290 and i have hit a few issues with centos
centos7
seems to be using systemd as such it needs a privileged container.  Has this always been the case
centos8
"Error: Failed to download metadata for repo 'appstream': Cannot prepare internal mirrorlist: No URLs in mirrorlist"
Looks like its related to EOL.  creating this CR to see if things build by default

Seems centos is working now so will close this down


How to show parameter hint for custom language

I want to show parameter hint for a custom language. I set parameterHints to true but doesn't know what to do next.

@naumanumer You need to follow a pattern similar to providing suggestions for example, but in your case it would be monaco.languages.registerSignatureHelpProvider
Example of completion item provider:
https://microsoft.github.io/monaco-editor/playground.html#extending-language-services-completion-provider-example
Example of hover provider:
https://microsoft.github.io/monaco-editor/playground.html#extending-language-services-hover-provider-example


Added Rust dependabot

This PR is focused on adding the Dependabot system that will check our rust and Github action dependencies for updates to crates and workflows.
It should then create appropriate PR fixes to our repository for us that we can approve on a case by case basis

I believe this needs to be merged into main in order to get it to run correctly.


NuGet transitive dependencies not covered

The newer version of NuGet (which came along around the time of .net core but is the suggested style for .net framework, .net core and .net standard) has support for transitive dependencies.
https://www.erikheemskerk.nl/transitive-nuget-dependencies-net-core-got-your-back/
This means that not all of the packages a project is dependent on are listed in the csproj/fsproj file. Unfortunately this project doesn't seem to make any deep checks on what these transitive dependencies are and so could miss vulnerable packages which have been included.
I'm not aware that the OSSIndex vulnerabilities listings account for these transitive dependencies so it would be a good idea to warn people about this issue (maybe in a big bold font?) before some kind of fix can be found?
This guy seems to have it figured out:
https://www.jerriepelser.com/blog/analyze-dotnet-project-dependencies-part-2/
And in fact he seems to have a similar project which lists outdated packages:
https://github.com/jerriep/dotnet-outdated
Hopefully this issue can be sorted at some point soon? I'm afraid I don't have a load of time to work on it at the moment.

Thanks for the report (and the PR). We will get on these shortly. Updating DevAudit in a variety of ways are approaching the top of the list again.
Sorry for the delay.

I am cleaning up older issues at the moment. The newest released version should have the transitive fix.


Remove unused precondition combination

Found this one by checking the alert.

I've triggered a build for you. Click here to see all failures if there's any.


Topic resources and topic pdf content

Topic resources (issue #98)
Topic pdf content (issue #61)
Policies for lessons and topics, whitespace and readability fixes, sorted imports, etcetc

Add PDF and some resources to seeder https://github.com/EscolaLMS/Courses/blob/main/database/seeders/CoursesSeeder.php then merge


[AS-1555] Analytics tracking for Fairs work on iOS

I've added all the events for iOS in the spreadsheet - https://docs.google.com/spreadsheets/d/1_VDAmRJz2K87FXDq7oh_tQSgwBv4syuZjaaR3sfetcI/edit#gid=849940367
@louislecluse my only question for you is if you think tappedView is an okay name for tapping on a tab. We have these tabs on the artist pages too, so I know that we'll be seeing more of these tabs throughout the ecosystem. I could also do tappedTab or tappedNewView, but there is already a tappedTabBar event for the icons at the bottom of the app.

Same problem for web, I looked at the currently firing event and it is "click" with context_module: NavigationTabs.
So I am creating a clickedNavigationTab event.
I think tappedView makes sense for iOS!

Ooo good call - I'm going to change it to tappedNavigationTab actually!

:rocket: PR was released in v1.37.0 :rocket:


[SPARK-9871][SparkR] Add expression functions into SparkR which have a variable parameter

Summary

Add lit function
Add concat, greatest, least functions

I think we need to improve collect function in order to implement struct function. Since collect doesn't work with arguments which includes a nested list variable. It seems that a list against struct still has jobj classes. So it would be better to solve this problem on another issue.
JIRA
[SPARK-9871] Add expression functions into SparkR which have a variable parameter - ASF JIRA

Merged build triggered.

Merged build started.

Test build #40858 has started for   PR 8194 at commit 2a5f084.

Test build #40858 has finished for   PR 8194 at commit 2a5f084.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
 implicit class StringToColumn(val sc: StringContext)

Merged build finished. Test PASSed.

@shivaram could you review it?

Thanks @yu-iskw. I left some comments inline. One more thing we need to do is to update our NAMESPACE file [1] with all the functions we have added so far (in this PR and in the previous one).
[1] https://github.com/apache/spark/blob/master/R/pkg/NAMESPACE

Merged build triggered.

Merged build started.

Test build #40938 has started for   PR 8194 at commit 50ed0e5.

Test build #40938 has finished for   PR 8194 at commit 50ed0e5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40938/
Test PASSed.

Merged build triggered.

Merged build started.

Test build #40939 has started for   PR 8194 at commit dc98a13.

Test build #40939 has finished for   PR 8194 at commit dc98a13.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/40939/
Test PASSed.

Merged build triggered.

Merged build started.

Test build #41002 has started for   PR 8194 at commit bf59a84.

Test build #41002 has finished for   PR 8194 at commit bf59a84.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41002/
Test PASSed.

@shivaram could you review it?

Define lit as a S4 function
Add the functions into the NAMESPACE file
Remove the unnecessary generic for struct

I have a question about column and col function in SparkR. Should we define them as S4 functions? They are currently defined as S3 functions.

Thanks @yu-iskw - I had a minor comment about the NAMESPACE but otherwise this LGTM

@shivaram thank you for your comment. I updated it!

Merged build triggered.

Merged build started.

Test build #41006 has started for   PR 8194 at commit 36407ef.

Test build #41006 has finished for   PR 8194 at commit 36407ef.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41006/
Test PASSed.

LGTM Merging this

Thank you for merging it!


BoAT User Guide-Software Dependency

Describe the bug
在Ubuntu下，可以使用apt install命令安装相应的包。在Cygwin下，使用Cygwin自带的Setup程序进行安装。
On Ubuntu, can use the apt install command to install the corresponding package. Under Cygwin, use the Setup program that comes with Cygwin to install.
Better translation could be:
On Ubuntu OS, you can use the apt install command to install the corresponding package. Under Cygwin OS, use the Setup program that comes with Cygwin to install.

Cygwin is not an operate system. Mey be the sentence should be On Ubuntu OS, you can use the apt install command to install the corresponding package. Under Cygwin, use the Setup program that comes with Cygwin to install..


Hyper-V Android emulator support

Describe the Bug
I see a lot of issues posted here about having trouble installing HAXM and the solution being to disable Hyper-V.
Disabling Hyper-V is something I would rather not do, so I searched the Web for "HAXM and Hyper-V."
I found this Microsoft page: https://devblogs.microsoft.com/visualstudio/hyper-v-android-emulator-support/.
It says that the "Windows 10 April 2018 Update... enables developers with Hyper-V enabled on their machines to use a hardware accelerated Android emulator, without needing to switch to Intel’s HAXM hypervisor."
The HAXM installer should check for this and advise that HAXM does not need to be installed.
Host Environment

HAXM version: 7.7.-

Host OS version: Windows 10 Pro version 21H1

Host OS architecture: x86_64

Host CPU model: i7-11700K

Host RAM size: 32 GB

(Optional) Host computer model: MSI Z590-A PRO motherboard

Guest Environment

Just trying to get Android Studio 4.2.2 installed. I got this error message:
...
Android SDK is up to date.
Running Intel® HAXM installer
Intel HAXM installation failed!
For more details, please check the installation log: C:\Users\danjam\AppData\Local\Temp\haxm_install-20210724_1835.log
Intel® HAXM installation failed. To install Intel® HAXM follow the instructions found at: https://github.com/intel/haxm/wiki/Installation-Instructions-on-Windows
To Reproduce
Install Android Studio 4.4.2
Expected Behavior
An install log with no errors.
Reproducibility
Don't know
Diagnostic Information
Host crash dump:
HAXM log:
Output folder: C:\Program Files\Intel\HAXM
Extract: checktool.exe
Execute: C:\Program Files\Intel\HAXM\checktool.exe --verbose
CPU vendor          *  GenuineIntel
Intel64 supported   *  Yes
VMX supported       -  No
VMX enabled         -  No
EPT supported       -  No
NX supported        *  Yes
NX enabled          *  Yes
Hyper-V disabled    -  No
OS version          *  Windows 10.0.19043
OS architecture     *  x86_64
Guest unoccupied    *  Yes. 0 guest(s)
Execute: cmd.exe /c type "F:\haxm-windows_v7_7_0\haxm_install-20210725_1258.log" > "C:\Users\danjam\AppData\Local\Temp\haxm_install-20210725_1258.log"
Delete file: F:\haxm-windows_v7_7_0\haxm_install-20210725_1258.log
Additional context

None

HAXM is integrated to Android studio, by default the Hyper-V is disabled for windows so AVD should leverage HAXM for acceleration. If you enable Hyper-V manually, the AVD used the window WHPX for acceleration. It makes sense.


How to change color DotSpan with selected day

I want change color DotSpan when I selected day
I want look like:

But ...

I had that issue too and it is solve thanks to DayViewDecorator

public class EventDecorator implements DayViewDecorator {

    private final  ArrayList<CalendarDay> dates;
    private final Context context;

    public EventDecorator(Context context, ArrayList<CalendarDay> dates) {
        this.dates = dates;
        this.context = context;
    }

    @Override
    public boolean shouldDecorate(CalendarDay day) {
        return dates.contains(day);
    }

    @Override
    public void decorate(DayViewFacade view) {
        Drawable drawable = ContextCompat.getDrawable(context,R.drawable.calcircleselector);
        assert drawable != null;
        view.setBackgroundDrawable(drawable);
    }
}

Just set "calcircleselector" drawable with the desired colors & shape
You can set multiple dayViewDecorators to the calendar, each one will represent a set of dates with a custom color
EventDecorator eventDecorator = new EventDecorator(getContext(), calDaysArr);
        binding.calendarView.addDecorator(eventDecorator);

You can show me "calcircleselector.xml" . please

I had that issue too and it is solve thanks to DayViewDecorator

public class EventDecorator implements DayViewDecorator {

    private final  ArrayList<CalendarDay> dates;
    private final Context context;

    public EventDecorator(Context context, ArrayList<CalendarDay> dates) {
        this.dates = dates;
        this.context = context;
    }

    @Override
    public boolean shouldDecorate(CalendarDay day) {
        return dates.contains(day);
    }

    @Override
    public void decorate(DayViewFacade view) {
        Drawable drawable = ContextCompat.getDrawable(context,R.drawable.calcircleselector);
        assert drawable != null;
        view.setBackgroundDrawable(drawable);
    }
}

Just set "calcircleselector" drawable with the desired colors & shape
You can set multiple dayViewDecorators to the calendar, each one will represent a set of dates with a custom color
EventDecorator eventDecorator = new EventDecorator(getContext(), calDaysArr);
        binding.calendarView.addDecorator(eventDecorator);

You can show me "calcircleselector.xml" . please

Sure, it is just a simple oval shape, and certainly customise it according to your needs
<?xml version="1.0" encoding="utf-8"?>
<shape xmlns:android="http://schemas.android.com/apk/res/android"
    android:shape="oval">
    <solid
        android:color="@color/color_blue"/>
</shape>

how to set the default appointment color blue and when i select the date the blue will be changed into white.


feat: community page

As promised in #76, I've made a community page. It features: a "subtron" at the top which is dark and light mode friendly; a section on the different communities one can become invlovled in; and some language communities that one can join. This should be a good starting point for a community section, and I would like to be let know of any amendments that can be made to make it a better one.
This is my first PR on this repository, and it probably won't be the last 😃.
Thank you for reading.

I'm sorry, I've been very busy for the past few days. I'll be able to get to this tomorrow hopefully.

I thought I'd get more free time, but I haven't (I've only gotten more busy), and I think it would be better if I let it to you guys. In any case, thank you for your patience and I hope I've helped you.

No problem @validity-check !


Artefacts with materials outputting negative emissive values

Hello! We integrated version 1.4.5 and found some artefacts when using materials with the "Allow Negative Emissive Color" checked and when trying to ouput any values below 0 in that material. When moving around the mesh with the material it will result in bright random color flashing.

Thank you for reporting, it is a known issue and should be fixed in next release.

Hello! We recently upgraded to 1.3.1 sdk and I'm still observing the issues when using negative emissive values. Do you know the sdk version where this is planned to be fixed?

Really sorry, the fix is in version 1.3.2, but it is postponed for some reason. A new release of XeSS SDK will be ready soon.

As a workaround, you can find libxess.dll version 1.3.2 in the latest patch of game Black Myth: Wukong if you have purchased it already: https://store.steampowered.com/news/app/2358720/view/527580578293220145?l=english


Update dependencies

Codecov Report

Merging #27 (5397bda) into main (34acf0f) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files


feat: added wave.svg in sign-up.js

Short Description
Added wave image at the end of the Sign-up page.
Fixes Issue
Closes #58
Changes proposed

Added wave.svg in sign-up.js
Place the wave image at the bottom.

Screenshots

You did a great job.
Thanks for your contribution
hope to see you more in the future


Throw TypeError if we get a Number to spanner.date.

Fixes #2097.

Coverage increased (+2.0e-05%) to 99.924% when pulling 326c64ed5a02837d6034d71ddafa15ce8865eda3 on lukesneeringer:spanner-date into a3c455f765822a24d87adb3c2469b5e31b8288f2 on GoogleCloudPlatform:master.

Thank you, LGTM, I just need to make sure my code style commits didn't break the tests before merging.

Coverage increased (+2.0e-05%) to 99.924% when pulling f0b53851af3884cdc4444442d8848b8716331f21 on lukesneeringer:spanner-date into a3c455f765822a24d87adb3c2469b5e31b8288f2 on GoogleCloudPlatform:master.


Data Frame -  summarize-columns example

in  https://lisp-stat.dev/docs/getting-started/data-frame/  documentation
(asdf:load-system :lisp-stat)
(in-package :ls-user)
(defdf mtcars (read-csv rdata:mtcars))
(summarize-column 'mtcars:mpg)
`The value
#(21 21 22.8d0 21.4d0 18.7d0 18.1d0 14.3d0 24.4d0 22.8d0
19.2d0 17.8d0 16.4d0 17.3d0 15.2d0 10.4d0 10.4d0 14.7d0
32.4d0 30.4d0 33.9d0 21.5d0 15.5d0 15.2d0 13.3d0 19.2d0
27.3d0 26 30.4d0 15.8d0 19.7d0 15 21.4d0)
is not of type
SYMBOL
[Condition of type TYPE-ERROR]
Restarts:
0: [RETRY] Retry SLIME REPL evaluation request.
1: [*ABORT] Return to SLIME's top level.
2: [ABORT] abort thread (#<THREAD "repl-thread" RUNNING {25C391B1}>)
Backtrace:
0: (DATA-FRAME::SUMMARIZE-GENERIC-VARIABLE #(21 21 22.8d0 21.4d0 18.7d0 18.1d0 ...))
1: (SB-INT:SIMPLE-EVAL-IN-LEXENV (SUMMARIZE-COLUMN (QUOTE MTCARS:MPG)) #)
2: (EVAL (SUMMARIZE-COLUMN (QUOTE MTCARS:MPG)))
--more--`

To make this work, you can use the 'augmented' version of the mtcars data set. You can do that with (load #P"LS:DATASETS;ls-mtcars").
Dropping into the debugger when you don't have the extra information isn't what we want though, so I'll have to look into this and add some checks.


chore: bdd-ghfjxy-import-1586294642 to 0.0.1

chore: Promote bdd-ghfjxy-import-1586294642 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Add bash autocompletion to OpenStack clients

This patch adds bash autocompletion to the following Openstack clients:
Cinder
Glance
Heat
Keystone
Neutron
Nova

Bash completion will be available in all enviroments where the OpenStack
clients are installed. The autocompletion scripts are placed into a new
directory, and are sourced only if their respective client is present in
the enviroment.
Closes-issue: https://github.com/rcbops/rpc-openstack/issues/469
Replaces-pull-request: https://github.com/rcbops/rpc-openstack/pull/193

:+1:

:+1:


Ferry cannot decompose fragment on TypeCondition

Assume the following schema :
  interface GenericValue {
    name: String
  }

  type TextValue implements GenericValue {
    name: String
    text: String
  }

  type NumberValue implements GenericValue {
    name: String
    number: Int
  }

  type Test {
    value: GenericValue
  }

  type CrashTest {
    a: Test
    b: Test
  }

  type Query {
    fixme: CrashTest
  }

And now, in your app, you query it like
fragment Text on TextValue {
  name
  text
}

fragment Number on NumberValue {
  name
  number
}

fragment CrashTestFragment on Test {
  value {
    ...Text
    ...Number
  }
}

query Test {
  fixme {
    a {
      ...CrashTestFragment
    }
    b {
      ...CrashTestFragment
    }
  }
}

Building the following will result in
[SEVERE] ferry_generator:graphql_builder on lib/src/views/questionnaires/questionnaires.graphql:

Bad state: No element
dart:collection                                          ListBase.firstWhere
package:gql_code_builder/src/operation/data.dart 457:8   _getFieldTypeNode
package:gql_code_builder/src/operation/data.dart 153:24  buildSelectionSetDataClasses.<fn>
dart:core                                                Iterable.toList
package:gql_code_builder/src/operation/data.dart 169:5   buildSelectionSetDataClasses
package:gql_code_builder/src/operation/data.dart 235:22  buildSelectionSetDataClasses.<fn>
dart:core                                                List.addAll
package:gql_code_builder/src/operation/data.dart 234:10  buildSelectionSetDataClasses
package:gql_code_builder/src/operation/data.dart 235:22  buildSelectionSetDataClasses.<fn>
dart:core                                                List.addAll
package:gql_code_builder/src/operation/data.dart 234:10  buildSelectionSetDataClasses
package:gql_code_builder/src/operation/data.dart 235:22  buildSelectionSetDataClasses.<fn>
dart:core                                                List.addAll
package:gql_code_builder/src/operation/data.dart 234:10  buildSelectionSetDataClasses
package:gql_code_builder/src/operation/data.dart 24:10   buildOperationDataClasses
package:gql_code_builder/data.dart 36:17                 buildDataLibrary.<fn>
dart:core                                                Iterable.toList
package:gql_code_builder/data.dart 46:8                  buildDataLibrary
package:ferry_generator/graphql_builder.dart 104:22      GraphqlBuilder.build

Until this is resolved, a workaround is to wrap your fragment with the type condition :
fragment CrashTestFragment on Test {
  value {
    ... on TextValue { ...Text }
    ... on NumberValue { ...Number }
  }
}

In case it's useful, source of graphql manifest : https://spec.graphql.org/October2021/#sec-Type-Conditions

the issue is here in buildSelectionSetDataClasses
    final typeDef = getTypeDefinitionNode(
        schemaSource.document,
        type,
      )!;
      final typeNode = _getFieldTypeNode(
        typeDef,
        node.name.value,
      );

we only get the type definition node for the type A the is queried, but the selection may come from a fragment that is on a subtype B of A
So, when we lookup the queried field in A, we can't find it, as it is only in B.
We need to check the selections list for FragmentSpreadNode, and use the type from that FragmentSpreadNode instead when possible.
Edge cases to think about:

what if selections occur with different types in different fragment spread nodes?


How to return bool instead of GraphNode

hello, mingyang,
I am very interested in your project, I am implementing smooth RM+, but I don't know how to compare GraphNode and float, such as
a = LiteEFG.sum(LiteEFG.maximum(self.regret_buffer + gradient - ev, 0.0))
a<=1.0

always returns GraphNode instead of bool, how to solve this problem?

LiteEFG is based on the computation graph so that the code scripts in init() are defining the graph instead of doing computation. Therefore, a<=1.0 will return a GraphNode with its value equal to a<=1.0

Thanks for your reply. How do I implement smooth RM+ in this case?

I guess you are referring to stable Predictive RM. To implement
if regret(a)<=1
    regret(a)=1

you may use LiteEFG.maximum(regret, 1.0).

No, I am referring to smooth Predictive RM+.  I have to determine whether
a = LiteEFG.sum(LiteEFG.maximum(self.regret_buffer + gradient - ev, 0.0))

is less than 1 or greater than 1 to decide whether to use
self.regret_buffer.inplace(LiteEFG.maximum(self.regret_buffer + gradient - ev, 0))

or
self.regret_buffer.inplace(self.regret_buffer + gradient - ev)
self.regret_buffer.project(distance="L2")

Additionally, to my knowladege, to implement stable Predictive RM+, we have to implement
if \sum_{a} regret(a)<=1:
    regret(a) = 1/(\sum_{a} 1 )

Therefore, you code LiteEFG.maximum(regret, 1.0) cannot be used to implement stable Predictive RM+.

Got it. You may try to do this for smooth Predictive RM+
            flg = LiteEFG.sum(LiteEFG.maximum(self.regret_buffer, 0.0)) >= 1.0
            self.flg_true = LiteEFG.maximum(self.regret_buffer, 0.0)
            self.flg_false = LiteEFG.project(self.regret_buffer, distance="L2")
            self.regret_buffer.inplace(self.flg_true * flg + self.flg_false * (1.0 - flg))

You may also use the same trick to implement stable Predictive RM+.

Thank you very much for your reply


Examples

Dear @karlmartens
Thank you for your outstanding work.
It would be great if you could add samples on how to use this plugin? Not all of dotnet developers could read source code in java.
Thanks

This is a good idea I will add some documentation.

I recently used this in another project and can link the build.gradle file when it is published.


fix rstudio link

@betatim sorry about the broken link, here is the fix

🍻 !

It entertains me far more than I'd ever admit in public to work together with someone who has the same surname as me but in German ... except you live in the US where they all speak English and I am in Switzerland where they speak German ... too easily amused :)

Haha, the irony did occur to me, besonders da ich aus Freiburg im Breisgau bin ;)


Add a private method macro

Currently it doesn't seem possible to create a method that is only visible inside the gobject impl. This is needed to e.g. implement private methods that accesses the properties without needing to make them public.

My suggestion would be to do this inside the module:
impl super::MyObject {
    fn blah(&self) { /* ... */ }
}

Oh, yeah, that works great!


217-providers-suggest-order

Resolves enhancement issue #217.
Changes include:

Now provider suggestion results are housed within a new DOM element child container for each provider, in order to maintain the configured order of providers for suggested results.
Clearing the parent this._suggestions is handled a little bit differently.  Instead of its innerHTML being set to '', each of the providers' child container is cleared out; again, this is to maintain the original configured order.

Some questions which might create more work for another PR(s):

I can't tell how the clearSuggestions method is being actively used. Is this a holdover from previous functionality? https://github.com/Esri/esri-leaflet-geocoder/blob/87c9b7ed5335cfba88631b757c41a1fbb3519493/src/Controls/Geosearch.js#L157  It is being called from here but doesn't quite make sense to me. https://github.com/Esri/esri-leaflet-geocoder/blob/87c9b7ed5335cfba88631b757c41a1fbb3519493/src/Classes/GeosearchCore.js#L107
Unrelated to this PR, there are cases when suggestions should be generated but are not.  This is due to this._lastValue being perhaps a little too strict and its value could maybe be set in more places. Consider a user experience when (1) getting suggestions then (2) blurring from the input element then (3) returning to the input element and attempting to get fresh suggestions for the exact same search text.   https://github.com/Esri/esri-leaflet-geocoder/blob/87c9b7ed5335cfba88631b757c41a1fbb3519493/src/Controls/Geosearch.js#L361-L362

@jwasilgeo what's a good way to see this in action? What code could/should I add to debug/sample.html?

Good question.  I would alter the contents of the providers config array in the sample page.
The order you set here should persist top-to-bottom in the suggestions ui panel (instead of the top-to-bottom order being defined by the order of suggest responses).
https://github.com/Esri/esri-leaflet-geocoder/blob/87c9b7ed5335cfba88631b757c41a1fbb3519493/debug/sample.html#L38
(Also tagging @jgravois for some of my history questions above.)

Code looks good to me, and it looks like that basic use case is working, when I change the order of providers I get this different behavior:

Looks like we do not have unit tests or e2e testing around how the DOM looks in the suggestion dropdown, so we will not worry about that for now (although it might be nice to add that in the future).

@gavinr @jgravois thank you both for your reviews and comments!  I'm going to keep this open for now and push some more changes.

@gavinr @jgravois I just pushed 3 more commits, each of which I hope have self-explanatory messages. If either of you could review again I'd really appreciate it.

@jgravois @gavinr I'm going to write a fresh list of changes now that I added more commits after your all's earlier reviews.  Thoughts?
Changes include:

Now provider suggestion results are housed within a new DOM element child container for each provider, in order to maintain the configured order of providers for suggested results.  (And using div instead of span when possible.)
Clearing the parent this._suggestions is handled a little bit differently.  Instead of its innerHTML being set to '', each of the providers' child container is cleared out; again, this is to maintain the original configured order.
Moved the hiding of the loading indicator to the end of all suggest requests.
Added more error handling UI cleanup when a provider suggest request fails.
Wrapped up some repeated lines into methods and removed no longer used lines of code.
Assigned this._lastValue when input text length is too short to perform suggestions on, so that the user can search again with the same text when it gets long enough, e.g. typing and expecting suggestions for "sa" (yes), then "s" (no), then "sa" again (yes).

given that @gavinr approved and you addressed my earlier feedback, I'd say 'merge at will'

thank you both


Fix wrong reset of command buffers in sample fragment_shading_rate_dynamic.

Description
Without this fix, I got the following validation layer error when running the fragment_shading_rate_dynamic sample:
Validation Error: [ VUID-vkQueueSubmit-pCommandBuffers-00070 ] Object 0: handle = 0x223ba20f060, type = VK_OBJECT_TYPE_COMMAND_BUFFER; | MessageID = 0xaeecdc0b | vkQueueSubmit(): pSubmits[0].pCommandBuffers[0] VkCommandBuffer 0x223ba20f060[] is unrecorded and contains no commands. The Vulkan spec states: Each element of the pCommandBuffers member of each element of pSubmits must be in the pending or executable state (https://vulkan.lunarg.com/doc/view/1.3.275.0/windows/1.3-extensions/vkspec.html#VUID-vkQueueSubmit-pCommandBuffers-00070)
The reason was, that the call to vkResetCommandPool not only reset the small_command_buffers, but also the compute_buffer[].command_buffers.
Creating the command_pool with VK_COMMAND_POOL_CREATE_RESET_COMMAND_BUFFER_BIT and resetting the small_command_buffers by calls to vkResetCommandBuffer resolves that error.
Build tested on Win10 with VS2022. Run tested on Win10 with NVidia GPU.
General Checklist:
Please ensure the following points are checked:

[x] My code follows the coding style
[x] I have reviewed file licenses
[x] I have commented any added functions (in line with Doxygen)
[x] I have commented any code that could be hard to understand
[x] My changes do not add any new compiler warnings
[x] My changes do not add any new validation layer errors or warnings
[x] I have used existing framework/helper functions where possible
[x] My changes do not add any regressions
[x] I have tested every sample to ensure everything runs correctly
[x] This PR describes the scope and expected impact of the changes I am making

Note: The Samples CI runs a number of checks including:

[x] I have updated the header Copyright to reflect the current year (CI build will fail if Copyright is out of date)
[ ] My changes build on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions

Sample Checklist
If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist:

[x] I have tested the sample on at least one compliant Vulkan implementation
[x] If the sample is vendor-specific, I have tagged it appropriately
[x] I have stated on what implementation the sample has been tested so that others can test on different implementations and platforms
[x] Any dependent assets have been merged and published in downstream modules
[x] For new samples, I have added a paragraph with a summary to the appropriate chapter in the readme of the folder that the sample belongs to e.g. api samples readme
[x] For new samples, I have added a tutorial README.md file to guide users through what they need to know to implement code using this feature. For example, see conditional_rendering
[x] For new samples, I have added a link to the Antora navigation so that the sample will be listed at the Vulkan documentation site

Yep, will do.  LMK if issues.


doesnt work

does not work for me, when executing nothing happens!

No info/details/error report. Closing this.


The addon will not auto-sell artifact relics from Legion, which can't even be used anymore

Title. ex: Collapsing Epoch, artifact relic. Will not be auto sold at vendor even though it's useless and unusable.

Duplicate of #106


Variables defined in the body of repeat loop should be visible in until and fixup blocks

Describe the bug
Variables defined in the body of repeat loop are not visible in until and fixup blocks.
To Reproduce
The following loop that used to work in Q# 0.* fails compilation because of unknown variable res:
use q = Qubit();
repeat {
    H(q);
    let res = M(q);
} until res == One;

Expected behavior
Variables defined in the body of repeat loop should be visible in until and fixup blocks. All three blocks are considered parts of the same loop, even though visually they are separate blocks. This makes using the loop a lot more convenient compared to defining a mutable variable before the loop and reassigning its value in the body.
System information

QDK 1.0.33

This was actually an intentional change, as repeat blocks being the one place where variables defined in one block are visible in another block caused some serious problems. The extra logic to handle this exceptional behavior caused led to crashes in name resolution AND required more work to explain why scopes didn't behave intuitively in this one construct. When combined with the fact that variable shadowing is now allowed, the exceptional scope behavior of repeat-loops was also a potential source of errors in the Q# code itself. The newer behavior is much easier to explain and requires no caveats or exceptions: the scope of a variable bound by a let or mutable statement is determined by its containing block.
For reference, this change was part of a bug fix that addressed two crashes found via fuzzing: https://github.com/microsoft/qsharp/pull/355


Very slow dependency resolving

I'm using RC5 version of sbt-coursier (addSbtPlugin("io.get-coursier" % "sbt-coursier" % "1.0.0-RC5") ).
When I issue compile in sbt for the first time after loading it it takes a really long time to resolve dependencies. It looks like this:
[info] Updating someModule (sbt classifiers)
http://repo.typesafe.com/typesafe/releases/org/flywaydb/flyway-sbt_2.10_0.13/4.2.0/flyway-sbt-4.2.0.pom.sha1
    0.0% [          ] 0 B (0 B / s)

I just started complile and it took it several minutes to do all that and actually start compiling (no artifacts were downloaded, just checking metadata).
I restarted sbt and compiled again - it's stuck at
https://repo.typesafe.com/typesafe/ivy-releases/org.flywaydb/flyway-sbt/scala_2.10/sbt_0.13/4.2.0/ivys/ivy.xml.sha1
    0.0% [          ] 0 B (0 B / s)

for a couple of minutes now.
This wasn't happening before RC3 (switched to it from M15).

The https://repo.typesafe.com/... repositories are usually quite slow in my experience too. Although once the corresponding file is in cache (from this repo or another), the https://repo.typesafe.com/... repo shouldn't be checked for it... (but new files are downloaded since 1.0.0-RC1)
sbt usually gives these slow repositories a high priority, but  https://github.com/coursier/coursier/commit/a91b955a9ff7765740b35eae2c286ad55d2b58d4 should now lower it from sbt-coursier. It will be included in 1.0.0-RC6, about to be released.

Cool! I updated to RC6 and it does seem to be faster now :+1:
Out of curiosity - why are those URLs slow? (like http://repo.typesafe.com/typesafe/releases/org/flywaydb/flyway-sbt_2.10_0.13/4.2.0/flyway-sbt-4.2.0.pom.sha1). curling them works pretty fast, lke 300ms for me and definitely not minutes.

The repo.typesafe.com just feels slow compared to Central, in my experience. Plus it doesn't seem to accept several connections in parallel, effectively answering only one of them at once.
(Closing the issue...)


Record entries

Added record_entries and record_generator to produce Dataclasses as output for data generation process.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 91.66667% with 3 lines in your changes missing coverage. Please review.

Project coverage is 96.84%. Comparing base (1927f49) to head (748ca93).

Files
Patch %
Lines

datacraft/builder.py
90.00%
3 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #55      +/-   ##
==========================================
+ Coverage   96.41%   96.84%   +0.43%
==========================================
  Files          70       70
  Lines        3907     3935      +28
==========================================
+ Hits         3767     3811      +44
+ Misses        140      124      -16

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


Columns may not support null values

Columns that contain null values that are returned from GET may be removing entries instead of passing on those nulls.
Reported by @AlexanderPico

Fixed in 6406e38


Increase Echo focusing beam damage

Created by: dubspool
Focus beam damage multiplier is 175% in OW2 but 200% in OW1.

mentioned in merge request ow1_emulator_backup!269

moved from ow1_emulator_backup#607


UTF-8 and ISO-8859-1 combination doesn't work.

I have my charset set to "Try to parse as UTF-8 and as   ISO-8859-1  , send as UTF-8 (recommended)", and UTF-8 works fine, but other people's ISO-8859-1 Scandinavian characters show as unknown characters in both HexChat and XChat Azure clients (both clients set to UTF-8). My ZNC version is 1.6.5+deb1, running on current Debian-stable.

Set it in network settings, not in user settings. User settings is for the
client encoding, between ZNC and your IRC client.
2017-08-11 8:51 GMT+01:00 apliedes<EMAIL_ADDRESS>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/znc/znc/issues/1433, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAT15EhdD7pKjgAglL3F1UbF1UPVjVmGks5sXAgcgaJpZM4O0WDg
.

Doh, that fixed it. Thank you, and sorry about the issue. It might be worthwhile to mention this at the client encoding settings though.

It is mentined there, right above it :(
Do you have any suggestion how to make it more clear?
2017-08-11 10:32 GMT+01:00 apliedes<EMAIL_ADDRESS>
Closed #1433 https://github.com/znc/znc/issues/1433.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/znc/znc/issues/1433#event-1202300722, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAT15ODqADQ6d278I9_4Rl4SBW3vSGf_ks5sXB-bgaJpZM4O0WDg
.

You're not the first one who is confused by this setting.
2017-08-11 10:32 GMT+01:00 apliedes<EMAIL_ADDRESS>
Closed #1433 https://github.com/znc/znc/issues/1433.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/znc/znc/issues/1433#event-1202300722, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAT15ODqADQ6d278I9_4Rl4SBW3vSGf_ks5sXB-bgaJpZM4O0WDg
.

Ah, it's not mentioned in my version (1.6.5+deb1). Hopefully the problem will go away once everyone updates.

It is there :)
https://github.com/znc/znc/blob/znc-1.6.5/modules/data/webadmin/tmpl/add_edit_user.tmpl#L291
2017-08-11 10:43 GMT+01:00 apliedes<EMAIL_ADDRESS>
Ah, it's not mentioned in my version (1.6.5+deb1). Hopefully the problem
will go away once everyone updates.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/znc/znc/issues/1433#issuecomment-321770687, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAT15BcyfYhIS8mfvvdaCkLhqHw-0NvSks5sXCJIgaJpZM4O0WDg
.


Runtime: Implement toplist comparisons API

For details, see (this doc).

Closed by https://github.com/rilldata/rill-developer/pull/2312


Eye patch is not drawn over right eye on some devices

When testing app on my Nexus 6P I noticed that the eye patch was not properly centered over the right eye. I am guessing this is because the x and y offsets are hard coded.
A potential solution is to use scaled height and width from bitmap adjusted to the canvas

Thanks! I was expecting some issues from hardcoding those values. The app was just to show what's possible with the APIs so I initially didn't give that much concern. I will explore that in future commits.


Flask error handler caching leads to inconsistent behavior

When Flask finds an error handler for a given exception, it caches it in the handler map.  This can lead to incorrect behavior for subclasses of the exception.
Here is some code demonstrating the problem:
from flask import Flask
from werkzeug.exceptions import InternalServerError

class E1(Exception):
    pass

class E2(Exception):
    pass

class E(E1, E2):
    pass

app = Flask(__name__)

@app.errorhandler(E2)
def handle_e2(e):
    return "E2", 500

@app.errorhandler(Exception)
def handle_exception(e):
    return "Exception", 500

@app.route("/<exception>", methods=['POST'])
def raise_exception(exception):
    exc = globals()[exception]
    raise exc

def test_errorhandler_precedence():
    client = app.test_client()

    response1 = client.post('/E1')
    assert response1.data == "Exception"

    response2 = client.post('/E')
    assert response2.data == "E2"

In this example, we have two exception classes, E1 and E2, and a third exception E that inherits from both.  We register error handlers for E2 and Exception.  When E is raised, Flask should invoke the E2 error handler as E2 comes before Exception in the MRO of E.  But if E1 has been raised in the past, then the Exception error handler will be "cached" as the handler for E1, so that handler will be invoked instead.

Related to #1433


[CONTENT] Add documentation for the "--dapr-http-max-request-size" argument for the CLI and "dapr.io/http-max-request-size"  annotation

What content needs to be created or modified?
Within this issue https://github.com/dapr/dapr/issues/2659 and this https://github.com/dapr/dapr/pull/2679 I am adding the possibility to define custom fasthttp server MaxRequestBodySize parameter to handle uploading of big files.
Describe the solution you'd like
Add documentation for the --dapr-http-max-request-size argument for the CLI and "dapr.io/http-max-request-size" annotation for the Kubernetes pods
Where should the new material be placed?
https://docs.dapr.io/operations/configuration/

@apuzyrevsky  - Can you create the docs for this update?
Need to update the following topics

https://v1-rc3.docs.dapr.io/operations/hosting/kubernetes/kubernetes-annotations/
Document the CLI command this is used with in the CLI reference docs.
Document in the https://docs.dapr.io/operations/configuration/ topic

Hi, @msfussell

It's already in the link that you specified. Here is the closed PR: https://github.com/dapr/docs/pull/1104
I did not add a CLI command, only a flag.
Here is one more PR: https://github.com/dapr/docs/pull/1183

Resolved. Closed


Clean up and standardize column names

We highly distributed the mapping of PHMSA raw data. This means we probably have some inconsistencies across our column naming conventions we should fix. These include:

Fix ordering of onshore/offshore and transmission/gathering
Check that categorizations are consistent across tables
Check missing and extra column errors for all tables to make sure we haven't mismapped anything

all the date fields
2022 pdf:

DATAFILE_AS_OF | Data as of date
FILING_DATE | System created value: Date when a record was filed with PHMSA DOT
FILING_DATE (duplicated) | Date Submitted
REPORT_DATE | System created value: Date when an Initial record was filed with PHMSA DOT
REPORT_DATE (duplicated) | Initial Date Submitted

pre 2000:

DOE	|	DATE	|	 	Date record was entered into the database. (YYYYMMDD)
DOC	|	DATE	|	 	Date record was changed in the database. (YYYYMMDD)

all the date fields
2022 pdf:

DATAFILE_AS_OF | Data as of date
FILING_DATE | System created value: Date when a record was filed with PHMSA DOT
FILING_DATE (duplicated) | Date Submitted
REPORT_DATE | System created value: Date when an Initial record was filed with PHMSA DOT
REPORT_DATE (duplicated) | Initial Date Submitted

pre 2000:

DOE	|	DATE	|	 	Date record was entered into the database. (YYYYMMDD)
DOC	|	DATE	|	 	Date record was changed in the database. (YYYYMMDD)

@cmgosnell Do you read this as DOE = REPORT_DATE? Sounds like DOC would be equivalent to the FILING_DATE only when the filing was a supplemental filing, so not a 1:1. Or should we treat them as 4 slightly different columns?


Stale/invalid mappings prevent pulling LCSC assignments from the schematic

Describe the bug
Sometimes the tool doesn't pick up LCSC numbers which it has previously written to the schematic. The slightly crowded screenshot below shows the issue. C3 has a part number assigned as shown in the bottom half of the image. In the PCB layout, the tool shows the stock level for this part (C107110), but nothing in the "LCSC" column in the table.

I think this is due to a conflict between the "mapping" file, and the information in the schematic. Deleting all of the mappings (via the "manage mappings" window) has resolved the problem (I think).
KiCad Version
Application: KiCad PCB Editor (64-bit)

Version: (6.0.5), release build

Libraries:
	wxWidgets 3.1.5
	libcurl/7.82.0-DEV Schannel zlib/1.2.12

Platform: Windows 10 (build 19044), 64-bit edition, 64 bit, Little endian, wxMSW

Build Info:
	Date: May  3 2022 00:36:16
	wxWidgets: 3.1.5 (wchar_t,wx containers)
	Boost: 1.79.0
	OCC: 7.6.0
	Curl: 7.82.0-DEV
	ngspice: 36
	Compiler: Visual C++ 1929 without C++ ABI

Build settings:
	KICAD_USE_OCC=ON
	KICAD_SPICE=ON

Does close and reopen the plugin help it that case?

Closing and reopening didn't seem to fix things. The problem went away when I deleted all the mappings (via the "manage mappings" menu), but I'm not very confident on what the exact solution was that made the symptom go away, unfortunately.

So that meany you no longer have a project that has these issue, right?
I guess there's no wa to figure out what went wrong without being able to reproduce it....
I'll close the issue and if you get the same kind of problems again, you can re-open.


[Versioning] 1.1.0

Quick change to put back the versioning of the software.

Hum, since it's written 1.1.0 we can put in the 1.1.0 milestone, but we would have to do a new PR just after to change 1.1.0 to 1.1.1. I agree if you want to put it in 1.1.0, if @fcollot agrees.

From this issue https://github.com/Inria-Asclepios/medInria-public/issues/101

(Flo said yes for the milestone on IRC)


Reducing flutter driver initialization timeout to 1 minute from 1 year

Flutter driver crashes (encountered internally) were causing overly long test runs terminating in a timeout. 1 minute seems like a more reasonable limit for driver initialization.
Global presubmit in internal CL: cl/542926104
Internal bug: https://b.corp.google.com/issues/288599376
Pre-launch Checklist

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

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

We don't use timeouts in our code. The year-long timeout effectively disables timeout on purpose. Instead, a test-global timeout is used to timeout the entire test.
See https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo#never-check-if-a-port-is-available-before-using-it-never-add-timeouts-and-other-race-conditions
You can pass a custom timeout to FlutterDriver.connect (API).

That makes sense. I'll close this and update the internal test instead.


Puppet error "Unknown function: 'mongodb_password'"

I´m trying to upgrade from module version 2.2.2 to the current one, while 3.0.0 did cause some errors as well why i choosed to take the most recent commit to 3.0.1-rc0.
Affected Puppet, Ruby, OS and module versions/distributions

Puppet:
agent => 5.5.17
server => 5.3.10

Ruby:
ruby => {
platform => "x86_64-linux",
sitedir => "/opt/puppetlabs/puppet/lib/ruby/site_ruby/2.4.0",
version => "2.4.9"
}

Distribution:
Bionic

Module version:
3.0.1-rc0 - commit 37dd6f0

How to reproduce (e.g Puppet code you use)
mongodb_user { 'backup':
      ensure        => present,
      password_hash => mongodb_password('backup', $mongo_backup_pw),
      database      => 'admin',
      roles         => ['backup','restore','backup_system_profile'],
}

What are you seeing
Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Unknown function: 'mongodb_password'
Compared to v2.2.2 there are some changes to the script and location:
https://github.com/voxpupuli/puppet-mongodb/blob/master/lib/puppet/functions/mongodb_password.rb
https://github.com/voxpupuli/puppet-mongodb/blob/v2.2.2/lib/puppet/parser/functions/mongodb_password.rb
What behaviour did you expect instead
Function mongodb_password is found.

I can't really explain why this would fail. @alexjfisher do you have any idea why hits could fail?

Since this is puppet 5 and the function uses the modern API, my guess would be the function is being called from a module (eg. a profile module) with a metadata.json containing a dependencies section, and in this section, puppet/mongodb is not listed.  Either add it, or declare no dependencies in your profile (an empty array).
From https://github.com/puppetlabs/puppet-specifications/blob/master/language/func-api.md

Note: A module that wants to call a function in another module must either have that module listed as a dependency, or have no dependencies entry at all in its metadata.json. This requirement is enforced at runtime to help users keep metadata for modules up to date as users then simply need to install a module and its dependencies. If a module does not have the correct dependencies listed the runtime will simply not find the function.

I think this behaviour was removed in puppet 6.

This was an excellent catch. Defining an empty hash did not help, specifying

"name": "puppet-mongodb",
"version_requirement": ">= 3.0.0"

resolved that issue. Thank´s a lot!


Test 80-test_cmp_http.t Kills the perl wrapper on failure

If there is an issue in 80-test_cmp_http.t, the process kills the perl wrapper process instead of the Mock server (if a Mock server is not properly registered. This should happen on any platform.
The error condition is as follows:
# The results of this test will end up in test-runs/test_cmp_http
Starting1..6
Current directory is /home/randall/openssl/test/recipes/80-test_cmp_http_data/Mock
Launching Mock server: ../../../../util/wrap.pl ../../../../apps/openssl cmp -config server.cnf
Mock server PID=1409286238
cmp: error setting accept on port 0
00000000:error:80001011:system library:BIO_socket:Address family not supported:/home/randall/openssl/crypto/bio/bio_sock2.c:51:calling socket()
00000000:error:10000076:BIO routines:BIO_socket:unable to create socket:/home/randall/openssl/crypto/bio/bio_sock2.c:53:
00000000:error:80001011:system library:acpt_state:Address family not supported:/home/randall/openssl/crypto/bio/bss_acpt.c:233:calling socket(::, 0)
00000000:error:10000076:BIO routines:acpt_state:unable to create socket:/home/randall/openssl/crypto/bio/bss_acpt.c:236:
Mock server output: cmp_main:/home/randall/openssl/apps/cmp.c:3080:CMP info: using section(s) 'cmp' of OpenSSL configuration file 'server.cnf'
Killing Mock server with PID=0Killed

This results because the following line:
https://github.com/openssl/openssl/blob/e6b6b18af3e85a6b5f0d8ea1070f7070557d6357/test/recipes/80-test_cmp_http.t#L314
is missing an if $pid.
A pull request is coming.

While there are other issues with this test, the error Killing Mock server with PID=0Killed was indication that perl was killed. The PR #22015 deals with only the perl issue.


Make it easier to override the base url

Is your feature request related to a problem? Please describe.
Yes. I love the theme but it's a pain to use when going through an NGINX reverse proxy with a subfolder (i.e. https://<host>/qbt/). In order to use this I had to:

Fork/clone the repo.
Change the <base> tag in index.html.
Change the root prop in http_config.json
Rebuild

Describe the solution you'd like
It would be great if the "base path" could be more dynamically overridden. Maybe via an HTTP header?

My current setup is to host it on a subdomain which provided the least amount of friction. It should be possible to override the base url, i'll have to investigate a bit.

I'm also using qbittorrent behind a reverse proxy, and am interested in overriding the base url to make this theme work.
@taschmidt could you elaborate on what changes you made to  index.html and http_config.json?

I actually currently use a different theme and I (unfortunately) have a short memory. Taking a quick peek, I believe you would need to change the href attrib on the <base> tag here to your subpath (i.e. "/qbt").
The modifications to http_config.json are less clear to me but I would imagine you would need to change "/api/v2" to "{your sub path}/api/v2" (i.e. "/qbt/api/v2") here.

So, how does one remove this theme if they didn't realize it would break qbittorrent's webui? Just deleting the theme's folder didn't seem to help.

@lightmaster easiest way I found to do it is to find your qbittorrent.[ini|conf] file and clear out the WebUI\RootFolder setting.


Stop RocketChat from stealing cursor

Currently when navigating to any of the keynote / tutorial / paper pages that have an embedded RocketChat, the cursor is grabbed by RocketChat and the page scrolls down past the other content. Would be nice if this did not happen.

Not sure how to fix this, but for what it's worth it only happens if you're logged in.

Definitely just a nice-to-have. I couldn't figure out how to get it either.


Add Xoriant as new partner

Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

@krisfreedain - Created a new PR and figured out the DCO issue . For some reason, I'm unable to do a local build of the site and have updated the xoriant.md file with the error fixed (hopefully). Please kindly check.

thanks @Ram-Kumaar-ML DCO works!
I'm getting an odd error
"project-website/_partners/xoriant.md: (): found character that cannot start any token while scanning for the next token at line 47 column 1"
so I'll investigate that and see what we need to fix.

@Ram-Kumaar-ML - checking in on this and seeing if there is anything I can do to help get your company updated.

@Ram-Kumaar-ML - Just checking in to see if you're able to help resolve this. Looks like there's still an error.


Fixes: Inspector doesn't work with firefox on Linux because of 'meta-shift' as default for all platforms but Windows

The 'meta-shift' shortcut won't work on firefox when used on linux. Also in KDE the shortcut is preset to draw mouse trails on the screen. Switching to control-shift on linux but is a little tricke to work with, because of the way it handles the click to activate the browser window again.
Having tried all sort of options/combinations it seems the best working for all browsers
is the combination of "control-shift" and holdMode: true
I'm stil wondering  if instead of checking
['linux', 'win32'].includes(process.platform) ? 'control-shift' : 'meta-shift',
it would make more sense to check just for mac by doing a
process.platform === 'darvin' ? 'meta-shift' : 'control-shift',
for sure it is cumbersome having to change the options in svelte.config.js manually every time when switching the platform (mac <-> linux)
Discussed some on:
https://discord.com/channels/457912077277855764/1052953876724207676
Note: The preset of meta-shift can be deactivated in systemsettings of KDE but firefox still won't work with it.

unfortunately there is no perfect key-combo for all platforms and browsers. meta-shift works on linux chromium based browsers.
I'd prefer we keep the current defaults. You could PR an addendum to the docs that gets into more detail about fine-tuning the combo for your needs.
In multi-environment setups, you could use process.env.SOME_ENV='my-combo' and keyCombo: process.env.SOME_ENV ?? 'fallback-combo'  to allow yourself or team members to override the default to something that works for them.
In the same direction I don't think holdMode should be changed to on by default on a single platform. If at all it should default to on for all platforms if thats what most users prefer.

unfortunately there is no perfect key-combo for all platforms and browsers. meta-shift works on linux chromium based browsers.
I'd prefer we keep the current defaults. You could PR an addendum to the docs that gets into more detail about fine-tuning the combo for your needs.
In multi-environment setups, you could use process.env.SOME_ENV='my-combo' and keyCombo: process.env.SOME_ENV ?? 'fallback-combo' to allow yourself or team members to override the default to something that works for them.
In the same direction I don't think holdMode should be changed to on by default on a single platform. If at all it should default to on for all platforms if thats what most users prefer.

I agree on the holdMode.
About the shortcut though, right now it is a workaround for mac based computers, crippling the functionality on other OS. As mentioned earlier meta-shift is a pre-registered combination on some DEs. 'control+shift' works for all browsers - with or without holdMode: true. holdMode : true just gave a better user experience.
I'll modify the patch accordingly to keep 'meta-shift'  as the exception just for the mac. That way it will work on more platforms and browsers including firefox + it won't be blocked by the DEs global shortcuts (e.g. KDE).
holdMode: true is a big :+1:  from me. But for sure it depends on what the majority of the user wants.

Now this is probably way better and cleaner:
Now you could simply:
	vitePlugin: {
		experimental: {
			inspector: {
				holdMode: true,
			}
		}
	}

where the holdmode is obiously a choice and have it working on all platforms. Surely it remains tricky and you'd need to find ppl to try it on more exotic platforms and see if there is any bug remaining.

I'd agree in general but argue that it doesn't work at all for many linux users. 1 or 2 days ago I was looking for the this repo and asked in the community chat. There were 2 or 3 more ppl using kde and it didn't work at all for them They were wondering about it. Also firefox is not working either.
I'd also argue as it is in experimental and it is "just" the linux users that get affected the harm might be smaller << but I guess that is a small consolation.
The environment env sounds like a feasible option. I'd still argue that ctrl+shift is more natural to most users inlcuding linux but ok.
summing up:
contra meta-shift on  linux

firefox not working
kde has a global shortcut, blocking it

pro(meta-shift on linux

it is the default since setting the workaround for mac
muscle memory of Linux users.

Me switching OS and computers often somehow don't worry about it as much and I keep adapting quickly, but that is just me.
Ultimately it all depends on documentation + I guess maybe it would be helpful to write a index.d.ts file for the plugins config options to help with intellisense and provide documentation inline.

meta-shift doesn't play along well with edge on linux

meta-shift does nothing for me in Chrome on Linux. I could only get alt-shift and control-shift to work.
I don't know what meta even is. MDN seems to indicate the key would say "meta", but I don't have such a key on my keyboard. I have a Windows key on the external keyboard I use, which doesn't seem to do anything in relation to inspector. I tried both meta-shift and os-shift as I believe the latter is what would be triggered. Neither did anything. I needed some code changes to test os-shift as inspector does not currently support it and it's possible I didn't implement it properly.
I also had a terrible time figuring out that I needed to type control rather than ctrl. It would be nice to accept both variants.

While we could add alias support for some keys, i think it's best to stick to the key event names. control is also heavily documented here https://github.com/sveltejs/vite-plugin-svelte/blob/main/docs/config.md#inspector
"meta" is triggered for me on manjaro-linux chromium by pressing the windows logo key on my keyboard. Not sure why it doesn't for you, but if you want to add support for os, i think it would need to be inserted in this switch and supported by getModifierState https://github.com/sveltejs/vite-plugin-svelte/blob/main/packages/vite-plugin-svelte/src/ui/inspector/Inspector.svelte#L134  (or have name === key, in which case it would already work)

closing in favor of #631  which also adds environment configuration


Create a department

Hi, is there a way to create departments with the library?

@MauriPastorini Is "departments" a setting of cybozu.com? This library cannot manage cybozu.com administration.
If you change this code to BASE_PATH = '/v1/' and use Kintone::Api#get_url, probably it work.
https://developer.kintone.io/hc/en-us/articles/115008478208


Q: Twig Extensions and Twig Functions

Is there a way to make extensions/functions/filters work with melody as well?

So all the custom twig extensions and so on have to be translated to plugin? What about stuff like https://symfony.com/doc/current/translation.html ?

Is there a tutorial to add custom plugins?

How do you work with translations btw?

We have custom plugins that expose a filter for translation strings.


Added .StyleCop file per project instead of global rule set

Fixes #18 : Stylecop doesn't seems to recognize parameters in sln.dotsettings
Changes proposed

This PR contains below items.

Removed older .dotSettings files from src location.
Added specified rule set file to suppress warnings from respective project. I added file for the same.

To fixed this issue I used steps mentioned in this answer: Stackoverflow
Once this PR approves I will add same steps in reported issue as a comment and will close it.

@atulmish , Please review above PR and if it works then merge it.
I tested it locally and by adding these files, warning which are mentioned in .dotsettings file get suppressed.
I added only warnings which were mentioned in .dotSettings file and removed that file.
Please review this PR, if it works on your machine then only merge it to develop.
If did not work, please let me know.


fix colors regressed from old styles

Fix a couple of resources that were unintentionally changed in the old styles.

/azp run


Not including imported files to compile

Hi ,
we have parent.scss and that import all child scss but this plugin not considering all child scss files
is there any way to include child as well
and also how to include multiple src and dest dirs ?
@import
"../../../test-core/scss/test-core",
"../vendor/vendor",
"../components/components";

Hello @venkiiitk, I'm closing the issue as it was left unanswered for months now. If you're still experiencing issue, then feel free to open the issue and answer questions in previous comment.


[FEATURE] LangSmith Threads support

Support for Threads in LangSmith: https://docs.smith.langchain.com/old/monitoring/faq/threads
By configuring a uuid, Ideally provided by the user request or using the LangSmith settings in Flowise.

+1 on this! Would be awesome

agree, this would improve the observability a lot!
It's also fairly easy to implement imo

Any update on this, is there a workaround ?

any update?


A lot of the code within the website is not linted

Do we want to use something like https://github.com/wooorm/eslint-md ?

I doubt that would work as our code is within the front matter rather than code blocks embedded within markdown.

next step for playground is to move the JS to files to try and get us back to one pass Assemble again. That would make linting easy as well


Add Cowbridge Comprehensive School Sixth Form

Add Cowbridge Comprehensive School Sixth Form
http://www.cowbridgecomprehensiveschool.co.uk/cms/_files/AS Options Booklet September 2017.pdf
Computer Science Page 19
Information and Communication
Technology (ICT) Page 43
Both of which are 2 Year A Level Courses

Email addresses are
@cowbridgecomprehensiveschool.co.uk
@cowbridgecs.co.uk
Both of which are used by teachers and pupils

Format of email
Pupil -<EMAIL_ADDRESS>Pupil -<EMAIL_ADDRESS>Staff  -<EMAIL_ADDRESS>Staff  -<EMAIL_ADDRESS>For example Foo Bar joining in 2017 would be
Pupil -<EMAIL_ADDRESS>Pupil -<EMAIL_ADDRESS>Staff  -<EMAIL_ADDRESS>Staff  -<EMAIL_ADDRESS>

@andrew159159 I am sorry for the delay with the request processing. Pull request is now merged. Please notice that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age. Thank you!


Is array-find-index polyfill necessary?

https://www.npmjs.com/package/array-find-index
https://github.com/benwiley4000/cassette/blob/fa9c48fdc13009972a80f54ca227d68c807d3cef/packages/core/src/utils/findTrackIndexByUrl.js#L1
From what I can see in the webpack config for core, @babel/preset-env preset is used with targets: undefined.

Sidenote, if no targets are specified, @babel/preset-env behaves exactly the same as @babel/preset-es2015, @babel/preset-es2016 and @babel/preset-es2017 together (or the deprecated babel-preset-latest).

Wouldn't Array.prototype.findIndex be transpiled by babel?

Good question!
Array.prototype.findIndex is part of the standard library, not the syntax, so it needs to be polyfilled. Sometimes people use babel-polyfill for this (which is different than the babel syntax transform).
Sooner or later I need to do an inventory of all the newer web platform features I'm using and figure out which ones actually need to be polyfilled and let people bring their own polyfills for the rest. But I think that would be a separate issue.

Are we ok to close this issue? Please let me know if you think I overlooked something. :)

Thanks for the answer, yes, it's babel-polyfill that takes care of that.
If the core-js package were used, the library could use the native implementation if it's present or polyfill it if it's not, doing something like import 'core-js/fn/array/find-index';, and that way you would more easily know which polyfills you use (for an inventory later on) by just looking for core-js.
But that would pollute global namespace and you as library author might not want to do that, in that case, I think core-js can also be used to import some polyfills as functions, that way there's no pollution but keep the perk of using just one package for pollyfills and make them easier to track.

Btw there are some other things that need to be polyfilled in certain cases - requestAnimationFrame, cancelAnimationFrame, Map and Set - but those are already dependencies of React 16 so I'm assuming they exist in the global scope.


Updating User registry API

https://github.com/agstack/agstack-website/issues/15

@MHA10 adding the user registry API. I made some updates for the descriptions

Merging this PR to check website update