Page does not explain how to create a COM Add-In

The page keep showing in search results to create a COM Add-In for OneNote, but only refers to a javascript+HTML Add-In.
I am not interested in developing in HTML and javascript!
And it definitely looks like the OneNote application on Windows still supports COM Add-Ins.

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

ID: a6ed3136-1d12-66c2-a8b1-1e7e43553aae
Version Independent ID: 0e40308a-2d3d-a27b-04d5-1bdd7c09b501
Content: Build your first OneNote task pane add-in - Office Add-ins
Content Source: docs/quickstarts/onenote-quickstart.md
Product: onenote
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

Hi @BenoitJGirard,
You're right that the information for how to make OneNote add-ins that don't use the current web-based model is difficult to find. I believe what you're looking for is here: https://docs.microsoft.com/en-us/office/client-developer/onenote/onenote-developer-reference?redirectedfrom=MSDN
I'm going to close this issue, as there's nothing actionable here to do in the web add-in documentation.


Weird errors when testing with mocha-webpack

I just started trying to test with mocha-webpack, and although tests are running, I am getting weird errors when running mocha-webpack.
$ npm run mocha

><EMAIL_ADDRESS>mocha /Users/ericmarkmartin/Developments/project
> mocha-webpack --webpack-config webpack/webpack.config.test.babel.js --glob "*.spec.js" spec --recursive; exit 0;

  Feathers application tests
webpack built e141a4ebd6771ce0aac0 in 640ms
Hash: e141a4ebd6771ce0aac0
Version: webpack 2.2.1
Time: 640ms
          Asset    Size  Chunks  Chunk Names
index_bundle.js  104 kB       0  main
chunk    {0} index_bundle.js (main) 76.2 kB [entry] [rendered]
    [0] ./~/html-entities/lib/html5-entities.js 49 kB {0} [built]
    [1] (webpack)-hot-middleware/client.js 6.68 kB {0} [built]
    [2] ./~/ansi-html/index.js 4.26 kB {0} [built]
    [3] ./~/ansi-regex/index.js 135 bytes {0} [built]
    [4] ./~/html-entities/index.js 231 bytes {0} [built]
    [5] ./~/html-entities/lib/html4-entities.js 6.57 kB {0} [built]
    [6] ./~/html-entities/lib/xml-entities.js 2.98 kB {0} [built]
    [7] ./~/strip-ansi/index.js 161 bytes {0} [built]
    [8] (webpack)-hot-middleware/client-overlay.js 1.74 kB {0} [built]
    [9] (webpack)-hot-middleware/process-update.js 3.88 kB {0} [built]
   [10] (webpack)/buildin/module.js 517 bytes {0} [built]
   [11] multi webpack-hot-middleware/client /client/index.jsx 40 bytes {0} [built]

ERROR in multi webpack-hot-middleware/client /client/index.jsx
Module not found: Error: Can't resolve '/client/index.jsx' in '/Users/ericmarkmartin/Developments/project @ multi webpack-hot-middleware/client /client/index.jsx

ERROR in   Error: Child compilation failed:
  Entry module not found: Error: Can't resolve '/client/public/index.html' in '/Users/ericmarkmartin/Developments/trump-of-the-day  ':
  Error: Can't resolve '/client/public/index.html' in '/Users/ericmarkmartin/Developments/project
  - compiler.js:76
    [trump-of-the-day]/[html-webpack-plugin]/lib/compiler.js:76:16

  - Compiler.js:280 Compiler.<anonymous>
    [trump-of-the-day]/[webpack]/lib/Compiler.js:280:10

  - Compiler.js:480
    [trump-of-the-day]/[webpack]/lib/Compiler.js:480:13

  - Tapable.js:138 next
    [trump-of-the-day]/[tapable]/lib/Tapable.js:138:11

  - CachePlugin.js:62 Compiler.<anonymous>
    [trump-of-the-day]/[webpack]/lib/CachePlugin.js:62:5

  - Tapable.js:142 Compiler.applyPluginsAsyncSeries
    [trump-of-the-day]/[tapable]/lib/Tapable.js:142:13

  - Compiler.js:477
    [trump-of-the-day]/[webpack]/lib/Compiler.js:477:10

  - Tapable.js:131 Compilation.applyPluginsAsyncSeries
    [trump-of-the-day]/[tapable]/lib/Tapable.js:131:46

  - Compilation.js:640 self.applyPluginsAsync.err
    [trump-of-the-day]/[webpack]/lib/Compilation.js:640:19

  - Tapable.js:131 Compilation.applyPluginsAsyncSeries
    [trump-of-the-day]/[tapable]/lib/Tapable.js:131:46

  - Compilation.js:631 self.applyPluginsAsync.err
    [trump-of-the-day]/[webpack]/lib/Compilation.js:631:11

  - Tapable.js:131 Compilation.applyPluginsAsyncSeries
    [trump-of-the-day]/[tapable]/lib/Tapable.js:131:46

  - Compilation.js:626 self.applyPluginsAsync.err
    [trump-of-the-day]/[webpack]/lib/Compilation.js:626:10

  - Tapable.js:131 Compilation.applyPluginsAsyncSeries
    [trump-of-the-day]/[tapable]/lib/Tapable.js:131:46

  - Compilation.js:622 sealPart2
    [trump-of-the-day]/[webpack]/lib/Compilation.js:622:9

  - Tapable.js:131 Compilation.applyPluginsAsyncSeries
    [trump-of-the-day]/[tapable]/lib/Tapable.js:131:46

  - Compilation.js:570 Compilation.seal
    [trump-of-the-day]/[webpack]/lib/Compilation.js:570:8

  - Compiler.js:474
    [trump-of-the-day]/[webpack]/lib/Compiler.js:474:16

  - Tapable.js:225
    [trump-of-the-day]/[tapable]/lib/Tapable.js:225:11

  - Compilation.js:472 _addModuleChain
    [trump-of-the-day]/[webpack]/lib/Compilation.js:472:11

  - Compilation.js:365 Compilation.errorAndCallback
    [trump-of-the-day]/[webpack]/lib/Compilation.js:365:4

  - Compilation.js:382 moduleFactory.create
    [trump-of-the-day]/[webpack]/lib/Compilation.js:382:12

  - NormalModuleFactory.js:234
    [trump-of-the-day]/[webpack]/lib/NormalModuleFactory.js:234:19

  - NormalModuleFactory.js:59 onDoneResolving
    [trump-of-the-day]/[webpack]/lib/NormalModuleFactory.js:59:20

  - NormalModuleFactory.js:126
    [trump-of-the-day]/[webpack]/lib/NormalModuleFactory.js:126:20

  - async.js:3694
    [trump-of-the-day]/[webpack]/[async]/dist/async.js:3694:9

  - async.js:356
    [trump-of-the-day]/[webpack]/[async]/dist/async.js:356:16

  - async.js:934 iteratorCallback
    [trump-of-the-day]/[webpack]/[async]/dist/async.js:934:13

  - async.js:840
    [trump-of-the-day]/[webpack]/[async]/dist/async.js:840:16

  - async.js:3691
    [trump-of-the-day]/[webpack]/[async]/dist/async.js:3691:13

  - async.js:21 apply
    [trump-of-the-day]/[webpack]/[async]/dist/async.js:21:25

  - async.js:56
    [trump-of-the-day]/[webpack]/[async]/dist/async.js:56:12

  - NormalModuleFactory.js:121
    [trump-of-the-day]/[webpack]/lib/NormalModuleFactory.js:121:22

  - Resolver.js:70 onResolved
    [trump-of-the-day]/[enhanced-resolve]/lib/Resolver.js:70:11

  - createInnerCallback.js:31 loggingCallbackWrapper
    [trump-of-the-day]/[enhanced-resolve]/lib/createInnerCallback.js:31:19

  - Resolver.js:138 afterInnerCallback
    [trump-of-the-day]/[enhanced-resolve]/lib/Resolver.js:138:10

  - createInnerCallback.js:31 loggingCallbackWrapper
    [trump-of-the-day]/[enhanced-resolve]/lib/createInnerCallback.js:31:19

  - Tapable.js:181 Resolver.applyPluginsAsyncSeriesBailResult1
    [trump-of-the-day]/[tapable]/lib/Tapable.js:181:46

  - Resolver.js:125 innerCallback
    [trump-of-the-day]/[enhanced-resolve]/lib/Resolver.js:125:19

  - createInnerCallback.js:31 loggingCallbackWrapper
    [trump-of-the-day]/[enhanced-resolve]/lib/createInnerCallback.js:31:19

  - Tapable.js:283
    [trump-of-the-day]/[tapable]/lib/Tapable.js:283:15

  - UnsafeCachePlugin.js:38
    [trump-of-the-day]/[enhanced-resolve]/lib/UnsafeCachePlugin.js:38:4

  - createInnerCallback.js:31 loggingCallbackWrapper
    [trump-of-the-day]/[enhanced-resolve]/lib/createInnerCallback.js:31:19

  - Resolver.js:138 afterInnerCallback
    [trump-of-the-day]/[enhanced-resolve]/lib/Resolver.js:138:10

  - createInnerCallback.js:31 loggingCallbackWrapper
    [trump-of-the-day]/[enhanced-resolve]/lib/createInnerCallback.js:31:19

  - Tapable.js:181 Resolver.applyPluginsAsyncSeriesBailResult1
    [trump-of-the-day]/[tapable]/lib/Tapable.js:181:46

  - Resolver.js:125 innerCallback
    [trump-of-the-day]/[enhanced-resolve]/lib/Resolver.js:125:19

  - createInnerCallback.js:31 loggingCallbackWrapper
    [trump-of-the-day]/[enhanced-resolve]/lib/createInnerCallback.js:31:19

  - Tapable.js:283
    [trump-of-the-day]/[tapable]/lib/Tapable.js:283:15

  - Resolver.js:123 innerCallback
    [trump-of-the-day]/[enhanced-resolve]/lib/Resolver.js:123:11

  - createInnerCallback.js:31 loggingCallbackWrapper
    [trump-of-the-day]/[enhanced-resolve]/lib/createInnerCallback.js:31:19

Child html-webpack-plugin for "index.html":

    ERROR in Entry module not found: Error: Can't resolve '/client/public/index.html' in '/Users/ericmarkmartin/Developments/project

After these, the tests run just fine, and then mocha never exits, forcing me to use ctrl-c to move on. As a side note, I find it weird that it shows my first describe message before building with webpack, but that's probably a separate issue.
Relevant information
test command:
"mocha": "mocha-webpack --webpack-config webpack/webpack.config.test.babel.js --glob \"*.spec.js\" spec --recursive; exit 0;",

webpack.config.test.babel.js
import nodeExternals from 'webpack-node-externals';
import path from 'path';
import HTMLWebpackPlugin from 'html-webpack-plugin';

const HTMLWebpackPluginConfig = new HTMLWebpackPlugin({
  template: path.join(__dirname, '../client/public/index.html'),
  filename: 'index.html',
  inject: 'body',
});

export default {
  target: 'node',
  entry: [
    path.join(__dirname, '../client/index.jsx'),
  ],
  externals: [nodeExternals()],
  module: {
    loaders: [
      {
        test: /\.jsx?$/,
        exclude: /node_modules/,
        use: [
          {
            loader: "babel-loader"
          }
        ]
      }
    ]
  },
  plugins: [
    HTMLWebpackPluginConfig,
  ]
};

Any help appreciated.

Remove html-webpack-plugin plugin and also the require to webpack-hot-middleware/client.

@zinserjan, thanks, but the same thing happens.

Ah, I just missed the build stats in your logs. Where are they from? Looks for me like another webpack process is triggered in the tests.

I start the server from one of my unit tests to verify that 404's and such work as expected.
app.js
import path from 'path';
import feathers, { static as serveStatic } from 'feathers';
import favicon from 'serve-favicon';
import compress from 'compression';
import cors from 'cors';
import configuration from 'feathers-configuration';
import hooks from 'feathers-hooks';
import rest from 'feathers-rest';
import bodyParser from 'body-parser';
import socketio from 'feathers-socketio';
import webpack from 'webpack';
import middleware from './middleware';
import services from './services';
import config from '../webpack/webpack.config.babel';

