Support multiple axes for operations in TF backend

Summary
Improves documentation for axis argument (i.e. we should accept a list of integers as well).
Related Issues
PR Overview

[y] This PR requires new unit tests [y/n] (make sure tests are included)
[y] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[y] This PR is backwards compatible [y/n]
[n] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Thanks for the PR!
While it's true that it would be nice if all those functions were to accept multiple axes through the axis argument, we don't know if they can really do it until we test it. All those tests are in keras/tests/keras/backend/backend_tests.py around line 352.
Some of them (like min or mean) are already tested with lists as axis, but some other not (like max). Do you think you would be able to add the missing tests? If not, no worries, I'll do it.

why only in TF backend? the tests have shown that it works in all backends :)

Thanks a lot for this PR!

why only in TF backend? the tests have shown that it works in all backends :)

Whoa, cool :)
Then we should probably update the documentation for the other backends.
Thank you


docs(ssr): add a missing closing parenthesis

Description

This fixes a small error in the code comment in SSR documentation.
resolve #15428
Additional context

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[x] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines, especially the Pull Request Guidelines.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Update the corresponding documentation if needed.
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

I see! Thank you for reviewing 🙂


Prowler v4 deprecations from v3

Deprecations from v3
General

Allowlist now is called Mutelist.
The --quiet option has been deprecated, now use the --status flag to select the finding's status you want to get from PASS, FAIL or MANUAL.
All INFO finding's status has changed to MANUAL.
The CSV output format is common for all the providers. Check the equivalences from v3 here.

We have deprecated some of our outputs formats:

The HTML is replaced for the new Prowler Dashboard, run prowler dashboard.
The native JSON is replaced for the JSON OCSF v1.1.0, common for all the providers. Here there is a table with the equivalences https://docs.prowler.com/projects/prowler-open-source/en/latest/tutorials/reporting/#json.

AWS

Deprecate the AWS flag --sts-endpoint-region since we use AWS STS regional tokens.
To send only FAILS to AWS Security Hub, now use either --send-sh-only-fails or --security-hub --status FAIL.

Read more about the v4 release here.

Note that as v4.1.0 Security Hub will receive muted fails.


Add Pelias Update Job

Checklist

[x] Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
[x] Any modified or new methods or classes have helpful JavaDoc and code is thoroughly commented
[x] The description lists all applicable issues this PR seeks to resolve
[x] The description lists any configuration setting(s) that differ from the default settings
[x] All tests and CI builds passing

Description
Along with https://github.com/ibi-group/datatools-ui/pull/699 will support sending a Deployment's GTFS feeds to Pelias along with some custom CSV

Thanks Evan! Assigning to Landon now for round 2.

Thanks Landon! All is updated


Import-SPOTaxonomy: crash when reused terms are in the importfile

Inter-tenant migration:

All terms (including 'reused' terms) are exported with Export-SPOTaxonomy correctly from the source tenant and have (obviously) the same ID as the source term.
The Import-SPOTaxonomy to the target tenant will crash however when it detects a dupicate term ID.

Feature Request: Would it be possible to have reused terms import successfully to a target tenant?

I believe this is fixed in latest. If you believe this is still an issue in the latest release please re-open and let us know.


Sync Subplots by Name

Currently only done upon initialization. Will work on making this property mutable post-initialization soon.

Now that #280 exists, we're going to need to figure out how to consolidate this :smile:

Anything you want me to do on my end? Or will you handle it?

We'll have to brainstorm a bit, probably at scipy sprint?
On Fri, Jul 7, 2023, 10:06 Arjun Putcha @.***> wrote:

Anything you want me to do on my end? Or will you handle it?
—
Reply to this email directly, view it on GitHub
https://github.com/kushalkolar/fastplotlib/pull/267#issuecomment-1625469622,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACHXXRAQVDV4VH6FYJ75OXLXPAJXDANCNFSM6AAAAAAZYZWKGU
.
You are receiving this because you commented.Message ID:
@.***>

superseeded by #382


V3 - > V4 Migration [V4 SIDE]

Fix for #505
===========

Migration on the V4 side will check if the bots list is empty, and then if that is true, it will check the app folder for a file called migration_marker.txt to see if the V4 side of migration has been performed
if the marker is not there, all the bots in /migration/ will be added to the MRU and the post-migration dialog will be displayed

Sorry, didn't have time to write tests. I can do that when I get back. 👍

Pull Request Test Coverage Report for Build 152

2 of 4 (50.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.008%) to 46.025%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/app/client/src/commands/uiCommands.ts
1
2
50.0%

packages/app/main/src/utils/getFilesInDir.ts
1
2
50.0%

Totals

Change from base Build 149:
0.008%

Covered Lines:
2875

Relevant Lines:
5212

💛  - Coveralls


Fixes&update

Fix switch case for supported operations
Minor optimization when getting method's name
Upgrade to Lotus v0.8.0

Merging but we still have negative values in the reconciliation (related to #66 )


some resolution video's image is broken

some video's image is not working, this images resolution is 1922x1082. I already matched video size. That video was get from this command.
 .\ffmpeg.exe -i MYRTSP_FROM_CAMERA -an -sn -vf scale=1922:1082 -c:v libx264 -vprofile baseline -bufsize 6M -b:v 300k -r 10 -pix_fmt yuv420p -tune zerolatency -preset ultrafast -g 120 -f rawvideo 1922_1082_5sec.h264

I found out some resolution not working

1602x942
1606x944

I want to stream rstp to my webpage and I set the video size for fitting my webpage. please help me. thanks.

have you resolved this problem？i got it,too

@huangqichao  nope

I solved this problem. firstly, using value which can print clearly and edit the size using css style. you should use same video output size, canvas size. and then edit canvas element css size.


Sync 05 13

Upstream Sync. Brings down two bug fixes and a small change to the installation script.
This PR should only merge once QA has verified the bug fixes

/hold ensure QA signs off first

Since it's FF I've split this PR into 2 bug PRs. The rest can be sync'ed later


Adding Makefile to the project.

Description
The issue involves addition of a Makefile to the repository. This will ease the process of Setting up the repository for new contributors.
Acceptance Criteria
One I run make,  build, test steps should run and the repository should be ready to contribute to.

Hello @RajatVaryani
Thank you for opening the issue.
intentionally Makefile was not created in project, rather promote contributors to install the 1build first and run the development command from 1build.yaml in this project.
Feel free address if something is missing from Contribution.md file or 1build.yaml.

Hello, I am fine with it. But there are few things we can improve. For example the we can have prerequisite for the project which will make the setup process easier. We can also get rid of go get -u github.com/golang/dep/cmd/dep dep ensure commands as we don't use dep anymore.

@RajatVaryani You are right.
There are some things to improve in the document.
Your help in updating doc is much appreciated :)


Does it support cjt188 protocol, electronic water meter

What would you like to be added/modified:
Why is this needed:

Water related agreements are being gradually supported. At present, sl651 and hj212 have been supported. The HiCJT188 agreement will be arranged after the Spring Festival

Water related agreements are being gradually supported. At present, sl651 and hj212 have been supported. The HiCJT188 agreement will be arranged after the Spring Festival


Reintroduce Legacy Navigation

Closes: #3936
What's Changed

Created 'NavContext,' 'NavGrouping,' and 'NavItem,' all of which inherit from 'NavMenuBase' and 'PermissionsMixin,' these classes are the new ui navigation equivalents of 'NavMenuTab,' 'NavMenuGroup,' and 'NavMenuItem.'
Reintroduced legacy UI navigation classes: 'NavMenuTab', 'NavMenuGroup', and 'NavMenuItem'.
Enhance 'core.apps.create_or_check_entry' to recursively create and sort 'registry["new_ui_nav_menu"]' items for new ui.

TODO

[ ] Explanation of Change(s)
[ ] Added change log fragment(s) (for more information see the documentation)
[ ] Attached Screenshots, Payload Example
[ ] Unit, Integration Tests
[ ] Documentation Updates (when adding/changing features)
[ ] Example Plugin Updates (when adding/changing features)
[ ] Outline Remaining Work, Constraints from Design

Probably need to also revert/update nautobot/docs/development/navigation-menu.md to undo most of the changes from #3652.


Refactor abc.run()

This is unnecessary because it's part of changing the algorithm. There's no point doing work on what is going to be obsolete code.


iOS 14.4. Photo upload from driver did not work.

-Login as driver user.
-Ensure driver has at least 1 entry under pending.
-click on entry and click delivered. User will see camera enabled to click photo.
-take a photo and select "use photo"
Observed: photo upload does not work.
Expected: upload should work and transaction completed.

This is caused by the missing directory for uploads in the backend. Would be fixed server side.

It seems fixed now.

This was caused by missing claims assignment for the users, would be fixed server side.

Tracked in https://github.com/bigpanther/trober/issues/105


WIP: Rolldown build

E2E Test Status
windows ci has error

Playground Name
Dev
Build
Reasons

alias
✅
✅

assets
✅
✅
- rollup hash asset using base64, but replace - to $  - disable rebuild test

assets-sanitize
✅
✅

backend-integration
✅
❌
https://github.com/rolldown/rolldown/issues/1842#issuecomment-2296345255

build-old
✅
✅
-

cli
✅
✅
-

cli-module
✅
✅
-

config
✅
✅
-

csp
✅
✅
-

css
✅
✅
-

css-codesplit
✅
❌
manualChunks

css-codesplit-cjs
✅
✅
-

css-dynamic-import
✅
❌
https://github.com/rolldown/rolldown/issues/1842

css-lightningcss
✅
✅
-

css-lightningcss-proxy
✅
✅
-

css-no-codesplit
✅
✅
-

css-sourcemap
✅
✅
-

data-uri
✅
✅
-

define
✅
✅
-

dynamic-import
✅
❌
https://github.com/rolldown/rolldown/issues/1842 https://github.com/rolldown/rolldown/issues/1843

dynamic-import-inline
-
✅

env
✅
✅
-

env-nested
✅
✅
-

extensions
✅
✅
-

external
✅
❌
https://github.com/rolldown/rolldown/issues/2041

fs-serve
✅
✅
-

glob-import
✅
❌
- remove empty CSS generate chunk https://github.com/rolldown/rolldown/issues/1842

hmr
✅
✅
-

hmr-root
✅
-
-

hmr-ssr
✅
✅
-

html
✅
✅
-

import-assertion
✅
✅
-

js-sourcemap
✅
❌
manualChunks

json
✅
✅
-

legacy
❌
❌
🟡pending support system format

lib
❌
❌
🟡pending support umd format

minify
-
✅
-

module-graph
✅
-
-

multiple-entrypoints
✅
✅
-

nested-deps
✅
✅
-

object-hooks
✅
❌
object hook sequential

optimize-deps
✅
❌
- the rolldown missing export always is error, but the rollup has error or warn level, ref https://github.com/rolldown/vite/pull/27/commits/bfc679efab9d3eb0ef3cee1611506518bcc33a23  -https://github.com/rolldown/rolldown/issues/2031  - using Array input generate name is not expected, the entry name will be like index~1.js

optimize-deps-no-discovery
✅
✅
-

optimize-missing-deps
✅
✅
-

preload
✅
✅
-

preserve-symlinks
✅
✅
-

proxy-bypass
✅
✅
-

proxy-hmr
✅
✅
-

resolve
✅
✅
- rolldown exit process issue caused vitest error if run test in parallel

resolve-config
-
✅

ssr
✅
✅
-

ssr-alias
✅
✅
-

ssr-conditions
✅
✅
-

ssr-deps
✅
✅
-

ssr-html
✅
✅
-

ssr-noexternal
✅
-
-

ssr-pug
✅
✅
-

ssr-resolve
-
✅
-

ssr-webworker
✅
✅

tailwind
✅
✅
-

