Enhancement: Crud-generator cli command similar to Nest JS

What's the feature you'd like to ask for.
I was a good user of Nest JS framework and was looking for a similar framework in python and was happy that I found 'Starlite'. So as I started exploring the 'Starlite' framework, I kind of missing tools similar to nest-cli. It enables the scaffolding of a project easily.
Wondering if starlite has any plans to implement a cli tools.
Additional context
Reference: https://docs.nestjs.com/recipes/crud-generator

This would be a great feature.  We'll be releasing a repository and service pattern for SQL Alchemy in an upcoming release, so these types of CLI enhancements could be really helpful for application scaffolding.

Hi, is this still open?
Is it ok to use click for this?
Also, should this be in this same repo or in a new one, e.g. litestar-cli?

Hi, is this still open?. I would like to do a prototype for this. Is it ok to use click for this? Also, should this be in this same repo or in a new one, e.g. litestar-cli?

Thanks for the intuitive @aarcex3 I believe this user @ch-iv has a "work in progress" repo going on here https://github.com/ch-iv/litestar-manage
Maybe you can join efforts there if you are both willing?


[audiofile] Bump to 1.1.2

[x] Changes comply with the maintainer guide.
[x] SHA512s are updated for each updated download.
[x] The "supports" clause reflects platforms that may be fixed by this new version.
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

The new version need re-make patche:
-- Applying patch fix-cmakeLists.patch
CMake Error at scripts/cmake/z_vcpkg_apply_patches.cmake:34 (message):
  Applying patch failed: Checking patch CMakeLists.txt...

  error: while searching for:

  #===============================================================================

  set (CMAKE_SUPPRESS_REGENERATION true)

  error: patch failed: CMakeLists.txt:42

  error: CMakeLists.txt: patch does not apply

Looks the patch can't delete.
1> CMake generation started for default configuration: 'x64-Debug'.
1> Found and using vcpkg toolchain file (F:/Feature-test/vcpkg/scripts/buildsystems/vcpkg.cmake).
1> Command line: "C:\WINDOWS\system32\cmd.exe" /c "%SYSTEMROOT%\System32\chcp.com 65001 >NUL && "C:\PROGRAM FILES\MICROSOFT VISUAL STUDIO\2022\COMMUNITY\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\CMake\bin\cmake.exe"  -G "Ninja"  -DCMAKE_BUILD_TYPE:STRING="Debug" -DCMAKE_INSTALL_PREFIX:PATH="C:\Users\test\source\repos\CMakeProject1\out\install\x64-Debug" -DCMAKE_C_COMPILER:FILEPATH="C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.41.34120/bin/Hostx64/x64/cl.exe" -DCMAKE_CXX_COMPILER:FILEPATH="C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.41.34120/bin/Hostx64/x64/cl.exe"   -DCMAKE_MAKE_PROGRAM="C:\PROGRAM FILES\MICROSOFT VISUAL STUDIO\2022\COMMUNITY\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\Ninja\ninja.exe" -DCMAKE_TOOLCHAIN_FILE="F:/Feature-test/vcpkg/scripts/buildsystems/vcpkg.cmake" "C:\Users\test\source\repos\CMakeProject1" 2>&1"
1> Working directory: C:\Users\test\source\repos\CMakeProject1\out\build\x64-Debug
1> [CMake] CMake Error at out/build/x64-debug/CMakeFiles/3.29.5-msvc4/CMakeSystem.cmake:6 (include):
1> [CMake]   include could not find requested file:
1> [CMake]
1> [CMake]     F:/Feature-test/openfbx/scripts/buildsystems/vcpkg.cmake
1> [CMake] Call Stack (most recent call first):
1> [CMake]   CMakeLists.txt:12 (project)
1> [CMake] CMake Error at CMakeProject1/CMakeLists.txt:7 (find_package):
1> [CMake]   Could not find a package configuration file provided by "AudioFile" with
1> [CMake]   any of the following names:
1> [CMake]
1> [CMake]     AudioFileConfig.cmake
1> [CMake]     audiofile-config.cmake
1> [CMake]
1> [CMake]   Add the installation prefix of "AudioFile" to CMAKE_PREFIX_PATH or set
1> [CMake]   "AudioFile_DIR" to a directory containing one of the above files.  If
1> [CMake]   "AudioFile" provides a separate development package or SDK, be sure it has
1> [CMake]   been installed.
1> [CMake]
1> [CMake]
1> [CMake] -- Configuring incomplete, errors occurred!
1> 'C:\WINDOWS\system32\cmd.exe' '/c "%SYSTEMROOT%\System32\chcp.com 65001 >NUL && "C:\PROGRAM FILES\MICROSOFT VISUAL STUDIO\2022\COMMUNITY\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\CMake\bin\cmake.exe"  -G "Ninja"  -DCMAKE_BUILD_TYPE:STRING="Debug" -DCMAKE_INSTALL_PREFIX:PATH="C:\Users\test\source\repos\CMakeProject1\out\install\x64-Debug" -DCMAKE_C_COMPILER:FILEPATH="C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.41.34120/bin/Hostx64/x64/cl.exe" -DCMAKE_CXX_COMPILER:FILEPATH="C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.41.34120/bin/Hostx64/x64/cl.exe"   -DCMAKE_MAKE_PROGRAM="C:\PROGRAM FILES\MICROSOFT VISUAL STUDIO\2022\COMMUNITY\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\Ninja\ninja.exe" -DCMAKE_TOOLCHAIN_FILE="F:/Feature-test/vcpkg/scripts/buildsystems/vcpkg.cmake" "C:\Users\test\source\repos\CMakeProject1" 2>&1"' execution failed with error: ''C:\WINDOWS\system32\cmd.exe' '/c "%SYSTEMROOT%\System32\chcp.com 65001 >NUL && "C:\PROGRAM FILES\MICROSOFT VISUAL STUDIO\2022\COMMUNITY\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\CMake\bin\cmake.exe"  -G "Ninja"  -DCMAKE_BUILD_TYPE:STRING="Debug" -DCMAKE_INSTALL_PREFIX:PATH="C:\Users\test\source\repos\CMakeProject1\out\install\x64-Debug" -DCMAKE_C_COMPILER:FILEPATH="C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.41.34120/bin/Hostx64/x64/cl.exe" -DCMAKE_CXX_COMPILER:FILEPATH="C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.41.34120/bin/Hostx64/x64/cl.exe"   -DCMAKE_MAKE_PROGRAM="C:\PROGRAM FILES\MICROSOFT VISUAL STUDIO\2022\COMMUNITY\COMMON7\IDE\COMMONEXTENSIONS\MICROSOFT\CMAKE\Ninja\ninja.exe" -DCMAKE_TOOLCHAIN_FILE="F:/Feature-test/vcpkg/scripts/buildsystems/vcpkg.cmake" "C:\Users\test\source\repos\CMakeProject1" 2>&1"' returned with exit code: 1'.

@JonLiu1993 Please re-check this PR. I've test it on windows and osx, they both work.
The error you mentioned above is actually a historical problem of this port.
It looks like your compiled audiofile does not have this PR applied

@JonLiu1993

Tested usage successfully by audiofile:x64-windows


Address Unused Recursive Filter issue

Describe the bug

When running the following
pan-os-php type=address in=api://IP actions=display/delete 'filter=(object is.unused.recursive)'
The "unused" objects being filtered are actually being used

Expected behavior
Can this filter work to make sure these objects are not referenced anywhere in the XML?
Current behavior
Unused filters not showing used in Network / Interface section
Possible solution
N/A
Steps to reproduce

Create an address object
Add that object to an interface as an IP
Run the filter to search for unused objects and the delete action

Screenshots
* ** ERROR ** * API reported a failure: "error" with the following addition infos:   vl794 cds gw <IP_ADDRESS> cannot be deleted because of references from: network -> interface -> ethernet -> ethernet1/4 -> layer3 -> units -> ethernet1/4.794 -> ip -> vl794 cds gw <IP_ADDRESS> network -> interface -> ethernet -> ethernet1/4 -> layer3 -> units -> ethernet1/4.794 -> ip -> vl794 cds gw <IP_ADDRESS>

 *** Backtrace ***
0 ****
       backtrace_print()
       ::/tools/pan-os-php/lib/pan_php_framework.php line 631
1 ****
       derr()
       ::/tools/pan-os-php/lib/misc-classes/PanAPIConnector.php line 1319
2 ****
   PanAPIConnector:<IP_ADDRESS>
       PanAPIConnector::sendRequest() @
           /tools/pan-os-php/lib/misc-classes/PanAPIConnector.php line 1950
3 ****
   PanAPIConnector:<IP_ADDRESS>
       PanAPIConnector::sendDeleteRequest() @
           /tools/pan-os-php/lib/object-classes/AddressStore.php line 627
4 ****
   PANConf: / VirtualSystem:vsys1 / AddressStore:addresses
       AddressStore::API_remove() @
           /tools/pan-os-php/utils/common/actions-address.php line 82
5 ****
       {closure}()
       ::/tools/pan-os-php/utils/common/CallContext.php line 116
6 ****
   Action:'delete'
       CallContext::executeAction() @
           /tools/pan-os-php/utils/lib/UTIL.php line 1976
7 ****
       UTIL::time_to_process_objects() @
           /tools/pan-os-php/utils/lib/UTIL.php line 231
8 ****
       UTIL::utilStart() @
           /tools/pan-os-php/utils/lib/UTIL.php line 215
9 ****
       UTIL::__construct() @
           /tools/pan-os-php/lib/misc-classes/PH.php line 1207
10 ****
       callPANOSPHP()
       ::/tools/pan-os-php/utils/pan-os-php.php line 118
11 ****
       require_once()
       ::Command line code line 1

## Context

Your Environment

Version used:
Environment name and version (e.g. Chrome 59, node.js 5.4, python 3.7.3):
Operating System and version (desktop or mobile):
Link to your project:

thanks for mentioning this bug:
it is still the same as:
https://github.com/PaloAltoNetworks/pan-os-php/issues/764
Which is not fixed since a long time.
No clue when I can bring in better validation for all this Panorama template stuff, which also differ if they are used directly in Firewall


added StyleKit

Project URL

https://github.com/146BC/StyleKit
Description

Added StyleKit to UI Section
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 7 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

Thx!


Does not build on ARM by default

The .bazelrc file prevents building on ARM platforms (i.e. AWS Graviton) because:
https://github.com/google/hashtable-benchmarks/blob/1e2f25fd81d4a15c582ff3a171d0756ff5d98d5f/.bazelrc#L3
It works after removing this line, but it would be nice to build on ARM by default without editing build files.

Happy to entertain patches, I am not really a bazel expert


soroban-rpc: Update getLedgerEntry to use the internal storage

What
Update the getLedgerEntry JSON-rpc endpoint to use the internal ledger-entry storage provided by #322
I've also had to:

