Override probot/stale defaults, if necessary

Per the discussion in https://github.com/exercism/discussions/issues/128 we
will be installing the probot/stale integration on the Exercism organization on
April 10th, 2017.
By default, probot will comment on issues that are older than 60 days, warning
that they are stale. If there is no movement in 7 days, the bot will close the issue.
By default, anything with the labels security or pinned will not be closed by
probot.
If you wish to override these settings, create a .github/stale.yml file as described
in https://github.com/probot/stale#usage, and make sure that it is merged
before April 10th.
If the defaults are fine for this repository, then there is nothing further to do.
You may close this issue.

It is at this point that I wish I had the time to use https://developer.github.com/v3/issues/comments/ to get some data on this: Which issues would have been auto-closed by probot, with a 60-day strategy? Should they have been closed?
What do I mean by "should", even? Well, it seems to me that there are various kinds of issues:

Issues where we've decided on a clear plan of action

Good examples: the good first patch issues.
These issues should probably stay open until the action is taken, no matter how stale they get.
Only exception probably is if they get so stale that the action they recommend is no longer relevant to the repo, but the bot does not have the ability to decide that, so I say the bot should never close these issues.
For these issues, the bot forces extra work on us, by forcing us to apply a label to exempt it.

Issues where there isn't a clear plan of action, perhaps discussions or questions.

Good examples... ??? I don't have one for the Haskell track right now.
For these it seems being stale implies not enough consensus to change the status quo and/or the querent is no longer interested in the question.
For these issues, the bot saves us work.

If the bot creates more work than it saves, then I would obviously oppose having it. Since I don't have that data right now, I'll defer my vote. Feel free to proceed and make a decision without having my vote.

I'm finally ready to look at the data.
I used https://gist.github.com/petertseng/04edd07d4b35cda2cbc1d8142b6fcc13 to get a rough idea of the issues that would have been closed by this integration. (Limitations: It doesn't look at pull requests, and it may have false positives for issues that are edited after they are closed)
In this track's history, 25/124 issues went stale (I deliberately do not include a # in front of each since I don't mean to reference those issues from this one):
470 Document periodic maintenance tasks (Stack snapshot update, cache clear)
469 Changes to Luhn tests
455 space-age: Typo in HINTS.md
452 discussion: Test cases about memory and/or performance: Should we have them?
442 stack.yaml: Avoiding multiplicity
416 Ensuring we are up to date with x-common
398 readme: Fix instructions for running the tests.
397 Allow marking an example as should-fail
274 Update config.json to match new specification
211 tests: Tracking migration to hspec with fail-fast.
130 travis: All GHCs use same cache; check whether that's OK
129 travis: Automatically invalidate caches if new package versions exist
125 clock: canonical test data has been improved
124 travis: Document why we test the GHC versions that we do
122 check-exercises.hs redundant import warning after GHC 7.10.x
119 travis: all builds install all GHC versions, rather than just the one they're testing
117 travis: 7.10.1 -> 7.10.3?
116 robot-simulator: "turning edge cases" are not necessarily edge cases
115 discussion: Should we return Maybe when there is a possibility of invalid input?
111 binary: improve tests for invalid numbers
 99 Verify "Largest Series Product" exercise implementation
 92 pov: Consider making tests insensitive to child ordering
 88 Move exercises to subdirectory
 85 Gigasecond - Could not find module ‘System.Locale’
 59 Binary search tree cannot be a monoid

Of these, I would say 88, 92, 99, 111, 116, 117, 119, 122, 124, 125, 129, 274, 397, 398, 455, 470 at least had clear enough calls to action that it seems like it doesn't make sense to close them even if they go stale. Then, 211 was known to be a long-term issue. So in 17 cases the bot would have caused extra work for the maintainers of this track and in only 8 cases the bot can save work.
As I promised, if the bot would create more work than it saves, I would oppose having it. So here is my vote in opposition.

Agreed. I think that in this track we've been really good about making issues actionable, so the bot doesn't make sense.

Hi. Guess nobody else wants to weigh in. Well, if there's one vote in opposition (mine) and no other votes, I suppose that means no bot, which is obviously fine with me. Let us know if that should change.


Update Black to 24.10

Trivy scan revealed a medium vulnerability in 21.9b0

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.85%. Comparing base (a350935) to head (9a868a0).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #21   +/-   ##
=======================================
  Coverage   84.85%   84.85%
=======================================
  Files           7        7
  Lines         284      284
=======================================
  Hits          241      241
  Misses         43       43

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


代码更新到最新版本之后报错

When you git pull the code to the latest version, an error will be reported, which was normal before.
I searched and found that the error started from the commit 9ab298dd49c86e8eff12e3ce70ed90a5514c6193, which is "[add llm export script](https://github.com/FunAudioLLM/CosyVoice/commit/9ab298dd49c86e8eff12e3ce70ed90a55 14c6193)".
The development environment is Win11 virtual environment Ubuntu 22.04.3 LTS,  python 3.8.
具体报错内容是：
Traceback (most recent call last):
File "webui.py", line 179, in
cosyvoice = CosyVoice(args.model_dir)
File<EMAIL_ADDRESS>line 30, in init
configs = load_hyperpyyaml(f)
File "/home/laobai/anaconda3/envs/cosyvoice/lib/python3.8/site-packages/hyperpyyaml/core.py", line 188, in load_hyperpyyaml
hparams = yaml.load(yaml_stream, Loader=loader)
File "/home/laobai/anaconda3/envs/cosyvoice/lib/python3.8/site-packages/yaml/init.py", line 81, in load
return loader.get_single_data()
File "/home/laobai/anaconda3/envs/cosyvoice/lib/python3.8/site-packages/ruamel/yaml/constructor.py", line 116, in get_single_data
return self.construct_document(node)
File "/home/laobai/anaconda3/envs/cosyvoice/lib/python3.8/site-packages/ruamel/yaml/constructor.py", line 120, in construct_document
data = self.construct_object(node)
File "/home/laobai/anaconda3/envs/cosyvoice/lib/python3.8/site-packages/ruamel/yaml/constructor.py", line 147, in construct_object
data = self.construct_non_recursive_object(node)
File "/home/laobai/anaconda3/envs/cosyvoice/lib/python3.8/site-packages/ruamel/yaml/constructor.py", line 188, in construct_non_recursive_object
for _dummy in generator:
File "/home/laobai/anaconda3/envs/cosyvoice/lib/python3.8/site-packages/ruamel/yaml/constructor.py", line 633, in construct_yaml_map
value = self.construct_mapping(node)
File "/home/laobai/anaconda3/envs/cosyvoice/lib/python3.8/site-packages/ruamel/yaml/constructor.py", line 429, in construct_mapping
return BaseConstructor.construct_mapping(self, node, deep=deep)
File "/home/laobai/anaconda3/envs/cosyvoice/lib/python3.8/site-packages/ruamel/yaml/constructor.py", line 244, in construct_mapping
value = self.construct_object(value_node, deep=deep)
File "/home/laobai/anaconda3/envs/cosyvoice/lib/python3.8/site-packages/ruamel/yaml/constructor.py", line 147, in construct_object
data = self.construct_non_recursive_object(node)
File "/home/laobai/anaconda3/envs/cosyvoice/lib/python3.8/site-packages/ruamel/yaml/constructor.py", line 183, in construct_non_recursive_object
data = constructor(self, tag_suffix, node)
File "/home/laobai/anaconda3/envs/cosyvoice/lib/python3.8/site-packages/hyperpyyaml/core.py", line 481, in construct_object
return callable(*args, **kwargs)
TypeError: ('Invalid argument to class cosyvoice.llm.llm.TransformerLM', "init() missing 1 required positional argument: 'sampling'")

重新下一下权重文件，缺了两个zip文件

重新下一下权重文件，缺了两个zip文件

谢谢了，可以了


[WIP] Add note that documents YAML compatibility bug

Add warning note that documents Issue 34146, which addresses YAML 1.1/1.2 compatibility issues in Kubernetes.
Fixes #36526

@sftim this is a first draft, I'm not even sure if the content belongs to this document ( I asked you in the issue comments). Thanks

Does it make sense to you, @chrishiestand , @reinvantveer?

Does it make sense to you, @chrishiestand , @reinvantveer?

Perfect!

/assign @sftim

Does it make sense to you, @chrishiestand , @reinvantveer?

I'm not a fan of this documentation closing kubernetes/kubernetes#34146, however we're approaching 7 years since the issue was opened and if there's no other fix in sight then... okay? And since I haven't contributed anything between then and now my vote ought to count for very little.
I think it's a good start although perhaps not comprehensive enough because like @2rs2ts and @fraenkel said, there are incompatibilities between 1.1 and 1.2 other than booleans, such as scientific notation/string types. I am not aware of any exhaustive but concise list detailing breaking changes, but if one exists then we ought to link to it or copy it. I wasn't able to find one after some quick googling, however your google fu may be better.
So I think before this is merged you should urge avoidance of all incompatibilities between 1.1 and 1.2, even if we don't list them explicitly. In other words, we k8s users should try to stick to the syntax/grammar intersection of yamls 1.1. and 1.2.
Thanks for your help @mrgiles!

The list of changes can be found here:
https://yaml.org/spec/1.2.2/ext/changes/
See the Changes in 1.2 section which describes what was removed/changed from 1.1

I'm not a fan of this documentation closing kubernetes/kubernetes#34146, however we're approaching 7 years since the issue was opened and if there's no other fix in sight then... okay? And since I haven't contributed anything between then and now my vote ought to count for very little.

@mrgiles I have edited the PR description to tweak that.

Thank you for the review, @chrishiestand. This PR was meant to close issue #36526, related to documentation, not the original issue, related to the actual YAML issue.
As @fraenkel notes, the YAML spec changes doc lists the differences among versions. There are, indeed, other changes that I may add to the note or, I could just add the link as a reference for the reader to explore further? The change list is rather long. @sftim what do you suggest?

I wouldn't list all the differences between YAML versions here. We're documenting Kubernetes, not YAML itself.

@sftim I added a change to the note and squashed the commits. Thanks

@sftim do you have any feedback on the latest change? How should I proceed? Thanks

@reylejano can you take a look at this PR? Thanks


How to cancel old requests?

Hey,
How I can cancel old request if I call some function of the request second time?
For example if I have get() function and call it twice with different params to cancel the first one and proceed with the second? Observables have swithMap but how I can do it with the ng2-resource-rest?

Please check README.

In the readme I can find only abortRequest() function which is not the same

It cancel will cancel the request.

I have this situation:
I want to abort the request but it seem that is not doing that.
The Resource:
class UserResource extends OAuthGuardResource {
    @ResourceAction({
        isArray: false,
        isLazy: true
    })
    query: ResourceMethod<IUserQueryInput, IUserShort[]>;
}

My Component:
class UserComponent implements OnInit {
    private query:any = null;

    constructor(private user:UserResource) { }

    ngOnInit() {
        this.refreshTable();
    }

