Delete duplicated extension

Description
VK_KHR_CREATE_RENDERPASS_2_EXTENSION_NAME is added twice, so delete one.
General Checklist:
Please ensure the following points are checked:

[x] My code follows the coding style
[x] I have reviewed file licenses
[ ] I have updated the header Copyright to reflect the current year (CI build will fail if Copyright is out of date)
[x] I have commented any added functions (in line with Doxygen)
[x] I have commented any code that could be hard to understand
[x] My changes do not add any new compiler warnings
[x] My changes do not add any new validation layer errors or warnings
[x] I have used existing framework/helper functions where possible
[x] My changes build on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions
[x] My changes do not add any regressions
[x] I have tested every sample to ensure everything runs correctly
[x] This PR describes the scope and expected impact of the changes I am making

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

[ ] I have tested the sample on at least one compliant Vulkan implementation
[ ] If the sample is vendor-specific, I have tagged it appropriately
[ ] I have stated on what implementation the sample has been tested so that others can test on different implementations and platforms
[ ] Any dependent assets have been merged and published in downstream modules
[ ] For new samples, I have added a paragraph with a summary to the appropriate chapter in the samples readme

@marty-johnson59 this is safe to merge :)

Merging with 3.  LMK if issues.


Meta: Create schema validation for raw material production chain

This is a meta ticket, providing a general description for the specific subtasks
Imagine a process chain were raw materials are produced. These raw materials could be used in a production process for products, which are subject to regulations on the declaration of environmental impacts at individual item level.
In this scenario we focus on the production of raw materials. The corresponding EPCIS events should contain

the CO2 equivalent (co2e) produced during this process step (as an example for enviromental impact data)
all physical measurements of the created raw material which could be necessary to calculate the share of co2e in subsequent processes, where (parts of) the raw materials are used. This could be volume, mass, density.

The overall process consists of three steps/ event models:

Production of the raw material - #10
Refinement of the raw material - #11
Shipping of the raw material - #12

The corresponding validation tasks are described in the particlar ticket for each event.

@RalphTro please have a look - currently only #10 is ready.
My intention is to identify non-trivial json schema validations, or even limits in schema validation. I hope it's not considered to be mean to any developer ;)


Exclude for dependencies

Fix [SR-688] (https://bugs.swift.org/browse/SR-688) bug.  The folders specified in exclude are handled for dependencies.

This issue was fixed during refactoring by @mxcl but I think it would be useful to keep the test for it.


Boot support?

Would you be open to adding Boot support to this project?

Absolutely, although I have no background or experience in boot; but am happy to take on PRs and/or add collaborators onto the project.
If we do go down the route of adding boot support however, it might be better to consider either:

renaming the project so it isn't specific to leiningen, or
create a completely separate boot-nvd project that depends on the core https://clojars.org/nvd-clojure/.

I haven't got any particular preferences here, but if there is an appetite for this, I would be interested in canvassing people's opinions.

I've got a PR almost ready. Is there a way to set the output directory of all the nvd tasks?

In terms of project structure, I suggest looking at environ.

Is there a way to set the output directory of all the nvd tasks?

There is a nvd.output-dir property which can be specified which is used by the reports to set the output directory, see https://github.com/rm-hull/lein-nvd/blob/master/src/nvd/report.clj#L40

nvd-clojure (part of this repo) is a discrete Clojure program that can be invoked regardless of tooling choices.
That might suffice to consider this issue fixed.
Cheers - V


Example does not actually use sns_extended_client

The library is imported, but then just shadowed further down in the code and will never be used.

import boto3
import sns_extended_client <--- This is imported here but never used

# Low level client
sns = boto3.client('sns')
sns.large_payload_support = 'bucket-name'

# boto SNS.Topic resource
resource = boto3.resource('sns')
topic = resource.Topic('topic-arn')
# rest of code omitted

The same goes for all the examples except the last one.

So just to provide a snippet that actually works; add this to your lambda handler:
import sns_extended_client
sns_extended_client_resource = sns_extended_client.SNSExtendedClientSession().resource(
    service_name="sns", region_name="eu-west-1"
)
topic = sns_extended_client_resource.Topic(TOPIC_ARN)
topic.large_payload_support = S3_EXTENDED_PAYLOAD_BUCKET

#... something something create json data or whatever ...

topic.publish(TopicArn=TOPIC_ARN, Message=json_data)


Auto download sdk

If Android SDK is not already installed, AndroidSdkToolsInstaller will download
the zip and extract the binaries.

Thanks for the comments folks! I accidentally moved this PR out of draft but I am still waiting for some feedback from legal. But the current update is that we will have to display the terms and conditions after the user agrees to have Bubblewrap install Android SDK. Then once they have agreed to terms and conditions, download and install can continue. I still need to find someone on Android legal to discuss specifically how best to get the text for license to be displayed. Like making sure we are displaying the most up-to-date license. So I'll move this back to draft for now.
I will also make sure to address the comments before I mark it as ready again. Thanks!

Thanks all for your review! I resolved the conflicts and I think is ready to merge, @andreban. Thanks!


Implement support for attributed strings

Implementation for the enhancement request regarding support for attributed strings

Thanks for your contribution. Closing this PR in favor of https://github.com/teodorpatras/EasyTipView/pull/185.

Thanks for your contribution. Closing this PR in favor of https://github.com/teodorpatras/EasyTipView/pull/185.


vue.js - recursive components doesn't get updated when using nested array from raw json

Version
2.5.13
Reproduction link
https://jsfiddle.net/znedj1ao/9/
Steps to reproduce
I am trying to create a tree with add function using computed data. I used the tree-view example in vuejs official homepage and combine it with the computed function that I created but found no luck in implementing it. I've been trying to solve this for 4 days already and still no luck so I am here trying to look for help.
What is expected?
When you click the add symbol  in the end of the list it will call the addChild function and it will successfully append the data.
What is actually happening?
the data gets appended but the recursive component is not reactive

As we stated in the issue helper instructions, the issue list is not the place to seek help. Please use the forum or the Discord chat.


Automatically load task list from newly created tokens

Maybe this is intentional, but imdone's tasks page doesn't get updated when files are changed right after I create a custom token. This GIF illustrates that: I have to close and reopen the tab to see the changes. (Edit: Another user in #91 said it's also possible to update the page by just issuing imdone again, but it didn't work here. I'm using Ubuntu 16.04.)

Wouldn't it be desirable to change such behavior?

Sorry I'm just getting back to you now, but I think this can be fixed easily, so I'll look at putting it into the next release.  I have to make sure to update the .imdone/config > code.include_lists property with your custom tag and call repository.refresh(cb).  Shouldn't be too much trouble for the next release.

No worries. Thanks!

This should be fixed in 2.0


Deb package should not replace configs

When you install marathon-consul from deb it will overwrite /etc/marathon-consul.d/config.json. Instead user should be asked what should happen.
https://www.debian.org/doc/debian-policy/ch-files.html#s-config-files

My idea is to change the name of default config created by .deb package to something like /etc/marathon-consul.d/config.json.template. Only downside is that new installations won't be as smooth as they are now.

I think adding this file to conffiles should be enough but can't verify it now :(

conffiles is also good idea but goxc doesn't support conffiles/postinst files.

goxc #86 suggest using fpm for generating debs. I think we can go with template option. It won't change current behavior becouse deb config is generated from default values so there shouldn't be any change in deb installation behavior.
@ojagodzinski Would you like to prepare pull request for it?

Prepared PR for debber https://github.com/debber/debber-v0.3/pull/2 (part of goxc used for creating debs). Once it's get merged we need to apply following patch
diff --git a/resources/debian/conffiles b/resources/debian/conffiles
new file mode 100644
index 0000000..834b1ae
--- /dev/null
+++ b/resources/debian/conffiles
@@ -0,0 +1,3 @@
+/etc/init/marathon-consul.conf
+/etc/marathon-consul.d/config.json
+/etc/systemd/system/marathon-consul.service

It looks like debber and goxc is no logner maintained and my fix is not enought. I'll try incorporate fpm.
Thanks @ojagodzinski for pointing out the problem and help with fpm
fpm --verbose -s dir -t deb \
        -n marathon-consul \
        -v 1.0.0-alpha7 \
        --url="https://github.com/allegro/marathon-consul" \
        --vendor=Allegro \
        --maintainer="Allegro Group<EMAIL_ADDRESS>\
        --description "Marathon-consul service (performs Marathon Tasks registration as Consul Services for service discovery) Marathon-consul takes information provided by the Marathon event bus and forwards it to Consul agents. It also re-syncs all the information from Marathon to Consul on startup and repeats it with given interval."


Bug in Docs - Code template links are not working on Agent Patterns page

Links for code templates on the Agent patterns page need to be corrected.

https://github.com/DAGWorks-Inc/burr/tree/main/examples/templates/multi_modal_agent
https://github.com/DAGWorks-Inc/burr/tree/main/examples/templates/multi_agent_collaboration
https://github.com/DAGWorks-Inc/burr/tree/main/examples/templates/agent_supervisor
https://github.com/DAGWorks-Inc/burr/tree/main/examples/templates/hierarchical_agent_teams

Current behavior
Link to code templates are 404s.
Additional context
This bug was found using an automation built by https://www.demodrive.tech/. We are looking at popular OSS projects and helping them ensure their documentation is error-proof.

Fixed in #478 thanks!


Fix running dev/bots/test.dart in Dart 2 mode

@Hixie does the windows failure make sense to you? It seems to be of a kind that should either fail everywhere or succeed everywhere :confused:

I don't know what 's causing it, but it is a flake that I've heard of. Log it and we'll address it when we look at flakes.

Logged and restarted the bot.

ping?


Fix create task with empty desc

Fixed bug where user is able to create a task with an empty description via the use of an invisible character.

Codecov Report

Merging #242 (6876677) into master (adf2d7c) will decrease coverage by 0.08%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #242      +/-   ##
============================================
- Coverage     51.58%   51.50%   -0.09%
  Complexity      595      595
============================================
  Files           107      107
  Lines          2489     2493       +4
  Branches        286      286
============================================
  Hits           1284     1284
- Misses         1145     1149       +4
  Partials         60       60

Impacted Files
Coverage Δ

...ava/manageezpz/logic/commands/EditTaskCommand.java
0.00% <0.00%> (ø)

...ageezpz/logic/parser/AddEmployeeCommandParser.java
100.00% <ø> (ø)

...geezpz/logic/parser/AddEventTaskCommandParser.java
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

This bug is resolved in PR #241


Add types for new cluster info api (_info/<target>)

Closes elastic/elasticsearch#96477

Hey @sethmlarson :) when you have some spare time, can you review this PR? thanks!

Thanks for submitting this! I'm going to have someone from the team review it for correctness :)


Godot Input)

