Updates for co-chair(s) to emeritus

This is a tracking issue to move @irvifa and @kbarnard10 to emeritus based on their own request. At this time there are no replacements but we have began announcing the transition and encouraging folks to step up. I plan on sending out an email to SIG Docs with a more direct call to action in the near future (/cc @parispittman).
Required changes
Co-chair transitions to emeritus require some updates.

Repo
file
PR

k/community
OWNERS_ALIASES
https://github.com/kubernetes/community/pull/5949

k/community
sig-docs/README.md
https://github.com/kubernetes/community/pull/5949

k/community
sig-docs/blog-subproject/README.md
https://github.com/kubernetes/community/pull/5949

k/community
sig-list.md
https://github.com/kubernetes/community/pull/5949

k/community
sigs.yaml
https://github.com/kubernetes/community/pull/5949

k/enhancements
OWNERS_ALIASES
https://github.com/kubernetes/enhancements/pull/2859

k/org
OWNERS_ALIASES
https://github.com/kubernetes/org/pull/2890

k/org
config/kubernetes/sig-docs/teams.yaml
https://github.com/kubernetes/org/pull/2890

k/org
config/kubernetes-sigs/sig-docs/teams.yaml
https://github.com/kubernetes/org/pull/2890

k/website
OWNERS
https://github.com/kubernetes/website/pull/29357

k/website
OWNERS_ALIASES
https://github.com/kubernetes/website/pull/29357

k/website
SECURITY_CONTACTS
https://github.com/kubernetes/website/pull/29357

k/k
OWNERS_ALIASES
https://github.com/kubernetes/kubernetes/pull/104355

k/test-infra
OWNERS_ALIASES
https://github.com/kubernetes/test-infra/pull/23240

k-sigs/reference-docs
OWNERS
https://github.com/kubernetes-sigs/reference-docs/pull/252

k-sigs/reference-docs
SECURITY_CONTACTS
https://github.com/kubernetes-sigs/reference-docs/pull/252

k/k8s.io
OWNERS_ALIASES
https://github.com/kubernetes/k8s.io/pull/2535

I would appreciate, but not require, a confirmation / "+1" from @irvifa and @kbarnard10 to demonstrate that I am not acting alone and this issue is opened in good faith for the sake of integrity & community.
Thank you
From @jimangel:
@kbarnard10 stepped into the co-chair role officially in early 2020 and is a big reason the blog is as successful as it is today. Thank you for all you work and effort in making SIG Docs a great place to be!
@irvifa became co-chair later in September of 2020 and brought an immense number of improvements to the localization efforts underway for SIG Docs, including a new monthly meeting dedicated to issues localizing documentation. Thank you!
/assign @kbhawkey @sftim @onlydole
/sig docs

Thanks for many valuable contributions @kbarnard10 and @irvifa
I see this as entirely in good faith.

🦸♀ @irvifa , @kbarnard10 🦸♀
🎉 Thank you for your many contributions to Sig Docs.

Hi all, thank you for your help and yes I’ll give my +1 for my part.

@jimangel What does it take to get started as a docs reviewer in English?

@geojaz! Nice to see you on the open source side of things! 😄
If you're specifically asking about becoming a reviewer, check out our roles and responsibilities section of the SIG Docs contributor guide.
I see you're already an org member, but that's traditionally the first recommendation. Details outlined here.
Other than that, it's a matter of learning how the k8s docs work, attending meetings, and providing additional reviews (even if not assigned on the PR). Generally building a track record of providing reliable docs content / reviews. If you're new to docs, you can observe how others review PRs too.
Two YouTube videos I'd recommend for new contributors to docs:

Writing for Developers: Take your Project Docs to the Next Level - Celeste Horgan, CNCF
Intro to Kubernetes Docs - Celeste Horgan, Linux Foundation; Brad Topol, IBM; Tim Bannister

If you're interested in contributing directly vs. reviewing, here is list of good first issues (all localizations, not just English).
Lastly, pop into the #SIG-Docs slack channel and say "hey!"" It's a great place to ask any questions if you're looking for something to work on. Cheers!

Just waiting on https://github.com/kubernetes/test-infra/pull/23240 I think.

Done
/close

Hmm seems prow missed the hook?
/close


Is there a future for atom !

After github was acquired, atom continued to update or was ready to merge with vscode ?

Duplicate of https://github.com/atom/atom/issues/17471.


Katib 0.10 Release Blog Post

I added Blog Post: "Kubeflow Katib: Scalable, Portable and Cloud Native System for AutoML".
Name is under discussion, if you have any other options please let me know.
/cc @jbottum @gaocegege @johnugeorge @castrojo @amsaha

@kramachandran @theadactyl Can you help with this Blog Post review, please ?

Hi! Thanks for tagging me. Can we get an LGTM from a subject matter expert aside from blog author? I think there are some copy edits needed here as well, which I'd love help with. Otherwise looks good to me -- happy to approve when those are taken care of.

Hi! Thanks for tagging me. Can we get an LGTM from a subject matter expert aside from blog author? I think there are some copy edits needed here as well, which I'd love help with. Otherwise looks good to me -- happy to approve when those are taken care of.

Thank you @theadactyl!
Please let me know which parts can be modified in this Blog from your point of view.

I removed WIP status.
/hold for the review.

Thank you so much for the review @theadactyl!
I addressed your comments.

@andreyvelich very nice effort.    I have 3 comments.
(this sentence might need some commas or broken in to two sentences) - Since Katib can execute various Kubernetes resources, users are able to run Experiments which can optimize not only ML models but also any software, code or program to make it more efficient with optimization algorithms provided by Katib.
(should "perform" be "provide a"?) We are continually working on the new Katib UI to perform better User Experience and native integration with Kubeflow central dashboard.
(this sentence probably needs to be broken into two or more sentences...and I struggle with best and worse in the same sentence) : Currently, Katib supports the Median Stopping Rule which stops a pending Trial X at step S if the Trial's best objective value by step S is worse than the median value of the running averages of all completed Trials' objectives reported up to step S.     Suggestion....   Currently, Katib supports the Median Stopping Rule.   The Medium Stopping rule stops if the Trial's best objective is lower than the median value of all Trials' objectives completed to that point.

Thank you for the comments @jbottum! What do you think ?

(this sentence might need some commas or broken in to two sentences) - Since Katib can execute various Kubernetes resources, users are able to run Experiments which can optimize not only ML models but also any software, code or program to make it more efficient with optimization algorithms provided by Katib.

Since Katib can execute various Kubernetes resources, users are able to run not only ML models optimization Experiments. They can also enhance any software, code or program to make it more efficient with optimization algorithms provided by Katib.

(should "perform" be "provide a"?) We are continually working on the new Katib UI to perform better User Experience and native integration with Kubeflow central dashboard.

Make sense, changed.

(this sentence probably needs to be broken into two or more sentences...and I struggle with best and worse in the same sentence) : Currently, Katib supports the Median Stopping Rule which stops a pending Trial X at step S if the Trial's best objective value by step S is worse than the median value of the running averages of all completed Trials' objectives reported up to step S. Suggestion.... Currently, Katib supports the Median Stopping Rule. The Medium Stopping rule stops if the Trial's best objective is lower than the median value of all Trials' objectives completed to that point.

Thanks! What do you think about this "Currently, Katib supports the Median Stopping Rule. The Medium Stopping rule stops a running Trial if the Trial's best objective value is lower than the median value of all succeeded Trials' objectives completed to that point."

LMK when all stakeholders are ready for merge.

/assign @gaocegege @johnugeorge @jbottum

/lgtm


Copy-ExcelWorkSheet makes the incorrect assumption about the layout of the data. Needs to copy all cells. it asks for append -Noheader

Copy-ExcelWorkSheet makes the incorrect assumption about the layout of the data. Needs to copy all cells. it asks for append -Noheader
Can you provide an example to use this

Please could you provide an example of what is happening, thank you

@dfinke  I am also looking at this. EEplus has "Add Worksheet using a pre-existing worksheet" (which doesn't work if there is conditional formatting !) but it also might be good to have Copy sheet X and then sheet Y  into sheet Z. (And don't create the header, or add autofilter, or add a title or anything else you might do when creating a new sheet) .
I wonder if we have a translation issue …
@berincdhas   when you say "it asks for" do you mean  "It prompts me to input"  or do you mean "it needs to have"

@jhoneill I'm watching the EPPlus work progress. Only concern is what will break when they are done porting it to dotnet. Not sure how long or how far they'll take it either.


fix issue with preferential gauges for unstake

Description
Allow users to unstake if a preferential gauge is not set. Removes restake pool page header if the pool is affected by a vulnerability.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Dependency changes
[ ] Code refactor / cleanup
[ ] Documentation or wording changes
[ ] Other

How should this be tested?
Test old bb-a-usd staking form shows correctly with a staked address
Visual context
Please provide any relevant visual context for UI changes or additions. This could be static screenshots or a loom screencast.
Checklist:

[ ] I have performed a self-review of my own code
[ ] I have requested at least 2 reviews (If the PR is significant enough, use best judgement here)
[ ] I have commented my code where relevant, particularly in hard-to-understand areas
[ ] If package-lock.json has changes, it was intentional.
[ ] The base of this PR is master if hotfix, develop if not

Converting this to a hotfix here #2420


regex version pinning is problematic

Consider a new project that relies on both regex and which:
% cargo new --lib simple
Created library `simple` package
% cd simple/tmp
% cargo add regex -q # this adds the most recent one, 1.10.2
% cargo add which --features=regex -q
error: failed to select a version for `regex`.
    ... required by package `which v5.0.0`
    ... which satisfies dependency `which = "^5.0.0"` of package `simple v0.1.0 (/tmp/simple)`
versions that meet the requirements `=1.9.6` are: 1.9.6

all possible versions conflict with previously selected packages.

  previously selected package `regex v1.10.2`
    ... which satisfies dependency `regex = "^1.10.2"` of package `simple v0.1.0 (/tmp/simple)`

failed to select a version for `regex` which could resolve this conflict

Cargo.toml pins regex version to =1.9.6 explaining this by MSRV 1.63, let's test that.
% cargo new --lib simple2 && cd simple2 && echo 1.63.0 > rust-toolchain
     Created library `simple2` package
% cargo add which -q
% cargo check
    Updating crates.io index
error: package `home v0.5.9` cannot be built because it requires rustc 1.70.0 or newer, while the currently active rustc version is 1.63.0

One way to work around this is to add a direct dependency on home:
% cargo add<EMAIL_ADDRESS>    Updating crates.io index
      Adding home =0.5.5 to dependencies.
% cargo check
    Updating crates.io index
   Compiling rustix v0.38.30
    Checking linux-raw-sys v0.4.13
    Checking bitflags v2.4.2
    Checking either v1.9.0
    Checking home v0.5.5
    Checking which v5.0.0
    Checking simple2 v0.1.0 (/tmp/simple2)
    Finished dev [unoptimized + debuginfo] target(s) in 2.22s

You can use similar approach to pin regex to the required version.
On the other hand to use which with more recent regex I have to either fork the whole crate or vendor it just to replace "= 1.9.6" with "1.9.6".
It would be great if you could release 5.0.1 or whatever version with regex unpinned and a short mention in the documentation explaining how exactly to compile it with 1.63.

