Fix tests again

Many cookie problems were already addressed by ether/etherpad-lite#4905, as of this writing still in PR. After that PR and this one, the tests work again on Firefox and Chrome, for me anyway.
Fixed here:
Some regressions from switching to async/await; helper.chrome$ was not ready.
One issue with css size imprecision.
One issue with how for some reason cookies don't have {:} characters espaped, while they do have others escaped.

I wouldn't worry too much about this project btw, we're rewriting :)

I can merge anyway and we can try get it stable but it's unlikely any of the code in this repo in it's current form will ship


[BUG] - Component Select - required attribute

NextUI Version
2.1.13
Describe the bug
https://nextui.org/docs/components/select
My IDE warns about empty required attributes - 18 pieces

Element Select doesn't have required attribute disableAnimation
Element Select doesn't have required attribute fullWidth
Element Select doesn't have required attribute isLabelPlaceholder
....

It's not a big problem, but I want...
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue

open webStorm IDE
create file .tsx
write
function Test(){ return <Select />; }

Expected behavior
The attribute settings are correctly specified for the component.
Screenshots or Videos

Operating System Version
macos
Browser
Chrome

I was not able to reproduce this anymore when I upgraded to NextUi 2.2.8. I think you can close this issue.

Not reproducible + inactivity.


Cannot react to firebase latest change after firebase record added or deleted.

I love this lib so much. But it cannot dynamically refresh data  after firebase record added/canged or deleted.
Is this a by design feature?

Sorry, my bad, it's doable.

Yes, as you found the dynamic refresh is just ridiculously easy using typescript, and observables. However, it is also a little bit hard to discover. I plan on publishing a complete example application to accompany this library, showing how to use it. I have another project coming up where I will be using this library, that is when I will make a lot more progress.

 I added a small function and fixed one some issues to adapt to new rxjs.

Thank you, when I turned my attention back to this project I will compare what you have pasted above with what is already in the software. I already know about a couple of problems to fix, I will try to fix all of the problems next time I work on it.


Style: code format problem

I am using PyCharm to develop our project. I am confused about how to let PyCharm format code in our convention. It seems that our community uses 2 incidents, but PyCharm always lets me format it to 4 incidents.
How can I solve this? Is there a style file I can import into my PyCharm?

@peb-peb hi, could you have a loock on #85 ? I set the incident to 2 in PyCharm and format our project. Is that kinds of code style we are looking?

Here you can find pylintrc that we use in our test environment. Probably you can load that file in PyCharm.

Hello @mshudrak @ZhengLin-Li
I think to solve this issue we can configure the code style settings in PyCharm to match the coding convention used by the community. By following the steps we can do it:

Open PyCharm and go to File -> Settings (or PyCharm -> Preferences on Mac).
In the left-hand pane, click on "Editor" and then click on "Code Style".
In the right-hand pane, we should see a list of programming languages that PyCharm supports. Select the language that we are working with (e.g., Python) and click on it.
We should now see a tab for "Tabs and Indents". Click on it and we should see an option for "Tab size" and "Indent".
Change the "Tab size" to 2 and "Indent" to 2 as well.
Click "OK" to save the changes.

After configuring the code style settings, PyCharm will automatically format the code according to the community's convention of 2 spaces per indent.
If we want to make sure that our code is formatted correctly, we can use the "Reformat Code" option in PyCharm to reformat the code in our file.


Cross-selling twig condition shifted as filter to the for loop, add twig condition to cross-selling

1. Why is this change necessary?
When you have multiple Active Cross-selling assign to one Product, but the first one has no assigns  products, the storefront output is not correct.
Case 1: (Default product page Layout is assigned)
Nothing will displayed in the storefront.
Case 2: (no product page Layout is assigned)
Only the title from the scross-selling is on screen only wehen you click on the title the slider appears.
2. What does this change do, exactly?

Condition shifted as filter to the for loop, so that the condition "if loop.first" is always right.
Add/change if condition, so that cross-selling will be correct displayed

3. Describe each step to reproduce the issue or behaviour.

Assign multiple cross sellings to one product.
Assign products to the second cross selling
dont asign any product to the first scross selling
See it in frontend

-Assign Default product page Layout to the Product
See it in frontend
4. Please link to the relevant issues (if any).
https://issues.shopware.com/issues/NEXT-17805
5. Checklist

[ ] I have written tests and verified that they fail without my change
[ ] I have squashed any insignificant commits
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

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

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-17883
Please use this issue to track the state of your pull request.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-17987
Please use this issue to track the state of your pull request.


Crashing on this file

https://github.com/augmento-ai/quant-reseach/blob/466ee8c32549d4553c79f22422367c0ac85c9e95/src/analysis_helper.py#L1
Im getting next error
[1] During: lowering "id=4[LoopNest(index_variable = parfor_index.66, range = (0, $62binary_subscr.5, 1))]{70: <ir.Block at src/analysis_helper.py (25)>}Var(parfor_index.66, analysis_helper.py:25)" at src/analysis_helper.py (25)

Just fixed a merge issue in analysis_helper.py, can you pull and check if the error still exists?
Else is there any more information about the error? Looks like it could be an error compiling the JIT functions by numba.


imp: check if there are unreviewed paths by METADATA files

Call custom API endpoint to validate if there are unreviewed paths by METADATA files on the repository. Ask user if he/she wants to continue if such paths exist.

Example:

This flow fails on the first unreviewed path. We do not return all unreviewed paths to reduce traffic and keep performance high.
Also maybe it is worth to roll out this via opt-in flag?


no video in room

Hi All
I am now trying to join the room. I see white webpage with a little mic button at the bottom but no video. Where do I start to look?

solved my problem, I was using the local ip address and when I used local host it was all good.

done

I'm glad it's solved :)


bug fix: Fix buddy list filtering.

When we filtered buddy lists, a recent change introduced some
bugs related to case-insensitive emails.  We now circumvent the
bug by indexing presense_info with user_ids.

Automated message from Dropbox CLA bot
@showell, it looks like you've already signed the Dropbox CLA. Thanks!

Merged, thanks @showell!


[Question] Any interest in an Insomnia plugin that would allow for faker type value creation?

Just thinking it would be cool to have many of the values in our queries be dynamic, similar to how faker can generate things like...
First name, last name, phone number etc.
https://github.com/marak/Faker.js/

That sounds like it could be super useful. The API also looks like it would map to a template tag fairly easily too.
{% faker 'name', 'firstName' %}
{% faker 'address', 'country' %}
...

@gschier my thoughts as well. However, I'm new to Insomnia and wasn't sure something like this already existed or not or if it would work implementing as a plugin.
I'm more then happy to take a peak though, if the interest / options are there.

There's nothing very similar to this yet.
Another plugin was created recently that does essentially the same thing (maps a Node.js module to a plugin). I suggest taking a look at it because this one would require a very similar approach: https://github.com/Gabb1995/insomnia-plugin-customtimestamp/blob/master/index.js

Cheers, already looking at it. Back in a few...

@gschier wait.... I can't seem to find examples of using plugin template tags inside the GQL request? How is this does from the app itself? I can only seem to get it to work in the Request URL or Header values.

You can only use template tags in the variables section of a graphql request. So, you'd have to create a GraphQL variable that uses the plugin, then reference that in your query.

@gschier Ah. Less then ideal.
Any interest in extending template tag usage into the body of the request?
What if your request is creating an array of things, each needing a unique name? I'd hate to have to create n:1 variables just to make that happen.
Going to hold off on the plugin for now. Was hoping to bash it out today, but its not feasible for our team to have to manually create variables for each dynamic value we want to inject.

Can you post some examples of where you want to use a template tag in our GraphQL query?
The reason it doesn't support it is because I imagined there would be a lot of scenarios where using Insomnia template syntax ({{ ... }} and {% ... %}) would break the GraphQL syntax, causing linting errors and breaking GraphQL autocomplete.

@gschier Well take my example above...
Ideally, you would not want to test/call your endpoint/methods with the same data each time...
mutation createVideo {
  createVideo(
		title: {% faker.lorem.sentence %},
		description: {% faker.lorem.paragraph %},
		duration: 30000,
		hasFull: {% faker.random.boolean %},
		hasDistilled: {% faker.random.boolean %},
		isEspresso: {% faker.random.boolean %},
		isFeatured: {% faker.random.boolean %},
		isPublished: {% faker.random.boolean %},
		isPublic: {% faker.random.boolean %},
		recordedAt: {% faker.date.past %},
) {
    id
    title
  }
}

Agree'd that how it would need to be implemented would be to add a step before the parser that would inject those values. Ideally, it would need to work with many types, String, Int etc.  I'm not familiar with how Insomnia works internally, so not sure how something like this could be done.
But to be transparent, I actually expected that when I was typing in the body and at the start of typing a value, that Control+Space would work to call out the template dropdown which would allow to select one. Then I expected that the GQL query would have those pill looking variable placeholders where the var would be injected, just like the Request URL area does.

Would be nice if fake data generation were native. Even better if I could choose what fake data I want by using autocomplete dropdown, maybe with a ctrl+shift+space shortcut. This dropdown would list all fake generators available (name, surname, creditcard, etc), and after selecting one, it would have a nice colored box in the json body, just like environment variables. Maybe something like:
{
  "name": fake:name,
  "credit-card": fake:credit-card
}

Imagine that on the Insomnia land page screenshots. Oh boy.

That makes me laugh every time, even after all these years.

I actually had the idea to create a Faker wrapper plugin for Insomnia today. Just so happens I came across this thread while looking through the issues for something else unrelated. I recently published the plugin on NPM. https://www.npmjs.com/package/insomnia-plugin-faker
Enjoy!
Brian

There is also this which is similar but uses Chance.js https://www.npmjs.com/package/insomnia-plugin-chance

Nice one @bbbco!

Wonderful!

@bbbco How can I pass date format to your plugin?

@starx you may have better luck posting your question to the repo directly: https://github.com/bbbco/insomnia-plugin-faker#readme


Vim installation fails during configure on ncurses

