Include AsciiDoc extension by default

Please consider adding the AsciiDoc extension to Theia Blueprint as a default extension: https://github.com/asciidoctor/asciidoctor-vscode.

@Jeeppler I'm not sure it is necessary to add by default, you can install it from the extensions-view given that it is available from open-vsx:

https://open-vsx.org/extension/asciidoctor/asciidoctor-vscode

The discussion falls under the same idea as https://github.com/eclipse-theia/theia-blueprint/issues/81#issue-862898518 and https://github.com/eclipse-theia/theia-blueprint/issues/81#issuecomment-823431155. We should not bloat the application with extensions that users may or may not need, instead they are free to install them at runtime.

@vince-fugnitto thanks for the explanation.


[turborepo] Do something smart if persistent: true tasks will (or have) exhausted available concurrency.

Describe the Bug
If you have more than 10 concurrent parallel tasks you can starve the rest of your graph and never process it.
Expected Behavior
We should do something smart here, probably a combination of both of these:

Warn or error in advance. After building the graph we can walk by distance from root by graph branch and identify the maximum number of persistent tasks and hard error.
If all execution slots are taken up by persistent tasks, grow available concurrency by one. (Only one non-persistent task is ... not great.)

Originally reported in #4107. Thanks, @loick!

If --concurrency isn't passed and if there are more than 10 persistent tasks to be executed, we could change the option to N+1?


Added properties to the items

Addressing nelmio/NelmioApiDocBundle#1092
Issue was closed for wrong reason and without addressing the problem

Since Items is an instance of Schema in swagger 3.0, I think we can stop distinguishing the two (deprecate Items in favor of Schema).

At the moment this library is targeting version 2

Indeed, but if we strictly followed 2.0 spec, your PR would not be mergeable as well since 2.0 spec specifies that items can't be a model.

That is right :)
But i was hoping for this to get merged for the same reason GuilhemN/swagger#3 got merged

Well if we add properties to Items I do no longer see the need to differenciate it from Schema. This is the biggest difference between them right now isn't it ?

Yes.
Shall i just make the Schema not final and have Items extend Schema ? This will keep BC while proving full functionality

Yes please :)

Done.
Fixed tests as well, in case you want to setup travis :)

Great, thank you @dragosprotung! :)


Check the existence of the gzipped path explicitly

We couldn't depend on the previous check and it cause errors randomly.
This also fixes issues like https://github.com/zeit/next.js/issues/677

@rauchg Check now.

Looking forward to removing the check, doesn't seem necessary


prompt improvement and dynamic review of the text

Hello, first thank you soo much for creating this project.
I want to know the way you finded the prompt used in the code ?
Then do you think we can improve the output quality by creating the prompt with openai gpt-builder ?
Do you think it's a good idea to review the translation within this code by GPT4.5-turbo to increase the quality of the translation ?
thanks for everything

@jb41 did you have time to review my last comment ?


pip install -r requirements.txt has some dependency/version issues

I have seen this error with other projects (not yours) when the python version was in mismatch. I currently have 3.9.6 (errors were reported with 3.11.x)...so not sure what may be going on here.

Collecting uvloop==0.17.0 (from -r requirements.txt (line 112))
Using cached uvloop-0.17.0.tar.gz (2.3 MB)
Preparing metadata (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [6 lines of output]
Traceback (most recent call last):
File "", line 2, in
File "", line 34, in
File "C:\Users\patbh\AppData\Local\Temp\pip-install-qa53rwky\uvloop_94d9148e502f4a8689747849ae1f0a57\setup.py", line 8, in
raise RuntimeError('uvloop does not support Windows at the moment')
RuntimeError: uvloop does not support Windows at the moment
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

just to get to the bottom of this,

raise RuntimeError('uvloop does not support Windows at the moment')

A later version of uvloop itself works with Windows, but not the 0.17.0 version? Why do you think it was a python version mismatch issue?
I need some help troubleshooting this as I'm using Linux as my daily driver, but if the fix is as simple as bumping up uvloop then it'd be easy to review and / or merge.

Thanks for the response. Let me try a few things and let you know. The reason why I suggested it could be the python version is that in the past that was the case for other projects that I cloned and ran from GH.

Unless I rebuild a whole bunch of packages, I will not be able to use the code on Windows. It will be nice if you can release a Windows working version as well. Thank you.

Solved in the latest commit of #7

Thank you.


add JSON string deserialization for nip15 StallData in nostr-ffi

This is needed in order to use the JSON parse function for StallData from foreign function interface as mentioned in #308

Description

Notes to the reviewers

Checklists
All Submissions:

[✅ ] I followed the contribution guidelines
[✅ ] I ran make precommit before committing

New Features:

[ ] I've added tests for the new feature
[✅ ] I've added docs for the new feature

Bugfixes:

[ ] This pull request breaks the existing API
[ ] I've added tests to reproduce the issue which are now passing
[ ] I'm linking the issue being fixed by this PR

@yukibtc
Thank you for adding JsonUtil for NIP15 structs in such short time. In this PR I'm trying to add it to the nostr-ffi nip-15 so I can finally use it from FFI.
I only added StallData for now since I'm not sure if this is correct implementation. Let me know if there's anything needed in this PR, I'm happy to modify it if needed.

Before merge, can you squash the commits?

Done!


Installation with GPU Support

I have a doubt in the installation section. Can the GPU support also be installed for computers with AMD Radeon GPU's or is it exclusively for NVIDIA GPU's?
Thanks.

Only NVIDIA GPUs are supported. More information here.


OC EFI ITLWM

Hi XXCoreRangerX
first of all thank you for providing and sharing your EFI with us, in my case it's working great on my HP 840 G4 i5 7300u touch
I tried to inject ITLWM to make my stock Intel 8265 work but in vain I can't make it to boot
If you would mind and share with us a working EFI for Intel stock users would be great
Thank you in advance
Have a good day

hello XXCoreRangerX
ApECID in config.plist is already set to 0
i tried upgrading your opencore EFI with latest one and updated all kexts and added the AirportItlwm kext too
but when i boot it gives me kernel panic like
this OC grabbed zero system-id for SB this is not allowed
Halting on critical error
maybe i missed something as i dont have good knowledge in hackintosh
i attached the EFI i upgraded to look on it
please if you can upload en EFI with itlwm it would be great
thank you in advance:
EFI.zip
)

Alright, your EFI looks fine. Except that the kext order is wrong. You do NOT want to place any kext before Lilu and VirtualSMC. That's the reason your laptop doesn't boot.
Try this EFI:
EFI.zip


When creating a challenge, error while building tasks

Creating an Overpass based challenge based on query below.
Starts building tasks, but after a while gives 'Error retrieving tasks from server'
And then MapRoulette becomes unavailable altogether.
It has quite a few tasks, perhaps that is the issue here?
area["name"="Durham County"]->.a;
(
  way(area.a)["tiger:reviewed"="no"];
);
out meta;
>;
out meta qt;

Part of the problem may be the huge number of tasks involved, which in this case was caused by a faulty Overpass query. This query returns both ways and nodes (from the > recurse down statement)  so it creates way more tasks than intended. Perhaps a warning is in order when an overpass result returns mixed feature types, which is almost certainly not what the user intended.

We should be using out geom rather than recursing down. When doing that you get both the ways and the nodes.


CNN-DSSM, LSTM-DSSM, MMoE, ShareBottom, video recommendation

CNN-DSSM, LSTM-DSSM, MMoE, ShareBottom, video recommendation

 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.