refactor the daemon.go module to make it testeable.
make the ledger storage stop grafefully when being closed (also to make it testable).
remove TestSimulateTransactionDeadlineError early (it wasn't worth fixing since we are going to remove the core dependency on simulateTransaction very soon.

Why
Core is deprecating its getLedgerEntry endpoint
Known limitations
N/A

Addresses https://github.com/stellar/go/issues/4720


use path provided rtcm converter for ntrip output [CPP-949]

having issues bundling rtcm3tosbp into qt app (unsure location of the binary),
this allows user to select any binary that consumes RTCM in stdin and grabs stdout when provided.
also just to allow testing purposes

with no converter provided, will just send RTCM data

Frontend and Release Workflow Started here

Frontend and Release Workflow Started here

Close?  I don't think we need this anymore.


Perform basic testing of aws extension through localstack

/cc @oscerd

Let me assign this to myself for the moment

There's localstack support from testcontainers https://www.testcontainers.org/modules/localstack/

The behaviour is not the same. We already seen this on camel-k with kinesis and other stuff

but could be enough to have some smoke tests ?
just to be sure that for example the integration can start and connect

@oscerd @ppalaga I think this can be closed right ?

Yes, I think so

Some AWS2 extensions still do not have proper tests, but I know which ones they are, so no problem with closing this one.


Variable initialization moved from constructor to declarations in MavlinkReceiver class and alphabetize var lists.

Hi,
This PR moves variable initialization out of the MavlinkReceiver constructor into mavlink_reciever.h at the variable declarations and changes the handle_message_play_tune method to utilize uORB rather than px4_open().  It also alphabetizes subscription and variables lists in mavlink_receiver.h.
@davids5, this work partly addresses feedback in PR #11192.
Hardware testing coonducted with QGroundControl 3.4.4 and pixhawk 4 (fmu-v5).
Please let me know if you have any questions on this PR. Thanks!
-Mark

@davids5 and @dagar, anything else you'd like to see done in this PR?

Bench tested with Gazebo, spot checked handle_message_hil_sensor, (impacted by deleting _hil_frame and _old_timestamp), tested other basic functionality and rebased against current master:

We have finally whittled this down to a fairly tidy diff!

@dagar and @davids5 , what can I do to get this on the agenda for Wednesday's call?

I've updated the title of this PR as the only remaining work pertains to the utilization of the DEFINE_PARAMETERS() macro and minor formatting and alphabetization of lists.

@dagar, @davids5, @bkueng,  would you be willing to look this over and let me know if there is anything you'd like to see done here?  Thanks for your feedback!

Here results from functional tests with QGroundControl querying the code/messages impacted by this PR:

Thanks for your feedback @davids5.  I certainly think it's safest to introduce after 1.9 ... hopefully soon!

Flight tested here:
https://review.px4.io/plot_app?log=1d103e72-aa9d-448a-898e-d043949ec6ab
Spot test requesting the battery params during flight can be seen in this screenshot:

Hi @bkueng , would you mind giving this PR your trained eye for a few minutes?  I am hoping to get it merged or closed soon.
I appreciate any feedback you have.  Thank you!

@davids5 and @dagar , I bench tested the latest commit in this PR on a pixhawk4 mini and pixhawk 4 by adding a PX4_INFO(update_params()) printout in the body of the update_params() method then setting parameters for each of the affected handle_message_XXX() methods:

Flight log from this PR here: https://review.px4.io/plot_app?log=d85db144-2341-4c60-945d-1ff167826cf5
Flight log from master here: https://review.px4.io/plot_app?log=2f3ecfb0-c665-4971-bd21-76b72f454120
This PR is good to go.  Please let me know if you have any additional questions or concerns on this PR!

Thanks @bkueng, @julianoes , @dagar , and @davids5 for each of your reviews on this PR!

@dagar or @davids5 , if not finalized by next week, could you please add this to the dev call? Thank you!

Rebased with current master.  Please let me know if you would like the commits squashed. Thanks for all of your review efforts!

Looks right, I'd just do something quick to test that it has the parameter values initially (eg UUID in mavlink flight_information) and that it's picking up changes.

Here is a screenshot of basic bench test exercising the affected methods in MavlinnkReceiver on pixhawk 4.  Test performed by first querying the param, then setting the param, then querying the param to verify the set value with the following: COM_FLIGHT_UUID, BAT_CRIT_THR, BAT_EMERGEN_THR, SENS_FLOW_MAXHGT, and SENS_FLOW_ROT.

Thanks @bkueng !


'default' is not exported by kaboom.js

I tried to use kaboom in a typescript project setup with vite; however it seems that the default import from kaboom is an empty object:
The error thrown on production build:
~/Documents/dev/games/flappy-like main
❯ yarn build
yarn run v1.22.10
warning package.json: No license field
$ tsc && vite build
vite v2.3.7 building for production...
transforming (2) src/main.tsError when using sourcemap for reporting an error: Can't resolve original location of error.
✓ 3 modules transformed.
'default' is not exported by node_modules/kaboom/dist/kaboom.js, imported by src/main.ts
file: /Users/renan/Documents/dev/games/flappy-like/src/main.ts:1:7
1: import kaboom from "kaboom";
          ^
2: const k = kaboom();
3: k.scene("start", () => {
error during build:
Error: 'default' is not exported by node_modules/kaboom/dist/kaboom.js, imported by src/main.ts
    at error (/Users/renan/Documents/dev/games/flappy-like/node_modules/rollup/dist/shared/rollup.js:7917:30)
    at Module.error (/Users/renan/Documents/dev/games/flappy-like/node_modules/rollup/dist/shared/rollup.js:9859:16)

Browser error on dev:

I don't know if I'm missing something in the vite config or typescript config:
The main.ts file:
import kaboom from 'kaboom'

const k = kaboom() // ⚠ Throw an error "Uncaught TypeError: kaboom is not a function"

k.scene('start', () => {
  // add a text at position (100, 100)
  k.add([
    k.text("ohhimark", 32),
    k.pos(100, 100),
  ]);
})

k.start('start')

tsconfig.json file
{
  "compilerOptions": {
    "target": "esnext",
    "module": "esnext",
    "lib": ["esnext", "DOM"],
    "moduleResolution": "node",
    "strict": true,
    "sourceMap": true,
    "resolveJsonModule": true,
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "importHelpers": true,
    "esModuleInterop": true,
    "noEmit": true,
    "noUnusedLocals": true,
    "noUnusedParameters": true,
    "noImplicitReturns": true,
    "noImplicitAny": false,
    "isolatedModules": true,
    "typeRoots": [
      "node_modules/**/*.d.ts"
    ],
  },
  "include": ["./src/*.ts"]
}

I created a repo so it will be easier to see the issue: https://github.com/DecampsRenan/kaboom-no-default-export
Just install dependencies, run yarn dev and open you browser devtools or try to run yarn build to see the errors I mentioned

I'm having the same issue. Tried with just javascript and typescript, using parcel the error I get is:
Uncaught TypeError: _kaboom.default is not a function
    parcelRequire<["kaboom.ts"]< kaboom.ts:6
    newRequire main.c39d6dcf.js:47
    localRequire main.c39d6dcf.js:53
    parcelRequire<["main.ts"]< main.ts:1
    newRequire main.c39d6dcf.js:47
    parcelRequire main.c39d6dcf.js:81
    <anonymous> main.c39d6dcf.js:120

Hello.
I have solved the problem using full path to kaboom packages.
Using vite and vanilla javascript, not typescript.
import kaboom from '../node_modules/kaboom/dist/kaboom';
With typescript must use .d.td module.

Looks like vite (is it using rollup?) is ignoring the exports field in kaboom's package.json.
"browser": "dist/kaboom.js",
"types": "dist/kaboom.d.ts",
"exports": {
	".": {
		"import": "./dist/kaboom.mjs",
		"require": "./dist/kaboom.cjs"
	},
	"./plugins/*": {
		"import": "./dist/plugins/*.mjs",
		"require": "./dist/plugins/*.cjs"
	}
},

I only tested package with esbuild and it works. I'll investigate into this, but if anyone is familiar with distributing packages for all cjs, umd and iife help is appreciated!

With parcel:
import kaboom from "../node_modules/kaboom/dist/kaboom.mjs";

adding the module and main field to point to the .mjs and .cjs (8055546)  seems to fix this, can anyone verify by cloning and installing the library locally? (it's possible your code might break due to other hanging breaking changes on master which is unrelated to this)

I tested it and it works too @slmjkdbtl ! adding .mjs and .cjs versions fixes the issue 👍
In my case I was able to bypass this issue (for vitejs) by adding an alias pointing to the kaboom package:
alias: {
      kaboom: path.resolve(projectRoot, 'node_modules/kaboom/dist/kaboom.mjs'),
}

But I confirm that adding the module and main fields is working fine

with react-scripts and a create-react-app project, the following worked for me:
import kaboom from "../node_modules/kaboom/dist/kaboom.mjs";

the file is a plain JavaScript file, I was not able to get it working with TypeScript

@cn-80 Can you try again with kaboom@next and import kaboom from "kaboom"?

I can still not use the newest version.
`import kaboom from "kaboom";
const k = kaboom({
scale: 2,
clearColor: [3, 211, 252, 1],
width: 240,
height: 384,
});
export default k;`
Installed version:
"version": "2000.0.0-beta.13",
Gives this error:
index.js:19 Uncaught TypeError: _kaboom__WEBPACK_IMPORTED_MODULE_0__.default.start is not a function

@WebdevAlexTV Are you using webpack?

Hmm I tested with create-react-app and webpack and they work on my local machine. Can you share config?

Oh start() is removed in favor of go(), is that the problem?

Oh, it seems that start() was the Problem. So import kaboom from "kaboom"; works fine :)

Nice! I believe the package / bundler issue is resolve, closing.


parse-ms does not work with require()

Hello,
I realized that parse-ms cannot be included with require() As i get this error

import parseMilliseconds from 'parse-ms'
^^^^^^
SyntaxError: Cannot use import statement outside a module

Thats why i used the source code from parse-ms to create parse-ms-2, https://www.npmjs.com/package/parse-ms-2
All the source is in github (https://github.com/dionsyran2/parse-ms-2), so you can have a look and make sure i am not giving you a virus or something.
Thanks for reading
Have a nice day,
dionsyran2

Hey! Change require to import and i find it works just fine!

Hey! Change require to import and i find it works just fine!

only that you need to change my project to module to use import, but if i do i cannot use require or import other modules, because they only work with require()

hmm, sometimes it doesnt require you to have require for all of them, i changed all my requires to import. clone your files to a new folder and try that.


synchronise loading swagger

after loading swagger in synchronise way, can remove buildValidations from exports

Done in https://github.com/Zooz/api-schema-builder/pull/20


Allow Prisma adaptor to take either integer or uuid for ID

We use uuid in order to have globally unique ID's across our entities. Prisma adaptor typings expect ID to be a string, despite changing it in the prisma schema.
I also cannot figure out where this type comes from because inside the index.d.ts for next-auth userId is in fact a string
EDIT: it seems to come from getUser method on the adaptor

EDIT2: I see that v4.0.0-next.15 typings have this fixed already. I'll close this now. Hopefully it helps others stumbling upon this


Version selection method

I tryed 0.1, 0.2, 0.6, 0.7.7 and 0.8.0 beta4 versions with Alan Wake Remastered, all not working with my GTX 1080 (first one crashes, other gives black screen with only HUD visible when DLSS enabled). Should I keep trying other version or it would be useless? If it's not useless to try other version, in what order should I do it (to except very simmilar versions that are useless to test both)?

Nothing you can do about it. Game uses the same gfx engine as the game "Control" which have similar issues.


please tag formal releases

Please consider assigning version numbers and tagging releases. Tags/releases
are quite useful for downstream package maintainers (in Debian and other distributions) to export source tarballs, automatically track new releases and to declare dependencies between packages. Read more in the Debian Upstream Guide.
Thank you.
See also

http://semver.org/
https://en.wikipedia.org/wiki/Software_versioning

https://github.com/coreos/go-tspi/releases

Awesome, thank you very much. :)


Fix not valid JSON

This JSON is not valid and provokes error at homebridge launch

I fixed this myself, thanks for the PR though!


xorgxrdp uses unpatched xrdp sources

Describe the bug
xrdp sources are patched with additional fields of xrdp client config. Unfortunately, xorgxrdp packaged with unpatched xrdp sources as seen here. This causes model and layout fields be overridden by value of rsakeys_ini_file.
Steps To Reproduce
Steps to reproduce the behavior:

Enable xrdp service by

{ services.xrdp.enable = true; }

Login to xrdp session.
Check xkb settings.

$ setxkbmap -print -verbose 10
Setting verbose level to 10
locale is C
Trying to load rules file ./rules/base...
Trying to load rules file /nix/store/01s51slwz2i6z019p6qb9298ws86zmq5-setxkbmap-1.3.2/share/X11/xkb/rules/base...
Success.
Applied rules from base:
rules:      base
model:      /run/xrdp/rsakeys.ini
layout:     s.ini
Trying to build keymap using the following components:
keycodes:   xfree86+aliases(qwerty)
types:      complete
compat:     complete
symbols:    pc+s.ini
geometry:   pc(pc104)
xkb_keymap {
	xkb_keycodes  { include "xfree86+aliases(qwerty)"	};
	xkb_types     { include "complete"	};
	xkb_compat    { include "complete"	};
	xkb_symbols   { include "pc+s.ini"	};
	xkb_geometry  { include "pc(pc104)"	};
};

Expected behavior
xkb settings should contain appropriate values. For example: model - pc104 and and layout - us.
Notify maintainers
@chvp

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.44, NixOS, 23.05 (Stoat), 23.05.20230809.9034b46`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.5`
 - channels(root): `""`
 - nixpkgs: `/home/adomas/.config/flakeInputs/nixpkgs`

PR at #250189


Off-by-one in block_capacity_hard_limits?

#include "plf_hive.h"
#include <cassert>

int main() {
  plf::hive<int> h({}, {2,2});
  auto soft = h.block_capacity_limits();
  auto hard = h.block_capacity_hard_limits();
  assert(hard.min <= soft.min && soft.max <= hard.max);
}

This program assert-fails because block_capacity_hard_limits claims the hard min is 3, but in fact check_capacities_conformance is happy with a soft min of 2. I assume the simple fix is to change 2 to 3 in check_capacities_conformance, or vice versa in block_capacity_hard_limits. But it might be a good idea to refactor check_capacities_conformance entirely:
    static inline void check_capacities_conformance(plf::hive_limits soft) {
        auto hard = block_capacity_hard_limits();
        if (!(hard.min <= soft.min && soft.min <= soft.max && soft.max <= hard.max)) {
            throw std::length_error("Supplied memory block capacities outside of allowable ranges");
        }
    }

Good idea, fixed in beta
On 24/04/2022 1:10 pm, Quuxplusone wrote:

|#include "plf_hive.h" #include  int main() { plf::hive
h({}, {2,2}); auto soft = h.block_capacity_limits(); auto hard =
h.block_capacity_hard_limits(); assert(hard.min <= soft.min && soft.max
<= hard.max); } |
This program assert-fails because |block_capacity_hard_limits| claims
the hard min is 3, but in fact |check_capacities_conformance| is happy
with a soft min of 2. I assume the simple fix is to change |2| to |3| in
|check_capacities_conformance|, or vice versa in
|block_capacity_hard_limits|. But it might be a good idea to refactor
|check_capacities_conformance| entirely:
|static inline void check_capacities_conformance(plf::hive_limits soft)
{ auto hard = block_capacity_hard_limits(); if (!(hard.min <= soft.min
&& soft.min <= soft.max && soft.max <= hard.max)) { throw
std::length_error("Supplied memory block capacities outside of allowable
ranges"); } } |
—
Reply to this email directly, view it on GitHub
https://github.com/mattreecebentley/plf_hive/issues/12, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABE4FIU6IOFG3O6QY7YIATTVGSNQZANCNFSM5UFO7LBQ.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Duplicate QRCodes in Database

Scanning a QRCode will always make a new document in FireStore, rather than checking and updating an existing document, resulting in duplicate codes in separate documents.
Then leads to being impossible to view this QRCode because readQRCode(...) {...} expects that there is only one document with the id, and returns nothing if there is not exactly 1 matching document.
Bug will likely be fixed shortly but documenting so I don't forget

Interestingly, the bug only seems to occur with QRCode5 from ScanMe.png
The Database currently has 10 unique documents each containing QRCode5, but has no other QRCodes with duplicate documents.
(And to clarify, when I say duplicate, I don't mean the data in each document is synched with eachother, each QRCode5 document has it's own scanners, comments, titles, etc., though of course each has the same ID)
How can the bug be replicated with other QRCodes? Further testing required.

This issue seems resolved now, scanning QRCodes with different accounts updates the existing firestore document, rather than creating new ones.
I think this issues should have been closed with the last commit to staging, but was left open by mistake.


Improve api coverage

This brings us so close to full coverage that I consider this closing #10. Now uncovered features are:

debug markers (because I have no idea yet how these work)
render bundles (because wgpu-native does not support them)
indirect drawing
a few missing features inside render calls: depth_stencil_state, resolve_target, depth_stencil_attachment

todo:

[ ] test coverage!

I dont see debug markers in the webgpu spec... Can you provide a link/reference so I can read up on them?

Would it be much effort to add the indirect drawing API? It seems like it would be necessary to support instanced rendering?

I dont see debug markers in the webgpu spec...

E.g. here. Not much info though. More here.

Would it be much effort to add the indirect drawing API?

I'll have a look. I initially planned to just implement all features that we might need in visvis2. But yeah ... we're pretty close to 100% now so maybe  just push through.

It seems like it would be necessary to support instanced rendering?

I don't think so.

https://www.khronos.org/opengl/wiki/Debug_Output
This page describes use of debug markers in more detail; apparently you can annotate your GPU commands with "markers" and have the annotations show up in traces/logs, to help you identify/locate things.

It seems like it would be necessary to support instanced rendering?

I don't think so.

OK then. Nevermind :)

I'm merging this. I will have a go at debug markers and indirect drawing in new PR's.


License File Needed

Please add a license file, so as to clarify what the code is available for.
https://choosealicense.com/

OK, after I make some important updates.
#4  #5

I would argue that it is best to do it before you issue those updates - it will allow potential contributors, such as myself, to contribute according to the license provided.
If you indicate which license you prefer, I can make a pull request to add one.

@balag12
Pull MIT license to me please.

#7

Closed.


fix wrong parameters description

It works great with:
        $filename = __DIR__ . time() . ".jpg";
        file_put_contents($filename, fopen($link, 'r'));
        $result = Request::sendPhoto(
            [
                'chat_id' => Config::getTelegramChat(),
            ],
            $filename
        );

@MBoretto why I should this, when i already know the path to image and it works?
Also I'm not sure that $telegram->getUploadPath() returns the correct path(e.g. on Win it shows: D:\Development\Web\projects\vk-to-telegram-bot\vendor\longman\telegram-bot\src/../Upload
I got his with: echo $telegram->getUploadPath();
By the way, I suppose that I need to use setUploadPath($folder) to set folder various from \vendor\longman\telegram-bot\src

You are right for what concern the wrong path on windows. He're you can see the  automatic initialization intended for linux, we can open an issue for this.
https://github.com/akalongman/php-telegram-bot/blob/master/src/Telegram.php#L160
The 'UploadPath()' is intended to have a know path where all the images to upload are stored, that's because can be used internally from the library. Of course you can use it also as you mention, but i would like to leave these information in the documentation.

@MBoretto thanks, now I understand meaning if this function. So I close PR.
Thanks for attention!


Assertion 'lit_is_valid_cesu8_string (string_p, string_size)' failed in ecma_new_ecma_string_from_utf8

Jerry version:
Checked revision: 0ef2418
Build command: ./tools/build.py --clean --debug

OS:
Ubuntu 16.04.3 LTS

Test case:
unescape("%f׊nd")

Download test case
Backtrace:
ICE: Assertion 'lit_is_valid_cesu8_string (string_p, string_size)' failed at jerryscript/jerry-core/ecma/base/ecma-helpers-string.c(ecma_new_ecma_string_from_utf8):154.
Error: ERR_FAILED_INTERNAL_ASSERTION

Program received signal SIGABRT, Aborted.
(gdb) bt
#0  0x00000000005e4d28 in raise ()
#1  0x00000000005e4eda in abort ()
#2  0x000000000057ec65 in jerry_port_fatal (code=ERR_FAILED_INTERNAL_ASSERTION)
    at jerryscript/jerry-port/default/default-fatal.c:71
#3  jerry_fatal (code=code@entry=ERR_FAILED_INTERNAL_ASSERTION) at jerryscript/jerry-core/jrt/jrt-fatals.c:58
#4  0x000000000057f350 in jerry_assert_fail (assertion=<optimized out>, file=<optimized out>, function=<optimized out>, line=<optimized out>)
    at jerryscript/jerry-core/jrt/jrt-fatals.c:82
#5  0x000000000040eac9 in ecma_new_ecma_string_from_utf8 (string_p=0x9c6458 <jerry_global_heap+296> "%f\327ýd", string_size=5)
    at jerryscript/jerry-core/ecma/base/ecma-helpers-string.c:154
#6  0x00000000004b65c6 in ecma_builtin_global_object_unescape.isra.12 (arg=<optimized out>)
    at jerryscript/jerry-core/ecma/builtin-objects/ecma-builtin-global.c:1336
#7  ecma_builtin_global_dispatch_routine (builtin_routine_id=<optimized out>, this_arg_value=<optimized out>, arguments_list=<optimized out>,
    arguments_number=<optimized out>) at jerryscript/jerry-core/ecma/builtin-objects/ecma-builtin-global.inc.h:215
#8  0x0000000000441430 in ecma_builtin_dispatch_routine.lto_priv.316 (builtin_object_id=<optimized out>, builtin_routine_id=<optimized out>,
    this_arg_value=72, arguments_list=0x7fffffffcf44, arguments_number=<optimized out>)
    at jerryscript/jerry-core/ecma/builtin-objects/ecma-builtins.inc.h:484
#9  0x000000000049dd72 in ecma_builtin_dispatch_call (obj_p=0x9c6440 <jerry_global_heap+272>, this_arg_value=72, arguments_list_p=0x7fffffffcf44,
    arguments_list_len=1) at jerryscript/jerry-core/ecma/builtin-objects/ecma-builtins.c:844
#10 0x0000000000571830 in ecma_op_function_call (func_obj_p=0x9c6440 <jerry_global_heap+272>, this_arg_value=72, arguments_list_p=0x7fffffffcf44,
    arguments_list_len=1) at jerryscript/jerry-core/ecma/operations/ecma-function-object.c:419
