Can we use gzip compressed files for GitHub pages?

JavaScript files generated by GopherJS is quite big. Further more, their revision history don't have to be kept. Let's compress them or move them away from the repository.

Created https://github.com/hajimehoshi/ebiten.pagestorage. Let's manage large files there!


Sweetie Bot doesn't recognize some collections and commands

!cute is returning "invalid command"
While the alias !pony is returning "collection does not exist"
!bucket returns "invalid command"

This was an issue with the aliases.


Support for ONNX mdoels with external tensors

This Pull Request enables Model class to handle ONNX models that use external files to store tensor data. In particular, it enables using models that exceed 2GB in size.
Key Changes:

Introduction of use_ext_tensors parameter:
Several functions now include a use_ext_tensors parameter. This new parameter dictates whether the model should operate with tensor data stored externally.

Setting the base_dir parameter:
To ensure the ONNX library can accurately locate the external tensor files, the base_dir parameter has been set in the numpy_helper.to_array() function. This necessitated adding a base_path parameter to several related functions.

New save_onnx_model() function:
Calls to onnx.save() are wrapped in a new function save_onnx_model(). This function is responsible for

enabling use of external tensors,
setting the ext_tensor_filename parameter (when provided)
removing any previous versions of the tensor file.

By removing outdated data, this ensures that the md5 digest calculation in the td module works as expected.

LGTM


Clear project settings when "Generate New Project" is selected

After generating a project and downloading it, clicking on "Generate new project" does not clear out the extensions or the project settings (GAV, etc). Any project info should be reset to default values when "Generate new project" is selected.

This is fixed


Fix not stopping dev server on sigterm.

If you run npm run dev, and then close app with cmd+q, dev server is still opened.

We already use concurrently --kill-others to run the dev script. Could this case still happened?

Yes concurrently sends SIGTERM to our dev server but It doesn't close well.
Check this video: https://yadi.sk/d/Xkc3qCZ1rjNk8
also I added closing webpackDevMiddleware. Without it build after quiting app does not wait until bundle, in next start. You can see it on second npm run dev in video

Thanks!


Async/Await adaptation?

Hi, my question is:
How would it work if you adapt to promises in NodeJS?

I should do so in the core code - I agree, it's a bit old code by now... I could improve on the interface quite a lot I believe. But I dislike many other details as well, so it would probably lead to a longer project which I don't feel I have time for right now, but could have in the near future due to what we are working with at my day to day work.
I keep this open to remind me and to tell you when, if I do, make such an update.


Error using multiple server/repos

Hi,
at first thanks for you plugin! It makes it easier to work with the AWS Codeartifact :)
At the moment we face an issue, if we work with several servers/profiles
<server>
   <id>server1</id>
   <username>user1</username>
   <password></password>
</server>
<server>
   <id>server1-snapshots</id>
   <username>user1</username> <!-- Same user as the first one -->
   <password></password>
</server>
<profiles>
        <profile>
            <id>server1</id>
            <activation>
                <activeByDefault>true</activeByDefault>
            </activation>
            <repositories>
                <repository>
                    <id>server1</id>
                    <url>...</url>
                    <snapshots>
                        <enabled>false</enabled>
                    </snapshots>
                    <releases>
                        <enabled>true</enabled>
                    </releases>
                </repository>
                <repository>
                    <id>server1-snapshots</id>
                    <url>...</url>
                    <snapshots>
                        <enabled>true</enabled>
                    </snapshots>
                    <releases>
                        <enabled>false</enabled>
                    </releases>
                </repository>
            </repositories>
        </profile>
    </profiles>

I can generate the credentials for server1, but if I try this for server1-snapshots, I get the following error:

I also tried to separate the repositories to two different profiles, but that does not do the trick.
Do I have to configure it differently?
Based on https://github.com/coderazzi/aws_codeartifact_maven/issues/3 I think your plugin should support it?

Hi, could you send me your settings.xml and .aws folder, removing any
sensitive information?
On Mon, Sep 18, 2023 at 6:07 AM philkohl @.***> wrote:

Hi,
at first thanks for you plugin! It makes it easier to work with the AWS
Codeartifact :)
At the moment we face an issue, if we work with several servers/profiles

server1
user1

server1-snapshots
user1

server1

true

server1
...

false

true

server1-snapshots
...

true

false

I can generate the credentials for server1, but if I try this for
server1-snapshots, I get the following error:
[image: image]
https://user-images.githubusercontent.com/65112414/268565035-cf4698a0-7e4b-433e-b696-65b74514d3ca.png
I also tried to separate the repositories to two different profiles, but
that does not do the trick.
Do I have to configure it differently?
Based on #3 https://github.com/coderazzi/aws_codeartifact_maven/issues/3
I think your plugin should support it?
—
Reply to this email directly, view it on GitHub
https://github.com/coderazzi/aws_codeartifact_maven/issues/10, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH54IMNEMDOI2EH2GWTPPTX27QJ7ANCNFSM6AAAAAA44GEMSI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

aws_codeartifact_maven.zip
Sure, I attached the two .m2 settings that I tried. The aws config is very basic. Thanks for taking a look!

Hi, little, silly bug, ashamed to say. I will release tomorrow asap
the solution ....
On Mon, Sep 18, 2023 at 12:29 PM philkohl @.***> wrote:

aws_codeartifact_maven.zip
Sure, I attached the two .m2 settings that I tried. The aws config is very basic. Thanks for taking a look!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

Hi, I have released version 3.2.3, covering the bug you found.
You could download it from the beta channel and test it:
https://plugins.jetbrains.com/plugin/16777-aws-codeartifact--maven/versions/beta
Best,
Lu
On Mon, Sep 18, 2023 at 9:37 PM coderazzi @.***> wrote:

Hi, little, silly bug, ashamed to say. I will release tomorrow asap
the solution ....
On Mon, Sep 18, 2023 at 12:29 PM philkohl @.***> wrote:

aws_codeartifact_maven.zip
Sure, I attached the two .m2 settings that I tried. The aws config is very basic. Thanks for taking a look!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

Hi, thank you very much for the fix! Now it's working :)


[DO NOT MERGE] Swagger Documentation For Ledger Digest Upload

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

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
[x] Ensure to copy the existing version into new directory structure for first commit (including refactoring) and then push new changes including version updates in separate commits. This is required to review the changes efficiently.
Adding a new service

[x] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[x] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[x] Removing API(s) in stable version
[x] Removing properties in stable version
[x] Removing API version(s) in stable version
[x] Updating API in stable or public preview version with Breaking Change Validation errors
[x] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

All the failed checks are expected.

the systemData lint diff errors are expected.
the .net SDK errors are also expected and we are working on .net sdk repo now to fix it.

all comments should be addressed @j5lim

adding do not merge because feature is not yet in public preview, will update

Hi @rewongmicrosoft , you need to fix prettier check &  modal validation failure first to make it as ready for merge status.


Add Array::Constructor

This fixes a few corner cases when using array types. One specific was reported by @solnic
Dry::Types['params.array<params.integer>'].member.('1')
# => Dry::Types::CoercionError: "1" cannot be coerced to array
# from /Users/gordon/dev/dry-rb/dry-types/lib/dry/types/coercions/params.rb:125:in `to_ary'

Now it works fine. This is the same solution we have for hash constructors so it potentially can be generalized but atm I just want to fix it as it is.

Well, turns out it doesn't really work (breaks dry-schema) and we need a generalized way of determining type composability, I'll be working on this.


Rmove alluxio worker hostname from ALLUXIO_JAVA_OPTS

We need remove alluxio worker hostname from ALLUXIO_JAVA_OPTS  to avoid that it overrides the alluxio.worker.hostname in ALLUXIO_WORKER_JAVA_OPTS.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10149/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10150/
Test PASSed.

Merged build finished. Test PASSed.

alluxio-bot, merge this please


Bridge Detail shows on side

…ose it too
Description
Bridges Detail is now dynamically populating on the side once clicked and now can close on x button on it.

[ ] This is a bugfix.
[x] This is a new feature.
[] This is part of a feature.

Organization

[x] Have you linked this pull request to a Trello card?

Review

[ ] Have you added two reviewers to this pull request?
[ ] Is this a work in process?
[x] Is this ready to be reviewed?

 Map Display


Create index.d.ts

Pull Request Template
Description
Adds basic type definitions its not 100% perfect but its most of the api
Fixes #21
Type of change
Please delete options that are not relevant.

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

Checklist:

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

Amazing, thanks for the PR!


chore(tooltips): not aligned in demo

http://valor-software.com/ng2-bootstrap/#getting-started
The tooltips are not aligned w/ their elements.
The tooltip on Datepicker demo doesn't appear.

Problem is bigger than you think, modals\popups\tooltips are pain in the code in ng2 as for now
I need to properly export position service and reuse it across all elements

implemented tooltips demo now aligned

Possibly related: the v4 bootstrap tooltips are missing their arrows.

interesting things about arrows in bs4
they are missing in bs4 off documentation
tooltip container has all this new classes
and in bs4 tooltip can be attached only to document.body
tooltip bs-tether-element bs-tether-abutted bs-tether-abutted-top bs-tether-element-attached-bottom bs-tether-element-attached-center bs-tether-target-attached-top bs-tether-target-attached-center fade bs-tether-enabled in

Does this mean that tether.min.js needs to be included alongside ng-bootstrap as per the v4 documentation?

not sure to be honest, I see where it is came from
https://github.com/twbs/bootstrap/blob/v4-dev/js/src/tooltip.js#L36
right now I am fixing system.js bundles, I will get to this a bit later


Issue about dragging the item to the last row of the table.

Hi,
First a great and easy to use plugin.
Second, I followed the example at here https://amendx.github.io/vue-dndrop/examples/table-drag.html. I found out that if I want to drag any item to the last row of the table, it just 'bounces' back to original position. It also happens in the demo from the documentation. Is there any way to fix it?
I tried this fix here https://github.com/amendx/vue-dndrop/issues/24 but I guess it's not really related to my issue though.
I have also read some sites saying I can change the container height, but not so sure how to actually do it. It would be awesome if you could give me a little bit more detailed description. Thanks!

Hi, @babyghost-ys. I'll be taking a look on that issue right away. Hold on!

It's because the "ghost" element is always added to the end of the list and the ghost element has pointer-events none set on it.  So the hitElement never registers the last ghost space in the table as a result of elementFromPoint.  The browser won't ever return a tbody for elementFromPoint, so the hitElement goes up to the table, and that's not a child of Container.
That results in the "pos" of dragResult being null, and that results in the cancelation of the drop.
// dropPlaceholderContainer.style.pointerEvents = 'none';

// ghost.style.pointerEvents = 'none';