    refreshTable():void {

        if(this.query && this.query.$resolved === false) {
            this.query.$abortRequest();
        }

        this.query = this.user.query();
        this.query.$observable
            .subscribe(
                (res: any) => {
                       // updating data
                },
                (error) => {
                       // Showing the error
                }
            );
    }

    onPageChange() {
        this.refreshTable();
    }
}

The idea is to refresh the table when change pagination, but aborting the request if the user change pagination before the other request has finished.
Thanks

With isLazy: true you have raw observable without $abortRequest function and any data transformation.
So in that case you have to handle abort with unsubscribe
class UserComponent implements OnInit {
    private query:any = null;

    constructor(private user:UserResource) { }

    ngOnInit() {
        this.refreshTable();
    }

    refreshTable():void {

        if(this.query) {
            this.query.unsubscribe();
        }

        this.query = this.user.query()
          .$observable
          .subscribe(
                (res: Response) => {
                       // updating data
                },
                (error) => {
                       // Showing the error
                },
                () => { // on complete
                      this.query = null;
                }
            );
    }

    onPageChange() {
        this.refreshTable();
    }
}

Good to know that. Thanks


Nether Cluster Spawning

General Info

[ ] I am running a modpack
[X] I can reproduce this issue consistently in single-player
[ ] I can reproduce this issue consistently in multi-player
[X] I have searched for this issue previously and it was either (1) not previously reported, or (2) previously fixed and I am having the same problem.
[ ] I am crashing and can provide my crash report(s)
[X] I am using the latest version of the mod

Forge version
43.2.14
Minecraft version
1.19.2
Mod version
1.2.4
Java version
18
Issue Description
I initially ran into this while on my personal modpack but it still happens when no other mod is installed. In the nether, a bunch of stuff spawns in clustered together (see attached screenshots). My interpretation of this is that it is attempting to place an amount of a feature (e.g. geysers) across an area, maybe a few chunks or something, but instead of spreading it out in across the area, it places as many as are allowed/allotted to that sector. For example if it decides that 9 geysers are to be in a given 100 block square, it places all the geysers in the first available 3 x 3 area. I saw others on here saying they were having nothing spawn in the nether but this seemed different enough that I thought it may be useful for fixing stuff. Now for the screenshots:

Additional Information
No response

having this error as well


Extract all theming CSS rules to jquery.mmenu.themes.css and avoid using wildcard selectors

Currently, theming jQuery.mmenu for your own layout requires that you redefine a lot of CSS rules. This could be improved by extracting all theming (font-family, size, colors, etc.) rules to jquery.mmenu.themes.css and avoiding the usage of wildcard (*) selectors.
Working with Bootstarp 3, the rule at line 133 on file jquery.mmenu.all.css messes with a lot of Bootstrap components (particularly with glyphicons):
.mm-list * {
    -webkit-text-size-adjust: none;
    font-family: Arial, Helvetica, sans-serif;
    font-size: 14px;
    text-shadow: none; }

For now, I just deleted this selector and some others for the sake of not wasting time.

@FrDH Seems like this is still a issue since you use @include font; in jquery.mmenu.scss

What do you mean with @include font; ??
The contents of jquery.mmenu.scss:
@import "jquery.mmenu.oncanvas";
@import "addons/jquery.mmenu.offcanvas";

The wildcard selector (*) is removed from the CSS.


Jax one-hot error

Report
https://discourse.scverse.org/t/scib-metrics-benchmarker-function-outputs-error/1682
Version information
No response

Fixed in #107


docs: fix code block in documentation

Fix closure of python code block
Description of changes:
Tiny change in the docs to fix a missing block closure in the markdown
Checklist

[x] Meet tenets criteria
[ ] Update tests. N/A
[x] Update docs
[x] PR title follows conventional commit semantics

Breaking change checklist

RFC issue #:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

View rendered docs/core/logger.md

Thank you so much for this @mozz100 !! Every doc PR makes me incredibly happy ;-)


Uncommitted GC - collect full path if part of repo storage namespace

When entry metadata physical path points to an object under the repository storage namespace - we consider it as data we manage.
Fix #4759

Added test for listing addresses of committed data.


Test audit job again

Testing audit job from https://github.com/apache/arrow-datafusion/pull/9182

it works! https://github.com/apache/arrow-datafusion/actions/runs/7877518929/job/21493850998?pr=9205


spelling in introduction.md

recieve -> receive
Description
Fix spelling

Thanks for the catch @triptec !

Actually looks like someone else opened the same typo fix here: https://github.com/continuedev/continue/pull/2433


Wrong error log

close: https://github.com/LunarG/gfxreconstruct/issues/548

Right now, Portal 2 doesn't use neither VK_EXT_buffer_device_address and VK_KHR_buffer_device_address extensions, so the error can't be regenerated. We also don't keep the old captured file to test.

https://github.com/locke-lunarg/Vulkan-Tools/commit/f6104490af091fe6bf5dccda9400330d20fd1123
I added VK_EXT_buffer_device_address into vkcube. It's captured and replayed successfully after I removed the GFXRECON_LOG_ERROR. (Some devices might not support VK_EXT_buffer_device_address) In this case, I think we should remove the GFXRECON_LOG_ERROR.

CI gfxreconstruct build # 1078 running.

CI gfxreconstruct build queued with queue ID 8766.

CI gfxreconstruct build # 1079 running.

CI gfxreconstruct build # 1079 passed.


Feature/vpn manager setup

Description
This PR sets up vpn manager using material UI and Next js.

Adds an execution script to read users from spreadsheet given and converts the data to JSON. pnpm process-new-hires.
Save the credentials provided here in a credentials.json.
Updatr .env as per .env.example

Fixes #678
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Screenshots
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation

@kelvinkipruto Have a look at this


#2974 Can not input line break between " and other command.

The issue was twofold:
1). Caret positioning after formatting was incorrect. For example, in
x |<- 1
typing ENTER produced
x
<|- 1

because caret position tracker only tracked changed range and offset from x was incorrect since autoformat removes trailing space. The fix is to for the tracking range to include next line do next token after the CRLF will be seen.
2). In
s <-
    "
     string
    "|x <- 1

upon ENTER formatter attempted to format     " not seeing that it is inside the string.

👍


Chunk get multiple accounts

Automatically chunk requests for get multiple accounts if the number of accounts is over 100 https://github.com/project-serum/anchor/blob/master/ts/src/utils/rpc.ts#L4

There's an implementation of this here https://github.com/solana-labs/dapp-scaffold/blob/eef68f4a46717b81549211594334d98bf358f87d/src/contexts/accounts.tsx#L432.

What does this achieve? I tried calling it on over 1000 accounts and it works fine.

What does this achieve? I tried calling it on over 1000 accounts and it works fine.

My understanding is that the JSON RPC api is limited to 100 accounts. Has that changed?

I see, I think there isn't a limit on the local validator but
"Default max is 100, by it can be overridden for a specific validator instance by the --rpc-max-multiple-accounts command-line argument"
I'll take this :)

My bad, I was using getProgramAccounts which doesn't seem to have this same limit.

Closed by #763.


Database help

can i convert offline uuid from uuid column to nickname in MYCUSTOMTABLE table? I configured it like this but nothing worked
database:
hostname: localhost
username: DATABASE USERNAME
password: DATABASE PASSWORD
config:
- database: DATABASE NAME
table: MYCUSTOMTABLE
convert: # Some examples, refer to documentation for explanations
- [uuid, column]

no

Should it create a player column in this table and write a nickname there?

It will be wonderful. Almost all normal economy plugins are made for uuid, but my site only accepts a nickname to synchronize currency between servers and site

Ok, some questions:

to clarify, your server is in offline mode?
with nickname, do you mean ingame name or really the nickname? (might be an EssentialsX thing? Often indicated with ~)
so you essentially need a database table with UUID/nickname combinations so your website can look up the nickname for a given uuid?

yes, the server is offline. I have CMI instead of EssentialX. Like this% player_name%, game nickname. The site can only connect to a table with columns nickname and balance, the name of the columns can be changed. And the economy plugin to uuid and balance (pocket - column). It is necessary that in one table there should be uuid and username and pocket (balance).
Economy plugin called MZP-Economy and MZP-Core API


Odd Issue

I am not sure why, but the images for the custom buttons and stuff do not work. The only thing this style changes for me is when I hover over and click a button it turns blue and the background is slightly more gray. On Linux, you can see the entry and buttons as you're supposed to, but only in the file dialog? On windows, nothing works but the slight color changes mentioned earlier. I do not believe this is an issue with my code. I am using the module.

Seems like you're using plain tkinter widgets, that can change only in their colors. Please change them to ttk widgets, since only they can be themed.


Uncaught OutOfRangeException

Hi, I'm getting the error:
Stack trace:
Message: Uncaught OutOfRangeException: Unknown field "errors" in xxx/vendor/layershifter/tld-extract/src/Result.php:220
File: xxx/vendor/layershifter/tld-extract/src/Result.php
Line: 220

thrown
#0 LayerShifter\TLDExtract\Result->__get('errors')

My code is straight from the example:
$extract = new LayerShifter\TLDExtract\Extract();
$result = $extract->parse('shop.github.com');
$result->getRegistrableDomain();

Composer install details:
Using version ^2.0 for layershifter/tld-extract
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Package operations: 5 installs, 0 updates, 0 removals
  - Installing symfony/polyfill-php72 (v1.10.0): Downloading (100%)
  - Installing symfony/polyfill-intl-idn (v1.10.0): Downloading (100%)
  - Installing layershifter/tld-support (1.1.1): Downloading (100%)
  - Installing layershifter/tld-database (1.0.65): Downloading (100%)
  - Installing layershifter/tld-extract (2.0.1): Downloading (100%)

Am I missing something or doing something wrong?
Many thanks,
jkns

<?php

include 'vendor/autoload.php';

$extract = new LayerShifter\TLDExtract\Extract();
$result = $extract->parse('shop.github.com');

var_dump(PHP_VERSION, $result->getRegistrableDomain());

string(5) "7.3.2"
string(10) "github.com"

Am I missing something or doing something wrong?

It seems yes, are you're trying to access $result->errors?

Ignore. Error was in my code that was already using $result as a variable!!!

<?php

include 'vendor/autoload.php';

$extract = new LayerShifter\TLDExtract\Extract();
$result = $extract->parse('shop.github.com');

var_dump(PHP_VERSION, $result->getRegistrableDomain());

string(5) "7.3.2"
string(10) "github.com"

Am I missing something or doing something wrong?

It seems yes, are you're trying to access $result->errors?

Many thanks for the quick reply, it was an embarrassing mistake that I made. Thanks for this package, good work.


segmentation fault.

In the Rotation branch... when I use flag "2" I get the following error:
C:\Users\Jam\Documents\Work\Arduino\MyProjects\MarqueeTextFast\MarqueeTextFast.ino: In function '_GLOBAL__sub_I_led':
C:\Users\Jam\Documents\Work\Arduino\MyProjects\MarqueeTextFast\MarqueeTextFast.ino:300:1: internal compiler error: Segmentation fault
}
^