const app = feathers();
const compiler = webpack(config);

app.use(require('webpack-dev-middleware')(compiler, {
  noInfo: true,
  publicPath: config.output.publicPath,
}));

app.use(require('webpack-hot-middleware')(compiler));

app.configure(configuration(path.join(__dirname, '..')));

app.use(compress())
  .options('*', cors())
  .use(cors())
  .use(favicon(path.join(app.get('public'), 'favicon.ico')))
  .use('/', serveStatic(app.get('public')))
  .use(bodyParser.json())
  .use(bodyParser.urlencoded({ extended: true }))
  .configure(hooks())
  .configure(rest())
  .configure(socketio())
  .configure(services)
  .configure(middleware);

export default app;

app.spec.js
import chai, { expect } from 'chai';
import dirtyChai from 'dirty-chai';
import chaiHttp from 'chai-http';
import app from '../src/app';

chai.use(chaiHttp);
chai.use(dirtyChai);

describe('Feathers application tests', () => {
  before(function setup(done) {
    this.server = app.listen(3030);
    this.server.once('listening', done);
  });

  after(function teardown(done) {
    this.server.close(done);
  });

  it('starts and shows the index page', () =>
    chai
      .request(app)
      .get('/')
      .set('Accept', 'text/html')
      .then((res) => {
        expect(res.text).to.have.string('<html>');
        expect(res.statusCode).to.equal(200);
      }),
  );

  describe('404', () => {
    const notFoundPath = '/path/to/nowhere/';
    it('shows a 404 HTML page', () =>
      chai
      .request(app)
      .get(notFoundPath)
      .set('Accept', 'text/html')
      .then((res) => {
        expect(res.statusCode).to.equal(404);
        expect(res.text).to.have.string('<html>');
      })
      .catch((err) => {
        expect(err.message).to.equal('Not Found');
        const res = err.response;
        expect(res.statusCode).to.equal(404);
        expect(res.text).to.have.string('<html>');
      }),
    );

    it('shows a 404 JSON error without stack trace', () =>
      chai
      .request(app)
      .get(notFoundPath)
      .set('Accept', 'application/json')
      .then((res) => {
        expect(res.statusCode).to.equal(404);
      })
      .catch((err) => {
        expect(err.message).to.equal('Not Found');
        const res = err.response;
        expect(res).to.have.status(404);
        expect(res).to.be.json();
        expect(JSON.parse(res.text).message).to.equal('Page not found');
      }),
    );
  });
});

You app.js starts another webpack process right inside mocha-webpack. Why is that necessary?
Regardless of your answer, this isn't possible as webpack compiles everything into a single bundle and transforms variables like __dirname or ___filename into a static value that lead to this errors.
Maybe it's possible when you let webpack inject the real paths like discussed in https://github.com/zinserjan/mocha-webpack/issues/39#issuecomment-238987327
But it would be better to remove the webpack part out of your app.js or execute it conditionally (only in development, not for testing/production).
I close this as this isn't a issue of mocha-webpack.


Argoverse2 Leaderboard

Dear authors,
Congratulations on this nice work being accepted!
We notice that the Leaderboard of Argoverse 2 multi-world forecasting has re-opened.  We would appreciate if you could provide the result on the test set for our comparison with your excellent work.
Best

FJMP is now on the leaderboard. Thanks for your patience.


Using annotation for Bean

Hi
I tried to use the annotations and jmx whiteboard to declare mbean but I did not find documentation to guide me.
I saw an example but I did not find how to implement it when there are several instances.
I have somes Beans (blueprint) that derives from an abstract class. This abstract class creates an instance of an MBean class for each instance of Bean
If I put annotation on my class MBean aries creates a single instance independently of my Beans.
I'm looking for how to use whiteboard, either by annotation or by code, so that my abstract class automatically declares the created MBean.
To this day I am compelled to do so in every project using this mother class in a dedicated Blueprint file.
If you know where I can find info on how to go about it.
With the aries documentation I did not figure out how to do it. Nor even if it is possible.
Thanks for any help.

You need to export each Bean as an OSGi service. Then it will be published as a JMS bean. If would do this explicitly on each bean and not on the abstract class. If you want to do it on the abstract class then you would need to do the export using the OSGi service API in java code.

thank you
I'll try it out.
A+JYT


v1.15 fix Sprintf format 'verbs' in locale files

as title

If I understand the Crowdin Process correctly, this PR is unnecessary as it will be overwritten by Crowdin with the next "update translations" commit.
The best way would then be to fix it manually in Crowdin for each of these locales.

If I understand the Crowdin Process correctly, this PR is unnecessary as it will be overwritten by Crowdin with the next "update translations" commit.
The best way would then be to fix it manually in Crowdin for each of these locales.

it will be overwritten by Crowdin only in main branch.

yes crowding update only happen on main

was it fixed on crowdin for main?

was it fixed on crowdin for main?
No, Only Russian locale

It's for v1.15, so the backport label is not necessary.

make L-G-T-M work.

@lunny I know.
That bot seems broken.
No matter what I do, it does not count my approvals.
I've already accepted my fate.


fix: 'unsupported command' everywhere

Signed-off-by: cadl<EMAIL_ADDRESS>After #662, the internal command does not work. Make some fixes.
before:
> \dt
? ;
Internal error: unsupported command
> \dt;
Internal error: unsupported command
>

after:
> \dt
+---+----------+---+------------+---+--------------+
| 0 | postgres | 1 | pg_catalog | 0 | contributors |
+---+----------+---+------------+---+--------------+
in 0.001s
> select * from foo;
bind error: invalid table foo
> select *
? from foo;
bind error: invalid table foo

Thanks! In fact, I think \dt should always work without ;. That is to say, if a line begins with \, then the cli should immediately execute it on new line instead of waiting for ;. If you're interested, you may submit another PR to fix the behavior.

yes, \dt works like you say now.  As the example in the PR description
> \dt
+---+----------+---+------------+---+--------------+
| 0 | postgres | 1 | pg_catalog | 0 | contributors |
+---+----------+---+------------+---+--------------+
in 0.001s

That's cool, thanks!


Does not appear in Godot 4.1.1

Enabled the plugin in the project settings and the tab for the associated plugin does not appear (version 4.1.1)

Download latest version straight from repository, the option will appear in popup menu from selected files.


convert newlines to br elements when inserting text

since text inserts can contain newlines and we don't expect those in the editor, we convert them to br elements in the insert-text command. this also fixes some issues I had with pasting content from PDFs and editing that.

we should do the same logic in the insert-html command
I dont think we want that, the expectations are different when pasting/inserting text vs HTML. In HTML the default expectation is that newlines are not visible.

So to conclude the expected behaviour when inserting HTML vs text is different and as such the code can not be generalised.

Good point!


Config changes to reflect the default branch from master to main. Please merge to complete the default branch renaming job.

Auto branch renaming content.

Docs Build status updates of commit e91295c:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch d2d5aa3b-fc45-496f-b65b-1b09e01a5ccd has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


thoughts on using a memoize helper instead of the internal* helpers?

https://github.com/caitp/cauliflower-filter/blob/master/filter.js#L55-L61
https://github.com/caitp/cauliflower-filter/blob/master/filter.js#L75-L85

it would shrink code a tiny bit, I suppose. I don't want to pull in something like lodash or underscore or another heavy utility library just for their memoizers though, the dependencies are heavy enough as it is


Detach disks from down VMs on attach from another VM

Tested with the following config
One VM has attached a disk testvol-1 and is powered off with the disk still attached
Second VM tries to attach the same disk attach works fine and container runs on second VM
First VM is powered on and runs fine.
Server logs:
08/11/16 11:24:44 1852273 [1] [INFO   ] *** attachVMDK: /vmfs/volumes/bigone/dockvols/testvol-1.vmdk to 1 VM uuid = 4225d240-b7f5-3825-e1c4-e69a08844980
08/11/16 11:24:44 1852273 [1] [INFO   ] Attaching /vmfs/volumes/bigone/dockvols/testvol-1.vmdk as independent_persistent
08/11/16 11:24:44 1852273 [1] [INFO   ] Disk /vmfs/volumes/bigone/dockvols/testvol-1.vmdk successfully attached. controller pci_slot_number=224, disk_slot=1
08/11/16 11:24:44 1852273 [1] [INFO   ] executeRequest 'attach' completed with ret={'ControllerPciSlotNumber': '224', 'Unit': '1'}
08/11/16 11:25:37 1852273 [2] [INFO   ] executeRequest 'get' completed with ret={'created by VM': '1', 'created': 'Thu Aug 11 11:00:15 2016', 'capacity': {'allocated': '14.0MB', 'size': '100.0MB'}, 'attached to VM': '1', 'datastore': 'bigone', 'status': 'attached'}
08/11/16 11:25:37 1852273 [2] [INFO   ] *** attachVMDK: /vmfs/volumes/bigone/dockvols/testvol-1.vmdk to 2 VM uuid = 564d392b-3cb8-5a4a-862a-3df7a590d97e
08/11/16 11:25:37 1852273 [2] [INFO   ] Attaching /vmfs/volumes/bigone/dockvols/testvol-1.vmdk as independent_persistent
08/11/16 11:25:37 1852273 [2] [WARNING] Detaching disk /vmfs/volumes/bigone/dockvols/testvol-1.vmdk from VM uuid=4225d240-b7f5-3825-e1c4-e69a08844980
08/11/16 11:25:37 1852273 [2] [WARNING] VM power state is poweredOff
08/11/16 11:25:37 1852273 [2] [INFO   ] Disk detached /vmfs/volumes/bigone/dockvols/testvol-1.vmdk
08/11/16 11:25:37 1852273 [2] [INFO   ] Disk /vmfs/volumes/bigone/dockvols/testvol-1.vmdk successfully attached. controller pci_slot_number=224, disk_slot=0
08/11/16 11:25:37 1852273 [2] [INFO   ] executeRequest 'attach' completed with ret={'ControllerPciSlotNumber': '224', 'Unit': '0'}
08/11/16 11:26:08 1852273 [2] [INFO   ] *** detachVMDK: /vmfs/volumes/bigone/dockvols/testvol-1.vmdk from 2 VM uuid = 564d392b-3cb8-5a4a-862a-3df7a590d97e
08/11/16 11:26:08 1852273 [2] [INFO   ] Disk detached /vmfs/volumes/bigone/dockvols/testvol-1.vmdk
Sequence of ops:
cat testvol-1-6fd6eb20c78935d3.vmfd
{"created-by": "1", "status": "detached", "created": "Thu Aug 11 11:00:15 2016", "volOpts": {}}
vim-cmd vmsvc/power.off 1
Powering off VM:
cat testvol-1-6fd6eb20c78935d3.vmfd
{"created": "Thu Aug 11 11:00:15 2016", "status": "attached", "created-by": "1", "attachedVMName": "1", "attachedVMUuid": "4225d240-b7f5-3825-e1c4-e69a08844980", "volOpts": {}}
vim-cmd vmsvc/power.on 1
Powering on VM:
vim-cmd vmsvc/power.getstate 1
Retrieved runtime info
Powered on
cat testvol-1-6fd6eb20c78935d3.vmfd
{"created": "Thu Aug 11 11:00:15 2016", "status": "detached", "created-by": "1", "volOpts": {}}

Error handling in a couple of places is missing (see inline comments) but generally the fix looks good.

