Add prop tests

Problem
The rewards program has some extensive test coverage, but it's missing
regression testing to ensure the arithmetic in reward and "last seen rate"
calculation is tight.
Summary of Changes
Add prop tests to test the rewards and rate arithmetic.

Almost finished integrating your feedback @joncinque, but wanted to verify: since we've changed the inputs using prop_compose!, I should be removing the persisted regression failure (proptest-regressions dir) as well, right? Since this would be for the previous input set.

Ok great! Yeah that would probably be better

@joncinque I've done some more calculations using 1e18, as you suggested, and I've noticed a few things.
First of all, the constraint for the calculation actually requires (current - last) * balance to be less than or equal to u64::MAX * 1e9, which is significantly smaller than u128::MAX. This is because of the conversion to u64 that happens at the end of the calculate_eligible_rewards function.
Secondly, the margins for token account balance and marginal rate are quite tight when using 1e18. For example, even though 10% of circulating SOL supply is extremely high, the maximum token account balance as a percentage of total token supply is extremely low (< 0.0001%).
Since we need the resulting rewards to be able to fit into a u64, I think these constraints actually make sense. That being said, I wonder if I should document somre more reasonable benchmarks for marginal rewards (< 10%) and/or update the documentation.


[FIX-21] building docs locally

Resolves #21.
Applied changes

Removed sorting argument from CitationsBibliography
Adjusted syntax of the latest version of makedocs using the plugin keyword
Fixed typo in docstrings
Removed docstring of the QEDbase module (this feature might be broken)
Added QEDbase as a dependency to docs

Why there are a make.jl and make_local.jl. Looks like the make.jl is not working standalone.

Please use an "absolute" path here:
https://github.com/QEDjl-project/QEDbase.jl/blob/b3a7ad63d9e95e1148be873c354ac46692bbed77/docs/make_local.jl#L3
project_path = Base.Filesystem.joinpath(Base.Filesystem.dirname(Base.source_path()), "..")

Please use an "absolute" path here:
https://github.com/QEDjl-project/QEDbase.jl/blob/b3a7ad63d9e95e1148be873c354ac46692bbed77/docs/make_local.jl#L3
project_path = Base.Filesystem.joinpath(Base.Filesystem.dirname(Base.source_path()), "..")

Oh, this is some ancient leftover. 0945480 removes that.

Looks good. The documentation builds and didn't saw anything strange in the rendered documentation.


fix: user cannot pay on non-notice project

What does this PR do and why?
Provide a description of what this PR does. Link to any relevant GitHub issues, Notion tasks or Discord discussions.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] (if relevant) I have tested this PR in all supported browsers.
[ ] (if relevant) I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #3991   👈

This comment was auto-generated by Graphite.


Set ACCOUNT_DEFAULT_HTTP_PROTOCOL to 'http' in production

allauth redirects to GitHub for the social login using a HTTP URL by default.
However the GitHub app is setup to expect a HTTPS URL. So we need to instruct
allauth to use one as well.
Fixes #1520.

Meh, sorry. Was inserting the title/description of a totaly different topic. Updated it.


[MSCA-1] - added test file

Description

I added sda to test something

Testing

I tested and made sure throug blbla that this bugfix was working by this maethodj

It is wrong

I have resovled it


PriceRange.High and PriceRange.Low are always 0

There seems to be a bug in the session.GetOffers() function. The returned PriceRange.High and PriceRange.Low are always zero. Here is a reproducible example:
package main

import (
	"context"
	"fmt"
	"log"
	"time"

	"github.com/krisukox/google-flights-api/flights"
)

func main() {
	session, err := flights.New()

	if (err != nil) {
		log.Fatal(err)
	}

	flightArgs := flights.Args{
		Date:        time.Date(2024, 6, 16, 0, 0, 0, 0, time.Now().Location()),
		ReturnDate:  time.Date(2024, 8, 11, 0, 0, 0, 0, time.Now().Location()),
		SrcCities:   []string{"San Diego"},
		DstCities:   []string{"New York"},
		Options:     flights.OptionsDefault(),
	}

	_, priceRange, err := session.GetOffers(
		context.Background(),
		flightArgs,
	)

	if (err != nil) {
		log.Fatal(err)
	}

	if priceRange != nil {
		fmt.Printf("High price %d\n", int(priceRange.High))
		fmt.Printf("Low price %d\n", int(priceRange.Low))
	}
}

This is the output:
High price 0
Low price 0

Thanks for working on this :).

Realized the high and low price refers to what Google Flights thinks is a usual high/low for the flight. Google Flights doesn't always provide this data so it is zero for some flights.


Netlify Build failed