Oh, that's interesting... It's the compiler that sigsegvd... Any more
details?
On Sat, 16 Mar 2019, 21:29 James Gohl<EMAIL_ADDRESS>wrote:

When I use flag "2" I get the following error:
C:\Users\Jam\Documents\Work\Arduino\MyProjects\MarqueeTextFast\MarqueeTextFast.ino:
In function '_GLOBAL__sub_I_led':
C:\Users\Jam\Documents\Work\Arduino\MyProjects\MarqueeTextFast\MarqueeTextFast.ino:300:1:
internal compiler error: Segmentation fault
}
^
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bartoszbielawski/LEDMatrixDriver/issues/31, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AD2sEIKbKosQVPx6UZRQl8YPxJzoA2WKks5vXVQvgaJpZM4b4A5P
.

C:\Users\Jam\Documents\Work\Arduino\MyProjects\MarqueeTextFast\MarqueeTextFast.ino: In function '_GLOBAL__sub_I_led':
C:\Users\Jam\Documents\Work\Arduino\MyProjects\MarqueeTextFast\MarqueeTextFast.ino:295:1: internal compiler error: Segmentation fault
}
^
Please submit a full bug report,
with preprocessed source if appropriate.
See http://gcc.gnu.org/bugs.html for instructions.
lto-wrapper.exe: fatal error: C:\Program Files (x86)\Arduino\hardware\tools\avr/bin/avr-gcc returned 1 exit status
compilation terminated.
c:/program files (x86)/arduino/hardware/tools/avr/bin/../lib/gcc/avr/5.4.0/../../../../avr/bin/ld.exe: error: lto-wrapper failed
collect2.exe: error: ld returned 1 exit status
exit status 1
Error compiling for board Arduino/Genuino Uno.

Can you tell me what's in that line? I don't have sources now.
On Sat, 16 Mar 2019, 21:32 James Gohl<EMAIL_ADDRESS>wrote:

C:\Users\Jam\Documents\Work\Arduino\MyProjects\MarqueeTextFast\MarqueeTextFast.ino:
In function '_GLOBAL__sub_I_led':
C:\Users\Jam\Documents\Work\Arduino\MyProjects\MarqueeTextFast\MarqueeTextFast.ino:295:1:
internal compiler error: Segmentation fault
}
^
Please submit a full bug report,
with preprocessed source if appropriate.
See http://gcc.gnu.org/bugs.html for instructions.
lto-wrapper.exe: fatal error: C:\Program Files
(x86)\Arduino\hardware\tools\avr/bin/avr-gcc returned 1 exit status
compilation terminated.
c:/program files
(x86)/arduino/hardware/tools/avr/bin/../lib/gcc/avr/5.4.0/../../../../avr/bin/ld.exe:
error: lto-wrapper failed
collect2.exe: error: ld returned 1 exit status
exit status 1
Error compiling for board Arduino/Genuino Uno.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bartoszbielawski/LEDMatrixDriver/issues/31#issuecomment-473590646,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD2sEHKh2jw36H2dyKlv24AObu4vWeP6ks5vXVTNgaJpZM4b4A5P
.

I can't find '_GLOBAL__sub_I_led' anywhere.
295:1 ... } is the last brace in my example.ino file.
Here is the last function:
void writeNext()
{
byte asc = text[textIndex] - 32;
// We need to handle space character since ignoring whitespace.
if (asc == 0)
{
scrollWhitespace = SPACE_WIDTH;
// Force change to next character in text[].
if (textIndex++ == textLen - 1)
{
textIndex = 0;
scrollWhitespace = LEDMATRIX_WIDTH;
}
return;
}
// Not set for this character, so find start column.
if (colLower == -1)
{
for (int c = 0; c < CHAR_WIDTH; c++)
{
if (font[asc][c] != 0)
{
colLower = c;
colIndex = c;
break;
}
}
}
// Not set for this character, so find finish column.
if (colUpper == CHAR_WIDTH)
{
for (int c = CHAR_WIDTH - 1; c > -1; c--)
{
if (font[asc][c] != 0)
{
colUpper = c;
break;
}
}
}
// Write the column to the right of the display.
led.setColumn(LEDMATRIX_WIDTH - 1, font[asc][colIndex]);
// Check if end of current character.
if (colIndex++ == colUpper)
{
// Reset column start and finsh.
colLower = -1; colUpper = CHAR_WIDTH;
// Add whitespace.
scrollWhitespace = CHAR_SPACING;
// Check if end of text string.
if (textIndex++ == textLen - 1)
{
  textIndex = 0;
  scrollWhitespace = LEDMATRIX_WIDTH;
}

}
}
Regards,
James Gohl
*Are you cyber streetwise?*https://www.cyberaware.gov.uk/
On Sat, 16 Mar 2019 at 20:47, Bartosz Bielawski<EMAIL_ADDRESS>wrote:

Can you tell me what's in that line? I don't have sources now.
On Sat, 16 Mar 2019, 21:32 James Gohl<EMAIL_ADDRESS>wrote:

C:\Users\Jam\Documents\Work\Arduino\MyProjects\MarqueeTextFast\MarqueeTextFast.ino:

In function '_GLOBAL__sub_I_led':

C:\Users\Jam\Documents\Work\Arduino\MyProjects\MarqueeTextFast\MarqueeTextFast.ino:295:1:

internal compiler error: Segmentation fault
}
^
Please submit a full bug report,
with preprocessed source if appropriate.
See http://gcc.gnu.org/bugs.html for instructions.
lto-wrapper.exe: fatal error: C:\Program Files
(x86)\Arduino\hardware\tools\avr/bin/avr-gcc returned 1 exit status
compilation terminated.
c:/program files

(x86)/arduino/hardware/tools/avr/bin/../lib/gcc/avr/5.4.0/../../../../avr/bin/ld.exe:

error: lto-wrapper failed
collect2.exe: error: ld returned 1 exit status
exit status 1
Error compiling for board Arduino/Genuino Uno.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<
https://github.com/bartoszbielawski/LEDMatrixDriver/issues/31#issuecomment-473590646
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AD2sEHKh2jw36H2dyKlv24AObu4vWeP6ks5vXVTNgaJpZM4b4A5P
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bartoszbielawski/LEDMatrixDriver/issues/31#issuecomment-473591599,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtJb3LfDyObdb_2y31sDS4RhYr6niwSks5vXVh2gaJpZM4b4A5P
.

The problem with this "bug" is that it doesn't actually happen in the library.
It's your compiler that critically fails (that is - crashes).
Can you try to run the set pixel example? This may tell us if the problem is in your code or in the library itself.

I have removed the line that calls setColumn and it doesn't crash the
compiler... if that helps.
Regards,
James Gohl
*Are you cyber streetwise?*https://www.cyberaware.gov.uk/
On Sat, 16 Mar 2019 at 22:06, Bartosz Bielawski<EMAIL_ADDRESS>wrote:

The problem with this "bug" is that it doesn't actually happen in the
library.
It's your compiler that critically fails (that is - crashes).
Can you try to run the set pixel example? This may tell us if the problem
is in your code or in the library itself.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bartoszbielawski/LEDMatrixDriver/issues/31#issuecomment-473596543,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtJb1PLOs4DgEQuuNGUwo7UwSChkUYzks5vXWrtgaJpZM4b4A5P
.

Okay, let me look at setColumn.
I just made your example work. For ESP8266 it doesn't crash compiler.

Learn about forward declarations. (writeNext() is called from loop()
that is before writeNext()),
LEDMatrixDriver led(LEDMATRIX_SEGMENTS, LEDMATRIX_CS_PIN, 7);
This was my reason for class enums... You can write it like this but why
not:
flags = LEDMatrixDriver::INVERT_Y | ....? Then you don't need to explain
what does 7 mean.

BB

calling setPixel is working fine.
Within setColumn... something is wrong in the "value" part of this code
perhaps...?
    setPixel(x, y, value & 1);
    value >>= 1;

Regards,
James Gohl
*Are you cyber streetwise?*https://www.cyberaware.gov.uk/
On Sat, 16 Mar 2019 at 22:24, James Gohl<EMAIL_ADDRESS>wrote:

I have removed the line that calls setColumn and it doesn't crash the
compiler... if that helps.
Regards,
James Gohl
*Are you cyber streetwise?*https://www.cyberaware.gov.uk/
On Sat, 16 Mar 2019 at 22:06, Bartosz Bielawski<EMAIL_ADDRESS>wrote:

The problem with this "bug" is that it doesn't actually happen in the
library.
It's your compiler that critically fails (that is - crashes).
Can you try to run the set pixel example? This may tell us if the problem
is in your code or in the library itself.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bartoszbielawski/LEDMatrixDriver/issues/31#issuecomment-473596543,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtJb1PLOs4DgEQuuNGUwo7UwSChkUYzks5vXWrtgaJpZM4b4A5P
.

okay so nothing has changed in setColumn since the last working version
which is really odd
Regards,
James Gohl
*Are you cyber streetwise?*https://www.cyberaware.gov.uk/
On Sat, 16 Mar 2019 at 22:30, James Gohl<EMAIL_ADDRESS>wrote:

calling setPixel is working fine.
Within setColumn... something is wrong in the "value" part of this code
perhaps...?
    setPixel(x, y, value & 1);
    value >>= 1;

Regards,
James Gohl
*Are you cyber streetwise?*https://www.cyberaware.gov.uk/
On Sat, 16 Mar 2019 at 22:24, James Gohl<EMAIL_ADDRESS>wrote:

I have removed the line that calls setColumn and it doesn't crash the
compiler... if that helps.
Regards,
James Gohl
*Are you cyber streetwise?*https://www.cyberaware.gov.uk/
On Sat, 16 Mar 2019 at 22:06, Bartosz Bielawski<EMAIL_ADDRESS>wrote:

The problem with this "bug" is that it doesn't actually happen in the
library.
It's your compiler that critically fails (that is - crashes).
Can you try to run the set pixel example? This may tell us if the
problem is in your code or in the library itself.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bartoszbielawski/LEDMatrixDriver/issues/31#issuecomment-473596543,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtJb1PLOs4DgEQuuNGUwo7UwSChkUYzks5vXWrtgaJpZM4b4A5P
.

So before you didn't use this method?
You can try to replace it with:
setPixel(x, y, value & 1 ? true: false);
value >>= 1;

But as you have noticed yourself - there was no change.
We can write it directly in the framebuffer but it shouldn't change anything.

Ok also noticed something odd, the compiler crash is intermittent. Out of
10 compile and uploads, 5 of them failed with the crash, and they occur in
no particular pattern.  Perhaps my Uno is faulty, unfortunately I only have
uno Uno ;)
Regards,
James Gohl
*Are you cyber streetwise?*https://www.cyberaware.gov.uk/
On Sat, 16 Mar 2019 at 22:37, Bartosz Bielawski<EMAIL_ADDRESS>wrote:

