Can pydot parse the dot file generated by joern

I am trying to use pydot to parse the dot file generated by joern, but I encounter following errors.
The graph content:
digraph ":program" {
"3" [label = <(METHOD,:program)1> ]
"74" [label = <(METHOD,anonymous)9> ]
"60" [label = <(METHOD,anonymous)11> ]
"61" [label = <(PARAM,this)11> ]
"62" [label = <(PARAM,err)11> ]
"63" [label = <(PARAM,stdout)11> ]
"64" [label = <(PARAM,stderr)11> ]
"65" [label = <(BLOCK,{\012    console.log(stdout);\012  },{\012    console.log(stdout);\012  })11> ]
"66" [label = <(BLOCK,{\012    console.log(stdout);\012  },{\012    console.log(stdout);\012  })11> ]
"67" [label = <(log,console.log(stdout))12> ]
"58" [label = <(<operator>.fieldAccess,console.log)12> ]
"57" [label = <(IDENTIFIER,console,console.log(stdout))12> ]
"59" [label = <(FIELD_IDENTIFIER,log,log)12> ]
"68" [label = <(IDENTIFIER,console,console.log(stdout))12> ]
"69" [label = <(IDENTIFIER,stdout,console.log(stdout))12> ]
"109" [label = <(LOCAL,console: ANY)> ]
"70" [label = <(MODIFIER,VIRTUAL)> ]
"71" [label = <(METHOD_RETURN,ANY)11> ]
"75" [label = <(PARAM,this)9> ]
"76" [label = <(BLOCK,{\012  console.log('running me');\012  exec('npx hardhat compile', (err, stdout, stderr) =>{\012    console.log(stdout);\012  });  \012},{\012  console.log('running me');\012  exec('npx hardhat compile', (err, stdout, stderr) =>{\012    console.log(stdout);\012  });  \012})9> ]
"77" [label = <(BLOCK,{\012  console.log('running me');\012  exec('npx hardhat compile', (err, stdout, stderr) =>{\012    console.log(stdout);\012  });  \012},{\012  console.log('running me');\012  exec('npx hardhat compile', (err, stdout, stderr) =>{\012    console.log(stdout);\012  });  \012})9> ]
"78" [label = <(log,console.log('running me'))10> ]
"54" [label = <(<operator>.fieldAccess,console.log)10> ]
"53" [label = <(IDENTIFIER,console,console.log('running me'))10> ]
"55" [label = <(FIELD_IDENTIFIER,log,log)10> ]
"79" [label = <(IDENTIFIER,console,console.log('running me'))10> ]
"80" [label = <(LITERAL,"running me",console.log('running me'))10> ]
"81" [label = <(exec,exec('npx hardhat compile', (err, stdout, stderr) =>{\012    console.log(stdout);\012  }))11> ]
"56" [label = <(IDENTIFIER,exec,exec('npx hardhat compile', (err, stdout, stderr) =>{\012    console.log(stdout);\012  }))11> ]
"82" [label = <(IDENTIFIER,this,exec('npx hardhat compile', (err, stdout, stderr) =>{\012    console.log(stdout);\012  }))11> ]
"83" [label = <(LITERAL,"npx hardhat compile",exec('npx hardhat compile', (err, stdout, stderr) =>{\012    console.log(stdout);\012  }))11> ]
"84" [label = <(METHOD_REF,anonymous,exec('npx hardhat compile', (err, stdout, stderr) =>{\012    console.log(stdout);\012  }))11> ]
"110" [label = <(LOCAL,exec: ANY)> ]
"111" [label = <(LOCAL,console: ANY)> ]
"85" [label = <(MODIFIER,VIRTUAL)> ]
"86" [label = <(METHOD_RETURN,ANY)9> ]
"72" [label = <(TYPE_DECL,anonymous)> ]
"93" [label = <(PARAM,this)1> ]
"4" [label = <(BLOCK,<empty>,<empty>)> ]
"5" [label = <(LOCAL,{ Command }: ANY)> ]
"19" [label = <(LOCAL,program: ANY)> ]
"30" [label = <(LOCAL,{ exec }: ANY)> ]
"10" [label = <(BLOCK,{ Command },{ Command })3> ]
"11" [label = <(LOCAL,_tmp_0: ANY)> ]
"94" [label = <(<operator>.assignment,_tmp_0 = require('commander'))3> ]
"95" [label = <(IDENTIFIER,_tmp_0,_tmp_0 = require('commander'))3> ]
"12" [label = <(require,require('commander'))3> ]
"9" [label = <(IDENTIFIER,require,require('commander'))3> ]
"13" [label = <(IDENTIFIER,this,require('commander'))3> ]
"14" [label = <(LITERAL,"commander",require('commander'))3> ]
"96" [label = <(<operator>.assignment,Command = _tmp_0.Command)3> ]
"15" [label = <(IDENTIFIER,Command,Command = _tmp_0.Command)3> ]
"16" [label = <(<operator>.fieldAccess,_tmp_0.Command)3> ]
"17" [label = <(IDENTIFIER,_tmp_0,Command = _tmp_0.Command)3> ]
"18" [label = <(FIELD_IDENTIFIER,Command,Command)3> ]
"97" [label = <(IDENTIFIER,_tmp_0,{ Command })3> ]
"98" [label = <(<operator>.assignment,const program = new Command())4> ]
"23" [label = <(IDENTIFIER,program,const program = new Command())4> ]
"20" [label = <(BLOCK,new Command(),new Command())4> ]
"21" [label = <(LOCAL,_tmp_1: ANY)> ]
"24" [label = <(<operator>.assignment,_tmp_1 = .alloc)4> ]
"25" [label = <(IDENTIFIER,_tmp_1,_tmp_1 = .alloc)4> ]
"26" [label = <(<operator>.alloc,.alloc)4> ]
"27" [label = <(<operator>.new,new Command())4> ]
"22" [label = <(IDENTIFIER,Command,new Command())4> ]
"28" [label = <(IDENTIFIER,_tmp_1,new Command())4> ]
"29" [label = <(IDENTIFIER,_tmp_1,new Command())4> ]
"34" [label = <(BLOCK,{ exec },{ exec })5> ]
"35" [label = <(LOCAL,_tmp_2: ANY)> ]
"99" [label = <(<operator>.assignment,_tmp_2 = require('child_process'))5> ]
"100" [label = <(IDENTIFIER,_tmp_2,_tmp_2 = require('child_process'))5> ]
"36" [label = <(require,require('child_process'))5> ]
"33" [label = <(IDENTIFIER,require,require('child_process'))5> ]
"37" [label = <(IDENTIFIER,this,require('child_process'))5> ]
"38" [label = <(LITERAL,"child_process",require('child_process'))5> ]
"101" [label = <(<operator>.assignment,exec = _tmp_2.exec)5> ]
"39" [label = <(IDENTIFIER,exec,exec = _tmp_2.exec)5> ]
"40" [label = <(<operator>.fieldAccess,_tmp_2.exec)5> ]
"41" [label = <(IDENTIFIER,_tmp_2,exec = _tmp_2.exec)5> ]
"42" [label = <(FIELD_IDENTIFIER,exec,exec)5> ]
"102" [label = <(IDENTIFIER,_tmp_2,{ exec })5> ]
"103" [label = <(action,program\012.command('compile')\012.action(()=>{\012  console.log('running me');\012  exec('npx hardhat compile', (err, stdout, stderr) =>{\012    console.log(stdout);\012  });  \012}))7> ]
"51" [label = <(<operator>.fieldAccess,(_tmp_3 = program\012.command('compile')).action)7> ]
"49" [label = <(<operator>.assignment,(_tmp_3 = program\012.command('compile')))7> ]
"50" [label = <(IDENTIFIER,_tmp_3,(_tmp_3 = program\012.command('compile')))7> ]
"46" [label = <(command,program\012.command('compile'))7> ]
"44" [label = <(<operator>.fieldAccess,program.command)7> ]
"43" [label = <(IDENTIFIER,program,program\012.command('compile'))7> ]
"45" [label = <(FIELD_IDENTIFIER,command,command)8> ]
"47" [label = <(IDENTIFIER,program,program\012.command('compile'))7> ]
"48" [label = <(LITERAL,"compile",program\012.command('compile'))8> ]
"52" [label = <(FIELD_IDENTIFIER,action,action)9> ]
"104" [label = <(IDENTIFIER,_tmp_3,program\012.command('compile')\012.action(()=>{\012  console.log('running me');\012  exec('npx hardhat compile', (err, stdout, stderr) =>{\012    console.log(stdout);\012  });  \012}))7> ]
"105" [label = <(METHOD_REF,anonymous,program\012.command('compile')\012.action(()=>{\012  console.log('running me');\012  exec('npx hardhat compile', (err, stdout, stderr) =>{\012    console.log(stdout);\012  });  \012}))9> ]
"106" [label = <(parse,program.parse())16> ]
"90" [label = <(<operator>.fieldAccess,program.parse)16> ]
"89" [label = <(IDENTIFIER,program,program.parse())16> ]
"91" [label = <(FIELD_IDENTIFIER,parse,parse)16> ]
"107" [label = <(IDENTIFIER,program,program.parse())16> ]
"112" [label = <(LOCAL,_tmp_3: ANY)> ]
"113" [label = <(LOCAL,exec: ANY)> ]
"114" [label = <(LOCAL,require: ANY)> ]
"115" [label = <(LOCAL,Command: ANY)> ]
"108" [label = <(METHOD_RETURN,ANY)> ]
"87" [label = <(TYPE_DECL,anonymous)> ]
  "3" -> "93"  [ label = "AST: "]
  "3" -> "74"  [ label = "AST: "]
  "3" -> "87"  [ label = "AST: "]
  "3" -> "4"  [ label = "AST: "]
  "3" -> "108"  [ label = "AST: "]
  "74" -> "75"  [ label = "AST: "]
  "74" -> "60"  [ label = "AST: "]
  "74" -> "72"  [ label = "AST: "]
  "74" -> "76"  [ label = "AST: "]
  "74" -> "85"  [ label = "AST: "]
  "74" -> "86"  [ label = "AST: "]
  "60" -> "61"  [ label = "AST: "]
  "60" -> "62"  [ label = "AST: "]
  "60" -> "63"  [ label = "AST: "]
  "60" -> "64"  [ label = "AST: "]
  "60" -> "65"  [ label = "AST: "]
  "60" -> "70"  [ label = "AST: "]
  "60" -> "71"  [ label = "AST: "]
  "65" -> "109"  [ label = "AST: "]
  "65" -> "66"  [ label = "AST: "]
  "66" -> "67"  [ label = "AST: "]
  "67" -> "58"  [ label = "AST: "]
  "67" -> "68"  [ label = "AST: "]
  "67" -> "69"  [ label = "AST: "]
  "58" -> "57"  [ label = "AST: "]
  "58" -> "59"  [ label = "AST: "]
  "76" -> "110"  [ label = "AST: "]
  "76" -> "111"  [ label = "AST: "]
  "76" -> "77"  [ label = "AST: "]
  "77" -> "78"  [ label = "AST: "]
  "77" -> "81"  [ label = "AST: "]
  "78" -> "54"  [ label = "AST: "]
  "78" -> "79"  [ label = "AST: "]
  "78" -> "80"  [ label = "AST: "]
  "54" -> "53"  [ label = "AST: "]
  "54" -> "55"  [ label = "AST: "]
  "81" -> "56"  [ label = "AST: "]
  "81" -> "82"  [ label = "AST: "]
  "81" -> "83"  [ label = "AST: "]
  "81" -> "84"  [ label = "AST: "]
  "4" -> "5"  [ label = "AST: "]
  "4" -> "19"  [ label = "AST: "]
  "4" -> "30"  [ label = "AST: "]
  "4" -> "112"  [ label = "AST: "]
  "4" -> "113"  [ label = "AST: "]
  "4" -> "114"  [ label = "AST: "]
  "4" -> "115"  [ label = "AST: "]
  "4" -> "10"  [ label = "AST: "]
  "4" -> "98"  [ label = "AST: "]
  "4" -> "34"  [ label = "AST: "]
  "4" -> "103"  [ label = "AST: "]
  "4" -> "106"  [ label = "AST: "]
  "10" -> "11"  [ label = "AST: "]
  "10" -> "94"  [ label = "AST: "]
  "10" -> "96"  [ label = "AST: "]
  "10" -> "97"  [ label = "AST: "]
  "94" -> "95"  [ label = "AST: "]
  "94" -> "12"  [ label = "AST: "]
  "12" -> "9"  [ label = "AST: "]
  "12" -> "13"  [ label = "AST: "]
  "12" -> "14"  [ label = "AST: "]
  "96" -> "15"  [ label = "AST: "]
  "96" -> "16"  [ label = "AST: "]
  "16" -> "17"  [ label = "AST: "]
  "16" -> "18"  [ label = "AST: "]
  "98" -> "23"  [ label = "AST: "]
  "98" -> "20"  [ label = "AST: "]
  "20" -> "21"  [ label = "AST: "]
  "20" -> "24"  [ label = "AST: "]
  "20" -> "27"  [ label = "AST: "]
  "20" -> "29"  [ label = "AST: "]
  "24" -> "25"  [ label = "AST: "]
  "24" -> "26"  [ label = "AST: "]
  "27" -> "22"  [ label = "AST: "]
  "27" -> "28"  [ label = "AST: "]
  "34" -> "35"  [ label = "AST: "]
  "34" -> "99"  [ label = "AST: "]
  "34" -> "101"  [ label = "AST: "]
  "34" -> "102"  [ label = "AST: "]
  "99" -> "100"  [ label = "AST: "]
  "99" -> "36"  [ label = "AST: "]
  "36" -> "33"  [ label = "AST: "]
  "36" -> "37"  [ label = "AST: "]
  "36" -> "38"  [ label = "AST: "]
  "101" -> "39"  [ label = "AST: "]
  "101" -> "40"  [ label = "AST: "]
  "40" -> "41"  [ label = "AST: "]
  "40" -> "42"  [ label = "AST: "]
  "103" -> "51"  [ label = "AST: "]
  "103" -> "104"  [ label = "AST: "]
  "103" -> "105"  [ label = "AST: "]
  "51" -> "49"  [ label = "AST: "]
  "51" -> "52"  [ label = "AST: "]
  "49" -> "50"  [ label = "AST: "]
  "49" -> "46"  [ label = "AST: "]
  "46" -> "44"  [ label = "AST: "]
  "46" -> "47"  [ label = "AST: "]
  "46" -> "48"  [ label = "AST: "]
  "44" -> "43"  [ label = "AST: "]
  "44" -> "45"  [ label = "AST: "]
  "106" -> "90"  [ label = "AST: "]
  "106" -> "107"  [ label = "AST: "]
  "90" -> "89"  [ label = "AST: "]
  "90" -> "91"  [ label = "AST: "]
  "98" -> "36"  [ label = "CFG: "]
  "103" -> "91"  [ label = "CFG: "]
  "106" -> "108"  [ label = "CFG: "]
  "94" -> "18"  [ label = "CFG: "]
  "96" -> "26"  [ label = "CFG: "]
  "99" -> "42"  [ label = "CFG: "]
  "101" -> "45"  [ label = "CFG: "]
  "51" -> "105"  [ label = "CFG: "]
  "105" -> "103"  [ label = "CFG: "]
  "90" -> "106"  [ label = "CFG: "]
  "12" -> "94"  [ label = "CFG: "]
  "16" -> "96"  [ label = "CFG: "]
  "24" -> "27"  [ label = "CFG: "]
  "27" -> "98"  [ label = "CFG: "]
  "36" -> "99"  [ label = "CFG: "]
  "40" -> "101"  [ label = "CFG: "]
  "49" -> "52"  [ label = "CFG: "]
  "52" -> "51"  [ label = "CFG: "]
  "91" -> "90"  [ label = "CFG: "]
  "18" -> "16"  [ label = "CFG: "]
  "26" -> "24"  [ label = "CFG: "]
  "42" -> "40"  [ label = "CFG: "]
  "46" -> "49"  [ label = "CFG: "]
  "44" -> "46"  [ label = "CFG: "]
  "45" -> "44"  [ label = "CFG: "]
  "3" -> "12"  [ label = "CFG: "]
  "94" -> "108"  [ label = "DDG: _tmp_0"]
  "94" -> "108"  [ label = "DDG: require('commander')"]
  "96" -> "108"  [ label = "DDG: Command"]
  "96" -> "108"  [ label = "DDG: _tmp_0.Command"]
  "24" -> "108"  [ label = "DDG: .alloc"]
  "27" -> "108"  [ label = "DDG: _tmp_1"]
  "27" -> "108"  [ label = "DDG: new Command()"]
  "99" -> "108"  [ label = "DDG: _tmp_2"]
  "36" -> "108"  [ label = "DDG: this"]
  "99" -> "108"  [ label = "DDG: require('child_process')"]
  "101" -> "108"  [ label = "DDG: exec"]
  "101" -> "108"  [ label = "DDG: _tmp_2.exec"]
  "49" -> "108"  [ label = "DDG: program\012.command('compile')"]
  "103" -> "108"  [ label = "DDG: _tmp_3"]
  "103" -> "108"  [ label = "DDG: program\012.command('compile')\012.action(()=>{\012  console.log('running me');\012  exec('npx hardhat compile', (err, stdout, stderr) =>{\012    console.log(stdout);\012  });  \012})"]
  "106" -> "108"  [ label = "DDG: program"]
  "106" -> "108"  [ label = "DDG: program.parse()"]
  "3" -> "93"  [ label = "DDG: "]
  "3" -> "10"  [ label = "DDG: "]
  "27" -> "98"  [ label = "DDG: new Command()"]
  "27" -> "98"  [ label = "DDG: _tmp_1"]
  "3" -> "98"  [ label = "DDG: "]
  "3" -> "34"  [ label = "DDG: "]
  "12" -> "94"  [ label = "DDG: "commander""]
  "12" -> "94"  [ label = "DDG: this"]
  "94" -> "96"  [ label = "DDG: _tmp_0"]
  "3" -> "97"  [ label = "DDG: "]
  "36" -> "99"  [ label = "DDG: "child_process""]
  "36" -> "99"  [ label = "DDG: this"]
  "99" -> "101"  [ label = "DDG: _tmp_2"]
  "3" -> "102"  [ label = "DDG: "]
  "49" -> "103"  [ label = "DDG: _tmp_3"]
  "3" -> "103"  [ label = "DDG: "]
  "46" -> "106"  [ label = "DDG: program"]
  "3" -> "106"  [ label = "DDG: "]
  "3" -> "24"  [ label = "DDG: "]
  "3" -> "29"  [ label = "DDG: "]
  "3" -> "9"  [ label = "DDG: "]
  "93" -> "12"  [ label = "DDG: this"]
  "3" -> "12"  [ label = "DDG: "]
  "3" -> "22"  [ label = "DDG: "]
  "24" -> "27"  [ label = "DDG: _tmp_1"]
  "3" -> "27"  [ label = "DDG: "]
  "3" -> "33"  [ label = "DDG: "]
  "12" -> "36"  [ label = "DDG: this"]
  "3" -> "36"  [ label = "DDG: "]
  "46" -> "49"  [ label = "DDG: "compile""]
  "46" -> "49"  [ label = "DDG: program"]
  "98" -> "46"  [ label = "DDG: program"]
  "3" -> "46"  [ label = "DDG: "]
}