I used your repozitory as a copy with my MD files.
I got this build log with errors:
10:00:56 PM: build-image version: 3b96229e32b19cdf63f20dc1fc9331cd240c2d01 (focal)
10:00:56 PM: buildbot version: 3b96229e32b19cdf63f20dc1fc9331cd240c2d01
10:00:56 PM: Fetching cached dependencies
10:00:56 PM: Failed to fetch cache, continuing with build
10:00:56 PM: Starting to prepare the repo for build
10:00:56 PM: No cached dependencies found. Cloning fresh repo
10:00:56 PM: git clone --filter=blob:none https://github.com/tomashak/obsidian_digital_garden
10:00:56 PM: Preparing Git Reference refs/heads/main
10:00:58 PM: Starting to install dependencies
10:00:58 PM: Python version set to 3.8
10:00:59 PM: Attempting Ruby version 2.7.2, read from environment
10:00:59 PM: Using Ruby version 2.7.2
10:01:00 PM: Started restoring cached go cache
10:01:00 PM: Finished restoring cached go cache
10:01:00 PM: go version go1.19.10 linux/amd64
10:01:01 PM: Using PHP version 8.0
10:01:02 PM: v18.16.0 is already installed.
10:01:03 PM: Now using node v18.16.0 (npm v9.5.1)
10:01:03 PM: Enabling Node.js Corepack
10:01:03 PM: Started restoring cached build plugins
10:01:03 PM: Finished restoring cached build plugins
10:01:03 PM: Started restoring cached corepack dependencies
10:01:03 PM: Finished restoring cached corepack dependencies
10:01:03 PM: Started restoring cached yarn cache
10:01:03 PM: Finished restoring cached yarn cache
10:01:03 PM: No yarn workspaces detected
10:01:03 PM: Started restoring cached node modules
10:01:03 PM: Finished restoring cached node modules
10:01:04 PM: Installing npm packages using Yarn version 1.22.19
10:01:04 PM: yarn install v1.22.19
10:01:04 PM: [1/4] Resolving packages...
10:01:04 PM: [2/4] Fetching packages...
10:01:26 PM: [3/4] Linking dependencies...
10:01:26 PM: warning "@emotion/react ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>10:01:26 PM: warning "@emotion/react > @emotion/babel-plugin ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>10:01:26 PM: warning "eslint-config-next > @typescript-eslint/parser > @typescript-eslint/typescript-estree ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| >= 3.2.0-dev || >= 3.3.0-dev || >= 3.4.0-dev || >= 3.5.0-dev || >= 3.6.0-dev || >= 3.6.0-beta || >= 3.7.0-dev || >= 3.7.0-beta".
10:01:39 PM: [4/4] Building fresh packages...
10:01:39 PM: Done in 35.16s.
10:01:39 PM: npm packages installed using Yarn
10:01:40 PM: Install dependencies script success
10:01:40 PM: Starting build script
10:01:41 PM: Detected 1 framework(s)
10:01:41 PM: "next" at version "12.1.0"
10:01:41 PM: Section completed: initializing
10:01:42 PM:
10:01:42 PM: Netlify Build
10:01:42 PM: ────────────────────────────────────────────────────────────────
10:01:42 PM:
10:01:42 PM: ❯ Version
10:01:42 PM:   @netlify/build 29.12.1
10:01:42 PM:
10:01:42 PM: ❯ Flags
10:01:42 PM:   baseRelDir: true
10:01:42 PM:   buildId: 6484d67609e5ab1baf7796b8
10:01:42 PM:   deployId: 6484d67609e5ab1baf7796ba
10:01:42 PM:
10:01:42 PM: ❯ Current directory
10:01:42 PM:   /opt/build/repo
10:01:42 PM:
10:01:42 PM: ❯ Config file
10:01:42 PM:   No config file was defined: using default values.
10:01:42 PM:
10:01:42 PM: ❯ Context
10:01:42 PM:   production
10:01:43 PM:
10:01:43 PM: ❯ Installing plugins
10:01:43 PM:    -<EMAIL_ADDRESS>10:01:51 PM:
10:01:51 PM: ❯ Using Next.js Runtime - v4.37.4
10:01:53 PM:
10:01:53 PM: @netlify/plugin-nextjs (onPreBuild event)
10:01:53 PM: ────────────────────────────────────────────────────────────────
10:01:53 PM:
10:01:53 PM: No Next.js cache to restore.
10:01:53 PM: Netlify configuration property "build.environment.NEXT_PRIVATE_TARGET" value changed.
10:01:53 PM:
10:01:53 PM: (@netlify/plugin-nextjs onPreBuild completed in 19ms)
10:01:53 PM:
10:01:53 PM: Build command from Netlify app
10:01:53 PM: ────────────────────────────────────────────────────────────────
10:01:53 PM:
10:01:53 PM: $ yarn build
10:01:53 PM: yarn run v1.22.19
10:01:53 PM: $ rm -rf graph-data.json && next build
10:01:54 PM: warn  - SWC minify release candidate enabled. https://nextjs.org/docs/messages/swc-minify-enabled
10:01:54 PM: warn  - No build cache found. Please configure build caching for faster rebuilds. Read more: https://nextjs.org/docs/messages/no-cache
10:01:54 PM: Browserslist: caniuse-lite is outdated. Please run:
10:01:54 PM:   npx browserslist@latest --update-db
10:01:54 PM:   Why you should do it regularly: https://github.com/browserslist/browserslist#browsers-data-updating
10:01:54 PM: info  - Checking validity of types...
10:01:54 PM: warn  - No ESLint configuration detected. Run next lint to begin setup
10:01:54 PM: info  - Creating an optimized production build...
10:01:55 PM: Browserslist: caniuse-lite is outdated. Please run:
10:01:55 PM: npx browserslist@latest --update-db
10:01:55 PM: Why you should do it regularly:
10:01:55 PM: https://github.com/browserslist/browserslist#browsers-data-updating
10:02:04 PM: info  - Compiled successfully
10:02:04 PM: info  - Collecting page data...
10:02:06 PM: Error: ENOENT: no such file or directory, lstat '/opt/build/repo/graph-data.json'
10:02:06 PM:     at Object.lstatSync (node:fs:1593:3)
10:02:06 PM:     at Object.isFile (/opt/build/repo/.next/server/chunks/403.js:264:62)
10:02:06 PM:     at constructGraphData (/opt/build/repo/.next/server/chunks/403.js:751:65)
10:02:06 PM:     at /opt/build/repo/.next/server/pages/index.js:69:100 {
10:02:06 PM:   errno: -2,
10:02:06 PM:   syscall: 'lstat',
10:02:06 PM:   code: 'ENOENT',
10:02:06 PM:   path: '/opt/build/repo/graph-data.json'
10:02:06 PM: }
10:02:07 PM: info  - Generating static pages (0/12)
10:02:11 PM: info  - Generating static pages (3/12)
10:02:11 PM: Error occurred prerendering page "/". Read more: https://nextjs.org/docs/messages/prerender-error
10:02:11 PM: Error: ENOENT: no such file or directory, open '/opt/build/repo/posts/index.md'
10:02:11 PM:     at Object.openSync (node:fs:601:3)
10:02:11 PM:     at Object.readFileSync (node:fs:469:35)
10:02:11 PM:     at Object.readFileSync (/opt/build/repo/.next/server/chunks/403.js:290:58)
10:02:11 PM:     at getSinglePost (/opt/build/repo/.next/server/chunks/403.js:704:85)
10:02:11 PM:     at getStaticProps (/opt/build/repo/.next/server/pages/index.js:72:92)
10:02:11 PM:     at renderToHTML (/opt/build/repo/node_modules/next/dist/server/render.js:465:26)
10:02:11 PM:     at async /opt/build/repo/node_modules/next/dist/export/worker.js:269:36
10:02:11 PM:     at async Span.traceAsyncFn (/opt/build/repo/node_modules/next/dist/trace/trace.js:75:20)
10:02:11 PM: info  - Generating static pages (6/12)
10:02:11 PM: info  - Generating static pages (9/12)
10:02:12 PM: info  - Generating static pages (12/12)
10:02:12 PM: > Build error occurred
10:02:12 PM: Error: Export encountered errors on following paths:
10:02:12 PM: 	/
10:02:12 PM:     at /opt/build/repo/node_modules/next/dist/export/index.js:498:19
10:02:12 PM:     at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
10:02:12 PM:     at async Span.traceAsyncFn (/opt/build/repo/node_modules/next/dist/trace/trace.js:75:20)
10:02:12 PM:     at async /opt/build/repo/node_modules/next/dist/build/index.js:1024:17
10:02:12 PM:     at async Span.traceAsyncFn (/opt/build/repo/node_modules/next/dist/trace/trace.js:75:20)
10:02:12 PM:     at async /opt/build/repo/node_modules/next/dist/build/index.js:898:13
10:02:12 PM:     at async Span.traceAsyncFn (/opt/build/repo/node_modules/next/dist/trace/trace.js:75:20)
10:02:12 PM:     at async Object.build [as default] (/opt/build/repo/node_modules/next/dist/build/index.js:82:25)
10:02:12 PM: error Command failed with exit code 1. (https://ntl.fyi/exit-code-1)
10:02:12 PM: info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
10:02:12 PM:
10:02:12 PM: "build.command" failed
10:02:12 PM: ────────────────────────────────────────────────────────────────
10:02:12 PM:
10:02:12 PM:   Error message
10:02:12 PM:   Command failed with exit code 1: yarn build (https://ntl.fyi/exit-code-1)
10:02:12 PM:
10:02:12 PM:   Error location
10:02:12 PM:   In Build command from Netlify app:
10:02:12 PM:   yarn build
10:02:12 PM:
10:02:12 PM:   Resolved config
10:02:12 PM:   build:
10:02:12 PM:     command: yarn build
10:02:12 PM:     commandOrigin: ui
10:02:12 PM:     environment:
10:02:12 PM:       - NEXT_PRIVATE_TARGET
10:02:12 PM:     publish: /opt/build/repo/.next
10:02:12 PM:     publishOrigin: ui
10:02:12 PM:   plugins:
10:02:12 PM:     - inputs: {}
10:02:12 PM:       origin: ui
10:02:12 PM:       package: '@netlify/plugin-nextjs'
10:02:13 PM: Caching artifacts
10:02:13 PM: Started saving node modules
10:02:13 PM: Finished saving node modules
10:02:13 PM: Started saving build plugins
10:02:13 PM: Finished saving build plugins
10:02:13 PM: Started saving corepack cache
10:02:13 PM: Finished saving corepack cache
10:02:13 PM: Started saving yarn cache
10:02:16 PM: Finished saving yarn cache
10:02:16 PM: Started saving pip cache
10:02:16 PM: Finished saving pip cache
10:02:16 PM: Started saving emacs cask dependencies
10:02:16 PM: Finished saving emacs cask dependencies
10:02:16 PM: Started saving maven dependencies
10:02:16 PM: Finished saving maven dependencies
10:02:16 PM: Started saving boot dependencies
10:02:16 PM: Finished saving boot dependencies
10:02:16 PM: Started saving rust rustup cache
10:02:16 PM: Finished saving rust rustup cache
10:02:16 PM: Started saving go dependencies
10:02:16 PM: Finished saving go dependencies
10:02:17 PM: Build failed due to a user error: Build script returned non-zero exit code: 2
10:02:17 PM: Failing build: Failed to build site
10:02:17 PM: Failed during stage 'building site': Build script returned non-zero exit code: 2 (https://ntl.fyi/exit-code-2)
10:02:17 PM: Finished processing build request in 1m21.644s

Your repository is missing an index.md file. Create one in the posts folder (https://github.com/tomashak/obsidian_digital_garden/tree/main/posts)


current_user - wrong number of arguments (given 0, expected 1)

I recently created a new rails app, and decided to use Devise 4.2.  In the past I have used version 3, and used to be able to do things like current_user.inspect for debugging and logging.
When I try that now on Devise 4.2, I get the following error:
wrong number of arguments (given 0, expected 1)
Am I doing something wrong? or did something change in those regards?
Thanks in advance!

Never mind, I was originally using a custom user auth, with a password_digest.  When I switched to devise, I forgot to remove that from my users model.  Once I did that this error was resolved.
Not quite sure what the conflict was, but will close this issue now.
Thanks again!

@pwgustafson thanks for your follow up post explaining how you fixed your problem. I had the same issue, was using a User class that I rolled myself until I switched to Devise. After removing the 'password digest' field I was able to query the user model without the error.


add context for expect

added because of https://roc.zulipchat.com/#narrow/stream/231634-beginners/topic/.E2.9C.94.20.60expect.60.20is.20wonderful!

Thanks @brian-carroll :heart:


Fix the co-design package versions on GitHub

The co-design package on GitHub packages (https://github.com/codidact/co-design/packages/245635) is obviously not in sync with the version published on NPM (https://www.npmjs.com/package/@codidact/co-design). Specifically, the GitHub packages version is 0.8.0, but the NPM one says version 0.12.0 is published.
Could we fix the package on GitHub by either:

Altering any release scripts to push to GitHub packages as well as npm and pushing say the last released version (as of this issue) to GitHub packages
Taking down the package from GitHub packages (maybe by making the repo private, taking the package down, and then marking it public again?)

This ensures consistency across platforms and ensures that clients will always be able to fetch the actual latest version of this package. Thanks!

Is this still an issue?


Valhalla on Flutter

Following on from this issue: https://github.com/valhalla/valhalla/issues/4893
What would be the challenges to overcome to achieve this? To get this kind of thing working in a flutter ios / mac os projects for example can be quite messy with ffi and linking static libraries in, in my experience

Sounds about right.
Valhalla recently dropped Conan in favor of vcpkg for managing build toolchains and dependencies. Conan was a major pain for mobile builds. Vcpkg makes this super easy using it's https://github.com/microsoft/vcpkg/tree/master/triplets (note the community folder inside this as well). I'm working right now on revamping the build process to use vcpkg and so far it's going great vs the old ways 😄
My goals are to get that published and building through CI pretty soon here. Hopefully that will help figuring out a flutter/ffi. The beauty of using valhalla on a mobile device, is you only really need a handful of functions for the preferred use case. That being a server runs the tile builds, then you simply sync the tiles or tar(s) to the device. My use case for example just uses the route function to fetch routes from a offline downloaded tileset (tar or folder). This is demoed for iOS and android in the Tests folders. That in mind, it may be simpler than you'd expect to get flutter ffi working.

Sounds awesome! I need /isochrones and /expansion for my use case. Is there some trickiness in including all those major endpoints? yeh I wouldn't recommend building the routing tiles on mobile haha

It depends. The process is:
1. Common Wrapper

Determine the minimum valhalla c++ to accomplish your goal. This typically means searching the valhalla src directory for the entry point of the server endpoint. In the case of route generation this was the valhalla actor.
Write a c++ function that takes a Valhalla config and simple args and returns simple arg(s) (typically json strings since valhalla supports web servers). See ValhallaActor::route.

2. Android

Write the JNI wrapper: Java_com_valhalla_valhalla_ValhallaKotlin_route. JNI looks a lot worse than it is, especially with AI assistance. Make sure to capture errors and return them in the string. This avoids c++ crashing the app with an uncaught exception.
Expose the JNI function in the kotlin JNI loader. See: ValhallaKotlin.kt#L10
Build a nice Kotlin function that cleanly implements the behavior (note this function will soon use OpenAPI typed models instead of pre-formatted json strings). Valhalla.kt#L12

3. Apple/iOS

Apple more readily supports c++, but we still need a wrapper to catch C++ exceptions (same as the JNI, but much cleaner). See: route
We make use of the function in the Objective C++ layer (this is likely to go away now that Swift-C++ interop is available)
Build a wrapper function that makes use of the OpenAPI codables.

Any additional language can either repeat the process like Android/Apple or it could use those call into the Android/Apple functions.
As a note, there are definitely opportunities to streamline the c++, share some common functionality like wrapping errors, add testing, etc.


The ZAP project vanishhed. Remove from smoke tests.

Codecov Report
Base: 92.94% // Head: 92.94% // No change to project coverage :thumbsup:

Coverage data is based on head (ac96ddc) compared to base (891aa95).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1643   +/-   ##
=======================================
  Coverage   92.94%   92.94%
=======================================
  Files         350      350
  Lines       25030    25030
=======================================
  Hits        23263    23263
  Misses       1767     1767

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


currentTerminal isn't being used when opening universal terminal via keybind

When opening a universal terminal via the key_key.ae2.wireless_terminal ("Open Wireless Terminal") keybind, it always opens the same terminal, with no way to influence which one that is.
In the case of using ExtendedAE's extended pattern provider terminal in a universal one, the extended pattern provider is opened each time. The item's currentTerminal nbt tag is not being used in this case.
The expected behavior here would be opening the most recently opened terminal (like it does when opening via right-clicking with the item), or alternatively being able to configure which one to open for the keybind.

key_key.ae2.wireless_terminal is supposed to open either the normal wireless terminal, or the wireless crafting terminal, preferably the wireless crafting terminal in the universal terminal (regardless what the current terminal is)
it seems tho as the wireless extended pattern provider terminal is providing that keybind as the one to open the extended pattern provider terminal (it should use the same  keybind ae2wtlibs wireless pattern provider terminal uses)
I'll make a PR to ExtendedAE to fix this

Awesome, thank you!


Ignores npm proxy settings

It looks like when running yarn docz dev it fails to load the local npm config.
I work behind a corporate proxy, which I have to configure in npm config .npmrc, however when docz tries to install its dependencies on first run, it ignores the npm config and just times out.

Hello !
Starting from version 2.0.0-rc.33 docz doesn't install any additional dependencies. So this problem should be solved.
Which version are you on ?


consider changing request lifecycle

A few months back the request lifecycle looked something like this:

User sends request to routing layer.
Routing layer forwards the request to the server.
Server responds to routing.
Routing responds to user.

We concluded that this was bad because this meant that the (potentially large) value now made two hops: Server to routing and routing to user. The simple solution at the time was to make the routing layer simply respond with the addresses of the correct server and have the user communicate directly with the server. The result was a request lifecycle that looks like this:

User sends a key to the routing layer.
The routing layer responds with the addresses of the set of servers responsible for this key.
The user caches these addresses and sends the request directly to the server.
The server responds.
The user relies on its cache for future requests until a server receives a message for a key it's not responsible for. The server then tells the user to invalidate its cache for that key and return to step 1.

However, we’ve since changed the architecture of the user & routing components to make them asynchronous. When a user sends a request, it includes it’s IP address in the request so that the routing layer knows where to respond to, because it might have to make a request to determine the correct replication factor for the key. In light of that change, we should change the request pattern:

User sends request to routing layer.
Routing layer forwards request along to one of the correct servers.
Server responds directly to user.

Pros:

This reduces the maximum number of roundtrips for a request from 4 to 3.
This could potentially be even faster if the routing layer is colocated with the storage layer (see #67).

Cons:

Forces the routing layer to be invoked on every single request. i.e., the minimum number of requests goes from 2 to 3. Whether this is a net negative or positive depends on the workload.

cc @cw75

For PUT requests, the VALUE will be passing through the routing thread I guess? This could be another drawback.


Tiny File Dialogs (suggestion)

This is sort of a continuation of my request to add a color picker 9 days ago. I ended up finding something on the GitHub for Portable File Dialogs.
https://sourceforge.net/projects/tinyfiledialogs/
This C++ library is also cross-platform and has more features than PFD (including a color picker). I haven't tested it out myself though so I don't know how well it works on different operating systems. It might be worth a look though.

Hello again! Thanks for the suggestion! The main problem I have with Tiny File Dialogs is that it is not asynchronous, while Portable File Dialogs behaves asynchronously by default. So I think this project should stick to PFD whenever possible. However, as you said it does not support native colour pickers, so we could make an exception here. I would try to integrate TFD and see if I can create a NativeColorPicker node with it.


[Question] jQuery Version via Webpack(er)

I'm trying to install the jQuery version of autocomplete.js via Webpacker into my Rails application. I've tried many combinations of plugins, aliases, and importing with no luck.
Here are (what I think) the relevant bits of my configuration. What is the correct/recommended way to get this working?
environment.js
const { environment } = require("@rails/webpacker")
const webpack = require("webpack")
const coffee = require("./loaders/coffee")

environment.plugins.prepend(
  "Provide",
  new webpack.ProvidePlugin({
    $: "jquery",
    jQuery: "jquery",
    algoliasearch: "algoliasearch",
    autocomplete: "autocomplete",
    "sticky-kit": "sticky-kit",
    "sticky-table-headers": "sticky-table-headers",
  })
)

const config = environment.toWebpackConfig()

config.resolve.alias = {
  jquery: "jquery/src/jquery",
  autocomplete: "autocomplete.js/index_jquery",
  "sticky-kit": "sticky-kit/dist/sticky-kit",
  "sticky-table-headers": "sticky-table-headers/js/jquery.stickytableheaders",
}

environment.loaders.append("coffee", coffee)

module.exports = environment

application.js
/* eslint no-console:0 */

// IE 11 polyfills
import "core-js"
import "element-closest"
import "classlist-polyfill"

// Stimulus
import { Application } from "stimulus"
import { definitionsFromContext } from "stimulus/webpack-helpers"

const application = Application.start()
const context = require.context("controllers", true, /.js$/)
application.load(definitionsFromContext(context))

// vendor
import {} from "jquery-ujs"
import {} from "sticky-kit"
window.Cookies = require("js-cookie")
require("waypoints/lib/jquery.waypoints.js")

I looked at your configuration the only line that looks like wrong to me is the following:
autocomplete: "autocomplete.js/index_jquery"

Did you tried something like:
autocomplete: "autocomplete.js/dist/autocomplete.jquery.js"

I'm not sure it will solve the issue though. If the issue persist, could you provide us a wokring example of your configuration? It will help a lot to better understand the issue.

autocomplete: "autocomplete.js/dist/autocomplete.jquery.js" worked great, thank you!


[Feature Request][Contribution Advice] Adding Vimtutor lessons

The vimtutor is an excellent resource and I always point new users to it when learning vim. I love the tutor, but I feel that it stops short of exploring some vim features that I use constantly myself. To remedy this, I'd like to contribute some of my Sunday's to adding lessons to the tutor.
Specifically, I was thinking to add lessons on:

Registers (yanking and pasting to and from, and listing) [replacing text with text from register]
Buffers (new buffer, listing, navigating {by number and completion}, closing)
Macros (creating, editing, using, highlight crossover with Registers)
Windows (splitting, navigating, closing) [already touched peripherally because of :help]
Tabs (new tabs, navigating, closing) [left till last because Buffers are a better solution to the problem most people try to solve with Tabs]

There are many things that could be added to the tutor, but the longer it gets the less useful it becomes overal. It's probably better to make a "part 2".  So the user would to the current tutor, then go use Vim for a while, and when feeling the need to learn more do "part 2".  It avoids "learning fatigue".

Understood. What would be the best way to add a part 2? There's already a cmdline switch for choosing languages, could a switch be added select a "second chapter"?

vimtutor -c 2
with a default of -c 1

I.e.
vimtutor [-g] [language] [-c] [chapter] [--list]

Where --list can list the chapters with a short description?

Seems the likeliest route is to add a tutor/tutor02.vim file to handle all files/translations for chapter two, as tutor/tutor.vim does for the original.
Chapter two files would be tutor02, tutor02.utf, ...tutor02.xx.*
vimtutor would select the appropriate chapter config based on -c cc
--list would need to be a hard-coded list like:
01 (default) - {en (default), bg, ca, cs, da, de, el, eo, es, fr, hr, hu, it, ja, ko, lv, nb, nl, no, pl, pt, ru, sk, sr, sv, tr, uk, vi, zh}
02 - {en (default)}

And updated when translations or chapters added.

I saw this was still open, what else can be done in the vimtutor section?
I thought about building a vimtutor lesson concerning the vimrc, and then perhaps include a 'settings pane' option in the vimtutor lesson where new users could get the basics of a .vimrc file laid out for them.
Things I would be adding:

Vimtutor lesson 3 - the vimrc file
A program to view, select and change settings fields in a vimrc, existing or not.
(Skill and Time Permitting) support for real time vimrc updating

This would be my first contribution to an Open source project, but I am fairly well versed in C and C++. I plan on opening another issue/making a pull request for this idea, but am also open to feedback. Please let me know what you might add or change about the idea!


It doesn't work

Followed the steps that are described on README but with no success. As seen on other tutorials I created webpack.config.js  file on the root directory. Added the code lines to call the plugin as said on the README but still got the 23 errors. Also searched in node_modules/react-scripts/config/webpack.config.js and added the piece of code from README. Searched for module.exports but it doesn't appear as it shows on many tutorials. It appears like:
// This is the production and development configuration.     // It is focused on developer experience, fast rebuilds, and a minimal bundle.     module.exports = function (webpackEnv) {       const isEnvDevelopment = webpackEnv === 'development';       const isEnvProduction = webpackEnv === 'production';
Any help will be much appreciated because I've tried everything what I found on internet. Downgrading webpack, using react-app-rewired,...
Nothing is working. Thought that this solution could fix the problem as it seems more concise. But no success.
Here the errors:
`Compiled with problems:X

ERROR in ./node_modules/body-parser/lib/read.js 24:11-26

Module not found: Error: Can't resolve 'zlib' in '/home/zaesar/Escritorio/coding/Github/webpack-intallation-exercise/webpack-installation-exercise/node_modules/body-parser/lib'

BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
    - add a fallback 'resolve.fallback: { "zlib": require.resolve("browserify-zlib") }'
    - install 'browserify-zlib'
If you don't want to include a polyfill, you can use an empty module like this:
    resolve.fallback: { "zlib": false }

ERROR in ./node_modules/content-disposition/index.js 19:15-39

Module not found: Error: Can't resolve 'path' in '/home/zaesar/Escritorio/coding/Github/webpack-intallation-exercise/webpack-installation-exercise/node_modules/content-disposition'

BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
    - add a fallback 'resolve.fallback: { "path": require.resolve("path-browserify") }'
    - install 'path-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
    resolve.fallback: { "path": false }

ERROR in ./node_modules/cookie-signature/index.js 4:13-30

Module not found: Error: Can't resolve 'crypto' in '/home/zaesar/Escritorio/coding/Github/webpack-intallation-exercise/webpack-installation-exercise/node_modules/cookie-signature'

BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
    - add a fallback 'resolve.fallback: { "crypto": require.resolve("crypto-browserify") }'
    - install 'crypto-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
    resolve.fallback: { "crypto": false }

ERROR in ./node_modules/destroy/index.js 15:17-41

Module not found: Error: Can't resolve 'fs' in '/home/zaesar/Escritorio/coding/Github/webpack-intallation-exercise/webpack-installation-exercise/node_modules/destroy'

ERROR in ./node_modules/destroy/index.js 17:13-30

Module not found: Error: Can't resolve 'stream' in '/home/zaesar/Escritorio/coding/Github/webpack-intallation-exercise/webpack-installation-exercise/node_modules/destroy'

BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
    - add a fallback 'resolve.fallback: { "stream": require.resolve("stream-browserify") }'
    - install 'stream-browserify'
If you don't want to include a polyfill, you can use an empty module like this:
    resolve.fallback: { "stream": false }

ERROR in ./node_modules/destroy/index.js 19:11-26

Module not found: Error: Can't resolve 'zlib' in '/home/zaesar/Escritorio/coding/Github/webpack-intallation-exercise/webpack-installation-exercise/node_modules/destroy'

BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

   ...`

By default, Create React App uses an internally-defined configuration. One way to access this configuration in order to modify it is to eject. Alternate methods appear to be listed here: https://marmelab.com/blog/2021/07/22/cra-webpack-no-eject.html

Thanks Richy but as I said on the first comment I used already react-app-rewired followed this tutorial as it suggests on the link that you provided. I tried dozens of stack overflow links. With no success. Any help or better explanation about the steps how to achieve it would be super appreciated.

I started a new app. wrote on the terminal npm install node-polyfill-webpack-plugin. Added on react-scripts/config/webpack.config.js the code showed on README:
`const NodePolyfillPlugin = require('node-polyfill-webpack-plugin');

module.exports = {
    // Other rules...
    plugins: [
	    new NodePolyfillPlugin()
    ]
};`

And agian the same problem:
`BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
 This is no longer the case. Verify if you need this module and configure a polyfill for it.`

Any suggestions would be appreciated, because I can't figure out what is wrong.

Added on react-scripts/config/webpack.config.js

I don't think that would work. Try ejecting within this new app.


in customHeadRender only 1 column needs to sort and not remaining elements

Expected Behavior

Current Behavior

Steps to Reproduce (for bugs)

Your Environment

Tech
Version

Material-UI

MUI-datatables

React

browser

etc

Suggest closing this issue.


Add ClusterQueue and Queue Metrics

What would you like to be added:
Metrics to track:

Queue/ClusterQueue pending workloads (gauge)
Queue/ClusterQueue admitted workloads (gauge)
Workload waiting time in the queue (histogram)
Admission cycle latency (histogram)

what else?

Oops, I forgot to put my name on this. I was already working on it. I hope you don't mind @kerthcet?

Plz go ahead.

I have nearly finished the Queue/ClusterQueue pending workloads (gauge), some tests left. if you haven't work on this one, I can finish this one to prevent redundant works.

/assign

Aldo, I guess the following represents progress on this issue, correct?

[x] Queue/ClusterQueue pending workloads (gauge)
[x] Admission cycle latency (histogram)
[ ] Queue/ClusterQueue admitted workloads (gauge)
[ ] Workload waiting time in the queue (histogram)

correct, thanks

@alculquicondor are you planning to add workload waiting time?


PR1 - Add output files containing Crawford's Rule proposal values from Grand Theory section 1.3.1.

Add both output-all.txt and output-unique.txt as reference points for all instances of spell damage (as calculated by my own spaghetti plate)

I'm not tarnishing this org with Java. I would never wish that fate upon anyone :'(

also, hi <3

Hiya :)
Hopefully the spaghetti won't matter as long as you can read it to change it if needed


[1.11] Fix handling of consumable item fluid containers.

Fixes #4539.
Also added some null checks and fixed some compiler warnings

@aidancbrady this currently breaks normal buckets!

Huh, I've tested it with buckets? :s weird

I know what went wrong with my testing. The Forge buckets where enabled. Only tested those not the default Vanilla buckets

ahhhh, that'd do it. I'm guessing you have tinkers installed?

Pfff, test World right from idea. *looks at folder*
Yep!


Double check priory dates in California's and Oregon's water rights data

I noticed that prior to "1900" priority date, California and Oregon mostly don't show water rights which is either not right, or very interesting. The same applies to North Dakota and the lower states down to Oklahoma. I know TX and NM data is incomplete in that regard. So it is worth checking and see if that is correct as we will be showing this at our Spring 2021 meeting.

This is a distribution of the number of records of priority data for CA, ND, NE, and KS.  Looking at the distribution, all four states have a low number of water rights available around that 1900 mark.
This feels like a source data issue.

That's a very interesting finding. It's probably not a problem but worth bringing up with those states when we talk with them. I assume this plot was done in Tableau. Let's work together to create such plots as examples for DPL prototype

You want DPL to work on this?
Is this something we should create on our own using tools like Tableau?  Or is this a feature we want DPL to create for the app?  If we ask DPL to create this for the app, that will increase the cost.

What I meant to say is that we create these example plots for DPL to see as they develop the full app later. We talked with them about this figure idea that they will develop later. This plot above shows them what we mean.

Growth of the # of water rights over time


Refactor ControllerInput to cleanup code.

Transform ControllerId (interface of constants) into a "static" class.
Fix a checkstyle issue.

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/399/
Hooray Jenkins reported success with all tests good!

Huh, today I learned that gets a Checkstyle warning :-)
Thanks!


B027 false positive with abc.abstractclassmethod

Adding an abstractmethod decorator does not make sense if an abstractclassmethod decorator is used instead:
When doing an automated fixes to the PyTorch codebase I ran into an issue in the wild:
class Foo(metaclass=abc.ABCMeta):
    @abc.abstractclassmethod
    def read(self):
        pass

should not be flagged for instance (but currently is).  This should be an easy exclusion to add to the rule and a good first issue.

Just gonna fix this real quick.


Fallback JSON parsing, handle missing HTML title tag.

If a JSON file has invalid formatting, try to recover gracefully.
If an HTML document is missing it's title tag, recover gracefully.

Sorry dude, I saw another error from prod... So I tacked it on here.

Sure, np. I do the same. =)

You just review too fast! Thanks.


List mined blocks and uncles in Address page

Address page does not list mined blocks and uncles. It only lists ordinary transactions.

We're in the process of developing a new orbiter, which would include the above info


Add support for Tls enabled postgresql connections

Setup
Versions

Rust: 1.66.0
Diesel: 2.0.2
Diesel_async: 0.2.0
Database: PostgreSQL
Operating System Linux

Feature Flags

diesel: postgres, time
diesel_async: postgres, bb8

Problem Description
When using Diesel's sync Pg implementation, I can connect via TLS, but when using the bb8 pooled connections via diesel-async I cannot.  I tracked this down to diesel-async always passing NoTls when creating a tokio-postgres connection.
What are you trying to accomplish?
I want to connect to a PostgreSQL database using TLS
What is the expected output?
I'd expect this to work
What is the actual output?
I cannot connect
Are you seeing any additional errors?
The error I see is that I have to connect over TLS
Steps to reproduce
Simply try to connect to a pgsql service which requires tls using a bb8 pooled diesel-async connection (frankly a non-pooled connection would exhibit the same issue)
If you don't have such a server, you can get a free account on bit.io
Checklist

[x] I have already looked over the issue tracker for similar possible closed issues.

[x] This issue can be reproduced on Rust's stable channel. (Your issue will be
closed if this is not the case)
[ ] This issue can be reproduced without requiring a third party crate

Notes
I'd hope for a postgres-openssl and postgres-nativetls pair of features for diesel-async which enable the relevant TLS implementation for the connections.  In the meantime I just have a hacky pile of code I'm not proud of:
Hacky tls connection
pub type Pool = bb8::Pool<AsyncPgConnection>;

fn establish_connection(url: &str) -> BoxFuture<ConnectionResult<AsyncPgConnection>> {
    (async {
        let builder = SslConnector::builder(SslMethod::tls())
            .map_err(|e| ConnectionError::BadConnection(e.to_string()))?;
        let connector = MakeTlsConnector::new(builder.build());
        let (client, connection) = tokio_postgres::connect(url, connector)
            .await
            .map_err(|e| ConnectionError::BadConnection(e.to_string()))?;
        tokio::spawn(async move {
            if let Err(e) = connection.await {
                eprintln!("connection error: {}", e);
            }
        });
        AsyncPgConnection::try_from(client).await
    })
    .boxed()
}

pub async fn create_pool(db_url: &str) -> Result<Pool, PoolError> {
    let config = AsyncDieselConnectionManager::<AsyncPgConnection>::new_with_setup(
        db_url,
        establish_connection,
    );
    bb8::Pool::builder().build(config).await
}

As already discussed in https://github.com/weiznich/diesel_async/pull/36 I'm not willing to add these additional dependencies behind feature flags as this makes it much harder to test the crate correctly. Your "hacky" solution is in fact the correct way to build a connection pool with TLS support.
I'm willing to accept a PR adding these information to the documentation.


Added go support and hello world example

About
Adding Go support and hello world example.
Run with bazel run //go
Progress https://github.com/EngFlow/example/issues/36

Can we make the example less trivial? Maybe add some dependencies and more than just a file?

Th first commits try to add support for the language. Maybe a test. This addresses this https://github.com/EngFlow/example/issues/36. I have  no experience with go, scala, swift,  so I am just adding a hello world example and continue from there as wee need to do some more work this repository https://github.com/EngFlow/example/issues/40.
Now, if you have a go example I can just copy and paste, then I change this PR, no problem whatsoever. Any example suggestion?

Probably out of scope for this PR but FYI: I think this will only work for remote execution if the execution platform is the same as the host platform. Our CI in this repo doesn't appear to cover remote execution.

Probably out of scope for this PR but FYI: I think this will only work for remote execution if the execution platform is the same as the host platform. Our CI in this repo doesn't appear to cover remote execution.

I think that's okay. These examples will be useful for running against a cluster with remote execution capabilities. I just ran it in our canary and it works. It is slow though.


Show pid on process list?

Thinking it would be nice if the pid was displayed next to the name of the process. I'm not what method is being used to generate the list, but @MrRio if that's something you're not opposed to I can send in a PR. Thoughts?

Seconding this.

Pid is essential to me. Vtop would be perfect with pid column.

+1


test file for autoencoder

Hi,
I would like to try autoencoder on my dataset. I could not find the test file for the autoencoder. Only provided test file is in shrec16. If possible, could you share the test file for autoencoder and part-seg?
Thanks in advance
edit: I saw the test part in the train file for the autoencoder, so I have to use the same for train and test right?

hey,have you solved the problem?How can I evaluate the autoencoder on my own data?thanks a lot!


Proposal: Embedding files as raw binary data

Background
For one of my own projects, I ran into the problem that the memory overhead for embedding some of the files involved was unacceptably high over the potential lifetime of the program.  This, combined with the discovery of the osext library for finding the path to the current executable prompted the development of the following scheme for embedding files, which I have working as far as I need for my own purposes at this point.
Relevant code can be found here for the writer and here for the reader
Currently, installing transcodebot and calling
transcodebot watch [binary_standin] [file_to_append]

Will pack $file_to_append onto $binary_standin and then unpack it to stdout.
High level description

Compile the binary normally
Write each file to embed to the end of the binary through a gzip.writer, noting the start and end point on the file.
Write the gathered start and end points, along with their names (paths) out to a json-encoded metadata block at the end of binary
Write the start position of said json data out as a magic number for the last 8 bytes of the file.

The self-extraction process looks like this, then:

Find the path to the source of the current process
Open the file
Read the last 8 bytes
Open a json.Decoder at the position written to the last 8 bytes
Decode the json into a struct in memory
Close the file

Then, when the process wants to access some of the data in the file, a file reader is opened on it that is simply started at $start_ptr and is limited to reading $size bytes
Visualization:
0                        400                       500              7000
+------------------------+-------------------------+----------------+---------------------+--------------+
| Executable Binary Data | Embeded File ./assets/A | Embeded Data B | Json metadata block | Magic Number |
+------------------------+-------------------------+----------------+---------------------+--------------+

In this case the json would look something like this:
{
  "Version": "0.1",
  "Data":
  {
    "./assets/A": { "start_ptr": 400, "size": 100 },
    "B":          { "start_ptr": 500, "size": 6500 }
  }
}

And the magic number would be 7000
Comparison with current system
Disadvantages:

Weird-looking executables
Relies on runtime time code to find static assets
May not work on a new operating system with a new binary format that go targets in the future

Advantages:

No additional files generated at compile time
Data storage format is self-describing, so it's possible for other programs to pull out the files/data stored in the binary
Process of appending data is self contained and doesn't need to interact with source code or the compiler
Smaller file sizes as there is no need to convert the binary data into a stringable format
Everything is stream based, so massive files (e.g. videos, executables) can be embedded at no memory cost for the compile process or compiled program
Can expose an api on uncompressed data that looks like an os.File, walks like an os.File, and quacks like an os.File
Fixes all of the problems with embedded static files mentioned on this page
Can fall back on compile-in method if need be

Known work to be done
I'll probably be doing these anyways:

Move the code into it's own repo with better examples
Clean up the names in the public api

Trivial and would be happy to add:

Uncompressed data api

Things I would need help with

Gluing packr high level api onto this
Writing new tests for packr where appropriate

Thanks for reading!

Alright, I have the library pulled out and have added a way of appending data without compressing it. Here is the library and here is a cli tool to play with the format. It should be noted that the library allows arbitrary mixing of compressed and uncompressed data but the cli tool only allows one or the other.

The new v2 API allows for implementing of a Store interface, so please PR up an implementation! :)

From "macOS Code Signing In Depth":

Signing a program will modify its main executable file. There are some situations where this will cause you trouble:

If your program has a self-verification mode that detects a change, your code may refuse to run.
Appending data to a Mach-O executable is expressly prohibited. Signature verifications on such files will fail.

@jpo-joyent Thanks for the heads up. Do you know if macOS is willing to run executables generated in this manner given enough coercion? I can't access a mac with admin rights or go installed to test myself. If it can run unsigned then that would fit my (and I think many others') use cases, despite being a bit of a nasty hack.
It should also be noted that executable reading data out of themselves like this may also cause 3rd party Windows antivirus to throw a fit, although I don't have concrete evidence of this actually happening.
All of this points to this being a "plaster warnings all over" store interface option (?), which I think would be the way to go.

This would have to be implemented as an optional store, not the normal default store. And yeah, sounds like warnings are definitely required for this proposal. Perhaps even a confirmation flag??
On Dec 13, 2018, 5:36 AM -0500, Patrick Nuckolls<EMAIL_ADDRESS>wrote:

@jpo-joyent Thanks for the heads up. Do you know if macOS is willing to run executables generated in this manner given enough coercion? I can't access a mac with admin rights or go installed to test myself. If it can run unsigned then that would fit my (and I think many others') use cases, despite being a bit of a nasty hack.
It should also be noted that executable reading data out of themselves like this may also cause 3rd party Windows antivirus to throw a fit, although I don't have concrete evidence of this actually happening.
All of this points to this being a "plaster warnings all over" store interface option (?), which I think would be the way to go.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Issue-560