tailwind-sourcemap
✅
✅
-

transform-plugin
✅
✅

tsconfig-json
✅
✅
-

tsconfig-json-load-error
✅
✅
-

wasm
✅
✅
-

worker
✅
❌
-https://github.com/rolldown/rolldown/issues/1842  -https://github.com/rolldown/rolldown/issues/1010

hook call cost for benchcases/20000modules

vite:esbuild
  |- transform: 25867596.34ms
  |- load: 1.68ms
  |- transform_ast: 0.00ms
  |- resolve: 0.00ms
vite:load-fallback
  |- load: 7975644.26ms
  |- transform: 7.36ms
  |- transform_ast: 0.02ms
  |- resolve: 0.00ms
alias
  |- resolve: 6765733.93ms
  |- load: 0.52ms
  |- transform: 0.05ms
  |- transform_ast: 0.00ms
vite:modulepreload-polyfill
  |- load: 2099766.14ms
  |- resolve: 196862.31ms
  |- transform: 0.16ms
  |- transform_ast: 0.00ms
vite:css
  |- transform: 1300225.05ms
  |- load: 67072.74ms
  |- transform_ast: 0.00ms
  |- resolve: 0.00ms
vite:resolve
  |- resolve: 644735.49ms
  |- load: 564377.40ms
  |- transform: 0.10ms
  |- transform_ast: 0.00ms
vite:json
  |- transform: 455512.22ms
  |- load: 0.70ms
  |- transform_ast: 0.02ms
  |- resolve: 0.00ms
vite:wasm-helper
  |- load: 338319.71ms
  |- transform: 2.35ms
  |- transform_ast: 0.01ms
  |- resolve: 0.00ms
vite:dynamic-import-vars
  |- load: 262145.11ms
  |- transform: 60529.64ms
  |- transform_ast: 0.01ms
  |- resolve: 0.00ms
vite:build-import-analysis
  |- load: 93937.80ms
  |- transform: 92939.95ms
  |- transform_ast: 0.01ms
  |- resolve: 0.00ms
vite:worker
  |- transform: 82788.87ms
  |- load: 68149.93ms
  |- transform_ast: 0.01ms
  |- resolve: 0.00ms
vite:asset
  |- load: 99090.04ms
  |- transform: 1.84ms
  |- transform_ast: 0.02ms
  |- resolve: 0.00ms
vite:define
  |- transform: 89704.89ms
  |- load: 1.10ms
  |- transform_ast: 0.00ms
  |- resolve: 0.00ms
vite:wasm-fallback
  |- load: 65423.90ms
  |- transform: 0.81ms
  |- transform_ast: 0.02ms
  |- resolve: 0.00ms
vite:html-inline-proxy
  |- load: 62983.32ms
  |- transform: 0.03ms
  |- transform_ast: 0.01ms
  |- resolve: 0.00ms
vite:css-post
  |- transform: 50803.16ms
  |- load: 0.48ms
  |- transform_ast: 0.01ms
  |- resolve: 0.00ms
vite:reporter
  |- transform: 48866.64ms
  |- load: 1.36ms
  |- resolve: 0.00ms
  |- transform_ast: 0.00ms
vite:asset-import-meta-url
  |- transform: 42374.47ms
  |- load: 0.31ms
  |- transform_ast: 0.02ms
  |- resolve: 0.00ms
vite:build-html
  |- transform: 41530.89ms
  |- load: 0.40ms
  |- transform_ast: 0.01ms
  |- resolve: 0.00ms
vite:worker-import-meta-url
  |- transform: 35687.16ms
  |- load: 0.49ms
  |- transform_ast: 0.00ms
  |- resolve: 0.00ms
vite:build-metadata
  |- load: 4.54ms
  |- transform: 2.74ms
  |- resolve: 1.78ms
  |- transform_ast: 0.01ms
glob_import_plugin
  |- transform_ast: 1.79ms
  |- transform: 0.44ms
  |- load: 0.40ms
  |- resolve: 0.00ms
rolldown:data-url
  |- transform: 1.25ms
  |- transform_ast: 0.02ms
  |- resolve: 0.00ms
  |- load: 0.00ms
vite:watch-package-data
  |- load: 0.32ms
  |- resolve: 0.14ms
  |- transform: 0.08ms
  |- transform_ast: 0.00ms

After bumping rolldown to 0.13.2 locally, I encountered this bug: https://github.com/rolldown/rolldown/issues/2183
It's strange that the reproduction doesn't work even though the test in this branch passes with the version before bumping (0.12.2-snapshot-aa5807f-20240902003235).

After bumping rolldown to 0.13.2 locally, I encountered this bug: rolldown/rolldown#2183 It's strange that the reproduction doesn't work even though the test in this branch passes with the version before bumping (0.12.2-snapshot-aa5807f-20240902003235).

What case you meet it, I can't found it.

After bumping rolldown to 0.13.2 locally, I encountered this bug: rolldown/rolldown#2183 It's strange that the reproduction doesn't work even though the test in this branch passes with the version before bumping (0.12.2-snapshot-aa5807f-20240902003235).

What case you meet it, I can't found it.

This test case failed because sub_1.css didn't exist.
https://github.com/rolldown/vite/blob/75371b9173d11116a6f775313c88325458ee7e51/playground/css/__tests__/same-file-name/css-same-file-name.spec.ts#L15-L17
This is the branch I'm testing on: https://github.com/rolldown/vite/compare/rolldown-build...rolldown:vite:rolldown-build-investigate (note that ./playground/css/__tests__/same-file-name/*.spec.ts is excluded in that branch)

Yeah. It's strange and I also can't found it at now, the test how to work 😭. I remember i changed it because here has a filename different and all file is exits...

Fixing https://github.com/rolldown/rolldown/issues/2183 unveiled a different bug: https://github.com/rolldown/rolldown/issues/2207

Fixing rolldown/rolldown#2183 unveiled a different bug: rolldown/rolldown#2207

Thank you. I will fixed it at next.

Closing as I rebased this PR on the main branch in #43


fix prepended/appended_text not rendering html content

This should be in the code, as the Crispy forms docs say that this is possible, and in the corresponding bootstrap3 component it is included.
fixes #94

Thanks for the PR. 👍
I think I'd prefer to fix this by calling mark_safe() here. This will allow for removal of |safe in all of the template packs, refs https://github.com/django-crispy-forms/django-crispy-forms/issues/296

Looking at this again. This works if you wrap the string in django's mark_safe.
The crispy-form docs have also since been updated to make the same recommendation.

Yeah, that's better.


The resource file 'res:/dx9/model/station/soe/soes1/soes1.red' doesn't load and results in errors

Loading the SOE station 'res:/dx9/model/station/soe/soes1/soes1.red' that was mentioned in the last ccpwgl_int.js file update doesn't load and results in the following error on Chrome and Canary:
WebGL: INVALID_ENUM: drawElements: invalid type