@pacak I have released crate version 6.0.0, which bumps the MSRV to 1.70, unpins regex, and updates all of our dependencies.


Fix position bug (drag in scaled mode)

We have position bug when dragging GridItem in scaled mode.
CodeSandbox example
This is old bug, and its have been already fixed. But its looks like it have been fixed twice, first in the react-grid-layout and second in react-draggable (may be in react-scalable too). As a result, we again have a positioning error.

Fixed in 1157e13639f4d83ede6c8524c1b0ef1c80e5d6cb


'typeof TableSearch__SvelteComponent_' is not assignable of type 'Constructor'

Describe the bug
Not sure what is going on and I can't find any similar errors in recent history on Github or Discord. When I am trying to render a Svelte component for testing following @testing-library/svelte docs, I am getting this crazy Typescript error but the test seem to be functioning fine.
For some reason this test is only appearing after I run vitest and if I restart my TSServer and never run vitest they do no appear. Once I have run vitest these errors pop up suddenly.
It seems to want to be of type SvelteComponentDev but if I try to use as typeof SvelteComponentDev I get an error saying it's already typed. Can anyone advise me if this is a mistake on my part or a bug that has just sprung up?
No overload matches this call.
  Overload 1 of 2, '(component: Constructor<SvelteComponentDev>, componentOptions?: SvelteComponentOptions<SvelteComponentDev> | undefined, renderOptions?: Omit<...> | undefined): RenderResult<...>', gave the following error.
    Argument of type 'typeof TableSearch__SvelteComponent_' is not assignable to parameter of type 'Constructor<SvelteComponentDev>'.
      Type 'TableSearch__SvelteComponent_' is missing the following properties from type 'SvelteComponentDev': $set, $on, $destroy, $$prop_def, and 5 more.
  Overload 2 of 2, '(component: Constructor<SvelteComponentDev>, componentOptions?: SvelteComponentOptions<SvelteComponentDev> | undefined, renderOptions?: RenderOptions<...> | undefined): RenderResult<...>', gave the following error.
    Argument of type 'typeof TableSearch__SvelteComponent_' is not assignable to parameter of type 'Constructor<SvelteComponentDev>'.ts(2769)

Reproduction
import { render, fireEvent, cleanup } from '@testing-library/svelte';
import { describe, it, expect, afterEach, vi } from 'vitest';

import { DataHandler } from '@vincjo/datatables';
import TableSearch from './TableSearch.svelte';

// Mock DataHandler
class MockDataHandler extends DataHandler {
  search(query: string) {
    query;
  }
}

describe('TableSearch.svelte', () => {
  afterEach(() => {
    cleanup();
  });

  it('renders TableSearch component', () => {
    const { getByTestId } = render(TableSearch, { props: { handler: new MockDataHandler() } });
    expect(getByTestId('table-search-form')).toBeTruthy();
  });

  it('renders search input field with correct attributes', () => {
    const { getByTestId } = render(TableSearch, { props: { handler: new MockDataHandler() } });

    const input = getByTestId('table-search-input'); // Use getByTestId to find the input element
    expect(input).toBeTruthy();
    expect(input.className).toContain('transition-all');
  });

  it('calls handler search method when input is changed', async () => {
    const handler = new MockDataHandler();
    const searchMock = vi.spyOn(handler, 'search');
    const { getByTestId } = render(TableSearch, { props: { handler } });

    const input = getByTestId('table-search-input');
    await fireEvent.input(input, { target: { value: 'test query' } });

    expect(searchMock).toHaveBeenCalled();
    expect(searchMock).toHaveBeenCalledWith('test query');
  });
});

System Info
System:
    OS: Linux 5.15 Ubuntu 22.04.2 LTS 22.04.2 LTS (Jammy Jellyfish)
    CPU: (12) x64 11th Gen Intel(R) Core(TM) i5-11600K @ 3.90GHz
    Memory: 219.06 MB / 1.86 GB
    Container: Yes
    Shell: 5.8.1 - /usr/bin/zsh
  Binaries:
    Node: 18.16.0 - ~/.nvm/versions/node/v18.16.0/bin/node
    npm: 9.6.4 - ~/.nvm/versions/node/v18.16.0/bin/npm
  npmPackages:
    @sveltejs/adapter-auto: ^2.0.1 => 2.0.1
    @sveltejs/adapter-node: ^1.2.4 => 1.2.4
    @sveltejs/kit: ^1.16.3 => 1.16.3
    @sveltejs/vite-plugin-svelte: ^2.2.0 => 2.2.0
    svelte: ^3.59.1 => 3.59.1
    vite: ^4.3.5 => 4.3.5

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

This is not related to Vitest in the slightest. Open an issue in @testing-library/svelte repo.

@sheremet-va Thank you, I understand.


Extract Only Sent Emails From Takeout?

Just to clarify, we only extract the sent emails from google takeout?
That should include sent & reply emails with their threads & original messages.
Or do we need to extract all email data?
Thanks again !

All email. Currently it uses sent as the reply and the received as prompt.


24 feat request prime admin

ZH CARD 24

postal code verification didn't seem to be working: https://www.loom.com/share/27c8bea6b39148919971302674756ef2 (verification could be tech debt unless it's a quick fix)

I believe we need to add a manual validator for the CAPostalCodeField as it only has validation on the Django form level and not the db level.

