esm export?

I'm trying to use rake-db and orchid-rom within my esm module but having issues as it's currently a commonjs module.
any chances we can get esm support?
import { rakeDb } from "rake-db";
         ^^^^^^
SyntaxError: Named export 'rakeDb' not found. The requested module 'rake-db' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from 'rake-db';
const { rakeDb } = pkg;

once i use the method above:
import rakedb_pkg from "rake-db";
const { rakeDb } = rakedb_pkg;

getting
(node:11244) Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.
(Use `node --trace-warnings ...` to show where the warning was created)
C:\Coding\wowtrack.gg\migrator2\node_modules\rake-db\dist\index.esm.js:1
import { singleQuote, quote, isRaw, getRaw, toArray, columnTypes, raw, getColumnTypes, getTableData, resetTableData, ColumnType, emptyObject, TransactionAdapter, logParamToLogObject, Adapter, columnsByType, instantiateColumn, codeToString, addCode, quoteObjectKey, primaryKeyToCode, indexToCode, foreignKeyToCode, TimestampColumn } from 'pqb';
^^^^^^

SyntaxError: Cannot use import statement outside a module
    at Object.compileFunction (node:vm:360:18)
    at wrapSafe (node:internal/modules/cjs/loader:1048:15)
    at Module._compile (node:internal/modules/cjs/loader:1083:27)
    at Module._extensions..js (node:internal/modules/cjs/loader:1173:10)
    at Module.load (node:internal/modules/cjs/loader:997:32)
    at Module._load (node:internal/modules/cjs/loader:838:12)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:170:29)
    at ModuleJob.run (node:internal/modules/esm/module_job:193:25)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:527:24)

also tried
import rakedb_package from "rake-db";

rakedb_package.rakeDb(
  {
    databaseURL,
    ssl: true,
  },
  {
    migrationsPath: "migrations",
  }
);

but same error
(node:2632) Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.
C:\Coding\wowtrack.gg\migrator2\node_modules\rake-db\dist\index.esm.js:1
import { singleQuote, quote, isRaw, getRaw, toArray, columnTypes, raw, getColumnTypes, getTableData, resetTableData, ColumnType, emptyObject, TransactionAdapter, logParamToLogObject, Adapter, columnsByType, instantiateColumn, codeToString, addCode, quoteObjectKey, primaryKeyToCode, indexToCode, foreignKeyToCode, TimestampColumn } from 'pqb';
^^^^^^

SyntaxError: Cannot use import statement outside a module
    at Object.compileFunction (node:vm:360:18)
    at wrapSafe (node:internal/modules/cjs/loader:1048:15)
    at Module._compile (node:internal/modules/cjs/loader:1083:27)
    at Module._extensions..js (node:internal/modules/cjs/loader:1173:10)
    at Module.load (node:internal/modules/cjs/loader:997:32)
    at Module._load (node:internal/modules/cjs/loader:838:12)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:170:29)
    at ModuleJob.run (node:internal/modules/esm/module_job:193:25)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:527:24)

Node.js v18.9.0

Thank you for bringing this out. It compiles to both commonjs and esm, but apparently there is some problem, I'll look into it.

I fixed all esm related issues and it should work now!
(update all libraries to the latest versions to try)
Here is a demo project for ESM:
https://github.com/romeerez/orchid-orm-examples/tree/main/packages/esm

I'll give it a try soon and report back.


questions about the inference procedure of sequential recommendation

in the templates.py file under sft folder there are 2 templates for sequential recommendations, one has candidates in input while the other does not, i wonder which one is used during the evaluation of sequential recommendation tasks? or both are used in some way? Thank you!

@tiebreaker4869
We use 2 templates(with/without candidates) while training. Refer to Train_task_group_mapping["SFTSeqRec"].
In model evaluation, we separate them into 2 classes of instruction. Test_task_group_mapping["SFTTestSeqRec"] without candidates, Test_task_group_mapping["SFTTestSeqRanking"] with candidates.
We use the one without candidates during the evaluation of sequential recommendation tasks.

helpful answers.  thanks for ur explanation!


Singularity containers for faster app server building

A persistently annoying issue is the slow installation process for the large set of R libraries required to run the MDI Stage 2 apps server. It cannot be sped up by any obvious means. We have had success with Singularity containers for running Stage 1 pipelines, which simplifies the similarly slow process of conda environment building for end users.

https://github.com/MiDataInt/mdi-pipelines-framework/releases/tag/v0.3.0

It begs the question of having MDI project support for OCI containers that run apps servers corresponding to a specific R version.
It is not a 1:1 correlation to pipelines, because the slow step for apps is building the bare bones server, not the pipeline-specific conda environments. Users will often need to add R packages for their target apps to a container image and save it as a new image.  That process would likely fall to tool suite developers, just as it does for pipelines, which would mean apps server containers would likely only offer one tool suite. Users would need to choose their tool suite before launching their web server, which breaks down the MDI's central design concept of a tool suite-based multi-server.
Other barriers are that end users would have to ensure installation of container software on their computers, which can be an impediment. At present they must install R, but that's more friendly than Docker or Singularity.
So, it's not obvious how this would go for apps servers. It may be that AWS AMIs - as already implemented - that facilitate rapid launching of pre-installed apps servers may be a superior solution to R Shiny containers. They both offer the advantage of not having to do the slowest part of the install, but the AWS instances make it much easier to install new tool suites on the fly.

After a bit more work implementing live tools with AWS AMIs and working with pipeline containers, I believe the optimal ways to offer pre-installed code environments for end users is different for Stage 1 pipelines and Stage 2 apps:

Stage 1 pipelines = Singularity containers
Stage 2 apps = AWS server AMIs

I am closing this issue for now, we can re-open it again later if someone identifies a clearer path toward used containers for Stage 2 apps.

re-opening this issue
Thinking about the concern that "apps server containers would likely only offer one tool suite" - from some developer and user perspectives, it might be good to think of a suite, not the MDI, as a fundamental unit of code distribution.  It would be natural for many to clone a suite from GitHub and execute it directly, without having to worry about installing the MDI - an executable in the suite could do that for the user.
From a container perspective, the conceptual change that makes more sense for apps servers is for tool suite containers to offer the entire suite, including both pipelines and apps. Users would indeed say "now I need to run suite X". Thus, the MDI could support each of the following conceptual approaches equally well:

MDI-centric: install the mdi, with as many tool suites as the user cares to add
suite-centric:  install as many tools suites as desired, each of which independently maintains its own supporting mdi installation

Additions to implement the suite-centric vision would include:

modifying _config.yml in a tool suite to declare container support, including external suite dependencies
add a renamable 'run' script in the tool suite base directory that calls 'singularity pull/run mdi @args'
add command 'build' to the Stage 1 jobManager to create and share a suite-level container with:

a versioned R base image to support apps servers, as 'mdi-web-server' already does
conda installed and used to build any required pipeline environments, as 'build-common.def' already does

Such uber-containers would be fairly big (several Gb), but would be self contained and guaranteed to run the suite correctly in both Stage 1 and Stage 2 with no slow or complex installation required by end users.

As of v0.4.2, the MDI offers mdi-centric and suite-centric install/run configurations. Developers can support suite-level containers for pipelines and/or apps as fits their needs (pipeline-level containers did not change). It is ~stable, but bugs may be encountered.
Along the way, it was clear that containers can benefit users even if a tool suite developer doesn't maintain their own container registry. MDI project administrators will maintain a registry for an empty suite-level container. If Singularity is available on the host system, the MDI will install and run the MDI app server within a Singularity container, with active directories bind-mounted to the container, including frameworks, suites, data, and sessions. R packages needed by a tool suite but not present in the container - and only those packages - are installed in the user's 'mdi/containers' folder. Thus, all users will enjoy much faster Stage 2 install times whether working in mdi-centri or suite-centric modes. Even R doesn't need to be installed on the host as it runs in the container - the only pre-requisite is Singularity.
Other changes implemented while developing Stage 2 container support speed R package installation by multi-threading, etc.
See the new repository for details:

https://github.com/MiDataInt/mdi-singularity-base


DropdownButtonFormField does not render InputDecoration correctly

Steps to Reproduce