I have bootstrapped the installation using the "Standalone Installation" guide (including installation of dupes/ncurses) on a RHEL 6 environment.
I get the following output when I try to install vim:
~ $ brew install vim
==> Downloading https://github.com/vim/vim/archive/v7.4.891.tar.gz
Already downloaded: /homes/bjorncs/.cache/Homebrew/vim-7.4.891.tar.gz
==> ./configure --prefix=/home/bjorncs/.linuxbrew --mandir=/home/bjorncs/.linuxbrew/Cellar/vim/7.4.891/share/man --enable-multibyte --with-tlib=ncurses --enable-cscope --with-features=huge --with-compiledby=Homebrew --enable-perlinterp --enable-pythoninterp --enable-rubyinterp --enable-gui=no --without-x
Last 15 lines from /homes/bjorncs/.cache/Homebrew/Logs/vim/01.configure:
checking for mode_t... yes
checking for off_t... yes
checking for pid_t... yes
checking for size_t... yes
checking for uid_t in sys/types.h... yes
checking for uint32_t... yes
checking whether time.h and sys/time.h may both be included... yes
checking for ino_t... yes
checking for dev_t... yes
checking whether byte ordering is bigendian... no
checking for rlim_t... yes
checking for stack_t... yes
checking whether stack_t has an ss_base field... no
checking --with-tlib argument... ncurses
checking for linking with ncurses library... configure: error: FAILED

brew test ncurses completes successfully and brew doctor is completes without warnings.

Run brew install --debug vim and enter shell. Source config.log from there and look for the last error. It might actually not be in any way connected with ncurses after all. Strange are autotools' ways.

After a reboot Linuxbrew was suddenly able to install vim. I tried to reproduce the issue without success.


Updated UUID and fs library to support Node v12.18.1

PR Checklist:

[x] I have run npm test locally and all tests are passing.
[x] I have added/updated tests for any new behavior.

[x] If this is a significant change, an issue has already been created where the problem / solution was discussed: [N/A, or add link to issue here]

PR Description

Resolved git issue

Git issue resolved


Message deduplication is broken

As a developer I am seeing one message object which is send to SQS. If I then enable content based deduplication I expect to only ever send one such object to my queue (in a certain time frame). What is happening is however a different story. Gosoline collects some attributes and stores an object in the queue with attribute and body fields. Thus, the traceId from the context can end up in my message. However, that ID is random, so I will always get a different message from SQSs perspective and content based deduplication no longer works.
To fix this, we should send a message deduplication ID to SQS together with the message instead of letting SQS handle this itself. We could also move the attributes to SQS attributes, but that does sound a lot more complicated.

The purpose of the stream producer / output / input / consumer stuff is to create a generic way to exchange messages. It has also the opportunity to exchange the underlying transport layer. As a drawback, you can't use any transport related features like sqs message deduplication, but this approach works very well for ~95% of our use cases and should be the same for others.
Hacking these features into the stream abstraction could be difficult and ugly. Instead, one could use the sqs transport directly. If you need message deduplication from sqs, use the sqs package. One could still use the message encoder from the stream package to have fully featured message creation.
Any thoughts about this?

Hmm, sounds reasonable... In that case I would argue that configuring a queue in the config.yml to use message deduplication and then using the generic interface should throw an error explaining the problem instead of silently working wrongly. Will take a look at how I can use a specialized transport instead.

Maybe we have to remove the configuration value? Doesn't make sense to have it configurable if it never works?


Added rake tasks for change.org petition fetch and moved the get_supp…

…orter_data as a task. Updated the main page with the link to change.org. Updated the README.md file with new instructions.

It works, cool! Some feedback:
You left a commented out key and secret in the Rakefile. Please remove those lines.
Since your commit message is so long, please separate it into a subject line and body, and wrap the body at 72 characters. This explains http://chris.beams.io/posts/git-commit/
Turn on trailing whitespace errors in your text editor, and fix them in the Rakefile.
On the homepage, can you make the Change.org alert box class to alert-box success to make it green?

Should not have checked in the key. I'l resubmit it.
Sent from my iPhone

On Nov 22, 2015, at 10:54 PM, Christopher Adams<EMAIL_ADDRESS>wrote:
It works, cool! Some feedback:
You left a commented out key and secret in the Rakefile. Please remove those lines.
Since your commit message is so long, please separate it into a subject line and body, and wrap the body at 72 characters. This explains http://chris.beams.io/posts/git-commit/
Turn on trailing whitespace errors in your text editor, and fix them in the Rakefile.
On the homepage, can you make the Change.org alert box class to alert-box success to make it green?
—
Reply to this email directly or view it on GitHub.


Fix CSS links, and create Material theme

Strangely, hrefs for CSS started by a /, instead of nothing that caused CSS not loading.
Also, I added a Material theme thanks to the colors of Material Oceanic theme from Material Theme UI.
A quick preview of how the theme is looking:

CDN links were fixed in https://github.com/xz/new.css-site/commit/25b50e802140a7d1552fd73f984d8fa1f3f2d439. Thank you!


Add example

Add a minimal example using
https://openneuro.org/datasets/ds000030/versions/1.0.0

No longer relevant hence closing this project. See https://github.com/htwangtw/functional_connectivity_bsms


Multiple cursor on Alt-Click instead of Ctrl-Click

Hello,
Currently brackets code editor and the test editor in codemirror website activates multicursor with ctrl-click .

Can this be remapped to alt-click by come config?

Trying this change as ctrl-click is mapped to jump to definition on most editors, so brackets too is moving to that.
Thanks,
Arun.

The configureMouse option should be able to do this.

found it!
Thanks for the quick Reply @marijnh
function _mouseHandlerOverride(cm, repeat, event) {
            return {
                addNew: event.altKey // alt key will init multi cursor instead of ctrl-key
            };
        }

new CodeMirror(container, {
configureMouse: _mouseHandlerOverride
}


Change Title (site name) but pages still display "ghost" as title

Do you need help or have a question? Please come chat in Slack: https://ghost.org/slack 👫.
If you're filing a bug 🐛, please include the following information:
Screenshot

Steps to Reproduce

From the admin page, go to "Settings: General"
Leave everything default, edit Title & Description and Save
Then when I go to "home" page and do refresh, the title still display "ghost", not the new title. Same thing as the top black bar at the top, still display "ghost". But initially while the page was loading, it displayed the new title ... but then after the page completely (or halfway) loaded, the new title was replaced (switched back) to "ghost".

Technical details

Casper Version: 2.0.4
Ghost Version: 1.6.1
Browser Version: Firefox 54.0.1
OS Version: Windows 10

@eendii this isn't a bug, if you want to use the site title instead of the logo you need to first remove the logo (or you can replace it with your own logo)


Detected that custom integration 'pirateweather' calls async_forward_entry_setup for integration, pirateweather

Describe the bug
Warning in log
Expected behavior
No warnings
Actual behavior
integrations works fine but warnings
Home Assistant version
2024.7.4
Integration version
1.5.3
Other details
2024-08-01 14:49:40.820 WARNING (MainThread) [homeassistant.helpers.frame] Detected that custom integration 'pirateweather' calls async_forward_entry_setup for integration, pirateweather with title: PirateWeather and entry_id: 10e46c8764e4a4edab3a3108534c259b, which is deprecated and will stop working in Home Assistant 2025.6, await async_forward_entry_setups instead at custom_components/pirateweather/__init__.py, line 122: await hass.config_entries.async_forward_entry_setup(entry, PLATFORMS[1]), please create a bug report at https://github.com/alexander0042/pirate-weather-ha/issues
Troubleshooting steps

[X] I have updated my Home Assistant installation to the latest version.
[X] I have updated the Pirate Weather Integration to the latest version.
[X] I have gone through the documentation before opening this issue.
[X] I have searched this repository and API Repository to see if the issue has already been reported.
[X] I have restarted my Home Assistant installation.
[X] I have queried the API in my browser to confirm the issue is not with the API.
[X] I have written an informative title.

Thanks for the report. I've fixed the version and I've released a beta version 1.5.4b1 with the fix. I've tested this on my end in a devcontainer and ran into no issues but it would be nice if you could try on your end to make sure everything is works as it should.

I can confirm the error is gone and the integrations is working fine.
Thanks!

Stable version has been released. Thanks for confirming that everything is working fine after the update.


Fix #2195 - Add better logging when building flawed chaincode

Description

Add logging of the errors when building faulty chaincode.
Also replaced fmt.Sprintf with direct formatting in the logger.
Motivation and Context

Fixes #2195
Gives a better understanding of why building failed by embedding the chaincode build log in the peer log.
How Has This Been Tested?

Brought up a peer and attempted to deploy chaincode with a compile error. The following was output in the peer log:
01:12:24.227 [peer] ensureConnected -> INFO 01f Touch service indicates no dropped connections
01:12:29.185 [dockercontroller] deployImage -> ERRO 020 Error building images: The command '/bin/sh -c go install github.com/hyperledger/fabric/examples/chaincode/go/chaincode_example02 && cp src/github.com/hyperledger/fabric/peer/core.yaml $GOPATH/bin && mv $GOPATH/bin/chaincode_example02 $GOPATH/bin/0cbc73e2e86c98dcfb79243d9761b00af7f882c3c0ac1261f4b4c12956c3015c0f9a79c322447360324045eb6acd961ea3aaa427ba48e2681714bc44d30b2969' returned a non-zero code: 2
01:12:29.186 [dockercontroller] deployImage -> ERRO 021 Image Output:
********************
Step 1 : FROM hyperledger/fabric-baseimage
 ---> 2b5669336d1d
Step 2 : COPY src $GOPATH/src
 ---> Using cache
 ---> 53bdb4e86e2a
Step 3 : WORKDIR $GOPATH
 ---> Using cache
 ---> 12254136b6ec
Step 4 : RUN go install github.com/hyperledger/fabric/examples/chaincode/go/chaincode_example02 && cp src/github.com/hyperledger/fabric/peer/core.yaml $GOPATH/bin && mv $GOPATH/bin/chaincode_example02 $GOPATH/bin/0cbc73e2e86c98dcfb79243d9761b00af7f882c3c0ac1261f4b4c12956c3015c0f9a79c322447360324045eb6acd961ea3aaa427ba48e2681714bc44d30b2969
 ---> Running in a53e381e3f77
# github.com/hyperledger/fabric/examples/chaincode/go/chaincode_example02
src/github.com/hyperledger/fabric/examples/chaincode/go/chaincode_example02/chaincode_example02.go:91: syntax error: unexpected HERE at end of statement

********************
01:12:29.186 [chaincode] Launch -> ERRO 022 launchAndWaitForRegister failed Error starting container: The command '/bin/sh -c go install github.com/hyperledger/fabric/examples/chaincode/go/chaincode_example02 && cp src/github.com/hyperledger/fabric/peer/core.yaml $GOPATH/bin && mv $GOPATH/bin/chaincode_example02 $GOPATH/bin/0cbc73e2e86c98dcfb79243d9761b00af7f882c3c0ac1261f4b4c12956c3015c0f9a79c322447360324045eb6acd961ea3aaa427ba48e2681714bc44d30b2969' returned a non-zero code: 2

Checklist:

[x] I have added a Signed-off-by.
[x] I have either added documentation to cover my changes or this change requires no new documentation.
[x] I have either added unit tests to cover my changes or this change requires no new tests.
[x] I have run golint and have fixed valid warnings in code I have added or modified. This tool generates false positives so you may choose to ignore some warnings. The goal is clean, consistent, and readable code.

Signed-off-by: Julian Carrivick<EMAIL_ADDRESS>

Travis failed due to #2266 which is fixed in #2267

reverify jenkins

Investigating Jenkins failure

Not going to be able to fix this before the weekend, will come back to next week :)