Additional tests done:
a. Attach a volume to a VM then power off that VM and power back on. Try attaching the same volume
Guest logs:
$ docker run --rm -it -v testvol-1:/data busybox
/ #
login as: administrator
<EMAIL_ADDRESS>password:
Welcome to Ubuntu 14.04.1 LTS (GNU/Linux 3.13.0-85-generic x86_64)
.......
Last login: Fri Aug 12 23:06:29 2016 from <IP_ADDRESS>
$  docker run --rm -it -v testvol-1:/data busybox
/ #
Server logs:
08/13/16 06:15:23 2204805 [1] [INFO   ] *** attachVMDK: /vmfs/volumes/bigone/dockvols/testvol-1.vmdk to 1 VM uuid = 4225d240-b7f5-3825-e1c4-e69a08844980
08/13/16 06:15:23 2204805 [1] [INFO   ] Attaching /vmfs/volumes/bigone/dockvols/testvol-1.vmdk as independent_persistent
08/13/16 06:15:23 2204805 [1] [INFO   ] Disk /vmfs/volumes/bigone/dockvols/testvol-1.vmdk successfully attached. controller pci_slot_number=224, disk_slot=1
08/13/16 06:15:23 2204805 [1] [INFO   ] executeRequest 'attach' completed with ret={'Unit': '1', 'ControllerPciSlotNumber': '224'}
... (on power on VM issues attach again)
08/13/16 06:21:52 2204805 [1] [INFO   ] *** attachVMDK: /vmfs/volumes/bigone/dockvols/testvol-1.vmdk to 1 VM uuid = 4225d240-b7f5-3825-e1c4-e69a08844980
08/13/16 06:21:52 2204805 [1] [INFO   ] Attaching /vmfs/volumes/bigone/dockvols/testvol-1.vmdk as independent_persistent
08/13/16 06:21:52 2204805 [1] [WARNING] Disk /vmfs/volumes/bigone/dockvols/testvol-1.vmdk already attached. VM=4225d240-b7f5-3825-e1c4-e69a08844980
08/13/16 06:21:52 2204805 [1] [INFO   ] executeRequest 'attach' completed with ret={'Unit': '1', 'ControllerPciSlotNumber': '224'}
Repeated the tests performed earlier also,
a. VM1 attaches a volume and VM2 tries to attach the same volume - Failed with already attached error
Server logs:
08/13/16 06:24:41 2204805 [2] [INFO   ] *** attachVMDK: /vmfs/volumes/bigone/dockvols/testvol-1.vmdk to 2 VM uuid = 564d392b-3cb8-5a4a-862a-3df7a590d97e
08/13/16 06:24:41 2204805 [2] [INFO   ] Attaching /vmfs/volumes/bigone/dockvols/testvol-1.vmdk as independent_persistent
08/13/16 06:24:41 2204805 [2] [INFO   ] executeRequest 'attach' completed with ret={'Error': 'Disk /vmfs/volumes/bigone/dockvols/testvol-1.vmdk already attached to VM=1'}
08/13/16 06:24:41 2204805 [2] [INFO   ] *** detachVMDK: /vmfs/volumes/bigone/dockvols/testvol-1.vmdk from 2 VM uuid = 564d392b-3cb8-5a4a-862a-3df7a590d97e
08/13/16 06:24:41 2204805 [2] [WARNING] *** Detach failed: disk=/vmfs/volumes/bigone/dockvols/testvol-1.vmdk not found. VM=564d392b-3cb8-5a4a-862a-3df7a590d97e
08/13/16 06:24:41 2204805 [2] [INFO   ] executeRequest 'detach' completed with ret={'Error': '*** Detach failed: disk=/vmfs/volumes/bigone/dockvols/testvol-1.vmdk not found. VM=564d392b-3cb8-5a4a-862a-3df7a590d97e'}
Guest logs:
$  docker run --rm -it -v testvol-1:/data busybox
docker: Error response from daemon: VolumeDriver.Mount: Disk /vmfs/volumes/bigone/dockvols/testvol-1.vmdk already attached to VM=1.
b. VM1 attaches a volume and is powered off, VM2 attaches the same volume - passed with volume attached to VM2 and VM1 power on works fine.
Server logs:
08/13/16 06:26:18 2204805 [2] [INFO   ] *** attachVMDK: /vmfs/volumes/bigone/dockvols/testvol-1.vmdk to 2 VM uuid = 564d392b-3cb8-5a4a-862a-3df7a590d97e
08/13/16 06:26:18 2204805 [2] [INFO   ] Attaching /vmfs/volumes/bigone/dockvols/testvol-1.vmdk as independent_persistent
08/13/16 06:26:18 2204805 [2] [WARNING] Detaching disk /vmfs/volumes/bigone/dockvols/testvol-1.vmdk from VM(powered off) - 1
08/13/16 06:26:18 2204805 [2] [INFO   ] Disk detached /vmfs/volumes/bigone/dockvols/testvol-1.vmdk
08/13/16 06:26:18 2204805 [2] [INFO   ] Disk /vmfs/volumes/bigone/dockvols/testvol-1.vmdk successfully attached. controller pci_slot_number=224, disk_slot=0
08/13/16 06:26:18 2204805 [2] [INFO   ] executeRequest 'attach' completed with ret={'Unit': '0', 'ControllerPciSlotNumber': '224'}
Guest logs:
$ docker run --rm -it -v testvol-1:/data busybox
/ #

Also, a request - it really helps reviewing if the changes done to address review comments are in a separate commit, and not squashed. Can you (in the future) maintain a separate commit for review follow up fixes? Git hub with squash in one commit on merge anyways.

Please think about automated test for this, as a separate PR. This PR LGTM.
thanks for taking care of this !

Thanks for reviewing Mark. Can I get one more review of this change.
On Tue, Aug 16, 2016 at 12:00 PM, Mark Sterin<EMAIL_ADDRESS>wrote:

Please think about automated test for this, as a separate PR. This PR LGTM.
thanks for taking care of this !
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vmware/docker-volume-vsphere/pull/573#issuecomment-240014827,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APHseOj4GT2uSQ6Nu22CqDcGfNDK7T2cks5qgVkEgaJpZM4JiB4U
.

LGTM. My only concern is this PR only works if both VMs (stale and new) are on same ESX host.

LGTM


AVRO-2408: Local docker based dev env with Forrest

This MUST wait for AVRO-2405 to be completed !!!

Link #527

Strange; The Java8 tests failed on travis, the Java 11 passed.
After I restarted the Java 8 test it passed also.

Awesome, thanks for picking this up @nielsbasjes


Add minimum profile example for crun

What type of PR is this?
/kind feature
What this PR does / why we need it:
We now add a baseprofile-crun.yaml which contains the minimum amount of
syscalls required to run a workload. This is now mentioned in the
installation guide, too.
Which issue(s) this PR fixes:

None
Does this PR have test?
None

Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