@sonichi @thinkall
Why are these changes needed?
Add to FAQ - How to get each agent message?
Related issue number
Closes #560
Checks

[x] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[x] I've made sure all auto checks have passed.

Codecov Report

Merging #578 (615c081) into main (b41b366) will decrease coverage by 0.24%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #578      +/-   ##
==========================================
- Coverage   34.04%   33.81%   -0.24%
==========================================
  Files          25       25
  Lines        3005     3005
  Branches      668      668
==========================================
- Hits         1023     1016       -7
- Misses       1906     1912       +6
- Partials       76       77       +1

Flag
Coverage Δ

unittests
33.81% <ø> (-0.17%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes

@sonichi  PR updated


SASL authentication

To support people who deploy their apps on Heroku.

@trungpham we do not have this feature, however we are looking forward in a little more details on this requested feature. Might you please explain the importance of this?


Auto-fix doing nothing

I am using Barackets Release 1.7 build 1.7.0-16898 (release b0a363b71) with brackets-eslint 2.5.0 installed by url.
Auto-fix seems to have no affect.

@PeterVermont  if you use command line with --fix does it do something? Extension is just supposed to do that. It doesn't fix every problem.
http://eslint.org/docs/user-guide/command-line-interface

Getting this same issue. Doing --fix from the command line works, but using the both the right-click menu and the option in the edit menu seem not to. No errors in the console, either.


fix: Update node_iam_role_arns locals to check for Windows Managed Node Groups

Description

This change adds additional checks for platform == "windows" on both local.node_iam_role_arns_non_windows and local.node_iam_role_arns_windows.
The additional validation in local.node_iam_role_arns_windows ensures that the correct configurations are added to aws-auth.
The additiional validation in local.node_iam_role_arns_non_windows ensures that terraform does not also create an extra incorrect entry in aws-auth for the node group.
To support this change, the eks_managed_node_group module has a new output (platform)
Motivation and Context

Resolves the issue described in this comment from #2350
Breaking Changes

No breaking changes found during my testing.
How Has This Been Tested?

[] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

👍

can someone approve this :-)