Create a DropdownButtonFormField
Supply the required arguments using DropdownMenuItem(child: Text('Some Text), value: 'Some Text')
Add the line: decoration: InputDecoration(labelText: 'Theme'), to the arguments list.

Expected results:
A label that appears in a reasonable location in relation to the DropDownButton.
Actual results:

I resolved this on my own, it appears that the argument value should also be required and I went down a debugging rabbit hole prematurely.  Is this something that should be fixed or is it "as intended"?

Reproducible code sample
import 'package:flutter/material.dart';

void main() => runApp(MaterialApp(home: MyApp()));

class MyApp extends StatefulWidget {
  @override
  _MyAppState createState() => _MyAppState();
}

class _MyAppState extends State<MyApp> {

  String value;

  @override
  Widget build(BuildContext context) {
    return SafeArea(
      child: Scaffold(
        body: Center(
          child: DropdownButtonFormField(
            decoration: InputDecoration(labelText: 'Theme'),
            onChanged: (newValue) {
              setState(() {
                value = newValue;
              });
            },
            value: value,
            items: [
              DropdownMenuItem(
                child: Text('Dark'),
//                value: 'Dark',
              ),
            ],
          ),
        ),
      ),
    );
  }
}

I was thinking about this a little bit and my first instinct was that we can add an assert at the constructor to ensure that DropdownMenuItem could never be null. However, I realized this would technically be breaking because users might use null as an actual value to display a particular DropdownMenuItem. For example:
DropdownButtonFormField(
  decoration: InputDecoration(labelText: 'Theme'),
  onChanged: (newValue) {
    setState(() {
      value = newValue;
    });
  },
  value: value,
  items: [
    DropdownMenuItem(
      child: Text('Dark'),
      value: 'Dark',
    ),
    DropdownMenuItem(
      child: Text('This value is null'),
      value: null,
    ),
    DropdownMenuItem(
      child: Text('Light'),
      value: 'Light',
    ),
  ],
);

I'm not sure if the breaking change is worth making since there are ways to prevent this from happening, and adding the assert has the potential to break an unknown number of users, but on the flip side of things, helps prevent running into this kind of problem in the first place.

One approach we could attempt to make is to have the InputDecorator acknowledge that there is a null value menu item and properly show the input decorator if a dropdown item with a null value is present.

Is this really worth to have? Allowing null values and expect values like this?
I mean it is very hard to see usage with null values. But we never know

Since both changes necessitate a breaking change (although one more breaking than the other), you could argue that we should just not allow a null value for DropdownMenuItem, but like you said, it's unknown how many developers actually use this pattern.
I think it's valid to go for the bigger breaking change as long as there's a migration path. For example, the migration guide would need to outline how a user would want to set value to null by tapping a DropdownMenuItem if we were to not allow a null value for DropdownMenuItem. If that's not possible, there needs to be a compelling argument for not allowing this behavior (which can be made in the design document that needs to be written as part of the breaking change steps).
From what I can tell from the way DropdownMenuItem is written, not allowing a null value might be a breaking change that cannot be worked around, so I'm concerned about the feasibility of not allowing null values.

I guess one way devs can create a workaround is:
DropdownButtonFormField(
  decoration: InputDecoration(labelText: 'Theme'),
  onChanged: (newValue) {
    setState(() {
      if (newValue == 'placeholder value') {
        value = null;
        return;
      }
      value = newValue;
    });
  },
  value: value,
  items: [
    DropdownMenuItem(
      child: Text('Dark'),
      value: 'Dark',
    ),
    DropdownMenuItem(
      child: Text('Set DropdownButton to null to show hint.'),
      value: 'placeholder value',
    ),
    DropdownMenuItem(
      child: Text('Light'),
      value: 'Light',
    ),
  ],
);

I think the primary situation would be that developers might want to set value to null in order to display the hint. Special logic can be incorporated into the onChanged callback to set that value to null with a unique value in DropdownMenuItem.value.

I think the primary situation would be that developers might want to set value to null in order to display the hint.

Are you talking about DropdownMenuItem or DropdownButtonFormField?

DropdownButtonFormField, but by using the null value passed into DropdownMenuItem. I'm trying to help come up with use cases for which a developer might want to pass null into DropdownMenuItem.value, and that's the biggest one I can think of.

You're right, thins can happen too. If the problem is with show/hide hint, we can add a param to decide wheter to show hide the hint.
Just thinking in possible ways to fix this

Reproducible on latest master channel.
flutter doctor -v
[✓] Flutter (Channel stable, 1.22.5, on Mac OS X 10.15.7 19H2 darwin-x64, locale en)
    • Flutter version 1.22.5 at /Users/pedromassango/dev/SDKs/flutter_stable
    • Framework revision<PHONE_NUMBER> (6 days ago), 2020-12-10 11:54:40 -0800
    • Engine revision ae90085a84
    • Dart version 2.10.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.1, Build version 12A7403
    • CocoaPods version 1.9.3

[!] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2020.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin installed
    • Dart plugin version 203.5981.152

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

[✓] Connected device (1 available)
    • sdk gphone x86 arm (mobile) • emulator-5554 • android-x86 • Android 11 (API 30) (emulator)

! Doctor found issues in 1 category.
Process finished with exit code 0

We also ran into this - in our case, its not that we want a null item in the item list, its that we set the original value to null because we don't want the list to be preset until the user selects a list item. The expected behavior is that it shows the hint and the user then selects an item from the list. Setting it to the first one in the list, makes it too easy for a user to simply bypass the entry even if it is required, because the first one is wrongly selected.

We also ran into this - in our case, its not that we want a null item in the item list, its that we set the original value to null because we don't want the list to be preset until the user selects a list item. The expected behavior is that it shows the hint and the user then selects an item from the list. Setting it to the first one in the list, makes it too easy for a user to simply bypass the entry even if it is required, because the first one is wrongly selected.

Hi everyone
This seems to be working fine now, here I created an example based on https://github.com/flutter/flutter/issues/56601#issuecomment-628156659, where null value was provided.
I don't see layout issues even after using the value in the menu item.

minimal code sample
import 'package:flutter/material.dart';

const List<String> fruits = ["Mango", "Apple", "Banana", "Strawberry"];

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Material App',
      theme: ThemeData.dark(),
      home: const Home(),
    );
  }
}

class Home extends StatefulWidget {
  const Home({Key? key}) : super(key: key);

  @override
  State<Home> createState() => _HomeState();
}

class _HomeState extends State<Home> {
  String? _dropdownbuttonFormValue;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Material App Bar'),
      ),
      body: Center(
        child: DropdownButtonFormField<String>(
          decoration: const InputDecoration(labelText: 'Theme'),
          onChanged: (newValue) {
            setState(() {
              _dropdownbuttonFormValue = newValue;
            });
          },
          value: _dropdownbuttonFormValue,
          items: const <DropdownMenuItem<String>>[
            DropdownMenuItem(
              child: Text('Dark'),
              value: 'Dark',
            ),
            DropdownMenuItem(
              child: Text('Set DropdownButton to null to show hint.'),
              value: null,
            ),
            DropdownMenuItem(
              child: Text('Light'),
              value: 'Light',
            ),
          ],
        ),
      ),
    );
  }
}

If the problem persists, please comment below, thanks


Travis / CodeCov / ReadTheDocs

This PR adds:

travis (built + tests)
codecov
readthedocs

Codecov Report

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

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   91.10%
=========================================
  Files             ?       29
  Lines             ?     1732
  Branches          ?        0
=========================================
  Hits              ?     1578
  Misses            ?      154
  Partials          ?        0

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 5679dea...9c1e3db. Read the comment docs.

Codecov Report

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

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   91.10%
=========================================
  Files             ?       29
  Lines             ?     1732
  Branches          ?        0
=========================================
  Hits              ?     1578
  Misses            ?      154
  Partials          ?        0

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 5679dea...9c1e3db. Read the comment docs.


Update ResponseNullFilter

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

#31806


vendor runc to fix issue#21808

Same as #21937 but for master

@tibor that was fixed in runc master.

@jhowardmsft failed on windowss twice in a row :(

@lk4d4 Put a LGTM on https://github.com/docker/docker/pull/21962 and merge :smiling_imp:
00:21:09.360 ----------------------------------------------------------------------
00:21:09.360 FAIL: docker_cli_logs_test.go:196: DockerSuite.TestLogsSinceFutureFollow
00:21:09.360
00:21:09.361 docker_cli_logs_test.go:218:
00:21:09.361     c.Assert(out, checker.Not(checker.HasLen), 0, check.Commentf("cannot read from empty log"))
00:21:09.361 ... obtained string = ""
00:21:09.362 ... n int = 0
00:21:09.362 ... cannot read from empty log
00:21:09.362
00:21:09.551
00:21:09.551 ----------------------------------------------------------------------

LGTM

LGTM


Cannot create shortcut

I have a android phone using the openlauncher. When I create a shortcut for my webapp, it says it is created but it does not appear on my launcher. I got webapps and openlauncher from the fdroid app store.

Turns out openlauncher does not support pinshortcuts


Wrong requirement (request -> requests)

Thank you very much @sfermigier !!! Pull Requests to fix, improve or add new functionalities are welcome! :)


Aliyun: 避免cache image失败

这个 PR 实现什么功能/修复什么问题:
避免cache image失败
是否需要 backport 到之前的 release 分支:

release/2.9.0
release/2.8.0

/cc @swordqiu @yousong

Codecov Report

Merging #759 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #759      +/-   ##
=========================================
- Coverage    7.36%   7.34%   -0.02%
=========================================
  Files         348     348
  Lines       55839   55839
=========================================
- Hits         4111    4103       -8
- Misses      51348   51356       +8
  Partials      380     380

Flag
Coverage Δ

#aFlag
7.34% <ø> (-0.02%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/util/rbacutils/rabc.go
69.69% <0%> (-3.04%)
:arrow_down:

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 0b7dc5e...0076e80. Read the comment docs.

/hold

/cherrypick release/2.9.0

/cherrypick release/2.8.0

/hold cancel

/approve
/lgtm


Behaviour of beforeEach and afterEach

I came across some weird behaviour today with ava where the following test gives the following output:
import test from 'ava';

test.beforeEach(() => {
  console.log('beforeEach');
});

test.afterEach(() => {
  console.log('afterEach');
});

test('Test 1', (t) => {
  t.pass();
});

test('Test 2', (t) => {
  t.pass();
});

beforeEach
beforeEach
afterEach
afterEach
  ✔ Test 1
  ✔ Test 2

  2 tests passed

It was my assumption that afterEach would be called after a test and before the beforeEach of the next test. Is this expected behaviour? (I am using 0.7 btw)
Thanks!

Yes, beforeEach and afterEach work just like you expected, no worries ;) You are seeing such output, because AVA runs tests in parallel. It means, that Test 2 does not wait for Test 1 to complete, but both tests run at the same time.

@vdemedes ah ok that makes sense! I missed the difference between the test files being run in parallel and the tests being run concurrently. Since the tests get run concurrently how would you recommend patching global objects for tests? The original reason I came across this issue was because I was trying to patch the process.nextTick method so that I could make Promises synchronous in my tests however the beforeEach (where I patched the method) was being run before the corresponding afterEach had a chance to tear down the patch.

AVA both runs test files and tests concurrently :)

Since the tests get run concurrently how would you recommend patching global objects for tests?

I'd recommend use before() and after() methods, here's the docs for them.

so that I could make Promises synchronous in my tests

I may not understand your goal correctly, but if I advise using async/await in your tests in order to write "synchronous looking" code.
Let me know if you have any more questions ;)