@juliancarrivick-ibm Thank you!

Rebased on master and all tests passed! Ready to be looked at :)

LGTM

LGTM


Add possibillity to skip tests

Hi,
it would be great if there was a possibillity to somehow skip certain tests.
Our workflow is like this at the moment:

We start to work on a feature by creating a gherkin test, that automatically tests our acceptance tests
We implement the feature and the steps behind a feature flag
We know we are done when the  tests are green
We then switch the feature flag

This means, our software is theoretically releasable, even though the WIP gherkin tests are not green yet.
We would like to be able to mark a test as WIP, so we can skip it in our CI/CD pipeline.

The newest version has tags and those work just fine for that. Thank you very much.


How to Get Answer From command /survey ?

hi,
can you please give an example , how to make the results from command /survey sent to the admin bot ?

I assume you're saving all data to the $notes variable?
After the last stage, when the survey command is finished, you can simply send whatever data from $notes that you like to to the admin user.
Have a look at how this is done in the example:
https://github.com/php-telegram-bot/example-bot/blob/cbaec2e66519da9bc01b7ca97e4b0c9666d7bb84/Commands/SurveyCommand.php#L241

Like this sir ? make two result ?
$this->conversation->update();
$out_text = '/Survey result:' . PHP_EOL;
unset($notes['state']);
foreach ($notes as $k => $v) {
$out_text .= PHP_EOL . ucfirst($k) . ': ' . $v;
}
            $data['photo']        = $notes['photo_id'];
            $data['reply_markup'] = Keyboard::remove(['selective' => true]);
            $data['caption']      = $out_text;
            $this->conversation->stop();

            $result = Request::sendPhoto($data);
            $result = Request::sendPhoto([
                    'chat_id' => '$admin_user',
                    'photo' => $data,
                ]);
            break;


forgot httpin modularscale.com

Thanks! Good catch. :smile:

You're welcome!👍

On 15 Apr 2015, at 17:40, Nick Pfisterer<EMAIL_ADDRESS>wrote:
Thanks! Good catch.
—
Reply to this email directly or view it on GitHub.


The first optimizer.step() execution cost extremely long time

Hello, thank you for providing the implementation of the paper. When I run the code, I found that when the optimizer.step() is called for the first time, it would take extremely long time.
For me, when pretrain llama_1b model on one A100 with batch_size == 1, running optimizer.step() for the first time cost me 70 seconds. But the time became normal (30ms) after the first step. Is this because of some tensor-register step?

It is because of SVD operations that compute the projection matrices at the beginning


ghttp post content bug

1. What version of Go and system type/arch are you using?

v1.14.1
2. What version of GoFrame are you using?

v1.12.1
3. Can this issue be reproduced with the latest release?
yes
4. What did you do?

var postData = `{
    "uid"         : 1,
    "sku_id"      : 10000,
    "amount"      : 19.99,
    "create_time" : "2020-03-26 12:00:00",
}`

j, _ := gjson.LoadJson(postData)

g.Client().RequestContent("POST", "http://<IP_ADDRESS>/xxxx", j.MustToJsonString())

then http server receive
[{     "uid"         : 1,     "sku_id"      : 10000,     "amount"      : 19.99,     "create_time" : "2020-03-26 12:00:00", }]
this code is running OK in gf version v1.11.7.
5. What did you expect to see?
6. What did you see instead?

我看看

@KiVirgil How does your server receive the JSON data? Can you provide the server codes receiving the JSON content?

My fault, i have test this code with g.Server(), then it work well.
It might be the bug of "github.com/go-macaron/binding", because i use this tool to do data binding and validation.
I will close this issue, and thanks for your help


Replaced @_inlineable and @_versioned with @usableFromInline

Motivation:
swfit build generated the deprecated warnings for @_inlineable and @_versioned
Modifications:
Replaced @_inlineable and @_versioned with @usableFromInline
Result:
swift builds with no warnings

Unfortunately we cannot do this until we drop Swift 4.0 support, which we are not doing for the foreseeable future. Given that this patch will become out of date very quickly, I think we’ll close this. Thanks for the offer!
(As a note, this warning should go away soon.)


question: different up headers by role

Hi,
thanks for this project, it's working great for us!
I'm using a configuration with an upstream OpenID provider. I have configured two authorization policies that match on the roles returned from the OpenID provider. This works well, and I can make different resources accessible through these policies.
    authorization policy oursite_member {
      set auth url https://portal.oursite.de/oauth2/oursite/
      [...]
      allow roles member
    }

    authorization policy oursite_admin {
      set auth url https://portal.oursite.de/oauth2/oursite/
      [...]
      allow roles admin
    }

Now, I'm using a few admin-only resources with proxy header auth. E.g.
grafana.oursite.de {
    authorize with oursite_admin
    reverse_proxy grafana:3000 {
      header_up X-Auth-User admin
    }
}

Now I'd like to make grafana accessible to non-admin members, but with restricted permissions (view only). For that it'd be most convenient to pass a different header value if a user only matches the oursite_member policy but not the ourisite_admin policy. Something like (imaginary syntax):
grafana.oursite.de {
    authorize with oursite_member
    reverse_proxy grafana:3000 {
      header_up X-Auth-User member
      if user_has_role admin {
        header_up X-Auth-User admin
      }
    }
}

Is this possible? What would be the correct syntax?
Thanks,
Franz

Is this possible? What would be the correct syntax?

@Frando , here is how you should go about it.
When authorize plugin passes a request to the next plugin, e.g. reverse_proxy, it inject a number of items into caddy context.
Please see https://github.com/greenpau/caddy-security/issues/66#issuecomment-1075744577
"{http.auth.user.id}"
"{http.auth.user.claim_id}"
"{http.auth.user.sub}"
"{http.auth.user.email}"
"{http.auth.user.name}"
"{http.auth.user.roles}"

You can then access these variables. Please see https://github.com/greenpau/caddy-security/issues/66#issuecomment-1075777964
Now, please open an issue https://caddy.community/ and ask how to match on the {http.auth.user.roles}.
Hope this helps!

Thanks @greenpau !
In the meantime I also found another way how to solve this, documenting here for others who might be interested:
I added an action add directive with a custom scope to the two user transforms for the different roles. As they are evaluated in-order, the second one overrides the first one if it applies. Thus, if the transforms are ordered from low-to-high privilege, the custom scope is set to the highest privilege the user has.
 transform user {
        match realm oursite
        match group member
        action add authlevel member as string
      }

      transform user {
        match realm oursite
        match group admin
        action add authlevel admin as string
      }

I then used this custom scope to inject a header with its value, which I then use e.g. in grafana to set a user account from the header value, which are assigned the respective access policies without having to go through the hurdles of configuring individual accounts for individual users (which we don't need here).
 authorization policy oursite_member {
      set auth url https://portal.oursite.de/oauth2/oursite/
      crypto key verify {env.JWT_SHARED_KEY}
      allow roles admin member
      validate bearer header
      inject header "X-Auth-Level" from authlevel
    }

    authorization policy oursite_admin {
      set auth url https://portal.oursite.de/oauth2/oursite/
      crypto key verify {env.JWT_SHARED_KEY}
      allow roles admin
      validate bearer header
      inject header "X-Auth-Level" from authlevel
    }

I might try your solution too, thanks for the hints! For now, this setup works great for us as well.

@Frando , perfect! Please consider sponsoring and promoting this project 😄


add textures for proposed sysinfo widget changes

for systeminfo widgets: networking (mod) and battery level (new)

tnx!

sorry Mario i had to revert the PR because the textures were added in the root and not in the resources folder

really?!  that is bizarre.
OMG - I just recreated the PR because I had accidentally deleted my branch the PR was against and right after I read this message -- too funny!
well hopefully you will see the new PR with them in the proper place.
Mario


injector: Split Envoy bootstrap config generation into smaller functions

This PR creates a 2 helper functions to create the Envoy bootstrap config.
No Functional Code Or Struct Changes  (this is proven by the fact that existing tests for the entire Envoy bootstrap config did not change)

getXdsCluster() creates the static XDS cluster struct
getStaticResources() creates the static_resources struct, which contains the XDS cluster

This is merely moving struct creation into a new function and adding a test (albeit duplicating an existing overall test)

Affected area:

New Functionality      [ ]
Documentation          [ ]
Install                [ ]
Control Plane          [ ]
CLI Tool               [ ]
Certificate Management [ ]
Networking             [ ]
Metrics                [ ]
SMI Policy             [ ]
Security               [ ]
Tests                  [ ]
CI System              [ ]
Performance            [ ]
Other                  [ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?

Codecov Report

Merging #2225 (61d3368) into main (7f1a775) will decrease coverage by 0.01%.
The diff coverage is 93.58%.

@@            Coverage Diff             @@
##             main    #2225      +/-   ##
==========================================
- Coverage   57.72%   57.71%   -0.02%
==========================================
  Files         145      145
  Lines        6304     6295       -9
==========================================
- Hits         3639     3633       -6
+ Misses       2661     2658       -3
  Partials        4        4

Impacted Files
Coverage Δ

pkg/envoy/cds/response.go
69.04% <73.68%> (-0.19%)
:arrow_down:

pkg/catalog/routes.go
79.18% <100.00%> (ø)

pkg/envoy/cds/cluster.go
96.80% <100.00%> (ø)

pkg/envoy/cds/tracing.go
100.00% <100.00%> (ø)

pkg/injector/envoy_config.go
93.96% <100.00%> (+0.05%)
:arrow_up:

Continue to review full report at Codecov.

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


Managing user consent for apps article doesn't show in TOC

The page at /microsoft-365/admin/misc/user-consent does not show up in the TOC -- when the page is opened, the TOC doesn't expand to show its location in the TOC. Is this an orphaned page that needs a redirect OR is it a bug on learn.microsoft that TOC doesn't expand upon opening the article? I attempted to search for the title in the TOC search field and it still didn't show. Please investigate and fix. Thanks.

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

ID: b13e4c14-5b0a-a4f6-1e65-7c79e49a0f10
Version Independent ID: ec4223f2-8127-7605-07a0-306c4d412fed
Content: Managing user consent to apps in Microsoft 365 - Microsoft 365 admin
Content Source: microsoft-365/admin/misc/user-consent.md
Service: microsoft-365-business
GitHub Login: @kwekuako
Microsoft Alias: kwekua

@kwekuako Please help us in resolving this issue. Thanks

@kwekuako , I see this as the TOC entry now. Seems like the above is addressed. Thoughts?

@ashishguptaiitb Thanks.


Flashing the original chip

Not a real issue but a question.
Die you manage to flash the original chip with a new firmware or did you replaced it with an d1-mini?
If you manage to flash it, what was the baud rate of the original chip and how did you flash it?
best regards
Sebastian

Hello, I flashed the original chip. It flashes the same way as the others.

Okay, do you have also reboot problems? if i turn both lights on, it will reboot

no worries you can turn on the 2 lights at the same time.

perfect, thank you :)


Feature: When Datatable is selectable, emit select event on checkbox or radio button activity

Example Scenario:
The user sees a DataTable with multi-selection checkboxes . The user checks two checkboxes from two table rows. The developer wishes to react to these checks but is unable to.
See data table demo: https://carbon-components-svelte.onrender.com/framed/DataTable/DataTableBatchSelectionToolbar?theme=white
Currently, only the selected row ids are updated when users select table rows using batch selection. This makes it difficult to build functionality / reactivity based on the user's actions.
Proposed solution:
Similar to on:click:row--expand , surface a on:click:row--select event on the DataTable and dispatch the row selection status when a row is selected/unselected as follows:
dispatch('click:row--select', { row, selected: <selection status> });

Released in v0.69.0


Performance

When the module is imported, there seems to add a good amount of input lag after typing spaces, it freezes actions done after it for a bit too long, maybe there's a way to reduce this, some way to do this asynchronously?

I don't get much of an input lag. Maybe you can attach a screen recording so I might be able to see what the problem is.

I don't think I'd be able to really record this well, but I could give it a try when I have some time, maybe with KeyViz or something. I did notice some other stuff though, when cmd_validator.ps1 is used, doing things like ctrl + a + backspace doesn't work consistently.

Has it always been like this? Or did this issue just appear when I fixed the problem with the square brackets?

Yes it was like this before the square brackets as well, just didn't report at the time.


Where is get_weight_initializer() defined?

I see get_weight_initializer() called in AirNet/inference.py but a search doesn't show it in your code or in TensorFlow.
Where is get_weight_initializer() defined?
Thanks!

There is a get_weight_initializer() method defined in AirNet/inference.py at line 219 ?
def get_weight_initializer():
    if(FLAGS.conv_init == "var_scale"):
        initializer = tf.contrib.layers.variance_scaling_initializer()
    elif(FLAGS.conv_init == "xavier"):
        initializer=tf.contrib.layers.xavier_initializer()
    else:
        raise ValueError("Chosen weight initializer does not exist")
    return initializer

Thank you.  I have no idea why my search missed that.

BTW I really like the rewrite of the TensorFlow-SegNet code you've done. Thank you!

No worries, glad I could help :)
Thank you!