We try to use a webgl extension for 32 bit index buffers for objects requiring more than 65536 vertices (https://www.khronos.org/registry/webgl/extensions/OES_element_index_uint/). SOES1 seems to be one of them. I would imagine this extension is available on most desktop machines. What is the OS on your machine?

Windows 8 64bit
On 14/05/2015 1:17 AM, "filipppavlov"<EMAIL_ADDRESS>wrote:

We try to use a webgl extension for 32 bit index buffers for objects
requiring more than 65536 vertices (
https://www.khronos.org/registry/webgl/extensions/OES_element_index_uint/).
SOES1 seems to be one of them. I would imagine this extension is available
on most desktop machines. What is the OS on your machine?
—
Reply to this email directly or view it on GitHub
https://github.com/ccpgames/ccpwgl/issues/27#issuecomment-101660134.

this.gl.getExtension("OES_element_index_uint") is missing from the Tw2Device function (Thanks @aurora). Once this is called the model will load although I get the following errors:

Error: WebGL: drawElements: bound element array buffer is too small for given count and offset
Missing Normals on the Station Glass.

This should be fixed with the latest commit.


No access to web pages when running with Screen Time

AdGuard version
2.11
Browser version

OS version
Ventura 13.4
What filters do you have enabled?
No response
What Stealth Mode options do you have enabled?
No response
Support ticket ID
No response
Issue Details
Steps to reproduce:

Open AdGuard (default settings)
Go to system settings - Screen Time - Content&Privacy - Content Restrictions - Access to Web Content - Limit Adult Websites - Customize then add example.org to Restricted sites
Open any browser and search for anything.

Expected Behavior
No issues while surfing websites
Actual Behavior
Websites don't open
Screenshots
Screenshot 1:

Additional Information
No response

Content filtering of screen time is breaking AdGuard as well as many other filtering apps, may break VPNs also.
It is recommended to turn off "Content & Privacy" of "Screen time" (reboot is needed to fully turn off it), and use AdGuard's functionality instead.

@Stillness-2 can we detect screen time and and alert the user about it?

Unfortunately, there is no way to clearly detect that content filtering of screen time is enabled.


multi-user: provide example

@SergeyYakubov

Nice! This works well for me. The only comment I have is that it seems to run at localhost:8080 instead of localhost:8080/app even after setting EP_PATH.

Indeed but I though those path will be managed outside of the container in a gateway or something.
Why do you need to have a specific path within that docker image? Also does that need to be a runtime or build time option for within docker handling?

@SergeyYakubov could correct me if I'm wrong, but because we're integrating these apps as tools into Galaxy (workflow management platform we're using) these apps get assigned paths dynamically. Like a user could potentially click and launch the tool (or multiple), and then Galaxy will start the app, assign it a path (partially using EP_PATH, etc. The ngnix config in the repo kinda illustrates what I'm talking about.

I get the dynamic part which means it is really at runtime and not build time.
But I guess my next question, does the service needs to maintain internally the path assigned on the fly by galaxy or the dynamic infrastructure of galaxy allow to rewrite URL and map them to the root of a running service?

Ah, I get what you're saying.Yeah, it's probably not an issue, as I believe we actually can just do this via Galaxy. I'm fine with merging this if @SergeyYakubov also approves.

I'm currently working on enabling that base path in our docker image. So worst case scenario, you will have that available if need be.

It depends on the app and how it is implemented. Galaxy can redirect calvera.ornl.gov/app/xxx/ to somehost/xxx. But if the app is unaware of /app and wants to route to /yyy instead of /app/yyy, we'll have a problem. Which might not be the case with Trame, I'm not sure here.

The app is aware internally and, therefore, can adjust any URL call if need be.

So, I tried integrating your branch with this repository's dev branch, and I was running into issues getting it working. Since the dev branch is where all of the actual development is happening, can you create an mr to that branch instead and see if it works there (from docker)?

Ok I'll start a new branch from dev


v0.8.3 upgrade issue

Hi there
I was upgrading from v0.8.2 and noticed that this error came up:
Argument 1 passed to Dingo\Api\Routing\ControllerReviser::revise() must be an instance of Dingo\Api\Routing\Route, instance of Illuminate\Routing\Route given, called in F:\xampp\htdocs\vendor\dingo\api\src\Event\RevisingHandler.php on line 51 and defined

In my controller:
<?php namespace API;

use \Illuminate\Routing\Controller;
use \Dingo\Api\Routing\ControllerTrait;

class LikeController extends \Controller {

    use ControllerTrait;

    public function blah($id)
    {
        ...
    }
}

I made it pretty much just as the docs say, but get the error. How can it be fixed?

Can I see your routes file please?
On Wed, 1 Apr 2015 18:29 Lorenc<EMAIL_ADDRESS>wrote:

Hi there
I was upgrading from v0.8.2 and noticed that this error came up:
Argument 1 passed to Dingo\Api\Routing\ControllerReviser::revise() must be an instance of Dingo\Api\Routing\Route, instance of Illuminate\Routing\Route given, called in F:\xampp\htdocs\vendor\dingo\api\src\Event\RevisingHandler.php on line 51 and defined`
In my controller:
.

Route::api(['version' => 'v1', 'prefix' => 'api'], function()
{

    // Likes
    Route::get('like/{id}', array('as' => 'api.like', 'uses' => 'API\LikeController@like', 'before' => 'auth'));
}

There you go

I noticed this error as well after an upgrade, must be something that got left behind. Any clue on where to look?

Not upgrading, simply trying to use for the first time. Get the same error.

Argument 1 passed to Dingo\Api\Routing\ControllerReviser::revise() must be an instance of Dingo\Api\Routing\Route, instance of Illuminate\Routing\Route given, called in /Users/randomdrake/Sites/smokereports/smokereports/laravel/vendor/dingo/api/src/Event/RevisingHandler.php on line 51 and defined

Downgraded from 0.8.* to 0.8.2 and it works just fine.

Just upgraded from 0.8.2, same issue.

+1 same issue after upgrading from 0.8.2 to 0.8.3

I get this error when i've got a route in the api endpoint with the same name as in the web endpoint.
Route::api(['version' => 'v1'], function () {
    Route::get('/update', '\App\Controllers\Api\TestController@test');
});
...
Route::get('/update', array('as' => 'update-yourself', 'uses' => 'HomeController@getUpdateYourself'));

Breaks on version 0.8.3. works on 0.8.2

Sorry guys but I'm no longer supporting Laravel 4.x going forward. Have to focus my attention on L5 and Lumen.


Configurable Artist and Title remapping

Allow to configure the artist and title web lookup, so things like multiple artist, exotic chars or personalized tags can be handled via TF:
For ex. [$stripprefix($meta(artist,0))] or [$replace(%title%,$char(40),,$char(41),)]...
Example from Lyrics Show 3:

Note this allows an infinite amount of possibilities to fix common problems looking for things.

Certainly a useful tool for power usages like yourself. If I understand correctly this would supersede the "Exclude text in brackets at the end  of artist/album names" checkbox that we currently have.
We'd need to either continue supporting that option, or set a default if users have that option enabled (presumably involving $cut and $strrchr).

Without regexp is not trivial, but here it is:

-> My [hey] artist [ho]
$puts(char,'[')$puts(tag,My '['hey']' artist '['ho']')$puts(spacer,$strrchr($get(tag),$get(char)))
$ifequal($strrchr($get(tag),']'),$len($get(tag)),$trim($left($get(tag),$sub($get(spacer),1))),$get(tag))
-> My [hey] artist

-> My [hey] artist
$puts(char,'[')$puts(tag,My '['hey']' artist)$puts(spacer,$strrchr($get(tag),$get(char)))
$ifequal($strrchr($get(tag),']'),$len($get(tag)),$trim($left($get(tag),$sub($get(spacer),1))),$get(tag))
-> My [hey] artist

It only removes the brackets if the last char is ']':
$puts(char,'[')$puts(tag,%ARTIST%)$puts(spacer,$strrchr($get(tag),$get(char)))
$ifequal($strrchr($get(tag),']'),$len($get(tag)),$trim($left($get(tag),$sub($get(spacer),1))),$get(tag))

Minified:
$ifequal($strrchr(%ARTIST%,']'),$len(%ARTIST%),$trim($left(%ARTIST%,$sub($strrchr(%ARTIST%,'['),1))),%ARTIST%)

Personally I would leave the option as is, since it's already coded, on top of the TF.


Offer more download options

It should be more convenient to download all icons, or only specific icons.

"Download all" button
Download option for single icons in the overlay
Possibly as SVG sprite sheet and icon font

This needs to be synced with the download options in the main icons repo.

The "Download all" button got addressed in #7. It also added an option copy icon SVGs to the clipboard, which addresses the second bullet.


Onlyoffice does not start when editing shared documents.

Onlyoffice does not start when editing shared documents.
Own documents can be edited.
I'm using onlyoffice-documentserver 5.3.1-26 on CentOS Linux 7.6.1810.
Owncloud version <IP_ADDRESS> (community).

#270
Fixed b429d15


Update all Linux images to ubuntu-18.04

Looks good. Although is there a way to inherit the matrix from a common place and allow overriding them?

I believe @danieljurek is working on this refactoring.

/azp run js - storage-blob - tests

All PR pipelines passed.  I started a live test pipeline for a little more validation, if this passes I think we are good to merge.
@danieljurek: I would like your signoff on this as well since you have been looking at these YML files recently.


feat: Add kmods installer module

Credits: @C0dePlayer
This is not ideal as it does not support custom kernels & it involves editing Containerfile.
Custom kernels is something that upstream should solve I believe, so it's not a concern we should direct to.
I believe there is no other way but to make users edit Containerfile for those files to be even pulled of.
I would like this to be through the recipe only, so I will put this as a draft until some better ideas come.
2nd part of this work:
https://github.com/ublue-os/bling/pull/89

How big are the akmods, like, would it hurt to just always COPY them in the Containerfile (as is done with bling rpms and files). This would make just the module be used to configure akmods installation.

 This would make just the module be used to configure akmods installation.

It's 2MB when I pulled it with podman. Not big imo

Then I'd probably pull main by default and just document it well that you need to change that if you're building asus or surface.

Then I'd probably pull main by default and just document it well that you need to change that if you're building asus or surface.

That's only needed if users want to use older version of Fedora as a base (GTS as the most used option of those).
Users can use -nvidia, -asus, -surface etc. builds as a base if they want those akmods.
 This would make just the module be used to configure akmods installation.
Also, the tag part could be main-${IMAGE_MAJOR_VERSION}, but if someone wanted to build main, asus & surface images they'd need to edit build.yml...

I forgot to mention, when I set ${IMAGE_MAJOR_VERSION}, it fails
The guy who originally made akmods module used this workaround:
https://github.com/C0dePlayer/silverflow/blob/live/Containerfile

Adding IMAGE_MAJOR_VERSION as an argument only works when building an image offline. Github Action fails.
So I think it's good as it is.
It's a shame that akmods repo doesn't have -latest tag available though. I may file an issue on this.

If this "latest" tag is not considered in akmods repo, I believe this is ready to merge, as 2nd part in bling is finally ready.

To merge, please update the branch @fiftydinar

To merge, please update the branch @fiftydinar

Updated


Tracking down instigators of SAVE

Love this package! :)
I'm using redux-storage-decorator-filter to restrict the save calls to a specific part of the state. I also just started using redux-sagas, so I put the three EFFECT_* actions into the storage blacklist, and they both work together beautifully.
However, I'm getting an extra REDUX_STORAGE_SAVE action which I have no clue why it's being dispatched, and there is no information I can see to help me figure it out. How do I track down the part of the code that caused the save operation to happen ... ?
Thanks !
-Guy

You're welcome! :sunglasses:
It should be pretty easy, as redux-storage simply dispatches a whole new action (or in other words: no special dispatching magic involved!). So the action right before REDUX_STORAGE_SAVE should be the action that has triggerd this save operation.
NOTE: If your using redux-storage-decorator-debounce this relation is no longer valid, as it's the main job of decorator-debounce to delay the save operation :wink:
Or is this the way you already tried to tackle it with no success?

Thanks for responding :)
I've tried to track it down like you said, but redux-sagas uses actions very liberally (almost every API call results in two actions) so it makes tracking down the source of the save very confusing. What I can say for sure after following a single flow, is that the save action is dispatched even if the filtered state doesn't change.
I'm also using redux-logger to log all of the actions, but just to make sure I listened in to the redux-sagas actions and JSON.stringified the state to the console. I'm seeing a bunch of EFFECT_TRIGGERED and EFFECT_RESOLVED, but the state does not change :(
The only thing that comes to mind is that maybe the before state and after state, while identical, aren't the same object - immutability etc. But it's really grasping at straws for me at this point...
Any suggestions as to how to further break this down and hunt the errant code?

@guybenron Would it be possible for you to checkout and use PR #102? This adds meta.origin to the action dispatched by redux-storage (only if NODE_ENV !== 'production').

Looks like it's what I need, however something came up and I won't be able to check it out :(
If and when it becomes relevant again, will definitely look into it...
Again, thank you !!


GROOVY-9637: Improve the performance of GString

Co-authored-by: Daniel Sun<EMAIL_ADDRESS>

I tried the PR and found no performance improvement. I assume toString() has not used cachedStringLiteral.
D:\_TEMP\GROOVY-9637>groovy -v
Groovy Version: 4.0.0-SNAPSHOT JVM: 11.0.7 Vendor: Azul Systems, Inc. OS: Windows 10

D:\_TEMP\GROOVY-9637>groovy Groovy9637.groovy
8758ms

D:\_TEMP\GROOVY-9637>groovy Groovy9637.groovy
8651ms

D:\_TEMP\GROOVY-9637>groovy Groovy9637.groovy
9018ms

FYI. https://github.com/apache/groovy/pull/1309#issuecomment-662822845

The performance is improved a lot ;-)
The costed time is reduced from 8700ms to 370ms
D:\_TEMP\GROOVY-9637>groovy -v
Groovy Version: 4.0.0-SNAPSHOT JVM: 11.0.7 Vendor: Azul Systems, Inc. OS: Windows 10

D:\_TEMP\GROOVY-9637>groovy Groovy9637.groovy
399ms

D:\_TEMP\GROOVY-9637>groovy Groovy9637.groovy
365ms

D:\_TEMP\GROOVY-9637>groovy Groovy9637.groovy
361ms

D:\_TEMP\GROOVY-9637>groovy Groovy9637.groovy
290ms

D:\_TEMP\GROOVY-9637>groovy Groovy9637.groovy
337ms

D:\_TEMP\GROOVY-9637>

It is still a breaking change but much fewer cases now:
def s = ['The ', ' brown '] as String[]
def v = ['quick', 'fox'] as Object[]
def gstr = new org.codehaus.groovy.runtime.GStringImpl(v, s)
println gstr // The quick brown fox
v[0] = 'sly'
s[1] = ' old '
println gstr // with caching: The quick brown fox, before: The sly old fox

Normally the input array isn't accessible, so perhaps not a problem.

Using GString#plus can return a cacheable GString and doesn't impact the existing GStrings:
long b = System.currentTimeMillis()
def g1 = "integer: ${1}, double: ${1.2d}, string: ${'x'}"
def g2 = "class: ${Map.class}, boolean: ${true}"
def g3 = g1 + ' ' + g2
for (int i = 0; i < 10000000; i++) {
	g1.toString()
	g2.toString()
	g3.toString()
}
long e = System.currentTimeMillis()
println "${e - b}ms"

It is still a breaking change but much fewer cases now:
def s = ['The ', ' brown '] as String[]
def v = ['quick', 'fox'] as Object[]
def gstr = new org.codehaus.groovy.runtime.GStringImpl(v, s)
println gstr // The quick brown fox
v[0] = 'sly'
s[1] = ' old '
println gstr // with caching: The quick brown fox, before: The sly old fox

Normally the input array isn't accessible, so perhaps not a problem.

Agreed :-)

@blackdrag If you get a chance, please cast your eye over this PR instead of the earlier one

@paulk-asert Should we take the multi-thread issues into account?

This again puts a big question mark on the split between GString and GStringImpl. I think code like those instanceof checks should be avoided. Could we shortly summarize the goal? I am not sure if the focus is set right in total.

Goal: improve the performance of GStrings where possible but without foregoing behavior where a value can be dynamically supplied by a closure including lazily evaluated
Highly desired: don't break binary compatibility unless we really have to to minimise impact on any frameworks that might be extending the existing classes in ways unknown to us
Highly desired: should be invisible to the user
Somewhat desired: don't break less commonly used practice of mutating strings/values if possible
Current approach: cache only where values are simple types and known to be immutable - and skip caching even in that case if the values/strings are accessed externally
The pre-existing design (class split and array inputs/outputs) all work against maximising the potential cases which could be cached, so the current PR is very much "handle quick wins" and nothing more. We could totally re-design those classes and have a major breaking change and deal with more cases but we'd possibly forgo some existing dynamic GString use cases and for a dynamic language you could argue little would be achieved because metaprogramming could alter what should be output each time. We currently have things like GROOVY-2599 and hard-coded InvokerHelper paths actually limiting metaprogramming impact somewhat but they might be things we want to fix down the track.

Goal: improve the performance of GStrings where possible but without foregoing behavior where a value can be dynamically supplied by a closure including lazily evaluated

I see that as the main goal and the other points less as strict constraints or subgoals, more as nice-to-have

Highly desired: don't break binary compatibility unless we really have to to minimise impact on any frameworks that might be extending the existing classes in ways unknown to us
Highly desired: should be invisible to the user
Somewhat desired: don't break less commonly used practice of mutating strings/values if possible
Current approach: cache only where all values are simple types and known to be immutable - and skip caching even in that case if the values/strings are accessed externally

is being able to subclass GString still desired. I mean it goes a little bit into binary compatibility, but of course only if you are actually subclassing GString and even then only maybe

The pre-existing design (class split and array inputs/outputs) all work against maximising the potential cases which could be cached, so the current PR is very much "handle quick wins" and nothing more. We could totally re-design those classes and have a major breaking change and deal with more cases but we'd possibly forgo some existing dynamic GString use cases and for a dynamic language you could argue little would be achieved because metaprogramming could alter what should be output each time. We currently have things like GROOVY-2599 and hard-coded InvokerHelper paths actually limiting metaprogramming impact somewhat but they might be things we want to fix down the track.
@danielsun1106 chime in if you have any differing view

I like that we have the optimization in GStringImpl now. The part with the internal in GString is still bothering me though:

return gs instanceof GStringImpl ? ((GStringImpl) gs).getStringList().toArray(EMPTY_STRING_ARRAY) : gs.getStrings();

I have not tried it out, but I would like this to be considered:

change implementation to work with lists instead of array
remove the getInternal methods and use getList methods instead

this will allow the list variant to become the fast case in the sense of not having to convert from an array...
Now I am rethinking the approach... what about if we

remove the list methods and internal methods in GString
have a "freeze" method that will return a new GString, which will potentially ignore changes to the array. I would use that in GStringImpl only though
have GString more or less like it is now, which means no string cache at all
give GStringImpl control over the optimization
calling getValues and getStrings on GString and GStringImpl will then still invalidate the cached String, unless the GStringImpl has been frozen
a frozen GStringImpl will always return the cached string
have non invalidating methods to return the values and strings arrays in GStringImpl and use them for toString
move the string building code of GString into a utility class and call it from GString and GStringImpl with their respective method getValues/getStrings methods

API writers can the use "freeze" if they want to look at the structure of a GString, without maybe causing a performance hit - but actually I think the cases where we look at the structure of a GString have much bigger code following that. Actually I think the freeze feature is more a nice-to-have. After all, not having freeze does not worsen the situation compared to today at all. And of course API users that mutate the arrays will still be able to change the GString.
To show what I mean: https://github.com/apache/groovy/pull/1327
Of course that can be improved by a bit of refactoring and moving the static methods as well as the logic for build and writeTo into a utility class, that is then used by GString and GStringImpl to avoid code duplication and have the only the call to the utility method and the getting of values and strings in the class itself.

Closing this one but see:
https://github.com/apache/groovy/pull/1329


Research: Badges/Achievements

Objective
To develop and implement an MVP of badges/achievements for DevHub by Q4, aligning with DevHub's long-term developer engagement goal.
Background
Developers are the most crucial part of DevHub, and their engagement is essential for building a thriving ecosystem. Gamification, through the introduction of badges, is a key component of this.
Scope
The MVP should encompass the following aspects:

Identify and design 10-15 badges that are relevant to developers (e.g. writing code, participating in hackathon, etc). Collaborate with each DevHub core leader to identify potential badges. Work with DevHub designer (Sasha) or community. Consider running an engagement campaign too.
Introduce a dedicated section on user profiles to display earned badges.
Establish a mechanism for awarding badges (manual/automatic?)
Establish a mechanism for evaluating success/metrics of these badges
Define a process for the community to request or suggest new badges
Explore giving DevHub communities the ability to introduce their own badges (maybe for future

Success Metrics
[Need to define what successful engagement looks like at launch and how we'll monitor over time]
Additional Considerations

Research examples from other developer focused badges
Ensure that badges are challenging yet achievable ... and provide developers with a sense of accomplishment/bragging rights
Consider various motivations (e.g. certainty, autonomy, meaning, profess, social inclusion)

I suggest implement badges fully on indexer side, a more detailed pros and cons is required in this research but the initial thought it would be much more extensible and parallel to the work on indexer side

Hey @ailisp! I would like to know more about this. Can we plan in a call?


yt=ft (ht) what is ft

I am very interested in your article. What do you mean by ft and gt when you prove time-variance, such as yt=ft (ht)?

Thank you for your interest. In the paper, ft describes the output layer and gt describes a hidden block in ForecastNet. The published paper may provide a better description for this. Here is a link
https://link.springer.com/chapter/10.1007/978-3-030-63836-8_48
(I have also updated the README file with this link)

Thank you for your interest. In the paper, ft describes the output layer and gt describes a hidden block in ForecastNet. The published paper may provide a better description for this. Here is a link
https://link.springer.com/chapter/10.1007/978-3-030-63836-8_48
(I have also updated the README file with this link)


DNMY: move mps writing to MPSWriter.jl

[ ] Register MPSWriter as an official package
[ ] Add MPSWriter to REQUIRE
[ ] Remove Pkg.clone links in travis and appveyor

Changes Unknown when pulling 06340f66e9862a1feb2080e14344e68c936a2259 on MPSWriter into ** on master**.

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

No coverage report found for master at 00a31b3.

Powered by Codecov. Last updated by [00a31b3...b168a75][cc-compare]
[cc-base-branch]: https://codecov.io/gh/odow/NEOS.jl/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/odow/NEOS.jl/compare/00a31b3c49acae0eeb74965409e30ef01109a51c...b168a75f31203183d20af4b5522a018dcc6eba1c?src=pr
[cc-pull]: https://codecov.io/gh/odow/NEOS.jl/pull/8?src=pr


Sourcing nodes fails when content contains protocol-relative image URLs (img src with double-slash)

I'm seeing the following error when sourcing my WordPress content:
 ERROR

 gatsby-source-wordpress  failed to process https://www.vietnamcoracle.com//pix6.agoda.net/hotelImages/636/6365854/6365854_19010317240071053512.jpg?s=480x360
MaxRedirectsError: Redirected 10 times. Aborting.

This is coming from a post that contains the following HTML:
<img src="//pix6.agoda.net/hotelImages/636/6365854/6365854_19010317240071053512.jpg?s=480x360" />

A rudimentary search of the source code seems to imply the cause may be here: https://github.com/gatsbyjs/gatsby-source-wordpress-experimental/blob/master/src/steps/source-nodes/create-nodes/process-node.js#L351
This check assumes that any img tag containing src="/ must be a self-hosted asset. Obviously this doesn't account for protocol relative URLs.

Hi @benknight , sorry for the delayed response here - protocol relative url's are an anti-pattern https://www.paulirish.com/2010/the-protocol-relative-url/
Since the images still show up (just not as Gatsby images) we won't be implemented logic to handle this - I recommend instead using https:// url's everywhere. Since it's an anti-pattern and potentially insecure - it isn't worth adding the additional complexity to handle that case in this plugin. If you disagree please open a feature request on our new GatsbyJS portal https://portal.gatsbyjs.com/gatsby-cloud
Thank you!

Yes I understand however because WordPress is a CMS that generates HTML from WYSIWYG as well as third-party scripts (in the example I posted it's a widget provided by Agoda.com), its not always within developer's control. I suppose a plugin could do search/replace on protocol relative URLs. Anyway we've since migrated to Next.js so not concerned about this issue anymore. Thanks anyway for the followup.


Remove --socket and clarify --socket-dir example

Fixes #6936.
Fixes #7639.

This change is

@knz, there was already an cockroach sql example for --socket-dir in 20.1, so I'm not sure what's different in 20.2.

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/4aa19df7cddad50698bc264e52894c9d0ea03d44/
Edited pages:

v20.2/cockroach-sql.md
v20.2/cockroach-start-single-node.md
v20.2/connection-parameters.md


Add "TotalPages" into the exported components

Add "TotalPages" component into the components export

[!IMPORTANT]
Add TotalPages to the export list in index.tsx.

Exports:

Add TotalPages to the export list in index.tsx for external use.

This description was created by  for 89c639bdca8140dbb3491aba245ac4f8b93a4cc0. It will automatically update as commits are pushed.

Hi !
I'm trying to use your viewer into a project. And looking at your source I found this TotalPages component was missing.

Thank you very much for your contribution! Just merged :)


Generated audio has no voice

Any idea why the generated audio has no voice. I trained the model with 10K utterances and training completed successfully also. I am attaching sample generated audio. Kindly give insight what wrong could have gone?
audio.zip

Same issue here. I trained the model on GPU and synthesized files but these files have no voice.

It would be much easier if you provided the log from training/synthesis to give any opinion on what may be wrong.

Got the same problem. Did anyone find a solution for it?

Please attach your training and/or synthesis log to allow for anybody to help


Add Sort

4. Feature: Sort by Tag Values
Purpose
Allows users to sort their contact list based on specific tag values.
Command format
sort t/TAG
Example commands

sort t/client

Parameters
Tag

Acceptable values: Tag key must exist
Error handling: If the tag does not exist, display: "Tag not found: Please provide an existing tag."

Outputs

Success: "Contacts sorted by [tag] tag."
Failure: "Tag not found: [tag]."

Still WIP due to changes in fundamental JSON mapping changes for Tag class. Moving this to milestone v1.3 for now.

WIP due to breaking changes in underlying JSON mapping for Tag class. Moving to milestone v1.3 for now


Mobx instead redux

Hello, and thank you for this awesome boilerplate.
Is this possible to exchange redux state management for Mobx?
If yes, how i can do it?

Out of curiosity, what would make you want to use MobX? Have you tried porting?

Ive used redux before, and didnt like a tonns of boilerplate that i have to write,
prefer the Mobx way.

Using Mobx would not allow us to use React Offline (#20)

I wholeheartedly agree Redux brings with it a lion's share of boilerplate, lifting the barriers to entry for many would-be coders and spurring minimalist Flux implementations like Exim. There is something to be said, however, for maintaining a course until a clear and favorable alternative emerges, and I do not believe MobX to be it.

jhabdas, im not sure, that here is the right place to discuss , however there is nothing Redux can do, that Mobx cant. And i don`t understand, why i have to write so many bolilerplate, if i can achieve same result with another tool?

@psycura I have not worked with Mobx before, but you can create a PR and we can test the application with Mobx, I can assist you in PR and where to make changes.
Also we can use both Mobx and Redux, Redux is mainly for Core code, any other module can be installed and used, there is no hard dependency on Redux.
Would love to study the differences and provide proper support. Right now busy with documentation and examples so that users can understand full power of boilerplate.
Thanks guys.

@psycua. Closing issue due to inactivity. Please re-open the issue of you are willing to continue to Mobx


dresden codak does not work despite being on the list

http://dresdencodak.com/2012/11/03/dark-science-22/

It should work now.


What is the scope of the marks?

[Enter feedback here]
What is the scope of the marks that are made?
Does each Record instance only see the marks that it made itself, or does it see marks made by other instances?
Are the marks persistent like an extra yes/no column in the table? Or are they e.g. local to a certain user of to... whatever?
It's easy to write a test to find it out, but it's less work if the docs tell me. Knowing the scope makes possible the decision when to use this feature.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 5c5dbc52-9925-14e5-c9f3-5fafa3ccdec3
Version Independent ID: 632decfa-ced4-9e06-2b80-ee1b645d5437
Content: Record.Mark([Boolean]) Method - Business Central
Content Source: dev-itpro/developer/methods-auto/record/record-mark-method.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Thanks for your feedback. With the next doc refresh the topic will be updated with some examples and a bit more explanation: The mark on a record is temporary, and disappears when the current session ends. The Mark method is valid for the specific record instance/variable, and not shared between all variables of the same record type in a session.
Thanks!


feat: drop items when cutting down the tree

When a Sprite is removed, add the dropped item to the map and show the sprite
There are a few conflicts with #31, but I'll fix those before merging!

I noticed when I try to pick up the rope, it goes into a loop of trying to try to store it

Yeah, that warning is intentional. The log is from the incomplete Task 2


Cleanup the "extended load test" overrides

The extended load tests has been implemented and enabled everywhere as a part of https://github.com/kubernetes/perf-tests/issues/704.
We've been running these tests in many places (also at 5k node scales) for a few months. I believe we have enough confidence to make all the changes the defaults and cleanup all the gating logic we have.
We should get rid of all these overrides: https://github.com/kubernetes/perf-tests/tree/master/clusterloader2/testing/load/experimental/overrides
And clean-up all places where they are used (e.g. 10+ occurrences in k8s.io/test-infra)

/good-first-issue

To clarify - the things to do are:

update default for things that are enabled everywhere (PVs are still disabled in couple tests)
then remove the overwrides
cleanup ability to override those things

Hi @wojtek-t  I would like to work on this issue.
I found 126 occurrences of "testing/load/experimental/override" in 7 files within  k8s.io/test-infra. All these occurrences use --test-cmd-args=--testoverrides=./testing/load/experimental/overrides/ with one of the files listed in https://github.com/kubernetes/perf-tests/tree/master/clusterloader2/testing/load/experimental/overrides .
I wanted a bit more clarification regarding changes and tests I can use to verify the changes. So as apart of PR to resolve this issue should I

Remove the --test-cmd-args=--testoverrides=./testing/load/experimental/overrides/ switches from k8s.io/test-infra ?
Remove the override files ?
Modify the ./clusterloader2/pkg/config/template_provider.go to cleanup ability to  override
?
Would it be possible to share more details related to "update default for things that are enabled everywhere"

Thanks,
Mani

Hey @maniSbindra, thanks for taking a look at this!
To give you an example of what "update defaults" means, let's take a look at the
https://github.com/kubernetes/perf-tests/blob/master/clusterloader2/testing/load/experimental/overrides/enable_configmaps.yaml
It overrides the ENABLE_CONFIGMAPS variable to true.
We'd like to change the default of ENABLE_CONFIGMAPS to true so the override is not needed anymore. To do that, you should find all places where this variable is defined and change the default to true, e.g. this one https://github.com/kubernetes/perf-tests/blob/d8184730fffaa304cc0ee5ef7125740ff26d9cdd/clusterloader2/testing/load/config.yaml#L20
Once you do that, the override will be no-op and it will be safe to delete it (first delete usage in test-infra, then delete the override file in perf-test).

Thank you @mm4tt

Hi @mm4tt I believe this issue can be closed now.
Thanks,
Mani

There is one more thing - for stuff that we don't disable anywhere (e.g. secrets), we should also cleanup the config file:
https://github.com/kubernetes/perf-tests/blob/master/clusterloader2/testing/load/config.yaml
[The simpler it will be, the better]

Hi just wanted to confirm my understanding. Currently only places (which I could find) in test-infra where we use not use the default values of these override variables are where we set CL2_ENABLE_PVS=false in sig-scalability-presubmit-jobs.yaml, and where we set CL2_ENABLE_NETWORKPOLICIES=true in sig-scalability-experimental-periodic-jobs.yaml. Does this mean that for the remaining 5 override variables (ENABLE_CONFIGMAPS, ENABLE_DAEMONSETS, ENABLE_JOBS, ENABLE_SECRETS, ENABLE_STATEFULSETS) we cleanup the config file mentioned?

Yes - that's exactly what I'm suggesting.

Hi @wojtek-t  for this issue to be closed, do I need to submit a PR deleting the file experimental-config.yaml?

Yes - it's out-of-date anyway. So let's please remove it and close this issue then.

Hi @wojtek-t ,
Can this issue be closed now that #1183 has been merged.
Thanks,
Mani

Yes - we can close it. Thanks!


[WIP] [Docs] add explicit info about customization of homepage

close https://github.com/go-gitea/gitea/issues/8693

look of draft1:

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@70fa80d). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #8694   +/-   ##
=========================================
  Coverage          ?   41.69%
=========================================
  Files             ?      531
  Lines             ?    68723
  Branches          ?        0
=========================================
  Hits              ?    28655
  Misses            ?    36395
  Partials          ?     3673

Continue to review full report at Codecov.

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

@lafriks can you look throu?


API for Configure Monitoring not functioning as expected.

Environment
Keptn 0.8.1

Client OS (e.g., Linux, macOS, Windows):
Keptn Version (keptn version): 0.8.1
Kubernetes Cloud Provider (e.g., GKE, AKS): microk8
Kubernetes version (kubectl version): 1.19

istio 1.9.1
Affected Component

[ ] Docs
[ ] CLI
[ ] Kubernetes Integration
[ ] Openshift Integration
[ ] Helm
[ ] Istio
[ ] Bridge
[ ] Approval
[ ] Datastore
[X] REST API
[ ] eventbroker / distributor
[ ] jmeter

Describe the bug

Created project in Keptn.
Call API to configure Dyntrace monitoring.
Then run a quality gate check.

This message is presented.
Status: no evaluation performed by lighthouse because no SLI-provider configured for project qualitygate
Then I tried to use the swagger interface to add the Dyntrace monitoring.
Calling the /v1/event API with the following payload.
{
 "data": {
  "project": "qualitygate",
  "configureMonitoring": {
     "type": "dynatrace"
   }
},
"datacontenttype": "application/json",
 "source": "Jenkins",
 "specversion": "1.0",
 "type": "sh.keptn.event.configure-monitoring.triggered"
}

This does return a keptnContext.

So, one would think it worked.
However, if you run the test again the same message appears.

Next, I used the keptn CLI.
Running this command,
keptn configure monitoring dynatrace --project=qualitygate
You get:
ID of Keptn context: ee0c05a2-25e9-48a8-bb76-10a8926dfcf1
Dynatrace monitoring setup done.
The following entities have been configured:

---Management Zones:---
  - Keptn: qualitygate: Created successfully
  - Keptn: qualitygate hardening: Created successfully

---Automatic Tagging Rules:---
  - keptn_service: Created successfully
  - keptn_stage: Created successfully
  - keptn_project: Created successfully
  - keptn_deployment: Created successfully

---Problem Notification:---
  - Successfully set up Keptn Alerting Profile and Problem Notifications

---Dashboard:---
  - Dynatrace dashboard created successfully. You can view it here: https://ibg73613.live.dynatrace.com/#dashboards

Now, if you run the test, everything works as expected.
Expected behavior
Expectation is for the API call to function the same as the CLI.
This call to the API should configure the Dynatrace monitoring.

Additional context
Please inform me if the API endpoint is incorrect. Everything I checked, indicates it is correct.
Also note, this causes an issue for the Jenkins Library as it uses the API to configure the Dynatrace Monitoring.
As of now, the Jenkins Library would appear broken to most users.

Hi @pcjeffmac,
can you please set the property: type: sh.keptn.event.monitoring.configure
Please let me know if it worked then.

It is closer, but the lighthouse service does not get the SLI provider,
I see this in the bridge,

This is what shows up in the lighthouse log,
{"timestamp":"2021-03-29T14:03:58.598516548Z","logLevel":"DEBUG","message":"SLI provider for project qualitygate is: "}
{"timestamp":"2021-03-29T14:03:58.598653194Z","logLevel":"DEBUG","message":"Send event: sh.keptn.event.get-sli.triggered"}
{"timestamp":"2021-03-29T14:05:29.500883059Z","logLevel":"DEBUG","message":"Received event: sh.keptn.event.monitoring.configure"}
{"timestamp":"2021-03-29T14:09:05.904068101Z","logLevel":"DEBUG","message":"Received event: sh.keptn.event.evaluation.triggered"}
{"timestamp":"2021-03-29T14:09:05.904271178Z","logLevel":"DEBUG","message":"Send event: sh.keptn.event.evaluation.started"}

You can see by the log, the SLI provider is not set.
So, I again used the CLI,
keptn configure monitoring dynatrace --project=qualitygate
Now it works, with this in the lighthouse log,
{"timestamp":"2021-03-29T14:09:06.14927593Z","logLevel":"DEBUG","message":"SLI provider for project qualitygate is: dynatrace"}
{"timestamp":"2021-03-29T14:09:06.149425787Z","logLevel":"DEBUG","message":"Send event: sh.keptn.event.get-sli.triggered"}
{"timestamp":"2021-03-29T14:09:09.416196622Z","logLevel":"DEBUG","message":"Received event: sh.keptn.event.get-sli.finished"}
{"timestamp":"2021-03-29T14:09:09.426890456Z","logLevel":"DEBUG","message":"Start to evaluate SLIs"}
{"timestamp":"2021-03-29T14:09:09.721868406Z","logLevel":"DEBUG","message":"Evaluation result: pass"}

You can now see that the SLI provider is set to Dynatrace.
This is the output in the bridege,

@pcjeffmac can you please change the payload for your API call to how it is defined here: https://github.com/keptn/spec/blob/0.1.7/cloudevents.md#configure-monitoring
(just remove the configureMonitoring property and add service).
I think there happened a mistake with the auto-generation of the keptn/spec, since the implementation still uses the old format which is version 0.1.7.

Looks like this payload works,
{
"data": {
"project": "qualitygate-simpletest",
"service":  "eval",
"type": "dynatrace"
},
"datacontenttype": "application/json",
"source": "Jenkins",
"specversion": "1.0",
"type": "sh.keptn.event.monitoring.configure"
}
Should I change the "specversion"?
Thanks,

No, you do not have to change the specversion.

Task:

[ ] Update of the keptn/spec to use the old format of sh.keptn.event.monitoring.configure
[ ] Mark this event as deprecated
[ ] File an issue that proposes to solve the "problem" of the misused event.


Feature Extraction Problem in RTX 3090

I am trying to extract feature_type = i3d in RTX 3090. But it freezes the terminal after printing this:
feature_type: i3d
stack_size: 12
step_size: 2
streams: rgb
flow_type: pwc
extraction_fps: null
device: cuda:0
on_extraction: print
output_path: ./output/i3d
tmp_path: ./tmp/i3d
keep_tmp_files: false
show_pred: false
config: null
video_paths: null
file_with_video_paths: ./sample/sample_video_paths.txt
Device: cuda:0
and doesn't do anything after this.

duplicate of #13


[Bug] [Forge 1.20.1] Multiplayer Syncing Problem on Player Join

When attempting to play Multiplayer/LAN with the accessories mod, only one player can access the accessory GUI at a time. The game will also crash when another player tries to access the accessory GUI screen. The crash is the same for both cases as it seems to be a client side crash.
Crash log: https://pastebin.com/av3zfkq1
In the case of a multiplayer server, only the player who joined the server first can access the accessory GUI, anyone who joins after can press the inventory button, but nothing will happen. If the first player disconnects and reconnects the second player can now open the GUI, but when the first player presses the inventory button, they will crash, and subsequently anyone who joins after, can't open the GUI again.
Also trying to use the "H" keybind to open the GUI does not work either and will also crash the game.
Server video demonstration: https://streamable.com/njru9f
For LAN hosted single player world its pretty much the same but no crash since the host cant disconnect and reconnect. Only the LAN host can access the accessory GUI and not any other players.
Singleplayer LAN video demonstration: https://streamable.com/l45uwa
Note: I have connector present, and I'm using regular Forge, but I have done a lot of troubleshooting, with and without connector installed, using NeoForge, using different curios mods and trinkets mods. I also tested on a clean profile with just accessories and cloth config installed, and it all ends up in the same bug/crash.
Forge Version: 47.3.3
Minecraft Version: 1.20.1
Accessory mods used:
accessories-1.0.0-alpha.24+1.20-neoforge
accessories_cclayer-7.4.0-alpha.13+1.20
accessories_tclayer-3.8.1-alpha.12+1.20

Wow this is a odd bug, will investigate such.

Testing within the development for 1.21 without compat layers, I can't reproduce the issue. Will attempt such within 1.21 and then test with both compat layers.

Seems to be involved with the OnDatapackSyncEvent on forge not firing leading to a problem where the client does not get the data required i.e. slot info and such.

Ok tracked it down to a bug within the code targeting the wrong entity when trying to reopen the screens for anyone who has AccessoriesScreen open when running dataSync

Further checking on the CCLayer seems to still have the bug, most investigating!

Pushed a fix for such to modrinth and curseforge

Just tested the new version. The issue seems fixed from my testing. And the visual bug from #43 is fixed as well, the accessories now appear, and the game no longer crashes. Thank you!
I also tested the keepInventory bug from #40 to see if it was related, and it still drops it seems. Interestingly though, that bug doesn't happen on the Alpha 23 version.

Will look into the keep inventory bug coming up but closing this as fixed


feat/creating the connection modal with github

Brief Information
The type of this pull request is:

[ ] bug fixing
[x] new feature
[ ] others

What does this PR do?
creating a modal to ask the user to connect to github if they are not already connected.

when clicking on my tasks button

when clicking the apply button

I am archiving this PR


Fixed bug, resolved #19 #20 #16

@swilliams Can you merge this in? It's a bug fix mostly...

@isiegel @iortz-ethology Can you guys merge this? The fate of all designers using sketch depends on it!
Kidding... but please do merge this in when you can ;)

Looks like I don't have write access to this repo.
This branch has no conflicts with the base branch
Only those with write access to this repository can merge pull requests.

@isiegel That's super lame! I don't have access anymore to merge anything. So... Whenever @swilliams is around, please let him know this is a a simple merge... Actually, if he gives me write access to this repo (since I'm trying to actively maintain it), that would be better :)

maybe @paulvantuyl can merge it in? (Getting desperate here hahaha)

You sure the fate of all designers using Sketch depends on it wasn't a joke? :)

Merged and I also added @seanriceaz as a contributor – hope you don’t care @swilliams. This is his baby, even if it’s Tallwave’s IP.


[CHALLENGE SUBMISSION]

Please provide the following information for your submission:
What is your name? (First, Last)
Kalyan Sista
Where is your GitHub Repo ?(With your challenge solution)
GitHub Repo                        : https://github.com/kalyanrajsista/AzureEats-Website
Public URL for the website   : https://terraform-lz-challenge1-poc.azurewebsites.net/
What Challenge is this for? (Challenge number between 1 and 5)
Challenge 2
(Optional) Anything else we should know? e.g., language used, location, blog post?
Location: Ottawa, Canada

Thanks for participating in the #TerraformOnAzure coding challenges!
We have attached your very special Astronaut Badger background, for you to use in all your upcoming video calls!
And to recognize you for your learning, you can collect your digital Azure Heroes badge on your mobile (you may need to install the Enjin wallet to collect it). Make sure to collect it today!
https://beam.enjin.io/claim/d32599695c8c28cfc24820492080fac0
Keep on learning and sharing!
Greetings from the #TerraformOnAzure team,
Suzanne, Floor, Erik & Nick


Connection pool having performance issue on managed cluster

Environment
HikariCP version: 2.6.1
JDK version: 1.8.0_112-b15
Database: Apache Derby <IP_ADDRESS>
Driver version: Apache Derby Client  <IP_ADDRESS>

We are running a web service in docker container on a managed cluster -- DC/OS 1.8, but seeing some weird performance issues:
2017-03-03T19:31:30,812Z DEBUG [HikariPool-1 connection closer] com.zaxxer.hikari.pool.PoolBase [cid:] HikariPool-1 - Closing connection org.apache.derby.client.net.NetConnection@33425171: (connection is evicted or dead)

2017-03-03T19:31:31,837Z DEBUG [HikariPool-1 connection adder] com.zaxxer.hikari.pool.HikariPool [cid:] HikariPool-1 - Added connection org.apache.derby.client.net.NetConnection@1c3c3711

2017-03-03T19:31:31,837Z DEBUG [HikariPool-1 connection adder] com.zaxxer.hikari.pool.HikariPool [cid:] HikariPool-1 - After adding stats (total=10, active=1, idle=9, waiting=0)

2017-03-03T19:31:35,817Z DEBUG [HikariPool-1 connection closer] com.zaxxer.hikari.pool.PoolBase [cid:] HikariPool-1 - Closing connection org.apache.derby.client.net.NetConnection@22ab3e78: (connection is evicted or dead)

2017-03-03T19:31:35,832Z DEBUG [HikariPool-1 connection adder] com.zaxxer.hikari.pool.HikariPool [cid:] HikariPool-1 - Added connection org.apache.derby.client.net.NetConnection@12fbcae6

2017-03-03T19:31:35,832Z DEBUG [HikariPool-1 connection adder] com.zaxxer.hikari.pool.HikariPool [cid:] HikariPool-1 - After adding stats (total=10, active=1, idle=9, waiting=0)

2017-03-03T19:31:40,823Z DEBUG [HikariPool-1 connection closer] com.zaxxer.hikari.pool.PoolBase [cid:] HikariPool-1 - Closing connection org.apache.derby.client.net.NetConnection@2c6cc45d: (connection is evicted or dead)

2017-03-03T19:31:41,845Z DEBUG [HikariPool-1 connection adder] com.zaxxer.hikari.pool.HikariPool [cid:] HikariPool-1 - Added connection org.apache.derby.client.net.NetConnection@16c45c61

2017-03-03T19:31:41,845Z DEBUG [HikariPool-1 connection adder] com.zaxxer.hikari.pool.HikariPool [cid:] HikariPool-1 - After adding stats (total=10, active=1, idle=9, waiting=0)

2017-03-03T19:31:45,828Z DEBUG [HikariPool-1 connection closer] com.zaxxer.hikari.pool.PoolBase [cid:] HikariPool-1 - Closing connection org.apache.derby.client.net.NetConnection@28535508: (connection is evicted or dead)

2017-03-03T19:31:45,844Z DEBUG [HikariPool-1 connection adder] com.zaxxer.hikari.pool.HikariPool [cid:] HikariPool-1 - Added connection org.apache.derby.client.net.NetConnection@53c6c0dd

2017-03-03T19:31:45,844Z DEBUG [HikariPool-1 connection adder] com.zaxxer.hikari.pool.HikariPool [cid:] HikariPool-1 - After adding stats (total=10, active=1, idle=9, waiting=0)

2017-03-03T19:31:48,805Z DEBUG [HikariPool-1 housekeeper] com.zaxxer.hikari.pool.HikariPool [cid:] HikariPool-1 - Pool stats (total=10, active=1, idle=9, waiting=0)

2017-03-03T19:31:50,833Z DEBUG [HikariPool-1 connection closer] com.zaxxer.hikari.pool.PoolBase [cid:] HikariPool-1 - Closing connection org.apache.derby.client.net.NetConnection@6d51ce66: (connection is evicted or dead)

2017-03-03T19:31:51,857Z DEBUG [HikariPool-1 connection adder] com.zaxxer.hikari.pool.HikariPool [cid:] HikariPool-1 - Added connection org.apache.derby.client.net.NetConnection@2ebfe10b

2017-03-03T19:31:51,857Z DEBUG [HikariPool-1 connection adder] com.zaxxer.hikari.pool.HikariPool [cid:] HikariPool-1 - After adding stats (total=10, active=1, idle=9, waiting=0)

<IP_ADDRESS> - - [2017-03-03T19:31:55,681Z] "GET /favicon.ico HTTP/1.1" 200 15086 "http://{serviceURI}/" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_5) AppleWebKit/537.36 (KHTML, like Gecko) Safari/537.36" "host" "-" 0/18 0 0 "8fc1137b-b88b-44a7-a04c-76235582d295"

2017-03-03T19:31:55,837Z DEBUG [dw-37 - GET /{serviceURI}/] com.zaxxer.hikari.pool.HikariPool [cid:da27c036-fb4e-458e-9c9f-56f194db76ac] HikariPool-1 - Timeout failure stats (total=9, active=0, idle=9, waiting=0)

2017-03-03T19:31:55,837Z DEBUG [HikariPool-1 connection closer] com.zaxxer.hikari.pool.PoolBase [cid:] HikariPool-1 - Closing connection org.apache.derby.client.net.NetConnection@1672754a: (connection is evicted or dead)

2017-03-03T19:31:55,842Z ERROR [dw-37 - GET /{serviceURI}/] io.dropwizard.jersey.errors.LoggingExceptionMapper [cid:da27c036-fb4e-458e-9c9f-56f194db76ac] Error handling a request: c5e0aac9460eba45
com.cerner.system.jdbc.JDBCException: java.sql.SQLTransientConnectionException: HikariPool-1 - Connection is not available, request timed out after 30035ms

From the log I suspect this would be the observed behavior:
The pool was primed to default maximum size of 10 at service startup, then during a call, the service asks for connection from the pool, for which the pool goes through its idle connections to hand one out, in this case all of these idle connections in the pool are dead or evicted, which I still don't know why. What is causing the performance issue is there seems to be a 3~5 seconds of interval between refreshing of stale connections, demonstrated in this log since we have 10 stale ones to refresh it eventually cause a time out.
We are not seeing this in a local MacOS X setup running the same docker container connecting to the same derby DB instance. On managed cluster, there could be some shared CPU resource limitation tied to closing or re-adding connections.
Anyway this service using HikariCP is not a heavy load service and we managed to work around this by not using a fixed pool size since we are not expecting constant spikes. However, this could be an issue where other critical service is running on this managed cluster, so I would like to hear more from your opinion of what might be a cause of this?
Thanks!

I suspect it is a Docker specific issue.  My company uses HikariCP along with Derby and has never encountered this problem.
If there are no other exceptions in the log indicating 08xxx SQLState errors, it is likely that the connection is being interrupted by the server or the Docker interconnect layer.
You could try to wireshark/tcpdump the traffic to see which side is responsible. A short term fix would be to set a maxLifetime that is fairly short, for example 3-5 minutes.


Add volumes for /run/xtables.lock

To prevent race conditions when multiple applications want to create/modify iptables rules.
Calico and Cilium already have these volumes.
This PR adds this volume also to flannel and kube-proxy.

Pull request for flannel: https://github.com/coreos/flannel/pull/1229

Thanks. Rebased, squashed, and tweaked some as 016d4ebd0c39c487d7eb7858aee7221dd5ef2744
I've still never observed an issue from lack of this, but its worth having.


Adapt kvrocks with ToplingDB

ToplingDB is compatible with RocksDB and provide higher performance, WebView, native support to prometheus, secondary instance on shared storage ...
ToplingDB distributed compaction can not be supported since kvrocks CompactionFilter need to access DB and calling DB::Get.
1. Build with ToplingDB
By default ToplingDB is not enabled, to compile kvrocks with ToplingDB, following instructions below:
1.1 Build ToplingDB
# ubuntu
sudo apt-get install -y git libjemalloc-dev libaio-dev libgflags-dev zlib1g-dev libbz2-dev libcurl4-gnutls-dev liburing-dev
# centos
sudo yum -y install git libaio-devel gcc-c++ gflags-devel zlib-devel bzip2-devel libcurl-devel liburing-devel

git clone https://github.com/topling/toplingdb
cd toplingdb
make -j`nproc` DEBUG_LEVEL=0 PREFIX=/path/to/toplingdb/install/dir install

1.2 Build kvrocks with ToplingDB
# build with ToplingDB, you should build and install ToplingDB first
export LD_LIBRARY_PATH=/path/to/toplingdb/install/dir/lib:${LD_LIBRARY_PATH}
env TOPLINGDB_HOME=/path/to/toplingdb-repo TOPLINGDB_LIBDIR=/path/to/toplingdb/install/dir/lib \
    ./x.py build -j`nproc` -DCMAKE_BUILD_TYPE=Release

# copy html files to kvrocks's ToplingDB document root, this makes webview pretty
cp /path/to/toplingdb-repo/sideplugin/rockside/src/topling/web/{index.html,style.css} \
   /path/to/kvrocks/toplingdb/document-root

2. Run with ToplingDB
env var TOPLING_CONF specifies ToplingDB json/yaml config which instructs kvrocks to run with ToplingDB.
If env var TOPLING_CONF is not specified, kvrocks will run without ToplingDB sideplugin and WebView...
User should change datadir in json config.
$ export LD_LIBRARY_PATH=/path/to/toplingdb/install/dir/lib:${LD_LIBRARY_PATH}
$ env TOPLING_CONF=kvtopling-community.json ./build/kvrocks -c kvrocks.conf

3. Run with ToplingDB shared storage secondary instance
Secondary instance on shared storage sharing same files of primary instance and keep catch up with primary.
$ env TOPLING_CONF=kvtopling-community-2nd.json ./build/kvrocks -c kvrocks.conf

Closing...
See https://github.com/apache/incubator-kvrocks/pull/1206#pullrequestreview-1229819085.

Benchmark with wikipedia data: 109GB, 38.5 M rows, average 2.8 KB per row.
Hardware: aliyun 2c16g, in random read, we also tested with 8c16g, disk is high performance ESSD(iops up to 60k).
community conf kvtopling-community.json using BlockBasedTable(block size is 8K).
enterprise conf kvtopling-enterprise.json using ToplingZipTable.

Random read after full compaction, in random read, it is CPU bounded, rand 8c means 8c16g.
block cache is 1G for BlockBasedTable, we have tried block cache 4G & 8G, but performance getting worse.

Topling's LICENSE is in Category X from the ASF License Policy. So we cannot integrate it within the Apache release.
That said, you can still provide the integration within topling's org and make necessary interface refactoring in the upstream (support pluggable storage backend, for example), and even send a blog to show the integration is supported by third-party at https://kvrocks.apache.org/blog.

Now ToplingDB license has updated to standard Apache 2.0: license

Hi @rockeet
Great thanks for your benchmark data and impressive work on the toplingdb. I have some questions about the benchmark data:
For chart 1, does the build speed mean the rocksdb SST building key-value speed?
For chart 2, how can we explain the rand key per second? and how it's related to the QPS.

Thanks for your clarification.

https://github.com/wacfork/incubator-kvrocks/tree/unstable/docker-compose#readme
kvrocks + rocksdb / topling 社区版 性能测试对比

For chart 3, it is 16 threads(16 client connections) in which each thread use mget with 64 keys per request.

hi , for chart 3,any idea why when blockcache was 4G & 8G, but performance will get worse.

For chart 3, it is 16 threads(16 client connections) in which each thread use mget with 64 keys per request.

hi , for chart 3,any idea why when blockcache was 4G & 8G, but performance will get worse.

I think it should because larger block cache evict OS page cache memory, thus page cache hit rate becomming lower, and OS page cache data is compressed, which can hold more data in same memory.

Now distributed compaction is supportted for metadata, which support strings data type and main key for other data types.
distributed compaction for subkey is not supportted, which needs DB::Get, thus can not be efficiently supportted.


Configurable JUnit test case separator

Current behavior
JUnit reporter (and perhaps other reporters) generate test cases such as [Feature] [Scenario] (i.e. with a single space separator) which makes it hard to distinguish both parts e.g. Another feature Runs first scenario:
Not shure if it's related to this library or Cypress itself.
<testsuite name="Another feature" timestamp="2020-07-31T16:49:37" tests="2" time="26.7660" failures="0">
    <testcase name="Another feature Runs first scenario" time="13.0000" classname="Runs first scenario">
    </testcase>
    <testcase name="Another feature Runs second scenario" time="13.7660" classname="Runs second scenario">
    </testcase>
</testsuite>

Desired behavior
A configuration option would be great.
I am thinking of something like:
"cypress-cucumber-preprocessor": {
  "testCaseTitleSeparatedBy": " - "
}

And the report would be:
<testsuite name="Another feature" timestamp="2020-07-31T16:49:37" tests="2" time="26.7660" failures="0">
    <testcase name="Another feature - Runs first scenario" time="13.0000" classname="Runs first scenario">
    </testcase>
    <testcase name="Another feature - Runs second scenario" time="13.7660" classname="Runs second scenario">
    </testcase>
</testsuite>

Test code to reproduce

Versions

Cypress version: 4.11.0
Preprocessor version: ^2.5.4
Node version: 12.14.1

As luck would have it, I've just been reading the JUnit Reporter docs whilst looking for something else, and I think I saw what you need. The option suiteTitleSeparatedBy looks about right:
https://github.com/michaelleeallen/mocha-junit-reporter#full-configuration-options

Due to personal reasons, the previous maintainers of this package are stepping down and handing the reigns over to me, a long-time contributor to the project and a user of it myself. This is a responsibility I'm very excited about. Furthermore, I'd like to thank @lgandecki ++ for all the work that they've done so far.
Read more about the transfer of ownership here.
The repository has however moved and all outstanding issues are being closed. This is not a reflection of the perceived importance of your reported issue. However, if after upgrading to the new version, you still find there to be an issue, feel free to open up another ticket or comment below. Please make sure to read CONTRIBUTING.md before doing so.


feat: allow creation of offchain proposals with weighted and quadratic voting type

Summary

Closes: #149
Closes: #151
Add support for weighted and quadratic voting on offchain spaces proposals
Both type are merged into a single PR, as they're 100% identical from the UI PoV.
New features

Add weighted and quadratic voting type when creating an offchain proposal
Add support for voting on weighted and quadratic proposals (for offchain spaces)

How to test
Proposal creation

Go to an offchain spaces
Create a proposal with weighted or quadratic voting type
It should create the proposal with the correct voting type (confirm by checking the pinned proposal metadata)

Voting

Vote on the each of the new proposals you just created
It should register the vote
Refresh the page to see the vote results: you should see the same results as when viewing the same proposal on snapshot v1

TODO

[ ] Waiting for new UI, currently using same UI as v1

UI part is still pending, but the vote logic can already by reviewed

Test with http://localhost:8080/#/s:pistachiodao.eth/proposal/0x21f64875abbca71762a980efae43ab62b546d54f19a208d0e61a5d7cee571a35
UI is temporary, ad borrowed from snapshot v1:
Vote UI:

Need more space on the x axis, or a different design

Votes results:

Need some improvement on the tooltip to increase legibility (maybe each choice on its own line, with a progress bar in each choice background)

@bonustrack maybe you have some ideas about temporary UI for this? Or maybe we should wait for final UI?

I have some idea, will do a Figma and send here

Here is my proposition, skip the % for now, and show input like this:

https://www.figma.com/proto/hV3qM6uaLDOo1lHWTfX56t/Untitled?type=design&node-id=1-243&t=eW5d44SnHhmS3n6b-1&scaling=min-zoom&page-id=0%3A1&mode=design

New UI implemented (with my take on the progress)


Add style to desktop version of portfolio website

Updated the images on the homepage and about me page.
Responsive functionality has been added to site to enable go view on mobile and desktop device

Rearranged the text beside the contact form
Applied font-family "inter" as in the figma design template
Rearranged the language, skill, and framework sections as recommended in the review


Fix issue with use of callback for async function

I see the follow error when I run the application - Serverless: Warning: handler 'hello' returned a promise and also use a callback! This is problematic and might cause issues in you lambda.
$ sls offline start
Serverless: Bundling with Webpack...

Serverless: Starting Offline: dev/us-east-1.

Serverless: Routes for hello:
Serverless: GET /hello

Serverless: Offline listening on http://localhost:3000

Serverless: GET /hello (λ: hello)
Serverless: The first request might take a few extra seconds
Serverless: [200] {"statusCode":200,"body":"{\"message\":\"Go Serverless v1.0! Your function executed successfully! (with a delay)\"}"}

Serverless: Warning: handler 'hello' returned a promise and also use a callback!
This is problematic and might cause issues in you lambda.

I fixed the error like described here - https://forum.serverless.com/t/how-to-use-node8-10-with-graphql-and-async-await/6459

Thank you! We made this change to the tutorial but didn't do the same here yet.


New version: PDQ.PDQDeploy version 19.3.626.0

Created by 🥟 Dumplings.
Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Chapter 7: player can move over monsters

Half way through Chapter 7 (dealing damage) the player can move over monsters!! This is because I think of the order we run systems in:

the player move
we run map_indexing
we run monster_ai
the player moves again

So when the player moves, map_indexing has not been refreshed with the new monster positions. I've solved it by changing the order in run_systems:
fn run_systems(&mut self) {
        let mut vis = VisibilitySystem{};
        vis.run_now(&self.ecs);
        let mut mob = MonsterAI{};
        mob.run_now(&self.ecs);
        let mut mapindex = MapIndexingSystem{};
        mapindex.run_now(&self.ecs);
        self.ecs.maintain();
    }

Originally posted by @Zireael07 in https://github.com/thebracket/rustrogueliketutorial/issues/28#issuecomment-552125591

Yeah, this fixes the issue - I just checked.


Implement setting bandwidth_limit on a servers network interface

Description

After the go-anxcloud release, we now implement the bandwidth_limit field on the server network interface.
This currently is rebased onto the dependabot branch to be able access the newly added field in go-anxcloud.
Checklist

[ ] added release notes to Unreleased section in CHANGELOG.md, if user facing change

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

The fact that the new autogenerated documentation is losing a lot of information of the previous iteration makes me disapprove this PR immediately. :/
Haven't checked the code itself yet, gonna do that in a sec. :eyes:
For now I'd prefer it if we somehow manually adjust the documentation and ensure that it matches the autogenerated version in a second PR.

I actually didn't give much thought to the docs generator lol Just said it will replace all files and hence I didnt look deeper into why they all changed.
I adjusted the docs accordingly and its all good now

Tried to test it manually, and the creation worked fine. However, a second invocation of terraform plan did not work.
To reproduce it:

Copy the example code from the official docs to a local .tf file somewhere.
Compile the binary, together with a version suffix.

I just patched the Makefile locally and ran make build, see below.

Point terraform to the override.
Run terraform apply. The server is built as intended and everything works, you can see them in the engine.
Run terraform plan again, after everything has been created.

I got the following output:
tf apply -auto-approve
╷
│ Warning: Provider development overrides are in effect
│
│ The following provider development overrides are set in the CLI configuration:
│  - anexia-it/anxcloud in /home/joster/repos/anexia-it/terraform-provider-anxcloud
│
│ The behavior may therefore not match any released version of the provider and applying changes may cause the state to become incompatible with published releases.
╵
data.anxcloud_core_location.anx04: Reading...
data.anxcloud_core_location.anx04: Read complete after 1s [id=52b5f6b2fd3a4a7eaaedf1a7c019e9ea]
anxcloud_vlan.example: Refreshing state... [id=c13d44bd070c4477a7cc9b1f182f445f]
anxcloud_network_prefix.v6: Refreshing state... [id=076ef87b174942df90085b20a5fae8d1]
anxcloud_network_prefix.v4: Refreshing state... [id=da9eaa5ca2db4ac6a10b54390a6e9082]
anxcloud_ip_address.v6: Refreshing state... [id=1873d26ab47a4adcaec023ece59f871e]
anxcloud_ip_address.v4: Refreshing state... [id=79f592ce58ac4d6894fcc40e4220224d]
anxcloud_virtual_server.example: Refreshing state... [id=e7a4f32fe52c484a8dc90678af06e3eb]

Planning failed. Terraform encountered an error while generating this plan.

╷
│ Error: Request cancelled
│
│ The plugin6.(*GRPCProvider).PlanResourceChange request was cancelled.
╵
error: Recipe `apply` failed on line 2 with exit code 1

addendum: makefile.patch
diff --git a/Makefile b/Makefile
index 8dace30908..ee4d0bdd79 100644
--- a/Makefile
+++ b/Makefile
@@ -5,8 +5,8 @@
 HOSTNAME=hashicorp.com
 NAMESPACE=anexia-it
 NAME=anxcloud
-BINARY=terraform-provider-${NAME}
-VERSION=0.3.1
+VERSION=0.7.0
+BINARY=terraform-provider-${NAME}_v${VERSION}
 OS_ARCH=linux_amd64
 GOLDFLAGS= -s -X github.com/anexia-it/terraform-provider-anxcloud.version=$(VERSION)

Everything should be addressed now


Configuration search ignores current dir

There is an error in the treewalking code as a config file in the current dir is never found.
A file in the oparent fiolder is correctly found.

Fixed per https://github.com/sthagen/laskea/commit/6e5c57050ca0e98c50b5c15956bb457a4780461d


Unexpected unknown command invocation 'class configure'

Hi!
I'm trying to follow the examples for classes from the documentation, but I can't get it to work. I'm getting syntax errors:
Unexpected unknown command invocation 'class configure blacklist {
    "blacklisted" "<IP_ADDRESS>"
  }'

