Supporting showing UTF-8 string as a payload

Now it only show the hex string

Could you share the link to the above screenshot?
Can't get action with payload.

This issue is a bit stale but it refers to action page like https://www.iotexscan.io/action/000086e6e3034c104025896f8af9d3e313b95957c076b3a5ebe1f42f8af3a8ad

I am still not sure about this.
Should I decode the payload as UTF-8 string?

An example: https://github.com/decred/dcrdata/issues/635


I made a custom .netkan file

Please update the CKAN listing if this works

Hey, it looks like instead of a .netkan file you pasted an inflated .ckan here, with a lot of hard coded values like mod version, download URL and hash etc. that should be auto-generated by netkan.exe itself.
The primary changes you want to make seem to be with the relationships and tags. Can you re-apply those changes to the raw netkan instead?
Otherwise, if you want, you can tell us what exactly you'd like to have changed, and we can help you out.

Hey, it looks like instead of a .netkan file you pasted an inflated .ckan here, with a lot of hard coded values like mod version, download URL and hash etc. that should be auto-generated by netkan.exe itself.
The primary changes you want to make seem to be with the relationships and tags. Can you re-apply those changes to the raw netkan instead?
Otherwise, if you want, you can tell us what exactly you'd like to have changed, and we can help you out.

re-apply those changes to the raw netkan instead?

Took care of this in the latest commit. Now the problem is the internal .ckan file that you mentioned on Discord.

re-apply those changes to the raw netkan instead?

Took care of this in the latest commit. Now the problem is the internal .ckan file that you mentioned on Discord.

So why didn't it give me the error when I inflated it here? I thought I was supposed to post the inflated version to you guys. I reread the instructions multiple times, but it was about 4:30am when I did that.

So why didn't it give me the error when I inflated it here? I thought I was supposed to post the inflated version to you guys. I reread the instructions multiple times, but it was about 4:30am when I did that.

So why didn't it give me the error when I inflated it here?

The first version of this pull request hard-coded the compatibility (and everything else) in the netkan, which bypasses the logic in question, but you really don't want that.

I thought I was supposed to post the inflated version to you guys.

No, the inflated .ckan file gets generated automatically by the bot and saved into the CKAN-meta repo. The NetKAN repo is for the pre-inflated .netkan file.

I reread the instructions multiple times, but it was about 4:30am when I did that.

If you can tell us where you thought you read that, we can try to make it clearer.

So why didn't it give me the error when I inflated it here?

The first version of this pull request hard-coded the compatibility (and everything else) in the netkan, which bypasses the logic in question, but you really don't want that.

I thought I was supposed to post the inflated version to you guys.

No, the inflated .ckan file gets generated automatically by the bot and saved into the CKAN-meta repo. The NetKAN repo is for the pre-inflated .netkan file.

I reread the instructions multiple times, but it was about 4:30am when I did that.

If you can tell us where you thought you read that, we can try to make it clearer.

Looks like the download was replaced and no longer contains a .ckan file. :+1:

Looks like the download was replaced and no longer contains a .ckan file. :+1:


New EID and RH signature type

Introduce a new signature type for allowing commitment to both an enrollment ID and a revocation handle
#21

@ale-linux , please, can you review this?


Improve CLI startup time

Program startup time was really slow. After inspecting it with -Ximporttime it turned out to be pulling in a lot of unnecessary packages from pkg_resources responsible only for debug functionality.
Solution: reduce the scope of the debug import.
Note that this type of optimization is more of a hack than a solution, but considering it significantly improves startup time for the 99%+ of use cases it is worth implementing.
Before the patch:
$ time python3.10 -m rivalcfg --help > /dev/null

real	0m0,156s
user	0m0,125s
sys	0m0,023s

$ time python3.10 -m rivalcfg -s 800

real	0m0,260s
user	0m0,129s
sys	0m0,021s

After the patch:
$ time python3.10 -m rivalcfg --help > /dev/null

real	0m0,069s
user	0m0,050s
sys	0m0,011s

$ time python3.10 -m rivalcfg -s 800

real	0m0,185s
user	0m0,060s
sys	0m0,015s

Hello,
Thank you for the PR! Please fix the codding style (missing blank line after import) to allow me to merge :)

Thank you, this will be released with the next version :)


Allow support for process aliases in config.toml file

Allow the user to use aliases for processes instead of having to type out lengthy process names:
aliases = """
network_inference.direct.flashweave as FW
network_inference.network.make_network as MN
"""
order = """
FW
MN
"""

OR
[network_inference.direct.flashweave]
alias = "FW"

[network_inference.network.make_network]
alias = "MN"

This issue is related to #197

No longer applicable


add missing axes params (plot_psd_topomap and related)

@jaeilepp added the missing axes args

[Current coverage][cc-pull] is 86.66%
[][cc-pull]

No coverage report found for master at 0324f00.

Powered by Codecov. Last updated by [0324f00...9c5cc79][cc-compare]
[cc-base-branch]: https://codecov.io/gh/mne-tools/mne-python/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/mne-tools/mne-python/compare/0324f0009d408a8c7d6b2c50cd035369112904d3...9c5cc79206150d6ff42333b244fd40e65ae9f9ad?src=pr
[cc-pull]: https://codecov.io/gh/mne-tools/mne-python/pull/3404?src=pr

Thanks @mmagnuski


RM Bridge web interface Code Learning page

using mini3 device: When I manually add MAC address, device name and RM2 in the RM Bridge web interface Code Learning page I get the "ERROR: HTTP/404"  message.  Do you have any suggestions?

That website has been giving me so many problems lately, try with the IP or try to debug the Web Console and see if there is an error there

Not sure how to get the code information with the IP address.  Thanks.
Michael Flamm
Sent from my iPhone

On Nov 28, 2017, at 10:48 PM, Jose Vera<EMAIL_ADDRESS>wrote:
That website has been giving me so many problems lately, try with the IP or try to debug the Web Console and see if there is an error there
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Did you get this working? I'm closing for now this bug


Implement Partial Update of User Profile Settings

Description
This pull request implements a RESTful API endpoint to handle partial updates of user profile settings. The endpoint allows authenticated users to modify specific parts of their profile information, such as name, email, or preferences. User preferences are stored as a JSON field in the user table of the database.
Key Features

API Endpoint Implementation:

[ ] The endpoint is accessible at /api/v1/users/.

[ ] Accepts HTTP PATCH requests.

Data Validation and Authorization:

[ ] Ensures the user making the request is authenticated.

[ ] Validates incoming data for correctness and security.

Updating User Profile:

[ ] Updates the user's profile information in the database upon successful validation.

[ ] Ensures critical fields (e.g., email) trigger appropriate verification processes if changed.

Response:

[ ] On success, returns a 200 OK status code with the updated profile information.

[ ] On failure, returns appropriate error codes with descriptive messages.

Related Issue
Link to issue ticket
Motivation and Context
The purpose of this implementation is to provide a secure and efficient backend service for users to update specific parts of their profile settings. The system ensures data validation and maintains user information integrity.
How Has This Been Tested?

[ ] Manual Testing: Verified the functionality using Postman by sending PATCH requests with different payloads.

[ ] Unit Tests: Added unit tests to check validation, authentication, and successful updates.

[ ] Integration Tests: Tested integration with other parts of the system to ensure seamless updates.

Screenshots (if appropriate):

Types of changes

[ ]  New feature (non-breaking change which adds functionality)

Checklist:

[ ] My code follows the code style of this project.
[ ]  My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ]  I have read the CONTRIBUTING document.
[ ]  I have added tests to cover my changes.
[ ] All new and existing tests passed.

Code Implementation:
python
Copy code
@settings.patch("/users", status_code=status.HTTP_200_OK, response_model=success_response)
async def update_profile_settings(
settings: UserAndProfileUpdate,
db: Session = Depends(get_db),
current_user: User = Depends(user_service.get_current_user)
):
"""
Update the authenticated user's profile. This endpoint allows partial updates.
"""
db_user = update_user_and_profile(db=db, user_id=current_user["id"], user_profile=settings)
if db_user is None:
    raise HTTPException(status_code=404, detail="User not found")
return success_response(
    status_code=200,
    message="Update created successfully",
    data=jsonable_encoder(db_user)
)

Merge conflict

Do this. All these profile settings files, all of the code should be in profile.py file for each of these folders. THis is for consistency sake

Done

Merge conflict

Merge conflict

Done please review

i need second review

Stale


rules failed: Central Sync Requirements Signature Validation

No public key: Key with id: (4f197542e4123f67) was not able to be located on http://keyserver.ubuntu.com:11371/. Upload your public key and try the operation again.
No public key: Key with id: (4f197542e4123f67) was not able to be located on https://keys.openpgp.org/. Upload your public key and try the operation again.
No public key: Key with id: (4f197542e4123f67) was not able to be located on http://pgp.mit.edu:11371/. Upload your public key and try the operation again.

done externally


doc typo

I think there is a documentation error for the scaleLabel:
The grid line configuration is nested under the scale configuration in the scaleLabel key.
grid line shoud be replaced with scale label.
Thanks for this awesome library!

thanks for reporting this @delcon

Fixed in #3360


Save failing examples indexes while running tests to avoid re-executing them

Description
Save failing examples indexes when the test has failed so that we don't have to re-execute the whole test with debug=True when instancing a debugging session
Type of Change

[ ] 📚 Examples / docs / tutorials / dependencies update
[ ] 🔧 Bug fix (non-breaking change which fixes an issue)
[x] 🥂 Improvement (non-breaking change which improves an existing feature)
[ ] 🚀 New feature (non-breaking change which adds functionality)
[ ] 💥 Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🔐 Security fix

It's also missing a handler of sub dataset creation.

1f85060

Done in 1f85060

It's also missing a handler of sub dataset creation.

Done in 1f85060


[backport] New style bilinear

Backport of #3917, will be rebased after #4021 merged.

Will merge after rebase and test.

Could you rebase the backport since #4021 has been is merged?

Merged the latest v3!

jenkins, test this please.

LGTM!


Rename Symbol Creates Conflicting File

VSCode Version: 1.18.1
OS Version: OS X 10.13.1

Was refactoring some code for a company I contract for. I'm not sure how I got into this state, but this issue happened twice today (VSCode was restarted between occurrences). This bug ended up causing data loss. I worked in the rogue file by accident and lost a bunch of work by mindlessly clicking the checkmark after being asked to compare conflicting files.
Video Demonstration
Steps to Reproduce:

Right click next to a function parameter in a .ts file
Select "Rename Symbol"
Notice the editor opens a new tab with a new file. Notice the new file has the same name as the old file except the first letter is capitalized. Notice that the symbol is not renamed in the original file, but it is renamed in this weird new file.

This seems very strange, but it has happened twice today and once resulted in data loss. It felt worth reporting.
Thanks,
Bobby

Additional observations:

Any variable in this file will trigger the issue, not just function parameters
The rogue file does not appear on disk prior to it being created by VSCode

The symptoms sound like https://github.com/Microsoft/TypeScript/issues/19329 to me. The new file's case is being changed here, correct?

@mjbvz yes, this looks like the same issue. can you please specify what additional info you need? i'd be happy to get whatever information you need as this has been a huge productivity drain over the last few days. thanks!

To check this:

Add "typescript.tsserver.log": "verbose" to your user settings
Close all editors in your workspace then restart vs code.
Now reproduce the problem
Open the output panel and go to the TypeScript section

In this log, look for the bad editor's filepath. See if there are any casing differences between this and the original path

A new file is being created, but it's being created by VSCode (or TS?). It's happening to me all the time and is starting to take quite a lot of time away from work.
I sometimes get it when I use "F12" to go to definition as well.

@Krossmaskinen The problem is that TS returns a file with the incorrect case to VS Code which causes us to create it on the disk. Can you please quickly go through the steps in my comment to confirm that this is indeed a duplicate of Microsoft/TypeScript#19329

@mjbvz it looks like it indeed. The difference is that in my case it's happening on files that never changed names. Still looks like a TS-problem though.

@Krossmaskinen Are you able to share the relevant log sections? Since there is no rename in this case, I think it may be a different TS issue


feat: remove Rafiki submodule

Context

fixes #264

Changes
Remove Git submodules and only use the Docker images to run Rafiki.
Other changes:

Renamed the current applications to wallet-*
Updated the workflows/labels to match the new names
Small cleanup
Updated Dockerfile - we are now using a slim image to reduce it's size.

Please let me know if I missed something.


New aka.ms link

Aka.ms link name
425show
Category (Optional. Select None to auto-select)
None

Thank you for submitting aka.ms/425show. Your link already exists at akaSearch.net. 🙏✅


suggestion: Add a switch interface for the hostname validation in the base-package

we know that the safety inspection mechanism in GO will check the hostname of certificate. If the security certificate does not configure DNS or IP, it will be unable to achieve the function of secure communication.
Here is the question:
Can we construct a switch API to select whether to verify the hostname of certificate, rather than verifying it all the time?
If I have a certificate that does not contain DNS and IP but I want to do secure communication, then I can enable the switch API and filter out the hostname validation.
I think it will be easier for developers to operate and more flexible to use. What's your opinion?

For questions about Go, see https://golang.org/wiki/Questions
I think what you're looking for is tls.Config.InsecureSkipVerify.

Thanks for your timely response!
tls.Config.InsecureSkipVerify ignores most of the certificate validation logic, but what I want to realize is just filtering out the hostname validation.
The certificate we use does not contain valid hostname (DNS or IP), but our usage situation is not for testing, so I put forward the above suggestion. Do you have any idea to resolve the contradictive of my situation?

I also agree that it would be nice to independently skip the hostname validation and the trust chain validation. We have certs that for certain reasons don't match up with the server's DNS name and are self-signed, but it would be nice for us to import our CA in tls Config and still have that part of the validation performed.


Changes for mid-deliverable 1.2

Implemented UI component for Event Details
Implemented Event Storage component
Created date, time, venue attributes for Event
Edited all main directory files affected
Deleted/edited some tests for the old function of BrowserPanel
Edited some failing tests
Closes #110 , closes #165

Should be for mid-v1.3 though


OSequence tests periodically fails on CI

You may check results there for example http://helios.orientdb.com/view/_2.2.x/job/orientdb-2.2.x/348/

Is this stil valid?

Yes, test is disabled to avoid periodical crash of CI.


⚠ ClamAV has degraded performance

In 9d9b3e1, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 92 ms

Resolved: ClamAV performance has improved in 9eb97ae.


Missing scenario from how to reactivate M365 subscription instructions

This document doesn't cover the scenario where the 'reactivate subscription' is greyed out.
This seems to occur if the subscription was a MAPS based subscription and is flagged as  commercial direct.
I am currently on a support call and even they do not know what i should do.
The partner center in UI guidance also doesn't cover this scenario - it says to use the extend button (which doesn't exist in this scenario)
https://github.com/MicrosoftDocs/microsoft-365-docs/blob/public/microsoft-365/commerce/subscriptions/reactivate-your-subscription.md
Example of greyed out reactivate subscription:

@scyto Thank you for your feedback. Global or billing admins can reactivate the subscription in the admin center if the subscription is in disabled status. As mentioned in the document, If you don't see Reactivate as an available action, call Support to reactivate your subscription as they can reactivate it in the backend. There could other issues and only support can help you in this case.
Let us know if support was able to reactivate or what action plan was provided, So we can update the document if need be.
Thanks
Sri

@scyto Thank you for your feedback. Global or billing admins can reactivate the subscription in the admin center if the subscription is in disabled status. As mentioned in the document, If you don't see Reactivate as an available action, call Support to reactivate your subscription as they can reactivate it in the backend. There could other issues and only support can help you in this case.
Let us know if support was able to reactivate or what action plan was provided, So we can update the document if need be.
Thanks
Sri

@msbemba  Thank you very much for the contribution and sharing this explanation. @scyto Hope this comment is helpful for you. Considering the nature of the issue, as advised, please open a service ticket in your tenant and follow with them for the resolution. We proceed here to close it.
We will be very happy if you can share the outcome or resolution with us if you see documentation update is required. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

@msbemba  Thank you very much for the contribution and sharing this explanation. @scyto Hope this comment is helpful for you. Considering the nature of the issue, as advised, please open a service ticket in your tenant and follow with them for the resolution. We proceed here to close it.
We will be very happy if you can share the outcome or resolution with us if you see documentation update is required. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Ensure dependency issues are not closed incorrectly

Fixes #345
Signed-off-by: Faseela K<EMAIL_ADDRESS>

