text:
Modal with templateUrl - 'items' undefined

Ionic version:  (check one with "x")
[ ] 1.x
[ ] 2.x
[ x] 3.x
I'm submitting a ...  (check one with "x")
[ x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
Receive an undefined error when presenting a modal that has its own .ts file and .html file as templateUrl
Expected behavior:
Modal presenting without any error
Steps to reproduce:
For Ionic 2 issues - http://plnkr.co/edit/te7JzQj5mxbIIrfSTo62?p=preview
I cannot reproduce the error in the forked Plunker.
Related code:
// new-location-modal.ts
import { Component } from '@angular/core';
import { Platform, NavParams, ViewController } from 'ionic-angular';

@Component({
  templateUrl: "new-location-modal.html"
})
export class NewLocationModalPage {

  constructor(public platform: Platform, public viewCtrl: ViewController) {
    console.log('NewLocationModal#constructor');
  }

  dismiss() {
    this.viewCtrl.dismiss();
  }
}

// new-location-modal.html
<ion-header>
  <ion-toolbar>
    <ion-title>
     Title
    </ion-title>
    </ion-toolbar>
</ion-header>
<ion-content>
  Content
</ion-content>

// new-event.ts
import { Component } from '@angular/core';
import { FormBuilder, FormGroup, Validators } from '@angular/forms';
import { ModalController, NavController, NavParams } from 'ionic-angular';
import { Platform, ViewController } from 'ionic-angular';

import { NewLocationModalPage } from '../../modals/new-location/new-location-modal';

@Component({
  selector: 'page-new-event',
  templateUrl: 'new-event.html'
})
export class NewEventPage {
  newEventForm: FormGroup;

  timeStarts: string = '';

  constructor(public modalCtrl: ModalController, public formBuilder: FormBuilder) {
    // Creating the newEventForm and specifying controls
    this.newEventForm = this.formBuilder.group({
      title: ['', Validators.required],
      sport: ['', Validators.required],
      timeStarts: ['', Validators.required],
      location: ['', Validators.required]
    });
  }

  ionViewDidLoad() {
    console.log('ionViewDidLoad NewEventPage');
  }

  presentNewLocationModal() {
    let modal = this.modalCtrl.create(NewLocationModalPage);
    modal.present();
  }
}

Other information:
EXCEPTION: Error in ./NewLocationModal class NewLocationModal - inline template:22:14 caused by: Cannot read property 'items' of undefined

TypeError: Cannot read property 'items' of undefined
    at CompiledTemplate.proxyViewClass.View_NewLocationModal0.detectChangesInternal (/AppModule/NewLocationModal/component.ngfactory.js:363:46)
    at CompiledTemplate.proxyViewClass.AppView.detectChanges (http://localhost:8100/build/main.js:117531:14)
    at CompiledTemplate.proxyViewClass.DebugAppView.detectChanges (http://localhost:8100/build/main.js:117726:44)
    at CompiledTemplate.proxyViewClass.AppView.internalDetectChanges (http://localhost:8100/build/main.js:117516:18)
    at CompiledTemplate.proxyViewClass.View_NewLocationModal_Host0.detectChangesInternal (/AppModule/NewLocationModal/host.ngfactory.js:29:19)
    at CompiledTemplate.proxyViewClass.AppView.detectChanges (http://localhost:8100/build/main.js:117531:14)
    at CompiledTemplate.proxyViewClass.DebugAppView.detectChanges (http://localhost:8100/build/main.js:117726:44)
    at ViewContainer.detectChangesInNestedViews (http://localhost:8100/build/main.js:117863:37)
    at CompiledTemplate.proxyViewClass.View_ModalCmp0.detectChangesInternal (/IonicModule/ModalCmp/component.ngfactory.js:46:14)
    at CompiledTemplate.proxyViewClass.AppView.detectChanges (http://localhost:8100/build/main.js:117531:14)
    at CompiledTemplate.proxyViewClass.DebugAppView.detectChanges (http://localhost:8100/build/main.js:117726:44)
    at CompiledTemplate.proxyViewClass.AppView.internalDetectChanges (http://localhost:8100/build/main.js:117516:18)
    at CompiledTemplate.proxyViewClass.View_ModalCmp_Host0.detectChangesInternal (/IonicModule/ModalCmp/host.ngfactory.js:30:19)
    at CompiledTemplate.proxyViewClass.AppView.detectChanges (http://localhost:8100/build/main.js:117531:14)
    at CompiledTemplate.proxyViewClass.DebugAppView.detectChanges (http://localhost:8100/build/main.js:117726:44)

StackOverflow:  http://stackoverflow.com/questions/44019627/ionic-2-modal-inline-template-typeerror-cannot-read-property-items-of-unde
Ionic Forum: https://forum.ionicframework.com/t/modalcmp-ionviewpreload-error/73988
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
global packages:

    @ionic/cli-utils : 1.1.2
    Ionic CLI        : 3.1.2

local packages:

    @ionic/app-scripts              : 1.3.7
    @ionic/cli-plugin-ionic-angular : 1.1.2
    Ionic Framework                 : ionic-angular 2.3.0

System:

    Node       : v6.9.2
    OS         : macOS Sierra
    Xcode      : Xcode 8.3.2 Build version 8E2002
    ios-deploy : 1.9.0
    ios-sim    : 5.0.13

Hello, thanks for opening an issue with us. Would you mind updating to the latest release of ionic and seeing if you still have this issue?

Thanks @jgw96  for your fast answer! It worked like a charm. But I have two questions though:

Can  I use this version for production?
Can I keep relying on the Ionic "current version" doc (https://ionicframework.com/docs/)? It won't differ?

No problem! Yep, you can. Ever release we do that is not marked as nightly, alpha or beta is production ready. Also, docs are updated every release to match the latest api of that release.


text:
feat $ionicSideMenus: add '$ionicSideMenuOpen'/ '$ionicSideMenuClose' event

feat $ionicSideMenus: add '$ionicSideMenuOpen'/ '$ionicSideMenuClose' event
add an event ($ionicSideMenuOpen or $ionicSideMenuClose) broadcast on menu open/close.
$ionicSideMenuOpen and $ionicSideMenuClose include the menu side (ie. 'left' or 'right').
fixes #3579

Thanks, we're going to be holding off on new features until post 1.0.
In the meantime, could you add a unit test checking for these events?

@perrygovier as requested, I've added unit tests.
Please consider letting this one slip into 1.0. It's not exactly a fully fledged 'feature' - but I know of at least 3 apps (my own included) that would really benefit from this event being broadcasted on menu open and closed. Forgive me for being obstinate/spoiled, but I can't really see a reason why not, aside from the floodgates argument, ie. 'if we let this minor addition slip through then we'll let them all through'.

This has been added to the 1.0.1 branch. Thanks! 7679690f57ad5a5d874a519d0ede9588a48bde9e

source: gharchive/pull-request

authors: perrygovier raibutera

repo: driftyco/ionic

license: mit

license_type: permissive

license_source: bigquery


text:
Catch errors in $cordovaFlashlight promise chain

Simplify boilerplate in $cordovaFlashlight and avoid missing flashlight plugin causing an unhandled exception (eg: while testing in a browser) by capturing the exception in the promise chain.

It's supposed to fail in the browser; better would be to add a $q.reject if we're in a browser; or add  a mock that can be called in the browser.

@gortok I'm not sure I understand; it is adding a $q.reject - Could you expand on what you mean please? I'm happy to edit and resubmit if it just needs something changing.

authors: david-byng gortok

repo: driftyco/ng-cordova

license: MIT

license_source: github-api


text:
[BUG]: Missing d1 as a valid input for driver

What version of drizzle-orm are you using?
0.30.10
What version of drizzle-kit are you using?
0.21.0
Describe the Bug
Following the documentation for migrating to D1 i'm getting the following error:
Invalid input  Either "turso", "libsql", "better-sqlite" are available options for "--driver"
Using this setup:
import { defineConfig } from "drizzle-kit";

export default defineConfig({
  dialect: "sqlite",
  driver: "d1", // The docs says "d1-http" but that's gives me an type error
  dbCredentials: {
    wranglerConfigPath: "wrangler.toml",
    dbName: "products",
  },
  schema: "./src/schema/*",
  out: "./drizzle",
}) satisfies Config;

Works fine to generate, but not migrate.

### Expected behavior

d1 to exist as a valid driver

### Environment & setup

_No response_

I'm getting type errors but this at config at least launches for drizzle studio for me
export default {
  schema: "./src/db/schema.ts",
  out: "./drizzle",
  dialect: "sqlite",
  driver: "d1-http",
  dbCredentials: {
    url: ".wrangler/state/v3/d1/miniflare-D1DatabaseObject/<id>,
  },
} satisfies Config;

Same issue. Here is my setup 👇🏿
{
  “dependencies": {
    “drizzle-orm": "^0.30.10"
  },
  “devDependencies": {
    “drizzle-kit": "^0.21.2"
  }
}

// drizzle.config.ts
import { defineConfig } from "drizzle-kit";

export default defineConfig({
  schema: "./server/database/schema",
  out: "./server/database/migrations",
  dialect: "sqlite", // <-- No good, even with this. Is this even required? 🤔
  driver: "d1",
  dbCredentials: {
    wranglerConfigPath: "./wrangler.toml",
    dbName: "coolkatznotrealdbname”,
  },
});

 Invalid input  Either "turso", "libsql", "better-sqlite" are available options for "--driver"

I have the same issue with a barebone setup. @collinstommy approach worked for me with slight adjustments:
import { defineConfig } from "drizzle-kit";

const isStudio = process.argv.includes("studio");

// [TODO] Always use "d1" driver when there's a solution for the issue: https://github.com/drizzle-team/drizzle-orm/issues/2292
//   driver: "d1",
//     dbCredentials: {
//     wranglerConfigPath: "./../../apps/web/wrangler.toml",
//     dbName: "main",
//   },
export default defineConfig({
  schema: "./schema.ts",
  out: "./drizzle",
  dialect: "sqlite",
  ...(isStudio
    ? {
        driver: "d1-http",
        dbCredentials: {
          wranglerConfigPath: "./../../apps/web/wrangler.toml",
          url: "file:./../../apps/web/.wrangler/state/v3/d1/miniflare-D1DatabaseObject/YOUR_DB_ID.sqlite",
        },
      }
    : {
        driver: "d1",
        dbCredentials: {
          wranglerConfigPath: "./../../apps/web/wrangler.toml",
          dbName: "main",
        },
      }),
});

I added file: to the url option as without it was throwing URL_INVALID and also check if it's a studio command, to make drizzle-kit generate work.

This should be fixed in drizzle-kit@0.21.3. If the issue persists, feel free to reopen it!

Hey @AndriiSherman I am trying with drizzle-kit@0.21.3 but still getting the same error.
bun db:migrate
$ drizzle-kit migrate
drizzle-kit: v0.21.3
drizzle-orm: v0.30.10

No config path provided, using default path
Reading config file '/workspaces/umesh.dev/drizzle.config.ts'
 Invalid input  Either "turso", "libsql", "better-sqlite" are available options for "--driver"
error: script "db:migrate" exited with code 1

Also here is the config
import { defineConfig } from "drizzle-kit";

export default defineConfig({
  schema: "./src/db/schema.ts",
  out: "./migrations",
  dialect: "sqlite",
  driver: "d1",
  dbCredentials: {
    wranglerConfigPath: "./wrangler.toml",
    dbName: "production_db",
  },
  verbose: true,
  strict: true,
  schemaFilter: ["public"],
  migrations: {
    table: "migrations",
    schema: "public",
  },
});

Apart from generate and check none of the commands like migrate, pull, push, studio seem to work.

I also can confirm still seeing this @AndriiSherman

@Akkuma @imumesh18
it's driver: 'd1-http', ts should've suggested, please restart ts server
we will update error text respectfully

@Akkuma @imumesh18 it's driver: 'd1-http', ts should've suggested, please restart ts server we will update error text respectfully

That worked. Yea reloading TS server was the ticket since it still said 'd1' was valid. Is there a way to choose whether to use local vs remote DBs like leveraging studio on the local data?

you should have 2 separate drizzle configs for remote and local
and then just
drizzle-kit studio --config drizzle.local.config.ts
drizzle-kit studio --config drizzle.config.ts

source: gharchive/issue

authors: Akkuma AlexBlokh AndriiSherman collinstommy imumesh18 kossnocorp oliwerhelsen they-cloned-me

repo: drizzle-team/drizzle-orm


text:
bugfix: Fix a bug that import statements for columns of type double are not inserted

close drizzle-team/drizzle-kit-mirror#403

it was closed automatically, sorry. I guess you would need to change base branch to main and reopen it

cc @Karibash

authors: AndriiSherman Karibash


text:
fix: Add union undefined as accepted optional type

Encounter #2550 while playing around with drizzle-zod. This adds | undefined to the end of the OptionalKey columns:
{
  id?: number | undefined
}

It should be the same as the previous one:
{
  id?: number
}

Wonder if update should have the same treatment?
https://github.com/drizzle-team/drizzle-orm/blob/c59440ce5c29892e54b661eceaab561b4927ab56/drizzle-orm/src/pg-core/query-builders/update.ts#L30-L36

Because of this I'm following the the discussion happening over at zod repositories (was not familiar with both zod nor drizzle).
https://github.com/colinhacks/zod/issues/635
I'll close this down.


text:
Add Drogon Debian Package

Add Drogon Debian Linux package.

I think it's unlikely. Only Debian Developers (only ~51 exists) are able to create and upload Debian packages. Nor did Debian provide a way to host PPAs. Instead, maybe creating a Ubuntu package and upload to Ubuntu's LaunchPad. Then Debian users can grab the file from LaunchPad.

I didn’t meant to create an official one, because exactly the reasons you provided.
I’m not really into Ubuntu.

Ah...

It is possible to generate .deb packages with CMake. They are not the highest quality, but I found them usable. The recipe looks like this: https://schlomp.space/tastytea/epubgrep/src/branch/main/cmake/packages.cmake. They can be generated with cpack -G DEB. I can make a pull request if you're interested?
If you want to provide them for download you would need to compile each package in the same OS it is targeting.

They can be generated with cpack -G DEB. I can make a pull request if you're interested?

This would be great. You are welcome 👍
The quality should be good enough for a first version.

If you want to provide them for download you would need to compile each package in the same OS it is targeting.

I don’t think we’ll do this at the moment.

Reminder to myself that Debian installation should also be added to the documentation.

authors: marty1885 rbugajewski tastytea

repo: drogonframework/drogon


text:
Replace rejection sampling and remove use of rand()

The old code uses rejection sampling. Trying to get number between 0~61 from 0~RAND_MAX (on Windows this is 15bits and on Linux 32bits) is extremely slow. This patch fixes that by using C++'s distribution class.
Also replaces rand() with C++ random engines since OpenBSD is complaining about it.
ld: warning: Utilities.cc(Utilities.cc.o:(drogon::utils::genRandomString(int)) in archive ../libdrogon.a): warning: rand() may return deterministic values, is that what you want?

@an-tao How can we fix the cpplint problem?

@an-tao How can we fix the cpplint problem?

Temporarily solved


text:
String at the end of is not found

It seems like a string is not considered valid unless there are at least 3 bytes following it before EOF.
good.txt: jstrings finds the Haiku
bad.txt: jstrings does not find the Haiku
I'm not sure if this is expected behavior or a bug, I just noticed it while testing and thought I might as well report it.

This was due to reaching the end of the file before the end of the string was detected. This should be fixed in v1.6.

authors: derula drojaazu

repo: drojaazu/jstrings


text:
Author/gravatar inconsistancy

Drone will report the opening of a pull request by putting the pull request author's github alias in the author field.  When drone reports the MERGING of a pull request, or generally just any direct commit to master, it puts the committer's email address in the author field, rather than their github alias.
When the email address is used for the author field, the gravatar hash will indeed point to the gravatar for that email address.  When the github alias is used for the author field, a different gravatar hash will appear, one that invariably displays the system default gravatar.  Presumably this is a gravatar generated from that alias, rather than an email address.
What this means is that a single user opening pull requests and merging others will appear in Drone results as two people, sometimes with their email address and correct gravatar, and other times with their github alias and a system default gravatar.  This is problematic when attempting to track activity in a user-centric fashion.
I'm not sure if this is something that can be changed on the drone side, or if it's a fundamental github issue, but hopefully there's a resolution here that will allow consistent email-address usage for all drone builds, so people can be tracked and gravatars can be relied upon.

closed in 0.4 branch. we now persist the avatar url

Nice!
@scottferg fyi

authors: Tathanen bradrydzewski

repo: drone/drone


text:
relocate api handlers, add swagger annotations

this change starts to add swagger annotations for /api handlers. Eventually we'll be able to auto-generate the swagger file instead of hand writing. woohoo!

I'm excited to see it generated

authors: bradrydzewski tboerger


text:
Fix typo in mysql migration file for org secrets

As mentioned in Gitter...

 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 quux


text:
(feature) Allow brace-less substitutions

Brackets are super useful when you want to process a variable while substituting, but when all you want is to substitute a variable, they make things harder to read.
echo ${foo}
echo $foo

It'd nice if this package allowed bracket less substitutions.

👋 hey there, if you need brace-less substitution we recommend using os.ExpandEnv.  This library was designed to support docker-compose [1] and drone, which only substitutes variables with curly braces and, by design, ignores variables without curly braces. Since we do not require such functionality in our upstream repositories, this is not something we plan to implement.
[1] https://docs.docker.com/compose/environment-variables/#substitute-environment-variables-in-compose-files

@bradrydzewski The reason I don't want to use os.ExpandEnv is that it can only substitute from the current processes environment. This package on the other hand allows substituting from arbitrary sources (including my use case of a map). I appreciate you have no intention to support this. Thanks for this great package :smile:.

Sorry, wrong function, you can use os.Expand instead 🙂
package main

import "os"

func main() {
	s := "hello ${NAME}"
	p := map[string]string{"NAME": "Jane"}

	s = os.Expand(s, func(k string) string {
		return p[k]
	})

	println(s)
}

Ok, wow, thnx that's pretty great. :+1:.

sorry for hijacking this issue, but docker-compose actually does support brace-less variable substitution.

Both $VARIABLE and ${VARIABLE} syntax are supported.

is the lack of support for this feature a definitive answer? i do have the use-case where i need to replace brace-less variables and variables with substitutions. i suppose i could do a double pass (first  envsubst.Eval followed by os.Expand), but the performance would suffer.

A braceless substitution would potentially break existing Drone definitions as people build on this behavior.

authors: UiP9AV6Y bradrydzewski mohkale tboerger

repo: drone/envsubst


text:
ANDROID_BUILD_TOP not set

I ran into this error while building the example code.
I tried to export ANDROID_BUILD_TOP in my bash profile. Didn't work.
I also browsed the previous questions. Didn't really help.
I have sdk and ndk installed. So now I am really confused.
Please help. Thank you.

AssertionError: $ANDROID_BUILD_TOP not set; you need to run lunch.
make: *** [GypAndroid.mk] Error 1
jni/Android.mk:3: ../../../GypAndroid.mk: No such file or directory
Traceback (most recent call last):
File "deps/gyp/gyp_main.py", line 18, in
sys.exit(gyp.script_main())
File "deps/gyp/pylib/gyp/init.py", line 533, in script_main
return main(sys.argv[1:])
File "deps/gyp/pylib/gyp/init.py", line 526, in main
return gyp_main(args)
File "deps/gyp/pylib/gyp/init.py", line 511, in gyp_main
generator.GenerateOutput(flat_list, targets, data, params)
File "deps/gyp/pylib/gyp/generator/android.py", line 970, in GenerateOutput
assert android_top_dir, '$ANDROID_BUILD_TOP not set; you need to run lunch.'
AssertionError: $ANDROID_BUILD_TOP not set; you need to run lunch.
make: *** [GypAndroid.mk] Error 1
jni/Android.mk:3: ../../../GypAndroid.mk: No such file or directory
/Users/****/Library/Android/sdk/ndk-bundle/build/core/build-all.mk:87: Android NDK: WARNING: No modules to build  your APP_MODULES definition is probably incorrect!
make: *** No rule to make target `../../../GypAndroid.mk'.  Stop.

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:ndkBuild'.

Process 'command '/Users/****/Library/Android/sdk/ndk-bundle/ndk-build'' finished with non-zero exit value 2

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 8s
14 actionable tasks: 1 executed, 13 up-to-date

If I recall correctly, you probably need to set ANDROID_NDK_HOME, or put ndk-build in your path so it can be discovered by the Makefile.

repo: dropbox/djinni


text:
SOAP Service Client framework

We have a requirement to consume SOAP services. Can you please advise, which framework we should use with dropwizard to write SOAP service client?

Hello @rpawarmca. The question of what is the best-of-breed Java SOAP client library is outside of Dropwizard's scope, since we'll never be in the business of providing a SOAP library. I think the dropwizard-users mailing list would be a more suitable venue for your question, so perhaps you could ask there?

Thanks @evnm , let me check on the mailing list.

Hey, this is a dropwizard SOAP solution if you are still interested :)
https://github.com/roskart/dropwizard-jaxws

authors: evnm gavarava rpawarmca

repo: dropwizard/dropwizard


text:
Dropwizard application 0.9 memory usage?

How to find the memory usage?
how much memory does dropwizard consume usually?

@snehaldkshah You can use the standard JVM (such as jstat or VisualVM) or operating system utilities to print the memory consumption of a Dropwizard application.

authors: joschi snehaldkshah


text:
[docs/di] Add link to DW 2 supported Guice bundle

Problem:

Docs reference Guice Bundle for DI, but there are multiple, and unclear which is the one that should be used.
Solution:

Add a link to the docs

Docs reference Guice Bundle for DI, but there are multiple, and unclear which is the one that should be used.

The one working with the latest stable version of Dropwizard is fine (https://github.com/xvik/dropwizard-guicey).

Yup, that's the one I linked

authors: OneCricketeer joschi


text:
Experimental index optimised direct VQ

Part of experimental 2020A mode as discussed in #217, codec2 side of this work in https://github.com/drowe67/codec2/pull/274.

Design tweaks to select between "direct VQ" (as currently used for FreeDV 2020) and "index optimised direct VQ".
Use binary switch algorithm tool to optimise each of the 4 stage VQs, for example:~/codec2/build_linux/misc/vq_binary_switch -d 18 -m 1000 split_stage1.f32 split_stage1_bs.f32

Listen to a few samples using lpcnet_enc/lpcnet_dec and see if it improves speech quality in channels with random errors (lpcnet_dec can introduce errors).
If effective integrate with modems and test resulting FreeDV mode in simulation and OTA.

Initial test with just stage1 optimised didn't improve the quality. -x option selects index optimised VQ.

Direct VQ, 1% BER:
sox ../wav/peter.wav -t raw -r 16000 - | ./src/lpcnet_enc -s | ./src/lpcnet_dec -s -b 0.01 | aplay -f S16_LE -r 16000
Index Optimised Direct VQ, 1% BER
sox ../wav/peter.wav -t raw -r 16000 - | ./src/lpcnet_enc -x | ./src/lpcnet_dec -x -b 0.001 | aplay -f S16_LE -r 16000

I have used the codec2 binary switch tool to develop some index optimised direct quantisers. lpcnet_quant.c has been hacked to introduce bit errors, so we can test the effect of index optimisation.
Typical command line for test:
sox ../wav/all.wav -t raw - | ./src/dump_data --c2pitch --test - - | ./src/quant_feat -i --mbest 5 -d 3 -q split_stage1_indopt.f32,split_stage2_indopt.f32,split_stage3_indopt.f32,split_stage4_indopt.f32 > /dev/null

VQ
BER
SD (dB)

Original direct split
0.0
3.8

Index optimised direct split
0.0
3.8

Original direct split
0.01
68.5

Index optimised direct split
0.01
33.7

When I listen on a short file (wia.wav) its sounds better too, less extreme clicks and pops.  It is pretty noisy at 1% BER, suspect this codec is more sensitive to bit errors.

OK it looks like the first stage VQ is super sensitive.  If we simulate with the first stage VQ (bits 0..10) error free, and add errors to all other bits:
sox ../wav/wia.wav -t raw -r 16000 - | ./src/lpcnet_enc -s | ./src/lpcnet_dec -s -b 0.04 --ber_st 11 | aplay -f S16_LE -r 16000

... it sounds pretty good.  The quality decreases gradually with increasing BER (like SSB), but doesn't break down and there are no loud clicks and pops.

This suggests the following waveform design:

Really strong protection of just bits [0..10] (the stage1 VQ quantiser).  For example a rate 0.5 LDPC code on just the first 11 bits.  This would mean a (n,k) = (52+11,52) = (63,52) rate 0.82 overall, so a modest 18% increase in RF bandwidth.  This could be a redesign of the current 2020 FEC (which covers all bits), or added to the prototype 2020A waveform (if it will fit in terms of RF bandwidth).  Or maybe both.
For listening tests it's unclear if the index optimisation is worth it.  It may provide additional protection if combined with FEC on the first vector, as per previous index optimisation results on a prototype lower bit rate codec.  The objective SD results above suggest it is useful.

OK it looks like the first stage VQ is super sensitive.  If we simulate with the first stage VQ (bits 0..10) error free, and add errors to all other bits:
sox ../wav/wia.wav -t raw -r 16000 - | ./src/lpcnet_enc -s | ./src/lpcnet_dec -s -b 0.04 --ber_st 11 | aplay -f S16_LE -r 16000

... it sounds pretty good.  The quality decreases gradually with increasing BER (like SSB), but doesn't break down and there are no loud clicks and pops.

This suggests the following waveform design:

Really strong protection of just bits [0..10] (the stage1 VQ quantiser).  For example a rate 0.5 LDPC code on just the first 11 bits.  This would mean a (n,k) = (52+11,52) = (63,52) rate 0.82 overall, so a modest 18% increase in RF bandwidth.  This could be a redesign of the current 2020 FEC (which covers all bits), or added to the prototype 2020A waveform (if it will fit in terms of RF bandwidth).  Or maybe both.
For listening tests it's unclear if the index optimisation is worth it.  It may provide additional protection if combined with FEC on the first vector, as per previous index optimisation results on a prototype lower bit rate codec.  The objective SD results above suggest it is useful.


text:
[feat] Show progress for "Moving install to composer root" (ddev composer create)

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem? Please describe
Commands like ddev composer create --prefer-dist laravel/laravel can take a minute while DDEV rsyncs the files to the project root directory ( composer-create.go:124), at least on my mac m1 air.
It would be great to see the overall progress of the rsync operation, also to be sure it doesn't got stuck.
Describe the solution you'd like
I found out that there is an rsync argument called --info=progress2, which was introduced with rsync 3.1.0.

There is also a --info=progress2 option that outputs
statistics based on the whole transfer, rather than
individual files.  Use this flag without outputting a
filename (e.g. avoid -v or specify --info=name0) if you
want to see how the transfer is doing without scrolling
the screen with a lot of names. (You don't need to specify
the --progress option in order to use --info=progress2.)
Source: manpage rsync

I quickly checked the rsync version of DDEV on Gitpod:
gitpod /workspace/d9simple (main) $ ddev exec rsync --version
rsync  version 3.2.3  protocol version 31

Unfortunately I'm not sure if DDEV container always use rsync > 3.1.0?
Is it safe to assume this?
Tested it quickly in gitpod and changed composer-create.go, worked on Gitpod:
	output.UserOut.Printf("Moving install to composer root")

		rsyncArgs := "-rltgopD --info=progress2" // Same as -a
		if runtime.GOOS == "windows" {
			rsyncArgs = "-rlD --info=progress2" // on windows can't do perms, owner, group, times
		}

		// output.UserOut.Printf("Running rsync with %s", rsyncArgs)

		stdout, stderr, err = app.Exec(&ddevapp.ExecOpts{
			Service: "web",
			Cmd:     fmt.Sprintf(`rsync %s "%s/" "%s/"`, rsyncArgs, containerInstallPath, app.GetComposerRoot(true, false)),
			Dir:     "/var/www/html",
			Tty:     isatty.IsTerminal(os.Stdin.Fd()),
		})

Describe alternatives you've considered
There is also the option to use a pipe with pv, but you need to calculate the number of files beforehand${RSYNC} ${RSYNC_ARGS} ${SOURCES} ${TARGET} | pv -l -e -p -s "$HOW_MANY_FILES". (Source, Demo-Gif see "Use pv command to monitor progress of rsync command" -> here)
Additionally there are maybe third party progressbar packages.
Additional context
Infos about ddev composer create (wrapper for composer create) can be found in DDEV docs
Thanks very much for suggestion & tips!

The web container currently has rsync v3.2.3; that's standard on Debian Bullseye, it won't be changing (except upward as time goes on).

PR would be great.

BTW, I recommend using ddev composer create --no-install everywhere these days. It avoids this whole thing, and a ddev composer install afterward is so much more satisfying.

Would love to see your solution @mandrasch , otherwise let's close this. Thanks!

I'll submit a PR (if I find time to test it locally as well) :+1:
Thanks for the great advice with ddev composer create --no-install! It seems to be much more 'readable' anyway, less 'magic' involved, might switch to that for my personal projects. :+1:

authors: mandrasch rfay

repo: drud/ddev


text:
Default implementation for getDouble().

Default implementation for getDouble() in Aggregator/BufferAggregator.
@b-slim

:+1:

@akashdw and @himanshug  why do we need a default implementation ?

@b-slim because all aggregators are failing to compile.
what is the downside of having default impl ?  that method would not have a meaningful implementation for the sketch based complex aggs.

Which aggregators are failing to compile ? this default implementation is not a meaningful one and it can lead the dev into forgetting about implementing it.

@b-slim e.g. https://github.com/druid-io/druid/blob/master/extensions-core/datasketches/src/main/java/io/druid/query/aggregation/datasketches/theta/SketchBufferAggregator.java#L123 has same and all the other sketch based aggregators (qualtiles, tuples,.. ) would have same impl

yeah that is the way to go every aggregator will implement what ever needed, but that's not a default implementation anyway same for other methods like getLong. Again not sure which code you said is not compiling ?

we have custom aggregators which are failing to compile and getDouble() can't have a meaningful implementation for any, hence the default.

also, about dev forgetting to implement it: wouldn't that specific aggregator end up throwing exception and then the dev would know that it needs to be implemented?

From a design perspective, I don't think throwing an exception is suppose to be the default implementation of such method it makes no sense IMO, it sends wrong message/question to any new developer.

also, about dev forgetting to implement it: wouldn't that specific aggregator end up throwing exception and then the dev would know that it needs to be implemented?

not necessarily, it can throw the exception on run time and not seeing it during development phase.
Not sure what is wrong with implementing those within the in-house custom aggregator and avoid polluting the interface ?

I see your point of enforcing that aggregators compilation fail so that owning dev takes a look at it and implement it appropriately. ( I wonder if documenting it in the release notes wouldn't be enough. )
On the other hand, there are many custom aggregators that different teams have (not necessarily one set of devs) and a general user complain about extra work needed to do upgrades. Intention of this PR is to reduce the extra work needed if possible. getDouble() would have the default impl proposed in this PR in most cases.
let us discuss it in the dev sync to arrive at a conclusion.

I support @b-slim here, default implementation makes more harm than good here IMO. If adding default impl to getDouble(), why not to add it to getFloat() and getLong() also? But if we do so, it looks like we give preference to Object aggregators, that actually false, numeric aggregators are more widely used overall.

The introduction of the getDouble is a backwards incompatible API change on a public API.  Forcing all implementations of all aggregators to add the same default implementation that is added here will only introduce friction for upgrading for anybody who has taken the time to implement their own aggregator.
Backwards incompatible API changes should only be made when absolutely necessary in order to facilitate new functionality.  Philosophical concerns are the least important element when discussing backwards incompatible API changes.  If there are philosophical concerns with the APIs, then a new set of APIs that won't have the same philosophical issues (both at inception and during evolution) should be introduced and we should migrate to them.
And yes, a default on getLong would've been called for if we had been on Java 8 at the time.  getFloat was part of the original interface, so defaulting that wouldn't have happened, but there's no reason for it to not happen.  If the request is to add defaults on getFloat and getLong as well as documentation for why we have the individual primitive getters to make it more clear to developers, that sounds reasonable to me.
Fwiw, if we really want to make it super simple to generate different types of columns, it might actually make sense to use getObject and just verify that it is a Double before converting it to the primitive double.  I.e. just figure out a way to completely circumvent the primitive getters altogether.

it was concluded that having a default implementation makes sense to have easy upgrade, however default implementation should return the result of getFloat() instead of throwing exception.
interface change would be noted in the release notes so that extension owners could change their implementations if needed.

@akashdw can you change the default impl to getFloat() ?

@b-slim @himanshug Thanks for reviewing the PR. I changed the default implementation to use getFloat().

👍 left a minor comment.

authors: akashdw b-slim cheddar himanshug leventov

repo: druid-io/druid


text:
Remove pached TestNG classes

Current version of maven-failsafe-plugin supports setting custom version of testrunfactory, so there is no need to have patched TestNG class anymore.
Reference task: SUREFIRE-622

Please rebase on master.

@leventov done

authors: leventov yuppie-flu


text:
Refinement in the L.Control.SyncView.js:_pullView function

This PR implements the following:

parsing of Lat and Lon to 6 digit float;
trigger dynamic sync / update of the proximity-origin-summary Lat and Lon elements (eventually present in case check/selection of "Show (anyway) the Origin coordinates as summary in the Exposed Form" option

This implementation syncs with latest improvements implemented in Geofield version 8.x-1.61., greatly aimed to enhance dynamic sync between Origin coordinates input and Origin coordinates summary elements.

@itamair refer to https://www.drupal.org/project/leaflet/issues/3254562#comment-15806814, I partly implement the "parsing of Lat and Lon to 6 digit float" logic for 1.9.3 release.
BTW, hard-coded with .geofield-lat-summary and .geofield-lon-summary shouldn't be a good idea when working with downstream integration, therefore sorry that I reject this idea.


text:
Install psmisc to support killall

https://github.com/dry-dock/u16/issues/30

I'll pause all the build_16all job and un-pause once all the changes are in.


text:
Remove the .txt and .swp Files

Delete the .txt and .swp files & .gitignore files with the .swp extension.

I'll do it

Merged

authors: WendyThress-Roux smr-j


text:
Splash Screen Activity Implementation

Hey,
This app needs a Splash Screen (The animation screen that comes when you opens a app),
Please note that

create a separate activity for this

create that activity inside the Ui package (folder) only

Download the APP LOGO

implement the required functions inside that activity

create pull request

Add the tag of Hacktoberfest or Hacktoberfest2022
Happy Hacking 👩💻

For Help You can search on Yt about creating a splash screen in Android (kotlin)

Can you please assign me this issue!

Can you please assign me this issue!

You can Start

Thanks !

hey , can I work on this issue

can you assign this to me


text:
[TO-PR] [Chores] Privacy Report Submission.

name: Chores
ratings:
locationTracking: 1
dataRetention: 1
userControl: 1
darkPatterns: 1
crossSiteTracking: 1
summary: "/privacy-report-card/details/Chores"
reference: "https://chores-561a7.web.app/login"
category: "privacy-report-detail"
contibutors: Bryan Obieyisi ckobieyisi@gmail.com
Chores's Privacy Policy - Summary

They collect hella data. Privacy? Forget it....

This issue was automatically created from an API call.


text:
Question: does this library support feldman verifiable sss ?

hi,
could not see how else to address this question. hence filing an issue. How can i use the library to use verifiable sss using feldman approach of homomorphic encryption ?
thanks,
regards,
Nilay

Dear @shoppertreat,
Indeed this libary does not support Feldman VSS (and it is considered not in scope for this library). At this moment, I have no recommendations for alternative software to use, sorry.
-- Daan

authors: dsprenkels shoppertreat

repo: dsprenkels/sss


text:
Second most repeated string in a sequence

Hi @dsrao711, greetings
I would like to add a solution to this problem
https://practice.geeksforgeeks.org/problems/second-most-repeated-string-in-a-sequence0534/1
Please assign this task to me.
Reference: DSA 450 sheet by Love Babbar
Thanks and Regards

Go ahead!


text:
fix som typo

see the changes.

Thanks!

authors: dtao flyyang


text:
DeprecationWarning: Buffer() is deprecated

Hi.
Thank you for this great library!
The problem
When I tried this library, there is warning message below at the developer console.
(node:80337) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.

The cause
This is because the description new Buffer() is included by this library.
The solution
I tried to search this description by the commend below.
ag -Q "new Buffer" --ignore browser/

Then I found 3 places where the description used.

https://github.com/dtjohnson/xlsx-populate/blob/master/blank/template.js#L4

This line should use Buffer.from

https://github.com/dtjohnson/xlsx-populate/blob/master/lib/blank.js#L4

This line should use Buffer.from

https://github.com/dtjohnson/xlsx-populate/blob/master/lib/Encryptor.js#L523

This line should use Buffer.alloc

Unfortunately, I don't have the right to push the branch and make PR...
Would you please solve this problem?
I've created new PR for this issue.  Please have a look.

@eddiecorrigall
Thanks.
I've created PR, so I'm happy if you check the PR. #169

repo: dtjohnson/xlsx-populate


text:
Support declarative macros generating bridge modules

I dug around but didn't see an issue about this open already. It's not currently possible (AFAIK) to use declarative macros to generate whole cxx::bridge modules. The proc-macro side will work as expected, but the C++ codegen is parsing the unexpanded source and so it misses those modules.
This is different from #274 in that I'm not looking to somehow do macro expansion inside a bridge, which is understandably not possible today. I'm not sure whether there are any better answers here, however.
A similar workaround as was suggested in #274 can be used - i.e. generate the source beforehand, either in build.rs or a previous build step, depending on build system - but that is decidedly less user friendly, so if there are any other promising approaches here I'd like to explore them. The only approaches I can personally think of are (to put it mildly) hacky, e.g. have the proc-macro write the input TokenStream out somewhere so that cxxbridge can also operate on that, perhaps on an opt-in basis. Any other ideas?

autocxx needs to do something similar. David kindly exposed a codegen interface via https://crates.io/crates/cxx-gen/0.7.51, which other macros can choose to use to achieve the cxx C++ side codegen.

Hrm, I suppose I could use that to implement my own proc-macro with the hacky “generate other files in a proc-macro” approach above. Is autocxx doing something like that today?

Any update on this issue? @sbrocket, were you able to find a work-around?

authors: adetaylor sbrocket seblaf1

repo: dtolnay/cxx


text:
add unit test for fields.go, result.go, rows.go.

What this PR does:
Which issue(s) this PR fixes:

For #72 .
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report

Merging #75 (7f12a38) into master (09ac125) will increase coverage by 1.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #75      +/-   ##
==========================================
+ Coverage   23.27%   24.44%   +1.17%
==========================================
  Files          45       45
  Lines        8285     8285
==========================================
+ Hits         1928     2025      +97
+ Misses       6093     5995      -98
- Partials      264      265       +1

Impacted Files
Coverage Δ

pkg/mysql/rows.go
6.56% <0.00%> (+6.56%)
:arrow_up:

pkg/mysql/fields.go
64.40% <0.00%> (+64.40%)
:arrow_up:

pkg/mysql/result.go
100.00% <0.00%> (+100.00%)
: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 09ac125...7f12a38. Read the comment docs.

LGTM

authors: cjphaha codecov-commenter dongzl

repo: dubbogo/arana


text:
Issue found on page 'JSON Type'

Page URL: https://duckdb.org/docs/data/json/json_type.html
The past participle of the irregular verb "to cast" is "cast", not "casted". This docs page, and probably others, wrongly uses "casted", when it should use "cast" for present and past, as in "has been cast from a string to JSON".
The web page below gives the right recommendation and shows the Google stats that back up the absolute guidance that "casted" should never be used in Modern written English with this meaning.
https://writingexplained.org/cast-vs-casted
"Casted" can be used to mean "assigned a [social] caste", or "having had a plaster cast put on a limb" but that's not what's happening here. I have never seen it used before in CS or software engineering publications.

Hi Alastair, thanks! I submitted a fix. It will be deployed in ~10 minutes to the site.

authors: alastai szarnyasg

repo: duckdb/duckdb-web


text:
This is a test PR

Task/Issue URL: https://app.asana.com/0/488551667048375/1202717423070850/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #2146   👈

This comment was auto-generated by Graphite.

repo: duckduckgo/Android


text:
Add FF roll back integration test

Task/Issue URL: https://app.asana.com/0/488551667048375/1206994138177836/f
Description
Adding integration test to cover FF roll backs
Steps to test this PR
NA, test pass

#4384  👈
develop

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

Join @aitorvs and the rest of your teammates on  Graphite


text:
filter by license

Thanks for this great initiative!
Would it please be possible to add an option to filter the results by license? e.g. only display the CC0 ones

Planned as part of #100

authors: duerrsimon flodebarre

repo: duerrsimon/bioicons


text:
Flatnotes version display?

Is there anywhere I can see the current version I am running?
When updating containers this would be helpful but I could not find a way to detect my version.
many thanks

There isn't at the moment, but I think this would be a good thing to add. I'll add this to my todo list.

repo: dullage/flatnotes


text:
24

Need someone to test clean update with 'run' option for me. I don't have an easy project to update currently.

@duluca terminal now has cleaner stdout and you guessed it green text :)

@kentonbmax Runs ng version but not an actual update

Used local-weather-app to test

@kentonbmax fixed portions of the code and moved updateNg implementation over to  #28. execa is the better way to go about this.

authors: duluca kentonbmax

repo: duluca/angular-evergreen

license: ISC


text:
Added support for excluded routes

Fixes issues with tests for previous PRs.

@dunglas 4af1f4d passed but I rolled back once I re-read the feature.
  Scenario: We should access excluded page which is behind secure
    Given I send POST request to "/protected-resource/allowed" with invalid csrf header
    Then the response code should be 200

What should happen in this case. A CSRF was supplied but it was invalid… however it probably shouldn't have been checked in the first place becuase the url was excluded.

@dunglas Is this good to merge or do you require something else?

@dunglas Is there anything holding this PR back?

Thank you very much for this nice addition @leevigraham!

Hi @dunglas,
any ETA when it will be released ?

authors: dunglas leevigraham qmorek

repo: dunglas/DunglasAngularCsrfBundle


text:
Properly handle + and / replacements in state value generation

This pull request resolves issue: https://github.com/duosecurity/duo_universal_nodejs/issues/28
Description
I modified the string replacements for + and / because the current version only replaces the first instance of these characters. Additionally, I've replace the - and _ values with Z because - and _ are invalid Base64 characters.
Motivation and Context
This issue causes intermittent failures of 2FA attempts in our client code because my original state value contains plus characters... sometimes... and the resulting state returned by the Duo authentication causes those plus signs to become space characters. Then they no longer match the original state when I check to see if they match for a user.
How Has This Been Tested?
Tested in my own production code...
Types of Changes

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

Regarding your comment about - and _ being invalid base64 characters... is this causing an problem for your application?  I'm not sure why the state generation was written with base64 involved; nothing about the OIDC spec says the state needs to be base64.

You're absolutely right, the state doesn't need to be base64. Your function name is "bytesToBase64url" so I assumed that meant it's expected to return a valid base64 string. Introducing - or _ makes it not a base64 string. That's the only reason why I changed that.

I've come up with a better way to do this. I'll be submitting another PR shortly.

authors: AaronAtDuo driverjb


text:
jquery is not the first module in minified vendor js

I was having issues with angular-bootstrap-switch (element.bootstrap-switch is not a function error) when building the production code. It turns out that, in the minified vendor file, angular was the very first, before jquery. No issues when doing gulp default because jquery is the first.  So, I had to do 'bower install jquery --save' and make sure it is the first dependency in bower.json. Problem solved!

Specially for this I have this code block in my bower.json:
  "overrides": {
    "angular": {
      "dependencies": {
        "jquery": "1.9+"
      }
    },
    "angular-i18n": {
      "main": "angular-locale_ru-ua.js",
      "dependencies": {
        "angular": "*"
      }
    },
    "angular-bootstrap-toggle-switch": {
      "dependencies": {
        "angular": "1.3+"
      }
    },
    "moment": {
      "main": [
        "moment.js",
        "locale/ru.js"
      ]
    },
    "bootstrap": {
      "main": [
        "dist/css/bootstrap.css",
        "dist/js/bootstrap.js",
        "dist/fonts/glyphicons-halflings-regular.eot",
        "dist/fonts/glyphicons-halflings-regular.svg",
        "dist/fonts/glyphicons-halflings-regular.ttf",
        "dist/fonts/glyphicons-halflings-regular.woff",
        "dist/fonts/glyphicons-halflings-regular.woff2"
      ]
    }

generator-ng-poly uses wiredep module for resolution dependencies and determine their order. Angular's bower.json not declares explicit dependency on jQuery as it can work without it, but angular-bootstrap-toggle-switch can't. Of course you can just override angular-bootstrap-toggle-switch to depend on jQuery but in my project this causes problems: after js concatenation jQuery was before angular-bootstrap-toggle-switch but after angular. And angular used jquery-lite. So I modified my bower.json and explicitly declared that angular-bootstrap-toggle-switch depends on angular and angular depends on jquery.

Thank you for the info guys.
@flipjs, I think it's a coincidence that explicitly listing jQuery as a dep in bower.json fixed the issue (unless you meant you provided the override). @xak2000's override should guarantee it works.
We should probably modify the default scaffolded bower.json to add jQuery as a dep for Angular if Bootstrap is being used. This issue has come up quite a few times already.

Thank you both. Will update my bower.json and use @xak2000 's config as guide.

authors: dustinspecker flipjs xak2000

repo: dustinspecker/generator-ng-poly


text:
How to convert cv::Mat to uchar3* for detectnet

Hi. I am receiving yuv frames from the camera. which is not possbile to capture by videoSourse::Capture() in detectnet.cpp as the prefered input format are only rgb8, rgba8, rgb32f, rgba32f . In that case I am trying to capture it with CV::VideoCapture  But the frames captured with CV are CV::Mat. How Can I convert those yuv frames captured with CV to those uchar3* so that I can feed it to detectnet. Any help would be appreciated. Thanks in Advance..
I saw some code related to cudaFromNumpy() in python like cude-from-cv.py and cude-from-numpy.py etc. But I can't find such functionalities in C++ implimentation of Jetson-utils

Hi @Nowfal1993, you may want to see this post, and use cvtColor() beforehand to convert YUV to RGB:
https://github.com/dusty-nv/jetson-inference/issues/129#issuecomment-324130091

Hi @dusty-nv , I tried #129, but I am unable to perform such conversion as mentioned there. The frames are not converting properly and the programs stop immediately after
[TRT]    device GPU, networks/SSD-Mobilenet-v1/ssd_mobilenet_v1_coco.uff initialized.
Query: I saw one file in the build/arch64/include/jetson-utils/cudaYUV.h and it seems it can be used to convert yuv422 to rgb8 or rgb32 format. Is it possoble? if so, how to integrate it to the input pipeline and to the detectnet. I stuck this point so long, Any help would be really appreciated.

Query: I saw one file in the build/arch64/include/jetson-utils/cudaYUV.h and it seems it can be used to convert yuv422 to rgb8 or rgb32 format. Is it possoble?

I would recommend to use the cudaConvertColor() function as opposed to calling the YUV<->RGB functions directly.  See examples here:
https://github.com/dusty-nv/jetson-inference/blob/master/docs/aux-image.md#color-conversion

// load the image as rgb8 (uchar3)
if( !loadImage("my_image.jpg", &imgInput, &width, &height) )
	return false;

Issue1: loadImage() function from jetson-utils/imageIO.h is meant to Load a  color image from disk into CUDA memory in  ucahr3/ucahr4/float3/float4 formats with pixels values 0-255. If thats the case how can load yuv image of type uint8

repo: dusty-nv/jetson-inference


text:
What are the correct arguments for onnx_export.py if my train_ssd.py resolution=600x600?

Hi @dusty-nv,
Below are my train_ssd.py arguments
!python3 train_ssd.py --dataset-type=voc --data=data/final --model-dir=models/final --batch-size=128 --workers=12 --epochs=50 --resolution=600

What are the correct arguments required for onnx_export.py? From my source code inspection, seems that width, height and batch-size are the key arguments
parser.add_argument('--width', type=int, default=300, help="input width of the model to be exported (in pixels)")
parser.add_argument('--height', type=int, default=300, help="input height of the model to be exported (in pixels)")
parser.add_argument('--batch-size', type=int, default=1, help="batch size of the model to be exported (default=1)")

and I expected the below code works
python3 onnx_export.py models=models/final --width=600 --height=600 --batch-size=128

but after running the code I found out that I received the following error:
Namespace(batch_size=128, height=600, input='', labels='labels.txt', model_dir='models/final', net='ssd-mobilenet', output='', width=600)
=> running on device cuda:0
=> found best checkpoint with loss 2.8281142711639404 (models/final/mb1-ssd-Epoch-46-Loss-2.8281142711639404.pth)
creating network:  ssd-mobilenet
num classes:       9
=> loading checkpoint:  models/final/mb1-ssd-Epoch-46-Loss-2.8281142711639404.pth
=> exporting model to ONNX...
/content/pytorch-ssd/vision/ssd/ssd.py:45: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
  if get_feature_map_size:
/content/pytorch-ssd/vision/ssd/ssd.py:75: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
  if get_feature_map_size:
/content/pytorch-ssd/vision/ssd/ssd.py:87: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
  if get_feature_map_size:
/content/pytorch-ssd/vision/ssd/ssd.py:93: TracerWarning: Converting a tensor to a Python boolean might cause the trace to be incorrect. We can't record the data flow of Python values, so this value will be treated as a constant in the future. This means that the trace might not generalize to other inputs!
  if get_feature_map_size:
Traceback (most recent call last):
  File "onnx_export.py", line 109, in <module>
    torch.onnx.export(net, dummy_input, args.output, verbose=True, input_names=input_names, output_names=output_names)
  File "/usr/local/lib/python3.8/dist-packages/torch/onnx/utils.py", line 504, in export
    _export(
  File "/usr/local/lib/python3.8/dist-packages/torch/onnx/utils.py", line 1529, in _export
    graph, params_dict, torch_out = _model_to_graph(
  File "/usr/local/lib/python3.8/dist-packages/torch/onnx/utils.py", line 1111, in _model_to_graph
    graph, params, torch_out, module = _create_jit_graph(model, args)
  File "/usr/local/lib/python3.8/dist-packages/torch/onnx/utils.py", line 987, in _create_jit_graph
    graph, torch_out = _trace_and_get_graph_from_model(model, args)
  File "/usr/local/lib/python3.8/dist-packages/torch/onnx/utils.py", line 891, in _trace_and_get_graph_from_model
    trace_graph, torch_out, inputs_states = torch.jit._get_trace_graph(
  File "/usr/local/lib/python3.8/dist-packages/torch/jit/_trace.py", line 1184, in _get_trace_graph
    outs = ONNXTracedModule(f, strict, _force_outplace, return_inputs, _return_inputs_states)(*args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/torch/jit/_trace.py", line 127, in forward
    graph, out = torch._C._create_graph_by_tracing(
  File "/usr/local/lib/python3.8/dist-packages/torch/jit/_trace.py", line 118, in wrapper
    outs.append(self.inner(*trace_inputs))
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1194, in _call_impl
    return forward_call(*input, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/module.py", line 1182, in _slow_forward
    result = self.forward(*input, **kwargs)
  File "/content/pytorch-ssd/vision/ssd/ssd.py", line 101, in forward
    boxes = box_utils.convert_locations_to_boxes(
  File "/content/pytorch-ssd/vision/utils/box_utils.py", line 104, in convert_locations_to_boxes
    locations[..., :2] * center_variance * priors[..., 2:] + priors[..., :2],
RuntimeError: The size of tensor a (11658) must match the size of tensor b (3000) at non-singleton dimension 1

Question:

I found out that using just 'python3 onny_export.py models=models/final' works, do those value matter?
If those values matter, how should I change the values?
Should I just alter the values from the res-512 branch to achieve the results?

Update
I checked out the res-512 branch for pytorch-ssd it worked
!python3 train_ssd.py --dataset-type=voc --data=data/final --model-dir=models/final --batch-size=128 --workers=12 --epochs=50
!python3 onnx_export.py --model-dir=models/final --batch-size=128

@dusty-nv may I know whether this is a bug or not?


text:
roadhog build 后 index.html 打开一片空白？？？？

使用 roadhog build 不是应该对项目进行打包成静态代码吗，然后放到服务器下应该能跑的才对啊，可是现在运行之后， index.html 浏览器打开一片空白，就没有人遇到这问题吗？？？？？？？？？？？

其它都开发完了，就差这最后一步了，死活不成功，了解了一下,roadhog 不是替代 webpack 简化配置进行代码打包吗，有没有知道的帮忙解答一下啊。

如果你用的是browserHistory 服务器配置可参考 https://github.com/facebookincubator/create-react-app/blob/master/packages/react-scripts/template/README.md#deployment

试了好久，发现在 tomcat 成功不了，使用的 nginx，配置了反向代理，现在已经可以跑起来了。不过又出现了新的问题，样式乱了，不知道是打包时没打上还是啥问题。

折腾半天还是自己给弄出来了，记录一下，给其它遇到该问题的人一个参考吧：
使用 npm run build 之后得到的 dist 目录里面都是静态资源，在 index.html 里面需要手动引入下样式文件

  <meta charset="UTF-8">

  <link rel="stylesheet" href="/index.css">

authors: dkvirus lanxxg

repo: dvajs/dva


text:
browserHistory 如何在开发环境和生成环境下使用?

根据官方文档
import {browserHistory} from 'dva/router';
// 1. Initialize
const app = dva({
history: browserHistory,
});
这样是正常的 但是 访问路由下面的其他页面 变成了 not found。
当url 带有# 的字符 就是正常的 。请指教

用browserHistory后, 需要服务器配置泛路由, 把所有路由都指向到index.html文件

请教 @lincenying @sorrycc
所以如果是采用dva-cli创建的脚手架，里面以dora作为开发server的时候，应该如何配置呢？
另外如果正常部署线上环境（用atools-build生成）又应该怎样做呢？

请问有教程或者demo么 求一份 谢谢@lincenying @sorrycc

开发模式下
用 dora-plugin-browser-history，记得指定 index 。
生成环境
需要服务端支持，配 nginx 或者在服务端(比如 node)批量处理路由，例子：https://github.com/sorrycc/dva-boilerplate-isomorphic/blob/master/server/server.js#L22

works like a charm ✌
thx @sorrycc

补充下 nginx 的简单配置：
server {
	listen 8088;
	listen [::]:8088;

	root /opt/web/static;

        location / {
          try_files $uri /index.html;
        }
}

把 atool-build 生产出来的
index.css
index.html
index.js
放到 nginx root 下就可以了，这里用的 /opt/web/static

        location / {
          try_files $uri /index.html;
        }

这样不刷新页面是没有问题的。 我遇到的问题是dev.site/foo/bar当路由到这个页面的时候，f5刷新， 那么js，css等资源文件的路径也都变化成了
dev.site/foo/bar/common.js
...
dev.site/foo/bar/index.css

不知道我这个情况怎么配置nginx谢谢

@xjdata 资源文件的链接用绝对路径，比如 /common.js。

@sorrycc  谢谢。

roadhog server 开发环境如何使用browser history呢

authors: ashersmtll chengzhenping librae8226 lincenying sorrycc xjdata


text:
请问Webpack.config.js中的env参数是如何设置的？

我尝试设置node_env到production，但是这里的参数始终是development，请问要如何配置才能让这里的env生效呢？
我得配置：
"start": "cross-env NODE_ENV=production dora -p 8089 --plugins "webpack,browser-history?index=/src/index.html"",
谢谢

不需要配的。npm run build 是 production，用于生产环境；npm start 是 development，用于开发环境。

有什么办法能自己手动改么

为啥要手动改？

我这边的后台接口有不止两种环境。
想在打包的时候自动根据环境修改后台接口

可以用非 NODE_ENV 的环境变量，然后在 webpack.config.js 里判断。

"build": "SELF=local && atool-build dev"

这样可以么，我用process.env.SELF读不到

SELF=local atool-build 就可以了，在 webpack.config.js 里判断 process.env.SELF。

多谢！！

authors: sorrycc sunOpar yutingjin


text:
ant-design 样式问题

想实现table 列宽度，然后超出部分省略号表示
{
"title": "商户名称",
"dataIndex": "name",
"key": "name_2",
width:'150px',
className:'nameWidth'
},
.nameWidth{
display: inline-block;
overflow: hidden;
text-overflow:ellipsis;
white-space: nowrap;
background-color: aqua;
}
不起作用，有小伙伴遇到过吗

"title": "商户名称",
"dataIndex": "name",
"key": "name_2",
width:'150px',
render(text) {
  return (
{text}
)
}
}

@MaYing929 建议自己调试哈，这属于前端基础

authors: MaYing929 cycgit nikogu


text:
Add onStarted hook method.

The reason to add this method is to add support for test static page.
Add some extraReducer, and invoke onStarted method when dva start is done,
in this way, a static mock feature is easy to be supported in a plugin.
see #1480

感谢 PR，我觉得先用其他的 mock 方案吧。

authors: TimYi sorrycc


text:
about the data type

thanks for your brilliant idea and opensource!
A question about the data type:
I notice that the bf16 and tf16 are both set True. I'm confused what will happen. Can they use together?

This is a really good question!
<I suppose you are referring to the bf16 and tf32 flags that we used in the training script, which are both set to True>
While they are both referring to the floating point data types, their usage is quite different. We are able to access tensors in different data types such as fp16/32 and bf16. In contrast, tf32 (TensorFloat 32) is an internal data type available in NVIDIA's Ampere-architecture GPUs and cannot be operated directly in pytorch code:
It's a special computational format/mode designed for TensorCore on Nvidia GPUs, speeding up mat_mul, etc. It has an 8-bit numerical range and a 10-bit precision range. When we are training/inference with fp32, we can simply enable tf32 to gain throughput improvement. It also supports fp16 or bf16 mixed precision. There was also an interesting discussion on benchmarking the gains using tf32 on different data types in https://github.com/huggingface/transformers/issues/14608#issuecomment-1004392537.
Basically, if you have Ampere GPUs such as 3080/90 or A100, you can simply enable tf32 to speedup your tasks. That's also why previous codebases such as Alpaca and Vicuna have set both flags to be True.

I will close this issue as it has been inactive for several days. Please feel free to re-open it if there are any other things to discuss.

authors: Michelleable TheSouthFrog yukang2017

repo: dvlab-research/LongLoRA


text:
Sign up

With the working User model it's time to add option for letting users sign up.

HTML form to submit user signup information to application
Data will be used to create a new user and save its attributes to the database
After signup process - render a profile page with the newly created user's information
Tests for the new functionality needed (also integration tests)

Full User signing, loging and editing (plus destroying option) was added. Including:

views
db migrate
tests

repo: dvnDC/shopone


text:
🛑 Nextcloud is down

In 231bf2c, Nextcloud (https://nextcloud.itclowd.de) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Nextcloud is back up in 7bddc4d after 2 hours, 46 minutes.

repo: dwaris/itclowd-uptime


text:
Fronend Images Loading issue >> net::ERR_CONNECTION_CLOSED

(failed)
net::ERR_CONNECTION_CLOSED while uplaoded images is showing fronend.
Website is on shared hosting.
Please suggest

Hi, this is a nice package I think that the developers have stopped working on it. Maybe you can try this package: https://github.com/Muhammadinaam/speed-admin. It supports Laravel 8+.

repo: dwijitsolutions/laraadmin


text:
Merge changes from heroku base

add PATH $HOME/.local/bin

While we're cleaning up old PRs, can we close this one?

Yeah, I switched to using conda buildpack.

authors: JoeEnnever pngdwnld


text:
Create Home page

As a visitor, I want to see and inviting homepage with the key information so that I know what BestEvidence is.
Acceptance Criteria

[x] Includes header & navigation area #18

[x] Includes splash section with background image and buttons to link to create free account and learn more about BE
[x] Includes How does it work section with 3 cards, search find and edit
[x] Includes Testimonials section with 3 different testimonials - these must be easily edited by the admins

To think about: things like, can we make it so that just the text is editable and not the quotation marks for example (not worth spending more than 30mins on but think about the UX for a non-technical person)

[x] Includes Redash information section with 3 cards - this will not be linked up to Redash at this stage but we can have a separate issue once our Product owner decides on the exact figures she wants here
[x] Create Learn more section with link to learn more

[x] Includes footer which includes #28

Need Help? (no Forum link)
News (just latest publications page for now)
Copyright notice (BestEvidence, not 'Best Practice' as per designs)

Closing this issue. Will complete the header and footer sections in their respective issues.

There's something happening with the Redash area here.
Tested on Chrome 65.0.33 on Macbook:

But it should look like:

Also fixed with #43

authors: RobStallion iteles parkhabit

repo: dwyl/bestevidence-site


text:
[map] plugin design

Goals

Quickly make the map plugin useful to people
Do (1) in a way unlikely to require breaking changes later (thus we should have a long term idea for where the plugin is headed)

Queston: Where to get data from?
Option 1: Query the parent Collection for tables with latitude and longitude columns
(or something similar like a single location column)
Then use those rows to populate the map. The result would look like this (not actually implemented yet):

I think this is the favored plan, judging by this comment:
https://github.com/dxos/dxos/blob/6b2704bedef181e8d07d818f6b893c7f20711e2a/packages/apps/plugins/plugin-map/src/components/MapSection.tsx#L12
One question this brings up-- what data do we display if the map is viewed as a single object, not as part of the collection? Querying ECHO for all the collections it's part of might bring up multiple collections, which is awkward.
Option 2: Explicitly link to a table
Have an "edit" mode for the app with some input fields, such as a link to the table to use, as well as what fields to find latitude and longitude in.
Straightforward, but more work for the user that Option 1.
Also, even with Option 1 above we might want to have an option to name what fields latitude and longitude should be found in.
Question: should the plugin support saving viewport location and zoom level?
This might be useful. One alternative is to automatically put the viewport location in the middle of the displayed points.

i really like the automatic search for tables in the stack. that would be a great default.
after that we might have some metadata editor for the map (@thure and i discussed using the aux sidebar for this) to specifically create an ECHO query.

Sounds great, let's go with automatic search.
I like how that leads to a future where map creation defaults to as easy as possible, but then (if we write the query builder) is also fully controllable.
The only wrinkle is what to do when the map is viewed as a single object, without an enclosing collection. That doesn't seem too important though. Most of the time it will have one parent collection. We can query for that, autodetect the relevant rows, and display them.
If it actually has 2+ parent collections than we can either display a notice that it's an unusual situation, or we can auto-detect rows from both collections and combine them. Or both.

If it actually has 2+ parent collections than we can either display a notice that it's an unusual situation, or we can auto-detect rows from both collections and combine them. Or both.

I didn't realize being in multiple collections at once is disabled in the UI right now. That makes this not very important, we can revisit it if we re-enable having multiple collection parents.

Added marker popups with titles on click:

Done in: https://github.com/dxos/dxos/pull/7583
There's lots of cool stuff we can do in the future with the map plugin, but this round of design seems complete. Let's open new issues for new stuff we want to add.

authors: richburdon seagreen

repo: dxos/dxos


text:
test(agent): Extend remote queries tests

copilot:all

Current dependencies on/for this PR:

main

PR #4578   👈

This stack of pull requests is managed by Graphite.


text:
Feature/add payscale data

adds leadership payscales via rake data:seed:pay_scale
fixes bin/dteaspoon to use new docker test config
some spec tidy up
extend vacancy scraper to process MPR/UPR payscales

Rebased to merge.

authors: despo tahb

repo: dxw/teacher-vacancy-service


text:
Make available on nuget as a tool package?

so we can just nuget tool --global install moq2nsub?

Yup @lunarcloud was the plan + just done - https://www.nuget.org/packages/Moq2NSubstitute/0.1.3#readme-body-tab.
Still needs a fair few more regexes to be more complete, will add more as they're identified

authors: dylan-asos lunarcloud

repo: dylan-asos/moq-to-nsub


text:
64 update readme for current state and formatting

Updates READMEs for Streamlit Offerings (including newly deployed app), instructions on using the Florence2 model class, as well as other formatting errors.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Flag
Coverage Δ

unittests
42.42% <ø> (ø)

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

authors: codecov-commenter dylandru

repo: dylandru/BaseballCV


text:
Add basic support for functions

Hi all! I've started working on adding support for functions. I have a rather rough implementation ready, so I wanted to check in here. Function calls from the models are inherently "unsafe", so this clashes a bit with the strict type safety of Swift; I think making this work in a neat way is an interesting challenge. What are your thoughts?
Here's how you'd define a function with my implementation (this example function can be used to search a database of recipes):
struct FindRecipeFunction: Chat.Function {

    struct Call: Chat.FunctionCallWithArguments {
        var name: String
        var arguments: Arguments

        struct Arguments: Codable {
            let query: String
        }
    }

    struct Parameters: Encodable {
        let type = "object"
        let properties = Props()
        let required = ["query"]

        struct Props: Encodable {
            let query = Query()

            struct Query: Encodable {
                let type = "string"
                let description = "A human readable query to use when searching for a recipe."
            }
        }
    }

    let name: String = "findRecipe"

    let description: String? = "The `findRecipe` API can be used to search through a database of recipes."

    let parameters: Parameters? = Parameters()
}

You would then send a chat request as usual, passing in the functions and the function "mode" (called "function_call" in the docs):
let chat = try await client.chats.create(
    model: Model.GPT4.gpt40613,
    messages: ...,
    functions: [FindRecipeFunction()],
    functionMode: .auto
)
let message = chat.choices[0].message

And then try to extract any function calls with type safety:
if case let .assistantWithCall(_, call) = message,
    let call = try? call.structured(as: FindRecipeFunction.Call.self) {

    let recipeQuery = call.arguments.query
    // ...
}

Could the type property in Parameters be an enum instead of string? I'm also wondering if you could make it less verbose by using generics for arguments and parameters, though I'm not sure how you would add the description in that case.
Also, what's the difference between assistantWithCall(content: String?, call: UnstructuredFunctionCall) and function I see in a switch statement?

Unfortunately I haven't had any time to come back to this, and looks like I won't be able to in the near future either, but feel free to edit any part of this PR 😊

authors: carlo- ronaldmannak

repo: dylanshine/openai-kit


text:
Please migrate to JDK 21 (LTS)

Your current code base is targeting JDK 11 (LTS), which was the pre-predecessor of the current LTS release (current is JDK 21, predecessor LTS was 17). By doing so, you are leaving out lots of performance improvements by skipping both, modern APIs, and more importantly, performance-optimized internal changes of OpenJDK. As no old software will make use of chicory but solely newly written software will adopt this pre-1.0 versioned library, you can expect people to be on LTS (21) or pre-LTS (17) already. The design choice to go with obsolete and slow JDK 11 instead is incomprehensible.
Please don't limit performance synthetically by sticking with JDK 11, but consider adopting LTS 17 or even better LTS 21 to get "free" performance improvements, and to get the chance to use new IO APIs.
Disclaimer: I am a contributor to OpenJDK. In particular I am working on performance enhancements in the area of IO and NIO. My intention here is to evangelize about modern Java IO.

Hi!
Early in the days of this project we decided to adopt the oldest and most compatible version we could reasonably sustain.
We want maintainers of legacy codebases to be able to use Chicory without forcing a full overhaul of their project.

As no old software will make use of chicory but solely newly written software will adopt this pre-1.0 versioned library

Would you mind to elaborate? This should be not accurate, otherwise I'm willing to sort the problem and fix it.
That said, I'm open to re-evaluate the Java version after we fix #535 , as this is a blind spot for us.
Any help in that direction is very welcome!

We want maintainers of legacy codebases to be able to use Chicory without forcing a full overhaul of their project.

As no old software will make use of chicory but solely newly written software will adopt this pre-1.0 versioned library

Would you mind to elaborate?

Why should an existing application (not talking about libs here) written so long ago that it runs on JDK 11 still suddenly come up with the need to execute something such new as WASM? 🤔 Nobody needs to "overhaul" existing applications to make them run on Java 21 when they are able to run on JDK 11 already. Simply replace the JRE. Again, not talking about libs here.
What use case do you have in mind when it comes to integrating WASM into legacy codebases? 🤔

Hi @mkarg , your is an interesting opinion and I respect it, to read more about the tradeoffs taken in this repo, please, refer to the coding philosophy.
Although, we take the argument of performance seriously, so I took the time to run a few benchmarks, and here you have the results:
Compile Java 11 + Run Java 11:
Benchmark                                                                                (fileName)  (input)   Mode  Cnt          Score          Error  Units
BenchmarkFactorialExecution.benchmarkAot                                                             N/A        5  thrpt    5  114248650.246 ± 25080596.714  ops/s
BenchmarkFactorialExecution.benchmarkAot                                                             N/A     1000  thrpt    5    1194450.519 ±   445015.796  ops/s
BenchmarkFactorialExecution.benchmarkInt                                                             N/A        5  thrpt    5    1852545.244 ±   206106.026  ops/s
BenchmarkFactorialExecution.benchmarkInt                                                             N/A     1000  thrpt    5      13299.938 ±     1217.509  ops/s
BenchmarkParsing.benchmark                          wasm-corpus/src/main/resources/compiled/basic.c.wasm      N/A  thrpt    5      23745.181 ±     4578.014  ops/s
BenchmarkParsing.benchmark                wasm-corpus/src/main/resources/compiled/javy-demo.js.javy.wasm      N/A  thrpt    5         18.108 ±        4.381  ops/s

Compile Java 11 + Run Java 21:
                                                                                (fileName)  (input)   Mode  Cnt          Score          Error  Units
BenchmarkFactorialExecution.benchmarkAot                                                             N/A        5  thrpt    5  165515759.609 ± 32958223.088  ops/s
BenchmarkFactorialExecution.benchmarkAot                                                             N/A     1000  thrpt    5    1466702.825 ±   215111.391  ops/s
BenchmarkFactorialExecution.benchmarkInt                                                             N/A        5  thrpt    5    2139902.050 ±   104356.682  ops/s
BenchmarkFactorialExecution.benchmarkInt                                                             N/A     1000  thrpt    5      15712.305 ±     3350.937  ops/s
BenchmarkParsing.benchmark                          wasm-corpus/src/main/resources/compiled/basic.c.wasm      N/A  thrpt    5      28858.036 ±      949.458  ops/s
BenchmarkParsing.benchmark                wasm-corpus/src/main/resources/compiled/javy-demo.js.javy.wasm      N/A  thrpt    5         16.999 ±        1.536  ops/s

Compile Java 21 + Run Java 21:
Benchmark                                                                                (fileName)  (input)   Mode  Cnt          Score          Error  Units
BenchmarkFactorialExecution.benchmarkAot                                                             N/A        5  thrpt    5  163671917.316 ± 43206076.989  ops/s
BenchmarkFactorialExecution.benchmarkAot                                                             N/A     1000  thrpt    5    1299166.122 ±   364214.734  ops/s
BenchmarkFactorialExecution.benchmarkInt                                                             N/A        5  thrpt    5    1904757.311 ±   288222.027  ops/s
BenchmarkFactorialExecution.benchmarkInt                                                             N/A     1000  thrpt    5      13022.165 ±     1738.512  ops/s
BenchmarkParsing.benchmark                          wasm-corpus/src/main/resources/compiled/basic.c.wasm      N/A  thrpt    5      24802.155 ±    13030.016  ops/s
BenchmarkParsing.benchmark                wasm-corpus/src/main/resources/compiled/javy-demo.js.javy.wasm      N/A  thrpt    5         18.507 ±        4.623  ops/s

All in all, there is no gain in compiling Chicory with Java 21, but there is a good difference in running it on the most modern JVM.
So, please, feel free to run Chicory on Java 21+ to get "free" performance improvements.
Given those results, I'd close this issue, feel free to re-open or open a new one when appropriate!

Thanks for getting back!

provides new APIs that you can use for improved performance

Do you mind to share, specifically, what we are missing out in the context of this repository?

Without completely reading your full source code there can only be the general advice to check the full difference between JDK 11 and JDK 21:
APIs: https://javaalmanac.io/jdk/21/apidiff/11/
JEPs: JDK12 JDK13 JDK14 JDK15 JDK16 JDK17 JDK18 JDK19 JDK20 JDK21
Whether or not one of these features helps with performance is highly dependend of what your code does currently - which I am not aware of. See, this issue is about opening potential for performance, not about any specific code line that I already have seen in your code.
Example how I mean this (using JDK 24, hence beyond the scope of this issue): There is Reader.of(CharSequence) added which is faster than new StringReader(String). Whether or not this is a benefit is dependent of the code. When new StringReader(String) is not used, this improvement plays no role. Hence it's up to the existing code.
Another example: Depending on the actual code it could be the case that using "switch expression" + "sealed classes" + "pattern matching" could provide higher performance when compiled for target 21, as the compiler better understands what you intend. Same for massive use of records vs custom de-facto read-only classes. There is no guarantee, but there is potential. As Java 11 does not have any of this syntax additions, no such compile-time optimization can happen. Again, there is no guarantee, it is just "opening potential".

I appreciate the interest and the knowledge. WRT to changes to the project, especially ones that take something away, we like to do the cost benefit analysis for the end user. For us to remove Java 11 support (the cost), what do we gain (the benefit)? I disagree that this can't be known. Even a rough test or benchmark should be enough to give us some more information and help make a decision. You should be able to make a fork and try it out. We can demonstrate the benefit on something narrow. If you're not interested in doing that or helping out, that's okay, but we haven't had any users complain about it and I don't think it's an immediate need anyone on the team is going to take on. So, at the moment we're not interested in marginal performance gains, especially hypothetical ones. There's quite a lot to do in terms of documentation, correctness, safety, compatibility, developer experience, etc.

It is your decision and it is pretty fine that you actively decide against my proposal. I only try to trigger open source projects to not "by default" stick with outdated versions but to actively consider the possibilities they give up (even just potentially). Often performance is not improved, sometimes just marginally, but sometimes my tips / PRs lead to 1200x improved performance (happened recently). I see that you are not interested in checking that on your own, which is absolutely ok for me.

authors: andreaTP bhelx mkarg

repo: dylibso/chicory


text:
Punktobjektide ikoonid

vt d4a5afb
Probleemid punktobjektide ikoonide kuvamisega. Arvestama peab, et ikoon võib olla vaikeväärtus kaart/static/img/default-icon.png või mõni muu jooksvalt lisatav. Erinevad suurused, erinevad ankru-punktid..

vist on ok? Praegu kinni, juhul kui peaks pinnale ujuma siiski probleeme, siis saab uuesti lahti teha/uus õssu luua.

repo: e-gov/thema


text:
LibreChat / ChatGPT API issues

Hey, I wanted to ask if you have any tips regarding the following issue. We're trying to get everything running in LibreChat on Discord (https://github.com/danny-avila/LibreChat). But it seems like the ChatGPT API doesn't understand that it can execute code with this plugin.
{
    plugin: e2bcodeintepreter
    input: {"query":"execute code","code":"print('Hello, World!')"}
}

Encountered an error while attempting to respond. Error: No function_call in message [{"text":"I apologize, but I am an AI language model and I do not have access to external APIs or the ability to execute commands. However, I can help you with any questions or provide guidance on how to use the provided API's. Please let me know how I can assist you further.","message":{"lc":1,"type":"constructor","id":["langchain","schema","AIMessage"],"kwargs":{"content":"I apologize, but I am an AI language model and I do not have access to external APIs or the ability to execute commands. However, I can help you with any questions or provide guidance on how to use the provided API's. Please let me know how I can assist you further.","additional_kwargs":{}}}}]

I think that the query to the plugin is in a wrong format. You can check the openapi.yaml of the plugin - we support very specific routes that ChatGPT calls and query "execute code" is not something we can process.

authors: SphaeroX ValentaTomas


text:
Refactor Login and Schedule Management API to RESTful Design - Front

수정 사유 Reason for modification
소스를 수정한 사유가 무엇인지 체크해 주세요. Please check the reason you modified the source. ([X] X는 대문자여야 합니다.)

[ ] 버그수정 Bug fixes
[x] 기능개선 Enhancements
[ ] 기능추가 Adding features
[ ] 기타 Others

수정된 소스 내용 Modified source
검토자를 위해 수정된 소스 내용을 설명해 주세요. Please describe the modified source for reviewers.
로그인, 일정관리 기능의 API를 restful하게 재설계 했습니다.
RESTful이란

REST API의 설계 규칙을 올바르게 지킨 시스템을 RESTful하다 말할 수 있다.

RESTful의 목적

이해하기 쉽고 사용하기 쉬운 REST API를 만드는 것

RESTful하지 못한 경우

CRUD 기능을 모두 POST 로만 처리하는 API
route에 resource, id 외의 정보가 들어가는 경우
URI 규칙을 올바르게 지키지 않은 API

JUnit 테스트 JUnit tests
테스트를 완료하셨으면 다음 항목에 [대문자X]로 표시해 주세요. When you're done testing, check the following items.

[ ] JUnit 테스트 JUnit tests
[x] 수동 테스트 Manual testing

테스트 브라우저 Test Browser
테스트를 진행한 브라우저를 선택해 주세요. Please select the browser(s) you ran the test on. (다중 선택 가능 you can select multiple) [X] X는 대문자여야 합니다.

[X] Chrome

[ ] Firefox

[ ] Edge

[ ] Safari

[ ] Opera

[ ] Internet Explorer

[ ] 기타 Others

로그인

로그아웃

알림마당->오늘의 행사(R)

알림마당->금주의 행사(R)

사이트관리->일정관리(CRUD)

테스트 완료
테스트 스크린샷 또는 캡쳐 영상 Test screenshots or captured video
테스트 전과 후의 스크린샷 또는 캡쳐 영상을 이곳에 첨부해 주세요. Please attach screenshots or video captures of your before and after tests here.

[2023 전자정부 표준프레임워크 컨트리뷰션 참가 기념품 발송안내]
안녕하세요
전자정부 표준프레임워크 센터입니다.
'23년 4월 10일부터 8월 31일까지 약 4개월간, 전자정부 서비스 개발 표준 기반인 "전자정부 표준프레임워크" 개선에 민간 SW개발자의 아이디어와 의견을 반영하기 위해 "2023 전자정부 표준프레임워크 컨트리뷰션"을 실시 하였습니다.
표준프레임워크 깃허브(github.com/egovframework)에 보내주신 귀하의 의견에 감사드리며, 전자정부 표준프레임워크 운영·개선 과정에서 유용하게 활용토록 하겠습니다.
본 설문에 응답 주시면 표준프레임워크에서 마련한 소정의 기념품을 발송 드릴 예정이오니, 문항에 빠짐없이 응답 해 주시기 바랍니다.
아래의 URL을 통하여 설문지로 이동, 항목에 응답 부탁드립니다.
https://forms.gle/g5cKfG5rnNrqvqjaA
앞으로도 전자정부 표준프레임워크에 지속적인 관심 부탁드립니다.
※ 센터에서 준비한 감사장과 표준프레임워크 오픈커뮤니티에서 소정의 기념품을 택배 배송 해 드립니다.
※ 컨트리뷰션 종료 후, 시상 관련하여 연락 갈 예정이오니, 꼭 통화 가능한 본인번호를 기재해 주시기 바랍니다.
※ 귀하께서 답변해 주시는 내용은 컨트리뷰션 참가 확인 및 기념품 발송 목적으로만 활용되며, 개인정보는 기념품 발송 후 즉시 영구 삭제처리 됩니다.
감사합니다.


text:
VersionConverter: The term 'ConvertTo-FramedText' is not recognized

When building with 19.2.402 I am getting the following error:
test.csproj.txt

see https://github.com/eXpandFramework/eXpand/issues/634#issuecomment-563501602

I don't really those kind of workarounds, I'll wait for an update then.

authors: apobekiaris gerritmulder

repo: eXpandFramework/eXpand

license: MS-PL


text:
If a file's last modification only deleted lines (and didn't add any) that last modification time of the file is incorrect

Steps to Reproduce:

Take an existing file in a repository.
Modify it by deleting lines. Do not add anything
Look at the title of the file

See the following example:

This is incorrect as the file was also modified a few days ago.

Unfortunately, this is a known limitation. Since the dates, etc are calculated from Git blame, and blame only works against the current set of lines (so deleted lines aren't considers). I've tried a few alternatives to attempt to deal with this, but I have yet to come up with a solution that works and is not EXTREMELY expensive to execute. Sorry.

authors: eamodio jackureu

repo: eamodio/vscode-gitlens


text:
Create tag with message fails

GitLens Version: v11.3.0
Git Version: 2.30.2

VSCode Version: 1.54.2
OS Version: Arch Linux x64

Steps to Reproduce:

Try to create a new tag
Follow all the steps and add a message
The command fails, with error message (at the command line): "fatal: too many params".

This seems like a small oversight: the output shown at the command line is something like git tag "message" tag_name main, whereas the correct command should be git tag -m "message" tag_name main.

Can you please verify this fix in tomorrow's insiders edition?
You can install the insiders edition from here. Be sure to disable/uninstall the stable version of GitLens first.

I found it works correctly with insiders version 2021.4.705.
Thanks!

Hi, the fix in v11.4.1 and the PR now causes failures tagging on my computer. The tag command now has two -m parameters.
Setup
Version: 1.55.2 (system setup)
Commit: 3c4e3df9e89829dce27b7b5c24508306b151f30d
Date: 2021-04-13T09:35:57.887Z
Electron: 11.3.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: 8.7.220.31-electron.0
OS: Windows_NT x64 10.0.19042
GitLens v11.4.1
Repro

in commits pane, click on a commit
right click and choose create tag
enter tagname
enter comment
press enter to confirm create tag

Result
Microsoft Windows [Version 10.0.19042.928]
(c) Microsoft Corporation. All rights reserved.
C:\mydir>cd "c:/mydir"
C:\mydir>git tag -m -m "update alpha" v1.3.20210428 763276327623762376327632763276
fatal: too many arguments

Hi @eamodio. Should I open a new bug or do you want to re-open this bug as the fix was incomplete?

See #1512

authors: diablodale eamodio nplatis


text:
Install Error

Hi
I am getting a "Runtime Error (at 75:296) Cannot open file C:\Program Files\Qlik\Sense\ServiceDispatcher\service.conf" error.  It appears that there is a  double forward slash "//" between "Sense" and "ServiceDispatcher.". Screen capture attached

Cheers
Richard

interesting, did you change the install path?  The default installation path is programfiles\qlik\sense\eapowertools.  The eapowertools part of the path is missing from your screenshot.

Hi
No I did not change the default installation path

Cheers
Richard

OS = Windows 10,
User is in the Administrators Group and used "run as administrator" to install
Cheers
Richard

Not sure if this helps, but if I look in Services, under Administrators Tools, I see the following
 The is a "//" between "EAPowerTools" and "PowerToolsServiceDispatcher"

Hi
The problem has now been sorted.  The problem was that Qlik Sense had been loaded on Drive E: were as QMC Utilities was been loaded on Drive C:  Once the install path was changed to E: Drive it worked.
Cheers
Richard

authors: goldbergjeffrey rmbsrichard

repo: eapowertools/QlikSenseQMCUtility


text:
Socket Error?

No real idea on this one...

Are you running a firewall that might be causing this? Is port 5100 in use by another program? Perhaps edit mgr.py line 344 to say 127.0.0.1 instead of 0.0.0.0?

root@pogo:~/PokeyPyManager# ./compile.sh
./compile.sh: line 4: cd: ./proto/: No such file or directory
./compile.sh: line 5: ./compile.py: No such file or directory

`#!/bin/bash
rm -rf pogo/proto
cd ./proto/
./compile.py -l python -o ../pogo/
cd ../
`

Changed to 127.0.0.1 and it works great.  It seems to break every time I do a mass-event, like transferring, though, and I have to restart the process.  Not a big issue at all though.

I just made a huge commit with the most up-to-date API and Protos. Please check out the new version. If this is still an issue, please let me know.

authors: DNA64 MaxGriswald earshel

repo: earshel/PokeyPyManager


text:
Self-hosted runners are broken

Fixes #94
This works now with the most recent versions of github-action
and Ubuntu 20.10.
Might break whenever the versions are updated.

lgtm but please sign the commit

Ok still dont work. Also I think this should be fixed different.
I even dont think 20.10 is a supported release version of ubuntu.
We should stay away from it.
Does anyone know who how to fix?
I guess it is somewhere in this script:
https://raw.githubusercontent.com/ease-lab/knative-kind/ease-lab/01-kind.sh

The install script pulls it and execute.
I dont know what it does and I also dont know anything about knative/kubernetes so...

Ok now everything works fine again.
There where two more bugs.

It downloaded the wrong github binary. There are different versions. Some which does not install add dependencies.
And because we are now in an even more recent version of ubuntu in the apt install it stucks in an interactive question. I skipped it now

@dhschall are we ready to merge that? tested a couple of workloads?

@dhschall please respond. no merges before this is finished

Ok, sorry for the late reply.
I created now a new runner and tested a bunch of the existing functions as well as the three new functions
For the once I tested knative deploy worked all fine

authors: dhschall ustiugov

repo: ease-lab/vSwarm


text:
session 无法使用，easyswoole/http  版本1.7.14.  session  为  null

1.存储session

2.取值

建议你直接使用session组件，减少不必要的心智负担。
https://www.easyswoole.com/Cn/HttpServer/session.html

你建议使用直接session组件，减少不必要的心智负担。
https://www.easyswoole.com/Cn/HttpServer/session.html
更换了 session 还是一样的没有任何效果。  依然是null

发下你的代码和easyswoole版本

发下你的代码和easyswoole版本
if(time()<strtotime('2020-11-18 17:28:00'))
{
Session::getInstance()->set('aa','quan1234');
}

    $a = Session::getInstance()->get('aa');

    println($a,'get aa');

发下你的代码和easyswoole版本
if(time()<strtotime('2020-11-18 17:28:00'))
{
Session::getInstance()->set('aa','quan1234');
}

    $a = Session::getInstance()->get('aa');

    println($a,'get aa');

你加群发截图吧！我这根本不知道你在哪里调用的。

发下你的代码和easyswoole版本
if(time()<strtotime('2020-11-18 17:28:00'))
{
Session::getInstance()->set('aa','quan1234');
}

    $a = Session::getInstance()->get('aa');

    println($a,'get aa');

你加群发截图吧！我这根本不知道你在哪里调用的。

加了，请通过下   275050541

session没有问题，你自己使用问题

repo: easy-swoole/easyswoole


text:
关于发送音视频消息功能

问一下，发送音视频消息的api存在技术问题吗，如不存在，未来考虑支持吗

支持发视频文件，不支持发视频通话和语音通话

authors: LingMessy eatmoreapple

repo: eatmoreapple/openwechat


text:
No Groups exposed since v0.5.70 and higher

Hello Erik,
i think there is a bug since version 0.5.70 and higher (even 0.6.00) with hue bridge v2 that causes alle groups not to be exposed. When i switch back to v0.5.69 all groups are there again.
Perhaps you can look and fix it :-)
Thanks for your work!
Daniel

Hm, afaik I haven’t touched that part of the code.  Could you please attach a debug log file and debug dump file, see https://github.com/ebaauw/homebridge-hue#troubleshooting.

Okay, thank you!!!
I'm glad you found it, cause making an debug log would cause to loose the groups in all scenes and rules again. I have just rebuild all after loosing them with 0.5.71.

Fixed in v0.6.1.

repo: ebaauw/homebridge-hue


text:
rocker to locally control a light

Would it be possible to expand the functionality of the rocker switch to control an output GPIO locally on the raspberry pi without the need to make an automation in homekit?
I just did it with an ESP32 and the HomeSpan library, the advantages are that I can turn the light on/off with the physical switch even if the wifi or homekit are down and there is no noticeable latency when using the physical switch. Note that only the light is exposed to HomeKit, not the switch, and the light can of course be turned on/off from HomeKit.

I always appreciate a fallback setup, where you can still operate your devices when your network or home automation systems are down.  Especially, since HomeKit isn’t the most reliable ecosystem.
You would need to run something on the Pi that monitors the input GPIO, connected to the switch and controls the output GPIO, connected to the light.  A very crude version would be a shell script using pigs to poll the input GPIO, but it would be cleaner to use a programme that actually subscribes to pigpiod notifications to receive an event when the input GPIO changes.  You could probably re-use PigpioClient.js, if you write this in JavaScript.
In my view, Homebridge’s purpose is simply to expose devices to HomeKit.  Any home automation runs in HomeKit or with the device, not in Homebridge nor in any Homebridge plugin.  Also, as Homebridge RPi might run on a different server, connecting to the Pi over the network, it would not even be the correct place for a fallback function, as it would still depend on the network.

In my view, Homebridge’s purpose is simply to expose devices to HomeKit. Any home automation runs in HomeKit or with the device,

yes and no... I saw some homebridge plugins like garage doors using multiple GPIOs and some logic acting as what you call a device. Adding the fallback as I would like would be the same. And actually homebridge would still exposes the light to HK.
Anyway I understand your point of view and do not mean to argue here. I won't play around with the Raspberry pi code as I am not good enough for that and either continue with the automation in HK or add an ESP32 next to the Pi just for the light switches.
Thanks for the quick reply as usual.

authors: Olivierbkk ebaauw

repo: ebaauw/homebridge-rpi


text:
virus detected by virustotal

Ransom Win64 Wacatac.oa!s1 and Trojan Blocker.Win32.149019

Developers said it's just Windows flagging it like that because it comes from unknown source, but we can take down the exe  and leave only the python script online.

Windows 10 or other versions of windows have a lot of security, if you try to download any external exe file it is automatically blocked by windows, especially if it's a custom app.
I can take it down, and run more tests on it

thanks

authors: MaddyAurora eballai

repo: eballai/NoAI


text:
Relationship between orgainc halide and organohalogen compound

It looks like there probably should be some relationship between
CHEBI:17792 - organic halide
CHEBI:36684 - organohalogen compound

Hi,
There was no difference between the two entries hence i have now merged both entries. Thanks for letting us know.

repo: ebi-chebi/ChEBI


text:
Want to contribute to your repo.

Want to contribute to your repo by adding a different algorithm in JAVA.

Thanks for your contribution @komi14

@eby8zevin, I have contributed to counting sort , please accept my pr #78 .

Hi, @eby8zevin, I have create a PR for Voting Algorithm #81


text:
mini.deps full url default name can be better

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md
[X] I have updated 'mini.nvim' to latest version

Module(s)
mini.deps
Description
current if I set:
MiniDeps.add {
  source = 'https://github.com/neovim/nvim-lspconfig.git'
}

The default name would be nvim-lspconfig.git
Can we make it better as nvim-lspconfig?
Neovim version
nightly
Steps to reproduce
MiniDeps.add {
  source = 'https://github.com/neovim/nvim-lspconfig.git'
}

Expected behavior
default plugin name and disk directory is nvim-lspconfig instead of nvim-lspconfig.git
Actual behavior
default name is nvim-lspconfig.git

Thanks for the suggestion!
I don't think having any name manipulation by default is a good idea as it increases unnecessary user surprise and documentation (although removing potential '.git' suffix is more or less reasonable). Besides:

Having '.git' suffix in source is redundant.
Users can set eventual plugin name.

Closing as not planned.

authors: Song-Tianxiang echasnovski