@jkimbo
You have two options.
Place the process.nextTick patch in before and after. These only get run once - so they shouldn't clobber each other.
If your mock needs to maintain context, use t.context to pass information between beforeEach and the actual test (don't store it as a variable outside the functions). Note that t.context is not available in before/after, only the Each variants.
Use t.serial for your tests that require these mocks. If you have any async IO in your tests (i.e. disk/network requests), then you will end up sacrificing some speed. Otherwise, it should be easy.

@vdemedes beat me by six seconds!
With a much better point regarding using async/await to boot.

Haha thanks guys! Very helpful!
To give some more context around why I'm patching process.nextTick, I'm running some code that triggers a fetch request. I patch fetch out to be a promise I control in the test and then I assert some state, resolve the promise and assert the resultant state. The problem I had with that workflow was that the promise resolve would queue up a microtask and so I couldn't assert the next state synchronously. For the readability of the test I patch the nextTick method, store up the tasks queued up and run them at once so that I don't have to make the test async. Some sudo code that hopefully makes it clearer (React test):
// mock fetch
global.fetch = sinon.stub();
const deferred = defer(); // create a deferred promise
fetch.returns(deferred.promise);

TestUtils.Simulate.click(ReactDOM.findDOMNode(stub.refs.button));

assert(fetch.calledOnce);
assert.equal(stub.state.searching, true);

// return a response
deferred.resolve({
  'status': 200,
  'json': () => ({
    'foo': 'bar',
  }),
});

runAllTicks(); // this runs all the tasks queued

assert.equal(stub.state.searching, false);

Anyway I hope that explains why I can't use async/await etc.
Thanks again for the help!

FYI I got this idea from jest: https://facebook.github.io/jest/docs/api.html#jest-runallticks

@jkimbo - I don't know a lot about React, so I may be missing something.
Why can't you just do:
deferred.resolve({...});

await deferred.promise;

assert.equal(stub.state.searching, false);

If that returns too quickly (i.e. await returns before state has settled), you could do:
import delay from 'delay';
// ...
await deferred.promise.then(delay(10)); // or delay(0), or delay(100), etc.

@jamestalmage that works really well thanks!


Streamline dockerfiles

Changes to both dockerfiles herein. Main changes:

api container no longer builds node from source
mongo container loads data on startup (though the context needed to be expanded to allow run the reloading script

Cool.  Yeah, this looks better.  Merging...


Any way to use on Android?

Hi!
I just wanted to know if there was any way I could use this on Android? Maybe through root?
Because this is based on linux, the process shouldn't be very different, right?

Hi @ShishirModi. The only way of replacing the system emoji on Android that I'm aware of is via a Magisk module (you can find examples on GitHub). For that, your device will indeed have to be rooted.
To be honest, I can't guarantee this font will work on Android, but if you do try let me know how that went (just out of curiosity).

I've tried installing the font, but apparently stuff works differently on Samsung phones. I'll keep you updated in case I get it to work!
Thank you!

@RaeesD that's good to hear 🙂
I think it would be nice to add this to the readme for future reference. If you want, you can open a PR adding the specific steps you followed (e.g., I guess you used some adb commands?) in the troubleshooting section.

A third option, if you don't have the time, I can just link to your comment.

@dmlls I wrote up a short guide. Thanks for the opportunity to contribute 👍

@RaeesD Thank you for the PR!
@ShishirModi I'll close this issue for now, but feel free to reopen if you find a way to make it work on Samsung.


add read_timeout configuration for HTTP requests

This adds the possibility to configure the read_timeout delay (in seconds) for API requests.

Coverage increased (+0.1%) to 26.415% when pulling d318990a8d0a24e5cb17fbb29a0f99744ab01597 on fakenine:set_timeout_requests into 667e2efe0c75ead8fb3c0b20fca0fbcd8cf26a2c on Cyberwatch:master.

Coverage increased (+0.1%) to 26.415% when pulling d318990a8d0a24e5cb17fbb29a0f99744ab01597 on fakenine:set_timeout_requests into 667e2efe0c75ead8fb3c0b20fca0fbcd8cf26a2c on Cyberwatch:master.


No target power

Since Latest commit, i get an error, No target power.
If i move 5V from the updi header, to 5V on the arduino ISP header, it works, and flashes the attiny412 (need 2 tries, as it should atm)
Using your programmet for the nano, from Tindie.

Hmm seems to have fixed itself.
Closing this

Thanks for reporting this.
I ran some tests on this repo and the one previous to enabling pullups for unused pins and the AVR132DA ... the results came out the same. I also tried various chip settings, but it had no effect on the results, so I used defaults.
I updated the Arduino Nano HV UPDI Programmer page to show Operational Test Results (scroll down near the bottom).
Let me know if you're finding any differences or new issues.

Thanks for reporting this.
I ran some tests on this repo and the one previous to enabling pullups for unused pins and the AVR132DA ... the results came out the same. I also tried various chip settings, but it had no effect on the results, so I used defaults.
I updated the Arduino Nano HV UPDI Programmer page to show Operational Test Results (scroll down near the bottom).
Let me know if you're finding any differences or new issues.

Thank you.
I have been able to reproduce the error situation. After flashing this code, the attiny412 gets in a state where I need to supply the 5V from 6 PIN header instead of UPDI header (T5).```
#include <avr/io.h>
#include <util/delay.h>

int main(void) {
  PORTA.DIR = PIN3_bm;

  while(1) {
    PORTA.OUTTGL = PIN3_bm;
    _delay_ms(500);
  }
}

Flashing this code, brings the expected functionality (T5) back again
#include <avr/io.h>
#include <util/delay.h>

#define LED_PIN PIN3_bm
#define LED_PORT PORTA

void setup() {
  LED_PORT.DIRSET = LED_PIN;
}

void loop() {
  if ((FUSE.SYSCFG0 & FUSE_RSTPINCFG_gm) == RSTPINCFG_UPDI_gc) {                     // UPDI mode
    ((millis() >> 7) & 15) ? LED_PORT.OUTCLR = LED_PIN : LED_PORT.OUTSET = LED_PIN;  // LED blinks at 0.5Hz
  }
  else if ((FUSE.SYSCFG0 & FUSE_RSTPINCFG_gm) == RSTPINCFG_RST_gc) {                 // Reset mode
    ((millis() >> 5) & 15) ? LED_PORT.OUTCLR = LED_PIN : LED_PORT.OUTSET = LED_PIN;  // LED blinks at 2Hz
  }
  else if ((FUSE.SYSCFG0 & FUSE_RSTPINCFG_gm) == RSTPINCFG_GPIO_gc) {                // GPIO mode
    ((millis() >> 3) & 15) ? LED_PORT.OUTCLR = LED_PIN : LED_PORT.OUTSET = LED_PIN;  // LED blinks at 8Hz
  }
  else {                                                                             // Invalid setting
    LED_PORT.OUTSET = LED_PIN;                                                       // LED remains ON
  }
}

I tested your fist code and couldn't duplicate the issue until I forgot to switch the programmer from HV Mode to UPDI Mode. Although leaving the programmer's jumper in HV mode or PCHV mode doesn't hurt the target's PA0 when its in UPDI mode, this can start the "toggle" pass/fail issue. Also, note that the T5V pin on the programmer comes from PORTC and is only good for about 60mA and its voltage is about 4.7-4.8V.
For simple UPDI programming, I recommend having the jumper in UPDI mode and using the 5V pin on the ISP header.
This jumper is difficult to access ... I think the long flexible tab versions would be easier to use.
I'll update the product page to show recommended connections and jumper settings.
From your images, it looks like you're using an older version of megaTinyCore.
I'm using 2.1.5 installed via Boards Manager which does show a few compiler warnings (these can be ignored) ... the GitHub version has this fixed.

Update megaTinyCore, still I can reproduce. I guess its the chip being used. I really dont know, the top has the text removed.
Examining the 12V pulse, seems OK, actually goes to ~16V very short.
Then again, I noticed you mentioned the use of the 6PIN header, so its ok really.

Thanks.

The voltage should be clipped to 12V by the zener D4 ... looks like you may have a bad solder connection or faulty diode.
I'll be happy to send a replacement asap (even if you manage to repair it).

Excellent, this looks more correct


Make sure Jackson annotations are taking into account in schema generation

e.g. if a field is renamed via Jackson annotations this should be picked up in schema extraction

So with this class, the schema should have a FirstName field, not a 'first` field:
public class Person {
    @JsonProperty("FirstName")
    private String first;
}

Yes, that is correct


Nickname Issues

https://projectpokemon.org/home/forums/topic/63302-pkmds-for-web/page/2/#comment-290560

I've been trying to change nicknames in the web version, and whenever I do, the game fills in the unused spaces with question marks. Is there a specific format I am missing when inputting these names?

Also noticing in Gen II, setting the nickname doesn't work.

Probably should do this and #56 at the same time.

Looks like the issue with Gen II string inputs (nickname and OT name at least) is specifically tied to a non-English version.


remove snapshot tests of ast

They are v brittle when I change the ast

In https://github.com/aDotInTheVoid/skate/commit/d838f29a50691cac68f9dfc264db66b63338f3a4, I didnt look at them at all, so I dont think they have value, as I'm blindly blessing. It might be worth adding a test to make sure they parse.


Update results.erb

Removed VAT deferral scheme. Updated details for the furlough scheme.
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠

content looks good


feat(rpc): add documentation for calculate_reward_percentiles

Basic doc comments is added for calculate_reward_percentiles.

keep 'em coming


Add Python 3.12 support

Support for Python 3.12 needs conan to be upgraded, which is not yet possible due to GitLab API...

Conan 2 & GitLab issue: https://gitlab.com/gitlab-org/gitlab/-/issues/333033


Multithread problem in JobSystem?

JobSystem::execute runs in threads, and calls JobSystem::pop(), and calls WorkStealingDequeue::pop
Job* pop(WorkQueue& workQueue) noexcept {
size_t index = workQueue.pop();
assert(index <= MAX_JOB_COUNT);
return !index ? nullptr : &mJobStorageBase[index - 1];
}
But WorkStealingDequeue::pop has a comment that "Must be called from the main thread"
Is this a problem?
PS: JobSystem::run -> JobSystem::put -> WorkStealingDequeue::push  has the same situation

It is not a problem in practice because each thread has its own work queue as part of the ThreadState.


Change to fix issue 977

This change is equivalent to the patch suggested on issue #977.

I just saw this change was already done on pull request #842 and it was merged on nginx_refactoring branch.

@zimmerle, I reopen the pull request for your consideration.


JLearnIt version 6.4

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/109801&drop=dogfoodAlpha

Closed, I prefer to recreate a new release as I also found a bug today.


Wrapper development link (links section) should point to Github not Gitlab

The wrapper development link (in the links section) is wrong, it should point to Github, not Gitlab

/cib


5 days delay in indexing

We got a report today about a package missing from the search results (https://github.com/jsdelivr/jsdelivr/issues/18387), and I see the first release was five days ago. There was also a similar issue recently (#928). Is there anything we could do to reduce the delay?

yep indeed.
It seems NPM bumped their sequence again and we are 60K updates behind.
Not so much we can do since we need to process update sequentially.
The same issue all over again :'(
We had lost metrics monitoring since a few weeks (and had no time to look up), I have added a more annoying alerting so that I don't skip those.

For JsDelivr we should really sync and find not an alternative but a decent way to fallback. Could be with a secondary real time index that has no additional info combined with an internal API to refresh cache.

Do you know what's slowing down the indexing most? I'm wondering how the fallback would work because if you visit a page like https://www.jsdelivr.com/package/npm/quantdom it's easy, but to get there, you'd usually search at https://www.jsdelivr.com/?query=quantdom and if that returns some results, there's no way for us to know we need to fallback.
At the same time, we use jsDelivrHits for sorting the search results and also plan to add an option for filtering based on moduleTypes/styleTypes fields, so using a lighter index always is not an option. Would it make sense to maybe insert packages to the main index immediately, with some kind of "in progress" flag, and update it later?

The index is back up to date now, thanks for standing by!


Run Commands as Another User

I'd like to be able to run fabric commands inside my vagrant VM as someone besides the vagrant user.
The plugin executes vagrant ssh-config and scrapes the output and pulls the value of 'User', which is then what it uses. This should be configurable.

@dgarstang You can do this off of my branch in https://github.com/wutali/vagrant-fabric/pull/11 with the -u flag.


Update gatsby

Fixes https://github.com/reactjs/reactjs.org/issues/589

I just tested the preview with Safari on an iPhone X running iOS 11.2.5 and the navigation bug is fixed. Well done!

Sweet, thank you so much for verifying!


Clearing Draper::ViewContext in RSpec

I had an issue with some specs not clearing the ViewContext and hence having undefined h.current_user. It doesn't make sense to clear for every type of spec?
https://github.com/drapergem/draper/blob/ed62f5555148b997eeb9feb365dd3dffcdb3d903/lib/draper/test/rspec_integration.rb#L12

The issue you are experiencing is actually due to the view context not being cleared after certain spec types. See the documentation in this pull request, as well as the referenced issue for more detail. Are you using decorators inside of jobs? That would be a scenario where the view context was built without current_user accessible and subsequent tests would fail.

Not using decorators inside jobs but it was a similar scenario. Thanks for the PR! I'll be useful for others.

Perfect. Glad we could help.


Injection doesn't seem to work (crashes or just does nothing)

I'm using the RPCS3 IPC build from that "Artifacts" section of the cirrus page along with the latest KAMI release version, I've tried Killzone 1 and 2 from Killzone Trilogy, the KAMI tool starts up, seems to detect RPCS3 and the game and then closes on both games. Trying it with Red Dead Redemption, the KAMI tool starts and stays working, but once enabled the mouse aiming doesn't seem to work, the mouse gets locked and I can use the mouse buttons but I can't move the camera at all with the mouse.

RDR hook author here. You may want to restart the game once or twice and then it should work. It's not super stable right now, I need to get my marbles together sometime and trace it out fully to ensure it works on every launch.

In my case, it doesn't even connect to RPCS3, I have the latest update and it just stays in "Unconnected".

Similar issues with Resistance 2 - Detects correctly but on game launch is bombs out without an error.

Update, trying again with the newest RPCS3 version (build 0.0.24-14129) and the latest KAMI release, same thing happens in both Killzone 1 and 2. The KAMI window appears, shows all the info as if it's detected and connected and then crashes a few seconds later.

I have this issue as well. I've only been able to test Resistance 1 and 2, as soon as either game opens/gets past the "Building SPU cache" screen, KAMI closes itself.
I've also tried a new clean install of the latest RPCS3 build (and I have .NET 6 Runtime installed), the only setting I changed was enabling IPC, and it still crashes.

I have a suspicion, might look into it this weekend

I take it this issue still isnt fixed? i used KAMI last year to play killzone 1 trying to play 2 now and its just constant crashing without fail

can this be fixed now?

It could have been fixed even before, it just hasn't been. Neither Juhn nor me (particularly not me) have been touching KAMI much since the first releases, although he did push up some changes to PCSX2 and RPCS3 to maintain the existing compatibility and features.
The project is not being actively / consistently worked on, that's why there is little to no progress. Both our attentions have been elsewhere for the past year. Please do not wait for this, there's no ETA, nor steady work being done.


Upgrade to support PHP 8.X and Latest twig version

Description
Te composer dependencies where refactored and php >= 8.0 was added as dependency.
Updated the twig/twig to the lasted ^3.x version.
Refactored all the code to work with PHP 8.X type hint sintax.
Motivation and context
Why is this change required? What problem does it solve?
The supporting version of PHP, should be updated as most of the new projects are using PHP 8.X. Meanwhile some dependent projects of this package that are using latest versions of PHP can't run properly as the twig/twig version is outdated.
How has this been tested?
The source code for tests was not changed (only fixed for new version of twig) and all the specification stays the same. I have run the phpspec and behat to guarantee package interfaces.
Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:
Go over all the following points, and put an x in all the boxes that apply.
Please, please, please, don't send your pull request until all of the boxes are ticked. Once your pull request is created, it will trigger a build on our continuous integration server to make sure your tests and code style pass.

[x] I have read the CONTRIBUTING document.
[x] My pull request addresses exactly one patch/feature.
[x] I have created a branch for this patch/feature.
[x] Each individual commit in the pull request is meaningful.
[x] I have added tests to cover my changes.
[x] If my change requires a change to the documentation, I have updated it accordingly.

If you're unsure about any of these, don't hesitate to ask. We're here to help!

The pipeline does not run tests properly

Should not be pushed to master. Should be in develop branch.


Unable to start gsp-local when WFH

WHAT
I'm unable to start a new GSP-cluster from home (using the VPN).
As you are aware, I got a running local version most days from the office.
When running the startup script over the VPN I get this:
Unable to pull images, which may be OK: running cmd: sudo kubeadm config images pull --config /var/lib/kubeadm.yaml: command failed: sudo kubeadm config images pull --config /var/lib/kubeadm.yaml
stdout:
stderr: failed to pull image "k8s.gcr.io/kube-apiserver:v1.12.0": output: Error response from daemon: Get https://k8s.gcr.io/v2/: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)

After which the script continues, until it falls over when it tries to apply templates.
If I'm not on the VPN, I don't get the error, but the same outcome (stuck on apply).
I've left the apply running for hours, and it's just not getting past the rule, destinationrule etc.
WHY
The being unable to pull the images issue may be related to hyperkit, but I'm not sure

I have also experienced issues with minikube networking on OS X when using the Cisco AnyConnect Client including but probably not limtied to:

VirtualBox based minikube instances requiring a full host reboot before I can get networking back
Network between guest and internet failing for Hyperkit based minikube instances requiring rebuild of minikube instance
Network between host and guest so slow that it is unusable.

All of those issue seem to persist even after no longer using the VPN client... I assume the AnyConnect client does some horrible hackery to hijack the network stack and is not cleaning up correctly.
If have not seen these networking issues with Docker based deployments, but unfortunately the Docker (ie Kind) based local setup has other issues right now.

We also saw this after having used the VPN without first rebooting.

@Krenair @chrisfarms
I've tried following this:

Restart machine
Get on VPN
Delete cluster (if it exists)
Create cluster

The startup still can't download the images (with the same error), and I'm currently on #25 at
Failed to apply /var/folders/jr/lv1mpg4s6cgby621kx99vjfm0000gr/T/tmp.iH691KUJ/gsp-istio/. Retrying in 10s...
I can't see any changes in the apply at all, all applied components stay unchanged and most istio components fail (rule, destinationrule etc).

See #832


insert into table using an array of array variable

alasql('CREATE TABLE country1(country string primary key, capital string)');
alasql('CREATE TABLE country2(country string primary key, capital string)');
alasql('CREATE TABLE country3(country string primary key, capital string)');
// i can insert values into country1 table using the following
alasql('INSERT INTO country1 VALUES \
("Malaysia1","Kuala Lumpur"), \
("China1","Beijing"), \
("United Kingdom1","London"), \
("Philippines1","Manila") \
')
// i can insert into country2 table using array of object
var data2 = [
{country:'Malaysia',capital:'Kuala Lumpur'},
{country:'China',capital:'Beijing'},
{country:'United Kingdom',capital:'London'},
{country:'Philippines',capital:'Manila'}
]
alasql.tables.country2.data = data2
// but i want to insert an array of array into country3 table
// so i try this but failed
var data3 = [
['Malaysia','Kuala Lumpur'],
['China','Beijing'],
['United Kingdom','London'],
['Philippines','Manila']
]
alasql('INSERT INTO country3 VALUES data3')   //failed
Does alasql provide any way to insert into table from an array of array?`

This is not supported.


Move to new, common version-tag for McStas and McXtrace

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

All god, merging


Empty source when showing metadata of UDR (external Stored Procedure)

Description
Metadata of UDRs (user defined routines) is not extracted, in this case metadata of external Stored Procedures.
DBeaver Version
Community Edition 24.0.5
Operating System
Win10, Win11
Database and driver

Firebird 4, Firebird 5
org.firebirdsql.jdbc:jaybird:RELEASE 5.0.4.java11

Steps to reproduce
Not as easy as with external stored functions, since there are no external stored procedures shipped with a Firebird installation. This system table query gives an idea of what to look for:
select trim(rdb$procedure_name) name, trim(rdb$engine_name) engine, trim(rdb$entrypoint) entrypoint
from rdb$procedures
where rdb$engine_name = 'UDR'

The outcome of metadata extraction should be something like this:
CREATE PROCEDURE WRITE_TXT (
    TEXT blob sub_type 1 NOT NULL,
    FILE_NAME varchar(500) NOT NULL,
    "MODE" varchar(10) NOT NULL )
RETURNS (
    STATUS varchar(100) )
EXTERNAL NAME 'pascaludr!write_txt'
ENGINE UDR

similar to the extraction of external stored functions.
Additional context
No response

Thank you for bug report.


[typography] add fontStyle, lineSpacing, lineHeight

Added typography properties as mentioned in #15879

@oliviertassinari what do you think about this commit?


[v1.1.2] docker-containerd task run hangs

Description
Below command hangs
# docker-containerd-ctr --namespace moby --address /var/run/docker/containerd/docker-containerd.sock task ls
in kubernetes node, where there is a orphan container. not sure whether the container will cause that

Steps to reproduce the issue:

there is a pod container left and its /pause container already exit(0) in kubernetes worker node
run the command below
docker-containerd-ctr --namespace moby --address /var/run/docker/containerd/docker-containerd.sock task ls

Describe the results you received:
above command no output
Describe the results you expected:
output container，container id， container status in stdout
Output of containerd --version:
containerd github.com/containerd/containerd v1.1.2 468a545b9edcd5932818eb9de8e72413e616e86e

Any other relevant information:
docker version
Client:
 Version:           18.06.2-ce
 API version:       1.38
 Go version:        go1.10.3
 Git commit:        6d37f41
 Built:             Sun Feb 10 03:47:42 2019
 OS/Arch:           linux/amd64
 Experimental:      false

Server:
 Engine:
  Version:          18.06.2-ce
  API version:      1.38 (minimum version 1.12)
  Go version:       go1.10.3
  Git commit:       6d37f41
  Built:            Sun Feb 10 03:46:06 2019
  OS/Arch:          linux/amd64
  Experimental:     false

containerd 1.1 and Docker 18.06 reached EOL.
Please try containerd 1.3 or 1.4 with Docker 19.03 or 20.10-beta.

https://github.com/containerd/containerd/issues/4654#issue-729326822
front part of containerd stack trace dump like
Oct 26 14:26:51 <node name> dockerd[31988]: time="2020-10-26T14:26:51+08:00" level=info msg="=== BEGIN goroutine stack dump ===
Oct 26 14:26:51 <node name> dockerd[31988]: goroutine 27 [running]:
Oct 26 14:26:51 <node name> dockerd[31988]: github.com/containerd/containerd/cmd/containerd/command.dumpStacks()
Oct 26 14:26:51 <node name> dockerd[31988]: #011/go/src/github.com/containerd/containerd/cmd/containerd/command/main_unix.go:78 +0x8c
Oct 26 14:26:51 <node name> dockerd[31988]: github.com/containerd/containerd/cmd/containerd/command.handleSignals.func1(0xc420530240, 0xc4205301e0, 0x55ffe1c25b80, 0xc420140010, 0xc42
02fe180)
Oct 26 14:26:51 <node name> dockerd[31988]: #011/go/src/github.com/containerd/containerd/cmd/containerd/command/main_unix.go:53 +0x274
Oct 26 14:26:51 <node name> dockerd[31988]: created by github.com/containerd/containerd/cmd/containerd/command.handleSignals
Oct 26 14:26:51 <node name> dockerd[31988]: #011/go/src/github.com/containerd/containerd/cmd/containerd/command/main_unix.go:43 +0x8b
Oct 26 14:26:51 <node name> dockerd[31988]: goroutine 1 [chan receive, 440862 minutes]:
Oct 26 14:26:51 <node name> dockerd[31988]: github.com/containerd/containerd/cmd/containerd/command.App.func1(0xc42048e000, 0xc42048e000, 0xc420517d07)
Oct 26 14:26:51 <node name> dockerd[31988]: #011/go/src/github.com/containerd/containerd/cmd/containerd/command/main.go:165 +0xa1b
Oct 26 14:26:51 <node name> dockerd[31988]: github.com/containerd/containerd/vendor/github.com/urfave/cli.HandleAction(0x55ffe1a127a0, 0x55ffe1c02758, 0xc42048e000, 0xc420530180, 0x0)
Oct 26 14:26:51 <node name> dockerd[31988]: #011/go/src/github.com/containerd/containerd/vendor/github.com/urfave/cli/app.go:502 +0xca
Oct 26 14:26:51 <node name> dockerd[31988]: github.com/containerd/containerd/vendor/github.com/urfave/cli.(*App).Run(0xc42004a1c0, 0xc420138060, 0x3, 0x3, 0x0, 0x0)
Oct 26 14:26:51 <node name> dockerd[31988]: #011/go/src/github.com/containerd/containerd/vendor/github.com/urfave/cli/app.go:268 +0x60e
Oct 26 14:26:51 <node name> dockerd[31988]: main.main()
Oct 26 14:26:51 <node name> dockerd[31988]: #011github.com/containerd/containerd/cmd/containerd/main.go:28 +0x51
Oct 26 14:26:51 <node name> dockerd[31988]: goroutine 20 [chan receive]:
Oct 26 14:26:51 <node name> dockerd[31988]: github.com/containerd/containerd/vendor/github.com/golang/glog.(*loggingT).flushDaemon(0x55ffe231baa0)
Oct 26 14:26:51 <node name> dockerd[31988]: #011/go/src/github.com/containerd/containerd/vendor/github.com/golang/glog/glog.go:879 +0x8d
Oct 26 14:26:51 <node name> dockerd[31988]: created by github.com/containerd/containerd/vendor/github.com/golang/glog.init.0
Oct 26 14:26:51 <node name> dockerd[31988]: #011/go/src/github.com/containerd/containerd/vendor/github.com/golang/glog/glog.go:410 +0x205
Oct 26 14:26:51 <node name> dockerd[31988]: goroutine 21 [syscall]:
Oct 26 14:26:51 <node name> dockerd[31988]: os/signal.signal_recv(0x55ffe1c15540)
Oct 26 14:26:51 <node name> dockerd[31988]: #011/usr/local/go/src/runtime/sigqueue.go:139 +0xa8
Oct 26 14:26:51 <node name> dockerd[31988]: os/signal.loop()
Oct 26 14:26:51 <node name> dockerd[31988]: #011/usr/local/go/src/os/signal/signal_unix.go:22 +0x24
Oct 26 14:26:51 <node name> dockerd[31988]: created by os/signal.init.0
Oct 26 14:26:51 <node name> dockerd[31988]: #011/usr/local/go/src/os/signal/signal_unix.go:28 +0x43
...

containerd 1.1 and Docker 18.06 reached EOL.
Please try containerd 1.3 or 1.4 with Docker 19.03 or 20.10-beta.

thanks for your answer, not sure that is a bug and as for production, upgradation is not preferred

@Davidrjx Sorry can't help you with your production scenarios as a open source project. If you can recreate the problem with supported versions, we can try to help (backports to whatever you support would be on your plate).

@dims thanks, very great but which versions are what you specified

I'm closing this as v1.1 is outdated, but feel free to reopen if this is reproducible with v1.4.

I'm closing this as v1.1 is outdated, but feel free to reopen if this is reproducible with v1.4.


[BUG] Could not retrieve stock videos: generate_subtitles() missing 2 required positional arguments: 'sentences' and 'audio_clips'

Describe the bug
Every time I try to generate a video it shows this error:
Could not retrieve stock videos: generate_subtitles() missing 2 required positional arguments: 'sentences' and 'audio_clips'
I'm using my local machine with ubuntu.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Linux

Additional context
I filled all the .env with all the providers but didn't work.

also happen to me;
[-] Error: generate_subtitles() missing 2 required positional arguments: 'sentences' and 'audio_clips'

OS: Debian Sid
Python version: Python 3.11.7

same error for me

Any solution guys?

same here.
I added:
subtitles_path = generate_subtitles(tts_path, sentences, paths)
in line 159 of main.py
Worked for me, but I still have the issue, that the audio is not added to the final video

Hi, this was an issue from someone who made a PR, I apologize for not noticing this earlier. Please pull from git again and the issue should be fine.

Hi, this was an issue from someone who made a PR, I apologize for not noticing this earlier. Please pull from git again and the issue should be fine.

Still doesn't work.

testing recently and still happen. The process is stoped and the final video not created.

That means you didn't run git pull, as you can see the bug was fixed: https://github.com/FujiwaraChoki/MoneyPrinter/blob/f112a7165a91a660b98333b3ed19baa951e635ef/Backend/main.py#L162

Thanks. It's now fixed.

testing recently and still happen. The process is stoped and the final video not created.

That means you didn't run git pull, as you can see the bug was fixed:
https://github.com/FujiwaraChoki/MoneyPrinter/blob/f112a7165a91a660b98333b3ed19baa951e635ef/Backend/main.py#L162

That line isn't the issue.
MoviePy - Writing audio in ../temp/b6354402-ac06-4c93-9a3d-224e2d472b84.mp3
MoviePy - Done.
[-] Error: generate_subtitles() missing 2 required positional arguments: 'sentences' and 'audio_clips'
<IP_ADDRESS> - - [07/Feb/2024 18:12:03] "POST /api/generate HTTP/1.1" 200 -

Maybe this is the cause?
Line 159: subtitles_path = generate_subtitles(tts_path)
Yes, I did git pull several times, now. Still latest version.

I can't find the line you're talking about - I suspect there's some problems with git, do you perhaps mean this line of code?
https://github.com/FujiwaraChoki/MoneyPrinter/blob/f112a7165a91a660b98333b3ed19baa951e635ef/Backend/main.py#L155

I would suggest removing MoneyPrinter and re-cloning it.


helper

Pull Request Test Coverage Report for Build 31

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.857%

Totals

Change from base Build 29:
0.0%

Covered Lines:
145

Relevant Lines:
175

💛  - Coveralls


Google Cast - castStyle.castViews missing

Reopen issue #122

@SotoiGhost , Is there any update on this?
i tried to uncomment and create a custom build but it cause some linker error
Undefined symbols for architecture arm64:
"OBJC_CLASS$_GCKUIStyleAttributesCastViews", referenced from:
objc-class-ref in registrar.o
"OBJC_CLASS$_GCKUIStyleAttributesConnectionController", referenced from:
objc-class-ref in registrar.o
"OBJC_CLASS$_GCKUIStyleAttributesDeviceChooser", referenced from:
objc-class-ref in registrar.o
"OBJC_CLASS$_GCKUIStyleAttributesDeviceControl", referenced from:
objc-class-ref in registrar.o
"OBJC_CLASS$_GCKUIStyleAttributesExpandedController", referenced from:
objc-class-ref in registrar.o
"OBJC_CLASS$_GCKUIStyleAttributesGuestModePairingDialog", referenced from:
objc-class-ref in registrar.o
"OBJC_CLASS$_GCKUIStyleAttributesInstructions", referenced from:
objc-class-ref in registrar.o
"OBJC_CLASS$_GCKUIStyleAttributesMediaControl", referenced from:
objc-class-ref in registrar.o
"OBJC_CLASS$_GCKUIStyleAttributesMiniController", referenced from:
objc-class-ref in registrar.o
"OBJC_CLASS$_GCKUIStyleAttributesTrackSelector", referenced from:
objc-class-ref in registrar.o
ld: symbol(s) not found for architecture arm64

@dationguyen Google changed those symbols visibility from external to private external making impossible to us to bind these classes, this is why we commented them. I filed a bug to Google and they already changed the visibility again to external. The component needs an update to 4.3.5

Cool, Thanks @SotoiGhost

@dationguyen Google Cast v4.4.2 is live now on NuGet. Hope this helps you. Best!


Community bootstrapping

As this has moved forward significantly, we should open this up to the Cloud Native contributor community at large :)

[ ] Schedule a monthly(?) community meeting / office hours (first meeting would be an intro to clomonitor)
[ ] Create #clomonitor Slack channel in CNCF Slack
[ ] Send out comms to maintainers on the purpose of clomonitor, and signal boost the community meeting / slack channel

@tegioz @cynthia-sg I'm happy to tackle all of this :) I just want to make sure you're both aware of it AND make sure when scheduling the meeting it fits in your calendars. Feel free to thread with questions!
/assign

Thanks for taking care of this @jeefy! 👍
Between 8:00 and 11:00am PT usually works for us any day, except the 2nd Tuesday of the month from 10:30 to 11:00am that we have the Artifact Hub meeting.

Slack channel's been created (and y'all should have been invited)
Virtual meetup's been scheduled ( https://community.cncf.io/clomonitor-community/ ) -- Once you're both registered for that website, I can make you admins of the whole Community and you can update/modify events as needed. :)
Tentatively I went Second Monday of the month at 8a PT (starting March 14th). But since it's (right now) just us, that can easily change.
Once the time is ACK'd and settled on, I can start working on comms to ping various projects about attending!

Awesome, time works fine for @cynthia-sg and me 👍


[P0] Prisma client is not able to push data to MongoDB

Bug description
Pushing data to Mongo DB not working for Prisma 5.14.0
Pushing and pulling data from MongoDB not working for Prisma 5.18.0
Error

How to reproduce

Try the same schema and client with MongoDB with prisma version: 5.13.0
Update the prisma to 5.14.0
Try to pull and push data from DB using the below shared code
It works for pull: findUnique but not for create
upgrade prisma to 5.18.0 - latest
it does not work for either operation
code:

async function createTagAction(data: TagIterface) {
  const validatedData = tagSchema.safeParse(data);

  if (!validatedData.success) {
    return {
      error: validatedData.error.format(),
    };
  }

  const { slug, description, imageUrl } = validatedData.data;
  try {
    const existingTag = await prisma.tag.findUnique({
      where: { slug },
    });

    if (existingTag) {
      console.log("Tag already exists");
      return {
        error: { slug: "Slug already exists" },
      };
    }

    console.log("Data being passed to Prisma:", validatedData.data);

    console.time("DB Operation");
    const newTag = await prisma.tag.create({
      data: {
        slug,
        description,
        imageUrl,
      },
    });
    console.timeEnd("DB Operation");

    return { success: true, tag: newTag };
  } catch (error) {
    console.error("Failed to create tag:", error);
    return { error: "Failed to create tag." };
  }
}

Expected behavior
Should be able to push and pull data from db with all prisma versions.
Prisma information

// Add your schema.prisma
generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider = "mongodb"
  url      = env("DATABASE_URL")
}

model ContactForm {
  id        String   @id @default(cuid()) @map("_id")
  name      String
  email     String
  message   String
  createdAt DateTime @default(now())
  updatedAt DateTime @updatedAt
}
model Tag {
  id    String @id @default(cuid()) @map("_id")
  slug  String @unique
  description String?
  imageUrl String?
  posts TagOnPost[]
}

// Add your code using Prisma Client
import { PrismaClient } from "@prisma/client";

const prismaClientSingleton = () => {
  return new PrismaClient();
};

type PrismaClientSingleton = ReturnType<typeof prismaClientSingleton>;

// eslint-disable-next-line
const globalForPrisma = globalThis as unknown as {
  prisma: PrismaClientSingleton | undefined;
};

const prisma = globalForPrisma.prisma ?? prismaClientSingleton();

export default prisma;

if (process.env.NODE_ENV !== "production") globalForPrisma.prisma = prisma;

Environment & setup

OS:  macOS
Database: MongoDB
Node.js version: v21.7.3

Prisma Version
prisma                  : 5.18.0
@prisma/client          : 5.18.0
prisma                  : 5.14.0
@prisma/client          : 5.14.0
prisma                  : 5.13.0
@prisma/client          : 5.13.0

Hey @deepshaswat, would you be able to provide us with a reproduction repository? That would be extremely helpful. Thank you

@Weakky
Repo: https://github.com/deepshaswat/deepshaswat.com
/apps/admin You just need to setup mongodb and upgrade the prisma in /packages/db and reproduce

@Weakky Were you able to reproduce the bug?

@jasonrudolph @endor @elithrar Can someone please check on this

Hello @deepshaswat,
Thank you for providing us with a reproduction. We will get back to you once we get to this in the upcoming days.
Please bare with us for now while we coordinate internally.


Improve support for recently released defineProps in Vue.

import type { NumberInputProps } from "@ark-ui/vue";

interface Props {
  label: string;
  id?: string;
  paddingRight?: number;
  paddingLeft?: number;
}

const props = defineProps<NumberInputProps & Props>();

Vue now supports passing types to the defineProps method since Vue 3.3.
However, with Ark types this shows different errors for some reason.
https://stackblitz.com/edit/nuxt-starter-3jzcrb?file=components%2FChildDialog.vue
Originally posted by @jiblett1000 in https://github.com/chakra-ui/ark/issues/918#issuecomment-1550347164
NB: this might be coming from the polymorphic factory types and might be solved by migrating to the asChild prop.

@Shyrro Thanks for this :smile:

Unfortuantely it turned out that this is not possbile due to intenstive use of generics in zag.js
Instead I wrote a script that turns a typescript interface into vue props. https://github.com/chakra-ui/ark/blob/main/scripts/src/generate-vue-props.ts
We going to use this auto generated props / emits instead.


Add get block async

Codecov Report

Merging #38 into dev will increase coverage by 0.16%.
The diff coverage is 67.44%.

@@             Coverage Diff              @@
##                dev      #38      +/-   ##
============================================
+ Coverage     56.18%   56.34%   +0.16%
- Complexity      167      168       +1
============================================
  Files            27       27
  Lines          1723     1750      +27
  Branches        114      113       -1
============================================
+ Hits            968      986      +18
- Misses          677      686       +9
  Partials         78       78

Impacted Files
Coverage Δ
Complexity Δ

...s/stub/fabric/performance/PureFabricCallSuite.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

...ic/performance/PureFabricSendTransactionSuite.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

...a/com/webank/wecross/stub/fabric/FabricDriver.java
57.93% <67.44%> (+0.96%)
20.00 <1.00> (+1.00)

Continue to review full report at Codecov.

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


The logging message appears only when the GroupAwareQueryManager is initialized.
Closes #1074.

@taegeonum @DifferentSC I've addressed your comments. Thanks!

LGTM. I'm merging it. Thanks!


General reflexion

[ ] Do a critical but constructive general reflexion over the whole process on the check and balances principles... Check in relation to goals in whitepaper whether they are met or not and potential issues.. Compare intent vs actual implemented prototype...

It might be a good idea to separate this task for the final report...

@Sofia-Pika has already done a pretty great job on documenting interface issues and a good start on the checks and balances.
@Xoeseko I was thinking of further developping the checks and balances to make sure we didn't miss anything
@Bayar4B It might be interesting for you to document the potential financial issues. I can give you more details but for example what we talked about that the only form of remuneration is speculation and that that is problematic if one wants to talk about a sustainable remuneration model. If you don't undersand something feel free to ask!


GUI Button cannot be pressed if keyboard is pressed, or for a second after

Tested versions

Reproducible in 4.1.1-stable, 4.3-stable and 4.4-dev4

System information
Godot v4.3.stable - Windows 10.0.19045 - Vulkan (Forward+) - integrated Intel(R) UHD Graphics (Intel Corporation; <IP_ADDRESS>4) - Intel(R) Core(TM) i7-10510U CPU @ 1.80GHz (8 Threads)
Issue description
After pressing any keyboard button, even ones that are not bound to actions, GUI Buttons cannot be pressed and remain inactive for about another second.
This may be intended behavior, but I found nothing in the documentation to indicate it.
Modifier keys such as shift work fine.
Mouse enter events work as well.
Tried to set it to the right mouse button but got the same result.
Steps to reproduce

Create a new project
Add a Button.
Connect the pressed signal to a function that prints a message
Start the project
Press a button, such as a number (Modifier keys such as Shift do not produce this behavior)
Try pressing the button
The function will not be called while the button is pressed, or for some time after.

Minimal reproduction project (MRP)
GUIButtonTest.zip

Had no issue trying to reproduce this in v4.4.dev.custom_build [277cb68e1]. Try making a MRP and check if its a problem with your project.godot.

Just found out it's not Godot-specific, and my laptop is just going crazy, doing the same in other programs as well. At least it's not me who is going crazy...
Sorry for the false alarm.


[5.9] Remove unnecessary top padding in navigation bar

Explanation: Removes an unnecessary top padding from the nav bar that we are removing everywhere anyway.
Scope: All pages
Issue: rdar://107692636
Risk: Small
Testing: There should be no visual change in doc and tutorial pages
Reviewer: @marinaaisa @SamLanier
Original PR: https://github.com/apple/swift-docc-render/pull/583

@swift-ci test


Handling situation of having two identically named packages

Given there are now two "torch2jax" packages, obviously, this makes it problematic if I want to install both and use them for slightly different purposes in the same virtual environment.
I am wondering what the way forward here is?

I apologize for not having a solution yet, a stopgap solution is now mentioned in the README.md under Related Work. I provide a helper script to install this package under an aliased named, see:

install_package_aliased.py

The aliased PyPI installation is now available!
$ pip install wrap-torch2jax
$ # then
$ python3
$ >>> from wrap_torch2jax import torch2jax, torch2jax_with_vjp

@adam-hartshorne


No output from iGPU

I have a bit of an unusual setup where two monitors are connected to my Nvidia card (GTX 960 using the nouveau driver) and a third monitor is connected to the iGPU via display port.
When I start sway the two monitors connected to the Nvidia card work as expected but the third monitor "hangs", appearing to do nothing. When I exit sway the monitor springs back to life displaying my shells login session on tty1.
I'm running sway in debug mode now and will post logs after I've looked through them. For now I'm wondering if anyone has tried to use Sway in a similar setup and had the same issues?
The third monitor does not show up in xrandr or via smaymsg:
❯ xrandr
Screen 0: minimum 320 x 200, current 2560 x 1080, maximum 8192 x 8192
XWAYLAND0 connected 1920x1080+0+0 (normal left inverted right x axis y axis) 500mm x 280mm
   1920x1080     59.96*+
XWAYLAND1 connected 2560x1080+0+0 (normal left inverted right x axis y axis) 670mm x 280mm
   2560x1080     59.98*+

❯ swaymsg -t get_outputs
Output DVI-I-1
  Geometry: 1920x1080 @ 0,0
  Scale factor: 1x
  Workspace: 1:一

Output HDMI-A-1
  Geometry: 2560x1080 @ 1920,0
  Scale factor: 1x
  Workspace: 2:二

Multi-GPU support is planned for wlroots, if possible, but will not be attempted earlier than that.


add chromedriverPorts cap which allows specifying multiple ports or a range

this helps support the multiple webview case, and would provide a supported 'workaround' for e.g. https://github.com/appium/appium/issues/11820

oh, also deprecate chromeDriverPort in favor of the more standard chromedriverPort, for parity with all our other chromedriver caps

while i was in there i also removed a discrepancy where the bootstrapPort opt was only a CLI opt and not a desired cap. now it can be either like all the rest.

ok, feedback addressed. this PR now also depends on https://github.com/appium/appium-base-driver/pull/314

a note need to update after this:

https://github.com/appium/appium/blob/master/docs/en/writing-running-appium/caps.md

published in 4.14.0


Clean install requires -allow-unauthenticated

Describe the bug
Following the instructions in https://docs.microsoft.com/en-us/cli/azure/install-azure-cli-apt?view=azure-cli-latest on a fresh travis build agent the installation fails if -allow-unauthenticated is not specified.
To Reproduce
git clone --depth=50 --branch=master https://github.com/jsacapdev/k8s-routing-slip.git jsacapdev/k8s-routing-slip
Cloning into 'jsacapdev/k8s-routing-slip'...
remote: Counting objects: 57, done.
remote: Compressing objects: 100% (42/42), done.
remote: Total 57 (delta 13), reused 50 (delta 8), pack-reused 0
Unpacking objects: 100% (57/57), done.
$ cd jsacapdev/k8s-routing-slip
$ git checkout -qf cdeae2bc90067ff0e662125a62bbacae41642564
services
0.02s$ sudo service docker start
start: Job is already running: docker
$ bash -c 'echo $BASH_VERSION'
4.3.11(1)-release
before_install.1
0.01s$ echo "deb [arch=amd64] https://packages.microsoft.com/repos/azure-cli/ wheezy main" | sudo tee /etc/apt/sources.list.d/azure-cli.list
deb [arch=amd64] https://packages.microsoft.com/repos/azure-cli/ wheezy main
before_install.2
1.02s$ sudo apt-key adv --keyserver packages.microsoft.com --recv-keys 52E16F86FEE04B979B07E28DB02C46DF417A0893
Executing: /tmp/tmp.uEQPwVlUCD/gpg.1.sh --keyserver
packages.microsoft.com
--recv-keys
52E16F86FEE04B979B07E28DB02C46DF417A0893
gpg: requesting key 417A0893 from hkp server packages.microsoft.com
gpg: key 417A0893: public key "MS Open Tech<EMAIL_ADDRESS>imported
gpg: Total number processed: 1
gpg:               imported: 1  (RSA: 1)
before_install.3
3.27s$ sudo apt-get -qq update
W: GPG error: https://packages.microsoft.com/repos/azure-cli wheezy InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY EB3E94ADBE1229CF
W: The repository 'https://packages.microsoft.com/repos/azure-cli wheezy InRelease' is not signed.
W: There is no public key available for the following key IDs:
EB3E94ADBE1229CF
before_install.4
0.82s$ sudo apt-get install -qq apt-transport-https
0.80s$ sudo apt-get install -qq azure-cli
WARNING: The following packages cannot be authenticated!
  azure-cli
E: There were unauthenticated packages and -y was used without --allow-unauthenticated
The command "sudo apt-get install -qq azure-cli" failed and exited with 100 during .
Your build has been stopped.

Expected behavior
The cli is installed
Environment summary
See https://travis-ci.org/jsacapdev/k8s-routing-slip/jobs/383136723 for full build details.

@troydai thanks, that resolved it


Cannot ingest data

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [x] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce
If you follow the READMEs to ingest data, it does not work.
Mention any other details that might be useful
The first issue is to create a table and enable streaming ingestion (analytics/adx/create-table/README.md). The sample mentions the following command:
.alter table Events policy streamingingestion enable;

But I have an error : Admin command cannot be executed as streaming ingestion is not enabled
I checked the documentation  and tried the following:
.alter table Events policy streamingingestion '{"IsEnabled": true}'

But this doesn't work either (same error)
Then I tried to ingest. The README (analytics/adx/ingest-client/README.md) does not mention that we need to package the code as a JAR (mvn package) and execute it (java -jar target/ingest.jar).
But it doesnt work as there are a few env variables missing ADX_ENDPOINT, ADX_CLIENT_ID, ADX_CLIENT_SECRET and ADX_TENANT_ID

This issue is stale because it has been open 170 days with no activity. Remove stale label or comment or this will be closed in 10 days


youtube-dl doesn't download and convert mp4 videos from youtube in the correct resolution

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Hi, when trying to download these links: "https://www.youtube.com/watch?v=ezopeXwUukQ", "https://www.youtube.com/watch?v=1WjxBLgsQy8", and "https://www.youtube.com/watch?v=0DdPkt2QOKk" using the "-v --ignore-errors --format "best[ext=mp4][height<=720]" -o %(title)s.%(ext)s --cache-dir "C:\Users\Computer User\Music\youtube-dl and libav 11.3 files cache and downloads" --batch-file Files.txt" command, youtube-dl downloads them as 540x360 and 480x320 mp4, or 720x480 mkv files, but other youtube downloaders are able to download and convert those same files from the available streams as 720x480 mp4 files. It seems that issue is that youtube-dl isn't able to download and convert the proper available streams and merge them to create mp4 files, so is there a way to fix that? If other youtube downloaders can do that, then why can't youtube-dl? Thanks.

Hi, is there a way to merge all of these similar service requests: "https://github.com/ytdl-org/youtube-dl/issues/20831", "https://github.com/ytdl-org/youtube-dl/issues/20832", "https://github.com/ytdl-org/youtube-dl/issues/20887", and "https://github.com/ytdl-org/youtube-dl/issues/20888" into one service request? Thanks.
Hi. You provided me with the "--format "best[ext=mp4][height<=720]" command to use to download and convert files as 720p mp4 a while ago when I wanted help with that, because I didn't know which command to use. Everything was working fine and youtube-dl was able to download videos properly and convert them in that format until recently. Can you please tell me what command to use for youtube-dl to download videos in 720p (in this case, in the available 720x480 resolution), and mux the available streams using ffmpeg into an mp4 file? So far, I've only found out how to get youtube-dl to download the videos in the correct 720x480 available resolution, but it saves them as mkv files, and I want to find out how to get it to use ffmpeg to convert them to mp4 files. I'd really appreciate it, since the faq is very confusting for average computer users. Youtube-dl is a very good program by the way, once people know how to use it. Thank you.


Show progress in title bar

When installing/upgrading multiple packages at once (including their dependencies), show the progress as simple numbers in the title bar along the name of the current package, e.g. [5/21]: qt5

Added in commit f16cb10


Add documentation and test for EVP_PBE_alg_add

Fixes openssl#18687

Checklist

[x] documentation is added or updated
[x] tests are added or updated

Compilation failures in test need addressing

I am not sure about backporting it to 3.0 as we do not usually allow PRs that just add a testcase.

My approval is for master branch.

I think this is okay for 3.0 too.  It fixes documentation as well as adding a test case.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged to master and 3.0 branches. Thank you.


mscs status "bad ps syntax"

Received "bad ps syntax" warning when running mscs status.
Error:
$ mscs status
Minecraft Server Status:
  alpha: not running.
  shimniok: running version 1.11.2 (0 of 5 users online).
    Process ID: 7673.
Warning: bad ps syntax, perhaps a bogus '-'? See http://procps.sf.net/faq.html
    Memory used: 349480 kB.

Tracing msctl yields:
+ ps -a -U minecraft -o+  pid,comm,args -ww
/usr/bin/perl -ne if ($_ =~ /^\s*(\d+)\s+java.+mscs-world=shimniok$/) { print $1; }
+ PID=7673
+ printf %d\n 7673
+ PID=7673
+ ps -h -p 7673 -o rss
Warning: bad ps syntax, perhaps a bogus '-'? See http://procps.sf.net/faq.html
+ printf     Memory used: %d kB.\n 409456
    Memory used: 409456 kB.
+ exit 0

ps version:
$ ps --version
procps version 3.2.8

Server info:
Linux Mint 13
uname:
Linux nogales 3.2.0-23-generic #36-Ubuntu SMP Tue Apr 10 20:39:51 UTC 2012 x86_64 x86_64 x86_64 GNU/Linux
Let me know if I can help troubleshoot/test/whatever.

Further investionation:
It appears -h is the offending flag. If the intent is to strip headers, use --no-headers (on this system, anyway).
So, changing line (190) from:
ps -h -p $PID -o rss
to:
ps --no-headers -p $PID -o rss
results in:
$ mscs status
Minecraft Server Status:
  alpha: not running.
  shimniok: running version 1.11.2 (0 of 5 users online).
    Process ID: 7673.
    Memory used: 459604 kB.

The --no-headers option is synonymous with -h on my system (Ubuntu 17.04):
$ ps --version
ps from procps-ng 3.3.12

I checked a few other systems, and the oldest version of procps-ng I could find, 3.3.9 has the same --no-headers options.  I would be fine with making this change, hopefully there isn't some variant of ps that only works with the -h option.  Do you want to submit a pull request?

Sure, sounds good; I'll submit a pull request. Is there a particular branch you'd like me to use or create for changes? Thanks!

You can create a new branch on your fork if you like and work there.
When you submit the PR, just set the base as master.

Pull request accepted, thanks for the patch. Closing this issue.


Atlasmaps that extend to edge of image

If an atlas map falls outside of the section image, the atlas map remains anchored outside (cannot be dragged in). Is there a solution for this? The only solution I can see is to reanchor with QuickNII so that the atlas template is within the image boundaries...

There are solutions for this - Gergely will look into it.

I discussed this with Gergely. For now users should anchor sections so that the atlas maps fall within the boundaries of the sections (the autocropping feature in Transform makes this more likely, in some cases it may be helpful not to autocrop images). In the long run, Gergely will implement support to solve this, for example by adding a border around sections...

Do you have some update here?

@Tevemadar Did you implement this?

No, this is not implemented fully. VisuAlign allows manipulating an extra 10% border around images, so data created in the current release will be compatible with the future one, but that border displays only empty space now.

The 10% border around the image is satisfactory for now.

closed 03.09.2021


insertArrowTable broken

What happens?
nothing - no error - nothing to report - no insertion
To Reproduce
with duck-wasm 1.28, main, and 1.28.1-dev132.0
js-arrow 14.02
const somedata = [
      { key1: "a", key2: 1 },
      { key1: "b", key2: 2 }
    ];
    await c.query("CREATE TABLE someData(key1 VARCHAR, key2 integer)");
    const arrowTable = ar.tableFromJSON(somedata);
    await c.insertArrowTable(arrowTable, {
      name: "somedata",
      //create: true,
      schema: "main"
    });
    await c.insertArrowTable(EOS, { name:  "somedata"});
    console.log("somedata", (await c.query("select * from somedata")).toArray());

returns an empty array
Browser/Environment:
Mac FF and Chrome
Device:
M2
DuckDB-Wasm Version:
1.28
DuckDB-Wasm Deployment:
npm
Full Name:
Andrew Grosser
Affiliation:
Sourcetable

I think you need to insert an EOS buffer (new Uint8Array([255, 255, 255, 255, 0, 0, 0, 0])) after the actual data, see this example: https://duckdb.org/docs/api/wasm/data_ingestion.html#apache-arrow

I tried that and no luck too.

I had the same issue. On a fresh @duckdb/wasm npm project<EMAIL_ADDRESS>is installed automatically. In this case, inserting using arrow works. Once you upgrade apache-arrow to 15.0.0, it fails.

I was also running into the same issue. TL;DR with careful selection of which arrow version was in my own package.json I was able to get things working.
Documented a little more here: https://github.com/duckdb/duckdb-wasm/discussions/1545#discussioncomment-8550373
I'd been experimenting with updating to 1.28.1-dev106.0 as it appears as though a new release might be coming and have noticed that I cannot load data (with or without the additional EOS buffer)
I was playing with different versions of apache-arrow in my own package.json and found that matching exactly (14.0.1 for dev106 and 15.0.0 for dev132) would allow me to successfully insert data WITHOUT having to add the EOS buffer.

Ah legend thanks - moving to dev132 and 15 fixes it!

Ah seems like there's still errors, with a valid arrow table I get:
Error: Type metadata cannot be none
    at _class.insertArrowFromIPCStream (bindings_base.ts:342:19)
    at _callee$ (worker_dispatcher.ts:370:36)
    at tryCatch (runtime.js:64:40)
    at Generator.invoke (runtime.js:299:22)
    at Generator.next (runtime.js:124:21)
    at asyncGeneratorStep (_async_to_generator.js:3:28)
    at _next (_async_to_generator.js:20:17)
    at _async_to_generator.js:27:13
    at new Promise (<anonymous>)
    at _async_to_generator.js:16:16
async_bindings.ts:150 Uncaught (in promise) Error: Type metadata cannot be none
    at _class.onMessage (async_bindings.ts:150:23)

Example valid IPC file that breaks it
255,255,255,255,224,0,0,0,16,0,0,0,0,0,10,0,16,0,14,0,7,0,8,0,10,0,0,0,0,0,0,1,16,0,0,0,0,0,4,0,8,0,8,0,0,0,4,0,8,0,0,0,4,0,0,0,1,0,0,0,4,0,0,0,176,255,255,255,20,0,0,0,0,0,0,1,20,0,0,0,0,0,0,12,140,0,0,0,1,0,0,0,103,0,0,0,1,0,0,0,20,0,0,0,16,0,16,0,0,0,0,0,11,0,12,0,0,0,4,0,16,0,0,0,12,0,0,0,0,0,0,0,84,0,0,0,1,0,0,0,20,0,0,0,16,0,24,0,4,0,11,0,19,0,20,0,0,0,12,0,16,0,0,0,20,0,0,0,0,0,0,1,20,0,0,0,0,0,0,2,24,0,0,0,1,0,0,0,116,0,0,0,0,0,0,0,8,0,12,0,8,0,7,0,8,0,0,0,0,0,0,1,32,0,0,0,252,255,255,255,4,0,4,0,4,0,0,0,0,0,0,0,255,255,255,255,216,0,0,0,20,0,0,0,0,0,0,0,12,0,22,0,20,0,15,0,16,0,4,0,12,0,0,0,16,0,0,0,0,0,0,0,0,0,0,3,16,0,0,0,4,0,10,0,24,0,12,0,8,0,4,0,10,0,0,0,20,0,0,0,104,0,0,0,1,0,0,0,0,0,0,0,0,0,0,0,5,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,8,0,0,0,0,0,0,0,8,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,8,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,8,0,0,0,0,0,0,0,8,0,0,0,0,0,0,0,0,0,0,0,3,0,0,0,1,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,1,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,1,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,1,0,0,0,5,0,0,0,0,0,0,0,255,255,255,255,0,0,0,0

I just tried this on my current working version (1.28.0 -- with arrow 13.0.0):
const byteArray = Uint8Array.from([
  255,255,255,255,224,0,0,0,16,0,0,0,0,0,10,0,16,0,14,0,7,0,8,
  0,10,0,0,0,0,0,0,1,16,0,0,0,0,0,4,0,8,0,8,0,0,0,4,0,8,0,0,0,
  4,0,0,0,1,0,0,0,4,0,0,0,176,255,255,255,20,0,0,0,0,0,0,1,20,
  0,0,0,0,0,0,12,140,0,0,0,1,0,0,0,103,0,0,0,1,0,0,0,20,0,0,0,
  16,0,16,0,0,0,0,0,11,0,12,0,0,0,4,0,16,0,0,0,12,0,0,0,0,0,0,
  0,84,0,0,0,1,0,0,0,20,0,0,0,16,0,24,0,4,0,11,0,19,0,20,0,0,0,
  12,0,16,0,0,0,20,0,0,0,0,0,0,1,20,0,0,0,0,0,0,2,24,0,0,0,1,0,
  0,0,116,0,0,0,0,0,0,0,8,0,12,0,8,0,7,0,8,0,0,0,0,0,0,1,32,0,
  0,0,252,255,255,255,4,0,4,0,4,0,0,0,0,0,0,0,255,255,255,255,
  216,0,0,0,20,0,0,0,0,0,0,0,12,0,22,0,20,0,15,0,16,0,4,0,12,0,
  0,0,16,0,0,0,0,0,0,0,0,0,0,3,16,0,0,0,4,0,10,0,24,0,12,0,8,0,
  4,0,10,0,0,0,20,0,0,0,104,0,0,0,1,0,0,0,0,0,0,0,0,0,0,0,5,0,
  0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,8,0,0,0,
  0,0,0,0,8,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,8,0,0,0,0,0,0,0,0,0,
  0,0,0,0,0,0,8,0,0,0,0,0,0,0,8,0,0,0,0,0,0,0,0,0,0,0,3,0,0,0,
  1,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,1,0,0,0,0,0,0,0,0,0,0,0,0,0,
  0,0,1,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,1,0,0,0,5,0,0,0,
  0,0,0,0,255,255,255,255,0,0,0,0
])

const dataTable = tableFromIPC(byteArray)
console.log(JSON.stringify(dataTable.toArray()))
await connection.insertArrowTable(dataTable, {
  name: "table_name"
})

const results = await connection.query("select * from \"table_name\"")
console.log(JSON.stringify(results.toArray()))

This seems to work just fine, but the original inserted table (dataTable) and the query result (result) seem to not successfully be capturing the nested struct.
output of the above for me is:
[{"g":[null]}]

Wondering if this is a problem with the nested struct (or duckdb's compatibility with loading in arrow structs).

The newest version works for me, but I had to prefix the "new List" like so:
new List(Field.new({ name: "l", metadata: new Map(), {...other details you need in both arrow and duck}
That matches up with the schema that duck produces in 1.28.1-dev132.0
@carlopi ever changing this will be tragic for people - would love to get a thought on this


Security issue: use of a dangerous C functions

Bug description:
Hello there, while testing the security of our iOS app, the Red Team observed that the iOS Starscream framework used a potentially dangerous function that could introduce a vulnerability if it is used incorrectly. Functions such as _memcpy and _strncpy were used, these could potentially lead to certain types of vulnerabilities.
https://scriptingxss.gitbook.io/embedded-appsec-best-practices/1_buffer_and_stack_overflow_protection
We’ll be happy if you could review this treats if possible,
Thanks 🙏
Environment:

Starscream Latest iOS version
Xcode

Starscream doesn't use any C calls like memcpy or strncpy directly. Those are implementation details from the higher CoreFoundation APIs that Starscream uses. If you feel there is a security issue with those, I would suggest reporting them to the Apple Radar/bug tracker so the Foundation team can review and patch it.


How do Solve this Issue in Replit

In the Console It's Show this:
events.js:353
throw er; // Unhandled 'error' event
^
Error [ERR_STREAM_WRITE_AFTER_END]: write after end
at writeAfterEnd (_http_outgoing.js:694:15)
at ServerResponse.end (_http_outgoing.js:815:7)
at /home/runner/Replitsh-v4/index.js:56:70
at FSReqCallback.readFileAfterClose [as oncomplete] (internal/fs/read_file_context.js:73:3)
Emitted 'error' event on ServerResponse instance at:
at writeAfterEndNT (_http_outgoing.js:753:7)
at processTicksAndRejections (internal/process/task_queues.js:83:21) {
code: 'ERR_STREAM_WRITE_AFTER_END'
}
Can Anyone Help me?

Sorry I sen in the wrong issue

I accidentally Click Close the issue

Can you share a link to the Repl you're using this in?

Im closing this due to no response.


⚠ Buyfrom.io has degraded performance

In 7261146, Buyfrom.io ($BUYFROM_IO_URL) experienced degraded performance:

HTTP code: 200
Response time: 5892 ms

Resolved: Buyfrom.io performance has improved in 38370b1.


Thick white lines appearning in clustermap when large genesets used

Describe the bug
I have a matrix of size 157 rows (genes) × 1023 columns (cells). When I create a clustermap plot, I get these thick white lines/ artifacts appearing in the image.

Expected behavior
When I reduce the size of my matrix 59 rows (genes) × 1023 columns (cells), the clustermap outputs as it should.

What can I do about this?
Desktop (please complete the following information):
-Linux Mint 20.1

Chrome
Jupyter Notebook

Additional context
Plots generated following this tutorial with own data

Could you please check whether there were missing values in the data of 157 rows (genes) × 1023 columns (cells)?
I think the white rows were missing values.

Thank you. The issue was caused because of rows or columns which contained all zeros across the dataframe. When the
the z_score parameter was used, naturally the calculation failed on these rows/columns and resulted in NaNs which caused the white lines in plots.

Great! Thanks for letting me know.


TinyUSB_config_additions (IDFGH-6344)

This PR includes updates to the TinyUSB descriptor configuration, adding missing configurations for Vendor, MIDI and DFU runtime endpoints.  The missing "dfu_rt_device.c" source file is added to the tinyusb CMakeLists.txt

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for your contribution.

Hello @dskulina
I'm sorry for getting back to you so late, by the time git history got polluted, but I understand that you want to merge these two commits: fb0b6a015558567aa4983a431dcdf47877d2fc94 dc3b955963b40ee2e79cfb54c56b8e983c435b8b
Have you tested this code with 8 active endpoints? Because there is HW limitation in ESP32S2 and ESP32S3 that limits you to maximum 6 endpoints (5 IN or OUT and 1 IN) at a time.

Thanks for your contribution again, will close due to short of feedback, feel free to reopen with more updates. Thanks.