Fix translation links in README and instructions

Some links were broken when README.md was capitalized, and some links were inconsistent/missing.

I think that, for simplicity's sake, as more translations are added, it would be beneficial to require the contributor to update the links in all README files. I don't think it'd be a bad thing if one "Translations" section were simply copy/pasted across all the files, even if it would lead to each file linking to itself once.
Also, I wanted to note the order in which the languages are listed. I've simply listed them in alphabetical order according to the romanization of each language (Chinese, English, French, Gujarati).
Example
## Translations
- [简体中文](docs/README.zh-CN.md)
- [English](/README.md)
- [Français](docs/README.fr-FR.md)
- [ગુજરાતી](docs/README.gu-GU.md)

Yes, I agree with that @piccoloser . I'll be merging this change.
Thanks !

Hey @piccoloser , The changes in the PR have broken links due to uppercase "Readme" in the links.

Oof, that's ironic. Let me take a look and I'll fix it!

@ParthJadhav I figured out what the issue was. I renamed all the README files with full capitalization, but for some reason that wasn't included in my commits. The issue would be fixed by just capitalizing the files.
Not sure why that wasn't included in the commits, I'll have to check on my environment.

@ParthJadhav Sorry for the double-tag. Should be fixed now.

Awesome @piccoloser . But there are still some broken links.
Chinese Readme -> Gujrati Readme (You missed a forward slash)
French Readme -> All other Readme
Gujarati Readme -> All other Readme

Okay, I'll take another look.

Okay, I think that fixes it.

Yes ! Thanks for it. I can merge it now.


[BEAM-6362] DO NOT MERGE remove --info flag in gradle invocations

This should significantly reduce log size, while still logging INFO and
above for failed tests.

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
 Should add the flag specifically for Go?

LGTM

@udim Not that I'm aware of. The Go Gradle we use (formerly called the blindpirate one) does seem to support better test parsing with Go1.10+. We may simply need to update to the latest version of the plugin: https://github.com/gogradle/gogradle#snapshot
Presumably it does that in the appropriate format for general gradle usage.
A cursory search doesn't seem to indicate much. The --info flag does print out the coverage report to the output, but I don't think it would do anything to affect debugability.

Thank you all!


view/services/db: QueryablePersistence

Currently the db service provides only two interfaces, Persistence and VersionedPersistence, with very limited query capabilities.
This issue asks to introduce a new interface called QueryablePersistence that offer rich queries.

this will be addressed when the requirements will come


Unit test Failing for test_score_analyses_runs_successfully_with_visualization_on

Describe the bug

Unit test failing.. during tear down.

To Reproduce

on Windows platform
Create new conda envionrment using the environment.yml
Run quick test: test should pass.
Run pytest full
Unit test fails at clean up process for C:\git\axondeepseg\test\testing\test_segmentation_scoring.py:77:

See detailed stack trace in log file attached. CMDConsoleLog.log
Reproduced in 2 Windows10 PCs, with or without nvidia GPU.
Reproduced in experimental GitHub Action Pytest

Platform (please complete the following information):

OS: Windows [Version 10.0.19042.867]
AxonDeepSeg version number: Master branch as of 2021-03-20T232341EST for 837562b4c1e343dcc80a7bb7a12d44e299977481

Updated reference to the experimental GitHub Action Unit Testing branch where this failed:

Failures line from PyTest
Exit code 1 from PyTest

Confirmed it is a Windows ONLY issue. Similar tests past in both Ubuntu and macOS.

Root cause identified and resolved as an indent is needed at Line79 for Windows to properly close the "with" clause.
This will be bundled with the upcoming TravisCI fix, which require addressing this to pass.

Thanks for finding this and resolving it @dyt811 ! Yeah, I recall encountering some really annoying permissions errors while trying to create/delete temp files in python for this project earlier on and thought I had resolved them all, but we've not touched on Windows for a long time now.


Add Post Secondary Course to domain of several properties

For the PESC transcript we find the need for these properties when describing Postsecondary Courses
P000057 Has Course Credit Units and P000058 Credit Value because Post-secondary courses carry credit too.
P001298 Course Academic Grade Scale Code has PS Section in domain, but would be useful at Course level too for when Sections aren't specified.
P001517 Has School Courses for the Exchange of Data Course Code, SCED is used in HE too
P001518 Has School Courses for the Exchange of Data Course Subject Area, used in HE too
P001312 Has Course Level Type, currently domain includes Course and PS Section, but not PS Course.
P000060  Has Course Grade Point Average Applicability, currently currently domain includes Course and PS Section, but not PS Course.
P000066 Course Subject Abbreviation, durrently domain incl Course not PS Course

@philbarker Would making :PostsecondayCourse a subclass of :Course resolve this issue?  Would doing so create any new issues with how you use :PostsecondaryCourse?

@aemnathanclinton that sounds right


File Organization

Hello!
I'd like to propose we create resbazaz-in-a-box style documentation for the entire org to increase transparency. I'm proposing a document structure like the one below where the conference, weekly meetups, and executive documentation would all have a distinct location.
.
├──  README
├──  Conference/
├── Timeline template
├── Agenda template
├── subcommittees/
│      ├── Administration/
|      ├── Survey templates
|       │      ├──Registration
|       │      ├── Evaluation
|       │     └── Speaker interest
│      ├── Marketing/
|       |      ├── Socials/
|       |    ├── How-to access resbaz socials
|       │   └── Guide to social media marketing
|      ├──  Guide to booking venue
|      └──  Guide to updating website annually at conference time
│      └── Operations/
|       |       ├──  Sponsors/
|       |        │     ├── Sponsorship template
|       |       ├──  Speakers/
|       |        │     ├── Invitation template
|       |       ├── Volunteers/
|      |         │   └──Invitation template
├──  Weekly Events/
├── C&C/
|        └── Logo
└──HH/
|        └── Logo
├──  Marketing/
├──logos/
├──General flier template
├──Branding guidelines
├──Socials/
|        └── Meetup
├──  Resources & Templates/
└── Executive/
└── Governing docs/
Please let me know your thoughts (and let me know if this issue is better placed in https://github.com/resbazaz/cookbook)

LOL ok i tried twice and I can't figure out how to make it look pretty??? please help!!!

:D you can put it in a code block for monospacing. Still would need to obsess over the right number of spaces, though.

@jcoliver of course! Thank you!
Well I can't count but I can draft documentation. I'll leave this for discussion!

I think this needs to be updated to separate conference organizational documents from community organizational documents.


VolumeWeightedAveragePrice returningobject type of  <ta.volume.VolumeWeightedAveragePrice at 0x19ffb5afb50>

Hi ,
After calling VolumeWeightedAveragePrice  function , it is returning below object type
<ta.volume.VolumeWeightedAveragePrice at 0x19ffb5afb50>
Could you please help on how to read values out of it

VolumeWeightedAveragePrice is a class, not a function. You have to call  volume_weighted_average_price() like
 VolumeWeightedAveragePrice(...).volume_weighted_average_price()


@connection should accept partial sort key

Feature Request / Bug
Consider the following example:
type User
@model
{
  id: ID!
  userSchoolId: ID!
  userTags: [UserTags]
    @connection(
      keyName: "UserTagsBySchoolByUserByCategoryByTag"
      fields: ["userSchoolId", "id"]
    )
}

type UserTags
  @model
  @key(fields: ["categoryId", "tagId", "userId"])
  @key(
    name: "UserTagsBySchoolByUserByCategoryByTag"
    fields: ["schoolId", "userId", "categoryId", "tagId"]
  ) {
    categoryId: ID!
    tagId: ID!
    schoolId: ID!
    userId: ID!
}

UserTags creates a GSI with a combined sort key.  This allows for multiple access patterns.  Get all UserTags by School.  Get all UserTags for User.  Get all UserTags for User by Category.
But if you run amplify api gql-compile on the example above, it would produce the following error:
## InvalidDirectiveError: Invalid @connection directive  usersTags. fields does not accept partial sort key
For the User to get all their tags though, all the sort keys aren't necessary.  A User only needs a schoolId and their id to get their tags.   The current workaround for this situation is:
type User
@model
{
  id: ID!
  userSchoolId: ID!
   userTags: [UserTags]
    @connection(
      keyName: "UserTagsBySchoolByUserByCategoryByTag"
      fields: ["userSchoolId", "id", "id", "id"]
    )
}

type UserTags
  @model
  @key(fields: ["categoryId", "tagId", "userId"])
  @key(
    name: "UserTagsBySchoolByUserByCategoryByTag"
    fields: ["schoolId", "userId", "categoryId", "tagId"]
  ) {
  categoryId: ID!
    tagId: ID!
    schoolId: ID!
    userId: ID!
}

Then modify User.userTags.req.vtl and remove user.id being passed as categoryId and tagId in the DB query

@duwerq we don't support partial sort keys yet, closing in favor of #5468

@ammarkarachi Sorry, I think there are two separate issues here. Issue mentioned in #5468 is that when using @connection, all fields must be non-nullable. What this means is that using @duwerq 's previous example, because User use @connection with fields ["userSchoolId", "id"], both userSchoolId and id need to be non-nullable or use a ! at the end. This is a bug and it is addressed by this pending pull request #5153 , because a user might not be affiliated with any school and forcing to assign a school id to a user is buggy.
type User
@model
{
  id: ID!
  userSchoolId: ID!
  userTags: [UserTags]
    @connection(
      keyName: "UserTagsBySchoolByUserByCategoryByTag"
      fields: ["userSchoolId", "id"]
    )
}

However, the issue with "fields does not accept partial sort key" is another bug and is totally different. User and UserTags is a 1 to many relationships. Therefore, User is creating a connection with UserTags with fields id and userSchoolId. It always provide partial sort key only because it is a 1 to many relationships. User doesn't know  categoryId or tagId. Given the current situation, we can't query UserTags within User, but we can query UserTags with fields id and userSchoolId (see the bug here?) A workaround is to separate the two queries.
I request to reopen this issue, as the two are totally different.


Question

Hello,
Great library.
I want to create a backend in .net core for manage push notifications.
I want use your library,

Is it safe to save the .json (serviceAccountKey) in the root of the project?

how would you create an example like this

{
	"to":"/topics/General",
    "data": {
        "message" : "my_custom_value",
        "other_key" : true,
        "body":"test"
     },
     "priority": "high"
}

I think it would be good in the documentation more examples of use.

Thanks a lot!!!

The ServiceAccountKey is your Private Key used to sign requests to Firebase. As such it used to authorize your requests to Firebase and you should never leak it into public.

What do you want to achieve? An example for sending data messages is here: https://github.com/bytefish/FcmSharp/tree/master/FcmSharp/Examples. An example for sending a Notification is given here: https://github.com/bytefish/FcmSharp/blob/master/FcmSharp/FcmSharp.Console/Program.cs.

Yes, there is indeed an ongoing issue at https://github.com/bytefish/FcmSharp/issues/34, which is used for discussion and should lead to more examples. The library itself is a 1:1 translation of the Firebase Cloud Messaging protocol, which has a good documentation at: https://firebase.google.com/docs/reference/fcm/rest/v1/projects.messages. So you probably also want to look there for additional explanations and examples.


Beta+dev

Merged locally dev into beta+dev
Only change is lifesteal description.
Let's not merge until everyone has checked nothing is missing.

looks ok to me but these are @RealBazou terms.


MQB folder missing

Hi,
When I try to update my unit from SD card I don't have MQB folder on the radio. What can I do?
MHI2_ER_VWG13_P4512_2
5NA035022C
H29.319.29_HIGH2GP_EU
V 4.12st - on radio mqbcoding/Console it says:
Script not found: /eso/hmi/engdefs/scripts/mqb/update_toolbox.sh
Regards!

What you mean with MQB folder? What error are you getting?

Hi, I do have the same issue. In green menu, the point mqbcoding is there. however if i click on it, no scripts etc are there. Only the message above the line with  the warning.
Below is only the update button.
the rest is empty.
If i click on update, its the same message above
Script not found: /eso/hmi/engdefs/scripts/mqb/update_toolbox.sh
I do have a MHI2_ER_AU57x_K3663_1
i tried to reinstall two times, no change
Best regards Tom

Ok , got the problem. i was using a bigger SD card wich could not be formated into FAT32. With a different SD, FAT32 and reinstall everything works


prevent closing os.Stderr (used in verbose test logging)

Overview
When running tests with the -v verbose flag, logOutput was getting set to os.Stderr. In some test that used test.command.Reopen(), this was causing os.Stderr (or /dev/stderr) to be closed, which caused problems later.
This PR just prevents closing logOutput when it is os.Stderr
Pull request checklist

[x] I have read the contributing guide.
[x] I have agreed to the Contributor License Agreement.
[x] I have updated the documentation.
[x] I have resolved any merge conflicts.
[ ] I have included tests that cover my changes.
[x] All new and existing tests pass.

Code review checklist
This is the checklist that the reviewer will follow while reviewing your pull request. You do not need to do anything with this checklist, but be aware of what the reviewer will be looking for.

[ ] Ensure that any changes to external docs have been included in this pull request.
[ ] If the changes require that minor/major versions need to be updated, tag the PR appropriately.
[ ] Ensure the new code is properly commented and follows Idiomatic Go.
[ ] Check that tests have been written and that they cover the new functionality.
[ ] Run tests and ensure they pass.
[ ] Build and run the code, performing any applicable integration testing.

@jaffee I updated this to include os.Stdout


Fixed #27273 -- Added a construct_change_message() admin utility function.

https://code.djangoproject.com/ticket/27273

@claudep, okay for you?

No problem for me.


Plugin doesnt work with WP 5.2.2 and WC 3.6.5

When update?

I tested the versions WP 5.2.2 and WC 3.6.5 and work and it works fine.
What's the problem?


Debug Info on output, when enter an empty "new" password

Output of restic version
restic 0.12.0 compiled with go1.15.8 on windows/386
How did you run restic exactly?
PS T:> restic -r xxx init
enter password for new repository:         (entering here just return)
an empty password is not a password
main.ReadPassword
/restic/cmd/restic/global.go:367
main.ReadPasswordTwice
/restic/cmd/restic/global.go:376
main.runInit
/restic/cmd/restic/cmd_init.go:61
main.glob..func12
/restic/cmd/restic/cmd_init.go:25
github.com/spf13/cobra.(*Command).execute
/home/build/go/pkg/mod/github.com/spf13/cobra@v0.0.5/command.go:826
github.com/spf13/cobra.(*Command).ExecuteC
/home/build/go/pkg/mod/github.com/spf13/cobra@v0.0.5/command.go:914
github.com/spf13/cobra.(*Command).Execute
/home/build/go/pkg/mod/github.com/spf13/cobra@v0.0.5/command.go:864
main.main
/restic/cmd/restic/main.go:98
runtime.main
/usr/local/go/src/runtime/proc.go:204
runtime.goexit
/usr/local/go/src/runtime/asm_386.s:1333
PS T:>
The same behaviour with command:
PS T:> restic -r restic-st key add
enter password for repository:
repository 3a1ebf59 opened successfully, password is correct
enter new password:
an empty password is not a password
main.ReadPassword
/restic/cmd/restic/global.go:367
main.ReadPasswordTwice
/restic/cmd/restic/global.go:376
main.getNewPassword

What backend/server/service did you use to store the repository?
Expected behavior
no text after "an empty password is not a password"
I guess, that is some debug information, not designed for final release

Actual behavior

Steps to reproduce the behavior
Just press "Return" when you asked for new password

Do you have any idea what may have caused this?
Do you have an idea how to solve the issue?
Did restic help you today? Did it make you happy in any way?
It' always a pleasure to work with restic!

I checked with the 64-version, same behaviour

restic 0.12.0 compiled with go1.15.8 on windows/amd64

But self-compiled versions from yesterdays master (standard and debug) do not show that.

restic 0.12.0.Stephan compiled with go1.16.4 on windows/amd64

This has been fixed in #3283


Use UTF-8 on Windows 10 Version 1903, fix #1195

Allows Ninja to use descriptions, filenames and environment variables with characters outside of the ANSI codepage on Windows. Build manifests are now UTF-8 by default (this change needs to be emphasized in the release notes).
WriteConsoleOutput doesn't support UTF-8, but it's deprecated on newer Windows 10 versions anyway (or as Microsoft likes to put it: "no longer a part of our ecosystem roadmap"). We'll use the VT100 sequence just as we do on Linux and macOS.
https://docs.microsoft.com/en-us/windows/uwp/design/globalizing/use-utf8-code-page
https://docs.microsoft.com/en-us/windows/console/writeconsoleoutput
https://docs.microsoft.com/de-de/windows/console/console-virtual-terminal-sequences
Fixes #1195 (I hope).

@jhasse I'm happy to see Ninja supporting Unicode, but I want to point out consequences to this change that should be taken into account:
This is a breaking change for ninja build files. Since Ninja feeds the raw bytes from those files into -A Win32 functions, this means that Ninja build files will now have to be encoded as utf-8, not ANSI (which is good, but a breaking change). In particular, cmake will have to be updated to not generate Ninja build files as ANSI.
There might also be repercussions with the "include prefix" feature. Ninja does binary comparison of the string coming from its build file and that coming from the cl.exe process output. Previously the first of these strings was ANSI and now it will be UTF-8, so this might cause mismatches depending on the encoding that cl.exe uses.

Indeed, thanks for pointing that out.
Note that this will only result in issues when there are non-ASCII characters in the build manifest. Something that would have resulted in problems on Windows anyway so I doubt that many people relied on that. It will still be the first point in the release notes ;)

