Tip

Hey man!
I see you are having issues with bundling the modules.
Just wanted to let you know that you can use https://github.com/mafintosh/prebuild for that
Cheers

Hello!
Thank you! I think about prebuilds for this module, but 'prebuild' use node-gyp for build module.
It this case, we should use cmake because libuiohook developed in cmake.

Hey!
I see, it looks like the JavaScript API of "prebuild" allows you to specify a custom backend.
Apparently, cmake-js (which you already seem to be using) claims it works "exactly" like node-gyp, so I wonder if it could be "dropped in" perhaps?
I might give this a try myself when I have some time on my hands.
Thanks for making this!

I have put together this bindings.gyp file which seems to work with node-gyp.
Not really tested thoroughly but anyways:
{
  "targets": [
    {
      "target_name": "iohook",
      "sources": [ "src/iohook.cc" ],
      "include_dirs": [
        "<!(node -e \"require('nan')\")",
        "libuiohook/include"
      ],
      "conditions": [
        [ 'OS!="win"', {
          "cflags+": [ "-std=c++14" ],
          "cflags_c+": [ "-std=c++14" ],
          "cflags_cc+": [ "-std=c++14" ],
        }],
        [ 'OS=="mac"', {
          "xcode_settings": {
            'GCC_ENABLE_CPP_RTTI': 'YES',
            'GCC_ENABLE_CPP_EXCEPTIONS': 'YES',
            'MACOSX_DEPLOYMENT_TARGET':'10.8',
            'CLANG_CXX_LIBRARY': 'libc++',
            'CLANG_CXX_LANGUAGE_STANDARD':'c++11',
            'GCC_VERSION': 'com.apple.compilers.llvm.clang.1_0'
          }
        }]
      ]
    }
  ]
}

Took a little from here: https://github.com/peteyy/native-addon-example/blob/cpp14/binding.gyp
Then from here to get rid of a compiler is deprecated warning on OSX: https://github.com/mapnik/node-mapnik/blob/c8dee11e7e8f29168cf4e96c46b13d4933246327/binding.gyp#L115-L120
Hope it helps

Ok never mind me that was super stupid, it's complaining about missing symbols of course since it needs a whole bunch more linked then "libuiohook/include" that I added up there.
Anyways, here was my previous try to get cmake working in the prebuild script:
var prebuild = require('prebuild')
// var cmake = require('cmake-js')
var cmake = require('node-cmake')
var nodeGyp = require('node-gyp')()
cmake.parseArgv = function (args) {
  nodeGyp.parseArgv(args)
  cmake.commands = nodeGyp.commands
  cmake.todo = nodeGyp.todo
}
var version = '0.8.0'
prebuild.build({
  log: {
    verbose: console.log.bind(console),
    error: console.log.bind(console)
  },
  pkg: require('./package.json'),
  gyp: cmake
}, version, function (err) {
  // ...
  console.log(err)
})

Not so great either as node-cmake and cmake-js are not fully compatible with node-gyp (see the parseArgv hack I did there)
Perhaps a little shell script that does what prebuild does is the way to go here...

Sorry for the spam, updated example that starts working better =)
var prebuild = require('prebuild')
var spawn = require('child_process').spawn

var version = '6.10.0'
const run = (args) => {
  spawn('./node_modules/.bin/ncmake', args).stdout.pipe(process.stdout)
}
const cmakeWrap = {
  todo: [],
  parseArgv: function (argv) {
    console.log('PARSE ARGV', arguments)
    this.todo.push({
      name: argv[2],
      args: argv.slice(2)
    })
  },
  commands: {
    build: run,
    clean: run,
    configure: run,
    rebuild: run,
    update: run,
    install: run,
    list: run
  }
}

prebuild.build({
  log: {
    verbose: console.log.bind(console),
    error: console.log.bind(console)
  },
  pkg: require('./package.json'),
  gyp: cmakeWrap
}, version, function (err) {
  // ...
  console.log(err)
})

Updated version that seems to work for electron too (replaces --target flag with --runtime-version flag)
var prebuild = require('prebuild/prebuild')
var upload = require('prebuild/upload')
var spawn = require('child_process').spawn
var async = require('async')
var extend = require('xtend')

const run = (args, cb) => {
  args = args.map(arg => {
    if (arg.indexOf('--target') > -1) {
      return arg.replace('--target', '--runtime-version')
    }
    return arg
  })
  const proc = spawn('./node_modules/.bin/cmake-js', args, { env: process.env })
  proc.stdout.pipe(process.stdout)
  proc.stderr.pipe(process.stderr)
  proc.on('exit', (code, sig) => {
    if (code === 1) {
      return cb(new Error(`Failed with signal: ${sig}`))
    }
    cb()
  })
}

const cmakeWrap = {
  todo: [],
  parseArgv: function (argv) {
    this.todo.push({
      name: argv[2],
      args: argv.slice(2)
    })
  },
  commands: {
    build: run,
    clean: run,
    configure: run,
    rebuild: run,
    update: run,
    install: run,
    list: run
  }
}

var opts = {
  target: process.versions.node,
  runtime: 'node',
  arch: process.arch,
  libc: process.env.LIBC,
  platform: process.platform,
  all: false,
  force: false,
  debug: false,
  verbose: false,
  path: '.',
  log: {
    verbose: console.log.bind(console),
    error: console.log.bind(console)
  },
  pkg: require('./package.json'),
  gyp: cmakeWrap,
  prebuild: require('node-abi').supportedTargets,
}
console.log(opts.prebuild)

var files = []
async.eachSeries(opts.prebuild, function (target, next) {
  prebuild(opts, target.target, target.runtime, function (err, tarGz) {
    if (err) return next(err)
    files.push(tarGz)
    next()
  })
}, function (err) {
  if (err) return onbuilderror(err)
  uploadFiles(files)
})

function uploadFiles (files) {
  console.log('Uploading ' + files.length + ' prebuilds(s) to Github releases')
  upload(extend(opts, {files: files}), function (err, result) {
    if (err) return onbuilderror(err)
    console.log('Found ' + result.old.length + ' prebuild(s) on Github')
    if (result.old.length) {
      result.old.forEach(function (build) {
        console.log('-> ' + build)
      })
    }
    console.log('Uploaded ' + result.new.length + ' new prebuild(s) to Github')
    if (result.new.length) {
      result.new.forEach(function (build) {
        console.log('-> ' + build)
      })
    }
  })
}

function onbuilderror (err) {
  if (!err) return
  console.error(err)
  process.exit(2)
}

You could use appveyor to build windows "prebuilds": https://www.appveyor.com/
Config which is similar to what we would need: https://github.com/Level/leveldown/blob/master/appveyor.yml
☁  iohook [master] ⚡ ls -lah prebuilds
drwxr-xr-x  15 koen  staff   510B Mar 22 00:44 .
drwxr-xr-x  19 koen  staff   646B Mar 22 00:44 ..
-rw-r--r--   1 koen  staff    23K Mar 22 00:42 iohook-v0.1.2-electron-v47-darwin-x64.tar.gz
-rw-r--r--   1 koen  staff    23K Mar 22 00:42 iohook-v0.1.2-electron-v48-darwin-x64.tar.gz
-rw-r--r--   1 koen  staff    23K Mar 22 00:43 iohook-v0.1.2-electron-v49-darwin-x64.tar.gz
-rw-r--r--   1 koen  staff    23K Mar 22 00:43 iohook-v0.1.2-electron-v50-darwin-x64.tar.gz
-rw-r--r--   1 koen  staff    23K Mar 22 00:44 iohook-v0.1.2-electron-v51-darwin-x64.tar.gz
-rw-r--r--   1 koen  staff    23K Mar 22 00:44 iohook-v0.1.2-electron-v53-darwin-x64.tar.gz
-rw-r--r--   1 koen  staff    23K Mar 22 00:03 iohook-v0.1.2-node-v11-darwin-x64.tar.gz
-rw-r--r--   1 koen  staff    23K Mar 22 00:03 iohook-v0.1.2-node-v14-darwin-x64.tar.gz
-rw-r--r--   1 koen  staff    23K Mar 22 00:04 iohook-v0.1.2-node-v46-darwin-x64.tar.gz
-rw-r--r--   1 koen  staff    23K Mar 22 00:04 iohook-v0.1.2-node-v47-darwin-x64.tar.gz
-rw-r--r--   1 koen  staff    23K Mar 22 00:04 iohook-v0.1.2-node-v48-darwin-x64.tar.gz
-rw-r--r--   1 koen  staff    23K Mar 22 00:04 iohook-v0.1.2-node-v51-darwin-x64.tar.gz

I have been trying out appveyor on my fork: https://github.com/vespakoen/iohook
It runs the build script, but I guess the CMakeFile needs some love for Windows still?
https://ci.appveyor.com/project/vespakoen/iohook

Hello! looks like you do big work today :)
But I already have build system in project and it not require any additional modules like prebuild.
All what need, setup target and run "npm run compile". Also, module already support automatic selection of prebuilds.
Problem not in building prebuild packages, how to store all needed versions?
In your case, you got 6 electron builds and 6 node builds, but it all only for darwin.
Module should support other linux/windows platforms, and it takes
6(versions) * 3(electron/node/nw.js) * 3(darwin/linux/windows) = 54(prebuild versions)
I can't place all versions in one package. Best way, store all versions in CDN and download it for each installation. I just don't know free CDNs for implementing this.
If you can help with it, I will be happy implement prebuild system in this module. I want to do it long time

Hey WilixLead!
That's great news =)
Fortunately, the builds are pretty small (23K each it seems).
The github releases would be your "free CDN", if you make a git tag, you can later upload files to it on github, or using their API (which is what the "prebuild" script does)
https://github.com/vespakoen/iohook/blob/master/build.js#L87
"prebuild-install" can easily install the .tar.gz's from the github releases for users installing your package (replacing the install.js script)
I was hoping to setup AppVeyor to produce (& "github release") the windows versions for us (x86 & x64)
cmake-js also claims that the builds it produces are "NW.js" compatible out of the box
NW.js: all CMake.js based native modules are compatible with NW.js out-of-the-box, there is no nw-gyp like magic required

Can you point me to the code that handles "module already support automatic selection of prebuilds." ?
I have tried compiling it with Visual Studio Code (the libuiohook github somewhere says it should work) but I haven't had any luck with it, I must also add that my C++ / Windows skills are near non-existant haha)
There is also jNativeHook which also uses libuiohook and uses Msys2 to compile it:
https://github.com/kwhat/jnativehook/wiki/Compiling
Which is what I was going to try next, but I guess I will wait for your response first this time =)

I just has success building libuiohook on windows using msys2, but this is not helping us in any way, we really need to get CMake working for it to produce the .node library, I haven't succeeded in this though so would love to hear you experiences. I also found https://github.com/ayoubserti/node-iohook, which also has a appveyor.yml conf, so I got excited for a second but noticed their builds are failing as well.
So I guess my question is, you claim to have been able to install this package on a windows machine using cmake-js, if this is the case, can you describe the steps you took / tools you used?
If I can get it working myself I'd be glad to help you automatically get those builds up on github / help you setup the prebuild-install script.