Maybe you should add an "on" statement similar to the one below to your "it" block?

    it "your description" {
      ...
      on class configure blacklist {
    "blacklisted" "<IP_ADDRESS>"
  } return "your return value"
      ...
    }

I'm using jtcl 2.8.0 and testcl 1.0.8 on Java 1.8.0_60, and start jtcl like this: /opt/jdk/bin/java -DTCLLIBPATH=/jtcl/TesTcl -cp /jtcl/jtcl-irule.jar:/jtcl/jtcl-2.8.0.jar tcl.lang.Shell.
Some incompatibility with my versions, or error in the invocation?

@jamespic: From the commit log, it looks like you contributed this functionality - thanks for the effort! Do you have any insight in what could cause this? I tried running the test/test_class.tcl file included in the release, and that does seem to work. However, it seems to be bypassing a lot of the "normal" import structure and instead including parts explicitly with source src/classes.tcl and so on. I'm guessing that is not what I as an end user should do?

@carlpett It's been a while since I worked with the codebase, and my Tcl isn't that strong (I basically learned just enough to solve our problem, on the project where I used this), but my recollection is that we used explicit source src/classes.tcl in our code.
I could readily believe that that's not the Tcl way, so feel free to refactor it to be more idiomatic.

@jamespic Thanks for the quick reply! I'm basically in the same boat, no tcl knowledge outside the quite simple stuff in irules. I'll try and see if I can get my rules and tests working by adding source then.
@landro: If this is how it is meant to be done, I could try to update the examples?