So before you didn't use this method?
You can try to replace it with:
setPixel(x, y, value & 1 ? true: false);
value >>= 1;
But as you have noticed yourself - there was no change.
We can write it directly in the framebuffer but it shouldn't change
anything.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bartoszbielawski/LEDMatrixDriver/issues/31#issuecomment-473598366,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtJbzX1H562KQTuSzGwNFqyCzTxROymks5vXXIygaJpZM4b4A5P
.

.... but then again, this is compiled on my machine first.
Regards,
James Gohl
*Are you cyber streetwise?*https://www.cyberaware.gov.uk/
On Sat, 16 Mar 2019 at 22:43, James Gohl<EMAIL_ADDRESS>wrote:

Ok also noticed something odd, the compiler crash is intermittent. Out of
10 compile and uploads, 5 of them failed with the crash, and they occur in
no particular pattern.  Perhaps my Uno is faulty, unfortunately I only have
uno Uno ;)
Regards,
James Gohl
*Are you cyber streetwise?*https://www.cyberaware.gov.uk/
On Sat, 16 Mar 2019 at 22:37, Bartosz Bielawski<EMAIL_ADDRESS>wrote:

So before you didn't use this method?
You can try to replace it with:
setPixel(x, y, value & 1 ? true: false);
value >>= 1;
But as you have noticed yourself - there was no change.
We can write it directly in the framebuffer but it shouldn't change
anything.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bartoszbielawski/LEDMatrixDriver/issues/31#issuecomment-473598366,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtJbzX1H562KQTuSzGwNFqyCzTxROymks5vXXIygaJpZM4b4A5P
.

Maybe there was an update of the Arduino IDE? You can always try to compile
it online using this Arduino Create.
Or some other editor - VSC + PlatformIO or something...

Updated arduino ide, didn't seem to update gcc though and made no
difference in the crash results.
I reverted back to my version of the driver which was working fine for
every compile and now I'm getting the same crash results.  How very odd.
I am going to guess the problem is at my end, why would I get intermitted
compiler results unless my ram is somehow corrupt?
Regards,
James Gohl
*Are you cyber streetwise?*https://www.cyberaware.gov.uk/
On Sat, 16 Mar 2019 at 22:54, Bartosz Bielawski<EMAIL_ADDRESS>wrote:

Maybe there was an update of the Arduino IDE? You can always try to compile
it online using this Arduino Create.
Or some other editor - VSC + PlatformIO or something...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bartoszbielawski/LEDMatrixDriver/issues/31#issuecomment-473599291,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtJb3CO-Lmif8Yaw2oQd_uvWtjIvumiks5vXXYzgaJpZM4b4A5P
.

I can't reproduce the crash after retarting my machine.

Learn about forward declarations. (writeNext() is called from loop() that is before writeNext()),

I know about forward declarations, yes anything that is called needs to be declared before, but gcc doesn't seem to care.

I'm pretty sure it's the Arduino editor that hides it from you. My compiler
(gcc for Tensilica Xtensa) certainly cares.
BB
niedz., 17 mar 2019 o 00:34 James Gohl<EMAIL_ADDRESS>napisał(a):

Learn about forward declarations. (writeNext() is called from
loop() that is before writeNext()),

I know about forward declarations, yes anything that is called needs to be
declared before, but gcc doesn't seem to care.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/bartoszbielawski/LEDMatrixDriver/issues/31#issuecomment-473601510,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD2sEJnr9fG6ACngUQzEMTtFjeUxGAt2ks5vXX-bgaJpZM4b4A5P
.


check realloc

File:
/libjxl-main/tools/jxlinfo.c: line 389
Code:
box_index += chunk_size - remaining;
box_data = realloc(box_data, box_size); // <-- check realloc

Use pull requests if you want to propose changes to the code base, this is a bug reporting page.

Thanks for the proposed changes, @maddin200!


OpenShift windows docker image could not access KVM kernel module

I created an Windows 10 image and I am able to run it locally inside the docker container.
When I test it in minishift I get the following error:
(process:27): GLib-WARNING **: 15:44:18.254: gmem.c:489: custom memory allocation vtable not supported
--
Could not access KVM kernel module: No such file or directory failed to initialize KVM: No such file or directory

Any ideas?

@emilorol Minishift should be running in Virtualbox which does not forward KVM. However e.g. VMWare forwards KVM and it should work. Not sure whether Minishift supports this.

I am using KVM instead of Virtualbox
$ minishift start
-- Starting profile 'minishift'
-- Check if deprecated options are used ... OK
-- Checking if https://github.com is reachable ... OK
-- Checking if requested OpenShift version 'v3.11.0' is valid ... OK
-- Checking if requested OpenShift version 'v3.11.0' is supported ... OK
-- Checking if requested hypervisor 'kvm' is supported on this platform ... OK
-- Checking if KVM driver is installed ...
   Driver is available at /usr/local/bin/docker-machine-driver-kvm ...
   Checking driver binary is executable ... OK
-- Checking if Libvirt is installed ... OK
-- Checking if Libvirt default network is present ... OK
-- Checking if Libvirt default network is active ... OK
-- Checking the ISO URL ... OK
-- Checking if provided oc flags are supported ... OK
-- Starting the OpenShift cluster using 'kvm' hypervisor ...
-- Starting Minishift VM ............... OK
-- Checking for IP address ... OK
-- Checking for nameservers ... OK
-- Checking if external host is reachable from the Minishift VM ...
   Pinging <IP_ADDRESS> ... OK
-- Checking HTTP connectivity from the VM ...
   Retrieving http://minishift.io/index.html ... OK
-- Checking if persistent storage volume is mounted ... OK
-- Checking available disk space ... 61% used OK
-- Writing current configuration for static assignment of IP address ... WARN
-- Mounting host folders
-- OpenShift cluster will be configured with ...
   Version: v3.11.0
-- Copying oc binary from the OpenShift container image to VM ... OK
-- Starting OpenShift cluster ...................

Related to https://github.com/dhiltgen/docker-machine-kvm/issues/36.

Yep, I did that still no luck. While the changes made the error go away, qemu keeps killing itself. The new error now is:
Waiting xvfb...
--
  | --- x11vnc loop: 1 ---
  | --- x11vnc loop: waiting for: 26
  | PORT=5900
  | (process:28): GLib-WARNING **: 13:49:17.054: gmem.c:489: custom memory allocation vtable not supported
  | /usr/bin/xvfb-run: line 181:    28 Killed                  DISPLAY=:$SERVERNUM XAUTHORITY=$AUTHFILE "$@" 2>&1
  | --- x11vnc loop: sleeping 500 ms ---
  | --- x11vnc loop: 2 ---
  | --- x11vnc loop: waiting for: 35
  | --- x11vnc loop: sleeping 500 ms ---
  | --- x11vnc loop: 3 ---

Any other idea?

@emilorol do you have sufficient memory allocated?

I have the config like this:
  "edge": {
    "default": "18.0",
    "versions": {
      "18.0": {
        "image": "nexus.internal.com:8443/microsoft/windows10:edge-18",
        "port": "4444",
        "path": "/wd/hub",
        "privileged": true,
        "resources": {
          "limits": {
            "cpu": "1.5",
            "memory": "2.5Gi"
          },
          "requests": {
            "cpu": "200m",
            "memory": "1Gi"
          }
        }
      }
    }
  },

But looking at the screen when the container runs I have the following:
browser
Image: microsoft/windows10
Ports: 4444/TCP, 5900/TCP
Mount: dshm → /dev/shm read-write
Mount: default-token-xb724 → /var/run/secrets/kubernetes.io/serviceaccount read-only
CPU: 500 millicores to 1 core
Memory: 512 MiB to 1 GiB

Is the memory not been allocated?

@emilorol seems to be. Are we speaking about Moon? If so could you provide more details about Moon version?

Yes, I am running Moon 1.3.8

@aandryashin any ideas?

What is host operating system? When container starting kernel loads kvm
modules, they should be the same on host and within container...

I am running on RedHat 7 and yes they are the same.

Try to rebuild windows image from ubuntu:16.04 or from RH... In that case
most probably kvm kernel modules will be compatible...
пт, 11 окт. 2019 г., 16:42 Emil Orol<EMAIL_ADDRESS>
I am running on RedHat 7 and yes they are the same.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aerokube/windows-images/issues/17?email_source=notifications&email_token=AAKY23NET2RKVN6PXFKVK3DQOB7E7A5CNFSM4I5E57GKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBABD5Q#issuecomment-541069814,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKY23MLMGOMTPXAEPNKINLQOB7E7ANCNFSM4I5E57GA
.

Yes, I tried that I builded the windows images on the same docker image and host from where I run them.

Still the same error?
сб, 12 окт. 2019 г., 17:56 Emil Orol<EMAIL_ADDRESS>
Yes, I tried that I builded the windows images on the same docker image
and host from where I run them.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aerokube/windows-images/issues/17?email_source=notifications&email_token=AAKY23P63FJB2VEJABDRB3LQOHQSPA5CNFSM4I5E57GKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBCBFRY#issuecomment-541332167,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKY23OEGIOBJADJUKZXEPDQOHQSPANCNFSM4I5E57GA
.

Going back to selenoid on a VM. Thank you for your support.


[HELP] GET current user login in Middleware

Hi guy, how can i get current user in Middleware
<?php

namespace App\Http\Middleware;

use Closure;

class CheckUploadImageBackend
{
    public function handle($request, Closure $next, $role= NULL)
    {
      dd(\Auth::guard('admin')->user());
        if (auth()->check() && auth()->user()->hasRole($role)) {
            return $next($request);
        }

        return redirect('login');
    }
}

Have you found any solution?
I like to update the config('filesystems.disks.local.root') as per login users.
For example:
filesystems.disks.local.root = public_path(\Auth::guard('admin')->user()->username);
How can I achieve that do you have any idea?


git svc show that all files are uncommited in wsl

VSCode Version: 1.39.2
OS Version: windows x64 1909
WSL:ubuntu

I clone a git repo form github wthin windows_git, and than I open this folder in vscode-remote  on WSL, git svc shows that all files are nucommited;

Do things work if you clone from within WSL?

Do things work if you clone from within WSL?

ok, I clone a repo in wsl, and it works well using vscode-remote

Alright, closing it then. That's the recommended way to do things.


Updating ticket status: notification e-mail has correct header but incorrect content

When updating the ticket status, the notification e-mail has the correct header but the incorrect content.  For example, if I close a ticket then the e-mail header says "Kebian updated My new ticket status to Complete" but the e-mail content says "Kebian changed the status of "My new ticket" from Pending to Pending"
This is caused by the default_reopen_status_id and default_close_status_id settings being seeded as false.  Shouldn't they instead be the following?
default_reopen_status_id = 1 (for Pending)
default_close_status_id = 2 (for Solved)

The complication is of course that the statuses can be changed by an administrator so 1 might not always be "Pending" and 2 might not always be "Solved" so I thought I'd ask for your thoughts before submitting a pull request.

I couldn't agree more


[SQL][DOCS] Update sql documentation

Updated examples using the new api and added DataFrame concept

Can one of the admins verify this patch?

Thanks. I've merged this.


Add docker workflow