I've opened CMake Issue 21866 for this, thanks.

#1918 proposes an additional tool to help generators determine the correct encoding for build.ninja files.


NaN timespan has shown

solve this issue

when we press next button for some instant is has shown this NaN:NaN value

Could you please assign to me and I have started to work on it

I have fixed this issue in my recent PR #58.
I removed the bottom time lapse which now looks like this -

And now it doesn't show the NaN value.


BIWI

The README instruction call for the BIWI data set.
I have requested the data set but was unable to obtain it.
Is there a work around this problem? Is there an alternative data set without having to create my own.
Can the demo be run without it?

The README instruction call for the BIWI data set. I have requested the data set but was unable to obtain it. Is there a work around this problem? Is there an alternative data set without having to create my own. Can the demo be run without it?

Hello, you may consider downloading the VOCA dataset and trying the code - it's available on their official website.


[Documentation] Broken link to Bert Fine-Tuning examples page

In the documentation here : https://github.com/dmlc/gluon-nlp/blob/master/docs/examples/sentence_embedding/bert.md in the "Conclusion" points to this broken link ( https://github.com/dmlc/gluon-nlp/blob/master/docs/model_zoo/bert/index.rst) for referencing other fine tuning scripts.
This probably needs to reference this page: - https://gluon-nlp.mxnet.io/master/model_zoo/bert/index.html

@animohan thanks for raising this issue. This markdown is used to render the notebook that you can find here: http://gluon-nlp.mxnet.io/master/examples/sentence_embedding/bert.html. That's the link we originally intended for it to work.
That said, I understand that It indeed might be a bit confusing since people may read the markdown directly. I will keep this issue open for collection suggestions on how to avoid this confusion

@szha what about always reference to the github folder (e.g. ../../../scripts/bert), and adding a .htaccess file to our web server so that scripts/bert are redirected to modelzoo/bert? We also need to make sure modelzoo/bert displays correctly (defaults to modelzoo/bert/index.html)

Redirection should work, though my worry is it might be error prone. The links that include 'scripts' are currently manually replaced in the makefile.

The solution to this issue is tracked in https://github.com/dmlc/gluon-nlp/issues/894.


make this project independent. I want to use it independently Net C # interactive plug-in

Make this project independent. I want to use it independently Net C # interactive plug-in. However, the current situation is to install plug-ins on vscode NET Interactive Notebooks needs to rely on 'jupyter', while 'jupyter' plug-ins need to rely on 'python' plug-ins, so they are bound together. I don't need to install Python plug-ins at all. I hate to install anything in Python on the machine because I don't need it. So I want to uninstall all Python plug-ins on vscode without affecting their use.

@DonJayamanne, @rebornix do you have any suggestions?

based on my understanding this extension doesn't need the jupyter extension at all, probably might need the render extension, apart from that i don't see why I've would need to install the jupyter extension.
if there's a dependency on jupyter extension, I'd check why that is there, and if there's no direct user of any jupyter extension api, then i think we can remove any such dependencies

.NET interactive depends on jupyter but it seems the user is reporting python extension installed too or maybe just been prompted to install themDr Diego Colombo PhDOn 17 Oct 2022, at 10:20, Don Jayamanne @.***> wrote:
based on my understanding this extension doesn't need the jupyter extension at all, probably might need the render extension, apart from that i don't see why I've would need to install the jupyter extension.
if there's a dependency on jupyter extension, I'd check why that is there, and if there's no direct user of any jupyter extension api, then i think we can remove any such dependencies
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

we depend on the jupyter extension for ipynb file handling


Revenue Duplicated

Hello,
I have implemented the beta node sdk, but I'm facing a problem while registering a new Revenue.. When I register it, and check it on the User Look-up, I see it duplicated.
The first one, in the Event Type (again, looking at the User Look-Up Table) is called [Amplitude] Revenue (with the properties correct and everything just fine)
The second one, that appears almost immediately after the first one, is with the [Amplitude] Revenue (Unverified). I'm supposing this should not be the behavior expected.
The events work just fine with a very similar code.
My function below (all the parameters where checked beforehand by other function)
async function trackRevenue(user_id, product_id, price, eventProperties, env) {
    require('@amplitude/analytics-node').init('MY_KEY_HERE')

    const event = new Revenue()
        .setProductId(product_id)
        .setPrice(price)
        .setQuantity(1)
        .setRevenueType((price > 0) ? `incoming` : `refund`)
        .setEventProperties(eventProperties)

    try {
        const result = await revenue(event, {
            user_id: user_id
        }).promise

        if (result.code !== 200) {
            console.log(`Error: : ${result.code}
            Received message: ${result.message}
            Created Event Server Side ${JSON.stringify(result.event)}`)
            return ({ error: true, message: result.message })

        }
        console.log(`Everything worked just fine. Success Message: ${result.message}`)
        return ({ error: false, message: result.message })
    } catch (err) {
        console.log(`Error on trackRevenue. Error received: ${err}`)
        return ({ error: true, message: err })
    }

}