@carlpett the way users should write tests, is by following the pattern found in the examples folder. The stuff found in tests is for testing the TesTcl itself (consider them unit-tests).
TesTcl heavily relies on the "unknown" command, which kicks in whenever an unknown/undefined proc is called, in this case the class command. I'll quickly check @jamespic's code to see if I can sort out what's going on. Stay tuned.

👍

@carlpett Could you please test the classes_fix branch and see if it works for you? I added an example that show how end users should use this functionality, see examples directory

@landro It works! Thanks a lot, great work!

fixed in 99ab81f78a45774fa6df4bd17d30a9051f7e3c16


[docs]: Increase size of CTA "Get Started" and Change Title and Subtitle to white

In https://next.material-ui.com, the CTA(call to action) is way too small(14px). Even though there is no firm font size, generally people use 16px as a minimum. I feel 14px is too less for the main call to action on the page. Although the contrast of the color of text and CTA follows the WCAG guidelines(4.61 contrast) but I don't think it's really eye-catching.

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Currently, in the homepage:

Expected Behavior 🤔
The color white would look much better, or at least the color which was used previously(the light blue color on the logo). Here's an image after changing only the color of the title and CTA to white and increasing size of CTA:

It looks much more eye catching according to me. Also, while hovering the CTA, we can change the border color and text color to the lighter color on the logo.
Context 🔦
I'd love to work on this but I'd like to know if this is something others also agree with. Also, could someone point me out to the file that contains the code for the hero section(the headline, CTA etc). I'm new to open source so any tips would be appreciated. Thanks a lot!

