text:
RHBPMS-4145: Language selection is not remembered on relogin in Chrome

See https://issues.jboss.org/browse/RHBPMS-4145

Part of an ensemble:

https://github.com/droolsjbpm/kie-wb-common/pull/457
https://github.com/droolsjbpm/kie-wb-distributions/pull/349

+1

@csadilek This PR is "good to go" now... :-)

authors: csadilek manstis

repo: droolsjbpm/kie-wb-common


text:
noexcept in jniDefaultSetPendingFromCurrent

I get the following error when trying to run make example_android. I have made the following changes to Application.mk:
NDK_TOOLCHAIN_VERSION := clang
APP_PLATFORM := android-23
APP_STL := c++_shared
I am using the r17b NDK toolchain, because GCC is deprecated and will be removed next iteration. It should be noted that if I change NDK_TOOLCHAIN_VERSION back to 4.9, everything builds fine.
[arm64-v8a] Compile++      : djinni_jni <= djinni_support.cpp
../../../support-lib/jni/djinni_support.cpp:591:9: error: 'jniDefaultSetPendingFromCurrent' has a non-throwing exception specification but can still throw [-Werror,-Wexceptions]
        throw;
        ^
../../../support-lib/jni/djinni_support.cpp:588:6: note: function declared non-throwing here
void jniDefaultSetPendingFromCurrent(JNIEnv * env, const char * /*ctx*/) noexcept {
     ^                                                                   ~~~~~~~~
1 error generated.

It seems that the jniDefaultSetPendingFromCurrent function in djinni_support.cpp is marked noexcept, yet it still throws:
void jniDefaultSetPendingFromCurrent(JNIEnv * env, const char * /*ctx*/) noexcept {
    assert(env);
    try {
        throw;
    } catch (const jni_exception & e) {
        e.set_as_pending(env);
        return;
    } catch (const std::exception & e) {
        env->ThrowNew(env->FindClass("java/lang/RuntimeException"), e.what());
    }

    // noexcept will call terminate() for anything not caught above (i.e.
    // exceptions which aren't std::exception subclasses).
}

See discussion on PR #367

source: gharchive/issue

authors: artwyman jonmcclung

repo: dropbox/djinni

license_type: permissive

license_source: bigquery


text:
running droppy start

im having trouble getting this to work. Ive tried both the install script and following the instructions from the Documentation. I run into
Missing devDependencies to compile resource cache, please reinstall or run `npm install --only=dev` inside the project directory
    at compile (/usr/lib/node_modules/@droppyjs/cli/node_modules/@droppyjs/server/lib/services/resources.js:261:15)
    at ReadFileContext.callback (/usr/lib/node_modules/@droppyjs/cli/node_modules/@droppyjs/server/lib/services/resources.js:181:14)
    at FSReqCallback.readFileAfterOpen [as oncomplete] (fs.js:281:13)

Hey @dtuando are you using yarn for this?

Yes. Using yarn

I've still had installation problems with the latest release too, looks like that needs more work. I can help debugging this if need to be, although I cannot currently remember how it is broken or what my workaround to get it running/upgraded was.

@dtuando can you drop your version of node and npm please?

Got the same error after installing with npm:
tobias@hpserver:~$ npm install @droppyjs/cli
npm WARN deprecated fsevents@2.1.3: "Please update to latest v2.3 or v2.2"

> core-js@3.21.1 postinstall /home/tobias/node_modules/core-js
> node -e "try{require('./postinstall')}catch(e){}"

Thank you for using core-js ( https://github.com/zloirock/core-js ) for polyfilling JavaScript standard library!

The project needs your help! Please consider supporting of core-js:
> https://opencollective.com/core-js
> https://patreon.com/zloirock
> https://paypal.me/zloirock
> bitcoin: bc1qlea7544qtsmj2rayg0lthvza9fau63ux0fstcz

Also, the author of core-js ( https://github.com/zloirock ) is looking for a good job -)

npm WARN saveError ENOENT: no such file or directory, open '/home/tobias/package.json'
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY: fsevents@~2.1.2 (node_modules/chokidar/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for fsevents@2.1.3: wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
npm WARN enoent ENOENT: no such file or directory, open '/home/tobias/package.json'
npm WARN pdfjs-dist@2.9.359 requires a peer of worker-loader@^3.0.7 but none is installed. You must install peer dependencies yourself.
npm WARN tobias No description
npm WARN tobias No repository field.
npm WARN tobias No README data
npm WARN tobias No license field.

+ @droppyjs/cli@0.2.2
added 118 packages from 1059 contributors and audited 119 packages in 14.102s

16 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

npm --version: 6.14.16
node --version: v12.22.10

@DerTobi75 you should install the cli globally
npm install-g  @droppyjs/cli

@markhughes
That results in the same Errors. I tried to install without -g because someone mentions that fixed the problem for him.

@markhughes
This is what I got, when I try to install the cli globally:
sudo npm install -g @droppyjs/cli
[sudo] password for tobias:
npm WARN deprecated fsevents@2.1.3: "Please update to latest v2.3 or v2.2"
/usr/bin/droppy -> /usr/lib/node_modules/@droppyjs/cli/bin/droppy

> core-js@3.21.1 postinstall /usr/lib/node_modules/@droppyjs/cli/node_modules/core-js
> node -e "try{require('./postinstall')}catch(e){}"

Thank you for using core-js ( https://github.com/zloirock/core-js ) for polyfilling JavaScript standard library!

The project needs your help! Please consider supporting of core-js:
> https://opencollective.com/core-js
> https://patreon.com/zloirock
> https://paypal.me/zloirock
> bitcoin: bc1qlea7544qtsmj2rayg0lthvza9fau63ux0fstcz

Also, the author of core-js ( https://github.com/zloirock ) is looking for a good job -)

npm WARN optional SKIPPING OPTIONAL DEPENDENCY: fsevents@~2.1.2 (node_modules/@droppyjs/cli/node_modules/chokidar/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for fsevents@2.1.3: wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
npm WARN pdfjs-dist@2.9.359 requires a peer of worker-loader@^3.0.7 but none is installed. You must install peer dependencies yourself.

+ @droppyjs/cli@0.2.2
added 118 packages from 1059 contributors in 11.11s

tobias@hpserver:~$ droppy start -c /home/tobias/.droppy/config -f /home/tobias/.droppy/files

           .:.
    :::  .:::::.   @droppyjs/server 0.2.2 running on node 12.22.10
  ..:::..  :::     config at /home/tobias/.droppy/config
   ':::'   :::     files at /home/tobias/.droppy/files
     '

2022-02-25 00:17:50 [INFO] Configuration: { listeners: [ { host: [ '0.0.0.0', '::' ], port: 8989, protocol: 'http' } ], public: false, timestamps: true, linkLength: 5, linkExtensions: false, logLevel: 2, maxFileSize: 0, updateInterval: 1000, pollingInterval: 0, keepAlive: 20000, uploadTimeout: 604800000, allowFrame: false, readOnly: false, ignorePatterns: [], watch: true, headers: {}}
2022-02-25 00:17:50 [INFO] Loading resources ...
2022-02-25 00:17:50 [INFO] ENOENT /home/tobias/.droppy/cache/cache.json, building cache ...
2022-02-25 00:17:50 [INFO] Loading resources done
2022-02-25 00:17:50 [ERROR] Missing devDependencies to compile resource cache, please reinstall or run `npm install --only=dev` inside the project directory
    at compile (/usr/lib/node_modules/@droppyjs/cli/node_modules/@droppyjs/server/lib/services/resources.js:261:15)
    at ReadFileContext.callback (/usr/lib/node_modules/@droppyjs/cli/node_modules/@droppyjs/server/lib/services/resources.js:181:14)
    at FSReqCallback.readFileAfterOpen [as oncomplete] (fs.js:261:13)

Sorry all. I see the issue, my mistake - fix is incoming.

Resolved in 0.2.3

authors: DerTobi75 Tronic dtuando markhughes

repo: droppyjs/droppy

license_source: github-api


text:
Test asynchronous JAX-RS method

Consider an asynchronous JAX-RS method, part of an OkResource:
@GET
@Path("/ok")
@ManagedAsync
public void getOk(@Suspended final AsyncResponse asyncResponse) {
    asyncResponse.resume("OK");
}

Dropwizard, based on Jersey, provides the useful ResourceTestRule. A test class for the above method would contain an instance of this rule.
@ClassRule
public static ResourceTestRule RULE = ResourceTestRule.builder()
  .addResource(new OkResource())
  .build();

A test method for the JAX-Rs method would look like
@Test
public void testGetOk()  {
    Response response = RULE.target("/ok).request().get();
    // assert ...
}

For synchronous JAX-RS methods this works fine. But for asynchronous JAX-RS like above this approach does not work:

WARN  [2019-06-26 09:15:15,811] org.glassfish.jersey.test.inmemory.InMemoryConnector: Asynchronous server side invocations are not supported by InMemoryContainer.

Is there a different way to test asynchronous JAX-RS methods without having to directly call the method (getOk in this case) with a mocked AsyncResponse?

@lutzhorn Have you tried using another test container?
https://jersey.github.io/documentation/2.28/test-framework.html#d0e17501

I will try that, thank you.

Works. Maybe I should read the documentation before opening an issue :)

authors: joschi lutzhorn

repo: dropwizard/dropwizard


text:
Dependency updates

Jackson 2.8.6
Guava 21.0
Mockito 2.6.1
Jetty 9.3.15.v20161220 (although this one might make problems, as usual)

Changes Unknown when pulling c7bc2c06b726efaeecbfd722c7faf4ff1fe1b55d on dependency-updates into ** on master**.

source: gharchive/pull-request

authors: coveralls joschi


text:
Add some commonly used and useful jersey request/response filters

X-Request-Id filter is based upon https://devcenter.heroku.com/articles/http-request-id
X-Runtime filter is based upon https://github.com/rack/rack/blob/master/lib/rack/runtime.rb

Coverage decreased (-0.3%) to 84.526% when pulling 274b375b9179b652d048ae1da00c23dae76a7b3e on jplock:jp-filters into 857cb187e6f882cb2d05b6ba8291a99861165b81 on dropwizard:master.

Coverage decreased (-0.3%) to 84.526% when pulling 1cb846b853418845d9ac12ef1bb7a07e32e0de4a on jplock:jp-filters into 857cb187e6f882cb2d05b6ba8291a99861165b81 on dropwizard:master.

Coverage decreased (-0.3%) to 84.526% when pulling 1cb846b853418845d9ac12ef1bb7a07e32e0de4a on jplock:jp-filters into 857cb187e6f882cb2d05b6ba8291a99861165b81 on dropwizard:master.

I'm going to play devils advocate on two of these new filters. No issues with the charset filter
RequestIdFilter could be modeled of a more opinionated stance on request tracking similar to this bundle. It doesn't handle requests that already have an id in the request header. It isn't utilizing MDC which enables rich correlation of logs events from the entire request. Though the MDC does require the user to property format their log patterns.
https://github.com/service-enabled/dropwizard-request-tracker
RuntimeFilter seems to be duplicating the functionality/purpose of the access logs and will likely report a different time since it will be based on when the filter executed vs when jetty logs the access event.

@mattnelson thanks for the feedback.  I think I can make the X-Request-Id better by looking for an existing X-Request-Id header in the request and using that value instead of generating a new UUID.  I'll take a look at the request-tracker link to see if there's other functionality which may be missing.  The X-Runtime filter was just to provide an easy mechanism for knowing how long an API response took.  If you think this is confusing, I can remove it, but I've found it useful in some of the API's I've built.

The request filter should probably be opt-in since UUID performance is highly dependent on the concurrent load.
[1] http://www.cowtowncoder.com/blog/archives/2010/10/entry_429.html
[2] https://github.com/Netflix/netflix-commons/blob/v0.3.0/netflix-commons-util/src/main/java/com/netflix/util/concurrent/ConcurrentUUIDFactory.java

@mattnelson none of these are required.  They are just provided in the code so someone can optionally use them.  They aren't being tied in by default.

They aren't being enabled by default.

That alleviates most of my concerns. I was thinking that these filters would be auto registered based on the annotations, but it does not appear that this package is added to the package scanning list.

@mattnelson I pushed up some updates to address your feedback

Coverage decreased (-0.5%) to 84.302% when pulling 3c2cafcc478cfffa7d52ade22e4abb5a5bf7b65e on jplock:jp-filters into 857cb187e6f882cb2d05b6ba8291a99861165b81 on dropwizard:master.

Coverage decreased (-0.5%) to 84.302% when pulling 3c2cafcc478cfffa7d52ade22e4abb5a5bf7b65e on jplock:jp-filters into 857cb187e6f882cb2d05b6ba8291a99861165b81 on dropwizard:master.

I pushed up some updates to address your feedback

Changes look good to me.

Thanks @jplock for the contribution, @joschi, @evnm, and @mattnelson for the review!

authors: arteam coveralls jplock mattnelson


text:
Semantic Versioning Documentation, for drud/ddev #352

The Problem:
We committed to support this and we just need some additional documentation outside the issue queues to both 1) make this more obvious to an end-users and 2) make this as part of our release processes.
The Fix:
See verbiage in PR.
Related Issue Link(s):
OP drud/ddev/issue/352

@rfay Added you as reviewer to get the official +1

repo: drud/community


text:
ModuleList is already part of Initialization

This class is already part of Initialization and seems to be unused. Please correct me if I'm wrong or if it's used externally for some reason.

LGTM

authors: himanshug xvrl

repo: druid-io/druid


text:
killDataSourceWhitelist in CoordinatorDynamicConfig accepts comma separated list of strings in addition to json array of strings so that coordinator console can do the updates correctly

#3095

👍

👍

authors: drcrallen gianm xvrl


text:
feat(drupal): Allow default install to work single node

Currently the default install won't fully work until Azure Files are setup so that the /var/www/html/sites/default/files are shared between the drupal and nginx pods.
Depending on the cloud this can be setup in 1...N ways, however we do provide examples for how this is specifically done either with Azure Files or Azure Shared Disk.
You can see a full example using Azure Files using the CSI Driver here:

https://github.com/drupalwxt/helm-drupal/blob/master/drupal/values-example-csi-azure.yaml

I was thinking we should make the default install work with the caveat it will only work when you only have 1 node in the cluster because the PVC can only attach to multiple nodes if it is the one same node or you will get a multi attach error.
This technically isn't best practice so wanted to see if this is something we should add as a default.

If you decide to move forward with this change, I would suggest to add some context (test purpose), limitation (single node k8s cluster) and instructions (helm command) on the readme page.

authors: PatrickMessierGoC sylus

repo: drupalwxt/helm-drupal

license: MIT


text:
Come up with a nice favicon and image?

The documentation could perhaps use a nice favicon and image?
If anyone's good with design etc... Perhaps throw some images in this issue?
(Here's the currently rendered documentation: http://axelrod.readthedocs.org/en/latest/)
All files are written using sphinx in rst and can be found in the docs directory.

Just pushed this which is slightly better but please do change it :)

repo: drvinceknight/Axelrod

license: mit


text:
Anyone using this in WinForms successfully?

all examples I saw were WPF. Did anyone use this in a WinForms project?
#56 is my open issue for WinForms
If someone has a running example for WinForms would be very helpful.
Even the demo WinForms project is not working...

It seems to work for me:

WindowsFormsApp1.zip
I have never used WinForms though, so really can't help you any further than this.

HA! thanks a lot!
I saw that we have the same source for using - but your demo works. So there must be another issue.
And it was the GUID of the project. Somehow my registry has some entries scrambled with this GUID. I created another one, and voila - my band can be selected for displaying in the taskbar.
Now I can focus on my other tasks, completing the tools...

authors: obelixA stnkl

repo: dsafa/CSDeskBand


text:
housekeeping: Update module cactus/go-statsd-client/statsd to v1

This PR contains the following updates:

Package
Type
Update
Change

github.com/cactus/go-statsd-client/statsd
require
major
v0.0.0-20200623234511-94959e3146b2 -> v1.0.1

Release Notes

cactus/go-statsd-client
v1.0.1
Compare Source

Renovate configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you check the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: backend/go.sum
Command failed: go get -d ./...
go: github.com/cactus/go-statsd-client/statsd@v1.0.1: reading github.com/cactus/go-statsd-client/statsd/statsd/go.mod at revision statsd/v1.0.1: unknown revision statsd/v1.0.1

repo: dschaller/clutch


text:
Revisit b5929e7

Just a reminder. We will retest when the next OpenDSS release is out anyway.

SVN r3477 adds some changes related to this. We will reapply and retest whenever new official  OpenDSS binaries are available (their latest release is affected by the issue r3477 mentions).

Changes reapplied. Before closing this, we should retest and check, then prepare a new release.


text:
PipelineWrapper has no attribute 'get_config'

@ScottyB I do not see the need of get_config anyway. We can just do something like:
config = self.surround.config

repo: dstil/surround


text:
Please add a feature to make presets for custom arguments

For the command line version, I have a few different presets I use, for example
-o "Music\%(channel)s\%(title)s" -x --embed-thumbnail --audio-format mp3 for music
-o "Videos/%(channel)s/%(title)s.%(ext)s" --embed-thumbnail --embed-chapters -f "bv*[height<=720]+ba/b[height<=720]" for videos
and looking in the config.json file, I feel like it wouldn't be hard to add a preset feature

Additionally, a thing where we can just use arguments instead of the presets given by the dropdowns and checkboxes would be nice, for custom formats at least

Hello. I did a little test. Let me know how this work:
https://github.com/GuyMicciche/yt-dlp-gui/releases/tag/2024.08.27

Looks good, the only small issue I see is that you can save multiple of the same named preset so there's multiple of identically named presets in the drop-down. I cloned the repo so I can make some contributions

@CraftyDuck100 Thank you!

@CraftyDuck100 I really liked this idea. At first, I wasn't sure how to implement it without losing the program's simplicity and changing the experience for older users. However, I was able to keep both aspects, so it can be used out of the box or customized for more advanced users.
All the customization options are in the config.json file. I will write a more detailed README to explain everything once I merge it into the main branch. For now, each preset only needs one required field called "args" which can be a list or a string, the rest are optional. This field contains the yt-dlp command arguments that will be executed. to add your music example I'll simply add a music preset after the mp3 preset:
...
"mp3": {
  ...
},
"music": {
  "args": "-o \"C:/Users/user/Desktop/Music/%(channel)s/%(title)s\" -x --embed-thumbnail --audio-format mp3"
}

You'll notice that all fields that aren't present in the config.json for the preset will be disabled in the gui.
I'd appreciate if you could do some testing on the config branch (refer to the log file if you have problems) if it works as expected I'll merge.

Alright I'll test it a bit

I apologize, I completely forgot about this. I will take a look as soon as I have time, which will be sometime tomorrow

Ended up changing the config file to toml to make it easier to configure. if you run into issues let me know.

Never got around to doing it, sorry, but thank you for all your hard work

authors: CraftyDuck100 GuyMicciche dsymbol

repo: dsymbol/yt-dlp-gui

license: Unlicense


text:
[Windows 10] npm install cash-global -g does not work

npm install cash-global -g
Restart pc
'ls' is not recognized as an internal or external command, operable program or batch file

However when I do cash it works as before. Am I missing something?
C:\Users\stani>node -v
v5.9.1
C:\Users\stani>npm -v
3.8.3

Was there any installation errors? Can you show me the text from the installation?

@dthree No errors whatsoever.
C:\Users\stani>npm install cash-global -g

> cash-global@0.2.0 preinstall C:\Users\stani\AppData\Roaming\npm\node_modules\.staging\cash-global-4e34e430
> node ./bin/preinstall.js

Installed:      clear
Installed:      cp
Installed:      false
Installed:      kill
Installed:      ls
Installed:      mkdir
Installed:      mv
Installed:      pwd
Installed:      touch
Installed:      true
Installed:      cat
Installed:      grep
Installed:      rm

C:\Users\stani\AppData\Roaming\npm\cash -> C:\Users\stani\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\bin\cash.js
C:\Users\stani\AppData\Roaming\npm\$ -> C:\Users\stani\AppData\Roaming\npm\node_modules\cash-global\node_modules\cash\bin\cash.js

> cash-global@0.2.0 postinstall C:\Users\stani\AppData\Roaming\npm\node_modules\cash-global
> node ./bin/postinstall.js

C:\Users\stani\AppData\Roaming\npm
`-- cash-global@0.2.0
  `-- cash@0.8.0
    +-- array-shuffle@1.0.0
    +-- chalk@1.1.3
    | +-- ansi-styles@2.2.1
    | +-- escape-string-regexp@1.0.5
    | +-- has-ansi@2.0.0
    | | `-- ansi-regex@2.0.0
    | +-- strip-ansi@3.0.1
    | `-- supports-color@2.0.0
    +-- filesize@3.2.1
    +-- fkill@3.1.0
    | +-- arrify@1.0.1
    | +-- pify@2.3.0
    | +-- pinkie-promise@2.0.0
    | | `-- pinkie@2.0.4
    | `-- taskkill@1.0.0
    +-- fs-extra@0.23.1
    | +-- graceful-fs@4.1.3
    | +-- jsonfile@2.2.3
    | +-- path-is-absolute@1.0.0
    | `-- rimraf@2.5.2
    |   `-- glob@7.0.3
    +-- glob@6.0.4
    | +-- inflight@1.0.4
    | | `-- wrappy@1.0.1
    | +-- inherits@2.0.1
    | +-- minimatch@3.0.0
    | | `-- brace-expansion@1.1.3
    | |   +-- balanced-match@0.3.0
    | |   `-- concat-map@0.0.1
    | `-- once@1.3.3
    +-- lodash@4.6.1
    +-- minimist@1.2.0
    +-- user-home@2.0.0
    | `-- os-homedir@1.0.1
    +-- username@1.0.1
    | `-- meow@3.7.0
    |   +-- camelcase-keys@2.1.0
    |   | `-- camelcase@2.1.1
    |   +-- decamelize@1.2.0
    |   +-- loud-rejection@1.3.0
    |   | +-- array-find-index@1.0.1
    |   | `-- signal-exit@2.1.2
    |   +-- map-obj@1.0.1
    |   +-- normalize-package-data@2.3.5
    |   | +-- hosted-git-info@2.1.4
    |   | +-- is-builtin-module@1.0.0
    |   | | `-- builtin-modules@1.1.1
    |   | +-- semver@5.1.0
    |   | `-- validate-npm-package-license@3.0.1
    |   |   +-- spdx-correct@1.0.2
    |   |   | `-- spdx-license-ids@1.2.0
    |   |   `-- spdx-expression-parse@1.0.2
    |   |     `-- spdx-exceptions@1.0.4
    |   +-- object-assign@4.0.1
    |   +-- read-pkg-up@1.0.1
    |   | +-- find-up@1.1.2
    |   | | `-- path-exists@2.1.0
    |   | `-- read-pkg@1.1.0
    |   |   +-- load-json-file@1.1.0
    |   |   | +-- parse-json@2.2.0
    |   |   | | `-- error-ex@1.3.0
    |   |   | |   `-- is-arrayish@0.2.1
    |   |   | `-- strip-bom@2.0.0
    |   |   |   `-- is-utf8@0.2.1
    |   |   `-- path-type@1.1.0
    |   +-- redent@1.0.0
    |   | +-- indent-string@2.1.0
    |   | | `-- repeating@2.0.0
    |   | |   `-- is-finite@1.0.1
    |   | `-- strip-indent@1.0.1
    |   |   `-- get-stdin@4.0.1
    |   `-- trim-newlines@1.0.0
    +-- vorpal@1.10.10
    | +-- babel-polyfill@6.7.4
    | | +-- babel-regenerator-runtime@6.5.0
    | | +-- babel-runtime@5.8.38
    | | | `-- core-js@1.2.6
    | | `-- core-js@2.2.1
    | +-- in-publish@2.0.0
    | +-- inquirer@0.11.0
    | | +-- ansi-escapes@1.3.0
    | | +-- cli-cursor@1.0.2
    | | | `-- restore-cursor@1.0.1
    | | |   +-- exit-hook@1.1.1
    | | |   `-- onetime@1.1.0
    | | +-- cli-width@1.1.1
    | | +-- figures@1.5.0
    | | +-- lodash@3.10.1
    | | +-- readline2@1.0.1
    | | | +-- code-point-at@1.0.0
    | | | | `-- number-is-nan@1.0.0
    | | | +-- is-fullwidth-code-point@1.0.0
    | | | `-- mute-stream@0.0.5
    | | +-- run-async@0.1.0
    | | +-- rx-lite@3.1.2
    | | `-- through@2.3.8
    | +-- log-update@1.0.2
    | +-- node-localstorage@0.6.0
    | `-- wrap-ansi@2.0.0
    |   `-- string-width@1.0.1
    +-- vorpal-autocomplete-fs@0.0.3
    +-- vorpal-grep@0.1.2
    | `-- glob@5.0.15
    `-- vorpal-less@0.0.13
      `-- slice-ansi@0.0.3

C:\Users\stani>ls
'ls' is not recognized as an internal or external command,
operable program or batch file.

C:\Users\stani>

Huh. Interesting.
Did it work before you restarted the PC?

Nope, I just tried it and it didn't work. :)

I am also having this problem on Windows 10 (build 1511) with Node js v5.9.0 and npm v3.8.5.
@stanislavromanov What language is used in your system? Russian?

Is the cash-ls module actually getting installed? You would have to check the location of your global installs (npm config get prefix might point you closer to the right spot, but I'm not 100% sure).
Next thing I can think of would be to check the value of your PATH env var. Try running echo %PATH% in a windows terminal to see its value (or echo $PATH in a cash terminal, if you're daring). Hope this helps!

@nfischer Well I don't think the problem is with PATH because every other npm module works fine. Even cash itself, just not cash-global

I had this problem back when I first tried this, from what I remember of my investigation then for some reason the *.bat wrapper scripts weren't getting created for the sub-modules, only cash and $.

Looks like a sequencing thing. NPM does bindings based on the bin object in package.json, and originally only cash and $ are there. Cash global then runs script hooks to re-write the package.json and then run the linkings again. However, it looks like NPM is inconsistent on how it does this, as certain Windows versions work as expected and others don't.

Well I guess we don't be needing this as Windows Bash replacement soon enough: http://blog.dustinkirkland.com/2016/03/ubuntu-on-windows.html

@stanislavromanov Holy Grail! :bomb: :boom:

Amazing, right? :smile: :fire: :bomb:

Nope, I just tried it and it didn't work. :)

good luck

May this be related to #74?

I have the same, I have an english version of windows 10.
npm -v
3.3.12

node -v
v5.9.1

OS Name:                   Microsoft Windows 10 Pro
OS Version:                10.0.10586 N/A Build 10586

Wouldn't you need to type in cash before using ls?

@SphinxTwo The entire point of cash-global is so you don't have to type cash before using the commands 😛.

I'm having the same issue on windows 8.1. Let me know if there is anything I can do to help :). Very cool idea, I miss my linux commands :).

+1, Having the same problem on two win10s

Same problem here, win10, node v7.3.0

Same here, win 10.

type cash first

authors: Arcanemagus Loilo SphinxTwo StefanDimov dthree gonzalocasas hoelter jaylamsen mrmlnc nfischer stanislavromanov tandyjames tangsilian xtreemrage

repo: dthree/cash


text:
nodemon messes up vorpal command history

when vorpal is launched from within nodemon, the command history is non-functional.   Instead of previous commands appearing with the up cursor you get  ^[[A

Same problem. Answered as the 1st entry in nodemon FAQ https://github.com/remy/nodemon/blob/master/faq.md#nodemon-doesnt-work-with-my-repl
If using the "restartable" option, nodemon takes over stdin to catch the manual restart command. Using  "restartable": false solves the problem.

@dkebler can you try that and let me know?
--
I threw this in the Wiki:
https://github.com/dthree/vorpal/wiki/Docs-|-Integration

just now saw this.  Thanks! working like a charm!

authors: Offirmo dkebler dthree

repo: dthree/vorpal


text:
crash when used as face-view labeling

I tried to use it as demonstrated in mvs_texturing, but for some meshes it crashes:

I can send the mesh, but I do not see an easy way to dump the mapmap problem to a file to send you the problem.

Hey, I ran into the exact same problem as yours. Did you work it out eventually?

but for some meshes it crashes:

For me, I didn't find out a single working mesh...

Hi there,
I'd be happy tro investigate that. If you send me the mesh and instructions to replicate, I'll take a shot at it (at least under Linux).

Hi there,
I'd be happy tro investigate that. If you send me the mesh and instructions to replicate, I'll take a shot at it (at least under Linux).
Hi, this is one of my meshes that is keep getting the segmentation fault. I have tried it on Linux(Ubuntu) and it works fine on it.
https://drive.google.com/file/d/1nJ9ZDvfvBihSqb-oXHONyZC1fqiuvODk/view?usp=sharing
I tried to debug it for serveral days now. I suspect it's related to some array alignment problems like the link below.
https://stackoverflow.com/questions/62324009/mm256-load-ps-cause-segmentation-fault-with-google-benchmark-in-debug-mode
BTW, I am using MinGW-w64 GCC (10.2.0) as compiler on windows10.

actually, you may not need the mesh. I ran into the same issue in mapmap_demo or mapmap_test.
It always crashed here.

I can't figure out why this can trigger a segmentation fault. After some tries, I changed the line a little bit.
diff --git a/mapmap/source/optimizer_instances/dp_node_solver.impl.h b/mapmap/source/optimizer_instances/dp_node_solver.impl.h
index 63498f8..5f82c87 100644
--- a/mapmap/source/optimizer_instances/dp_node_solver.impl.h
+++ b/mapmap/source/optimizer_instances/dp_node_solver.impl.h
@@ -246,7 +246,8 @@ get_independent_of_parent_costs(
         label_from_offset(node_id, 0);

     /* vector holding costs */
-    _v_t<COSTTYPE, SIMDWIDTH> cost = v_init<COSTTYPE, SIMDWIDTH>();
+    _s_t<COSTTYPE, SIMDWIDTH> test = 0;
+    _v_t<COSTTYPE, SIMDWIDTH> cost = v_init<COSTTYPE, SIMDWIDTH>(test);

     /* retrieve label vector for this offset */
     _iv_t<COSTTYPE, SIMDWIDTH> l = m_node->c_labels->

Then I not longer received the segmentation fault in that line. Instead, I crashed in the few lines below.
    cost = c_unary->supports_enumerable_costs() ?
        v_add<COSTTYPE, SIMDWIDTH>(cost,
        c_unary->get_unary_costs_enum_offset(l_i)) :
        v_add<COSTTYPE, SIMDWIDTH>(cost, c_unary->get_unary_costs(l));

Interesting - so far, I seem to be unable to reproduce it. Can you give me the rough outline of your system (especially CPU and OS) so I can narrow this down to the right instruction set (AVX/2, SSE, ...)?

One further thing to try would be edit all v_load variants in source/vector_math.impl.h so they always use the unaligned intrinsics (loadu). I'll see if I can get my hands on a windows machine to test that.

Interesting - so far, I seem to be unable to reproduce it. Can you name me the CPU you're using so I can narrow this down to the right instruction set (AVX/2, SSE, ...)?

My CPU is Intel(R) Core(TM) i3-8100 CPU @ 3.60GHz (4 CPUs), ~3.6GHz

One further thing to try would be edit all v_load variants in source/vector_math.impl.h so they always use the unaligned intrinsics (loadu). I'll see if I can get my hands on a windows machine to test that.

Just tested it, no luck though.

Surprisingly, I tested mapmap_demo compiled by MVSC and NO SegFault! Though as https://github.com/dthuerck/mapmap_cpu/issues/21 mention, I have to change the line in prev_level().
I was using MinGW-GCC 10.2.0 as compiler, maybe the tbb provieded by msys2's package manager has some issues. I will try to rebuild it to see if the problem still exsits.

I can confirm that my bug is related to the compiler, there've been serval discussion about this bug.
https://stackoverflow.com/questions/30928265/mingw64-is-incapable-of-32-byte-stack-alignment-required-for-avx-on-windows-x64
https://sourceforge.net/p/mingw-w64/mailman/message/34485783/

Ah, too bad, it's a compiler bug. Are you able to switch to VS? I'll push the change in #21 upstream.
Regarding this problem, I only see two possibilities going forward:

Avoid MinGW ;)
Move variables from stack to heap.

Ah, too bad, it's a compiler bug. Are you able to switch to VS? I'll push the change in #21 upstream.
Regarding this problem, I only see two possibilities going forward:
1. Avoid MinGW ;)

2. Move variables from stack to heap.

Yeah, but I was able to avoid the problem by setting simd width to 4 using mingw. I don't know what it will do to the final outcome though.

@xubury @dthuerck
I am facing the same issue reported here by @cdcseacave. I packed the problem to a bin file in this issue https://github.com/dthuerck/mapmap_cpu/issues/37#issuecomment-1218231886

I think I might have found the reason, looks like the "labels" within the "label_set" are assumed to be sorted. If not, bad things happen. Correct?

Solved by making sure labels are sorted.
cc: @cdcseacave in case you are still interested in trying it again.

Thanks for investigating!

authors: cdcseacave dthuerck melhashash xubury


text:
Structs with unsized final field

I'd like to be able to do something like
struct Foo<T: ?Sized>(x: Bar, y: T);

impl<T: ?Sized + DynClone> DynClone for Foo<T> { ... }

But I'll need to call something which just initializes the pointer, rather than allocates and initializes.

This is not something I would want to support in this crate.

repo: dtolnay/dyn-clone


text:
Expose the quote::quote_each_token macro as something like quote_into!()

If this were exposed, users of the crate would be able to easily directly extend an existing token stream, rather than creating a new one an immediately deallocating it. It might not matter all that much, given that the root of a token stream doesn't have all that many TokenTrees in it (i.e. quote!(impl Foo { /* 1000 lines of code */ }).into_iter().count() == 3) but given that the functionality already exists, I think it couldn't hurt to expose it. Something that looks like:
let mut output = input.into_token_stream();

for x in impls_or_extra_type_decls {
    let (a, b, c) = x;
    quote_into! { output,
        ... #a ... #b ... #c
    }
}

output.into()

As you said, it doesn't make a difference, so I would prefer not to expose an additional API for this.

As you said, it doesn't make a difference, so I would prefer not to expose an additional API for this.

repo: dtolnay/quote


text:
Add a mutable version of Visitor

I want to be able to walk over an AST and do substitutions/mutations in-place. I think there should be a version of Visitor which mutates the AST.
eg.
pub trait VisitorMut: Sized {
    fn visit_ident(&mut self, _ident: &mut Ident) {}
    fn visit_derive_input(&mut self, derive_input: &mut DeriveInput) {
        walk_derive_input_mut(self, derive_input)
    }
    fn visit_ty(&mut self, ty: &mut Ty) {
        walk_ty_mut(self, ty)
    }
    ... // etc.
}

Thoughts?

I think that this should probably be part of an effort to unify the visitor-style structs to use a common codepath of some type, as right now they are each managed separately and definitely have bugs in them (as in they miss some subtrees or nodes, or don't have enough visit methods).
If we could use custom derives it would be awesome to #[derive(Visitors)] on these types.

This has been fixed in #206

authors: canndrew mystor

repo: dtolnay/syn


text:
Possible misuse of CAPI causing wal files to remain intact and mek DB inaccessible

Here is the context (have a look at the reply on this too):
https://github.com/duckdb/duckdb/issues/10002#issuecomment-2563013881

This was resolved in the linked thread.

@jraymakers Just wanted to say... the new .stream() api is inanely fast compared to previous versions'  .run() api... i saw like 10x improvement... my use case was to run query on entire data set, but only read first 2048 rows from resultset.
Awesome work! Thanks..
May I sugest to add the usage of .stream() API in npm docs too? Currently it doesn;t list it's usage.. but, only the call... all other api's show using resultset too. I had to refer to this PR to learn how to use it: https://github.com/duckdb/duckdb-node-neo/pull/81/files

Glad to hear it!
I'll think about how I can make the different ways of running results more clear. There are a lot of combinations, so it's tricky to document them all clearly. I did add a bit about streamAndReadUntil, which is my recommended way of doing streaming, but I probably didn't make this clear enough.

@jraymakers yep.. i think so many very similar looking API's are adding a bit cognitive load when deciding which one to use. Maybe grouping them by use-cases might help.

@jraymakers Was trying new streamAndReadUntil api... getting this crash:
No tables to drop.
Tables dropped. Loading data and getting count...

<--- Last few GCs --->

[5009:0x110008000]    32393 ms: Scavenge (reduce) 2047.7 (2082.4) -> 2047.2 (2082.9) MB, 22.29 / 0.00 ms  (average mu = 0.386, current mu = 0.387) allocation failure;
[5009:0x110008000]    32568 ms: Mark-Compact (reduce) 2048.0 (2082.9) -> 2047.3 (2083.1) MB, 112.96 / 0.00 ms  (+ 183.2 ms in 39 steps since start of marking, biggest step 6.4 ms, walltime since start of marking 355 ms) (average mu = 0.374, current mu = 0

<--- JS stacktrace --->

FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
----- Native stack trace -----

 1: 0x102a3959c node::Abort() [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
 2: 0x102a3979c node::ModifyCodeGenerationFromStrings(v8::Local<v8::Context>, v8::Local<v8::Value>, bool) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
 3: 0x102bbe95c v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, v8::OOMDetails const&) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
 4: 0x102d93050 v8::internal::Heap::GarbageCollectionReasonToString(v8::internal::GarbageCollectionReason) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
 5: 0x102d96f04 v8::internal::Heap::CollectGarbageShared(v8::internal::LocalHeap*, v8::internal::GarbageCollectionReason) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
 6: 0x102d93968 v8::internal::Heap::PerformGarbageCollection(v8::internal::GarbageCollector, v8::internal::GarbageCollectionReason, char const*) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
 7: 0x102d916f0 v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
 8: 0x102d88344 v8::internal::HeapAllocator::AllocateRawWithLightRetrySlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
 9: 0x102d88ba4 v8::internal::HeapAllocator::AllocateRawWithRetryOrFailSlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
10: 0x102d6d800 v8::internal::Factory::AllocateRawWithAllocationSite(v8::internal::Handle<v8::internal::Map>, v8::internal::AllocationType, v8::internal::Handle<v8::internal::AllocationSite>) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
11: 0x102d7bd50 v8::internal::Factory::NewJSArrayBufferView(v8::internal::Handle<v8::internal::Map>, v8::internal::Handle<v8::internal::FixedArrayBase>, v8::internal::Handle<v8::internal::JSArrayBuffer>, unsigned long, unsigned long) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
12: 0x102d7c214 v8::internal::Factory::NewJSTypedArray(v8::internal::ExternalArrayType, v8::internal::Handle<v8::internal::JSArrayBuffer>, unsigned long, unsigned long, bool) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
13: 0x102be7088 v8::Uint8Array::New(v8::Local<v8::ArrayBuffer>, unsigned long, unsigned long) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
14: 0x102a10814 node::Buffer::Copy(node::Environment*, char const*, unsigned long) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
15: 0x102a1065c node::Buffer::Copy(v8::Isolate*, char const*, unsigned long) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
16: 0x102a070d4 napi_create_buffer_copy [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
17: 0x10861d6b0 DuckDBNodeAddon::get_data_from_pointer(Napi::CallbackInfo const&) [/Users/nitinbansal/dev/gitlab/nodejs/zenquery/node_modules/@duckdb/node-bindings-darwin-arm64/duckdb.node]
18: 0x10860c74c Napi::InstanceWrap<DuckDBNodeAddon>::InstanceMethodCallbackWrapper(napi_env__*, napi_callback_info__*) [/Users/nitinbansal/dev/gitlab/nodejs/zenquery/node_modules/@duckdb/node-bindings-darwin-arm64/duckdb.node]
19: 0x1029f797c v8impl::(anonymous namespace)::FunctionCallbackWrapper::Invoke(v8::FunctionCallbackInfo<v8::Value> const&) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
20: 0x10342def0 Builtins_CallApiCallback [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
21: 0x1380751f4
22: 0x138075e74
23: 0x103463210 Builtins_AsyncFunctionAwaitResolveClosure [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
24: 0x103510fb8 Builtins_PromiseFulfillReactionJob [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
25: 0x103452b94 Builtins_RunMicrotasks [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
26: 0x10342a3f4 Builtins_JSRunMicrotasksEntry [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
27: 0x102d004d0 v8::internal::(anonymous namespace)::Invoke(v8::internal::Isolate*, v8::internal::(anonymous namespace)::InvokeParams const&) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
28: 0x102d009bc v8::internal::(anonymous namespace)::InvokeWithTryCatch(v8::internal::Isolate*, v8::internal::(anonymous namespace)::InvokeParams const&) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
29: 0x102d00b98 v8::internal::Execution::TryRunMicrotasks(v8::internal::Isolate*, v8::internal::MicrotaskQueue*) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
30: 0x102d27d64 v8::internal::MicrotaskQueue::RunMicrotasks(v8::internal::Isolate*) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
31: 0x102d28500 v8::internal::MicrotaskQueue::PerformCheckpoint(v8::Isolate*) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
32: 0x102968c64 node::InternalCallbackScope::Close() [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
33: 0x10296877c node::CallbackScope::~CallbackScope() [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
34: 0x102a0885c (anonymous namespace)::uvimpl::Work::AfterThreadPoolWork(int) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
35: 0x102a08c50 node::ThreadPoolWork::ScheduleWork()::'lambda'(uv_work_s*, int)::operator()(uv_work_s*, int) const [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
36: 0x10340639c uv__work_done [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
37: 0x103409dec uv__async_io [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
38: 0x10341bec4 uv__io_poll [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
39: 0x10340a3b0 uv_run [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
40: 0x102969754 node::SpinEventLoopInternal(node::Environment*) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
41: 0x102a79c6c node::NodeMainInstance::Run(node::ExitCode*, node::Environment*) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
42: 0x102a79a08 node::NodeMainInstance::Run() [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
43: 0x102a03718 node::Start(int, char**) [/Users/nitinbansal/.nvm/versions/node/v20.11.1/bin/node]
44: 0x1831da0e0 start [/usr/lib/dyld]

Test code
async function run7(connection) {
    const dataFilePaths = [
        '/Users/nitinbansal/Downloads/mix data/generated.csv',
        '/Users/nitinbansal/Downloads/hackernews_10k_dec 2024.csv'
    ];
    const tableNames = ['table_1', 'table_2'];

    await useConnection(DB_FILE_PATH, async (connection) => {
        await dropTables(connection, true);

        console.log(`Tables dropped. Loading data and getting count...`);

        await loadFile(connection, dataFilePaths[0], tableNames[0], 3);
        await loadFile(connection, dataFilePaths[1], tableNames[1], 3);

        let result = null;
        for(let k = 0; k < 200; k++) {
            console.log(`iteration: ${k+1}`);

            result = await runUsingStreamAndReadUntil(connection, `select * from table_1`, 2048);
            console.log(`selected row count ${result?.rows?.length}`)

            result = await runUsingStreamAndReadUntil(connection, `select * from table_2`, 2048);
            console.log(`selected row count ${result?.rows?.length}`)
        }
    });
}

async function useConnection(dbFilePath, callback) {
    let connection = null;
    try {
        connection = await initConnection(dbFilePath);
        return await callback(connection); // Return the result of the callback
    } catch (error) {
        console.error("Error while using connection:", error);
        throw error;
    } finally {
        if(connection) {
            connection.disconnect();
            // connection = null; // todo: prod: use close() if available
        }
    }
}

async function initConnection(dbFilePath) {
    const instance = await DuckDBInstance.create(dbFilePath);
    return await instance.connect();
}

async function runUsingStreamAndReadUntil(connection, query, maxRowsToFetch = 2048) {
    const result = await connection.streamAndReadUntil(query, maxRowsToFetch);

    const collectedRows = [];

    while(true) {
        const rows = result.getRows();
        if (!rows || rows.length === 0) {
            break;
        }

        console.log(`got ${rows.length} rows`);

        rows.forEach(row => collectedRows.push(row));
    }

    return collectedRows;
}

Source file details

hackernews_10k_dec 2024.csv: 10K rows, 4.1MB CSV
generated.csv: 9M rows, 1.03GB CSV

Also, from docs and PR both, it's not clear how to proceed and use result returned from streamAndReadUntil call. So, I tried chunk based extraction too (on top of rows based on shown above).. net result is same for both... crash

while(true) {
        const chunk = await result.fetchChunk();
        if (!chunk || chunk.rowCount === 0) {
            break;
        }

        for (const row of chunk.getRows()) {
            collectedRows.push(row);
        }
    }

@jraymakers  continuing with above...
I know it's heap related error, but, the same code using .stream() API works without throwing any error.

It looks like you're running into a 2 GB heap limit in your Node environment. Since one of your files is 1 GB and you're creating multiple DuckDB instances in a loop, it's not very surprising you'd hit this limit.
A few options/recommendations:

Don't create multiple DuckDB instances in a loop, especially if you're reading in large files. You'll likely end up with multiple copies of those large files in memory.
If you do want to create multiple instances (though I'm not sure why you would), then explicitly DETACH large databases/files when you're done. This, I think, is likely to free up the memory they're using. Note that disconnecting does not perform this cleanup.
Increase the memory limit of your Node process, using the --max-old-space-size command-line flag.

I agree the documentation of the various ways to run SQL and get results could be improved, but I suspect that isn't what's causing your problem here. If you think it is, please send the exact differences. The streamAndReadUntil helper is just a wrapper around calls to stream and fetchChunk, so it's not clear why it would be any different.

@jraymakers Thanks for the info. A few points:

I believe disconnect/close should immediately free used memory (or mark it as freed for OS to reclaim) since the programmer is making it explicitly clear that they are not going to use it again. This is pretty standard expectation of connections and associated resources.
The difference between stream and streamAndReadUntil is indeed there. Since switching over to stream api yesterday, not a single time heap crash has occured when testing for same files.
The problem with maintaining a single connection is that it forces global state in an application. I could wrap it in a global function, but yet another problem of that long lives connection is that it might have become stale (for whatever reason). This can be handled by reconnecting. But then it becomes a burden on some other part of the code. Another way would be issue a simple select 1 statement, but then this will have to be issued before every lease of the connection. This is where connection pooling helps since it can in background from time to time issue this test query.. But I implement that either here because of restriction of having just one connection. So, overall, im stuck.

I'm mostly from java background, so, im a bit biased to think that way, but, the current behaviour does seem odd where API is not doing fully what it's saying.
Attaching GIF recording of same showing how readUntil is crashing whereas stream is working fine while keeping everything else as-is.

Attaching video recording link instead. This shows without doubt the observed difference
https://drive.google.com/file/d/1HM3J0DKNvY8FhN3EaBiML_DvWZltrtCI/view?usp=sharing
Also, here's the code (I have packed all relevant methods in a single file for easier testing)
import {DuckDBInstance} from "@duckdb/node-api";
import fs from 'node:fs';

async function initConnection(dbFilePath) {
    const instance = await DuckDBInstance.create(dbFilePath);
    return await instance.connect();
}

async function runUsingStreamAndReadUntil(connection, query, maxRowsToFetch = 2048) {
    const result = await connection.streamAndReadUntil(query, maxRowsToFetch);

    const collectedRows = [];

    while(true) {
        const rows = result.getRows();
        if (!rows || rows.length === 0) {
            break;
        }

        for (const row of rows) {
            collectedRows.push(row);
        }
    }

    return collectedRows;
}

async function runUsingStream(connection, query, maxRowsToFetch = 2048) {
    const result = await connection.stream(query);

    const collectedRows = [];
    let rowsFetched = 0;

    while (maxRowsToFetch < 0 || (maxRowsToFetch > -1 && rowsFetched < maxRowsToFetch)) {
        const chunk = await result.fetchChunk();
        if (!chunk || chunk.rowCount === 0) {
            break;
        }

        for (const row of chunk.getRows()) {
            if (maxRowsToFetch > -1 && rowsFetched >= maxRowsToFetch) break;
            collectedRows.push(row);
            rowsFetched++;
        }
    }

    return collectedRows;
}

async function loadFile(connection, filepath, tableName) {
    let query = `CREATE TABLE ${tableName} AS SELECT * FROM read_csv_auto('${filepath}')`;
    await connection.run(query);
}

async function deleteFile(filePath) {
    try {
        await fs.unlinkSync(filePath);
    } catch (err) {
        if (err.code !== 'ENOENT') {
            throw err;
        } else {
            // ignore
        }
    }
}

async function main() {
    const DB_FILE_PATH = './db.bin';
    await deleteFile(DB_FILE_PATH);
    await new Promise(resolve => setTimeout(resolve, 1000));

    const dataFilePaths = [
        '/Users/nitinbansal/Downloads/mix data/generated.csv',
        '/Users/nitinbansal/Downloads/hackernews_10k_dec 2024.csv'
    ];
    const tableNames = ['table_1', 'table_2'];

    const connection = await initConnection();

    await loadFile(connection, dataFilePaths[0], tableNames[0]);
    await loadFile(connection, dataFilePaths[1], tableNames[1]);

    let rows = null;
    for(let k = 0; k < 200; k++) {
        console.log(`iteration: ${k+1}`);

        // runUsingStreamAndReadUntil
        // runUsingStream
        rows = await runUsingStreamAndReadUntil(connection, `select * from table_1`, 2048);
        console.log(`selected row count ${rows.length}`)

        rows = await runUsingStreamAndReadUntil(connection, `select * from table_2`, 2048);
        console.log(`selected row count ${rows.length}`)
    }
}

main().catch(console.error);

And here are the data files links (please uncompress larger file)

Hackernews: https://drive.google.com/file/d/1I-f5eDWVIOYonBXmSh-PdqgqJchijOOJ/view?usp=sharing
Generated: https://drive.google.com/file/d/1kGwu62p8YAjKmEdAxCLSsp4HMl9q1Z9o/view?usp=sharing

Let me know if any other details are needed.
Thanks..

Thanks for the detailed reproduction! I think I see what's going on now.
The problem is a misunderstanding about how the getRows method of DuckDBResultReader (returned by streamAndReadUntil) behaves. It returns all the rows that have been read so far. This is different from the fetchChunk method of DuckDBResult (returned by stream), which tries to fetch the next chunk, and will return null or a zero-length chunk if there are none left to fetch.
So, the while(true) loop in runUsingStreamAndReadUntil will never terminate; it will continue to get and append the same set of rows until memory runs out.
(Note that DuckDBResult also has a getRows method, which fully materializes the result and returns all rows, even if it's a streaming result.)
I apologize for not making this distinction clearer. Obviously the various methods could use more documentation. I'll move that up in my priority list.

Regarding your other comments about connections:
The way DuckDB connections behave is different from other databases. It may take some getting used to. While the lifetime of some state is tied to a connection, such as temporary objects, most in-memory state has a lifetime tied to the instance, not the connection. An instance can have more than one connection. Closing a connection does not affect the instance-level state.
Specifically, if you create a (non-temporary) table using one connection to an instance, and then close that connection and create a new connection to the same instance, that table will still be around. The way to remove a table from an instance is either to DROP it or DETACH its containing database.
Additionally, connections in DuckDB are not network connections. They're just objects in memory. They cannot go stale. There's no need for, or benefit from, issuing occasional queries (e.g. select 1) on a connection.
In my understanding, these are some recommended patterns of use for DuckDB:

Create a single instance per process. (Using multiple instances in the same process might be useful in some advanced scenarios; if this is done, special care must be taken if the same data files are attached to both, or corruption can occur.)
Create multiple connections to an instance if you want to run multiple queries in parallel, or if you want different scopes for temporary objects (or other per-connection state such as prepared statements, settings, and variables).
Keep connections and instances around as long as you'd like. No special action is needed to keep them usable.

(Also, some context: While I communicate frequently with the DuckDB team as part of my work, both as a member of the MotherDuck team and as an author of Node Neo, I didn't build DuckDB nor do I control its design. The above is what I've learned, not something I have any control over.)

@jraymakers  Thanks.. verified working without crashing with this:
const result = await connection.streamAndReadUntil(query, maxRowsToFetch);
return result.getRows().slice(0, maxRowsToFetch);

Regarding single connection, if I can rely on it not going stale (since it's just like any other variable), that makes things much simple.
And yes, docs do need an update.
Thanks for your time..

authors: freakynit jraymakers

repo: duckdb/duckdb-node-neo


text:
Migrate fragments to view binding

Task/Issue URL:https://github.com/duckduckgo/Android/issues/1478
Description
Migrated fragments to ViewBinding.
Does not include BrowserTabFragment because couldn't figure out how to migrate PopupMenu to view binding.

@cmonfortep Covered all fragments mentioned in https://github.com/duckduckgo/Android/issues/1478 except BrowserTabFragment. Hopefully that can be migrated as a separate PR as its a big file with a lot of bindings.

Hi @droid-it, we have another contribution that migrates some of the fragments on this PR (https://github.com/duckduckgo/Android/pull/1521). I wanted to know if you will be able to complete this contribution.

@cmonfortep sorry for the delayed response. I am on a vacation until next weekend. I'll close this PR to unblock.

authors: cmonfortep droid-it

repo: duckduckgo/Android


text:
Remove CPM experiment config

Please review the release process for BrowserServicesKit here.
Required:
Task/Issue URL: https://app.asana.com/0/1206594217714678/1207420864578039/f
iOS PR: https://github.com/duckduckgo/iOS/pull/3561
macOS PR: https://github.com/duckduckgo/macos-browser/pull/3534
What kind of version bump will this require?: Major/Minor/Patch
Optional:
Tech Design URL:
CC:
Description:

Steps to test this PR:
1.
1.

OS Testing:

[ ] iOS 14
[ ] iOS 15
[ ] iOS 16
[ ] macOS 10.15
[ ] macOS 11
[ ] macOS 12

Internal references:
Software Engineering Expectations
Technical Design Template

@federicocappelli not sure why the tests are failing, is it a known flakiness?

repo: duckduckgo/BrowserServicesKit


text:
Redesign "Use Duck Duck Go in Safari" screen

Task/Issue URL: https://app.asana.com/0/361428290920652/689645049709948
Tech Design URL: N/A
Description:

Update "Use DuckDuckGo in Safari" screen to better match new modal popup design
Remove disclosure indicators from modal popups in settings
Switch modals popups to cornflour blue

Steps to test this PR:

Go to Settings
Open "Use DuckDuckGo in Safari" and ensure it renders ok
Switch from portrait to landscape mode and again ensure it renders fine
Press the "Done" button to close it
Open "Use DuckDuckGo in Safari" again
Press the 'x' button to close it
Repeat on a few different devices

Internal references:
Software Engineering Expectations
Technical Design Template

Good call on the notification, I have updated that just to keep the dev branch in good condition. As for onboaridng screen that's an open question as it uses a different blue color to the others. Will investigate and let you know

repo: duckduckgo/iOS


text:
想請問~點擊 lock 顯示出來的文字 要怎樣做修改

非常好用謝謝!

解決!

repo: dueeeke/DKVideoPlayer


text:
arduino-cli lib install git-url

Great build tasks! There is issue with the current implementation with the lib install task, in that the command formed for library install using git is using
--git-url="...."
but should be
--git-url "..."
the equal sign breaks the command. Easy fix though in lib_install/lib_installV1/index.js, line 41. The same is probably true for the zip option (line 50) even though I did not try that.
I am not sure whether this equal sign vs. space issue also screws up the other command line options. According to the docs, space should be used to separate args from key....
Related: library install (and zip install) will only work if unsafe installs are enabled. This has to be done prior to calling the action. There are really two ways to accomplish that, adding a arduino-cli.yaml file to the repo with the appropriate keys or creating the file on the build agent on the fly using bash and
$(RUNNER_TOOLSDIRECTORY)/arduino-cli/$(arduino-cli-version)/Linux_64bit/arduino-cli config init
$(RUNNER_TOOLSDIRECTORY)/arduino-cli/$(arduino-cli-version)/Linux_64bit/arduino-cli config set library.enable_unsafe_install true
$(RUNNER_TOOLSDIRECTORY)/arduino-cli/$(arduino-cli-version)/Linux_64bit/arduino-cli config set board_manager.additional_urls ....
          ... # other config keys

I do the latter, because I do not really want the config file in the repo, as for desk side development I use VSCode with full arduino. It be great to add an action for creating the config file and adding the key/value. There are only like a handful of keys or so :)

Thank you! Great input for a new release.

authors: duglah thor-schueler

repo: duglah/azuredevops-arduino-cli-task


text:
Firebase rules and configuration

Hi! I really appreciated your job.
Can you please provide also the Firebase configurations (rules, etc) so that it's possible to create an exact clone?
Thank you

@facconi Did you ever figure this out?

Unfortunately not, but I have stopped using Firebase.


text:
rust: add cargo and rustfmt?

Having a clean nixos system (no cargo etc installed) and opening a rust file in a rust project, rust-analyzer complains that it doesn't find cargo. Does it make sense to add it to the nix-shell? Same for rustfmt, which allows the lsp to format the buffer.
https://github.com/dundalek/lazy-lsp.nvim/blob/87305e02c8f167709598eea4b76542540d8366ff/lua/lazy-lsp/servers.lua#L234
I wanted to test this locally, but didn't get it working. I'm using lazy.nvim and edited the servers.lua file in the installed plugin, but didn't see any effect. Any hints?

Another idea:
Many rust projects store their rust version and tooling in a rust-toolchain.toml. This can be read automatically by nix and produce a nix-shell with the right rust version and tooling installed: https://github.com/oxalica/rust-overlay
Not sure how practical that is in the scope of this nvim plugin.

Thanks for the report. I pushed a fix https://github.com/dundalek/lazy-lsp.nvim/commit/e7638bd348c09c27f7fbcba758a6cd8aeca4ce37, please give it a try.
There were actually two things going on:

The default lspconfig uses cargo to workspace path which fails even before nix-shell wrapper even runs. I added fallback with a simpler detection.
Added cargo and rustfmt dependencies to make sure things works properly once rust-analyzer starts.

Another idea: Many rust projects store their rust version and tooling in a rust-toolchain.toml. This can be read automatically by nix and produce a nix-shell with the right rust version and tooling installed: https://github.com/oxalica/rust-overlay
Not sure how practical that is in the scope of this nvim plugin.

Yeah, I am thinking if one uses special tooling that should be configured in user config.

Thanks for the report. I pushed a fix e7638bd, please give it a try.

Works very well, thank you!

Sweet, always great to get confirmation that it works on another computer, thanks!

authors: dundalek fdietze


text:
[Project structure] split g4 and some other antlr file into other source folder

I thinks the project structure like above more beautiful :D

done


text:
Can't use Frankenphp as a golang library

What happened?
I grabbed an example of using Franken as a go library
package main

import (
	"log"
	"net/http"

	"github.com/dunglas/frankenphp"
)

func main() {
	if err := frankenphp.Init(); err != nil {
		panic(err)
	}
	defer frankenphp.Shutdown()

	http.HandleFunc("/", func(w http.ResponseWriter, r *http.Request) {
		req, err := frankenphp.NewRequestWithContext(r, frankenphp.WithRequestDocumentRoot("/var/www/html", false))
		if err != nil {
			panic(err)
		}

		if err := frankenphp.ServeHTTP(w, req); err != nil {
			panic(err)
		}
	})
	log.Fatal(http.ListenAndServe(":8080", nil))

> go mod init example
> go mod tidy
...

❯ go build .
# github.com/dunglas/frankenphp/internal/watcher
watcher.c:5:10: fatal error: wtr/watcher-c.h: No such file or directory
    5 | #include "wtr/watcher-c.h"
      |          ^~~~~~~~~~~~~~~~~
compilation terminated.

I thought perhaps watcher-c.h is in the php code?  but I can't locate it.   Where does that file come from?
Build Type
Custom (tell us more in the description)
Worker Mode
Yes
Operating System
GNU/Linux
CPU Architecture
x86_64
PHP configuration
See above

Relevant log output
As above

Ok I seems I need to install watcher.
Why I went wrong here is because the documentation says
Install "Optional" Dependencies.
Since the title says "Optional".  I skipped it.   But it's not optional unless you turn those features off.  That's unclear.  I suggest you amend the documentation.
Unfortunately, watcher has really bad documentation on how to actually install it and make it available.
According to the watcher docs, I followed the steps and watcher is still not found.   And it's been way too many years that I have forgotten how the C compiler finds files.   From memory, INCLUDE environment vars or something.  But there simply are no examples to get this off the ground with golang.  It's a horrible onboarding experience.
Looks like I can copy the watcher includes to:
sudo cp -rp watcher-next/include/wtr /usr/local/include/
So this problem is resolved but now I get
❯ go build .
# github.com/dunglas/frankenphp
../go/pkg/mod/github.com/dunglas/frankenphp@v1.3.0/cgi.go:3:11: fatal error: php_variables.h: No such file or directory
    3 | // #include <php_variables.h>
      |           ^~~~~~~~~~~~~~~~~
compilation terminated.

Next battle.   That's a file in the php sources.

To build without the watcher you can do something like this in the meantime I think:
go build -tags nowatcher

I got a little further.
❯ CGO_CFLAGS="-I/home/hookenz/franken/php-8.2.25/main -I/home/hookenz/franken/php-8.2.25/Zend -I/home/hookenz/franken/php-8.2.25 -I /home/hookenz/franken/php-8.2.25/TSRM" go build .
# frank
/snap/go/10743/pkg/tool/linux_amd64/link: running gcc failed: exit status 1
/usr/bin/gcc -m64 -o $WORK/b001/exe/a.out -Wl,--export-dynamic-symbol=_cgo_panic -Wl,--export-dynamic-symbol=_cgo_topofstack -Wl,--export-dynamic-symbol=crosscall2 -Wl,--export-dynamic-symbol=go_apache_request_headers -Wl,--export-dynamic-symbol=go_frankenphp_finish_request -Wl,--export-dynamic-symbol=go_frankenphp_release_known_variable_keys -Wl,--export-dynamic-symbol=go_frankenphp_worker_handle_request_start -Wl,--export-dynamic-symbol=go_getenv -Wl,--export-dynamic-symbol=go_getfullenv -Wl,--export-dynamic-symbol=go_handle_file_watcher_event -Wl,--export-dynamic-symbol=go_handle_request -Wl,--export-dynamic-symbol=go_log -Wl,--export-dynamic-symbol=go_putenv -Wl,--export-dynamic-symbol=go_read_cookies -Wl,--export-dynamic-symbol=go_read_post -Wl,--export-dynamic-symbol=go_register_variables -Wl,--export-dynamic-symbol=go_sapi_flush -Wl,--export-dynamic-symbol=go_sapi_getenv -Wl,--export-dynamic-symbol=go_shutdown -Wl,--export-dynamic-symbol=go_ub_write -Wl,--export-dynamic-symbol=go_write_headers -Wl,--compress-debug-sections=zlib /tmp/go-link-124271613/go.o /tmp/go-link-124271613/000000.o /tmp/go-link-124271613/000001.o /tmp/go-link-124271613/000002.o /tmp/go-link-124271613/000003.o /tmp/go-link-124271613/000004.o /tmp/go-link-124271613/000005.o /tmp/go-link-124271613/000006.o /tmp/go-link-124271613/000007.o /tmp/go-link-124271613/000008.o /tmp/go-link-124271613/000009.o /tmp/go-link-124271613/000010.o /tmp/go-link-124271613/000011.o /tmp/go-link-124271613/000012.o /tmp/go-link-124271613/000013.o /tmp/go-link-124271613/000014.o /tmp/go-link-124271613/000015.o /tmp/go-link-124271613/000016.o /tmp/go-link-124271613/000017.o /tmp/go-link-124271613/000018.o /tmp/go-link-124271613/000019.o /tmp/go-link-124271613/000020.o /tmp/go-link-124271613/000021.o /tmp/go-link-124271613/000022.o /tmp/go-link-124271613/000023.o /tmp/go-link-124271613/000024.o /tmp/go-link-124271613/000025.o /tmp/go-link-124271613/000026.o /tmp/go-link-124271613/000027.o /tmp/go-link-124271613/000028.o /tmp/go-link-124271613/000029.o /tmp/go-link-124271613/000030.o /tmp/go-link-124271613/000031.o -O2 -g -lresolv -L/usr/local/lib -L/usr/lib -lphp -ldl -lm -lutil -O2 -g -lresolv -O2 -g -lpthread -O2 -g -lwatcher-c -lstdc++ -O2 -g -no-pie
/usr/bin/ld: cannot find -lphp: No such file or directory
collect2: error: ld returned 1 exit status

Ok, I've solved it.   I need to "install" the PHP binary and all it's development files with
sudo make install.
Then simply run go build .
I also needed to copy the watcher
sudo cp -rp watcher-next/include/wtr /usr/local/include/
It would be awesome if the docs could be updated with more full and clear examples.  But I think I got there in the end.

Don't hesitate to send docs PR, improving this is on my (long) todo list, but any help is welcome.

Ok, I've solved it. I need to "install" the PHP binary and all it's development files with.

Don't forget it needs to be a ZTS build @hookenz!

authors: AlliBalliBaba dunglas hookenz withinboredom

repo: dunglas/frankenphp


text:
Adding better support for AWS China and GovCloud

Adds support for using the correct default region in more places throughout the code. Tested as working in GovCloud, but no data on AWS China.

 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.

authors: CLAassistant benjamin-jones

repo: duo-labs/cloudmapper


text:
DUPLO-25418 Fixed name validation issue

Overview
Added name validator
Summary of changes
Added validation to storage account sub resources duplocloud_azure_storageclass_blob, duplocloud_azure_storageclass_queue, duplocloud_azure_storageclass_table
This PR does the following:

Added validation at schema level
...

Testing performed

[ ] Using unit tests
[ ] Manually, on my local system
[✓ ] Manually, on a remote test system

Describe any breaking changes

...

Task linked: DUPLO-25418 TF provider: Storage class 'name' should not allow more than 63 characters for resource duplocloud_azure_storageclass_blob also there should be a validation error for invalid name.

authors: Duplo-nikv zafarabbas


text:
Cloudflare R2 put_object_stream fails every time with SignatureDoesNotMatch error

Describe the bug
When using the library with Cloudflare R2, the put_object_stream function fails every time with an 8MB file of all 0s (and any other large file I tired).  It gives the following error:
Error: Got HTTP 403 with content '<?xml version="1.0" encoding="UTF-8"?><Error>SignatureDoesNotMatch<Message>The request signature we calculated does not match the signature you provided. Check your secret access key and signing method. </Message></Error>'

To Reproduce
use s3::{creds::Credentials, Bucket, Region};
use std::error::Error;

#[tokio::main]
async fn main() -> Result<(), Box<dyn Error>> {
    let account_id = std::env::var("S3_ID")?;
    let access_key_id = std::env::var("S3_ACCESS_KEY_ID")?;
    let secret_access_key = std::env::var("S3_SECRET_ACCESS_KEY")?;
    let credentials = Credentials::new(
        Some(&access_key_id),
        Some(&secret_access_key),
        None,
        None,
        None,
    )?;
    let bucket =
        Bucket::new("multipart-testing", Region::R2 { account_id }, credentials)?.with_path_style();

    // empty_file_8_mb = `dd if=/dev/zero of=empty_file_8_mb bs=1 count=8388608`
    let mut file = tokio::fs::File::open("empty_file_8_mb").await?;
    let result = bucket
        .put_object_stream(&mut file, "empty_file_8_mb")
        .await?;

    dbg!(&result);

    Ok(())
}

Expected behavior
I expect it to upload the file with no errors.
Environment

cargo 1.70.0 (ec8a8a0ca 2023-04-25)
rust-s3 = "0.33.0"

Additional Context
I tried various combinations of nightly, the 0.34.0-beta3 beta, using slashes, not using slashes, always the same result.
I also tried using put_multipart_chunk manually from an existing multipart upload to ensure it was not a concurrency issue and I still get the same error.
Uploading the entire object from memory with put_object works fine.
Test file created with dd if=/dev/zero of=empty_file_8_mb bs=1 count=8388608

Added a test for streaming big objects into R2 which is currently passing

authors: Sharpiro durch


text:
🛑 Mention Test Bot is down

In e5b89d2, Mention Test Bot (https://backend.isbotdown.com/bots/mentiontestbot) was down:

HTTP code: 200
Response time: 116 ms

Resolved: Mention Test Bot is back up in 04a6929.

repo: durof/status


text:
🛑 Mention Robot is down

In 4455791, Mention Robot (https://backend.isbotdown.com/bots/mentionrobot) was down:

HTTP code: 200
Response time: 90 ms

Resolved: Mention Robot is back up in 43d7df3.


text:
🛑 Mention Bot is down

In 1235f86, Mention Bot (https://backend.isbotdown.com/bots/mentionbot) was down:

HTTP code: 200
Response time: 155 ms

Resolved: Mention Bot is back up in 3be8a47 after 2 hours, 55 minutes.


text:
Implement handlers for wire messages Mempool and Pong

level=debug msg="received message with unknown topic" topic=mempool
level=debug msg="received message with unknown topic" topic=pong

Unknown topics reported for two wire messages multiple times on testnet

Fixed via #872

Fixed via #872

repo: dusk-network/dusk-blockchain


text:
[Ahuna II] [Nice-to-have] Search by editor

As per bottom of the framing issue

Full figma (source of truth)
Full issue
Even clearer specs via the prototype

Not prio imho. Closing.

repo: dust-tt/dust


text:
Shortened skill orders don't have full accuracy for first 3-4 skills

A skill order like http://champion.gg/champion/Quinn is shortened to "Q>E>W" in both the Most Frequent and Highest Win Rate builds. But one starts with "QEWQ" and the other starts with "QEQW". I recommend you separate the first 3-4 skills and list them in addition to the shortened order.
I mentioned this in the reddit thread but I'm not sure if I was clear about what I meant. See https://github.com/mpcomplete/lisa/blob/master/lib/js/outputFormat.js#L84 for the way I handle this (there's no need to sort the skills - you know if a skill is finished by level 9 that it is maxed, and by level 13 that it is second-to-max.)

Trinkets | Q.E.W.Q - Q>E>W ?

Perfect!

authors: dustinblackman mpcomplete

repo: dustinblackman/Championify


text:
Run re-trained model without the tensorRT optimization

Hello,
I'm having problem after re-train because I'm using the Drive OS release and the version of the TensorRT is 5.1.4. As I saw in different posts, I need the version 7 to run the re-trained model in .onnx format using the detectnet.
Do you know how can a run my re-trained model without the optimization of the TensorRT? or... maybe you can help me with the error:
--- End node ---
ERROR: ModelImporter.cpp:288 In function importModel:
[5] Assertion failed: tensors.count(input_name)
[TRT]    failed to parse ONNX model 'models/fruit/ssd-mobilenet.onnx'
[TRT]    device GPU, failed to load models/fruit/ssd-mobilenet.onnx
[TRT]    detectNet -- failed to initialize.
detectnet:  failed to load detectNet model
Thanks in advance

Hi @DanielYG, I don't know of a way to make it work in older TensorRT, so the fallback would be to run it in PyTorch.  You could just use the pytorch_ssd code to run it in PyTorch.  You wouldn't need to use jetson-inference / detectNet for that, since I only support TensorRT in jetson-inference.

authors: DanielYG dusty-nv

repo: dusty-nv/jetson-inference


text:
(un)Tar of models fails in cmake. Models have Windows special charaters in their names.

After the models are downloaded in the CMake process, for example as "Super-Resolution-BSD500.tar.gz", the tar step fails:
XXXXXX - ´ Super-Resolution-BSD500.tar.gz\r´ saved [640910/640910]

tar (child): Super-Resolution-BSD500.tar.gz: Cannot open: No such file or directory.
tar (child): Error is not recoverable: exiting now
tar: Child returned status 2

Seems that somehow the filename got encoded with Windows line endings '\r'.
Other examples are:
'multiped-500.tar.gz'$'\r'
'facenet-120.tar.g'$'\r'
'googlenet_noprob.prototxt'$'\r'
Not terrible, since the cmake and make scripts dont' really fail and you can actually extract those files with the correct name and move them to the networks directory, but is still an error.

Hmm, I just checked the file from Ubuntu with cat -e CMakePreBuild.sh, and it only showed Unix line-endings ($) and not Windows line endings (^M$).
Maybe it was a temporary glitch from a previous version of the file.  Can you try cloning from master again and seeing if the issue persists?

Apologies, I think I didn't explain myself correctly. The issue wasn't with the CMake script, but with the name of the models stored at public.boxcloud.com which are downloaded. Those were the ones with windows special characters in their name, so the tar steps at the CMakePreBuild.sh failed.
In any case, just tested it again and the issue is gone. I guess someone updated them?  I'll close the issue now.
Thanks Dusty

authors: Rmwak dusty-nv


text:
Syntax highlighting added ✨

:shipit: and profit!

Thanks

authors: duythien rmariuzzo

repo: duythien/cashier


text:
Add pegasus_draw() and draft Pegasus intro

Thanks, @boothby!

Codecov Report

Merging #94 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #94   +/-   ##
=======================================
  Coverage   80.58%   80.58%
=======================================
  Files          26       26
  Lines        1339     1339
=======================================
  Hits         1079     1079
  Misses        260      260

Impacted Files
Coverage Δ

dwave_networkx/generators/pegasus.py
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

authors: JoelPasvolsky codecov-io


text:
Drop handler: Drop file in the directory

Hi,
I can't handle explorer drop event when I drop file not on the another file but in the empty place. When I drop file on another file that works correct.
My code is:
    [ComVisible(true)]
    [COMServerAssociation(AssociationType.AllFiles)]
    public class CustomDropHandler : SharpDropHandler
    {
        protected override void DragEnter(DragEventArgs dragEventArgs)
        {
            dragEventArgs.Effect = DragDropEffects.Move;
        }

        protected override void Drop(DragEventArgs dragEventArgs)
        {
            string message = "";
            foreach (string path in DragItems)
            {
                message += path;
            }
            MessageBox.Show(message);
        }
    }

Having the same issue I think.  I cannot use a folder as the drop target (either by dragging over a folder or the blank space).  Dragging over files displays the "link" annotation and show the correct events firing in the debugger, but not folders.  I have tried the following annotations:
[COMServerAssociation(AssociationType.Class, @"Directory\Background")]
[COMServerAssociation(AssociationType.AllFiles)]
[COMServerAssociation(AssociationType.UnknownFiles)]
[COMServerAssociation(AssociationType.Directory)]
I am starting to think folders are not supported as drop targets by this version of windows (Windows 7).

Is there any fix on Drag and Drop into folder or Drive?

      dmwkerr,

Thank you for you quick answer.
I tried to use AssociationType.Directory, but that doesn't work too.

@MadMaxidrom: Have you got the solution for the drop to folder?

Try:
[COMServerAssociation(AssociationType.DirectoryBackground)]

authors: MadMaxidrom SimplexAuctor SivaprasadTamatam dwmkerr

repo: dwmkerr/sharpshell


text:
chore(plugin-space): Save reference to a directory on a plugin level

Motivation / Background

This Pull Request ...
Details

Additional information

Current dependencies on/for this PR:

main

PR #5336   👈

This stack of pull requests is managed by Graphite.

repo: dxos/dxos


text:
Update CHANGELOG for Release 45

Task list should show period_date
RUN ON PRODUCTION: remove submissions without task and associated models
RM858 and RM3710 changes to DW export
Reinstate validations on remaining October frameworks (2/2)
Users should be soft deleted
Remove Docker configuration for local development
Add developer strategy to API/omniauth so that development doesn't require Google auth setup per-developer

It's missing the diff URL at the bottom.

authors: james lozette

repo: dxw/DataSubmissionServiceAPI


text:
Function to calculate derivative of vis map

Similar to calcVisMap, but that calculates the visibility rate.

Re-opening. There was a bug in the satvis repo that was fixed that makes refactoring required here.

See #107 for some of the fixes


text:
support for Nobara

Description
I'd like to see support for this fedorda based distro Nobara.
tried to do it myself but messed up user file in root dir and had to reinstall to fix yeah i didn't make a back up first.

[x] Does this issue still occur in the master branch? (Required if issue)
would like assii logo

Neofetch version
neofetch-7.1.0-7.fc36.noarch
Screenshot
Config file
default now as my edited one screwed the pooch.
Verbose log

Run neofetch -vv 2> neofetchlog
Upload the contents of neofetchlog to pastebin, gist or equivalent.

_
logo-wip2-web.txt
is a copy of logo i made in gimp from site logo  needs to be fixed though not perfect.

repo: dylanaraps/neofetch


text:
Add ravynOS platform

Description
ravynOS (GitHub) is a new BSD-based OS that aims to be compatible with macOS at an (application) source and binary level. It is sort of a hybrid between Darwin and FreeBSD.
This PR adds the ravynOS platform detection logic and ASCII logo.
Features
Issues
TODO

Awesome, thank you!


text:
Add eslint-plugin-* and @types/* to ignore matches list for unused

removes false positives for eslint plugins and typescript types

Note fixes #202

<3

authors: amilajack corbinu

repo: dylang/npm-check


text:
[Snyk] Fix for 1 vulnerabilities

Snyk has created this PR to fix 1 vulnerabilities in the yarn dependencies of this project.
Snyk changed the following file(s):

plugins/app-backend/package.json

Note for zero-installs users
If you are using the Yarn feature zero-installs that was introduced in Yarn V2, note that this PR does not update the .yarn/cache/ directory meaning this code cannot be pulled and immediately developed on as one would expect for a zero-install project - you will need to run yarn to update the contents of the ./yarn/cache directory.
If you are not using zero-install you can ignore this as your flow should likely be unchanged.

⚠ Warning
Failed to update the yarn.lock, please update manually before merging.

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Directory Traversal SNYK-JS-SOURCEMAPSUPPORT-6112477
  170

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Directory Traversal

:tada: Snyk checks have passed. No issues have been found so far.
:white_check_mark: security/snyk check is complete. No issues have been found. (View Details)
:white_check_mark: license/snyk check is complete. No issues have been found. (View Details)

repo: dylansnyk/backstage


text:
[Snyk] Fix for 1 vulnerabilities

Snyk has created this PR to fix 1 vulnerabilities in the yarn dependencies of this project.
Snyk changed the following file(s):

plugins/search-backend-module-techdocs/package.json

Note for zero-installs users
If you are using the Yarn feature zero-installs that was introduced in Yarn V2, note that this PR does not update the .yarn/cache/ directory meaning this code cannot be pulled and immediately developed on as one would expect for a zero-install project - you will need to run yarn to update the contents of the ./yarn/cache directory.
If you are not using zero-install you can ignore this as your flow should likely be unchanged.

⚠ Warning
Failed to update the yarn.lock, please update manually before merging.

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Improper Input Validation SNYK-JS-NANOID-8492085
  44

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Improper Input Validation

:tada: Snyk checks have passed. No issues have been found so far.
:white_check_mark: security/snyk check is complete. No issues have been found. (View Details)


text:
Add progress wheel when loading results

When 10 or more pages are retrieved, it can take more than a few seconds to load the page.

Use the following progress spinner code from loading.io/css.
HTML

CSS
.lds-spinner,
.lds-spinner div,
.lds-spinner div:after {
  box-sizing: border-box;
}
.lds-spinner {
  color: currentColor;
  display: inline-block;
  position: relative;
  width: 80px;
  height: 80px;
}
.lds-spinner div {
  transform-origin: 40px 40px;
  animation: lds-spinner 1.2s linear infinite;
}
.lds-spinner div:after {
  content: " ";
  display: block;
  position: absolute;
  top: 3.2px;
  left: 36.8px;
  width: 6.4px;
  height: 17.6px;
  border-radius: 20%;
  background: currentColor;
}
.lds-spinner div:nth-child(1) {
  transform: rotate(0deg);
  animation-delay: -1.1s;
}
.lds-spinner div:nth-child(2) {
  transform: rotate(30deg);
  animation-delay: -1s;
}
.lds-spinner div:nth-child(3) {
  transform: rotate(60deg);
  animation-delay: -0.9s;
}
.lds-spinner div:nth-child(4) {
  transform: rotate(90deg);
  animation-delay: -0.8s;
}
.lds-spinner div:nth-child(5) {
  transform: rotate(120deg);
  animation-delay: -0.7s;
}
.lds-spinner div:nth-child(6) {
  transform: rotate(150deg);
  animation-delay: -0.6s;
}
.lds-spinner div:nth-child(7) {
  transform: rotate(180deg);
  animation-delay: -0.5s;
}
.lds-spinner div:nth-child(8) {
  transform: rotate(210deg);
  animation-delay: -0.4s;
}
.lds-spinner div:nth-child(9) {
  transform: rotate(240deg);
  animation-delay: -0.3s;
}
.lds-spinner div:nth-child(10) {
  transform: rotate(270deg);
  animation-delay: -0.2s;
}
.lds-spinner div:nth-child(11) {
  transform: rotate(300deg);
  animation-delay: -0.1s;
}
.lds-spinner div:nth-child(12) {
  transform: rotate(330deg);
  animation-delay: 0s;
}
@keyframes lds-spinner {
  0% {
    opacity: 1;
  }
  100% {
    opacity: 0;
  }
}

Need to review how to load the spinner while the page is still loading. Currently, the API call happens before the rest of the page is loaded.
May need to convert that section into a function and then call the function at the bottom of the page after everything else has loaded.

May consider using Ajax fetch() to call query string parameter on same page, and then add PHP isset($_GET[]) to run the code that will build the HTML grid of actors.

repo: dynamiccookies/radarr-actor-import


text:
Remove dependency on grunt

Given that there are very little reasons to keep using Grunt in modern Node.js ecosystem, consuming used libraries (such as JSHint) directly through npm might be more maintainable than using grunt with plugins.

Closing this due to no recent activity. If this is still a problem or you have questions please feel free to comment again and we can try to help further.

I'd totally be willing to review a PR for this if anyone wants to submit one!

Sorry, was a bit hectic on my side due to switching job and apartment. Will try to get to it this week.

@kibertoad No problem! Looking forward to it!!

authors: fishcharlie kibertoad

repo: dynamoosejs/dynamoose


text:
担当者にならなかったメールを送信する

担当者割り当てを初めて行ったとき（ツアー状態コードが割り当て前 → 割り当てになるタイミング）のみ送信。
担当者を外す処理の仕様を考えてから実装。

担当にならなかったメール条件

初めて担当者を割り当てた際に、割当たらなかったガイドのみ送信

担当者リセットをすると条件の判定ができなくなってしまう。
テーブルにフラグを足すのは何か違う気がする。

担当リセットAPI、機能を削除するべき？そうすれば状態コードが前に戻ることが無くなる。
再決定は可能。

#231 とまとめます。


text:
How to get the API key from Defastra?

I tried but couldn't get one.
Do you have clear instructions?

Bumping this. Can not find where to get the key. Even willing to purchase a key yet it their site only has a documentation page and no actual main domain for signing up to buy access to their api... Is this project defunct now??


text:
Fullscreen Search: New pattern for mobile web

More details to follow...

We can close this ticket if we feel like this is more of an adaptive pattern that responsive.

repo: eBay/mindpatterns


text:
[20334] Configure the Log through YAML

Merge after:

https://github.com/eProsima/Fast-DDS/pull/4292
https://github.com/eProsima/dev-utils/pull/92
https://github.com/eProsima/DDS-Pipe/pull/86

Codecov Report
Attention: Patch coverage is 8.82353% with 93 lines in your changes are missing coverage. Please review.

Project coverage is 19.89%. Comparing base (ea973cb) to head (1a0dd10).
Report is 2 commits behind head on main.

:exclamation: Current head 1a0dd10 differs from pull request most recent head 4c6eb0f. Consider uploading reports for the commit 4c6eb0f to get more accurate results

Files
Patch %
Lines

ddsrecorder/src/cpp/main.cpp
0.00%
22 Missing :warning:

ddsreplayer/src/cpp/main.cpp
0.00%
20 Missing :warning:

...src/cpp/user_interface/arguments_configuration.cpp
0.00%
12 Missing :warning:

...src/cpp/user_interface/arguments_configuration.cpp
0.00%
12 Missing :warning:

..._yaml/src/cpp/recorder/YamlReaderConfiguration.cpp
27.27%
3 Missing and 5 partials :warning:

..._yaml/src/cpp/replayer/YamlReaderConfiguration.cpp
27.27%
0 Missing and 8 partials :warning:

..._yaml/src/cpp/replayer/CommandlineArgsReplayer.cpp
14.28%
2 Missing and 4 partials :warning:

..._yaml/src/cpp/recorder/CommandlineArgsRecorder.cpp
28.57%
2 Missing and 3 partials :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #115      +/-   ##
==========================================
+ Coverage   19.24%   19.89%   +0.65%
==========================================
  Files          32       34       +2
  Lines        2832     2870      +38
  Branches     1501     1522      +21
==========================================
+ Hits          545      571      +26
+ Misses       1802     1780      -22
- Partials      485      519      +34

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

repo: eProsima/DDS-Record-Replay


text:
fastrtps1.5.0 participant discovery problem

Software ENV: Ubuntu 16.04
fastrtps version: 1.5.0
Resource: source code
Hardware ENV: two hosts located in a LAN with IP address as, e.g., A(192.168.1.6), B(192.168.1.7)
Problem description:
Case 1: When I run the simple talker/listener case, e.g., talker(publisher) on A, listener(subscriber) on B, it works well.
Case 2: The case is more complex. On host A, start a participant which pub some topic, while on the same host A, there is two subscribers subscribing this topic. On B, start a participant which has a subscriber subscribing the same topic. The result is: the subscriber of the participant on host B is not discovered by participant on host A; but the two subscribers of host A have been discovered. In other words, when there is more subscribers with the same topic subscribed located on different hosts, the ones NOT located with same host as the publisher is not discovered correctly.
Any help is appropriated, thanks in advance!

Hi @Tony301X
I just tested with our HelloWorldExample and everything worked fine.

As you can see, the upper pane is connected to a machine named ubuntu59 while the rest of panes are local. I launched a subscriber in ubuntu59 (like in your host B), and two subscribers and a publisher on my machine (acting as host A).
Can you provide us more information to reproduce the issue like each instance configuration, the exact command used and the launching order?
Thank you very much.

As we are unable to reproduce the issue and no more information was provided, I will close the issue.

repo: eProsima/Fast-RTPS


text:
Broken link in example pcrglob run

Under the header Load forcing data, hyperlink to "preprocessing examples" is broken.
It should point to https://ewatercycle.readthedocs.io/en/latest/user_guide/02_forcing.html

already fixed by the merging of the cleanup Rolf branch.

authors: BSchilperoort RolfHut


text:
Display related pull requests in an issue's detail view

Description
This pull request add a list of related pull requests the detail view of an issue. The result looks like this:

Related pull requests is read-only and the section is completely hidden if there is not related PRs.
Fixes # 40
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Tests have been added on the IssueController to validate that the section is displayed correctly.
Test Configuration:

Firmware version: ruby 2.4.5p335 (2018-10-18 revision 65137) [x86_64-darwin18]
Hardware: macOS 10.14.5 Mojave
Toolchain: Redmine 3.4.5.stable
SDK: Rails 4.2.8

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Fix #40


text:
Metrics are not following a same principle

In the calculation of metrics like request count, status code count, min duration, max duration, all data are weighted equally.
In the calculation of metrics like m1, m5, the moving average algorithm is used, which gives more weight to recent data.
Metrics like duration percentiles are actually only using the recent data for calculation.
These metrics should follow the same principle to weight data by default, exceptional cases should be handled carefully.

This difference was explained and not wrong.

repo: easegress-io/easegress