Allow PyGrid to serve plan operations as either a list of individual commands (syft.js) or as TorchScript (Android and iOS)

Note: This issue must be done first: https://github.com/OpenMined/PySyft/issues/2994
Once PyGrid supports the hosting of plans and protocols, and once PySyft supports the ability to compile plans as either TorchScript or as a list of procedures and stateful tensors, we must have some way for PyGrid to send the correct type of plan to the correct worker.
When a worker authenticates itself with PyGrid, it must inform PyGrid what type of plan it would like to receive: TorchScript or a list of operations. PyGrid must then retrieve the appropriate protocol and plan, and convert that plan into the appropriate format (if necessary) before returning it to the worker.

Closing in favor of https://github.com/OpenMined/threepio/issues/49


Simulation instability causes crash

When the simulation is unstable, sometimes, I see this error message:
ERROR: LoadError: BoundsError: attempt to access 94-element Vector{Subzero.Status} at index [102]
Stacktrace:
  [1] getindex
    @ ./essentials.jl:13 [inlined]
  [2] fuse_floes!(floes::StructArrays.StructVector{…}, max_floe_id::Int64, floe_settings::FloeSettings{…}, Δt::Int64, rng::Xoshiro)
    @ Subzero ~/.julia/dev/Subzero/src/physical_processes/simplification.jl:243
  [3] simplify_floes!(model::Model{…}, max_floe_id::Int64, simp_settings::SimplificationSettings{…}, collision_settings::CollisionSettings{…}, floe_settings::FloeSettings{…}, Δt::Int64, rng::Xoshiro)
    @ Subzero ~/.julia/dev/Subzero/src/physical_processes/simplification.jl:366
  [4] timestep_sim!(sim::Simulation{…}, tstep::Int64, start_tstep::Int64)
    @ Subzero ~/.julia/dev/Subzero/src/simulation_components/simulation.jl:204
  [5] run!(sim::Simulation{…}; logger::Nothing, messages_per_tstep::Int64, start_tstep::Int64)
    @ Subzero ~/.julia/dev/Subzero/src/simulation_components/simulation.jl:291
  [6] run!
    @ ~/.julia/dev/Subzero/src/simulation_components/simulation.jl:286 [inlined]
  [7] macro expansion
    @ ./timing.jl:279 [inlined]
  [8] run_time!(simulation::Simulation{…})
    @ Main ~/.julia/dev/Subzero/examples/simple_strait.jl:80
  [9] top-level scope
    @ ~/.julia/dev/Subzero/examples/simple_strait.jl:100
 [10] include(fname::String)
    @ Base.MainInclude ./client.jl:489
 [11] top-level scope
    @ REPL[10]:1
in expression starting at /Users/skylargering/.julia/dev/Subzero/examples/simple_strait.jl:100
Some type information was truncated. Use `show(err)` to see complete types.

I think that the instability is causing some of the floes to be removed and therefore, when it is time to fuse them, the floe list is all screwed up.

I receive the same error in some tests with instability (kinetic energy growth) that occurs when the timestep is too large. I can reproduce this in simulation: energy_conservation_test_error.txt (note that the extension is changed from .jl to .txt so that I can attach it). The simulation
This simulation has most physical settings turned off (including fracturing), with floe_floe_max_overlap = 1.0, and in a doubly-periodic domain.
Note that:

If I decrease the timestep from Δt = 10 to Δt = 1 (and adjust nΔt to produce the same total simulation length), then I no longer receive this error (I think floes don't intersect enough to try and fuse)
The initial floe array size is 248. I receive the error attempt to access 244-element Vector{Subzero.Status} at index [257]
If I close the domain (replace all PeriodicBoundary with CollisionBoundary) then I no longer receive this error regardless of Δt values that I tested (as high as Δt = 1000)

Based on the last point, I suspect that the issue is related to the ghost floes included for periodic boundaries


Use a custom stack snapshot and improve CI cache

We now use a custom stack snapshot instead of specifying extra-deps in stack.yaml and tweak the CI caching scripts.
Before the CI build caching suffered from a couple of problems:

A cache for STACK_ROOT would always be uploaded after the build. (I don’t know why.)
We cached the build artifacts in .stack-work keyed by the has of stack.yaml. This resulted in many builds sharing building artifacts which could potentially result in bugs.

With this commit we stop caching build artifacts and improve the caching strategy for STACK_ROOT.
By using a snapshot the artifacts for the extra packages (e.g. cbor) are build in STACK_ROOT and cached. (Including the extra packages in the cache was the initial reason for caching .stack-work)
The main cache ID for STACK_ROOT is determined by the hash of snapshot.yaml and package.yaml. (Ideally the cache ID should be the hash of the output of stack ls dependencies, but this requires some further modifications.)
In addition we have a STACK_ROOT cache for the master branch. This cache is always uploaded for a master build. If a cache with a specific hash does not exist we load the master cache.

Do snapshots support git dependencies? I figure we will spend a lot of time building those redundantly down the road.

Yes, they support git dependencies. Fetching the source is done locally in .stack-work so it needs to be repeated for every build. But then the source is hashed in the build happens in the global stack root which is cached. So on a subsequent build we check out the source again but then find the already built version.


Incorrect setting of gyro sensitivity in LSM6DSO driver

Describe the bug
The LSM6DSO driver does not set the gyro sensitivity (full-scale setting) correctly. An example walk-through of the issue when trying to set ±2000°/s full-scale:
Zephyr driver gyro configuration (hard-coded magic numbers that don't align with stmemsc API):
https://github.com/zephyrproject-rtos/zephyr/blob/13bf2e5e48737f5971bdfa2ee683453e379403ea/drivers/sensor/lsm6dso/lsm6dso.h#L67-L86
LSM6DSO_DEFAULT_GYRO_FULLSCALE used:
https://github.com/zephyrproject-rtos/zephyr/blob/9407add27c16f5a10eb1b508f8cb6a482bfc62ed/drivers/sensor/lsm6dso/lsm6dso.c#L782
Magic numbers passed through directly into stmemsc API:
https://github.com/zephyrproject-rtos/zephyr/blob/9407add27c16f5a10eb1b508f8cb6a482bfc62ed/drivers/sensor/lsm6dso/lsm6dso.c#L136-L146
Expected values for lsm6dso_gy_full_scale_set function call:
https://github.com/zephyrproject-rtos/hal_st/blob/575de9d461aa6f430cf62c58a053675377e700f3/sensor/stmemsc/lsm6dso_STdC/driver/lsm6dso_reg.h#L2775-L2784
The stmemsc API is such that the value of the lsm6dso_fs_g_t enum is directly used to set the FS[1:0]_G and FS_125 fields in the CTRL2_G register. The values that the Zephyr driver are passing it appear to bear no relation to the lsm6dso_fs_g_t values and the end result is that, when requesting ±2000°/s full scale the gyro is actually configured for ±125°/s full scale:

LSM6DSO_DEFAULT_GYRO_FULLSCALE == 3
lsm6dso_gy_full_scale_set sets the fs_g member of the lsm6dso_ctrl2_g_t struct, which effectively results in a left shift by 1, setting the lower nibble to 6 (1010 in binary). Since the FS_125 bit is set, the value of the FS[1:0]_G field is ignored by the IMU.

@richardbarlow If you want to take a look to the fix I would appreciate it. Thanks very much for reporting this bad issue.

@avisconti Thank you for fixing the problem so promptly! I am not in a position to test the fix on real hardware at the moment, but reading through the change it looks good :+1:
I'll try to get the fix tested in the next couple of days.

@avisconti Thank you for fixing the problem so promptly! I am not in a position to test the fix on real hardware at the moment, but reading through the change it looks good +1
I'll try to get the fix tested in the next couple of days.

Thanks very much!


refactor(prettier): move printer.rs to its own folder

Current dependencies on/for this PR:

main

PR #1294   👈

This stack of pull requests is managed by Graphite.

Merge activity

Nov 14, 12:42 AM: @Boshen merged this pull request with Graphite.


Add version command

Two comments:

Currently one requires access to a OCP cluster to run opcap version. This shouldn't be necessary. Opened https://github.com/opdev/opcap/issues/120 to track this.
We could think of renaming the module from opcap to github.com/opdev/opcap, which is usually done in similar go projects. This requires to change imports all over the place though. I don't know if anyone has a strong preference here (I don't).

This requires to change imports all over the place though.

Hi @mgoerens , both comments are relevant! Thanks a lot! Please open an issue for the second one as well and elaborate more on that to help folks understand the problem. Although it changes imports in multiple places it seems to be a good first issue since it would require just replacing one line in multiple files if I got that right.

/lgtm


Remove UrlMaker and associated code

Now that models know their own public route (through the public_path and public_url methods), we no longer need to use magic to guess what their route could be.
Therefore removing all of the code that was previously used to do this.
Trello card
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
Follow these steps if you are doing a Rails upgrade.

Wondered if you could add a basic line to each of the first few commits to just explain the context of them so they can be understood individually if e.g. looking at a blame locally, but minor so approving anyway.

Good idea 👍.  Have added a more detailed explanation to each commit.


Fix a bug with misnamed global variable in parser

Oops! The name of the global variable the parser uses was wrong!
This commit fixes that problem and also adds an extra assertion to
ensure that the non-reentrant parser is never called recursively.

Codecov Report

Merging #143 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted Files
Coverage Δ

titan-compiler/parser.lua
95.79% <100%> (+0.07%)
:arrow_up:

Continue to review full report at Codecov.

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


Improve /project_stats display on landscape small screen

I find myself displaying /project_stats to others on my phone.  Going landscape is great for the charts, but is marred because the top black area takes too much space.  It should be one line, not two.

The problem may be that we allow room for a progress bar, but on this page that will never happen :-).