The error information:
(TypeError)can only concatenate str (not "ParseException") to str

Hello, I was also looking for this solution. And I found that the latest version on master branch has fixed this bug yet.
However, the package on pypi was still not updated since 2021.
I think pulling the latest master branch, and pack up your own latest wheel will solve this problem.

@HJX-zhanS Hello, the newest version 2.0.0 has been released, please try this version. I have tested that, and this graph was parsed successfully.


Add Windows hostprocess support

Add the correct specs fields to https://github.com/containerd/nerdctl/blob/f316402913c4239a2388b7467bcffc9735dd8327/cmd/nerdctl/run_windows.go#L42
Something like:
hpc, err := cmd.Flags().GetBool("host-process")
if err != nil {
  return nil, err
}
if hpc {
  //TODO append the right hpc opts
}

See https://github.com/containerd/containerd/pull/5131 for details on how to set the right spec fields.

Planning to look at this next week

@AkihiroSuda What's the story for how certain things should be handled if they require a specific Containerd version? hostprocess (Windows' version of a privileged container) will be a feature for Containerd 1.6 and currently only works in the beta releases until a full release.

What's the story for how certain things should be handled if they require a specific Containerd version?

Just mentioning that in docs/windows.md would be fine

Ok awesome, thanks!


Code annotations scottbrabson

Annotations inside of the files will be between 💻  these bad boys 💻 - for issue #23
Files:
/web/src/modules/user/Profile.js

Add File component to choose which profile image to upload and button to submit (or submit on selection)
Add profile description field
Add shipping address field
BACKEND (NEW) need to work with BE on what the API contract will look like for new user properties: user image, user description, user shipping address
BACKEND (UPDATE) need to work with BE to implement PATCH for: user email, user shipping address, user description, user image

/web/src/modules/user/api/actions.js

setUser action to include the new image, description, delivery address, delivery date fields that will be added to user profile

/web/src/modules/user/History.js (NEW)

Similar to the implementation of Subscriptions.js under the User component
will call getHistoryByUser, dispatches action from inside the history component (web/src/modules/history.js)

/web/src/modules/common/header/Header.js

need new MenuItem (rendered after auth) to direct user to user's order history/details (does not exist yet, will be NEW component(s))

/web/src/modules/history.js (NEW)

to follow convention, we may want to copy the format of the subscriptions or crates files/component structure
on mount, a REQUEST action is dispatched (from /web/src/modules/history/api/action.js), is in loading state until receiving response, then RESPONSE action is dispatched
BACKEND (NEW) need to work with BE to create new properties for subscriptions/crates/users to create relationships between data to display meaningful history
Items can only have a delivery date that falls within the user's desired delivery schedule

Closing for backlog grooming. Thanks for your annotations! Don't forget to submit a link to your branch if you haven't already here


How is loss (and average loss) computed in Canned estimators?

Documentation only specifies:

loss (mean loss per mini-batch) and the average_loss (mean loss per sample)

I understand that this might seem like enough, but since there are several, very different algorithms for computing loss (absolute, relative, square etc.), this can make a big difference.
Especially if you are comparing your custom estimator, with canned alternatives.
For example, I make my own DNN classifier, I run it and get eval dict with keys loss, accuracy, and global_step.  But does the loss key represent the same thing it does in canned equivalent? And even if it does, which algorithm I should pick if I want to compute average loss?
If you are using the canned estimators as a baseline, and many beginners do, then making wrong assumptions about metrics is the worst possible option.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce
Mobile device

This is a deficiency in documentation, not a bug report.

Closing this, feel free to reopen if running into any errors.


#ERROR# TypeError: Cannot read property 'id' of undefined

See this run: https://github.com/dotnet/upgrade-assistant/actions/runs/3093629638/jobs/5006207314#step:2:8
Not really sure what's going wrong

Also seeing the same thing: https://github.com/ecadlabs/taqueria/actions/runs/3285952561
Workflow config:
name: Add new PR's to the scrum board