bump

bump

I would also greatly appreciate a review of this. Thanks!

I've nearly submitted this exact same code in a different PR today.  Even with today's master, the changes are the same.  It would be great to get this reviewed and pulled in so managed windows node groups can be functional instances.

Can we please get this reviewed and merged? It's really needed!

how has this been tested and validated?

+1 this PR would benefit us greatly

how has this been tested and validated?

PR was tested using the eks_managed_node_group example module, with some modifications to use existing VPC related resources since those are managed elsewhere within my org. I do not have access to a sandbox account so that's the closest I can get.

This looks good in some limited testing, given the problems this is currently causing I think it should be pulled in.

+1 Can we please get this merged ?

Would be really nice to see this merged.

+1 Would be great to see this merged.

This PR is included in version 19.16.0 :tada:


wrong cci calculation

for cci calculation, we should use "mean absolute deviation (mad)" not standard deviation. please correct it.
unfortunately, pandas rolling doesn't provide mad function. we have to find it by our own.
here I'm posting my code which is used for cci calculation.
pp = (data['CLOSE'] + data['LOW'] + data['HIGH'] )/3
mad = lambda x : np.mean(np.abs(x-np.mean(x)))
data['CCI']= (pp-pp.rolling(period).mean())/(0.015 * pp.rolling(period).apply(mad,True))

