Manhuaplus.org not working

Source information
Manhuaplus.org
Source language
English
Steps to reproduce
Don't download
Expected behavior
The download should happen
Actual behavior
File downloading
Mihon/Tachiyomi version
0.16.3
Android version
Lineageos 21 (Android 14)
Other details
I'm using a raspberry pi 4B
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

Sorry on my errors in t
How to produce the bug simply download the first chapter of any manhua it will say the number of page but not downloading anything

the issue is not from extension.
chek your network or your firewall, if you use an adblock in raspberry pi chek if it block download.

cannot reproduce


[Feature] 嵌套子表格

What problem does this feature solve?
https://ant.design/components/table-cn#table-demo-nested-table
What does the proposed API look like?
无

感谢你的反馈！我们已经收到你的问题，并将在后续进行处理。
如果有开发者对这个 issue 感兴趣，请在issue下方留言“认领该issue”，留言后开始你的创作提pr给我们！
欢迎大家参与开源共建！我们非常感激每一位贡献者，提交pr后我们将为参与共建的开发者准备一份礼品。
参与共建可参考：https://visactor.io/vtable/guide/Contribution_Guide
与官方开发者交流问题可以下载飞书app扫码加入飞书群：

再次感谢你的支持！
Thank you for your feedback! We have received your question and will deal with it later.
If any developer is interested in this issue, please leave a message "claim this issue" below the issue. Welcome to participate in open source co-construction!
We are very grateful to every contributor. After submitting the PR, we will prepare a gift for the developers who participated in the co-construction.
For participation in co-construction, please refer to: https://visactor.io/vtable/guide/Contribution_Guide
To communicate with official developers, you can download the Feishu app and scan the QR code to join the Feishu group！
Thank you again for your support!


fix(@angular/build): check inlineSourceMap option with isolated modules optimization

The isolated modules transpilation option that was recently introduced is currently disabled when source maps are enabled. TypeScript has two mutually exclusive options that can be used to check for source map usage. Both of these options (sourceMap and inlineSourceMap) are now checked to determine if the direct transpilation optimization can be used.

The changes were merged into the following branches: main, 18.1.x


when insert video file like ".MP4" or ".AVI", it seem not work

I  suggest considering the lowcase and upcase

Fixed. Thanks.


[Admin] Publish

Docs Build status updates of commit e8ea3d3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/concepts/browsers-used-by-office-web-add-ins.md
:white_check_mark:Succeeded
View

docs/develop/configure-your-add-in-to-use-a-shared-runtime.md
:white_check_mark:Succeeded
View

docs/images/custom-functions-in-browser-runtime.png
:white_check_mark:Succeeded
View

docs/outlook/add-mobile-support.md
:white_check_mark:Succeeded
View

docs/outlook/outlook-add-ins-overview.md
:white_check_mark:Succeeded
View

docs/outlook/testing-and-tips.md
:white_check_mark:Succeeded
View

docs/reference/objectmodel/preview-requirement-set/office.context.mailbox.item.md
:white_check_mark:Succeeded
View

docs/reference/objectmodel/requirement-set-1.1/office.context.mailbox.item.md
:white_check_mark:Succeeded
View

docs/reference/objectmodel/requirement-set-1.10/office.context.mailbox.item.md
:white_check_mark:Succeeded
View

docs/reference/objectmodel/requirement-set-1.2/office.context.mailbox.item.md
:white_check_mark:Succeeded
View

docs/reference/objectmodel/requirement-set-1.3/office.context.mailbox.item.md
:white_check_mark:Succeeded
View

docs/reference/objectmodel/requirement-set-1.4/office.context.mailbox.item.md
:white_check_mark:Succeeded
View

docs/reference/objectmodel/requirement-set-1.5/office.context.mailbox.item.md
:white_check_mark:Succeeded
View

docs/reference/objectmodel/requirement-set-1.6/office.context.mailbox.item.md
:white_check_mark:Succeeded
View

docs/reference/objectmodel/requirement-set-1.7/office.context.mailbox.item.md
:white_check_mark:Succeeded
View

docs/reference/objectmodel/requirement-set-1.8/office.context.mailbox.item.md
:white_check_mark:Succeeded
View

docs/reference/objectmodel/requirement-set-1.9/office.context.mailbox.item.md
:white_check_mark:Succeeded
View

docs/reference/requirement-sets/excel-api-online-requirement-set.md
:white_check_mark:Succeeded
View

docs/reference/requirement-sets/excel-preview-apis.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


[New API] : Result marks Api

Brief Description 🤓
As you closed my previous issue and told to add some more features like particular studnt marks,etc
so that you can give level 3 label
in PR : #140
and i can not work on closed issue so this issue is adding more advanced functionalities
How will you develop it?🤩
Reuslt Marks API
with 2roles
student role and treacher Role
Crud operation with marks and student as well.
Deployment
None
Select the open source program you are contributing to
Other
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINES of this project.

Greetings @Yashgabani845🎀 ! , We are excited to have you dive into another issue with us!🚀
Your involvement in our project has been invaluable, and we're confident that your skills and insights will help us conquer this challenge😇.  Use command /gssoc24 to self assign the issue.  This issue represents an exciting opportunity for us to improve and enhance our project, and we are thrilled to have you on board.
We request you to follow CONTRIBUTING GUIDELINES.
Looking for your PR soon!
In case of any issues, you can contact us on Discord.❣

/gssoc24

Hey @Yashgabani845 ! I wanted to inform you that we have closed issue.Thank you for your understanding, and we look forward to your continued engagement with our repository🤗.
In case of any issues, you can contact us on Discord.
Thank you! ❣
Happy Coding! ✨
Will See you soon❣


New SD card,format and FBI (second time?)