Your Environment 🌎

`npx @material-ui/envinfo`
  System:
    OS: macOS 11.5.2
  Binaries:
    Node: 15.11.0 - /usr/local/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 7.15.1 - /usr/local/bin/npm
  Browsers:
    Chrome: 92.0.4515.159
    Edge: 92.0.902.62
    Firefox: 87.0
    Safari: 14.1.2
  npmPackages:
    @material-ui/core: ^4.11.3 => 4.11.3
    @material-ui/icons: ^4.11.2 => 4.11.2
    @material-ui/lab: * => 4.0.0-alpha.57
    @material-ui/styles: ^4.11.3 => 4.11.3
    @material-ui/system:  4.11.3
    @material-ui/types:  5.1.0
    @material-ui/utils:  4.11.2
    @types/react: ^17.0.14 => 17.0.14
    react: ^16.14.0 => 16.14.0
    react-dom: ^16.14.0 => 16.14.0
    typescript: ^4.3.2 => 4.3.2

Thanks for the report and suggestion. I would leave this to @danilo-leal (he is our awesome designer ;))

Hey @akashshyamdev, thank you for reporting this one up.
We haven't really given much attention to the current home page as we have a new very different one baking in the oven that will be released very soon, with v5 stable, substituting this one. If you still feel like pushing your changes forward (I'm ok with them), go ahead! Just know they would probably be up for a short time :)