fix: adds dependabot-preview account to admins

After GH acquisition of dependabot, as they are integration more closely, old account opening PRs has been changed to dependabot-preview[bot]
This adds aforementioned account to the admin list so the PRs are automatically tested.

@aditya-konarde do you also review admin list for no longer valid user accounts? (i.e. those who are not part of the project anymore)


build: strip the lipo'ed target placeholder

When building non-MachO targets, there is no support for fat binaries.
As a result, the lipo target does nothing.  Because we do not support
variant builds for ELF targets, this has not resulted in any issues.
However, the PE/COFF (Windows) target suppots multiple variants.  In
such a case, the "lipo" target will copy a number of builds, leaving the
binary to be an arbitrary variant.  Instead, prefer to always use the
architectural variant path.  In place, create an empty target which
serves as a means of collecting dependencies.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 49d885e318ee0336c202b8ad57b730f86b4230a1

@swift-ci please test

@swift-ci please test Linux platform

Build failed
Swift Test Linux Platform
Git Sha - 49d885e318ee0336c202b8ad57b730f86b4230a1

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 49d885e318ee0336c202b8ad57b730f86b4230a1

Can one of the admins verify this patch?

Please update the base branch to main by Oct 5th otherwise the pull request will be closed automatically.

How to change the base branch: (Link)
More detail about the branch update: (Link)


Report generation issue via JUnit Listener in Gradle

Gradle issues-

There are number of tickets opened for this, but no fix is available yet(7-Nov-2018).
* - https://discuss.gradle.org/t/testrunfinished-not-run-in-junit-integration/14644
* - https://github.com/gradle/gradle/issues/842
* - couple of more related tickets.

Gradle does not deal with JUnit listeners.
This should be handled gracefully by Zerocode to bypass listener and generate extent reports.
The usual JUnit xml report is still generated at the usual output folder anyway.
But Zerocode extent reports are useful and handy for step recognition and search and filter operation, as well as Chart, CSV format and more. Hence a fix would help all Gradle folks.
Let's put this into a Given When Then -
GIVEN I am a gradle user and using Gradle as build tool for setting up Zerocode test projects
WHEN I invoke gradelew clean build
THEN the desired tests will run
AND after which the CSV and HTML search and filter reports will be generated
Actual-

The desired tests are run successfully -usual stuff
No extents reports are there. - Ah!(not expected from Gradle)

How do you know it's a gradle issue?

The same setup works via mvn clean install as well as IDE(Eclipse n IntelliJ)

PR in progress : https://github.com/authorjapps/zerocode/pull/152

Version - 1.2.11
Gradle: group: 'org.jsmart', name: 'zerocode-rest-bdd', version:'1.2.11'
Gradle Steps (sample build.gradle - HelloWorld repo ) :

Clear the target folder
Configure the test to generate charts and csv

test {
    systemProperty 'zerocode.junit', 'gen-smart-charts-csv-reports'
    include 'org/jsmart/zerocode/testhelp/tests/HelloWorldGitHubSuite.class'
}

Issue a clean build command as below

  /gradlew clean build

See the target folder for these two reports

zerocode-junit-granular-report.csv
zerocode-junit-interactive-fuzzy-search.html


react-icons lib not working

My code:
import React, { Component, PropTypes, Children } from 'react';
import styles from './styles.css';
import NavBar from '../NavBar';
import { FaBeer } from 'react-icons/lib/fa/';

export default class App extends Component {
  render() {
    return (
      <div className={styles.container}>
        <FaBeer />
        {Children.toArray(this.props.children)}
      </div>
    );
  }
}