@vespakoen  The time I added appveyor hook for continue integration, Appveyor didn't support VS2013 and use a fixed CMake version.
@WilixLead  Thank you for picking my code and continue working. I didn't really find time to fix all known issues. I think I deserve a credit on your module ;)

I bit the bullet and installed the VS Build tools / CMake in a Windows 10 VM and finally was able to compile it myself by using a whopping combination of tools (node / git / msys2 / autoconf / pkg-utils etc).
I have tried out a bunch of configurations but the missing config.h / config.h.in, generated through the ./autogen.sh / ./configure was the missing link to get it to compile.
So I went and found this thing on the internet: https://github.com/schismtracker/schismtracker/blob/master/appveyor.yml
And modified it to our needs here: https://github.com/vespakoen/iohook/blob/master/appveyor.yml
The -Dsnprintf=_snprintf option that is added for certain VS versions seems to be incorrect, I had to remove that to make it work on VS2015 in my Windows 10 (64 bit) VM.
The compilation on AppVeyor with VS 2013 succeeded to build libuiohook (!!), but has a couple of issues with iohook.cc still https://ci.appveyor.com/project/vespakoen/iohook/build/61/job/0jbyavfnfi5gb2ji
I will now see how well it works on node-iohook.
Since both of you guys are here now, can we merge these 2 packages? if so / into which one / can I be a contributor?

I think you are going to like this...
https://github.com/vespakoen/iohook/releases/tag/v0.1.3

Now checking if they actually work... ;)

Just build them on my OSX machine, and published a version to NPM to test it out, first test when installing it on my mac works:
  testing  npm install --save iohook-prebuild-test

><EMAIL_ADDRESS>install /Users/koen/Projects/testing/node_modules/iohook-prebuild-test
> prebuild-install

prebuild-install info begin Prebuild-install version 2.1.1
prebuild-install info looking for local prebuild @ prebuilds/iohook-prebuild-test-v0.1.3-node-v48-darwin-x64.tar.gz
prebuild-install info looking for cached prebuild @ /Users/koen/.npm/_prebuilds/https-github.com-vespakoen-iohook-releases-download-v0.1.3-iohook-prebuild-test-v0.1.3-node-v48-darwin-x64.tar.gz
prebuild-install http request GET https://github.com/vespakoen/iohook/releases/download/v0.1.3/iohook-prebuild-test-v0.1.3-node-v48-darwin-x64.tar.gz
prebuild-install http 200 https://github.com/vespakoen/iohook/releases/download/v0.1.3/iohook-prebuild-test-v0.1.3-node-v48-darwin-x64.tar.gz
prebuild-install info downloading to @ /Users/koen/.npm/_prebuilds/https-github.com-vespakoen-iohook-releases-download-v0.1.3-iohook-prebuild-test-v0.1.3-node-v48-darwin-x64.tar.gz.80561-f813d97878fef.tmp
prebuild-install info renaming to @ /Users/koen/.npm/_prebuilds/https-github.com-vespakoen-iohook-releases-download-v0.1.3-iohook-prebuild-test-v0.1.3-node-v48-darwin-x64.tar.gz
prebuild-install info unpacking @ /Users/koen/.npm/_prebuilds/https-github.com-vespakoen-iohook-releases-download-v0.1.3-iohook-prebuild-test-v0.1.3-node-v48-darwin-x64.tar.gz
prebuild-install info unpack resolved to /Users/koen/Projects/testing/node_modules/iohook-prebuild-test/build/Release/iohook.node
prebuild-install info unpack required /Users/koen/Projects/testing/node_modules/iohook-prebuild-test/build/Release/iohook.node successfully
prebuild-install info install Prebuild successfully installed!
<EMAIL_ADDRESS>/Users/koen/Projects/testing
└─┬<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ ├─┬<EMAIL_ADDRESS>    │ │ ├──<EMAIL_ADDRESS>    │ │ └─┬<EMAIL_ADDRESS>    │ │   ├──<EMAIL_ADDRESS>    │ │   ├──<EMAIL_ADDRESS>    │ │   ├──<EMAIL_ADDRESS>    │ │   ├──<EMAIL_ADDRESS>    │ │   ├──<EMAIL_ADDRESS>    │ │   ├──<EMAIL_ADDRESS>    │ │   └──<EMAIL_ADDRESS>    │ ├──<EMAIL_ADDRESS>    │ ├─┬<EMAIL_ADDRESS>    │ │ ├──<EMAIL_ADDRESS>    │ │ ├──<EMAIL_ADDRESS>    │ │ ├──<EMAIL_ADDRESS>    │ │ ├──<EMAIL_ADDRESS>    │ │ ├─┬<EMAIL_ADDRESS>    │ │ │ ├──<EMAIL_ADDRESS>    │ │ │ └─┬<EMAIL_ADDRESS>    │ │ │   └──<EMAIL_ADDRESS>    │ │ ├─┬<EMAIL_ADDRESS>    │ │ │ └──<EMAIL_ADDRESS>    │ │ └──<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ ├──<EMAIL_ADDRESS>    │ └─┬<EMAIL_ADDRESS>    │   └──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ ├──<EMAIL_ADDRESS>    │ ├──<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ ├──<EMAIL_ADDRESS>    │ ├─┬<EMAIL_ADDRESS>    │ │ └──<EMAIL_ADDRESS>    │ └─┬<EMAIL_ADDRESS>    │   └──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    └──<EMAIL_ADDRESS>
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.
☁  testing  nano index.js
☁  testing  node index.js
{ button: 0, clicks: 0, x: 664, y: 676, type: 'mousemove' }
{ button: 0, clicks: 0, x: 658, y: 679, type: 'mousemove' }
{ button: 0, clicks: 0, x: 652, y: 682, type: 'mousemove' }
{ button: 0, clicks: 0, x: 648, y: 685, type: 'mousemove' }
{ button: 0, clicks: 0, x: 642, y: 688, type: 'mousemove' }

Wow. How many information :)  Sorry, I just will be busy this days.
I can't approve your pull request, it contains changes in packages.json, like
"name": "iohook-prebuild-test"
And why you change "class" to vars? And other es5/6 const/lets and etc?