#11 0x00000000005c5f7d in opfunc_call.isra.1 () at jerryscript/jerry-core/vm/vm.c:408
#12 vm_execute (frame_ctx_p=frame_ctx_p@entry=0x7fffffffcf50, arg_p=<optimized out>, arg_list_len=<optimized out>)
    at jerryscript/jerry-core/vm/vm.c:2743
#13 0x00000000005cbbba in vm_run (bytecode_header_p=<optimized out>, this_binding_value=<optimized out>, lex_env_p=<optimized out>,
    is_eval_code=<optimized out>, arg_list_p=<optimized out>, arg_list_len=<optimized out>)
    at jerryscript/jerry-core/vm/vm.c:2823
#14 0x00000000004094b8 in vm_run_global (bytecode_p=0x9c6568 <jerry_global_heap+568>) at jerryscript/jerry-core/vm/vm.c:231
#15 jerry_run (func_val=259) at jerryscript/jerry-core/api/jerry.c:444
#16 main (argc=<optimized out>, argv=<optimized out>) at jerryscript/jerry-main/main-unix.c:707

Found by  Fuzzinator

@yichoi @LaszloLango  Can we close this issue?


Examples of arrays are being merged

From discussion #361.
test("should return array of arrays", () => {
  expect(
    getExamples(
      withMeta(z.array(z.number().int())).example([1, 2]).example([3, 4]),
      false
    )
  ).toEqual([
    [1, 2],
    [3, 4],
  ]);
});