This approach should work according to below comment/issue:
https://github.com/mxstbr/react-boilerplate/issues/943#issuecomment-245089173
Error message:
ERROR in ./~/react-icon-base/index.js
Module parse failed: /home/x/Desktop/repos/frontend/node_modules/react-icon-base/index.js Unexpected token (3:50)
You may need an appropriate loader to handle this file type.
| import { default as React, PropTypes } from 'react'
|
| const IconBase = ({ children, color, size, style, ...props }, { reactIconBase = {} }) => {
|   const computedSize = size || reactIconBase.size || '1em'
|   return (
 @ ./~/react-icons/lib/fa/500px.js 18:15-41

In both your example and in there, looks like it doesn't recognise the ... object rest properties. I think it needs to be enabled using Babel.

@williamgranli This is something that it seems many react-icons users have run into, I would suggest looking there for answers. The most common solution that I have seen is to add a few lines to the webpack config (for this boilerplate in particular, the base webpack config).
If you look at an old react-icons issue, with the state the library is currently in, your imports from react-icons need to be loaded with babel-loader, rather than a file loader, or at least that has been my simplest go-to on small projects thus far.
I admittedly do not understand the process well enough to tell if this is an unnecessary work-around (i.e. if there is a significantly better way of doing it).


Remove bounds slot on Boolean

Fixes https://github.com/holoviz/param/issues/699
I could not see a good reason to keep bounds on the Boolean attribute.

Thanks for the review, merging.


Remove entrypoint from app.yaml

Fixes #3829

It's a good idea to open an issue first for discussion.

[ ] I have followed Sample Format Guide
[ ] pom.xml parent set to latest shared-configuration
[ ] Appropriate changes to README are included in PR
[ ] API's need to be enabled to test (tell us)
[ ] Environment Variables need to be set (ask us to set them)
[ ] Tests pass:   mvn clean verify required
[ ] Lint  passes: mvn -P lint checkstyle:check required
[ ] Static Analysis:  mvn -P lint clean compile pmd:cpd-check spotbugs:check advisory only
[ ] Please merge this PR for me once it is approved.

LGTM - should there be some kind of change to a README? Or a comment about entry points no longer be necessary?

This should have been removed with the updates to the maven plugin and gcloud.


Migrating to Manifest V3

What problem does this feature solve?
Hi. This is Mengdi from React DevTools team.
According to Chrome, all extensions must be migrated to Manifest V3 by end of this year. Otherwise Chrome will no longer run them  https://developer.chrome.com/docs/extensions/mv3/mv2-sunset/
What does the proposed API look like?
Do Vue devtools team have a plan to migrate to V3 in near future?

We already have a working branch: https://github.com/vuejs/devtools/tree/manifest-v3
We are waiting for Firefox to support it (+ some testing) to make the switch.

Awesome. Looks like it's already done then.

@mondaychen How to install and use it?

Hi @Akryum I had some trouble migrating React DevTools. After I took a closer look at your v3 branch, I think we took the same approach and you might encounter the same issue.
In the past we both use a sync way to inject the hook, like this
const script = document.createElement('script')
script.textContent = source // code source as string
document.documentElement.appendChild(script)

Because this is no longer allowed in V3, we switch to using a pre-built JS file as src
const script = document.createElement('script')
script.src = chrome.runtime.getURL('build/hook-exec.js')

However, this is approach is async and can cause race condition. In my test, if react-dom is loaded fast enough (usually happens on cached production build), it failed to detect the hook and thus assume the does not have devtools installed.
Can you give it a test (load the V3 extension on a prod build small vue.js website) and see if this is an issue on your side too?

@Akryum What is the current status on this?
According to https://developer.chrome.com/blog/mv2-transition :

January 2023: The Chrome browser will no longer run Manifest V2 extensions. Developers may no longer push updates to existing Manifest V2 extensions

This even states

After January 2023, Chrome browser will no longer run Manifest V2 extensions and developers may no longer be able to push updates to existing Manifest V2 extensions. With the use of enterprise policy, the MV2 extensions can function on Chrome deployments till June 2023. It's recommended that you make sure all your Chrome extensions are Manifest V3 compliant before MV2 is completely deprecated.

Which does not seem to be true because vue devtools still works for me, but nevertheless it might be time to do something?

They backed down on this, I don't think there is a date anymore, I need to check

我遇到了同样的问题，

Hi @Akryum I had some trouble migrating React DevTools. After I took a closer look at your v3 branch, I think we took the same approach and you might encounter the same issue. In the past we both use a sync way to inject the hook, like this
const script = document.createElement('script')
script.textContent = source // code source as string
document.documentElement.appendChild(script)

Because this is no longer allowed in V3, we switch to using a pre-built JS file as src
const script = document.createElement('script')
script.src = chrome.runtime.getURL('build/hook-exec.js')

However, this is approach is async and can cause race condition. In my test, if react-dom is loaded fast enough (usually happens on cached production build), it failed to detect the hook and thus assume the browser does not have devtools installed.
Can you give it a test (load the V3 extension on a prod build small vue.js website) and see if this is an issue on your side too?

I'm having the same problem with upgrading to mv3, how is google doing now? looks like an upgrade to mv3, a long way off!


[Bug] Unable To Download GenshinImpact (International server)

Checklist

[X] My issue was not mentioned by others, and it is not a duplicate issue.

Description
The download process is abruptly interrupted and an error log is thrown
Reproduction Steps

Expected Behavior
No response
Screenshots
No response
Starward Version
0.12.0
Windows Version
Windows 11  22631.3880
Log
[13:03:52.235] [INFO]
Welcome to Starward v0.12.0
System: Microsoft Windows NT 10.0.22631.0
Command Line: D:\Starward\app-0.12.0\Starward.dll download --biz hk4e_global --loc D:\Starward\GameFiles\GenhinImpact --lang 15

[13:03:52.254] [INFO] Starward.Services.DatabaseService
Database path is 'D:\Starward'

[13:03:52.268] [INFO] Starward.Services.DatabaseService
Database version is 9, target version is 9.

[13:03:52.422] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Initialize install game service
biz: hk4e_global
path: "D:\Starward\GameFiles\GenhinImpact"
lang: All
repair: False
reinstall: False

[13:03:52.443] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Start install game, skipVerify: False

[13:03:52.444] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Prepare for download.

[13:03:53.748] [WARN] Starward.Services.Launcher.GameLauncherService
config.ini not found: "D:\Starward\GameFiles\GenhinImpact\config.ini"

[13:03:54.400] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Request head: "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/GenshinImpact_4.8.0.zip.001"

[13:03:54.400] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Request head: "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/GenshinImpact_4.8.0.zip.003"

[13:03:54.400] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Request head: "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/GenshinImpact_4.8.0.zip.002"

[13:03:54.401] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Request head: "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/GenshinImpact_4.8.0.zip.004"

[13:03:54.401] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Request head: "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/GenshinImpact_4.8.0.zip.005"

[13:03:54.401] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Request head: "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/GenshinImpact_4.8.0.zip.006"

[13:03:54.401] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Request head: "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/Audio_Chinese_4.8.0.zip"

[13:03:54.401] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Request head: "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/Audio_English(US)_4.8.0.zip"

[13:03:54.401] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Request head: "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/Audio_Japanese_4.8.0.zip"

[13:03:54.401] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Request head: "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/Audio_Korean_4.8.0.zip"

[13:03:54.679] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Start download files.

[13:03:54.679] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
10 files need to download.

[13:03:54.682] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Download: FileName "GenshinImpact_4.8.0.zip.001", Url "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/GenshinImpact_4.8.0.zip.001"

[13:03:54.683] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Download: FileName "GenshinImpact_4.8.0.zip.002", Url "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/GenshinImpact_4.8.0.zip.002"

[13:03:54.690] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Download: FileName "GenshinImpact_4.8.0.zip.003", Url "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/GenshinImpact_4.8.0.zip.003"

[13:03:54.690] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Download: FileName "GenshinImpact_4.8.0.zip.005", Url "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/GenshinImpact_4.8.0.zip.005"

[13:03:54.690] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Download: FileName "GenshinImpact_4.8.0.zip.004", Url "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/GenshinImpact_4.8.0.zip.004"

[13:03:54.690] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Download: FileName "GenshinImpact_4.8.0.zip.006", Url "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/GenshinImpact_4.8.0.zip.006"

[13:03:54.690] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Download: FileName "Audio_Chinese_4.8.0.zip", Url "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/Audio_Chinese_4.8.0.zip"

[13:03:54.690] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Download: FileName "Audio_English(US)_4.8.0.zip", Url "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/Audio_English(US)_4.8.0.zip"

[13:03:54.690] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Download: FileName "Audio_Korean_4.8.0.zip", Url "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/Audio_Korean_4.8.0.zip"

[13:03:54.691] [INFO] Starward.Services.InstallGame.GenshinInstallGameService
Download: FileName "Audio_Japanese_4.8.0.zip", Url "https://autopatchhk.yuanshen.com/client_app/download/pc_zip/20240705190845_sEVV7ebRBuMrOheE/Audio_Japanese_4.8.0.zip"

[13:08:32.729] [EROR] Starward.Services.InstallGame.GenshinInstallGameService
Install game error.
System.Net.Http.HttpIOException: The response ended prematurely, with at least<PHONE_NUMBER> additional bytes expected. (ResponseEnded)
   at System.Net.Http.HttpConnection.ContentLengthReadStream.ReadAsync(Memory`1, CancellationToken)
   at Starward.Services.InstallGame.InstallGameService.DownloadFileAsync(DownloadFileTask task, CancellationToken cancellationToken, Boolean noTmp) in D:\a\Starward\Starward\src\Starward\Services\InstallGame\InstallGameService.cs:line 813
   at Starward.Services.InstallGame.InstallGameService.<>c__DisplayClass76_0.<<DownloadAsync>b__2>d.MoveNext() in D:\a\Starward\Starward\src\Starward\Services\InstallGame\InstallGameService.cs:line 546
--- End of stack trace from previous location ---
   at System.Threading.Tasks.Parallel.<>c__53`1.<<ForEachAsync>b__53_0>d.MoveNext()
--- End of stack trace from previous location ---
   at Starward.Services.InstallGame.InstallGameService.DownloadAsync(CancellationToken cancellationToken, Boolean noTmp) in D:\a\Starward\Starward\src\Starward\Services\InstallGame\InstallGameService.cs:line 540
   at Starward.Services.InstallGame.InstallGameService.StartAsync(Boolean skipVerify) in D:\a\Starward\Starward\src\Starward\Services\InstallGame\InstallGameService.cs:line 251

Additional Context
No response

It's a network error. The automatic retry mechanism will be added on latter version, and the download experience will be better


⚠ Bofrost.at has degraded performance

In bba2371, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1885 ms

Resolved: Bofrost.at performance has improved in a765847 after 1 hour, 33 minutes.


Merge working features into master for live deployment.

What:
Merge footer, flick-of-the-pen, and araths sanity changes into the live site.
Why:
Someones gotta do it.

Never merge your own changes!
But it's a strange time.


chore: move ledgers to di

Summary of Changes
Move the ledgers from config so they are managed and access through dependency injection.
Related Issues
n/a
Pull Request Checklist
Tick all boxes below to demonstrate that you have completed the respective task. If the item does not apply to your this PR check it anyway to make it apparent that there's nothing to do.

[x] All commits contain a DCO Signed-off-by line (we use the DCO GitHub app to enforce this);
[x] Updated LICENSE-3RD-PARTY.md for any added dependencies or vendored components;
[x] Updated documentation as needed for changed code and new or modified features;
[x] Added sufficient tests so that overall code coverage is not reduced.

If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!
Pro Tip 🤓

Read our contribution guide at least once; it will save you a few review cycles!
Your PR will likely not be reviewed until all the above boxes are checked and all automated tests have passed.

PR template adapted from the Python attrs project.

Is there a way to inject the agent modules (getAgentModules from /utils/agent.ts), which is the one that uses those ledger configs?
If it's possible, even the agent modules would be injectable, allowing us to use a custom did resolver for example.


no PR triggers on release builds

Description of the change
This prevents the release builds to be triggered by a pull request.

:+1: This seems reasonable. It matches what CI runs upstream is actually doing. (I wonder if they have set this in the DevOps UI?)

Probably they have set this using UI. I will make a PR to upstream as well.

I'd like to merge this, if you're okay with that.

Let me write a better issue text. 😄 I want to follow the template.

I will merge this together with #31. I want to avoid failed tests on the master.

This is merged upstream!
https://github.com/atom/atom/pull/21061


[BUG] BlobClient.ExistsAsync with URI throws exception in Blazor Webassembly

Library name and version
Azure.Storate.Blobs 12.16.0
Describe the bug
Calling blobClient.ExistsAsync() in a balzor web assembly application causes an exception.
This seems to be new behaivour with existing code - as this used to work fine - but possibly something in the azure side has changed?
Expected behavior
Should return true if the blob exists and return false if it does not exist.
Actual behavior
throws an exception:
TypeError: Cannot read properties of null (reading 'getReader')
Reproduction Steps
Create a new blank webassembly application
add a reference to Azure.Storate.Blobs 12.16.0
on the index page add code::
@page "/"
@using Azure.Storage.Blobs
@using Azure.Storage.Sas

Hello, world!

<button @onclick="DoesItExist">Click</button>

exists @exists
message @message

@code
{
    string file = "FILENAME";
    string container = "CONTAINER_NAME";
    string connectionString = "CONNECTION_STRING";

    bool exists = false;
    string message = String.Empty;

    public async Task DoesItExist()

    {
        try
        {
            var serviceClient = new BlobServiceClient(connectionString);
            var containerClient = serviceClient.GetBlobContainerClient(container);
            var sasUri = GetServiceSasUriForContainer(containerClient, false);
            BlobContainerClient sasClient = new BlobContainerClient(sasUri);

            var blobClient = sasClient.GetBlobClient(file);

            if (await blobClient.ExistsAsync())
            {
                exists = true;
            }
            else
            {
                message = "is not there";
            }
        }
        catch (Exception ex)
        {
            message = ex.Message;
        }
    }

    private static Uri GetServiceSasUriForContainer(BlobContainerClient containerClient, bool asReadonly)
    {
        BlobSasBuilder sasBuilder = new BlobSasBuilder()
            {
                BlobContainerName = containerClient.Name,
                Resource = "c"
            };

        sasBuilder.ExpiresOn = DateTimeOffset.UtcNow.AddHours(1);
        if (asReadonly)
        {
            sasBuilder.SetPermissions(BlobContainerSasPermissions.List | BlobContainerSasPermissions.Read);
        }
        else
        {
            sasBuilder.SetPermissions(BlobContainerSasPermissions.All);
        }

        Uri sasUri = containerClient.GenerateSasUri(sasBuilder);
        return sasUri;
    }
}

Environment
WIndows 10 OS on local maching and also on azure webpps
.Net 7 (also seems to happen on .Net 6)

I have same issue, I am using a Connection string directly. Here my code just when try to get the BlobContent
        try         {             BlobClient blobClient = GetBlogClient(container, fileName);             BlobDownloadResult response = await blobClient.DownloadContentAsync();             return response.Content.ToArray();         }         catch         {             return new byte[0];         }
I get a exception about max retry, I didn't modify defatul value about 6 retry.The exception also is fired if you try to request for
if(await blobClient.ExistsAsync())             {                 BlobDownloadResult response = await blobClient.DownloadContentAsync();                 return response.Content.ToArray();             }
But BlobClient get correctly the Blob Uri and I can request for the Uri directly like a link in my Blazoe WebAssembly App.
I want to recovery teh bytes to avoid cache issues I am having with the browsers when only use the Blob Uri.

Adding service attention label.

Does this issue only occur in Blazor? Because TypeError: Cannot read properties of null (reading 'getReader') isn't an error we return?

I have only reproduced this error in Blazor WASM.
The error started happening on a production site without any deployment or code change. As far as I'm aware nothing updated on the site, we just came in one morning to find the logs full of errors and traced it back to this issue. We have since patched the site to not bother checking the blob exists and just handling the not found error as needed.
I originally raised this with the Azure team on the assumption that something must have changed at a platform level which changed the response the blob store returned for this method, but they insisted it was a code error and nothing had changed on the azure platform - so unfortunately I don't have much information about quite where this bug might have been introduced.

This repo handles Azure SDK issues. So I can't speak for Blazor WASM and any other azure platforms you may be using.
This doesn't sound like an Azure.Storage.Blobs issue since you didn't upgrade the version and it happened out of nowhere.
Is there more logs or stack traces behind this, that could help us narrow down if this is an Azure Storage SDK issue or not?

I have reloaded the demo code that I used to repro this issue and the issue seems to be resolved. So, again, the same code, the same data, nothing changed in the source, and the issue has gone away. This again, points to a problem somewhere in the Azure platform which is either intermittent or now resolved.

I just tried this with the latest Azure.Storage.Blobs 12.19.0-beta and it gives the error.


updated indirect dependency

we need to update dompurify, which is an indirect dependency. here is how i did it:

in package.json add a resolution forcing dompurify to 2.4.9
yarn install (this updates yarn.lock, but not the way we want it)
revert the change in package.json
remove node_modules (rm -rf node_modules)
yarn install
commit yarn.lock

closing, will do https://github.com/grafana/grafana-json-datasource/pull/419


Create inband management tests to loopback0 interfaces

Change Summary
Types of changes

[ ] 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)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes)
[x] Documentation content changes
[ ] Other (please describe):

Related Issue(s)

Fixes issue #668
Component(s) name
arista.avd.eos_validate_state
Proposed changes
loopback0 reachability has added tests for L2 leafs, sourcing from inb management (validated on routing table, so no specific source).
Reporting in CSV/MD files are also added.
How to test

Checklist:

[x] My code has been rebased from devel before I start
[x] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have updated molecule CI testing accordingly
[ ] All new and existing tests passed (pre-commit, make linting and make sanity-lint).

@xaviramon Please try to make this more device type neutral, so we just call it "Inband Management test to loopback0" or similar, and you can use the new variables switch_inband_management_ip and switch_inband_management_ip to test if we have inband management available and to set as source if needed.

@ClausHolbechArista @onurgashi  upgraded as discussed

400+ files changed hahaha! =D

Please @onurgashi and @ClausHolbechArista validate if that works.
Discussed with @titom73 and I can't see a better way to validate inband source interface and iterate over all loopback0 defined.
Improved reporting during ansible execution, so user can validate which interface is being validated on which device.
At reporting, executed ping command is parsed in order to get last word, corresponding to source interface.


Not able to connect to slack

From the apache pinot official webpage , I tried to connect to slack channel it gave me an error like this

try this, apache-pinot.slack.com

cc @xiangfu0


feature request: add possibility even if camera provide snapshot url to make one from the stream

I bought IMOU Bullet 2E 4MP camera, but it provides screenshots at low resolution. I changed fast and durty the code of your adapter to always make screenshot from the video, not using one from the camera. Would be nice if you can add in the setup of the adapter possibility not to use screenshot from the camera if it is there ( or change the logic, if one selects to generate screenshot from video, then ignore the one provided by camera).
Thank you!

An ffmpeg error occurred: Output stream closed occurs in the logs, but screenshot is made fine.

as always - add versions:
adapter:
admin:
js-controller:
node:
o/s:
And please add complet log from adapter

as soon as you request screenshot from adapter, generating one from stream video, this error appears in the log: An ffmpeg error occurred: Output stream closed. I guess it is from onvif library, that you are using. I also noticed that after several hours one need to restart onvif adapter, otherwise it stops to react somehow and the values are not updated anymore (that camera detected any movement). as soon as one restarts adapter, all start to work again.

do you have logs.. normally the adapter relogin every 30min to the camera

onvif.0
2023-09-01 18:56:36.400	error	An ffmpeg error occurred: Output stream closed
onvif.0
2023-09-01 18:55:12.839	info	Starting snapshot server
onvif.0
2023-09-01 18:55:12.838	info	Finished onvif discovery
onvif.0
2023-09-01 18:55:07.833	info	Start onvif discovery
onvif.0
2023-09-01 18:55:05.527	warn	No presets found for <IP_ADDRESS>:80 Error: ONVIF SOAP Fault: This optional method is not implemented
onvif.0
2023-09-01 18:55:03.240	info	Device successful initialized: <IP_ADDRESS>:80
onvif.0
2023-09-01 18:55:01.180	info	Found Adapter Device: onvif.0.192_168_1_145_80 LCIPC-F42F <IP_ADDRESS>:80
onvif.0
2023-09-01 18:55:01.166	info	starting. Version 1.0.5 in /opt/iobroker/node_modules/iobroker.onvif, node: v18.17.0, js-controller: 4.0.24

Maybe you can enable debug log and check if every 30min a reconnect happens

will collect some logs and will send later


Skeleton for Wallet App

Closes #5.

Extracted shared logic to "logic" package.
Extracted shared theme to "design" package.
Changed fonts approach, since there is no way to load local font files from a directory outside package root. Now they are acquired from Google Fonts.

To do:

Find a way to serve components with default less styles from "design" package.
Make a "components" directory in the "design" package with the layout components and the shared components from apps (like Login).
Extract common devDependencies to monorepo.
Extract common tsconfig, eslint, etc. config files.

I'm not sure if I should extract the dependencies to the lerna root right away.

I would only pull out devDependencies to lerna root. Feel free to do that now.
When you find yourself cut-and-pasting code (components), then it would be good to make a separate package for that - one for components, another for logic. I think you will get much more of that in the next pr.

Now is a great time to start doing that refactoring

Great, I wasn't sure if it was the right moment.

I would say those TODOs are nice, but not needed in this PR.

Find a way to serve components with default less styles from "design" package.
Make a "components" directory in the "design" package with the layout components and the shared components from apps (like Login).

I would work on that when you start building out all the component flow of the wallet and find all the components you want to reuse, so you can test the approach on more than one example.
This PR is big enough, time for a new one :)