[Pick 2.0](inverted index) fix wrong need read data opt when encounters columnA > columnB predicate (#33855)

Proposed changes
Issue Number: close #xxx
Pick From #33855
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


chore(master): release 2.3.1

:robot: I have created a release beep boop
2.3.1 (2024-10-03)
Bug Fixes

strip tmpDir from output dir (#108) (80c92bf)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v2.3.1
:sunflower:


Add targetPort in webhookdefinitions while scaffolding CSV

What did you do?
Attempted to generate a CSV using the SDK for an operator that includes a Webhook.
What did you expect to see?
The SDK does not define a containerPort or targetPort for the webhook:

OLM defaults containerPorts to 443 if not specified.
OLM will set the targetPort to the value of the containerPort if not specified

Given the above, the Operator-SDK will only generate a correct CSV if the webhook server is configured to listen on port 443, which is not default behavior for webhooks generated by the SDK. The SDK should use available code to ID the targetPort and set this value in the CSV.
What did you see instead? Under which circumstances?
The SDK should use available code to ID the targetPort and set this value in the CSV.
Environment:
$ operator-sdk version
operator-sdk version: "v1.0.1", commit: "4169b318b578156ed56530f373d328276d040a1b", kubernetes version: "v1.18.2", go version: "go1.15.2 darwin/amd64", GOOS: "darwin", GOARCH: "amd64"
Possible Solution
Scaffold csv with containerPort specified in webhookDescription.

Closing this since https://github.com/operator-framework/operator-sdk/issues/4120 was reopened.


Sql Grammar for datetime2 format

Laravel on MSSQL is using by default datetime format to dates, but as I
learn, the date format returned by the mssql can change by server to
server, so the ideal is use the new format datetime2, on datetime2 the
default lenght is 7, so I think it is logical to update the
getDateFormat function to deal with the default value

I think would be nice too change all datetime creation to datetime2(7) on migrations

Wouldn't this be a breaking change to databases using the existing format?

maybe an modification to be made on laravel 5.3 ?


Internal clean up.

Add necessary headers to convenient the import to google.
Require holding mutex for some function that access the variables that guard by mutex.

@wally-the-cartographer merge

Merge requested by authorized user gaschler. Merge queue now has a length of 1.


Python Queue Function Unable to Locate Modules

From @MisterJames on January 6, 2017 21:16
I am using Python 2.7 and have a module which cannot install via pip. I have created a wheel using these instructions and uploaded the wheel into a wheelhouse directory in the function.

And here is the wheel in that directory:

I've added a requirements.txt to the root of the function with the following content:
--find-links wheelhouse
yara-python==3.5.0

If I run d:\python27\python.exe -m pip install -r requirements.txt from the Kudu console I see the following error:
error: Microsoft Visual C++ 9.0 is required (Unable to find vcvarsall.bat). Get it from http://aka.ms/vcpython27

But I don't think the above is a fruitful path to follow as I won't have the rights I need to get vc++ installed methinks.  And my understanding was that if the wheel were being used, that it shouldn't need a compile at this juncture.
I have added a import yara statement to the top of my Python script. If I try to save/run the Function from the Azure Portal, I see the following error in the logs:
2017-01-06T21:03:22.454 Exception while executing function: Functions.buildRule. Microsoft.Azure.WebJobs.Script: Traceback (most recent call last):
File "D:\home\site\wwwroot\buildRule\run.py", line 2, in <module>
import yara
ImportError: No module named yara

Is there another way to bring in this dependency? Is there a way to hint to Python/Functions that I want to use the precompiled wheel?
Copied from original issue: Azure/azure-webjobs-sdk-templates#388

From @MisterJames on January 8, 2017 5:6
Great, thanks for the suggestions @tohling, I'll explore this to see if Python 3 is a solution here.

From @tohling on January 6, 2017 23:27
Resolving the error "Unable to find vcvarsall.bat" requires the installation of Microsoft Visual C++ Compiler for Python 2.7 (See Details section)
Even if we were able to install this package, there are 2 issues worth noting:

Under the System Requirements section, it states,
Supported Operating System
Windows 7, Windows 8, Windows 8.1
The Azure Functions infrastructure runs on Windows Server 2012.  There may still be issues during runtime.

We may engage VS team for an update, but under the Details section, it states,
"This compiler package is entirely unsupported..."

Have you tried using Python 3 to see if it will resolve the issue?  Here are the instructions: https://github.com/Azure/azure-webjobs-sdk-script/wiki/Using-a-custom-version-of-Python

Will reopen if there is further issues


What about 4D images?

Should atlasreader also accept 4D images? At the moment it accepts 4D images, but it only uses the first volume. I think I'm ok with that (for the moment).
Otherwise, I could imagine looping through the volumes and adding an additional postfix after the filename.
Any thoughts?

I think that accepting 4D images might get a little tricky, so I'm more comfortable leaving it as 3D and just accepting the first volume. Alternatively, we could add a parameter to specify which volume should be used?

@rmarkello - Using a parameter is probably the best solution! But in my opinion, we can leave it on the side until someone requests this feature.
Except if you want to implement it :-) Otherwise you can close this issue.

Closing this issue for now.


[Question] How to get list of bot contacts?

How is it possible to get the contact list of my Bot?

I'm sorry I didn't understand clearly can you please specify if it is any specific channel you want to get the contact list from and what framework you are using for the Bot.

Sorry, so I have created my Bot in Python and using REST API interface to connect to BotFramework (my bot is connected to Skype) .
When I start my python app it can handle contactRelationUpdate -s, so that it knows the list of contacts who are connected with my bot, however if I stop my python app and after a while im restarting i would like to know that list of contacts again, and that's why I would like to some REST API call to the botframework and I'm expecting the list of users who are still contacted with my bot.
For example:

bot_handler.py started, which is handling BobBot's requests (BobBot has skype channel):
X joins to BobBot on skype -> bot_handler will get message and know that X joined BobBot
Y joins to BobBot on skype -> bot_handler will get message and know that Y joined BobBot
bot_handler.py is stopped
Z joins to BobBot on skype -> bot_handler will not get this message as it is not running
Y leaves BobBot on skype -> bot_handler will not get this message as it is not running
bot_handler is restared
How can I know the current status? How can bot_handler know the current contact list of BobBot (which is actually: X and Z)

i hope this clarifies my querry, and thanks in advance for the answers!

Anyone please answer.
@fanidamj @nwhitmont

@leonard88 I sounds like the issue can be easily resolved with using storage that persists when you bot is restarted.  Please open a new issue if you have any other questions.


Is it possible to use with AMD GPUs?

Is it possible to adapt/use this with an AMD GPU? I'm not sure on the technical side of things how this works/could work with AMD.
(This project looks awesome, hope I'll be able to try it one day)

Hey, appreciate your interest in the project.  About running it on AMD GPUs, honestly, I'm not too sure since I've only used NVIDIA. Maybe you could try running the API with python directly without Docker and see how it goes, the libraries I used to run the models are from huggingface so they might have some information about it.


Hide returning()/returningAll() when using unsupported db

It would be nice to have something to prevent returning()/returningAll() from showing when using MySQL for example. I wrote all my code when moving from prisma to kysely and it wasn't until I actually ran the code that I realised the mistake I had made.
Also see: https://github.com/kysely-org/kysely/issues/264

This has been discussed but it's too difficult to implement cleanly. We'd basically need a copy of kysely for each dialect and that doesn't work with third party dialects.


SVG Grouping the siblings

Hello Team,
Add multiple-element
Make it group
assign the ID
when I click on inner element it doesn't give me Id, because it created liner group instead of subgroup
Thanks in advance

I need below structure
The SVG Group Element is a container that contains all child SVG elements defined inside of it.
1
2
3
4      ....
5
6
7
but mx groupd follows below structure
1
2
3
4 ....
5
6
7

Hello
The SVG Group Element can be nested inside of other SVG Group Elements:
I require blow structure
<g>   <g>     <g>       ....     </g>   </g> </g>


feat: refactor deploy pipeline to split build and deploy

Part of https://github.com/ministryofjustice/find-moj-data/issues/590
The current pipeline has a couple of problems:

We use github's environment protection, but we access environment scoped variables in multiple jobs per environment, so we are prompted twice for approval to continue.

We build the docker image once per environment, so we can't guarantee the integrity of the one we end up with in prod.

To address this I have moved the ECR variables to repository level, so they can be used without setting an environment.
In deploy-staged.yml, we now build the image once, without requiring an extra approval step. Since we have repositories for each environment, the image is pushed to the next environment as soon as we have successfully deployed to the preceding environments.
I've also removed deploy-to-dev.yml, which triggered for every PR, because

It required us to have an otherwise unnecessary approval gate on dev, to prevent us unintentionally deploying on top of a branch somebody else is testing
It made it impossible to distinguish between "tests are still running" and "tests passed but the branch wasn't deployed" when looking at a list of PRs
We have a manually triggered deploy-dev-from-branch.yml which deploys a branch to dev without these problems.

ECR_ROLE_TO_ASSUME is set by https://github.com/ministryofjustice/cloud-platform-terraform-ecr-credentials/blob/main/main.tf and configured with a list of github environments (example)
Because of this configuration, all the secrets/variables for connecting to the ECR repo are currently associated with those environments, rather than being set at the repository level.
I think we need to change this because otherwise, the build job needs to set the environment key, which we don't want. If the environment key is added back in then github will add the approval gate.


Address incorrect placement of period

Fix issue where the period is mistakenly placed within the backticks, and remove some rogue periods.

@TylerMSFT
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


GH: Extend Speckle Object K/V is not working as expected.

Reported in: https://speckle.community/t/extend-keys-component-mixes-values/3786/8
The problem
I think something is off with our data matching logic, but it also may be that what we want to do goes against all Grasshopper common-sense and it may be better to re-think this.
At the very least, we should flag any limitations as errors so the user is aware that they're trying to do something out of the scope of that component.
In essence, we need to be able to automatically differentiate if the value of a specific provided key is an item or a list. The way this is done is by bypassing GH data matching capabilities and running our own matching logic based on the path of the current item to extend {X;Y}.
We fetch the key list for each object by finding the branch in the key input with the same path as the object.
Given a specific path, we'll look for the specific "subTree" in the value input tree (i.e. all the branches that start with {X;Y;...}.
Once that subTree is obtained, we iterate through the branches assuming each branch contains the value/s of each key.
Somewhere along the way this is failing to work as intended.
Possible solutions

Stop fighting grasshopper and flag this component (and the object by k/v) as obsolete. This is the easy solution, it requires 0 maintenance for us. Downside is that users would have to create their own python scripts to replace these components. Upside is that they'd have total control over what is a list, what is a tree and how things are mapped.

Determining what scenarios would not work under this logic and report them as errors, preventing execution and any weird behaviours that may happen. Downside of this is that this is very annoying to the user, and would require some pretty detailed explanations of why and what to do to fix it. Upside is that no mixed data should ever be possible.

Reviewed with @paloknapo


Registry key is not present in install

And this causes an error that can be fixed with a documentation update.

@brianreidc7 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Resolved with https://github.com/MicrosoftDocs/azure-docs-pr/pull/88185

#sign-off


Update scala-library to 2.13.14

About this PR
📦 Updates org.scala-lang:scala-library from 2.13.10 to 2.13.14
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

🔍 Files still referring to the old version number
The following files still refer to the old version number (2.13.10).
You might want to review and update them manually.
src/config.scala

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang", artifactId = "scala-library" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-lang", artifactId = "scala-library" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:1

Done in https://github.com/alexarchambault/scalacheck-shapeless/pull/299


XFA - Add a missing method to XFAAttribute, to prevent breaking errors (issue 13748)

This is yet another case where I've got no idea if the patch is correct, but it does at least fix a breaking error :-)
Note how in the Binder._bindValue method, we're assuming that if a data-value exists then it'll also be possible to actually access it. For the XFAAttribute-implementation however, the second method is missing and that's what causes the breaking errors in issue #13748.
Please note that another possible way of "fixing" the error wouldn't been to simply change the exists-check to return false, and I could see that being a preferred solution.
However, the reason for submitting the current patch is that we get fewer warnings about Nodes with mis-matched types this way.

/botio test

Is this fixing #13748 completely?

Is this fixing #13748 completely?

Unfortunately not, since there's a bunch of scripting in that document as well (I've updated the tags accordingly).

/botio makeref


[dsch] adding the type-hidden properties error to Ok and value to Err…

… of type never

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

8 of 8 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.7%) to 100.0%

Totals

Change from base Build<PHONE_NUMBER>8:
1.7%

Covered Lines:
1367

Relevant Lines:
1367

💛  - Coveralls


New version: Wilfred.difftastic version 0.56.1

Pull request has been created with komac v2.1.1 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Packages\Wilfred.difftastic__DefaultSource\difft.exe returned exit code: 2
Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Links\difft.exe returned exit code: 2

(Automated response - build 861.)

For moderators
This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping Exorcism0666.