Hi @angopinath ,
Thank you for your comment. Yes, I am wrong.
I have checked your mad function and I am not sure. Whey mean(mean())? Probably we should use mean(std()). What do you think?
mad = lambda x : np.mean(np.abs(x-np.mean(x)))
or
mad = lambda x : np.mean(np.abs(x-np.std(x)))

Any resolution to this?

Current:
pp = (high + low + close) / 3.0     cci = (pp - pp.rolling(n, min_periods=0).mean()) / (c * pp.rolling(n, min_periods=0).std())     if fillna:         cci = cci.replace([np.inf, -np.inf], np.nan).fillna(0)     return pd.Series(cci, name='cci')
Based on another library and wikipedia
I think cci = (pp - pp.rolling(n, min_periods=0).mean()) / (c * pp.rolling(n, min_periods=0).mean()) might be better.
On the contary investopedia agrees with the original implemenation.
I am not familiar with the practical differences between tho two. I'd suggest a (optional) boolean flag to be used, so the programmer decide, I think this is a good middle ground.
If you wish, I would be happy to make the change a create a pull (FYI it would be my first)

To match the formula on Investopedia, this is the correct code, so far as I can see.
pp = (data['CLOSE'] + data['LOW'] + data['HIGH'] )/3   (pp-pp.rolling(20).mean()) / (0.15 * abs(pp-pp.rolling(20).mean()).rolling(20).mean())
The denominator takes the absolute values of the last (period) differences and averages them, which is what the Investopedia article refers to as the "mean deviation".

I think this is the correct implementation.
    pp = (high + low + close) / 3.0
    mad = lambda x : np.mean(np.abs(x-np.mean(x)))
    cci = ((pp - pp.rolling(n, min_periods=0).mean())
           / (c * pp.rolling(n, min_periods=0).apply(mad, True)))

It is included in the new ta version 0.4.6.
Thank you to all of you.

Remember: Tradingview's default settings in CCI indicator is calculated on Close not hlc3.

So if you get different numbers... correct the settings on TV.


powershell equivalent of KList

What to know whether PS come with the cmdlet that equivalent of Klist.
KList: This is a great command line tool that lists Kerberos tickets as well as being able to purge Kerberos tickets. The nice thing about this tool is that you can selectively purge Kerberos tickets rather than deleting all tickets like the KerbTray utility does.

You can run klist from PowerShell. Why do you need cmdlrt?


[FEATURE REQUEST] Not Number Treshold in Month view

I'm trying to use this plugin for a habit tracker but for "Boolean Habits", habits that are done, or aren't. But I dont find a way to use it to make the day in the month view to be marked. Is there a way to do it?
if there aren't I would be grateful if you make this a feature.
(sorry for the bad english)

You can use tasks (e.g. - [ ] habit) in your daily notes and track them using searchType task.done.
Example trackers are here

Will close this issue, you can reopen it if you still get problems with it.


Infer rels based on heads and tails

This PR added inference on _relation_s between a known pair of head and tail. Can anybody review the change? Thanks.

looks fine, but only TransE has been changed accordingly.

Hi, @louisccc, is TransE not the only model class providing inference methods currently?

hi @baxtree yea.. for the unit tests we now only have tested TransE. I will be adding soon :)

Oh cool! That will be really useful across all models. 👍


[PECO-348] Enable direct results by default

Previously user had to explicitly pass maxRows to use direct results feature. Now if maxRows is omitted, it will default to 100000  (same value we use for fetchChunk/fetchAll defaults), and to disable direct results maxRows should be explicitly set to null (e.g. for use with runAsync)

Codecov Report
Base: 94.94% // Head: 94.94% // Increases project coverage by +0.00% :tada:

Coverage data is based on head (47981d4) compared to base (73874d4).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #70   +/-   ##
=======================================
  Coverage   94.94%   94.94%
=======================================
  Files          50       49    -1
  Lines         672      673    +1
  Branches       90       90
=======================================
+ Hits          638      639    +1
  Misses         17       17
  Partials       17       17

Impacted Files
Coverage Δ

lib/DBSQLOperation/index.ts
100.00% <100.00%> (ø)