PS: I receive a success message from amplitude
Environment

Node Version: 16.14.0
Installation Method NPM

Print:

Thanks for the support!

Hi, @renatotr ,
Thanks for asking here!
I think this is expected behavior, here is the reason of "Revenue (Unverified)":

If a revenue event is unverified, that does not mean it failed verification. It means Amplitude did not attempt to verify it since it came through the HTTP API, Javascript SDK, or because your project did not include any revenue verification keys. If verification is not on, this event will be logged for all revenue events.

If you want to learn more, we have a doc mentioning how to enable the revenue verification.
Thanks!


How to fix WslRegisterDistribution failed with error: 0x800106d9 ?

How to fix WslRegisterDistribution failed with error: 0x800106d9 ?

My windows specification :
Edition	Windows 10 Pro
Version	20H2
Installed on	12/22/2020
OS build	19042.867
Experience	Windows Feature Experience Pack 120.2212.551.0
I have enabled the "Windows Subsystem for Linux" optional feature.
I hava enabled the 'Virtual Machine Platform' optional component
I have installed the wsl kernel
I have set WSL 2 as my default version
I tried Ubuntu20.04 , debain and kali, and then both encountered the same problem
I tried restarting and reinstalling, but the problem is still not solved

Looks like EPT_S_NOT_REGISTERED, could you take a trace?
/logs


RPP Tensor Audio Support - To Decibels

To be reviewed and merged after PR #224 (which adds initial audio support)

Adds support for audio functionality - To Decibels
Adds relevant QA and performance tests

@r-abishek can you check the CI results for this PR?

@r-abishek can you check the CI results for this PR?

@kiritigowda It passes on local, but there seems to be an issue with the CI unable to initiate runs.
Screenshot of test PASS on local Ubuntu 20.04:

@r-abishek can you check the CI results for this PR?

@kiritigowda It passes on local, but there seems to be an issue with the CI unable to initiate runs. Screenshot of test PASS on local Ubuntu 20.04:

@r-abishek can you try the new org CI?

@kiritigowda There are two issues:

This PR needs to verify Audio Tests to run on CI, that's not running yet due to the inability to install libsndfile:

Docker for RHEL8 seems to not build, says cannot reach network:

Other than these, other tests pass.

@r-abishek all test passing.


Look for correct version in list of possible versions

Look for correct version in list of possible versions
Description
Changed logic in minemeld.ft.taxii.py to look for appropriate version in a list of available versions as opposed to comparing first version in list to what we want.
Motivation and Context
fixes #337
How Has This Been Tested?
I have used this change in a custom written extension based on this miner.
Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist

[N/A] I have updated the documentation accordingly.
[N/A] I have read the CONTRIBUTING document.
[N/A] I have added tests to cover my changes if appropriate.
[N/A ] All new and existing tests passed.

I accept all terms of the license

Thanks! Could you state that you accept the Apache license?

I accept all terms of the Apache License version 2.0


Add exclude: param to relativize_paths filter

Implements nanoc/features/issues/41.
Detailed description
This allows specifying a list of excludes in the relativize_paths filter, e.g.
# excludes /foo, /foo/bar, …
# does not exclude /fooooo, /qux/foo, /qux/foo/asdf, …
filter :relativize_paths, type: :html, exclude: '/foo'

# excludes /foo, /foo/asdf, /bar, /bar/asdf, …
# does not exclude /fooooo, /qux/foo, /qux/foo/asdf, /qux/bar, …
filter :relativize_paths, type: :html, exclude: ['/foo', '/bar']

# excludes /foo/bar, /bar, /archive, /blog/archive/something, …
filter :relativize_paths, type: :html, exclude: /ar/

It can be used in different ways:

string vs. regexp: When given a regular expression, the path will not be relativized if the regular expression matches. When given a string, the path will not be relativized when the path starts with one or more components that matches the given string.

single object vs. array: The parameter can be either a single string or regexp, or an array of them.

To do

[x] Exclude only when fully matching a path component
[x] Allow specifying string instead of an array of strings
[x] Allow specifying an (array of) regex instead of an (array of) strings

After merge, before release:

Document

👍


Parallel/Redundant Versions Of Reactions

There are a number of cases where the "same" overall reaction is represented at multiple different levels of resolution in different reactions/series of reactions:

[x] The aconitase reaction (citrate <-> isocitrate) can be carried out in

one step:MAR04456
two steps: MAR04458 (citrate <-> cis-aconitate + H2O) and MAR04589 (cis-aconitate + H2O <-> isocitrate)

[x] The cytosolic isocitrate dehydrogenase (IDH) reaction (isocitrate + NADP+ -> AKG + CO2 + NADPH) can be carried out in

one step: MAR00710
two steps: MAR04585 (isocitrate + NADP+ <-> oxalosuccinate + NADPH) and MAR04111 (oxalosuccinate -> AKG + CO2)

[x] The peroxisomal IDH reaction can be carried out in

one step: MAR00712
two steps: MAR04587 (isocitrate + NADP+ <-> oxalosuccinate + NADPH) and MAR04113 (oxalosuccinate -> AKG + CO2)

[x] The NADP(H)-dependent mitochondrial IDH reaction can be carried out in

one step: MAR03958
two steps: MAR04586 (isocitrate + NADP+ <-> oxalosuccinate + NADPH) and MAR04112 (oxalosuccinate -> AKG + CO2)

[x] The pyruvate dehydrogenase (PDH) reaction (pyruvate + NAD+ -> acetyl-CoA + CO2 + NADH) can be carried out in

one step: MAR04137
three steps: MAR06410 (pyruvate + lipoamide + H+ -> S-acetyldihydrolipoamide + CO2), MAR06412 (S-acetyldihydrolipoamide + CoA -> acetyl-CoA + dihydrolipoamide), and MAR06409 (lipoamide + NADH + H+ <-> dihydrolipoamide + NAD+)

[x] The 2-oxoglutarate dehydrogenase (OGDH) reaction (AKG + CoA + NAD+ -> succinyl-CoA + CO2 + NADH) can be carried out in

one step: MAR05297
three steps: MAR06411 (AKG + lipoamide + H+ -> S-succinyldihydrolipoamide + CO2), MAR06414 (S-succinyldihydrolipoamide + CoA -> succinyl-CoA + dihydrolipoamide) and MAR06409 (lipoamide + NADH + H+ <-> dihydrolipoamide + NAD+)
four steps: MAR04209 (AKG + thiamin-PP + H+ -> 3-carboxy-1-hydroxypropyl-ThPP + CO2), MAR06413 (3-carboxy-1-hydroxypropyl-ThPP + lipoamide -> S-succinyldihydrolipoamide + thiamin-PP), MAR06414 (S-succinyldihydrolipoamide + CoA -> succinyl-CoA + dihydrolipoamide) and MAR06409 (lipoamide + NADH + H+ <-> dihydrolipoamide + NAD+)

[x] The glycine cleavage system (GCS) reaction (glycine + THF + NAD+ -> 5,10-methylene-THF + CO2 + NH3 + NADH + H+) can be carried out in

one step: MAR03923
three steps: MAR08433 (glycine + lipoamide + H+ -> S-aminomethyldihydrolipoamide + CO2), MAR08434 (S-aminomethyldihydrolipoamide + THF -> 5,10-methylene-THF + NH3 + dihydrolipoamide + H+), and MAR06409 (lipoamide + NADH + H+ <-> dihydrolipoamide + NAD+)
three steps: MAR08435 (glycine + lipoylprotein + H+ -> S-aminomethyldihydrolipoylprotein + CO2), MAR08436 (S-aminomethyldihydrolipoylprotein + THF -> 5,10-methylene-THF + NH3 + dihydrolipoylprotein + H+), and MAR08437 (lipoylprotein + NADH + H+ <-> dihydrolipoylprotein + NAD+)