[!warning]
With the new version of Komac, numerous Pull Requests may contain bugs or unwanted elements. If such issues arise, please tag russellbanks so that these bugs can be addressed in future updates.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

@wingetbot waivers Add Validation-Executable-Error

Completing validation.


[mesheryctl] mesheryctl mesh subcommand for viewing SMI conformance test results

See mesheryctl Command Tracker for current status of commands.
Current Behavior
There is no command to view SMI conformance results using meshery CLI.
Desired Behavior
We should have a subcommand for mesheryctl mesh validate for viewing the results by supplying the ID of a conformance test.
One, so as to create a similar experience as the perf commands. Two, to facilitate immediate display of results in GH action.
Implementation
Design spec: https://docs.google.com/document/d/1E0u9twIcHrnEa0NgytNxobQVRh6EnujfQU5Y0Ocvdq8/edit#

Contributor Resources

⌨ mesheryctl Contributing Guide
Beginner's guide to contributing to Meshery and mesheryctl
mesheryctl Command Tracker
Meshery CLI Commands and Documentation
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

@DelusionalOptimist this is a pending feature from a long time, because there's no api available to get a single Test Result.


⚠ Bofrost.fr has degraded performance

In bad55c5, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1442 ms

Resolved: Bofrost.fr performance has improved in de39ec9.


⚠ Bofrost.gr has degraded performance