lib/DBSQLSession.ts
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@moderakh I implemented this in a pretty straightforward manner, however, here are some thought on this. Since direct results is barely compatible with runAsync, we could use it to disable direct results. So if runAsync = false we could set default value for maxRows/use user-provided value, but if runAsync = true we could ignore maxRows and effectively disable direct results feature. Considering that runAsync defaults to false, it will mean that direct results will be enabled by default. This approach will make null-alternative not needed at all. WDYT?

@moderakh I implemented this in a pretty straightforward manner, however, here are some thought on this. Since direct results is barely compatible with runAsync, we could use it to disable direct results. So if runAsync = false we could set default value for maxRows/use user-provided value, but if runAsync = true we could ignore maxRows and effectively disable direct results feature. Considering that runAsync defaults to false, it will mean that direct results will be enabled by default. This approach will make null-alternative not needed at all. WDYT?

@kravets-levko we should be consistent in terms of behaviour across our DBSQL SDKs. I wonder if you checked other GA sdks. How are we doing this in JDBC/ODBC/PySQL?


Rename repo to UCAN Community Website

We should rename this repo to UCAN Community Website or ucan.xyz or similar

Yeah agreed, let's rename it. 💯
Maybe ucan-community-web? I think making the purpose of the site explicit in the name will help folks find it.

The repository has been renamed to ucan-community-web.


New version: Surfshark.Surfshark version <IP_ADDRESS>9

Created by 🥟 Dumplings in workflow run #6513.

Log
Updated: <IP_ADDRESS>9 → <IP_ADDRESS>9
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

I believe Policy 2.3 has been covered for this package.

https://github.com/microsoft/winget-pkgs/pull/162345#issue-comment-box


Add support for Vala

Hi, thanks for this cool tool you built!
It would be super nice if Vala was supported. I guess this shouldn't be hard to do as Vala have a tool called valadoc to generate documentations from files, it uses GIR to get and generate the docs.
Vala is a cool OOP language that compiles into C code via glib, then C code is compiled using any C compiler you want.
Thanks in advance for the future responses ;)

Sounds promising. Let's see if other Vala fans jump in! :smile:


Add gocyclo check to the CI

Make sure gocyclo is ran in the parser during the CI

It's not clear to me what this means.


RFC about (optionally) using bgpq3 to fetch data

I have noticed that certain as-set objects that include many ASNs and as-sets are quite slow to process or even yield memory error.
While running couple of simple tests with bgpq3, its performance is magnitudes faster.
What about providing a patch that detects if bgpq3 is present and using it instead of the built-in fetcher?

@bbonev you are correct, bgpq3 does process the information today a whole lot quicker, I think it may have to do with the way php handles streams of data, will look into this further.

Nope, I did some tests and the difference comes from parallel execution in bgpq3. Implementing parallelism is not impossible with php sockets, but I think that using bgpq3 as a back-end fetcher would be better.
I will prepare a PR as time permits


fix(firestore-bigquery-export): clean column names for generating schema views

fixes #247
Screenshot of successfully generating schema view without error

Screenshot of view in Big Query updated with column name successfully changed to colon_colon.

Codecov Report

Merging #304 into next will decrease coverage by 1.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             next     #304      +/-   ##
==========================================
- Coverage   40.09%   38.92%   -1.18%
==========================================
  Files          20       20
  Lines         631      650      +19
  Branches       83       87       +4
==========================================
  Hits          253      253
- Misses        376      395      +19
  Partials        2        2

Impacted Files
Coverage Δ

storage-resize-images/functions/src/index.ts
0.00% <0.00%> (ø)


Winget installs zoom's installs through .msi package

Hello,
Just a few minutes ago i installed zoom meetings (zoom.zoom) via WinGet , instead of installing through .exe , it installed through .msi , which increased the application size to 148MB , whereas , the .exe version from their website (zoom.us) is 9.6MB , NOTE: application size is measured in control panel/programs . I hope , .msi version would be replaced with .exe version of the corresponding application.
Thanks and regards,
Manas

Feel free to just edit the download link in manifest.

The full installer is used because it installs for all users, and has more metadata for winget to work with. The smaller installer still ends up being the same size, it just doesn't report all of the files to the add/remove programs table. It probably doesn't matter which one we ship (although we should also add the exe installer since it doesn't require admin rights IIRC).

Did a little digging, found only the x64 installer tho, but the links are https://cdn.zoom.us/prod/<IP_ADDRESS>1/x64/ZoomInstallerFull.exe (installs for user) and https://cdn.zoom.us/prod/<IP_ADDRESS>1/x64/ZoomInstaller.exe
Zoom claims to be 9.77 MB in Revo Uninstall but the actual folder is 143 MB

And yeah i agree we should add the exe for InstallerScope: User compatibility, however the exe version is 5.6.6 (961) which might cause issues?


#1045 : disable radio buttons and email field while app is generating

Fixes #1045 .
Changes :
1) Radio buttons and email input are disabled while generating the app.
2) Added spaces between four buttons
Opinions @Princu7  @aayusharora  @mariobehling ?

Codecov Report

Merging #1191 into development will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           development    #1191   +/-   ##
============================================
  Coverage        63.74%   63.74%
============================================
  Files                9        9
  Lines             1451     1451
  Branches           218      218
============================================
  Hits               925      925
  Misses             526      526

Continue to review full report at Codecov.

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

@sumedh123 Nice work. Looks good. Can you please also provide the screenshot with the updated view of the buttons with spaces between them. It is not visible in the uploaded gif. Thanks!!

@Princu7

@sumedh123 Thanks for uploading the image of the buttons. In my opinion, they look a bit too much far apart from each other. I kinda like the previous look of the buttons a bit more than this. But, the user experience is different for every person so I might be wrong. Let's take more inputs and then proceed with this look. Hope this is okay. Thanks.
@aayusharora Inputs, please?

@Princu7  Shall I get those breaking spaces removed? Now I am feeling that the previous view was better.

@sumedh123 Please make the space less.

@aayusharora @Princu7


media query "handheld" not found

handheld keyword, part of CSS media queries, is not found... why?

handheld seems to be deprecated.
https://www.w3.org/TR/mediaqueries-4/#valdef-media-handheld

There is quite clear stance about this at cssmediaqueries.com:

Most important; the handheld media type is not used by smartphones and tablets. Smartphones respond to screen. So in effect, the handheld media type is quite useless.
-- http://cssmediaqueries.com/what-are-css-media-queries.html

(Didn't know this myself either.)


undefined method parse_csv with example script

I've got smartermeter downloading my daily data now, so I tried the sample script from the readme.  I get this error:
./smarter.rb:8:in `<main>': undefined method `parse_csv' for SmarterMeter::Samples:Class (NoMethodError)

This is from the script pasted exactly from the readme.  Any suggestions for how I troubleshoot that?

Sorry about that. The Readme is out of date, that method should be
parse_espi. I'll update the docs when I get a chance.

I shot you a pull request with that fixed and fixed formatting for the pachube info.  Still unsure how to get pachube to work tho.

Since I merged your pull I am going to close this as well.


[BUG]: Elastic Search is not returning more than 10k records

What is the bug related to?
Registry Core
What went wrong?
Search request is not returning more than 10k records even if limit is set more than 10k and records are available.
What did you expect to see?
Search request should return all the records available with the given limit.
Additional Context
Elastic Search might have some threshold limit, which might be configurable
Relevant logs/output (if any)
No response

This is not a bug
Elastic Search has this limit of 10k max_result_window index by default. which is dynamic meaning that we need to update it for each entity.
We can follow these steps to allow indexing for more than 10k records:

The below is sample cURL for updating max result window in elastic search
curl --location --request PUT 'http://localhost:9200/:entityName/_settings?preserve_existing=false' \
    --header 'Content-Type: application/json' \
    --data '{
        "index.max_result_window": "50000"
    }'

where entityName can be schema, revokedCredential, student, teacher

Update max_result_window for all the entities including schema, revokedCredential.

Set environment variable search_limit for registry. i.e. 50000

Restart registry service


UIScreen.scale and output images

New Issue Checklist

[x] I have read and understood the CONTRIBUTING guide
[x] I have read the Documentation
[x] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
iOS

Platform Version
11.x/10.x

SDWebImage Version
4.1.2

Integration Method
cocoapods

Xcode Version
Xcode 9.3

Repro rate
100%

Repro with our demo prj
Probably

Demo project link
n/a