The existence of these parallel/duplicate/redundant representations of the "same" overall reaction often causes problems for me (I'm working on a new method for incorporating expression data into a generic GEM to create context-specific GEMs), so I usually block or remove the "lower-resolution" versions of these reactions. I don't know enough about the rationale/criteria/thought process behind the curation of Human1 to know if the "lower resolution" reactions

MAR04456, MAR00710, MAR00712, MAR03958, MAR04137, MAR05297, MAR06411, MAR06414, MAR03923

should be removed, if the "higher resolution" reactions

MAR04458, MAR04459, MAR04585, MAR04111, MAR04587, MAR04113, MAR04586, MAR04112, MAR06410, MAR06412, MAR06409, MAR06411, MAR06414, MAR04209, MAR06413, MAR08433, MAR08434, MAR08435, MAR08436, MAR08437

should be removed, or if it's fine to keep all of them.

[x] However, I will say that it seems like MAR08435, MAR08436, and MAR08437 should be removed, because:

The only difference between (MAR08433, MAR08434, MAR06409) and (MAR08435, MAR08436, MAR08437) is that the first series uses lipoamide (MAM02393m) and dihydrolipoamide (MAM02393m) while the second uses lipoylprotein (MAM02400m) and dihydrolipoylprotein (MAM01703m)
MAR08435 and MAR08437 are the only two reactions that the "lipoylprotein" and "dihydrolipoylprotein" metabolites appear in
The "high resolution" versions of other comparable dehydrogenase reactions (e.g. MAR06410 and MAR06412 for PDH; MAR04209, MAR06413, and MAR06414 for OGDH) use (dihydro)lipoamide and not (dihydro)lipoylprotein

@Devlin-Moyer thanks for reporting this, we will investigate this and reach you back later

reformatted cases of reaction sets with different resolution as check box for downstream processing

thank you very much @Devlin-Moyer! You are absolutely correct about the co-existence of low and high resolution reactions for pyruvate dehydrogenase, 2-oxoglutarate dehydrogenase, as well as glycine cleavage system. And this should be fixed.

If keeping the high resolution reactions for these complexes, i.e. splitting one to several sub-reactions, maybe their GPRs should also be reduced to specific component. @Devlin-Moyer what do you think?

yea I think that would make sense, but I don't already know exactly which genes should catalyze each step. Also I'm not sure where we'd put structural components, like PDHX (ENSG00000110435; also known as "E3 binding protein"), which doesn't directly participate in catalysis of the PDH reaction, but still plays an important role in enabling high fluxes by keeping DLD near PDHB (ENSG00000168291). Ofc this would probably be pretty easy for the glycine cleavage system, since we know those genes are all enzymes that directly catalyze a specific step of the process and none are structural because they're all free-floating

The 2-oxoglutarate dehydrogenase (OGDH) reaction (AKG + CoA + NAD+ -> succinyl-CoA + CO2 + NADH) can be carried out in
one step: MAR05297
three steps: MAR06411 (AKG + lipoamide + H+ -> S-succinyldihydrolipoamide + CO2), MAR06414 (S-succinyldihydrolipoamide + CoA -> succinyl-CoA + dihydrolipoamide) and MAR06409 (lipoamide + NADH + H+ <-> dihydrolipoamide + NAD+)
four steps: MAR04209 (AKG + thiamin-PP + H+ -> 3-carboxy-1-hydroxypropyl-ThPP + CO2), MAR06413 (3-carboxy-1-hydroxypropyl-ThPP + lipoamide -> S-succinyldihydrolipoamide + thiamin-PP), MAR06414 (S-succinyldihydrolipoamide + CoA -> succinyl-CoA + dihydrolipoamide) and MAR06409 (lipoamide + NADH + H+ <-> dihydrolipoamide + NAD+)

Regarding the two versions of high resolution reaction-sets for 2-oxoglutarate dehydrogenase complex: (MAR06411 + MAR06414 + MAR06409) and (MAR04209 + MAR06413 + MAR06414 + MAR06409). The only difference is that MAR06411 from the former is splitted into MAR04209 and MAR06413 from the latter, by introducing two metabolites: 3-carboxy-1-hydroxypropyl-ThPP and thiamin-PP. While 3-carboxy-1-hydroxypropyl-ThPP only occurs between these two reactions, thiamin-PP actually functions as cofactor in this step, according to this paper.

Likewise the case of MAR06409 (#529), some other sub-reactions should be modified/shrinked to more specific component in GPRs:
MAR06412: DLAT (ENSG00000150768)
MAR08433: GLDC (ENSG00000178445)
MAR08434: AMT (ENSG00000145020)

This might just be an annoying semantic point, but it feels relevant to point out that lipoamide is also a cofactor here; they're both bound to a subunit of the enzyme complex and aren't produced or consumed in the overall reaction. Is the point you're making that thiamin-PP goes in as thiamin-PP and comes out as thiamin-PP (i.e. exactly the same metabolite), while lipoamide goes in as lipoamide and then comes out as dihydrolipoamide, which needs DLD to turn it back into lipoamide while also reducing NAD+?
While normal FBA would just have the thiamin-PP cycle between MAR04209 and MAR06413 without any meaningful impact on the rest of the network, I happened to come across this paper earlier this week, which introduces limed-FBA, in which case the thiamin-PP would actually do something. The idea with limed-FBA is to stop cycles from happening where no metabolites come in or out, and they accomplish this by adding a "dilution" reaction for each metabolite whose flux is constrained to be a small fraction of the sum of the absolute values of all the other fluxes involving that metabolite. So the argument for keeping MAR04209 and MAR06413 is that someone might want to use Human1 with something like limed-FBA to account for the fact that real cells cannot perfectly recycle their thiamin-PP indefinitely and occasionally need to replace it with new thiamine.
Also, now feels like a good time to mention MAR08746: pyruvate[c] + thiamin-PP[c] + H+[c] -> 2-(alpha-hydroxyethyl)thiamine-diphosphate[c], with a GPR of (PDHA1 and PDHB) or (PDHA2 and PDHB). It was probably supposed to be a "higher resolution" version of the PDH reaction, like MAR04209 is for OGDH, but no other reactions involve 2-(alpha-hydroxyethyl)thiamine-diphosphate (MAM00561c), and it's in the wrong compartment (cytosol not mitochondria). Note how the top left of figure 1 from this paper and figure 2 from this paper show exactly the same structure as the page for MAM00561c. So anyway if you remove MAR04209 because of the way it involves thiamine, you should probably also remove MAR08746 and MAM00561c, and if you keep MAR04209, you might also want to fix MAR08746 by moving it to the mitochondrial compartment (which would also involve changing MAM00561c into MAM00561m) and adding another reaction to convert mitochondrial 2-(alpha-hydroxyethyl)thiamine-diphosphate and lipoamide into S-acetyldihydrolipoamide and thiamin-PP, i.e. MAM00561m + MAM02393m -> MAM02869m + MAM02984m

MAR06412: DLAT (ENSG00000150768)
MAR08433: GLDC (ENSG00000178445) and GCSH (ENSG00000140905)
MAR08434: AMT (ENSG00000145020)

The suggested GPRs for MAR06412 and MAR08433 look fine to me, but I would also add GCSH (ENSG00000140905) to the GPR of MAR08434, because diagrams of the GCS (e.g. figure 1 of this paper) always show the lipoamide bound to GCSH in every step

I would also add GCSH (ENSG00000140905) to the GPR of MAR08434, because diagrams of the GCS (e.g. figure 1 of this paper) always show the lipoamide bound to GCSH in every step

nice citation - GPRs updated in 20961d5742109dace2d0a489e2801984e040bec5, please re-review #538

if you remove MAR04209 because of the way it involves thiamine, you should probably also remove MAR08746 and MAM00561c, and if you keep MAR04209, you might also want to fix MAR08746 by moving it to the mitochondrial compartment (which would also involve changing MAM00561c into MAM00561m) and adding another reaction to convert mitochondrial 2-(alpha-hydroxyethyl)thiamine-diphosphate and lipoamide into S-acetyldihydrolipoamide and thiamin-PP, i.e. MAM00561m + MAM02393m -> MAM02869m + MAM02984m

Again, any modification to HumanGEM should be evidence-based. To keep the super high resolution version by involving thiamine-PP to krebs cycle reactions, can you please provide more references that supporting this claim.

This paper detected2-hydroxyethyl-ThPP as an intermediate in the human PDH complex (see third paragraph of Results section and figure 1A). As did this paper; see the second paragraph in the "Kinetic Analysis of the E1 Decarboxylation Reaction under Single-Turnover Conditions by Quench Flow−1H NMR Spectroscopy" subsection.
This paper observed 3-carboxy-1-hydroxypropyl-ThPP as an intermediate in the human OGDH complex; compare the structure in Scheme 1 labeled "C2-(alpha-hydroxy)-gamma-carboxypropylidene-ThPP" to this structure of 3-carboxy-1-hydroxypropyl-ThPP.
I'm not entirely sure what evidence would support the notion that including these thiamin-PP intermediates is necessary; the reason I mentioned limed-FBA earlier was to point out that including them could be useful, and specifically that removing them could make Human-GEM less useful.

Very nice!

This paper detected 2-hydroxyethyl-ThPP as an intermediate in the human PDH complex (see third paragraph of Results section and figure 1A). As did this paper; see the second paragraph in the "Kinetic Analysis of the E1 Decarboxylation Reaction under Single-Turnover Conditions by Quench Flow−1H NMR Spectroscopy" subsection.

these studies explicitly demonstrate that the pyruvate decarboxylation process catalyzed by PDH E1 component is thiamin-dependent.

This paper observed 3-carboxy-1-hydroxypropyl-ThPP as an intermediate in the human OGDH complex; compare the structure in Scheme 1 labeled "C2-(alpha-hydroxy)-gamma-carboxypropylidene-ThPP" to this structure of 3-carboxy-1-hydroxypropyl-ThPP.

Overall, it seems the intermediate enamines involved in both PDH and OGDH processes are not structurally finalized, in contrast to 2-hydroxyethyl-ThPP (MAM00561c) and 3-carboxy-1-hydroxypropyl-ThPP (MAM00765m) in HumanGEM.

I'm not entirely sure what evidence would support the notion that including these thiamin-PP intermediates is necessary; the reason I mentioned limed-FBA earlier was to point out that including them could be useful, and specifically that removing them could make Human-GEM less useful.

Based on these studies, inclusion of thiamin-PP intermediates may help in:

understanding potential interaction to ROS generation, as reported in this JBC paper
more accurate simulation of thiamine metabolism together with this existing subsystem

@Devlin-Moyer just wonder if you have a chance to look into the situation of isocitrate dehydrogenase (ICDH) reactions, after PDH and OGDH?

If you're asking if I think we need to alter the GPRs for any of the IDH reactions to represent which specific subunits catalyze each step, we do not need to make any such changes; IDH1 and IDH2 both function as homodimers that are individually capable of catalyzing both steps in the conversion of citrate to isocitrate or vice versa (source), and you need all three subunits of IDH3 to be expressed (IDHA, IDHB, and IDHG) for full enzymatic activity (source), and the GPRs I proposed in #523 and you implemented in #525 already represent this accurately.

fixed in #541


Add Goth Amino

This change adds the Goth Scene Amino icon.  The original was traced from the official icon found on the Google Play Store page.

I’m not going to add an unofficial (i.e. traced) icon. If you want me to add the Amino logo I’m happy to.

Okay.  The generic Amino logo won't work for my use-case, where I'm creating a button to visit a specific Amino community (Goth Amino).
The official Goth Amino logo appears to be vector art, but I could only find it served as a png.


Unresolved char entity causes STDOUT output to stop

Hi Andrew,
I have an XML file that is not well-formed - it may or may not have a DOCTYPE, but the real problem is that the XML contains a character entity that is not resolved. This currently causes the doctype tool to stop outputting the file contents to STDOUT. In other words, if there is an error, the tool will only pass the content to STDOUT until that point.
The larger issue is that I have a bunch of XML files with character entities but mostly without DOCTYPE declarations that are being linked to a main XML (with DOCTYPE) via XInclude by a system that doesn't appear to think that this is a problem. In order to process the XInclude targets outside this system, I need to first inject them with a DOCTYPE declaration so the character entities can be resolved in a later step.
Many thanks.

Hi Ari,
Thanks for reporting this.
I believe the immediate cause is that the wrapper around the parser I am using (expat) understandably abandons the parse after encountering a well-formedness error -- so it's output ceases as soon as an undeclared entity appears.
I have delved into the lower-level callbacks expat provides as part of investigating some of the other feature requests for this tool, but have not yet discovered a way to preserve the value of the undeclared entity.
It does however provide location information, so a hack might be to simply append the rest of the document as unparsed text from that point. But like I say, a hack...
Andrew


Font weight not respected in webkit

I believe I ran into this with variable fonts. The workaround I've used is to provide font-variation-settings: "wght" apart from font-weight property.
https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_Fonts/Variable_Fonts_Guide#weight

I think this might be a next/font issue :/

FWIW we've had the same problem on other websites (e.g. my personal website https://tobiasbernard.com), and it seems to be specific to WebkitGTK, rather than all of Webkit.

It seems to be fine in gnome web now?

Seems fixed indeed!


Output other than HTML?

Is there currently a way to put out something else than HTML? Or is there at least an easy to hook into @behaviour to write “output filters”?
I'd be very interested in trying to provide some LaTeX printer if it is currently missing.

It is undocumented but yes, there is. Formatters aren't behaviours yet, but I'll turn them into formal behaviours in the future.
Basically, a formatter converts a flat list of tokens into a string. For the HTML formatter, that string is HTML. You can write a formatter that converts the tokens into LaTeX with no problem.
My main problem here is a design one: I'm still not sure of whether there should be a very unified interface to all formaters, or if each formatter should work in a unique way.

Actually, a behaviour might be overkill. I'm planning on formally making the token format part of the public API, and use the metadata for possible extensions, so you should be able to weite your own formatter just by processing these tokens (which are very simple).

Well a binary is an iolist but not vice-versa, so iolists should be the output (especially since you can then just convert it to a binary anyway if necessary, or feed it straight to a network socket or so).
But otherwise unless the system itself needs the formatters and not the other way around, a behaviour itself is not needed, just a good API for accessing the tokens.

Well a binary is an iolist but not vice-versa, so iolists should be the output

The function that outouts a binary just renders the iolist returned by the other function into a binary.

a behaviour itself is not needed, just a good API for accessing the tokens.

A token is just a {atom(), Map.t(), iodata()}. There isn't really much of an API xD

The function that outouts a binary just renders the iolist returned by the other function into a binary.

You don't always want that though as keeping it as an iolist can often be faster, so it is a good base.  :-)


PushConstant gets mistakenly recognized as binding

The following code,  produces the following error

[[vk::binding(0, 0)]] RWTexture2D<float4> output;
[[vk::push_constant]] RaytracingAccelerationStructure tlas;

[shader("raygeneration")]
void rayGenMain()
{}

VUID-VkRayTracingPipelineCreateInfoKHR-layout-07988(ERROR / SPEC): msgNum: -859700687 - Validation Error: [ VUID-VkRayTracingPipelineCreateInfoKHR-layout-07988 ] Object 0: handle = 0x4fac1c0000000032, type = VK_OBJECT_TYPE_SHADER_MODULE; Object 1: handle = 0xa808d50000000033, type = VK_OBJECT_TYPE_PIPELINE_LAYOUT; | MessageID = 0xccc20231 | vkCreateRayTracingPipelinesKHR(): pCreateInfos[0].pStages[0] SPIR-V (VK_SHADER_STAGE_RAYGEN_BIT_KHR) uses descriptor [Set 0, Binding 1, variable "tlas"] (type VK_DESCRIPTOR_TYPE_ACCELERATION_STRUCTURE_KHR) but was not declared in the pipeline layout.
The Vulkan spec states: If a resource variables is declared in a shader, a descriptor slot in layout must match the shader stage (https://vulkan.lunarg.com/doc/view/1.3.296.0/windows/1.3-extensions/vkspec.html#VUID-VkRayTracingPipelineCreateInfoKHR-layout-07988)
    Objects: 2
        [0] 0x4fac1c0000000032, type: 15, name: NULL
        [1] 0xa808d50000000033, type: 17, name: NULL

For some reason the pushconstant is a descriptor binding (set 0, binding 1)

RaytracingAccelerationStructure is an opaque handle and it is not possible to place it as a push constant in SPIRV. Therefore the [vk::push_constant] attribute has no meaning and is ignored by the compiler.

Oh ok but cast from uint64 should work? At least in glsl this worked.

Casting from uint64 to AccelerationStructure is not valid in HLSL and it is not supported in Slang either. D3D does not specify that AccelerationStructure can be constructed from uint64. If we add such support in Slang, it won't be a cross-platform operation either.

I guess I could hack something together with inline spirv?

That may be possible and certainly worth a try.

@fknfilewalker does the WAR that @csyonghe suggest work for you? If so - are you ok with closing this issue for now?

I did try this
__glsl_extension(GL_EXT_ray_tracing)
[ForceInline]
RaytracingAccelerationStructure addressToAccelerationStructure(uint64_t address)
{
    return spirv_asm {
        OpCapability RayTracingKHR;
        OpExtension "SPV_KHR_ray_tracing";
        result:$$RaytracingAccelerationStructure = OpConvertUToAccelerationStructureKHR %address
    };
}

but it did not work. Maybe you see the error otherwise it is fine for now.

Casting from uint64 to AccelerationStructure is not valid in HLSL and it is not supported in Slang either. D3D does not specify that AccelerationStructure can be constructed from uint64. If we add such support in Slang, it won't be a cross-platform operation either.

This is a temporary constraint in HLSL / DXR spec that will likely be addressed eventually, particularly given that they are moving to adopt SPIR-V for their IR. (Still, I can bring this up to Martin, perhaps we could get some correction to the DXR spec about the sizeof an acceleration structure handle...)
I'd advise against us constraining Slang to cater the restrictions of one particular target backend (DXR) over the others. In both OptiX and in Vulkan, constructing an accel from a pointer (uint64) is the standard way to get accel handles into SBT / push constant structs.
For the moment, we might want to consider exposing this via an intrinsic that's made available through Slang's compatibility system, similar to how we handle other features that HLSL (or any other target backend API) might not currently support.

@fknfilewalker does the WAR that @csyonghe suggest work for you? If so - are you ok with closing this issue for now?

For the above, we can pivot discussion to issue #5801.

PR https://github.com/shader-slang/slang/pull/5967 added RaytracingAccelerationStructure(uint64_t).


compatibility check, not listed here, listed in Tasmota templates repository

A while ago I got a smart plug that unfortunately depends on an internet connection, after having a look at the communication with an Android equivalent of Wireshark, it seems it's a Tuya smart plug.
This particular smart plug isn't listed on this github page as compatible, however, it is listed in the Tasmota templates repository -> https://github.com/blakadder/templates/blob/master/_templates/hIhome_WPP-16S1
I haven't tried to flash it yet, but since it's listed as compatible with Tasmota, does that also mean it's compatible with Tuya-convert?
edit:
Well, I tried it. It seems to work :) , though it needs a recalibration and the network scan app doesn't seem to pick up the hostname or mac.
As for Tuya-convert, is it normal that the AP vtrust-flash remains active?

Being compatible with Tasmota doesn't necessarily mean it's compatible with tuya-convert. It needs to be running Tuya firmware on an ESP82xx.
Also note that while a particular model may be listed as compatible on our wiki, they have recently started to change the microcontroller in these devices so it is hard to tell from the model alone. The only way to find out may just be to try it.
To end your session and close the vtrust-flash AP, run stop_flash.sh.

Let me know if you have any more questions and good luck!


String.fsFormat("%A") conflicts with Number.prototype[Symbol.iterator]

Description
Another issue causes infinite loop and browser hanging. :disappointed:
Repro steps
Please provide the steps required to reproduce the problem

Create a page and load core-js as polyfill.
Compile such code and run in page printf "%A" [123]

Expected behavior
The list [123] is print out in console.
Actual behavior
The browser hangs, must to kill the page.
Investigate cause

Related code is here.
The core-js implements some APIs on Number to make v[Symbol.iterator] to be a truely value. (Concretely, the truely value points to entries function in Chrome.)
For k="0",v=123 case, the JSON.stringify callback goes to Array.from(v) branch and returning an array [1,2,3].
For upcoming every number in [1,2,3], the same thing happens.
The JSON.stringify callback goes infinitely.

 Thanks a lot for the detailed information! I've fixed the bug and published the new versions (fable-compiler 0.2.3 & fable-core 0.0.12). Github automatically closed the issue but please reopen if it's not working for you yet.
JSON.stringify was just a cheap trick to emulate printfn "%A" but probably in the future we'll have to implement something more "serious" ;)

Damn! This was caused by my latest hack trying to improve printfn "%A" output in certain scenarios. :(
I agree with Alfonso. JSON.stringify is not the best solution, but it is doing the job for now.
I will make some tests to at least cover these cases discovered by @lijunle.

Aaaaand... now I remember WHY I didn't write those tests in the first place.
Since the JS output is not 100% faithful to F# it's hard to make a test pass in both F# and Js. :(
So, I'm leaving it as is for now. And let's hope we don't shoot ourselves in the foot again next time someone changes this code ;)

I understand that it is hard to test. This is related to an extended dependency modifying this

@fdcastel Ideally all tests should have the same result in F# and compiled JS, but I guess we can allow some differences in cases like this one. I'm already using the MOCHA symbol for similar situations, feel free to do the same with the formatting tests :)

I have verified the fix. Thanks very much!


Scriptable RandomOrder

Changed the RandomOrder transform to use torch.randperm and thus make it Scriptable

This PR also lifts the restriction of RandomOrder to take only PIL.Image as input.
https://pytorch.org/vision/0.8/transforms.html#transforms-on-pil-image-only


Add an ability to check consumer lag

I need to get consumer lag for monitoring purposes. Added a method that returns output similar to bin/kafka-run-class.sh kafka.admin.ConsumerGroupCommand --describe --group <groupId>

Can you please rename client.getOffset to something like getPartitionOffset and also drop all changes to package.json file.

Sure thing, bump.

Did you have any random failures previously? I'm now getting this:
2017-03-06T08:24:58.993Z ERROR group-consumer1 Full rejoin attempt failed: NoKafkaConnectionError [[object Object]]: Connection closed { [NoKafkaConnectionError: Connection closed]
I don't think it's related to my changes.

Try restarting the build. I had some random network related errors with Travis too.

Updated my comment. I can't re-run this build, as I'm not the owner. Maybe you need to increase the sleep 10. Not sure.


Performance difference when model is trained on the same data?

Hello,
I already posted a similar issue where I described that I had problem using a checkpoint (when saved with checkpoint_save_mode=2) on a different machine. So I retrained the model on the same data (with checkpoint_save_mode=1), and the performance of the checkpoints is so different. Example:
sentence = 'I am a cool dude!  What about [ASP]ethereum[ASP]?'
model1 = APCCheckpointManager.get_sentiment_classifier(checkpoint='fast_lsa_t_Crypto_acc_79.74_f1_80.21')
model2 =APCCheckpointManager.get_sentiment_classifier(checkpoint='fast_lsa_t_Crypto_acc_83.62_f1_84.14_state_dict')
model1.infer(sentence) -> i am cool dude! # bitcoin -> 1.0(confidence:Negative) sucks. what about ethereum -> 0.999(confidence:Neutral)?
model2.infer(sentence) -> i am a cool dude! # bitcoin -> 0.998(confidence:Negative) sucks. what about ethereum -> 0.917(confidence:Positive)?
Obviously, model1 is correct, because the sentiment for ethereum should be neutral. Why is this problem happening when I train on the same dataset, and only change checkpoint_save_mode, why is there different predictions?

@yangheng95 , can this be a possible reason:
checkpoint_save_mode =1 will only save the state_dict (so the trained pytorch model on top of bert). But the fine-tuned bert WILL NOT be saved, so this is probably why I get performance decrease.
While checkpoint_save_mode=2 will save the trained pytorch model + the fine-tuned bert (so the full trained model).

OK. This is useful to know. More evaluations you mean more epochs, or to run the training more times?

Run inference on more cases to confirm the issue you found

Ok there are slight performance differencess (+-20%), but I think it just needs more training data to be more conclusive.


🙏复制支持输出html格式

🖋 Description
目前选中数据复制到rte编辑器会变成文本，对比竞品皆会生成表格，如：

🏞 What problem does this feature solve
🧐 Suggest the API

name
type
default
description

-
-
-
-

:tada: This issue has been resolved in version @antv/s2-v1.25.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


FormRecognizer - Update code generation path, FR SDK class name

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Can one of the admins verify this patch?

Automation for azure-sdk-for-python

Automation for azure-sdk-for-js

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-java

Automation for azure-sdk-for-go

@sarangan12 Is the Ruby failure expected? If not, please merge the PR.

NewRP, Ruby and .NET failure is expected. Merging.


Add validation to AnDOGeneration

Currently the validation method of the AnDOGeneration is NotImplemented https://github.com/INT-NIT/AnDO/blob/cef40788bddb2ffc11966b3e60ae6e36a2482853/ando/tools/generator/AnDOGenerator.py#L235
With the structure of the repository getting more stable, we should add a call to the AnDOChecker here.

solved for now by #89. The validation is still not called automatically after the generation of the structure as not all required components are present yet.


Generate properties for templates using TemplatePartAttribute

There is a new attribute ported from WPF.
It will be useful to have optional code generation for "parts" of templates.
As an example, user code:
[TemplatePart("PART_TextPresenter", typeof(TextPresenter))]
public partial class TextBox
{
    protected override void OnAttachedToVisualTree(VisualTreeAttachmentEventArgs e)
    {
        base.OnAttachedToVisualTree(e);

        if (IsFocused)
        {
            TextPresenter?.ShowCaret(); // TextPresenter is generated
        }
    }
}

Additional generated code, adds new property and sets value to it:
#nullable enable
public partial class TextBox
{
    private TextPresenter? TextPresenter { get; set; }

    static TextBox()
    {
         TemplateAppliedEvent.AddClassHandler<TextBox>((control, args) =>
         {
             control.TextPresenter = args.NameScope.Get<TextPresenter>("PART_TextPresenter");
         });
    }
}

Problems:

Only one static ctor can be created for class, if we want to handle TemplateAppliedEvent to fill these properties.
Only one override of OnTemplateApplied method is possible, if we want to use it to fill these properties.

Which means, we most likely will need to modify Avalonia to add yet another way to read namescope after template was applied.

@kekekeks @worldbeater if you have any thoughts.

I think code generation is a good option here and like the ideas. I'll have to read more into what WinUI is doing when I first noticed this (https://github.com/AvaloniaUI/Avalonia/issues/7432#issuecomment-1066183609). They are also generating an enum so there are no magic strings anywhere.
See: https://github.com/microsoft/microsoft-ui-xaml/blob/2a6d2d69c809c141ca7916c9730fd5305487501e/dev/TeachingTip/TeachingTipTemplatePartHelpers.h
There is also a pattern that is used for UWP that removes strings -- it uses nameof() of the fields instead. This wouldn't work for these ideas because someone still has to define the name whether it's a field or a string so this is just FYI:
    [TemplatePart(Name = nameof(ColorPicker.AlphaChannelSlider),          Type = typeof(ColorPickerSlider))]
    [TemplatePart(Name = nameof(ColorPicker.AlphaChannelTextBox),         Type = typeof(TextBox))]
    public partial class ColorPicker : Microsoft.UI.Xaml.Controls.ColorPicker
    {
        private TextBox           AlphaChannelTextBox;
        private ColorPickerSlider AlphaChannelSlider;

        protected override void OnApplyTemplate()
        {
            this.AlphaChannelTextBox = this.GetTemplateChild<TextBox>(nameof(AlphaChannelTextBox));
            this.AlphaChannelSlider = this.GetTemplateChild<ColorPickerSlider>(nameof(AlphaChannelSlider));
        }
    }

https://github.com/CommunityToolkit/WindowsCommunityToolkit/blob/main/Microsoft.Toolkit.Uwp.UI.Controls.Input/ColorPicker/ColorPicker.cs

How exactly would custom OnApplyTemplate logic work with such generated code? Should we generate a partial method to be called from OnApplyTemplate?

How exactly would custom OnApplyTemplate logic work with such generated code? Should we generate a partial method to be called from OnApplyTemplate?

This is the best solution I can think of as well. Just do it like InitializeComponent() in a separate partial class file.

add yet another way to read namescope after template was applied.

Just pass the arguments.
private void InitializeTemplateParts(TemplateAppliedEventArgs e)
{

}

This method can be added to the avalonia base control class, and called just before OnApplyTemplate. As an option. Probably with some "hide from intellisense" attributes. A bit hacky, but transparent for user.
protected virtual void InitializeTemplateParts(TemplateAppliedEventArgs e);

This method can be added to the avalonia base control class, and called just before OnApplyTemplate.

Regarding the implementation details of the new generator, I guess it is worth introducing a new class similar to AvaloniaNameGenerator named e.g. TemplatePartGenerator, then adding a new option to GeneratorOptions that would control if the new generator is enabled, and then invoking the generator in the composition root.
The code that scans the assembly and finds all classes annotated with the TemplatePart attribute can be found here https://github.com/reactivemarbles/ObservableEvents/blob/main/src/ReactiveMarbles.ObservableEvents.SourceGenerator/SyntaxReceiver.cs#L17 Most likely we'll need to implement ISyntaxReceiver that receives all relevant classes with TemplatePart attributes and then generates the code.

I think we have a working solution all ideas together.

@robloo @maxkatz6 @kekekeks to sum up, for the following annotated class:
[TemplatePart("PART_Text", typeof(TextPresenter))]
[TemplatePart("PART_AcceptButton", typeof(Button))]
public partial class TextBox
{
    protected override void OnApplyTemplate(TemplateAppliedEventArgs e)
    {
        base.OnApplyTemplate(e);
        this.InitializeTemplateParts(e);

        // Text.ShowCaret();
        // AcceptButton.IsEnabled = false;
    }
}

We are going to generate the following partial class:
public partial class TextBox
{
    private global::Avalonia.Controls.Presenters.TextPresenter TextPresenter;
    private global::Avalonia.Controls.Button AcceptButton;

    private InitializeTemplateParts(global::Avalonia.Controls.Primitives.TemplateAppliedEventArgs e)
    {
        TextPresenter = e.NameScope.Get<global::Avalonia.Controls.Presenters.TextPresenter>("PART_Text");
        AcceptButton = e.NameScope.Get<global::Avalonia.Controls.Button>("PART_AcceptButton");
    }
}

Is this correct?

There was a lot of further discussion in https://github.com/AvaloniaUI/Avalonia/issues/9093 related to how to define the string constants and then enforce them statically in XAML. I think that discussion isn't fully closed yet and it may influence what is done here. Feel free to jump into that conversation as well. That said, the TemplatePart attribute isn't going to change so however the string name gets into the attribute is irrelevant for you.
Concerning the code generation:

I would consider using .Find instead of .Get to fail gracefully if control authors don't do what they need to do. However, that would require nullable types so maybe isn't worth it. We definitely need good exception handling and build outputs here to say which template part isn't found though.
Technically instead of TextPresenter the control should be named just Text. Not all template parts currently have good names...


Blur exact location (for privacy concerns)

From comment of Florence on deliverable early alert system v1:

In the light of the EU GDPR, it seems that we would not be able to show the precise location with exact geographical coordinates if the species is located in a private domain. It would need to be a bit less precise, so that it is not possible to determine a private address. We could keep the precise location for internal use by public authorities but not for open data. At least, that’s what our consultants advised us to do. Do you plan to/ Is it feasible from an IT perspective to distinguish data flows for public vs internal use?

While tackling this problem, I think we should make a distinction between:

data published on GBIF by organizations out of the LIFE RIPARIAS consortium and/or outside the RIPARIAS project (iNaturalist, Waarnemingen.be, etc.)
data published on GBIF by LIFE RIPARIAS partners (CRs, DEMNA, etc.) and/or within RIPARIAS project (RATO vzw, province East Flanders, etc.)

Possible solutions
I don't think we can make a distinction between public/private domains. Even if we would have shapefiles for it, doing such an intersection means introducing a gigantic burden to the system. We should opt for more general solutions.
General solution
I think we can handle this by a front end general solution:

set a minimum radius of 50m to the visualized points even if the coordinate uncertainty is set to a value below it
round up the geographical coordinates shown on screen to three decimal figures

Notice that the we still provide the link to the correspondent page on iNaturalist/waarnemingen.be where the values are precise. In this way field managers have more control and can also see contact details about the observer if provided.
Notice also that the data stored in db are precise.
Solution for publishing (2)
For data in 2, we will have to updated our guidelines (version 1 already published) on how to handle sensitive data while publishing. The main idea is to inform as much as possible and to give the citizens the right to ask for removing/obscuring some observations. But this is out of scope for this repository.
@niconoe: do you agree on this? Any other (more feasible) idea?

Hey @damianooldoni, that's definitely doable, with the following caveats:

I'd document the process on the website, so users know the precision is limited, and why
as you say, we'll link to the source anyway (but the direct source would be the record at GBIF, not observations.be, iNaturalist, ...)
Choosing a perfect threshold is difficult: 50m may encompass many buildings in Brussel's most populated zones, but wouldn't help privacy much if in the middle of the Royal Domain of Laeken. Anyway, I'd keep it around 50 meters to avoid hindering our users too much

According to the latest exchanges with the legal team, this "feature" is not needed anymore. Closing.

Sorry @niconoe: I would leave it open as long as we don't have the official approval of the final version of the guideline.

@damianooldoni: do you know more clearly what to do for the very first public release in a couple of days?

We do not blur anything.

Brilliant! Can I close this issue for now, or do you prefer me to move it out of the milestone so we can revisit the question later?

This depends on the final outcome of the GDPR discussion, which is still ongoing unfortunately. So, maybe it's better to put it on hold instead of closing. Moving out of the milestone? YES.

We can close this issue. Sorry for letting it open for so long after the publication of the rules.


"Save as shell script" suggests *.bashrc instead of *.sh

Issue Type: Bug
When I go to save a new file as a shell script using File > Save As, not having marked the syntax as shell script yet in the editor; it suggests I name it (filename).bashrc, instead of (filename).sh
VS Code version: Code 1.49.2 (e5e9e69aed6e1984f7499b7af85b3d05f9a6883a, 2020-09-24T16:23:52.277Z)
OS version: Darwin x64 19.6.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz (16 x 2300)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (0.28GB free)

Process Argv
--crash-reporter-id 52447d5c-eb10-4f79-af5d-122dbf24b8fc

Screen Reader
no

VM
0%

Sounds familiar.

I can't reproduce with 1.49.2 and no extensions installed.
Do you have any shell related extensions installed?


Cleanup files and state from old onboarding flow

Expected Behavior
No dead code
Current Behavior
All files and some unused redux state are still in the mobile source

@sallyjyl Let's give the new flow a week or so to bake and if it all looks good let's pick this task up for next sprint


Make tic-tac-toe UI better

For now, the UI for the actual game does not look great. Use CSS styling to make it better looking.
Suggestions:
Add little animations

I have a PR out for this issue. Can you please assign to me?

Sure.
Are you done with your PR?

Yes, but now it looks like there are some conflicts that need resolved. Ill do that ASAP.

@arujjval fixed merge conflicts by combining the latest updates from other PRs with mine. Checked for issues after making the changes and everything works and looks great. PR is ready for approval.

Okay, mention the PR here

Feature/UI-and-functional-updates #6

@arujjval can i also help in the ui? There many things left to add in the website

@arujjval you didn't merge my PR.. but you marked this as complete. Was there an issue with the code?


Shipyard-controller stops sequence when a task returns result=fail

Currently, the shipyard-controller stops the orchestration of a sequence when a task returns status=errored.
Given the following shipyard, the shipyard-controller also stops orchestrating the task sequence when result=fail for any task.
- name: "staging"
  sequences:
  - name: "artifact-delivery"
    tasks:
    - name: "deployment"
      properties:
        deploymentstrategy: "blue_green_service"
    - name: "test"
      properties:
        teststrategy: "performance"
    - name: "evaluation"
    - name: "release"

Task:

Change the behaviour of the shipyard-controller to stop on result=fail

Definition of done:

[ ] A failed test will stop, e.g., the artifact-delivery sequence.

Please consider: https://github.com/keptn/spec/issues/52


Optimize FusedBatchNorm (and fix a bug).

I discovered experimentally that cudnn computations can be performed in place. Therefore there is no need to allocate two temporary tensors in FusedBatchNorm for GPU and data format NHWC. One is enough. This lowers memory consumption, and hence increases the maximum possible batch size.
This might seem risky (because NVIDIA doesn't mention the property), but in fact the current implementation already uses it: by doing forward_input_or_allocate_output in FusedBatchNormOp.
If data format is NCHW, and the input is forwarded, then cudnn would be forced
to do the computation in place (see line 247). This is how I discovered that the whole approach works:
I was trying to see if forwarding the input is a bug or not.
I added several tests to ensure that the change is correct.
While doing this, I discovered that ops_testutil does not properly synchronize at the end.
The reason seems to be the call context_->eigen_gpu_device().synchronize(). Somehow
it does nothing. I think the problem is that Eigen is not compiled with the flag EIGEN_CUDACC.
So I changed it to GPUUtil::Sync(device_.get()).

Can one of the admins verify this patch?

/CC @zheng-xq


Couldn't receive txs failed to fetch raw transaction: code -5 message No such mempool or blockchain transaction. Use gettransaction for wallet transactions. error: failed to get transaction for

The main_index always reports an error and is stuck in the 767437 block.
The error log is as follows.
[2024-03-02T06:23:59Z ERROR ord::index::updater] Couldn't receive txs failed to fetch raw transaction: code -5 message No such mempool or blockchain transaction. Use gettransaction for wallet transactions.
error: failed to get transaction for 113a114c8bf57e8bc317591c458c0546a71805a3b536760f32abb26e68e21ff7
Txhash may be any one in this block.
I can use the bitcoin command line to find the correct raw transaction.

My Bitcoin node has been fully synchronized and txindex is enabled. The following is my Bitcoin node configuration.
[main]
server=1
rpcuser=veryyoung
rpcpassword=
txindex=1
Can you help me solve this problem? Thanks

The getindexinfo command returns
{   "txindex": {     "synced": true,     "best_block_height": 832891   } }
synced is true, and the best_block_height is the current latest block height

This shouldn't happen if txindex is synced.
Can you make sure that you do not have another bitcoin node without txindex that ord is trying to connect to? Also, can you double check the network in main_index is set to mainnet?

Hey @veryyoung does this solve the issue?

@samedcildir @nathanddrake
Hi, thank you for your replies. My node data was downloaded from the Internet and may lack the index of previous data. I am running the bitcoin client with the -reindex parameter and it may take a day or two to complete. I'll test how things go after everything is done.
Thanks
Thanks

@samedcildir @nathanddrake Hi, thank you for your replies. My node data was downloaded from the Internet and may lack the index of previous data. I am running the bitcoin client with the -reindex parameter and it may take a day or two to complete. I'll test how things go after everything is done.
Thanks
Thanks

Problem solved, thanks for you help!


PHP role usage on Debian Buster in Chapter 6 may need update

From this issue: https://github.com/geerlingguy/ansible-role-php/issues/281
@luisnabais said:

Maybe it should be updated/corrected on the Ansible for Devops book, on page "142", Chapter 6 - Playbook Organization - Roles, Includes, and Imports.

Regarding the example in a debian10 Vagrant Box, he got the error:
TASK [geerlingguy.php : Ensure PHP packages are installed.] ************************************************************************************************************************************************
fatal: [debian]: FAILED! => {"changed": false, "msg": "No package matching 'php7.0-common' is available"}

I'm assuming the example is related to:
---
- hosts: all
  become: yes

  roles:
    - geerlingguy.mysql
    - geerlingguy.apache
    - geerlingguy.php
    - geerlingguy.php-mysql

I just tried that example with the following Vagrantfile:
# -*- mode: ruby -*-
# vi: set ft=ruby :

VAGRANTFILE_API_VERSION = "2"

Vagrant.configure(VAGRANTFILE_API_VERSION) do |config|
  config.vm.box = "geerlingguy/ubuntu2004"
  config.vm.network :private_network, ip: "<IP_ADDRESS>"
  config.vm.hostname = "lampy.test"
  config.ssh.insert_key = false

  config.vm.provider :virtualbox do |v|
    v.memory = 1024
  end

  # Ansible provisioning.
  config.vm.provision "ansible" do |ansible|
    ansible.playbook = "playbook.yml"
  end
end

And I ran into the same error:
TASK [geerlingguy.php : Ensure PHP packages are installed.] ********************
fatal: [default]: FAILED! => {"changed": false, "msg": "No package matching 'php7.0-common' is available"}

So yeah, it would be good to get the PHP role updated to work with the latest versions of Ubuntu and Debian!

I'm going to wrap both of those playbooks into the automated tests included with this repository, too, so if there's an issue in the future it will be easier to trace!


Adding Viva Card samples

Description
This PR includes the design guidance samples for adaptive cards related on the Viva Connections Quick view guidance based on discussion in email.
Microsoft Reviewers: Open in CodeFlow

Problematic template has been removed. Not sure what exactly was wrong with that, but let's ensure that we get at least the other samples from the design team to be available in the gallery.

Closing due to lack of activity. Please feel free to reopen with fixes for jwoo-msft's feedback. Thanks!


Small fixes

AbstractCollection uses itself as \ArrayAccess. It does not call directly offset[Set/Get/Exists/Unset] methods. This change accomplish this coding style.
Rename ValueToStringTaintTest.php to ValueToStringTraitTest.php

Coverage remained the same at 100.0% when pulling 4f6ded8638f719e5f9c0346cc0814d10f2cafb9b on eclipxe13:smallfixes into ecb0b5423d558d01c2063e31215119f602844e4b on ramsey:master.

Thanks!


[FIX] Add rate-limit to authentication routes and other necessary part.

Describe the bug
The authentication routes without rate-limits are open to all kinds of attacks and abuse like:

Brute-Force Attacks:
These attacks involve trying different password combinations repeatedly to gain unauthorized access. Rate limiting can significantly slow down these attempts, making them less effective.

DDoS Attacks:
Distributed Denial of Service (DDoS) attacks aim to overwhelm a system with numerous requests, making it unavailable to legitimate users. Rate limiting can help mitigate the impact of these attacks by preventing excessive traffic.

API Abuse:
Some malicious actors might try to exploit APIs by making excessive requests to gather data or perform unauthorized actions. Rate limiting can prevent such abuse.

Bot Attacks:
Bots can be used to automate login attempts or other malicious activities. Rate limiting can hinder the effectiveness of these bots.

Throttling:
If a system is experiencing high load, rate limiting can help prevent it from being overwhelmed by excessive login requests.

Expected behavior
When concurrency is used in accessing the resources on the API, it comes to a point where the API slows down, with rate-limiting, the API is expected to block or deny access to this kind of abuse.
Related Issue
Rate limit

@MikeSoft007

Assign to me


Performance Question about service instnace : getInstance(i Injector) (T, error)

func (s *serviceEager[T]) getInstance(i Injector) (T, error) {
	frame, ok := stacktrace.NewFrameFromCaller()
	if ok {
		s.mu.Lock()
		s.invokationFrames = append(s.invokationFrames, frame) // @TODO: potential memory leak
		s.mu.Unlock()
	}

	return s.instance, nil
}

above code is from https://github.com/samber/do/blob/v2-🚀/service_eager.go,
right now I am using do by generate a global singleton scope and get the service on demand. but there is a call to get call
stack everytime I try to get the instnace
frame, ok := stacktrace.NewFrameFromCaller(), in fact this methos is not exported just for internal usagae. It will decrease the system performance. is there way to optimize the design?

3 ideas come to my mind:

add a parameter in the root scope
add a parameter globally in the lib (eg: do.TraceInvocation = true)
enable only if a debugging library is loaded (like github.com/samber/do/http/gin/v2)

I'm adding this to v2.1 roadmap.


Admission webhook for KongIngress that warns about deprecation of certain fields

Is there an existing issue for this?

[X] I have searched the existing issues

Problem Statement
https://github.com/Kong/kubernetes-ingress-controller/issues/2941 marks some of the KongIngress fields as deprecated and scheduled for removal in 3.0 (https://github.com/Kong/kubernetes-ingress-controller/issues/3018).
It would be cool if kubectl apply of a KongIngress resource that uses the deprecated functionality showed a visual warning. This is a common behavior pattern in the Kubernetes API world.
Proposed Solution
Implement an admission webhook for KongIngress that, if notices that a deprecated field of KongIngress is set, emits a warning that kubectl prints to stderr.
Additional information
https://kubernetes.io/docs/reference/access-authn-authz/extensible-admission-controllers/#response

Admission webhooks can optionally return warning messages that are returned to the requesting client in HTTP Warning headers with a warning code of 299. Warnings can be sent with allowed or rejected admission responses.

Acceptance Criteria

[ ] As a kubectl user, if I kubectl apply a KongIngress that has a deprecated field set, I get a visual warning.

Marked this release/not-required as it's a last-second inclusion in the 2.8 milestone. It would be a great enhancement for the UX (that would help our users migrate away on the deprecation journey), though.


Modified readme circleCI

Fixed misspelling at CircleCi.md an added extra steps

[AUTOMATED] Latest version has been pushed to preview.
Preview deployed at https://preview-f260e17.dev.nextjs-boilerplate.jam3.cloud.jam3.net


Allow multiple resource files

This is in reference to:
#242 Enable importing multiple resource files
I changed resource files to a dictionary for quick access, the key and value are the same (the path).

@ericsciple Can you review when you get a chance?

looks good to me, lets merge it

Lets merge it, We would like to consume this change for update2. We have a library which is consumed by few tasks for which we want to add additional localization resource.

@kasubram Merge complete.

done

thanks folks


UI/Display error with text at location enabling screen

Avoid duplicates

[x] Bug is not mentioned in the FAQ
[x] Bug is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
[x] Bug is not already reported in another issue

Technical details

Device name: Fairphone 3 (FP3)
Android version: 10 (Stock ROM)
App version: 2.0.3

Describe the bug
When the location services are disabled in Android system settings, the screen that asks the user to enable that has a display error with overlapping text.
Steps to reproduce the issue

Disable location services completely.
On the home page click on “location services disabled” (”Standortdienste deaktiviert”)
You get to a screen titled with “risk assessment” (”Risikoermittlung”).
Up until the menu for other countries it looks all right. And further down the static explanation text (”Um zu erkennen…”) also looks ok. Only the big interactive part/prompt with the button for the user is absolutely distorted.

The texts “Ermittlungszeitraum aktivieren” (”enable detection period”) and “Standortverwendung erlauben” (”enable location access”) seem to overlay each other:

BTW, the button works though. :)
(English translations are my guess, I have not double-checked whether these are
Expected behaviour

If I had to guess, I’d say this has been introduced with the latest update.
Using the app in German language.
Screen size is according to Wikipedia:
5.65 in (144 mm)
1080×2160 px FullHD+
427 ppi
Reproducible in dark and light mode.

Also reproducible with CCTG v2.0.3: https://codeberg.org/corona-contact-tracing-germany/cwa-android/issues/120 (Same bug text copied to both issues, only images changed.)

This issue is also reproducible on:
Device name: Samsung Galaxy A5 (2017)
Android version: 8.0.0
App version: 1.15.2 and 2.0.3
It is a regression from version 1.14.3. In this version the display shown by tapping on "LOCATION SERVICES DEACTIVATED" is correct (with no text overwriting other text).
After updating by hand to 1.15.2 the "Period Logged" and "Allow location access" views on the Exposure Logging screen overwrite each other. This problem is unchanged by then updating to 2.0.3.

@rugk @MikeMcC399 Thanks for reporting. Internal Tracking ID: EXPOSUREAPP-6711

Corona-Warn-App Open Source Team

This issue happens as well in version 2.0.4 with Fairphone 3 in Android 10.

@Deepunkt

This issue happens as well in version 2.0.4 with Fairphone 3 in Android 10.

2.0.4 is just a minor bug-fix on top of 2.0.3 to fix issue https://github.com/corona-warn-app/cwa-app-android/issues/2909 scanning QR codes, so I would not expect this issue (#2940) to have been fixed yet.

I opened a PR for this issue: #2947.

There is a similar issue with the RAT information screen. I will create an issue as soon as I find some time.

@rugk
I checked with
Device name: Samsung Galaxy A5 (2017)
Android version: 8.0.0
App version: 2.2.1
that the issue is fixed. It would be good to hear back from you when you have the opportunity to check with the new version on your device as well.

Tested with v2.1⅕ (same device as in OP etc.): Not fixed.
Gonna update… :upside_down_face:

@rugk
Can you confirm the fix in CWA release 2.2.? Thanks

Corona-Warn-App Open Source Team

I can confirm it’s fixed in v2.3.4. :slightly_smiling_face:


Feature/lit 2382 js sdk refactor crypto and other wasm based modules

Description
Rewrite the wasm utilities in a unified Rust crate, and expose them type-safely.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

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

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

Closing to be reopened in a new PR.


WebSockets appear to be broken in 0.5.1

Web sockets (even using the example) do not work.  They startup and report enabling, but do not connect at all.  I've also tried using h3 bindings and could not get anything to connect.
Easy way to test, run SSR/Basic example and attempt to connect.

https://github.com/nksaraf/vinxi/pull/439

What I did for now is patch node_modules/vinxi/lib/nitro-dev.js with the fix in that PR. (e.g. pnpm patch vinxi)
diff --git a/lib/nitro-dev.js b/lib/nitro-dev.js
index 83b082422216bb63abf201550cc4b380db794e1b..1fe4826e21bbfd9b2d062fe3fcbf4460c2991ce5 100644
--- a/lib/nitro-dev.js
+++ b/lib/nitro-dev.js
@@ -293,26 +293,7 @@ export async function createDevServer(nitro) {
 	// 	}),
 	// );

-	const adapter = wsAdapter({
-		...wsApp.websocket,
-		hooks: {
-			open: (event) => {
-				event.ctx.node.req.url = event.ctx.node.req.originalUrl;
-			},
-			message: (event) => {
-				event.ctx.node.req.url = event.ctx.node.req.originalUrl;
-			},
-			close: (event) => {
-				event.ctx.node.req.url = event.ctx.node.req.originalUrl;
-			},
-			error: (event) => {
-				event.ctx.node.req.url = event.ctx.node.req.originalUrl;
-			},
-			upgrade: (event) => {
-				// event.ctx.node.req.url = event.ctx.node.req.originalUrl;
-			},
-		},
-	});
+	const adapter = wsAdapter(wsApp.websocket);
 	// Listen
 	/** @type {import("@vinxi/listhen").Listener[]}  */
 	let listeners = [];

Not wanting to patch like this as this is for production.  Also it looks like it's the nitro dev server?  Is that correct or am I misreading?


Support ssl=None in connect w/ ws:// URL

Currently, if you pass ssl=None like this:
        if tls:
            use_tls_ctx = SSL_VERIFICATION_CONTEXT

        else:
            use_tls_ctx = None

        websocket = await websockets.client.connect(
            str(loc),
            ssl = SSL_VERIFICATION_CONTEXT,
            max_size = 10 * (2 ** 20)    # Max incoming msg size 10 MiB
        )

you get something like this:
Traceback (most recent call last):
  File "c:\users\badgerdoc\dropbox\documents\projects--muterra\py_hypergolix\hypergolix\comms.py", line 719, in loop_run
    **self._conn_kwargs
  File "C:\Python35\lib\asyncio\coroutines.py", line 105, in __next__
    return self.gen.send(None)
  File "c:\users\badgerdoc\dropbox\documents\projects--muterra\py_hypergolix\hypergolix\comms.py", line 418, in new
    max_size = 10 * (2 ** 20)    # Max incoming msg size 10 MiB
  File "C:\Python35\lib\site-packages\websockets\py35\client.py", line 19, in __await__
    return (yield from self.client)
  File "C:\Python35\lib\site-packages\websockets\client.py", line 151, in connect
    raise ValueError("connect() received a SSL context for a ws:// URI. "
ValueError: connect() received a SSL context for a ws:// URI. Use a wss:// URI to enable TLS.

The docs on AbstractEventLoop.create_connection makes it look like ssl=None is acceptable. It would be nice (I mean, it's definitely splitting hairs! It took me all of three additional lines of code to have a workaraound, but...) if ssl=None would still work with a ws:// URL, as a secondary (yes, perhaps redundant) way of specifying "not using TLS here".

This line means that not specifying ssl and specifying ssl=None are equivalent on the server side.
This line should be changed to achieve the same effect on the client side.
I will accept a PR that changes elif 'ssl' in kwds to elif kwds.get('ssl') is not None. Just make sure tests still pass with tox with that change.

👍 I'll see what I can do!


Ingestion resource with APIs for ingestion via file/URI

Description
Adding APIs for uploading data to an offline table. These are meant for quick testing/trials, and not intended for production usage. We will be using these in the Cluster Manager to introduce a "Test a file" flow.
Pending: yet to add some tests
File ingest
curl "http://localhost:9000/ingestFromFile?tableName=transcript_OFFLINE&batchConfigMapStr={"inputFormat":"json"}"  \
-F<EMAIL_ADDRESS>
URI ingest
Note:

The fs details only need to be provided if scheme is not already registered.
Credentials will by default be taken from local aws credentials

curl "http://localhost:9000/ingestFromURI?tableName=transcript_OFFLINE&
batchConfigMapStr={"inputFormat":"json","input.fs.className":"org.apache.pinot.plugin.filesystem.S3PinotFS","input.fs.prop.region":"us-central","input.fs.prop.accessKey":"foo","input.fs.prop.secretKey":"bar"}
&sourceURIStr=s3://test.bucket/jsondata/part1.json"
  -X POST

Release Notes
Introduced 2 new APIs /ingestFromFile and /ingestFromURI

Codecov Report

Merging #6354 (ec99e83) into master (1beaab5) will decrease coverage by 1.24%.
The diff coverage is 45.12%.

@@            Coverage Diff             @@
##           master    #6354      +/-   ##
==========================================
- Coverage   66.44%   65.20%   -1.25%
==========================================
  Files        1075     1294     +219
  Lines       54773    62294    +7521
  Branches     8168     9023     +855
==========================================
+ Hits        36396    40619    +4223
- Misses      15700    18780    +3080
- Partials     2677     2895     +218

Flag
Coverage Δ

unittests
65.20% <45.12%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...e/pinot/broker/api/resources/PinotBrokerDebug.java
0.00% <0.00%> (-79.32%)
:arrow_down:

...ot/broker/broker/AllowAllAccessControlFactory.java
71.42% <ø> (-28.58%)
:arrow_down:

.../helix/BrokerUserDefinedMessageHandlerFactory.java
33.96% <0.00%> (-32.71%)
:arrow_down:

...ker/routing/instanceselector/InstanceSelector.java
100.00% <ø> (ø)

...ava/org/apache/pinot/client/AbstractResultSet.java
66.66% <0.00%> (+9.52%)
:arrow_up:

.../main/java/org/apache/pinot/client/Connection.java
35.55% <0.00%> (-13.29%)
:arrow_down:

...inot/client/JsonAsyncHttpPinotClientTransport.java
10.90% <0.00%> (-51.10%)
:arrow_down:

...not/common/assignment/InstancePartitionsUtils.java
73.80% <ø> (+0.63%)
:arrow_up:

...common/config/tuner/NoOpTableTableConfigTuner.java
100.00% <ø> (ø)

...ot/common/config/tuner/RealTimeAutoIndexTuner.java
100.00% <ø> (ø)

... and 1143 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 4183ffe...ec99e83. Read the comment docs.

some thoughts re API design. I understand that this one is mainly for testing purposes so it probably doesn't matter too much.
If this is to ever become a production feature for "quickly uploading some data" it may be worthwhile to avoid the "json blob" as a URL fragment in the upload methods.
This could be achieved by either (a) using a JSON payload with the batch configs, as well as a base64-encoded representation of the data, or (b) split out the multipart-upload and launching the ingestion job with "batch configs" as two separate invocations.
Both have merit:
(a) the JSON one is straight forward and makes this a clean API at the cost of expanding the size of the uploaded data via encoding. If the data is tiny anyways, this would be my suggested approach.
(b) if we want to operate the controller like a small temp file server, then might as well have the ability to upload a multipart file. Then, separately, call an API to invoke the ingestion "locally" with a JSON payload that has the configs and a pointer to the uploaded file

some thoughts re API design. I understand that this one is mainly for testing purposes so it probably doesn't matter too much.
If this is to ever become a production feature for "quickly uploading some data" it may be worthwhile to avoid the "json blob" as a URL fragment in the upload methods.
This could be achieved by either (a) using a JSON payload with the batch configs, as well as a base64-encoded representation of the data, or (b) split out the multipart-upload and launching the ingestion job with "batch configs" as two separate invocations.
Both have merit:
(a) the JSON one is straight forward and makes this a clean API at the cost of expanding the size of the uploaded data via encoding. If the data is tiny anyways, this would be my suggested approach.
(b) if we want to operate the controller like a small temp file server, then might as well have the ability to upload a multipart file. Then, separately, call an API to invoke the ingestion "locally" with a JSON payload that has the configs and a pointer to the uploaded file

Thanks for the thoughts Alex. Yes the json blob as a query param is certainly ugly. But this is strictly going to be used for testing purposes, and unllikely that it be operated in a productionized setup. For production, we will always expect users to use the deep store for dropping files and a minion task picking them up periodically/on trigger.
Added a comment on the methods and in the swagger APIs.


Verifier Does Not Compute Extended Base Hash

The verifier implementation does not compute the extended base hash ad instead defaults to zero; without this, the verifier is incomplete.
See the TODO in the verifier implementation for missing code.

Hi, thanks for raising this issue - we've recently transitioned the primary ElectionGuard reference implementation to the ElectionGuard-Python repository. The team believes this issue has been resolved or mitigated there, so closing this issue for now as this repository will no longer be actively maintained. Please feel free to check and leave comments in the latest repo.


Modify GetComponent Method to reflect new CWTOutput object

https://github.com/avcarr2/FCWTNET/blob/f4dfb6842f356466f743610df35d327a6a1eee01/FCWTAPI/CWTObject.cs#L89

Getting rid of the GetComponent methods entirely