Ok I'd be glad to push said changes. Just one thing, do you know where the file for the hero section on the landing page is? I've searched in '/docs/src/pages/landing' but I can see all the sections except the hero section(with heading). In the screenshot above, I had edited the page through dev tools.

@akashshyamdev There you go!


Use build args to capture build time data

Acceptance Criteria:

Use the --build-args flag when running docker build to pass in the following items (and more if needed)
BUILD_DATE
VERSION

You will need to do the following in the dockerfile to persist the data into the image:
ARG VERSION
ARG BUILD_DATE
ENV VERSION $VERSION
ENV BUILD_DATE $BUILD_DATE

This issue was moved to teamhephy/fluentd#6


colorcolumn not visible

I use 80 characters colorcolumn and that is not visible:

thats because some guys didnt want colorcolumn to show up. so i tried to disable it with setting that fg  as similar to background color. Should i consider a bitt lighter + a bit darker foreground to it ?

@Mangeshrex color similar to line numbers would even work because we usually don't focus on colorcolumn much until we cross it so it would be non disturbing and even visible

@Mangeshrex color similar to line numbers would even work because we usually don't focus on colorcolumn much until we cross it so it would be non disturbing and even visible

Ok I'll add a bit lighter color than the background

Still colorcolumn is not visible....

Comments right? Ill push that too. Didn't get time to boot