// css[("." + ghostClass + " *")] = {
//     'pointer-events': 'none'
//   };

I tried moving the container up a level to be the table, but too many places require the list of dragables to be direct children of the container.


Content Management System

Dependent upon input from NASA Web Modernization working group

📆 february status: deferred to focus on developing a more refined plan: #70


Segementation Fault Running setup.sh on Raspbian

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:
When running sudo bash setup.sh config.json -s 123456, the SDK errors out when building with the following error:
setup.sh: line 252:  8922 Segmentation fault      cmake "$SOURCE_PATH/avs-device-sdk" -DCMAKE_BUILD_TYPE=DEBUG "${CMAKE_PLATFORM_SPECIFIC[@]}"
What is the expected behavior?
It should build successfully.
What behavior are you observing?
Failed build with segmentation fault.
==============> CLONING SDK ==============

Cloning into 'avs-device-sdk'...
remote: Enumerating objects: 16, done.
remote: Counting objects: 100% (16/16), done.
remote: Compressing objects: 100% (16/16), done.
remote: Total 8204 (delta 1), reused 7 (delta 0), pack-reused 8188
Receiving objects: 100% (8204/8204), 13.65 MiB | 5.72 MiB/s, done.
Resolving deltas: 100% (3596/3596), done.

==============> BUILDING SDK ==============

setup.sh: line 252:  8922 Segmentation fault      cmake "$SOURCE_PATH/avs-device-sdk" -DCMAKE_BUILD_TYPE=DEBUG "${CMAKE_PLATFORM_SPECIFIC[@]}"

Provide the steps to reproduce the issue, if applicable:
Download files.
Run sudo bash setup.sh config.json -s 123456.
Get to BUILDING SDK.
Get segmentation fault.
Post issue to GitHub.
Tell us about your environment:
pi@pi3:~ $ uname -a
Linux pi3 4.14.79-v7+ #1159 SMP Sun Nov 4 17:50:20 GMT 2018 armv7l GNU/Linux
pi@pi3:~ $ cat /etc/issue
Raspbian GNU/Linux 9 \n \l
pi@pi3:~ $ cat /etc/os-release
PRETTY_NAME="Raspbian GNU/Linux 9 (stretch)"
NAME="Raspbian GNU/Linux"
VERSION_ID="9"
VERSION="9 (stretch)"
ID=raspbian
ID_LIKE=debian
HOME_URL="http://www.raspbian.org/"
SUPPORT_URL="http://www.raspbian.org/RaspbianForums"
BUG_REPORT_URL="http://www.raspbian.org/RaspbianBugs"
pi@pi3:~ $ lsb_release -da
No LSB modules are available.
Distributor ID: Raspbian
Description:    DebianEdu/Skolelinux
Release:        9.6
Codename:       stretch
pi@pi3:~ $ hostnamectl
   Static hostname: pi3
         Icon name: computer
        Machine ID: 8c5604e85c924195aef34a3e4079998d
           Boot ID: 843c1eec829c4099b664a9fb1d96b4d4
  Operating System: Raspbian GNU/Linux 9 (stretch)
            Kernel: Linux 4.14.79-v7+
      Architecture: arm
pi@pi3:~ $ sudo apt list --upgradeable
Listing... Done

What version of the AVS Device SDK are you using?
  1.11

Tell us what hardware you're using:

[ ] Desktop / Laptop
[x] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[ ] Linux
[ ] MacOS
[x] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

Also, for whatever it's worth, I noticed #878 was closed without ever finding a resolution after some more searching, so here is was was requested there:
pi@pi3:~/alexa $ ls
avs-device-sdk  build  config.json  db  genConfig.sh  pi.sh  setup.sh  sounds  third-party
pi@pi3:~/alexa $ cd build
pi@pi3:~/alexa/build $ ls
pi@pi3:~/alexa/build $ cmake ../avs-device-sdk --debug-output \
>     -DSENSORY_KEY_WORD_DETECTOR=ON \
>     -DSENSORY_KEY_WORD_DETECTOR_LIB_PATH=../third-party/alexa-rpi/lib/libsnsr.a \
>     -DSENSORY_KEY_WORD_DETECTOR_INCLUDE_DIR=../third-party/alexa-rpi/include \
>     -DGSTREAMER_MEDIA_PLAYER=ON -DPORTAUDIO=ON \
>     -DPORTAUDIO_LIB_PATH="../third-party/portaudio/lib/.libs/libportaudio.a" \
>     -DPORTAUDIO_INCLUDE_DIR="../third-party/portaudio/include" \
>     -DCMAKE_BUILD_TYPE=DEBUG
Segmentation fault
pi@pi3:~/alexa/build $ cmake ../avs-device-sdk --trace \
>     -DSENSORY_KEY_WORD_DETECTOR=ON \
>     -DSENSORY_KEY_WORD_DETECTOR_LIB_PATH=../third-party/alexa-rpi/lib/libsnsr.a \
>     -DSENSORY_KEY_WORD_DETECTOR_INCLUDE_DIR=../third-party/alexa-rpi/include \
>     -DGSTREAMER_MEDIA_PLAYER=ON -DPORTAUDIO=ON \
>     -DPORTAUDIO_LIB_PATH="../third-party/portaudio/lib/.libs/libportaudio.a" \
>     -DPORTAUDIO_INCLUDE_DIR="../third-party/portaudio/include" \
>     -DCMAKE_BUILD_TYPE=DEBUG
Segmentation fault
pi@pi3:~/alexa/build $ apt -s install cmake
NOTE: This is only a simulation!
      apt needs root privileges for real execution.
      Keep also in mind that locking is deactivated,
      so don't depend on the relevance to the real current situation!
Reading package lists... Done
Building dependency tree
Reading state information... Done
cmake is already the newest version (3.7.2-1).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
pi@pi3:~/alexa/build $

Solved my own issue. Cmake itself is throwing the error no matter what. Either reinstall Raspbian or install the latest version of Cmake manually, for example:
sudo mkdir /opt/cmake
cd /opt/cmake
sudo wget https://github.com/Kitware/CMake/releases/download/v3.14.0-rc1/cmake-3.14.0-rc1-Linux-x86_64.sh
sudo sh cmake-3.14.0-rc1-Linux-x86_64.sh --prefix=/opt/cmake
sudo ln -s /opt/cmake/bin/cmake /usr/local/bin/cmake

Though you'll obviously want to get the latest cmake from here.
Hope anybody else who has this issue will find this!


Handle overflows more gracefully

Right now can get "stuck" at an ID -- probably just switch trying to compute 50 most recent after an id.

Donezo


Change colors for blockquote, forms etc...

Hey,
the blockquote element is by default red and the forms are teal by default. I didn't find a way to override it with classes. Is that planned for the future?
Phillipp

That's a good question, I guess for now you could do something like: http://codepen.io/anon/pen/eNPVNe

Yeah that could be a hotfix. Have my two colors in a config file so i can change the colors on the whole page by editing my config file. So it would be cool for the future to controll even blockquotes, forms and so on via classes.

You won't be able to override it with any prepackaged classes, but you can use Sass to change all the variables to your liking


Can´t migrate in Firebird3.0

I get this error when migrating, how can I increase this size or something else to be done?
raise OperationalError(message, gds_codes, sql_code)