i think the tests will need adjusting on this also (as well as #356 )

actually not - it just tests against the CONSTANT i think


no space before function paren

Hmm, one point of the linter is to have uniform code. I don't care so much which way it goes, but I do like that it enforces one way. It can get a little nitpicky sometimes, but it's sort of a slippery slope to start disabling checks.

Coverage remained the same at 82.579% when pulling aedb474000f948107e91e1b57fe5f12ebda1ab98 on mparke:no-func-space into 15d8d5e0bd15c2fbbb2f89fcfdf742a7907edaf5 on linode:master.

we recently re-enabled anon functions, which you couldn't write before, this just enables function() {} vs function () {}

I understand. My point is that now you can write function() {} or function () {}: we lose style consistency. (And even then, this obviously isn't the first PR to loosen things up. ) I really don't care as much about any of the linter points except for that the linter enforces some particular style.


Enable variable permissions for graders

Motivation and Context
This allows admin to enable/disable various permissions for graders.

Your Changes

Description:
Created a new table that contains all the configurable grader actions and admin can enable/disable them. Enforced those permissions using Action policy for various controllers. There will be structural changes in the user interface of the application. Also creating a new user interface for the purpose of this PR.
Type of change (select all that apply):

[x] Breaking change (fix or feature that would cause existing functionality to change)

Testing
Manually tested this PR through the web interface
Questions and Comments (if applicable)

Checklist

[x] I have performed a self-review of my own code.
[x] I have added tests for my changes, if applicable.
[ ] I have updated the Changelog.md file.
[ ] I have described any required documentation changes below, if applicable.
[x] I have fixed any Hound bot comments (check after opening pull request).
[x] I have verified that the TravisCI tests have passed (check after opening pull request).
[x] I have reviewed the test coverage changes reported on Coveralls (check after opening pull request).

Required documentation changes (if applicable)

@ANUSHIYAPRIYA okay, I've started working through this PR now. I will focus first on simplifying the design of the permissions themselves, and also mention opportunities to break out smaller parts.

create_delete_annotations: I want you to remove this permission altogether. TAs should always be able to add/modify annotations in the results/edit view, and for annotation_controllers/index we will just use the manage_annotations setting. Please pull out the changes related to AnnotationPolicy (and the corresponding changes in the results view files) into a separate PR. (And so they should no longer appear here.) You can keep the AnnotationCategoriesPolicy here, though, since that will rely on the manage_annotations setting.
create_notes: You can remove this one altogether. TAs can always create notes.
manage_exam_templates: You can remove this one, and replace it with manage_assignments (i.e., if TAs can manage assignment, then they can manage exam templates).
manage_reviewers: You can remove this one, and replace it with manage_assignments (i.e., if TAs can manage assignment, then they can assign reviewers).
manage_assignments/manage_grade_entry_forms: Merge these into a single permission, manage_assessments.

@ANUSHIYAPRIYA okay everything looks good! I forgot to ask you to update the Changelog, so please update that file (and refer to this PR), and then I'll merge this in.


Linux Addons

Is there a linux version of blenderseed?

There are unofficial builds as well as ways to make your own.
Unofficial builds from Discord as well as one I created.
If you want to build yourself, there is a WIP Makefile from Discord and my Docker Container which builds Blender as well as Blenderseed for you.

Thanks, for you quickly answer.


[Bug]: LastAsyncOperation is failed, but resource has been marked as ready

Is there an existing issue for this?

[X] I have searched the existing issues

Affected Resource(s)
At least these two:

network.azure.upbound.io/v1beta1 - ApplicationGateway
managedidentity.azure.upbound.io/v1beta1 - FederatedIdentityCredential

Resource MRs required to reproduce the bug
No response
Steps to Reproduce

Create ApplicationGateway with proper configuration that does deploy it successfully
Add spec.forProvider.authenticationCertificate object with invalid name
Observe LastAsyncOperation and verify that resource status is still ready

If you use nested compositions, deploy ApplicationGateway into child composition and verify that also parent composition is showing everything as ready.
What happened?
Expected: Resource and compositions would be reporting resource as failed in all levels (parent and child composition and resource itself).
Observed: Resource, its composition and parent composition are showing everything as ready. Resource's LastActivityOperation is showing an error that prevents applying the change and therefor resource isn't matching the state that have been written into Git and pushed into cluster from there.
In these screenshots you see obvious failure that is rightfully failing, but also how these resources do look like with common ways of checking their status:
Output from kubectl get managed

Parent composition

Actual ApplicationGateway resource

Relevant Error Output Snippet
No response
Crossplane Version
1.15.1
Provider Version
xpkg.upbound.io/upbound/provider-azure-network:v1.0.0, xpkg.upbound.io/upbound/provider-azure-managedidentity:v1.0.0
Kubernetes Version
v1.27.7
Kubernetes Distribution
AKS
Additional Info
No response

@hannupiki-eficode thanks for reporting. This issue is similar to https://github.com/crossplane-contrib/provider-upjet-aws/issues/1164 . The cause is explained at https://github.com/crossplane-contrib/provider-upjet-aws/issues/1164#issuecomment-2058500394 and we will follow up there.

I'm closing this issue because we introduced a fix for the issue in provider v1.1.0


Benchmarking single-threaded execution

Add option to script to pin the execution to a single thread.

It is possible to use the ThreadsNum environment variable, which is already implemented by the C++ code:
ThreadsNum=1 ./scripts/run.sh

...
Run query1: cpp/cmake-build-release/sigmod2014pc_cpp csvs/p1k/ PARAM 1 757 383 0
cpp,1,Threads: 1/4
q1,1250896,119027
...

@marci543 sorry, the original post was unclear. I'd like to have an option to pin a benchmark to a single (logical) CPU core. This way, we can benchmark the single-threaded performance of existing solutions. See details in the 'threats to validity' section of the paper.


XSS: Javascript execution through links

Posting this:
[Just Regular Link](javascript:alert(1))

Renders
A link which will show alert on click. Should this be considered an XSS vulnerability and mitigated by library itself or by the users by creating a custom parser?

@iamareebjamal, I think an option is needed to filter out some URI formats for links. I will add it. That way custom link resolver will only be needed for special cases.

@vsch how to use SUPRESSED_LINKS ? could you give an example of configuration?

@bvn13, it is enabled by default but if you want to change the RegEx used for suppressing links you can add .set(HtmlRenderer.SUPPRESSED_LINKS, "your reg ex here") to your Parser/Renderer build options.
See the flexmark-java-samples module directory for examples.


Change reader type from io.Reader to *io.PipeReader in read function

Modified the function signature to accept *io.PipeReader for more specific use cases.

According to doc, this is correct: https://pkg.go.dev/io#Pipe


Add Sonarr, Radarr and Prowlarr to Jellyfin stack

https://www.fuzzygrim.com/posts/jellyfin

Use qbitorrent+VPN with a Windscribe VPN subscription for torrenting

Sonarr, Radarr and Prowlarr docker images


Update to<PHONE_NUMBER>1249b12b031d82aaeb3bb9d56c1591

Only changes from<PHONE_NUMBER>1249b12b031d82aaeb3bb9d56c1591
and e72407495e0ecc4158985e4605d5f84e0e712b7f were brought in as these
were the only relevant changes.
Signed-off-by: Kartik Null Cating-Subramanian<EMAIL_ADDRESS>

🚀 👍


Roles relations and inheritance

Hello! Great work!
I want to ask you about Roles relations and inheritance, similar to the way Zend Framework does ACL. For example, user inherits permissions from guest and admin inherits from user. This would make the process of defining roles much more faster and simpler.
Does entrust offer any support for this?

/push... would be super nice to have the possibility of inheritance

/push, this would be fantastic

:+1

This was opened in 2015, Any updates?
@Zizaco

this would be very useful, is this in the roadmap?

/push I am also very interested in it!
How do you solve the missing inheritance problem actually?  Or do you check every role, like $user->hasRole(['guest', 'user', 'admin']); ?

I suppose both would work,  currently we give our users multiple roles - for example Admin would get guest, user and admin.


Invalid JSON utterance results

Hello,
I have the server installed and running one of your samples using your HTTP API interface (see below).  The output from the server looks like invalid JSON data.  I must have something configured incorrectly.
Any thoughts would be greatly appreciated.
Thank you
$curl -T test/data/english_test.wav "http://localhost:8888/client/dynamic/recognize"
$python kaldigstserver/master_server.py --port=8888
DEBUG 2018-08-02 11:53:40,136 Starting up server
INFO 2018-08-02 11:53:58,416 101 GET /worker/ws/speech (<IP_ADDRESS>) 0.58ms
INFO 2018-08-02 11:53:58,416 New worker available <main.WorkerSocketHandler object at 0x7f4c4c54a110>
INFO 2018-08-02 11:54:57,842 f07e5081-e669-470e-95f4-bcc22f3e5796: OPEN: user='none', content='none'
INFO 2018-08-02 11:54:57,842 f07e5081-e669-470e-95f4-bcc22f3e5796: Using worker <main.HttpChunkedRecognizeHandler object at 0x7f4c4c54ac90>
INFO 2018-08-02 11:54:57,845 f07e5081-e669-470e-95f4-bcc22f3e5796: Handling the end of chunked recognize request
INFO 2018-08-02 11:54:57,845 f07e5081-e669-470e-95f4-bcc22f3e5796: yielding...
INFO 2018-08-02 11:54:57,845 f07e5081-e669-470e-95f4-bcc22f3e5796: Waiting for final result...
INFO 2018-08-02 11:55:01,236 f07e5081-e669-470e-95f4-bcc22f3e5796: Receiving event {u'status': 0, u'segment': 0, u'result': {u'hypotheses': [{u'transcript': u'ONE.'}], u'final': Fa... from worker
INFO 2018-08-02 11:55:01,542 f07e5081-e669-470e-95f4-bcc22f3e5796: Receiving event {u'status': 0, u'segment': 0, u'result': {u'hypotheses': [{u'transcript': u'ONE TWO.'}], u'final'... from worker

You are showing the output from the log, not the actual JSON response from the server. The log prints truncated JSON response by design.

Thanks for clarifying the log output.
When I run the following example (english_test.wav) it displays the following utterance result.  This should be ONE TWO THREE FOUR FIVE SIX SEVEN EIGHT but returns O.
$ curl -T english_test.wav "http://localhost:8888/client/dynamic/recognize"
{"status": 0, "hypotheses": [{"utterance": "O."}], "id": "bc728167-4d3d-4ac2-ba73-7f469e192aef"}

Can you post you configuration file?


Consider renaning Global Context to Global State

There are a lot of "contexts" throughout the code. Renaming this will a) more easily/intuitively communicate the function of this feature, b) help reduce the overloading of the term "context".
Same with Request Context -> Request State.

Right now, there are three, I believe.  There's the GraphQLContext trait provided by the Juniper crate that we implement in Warpgrapher. There's a GlobalContext we expose to the client of Warpgrapher, and a RequestContext that we expose to the client of Warpgrapher.
If I may propose an alternative solution.  It made sense to have both a GlobalContext and RequestContext when we were providing the server that accepted and responded to HTTP requests.  Now that we've split out the engine from the integration with any particular server, it could be entirely the client's responsibility to martial the context arguments to pass into the execute function, but mutably, so that extensions can modify data as needed.  Anything they pass to Warpgrapher will be passed for each invocation of execute.
Thus, I would propose we get rid of the GlobalContext entirely, keeping only a single Warpgrapher Context trait to be implemented by the client library.  Because we treat the GraphQLContext as entirely internal, for Warpgrapher's use, and expose only the Warpgrapher context to the client, this should mitigate the risk of any confusion.  The client library can source the data for their context anywhere they want, whether it's global across all requests or local to a single request.  They would then pass the context into the execute method, where it would be modified as needed by extension calls.

Interesting idea - I'll give this some thought.


NODE-2784: sets primary read preference for writes

The fix itself LGTM. Maybe I'm jumping the gun before re-requesting reviews, but will there be tests added to validate that DDL operations are all using fixed read preferences?

@nbbeeken good catch, that was accidental :(
@emadum also good catch, updated!


Gocode doesn't search packages

Hello, I'm running weekly.2012-02-07 +52ba9506bd99.
I installed gocode with:
go get -v github.com/nsf/gocode
go install -v github.com/nsf/gocode
I use vim, and have run:
...vim/update.bash
My ~/.profile:
export GOROOT=$HOME/go
export GOARCH=amd64
export GOOS=linux
export GOBIN=$GOROOT/bin
export GOPATH=$HOME/gopkgs:$HOME/code/go
export PATH=$GOBIN:$HOME/gopkgs/bin:$HOME/code/go/bin:$PATH
export GAEGO=$HOME/google_appengine
ISSUE:
The gocode plugin works,,but it does not search for definitions in packages. There is a godoc in ~/gopkgs/bin though.

If it works, then it searches for definitions in packages? What makes you think it doesn't? Does it autocomplete this?
package main
import "fmt"
func main() {
    fmt.
}

If yes, then what packages do you want to find? I'm pretty sure GOPATH works too.

I should mention also that gocode takes type information from compiled packages. If you haven't compiled one or more of your packages in the GOPATH it won't find them.
Another thing I've noticed that you have some env var related to google app engine, if you want gocode to look for their packages it's another issue. It is solvable though.

It does not autocomplete the code. I only get:
package
main
import
fmt
func

It seems that you're using the wrong autocompletion type, in vim you need use omnifunc autocompletion, it is invoked by  by default. As you can see your autocompletion proposals are just words in that file, that's the default thing in vim.

Now I get "Omni completion ... Pattern not found". I'm obviously no
omni completion expert :(
On Fri, Feb 24, 2012 at 1:27 AM, nsf
<EMAIL_ADDRESS>wrote:

It seems that you're using the wrong autocompletion type, in vim you need use omnifunc autocompletion, it is invoked by  by default. As you can see your autocompletion proposals are just words in that file, that's the default thing in vim.

Reply to this email directly or view it on GitHub:
https://github.com/nsf/gocode/issues/74#issuecomment-4149034

I don't think it's related to the dvorak. Probably installation issues. Try typing set :omnifunc it should print omnifunc=gocomplete#Complete, if that's correct, then it must be something on the gocode installation side. Sometimes people run their from the gnome or some other DE without using shell, but their PATH environment variable gets modified by the .bashrc. It results in the fact that vim alone doesn't know where gocode is, even though your shell knows that. That could be an issue. I've just checked, vim prints "pattern not found" when there is no 'gocode' executable on PATH.

You can check your $PATH in vim doing this: :echo $PATH

Reinstalled everything (go, gocode, vim), and it works!! I must have failed in some serious way.
Great plugin anyway. Wish you all the best!

Good, closing the issue then.


Gh-1357 parquet-writer-bugfix

Codecov Report

Merging #1358 into develop will decrease coverage by 0.23%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #1358      +/-   ##
=============================================
- Coverage      65.27%   65.03%   -0.24%
+ Complexity      1541     1494      -47
=============================================
  Files            694      590     -104
  Lines          22188    18186    -4002
  Branches        2762     2136     -626
=============================================
- Hits           14484    11828    -2656
+ Misses          6572     5500    -1072
+ Partials        1132      858     -274

Impacted Files
Coverage Δ
Complexity Δ

...isation/impl/ArrayListStringParquetSerialiser.java

...affer/rest/service/v1/example/ExamplesService.java

...parquetstore/utils/GafferGroupObjectConverter.java

...fer/rest/service/v2/example/ExamplesServiceV2.java

.../gchq/gaffer/rest/service/v1/OperationService.java

...e/serialisation/impl/IntegerParquetSerialiser.java

...r/parquetstore/io/writer/ParquetElementWriter.java

...chq/gaffer/rest/example/ExampleFilterFunction.java

...re/serialisation/impl/StringParquetSerialiser.java

...ore/serialisation/impl/FloatParquetSerialiser.java

... and 95 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 e0a1aec...6ae17b7. Read the comment docs.


Help with infracost comment on commit

I am trying to do a CI/CD with infracost
For comment on PR all does normaly
But when i try to comment on a commit i receive a error:
Error: Type mismatch on variable $commitSha and argument oid (String! / GitObjectID)
Command:
infracost comment github --policy-path infracost-policy.rego --path infracost.json --repo mpcabral/infracost-test --commit 27351c2c0bd2aefe3e3a52bbf99f8b8be73e382e --github-token ****
Am I doing something wrong?

@mpcabral Hi! Thank you for creating the issue! I was able to reproduce this and seems like to be a bug in CLI. The error message is from GitHub's GraphQL API saying that it expects commit as a GitObjectID value, but we send it as a string instead when it fetches existing comments.
We tested the functionality thoroughly during implementation and it worked fine. I tried to find any changes in API's changelog. The fix looks pretty easy to me. But it will be available only with the next release.
This happens only with the default --behavior update when CLI tries to find its existing comment and update it. If you add to your command --behavior new it should post a comment successfully. However, this option might be a bit noisy as it will create a new comment on every run. Here more info on behaviors: https://www.infracost.io/docs/features/cli_commands/#comment-on-pull-requests.
Please let me know if this helps. :)

Yes this helped me a lot and I managed to complete my pipeline
Just out of curiosity: is there any option to detect a cost policy failure and stop my pipeline without needing to enter the comment in the commit?

Just out of curiosity: is there any option to detect a cost policy failure and stop my pipeline without needing to enter the comment in the commit?

@mpcabral we don't have that option in the infracost comment command yet, but you could do it by running OPA separately. Something like this could work:
# Generate Infracost breakdown
infracost breakdown --plan.json --format=json --out-file=infracost.json

# Run the policy
opa eval --input infracost.json -d infracost-policy.rego --format pretty "data.infracost.deny" | tee opa.out

# If the policy fails, log the details and exit
denyReasons=$(<opa.out)
if [ "$denyReasons" != "[]" ]; then
  echo -e "Policy check failed:\n$denyReasons"
  exit 1
# If the policy passes, continue and post the comment
else
  echo "Policy check passed."
  infracost comment github --policy-path infracost-policy.rego --path infracost.json --repo $GITHUB_REPO --commit $GITHUB_COMMIT --github-token $GITHUB_TOKEN
fi

@mpcabral We just merged the fix to master. It will go live with the next release. It automatically closed this issue too. Please feel free to reopen if you think there is anything unresolved. Thanks again for reporting it! :)
BTW, have you had a chance to look at Alistair's comment about the policy? If yes, has it worked for you?

Hey @mpcabral, this has now been released as part of the v0.10 release. Thanks for bringing this to our attention.


Add ANCM

Fixes aspnet/CoreCLR#253. Since we're taking over development on AspNetCoreModule we should build it in Universe instead of in the CoreCLR mirror.

See https://github.com/aspnet/CoreCLR/issues/253#issuecomment-319762365


Allow to change symbol in a feature

Now the symbol property is not added in the feature so it can not be changed whith hover or click. It would be nice to have this feature available in the API.
const viz = new carto.Viz(`
  width: 30
  symbol: image('${marker}')
`);

[...]

feature.symbol.blendTo(`image('${markerSelected}')`);
feature.symbol.reset();

I think this can be done by modifyng https://github.com/CartoDB/carto-vl/blob/master/src/interactivity/feature.js
After supporting blending betweem images, which is an existing issue that is already under development.


Adding company list

Part of the MVP: #84
Note: faked an example, until backend responds.
Example:

This feature won't should go live for now. Let's reopen it in the future!
Thanks!


added clabverter dockerfile

fix #17
This is a minimal dockerfile for clabverter. Need to figure out how to include it in the build/release pipelines which use devspace...

Need to figure out how to include it in the build/release pipelines which use devspace...

this I can be helpful with :p


AIAvatarVRM

The demo scene shows AIAvatarVRM as missing.  Did I miss a step?

Sorry, I missed the extensions package.  :)


Homebrew CI Test Failure: "thread 'main' panicked"

I'm trying to update Homebrew's Rust to 1.48 here: https://github.com/Homebrew/homebrew-core/pull/66285
Homebrew CI tested all packages that depend on Rust. This revealed the following error from ktmpl:
Backtrace

❯ RUST_BACKTRACE=full ktmpl test.yml -p PORT 8080
thread 'main' panicked at 'attempted to leave type `linked_hash_map::Node<yaml::Yaml, yaml::Yaml>` uninitialized, which is invalid', /private/tmp/rust-20201211-36507-3y6ny0/rustc-1.48.0-src/library/core/src/mem/mod.rs:658:9
stack backtrace:
   0:        0x104889a5e - <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt::h650be7d6317a1169
   1:        0x1048b601e - core::fmt::write::h451c4858d0fea670
   2:        0x1048897ca - std::io::Write::write_fmt::had0d660ea6c32991
   3:        0x1048a384f - std::panicking::default_hook::{{closure}}::h7f9d4cee5e038ae9
   4:        0x1048a3543 - std::panicking::default_hook::h5fb4408d2b8d41f0
   5:        0x1048a3d7a - std::panicking::rust_panic_with_hook::h6f223595664a4622
   6:        0x10488a2fb - std::panicking::begin_panic_handler::{{closure}}::ha7895c801e815942
   7:        0x104889bd8 - std::sys_common::backtrace::__rust_end_short_backtrace::hafb1945b261ddb13
   8:        0x1048a3953 - _rust_begin_unwind
   9:        0x1048c0baf - core::panicking::panic_fmt::h23cb1548c6906f6a
  10:        0x1048c0b07 - core::panicking::panic::h97965fc650fa4c16
  11:        0x10482f6ce - linked_hash_map::LinkedHashMap<K,V,S>::insert::h6aed8966938aeef9
  12:        0x10482d900 - yaml_rust::yaml::YamlLoader::insert_new_node::h84a2dc6aca853064
  13:        0x10482d2fd - <yaml_rust::yaml::YamlLoader as yaml_rust::parser::MarkedEventReceiver>::on_event::h90cc8ed523819ea3
  14:        0x104820ebf - yaml_rust::parser::Parser<T>::load_node::hea42f316d6ec0f94
  15:        0x1048213fe - yaml_rust::parser::Parser<T>::load_mapping::h657e040bebe66fbe
  16:        0x10482095b - yaml_rust::parser::Parser<T>::load::hfaa369a92904d439
  17:        0x10482daa8 - yaml_rust::yaml::YamlLoader::load_from_str::hbc8c4808e30bc98b
  18:        0x104797f36 - ktmpl::template::Template::new::h37f21bc74c7d28da
  19:        0x10478e679 - ktmpl::main::h10bda75d6a75f5e5
  20:        0x10478c38a - std::sys_common::backtrace::__rust_begin_short_backtrace::he46e50b20aaf6f38
  21:        0x10478c3ac - std::rt::lang_start::{{closure}}::h2102adef0070e7f3
  22:        0x1048a10ad - std::rt::lang_start_internal::h2f38ef5052441758
  23:        0x10478f4d9 - _main

For reference, these are the contents of test.yml.
Test File

---
kind: "Template"
apiVersion: "v1"
metadata:
  name: "test"
objects:
  - kind: "Service"
    apiVersion: "v1"
    metdata:
      name: "test"
    spec:
      ports:
        - name: "test"
          protocol: "TCP"
          targetPort: "$((PORT))"
      selector:
        app: "test"
parameters:
  - name: "PORT"
    description: "The port the service should run on"
    required: true
    parameterType: "int"

I haven't had the pleasure of using ktmpl, so I'm not really sure what's going on. I'd appreciate some assistance here. Is the issue with test.yml? It worked with previous CI runs.
For reference, I ran this on macOS 11.0, but the error also appears on macOS 10.14 and 10.15. (All Intel Macs.)
I should also mention that I saw deprecation warnings upon compilation. More details available if you need it.

This might be fixed by https://github.com/Homebrew/homebrew-core/pull/66914.

Yep, I think so too. This is a hilariously embarrassing oversight. Will close this when CI on your PR is green.

CI is green. 💚

This is started happening again. It's because of yaml-rust - bumping the version 0.4.5 fixes the issue for me. ktmpl currently used 0.4.2.


Calling Save() doesn't flush to the stream

Description
I am writing a wrapper class for SpreadsheetDocument. I would like to be able to Save the document more than once. Currently the only way to force SpreadsheetDocument to flush to the stream is by calling Close() on it. Is there anyway to avoid that?
Information

.NET Target: .Net Standard 2.0
DocumentFormat.OpenXml Version: 2.7.2

Repro
using DocumentFormat.OpenXml;
using DocumentFormat.OpenXml.Packaging;
using DocumentFormat.OpenXml.Spreadsheet;
using System;
using System.IO;

namespace OpenXmlDemo
{
    class Program
    {
        static void Main(string[] args)
        {
            using (var stream = new MemoryStream())
            {
                var document = SpreadsheetDocument.Create(stream: stream,
                                                          type: SpreadsheetDocumentType.Workbook,
                                                          autoSave: true);

                var workbookPart = document.AddWorkbookPart();
                var workbook = workbookPart.Workbook = new Workbook();
                var sheetsElement = new Sheets();
                workbook.AppendChild(sheetsElement);

                InsertSheet(document, workbookPart, sheetsElement, "Sheet A", 1);

                // Save the workbook and document
                workbook.Save();
                document.Save();

                // document.Close();

                // Returns an empty array if the document isn't closed
                // stream.Position = 0; // Some places suggested this, unfortunately this doesn't seem to help.
                var bytes = stream.ToArray();
                File.WriteAllBytes(@"D:\sheet.xlsx", bytes);
            }
        }

        private static Sheet InsertSheet(SpreadsheetDocument doc,
                                      WorkbookPart workbookpart,
                                      Sheets sheetsElement,
                                      string name,
                                      uint id)
        {
            WorksheetPart newWorksheetPart = workbookpart.AddNewPart<WorksheetPart>();
            newWorksheetPart.Worksheet = new Worksheet(new SheetData());

            string relationshipId = workbookpart.GetIdOfPart(newWorksheetPart);

            // Give the new worksheet a name.
            if (string.IsNullOrEmpty(name))
                name = "Sheet" + id;

            // Append the new worksheet and associate it with the workbook.
            Sheet sheet = new Sheet() { Id = relationshipId, SheetId = id, Name = name };
            sheetsElement.AppendChild(sheet);

            return sheet;
        }
    }
}

Observed
The stream is empty.
Expected
The stream should contain the document.

I've encountered this as well. I think it may be a limitation of System.IO.Packaging, but I'll look into it

This is blocked by a bug in .NET Core. I've opened an issue here: https://github.com/dotnet/corefx/issues/25273

@encrypt0r I have a PR open to enable this with a flag on OpenXmlPackage.CanSave indicating whether the platform fully supports saving. It will still only work on .NET Framework, but the flag should help understand at runtime if saving will be successful or not. Please check it out at #468 and leave any feedback. This would be released in a 2.9.0 release.


Build error when installing with LuaRocks

Sorry if this ends up being a noob mistake; I looked for a while but couldn't find anything that suggested a solution.
$ sudo luarocks install luafilesystem
Installing https://luarocks.org/luafilesystem-1.7.0-2.src.rock
gcc -O2 -fPIC -I/usr/include/lua5.3 -c src/lfs.c -o src/lfs.o
src/lfs.c:66:10: fatal error: lua.h: No such file or directory
 #include <lua.h>
          ^~~~~~~
compilation terminated.

Error: Build error: Failed compiling object src/lfs.o

I'm running the current release of Ubuntu Studio and the (I would assume) current version of LuaRocks from apt-get.
Cheers, and please let me know if there's more information I can provide.

Sorry, it was me. Installing the lua5.3 package doesn't include liblua5.3-dev even though Ubuntu's page for the Lua package says it does.


Downloading webpage ERROR: Unable to extract uploader id

Checklist

[ ] I'm reporting a broken site support issue
[ ] I've verified that I'm running youtube-dl version 2021.12.17
[ ] I've checked that all provided URLs are alive and playable in a browser
[ ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ ] I've searched the bugtracker for similar bug reports including closed ones
[ ] I've read bugs section in FAQ

Verbose log

PASTE VERBOSE LOG HERE

Description

youtube-dl https://www.youtube.com/watch?v=JYNyQcblzgI
[youtube] JYNyQcblzgI: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

This is a duplicate of https://github.com/ytdl-org/youtube-dl/issues/31530

This seems to be solved in the most recent source code.

Install python
Install this package: pip install git+https://github.com/ytdl-org/youtube-dl
This way it worked for me.

but i was able to use it.

When I did the following:

Install python
Install this package: pip install git+https://github.com/ytdl-org/youtube-dl
2.2 It was complaining about pip3.8 file not available in usr/bin location , then I moved the file pip3.8 from usr/local/bin/ to usr/bin and it worked fine.

@szabgab  I am also having same issue.

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', '11Yr3YhdUrE']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Git HEAD: ac0663ae5
[debug] Python version 3.10.11 (CPython) - macOS-13.3.1-arm64-arm-64bit
[debug] exe versions: ffmpeg 6.0, ffprobe 6.0
[debug] Proxy map: {}
[youtube] 11Yr3YhdUrE: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/opt/homebrew/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "/opt/homebrew/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "/opt/homebrew/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/opt/homebrew/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/extractor/youtube.py", line 1794, in _real_extract
    'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,
  File "/opt/homebrew/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.```


Missing "credential type" translations for Registries

In the current environment for the new UX, the translations for the credential types for Registries (registration, business_number and relationship) are missing.  Please add them.
See: https://frontend-v2-dev.apps.silver.devops.gov.bc.ca/search?q=inc&category%3Aentity_type=&credential_type_id=&inactive=&page=1

Those values are fetched from the credential_type.description field. This is more a problem on the issuer's end since they did not provide a proper description for their credential schema when they published it.

True -- but we control the user end and if we are going to show this to the testers, we don't want them all reporting that as a problem in the UX.  Better to patch the data in the database instance.  I believe @amanji knows how to do this.

We can do a db patch for now to update the credential type definition. Later we can choose to re issue a schema with updated metadata.

We've decided to add two new db fields for schema name localizations similar to what we do for credential attribute/claims


module.js:438 Uncaught Error: Cannot find module 'underscore'

I'm getting the above error in the chrome console on the lastest git pull. Electrum window looks like this:

Hi, Mariodian! I ran into this issue as well - and was able to resolve it by deleting my local node_modules directory and rerunning npm install.
Since npm doesn't revalidate recursive dependency chains during subsequent install processes, a failed/partial initial install can result in a node_modules state which npm doesn't know how to recover from.
https://github.com/npm/npm/issues/1341

Either way we failed to explicitly declare underscore as a dependency in our package.json and since we're directly importing it, we should be.
Anyhow, I've fixed it here (should be merged into master soon):
https://github.com/OpenBazaar/openbazaar-desktop/pull/14


test(RHTAPWATCH-1043): add kubelint to CI

Status:

The linter is running - it might not be the most ideal way to make it run for all kustomization files (e.g. some nested files may run multiple times -- not sure if that's a problem), but it's working.

I applied some fixes to some of the issues. Some of them are breaking the deployment (I think the main issue is low resources that need to be tweaked). This needs to be looked on one by one.

Some fixes are still missing. Need to attend to those as well.


server: wrap leader transfer operator as must succeed once operator.

why not handling directly in LeaderTransfer?
I think it breaks the semantics for Once.

@siddontang Agreed, PTAL.

LGTM

LGTM

@shenli  @siddontang
After the discuss, we all think using hb count is better, so i update this check.
PTAL

LGTM


V2 -- Divide env config into appEnv and nodeEnv

Currently we set the NODE_ENV by mapping 1:1 to our Webpack configs. When working with multiple environments (e.g. dev, staging, dist) I found the requirement for finer control. E.g. to achieve this
dev     -> NODE_ENV=dev,        APP_ENV=dev
staging -> NODE_ENV=production, APP_ENV=staging
dist    -> NODE_ENV=production, APP_ENV=dist

In this example, staging and dist should both build with production optimizations (e.g. webpack -p and NODE_ENV=production). However, staging may differ in other conditional flags used throughout the app. This means, inside the project I need to distinguish between code targeted for staging and production, which I do by means of the APP_ENV setting.
Initially I was using the setting from src/config/<env>.js to pick appropriate values. However, in the case of dynamic requires Webpack would not eliminate dead code that way, so I'd propose using the DefinePlugin and introduce two changes to the configs:
// Somewhere inside conf/webpack/Staging.js
get env() {
  return process.env.NODE_ENV || 'production';
}

get appEnv() {
  return process.env.APP_ENV || 'staging';
}

this.config = {
  plugins: [
    new webpack.DefinePlugin({
      'process.env': {
        NODE_ENV: JSON.stringify(this.env),
        APP_ENV: JSON.stringify(this.appEnv)
      }
    }),
    // ...
  ]
};

This way, process.env.APP_ENV becomes available in code and everything inside the branch will be eliminated from the bundle size when the condition is not met (I am using it for things like requiring gobal mocks, adding dev routes, dev components, etc.).
@weblogixx would this addition be okay for you?

Hi @sthzg, more than ok. I use it nearly the same way in some current projects where I use the global conf to set api url targets for dev, staging and production usage. Currently, I just copied the base dev config and adjusted it accordingly (which was way harder when using v2 of the generator). For me this is totally fine.


Make Settings for the individual extensions

You often don't want to use all the extensions, so this adds project settings to control which extensions are enabled for the addon.
I've found it useful for my current game.

So because this requires 4.4 should we branch this or hold it off?
It's fine on my local branch but not sure if we want to restrict this to 4.4? Is there a version define we can do to make this back compat?

So because this requires 4.4 should we branch this or hold it off?
It's fine on my local branch but not sure if we want to restrict this to 4.4? Is there a version define we can do to make this back compat?

It's just something to keep in mind when we're thinking about the timing of merging this. Like, just to consciously decided when master is no longer Godot 4.3-compatible, and not accidentally merge something that requires 4.4 before we mean to. However, I guess our CI seems to have caught it in this case anyway :-)

I'm concerned about this approach as if not done right, this would be another mechanism for users to shoot themselves in the foot..
We have extensions that depend on each other (e.g: fb_scene_extension & fb_spatial_entity_extension), but the proposed mechanism does not take this into account, which means that as a user, I could enable fb_scene_extension and disable fb_spatial_entity_extension and be confused about why the former is not working.
The proper mechanism IMO would be to group related extensions under a common setting, so you enable / disable all related settings together. However this would increase the complexity and maintenance burden of this feature..

It would be a nice improvement with error messages or similar. Not too bothered as I've catered to my needs lovely but I without the ability to turn off stuff you don't need honestly this addon becomes bloat that is unusable in serious projects.
The question is if we need to make all those improvements in this PR or in a later one. I think the maintenance burden is required if you want to keep it friendly.
Maybe in each extension wrapper we add a dependency macro? To enforce enable and order of init

Personally, I think implementing this is worth the maintenance burden.
Right now, we enable all extensions, but don't necessarily use them. Imagine we implemented all the vendor extensions supported by Meta, for example: that's like ~50 extensions. On the Godot side, this means registering ~50 extension wrappers in Godot, which all get their "hooks" invoked (ie _on_process(), _on_pre_render(), etc). And also internally within the OpenXR runtime, it's going to initialize everything for all ~50 extensions to be used.
If a given app is only going to use, for example, 2 of those extensions, it's much cleaner to just not enable the other ~48.
The biggest downside to this in my mind is that, up until now, we've been primarily using export settings for this, and now we end up project settings too. We'll need to figure out how we want to split these things up between the two places.

Just registering you want an extension enabled to the runtime on boot costs memory and perf.
The issue on using export settings is it leads to a false testing experience for non exporting builds which increase the "release only" style bug


Submit a Tool

Adding two tools for ecosystem of the aerospace hacking tools website.
Tools:

Orbitron - Satellite Tracking System
MMSSTV

Thanks so much for contributing 🚀✨


Ability to hide app from Dock

Ability to hide app from Dock
There's a window-level .skip_taskbar() method, but as far as I can tell nothing for hiding the app itself from the Dock

skip_taskbar should always be for window-level.
I am assuming you are on macOs and unfortunately it is not supported atm because on macOs it is app-level and we still haven't decided how to introduce this.

I think we forgot to document that it doesn't work on macOs

This could be accomplished with set_activation_policy
https://github.com/tauri-apps/tao/blob/dev/examples/system_tray_no_menu.rs#L36
But, tauri do not expose it yet.

@amrbashir Yeah, I agree that skip_taskbar should only be window-level.
In Electron they just have dock.hide() and dock.show(), maybe Tauri could have set_dock_visibility(bool)


View

Where are php-view, twig-view, csrf, flash message?
How to implement php-view using league/container or php-di???

All our packages are located here https://github.com/slimphp


Add MAX32660 target

This pull request adds a target descriptor for the MAX32660 device. The descriptor has been generated using target-gen, using this CMSIS pack as input.

Note that this is untested because I don't have experience building probe-rs and related tools yet. I intend for this to change, but if someone manages to test me before that happens, I'd be grateful :)

I was able to download a binary using cargo embed, attach a debugger using the probe-rs vscode extension and stop/step during the debugging session. I'm not sure if I need to do anything else here, how can we move forward?

The halt you get in cortex-m/and.S is because you have "haltAfterReset" in your launch.json.
You can set that to false and then the first halt will be your breakpoint.

There has been some recent changes in target-gen.  I would appreciate if you can update from master, and then re generate the target.  Once you have done that and re-run your test, we should be able to promote this :-)

Thanks, I've re-generated the target file, built a debugger from local sources (cargo install --force --path debugger) and ran my launch config. It seems nothing broke in the process :)

bors r+


Use effect syntax for OCaml 5.3

This currently needs https://github.com/ocaml/ocaml/pull/12309 (soon to be merged) and opam pin add multicont --dev as that also fails to compile with the released version.

This is expected to fail in ocaml-ci until https://github.com/ocaml/ocaml/pull/12309 is merged. You can try it out quickly by using opam compiler '#12309' to get an opam switch the PR included.

[..] opam pin add multicont --dev as that also fails to compile with the released version.

Do you want me to prepare a new release?

@dhil a new release would be great! I think it's https://github.com/dhil/ocaml-multicont/pull/8 that is the problem, and you've merged that into your development tree.

It's green now in CI so I've gone ahead and merged this, and also added a note to the README that users who want a pre-effect-syntax can use the 5.1.1 tag instead.


IHittable: add snapping and respect data area clipping

Feature Suggestion
Feature description:
Hello Scott, I just played a bit with the IHittable interface, and here are some comments about it and potential ideas to improve it I hope

The current implementation of IHittable does not account for margins, as IDraggable does. There is no snap distance, making it difficult to implement and use IHittable on points (instead of polygons)
At the moment, it seems IHittable can only be activated/disactivated, allowing to hit an object on an event (e.g) mouse click could be a nice addition if not already possible. This would improve performance for IHittable ScatterPlots
I think the detection performed on draggables with the isUnderMouse is redundant with what should be implemented in HitTest, especially considering the addition of the snap distances. There is a probable refactor to be done here since IDraggable seems to be a IHittable (or more likely IHoverable but that just semantics) with extra steps.

The most difficult thing, with respect to the current implementation seems to be the ability to catch mouse events. Do you think this could be done without too much hassle ? Also that would be really nice to have that in v5 don't you think ?

Hi @BambOoxX! Sorry it took me so long to get to this. I've been putting all my time lately into moving #1955 forward (with discussions in #1966). I'd like to address this issue so it doesn't keep floating out there.
Interactivity Features

nice to have that in v5

Mouse interactivity in ScottPlot4 has been somewhat painful to develop, maintain, and extend over the years. This is in part because GUI programming is hard, it's been difficult for me to predict which behaviors users will want and to make those tasks easy to achieve, and finally ScottPlot4's clunky API hasn't been making this much easier.
With ScottPlot 5 I certainly do want all this to be easier, but I haven't figured out an API to support it that I'm happy with yet, so for the next few months ScottPlot 5 will intentionally not support any mouse interactivity 😈
My plan is to figure out a way to make it extremely easy for users to add their own mouse interactivity to interact with plottables. I'm really motivated to work on that API and get it released as soon as possible, and I have limited enthusiasm for adding new features to ScottPlot 4. However if we can achieve some quick wins on this front, I'm all for it!
IHittable should respect margins and support snap distance
I think I can add these features without much trouble. I'll leave this issue open to track progress on this front.
Other topics
To be honest I don't really understand your point 2 😅
Regarding overlap between Hittable, Draggable, and Hoverable, I totally agree. I don't want to break the public API by refactoring this in ScottPlot 4, so I think things will remain mostly frozen as they are, but I've definitely learned some painful lessons about how not to design an API for ScottPlot 5 🙃

The most difficult thing, with respect to the current implementation seems to be the ability to catch mouse events. Do you think this could be done without too much hassle ?

See the code at the bottom of #1844 - does this adequately demonstrate how to achieve the functionality you're looking for?

I'm creating a demo app to make usage more discoverable

Dang, this is brutally complex 😝
All the units are in coordinate units (not pixel units), so there's no way to know "units per pixel", so there's no way to know if the cursor is N pixels away from the bounding box.
I keep coding-out basic ideas for how to implement this but they either blow up in complexity or get really messy (like storing previous pixel size states from one render to another).
This may be a problem that needs to be solved on the client side (in which case I can provide an example to make it easier).
I'm going to put this down for the rest of today but pick this up tomorrow 👍

Hi. No worries, I've seen what you are doing on v5, and I'm looking forward to trying this!
Regarding point no 2.
AFAIK, at the moment, the detection of objects on the plot, either draggable or hittable is performed continuously, so each time the mouse moves, that triggers a new set of computations, such as hitTest or isUnderMouse.
My proposal was to have the possibility to perform the triggering of these computations a bit less expensive by doing them on a specific event, here Mouse Click.
The example with the hittable scatter plot is : I have a set of points plotted and when clicking on one of these points I want to be able to find which one of them.
I hope it's clearer for you now.

Regarding v5. I know very little about such complex designs, but would it be possible for each plottable to have a MouseEvent  and KeyboardEvent property so that one could link actions to plottables?
Also, I think some sort of ReturnOnEvent could have practical use. Because in general when I want interaction in a plottable it's because I need to retrieve some information about it related to the action I performed with either  mouse or key.
Not sure if this helps but these are my two cents

How to select the nearest point on click (not on mouse move)

I have a set of points plotted and when clicking on one of these points I want to be able to find which one of them.

Hi @BambOoxX, I think I found a solution! Let me know if this helps you get your job done, and if not I'm happy to work together to come up with one. We can continue the conversation here or on the Discord! (#1966)
Does this code/screenshot adequately demonstrate the idea? This can be achieved using the existing published library.

public partial class Form1 : Form
{
    Random Rand = new(0);
    double[] Xs;
    double[] Ys;
    ScottPlot.Plottable.MarkerPlot Marker;
    double SnapDistancePx = 50;

    public Form1()
    {
        InitializeComponent();
        (Xs, Ys) = DataGen.RandomWalk2D(Rand, 50);
        formsPlot1.Plot.AddScatter(Xs, Ys);
        Marker = formsPlot1.Plot.AddMarker(0, 0, MarkerShape.openCircle, 20, Color.Red);
        formsPlot1.LeftClicked += FormsPlot1_LeftClicked;
        formsPlot1.Plot.Title("Waiting for click...");
        formsPlot1.Refresh();
    }

    private void FormsPlot1_LeftClicked(object sender, EventArgs e)
    {
        (double mousePixelX, double mousePixelY) = formsPlot1.GetMousePixel();
        (double mouseX, double mouseY) = formsPlot1.GetMouseCoordinates();

        // determine the point in the scatter plot closest to the mouse
        double closestDistance = double.PositiveInfinity;
        int closestIndex = 0;
        for (int i = 0; i < Xs.Length; i++)
        {
            (double x, double y) = formsPlot1.Plot.GetPixel(Xs[i], Ys[i]);
            double dX = mousePixelX - x;
            double dY = mousePixelY - y;
            double distance = Math.Sqrt(dX * dX + dY * dY);
            if (distance < closestDistance)
            {
                closestDistance = distance;
                closestIndex = i;
            }
        }

        // take action based on whether the click engaged a point
        if (closestDistance < SnapDistancePx)
        {
            double x = Xs[closestIndex];
            double y = Ys[closestIndex];
            formsPlot1.Plot.Title($"Clicked point [{closestIndex}] (X={x:0.00}, Y={y:0.00})");
            Marker.IsVisible = true;
            Marker.X = x;
            Marker.Y = y;
        }
        else
        {
            formsPlot1.Plot.Title($"Clicked empty space (X={mouseX:0.00}, Y={mouseY:0.00})");
            Marker.IsVisible = false;
        }

        formsPlot1.Refresh();
    }
}

Let's not modify IHittable
One of my big design goals for ScottPlot 5 is to make it easier to customize behavior using client-side code (rather than requiring a developer to anticipate and build-in all the various combinations of functionality users may want).
I still have some work to do figuring out the best API here, but I'm considering not supporting IHittable or even IDraggable, but instead trying to make it as easy as possible for the user to implement their own hit detection and/or dragging code on their side. We'll see how it goes 🤞
I'm happy this issue resulted in a new WinForms demo showing how IHittable works (#1985), but I don't think I'll go all the way and add the complexity of unit/pixel state management necessary for IHittable to support pixel snapping.


Failed to parse: Proxies

Failed to parse: Proxies using hostname:port:username:password seem you didnt add including user and password on it

The issue has been fixed, thanks for letting us know. ✅
Don't forget to star the repository if you enjoy using our bot 😁


The BaseOutputPath/OutputPath property is not set for project 'kdmapper.vcxproj'

The BaseOutputPath/OutputPath property is not set for project 'kdmapper.vcxproj'.  Please check to make sure that you have specified a valid combination of Configuration and Platform for this project.  Configuration='Debug'  Platform='x64'.  This error may also appear if some other project is trying to follow a project-to-project reference to this project, this project has been unloaded or is not included in the solution, and the referencing project does not build using the same or an equivalent Configuration or Platform.	kdmapper	E:\VSCODE2022\MSBuild\Current\Bin\amd64\Microsoft.Common.CurrentVersion.targets	824

Make sure your Visual Studio is working properly, and you download the entire source correctly


Jballerina runtime reflection api for call functions from its name

Description:
$Subject

This will be fixed with https://github.com/ballerina-platform/ballerina-lang/issues/36559


Conn_id dinamic

hello. I have a suggestion for improvement. in the conn_id, it could be passed as a parameter in the DAG, if not informed, use the default. I have seen this need in clients with different servers. thanks

Hello, the conn_id can be passed as a parameter in both the pipeline and workflow operators (the parameter is called "hop_conn_id"). Is this what you meant?

Exactly.
I would like to use a name other than hop_default.
Reading the plugin code, I understood that it is fixed, not being possible to change or inform another value.
I read the Pentaho plugin and understood that it is possible.

Hi @pauloricardoferreira ,
It seems that you probably misunderstood the code.
https://github.com/damavis/airflow-hop-plugin/blob/ef94795d706cb2beaf234356a06d1d7dd28955c7/airflow_hop/operators.py#L61
This is the default value when not provided, but you can provide it if you want it this way:
my_workflow =  HopWorkflowOperator(workflow='workflows/my_workflow.hwf',
                                   project='my_project',
                                   log_level='Basic',
                                   hop_conn_id='another_hop_connection')

Tks


Updating value type to match available option types

Overview
When selecting a utility provider and tariff, we receive the tarriffId as a number. When I pass in that number into the value field of the SelectInputNative, I get a type error that it doesn't match a string:

I believe SelectInputNative should accept a value that matches the types of the option's keys.
Github Issue or Trello Card
This PR addresses this issue:
https://trello.com/c/74yuGx9I/1943-utility-rate-selection-limited-to-the-first-entry
What type of change is this?

[x] Bug fix (non-breaking change which fixes an issue)
[x] Updating documentation.

Before Doc:

After Doc:

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

UI Checklist

[x] I have conducted visual UAT on my changes/features.
[x] My solution works well on desktop, tablet, and mobile browsers.

 Utility Rate selection limited to the first entry

 Utility Rate selection limited to the first entry


⚠ Avatar API Endpoint has degraded performance

In 51d7cef, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 1141 ms

Resolved: Avatar API Endpoint performance has improved in 7efb3a2 after 5 minutes.


[SSR][vite:ssr-manifest] The "path" argument must be of type string. Received undefined

Describe the bug
rendering chunks (56)...[vite:ssr-manifest] The "path" argument must be of type string. Received undefined
error during build:
TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received undefined
    at new NodeError (internal/errors.js:322:7)
    at validateString (internal/validators.js:124:11)
    at Object.basename (path.js:1281:5)
    at Object.generateBundle (/Users/ying/workspace/pro-vip/node_modules/vite/dist/node/chunks/dep-27bc1ab8.js:38775:52)
    at /Users/ying/workspace/pro-vip/node_modules/rollup/dist/shared/rollup.js:22823:37
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Reproduction
dscsc
System Info
System:
    OS: macOS 12.5
    CPU: (6) x64 Intel(R) Core(TM) i5-8500B CPU @ 3.00GHz
    Memory: 2.80 GB / 32.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 14.19.0 - ~/.nvm/versions/node/v14.19.0/bin/node
    Yarn: 1.22.18 - /usr/local/bin/yarn
    npm: 6.14.16 - ~/.nvm/versions/node/v14.19.0/bin/npm
  Browsers:
    Chrome: 103.0.5060.134
    Safari: 15.6
  npmPackages:
    @vitejs/plugin-legacy: ^1.8.0 => 1.8.1
    @vitejs/plugin-vue: ^2.3.1 => 2.3.1
    @vitejs/plugin-vue-jsx: ^1.3.9 => 1.3.10
    vite: ^2.9.1 => 2.9.5

Used Package Manager
yarn
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.


2/26/23 PR Comments

Addressing comments from this PR: https://github.com/bloXroute-Labs/hummingbot/pull/15

Addressing comments from this PR: #15

Do we want to start adding the test coverage hummingbot requires? They are gonna comment about that right off of the bat I would think


Search not working

The neurodesk.org website search currently doesn't work.
It looks like we are using lunr to do this? Is this what we need to set it up?
https://codewithhugo.com/hugo-lunrjs-search-index/
@aswinnarayanan, was it never set-up or is just something simple broken?
Another option I found is using this search provider:
https://docsearch.algolia.com/
Any ideas what's causing the issue and how to best fix it :?

I'll have a look

Lets try algolia. I've applied for their docsearch, and it is already integrated into hugo docsy.

Am fixing the Algolia search. I think I sent them the wrong domain (neurodesk.org instead of www.neurodesk.org)
Have asked them to update

cool - that works actually quite well :)