v3.1.stable.offficial / Windows 10

one Key isn't detected if you press left, up, right  at the same time (arrowkeys)

bug report.zip

Most likely, this is not Godot's fault, but a phenomenon known as "keyboard ghosting".

^
This. It's a hardware problem.

Thx


fix accessible colors and set up more neutrals

Fixes
fixes https://github.com/Police-Data-Accessibility-Project/design-system/issues/92
Description
I've been playing with these colors and being indecisive, so I figured it was time to just submit a small PR that fixes the accessibility and play with things later.

defines 50–950 values for brand gold and wine where 500 is the original hex value

gold's default is 600, putting it >3 accessibility-wise
wine's default is 500, the original hex value
50 and 950 are not pure black or white, just close

adds goldneutral and wineneutral to the playground, but nothing is using them yet

50 and 950 are not pure black or white, just close
substituting neutral for goldneutral or wineneutral often looks very nice, but can be weird

moves blue out of the brand section out of pure pedantry (it's not a brand color)

@joshuagraber it may be bad form to add all those neutral variants we aren't using yet, LMK and I will stash them somewhere. I do want to incorporate this soon, but I want that exercise to stop slowing down the gold accessibility thing

@josh-chamberlain I don't think it's bad form. This looks good, and it'll be nice to have the full range of options available.
Do you think we can merge this to beta? Or do you need this more urgently in main?


Adding setHeaders method

Allows to set custom http headers via:
jsonrpc.setHeaders({'X-Auth':123});

and/or
jsonrpcConfigProvider.set(headers:{'xy':123,'z':45});

Dear Jens, thank you for your pull request. I think this is a good addition.
Could you please add unit tests for both the setting of headers via the configuration and via the request call itself? Then I can merge this.

Actually, after thinking about it a bit more, I think it should be implemented slightly differently. These headers could be use for example for authentication. And in that case, you want to be able to send different headers to different JSON-RPC servers.
So I'll implement this, but with the headers option inside the servers argument of jsonrpcConfig. Or, if you want to do that, that's fine too of course.

He Joost, thats a good point, i have changed it in this direction.
May you can add the tests?

Dear Jens, thanks again. I've made my own code and tests based on your pull request. So I'm closing this pull request.
Your request has been fulfilled in version 0.2.0: the servers argument now has a headers field for your custom headers. See the new example file for a working example.
I hope this helps you! And if not, let me know.

He Joost, actually, the idea with the setHeaders method was to have the ability to set headers at runtime. For example after a authentication token has been received.
If i see this correct, i can not inject data into the .config block later or rather it wont be recognized by the jsonrpc service at runtime.
But maybe I overlooked a different way to achieve this?

Hm, I understand your needs. And with the current setup, I am pretty sure that what you want is impossible. Let me think a bit on how to solve that in the best way!


Android - Chat - Badge "stg" is missing  in the chat list

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open the internal tester program New Expensify app
Login with any user

Expected Result:
Must be marked "stg" badge in the sign in page and in the chat list page
Actual Result:
There is no "stg" badge in the sign in page and in the chat list page
Workaround:
Visual
Platform:

Where is this issue occurring?

Android

Version Number: 1.1.77-2
Reproducible in staging?: Y
Reproducible in production?: -
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
View all open jobs on GitHub

Setting as internal, since external collaborators can't really access the staging environment, and monthly, since it is not a big deal.

Issue not reproducible during KI retests. (First week)

Issue not reproducible during KI retests. (Second week)

Issue not reproducible during KI retests. (Third week)Closing


encoding/json: Encode results in invalid json if json.Number is not a valid number

When encoding a struct with a json.Number containing an invalid number the resulting json is invalid.
See: http://play.golang.org/p/Xx4GjIgL_X
This is because of the exception made at: https://github.com/golang/go/blob/master/src/encoding/json/encode.go#L533
If this is intentional I think it should be documented in the json.Number documentation. If not it should be fixed by trying the number.Float64() method first and if it fails writing the result as a String enclosed in quotes.

Interesting. Too late for Go 1.5 but worth looking into early in Go 1.6.
Probably there needs to be a separate routine to validate the form of
JSON number syntax. It's not exactly what strconv.ParseFloat requires,
so using Float64 is only an approximation.
Also, probably both Marshal and Unmarshal should use it.
Unmarshaling "invalid" into a json.Number should not store 'invalid'.
It should be an error.
I think.

I wrote a simple Valid() method for the Number type: http://play.golang.org/p/USf7fIxih7
This method could be used at decode.go#L783 to validate the number before setting it. I guess it's best to return an error when it's not valid.
The same goes for encode.go#L532.
If you want I can submit a diff for this using the Contribution Guidelines?
The function is based on this official JSON diagram:

It is quite a lot faster than using a regexp based method:
BenchmarkNumberValid		50000000	        37.5 ns/op
BenchmarkNumberValidRegexp	 2000000	       846 ns/op

Yes, please mail your change, thanks. Note that the tree is frozen for Go 1.5, so a reviewer will mostly likely mark the change as being for Go 1.6, and it won't be reviewed until the tree reopens.

Done: https://go-review.googlesource.com/#/c/12250/

While fuzzy testing I found some edge cases and fixed them.
I also added a regexp benchmark function to show the speed increase from not using the regexp:
BenchmarkNumberIsValid-8      	50000000	        27.8 ns/op
BenchmarkNumberIsValidRegexp-8	 2000000	       881 ns/op


faster Component.extract()

fixes slowness in Component.extract() reported in https://github.com/gdsfactory/gdsfactory/issues/3086
Before
Layer 0 Time: 0.121232 s
Layer 4 Time: 31.529844 s
Layer 0 and 4 Time: 34.504553 s
After
Layer 0 Time: 0.060577 s
Layer 4 Time: 0.001984 s
Layer 0 and 4 Time: 0.002031 s
thanks @octavephotonics for reporting this!

For fastest performance you should use this in the recursive case https://klayout.de/doc-qt5/code/class_Cell.html#method41 and copy_shapes in the non-recursive case
@joamatab


Pagination with theme gem

I'm working on a theme gem for Octopress 3 and I'm having trouble getting Jekyll to generate paginated pages from the index.html inside the gem. Has anyone else had issues with this?
Things I've noticed:

If no pagination is required (i.e. no page2 gets created), it doesn't show it's head
If pagination is required, it throws the following error, once for every paginated page it's trying to create (excluding the first):

Error reading file /path/to/gem/test/index.html: No such file or directory @ rb_sysopen - /path/to/myGem/test/index.html

If I move the index.html from ../myGem/assets/pages/ to ../myGem/test/ it paginates everything correctly.

I'm trying to avoid making someone copy code from my repo to get the gem working correctly, so I don't really see this as a solution.

I'm looking at the Genesis Theme and I can't find anything egregiously wrong with what I'm doing. Does anyone have any insights?
Link to my code.

@grantbachman Currently there is no way to get Jekyll pagination to work with Octopress Ink plugins. I'm working on an alternate solution that doesn't use the Jekyll paginate plugin.

Jekyll's pagination is broken anyway. It only really works with the index page and assumes a lot.
I've been fighting with it for months...
+1 for creating our own plugin.

Thanks @imathis! Glad to know I'm not crazy :) Keep up the good work with this, Octopress 3 is turning out awesome!

I'm working on pagination here: octopress-paginate. So far it's working pretty well. It's not ready to be used yet, but soon I think we'll be able to have pagination.


oops

I thought I had already reverted/fixed that. Look like line 398 in my current version.

OK cool.


Parser adding an extra line

Hello.
I came across an interesting behavior when playing around with Markdown.
Looks like Markdown is breaking my string in a wrong way and it is breaking my the text in an extra unnecessary line.
Using the following string, I got the outcome as displayed by the image above:
'
The most popular lib
'

On the other hand, when I add any content that is not an HTML tag as first child of the
, the break line does not happen:
'
aThe most popular lib
'

You can play with this example in this codesandbox:
https://codesandbox.io/s/html-react-parser-forked-xocqu
It looks like when the content of the first
 tag is another HTML tag, the parser is breaking the following content to another line.

Hmm, that does seem like a bug. The "b" tag should be detected as an inline HTML tag and processed accordingly.

Hello,
Just dealing with the same issue but with a custom override this time.
Is this going to be fixed at any time soon?

I seem to be running into a similar issue with anchor tags. I was able to work around it using forceInline but I don't think that's a solution I can use across the board unfortunately (since we do want breaks for stuff like headers).

I see the same thing with "mark" tags, although it seems to only happen when the tag is at the beginning of a line,e.g.
github:
foo bar
markdown-to-jsx:
foo
bar
github:
foo bar baz
markdown-to-jsx:
foo bar baz


Add documentation for the LEAP connector

This PR adds documentation for the LEAP connector.
As a custom connector (pre-certification), the LEAP connector has been successfully evaluated by ~25 law firms, representing ~800 users, in multiple countries over the past six (6) months.
Pending Microsoft's capacity and approval, we hope to have the LEAP connector certified and released at Microsoft's earliest opportunity.

Thank you in advance.
@microsoft-github-policy-service agree [company="LEAP"]

@microsoft-github-policy-service agree [company="LEAP"]

@microsoft-github-policy-service agree company="LEAP"

@appintbot-leapdev - Thanks for your pull request. A member of the content team will review it.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Can we please have an update on this PR? Thank you in advance.

@DougKlopfenstein I understand that other connector documentation PRs are assigned to you. We previously worked with @bezhan-msft, who has recently departed (email address is now hard bouncing with no auto-response). Can you please assist us? Our target is the certified connector's inclusion in the Power BI Desktop February 2024 or March 2024 release. Thank you in advance.


How can create custom arrows between blocks.

Hi,
For my project, I want to create custom back arrows between blocks, something like many to one relationship.
Could you please suggest where I should make changes in the flow.js javascript code to make this work.
Thanks and Regards,
Bhavin Jawade

I'm not sure how that would look like, but the arrows are drawn here, here, here, and here. Probably should make it into a function to be fair.
The way it works is by manually setting the SVG coordinates to draw the arrow, so you can easily draw anything you like.

Actually it's much easier to do that now. I've made two functions (drawArrow and updateArrow) to draw and update arrows respectively. You can find them here.


Fix stdout encoding for python2 and python3 in test262-harness.py

Traceback (most recent call last):
  File "C:\work\study\languages\typescript\jerryscript\tools\runners\test262-harness.py", line 1019, in <module>
    sys.exit(main())
  File "C:\work\study\languages\typescript\jerryscript\tools\runners\test262-harness.py", line 1009, in main
    code = test_suite.run(options.command, args,
  File "C:\work\study\languages\typescript\jerryscript\tools\runners\test262-harness.py", line 930, in run
    progress.has_run(result)
  File "C:\work\study\languages\typescript\jerryscript\tools\runners\test262-harness.py", line 753, in has_run
    result.report_outcome(True)
  File "C:\work\study\languages\typescript\jerryscript\tools\runners\test262-harness.py", line 492, in report_outcome
    self.write_output(sys.stdout)
  File "C:\work\study\languages\typescript\jerryscript\tools\runners\test262-harness.py", line 506, in write_output
    target.write("--- errors ---  \n %s" % error)
UnicodeEncodeError: 'gbk' codec can't encode character '\xab' in position 52: illegal multibyte sequence
...............................................=== Summary - updating excludelist ===

JerryScript-DCO-1.0-Signed-off-by: Yonggang Luo<EMAIL_ADDRESS>

I'm getting confused. What is the difference between #4499 (this PR), #4492, and #4390? All claim to fix encoding problems in test262-harness.py .

I'm getting confused. What is the difference between #4499 (this PR), #4492, and #4390? All claim to fix encoding problems in test262-harness.py .

I'm getting confused. What is the difference between #4499 (this PR), #4492, and #4390? All claim to fix encoding problems in test262-harness.py .

Do I need to merge them into a single patch?
I split them for easier to review.

I'm getting confused. What is the difference between #4499 (this PR), #4492, and #4390? All claim to fix encoding problems in test262-harness.py .

Do I need to merge them into a single patch?
I split them for easier to review.

Let's start with this PR and #4492 . What's the difference between them? They contains the exact same changes.

Let's start with this PR and #4492 . What's the difference between them? They contains the exact same changes.

Let's start with this PR and #4492 . What's the difference between them? They contains the exact same changes.

Oh sorry, mis created pr

Let's start with this PR and #4492 . What's the difference between them? They contains the exact same changes.

Oh sorry, mis created pr


[UBS-Admin. Pricing page] Dropdown lists of the menu on the Pricing page ('Tariffs') is not localized for EN

Environment: Windows 10 Pro, Chrome 101
Reproducible: always
Build found: May 18, 2022
Preconditions
User is logged in as Admin. EN locale is set for the page.
Steps to reproduce

Go to UBS Admin Tariffs page: https://ita-social-projects.github.io/GreenCityClient/#/ubs-admin/tariffs
Click on the 'Region' dropdown. Pay attention to the items in the dropdown list.
Select any entry in the 'Region' dropdown menu.
Click on the 'City' dropdown. Pay attention to the items in the dropdown list.
Click on the 'Courier' dropdown. Pay attention to the items in the dropdown list.
Click on the 'Receiving station' dropdown. Pay attention to the items in the dropdown list.
Click on the 'State' dropdown. Pay attention to the items in the dropdown list.

Actual result
Items in the dropdown lists are not localized according to EN locale.
Expected result
Items in the dropdown lists are localized according to EN locale.
User story and test case links
User story #3835
Test case"

We don't localized for EN admin part of project


Update dps mesh mailbox in staging

Update this mailbox to one we have access to so we can see what gets sent to it for testing purposes.

I trust that's the right combination of characters!

🤞


Protect "Generator" from non-admin

Hi,
I am trying to protect all dashboard links (/admin/*) from being accessible by non-admin.
So far, the code below can get the job done.
/routes/web.php
Route::group(['prefix' => 'admin', 'middleware' => ['auth', 'roles'], 'roles' => 'admin'], function () {
	Route::get('/', 'Admin\AdminController@index');
	Route::get('/give-role-permissions', 'Admin\AdminController@getGiveRolePermissions');
	Route::post('/give-role-permissions', 'Admin\AdminController@postGiveRolePermissions');
	Route::resource('/roles', 'Admin\RolesController');
	Route::resource('/permissions', 'Admin\PermissionsController');
	Route::resource('/users', 'Admin\UsersController');
});

There is the link 'admin/generator' which can be accessed publicly and it seems like there is some protection already in  https://github.com/appzcoder/laravel-admin/blob/master/src/routes.php, but unfortunately it did not work in my case.
Can anyone give me hint how to get this done?
(FYI: I started Laravel a few days ago, so I am very beginner. Thank for your patience)
Regarding to this issue, my other thinking is that I made mistake at the first place which I installed this package by using composer require but not require-dev.  But I do need the dashboard functions.
Thank for any answers

Hi @Sengchheang
I've put the route into main route so you can change it anytime. Comited at fd59b70

Thank you @sohelamin
The change works like magic.


Disable Wi-Fi Direct (p2p)

Hotspot does not use P2P. Disable it by adding p2p_disabled=1 to wpa_supplicant.conf.

Irrelevant, closing. Our WLAN Pi modes don't touch wpa_supplicant.conf.
Jerry already added p2p_disabled=1 to wpa_supplicant.conf in pi-gen, which is all we need:
https://github.com/WLAN-Pi/pi-gen/commit/00e056021ed160f25ebeaf61be16df53dd946ede


ResourceExhausted: 429 received metadata size exceeds soft limit

Environment details

OS: Dockerfile base image: python:3.11
Python version: 3.11
pip version: 24.0
google-auth version: 2.29.0

Description
We have created a simple Flask server and deployed it to GCP as Cloud Run Service. We are also using few other dependencies:
google-api-core==2.19.0
google-cloud-aiplatform==1.49.0 # we needed to pin this version, because of depraction warning somewhere in ai-vertex, might be worth checking in future
google-cloud-logging==3.10.0
langchain==0.2.1
langchain_community==0.2.1
langchain-google-vertexai==1.0.4

snippet of the code:
import google.cloud.logging
from langchain_google_vertexai import VertexAI

# setup logging
client = google.cloud.logging.Client()
client.setup_logging()

# in the endpoint
llm_model = VertexAI(
        model_name="text-bison",
        max_output_tokens=256,
        temperature=1,
        top_p=0.8,
        top_k=40,
        verbose=True,
    )

We don't do anything more sophisticated than that. After deployment, it ran fine for few hours and then we started to received warnings:
Retrying langchain_google_vertexai.llms._completion_with_retry.<locals>._completion_with_retry_inner in 4.0 seconds as it raised ResourceExhausted: 429 received metadata size exceeds soft limit (16711 vs. 16384);  :path:90B :authority:79B :method:43B :scheme:44B content-type:60B te:42B grpc-accept-encoding:75B user-agent:100B grpc-trace-bin:103B pc-low-fwd-bin:77B x-goog-request-params:148B x-goog-api-client:12052B x-goog-api-client:62B authorization:1076B x-google-gfe-frontline-info:836B x-google-gfe-timestamp-trace:76B x-google-gfe-verified-user-ip:76B x-gfe-signed-request-headers:472B x-google-gfe-location-info:74B x-gfe-ssl:44B x-google-gfe-tls-base64urlclienthelloprotobuf:299B x-user-ip:56B x-google-service:105B x-google-gfe-service-trace:115B x-google-gfe-backend-timeout-ms:71B accept-encoding:56B x-google-peer-delegation-chain-bin:92B x-google-request-uid:138B x-google-dappertraceinfo:111B.

You can see that there are two fields named x-goog-api-client and one is growing out of proportion.
Later on it grows even bigger and we started to received it on almost every request. The server also started to timeout as it was unable to serve those requests.
Retrying langchain_google_vertexai.llms._completion_with_retry.<locals>._completion_with_retry_inner in 4.0 seconds as it raised ResourceExhausted: 429 received metadata size exceeds soft limit (27114 vs. 16384);  :path:90B :authority:79B :method:43B :scheme:44B content-type:60B te:42B grpc-accept-encoding:75B user-agent:100B grpc-trace-bin:103B pc-low-fwd-bin:77B x-goog-request-params:148B x-goog-api-client:22452B x-goog-api-client:62B authorization:1076B x-google-gfe-frontline-info:837B x-google-gfe-timestamp-trace:76B x-google-gfe-verified-user-ip:76B x-gfe-signed-request-headers:472B x-google-gfe-location-info:74B x-gfe-ssl:44B x-google-gfe-tls-base64urlclienthelloprotobuf:299B x-user-ip:56B x-google-service:105B x-google-gfe-service-trace:115B x-google-gfe-backend-timeout-ms:71B accept-encoding:56B x-google-peer-delegation-chain-bin:92B x-google-request-uid:140B x-google-dappertraceinfo:111B.

It looks like something is appended to this field and it overflows after some time. I found a place in the copde of the library tha could cause it: https://github.com/googleapis/google-auth-library-python/blob/main/google/auth/metrics.py#L138-L154
I'm looking for some guidance what could cause such warning and overflow in requests.
Steps to reproduce
I'm not really sure, error only occurred after few hours (serving few thousands requests)
I have also created this issue in google auth library repo, but maybe someone here will be able to help.
Let me know if I can help you somehow or provide any additional info!

how exactly do you invoke the model?
on a separate note, have you considered switching to Gemini?

We use load_summarize_chain with either stuff or map_reduce. Then we call invoke() on those chains.
We're happy with PaLM2 for our use case here


Feature Request

Hello,
I want to run a server with a lobby server and 2 game servers. However, if the lobby server goes offline, no one can join my server even if my other two game servers are running.
Also, I want to use citizens so my players can change servers. Attaching a command to the citizen “/exaroton switch [server]” does not work.
Please add these features.
From,
ORC

If exaraton has their own "custom" velocity then this comment will not be beneficial.
For the first part, you can use multiple servers as a fallback incase one lobby server goes down and check out the velocity config or the velocity wiki
For the second part, you dont use exaroton, you use /server <servername_specified_in_velocityconfig>, and then add it to an npc using citizens.


Feature/shared model proto txresp

What is this pull request?
This pr contains shared_model's transaction_response implementation for proto loading
How to use the features provided in the pull request?
You can use this patch for testing purposes:
diff --git a/shared_model/backend/protobuf/CMakeLists.txt b/shared_model/backend/protobuf/CMakeLists.txt
index bbaa66d3..9101b61a 100644
--- a/shared_model/backend/protobuf/CMakeLists.txt
+++ b/shared_model/backend/protobuf/CMakeLists.txt
@@ -1,4 +1,4 @@
-add_library(shared_model_proto_backend
+add_executable(shared_model_proto_backend
     impl.cpp
     )

diff --git a/shared_model/backend/protobuf/impl.cpp b/shared_model/backend/protobuf/impl.cpp
index 21353c0d..683753eb 100644
--- a/shared_model/backend/protobuf/impl.cpp
+++ b/shared_model/backend/protobuf/impl.cpp
@@ -15,5 +15,11 @@
  * limitations under the License.
  */

-#include "backend/protobuf/commands/proto_command.hpp"
-#include "backend/protobuf/commands/proto_add_asset_quantity.hpp"
+#include "backend/protobuf/transaction_responses/proto_tx_response.hpp"
+
+int main() {
+  iroha::protocol::ToriiResponse r;
+  r.set_tx_status(iroha::protocol::COMMITTED);
+  shared_model::proto::TransactionResponse resp(r);
+  shared_model::proto::TransactionResponse(iroha::protocol::ToriiResponse{});
+}

PR contains the following tx responses

[x] committed
[x] stateful fail
[x] stateful valid
[x] stateless fail
[x] stateless valid
[x] unknown

Codecov Report

Merging #712 into feature/shared_model will not change coverage.
The diff coverage is n/a.

@@                  Coverage Diff                  @@
##           feature/shared_model     #712   +/-   ##
=====================================================
  Coverage                 43.49%   43.49%
=====================================================
  Files                       298      298
  Lines                     12022    12022
  Branches                   6599     6599
=====================================================
  Hits                       5229     5229
  Misses                      721      721
  Partials                   6072     6072

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 12d58e2...4b3397b. Read the comment docs.


Craedel spring cloud config

Hi,
this is a service release for BAEL-90 and BAEL-219.
Main changes:

Renamed modules
Changed output directory to Docker input directory
Added a README.md

Now both tutorials are working standalone and together.
Regards,
Christian

Already merged


Scheduled messages

Documentation for: https://github.com/mattermost/mattermost/pull/28932
@harshilsharma63 - FYI This docs PR includes additional updates beyond scheduled messages.

@cwarnermm here are a few images you can use. These might be a bit cleaner:
For 'manage scheduled messages'

For troubleshooting

These are @2x resolution, but should be set to 600 width

@asaadmahmood can you review this?

Sure @matthewbirtch

Thanks for all the great feedback, @harshilsharma63! I've incorporated your feedback in my latest commit.
@asaadmahmood - A new generated docs preview will be generated using my latest commit. Looking forward to any feedback you have.

Looking forward to your input, @asaadmahmood!


upgrade go chassis, fix graceful shutdown panic

Pull Request Test Coverage Report for Build 134

0 of 0   changed or added relevant lines in 0 files are covered.
10 unchanged lines in 4 files lost coverage.
Overall coverage increased (+0.1%) to 46.676%

Files with Coverage Reduction
New Missed Lines
%

pkg/egress/egress_config.go
2
69.86%

protocol/dubbo/schema/schema.go
2
0.0%

adminapi/router.go
3
25.93%

protocol/dubbo/dubbo/respond.go
3
37.74%

Totals

Change from base Build 132:
0.1%

Covered Lines:
1018

Relevant Lines:
2181

💛  - Coveralls


Docs: Installation from package

This PR ...
... adds information to docs about Linux distributions providing official Gitea packages:

Arch Linux
Arch Linux ARM

... changes

Alpine Linux: Content shrinking and fixing minor typo.

... removes

Debian: There are no Debian packages available, so i removed this part completely.

If you know other distributions with official packages: Please let me know and I'll add them as well.

I never tested it ... but FreeBSD has it too: https://svnweb.freebsd.org/ports/head/www/gitea/

@gsilvan as it's mentioned for FreeBSD we should perhaps mention that the alpine package install a FHS compliant gitea binary (https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/12065/diffs) ?


fix usage of updatePath

updatePath takes a (true) bool as second param.
see signature of function updatePath(path, avoidRouterUpdate)

This should be fixed when https://github.com/jlongster/redux-simple-router/pull/38 is merged

I believe #38 fixed it which I just merged. Sorry for the duplicate work @sokra and thanks a lot for the PR! That patch was a big one and I usually like to do smaller ones but that implemented a large feature.

No problem


ABP Frame Counter slow writing?

Hello,
First thanks for the great work to this firmware. It fixes my problem with MKR13x0 Boards and resetting frame counters at board restart. (I run in ABP mode, with TPL5100 timer with battery in field, restarting the board every hour to make measurements and send the values to TTN).
Now I noticed, that it needs about 3 seconds after sending a message to store the frame counters to EEPROM.
so if I shut down the MKR node to early after sending, the FCU is not inreasing at the board and the ttn gateway does not forward the message to the application.
The screenshot (Live data from TTN gateway) shows the behavior of the node with dev. address ending with "13 8B":
at time 13:12 i set the delay(5000) between message send and shutdown - it works, FCU is increasing.
at time 13:14 i set the delay only 2 seconds - the FCU at the board doesn not increasing and hence it is not working.
at time 13:17 I set the delay to 3 seconds - it seems to be sufficient.

Is it possible to speed up the writing process? The measurement and sending message takes about 5 seconds - so 3 seconds delay after sending to store the FCU is a lot and increases battery consumption by factor 1.6.

Hello,
Thanks for reporting this. Would you mind elaborating on what you mean by shutting down the node? Does that involve powering off the LoRaWAN modem? In other words, does the LoRaWAN modem lose the memory contents when you shut down your node?
Jan

Hello Jan,
With "shutting down" the node i mean to power off the MKRWAN1300 (as TTN node).
I am using TPL5110 (timer) for low-power management. First the timer gives 5V to the node (VIN). Hence, the node power on, makes measurements and sends the values to TTN network (takes about 5 seconds). At the end of the program the node gives a signal to the TPL5110 timer to  safely turn off the power (VIN = 0V at node). One hour later the timer wakes up and power on the node again. in the meantime (during the node is switched off) the timer runs with very low power (I can't see big difference to self-discharge of battery).
With the orignal MKRWAN firmware I had the problem, that the frame counter is always starting at zero after turn power on. That behavior is not supported well by TTN (Frame Counter Reset an MAC settings did not work properly). And i do not want to make OTAA every time after power on.
After flashing firmware of the node with "lora-modem" the framecounter ist stored somehow (I think in modems EEPROM), even after turn power off and on. - but, only, when I am waiting 3 seconds after sending the message.
I use MKRWAN.h library for sending a message with following code (short version):
modem.begin(EU868)
modem.joinABP(devAddr, nwkSKey, appSKey)
modem.setPort(4);
modem.dataRate(5);
modem.setADR(false);
modem.beginPacket();
modem.write(data, dataSize);
err = modem.endPacket(false);
delay(3000); //waiting for frame counter
digitalWrite(donePin, HIGH); //signal to TPL5110 to power off
I mean for me 3s would be ok. But I was wondering about that behavior. It came out, because frame counters increased by power the node at USB port and not when power off by the timer.

I will take a look at it. I am not sure if I can speed things up. This functionality is controlled by the embedded LoRaMac-node library which determines when to save the crypto state (which the FCnt counter is part of) to NVM. It could be that it is waiting for some LoRaWAN-related timers to fire.
One potential work-around might be to implement an asynchronous notification sent from the modem to the host to indicate that the modem is idle, i.e., that there are no more pending tasks, including unsaved NVM data.
By the way, the LoRaMac-node library this firmware uses has not been designed for this kind of power-on power-off operation. It keeps a lot of data in RAM and not all of it is saved to NVM. Most of it can be recovered after the RAM contents has been lost, but probably not all, depending on what state your device and your network are in. It is generally recommended to operate such devices in an OTAA mode with periodic JOINs, e.g., if it cannot communicate with the network.


feat: Add train_test_split

randomly partition a dataset into subsets while ensuring reproducibility
Reference:

https://scikit-learn.org/stable/modules/generated/sklearn.model_selection.train_test_split.html
https://rsample.tidymodels.org/reference/initial_split.html

https://github.com/ibis-project/ibis-ml/pull/60 implements a basic (approximate) train-test split using Ibis. It could be very nice to wrap this up as part of a utility in IbisML, but it would be the only non-Step utility at this time. Maybe it's sufficient to just show it in the demo notebook for now? I'm not sure.

demonstrating how it's done w/ sufficient explanation seems fine for now

demonstrating how it's done w/ sufficient explanation seems fine for now

Updated the issue title to reflect this.

Explanation is in the tutorial; @jitingxu1 created https://github.com/ibis-project/ibis-ml/pull/124 with an implementation, so let's close this as completed.


Navigation

Portfolio Navigation
User Story
AS AN employer
I WANT to view a potential employee's deployed portfolio of work samples
SO THAT I can review samples of their work and assess whether they're a good candidate for an open position
Acceptance Criteria
GIVEN I need to sample a potential employee's previous work
WHEN I load their portfolio
THEN I am presented with the developer's name, a recent photo or avatar, and links to sections about them, their work, and how to contact them
WHEN I click one of the links in the navigation
THEN the UI scrolls to the corresponding section

Website displays developer's name, avatar and links to sections about them, their work, and has a contact form. Navigation links correspond correctly.


Space missing

"Get-WindowsPackage--Online"
There should be a space before the "online" option

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

ID: 0e8be60e-5712-efd6-df26-0c6e74aff2bb
Version Independent ID: 8a132ca9-3d8d-dd4a-847d-50d3bb77c31f
Content: Manage Nano Server
Content Source: WindowsServerDocs/get-started/Manage-Nano-Server.md
Service: na
Product: windows-server
Technology: server-nano
GitHub Login: @jaimeo
Microsoft Alias: jaimeo

@JasonGerend

Unless someone else has started the process to add the required change,
feel free to ping me for a quick PR to solve the issue.

@illfated: go for it

Hmmmm, I think that should be '-online' instead of '--online'. Didn't notice that until now, sorry for that.
https://docs.microsoft.com/en-us/powershell/module/dism/get-windowspackage

Fair enough. Thanks for the link, although it is better to leave it unmodified in comments:
https://docs.microsoft.com/en-us/powershell/module/dism/get-windowspackage
I will get that change added in the PR.

There you go. Feel free to suggest other modifications to the page, if required.


Add documentation around identifying Red Hat components using purl

This also adds a minimal skeleton for a docs site using mkdocs and the material theme.

Looks good to me.


[NavigatorIOS]'navigationContext' property is not properly initialized

All NavigatorIOS instances shares the same initial navigationContext object, due to the nature of React.createClass. If one of the instances modify that navigationContext object, subsequent instances will be initialized with the modified object.
Example:

NavigatorIOSInstA mounts
NavigatorIOSInstA unmounts, causes NavigatorIOSInstA.navigationContext._eventEmitter = null
NavigatorIOSInstB mounts, now NavigatorIOSInstB.navigationContext._eventEmitter is null

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out it's new home: https://productpains.com/post/react-native/navigatoriosnavigationcontext-property-is-not-properly-initialized


Release template expressions in inline values are not rendered

Extracted from https://sweetops.slack.com/archives/CE5NGCB9Q/p1571400567057100

+1 for that feature, would like to use it for name overrides with Environment.Name in values


Build integration tests as part of CI

This builds, but does not run, the integration tests as part of the CI. Since most of the errors seen with integration tests are build-related, this should hopefully catch those issues before they hit master.

Looks good to me.  Just waiting on CI to come back green.

This seems like as good a place to ask as any...
Should we remove doc generation/upload from the Travis CI? Now that all the crates are individually shipped, docs.rs does a good job of generating and hosting them. Removing that step from CI would shorten each CI run by ~280s (based on the most recent CI run).
The time lost by adding integration tests building could pretty much be entirely regained by doing that.

I'd wait for @matthewkmayer to weigh in since he set it up, but I'm all for getting the build time back.

Huh, looks like AppVeyor didn't like the command, will fix that.
Also, apparently we would only reclaim a ~40% of the added time. Integration test building took 720s.

This rings a bell from a while ago. Is building the integration tests literally recompiling the whole thing?

It is, yeah. Putting integration tests into the workspace may prevent recompilation, I'm not sure.

Now that this has successfully gone through both AppVeyor and Travis, it's easy to get an idea of how it affects the CI time.
For AppVeyor, it basically doubled.
For Travis, it increased by about 50%. Travis also does a lot more, like doc generation, so this isn't surprising.
It's not like we're going to run into timeout issues again, as those were related to a long build step without output, which is not the case here. But I'm concerned about slowing down CI too much since we can only run a single CI build at a time. The slower a single build takes, the more everything gets backed up.
However, I also feel that building integration tests is important. We're going to want to run the tests eventually, so we'll need to build them anyways. Best to just bite the bullet now and get used to it?

This looks good to me.  A good first step before running integration tests, which is a little more complex due to possible race conditions (bucket naming collisions, etc...).
Re: docs on github pages or docs.rs: let's make a new ticket to discuss that. 😄

https://github.com/rusoto/rusoto/issues/754 is for docs.rs vs github pages.


rename body to reqest_body  and response_body

In our code, we use the generic body for both response_body and request_body.  If we can explicitly spell out request_body and response_body, the code will easier to understand.

Ok, agreed

This is done?

This is done?

Not yet. We need some more renaming to be done


[SPARK-6754] Remove unnecessary TaskContextHelper

The TaskContextHelper was originally necessary because TaskContext was written in Java, which does
not have a way to specify that classes are package-private, so TaskContextHelper existed to work
around this. Now that TaskContext has been re-written in Scala, this class is no longer necessary.
@rxin can you look at this? It looks like you missed this bit of cleanup when you moved TaskContext from Java to Scala in #4324
cc @ScrapCodes and @pwendell who added this originally.

Test build #29811 has started for   PR 5402 at commit f089800.

LGTM

Test build #29811 has finished for   PR 5402 at commit f089800.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

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

LGTM
On Tue, Apr 7, 2015 at 6:26 PM, UCB AMPLab<EMAIL_ADDRESS>wrote:

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29811/
Test PASSed.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/5402#issuecomment-90748988.

(Though it is redundant to say. :) ) LGTM too.

I'm merging this in master. Thanks.


Create dataset loader for Multilingual-ALPACA

Dataloader name: multilingual_alpaca/multilingual_alpaca.py
DataCatalogue: http://seacrowd.github.io/seacrowd-catalogue/card.html?multilingual_alpaca

Dataset
multilingual_alpaca

Description
For multilingual general task instruction data, we incorporate ALPACA dataset (Taori et al., 2023), which consists of 52k English questions and corresponding response, and we obtain its foreign version with in-house translation engine.  The six languages are Arabic (Ar), Greek (El), Hindi (Hi), Turkish (Tr), Vietnamese (Vi), Chinese (Zh).

Subsets
-

Languages
vie

Tasks
Chatbot

License
Unknown (unknown)

Homepage
https://drive.google.com/file/d/1bkejieKDJFDJ45UmQYiY4eeqpGBwj-r-/view

HF URL
-

Paper URL
-

#self-assign


Merlin env

change  env to merlin from rapids.

@bschifferer Not sure if I should have updated the docker-compose files... I didnt know if they were locked to the 0.4 tag for a reason?


Invalid or incomplete POST params => ActionController::RoutingError

Environment

Ruby 2.6.5
Rails <IP_ADDRESS>
Simple Form 5.0.2

Current behavior
I have scopes in my model :
scope :animals, -> { where(type_service: 'animals') }
scope :active, -> { where(active: true) }

The form return an error : Invalid or incomplete POST params and a RoutingError
doesn't  works with radio_buttons &  check_boxes
<%= poi.association :services, :as => :radio_buttons, :collection => Service.options_sheets %>
<%= poi.association :services, :as => :check_boxes, :collection => Service.animals.active %>

works with radio_buttons or check_boxes for the two fields
<%= poi.association :services, :as => :radio_buttons, :collection => Service.options_sheets %>
<%= poi.association :services, :as => :radio_buttons, :collection => Service.animals.active %>

works
<%= poi.association :services, :as => :check_boxes, :collection => Service.options_sheets %>
<%= poi.association :services, :as => :check_boxes, :collection => Service.animals.active %>

doesn't  works with select and check_boxes
<%= poi.association :services, :as => :select, :collection => Service.options_sheets %>
<%= poi.association :services, :as => :check_boxes, :collection => Service.animals.active %>

but works with the reverse
<%= poi.association :services, :as => :check_boxes, :collection => Service.options_sheets %>
<%= poi.association :services, :as => :select, :collection => Service.animals.active %>

I am seeing the same thing with a similar setup. Any idea what the root cause might be?

Hey! I'm closing this for now since there's no an easy way of reproducing it without simple app. Feel free to submit it so we can investigate.


View (already confirmed or rejected) transaction details (in the wallet)

Story
As a user
I want to see the details of my completed transaction
So that I can check the details, share to prove it if needed and troubleshoot if something doesn't seem right
Acceptance Criteria

[ ] I can click on a transaction from the list and see the same details I'd seen to approve / reject when I first received them
[ ] There is a way to navigate back to list of transactions
[ ] I can see whether I confirmed or rejected the transaction
[ ] There is a link to the block explorer page for that transaction
[ ] I can see the transaction hash and copy it to clipboard
[ ] I can see what network it was on
[ ] I can see when I responded (confirmed or rejected)
[ ] I can see when it was submitted to the network
[ ] There is a backup, generic "Transaction couldn't be found" page
[ ] If someone renames their key, the key would have the new name
[ ] I can still understand the historic order transaction when a market no longer exists e.g. we've found a way to store it

Tasks

[ ] Detailed sketch
[ ] Visual design
[ ] Implement UI
[ ] Ensure the data can be shared between this and the transaction list
[ ] Render the data on the page
[ ] Integrate with backend
[ ] Write unit tests
[ ] Write end to end tests
[ ] Code review
[ ] Team verify acceptance criteria met
[ ] Exploratory QA testing
[ ] Capture bugs and new edge cases in tickets (this one or new TBD)
[ ] Write automated regression tests for the A/Cs (expanding on A/Cs where possible)
[ ] Add story and A/Cs to spec doc
[ ] Review with stakeholders
[ ] Release

Design
https://www.figma.com/file/rFlT1Ip2ot7Zfr2E5Is3Pd/Browser-Wallet?type=design&node-id=3261-871&t=2iJhdxAg7pulFpV3-0

Sketch

Additional details / background info

This is all covered by the ACs that Dex is now delivering on #143


Sign software for OS X

My work computer is provisioned to disallow OS X software from unidentified developers. Could you get Thyme officially signed? It's pretty easy, as far as I know.
Thanks for your work! I love using Thyme on my personal computer.

I don't have an Apple Developer Connection.


fix(group): initialize items before resetting #57

Alaska Airlines Pull Request
Fixes #57
Summary:
This makes sure this.items is initialized before resetting the radio group. In a Svelte app, this was consistently happening. I'm not sure why, but somehow the individual items "reset" event was being fired before the "slotchange" event that initializes the group. I added a test that failed before this change -- it's not quite the same scenario, but captures the same behavior.
I tested this change inside AuroSvelteDemo and it fixes the error.
Type of change:
Please delete options that are not relevant.

[x] Revision of an existing capability

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

:tada: This PR is included in version 1.5.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Can't save to clipboard on macOS

I can successfully save to disk, but I can't save to the clipboard. I even complied it fully (as a troubleshooting step) versus the default (which also didn't work) and it still yields this error crashing neovim.
Error: Process exited with error code 138
macOS 14.5 (23F79)
neovim version
NVIM VO. 10.0
Build type: Release
LuaJIT 2.1.1713484068

Using kitty 0.36.0

Hi @jondkinney, can you provide screen record for describe this issue? Cause I'm on MacOS but Apple Silicon, I can't reproduce this issue


Add support for compatibility date and flags

Description
This PR adds support for the compatibility_date and compatibility_flags options when uploading a worker script.
https://developers.cloudflare.com/workers/platform/compatibility-dates/
Has your change been tested?

test included

Types of changes
What sort of change does your code introduce/modify?

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] This change is using publicly documented (api.cloudflare.com or developers.cloudflare.com) and stable APIs.

/fyi @GustavoCaso - attempting to get https://github.com/Shopify/cloudflare-go/pull/13 upstreamed

Codecov Report

Merging #1177 (468a02d) into master (6153c1e) will decrease coverage by 0.02%.
The diff coverage is 50.22%.

@@            Coverage Diff             @@
##           master    #1177      +/-   ##
==========================================
- Coverage   49.40%   49.37%   -0.03%
==========================================
  Files         127      128       +1
  Lines       12290    12416     +126
==========================================
+ Hits         6072     6131      +59
- Misses       4840     4883      +43
- Partials     1378     1402      +24

Impacted Files
Coverage Δ

access_organization.go
64.70% <ø> (ø)

cloudflare_experimental.go
0.00% <0.00%> (ø)

teams_rules.go
44.94% <0.00%> (-0.52%)
:arrow_down:

utils.go
72.72% <ø> (ø)

cloudflare.go
68.37% <14.28%> (-0.34%)
:arrow_down:

mtls_certificates.go
26.59% <26.59%> (ø)

origin_ca.go
57.26% <90.90%> (-2.09%)
:arrow_down:

dns.go
72.72% <96.36%> (+9.00%)
:arrow_up:

devices_managed_networks.go
41.55% <100.00%> (+9.09%)
:arrow_up:

email_routing_destination.go
66.66% <100.00%> (+0.41%)
:arrow_up:

... and 10 more

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

thanks @sbfaulkner 🍰


It appears that the SQL endpoint is disabled. The console will fall back to native Druid APIs

This is not an issue this is the correct behaviour. Closing this, please reopen specifying exact problem if any. Thanks


Possible bug with token offsets/results in REST API query response

I'm seeing some oddities in the token & offsets in some of my basic query responses.  I've narrowed down to a simple test case using the != operator on a single sentence.
The basic sentence in play is:  Topical imiquimod and fluorouracil are also potential , but less supported , treatments .
The text has had DRUG entities previously identified and DRUG tagged tokens have an entity value in the form of: |DRUG:<name>|
In this sentence the tokens: imiquimod,  fluorouracil are tagged as DRUGs on the entity axis, all of the other tokens are tagged as '0' for the entity.
Running a simple rest API query of:  (?<notdrug_green>[entity!=/|DRUG.*/])
I would have expected to get every token except the two drug names returned as match spans, but instead I am seeing tokens starting with the token 'also' being returned in the response. I'm not sure why the earlier non-DRUG tokens (Topical, and, are) aren't being identified.
Here is the actual REST API result payload returned for that sentence for that query (I'm also attaching the JSON input doc to this issue):
{
"sentenceId": 29,
"score": 0,
"documentId": "S0002838X12602470",
"sentenceIndex": 29,
"words": [
"Topical",
"imiquimod",
"and",
"fluorouracil",
"are",
"also",
"potential",
",",
"but",
"less",
"supported",
",",
"treatments",
"."
],
"matches": [
{
"span": {
"start": 5,
"end": 6
},
"captures": [
{
"notdrug_green": {
"span": {
"start": 5,
"end": 6
},
"captures": []
}
}
]
},
{
"span": {
"start": 6,
"end": 7
},
"captures": [
{
"notdrug_green": {
"span": {
"start": 6,
"end": 7
},
"captures": []
}
}
]
},
{
"span": {
"start": 7,
"end": 8
},
"captures": [
{
"notdrug_green": {
"span": {
"start": 7,
"end": 8
},
"captures": []
}
}
]
},
{
"span": {
"start": 8,
"end": 9
},
"captures": [
{
"notdrug_green": {
"span": {
"start": 8,
"end": 9
},
"captures": []
}
}
]
},
{
"span": {
"start": 9,
"end": 10
},
"captures": [
{
"notdrug_green": {
"span": {
"start": 9,
"end": 10
},
"captures": []
}
}
]
},
{
"span": {
"start": 10,
"end": 11
},
"captures": [
{
"notdrug_green": {
"span": {
"start": 10,
"end": 11
},
"captures": []
}
}
]
}
S0002838X12602470.json.txt

Thanks for bringing this to our attention, @curtkohler. I'll reproduce and get back to you.

Thanks again for bringing this to our attention, @curtkohler. The latest snapshot should now work as expected.


Upgrade to elm/http version 2.0.0

This is similar to @michaelwebb76's PR #16, but it provides the body with a BadStatus response, so that the AuthorizationCode request can parse the returned JSON for the error.
I have tested the AuthorizationCode branch in billstclair/elm-oauth-middleware.
PR #16 is simpler, and should be sufficient for other than AuthorizationCode errors. If you choose that one, I'll probably make my own copy of the AuthorizationCode code in elm-oauth-middleware.

@billstclair Apologizes for the time it took me to get back to you.
Yet, indeed, I reviewed both PRs and, made a few changes to the one from @michaelwebb76 before merging it.
I first had a hard-time understanding why in your PR you were redefining the HTTP error types and re-read carefully your description above. Indeed, I don't understand why the native elm http library doesn't include the body for the BadStatus branch when using the basic expect*** helper. I guess there's a valid design decision behind this but I fail at grasping it here :/ ...
Having said that, including a superset of the HTTP error within the OAuth library feels wrong and feels like accommodating here for flaws in the built-in http package. That's not something I am really willing to do, hence why I didn't merge this. If you can work around this, then it's all good and I'd actually be really keen to have a look at how to maybe include that in the troubleshooting section of the README :+1:

Working around my kluge on the package as you updated it ended up being very easy, and only required a small change to the receipt of the token from the billstclair/elm-oauth-middleware server, something that nobody but me may ever do. I applaud you for discarding my proposal.


Change options of 2nd dropdown depending on the choice of the 1st

Hello there, I'm testing this tool in order to check if I use it on the project I'm currently working on. However, there's something that I think it's still uncovered or if it isn't, then it hasn't been specified in the documentation.
I have two dropdowns: the first one has a list of countries and the second a list of provinces. Depending on the country I choose, I make an ajax request to retrieve the provinces that belong to the selected country.
However, I don't find a way to implement this on the survey. Anyone knows how? If I can achieve this, then I'll be able to use it on my project, otherwise maybe I'd use it when I don't require to ask this.
Thank you for your attention :)

@DulceRusso, Hello thanks for your interest. I think, that we already have this enhancement. Please check #393:
from Andrew Telnov:

country -> state -> city?
I was thinking to add this functionality from the beginning, but then decided to wait until somebody actually ask for it :)

Is it the same feature?

@DulceRusso Are you using a web rest service?
Thank you,
Andrew

@DulceRusso I have found a plunker demo: https://plnkr.co/edit/0sZFOKzlAlWUujDpgQuN?p=preview
Thank you,
Andrew

Hello. Thank you for your answer. That's what I was looking for. And nope, I'm not using a web rest service. I have the information stored in a database and I retrieve it with Ajax. This returns an array of choice html elements to add them in a select.
But, I think by just returning a json the drop down can read it correctly, right? Advice on this part would be very welcome. Once again, thank you.
And, actually I do this with three drop downs:
Country->Province->Community

@DulceRusso you have to use survey.onValueChanged event. Here is the example on plunker: http://plnkr.co/edit/FfYvPB?p=preview
You have to call your ajax request onValueChanged and then on ajax call back set the updated choices to the dropdown question.
Thank you,
Andrew

Hi @andrewtelnov, looking at your example on plunker - is it possible to use {variable} when using matrix dropdown questions? E.g Column2 depend on Column1.

@xraywu Sure, you should have to use survey.onMatrixCellValueChanged event.
Here is the example: http://plnkr.co/edit/r80ZfA?p=preview

@andrewtelnov many thanks! Actually I mean importing the choices via a restful api with variable embedded in the url (example in https://plnkr.co/edit/0sZFOKzlAlWUujDpgQuN?p=preview). But I guess onMatrixCellValueChanged can do the trick 👍

@xraywu Yes, it is better to use the event at the current moment. You may use the variable, like {mytrixquestion[rowIndex].columnName}, but unfortunately there is no currentRow value, so you may write {currentRow.columnName}. You may only to use the absolute row index and it is tricky.
Thank you,
Andrew

@xraywu We have implemented the row variable, it returns the current row values. You may use the following syntax: {row.columnName}
Thank you,
Andrew


Gutter Decoration

Fresh install, with the latest versions of the dependencies, it seems that the gutter decarations for the colors and playing the sound have stopped working.
http://i.imgur.com/7wTcfXt.png

I'll look into this.

The linter was updated and changed the name of its gutter, which we use for our own decorations right now. We'll stop using their gutter soon, but for now this solution should work.

That seemed like the case, its working now though.
Thank you so much for your work! Your atom package is pretty much the best thing to use to make filters!


don't use sync-freelist=true in lnd.conf

https://twitter.com/c_otto83/status/1552337039957413888
We can remove this option from the lnd.conf going forward as has a potential to cause problems.
Supposed to speed up the startup, but can cause high load in case of many channels.
To be clear I didn't experience force closes on v0.15. It is probably affecting nodes with many channels whuch shouldn't run on an RPi anyway.

I've experienced both more FCs and an extremely fast growing channel.db with sync-freelist=true.
Ran both with true and off for 2 weeks, and can validate it was this single setting.
Since false, my channel.db grew by 100MB in 7 days, vs before 600MB-1GB per day, with 120 channels.

@openoms do you think we should have this as default for new users or kind of force-change this with the next update?

Yes, I think it can be removed from the default and if present should be changed to false with lnd.check.sh . I keep it off now and the node restarts are roughly the same.

Final Testing: On release candidate check that it gets removes on update from v1.8 to v1.9


Switch to com.github.mwiede:jsch

... for reasons named in https://www.matez.de/index.php/2020/06/22/the-future-of-jsch-without-ssh-rsa/

Do we need to do it now?
I’m always a bit nervous when switching some security component to a new artifact coming out of the blue.
That’s how good supply chain attacks usually start.
If it’s not pressing, I would wait for a bit.

We (Camel Quarkus) would prefer getting this not earlier than Quarkus 2.9.
A similar migration is in progress in Camel by @ggrzybek in https://github.com/apache/camel/pull/7241 . That one is scheduled for Camel 3.17 that still does not have an ETA.

Are we likely to get this done any time soon? While there's some divergence between this extension and Camel, we have to maintain some workarounds in Camel Quarkus. Which I'd like to avoid doing across multiple releases if possible.

If you're confident this is a safe move then I suppose we should do it, provided @gastaldi agrees.

+1, that makes sense.
Have anyone tried simply replacing it to verify if any tests fail?

Have anyone tried simply replacing it to verify if any tests fail?

Seems there are some extra classes that need to be registered for reflection as the native itests fail. I could try to take a look into it next week and create a PR after I'm done with the CQ 2.9.0 release.


Carousel Indicators for rotators cannot be changed to round shape.

When suggesting a feature, include:
Carousel indicators appear as squares however the does not appear to be documentation or availability to style or change the style of the indicators.  Css/Scss styling have produced no changes.
See below:
Bootstrap 4 Carousel Indicators

Bootstrap 3 Carousel Indicators

This is something that prevents us from utilizing bootstrap 4 due to the change in appearance of the feature from past version to future version.

Bootstrap 3 didn't provide alternative carousel indicator styles, so there's little change BS4 will either -- though, you could certainly propose a new feature. Luckily, it's relatively easy to create custom indicators on your own:
.carousel-indicators--round li {
    border-radius: 10px;
    margin: 1px 3px;
    height: 10px;
    max-width: 10px;
    border: 1px solid #FFF;
    background-color: transparent;
}
.carousel-indicators .active {
    background-color: #fff;
    max-width: 12px;
    margin: 0 3px;
    height: 12px;
}

Just add the class .carousel-indicators--round to .carousel-indicators.
Please keep in mind that Bootstrap is just a 'base' CSS framework. It does not have all of the answers. You should supplement the missing components with your own CSS in your projects.

It's very possible to change these with custom CSS, so closing out.


core: Check we are not running the legacy camera stack

If we definitely are, give up. Anything else - even if not a Pi at all

we let through.

Signed-off-by: David Plowman<EMAIL_ADDRESS>

@naushir Failure looks to be another timing thing... time to buy those cheapo SSDs!


👋🏼  Remove ghosticon.jpg from assets

refs TryGhost/Ghost#8221
needs TryGhost/Ghost#8285
Remove ghosticon.jpg as it's not used anymore.

Coverage increased (+0.05%) to 74.391% when pulling cf1efd5b1cdadf3f90eec7f7ee25168d25b68f2e on AileenCGN:remove-ghosticon.jpg into 4d2441873f595f9795ab1ae50ba061ddcc1e6dd2 on TryGhost:master.


Bug: Highlights "Home" and "Following" CSS not consistent

Describe the bug
https://github.com/open-sauced/app/assets/23109390/8c486770-0c8e-4fe2-9034-fde0bada0b48
Notice how, as I switch between the "Home" and "Following" tab in the highlights page how the CSS for each card is not consistent. These should be the same between both (just like it would be between posts on Twitter between tweets in "For You" and "Following": https://twitter.com/home)
Steps to reproduce

Go to https://insights.opensauced.pizza/feed
Flip between "Home" and "Following"
Notice the CSS inconsistency between cards

Browsers
Chrome
Additional context (Is this in dev or production?)
No response
Code of Conduct

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

Contributing Docs

[X] I agree to follow this project's Contribution Docs

The Contributor Highlight Card seems to be wrapped in different CSS in Following and Home tabs, giving extra padding in the Following tab:
In Home tab:

                <ContributorHighlightCard
                    ...
                />

where as in Following tab:

                <ContributorHighlightCard
                    ...
                />

Thanks @5hraddha! If you'd like to work on it you can use the .take command.

.take


ImportError: No module named pathlib

I tried to install it but seems I do not have this module pathlib. Should this package installer install it, when I do not have it?
$ pip install st-package-reviewer
Collecting st-package-reviewer
  Downloading st_package_reviewer-0.2.2.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "c:\users\profes~1\appdata\local\temp\pip-build-exvhzl\st-package-reviewer\setup.py", line 2, in <module>
        from pathlib import Path
    ImportError: No module named pathlib

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in c:\users\profes~1\appdata\local\temp\pip-build-exvhzl\st-package-reviewer\

I am on Windows 10 with anaconda conda --version -> conda 4.3.18.
Later by following:

https://github.com/nvbn/thefuck/issues/287 OSX: "ImportError: No module named pathlib"

I installed successfully this package. However after trying to run it I got this error:
$ st_package_reviewer
Traceback (most recent call last):
  File "f:\python27\lib\runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "f:\python27\lib\runpy.py", line 72, in _run_code
    exec code in run_globals
  File "F:\Python27\Scripts\st_package_reviewer.exe\__main__.py", line 5, in <module>
  File "f:\python27\lib\site-packages\st_package_reviewer\__main__.py", line 106
    print("### Repository checks ###", file=out)
                                           ^
SyntaxError: invalid syntax

I just noticed this required python 3.4 while I am on python 2.7.13


fix(core): Only assume empty square if reported square size is 1 and there are no txs

A bug was caught on blockspacerace network that caused a data root mismatch between bridge nodes and core nodes because celestia-node incorrectly assumes an empty square if there are no transactions which is not the case as squares can contain only tail-padding even if no transactions land in the block itself.
We also will now:

rely on using data.SquareSize from the block data for extending shares instead of calculating square size from len(shares), and
panic on data root mismatch

Future considerations:

we need a fuzzing suite against our block extension mechanism that can generate all kinds of blocks (our integration testing doesn't test for edge cases like this.
we need to audit the rest of the codebase to ensure we perform the same calculations over square size or assume empty square where we shouldn't be.

Codecov Report

Merging #1952 (d92e482) into main (b4fe04d) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1952      +/-   ##
==========================================
+ Coverage   57.72%   57.75%   +0.02%
==========================================
  Files         245      245
  Lines       16186    16184       -2
==========================================
+ Hits         9344     9347       +3
+ Misses       5891     5888       -3
+ Partials      951      949       -2

Impacted Files
Coverage Δ

core/eds.go
63.15% <100.00%> (-1.85%)
:arrow_down:

header/header.go
63.30% <100.00%> (-0.34%)
:arrow_down:

... and 3 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[cleanup][build] Remove docker build and publish scripts

Motivation

Those scripts are not used anymore in the release process
Verifying this change

[ ] Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[ ] doc-not-needed
[x] doc-complete

Matching PR in forked repository
PR in forked repository:

@cbornet Thanks for your contribution! As you mark this PR as doc-complete, can you share the related doc page or change set?

I suggest that we should update the build and publish scripts instead of removing them.

+1. In addition, it is necessary to handle existing maintenance branches. Otherwise, the "latest" tag will always point to the image that was published last. That should be prevented.

Closing this one as we will update the scripts instead.


ARM template in this page throws 404

Here is the link which throws 404 : https://azure.microsoft.com/documentation/templates/demos/dns-forwarder/
@KumudD Can you take a look at this ?

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

ID: a7b1b215-0434-145b-cd11-95d0f32bc015
Version Independent ID: d328a868-a151-1985-2b62-f3884f7ad76c
Content: Name resolution for resources in Azure virtual networks
Content Source: articles/virtual-network/virtual-networks-name-resolution-for-vms-and-role-instances.md
Service: virtual-network
GitHub Login: @rohinkoul
Microsoft Alias: rohink

@msrini-MSFT I updated the link and should reflect later today.
Cheers!

#please-close


索引长度太长了

建表
DROP TABLE IF EXISTS `ClientDetails`;
CREATE TABLE `ClientDetails` (
  `appId` varchar(256) NOT NULL,
  `resourceIds` varchar(256) DEFAULT NULL,
  `appSecret` varchar(256) DEFAULT NULL,
  `scope` varchar(256) DEFAULT NULL,
  `grantTypes` varchar(256) DEFAULT NULL,
  `redirectUrl` varchar(256) DEFAULT NULL,
  `authorities` varchar(256) DEFAULT NULL,
  `access_token_validity` int(11) DEFAULT NULL,
  `refresh_token_validity` int(11) DEFAULT NULL,
  `additionalInformation` varchar(4096) DEFAULT NULL,
  `autoApproveScopes` varchar(256) DEFAULT NULL,
  PRIMARY KEY (`appId`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;

提示：
Specified key was too long; max key length is 767 bytes

这个字段命名风格也不统一。

看样子是MySQL版本的问题，我的是MySQL 5.6

767 bytes is the stated prefix limitation for InnoDB tables in MySQL version 5.6 (and prior versions). It's 1,000 bytes long for MyISAM tables. In MySQL version 5.7 and upwards this limit has been increased to 3072 bytes.
You also have to be aware that if you set an index on a big char or varchar field which is utf8mb4 encoded, you have to divide the max index prefix length of 767 bytes (or 3072 bytes) by 4 resulting in 191. This is because the maximum length of a utf8mb4 character is four bytes. For a utf8 character it would be three bytes resulting in max index prefix length of 254.

5.6里，utf8mb4编码索引长度最多191。
参考

看样子是MySQL版本的问题，我的是MySQL 5.6

767 bytes is the stated prefix limitation for InnoDB tables in MySQL version 5.6 (and prior versions). It's 1,000 bytes long for MyISAM tables. In MySQL version 5.7 and upwards this limit has been increased to 3072 bytes.
You also have to be aware that if you set an index on a big char or varchar field which is utf8mb4 encoded, you have to divide the max index prefix length of 767 bytes (or 3072 bytes) by 4 resulting in 191. This is because the maximum length of a utf8mb4 character is four bytes. For a utf8 character it would be three bytes resulting in max index prefix length of 254.

5.6里，utf8mb4编码索引长度最多191。话说这些主键也不用varchar(256)吧。
参考

具体的话是参考的官方的表结构：https://github.com/spring-projects/spring-security-oauth/blob/master/spring-security-oauth2/src/test/resources/schema.sql  ，当然是和MySQL版本有关，我用的是MySQL5.7，官方的表结构某些数据类型也是需要修改的


Multi module compilation error: Dynamic features with dashed names

I added a dynamic module to my multi-module project. Most of my modules contain dashes in their names and I haven't got any problems in compiling them, but when I add a dynamic feature model called "data-dropbox" I get the following error:
The following module names contain invalid characters. Names can only contain letters, digits and underscores.
	-> data-dropbox

Here is the structure of the project:

I'm not sure if this is the right place to report the problem, since in the Issue Tracker there is no option to create a "tools/gradle" problem.

I'm using AS 3.2 Canary 15 with Android Tools 28.0.0-rc1.

For the time being, I recommend to use either underscores or camelCase as naming convention.
You can report this under https://issuetracker.google.com/issues/new?component=398856 and it'll get routed to the correct component.

Done. Thank you!

https://issuetracker.google.com/issues/109923677
I expected they add the support for dash. Documentation still has examples with dash.   https://developer.android.com/studio/projects/dynamic-delivery

Thanks for pointing out that the documentation needs updating here.
As it stands there's no support for dash in feature names.
The docs will be updated to reflect this shortly.

Is it still planned to support modules with dashes? AS 3.4 allows only underscores.


replace dar by duprelatour in the doc

Codecov Report

Merging #2 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #2   +/-   ##
=======================================
  Coverage   80.81%   80.81%
=======================================
  Files          33       33
  Lines        2961     2961
=======================================
  Hits         2393     2393
  Misses        568      568

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 c54eb00...4a74d39. Read the comment docs.


Refactor Serverless, To #21657266

Signed-off-by: cheyang<EMAIL_ADDRESS>
Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #2 (b85a073) into master (c49cb87) will decrease coverage by 0.16%.
The diff coverage is 9.09%.

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
- Coverage   70.56%   70.40%   -0.17%
==========================================
  Files         222      224       +2
  Lines       12153    12175      +22
==========================================
- Hits         8576     8572       -4
- Misses       2659     2682      +23
- Partials      918      921       +3

Impacted Files
Coverage Δ

pkg/common/constants.go
100.00% <ø> (ø)

pkg/ddc/base/runtime.go
72.66% <0.00%> (-1.07%)
:arrow_down:

pkg/utils/serverless.go
0.00% <0.00%> (ø)

pkg/webhook/plugins/fusesidecar/fuse_sidecar.go
0.00% <0.00%> (ø)

pkg/webhook/plugins/plugins.go
88.88% <50.00%> (-11.12%)
:arrow_down:

pkg/ddc/goosefs/metadata.go
60.98% <0.00%> (-3.30%)
:arrow_down:

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 c49cb87...b85a073. Read the comment docs.


bevy_audio: ECS-based API redesign

Objective
Improve the bevy_audio API to make it more user-friendly and ECS-idiomatic. This PR is a first-pass at addressing some of the most obvious (to me) problems. In the interest of keeping the scope small, further improvements can be done in future PRs.
The current bevy_audio API is very clunky to work with, due to how it (ab)uses bevy assets to represent audio sinks.
The user needs to write a lot of boilerplate (accessing Res<Assets<AudioSink>>) and deal with a lot of cognitive overhead (worry about strong vs. weak handles, etc.) in order to control audio playback.
Audio playback is initiated via a centralized Audio resource, which makes it difficult to keep track of many different sounds playing in a typical game.
Further, everything carries a generic type parameter for the sound source type, making it difficult to mix custom sound sources (such as procedurally generated audio or unofficial formats) with regular audio assets.
Let's fix these issues.
Solution
Refactor bevy_audio to a more idiomatic ECS API. Remove the Audio resource. Do everything via entities and components instead.
Audio playback data is now stored in components:

PlaybackSettings, SpatialSettings, Handle<AudioSource> are now components. The user inserts them to tell Bevy to play a sound and configure the initial playback parameters.
AudioSink, SpatialAudioSink are now components instead of special magical "asset" types. They are inserted by Bevy when it actually begins playing the sound, and can be queried for by the user in order to control the sound during playback.

Bundles: AudioBundle and SpatialAudioBundle are available to make it easy for users to play sounds. Spawn an entity with one of these bundles (or insert them to a complex entity alongside other stuff) to play a sound.
Each entity represents a sound to be played.
There is also a new "auto-despawn" feature (activated using PlaybackSettings), which, if enabled, tells Bevy to despawn entities when the sink playback finishes. This allows for "fire-and-forget" sound playback. Users can simply
spawn entities whenever they want to play sounds and not have to worry about leaking memory.
Unsolved Questions
I think the current design is fine. I'd be happy for it to be merged. It has some possibly-surprising usability pitfalls, but I think it is still much better than the old bevy_audio. Here are some discussion questions for things that we could further improve. I'm undecided on these questions, which is why I didn't implement them. We should decide which of these should be addressed in this PR, and what should be left for future PRs. Or if they should be addressed at all.
What happens when sounds start playing?
Currently, the audio sink components are inserted and the bundle components are kept. Should Bevy remove the bundle components? Something else?
The current design allows an entity to be reused for playing the same sound with the same parameters repeatedly. This is a niche use case I'd like to be supported, but if we have to give it up for a simpler design, I'd be fine with that.
What happens if users remove any of the components themselves?
As described above, currently, entities can be reused. Removing the audio sink causes it to be "detached" (I kept the old Drop impl), so the sound keeps playing. However, if the audio bundle components are not removed, Bevy will detect this entity as a "queued" sound entity again (has the bundle compoenents, without a sink component), just like before playing the sound the first time, and start playing the sound again.
This behavior might be surprising? Should we do something different?
Should mutations to PlaybackSettings be applied to the audio sink?
We currently do not do that. PlaybackSettings is just for the initial settings when the sound starts playing. This is clearly documented.
Do we want to keep this behavior, or do we want to allow users to use PlaybackSettings instead of AudioSink/SpatialAudioSink to control sounds during playback too?
I think I prefer for them to be kept separate. It is not a bad mental model once you understand it, and it is documented.
Should AudioSink and SpatialAudioSink be unified into a single component type?
They provide a similar API (via the AudioSinkPlayback trait) and it might be annoying for users to have to deal with both of them. The unification could be done using an enum that is matched on internally by the methods. Spatial audio has extra features, so this might make it harder to access. I think we shouldn't.
Automatic synchronization of spatial sound properties from Transforms?
Should Bevy automatically apply changes to Transforms to spatial audio entities? How do we distinguish between listener and emitter? Which one does the transform represent? Where should the other one come from?
Alternatively, leave this problem for now, and address it in a future PR. Or do nothing, and let users deal with it, as shown in the spatial_audio_2d and spatial_audio_3d examples.

Changelog
Added:

AudioBundle/SpatialAudioBundle, add them to entities to play sounds.

Removed:

The Audio resource.
AudioOutput is no longer pub.

Changed:

AudioSink, SpatialAudioSink are now components instead of assets.

Migration Guide
// TODO: write a more detailed migration guide, after the "unsolved questions" are answered and this PR is finalized.
Before:

/// Need to store handles somewhere
#[derive(Resource)]
struct MyMusic {
    sink: Handle<AudioSink>,
}

fn play_music(
    asset_server: Res<AssetServer>,
    audio: Res<Audio>,
    audio_sinks: Res<Assets<AudioSink>>,
    mut commands: Commands,
) {
    let weak_handle = audio.play_with_settings(
        asset_server.load("music.ogg"),
        PlaybackSettings::LOOP.with_volume(0.5),
    );
    // upgrade to strong handle and store it
    commands.insert_resource(MyMusic {
        sink: audio_sinks.get_handle(weak_handle),
    });
}

fn toggle_pause_music(
    audio_sinks: Res<Assets<AudioSink>>,
    mymusic: Option<Res<MyMusic>>,
) {
    if let Some(mymusic) = &mymusic {
        if let Some(sink) = audio_sinks.get(&mymusic.sink) {
            sink.toggle();
        }
    }
}

Now:
/// Marker component for our music entity
#[derive(Component)]
struct MyMusic;

fn play_music(
    mut commands: Commands,
    asset_server: Res<AssetServer>,
) {
    commands.spawn((
        AudioBundle::from_audio_source(asset_server.load("music.ogg"))
            .with_settings(PlaybackSettings::LOOP.with_volume(0.5)),
        MyMusic,
    ));
}

fn toggle_pause_music(
    // `AudioSink` will be inserted by Bevy when the audio starts playing
    query_music: Query<&AudioSink, With<MyMusic>>,
) {
    if let Ok(sink) = query.get_single() {
        sink.toggle();
    }
}

I'm a big fan of this design, good to see audio get a usability upgrade 👍

I added one more variant for PlaybackMode: Remove. It will cause the audio-related components to be removed from the entity when playback ends, without outright despawning the entity. I think this will be very useful for those use cases where one might want to play audio by reusing an existing entity (such as if there is an entity that also has other components for other purposes).
This begs the question: should PlaybackMode::Once even exist? Does it even make sense to retain audio-related components, if sound is no longer playing?


MVP: Added End to End testing of Reckoner

Added a full end-to-end test with installing charts with reckoner on python 2 & 3
Added a basic yaml course to test with

This is an MVP and isn't efficient or easily developed upon. I think there needs to be more thought put into the "tests" themselves. The MVP aspect is to assure it's possible to run reckoner against a kubernetes cluster with helm. The output results testing from there is the hard part.