In af16b55, Bofrost.gr (https://www.bofrost.gr) experienced degraded performance:

HTTP code: 200
Response time: 4006 ms

Resolved: Bofrost.gr performance has improved in 2322741.


Update reward on testnet.

I am pretty sure some tests are also need to be updated accordingly. Please run truffle tests with testrpc (not the special command line needed) and see what fails.

Good call.  I'll update (and make sure they work) those and then update the PR.

This one should be good to go now.


Fixes aws monorepo example

Updates references to latest sst package
Adds sst as a dependency on all projects

Thanks.


XMLException on download

Which service(blob, file, queue, table) does this issue concern?
blob
Which version of the Azurite was used?
3.11.0
Where do you get Azurite? (npm, DockerHub, NuGet, Visual Studio Code Extension)
dockerhub
What's the Node.js version?
unknown should be the one baked in docker image
What problem was encountered?
System.Xml.XmlException: 'Root element is missing.' on download
Steps to reproduce the issue?
Download in c# code
Have you found a mitigation/solution?
No

@bigpod98
Please see my comments in https://github.com/Azure/Azurite/issues/718
I can't repro this issue with Azurite 3.11.0 in docker and Azure Storage Explorer 1.18.1.
Please give the debug log, and the detail step to repro it.

Close this issue as #718 is also closed, and not Azurite code issue.


moving side effects to componentWillReceiveProps

Moved side effects from shouldComponentUpdate to componentWillReceiveProps since side effects in shouldComponentUpdate should be avoided.

Coverage remained the same at 96.442% when pulling a5cc8473d91d7a070094e82c947133ed8ceee7d1 on manuelbieh:master into 3381e154e025e094931b3d69f50e4c2f6608f31a on boundlessgeo:master.

thanks, I will check this out tomorrow morning

Glad to hear that 😉


fix(rust-analyzer): add _musl environment

@mason-org/triage notify(Reason::NewPullRequest)
This comment ensures a notification is delivered to maintainers.
 🤖 This is an automated comment.    📖 Source code


iOS and Android 3rd Party mobile apps cannot provide the device compliance state at login

I've raised this with Microsoft Support (ref:120021922001217) and they confirmed this does not work for the combination of Intune & AAD Conditional Access policies that require managed devices, & 3rd party mobile applications. In our case Workday, SAP Concur, Citrix Files as examples. The Microsoft mobile applications, Outlook, Teams etc. work perfectly, but other 3rd party apps are blocked from access and the logs show no device compliance information is passed by the 3rd party application to AAD.

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

ID: e7aca4a4-9951-2132-3910-cf83ba66a048
Version Independent ID: 2509fd1f-60e7-66ce-fc7e-c16b8cf73865
Content: Conditional Access require managed device - Azure Active Directory
Content Source: articles/active-directory/conditional-access/require-managed-devices.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@m-xen
Thanks for your feedback! We will investigate and update as appropriate.

Email out to PG for feedback.

Some third party apps may not see the state data. Will add an update to the doc in the near future. #please-close

Some third party apps may not see the state data. Will add an update to the doc in the near future. #please-close


How to find current location of the user using map_view in flutter?

I would like to set initialCameraPosition to point to the user's current position rather than hard code a longitude and latitude.
  showMap() {
     mapView.show(
      new MapOptions(
          mapViewType: MapViewType.normal,
          showUserLocation: true,
          initialCameraPosition: new CameraPosition(
              new Location(23.2599, 77.4126), 10.0),
          title: "Parking Spaces"),
      toolbarActions: [new ToolbarAction("Close", 1)]
      );

You can use mapView.onLocationUpdated.
mapView.onLocationUpdated.listen((location) => myUserLocation = location);

@matthewtsmith - As of this writing flutter_google_map_view (this lib is) deprecated, but it's worth noting that a complete version of the above
mapView.onLocationUpdated
        .listen((Location location) {
        mapView.setCameraPosition(new CameraPosition(location, 10.0));
    });

Will keep updating the map's location.


Publish Stage

What is the purpose of the change

ClickUp Task
ClickUp Task URL
Brief Changelog

Testing and Verifying

Documentation and Release Note

oops squashed by accident 😬
Is there a way to default the button for stage -> master to not be squash?

oops squashed by accident 😬
Is there a way to default the button for stage -> master to not be squash?

@sunnya97 I don't think so, at least we weren't able to figure it out. We either have to remove squashing for everything or not.
So long as we don't need to revert, we should be fine


Issue with useSeoMeta - og:image Not Displayed on Facebook Messenger and WhatsApp

Environment

Operating System: Linux
Node Version:     v22.2.0
Nuxt Version:     3.12.3
CLI Version:      3.12.0
Nitro Version:    2.9.7
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, runtimeConfig, css, modules, plugins, googleFonts, pinia, colorMode, postcss, router, vite, compatibilityDate
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction

Set up a Nuxt 3 project with SSR enabled.
Create a dynamic page pages/posts/[id].vue.
Use useSeoMeta to set the following meta tags:

// posts/[id].vue
[...]
const store = useStore();
const route = useRoute();
const post = computed(() => get(store, "post"));

useAsyncData("post", async () => {
  try {
    const { id } = route.params;
    await store.fetchPost(id);
    fetchSimilarPosts();
    return true;
  } catch (error) {
    console.log("Error: ", error);
  }

  return true;
});

useSeoMeta({
  title: () => post.value?.title,
  description: () => post.value?.subtitle,
  keywords: () => post.value?.categoryName,
  author: () => post.value?.authorBy,
  ogTitle: () => post.value?.title,
  ogDescription: () => post.value?.subtitle,
  ogImage: () => post.value?.thumbnail,
  ogImageSecureUrl: () => post.value?.thumbnail,
  ogImage_alt: () => post.value?.title,
  ogType: () => "article",
  ogUrl: () => `https://mywebsiteurl.com/posts/${post.value?.slug}`,
  twitterCard: () => "summary_large_image",
  twitterTitle: () => post.value?.title,
  twitterDescription: () => post.value?.subtitle,
  twitterImage: () => post.value?.thumbnail,
  ogUpdated_time: () => +new Date(),
});

Describe the bug
Steps Taken:

Cleared Facebook cache using the Facebook debugger tool.
Tested with Facebook debugger - no issues found.
Tried adding meta tags directly using <Head> and <Meta>components.
Included dimensions (width, height) and type for og:image.
Added og:image:secure_url.
Created a Facebook App and added fb:app_id to the meta tags.

Expected Behavior:
The og:image should display correctly on Facebook Messenger and WhatsApp when sharing posts.
Actual Behavior:
The og:image does not display on Facebook Messenger and WhatsApp, though the title and description are shown correctly. The meta tags work correctly on other platforms (Telegram, LinkedIn, Twitter, LINE, Facebook feed).
Additional context
I suspect the issue might be due to the reactive nature of post.value causing the meta tags to be added later in the , which might lead to them being ignored if there is a size limit.
Here's from Meta doc

However, I am not able to move all the meta tag from useSeoMeta to the top order in head tag.
Any specific configurations or steps to ensure the og:image meta tag works correctly on Facebook Messenger and WhatsApp would be greatly appreciated.
Logs
No response

我也出现了这种类似问题 我在nuxt.config.ts中写入
app: {
head: {
title: 'title_1',
}
}

然后在  pages/a.vue 中写入
useSeoMeta(title:'title_2') 活着 useHead(title:'title_2')
使用npm run generate打包后 发现 next/a/index 中出现的是 <title>title_1</title>虽然页面标题会显示title_1
但会导致在其他平台分享xxxxx/a 链接时 的标题是title_1而不是我想要的title_2


Fix gps on zephyr 3.6, PDB for b3rb, VMU for rdd2

Appears that the transition from zephyr 3.5->3.6 made the canhubk3 default to eth slave mode and requires running on the navqplus:
sudo ethtool -s eth0 master-slave forced-master
sudo ifconfig eth0 <IP_ADDRESS> netmask <IP_ADDRESS>


sqlalchemy_url config doesn't work

Tried to pass in sqlalchemy_url but it didn't seem to work for me, looks like it's just ignored?

It turns out it was just ignored. I have pushed a fix now, so it behaves as I described.


[release-4.15] OCPBUGS-31377: Manually cherry pick #3782 to 4.15

Manually cherry-pick #3782 to release-4.15.

/jira refresh

/jira refresh

@bryan-cox @Patryk-Stefanski Could you help approve this manual cherry-pick ?

/lgtm

/approve

/test e2e-kubevirt-aws-ovn

It's step phase pre that failed again.
/unhold

/test e2e-kubevirt-aws-ovn

/test e2e-aws


(WIP) Add preliminary support for aws_cognito_userpool/aws_cognito_userpool

❯ ./terraformer import aws --resources cognito --regions ap-southeast-1
2020/03/27 13:53:05 aws importing region ap-southeast-1
2020/03/27 13:53:05 aws importing... cognito
2020/03/27 13:53:05 aws importing cognito_identity_pool: ap-southeast-1:9e3f8239-0c8a-43ab-ac5c-c936b14649d2
2020/03/27 13:53:05 aws importing cognito_user_pool: ap-southeast-1_bZWf5VS8d
2020/03/27 13:53:10 Refreshing state... aws_cognito_identity_pool.tfer--test_cognitoidentity
2020/03/27 13:53:10 Refreshing state... aws_cognito_user_pool.tfer--testUserPool
2020/03/27 13:53:11 aws Connecting....
2020/03/27 13:53:11 aws save cognito
2020/03/27 13:53:11 aws save tfstate for cognito

Support of aws_cognito_user_pool_client is not straightforward in the call parameters for CreateResource, investigating the change along the ReadResource path

@googlebot I fixed it.


feat: Add optional device_name parameter for the aws_customer_gateway

Description

Adding optional param device_name to the cgw
Motivation and Context

I've got few existing VPN tunnels I'd like to move under tf management, and some cgws have that device_name set, but module doesn't allow to set it. Missing device_name results in cgw recreation.
Breaking Changes

n/a
How Has This Been Tested?

[ ] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[ ] I have tested and validated these changes using one or more of the provided examples/* projects
[x] I have executed pre-commit run -a on my pull request

Trivial change, so just tested in my env this solves the problem.

@antonbabenko could you review, plz 🙏

ping @antonbabenko 🙏

ping @antonbabenko 🙏


[Bug]: User accounts without finishing e-mail verification unintendedly removed

What happened?
First of all, thank you for developing such an outstanding tool! I really appreciate the effort and skill that has gone into making it so useful and effective.
It is unclear whether this is a bug, or intended behavior, or just our misunderstanding. However, we appreciate if you would check the following behavior.
User accounts with unverified email addresses are suddenly being deleted. This issue occurs when Email Password Reset is enabled and the email-related settings are active in the .env file, and the setting ALLOW_UNVERIFIED_EMAIL_LOGIN is set to true.
Under these conditions, when a new account is created, the user should be able to use LibreChat without email verification. However, in practice, an expiresAt field is added to the database, causing the account to be deleted one week after the expiration period.
This phenomenon leads to two UX-related issues:
Since users can log in without completing email verification, they may be unaware that their email address remains unverified.
Administrators may use these settings not for the purpose of enhancing security through user authentication, but merely to provide a password reset service. When these settings are configured with the latter intention, user accounts may be deleted contrary to the administrator's intent.
Steps to Reproduce

Configure .env file as follows:
ALLOW_EMAIL_LOGIN=true
ALLOW_REGISTRATION=true
ALLOW_SOCIAL_LOGIN=false
ALLOW_SOCIAL_REGISTRATION=false
ALLOW_PASSWORD_RESET=true
ALLOW_UNVERIFIED_EMAIL_LOGIN=true

EMAIL_SERVICE=
EMAIL_HOST=smtp.example.com
EMAIL_PORT=465
EMAIL_ENCRYPTION=tls
EMAIL_ENCRYPTION_HOSTNAME=
EMAIL_ALLOW_SELFSIGNED=
<EMAIL_ADDRESS>EMAIL_PASSWORD=example_password
EMAIL_FROM_NAME=MyOrganizationName_LibreChat_Admin
<EMAIL_ADDRESS>2. Create user account, but ignore the received email and do not push the Verify Email button in the mail body.
3. Wait two weeks after account creation, or just check the raw database record which contains expriesAt field.
What browsers are you seeing the problem on?
Microsoft Edge
Relevant log output
I could not find the log about user deletion, but the raw mongodb record of the unverified user is following (I modified some fields for privacy reasons.)

  {
    _id: ObjectId('12345678901234567890'),
    name: 'User Name',
    username: 'example',
    email<EMAIL_ADDRESS>    emailVerified: false,
    password: '12345678901234567890',
    avatar: null,
    provider: 'local',
    role: 'USER',
    plugins: [],
    expiresAt: ISODate('2024-10-16T01:48:34.263Z'),
    termsAccepted: false,
    refreshToken: [],
    createdAt: ISODate('2024-10-09T01:48:34.265Z'),
    updatedAt: ISODate('2024-10-09T01:48:34.265Z'),
    __v: 0
  },

Screenshots
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

thanks for reporting. while implementing email verification, I didn't think about this

@berry-13 I think this might've been fixed already, but I will have to test it to be sure.

Sorry, I was using older version #4291.

@tak-kanno No worries, I can see this is indeed happening as you said.
This is a bit of an edge case, since it's implied that enabling email services would require email verification. However, to boost customization and since configuration allows many options for this, I will follow the logic you expect.
It should be relatively easy to fix, but I'm sorry your users were affected.

Closing with #4482:

If the system is set to allow unverified email logins, it will no longer set an expiration date for new user accounts.

For existing users created before this change, if unverified emails are now allowed, the system will remove any expiration date from their account.

The system will still send verification emails if email services are enabled, even if unverified logins are allowed.

Since users can log in without completing email verification, they may be unaware that their email address remains unverified.

I agree on this point, that users should be reminded on every login that their email remains unverified. @berry-13 maybe you can help with this.


Feature to start process in camunda

Missing copyright holder name in license file: https://github.com/bigbank-as/go-camunda-client/blob/master/LICENSE#L189


Makefile: add target for release tarball

Add a target to automatically create the vendor tarball and the extra
files. This can only be used on a version which has not the -dev suffix
to prevent user errors.

@baude @mheon @flouthoc PTAL
This should automatically create the release files which then should be uploaded to github. I think it would be great if we could make this automatically in CI.

Maybe add an rm -rf for the vendor directory in there to ensure it doesn't get committed to the repo?
Otherwise LGTM

/lgtm


set/get doesn't trigger when setting dictionary/array by index

Godot version
Godot 4.1
System information
Godot v4.1.stable - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce 930MX (NVIDIA; <IP_ADDRESS>3) - Intel(R) Core(TM) i7-6600U CPU @ 2.60GHz (4 Threads)
Issue description
the set function of dictionary/array doesn't trigger when trying to set specific key or index
in godot 3.x you can do it fine with setget, but in godot 4.x it's looks like an old bug come again #37737
Steps to reproduce
in any script, write any dictionary/array with "set" and it's function with print line
then try to set key/index anywhere in your code (like _ready function)
example :
var data : Dictionary = {
	"money":300,
	"items":1,
	"slots":1
} : set = _new_data

func _new_data(value):
	data = value
	print("function triggered")

func _ready():
	data["money"] = 200

run the script to find no output
Minimal reproduction project
N/A

This is as expected and intended and not a bug IMO, it doesn't assign to the dictionary, it assigns to an index in it, it's an optimisation

is so, then every time i need to set one key, i need to re-set the whole dictionary?
in every line >->?

If you want this behaviour yes, and there you see what performance issue having that by default would be arguably

what about thing like set("dictionary:key", value) to trigger the setter?
set("data:money", 100)
if there is no way like this, then you can close the issue

Does it trigger it, I can't test at the moment
But to be clear, making it assign a dictionary or array each time it's modified would be bad for performance
It would also be confusing, as you could do:
var arr_copy = my_array
arr_copy[1] = y

And it wouldn't fire, even if it would when doing my_array[1] = y

nice to know it's intended and not but.
but right now what is the best way to do (asking for a good line of code)

For the future note that this isn't for support questions, please use the other community channels for that.
You will probably just have to set it manually, also see the other issues for suggestions, as people have already answered that there


#1421: For CircuitBreaker fallback scenario remove from requests attr…

Fix for issue #1421: CircuitBreaker redirection to external URI does not work

Closing in favor of #1489


Connect-EFlowVM / too many authentication failures / edge case for SSH / Solution Found

I was using my desktop computer to SSH into at least 10 or more Raspberry PI in the development of an application before starting the install for EFLOW... BTW.. this is the result of about 5 days of intense effort and going down a rabbit hole of Linux, docker, IOT Hub, etc.
Windows 10/11 using the "Default Switch" followed the guidance of: https://learn.microsoft.com/en-us/azure/iot-edge/how-to-provision-single-device-linux-on-windows-symmetric?view=iotedge-1.4&tabs=azure-portal
which resulted in the failure to connect via ssh:

Expected behavior
Connection the the VM via SSH, but with NO IPaddress being assigned, that was NOT possible
Screenshots
the solution

analysis leading up to the resolution:

Hi @codeputer,
Thanks for reaching out and sharing your investigation. A couple of things:

Our EFLOW VM is not using SSH over TCP/IP; we are using a vSock (Hyper-V socket between the host OS and guest OS) - For more information, see EFLOW Security.

Because we use vSock, there's no need for an IP address to establish the SSH connection. Also, regular ssh service won't run, and we have our service for vSock connections.

Finally, you need authentication; ssh is only enabled for certificate-based auth, and the cert is created and stored during EFLOW installation.

Thanks,
Francisco


_pickle.PicklingError: Can't pickle <class 'numpy.core._exceptions.UFuncTypeError'>: it's not the same object as numpy.core._exceptions.UFuncTypeError

when i use
dict(type='PhotoMetricDistortion',
brightness_delta=32,
contrast_range=(0.5, 1.5),
saturation_range=(0.5, 1.5),
hue_delta=18)
in config file for training
i got the following error:
Traceback (most recent call last):
File "/usr/lib/python3.6/multiprocessing/queues.py", line 234, in _feed
obj = _ForkingPickler.dumps(obj)
File "/usr/lib/python3.6/multiprocessing/reduction.py", line 51, in dumps
cls(buf, protocol).dump(obj)
_pickle.PicklingError: Can't pickle <class 'numpy.core._exceptions.UFuncTypeError'>: it's not the same object as numpy.core._exceptions.UFuncTypeError
that's all info.
i found the following code raise the error:
delta = random.uniform(-self.brightness_delta,self.brightness_delta)
img+=delta
in PhotoMetricDistortion class in pipelines/transforms.py file
other familiar code raise the error too.
i don't know why?
i found if delta is negative or float,it will raise the error.positive integer is ok.

when i use  img = img + delta to replace img+=delta,it's ok...

Do you use the config ssd300_coco.py, I can't reproduce your error with this config. Could you specify which config you use and what modification you apply to? Could you provide more detail information about your environment?

when i use PhotoMetricDistortion in retinanet training, I met exactly same error

Do you use the config ssd300_coco.py, I can't reproduce your error with this config. Could you specify which config you use and what modification you apply to? Could you provide more detail information about your environment?

I use the faster_rcnn_r50_fpn. torch-1.1.0, CUDA-10.1, numpy-1.18.2.Thanks. As mentioned above, when i replace '+=','*=',and such operation in PhotoMetricDistortion, the error disappear.

I found that the problem is that I didn't load image with float32=True, cv2.cvtColor doesn't support float64. That causes this wired error here:)

Do you use the config ssd300_coco.py, I can't reproduce your error with this config. Could you specify which config you use and what modification you apply to? Could you provide more detail information about your environment?

I use the faster_rcnn_r50_fpn. torch-1.1.0, CUDA-10.1, numpy-1.18.2.Thanks. As mentioned above, when i replace '+=','*=',and such operation in PhotoMetricDistortion, the error disappear.

@BChunlei  Just as @edwardyangxin  mentioned, when using PhotoMetricDistortion transformation, it is necessary to convert the image to np.float32 firstly just as what SSD does, you can refer here. But it is indeed not very friendly, would you like to create a PR to make it more friendly, i.e., report a reminder that user needs to set to_float32 flag?

Do you use the config ssd300_coco.py, I can't reproduce your error with this config. Could you specify which config you use and what modification you apply to? Could you provide more detail information about your environment?

I use the faster_rcnn_r50_fpn. torch-1.1.0, CUDA-10.1, numpy-1.18.2.Thanks. As mentioned above, when i replace '+=','*=',and such operation in PhotoMetricDistortion, the error disappear.

@BChunlei , Thank you for your answer.
So, why? what's the difference?


getting more than one prediction

you were assuming only one result coming back, i added this small method
  formatResults (data) {
    const lines = data.trim().split('__label__')

    const predictions = lines.map((line) => {
      const params = line.split(' ')
      const label = params[0]
      const confidence = Number(params[1])
      return { label, confidence }
    })

    return (predictions)
  },

@dcsan Thanks a lot. I'm working on an update :)

one thing with this is that the parameters to the process are added at startup when you load the FT object and models into memory. if you want to change the parameters, you'd have to reload the whole object, as far as I can see?

@dcsan So many problems with the way it works. I think it's better to use node-gyp but I have never tried it.
FWI, our company'll start using ML as a service instead of FastText.

ML as a service from where? I'm finding fastText is giving quite good results compared to some other cloud APIs and of course it's much quicker.

@dcsan Amazon ML is identical to FastText


Add Tags in Taglist on Enter

I would like the possibility of adding new tags in the TagList Control using the Enter key.

Enter key is reserved for selecting Tags from the suggested tags list. On Enter key press, control will be forced to resolve the tag.

If there is an existing tag in the suggested list, then the first one is selected from a list
If there is no existing tags and creating new is allowed, then currently typed text will be resolved as new Tag

Submitting a tag via keyboard is possible too:
From a box where the tags are added press "Tab" (to move the focus to the add (+) button) and press Enter.
Please, reopen the issue if I misinterpreted the request.


Is it possible to analyze a part of cells in one loom.file according to CellID

I have a loom.file of 3,000 cells (from SmartSeq2). But I only want to analyze one type of about 2,000 cells.
Can I select these cells from the loom.file according to CellIDs?
Or do I have to create another loom.file by velocyto run?

You can select the cells that you want to analyse by making a new filtered loom file. However the simplest way to pick a subset of cells is to do it as a filtering steps at the beginning of analysis you can do this using the velocyto's filter_cells method.

I tried
vlm.filter_cells(vlm.ca['CellID']==cellID_of_T_cells)
cellID_of_T_cells is the list of cellIDs
and I got
ValueError: Lengths must match to compare

This is not a problem of velocyto.
 Are you sure you are doing this in the right way? Maybe you should use the numpy.in1d function not the "==" statement.

Thanks for your reply!
I now get the correct result!
velocyto is really a great algorithm,it can deal with datasets that other pseudotime algorithms can't do!


Quicklook like toy

Description of the new feature / enhancement
Peek function is not as fast and smart as Quicklook from MS store.
Problem: currently when Peek opens a pdf you can only navigate the content of that pdf when pressing keyboard arrows.
Improve: this function should work like that only when the focus is on Peek's window. It's necessary to add the case when you are focusing the file explorer so you can peek all the files with keyboard arrows.
Scenario when this would be used?
navigating a list of pdfs or videos without having to close and open each time Peek window
Supporting information
No response

hitting left / right should navigate to the next file.  I just tried this with a bunch of video files and PDFs and Peek operated as expected.
From File Explorer, if you select a few of the files and invoke Peek, it will only go through the items selected too
Can you provide more detail on how you're selecting, maybe some repro steps?
/needinfo

@crutkas
What @nikemaul means is switch the preview file in an open Peek window by selecting an other on in the File Explorer window. (Technically something like on FileExplorerNewSelection event change previewed file in Peek.)
I know that we have (multiple) other issues that request the same.


Revocation contracts - Need help with proxy

I added credential revocation smart contracts and tests in Hardhat. The revocation worked when testing the contract directly on the network, but I wasn't successful when using the proxy contract. I need help to fix the problem with the proxy!
@Toktar #54

Hello @vdsilveira ! Thank you, that's great!
Could you fix DCO and use only English in commit messages please?
Also, I hope to see a design document or/and specification for this revocation feature, because it is not Anoncreds Revocation and not a Revocation List. Maybe I lost something, sorry.
And tell more about your proxy (upgrade / role map?) issue. I will be happy to help.


published to npm as swipe-js

@davidchase you need to install jQuery, then use browserify or webpack

@nickleefly yes, i do not wish to use jquery, so i removed it in my situation and did a simple module.exports = Swipe
so i can do this -> var Swipe = require('swipe-js'); var swipe = new Swipe(); sans jquery :smile:


Scenario reports: Dutch and English version should differ less

The Dutch and English version of (both the regional and the main) scenario reports differ a lot.
It seems that most effort has been put in the Dutch version. So, let's align the English version with that one.

Assigning @MartLubben as he made most of the changes to the Dutch version

@DorinevanderVlies I fully agree, although I have a remark.
It could take some time to translate everything. I expect it takes 1-2 days. A google translate alone is difficult, due to al the mark-up and queries.

I couldn't wait and found an easy way to translate it without losing the code. I think I need 0.5 days create a nice English version of the scenario report :).

Nice, let's discuss this in next week's team meeting

If these scenario reports are updated, please also take into account issue #3638.
Additionally, it is recommended to examine if certain hardcoded values need to be updated, for example national or EU targets  (see issue #3794).


Fix method theme

Fixes #45
This PR fixes the methods level rendering to match functions style.

CSS selector was not picking up correct element with targeted method class. (this was one problem but it did not fully match functions theme style)
Adjusts all elements style under method and static method html class to be consistent with functions styling theme

Compared to functions theme rendering:

Follow up on separate PR:

Add a methods.rst example under docs/sphinx-guide/

 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.

@1ucian0 @javabster This PR is ready for review/comments :)

@karlaspuldaro Thanks for your contribution! This looks great! I will review your PR soon.

This looks great to me, thanks so much @karlaspuldaro!
To answer your questions:

yes I think it would be great to include the staticmethod html class with these changes too
a methods.rst example would be helpful in the docs guidelines, but you can add that in a new PR as it is slightly out of scope and not blocking for this work

@javabster Thank you for reviewing! I believe all comments should be addressed, and I created a new issue to track the follow up idea of adding method.rst to docs guide.


Synchronization of Hudi partitioned tables

We are using the batch synchronization tool and can successfully access the tables from Athena in our new region except for some Hudi partitioned tables that we have in our origin region, when we query them from Athena we simply get an empty result, no error whatsoever.
Is there some consideration about partitioned tables that's no included in the readme?

Please try the updates in the partitionfix branch.


Added Chrome 81 & 83

Chrome 83 comes out of beta on May 19. (https://www.chromestatus.com/features/schedule)
Another pull request will be needed when this date comes around

Coverage remained the same at 100.0% when pulling 044e94ddc5ed2164b31e33e042452b767c70c3e5 on benjamindoe:patch-1 into 4899110cd2f87b01e04ced62dbb9dec541031dee on WhichBrowser:master.

Chrome update has been merged.


feat(governance): add Harsh Thakur as new reviewer

Refer: GOVERNANCE.md.
@RealHarshThakur has been actively involved in the development of Openebs project with major contributions to NDM and openebs charts.
Major contributions:

PRs raised, reviewed
Issues

Based on his interest and commitment to contributing to OpenEBS and involvment in the OpenEBS NDM community meetings, we take pride in adding him as a reviewer (#control-plane-maintainers) to the OpenEBS project.
Signed-off-by: Akhil Mohan<EMAIL_ADDRESS>

Awesome! Thank you @RealHarshThakur for your continued contributions!!
+1


ESC doesn't work on its own on posix

Pressing the ESC key on posix systems will not be returned by readkey().
This is expected as it, as the name implies, escapes the next char. It is used to mark longer sequence of multiple keys. readchar can't know the difference if it stems from a single press of ESC or if it is the beginning of a longer sequence

One option would be to implement a timeout on reads. This would require #92 to be merged and enable non-blocking reads, so that a timer could be checked again in parallel to waiting for the next char


Created Radixsort.py

Radix Sort Algorithm
Do the following for each digit i where i varies from the least significant digit to the most significant digit.
Sort input array using counting sort (or any stable sort) according to the i\’th digit.

Kindly give Hacktoberfest tag.
Thanks in advance!!

Please merge this ,


Add more testcase

Codecov Report

Merging #11 (33e58ea) into main (3b6ddeb) will increase coverage by 2.15%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
+ Coverage   79.62%   81.77%   +2.15%
==========================================
  Files           9       11       +2
  Lines         584      609      +25
==========================================
+ Hits          465      498      +33
+ Misses        102       97       -5
+ Partials       17       14       -3

Impacted Files
Coverage Δ

pkg/metrics/server.go
70.58% <100.00%> (ø)

pkg/runner/runner.go
54.12% <100.00%> (+11.78%)
:arrow_up:

pkg/metrics/metrics.go
0.00% <0.00%> (ø)

pkg/proxy/tlsconn.go
89.41% <0.00%> (+10.58%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


BioSample (WIP)

BioSample represents an object currently referred to by readgroups and callsets, namely the sample a given read group or call set refers to. This PR is for fleshing out the functionality and changes proposed in schemas.
https://github.com/ga4gh/schemas/pull/559
The main issues are, what queries are we expected to fulfill and how?

bioSampleId is an attribute of read groups and call sets that needs to point to a biosample record.

How do we properly set the bioSampleId to point to a bioSample record when these values are currently pulled verbatim using pysam?

When returning a read group set or call set protocol element we could perform an internal lookup based on the name and replace the bioSampleId with the result of that call.

The datarepo could load the header/metadata from its files into a database and keep a pointer to the file in the database as opposed to using pysam to handle all queries.

On server start we could iterate through each callset and readgroup and create a biosample record for each unique name.

We need to support the query, "Show me reads with this bioSampleId". I believe this is a change to searchReadsRequest and callsets, unless we expect the metadata service to consume all other services.

I expect this general issue to continue to crop up since we are using pysam as a database. The most sensible solution to me is to use basic CRUD patterns. Adding a new readgroup to the server involves HTTP posting the record details as well as the file's path. This creates a row in the readgroup table, as well as a row in the biosample table, which can later be updated to hold more detailed information.
Requests about a readgroup will return data from a table; only when returning alignments or seeking within the file do we open pysam.
@jeromekelleher for the purposes of this PR I've added the ability to perform CRUD operations on metadata objects. This is not meant to leak back into the schemas. Since these objects do not have a randomly accessible filetype backing them I believe having the ability to use the API to update and edit them gives us internal consistency and better data quality: a poorly formed create request is denied at creation time and there is a "safer" method than hand editing JSON for updating.

How are the sqlite databases organized?  One per server, one per dataset, etc.
One table per metadata?
How do ids works?
What is rationale for creating an internal write API vs just loading the database via the gtrepo command?

One sqlite DB per service, one table per class, and ids are randomly assigned during creation.
The reason for having a write API is testing, data quality, better error handling, etc. Letting users hand edit JSON files is error prone. The server needs to be restarted when new data are added? Does a parsing error keep the server from starting? Can we guarantee persistent identifiers if they are generated by an offline process?
I'll see about implementing some other ingestion mechanism using garepo. Do you think we might be going further in the direction of configuration over convention?

So one data for all datasets?
The question about a write API isn't handed editing, it's going
through http vs directly to the database.
which identifiers are persistent?
Time for some design work rather than code.
David Steinberg<EMAIL_ADDRESS>writes:

One sqlite DB per service, one table per class, and ids are randomly assigned
during creation.
The reason for having a write API is testing, data quality, better error
handling, etc. Letting users hand edit JSON files is error prone. The server
needs to be restarted when new data are added? Does a parsing error keep the
server from starting? Can we guarantee persistent identifiers if they are
generated by an offline process?
I'll see about implementing some other ingestion mechanism using garepo. Do you
think we might be going further in the direction of configuration over
convention?
—
Reply to this email directly or view it on GitHub.*

I haven't gone through this in detail @david4096, but here is some quick feedback on the write API. This is a reference implementation of the GA4GH schemas, and as such we certainly should not be adding in our own extensions to the API. I'm OK with the root URL returning a HTML summary of the data held in the repository as this is a useful user tool, and I doubt anyone would be obtuse enough to think that scraping the HTML for information would be portable across servers. Other than that, we should strictly support the API and no more. So, in general any addition to the internet facing surface of the server will be received negatively.
In any case, I think your arguments are actually satisfied by the repo management tool. Certainly we shouldn't be hand editing any JSON files. The repo administrator should do all of his/her data repository administration through the repo administration CLI that we provide. This is our own software, not under any external API requirements and so we can do what we want with it. So, I think any future updates to  how we handle data will need to include updates to the repo manager. Do this make sense?

If the refserver goal was to have a remotely administered
server, an internal API would be a reasonable approach.
However, that is not currently a goal, so adding HTTP
administration is an unnecessarily layer.
Well structure code in libraries used by commands can evolve to
support a write API when we get to that phase.
We do need to get top-down discover APIs into the schema
specification.
Jerome Kelleher<EMAIL_ADDRESS>writes:

I haven't gone through this in detail @david4096, but here is some quick
feedback on the write API. This is a reference implementation of the GA4GH
schemas, and as such we certainly should not be adding in our own extensions to
the API. I'm OK with the root URL returning a HTML summary of the data held in
the repository as this is a useful user tool, and I doubt anyone would be
obtuse enough to think that scraping the HTML for information would be portable
across servers. Other than that, we should strictly support the API and no
more. So, in general any addition to the internet facing surface of the server
will be received negatively.
In any case, I think your arguments are actually satisfied by the repo
management tool. Certainly we shouldn't be hand editing any JSON files. The
repo administrator should do all of his/her data repository administration
through the repo administration CLI that we provide. This is our own software,
not under any external API requirements and so we can do what we want with it.
So, I think any future updates to how we handle data will need to include
updates to the repo manager. Do this make sense?
—
Reply to this email directly or view it on GitHub.*


original latex source release

Fantastic work, thanks for the release.
Is there any chance you could also release the original latex files? Even just privately. I'm interested in this for a scientific OCR project.
Thanks a lot!

Thanks for your interest. Unfortunately, the parsing results from the mathpix api are not page-specific, i.e., given a pdf, a parsed doc like markdown will be obtained, which contains the whole content of the pdf, not separated by the page.

Understood! Would it be possible to release them still? And what about the original LaTeX files? Thanks a lot!

Hi, you can download it from the following sharing link. Please DO NOT use it for commercial purposes, just for research. And, feel free to cite our MathPile paper!  :)
https://drive.google.com/file/d/1m1UIhpXT3_9LR61BN3JB0xyUJZqVzumR/view?usp=sharing

Thank you!
On Fri, 5 Jul 2024, 10:07 Zengzhi Wang, @.***> wrote:

Hi, you can download it from the following sharing link. Please DO NOT use
it for commercial purposes, just for research. And, feel free to cite our MathPile
paper https://arxiv.org/abs/2312.17120! :)
https://drive.google.com/file/d/1m1UIhpXT3_9LR61BN3JB0xyUJZqVzumR/view?usp=sharing
—
Reply to this email directly, view it on GitHub
https://github.com/GAIR-NLP/MathPile/issues/6#issuecomment-2210402551,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJBFWJXZGZGPQ6D4DMEG5DZKZH5TAVCNFSM6AAAAABKH2QZS2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJQGQYDENJVGE
.
You are receiving this because you authored the thread.Message ID:
@.***>


add custom theme path to intializer config

it allows you to set a new path to look for themes instead of being hard coded. there is a example of how to set up up in the readme.

@TopRoupi Thanks for your PR. I think we can add new api like the below to make this easy.
self.register_theme :name, assets_path: Rails.root.join("app", "rpictogrify", "themes")

@TopRoupi It bumps to 0.5.0 🎉 We can register a new theme like below
Rpictogrify.configure do
  self.register_theme :custom, assets_path: Rails.root.join("app", "rpictogrify", "themes", "custom")
end

thx


Use the build system to build artifacts for distribution

Instead of using a collection of Python scripts to generate the artifacts we
want to distribute from this repository, we should just use the build system.
This CL adds a top-level |dist| target that creates these artifacts in a "dist"
directory of the build output.

lgtm


Making the Navbar on Audio Page responsive

The Navbar on audio Page is not responsive.

can I take up this Issue ?

@SahilDahekar Go ahead

yep

I have taken care of this issue in issue #116. Please see once.

@ShivanshAr97 Your PR got merged but it still got issues and also i have added additional feature of toogling the hamburger svg with cross svg.


[Feature][JsonSplit] snowFlakeUtils

add snow flake. we can use it like SnowFlakeUtils.getInstance().nextId() .But you should need to handle the exception

Codecov Report

:exclamation: No coverage uploaded for pull request base (json_split@d67b335). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             json_split    #4432   +/-   ##
=============================================
  Coverage              ?   43.41%
  Complexity            ?     3203
=============================================
  Files                 ?      472
  Lines                 ?    21842
  Branches              ?     2652
=============================================
  Hits                  ?     9483
  Misses                ?    11451
  Partials              ?      908

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 d67b335...2bba55c. Read the comment docs.

Codecov Report

:exclamation: No coverage uploaded for pull request base (json_split@d67b335). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             json_split    #4432   +/-   ##
=============================================
  Coverage              ?   43.41%
  Complexity            ?     3203
=============================================
  Files                 ?      472
  Lines                 ?    21842
  Branches              ?     2652
=============================================
  Hits                  ?     9483
  Misses                ?    11451
  Partials              ?      908

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 d67b335...2bba55c. Read the comment docs.


Make rand_distr optional

rand_distr is only used in dataset module
https://github.com/smartcorelib/smartcore/blob/162bed2aa2242e35e02c875821d0d7abaf054070/src/dataset/generator.rs#L5

Codecov Report

Merging #161 (4ed7ddc) into development (436da10) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           development     #161      +/-   ##
===============================================
- Coverage        84.25%   84.18%   -0.07%
===============================================
  Files               85       85
  Lines             9196     9196
===============================================
- Hits              7748     7742       -6
- Misses            1448     1454       +6

Impacted Files
Coverage Δ

src/optimization/line_search.rs
82.00% <0.00%> (-8.00%)
:arrow_down:

src/optimization/first_order/lbfgs.rs
92.85% <0.00%> (-1.59%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


guide: Running Experiments

We are removing some parts of the experiments trail that may be better put into a UG document.
Related to #2548
A guide dedicated to running experiments should contain:

Running experiments
Running experiments in parallel
Managing experiments queue
Naming experiments

Please add a checkbox in epic #2548.


Make Prime Minister dynamic on "How government works" page

This is very much a first stab at this. Be gentle, I don't write code anymore. If it looks vaguely ok I'll write a test for it and get the other tests passing.

I think this looks fine 👍

@gpeng: Thanks, I think I'm almost there. The final(ish) thing that's confusing me is that the "Unpublishing a document and redirecting" feature is creating a FactoryGirl publication which is somehow going through the HomeController#how_government_works method.

It's coincidentally using that page in the feature here https://github.com/alphagov/whitehall/blob/master/features/step_definitions/unpublishing_published_documents_steps.rb#L90 but the there's no MinisterialRole being setup within the feature so you're getting the no method on nil thing. You could probably just choose another page for that step.

@gpeng: Ooh thanks! Tests are passing now so this is good for somebody to have a look at.

👍 looks good to me (again)


Web Api validation with custom validators using Unity DI not using custom validator - bug or feature?

Hi there
I think I found a bug.
I have some model
[Validator(typeof(AccompanyingDocumentModelValidator))]
public class AccompanyingDocumentModel
{
and the Validator
public class AccompanyingDocumentModelValidator : AbstractValidator
{
registered with Unity in OWIN Startup
FluentValidationModelValidatorProvider
.Configure(config, provider => provider.ValidatorFactory = container.Resolve());
The VAlidatorFactory:
public class UnityValidatorFactory : IValidatorFactory
{
private readonly IUnityContainer _container;
	public UnityValidatorFactory()
	{
		_container = null;
	}

	public UnityValidatorFactory(IUnityContainer container)
	{
		_container = container;
	}

	public IValidator CreateInstance(Type validatorType)
	{
		return _container.Resolve(validatorType) as IValidator;
	}

	public IValidator<T> GetValidator<T>()
	{
		return (IValidator<T>)this.GetValidator(typeof(T));
	}

	/// <summary>
	/// Gets a validator for the appropriate type.
	/// </summary>
	public IValidator GetValidator(Type type)
	{
		if (type == null)
			return null;

		var attribute = (ValidatorAttribute)Attribute.GetCustomAttribute(type, typeof(ValidatorAttribute));

		if (attribute == null || attribute.ValidatorType == null)
			return this.CreateInstance(typeof(IValidator<>).MakeGenericType(type));

		return this.CreateInstance(attribute.ValidatorType) as IValidator;
	}

}

and UnitiyContainer filled:
p_Container.RegisterType<IValidator, AccompanyingDocumentModelValidator>(new HierarchicalLifetimeManager());
p_Container.RegisterType IValidatorFactory,
UnityValidatorFactory>(new HierarchicalLifetimeManager());
This results in FluentValidationBodyModelValidator calls
private bool ValidateNodeAndChildren(ModelMetadata metadata, ValidationContext validationContext, object container)
{
object model = metadata.Model;
bool isValid = true;
		// Optimization: we don't need to recursively traverse the graph for null and primitive types
		if (model == null || IsSimpleType(model.GetType()))
		{
			return ShallowValidate(metadata, validationContext, container);
		}

		// Check to avoid infinite recursion. This can happen with cycles in an object graph.
		if (validationContext.Visited.Contains(model))
		{
			return true;
		}
		validationContext.Visited.Add(model);

		// Validate the children first - depth-first traversal
		IEnumerable enumerableModel = model as IEnumerable;
		if (enumerableModel == null)
		{
			isValid = this.ValidateProperties(metadata, validationContext);
		}
		else
		{
			isValid = this.ValidateElements(enumerableModel, validationContext);
		}

		// Validate this node as well
		isValid = ShallowValidate(metadata, validationContext, container) && isValid;

		// Pop the object so that it can be validated again in a different path
		validationContext.Visited.Remove(model);

		return isValid;
	}

The bug is that it does not use the custom  validator but some standard ones( IValidator) for every single Property of the APiControllers Method parameter.
So I changed it to the following checking for metadata's model Custom validation attribute. if present, it goes straight to the validation of the VAlidator decorated model using the assigned custom validator.
private bool ValidateNodeAndChildren(ModelMetadata metadata, ValidationContext validationContext, object container)
{
object model = metadata.Model;
bool isValid = true;
		// Optimization: we don't need to recursively traverse the graph for null and primitive types
		if (model == null || IsSimpleType(model.GetType()))
		{
			return ShallowValidate(metadata, validationContext, container);
		}

		// Check to avoid infinite recursion. This can happen with cycles in an object graph.
		if (validationContext.Visited.Contains(model))
		{
			return true;
		}
		validationContext.Visited.Add(model);

		if (metadata.ModelType.GetCustomAttribute(typeof(ValidatorAttribute), false) != null)
		{
			isValid = ShallowValidate(metadata, validationContext, container) && isValid;
		}
		else
		{
			// Validate the children first - depth-first traversal
			IEnumerable enumerableModel = model as IEnumerable;
			if (enumerableModel == null)
			{
				isValid = this.ValidateProperties(metadata, validationContext);
			}
			else
			{
				isValid = this.ValidateElements(enumerableModel, validationContext);
			}

			// Validate this node as well
			isValid = ShallowValidate(metadata, validationContext, container) && isValid;
		}

		// Pop the object so that it can be validated again in a different path
		validationContext.Visited.Remove(model);

		return isValid;
	}

Did I miss something? or is this indeed a bug?

I'm a bit confused with what you're trying to do.
The ValidatorAttribute is for use with the AttributedValidatorFactory. When you ask it to create a validator, it will check to see if that attribute is present on the type and create the corresponding validator.
If you're using an IoC container, then the ValidatorAttribute is redundant. The entire point of using the container is that it is responsible for assocating validators with their type - you shouldn't be using the attribute for this.
Basically, if you want to associate a specific validator with a type, you should be registering this with your container (I don't use unity, so I can't help with the exact code for this), not using the ValidatorAttribute.

I registerd the validator like
p_Container.RegisterType IValidator, AccompanyingDocumentModelValidator (new HierarchicalLifetimeManager());
Then
public class AccompanyingDocumentModel : IValidatableObject
{
...
public AccompanyingDocumentModel(IValidator p_Validator)
{
this.m_Validator = p_Validator;
}
	public IEnumerable<ValidationResult> Validate(ValidationContext p_ValidationContext)
	{
		return this.m_Validator.Validate(this).ToValidationResult();
	}

But at the time the constructor got called the validator was not yet resolved, being null.
What did I wrong? How did you registering the validator with your IOC Framework? Even if it is not unity I could "translate" it to unity.

And how does the FluentValidationBodyModelValidator Validate Method know that there is validator registered for that particular type not digging deeper using ValidateNodeAndChildren?

It calls into the ModelValidatorProvider, which in turn uses the ValidatorFactory that you specify in your call to FluentValidationModelValidatorProvider.Configure to ask for the correct validator.
With StructureMap I'd have a registry that calls For<IValidator<MyType>>().Use<MyValidator>()

I think i did the same with unity
 p_Container.RegisterType<IValidatorFactory, UnityValidatorFactory>(new HierarchicalLifetimeManager()); 			p_Container.RegisterType<IValidator<AccompanyingDocumentModel>, AccompanyingDocumentModelValidator>(new HierarchicalLifetimeManager());

OK, so if you trace through the code, is your UnityValidatorFactory's GetValidator method being invoked?

yes but only with string as requested type

OK that doesn't sound right - it should be called with your model type. What does your controller action look like?

[ValidationResponseFilter]		 		public async Task<IHttpActionResult> Post(AccompanyingDocumentModel p_AccompanyingDocumentModel)

Hard to tell what the problem could be. If you could put together a (small) sample project that reproduces the issue then I can look into it further.

How do you register ein WEb Api OWIN? Maybe that is wrong?
 FluentValidationModelValidatorProvider 				.Configure(config, provider => provider.ValidatorFactory = container.Resolve<IValidatorFactory>());

I don't think that's the problem. If that was the issue then your UnityValidatorFactory's GetValidator method wouldn't be called at all. If you could put together that sample, then I'll take a look at it.

But why does it use the FluentValidationBodyModelValidator in any case? This one never cares about custom validators.
`public static void Configure(HttpConfiguration configuration, Action configurationExpression = null) {
configurationExpression = configurationExpression ?? delegate { };
		var provider = new FluentValidationModelValidatorProvider();
		configurationExpression(provider);
	    configuration.Services.Replace(typeof(IBodyModelValidator), new FluentValidationBodyModelValidator());
		configuration.Services.Add(typeof(ModelValidatorProvider), provider);
	}`

But why does it use the FluentValidationBodyModelValidator in any case

It's to work around a bug in WebApi to do with generating error messages - it's not relevant to your issue.  The default BodyModelValidator doesn't play nicely with FluentValidation. The FluentValidationBodyModelValidator has a special case for running FluentValidation's validators that make's sure that property names are built correctly.

This one never cares about custom validators.
Instantiation of the validators isn't handled by the BodyModelValidator - it's handled by the provider.

Is there any Code Example with WebApi, OWin, custom validator and some IOC Container?

No, sorry

Cleanup: Closing this as it hasn't been active in some time. Feel free to re-open if necessary.


compile -> implementation in gradle info

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the contributing guidelines.

Changes

[ ] Internal code
[ ] Library interface (affecting end-user code)
[x] Documentation
[ ] Other: _____

Closes Issue: NaN
Description
Replaced compile with implementation in gradle info of Readme.md

Please rebase your branch to be inline with develop

Actually... let me think about it... since this is only in the readme and minn apparently only does those changes to master

For consistency, changing the compile ('net.dv8tion:JDA:VERSION') { a few lines down as well would make sense.

Closing due to inactivity.


How to receive webhooks from a public channel?

I successfully created a bot, that receives webhooks, if I chat with the bot directly.
I also want to receive webhooks and handle webhooks from other channels. For example a public channel.

I added the boot as an Admin to the channel.
Set allow message from unknown chats to true and added the chat id to the database.
I set the privacy to disabled.

Did I miss something?

Receiving Webhooks from a Group works fine.


Fix the UI of Search result page.

what exaclty do want to be fixed in search page?

The image of books was good but somehow they are not right fir you see the production. It should be like the readme screenshots.


Issue compiling embedded-svc on rustc 1.75.0-nightly (fee5518cd 2023-11-05)

Hi there,
I recently updated to the most recent version of rust nightly, and was presented with this error when compiling embedded-svc:
   Compiling nb v1.1.0
   Compiling byteorder v1.5.0
   Compiling stable_deref_trait v1.2.0
   Compiling enumset v1.1.3
   Compiling const_format v0.2.32
   Compiling libc v0.2.149
   Compiling atomic-waker v1.1.2
   Compiling void v1.0.2
   Compiling nb v0.1.3
   Compiling embedded-hal v1.0.0-alpha.10
   Compiling anyhow v1.0.75
   Compiling embedded-hal v0.2.7
   Compiling embedded-hal-nb v1.0.0-alpha.2
   Compiling embedded-can v0.4.1
   Compiling hash32 v0.2.1
   Compiling esp-idf-hal v0.41.2
   Compiling serde v1.0.189
   Compiling no-std-net v0.5.0
   Compiling embedded-io v0.4.0
   Compiling esp-idf-sys v0.33.2
   Compiling esp-idf-svc v0.46.2
   Compiling uncased v0.9.9
   Compiling log v0.4.20
   Compiling heapless v0.7.16
   Compiling bytemuck v1.14.0
   Compiling embassy-futures v0.1.0
   Compiling rgb v0.8.36
   Compiling trainer v0.1.0 (/home/jesse/dev/rust/experimental-embed-esp32/trainer)
   Compiling embedded-svc v0.25.3
error[E0277]: `(dyn FnOnce() -> T + Send + 'static)` cannot be sent between threads safely
   --> /home/jesse/.cargo/registry/src/index.crates.io-6f17d22bba15001f/embedded-svc-0.25.3/src/utils/asyncify.rs:72:11
    |
72  |         = BlockingFuture<T> where T: Send;
    |           ^^^^^^^^^^^^^^^^^ `(dyn FnOnce() -> T + Send + 'static)` cannot be sent between threads safely
    |
    = help: the trait `Send` is not implemented for `(dyn FnOnce() -> T + Send + 'static)`
    = note: required for `Unique<(dyn FnOnce() -> T + Send + 'static)>` to implement `Send`
note: required because it appears within the type `Box<dyn FnOnce() -> T + Send>`
   --> /home/jesse/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/alloc/src/boxed.rs:195:12
    |
195 | pub struct Box<
    |            ^^^
note: required because it appears within the type `Option<Box<dyn FnOnce() -> T + Send>>`
   --> /home/jesse/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core/src/option.rs:569:10
    |
569 | pub enum Option<T> {
    |          ^^^^^^
note: required because it appears within the type `BlockingFuture<T>`
   --> /home/jesse/.cargo/registry/src/index.crates.io-6f17d22bba15001f/embedded-svc-0.25.3/src/utils/asyncify.rs:87:16
    |
87  |     pub struct BlockingFuture<T> {
    |                ^^^^^^^^^^^^^^
note: required by a bound in `unblocker::Unblocker::UnblockFuture`
   --> /home/jesse/.cargo/registry/src/index.crates.io-6f17d22bba15001f/embedded-svc-0.25.3/src/executor.rs:95:57
    |
95  |             type UnblockFuture<T>: Future<Output = T> + Send
    |                                                         ^^^^ required by this bound in `Unblocker::UnblockFuture`

For more information about this error, try `rustc --explain E0277`.
error: could not compile `embedded-svc` (lib) due to previous error
warning: build failed, waiting for other jobs to finish...

Reverting to nightly-2023-10-15 resolves my issue, so it's not a deal breaker but wanted to report for your attention.
Let me know if you need any more information from my end!
Reporting
Just reporting this

Fixed yesterday.


Remove setLeagues

Remove setLeagues method usage
This PR removes the setLeagues method that was being used on the SoccerAPIClient objects. After inspecting the API docs I didn't find any trace of this method, so it seems it was deprecated. I also didn't find a method that could be used as replacement. So for now I think it's best to remove the use of this method.
Please tell us , the type of Change you are submiting:
Select one of the following:

[ X ] Bug
[ ] Feature
[ ] Minor Change

Call to undefined method Sportmonks\SoccerAPI\Requests\LiveScore::setLeagues()

Does it fix an existing issue? Please tell us which one
Fixes issue #25
Description of what's been changed
Use of the method setLeagues was removed from the SoccerAPIController

Closed because you didn't used the most recent code and you removed code that is used when you visit a league-specific page (like /leagues/208)


Use text-table to align console output

Use text-table to align console output

Delay

text-table has work not well with Unicode characters, see issue

You need to pass a string length function that would take wide characters into account.
var table = require('text-table'),
    wcwidth = require('wcwidth');
var t = table([
    [ '中国好声音', '1024000' ],
    [ '舞蹈', '33450' ],
    [ 'foo', '1006' ],
    [ '中文对齐有点儿问题哦', '45.08' ]
], {
  align: [ 'l', 'r' ],
  stringLength: wcwidth
});

console.log(t);

This one will be better: https://github.com/timoxley/columnify


Develop

Codecov Report

Merging #43 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #43   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          11     11
  Lines         544    544
  Branches       19     19
=====================================
  Hits          544    544

Impacted Files
Coverage Δ

src/puml-writer.spec.ts
100% <ø> (ø)
:arrow_up:

src/interface.ts
100% <ø> (ø)
:arrow_up:

src/puml-writer.ts
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


ci: Test lint PR title

For consistency, we should start the PR title's subject (the part after the feat:/fix: etc prefix) with a lowercase letter
See https://github.com/amannn/action-semantic-pull-request for context on the settings

While maintaining consistency, it is advisable to focus more on the overall solution and propose a comprehensive standard for document editing rather than modifying a specific currently accessible link. This is more efficient.

While maintaining consistency, it is advisable to focus more on the overall solution and propose a comprehensive standard for document editing rather than modifying a specific currently accessible link. This is more efficient.

I'm not sure I can follow, can you elaborate?

在保持一致性的同时，建议更多地关注整体解决方案，并提出文档编辑的全面标准，而不是修改当前可访问的特定链接。这样效率更高。

我不确定我能听懂，你能详细说明一下吗？

While maintaining consistency, it is advisable to focus more on the overall solution and propose a comprehensive standard for document editing rather than modifying a specific currently accessible link. This is more efficient.

I'm not sure I can follow, can you elaborate?

Apologies, English is not my native language, but I am trying to make you understand my point: the issue of case sensitivity in links is not an exception in this repository. After the document was updated yesterday, I found some incorrect hyperlinks (and I am preparing to submit them). Perhaps the official developers should introduce more detailed contribution guidelines and inspection methods to reduce errors? Or maybe perform a thorough check of the page before submitting?

While maintaining consistency, it is advisable to focus more on the overall solution and propose a comprehensive standard for document editing rather than modifying a specific currently accessible link. This is more efficient.

I'm not sure I can follow, can you elaborate?

Apologies, English is not my native language, but I am trying to make you understand my point: the issue of case sensitivity in links is not an exception in this repository. After the document was updated yesterday, I found some incorrect hyperlinks (and I am preparing to submit them). Perhaps the official developers should introduce more detailed contribution guidelines and inspection methods to reduce errors? Or maybe perform a thorough check of the page before submitting?

I think there is a misunderstanding - this PR has nothing to do with your contributions. Definitely agree we should check the correctness of links before merging PRs. This PR adds a simple lint check for the title of PRs (to lowercase the first word of the subject)

I think there is a misunderstanding - this PR has nothing to do with your contributions. Definitely agree we should check the correctness of links before merging PRs. This PR adds a simple lint check for the title of PRs (to lowercase the first word of the subject)

I suppose it was a misunderstanding on my part, thank you for clarifying.


Dropped request handling

There needs to be a mechanism to handle dropped requests/error response codes. Would it be better to implement it through a callback in the settings, or through a pipeline?

To clarify, when I say handle, I mean "to do something with the request, like update a database)

I suggest you moving onto Request / Response Middlewares.
This way we can integrate current middleware type as Request Middleware and provide a way to basically tag, filter or do something else upon getting new response.

Hmmm. This would mean a middleware piping step prior to piping the success response through the item pipeline.
Let's call this hypothetical step Response Middlewares.
Should the sole responsibility of the Response Middlewares be to do something with the responses, successful or not? This could be the step where we check for retries, and tag responses, or basically control the flow of the response (whether to be dropped, logged, piped etc).
Currently, as is, only code 200 responses are passed to the item pipelines

I think, that the more flexibility you have – the better.
The response middleware list can contain only one middleware by default, like AllowStatusCodes with the tuple of 200, 201 as argument. This way it'll provide both sensible defaults and flexible way to inject some additional code, based on what actions you actually want to happen.
Because when the status code IS the information you'd like to gather, you'll need some way to override the defaults in a non-hacky way.


Add support for images, or at least fail gracefully

Currently, the preview fails with no error messages if there are any images embedded in the document.  It would be nice if it could output a warning instead.

bump
Is there an update on this?
If I insert an image in a markdown file and then try to Live Preview it, Sublime Text will crash then the window or program will close.

Another bump.
Embedding images from the web seems to work (it will display a loading icon and then the image, once it is loaded).
But referencing local images (i.e. relative paths such as ./image.jpg) still crashes the preview. It will stop updating on ANY changes and only work again once the relative link is removed.


[WIP] Add codegen for function type and function pointer load

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Fixes #
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, releated PRs, TODO items or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #15161 into master will increase coverage by 0.18%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master   #15161      +/-   ##
============================================
+ Coverage     22.99%   23.17%   +0.18%
+ Complexity     4970     4968       -2
============================================
  Files          1340     1302      -38
  Lines         66792    65673    -1119
  Branches       9064     8874     -190
============================================
- Hits          15357    15221     -136
+ Misses        49741    48773     -968
+ Partials       1694     1679      -15

Impacted Files
Coverage Δ
Complexity Δ

...allerinalang/compiler/bir/model/BIRTerminator.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...erinalang/compiler/bir/model/BIRNonTerminator.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...lerinalang/compiler/bir/model/InstructionKind.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...va/org/wso2/ballerinalang/compiler/bir/BIRGen.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...lang/compiler/bir/writer/BIRInstructionWriter.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...2/ballerinalang/compiler/bir/model/BIRVisitor.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...lang/langserver/formatting/FormattingNodeTree.java
72.67% <0%> (-0.01%)
665% <0%> (-2%)

...erinalang/compiler/parser/BLangParserListener.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

tool-plugins/vscode/src/utils/logger.ts

tool-plugins/vscode/src/test-runner/runner.ts

... and 36 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 29b44ce...6267134. Read the comment docs.