fix: pgn annotation for illegal moves

fixes https://github.com/Disservin/fast-chess/issues/523

i think this should fix it ?

it worked:
[Event "Fast-Chess Tournament"]
[Site "?"]
[Date "2024.07.10"]
[Round "3"]
[White "Engine1"]
[Black "Engine2"]
[Result "0-1"]
[SetUp "1"]
[FEN "rnbqk2r/pp3ppp/2pbpn2/8/3Pp2N/2P2P2/PP4PP/RNBQKB1R w KQkq - 1 7"]
[GameDuration "00:00:01"]
[GameStartTime "2024-07-10T02:56:34 +0700"]
[GameEndTime "2024-07-10T02:56:36 +0700"]
[PlyCount "3"]
[Termination "illegal move"]
[TimeControl "10+0.1"]

1. fxe4 {-0.73/16, 0.391s} Nxe4 {+0.22/17, 0.455s}
2. f2c3 {-0.59/19, 0.394s, Engine1 made an illegal move} 0-1


register istio operator v1alph2 api.

@morvencao: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_operator
1d56fd49ddd117c64a3f41737dd8f585c1e37306
link
/test gencheck_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_operator
1d56fd49ddd117c64a3f41737dd8f585c1e37306
link
/test gencheck_operator

lint_operator
1d56fd49ddd117c64a3f41737dd8f585c1e37306
link
/test lint_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_operator
1d56fd49ddd117c64a3f41737dd8f585c1e37306
link
/test lint_operator

gencheck_operator
8f0b2f0c35490a7945c7b63517c79c552d47827a
link
/test gencheck_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


where to set proxy address？

Hi, we use vpn to access binance in china, so it seems we need set proxy in some where in backtrader_binance, or we cannot access binance. I wonder where to set proxy in backtrader_binance.

Hi, we use vpn to access binance in china, so it seems we need set proxy in some where in backtrader_binance, or we cannot access binance. I wonder where to set proxy in backtrader_binance.

Hi! ) If you use windows - you can use that solution https://github.com/WISEPLAT/python-proxy
It will redirect all youe traffic through proxy address and port which ones you specify.


Add ChunkLayout class

Provides information about the chunk coordinate space for better cross-implementation support. The current API assume all implementations use the same chunk layout or leaves it undefined.
See https://github.com/SpongePowered/Sponge/issues/228

This is extremely awesome. Especially because this could mean sponge could work with a Cubic Chunks system.

Linking #546 since there are remarkable similarities in goals that these accomplish...


Change transaction types to serializable

Hello,
What difference does it make to enable the transaction types to serializable?
Should we enable or disable it by default?
Thank you for your work!

Disable by default. It's an overly strict mode for when your store is
basically inoperable.
On Tue, Jun 23, 2015 at 8:27 AM, vmaph2m<EMAIL_ADDRESS>wrote:

Hello,
What difference does it make to enable the transaction types to
serializable?
Should we enable or disable it by default?
Thank you for your work!
—
Reply to this email directly or view it on GitHub
https://github.com/philwinkle/Philwinkle_DeadlockRetry/issues/13.

--
Follow me on Twitter - @philwinkle
Friend me on Facebook - http://www.facebook.com/philwinkle


Missing arithmetic operation - xadd

Description
When executing xadd instruction the add operation is forgotten.
Reference:
xadd: Ref. Intel 64 and IA-32 Architecture Software Developer's Manual Vol. 2C 5-580
Affected instructions:
0x0fc1c0
0x640fc1c0
0x64670fc1c0
0x670fc1c0

Reproduction guide
Instruction:
00000000  0FC1C0            xadd eax,eax

Input:
pyvex.IRSB("\x0F\xC1\xC0", 0x8048000, archinfo.ArchX86())

Observed output:
IRSB {
   t0:Ity_I32 t1:Ity_I32 t2:Ity_I32 t3:Ity_I32

   00 | ------ IMark(0x0, 3, 0) ------
   01 | t0 = GET:I32(eax)
   02 | PUT(cc_op) = 0x00000003
   03 | PUT(cc_dep1) = t0
   04 | PUT(cc_dep2) = t0
   05 | PUT(cc_ndep) = 0x00000000
   06 | PUT(eax) = t0
   NEXT: PUT(eip) = 0x00000003; Ijk_Boring
}

Expected output:
The add operation is included.
System Info
OS:
# uname -a
Linux ubuntu 4.10.0-28-generic #32-Ubuntu SMP Fri Jun 30 05:32:18 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
# cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=17.04
DISTRIB_CODENAME=zesty
DISTRIB_DESCRIPTION="Ubuntu 17.04"

PyVEX:
#  pip freeze | grep pyvex
pyvex==<IP_ADDRESS>

It seems  that this bug has not been repaired by PyVEX. I supposed that maybe this instruction won't appear in actual software.

It seems  that this bug has not been repaired by PyVEX. I supposed that maybe this instruction won't appear in actual software.

Thanks for your interest. Quick google search shows this:
https://stackoverflow.com/questions/57205494/why-is-there-a-locked-xadd-instruction-in-this-disassambled-stdstring-dtor

Thanks for your interest. Quick google search shows this:
https://stackoverflow.com/questions/57205494/why-is-there-a-locked-xadd-instruction-in-this-disassambled-stdstring-dtor

I know there is “xadd", but is there "xadd eax, eax" in the commodity software? Thanks.

I know there is “xadd", but is there "xadd eax, eax" in the commodity software? Thanks.

Don't know 😄 But those forms can always appear in the future, if not now.

Don't know 😄 But those forms can always appear in the future, if not now.


Scrolling through model

@Jasper8888 commented on Thu Feb 14 2019
Hi,
It would be usefull if one can scroll through a model by left-click on the canvas and hold the button. Mouse movement will now navigate through the model.

Documentdetails
⚠ Dit gedeelte niet bewerken. Het is vereist om problemen te koppelen tussen docs.microsoft.com en GitHub.

ID: f4cfe8c6-b047-1277-8726-0d45d3630b8d
Version Independent ID: b2e24404-209e-f043-b749-2e2e865a9e79
Content: Modelweergave in Power BI Desktop gebruiken (preview) - Power BI
Content Source: powerbi-docs/desktop-modeling-view.md
Service: powerbi
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Hi team,
The user is making a suggestion on the functionality of the product.
best

Thanks @CeciAc. @Jasper8888 - You can submit product feedback or feature ideas by going to https://ideas.powerbi.com. You can also engage with the Power BI engineering team by being active in the Power BI Community, at https://community.powerbi.com.


bug: rollup inlines external dependency because import path was re-written

Type of Issue

[x] Bug Report
[ ] Feature Request

Description
FESM bundles include inlined external dependencies.
How To Reproduce
Exmaple w/ ngx-translate:
The ng-package.json:
{
    "$schema": "../../node_modules/ng-packagr/lib/ng-package.schema.json",
    "lib": {
        "entryFile": "public_api.ts",
        "externals": {
            "@ngx-translate/core": "ngx-translate-core"
        }
    }
}

The source code:
import { TranslateService } from '@ngx-translate/core';

export class MyLib {
  // do stuff with TranslateService
}

In the .ng_build directory, the generated .js file ends up with the following import:
import { TranslateModule, TranslateLoader } from '@ngx-translate/core/index';

The import path from @ngx-translate/core in the source was re-written to @ngx-translate/core/index,  thus the externals mapping does not match and rollup inlines the third-party dependency.

To confirm the cause, the following setting then produces a good FESM bundle!
{
    "lib": {
        "entryFile": "public_api.ts",
        "externals": {
            "@ngx-translate/core": "ngx-translate-core",
            "@ngx-translate/core/index": "ngx-translate-core"
        }
    }
}

FESM Bundle:
import { TranslateLoader, TranslateModule } from '@ngx-translate/core/index';

Expected Behaviour
Externals dependencies shouldn't be rewritten.
Usage of lib.externals in ng-package.json SHOULD BE the TypeScript import path from sources!
In above examples: @ngx-translate/core.
Version Information
ng-packagr: v1.0.0-pre.14
node: v8.1.1
@angular: v4.3.3
@ngx-translate/core: v7.1.0
rxjs: 5.4.2
zone.js: 0.8.16

Cause explained in angular/angular#16084
Issue tracked by angular/tsickle#376

In Angular, there is one place where they set to false and one to true
tsc-wrapped sets it to false:
https://github.com/angular/angular/blob/6af2396cf75a6c6f691c7551b89440a217d9de93/packages/tsc-wrapped/src/main.ts#L68-L69
compiler-cli sets it to true:
https://github.com/angular/angular/blob/450a13dea03b5b3cd3b92c5a840a6536ec003f83/packages/compiler-cli/src/main.ts#L58-L64

See also angular/angular#16152 (back-dating to April...Oo)

Is it why I get the Field 'browser' doesn't contain a valid alias configuration all over the place?
Is there any work around?

@avatsaev Possibly yes. Possibly no. Angular CLI's error messaging can be quite verbosive w/o concrete indication of the cause. The above error message often raises when an import cannot be properly resolved.

Could you please help me out, I've been pulling my hair for 2 days, every time I install the package from archive or npm, I get this error when I build the host angular app with angular cli:

ERROR in ./node_modules/ngx-codemirror/ngx-codemirror/ngx-codemirror.es5.js
Module not found: Error: Can't resolve '../../lib/codemirror' in '/Users/avatsaev/DEV/Angular/LIBS/ngx-codemirror/node_modules/ngx-codemirror/ngx-codemirror'
resolve '../../lib/codemirror' in '/Users/avatsaev/DEV/Angular/LIBS/ngx-codemirror/node_modules/ngx-codemirror/ngx-codemirror'
  using description file: /Users/avatsaev/DEV/Angular/LIBS/ngx-codemirror/node_modules/ngx-codemirror/package.json (relative path: ./ngx-codemirror)
    Field 'browser' doesn't contain a valid alias configuration
  after using description file: /Users/avatsaev/DEV/Angular/LIBS/ngx-codemirror/node_modules/ngx-codemirror/package.json (relative path: ./ngx-codemirror)
    using description file: /Users/avatsaev/DEV/Angular/LIBS/ngx-codemirror/package.json (relative path: ./node_modules/lib/codemirror)
      no extension
        Field 'browser' doesn't contain a valid alias configuration
        /Users/avatsaev/DEV/Angular/LIBS/ngx-codemirror/node_modules/lib/codemirror doesn't exist
      .ts
        Field 'browser' doesn't contain a valid alias configuration
        /Users/avatsaev/DEV/Angular/LIBS/ngx-codemirror/node_modules/lib/codemirror.ts doesn't exist
      .js
        Field 'browser' doesn't contain a valid alias configuration
        /Users/avatsaev/DEV/Angular/LIBS/ngx-codemirror/node_modules/lib/codemirror.js doesn't exist
      as directory
        /Users/avatsaev/DEV/Angular/LIBS/ngx-codemirror/node_modules/lib/codemirror doesn't exist

this is my ng-package.json:
{
  "$schema": "./node_modules/ng-packagr/ng-package.schema.json",
  "lib": {
    "name": "ngx-codemirror",
    "entryFile": "src/app/lib/index.ts",
    "externals": {
      "codemirror/index": "codemirror"
    }
  }
}