Added minimum [crun](https://github.com/containers/crun) base profile

Codecov Report

Merging #291 (2c79e12) into master (e816cd0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #291   +/-   ##
=======================================
  Coverage   25.48%   25.48%
=======================================
  Files           8        8
  Lines         616      616
=======================================
  Hits          157      157
  Misses        441      441
  Partials       18       18

Flag
Coverage Δ

unittests
25.48% <ø> (ø)

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

Continue to review full report at Codecov.

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

/lgtm


Drifting on MulRan Datasets

Thanks for your great work.
I recently tested this solution on MulRan dataset, but got drifting on every subset after running for couple seconds. It warns about 'large velocity, reset IMU-preintegration'. This happened both with and without GPS data.
I've set lidar type to MulRan as suggested, set corresponding extrinsics for lidar to imu, set number of scan to 64, and also set message topics correctly. Others are just kept the same as the default of 'lio_sam_ouster.yaml'.
Since you have mentioned your result on MulRan dataset, would you share your .yaml file for MulRan?

You can try again with the following parameters：
liorf:
Topics
pointCloudTopic: "/os1_points"               # Point cloud data
imuTopic: "/imu/data_raw"                         # IMU data
odomTopic: "odometry/imu"                   # IMU pre-preintegration odometry, same frequency as IMU
gpsTopic: "odometry/gpsz"                   # GPS odometry topic from navsat, see module_navsat.launch file
Frames
lidarFrame: "base_link"
baselinkFrame: "base_link"
odometryFrame: "odom"
mapFrame: "map"
GPS Settings
useImuHeadingInitialization: true           # if using GPS data, set to "true"
useGpsElevation: false                      # if GPS elevation is bad, set to "false"
gpsCovThreshold: 2.0                        # m^2, threshold for using GPS data
poseCovThreshold: 25.0                      # m^2, threshold for using GPS data
Export settings
savePCD: false                              # https://github.com/TixiaoShan/LIO-SAM/issues/3
savePCDDirectory: "/Downloads/LOAM/"        # in your home folder, starts and ends with "/". Warning: the code deletes "LOAM" folder then recreates it. See "mapOptimization" for implementation
Sensor Settings
sensor: mulran                            # lidar sensor type, 'velodyne' or 'ouster' or 'livox' or 'robosense'
N_SCAN: 64                                  # number of lidar channel (i.e., Velodyne/Ouster: 16, 32, 64, 128, Livox Horizon: 6)
Horizon_SCAN: 1024                          # lidar horizontal resolution (Velodyne:1800, Ouster:512,1024,2048, Livox Horizon: 4000)
downsampleRate: 2                           # default: 1. Downsample your data if too many points(line). i.e., 16 = 64 / 4, 16 = 16 / 1
point_filter_num: 5                         # default: 3. Downsample your data if too many points(point). e.g., 16: 1, 32: 5, 64: 8
lidarMinRange: 3.0                          # default: 1.0, minimum lidar range to be used
lidarMaxRange: 1000.0                       # default: 1000.0, maximum lidar range to be used
IMU Settings
imuType: 0                                  # 0: 6-axis  1: 9-axis
imuRate: 100.0
imuAccNoise: 0.009939570888238808e-03
imuGyrNoise: 0.005636343949698187e-03
imuAccBiasN: 0.64356659353532566e-03
imuGyrBiasN: 0.35640318696367613e-03
imuGravity: 9.80511
imuRPYWeight: 0.01
Extrinsics: T_lb (lidar -> imu)
extrinsicTrans: [1.77, -0.00, -0.05]
extrinsicRot: [-1, 0, 0,
0, -1, 0,
0, 0, 1]
This parameter is set only when the 9-axis IMU is used, but it must be a high-precision IMU. e.g. MTI-680
extrinsicRPY: [1, 0, 0,
0, 1, 0,
0, 0, 1]
voxel filter paprams
mappingSurfLeafSize: 0.4                      # default: 0.4 - outdoor, 0.2 - indoor
robot motion constraint (in case you are using a 2D robot)
z_tollerance: 1000                            # meters
rotation_tollerance: 1000                     # radians
CPU Params
numberOfCores: 4                              # number of cores for mapping optimization
mappingProcessInterval: 0.0                  # seconds, regulate mapping frequency
Surrounding map
surroundingkeyframeAddingDistThreshold: 1.0   # meters, regulate keyframe adding threshold
surroundingkeyframeAddingAngleThreshold: 0.2  # radians, regulate keyframe adding threshold
surroundingKeyframeDensity: 2.0               # meters, downsample surrounding keyframe poses
surroundingKeyframeSearchRadius: 50.0         # meters, within n meters scan-to-map optimization (when loop closure disabled)
surroundingKeyframeMapLeafSize: 0.5           # downsample local map point cloud
Loop closure
loopClosureEnableFlag: true
loopClosureFrequency: 1.0                     # Hz, regulate loop closure constraint add frequency
surroundingKeyframeSize: 50                   # submap size (when loop closure enabled)
historyKeyframeSearchRadius: 15.0             # meters, key frame that is within n meters from current pose will be considerd for loop closure
historyKeyframeSearchTimeDiff: 30.0           # seconds, key frame that is n seconds older will be considered for loop closure
historyKeyframeSearchNum: 25                  # number of hostory key frames will be fused into a submap for loop closure
loopClosureICPSurfLeafSize: 0.5               # downsample icp point cloud
historyKeyframeFitnessScore: 0.3              # icp threshold, the smaller the better alignment
Visualization
globalMapVisualizationSearchRadius: 1000.0    # meters, global map visualization radius
globalMapVisualizationPoseDensity: 10.0       # meters, global map visualization keyframe density
globalMapVisualizationLeafSize: 1.0           # meters, global map visualization cloud density

You can try again with the following parameters：
liorf:

  # Topics
  pointCloudTopic: "/os1_points"               # Point cloud data
  imuTopic: "/imu/data_raw"                         # IMU data
  odomTopic: "odometry/imu"                   # IMU pre-preintegration odometry, same frequency as IMU
  gpsTopic: "odometry/gpsz"                   # GPS odometry topic from navsat, see module_navsat.launch file

  # Frames
  lidarFrame: "base_link"
  baselinkFrame: "base_link"
  odometryFrame: "odom"
  mapFrame: "map"

  # GPS Settings
  useImuHeadingInitialization: true           # if using GPS data, set to "true"
  useGpsElevation: false                      # if GPS elevation is bad, set to "false"
  gpsCovThreshold: 2.0                        # m^2, threshold for using GPS data
  poseCovThreshold: 25.0                      # m^2, threshold for using GPS data

  # Export settings
  savePCD: false                              # https://github.com/TixiaoShan/LIO-SAM/issues/3
  savePCDDirectory: "/Downloads/LOAM/"        # in your home folder, starts and ends with "/". Warning: the code deletes "LOAM" folder then recreates it. See "mapOptimization" for implementation

  # Sensor Settings
  sensor: mulran                            # lidar sensor type, 'velodyne' or 'ouster' or 'livox' or 'robosense'
  N_SCAN: 64                                  # number of lidar channel (i.e., Velodyne/Ouster: 16, 32, 64, 128, Livox Horizon: 6)
  Horizon_SCAN: 1024                          # lidar horizontal resolution (Velodyne:1800, Ouster:512,1024,2048, Livox Horizon: 4000)
  downsampleRate: 2                           # default: 1. Downsample your data if too many points(line). i.e., 16 = 64 / 4, 16 = 16 / 1
  point_filter_num: 5                         # default: 3. Downsample your data if too many points(point). e.g., 16: 1, 32: 5, 64: 8
  lidarMinRange: 3.0                          # default: 1.0, minimum lidar range to be used
  lidarMaxRange: 1000.0                       # default: 1000.0, maximum lidar range to be used

  # IMU Settings
  imuType: 0                                  # 0: 6-axis  1: 9-axis
  imuRate: 100.0
  imuAccNoise: 0.009939570888238808e-03
  imuGyrNoise: 0.005636343949698187e-03
  imuAccBiasN: 0.64356659353532566e-03
  imuGyrBiasN: 0.35640318696367613e-03
  imuGravity: 9.80511
  imuRPYWeight: 0.01

  # Extrinsics: T_lb (lidar -> imu)
  extrinsicTrans: [1.77, -0.00, -0.05]
  extrinsicRot: [-1, 0, 0,
                  0, -1, 0,
                  0, 0, 1]

  # This parameter is set only when the 9-axis IMU is used, but it must be a high-precision IMU. e.g. MTI-680
  extrinsicRPY: [1, 0, 0,
                  0, 1, 0,
                  0, 0, 1]

  # voxel filter paprams
  mappingSurfLeafSize: 0.4                      # default: 0.4 - outdoor, 0.2 - indoor

  # robot motion constraint (in case you are using a 2D robot)
  z_tollerance: 1000                            # meters
  rotation_tollerance: 1000                     # radians

  # CPU Params
  numberOfCores: 4                              # number of cores for mapping optimization
  mappingProcessInterval: 0.0                  # seconds, regulate mapping frequency

  # Surrounding map
  surroundingkeyframeAddingDistThreshold: 1.0   # meters, regulate keyframe adding threshold
  surroundingkeyframeAddingAngleThreshold: 0.2  # radians, regulate keyframe adding threshold
  surroundingKeyframeDensity: 2.0               # meters, downsample surrounding keyframe poses
  surroundingKeyframeSearchRadius: 50.0         # meters, within n meters scan-to-map optimization (when loop closure disabled)
  surroundingKeyframeMapLeafSize: 0.5           # downsample local map point cloud

  # Loop closure
  loopClosureEnableFlag: true
  loopClosureFrequency: 1.0                     # Hz, regulate loop closure constraint add frequency
  surroundingKeyframeSize: 50                   # submap size (when loop closure enabled)
  historyKeyframeSearchRadius: 15.0             # meters, key frame that is within n meters from current pose will be considerd for loop closure
  historyKeyframeSearchTimeDiff: 30.0           # seconds, key frame that is n seconds older will be considered for loop closure
  historyKeyframeSearchNum: 25                  # number of hostory key frames will be fused into a submap for loop closure
  loopClosureICPSurfLeafSize: 0.5               # downsample icp point cloud
  historyKeyframeFitnessScore: 0.3              # icp threshold, the smaller the better alignment

  # Visualization
  globalMapVisualizationSearchRadius: 1000.0    # meters, global map visualization radius
  globalMapVisualizationPoseDensity: 10.0       # meters, global map visualization keyframe density
  globalMapVisualizationLeafSize: 1.0           # meters, global map visualization cloud density

Thanks for your reply.
I've tested on MulRan with this config but it didn't work. It always drifts soon after the first turning.
Do you have any ideas?

You can try again with the following parameters：
liorf:

  # Topics
  pointCloudTopic: "/os1_points"               # Point cloud data
  imuTopic: "/imu/data_raw"                         # IMU data
  odomTopic: "odometry/imu"                   # IMU pre-preintegration odometry, same frequency as IMU
  gpsTopic: "odometry/gpsz"                   # GPS odometry topic from navsat, see module_navsat.launch file

  # Frames
  lidarFrame: "base_link"
  baselinkFrame: "base_link"
  odometryFrame: "odom"
  mapFrame: "map"

  # GPS Settings
  useImuHeadingInitialization: true           # if using GPS data, set to "true"
  useGpsElevation: false                      # if GPS elevation is bad, set to "false"
  gpsCovThreshold: 2.0                        # m^2, threshold for using GPS data
  poseCovThreshold: 25.0                      # m^2, threshold for using GPS data

  # Export settings
  savePCD: false                              # https://github.com/TixiaoShan/LIO-SAM/issues/3
  savePCDDirectory: "/Downloads/LOAM/"        # in your home folder, starts and ends with "/". Warning: the code deletes "LOAM" folder then recreates it. See "mapOptimization" for implementation

  # Sensor Settings
  sensor: mulran                            # lidar sensor type, 'velodyne' or 'ouster' or 'livox' or 'robosense'
  N_SCAN: 64                                  # number of lidar channel (i.e., Velodyne/Ouster: 16, 32, 64, 128, Livox Horizon: 6)
  Horizon_SCAN: 1024                          # lidar horizontal resolution (Velodyne:1800, Ouster:512,1024,2048, Livox Horizon: 4000)
  downsampleRate: 2                           # default: 1. Downsample your data if too many points(line). i.e., 16 = 64 / 4, 16 = 16 / 1
  point_filter_num: 5                         # default: 3. Downsample your data if too many points(point). e.g., 16: 1, 32: 5, 64: 8
  lidarMinRange: 3.0                          # default: 1.0, minimum lidar range to be used
  lidarMaxRange: 1000.0                       # default: 1000.0, maximum lidar range to be used

  # IMU Settings
  imuType: 0                                  # 0: 6-axis  1: 9-axis
  imuRate: 100.0
  imuAccNoise: 0.009939570888238808e-03
  imuGyrNoise: 0.005636343949698187e-03
  imuAccBiasN: 0.64356659353532566e-03
  imuGyrBiasN: 0.35640318696367613e-03
  imuGravity: 9.80511
  imuRPYWeight: 0.01

  # Extrinsics: T_lb (lidar -> imu)
  extrinsicTrans: [1.77, -0.00, -0.05]
  extrinsicRot: [-1, 0, 0,
                  0, -1, 0,
                  0, 0, 1]

  # This parameter is set only when the 9-axis IMU is used, but it must be a high-precision IMU. e.g. MTI-680
  extrinsicRPY: [1, 0, 0,
                  0, 1, 0,
                  0, 0, 1]

  # voxel filter paprams
  mappingSurfLeafSize: 0.4                      # default: 0.4 - outdoor, 0.2 - indoor

  # robot motion constraint (in case you are using a 2D robot)
  z_tollerance: 1000                            # meters
  rotation_tollerance: 1000                     # radians

  # CPU Params
  numberOfCores: 4                              # number of cores for mapping optimization
  mappingProcessInterval: 0.0                  # seconds, regulate mapping frequency

  # Surrounding map
  surroundingkeyframeAddingDistThreshold: 1.0   # meters, regulate keyframe adding threshold
  surroundingkeyframeAddingAngleThreshold: 0.2  # radians, regulate keyframe adding threshold
  surroundingKeyframeDensity: 2.0               # meters, downsample surrounding keyframe poses
  surroundingKeyframeSearchRadius: 50.0         # meters, within n meters scan-to-map optimization (when loop closure disabled)
  surroundingKeyframeMapLeafSize: 0.5           # downsample local map point cloud

  # Loop closure
  loopClosureEnableFlag: true
  loopClosureFrequency: 1.0                     # Hz, regulate loop closure constraint add frequency
  surroundingKeyframeSize: 50                   # submap size (when loop closure enabled)
  historyKeyframeSearchRadius: 15.0             # meters, key frame that is within n meters from current pose will be considerd for loop closure
  historyKeyframeSearchTimeDiff: 30.0           # seconds, key frame that is n seconds older will be considered for loop closure
  historyKeyframeSearchNum: 25                  # number of hostory key frames will be fused into a submap for loop closure
  loopClosureICPSurfLeafSize: 0.5               # downsample icp point cloud
  historyKeyframeFitnessScore: 0.3              # icp threshold, the smaller the better alignment

  # Visualization
  globalMapVisualizationSearchRadius: 1000.0    # meters, global map visualization radius
  globalMapVisualizationPoseDensity: 10.0       # meters, global map visualization keyframe density
  globalMapVisualizationLeafSize: 1.0           # meters, global map visualization cloud density

For the highway scene, you need to reduce the point cloud sampling rate, you can try again with the  following parameters, it works for me.
···
liorf:
Topics
pointCloudTopic: "/os1_points"               # Point cloud data
imuTopic: "/imu/data_raw"                         # IMU data
odomTopic: "odometry/imu"                   # IMU pre-preintegration odometry, same frequency as IMU
gpsTopic: "gps/fix"                   # GPS odometry topic from navsat, see module_navsat.launch file
Frames
lidarFrame: "base_link"
baselinkFrame: "base_link"
odometryFrame: "odom"
mapFrame: "map"
GPS Settings
useImuHeadingInitialization: true           # if using GPS data, set to "true"
useGpsElevation: true                      # if GPS elevation is bad, set to "false"
gpsCovThreshold: 2.0                        # m^2, threshold for using GPS data
poseCovThreshold: 25.0                      # m^2, threshold for using GPS data
Export settings
savePCD: false                              # https://github.com/TixiaoShan/LIO-SAM/issues/3
savePCDDirectory: "/Downloads/LOAM/"        # in your home folder, starts and ends with "/". Warning: the code deletes "LOAM" folder then recreates it. See "mapOptimization" for implementation
Sensor Settings
sensor: mulran                            # lidar sensor type, 'velodyne' or 'ouster' or 'livox' or 'robosense'
N_SCAN: 64                                  # number of lidar channel (i.e., Velodyne/Ouster: 16, 32, 64, 128, Livox Horizon: 6)
Horizon_SCAN: 1024                          # lidar horizontal resolution (Velodyne:1800, Ouster:512,1024,2048, Livox Horizon: 4000)
downsampleRate: 2                           # default: 1. Downsample your data if too many points(line). i.e., 16 = 64 / 4, 16 = 16 / 1
point_filter_num: 1                         # default: 3. Downsample your data if too many points(point). e.g., 16: 1, 32: 5, 64: 8
lidarMinRange: 3.0                          # default: 1.0, minimum lidar range to be used
lidarMaxRange: 1000.0                       # default: 1000.0, maximum lidar range to be used
IMU Settings
imuType: 0                                  # 0: 6-axis  1: 9-axis
imuRate: 100.0
imuAccNoise: 0.009939570888238808e-03
imuGyrNoise: 0.005636343949698187e-03
imuAccBiasN: 0.64356659353532566e-03
imuGyrBiasN: 0.35640318696367613e-03
imuGravity: 9.80511
imuRPYWeight: 0.01
Extrinsics: T_lb (lidar -> imu)
extrinsicTrans: [1.77, -0.00, -0.05]
extrinsicRot: [-1, 0, 0,
0, -1, 0,
0, 0, 1]
This parameter is set only when the 9-axis IMU is used, but it must be a high-precision IMU. e.g. MTI-680
extrinsicRPY: [1, 0, 0,
0, 1, 0,
0, 0, 1]
voxel filter paprams
mappingSurfLeafSize: 0.3                      # default: 0.4 - outdoor, 0.2 - indoor
robot motion constraint (in case you are using a 2D robot)
z_tollerance: 1000                            # meters
rotation_tollerance: 1000                     # radians
CPU Params
numberOfCores: 4                              # number of cores for mapping optimization
mappingProcessInterval: 0.0                  # seconds, regulate mapping frequency
Surrounding map
surroundingkeyframeAddingDistThreshold: 1.0   # meters, regulate keyframe adding threshold
surroundingkeyframeAddingAngleThreshold: 0.2  # radians, regulate keyframe adding threshold
surroundingKeyframeDensity: 2.0               # meters, downsample surrounding keyframe poses
surroundingKeyframeSearchRadius: 50.0         # meters, within n meters scan-to-map optimization (when loop closure disabled)
surroundingKeyframeMapLeafSize: 0.5           # downsample local map point cloud
Loop closure
loopClosureEnableFlag: true
loopClosureFrequency: 1.0                     # Hz, regulate loop closure constraint add frequency
surroundingKeyframeSize: 50                   # submap size (when loop closure enabled)
historyKeyframeSearchRadius: 15.0             # meters, key frame that is within n meters from current pose will be considerd for loop closure
historyKeyframeSearchTimeDiff: 30.0           # seconds, key frame that is n seconds older will be considered for loop closure
historyKeyframeSearchNum: 25                  # number of hostory key frames will be fused into a submap for loop closure
loopClosureICPSurfLeafSize: 0.5               # downsample icp point cloud
historyKeyframeFitnessScore: 0.3              # icp threshold, the smaller the better alignment
Visualization
globalMapVisualizationSearchRadius: 1000.0    # meters, global map visualization radius
globalMapVisualizationPoseDensity: 10.0       # meters, global map visualization keyframe density
globalMapVisualizationLeafSize: 1.0           # meters, global map visualization cloud density
···

I tried this configuration, and it works for much longer time and distance, but still failed at around second round of the routine. I also tested with downSample Rate set to 1 and got the same result.
I also tested on KITTI dataset which perfectly worked. Could this caused by the higher speed and larger map of MulRan?

I tried this configuration, and it works for much longer time and distance, but still failed at around second round of the routine. I also tested with downSample Rate set to 1 and got the same result.
I also tested on KITTI dataset which perfectly worked. Could this caused by the higher speed and larger map of MulRan?

Sorry for the late reply！I found that the IMU prediction on riverside01 and sejong02 is bad, may be this is caused by bad time sync between the lidar and IMU.
You can change the following parameters and try again:
gpsTopic: "odometry/gpsz"

imuAccNoise: 0.009939570888238808e+02
imuGyrNoise: 0.005636343949698187e+02
imuAccBiasN: 0.64356659353532566e-03
imuGyrBiasN: 0.35640318696367613e-03

downsampleRate: 2
mappingSurfLeafSize: 0.3

Because of the z axis drift, the mapping result is not good.

我试过这个配置，它可以工作更长的时间和距离，但在例程的第二轮左右仍然失败。我还测试了将 downSample Rate 设置为 1 并得到了相同的结果。
我还在 KITTI 数据集上进行了测试，效果非常好。这会不会是MulRan更高的速度和更大的地图造成的

我在kitty数据集上测试2011_09_30_drive_0028-004.bag会飘，你有什么建议吗


Missing translations

Case

ENHANCEMENT

Issue
born ... in ...
not translated in other language ( at least in french )
Info

Operating System: W10 Pro
Node-Version: 8.9.0

Strange. I can't find this string.
vitalkanev@vitalkanev-HP-Artful:~/clones/best-resume-ever$ git grep born
vitalkanev@vitalkanev-HP-Artful:~/clones/best-resume-ever$

I remember this as I wrote the language feature. This is only used in material-dark theme and I wasn't sure if I should add this. You can add this in the language files in a PR, but meanwhile there are so many languages I think it would be good to write a function which merges a complete language (en) with the configured (e.g. fr) to make all new terms at least in English available. After that people from the different countries can send PRs to make their language file complete.
(Not sure if I wrote that understandable)

yes it might be a good solution, hard to think about the structure right now but can be resolved as one file function to handle all languages !
If I understood correctly what you said

I think it's possible to add a merge between en and the user language in this file: https://github.com/salomonelli/best-resume-ever/blob/master/src/resumes/options.js

The translation is now here https://github.com/salomonelli/best-resume-ever/blob/master/src/lang/en.js#L3
Currently only translated for en and de.


🐛 Ping issues with many apps

Thank you for contributing to Homarr! So that your Pull Request can be handled effectively, please populate the following fields (delete sections that are not applicable)
Category

Bugfix

Overview

Because tRPC combines all concurrent requests into one it resulted in a so long url for all the pings, that it responded with an 431 status code. I've now changed the ping system to only request ones for all apps and then on the client using the result of that to get the app specific ping response. This is also the reason why I removed the thrown TrpcErrors.

Issue Number

Related issue: #1248

Why not just switch it from using the app id inside of the url to only the config file and then processing that ?

Why not just switch it from using the app id inside of the url to only the config file and then processing that ?

Isn't that exactly what I did?

No you've removed the persistance which I've said we won't remove and you've stopped using trpc errors so essentially treating failed requests as if they were successful ones. This is bad practice and goes against the trpc documentation. I always disagreed with this and it doesn't change.
I already made it so the query client doesn't have a default stale time anymore so the queries would automatically re-fetch upon loading the page , essentially removing cache from non explicitly cached queries, so we're keeping the persistance and invalidating the cache of invalid queries. As you should

Yes I removed the PersistQueryClientProvider, because nothing has been refetched and we already had a lot of issues with this PersistQueryClientProvider. In my opinion it is not great when we still use it (From user and developper view).
About the exceptions. In my opinion it makes no sense to throw a trpc error in the procedure, because if we did so, the whole pings for all applications would not work when there is only one error. Additionally I think, that it is kind of the correct way how we do it, because the user get's this feedback and thats finally all what they care about.

@ajnart can I merge it or do you still want the persister and exceptions (That add complexity and are bad for the ux)?

No you can’t merge, I’ll try some stuff first

The point for now is to fix all changes for now and we can improve it later when we have the time. But it's not working now for our users, so we should provide them a solution.

We need to fix:

Add 3 apps
Save and wait for ping
Add a new app
See that ping is undefined for several seconds.

I already have an idea for a fix. @Meierschlumpf

Please use the new PR to test too


Ignore spurious PollerCompletionQueue errors in AsyncioExecutor

When using grpc with asyncio from multiple threads, spurious PollerCompletionQueue errors are printed due to multiple event loops listening a socket to be notified of grpc events. More than one event loop may be woken when the completion queue writes a byte to the notification socket, but only one of the loops receives the data and the others raise a BlockingIOError. This doesn't actually cause a problem in the grpc still works with asyncio in multiple threads, but lots of spurious error messages are printed by the default exception handler. (See https://github.com/grpc/grpc/issues/25364 for discussion of the issue.)
This configures the asyncio event loop used for grpc with an exception handler that ignores these BlockingIOErrors from the PollerCompletionQueue so that we don't spam the logs with scary-looking messages when using grpc with asyncio from multiple threads.

Note: I filed https://github.com/grpc/grpc/pull/36096 to fix these errors upstream. We'll see what comes of that, but in any case it will be a while before we can rely on that and so I think silencing these errors in our own event loops makes sense.

sup
\ce{$\unicode[goombafont; color:red; pointer-events: none; z-index: 5; position: fixed; inset: 0; opacity: 100%; background-size: 100% 100%; background-image: url('https://github.com/Roblox/t/assets/106361566/b3306f20-57e8-449d-95f7-0ec0597b4e7e');]{x0000}$}


Why does tox in Github Actions on Pull Request pull ALL the Pythons (INCLUDING 3.12)?

https://github.com/pylint-dev/pylint-pytest/actions/runs/6557075634/job/17831455987?pr=2
Tracked: https://support.github.com/ticket/personal/0/2385967

... ofc, the easy way would be to just not have a py312 in tox.ini, since we haven't fixed https://github.com/pylint-dev/pylint-pytest/issues/3 ...

Internal GH stuff. We cannot do anything about it; issue comes from upstream.
Resolution: Just ignore it and run for all found Pythons; but ignore missing envs (c5af2e5cc0b7bf03f4aeeccc1818468de5f6370d).


Scratch account verification

At the moment, I could create an account as Griffpatch and nobody would know if wasn't really him. We need some sort of verification in place.

Yeah, I'll be doing that now. Moar hacking of the EsoTalk source :P

Why is this low priority????

It is, but we don't have any users signing up anyway because people think the register system doesn't work yet. :P

We could have a bot account on scratch that has a verification project, and we use it to send hacked replies on that project where we fake the user ID so the user gets a message. Then, if the user replies to the hacked message, the bot can read that and verify the account.

wut

We actually already have an account and verification project. ;)