hi!
I will start doing part 4 of plailect's guide today after after almost 1 month of completing part 3.
I am using a 2ds and I bought a new sd card 32GB at part 3 to continue.
It seems that I have formated the new sd card following this guide: https://github.com/Plailect/Guide/issues/231 at post #6 by @nickthegamer5 (which he helped me)
Will that be a problem?
Because I saw on gbatemp.net that I have to finish the guide first,then change to a new sd card
Also If I format rednand 3 times just to be sure(because I did format the new sd card with emunand 9 and I don't know if it counts) will the rednand and sysnand be linked?
Also the guide at part 4 in instructions step II-Injecting FBI, says to inject FBI to H&S but I have already done that at 9.2.0 Update Page of Plailect's Guide.Must I do it again?
thank you!

Formatting it extra times won't affect anything.
If you can open FBI, you don't have to inject it again.

Thanks! I just finished your guide!you are awesome!
Although I have a little problem now:I injected back the h&s app(following part 5 section VI,last 4 steps) that I had from the 9.2.0 update(I was on <IP_ADDRESS>E and used rxtools 3.0 beta on june 28) and it is working but the icon is still FBI.
FBI.cia is installed too so I have 2 fbi's but the h&s app shows on the upper screen that it is h&s and the cia file shows that it is indeed FBI
what should I do?


set-up-a-github-repo-to-publish-a-marketplace-item

Please use the form below, leaving the prefilled data to help us. Thank you.
Page link: set-up-a-github-repo-to-publish-a-marketplace-item
Document link: set-up-repo.md
My Issue/Suggestion
You can't actually create a repo in this form "Mendix/MyFirstApp", because github does not accept the forward slash

Hi @scottf thanks for your feedback, this was meant to be a generic UpperCamelCase example, I've clarified it in this commit: https://github.com/mendix/docs/commit/60100aa86b908bcfb1639a374276e172b9e91c7a
Cheers,
Adam


Demo showcase doesn't match current version (v0.7.1)

Just a heads up that the current demo showcase at http://docs.raneto.com/ is not up-to-date.
Example:
http://docs.raneto.com/templates/customizing-the-template
still has:
All of the template views can be found in the views folder

The source code for the demo page is already up-to-date (https://github.com/gilbitron/Raneto/blob/0.7.1/content/templates/customizing-the-template.md)
All of the template views can be found in the themes/default/

The demo showcase probably just has to be redeployed with the current content.

Thanks @draptik. I'm not sure how this is dont currenty but will find out
@gilbitron is updating the documentation as simple as editing the gh-pages branch?

@ryanlelek No it's hosted on Heroku. I've just pushed the new version to the demo site. I needed to make some changes to pakcage.json to make it work.

Cool. Thanks for the quick fix @gilbitron !


Clarification on ACL vs Azure IAM permissions

The documentation states:

The following table lists services that can have access to your storage account data if the resource instances of those services are given the appropriate permission. To grant permission, you must explicitly assign an Azure role to the system-assigned managed identity for each resource instance. In this case, the scope of access for the instance corresponds to the Azure role assigned to the managed identity.

However, in reality it does not require an Azure IAM assignment. The firewall with "Allow trusted services" already opens for the MSI if it does not have any Azure Role on the resource (e.g. Blob Data Contributor, Blob Data Reader) but just an ACL assignment on a specific folder. Tested with ADF.
Could you please review this paragraph of the documentation with what is actually checked?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 2632af75-7664-b79b-ac5b-c7c17e6a609a
Version Independent ID: 6ecf71b6-77c4-3ab6-8ec9-353ea407d70f
Content: Configure Azure Storage firewalls and virtual networks
Content Source: articles/storage/common/storage-network-security.md
Service: storage
Sub-service: common
GitHub Login: @normesta
Microsoft Alias: normesta

@fmms Thanks for reporting this. I have assigned the issue to the content author to investigate further and update the document as appropriate.

Thanks for asking @fmms. I've updated the text to say "also" instead of "now" as the current phrasing is causing confusion. The statement is intended to communicate that the feature is supported in accounts that have a hierarchical namespace as well as those that don't.  Thank you!
#please-close

My apologies @fmms this was a response to another issue. Your issue still requires our attention. @DerekHerman-MSFT do you know how to reopen this issue?

@DerekHerman-MSFT please reopen!

We have opened a support case regarding this documentation issue. In the case<PHONE_NUMBER>002488 it was confirmed that indeed the documentation is wrong. Documentation will be updated, though there is no ETA right now.

Ok. I confirmed that ACLs work just fine in this context. The doc has been updated to make that clear. Thank you @fmms
#please-close

thanks. For everyone else, fix is comitted with https://github.com/MicrosoftDocs/azure-docs/commit/39b426db4d90d257b0e79dba1046f412a8ebb27e#diff-f7331dc8d6085d8fa484badcb565f9c42b7f3fbc3d85fe7ff48b3b854a727c2a .


[Improvement][AMS] keyed change table don't delete TTL files in time when expire

Now, keyed change table has 2 properties to control files
change.data.ttl.minutes: check change table data file whether in valid time, if not then delete from iceberg snapshot
snapshot.change.keep.minutes:check change table snapshot whether in valid time, if not then delete snapshot from file system(include deleted data files and metadata files)
so, the data files will keep ttlTime + expireTime.

cc @hzluting    does it have been handled?

No, but now you can set the expire time to 0. The data files will delete after the TTL time.

Closed as not planed.


Issue with phrases.json

this did not happen to me, it happen to some other dude he sent me the image of the error.

pls tell me how to fix i will try to fix and learn js in the process thanks also can you give me your discord

this guy didn't download phrases.json

Yea but the updater needs to download it automatically right

Yeah true but the old version of the updater needs to download the new version of the updater right


Get structure in line with Go

Most of the actual code is the same as before. Changes:

Moved all routing code to separate project
Removed router code from Props, replaced with Spawner abstraction
Renamed Routing to Router
Renamed Remoting to Remote

Thanks!


[Compress PDF] Error 422

Extension
https://www.raycast.com/felixhaeberle/compress-pdf
Description
When I try to compress pdf I get Error 422.
Steps To Reproduce
Add plugin
Register cloudconvert
create API key
Add to Raycast
Add file to compress
Error 422
Current Behaviour
When I try to compress pdf I get Error 422.
Expected Behaviour
Compressed PDF in folder.
Wishful was more explenation about the API key.

Same error


From private repo

@v-alje : Thanks for your contribution to the Azure documentation! The author, @echuvyrov, has been notified to review your proposed change.

This is a massive PR, may I suggest breaking it down into small chunks by topic? For example, the change to Terraform docs (that looks great, btw, thanks!) could go into a single PR that I can approve. I shouldn't be able to approve any other changes since I didn't author the original docs.


obsidian glass slabs invisible from one side.

Obsidian glass slabs are invisible from one side, while appearing perfectly normal from the other.

Is this a duplicate of https://github.com/minetest-mods/moreblocks/issues/32 ? Note, there is a fix for that issue...

I'm not sure, created the issue for sokomine as he requested. Never saw the problem or looked into it. Sounds like it could be a simple as a flipped normal.

Already fixed by f92a938d7590e68c2a34d45866a52aa09d6c8fd6


Add forceCdata option to builder to force the use of CDATA tags for all content.

This PR adds a forceCdata option to the XML builder that wraps all text nodes in CDATA even when it is not required. This option is set to false by default. A new test exercises the forceCdata option using a small sample of text that does not require escaping.

Coverage remained the same at 97.633% when pulling 16ce472b58dcdbceacc66413a2bacd9254d132d9 on mdaly:force-cdata-builder-option into 434abf4ca99fc780ae2f905a400449889dd88520 on Leonidas-from-XIV:master.

Why would I want to wrap things in unnecessary CDATA?

We've run into a situation where a parser (that we don't control) requires CDATA on all text content, like what's described in #321.  I'm not sure how common this is, but the ability to always force CDATA has served us well.

Truth be told, I am not very fond of outputting nonsensical XML just because something that claims it parses XML does not do it properly. It is like saying "we have a parser that doesn't handle characters, so we should encode every character via XML entities".

@Leonidas-from-XIV  We met the same problem, some company require CDATA on all text content, but we can't say they're wrong and fix for us. So we have to change our code in workspace use CDATA force. We hope there'll be coming a long-term solution.

Same for us. One of the german services requires all the data to be wrapped into CDATA

Same problem here

+1
The Tecent wechat api require CDATA on all content.

Same problem here. Now I have to use a template string instead of xml2js to force CDATA. Painful as it might, I still consider a forceCdata option quite useful.

Anything?

Same problem with Wechat. Now I have to use a template string instead of xml2js to force CDATA. Painful as it might, I still consider a forceCdata option quite useful.

Just wanted to elaborate on this suggestion. It could be implemented as the following:

Walk through all object fields that need to be in CDATA tag and append to their values a string that will force CDATA generation e.g. value += '<FORCECDATA>'.
Build XML.
Remove all occurrences of the appended string from the XML string, e.g. xml = xml.ReplaceAll('<FORCECDATA>', '').


Integrate the OpenSWATH into Galaxy

More informations here:

http://www.openswath.org/example_OpenSWATH.zip
http://www.openswath.org/openswath_instructions.html

Programs needed:

[ ] xinteract (TPP)
[ ] spectrast (PP)
[ ] spectrast2spectrast_irt.py
[ ] spectrast
[ ] spectrast2tsv.py
[ ] ConvertTSVToTraML*
[x] OpenSwathDecoyGenerator*
[ ] OpenSWATHWorkflow*

*All OpenMS tools can be generated automatically, the wrapper quality varies

The TPP tools are so hard to wrap since they create files with so many references embedded right in the file - which is not compatible with the way Galaxy stores files or handles outputs. It might be worth wrapping large portions of the workflow into a single tool for this reason.

John and Bruening - do you think using PeptideShaker outputs instead of TPP tools would be an alternative?

I hope that @iracooke awesome protk library can help here. Otherwise we can fallback to one single wrapper.
Many of these tools are also integrated in OpenMS, or this is wip. So we do not need to take care of the links at some point.

In the list, should that be "spectrast (PP) (TPP)"?
It looks like it's listed twice, is the second one from another source?

@trevor I hope you can also edit the list above. It was listed twice because it is used twice in the workflow. I corrected this mistake. Thanks.

A few additional tools that are pretty integral to the workflow:
pyprophet https://github.com/uweschmitt/pyprophet and a feature_alignment.py script (written by Hannes et al at ETH) to combine and align results files within a single experiment (https://code.google.com/p/msproteomicstools/source/browse/trunk/analysis/alignment/feature_alignment.py).

Thanks @sparker43 I added it to the list.

added ConvertTSVToTraML and OpenSWATHWorkflow in https://github.com/bgruening/galaxytools/commit/68b57f3370af48f8f0b31c5a4cdad2e6ca38e529

I will start to work on the missing tools here: https://github.com/galaxyproteomics/tools-galaxyp/tree/msproteomicstools
Contributions welcome!

Inittial wrappers for pyprophet and feature_alignment are included. Does anyone have small test files?

Hello Sarah,
Do you have small test files for pyprophet and feature alignment tool?
Regards,
Pratik
Pratik Jagtap,
Managing Director, Center for Mass Spectrometry and Proteomics,
43 Gortner Laboratory, 1479 Gortner Avenue, St. Paul, MN 55108
Phone:<PHONE_NUMBER>
On Tue, Mar 3, 2015 at 11:54 AM, Björn Grüning<EMAIL_ADDRESS>wrote:

Inittial wrappers for pyprophet and feature_alignment are included. Does
anyone have small test files?
—
Reply to this email directly or view it on GitHub
https://github.com/galaxyproteomics/tools-galaxyp/issues/10#issuecomment-76997537
.

I suggest we narrow the scope of the xinteract tool since xinteract is basically just a wrapper for other tools (various Prophets, libra, asapratio etc).
Is the required functionality covered by protk tpp prophets tools?
https://github.com/iracooke/protk-galaxytools/tree/master/tpp-prophets

xinteract is not needed anymore, @iracooke has the missing functionality already in protk


Fix and refactor the scope of some resources

There was a bug in TestResourceScope.MATCHING_RESOURCES which made this work and was fixed in https://github.com/quarkusio/quarkus/pull/44279, I change it in this PR https://github.com/quarkus-qe/beefy-scenarios/pull/505 but it seems I wrongy understand the docs back then.
Not it should work as:

RESTRICTED_TO_CLASS start for every test class, every test class need anotation
MATCHING_RESOURCES start for specific group of test classes, every test class need anotation
GLOBAL start for every test class, only one annotation

I change it to cover all 3 usages.

Pleas add commit similar to https://github.com/quarkus-qe/beefy-scenarios/commit/c48ec1d1423aab1ca609a314719fbc003dde95fb to increase the memory to 8g


Fix titles and schema references on windows

Tested at https://github.com/deco-sites/starting/pull/102 using windows.


add resetProperties()

I've created a private resetProperties method to encapsulate the property resets. This makes the close method more concise and easier to understand. It also separates the concerns of closing the cURL resource and resetting object properties for better code organization.

The code has been formatted by VSC, I can remove the formatting if it is req.


Update readme examples for Swift 3.0

Since the Swift 3.0 migration has changed almost all of the APIs in the SDK, we should update the readme to reflect those changes. In particular, the code examples need to be updated so that they conform to the Swift 3.0 version of the SDK.

Complete with 58d35f8c2babe418680963c5772336f9d94fb347


add getclaimbyid

Its probably worth keeping consistency with RPC command getvalueforname here, as we don't want to end up with two separate handlers in lbryum or whatever application using these two commands.
When there is no claim that exists, getvalueforname returns an empty dictionary (instead of -1).
getvalueforname also does not format the amount value to credits.
getvalueforname also returns effective amount in credits which we can also do here.

@kaykurokawa updated the return fields

LGTM


The project has no license

Hi @domluna, @andrewpaster,
Notice that our project doesn't have license, that's a little unprofessional for a open source project, especially there has much extends adoption and development in an online education project. "No license terms" means users don't know it can be used freely. Another reference is Can I call my program "Open Source" even if I don't use an approved license?.
So here I suggest the project add license, recommend we choose one of the below license, firstly because they're all Permissive Licenses, but they have different redistribution disclaimer:

MIT
Apache-2.0
BSD-3-Clause

Regards,
Brandon

Added!


GPU memory usage at benchmark

I want to know the GPU memory usage,because I'm out of memory when I test the benchmark,
model: Qwen1.5 0.5B Chat
GPU: RTX3090
command:bash scripts/infintebench.sh
bash scripts/longbench.sh
result:cuda out of memory.

Hi! Could you provide more information, such as your configuration and which dataset you were evaluating when the out of memory issue occurred?

Hi! Could you provide more information, such as your configuration and which dataset you were evaluating when the out of memory issue occurred?

config.json
model:
  type: inf-llm
  path: Qwen1.5-0.5B-Chat
  block_size: 128
  n_init: 128
  n_local: 4096
  topk: 16
  repr_topk: 4
  max_cached_block: 32
  exc_block_size: 512
  score_decay: 0.1
  fattn: true
  base: 1000000
  distance_scale: 1.0

max_len:<PHONE_NUMBER>
chunk_size: 8192
conv_type: qwen

bash scripts/longbench.sh
mkdir: cannot create directory ‘benchmark/longbench-result’: File exists
Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
Pred narrativeqa
  0%|                                                                                | 0/200 [00:00<?, ?it/s]huggingface/tokenizers: The current process just got forked, after parallelism has already been used. Disabling parallelism to avoid deadlocks...
To disable this warning, you can either:
        - Avoid using `tokenizers` before the fork if possible
        - Explicitly set the environment variable TOKENIZERS_PARALLELISM=(true | false)
huggingface/tokenizers: The current process just got forked, after parallelism has already been used. Disabling parallelism to avoid deadlocks...
To disable this warning, you can either:
        - Avoid using `tokenizers` before the fork if possible
        - Explicitly set the environment variable TOKENIZERS_PARALLELISM=(true | false)
  0%|                                                                                | 0/200 [00:05<?, ?it/s]
Traceback (most recent call last):
  File "/home/luhao/InfLLM-main2/benchmark/pred.py", line 299, in <module>
    preds = get_pred(
  File "/home/luhao/InfLLM-main2/benchmark/pred.py", line 241, in get_pred
    output = searcher.generate(
  File "/root/anaconda3/envs/train/lib/python3.9/site-packages/inf_llm-0.0.1-py3.9.egg/inf_llm/utils/greedy_search.py", line 33, in generate
  File "/root/anaconda3/envs/train/lib/python3.9/site-packages/inf_llm-0.0.1-py3.9.egg/inf_llm/utils/greedy_search.py", line 55, in _decode
  File "/root/anaconda3/envs/train/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/root/anaconda3/envs/train/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/root/anaconda3/envs/train/lib/python3.9/site-packages/transformers/models/qwen2/modeling_qwen2.py", line 1173, in forward
    outputs = self.model(
  File "/root/anaconda3/envs/train/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/root/anaconda3/envs/train/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/root/anaconda3/envs/train/lib/python3.9/site-packages/inf_llm-0.0.1-py3.9.egg/inf_llm/utils/patch.py", line 98, in model_forward
  File "/root/anaconda3/envs/train/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/root/anaconda3/envs/train/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/root/anaconda3/envs/train/lib/python3.9/site-packages/transformers/models/qwen2/modeling_qwen2.py", line 773, in forward
    hidden_states, self_attn_weights, present_key_value = self.self_attn(
  File "/root/anaconda3/envs/train/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/root/anaconda3/envs/train/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
    return forward_call(*args, **kwargs)
  File "/root/anaconda3/envs/train/lib/python3.9/site-packages/inf_llm-0.0.1-py3.9.egg/inf_llm/utils/patch.py", line 16, in hf_forward
  File "/root/anaconda3/envs/train/lib/python3.9/site-packages/inf_llm-0.0.1-py3.9.egg/inf_llm/attention/inf_llm.py", line 60, in forward
  File "/root/anaconda3/envs/train/lib/python3.9/site-packages/inf_llm-0.0.1-py3.9.egg/inf_llm/attention/context_manager.py", line 726, in append
  File "/root/anaconda3/envs/train/lib/python3.9/site-packages/inf_llm-0.0.1-py3.9.egg/inf_llm/attention/context_manager.py", line 616, in append_global
  File "/root/anaconda3/envs/train/lib/python3.9/site-packages/inf_llm-0.0.1-py3.9.egg/inf_llm/attention/context_manager.py", line 20, in __init__
RuntimeError: CUDA error: out of memory
CUDA kernel errors might be asynchronously reported at some other API call, so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
Compile with `TORCH_USE_CUDA_DSA` to enable device-side assertions.

/root/anaconda3/envs/train/lib/python3.9/site-packages/fuzzywuzzy/fuzz.py:11: UserWarning: Using slow pure-python SequenceMatcher. Install python-Levenshtein to remove this warning
  warnings.warn('Using slow pure-python SequenceMatcher. Install python-Levenshtein to remove this warning')

When I use Qwen1.5-0.5B-Chat_mistral is no problem,mistral arch.

I tested narrative qa on an A40 (48G) using your settings and limited the CUDA memory usage to 24G. No out of memory issue occurs. Did you use the model Qwen/Qwen1.5-0.5B-Chat from the Hugging Face Hub?


chore(main): release 0.0.17

:robot: I have created a release beep boop
0.0.17 (2024-06-05)
Bug Fixes

correct IP range failure logic; add support for configurable ping packet size for MTU checks (#200) (dec0b85)

Other

deps: update kubernetes packages to v0.30.1 (#180) (1347078)
deps: update module github.com/go-logr/logr to v1.4.2 (#195) (86beecc)
deps: update module github.com/onsi/ginkgo/v2 to v2.19.0 (#198) (668a91d)
deps: update module github.com/onsi/gomega to v1.33.1 (#181) (592a773)
deps: update module github.com/validator-labs/validator to v0.0.41 (#196) (49bff7d)
deps: update module sigs.k8s.io/cluster-api to v1.7.2 (#179) (5c1f475)
deps: update module sigs.k8s.io/controller-runtime to v0.18.4 (#202) (fa9c1fb)
deps: bump golang.org/x/net from 0.21.0 to 0.23.0 (#182) (7199ff8)
deps: pin googleapis/release-please-action action to f3969c0 (#189) (12b3380)
deps: update actions/checkout digest to 0ad4b8f (#183) (3d3f24d)
deps: update actions/checkout digest to a5ac7e5 (#190) (b08f14e)
deps: update actions/setup-go digest to cdcb360 (#192) (f6a4347)
deps: update anchore/sbom-action action to v0.16.0 (#197) (8741cc9)
deps: update azure/setup-helm digest to fe7b79c (#178) (adbd1c0)
deps: update codecov/codecov-action digest to 125fc84 (#191) (c2e67e3)
deps: update codecov/codecov-action digest to 6d79887 (#175) (07e51ea)
deps: update dependency go to v1.22.4 (#199) (2e5bbe9)
deps: update docker/login-action digest to 0d4c9c5 (#193) (4aede1f)
deps: update docker/setup-buildx-action digest to d70bba7 (#176) (92ace5b)
deps: update gcr.io/spectro-images-public/golang docker tag to v1.22 (#144) (ad2a4f1)
deps: update helm/kind-action action to v1.10.0 (#184) (d47ff92)
deps: update softprops/action-gh-release digest to 69320db (#194) (2b72308)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/validator-labs/validator-plugin-network/releases/tag/v0.0.17 :sunflower:


cat-blog.md

Fixes #118
Preview

@cvojslavek Is there a reason this link is italic?

For the ONRResource link, we should add a note that the link won't work if you're not on the vpn.

@mcharg It's italicized because it is the title of the post. Maybe it is not as formal as a blog post.
Good call. What about: ONRResource has cross training and job rotation opportunities (connect to VPN to access).

@cvojslavek I think that works.

@jlouis-onrr @mojobnichols can one of you figure out why this branch has conflict and I can't merge it? Thanks

@Maroyafaied, the conflict is now resolved.


Setting network offline in devtools does not affect localhost server

In section 3 of the "Caching Files with ServiceWorker" lab in the PWA training (https://developers.google.com/web/ilt/pwa/lab-caching-files-with-service-worker#3_serve_files_from_the_cache), the student is instructed to "take the app offline".  The hyperlink on that instruction tells how to use devtools to set the browser to offline mode, but the server being used is on localhost.  Since localhost does not need the network, setting the browser to offline doesn't prevent the files from being fetched from the server.
Instead, I simply stopped the server and the behavior was as expected - cached items were displayed, non-cached items came back with a 404 error.
Fix:  The instructions should tell the student to stop the server when they want to test the cache and restart it when they want to resume normal operation.  These instructions appear several times in this tutorial.
BTW, I'm glad to know about the instruction for disabling the network using devtools, but it just doesn't work in this particular case.

Hi again Glenn - thanks for filing another bug!
Although it certainly doesn't seem like it, Chrome's offline mode actually does work for localhost! ... usually ;) There is currently a bug that causes unexpected behavior in offline mode, but it has been corrected in Chrome 63.
Unfortunately those instructions are scattered throughout dozens of tutorials, so we probably won't be able to update them before Chrome 63 comes out anyways. But in the meantime, shutting down the server works just as well.


[Currying]

Is this a unique topic?

[x] I have checked "open" AND "closed" issues and this is not a duplicate

Is your topic request related to a problem/unavailable functionality? Please describe.
I often find it challenging to convey the benefits and usage of advanced JavaScript concepts, such as currying. Many developers struggle with understanding the difference between currying and partial application, and how these concepts can enhance code reusability and readability. Additionally, existing resources often lack practical examples demonstrating how to implement these concepts in real-world scenarios.
Proposed Solution
I propose adding a topic on Currying in JavaScript to the JavaScript Handbook. This topic will cover:

Why Currying is Useful: Explanation of how currying helps in creating more reusable and composable functions.
Comparison with Partial Application: Clarifying the differences between currying and partial application with examples.
Practical Implementation: A simple project demonstrating the advantages of currying through a Functional Discount Calculator that dynamically applies discount rates to product prices.

The implementation of this topic can draw inspiration from other projects and articles focused on functional programming in JavaScript. For example, the article on Understanding Currying provides a foundational understanding that can be expanded upon with hands-on examples.
Screenshots

Do you want to work on this issue?
Yes
If "yes" to above, please explain how you would technically implement this (issue will not be assigned if this is skipped)
To implement this topic, I will:

Create a new section in the JavaScript Handbook dedicated to Currying in JavaScript.
Write a comprehensive explanation of what currying is and why it's beneficial, along with a comparison to partial application.
Develop code examples that illustrate basic currying and partial application, including a practical use case, like the Functional Discount Calculator.
Ensure that the examples are well-documented and easily understandable for developers at various skill levels.
Organize the project files into a clear directory structure, making it easy for contributors to access and understand the implementation.

I will reference existing JavaScript functional programming concepts and use well-established practices to ensure clarity and maintainability.

Hi! I've created a pull request to address this issue. You can check it out here: #64 .
Looking forward to your feedback!
Direct Link: https://github.com/neerajrathore/javascript-handbook/pull/64#issue-2568811002

Hello @neerajrathore I have already created a Pull Request with respect to this Issue. Please take a look at it and suggest any changes. Please merge if the changes are acceptable.
Pull Request #64

@ArpitCS yes I am reviewing your changes now


Add lpush rpush insertion capability. Serialize Kakfa struct to json

Hi @jvenant!
Thanks for creating this pull request! Much appreciated. The LPush and RPUSH functionality look great. I would prefer that the JSON part be done in a single message transformation. This is pretty common functionality which other connectors could benefit from. Take a look at this transformation and see if it works for you.
J

Ok, you're right : I remove the Json/Struct conversion


cannot "upgrade" from get to get -vcs

Effectively reproduced by:
> [!net] skip
> [!exec:git] skip
> env GOPROXY=
> cd repo
> go get<EMAIL_ADDRESS>> env GOHACK=/gohack
> gohack get rsc.io/quote
rsc.io/quote => /gohack/rsc.io/quote
> gohack undo
> gohack get -vcs rsc.io/quote

I think this issue also raises the question of how you "downgrade" in the case you subsequently call bare get (because the assumption would be, in that case, that you aren't working in a VCS checkout).

Just encountered this myself:
$ gohack get -vcs github.com/rogpeppe/go-internal
cannot update VCS dir for github.com/rogpeppe/go-internal: cannot get info: cannot get VCS info from "/home/mvdan/gohack/github.com/rogpeppe/go-internal": fatal: not a git repository (or any parent up to mount point /)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
all modules failed; not replacing anything
$ # remove ~/gohack
$ gohack get -vcs github.com/rogpeppe/go-internal
creating<EMAIL_ADDRESS>github.com/rogpeppe/go-internal => /home/mvdan/gohack/github.com/rogpeppe/go-internal


Ad prefetch internal pr

Type of change

[ ] Bugfix
[ ] Feature
[ ] New bidder adapter
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Does this change affect user-facing APIs or examples documented on http://prebid.org?
[ ] Other

Description of change

test parameters for validating bids

{
  bidder: '<bidder name>',
  params: {
    // ...
  }
}

Be sure to test the integration with your adserver using the Hello World sample page.

contact email of the adapter’s maintainer
[ ] official adapter submission

For any changes that affect user-facing APIs or example code documented on http://prebid.org, please provide:

A link to a PR on the docs repo at https://github.com/prebid/prebid.github.io/

Other information

@ym-abaranov Can we use the consent module from prebid? It seems we are duplicating code from that module.

@ym-abaranov Can we use the consent module from prebid? It seems we are duplicating code from that module.

Unfortunately, those are not exported. Also, IabConsent in case of no cookie will wait for user action which is not necessary, so we're using timeout there. Hopefully, pbjs team could either export them and add cb timeout when they will review the pr or decide to keep it as-is.


Implement Pink Elephant

As follow-up for #521, since we have an accepted proposal for Pink Elephant, here is the implementation items for Pink Elephant (and more broadly any theme in the future)

[ ] Create src/theme_pink_elephant.scss

This should be a copy of src/theme_dark.scss with line 13 changed to reflect Pink Elephant rather than dark theme

[ ] Create src/themes/oui/oui_colors_pink_elephant.scss

This should be a copy of src/themes/oui/oui_colors_dark.scss for now. These colors will change in the future

[ ] Create src-docs/src/theme_pink_elephant.scss

This should be a copy of src-docs/src/theme_dark.scss with line 16 changed to reflect Pink Elephant rather than dark theme

[ ] Add Pink Elephant to the list of themes (OUI_THEMES) in src/themes/themes.ts

text: Pink Elephant
value: pink-elephant

[ ] Register the theme in src-docs/src/index.js

Import the theme like the other themes: https://github.com/opensearch-project/oui/blob/f5d7aa66fa05e211f60b20cd26612472da9e5ee0/src-docs/src/index.js#L25-L28
Register the theme like the other themes: https://github.com/opensearch-project/oui/blob/f5d7aa66fa05e211f60b20cd26612472da9e5ee0/src-docs/src/index.js#L33-L36

The implementation of this task can be based off the work done here: https://github.com/opensearch-project/oui/compare/main...BSFishy:oui:pink_elephant

@BSFishy Can we close this now? Is there any work remaining to be done?


REVIGO Core Installation Instructions

Dear REVIGO team,
I would like to install the REVIGO Core package and run the python examples. Unfortunately there seems to be no installation instruction. How can I install the package?
Thank you and kind regards

To compile the RevigoCore library you need the Visual Studio code, or Visual Studio (if you are using Windows platform), a .NET installation (from Microsoft or Mono.NET from a Mono Project) and a set of precompiled databases (see RevigoWeb for links). To compile a .NET project inside Visual Studio code there are numerous examples available online so I'm not going into details.

Installation did not work for us. "Just google it" also does not really qualify as a proper installation instruction. Luckily there are enough alternative implementations of what REVIGO does so it is not necessary to waste more time and eventually a reference on it.

Well, we are happy that you found an alternative. I guess you asked for precompiled binaries so you don't have to compile the code yourself. The binary release is planned for the end of the February 2023.


Remove cookbook & refer to new home

Now that it’s a new home, let’s send folks over there.
Attn: @tpetr @wsorenson

Lemme try this again…


Update Program.cs

Bail -> Fail

Hi, why replace bail with fail?

Whats means Bail?

Bail is American English slang that colloquially means "to leave", but in a more negative context.  Let's bail!"
I appreciate the attention to potential typos, but this was intended as is. If you find anything else though, PR's are certainly appreciated. :)


When using gevent/greenlets, highly concurrent requests can return incorrect responses

When sending multiple concurrent requests through gevent/greenlets, requests from one greenlet can sometimes leak their state into other greenlets. So a request will get the response from a different query.
This may be related to a similar issue, under the same circumstances where a "Bad Status Line" exception is thrown when reading a response. Doing a TCPDump shows that the data coming across the wire is correct, but pyArango seems to drop portions of the incoming stream.

I am not familiar with greenlets, but if all threads are behind the same connection query results might get swaped. You could try having several connections, running each query in a different processes instead of threads or you could implement a way to associate each thread to the right result by having them share a lookup table of results.

Hi @bkp-billy @tariqdaouda @dothebart ,
I have fixed it by using a different session for each request and also implemented, fixed connection for concurrent requests through gevent/greenlets
Check out the fork at dilipvamsi/pyArango/tree/origin/geevent
Regards,
Dilip Vamsi.

Hi @bkp-billy @tariqdaouda @dothebart ,
As the request.Session is not thread safe it is better to create a session for each request rather than using the same request which fixes everything.


Upgrade Linux Kernel for flatcar-3637 from 6.1.34 to 6.1.36

Subject says it all.

See instead https://github.com/flatcar/scripts/pull/961


Could not found template ! problem with quarkus

Hey there!
i'm tryin to use pebble template in quarkus project! i put my pebble file in ressource/templates.
after setting the path this way PebbleTemplate compiledTemplate = engine.getTemplate("templates/cra.peb"); and running the project i got this error :
com.mitchellbosecke.pebble.error.LoaderException: Could not find template "templates/cra.peb" (?:?)
Does anyone know what seems to be wrong ? been stuck for a while and i don't know how to proceed to correct this error, i would appreciate some help.
Thanks in advance.

I have the same error message but the context is different.
The problem in my case is that the error appears from time to time but the template is rendered somehow (I cannot explain this - maybe two successive fast request for the same template).
2021-08-20 09:24:45 ERROR DefaultErrorHandler:186 - Could not find template "users" (?:?)
com.mitchellbosecke.pebble.error.LoaderException: Could not find template "users" (?:?)
	at com.mitchellbosecke.pebble.loader.DelegatingLoader.getReader(DelegatingLoader.java:69)
	at com.mitchellbosecke.pebble.loader.DelegatingLoader.getReader(DelegatingLoader.java:24)
	at com.mitchellbosecke.pebble.PebbleEngine.getPebbleTemplate(PebbleEngine.java:151)
	at com.mitchellbosecke.pebble.PebbleEngine.lambda$getTemplate$1(PebbleEngine.java:146)
	at java.base/java.util.concurrent.ConcurrentHashMap.computeIfAbsent(ConcurrentHashMap.java:1705)
	at com.mitchellbosecke.pebble.cache.template.ConcurrentMapTemplateCache.computeIfAbsent(ConcurrentMapTemplateCache.java:24)
	at com.mitchellbosecke.pebble.cache.template.ConcurrentMapTemplateCache.computeIfAbsent(ConcurrentMapTemplateCache.java:9)
	at com.mitchellbosecke.pebble.PebbleEngine.getTemplate(PebbleEngine.java:146)
	at com.mitchellbosecke.pebble.PebbleEngine.getTemplate(PebbleEngine.java:117)
	at ro.pippo.pebble.PebbleTemplateEngine.renderResource(PebbleTemplateEngine.java:149)
	at ro.pippo.core.Response.renderToString(Response.java:1057)
	at ro.pippo.core.Response.render(Response.java:1023)

Related to above stacktrace, in the end is executed engine.getTemplate(templateName) (where engine is an instance of PebbleEngine).
I use Java 11 and Pebble 3.0.5.

I found the problem in my case. It's something wrong in our code.


LSP does not attach when workspace contains too many files

Hey, thanks for the project, looking forward to give it a go to use Neovim with my Obsidian vault.
I set it up with Mason and Lspconfig and seems to be attaching for non-obsidian markdown files. However, when I open an obsidian markdown file, it wont attach to it. According to LSP log is not even trying to!
I tried removing some files/folders from my obsidian vault until I removed Journal/ and the LSP finally got attached. There is nothing special in this directory, just 550 markdown files with names in ISO format dates (eg: 2024-01-19.md)
Thanks!

Ah okay right. Thanks for reporting.
Do you or have you used logseq by any chance?

Yep, I moved from Logseq to Obsidian time ago.

Just for reference, I have 2071 markdown files total in my Obsidian vault

This is an odd one, but I hope I can help.
I have 4400 files and I use this every day, so I don't think that is the issue.
Could you post your LspInfo for when the server does attach?

Hmmm, interesting, in that case, there may be a file in my Journal directory that breaks moxide parsing maybe. Is there anyway of dumping debug info?
LspInfo with Journal:
 Press q or <Esc> to close this window. Press <Tab> to view server doc.

 Language client log: /Users/pwntester/.local/state/nvim/lsp.log
 Detected filetype:   markdown

 0 client(s) attached to this buffer:

 Other clients that match the filetype: markdown

 Config: markdown_oxide
 	filetypes:         markdown
 	root directory:    /Users/pwntester/obsidian
 	cmd:               /Users/pwntester/.local/share/nvim/mason/bin/markdown-oxide
 	cmd is executable: true
 	autostart:         true
 	custom handlers:

 Configured servers list: jsonls, yamlls, gopls, tsserver, lua_ls, pyright, dockerls, codeqlls, markdown_oxide, bashls

LspInfo without Journal:
 Press q or <Esc> to close this window. Press <Tab> to view server doc.

 Language client log: /Users/pwntester/.local/state/nvim/lsp.log
 Detected filetype:   markdown

 1 client(s) attached to this buffer:

 Client: markdown_oxide (id: 1, bufnr: [1])
 	filetypes:       markdown
 	autostart:       true
 	root directory:  /Users/pwntester/obsidian
 	cmd:             /Users/pwntester/.local/share/nvim/mason/bin/markdown-oxide

 Configured servers list: jsonls, bashls, yamlls, gopls, tsserver, lua_ls, pyright, codeqlls, markdown_oxide, dockerls

Thank you!
Could you post logs from /Users/pwntester/.local/state/nvim/lsp.log? Perhaps enable debug too by a vim.lsp.set_log_level("debug")

I think I found the issue. There are some entries which are really big for a text file:
❯ du -ah . | sort -rh | head -n 17
110M    .
 22M    ./2023-09-19.md
 20M    ./2023-09-20.md
 15M    ./2023-09-11.md
 14M    ./2023-09-21.md
6.7M    ./2023-10-02.md
5.9M    ./2023-10-20.md
4.9M    ./2023-09-26.md
4.7M    ./2023-09-05.md
3.8M    ./2023-09-22.md
2.8M    ./2023-10-05.md
2.4M    ./2023-10-06.md
2.3M    ./2023-10-03.md
2.2M    ./2023-09-25.md
1.0M    ./2023-09-28.md
392K    ./2023-10-31.md
348K    ./2024-01-16.md

So, perhaps, it was not crashing, just processing them and taking a long time. Seems like an Obsidian plugin went crazy those days and duplicated a lots of lines. After deduping the lines and getting back to normal size, everything seems to be working fine, thanks!!

😳
The parser is fast but not that fast!
Haha great that you figured it out.

Happy to find another with so many notes

It appears that we can close this now.

Thanks a lot!


config-loader: Fix failing tests on windows

The / path is resolved to C:\\ on Windows. Hence, when reading a file /mock-file.yaml the path is resolved to C:\\mock-file.yaml and it does not exist.
Credits to @freben for doing this in https://github.com/backstage/backstage/pull/3852
See https://github.com/backstage/backstage/runs/1761948460 for the concerned tests.
Closes #4242

Aight, will check back in a day or two to see if the Windows build finished.

Aight, will check back in a day or two to see if the Windows build finished.


Add default port

Add default port if not present in redis-cluster-nodes

Why I'm keeping opening PRs to original repo instead of my fork 🤦🏼 SORRY!


Filter grooming: how user knows what (s)he has chosen?

24.2. monday discussed filters overall: how to make them better for user to view and choose

Merged into #2106


The linked sample does not have the 12th step

The linked sample does not have the 12th step which is the addition of FirebaseMessagingService. It is using instead the NotificationHandler

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: affa806e-ed31-20b7-9aee-2b56572225ab
Version Independent ID: 5dea14a7-065e-c21b-712c-0cee7d3ffb16
Content: Send push notifications to Android using Azure Notification Hubs and Firebase
Content Source: articles/notification-hubs/notification-hubs-android-push-notification-google-fcm-get-started.md
Service: notification-hubs
GitHub Login: @sethmanheim
Microsoft Alias: sethm

@razvansalajan Thanks for the feedback! We are currently investigating and will update you shortly.

Had a discussion with @razvansalajan offline. Currently, the NotificationHandler wraps the FirebaseMessagingService and the tutorial should work as-is.

@razvansalajan We will now close this out but if you feel you need more information please just let us know.


Add build configuration with browserlist

Related to https://github.com/CartoDB/carto-vl/issues/28
This PR adds configuration to Webpack so the bundling process makes transpilation using babel-loader and a list of supported browsers (browserslist)

browserslist has not been defined yet with more details.
It currently includes just defaults

Blocked by #984 (because of the carto.Map removal)


Add new meta events

Add new meta events

Ley-line Anomaly
Lake Doric
Crystal Oasis
Domain of Istan
Jahai Bluffs
Thunderhead Peaks

This project  still active?


Login button visible with private resource using nss #78

Not yet sure if this requires a solid-ui, auth, or NSS solution.
This is adding more detail to:
https://github.com/solid/solidos/issues/70#issuecomment-940146896
"On NSS : Login to server-root, on your pod, click on the goto link of a private container. The opened page will show you as logged out but it will also show you private resources."
I have the problem happening sometime after having logged in (possibly days?) when I use a bookmark to go straight to a private resource.
It looks like the problem can occur simply because UI.authn.authSession.info.isLoggedIn=false even when an NSS cookie is set and therefore private access is indeed possible.
To replicate:

Go to a private resource on solidcommunity.net
Login
Delete all localStorage but keep cookies
Refresh page
Private resource is shown and editable but login button is also shown and UI.authn.authSession.info.isLoggedIn=false

If loggedIn status was based on wac-allow it would be correct, but also dependent on the resource being viewed.
I haven't checked but my guess would be that solid-client-authn-browser relies on the redirect dance to determine logged in status and this is somehow able to be bypassed when the cookie is used??
One fix for the UI to be correct would be to clear the cookie if UI.authn.authSession.info.isLoggedIn=false ...
Edit: isLoggedIn gets set by the redirect handler
https://github.com/inrupt/solid-client-authn-js/blob/287f3b8b89d7b0cfb94c8dc251d51eb5380b6412/packages/browser/src/login/oidc/redirectHandler/AuthCodeRedirectHandler.ts#L222
If checkuser fails in solid-ui, mashlib still tries to render the subject using outliner from solid-panes, which succeeds in fetching it as if it is a public resource because the cookie allows it.

This should maybe be a new issue, but it is the converse of this one, so I thought it's worth mentioning here.
When using the community solid server the NSS cookie is obviously never set so the login state is always correct.
Instead, mashlib breaks for things like private images. Mashlib does not seem to make an authenticated request - it relies on authentication via the NSS cookie.
On CSS the image fails to load because the DPOP token is not provided and the cookie is not available.
We're caught in the middle of the two authentication mechanisms and we either need to completely switch to DPOP and drop the cookie, or explicitly prefer the cookie for requests on the pod's subdomain (only available using NSS) and explicitly fallback to DPOP when unavailable...?
I notice that solid-client-authn-browser does have some kind of temporary support for cookie based auth for requests, but it's not clear to me that mashlib is using this or that it is useful in this context.
https://github.com/inrupt/solid-client-authn-js/blob/784d86c907b4f0c1047b0ee3371df682eadc03ff/packages/browser/src/login/oidc/redirectHandler/AuthCodeRedirectHandler.ts#L244

@timbl @theRealImy
Is this a very general question with solidos or rdflib to default fetch with credentials: 'include'
https://developer.mozilla.org/fr/docs/Web/API/Request/credentials

@josephguillaume

When using the community solid server the NSS cookie is obviously never set so the login state is always correct.
Instead, mashlib breaks for things like private images. Mashlib does not seem to make an authenticated request - it relies on authentication via the NSS cookie. On CSS the image fails to load because the DPOP token is not provided and the cookie is not available.

For querying CSS are you using SolidOS webApp (/browse.html) or CSS with an internal SolidOS (https://github.com/solid/community-server-recipes) ?

The CSS case is using an internal SolidOS using community-server-recipes, so the same-site cookie mechanism would work if CSS supported it (which presumably would require it to be in a solid spec somewhere). It's also not a CORS issue.
It's not clear to me that credentials: 'include' would actually use the custom fetch defined by solid-client-authn-browser, which is what would be needed.
Note also that in the case of images we're not using fetch directly at all - the url in e.g. slideshow pane is currently specified using the src attribute.

@josephguillaume   Thanks. I also found this.
https://www.twelve21.io/how-to-access-images-securely-with-oauth-2-0/#:~:text=4. USE SERVICE WORKERS
@RubenVerborgh have you any idea to feed our discussion ? Is there a specification issue ?

The client/server login state disagreement is a longstanding NSS issue indeed.
The path we've chosen with CSS is to not implement the cookie approach, because the security implications are not clear. This indeed means that non-public resources can only be accessed through apps.
The modern workaround that was recently recommended to me is the Blob API.

Using the Blob API works fine for me and component-based development environments can abstract out the complexity, e.g.
https://forum.solidproject.org/t/how-to-add-inline-content-in-an-authenticated-solid-app/4929/5
I'd suggest the next step for this issue is to turn off cookie based authorization for NSS, which would then mean it has the same behaviour as CSS.
Presumably this would be done / would require a new config option here:
https://github.com/solid/node-solid-server/blob/830f8849f345e40e471c9efb15dc443825cdbda2/lib/create-app.js#L198
This will obviously break a fair few things in mashlib, but fixing them while using CSS as the test server seems like it provides a clear transition path.
Until the transition is complete, the login status on NSS+mashlib would be unreliable, but I don't think it's worth implementing an additional cookie detection routine if it just gets phased out eventually.


How do you load/define dapr components?

I'm not seeing it in the dapr sample, it seems to just start a sidecar that does nothing?
The sample could, for example, demonstrate using a cron binding
Edit:
This is how
    .WithDaprSidecar(new DaprSidecarOptions
    {
        AppId = "your-api",
        ResourcesPaths = [
            "../your/components",
        ],
    });

@paulyuk thoughts?

@philliphoff is this still relevant

@danmoseley The Dapr sample in this repo doesn't require any components as it uses only method invocation.  The Dapr sample in the dotnet/aspire repo does, as it makes use of pub-sub and state-store components.
What's the thinking on where proper Aspire samples ought to live, here or in the aspire repo itself?  If here, should we move over the one from the aspire repo?  It seems like it'd be an extra headache to try to maintain both.

What's the thinking on where proper Aspire samples ought to live, here or in the aspire repo itself? If here, should we move over the one from the aspire repo? It seems like it'd be an extra headache to try to maintain both. (See related https://github.com/dotnet/aspire-samples/issues/82.)

Proper samples live in this repo. We just renamed the directory in the other repo to "playground" to better describe its purpose (i.e. to make it easier for folks working on Aspire itself to have apps that use features in the repo).
I'd update the sample in this repo to include showing relevant features.


Enable Nvidia GBM patches for Sway

Nvidia proprietary driver 495+ supports GBM and Sway can work just fine: https://forums.developer.nvidia.com/t/nvidia-495-on-sway-tutorial-questions-arch-based-distros/192212
Though Sway developers reject to support the proprietary driver: https://github.com/swaywm/sway/issues/6342#issuecomment-867450822
So, all the relevant patches should be adopted by individual distros.

I guess we would need to add an option which would automatically set this flag in the session descriptor or the wrapper.


Investigate PR templates

I'd like to see that PRs link to an issue (if applicable, I don't think creating an issue is necessary to submit a PR).  Is there anything else we might want to make sure we include in a PR?

I can't think of anything - I was initially thinking that a description of what the PR adds should be included.  However, if we're not creating release notes as we're continuously updating things, I don't think that matters.  And as a cookbook it's fairly self-explanatory what a new recipe adds.


[Flutter3.3.8] desktop触摸板两指滚动失效ListView

Steps to Reproduce

Execute flutter run on the code sample
...
...

Expected results:
Actual results:

Code sample

Logs

ListView点击拖动滚动没问题，两指上下滑动不了

(Using Google translate):
"There is no problem with clicking, dragging and scrolling on ListView, but two fingers cannot slide up and down. Using flutter 3.3.8, I feel that the SDK may have disabled this feature."

Hi @q819438477
When opening an issue, please try to provide all or some of the information indicated in the template so that we can address this issue properly.

Issue Template
If you are looking for support, please check out our documentation
or consider asking a question on Stack Overflow:
- https://flutter.dev/
- https://api.flutter.dev/
- https://stackoverflow.com/questions/tagged/flutter?sort=frequent

If you have found a bug or if our documentation doesn't have an answer
to what you're looking for, then fill our the template below. Please read
our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports

Steps to Reproduce

Please tell us exactly how to reproduce the problem you are running into.

Please attach a small application (ideally just one main.dart file) that
reproduces the problem. You could use https://gist.github.com/ for this.

If the problem is with your application's rendering, then please attach
a screenshot and explain what the problem is.

1. ...
2. ...
3. ...

Please tell us which target platform(s) the problem occurs (Android / iOS / Web / macOS / Linux / Windows)
Which target OS version, for Web, browser, is the test system running?
Does the problem occur on emulator/simulator as well as on physical devices?

Target Platform:
Target OS version/browser:
Devices:

Run your application with `flutter run --verbose` and attach all the
log output below between the lines with the backticks. If there is an
exception, please see if the error message includes enough information
to explain how to solve the issue.

Run `flutter analyze` and attach any output of that command below.
If there are any analysis errors, try resolving them before filing this issue.

Finally, paste the output of running `flutter doctor -v` here.

To be more precise, please provide the information indicated below:

A completed and minimal reproducible code sample
The output of flutter doctor -v
Reproduction steps, Expected results, Actual results
Which environment did this issue happen in? (MacOS/Windows/Linux)

Thank you!

Lowered to flutter sdk3.05, the touch pad can slide up and down with two fingers

Tested multiple Windows and Mac models. The function of disabling the Fluttersdk above 3.0.5 was tested. The touchpad swiped up and down to scroll the listview list

Please check this breaking change related to trackpad gestures introduced in 3.3.0 and see if it helps in your case.

Same problem!
You know, Macbook users are more used to using a trackpad than a mouse.
When I upgraded the flutter from 3.0.5 to 3.3.9, I found that the scrolling of the ListView widget could no longer be done with touchpad gestures (two fingers sliding up and down). It seems that this behavior is disabled.
There's no mention of changes to Flutter3.3 touch pad. This is a very common operation and should not be banned by default.
[✓] Flutter (Channel stable, 3.3.9, on macOS 11.6.1 20G224 darwin-x64, locale zh-Hans-CN)
• Flutter version 3.3.9 on channel stable at /Users/mac/flutter3.3
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision b8f7f1f986 (4 周前), 2022-11-23 06:43:51 +0900
• Engine revision 8f2221fbef
• Dart version 2.18.5
• DevTools version 2.15.0
• Pub download mirror https://pub.flutter-io.cn
• Flutter download mirror https://storage.flutter-io.cn
[!] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
• Android SDK at /Users/mac/Library/Android/sdk
✗ cmdline-tools component is missing
Run path/to/sdkmanager --install "cmdline-tools;latest"
See https://developer.android.com/studio/command-line for more details.
✗ Android license status unknown.
Run flutter doctor --android-licenses to accept the SDK licenses.
See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.
[✓] Xcode - develop for iOS and macOS (Xcode 13.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 13A1030d
• CocoaPods version 1.11.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2021.2)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
[✓] IntelliJ IDEA Ultimate Edition (version 2019.3.3)
• IntelliJ at /Applications/IntelliJ IDEA.app
• Flutter plugin version 45.1.2
• Dart plugin version 193.7547
[✓] Connected device (2 available)
• macOS (desktop) • macos  • darwin-x64     • macOS 11.6.1 20G224 darwin-x64
• Chrome (web)    • chrome • web-javascript • Google Chrome 108.0.5359.124
[✓] HTTP Host Availability
• All required HTTP hosts are available
! Doctor found issues in 1 category.
Process finished with exit code 0


Adding release notes for 1.0.0

Signed-off-by: Abbas Hussain<EMAIL_ADDRESS>Description
Adding release notes for OpenSearch 1.0.0 (GA)
This is generated using the tool: https://github.com/ariatemplates/git-release-notes
Issues Resolved
#789
Check List

[ ] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] 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 Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

For release versions I believe we do not need to have a qualifier, i would prefer to have 1.0.0 instead of 1.0.0-ga.
And as you brought up we can wait for other PRs to be merged into 1.0...

start gradle check

start gradle check

The test failure in "Log 266":
> Task :server:internalClusterTest

REPRODUCE WITH: ./gradlew ':server:internalClusterTest' --tests "org.opensearch.cluster.ClusterHealthIT.testHealthOnMasterFailover" -Dtests.seed=5C28067404DD84A9 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1" -Dtests.locale=lt-LT -Dtests.timezone=Asia/Almaty -Druntime.java=15

org.opensearch.cluster.ClusterHealthIT > testHealthOnMasterFailover FAILED
    java.util.concurrent.ExecutionException: MasterNotDiscoveredException[NodeClosedException[node closed {node_s0}{ZMcSsUfoSCqq3rrVnWv1XA}{iyf5QSZjRZSqp6_EtnYkqA}{<IP_ADDRESS>}{<IP_ADDRESS>:45327}{imr}]]; nested: NodeClosedException[node closed {node_s0}{ZMcSsUfoSCqq3rrVnWv1XA}{iyf5QSZjRZSqp6_EtnYkqA}{<IP_ADDRESS>}{<IP_ADDRESS>:45327}{imr}];

The test failure in "Log 275":
> Task :server:internalClusterTest

REPRODUCE WITH: ./gradlew ':server:internalClusterTest' --tests "org.opensearch.indices.DateMathIndexExpressionsIntegrationIT.testCreateIndexWithDateMathExpression" -Dtests.seed=E872F82509438444 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1" -Dtests.locale=nn-NO -Dtests.timezone=Asia/Bishkek -Druntime.java=15

org.opensearch.indices.DateMathIndexExpressionsIntegrationIT > testCreateIndexWithDateMathExpression FAILED
    [.marvel-2021.06.30/Z2eGPS5kQ0-KTI1u2Myptw] ResourceAlreadyExistsException[index [.marvel-2021.06.30/Z2eGPS5kQ0-KTI1u2Myptw] already exists]

Neither can be reproduced in main branch. 🤔

@tlfeng Its strange that a text file with no related tests is causing other test failures 🤔

@tlfeng Its strange that a text file with no related tests is causing other test failures 🤔

Exactly! 😂 😂 However, since it can't be reproduced by me, it's harder to debug ...

start gradle check

start gradle check


Support for Tuples as input values for PyTorch models

This PR adds support for Tuples as input values for PyTorch models.
Description
It is a simple PR which combines input values to Tuples if NDArray's name have '\w+()` syntax . Also there is a test.

Codecov Report

Merging #1436 (7d0115c) into master (bb5073f) will decrease coverage by 1.41%.
The diff coverage is 47.99%.

@@             Coverage Diff              @@
##             master    #1436      +/-   ##
============================================
- Coverage     72.08%   70.67%   -1.42%
- Complexity     5126     5298     +172
============================================
  Files           473      493      +20
  Lines         21970    23215    +1245
  Branches       2351     2532     +181
============================================
+ Hits          15838    16408     +570
- Misses         4925     5532     +607
- Partials       1207     1275      +68

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <ø> (-5.24%)
:arrow_down:

...odality/cv/translator/BigGANTranslatorFactory.java
33.33% <0.00%> (+8.33%)
:arrow_up:

...nslator/InstanceSegmentationTranslatorFactory.java
14.28% <0.00%> (-3.90%)
:arrow_down:

.../modality/cv/translator/YoloTranslatorFactory.java
8.33% <0.00%> (-1.67%)
:arrow_down:

...i/djl/modality/cv/translator/YoloV5Translator.java
5.69% <0.00%> (ø)

...odality/cv/translator/YoloV5TranslatorFactory.java
8.33% <0.00%> (-1.67%)
:arrow_down:

...modality/nlp/embedding/TrainableWordEmbedding.java
45.00% <0.00%> (-3.34%)
:arrow_down:

...pi/src/main/java/ai/djl/ndarray/BytesSupplier.java
54.54% <0.00%> (-12.13%)
:arrow_down:

...i/src/main/java/ai/djl/ndarray/NDArrayAdapter.java
15.23% <0.00%> (-0.83%)
:arrow_down:

... and 156 more

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 6c7acba...7d0115c. Read the comment docs.

@zakhio
Thanks for your contribution, this looks great.


Camera picture in views page

Is it possible at some point in future to make camera picture to show up as card in view page, so I can see it immediately, without clicking on entity?

It would be possible with #217


Demo API key in examples is not working

Hi,
Demo API key is not working - I receive bad response 500.
https://github.com/2captcha/2captcha-python/blob/master/examples/recaptcha_v2.py

Pretty sure it shouldn't work anyways.

there's no demo API key in the example, you should always use your own API key


Error: CI_Input::post_get() in Datatable.php on line 136

I got this error with the latest version of Codeigniter-DataTables,
Can you please help?
Fatal error: Call to undefined method CI_Input::post_get() in /var/www/html/test/application/libraries/Datatable.php on line 136

hi @sivapc  , the problem is , you don't have post_get() function in input.php (/system/core/input.php).
please check  .
so, go to your /system/core/input.php and add post_get() function. that works for me.


Node 0.6 support (now supported by connect)

Node 0.6 support would be very nice, and also attainable, now that it is supported by connect.
https://github.com/senchalabs/connect/blob/master/package.json#L19

Now supported in 0.2.6 using connect 1.X


Invalid url

Hello !
My question must be stupid!
I installed evilginx2, and configured it like this:
"config domain google.com
config ip xx.xx.xxx.xx
phishlets hostname reddit reddit.google.com
lures create reddit
lures edit 7 redirect_url https://www.reddit.com/r/Whatcouldgowrong/comments/mjug1i/removing_the_prime_minister_from_the_podium/
lures get-url 7 "
I get a link, but the link is invalid ...
Thank you !

I think you just not own google.com

I think it's from your phishlet.. try using another phishlet


errors.py

Why should I merge this? The exception you created is not used within the library and no description is provided.


Enable Broadcast for the UdpTransportTarget

How can I enable broadcasts for the UdpTransportTarget object in getCmd and setCmd use cases?

For IPv4 broadcast, you can refer to the example,
https://github.com/lextudio/pysnmp/blob/v5.0.24/examples/v1arch/asyncore/manager/cmdgen/broadcast-agent-discovery.py#L88


python312Packages.pydrawise: 2024.6.1 -> 2024.6.3

Diff: https://github.com/dknowles2/pydrawise/compare/refs/tags/2024.6.1...2024.6.3
Changelog: https://github.com/dknowles2/pydrawise/releases/tag/2024.6.3
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

5 packages built:

home-assistant-component-tests.hydrawise
python311Packages.pydrawise
python311Packages.pydrawise.dist
python312Packages.pydrawise
python312Packages.pydrawise.dist

Sorry, master is on 2024.6.2, which causes the merge conflict.


please add demo colab

please add demo colab for test my own imges

If you create one, I'll be happy to add it


panic when pressing  within the terminal

I'm using macOS and fish shell.
Whenever I just hold Ctrl + l in kitty terminal, I get this error
❯ thread 'main' panicked at library/std/src/io/stdio.rs:1118:9:
failed printing to stdout: Broken pipe (os error 32)
stack backtrace:
   0:        0x104e13340 - <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt::h1f3776e0b5c7517d
   1:        0x104e348d8 - core::fmt::write::heedef092c8c0962e
   2:        0x104e10180 - std::io::Write::write_fmt::h7178e8e2ea928914
   3:        0x104e13198 - std::sys_common::backtrace::print::h417292deb95532ed
   4:        0x104e142f4 - std::panicking::default_hook::{{closure}}::h0cb68f1228c4613a
   5:        0x104e13fe8 - std::panicking::default_hook::h24535936bc1f51de
   6:        0x104e14bac - std::panicking::rust_panic_with_hook::h5db4d2345b297bed
   7:        0x104e145dc - std::panicking::begin_panic_handler::{{closure}}::h3fd558f09a0d5492
   8:        0x104e137c8 - std::sys_common::backtrace::__rust_end_short_backtrace::hfc76eebe1ce501b2
   9:        0x104e1434c - _rust_begin_unwind
  10:        0x104e57f54 - core::panicking::panic_fmt::hc2b459a5bd3dce66
  11:        0x104e0eed8 - std::io::stdio::_print::h64cfa4dfe0b98263
  12:        0x104a59230 - ghr::cmd::shell::Cmd::run::hb8f437acdbdcad84
  13:        0x104a0f46c - ghr::cmd::Cli::run::{{closure}}::hb3c5f839659e0c79
  14:        0x104a093b4 - tokio::runtime::park::CachedParkThread::block_on::h80775a4ea28658f5
  15:        0x1049993bc - ghr::main::hf0b9b03fd605d3e1
  16:        0x104a05240 - std::sys_common::backtrace::__rust_begin_short_backtrace::h4a000ba23e4eb156
  17:        0x1049d63b8 - std::rt::lang_start::{{closure}}::h45b22d0143ace0df
  18:        0x104e0850c - std::rt::lang_start_internal::hecc68fef83c8f44d
  ~

@yujinyuz I am also a macOS + fish user, but I can't reproduce the issue. How's your config.fish?

Oops. I guess it was partly my fault since I have this
# ~/.config/fish/conf.d/_ghr_init.fish
ghr shell fish | source

And I'm also using tide so it might be calling something under the hood.
But changing the config to
! set -q GHR_INIT; and set -gx GHR_INIT true; and ghr shell fish | source

made the error go away

hmm. If sourcing the same line twice can reproduce the problem, maybe we can investigate further what's going on and decide whether or not we should take care of it or not


Use alloc::VecDeque rather than RingBufffer in device

I'm not sure why we didn't do this in the first place. Removes a dependency. The ring buffer crate we are currently using would make sense if we weren't using an allocator but we are.

Merged in https://github.com/frostsnap/frostsnap/pull/111


Does this tool support --no-sign-request?

Hi, I am using this tool for downloading open access s3 data, which does not require credentials. How can I add the parameter "--no-sign-request"?

Data Transfer Hub support no-sign request

The new version 2 already supported no-sign request.


Add Meatless Industries from Kerbal Stuff

This pull request was automatically generated by Kerbal Stuff on behalf of DuoDex, to add Meatless Industries to CKAN.
Please direct questions about this pull request to DuoDex.

This is, in accordance with its KS page an identical copy of BaconLabs, just without mentioning the word Bacon... "for vegetarians".
What is the policy on things like these, I can't really think of a case quite like it.

Oh goodness, this is beautiful.

What is the policy on things like these, I can't really think of a case quite like it.

We index it. We're the Comprehensive Kerbal Archive Network, after all. :)
Except for when a mod is actually hostile (eg: installs a rootkit), I hope we won't be policing what can and cannot appear in the mod list. (And if we do, we've made sure all our source and metadata can be forked so another group can start the FreedomKAN in response.)

:+1: As long as it abides by the terms of the original license and Kerbal Stuff doesn't remove it for their own policy reasons¹, we should index it.
--
¹ I could actually see KS prohibiting low effort forks of mods to prevent clutter from everyone and their mama creating their own personal version of a mod if this becomes more prevelant, as with flag and craft "mods".


adding new merche

👷 Deploy Preview for sparkly-mousse-72daf9 processing.

Name
Link

🔨 Latest commit
791ed2302f795f5b70d3fe0731a0c4c432845df5

🔍 Latest deploy log
https://app.netlify.com/sites/sparkly-mousse-72daf9/deploys/6435eb33f9f77f0007ac2e87


update init image to 0.13.0

update init-image to 0.13.0 this includes fix https://github.com/openshift/odo-init-image/pull/38

/approve

/retest

/retest

Typo in image name :-( This is why the tests were failing.
/retest

/lgtm

/approve


Git commit box does not warn long messages

Testing #20999
There is a setting to enable warning for long commit messages. Even though it is enabled, there is no warning shown for long messages. Also it would be helpful to update the description of the setting about the length of the message

Hey @joaomoreno! I'd very much like this feature after starting using visual studio code. Can I pick this up and submit a patch? If so, something I should keep in mind?

I admire your effort, but this won't be a simple task, since git is now an extension. You'll be limited by the extension API.
If you want to contribute to VS Code I suggest to go through the help wanted issues first.

I still miss it a lot. There are a couple stupid things I do to work around this a bit though:

type the message in the editor on an empty line first and see its length according to the status bar. e.g. if the cursor is at the end of the line and the status bar says Ln 15, Col49, then it should be 48 characters long.

(on mac), make the font of the input box monospaced by adding css to vscode's own css (located at /Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/electron-browser/workbench.main.css) and pasting this below all of the css

.scm-viewlet>.scm-editor>.monaco-inputbox {
  font-family: monospace;
}

that does make a warning pop up in vscode that it's unsupported though, and it will have to be redone every time a new version of vscode is installed

I like how Atom tucks a character countdown in the corner.
Yellow after 55 characters, then red after 72 characters. Negative numbers to count excess characters are handy too.

@jayjun
You gave me an idea and it totally works on the Mac version (1.17.2)!!! (I don't know if you have a Mac, so this might only be for Mac users).

You can add additional JavaScript to the VS Code app, so why not add some DOM-manipulation!?!
You can accomplish it using the following steps:

find the Visual Studio Code.app application file in your Mac's Application folder
right-click the app and click Show Package Contents
navigate to the Contents/Resources/app/out folder
open the paths.js file
add the following code to the bottom of the file:

window.addEventListener("keyup", gitCharacterCounter, false);

function gitCharacterCounter() {
  var mirror = document.getElementsByClassName("mirror")[0]

  if (mirror) {
      var length = mirror.textContent.replace('Message (press Cmd+Enter to commit)', '').length
      var titleLabel = document.getElementsByClassName("title-label")[0]
      var titleSpan = titleLabel.getElementsByTagName('span')[0]
      var labelBase = titleSpan.textContent.replace(/\([0-9]{0,100}\)/,'')

      titleSpan.innerHTML = labelBase + ' (' + length + ')'
  }
};

that's it! You will have to reload the current window of VS Code you're using

@scottclayton Wow, that’s an amazing hack!
I found the actual code responsible for those elements. InputBox listener here and getTitle() here.
I think adding a counter to the actual InputBox or surrounding elements is more acceptable since multiple SCMs could be active.
Would love to work with you to get a pull request in!

@jayjun Thanks!
I'm totally on board for working on a PR for restoring the feature—I've been missing it nearly every day since its removal.
I think implementing a running count of characters plus a warning when a certain length is exceeded—(configurable as settings for whether or not the warning happens and also what the length limit would be for the warning when exceeded)—would be a nice approach, since we would be getting the character count for free every time we check if the length exceeds the limit anyway. What do you think?
I couldn't see any active PRs for the feature, so I guess this is our time to shine.
I have no idea how people on github communicate with one another, so I've made a gist for us to discuss it if you're keen: https://gist.github.com/scottclayton/2c463746002ed2070ab1914bb7c7cafb

@scottclayton I submitted a WIP pull request. Let’s continue our discussion/hacking there!

Fixed by https://github.com/Microsoft/vscode/pull/36890


feat(playground): add substrait validator playground

An attempt to close #6.
Demo
This adds a playground directory that consists of a substrait-playground crate that provides wasm bindings for the validator. It uses the pbjson branch of the substrait-rs bindings to deserialize the provided JSON input. A very basic svelte app renders an input field and the validator output for that. CI can deploy this to GitHub pages.
I did have to make some modifications in the validator because a url method was not implemented for the wasm target.
I'm just putting this out here for visibility. This still requires some front-end work to be usable. And there are many opportunities to improve:

reduce the wasm blob size (currently 4MB)
add selector for example JSON inputs
add SQL input
add binary input
match substrait website style
(maybe) use validator protobuf and render in the browser based on that (instead of using the validator html output)

To build this locally you need nodejs:
cd playground
npm ci
npm run dev

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Conda recipe

What happened?
The documentation states that we have an earthkit recipe but it doesn't seem to exist:
https://earthkit.readthedocs.io/en/latest/install.html
What are the steps to reproduce the bug?
Either upload an earthkit recipe or remove this approach from the documentation.
Version
latest
Platform (OS and architecture)

Relevant log output
No response
Accompanying data
No response
Organisation
No response

Documentation has been updated.


Add resource requests and limits in pod spec in manifest files

Need to allow prometheus pods to use the entire node.

what is the current limitation?

prombench:v1.0.0 specified the resource limits almost equal to the node CPU & memory capacity.  https://github.com/prometheus/prombench/blob/v1.0.0/spec.example.yaml#L8

there should be a way to remove all kind of limitations. at least with docker unless you specify limitations it uses all node power.

Using all node-power might cause node to crash


Please ignore. This is for testing

This is for testing.

@bazel-io fork 7.0.1

@bazel-io fork 7.0.1

@bazel-io fork 7.0.1

@bazel-io fork 7.0.1

@bazel-io fork 7.0.1

@bazel-io fork 7.0.1

@bazel-io fork 7.0.1

@bazel-io fork 7.0.1


ESLint 5 Breaking Change: Globs to lint MUST be valid

I recently migrated Lynt to ESLint v5, but there was one breaking change I forgot to take account of: Linting nonexistant files from the command line is now a fatal error. If globs were not being passed to lynt (e.g. you were just doing lynt --react without specifying files to lint), I was passing in the following globs for you: ['**/*.js', '**/*.jsx']. However, this will now cause an issue with ESLint if you don't have any .jsx files because the globs that you pass in MUST exist now.
To fix this, I will need to "deglob" these default glob patterns and pass in a valid list of files to ESLint.

This is fixed as of lynt v0.5.5.


Add Login URL Under Registration Form Button

User Story: Add Login URL Under Registration Form Button
Title
As a user, I want to have a login link below the registration form, so I can easily navigate to the login page if I already have an account.
Description

As a user visiting the registration page.
I want to see a link to the login page directly below the "Register" button.
So that I can quickly switch to the login page if I realize I already have an account.

Acceptance Criteria

Given I am on the registration page,
When I scroll down to the bottom of the form,
Then I should see a text link that reads "Already have an account? Login here."

Given I click on the "Login here" link,
When I click the link,
Then I should be redirected to the login page.

Mockup

Technical Notes

The login link should be styled consistently with the rest of the form elements.
The link text should be easily clickable and clearly visible to the user.
The login link should redirect to the existing login page route (e.g., /login).

Tasks

Update the registration form component to include a Link component under the registration button.
Style the link to align with the current design language of the application.
Implement navigation to the login page when the link is clicked.
Test the feature to ensure the link is functioning as expected on different screen sizes and devices.

Attachments

Hi, I will work on this feature.

For sure , @komma-charan-dev

Hi, I would like to work on this


Adding MapperServiceProvider for plugins

Description
This PR introduces a new functional interface, MapperServiceProvider, to support the Query Insights plugin in accessing field types for search requests. The IndicesService holds the index mappings for each index, but rather than exposing the entire service to plugins, this interface allows them to retrieve the MapperService for specific indices through the getMapperService(Index index) method.
Related Issues
RFC https://github.com/opensearch-project/query-insights/issues/69
PR https://github.com/opensearch-project/query-insights/pull/130
Check List

[ ] Functionality includes testing.
[ ] API changes companion pull request created, if applicable.
[ ] Public documentation issue/PR created, if applicable.

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.

@dzane17 could you please provide more context behind this change: why do you need to expose the IndicesService to plugins? thank you

@reta thanks for the feedback, I added to the PR description

@reta thanks for the feedback, I added to the PR description

Thanks @dzane17 , it still does not answer the question (but leads to more questions):

why you are trying to export the IndicesService but referring to MapperService?
in which context you need access to MapperService?

Could you please share the reference to the pull request or associated issue so it would be possible to understand the problem you are solving that asks for such solution.

@reta

MapperService is per index so instead of passing multiple instances, I originally passed the entire IndicesService which contains MapperService for each index. However with @jainankitk's suggestion of MapperServiceProvider, we are now exposing a single method instead of IndicesService.
Field type is needed for query shape in Query Insights. Query shape generation occurs after shard search, on the coordinator node. See onRequestEnd() here.

I have linked the related RFC and PR above.

I have linked the related RFC and PR above.

Thanks @dzane17 , MapperService is contextual component, I believe what you need is to use getGuiceServiceClasses extension instead of createComponents:
    @Override
    public Collection<Class<? extends LifecycleComponent>> getGuiceServiceClasses() {
        return List.of(QueryInsightsService.class, QueryInsightsListener.class);
    }

In this case, you could inject IndicesService directly and create to QueryInsightsService / QueryInsightsListener (when appropriate):
@Inject
    public QueryInsightsService(
        final ClusterSettings clusterSettings,
        final ThreadPool threadPool,
        final Client client,
        final MetricsRegistry metricsRegistry,
        final NamedXContentRegistry namedXContentRegistry,
        final IndicesService ndicesService
    ) {
  }

(same applies to QueryInsightsListener)

I have linked the related RFC and PR above.

Thanks @dzane17 , MapperService is contextual component, I believe what you need is to use getGuiceServiceClasses extension instead of createComponents:
    @Override
    public Collection<Class<? extends LifecycleComponent>> getGuiceServiceClasses() {
        return List.of(QueryInsightsService.class, QueryInsightsListener.class);
    }

In this case, you could inject IndicesService directly and create QueryInsightsService / QueryInsightsListener (when appropriate):
@Inject
    public QueryInsightsService(
        final ClusterSettings clusterSettings,
        final ThreadPool threadPool,
        final Client client,
        final MetricsRegistry metricsRegistry,
        final NamedXContentRegistry namedXContentRegistry,
        final IndicesService ndicesService
    ) {
  }

(same applies to QueryInsightsListener)

Thanks @reta for this suggestion. Definitely something we should consider. Although, QueryInsightsListener is added to the SearchOperationsListener list in the core when returned via createComponents. Wondering if the same behavior persists with injection using GuiceServiceClasses

Wondering if the same behavior persists with injection using GuiceServiceClasses

It has to, if not - that would be a good fix to have, thanks@dzane17

@dzane17 @jainankitk I haven't taken a deeper dive into this PR yet, but would this work for your use-case:

Implement onIndexModule (similar to security): https://github.com/opensearch-project/security/blob/830b341453908332e4004d519ac2e0b099dca1f6/src/main/java/org/opensearch/security/OpenSearchSecurityPlugin.java#L689
Get instance of mapperService like this: https://github.com/opensearch-project/security/blob/main/src/main/java/org/opensearch/security/configuration/SecurityFlsDlsIndexSearcherWrapper.java#L80

Okay, we will need to add pluginLifecycleComponents to the SearchRequestOperationsCompositeListenerFactory list.

@jainankitk why do you need that? The QueryInsightsListener could implement the LifecycleComponent and that should be sufficient

@reta - How else will the QueryInsightsListener get notified of the search events, if the QueryInsightsListener is not returned as part of pluginComponents anymore. Below change was added to include the QueryInsightsListener for search events:
pluginComponents.stream()
                            .filter(p -> p instanceof SearchRequestOperationsListener)
                            .map(p -> (SearchRequestOperationsListener) p)

@reta - How else will the QueryInsightsListener get notified of the search events, if the QueryInsightsListener is not returned as part of pluginComponents anymore.

@jainankitk that is exactly the issue that @dzane17 brought up and we may need to fix: the instances of classes returned by getGuiceServiceClasses() could implement SearchRequestOperationsListener as well but this fact is completely ignored at the moment.

@reta @jainankitk I originally intended to solve this by moving the creation of SearchRequestOperationsCompositeListenerFactory  to later in node.java so that we can capture listeners from both pluginComponents and pluginLifecycleComponents.
            final SearchRequestOperationsCompositeListenerFactory searchRequestOperationsCompositeListenerFactory =
                new SearchRequestOperationsCompositeListenerFactory(
                    Stream.concat(
                        Stream.of(
                            searchRequestStats,
                            searchRequestSlowLog,
                            searchTaskRequestOperationsListener,
                            queryGroupRequestOperationListener
                        ),
                        Stream.concat(
                            pluginComponents.stream()
                                .filter(SearchRequestOperationsListener.class::isInstance)
                                .map(SearchRequestOperationsListener.class::cast),
                            pluginLifecycleComponents.stream()
                                .filter(SearchRequestOperationsListener.class::isInstance)
                                .map(SearchRequestOperationsListener.class::cast)
                        )
                    ).toArray(SearchRequestOperationsListener[]::new)
                );

However, pluginLifecycleComponents & getGuiceServiceClasses() are called after searchRequestOperationsCompositeListenerFactory (and all other injectables) are already bound to the instance here.
We face a paradox where getGuiceServiceClasses() relies on dependency injection, but we are now attempting to return an object to be bound (QueryInsightsListener) from getGuiceServiceClasses().

We face a paradox where getGuiceServiceClasses() relies on dependency injection, but we are now attempting to return an object to be bound (QueryInsightsListener) from getGuiceServiceClasses().

Oh I see, thanks @dzane17 , that's unfortunate, let me think it through a bit

Okay, we will need to add pluginLifecycleComponents to the SearchRequestOperationsCompositeListenerFactory list.

@jainankitk why do you need that? The QueryInsightsListener could implement the LifecycleComponent and that should be sufficient

@reta - How else will the QueryInsightsListener get notified of the search events, if the QueryInsightsListener is not returned as part of pluginComponents anymore. Below change was added to include the QueryInsightsListener for search events:
pluginComponents.stream()
                            .filter(p -> p instanceof SearchRequestOperationsListener)
                            .map(p -> (SearchRequestOperationsListener) p)

@jainankitk Idk if this would be helpful as well, but Security creates a SearchOperationListener within onIndexModule: https://github.com/opensearch-project/security/blob/830b341453908332e4004d519ac2e0b099dca1f6/src/main/java/org/opensearch/security/OpenSearchSecurityPlugin.java#L754-L837

The use-case for Security is Fls/Dls/FieldMasking feature which cluster administrators use to hide sensitive data stored in the clusters to users. Security must intercept search operations to apply the rules.

Thanks @cwperks , I think onIndexModule is quite a viable alternative as well

The use-case for Security is Fls/Dls/FieldMasking feature which cluster administrators use to hide sensitive data stored in the clusters to users. Security must intercept search operations to apply the rules.

Thanks @cwperks , I think onIndexModule is quite a viable alternative as well

@cwperks - Please correct me if I am wrong, but onIndexModule can only be leveraged for listening shard level search events, not request level. We need the search request listener for what we are trying to achieve here! cc: @reta

@cwperks - Please correct me if I am wrong, but onIndexModule can only be leveraged for listening shard level search events, not request level. We need the search request listener for what we are trying to achieve here! cc: @reta

Correct, its shard-level. To my knowledge, the only request-level extension point is the handlerWrapper, but that is reserved for the security plugin to perform auth.
My knowledge of how the internals of Search work is limited, but my understanding is that a SearchRequest lands on a coordinator node that then uses the routing table to figure out where to route the internal shard search requests to. It then sends out shard search requests to various nodes and collects the results before returning the results to the client.
Is it possible to distinguish the request that lands on the coordinator node vs. the shard requests?
iirc on the transport layer its the difference between indices:data/read/search vs indices:data/read/search[s]

@cwperks - Please correct me if I am wrong, but onIndexModule can only be leveraged for listening shard level search events, not request level. We need the search request listener for what we are trying to achieve here! cc: @reta

Got it, so here is the way we could make it work, which is a bit cumbersome but solves the dependency problem, the QueryInsightsService in injected as the service class:
    @Override
    public Collection<Class<? extends LifecycleComponent>> getGuiceServiceClasses() {
        return List.of(QueryInsightsService.class);
    }

Since QueryInsightsListener is created before QueryInsightsService (and it is injectable), we could inject QueryInsightsListener right there and initialize it with the indicesService or QueryInsightsService (fe using setter) since none of this services are needed during QueryInsightsListener initialization:
@Inject
    public QueryInsightsService(
        final ClusterSettings clusterSettings,
        final ThreadPool threadPool,
        final Client client,
        final MetricsRegistry metricsRegistry,
        final NamedXContentRegistry namedXContentRegistry,
        final IndicesService indicesService,
        final QueryInsightsListener listener
    ) {
  }

Seems due to caching changes:
[Test Result](https://build.ci.opensearch.org/job/gradle-check/49029/testReport/) (1 failure / ±0)

    [org.opensearch.indices.IndicesRequestCacheCleanupIT.testDynamicStalenessThresholdUpdate](https://build.ci.opensearch.org/job/gradle-check/49029/testReport/junit/org.opensearch.indices/IndicesRequestCacheCleanupIT/testDynamicStalenessThresholdUpdate/)