on:
  pull_request:
    types:
      - opened
      - ready_for_review

jobs:
  add-pr-to-taqueria-dev-project:
    name: Add PR to project
    runs-on: ubuntu-latest
    steps:
    - name: New PR
      if: ${{ !github.event.pull_request.ready_for_review }}
      uses<EMAIL_ADDRESS>      with:
        action-token: "${{ secrets.ADD_TO_PROJECT_PAT }}"
        project-url: "https://github.com/orgs/ecadlabs/projects/9"
        column-name: "🚀  Current Sprint"
    - name: PR Ready for review
      if: ${{ github.event.pull_request.ready_for_review }}
      uses<EMAIL_ADDRESS>      with:
        action-token: "${{ secrets.ADD_TO_PROJECT_PAT }}"
        project-url: "https://github.com/orgs/ecadlabs/projects/9"
        column-name: "👀 In review"


Dispatch test queue updates

In #53 we created functionality to add patients to a test queue, currently implemented in localstorage.
Once the backend in #90 is implemented, we will need to add functionality to dispatch this information from localstorage to the backend database.

Is this story- Save test queue to the cloud (like not just in the browser)?

This one is a technical task, I don't think it needs a story. It's an output from the story in #53.

Moving to 4 with the epic

Already covered!


Override sub dependency with my own

[x] I am on the latest Poetry version.
[x] I have searched the issues of this repo and believe that this is not a duplicate.
[x] If an exception occurs when executing a command, I executed it again in debug mode (-vvv option).

OS version and name:  MacOS
Poetry version: Poetry 0.11.5

Issue
My app is depending on the wsproto library. wsproto itself is depending on h11 >=0.7.0,<0.8.0.
It happens that h11 has a bug which I fixed; the h11 project has merged this but not yet published an update.
I'd like to therefore install the master branch of h11. I am doing this:
poetry add wsproto
poetry add --git https://github.com/miracle2k/h11.git h11

However, it seems that my "git overrride" is ignored:
$ poetry show
atomicwrites   1.2.1  Atomic file writes.
attrs          18.2.0 Classes Without Boilerplate
h11            0.7.0  A pure-Python, bring-your-own-I/O implementation of HTTP/1.1
more-itertools 4.3.0  More routines for operating on iterables, beyond itertools
pluggy         0.7.1  plugin and hook calling mechanisms for python
py             1.6.0  library with cross-python path, ini-parsing, io, code, log facilities
pytest         3.8.0  pytest: simple powerful testing with Python
six            1.11.0 Python 2 and 3 compatibility utilities
wsproto        0.11.0 WebSockets state-machine based protocol implementation

$ poetry run  pip freeze
atomicwrites==1.2.1
attrs==18.2.0
h11==0.7.0
more-itertools==4.3.0
pluggy==0.7.1
py==1.6.0
pytest==3.8.0
six==1.11.0
wsproto==0.11.0

If I poetry remove wsproto, it installs the correct git dependency and shows as the version h11==0.8.1+dev.
At the very least, I would expect here to be conflict: if wsproto requires a version <0.8, but I require a specific git install with a version >0.8, this should be unresolvable.
Ideally, I'd like a way to to ignore the overlay restrictive wsproto version range and install the version I tell it to.

This looks like the same issue as in #991 where poetry doesn't seem to have any idea what the version provided by a git repo is. The only workaround I've found so far (and it feels like a disappointing one to me) is to fork the package that depends on I've been forced to fork the problematic packages, add a pyproject.toml with the problematic relationship specified as <package_name> ="*" instead of the correct version specifier, and to use my fork as the source of the package.

Tracked in #991. Closing this issue.


ISC license?

I noticed some of your other related projects were MIT and wondered if it was an oversight here and here. Or maybe that one is the odd one out. Either way I figured I'd ask.

I default to ISC, but when forking or creating a derivative work of an MIT licensed project, it's more straightforward to use the same license.
Minizlib is derivative of code in node.js.  Minipass-fetch is a derivative work of node-fetch.  So those two are MIT-licensed, but everything else I do is usually ISC.


fix: ArgumentOutOfRangeException when no hits

ArgumentOutOfRangeException when no hits.
https://github.com/marijnz/unity-editor-spotlight/blob/master/Assets/Editor/EditorSpotlight.cs#L334

Good idea, I noticed this but too lazy to fix it. Thanks!


v0.18.0 Docker image has no viewer UI

As of the changes in #940/#941, the image that is pushed on release loads only the following HTML at /:

.keep

I'm guessing this is because the "Publish Docker container" step does not include context: ., and thus is working on the Git context, which does not include the local filesystem changes like the addition of ui/dist.

@jchamberlain thanks for the report. There's one other report of someone having issues with the viewer: https://github.com/go-spatial/tegola/issues/942

Thanks @ARolek, I just left a comment there. It appears to be an unrelated issue: the output of tegola version always says "ui viewer: viewer not build" even when it did build successfully.

To follow up on @jchamberlain report, the following step in the CI appears to be where the issue is happening:
https://github.com/go-spatial/tegola/blob/096b5d90328ed4b494bedf9aa780a5b065797122/.github/workflows/on_release_publish.yml#L209-L216
And the context @jchamberlain is talking about is documented here

@manastyretskyi can you take a quick look at this one? Seems like all we need to do is add context: . to the action.

@ARolek I will try to take a look at it later today


Implement Debug for most types

I've wrote an implementation of Debug for most types, but I can't figure out how to do FrameBufferResponse, since it gives an iterator and not a slice.
Also should it print out all the framebuffer modes? There can be quite a lot of them (65 in my qemu configuration).

Also, you'll have to rebase from #32.


Rearrange tabs in napari

Hey @qin-yu, I would like to propose the following changes.

merge the predictions and segmentation  in a single Extra workflows (to save space for the upcoming training/dataset tab)
remove the watershed in segmentation and add back the functionalities in Main
Always use the dropdown menu for Extra workflow (i.e., remove the TOO_MANY_WIDGES = False check). Having all the widgets open adds a lot of clutter to the UI.

merge the predictions and segmentation  in a single Extra workflows (to save space for the upcoming training/dataset tab)
remove the watershed in segmentation and add back the functionalities in Main

These changes doesn't really affect me and might be new-user-friendly.

Always use the dropdown menu for Extra workflow (i.e., remove the TOO_MANY_WIDGES = False check). Having all the widgets open adds a lot of clutter to the UI.

I would like to have easier access to all widgets for more direct feedback ("on_change" events) during development, I propose to have an environment variable to allow this (that's why I did a TOO_MANY_WIDGES, but didn't have time to finish)
The workflow for me to segment or to show new users how to segment using PlantSeg is the following:

(Add a model)
Load an image
(Gaussian blur)
Run prediction
Run watershed
Run filter (the one I developed)
Run GASP
Run filter
Export segmentation

So having "Data", "Main", and "Extra" doesn't add complexity for inference. I would say your proposal is nice overall.

Fixed by #249


http/1.1 "500 Internal Server Error ERROR:    Exception in ASGI application"

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
# import libraries
from fastapi import FastAPI
from pycaret.classification import *
import uvicorn
import pandas as pd
import warnings
warnings.filterwarnings('ignore')

# define the app object
app = FastAPI()

# Load the model
model = load_model('rf_class_heart_failure_model')

# define function

@app.post('/predict')
def predict(age, sex, chest_pain, r_blood_pressure, cholesterol, f_blood_sugar, r_ecg, max_heart_rate, exer_angina,
            oldpeak, slope_peak, major_vessels, thal):
    data = pd.DataFrame([[age, sex, chest_pain, r_blood_pressure, cholesterol, f_blood_sugar, r_ecg, max_heart_rate,
                          exer_angina, oldpeak, slope_peak, major_vessels, thal]])
    data.columns = ['age', 'sex', 'chest_pain', 'r_blood_pressure', 'cholesterol', 'f_blood_sugar', 'r_ecg',
                    'max_heart_rate', 'exer_angina', 'oldpeak', 'slope_peak', 'major_vessels', 'thal']

    predictions = predict(model, data=data)
    print(data)
    return {'prediction': int(predictions['Label'][0])}

if __name__ == '__main__':
    uvicorn.run(app, host='<IP_ADDRESS>', port=8000)

Description
I built a model which I want to deploy using fastapi, however when I run uvicorn main:app I get {"detail":"Not Found"} on <IP_ADDRESS>:8000, and internal server error when I try to execute from <IP_ADDRESS>:8000/docs.

Error: Internal Server ErrorResponse bodyDownloadInternal Server ErrorResponse headers content-length: 21  content-type: text/plain; charset=utf-8  date: Sat,20 Aug 2022 12:22:30 GMT  server: uvicorn

Operating System
Windows
Operating System Details
Am using Windows 11
FastAPI Version
(env) PS D:\New\pycaret> pip show fastapi Name: fastapi Version: 0.79.1 Summary: FastAPI framework, high performance, easy to learn, fast to code, ready for production Home-page: https://github.com/tiangolo/fastapi Author: Sebastián Ramírez Author-email<EMAIL_ADDRESS>License: Location: d:\new\pycaret\env\lib\site-packages Requires: pydantic, starlette Required-by:
Python Version
Python 3.8.8
Additional Context
No response

{"detail":"Not Found"}

is expected when you hit on <IP_ADDRESS>:8000. It is because you do not have any route to "/".

As of InternalServerError it is most likely propagated from one of your internal methods. You can try encapsulating them into a try block to find what is the error.
try:
    predict(...)
except Exception as e:
    print(e)

@iudeen this is the error am now getting.

TypeError: Failed to execute 'fetch' on 'Window': Failed to parse URL from http://<IP_ADDRESS>:8000predict?age=70&sex=1&chest_pain=4&r_blood_pressure=130&cholesterol=322&f_blood_sugar=0&r_ecg=2&max_heart_rate=109&exer_angina=0&oldpeak=2.4&slope_peak=2&major_vessels=3&thal=3

TypeError: Failed to execute 'fetch' on 'Window': Failed to parse URL from

This error appears to be coming from JS. You should be calling http://<IP_ADDRESS>:8000/predict?....

Your function is named "predict":
def predict(...

And you called "predict" in your "predict" function:
predictions = predict(model, data=data)

It calls itself and causes infinite loop.  You should solve it and check again.

conn.execute(oficina.insert().values(new_ofic))
AttributeError: 'APIRouter' object has no attribute 'insert'
At At record insert time it throws this error and doesn't insert anything into the table.
Your help please


v0.9.6 unable to change resolution in fullscreen mode

Since updating to v0.9.6 I'm unable to change the resolution in fullscreen mode to anything but 4k which is my desktop resolution. Also setting screen percentage to 50 in the INI to achieve 1440p doesn't have any effect.

Can you post your SMTVFix.log file please?

Hey Lyall, so the odd thing, today when I tried to recreate the bug to have a fresh log file ready for you, the game would correctly change resolutions again. I'm closing this issue for now. Thanks for your work!


Implement listbox control

Implement a listbox control to override game list management.
Provide the following features:

scrolling of items by arrows, page up / down, home / end
jump to a specific item by typing the first letters
highlighting of occurrences in items of a specific string
ability to hide items that do not match the string used for highlighting
ability to view items on multiple columns (useful for rendering a list of files)

The following remain to be implemented:

the display of only the items that match the string for highlighting
the arrangement on multiple columns

The items filtering is done into the class that handle programs list, so is not necessary. A clear method to empty the listbox it's enough.


hotel-booking-frontend: Finalize-Login

In this PR, I have done the following tasks.

Added redirect after login.
Created a logged-in state in the redux store.
Added Add validation for login component.

👋 @angeluray, That is not mine. That is Tadesse's implementation, Anyway I will solve it but there is no such type of errors in my local please pull the latest "register.jsx" it will fix the errors

On my side, it works just like this.


[REQ] Web dashboard

Plotman is awesome! It will be much nicer if we have a web dashboard for monitoring ongoing tasks!
I may put myself into making a PR to implement this.. any comments?

+1 from me :)