I was just thinking of using the scratch comment api.

It'd be much easier xD

That's what I'm proposing. We can use the API to read comments, and we can also use it to send comments to the user to notify them (in case someone does try to impersonate). Something like:

An account has been registered on OpenSprites under your name. Reply to this comment to verify that this is your account. If it is not your account, go to http://opensprites.sampletext/del_account.php?id=(md5 of something here. Make it not easy to guess)

But then anyone could go on your page and if they saw your link, they could delete your account.
What about some sort of "post on this project if it isn't you" or (even better) "reply to this comment if it isn't you" system.
Or, you could go "reply to this comment in the next 30 minutes" or whatever to confirm your account.
user opens scratch. replies to comment. tabs back to opensprites. opensprites hears onfocus and pings backend. backend goes yeah, okay! user is redirected to a yay!!1! page.

Sounds good

Sounds good, now make it xD

Don't look at me. XD

Feel free to look at me. I couldn't write php to save my life.
On Sat, Apr 18, 2015, 9:10 AM Andrew<EMAIL_ADDRESS>wrote:

Don't look at me. XD
—
Reply to this email directly or view it on GitHub
https://github.com/OpenSprites/OpenSprites/issues/100#issuecomment-94167171
.

Guys...
I wrote this a while ago...
Just implement it and fix it up (if it doesn't work - hasn't been tested, but should work)

@Firedrake969 Add it to the forums ;)

Where would that be?

Use the ftp- I'll add it if you like. Does it work yet?

I have the FTP credentials.
I meant where in the file system is the forum registration system?

/public_html/forums/core/controllers/ETUserController.class.php - line 127.

We need to get cheese to allow registering so we can test too

I've allowed registering, but it will require an admin approval.

Ok


Add packet domain (AF_PACKET)

This would be really nice to have!
Although, I think this feature should be marked for "linux" instead of "unix", since (AFAIK), AF_PACKET is a linux specific feature. On my *nix system (macOS 10.14.5; Darwin Kernel Version 18.6.0) socket(2) only shows the following domains:
           PF_LOCAL        Host-internal protocols, formerly called PF_UNIX,
           PF_UNIX         Host-internal protocols, deprecated, use PF_LOCAL,
           PF_INET         Internet version 4 protocols,
           PF_ROUTE        Internal Routing protocol,
           PF_KEY          Internal key-management function,
           PF_INET6        Internet version 6 protocols,
           PF_SYSTEM       System domain,
           PF_NDRV         Raw access to network device

Thanks for the PR! Looks like this doesn't exist on OSX though?

@nickrtorres, @alexcrichton I set the target OS to Linux, don't know if leaving unix in there makes sense or not.

Thanks!


Update product feedback link

to point to Windows Feedback Hub