Issue Description and Steps
Good day! This may or may not be an issue depending on the design intent around images for this library. I've recently come across what appears to be an issue in SDWebImage where downloaded remote images have an incorrect UIImage.scale for their device scale if the URL doesn't contain any scale information (@3x, @2x, etc). This results in images with implicit sizes that are waay bigger than they should be, which confuses the implicit sizing of UIImageView, which ultimately affects auto-layout and layout in general.
Example
Given a UIImageView inside of an arbitrary layout with a given fixed width (375) and dynamic height, I would like to fetch a remote image and have the resulting image auto-sized with auto layout. I know the aspect ratio I would like for this image to have, let's say 3:2 for this example.
Given my URL parameters (https://placekitten.com/{x}/{y}), a width, aspect ratio, and screen scale (let's say iPhone Plus — 3x scale), I have enough information to construct a URL to fetch a remote image:
let aspect: CGFloat  = 3.0/2.0
let scale = UIScreen.main.scale
let kittenURL = URL(string: "https://placekitten.com/\(Int(imageView.bounds.width*scale))/\(Int(imageView.bounds.width/aspect * scale))")! // https://placekitten.com/1125/750

The resulting image that is returned using SDWebImage for this URL has an image.scale == 1.0 and image.size = {1125.0, 750.0}. This makes the resulting implicit content size of UIImageView to be {1125.0, 750.0} instead of the correct parameters of {375.0, 250.0} @ 3x scale. This affects not only the implicit content size, but the image content mode resizing parameters of UIImageView with an image it thinks is much larger than it actually is.
The only mentions of the scale API on UIScreen.mainScreen are located in the GIF construction in UIImage+GIF.m:42 and in SDWebImageCompat.m:48, which appears to use it exclusively to know if it needs to even search for the @-scale modifiers in the original URL. I can hack usage of the screenScale API into SDWebImageCompat.m:50 as the default scale and this works marvelously (for my specific application), but I doubt that is the correct solve for this issue.
Do you have any guidance on how to achieve the correct output that I am looking for? I searched for relevant issues and didn't find anything that sounded similar to this. If this does in fact sound like a bug internally, what can I do to help this get resolved?
Thank you in advance! 🍻

This feature is already added in #2266. Which allow you to provide a extra scale factor arg to specify the target image scale. This will effect all the coders to use that information as scale factor and we will not call create a extra UIImage instance by grabing the previous CGImage and the new scale factor. See more in that PR.
imageView.sd_setImage(with: url, placeholderImage: nil, options:[], context: [.imageScaleFactor: 3], progress: nil, completed: nil)

In 4.x, seems we do not have a plan to add this feature. However, current you have one solution:
You can try to use cache key filter, to remap your remote URL format (like you say https://placekitten.com/{x}/{y}), to a string key that contains the scale factor(like https://placekitten.com/{x}/{y}@3x.jpg). Then the build-in process in 4.x will treat this url as a scaled url and will use 3x for scale factor.
This sounds a little trick, but since current we do not have plan to add a entire context arg in 4.x version, there are no something other good way to solve this.

I implemented the above workaround and it does in fact work for the images in question. Since this method affects all image requests in my application, it will require a regression on images. I'm confident this is the better method for image fetching for my application, though.
Any timeline on 5.0 release date?

@larsacus 5.x seems take so long time. We have to wait for #2285 to finish.
About current workaround - a more tricky way in 4.x. Since the cacheKeyFilter accept a NSURL instance, which original from the View Category method, you can try to pass extra information on that NSURL instance, for example, using setResourceValue:forKey:error: or Objective-C associate object way...Then in your cacheKeyFilter block, check the asscoiate value about scale factor, and return a modified cache key with @2x. or @3x.


add missing arm64

Fixes #385

Thanks!

Hi Guys,
Thank you for the fix, Could you release new tag for it, please?


Conflicts with @types/eslint due to eslint-formatter-pretty dependency

Due to the usage of eslint-formatter-pretty in version 4 I get some conflicts with newer versions of eslint plugins. Here with npm ci
npm error code EUSAGE
npm error
npm error `npm ci` can only install packages when your package.json and package-lock.json or npm-shrinkwrap.json are in sync. Please update your lock file with `npm install` before continuing.
npm error
npm error Invalid: lock file's<EMAIL_ADDRESS>does not satisfy<EMAIL_ADDRESS>npm error Missing<EMAIL_ADDRESS>from lock file

In the end I would say, that eslint-formatter-pretty should not specify any types package as a prod dependency.
foxriver76@MBPvonMoritz2 ioBroker.js-controller % npm ls @types/eslint
npm error code ELSPROBLEMS
npm error invalid<EMAIL_ADDRESS>/Users/foxriver76/workspaces/ioBroker.js-controller/node_modules/@types/eslint
iobroker.js-controller-mono@ /Users/foxriver76/workspaces/ioBroker.js-controller
├─┬<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>invalid: ">=8.0.0" from node_modules/eslint-plugin-prettier
└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    └──<EMAIL_ADDRESS>deduped invalid: ">=8.0.0" from node_modules/eslint-plugin-prettier

In the end I would say, that eslint-formatter-pretty should not specify any types package as a prod dependency.

This would be wrong because it uses types from that package in prod.

In the end I would say, that eslint-formatter-pretty should not specify any types package as a prod dependency.

This would be wrong because it uses types from that package in prod.

Where do you find it using types in Prod and how would that look like tbh I cannot imagine as types can only be used on the type level?

However, if this is the case, then likely it should be followed the recommendations for such scenarios. Which from my point of view is specifying it as a peer dep and with only a lower bound on the version, like also done here https://github.com/prettier/eslint-plugin-prettier/blob/51324d9cdf8b883e2a392904d0f73da9eca6a808/package.json#L52
As you are deeper in the project please let me know what you think @sindresorhus


BLD: pin tzlocal back

Due to incompatible changes in v3, pin to older versions.

Did it cause a significant CI failure rate?

@tacaswell, I see now that the CI failures triggered you to create these PRs:

https://github.com/bluesky/suitcase-jsonl/pull/19
https://github.com/bluesky/suitcase-msgpack/pull/14

@MikeHart85, do you mind taking care of the releases for the above suitcase-* fixes?


Creating VirtualBox machine no longer works after rc4

Since grabbing the latest version (rc4), I now get this output when trying to create a machine ...

PS C:> docker-machine create default -d virtualbox
Creating CA: C:\Users\Dan.docker\machine\certs\ca.pem
Creating client certificate: C:\Users\Dan.docker\machine\certs\cert.pem
Running pre-create checks...
Creating machine...
Waiting for machine to be running, this may take a few minutes...
Machine is running, waiting for SSH to be available...
Detecting operating system of created instance...
Provisioning created instance...
Copying certs to the local machine directory...
Copying certs to the remote machine...
Setting Docker configuration on the remote daemon...
WARNING >>>
This machine has been allocated an IP address, but Docker Machine could not
reach it successfully.
SSH for the machine should still work, but connecting to exposed ports, such as
the Docker daemon port (usually :2376), may not work properly.
You may need to add the route manually, or use another related workaround.
This could be due to a VPN, proxy, or host file configuration issue.
You also might want to clear any VirtualBox host only interfaces you are not using.
To see how to connect Docker to this machine, run: C:\Program Files\Docker Toolbox\docker-machine.exe env default

(see the first line in the quote above for the command I'm using)
This is on Windows 10, and I've seen this happen on two machines - my work machine, and my personal laptop.
I've tried deleting my entire .docker folder and all VirtualBox machines, but it doesn't make a difference. I don't have any VirtualBox hostonly interfaces other than the default VirtualBox one.
No machines seem to have been created in VirtualBox after this command fails.

Although strangely, I've just tried reverting to rc2 (I couldn't use rc3 due to #2093), but it's still doing it. Strange that I've seen it on both my work machine (yesterday) and my personal laptop (today) after upgrading to rc4 though. Are there any caches stored anywhere other than the <user>\.docker folder?

I am facing similar problem on Windows 7 + VirtualBox 5.0.6.
The issue  is multiple Host-Only networks get created. and Multiple docker machines have same ip <IP_ADDRESS>
Only way, I am able to fix it by retstarting the machine to restart VirtualBox service. Also before restarting machine, delete newly created hostonly network.
I started using Docker machine recently so canot confirm the behaviour on older versions, but I found this issue on rc3 and rc4

I think this problem was faced with earlier builds too. https://github.com/docker/machine/issues/1996

This isn't the same issue as I'm seeing. I only have one Docker Machine, and there aren't any VirtualBox host only network interfaces created.

@kunalkushwaha @dracan Can you all please both make a gist.github.com of the output of running docker-machine --debug create -d virtualbox default?  It will help immensely with fixing this bug since we will be able to see the conditions in our parsing of the VirtualBox command output that cause Machine to go sideways.
If you can get it in today, even better as the release is planned for tomorrow :)

I am sorry as won't be able to provide log today. My dev machine not with
me today.
Regards,
Kunal Kushwaha

Sent from mobile device
On 03-Nov-2015 5:18 am, "Nathan LeClaire"<EMAIL_ADDRESS>wrote:

@kunalkushwaha https://github.com/kunalkushwaha @dracan
https://github.com/dracan Can you all please both make a gist.github.com
of the output of running docker-machine --debug create -d virtualbox
default? It will help immensely with fixing this bug since we will be
able to see the conditions in our parsing of the VirtualBox command output
that cause Machine to go sideways.
If you can get it in today, even better as the release is planned for
tomorrow :)
—
Reply to this email directly or view it on GitHub
https://github.com/docker/machine/issues/2136#issuecomment-153144933.

I got an error when this call is issued to VirtualBox:
(default) DBG | COMMAND: C:\Program Files\Oracle\VirtualBox\VBoxManage.exe modifyvm default --natpf1 delete ssh
(default) DBG | STDOUT:
(default) DBG | {
(default) DBG | }
(default) DBG | STDERR:
(default) DBG | {
(default) DBG | VBoxManage.exe: error: Code E_INVALIDARG (0x80070057) - One or more arguments are invalid (extended info not available)
(default) DBG | VBoxManage.exe: error: Context: "RemoveRedirect(Bstr(ValueUnion.psz).raw())" at line 1767 of file VBoxManageModifyVM.cpp
(default) DBG | }

Please find output of docker-machine --debug create -d virtualbox default
https://gist.github.com/kunalkushwaha/de636308da3767511f6f

@kunalkushwaha Please try clearing all host-only networks and attempting the 0.5.0 release when you get a chance.

It is still reproducible.
Steps to reproduce this error.

Clean all host only network - Keep only default one.
Put Windows 7 machine on sleep.
Activate Windows machine .
docker-machine --debug create -d virtualbox default

This creation process creates two more host-only networks in VirtualBox.

NOTE: If windows start with fresh boot, Until, windows dont go into sleep mode, this error does not occur.
https://gist.github.com/kunalkushwaha/95fdc4e9e6d5328bc8e1

Sorry for the huge delay in getting you the output of my command. Below is the debug output. This is with the very latest 1.5.0 version.
This was run when I had no machines, I've deleted my .docker directory, and I've cleared all host only networks in VirtualBox.
One point to note is that the hostonly interface that gets created in virtualbox is called 'VirtualBox Host-Only Ethernet Adapter #3'. I can't find a way of resetting this #3 postfix. I wonder if this is the issue? Is Docker Machine expecting the name without it? The output below suggests that's the case.
PS C:\Scripts\docker> docker-machine --debug create -d virtualbox default
Docker Machine Version: 0.5.0 (04cfa58)
Found binary path at C:\Program Files\Docker Toolbox\docker-machine-driver-virtualbox.exe
Launching plugin server for driver virtualbox
Plugin server listening at address <IP_ADDRESS>:61543
() Calling RpcServerDriver.GetVersion
Using API Version 1
() Calling RpcServerDriver.SetConfigRaw
() Calling RpcServerDriver.GetMachineName
(flag-lookup) Calling RpcServerDriver.GetCreateFlags
Making call to close connection to plugin binary
Making call to close driver server
(flag-lookup) Calling RpcServerDriver.Close
Successfully made call to close driver server
Found binary path at C:\Program Files\Docker Toolbox\docker-machine-driver-virtualbox.exe
Launching plugin server for driver virtualbox
Plugin server listening at address <IP_ADDRESS>:61547
() Calling RpcServerDriver.GetVersion
Using API Version 1
() Calling RpcServerDriver.SetConfigRaw
() Calling RpcServerDriver.GetMachineName
(default) Calling RpcServerDriver.GetMachineName
(default) Calling RpcServerDriver.DriverName
(default) Calling RpcServerDriver.GetCreateFlags
(default) Calling RpcServerDriver.SetConfigFromFlags
Creating CA: C:\Users\DanClarke\.docker\machine\certs\ca.pem
Creating client certificate: C:\Users\DanClarke\.docker\machine\certs\cert.pem
Running pre-create checks...
(default) Calling RpcServerDriver.PreCreateCheck
(default) DBG | COMMAND: C:\Program Files\Oracle\VirtualBox\VBoxManage.exe
(default) Calling RpcServerDriver.GetConfigRaw
(default) Calling RpcServerDriver.GetConfigRaw
Creating machine...
(default) Calling RpcServerDriver.Create
(default) OUT | Image cache does not exist, creating it at C:\Users\DanClarke\.docker\machine\cache...
(default) OUT | No default boot2docker iso found locally, downloading the latest release...
(default) OUT | Latest release for github.com/boot2docker/boot2docker is v1.9.0
(default) OUT |
(default) OUT | Downloading https://github.com/boot2docker/boot2docker/releases/download/v1.9.0/boot2docker.iso to C:\Users\DanClarke\.docker\machine\cache\boot2docker.iso...
(default) DBG | COMMAND: wmic cpu get VirtualizationFirmwareEnabled
(default) OUT | Creating VirtualBox VM...
(default) OUT | Creating SSH key...
(default) DBG | STDOUT:
(default) DBG | {
(default) DBG | VirtualizationFirmwareEnabled
(default) DBG | TRUE
(default) DBG |
(default) DBG | }
(default) DBG | Creating disk image...
(default) DBG | Creating 20000 MB hard disk image...
(default) DBG | COMMAND: C:\Program Files\Oracle\VirtualBox\VBoxManage.exe createvm --basefolder C:\Users\DanClarke\.docker\machine\machines\default --name default --register
(default) DBG | STDOUT:
(default) DBG | {
(default) DBG | Virtual machine 'default' is created and registered.
(default) DBG | UUID: 6a67d65b-6453-41ed-be19-90131c7a9d2b
(default) DBG | Settings file: 'C:\Users\DanClarke\.docker\machine\machines\default\default\default.vbox'
(default) DBG | }
(default) DBG | STDERR:
(default) DBG | {
(default) DBG | }
(default) DBG | VM CPUS: 1
(default) DBG | VM Memory: 1024
(default) DBG | COMMAND: C:\Program Files\Oracle\VirtualBox\VBoxManage.exe modifyvm default --firmware bios --bioslogofadein off --bioslogofadeout off --bioslogodisplaytime 0 --biosbootmenu disabled --ostype Linux26_64 --cpus 1 --memory 1024 --acpi on --ioapic on --rtcuseutc on --natdnshostresolver1 off --natdnsproxy1 off --cpuhotplug off --pae on --hpet on --hwvirtex on --nestedpaging on --largepages on --vtxvpid on --accelerate3d off --boot1 dvd
(default) DBG | STDOUT:
(default) DBG | {
(default) DBG | }
(default) DBG | STDERR:
(default) DBG | {
(default) DBG | }
(default) DBG | COMMAND: C:\Program Files\Oracle\VirtualBox\VBoxManage.exe modifyvm default --nic1 nat --nictype1 82540EM --cableconnected1 on
(default) DBG | STDOUT:
(default) DBG | {
(default) DBG | }
(default) DBG | STDERR:
(default) DBG | {
(default) DBG | }
(default) DBG | using <IP_ADDRESS> for dhcp address
(default) DBG | COMMAND: C:\Program Files\Oracle\VirtualBox\VBoxManage.exe list hostonlyifs
(default) DBG | STDOUT:
(default) DBG | {
(default) DBG | }
(default) DBG | STDERR:
(default) DBG | {
(default) DBG | }
(default) DBG | COMMAND: C:\Program Files\Oracle\VirtualBox\VBoxManage.exe hostonlyif create
(default) DBG | STDOUT:
(default) DBG | {
(default) DBG | Interface 'VirtualBox Host-Only Ethernet Adapter' was successfully created
(default) DBG | }
(default) DBG | STDERR:
(default) DBG | {
(default) DBG | 0%...10%...20%...30%...40%...50%...60%...70%...80%...90%...100%
(default) DBG | }
(default) DBG | COMMAND: C:\Program Files\Oracle\VirtualBox\VBoxManage.exe hostonlyif ipconfig VirtualBox Host-Only Ethernet Adapter --ip <IP_ADDRESS> --netmask <IP_ADDRESS>
(default) DBG | STDOUT:
(default) DBG | {
Error creating machine: Error in driver during machine creation: C:\Program Files\Oracle\VirtualBox\VBoxManage.exe hostonlyif ipconfig VirtualBox Host-Only Ethernet Adapter --ip <IP_ADDRESS> --netmask <IP_ADDRESS> failed:
VBoxManage.exe: error: The host network interface with the given name could not be found
VBoxManage.exe: error: Details: code E_INVALIDARG (0x80070057), component HostWrap, interface IHost, callee IUnknown
VBoxManage.exe: error: Context: "FindHostNetworkInterfaceByName(Bstr(pszName).raw(), hif.asOutParam())" at line 205 of file VBoxManageHostonly.cpp

@nathanleclaire Gist out debug output after upgrading to 0.5 and clearing all host only network interfaces
https://gist.github.com/rawkode/fecb665d3a2598fc3b09

Thanks for following up @dracan @rawkode will take a look.

Is there anything else I can provide that will help debug this? Or do you have an idea of what it might be and I could investigate further myself?

For me it helped to update to latest Virtualbox 5.0.10 on my machine and install a fresh copy of the docker toolbox...

It worked on my machine when I done this, but failed on two of my colleagues machines. Whatever the problem is, it appears to be rather fickle :8ball:

@rawkode If you could Gist the debug output in the failing cases it would be great.  We might not be able to fix it right away but we're always looking for extra information on host system state to add for our tests (which we are currently beefing up).

https://gist.github.com/gregory/3b3c4768df89adb3780d

Having this issue as well.

@gregory As mentioned in the error line https://gist.github.com/gregory/3b3c4768df89adb3780d#file-debug-docker-machine-L202, you have conflicting / invalid host only interfaces.  I'd recommend removing all of them that you can from the VirtualBox GUI -- (in menu bar: VirtualBox => Preferences => Network => Host-only networks).  In your case, looks like vboxnet3 and vboxnet5 are the troubled ones.

@caleb-allen Please remove any existing machines and post output of docker-machine --debug create default for us to take a look at.  Make sure to use Machine 0.5.2.

@nathanleclaire removing the invalid host only interfaces from VirtualBox did the trick.

Nice to hear @caleb-allen.  Maybe in the future we can do so automatically, but I think it's hard to do safely.

Closing this issue since OP (@dracan) has not checked in for a while.

I am facing a similar problem. issue #2497

Clearing all host-only networks and $HOME/.docker folder does the trick for me.

I feel rather silly now, but in-case anyone else is daft enough to not realise:
This stopped working for me because docker-machine creates VM's on the same subnet as our internal network (192.168.99) - --virtualbox-hostonly-cidr "<IP_ADDRESS>/24" helps :+1:

I had the same problem followed the suggestion above from @zhangcheng-aa, worked great.

Hit same problem recreating a machine with same name but more memory. Following advice from @zhangcheng-aa worked for me too.

@zhangcheng-aa fix did the trick.

Just adding to the mix here, having tried most things @rawkode's fix worked for me. Windows 10, recently everything broke, and this has got me up and running again.
Incidentally, contrary to past experience and lots of comments, having multiple host-only adaptors didn't cause an issue after having created the vm with the --virtualbox-hostonly-cidr "<IP_ADDRESS>/24" flag.
So the full command is: docker-machine create -d virtualbox --virtualbox-hostonly-cidr "<IP_ADDRESS>/24" default.

I get this problem on a regular basis. I delete the docker machine, recreate it, then a few days later the problem reappears.
Debug output is here

As said by Ajeet-Ganga if system is connected to any VPN, disconnect it
If you are using Docker Quickstart Terminal relauch it
If you are using Kitematic again use:Delete VM and retry setup
But that is not the root solution, need to check how it can be resolved infact when its connected to VPN...

Disconnecting from VPN solved this problem for me. Thanks for the suggestion, @Ajeet-Ganga!

Are you able to connect infact, when vpn is connected, if so please share fix details i will try with my system

I'm facing the same issue in Arch Linux with virtualbox 5.1.10 r112026. I share my situation and solution in case anyone still not getting it resolved.
First please check your hostonly network, if it does not have any ip address, then you met same problem as me.
$ ip addr
...
4: vboxnet0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc fq_codel state UP group default qlen 1000
    link/ether 0a:00:27:00:00:00 brd ff:ff:ff:ff:ff:ff

In my case, the hostonly network was created but there is no IP address assigned to it, which cause it can not route from host to guest vm. Here is my solution:
Add ip address manually to your hostonly network, assume you use default cidr:
$ sudo ip addr add <IP_ADDRESS>/24 dev vboxnet0
$ ip addr
4: vboxnet0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc fq_codel state UP group default qlen 1000
    link/ether 0a:00:27:00:00:00 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/24 scope global vboxnet0
       valid_lft forever preferred_lft forever
    inet6 fe80::800:27ff:fe00:0/64 scope link
       valid_lft forever preferred_lft forever

and the route will automatically be added:
$ ip route
default via xxx.xxx.xxx.xxx dev eno1 proto static metric 202
<IP_ADDRESS>/24 dev vboxnet0 proto kernel scope link src <IP_ADDRESS>

then docker-machine can connect to the guest vm. Hope it helps.

Faced the same scenario as @justlaputa (Arch Linux, virtualbox 5.1.22 r115126, docker-machine 0.11.0) however the scenario was a little different for me. Not only was the IP address not assigned, the vboxnet0 interface was DOWN as well. I fixed this by running sudo ip link set dev vboxnet0 up and then assigning an IP address.

For those with a firewall in place, maybe you'll have to open the port 2376:
sudo ufw allow from any to localhost port 2376 proto tcp
More on: http://www.configserverfirewall.com/ufw-ubuntu-firewall/ubuntu-firewall-open-port/

Hi,
I'm using the latest version of Docker Toolbox 18.03.0-ce and latest VB version on Windows 7. Up until yesterday when I connected to a remote  host using VPN I didn't have any problems but since then I've experienced the same errors. I've unistalled, deleted and installed again everything related to the toolbox and VB but nothing helps. I've also tried a number of different solutions found here and on: https://github.com/docker/toolbox/issues/537 but nothing helps. I went to the VB GUI and saw that for the machine running, the Adapter 2 was host-only. I changed that to NAT and then the output of docker version was:
Client:
Version:       18.03.0-ce
API version:   1.37
Go version:    go1.9.4
Git commit:    0520e24302
Built: Fri Mar 23 08:31:36 2018
OS/Arch:       windows/amd64
Experimental:  false
Orchestrator:  swarm
error during connect: Get https://<IP_ADDRESS>:2376/v1.37/version: dial tcp <IP_ADDRESS>:2376: connectex: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond.
which is better than before but still it doesn't work.... Please provide any info if you can...


Addon Name

Addon Manifest URL
https://2ecbbd610840-hi-movies.baby-beamup.club/manifest.json
Addon Description
No response
Language of Content
None
Choose Labels

[ ] anime
[ ] asian drama
[ ] bollywood
[ ] cartoons
[ ] debrid support
[ ] http streams
[ ] live tv
[ ] metadata
[ ] misc
[ ] movies
[ ] music
[ ] nsfw
[ ] podcasts
[ ] radios
[ ] subtitles
[ ] torrents
[ ] tv shows
[ ] video games

this is a duplicate