I'm happy with a monitor, ie non interactive.  just grab the stats and details and either run a little python http interface and present a couple html tables and css to make it pretty, or even just shove stats into a json or xml file and have a static html/js page to read that and update on changes.
Right now I keep a couple ssh/screen instances open to watch interactive.  Be nicer as a web interface.

2 people give +1 on this plus me it's 3 ppl already... I am going to work on this, expect a PR soon...

I have a homegrown solution now where I just have a cronjob that runs every 2 mins and outputs a formatted version of plotman status to my nginx root so I'm able to view it in a browser. It's as simple as:
plotman status | /usr/bin/txt2html > /var/www/html/plotman.html

I'll likely update that pipeline to insert an HTML meta auto-refresh tag, so that the page auto refreshes at a regular interval.
But I agree a native web interface with more details would be nicer.

I have a homegrown solution now where I just have a cronjob that runs every 2 mins and outputs a formatted version of plotman status to my nginx root so I'm able to view it in a browser. It's as simple as:
plotman status | /usr/bin/txt2html > /var/www/html/plotman.html

I'll likely update that pipeline to insert an HTML meta auto-refresh tag, so that the page auto refreshes at a regular interval.
But I agree a native web interface with more details would be nicer.

This is pretty cool. Let me start with just simple text-output to browser first then we craft some interactive dashboard.

Duplicate of #9


Can't seem to resolve some recipes

Required Information
Class: crp
Level: 61
Craftsmanship: 230
Control: 267
CP: 268
Recipe Name: Beech Lumber
Recipe Level: 61
Solver Seed: (Look at the top of the Execution Log on the Solver page)
Actuall Behaviour
For some reason if i try to solve Beech Lumber with my CRP the button "start" is greyed out, you can try this my loading my json, it also seems to affect some jobs and not others. (i tried it in chrome)
I can go around it by going in the simulator and pressing the solve key, but this is a strange behavior that started a few hours ago.
....
f14_ts_skills.zip

It's because you have actions that are not available with your current job still in your initial guess.
In this case, careful synthesis. Remove those actions with edit, and you can solve again.
Do know that whatever you input in the 'Simulator' tab will be taken as initial guesses for the solver. This means that it will take that sequence of actions as 'base' and try to mutate from there. It's best to keep the initial guess small, like an opener.

Ok got it, not sure why it would put itself there in the first place, also if i go and edit my crafter (crafter-attributes) and select NONE in avail actions and then press guaranteed one would think that by doing that it would reset it to a working state, this does not seem to happen.
Anyway, thanks for making a tool that greatly helps, would be a small nightmare to figure out sane rotations without it :)


Sign DBeaver plugin jars in Eclipse repositories

Every since I started using DBeaver, when I install updates in Eclipse, I get a security warning. I install anyway, but the next time, it happens again,
Warning: You are installing software that contains unsigned content. The authenticity or validity of this software cannot be established. Do you want to continue with the installation?
/Users/joelsolon/.p2/pool/plugins/com.github.jsqlparser_1.2.0.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.core_5.1.3.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.core.eclipse_5.1.3.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.data.gis_1.0.12.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ext.derby_1.1.73.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ext.erd_2.0.90.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ext.firebird_1.0.89.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ext.generic_2.3.74.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ext.h2_1.0.43.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ext.hana_1.0.17.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ext.hsqldb_1.0.35.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ext.import_config_2.1.51.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ext.informix_1.0.79.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ext.mockdata_1.0.14.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ext.mssql_1.0.82.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ext.netezza_1.0.53.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ext.phoenix_1.0.57.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ext.postgresql_2.1.61.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ext.sqlite_1.0.37.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ext.teradata_1.0.56.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ext.ui.locks_1.0.33.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ext.vertica_2.0.18.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.model_1.0.77.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.net.ssh_1.0.9.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.net.ssh.jsch_1.0.9.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.net.ssh.ui_1.0.9.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.runtime.ide.core_2.1.10.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.runtime.ide.ui_1.1.10.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.dbeaver.ui_5.1.3.jar
/Users/joelsolon/.p2/pool/plugins/org.jkiss.utils_2.1.71.jar
/Users/joelsolon/.p2/pool/features/org.jkiss.dbeaver.core.feature_5.1.3
/Users/joelsolon/.p2/pool/features/org.jkiss.dbeaver.ext.generic.feature_2.1.74
/Users/joelsolon/.p2/pool/features/org.jkiss.dbeaver.runtime.feature_5.1.3

Yes, we do not sign jar files. But we should, at least for public Eclipse repositories. Mark as FR.

We'll add jars sign in 5.1.5. All jars in all versions will be signed by our certificate.
However Windows and MacOS installers will still contain unsigned jars (because of pack200 usage). But this shouldn't be a problem as installers are signed themselves.

For the record. I've stuck with slow jarsigner utility problem.
Eventually I found http://insights.sigasi.com/tech/slow_jarsigner_centos7.html which fixes /dev/random performance.
Running sudo apt install rng-tools on Ubuntu helps.

We still have some unsigned 3rd party jars.
com.vividsolutions.jts
org.apache.commons.jexl
com.github.jsqlparser
net.sf.opencsv
Not sure how to handle them. I could sign them with out certificate (just like Eclipse Orbit does with 3rd party bundles).

Hi. Has the signing of JARs been disabled/broken on recent versions?


[QUERY] Azure-cosmos recommended version