Signed-off-by: Sayali Gaikawad<EMAIL_ADDRESS>Description
This PR adds docker workflow which will publish the customized jenkins image on dockerhub which will then be used to set up jenkins on an EC2 server.
Issues Resolved
Part of #106
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

LGTM


testing the run-in-postman button

Coverage remained the same at 100.0% when pulling a847fe5c78971afde0a8444f7fd66a9e1d405baf on postman-button into 7ab7a89424c47d22b9e0c2b5d3eaa30cc7242c61 on master.

Coverage remained the same at 100.0% when pulling 4aa44bfbefaf752336bc62e517f51ea0f465b754 on postman-button into 0d11398e05dd7e8e20c396cb0e400722ba696e24 on master.

Coverage remained the same at 100.0% when pulling 5bc4318d42ca5b4a5665e4a740d32c2f36496c05 on postman-button into d40e6050449f8c9996d66e79d49e7fbaba5d0c89 on master.

Coverage remained the same at 100.0% when pulling e0f7a74ddea447f3bff2c5403567ed88eb08a17b on postman-button into c897311a4ab54cfcdb6d37c0fcecb954e6d4c0c6 on master.

Coverage remained the same at 100.0% when pulling e7240edbdbc6e1a1ea38931522cf9dd866cc109d on postman-button into 17e61b5009123b8651fab22bbd6945d53149cfad on master.


Fix ESLint issues about too long lines

1. Why is this change necessary?
While rebuilding the administration with ./psh.phar administration:build, ESLint complained about lines exceeding the maximum line length of 125.
2. What does this change do, exactly?
I've choped down the too long lines.
3. Describe each step to reproduce the issue or behaviour.
Run ./psh.phar administration:build on the current master.  Please link to the relevant issues (if any).
➖
5. Which documentation changes (if any) need to be made because of this PR?
➖
6. Checklist

[X] I have written tests and verified that they fail without my change
[X] I have squashed any insignificant commits
[X] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I have read the contribution requirements and fulfil them.

thanks for your contribution @Crease29 :+1: :tada: :blue_heart:


Add entries of python module to use hidapi

Please add the following dependency to the rosdep database.
Package name:

python-hid
python3-hid
python-hidapi
python3-hidapi
python-hidapi-cffi
python3-hidapi-cffi

Package Upstream Source:
The names of these python modules are extremely confusing and are organized in the table below.

Index
Proposed rosdep name
Upstream Source
Name to import
Name in pypi
Name in github
Name in apt
Architecture

1
python[3]-hid
https://github.com/apmorton/pyhidapi
hid
hid
pyhidapi
-
ctypes

2
python[3]-hidapi
https://github.com/trezor/cython-hidapi
hid or hidraw
hidapi
cython-hidapi
python[3]-hid
Cython/cython3

3
python[3]-hidapi-cffi
https://github.com/jbaiter/hidapi-cffi
hidapi
hidapi-cffi
hidapi-cffi
python[3]-hidapi
cffi or python3-cffi

As shown in the table above, each library uses a different name depending on the context, and I proposes following naming rule for these python packages.
python-<PyPi name>

Purpose of using this:
These python modules provide interface to access hidapi.
You can work with Human Interface Devices such as mouses and keyboards, etc.
Distro packaging links:
Links to Distribution Packages

python[3]-hid

PyPi: https://pypi.org/project/hid/
Debian: https://packages.debian.org/

REQUIRED: not available

Ubuntu: https://packages.ubuntu.com/

REQUIRED: not available

python[3]-hidapi

Debian: https://packages.debian.org/en/buster/python3-hid, https://packages.debian.org/en/buster/python-hid

REQUIRED

Ubuntu: https://packages.ubuntu.com/focal/python3-hid

REQUIRED

python[3]-hidapi-cffi

Debian: https://packages.debian.org/en/buster/python3-hidapi, https://packages.debian.org/en/buster/python-hidapi

REQUIRED

Ubuntu: https://packages.ubuntu.com/focal/python3-hidapi

REQUIRED

How should we do? Should I remove rosdep entries for python2?

Yes, I would recommend removing the rosdep entry  for Python2. In fact, I would recommend generally contributing a single rule per pull request, that way these issues can be handled separately.

@emersonknapp Thank you for your approval, and sorry for my late comment.
I expect the tests will pass if it runs. Please approve of running the workflow.
Thank you.

@cottsay I committed your suggestion. Thank you.

@cottsay, Thank you for your reviews.
To tell the truth, what I really want to add is python-hidapi. So, at the moment, I decided to add only python-hidapi and python-hidapi-cffi.
I would be grateful if you would review my commit.


Upgrade puppetdb to 2.0.0

This commit upgrades puppetdb on precise machines to 2.0.0.
2.0.0 requires a newer version of the JDK than 6. 6 seems to be provided by the old puppetdb package so I've uninstalled it and manually installed 7.
I've also modified the upstart file to allow puppetdb to start - the change I've made here is similar to the change the package made to its own init file (which we purge).

I wonder if now's a good time for us to go back to using the SysV init script provided by the PuppetDB package, instead of providing our own Upstart script. It'd be less for us to maintain and we use SysV for a lot of other services.

LGTM, merge when ready :+1:

Actually I lied just now when I said it doesn't install a new JDK:
vagrant@puppetmaster-1:~$ apt-cache policy puppetdb | grep Installed
  Installed: 2.0.0-1puppetlabs1
vagrant@puppetmaster-1:~$ apt-cache depends puppetdb
puppetdb
 |Depends: <java7-runtime-headless>
    openjdk-7-jre-headless
    oracle-java7-installer
  Depends: <j2re1.7>
  Depends: adduser
 |Depends: puppet
  Depends: <puppet-agent>
  Suggests: postgresql

Still ok or shall we remove the explicit JDK installation?

Probably good to be explicit though I'm ambivalent.

If the package pulls the dependency I'd say it's better to not be explicit as it's another thing to potentially change in the future. Otherwise looks good :+1:

Are we moving to 2.0 as it's the last version compatible with Puppet 3.* ?

I didn't know that was the case, but picked 2.0 because it's a version with packages available for both 12.04 and 14.04 so we can upgrade the operating system independent of the package.

Going to close this until I've had a chance to test it more, I think it's breaking sometimes.


Add more integration tests to check Diktat compatibility with KtLint

Update: we already have Gradle plug-in tests available under diktat-gradle-plugin/src/functionalTest/,

broken with #1428,
re-enabled with #1579,

so the primary focus of this task is the Spotless plug-in (see below).
Diktat Gradle plug-in
Probably DiktatJavaExecTaskTest, feel free to suggest a different test class name.

Check that the plug-in is indeed executed and reports errors (i. e. ./gradlew diktatCheck) against invalid input should produce errors from diktat-ruleset namespace.

plain reporter
sarif reporter

Check that KtLint-standard rules are indeed excluded, i. e. none of the errors from the standard or experimental namespaces are triggered by the invalid input (be sure to also disable the corresponding Diktat rules via diktat-analysis.yml). This test should fail if changes from #1568 are temporarily rolled back.

Spotless Gradle plug-in

Check that diffplug/spotless, when run as a Gradle plug-in with the snapshot Diktat version, produces expected errors and has no regressions like #1559.

plain reporter
sarif reporter

Basically, we need a test which would, one way or another, launch a forked JVM against a test project, something we do in DiktatSmokeTest.
At the same time, I'd vote against using save-cli for this purpose and instead look for smth similar to khmarbaise/maven-it-extension for JUnit 5, for two reasons:

save-cli takes a long time to download over a slow network,
a forked JVM is much easier to debug in case of test failures.

@petertrr, @nulls, WDYT?

we need also test spotless maven plugin

Sounds reasonable; our functionalTest for Gradle plugin can be enhanced to run different scenarios as well as projects with spotless plugin using Gradle testkit.

Depends on https://github.com/saveourtool/save-cli/issues/521


[window function] support min max with self define sliding window and optimize segment tree .

Which issue does this PR close?

Closes #4603.
Rationale for this change
There are three kind of window:

RANGE BETWEEN UNBOUNDED PRECEDING AND UNBOUNDED FOLLOWING: compute once.
RANGE BETWEEN UNBOUNDED PRECEDING AND CURRENT ROW:  use Aggregator  compute cumulative state
ROWS BETWEEN 3 PRECEDING AND 3 FOLLOWING:  need Aggregator support remove sub-state, for now min_max agg func not support this remove operator

From paper For aggregate functions like min, max, count, and sum, the Segment Tree uses the obvious corresponding aggregate function.
So i try to use segment-tree to support this.
What changes are included in this PR?

Are these changes tested?

yes, also test result with spark-sql
Are there any user-facing changes?

@alamb  @mustafasrepo PTAL

@alamb @ @mustafasrepo PTAL

Thanks @Ted-Jiang for the contribution. I think in the long run breaking the update_batch and retract_batch API makes code hard to maintain. I wonder whether is there any way to hide SegmentTree implementation behind update_batch and retract_batch calls (I will experiment with it). Also your implementation stores all batch at the beginning and queries its result according to range. There is also a data structure in the discussion that supports push, pop, get (update, retract, evaluate in our case) with amortized O(1) complexity. I think this data structure is better for our use case in terms of complexity and support for incremental implementation. I have opened an issue previously discussing these points #4402. I am putting it here, in case it helps.

I plan to review this carefully tomorrow

I think in the long run breaking the update_batch and retract_batch API makes code hard to maintain.

Yes, I agree with this. Now window function with agg reuse the aggExpr, if we try to imply retract_batch for min max, i think it will also break a lot of api for avoiding effect the efficiency of non custom window frames situation. So I prefer keep this alone, is there any solution🤔

I think this data structure is better for our use case

Yes, you are right, from the paper why need stores all batch at the beginning  i think it will make inter-partition parallelism (but not in this pr 😂)

@berkaycpp is working on a PR to support  MIN, MAX with custom window frames. Thinking about how to utilize existing API lead us to use design in the discussion.  I think this PR will be ready in a couple of days (Main components are already there, it needs minor refactors).  I think implementation in this PR is better design for future (both in terms of algorithmic complexity[it is an amortized constant algorithm and uses finite memory] and readability). @alamb What do you think about this discussion?