django.db.utils.OperationalError: ('Dynamic SQL Error\nSQL error code = -104\nNa
me longer than database column size\n', -104)

Can you tell me what changes you've made to the model?
Please upload the migration file if you can!

Seems to be an Firebird engine implementation issue , could you test with Firebird 4.0 ?
https://stackoverflow.com/questions/20155517/is-it-possible-to-extend-firebird-table-name-length

Object name maxlenght matches the value of firebird4
https://github.com/nakagami/djfirebirdsql/blob/master/djfirebirdsql/operations.py#L305

In Firebird 3 maxlenght is 31

I see, I understand.
I don't want to support many versions of Firebird,
but Firebird 3.0 will still be used in the future, so I'll think about it.


test_add_builtin_account test is flaky

Problem
https://buildkite.com/anza/agave/builds/5840#01900827-c057-4097-b27d-796d0052dd5f
The last assertion in test_add_builtin_account failed:
        // No replacement should have happened
        assert_eq!(bank.get_account_modified_slot(&program_id).unwrap().1, slot);

Proposed Solution

So, new builtin accounts are created with rent epoch == 0 because INITIAL_RENT_EPOCH == 0 and this function is used in add_builtin_account:
    fn inherit_specially_retained_account_fields(
        &self,
        old_account: &Option<AccountSharedData>,
    ) -> InheritableAccountFields {
        const RENT_UNADJUSTED_INITIAL_BALANCE: u64 = 1;

        (
            old_account
                .as_ref()
                .map(|a| a.lamports())
                .unwrap_or(RENT_UNADJUSTED_INITIAL_BALANCE),
            old_account
                .as_ref()
                .map(|a| a.rent_epoch())
                .unwrap_or(INITIAL_RENT_EPOCH),
        )
    }

The reason this test is flaky is that sometimes the test builtin account gets rent collected in a partitioned rent pass and despite skip rewrites being enabled, we don't skip rewrites if the account in question has a rent epoch of 0:
    fn skip_rewrite(rent_amount: u64, account: &AccountSharedData) -> bool {
        // if rent was != 0
        // or special case for default rent value
        // these cannot be skipped and must be written
        rent_amount == 0 && account.rent_epoch() != 0
    }
    ```

cc @brooksprumo @jeffwashington, can one of you fix the test?

Btw I think it make more sense for builtins to be added with the RENT_EXEMPT_RENT_EPOCH but probably not important to fix because we are unlikely to add more builtins


Add title header on all pages in the organiser view

All organiser pages should have a title header. Pages in the organizer view in most cases do not have a title, e.g. the page “Call for Participation” does not show the title “Call for Participation”

Hi @mariobehling, I am claiming this Issue!


Anvil no longer returns event filter logs for impersonated accounts

Component
Anvil
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
anvil 0.1.0 (3ca8723 2023-01-28T00:05:49.406699Z)
What command(s) is the bug in?
No response
Operating System
None
Describe the bug
Seems that after https://github.com/foundry-rs/foundry/commit/b980e3cf97888a591e8f33a92fbe9f7cba364826, Anvil no longer returns event filter logs for transactions sent from impersonated accounts.
Steps to reproduce

Impersonate vitalik.eth

curl http://localhost:8545 -X POST -H "Content-Type: application/json" --data '{"jsonrpc":"2.0","method":"anvil_impersonateAccount","params":["0xd8da6bf26964af9d7eed9e03e53415d37aa96045"],"id":0}'

Create a new filter

curl http://localhost:8545 -X POST -H "Content-Type: application/json" --data '{"jsonrpc":"2.0","method":"eth_newFilter","params":[{}],"id":1}'

Send a couple of transactions that sends 1 USDC to an arbitrary address (uses transferFrom on USDC contract).

curl http://localhost:8545 -X POST -H "Content-Type: application/json" --data '{"jsonrpc":"2.0","method":"eth_sendTransaction","params":[{ "from":"0xd8da6bf26964af9d7eed9e03e53415d37aa96045", "to":"0xa0b86991c6218b36c1d19d4a2e9eb0ce3606eb48", "data":"0xa9059cbb000000000000000000000000f39fd6e51aad88f6f4ce6ab8827279cfffb922660000000000000000000000000000000000000000000000000000000000000001" }],"id":2}'

curl http://localhost:8545 -X POST -H "Content-Type: application/json" --data '{"jsonrpc":"2.0","method":"eth_sendTransaction","params":[{ "from":"0xd8da6bf26964af9d7eed9e03e53415d37aa96045", "to":"0xa0b86991c6218b36c1d19d4a2e9eb0ce3606eb48", "data":"0xa9059cbb000000000000000000000000f39fd6e51aad88f6f4ce6ab8827279cfffb922660000000000000000000000000000000000000000000000000000000000000001" }],"id":3}'

Use the filter ID from step 2. to try obtain logs

curl http://localhost:8545 -X POST -H "Content-Type: application/json" --data '{"jsonrpc":"2.0","method":"eth_getFilterLogs","params":["<INSERT_FILTER_ID_HERE_PLZ>"],"id":4}'

Seems that the above RPC call returns an array of no items, however, I'd expect 2 event logs to be populated (previous behavior).

ah sigh this is so cursed -.-
checking

❤


Link to old tutorial in README.md is broken

404 link to "old tutorial" in README.md:

https://github.com/allenai/allennlp/blob/master/tutorials/getting_started/semantic_parsing.md

It looks like that it's moved to the "docs" directory:

https://github.com/allenai/allennlp/blob/master/docs/tutorials/getting_started/semantic_parsing.md

is gone from there too now


Fix Custom Template check and Add ability to redact entire field

Minor comment that requires attention.

Rest of it looks good to me.


Add other pnm (pbm, pgm, pam) decoders

Adds decoders and format recognition for the following formats from the netbpm package:
Portable Bit Map, Portable Gray Map and Portable Arbitrary Map
Details
The pam format decoder recognizes the tuple types BLACKANDWHITE, GRAYSCALE and RGB and explicitely recognizes but rejects their _ALPHA variants (support for these might come in another patch).
Tests for the new formats are not yet included but will arrive in the next week, this should stay open until then.
This should deprecate the pam module, not yet part of this pull request. In case it is desired to explicitely test for the ppm format instead, the decoder class offers a method to query the specific image subtype after the header has been read and a wrapper for ppm specificially has been put into the place of the old decoder.
Related Issues
This closes #664

Tell me when you're ready.

I'll add some more tests tomorrow and on the weekend, see if anything turns up in pam especially. Then this should be good to go probably on Saturday. One question, should mod pnm be deprecated right away or not? And if so, what's the preferred way?

@nwin what do you think?

Tests are currently done in pnm as far as pam, pbm and pgm are concerned and in ppm for ppm. Turned out I still had bugs before. All good on my end, ready to go.

I think it would be best to deprecate mod pnm right away.

This brings up something interesting, maybe one would want to assert the image format to be ppm specifically. There is currently no replacement for it when ImageFormat::PPM is removed. Maybe the ImageFormat::PNM could get the additional subtype parameter from the decoder struct so this can be queried/specified.

Also, the encoder is still only written for ppm, so I've just deprecated the decoder for the moment.

I think we can merge this now and make changes to ImageFormat later if needed.

Possibly when the encoder has been transported to pnm or at 0.18. Who knows, maybe both at once? But the former would be a good reason, as it would support specifying the intended output encoding in a uniform way with one implementation.

Merging.


vim: rebase patch to vim 8.0.0983

Signed-off-by: Yunguo Wei<EMAIL_ADDRESS>

I know that Jason had done the same patch, but I can't find his pull request at the moment, so I've merged this one.


isActionOf - Argument contains array with invalid element at index 0

I'm creating action and use it with isActionOf according to the docs:
import { isActionOf, action } from 'typesafe-actions';

export const ACTION = (id) => action('Name', { id });

actions$.pipe(
  filter(isActionOf(ACTION)),
)

But I get an error:

Argument contains array with invalid element at index 0, it should be an action-creator instance from "typesafe-actions"

What am I missing?
Thanks.
"typescript": "~3.9.3"

Care to share how you fixed it/why you closed the issue?

I changed it to:
export const ACTION = (id) => createAction('Name', ({ id }) => ({ id }))();


No update for Firefox (stuck at 1.2.0)

Hi,
The Chrome extension has been updated to 1.3.0 several days ago but I'm still on 1.2.0 for Firefox.
Is there an issue with Firefox?
Thank you.

Review Process :( 166/210. Cookie AutoDelete (and probably other extensions with a moderately sized codebase) has problems with long reviews in AMO. First time it took about 3 months and it was getting stuck at certain parts of the queue.
At the current rate of this update queue, probably 1-2 weeks.
Do you need a signed version by any chance?

Wow! It's the first time that I'm checking an extension on 2 browsers.
I know the first submission to AMO takes a while but I didn't know that we have such a delay for updates...
I'm going to grab the update from the releases tab.

It's not signed so unless you are on FF Developer or Nightly (and turn off signing), it won't install.

Oh yes you're right then please send a signed version.

I added the unlisted signed version under the 1.3.0 release (I added the RC because AMO won't allow me to upload the same '1.3.0' version). Also note that it's unlisted, so it won't autoupdate.

I think it might be possible to set up the automatic builds to do automatic signing using AMOs API, so maybe something I look into in the future.

Auto-updates are always disabled on my end.
Update done, thanks.


Add test for Neo4jGraphMergeBug

The test is very raw and can almost guaranteed be minimised to drill down into the error

This was only an error in the usage of Neo4j graph merge rather than the program logic.


Override equals and hashcode for all entities.

Could you expand more on what the issue is here? I don't understand.

The equals and hashcode methods need to overridden so that objects can be compared.


fix: align reports icon

GitHub Issue (if applicable): #2590
Trello Link (if applicable):
Explanation of Bugfix/Feature/Modification:
Wrapped the Reports icon with display flex so it will be centered when the sidebar is collapsed
After the changes

Thanks for the fix, took a look at this given @kylehaugland's suggestions and just add the navlink class to our element and removed the highlighting.


Support for fully custom viewports not yet implemented

When i run example locally,i got many error like this

I wonder if this is my fault.
I got the same error when i run the example "https://www.cornerstonejs.org/docs/tutorials/basic-segmentation-tools".

Hello, this problem occurred when I ran this example(https://www.cornerstonejs.org/docs/tutorials/basic-volume). Have you encountered it?

resolved

resolved

How did you solve it?


Workflow: Changing sky dart files in the engine does not propagate to app flutter run.

Flutter apps seem to cache the sky dart libraries in the build directory, which causes them to not be updated when new changes are made and built.
Currently, the build directory should be cleared anytime dart changes are made in the engine when testing on real apps.

Duplicate of https://github.com/flutter/flutter/issues/16604


Ztunnel may want to use a different DNS resolution strategy

Currently, we use the default from hickory, Ipv4thenIpv6. We might want to consider changing this, possibly conditionally. More research is needed here.

https://github.com/istio/ztunnel/commit/9e5638b427668e64431b185cae71119069a29bb8


Infer docstring style

It would be nice if we could simply infer the doc string style automatically and choose the appropriate
parser. This would especially be useful when:

the code is the result of combining code from other projects using different doc styles
you are changing the official doc-string style and don't want to have to convert all your strings at once
you are subclassing from another library using a different doc string format and want to include inherited members

So, nice idea! Currently, each object can have its own docstring style attached. But you listed some cases that would still not be supported by this. I'll think about it :slightly_smiling_face:

Instead of inferring the docstring style at once, it could be easier to infer the section styles independently. The rationale is that the style of a section is usually clear while the style of a docstring could be ambiguous. In fact, I've seen mix-styled docstrings.
This approach could actually simplify the way griffe parses docstrings as a lot of the parsing is common to all styles. Also, instead of choosing a style in mkdocs.yml you could rank the styles in the order of preference.

This approach could actually simplify the way griffe parses docstrings as a lot of the parsing is common to all styles. It would also become easier to add parsing rules for custom sections.

Could you expand on that? It's true that the Google and Numpy styles have a lot in common, though subtle differences remain. What do you mean by "parsing rules for custom sections"?

This approach could actually simplify the way griffe parses docstrings as a lot of the parsing is common to all styles. It would also become easier to add parsing rules for custom sections.

I was thinking of instead of having a docstring parse function common to all styles, which calls the "section" parsers implemented for each styles.
Thinking back on that, it would work only if the space policy between sections is the same across all styles. But for similar enough styles you could "share" it and change the parsing style for each section.

What do you mean by "parsing rules for custom sections"?

If I understand correctly, the sections that are availabe are limited by the DocstringSection classes (Parameters, Returs,...) currently implemented which are mapped with section titles. My idea was to allow parameters-like, returns-like, examples-like, note-like ... section parsing with different section titles by adding rules section title -> parser. sphinx.ext.napoleon allows to do that.
Also, maybe some current section could be merged? I am thinking of Returns and Yields.

I'm not sure supporting multiple-style docstrings is worth the effort. Even if you're switching from one style to another in a big codebase, docstrings themselves are usually short enough that you can stick to a single style, and not just switch one or two sections in the docstring. And if it's not about switching styles, then I'd consider it a very niche use-case, again not worth the effort.
Your argument about factorizing the code still stands though. It's not the prettiest, DRYest code I've written, it can surely be refactored to something more elegant and reusable :slightly_smiling_face:

allow parameters-like, returns-like, examples-like

Custom titles for sections are already supported! There's an example here in Griffe's own docs (unfold the source code). I don't think more customization is welcomed in docstring parsers. If you deviate from the spec, then all the tooling around docstrings stop working (linters, IDEs, ...). Just my opinion of course, happy to read counter-examples!

Tooling for docstring styles is not so great in many tools. (e.g. PyCharm still doesn't support google style very well), but obviously no one would recommend that anyone intentionally use multiple styles in the same doc string, yet it does happen sometimes.
It makes sense for linters and the like to be more strict about doc-string styles, but people want their doc generator to just magically do the "right thing" if at all possible.
I think that it would be fantastic if inference was just at the docstring level, but it would be even nicer if it could be done per section.
However, maybe anyone who wants this should provide some real-world examples doc-strings using multiple styles.

Here's some very rough/ugly script to count styles in packages docstrings:
import os
import re
from contextlib import suppress
from griffe.exceptions import AliasResolutionError
from griffe.loader import GriffeLoader

packages_paths = []

numpy_docstrings = 0
google_docstrings = 0
sphinx_docstrings = 0
unknown_docstrings = 0
markdown_docstrings = 0
rst_docstrings = 0

md_inline_code = re.compile(r"`[^`]+`")
rst_inline_code = re.compile(r"``[^`]+``")

def prompt_docstring(docstring):
    global numpy_docstrings
    global google_docstrings
    global sphinx_docstrings
    global unknown_docstrings
    global markdown_docstrings
    global rst_docstrings
    if docstring:
        for markup in (
            ":meth:",
            ":func:",
            ":attr:",
            ":mod:",
            ":class:",
            ":raises:",
            ":raise ",
            ":note:",
            ":param ",
            ":return:",
            ":returns:",
            ":rtype:",
        ):
            if markup in docstring.value:
                sphinx_docstrings += 1
                return
        for markup in (
            "Args:\n  ",
            "Arguments:\n  ",
            "Attributes:\n  ",
            "Raises:\n  ",
            "Returns:\n  ",
            "Yields:\n  ",
            "Example:\n  ",
            "Examples:\n  ",
        ):
            if markup in docstring.value:
                google_docstrings += 1
                return
        for markup in (
            "Args\n----",
            "Arguments\n---------",
            "Attributes\n----------",
            "Parameters\n----------",
            "Raises\n------",
            "Returns\n-------",
            "Yields\n------",
            "Methods\n-------",
        ):
            if markup in docstring.value:
                numpy_docstrings += 1
                return
        for markup in (
            "```\n",
        ):
            if markup in docstring.value:
                markdown_docstrings += 1
                return
        for markup in (
            ".. autofunction::",
            ".. code::",
            ".. todo::",
            ".. note::",
            ".. warning::",
            ".. versionchanged::",
            ".. versionadded::",
            "::\n\n    ",
        ):
            if markup in docstring.value:
                rst_docstrings += 1
                return

        if rst_inline_code.search(docstring.value):
            rst_docstrings += 1
            return
        if md_inline_code.search(docstring.value):
            markdown_docstrings += 1
            return

        unknown_docstrings += 1

def iter_docstrings(obj):
    try:
        prompt_docstring(obj.docstring)
    except AliasResolutionError:
        return
    for member in obj.members.values():
        if not member.is_alias:
            iter_docstrings(member)

if __name__ == "__main__":
    loader = GriffeLoader(allow_inspection=False)
    for index, package_path in enumerate(packages_paths, 1):
        print(f"\r{index}/{len(packages_paths)}", end="")
        with suppress(Exception):
            package = loader.load_module(package_path)
            iter_docstrings(package)

    print("Google", google_docstrings)
    print("Numpy", numpy_docstrings)
    print("Sphinx", sphinx_docstrings)
    print("Markdown", markdown_docstrings)
    print("RST", rst_docstrings)
    print("Unknown", unknown_docstrings)

It does not search for multi-style docstrings. To do it, the words search should probably be replaced with regexes, and all regexes should be tested.

It will available in next Insiders version.

Aww, you didn't use my lib

@SamuelMarks no indeed, as I couldn't find anything public in your library that would allow me to do this. But maybe I missed it?

Ah, I now see derive_docstring_format, but the tokens are quite limited. Even if they weren't, I wouldn't want to depend on your entire library just for this tiny part 😅


Sales

How about the sales of our product? How much is it? The people should like? And the most critics? We need to make an acceptable work! Where is the community? This is what i called Marketing!
It's a good idea try to get costumers that are interested on Dizar or maintain it just for us?

Yes, it's a good idea. Just be aware of the price: it can be about some 20 or 15, but not more that this!

This is good, because, even it's a low price, we should get money!

This can be difficult 'cause we don't have all recources for make a game engine!


Documented Use with SecrurityComponent.

Please send this PR again when we are done with the CakePHP 3 version of the plugin


Got a different huber_loss result

when I just use the huber loss in file "03_linreg_placeholder" in line 36, I get different results from the notes.
w=-4.2249994 b=78.485054

I get exactly the same result, w=-4.2249994 b=78.485054


Output.path is dist, how to delete only js folder and css folder in dist

Issue description or question
Output.path is dist, how to delete only js folder and css folder in dist

Webpack Config
output: {
    path: resolvePath("../dist"),
    publicPath: customConf.publicPath,
    filename: isDev ? `[name].[hash].js` : `js/[name].[chunkhash:8].js`,
    chunkFilename: isDev ? `[name].[hash].js` : `js/[name].[chunkhash:8].js`
  },

This should point you in the right direction:
new CleanWebpackPlugin({
    // patterns are relative to webpack's output.path
    //
    // See https://github.com/sindresorhus/del#patterns for pattern matching documentation
    cleanOnceBeforeBuildPatterns: ['js/**/*', 'css/**/*'],
});


Fill in pass.rs

Variants for every pass
as_str should return hard-coded names that correspond to the binaryen pass names
description should query the binaryen PassRegistry api

done


Undefined Index

Getting the following error when attempting to run.

In Query.php line 189:
Undefined index: fetch_order

To Reproduce
Just run, I believe the bug was introduced in version 1.2.4
Additional context
Managed to get around the error by adding the following to the options array in config.php
'fetch_order' => 'asc'
However, when you do this, the same error then appears again but this time with

In Query.php line 189:
Undefined index: message_key

Again, adding this to the options array resolves the issue, but it appears it is not pulling the default from the provided options array, so possibly an issue with the merge?
'message_key' => 'id'
Had to pull the values from the Query.php file as the default values are not listed in the documentation, so be good to have those also.
Hopefully this can get resolved, without the need for the work-arounds above.
EDIT:
Also noticed it does the same thing in the imap.php files for FT_UID need to put it in a quote mark to get it working

Hi @empsean ,
thanks for your report. I'll take a closer look at it tonight :)
Best regards


Enable debug logs for github actions

In the process of releasing 0.4.1, we noticed that the following github action semver-release-action failed and it wasn't easy to pin-point the problem based on the error message that was printed:
  Run zwaldowski/semver-release-action@v11s
    github_token: ***
Run zwaldowski/semver-release-action@v1
  with:
    dry_run: true
    bump: pr/patch
    github_token: ***
##[error]Cannot read property 'toString' of null

It would be nicer if we could get better logs so that the problem becomes evident and takes lesser time to fix.
We might like to look at the information available in this page : https://github.com/actions/toolkit/blob/master/docs/action-debugging.md

We can enable debug logs by setting an env variable. We will only do this if there is a problem.
Closing this task as there is nothing we can do atm.


API login/Basic fails on 1.50.1 for installer

For the last 16 months I have been able to use /api/login/Basic with username=installer.
Yesterday my PW firmware was updated from 1.49.0 to 1.50.1 and my code fails at login with the response:
{"code":401,"error":"roles [Provider_Engineer] require physical access","message":"Login Error"}
There is also a new screen when installer is selected in the Kiosk.  It states that the PW switch must be toggled (as proof of physical access, I presume).
Any ideas?

I too had this issue yesterday when the new firmware went on. I don't think there's a way around it using the local gateway now they have introduced physical confirmation.
I've switched my code over to using the public API now, which actually seems to work better as the process doesn't stop the Powerwall from operating when making a change like with the local gateway. Changes to reserve seem to be instant.
https://www.teslaapi.io/powerwalls/commands
All you need to do is get the token from /oauth/token then you can set the reserve % via /api/1/energy_sites/:siteid/backup

@spoonwzd can you share how you do this?  I assume that the reserve percentage is a parameter on the call?  The API doco doesn't have this detail.  e.g. is it /api/etc/backup?reserve=10 or api/etc/backup/reserve/10?

This is so annoying but perhaps inevitable.

You simply POST the backup_reserve_percent value in JSON to the /api/1/energy_sites/energy_site_id/backup API URL along with the authorization bearer @BJReplay
This is how I'm doing it in node-red with a custom JavaScript function:
msg.headers = {
Authorization: "Bearer " + MY_AUTH_TOKEN,
"content-type": "application/json"
}
msg.method = "POST"
msg.url = "https://owner-api.teslamotors.com/api/1/energy_sites/MY_SITE_ID/backup"
msg.payload = {
"backup_reserve_percent": 0
}

You simply POST the backup_reserve_percent value in JSON to the /api/1/energy_sites/energy_site_id/backup API URL along with the authorization bearer @BJReplay
This is how I'm doing it in node-red with a custom JavaScript function:
msg.headers = {
Authorization: "Bearer " + MY_AUTH_TOKEN,
"content-type": "application/json"
}
msg.method = "POST"
msg.url = "https://owner-api.teslamotors.com/api/1/energy_sites/MY_SITE_ID/backup"
msg.payload = {
"backup_reserve_percent": 77

I don't suppose you've got something similar for changing modes have you?

I don't suppose you've got something similar for changing modes have you?

I haven't tried changing the mode, but I'm going to assume it's just the same as the above example, just use /api/1/energy_sites/energy_site_id/operation and set "mode" in the JSON payload with "self_consumption" or whatever you're looking to set.

Ahh, so annoying.  I'm testing using postman and I'm getting
{
    "response": null,
    "error": "https://powergate.prd.sn.tesla.services:443/api/powerwalls/MY_GATEWAY_ID/backup => Method Not Allowed",
    "error_description": ""
}

In response to:
Method: POST
URL: https://owner-api.teslamotors.com/api/1/powerwalls/MY_SITE_ID/backup
Headers: as noted above (Bearer & auth token, content-type)
Body: {"backup_reserve_percent": 10}

Is it that you need to be setting reserve at 'energy_sites' not 'powerwalls' @BJReplay ?

@spoonwzd Yes!  Thanks!  Right.  powerwalls for getting status (or maybe that works with energy_sites too - will try), energy_sites for setting status.

@spoonwzd Yes! Thanks! Right. powerwalls for getting status (or maybe that works with energy_sites too - will try), energy_sites for setting status.

Nice. :) I'm still getting live power flow stats from the local gateway as I don't want to be hammering the public API.

@HeraldCoupe Yes, I have:
POST to https://owner-api.teslamotors.com/api/1/energy_sites/YOUR_INTEGER_SITE_ID/operation
Headers:
Authorization Bearer as above
Content-Type application/json
Body:
{"default_real_mode":"self_consumption"}
or
{"default_real_mode":"backup"}

Great work everyone :)

Yes, great work and thank you. I'll have a go at this myself in the coming days.
The next question is.... how long before Tesla stop us from using this API too?

@HeraldCoupe Yes, I have:
POST to https://owner-api.teslamotors.com/api/1/energy_sites/YOUR_INTEGER_SITE_ID/operation
Headers:
Authorization Bearer as above
Content-Type application/json
Body:
{"default_real_mode":"self_consumption"}
or
{"default_real_mode":"backup"}

Brilliant, thank you.
I've made progress today, I can get the token and energy_site_id so I'll move onto getting the mode then I'll attempt to set the mode.

You can use "https://owner-api.teslamotors.com/api/1/energy_sites/YOUR_INTEGER_SITE/site_info" to get "backup_reserve_percent", "default_real_mode" as well as "storm_mode_enabled" and a bunch of other potentially useful bits of info.
It requires the usual authorization bearer stuff.

Note that the site ID is not an integer (as I suggested by referring to it as YOUR_INTEGER_SITE) - more like a BIGINT or long numeric.  I've seen an example which is 12 digits long.

Any idea how to force the "commit" for the setings.
In general from what I see calling the cloud API , is introducing around 15-20 minute delay, closer to 15 minute.
Any way for the settings to take effect.
The Application shows the setting right away, so its going to Tesla server, but the execution of Gateway is taking around 15 minutes.
Is there a way to call ?
/config/completed  or /sitemaster/run from the CLOUD?
https://github.com/fkhera/powerwallCloud/blob/master/powerwallBackup.py#L32

Interesting. I'd expected to have this problem too but actually the change appears to be pretty instant for me. I've confirmed it using the graph in the Tesla app.

Is there a way to call ?
/config/completed or /sitemaster/run from the CLOUD?

I don't change mode - simply adjusting the reserve fulfills all my needs.
When using the public API confirmation (/config/completed or /sitemaster/run) is not required and the change is all but instant on my Powerwall.

I just stay in the time based control mode, and adjust reserve %.
Today I noticed there is about 10-15 minute delay on my phone.
Our current peak hours are 5am-9am and 5pm-9pm.
One long peak defined as 5am-9pm in Tesla peak and controlled by adjusting
reserves.
This morning I am supposed to be at zero %, but I had another script set me
to 100% at around six to 6:50am.
At 6:50 am on my phone , I manually adjusted from 100% to force discharge 0%
We did not see change of standby to 'discharging' until around 7:05am.
Also seeing around 15 minute delay.
So I guess the delay matches the phone so must be Tesla server or Tesla
Gateway related.
On Tue, Nov 3, 2020 at 1:32 AM spoonwzd<EMAIL_ADDRESS>wrote:

Is there a way to call ?
/config/completed or /sitemaster/run from the CLOUD?
I don't change mode - simply adjusting the reserve fulfills all my needs.
When using the public API confirmation (/config/completed or
/sitemaster/run) is not required and the change is all but instant on my
Powerwall.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/vloschiavo/powerwall2/issues/34#issuecomment-720975557,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABU4JYC7WSRY562RGE2VOELSN65ZHANCNFSM4S32I5GA
.

I wonder if it's something to do with the mode you're in? I'm in self_consumption mode and never switch away from it - I just set the battery reserve and the change is actioned on the Powerwall instantly.

I wonder if it's something to do with the mode you're in? I'm in self_consumption mode and never switch away from it - I just set the battery reserve and the change is actioned on the Powerwall instantly.

Thanks for the insight. I tried this - for Time Based Control, looking at a separate energy monitor (ie. not relying on the Tesla app) changes took between 15 and 45 minutes to take effect. Self consumption - as you experienced - pretty  immediate.
Discovered another factor - under TBC and Powerwall controlled charging, charge power is consistently c.3.8KW.  When charging due to increased reserve % under TBC - 5KW. Understandable to an extent, to ensure the reserve is put in place asap.  But charging under Self Consumption forced by increased reserve.....1.7KW, consistent over a number of tests.
Any views why that might be (and how, potentially, to increase it)? I ask the question as although SC seems to give the control response needed, that charge rate is just too low for my scenario.

Any views why that might be (and how, potentially, to increase it)?

Yep, setting the mode to backup will double the charge rate to 3.3kW.  You will need to monitor progress, and when you hit your target you'll need to swap back into SC.

I can confirm that the max charge rate in self consumption mode is 1.7kW and there's no way to increase it when in this mode.

Luckily I have a 10-hour off-peak window so I can achieve a full charge within this time frame.

Any views why that might be (and how, potentially, to increase it)?

Yep, setting the mode to backup will double the charge rate to 3.3kW. You will need to monitor progress, and when you hit your target you'll need to swap back into SC.

In backup charge rate on my unit is 5KW. But agree with the approach you suggest - i'll test the required charge against available time and if the SC rate can't meet that, its a simple calculation to determine the charge profile in the off peak period.
Can understand why backup charges at the full 5KW rate, I have asked my installer why the rates are different in SC/TBC. Will share any response.

I did query Tesla on the difference in charging rates (1.7KW in Self Consumption, 5KW in Backup, 3.7KW in TBC).
(Eventual) response was that 3.7KW was chosen as a balance between charge rate & battery lifetime....but that 1.7KW in SC couldn't (wouldn't ?) be changed. Go figure.