KYLIN-3547 add boolean DataTypeOrder

https://issues.apache.org/jira/browse/KYLIN-3547

Can one of the admins verify this patch?


Updated to php-coveralls/php-coveralls

With version 2 package has been renamed from satooshi/php-coveralls to php-coveralls/php-coveralls, and the script has been renamed from coveralls to php-coveralls

Thanks, @webimpress


Chatbot Agent Tools deviates from prompt instruction not staying within index context

Hi guys, wondering if is it possible to ensure indexes do not deviate from the actual provided context within agent tools
index = GPTSimpleVectorIndex.from_documents(documents)
The above loads an index based on website data, below prompt seems to ensure any questions that deviate from context are not answered.
QQA_PROMPT_TMPL = (
    "We have provided context information below. \n"
    "---------------------\n"
    "{context_str}"
    "\n---------------------\n"
    "Given this information, please answer only based on context if the question is not part of context do not answer. Instead respond with I don't know: {query_str} also return back the context used as part of the answer\n"
)
QA_PROMPT = QuestionAnswerPrompt(QA_PROMPT_TMPL)

print(index.query("Who is the president of france",
                  text_qa_template=QA_PROMPT,
                  response_mode="compact"))

This returns
I don't know. Context: title: Reldyn, length: 1215, excerpt: Leverage digital transformation with us for efficient and better customer experience., byline: None, dir: None, lang: en, siteName: None.
.
Tried a similar approach using agents and tools where the tools loads up the GPTSimpleVectorIndex based off the same web content as above example, at times it works perfectly. Sometimes it just loses the instruction to keep within the context. Any points to ensure am approaching this correctly would be appreciated.

 prompt = PromptTemplate(
        template="""You are a personal assistant for Reldyn company your job is to answer questions. Use only context Website Index to provide answers.
                    Do not provide any answers that deviate from your tools documents. If you don't know the answer, just say "Hmm, Im not sure." Don't try to make up an answer up.:
                   --------
                   Question: {query}
                  """,
        input_variables=["query"],
    )

tools = [Tool(
    name="Website Index",
    func=lambda q: str(index.query(q)),
    description=f"useful for when you want to answer questions from indexed text. Always, you must try the index first, only answer based on index text",
)]

memory = ConversationBufferMemory(memory_key="chat_history")
agent_chain = initialize_agent(
    tools,
    llm,
    agent="zero-shot-react-description",
    memory=memory,
    verbose=True,
    max_iterations=5,
)

agent_prompt = prompt.format(query=user_input)
response = str(agent_chain.run(agent_prompt)).strip()

Got this to work by providing more strict instructions to focus on the index context and the tool description. the key seems to be playing around with the prompt and tool description. Being too strict produces more direct responses but also limits the range of answers. This might be good for use cases where you ideally don't want the bot deviating and going off on a tangent

Super interesting investigations. Ya, right now there's not too many great ways to fix hallucination beyond prompt engineering. We hope the future instruction fine-tuned models will be better and better.


[Question] How to authenticate with the Microsoft Graph API from bot app?

Bot Info

SDK Platform: Node.js
Active Channels: Cortana
Deployment Environment: Azure App Service