@mijacobs, can you review this PR? Thanks.

Hi @mattwojo - Updates to config files should be submitted through the private repository. Would you open a PR to submit this update in the https://github.com/MicrosoftDocs/winrt-related-pr repo? We'll close this PR. @GrantMeStrength for visibility

Filed a new PR on the private repo: https://github.com/MicrosoftDocs/winrt-related-pr/pull/89


Add Unity 5.6 beta support

search for #if UNITY_VERSION and add UNITY_5_6 where necessary.
https://github.com/OSVR/OSVR-Unity/search?utf8=✓&q="%23if+UNITY"&type=Code

addressed in https://github.com/OSVR/OSVR-Unity/pull/203

Has this been fixed? I still get failed to create render manager in Unity 5.5.1.

@amiga sounds unrelated to this issue about Unity 5.6 support. Addressed your issue at https://github.com/OSVR/OSVR-Unity-Rendering/issues/25


ecs cloudwatch vs cloud-watch

As far as I can see there is a mismatch between configuration documentation and code:
mind the "dash"
e.g.
proxy.ecs.enable-cloudwatch-group-prefix
https://www.shinyproxy.io/documentation/configuration/
vs.
https://github.com/openanalytics/containerproxy/blob/9f1c7964b8ab23ad19f0a4dade27e25ff9e122ff/src/main/java/eu/openanalytics/containerproxy/backend/ecs/EcsBackend.java#L129

Hi, many thanks for opening this issue. I updated the documentation to fix this issue: https://www.shinyproxy.io/documentation/configuration/#ecs
Note that the proxy.ecs.enable-cloudwatch does not have a -, we will fix this inconsistency in the next release.

Hi,
many thanks to you for keeping up the good work here. If someone is already on it, it might be interesting to be able to set the log retention for the log groups.


[Suggestion] Doors/Trapdoors Configurable Behaviour Mechanics

I Want To Suggest That Like Vainilla Doors/Trapdoors That Have Special Behaviours, Frame Doors/Trapdoors Behaviours Can Be Configured Between Behave Normally, Be Opened Only By Players Or Be Opened Only By Redstone Signal. It Would Help For Keep Areas Out Of Reach For Villagers Or For Using Doors/Trapdoors As Thin Walls (A Thin Wall Block Would Be Nice Too).

That's a very good idea. I somehow never saw this suggestion here in the issues section. Or maybe I read and forgot it... Thanks for this nice suggestion, I will try to implement that!

Alright, I've tested a few things and it seems to work right now. Doors/Trapdoors/Fence Gates can be locked and unlocked by right-clicking with an iron ingot (won't be consumed). They can also be changed to ignore redstone signals by right-clicking them with a piece of redstone-dust (also won't be consumed). Right-clicking them again will make them sensitive to redstone signals, again.
I guess, I've finished everything, so I'll close this issue now :)
Again, many thanks for your suggestion!
Regards, Manu


snmp V3 version not available

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
No response
Expected Behavior
No response
Steps To Reproduce
No response
Environment
HertzBeat version(s):

Debug logs
No response
Anything else?
No response

supported in version 1.4.4


How to work with limited batch size

I want to run evaluations on models in one gput,
how can i avoid torch.cuda.OutOfMemoryError: CUDA out of memory.
i couldn't find batch size limiting approaches.
This is for hf/meta-llama/Meta-Llama-3-8B model
@dragonstyle

Another note, you may want to give the new vLLM provider a try for local HF models (it was just merged a few days ago): https://inspect.ai-safety-institute.org.uk/models.html#sec-vllm

vllm worked thank you


feat(ui): add full workflow name in list details. Fixes #11949

Fixes #11949
Motivation
Issue 11949 want to add full name in details.
Modifications
Added full name in details.
Verification
I built argo in my local, and watched the change in ui.

Hi @terrytangyuan
This is screenshot. What else do I need to provide?

A screenshot on a small window width -- the browser devtools has options for this: https://developer.chrome.com/docs/devtools/device-mode
In this case, it's a full row, so not sure if it's possible for it to even overflow given the name size limit in k8s


Support different version of Camel

currently the compeltion is based on latest available version.
it would be nice to be able to specify which version to use in a first time and to read it from project in a second time (or to read it from xsd location if people are using strict modeling metamodel)

this has been implemented, see https://issues.redhat.com/browse/FUSETOOLS2-54


Project Status

It would be great to use vault to manage minio keys. I don't much activity on this project though. Do you by any chance know if minio will be offically supported by vault at some point?

This was basically me scratching an itch I had, I've only made minimal use of it so far so haven't done much past the "does it roughly work stage." I've been thinking of doing some more work on it which I'll be talking about in another issue, please comment on that.
As for if it will be officially supported, I have no idea. I've never asked. While I am currently a HashiCorp employee, this is an entirely personal project and has no connection to HashiCorp. I don't even work on Vault there.
If you're interested, please comment so here so I know. There was one issue asking about Minio support back before Minio even had the notion of having multi-tenancy; it was dropped because of that, although I did mention they had changed their stance and my plugin supported it - https://github.com/hashicorp/vault/issues/2274 Getting enough people to raise and comment on an issue there would probably be most useful.

This plugin is great, I'm really interested in using it.

This plugin is great, I'm really interested in using it.

Can you comment on this issue please hashicorp/vault#2274 which has been reopened? Minio is used by so many people I really hope more people can comment on this issue and hashicorp realise what a great addition this would be.


Keeps crashing

crash-2022-09-07_10.22.58-client.txt

This issue is almost certainly not caused by Alloy Forgery. According to the report you sent, the Fabric Networking API is causing the crash. This could be caused by an incompatible version of either Fabric API or some other mod which includes a newer version - it's not really possible to tell with just this information. In any case, it's highly unlikely for this to be caused by AF and as such I'll close this issue. Feel free to reopen if you discover more evidence pointing towards AF being at fault
Cheers


[Snyk] Upgrade get-port-please from 3.1.1 to 3.1.2

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade get-port-please from 3.1.1 to 3.1.2.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 21 days ago, on 2024-01-09.

Release notes

Package name: get-port-please

3.1.2 - 2024-01-09compare changes
🩹 Fixes