We can factor in many dimensions when deciding the optimal data structure to maintain the aggregation state, that would be a nice discussion to have. (and I'm looking forward to it!)
However, I do think we should be conforming to the API and not special casing just for this. As the PR @mustafasrepo linked to shows, it is not hard at all to do this (the PR is very small). I see this is a case where there is no good reason to accumulate technical debt.

Supporting retract_batch on the existing min / max aggregators in https://github.com/apache/arrow-datafusion/blob/master/datafusion/physical-expr/src/aggregate/min_max.rs is probably a bad idea as it will slow down the non-window version
I wonder if it would be possible to make new accumulators that did support retract batch and use them instead of MinAccumulator and MaxAccumulator when operating on bounded window functions

Agreed, this makes sense. Planner can choose which version to use depending on the query.

I think implementation in this https://github.com/synnada-ai/arrow-datafusion/pull/29 is better design for future (both in terms of algorithmic complexity[it is an amortized constant algorithm and uses finite memory] and readability). @alamb What do you think about this discussion?

I don't have any strong opinion on the specific data structure used. My ideal workflow is:

Whoever contributes the code initially gets to decide on the initial algorithm
The initial PR includes sufficient tests that  ensure subsequent algorithmic improvements remain correct
We accept follow on PRs to improve the algorithms that show improved performance in benchmarks

Does that make sense?

It makes sense to me. I will check with @mustafasrepo tomorrow but I suspect he will agree 🙂 Good testing + reasonable initial algorithm is sufficient for a first implementation. We can go with whichever implementation that satisfies these baselines and improvements will always follow later.
On the other hand, I also think that one should avoid breaking a design pattern to implement incremental new functionality unless there is a good justification for doing so. In this case, I do not see a compelling reason to break the design, especially given that there are easy ways to do this within the given design. What do you think, @alamb?

@Ted-Jiang, would you like to collaborate with @mustafasrepo to explore how we can add a new MIN/MAX accumulator that conforms to the update/retract API? I would love to help you guys in the effort too.

@mustafasrepo Sure, You suggestions all make sense to me. As There's been a lot of work in https://github.com/synnada-ai/arrow-datafusion/pull/29. I will close this pr and wait for  and be glad to review it 😄.


File extension comparison treats some distinct extensions as equal

Bug Report

Q
A

Version(s)
HEAD (2.15.x)

Summary
Certain file extensions are considered equal by validators in some contexts.  For example, Laminas\Validator\File\ExcludeExtension#isValid will treat file extensions 10 and 010 as equal (example.10 and example.010 have the same file extension in its eyes).  This is a result of using == and in_array() without strict mode to check strings.  The following both return true in PHP:
'10' == '010';
in_array('10', ['010']);

Expected behavior
example.10 should never be considered to have the file extension 010.

https://github.com/laminas/laminas-validator/pull/104

Closed via #355


security/acme-client: DNS-01 nsupdate: Help for secret key should mention format

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[X] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[X] When the request is meant for an existing plugin, I've added its name to the title.

Is your feature request related to a problem? Please describe.
It was unclear for me what to enter in the text field "Secret Key" for the challenge type "DNS-01", DNS Service "nsupdate (RFC 2136)". The help text currently says:
Requires the whole key file in a format that is compatible with nsupdate.
I entered the content of the key file as produced by tsig-keygen resp. dnssec-keygen:
Private-key-format: v1.3
Algorithm: 165 (HMAC_SHA512)
Key: 7DXXX...
Bits: AAA=
Created:<PHONE_NUMBER>3656
Publish:<PHONE_NUMBER>3656
Activate:<PHONE_NUMBER>3656

...which did not work.
After lengthy researches I found that the form, which is used in named.conf to define keys, did work:
key "_acme-challenge.example.com." {
  algorithm hmac-sha512;
  secret "7DXXX...";
};

The help text did not help me much. That's why I am asking to change it.
I don't know whether the code could work with the contents of a key file from tsig-keygen (I think it would need the pair of files K<xxx>.key and K<xxx>.private), if yes, then this request to change the help text is almost obsolete and there should be some testing with that key file format or describe how to use it.
Describe the solution you'd like
Since nsupdate supports two formats for key files, the help text is not wrong, but could be more elaborate to help. Please change the help text to say content of the key file, not to confuse with a path to a key file. Secondly, for as long as it is unclear whether the key file format from  tsig-keygen does work or not, mention the format that works:
Enter the the key exactly as it is defined in named.conf of your bind server configuration, use the whole key{}-statement.
Please change this if needed, I am not a native speaker.
Describe alternatives you've considered
Alternatively:
Requires the whole content of the key file in a format that is compatible with nsupdate, which is one of two formats:  a single key statement as used in named.conf of the bind server software, which was generated by ddns-confgen; or the content of the key file generated by dnssec-keygen resp. tsig-keygen.
Additional context
It might be possible that even SIG(0) key pairs are supported. In this case the help text should mention this as well.

Yes, it's pretty cumbersome that one has to specify it in named.conf format. I'll consider an improvement for one of the next releases.

I don't feel this to be cumbersome: I have to add it anyway to the name server config, so it's easy to paste it to the plugin just afterwards. The only thing is: one has to know about it.
In case the one using the key and the one defining it on the server is not the same person, it looks different though. If you come up with a good solution, that's great, if you have other things to do first, changing the help text would be good for now. Thank you!

A better alternative might be adding another input field for nsupdate queries and splitting it into "key name" and "key"; I'm using Technitium DNS and it would be even more cumbersome for me to hand-craft an nsupdate-acceptable file from those two things I can cut&paste from it.


CPP: Make LambdaCaptures Elements

This is a fixed up part of something I was working on in 2017.  A LambdaCapture has a Location so there's no reason for it not to be an Element like everything else.
When someone is prepared to merge this I will write a (trivial) upgrade script for it.

LGTM, but can you elaborate on the script you plan to write?

can you elaborate on the script you plan to write?

By 'trivial', I mean that the upgrade script won't do anything, but it's required as the .dbscheme has changed and won't be considered compatible otherwise (see https://wiki.semmle.com/display/IN/Creating+an+ODASA+Database+Upgrade+Script for some background on this topic).  I don't want to write it until the last minute because, if somebody else changes the .dbscheme before it is merged, it would have to be redone.

I'm happy to merge this once there's an upgrade script. Remember that upgrade scripts go in this repo now. I've asked on Slack whether the internal repo also needs to be updated.

Added an upgrade script (https://git.semmle.com/Semmle/code/pull/33623) and updated stats (here).  I haven't done this for a while so I followed the steps on the wiki, hopefully I've got it right.

... pretty sure it's wrong.  I'll have to fix this another day.

Upgrade script moved to here and updated.  https://git.semmle.com/Semmle/code/pull/33623 should still be merged at the same time as it contains the LegacyLanguage.java change, which I assume is still needed.

Upgrade script is still valid.  Please review and merge.

Updated.


Training on Tesla K80

Hi,
Using Tesla K80 to train the model is giving the following error. Does the model require specific GPU architecture for training?
File "train.py", line 290, in
main()
File "train.py", line 50, in main
mp.spawn(run, nprocs=n_gpus, args=(n_gpus, hps,))
File "/anaconda/envs/vits/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 200, in spawn
return start_processes(fn, args, nprocs, join, daemon, start_method='spawn')
File "/anaconda/envs/vits/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 158, in start_processes
while not context.join():
File "/anaconda/envs/vits/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 119, in join
raise Exception(msg)
Exception:
-- Process 0 terminated with the following error:
Traceback (most recent call last):
File "/anaconda/envs/vits/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 20, in _wrap
fn(i, *args)
File "/media/hdd1tb/tts-VITS/vits-main/train.py", line 117, in run
train_and_evaluate(rank, epoch, hps, [net_g, net_d], [optim_g, optim_d], [scheduler_g, scheduler_d], scaler, [train_loader, eval_loader], logger, [writer, writer_eval])
File "/media/hdd1tb/tts-VITS/vits-main/train.py", line 162, in train_and_evaluate
hps.data.mel_fmax
File "/media/hdd1tb/tts-VITS/vits-main/mel_processing.py", line 105, in mel_spectrogram_torch
center=center, pad_mode='reflect', normalized=False, onesided=True)
File "/anaconda/envs/vits/lib/python3.7/site-packages/torch/functional.py", line 465, in stft
return _VF.stft(input, n_fft, hop_length, win_length, window, normalized, onesided)
RuntimeError: cuFFT doesn't support signals of half type with compute capability less than SM_53, but the device containing input half tensor only has SM_37

Did you try "fp16_run": false ?

You cannot train vits on a K80. K80's are a very weak GPU. You need at least a Tesla P100 or a T4 in order to avoid errors when training. A valid explanation for this is because K80's don't have enough memory for training.

You cannot train vits on a K80. K80's are a very weak GPU. You need at least a Tesla P100 or a T4 in order to avoid errors when training. A valid explanation for this is because K80's don't have enough memory for training.

That's not true. You can train on any GPU which supports cuda, but have to set fitting batch size. It reduce resulting quality, true, but it doesn't mean "you cannot train".


Custom font for label

can you please add opportunity for adding custom font for label. Right now i copied your TextInputLayout.js and add custom font manually. Thanks

I think is better to make pull request and write to zbtang in social network)


Mono: Add reload icon to refresh export variables in the editor

Godot version:
3.0/3.1
OS/device including version:
All
Issue description:
Currently sometimes the editor already doesn't consistently reload exported variables in the editor, but another issue is when dev's are building outside of the editor using msbuild. There's no way to reload the editor to see those changes
Suggest:
Give us a little reload icon to refresh the editor without exiting the editor to detect these changes on command.

@exts Please re-test with Godot 3.2.3 and/or the master branch.
If this issue does still exist for you, then you will need to include a minimal reproduction project.


Apparent mismatch in service name in anonymous authentication for hub rate limit preview

Folllowing the docs for how to preview your rate limit anonymously, cf.
File: docker-hub/download-rate-limit.md
getting a tocken with
TOKEN=$(curl -Lvs \
  "https://auth.docker.io/token?service=registry-1.docker.io&scope=repository:ratelimitpreview/test:pull" | \
   jq -r .token)

running the second request
curl -Lvs -H "Authorization: Bearer $TOKEN" https://registry-1.docker.io/v2/ratelimitpreview/test/manifests/latest

I get a the following respose in curl
< HTTP/1.1 401 Unauthorized
< Content-Type: application/json
< Docker-Distribution-Api-Version: registry/2.0
< Www-Authenticate: Bearer realm="https://auth.docker.io/token",service="registry.docker.io",scope="repository:ratelimitpreview/test:pull",error="invalid_token"
< Date: Thu, 29 Oct 2020 11:25:38 GMT
< Content-Length: 164
< Strict-Transport-Security: max-age=31536000