Issue Description
I have a bot service running on node.js with Cortana channel enabled. I have also enabled 'user sign-in' option at skill invocation to manage user identity.
https://docs.microsoft.com/en-us/cortana/skills/configure-connected-account
Moreover, I have created an app on apps.dev.microsoft.com which has permissions to access our MS graph. When I use graph explorer in browser and login with my corp. account I can get results from following GET.
https://developer.microsoft.com/en-us/graph/docs/api-reference/v1.0/api/calendar_list_events
but, when I access same code from bot I get error "Bearer access token is empty.". After digging around I found we need access token to use graph Api. But, currently we are facing sign-in issue for our user as we use Azure AD as identity provider and not Microsoft identity server. Looks like Cortana can't be configured to use AAD yet.
So in this case, is there any other way to access MS graph Api in bot service? Please make a note that we can have an app which has access to ms graph and real issue is use can't sign in to Cortana due to AD vs MS AD issue otherwise we would have used following code to grab access token.
var authorizationToken = session.message.entities.find((e) => {
return e.type === 'AuthorizationToken';
}); // we cant use this as user's are enable to sign in yet.
Code Example
var url = "https://graph.microsoft.com/v1.0/users/{office 365emailAddress}/calendar/events"
        var meetingEvents = [];
        request(url, function (error, response, body) {
            if (body) {
                var json = JSON.parse(body);
                if (json.value) {
                    meetingEvents.push(json)
                } else {
                    return;
                }
            }
            if(error){
                reject({error: error});
              }
        });

Expected Behavior
I would like to get API result without having to user logged in as I have an app which has access to ms graph.

+1

Finally, I was able to resolve it by having to have 2 apps.
App1 - Deals with just user sign-in from AAD V1
App 2 - Has full access to graph with App permissions (not delegated) which I use to get token and call graph API.


fix: version removed from plugin group

What type of PR is this?

/kind bug
What this PR does / why we need it:
The plugin group includes the version, which is wrong. This PR removes the version from the group.
Which issue(s) this PR fixes:

Fixes #
Does this PR introduce a user-facing change?:

NONE

/approve


Loading code in demos should link back to "here's how to load"

Describe the bug
The dataset module makes our demos cleaner, but it does make it harder for a user to work out how to load their own data. We should link to the demos for loading data to clarify this.
(StellarGraph 1.0.0rc1 dogfooding)

Hidden loading code in datasets class, need to point back to it.
Could be automated.
More notebook operations...
Is important - was consistent in dogfooding that users didn't know how to load data and had to work it out.
Could do manually in each notebook but preference is to do automatically.
Maybe when dataset load is called, that can print output back to notebook formatted with link to other notebooks to explain how to load data, and readTheDocs for loading function.
or
Tag loading cell and markdown cell before it with a link or description to appropriate destination.


Missing file for bare-metal installation

Running on a MacOSX 10.14.6, with pip 21.1.1. Grabbing the requirements file throws an error. I'd like to install the bits to pull out bits to examine the functionality of the code.
Referencing the instructions : https://mriqc.readthedocs.io/en/latest/install.html
pip install -r https://raw.githubusercontent.com/poldracklab/mriqc/master/requirements.txt
ERROR: 404 Client Error: Not Found for url: https://raw.githubusercontent.com/poldracklab/mriqc/master/requirements.txt

I believe the documentation is out of date - just skipping that pip install should do.

thanks, i'll give a try later today -- just noticed you reply.

Documentation has been updated. Please reopen if this problem persists.


loadScene vs loadSceneHierarchy/loadSceneSettings

According to https://github.com/playcanvas/engine/pull/284 loadScene is meant for engine users, and loadSceneHierarchy and loadSceneSettings are meant for convenience. However, loadScene is not documented.

Is loadScene a public API?
What is the behavior difference between the different loaders?

loadScene was made public in release 1.26.1.


Depostion form - Save button difficult to reach

Tester notes that:
It is possible to scroll down in the page so that you aren’t able to view the “Save” button at the top of the page, so you have to scroll all the way up to save, a bit annoying. Would be nicer if it is more easily accessed during the course of input, to ensure that people can do so regularly during the process.

Yes, will be implemented to have it available along the deposition form.


feature: Add blink.cmp hl groups

Did you check the docs?

[X] I have read all the solarized-osaka.nvim docs

Is your feature request related to a problem? Please describe.
blink.cmp provides option to use nvim-cmp hl groups for the time being but that is marked as something that will be removed in the future.
Currently if you are using it with that behavior a lot of colors are not working properly and the completion menu does not seem to be slightly transparent but instead renders as solid color.
Also the windows that show additional help related to an entry are also transparent and exhibiting such behavior making it difficult to use.
Describe the solution you'd like
Properly styled hl groups so the behavior is consistent.
Describe alternatives you've considered
Styled some to use for the time being but the menu issues i could not figure out.
My guess is that float windows that appear for menu and the additional info windows use different attributes compared to what nvim-cmp might be using.
BlinkCmp highlight group for blink.cmp uses NormalFloat for documentation windows which might be different from what nvim-cmp uses but couldn't find exactly what nvim-cmp is using.H
Similarly the completion menu window BlinkCmpMenu highlight is a PMenu which might be different from nvim-cmp defaults.
Additional context
I can make a pull request with defaults as same as nvim-cmp but the types of windows and their different behaviors are something I'm not knowledgeable about so better for someone who understands it to implement.

Hey, @Redoxahmii! Thanks so much for your contributions to this theme.
I'm currently busy with my project and can't find spare time to maintain the theme at the moment.
I'd be happy to welcome you as a collaborator of this repository.
I've sent an invitation, so please join it if you are interested.
There is absolutely no problem with adding blink.cmp.
Just like TokyoNight.nvim, it'd be nice to do some refactoring to split theme.lua into groups:
https://github.com/folke/tokyonight.nvim/tree/main/lua/tokyonight/groups

Thank you for the invite.
I had some issue with how the colors were being displayed when working with blink.cmp but for the time being I will add the colors so they are the same as nvim-cmp.
Could you please tell how I should push code now i would prefer not pushing into the repo directly but create a pull request so we both can take a look before it gets pushed into main.
Or I can create a separate branch in the main project where I could implement and than merge later.

For the refactoring part I will take a look into tokyonight and if it seems probable create a pull for it.

You can create PRs and merge them freely if they are minor changes, so I can quickly track the changes you made.
If you would like to make a major change or something that you think would need my review like the refactoring work, add me as a reviewer in the PR.

fixed in https://github.com/craftzdog/solarized-osaka.nvim/pull/49


End standout escape sequence not working in preview.

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

Problem / Steps to reproduce
The start standout escape sequence works fine in the preview, but the corresponding end sequence appears to not be working. The reset-all sequence works fine, and does end the standout mode.
echo "AAA $(tput smso)BBB$(tput rmso) CCC$(tput sgr0) DDD"
echo foo | fzf --preview='echo "AAA $(tput smso)BBB$(tput rmso) CCC$(tput sgr0) DDD"' --height=5

Thanks for the report. It should be fixed now.


Support for sending messages synchronously to the Kafka broker.

Adds two fields to the producer-configuration that allows to set the send mode (async or sync) and an optional timeout in sync mode. Default behavior is async.
This allows for an upstream application to verify that the message have been handed off to Kafka following the delivery options specified, and not just residing in memory at the producer.

Hi Martin,
Thanks for the contribution. Please sign the CLA as explained here (if you didn't already).
Thanks,
Marius

Thanks for the feedback Marius.
I changed as you suggested and signed the CLA, and as asked in the readme:
I have signed and agree to the terms of the SpringSource Individual Contributor License Agreement.

Merged w/minor updates here: https://github.com/spring-projects/spring-integration-kafka/commit/14110e3741722e09ab64848aa1d8da9c1b500941


API: Added goto_repodir(3) to help new module developers

Fixed issue #362.
This commit is to append goto_repodir(3) API in order that the developer
can make new module more easily.
Changes proposed in this PR:

Added goto_repodir(3)
Updated the existing document

Signed-off-by: Geunsik Lim<EMAIL_ADDRESS>
[Template] PR Description
In general, github system automatically copies your commit message for your convenience.
Please remove unused part of the template after writing your own PR description with this template.
$ git commit -s filename1 filename2 ... [enter]

Summarize changes in around 50 characters or less

More detailed explanatory text, if necessary. Wrap it to about 72
characters or so. In some contexts, the first line is treated as the
subject of the commit and the rest of the text as the body. The
blank line separating the summary from the body is critical;
various tools like `log`, `shortlog` and `rebase` can get confused
if you run the two together.

Further paragraphs come after blank lines.

**Changes proposed in this PR:**
- Bullet points are okay, too
- Typically a hyphen or asterisk is used for the bullet, preceded
  by a single space, with blank lines in between, but conventions vary here.

Resolves: #123
See also: #456, #789

**Self evaluation:**
1. Build test: [ ]Passed [ ]Failed [*]Skipped
2. Run test: [ ]Passed [ ]Failed [* ]Skipped

**How to evaluate:**
1. Describe how to evaluate in order to be reproduced by reviewer(s).

Add signed-off message automatically by running **$git commit -s ...** command.

$ git push origin <your_branch_name>

:memo: Version: 1.20180820. Thank you for submitting PR #397. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://invain.mooo.com/.


update(docs): sync the docs with chaos-chart via embedmd command

Signed-off-by<EMAIL_ADDRESS>This PR is for:

sync the docs with chaos-chart via embedmd command

Whenever you will have the next PR in chos-charts try to add the unit of NETWORK_DELAY unit.


Create a basic Program Interface

Description
Adds a very basic interface for programs that only allows getting and create (basic).
Paired with @kendrickt
Checklist

[ ] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

Issue(s)
Fixes #98

Thoughts on including actions on blocks in this PR?

Messed up git, so re-created with bug fixes in https://github.com/seattle-uat/universal-application-tool/pull/125


libcurl: Use CMake to compile on Android

Fixes Android on Windows
I had to pass a lot of options (I only supported option using lib supporting Android) and made a handledependency function for that.
It seems to be a common need, wouldn't it be nice to make it part of cmake, maybe in package.tools.cmake?

It will fail on Windows until https://github.com/xmake-io/xmake-repo/pull/3643 is merged

It fails on Android because fseeko is only available starting with API 24, and CMake wrongly detects it as available. Maybe a missing CMake config, or a CMake bug?
check_symbol_exists(fseeko         "${CURL_INCLUDES};stdio.h" HAVE_FSEEKO)

#if __ANDROID_API__ >= 24
int fgetpos(FILE* _Nonnull __fp, fpos_t* _Nonnull __pos) __RENAME(fgetpos64) __INTRODUCED_IN(24);
int fsetpos(FILE* _Nonnull __fp, const fpos_t* _Nonnull __pos) __RENAME(fsetpos64) __INTRODUCED_IN(24);
int fseeko(FILE* _Nonnull __fp, off_t __offset, int __whence) __RENAME(fseeko64) __INTRODUCED_IN(24);
off_t ftello(FILE* _Nonnull __fp) __RENAME(ftello64) __INTRODUCED_IN(24);
#endif /* __ANDROID_API__ >= 24 */