Updated to algolia search


Create TV browse screen with default cards and model data

Requires to use the BrowseFragment and populate the rows with proper data.


Pod Disruption Budget for grafana pods

Do we need a pdb spec for grafana pods or is it an overkill for grafana ? Perhaps we should add one. Any thoughts on this ?

@AdheipSingh PDB spec would be a very welcome addition.

@pb82 yes i would add it for sure, i am on travel, apologies for the delay ill add it for sure.

No problem at all, thanks!

Any update on PDB ? Is it included in spec currently ?

Any update on PDB ? Is it included in spec currently ?


Recalculate Transform Style onMouseDown

Hey I am writing a tiny svelte library which changes size of the elements for which I change transform style of the element,
I also rely on this library to drag  elements around for a different project.  When I  change transform style externally it doesn't move from that position rather from last dragged position

Hey I am writing a tiny svelte library which changes size of the elements for which I change transform style of the element,
I also rely on this library to drag  elements around for a different project.  When I  change transform style externally it doesn't move from that position rather from last dragged position

If i understand correctly, your own transform:  rules are overriding neodrag's transformations? If that's the case, check out legacyTranslate or transform option in the docs

Thanks, that fixed it. But wouldn't it still cause problems later down the line when a different library changes translate externally.
Feel feel free to close this issue