upon detailed inspection, even though I requested the host registry-1.docker.io, the server requires authentication for the service registry.docker.io.
After changing the first request to respect the requested service in the authentication challenge (thank god i've spent some hours fighting with these kind of challenges before), it worked:
```bash
TOKEN=$(curl -Lvs \
  "https://auth.docker.io/token?service=registry.docker.io&scope=repository:ratelimitpreview/test:pull" | \
   jq -r .token)
curl -Lvs -H "Authorization: Bearer $TOKEN" https://registry-1.docker.io/v2/ratelimitpreview/test/manifests/latest

So I suggest the docs be changed, more precisely the curl for the token should be adjusted by changing the service.
Interestingly, adjusting the host in the second request to registry.docker.io yielded

503 Service Unavailable

No server is available to handle this request.

I should add that I did not test the username and password based process.

fixed by https://github.com/docker/docker.github.io/pull/11656 (thanks for reporting!)


Finalize the Components - Overview page description and intro copy

@JennySanchez commented on Mon Aug 03 2020
User Story

As an IBM.com Library components adopter,

I need a simple but straight forward paragraph on the Components overview page list and the intro section of each component website page

so that I can learn about the components before I spend more time learning about the rest of the detail

Additional information

The current descriptions can be viewed in the Components - Overview page (Component list for Q3 update) BOX document. We need to make updates and make sure the content reflects the latest information.
The website stream will utilize the finalized content for website page publishing.

Acceptance criteria

[ ] Content is updated and finalized
[ ] Content is ready for the website stream to make updates to the overview page and implement for the Components section content publishing.

@oliviaflory FYI on this task issue existed based on our conversation last week.
@wonilsuhibm Thank you in advance for pairing up with Olivia for Components description and individual component intro section copy crafting.

@wonilsuhibm @oliviaflory Thank you for getting started on this task. As mention in the Slack conversation, I made additional comments for consideration and they are in green color text. Let me know if you have any questions.

Olivia will check with Jenny to see whether this issue can be closed or should be moved to sprint 20-17.


Service Look up error: Load balancer does not have available server for client

For full disclosure, I've raised this issue on SO but after a day, didn't get a single response. It may be too complicated for someone not knowing the ins and outs of Spring Cloud, so the chances that I'll ever get an answer is slim. I've now spent almost 2 days troubleshooting this, without any luck so far.
I won't repeat the question here but I'm posting a link to the SO question. If this is not the correct protocol around here, let me know and I'll be happy to elaborate.
http://stackoverflow.com/questions/34500174/spring-cloud-service-look-up-error-load-balancer-does-not-have-available-server

On further investigation, I found that if eureka.client.fetchRegistry is false, the various shuffle methods in Applications are not called and hence Applications.shuffleVirtualHostNameMap is never populated. This map is used later for look up in the method Applications.getInstancesByVirtualHostName that then fails.
I don't understand why a client would be forced to download the registry. They may choose to make the network trip each time or get delta when necessary.

fetching the registry is how eureka is able to tell ribbon (loadbalancer) about what servers are available.

Are you find a way to solve the issue?

@linjpg please ask on stack overflow as this issue is closed and the problem documented above.

May be the service provider's spring.application.name the value of the problem is too long, the value of the short enough to try


Rewritten TESTING.md

Description of the PR

Finally.

Testing

unit
integration
manual

- [ ] new unit tests created
- [ ] all unit tests pass

- [ ] new integration tests created
- [ ] all integration tests pass

- [ ] manual testing went well

Have you updated the TESTING.md or other relevant documentation?

[x] Yes
[x] Not yet. I will do it next.
[ ] Not relevant

This testing procedure was used in manual testing for Sprint 7 (#187). There is an open question on when the "missing Scala plugin" notification should be shown, and it's strongly related to #224.

I'm afraid to sound unpopular among @OlliKiljunen , but is there any chance to have screenshots for some "ASSERTION" stages (especially in the cases of the complex UI dialogs (like 6.x.))?

I'm afraid to sound unpopular among @OlliKiljunen , but is there any chance to have screenshots for some "ASSERTION" stages (especially in the cases of the complex UI dialogs (like 6.x.))?

I'll add some. :)
An issue with screenshots is that they become easily obsolete and, thus, are tedious to maintain. But I agree it would make testing easier if there were screenshots to compare with.

An issue with screenshots is that they become easily obsolete and, thus, are tedious to maintain. But I agree it would make testing easier if there were screenshots to compare with.

we would need them when we will be giving the plugin for testing to TA's (read soonish, but not explicitly now)


feat: merge formatters and linters

What this PR does / why we need it
This PR merges the linters and fmt scripts, mostly, together. They now share the same linters/<LANGUAGE>.sh script for bother linting and formatting.
Misc things:

Better non-terminal handling
When tool outputs information (in a supported terminal), don't rewrite the line
When errored, with no output, append the time then show the error
Fixes asdf plugin list issue

Jira ID
DT-2571 DT-2572

Notes for your reviewers

OK, I've tested this on bulkworkflow and authz and it works great. Reasonably confident this is good to merge post-release.


fix: Ignore changes to snapshot_identifier

Description
Ignore changes to snapshot_identifier. Fixes #270
Motivation and Context
snapshot_identifier is only used when creating the DB instance. However, if the parameter is changed post-creation it causes the DB instance to be deleted and re-created.
This is the solution recommended by Hashicorp: https://discuss.hashicorp.com/t/rds-restored-from-snapshot-is-destroyed-on-next-terraform-apply/3693/4
Breaking Changes
This change should not break anything, unless a user is relying on changing this parameter to trigger the creation of a DB instance.
How Has This Been Tested?
We are using this change in our private fork of this module.

Close/re-open to re-trigger tests

I don't understand why the "Semantic Pull Request" check is still failing. 🤷

I'd suggest having two aws_db_instance resources. One with ignore_changes, one without.
In some cases I want to trigger recreating db instance based on snapshot identifier.

hi @robinbowes - thanks for the PR. this is always something very tricky/difficult to work with in modules (lifecycle hooks). I think the safest approach is as @okgolove stated but that adds a significant amount of overhead and complexity for little gain - I'm not sure what we will be able to do here (hoping Hashicorp provide a variabilzed version of lifecycle hooks soon 🤞🏼)

closing this in favor of what is provided by the latest release of the upstream AWS provider https://github.com/hashicorp/terraform-provider-aws/pull/18013


openning buffers in new tabs causes error

While opening or changing buffers inside newly created tabs I'm getting the same error message over and over again:
Error detected while processing function 32_BEActivateBuffer:
line 18:
E684: list index out of range: 3
After pressing enter buffer opens, but it's really annoying. It only occurs inside newly created tabs.
Any ideas on what's  causing this and what to do about it?
Thank you very much.

OK, the problem is the bufexplorer plugin. I've removed it from bundles/rbtweaks/plugin  and life is perfect again.
Here's the article which brought me on the right track : http://vim.1045645.n5.nabble.com/Fwd-E684-error-when-changing-buffers-td4753709.html
Cheers!

I have fixed this in the 3.0 branch.


Language beta

Update README and flag Language as Beta.

Are there any other changes that need to be made to Natural Language? Or is the current release confirmed to be Beta?

Nope, we actually got the code to beta quality a couple of weeks ago code wise. I just forgot to communicate to update the documentation.

Right. Also, I remember you saying we were waiting until mid-April to declare it Beta. Glad to get this official.


✨ add twitch

from twitchio.ext import commands
class Bot(commands.Bot):
def __init__(self):
    # Initialise our Bot with our access token, prefix and a list of channels to join on boot...
    super().__init__(token='ACCESS_TOKEN', prefix='?', initial_channels=['...'])

async def event_ready(self):
    # We are logged in and ready to chat and use commands...
    print(f'Logged in as | {self.nick}')
    print(f'User id is | {self.user_id}')

@commands.command()
async def hello(self, ctx: commands.Context):
    # Send a hello back!
    await ctx.send(f'Hello {ctx.author.name}!')

bot = Bot()
bot.run()

Initial release and test done. more testing needed.


Widgets issue

Dev Home version
0.1701.597.0
Windows build number
10.0.22631.4169
Other software
OS Build Version: 10.0.22631.4169.amd64fre.ni_release.220506-1250
.NET Version: .NET 8.0.7
Steps to reproduce the bug
Just open Dev Home app
Expected result
Installed widgets displayed on dashboard
Actual result
No widgets displayed on dashboard, error message displayed instead
Included System Information
CPU: AMD Ryzen 7 5800H with Radeon Graphics
Physical Memory: 13.86GB (6.33GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
9932MartCliment.WingetUIWidgets_<IP_ADDRESS>_x64__g91dtg5srk15g (Widgets for UniGetUI (formerly WingetUI))
Microsoft.Windows.DevHome_0.1701.597.0_x64__8wekyb3d8bbwe (Core Widget Extension)
Microsoft.Windows.DevHome_0.1701.597.0_x64__8wekyb3d8bbwe (Hyper-V Extension)
Microsoft.Windows.DevHome_0.1701.597.0_x64__8wekyb3d8bbwe (Windows Subsystem for Linux Extension)
Microsoft.Windows.DevHome_0.1701.597.0_x64__8wekyb3d8bbwe (Git)
Microsoft.Windows.DevHomeGitHubExtension_0.1700.597.0_x64__8wekyb3d8bbwe (Dev Home GitHub Extension (Preview))
Widget Service:
MicrosoftWindows.Client.WebExperience_524.23700.10.0_x64__cw5n1h2txyewy

Duplicate of #2945
Please try this workaround while we work on the issue:
A workaround for this particular error is to go to the Windows Widget Board (keyboard shortcut: win+w) and click the + button to add a widget. You don't actually have to add one, but bringing up the dialog gets things started such that Dev Home should be back to normal.


Upgrade React Native to 0.26

Closes #1026. They're not kidding about those breaking changes!
Starting 0.26:

React API must be now required from react package

Fixes https://github.com/DoSomething/LetsDoThis-iOS/issues/1026#issuecomment-221123773 thanks to  https://github.com/facebook/react-native/issues/7697#issuecomment-221126318
@lkpttn To run the build, you'll need to update your local setup via npm update && pod install

Testing UserView and get a new warning for Profiles with an empty section:

May 24 08:22:17  DoSomething[3496] : Warning: In next release empty section headers will be rendered. In this release you can use 'enableEmptySections' flag to render empty section headers.


Do the best thing with webpack dynamic import commons chunk stuff

At the moment if two pages use the same massive library, each page chunk will have a copy of the library. This might not be bad, as it still makes the main page load fast, but I think there is a way it will do some cleverer logic in the future.
The discussion about it is https://github.com/webpack/webpack/issues/4392 and a WIP PR is https://github.com/webpack/webpack/pull/4255. Just will follow them and see how it goes.

I think the current behaviour is ok now I added an async commons chunk thing and we removed a few bloated deps.

@nicksellen could you elaborate for the rest of us struggling with this?

@wbern
This might be the relevant bit: https://github.com/yunity/karrot-frontend/blob/281420ebece873bdef37bfdffbd00a63d98fa332/build/webpack.prod.conf.js#L87
It joins some commonly used async imports into one file (0.<hash>.js): https://karrot-dev.foodsaving.world/bundlesize.html

Yes, indeed. The plugin I added (in addition to existing vendor and manifest CommonsChunkPlugin's) was:
new webpack.optimize.CommonsChunkPlugin({
    children: true,
    async: true,
    minChunks: 3,
}),

With: http://nicksellen.co.uk/upld/bundlesize-with-async.html
Without: http://nicksellen.co.uk/upld/bundlesize.html
I don't know what it will do when https://github.com/webpack/webpack/pull/4255/files is merged, but I am happy with current behaviour now.
Need to write some actual application code instead of just webpack configs ;)


[Bug Report] Default game of the Bot doesnt refresh correctly

Bug Description
In the config the game is set to default.
When the Bot is started fresh it will display "Type !help".
When you start a Song it will also correctly display the name but after the song is finished it will just stay empty.
Steps to Reproduce

Start Bot (Display "type !help")
Play song (Display Songname)
Wait for song to be over (Displays "")
(4. Restart Bot to get "Type !help" againt)

Expected Result
When no song is playing it should be set back to the default again
Debug Output
Windows Server 2022 21H2
Bot Version 0.3.9
Additional Info
No response
Checklist

[X] I have looked for information about this within the documentation
[X] I have searched for similar issues on the issues page
[X] I am running the latest version of the bot:

Hello,
I'm checking this. In order to reproduce this issue, it is important to set the following attribute on config file to true
songinstatus=true by default it is false.

Thank you :) The bot works very well in every other regard ^^


Reserved words in parameter names cause issues

In my project Artboard 1 and Artboard 2 are both similar. They both have 'Test' Text in centre but in Artboard 1, the parameters of Text namely 'TEXT PARAMETER' and 'COLOR PARAMETER' are typed to null. When I click on export all widgets it does not give any error message but only exports Artboard 2 and does not export Artboard 1 in which both parameters are set to null. This also happens when any one of the parameter is set to null. Though null cannot be set as parameter but plugin also does not show any error message. It should show error message namely 'Null cannot be set as parameter in TEXT PARAMETER or COLOR PARAMETER'.
Maybe my xd file could help so here is my xd file in zip in which this issue occurs:
XD to Flutter 2.zip

Ah, I see. Yes, I expect any reserved word will cause issues as a param name. The fix would be to assemble a list of reserved words in Dart, and then either throwing an error, or modifying the name - likely the latter to avoid a bunch of UI.

This would almost certainly be true for widget names as well.

Up till now I was able to see this error only with null. For this, I had checked undefined and many other similar one.

Fixed in 03b5227c07b32c160ddd611d774cdeebc10c496b


OAuth 2.0: Enable client_secret parameter to be omitted if the client secret is an empty string

[] Brief description of changes applied
This commit will enable this behavior for OAuth:
client_secret: The client MAY omit the parameter if the client secret is an empty string.

[] Any documentation on how to configure, test
The OAuth 2.0 Authorization Framework
https://tools.ietf.org/html/rfc6749#section-2.3.1
OAuth best practice
https://tools.ietf.org/html/rfc8252#section-8.5

[] Any possible limitations, side effects, etc

Allowing client secret to not be a must pass parameter seems to be a best practice for OAuth 2.0 that is implementation in Native Apps (see https://tools.ietf.org/html/rfc8252#section-8.5)
It is also written that the client MAY omit the parameter if the client secret is an empty string. Which is currently not implemented in CAS (from what I have tested) (see https://tools.ietf.org/html/rfc6749#section-2.3.1)
If I am not correct, please point it out, since I am not expert in OAuth 2.0.
Also, the following commit might not the only place need modification, just a start.

[] Reference any other pull requests that might be related.

 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.

Apart from the source code and @mmoayyed 's comment, the use case seems valid to me.
You should also sync the PR with the master.

Thank you @NgSekLong . I read over the spec and I think this is a valuable change. I merged the change out of band, and this will be included in RC4.


imrove the readabilty of wanning that have a "location prefix"

motivation: diagnostics with "location prefix" print that prefix before the diagnostics severity prefix (eg warning, errror) which is less clear
changes:

always print severity prefix first
improve handling of diagnostics with newline

@swift-ci please smoke test

CI issue should be addressed by https://github.com/apple/swift-package-manager/pull/4124

@swift-ci please smoke test macOS


Docker

@DrewRomanyk did you get the app to work locally? I'm getting a ECONNREFUSED error when attempting to connect with the db.

Works! 👍


Store symbol byte offset in IndexSymbol

Adds the offset field to the IndexSymbol struct. This lets code using swift::index functions skip any line/column -> offset reverse calculation if the specific byte offset location is desired.

@swift-ci smoke test and merge


Version 11 support

Currently I can’t get the Modul to work in version 11 of Foundry VTT

Please!!!

v11?

+1

Works fine for me...

I'll probably have time to look at it next weekend

the sub menu for opening links after clicking the original Icon is not working. I have two links set up similar to the way you have it in documentation but it's not launching the selection menu for links from the module.

are you in an active scene? it works fine for me when I'm in a scene. but when I'm not the submenu does not appear. But neither do the submenu's for the other controls.


Why does this Application try to make a remote access to a server?

Your application makes a remote access request to a server host which specifically mentions evildog1, it does it in the background - WITHOUT the permission of the user. It makes use of remote capability in Windows 10, WITHOUT the authorization of the owner.
I'm warning you, you better stop such activities otherwise I'm going to have to contact a few security researchers, @evildog1 can you explain?
There's background activity going on and it significantly slows down the user's PC, hogging up memory. I'm reporting both of you to Github.

Wat.
Apktool does not.

I assume you are using APK Easy Tool. This is probably update check, nothing more than that. I'm been using it for years without any malcious activity
You may have downloaded the software from the untrusted site then, it's your fault


#86 Updated CSS styles to use larger fonts

@jwcnewton - Merged! Can you add yourself to the Contributor List too?


Louis pull

Small updates:
Minimal_solver:

added clear_memory function for memory management

rtm_solver:

Updated Burdiscope to Refractometer

I think I have fixed with my most recent commit b -> r, let me know if you can see it as I am not sure if the pull request has updated to reflect this.

I think I have fixed with my most recent commit b -> r, let me know if you can see it as I am not sure if the pull request has updated to reflect this.

Yes, I can see the changes.


Add read access to settings in csv::WriterBuilder

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
While implementing CSV writing in DataFusion (see https://github.com/apache/arrow-datafusion/pull/7390/files), we would like to be able to check the value of has_headers before actually constructing a csv writer. However, the current API has no way to do read the current values (only modify them): https://docs.rs/arrow-csv/45.0.0/arrow_csv/writer/struct.WriterBuilder.html
Describe the solution you'd like
It would be nice to have read only access to the fields. Maybe something like
    let builder = WriterBuilder::new().has_headers(false);

    let has_headers = builder.get_has_headers()

It is somewhat unfortunate that the builder already uses has_headers to set the field names rather than with_has_headers
Describe alternatives you've considered
We can keep a copy of has_headers around as @devinjdangelo  has done in https://github.com/apache/arrow-datafusion/pull/7390/files
Additional context

Looks it is easy.

label_issue.py automatically added labels {'arrow'} from #4909


6x6 Sudoku Game using Vanilla JS

WebMasterLog Repository (Proposing new issue)
:red_circle: Project Title :  6x6 Sudoku Game
:red_circle: Aim :  Develop a Web game that allows user to play  6x6 Sudoku Game in Easy, Medium or Hard level.
:red_circle: Tech Stack Used : HTML, CSS, JS
:red_circle: Category :  Vanilla JS

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be all the main components.

Cover image of the application, if project will be deployed in future, with the name of the folder in Kebab-Case (hypen-separated).
Main files in formatted state so that code will be readable.
Other Images - In seprate folder named Images used to form the application.

Inside the Project folder, the README.md file must be filled up properly, with proper visualizations and descriptions.

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.

:white_check_mark: To be Mentioned while taking the issue :

Full name :  Surajit Maity
GitHub Profile Link : https://github.com/Surajit0573
What is your participant role?  JWOC contributor

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@Surajit0573 Assigned to you. Follow the contributing guidelines carefully.


Chunked transfer encoding on HTTP requests

Supersedes #775.

/cc @sigmavirus24

:+1: from me.

Minor style comments, rest looks good. Ping when ready. :) :custard:

Ping!


Explain why MQTT is a good fit for RSMP

Initial draft of why MQTT is a good fit for RSMP

I suggest splitting the text into somewhat smaller sections with headlines.

Asist mentioned the use-case of a TLC that needs to send status updates to two different authorities, which would imply to different brokers. Perhaps we should have some diagrams showing the most common use-cases (single supervisior, multiple supervisors, multiple authorities, site-to-site, supervisor-to-supervisor, etc)?


multilingual ?

Hi;
I tried to make this template multilingual but I'm not an Hugo expert
it is possible to make the template multilingual and/or if it's already the case to explain a little bit or it should be working ?

Thanks for taking the time to open an issue.
Currently we don't fully support multilingual sites. It should be possible as each fragment is it's own content file and using the Hugo specific split within the content directory should work, but we haven't fully tested this.
If you are testing this out, please provide feedback and we might be able to fix anything that comes up.
There is a feature tracking issue already up: #14

Some additional notes:
Content translation should work out of the box with the below technique as far as I can see.
Recommendation would be either directory based splitting: https://gohugo.io/content-management/multilingual/#translation-by-content-directory
Or file based: https://gohugo.io/content-management/multilingual/#translation-by-filename
We don't do anything special here so config translation should work as well.
For the config file translation take a look here: https://gohugo.io/content-management/multilingual/#configure-languages
We reuse menus for nav etc. So this should also be possible.
And for menus: https://gohugo.io/content-management/multilingual/#menus

I am also interested in i18n.
It found it hard to translate hugo sites in general because you have to do it a bit differently depending on the content types. ( config, file, content dir).
I wonder if you could add a more structured wrapper in syna so its easier for mortals to manage the translation ? Maybe in the make file would be a good start ? there you you could have formal wrappers for doing it so people are so so lost.
Maybe later we could add a web GUI for it to make it even easier for mortals.

In general with Syna is should already be a bit easier as almost everything that is user facing is a content file. So one can split it into en/, de/ etc. directories or specifically per file as far as I can see.
A web GUI is being thought about and might use netlify CMS as a first iteration.
Not sure, what specific makefile wrappers make sense. It might be really tooling specific so harder to implement on a general level. Basically any service/tool doing a per file translation should work.
That being said. It's not fully supported yet, but we would love to add it.
Contributions are always welcome as well ;)


black screen and reboot after installation

Please paste the output of the following command here: sudo edit-chroot -all

chronos@localhost / $ sudo edit-chroot -all
Password:
name: xenial
Enter encryption passphrase for xenial:
encrypted: yes, unlocked
Entering /mnt/stateful_partition/crouton/chroots/xenial...
crouton: version 1-20170619181610~master:980f090f
release: xenial
architecture: arm64
xmethod: xorg
targets: lxde
host: version 9460.60.0 (Official Build) stable-channel kevin
kernel: Linux localhost 4.4.52-08254-ge4cd28825ca4 #1 SMP PREEMPT Mon Jun 5 16:59:00 PDT 2017 aarch64 aarch64 aarch64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...
chronos@localhost / $
Please describe your issue:
black screen and reboot after installation
If known, describe the steps to reproduce the issue:

I have the same issue on my Samsung Chromebook Plus if I don't run it with xiwi.

Mostly likely dup of: https://github.com/dnschneid/crouton/issues/3195
Please track it there.


Does it support callback functions similar to langchain? For example, callbacks during tool calls

Does it support callback functions similar to langchain? For example, callbacks during tool calls

Hi, this feature is in progress. Callbacks for llm requests/responses/errors are already supported. Next steps are to implement callbacks for RAG and tools. Could you please provide more details about the required functionality? How do you plan to use this feature? Thanks!