how do I test this part of the AC? I was able to submit multiple requests for both mock companies and never hit a NO page (I think because you don't get the NO until a prime admin is approved, not just submitted). Could we have a mock prime user with 'approved' status to test this?

Given I am an external user logged in to my Business BCeID account
When my company already exists in the Registration database and a prime admin has already been assigned (approved) to my company
Then I cannot submit a request to be prime admin for my company (subsequent ticket will address what page the user should be presented with instead)

I'm going to create a new userOperator fixture to handle this.

I wasn't able to submit the form when I added a parent company. I had some sort of validation error but I couldn't figure out what it was: https://www.loom.com/share/23cf50fd938a41b0a85ce9c103b62f9e

@BCerki I couldn't determine which field caused the error. However, I believe the issue is resolved as I utilized your province's constant.


x/tools/gopls: implement callHierarchy

I'd love to see gopls implement the new LSP Call Hierarchy feature.  I'd look forward to using it in VSCode which already supports it.
This was discussed as a "Preview Feature" in the VSCode release notes from March:
https://code.visualstudio.com/updates/v1_33#_preview-features
At the time, this LSP API was still in preview, but with the upcoming 3.16 LSP specification, I believe that this is no longer the case:
https://microsoft.github.io/language-server-protocol/specifications/specification-3-16/
As you can see from the screenshot included in the VSCode release notes, VSCode provides a nice ui when the language server implements this:

I see that others have requested this feature on the vscode-go repo, but I don't see a ticket here:
https://github.com/microsoft/vscode-go/issues/2682
I share the sentiment of the other user: this would be a great addition and would be very useful for me in my day to day work.

I wanted to follow up on this ticket.  I think that this type of interface would be a big asset when trying to quickly understand a piece of code

Agreed! We intend to support all LSP features in the future, but this isn't as high priority as some other bug fixes / features that we're currently working on.

Got it.  Sounds good

Closed by commit b793a13


Empty formData attaching multipartfile

Issue Info
Always empty formData when am attaching a multipartfile
| Dio Version                    | 3.0.10
Issue Description and Steps
`FormData formData = FormData.fromMap({
    "name": "wendux",
    "photo": await MultipartFile.fromFile(
      file.path,
      filename: fileName,
    ),
  });
  print(formData.fields);`

It printing [MapEntry(name: wendux)]
'photo' formdata is never added. Couldn't find the solution. Need urgent help/suggestions.

Having the same issue, How did you solved it?

Having the same issue, How did you solved it?

Its little bit old issue. Can you share your code and issue?


the daterangepicker and the text input are not update while updating model manually

Hello every body.
Thank you for your plugin, it's very nice. But i think there is an issue. When i update the model manually, the text input and the daterangepicker is not updated. Can you help me to fix? Many thank

+1

@vincez i'm using this one
https://github.com/fragaria/angular-daterangepicker
if you can not wait to fix, you can use it

@trangchuvn , thanks so much for the quick reply! Are there any other issues with fragaria? Is it better than luisfarzati?

yes. it's good. i'm using it for my application. this issue is not appear on fragaria

@trangchuvn , one more question. With the fragaria library, I cannot seem to get "options" to work at all - the library seems to just ignore my options. Have you had any issues with options in fragaria? Can you show me how you initialize one of your pickers?

index.html

     <input date-range-picker id="daterange3" name="daterange3" class="form-control" type="text" ng-model="data.dateRange" options="data.opts" readonly="true"/>

Controller.js
$scope.data = {
 opts:{
        opens:"right",
        format:"DD MMM, YYYY",
        ranges: {
					'Today': [moment(), moment()],
					'Yesterday': [moment().subtract(1, 'days'), moment().subtract(1, 'days')],
					'Last 7 days': [moment().subtract(7, 'days'), moment()],
					'Last 30 days': [moment().subtract(30, 'days'), moment()],
					'This month': [moment().startOf('month'), moment().endOf('month')]}
    },
    dateRange:{ startDate: moment().startOf('month'), endDate: moment().endOf('month') },
}

Basically when daterangepickers 'show:' function triggers, it should redraw calendar in case input value has been changed or not (which can happen, for ex: when we get input after rest service response, it takes time you know). As a hotfix, I've injected value recheck code into show: function:
// marek_helmes 16/03/2016
// Fix bug when date wasn't updating with angular trigger
var dateString = $(this.element).val(), start = null, end = null;
    if(dateString.length) {
        var dateSplit = dateString.split(this.separator);
        if (dateSplit.length === 2) {
            start = moment(dateSplit[0], this.format);
            end = moment(dateSplit[1], this.format);
        }
        if (this.singleDatePicker || start === null || end === null) {
            start = moment(dateString, this.format);
            end = start;
        }
    this.startDate = start;
    this.endDate = end;
    this.setCustomDates(start, end);
}
// fix end

It should be inserted after:
show: function (e) {
    if (this.isShowing) {
        return;
    }

Hope it helps.


os/exec: Cannot execute command with space in the name on Windows, when there are parameters

Go 1.7.1 on windows-amd64, Windows 10 latest.
Consider a test project:
src/github.com/calmh/wincmdtest/
    main.go
    folder name/
        test.bat

main.go contents:
package main

import (
    "fmt"
    "os/exec"
    "strings"
)

func main() {
    execCmd("./folder name/test.bat")
    execCmd("./folder name/test.bat", "one param")
}

func execCmd(path string, args ...string) {
    fmt.Printf("Running: %q %q\n", path, strings.Join(args, " "))
    cmd := exec.Command(path, args...)
    bs, err := cmd.CombinedOutput()
    fmt.Printf("Output: %s", bs)
    fmt.Printf("Error: %v\n\n", err)
}

folder name/test.bat contents:
@echo off
echo Success

Expected output is two runs with "Success" in them.
Actual:
C:\Users\jb\Go\src\github.com\calmh\wincmdtest>go run main.go
Running: "./folder name/test.bat" ""
Output: Success
Error: <nil>

Running: "./folder name/test.bat" "one param"
Output: '.' is not recognized as an internal or external command,
operable program or batch file.
Error: exit status 1

It appears that having params on a command, where the command contains a space, breaks the parsing of it. I haven't been able to work around this by experimenting with various ways of quoting the command, using backslashes or slashes, etc.

I started following this down through syscall.StartProcess, but not really being a Windows programmer I ran into unacceptable levels of nope almost immediately so probably won't try to fix this myself, sorry. :(

It appears that having params on a command, where the command contains a space, breaks the parsing of it.

Go encodes child process parameters in a way that is understood by most programs. Go uses rules similar to what CommandLineToArgvW implements.
Unfortunately, your child process is cmd.exe (cmd.exe is called to execute the batch file you've requested). And cmd.exe parses its input parameters differently.
You can read this very long post http://daviddeley.com/autohotkey/parameters/parameters.htm#WIN about it all.

I haven't been able to work around this by experimenting with various ways of quoting the command, using backslashes or slashes, etc.

You should stop using exec.Command to build your child process command line, and build it yourself (as per rules described in the doco I mentioned). Then you can pass raw command line to your child process by setting exec.Cmd.SysProcAttr to syscall.SysProcAttr with CmdLine set appropriately.
Maybe we could fix this problem by changing Go encoding algorithm to use cmd.exe rules every time we execute batch file.
There is more of the same discussion on issue #15566.
Alex

I guess this should be fixed by small change.
diff --git a/src/syscall/exec_windows.go b/src/syscall/exec_windows.go
index cafce1e..a0e1f56 100644
--- a/src/syscall/exec_windows.go
+++ b/src/syscall/exec_windows.go
@@ -91,6 +91,9 @@ func makeCmdLine(args []string) string {
 		}
 		s += EscapeArg(v)
 	}
+	if s != "" && s[0] == '"' && len(args) > 1 {
+		s = `"` + s + `"`
+	}
 	return s
 }

If the first argument contain spaces, and it have arguments (no matter if the argument not contains spaces), whole of string should be quoted.
c:\dev\go-sandbox\space>go run main.go
Running: ".\\folder name\\test.bat" ""
Output: Success
Error: <nil>

Running: ".\\folder name\\test.bat" "one param"
Output: Success
Error: <nil>

Postponing decisions about this to Go 1.9.

+1 This is still an issue here, also see: https://github.com/docker/machine/issues/3152

@cannect docker/machine#3152 title mentions "powershell", but this issue is about running batch files. How do you know that if we make running batch file work, that it will fix docker/machine#3152 ?
Is there a simple way to reproduce docker/machine#3152 ?
Alex

As you can see here: https://github.com/docker/machine/issues/3152#issuecomment-261811531 there is some confidence we think it is related with this issue.

@cannect do you know of a simple way to reproduce docker/machine#3152 ?
How did you personally discovered docker/machine#3152 ? Did you encountered that problem yourself?
Alex

Hi @alexbrainman ,
I had exact the same issues as described in the first post of docker/machine#3152.
I do not have the knowledge to reproduce that issue. But I can tell you my setup:

WIndows 10 Pro
Docker for Windows
Started two docker machines in Windows 10 bash, listed as active in docker-machine ls
Looked those docker machines up with: docker-machine ls in Powershell, machines not active. (but they were active of course but not listed as such in Powershell)

Unfortunately I have no experience whatsoever with Go.

@cannect thank you very much for these steps. But I have never used Docker for Windows. How do I install it and use it?

Docker for Windows

How do I install it? What are the steps?

Looked those docker machines up with: docker-machine ls in Powershell, machines not active. (but they were active of course but not listed as such in Powershell)

What are the commands I should run to reproduce this? What did you do? What did you expect to see? What did you see instead?
Thank you
Alex

I agree with https://github.com/golang/go/issues/17149#issuecomment-257518448 that the answer here should be to rewrite the path for argv[0] from using slash to backslash. We should try to fix this early in the Go 1.12 cycle.

Any progress on this issue?
I'd like to add that it only seems to be a problem when both the command and the parameter contain spaces.  If only one or the other has a space then everything works.  I've expanded on @calmh's example to demonstrate:
main.go:
package main

import (
    "fmt"
    "os/exec"
    "strings"
)

func main() {
    execCmd("C:/Users/blye/Go/src/github.com/benlye/cmdtest/folder name/test.bat")
    execCmd("C:/Users/blye/Go/src/github.com/benlye/cmdtest/folder name/test.bat", "oneparam")
    execCmd("C:/Users/blye/Go/src/github.com/benlye/cmdtest/folder name/test.bat", "one param")

    execCmd("C:/Users/blye/Go/src/github.com/benlye/cmdtest/foldername/test.bat")
    execCmd("C:/Users/blye/Go/src/github.com/benlye/cmdtest/foldername/test.bat", "oneparam")
    execCmd("C:/Users/blye/Go/src/github.com/benlye/cmdtest/foldername/test.bat", "one param")
    }

func execCmd(path string, args ...string) {
    fmt.Printf("Running: %q %q\n", path, strings.Join(args, " "))
    cmd := exec.Command(path, args...)
    bs, err := cmd.CombinedOutput()
    fmt.Printf("Output: %s", bs)
    fmt.Printf("Error: %v\n\n", err)
}

Produces only one failure (the 3rd test):
C:\Users\blye\Go\src\github.com\benlye\cmdtest>go run main.go
Running: "C:/Users/blye/Go/src/github.com/benlye/cmdtest/folder name/test.bat" ""
Output: Success
Error: <nil>

Running: "C:/Users/blye/Go/src/github.com/benlye/cmdtest/folder name/test.bat" "oneparam"
Output: Success
Error: <nil>

Running: "C:/Users/blye/Go/src/github.com/benlye/cmdtest/folder name/test.bat" "one param"
Output: 'C:/Users/blye/Go/src/github.com/benlye/cmdtest/folder' is not recognized as an internal or external command,
operable program or batch file.
Error: exit status 1

Running: "C:/Users/blye/Go/src/github.com/benlye/cmdtest/foldername/test.bat" ""
Output: Success
Error: <nil>

Running: "C:/Users/blye/Go/src/github.com/benlye/cmdtest/foldername/test.bat" "oneparam"
Output: Success
Error: <nil>

Running: "C:/Users/blye/Go/src/github.com/benlye/cmdtest/foldername/test.bat" "one param"
Output: Success
Error: <nil>

Any progress on this issue?

I don't believe anyone working on this.

I'd like to add that it only seems to be a problem when both the command and the parameter contain spaces. ...

I am pretty sure we know what the problem is (see https://github.com/golang/go/issues/17149#issuecomment-248822918 for details). We just have not decided how to fix it.
Alex

For those stuck by this bug:
As https://github.com/golang/go/issues/17149#issuecomment-473976818 mentioned, this problem happened only when both binary path and arguments have space.

Why not fixed?

Wow. What a bug. XD

It took me a while to figure out what was wrong, I had to use Process Monitor in order to see what Go was passing as arguments to the process, any news on a fix?

any news on a fix?

I am not working on this issue, if that is what you are asking.
Alex

For anyone needing a more concrete example of how to do the manual path construction, it works something like this:
cmd := exec.Command(os.Getenv("SystemRoot")+`\System32\cmd.exe`)
args :=  []string{`/c`, `""C:\Program Files\echo.bat"`, `"hello friend""`}
cmd.SysProcAttr = &syscall.SysProcAttr{CmdLine: " " + strings.Join(args, " ")}
out, _ := cmd.CombinedOutput()
fmt.Printf("Output: %s", out)

Note that you're on the hook to have everything on the CmdLine escaped/quoted/etc properly, which can be a real chore.

Here's an updated version of @ItsMattL's workaround:
// command to execute, may contain quotes, backslashes and spaces
var commandLine = `"C:\Program Files\echo.bat" "hello friend"`

var comSpec = os.Getenv("COMSPEC")
if comSpec == "" {
	comSpec = os.Getenv("SystemRoot") + "\\System32\\cmd.exe"
}
childProcess = exec.Command(comSpec)
childProcess.SysProcAttr = &syscall.SysProcAttr{CmdLine: "/C \"" + commandLine + "\""}

// Then execute and read the output
out, _ := childProcess.CombinedOutput()
fmt.Printf("Output: %s", out)

For Linux system:
`input = strings.ReplaceAll(path, " ", "\ ")
		args := []string{
			"ffmpeg",
			fmt.Sprintf("-i %s", input),
			fmt.Sprintf("-vf %s", "scale=-1:360"),
			fmt.Sprintf("-c:v %s", "libx264"),
			fmt.Sprintf("-crf %s", "18"),
			fmt.Sprintf("-c:a %s", "copy"),
			"b.mp4",
			"-y",
		}
		myCmd := strings.Join(args, " ")
		cmd := exec.Command("bash","-c", myCmd)
`

Here's an updated version of @ItsMattL's workaround:
// command to execute, may contain quotes, backslashes and spaces
var commandLine = `"C:\Program Files\echo.bat" "hello friend"`

var comSpec = os.Getenv("COMSPEC")
if comSpec == "" {
	comSpec = os.Getenv("SystemRoot") + "\\System32\\cmd.exe"
}
childProcess = exec.Command(comSpec)
childProcess.SysProcAttr = &syscall.SysProcAttr{CmdLine: "/C \"" + commandLine + "\""}

// Then execute and read the output
out, _ := childProcess.CombinedOutput()
fmt.Printf("Output: %s", out)

At this risk of sounding pedantic 😅, should the executable path be included in the CmdLine, too? Not sure if it makes a difference in practice, but reasons I ask:

The docs for the underlying Windows API mentions it.
If you use exec.Command without the SysProcAttr escape hatch, it automatically prepends the executable to the args (see https://github.com/golang/go/blob/master/src/os/exec/exec.go#L379). Those args ultimate are used to generate the cmdLine (see https://github.com/golang/go/blob/master/src/syscall/exec_windows.go#L302)
If you run the batch file from Windows Run and watch the procmon output, it includes the exe in the command line.

If you run the example code and watch the procmon output, it doesn't include the exe in the command line.

@rafd123 I think you are right! So, the proper code becomes:
// command to execute, may contain quotes, backslashes and spaces
var commandLine = `"C:\Program Files\echo.bat" "hello friend"`

var comSpec = os.Getenv("COMSPEC")
if comSpec == "" {
	comSpec = os.Getenv("SystemRoot") + "\\System32\\cmd.exe"
}
childProcess = exec.Command(comSpec)
childProcess.SysProcAttr = &syscall.SysProcAttr{CmdLine: comSpec + " /C \"" + commandLine + "\""}

// Then execute and read the output
out, _ := childProcess.CombinedOutput()
fmt.Printf("Output: %s", out)

What do you think about something like this as a general solution?
Caveats:

syscall.EscapeArg probably needs to be replaced with something that is aware of cmd.exe's escaping rules in order to be airtight; see this.
I can imagine adding handling for the problems inherent with running msiexec, too.

exec.go
package exec

import (
	"os/exec"
)

// MakeCmd returns the Cmd struct to execute the named program with
// the given arguments.
//
// It handles the case where the program is a Windows batch file
// and the implication it has on argument quoting.
func MakeCmd(name string, args ...string) (*exec.Cmd, error) {
	return makeCmd(name, args...)
}

exec_windows.go
package exec

import (
	"fmt"
	"os"
	"os/exec"
	"path/filepath"
	"strings"
	"syscall"
)

func makeCmd(name string, args ...string) (*exec.Cmd, error) {
	if len(args) == 0 {
		return exec.Command(name), nil
	}

	name, err := exec.LookPath(name)
	if err != nil {
		return nil, err
	}

	if !isBatchFile(name) {
		return exec.Command(name, args...), nil
	}

	argsEscaped := make([]string, len(args)+1)
	argsEscaped[0] = syscall.EscapeArg(name)
	for i, a := range args {
		argsEscaped[i+1] = syscall.EscapeArg(a)
	}

	shell := os.Getenv("COMSPEC")
	if shell == "" {
		shell = "cmd.exe" // Will be expanded by exec.LookPath in exec.Command
	}

	cmd := exec.Command(shell)
	cmd.Args = nil
	cmd.SysProcAttr = &syscall.SysProcAttr{
		CmdLine: fmt.Sprintf(`%s /c "%s"`, syscall.EscapeArg(cmd.Path), strings.Join(argsEscaped, " ")),
	}

	return cmd, nil
}

func isBatchFile(path string) bool {
	ext := filepath.Ext(path)
	return strings.EqualFold(ext, ".bat") || strings.EqualFold(ext, ".cmd")
}

exec_other.go
//go:build !windows

package exec

func makeCmd(name string, args ...string) (*exec.Cmd, error) {
	return exec.Command(name, args...), nil
}

@rafd123 Looks like a good improvement. However I don't see why we would need to handle batch files differently. It's important to pass them through CMD.EXE too, so they need the same escaping.
And yeah, you would need to follow the excellent article that you linked to, and escape meta characters with ^. Otherwise, there is a serious security issue where the one could inject commands by using "&malicious.exe in any of the arguments.
We need a Windows-specific escaping mechanism.

Digging into this a bit further, I noticed that the dotnet core equivalent of exec.Command does not suffer from this problem. Also note that dotnet is cross-platform these days and has a similar abstraction over OS-specific APIs for creating processes.
Specifically, this works fine in dotnet:
using System.Diagnostics;

Process.Start(@"C:\Program Files\echo.bat", new[] {"hello world"});

Debugging how both golang and dotnet ultimately call Windows' CreateProcess API, I noticed one meaningful difference: dotnet passes null for the optional lpApplicationName parameter of CreateProcess while golang does not.
This is how dotnet calls CreateProcess: https://github.com/dotnet/runtime/blob/2d411c4dfc1d71b2387ac64089014ec811ad7af0/src/libraries/System.Diagnostics.Process/src/System/Diagnostics/Process.Windows.cs#L578
This is how golang calls CreateProcess (note the use of argv0p...which in practice maps to the name argument of exec.Command...also note that in practice argvp will also contain the name argument of exec.Command): https://github.com/golang/go/blob/261fe25c83a94fc3defe064baed3944cd3d16959/src/syscall/exec_windows.go#L416
Empirically, I can also confirm that if I force golang to pass nil for lpApplicationName in the code snippet above, executing exec.Command(`C:\Program Files\echo.bat`, "hello world") works without resorting to use the SysProcAttr escape hatch.
All that said:

Specifying lpApplicationName for CreateProcess is cited as a security best-practice as a fail-safe should the caller fail to add quotes around the path of the executable in lpCommandLine. FWIW, dotnet ensures that the executable is quoted for this very reason, and it turns out golang does, too, via makeCmdLine's use of appendEscapeArg.
I can't find any official documentation around this subtle behavior difference with CreateProcess as to why it works when NOT specifying lpApplicationName, so at this point it's all circumstantial. All I was able to find is someone else pointing out this suble behavioral difference way back in 2001. 😅
It's entirely possible I'm glossing over some other important detail. I'm stabbing in the dark at this point. 😅

@rafd123 Following what is done in .NET Core, and adapting it to the expected behavior in Go is probably the best we can do. Thank you for the excellent research!
In your suggested makeCmd(), you would just need to add proper escaping for CMD.EXE. We probably need to use ^ instead of " to surround the arguments. WDYT?

(CC @golang/windows)

It's still exist in golang 1.20, e.g:
cmd.exe /c copy/b "input 1.ts"+"input 2.ts" ouput.ts
I guess that golang can automatically add double quotes to paths that contain Spaces, but many commands have their own coding rules, such as copy/b. The "+" in copy/b means concatenating the input files, but golang cannot parse it and cannot add double quotes to paths of input files that contain Spaces correctly.

@bcmills @golang/windows I'd like to submit a PR that modifies syscall/exec to address this issue, and I'd like to get buyoff on the approach before doing so.
TL:DR modify syscall/exec on Windows to pass nil for the lpApplicationName arg to CreateProcess() and CreateProcessAsUser() to mimic what dotnet core does since dotnet doesn't exhibit this issue due to this difference. For full background, see this comment: https://github.com/golang/go/issues/17149#issuecomment-1426878988
Concretely, this is the code change I'd like to propose: https://github.com/rafd123/go/commit/6edfd65a2a2ac24e8546ad285821e9e3e4345d1a#diff-ec673c10a75fe2d2faa9c788e03823294b263c68cc3de50f4a0865a53e28f3a3
My biggest concern with this change is that it assumes that syscall/exec.StartProcess's argv0 is represented within argv[0]. This assumption could be violated if, for example, folks don't use os/exec.Command and instead manually construct os/exec.Cmd without including os/exec.Cmd.Path in os/exec.Cmd.Args. I already ran into a violation of this assumption in one of the tests here: https://github.com/rafd123/go/commit/6edfd65a2a2ac24e8546ad285821e9e3e4345d1a#diff-f0ee8d33c527f62f2e48195ea748d2d7295c26910144012ee6e108fbf08f21a6
That said, there seems to be a precendent for this assumption in syscall/exec_unix.go here: https://github.com/golang/go/blob/e8c8b79f000515e086012df632f01fc0ec21076b/src/syscall/exec_unix.go#L169-L171
What do you think? Is this a reasonable approach? If so, I'll submit a PR.

@rafd123, on Unix it is possible for an exec.Cmd to have an Args[0] different from its Path. I would expect the same to be possible in general on Windows too, which would seem to imply calling CreateProcessW with an ApplicationName that matches Path and a CommandLine that matches Args.
But maybe we could leave ApplicationName nil when Path is already equal to Args[0], as would be the case if the user already called exec.LookPath to resolve the path passed to exec.Command?
Or, since this quoting seems to be mostly specific to cmd.exe and commands that implicitly reinvoke %COMSPEC% (specifically, commands with the extensions .bat and .cmd), maybe we should change the quoting algorithm to match that used by cmd.exe when the resolved Path has a .bat or .cmd extension? (Are there cases where that would regress existing programs, or do we expect that those are basically all using the SysProcAttr.CmdLine workaround already?)

I've been struggling with this bug and have recreated the proposed fixes in this conversation in my app.  The latest proposal works well for the use case in the OP, but I think it should be noted that when using cmd.exe /c cmd metachars also need to be handled (not just spaces and quotes).  These need to be further escaped with ^, though for reasons I can't yet explain that needs to itself be escaped as ^^^ in the final CmdLine string used in SysProcAttrs.

Based on the fuzz test added in https://go.dev/cl/530275, I believe that one part of the fix for this will be to change syscall.makeCmdLine to use the same approach as was taken for windows.ComposeCommandLine in that change.
Another part of the fix for this issue will probably be to add a fuzz test to os/exec, similar to windows.FuzzComposeCommandLine but using os/exec to start a Go program that prints its arguments (compare cmd/go/internal/work.FuzzSplitPkgConfigOutput). That will ensure that os.commandLineToArgv remains consistent with syscall.makeCmdLine.
Finally, we should either change os.commandLineToArgv to use CommandLineToArgvW directly, or add a fuzz test to ensure that its implementation is consistent with some other Windows function or program (perhaps CommandLineToArgvW, cmd.exe, or some other well-defined reference program).

I'm coming here directly from arduino IDE discussion. Yes, that IDE still doesn't work for people that have 2-part username with space in Windows, as path contain a space and we get an error while copying.
I find it hard to believe that a bug of this size persist for 8 years. I previously lost a couple of hours on debugging and fixing scripts, as it was still faster than recreating the whole environment elsewhere. Do you think it will ever be fixed?

@qmuntal Want to take a look at this?  Thanks.

I see two alternative solutions here:

Leave lpApplicationName nil when Path is already equal to Args[0]. Not that we can't unconditionally set lpApplicationName and prepend Path to Args because that will break applications that don't expect the application name to be in Args[0]. This fix works for the common case of initializing the command using exec.Command, as it always prepend the Path to Args. On the other hand, it won't work if os.ProcAttr.Dir is set and Path is relative, as in this we would have to modify Args[0] to make it absolute, which will break applications expecting Args[0] to be the application base name. I've prototyped this solution in CL 620915.

Detect if Path points to a .bat or .cmd file, and if so, adapt the argument quoting and escaping rules. This has the benefit of potentially fixing other issues, for example #15566, but doing it right is more complex than the previous solution. Also, the @golang/security has mentioned in the past that they would rather not open this can of worms.

I personally prefer the second option, as it is more complete, but I'm also good with the first one.

Detect if Path points to a .bat or .cmd file, and if so, adapt the argument quoting and escaping rules. This has the benefit of potentially fixing other issues, for example syscall: exec_windows.go: arguments should not be escaped to work with msiexec #15566,

I also prefer that option.
But what are the .bat or .cmd files argument quoting and escaping rules? Are they documented somewhere?
I have a feeling that msiexec argument quoting and escaping rules are different again. Are these documented somewhere?
Alex

From the security perspective, the main pain point here is that the rules are not immediately obvious, so implementing multiple escaping schemes seems likely to result in hard to parse semantics, both for us and for the user (although, arguably, this is already true for the latter), which tends to be a recipe for security issues down the road.
I don't think we would have a problem with the second solution given two things:

Detection of bat/cmd is reliable. Does Windows enforce the property that a filename ending in .bat/.cmd means its a batch file? Is there anything else we'd need to do to make sure we aren't using the wrong escaping scheme?
The escaping rules we implement are documented (at least... somewhat) by Microsoft. We currently implement a vague approximation of how CommandLineToArgvW works, but if I'm remembering correctly some of that is based on guesswork. Ideally we would be able to say for executables we using X escaping algorithm, and for batch files we use Y algorithm (and, possibly, for misexec we use Z algorithm).

The bat/cmd quoting rules are well defined in here: https://learn.microsoft.com/en-us/windows-server/administration/windows-commands/cmd.
I've been investigating this issue a little bit more and I no longer think that we should jump to supporting bat/cmd scripts directly. I've submitted #69939 with an middle-wround alternative.


Added tibobeijen.nl to site showcase

Added my personal programming blog to the site showcase. Thumbnail is 600x400.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Where is the frontend?

People have been adding to this repo for months now, but it seems that there is still no Frontend to render this content. Any updates on the status of this? Can the community help? Could this be a community effort?

Thank you very much for your interest. At this point in time, we are hard at work on creating the best design possible. As it stands right now, all I can say is that we intend to go live in the near future and we will of course inform the community as soon as that happens.

@pReya If you haven't seen it already on our social media channels or our forum, I'm happy to tell you there is a front end now: community.netcup.com


Spacemacs Keybinding Migration Guide

So I just migrated from Spacemacs over to Doom and... I'm really enjoying the experience so far. Everything seems so much more tightly integrated and fluid and stable and fast that I am just enamored so far.
However, I cam across this blurb in the README:
Have you migrated from Spacemacs? Help me flesh out this section by letting me know what kind of hurdles you faced in doing so. You’ll find me on our Discord server.

I didn't know what channel to use in the Discord for this type of feedback so I thought I would make it an issue.
The biggest problem that I'm having right now with the transition from Spacemacs to Doom is adjusting to the key bindings. As most people probably know, the key bindings to Spacemacs are mnemonic - which really helps with the whole "memorizing a billion keybindings" thing.
Now that I'm transitioning, I'm having a difficult time learning the key combinations and not instinctively using the old key combos. I find myself trying SPC p t to bring up treemacs and SPC g s to bring up a Magit status buffer. Oh, and SPC c l to comment out lines - I've been trying to use the comment line keybind (C-x C-;) and keep hitting (C-x C-l) instead lol.
My suggestion is to have migration documentation that helps with learning new keybindings. Perhaps a cheat sheet that details common actions taken and translates them to Doom keybindings? I've also been thinking about writing a private module that just replaces all the Doom keybindings with the ones from Spacemacs. Your thoughts?

(also I can help with this if it becomes a markdown page)

I didn't know what channel to use in the Discord for this type of feedback so I thought I would make it an issue.

The #development channel is best suited for these kinds of discussions.

My suggestion is to have migration documentation that helps with learning new keybindings. Perhaps a cheat sheet that details common actions taken and translates them to Doom keybindings?

I'm all for it. Trouble is: I'm clueless on the matter -- I've never used Spacemacs at length. It'd be a great help if you could flesh out the Spacemacs migration stub in our Getting Started page. Or at least include a link there to a spacemacs cheatsheet from there.

I've also been thinking about writing a private module that just replaces all the Doom keybindings with the ones from Spacemacs. Your thoughts?

Some ex-Spacemacs users have likely done this already -- at least for their favorite keybinds. Is there a spacemacs cheatsheet somewhere I could refer to?
I've considered writing a Spacemacs bindings module, but I don't want the responsibility of maintaining a module I have no insight on. Still, I will eventually add a community-contributed module repo (a MELPA for Doom modules, essentially), and such a module would be an excellent addition to it.

Hi all, I love doom and also spacemacs, so I took sometime to work on the spacemacs module. Please check:
https://github.com/chenyanming/spacemacs_module_for_doom
The current status:

Default spacemacs leader keybindings.
All org layer keybindings.
Github layer keybindings.
PDF layer keybindings.
epub layer keybindings.
dash layer keybindings.

I was looking for an alternative to spacemacs since it has become too slow for my use case. Found doom-emacs yesterday and found this issue listed here.
Today I installed doom-emacs. I can see how it is a lot faster than spacemacs, thank you and keep up the good work @hlissner, your optimisations are phenomenal.
I agree with @howdoicomputer regarding the ease to remember keybindings from spacemacs. One of the features or spacemacs is Mnemonic key bindings, with feels very natural. I've pressed SPC p t for treemacs and SPC p l for layout so many times already for no use.
I've tried out @chenyanming 's spacemacs module setup. It works out of the box for many things. Perhaps we can work in bringing some of it natively to doom?
Another thing I noticed is Spacemacs it has great documentation. They've built a static website from the org-mode file documentation. It's quite nice to have it. Is this something we can do for doom?
Overall what I'd like to say here, is doom-emacs looks very promising and is fast. We could make it a bit more better with mnemonic key bindings and better documentation.

Slightly off-topic: people who have used both Spacemacs and Doom, do you recommend not switching the default keybindings to Spacemacs-like? Is there some long-term reason for why I should try to learn doom keybindings rather than changing them to spacemacs defaults?

I just came across this discussion. I've been a Spacemacs user for a few years, and have been recently bouncing back and forth between it and Doom, in trying to decide where I want to land. There's a lot about Doom Emacs to like.
I have definitely found the keybindings to be a sticking point in transitioning, especially since I'm a non-evil user. Similar to Doom, Spacemacs has well thought out keybindings for evil users, initiated by SPC as a leader key. A critical difference for me, though, is when using non-evil standard emacs bindings. In Spacemacs, nearly all of the fabulous Spacemacs menus/keys easily come along for the ride if you just change the leader config. For me, I've set my leader to be M-\ (since it's an uncommon chord and it's about as comfortable as alt-tab), and then all of the key menus work great.
Doom, on the other hand, has an amazing set of keybindings for evil. I found them so amazing and convenient that I even spent a week moving over to using evil just so I could use Doom as it was intended. I may still land there, but I honestly prefer to use non-modal editing and would prefer not to go to the dark side. :) So I tried the Spacemacs trick of going non-evil and setting my own leader, which Doom lets you do, but I found it to be quite messy. A small subset of the Doom leader-based keybindings came along for the ride, since the Doom non-evil system is largely (but not entirely) to use the original standard keybindings. As best as I understand it, in non-evil mode for Doom ctrl-c serves two purposes; it is the default leader key as a replacement for SPC, but it also used in its usual role with vanilla emacs bindings. When I reconfigured the leader to M-, a few keys moved, but a whole lot of them stayed with ctrl-c. I found configuring my own leader not to be very useful, since it left a whole lot behind with ctrl-c. I realized I was happier just leaving ctrl-c fixed as the leader rather than configuring my own as I do in Spacemacs, but I found myself back in the land of using ctrl-key combos all over the place, which is what I gleefully left when I went to Spacemacs years ago.
To recap, one of my favorite parts of Spacemacs is the fantastic menu system that I can invoke with a leader I can configure myself on non-evil mode. It would be awesome if the Doom evil system could similarly be transformed. It's not that I would need something that mimics Spacemacs in the actual keys. Mimicing the Doom system would be fine. The key thing I'd love to see would be a non-chord-based approach to the menus (apart from perhaps the usable configured leader key). That's the #1 thing possibly holding me back at Spacemacs at this point. Spacemacs gets me the best of both worlds, from my own perspective, on keys; I get emacs-style non-evil editing, and a very convenient menu system that's not based around lots of ctrl-key combinations. Currently in Doom, it seems I have to choose one or the other.
Thanks again for all of the hard work!


How to i18n ?  Multi-language

Have you guys tried to do ?

I've successfully added react-intl and react-intl-redux with the babel-plugin-react-intl to handle multilanguage. Have a look at those!

I am use https://medium.com/@ryandrewjohnson/adding-multi-language-support-to-your-react-redux-app-cf6e64250050 👍


MixItUp MultiFilter premium extension not available on my account after purchase

Hello,
I can't find the multifilter extension script file under my account. I don't see any download link to get the mixitup-multifilter.min.js or any scripts.
Pleas provide support on this topic, I need to move forward asap.
Thanks

Responded via email.


Refactor buffer cleaning in InternalStreamConnection.sendAndReceive

This change was extracted from https://github.com/mongodb/mongo-java-driver/pull/883.

Somehow it confuses me. Maybe because I've never written code like this that takes advantage of Closeable idempotency so explicitly.

Actually, you did here the same exact thing :ъ


concurrent odometer

Closes #742

Current dependencies on/for this PR:

main

PR #648

PR #618

PR #674

PR #675

PR #676

PR #678

PR #682

PR #743   👈
PR #684

This comment was auto-generated by Graphite.

Closing, as concurrency will be handled within sequential odometers instead of being a separate constructor..


Interlinking of visuals in a dashbaord

Hi,
I am new to superset. My requirement is to create dashboards having interlinked visuals. If I click on any part, a bar or a Slice of pie, other visuals should be updated. I am not getting how to do that. I read somewhere that we will have to make code changes but I am not getting proper way to do that.

Hi, this is feature is on our roadmap, to receive status update or contribute to the project, please join our slack channel - https://apache-superset.slack.com/archives/C0188FT3V2S #dashboar-filters.


ContextMenu

[x] I have followed (at least) the PR section of the contributing guide.

Let's stick to https://github.com/mui-org/material-ui/pull/17839.


[env] Support JNI of FlinkEnv

What is the purpose of the change
This pull request introduces the JNI interfaces for FlinkEnv
Brief change log

introduces JNI interfaces of FlinkEnv

Verifying this change
New tests will be introduces later once the FlinkFileSystem interface is fully implemented.

@masteryhx The CI build failed because the method "FlinkFileSystem::Create" has not been implemented. This problem should  be resolved after the "pr#13 ([env] Implement all methods of env_flink )" is merged.

@masteryhx I have rebased the code with ""pr#13 ", and all CI checks have passed. Could you please take a look at this pr?


Add german translation

Added a translation for german commands

Ough, you can't use spaces in method names :)
Please pick a syntax which fits better to German language:

'mache einen Post Request' => macheEinenPostRequest
'mache einen Post Request' => mache_einen_post_request

Both are valid. But no spaces pls!

I would prefer the 1 style for js, 2 style is for ruby or python.

I am doing style 1 since all the other translation files are also style 1


[FFM-10859]: Create 'HashedRepo' for fetching Features & Segments

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 3 committers have signed the CLA.:white_check_mark: jcox250:x: akiraqb:x: rssnyderYou have signed the CLA already but the status is still pending? Let us recheck it.


Fix #466: Type of setState argument should be Partial

Coverage remained the same at 97.513% when pulling 386bb3a4f55bb77b0fe2c07193ec48063f41f67b on mseddon:setstate-typing into 75acb15ff7f9439ed89b8e1fd82d39961fdfa84d on developit:master.

/cc @kruczy @jrf0110 who know a bunch about tsd

Coverage remained the same at 97.522% when pulling 9fbcdb1bb32239f529888f39aa16ae6fee3cedf6 on mseddon:setstate-typing into 074aa7a3d44745b7e846a6e1e34a782d65261bf2 on developit:master.

Changes Unknown when pulling c08c7e2cac374f359151fc84a2c9bfafede6f9ce on mseddon:setstate-typing into ** on developit:master**.

I've just done some testing with TypeScript 2.0 and it looks like it is quite easy for users of earlier versions of TypeScript to add a "polyfill" for Partial<T> which would allow them to use these latest definitions:
type Partial<T> = T;

With this, any reference to Partial<T> is treated as if it were just T. On that basis, one option is that we could make this change but perhaps add a comment in the type definitions for the benefit of anyone using earlier TS versions, eg:
// Note: Partial<T> is a TS 2.1 feature. See <link to this PR>
setState(state: Partial<State>)

Thoughts?

@robertknight The problem with that "polyfill" is that it's not. Nothing would be optional. Instead aliasing as any would be closer even :(
I think saying that you need TS 2.1+ is fine. TS 2.0 to 2.1 does not introduce breaking changes, even we upgraded from 1.3 to 2.1 and it was fine. Everything was stashed away with flags anyway.

@robertknight The problem with that "polyfill" is that it's not.
Nothing would be optional. Instead aliasing as any would be closer even :(

That's why I put "polyfill" in quotes. It would then work the same as it does now using the current setState typings, so there would be no regression. Once users update to TS 2.1 they will be able to take advantage of the proper feature.
So to update the status here, the current recommendation from the TypeScript team based on the discussion for the React typings is that there should be a grace period after a new TS release before releasing typings updates that depend on the new release.
I would suggest that we track https://github.com/DefinitelyTyped/DefinitelyTyped/pull/13155 and once that is merged then we can do the same here. That way Preact and React typings will be in sync.

Then yup. I completely agree.
On Fri, 13 Jan 2017, 20:15 Robert Knight<EMAIL_ADDRESS>wrote:

@robertknight https://github.com/robertknight The problem with that
"polyfill" is that it's not.
Nothing would be optional. Instead aliasing as any would be closer even :(
That's why I put "polyfill" in quotes. It would then work the same as it
does now using the current setState typings, so there would be no
regression. Once users update to TS 2.1 they will be able to take advantage
of the proper feature.
So to update the status here, the current recommendation from the
TypeScript team based on the discussion for the React typings is that there
should be a grace period after a new TS release before releasing typings
updates that depend on the new release.
I would suggest that we track DefinitelyTyped/DefinitelyTyped#13155
https://github.com/DefinitelyTyped/DefinitelyTyped/pull/13155 and once
that is merged then we can do the same here. That way Preact and React
typings will be in sync.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/developit/preact/pull/467#issuecomment-272522334, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AB35WDbkQONjrDf7rGAmk8nQEvTp_Dntks5rR807gaJpZM4LS_xP
.

The corresponding PR for React has been merged upstream. setState in the React definitions uses Pick rather than Partial: https://github.com/DefinitelyTyped/DefinitelyTyped/blob/99d41f818c2de87790ce1f02956cbc39bce40f40/react/index.d.ts#L169

Coverage remained the same at 97.695% when pulling d9a0755a847325a0ce618c5b04b2099939acc256 on mseddon:setstate-typing into 472cb2b2abd4a229e87f7ea25c775002a73eb431 on developit:master.

@robertknight yes. The dropped "optional key"  support in state and chose pick.
https://github.com/DefinitelyTyped/DefinitelyTyped/pull/13155 based of https://github.com/Microsoft/TypeScript/issues/12793#issuecomment-266055843
Preact could follow this.

Updated to use Pick, and added the other signature.

Coverage remained the same at 97.695% when pulling e063e2bda7bc35131d56d9034ab57aab8a6ea997 on mseddon:setstate-typing into 472cb2b2abd4a229e87f7ea25c775002a73eb431 on developit:master.

Coverage remained the same at 97.695% when pulling a7d85f4cd009fa0ebdd2c77a1bdbcfd843c20f1b on mseddon:setstate-typing into 76fcbeee04ec89cfa267888dc3e5e7e7bc937859 on developit:master.

🎉

Awesome, so happy to have this merged :D


Issue 267

I'll let @sophieeng review. :)

@sophieeng test.

I changed the protect and analysis icons to mesh better with the other icons we have present. (This may not matter because we are removing the button anyway).

I think this looks good!


You cannot delete extensions that have already been cancelled

Description:
If the license for an extension has already been canceled, this will be indicated. You can only click on "Cancel and delete" in the extensions. However, this fails because the license has already been canceled. It is, therefore, not possible to remove the extension via the admin.
Expected result:
You can delete the extension.
Actual result:
You can't delete the extension.

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-33104.
[created from NEXT-33104, comment 486118]

[public] Hello,
Within our company roadmap and work capacity, we try to address each bug or improvement request but admit that not each one will be resolved. To continue our culture of honesty and openness, we are closing this issue to focus on our roadmap on behalf of all Shopware users.
If you feel like this issue is still impacting you, please create a new issue and let us know. Thank you again for your valuable feedback.
[created from NEXT-33104, comment 501290]


refactor: reduce usage of json.Marshal and consider alternatives

We marshal to/from a bunch of types during the lifecycle of a step run. We should cache the raw JSON bytes and converted objects, wherever possible, and consider an alternative like go-json to speed up decoding.

Hey @abelanger5 if this is open can i take this one up?


Add interface for test purposes to Quix/KafkaStreamingClient

Is your feature request related to a problem? Please describe.
As a software developer I wish to have tests exercising the code where I use either of these clients.
Describe the solution you'd like
Add an interface to both. Check if a common interface is possible, but it is not mandatory.
Describe alternatives you've considered
Use of https://github.com/quixio/quix-streams/blob/6fa5e331be801171d773b835375779e62fcaa280/src/CsharpClient/QuixStreams.Streaming.UnitTests/Helpers/TestStreamingClient.cs#L8 is possible for some testing, but not always the best.
Additional context
Might need to treat this interface with mostly testing goal, rather than an unbreakable contract. Not introducing breaking change however is still vastly preferred.

TestStreamingClient should implement both, so can use it in place of either.


error format verb %w is not recognized

https://github.com/dunstontc/vscode-go-syntax/blob/ea4e765872e35be141d8e71f38b24be91a0c4000/syntaxes/go.tmLanguage.json#L58C9-L65C11
fmt.Errorf("some error information: %w", err)

The %w is not recognized as %v is, for example.
append w.

created pull request with fix instead


Process finished with exit code -1073740940 (0xC0000374)

It will appear when running for a long time, about 30 minutes, the action is to repeat the initialization frequently, save the picture and close
Process finished with exit code -1073740940 (0xC0000374)

same on my side, did you reslove this issue

same on my side, did you reslove this issue

I give up using it,  turn to OpenCV VideoCapture no this issue.


Convert commands to lua, add WorkspacesRename, update README

Thanks for accepting my last PR!
I went to add the new stuff to the README and realized that I hadn't added a :WorkspacesRename command. I went ahead and converted the command setup to lua and then added the missing command.

Thanks again! Looks good, merging! 🥳


Trivial issue about utils _VERSION

This simple program
local utils = require "pl.utils"
print("Penlight "..utils._VERSION)
prints "Penlight 1.5.2" with the 1.5.4 version of Penlight
utils.lua, line 14 has
_VERSION = "1.5.2",
Sorry this is such a trivial issue, thought I'd point it out and leave it to you from here.
Always thanks for a great library,
Jeff

Fixed (although that's doesn't do much with the version already released).


Unhandled exception:
Dummy exception to set exit code.
#0      _rootHandleUncaughtError.<anonymous closure> (dart:async/zone.dart:1112:29)
#1      _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#2      _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#3      _Timer._runTimers (dart:isolate/runtime/libtimer_impl.dart:391:30)
#4      _Timer._handleMessage (dart:isolate/runtime/libtimer_impl.dart:416:5)
#5      _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:171:12)

@stevenroose I had issues with the dart environment (mailed you about it) and couldn't get the test suite running, hence the tests were never run. I'll have a look and see if i can get them sorted out some time.

Yeah I made a commit removing some old unused packages that also broke the
compilation on my machine. So now I can run them again.
Must have missed your e-mail. Usually better to create issues, I have
filters for that :p
On Fri, Feb 22, 2019, 14:14 Duncan Hoggan<EMAIL_ADDRESS>wrote:

@stevenroose https://github.com/stevenroose I had issues with the dart
environment (mailed you about it) and couldn't get the test suite running,
hence the tests were never run. I'll have a look and see if i can get them
sorted out some time.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PointyCastle/pointycastle/issues/173#issuecomment-466409879,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0F3El6WO8VmyGWJWXC-4k6FjxbyunUks5vP_tcgaJpZM4bFlEO
.

I've checked the test and it is structured correctly the length warning is also correct. I will adjust the test data accordingly, won't be difficult, just need the correct input parameters. Also you did reply on the mail, your rules are working fine 😁.


VirtualBoxOVA does not work for more than 64gb baseImageSize

Issue description
The standard way to create a nixos virtual box .ova files (config.system.build.virtualBoxOVA) does not work if the baseImageSize is more than 64gb.
Steps to reproduce
Specify baseImageSize more than 64gb
virtualbox = {
    baseImageSize = 65 * 1024; # in MiB
}

Technical details
The VM creation hangs with this output
[ 3.565686] This architecture does not have kernel memory protection.
[ 3.595800] EXT4-fs (vda1): mounted filesystem with ordered data mode. Opts:
[ 6.028488] random: crng init done

This is the strace from cptofs which does 100% cpu usage and does not finish.
timer_settime(0, 0, {it_interval={tv_sec=0, tv_nsec=0}, it_value={tv_sec=0, tv_nsec=9839123}}, NULL) = 0
timer_settime(0, 0, {it_interval={tv_sec=0, tv_nsec=0}, it_value={tv_sec=0, tv_nsec=9764387}}, NULL) = 0

Please run nix-shell -p nix-info --run "nix-info -m" and paste the
results.

system: "x86_64-linux"
host os: Linux 4.14.104, NixOS, 19.09pre171786.34aa254f9eb (Loris)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.2
channels(root): "nixos-19.09pre171786.34aa254f9eb"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

This was encountered in #49317 as well.

@matthewbauer I have this workaround #60246 to add an extra disk.

Does the extra disk not hit the 64gb limit?

No, the limit is only because of the bug in cptofs, and I do not use this tool. The extra disk is empty with only a filesystem.

@dfordivam could you open a bug upstream about cptofs being broken with images > 64GB? I doubt that's a nix-specific thing…

Why is cptofs being used here (specifically in make-disk-image)? It's anyway running in runInLinuxVM so couldn't we just mount the filesystem and cp?

Oh I see cptofs is part of preVM which is done before the VM is started. But couldn't it be avoided by sharing $root via 9P and copying the contents inside the VM?

Maybe, but cptofs shouldn’t have a 64G limit, right?

cptofs definitely shouldn't just burn CPU when working on >64G volumes, so we definitely should open a bug about that upstream.

This is definitely important to me.
Trying to encode sensible defaults into virtual appliances we plan on provisioning, and would love to see this fixed vs implementing more hacky workarounds than we already have regarding ESXi compatibility...

Still important though I’m not sure it makes sense to have this issue open if we’re just waiting on https://github.com/lkl/linux/issues/466.

Still quite important, but blocked on upstream


Move to Dagger 2

Dagger 1.x is not likely to continue. Dagger 2 is.
I'm interested enough to help port everything here (and maybe fix some bugs etc along the way), plus revamp upstream feign accordingly.
Please ping this issue if you feel strongly that feign should not move to Dagger 2.

The Dagger 2 approach is significantly incompatible with Dagger 1, particularly as multiple components with singleton scopes are unsupported, as is overrides. It is probably better to remove dagger so that callers can choose dagger 1, dagger 2, spring, guice, etc.


Feat: Add static images for development

Description
Just a small PR that adds support for adding images to the website.
This doesn't correct all image references, but only shows how to add additional static images for development's sake.
Only the alanturing.png was supported for the sake of example.
Interested parties may follow this PR as a guide to add the remaining images to the site.
This PR closes #71

Hey @jhlink, I'm not being able to see the images on the screen currently

@anoura12 , oh good catch. Looks like I didn't upload those files. One sec.

@anoura12 , uploaded.

Yay, I can see them now! Looks great, @jhlink !!
Ready to merge.


Generate APK file

Hi, can you tell why my app can't generate the apk file !? thank you in advance.
I added these lines to package.json :
"build:development:ios": "ionic cordova build ios",
"build:development:android": "ionic cordova build android",
"build:production:ios": "ionic cordova build ios --prod --release",
"build:production:android": "ionic cordova build android --prod --release"

When i execute this command "npm run build:development:android" by CLI: it displays these errors:
_You have been opted out of telemetry. To change this, run: cordova telemetry on.
Android Studio project detected
(node:34354) UnhandledPromiseRejectionWarning: CordovaError: Failed to find 'JAVA_HOME' environment variable. Try setting it manually.
at /Users/mac/Desktop/projets/cielapp/platforms/android/cordova/lib/check_reqs.js:177:35
at _rejected (/Users/mac/Desktop/projets/cielapp/platforms/android/cordova/node_modules/q/q.js:864:24)
at /Users/mac/Desktop/projets/cielapp/platforms/android/cordova/node_modules/q/q.js:890:30
at Promise.when (/Users/mac/Desktop/projets/cielapp/platforms/android/cordova/node_modules/q/q.js:1142:31)
at Promise.promise.promiseDispatch (/Users/mac/Desktop/projets/cielapp/platforms/android/cordova/node_modules/q/q.js:808:41)
at /Users/mac/Desktop/projets/cielapp/platforms/android/cordova/node_modules/q/q.js:624:44
at runSingle (/Users/mac/Desktop/projets/cielapp/platforms/android/cordova/node_modules/q/q.js:137:13)
at flush (/Users/mac/Desktop/projets/cielapp/platforms/android/cordova/node_modules/q/q.js:125:13)
at process.tickCallback (internal/process/next_tick.js:61:11)
(node:34354) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:34354) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Hi, i resolved the problem (this for MacOs of course) :
First of all, this command was very important for me,
cordova requirements
Steps:

I deactivated telemetry :
cordova telemetry off

I downloaded and installed JDK version 8 :
http://www.oracle.com/technetwork/java/javase/downloads/jdk8-downloads-2133151.html

N.B: maybe you can find a version problem if you do not install this version, so anyway you can try to delete the wrong version if it does not work ! For example, i installed the 10 version but it displays error in building, so i deleted it by this:
sudo rm -rf jdk-10.0.2.jdk

I uploaded Gradle :
https://gradle.org/install/#manually

N.B: I uploaded the complete version.

I putted the Gradle folder in the folder of my project (i think this is optional to choose the path !)

Return to user folder :
cd ~/

Create a file called .bash_profile in user folder
touch .bash_profile

Open this file
open -e .bash_profile

I wrote the content below in this file :

**

export JAVA_HOME=${/usr/libexec/java_home}
export GRADLE_HOME=/Users/mac/Desktop/Projets/gradle-4.10.1
export PATH=$PATH:$GRADLE_HOME/bin
**

N.B: you can see that i putted the Gradle folder (pay attention to the version that you will upload) in "Desktop/Projets".

i executed this command to install Gradle :
gradle


Port MQTT as s-c-s sink

From @sabbyanandan on November 14, 2015 0:14
As a developer, I'd like to move mqtt module from XD to s-c-s repo, so I can use it as sink to build streaming pipeline.
Acceptance:

mqtt as sink module is available in spring-cloud-stream-modules repo
maven repo includes mqtt module with respective group/artifact/version detail

Copied from original issue: spring-cloud/spring-cloud-stream-modules#79

This issue was moved to spring-cloud-stream-app-starters/app-starters-release#13


Uno R4 WiFi OTA update

Apply after #99
Needs WiFi firmware 0.3.0 -> https://github.com/arduino/uno-r4-wifi-usb-bridge/releases/tag/0.3.0-rc1

Uno R4 WiFi OTA

ci is failing because library is not included in the previous core releases.


Spacing out include of .md.part from youtube if has only one line

Here is a file.md.part:
Test

Here is a file.md:
---
title: File
---

{{< include file="/file.md.part" >}}

{{< youtube -dJolYw8tnk >}}

It renders without any space below the text, and looks cramped:

Next, if you make the .md.part have multiple lines, the rendering suddenly adds space below the include:
Test

Test

Unfortunately, use of
 or   adds too much vertical whitespace.
Can Geekdocs automatically add space below the single-line include of a .md.part?
Alternately, can Geekdocs have the ability to insert a small vertical space between two lines?

Fwiw, this
 hack seems accomplishes the job:
Text

The behavior is not related to the theme and affects all short codes, not only the custom include shortcode. Adding two or more youtube shortcodes right after each other will also not render a spacing.
You might want to take a look at https://github.com/gohugoio/hugo/issues/1642 where also some workarounds are described.
Why you don't want to use
{{< youtube -dJolYw8tnk >}}

{{< youtube -dJolYw8tnk >}}

or
{{< youtube -dJolYw8tnk >}}
\
{{< youtube -dJolYw8tnk >}}

is a bit unclear to me because both variants are shorter and the spacing is smaller that what

 creates, at least for me.

Okay I follow.  For me in the .md content file:

\

We can see

 has slightly less spacing.  It matches using

 at the bottom of the .md.part file.

Thanks @xoxys for sharing a relevant link.  Since it seems the bug is outside of this theme, I will close this out!
Feel free to reopen or follow up! 👌


Mesa 22.2.0

Checklist
https://docs.mesa3d.org/relnotes/22.2.0.html

[x] Checked the nixpkgs master branch

[x] Checked the nixpkgs pull requests

Project name
nix search name: legacyPackages.x86_64-linux.mesa

current version: 22.1.7
desired version: 22.2.0
Notify maintainers

maintainers: @primeos @vcunat
Note for maintainers
Please tag this issue in your PR.

https://github.com/NixOS/nixpkgs/pull/192243#issuecomment-1254183723


Parsing error with post keys

I'm not quite sure if I'm missing something here, but I think I found a parsing error. This seems to be a minimal working example:
{
  post.postA = ''
  '';
}

When opening this in Helix, I get this error:
helix_view::document [ERROR] TS parser failed, disabeling TS for the current buffer: Err(InvalidRanges)

It seems to happen when postFoo.postBar is the key of an attribute set. I found it while trying to do this:
{
  systemd.services.postgresql.postStart = ''
  '';
}

I'm not entirely sure what's going on here. Feel free to close if this doesn't actually have anything to do with this parser.

this is a problem on the helix side see https://github.com/helix-editor/helix/issues/3816


Add bbo.do to the webring

I would love to add my personal homepage, bbo.do, to the webring. I have added the icon in a link in the global navbar, present on all subpages. It should render on the top right, next to the github icon, and the button to change the theme.
If at all possible, I would love to be able to land at number 64. I have added the entry at the approximate location, but I'm guessing this is subject to change due to dead links being pruned.
Your projects are a cotinuous inspiration, thanks a ton.
xoxo
bodo

Hi Bodo, currently, all your projects are hosted off-site and do not have pages on the site itself, so I cannot merge the PR, but if you do create pages for the projects I'd be happy to add you as number 64 ;) Nice website btw, I'd love to add it when it's a bit further down the line. Keep me posted!


Training Pipeline

Hi,
Thank you for sharing your impressive work on RollingDepth! The results are outstanding.
I'm interested in training the model on a small dataset so I can improve my understanding of it. From reviewing the repository, it seems that the current code is primarily designed for inference. I was wondering if you plan to release the training pipeline or any code that illustrates how the model was trained? If this code already exists in the repository, could you point me to where it is? This would be incredibly helpful.
Thank you in advance!

I just realised there was a closed issue asking the same question, and I apologise for not seeing it first!


Support for Vite 3.x.x

Hi Team,
First, I would like to congrats on this awesome tool. thanks for sharing and contributing.
Will you have support for React@18, React-Router-Dom@6, and vite@3? It will be awesome to have news about this.
Thanks so much!

Hi! Thanks for the kind words. Vite 4 support landed in 0.17.0.
For React 18 there's already an issue open so we can track it there: https://github.com/frandiox/vite-ssr/issues/166


What do the models look like?

Your example is incomplete. You have
app.use(graffiti.express({
  prefix: '/graphql',
  adapter: graffitiMongoose,
  models: [User, Cat]
}));

But don't tell us what User and Cat look like. That's kind of important, isn't it?
Is there a mapping between the models and the database? Or does graffiti expose all DB fields? Is that not a security risk?
How does it handle relationships? Or it doesn't, because it only supports Mongo documents which already hold the relationships?

The models are available in the example app: https://github.com/RisingStack/graffiti-example

@mnpenner @jeremy-hunter thanks for the help! Yes these the adapter related models, in this case mongoose models while you are using graffiti with the graffiti-mongoose adapter.


Install fail for python 3

pip3 install git-lint

is trying to install dependency futures even for python 3.
Fix would be to remove futures from install_requires in setup.py

Not removed but just add the dep as
 extras_require={':python_version == "2.7"': ['futures']}

Thanks for the report and the PR.
It's fixed now. I deployed version 0.1.1 to pypi.


Lint issues caused by capital characters in error string

Description
Found 5 lint suggestions; failing.
225
##[error]/home/runner/work/l5act/l5act/handlers/mesh_ops_handlers.go:77:22: error strings should not be capitalized or end with punctuation or a newline
226
##[error]/home/runner/work/l5act/l5act/handlers/mesh_ops_handlers.go:187:21: error strings should not be capitalized or end with punctuation or a newline
227
##[error]/home/runner/work/l5act/l5act/handlers/mesh_ops_handlers.go:254:21: error strings should not be capitalized or end with punctuation or a newline
228
##[error]/home/runner/work/l5act/l5act/handlers/mesh_ops_handlers.go:282:2: var operationId should be operationID
229
##[error]/home/runner/work/l5act/l5act/handlers/mesh_ops_handlers.go:346:21: error strings should not be capitalized or end with punctuation or a newline
230
Found 6 lint suggestions; failing.
231
##[error]/home/runner/work/l5act/l5act/helpers/session_badger_persister.go:55:26: error strings should not be capitalized or end with punctuation or a newline
232
##[error]/home/runner/work/l5act/l5act/helpers/session_badger_persister.go:59:26: error strings should not be capitalized or end with punctuation or a newline
233
##[error]/home/runner/work/l5act/l5act/helpers/session_badger_persister.go:99:21: error strings should not be capitalized or end with punctuation or a newline
234
##[error]/home/runner/work/l5act/l5act/helpers/session_badger_persister.go:103:21: error strings should not be capitalized or end with punctuation or a newline
235
##[error]/home/runner/work/l5act/l5act/helpers/session_badger_persister.go:124:21: error strings should not be capitalized or end with punctuation or a newline
236
##[error]/home/runner/work/l5act/l5act/helpers/session_badger_persister.go:128:21: error strings should not be capitalized or end with punctuation or a newline

This is closed by recent linter change.


Prefer "--disk-size" over "--size"

The --size option can be ambiguous, so we should prefer a --disk-size option.  The --size option will remain for compatibility, but will not be documented.
This affects:

db:create
db:restart
db:replicate
backup:restore

Codecov Report

Merging #256 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #256      +/-   ##
==========================================
+ Coverage   96.13%   96.14%   +0.01%
==========================================
  Files          83       83
  Lines        3696     3714      +18
==========================================
+ Hits         3553     3571      +18
  Misses        143      143

Impacted Files
Coverage Δ

lib/aptible/cli/subcommands/db.rb
97.7% <100%> (+0.06%)
:arrow_up:

spec/aptible/cli/subcommands/db_spec.rb
100% <100%> (ø)
:arrow_up:

lib/aptible/cli/subcommands/backup.rb
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


API call to translate .rawgraph into .jpeg/.png/etc

(I've made a good-faith, hour+ attempt to search through docs and hope this question is reasonable)
Goal:  translate a .rawgraph file into a rendered graph file programatically.
Is there a REST-like call to the server, or do I need to use the code.
I couldn't find any docs on this in Wiki (not enough context to understand the API section).

Hi @cuddihyge your question is reasonable :)
We're publishing the rawgraphs-core (the library used by rawgraphs app) code and docs in the next weeks, and we'll explain in detail the process of rendering a rawgraphs chart from your javascript code (also from a project).
Here you can see an early preview of the API usage for this purpose:
https://observablehq.com/@bianchimro/test-rawgraphs
but keep in mind that the api is not totally stable and it could slightly change before the public release.
Also note that the example I linked lives in observablehq environment .. so the syntax is not "vanilla" javascript.

Hi @cuddihyge, we just released the rawgraphs-core library.
The api is not 100% stable and docs not 100% complete, but you should find all the information needed to perform your task, the example I linked in the previous comment is still a valid example.


[Tech support] Colored and numberer feature not working

I self-hosted the latest version of fiche.
I use Caddy as my webserver.
https://example.com, https://l.example.com {
  timeouts none
  tls<EMAIL_ADDRESS>  root /to/path/codes
  index index.txt index.html
}

However, after I manually prepend l. , it will still output plain text without any colored or numbered content.
cc #68 @Evidlo

You have to self-host the colouring service as well. You can use our script: https://github.com/solusipse/fiche/tree/master/extras/lines
Let me know if you need further help


Crash on Android : Unable to instantiate receiver com.kaya.playify.playify.PlayifyPlayer

Hey , am facing a crash on my android device , with error :
Unable to instantiate receiver com.kaya.playify.playify.PlayifyPlayer: java.lang.ClassCastException: com.kaya.playify.playify.PlayifyPlayer cannot be cast to android.content.BroadcastReceiver
only when i try to change Volume ( either programatically , or using hardware button )
I got this crash only on Samsung device ( Android version 12 )
And it was working fine on One plus ( Android Version 11 )

I have the same issue:
Fatal Exception: java.lang.RuntimeException
Unable to instantiate receiver com.kaya.playify.playify.PlayifyPlayer: java.lang.ClassCastException: com.kaya.playify.playify.PlayifyPlayer cannot be cast to android.content.BroadcastReceiver


Match mcardGCN.c for mq-j, mq-u, ce-u and ce-e

this doesn't do mq-e because I don't feel motivated enough atm but feel free to work on it, some parts aren't very pretty (mcardWrite for instance) but I'm out of ideas, also should I add the accurate struct offsets? it's only a difference of 0x4 (mCard->isBroken doesn't exist on mq-j)

I might try to play with making things look a little prettier, I'll try not to take too long though


Disable DANDI test temporarily (again)

CI breaking again due to failure on DANDI server
Good to catch so we can warn them and get a fix ASAP
But for years this test has run without hardly ever causing issues, wonder what's been going on lately

The better long-term solution might just be to turn this one test into a separate check run outside of the global pytest, kind of like how we treat imports
and then make the result of the test a non-requirement for merging, so we can be aware of upload issues as they happen (since that's pretty important for us and user base), but not hold up our own dev cycle

it looks like the MaxOneRecordingInterface is having problems for Python 3.11

Great, yet another intermittent issue then - looks like rerunning worked the second time 🤨


AdsToJava Maven version mismatch

When building the AdsToJava library, the latest version of Maven is not compatible with the POM file.
Recommend updating the POM file to reference Maven version 1.8 and place a note in the build procedure stating the required Maven version for the library.

Hi @robertsonj3,
thank you for your feedback.
You are correct, Maven 3.9+ requires JDK 8 or above to execute.
Your issue should be resolved by 9b970de1c36df58a7a5da3864b5f0a068fbdf760
Best Regards, Jannik Voss


Slack Socket Mode code cause application break

Describe the bug
After pulling the latest code from the master branch that contains Slack socket mode, the application cannot start. Previously, I was using Slack event mode and already set the following env:

SLACK_SIGNING_SECRET
SLACK_API_BOT_TOKEN

To Reproduce
Steps to reproduce the behavior:

Pull the latest code from commit 7cce8b7a9f3c2e051ba7472a9846fd6ab6787a23
Deploy on my server
Application cannot start

Expected behavior
Application starts with Slack Event mode enabled and working as usual.
Error From Server Log
Traceback (most recent call last):
File "/usr/local/lib/python3.8/site-packages/sentry_sdk/integrations/aiohttp.py", line 25, in <module>
from aiohttp import __version__ as AIOHTTP_VERSION
ModuleNotFoundError: No module named 'aiohttp'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):

File "/usr/local/bin/dispatch", line 5, in <module>
from dispatch.cli import entrypoint
File "/usr/local/lib/python3.8/site-packages/dispatch/cli.py", line 15, in <module>
from .main import *  # noqa
File "/usr/local/lib/python3.8/site-packages/dispatch/main.py", line 20, in <module>
from .extensions import configure_extensions
File "/usr/local/lib/python3.8/site-packages/dispatch/extensions.py", line 5, in <module>
from sentry_sdk.integrations.aiohttp import AioHttpIntegration
File "/usr/local/lib/python3.8/site-packages/sentry_sdk/integrations/aiohttp.py", line 28, in <module>
raise DidNotEnable("AIOHTTP not installed")
sentry_sdk.integrations.DidNotEnable: AIOHTTP not installed

I got this error as well. I was able to fix it by adding aiohttp to the requirements-base.txt file.

I got this error as well. I was able to fix it by adding aiohttp to the requirements-base.txt file.

I think slackclient was using aiohttp and hence it was transitively imported and now has to be explicitly imported. Just a guess.

I think slackclient was using aiohttp and hence it was transitively imported and now has to be explicitly imported. Just a guess.

Yes, it seems that aiohttp is now an optional dependency of slack_sdk where in slackclient it was a required dependency. I will get this added the the requirements file.

Yes, it seems that aiohttp is now an optional dependency of slack_sdk where in slackclient it was a required dependency. I will get this added the the requirements file.


Fix btcd running on server

Tried to delete btcd data and run again. However the problem is not solved. Error appears that cannot continue syncing:
2024-10-07 14:42:51.045 [INF] CHAN: Adding orphan block<PHONE_NUMBER>00000000006fb8f4a5d906e9c0112d5a97188f392407ab8e95bd81 with parent<PHONE_NUMBER>0000000002864156220f1093e76caf233009c1b6be9ee0d810ac29 2024-10-07 14:42:51.067 [INF] CHAN: Adding orphan block<PHONE_NUMBER>00000000028f255235dca42b10e5da593c2d4eb006cb329a041587 with parent<PHONE_NUMBER>00000000006fb8f4a5d906e9c0112d5a97188f392407ab8e95bd81 2024-10-07 14:42:51.182 [INF] CHAN: Adding orphan block<PHONE_NUMBER>00000000024590edc9d2d4878b32a4944dfda1a3929a6e4c9c3592 with parent<PHONE_NUMBER>00000000028f255235dca42b10e5da593c2d4eb006cb329a041587 2024-10-07 14:42:51.216 [INF] CHAN: Adding orphan block<PHONE_NUMBER>0000000001e2486f12c01ca0f76481b40181bf6f8f48802ade8c49 with parent<PHONE_NUMBER>00000000024590edc9d2d4878b32a4944dfda1a3929a6e4c9c3592 2024-10-07 14:42:51.341 [INF] CHAN: Adding orphan block<PHONE_NUMBER>0000000002c7b5bcf3372c7441e79bda1310c53f35eb59483b9092 with parent<PHONE_NUMBER>0000000001e2486f12c01ca0f76481b40181bf6f8f48802ade8c49 2024-10-07 14:42:51.378 [INF] CHAN: Adding orphan block<PHONE_NUMBER>00000000005ea10805f8ab474b9888bdc2c2840cd2e5529bbd0d49 with parent<PHONE_NUMBER>0000000002c7b5bcf3372c7441e79bda1310c53f35eb59483b9092 2024-10-07 14:42:51.499 [INF] CHAN: Adding orphan block<PHONE_NUMBER>0000000001428e153a325e9aa859589a80e8b0271d1ba48e8749c7 with parent<PHONE_NUMBER>00000000005ea10805f8ab474b9888bdc2c2840cd2e5529bbd0d49 2024-10-07 14:42:51.537 [INF] CHAN: Adding orphan block<PHONE_NUMBER>00000000004648af338d38563d26c3a5bef3ca9582ea2ccb72f8ea with parent<PHONE_NUMBER>0000000001428e153a325e9aa859589a80e8b0271d1ba48e8749c7 2024-10-07 14:42:51.661 [INF] CHAN: Adding orphan block<PHONE_NUMBER>0000000001a427c71546cda9a5577d5e38bc95a5d3450df7c1d26f with parent<PHONE_NUMBER>00000000004648af338d38563d26c3a5bef3ca9582ea2ccb72f8ea 2024-10-07 14:42:51.695 [INF] CHAN: Adding orphan block<PHONE_NUMBER>0000000002daff4047da69c658a1badb00d14d7d3e709f76b8bf3b with parent<PHONE_NUMBER>0000000001a427c71546cda9a5577d5e38bc95a5d3450df7c1d26f
Tried to find out and some related btcd issues are as follows:
https://github.com/btcsuite/btcd/issues/2121
https://github.com/btcsuite/btcd/issues/2090

Solved by running the latest source code version of btcd instead of the latest release version
Syncing...

Any plans on releasing soon a version that solves this issue?

Any plans on releasing soon a version that solves this issue?

Actually I use btcd to solve my problem in another project
I think btcd managers will release it soon because it might be a problem for many people
Since the merged PR solved this problem: https://github.com/btcsuite/btcd/pull/2208


Converting parameter from integer to string which fits to cron function

@see: #13

Merged, thanks !


Support Hashtags

Any plans on supporting hashtags (# and @)?
So, marking them on handling tabs?

Thanks for the suggestion. I released a new version which supports hashtags, mentions and links. Let me know what you think and if it works for you.

Looks great 👍
Thanks 😀


Upgraded to Angular 6 and fixed for Bootstrap 4 support

Upgraded to Angular 6 and fixed for Bootstrap 4 support

will try to work on updating the readme file next

Awesome. I'm working on bringing my bootstrap 4 support changes in. Looks like the CI stuff is still broken too. Hopefully we can get a release out soon.

would love to assist you with the CI stuff if possible still trying to get used to travis. if possible if you could email me on<EMAIL_ADDRESS>on how you CI is setup would really like to take a look at it too.

will do hey.


[BUG] failed to use input visual spec

from Rath Issues #139: [BUG] visual spec failed to bring to graphic-walker