it fails with:
 FAIL  tests/unit/common-helpers.spec.ts
  ● Common Helpers › getExamples() › should return array of arrays

    expect(received).toEqual(expected) // deep equality

    - Expected  - 4
    + Received  + 0

      Array [
    -   Array [
        1,
        2,
    -   ],
    -   Array [
        3,
        4,
    -   ],
      ]

So, getExamples merges the arrays into one.
I believe it is because of concat at https://github.com/RobinTail/express-zod-api/blob/1f6a9432046c8a1b522d15ef6a08ac194f957c47/src/common-helpers.ts#L125
Originally posted by @McMerph in https://github.com/RobinTail/express-zod-api/discussions/361#discussioncomment-4994650

Fixed in version 9.4.2 🚀 , @McMerph


CHANGELOG update

Coverage remained the same at 32.21% when pulling 40c9ade6b832faddac87d44cabdaf01c8f1f3407 on 4.1.0 into 1c84953edcd0334e8c7f3814e9ba01ad604e58e3 on master.


Add CUDA support

Could you please add Nvidia CUDA Acceleration support? Thanks.

You can specify hwaccel_api: nvenc in your pipeline config, given that your ffmpeg is built with CUDA support.

This should already be supported.  Let us know if the suggestion of @meryacine does not work for you.  Thanks!


Upgrade wallet connect v1 up to 1.7.0 version, move dependency to spm #5281

Closes #5281, Closes #5087

looks like yes, we don't need it anymore.

This PR stops using the fork and switches to using the upstream SPM right? Then we don't have to do anything else.


Attempt Upgrade to gcc 4.9

Experimenting with Travis config.