Query/Question
In azure-cosmos changelog you recommend to use version 4.31.0 and above (https://github.com/Azure/azure-sdk-for-java/blob/main/sdk/cosmos/azure-cosmos/CHANGELOG.md#4310-2022-06-08).
Why do you recommend this ?
What features or bugfixes justify this?
Thanks

@esciullo - there have been several improvements and critical bug fixes in the Java SDK which are all listed in the changelog below this specific version like memory leak fixes, and jackson dependency updates.
On top of that, azure-cosmos depends on azure-core and azure-core-http-netty modules for reactor and netty dependencies. Those also have critical fixes in the older versions, and that's why recommend this version.
If you want to get more details, I would recommend looking at release notes for azure-core, and azure-core-http-netty for 3rd party library critical fixes.


GEODE-9001: Update Documents to be match recent code changes in compatible-with-redis commands

…compatibility commands

Update docs for INCRBYFLOAT

Update docs for HGET

move HEXISTS and STRLEN to supported commands

Move HINCRBYFLOAT to supported

move DECR to list of supported commands

Move MGET and HDEL to supported

add hkeys to list of supported commands

add incrby to list of supported commands

Move SETNX and GETRANGE to supported

update RedisCommandType & fix missing changes in doc files

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

Thanks for making those updates! Weird thing I just noticed... it looks like the lists of supported and unsupported commands have not been updated in the SupportedCommandsJUnitTest to match the changes in RedisCommandType ...I'm not sure how those tests are passing in the pipeline, but there are failures when I run them locally.

updated with develop

updated with develop

Oh, that makes so much sense now... the pipeline rebases with develop before testing. :octocat: 🤦


feat: add auto PR reviewer

Purpose

Clear purpose of this PR

Proposed Changes
What does this PR add, remove or fix?
Issues

Links to relevant issues

Testing

How was it tested, what were the results?

CodeRabbit will be used


Issue when running tests on Angular Web App using serenityJS + Jasmine

Hi @jan-molak and @wakaleo, we are trying to run component testing on Angular web app using serenityJS + Jasmine but encountered the error below. Any idea how we can resolve this issue?

Hi @zhenwei94! It's hard to tell without seeing the code. Would you be able to create a minimal reproducible example so I can try to reproduce the issue?
Also, you do have zone-testing in your src/test.ts, right?
import 'zone.js/dist/zone-testing';
import { getTestBed } from '@angular/core/testing';

Hi @zhenwei94
Are your sure you can simply kick out Karma as test runner to do unit / component test in Angular as try in your app.component.spec.ts (https://blog.bitsrc.io/understanding-test-configurations-in-angular-2f8b3d1fbd00) ?
Not sure, if Serenity/JS is still capable of this, at all: https://serenity-js.org/handbook/integration/architecture.html

Hi @zhenwei94 , it's been a while, have you found a solution for this? I had a look into your repository again. I still think that this is not necessarily a problem of Serenity/JS at all but with the Jasmine setup in the project.

Let's close this issue. You can request to reopen this issue at any time, if it's still a problem on your side. Maybe you have new information and we could try to dive a bit deeper.


[Epic]: :moneybag: Fees - Phase 1 - Flat Fees

Functional Overview
The work in this epic allows the following flows, which is captured in the e2e tests added.
We might have a public/private token USDP, and a public token used for paying fees AZT. Alice will be able to send USDP to Bob publicly and privately. Alice will be charged a flat fee of 1 AZT for her transaction. Alice will be able to pay this fee in one of two ways:

Her balance of AZT is debited
By transferring USDP to a fee payment contract, which has its AZT debited.

In both cases, the sequencer's balance of AZT is incremented by 1.
Specifying fee information in a transaction will be optional. Requiring fee information by the sequencer will be optional. As such, all existing tests should pass.
Out of Scope
The following items are out of scope for this epic:

Specifying gas limits as part of the transaction (since it will always be 1)
Convergence by the PXE to determine optimal fee
Gas metering of any kind
Gas burning
Handling reverts properly (reverting side effects from app logic, but keeping side effects from fee prep/distribution)

Technical Overview
This epic will:

establish plumbing needed to communicate that different side effects are part of different phases of the transaction to the sequencer (fee prep, app logic)
have the sequencer process transactions as a state machine: execute fee preparation (private and public parts), followed by app logic, followed by distribution
distribute a flat fee to sequencer (address will be specified by the transaction, using dynamic coinbase is blocked by #3824 )
toggle mandatory fee charging in sequencer (e.g. so in e2e tests we can setup without paying fees and later turn fees on)

### End to end flat fees
- [ ] https://github.com/AztecProtocol/aztec-packages/issues/4084
- [ ] https://github.com/AztecProtocol/aztec-packages/issues/4391
- [ ] https://github.com/AztecProtocol/aztec-packages/issues/4388
- [ ] https://github.com/AztecProtocol/aztec-packages/issues/4389
- [ ] https://github.com/AztecProtocol/aztec-packages/issues/4334
- [ ] https://github.com/AztecProtocol/aztec-packages/issues/4087
- [ ] https://github.com/AztecProtocol/aztec-packages/issues/4333
- [ ] https://github.com/AztecProtocol/aztec-packages/issues/4092
- [ ] https://github.com/AztecProtocol/aztec-packages/issues/4094
- [ ] https://github.com/AztecProtocol/aztec-packages/issues/4095
- [ ] https://github.com/AztecProtocol/aztec-packages/issues/4083
- [ ] https://github.com/AztecProtocol/aztec-packages/issues/4088
- [ ] https://github.com/AztecProtocol/aztec-packages/issues/4099
- [ ] https://github.com/AztecProtocol/aztec-packages/issues/4327

Related:

#4287

This whole epic was finished at the offsite :)


Design Meeting Notes, 6/15/2022

More-Specific TemplateStringsArray
https://github.com/microsoft/TypeScript/pull/49552

We have a sort of gap in what you can express between function calls and tagged templates.
Some things that are lacking

Being able to infer specific values and raw values in place of a more-broad TemplateStringsArray type.
Inference in the face of intersections.

Why? Why now?

Has come up in a few issues.
Relatively little work.
Wanted to power more powerful mechanisms using plug-ins in tsc, but the type-system might be powerful enough now.
It's an asymmetry with function calls.

We said don't do this (https://github.com/microsoft/TypeScript/issues/45504). The type system isn't optimized to be a parser. Function calls maybe provide enough "syntactic salt". So why do we want to incentivize this?

You can already do this today.
Also, language service plugins can give you errors, but you can't get a better type.

Tuple inferences may be good regardless of this change.
Can give more specific return types, can validate the string, and even use inference to validate the string.

Should make sure the DSL libraries we're trying to model here are actually tested.
But also, the error messages that these produce will be totally unappetizing.

Conclusion

Not 100% sold across the team, but we can convince ourself that it's reasonable to bring in. Not a priority for TS 4.8, but if it gets in, it gets in.

A Tour of Module Resolution Strategies
https://gist.github.com/andrewbranch/3020c4e24092bd37f7e210d6f050ef26

Node.js 16+ ESM support was a good focus, but lots of other resolvers that have a non-consistent set of features.
Been telling people "you need to reference .js as a module extension" for Node.js.

If that's too painful, but maybe you shouldn't use the new mode if you're not using Node - but then what should people be doing?
Usually the best, most-maximal one is just called, uh, node.

People using bundlers are often best-off targeting --moduleResolution node.

But then every bundler supports the exports field.
But then our old node mode doesn't understand that.

Maybe we should pick that into node?

Seems like a "duh!" but then...why don't we see lots of demand for this!?

Hypotheses:

Relatively uncommon to use exports.
ESM/CJS APIs are similar enough.
People don't know how to ask, especially because this is so confusing.

This is all very confusing - if people are so frustrated, couldn't we transform paths based on .ts to .js?

Always comes up - but it amounts to a transformation from a "TypeScript-specific module system" to several target module systems, and they need to be able to compose well.

How do you deal with lies in .d.ts resolution?
Do the .ts extensions get preserved in .d.ts files?
How does it deal with dynamic imports? Do you need a TS dynamic import helper for non-static imports?
How do you refer to dependencies in packages which are exposed as .js files? They aren't .ts files.

The point of this table is not to try to model every single bundler behavior, it's to try to figure out the common module targets and where we have a gap.
So one of those common targets is places where you don't need a compiler and the runtime/resolver can do the TS -> JS transform (e.g. Deno, bundlers, etc.).
No conclusion, but at least feel like we're getting a better understanding of the lay of the land.

The module resolution is extremely confusing, and this doesn't even take into account tools like ESLint and rules that traverse the file system (import/no-unresolved, etc). When a TS file has imports with an extension that point to a non-existent file, it makes this very hard to understand.
What if the extension in imports were ditched, and instead assumed/derived by compiler options?

moduleResolution: nodenext + module: commonjs === .cjs (.cts lookup)
moduleResolution: nodenext + module: es/nodenext === .mjs (.mjs lookup)
moduleResolution: node + module: commonjs === .js (.ts lookup)
moduleResolution: node + module: es/nodenext === .js (.ts lookup)

Or if that is still too confusing, being explicit with an outExtension compiler option is probably the path of least resistance.

That just moves the problem elsewhere and introduces more configuration. You can't unconditionally add file extensions -- importing from fs/promises should not print fs/promises.js, that won't work. Yet people can and do use path mappings combined with bundler configuration to make paths that look indistinguishable from that:
import 'vs/editor/editor.all';

so now we need some additional configuration that controls which imports of the form 'a/b get rewritten to including file extensions. But it's worse - is the file editor.all.ts, or editor.all/index.ts?
All of this configuration effectively takes us backwards in the journey to making sure files are transpilable without needing extra information from the file system or from other files in the system, which is a key source of performance gain in a ton of scenarios including extant bundlers like esbuild, and our own ability to parallelize emit in the future.

but then...why don't we see lots of demand for this!?

Please add "we hand-write ambient declarations" to the "hypotheses" 😅 https://github.com/facebook/docusaurus/blob/5746c58f4145f56b92512095bce71428638d9d45/packages/docusaurus-plugin-content-docs/src/plugin-content-docs.d.ts#L620-L622

@lemonmade I think you may have lost me between what configuration you are using for the development of Quilt and what configuration Quilt is generating for users. E.g., you said

This tool produces no .js files

but the exports config mentions JS files. I also see in your project’s root tsconfig.json you have an outDir set, so I’m not quite following who has no .js files. Seems like everyone does 😄

Maybe we should pick that into node?

Seems like a "duh!" but then...why don't we see lots of demand for this!?

Consider this a voice of demand then 😄 In SvelteKit we have to keep a somewhat ugly ambient modules hack because of this (similar to @Josh-Cena) (https://github.com/sveltejs/kit/blob/master/packages/kit/types/ambient.d.ts#L71) and we'd very much like to switch to using export maps only instead, which right now doesn't work because the export resolution is not part of the node resolution algorithm. Since Node 12.X export maps present means they are used regardless of whether or not you use the commonjs or esm resolution, so I feel like a backport makes sense. But I also understand that maybe some people with older Node versions don't want this behavior, so we need an additional node10 setting now which preserves the old behavior? I certainly don't envy you 😅
It's also very strange that we need to reference ambient files (d.ts files) like import "./ambient.js" which feels wrong (is this related to the "lie" statement in the meeting notes?)


remove LastName from config.js

This is needed for 208 to avoid the dreaded
ERROR running force:org:create:  No such column 'LastName' on sobject of type ScratchOrgInfo.
error.

People who are still cloning and using this in 206 are now running into org create failures. See Issue #131.
Maybe you could start versioning/tagging this repo. At least for the 206/208 changeover period.


Quickstart links to html/javascript documentation

"Quickstart: Reading and writing a file" links to html / javascript documentation on Windows 8. It should link to more a more recent quickstart.

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

ID: 2b109100-bcd9-bf8e-b1bb-2f191b9a7c40
Version Independent ID: aa58ff16-a56c-0e4f-0c93-284d007a2460
Content: IRandomAccessStream Interface (Windows.Storage.Streams) - Windows UWP applications
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.storage.streams.irandomaccessstream.yml&version=GBlive&_a=contents
Product: uwp
Technology: winrt-reference
Git Login: lastnameholiu
Microsoft Alias: alholiu

Hi @wldevries , and thanks for bringing this to our attention. I went ahead and updated the quickstart link to point to our most updated one here. Thanks for your contribution and be well.


Can we start creating standard docs for this package?

🚀 Feature
Separate docs website, or much more detail in readmes throughout the website.
Motivation & Examples
Tell us why the feature is useful.

Package in its current form is very difficult to get started with (I'm trying to deploy one of these models to a raspberry pi in some form which avoids the need to install torch etc. ideally ONNX).

Describe what the feature would look like, if it is implemented.

Website documenting typical workflows and benchmarks on different machines for different models

I'd be keen to work on all of this.

Caveat - I had not found this notebook https://github.com/facebookresearch/d2go/blob/main/demo/d2go_beginner.ipynb which I believe resolves most of my frustrations.

There also seems to be some tooling provided for knowledge distillation but no docs on how to use it?

@GeorgePearse Thanks for the interest, there might be certain experimental features in the codebase but mainly for internal use, we'll update the doc if we decide to release them.

What's the status of distillation? If I could specify a torchscript path to an X101 from detecctron2 that would be great.


WIP: Add Validate v1.13.0 with JavaRosa v2.15.0

I can confirm that the problem here is with ODK Validate, not the conversion.
If I change run_odk_validate to False in pyxform/tests_v1/test_support_external_instances.py and run nosetests pyxform.tests_v1.test_support_external_instances, the tests pass.


Change Scale component from Scale(f32) to Scale(Vec3)

Other game engines have scale as Vector3, as there are many usecases where you only want to scale a single dimension. I think this should be a fairly simple change to make.

There is a NonUniformScale type too that has 3 components, that should work I think.

I'll mark this as documentation because I think this needs to be made more visible, this is not the first time this came up.
If NonUniformScale does not work for you then let me know and I'll re-tag this issue.

Thanks! It does work with one nitpicky caveat.
 Is there a way to override it? I tried slapping NonUniformScale on top and it just breaks all transformations. Of course, I could just copy paste the SpriteSheetComponents and the defaults provided and make my own bundle, but I'll end up with quite a bunch of duplicated code by doing this if I wish to always use NonUniformScale!

I wonder if it would make sense to add a #[bundle(option)] attribute in #[derive(Bundle)] to make optional components work.

I think this has been fixed in the move to the Transform component


Connection error when running gradio_app.py

huggingface_hub.utils._errors.LocalEntryNotFoundError: Connection error, and we cannot find the requested files in the disk cache. Please try again or make sure your Internet connection is on.
Is it simply an Internet error?

yeah, just repeat downloading them ......


Spark 3.0.2 shim no longer a snapshot shim

Spark 3.0.2 has been released, so this updates the 3.0.2 shim to no longer be a snapshot shim so it will be shipped as part of the 0.4 release.

build

I think we want to remove the -snapshot support from the service provider:
https://github.com/NVIDIA/spark-rapids/blob/branch-0.4/shims/spark302/src/main/scala/com/nvidia/spark/rapids/shims/spark302/SparkShimServiceProvider.scala#L23

build


Revert "[SR-6978] Implement support for simultaneous vX.X.X and X.X.X…

… style tags (#1503)"
This reverts commit a0087c7bc47a20d059c54e81a196c4e3492396e1.
Reverting this commit because it is causing resolution error in some of
the community packages. We need to understand the bug and write a test
that would have caught this issue. To reproduce, clone the community
package Moya @ 4b3ff7e and run swift-build.

@swift-ci please smoke test

/cc @Vercantez


Add filter for ADE API errors & remove v1 update route

Maybe creating a specific error handler for those : https://flask.palletsprojects.com/en/2.0.x/errorhandling/#error-handlers

I don't see what adding a specific error handler will add compared to what we have now (e.g. all bad API requests are logged on the admin page).

Actually, @SnaKyEyeS , shouldn't we use the error message from Python and not from HTML files ? It would make more sense if we want to have different messages for the different errors possible.
This current error message displayed on ADE Scheduler is "hardcoded" in the HTML but it could use the message raised by the exception, no?

Yeah I don't exactly remember how we handled that to be honest... I'd need to get a look at that and change the way we defined things accross all error callbacks, which is not as simple as that I think :)


feat (aurelia-validatejs): custom validation rule support

Add ability to decorators and fluent API in aurelia-validationjs for developer
to extend validate.js with custom validation routines.
issue #4 https://github.com/aurelia/validatejs/issues/4

closing- this library is deprecated


Update mustache format in archetype files

Currently, we have following style in mustache template files
{{#header}}
{{.}}
{{/header}}

It is a general best-practice to do the test and the {{.}} on the same line, since it is not so vertical-white-space verbose.
example:
{{#header}}{{.}}
{{/header}}

Closed by #6287 and #6286


[SPARK-39819][SQL] DS V2 aggregate push down can work with Top N or Paging (Sort with expressions)

What changes were proposed in this pull request?
Currently, DS V2 aggregate push-down cannot work with DS V2 Top N push-down (ORDER BY col LIMIT m) or DS V2 Paging push-down (ORDER BY col LIMIT m OFFSET n).
If we can push down aggregate with Top N or Paging, it will be better performance.
This PR only let aggregate pushed down with ORDER BY expressions which must be GROUP BY expressions.
The idea of this PR are:

When we give an expectation outputs of ScanBuilderHolder, holding the map from expectation outputs to origin expressions (contains origin columns).
When we try to push down Top N or Paging, we need restore the origin expressions for SortOrder.

Why are the changes needed?
Let DS V2 aggregate push down can work with Top N or Paging (Sort with group expressions), then users can get the better performance.
Does this PR introduce any user-facing change?
'No'.
New feature.
How was this patch tested?
New test cases.

ping @huaxingao cc @cloud-fan

sorry there are conflicts...

sorry there are conflicts...

Let's fix it.  :)

thanks, merging to master!

@cloud-fan Thank you very much !


Features & fixes for next release

This PR prepare the next 0.2.4 release with the following updates:

Add support for Verdaccio 4.4
Add retry mechanism on Bucket initialization
Updated dependencies

@juanpicado Thanks :)


Rename setters/getters/builders for BigQuery classes to meet proto conventions

Coverage decreased (-0.1%) to 84.539% when pulling fbe1aeb0a228d73d2643a5783320ae8d82c13c89 on mziccard:bigquery-rename into 513ca7034aca5395ab4e1b7a36899ffa56566af8 on GoogleCloudPlatform:master.

Coverage decreased (-0.07%) to 84.494% when pulling cbc7054738aed85ed18231f349f63980592c4879 on mziccard:bigquery-rename into 8bf9eab077037ba5d16ab45d92aef9a5fae5c0b8 on GoogleCloudPlatform:master.

/cc @lesv I rebased and added a commit to make deprecated methods call renamed ones. PTAL

Coverage decreased (-0.07%) to 84.493% when pulling 23730c45a5d3aa4647de6e8c93a2b641d1484a83 on mziccard:bigquery-rename into 8bf9eab077037ba5d16ab45d92aef9a5fae5c0b8 on GoogleCloudPlatform:master.


Enhanced block_from_array with label names

Fixes #685, adding the ability to create tensor blocks from dense tensors of arbitrary order, specifying the names of each axis.

[ ] Tests updated (for new features and bugfixes)?
[ ] Documentation updated (for new features)?
[x] Issue referenced (for PRs that solve an issue)?

Reviewer checklist

[ ] CHANGELOG updated with public API or any other important changes?

@Luthaf  if this looks reasonable, I'll update the tests.


Added new project - Machine learning

Added a new project movie recommender system
Related Issue
Closes the issue number #555
Description
A movie recommender system is a type of algorithmic software that provides personalized recommendations of movies to users based on their viewing history, preferences, and behavior.

Please check the PR @priyankarpal
Thankyou

successfully deployed @priyankarpal


Update tooltips and UI for each filter in Data Processing

Background
Link to issue
https://biomage.atlassian.net/browse/BIOMAGE-704
Link to staging deployment URL
https://ui-91go8oejz3b1ryknh1d86wzo7o.scp-staging.biomage.net/
Links to any Pull Requests related to this
N/A
Anything else the reviewers should know about the changes here
Also fixed some usability problems regarding responsiveness of plots and the data processing page in general by using flex layouts.
Changes
Code changes
Definition of DONE
Your changes will be ready for merging after each of the steps below have been completed:
Testing

[x] Unit tests written
[x] Tested locally with Inframock
[x] Deployed to staging

To set up easy local testing with inframock, follow the instructions here: https://github.com/biomage-ltd/inframock
To deploy to the staging environment, follow the instructions here: https://github.com/biomage-ltd/biomage-utils
Documentation updates
Is all relevant documentation updated to reflect the proposed changes in this PR?

[x] Relevant Github READMEs updated
[x] Relevant wiki pages created/updated

Approvers

[ ] Approved by a member of the core engineering team
[ ] (UX changes) Approved by vickymorrison (this is her username, tag her if you need approval)

Just before merging:

[ ] After the PR is approved, the unstage script in here: https://github.com/biomage-ltd/biomage-utils is executed. This script cleans up your deployment to staging

Optional

[ ] Photo of a cute animal attached to this PR

Looks good, but I prefer the loading bar just beside the Next / Prev button. That's just my preference though,

Looks good, but I prefer the loading bar just beside the Next / Prev button. That's just my preference though,

Same, but it would make it substantially more difficult to properly make things responsive.

@aerlaut -- Fixed.

Point no. 4. in the ticket asked for the dropdown to be removed from the Mitochondrial Content Filter. Is this going to be implemented?

I misunderstood, I thought it was only the other mock options that need to be removed.


Cosmetics/Refactoring

This PR is purely cosmetic refactoring not intended to have any logical changes.
Dependency:

https://github.com/o1-labs/snarky/pull/833

!ci-build-me

!ci-build-me

The dependency cosmetic PR https://github.com/o1-labs/snarky/pull/833 cannot be merged right now because their CI is broken. So I removed the snarky repo update and will merge only the local changes.

!ci-build-me


"database table is locked" while creating an issue via API

Gitea version (or commit ref): 85414d8
Git version: (docker)
Operating system: (docker)
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes
[ ] No
[ ] Not relevant

Description
When I try to create an issue via the API, it returns the error "database table is locked" when using SQLite. If this error gets returned the issue has been created but the status hasn't changed.
2018/06/18 09:04:29 [...api/v1/repo/issue.go:203 CreateIssue()] [E] ChangeStatus: database table is locked: label
2018/06/18 09:04:29 [...api/v1/repo/issue.go:203 CreateIssue()] [E] ChangeStatus: database table is locked: label
2018/06/18 09:04:30 [...api/v1/repo/issue.go:203 CreateIssue()] [E] ChangeStatus: database table is locked: label
2018/06/18 09:04:31 [...api/v1/repo/issue.go:203 CreateIssue()] [E] ChangeStatus: database table is locked: label
2018/06/18 09:04:36 [...api/v1/repo/issue.go:203 CreateIssue()] [E] ChangeStatus: database table is locked: label
2018/06/18 09:04:39 [...api/v1/repo/issue.go:203 CreateIssue()] [E] ChangeStatus: database table is locked: comment
2018/06/18 09:04:40 [...api/v1/repo/issue.go:203 CreateIssue()] [E] ChangeStatus: database table is locked: label
2018/06/18 09:04:42 [...api/v1/repo/issue.go:203 CreateIssue()] [E] ChangeStatus: database table is locked: label
2018/06/18 09:04:43 [...api/v1/repo/issue.go:203 CreateIssue()] [E] ChangeStatus: database table is locked: label
2018/06/18 09:04:44 [...api/v1/repo/issue.go:203 CreateIssue()] [E] ChangeStatus: database table is locked: label

Isn't that duplicate for #2040 ?

@lafriks You're right. But I found a solution for this problem by enabling WAL mode.


Author (in 2 columns) occupying too much space

Description
Author (in 2 columns) occupying too much space
https://arxiv.org/html/2404.18416v2
(Optional:) Please add any files, screenshots, or other information here.

(Required) What is this issue most closely related to? Select one.
Choose One
Internal issue ID
ed321c3a-72dd-40c9-8582-ce90f1685ecc
Paper URL
https://arxiv.org/html/2404.18416v2
Browser
Chrome/<IP_ADDRESS>
Device Type
Desktop

This is especially relevant for large collaborations, I always have to first navigte to the abstract. I understand the point of keeping this structure, but for more convenient reading in the browser might I suggest making the list collapsible, or collapsing it by default? That could be either above a certain count, or if the collaboration is named in the metadata.


[RESOLVED] Unavailable by ICMP ping

Problem started at 01:00:14 on 2023.04.19
Problem name: Unavailable by ICMP ping
Host: MF-FG
Severity: High
Operational data: Down (0)
Original problem ID: 18946
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23130&eventid=18946

Problem has been resolved in 6h 10m 0s at 07:10:14 on 2023.04.19
Problem name: Unavailable by ICMP ping
Host: MF-FG
Severity: High
Original problem ID: 18946


[turborepo] unable to parse yaml: mapping values are not allowed in this context at line 24 column 15

What version of Turborepo are you using?
1.10.12
What package manager are you using / does the bug impact?
Yarn v1
What operating system are you using?
Mac
Describe the Bug
When running turbo build, I keep getting this error. It's not clear to me which yaml file it's referrring to... can someone help me clarify?
Running "turbo run build"
--
00:39:14.085 | WARNING  Issues occurred when constructing package graph. Turbo will function, but some features may not be available:  1 error occurred:
00:39:14.085 | * unable to parse yaml: mapping values are not allowed in this context at line 24 column 15

If referring to the yarn.lock file, the content at line 24 is below, but it's unclear how I'm supposed to resolve this.
 DO NOT EDIT THIS FILE DIRECTLY.
# yarn lockfile v1

// ....

"<EMAIL_ADDRESS>  version "2.2.1"
  resolved<EMAIL_ADDRESS>  integrity sha512-lFM.....
  dependencies: // <<<<<<<<<<<<<<<<<<<<<<<< This is line 24
    "@jridgewell/gen-mapping" "^0.3.0"
    "@jridgewell/trace-mapping" "^0.3.9"

I'm raising this as an issue due to this similar issue.
I create my project using npx create-turbo@latest
Expected Behavior

Should not have malformed yaml
Should clarify which yaml file has the error

To Reproduce
Running "turbo run build"
--
00:39:14.085 | WARNING  Issues occurred when constructing package graph. Turbo will function, but some features may not be available:  1 error occurred:
00:39:14.085 | * unable to parse yaml: mapping values are not allowed in this context at line 24 column 15

Reproduction Repo
No response

Do you have a packageManager set in your package.json? If not, can you set it to the version of yarn that you expect?

If I had to guess, I believe that this would be fixed by https://github.com/vercel/turbo/pull/5986

If anyone encounters this please try upgrading to<EMAIL_ADDRESS>and reopen if the problem persists.


Sum of Digits in C | Math | GSSoC'21

💥 Proposal
Sum  of Digits in C
Have you read the Contributing Guidelines on Pull Requests?
YES

/assign


Allow enabling Go Pprof

If the profile.port argument is set to something different than 0, it will add a pprof endpoint so we can inspect what's going on FLP in terms of memory and CPU.

Codecov Report

Merging #308 (1bf6822) into main (ef4a155) will decrease coverage by 0.10%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #308      +/-   ##
==========================================
- Coverage   68.67%   68.57%   -0.11%
==========================================
  Files          81       81
  Lines        4699     4706       +7
==========================================
  Hits         3227     3227
- Misses       1276     1283       +7
  Partials      196      196

Flag
Coverage Δ

unittests
68.57% <0.00%> (-0.11%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

cmd/flowlogs-pipeline/main.go
0.00% <0.00%> (ø)

pkg/config/config.go
63.63% <ø> (ø)

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

/lgtm
but a short documentation on how to use it would be nice

@jotak I'll add the actual documentation in the NOO repo, as a new PR.


Discussion: code change for minimap integration

Hi ! Happy new year to all.
I've had a look at how to build a minimap integration for this package, as @t9md noted elsewhere he is not a user of minimap himself.
I have something basic working, but I've had to make some modifications to quick-highlight as one of the current optimizations (highlighting only what's visible) means we would also show, in the minimap, those highlights only. In the context of minimap it kind of defeats its purpose, in my opinion.
So, I'm now creating all highlights (even invisible ones), which could affect performance negatively, especially for large files.
On the other side, there's currently two subscriptions to the editor scroll events that I can remove, so this will give a smoother experience during scroll, especially on large files.
What do you think @t9md ? Should such changes live in a fork, or could they be merged here ? I know some people do not use minimap, so if the change mentioned above sounds bad to you, I understand you'll be cautious about it.

Thanks, I think it's mergable.
But if performance impact is big, I want make all-highlight-optional(auto detect minimap-quick-highlight and  passing option to rendering method of quickhl?).
If you have working minimap package repo, I can update my quick-hl by myself.

Thanks, I think it's mergable.
But if performance impact is big, I want make all-highlight-optional(auto detect minimap-quick-highlight and  passing option to rendering method of quickhl?).
If you have working minimap package repo, I can update my quick-hl by myself.

Here's the early working repo, not published on apm yet:
https://github.com/gouegd/atom-minimap-quick-highlight
And here are the changes I did on a local version of quick-highlight:
https://github.com/gouegd/atom-quick-highlight/commit/76c5049ef1ff22843815f1105daf96d6f78e0085
I believe any perf impact would be visible only on large files. Maybe the size of the file would be a good trigger to decide whether to rerender on scroll or to render everything once.

Here's the early working repo, not published on apm yet:
https://github.com/gouegd/atom-minimap-quick-highlight
And here are the changes I did on a local version of quick-highlight:
https://github.com/gouegd/atom-quick-highlight/commit/76c5049ef1ff22843815f1105daf96d6f78e0085
I believe any perf impact would be visible only on large files. Maybe the size of the file would be a good trigger to decide whether to rerender on scroll or to render everything once.

Thank, will check tomorrow.

Thank, will check tomorrow.

@t9md just updated, it's pretty cool now - it works for manual toggles too, and with the proper colors.

@gouegd Checked, cool!
I'm working on branch provide-service-for-minimap. for PR #13.
Pls check comment in #13 for the required code change of your minimap-quick-highlight.

@t9md ok, I just pushed this small change.
Time to sleep here, I'll check back tomorrow 👍

Me too sleepy!
On Tue, Jan 3, 2017 at 7:03 Greg D<EMAIL_ADDRESS>wrote:

@t9md https://github.com/t9md ok, I just pushed this small change.
Time to sleep here, I'll check back tomorrow 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/t9md/atom-quick-highlight/issues/12#issuecomment-270024942,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJeRTKZlDjl9xcInEfJqseZ_1-jpamwks5rOXQYgaJpZM4LYvnC
.

Done in https://github.com/t9md/atom-quick-highlight/pull/17


test: add batcher client to sequencer end_to_end_test

This PR conflicts with #1296.
In this PR we do not create a mock consensus manager but add the least amount of code to allow us to add the batcher to the flow test.

This change is

#1352  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ArniStarkware and the rest of your teammates on  Graphite

crates/tests-integration/tests/end_to_end_test.rs line 69 at r1 (raw file):
Previously, yair-starkware (Yair) wrote…
What about build_proposal & get_proposal_content? both are already in main

build_proposal is covered in the next PR: #1386. (Was not possible before: #1384).
get_proposal_contexnt can not be added to the test yet as the BlockBuilderTrait is not completely implemented.

crates/tests-integration/tests/end_to_end_test.rs line 66 at r1 (raw file):
Previously, yair-starkware (Yair) wrote…
I thought this was in the context of the setup.
Please add a todo to get the height from the mock_running_system object.

Done.


Reduce amount of implemented code on Expressions

To reduce the amount of implemented code on Expressions and Nodes, we can heavily use UnaryExpression, BinaryExpression, and so on.
Right now, these interfaces are just implementing Children(), Resolved() and IsNullable().
We could implement also TransformUp function as following:

Create a new interface in charge of return a new instance of the implemented expression:

type ExpressionInstanceGenerator interface {
	Generate(exprs ...sql.Expression) (sql.Expression, error)
}

With this, we will be able to implement this interface on Unary and Binary expressions:

type BinaryExpression struct {
	ExpressionInstanceGenerator
	Left  sql.Expression
	Right sql.Expression
}

After that, TransformUp method can be implemented on BinaryExpression. Generate method is the one that will be implemented on the other Expressions:

func (p *BinaryExpression) TransformUp(f sql.TransformExprFunc) (sql.Expression, error) {
	var err error

	var l sql.Expression
	if p.Left != nil {
		l, err = p.Left.TransformUp(f)
		if err != nil {
			return nil, err
		}
	}
	var r sql.Expression
	if p.Left != nil {
		r, err = p.Right.TransformUp(f)
		if err != nil {
			return nil, err
		}
	}

	exp, err := p.Generate(l, r)
	if err != nil {
		return nil, err
	}

	return f(exp)
}

Doing this, we will reduce bugs on bad implemented TransformUp functions, also we can create generic Tests suites for this kind of expressions.

What about this adding a WithChildren method to the Expression interface and leverage the already present Children method.
type Expression interface {
  WithChildren(...sql.Expression) (sql.Expression, error) // add this
  // remove TransformUp
}

Then have a:
func Transform(e sql.Expression, f sql.TransformExprFunc) (sql.Expression, error) {
  var children []sql.Expression
  for _, c := range e.Children() {
    c, err := f(c)
    if err != nil {
        return nil, err
    }
    children = append(children, c)
  }

  return e.WithChildren(children)
}


New author page should exist

Scenario
When a user visits the new author page
Then the page should render
Hints
Currently, the application contains only a single route (the root at '/', see config/routes.rb), a single view (app/views/home/index.html.erb) and a single controller (app/controllers/home_controller.rb), which connects the two.
All available routes of the application can be displayed using rails routes.
Before you begin implementing this feature, write a test to document its absence.
The file spec/features/author/new_author_spec.rb already contains the needed test case, it only needs to be un-commented.
The describe block defines which aspect of the the software will be tested here.
Specifying the feature type automatically includes helper methods that allow to simulate browser actions in a test.
Inside describe, each it block is a single test case.
In this example, the test tries to access the URL of the new-author page.
This fails because no such path is defined yet.
Commit the failing test case.
Now, this test needs to be made to pass. See the Rails tutorial for instructions to create the appropriate controllers, views and setting up routes for authors.
Instead of articles (as in the tutorial), this exercise is concerned with authors.
You can learn more about RSpec, the testing framework, in this documentation
Error

Got NameError: undefined local variable or method `new_author_path' for #RSpec::ExampleGroups::NewAuthorPage:0x0000000006083a08
Did you mean?  new_polymorphic_path

1/44 exercise tests have passed

Sorry, it's not working for me yet:
Scenario
When a user visits the new author page
Then the page should render
Error

Got NameError: undefined local variable or method `new_author_path' for #RSpec::ExampleGroups::NewAuthorPage:0x00000000054b1400
Did you mean?  new_polymorphic_path

If you have problems solving this task, please don't hesitate to contact the teaching team!

Sorry, it's not working for me yet:
Scenario
When a user visits the new author page
Then the page should render
Error

Got NameError: undefined local variable or method `new_author_path' for #RSpec::ExampleGroups::NewAuthorPage:0x0000000005721d70
Did you mean?  new_polymorphic_path

If you have problems solving this task, please don't hesitate to contact the teaching team!

Sorry, it's not working for me yet:
Scenario
When a user visits the new author page
Then the page should render
Error

Got NameError: undefined local variable or method `new_author_path' for #RSpec::ExampleGroups::NewAuthorPage:0x0000000004e469a8
Did you mean?  new_polymorphic_path

If you have problems solving this task, please don't hesitate to contact the teaching team!

Sorry, it's not working for me yet:
Scenario
When a user visits the new author page
Then the page should render
Error

Got NameError: undefined local variable or method `new_author_path' for #RSpec::ExampleGroups::NewAuthorPage:0x0000000004fe5340
Did you mean?  new_polymorphic_path

If you have problems solving this task, please don't hesitate to contact the teaching team!


fix insert partial tablet with binary NullPointer bug

When using insertTablet, we create a Binary[100], then insert 10 values.
The getValueBytesSize do not use the rowCount 10 when calculating the size.
#2179

has cherry-picked to rel/0.11


Question: Is an API to this project considered for the future?

Hello, I read the project summary and I like the use case.
I didn't try it yet. But I got the idea or thought that it would be practical to offer something API like.
So that other apps or services could quickly hook in on this specific use case and retrieve the results.
This is just quick thought / idea and I'm curios.
Looking forward to a short answer.
Have a really nice day! 👋

👋Thank you for your interest in this project! This project was designed from the start to be a ready-to-use local application. You only need to install the dependencies with conda to launch the GUI. Currently there is no plan to make it a service API, because the program involves model inference during runtime, which requires substantial computing resources. We do not have enough spare resources to support such a service yet. Of course, you are also welcome to build upon this project to enable server-side deployment.

Thank you for the detailed answer!


[WIP] Add detecting DPU

I think we need an override for this that is optional as part of a CR. The reason is that it could be common for a customer to have a DPU that they are perhaps using for something else (i.e. SmartNIC/NIC mode) and only want one DPU to be used.

This code only detects the devices, but doesn't do anything with it yet. We need to add code in the future to determine which devices we will configure/handle. Only when we have a full list of devices, can we decide which ones should not be handled. If we start handling configuration settings here, then the code will do two things: "detect and determine if we need to manage it or not".

/lgtm
/override ci/prow/e2e-dpu-operator-tests

/override ci/prow/e2e-dpu-operator-tests

FYI there is a bug introduced in this PR:
[root@localhost dpu-operator]# oc logs -n openshift-dpu-operator dpu-daemon-d45nh
2024-07-17T15:14:42Z    INFO    Daemon Init     Daemon init
2024-07-17T15:14:42Z    INFO    Daemon Init     Detected OpenShift      {"flavour": "MicroShift"}
2024-07-17T15:14:42Z    INFO    Daemon Init     Prepared CNI binary     {"path": "/opt/cni/bin/dpu-cni"}
2024-07-17T15:14:42Z    INFO    Daemon Init     Autodetected mode       {"isDPU": false}
2024-07-17T15:14:43Z    ERROR   Daemon Init     Failed to start daemon  {"error": "Failed to detect any DPU devices"}
main.main
        /workspace/cmd/daemon/daemon.go:165
runtime.main
        /usr/lib/golang/src/runtime/proc.go:271


Using real tailwind names in style

I thought i would get feedback on the option of having real tailwind names in a style prop on objects.
e.g.
<Text styleName="text-blue-500">Blue Text</Text>;

Similar to: https://github.com/kristerkari/react-native-css-modules/blob/master/docs/setup-stylename.md
Any thoughts on why this may or may not be possible?

Hi @nicwhitts
I never read or heard something from this package/setup (trying it as we speak)
I like what I see and you can just use the original tailwindcss package in here so go ahead as it has not the limitations react-native-tailwindcss has. (eg.: plugins are not supported in react-native-tailwindcss)
The only thing I am now testing is how it handles the difference between css and Yoga (react-native style engine).
I see that it is a direct translator of CSS to JS styles.
This is a problem as for example a box-shadow is extremely different in the react native styling world.
The reason I created this package is that I wanted something I would not need to think to hard about for things like a box-shadow or an object-fit.
That these things that are differently named or sometimes supported but in a different way (eg.: elevation is a shadow effect on android)
This means that I would suggest to use react-native-css-modules for the small and base styles if you want too. When a error occurs when a style is not supported or recognised you can Use react-native-tailwindcss to fill in the gaps.
Hope this explains a bit why react-native-tailwindcss came to be.
greetings
@TVke

That explains it!
Something i think you might be interested in, i just did a quick spike test and this seems to work:
(Will continue to test all the styles.)
There is a package made for react native tachyons here:
https://github.com/shipt/react-native-tachyons
I built a stylesheet object using your generator but without translating the keys, and then returned it in the build method of the tachyons
https://github.com/shipt/react-native-tachyons/blob/master/src/build.js
So basically the tachyons library is now using the tailwind syntax
And it works with both the styled and T:
<Text style={T('text-red-500')}>Test text</Text>

or
const TestText = styled(Text)`text-red-500`

Hey @nicwhitts
The first idea I had was to do stuff like:
<Text style={[t['text-red-500'], t['text-center']]}>Test text</Text>

The big problem I found was the typing work was huge. One of the goals of the original tailwindcss is speed. In the syntax above I tried a few pages but got tired of the brackets and the dashes.
I then thought that the object syntax was way cleaner, faster and more readable. The only compromise  I needed to make is the dashes (in an early version I tried _).
So after much trying I decided that the CamelCase syntax provided the best naming convention for it's speed and clarity.
I like the way you thing and I have considered a lot and the tachyons seem really cool but for now it wil stay on the CamelCase.
I'm writing docs for this package and I think there is a page you will definitely would want to read:
https://tvke.github.io/react-native-tailwindcss/
greetings
@TVke

That's cool :)
Good job by the way


Discrepancy between data fields and calendar entry creation

So when I look at an entry I make by clicking edit, the times listed for creation and due date are listed correctly in my local time. When I click to create a gcal entry, it's put on my calendar in eastern time instead.

This is working for me on staging but not on production. Is that expected?

@dreeves i think the latest changes didn't get deployed... how is it now?

Fix confirmed!


Runtime error after successful compile on example load_model

After following the instructions to load a model from the README, I can successfully compile the load_model example; however, when run, I receive a runtime error: Illegal instruction (core dumped)
I am running Ubuntu 19.10, GCC 9.2.1, the latest TensorFlow C API (libtensorflow-cpu-linux-x86_64-1.15.0).
Any tips on what I could be missing here?
Thank you!

I haven't tried using g++9, can you try it using g++7, to eliminate that possibility?

I proceed to close the issue due to lack of activity


Fail to build

Hello everyone,
Congratulation, for this repository and your work on it.
I face a problem with the build

I face, also, another problem with the quick start command ./gradlew spring-security-webauthn-samples/javaconfig/webauthn/spa:bootRun

Could you paste full log of the build?
Sorry, quick start command has changed.
Please execute the following:
./gradlew samples:javaconfig:webauthn:spa:bootRun

Using your command it worked!!


playground: Better crowdfunding example

Gets rid of the "NegativeNumber" error that appears occasionally in the logs
Uses wallet ID (int) instead of wallet public key to identify campaign owner

The problems that this deals with only occur in the playground - in plutus-use-cases we can enter the correct public key as a parameter and we also don't run the risk of producing a transaction that causes a NegativeValue error because we always select the correct currency that corresponds to Ada.
So the only reason to also update plutus-use-cases is for greater consistency

So the only reason to also update plutus-use-cases is for greater consistency

Seems like a good reason :) The more these two drift from each other the more annoying they are to update!

The crowdfunding contract is now exactly the same in plutus-use-cases and plutus-playground-server

I am not 100% sure that this ticket created the below UI changes, but it seems the most relevant to me, so I will add the comments here (the sl-plutus has > 70 members).
So while trying to check if this ticket was deployed on dev environment (https://kris.plutus.iohkdev.io/) I observed that for the Crowdfunding contract, the main field names are no longer displayed. Check the below picture.

My concern is that it makes the workflow much less intuitive - you don't know anymore what each value represents in the context of the contract.


Oracle exception message is different when using unsigned int vs unsigned long long

I'm seeing an issue when reading an oracle exception message in c++.
Here's the oracle table and constraint i've set up:
CREATE TABLE TestTableTbl(
id NUMBER
);
ALTER TABLE TestTableTbl
ADD CONSTRAINT TestTableTbl_id_uk
UNIQUE (id);
Here is the oracle package i've defined:
CREATE OR REPLACE PACKAGE TestAdapter IS
PROCEDURE TestInsert(
p_in	IN NUMBER,
p_out	OUT NUMBER
);
END TestAdapter;
CREATE OR REPLACE PACKAGE BODY TestAdapter IS
PROCEDURE TestInsert(
p_in	IN NUMBER,
p_out	OUT NUMBER
)
AS
BEGIN
INSERT INTO PlatSys.TestTableTbl ( id )
VALUES
(
p_in
)
RETURNING id
INTO p_out;
END TestInsert;
END TestAdapter;
Here is the C++ code i'm running:
try
{
ocilib::Pool pool(connect, user, pwd, ocilib::Pool::SessionPool, 0, 3);
ocilib::Connection connection = pool.GetConnection();
ocilib::Statement statement(connection);
int in = 1;
unsigned long long out = 0; // scenario A
unsigned int out = 0; // scenario B
statement.Prepare(_T("begin TestAdapter.TestInsert( :1, :2 ); end;"));
statement.Bind(_T("1"), in, ocilib::BindInfo::In);
statement.Bind(_T("2"), out, ocilib::BindInfo::Out);
statement.ExecutePrepared();
}
catch (ocilib::Exception e)
{
std::cout << e.what() << std::endl;
}
I've inserted one row into the TestTableTbl with id = 1.
When I run the C++ code with scenario A in place e.what doesn't reflect the constraint violation i'm expecting.
When I run the C++ code with scenario B e.what does show the constraint violation i'm expecting.
Why is this happening? Am I doing something wrong when using unsigned long long vs unsigned int?

Hi,
I will test your code and check the source of the issue.
Basically,  when binding 32bits integers, the host variable address is directly passed to oracle.
When binding 64bits integers, ocilib is using an OCINumber (OCI C counter part of server side generic NUMBER) and does assignment from/to host variable before/after statement execution (Oracle only support native bindings for 64bits integers starting with 12cR1)
In that process, ocilib might loose the original error as more internal intermediate code is executed.
I will check it and find it a fix for it this week.
Thanks for reporting issues :)
Best regards,
Vincent

Great. When do you think the release will be out with this fix?
From: Vincent Rogier<EMAIL_ADDRESS>Sent: Tuesday, May 2, 2017 1:51 AM
To: vrogier/ocilib<EMAIL_ADDRESS>Cc: Glassy Peou<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [vrogier/ocilib] Oracle exception message is different when using unsigned int vs unsigned long long (#86)
Closed #86https://github.com/vrogier/ocilib/issues/86.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/vrogier/ocilib/issues/86#event-1065151607, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHRMP5Hmj6p5CLAZChwe6shAeHqg2UXwks5r1u5agaJpZM4NIHX5.


IPNS Pinning - Republishing/reviving

a) Create endpoint that accepts a IPNS Path ("/ipns/k51...") and private key, then resolves it. This function should return the data that was resolved as confirmation (think gateway.ipfs.io/ipfs/Qm....). Then behind the scenes, node continues to keep record alive by republishing.
AND
b) Create endpoint similar to PutKey() (aka PutKeyV1), that accepts a IPFS Path and a private key. This function should publish for the first time and return the IPFS Path as confirmation. Does not delete key, thus allowing node to automatically keep record alive by republishing.
In practice, changes must be made:

Endpoints will need to renamed and reworked to be scoped tighter.
a) PutKeyV2 - take provided private key and store it in node. Does not delete key.
b) PublishV1 = PutKeyV1 - This allows for publishing updates w/o running their own node. And allows others the opportunity to implement their own republishing mechanics and key management systems.
c) PublishV2 - take ipfs path and name of key (string) and basically runs ipfs publish <ipfs_path> <key_name> to publish IPNS Record and republish automatically.
d) PublishV3 - take ipns path and private key, resolve data, if resolved, store key, and allow node to republish automatically.

PutKeyV1 currently gets a key&ipfsCID, stores key, publishes an IPNS record, and deletes key.

Okay first implemented here: https://github.com/mrodriguez3313/IPNSGoServer/pull/17

Publishing and republishing as well as get have been implemented here https://github.com/mrodriguez3313/IPNSGoServer/pull/20
The last thing that is needed to be done (which will be easier) will be

c) PublishV2 - take ipfs path and name of key (string) and basically runs ipfs publish <ipfs_path> <key_name> to publish IPNS Record and republish automatically.

This way to publish makes sense because of the fact that someone can use PostKey() to add their key to own node and now they just need to supply the content to publish. aka they don't need to send the key over the web again.

PostRecord works as expected where if one publishing a record through the service. Then the remote node will handle republishing automatically.
Put record accepts a private key and key ID to allow for republishing in the future. The problem with the state of this endpoint as it is today is: that is not enough for the remote node to handle republishing automatically. For the node to know to automatically republish the record on users behalf, the remote node is the one that needs to do the publishing.
Im thinking to solve this, I can peek at the record's expiration time, and attempt to republish the content at that later time. The problem arises that the user's node may also be online and thus two nodes would republish the content and now there would be a major nonce conflict and even content version problem.


Test out upgrading React

Possible tool to help with transition: https://github.com/reactjs/react-codemod

Needs to be 15.6.2 -- last version before 16

Merged into refactor/upgrade-react-router


QR Scanner

Description
Write a script to generate QR code from given text and also to extract text from the QR code.
Technologies and Languages

Python

Any other language and technologies are also welcome
Aditional Work
Make a GUI for this

can you assign this work to me please

Okay sure..

Great!. Looking forward to it

sorry brother you can assign this to someone else. In my Pycharm some modules are not working.

@yashqwerty1  did the modules work?

nope bro sorry it is showing no module named '' "

should I un-assign u then?  Well u can always give a try after wards if you are interested to do this project

Sure Thanks for giving me this assignment.
I am going to come back with a solution for my problem.

I will love to work on this

Sure. Go ahead 👍

Also do I have to implement Webcam capture in it.
If yes then real-time or a single photo capture

@bislara Please reply

Hey sorry.
If you can implement it in real-time then it would be really great. If you are facing issues, then you can go ahead with a picture.

Ok

Any update? Feel free to reach out to me if you need any help.

@Samikdevel  Any progress?

Yes, Sorry for not replying.
I had some tests going on and my pc got corrupted so I have to start again.
But hopefully I will do it in 3-4 days.

Okay cool no problem

Done


Erosion on result image after affine registration

Hello,
I am currently developing a medical image search software in Java, which should include a registration module. I decided to give SimpleElastix a go but ran into the a problem. The result image seems to be eroded (white lines around object edges) and I have no clue why.
Source image
Target image
Result image
This are the steps i have done:

I have two Java BufferedImage which I convert into itk.simple.Image (reading and writing the pixel values). This is how the pixels are written:

VectorUInt32 vector = new VectorUInt32();
//i and j are pixel coordinates
vector.add(i);
vector.add(j);
//images are unsigned. Since Java does not support unsigned data, 0xFF is "added"
image.setPixelAsUInt16(vector, databuffer[i*width+j] & 0xFF);

I register the two images using the following code:

VectorOfParameterMap parameterMapVector = new VectorOfParameterMap();
ParameterMap rigidMap = elastixImageFilter.getDefaultParameterMap("affine");
parameterMapVector.add(rigidMap);
elastixImageFilter.setParameterMap(parameterMapVector);
elastixImageFilter.execute();
Image result = elastixImageFilter.getResultImage();

convert the itk.simple.Image to a Java BufferedImage. Here is a small code extract showing the basic logic:

VectorUInt32 vector = new VectorUInt32();
//i and j are the pixel coordinates
vector.add(i);
vector.add(j);
intPictureValues[t] = (int)itkImage.getPixelAsFloat(vector);
...
bytePictureValues[i] = (byte)intPictureValues[i];
DataBuffer dataBuffer = new DataBufferByte(bytePictureValues, 1);

Hi. Sorry for the late reply. Feel free to reopen if this is still an issue.


:rainbow: Rainbow easter egg added - press r to see the magic!

🌈🌈🌈🌈🌈🌈🌈🌈🌈🌈🌈🌈🌈🌈🌈🌈🌈🌈🌈🌈🌈🌈🌈🌈🌈🌈🌈

haha nice 🌈


updated validations.py

now it returns false for usernames with '.' or '_'  in the begining
Fixes #<issue_number_goes_here>

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

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

created a pull req after editing the validatiosn.py file