I changed some stuff to ES5 to make it work in older environments.
I also might have sprinkled some const / lets in there since I mostly write ES6 these days and I guess it has become a habit.
I know the pull is not ready to go, I had to change the package name to be able to publish it to npm for example, the "prebuild-install" package determines the git releases url from the git url in the package json, not the regular one, but the one that NPM adds when you install a package trought it (basically, "npm link" or "npm install git+https://github.com/vespakoen/my-fork-of-your-thing.git" didn't work.
Also there are waaay to many commits from testing out the windows build on AppVeyor (triggering builds through commits)
Anyways, I tested some of the built packages on OSX and Windows and NW.js, they all worked!
(see my fork's the releases page for all the builds I created: https://github.com/vespakoen/iohook/releases)
I did the OSX builds on my own machine, the windows builds on AppVeyor, no linux builds yet (could probably be automated through travis)
Funny thing is, I was trying to use this project for a (Adobe) After Effects extension, which uses NW.js, turns out it doesn't support native extensions, so now I made a new CMakeLists.txt file that builds one of libuiohook's examples into a binary, and I added jsoncpp in there to log things I need to json (If you are interested in the code, I can share), now I simply start my binary as a subprocess and parse the json it outputs on stdout.
I also got rid of using "node-abi" and other weird hacks to get a list of all versions / platforms / abi's (what a mess it is actually!)
and hand crafted a list in "support.js" (https://github.com/vespakoen/iohook/blob/228b5d56942370061a196d79a7eba2af4e242787/support.js)
Hope you can figure it out a bit and take it from here, but I'll be happy to help / answer questions.

@vespakoen Great work! I make manual merge your commits in local machine.
Build and upload works great in MacOS! Cool!
Now I try build linux on ubuntu and after that, try to use your AppVeyor config. I think it will works too.
But, I keep my ES6 style (classes and const/let). It works on older node.js versions and I think we no need support more older then v4.x where it work only with --harmony flag
Great job! After testing I upload all to repo and npm

Cool man, as you wish! Enjoy!

Hello! Sorry for long silent. I be in vocation 🇲🇽 🌮  :)
Now I continue updating module. I made some small fixes, but your job is great. Now I try configure travis-ci and appveior for automatic build publish on push.
Also I plain keep compilation if build is not found. I think it helps to other peoples use this module in specific versions of node/electron/etc..

Ok. I push new version with released prebuild system. Still need some updates for support more environments like io.js/nw.js and some older versions of node.js


Autofocus the title field

This is very useful when using shortcuts (eg: c) - it means you don't have to touch the mouse at all to start typing.

Thank you :+1:

Tagged on 1.4.0 (https://github.com/AsgardCms/Menu/releases/tag/1.4.0)


[Component Proposal] Marketplace: Product item with Thumbnail

Is this an extension to an existing component, pattern, or a brand new asset?
Item from the EDS.
What is the rationale for this contribution.
A compound component for displaying resources inside a tile catalog wrapper to easier, more attractive browsing than tabular forms.
Detail the intended behaviour
A sister to the 'Product item with Icon' component as well as the 'featured tile' component, this displays a resource on a tile with a small thumbnail, title, subtitle and tertiary button, including a hover state when users hover over the thumbnail.
Example in the PAL:

It can be found as the second component at this link: https://ibm.invisionapp.com/share/U2NYW8D7DFV#/screens/319196743
Example from other products
Inside the PAL patterns - for the Digital Twin marketplace:

And again coupled with the featured tile in the AS function catalog design proposals:

Note: this is a sibling issue to:

[Contribution] Marketplace Featured Tile #133
[Contribution] Product item with Icon (example: file) #136
[Tile catalog] Update/deprecate/evolve into wrapper for catalog type patterns. #137

Seems like a very specific card type that would be better maintained at the app level


RuntimeError: Input type (float) and bias type (c10::Half) should be the same

Problem here:
Post-processing
video = self.decode_latents(latents)
File "/dsk1/ivan/AnimateDiff/animatediff/pipelines/pipeline_animation.py", line 419, in call
video = self.decode_latents(latents)
File "/dsk1/ivan/AnimateDiff/animatediff/pipelines/pipeline_animation.py", line 246, in decode_latents
video.append(self.vae.decode(latents[frame_idx:frame_idx+1]).sample)

Install this:
pip install xformers
pip install accelerate


Add negative filter option

The current filter option allows:
# Match all songs by Muse
script.py -f "artist:Muse"

Negative filter option allows:
# Match all songs NOT by Muse
script.py -F "artist:Muse"

# Match all songs not by Muse OR from an album with Blackholes in the name
script.py -F "artist:Muse -F "album:Blackholes"

The option would work alongside the positive filter option in the form of  'and not' or but not:
# Match all versions of the song "Yesterday" NOT by The Beatles (aka any of the couple thousand covers)
script.py -f "title:Yesterday" -F "artist:Beatles"

It could possibly have its own 'all' option (-A), though I haven't really thought of any useful cases for it. If anyone can think of a good example use of it, let me know.

The base funcitonality will be provided by the new standalone wrapper in this issue


Do not activate completion after tab or space

Hey! sometimes I just want to indent without triggering autocompletion. For example If I write a letter, quit the autocompletion menu and start pressing tab the autocompletion menu will appear again. I believe this would be done with the regex in config but I'm not so sure how to achieve it.. any help? thanks in advance!

Sorry for the late response, this worked perfectly fine! thanks. Just some notes:
I ended with this function for the get_trigger_characters property:
local function get_trigger_characters(trigger_characters)
   local filter_characters = function(char)
      return char ~= ' ' and char ~= '\t'
   end
   return vim.tbl_filter(filter_characters, trigger_characters)
end

since I needed also tab to be skipped from triggering the autocompletion.
And an additional comment is that I was having this issue with javascript, but I guess this config overrides it for all language servers and that's what I wanted.


Remove not needed step for port forwarding for UI E2E tests

Closing as it seems it cannot be done this way.


SSLV3_ALERT_HANDSHAKE_FAILURE for istio virtual service

Bug Description
I am trying to deploy the following VirtualService:
apiVersion: networking.istio.io/v1beta1
kind: VirtualService
metadata:
  creationTimestamp: "2023-03-16T18:50:14Z"
  generation: 5
  name: shopchat
  namespace: shopchat
  resourceVersion: "470270779"
  uid: 3c0e43a3-4063-4da4-9f71-5ad9e69e59b2
spec:
  gateways:
  - default/webapp-gateway
  hosts:
  - dashboard.scale.com
  http:
  - match:
    - uri:
        prefix: /shopchat/api
    rewrite:
      authority: widget.ecommca.co
      uri: /
    route:
    - destination:
        host: widget.ecommca.co
        port:
          number: 443
  - match:
    - uri:
        prefix: /shopchat
    rewrite:
      authority: e-commerce-widget.web.app
      uri: /
    route:
    - destination:
        host: e-commerce-widget.web.app
        port:
          number: 443

with the following respective ServiceEntry and DestinationRule
apiVersion: networking.istio.io/v1beta1
kind: ServiceEntry
metadata:
  creationTimestamp: "2023-03-16T18:36:02Z"
  generation: 8
  name: ecommca-widget
  namespace: shopchat
  resourceVersion: "471322369"
  uid: 49223229-eae1-440c-9d59-8d01a175320d
spec:
  hosts:
  - widget.ecommca.co
  location: MESH_EXTERNAL
  ports:
  - name: tls
    number: 443
    protocol: TLS
  resolution: DNS

apiVersion: networking.istio.io/v1beta1
kind: DestinationRule
metadata:
  creationTimestamp: "2023-03-16T18:50:14Z"
  generation: 8
  name: ecommca-widget
  namespace: shopchat
  resourceVersion: "470379901"
  uid: 72870ff3-2053-4e48-9e96-d5fc9d3a1c10
spec:
  host: widget.ecommca.co
  trafficPolicy:
    loadBalancer:
      simple: ROUND_ROBIN
    portLevelSettings:
    - port:
        number: 443
      tls:
        mode: SIMPLE

but when i hit curl -kv https://dashboard.scale.com/shopchat/api i get:
*   Trying <IP_ADDRESS>:443...
* Connected to dashboard.scale.com (<IP_ADDRESS>) port 443 (#0)
* ALPN: offers h2
* ALPN: offers http/1.1
* (304) (OUT), TLS handshake, Client hello (1):
* (304) (IN), TLS handshake, Server hello (2):
* (304) (IN), TLS handshake, Unknown (8):
* (304) (IN), TLS handshake, Certificate (11):
* (304) (IN), TLS handshake, CERT verify (15):
* (304) (IN), TLS handshake, Finished (20):
* (304) (OUT), TLS handshake, Finished (20):
* SSL connection using TLSv1.3 / AEAD-CHACHA20-POLY1305-SHA256
* ALPN: server accepted h2
* Server certificate:
*  subject: C=US; ST=California; L=San Francisco; O=Cloudflare, Inc.; CN=scale.com
*  start date: May  4 00:00:00 2022 GMT
*  expire date: May  4 23:59:59 2023 GMT
*  issuer: C=US; O=Cloudflare, Inc.; CN=Cloudflare Inc ECC CA-3
*  SSL certificate verify ok.
* Using HTTP2, server supports multiplexing
* Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0
* h2h3 [:method: GET]
* h2h3 [:path: /shopchat/api]
* h2h3 [:scheme: https]
* h2h3 [:authority: dashboard.scale.com]
* h2h3 [user-agent: curl/7.86.0]
* h2h3 [accept: */*]
* Using Stream ID: 1 (easy handle 0x148011e00)
> GET /shopchat/api HTTP/2
> Host: dashboard.scale.com
> user-agent: curl/7.86.0
> accept: */*
>
* Connection state changed (MAX_CONCURRENT_STREAMS == 256)!
< HTTP/2 503
< date: Fri, 17 Mar 2023 02:37:40 GMT
< content-type: text/plain
< content-length: 296
< cf-cache-status: DYNAMIC
< strict-transport-security: max-age=2592000; preload
< x-content-type-options: nosniff
< server: cloudflare
< cf-ray: 7a91daf73bdb17a1-EWR
< alt-svc: h3=":443"; ma=86400, h3-29=":443"; ma=86400
<
* Connection #0 to host dashboard.scale.com left intact
upstream connect error or disconnect/reset before headers. retried and the latest reset reason: connection failure, transport failure reason: TLS error: 268436496:SSL routines:OPENSSL_internal:SSLV3_ALERT_HANDSHAKE_FAILURE 268435610:SSL routines:OPENSSL_internal:HANDSHAKE_FAILURE_ON_CLIENT_HELLO%

i am not very familiar with networking. Is there something i am missing here?
Version
➜  kube kubectl version --short
Client Version: v1.23.10
Server Version: v1.23.14-eks-ffeb93d

Additional Information
No response

looks like https://istio.io/latest/docs/ops/common-problems/network-issues/#double-tls

i tried:
apiVersion: networking.istio.io/v1beta1
kind: ServiceEntry
metadata:
  creationTimestamp: "2023-03-16T18:36:02Z"
  generation: 9
  name: ecommca-widget
  namespace: shopchat
  resourceVersion: "473977206"
  uid: 49223229-eae1-440c-9d59-8d01a175320d
spec:
  hosts:
  - widget.ecommca.co
  location: MESH_EXTERNAL
  ports:
  - name: http
    number: 443
    protocol: HTTP
  resolution: DNS

and i get the same error
i also tried:
apiVersion: networking.istio.io/v1beta1
kind: ServiceEntry
metadata:
  creationTimestamp: "2023-03-16T18:36:02Z"
  generation: 10
  name: ecommca-widget
  namespace: shopchat
  resourceVersion: "473991911"
  uid: 49223229-eae1-440c-9d59-8d01a175320d
spec:
  hosts:
  - widget.ecommca.co
  location: MESH_EXTERNAL
  ports:
  - name: http
    number: 80
    protocol: HTTP
    targetPort: 443
  resolution: DNS

and that gives me a 503 weirdly


Make more levels

Maybe 30 will do
Might need more mechanics to make this work (like a void tile?)

a void tile.
maybe a teleport tile

The levels need to teach players tricks to prevent difficulty curve being too steep and to remove the need to give lots of instructions

folders pass through the opposite end of factories

cross intersection

can pass through factories even if not needed
use reverse belt to pass folders back through the same factory
using gates as substitutes for belts (straight through, turning, reverse)
how to use minimal amount of belts (gates at the last minute)
use gates as two turns (turn one way when it has a job, turn the other when that job is done)

Should wait until #8 before adding more levels


"checked" getter/setter

Hi,
I have added checked getter/setter. This should also resolve issue #121.
Regards, Ognjen

Excellent. Merging now...
The three commits I made were the following:

I changed back the version of symfony/css-selector, as it was enforcing developers to use a more modern version of PHP. I am dropping the dependency of symfony/css-selector by implementing the task of CSS to XPath queries in a much simpler form, here: github.com/PhpGt/CssXPath
Added a comment linking to issue #161 where we should be able to use $this->form to obtain a reference to the input element's containing form.
Checking the HTML specification document, it seems that any value is permitted as the value for the "checked" attribute, including an empty string.

Once again, thanks so much Ognjen, your contributions are greatly appreciated.

Great. You are welcome.


Config reader should support FiniteDuration

Duration was one of the harder codecs to copypaste from pureconfig, at this point we might wanna think about just depending on it...

It's not neccessary to copypaste.
We should support simple forms like:
1 s, 1s, 100ms, 100 ms, etc.


Update to work with new legacy fabric

Legacy Fabric's Intermediaries changed in Dec 2021 (thus making any mod built before unusable) so this updates it

@BoogieMonster1O1 Thanks for the PR, sorry I haven't merged it until now!


Support the other variant of unconventional-import-alias

🚀  Feature request
Currently, ruff supports enforcing importing packages/modules under aliases. It would be, however, nice to have a feature to forbid some aliases which might be considered non-standard at some companies.
For example, with a hypothetical configuration
[tool.ruff.flake8-import-conventions.forbidden-aliases]
"tensorflow.keras.backend" = "K"

the following import
>>> import tensorflow.keras.backend as K

should raise an error.
Thanks a lot for all your hard work. Ruff is an amazing tool ❤

Ooh, I almost replied saying we support this via aliases, but you're talking about the inverse: ban certain aliases. That makes sense to me! (And thank you for the kind words :))


Need a inference code for yolo-nas , so that i can get output in x1,y1,x2,y2 in terms of bounding box , along with the conf

I'm using Opencv , and i need to be able to put my image frame from the opencv which ofcourse is in numpy.
But the abstracted code you provide don't seem to give an option to get output in the form of x's and y's and conf score ,
Or it might be hiden in some different Class hiearchy , i'm not sure , cause i could not find it . would be very helpful if you could
help with it .
Arigato

Hi @abhinavisgood, it sounds similar to this issue, am I right?
Feel free if you have more questions :)

duplicated #907


Enh: allow to give install and global option to underlying pip of rez…

@nerdvegas  I wish this would work directly but strangely the argument parser seems to miss the argument I give, like with :
rez-pip --install PySide --install-option --qmake=/usr/bin/_qmake
I get :
rez pip: error: argument --install-option: expected one argument
seems related to LazyArgumentParser and 'co but I don't find what's the problem..

Hmm, actually I have started a quite better change/PR than this one:
I add --pip-before-option and --pip-option, which can be used to specify any pip argument(s), and not only the install-option and global-option ones..
will push it ASAP..


Crash while extension view and preferences json are open at the same time

Type: Bug
This is a weird one,
when opening an extension's view (by selecting an installed extension to see the details) and searching in it (Ctrl+F), then opening the preferences (Shift+Ctrl+P -> User Preferences JSON) and then going back to the extension tab, the details view is empty and vscode immediatly crashes without a warning or anything.
This is reproducible on my machine, but not on two others I tried.
I tried the --disable-extensions option but it resulted in a crash as well.
VS Code version: Code 1.76.0 (92da9481c0904c6adfe372c12da3b7748d74bdcb, 2023-03-01T10:22:44.506Z)
OS version: Windows_NT x64 10.0.19044
Modes: Restricted
dmp.zip
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz (8 x 2592)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.82GB (5.21GB free)

Process Argv
--crash-reporter-id b83564e7-5db5-47e7-86b0-ee22ee0628a5

Screen Reader
no

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

vscode-markdownlint
Dav
0.49.0

vscode-docker
ms-
1.24.0

remote-containers
ms-
0.282.0

hexeditor
ms-
1.9.11

vscode-yaml
red
1.11.0

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
pynewext54:30669237
azure-dev_surveyone:30548225
pyindex848:30662994
nodejswelcome1cf:30587006
3biah626:30602489
pyind779:30671433
f6dab269:30613381
pythonsymbol12:30671437
a9j8j154:30646983
vsccsb:30677849
pythonb192cf:30669361
funwalk2cf:30676044

I added the .dmp file to the original post, the execution used the --disable-extensions option. hope this helps.

There's a crash in the find impl
symbolicated.log

I tested it just now with 1.76.1, issue is still present.

Merging to https://github.com/microsoft/vscode/issues/177046


Client interface "runner", execution modules case

Hello,
i'm trying to use "runner" interface provided by a plugin for initiating some tasks, that are intendent to run on a salt-master, for example docker.pull execution module.
Currently I'm getting an error because plugin is always sending 'mods' argument, even if it is empty:
Sending JSON: {"client":"runner","mods":"","fun":"salt.cmd","arg":["docker.pull","nginx:latest"],"kwarg":{}}
Received response: ["Exception occurred in runner salt.cmd: Traceback (most recent call last):\n  File \"/usr/lib/python2.7/dist-packages/salt/client/mixins.py\", line 382, in _low\n    data['return'] = self.functions[fun](*args, **kwargs)\n  File \"/usr/lib/python2.7/dist-packages/salt/runners/salt.py\", line 69, in cmd\n    utils=utils(__opts__)).get(fun)(*args, **kws)\nTypeError: pull() got an unexpected keyword argument 'mods'\n"]

Some modules ignores it and execute correctly, like docker.images:
Sending JSON: {"client":"runner","mods":"","fun":"salt.cmd","arg":"docker.images","kwarg":{}}
Received response: [{"sha256:e548f1a579cf529320a3c1d8789399e5fea0bfaa04cbb70d03890afafb748a2f":{"Labels":{"maintainer":"NGINX Docker Maintainers <docker-maint@nginx.com>"},"Time_Created_Local":"2018-02-21 01:52:15 MSK","VirtualSize_Human":"103.6 MiB","Size_Human":"103.6 MiB","VirtualSize":108664124,"SharedSize":-1,"ParentId":"","Time_Created_Epoch":1519167135,"RepoTags":["nginx:latest"],"RepoDigests":["nginx@sha256:4771d09578c7c6a65299e110b3ee1c0a2592f5ea2618d23e4ffe7a4cab1ce5de"],"Containers":-1,"Size":108664124}}]

I can reproduce the request with curl, excluding 'mods', which is making it execute correctly:
# curl -sSk https://localhost:8000 -H 'X-Auth-Token: 6f22e4abace6692d2ca1b64df698edee933f0339' -d client=runner -d fun=salt.cmd -d arg=docker.pull -d arg="nginx:latest"
{"return": [{"Status": "Image is up to date for nginx:latest", "Time_Elapsed": 3.1092309951782227}]}

Is there any way to make 'mods' field in runner client interface disablable?
Thanks :)

Thanks for the report. Chalking this up as a bug. I'll investigate and see

This was just corrected and released as 3.1.6. It was just uploaded, and should reach the Jenkins mirrors over the next 24 hours.
The mods key will now only be submitted to the salt-api when it contains data.


Compatibility with Rollup

Expected behavior
A passing build, as suggested in the documentation.
Observed behavior
Rollup throws an error:
Error: 'default' is not exported by node_modules/bugsnag-js/dist/bugsnag.js
Steps to reproduce

Create a file, e.g. bugsnag.js
import bugsnag from 'bugsnag-js';
Build the file with Rollup
Rollup throws the error above

Version
4.6.3
Additional information
I use rollup-stream with rollup-plugin-node-resolve to work with packages installed through NPM.
Can't comment on Issues?
Some users have been unable to comment on Github issues when an adblocker extension is enabled.
We recommend temporarily disabling the extension, or if that fails, contacting<EMAIL_ADDRESS>

Hi @pieterbeulque, you'll need to add rollup-plugin-commonjs to your rollup build. We do export a default property but I don't know why rollup won't pick it up.

That was it! Thanks a lot.


Add Aprisyta to Contributors list

Thank You for helping me make my first contribution.

@Aprisyta, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added Facebook share and tweet feature to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


Optimize flattenBySketchAllClass()

Could probably be made more efficient. Low priority.

Specifically less use of array spread operator


assertion failures seen in fetcher

While examining log files during testing I saw:
2023-10-19 04:17:22,644 | INFO | scrapy.extensions.throttle | slot: noticias.uol.com.br | conc: 1 | delay: 5000 ms (+0) | latency:  303 ms | size:   456 bytes
2023-10-19 04:17:22,645 | DEBUG | scrapy.core.engine | Crawled (403) <GET https://noticias.uol.com.br/politica/ultimas-noticias/2023/10/17/relatorio-da-cpi-de-81-pede-indiciamento-de-bolsonaro-por-tentar-golpe.htm> (referer: None)
2023-10-19 04:17:22,747 | ERROR | scrapy.core.scraper | Spider error processing <GET https://noticias.uol.com.br/politica/ultimas-noticias/2023/10/17/relatorio-da-cpi-de-81-pede-indiciamento-de-bolsonaro-por-tentar-golpe.htm> (referer: None)
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/twisted/internet/defer.py", line 892, in _runCallbacks
    current.result = callback(  # type: ignore[misc]
  File "/app/indexer/workers/fetcher/batch_spider.py", line 81, in on_error
    self.cb(story)
  File "/app/indexer/workers/fetcher/fetch_worker.py", line 138, in scrapy_cb
    assert http_meta.final_url is not None
AssertionError

Perhaps both assertions in indexer/workers/fetcher/batch_spider.py should be if statements that set status_label to make the events visible?

Is there a counter for "failed to make a connection"?  rss-fetcher splits this out (in perhaps too many counters).
A counter for "too many redirects"?

I'm pretty sure the assert failures are a thing of the past, BUT I'd still be interested in seeing counters for "connection failed" and "too many redirects"; so perhaps this bug should be re-titled?

No evidence that the problem still exists (checked logs from all 20 batches of last fetch).
Closing.


Short solution needed: "QDA" (python-scikit-learn)

Please help us write most modern and shortest code solution for this issue:
QDA (technology: python-scikit-learn)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

Using QDA classifier example


Adds SkaffoldDownloader.

Downloads the latest release of skaffold by operating system.

Btw, feel free to take and work on any of the issues @GoogleContainerTools/java-tools

Other than the exception thing I think this looks good. Are you planning on making any changes to this?


"MP4 encoder is not available on this platform" on MacOS

This appeared when I'm using FrameCapturer on MacOS, Unity 2017.0f1. But I can see that there are some plugins, and I guess that fccore.bundle is for MacOS?

MP4 recording is available only on Windows for now.

Thanks for replying! We are making videos with Unity on MacOS. Is there any good way of output video(.mp4)?

any other recorders should work. you can record as exr sequence (best for non-realtime project) or webm video and covert it to mp4 by other tools (e.g. ffmpeg),

Is there any document of FrameCapturer which can explain each and every params in details? Especially those in encoder settings.

BTW, is there any good document for making video by Unity? Including FrameCapturer usage and ffmpeg to output high quality video. I think "Adam" is awesome but I just cannot output this kind of extremely great quality.

there is no proper document for now, but FrameCapturer exposes only basic and generic encoder parameters. I think ffmpeg's document can explain most of it.
Adam is recorded by internal tools and  encoded with ffmpeg.

I see. Is there any great group in which there are people who want to create video with Unity that I can join?

I have no idea.


fix parsing annotations data for CR

Description
Changes proposed in this pull request:

parsing data from yaml to put it to jsonpath.

Related issue(s)

/retest

/retest


DBZ-7067 Improve Oracle logging for COMMITs

https://issues.redhat.com/browse/DBZ-7067

/packit test --labels oracle

Checked the XStream test locally and it passed.
As this is a trivial change, I'm going to proceed and merge this myself.


Concurrent top-level detection

This PR gets the semantics for detecting when to make top-level code a concurrent context. For compatibility reasons, we want scripts written in top-level code without concurrency today in Swift 5 to work exactly the same way when the async top-level lands. To ensure this, the top-level needs to detect when concurrency features are being used before behaving as an async context. The implementation of this is to detect when an await keyword has been used before switching the top-level code context to an asynchronous context. If there are no suspension points directly in the top-level, the context is not asynchronous.
The implications of making the top-level code an asynchronous context include overload resolution behavior and semantics around spinning up a runloop in the implicit async_Main function. These two changes to behavior could cause unintended script breakage. Thus, if there are no await calls made, the top-level will behave exactly as it did before.

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 9c02a3e27ccec84fbfdc6f2216bbf8e10038574f

@swift-ci please test macos

Build failed
Swift Test OS X Platform
Git Sha - 9c02a3e27ccec84fbfdc6f2216bbf8e10038574f

@swift-ci please test macOS


tf.keras.layers.DepthwiseConv2D and tf.keras.layers.SeparableConv2D does not handle kernel regularizer correctly.

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug, a feature request, or a significant problem with the
documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go
here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script
provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): CentOS 7.3
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue
happens on a mobile device: NA
TensorFlow installed from (source or binary): binary, from anaconda
TensorFlow version (use command below): 2.2.0
Python version: 3.7.7
Bazel version (if compiling from source): NA
GCC/Compiler version (if compiling from source): NA
CUDA/cuDNN version: 10.1 / 7.6.5
GPU model and memory: NVIDIA Tesla V100 / 32GB
Exact command to reproduce: Source code below

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with:
python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
kernel_regularizer parameter on tf.keras.layers.DepthwiseConv2D and  tf.keras.layers.SeparableConv2D does not register the loss to model losses, despite of bias_regularizer works. I'll describe full test code below.
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.
Expected case 1: Dense layer
m = tf.keras.models.Sequential()
m.add(tf.keras.layers.Dense(3, input_dim=5, kernel_regularizer=tf.keras.regularizers.l2(1.0E-01), bias_regularizer=tf.keras.regularizers.l2(1.0E-01)))
m.add(tf.keras.layers.Dense(3, input_dim=5, kernel_regularizer=tf.keras.regularizers.l2(1.0E-01)))
print(m.losses)

Output:
[<tf.Tensor: shape=(), dtype=float32, numpy=0.19434488>,
 <tf.Tensor: shape=(), dtype=float32, numpy=0.0>,
 <tf.Tensor: shape=(), dtype=float32, numpy=0.22874394>]

Expected case 2: Conv2D layer
m = tf.keras.models.Sequential()
m.add(tf.keras.layers.Input(32, 32, 3))
m.add(tf.keras.layers.Conv2D(3, 3, kernel_regularizer=tf.keras.regularizers.l2(1.0E-01), bias_regularizer=tf.keras.regularizers.l2(1.0E-01)))
m.add(tf.keras.layers.Conv2D(3, 3, kernel_regularizer=tf.keras.regularizers.l2(1.0E-01)))
print(m.losses)

Output:
[<tf.Tensor: shape=(), dtype=float32, numpy=0.29766324>,
 <tf.Tensor: shape=(), dtype=float32, numpy=0.0>,
 <tf.Tensor: shape=(), dtype=float32, numpy=0.7301255>]

Expected case 3: ConvLSTM2D layer
m = tf.keras.models.Sequential()
m.add(tf.keras.layers.Input(3, 32, 32, 3))
m.add(tf.keras.layers.ConvLSTM2D(3, 3, return_sequences=True, kernel_regularizer=tf.keras.regularizers.l2(1.0E-01), bias_regularizer=tf.keras.regularizers.l2(1.0E-01)))
m.add(tf.keras.layers.ConvLSTM2D(3, 3, kernel_regularizer=tf.keras.regularizers.l2(1.0E-01)))
print(m.losses)

Output:
[<tf.Tensor: shape=(), dtype=float32, numpy=0.47271797>,
 <tf.Tensor: shape=(), dtype=float32, numpy=0.3>,
 <tf.Tensor: shape=(), dtype=float32, numpy=0.49562755>]

Does not expected case 1: DepthwiseConv2D layer
m = tf.keras.models.Sequential()
m.add(tf.keras.layers.Input(32, 32, 3))
m.add(tf.keras.layers.DepthwiseConv2D(3, 3, kernel_regularizer=tf.keras.regularizers.l2(1.0E-01), bias_regularizer=tf.keras.regularizers.l2(1.0E-01)))
m.add(tf.keras.layers.DepthwiseConv2D(3, 3, kernel_regularizer=tf.keras.regularizers.l2(1.0E-01)))
print(m.losses)

Output:
[<tf.Tensor: shape=(), dtype=float32, numpy=0.0>]

Does not expected case 2: SeparableConv2D layer
m = tf.keras.models.Sequential()
m.add(tf.keras.layers.Input(32, 32, 3))
m.add(tf.keras.layers.SeparableConv2D(3, 3, kernel_regularizer=tf.keras.regularizers.l2(1.0E-01), bias_regularizer=tf.keras.regularizers.l2(1.0E-01)))
m.add(tf.keras.layers.SeparableConv2D(3, 3, kernel_regularizer=tf.keras.regularizers.l2(1.0E-01)))
print(m.losses)

Output:
[<tf.Tensor: shape=(), dtype=float32, numpy=0.0>]

Am I have a mistake using regularizer with these layers?

SeparableConv1D also has same issue.
Case Conv1D:
m = tf.keras.models.Sequential()
m.add(tf.keras.layers.Input(32, 3))
m.add(tf.keras.layers.Conv1D(3, 3, kernel_regularizer=tf.keras.regularizers.l2(1.0E-01), bias_regularizer=tf.keras.regularizers.l2(1.0E-01)))
m.add(tf.keras.layers.Conv1D(3, 3, kernel_regularizer=tf.keras.regularizers.l2(1.0E-01)))
print(m.losses)

Output:
[<tf.Tensor: shape=(), dtype=float32, numpy=0.30436406>,
 <tf.Tensor: shape=(), dtype=float32, numpy=0.0>,
 <tf.Tensor: shape=(), dtype=float32, numpy=0.31131223>]

Case SeparableConv1D:
m = tf.keras.models.Sequential()
m.add(tf.keras.layers.Input(32, 3))
m.add(tf.keras.layers.SeparableConv1D(3, 3, kernel_regularizer=tf.keras.regularizers.l2(1.0E-01), bias_regularizer=tf.keras.regularizers.l2(1.0E-01)))
m.add(tf.keras.layers.SeparableConv1D(3, 3, kernel_regularizer=tf.keras.regularizers.l2(1.0E-01)))
print(m.losses)

Output:
[<tf.Tensor: shape=(), dtype=float32, numpy=0.0>]

@jyhpsycho  Have you test it on tf.keras.layers.Conv2D and tf.nn.depthwise_conv2d ?

@jyhpsycho
Request you to share colab link or complete code snippet with supporting files to reproduce the issue in our environment.It helps us in localizing the issue faster.Thanks!

@ravikyram
Oh, I found that some type on my examples(shape of input layer should be tuple, but not). I'll upload IPython notebook file and converted .py script for reproduce it.
41093.zip

I have tried in colab with TF versions 2.2, nightly versions and was able to reproduce the issue.Please, find the gist here.Thanks!

Please pay attention that according to the docs there is no kernel_regularizer parameter in DepthwiseConv2D, but there is depthwise_regularizer: https://www.tensorflow.org/api_docs/python/tf/keras/layers/DepthwiseConv2D
The problem would be then that while keras accepts the both, it does not apply kernel_regularizer without actually warning the user.

@mkarpushin-upstride The document seems to be changed after v2.3 release. On v2.2 there's no depthwise_regularizer. The end-user cannot know what the option name should be used for specifying regularizer at that time, even if depthwise_regularizer works with v2.2.
https://www.tensorflow.org/versions/r2.2/api_docs/python/tf/keras/layers/ConvLSTM2D
BTW, if depthwise_regularizer works on 2.2 and further version, this issue can be closed.

https://colab.research.google.com/gist/jyhpsycho/3fd5a2d8ff6071c013137a69af6acb18/untitled92.ipynb
I checked the depthwise_regularizer option (based on @ravikyram's colab code) and it seems to work with TF 2.2. @mkarpushin-upstride Thank you for clarification.
If there's any problem with test code, feel free to reopen this issue.


Initializable: initializer modifier multiple inheritance broken by _isConstructor() check

Require check now includes _isConstructor() check which will never be true with upgradeable contracts as we don't use their constructors. I assume this has been accidentally integrated over from @openzeppelin/contracts repo.

💻 Environment

📝 Details

🔢 Code to reproduce bug

Hello @playmint-jbr
This is on purpose. While we do not use constructors to initialize the upgradeable contracts themselves, we do perform initialization, using an upgradeable contract's logic, from the constructor of a Proxy.
This check ensure that if a proxy is currently being constructed, and that the initializer is called as part of this construction, then the call is valid. One particular thing to note is that reentrancy cannot happen under such circumstances, because you cannot call a contract being constructed (the proxy).

So you broke multiple inheritance on purpose?
On Wed, 12 Jan 2022, 19:06 Hadrien Croubois, @.***>
wrote:

Hello @playmint-jbr https://github.com/playmint-jbr
This is on purpose. While we do not use constructors to initialize the
upgradeable contracts themselves, we do perform initialization, using an
upgradeable contract's logic, from the constructor of a Proxy.
This check ensure that if a proxy is currently being constructed, and that
the initializer is called as part of this construction, then the call is
valid. One particular thing to note is that reentrancy cannot happen under
such circumstances, because you cannot call a contract being constructed
(the proxy).
—
Reply to this email directly, view it on GitHub
https://github.com/OpenZeppelin/openzeppelin-contracts-upgradeable/issues/142#issuecomment-1011363059,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVTIKJ5CKBJF42P53AHVVGTUVXGMPANCNFSM5LZNC3AA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
<OpenZeppelin/openzeppelin-contracts-upgradeable/issues/142/1011363059@
github.com>

Have you checked this release note?

And the Initializable contract present in @openzeppelin/contracts is targeted at being used with upgradeable contracts. It is by design that it's the exact same code on both versions of the repo.

Can you point me to an example where the constructor of a proxy calls an initializer from the constructor? All of the examples I can find in this repo have the initializer as a separate function which is called after construction.

Never mind, found an example here https://eips.ethereum.org/EIPS/eip-1822#erc-20-token


Sparkc 412 master

Build against Scala 2.11 finished with success

Build against Scala 2.10 finished with failure

retest this please

Build against Scala 2.11 finished with success

Build against Scala 2.10 finished with success

retest this please

Build against Scala 2.11 finished with success

Build against Scala 2.10 finished with success

retest this please

Build against Scala 2.11 finished with success

Build against Scala 2.10 finished with success

retest this please

Build against Scala 2.11 finished with success

Build against Scala 2.10 finished with success


Rename to MAVLink SDK

There are some concerns with the "Dronecode" branding of the project, such as:

Confusion between the Dronecode Foundation and the SDK
Confusion on slack (#sdk vs #dronecode)
Makes it sound like it is limited to PX4 and may be offensive to other communities (even though there is some work regarding adding support for APM in #728)

The proposition here is to rename it to "MAVLink SDK", as it is a MAVLink implementation and a high-level API in different languages. As a consequence, the project would move to the MAVLink organization.
And RFC has been open on the MAVLink repo to see if that would be acceptable from the MAVLink side.
Please feel free to share your opinion and discuss it here :blush:!

This is a good clarification and moving it to MAVLink sounds like the best solution. I'd like to add that as a MAVLink SDK it will need MAVLink command passthrough functions to provide complete MAVLink functionality.

I'd like to add that as a MAVLink SDK it will need MAVLink command passthrough functions to provide complete MAVLink functionality.

That's not the intent and not easily possible. The intent is to provide an SDK with certain functionality which uses MAVLink under the hood for the communcation to the drone. However, it does not necessarily expose the MAVLink API. If the raw MAVLink API is required, an implementation of MAVLink in a specific programming language should be used instead.
For the C++ library it is possible to export the C API as a passthrough / backdoor. However, if this API would be passed on to the language bindings, either all of the MAVLink messages would need to be added to the protobuf messaging or everything would need to be assembled at runtime which would require the SDK to parse the whole xml MAVLink definitions.

Thanks @laurenwinter for the comment!
I agree with Julian, though, and I would like to add that IMO, the passthrough is not a very nice way to provide "complete MAVLink functionality" (as in: not super elegant, not community-friendly, and soon not faster). Let's compare what it (currently) takes to add a feature to the SDK and what it takes to use the passthrough (again, that's my opinion and maybe I am missing something there):
SDK

Define the API as a proto definition
Implement the mavlink logic in a C++ plugin
Implement the gRPC server boilerplate to bring that to the language bindings (Swift, Python, Java, ...)
Auto-generate the language bindings

Passthrough
A. Define the interface for the new features (not necessary, but any readable piece of code would likely do it: that is about putting one feature in one function)
B. Implement the mavlink logic through the passthrough
Note that:

1 is trivial and corresponds to A.
2 corresponds to B.
3 is currently "boilerplate" to write, but we are working on having that auto-generated (that's one of the top priorities). Once finished, it will be merged with 4.
4 comes almost for free ("just run the auto-generation script").

As a conclusion, I understand that the passthrough may seem faster to implement now (because of 3), but we are putting a lot of effort into removing the costs related to 3.
Also, what is implemented into the SDK is "contributed" to the community in all the supported languages, whereas whatever is written with the passthrough will never come back upstream, and is technically limited to C++. And in that sense, using the passthrough goes "against" the community.
Not sure if that makes sense, but under those conditions, I don't see a use-case that actually wants the mavlink passthrough. What you want is the full auto-generation, and we are working hard on that! :blush:.

Back to the topic of this issue, it seems to have more activity on the MAVLink RFC, so let's close this one in favor of the RFC.


Need Write Access Please

Hi! I have some code ready to push - need write access to this new repo. Thanks!
After that I will hax ur passwords! haha get rekt scrub also I know ur ip sp threr

I gave you permission, but next time please watch your grammar and don't threaten us programmers at the GA Software Creation Facility. As Lead Senior Application Integration Engineer I do have permission to block you from accessing our software.
CONSIDER THIS A WARNING

LOL LOL LOL 😂 😆 🤣

I'm dying

➜  GASOS git:(master) git push
remote: Permission to germantownacademySOS/GASOS.git denied to sfraser.
fatal: unable to access 'https://github.com/germantownacademySOS/GASOS/': The requested URL returned error: 403
➜  GASOS git:(master)

😡

Start a join.me

no wait

click this

nice try

What?

was relative URL - went to github

ugh

Click it again

I'll join it I know where you are

Still getting this when I press the "Create a file" button on this repo:

You’re creating a file in a project you don’t have write access to. Submitting a change will create the file in a new branch in your fork sfraser/GASOS, so you can send a pull request.

Yay you fixed it! I got the invitation and accepted - thanks! 👍🏽😁


Screens on both devices stay on during entire transfer

Describe the bug
I sent a large file (~1 GB) between 2 Android devices. The screen stayed on during the entire transfer duration. This is obviously bad for battery.
To reproduce

Send a very large file.
Wait for the screen to time out as per system settings.

Expected behavior
The screen should turn off after the expected time out period. The transfer can continue with the screen off.
Screenshots
No response
Desktop (please complete the following information)
No response
Smartphone (please complete the following information)

Version: 1.10.0

Additional context
The long-term solution is #275, but a simple mitigation is just allowing the screen to turn off.

This behaviour is on purpose to avoid any interuptions of the file transfer. This behaviour is vendor dependent though so I don't know how we should fix this

I've been using https://play.google.com/store/apps/details?id=com.neximolabs.blackr to turn off the screen, was wondering if this could be implemented into the android app or at least allow the app to hide both the status and navigation bars so that Blackr can completely turn off the screen while on the transfer page.

I've been using Blackr to turn off the screen, was wondering if this could be implemented into the android app or at least allow the app to hide both the status and navigation bars so that Blackr can completely turn off the screen while on the transfer page.

It should be fine. This can be implemented as an independent feature though.
The main issue here is that the file transfer might be paused if the app is minimized.
There is no background task handler implemented yet.

I think this was the case when I tried this out on an iPhone.
Because you seem to really want the display to be turned off, I can add make the "keep screen on" optional (opt-out)


add an index endpoint for Content Block Editions

Can be merged after this PR https://github.com/alphagov/whitehall/pull/9208 ✅
Changes in this PR

adds a simple JSON response listing all Content Block Editions.

Merging in @Harriethw's absence 👍


Make Elasticsearch docs page procedural

Let's make this page procedural with step-by-step instructions.
https://docs.open-metadata.org/connectors/elastic-search
In its current form the distinction between the two config files is unclear.
It is also not clear that we should be working from clone of the repo to get the ingestion subdirectory.
The above seem to be problems across the connectors docs pages.

It should be separated from connectors page, as this is internal to the service itself and not necessarily a connector to fetch metadata inside elasticsearch

Lets also add some of these details https://docs.google.com/document/d/1-Z7JEhevp3zu780TDXiGPJ6FJsSIScgWGNGL_Pj3Vrk/edit#heading=h.34kftgotq69z on how to access a SSL enable elastic search cluster


Rustup

Also fix josh build (they have a deny(warnings) that's causing issues).

@bors r+

:pushpin: Commit d36c3e7cac5d97aefd86f0a2ecb93e9b29f70324 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit d36c3e7cac5d97aefd86f0a2ecb93e9b29f70324 with merge a2533ed2bc32064adf14ebcad609885c310c7fb4...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing a2533ed2bc32064adf14ebcad609885c310c7fb4 to master...


feat: add math/base/special/exp2f

Resolves #649
Description

What is the purpose of this pull request?

This pull request:

adds math/base/special/exp2f

Related Issues

Does this pull request have any related issues?

This pull request:

resolves #649

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

/stdlib update-copyright-years

good morning @Planeshifter! I've wrapped the intermediate results and final answers in float64ToFloat32(). If there's anything else that needs to be changed here please let me know.

Also, as mentioned in the questions section of the PR, what should i do about benchmark/c/cephes since it doesn't seem to compile locally and it fails on CI as well. Is there something that I'm missing from my side.


UI Error when launching catalog app in v2.2.4-rc12

rancher/rancher:v2.2.4-rc12

Deploy Rancher
Create a cluster
Go to the apps page
Click launch app
Select an app

A full page ui error is shown: Cannot read property '_routerMicrolib' of undefined

Default ui-index in RC is local, switching to https://releases.rancher.com/ui/2.2.66/index.html allowed to workaround

Fixed in UI v2.2.68 https://github.com/rancher/ui/pull/3012/files/747b10ba4a8ade21e9331ed98806bdf0c95e08cc#r287244076

master is good. So we don't need to fix it in master

in rc13

rancher/rancher:v2.2.4-rc13

Installed Rancher
Created cluster
Able to create apps correctly.


crash inside Error.isCanceled (instance of #649)

Following request to open new issue left in issue #649
I caused this issue by following code:
    .then { _ -> Promise<Void> in
        throw NSError()
    }

for whatever reason, swift allows to create empty NSError (frankly,  looks like a bug, there is no empty constructor defined for NSError in Swift)
version: 6.13.1 installed using CocoaPods

Not really sure what to say except either:

Don’t do that
Report it to Apple

As far as I know our code is legal Swift that shouldn’t crash.

It actually isn't?
There is no default constructor for NSError:
https://developer.apple.com/documentation/foundation/nserror
Actually, I distinctly remember failing to create empty NSError before (as in syntax error), so, it appears to be a resent Swift bug.
Maybe all other instances of #649 was similar bugs in NSError bridging?

If you can find a line of code in PromiseKit that is illegal we will correct it.
As it stands you are calling NSError(). If you can submit a PR to fix your code on this repo I will merge it.

I mean, it looks like it was a bit of the mystery to you, so, I decided to report one possible source of the issue 🤷♂
As to my attempts to find the source of the issue (which was that NSError() in my experiments), I did not found a workaround before I found the source of troubles. And frankly, I don't see how one could be possible. Your code should work, this the a legal way to do things.

K, thanks. This is the issue tracker, so if it's not a bug I will move this to the discussions board.


Does pip ships flit_core internally?

I am trying to make a new python project using PEP 621.
I firstly updated pip using python -m pip install --upgrade pip.
So pip version is the following:
python -m pip  --version
pip 21.3.1 from /Users/kbumsik/.pyenv/versions/3.10.0/lib/python3.10/site-packages/pip (python 3.10)

My pyproject.toml was like this:
[build-system]
requires = ["flit_core >=3.2,<4"]
build-backend = "flit_core.buildapi"

[project]
...
...

Then running pip install -e . or pip install . works, successfully installs dependencies.
But the point is, I even didn't install flit yet, I mean, I didn't python -m pip install flit yet and I can't import flit_core in the Python shell.
Does the latest pip ships flit_core internally?
Thank you.

Per https://www.python.org/dev/peps/pep-0518/, it doesn't need to. flit is installed into an isolated environment and used from there to build the package.

To expand that a bit: pip doesn't ship flit_core, but it knows how to get it when it needs it. The line requires = ["flit_core >=3.2,<4"] tells it that it will need flit_core for this package. And the reason you can't import it afterwards is that pip creates an isolated build environment, installs packages there, uses that to build your package, and then deletes the environment afterwards. So it doesn't install flit_core in the environment you're working in.

You can read more about the whole "isolated build environment" stuation with pyproject.toml based projects in https://pip.pypa.io/en/stable/reference/build-system/pyproject-toml/.

Thank you so much all! I read through your links during the weekend I finally got the idea of pyproject.toml. It looks very confusing at first but I think it is a great way to support various build systems!


Add two new badge designs

Guys, what do you think of these designs?

+1
On Sat, Dec 19, 2015 at 3:59 PM, Feross Aboukhadijeh <
<EMAIL_ADDRESS>wrote:

[image: standard-outline]
https://cloud.githubusercontent.com/assets/121766/11915741/8506174c-a719-11e5-8b62-cf560b4fdb1b.png
[image: standard-outline-white]
https://cloud.githubusercontent.com/assets/121766/11915742/88b6a910-a719-11e5-9d3e-4e4b7315d2c1.png
—
Reply to this email directly or view it on GitHub
https://github.com/feross/standard/pull/365#issuecomment-166034662.

NACK,  these are sticker designs.  IMHO the badge should still be the square.

@dcousens Yep, that's what I meant.

definetely better.

:+1:


Elasticsearch semantic convention

Adjust Elasticsearch semantic convention for our main branch.
DELETE User request
Before changes https://app.signalfx.com/#/apm/traces/5bb939f171aa326f6fd386188223a9bb
After changes https://app.signalfx.com/#/apm/traces/5c97a185812bd86135b0e6daf15e2c91

Here are have some other comments which can be addressed in a separate PR:

Should we not rename the elasticsearch.method tag into db.operation? Probably yes.
Should we not rename the elasticsearch.action tag into db.statement?  Probably not - I would need to read more about Elasticsearch.


Package: Get ready for v0.2.0

Codecov Report
Base: 63.03% // Head: 63.03% // No change to project coverage :thumbsup:

Coverage data is based on head (693ca59) compared to base (02dce41).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #149   +/-   ##
=======================================
  Coverage   63.03%   63.03%
=======================================
  Files          20       20
  Lines         633      633
  Branches      151      151
=======================================
  Hits          399      399
  Misses        222      222
  Partials       12       12

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


net: add multi listener impl for net.Listener

This adds an implementation of net.Listener which uses synchronous multiplexing to listen on and accept connections from multiple addrs.

What type of PR is this?
/kind feature
What this PR does / why we need it:
This PR adds an implementation of net.Listener which uses synchronous multiplexing to listen to and accept connections from multiple addresses.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Release note:
NONE

/cc @thockin @danwinship @aojea

For the testing part should I directly add an integration test that listens on net.InterfaceAddrs on multiple ports and then try to connect to server for validation?

/cc @uablrek

For windows, we might need a goroutine-based async multiplexer as select is not supported.

do we really need this?

Currently, for v1alpha2 I spawn multiple helathz and metrics servers in go-routines, this could really help there.
Also, Tim suggested this here

do we really need this?

My vote is on a library function because of testability. I can imagine some "what if"s that should be tested in (good) unit-tests at one place.
I also want to emphasise my "context" comment:

Also, I think your NewMultiListener should take a context as parameter. net.Listen uses ListenConfig which feels like a work-around for keeping backward compatibility.

do we really need this?

Currently, for v1alpha2 I spawn multiple helathz and metrics servers in go-routines, this could really help there.
Also, Tim suggested this here

KEP-2438 Dual-Stack API Server needs it too

But you really shouldn't be implementing this in terms of syscall. Use goroutines. Trust the runtime.

You must have support for context. Please see https://github.com/kubernetes/utils/pull/311#discussion_r1614677089. I will make a better review later.

I think the multi listener must be backward compatible. That means that a call like:
l,e := MultiListen(ctx, []string{":8080"})

must work, and listen on both families, and:
l,e := MultiListen(ctx, []string{"<IP_ADDRESS>:8080"})
// or
l,e := MultiListen(ctx, []string{"[::]:8080"})

must also listen on both families.

You must have support for context. Please see #311 (comment). I will make a better review later.

net.Listener interface won't allow Accept to consume context, should I store the context with the structure to check context in the Accept call?

Actually, I don't think you have to store it. Just create the Listerners with something like:
l,e := net.ListenConfig{}.Listen(ctx, network, address)

Then I suppose all Accept() will exit with an error if the ctx is canceled

The multi-listener will have different behavior compared to the standard Listener, as mentioned in https://github.com/kubernetes/utils/pull/311#discussion_r1641779949. I can't see a way to make them exactly the same, but the differences must be investigated and documented. Otherwise users will assume that they are totally compatible. I can make some tests, but not this week.
Related is how TCP backlog works in Linux. I thought I knew (I didn't), and have been hit by half-open connection problems at least once (in ~20y, so it is very rare).

Actually, I don't think you have to store it. Just create the Listerners with something like:
l,e := net.ListenConfig{}.Listen(ctx, network, address)

Then I suppose all Accept() will exit with an error if the ctx is canceled

@uablrek https://github.com/golang/go/issues/50861

Actually, I don't think you have to store it. Just create the Listerners with something like:
l,e := net.ListenConfig{}.Listen(ctx, network, address)

Then I suppose all Accept() will exit with an error if the ctx is canceled

@uablrek golang/go#50861

Is there any way to break a blocking Accept()??

OK, so...
We could do:
MultiListen(ctx context.Context, network string, addresses []string) (net.Listener, error)

where the semantics are basically:
for i := range addresses {
        lc.Listen(ctx, network, addresses[i])
}

with no added intelligence.
If we want per-address network fields, then I think a more logical API would be drop MultiListen and instead have the caller do
ml := netutils.NewMultiListener()
err := ml.Listen(ctx, "tcp4", "<IP_ADDRESS>:80")
if err != nil ...
err = ml.Listen(ctx, "tcp6", "[::]:80")
if err != nil ...
...

I think both of these are equally unambiguous, though the latter is more flexible.
I can imagine that for some callers, being able to separately listen on "tcp4", "<IP_ADDRESS>:80" and "tcp6", "[::]:80" might make more sense given their internal organization?
Regardless of which of the above approaches we go to, the other question is: what should happen if you listen on both "tcp", "<IP_ADDRESS>:80" and "tcp", "[::]:80" (which is one of the bugs that got us here, https://github.com/kubernetes/kubernetes/issues/114702)? Should MultiListener DTRT here, or should it just do literally what you asked, and end up returning an error (meaning that it's up to callers to know better than to do that).

Taking (or building) a list of individual Listener is about as flexible as gets.  It's not what I meant, but I can see the appeal.  Given that the primary consumer of this API will be kubernetes, I suspect that's more flexible that we need, but it seems low-cost to do it that way.  I see now the argument that dual-stack is a primary motivator, so a single network argument for a list of IPs may be pointless.
WRT zero addresses, if we make the caller call Listen(), then it should be on them to not collide - we should not be too smart.  Effectively this becomes an accept-mux - we need to nail those semantics.


Add in more granular DeCLUTR results

Add in more granular DeCLUTR results to the report (possibly appendix) and highlight the improvement at lower epoch counts.
Calculate metrics for:

downstream pseudo-tasks
GLUE
Topic Modelling

Added in Report v1.1


/sql/datetimetype: fix ranges to match mysql

Leaving comments on the parent issue: https://github.com/dolthub/dolt/issues/5035

Parent issue: https://github.com/dolthub/go-mysql-server/pull/1490


fix(v2): various minor bugs with location hash

Motivation
After merge #4162, our anchor links were broken. It also turned out that when we open any page with the hideable navbar enabled, one of useEffect hook is wrongly triggered.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Preview.
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

not sure to understand what was the problem in the first place 😅 but didn't notice anything wrong in preview so let's merge


feat: reserve variable names starting with $

#150  👈 (View in Graphite)
main

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


Provide means to sort Cw CreateWindow::value_list to avoid panic

I was surprised by:
thread 'main' panicked at 'CreateWindow::value_list must be sorted!'

If ordering is required (I'm not sure if that is true?), then there should be some way to see what the correct order should be from the documentation, or, more ergonomically, Cw should implement Ord and allow the data to be sorted in code.
As it stands, the source of this crate has to be examined to find the private method that is used to check for this condition to determine what that ordering should be.

This requirement comes from XCB directly that specifies that value_list must be sorted in the order of the Cw enum.
I don't sort in CreateWindow to avoid runtime penalty.
I will add proper documentation for this.
The value_list is generally defined at compile time, so it is best to define in the right order rather than to sort it at run-time.

Makes sense.  I think it would be good to define Ord for situations where the value list might be created dynamically though!

#207 implements Ord for Cw and similar types.
Note it is still not enough for x::CreateWindow request where value_list must be sorted AND have distinct values (at most one of each Cw member)

Thanks!


TypeError: to() received an invalid combination of arguments - got (device), but expected one of:  * (torch.device device, torch.dtype dtype)  * (torch.dtype dtype)       didn't match because some of the arguments have invalid types: (device)  * (Tensor other)       didn't match because some of the arguments have invalid types: (device)

When I carried out the project I got this promblem:
Traceback (most recent call last):
File "/home/bruce/mgn/mgn/mgn.py", line 300, in
run()
File "/home/bruce/mgn/mgn/mgn.py", line 243, in run
mgn = MGN(num_classes=len(train_dataset.unique_ids)).to(DEVICE)
File "/home/bruce/.conda/envs/snowflake/lib/python3.6/site-packages/torch/nn/modules/module.py", line 393, in to
return self._apply(lambda t: t.to(device))
File "/home/bruce/.conda/envs/snowflake/lib/python3.6/site-packages/torch/nn/modules/module.py", line 176, in _apply
module._apply(fn)
File "/home/bruce/.conda/envs/snowflake/lib/python3.6/site-packages/torch/nn/modules/module.py", line 176, in _apply
module._apply(fn)
File "/home/bruce/.conda/envs/snowflake/lib/python3.6/site-packages/torch/nn/modules/module.py", line 182, in _apply
param.data = fn(param.data)
File "/home/bruce/.conda/envs/snowflake/lib/python3.6/site-packages/torch/nn/modules/module.py", line 393, in
return self._apply(lambda t: t.to(device))
TypeError: to() received an invalid combination of arguments - got (device), but expected one of:

(torch.device device, torch.dtype dtype)
(torch.dtype dtype)
didn't match because some of the arguments have invalid types: (device)
(Tensor other)
didn't match because some of the arguments have invalid types: (device)
What can I do to solve it?

python2 or python3? I only test with python2. https://pytorch.org/docs/stable/tensors.html#torch.Tensor.to
You can print DEVICE and check pytorch doc to(device, dtype=None) → Tensor

Also make sure to use pytorch=0.4.0


Add collect support in dotnet test

@Faizan2304,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Release 1.1.1

Just waiting on https://github.com/dataiku/dss-plugin-nlp-offline-translation/pull/12, then we will publish a release 1.1.1 of the plugin

Should have made this a draft but forgot, will reopen anyway


Maint/main/ember 5.8.1

Yeah, this needs more work.

Closing in favor of https://github.com/puppetlabs/ivy-tabs/pull/51


What is the resolution of surface reconstruction?

How much resolution is used to reconstruction the synthetic rooms dataset in your article? 128^3 or 256^3?

Should be 128^3, what makes you think it's 256^3? Curious

Should be 128^3, what makes you think it's 256^3? Curious

I also think it 's 128^3. Thank you

K, i'll close this one. None of our config files and paper mention the 256 resolution, only 128.


[Event Change Request] report 5753 "Get Source Documents"  - OnAfterSalesLineOnPreDataItem / OnAfterPurchaseLineOnPreDataItem / OnAfterServiceLineOnPreDataItem

Could you please include the sender on the events OnAfterSalesLineOnPreDataItem, OnAfterPurchaseLineOnPreDataItem and OnAfterServiceLineOnPreDataItem ?
//+Change
//-[IntegrationEvent(false, false)]
[IntegrationEvent(true, false)]
//-Change
local procedure OnAfterSalesLineOnPreDataItem(var SalesLine: Record "Sales Line"; OneHeaderCreated: Boolean; WhseShptHeader: Record "Warehouse Shipment Header"; WhseReceiptHeader: Record "Warehouse Receipt Header")
begin
end;

//+Change
//-[IntegrationEvent(false, false)]
[IntegrationEvent(true, false)]
//-Change
local procedure OnAfterPurchaseLineOnPreDataItem(var PurchaseLine: Record "Purchase Line"; OneHeaderCreated: Boolean; WhseShptHeader: Record "Warehouse Shipment Header"; WhseReceiptHeader: Record "Warehouse Receipt Header")
begin
end;

//+Change
//-[IntegrationEvent(false, false)]
[IntegrationEvent(true, false)]
//-Change
local procedure OnAfterServiceLineOnPreDataItem(var ServiceLine: Record "Service Line"; OneHeaderCreated: Boolean; WhseShptHeader: Record "Warehouse Shipment Header"; WhseReceiptHeader: Record "Warehouse Receipt Header")
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 47288.


Native Compress Option

What's the Problem?
Lack of native 'Compress to Zip' option when right clicking files/folders.
Solution/Idea
Add a 'Compress to Zip' option to right click menu, as is present in File Explorer

Alternatives
Adding a third party compression tool which can then be added to context menus.
Priorities
No response
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.22000.556
Windows Version
Windows 11 Home, 21H2, 22000.556
Comments
No response

Merging with #45


Add examples

Hey Alexey
great project :) by change you can add a few examples in the documentation? that anyone just can run easy like ./gentee example1.g and see a full script with gentee?
I'm wrong or this is not related to this project? https://www.gentee.com/programming/samples/

Hi,
www.gentee.com is the old project of another programming laguage with the same name. I am going to close it later.
At this moment, there is not an example area, but I think I will add some examples. Anybody may send ideas for examples.
Now you look at gentee sources in tests/_test and tests/stdlib/_test files. Each test file contains some small scripts for testing purpose.

I have added examples folder with Gentee examples.


Changes for r0.7

Inculding https://github.com/tensorflow/tensorflow/pull/1072
Disabling //tensorflow/core/ops/array_grad_test.cc for GPU builds
Minor tweaks to ci_parameterzied_build.sh (a missing-space bug fix, env var display and timestamps)
Version bump from 0.6.0 to 0.7.0

Can one of the admins verify this patch?

@vrv version.h updated.

@vrv's comments are addressed. This PR is squashed and ready to be merged.


rolling restart

Description
Rolling restart of the cluster:

User asks for restart usig cli: bubuku-cli rolling-restart --image-tag 123 --instnce-type t2.nano
Bubuku creates restart assignment and triggers an action for rolling restart for the first broker in assignment at the same time checking for cluster state using kafka jmx metrics
Restart: broker is stopped, volume is detached, instance terminated, instance launched and broker is started
Once broker is being restarted is up, broker is making restart triggers rolling restart action with restart assignment without the broker which was restarted
Restart assignment is empty then rolling restart is finished

This is really cool 👍
Some corner cases that could happen but I'm not sure we need to address them right now:

Kafka leader imbalance could make this pause permanently between restarts. We've seen that happening. Let's see how it goes.
The action is consumed and posted back on each step. The problem with that is that some problems during the execution of a step could cause an unrecoverable failure. I mean, if at step 3 out of 5 something bad happens and an unrecoverable exception is launched, wouldn't that cause the complete process to stop and we would have to somehow identify where it stopped? Please, correct me if I understood it wrong.

Thank you for review!

It is intentionally blocking restart if there is prefered replica
imbalance, we observed in the past version of Kafka, it could imfluence
behavior. I agree, we need to observe it for sometime.

The restart can stop if bubuku performing restart was stopped, in that
case the restart should be started again from scratch. This is
intentionally done like that, because it is quite bad situation, when
broker is down while it is restarting other one. The state is not saved,
this is not perfect, we can improve it if required. If one of the steps
fails with exception, it will be triggered again.

On Thu 20. Dec 2018 at 12:50, Ricardo de Cillo<EMAIL_ADDRESS>wrote:

This is really cool 👍
Some corner cases that could happen but I'm not sure we need to address
them right now:

Kafka leader imbalance could make this pause permanently between
restarts. We've seen that happening. Let's see how it goes.
The action is consumed and posted back on each step. The problem
with that is that some problems during the execution of a step could cause
an unrecoverable failure. I mean, if at step 3 out of 5 something bad
happens and an unrecoverable exception is launched, wouldn't that cause the
complete process to stop and we would have to somehow identify where it
stopped? Please, correct me if I understood it wrong.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zalando-nakadi/bubuku/pull/124#issuecomment-448972520,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEo3QmS5BsvbPbJDZ8S-VBU1fcGkOjGQks5u63mVgaJpZM4ZbtFn
.

--
With great enthusiasm,
Andrey

Ok, now I remember that bubuku has a "catch all" exception and it would retry the action, right? So the only chance of having a partial rolling restart would be to kill bubuku process with -9, for example. If that's the case, then I agree that it's quite dramatic event and maybe we would like to start all over.

👍

I checked the lastest change. Was it necessary because it would retry too often?


ENH/TST: optimize CDI code,

The radial_grid from core is for 2D case only, so I still keep the _dist function.

Other comments on this, @tacaswell ?

Does it actually go faster?

Around 5% faster, but the code is much clean.


Contributing to: NewSource.json

Adding/Updating NewSource.json

Hi @rowanwins! Thanks for your data here. Could you rename the file and include the coverage information? See our contributor guidelines for more information about those fields.

Hi @iandees , have renamed the file and added the coverage. Apologies for missing this, first time contributor!

Hey @rowanwins , you have an extraneous comma in there.  Easy enough to make that mistake.  I would recommend running your json in something like JSONlint to make sure it's valid JSON.
Also, per the contributor guidelines referenced previously, you should have a "type" field, which in this case would be set to "http"


Course list refreshes when course is deleted. Added CourseListStudentCommand.

Coverage decreased (-0.5%) to 61.841% when pulling c9fa206be42f972d8219ac6ca36d2eefe5b64334 on m-aslam-mj2:stage into 0c7966e41bae99cb4655a3a77ceba4dc4a00286a on CS2113-AY1819S1-T16-1:master.


Engine Rewrite

As promised, a WIP rewrite of the repository in C++ with classes and stuff.
Todo

[x] Create & destroy windows
[x] App class that controls the window (currently only 1 window, but support for more can be added later)
[ ] Add documentation to functions etc.
[ ] Complete the Renderer class
[ ] Handle file I/O for shaders and whatever other files are needed
[ ] Handle user input
[ ] Handle more GLFW events
[ ] Incorporate other PRs and add their creators as co-authors so as to not overwrite already resolved issues

Maybe add

[ ] Cbengine API (perhaps with C compatiblility)

Closes #6

Likely will close all current issues and PRs

I don't really have time to correctly review PRs. My big exams are in about 2 months so I am revising. I'll get back to this in less than a week.

Hi CeeBox, hopefully all is well. I am graduating high school in about a week :). Anyways, I'm gonna be kinda busy getting that all set up and next steps for education and what not. Is the engine still something you plan on working on over next couple months (perhaps longer)?


force DeleteBucket command will failed on file tree height more than 2

Force DeleteBucket command will be failed because of empty keys (folder's keys)
Althrough DeleteBucket, ListBuckets result show that the bucket has already removed. But DeleteBucket still got the error.

The reason found!
To get compatibility from POSIX, maybe, cfs treat a folder path as a KV return to ListObject. But for the reason of POSIX, we should delete the leaf files before delete the folder. Althrough we can truely delete the folder, DeleteObject for folder's KV still return OK. But when DeleteBucket, ListObject will be invoke to check whether the bucket is empty. The ListObject shows not!


(ruby) Add packages for the different major versions

Add packages for the different major versions
Expected Behavior
I should be able to install Ruby 2.4, 2.5, or 2.6 instead of just the latest Ruby. A lot of people have applications that need a certain Ruby version to run.
Current Behavior
Right now the package is just the latest Ruby.
Possible Solution
Create a ruby24, ruby25, and ruby26 package in addition to the current package. That way someone that wants the latest can just install "ruby".
Steps to Reproduce (for bugs)

Install ruby package
Notice it's not 2.5

Context
N/A
Your Environment
N/A

I must have missed this when it was initially posted.
But let me say my opinion on the matter.
I strongly disagree with having multiple packages only for the usage of specific major/minor packages, if you want a polluted repository with duplicated packages, then this is exactly what you need to make that happen.
Using different packages specific to each major/minor version should be used very sparingly, and IMO should be avoided in most cases.
There is already a way to install a specific version of a package by specifying the --version argument when calling choco install.
Say for instance you want the <IP_ADDRESS> version of the package, you would then install the package with choco install ruby --version <IP_ADDRESS>.
Only specifying the major.minor version of the package and getting the latest version matching that range isn't possible yet though. But there is an open issue for such an implementation in choco: https://github.com/chocolatey/choco/issues/1278

I will close this as this will probably be possible one day by choco client and one can easily create script that does that.


参加aircoin内测

Hello, I want to sign up as a volunteer for the AIRcash software test. I understand some blockchain knowledge and have a certain foundation in English. I very much look forward to volunteers who can participate in this test.

Yes


Semantic API returning incorrect module symbol for value lang library symbols

Description:
Please refer to the below code. When trying to access the module symbol from the value library's jsonDecimal symbol returns lang.annotations library as the result (which is incorrect).
import ballerina/lang.value;

public function main() {
    value:JsonDecimal jsonDecimal = 23;
}

Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Had a look at this. It seems this is a bug in the compiler. In the internal symbol for JsonDecimal, the package ID is correct. But the owner symbol is set to the module symbol of lang.annotations.


Sharepoint API call to get all Documents

Hi @ificator
I need you help in understanding SharePoint
When i logged in to Office 365 with my business account, i see Sites Links there, some sites which my organisation shared with me.
When i click on Site link it shows "Document Library" and "Document sharing" links on left.
Click on "Document Library" it takes me to
https://myorganisation.sharepoint.com/Document Library/Forms/AllItems.aspx
And Click on "Document sharing" it takes me to
https://myorganisation.sharepoint.com/Documents sharing/Forms/AllItems.aspx
The API call i m using to get shared documents is following
/_api/web/getFolderByServerRelativeUrl('/Shared%20Documents')
but the Documents above call return does not contain any document from  "Document Library" and "Document sharing".
So do i need to make a separate call to get "Document Library" and "Document sharing" docs?
OR
Is there a common API call to get all Documents?
Please advice

Generally speaking, document libraries are treated as the scope of both the old (what you're referencing via _api/web) and the new (via _api/v2.0), and so when you make such a query you're only going to be getting back items within the specific document library. The only other option I can think of would be to drive and use search to find all the documents that are shared with you, but search is by definition not enumeration and so it won't have the same correctness guarantees.


Implement Consumers Pause

Resolves https://github.com/nats-io/nats.java/issues/1084

Waiting for a fix to https://github.com/nats-io/nats-server/issues/5163.


Creating persistent volume does not work with Mesos 1.0

Mesos 1.0 now enforces that DiskInfo principal is equal to framework/operator principal (https://issues.apache.org/jira/browse/MESOS-5005) or else the creation of persistent volume fails. So we need to set this during the create operation.

It looks like this issue is resolved in master. The disk-info principal is currently set.

@verma7 Is this resolved for you now ? If yes, can you please close this.

@kow3ns  @mohitsoni Are you sure this issue has been resolved?
I still see that the principal is not set here:

https://github.com/mesosphere/dcos-cassandra-service/blob/master/cassandra-commons/src/main/java/com/mesosphere/dcos/cassandra/common/util/TaskUtils.java#L111, or
https://github.com/mesosphere/dcos-commons/blob/master/src/main/java/org/apache/mesos/offer/OfferEvaluator.java#L325,

but is being set here:
3. https://github.com/mesosphere/dcos-commons/blob/master/src/main/java/org/apache/mesos/offer/ResourceUtils.java#L424
Are 1 and 2 not being used? Is so we should remove them or set the principal there too.

Reply from @kow3ns through Slack:
analysis of the call trace for the function mentioned in the issue leads me to believe it is dead code
dcos-commons does the Volume creation and it sets the principal as per the Mesos requirement

Closing the issue as we are not affected by it anymore.


[SPARK-44740][CONNECT][FOLLOW] Fix metadata values for Artifacts

What changes were proposed in this pull request?
This is a followup for a previous fix where we did not properly propagate the metadata from the main client into the dependent stubs.
Why are the changes needed?
compatibility
Does this PR introduce any user-facing change?
No
How was this patch tested?
Existing UT

merged to master and branch-3.5


sudo: 1.8.16 -> 1.8.17p1

Motivation for this change
Update sudo to the last version
Things done

[X] Tested using sandboxing
(nix.useChroot on NixOS,
or option build-use-chroot in nix.conf
on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

From https://www.sudo.ws/stable.html#1.8.17p1:
Fixed a bug introduced in 1.8.17 where the user's groups were not set on systems that don't use PAM.
https://bugzilla.sudo.ws/show_bug.cgi?id=749

Built locally, works fine. Travis' failure seems to be due to mysqlworkbench not compiling.

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable


FIP-18: Chain-level public address

Allows get_pub_address to return chain level public address if no token is found and the default chain is assigned.


EntityFramework Version

Is there a way we can get AutoMapper.EF6 to work with all versions of EF6 and not just EF 6.1.3?

You can always just copy the extension methods I think. But yes, I should
target the lowest version I can.
On Mon, Oct 16, 2017 at 10:45 AM, krptodr<EMAIL_ADDRESS>wrote:

Is there a way we can get AutoMapper.EF6 to work with all versions of EF6?
I dont believe the ODAC tools for Oracle.ManagedDataAccess work fully with
the latest version of EF 6.1.3.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/AutoMapper/AutoMapper.EF6/issues/13, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAGYMktiWqhywenq_b1WenzlPWyXtJ38ks5ss3oUgaJpZM4P61Te
.

Ah, I can't, I have a dependency that targets 6.1.3.


[DO NOT MERGE] Auto PR to update dependencies of istio

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
1115417e78cd6bcbb2c18f93a1b6d4524ec54e53
link
/test istio-pilot-e2e

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