@lehecka This seems to work for gcc4.9, can you take a look? Do you want clang as well (which isn't working)?

Seems like clang is just timing out ... and my attempt to extend the timeout failed :-/

We unfortunately need support for both clang and gcc, as both are used internally.

I know this is going to take a lot of tries to figure this out. Thank you for doing this!
I tried this and ended up convincing travis to use gcc 4.9, but the build was messed up enough that the tests froze and never completed: https://github.com/lehecka/reactivesocket-cpp/commit/17a592c7d4ed3ab11c97162325a633cdcdefc95d

the latest attempt from me is here: https://travis-ci.org/lehecka/reactivesocket-cpp/builds/178380361
it looks like it all compiles, but gets stuck in executing tests.
Maybe we can disable tests and see if they are all messed up or just one test is messed up..

closing in favor of https://github.com/ReactiveSocket/reactivesocket-cpp/pull/253


This application has a distinct lack of bald people

If anything, it lacks hairs! This Android looks quite bald to me:


[Feature]: meltano add to project without install

Feature scope
CLI (options, error messages, logging, etc.)
Description
As a user, I need to add to my meltano project without install
Adding to a meltano project is currently a very slow operation.  Our requirement is to add all the plugins to the project and then later build (i.e. 'meltano install') the whole project.  This means that the initial 'meltano add' is unnecessarily slow as we don't use any of the pip-installed modules from the initial 'meltano add'
Work arounds considered:

directly add a meltano.yml format file with our plugin definition to a directory with the root meltano.yml includes referencing that directory path & directly save a lock file to the /plugins directory  (essentially avoiding using 'meltano add' at all)

This proposal is to be able to add to the meltano project rapidly without installing all pip dependencies, and we'd ideally like to be able to freeze those dependencies.
meltano add --skip-install should

download the lock file
add to the meltano.yml
add all the lock and meltano.yml files for any requires plugins (https://github.com/meltano/hub/pull/505 & https://github.com/meltano/meltano/issues/6107)

Related to:
This feature is related to https://github.com/meltano/meltano/issues/6416
As a user, I need to add to my meltano project without install and create pip freeze
meltano add --skip-install --freeze consider whether this --skip-install works if --freeze is used (assuming https://github.com/meltano/meltano/issues/6416 might implement --freeze)

This was resolved by #6578 and can be closed. May need to revisit as per #6416.

Closed via https://github.com/meltano/meltano/pull/6578


simplifying use of mathjax-node-sre

As an author, I want my content to be as accessible as possible.

I don't see a natural way for doing this with npm.
For the record, one just needs to change/add the dependency and change the require('mathjax-node') call in lib/main.js. Since SRE is a very large dependency, I'd rather not have it as a default dependency here.

If anyone has an idea how to provide alternative dependencies like this, please let me know.

Here's a simple idea.
try {
  const mathjax = require('mathjax-node-sre');
}
catch(e) {
  const mathjax = require('mathjax-node');
}

This way, if a project installs both mathjax-node-page and mathjax-node-sre, we switch automatically.

Actually more complicated, cf. https://gist.github.com/FGRibreau/3135914#gistcomment-579762 but seems to work. Not sure how to integrate it into the cli yet.

For the record, one just needs to change/add the dependency and change the require('mathjax-node') call in lib/main.js.

Doing that seems to replace svg output with MathML output and do nothing else. What is the expected result?

Huh. That's weird. I have been extending mathjax-node-sre recently and something must have broken (and for obvious reasons there isn't a test for this yet :) )

Ah, I know what's happening. It's sort of a problem on both sides -- mathjax-node-sre always generated MathML (which it probably shouldn't) and mathjax-node-page blindly injects any output it finds instead of checking the configuration, with mathml coming last - thus overwriting previous svg output (which it probably shouldn't).

Thanks for 5d8f90c71167c13448. Now I can see the result. However I'm disappointed to see that neither lynx nor w3m display the title tag of inline svg whereas they both display the desc tag. Is there a reason to place the SRE output in title rather than desc?
Also it seems the original TeX source disappears completely from the output html. I think that, even with a very smart SRE, there will always be cases where users will want to read the source for disambiguation. I don't know where to put it and how to access it. But losing information seems unwise.

Is there a reason to place the SRE output in title rather than desc?

Yes, better screenreader support.

there will always be cases where users will want to read the source for disambiguation.

I understand the sentiment. I personally like spans behind the mathjax output which I hide but leave accessible to copy&paste (so you can copy a paragraph with TeX).
Might be a nice optional feature.

Is there a reason to place the SRE output in title rather than desc?

Yes, better screenreader support.

I'm not sure to understand what you call a screenreader here. Do you mean something like Orca?
I know only one blind mathematician. He always says that software trying to make a graphical UI accessible (like Orca) are not usable. He browses the web using lynx (and a braille display driven by brltty) and reads the TeX source of maths papers when he wants to really understand. But he also listen to math papers (recorded by humans) when he wants to get a superficial understanding.
So one could imagine something like lynx + festival reading a web page processed by mathjax-node-sre but with the possibility of stopping to read the source TeX at any time. Or simply using lynx, without speech output, but get the TeX source instead of formulas. This is still better than reading the full TeX source of a paper because you get hypertext links and less markup.

I personally like spans behind the mathjax output which I hide but leave accessible to copy&paste.

Yes, this is also a very good reason to keep the source somewhere.

I'm not sure to understand what you call a screenreader here.

Cf. https://en.wikipedia.org/wiki/Screen_reader.
But I should have been more detailed: support across the accessibility tool chain which includes but is not limited to browsers populating the accessibility tree with suitable information, operating systems exposing that information in their accessibility APIs and screenreaders leveraging that information.

Fixed via #50

Could you elaborate on that "Fixed via #50"? What should I do to be able to run something like mjpage < in.html > out.html" and get math rendered as SVG together with SRE output somewhere (preferably in <desc> tag inside svg but if you insist on putting it only in <title> I can still run a postprocessor to fix it).
Also, did you also fixed the "keep source code somewhere" sub-issue?


How to roll out new version of vmss agent if vmss scale set is used?

When creating through the portal, 1 agent pool = 1 vmss. What if we want to roll out a new version? How can we 'use' the same name so there will be no changes in the pipelines?

@majorvin Thank you for the question.
Please provide a link to the documentation and we can assign an author for review, though note the support options here are limited to examples only.
Otherwise, for more specific use-case scenarios, see:

Azure DevOps support
Azure DevOps community
Azure DevOps on StackOverflow

https://docs.microsoft.com/en-us/azure/devops/pipelines/agents/scale-set-agents?view=azure-devops

@majorvin Thank you for the document link, assigning this to the author for further review.

Please also note, if the issue is product-related or not yet supported, you'll need to use Report a problem or Suggest a feature in the Azure DevOps community.
Or, for additional support options, see:

Azure DevOps Support
Azure DevOps on StackOverflow


Fallback to getDefaultGateway() if "bridge" is unavailable.

Fixes #2239

This was released in https://github.com/testcontainers/testcontainers-java/releases/tag/1.14.0 🎉
Thanks for the contribution!


Issues with WSL showing?

Description

Steps To Reproduce
it doesn't show
Steps to fix

eg. Fix crash that occurs when...
eg. Correct action happens when clicking...

Files Version
w11
Windows Version
w11
Log file
.

Thanks for the feedback. Please make a new issue following the issue templates. It tells you how to fill it out if you don't know what to put for a section.

I think we have all the info we need here

I've seen this issue on devices the upgraded from Windows 10.

I've seen this issue on devices that upgraded from Windows 10.
No sir, this is a brand new ssd and windows 11 installed there is no win10 ever installed on this,  I have uninstalled and reinstalled the sub system for Linux.

Do you have a distro installed and setup. Also what version of WSL are you using?

@meemore do you have any distros installed?


Added new merging tab.

Added Merging tab.
Empty state

Queue contains requests

Looks good. Should we make the layout change toggleable or dependent on the merge blocking feature? Or should we make this the default appearance regardless?
I don't have a strong preference.

My preference will be to keep it as default appearance regardless of the state of merge blocking feature. It will keep the behaviour consistent and code clean.


[CT-1729] rpc requests fail on dbt version 1.3.1

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
I am running dbt-rpc server with following packages:
dbt-snowflake==1.3.0
dbt-rpc==0.2.1
On executing any rpc request (like status check), rpc server returns following response:
TypeError: _get_responses() takes 3 positional arguments but 4 were given
Downgrading packages to earlier version returns same results.
Expected Behavior
rpc server must return correct respose
Steps To Reproduce

Environment with following packages:
dbt-snowflake==1.3.0
dbt-rpc==0.2.1
run server status check like following:
{
"jsonrpc": "2.0",
"method": "status",
"id": "2db9a2fe-9a39-41ef-828c-25e04dd6b07d"
}
get the following error:
TypeError: _get_responses() takes 3 positional arguments but 4 were given

Relevant log output
Installing dbt packages
16:20:16  Running with dbt=1.3.1
16:20:17  Installing dbt-labs/dbt_utils
16:20:19    Installed from version 0.9.1
16:20:19    Updated version available: 1.0.0
16:20:19
16:20:19  Updates available for packages: ['dbt-labs/dbt_utils']
Update your versions in packages.yml, then run dbt deps
Error on request:
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/werkzeug/serving.py", line 335, in run_wsgi
    execute(self.server.app)
  File "/usr/local/lib/python3.8/site-packages/werkzeug/serving.py", line 322, in execute
    application_iter = app(environ, start_response)
  File "/usr/local/lib/python3.8/site-packages/werkzeug/middleware/dispatcher.py", line 78, in __call__
    return app(environ, start_response)
  File "/usr/local/lib/python3.8/site-packages/werkzeug/wrappers/request.py", line 187, in application
    resp = f(*args[:-2] + (request,))
  File "/usr/local/lib/python3.8/site-packages/dbt_rpc/task/server.py", line 152, in handle_jsonrpc_request
    jsonrpc_response = ResponseManager.handle(
  File "/usr/local/lib/python3.8/site-packages/dbt_rpc/rpc/response_manager.py", line 143, in handle
    result = cls.handle_valid_request(
  File "/usr/local/lib/python3.8/site-packages/dbt_rpc/rpc/response_manager.py", line 88, in handle_valid_request
    return cls.handle_request(request, dispatcher)
  File "/usr/local/lib/python3.8/site-packages/jsonrpc/manager.py", line 75, in handle_request
    responses = [r for r in cls._get_responses(rs, dispatcher, context)
TypeError: _get_responses() takes 3 positional arguments but 4 were given

Environment
No response
Which database adapter are you using with dbt?
snowflake
Additional Context
No response

Could be related to json-rpc dependency update that happened on Christmas [json-rpc PyPi] + this change in setup.py?
https://github.com/dbt-labs/dbt-rpc/commit/43a13999441268d220236bc230791554daa78320

@timopetmanson is spot on. We've fixed this for now by capping the version of json-rpc required by dbt-rpc: https://github.com/dbt-labs/dbt-rpc/pull/114
If you're using an older version of dbt-rpc with an older version of dbt-core, you should specify the explicit dependency version yourself:
json-rpc<1.14.0

There's another open issue that proposes updating the dbt-rpc code for compatibility with the changes in json-rpc==1.14.0: https://github.com/dbt-labs/dbt-rpc/issues/116
This problem is limited to dbt-rpc, and that's the place where we'll want to make the fix. I'm going to close this dbt-core issue in the meantime.


Incorrect information in the comment block for the function Str::orderedUuid()

Laravel Version
10.44.0
PHP Version
8.3.1
Database Driver & Version
MySQL 8.2.0
Description
In the comment block for the function Str::orderedUuid() it is incorrectly stated:

Generate a time-ordered UUID (version 4).

Correctly it should be written:

Generate a time-ordered modification of UUID (version 4) containing a time stamp.

By definition

Version 4 UUIDs [...] do not contain any information about the time they are created [...].

While this is not technically a bug breaking execution, this error in the statement (claiming to be version 4) might trap a developer into using this function by believing that the generated UUIDs do not contain any time information, leading to unintended leakage of (time) information.
Strictly speaking, Str::orderedUuid() doesn't produce UUIDs fully complying to RFC 4122, but just the customization "Timestamp-first COMB Codec":

In 2002, Jimmy Nilsson recognized this problem with random UUIDs and proposed a solution he called “COMBs” (see “The Cost of GUIDs as Primary Keys”).

Changing the function name would break implementations, but at least the comment should be updated.
Steps To Reproduce
See description.

Hey there, thanks for reporting this issue.
If you notice improper DocBlock, PHPStan, or IDE warnings while using Laravel, do not create a GitHub issue. Instead, please submit a pull request to fix the problem. Also see our contribution guide.
Thanks!


Create event api tests

Basic Information
Create tests for all the event api endpoints. Make sure to also test negative cases e.g. 400
Other Information
[Are there any additional information that we should know about?]

Should we also include RSVP testcases as well?

Should we also include RSVP testcases as well?

No that's okay for now because it requires auth to be merged in first


[FEAT]: Add a dataset to a project

Describe the solution you'd like

[ ] Add the ability to tie a dataset from the Resources Page to any project from the user
[ ] Add the ability to upload a dataset from a file or the Data Explorer to a project directly (which will automatically add it to the Resources Page

Note that the outstanding PR: https://github.com/DARPA-ASKEM/TERArium/pull/133 includes initial implementation of a client-side store to tracks all project resources including datasets, models, and papers.
Ideally, the appropriate solution to include a client-side store (i.e., cache) that is in sync with a backend store for persistence.

Note sure what is missing here to close this issue @dgauldie


mielecloudservice crashes after update to 6.0.0

`

host.homie
2022-04-26 20:09:33.784
info
Do not restart adapter system.adapter.mielecloudservice.0 because disabled or deleted

host.homie
2022-04-26 20:09:33.783
error
instance system.adapter.mielecloudservice.0 terminated with code 3 (NO_ADAPTER_CONFIG_FOUND)

mielecloudservice.0
2022-04-26 20:09:33.239
warn
Terminated (NO_ADAPTER_CONFIG_FOUND): Without reason

mielecloudservice.0
2022-04-26 20:09:33.233
error
adapter disabled

mielecloudservice.0
2022-04-26 20:09:33.209
info
Plugin sentry Sentry Plugin disabled for this process because sending of statistic data is disabled for the system

host.homie
2022-04-26 20:09:15.951
info
"system.adapter.mielecloudservice.0" disabled

host.homie
2022-04-26 20:09:02.319
info
Restart adapter system.adapter.mielecloudservice.0 because enabled

host.homie
2022-04-26 20:09:02.318
error
instance system.adapter.mielecloudservice.0 terminated with code 6 (UNCAUGHT_EXCEPTION)

mielecloudservice.0
2022-04-26 20:09:01.773
warn
Terminated (UNCAUGHT_EXCEPTION): Without reason

mielecloudservice.0
2022-04-26 20:09:01.772
info
terminating

mielecloudservice.0
2022-04-26 20:09:01.762
error
Cannot read properties of undefined (reading 'unit')

mielecloudservice.0
2022-04-26 20:09:01.762
error
TypeError: Cannot read properties of undefined (reading 'unit') at Object.module.exports.splitMieleDevices (/opt/iobroker/node_modules/iobroker.mielecloudservice/source/mieleTools.js:420:212) at EventSource. (/opt/iobroker/node_modules/iobroker.mielecloudservice/main.js:102:32) at EventSource.emit (node:events:526:28) at _emit (/opt/iobroker/node_modules/eventsource/lib/eventsource.js:277:17) at parseEventStreamLine (/opt/iobroker/node_modules/eventsource/lib/eventsource.js:292:9) at IncomingMessage. (/opt/iobroker/node_modules/eventsource/lib/eventsource.js:249:11) at IncomingMessage.emit (node:events:526:28) at addChunk (node:internal/streams/readable:315:12) at readableAddChunk (node:internal/streams/readable:289:9) at IncomingMessage.Readable.push (node:internal/streams/readable:228:10)

mielecloudservice.0
2022-04-26 20:09:01.761
error
unhandled promise rejection: Cannot read properties of undefined (reading 'unit')

mielecloudservice.0
2022-04-26 20:09:01.760
error
Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch().

mielecloudservice.0
2022-04-26 20:09:01.739
info
Server Sent Events-Connection has been (re)established @Miele-API.

mielecloudservice.0
2022-04-26 20:09:01.636
info
Registering for all appliance events at Miele API.

mielecloudservice.0
2022-04-26 20:09:01.247
info
Login attempt #1 @Miele-API

mielecloudservice.0
2022-04-26 20:09:01.220
info
starting. Version 6.0.0 in /opt/iobroker/node_modules/iobroker.mielecloudservice, node: v16.14.2, js-controller: 4.0.23

mielecloudservice.0
2022-04-26 20:09:01.141
info
Plugin sentry Sentry Plugin disabled for this process because sending of statistic data is disabled for the system

host.homie
2022-04-26 20:09:00.236
info
instance system.adapter.mielecloudservice.0 started with pid 12884

host.homie
2022-04-26 20:08:30.190
info
Restart adapter system.adapter.mielecloudservice.0 because enabled

`

Plattform linux Betriebssystem linux Architektur x64 CPUs 2 Geschwindigkeit 1996 MHz Modell Intel Core i7 9xx (Nehalem Class Core i7) RAM 7.79 GB System-Betriebszeit 1 T. 04:46:48 Node.js v16.14.2 (Empfohlene Version v14.19.1) time<PHONE_NUMBER>724 timeOffset -120 Anzahl der Adapter 490 NPM 8.5.0 Datenträgergröße 502.35 GB freier Festplattenspeicher 484.07 GB Betriebszeit 1 T. 04:46:32 Aktive Instanzen 21 Weg /opt/iobroker/ Hostname homie

Okay. There seems to be something bad inside your devices message, which was covered by the code before and got lost during rewrite. Please do a debug log run and provide the log here. I need to take a deeper look.

@aski71
After doing some research I'm pretty sure that V6.1.0 will fix your issue.
I removed some useless code for freezers which may have caused the exception. Please give it a try and report the result here.
But I'm anyways intrested in your debug log - since the device-state and the device-actions messages are logged there. I'd like to collect them to get a kind of device library for adapter testing and debugging purposes.


Rename Start Collaborating to ...

On http://console.prod-preview.openshift.io/home rename the button.
@qodfathr please provide the new name
Please then assign to me to make code fix, or send PR :-)

@muruGanesan is working onthe designs for this

@pmuir, Jessica worked on the screen. You can see the (wireframe) mock up  here: https://redhat.invisionapp.com/share/XZAOBAOPB#/screens

@muruGanesan is this ready for us to start implementing? If so, can @SMahil do an initial html/css pass through for us? /cc @maxandersen @michaelkleinhenz

I think we should expand the scope of the issue to "implement the homescreen design".
@SMahil are you able to take this on?

Sure @joshuawilson

thanks @SMahil

This appears fixed, closing.


Build cache for Travis CI

I think, this is may be useful: https://restic.net/blog/2018-09-02/travis-build-cache
PS: and .travis.yml missed go 1.12.x
Send PR?

Yes please.
On Sat., 17 Aug. 2019, 23:06 Lord Alfred<EMAIL_ADDRESS>wrote:

I think, this is may be useful:
https://restic.net/blog/2018-09-02/travis-build-cache
PS: and .travis.yml missed go 1.12.x
Send PR?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/flashmob/go-guerrilla/issues/177?email_source=notifications&email_token=AAE6MP6UIB44RBZ7WQVIJ5LQFAAVHA5CNFSM4IMPQAY2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HFZBZKQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE6MPYJ2E5DHIK7CAKQVSTQFAAVHANCNFSM4IMPQAYQ
.


Nested complex types not documented

Problem
I have a POST method:
public async Task<IActionResult> Post([FromBody] Person newMember)

With the Person model being:
public class Person {
    /// The name of the new member.
    public string Name { get; set; }

    /// The pet the new member likes the most.
    public Pet FavouritePet { get; set; }

    /// The pet the new member likes the least.
    public Pet LeastFavouritePet { get; set; }
}

Here the 'pet'-properties are not documented in the open-api spec. There is only a $ref to another schema but no description to show what the properties are. In contrast this description is available for non-complex members like eg Name.
What I would like to see
Documentation for complex types like nested objects.

Dup of #1303. Long story short - enable the UseAllOfToExtendReferenceSchemas setting.

@domaindrivendev  that is not really the solution I was looking for.
By using the UseAllOfToExtendReferenceSchemas, the schema results in:
"Person": {
  "type": "object",
  "properties": {
    "name": {
      "type": "string",
      "description": "The name of the new member."
    },
    "favouritePet ": {
      "allOf": [
        {
          "$ref": "#/components/schemas/Pet"
        }
      ],
      "description": "The documentation of the a Pet."
    },
    "leastFavouritePet ": {
      "allOf": [
        {
          "$ref": "#/components/schemas/Pet"
        }
      ],
      "description": "The documentation of the a Pet."
    },
  },
  "additionalProperties": false
},

Where I would expect it to be:
"Person": {
  "type": "object",
  "properties": {
    "name": {
      "type": "string",
      "description": "The name of the new member."
    },
    "favouritePet ": {
      "allOf": [
        {
          "$ref": "#/components/schemas/Pet"
        }
      ],
      "description": "The pet the new member likes the most."
    },
    "leastFavouritePet ": {
      "allOf": [
        {
          "$ref": "#/components/schemas/Pet"
        }
      ],
      "description": "The pet the new member likes the least."
    },
  },
  "additionalProperties": false
},

The description of the nested object is taken from the class, not the properties.

Can this issue be reopened?

Hi! I second this issue. I'm looking for the way to have the property's XML documentation in the description field.

I am running into a similar issue with the above, because if I have a property that is marked as deprecated using ObsoleteAttribute, but the property is a referenced complex type and not something simple like a string or a date, it does not show up as deprecated, because it is showing the documentation for the referenced type instead.
I will throw together a simple example and create a separate issue.  Stay tuned.

I'm sure there was another issue or PR related to this recently, but I can't find it...

@martincostello, I have been studying this and with this simple change:

We are filling the description of the property(It could be an enum or a ComplexType, so the issue #2120 would be the same.
The problem here seems to be also the Microsoft.OpenApi provider because it seems to be just serializing the ref keyword (It does not serialize the Description), with this image I show you in the outputWindow that the Description of the property is being filled:

But it's never serialized into the Json.
So if you just change the CrudActionsController putting a description over Status and you do the change I propose you will find that SwashBuckle is filling the info, but it's not serialized.
I pass you here the change I did over CrudActionsController
public class Product
{
    ///
    /// Uniquely identifies the product
    ///
    public int Id { get; set; }

    ///
    /// Describes the product
    ///
    public string Description { get; set; }

    ///
    /// Status of the Product
    ///

    public ProductStatus Status { get; set; }

    public ProductStatus? Status2 { get; set; }

    ///
    /// SubProduct
    ///
    public SubProduct SubProduct { get; set; }
}
public class SubProduct
{
    public int MyProperty { get; set; }
}

Bump to prevent staleness

@wim07101993
I have just tested with this code:
using System.Reflection;
using Microsoft.AspNetCore.Mvc;
using Microsoft.OpenApi.Models;

var builder = WebApplication.CreateBuilder(args);

// Add services to the container.

builder.Services.AddControllers();
// Learn more about configuring Swagger/OpenAPI at https://aka.ms/aspnetcore/swashbuckle
builder.Services.AddEndpointsApiExplorer();
builder.Services.AddSwaggerGen(c =>
{
	c.IncludeXmlComments(
	  Path.Combine(AppContext.BaseDirectory, $"{Assembly.GetExecutingAssembly().GetName().Name}.xml"),
	  true);
	c.UseAllOfToExtendReferenceSchemas();
	c.SwaggerDoc("v1", new OpenApiInfo { Title = "Test API", Version = "1" });
});
//builder.Services.AddSwaggerGenNewtonsoftSupport();

var app = builder.Build();

// Configure the HTTP request pipeline.
if (app.Environment.IsDevelopment())
{
	app.UseSwagger();
	app.UseSwaggerUI();
}

app.UseAuthorization();

app.MapControllers();

app.Run();

public class TestController : ControllerBase
{
	[HttpPost("Person")]
	public async Task<IActionResult> Post([FromBody] Person newMember) => NotFound();

}
public class Person
{
	/// The name of the new member.
	public string Name { get; set; }

	/// The pet the new member likes the most.
	public Pet FavouritePet { get; set; }

	/// The pet the new member likes the least.
	public Pet LeastFavouritePet { get; set; }
}
///
/// Summary for Pet
///
public class Pet
{
	public string Name { get; set; }
}

And with SwashBuckle latest version and also with version 6.0.0 and both of them produced the same OpenApi document:
{
  "openapi": "3.0.1",
  "info": {
    "title": "Test API",
    "version": "1"
  },
  "paths": {
    "/Person": {
      "post": {
        "tags": [
          "Test"
        ],
        "requestBody": {
          "content": {
            "application/json": {
              "schema": {
                "allOf": [
                  {
                    "$ref": "#/components/schemas/Person"
                  }
                ]
              }
            },
            "text/json": {
              "schema": {
                "allOf": [
                  {
                    "$ref": "#/components/schemas/Person"
                  }
                ]
              }
            },
            "application/*+json": {
              "schema": {
                "allOf": [
                  {
                    "$ref": "#/components/schemas/Person"
                  }
                ]
              }
            }
          }
        },
        "responses": {
          "200": {
            "description": "OK"
          }
        }
      }
    }
  },
  "components": {
    "schemas": {
      "Person": {
        "type": "object",
        "properties": {
          "name": {
            "type": "string",
            "description": "The name of the new member.",
            "nullable": true
          },
          "favouritePet": {
            "allOf": [
              {
                "$ref": "#/components/schemas/Pet"
              }
            ],
            "description": "The pet the new member likes the most.",
            "nullable": true
          },
          "leastFavouritePet": {
            "allOf": [
              {
                "$ref": "#/components/schemas/Pet"
              }
            ],
            "description": "The pet the new member likes the least.",
            "nullable": true
          }
        },
        "additionalProperties": false
      },
      "Pet": {
        "type": "object",
        "properties": {
          "name": {
            "type": "string",
            "nullable": true
          }
        },
        "additionalProperties": false,
        "description": "Summary for Pet"
      }
    }
  }
}

I have also tested switching from dotnet6 to dotnet8, and both of them worked. Could you please see if this issue persist?

Issue seems to be solved. Thanks


Update URLs in map helper, remove temporary test changes

Summary of changes
Asana Ticket: 💲New fares | Update map targets

some tests were changed late last night to get a build to pass in CI, those changes are reverted here
use vanity URLs for map PDFs so content can manage this fully without requiring code changes when maps are updated

Assigned to: @meagonqz

Were you able to revert https://github.com/mbta/dotcom/pull/69/files#diff-61adea710aa3b31d7e497ed8e07397ddR41?

@meagonqz updated


Update nameplate capacity for standalone battery systems

Description
-Change capacity nameplate (system_capacity) to maximum discharge power AC for standalone battery systems
-Affects basis for CBI calculations
-Adds ui variable to defaults for capacity payments
Fixes #1036
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] This change requires a documentation update

Checklist:
If you have added a new compute module in a SSC pull request related to this one, be sure to check the Process Requirements.

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

I agree with Brian: For PV-Battery and Generic-Battery, the variable label should be Battery maximum discharge power, MWac, and the last sentence of the UI text should be revised to read "The battery maximum discharge power is shown here for reference only." You can move the label about inflation to the left to make room.
Also, for the Standalone Battery configuration, I think the variable labels are likely to overlap on Mac and Linux, so maybe move the capacity payment amount and capacity payment escalation to the left to allow more vertical space between the variables.


Add support for kotlin coroutines

It would be great to be able to declare suspend function like:
@Bean
suspend fun uppercase(value: String): String {
	return value.toUpperCase()
}

Please feel free to contribute

Please feel free to contribute

I have done an adapter in on one of my project but it's the good way to integrate it in spring-cloud-function.
To find the best way to do it, I should probably start by looking at how coroutine was integrated in spring boot webflux.
Do you know where coroutine's integration is done in spring boot webflux?

I have done an adapter in on one of my project but it's the good way to integrate it in spring-cloud-function.
To find the best way to do it, I should probably start by looking at how coroutine was integrated in spring boot webflux.
Do you know where coroutine's integration is done in spring boot webflux?

I do not, but perhaps @sdeleuze can point you in the right direction.

I do not, but perhaps @sdeleuze can point you in the right direction.

@olegz You could maybe try to leverage CoroutinesUtils#invokeSuspendingFunction.

HI @olegz, I have started working on this feature: WIP Pull request on my repo
I have start by implementing the possibility to declare suspend function as suspend (String) -> String.
I map it as (Function<Flux<>, Flux<>>) and call it for for each item of the Flux
But as with Flux, we can only use Function<Flux<*>, Flux<*>>, should I do the same with suspend function and only implement the mapping for suspend (Flow<String>) -> Flow<String> ?

I believe this has been addressed

Could you provide a working example with the new version, that is a suspended consumer? I couldn't get it working, and the documentation only mentiones non suspended examples. Thanks

Hi @tarjanik,
I told @olegz I would work on the documentation, but I didn't took the time to do.
I will try to work on i it this weekend.

Hi @apoupard
Thanks. One more question: is it supposed to work with Rabbitmq messaging, or only for controllers?


Intl: is not installed, or has been disabled.

Hello! I don't know if it's an issue per se but whatever I have tried I cannot install intl to my website. Pecl install can't build it, with apk add it does install it but the installation is not persistent when I have to restart the app. I follow every official and unofficial guide I have found but no persistence whatsoever.

Persistance can be solved by adding the commands to install in the startup script.

Hello! It doesn't work with a startup script either... I follow this guide: https://github.com/Azure/wordpress-linux-appservice/blob/main/WordPress/wordpress_adding_php_extensions.md and when the app restarts after step 7 the module is nowhere to be found...

That guide doesn't mention the startup script. That guide takes a module and places it within /home/site, which is persisted between restarts. If it's not, then WEBSITES_ENABLE_APP_SERVICE_STORAGE should be set to true.
Instead, I recommend following this guide to use apk add in a startup script, which you mentioned works as you expect.

I have checked and double-checked everything to no avail. I have managed to have it installed with the startup script on /home/dev/ every time it restarts but the website doesn't see it as enabled. It is just installed. Any advice?

Closing this as duplicate, as there is another recent thread.


[CARBONDATA-2817]Thread Leak in Update and in No sort flow

Issue :-  After Update Command is finished , Loading threads are not getting stopped.
Root Cause :-

In Update flow  DataLoadExecutor 's close method is not called so all Executors services are not closed.
In Exceptions are not handled property in AFDW class's closeExecutorService() which is cuasing Thread leak if Job is killed from SparkUI..

Solution :-

Add Task Completion Listener and  call close method of DataLoadExecutor  to it .
Handle Exception in closeExecutor Service so that all Writer steps Threads can be closed.

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?
NO
[ ] Any backward compatibility impacted?
NO
[ ] Document update required?
NO
[ ] Testing done
Verified in cluster manually .
[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
NA

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/7748/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/7772/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/6496/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/6137/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/6153/

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/6155/

retest this please

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/7785/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/6510/

@BJangir Please handle thread leak scenario for BatchSortWriter in case of any exception. DataWriterBatchProcessorStepImpl.java

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/6193/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/7817/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/6541/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/7830/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/6555/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/6209/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/7835/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/6560/

LGTM


Add option to disable the de-tiling

Description
The detiling can cause heavy performance drops and is currently automatically activated when applying any texture to the terrain. The calculations are being done even if all your textures in the texture list are set to not use texture detiling. It would be great to not have to use the detiling if no textures use it.

The detiling can cause heavy performance drops

Have you tested this theory with our shader? You could test it in a few minutes. You could also disable it yourself, but I think you'll find it is not as expensive as you think.

Detiling is already branched off in the shader if its not used.
https://github.com/TokisanGames/Terrain3D/blob/main/src/shaders/main.glsl#L240
Modern cards spend no time on it if detiling is 0. You can avoid the performance hit by setting all textures to 0. Or customize the shader and remove the code entirely.


boss_db process bottleneck

I was just reading boss_db.erl and realized it makes calls to a single process called boss_db to perform database operations. Wouldn't that result in One Giant Bottleneck?

Indeed it does! Well, at least it's only one bottleneck per Erlang node. ;)
Revamping BossDB to be sane is a priority for 1.0. Let me know if you're interested!

I've revamped BossDB to use connection/worker pools with Poolboy. Closing this issue.


Incorrect descriptor starting bytes

Hello, i am using an A3 lidar, I already changed the rplidar.py baudrate , now I am getting errors in rplidar line 211: dsize, is _single, dtype = self._read_descriptor()
and in line 189 in _read_descriptor
raise RPLidarException('Incorrect descriptor starting bytes')
rplidar.RPLidarException: Incorrect descriptor starting bytes
anyone has any idea how to fix this?

same  problem

I was having this same problem when I stopped my script without properly closing my serial connection/disconnecting from the lidar.  I solved it by catching any RPLidarException and then executing
lidar.stop_motor()
lidar.stop()
lidar.disconnect()
lidar = RPLidar(port)

Not all of these lines may be necessary, but in general, I found disconnecting and reconnecting solved my problem.

I can confirm the above - happens when you don't have a clean shutdown - catch the error and restart the device:
def collect_data(client):
    try:
        lidar = RPLidar(None, PORT_NAME)
        for scan in lidar.iter_scans():
            post(client, scan)
    except RPLidarException:
        lidar.stop()
        lidar.disconnect()
        collect_data(client)


Is there some benchmark data?

Is there some benchmark information about logcabin performance?

There's some information in chapter 10 of my PhD dissertation. However, that was done on a different version of the code. The current code has progressed a fair bit, so it may be better and worse in some regards. LogCabin master is specifically missing pipelining (#97), since the implementation wasn't quite ready to merge. See #95 for some more background.

Closing this issue. Feel free to re-open if you have more questions.


Fix duplicated files by removing the duplications

What does this PR do?
Remove duplicated files
Description of Task to be completed?
Improves code quality by removing duplicated files
Updates code climate by excluding certain files

Team, the squashing didn't work as I hoped and I tried doing it again but it failed. Sorry about this. Next time will try not making the same mistake


Is there a way to use client certificates with SolrNet.Cloud?

I was using client certificates with SolrNet before, and I am thinking of moving to SolrNet.Cloud now.
However, I see that there is no method for setting an ISolrConnection for the library.
SolrNet had this method:
var postConnection = new PostSolrConnection(connection, solrServerUrl);
Startup.Init<MyCollectionDocument>(postConnection);

...where you could customize the connection.
For SolrNet.Cloud there is only this one, that does not accept any ISolrConnection instance:
var cloudStateProvider = new SolrNet.Cloud.ZooKeeperClient.SolrCloudStateProvider(settings.SolrZookeeperUrl, settings.SolrTimeoutInSeconds);
await SolrNet.Cloud.Startup.InitAsync<MyCollectionDocument>(cloudStateProvider: cloudStateProvider, collectionName: CollectionName, isPostConnection: true);

Am I missing something?
Or this is a missing feature?
Update:
Hm... I see that SolrNet.Cloud depends on SolrNet, so perhaps the old method is used for cloud as well...
But calling both methods from above tries to register ISolrBasicOperations twice, resulting in error:
System.ApplicationException: Key 'SolrNet.ISolrBasicOperations`1[[Admincontrol.Common.FileService.SolrDocument, Admincontrol.Common.FileService, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]]' already registered in container
   at SolrNet.Utils.Container.Register(String key, Type serviceType, Converter`2 factory)
   at SolrNet.Cloud.Startup.<InitAsync>d__6`1.MoveNext()

I ended up overriding the ISolrOperationsProvider in my DI Container after the call to InitAsync().
Not the most beautiful thing, but seems to work...

Also, the docs here don't seem to be up-to-date:
https://github.com/SolrNet/SolrNet/blob/master/Documentation/Basic-usage-cloud.md
In the latest version of the lib, I see that SolrNet.Startup.Init<T>() does not accept a SolrCloudStateProvider and you obviously have to call SolrNet.Cloud.Startup.InitAsync() for initialization.

Fixed docs in https://github.com/SolrNet/SolrNet/commit/e602ed209b826ab4ac8b340fa6882e4a205d0da5

@kostas-kelgeorgiadis Could you share the override you did, I'm currently reviewing this part of the code and I might include a proper fix in a later PR.

Sure.
Here you go (there is some extra code that prevents deadlocks in web applications):
var cert = LoadCertificate(settings);

using (NoSynchronizationContextScope.Enter())
{
	var container = SolrNet.Cloud.Startup.Container;

	container.RemoveAll<SolrNet.Cloud.ISolrOperationsProvider>();
	var cloudStateProvider = new SolrNet.Cloud.ZooKeeperClient.SolrCloudStateProvider(settings.ZookeeperConnectionString, settings.TimeoutInSeconds);
	SolrNet.Cloud.Startup.InitAsync<T>(cloudStateProvider: cloudStateProvider, collectionName: settings.CollectionName, isPostConnection: true)
		.ConfigureAwait(false)
		.GetAwaiter()
		.GetResult();

	var requestFactory = cert is null
		? new HttpWebRequestFactory() as IHttpWebRequestFactory
		: new CertificateHttpWebRequestFactory(cert);
	container.RemoveAll<SolrNet.Cloud.ISolrOperationsProvider>();
	container.Register<SolrNet.Cloud.ISolrOperationsProvider>(c => new MySolrOperationsProvider(requestFactory, settings.TimeoutInSeconds));
}

public class MySolrOperationsProvider : SolrNet.Cloud.ISolrOperationsProvider
{
    private readonly IHttpWebRequestFactory _requestFactory;
    private readonly int _timeOutSeconds;

    public MySolrOperationsProvider(IHttpWebRequestFactory requestFactory, int timeOutSeconds)
    {
        _requestFactory = requestFactory;
        _timeOutSeconds = timeOutSeconds;
    }

    public ISolrBasicOperations<T> GetBasicOperations<T>(string url, bool isPostConnection = false)
    {
        var connection = new SolrNet.Impl.SolrConnection(url);
        connection.HttpWebRequestFactory = _requestFactory;
        connection.Timeout = (int)TimeSpan.FromSeconds(_timeOutSeconds).TotalMilliseconds;

        if (!isPostConnection)
        {
            return SolrNet.SolrNet.GetBasicServer<T>(connection);
        }
        else
        {
            var postConnection = new SolrNet.Impl.PostSolrConnection(connection, url);
            postConnection.HttpWebRequestFactory = _requestFactory;
            return SolrNet.SolrNet.GetBasicServer<T>(postConnection);
        }
    }

    public ISolrOperations<T> GetOperations<T>(string url, bool isPostConnection = false)
    {
        var connection = new SolrNet.Impl.SolrConnection(url);
        connection.HttpWebRequestFactory = _requestFactory;
        connection.Timeout = (int)TimeSpan.FromSeconds(_timeOutSeconds).TotalMilliseconds;

        if (!isPostConnection)
        {
            return SolrNet.SolrNet.GetServer<T>(connection);
        }
        else
        {
            var postConnection = new SolrNet.Impl.PostSolrConnection(connection, url);
            postConnection.HttpWebRequestFactory = _requestFactory;
            return SolrNet.SolrNet.GetServer<T>(postConnection);
        }
    }
}

public static class NoSynchronizationContextScope
{
    public static IDisposable Enter()
    {
        var context = SynchronizationContext.Current;
        SynchronizationContext.SetSynchronizationContext(null);
        return new Restorer(context);
    }

    private struct Restorer : IDisposable
    {
        private readonly SynchronizationContext _synchronizationContext;

        public Restorer(SynchronizationContext synchronizationContext)
        {
            _synchronizationContext = synchronizationContext;
        }

        public void Dispose()
        {
            SynchronizationContext.SetSynchronizationContext(_synchronizationContext);
        }
    }
}

public class CertificateHttpWebRequestFactory : IHttpWebRequestFactory
{
    private readonly X509Certificate _certificate;

    public CertificateHttpWebRequestFactory(X509Certificate certificate)
    {
        _certificate = certificate;
    }

    public IHttpWebRequest Create(Uri url)
    {
        var req = (HttpWebRequest)WebRequest.Create(url);
        req.ClientCertificates.Add(_certificate);
        return new HttpWebRequestAdapter(req);
    }

    public IHttpWebRequest Create(string url)
    {
        return Create(new Uri(url));
    }
}

However, I think I've found an extra problem, which I am not sure if it is a problem of my additions or a problem in the library.
It appears that registering and resolving more than one types of documents does not work as I was expecting.
So, if you call the 1st snippet of code for different types of , it fails to register and resolve them both.
It only works with one document type.
I hope this helps.

Added a PR for this, with some extra improvements.


All dialogs and side panels in the kit are broken (no-op)

Does this bug already exist in our backlog?

[X] I have checked and confirm this is a new bug.

Describe the issue
Unable to edit environment variables and Customized emails.
Expected Behavior
When editing emails and environment variables when click save it should save the changes I made. Currently, nothing happens when I edit and click save. Already used different Browser, issue still persist.
What solution are you experiencing the issue with?
Core
What solution version are you using?
4.31
What app or flow are you having the issue with?
CoE Admin Command Center
What method are you using to get inventory and telemetry?
None
Steps To Reproduce
Go to CoE Admin Command Center
Go to Customized emails or environment variable
In my case I selected "Edit mail: Send list to manager - failed or timed out". Edit the body
Nothing happens, it just sit on the dialogue to save.
Anything else?
n/a
AB#3253
AB#3317

The broader team is working on an update to Creator Kit which we expect early to mid next week at which point I will use it to update the apps in the kit before we ship the June release.
CoE side tracking bug: https://github.com/microsoft/coe-starter-kit/issues/8350

I got an early drop of the new creator kit and confirmed I am unblocked from doing all these fixes so we will be able to fix for June release


chore: Reduce and restrict innerHTML use

SUMMARY

innerHTML is gross... this PR adds linting rules to restrict use of this method, and replaces or annotates existing usage.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

There are a few plugins that were touched that need testing:

Chord
Heatmap
NVD3

Dynamic plugins have their web URL bundle sanitized... this should work, but if not, very few people will be affected since this is an Alpha feature.
Icon file paths are also being sanitized - we should check for these in general, though it seems things should work.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

This'll need tests for superset-ui-core utils added if we want to bother with that approach.

👍 this approach looks good and should help with any future issues in this area. Not sure why the frontend build is failing. Would be good to bring up an ephemeral env to test this more thoroughly


Proper alignment and Attractive Heading

hey @girordo I want to work on this issue will you assign it to me

while making pr make sure add ss to highlight your change


Create a tutorial video ？

maybe create a tutorial about thinkvim 。will be nice ？

There is a video out there on youtube but it is not english. An english one could be really nice. Also there should be more color schema options. Or we should think a better way to customize it.

@emeentag  you can change the colorschema by leader+ f+c

@emeentag you can change the colorschema by leader+ f+c

Thanks a lot works great but it is not changing the tabs color. I think it could be also fixed with airblade properties but i am not sure. Also we should update the doc section ColorScheme for this shortcut. And for the Defx usage we should definetly give another option like Nerdtree out of the box. Still there are some essential behaviours are missing. It is ok you can do many thing by calling methods but most of the time they are not working properly. i.e https://github.com/Shougo/defx.nvim/issues/87#issuecomment-503025538

@emeentag  i had provide the nerdtree .but the i found the defx is more useful than nerdtree. i will add it agin.

@taigacute thx a lot. This does not mean we can couple or triple or plugins etc. But you know there are some out there which are essentials and basics for most of the vim-ers. And as i see defx is new one so making a new one by default and as only option does not good to have in very first step. So mature plugins always can be attractive and a good point to start making a transition to a new one. Do you want me to create a feature request for it?

@taigacute   i have added the nerdtree .please update. the key is localleader + e

@taigacute thx a lot. it is better to create a feature request ticket for this change in a specific title. You know if people search it on internet this type of changes will be liste. No body expect this kind of change is implemented under a video tutorial ticket. Better to create separate tickets in next time.

i check the problem. you need open the current file in defx?

Exactly but it is not convenient. In nerd tree it is just map <leader>r :NERDTreeFind<cr> thats it

@emeentag  right?

Exactly! Anyway we are moving away the context  of the topic :D thx @taigacute

i used leader+ e. it can do this thing.

Yeah but when you make a file search and then select a file and then locate to that file it is massing up the things. I already swtich to nerdtree so i need time to try defx

i think this issue should be close and recreate a new issue about video


Update run script to use full path of forseti_*

Adding cla: yes label, as the PR author is a googler at the time of this PR was created.


Usage on OSX

While I work on Ubuntu, most of my colleagues use OSX. On Docker, there is docker-machine which allows to use docker in a VM rather transparently. Is there anything like that for Vagga? If not, is there any example of, say, a working vagrant configuration or something like that which could allow oSX users to run vagga?
I am not very familiar with Vagrant. I guess a basic example could be easy to setup, but one needs to take into account sharing of the work folder with proper user permissions, networks sharing and so on, so that everything works as smoothly as with vagga.

@andreaferretti Vagga is a Linux only software.
It uses some Linux kernel specific features(same story with Docker, although Docker over time created tools to power up VMs and run Linux inside them).
To be able to use Vagga they need to start a VM with Linux, through any method they want(Vagrant, VirtualBox + Linux image, VMWare Player + Linux image...).
A relatively modern Linux kernel is needed. For example it works with Ubuntu 12.04 LTS(almost 4 years).
Sadly, I am not familiar with Vagrant myself.
The main maintainer is currently on vacation, he might have some other suggestions...

@andreaferretti, you may try https://github.com/rrader/vagrant-vagga . I'm not sure you can use it as easy as a vagga on linux. (it creates a virtual machine, you need to shutdown it at some time, running vagga in multiple projects requires multiple virtual machines AFAIR, and so on).
Anyway, I may put some example Vagrantfile in the documentation. Is it something you are asking for?

@tailhook Thank you, this is exactly what I had in mind! I have an example Vagrantfile here but it works so so.
I think the plugin you linked is exactly what my colleagues need! Thank you again for your quick help! :-)


[FLOC-2299] Release notes warning

Looks fine.
But perhaps we should document some of the most important currently known limitations  before the release.
Or link to a list of issues with a "known limitations" tag...or something.
Or maybe that's just the entire list of bug issues.
Anyway lets merge this and improve things later.


Make From for AssetPath parse label.

Objective
When you use Path and join through AssetServer::load, server doesn't recognize labels.
It will be inconvenient if you have to hard code full str every time you load.
Solution
Thus, as convenience purpose, modified impl of From<PathBuf> for AssetPath to parse labels.

Changelog
Migration Guide

Backlog cleanup: closing due to inactivity. Could still be a good QoL change but can always be resurrected, if the concerns over panicking can be overcome.


Enable GPU on multi-node cluster

Please run microk8s.inspect and attach the generated tarball to this issue.
inspection-report-20200121_142427.tar.gz
I set up microk8s according to the quick start guide and added another node according to this.
I'm trying to enable GPU support as the node that I added has GPUs.
When I run microk8s.enable gpu on the master I get:
Enabling NVIDIA GPU
Aborting: NVIDIA kernel module not loaded.
Please ensure you have CUDA capable hardware and the NVIDIA drivers installed.
Failed to enable gpu

But then when I run microk8s.enable gpu on the node I get:
This MicroK8s deployment is acting as a node in a cluster. Please use the microk8s.enable on the master.

So what's the correct way to do this?
Thanks

@ktsakalozos, I tried your suggestion, but it doesn't appear to work. It doesn't fail with any error message, but the cluster does not appear to see the gpus
kubectl describe nodes  |  tr -d '\000' | sed -n -e '/^Name/,/Roles/p' -e '/^Capacity/,/Allocatable/p' -e '/^Allocated resources/,/Events/p'  | grep -e Name  -e  nvidia.com  | perl -pe 's/\n//'  |  perl -pe 's/Name:/\n/g' | sed 's/nvidia.com\/gpu:\?//g'  | sed '1s/^/Node Available(GPUs)  Used(GPUs)/' | sed 's/$/ 0 0 0/'  | awk '{print $1, $2, $3}'  | column -t

Output:
Node         Available(GPUs)  Used(GPUs)
node1                   0                0
node1                   0                0

I figure out the solution: you need to use the enable command on the nodes, but before joining them to the master.
microk8s enable gpu

Then, you run this command on the master:
microk8s.kubectl apply -f https://raw.githubusercontent.com/ubuntu/microk8s/master/microk8s-resources/actions/gpu.yaml

The gpus will be visible in the nodes description.

@croesuslab how do you see gpu is available?

They appear in the nodes description as "nvidia.com gpu"
You can use this command to get a better formatted view:
kubectl describe nodes  |  tr -d '\000' | sed -n -e '/^Name/,/Roles/p' -e '/^Capacity/,/Allocatable/p' -e '/^Allocated resources/,/Events/p'  | grep -e Name  -e  nvidia.com  | perl -pe 's/\n//'  |  perl -pe 's/Name:/\n/g' | sed 's/nvidia.com\/gpu:\?//g'  | sed '1s/^/Node Available(GPUs)  Used(GPUs)/' | sed 's/$/ 0 0 0/'  | awk '{print $1, $2, $3}'  | column -t


Updated port range per #266 and add FAQ entry per #256

This PR adds two updates on the FAQ page. First, it corrects the port ranges used by NDT referenced in #266 and adds an entry to mention where and how we host our servers, addressing #256

This change is

@georgiamoon I made an update based on your feedback.

@critzo should be good with a link to infrastructure in that first paragraph. LGTM.


Improve gui updating in make and unmake movce in terms of performance

move more logic into board where it should be...


Video Streaming support

This is a very cool project.
Just wanted to ask how difficult will it be to implement Video Streaming with Caching using AVAssetResourceLoader? Is it possible?

Yes it's possible. The most part of code is the same. The difference in AVPlayer initialization. You should add AVPlayerLayer as described here: https://developer.apple.com/library/ios/documentation/AudioVideo/Conceptual/AVFoundationPG/Articles/02_Playback.html#//apple_ref/doc/uid/TP40010188-CH3-SW2

@leshkoapps Thanks for the reply. I ended up writing a custom class conforming to the AVAssetResourceLoaderDelegate protocol and it worked.
A suggestion to make this project more popular would be to have the stream end point more flexible,
As of now I feel this is centred around Yandex.
Thanks again. :)

on ne peux pas active sa desoler


Evaluate holding current day's key

Currently, a key that is still valid (rolling period + rolling period count covers the current time) causes a whole publish API request to fail.
As this has come up a couple of times, perhaps it is better to hold the still valid key until the end of the UTC day, but allow others to be sent out immediately.
/assign

To be clear, this is to handle a still valid key being uploaded. Rather than rejecting the publish, we would embargo that key, not including it in any batches, until after it has expired + additional time to handle skew?

Correct.


[FLINK-10958] [table] Add overload support for user defined function

What is the purpose of the change
This pull request adds the overload support for user defined function.
Brief change log

add logical in UserDefinedFunctionUtil#getUserDefinedMethod which identifies the methods which are more applicable than others when multiple methods are applicable.

Verifying this change
This change added tests and can be verified as follows:

Added test that validates that overload take effects

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)

@sunjincheng121 thanks a lot for the review. I have updated the PR according to your comments. Could you help to take a look? Thanks a lot.

@dianfu Thanks for the update!
I think the PR is in a good shape. +1 to merged.
Best,
Jincheng


Better makefile

Improving makefile

With these changes, you would only need to do make -j db when you start working on the project, then you can do make -j tests and make -j ignored whenever you need to test your changes, and when you are finished working you may do make -j stop.
Note: -j flag is optional but recommended.

I would like to merge this to master and then merge it to my branch  which I provide sqlx support

@ayrat555 I think commit fc1b0c7 does what you wanted.

Can this be merged ?
I would like to merge it also in my pr


Version 3.1.1

Is version 3.1.1 coming anytime soon? :)

versio 3.1.2 is out now

Looks like we missed 3.1.1. There is a rebase for 3.1.2 in PR.


Remove All from selected options once another node is selected

This has been addressed in the new implementation.