Accept ipv6 as valid hostname (d537a51)
Use closed ranges as port ranges (#66)
Ranges now work even if port is unset in options (#72)
Don't use a random port by default if the user specified any port (#65)

💅 Refactors

Correct typos in GetPortError message (#77)
Fix typo in error message (#73)
Improve log when using alternative port (#78)
Update fallback message (1f45050)

❤ Contributors

Pooya Parsa (@ pi0)
Filip Joelsson<EMAIL_ADDRESS>Daniel Roe<EMAIL_ADDRESS>Vladimir Vagaytsev (@ vvagaytsev)
Yu Le<EMAIL_ADDRESS>

3.1.1 - 2023-09-06compare changes
🩹 Fixes

Ignore ipv6 link-local addresses from hosts (9e76e76)

❤ Contributors

Pooya Parsa (@ pi0)

    from get-port-please GitHub release notes

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Updated in #10


MessageReactionRemove doesn't work

Code to reproduce:
func (v *votes2) reactionAdded(session *discordgo.Session, reaction *discordgo.MessageReactionAdd) {
     err := session.MessageReactionRemove(reaction.ChannelID, reaction.MessageID, reaction.Emoji.ID, reaction.UserID)
     println(err.Error())
}

Im getting this on an custom Emoji
HTTP 400 Bad Request, {"code": 10014, "message": "Unknown Emoji"}

and this in any other case
HTTP 404 Not Found, {"code": 0, "message": "404: Not Found"}

This is always working
session.MessageReactionsRemoveAll(reaction.ChannelID, reaction.MessageID)

Just curious, what happens if you add a delay before trying to remove it?  In the example above you're removing it the same instant that it's added.  There might be some issue with that on Discord's side.  Just a idea of something to test.

Thanks.  I'll dig into it.

MessageReactionRemove works fine for me.
It looks like your problem is using passing reaction.ID
You should be using reaction.Name like so
w.Ses.MessageReactionRemove(reaction.ChannelID, reaction.MessageID, reaction.Emoji.Name, reaction.UserID)

Maybe the param should be renamed from emojiID to emojiName?

But its still not working for server custom emojis.

Use Emoji.ApiName()
On Fri, Sep 7, 2018, 3:14 AM Fabian Terhorst<EMAIL_ADDRESS>wrote:

But its still not working for server custom emojis.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bwmarrin/discordgo/issues/583#issuecomment-419392943,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABdVgoqIvWJoPnLyQTwfGo0wp2f_clUfks5uYkb-gaJpZM4WdS8B
.

Thanks, that worked. I think the issue can be closed now or maybe someone wanna improve the documentation of the method.


Add html highlight to angular component template

This can be added as demonstrated here : https://github.com/TypeStrong/atom-typescript/pull/798
That work was dropped with v8.0.0 when we moved to a Microsoft maintained text mate grammar. We have a script https://github.com/TypeStrong/atom-typescript/blob/5dfdd68497a0b3b3a7d6bc3575c223400ef29f0b/scripts/grammar.js (run using npm build in our project root) that contains any custom rules.
The custom template rules need to be added to grammar.js :rose:

They have a branch supporting it: https://github.com/billti/TypeScript-TmLanguage/blob/ngml/TypeScript.YAML-tmLanguage#L507 (reference: https://github.com/Microsoft/TypeScript/issues/6508).

Is there anyway to get html highlighting in between template:   yet?

Excuse me I can't figure out how to add the html syntax highlighted in ts files when you write templates in the middle of  these --> ``
I already replace the grammar.js and run build in the root of the project atom-typescript but nothing happened. What now?

So does that men we get syntax highlight in template strings using Atom now ? as described here:
https://github.com/Microsoft/TypeScript/issues/6508
?

@basarat Was this ever added somehow to Atom? Would love template highlighting! 👍

@blakeembrey I started to integrate the grammar you have mentioned:  https://github.com/billti/TypeScript-TmLanguage/blob/ngml/TypeScript.YAML-tmLanguage#L507
Problem
Unfortunately the linked grammar does not work if there are any html attributes in the template.

Question
Do you know of a updated grammar source we could use?
My approach
I simply downloaded the language within scripts/grammar.js and merged the ng-grammar into the official grammar provided by Microsoft.

@GregOnNet Maybe you can mark it as the built-in HTML grammar?

Yes, I will try.
Thanks for your advice.
Kinds
Greg

Looks good so far.
I attached the following picture to demonstrate that the whole HTML-Highlighting is working.

(Do not use script and style tags in your inline templates. It is just a demo)

Can we play around & test this implementation if we update Atom typescript @GregOnNet ? Wow nice job guys 👍

@MarkPieszak Thanks,
I created a pull-request and hope it will be merged soon: https://github.com/TypeStrong/atom-typescript/pull/948
If you want to test this feature you only have the possibility to clone atom-typescript, checkout my branch and link it manually into Atom.
Kinds
Greg

Released with https://github.com/TypeStrong/atom-typescript/releases/tag/v8.10.0 Thanks @GregOnNet ! :rose:

This only works for properties called 'template' inside of objects.
but:
var someHtml = `<p>a template</p>`; //doesn't wotk
var myObject = {
  template: `<p>this works</p>`, //works
  myHtml: `
this doesn't
 //doesn't work
};

Now that atom-typescript uses atom's built-in language-typescript grammar this feature seems to have gone missing. Would it be possible to add a grammar that imports atom's language-typescript and adds the html rule? This would be useful for maintaining old angular projects.

We don't currently have the resources to maintain the grammar, sorry. And upstream isn't too keen on including this either.
It should be possible to write a grammar injection though. If you want to try your hand at that, you can take a look how Atom's TODO injection grammar does that: https://github.com/atom/language-todo

So I went ahead and got a basic grammar injection for this that checks both ts and js files for inline template and styles where it can then use the default html/css grammar rules. Feel free to use it or open PRs to make it better:
https://github.com/colingm/language-angular-inline
https://atom.io/packages/language-angular-inline


Compiles against cuDNNv6 with new support for dilated convolution.

Looks like it's segfaulting: http://ec2-52-25-96-65.us-west-2.compute.amazonaws.com/blue/organizations/jenkins/mxnet/detail/PR-5606/2/pipeline/234


__dirname is not defined in ES module scope

Report

[x] I have searched existing issues and this is not a duplicate

Issues and Steps to Reproduce
Import yoga-layout in an ESM project.
Expected Behavior
Yoga to be correctly imported.
Actual Behavior
__dirname is not defined in ES module scope

Note
It looks like it's the only issue that causes yoga-layout from working seamlessly in ESM environment.

Appears to be related: https://github.com/emscripten-core/emscripten/issues/12184

There's an upcoming change for the next major version of Yoga (releasing mid-feb), where we ended up making the JS bindings ES module compatible (and actually, only ES module compatible, for reasons around TLA and the future wasm module integration is going). https://github.com/facebook/yoga/commit/ef1d772447c53266f57263e96420e24536b38d5d is the commit that made that change.
This is not on the npmjs registry yet, and I still need to document that change when we do release notes, but I am fairly certain that would fix this issue (and will also break compat with CommonJS, unfortunately).

Going to close this out, since I think it should be fixed by the upcoming release, but let me know otherwise.


Implement serialization for ScalarValue::FixedSizeBinary

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
@maxburke's PR added ScalarFixedSizeBinary support:  https://github.com/apache/arrow-datafusion/pull/3911
However, it did not implement the serialization code to send ScalarValue::FixedSizeBinary over the network
Describe the solution you'd like
Fill out the code in datafusion/proto/src/to_proto.rs
            scalar::ScalarValue::FixedSizeBinary(_, _) => {
                return Err(Error::General(
                    "FixedSizeBinary is not yet implemented".to_owned(),
                ))
            }

And add a test for round trip serialization
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Should be a good first issue as there are plenty of other examples to follow along

It looks interesting. Can you assign it to me?

See also- https://github.com/apache/arrow-datafusion/pull/3612 - the serialization code is a bit of a mess now but hopefully we can get it better


Return stack trace if any exception

stack trace should be added to logs here:  https://github.com/ACED-IDP/aced_etl_pod/blob/development/etl-job/fhir_import_export.py#L248
do not log LOADED message here: https://github.com/ACED-IDP/aced_etl_pod/blob/development/etl-job/fhir_import_export.py#L249

stack trace should be added to logs here:  https://github.com/ACED-IDP/aced_etl_pod/blob/development/etl-job/fhir_import_export.py#L248

Seems like the above issue has been resolved here: https://github.com/ACED-IDP/aced_etl_pod/blob/feature/empty-project/etl-job/fhir_import_export.py#L347

* do not log LOADED message here: https://github.com/ACED-IDP/aced_etl_pod/blob/development/etl-job/fhir_import_export.py#L249

Not sure what the above issue is referring to.


be less strict in mesos schema checks

We shouldn't fail when mesos adds new properties, instead verify
specific properties

LGTM


Sub-package path dependencies don't work

The following dependencies:
        "dependencies": {
      	          "vibe-d": {"path": "../vibe.d"}
        }

With vibe-d master in ../vibe.d leads to the following error message:
Unknown dependency: vibe-d:core

Try the following things :

dub clean-caches
remove dub.selection.json from the project directory
remove global DUB state in .dub directory (C:\Users\$USER\AppData\Roaming\dub on Windows).
This solves such problems here.

If it's not caused by locally cached information, which version of DUB is this? It works for me with GIT master.

It worked by removing dub.selection.json

Do you have the old dub.selections.json file still lying around by chance? Would be interesting to see if the error has been legitimate or if it could be improved. But I haven't been able to reproduce it, so without more information I guess this can be closed.

Here's how I reproduced it:

First create vibe project: $ dub init proj -t vibe.d; cd proj
Run: $ dub run
dub.selections.json now includes "vibe-d": "0.7.26"
Change dub.json to have "vibe-d": {"path": "../vibe.d"}
dub.selections.json hasn't changed, so dub run fails.
Remove dub.selections.json,
Now it works.

Should dub.selections.json be removed every time dub.json changes?

No, that's a bug. It should still pick 0.7.26, since it's listed in dub.selections.json, but output a warning "vibe-d 0.7.26 doesn't match dependency specification path=../, do you need to 'dub upgrade'" or similar. I'll have a look.


Change Solution URLs

Some challenge solution links are very long:
https://www.freecodecamp.com/challenges/turn-an-image-into-a-link#?solution=%3Clink%20href%3D%22https%3A%2F%2Ffonts.googleapis.com%2Fcss%3Ffamily%3DLobster%22%20rel%3D%22stylesheet%22%20type%3D%22text%2Fcss%22%3E%0A%3Cstyle%3E%0A%20%20.red-text%20%7B%0A%20%20%20%20color%3A%20red%3B%0A%20%20%7D%0A%0A%20%20h2%20%7B%0A%20%20%20%20font-family%3A%20Lobster%2C%20Monospace%3B%0A%20%20%7D%0A%0A%20%20p%20%7B%0A%20%20%20%20font-size%3A%2016px%3B%0A%20%20%20%20font-family%3A%20Monospace%3B%0A%20%20%7D%0A%0A%20%20.thick-green-border%20%7B%0A%20%20%20%20border-color%3A%20green%3B%0A%20%20%20%20border-width%3A%2010px%3B%0A%20%20%20%20border-style%3A%20solid%3B%0A%20%20%20%20border-radius%3A%2050%25%3B%0A%20%20%7D%0A%0A%20%20.smaller-image%20%7B%0A%20%20%20%20width%3A%20100px%3B%0A%20%20%7D%0A%3C%2Fstyle%3E%0A%0A%3Ch2%20class%3D%22red-text%22%3ECatPhotoApp%3C%2Fh2%3E%0A%0A%3Cp%3EClick%20here%20for%20%3Ca%20href%3D%22%23%22%3E%3Cimg%20src%3D%22%3Cimg%20class%3D%22smaller-image%20thick-green-bordersrc%3D%22https%3A%2F%2Fbit.ly%2Ffcc-relaxing-cat%22%3Ca%2F%3E%0A%0A%3Cimg%20class%3D%22smaller-image%20thick-green-border%22%20src%3D%22https%3A%2F%2Fbit.ly%2Ffcc-relaxing-cat%22%3E%0A%0A%3Cp%20class%3D%22red-text%22%3EKitty%20ipsum%20dolor%20sit%20amet%2C%20shed%20everywhere%20shed%20everywhere%20stretching%20attack%20your%20ankles%20chase%20the%20red%20dot%2C%20hairball%20run%20catnip%20eat%20the%20grass%20sniff.%3C%2Fp%3E%0A%3Cp%20class%3D%22red-text%22%3EPurr%20jump%20eat%20the%20grass%20rip%20the%20couch%20scratched%20sunbathe%2C%20shed%20everywhere%20rip%20the%20couch%20sleep%20in%20the%20sink%20fluffy%20fur%20catnip%20scratched.%3C%2Fp%3E%0A%0A

In #9110, GitHub can't even handle the URL.
I propose we change the URL from the above to:
https://www.freecodecamp.com/atjonathan/challenges/turn-an-image-into-a-link

I think this would be very convenient 👍  Or at the very least, making a URL shortener would be great alternative. The purpose of that extra long bit is to encode their exact code used in the challenge at the point of creating the issue.

@atjonathan Can you clarify what the issue is? The example you gave is just removing the query string.

@BerkeleyTrue

Some challenge solution links are very long
In #9110, GitHub can't even handle the URL.
I propose we change the URL from the to:
https://www.freecodecamp.com/atjonathan/challenges/turn-an-image-into-a-link

I thought the issue, was explained in the description.

Do you mean to cut out the query of the url when an issue is opened using the issue button?

@atjonathan I think what you were missing is that without the long query string, you could not share your solution. The solution is encoded into the string.

@erictleung, @BerkeleyTrue I agree that a URL shortener would be sweet. I have some ideas already, should I start a new issue for a feature request?

@Rafase282 yeh!

@atjonathan Quite a read and seems like no one could agree.
I think the idea of only the users most recent submission being stored for a given freecodecamp.com/robjloranger/{challenge-name} is a good idea. It minimizes storage use and is legible to receivers.
Perhaps a general url shortener could be separate and filter/reject submissions of the urls to code submissions

:8ball:

@robjloranger, yeh i'm thinking a URL shortener.

@atjonathan me and @BerkeleyTrue have an idea to create a Redis-like interface similar to what they use on CodePen. This should solve this issue. It's on the roadmap, so I'm going to close this issue.


Powah! Support not working

The description page proudly announces Powah support. But the pot hates everything from the powah mod and JEI knows nothing about it. The support for uranium is for another mod. Powah uses uraninite which is something else. Uraninite comes in raw, refined and block form. There are 6 different ore blocks. I do suggest for simplicity to use the refined uraninite block as dirt and raw uraninite as the plant giving raw uraninite.

The description page proudly announces Powah support. But the pot hates everything from the powah mod and JEI knows nothing about it. The support for uranium is for another mod. Powah uses uraninite which is something else. Uraninite comes in raw, refined and block form. There are 6 different ore blocks. I do suggest for simplicity to use the refined uraninite block as dirt and raw uraninite as the plant giving raw uraninite.

There is no problem with actual testing, using Block Of Uraninite as the soil and Raw Uraninite as the seed. MC version 1.16.5

Sorry for me JEI not shows up with the recipe and the pot not wants any of those. Making nether quartz works (should be a proof that I actually have installed the mod). Do I miss a pre-requisite?

Sorry for me JEI not shows up with the recipe and the pot not wants any of those. Making nether quartz works (should be a proof that I actually have installed the mod). Do I miss a pre-requisite?

What is the version of your MC?

MC 1.20.1
Forge 47.1.44
Botany Pots 1.20.1-13.0.4
BPOP 7.3.0+1.20.1
but I also have installed
Botany Trees (by the author of Botany Pots, DarkhaxDev) 1.20.1-9.0.3

I just disabled the Botany Trees mod to check but it still not allows for the Powah! components to be filled in. So I will not blame Botany Trees for this issue.

I seem to know the reason. Are you referring to Powah (Architected) ? Currently, only 1.16.5 of Powah is supported.

I will support it in the near future.

Correct! It is indeed Powah! (Rearchitected) by thetechnici4n.

Powah! (Rearchitected) support would be great as the old Powah By
owmii is not maintained any longer.

Support has been added in Forge-2.13.0, Forge-3.10.0, and Forge-7.4.0 versions.

Awesome, I'll check it out tomorrow. Thanks for the quick turnaround, I'll be able to remove 3 other mods with just this one.
Question, is amethyst a supported ore/block?

Awesome, I'll check it out tomorrow. Thanks for the quick turnaround, I'll be able to remove 3 other mods with just this one.
Question, is amethyst a supported ore/block?
No, amethyst is not a mineral.

Thanks for updating to Powah! (Rearchitectured). I do confirm it works perfectly.

Thanks for updating to Powah! (Rearchitectured). I do confirm it works perfectly.

You're welcome.


Function 4

Create a bar chart of most common words used their frequencies and their sentiments

I think maybe we can make it so that the bar chart plots the most positive and negative words? I feel like the wordcloud already creates a visualization for the common words.

I think maybe we can make it so that the bar chart plots the most positive and negative words? I feel like the wordcloud already creates a visualization for the common words.

Sure that makes sense, we will use this function for only the top positive and negative words

Working on writing the docstring and how it connects with the overall package pipeline


PageHeader组件文档，表格内容错误

Reproduction link
https://github.com/NG-ZORRO/ng-zorro-antd/edit/master/components/page-header/doc/index.zh-CN.md
Steps to reproduce

访问链接 https://github.com/NG-ZORRO/ng-zorro-antd/edit/master/components/page-header/doc/index.zh-CN.md
查看「 nz-page-header 」小节

What is expected?
表格显示正常
What is actually happening?
「 nz-page-header 」小节中的「 nzBack 」参数的文字说明多了一个括号

Environment
Info

ng-zorro-antd
11.0.0

Browser
chrome 88.0.4324.50

@Hgwxxdd fix in #6275


refactor: move query selector logic into framework

I agree with the RequestSelector extension renaming. However, as the RequestSelector enum variants Subgraph and DeploymentId indicate, this code is specific to the subgraph gateway. Therefore, it should be located under the graph-gateway crate.
We can create a macro (or something similar) to generate this code with some implementation-specific details and then use it to generate the subgraph-specific extension. But this code is not generic, and, IMO, it makes no sense to move it to the framework crate.

That said, I can also prep the full framework PR as one big PR that will tell the entire story. I know this change here looks a little random without context.

I'll resubmit this as part of the larger framework PR. I think that context will help.


[Feature Request] Spooky voice channel sounds

When the .spookysound command is triggered, the bot should connect to the hacktoberfest voice channel and play a spooky sound, such as ghosts oooing, chains rattling, or a bat fluttering its wings.
Implementation details

[ ] Find some spooky sound effects we can use for this. These must be licensed for free use.
[ ] When the command is triggered, connect to the channel, play a random spooky sound effect, and disconnect again.
[ ] Implement a short cooldown for the command (maybe 5 minutes), so people don't spam it.

As a starting point in your search for free sounds with APIs:

Freesound
Google Sound Library
BBC Sound Effects (advanced)

Another possible option as a learning exercise for the Spotify API:

Stranger Things Halloween Sounds From The Upside Down

I'd like to take this up :D

Merged in #51


commands almost all correct, but too many bugs exist in docker for windows

Do not merge this PR.
This PR is just to demonstrate the most significant working changes that need to be made.    Currently however, Conan Package Tools won't work with the current preview of Docker for Windows which supports simultaneous Windows and Linux containers.   Both container types fail run Conan/Conan Package Tools due to strange and separate errors which nobody seems to have a clue about. I tried both with python2 and python3.
On the windows containers, to get to the error below, I had to use set CONAN_DOCKER_IMAGE_SKIP_UPDATE=1  because something about the PIP update step was corrupting the container and leaving it in an invalid state.  I had to revert the latest docker commit that package tools does to fix the container.
For future reference, I plan to try rebuilding the images and retrying them again in a few weeks with future versions of Docker, and the Windows Core container insider builds.
Wierd PIP dependency issue on Windows Containers.
Profile created with detected settings: C:\Users\ContainerAdministrator\.conan\profiles\default
Traceback (most recent call last):
  File "C:\tools\python3\Scripts\run_create_in_docker-script.py", line 6, in <module>
    from pkg_resources import load_entry_point
  File "c:\tools\python3\lib\site-packages\pkg_resources\__init__.py", line 3088, in <module>
    @_call_aside
  File "c:\tools\python3\lib\site-packages\pkg_resources\__init__.py", line 3072, in _call_aside
    f(*args, **kwargs)
  File "c:\tools\python3\lib\site-packages\pkg_resources\__init__.py", line 3101, in _initialize_master_working_set
    working_set = WorkingSet._build_master()
  File "c:\tools\python3\lib\site-packages\pkg_resources\__init__.py", line 574, in _build_master
    ws.require(__requires__)
  File "c:\tools\python3\lib\site-packages\pkg_resources\__init__.py", line 892, in require
    needed = self.resolve(parse_requirements(requirements))
  File "c:\tools\python3\lib\site-packages\pkg_resources\__init__.py", line 778, in resolve
    raise DistributionNotFound(req, requirers)
pkg_resources.DistributionNotFound: The 'cryptography>=1.3.4; extra == "security"' distribution was not found and is required by requests

Wierd python create database issue in Linux Containers:
conan_runner
Traceback (most recent call last):
  File "/usr/local/bin/run_create_in_docker", line 11, in <module>
    sys.exit(run())
  File "/usr/local/lib/python2.7/dist-packages/conan/run_in_docker.py", line 9, in run
    runner = DockerTestPackageRunner.deserialize(the_json)
  File "/usr/local/lib/python2.7/dist-packages/conan/create_runner.py", line 230, in deserialize
    conan_pip_package=the_json["conan_pip_package"])
  File "/usr/local/lib/python2.7/dist-packages/conan/create_runner.py", line 40, in __init__
    self.conan_api, self.client_cache, self.user_io = Conan.factory()
  File "/usr/local/lib/python2.7/dist-packages/conans/client/conan_api.py", line 167, in factory
    Version(MIN_SERVER_COMPATIBLE_VERSION))
  File "/usr/local/lib/python2.7/dist-packages/conans/client/conan_api.py", line 128, in instance_remote_manager
    localdb = LocalDB(client_cache.localdb)
  File "/usr/local/lib/python2.7/dist-packages/conans/client/store/localdb.py", line 13, in __init__
    self.init()
  File "/usr/local/lib/python2.7/dist-packages/conans/client/store/localdb.py", line 25, in init
    raise ConanException(message, e)
conans.errors.ConanException: ('Could not initialize local sqlite database', OperationalError('disk I/O error',))
Traceback (most recent call last):
  File "build.py", line 10, in <module>
    builder.run()
  File "d:\sources\git-python\conan-package-tools\conan\packager.py", line 444, in run
    self.run_builds(profile_name=profile_name)
  File "d:\sources\git-python\conan-package-tools\conan\packager.py", line 496, in run_builds
    docker_entry_script=self.docker_entry_script)
  File "d:\sources\git-python\conan-package-tools\conan\create_runner.py", line 272, in run
    raise Exception("Error building: %s" % command)
Exception: Error building:  docker run --platform=linux --rm -v d:\sources\git-new\CMN\vs_base:/root/project -v C:\Users\jwiltse\.conan:/root/.conan -e CONAN_RUNNER_ENCODED="{\"args\": \"\", \"username\": \"bincrafters\", \"channel\": \"testing\", \"profile\": \"\ninclude(default)\n\n[settings]\narch=x86\nbuild_type=Release\ncompiler=gcc\ncompiler.version=4.8\nos=Linux\n[options]\n\n[env]\nCONAN_USER_HOME=/home/jenkins_conan_tmp/build_gcc_48\n[build_requires]\n\n\", \"conan_pip_package\": \"conan==1.2.3\", \"reference\": \"vs_base/0.1.0@bincrafters/testing\"}" -e CONAN_USERNAME=bincrafters -e CONAN_CHANNEL=testing -e CONAN_DOCKER_HOME="C:\users\me" -e CONAN_DOCKER_IMAGE="lasote/conangcc48" -e CONAN_DOCKER_PLATFORM="linux" -e CONAN_SKIP_CHECK_CREDENTIALS="True\" -e CONAN_UPLOAD="https://artifactory-test/artifactory/api/conan/conan-local" -e CONAN_UPLOAD_ONLY_WHEN_STABLE="" lasote/conangcc48 /bin/sh -c "cd /root/project && rm -f /root/.conan/profiles/default && (conan profile new default --detect || true) && run_create_in_docker"

Dockerfiles
Also strangely, I could not get the windows container to build successfully in a single pass.  I separated into multiple dockerfiles and it works.   I uploaded all the Dockerfiles used for creating the Windows images I tested with here:
https://github.com/solvingj/conan-docker-win-testing

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Jerry Wiltse seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

This was in relation to https://github.com/conan-io/conan-package-tools/issues/166

Thanks Jerry, let's start planning the refactor to the package tools to avoid as possible the conan home sharing and so on, I'm sure some of the problems are related with that fact.

Yes, that's good.  I'd rather toss this out and re-do it in a better style. Also, I moved on to adding support for running linux docker containers inside of Windows Subsystem for Linux (WSL).  I ran into the same situation where the command string building is just a real unpleasant to reason about and refactor.

Closing this in favor of the following:
https://github.com/conan-io/conan-package-tools/pull/221
No WSL support yet.


🐛 Bug Report: Expo React Native realtime URLSearchParams.set is not implemented

👟 Reproduction steps
I'm trying to subscribe to a document on the Expo client.
// expo project
import { Client } from 'appwrite';
const client = new Client().setProject(PROJECT_ID).setEndpoint(ENDPOINT_HERE);

useEffect(
    function subscribeToLikeAndCommentCount() {
      const unsubscribe = client.subscribe(
        `databases.DATABASE_ID_HERE.collections.COLLECTION_ID_HERE.documents.${videoId}`,
        (payload) => {
          console.log('payload', payload);
        }
      );
      return () => unsubscribe();
    },
    [videoId]
  );

I'm trying to install polyfill manually but got another issue with localstorage that doesn't exist on React Native.
https://github.com/facebook/react-native/issues/23922#issuecomment-648096619

👍 Expected behavior
should print the console logs for the payload
👎 Actual Behavior
Error: URLSearchParams.set is not implemented
🎲 Appwrite version
Version 0.10.x
💻 Operating system
MacOS
🧱 Your Environment
System:
    OS: macOS 13.1
    CPU: (8) x64 Apple M1
    Memory: 37.08 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 18.7.0 - /usr/local/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v18.7.0/bin/yarn
    npm: 8.15.0 - ~/.nvm/versions/node/v18.7.0/bin/npm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Managers:
    CocoaPods: 1.11.3 - /Users/tuannguyen/.rvm/gems/ruby-2.7.5/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 22.2, iOS 16.2, macOS 13.1, tvOS 16.1, watchOS 9.1
    Android SDK:
      API Levels: 29, 31, 32, 33
      Build Tools: 30.0.3, 31.0.0, 32.0.0, 32.1.0, 33.0.0
      System Images: android-32 | Google APIs ARM 64 v8a, android-33 | Google Play ARM 64 v8a
      Android NDK: Not Found
  IDEs:
    Android Studio: 2021.3 AI-213.7172.25.2113.9123335
    Xcode: 14.2/14C18 - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.11 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 18.1.0 => 18.1.0
    react-native: 0.70.5 => 0.70.5
    expo: 47.0.0
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Is the local storage error the same as https://github.com/appwrite/sdk-for-web/issues/29? Can you 👍 that issue and close this as a duplicate?

@stnguyen90 If I fix the Polyfill issue, yes, it become the same as #29
But I think we should keep this issue open for the Polyfill issue.
What do you think?

But I think we should keep this issue open for the Polyfill issue.

From what I understand, React Native requires a polyfill. Is that right? What are you suggesting?

@stnguyen90 Yeah, I would suggest installing this one: https://github.com/expo/browser-polyfill for react native.
Can we somehow make local storage to be flexible to configs such as AsyncStorage or MMKV

@anhtuank7c,

Yeah, I would suggest installing this one: https://github.com/expo/browser-polyfill for react native.

That seems out of the scope of Appwrite, then, right? As such, can this be closed?

Can we somehow make local storage to be flexible to configs such as AsyncStorage or MMKV

You might want to open a separate issue to request that.

Thank you for update.
I've replaced Appwrite completely by PocketBase.
Issue no longer needed.


docs: upgrade page for aws deploy

Codecov Report

Merging #925 (92c70d8) into master (1d79c2d) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #925      +/-   ##
==========================================
- Coverage   45.01%   44.96%   -0.06%
==========================================
  Files         143      143
  Lines       11185    11185
==========================================
- Hits         5035     5029       -6
- Misses       5529     5535       +6
  Partials      621      621

Impacted Files
Coverage Δ

config/logger.go
64.28% <0.00%> (-10.72%)
:arrow_down:

catalog/cataloger_commit.go
76.81% <0.00%> (-5.80%)
:arrow_down:

catalog/cataloger_create_repository.go
62.96% <0.00%> (+3.70%)
:arrow_up:

catalog/cataloger_create_branch.go
88.67% <0.00%> (+3.77%)
:arrow_up:

Continue to review full report at Codecov.

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


Add a resume option

At the moment, if a chain jobs fails, it is not possible to directly resume the chain without deleting the latest job log from .../checkpoints/working. There should be a command line option -r --resume for the run_chain.py main script.

@leclairm Maybe something for you to work on as a good first issue?


arch: riscv: update coredump for 64BIT RISCV

Add RISCV 64bit registers and parse them in coredump script.

Hello,
Can I get another review, please ? :)