Firstly I would like to thank everyone on this thread for how instructional this has been - its been brilliant.
Secondly - and this may not be apparent to others so I thought I would share. If you have a tariff that does not differ between night and day (in the UK we call this Economy7) - then you cannot set the reserve. This is what I see on the Octopus Tesla Tariff -:

Thats after calling -:
https://owner-api.teslamotors.com/api/1/energy_sites/{site-id-number}/backup
with a body of -:
{ "backup_reserve_percent" : 50 }
from Postman. (ps I've removed my site-id and replaced it with {site-id-number}.

Can you set reserve in your phone ??
On Thu, Dec 10, 2020 at 3:09 PM Richard Hill<EMAIL_ADDRESS>wrote:

Firstly I would like to thank everyone on this thread for how
instructional this has been - its been brilliant.
Secondly - and this may not be apparent to others so I thought I would
share. If you have a tariff that does not differ between night and day (in
the UK we call this Economy7) - then you cannot set the reserve. This is
what I see on the Octopus Tesla Tariff -:
[image: image]
https://user-images.githubusercontent.com/1540464/101835706-16b17980-3b34-11eb-83bd-678144e88cb5.png
Thats after calling -:
https://owner-api.teslamotors.com/api/1/energy_sites/{site-id-number}/backup
with a body of -:
{ "backup_reserve_percent" : 50 }
from Postman. (ps I've removed my site-id and replaced it with
{site-id-number}.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/vloschiavo/powerwall2/issues/34#issuecomment-742831363,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABU4JYDG6TNMBET6674WMZDSUFBLDANCNFSM4S32I5GA
.

@fkhera  - no - its been removed.
I noticed that when I check the Data end point a few settings are set to false eg -:
Under components -:
"backup" : false
"configurable": false,

and
        "backup": {
            "backup_reserve_percent": 0,
            "events": null
        },

closed.  See issue #41 for an update re: version 20.49.0


Adding support for epub

Making epub the default format. As Amazon is removing mobi support.
I also fixed the title of the book.

Hello, thank you for your PR. I'm actually not using this anymore but I'm happy to merge it.


Docs: Record of decisions and research

I think everything is covered here now so going to go ahead and merge it!


use System.arraycopy

use System.arraycopy instead of loop.

Coverage increased (+0.003%) to 90.563% when pulling 4a92484dcc6fbe1ebd8b12c0820da23724d6ad7a on XenoAmess:use_System.arraycopy into 715d89d2625b09915feb1dd1b60270842eeffb7a on apache:master.

no. pause this pr.
please wait for my performance test.

for changes in DSCompiler.java and Logit.java :
please see performance test in https://github.com/XenoAmess/performance_tests_for_java/blob/master/src/test/java/com/xenoamess/performance/ArrayFill01PerformanceTest.java
and result in https://github.com/XenoAmess/performance_tests_for_java/blob/master/result/result.txt

related informations about this change: https://stackoverflow.com/questions/8526907/is-javas-system-arraycopy-efficient-for-small-arrays
do I still need a jmh performance benchmark for this pr?


Remove hard-coded 'magic' from the generic-api-key rule

Description:
This replaces the hardcoded logic in detect.go with a regex. In testing across 3,000 repositories, there was virtually no difference between these implementations.
The benefit is that the constraints are explicit (not hidden in code), and people can take the config and tweak to be as they desire.
Checklist:

[x] Does your PR pass tests?
[x] Have you written new tests for your changes?
[ ] Have you lint your code locally prior to submission?

I like this better than the harddcoded number check.
But the regex '''^[a-zA-Z_.-]+$''' would have to match on only the secret.
Is it possible to then also have other regex with regexTarget = "match" or regexTarget = "line", as in #1587, or with existing custom gitleaks.toml that modify the generic-api-key rule?
Possibly with the the new multiple allow lists?

Is it possible to then also have other regex with regexTarget = "match" or regexTarget = "line", as in #1587, or with existing custom gitleaks.toml that modify the generic-api-key rule?
Possibly with the the new multiple allow lists?

Absolutely. The plan is to keep a single allowlist for a bit longer, then we could implement this.


remove pidofproc function

The pidofproc function should not be redefined here because the same function from the LSB works correctly. Using pgrep will not work correctly if you have a file named similarly to the "daemon-name" open (i.e. "/etc/opt/influxdb/influxdb.conf").
This is with 0.9.2 installed on a base 64-Bit Debian Wheezy system, using the .deb from InfluxDB.com.
It's easy to recreate this situation by simply opening the configuration file (i.e. sudo vi /etc/opt/influxdb/influxdb.conf), and running /etc/init.d/influxdb status. With the configuration file open, pgrep -f /opt/influxdb/influxd picks up multiple pids, which causes the init script to report incorrectly that the influxd process is not running (e.g. influxd Process is not running [ FAILED ]).

Thanks @daachi -- I too noticed this in the script and was wondering why it was there since it is defined elsewhere as you say, but left it for now.
My concern is what will happen on distros that do not have LSB. Is that a concern?

This is a good point. This commit uses regex syntax to check for influxd command line, and also checks for the existence of the pidofproc function so that we're not overriding it if it exists.
I think this addresses your concerns?

Thanks @daachi -- I do have some questions before we can merge. Also, can you sign the CLA if you have not already done so?
https://influxdb.com/community/cla.html

I've signed the CLA, for sure.
Also, we're running into some other issues and I will have additional commits related to this. I'd like very much to hear your feedback as well.

So, otoolep, these commits updates the syntax of the pgrep command, as you suggested, corrects indentation, and wraps the remaining functions which can also be found in the LSB.
The problem remains that the start function still doesn't work correctly. This can be verified, for example, by opening the config file (from a fresh install) and setting enabled = true under the collectd section which will render the daemon unable to start. The init script will report incorrectly that the daemon has started.
Before 88307bf2 I had this script working by adding an additional sleep 5 to the top of the wait_for_startup function (which has since been removed). Not elegant, but correct and functional.
I don't know if you will accept this without that fix, but let me know if you want me to fix that as well, or if I should open another PR.

Closed in favour of https://github.com/influxdb/influxdb/pull/4227
Thanks @daachi


wget: bad address 'downlaods.openwrt.org'

uday@uday-thinkpad01:~/go/src/github.com/dockarray1/agent$ docker build --no-cache -t "maestrocm/agent" .
Sending build context to Docker daemon 35.45 MB
Step 1/6 : FROM progrium/busybox
---> a67699e37dbd
Step 2/6 : MAINTAINER Ron Messana<EMAIL_ADDRESS>---> Running in 31c0b5ec04b2
---> b14883096770
Removing intermediate container 31c0b5ec04b2
Step 3/6 : RUN opkg-install libseccomp ca-certificates
---> Running in 516619121e72
wget: bad address 'downlaods.openwrt.org'
wget: bad address 'downlaods.openwrt.org'
wget: bad address 'downlaods.openwrt.org'

I can't see anywhere in our commit history where we spelt downloads like that, weird! Try running docker pull progrium/busybox to get the latest copy of the Docker image and build your container again.


Humanize ordinals

As in https://github.com/dustin/go-humanize/blob/master/ordinals.go I would like to be able to pass an integer to the humanize function and have it return an ordinal.
eg. in a shortcode:
{{ $url := "/data/rallies.json" }}

{{ range $rally := getJSON $url }}
{{ if modBool $rally.order 2 }}

{{ $rally.order | humanize }} rally - {{ $rally.location }}

{{ end }}
{{ end }}

Should render as

1st rally - Sydney
3rd rally - Melbourne
5th rally - Adelaide

@waddles, we use the inflect package instead of go-humanize, but we can add an ordinalize func.

We should think a little about this and see in connection with the multi-language work being done somewhere else.

I'm looking for a way to contribute and this seems like it could be an good candidate for a first PR.
@bep would you envision this being a new template function or an addition to the existing humanize template function which could behave differently depending on what was passed in? Additionally, back in February you mentioned the multi-language work being done, how could these two things relate?
I apologize for the questions, I'm not super familiar with the design goals of hugo, just curious what a good approach would look like to you before I go down a path.

Actually the inflect package has a https://godoc.org/bitbucket.org/pkg/inflect#Ruleset.Ordinalize function already but I decided I didn't need it after all.

Even if you don't need it, someone else probably will and it's a simply enough thing to support I think.

@michaelorr I have no clue how this intersects with multilanguage, but the thing is that for Norwegians (like myself), Chinese etc. having ordinals that creates 1st, 2nd etc. is of not much use ...
But since we already have a humanize func it would sensible to change it so:
If the argument is one of the integer types, then return ordinal.
PR welcomed.

https://github.com/spf13/hugo/blob/273a68400ff081d05508afbcb6d158711b865307/tpl/template_funcs.go#L1728 would be the place to put it

No, https://github.com/spf13/hugo/blob/273a68400ff081d05508afbcb6d158711b865307/tpl/template_funcs.go#L1707

@bep First stab is here: https://github.com/spf13/hugo/pull/2197
There are a few things I want to take care of before it's ready but they are noted in the PR comment.

PR with the fix is ready: https://github.com/spf13/hugo/pull/2197


Flag excessively long records with a broken delimiter issue and skip importing

It seems like the importer should not be ingesting excessively long records, because it likely indicates a broken delimiter issue:
https://github.com/CatalogueOfLife/data/issues/534
http://api.checklistbank.org/dataset/9893/reference/069e92c9-83df-4617-b138-ea8b978784c9
It is not safe to assume that thousands of data sources will always provide perfect data, so the importer ought to be able to gracefully handle common problems like broken delimiters. We can get those Brachyura records fixed, but another dozen will likely pop-up elsewhere from other typos being introduced over time.
Could we please add a broken delimiter issue?

That is tricky. What exactly do you regard as an excessive long record?

The example you created gets interpreted as a TexTree format which is not ideal. That is sth I should be able to handle more strictly. If you set the options to coldp and reimport it things look better.

See also 767 Previous Line Skipped issues:

The CSV record before this one had to be ignored because it had no content or wrong number of columns. Often indicates a serious delimiter problem

Maybe that should be red? Not sure if there can be also harmless cases

I think we'd have to come up with some sensible limits for each field, and I'm willing to help provide that by analyzing the COL project data. It's excellent that the data gets imported into the verbatim table and provides the opportunity for flagging it with a potential broken delimiter issue, and/or maybe abnormal length issue.

unusual length seems like a good idea if you can come up with conservative length limits for each of the ColDP, but maybe also DWC terms. That should be straight forward to include during imports!

Just to be clear, we cannot tie an issue to an individual column. It will always be the entire verbatim record that is flagged if one of the columns exceeds the expectation. So maybe better unusual value length?

Just to be clear, we cannot tie an issue to an individual column. It will always be the entire verbatim record that is flagged if one of the columns exceeds the expectation. So maybe better unusual value length?

I think that would still work if any column is unusually long. Although ideally when ingesting the verbatim data into the interpreted database, I wonder if we should be skipping importing unusually long values? Especially for references, because it's not that big of a deal to lose a reference especially if it is most likely bad data. Having 100 references as a name reference is probably not going to be helpful to most people.


retriable error bug?

(apologies for the bad issue title, I don't have enough context yet so this is the best I could name the issue but feel free to update it)
so I noticed after bumping from v0.10.2 to v0.10.3, my producer was not working. I was using sync producer so it was just stuck until context got canceled. I figured to use the bench example to try reproducing and it was not producing anything as well:
$ ./bench -brokers localhost:61810 -topic test
0.00 MiB/s; 0.00k records/s
0.00 MiB/s; 0.00k records/s
0.00 MiB/s; 0.00k records/s

after some testing, I think there's something with the retriable code. this change makes it work:
diff --git a/pkg/kgo/client.go b/pkg/kgo/client.go
index 486708c..9db4d30 100644
--- a/pkg/kgo/client.go
+++ b/pkg/kgo/client.go
@@ -619,6 +619,7 @@ start:
 				// just retry now. If the error is *not* retriable but
 				// is a broker-specific network error, and the next
 				// broker is different than the current, we also retry.
+				fmt.Printf("retriable: err=%v retryErr=%v\n", err, retryErr)
 				if r.cl.shouldRetry(tries, err) || r.cl.shouldRetry(tries, retryErr) {
 					if r.cl.waitTries(ctx, tries) {
 						next, nextErr = r.br()
diff --git a/pkg/kgo/errors.go b/pkg/kgo/errors.go
index e4995bf..af17799 100644
--- a/pkg/kgo/errors.go
+++ b/pkg/kgo/errors.go
@@ -11,7 +11,7 @@ import (

 func isRetriableBrokerErr(err error) bool {
 	if err == nil { // sanity
-		return true
+		// return true
 	}
 	// https://github.com/golang/go/issues/45729
 	//
diff --git a/pkg/kgo/go115.go b/pkg/kgo/go115.go
index 634f131..473672c 100644
--- a/pkg/kgo/go115.go
+++ b/pkg/kgo/go115.go
@@ -6,5 +6,8 @@ package kgo
 import "strings"

 func isNetClosedErr(err error) bool {
+	if err == nil { // sanity
+		return false
+	}
 	return strings.Contains(err.Error(), "use of closed network connection")
 }

$ ./bench -brokers localhost:61810 -topic test
retriable: err=broker is too old; the broker has already indicated it will not know how to handle the request retryErr=<nil>
33.99 MiB/s; 356.41k records/s
12.99 MiB/s; 136.23k records/s
12.99 MiB/s; 136.23k records/s
38.98 MiB/s; 408.69k records/s
38.98 MiB/s; 408.69k records/s

I left the debug log and reverted the rest and this is what I get:
$ ./bench -brokers localhost:61810 -topic test
retriable: err=broker is too old; the broker has already indicated it will not know how to handle the request retryErr=<nil>
retriable: err=broker is too old; the broker has already indicated it will not know how to handle the request retryErr=<nil>
retriable: err=broker is too old; the broker has already indicated it will not know how to handle the request retryErr=<nil>
retriable: err=broker is too old; the broker has already indicated it will not know how to handle the request retryErr=<nil>
0.00 MiB/s; 0.00k records/s
retriable: err=broker is too old; the broker has already indicated it will not know how to handle the request retryErr=<nil>
0.00 MiB/s; 0.00k records/s
retriable: err=broker is too old; the broker has already indicated it will not know how to handle the request retryErr=<nil>
0.00 MiB/s; 0.00k records/s
retriable: err=broker is too old; the broker has already indicated it will not know how to handle the request retryErr=<nil>

for what it's worth, I am using the latest redpanda:
$ rpk version
v21.8.1 (rev 3522ce7e9e4b9fbe62b9474f80138c4deeb8359b)

Good find, this is from the idempotent request being retried.
In client.go,
 622                                 if r.cl.shouldRetry(tries, err) || r.cl.shouldRetry(tries, retryErr) {

we hit this case when one or the other error is non-nil, and shouldRetry previously returned false unless it was truly retriable. Now it returns true due to what you found.
I chose true because no error should mean retriable, but I'm going to change it to false because we specifically only want to evaluate if a non-nil error is retriable.

I've tagged the fix in v0.11.1. v1 should be released either tomorrow or Monday (unless you find a very tricky bug :smile:).


i18n(zh-cn): Update 2-pages/5.mdx

Description (required)
Update 2-pages/5.mdx
NOTE: The doc itself is up-to-date, so I commit some indent to update the status of Docs translation.
Related issues & labels (optional)

Suggested label: i18n

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note

zh-cn/tutorial/2-pages/5.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Service provider pretty name is shared

See from my talk https://youtu.be/Uv0ivz5xej4?t=1213, look at the columns. Something is wrong:

Hi @sttts,
It happened because there are one controller for each bind provider cluster. But there is no filtering in APIServiceBinding controller. APIServiceBinding controller is initialized with its provider namespace. So, when controller reconciles an APIServiceBinding from other provider, it sets incorrect pretty name. That's why Provider name is inconsistent. Linked PR should solve the issue.


update to 10.1.37

Hi
The current available version of MariaDB 10.1 in IUS is 10.1.35, however upstream MariaDB is now up to 10.1.37: https://mariadb.org/mariadb-10-1-37-now-available/
There is at least one problematic issue we've seen in 10.1.35 where it doesn't always clean up temporary tables from disk https://jira.mariadb.org/browse/MDEV-16647 which was marked as fixed in 10.1.36: https://mariadb.com/kb/en/library/mariadb-10136-changelog/
Could we please get the latest version available when you get a chance?
Thanks

@smokris Any interest in helping out with this one?

This package has been updated to 10.1.40 and is currently available from ius-testing.  I'll move it to stable in about a week.


[Addon]Add one more cloud resource: Alibaba DNS

Description of your changes
Add one more cloud resource: Alibaba DNS.
How has this code been tested?
We've use it to create new DNS records on our domain in our ACK cluster, it works. I replace the terraform configuration with alibaba official terraform module: https://github.com/terraform-alicloud-modules/terraform-alicloud-dns.git
see example.zip
Checklist

I have:

[x] Title of the PR starts with type (e.g. [Addon] or [Trait]).
[x] Updated/Added any relevant [documentation] and [examples].
[ ] Unit/E2E Tests added.

 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.:x: FreeceLian sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@FreeceLian That's wonderful! Thanks for your contribution.
Please fix DCO issue. Would you please also generate docs for Alibaba Cloud DNS http://kubevela.net/docs/platform-engineers/components/component-terraform?

@FreeceLian That's wonderful! Thanks for your contribution. Please fix DCO issue. Would you please also generate docs for Alibaba Cloud DNS http://kubevela.net/docs/platform-engineers/components/component-terraform?

I will generate docs later.


Automated application of SASS and CSS lint

This is a...

[x] Feature Request
[ ] Bug Report

Problem:
SASS and CSS files are not maintained under any rules and automated.
Proposed Solution:
Use some linter such as stylelint in CI.
Page to Update:
n/a

Blocked by #10606
/assign
/kind cleanup


[Package Request]: I want to help add the new nvidia app

How can we help?
I would like help so I can submit the manifest.
Please read and ensure the following

[x] The installer meets the above requirements

Please provide the following information
Download Page Url: https://www.nvidia.com/en-us/software/nvidia-app/
Publisher: NVIDIA
Package Name: NVIDIA app
Description: GPU Companion
Package Version: <IP_ADDRESS>
Installer Url: https://us.download.nvidia.com/nvapp/client/<IP_ADDRESS>/NVIDIA_app_v<IP_ADDRESS>.exe

[!NOTE]

Duplicate of #140696

[!CAUTION]
This is a https://github.com/microsoft/winget-pkgs/labels/Blocking-Issue, because of https://github.com/microsoft/winget-pkgs/labels/Hardware

Will be resolved by #190775

Close with reason: Duplicate of #140696;

Duplicate of #140696


register function is not properly exported in TS definition

Describe the bug
register function is not properly exported in TS definitions
So, it s not possible to use properly register function in typescript project and vue-advanced-chat component without an compilation error...
Steps to reproduce
I have created a repo https://github.com/rdhainaut/vue-advanced-chat-register-issue
Note: I have generated a basic vue 2 project from vue-cli (with typescript)
Expected behavior
no error in ts project
Fix
//vue-advanced-chat/types.index.d.ts
export function register(): void;

Thanks, could you open a PR with the fix?

Thx!


Search product by EAN

I had problems with pushing the commit. Please ignore/close the two requests down there...
#207 #202
-> http://jira.shopware.de/?ticket=SW-9795

Just for later reference: https://github.com/bcremer/shopware/pull/1

Dear Contributor,
Thank you for your contribution to Shopware. We value your engagement.
Unfortunately we have to close all existing Pull Requests.
This is due to the new Shopware 5 Major Release and a restructuring of the branch names.
If your contribution is still relevant for Shopware 5, a new pull request is highly appreciated.
We are very sorry for this inconvenience.
Greetings from the Shopware Team.


Update ONLINECOURSES.json

I've added a free AWS DevOps course to the list. It is a course that I follow and I found it helpful so I added it here

I've added a free AWS DevOps course to the list. It is a course that I follow and I found it helpful so I added it here

Apologies for the delay hacktoberfest-accepted thank you @prajwal-charizardX


Find replacement for CONFIG_MBEDTLS_ALLOW_UNSUPPORTED_CRITICAL_EXT (IDFGH-11195)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
This flag is currently without any effect, as it has been removed upstream (see https://github.com/Mbed-TLS/mbedtls/pull/4477).
This feature is crucial as major vendors of certificates use such unsupported extensions (see https://github.com/Mbed-TLS/mbedtls/issues/2605).
I think there are two options to replace this functionality:

Replicate this functionality by implementing the callback (mbedtls_x509_crt_parse_der_with_ext_cb) and ignore unknown extensions (this is deemed insecure by mbedtls maintainers)
Allow the user to service such a callback

Thanks for notifying. We will remove this config from ESP32 mbedtls port. However, the alternative solution for this must be discussed in the upstream community itself. One of the suggestion can be found here: https://github.com/Mbed-TLS/mbedtls/pull/3564#issuecomment-674825340

Fixed with https://github.com/espressif/esp-idf/commit/0c3ed4f540cc6ecd9d148cc2fd1767eeb7d860a3


autogenview: store API parameters as router query for history

This would store pagination, filter and search context in route query
which would allow for browser history navigation in list views.
Fixes #451
Signed-off-by: Rohit Yadav<EMAIL_ADDRESS>

Please test and review @davidjumani

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/454 (JID-2123)

Converted to draft, as this may not be complete yet
With @davidjumani

@davidjumani no update/progress on this?

Have tested for VMs, Images and Networks. Testing the rest in progress

TODO once done, add the functionality to redirect to running / stopped vms via dasboard

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/454 (JID-2205)

Hold-on merging this, we may need more tests.

Ready for review and testing @davidjumani
@blueorangutan package

@rhtyd a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/454 (JID-2227)

Please review/test @davidjumani @Pearl1594 thanks

Did another round of test, please continue testing master instead @Pearl1594 @davidjumani


Placeholder not working for [String] type

tags:
	type: [String]
	label: 'Schlagwörter'
	optional: true
	autoform:
		type: 'tags'
		afFieldInput:
			placeholder: 'Use comma to end tag'
			tagClass: 'label label-default'

The placeholder value for type: [String] is not showing in the autoform field. Works on my String and Date types. Please advise.

It would be up to the package that is providing the "tags" custom input type to support placeholders.


UI does not render CPI values in DS when selecting a newer kubernetes version to update

Internal reference: SURE-8310
Reported in 2.8.3
The downstream cluster was initially created using Terraform script, which clearly contained what we needed to make Vmware integration work. The TF includes the values for the CPI and CSI charts configuration.
The cluster is up and running and working as expected. The issue happens when selecting a newer kubernetes version to upgrade the cluster. In Rancher UI > Cluster Management > Select cluster > Edit Config > Cluster Configuration >Basics > Select a kubernetes version to upgrade cluster > Before saving to apply the upgrade, check the CSI and CPI in the Add-On Config tab.
The values will appear as empty for CPI when they should be accurately rendered.
Workaround:
A) Adding/Typing manually the CPI values in the Rancher UI
B) Modified the helm charts with additional manifests to indicated that some values are true
@rak-phillip was able to reproduce this in 2.9-head as of filing:
Reproduction Steps

Provision a new vSphere RKE2 cluster via Rancher Dashboard

Make the following changes under Cluster Configuration => Basics

Change Kubernetes Version to the lowest available version in the list
Change Cloud Provider to vSphere

Under Cluster Configuration => Add-On Config, make the following adjustments

Enter values for vSphere CPI Configuration => vCenter Host, Data Centers, Username, & Password
Enter values for vSphere CSI Configuration => vCenter Host, Data Centers, Username, & Password

Ensure that all other required for provisioning a vSphere cluster are populated and correct; click Create
Wait for the vSphere cluster to enter a ready state

Modify vSphere cluster created in step 1

Under Cluster Configuration => Basics, modify the Kubernetes version to the latest version in the list
Under Cluster Configuration => Add-On Config

All values for vSphere CPI Configuration are missing
All values for vSphere CSI Configuration are preserved

Optionally, if we change the selected Kubernetes version back to the value we selected in Step 1, we can observe that the vSphere CPI Configuration values are populated with their original values.

Possible candidate for 2.9.x backport


Elastalert still not working with ELK 7.4.0

Hi ,
I had deployed Elastic search  and kibana 7.4.0 versions and elastalert as 0.2.1 from the below docker image.
jertel/elastalert-docker:0.2.1
I deploy it in my kube cluster , but when trying to run the config i am getting the below error.
Need some help on this to fix.
ERROR:root:Error finding recent pending alerts: RequestError(400, u'search_phase_execution_exception', u'No mapping found for [alert_time] in order to sort on') {'sort': {'alert_time': {'order': 'asc'}}, 'query': {'bool': {'filter': {'range': {'alert_time': {'to': '2020-04-10T08:24:53.098208Z', 'from': '2020-04-08T08:24:53.098187Z'}}}, 'must': {'query_string': {'query': '!exists:aggregate_id AND alert_sent:false'}}}}}
Traceback (most recent call last):
File "/opt/elastalert/elastalert/elastalert.py", line 1526, in find_recent_pending_alerts
size=1000)
File "/usr/lib/python2.7/site-packages/elasticsearch/client/utils.py", line 84, in _wrapped
return func(*args, params=params, kwargs)
File "/usr/lib/python2.7/site-packages/elasticsearch/client/init.py", line 852, in search
"GET", _make_path(index, doc_type, "_search"), params=params, body=body
File "/usr/lib/python2.7/site-packages/elasticsearch/transport.py", line 358, in perform_request
timeout=timeout,
File "/usr/lib/python2.7/site-packages/elasticsearch/connection/http_requests.py", line 161, in perform_request
self._raise_error(response.status_code, raw_data)
File "/usr/lib/python2.7/site-packages/elasticsearch/connection/base.py", line 230, in _raise_error
status_code, error_message, additional_info
RequestError: RequestError(400, u'search_phase_execution_exception', u'No mapping found for [alert_time] in order to sort on')
WARNING:elasticsearch:GET http://localhost:9200/elastalert_status/elastalert_status/_search?_source_include=endtime%2Crule_name&size=1 [status:400 request:0.003s]
ERROR:root:Error querying for last run: RequestError(400, u'illegal_argument_exception', u'request [/elastalert_status/elastalert_status/_search] contains unrecognized parameter: [_source_include] -> did you mean any of [_source_includes, _source_excludes]?')
WARNING:elasticsearch:GET http://localhost:9200/logstash/_search?_source_include=%40timestamp%2C%2A&ignore_unavailable=true&scroll=30s&size=10000 [status:400 request:0.009s]
ERROR:root:Error running query: RequestError(400, u'illegal_argument_exception', u'request [/logstash/_search] contains unrecognized parameter: [_source_include] -> did you mean any of [_source_includes, _source_excludes]?')
INFO:elastalert:Ran Logmessages from 2020-04-10 10:09 CEST to 2020-04-10 10:24 CEST: 0 query hits (0 already seen), 0 matches, 0 alerts sent
INFO:elastalert:Sleeping for 59.939058 seconds

I fixed this myself

i have the same error. plz help me

i have the same error. plz help me @vinodhkumart

@taegeum Just remove elastalert_status indices and try freshly


SyntaxError in Punycode.js Line 55

Not sure if this is an uglify-js bug, but when I run a webpack build that uses uglify-js I get this error in the output:
ERROR in bundle.js from UglifyJs
SyntaxError: Unexpected token: name (length) [./~/punycode/punycode.js:55,0]

Line 55 is a map function.
function map(array, fn) {
	const result = [];
	let length = array.length;
	while (length--) {
		result[length] = fn(array[length]);
	}
	return result;
}

Any ideas? Breaking my builds at the moment and I'm at a loss.

Turns out when I dove into my bunde.js file, I found this to be breaking in Firefox 49.0.2
https://github.com/bestiejs/punycode.js/blob/master/punycode.js#L302
and elsewhere in the encode function.
Wherever there was
for (const currentValue of input)...
It was displaying this error message: SyntaxError: missing = in const declaration
So I removed const
for (currentValue of input)...
In all 3 locations and it works fine.
It still isn't explaining why uglify-js is barking, though.
I'm using<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

See the README:

The current version supports recent versions of Node.js only. For the old version that offers the same functionality with broader support, including Rhino, Ringo, Narwhal, and web browsers, see v1.4.1.

I'm using node 6 LTS??
On Saturday, October 22, 2016, Mathias Bynens<EMAIL_ADDRESS>wrote:

See the README https://github.com/bestiejs/punycode.js#readme:
The current version supports recent versions of Node.js only. For the old
version that offers the same functionality with broader support, including
Rhino, Ringo, Narwhal, and web browsers, see v1.4.1
https://github.com/bestiejs/punycode.js/releases/tag/v1.4.1.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bestiejs/punycode.js/issues/57#issuecomment-255540034,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAC8agF0L620geLrgea-OjtzWKqIVC6oks5q2kBAgaJpZM4Kd5hq
.

--
Sent from Joe In Real Life

I was responding to your comment mentioning Firefox 49.0.2. Here’s the Firefox bug you’re seeing: https://bugzilla.mozilla.org/show_bug.cgi?id=449811
The uglify-js issue you’re seeing sounds like a bug with that project :/ Sorry I can’t be of more help. Please let me know if you file an issue there!

Cool thanks!
On Tuesday, October 25, 2016, Mathias Bynens<EMAIL_ADDRESS>wrote:

I was responding to your comment mentioning Firefox 49.0.2. Here’s the
Firefox bug you’re seeing: https://bugzilla.mozilla.org/
show_bug.cgi?id=449811
The uglify-js issue you’re seeing sounds like a bug with that project :/
Sorry I can’t be of more help. Please let me know if you file an issue
there!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bestiejs/punycode.js/issues/57#issuecomment-256156498,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAC8apoRbgFx00pDiHloKft0y0JZJyzUks5q3l3lgaJpZM4Kd5hq
.

--
Sent from Joe In Real Life


Add an icon in line with the new visual guidelines of MacOS

The icon of OpenRefine is out of place with the new graphic charter for m

Proposed solution

Add an icon in line with the new visual guidelines of MacOS

Available solution

https://macosicons.com/#/open refine

Thanks for bringing this up!
It's unclear to me if we can simply reuse the icon you pointed at on macosicons.com. This repository that's associated to the website is in GPL v3, which would not work, but that's not necessarily the license that applies to the icons themselves.
We could also redo the adaptation on our side, following the same idea.
Once we have a suitable ICNS file at hand, it can simply replace this one:
https://github.com/OpenRefine/OpenRefine/blob/master/graphics/icon/openrefine.icns

The licensing discussion is here: https://github.com/elrumo/macOS_Big_Sur_icons_replacements/discussions/1382
It sounds like enough of a mess that we should just modify our own.
Perhaps a good task for the @OpenRefine/designers ?

https://www.figma.com/file/dJSxFNE0AmXnDgwb1nIzXM/Official-macOS-App-Icon-Template-(Community)?type=design&node-id=0-58&mode=design&t=jau7wfipCZUcRUNF-0
@tfmorris @wetneb I redid something in Figma from a template: https://openrefine.org/img/openrefine_logo.svg

The App icons guidelines state that we CAN go a bit larger with the OpenRefine gem coming a bit closer towards the edge of the app Icon grid bounding box.  It looks like we might have constrained the gem a bit too much in your design image above, where it's well within the "Icon grid" square (which is inside the "Icon grid bounding box".  You can probably increase the gem size so that it just touches the edge of the "Icon grid bounding box"?

Keep primary content within the icon grid bounding box; keep all content within the outer bounding box. If an icon’s primary content extends beyond the icon grid bounding box, it tends to look out of place. If you overlay a tool on your icon, it works well to align the tool’s top edge with the outer bounding box and its bottom edge with the inner bounding box, as shown below. You can use the grid to help you position items within an icon and to ensure that centered inner elements like circles use a size that’s consistent with other icons in the system.

Also, we SHOULD add a faint drop shadow under the gem and follow the style of the app-icon template for that.  For example, look at the Safari icon compass and notice the faint drop shadow of the red/white compass needle that is lower-right of the needle.  Apple is encouraging the use of faint drop shadows now when it can keep the icon clear and distinct and there's no text within its image.  We could also do a recessed look, which is an option now as well since Sonoma changed the visual style somewhat.  Read the guidelines and notice the red Music app icon for the macOS where the notes look recessed.  They mention this in the paragraph.

Create a design that works well on multiple platforms so it feels at home on each. If your app or game runs on more than one platform, use similar images and color palettes in all icons while rendering them in the style that’s appropriate for each platform. For example, in iOS, tvOS, and watchOS, the Music app icon depicts the white musical notes on a red background using a streamlined, graphical style; macOS displays the same elements, while adding shadow that makes the notes look recessed. Similarly, the Music app icon in visionOS uses the same color scheme and content, but offers a true 3D appearance when viewed while wearing the device.

Use the drop shadow in the icon-design template. The app-icon template includes the system-defined drop shadow that helps your app icon coordinate with other macOS icons.

Consider using interior shadows and highlights to add definition and realism. For example, the Mail app icon uses both shadows and highlights to give the envelope authenticity and to suggest that the flap is slightly open. In icons that include a tool that floats above a background — such as TextEdit or Xcode — interior shadows can strengthen the perception of depth and make the tool look real. Use shadows and highlights that suggest a light source facing the icon, positioned just above center and tilted slightly downward.


Unit test contributions?

Hello :wave:
Thank you for sharing your nice code :stars:  :earth_asia:!
I am experimenting with exoJAX and see an opportunity to add some unit tests.  Would you welcome contributions of unit tests?  If so, are you open to adopting the py.test framework?  This would be an optional installation, and I think nothing would change to the codebase other than the addition of a tests/ directory in the root of the project with some test scripts.
The main things I want to test automatically are:

[ ] That the installation proceeded correctly (useful when building both exojax and numpyro from source on different machines)
[ ] That the GPU-dependent codes are working with the right CUDA version
[ ] That basic methods work as expected.

Hello! Yes, sure. Thank you for proposing this. Honestly, I am not familiar with pytest. Let me know if there is anything I need to do.


quetz is incompatible with sqlalchemy>=2.0.0

quetz is currently not compatible with sqlalchemy>=2.0.0. This can be seen in this run. This should either be resolved through a pin or by making quetz compatible.

This seems to be because http://repo.mamba.pm/conda-forge doesn't have sqlalchemy>=2.0.0 yet.


revision_count is inaccurate under some conditions

MS/SD looks at the revision information from the SE API's posts/{ids}/revisions endpoint, but does not filter out the items in the returned data that are a change in the post's state rather than actual edits. Non-edits in the SE API revisions data don't have a revision_number.

Relevant code: https://github.com/Charcoal-SE/metasmoke/blob/4f2e42a9608f63c5be81be594e8a1cd115c64686/app/models/concerns/post_concerns/autoflagging.rb#L200


scripts/pkg: improved detection of active mounts

On systems which have /var/lib/rkt as a separate partition, the active
mount detection in before-remove needs to not get confused by the
presence of /var/lib/rkt itself as a mount.  Therefore a longer path
is used for active mount detection.
Fixes #3706

@squeed PTAL

Oh man, good catch. Checking.

LGTM!


Blackbox = true not respected for arrays

If you add blackbox = true to an array field - it still asks for the child definition and throws an error. e.g.
  methods: {
    type: Array,
    optional: true,
    blackbox: true
  }

Throws this error:
.. \"methods\" is Array type but the schema does not include a \"methods.$\" definition for the array items\"\n    at checkAndScrubDefinition

Try the following (I haven't tested, but let me know if it works):
methods: {
    type: Array
}
'methods.$': {
    type: Object,
    blackbox: true
}

That's what I'm currently using.  But that's contrary to the document API. With blackbox set you shouldn't have to set 'methods.$'.
methods: {
    type: Array
},
'methods.$': {
    type: Object,
    blackbox: true
}

I'm not sure the documentation mentions using blackbox: true on a field with type: Array; isn't it only intended for use with type: Object?

Rereading the doc's you are right. Think this is a new thing from 6th Feb (3e1cacf0cc3a67e04672bcca7f666b62b263e243), which explains why I only started getting problems recently.
Cheers @chrisbobbe!


Parquet writer should not write any min/max data to ColumnIndex when all values are null

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
While investigating #6310 I noticed that when a page contains only null values, min and max for that page will be set to vec![0; 1]. Because of this, a byte array of length 1 will be written. Instead an empty vec should be passed so spurious min/max values are not produced.
Describe the solution you'd like
Rather than vec![0; 1], vec![] should be passed to ColumnIndexBuilder::append. This will reduce the column index size a tiny bit.
Describe alternatives you've considered
It could be the current behavior is desired, so no change would be necessary.
Additional context
The current behavior isn't a bug, but is a little wasteful.

take

label_issue.py automatically added labels {'parquet'} from #6316


Update index.rst

Removed step that required to add 'django.template.loaders.app_directories.Loader' is in TEMPLATES setting as it's no longer needed.

Thanks you're right! Let's remove this.