Yeah comments are looking awesome

Updated the changes


Expected { after 'if' condition even with single lines

Using<EMAIL_ADDRESS>the following code fails:
const test = false
if (test)
  console.log(false)
else
  console.log(true)

Error:
  /Users/jonas/Development/sandbox/test/test.js:2:1: Expected { after 'if' condition.
  /Users/jonas/Development/sandbox/test/test.js:4:1: Expected { after 'else'.

whereas this one works:
const test = false
if (test) console.log(false)
else console.log(true)

Could be related to https://github.com/feross/standard/issues/664 , but the issue has been closed and this issue still exists
This makes it difficult to use single-line if blocks when they are too long to fit on the same line.

I think this is intentional, I at least prefer this behaviour

The error says Expected { after 'if' condition.
If I add the curly brackets like this:
const test = false
if (test) {
  console.log(false)
}
else {
  console.log(true)
}

It says there's an unnecessary { after if (due to being 1 line)
This is confusing to say the least.

I think that it's like so,
single line:
if (thisIsAll) onOneLine()

multiple lines:
if (thisIsActually)
  twoLine()

@jonaswindey
const test = false
if (test) {
  console.log(false)
} else {
  console.log(true)
}

Ok, I guess that rule changed than? I remember that
const test = false
if (test) {
  console.log(false)
} else {
  console.log(true)
}

Gave  Unnecessary { after 'if' condition. because it only contained one line (or other ifs)
Thanks for clarifying


Error on building on freshly installed Docker Enviroment

Unhandled Exception: Microsoft.Data.Sqlite.SqliteException: SQLite Error 10: 'disk I/O error'.
at Microsoft.Data.Sqlite.Interop.MarshalEx.ThrowExceptionForRC(Int32 rc, Sqlite3Handle db)
at Microsoft.Data.Sqlite.SqliteCommand.ExecuteReader(CommandBehavior behavior)
at Microsoft.Data.Sqlite.SqliteCommand.ExecuteScalar()
at Microsoft.EntityFrameworkCore.Storage.Internal.RelationalCommand.Execute(IRelationalConnection connection, String executeMethod, IReadOnlyDictionary2 parameterValues, Boolean closeConnection)    at Microsoft.EntityFrameworkCore.Migrations.HistoryRepository.Exists()    at Microsoft.EntityFrameworkCore.Migrations.HistoryRepository.GetAppliedMigrations()    at Microsoft.EntityFrameworkCore.RelationalDatabaseFacadeExtensions.GetAppliedMigrations(DatabaseFacade databaseFacade)    at Microsoft.EntityFrameworkCore.RelationalDatabaseFacadeExtensions.GetPendingMigrations(DatabaseFacade databaseFacade)    at NadekoBot.Services.DbService.GetDbContext() in /opt/NadekoBot/src/NadekoBot/Services/DbService.cs:line 28    at NadekoBot.NadekoBot..ctor(Int32 shardId, Int32 parentProcessId, Nullable1 port) in /opt/NadekoBot/src/NadekoBot/NadekoBot.cs:line 86
at NadekoBot.Program.Main(String[] args) in /opt/NadekoBot/src/NadekoBot/Program.cs:line 15
Platform: Windows 10 x64 Home Edition
Tool: Docker Toolbox 17.07.0-ce

Oh, you're on windows, that docker is for unix environments. I believe there's a difference between them (I may be completely wrong here)

Whose docker do you use? Also, have you first tried installing and running nadeko normally on your computer?
Unrelated question?: why you want to run NadekoBot in a docker?

@Poag sorry for the ping, are we able to use the image you created on docker for windows or not?

Shouldn't make any difference, it takes everything it needs with the container.
They appearing to be having an issue reading from disk on the local device. So it would appear docker doesn't have rights into windows to r/w the local filesystem.
But its windows so...why would you run the docker there? I'd go as far to say its actually inadvisable to run it in windows.