My external dependency is codemirror and I use the typings from @types/codemirror
And when I package the lib without externals defined in ng-package.json i get these warnings:
'codemirror/index' is imported by .ng_build/ts/ngx-codemirror/directives/codemirror/codemirror.directive.js, but could not be resolved – treating it as an external dependency
'codemirror/index' is imported by .ng_build/ngx-codemirror/ngx-codemirror.es5.js, but could not be resolved – treating it as an external dependency
No name was provided for external module 'codemirror/index' in options.globals – guessing 'index'

Did you try the workaround as described above?
{
    "lib": {
        "entryFile": "public_api.ts",
        "externals": {
            "@ngx-translate/core": "ngx-translate-core",
            "@ngx-translate/core/index": "ngx-translate-core"
        }
    }
}

isn't it what I did?
{
  "$schema": "./node_modules/ng-packagr/ng-package.schema.json",
  "lib": {
    "name": "ngx-codemirror",
    "entryFile": "src/app/lib/index.ts",
    "externals": {
      "codemirror/index": "codemirror"
    }
  }
}```

Did you add both? codemirror and codemirror/index?
If still not working, you could try to check the global token "codemirror" - which one do they use? Seems like they write "CodeMirror" ... is it case sensitive? see #119
If then still not working, you could try to upgrade to<EMAIL_ADDRESS>since it has added rollup-plugin-commonjs support and that may (maybe?) solve your issue.
When you find the solution, please let us know so we can keep this as documentation on how to configure external libraries.

So I did add both as you suggested (with 1.2.0):
Here is the public repo: https://github.com/avatsaev/ngx-codemirror
"lib": {
    "name": "ngx-codemirror",
    "entryFile": "src/app/lib/index.ts",
    "externals": {
      "codemirror/index": "codemirror",
      "codemirror": "codemirror"
    }
  }

and got this error:

ERROR in ./node_modules/ngx-codemirror/ngx-codemirror/ngx-codemirror.es5.js
Module not found: Error: Can't resolve 'codemirror/index' in '/Users/avatsaev/DEV/Angular/LIBS/ngx-codemirror/node_modules/ngx-codemirror/ngx-codemirror'
resolve 'codemirror/index' in '/Users/avatsaev/DEV/Angular/LIBS/ngx-codemirror/node_modules/ngx-codemirror/ngx-codemirror'
  Parsed request is a module
  using description file: /Users/avatsaev/DEV/Angular/LIBS/ngx-codemirror/node_modules/ngx-codemirror/package.json (relative path: ./ngx-codemirror)
    Field 'browser' doesn't contain a valid alias configuration
  after using description file: /Users/avatsaev/DEV/Angular/LIBS/ngx-codemirror/node_modules/ngx-codemirror/package.json (relative path: ./ngx-codemirror)
    resolve as module
      /Users/avatsaev/DEV/Angular/LIBS/ngx-codemirror/node_modules/ngx-codemirror/ngx-codemirror/node_modules doesn't exist or is not a directory
      /Users/avatsaev/DEV/Angular/LIBS/ngx-codemirror/node_modules/ngx-codemirror/node_modules doesn't exist or is not a directory
      /Users/avatsaev/DEV/Angular/LIBS/ngx-codemirror/node_modules/node_modules doesn't exist or is not a directory
      /Users/avatsaev/DEV/Angular/LIBS/node_modules doesn't exist or is not a directory
      /Users/avatsaev/DEV/Angular/node_modules doesn't exist or is not a directory
      /Users/avatsaev/DEV/node_modules doesn't exist or is not a directory
      /Users/avatsaev/node_modules doesn't exist or is not a directory
      /Users/node_modules doesn't exist or is not a directory
      /node_modules doesn't exist or is not a directory

I have the same problem with Mousetrap

Same problems with @swimlane/ngx-datatable, which is unfortunate because soooo close to having converted my entire manual build process over to jsut ng-packagr + asset bundling

Same problem with angular-calendar

Hi all, @avatsaev @ferreirarod @ronnyek @Ks89,
I wanted took a look at the ngx-codemirror error. I think it should be representative for what most of you are experiencing. This is what I got:
$ git checkout https://github.com/avatsaev/ngx-codemirror.git && cd ngx-codemirror
$ yarn install && yarn pkg
yarn install v1.1.0
info No lockfile found.
[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
[3/4] 🔗  Linking dependencies...
warning<EMAIL_ADDRESS>has unmet peer dependency "webpack-sources@>= 1.0.0".
[4/4] 📃  Building fresh packages...
success Saved lockfile.
✨  Done in 381.03s.
yarn run v1.1.0
$ ng-packagr -p ng-package.json && cd dist && npm pack
Building Angular library from ng-package.json
Built Angular library from ~/github/ngx-codemirror, written to ~/github/ngx-codemirror/dist
ngx-codemirror-1.0.0.tgz
✨  Done in 43.61s.

This is a build success for the library.
What exactly is the build error? Can you provide repros?

It builds no problem, but try to use the resulting pkg in a host angular cli app, then compile the app in prod mode, you’ll get an error

Sorry. I stopped one step short:
$ cd dist && yarn link && cd ..
$ yarn link "ngx-codemirror"
$ yarn build
ERROR in ./dist/ngx-codemirror/ngx-codemirror.es5.js
Module not found: Error: Can't resolve 'codemirror/index' in '~/github/ngx-codemirror/dist/ngx-codemirror'
resolve 'codemirror/index' in '~/github/ngx-codemirror/dist/ngx-codemirror'
  Parsed request is a module
  using description file: ~github/ngx-codemirror/dist/package.json (relative path: ./ngx-codemirror)
    Field 'browser' doesn't contain a valid alias configuration
  after using description file: ~github/ngx-codemirror/dist/package.json (relative path: ./ngx-codemirror)
    resolve as module

Modified app.module.ts to try to find out why the import cannot be resolved.
//import {CodeMirrorModule} from 'ngx-codemirror';
import { fromTextArea } from 'codemirror/index';

It builds.

Then, I modified ngx-codemirror.es5.js by hand:
import { Directive, ElementRef, EventEmitter, Input, NgModule, Output } from '@angular/core';
import { CommonModule } from '@angular/common';
import { fromTextArea } from 'codemirror';
var CodeMirrorDirective = /** @class */ (function () {

and app.module.ts:
import {CodeMirrorModule} from 'ngx-codemirror';

And yarn build:
Time: 22340ms
chunk {inline} inline.bundle.js, inline.bundle.js.map (inline) 5.83 kB [entry] [rendered]
chunk {main} main.bundle.js, main.bundle.js.map (main) 636 kB {vendor} [initial] [rendered]
chunk {polyfills} polyfills.bundle.js, polyfills.bundle.js.map (polyfills) 217 kB {inline} [initial] [rendered]
chunk {styles} styles.bundle.js, styles.bundle.js.map (styles) 20.4 kB {inline} [initial] [rendered]
chunk {vendor} vendor.bundle.js, vendor.bundle.js.map (vendor) 2.33 MB [initial] [rendered]
✨  Done in 28.95s.

I see two possibilities:

either wait for tsickle#376 to be fixed
or try to get PR angular#19083 so good that it gets merged

"Field 'browser' doesn't contain a valid alias configuration" error message is so un-descriptive of the issue and it seems to happen quite often ..

very true, and there is a lot of these confusing build system err msgs.

I'm running into a similar situation with a library I'm trying to create with ng-packagr. The library builds fine, but when I try to npm link from a host angular-cli app, I get a similar error. The library is simple and has no peerDependencies besides @angular/core, @angular/common, rxjs and zone.js, but I get a Field 'browser' doesn't contain a valid alias configuration error as well, with it being unable to resolve any of the .d.ts files generated by the library.
Using angular 4.4.4, cli 1.4.4, ng-packagr 1.2.1, though I tried a bunch of different combinations to no avail.

@dherges The library produced by ng-packagr looks structurally similar to @angular's packages (the recommended angular package format), so what's the difference between what they're doing and what we're doing?  I tried changing tsconfig in the library to have
"angularCompilerOptions": {
		"annotateForClosureCompiler": false

(which seems to be related to the issues above) but that didn't work either.

Hi @sumigoma,
please make sure that your issue IS or IS NOT the appended /index path.
If it's not, please check whether your issue is solved by #117 (or #132).

You're right, my issue actually looks more like #117. I did encounter the appended /index path problem at one point as well, but adding externals: to ng-package.json did the trick for that issue.

though not much ng-packagr can do about it, it's causing a lot of confusion and thus it's high prio

As of the release of angular 4.4.5, this should no longer be an issue due to (this fix)[https://github.com/angular/angular/pull/19579]

Seems to be working with 4.4.5

Is this issue solved by angular/angular#19579 ?
I think it is.
If yes, this issue can be closed.

Should the dependency on @angular/tsc-wrapped be bumped to ^4.4.5 to work for it properly?

I confirm that now it's working also with Mousetrap. Well done and thank u.

Let's bump the dependency.

Closing this issue as it should be solved by the change in @angular/tsc-wrapped!


Significant lag in dispatching?

I'm a newcomer to Redux Saga, and am trying to use it in a performance-sensitive application.
My setup is super simple, adapted from the intro tutorial.
In my React component, I bind an event on click to dispatch the following action:
export function selectArtist(artist) {
  console.log("Select Artist called in action", performance.now())
  return {
    type: SELECT_ARTIST,
    artist
  }
}

The action is responded to with this saga:
export function* selectArtist(action) {
  yield put(markArtistAsSelected(action.node));
}

export function* watchSelectArtist() {
  yield* takeEvery(SELECT_ARTIST, selectArtist)
}

The reducer handles the action put by the saga:
export default function reducer(state = fromJS(nodesData), action) {
  switch (action.type) {
    case MARK_ARTIST_AS_SELECTED:
      console.log("Reducer hit!", performance.now());
      // other stuff
  }
}

My problem is with the performance of this implementation. You can see I added some logs with performance.now() to benchmark. This implementation takes ~60-90ms to get from the action to the reducer.
Comparatively, using Redux Thunk takes 1-2ms.
Am I doing something wrong, or is this just the inherent cost of using generators? In most applications, 60ms might not be too bad but it's noticeable for what I'm doing.
I really like the API and methodology of redux saga :) I'm really hoping there's a way to get that timing down!
Full source here: https://github.com/joshwcomeau/panther

There are 2 possible cases here:

The saga is slow and takes 90ms to dispatch the new action
The saga is delayed for something like 90ms for unknown reasons

The saga code is called AFTER your store's root reducer is called and has computed the new state.
I suspect that your reducer is slow, and that the saga is delayed by 90ms because it is the time your reducer takes to handle that event.
You should try to log performance around these updates:
return state.updateIn([FUTURE, 'nodes'], nodes => {
        return nodes.map( (node, index) => {
          return nodeIndex === index
          ? node.set( 'selected', true )
          : node.set( 'rejected', true );
        });
      });

I may be wrong but it is likely to me that this part of your app is slower than you think and takes 90ms.
I use redux-saga and did not notice any performance problem with it.
If that's not the case don't hesitate to show the former redux-thunk version (including log statements).

@slorber I did the following:
      console.log("Start new state", performance.now())

      const newState = state.updateIn([FUTURE, 'nodes'], nodes => {
        return nodes.map( (node, index) => {
          return nodeIndex === index
          ? node.set( 'selected', true )
          : node.set( 'rejected', true );
        });
      });

      console.log("Finish new state", performance.now())

      return newState

And got the following:
Start new state 5387.865000000001
Finish new state 5388.795

So yeah, that part takes less than 1 millisecond. It isn't the problem.
I did notice, when putting more logs in the reducer, that there is a lot of stuff happening and I'm not sure why:

This looks wrong for so many reasons. Things are being resolved before being triggered.
Here's what the redux thunk version looks like:

At a loss as to what could be wrong, I followed the intro tutorial pretty closely.. Appreciate any light anyone can shed!

Can you log the execution time of each action inside the reducer? Because here we only see the time when the reducer receives the action, but it does not show how much time does this action takes to be handled.
Also it would help if you log the time around each dispatch.
Also do you use a saga monitor? Can you try without it?
Can you also try to use actionCreators/thunks without disabling the saga middleware?

Not sure what you mean by the execution time of the action inside the reducer, I believe I am doing that. I start the "clock" before the state mutation, and end it right before returning the new state. I believe if the reducer's operations was the problem, it would be consistently slow between Redux Saga and the Thunk alternative. I could be mistaken though.
I am not using a saga monitor.
I'm at work now so I can't edit it right now, but I will post what you've asked for this evening :) Thanks for your help!
The other thing I'm unclear about is if that sequence of events in the reducer is correct. Is there a bug in my saga code? Why are 3 effects being triggered, and why are they resolved before being triggered?

@joshwcomeau as far as I know redux-saga do dispatch internal events so that we could plug additional devtools based on these (like the saga monitor I mention).
It seems that each of these events are 15ms from each others, so there is something weird that takes time because if you don't have a saga monitor nobody else should be interested by these events...
Maybe try to use redux-ignore so that these events never gets dispatched to your reducer?

@joshwcomeau put effects are dispatched on the next microtask (Promise.then) but 60-90ms is too high for the Promise.then latency even with a polyfilled version.
I think there is time lost in dispatching Saga monitor events (monitor events are only dispatched in the development mode) : Each time a Saga yield an effect, a monitoring event is dispatched and will go throw the entire store.dispatch pipeline (unless intercepted by the Saga monitor). So you may try to disable dev. mode to see the impact.
I did some quick experiments on the counter demo (sceenshots below), the 1st in dev mode (monitoring enabled) and the 2nd with monitoring disabled. You can see the diff between the 2 effects
With monitoring enabled

With monitoring disabled

As for the Effect-triggred/resolved order; i think the first Effect resolved action belongs to a formed effect, you can verify by logging the whole Effect action (like above) so you can see the actual effectIds in the sequence

@slorber Thanks for the tip about redux-ignore!
@yelouafi thanks so much for spending the time experimenting with this :) am I correct in assuming that development mode is disabled the same way as in production? By setting process.env.NODE_ENV?
Will post an update later when I've had the chance to experiment with dev/prod mode.

Success! Switching to production mode lowers it to just 11ms :D
So glad I get to keep using redux saga. Thanks again @slorber and @yelouafi!

great!
@yelouafi how comes this dev mode is so expensive?

while(true) {
  yield take(ACTION)
  yield fork(worker)
}

when ACTION is dispatched there will be an EFFECT_RESOLVED for the take effect, then a pair EFFECT_TRIGGERED/RESOLVED for the fork effect. Each event will be dispatched in a microstask and will also trigger a full Store.dispatch cycle. If there is no Saga monitor to intercept events, then the actions will also be invoked on the entire reducer tree
I think it's the accumulated effect of both monitor events dispatching and the overhead of microtasks

makes sense but what I see in source code is that the reudcer tree of @joshwcomeau is pretty small and will ignore fastly all these redux-saga events. The overhead of microtasks is unlikely to delay each dispatchs of 15ms.
Maybe there's an existing middleware that does not handle redux-saga effects in an efficient way. I suggest some middleware is trying to log the effect actions, and in some cases it may not be performant because you put tasks in these effects. I think redux-saga should rather avoid to dispatch unserializable events containing objects with methods, like promises or errors. And it may be the cause of this latency no?

Maybe. I'll have to check that

Hi,
This should not be closed, as it is not solved: switching to prod isn't a solution, I may not want my saga to lag on dev too.
There should be some way to disable those debug events by configuring redux-saga (or some doc about it, I may have missed something :) ).

@eric-burel please share ur sample repository with issue reproduced. The issue mentioned in this thread no longer exists (or at least the underlaying reason might be different) because redux-saga is no longer dispatching any debug events - instead it allows you to pass in sagaMonitor object if you want debug this way, but it is an opt-in and doesnt do anything if not passed explicitly.

Okay, actually redux-logger seemed to be the responsible of lag issues in my case, it was the first I investigated and it succeed, saga does not seem to create much troubles. Thanks for the very quick answer!

Is there any way to put redux-saga into production mode without fiddling with NODE_ENV? E.g. if I want to run React in dev mode but redux-saga in prod mode?

@csvan you always can patch-package Redux Saga. I wonder though what's causing the slowness - could you share a performance profile or something?

@Andarist after careful digging it turned out our profile was misleading and that the problem was in another package. Thanks for the help anyway 😊


Resolve: Service not found

Debian 8 Server
[15:53:43][0x00007fdadb7fe700] miner#5 | Starting thread #5 (CPU-XENONCAT-AVX)
[15:53:43][0x00007fdada7fc700] miner#7 | Starting thread #7 (CPU-XENONCAT-AVX)
[15:53:43][0x00007fdaeac4b700] stratum | Connecting to stratum server usa:usa
[15:53:43][0x00007fdad8ff9700] miner#10 | Starting thread #10 (CPU-XENONCAT-AVX)
[15:53:43][0x00007fdad9ffb700] miner#8 | Starting thread #8 (CPU-XENONCAT-AVX)
[15:53:43][0x00007fdad97fa700] miner#9 | Starting thread #9 (CPU-XENONCAT-AVX)
[15:53:43][0x00007fda6af08700] miner#11 | Starting thread #11 (CPU-XENONCAT-AVX)
[15:53:43][0x00007fdaeac4b700] stratum | resolve: Service not found
[15:53:43][0x00007fdaeac4b700] stratum | Reconnecting in 3 seconds...
[15:53:46][0x00007fdaeac4b700] stratum | Connecting to stratum server usa:usa
[15:53:46][0x00007fdaeac4b700] stratum | resolve: Service not found
[15:53:46][0x00007fdaeac4b700] stratum | Reconnecting in 3 seconds...
[15:53:49][0x00007fdaeac4b700] stratum | Connecting to stratum server usa:usa
[15:53:49][0x00007fdaeac4b700] stratum | resolve: Service not found
[15:53:49][0x00007fdaeac4b700] stratum | Reconnecting in 3 seconds...
[15:53:52][0x00007fdaeac4b700] stratum | Connecting to stratum server usa:usa
[15:53:52][0x00007fdaeac4b700] stratum | resolve: Service not found
[15:53:52][0x00007fdaeac4b700] stratum | Reconnecting in 3 seconds...
[15:53:54][0x00007fdaecf4b740] Speed [300 sec]: 0 I/s, 0 Sols/s
[15:53:55][0x00007fdaeac4b700] stratum | Connecting to stratum server usa:usa
[15:53:55][0x00007fdaeac4b700] stratum | resolve: Service not found
[15:53:55][0x00007fdaeac4b700] stratum | Reconnecting in 3 seconds...
[15:53:58][0x00007fdaeac4b700] stratum | Connecting to stratum server usa:usa
[15:53:58][0x00007fdaeac4b700] stratum | resolve: Service not found
[15:53:58][0x00007fdaeac4b700] stratum | Reconnecting in 3 seconds...
[15:54:01][0x00007fdaeac4b700] stratum | Connecting to stratum server usa:usa
[15:54:01][0x00007fdaeac4b700] stratum | resolve: Service not found
[15:54:01][0x00007fdaeac4b700] stratum | Reconnecting in 3 seconds...
Does the same for EU as well.

Same on Ubuntu 16.04, with both -l eu or -l full-addr:port; It must be in NH servers, as other pools work ok.
EDIT: the URL in Readme fails as above, the stratum.. works ok.

Which stratum address did you get working?

I got
./nheqminer_cpu -l equihash.usa.nicehash.com:3357 -u address -t 6 to work. Guess I will do that for now until the script is fixed

Our servers were down at maintenance if this issue still persists reopen it.


feat(core): policy support for LIST of kas grants

Resolves https://github.com/opentdf/platform/issues/1295 (partially)

[x] new db queries in sqlc (see query.sql and readme) to support this list API
[x] integration tests & fixtures

Service endpoints/RPCs require a release of /protocol/go as a dependency with the new proto gencode from this PR, and will therefore be in a follow-up PR


Nightly error

After @mhegazy's comment I wanted to give the new codegen a try and downloaded 2.1.0-dev.20160907 (I am currently using Babel js).
The result on my codebase is a compiler crash:
TypeError: Cannot read property 'text' of undefined
    at addExportMemberAssignmentsForBindingName (D:\****\node_modules\typescript\lib\typescript.js:45135:95)
    at addExportMemberAssignmentsForBindingName (D:\****\node_modules\typescript\lib\typescript.js:45131:21)
    at visitVariableStatement (D:\****\node_modules\typescript\lib\typescript.js:45119:17)
    at visitor (D:\****\node_modules\typescript\lib\typescript.js:44882:28)
    at Object.visitNodes (D:\****\node_modules\typescript\lib\typescript.js:40012:48)
    at transformAsynchronousModuleBody (D:\****\node_modules\typescript\lib\typescript.js:44836:40)
    at transformAsynchronousModule (D:\****\node_modules\typescript\lib\typescript.js:44822:41)
    at transformAMDModule (D:\****\node_modules\typescript\lib\typescript.js:44757:20)
    at transformSourceFile (D:\****\node_modules\typescript\lib\typescript.js:44719:31)
    at D:\****\node_modules\typescript\lib\typescript.js:51196:89

I am sorry but I can't put up a repro our codebase is large and confidential :(

I found a simple repro. This issue occurs when you have a top level variable declaration with an array binding pattern that has a missing element and also have export specifiers in the same file:
const [a, , b] = [1, 2, 3];
export { a, b };

We parse the missing element as an OmittedExpression but incorrectly cast it to a BindingElement which always expects to have a valid name property.
I have a fix I will be pushing up shortly, and should be available in the next nightly build.

@rbuckton We definitely have this pattern somehwere in our code. I know because it was introduced very recently, like during the past week ;) So I guess that's it!

@rbuckton sent out a fix. Keep an eye out for the next nightly (~2-3 hours).

@rbuckton @DanielRosenwasser Will test again with our codebase on Monday.

Tested today with dev.20160912, the compiler crash is gone. Thanks.


PNG file stored in the flash of microcontroller

My board is stm32f429 discovery and I need to display a png image which is stored in the flash memory of the microcontroller. After reading "Use PNG images in LittlevGL"   I tried to display a 30 * 30 pixels, png image.
I used the codes in "PNG decoder for LittelvGL" and found out that I have to change some codes in lodepng.c file I made these changes:
static void* lodepng_malloc(size_t size)
{
#ifdef LODEPNG_MAX_ALLOC
	if(size > LODEPNG_MAX_ALLOC) return 0;
#endif
	// printf("%s - %zd\n", __FUNCTION__, size);
	return lv_mem_alloc(size); //malloc(size);
}

static void* lodepng_realloc(void* ptr, size_t new_size)
{
#ifdef LODEPNG_MAX_ALLOC
	if(new_size > LODEPNG_MAX_ALLOC) return 0;
#endif
	// printf("%s - %zd\n", __FUNCTION__, new_size);
	return lv_mem_realloc(ptr, new_size);//realloc(ptr, new_size);
}

static void lodepng_free(void* ptr)
{
	// printf("%s\n", __FUNCTION__);
	lv_mem_free(ptr);//free(ptr);
}

afer this I succeded but with the swaped colors of red and blue. why?
Inorder to solving this problem in convert_color_depth  function routin in png_decoder.c, I changed
c = LV_COLOR_MAKE(img_argb[i].red, img_argb[i].green, img_argb[i].blue);
to
c = LV_COLOR_MAKE(img_argb[i].blue, img_argb[i].green, img_argb[i].red); .
I also have to mention that I use freertos and these are my defines in lv_conf.h
#define LV_MEM_CUSTOM_INCLUDE "FreeRTOS.h"   /*Header for the dynamic memory function*/
#define LV_MEM_CUSTOM_ALLOC   pvPortMalloc       /*Wrapper to malloc*/
#define LV_MEM_CUSTOM_FREE    vPortFree         /*Wrapper to free*/

Now I want to display an 800 * 480 png image which needs 800 * 400 * 4 bytes for decoding and as you know this space is not available in the internal ram.
So how can I assign the bank 2 of external ram to this? -bank 1 is used for LCD display buffer-

It seems to me we can't improve the speed further so I close this issue.
I hope you can use it with the current performance.


zstd compression wastes too much time on memory allocation due to lack of destination buffer

Description
zstd compression wastes too much time on malloc due to lack of destination buffer
Versions
Sarama v1.41.0

Sarama
Kafka
Go

1.41.0
3.8.0
1.21.4

Configuration

Producer.Compression = CompressionZSTD

Logs
logs: malloc

goroutine 12165 [GC assist wait]:
runtime.gopark(0xc04dfab240?, 0x12d9c8b?, 0x0?, 0x0?, 0x74b8a57f60a8?)
/usr/local/go/src/runtime/proc.go:398 +0xce fp=0xc04dfab1c8 sp=0xc04dfab1a8 pc=0x13028ae
runtime.goparkunlock(...)
/usr/local/go/src/runtime/proc.go:404
runtime.gcParkAssist()
/usr/local/go/src/runtime/mgcmark.go:652 +0xe7 fp=0xc04dfab200 sp=0xc04dfab1c8 pc=0x12e47e7
runtime.gcAssistAlloc(0xc00f48a820)
/usr/local/go/src/runtime/mgcmark.go:509 +0x22a fp=0xc04dfab260 sp=0xc04dfab200 pc=0x12e424a
runtime.deductAssistCredit(0xc057881ea0?)
/usr/local/go/src/runtime/malloc.go:1291 +0x54 fp=0xc04dfab288 sp=0xc04dfab260 pc=0x12d27f4
runtime.mallocgc(0x37e, 0x3544380, 0x1)
/usr/local/go/src/runtime/malloc.go:1006 +0xc9 fp=0xc04dfab2f0 sp=0xc04dfab288 pc=0x12d2029
runtime.makeslice(0x389d880?, 0xc014798b60?, 0x74b80c61ae10?)
/usr/local/go/src/runtime/slice.go:103 +0x49 fp=0xc04dfab318 sp=0xc04dfab2f0 pc=0x13180e9
github.com/klauspost/compress/zstd.(*Encoder).EncodeAll(0xc0594a28c0, {0xc02221e380, 0x37e, 0x37e}, {0x0, 0x0, 0x0})
/go/pkg/mod/github.com/klauspost/compress@v1.17.4/zstd/encoder.go:516 +0x313 fp=0xc04dfab498 sp=0xc04dfab318 pc=0x1fd3b93
github.com/IBM/sarama.zstdCompress({0x0?}, {0x0, 0x0, 0x0}, {0xc02221e380, 0x37e, 0x37e})
/go/pkg/mod/github.com/!i!b!m/sarama@v1.41.0/zstd.go:71 +0x86 fp=0xc04dfab510 sp=0xc04dfab498 pc=0x20495c6
github.com/IBM/sarama.compress(0x20?, 0xc047086b88?, {0xc02221e380, 0x37e, 0x37e})
/go/pkg/mod/github.com/!i!b!m/sarama@v1.41.0/compress.go:190 +0xdae fp=0xc04dfaba38 sp=0xc04dfab510 pc=0x200bb4e
github.com/IBM/sarama.(*RecordBatch).encodeRecords(0xc009c36140, {0x3ea55e8?, 0xc008c79480?})
/go/pkg/mod/github.com/!i!b!m/sarama@v1.41.0/record_batch.go:206 +0x9a fp=0xc04dfaba80 sp=0xc04dfaba38 pc=0x203d7da
github.com/IBM/sarama.(*RecordBatch).encode(0xc009c36140, {0x3ea55e8, 0xc008c79480})
/go/pkg/mod/github.com/!i!b!m/sarama@v1.41.0/record_batch.go:88 +0x231 fp=0xc04dfabae0 sp=0xc04dfaba80 pc=0x203cd31
github.com/IBM/sarama.(*Records).encode(0xc008c79480?, {0x3ea55e8?, 0xc008c79480?})
/go/pkg/mod/github.com/!i!b!m/sarama@v1.41.0/records.go:62 +0x98 fp=0xc04dfabb28 sp=0xc04dfabae0 pc=0x203dab8
github.com/IBM/sarama.(*ProduceRequest).encode(0xc008c79440, {0x3ea55e8, 0xc008c79480})
/go/pkg/mod/github.com/!i!b!m/sarama@v1.41.0/produce_request.go:108 +0x5e7 fp=0xc04dfabcd8 sp=0xc04dfabb28 pc=0x2035367
github.com/IBM/sarama.(*request).encode(0xc0161f6450, {0x3ea55e8, 0xc008c79480})
/go/pkg/mod/github.com/!i!b!m/sarama@v1.41.0/request.go:43 +0x13c fp=0xc04dfabd00 sp=0xc04dfabcd8 pc=0x203e4dc
github.com/IBM/sarama.encode({0x3e67160, 0xc0161f6450}, {0x3e91630?, 0xc007a2f3b0})
/go/pkg/mod/github.com/!i!b!m/sarama@v1.41.0/encoder_decoder.go:29 +0x91 fp=0xc04dfabd70 sp=0xc04dfabd00 pc=0x201af31
github.com/IBM/sarama.(*Broker).sendInternal(0xc008100800, {0x3e8dc30, 0xc008c79440}, 0xc02e47a880)
/go/pkg/mod/github.com/!i!b!m/sarama@v1.41.0/broker.go:1001 +0x16b fp=0xc04dfabe50 sp=0xc04dfabd70 pc=0x1ffc94b
github.com/IBM/sarama.(*Broker).sendWithPromise(0xc008100800, {0x3e8dc30, 0xc008c79440}, 0x0?)
/go/pkg/mod/github.com/!i!b!m/sarama@v1.41.0/broker.go:991 +0x9e fp=0xc04dfabe88 sp=0xc04dfabe50 pc=0x1ffc75e
github.com/IBM/sarama.(*Broker).AsyncProduce(0xc008100800, 0xc008c79440, 0xc008c79460?)
/go/pkg/mod/github.com/!i!b!m/sarama@v1.41.0/broker.go:470 +0x1cc fp=0xc04dfabf10 sp=0xc04dfabe88 pc=0x1ffa88c
github.com/IBM/sarama.(*asyncProducer).newBrokerProducer.func1()
/go/pkg/mod/github.com/!i!b!m/sarama@v1.41.0/async_producer.go:828 +0x145 fp=0xc04dfabfa8 sp=0xc04dfabf10 pc=0x1ff2cc5
github.com/IBM/sarama.withRecover(0xc0105c9770?)
/go/pkg/mod/github.com/!i!b!m/sarama@v1.41.0/utils.go:43 +0x33 fp=0xc04dfabfc8 sp=0xc04dfabfa8 pc=0x2048a13
github.com/IBM/sarama.(*asyncProducer).newBrokerProducer.func4()
/go/pkg/mod/github.com/!i!b!m/sarama@v1.41.0/async_producer.go:793 +0x25 fp=0xc04dfabfe0 sp=0xc04dfabfc8 pc=0x1ff2b45
runtime.goexit()
/usr/local/go/src/runtime/asm_amd64.s:1650 +0x1 fp=0xc04dfabfe8 sp=0xc04dfabfe0 pc=0x1336601
created by github.com/IBM/sarama.(*asyncProducer).newBrokerProducer in goroutine 16568
/go/pkg/mod/github.com/!i!b!m/sarama@v1.41.0/async_producer.go:793 +0x2e5

Additional Context
We are trying to switch our compression algorithm from snappy to zstd, we are seeing about 25% more compressed bytes which is good but we are also seeing more CPU usage which is bad. We ended up adding 60% more pods to handle the same traffic load.
We are using sarama v1.14.0 and klauspost v1.71.4.
We generated a few stack traces and found about 75 go routines are stuck waiting for malloc, I haven't find a zstd go routine doing any real work except waiting for the malloc.
I think the malloc is from this line:
https://github.com/klauspost/compress/blob/v1.17.4/zstd/encoder.go#L516
// If less than 1MB, allocate a buffer up front.
if len(dst) == 0 && cap(dst) == 0 && len(src) < 1<<20 && !e.o.lowMem {
dst = make([]byte, 0, len(src))
}
Related issue is also fired on zstd compression library:
https://github.com/klauspost/compress/issues/987
After discussion with clauspost/compress community, the recommendation is to pass a pre-allocated dst buffer instead of nil (2nd argument in the below code segment):
https://github.com/IBM/sarama/blob/main/compress.go#L190
case CompressionZSTD:
return zstdCompress(ZstdEncoderParams{level}, nil, data)

If sarama community thinks this is the right way to fix this problem to make zstd compression performant, I can submit a PR to add the following config option for zstd compression path:
Producer.Compression.ZStandard.DestinationBufferSize = 4096
In zstd.go, each ZstdEncoder can have pre-allocated slice dstBuffer of DesintationBufferSize if this param is set.  When function zstdCompress() is invoked and when dst argument is nil and len(src) is less than DestinationBufferSize, we will use this pre-allocated dstBuffer slice to call the underlying zstd library.


VEGA-789 Add tasks "tab" page

Codecov Report

Merging #12 (46dd568) into main (d89dd57) will increase coverage by 2.10%.
The diff coverage is 93.54%.

@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
+ Coverage   86.30%   88.40%   +2.10%
==========================================
  Files           7       10       +3
  Lines         146      207      +61
==========================================
+ Hits          126      183      +57
- Misses         12       14       +2
- Partials        8       10       +2

Impacted Files
Coverage Δ

internal/sirius/cases_by_assignee.go
70.58% <ø> (ø)

...ternal/sirius/cases_with_open_tasks_by_assignee.go
83.33% <83.33%> (ø)

internal/server/server.go
90.90% <100.00%> (+0.71%)
:arrow_up:

internal/server/tasks.go
100.00% <100.00%> (ø)

internal/sirius/pagination.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


MINOR: Fix typo in MetadataVersion.IBP_4_0_IV0

This patch fixes a typo in MetadataVersion.IBP_4_0_IV0. It should be 0 not O.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Sorry I missed this on the last review. Let's make sure the build passes.

Really hard to catch so no worries.

Cherrypicked to 3.8 as well.


solved, unsubmitted kata, after refresh the code disappiered, looks completed on profile

Scenario:
I solved a kata, but did not submitted my code yet, because I was on the way of refactoring. I accidentally pressed the refresh button. My whole code disappeared and returned to the beginning state, but the kata was under the completed katas in my profile, but I can't see my solution or others solution (at least my solution would be great to see again), it behaves uncompleted (which is ok, because it's not submitted yet).

Your code should be on the Past Solutions tab of the training page. Isn't it there?

Oh, ok, I can see mine somehow now on completed...sorry for your inconvenience...


swipingEnabled does not work with UnderCenterPanel position

Swiping on tableView cells does not work when sidePanelPosition  is set to either UnderCenterPanelLeft or UnderCenterPanelRight . I have swipingEnabled set to true.

swipingEnabled is only for OverCenterPanelRight or UnderCenterPanelRight. For the positions you wrote above, you have panning, which is based on a UIScreenEdgePanGestureRecognizerso this means you have to pan from the edge of the screen. For further details, please check the Readme file.

Is it possible to have panning enabled so I can swipe to show left menu and have tableView cells swipe to delete correctly? It is 2 opposite swipe directions.


Please enable IIS Rewrite module by default in aspnet-docker image

It would be great to have access to the IIS rewrite module in IIS by default without having to modify or create our own image.  I think this makes sense as this module is available by default in Azure Websites.  Can this be added by default?

Sorry, we have no plans to add this module by default, as we have many users who do not wish to have this additional module in their base image, and it is not difficult to add it in derived images.

Just in case someone comes across this thread. Using ADD will cause docker to issue web request each time it builds the container. To avoid this and cache this layer properly, you may replace ADD with Invoke-WebRequest like this:
FROM mcr.microsoft.com/dotnet/framework/aspnet:4.8-windowsservercore-1909
ARG source
# Install IISRewrite Module
RUN Invoke-WebRequest http://download.microsoft.com/download/D/D/E/DDE57C26-C62C-4C59-A1BB-31D58B36ADA2/rewrite_amd64_en-US.msi -OutFile c:/inetpub/rewrite_amd64_en-US.msi
RUN powershell -Command Start-Process c:/inetpub/rewrite_amd64_en-US.msi -ArgumentList "/qn" -Wait

@47th Note that works pretty well for small files, but iwr is horridly slow and probably shouldn't be used for large files.  Start-BitsTransfer is a lot faster, but doesn't work on all web servers..

Using ADD is not a Dockerfile best practice.  In a case like this is you end up carrying the msi in the resulting image which is causing bloat.  See ADD or COPY best practices for details.  Keep in mind windows supports curl which is a nice alternative here.

I am trying to install this module with this dockerfile. is it possible to install it with in linux server. I always get errors like "powershell notfound", "Invoke-WebRequest notfound"
FROM mcr.microsoft.com/dotnet/aspnet:8.0 AS base
ENV TZ=Europe/Istanbul
USER $APP_UID
WORKDIR /app
EXPOSE 8080

FROM mcr.microsoft.com/dotnet/sdk:8.0 AS build
ARG BUILD_CONFIGURATION=Release
WORKDIR /src

COPY ["Sekmen.Web/Sekmen.Web.csproj", "Sekmen.Web/"]
RUN dotnet restore "Sekmen.Web/Sekmen.Web.csproj"

COPY . .
WORKDIR "/src/Sekmen.Web"
RUN dotnet build "Sekmen.Web.csproj" -c "$BUILD_CONFIGURATION" -o /app/build

FROM build AS publish
ARG BUILD_CONFIGURATION=Release
RUN dotnet publish "Sekmen.Web.csproj" -c "$BUILD_CONFIGURATION" -o /app/publish /p:UseAppHost=false

FROM base AS final
WORKDIR /app
COPY --from=publish /app/publish .

ENTRYPOINT ["dotnet", "Sekmen.Web.dll"]

I am trying to install this module with this dockerfile. is it possible to install it with in linux server. I always get errors like "powershell notfound", "Invoke-WebRequest notfound"

Why are you trying to run IIS on Linux?  It's a Windows web server.  There's plenty of great Linux web servers that will support .NET just fine.

@sekmenhuseyin IIS and URL rewrite module only work on Windows. If you need the URL rewrite functionality on Linux, you can for example use URL rewriting middleware.


Add support to Unmarshalers

Hello,
I noticed you can't use UnmarshalJSON with this library for custom types, which causes errors since it cannot set the value.
Please tell me if there is anything to change here. All test have passed.

@Cirivas Thank you for your contribution


Add enhanced location changed worker

I've added the EnhancedLocationChangedWorker.

Closing this refactoring PR as we've discussed that we will create a single refactoring PR that will be reviewed as a whole.


No release notes for 5.0.0?

I can't find them in the releases or in history.md, yet my tests are failing after the upgrade. Where might I be able to find release notes?

Sorry about that, thanks for asking. It’s been rather busy upgrading the whole ecosystem.
Nothing much changed in retext itself, but unified (the underlying interface) received quite some changes. You can find more on that on the Releases page.


Test inference warning

warning: An explicit [[test]] section is specified in Cargo.toml which currently
disables Cargo from automatically inferring other test targets.
This inference behavior will change in the Rust 2018 edition and the following
files will be included as a test target:

/home/travis/build/pingcap/rust-rocksdb/tests/test_rate_limiter.rs
/home/travis/build/pingcap/rust-rocksdb/tests/test_delete_files_in_range.rs
/home/travis/build/pingcap/rust-rocksdb/tests/test_delete_range.rs
/home/travis/build/pingcap/rust-rocksdb/tests/test_compact_range.rs
/home/travis/build/pingcap/rust-rocksdb/tests/test_prefix_extractor.rs
/home/travis/build/pingcap/rust-rocksdb/tests/test_ingest_external_file.rs
/home/travis/build/pingcap/rust-rocksdb/tests/test_event_listener.rs
/home/travis/build/pingcap/rust-rocksdb/tests/test_slice_transform.rs
/home/travis/build/pingcap/rust-rocksdb/tests/test_ttl.rs
/home/travis/build/pingcap/rust-rocksdb/tests/test_iterator.rs
/home/travis/build/pingcap/rust-rocksdb/tests/test_column_family.rs
/home/travis/build/pingcap/rust-rocksdb/tests/test_rocksdb_options.rs
/home/travis/build/pingcap/rust-rocksdb/tests/test_multithreaded.rs
/home/travis/build/pingcap/rust-rocksdb/tests/test_read_only.rs
/home/travis/build/pingcap/rust-rocksdb/tests/test_statistics.rs
/home/travis/build/pingcap/rust-rocksdb/tests/test_table_properties.rs
/home/travis/build/pingcap/rust-rocksdb/tests/test_metadata.rs
/home/travis/build/pingcap/rust-rocksdb/tests/test_compaction_filter.rs

This is likely to break cargo build or cargo test as these files may not be
ready to be compiled as a test target today. You can future-proof yourself
and disable this warning by adding autotests = false to your [package]
section. You may also move the files to a location where Cargo would not
automatically infer them to be a target, such as in subfolders.
For more information on this warning you can consult
https://github.com/rust-lang/cargo/issues/5330

Fixed in https://github.com/pingcap/rust-rocksdb/pull/243


Add a way to force the refresh on the Indexer bulk request

Being able to set $bulk->setRequestParam('refresh', 'wait_for'); could be useful.
Same goes for pipeline.

Being able to set $bulk->setRequestParam('refresh', 'wait_for'); could be useful.
Same goes for pipeline.

In fact I would extend to: being able to set any requestParam -> https://www.elastic.co/guide/en/elasticsearch/reference/current/docs-bulk.html#docs-bulk-api-query-params
Very basic implementation idea :
Maybe Indexer->setBulkRequestsParam(string $paramName, $value)
Each value stored in an associative array (like Elastica Bulk->setRequestParam()) and each time getCurrentBulk instantiate a new Bulk object, the array is read to transfer the parameters to the new Bulk

It has been done!


Creating a credentials.dat with temporary causes issues

Hi,
Seems that the package you're using to create the backup-temp folder is creating this with a 0777 permission, thus preventing majority of MySQL installations to ignore the config file.
Here's the output from mine:
mysql: [Warning] World-writable config file '/var/www/static/storage/app/backup-temp/temp/credentials.dat' is ignored.
ERROR 1045 (28000): Access denied for user 'root'@'localhost' (using password: NO)

I also added a lot to ensure it was doing everything correctly prior to it & seems correct, however here's also a copy of the command it's trying to execute
{"cmd":"mysql --defaults-extra-file=\"/var/www/static/storage/app/backup-temp/temp/credentials.dat\" tenant_ZKaQ3K < /var/www/static/storage/app/backup-restore-temp/2023-10-17-04-21-54-e4558f5d-710d-4ff4-b1e5-c97b89bb27cc/db-dumps/mysql-tenant_ZKaQ3K.sql"}

A simple fix for this, would be to chmod the file once it's created to 0644, I would also like to note that I'm doing this locally on Vagrant and using shared directories which could be the issue; however would persist with a few users.

Thanks for reporting.
I honestly can't tell right know what the right permission settings would be for the storage-folder; have to do some light research on this as well.
We also currently use file_put_contents to store that file. Might have to switch to the Storage-facade.
https://laravel.com/docs/10.x/filesystem#local-files-and-visibility
IMHO I would like to refactor this whole "credentials" part completely, but that's not something I will do in the next few days.

@CLCWebsiteServices Currently thinking about removing the whole approach of using a credentials/extra-file.
Could you maybe share when you're using this package to restore a backup? From your original comment, it looks like you're running this on a production server. What's your use case?

Sorry for the long delay, I didn't receive a notification to reply!.
But yeah, we're currently running a multi-tenancy platform of which we wanted users to create manual backups & manually restore when they wish. albeit that's why we used this package but the issue we had was tenancy was using the Storage facade, whereas you was storing manually into storage folder, so couldn't access it directly.
Here's what we did instead of using the credentials file!
Inside my RestoreTenant Job, I done the following:
private function extendDbFactory(): void
{
    DbImporterFactory::extend('mysql', new MySql());
}

And then my MySql class looks like the following:
<?php

namespace App\Domains\Backup\Databases;

use Illuminate\Contracts\Encryption\DecryptException;
use Spatie\Backup\Exceptions\CannotCreateDbDumper;
use Spatie\Backup\Tasks\Backup\DbDumperFactory;
use Wnx\LaravelBackupRestore\Databases\DbImporter;

class MySql extends DbImporter
{
    /**
     * @throws CannotCreateDbDumper
     */
    public function getImportCommand(string $dumpFile, string $connection): string
    {
        $dumper = DbDumperFactory::createFromConnection($connection);
        $importToDatabase = $dumper->getDbName();

        $credentialsArray = [
            'host' => config('database.connections.mysql.host'),
            'port' => config('database.connections.mysql.port'),
            'user' => config('database.connections.mysql.username'),
            'password' => config('database.connections.mysql.password'),
        ];

        if (str($dumpFile)->endsWith('gz')) {
            $command = $this->getMySqlImportCommandForCompressedDump($dumpFile, $credentialsArray, $importToDatabase);
        } else {
            $command = $this->getMySqlImportCommandForUncompressedDump($dumpFile, $credentialsArray, $importToDatabase);
        }

        return $command;
    }

    public function getCliName(): string
    {
        return 'mysql';
    }

    private function getMySqlImportCommandForCompressedDump(string $dumpFile, array $credentialsArray, string $importToDatabase): string
    {
        return collect([
            "gunzip < {$dumpFile}",
            '|',
            'mysql',
            '-u', $credentialsArray['user'],
            "-p'".$this->decryptIfEncrypted($credentialsArray['password'])."'",
            '-P', $credentialsArray['port'],
            isset($credentialsArray['host']) ? '-h '.$credentialsArray['host'] : '',
            $importToDatabase,
        ])->filter()->implode(' ');
    }

    private function getMySqlImportCommandForUncompressedDump(string $dumpFile, array $credentialsArray, string $importToDatabase): string
    {
        return collect([
            'mysql',
            '-u', $credentialsArray['user'],
            "-p'".$this->decryptIfEncrypted($credentialsArray['password'])."'",
            '-P', $credentialsArray['port'],
            isset($credentialsArray['host']) ? '-h '.$credentialsArray['host'] : '',
            $importToDatabase,
            '<',
            $dumpFile,
        ])->filter()->implode(' ');
    }

    private function decryptIfEncrypted($value)
    {
        try {
            return decrypt($value);
        } catch (DecryptException $e) {
            return $value;
        }
    }
}

@CLCWebsiteServices Thanks for sharing your workaround.
Will probably update the package to drop the credentials file as well and use the -u, -p and -P flags directly.
I think I've used the credentials.dat file, as I saw that the underlying Spatie package exposed this. Let's keep it simple and pass the credentials to the command directly.

I've updated the package to no longer use credentials file. Instead we pass username and password as arguments, like what you've done in your application. (#42)
A new version has been released: v1.1.3.
Note:
I didn't implement the decryptIfEncrypted-method, as this caused a couple of issues in our test suite. So you might have to stick with your implementation for a while.
I plan to rework the whole DbImporter-classes in v2 or even extract them to their own package.


fix(module:color-picker): avoid emitted twice nzOnChange event

PR Checklist
Please check if your PR fulfills the following requirements:

[X] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?
When we change the color of color-picket nzOnChange event is emitted twice
Issue Number: #8502
What is the new behavior?
When we change the color of color picker nzOnChange event is emitted only one time
Does this PR introduce a breaking change?
[ ] Yes
[X] No

Other information

LGTM


Selected Page/ Jump to certain page

HI Allen
I am using your react-bootstrap-table
for pagination , now its can only go to prev and next
is there any way , I give user a input and let user decide which page to go ?

Please check #1709.


Timestampable - add db field type parameter

We came across this issue when working with a Postgres database. The datetime field in pg ignores the timezone which was causing us some headaches. This PR allows you to define the field type as a parameter rather than (as we have done in our current project) overriding the whole subscriber class - which, incidentally, has a number of private methods and properties which make for an unnecessarily large override file. Have added some docs too.

:+1: @lopsided Tests are broken, can you fix it ?

Whoops sorry missed that. Have fixed the test and squashed. Let me know if I've missed anything else
@NiR-

thanks!


[WIP] integrate styleguide with hot reload

@visualjerk sadly we get errors on build as all external packages that are uses inside the styleguides can not be found.
Module build failed (from ./styleguide/src/loader/jsdoc-loader.js):
Error: Cannot find module 'vue-docgen-api'```
.
.
.
Module not found: Error: Can't resolve 'vue-click-outside' in '/nitro-web/styleguide/src/system/components/navigation/Menu'```
.
.
.
ERROR in ./styleguide/src/system/components/data-input/shared/input.js
Module not found: Error: Can't resolve 'async-validator' in '/nitro-web/styleguide/src/system/components/data-input/shared'
 @ ./styleguide/src/system/components/data-input/shared/input.js 4:0-37 136:26-32```

See: https://travis-ci.com/Human-Connection/Nitro-Web/builds/93019740

@roschaefer could you please check if this runs with your local development setup? For me its not somehow.

I did
git fetch origin
git checkout better-styleguide-integration # now on c0252d0194f77bfc48a98bae04d995082d014e79
docker-compose up --build

http://localhost:3000/login looks great.
Everything else would just irritate me as it's the promise of docker to have a reliable and portable setup. On the other hand we need to make sure that people can do the setup without using docker. What exactly is the problem on your machine @appinteractive ?

@roschaefer docker fails regularly on my machine. Now it states that I do not have enough space (16GB)... yeah... thats also a reason why I try to avoid using docker on my local machine... it just does not work like it should.

so @visualjerk & @roschaefer should we merge?

@appinteractive I would recommend to do a quick local test with this branch and the API running, so you can check if all views work as expected.


Custom IP for listening

I'd need an ability to specify which IP to listen to. I use the --web-port option but I need to pair it with a custom IP.
The reason for this is that I'm serving this debugger behind a proxy so that I'd get some extra privacy. I'd like to attach node-inspector for a local IP that is not visible to the outer world - and so it'd only be visible through that proxy.

There are two pull requests that currently support this feature.


scrollBehavior fires before useAsyncData is finished

Environment

Operating System: Linux
Node Version:     v16.14.2
Nuxt Version:     3.0.0-rc.3
Package Manager<EMAIL_ADDRESS>Builder:  vite
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-2vxhdz?file=app/router.options.ts

Scroll the main page a little bit.
go to "AboutPage"

Describe the bug
useAsyncData() blocks route navigation by default but scrollBehavior is not aware of that i guess so it fires before route navigation which causes scroll to top of page and then after data is fetched we are redirected to new page.
Additional context
So for now only way to handle this case is instead of using scrollBehavior use lazy: true in useAsyncData and after its finished use for example window.scrollTo(0,0) in component itself?
from docs: https://v3.nuxtjs.org/api/composables/use-async-data/#params

Under the hood, lazy: false uses  to block the loading of the route before the data has been fetched. Consider using lazy: true and implementing a loading state instead for a snappier user experience.

On top of that without defining scrollBehavior when routing to new route, page is not scrolled to top by default, instead scroll stays at the same position as before - is it desired behavior or a bug?
Logs
No response

Was this problem fixed? I use nuxt 3.10.2 and the problem is still here. ScrollBehavior fires before useAsyncData is finished.


MINOR: Update source link in interactive query page

Currently, the source reference are all pointing to the 1.0 version codes, which is obviously wrong. Update to the current dotVersion. (ref: https://github.com/apache/kafka/blob/trunk/docs/streams/developer-guide/datatypes.html#L173)
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@vvcephei , could you help review? Thanks.

@mimaison , could you help review this small PR? Thanks.

@mimaison  @vvcephei , could you help review this small PR? Thanks.


4 June: Certificate Chains in ARP

[DEP-2808] Add client certificate chain checking to Access Path Restrictions
For release w/e 29 May 2020?

Released 4 June


email verification

What does this PR do and why?
Provide a description of what this PR does. Link to any relevant GitHub issues, Notion tasks or Discord discussions.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] I have tested this PR in all supported browsers.
[ ] I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #3156

PR #3157

PR #3158

PR #3159   👈

This comment was auto-generated by Graphite.


combine fd and eva_id arguments into one admin type

It's my understanding that the fd and eva_id arguments to the driver functions are tightly coupled in meaning. For example, when you initialize with hb_mc_init_device(fd,eva_id,fname,tile,ntiles),  is it not the case that the eva_id is tied to that fd?
If my understanding is correct, right now different driver functions omit the fd argument if you don't need it. This exposes a little more to the users of this library than it should. What would be better is if there were a single type called device_t that is passed to all of these functions. Underneath the hood this could be a C struct that contains information relevant to memory allocation (like eva_id) and its low-level device descriptor fd.

Fixed in spades in #213


Add ParentIsTagHelper property to AttributeCompletionContext

Follow up to https://github.com/aspnet/Razor/issues/1572
Right now we hard code the value of this property here https://github.com/aspnet/Razor/blob/dev/src/Microsoft.CodeAnalysis.Razor.Workspaces/DefaultTagHelperCompletionService.cs#L63. This should instead come from the AttributeCompletionContext similar to ElementCompletionContext.

1b5bd4afacbcdc0a8b8e9837d91460e0411491ab


Consider quoting all provider names

We encountered an issue with pulumi-null's provider name being evaluated to the null value in Actions.
https://github.com/pulumi/ci-mgmt/pull/592 is attempting to fix that; however we should probably quote all provider names in our Actions as per code review suggestion.

Fixed as part of https://github.com/pulumi/ci-mgmt/pull/943


Discussion about the hubness problem

Hi,
Thanks for your open-source work.
I try QB-Norm for my customed model, which is based on CLIP4Clip. However, I find there exists performance degradation when I use 9K training split as the query bank. Meanwhile, I obtain the performance gain using Val split as the query bank.
I guess the assumption for the hubness problem in Figure 1 may need to be discussed more. In my experiments, the hub X2 is different in the Val query bank and training query bank. Maybe this is the reason why training query bank does not work.
I am unfamiliar with the hubness problem, look forward to a detailed discussion or suggestion. If I misunderstand the assumption, please tell me directly:) Thanks a lot.

Hi,
Thanks for raising this discussion! So, the hubness phenomena appears for retrieval models. In order to measure the performance, this is an important aspect relative to the testing split.
With the querybank, we are trying to estimate what videos are hubs, but in order to get an improvement in performance, we need to identify the potential hubs relative to the testing split. So, if there is a domain gap between the testing and the querybank it may lead to performance degradation depending on the normalization method used. We cover several cases in the paper in Table 2. There you can see that depending on what is the source of the queries from the querybank, there might be a degradation in performance. So, it is clear that the nature of the querybank has an impact on performance. However, we obtained a boost in performance on all dataset when we used all the queries from training.
I assume this domain gap appears in your case if using validation queries works, but using the training queries doesn't.
What happens if you use more/less queries from training? Are the sizes of the querybank comparable between training and validation? What happens if you combine validation and training? From our experience, it would benefit to have as many queries as possible in the querybank in order to mitigate this domain shift aspect.
Also, the beta parameter might influence the performance, so maybe it is worth trying to validate it on a separate split. CLIP4Clip scales the final similarity matrix (not sure if this is the case for you as well), so try using smaller beta values (in range of 0-2).
P.S. You can also find a more detailed discussion about hubness and QB-Norm here https://www.youtube.com/watch?v=KSgmKIbyA8M&t=3s.
Thanks!

Hi,
Thanks for your explanation. I will try the above suggestions as you said.
I believe a domain gap exists between the train and set, but it may not be the main reason. This is because we both utilize similar architecture, such as CLIP4Clip. I notice that CLIP2Video uses NetVLAD, a clustering method. Meanwhile, QB-Norm uses CLIP4Clip with a temporal transformer, including self-attention modules.
My experiments use mean-pooling to extract video-level features without extra clustering methods or attention mechanisms. Is there any connection between the hubness problem and specific architecture design (e.g., dot production/clustering)? I will try QB-Norm with related designs.
Thanks for your advice and explanation again :)


Setline with animated_text don't work

Hi !
All is in the title when I have a line with animated_text (add with "addline"), it is impossible to change it with "setline", I need to remove it and "addline" again I am in 1.8.9 and holograms 2.9.1-SNAPSHOT
Best Regards,
Downerr, Feloria Owner

Fixed in 2c3f9225


@conda-forge-admin, please add user @xylar

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added user @xylar in conda-forge/google-cloud-runtimeconfig-feedstock#14.


add new format hh:mm

i have added the functionality requested by the issue #3, also added the tests and they all work properly, and also updated the README.md file.
check it out!

Thank you for the well-detailed feedback. i'll start working on it now.

@Goldob  hello again, and thank you for taking the time to review my changes:
i added the backward compatibility to all the code and all initial tests are working, also the new 10 tests that i added just to make sure everything is working in both directions.
i hope you like everything and if there's anything just let me know and i'll fix it.

Looks good at first glance. I'll look deeper into the code some time later and let you know.

I think i can do that. I'll give it a go and let you know. Thanks

@Goldob hey again, i made the necessary change and kept all your test and my pervious tests and all of them are passing and also i added a few tests for the symmetrical conversions just like your example above. take a look at it, hope you like it. if anything is required please let me know. thanks

My pleasure, that would be great. If you have any thing else for me let me know.


Unable to access GVL instance

When attempting to access a GVL instance (ie. launched with Cloudlaunch) any attempt to access the API via BioBlend is redirected to the keycloak login page.

Problem was caused by a missing '/' (slash) at the end of the Galaxy server URL.


Fix ILI's right content

Fixes #212

Changes proposed in this Pull Request:

Added align items center to ILI's right content

Screenshots:

Works like a charm!

Works like a charm!


Significantly increase contrast for the dark theme.

Establish separate, exclusive styles for the code editor in dark mode.
Adjust dark mode editor colors to maintain >= 8.0 contrast ratio (most colors are >= 10).
Change the material primary color for embedded dark mode to a much lighter blue.
Ligten the output label color.

Main UI before:

Main UI after:

Embedded UI before:

Embedded UI after:

Fixes #1628

@mit-mit if you have a chance to take a look, that'd be awesome. Perhaps the most striking change is the switch from dark blue to cyan in the embedded view. I can shift that back towards the blues in the Flutter logo if you'd like.

The change looks good to me! Thank you!

Thanks Zoey!

Sorry, did catch this as I was out on vacation last week. Did we run this change by our designer for review?

I personally find it a bit odd looking that the background has different colors for the panes on the right and left. Why not use the same background color?

I would like to put this on hold until we can get a recommendation from our designer: https://github.com/dart-lang/dart-pad/pull/1774

How long will that take? I would prefer not to hold back an
accessibility improvement while we decide what color the bikeshed should be.
If it's going to take more than a week, I'd prefer if we go ahead and
deploy it. It's just as easy to make an update later.
On Tue, Feb 23, 2021 at 2:45 PM John Ryan<EMAIL_ADDRESS>wrote:

I would like to put this on hold until we can get a recommendation from
our designer: #1774 https://github.com/dart-lang/dart-pad/pull/1774
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/dart-pad/pull/1767#issuecomment-784464034,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB5I43KENRZCSLTDQ2HF3TTAQAXFANCNFSM4XXIYKVQ
.

And I'd like to have the designer who created the design actually validate the design change against the base design (which was created with accessibility in mind) before we deploy any changes.

It sounds like he will look at this next week.

I am not okay with this. I will be following up once I'm back in the office.

Sounds good. Having a designer to take a look sounds like a safer approach!
Thanks,
Zoey
On Tue, Feb 23, 2021 at 2:37 PM Stargirl Flowers<EMAIL_ADDRESS>wrote:

I am not okay with this. I will be following up once I'm back in the
office.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/dart-pad/pull/1767#issuecomment-784563415,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMR3232M56BMJVYDOHXKQ4TTAQUZ7ANCNFSM4XXIYKVQ
.

Apologies if my last comment was dismissive. I'm 100% supportive of resolving the problem raised by this PR. It looks some of our color combinations don't meet accessibility guidelines, and we should fix that.
My pushback was on the process. I'm against one-off design changes that don't consult with our designer. All I'm asking is that we go back and get additional feedback. For example, we may need to pick slightly different color combinations to stay consistent with other Dart sites. And ss part of that process we'll also consider if those related Dart sites need similar changes.

We're in an odd situation here with Mit OOO last week and Thea OOO now. Accessibility and great design aren't mutually exclusive, and I feel pretty confident we can arrive at a finished product that works for both, so I'd like to postpone any actual decision-making until everyone that wants to have a voice can have it.
In the meantime, though, review is great, thinking is great, and if this PR ends up inspiring better accessibility across Dart's properties, that'd be a great outcome.

FWIW, Material Design has a tool for evaluating accessibility of color schemes here: https://material.io/resources/color/#!/?view.left=1&view.right=0
Maybe this tool can help us evaluate our options and also stay in compliance with Material Design.


Add all-requirements.txt

Should include all "optional" modules as well. Things that some modules try to import and will fail or fallback in case of failure.

try pip install fonttools[all] or pip install -e .[all] (for local, "editable" installs)


Parsing the ICD-10-PCS codes from Excel into JSON

Create and save JSON objects of all data from the sheet "ALL 2022 ICD-10-PCS CODES" in the shared excel file.

First add the ICD10 Equivalent code and then parse the whole sheet

Just enter the ICD-10 codes. Enter the codes along with procedure name. Search on the basis of both. Include multiple selection as well.

https://docs.google.com/spreadsheets/d/1JlNXmlUtGO8Fs8bY-iLSaZgM2FVDfGmOBZz5ZV5_J44/edit?usp=sharing

https://docs.google.com/spreadsheets/d/169rnJVPSOIVTgHbyYkyNnxluWBXZAm30Jzo7wtOCVnY/edit?usp=sharing

@HammadAli55 can you please parse the codes from the docs to the mongo collection?


permission level increase to access custom yolo weight

custom_yolo.wt file permission requested to @kitrak-rev

Hi,
I have updated the access permissions. You should be able to download the weight file now.


Correct name of ignored directory

In .gitignore, /target/ notation just ignores openrr/target, however it does not other target directories in other hierarchies, such as openrr/arci/target. This PR fixes the problem.

I don't think that arci/target is generated, because we uses workspace.
All binaries should be located in target/ even if you cargo build in arci.
I want to know why you are in such situation.

I don't think that arci/target is generated, because we uses workspace.
All binaries should be located in target/ even if you cargo build in arci.
I want to know why you are in such situation.

It seems that target or target/ is more popular.
https://github.com/rust-lang/cargo/blob/master/.gitignore
https://github.com/rust-lang/futures-rs/blob/master/.gitignore
https://github.com/crossbeam-rs/crossbeam/blob/master/.gitignore

It seems that target or target/ is more popular.
https://github.com/rust-lang/cargo/blob/master/.gitignore
https://github.com/rust-lang/futures-rs/blob/master/.gitignore
https://github.com/crossbeam-rs/crossbeam/blob/master/.gitignore

Thanks!

Thanks!


Fix for PR #653 : add back 1 line that got accidentally removed

This line of code got accidentally deleted in PR #653, and since it's in the untested "#ifdef ALLOW_GGL90_HORIZDIFF" part this did not get caught in testreport.
What changes does this PR introduce?
bug fix
What is the current behaviour?
ggl90_calc.F does not compile anymore when ALLOW_GGL90_HORIZDIFF is defined.
What is the new behaviour
fixed
Does this PR introduce a breaking change?
Other information:
Suggested addition to tag-index
not needed if merged right after PR #653 (almost part of it)

Going also to do minor cleaning in ggl90_calc_diff.F and ggl90_calc_visc.F

@mjlosch Thanks. Going to merge this PR very soon.


Improve look-and-feel of notifications in Gradle AEM Trailer

Could you pls make the notifications in Gradle AEM Trailer to look better? The font on those is very small and it is hard to read the message.
Could you also make the name of the machine more prominent in the notification? I would like to be able to tell where the error happened right away I see the message.
Btw, what would you say about having configuration option telling how long the notification should be visible?

about name: https://github.com/Cognifide/gradle-aem-plugin#instance-conventions
if you would like to insert any name here, I could introduce new property label or displayName (instance definitions) in which you could put any name you want to be displayed here... or you could propose, what instead of that convention name should be displayed. I have no idea right now ;)
about font size, no problem, it will be increased.  I agree completely

@jakubprzybytek I improved (in my opinion :-)) look and feel of notifications. They are still dark, but the text has a much better contrast. What do you think?

I also tried to enlarge the size of the notification, but that would require forking the library we use of using a different one - a lot more effort.


#2203 issue fix

Description
#2203 issue fix ,  onBoardingViewController second next scene pushed only once now
Screenshots

@nickguendling  hi nick this is my first PR to open source project why  it's keep telling me
build-and-test Expected — Waiting for status to be reported ?

@mozead1996 Can you update the PR so that it matches the current HEAD ("Update Branch"), maybe the check will run then.

Weird afaik the checks should run instantly...
Circleci has a minor service outage, but macOS shouldn't be affected...

Thanx for the support - we will review the bugfix

Internal Tracking ID: EXPOSUREAPP-5771


gnome-user-docs: 3.38.1 -> 3.38.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for gnome-user-docs is: "User and system administration help for the GNOME desktop"
meta.homepage for gnome-user-docs is: "https://help.gnome.org/users/gnome-help/"
meta.changelog for gnome-user-docs is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

directory tree listing: https://gist.github.com/a19e0fb40055bebaabcc89459a0ef1dc

du listing: https://gist.github.com/1bbfbbba31fa86b0cf3257b4dea2d5f2

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
gnome-user-docs
gnome3.gnome-user-docs

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fi1mxvs7hzlwnjhmndamsjhgp7l24s1y-gnome-user-docs-3.38.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gnome-user-docs https://github.com/r-ryantm/nixpkgs/archive/e6584071eaa81aec150d8ea86880592e0c6af647.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fi1mxvs7hzlwnjhmndamsjhgp7l24s1y-gnome-user-docs-3.38.2
ls -la /nix/store/fi1mxvs7hzlwnjhmndamsjhgp7l24s1y-gnome-user-docs-3.38.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package built:

gnome-user-docs (gnome_user_docs)

Maintainer pings
cc @hedning @jtojnar @worldofpeace @dasj19 for testing.

Changes look okay:

3.38.2

Improvements to docs for Contacts (Pranali Deshmukh)
Improvements to docs for Files (Andre Klapper)
Updates to docs for background settings (Ivan Stanton)
Updated translations:
ca    (Jordi Mas)
cs    (Marek Černocký)
fa    (Danial Behzadi)
fi    (Jiri Grönroos)
id    (Andika Triwidada)
pl    (Piotr Drąg)
sv    (Anders Jonsson)


Rewrote ZOrder implementation for speed & glory

The old ZOrder implementation was very expensive. It involved keeping a sorted list up to date at all times, and a lot of code sites depended on the ZOrder API when what they really wanted was just to iterate over the list of children (independent of ZOrder).
This new implementation:

cleans up the code base to use the correct/intended APIs instead of the ZOrder ones.
reimplements ZOrder completely to be a lazy-evaluated and cached list, which is only sorted if needed
adds test for the crucial OnZOrderInvalidated callback which is needed by ITreeRenderer
makes sure ITreeRenderer doesn't dispatch zorder change on every event, but deferrs until the end of frame

This PR contains:

[x] Changelog
[-] Documentation - no relveant public APIs introduced or changed
[x] Tests

This failure seems relevant:
FAILED:  AllTests.Fuse.Controls.Test.PanelTest.ZOrder
         Assertion failed in 'ZOrder', at /Users/travis/build/fusetools/fuselibs-public/Source/Fuse.Controls/Tests/Panel.Test.uno:308
           Expected: Fuse.Controls.Panel, Name: B2
           But got : Fuse.Controls.Panel, Name: B1

On it

Hm, weird @kusma , that test doesn't fail locally...

Not able to reproduce this locally even when running all tests. Retrigger @kusma  ?

Curious. The test passes locally, AND on AppVeyor, but fails on Travis..

@Duckers: Travis is on macOS, and run AllTests.unoproj rather than the tests individually. Perhaps that sheds some light on the problem?

@kusma : Unable to reproduce it on my mac either :/

Hum! I was able to reproduce it now. It seems to be discrepencies in how Array.Sort works in .NET vs C++

Tests were failing because the .NET and C++ implementations of Array.Sort coped differently with items with the equal sorting value. The previous implementation of ZOrder computed the natural Z-Order before sorting to disambiguate.
Fixed in latest commit.

@Duckers: Could you please squash that last commit into cb02d46? You being frustrated during development isn't useful information for the future ;)

@kusma: done

@kusma: done

Yay! Finally green!


How to achieve the same level of accuracy as in the paper?

Thank you for sharing your code.
I have a question about the evaluation result of DiLiGenT dataset in the paper.
I have trained from scratch on the DiLiGenT dataset and have not been able to achieve the same level of accuracy as the paper. However, I used the weights you shared and am getting the same accuracy results as the paper. Are the settings in this repository different from the ones used to obtain the results in Table 3 of the paper?
Also, do you have any advice on how to obtain the results in Table 3 of the paper?
These are the normal estimation results I got.

ball
bear
buddha
cat
cow
goblet
harvest
Pot1
pot2
reading
avg

1.62
3.99
9.28
7.06
6.43
9.22
15.21
9.00
6.38
11.67
7.99

Thank you.

I encountered the same problem as you and my normal estimation results were:
nml_mae1.684 3.859 9.584 7.217 6.436 9.056 14.989 9.161 6.285 11.604 7.988(avg)
Have you solved it?

No, I haven’t. I tried adjusting various parameters but could not reproduce the results.


feat: add "register here" button for workshops on the Initiatives page (resolves #441)

[X] This pull request has been linted by running npm run lint without errors
[X] This pull request has been tested by running npm run start and reviewing affected routes
[X] This pull request has been built by running npm run build without errors
[X] This isn't a duplicate of an existing pull request

Description
Add "register here" button for workshops on the Initiatives page. The design for it can be found in the figma
Steps to test

Check Airtable "wecount_dev" base. In the "workshops" table, some workshops have URLs defined in "registration_url" field, some don't have;
Go to the Initiatives page, workshops have "registration_url" defined in Airtable have "register here" buttons displayed. The style of this button should match the design;
Test with UIO and the button should respond correctly.

Expected behavior:
See above.

Access the netlify deploy preview for this pull request hits "access denied" error.
Re-issue the PR to test if this is an one time issue with Netlify.


Stop tracking Cargo.lock in git

Why are we tracking Cargo.lock in git? I'm questioning this for the same reason as in #4549. It's creating conflicts in PRs although the severity is not as bad as there

Here is general explanation why tracking Cargo.lock might be helpful.


[camera] Camera viewer is over exposed as a result images taken w/ flash mode come out washed out on Samsung devices with Android 11

When trying to take a picture with the FlashMode set to auto or always, the image is over exposed causing everything to be extremely washed out.
I have set the ExposureMode and FocusMode to auto (which it defaults to as well) with no luck.
This issue was not present on my Samsung S9 running Android 10 or on iOS
The issue appears to be on Android 11 on my Samsung S21 or my other test device Samsung S10
Steps to Reproduce

Initialize the camera
Call the takePicture() method
View the image
Image is over exposed

Preview

Logs

Logs

[✓] Flutter (Channel stable, 2.2.3, on macOS 11.5.1 20G80 darwin-x64, locale en-US)
    • Flutter version 2.2.3 at /Users/anthony/fvm/versions/stable
    • Framework revision f4abaa0735 (6 weeks ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0-rc3)
    • Android SDK at /Users/anthony/Library/Android/sdk
    • Platform android-30, build-tools 31.0.0-rc3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5.1, Build version 12E507
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] VS Code (version 1.58.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.25.0

[✓] Connected device (2 available)
    • SM G991U1 (mobile) • R5CR717Z9SZ • android-arm64  • Android 11 (API 30)
    • Chrome (web)       • chrome      • web-javascript • Google Chrome 92.0.4515.131

• No issues found!

Hi @anthonycastelli
Thanks for filing the issue, I can't reproduce the issue on a Samsung device running Android 11 (Unfortunately I don't have a Samsung device with Android 10). Not only when making pictures, but just camera viewer without taking a photo also looks over exposed on Samsung or it shifts expose drastically when moving the camera between different objects.
Comparison

Version
camera: ^0.8.1+7
Code Sample
https://pub.dev/packages/camera/example

stable
master

❌
❌

Check flutter doctor -v outputs for each channel below

flutter doctor -v
[✓] Flutter (Channel stable, 2.2.3, on macOS 12.0 21A5294g darwin-x64, locale en-GB)
    • Flutter version 2.2.3 at /Users/tahatesser/Code/flutter_stable
    • Framework revision f4abaa0735 (6 weeks ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/tahatesser/Code/SDK
    • Platform android-31, build-tools 30.0.3
    • ANDROID_HOME = /Users/tahatesser/Code/SDK
    • Java binary at: /Users/tahatesser/Code/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment Temurin-11.0.12+7 (build 11.0.12+7)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode-beta.app/Contents/Developer
    • Xcode 13.0, Build version 13A5212g
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (version 2020.3)
    • Android Studio at /Users/tahatesser/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7583922/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to find bundled Java version.
    • Try updating or re-installing Android Studio.

[✓] VS Code (version 1.59.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.25.0

[✓] Connected device (3 available)
    • SM M025F (mobile) • R9ZR205XX0A • android-arm    • Android 11 (API 30)
    • macOS (desktop)   • macos       • darwin-x64     • macOS 12.0 21A5294g darwin-x64
    • Chrome (web)      • chrome      • web-javascript • Google Chrome 92.0.4515.131

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 2.5.0-6.0.pre.63, on macOS 12.0 21A5294g darwin-x64, locale en-GB)
    • Flutter version 2.5.0-6.0.pre.63 at /Users/tahatesser/Code/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 047d35d8ba (7 hours ago), 2021-08-12 12:47:05 +0800
    • Engine revision a2e60472a9
    • Dart version 2.15.0 (build 2.15.0-5.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/tahatesser/Code/SDK
    • Platform android-31, build-tools 30.0.3
    • ANDROID_HOME = /Users/tahatesser/Code/SDK
    • Java binary at: /Users/tahatesser/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7583922/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode-beta.app/Contents/Developer
    • Xcode 13.0, Build version 13A5212g
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Users/tahatesser/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7583922/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] Android Studio (version 2020.3)
    • Android Studio at /Users/tahatesser/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7583922/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] VS Code (version 1.59.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.25.0

[✓] Connected device (3 available)
    • SM M025F (mobile) • R9ZR205XX0A • android-arm    • Android 11 (API 30)
    • macOS (desktop)   • macos       • darwin-x64     • macOS 12.0 21A5294g darwin-x64
    • Chrome (web)      • chrome      • web-javascript • Google Chrome 92.0.4515.131

• No issues found!

✅: No Issue ❌: Issue reproduced

@TahaTesser If you set the flash to Always, does it get extremely washed out for you too? I can put together an example project if needed as well.

@anthonycastelli
I just borrowed Samsung A6 (Android 10), it doesn't look washed out

@anthonycastelli
Yes, always on Flash but as I said overall camera viewer is already over-exposed and since I reproduce the issue using an official sample and clearly Android 11 results matches your result, I don't think we need a project from you.

@TahaTesser Okay. Perfect. Glad it can be reproduced. Just wanted to get all of the info to you that you needed.
This is a bit of a blocker for us so if any workaround or fix is discovered, definitely let me know!

@TahaTesser Just wanted to circle back and see if there was any progress on this?

@TahaTesser did you resolved this? i have the same problem with exposed images with auto/always flash mode, i played it in a samsung galaxy note 10 with android 11 with ^0.8.1+7 version, with latest camera version the flash is get in trouble.

@anthonycastelli hello!
Tell please:

Did u solve this problem?
Looks like I have the same problem yea?

@anthonycastelli @TahaTesser
Hey yo!
Tell please:

Is there a solution to this problem now?
Looks like I have the same problem yea?

@avmax No, I did not resolve it. For the app I was building, I did find a work around, and that was to turn the flash on torch mode vs auto mode for Android. This served the correct purpose I needed, but I know this isn't something everyone can do.

@anthonycastelli thank you for answer and workaround! Tried the same this night :D
Did u add delay for any milliseconds between switching the torch on and making a photoshot?

@avmax I did yes. I think I used half a second or something, but this was all triggered on the initState method.

I can't believe this issue exists for over a year now and nothing has been done to fix it!
Do people realize that this bug together with this issue makes Flutter effectively unusable for apps with a camera feature with a bigger audience?
I can't put the camera on torch mode. I am working on a serious business app with thousands of users. How am I supposed to make it work on Android? Is there any workaround?

@anthonycastelli @avmax
I guess I fixed the bug. Can you guys confirm by putting the following lines in the pubspec.yaml of your project:
camera:
  git:
    url: git@github.com:gerken-tss/plugins.git
    ref: issue-88070-overexposure
    path: packages/camera/camera

If you can confirm, I will create a PR.
Thank you in advance!

This is same place. Taken with camera app and with my app(with flash mode)
Device: Google Pixel 6a, Version: Android 13