HDF5 -> nump standalone function

@martacamps , can you add the pre-processing you already have to the example_CFD project directory?


Feedback for: cs-engine/install.md

File: cs-engine/install.md
There is no such thing as SLES12.3.
https://www.suse.com/releasenotes/
Please check with @mavenugo on the right version

@mavenugo @cpuguy83 what version of SLES does CS Engine support, given that SLES 12 only seems to have 2 service packs.


Expand fit functionality: parameter correlations and MINOS errors

Story
As a user, I want access to parameter correlations obtained by a fit and MINOS errors to study the fit results in more detail.
Assumptions

either access to minuit is possible via pyhf.infer (depending on https://github.com/scikit-hep/pyhf/issues/881), or re-implemented in cabinetry
the minuit backend provides this functionality, while other backends may not provide the same functionality and users have to switch backends if they want MINOS errors

Acceptance criteria

parameter correlations are optionally returned from a fit and visualized as a correlation matrix
MINOS errors are optionally available when using the minuit backend
when backends other than minuit are used, users are informed if the functionality is unavailable

This iminuit tutorial shows some of the available functionality: https://nbviewer.jupyter.org/github/scikit-hep/iminuit/blob/master/tutorial/basic_tutorial.ipynb

#61 partially addresses this with a new fitting function that directly uses iminuit.Minuit. It returns the correlation matrix, and visualization for it is also included.

iminuit issue related to MINOS erros: https://github.com/scikit-hep/iminuit/issues/479

Related pyhf issue: https://github.com/scikit-hep/pyhf/issues/1099, which would allow for cleaner implementation when using pyhf.infer.

The last part of this issue, MINOS errors, will be added with #132. There are no other minimizers supported besides Minuit so far, so this:

when backends other than minuit are used, users are informed if the functionality is unavailable

will have to be added when other minimizers are supported.


Issue 376

this is awesome!! thanks a lot for working on this pipeline
only issue I see is that it seems that the virtualenv folder was committed, which results in having ~6900 new files on the repo

Hello @Franko1307! Thanks for updating the PR.

In the file pipelines/perihelion-mercury/py-scripts/base_solution.py, following are the PEP8 issues :

Line 13:12: E221 multiple spaces before operator
Line 14:13: E221 multiple spaces before operator
Line 16:13: E221 multiple spaces before operator
Line 21:14: E261 at least two spaces before inline comment
Line 22:14: E261 at least two spaces before inline comment
Line 23:3: E221 multiple spaces before operator
Line 23:14: E261 at least two spaces before inline comment
Line 24:14: E261 at least two spaces before inline comment
Line 32:63: E202 whitespace before ')'
Line 60:24: E201 whitespace after '('
Line 60:52: E202 whitespace before ')'

In the file pipelines/perihelion-mercury/py-scripts/perihelion.py, following are the PEP8 issues :

Line 17:12: E221 multiple spaces before operator
Line 18:13: E221 multiple spaces before operator
Line 20:13: E221 multiple spaces before operator
Line 25:14: E261 at least two spaces before inline comment
Line 26:14: E261 at least two spaces before inline comment
Line 27:3: E221 multiple spaces before operator
Line 27:14: E261 at least two spaces before inline comment
Line 28:14: E261 at least two spaces before inline comment
Line 36:63: E202 whitespace before ')'
Line 45:1: E302 expected 2 blank lines, found 1
Line 63:24: E201 whitespace after '('
Line 63:52: E202 whitespace before ')'
Line 71:1: E302 expected 2 blank lines, found 1
Line 73:17: E201 whitespace after '('
Line 73:49: E202 whitespace before ')'
Line 76:1: E305 expected 2 blank lines after class or function definition, found 1
Line 76:26: E261 at least two spaces before inline comment
Line 77:6: E221 multiple spaces before operator
Line 78:5: E221 multiple spaces before operator
Line 80:6: E221 multiple spaces before operator
Line 81:10: E221 multiple spaces before operator
Line 83:10: E221 multiple spaces before operator
Line 88:15: E221 multiple spaces before operator
Line 108:80: E501 line too long (81 > 79 characters)
Line 113:80: E501 line too long (88 > 79 characters)
Line 117:7: W292 no newline at end of file


Update claude extension

Description
Added support for the beta 8192 output for sonnet. 3.5 (added as an option on the extension preferences)
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

About the constants, yeah we could do it that way but I think you can only use one beta header at a time with the api, in the end it depends on how Anthrophic implements the longer outputs when it's not in beta anymore or if they add other beta features.

I think the useBetaFeatures preference flag is overkill for this functionality. Removing the flag should simplify both the implementation and usage. Can you update the PR to reflect those changes? So the beta header is always used when Sonnet 3.5 is selected and keep the code for allowing users to select up to 8192 tokens for Sonnet 3.5. I'm indifferent about the constants, that's your call.
I'm assuming this will become part of the standard API soon, so I want to minimize the changes made.
Besides that, thanks for the quick spot and implementation! 🚀 (I also follow @alexalbert__ now)

Works for me! I'll make the changes later today

Hey, it seems like not all references to the preference flag have been removed. Until those are cleaned up I don't think we should merge this PR.

Is this good now? 😅


Does the server check cert-to-name when establishing a TLS connection?

In file tls_listen.xml
<cert-to-name>
    <id>1</id>
    <fingerprint>02:E9:38:1F:F6:8B:62:DE:0A:0B:C5:03:81:A8:03:49:A0:00:7F:8B:F3</fingerprint>
    <map-type xmlns:x509c2n="urn:ietf:params:xml:ns:yang:ietf-x509-cert-to-name">x509c2n:specified</map-type>
    <name>root</name>
</cert-to-name>

Does the server check name root in client username? No matter what I configure the name to, it can establish the TLS connection normally. It looks like the server doesn't check the name.
    VRB("Cert verify CTN: new client username recognized as \"%s\".", session->username);

    if (server_opts.user_verify_clb && !server_opts.user_verify_clb(session)) {
        VRB("Cert verify: user verify callback revoked authorization.");
        X509_STORE_CTX_set_error(x509_ctx, X509_V_ERR_APPLICATION_VERIFICATION);
        return 0;
    }

    return 1;

It looks like server_opts.user_verify_clb is NULL.

It looks like the server doesn't check the name.

What "name"? I do not understand the question at all. In TLS, the client identifies itself by a certificate and the server needs to assign some username for this client. It does so using the cert-to-name process, which is, in short, looking for a matching fingerprint in the configuration and then deriving the username from the certificate according to the configuration. In this case it is just a fixed username no matter what information is in the certificate.


Version Catalogs documentation is outdated regards using values defined in Versions scope

The documentation defined here says we should use libs.versions.yourVersion.get() to retrieve values declared in version catalog's versions block, but in fact this method only works if you add .asProvider() before the get() function.
Incorrect example: libs.versions.compose.get()
Working example: libs.versions.compose.asProvider().get()

Expected Behavior
In a gradle file we should be able to retrieve version values using libs.versions.yourVersion.get()
Current Behavior
In a gradle file when using .get() function in a previously declared version of the version catalog an "unsolved reference" error is thrown, as we can see in the picture below:

Context
I was migrating my app from groovy + buildscript extensions to kotlin-dsl + version catalogs. To declare that a module can use Jetpack compose we must assign the compiler version to be used with compose (which is the same value of the current compose version), and then when i tried to use libs.versions.compose.get()the error occurred.
Steps to Reproduce

Enable version catalogs in your project
Create a version catalog TOML file
Declare a value in the versions block
Try to access the value using libs.versions.yourValue.get()
Notice the unsolved reference error

Your Environment

MacOs Monterey
Gradle Version 7.4.2
Android Gradle Plugin 7.1.2
Android Studio Version Bumblebee 2021.1.1 Patch 2

This is actually almost true. If there's a nested accessor, you need to call asProvider() first. So here you need to call asProvider() because you have a child node to compose I guess.


[0.10] Several BibTex fixes

This PR is still missing a fix for the broken file picker on mac (not able to select any bibtex-related files) and windows (able to change file type to "any"). @Severino could you please check this at least for windows? Might be the same fix for mac.

I added another fix, because adding new items of type inbook was not possible due to broken validation (no fields displayed)

I forgot to add the fix for the files that were not selectable on windows and map. Those are now in the PR #500


REFACTOR, more lighter, now it is too much Program to Interface

Reduce so many interfaces in internal, because of:

It is unnecessary for the internal logic
It reduce runtime performance
Doesn't improve readable

This has been covered on okdownload.


Add missing UI docs

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
Complete output when running fastlane, including the stack trace and command used
The UI docs were removed by https://github.com/fastlane/fastlane/commit/2eb22ef562e5d2fcf088a765d97c0e49f83dd7d5 but not added to docs.fastlane.com.

@revolter Created a PR for this https://github.com/fastlane/docs/pull/447


Golang network filter missing functions

I'm experimenting with the new golang network filter in v1.27.0
and I noticed that native C++ filters are exposed to more functions than what the golang filter is exposing.
for instance, the ConnectionCallback interface: https://github.com/envoyproxy/envoy/blob/f17f1fdd644cc4b32f65579184757e3975843db6/contrib/golang/common/go/api/filter.go#L176-L183
exposes 3 methods,
but from what it looks like the connection callback should have more methods such as requestedServerName() to retreive the SNI of the requested server and so on..
so, am I missing something? or is there a plan to add any missing methods to the connection callback later on?

Yep, the network filter is the early iteration, new features in the TODO list.
Could you please describe your case a bit, and is there other APIs you needed? except the requestedServerName.
cc @antJack

is there a page to track the TODO list you mentioned?

not yet, feel free to create issues, or even PRs, when you need new features.

And for my use case, I wanted to use requestedServerName to examine some custom behaviour that will mint a self signed certificate in the bacground based on the SNI on the request,

Do you want to generate a self signed certificate, on demand, by using Golang, based on the SNI?
if yes, I think it might be too late?

I was planning to use the generated self signed certificate on subsequent requests with that SNI using xDS,
so it could be late but not that late.

@or-adar oh, I see. then, how does first SSL/TLS connection handshake? is there already a useable bootstrap certificate?

@doujiang24 No, the first connection is just forwarded as-is and the TLS handshake won't be handled by Envoy in that case.
but coming to think of it, maybe it won't be really great for what I'll be trying to acheive.
do you think I should use a different approach? like creating a custom handshaker that'll generate certificates for each domain that a user has requested at runtime? (if custom_hanshaker is the case, do you know of any good examples that use it?)

yep, custom_hanshaker could be a workable choice, but it's a bit overhead if we just need generate certificates at runtime.
it will take over the SSL session resumption and SSL ticket, and etc.
actually, I had implemented a custom_handshaker that support using Golang to hand generate certificate at runtime, but, not opensource yet. I might create a PR for it when I have enough free time.
But, I do think a better choice, might be a custom_certificate_specifier, similar to custom_handshaker, so that the golang extension that implemented the custom_certificate_specifier do not need to care about SSL session/ticket.

it could produce a bit overhead but it can be tolerated, as theoretically I will only need to generate certificates only once (per domain), and reuse existing certificates for consequent requests with that same domain.
I haven't heard about custom_certificate_specifier or got to find anything about it in the docs/codebase,
do you have any reference?
Also, does creating custom_certificate_specifier/custom_handshaker requires forking the project?
or you can write the custom logic in a separate codebase and reference it with a compiled binary that you mount to a container running envoy (similar to how it could be acheived with wasm/golang network filter)?
I just never seen any examples of custom_handshaker so if you have any known exaples or references I would be happy if you could share them!

just to be clear, the golang extension for custom_certificate_specifier/custom_handshaker is something that is planned?

yep, it's on our TODO list, but not a high priority. anyway, I'll create a PR for the draft version.

And the PR you are talking about showcases a golang extension that allows extending custom_certificate_specifier/custom_handshaker?

yep, that's it, but it might not be a fully workable stage, might be a draft version at first.

oh I see, thank you for clarifying!
Looking forward to seeing the progress

Hi @or-adar, @doujiang24
I’m currently researching how to do selective TLS bumping that will affect incoming connections.
i.e hold incoming connection, accepted by Envoy, before TLS handshake is made, generate certificate for them if needed and perform TLS bumping.
I, too, didn’t find a way to do the above. I succeeded in implementing selective TLS bumping but it won’t work with newly incoming connections for which I didn’t create certificate yet (as the certificate creation happens at a later point where the handshake was already done).
@doujiang24 ,
Regarding your response on using custom_handshaker, I would like to avoid code changes in Envoy as much as possible.
Sorry if I’m asking the same question, but is there any plan to provide Go API to do the above without the need to change Envoy’s code base? (A POC/draft will be good to look at as well)
Thanks!

is there any plan to provide Go API to do the above without the need to change Envoy’s code base?

yep, just like the current Golang filter extension: https://www.envoyproxy.io/docs/envoy/latest/configuration/http/http_filters/golang_filter

Hello @or-adar and @BenAgai , here is the draft PR: https://github.com/envoyproxy/envoy/pull/29383
But I may not have enough time to finish it in short, welcome to continue it, or other better ways, if you are instersted it.

Thanks @doujiang24  for the reply!
I hope #29383 will be prioritized as I believe the above feature is desired by more people.
Have a good week!


Hide games you've already rated (Smart/Classic filter)

Create a new feed-like API that returns {id,modified} pairs for all games you've left ratings on.
Also consider adding some compatible way to mark a game as "I can't rate this", "Ignore", or other states we decide to add, it might be useful. May or may not have something to do with List Making #961.
The "games I've rated" list can be used to omit games from the Smart filter display client side (or alternatively gray them out/set low opacity on items you've done).
The switch to 24 responses should still give you plenty of other games even if we do decide to not show games.

Some folks like to find their own game on the lists. So it may be more desirable to gray out/lower opacity than to outright remove them. An overlay that suggests why they're hidden could be useful, say a checkmark :white_check_mark: for games you've rated, and something else for your game.

I see that this was for the LD52 milestone but just want to bump this and say that it would be great if previously played games could be filtered out or clearly marked and that it's not only smart/classic filter, at least danger but also zero would greatly benefit from this so probably valid for all sort modes


Progress Bars CSS styling not being applied or updated (Calorie Tracker)

Styling for the progress bars are not being updated. Going to default Django progress bar styling

Clear the browser cache or use a versioning strategy for static assets to force updates.


⚠ Guya.moe Proxied has degraded performance

In 2ba90ef, Guya.moe Proxied (https://ice.guya.moe/) experienced degraded performance:

HTTP code: 200
Response time: 12183 ms

Resolved: Guya.moe Proxied performance has improved in b8c2aef after 13 minutes.