miniz-sys makes x86_64-w64-mingw32-* freak out

Compiling a project that transitively depends on flate2. Got a university project to deliver by monday.
Doing it like this:
cargo rustc --release --target=x86_64-pc-windows-gnu -- -C linker=x86_64-w64-mingw32-gcc

Always works flawless. Not this time. Got linker error instead
  = note: collect2: error: ld returned 1 exit status

A scary one. Not even a bleept about an undefined reference or some such.
Instead turns out some elfs got into a miniz rlib:
$ x86_64-w64-mingw32-objdump  -t "path/target/x86_64-pc-windows-gnu/release/deps/libminiz_sys-f01b4045f82726c9.rlib"
In archive path/target/x86_64-pc-windows-gnu/release/deps/libminiz_sys-f01b4045f82726c9.rlib:

miniz_sys-f01b4045f82726c9.0.o:     file format pe-x86-64
<snip>
miniz.o:     file format elf64-x86-64

What probably happened is gcc crate using… gcc to compile the miniz.o, which is why it ended being an elf.
Making
$ echo $CC
x86_64-w64-mingw32-gcc

worked around the issue for the time being. Would be good to figure how to deal with this incompatibility.

Could you provide full gists of logs and such? It'd also be helpful to look at the output file generated by the miniz-sys crate. As a last ditch attempt you can also enable the zlib feature to use libz-sys instead which links to the system zlib bindings on MinGW I believe.
Right now miniz-sys uses gcc-rs which should handle this case automatically but the full logs will help debug this issue to see what's going on.

Right, so it turns out to be another case of me having $CC set to clang by default. Moving the rant to gcc-rs :)


Publish test results in appveyor jobs

Signed-off-by: oleg.hoefling<EMAIL_ADDRESS>

Codecov Report

Merging #150 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #150      +/-   ##
==========================================
+ Coverage   74.42%   74.44%   +0.01%
==========================================
  Files          12       12
  Lines        2057     2058       +1
==========================================
+ Hits         1531     1532       +1
  Misses        526      526

Impacted Files
Coverage Δ

src/main.c
77.45% <0.00%> (+0.22%)
:arrow_up:

Continue to review full report at Codecov.

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


chore: add more reviewers

What this PR does / why we need it:
Add more reviewers.
Release note:

None

/lgtm
/approve


License for ip-api.com is too restrictive

The current geo function uses "ip-api.com", which does not allow for non-commercial use. Assuming that someone is going to want to run Squyre in a commercial environment at some point, this is too restrictive - they would have to disable this function in order to avoid breaching the terms of service.
ipapi.com (note the missing hyphen) is used by some other tools like this, and has better terms of use. It does require signup/API key, but that is fine.
Suggest reworking the function to use ipapi.com instead of ip-api.com.

Resolved by changing the geo ip provider to ipapi.com. This function now requires an API key to function.


Selected text is not very visible

When selecting text, the highlight can be easily lost with the background. Could a different color or lighter shade of gray be used? In the following screenshot, the text isCustomerLoggedInEpic is selected.

I recently wrote the theme by hand instead of doing the usual automatic conversion from Sublime's theme… with that said, I'll take a look!
Thanks!

I love this theme, thanks for the quick fix!


Action __construct

Hi,
any suggestion
class Delete extends DestructiveAction
{

    protected $modelId;

    public function __construct($modelId)
    {
        $this->modelId = $modelId;
    }

How to use it with
$response->assertActionsInclude(Delete::class)

It will create error: Unresolvable dependency resolving [Parameter #0 [  $modelId ]] in class App\Nova\Actions\Delete
while
$response->assertActionsInclude(new Delete(1))

Creates:  Illegal offset type in isset or empty
any suggestion how to assert Nova Actions that have constructor

$response->assertActions(function($actions) {
    return $actions[0]->id == 1;
});

I would use the callback approach for that


terraform-providers: various

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Manually managing licences is a maintenance burden when it can be resolved by a simple upstream fix. Removing packages without a working spdx_id for now, they can be added back when they have been fixed upstream.
gitlab is reading the license from the "old" github repo, it's correct at the moment and hopefully it won't change. Adding proper support for providers hosted on gitlab isn't worth the hassle for one provider.

I pinged the few terraform provider that had no license at all.

Cloudscale's terraform provider was also fixed. Not released though: https://github.com/cloudscale-ch/terraform-provider-cloudscale/commit/6b8a3616ac6da1b364562584c90e7e9709e71da3

https://github.com/logicmonitor/terraform-provider-logicmonitor
https://github.com/ondrejsika/terraform-provider-vercel
These two don't have a license but it's a simple fix.
https://github.com/vmware/terraform-provider-nsxt
License but no spdx_id, the other vmware terraform repos do have a spdx_id so upstream should be fine with fixing it.
https://github.com/checkly/terraform-provider-checkly
Actually seems like the github api is broken for this one, it has a license but it's ignored? The repo is fork but it's doesn't actually point to the original repo. It was also added to nixpkgs this year so it's likely still being used, I've added an override for it as I doubt it can be fixed in the repo. Guess I'll open a ticket with github.
I'm planning to merge this in ~24hrs.


Could not decode query

test file: prosesittes.lua
error is coming from languagetool: Error: Could not decode query. Query length: 613 Request method: POST
comes from this_line

This means on the langtool side javas URLDecoder.decode call failed. -> we are not properly encoding the url?

fixed in: 7498ec2f077bbf3873825d898f11f6c2851e36e5


Premake modularization and customization

Describe the issue (if no issue has been made)
As of now, there is no easy way for a game to include the Hazel/Hazelnut/Sandbox projects into their game workspace other than copy pasting the premake project definitions over (and maintaining them as Hazel evolves). Splitting the projects up, similar to the dependencies of Hazel (GLFW, Glad and imgui) have their own premake file.
While implementing these premake changes, this PR also adds a .editorconfig file as suggested in #52 and implements the premake customization for this as well. This way PRs made to Hazel will have automatically the correct code layout, such that maintainers of Hazel don't have to correct these every time. This also means that contributors to Hazel will not have to change their own preferences.
PR impact (Make sure to add closing keywords)
List of related issues/PRs this will solve:

Impact
Issue/PR

Issues this solves
Resolves #242

Other PRs this solves
Closes #52

Proposed fix (Make sure you've read on how to contribute to Hazel)

This PR modularizes the premake files for Hazel, Hazelnut and Sandbox as described in #242. This way it is easy to include them in other (game project) workspaces that depend on Hazel.
This PR adds a .editorconfig file and adds it as a workspace scope included file such that any edits to the Hazel workspace will have the correct code layout as was suggested in #52.

Additional context
Tested the new premake files for VS2019 and tested compilation/running of Hazelnut and Sandbox.
Also tested the .editorconfig settings in various files in the different projects, and works as intended.

Still marked as a draft as I want to add https://github.com/TheCherno/Hazel/issues/241#issuecomment-670933630 to this as well. I do need some help with it...

Ready for review. Fixed last issues in https://github.com/TheCherno/Hazel/issues/241#issuecomment-670947871.


Adjust docs on select

Related to https://github.com/rust-lang-nursery/futures-rs/pull/1172, adding to select docs behavior when one stream ends.
Docs for zip specify this behavior, so no changes needed there.

LGTM aside from "or error" nit

Replaced with https://github.com/rust-lang-nursery/futures-rs/pull/1192

@dbcfd you might know this already, but just in case you don't: Instead of creating the new select-docs-2 branch, you could have force pushed to your existing select-docs branch to replace its content. PRs on GitHub automatically update whenever changes are made to their underlying branch.
Anyhow, thanks for your contribution! (Also, don't hesitate to provide further PRs if you want to change other stuff in the docs 😉)


Error: spawn EACCES

./hiddenwallet
Uncaught Exception...
Error: spawn EACCES
at exports._errnoException (util.js:1024:11)
at ChildProcess.spawn (internal/child_process.js:325:11)
at Object.exports.spawn (child_process.js:505:9)
at App.startApi (/home/user/Downloads/HiddenWallet-0.6.3-linux/resources/app.asar/main.js:49:31)
at emitTwo (events.js:130:20)
at App.emit (events.js:213:7)
This solved my problem:
sudo chmod -R a+rwx ./HiddenWallet-0.6.3-linux

Thanks. The front end was about to start the back end's daemon, but it didn't have enough access. In my ubuntu virtual machines it doesn't happen. I add sudo to the docs, hopefully nobody else will run into this.

If it's not too much to ask, could you try to download again and run with sudo ./hiddenwallet. If that does work, I'll leave that in the docs, because it's simpler, if that doesn't I'll add your sudo chmod -R a+rwx ./HiddenWallet-0.6.3-linux suggestion.

Isn't it more secure to use chmod instead of running a binary with sudo?

True. I was approaching it from a convenience point of view. Anyway I already added your chmod, because it turns out the sudo doesn't work for everyone. Though I left the sudo in it, too, just to be sure. I should educate myself from linux voodoo more.

The chmod is the solution, executables need the +x bit on Linux, otherwise trying to execute them gives a permission error.
Executing with `sudo won't help and even harmful, it escalates any potential vulnerability to a root exploit, as well as potentially cluttering root-owned files over your home directory. And even as root, executables needs the +x bit.

I removed sudo and kept chmod.

chmod -R a+rwx ./HiddenWallet-0.6.3-linux will make the whole directory writable by any local user. I recommend to remove this step from the documentation (on release page) and instead use chmod -R 700 <dir> untill we figure out which files needs to be executable.

YMMV, but here this was enough after unzipping:
chmod +x hiddenwallet resources/HiddenWallet.Daemon/bin/dist/current-target/HiddenWallet.Daemon

The only thing that I manually modify from the software inside the directory next to the executable is a TorData folder, which for some strange reason, if I can recall didn't work properly when I placed it into the actual data folder (%appdata%, "HOME" environment variable).
In fact the TorData is not manually modified by the software, the software starts a tor instance and the tor process will manually create and write into the TorData.
Anyway I will spend time on this issue after the testing is over.

There is a new version of the wallet, called Wallet Wasabi, that's a complete reimplementation of the wallet. GUI will be reimplemented in Avalonia, not Electron. This part of the codebase will not be reused.


Update reference to first UF3 paper (npj computational materials).

The README and pypi description have links to the arxiv paper. Now that the first UF3 paper has been accepted, the arXiv link should be replaced.
https://doi.org/10.1038/s41524-023-01092-7

This URL In the README appears to just point back to the repo; hovering over the link doesn't go where I'd expect (red box edited in):

The rST is probably mangled, I always forget how links are supposed to work in it: https://raw.githubusercontent.com/uf3/uf3/develop/readme.rst

Thank you for your comment. When I updated the page, I messed up. I will fix it.

pr #114 fixed this issue, closing!!


Expose HttpContext in Transmit Events

Package version
0.2.3
Describe the bug
For some reasons, we sometimes need to access the HttpContext in Transmit events to perform certains actions based on the current user, wich is currently not possible as we only have access to the randomly generated UUID.
Here is an example :
transmit.on('subscribe', async ({ channel, uid, auth }) => {
  const myChannel = await Channel.find(channel)
  if (myChannel) {
    transmit.broadcast(`channel/${channel}/join`, { user: auth.user })
    logger.info(`User ${auth.user.username} joined channel ${myChannel?.name}`)
  }
})
transmit.on('unsubscribe', async ({ channel, uid, auth }) => {
  const myChannel = await Channel.find(channel)
  if (myChannel) {
    transmit.broadcast(`channel/${channel}/leave`, { user: auth.user })
    logger.info(`User ${auth.user.username} leaved channel ${myChannel?.name}`)
  }
})

Reproduction repo
No response

Released in 0.3.0 if you want to try and give some feedback! 👍🏻


amq 7 broker does not have any addresses or topics configured

In amq broker documentation it is said (somewhere), that using the autocreate feature in production deployments is an antipattern.
All queues and topics used should be created in broker.xml configuration, which means that in the openshift broker deployment template, the alert topic needs to be created.

@jdanekrh if I understand correctly you mean mentioned the names here - https://github.com/jboss-container-images/jboss-amq-7-broker-openshift-image/blob/16108bf61f8d5403e11e412843259797410e99be/templates/amq-broker-74-basic.yaml#L191-L195
but this is only applicable only for Address and Queues names.

Yes, that's what I meant. @rvais how do I declare a topic in the broker config. There is AMQ_QUEUE, but not AMQ_TOPIC there!

There is no "TOPIC" thing per se. You set the queue as multicast type queue.

So I take my previous comment back ... AMQ_QUEUE creates ANYCAST and AMQ_ADDRESS creates MULTICAST address ...


Allocate IP addresses for pods running on Kubernetes Windows agents

This PR allocates IP addresses for Kubernetes Windows agents, bringing Windows to parity with Linux.

[x] Fix whitespace issues in winagent.t. (This is in its own commit.
[x] Allocate IP addresses for containers running on Windows agents.

I still need to install CNI plugin on Windows agents. That will be part of another PR due to external dependencies.

This change is

@ofiliz,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Any more comments?

This PR has been waiting for more than a month. Do you have any other comments? If not, I'd like to rebase this again, hopefully for the last time.

@anhowe @colemickens Can you take a look?

Does the outbound internet work with this solution?

@ofiliz I would like to understand better the fix or plans to fix around outbound internet #561

This PR has nothing to do with pod outbound internet connectivity. Completely separate. Let's not gate one on the other.

@ofiliz - if this fix goes in I'm worried we will get support questions on "why doesn't outbound internet not work with CNI for windows"?  Currently we say, CNI is not supported for Windows.  Let me ask around to see if it will be ok to say Outbound internet for CNI is not supported for Windows.

@anhowe - how do we want to proceed on this issue

I'm not comfortable merging in such a substantial change without test coverage. Perhaps we can have someone from Ex-Deis pick this branch up/rebase/add tests to free Onur up. Will defer to you, @anhowe.

Substantial change? It is 5 lines? Please read the description.

Substantial change? It is 5 lines? Please read the description.

It's nearly 200 lines of changes, according to the diff.

I can see that the actual code that was changed within the template is only 5 lines, but I can also see @seanknox's point about the file's whitespacing was changed in this PR. Is that necessary or can that be pulled out separately?

I missed this comment explaining the rationale. Don't have any handle on how acs-engine works but from a Go template point of view, this looks good to me. Don't consider my naive experience as an official LGTM, though.

@acs-bot test this please

@acs-bot test this please


Package 3.0 components

All 3.0 components need to be packaged and distributed through OBS.
TBD: Do we want a new OBS repo for them, or do we want to overwrite the exisiting one ?

We want a new repo. We're already versioning them, so we can just add a 3.0 one next to the others.

@sameo @dlespiau could you please tag the components in github.com/clearlinux (runtime, shim, proxy to enable packaging @ OBS. 3.0.0-alpha0 might work

@sameo @dlespiau Nevermind, I will use my own clones and see whtat's the best approach for taggging

@gorozco1 Any updates on that one ?

@gorozco1 is this complete?

@amyleeland yes, sorry for not closing weeks ago completed
http://download.opensuse.org/repositories/home:/clearcontainers:/clear-containers-3/


[Backport] [2.x] Bump com.networknt:json-schema-validator from 1.0.78 to 1.0.81 (#7460)

Backport of https://github.com/opensearch-project/OpenSearch/pull/7460 to 2.x

Closed by accident, unable to reopen so here is the replacement https://github.com/opensearch-project/OpenSearch/pull/7483


examples/ext4dist.bpf.c seems to have too few entries allowed in the latency map

examples/ext4dist.bpf.c appears to specify the maximum capacity of the ext4_latency_seconds hash map as MAX_LATENCY_SLOT + 1 entries. However, the code seems like it will try to store up to that many separate bucket entries for each separate file operation (so MAX_LATENCY_SLOT entries for reads, and again for writes, and so on). This seems like it doesn't fit in the maximum entries, and I believe it explains some oddities I saw in actual use that went away when I updated this to multiply the limit by the number of operations. Since I'm inexperienced with eBPF programming, I may be missing something here.
If this is a real bug, it seems likely that this lack of accounting for multiple operation types may exist in other examples as well. In some cases, such as biolatency.bpf.c, this may be normally camouflaged by the fact that few people have anywhere near 255 disks, so the extra space for unused disks compensates for the lack of accounting for the various operations for most people (with modest numbers of disks).

You are right. Would you like to make a PR to fix this?
cc @alebsys

Unfortunately I feel I don't know what the right fix for this would be. Define a new enum element at the end, called maybe F_MAX, and multiply by that in the max_entries definition?
(Fixing things like biolatency.bpf.c looks even trickier since the possible req_ops values are non-contiguous, with REQ_OP_LAST set to 36 but only 14 REQ_OP_* values currently defined.)

Define a new enum element at the end, called maybe F_MAX, and multiply by that in the max_entries definition?

That's what I would do.
For biolatency you can add a new define in ebpf code itself with the number of ops we actually use.


Can not install it on React 17.0.2

Hi there! Great work, although I cannot install it on React 17.0.2. It search for React ^16.12.0

Hey, please attach the error message

Same on React 18.2.0
warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>


full version of that broken merge (it was mistake)

Anyone wondering why there was a giant merge with lots of broken features? well uh... he  tried to help me, and accidentally merged (oh no)

note: the readme screenshot 5 is bugged, it is looking for the screenshot url for this version of the branch, it should be fixed for both of us on merge

i have been telling him to merge it for the past few days, he said yes, and still hasnt? at least i have time to add more stuff


Make SingleConfigurableTaskTest.testIniCommentsAtEnd() pass on Windows

IzPack project » izpack #728 SUCCESS
This pull request looks good
(what's this?)

Is this the same like https://jira.codehaus.org/browse/IZPACK-1224 ?
Do you have a piece of log what failed?

Hi
I'm not sure. I see spaces in the path of IZPACK-1224 that might lead to the FileNotFoundException.
In my case this is the failing output. I think the issue is the empty line after # country mapping in the to.ini.

+++ C:\Users\admin\AppData\Local\Temp\junit691123585348814144\to.ini +++
[section1]
# country mapping

lang.1.key=de
lang.2.key=en
lang.3.key=it

# additional countries
#lang.4.key=at
#lang.5.key=cs

+++ D:\git\izpack\izpack-util\target\test-classes\com\izforge\izpack\util\config\expected_after_merge.ini +++
[section1]
# country mapping
lang.1.key=de
lang.2.key=en
lang.3.key=it

# additional countries
#lang.4.key=at
#lang.5.key=cs

Whit the change in the PR the empty line is not in the log and the test pass.

IzPack project » izpack #730 SUCCESS
This pull request looks good
(what's this?)

Thanks for contributing!


Make it possible to extract the underlying operations

It'd be nice to be able to extract the exact shift/multiply/add/negate operations from the types here, instead of only being able to do arithmetic without the ability to peer inside.
One use case for this would be when you're writing a compiler and you want to emit code where the dividend is not known.

I am considering eventually adding more algorithms, and having the crate select the best implementation for a given platform using cfg attributes, so I would rather have QuickDiv keep everything private for now. However, it'd probably make sense then to split it into two crates, one with the current interface, and the other containing the actual algorithms with everything fully public.


[BUG] It cannot catch and listen for "mouseleave" and "mouseenter" events.

[BUG] It cannot catch and listen for "mouseleave" and "mouseenter" events.

As mentioned on this issue #7, some events cannot be caught by events.js#L105. Because the window to which the event listener is attached does not need to listen to it and does not trigger any event.
The following approach can be used to fix this problem,

// If no event of this type yet, setup
if (!activeEvents[type]) {
   activeEvents[type] = [];

   ['mouseleave','mouseenter'].indexOf(type) >= 0 ?
   document.addEventListener(type, eventHandler, true) :
   window.addEventListener(type, eventHandler, true);
}

Note: Some events such as onstorage only works on window. For this, we cannot set all of them as document.

Ok now I have tested this approach and I can confirm it works.

If you have a better and simpler approach like #8, please let me know.
Other events that might exhibit typical behavior such as mouseleave and mouseenter should be examined individually and a list made. (So they can be attached to document instead of window)

Why was this closed?

@cferdinandi
I have a new idea about this and this problem is out of date. I added a new issue.


Where is the source code?

In the old site, each hook had its source code in full display, which was great.
With this new site, the source code seems to be nowhere to be found. Instead, there is a small demo and a lot of promotional material.
Even the links in the github readme just link back to the website, and do not demonstrate the source for any of the hooks?
How do I lookup the actual source code for the hooks? Why was this made so difficult to find?

It's in index.js file

It's in index.js file


raft: fix raft node start bug

raft node should set initial prev hard state to empty.
Or it will not send the first hard coded state to application
until the state changes again.
This commit fixes the issue. It introduce a small overhead, that
the same state might send to application twice when restarting.
But this is fine.
Fix #2877
/cc @bdarnell Does multiNode have the same issue?

lgtm

I think MultiNode has the same bug (https://github.com/coreos/etcd/blob/de1c9c08e1d7c5e91c70581ced9d989b6e18fc17/raft/multinode.go#L216), but the bug doesn't occur with the way we use MultiNode in Cockroach (We never start a node from log index 0; we always start with a snapshot with a non-zero log index).

@bdarnell OK. I will try to "fix" multiNode in a follow up pr.

So with 2.0.11, what am I doing to cause it? Is there some workaround or way I can make sure I don't hit this.  (I'm hitting it every single time in my CI system)

@eparis Do no shutdown the etcd member too fast (wait for at least a second).

thank you!  /me cusses at ansible.

@eparis Let me know if you still hit this. It is still the case, then it might be another bug.

I can confirm with my CI system when I stopped my playbook from doing

start etcd for the very first time
immediately stop etcd
immediately start etcd again

with 2.0.11 things seem to be working okay. I'll try to write a new test that does the tight create/restart with code from HEAD.


test-branch/execute-job/comment-from-wrong-user-on-pr/26082941-1132-460d-bd23-36621461819c

PR for testing d16e5e03f4ccb6e084221a0e57265ba2cc6b5055

/canonical/self-hosted-runners/run-workflows d16e5e03f4ccb6e084221a0e57265ba2cc6b5055


Neovim Support

Describe the feature
It would be awesome to get Neovim support for million lint. It seems like the most common LSPs are coc.nvim and neovim-lsp.
Additional information

[ ] Would you be willing to help implement this feature?

I also wish for this

@hexcowboy Hey it's on the roadmap! Right now we're heads down stabilizing the VSCode extension


Quit application with lowercase Q

Hi!
First of all thanks for creating and maintaining this project. I use this for my Sway setup and it works great, even better than the X11 GUIs in my opinion!
I was just wondering if it's possible to set the exit/quit keybinding to lowercase Q instead of capital Q because it would be more consistent with other TUI applications like pulsemixer.
Thanks for considering!
 I'm sorry, I should've checked the documentation first. Usually I do that, but I didn't do it this time :sweat_smile:
It's great that you allow custom keybindings, that's amazing!
And thank you for the quick reply, very much appreciated!
I will jut close this issue then. Thanks again for your wonderful work :grin:


Waiting cache populated takes forever

Hi,
Sometimes, it takes forever to wait for cache populated. I have to restart Neovim to make it work from time to time.

Moreover, how to make Neuron start caching when I first open Neovim. At this moment, I have to gzr everytime I open my zettel note.

I figured it out. All I need is cd to my note directory and open Neovim from there. I normally use Startify plugin to quickly open my note. And this way doesn't work with Neuron.

We should address this.

I'm having the same issue. But for me, it never ends.
May very well be related to https://github.com/fiatjaf/neuron.vim/issues/22 - which might only be a problem on my system!

I'll have a look at this one too since I am seeing it - I think the caching "issue" should probably be non-blocking and maybe it should only use it once it is available and fallback to a live query when not ready. Maybe also a setting to turn it off all together.

The problem is that neuron.vim should be still prepopulating the cache whenever one opens any Zettel, unless this configuration is missing something, which is very likely given my inexperience with Vim plugins:
https://github.com/fiatjaf/neuron.vim/blob/16464a8831bcaec0bd1f274b653802cb4036a8f3/ftdetect/neuron.vim#L15-L19
@babygau's issue with opening from different directories shouldn't be a concern, I don't understand it.

Actually these parts might be relevant:
https://github.com/fiatjaf/neuron.vim/blob/16464a8831bcaec0bd1f274b653802cb4036a8f3/ftdetect/neuron.vim#L2-L12
I can't simulate any situation in which fnamemodify(expand("%:p") wouldn't give me the correct path, but maybe in your system that happens when you open zettels from other places. The solution would be to manually set g:neuron_dir.

cool and thanks for the lead - I'll come back with what I find 👍

Probably relevant for my case: I often open the directory, which starts in Explore mode - showing the file tree - so when I gzz I don't technically have any zettel open.

@vcavallo fnamemodify(expand("%:p") doesn't work on these conditions. We must find something that gives us the current directory so we can identify it as part of the Zettelkasten.

I have fixed one problem so far. In my case it was because my path had a space in it - needed to run the dir through fnameescape. @vcavallo when you run vim . is that inside your zettelkasten directory? Also, do you have g:neuron_dir set in your vimrc or not?

Ok, I have created a PR for this: https://github.com/fiatjaf/neuron.vim/pull/27

I believe this has been addressed, let me know if you still have issues.

Hi ! I also have the message waiting until chache is populated...and then nothing happens.
I haven't quite understood what's the matter here...

Hi @jrekier thanks for that - could you tell me the following:

how did you start neovim / vim (did you open a file or no file)
which directory were you in, were you in the zettelkasten directory?
what is the full path name for you directory?

Hi @fenetikm, thanks for your comment. Here are the answers to your questions:

I tried both: opening a file or opening neovim on its own. The bug is there regardless
again, tried from both inside and outside my zettelkasten directory
quite simply, this is: /Users/jerem/zettelkasten

Hope it's helpful ! Thanks for your help !

Hi @jrekier thanks for that - just wanted to check that you are on the latest version of the plugin too?
Could you please give me a list of steps that you did. e.g.

Opened file
Moved cursor over zettel ([[abcd1234]])
Typed gzo

Thanks.

Hi @fenetikm
I think I am using the latest version. I have just moved to this fork from the original plugin yesterday. I started off by cleaning my list of plugins and installed the latest one. Here is a screeshot of where I'm standing after PlugUpdate

The bug happens after I do the following:

open terminal and type in neuron search -e, select and open a zettel
then typing gzz anywhere produces the following message waiting until chache is populated...which stays there and nothing happens.

alternative to 1. : the bug also happens when I open a zettel directly (i.e. not using the neuron command), e.g. nvim vi 03420c42.md will open a zettel in neovim and then step 2. proceeds with the same bug.
Here is the content of my /Users/jerem/zettelkasten/neuron.dhall:
{ siteTitle =
    "My Zettelkasten"
, theme =
    "red"
}

I am missing something there ?

Thanks for the details - will see if I can reproduce what you are seeing and let you know.

@jrekier oh hang on, which version of neuron are you running? (run neuron --version) - I am on <IP_ADDRESS>.

Oh my! I'm on <IP_ADDRESS> ...
I just did a git pull on the neuron directory and this did not fix it! I must be doing something wrong with nix. How can I update to the latest version ? (sorry for the trouble !)

Ahh hopefully that is it. Have a look at the installation page that also mentions how to upgrade it.
nix-env -if https://github.com/srid/neuron/archive/master.tar.gz

yes!! That seems to have fixed it! Just checking the other commands atm and everything seems to fall into place.
Thank you so much for your help @fenetikm !

No probs at all - finding this and what happened to you may help others. Cheers.

Not sure if it is helpful, or another bug - I found that having this in my vimrc would have the unwanted effect:
let g:neuron_dir="~/Documents/zettle/"

By removing it, I'm able to use gzz and it is almost instant. I need to be in the zettel directory however. I figure I need declare the path in another way.

Thanks @craigdallimore - yeah was just about to ask what the complete path was and if that would work. Cheers.

I also wanted to add, that setting the neuron_dir path in my config had me stuck with the waiting until cache is populated... prompt when opening vim outside the zettelkasten directory and runnging gzz
Making the following change fixed the problem for me
let g:neuron_dir=expand("~/Documents/zettle")
to
let g:neuron_dir=expand("~/Documents/zettle/")
So make sure that:

you have a neuron.dhall in your zettelkasten directory
have your zettelkasten directory set in your vimrc
that the path should contain a trailing slash like let g:neuron_dir=expand("~/zettelkasten/")


RC-3.0 Feature - method for manually setting empty states

Added shouldShowEmptyState method to donut and stacked-area APIs to set the empty state manually (even if data is given) to see the empty states as described in https://github.com/eventbrite/britecharts/issues/483
Description
Empty state will be known using shouldShowEmptyState. For the donut chart, it will show an empty slice with no info for the centered text (we can switch to make it show 0% if needed).
Update: changed logic in drawLegend to show 0% if shouldShowEmptyState is true
Motivation and Context
https://github.com/eventbrite/britecharts/issues/483
How Has This Been Tested?
Added two new setter/getter, one for each chart.
Screenshots (if appropriate):
donut.sholdShowEmptyState(true):

stackedArea.sholdShowEmptyState(true):

Types of changes

[ ] Refactor (changes the way we code something without changing its functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style guide of this project.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Merged into https://github.com/eventbrite/britecharts/tree/release/3.0.0-rc1
Thanks a lot!


bump openwhisk-client-js to 3.21.3

The build is failing due to what seems like an unrelated issue: https://travis-ci.com/github/apache/openwhisk-wskdebug/jobs/428968755
But we should fix nonetheless.
My guess is that a newer circleci/node image was published with changes where a certain path is missing or different.
We might be able to fix by specifying a fixed, older version of the base image here: https://github.com/apache/openwhisk-wskdebug/blob/master/test/install/Dockerfile

I had to do something similar in openwhisk-runtime-nodejs (switch to node:stretch-lts instead of node:stretch) to sidestep picking up node 15.1.


SA1015 false positive on object initializer

https://github.com/n0099/open-tbm/commit/b29fea0747d7ce3bce5e2000429fd4a417d4655e#diff-ca28356cfa73fff3341e8571881b0401158b49858f07d83d30e1ece640b15fe2R19
protected override Lazy<Dictionary<Type, AddSplitRevisionsDelegate>>
    AddSplitRevisionsDelegatesKeyByEntityType =>
    _addSplitRevisionsDelegatesKeyByEntityType ??= new(() => new()
    {
        {typeof(ReplyRevision.SplitFloor), AddSplitRevisions<ReplyRevision.SplitFloor>},
        // SA1015 Closing generic bracket should be followed by a space              ^
        {typeof(ReplyRevision.SplitSubReplyCount), AddSplitRevisions<ReplyRevision.SplitSubReplyCount>},
        // SA1015 Closing generic bracket should be followed by a space                              ^
        {typeof(ReplyRevision.SplitAgreeCount), AddSplitRevisions<ReplyRevision.SplitAgreeCount>}
        // SA1015 Closing generic bracket should be followed by a space                        ^
    });

Just to clarify: Normally, those three lines would also get a SA1012 since they don't have a space after the opening brace and a SA1013 since they don't have a space before the closing brace. I assume that you have intentionally turned both of them off, since you only mention SA1015?

I assume that you have intentionally turned both of them off

Yes: https://github.com/n0099/open-tbm/commit/b2910ff273366680c028dbef3f3065dfb58abc3a#diff-dee8b4f6b469005cc91d8562bba59ac8c3b8b151d08f85ed8f8b613ace4132ecR23

Ok, well I agree with you. I also think this is a bug.


refactor(GUI): turn the update notifier modal into a native dialog

Electron v1.6.1 introduced checkbox support to the native message
dialog, giving us everything that was needed to implement the update
notifier modal using a native dialog.
This change allows us to get rid of a lot code.

See: https://github.com/electron/electron/pull/8590
Change-Type: patch
Changelog-Entry: Turn the update notifier modal into a native dialog.
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

Lol, I left my comment before noticing Juan had already requested a view from Benedict ;-)

@Shou Let me know how it looks now!

Hmmm, what happens if the user actually ticks "Remind me again in 7 days" but then clicks "Download" anyway? Perhaps we should only update lastSleptUpdateNotify if (results.sleepUpdateCheck && !results.agreed) ?
And I wonder if installing a new Etcher version should automatically 'reset' the lastSleptUpdateNotify too? (which I guess we could do by saving the version of Etcher used to do the last update-check)

Hmmm, what happens if the user actually ticks "Remind me again in 7 days" but then clicks "Download" anyway? Perhaps we should only update lastSleptUpdateNotify if (results.sleepUpdateCheck && !results.agreed) ?

Sounds reasonable!

And I wonder if installing a new Etcher version should automatically 'reset' the lastSleptUpdateNotify too? (which I guess we could do by saving the version of Etcher used to do the last update-check)

Interesting. Let's store the version in the settings, and use it in lib/gui/app.js to set ignoreSleepUpdateCheck if so.

Interesting. Let's store the version in the settings, and use it in lib/gui/app.js to set ignoreSleepUpdateCheck if so.

Are you planning to make that change in this PR, or in a separate PR?

I just installed rc5 from the bintray apt-get, and noticed that the settings screen still has the "Include unstable update channel", which looks a bit odd. Perhaps if ETCHER_DISABLE_UPDATES is set (as it is for apt-get installed versions of Etcher), then that setting should be automatically hidden?

Are you planning to make that change in this PR, or in a separate PR?

I just pushed this change :)

I just installed rc5 from the bintray apt-get, and noticed that the settings screen still has the "Include unstable update channel", which looks a bit odd. Perhaps if ETCHER_DISABLE_UPDATES is set (as it is for apt-get installed versions of Etcher), then that setting should be automatically hidden?

Done!

@Shou @lurch How do you feel about this?

I still want to add some comments to this, but just busy looking at the filename-escaping issues ATM :-/

Re-ping @lurch @Shou


DPL-947 - As a long read user (Craig) I need the DNA control complex box barcode to be required on a v12 Sequel IIe run and included on the sample sheet because it is required for SMRT LInk and we are currently having to enter it manually

User story
As a long read user (Craig) I need the DNA control complex box barcode to be required on a v12 Sequel IIe run and included on the sample sheet because it is required for SMRT LInk and we are currently having to enter it manually
Who are the primary contacts for this story
Craig / Steve
Who is the nominated tester for UAT
Craig / Long Read
Acceptance criteria
To be considered successful the solution must allow:

[ ] When a new v12 Sequel IIe run is created a DNA control complex box barcode can be added
[ ] When a new v12 Sequel IIe run is created a DNA control complex box barcode should be required
[ ] When a new v12 Sequel IIe sample sheet is created it should have a DNA control complex barcode
[ ] SMRT Link Version v11 should be made inactive in the rake task and database for UAT and production
[ ] Sample sheets should be sorted by plate and then wells [linked to another issue but so trivial not worth doing as a new story]

Dependencies
This story is blocked by the following dependencies:

#<issue_no.>
sanger/#<issue_no.>

References
This story has a non-blocking relationship with:

#<issue_no.>
sanger/#<issue_no.>

Additional context
Add any other context or screenshots about the feature request here.

Closed as done.


argo for constructors

Hi,
I would like to ask, if it possible to use argo for class constructors? I know boost parameter has some work-around for it, but I'm not so sure how to do it for argo.
I am trying to write a wrapper class for an interface to Fortran. For example, I have an object called pgrid, which in Fortran is created by
CALL dbcsr_t_pgrid_create(fcomm, dims, pgrid, map1_2d, map2_2d, nsplit, dimsplit)
and most arguments are optional, so I would ideally like to write a nice constructor for a class pgrid  with named parameters.

I don't see why not but I'd have to think about it.
You should just make the constructor the facade function and then
everything should just work...
I'll give it a think over the weekend
On Fri, Nov 29, 2019, 10:47 Maximilien Ambroise<EMAIL_ADDRESS>wrote:

Hi,
I would like to ask, if it possible to use argo for class constructors? I
know boost parameter has some work-around for it, but I'm not so sure how
to do it for argo.
I am trying to write a wrapper class for an interface to Fortran. For
example, I have an object called pgrid, which in Fortran is created by
CALL dbcsr_t_pgrid_create(fcomm, dims, pgrid, map1_2d, map2_2d, nsplit,
dimsplit)
and most arguments are optional, so I would ideally like to write a nice
constructor for a class pgrid with named parameters.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rmpowell77/LIAW_2017_param/issues/6?email_source=notifications&email_token=AAA5R5J6RRTDHHG7HGEPMRLQWDXKHA5CNFSM4JS56WZKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4H43NRJQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA5R5J3QY2VR67WG3A3W2TQWDXKHANCNFSM4JS56WZA
.


Provide a way to integrate/configure googleapis/code-suggester so openrewrite diff is displayed as GitHub PR suggestions

What problem are you trying to solve?
It would be nice if OpenRewrite suggestions could be displayed as PR suggestions in GitHub UI.
Describe the solution you'd like

"check style workflow" executes OpenRewrite, produces a diff/patch/whatever file as workflow artifact
A separate on: workflow_run: types: completed workflow picks up the attached patch artifact and calls https://github.com/googleapis/code-suggester to create code suggestions

A dedicated on: workflow_run workflow would be able to access secrets, so it would be relatively secure, and it would not allow PR builds to access secrets.
Unfortunately, the current googleapis/code-suggester does not support comments for suggestions. In other words, it creates only ```suggestion ... ``` comment, and it would probably be great to pass extra information like the rule name to the message.
For instance, ErrorProne's messages include a link with detailed explanation and the summary:
ERROR: example/myproject/BUILD:29:1: Java compilation in rule '//example/myproject:hello'
ShortSet.java:6: error: [CollectionIncompatibleType] Argument 'i - 1' should not be passed to this method;
its type int is not compatible with its collection's type argument Short
      s.remove(i - 1);
              ^
    (see http://errorprone.info/bugpattern/CollectionIncompatibleType)

OpenRewrite could probably add a link to the rule documentation so the users could know the intention behind the rule.
Additional context

https://github.com/timtebeek/code-suggester-action-reproducer/pull/1
https://github.com/openrewrite/rewrite-gradle-plugin/issues/255
https://docs.github.com/en/rest/pulls/reviews?apiVersion=2022-11-28#create-a-review-for-a-pull-request
INFRA-25352: Provide user / personal access token for a user that could add PR suggestions for automatic code reviews

It looks like pull requests=read and write should be enough for the action to create suggestions.
Unfortunately, it would imply the commenter user would have permission to merge PRs.

As far as I understand reviewdog, it supports diff as input, which should be enough to send the patch file to the API and then one has the suggestions?
It can even made be more simpler - if the diff is existing in the working tree: https://github.com/reviewdog/action-suggester
(I put that as TODO on my list; I wanted to share fast, because we made good experiences in the context of checkstyle with reviewdog)

Thanks for the helpful link @koppor; hadn't come across reviewdog before, and seems a convenient option for other platforms than GitHub indeed. For now I'd started to work with code-suggester in these two PRs, and will expand that with recipe runs:

https://github.com/openrewrite/gh-automation/pull/48
https://github.com/openrewrite/rewrite-migrate-java/pull/402

That approach helpfully hides the implementation details in a shared workflow, such that we can always switch as needed. I'd for now considered this something to use on the OpenRewrite repositories itself, but we can look at opening that up as an action ourselves in the future as well. Input & help welcome. :)

As part of https://github.com/openrewrite/gh-automation/issues/45 we now have two new workflows that together review PRs:
The first one receives a PR and runs recipes, while the second comments any changes found as code suggestions.

https://github.com/openrewrite/gh-automation/blob/main/.github/workflows/receive-pr.yml
https://github.com/openrewrite/gh-automation/blob/main/.github/workflows/comment-pr.yml

An example of such automated review comments can be seen here https://github.com/openrewrite/rewrite-testing-frameworks/pull/343#pullrequestreview-1862541678
We're likely to refine and expand this approach still, but I believe this is a good first step to show how this can be achieved. We don't yet insert accompanying comments, and that's perhaps indeed not supported by code-suggester, but good enough for use on our own PRs for now.
Further input and suggestions welcome, but I think we've achieved enough to close this issue already; I'll monitor any comments that arrive still, and can reopen if needed.


RuntimeError: Expected to have finished reduction in the prior iteration before starting a new one.

I try to train a model using UTF101 dataset.
My GPU is GTX 2060S,
and try to input:
./tools/dist_train_recognizer.sh configs/TSN/ucf101/tsn_rgb_bninception.py 1--validate
NOT work
2020-04-03 15:40:26,326 - INFO - Distributed training: True
2020-04-03 15:40:26,506 - WARNING - The model and loaded state dict do not match exactly
unexpected key in source state_dict: fc.weight, fc.bias
2020-04-03 15:40:29,750 - INFO - Start running, host: ethan@ethan-ideacentre-Y700-34ISH, work_dir: /media/ethan/D/Project/mmaction/work_dirs/tsn_2d_rgb_bninception_seg_3_f1s1_b32_g8
2020-04-03 15:40:29,750 - INFO - workflow: [('train', 1)], max: 80 epochs
Traceback (most recent call last):
File "./tools/train_recognizer.py", line 90, in
main()
File "./tools/train_recognizer.py", line 86, in main
logger=logger)
File "/media/ethan/D/Project/mmaction/mmaction/apis/train.py", line 58, in train_network
_dist_train(model, dataset, cfg, validate=validate)
File "/media/ethan/D/Project/mmaction/mmaction/apis/train.py", line 103, in _dist_train
runner.run(data_loaders, cfg.workflow, cfg.total_epochs)
File "/home/ethan/anaconda3/envs/torch/lib/python3.7/site-packages/mmcv-0.4.2-py3.7-linux-x86_64.egg/mmcv/runner/runner.py", line 359, in run
epoch_runner(data_loaders[i], **kwargs)
File "/home/ethan/anaconda3/envs/torch/lib/python3.7/site-packages/mmcv-0.4.2-py3.7-linux-x86_64.egg/mmcv/runner/runner.py", line 263, in train
self.model, data_batch, train_mode=True, **kwargs)
File "/media/ethan/D/Project/mmaction/mmaction/apis/train.py", line 37, in batch_processor
losses = model(**data)
File "/home/ethan/anaconda3/envs/torch/lib/python3.7/site-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/home/ethan/anaconda3/envs/torch/lib/python3.7/site-packages/torch/nn/parallel/distributed.py", line 392, in forward
self.reducer.prepare_for_backward([])
RuntimeError: Expected to have finished reduction in the prior iteration before starting a new one. This error indicates that your module has parameters that were not used in producing its output (the return value of forward). You can enable unused parameter detection by passing the keyword argument find_unused_parameters=True to torch.nn.parallel.DistributedDataParallel. If you already have this argument set, then the distributed data parallel module wasn't able to locate the output tensors in the return value of your module's forward function. Please include the structure of the return value of forward of your module when reporting this issue (e.g. list, dict, iterable). (prepare_for_backward at /opt/conda/conda-bld/pytorch_1556653114079/work/torch/csrc/distributed/c10d/reducer.cpp:408)
frame #0: c10::Error::Error(c10::SourceLocation, std::string const&) + 0x45 (0x7f26c088bdc5 in /home/ethan/anaconda3/envs/torch/lib/python3.7/site-packages/torch/lib/libc10.so)
frame #1: c10d::Reducer::prepare_for_backward(std::vector<torch::autograd::Variable, std::allocatortorch::autograd::Variable > const&) + 0x5ff (0x7f26efe3dbbf in /home/ethan/anaconda3/envs/torch/lib/python3.7/site-packages/torch/lib/libtorch_python.so)
frame #2:  + 0x6cb6c8 (0x7f26efe336c8 in /home/ethan/anaconda3/envs/torch/lib/python3.7/site-packages/torch/lib/libtorch_python.so)
frame #3:  + 0x12d07a (0x7f26ef89507a in /home/ethan/anaconda3/envs/torch/lib/python3.7/site-packages/torch/lib/libtorch_python.so)
frame #4: _PyMethodDef_RawFastCallKeywords + 0x264 (0x5578a8593ab4 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #5: _PyCFunction_FastCallKeywords + 0x21 (0x5578a8593bd1 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #6: _PyEval_EvalFrameDefault + 0x5389 (0x5578a85faa39 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #7: _PyEval_EvalCodeWithName + 0x2f9 (0x5578a853f389 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #8: _PyFunction_FastCallDict + 0x3ff (0x5578a85406ef in /home/ethan/anaconda3/envs/torch/bin/python)
frame #9: _PyObject_Call_Prepend + 0x63 (0x5578a855fa73 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #10: PyObject_Call + 0x6e (0x5578a8551fde in /home/ethan/anaconda3/envs/torch/bin/python)
frame #11: _PyEval_EvalFrameDefault + 0x1e9d (0x5578a85f754d in /home/ethan/anaconda3/envs/torch/bin/python)
frame #12: _PyEval_EvalCodeWithName + 0x2f9 (0x5578a853f389 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #13: _PyFunction_FastCallDict + 0x3ff (0x5578a85406ef in /home/ethan/anaconda3/envs/torch/bin/python)
frame #14: _PyObject_Call_Prepend + 0x63 (0x5578a855fa73 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #15:  + 0x17d27a (0x5578a85a727a in /home/ethan/anaconda3/envs/torch/bin/python)
frame #16: PyObject_Call + 0x6e (0x5578a8551fde in /home/ethan/anaconda3/envs/torch/bin/python)
frame #17: _PyEval_EvalFrameDefault + 0x1e9d (0x5578a85f754d in /home/ethan/anaconda3/envs/torch/bin/python)
frame #18: _PyEval_EvalCodeWithName + 0x2f9 (0x5578a853f389 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #19: _PyFunction_FastCallDict + 0x3ff (0x5578a85406ef in /home/ethan/anaconda3/envs/torch/bin/python)
frame #20: _PyEval_EvalFrameDefault + 0x1e9d (0x5578a85f754d in /home/ethan/anaconda3/envs/torch/bin/python)
frame #21: _PyEval_EvalCodeWithName + 0x2f9 (0x5578a853f389 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #22: _PyFunction_FastCallDict + 0x1d5 (0x5578a85404c5 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #23: _PyObject_Call_Prepend + 0x63 (0x5578a855fa73 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #24: PyObject_Call + 0x6e (0x5578a8551fde in /home/ethan/anaconda3/envs/torch/bin/python)
frame #25: _PyEval_EvalFrameDefault + 0x1e9d (0x5578a85f754d in /home/ethan/anaconda3/envs/torch/bin/python)
frame #26: _PyEval_EvalCodeWithName + 0x2f9 (0x5578a853f389 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #27: _PyFunction_FastCallKeywords + 0x387 (0x5578a85932b7 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #28: _PyEval_EvalFrameDefault + 0x690 (0x5578a85f5d40 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #29: _PyEval_EvalCodeWithName + 0x2f9 (0x5578a853f389 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #30: _PyFunction_FastCallKeywords + 0x387 (0x5578a85932b7 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #31: _PyEval_EvalFrameDefault + 0x14d4 (0x5578a85f6b84 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #32: _PyEval_EvalCodeWithName + 0x2f9 (0x5578a853f389 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #33: _PyFunction_FastCallKeywords + 0x387 (0x5578a85932b7 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #34: _PyEval_EvalFrameDefault + 0x14d4 (0x5578a85f6b84 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #35: _PyFunction_FastCallKeywords + 0xfb (0x5578a859302b in /home/ethan/anaconda3/envs/torch/bin/python)
frame #36: _PyEval_EvalFrameDefault + 0x416 (0x5578a85f5ac6 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #37: _PyEval_EvalCodeWithName + 0x2f9 (0x5578a853f389 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #38: PyEval_EvalCodeEx + 0x44 (0x5578a85402b4 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #39: PyEval_EvalCode + 0x1c (0x5578a85402dc in /home/ethan/anaconda3/envs/torch/bin/python)
frame #40:  + 0x22c664 (0x5578a8656664 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #41: PyRun_FileExFlags + 0xa1 (0x5578a8660a91 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #42: PyRun_SimpleFileExFlags + 0x1c3 (0x5578a8660c83 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #43:  + 0x237db5 (0x5578a8661db5 in /home/ethan/anaconda3/envs/torch/bin/python)
frame #44: _Py_UnixMain + 0x3c (0x5578a8661edc in /home/ethan/anaconda3/envs/torch/bin/python)
frame #45: __libc_start_main + 0xe7 (0x7f26fee80b97 in /lib/x86_64-linux-gnu/libc.so.6)
frame #46:  + 0x1db3e0 (0x5578a86053e0 in /home/ethan/anaconda3/envs/torch/bin/python)
Traceback (most recent call last):
File "/home/ethan/anaconda3/envs/torch/lib/python3.7/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/home/ethan/anaconda3/envs/torch/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/ethan/anaconda3/envs/torch/lib/python3.7/site-packages/torch/distributed/launch.py", line 235, in
main()
File "/home/ethan/anaconda3/envs/torch/lib/python3.7/site-packages/torch/distributed/launch.py", line 231, in main
cmd=process.args)
subprocess.CalledProcessError: Command '['/home/ethan/anaconda3/envs/torch/bin/python', '-u', './tools/train_recognizer.py', '--local_rank=0', 'configs/TSN/ucf101/tsn_rgb_bninception.py', '--launcher', 'pytorch']' returned non-zero exit status 1.
my pytorch version is 1.1.0
any solution?

This error is caused if you use the .sh script with argument of 1 for GPU count. I was able to run successfully for single GPU by running train_recognizer.py independently(without using the shell script).

I have same problem.  do you fix this ??

Me too. Having same problem with 2 for GPU.

The logs:
RuntimeError: Expected to have finished reduction in the prior iteration before starting a new one. This error indicates that your module has parameters that were not used in producing its output (the return value of forward). You can enable unused parameter detection by passing the keyword argument find_unused_parameters=True to torch.nn.parallel.DistributedDataParallel. If you already have this argument set, then the distributed data parallel module wasn't able to locate the output tensors in the return value of your module's forward function. Please include the structure of the return value of forward of your module when reporting this issue (e.g. list, dict, iterable)
fix:
https://github.com/open-mmlab/mmaction/blob/c7e3b7c11fb94131be9b48a8e3d510589addc3ce/mmaction/apis/train.py#L73
from: find_unused_parameters = cfg.get('find_unused_parameters', False)
to: find_unused_parameters = cfg.get('find_unused_parameters', True)

The logs:
RuntimeError: Expected to have finished reduction in the prior iteration before starting a new one. This error indicates that your module has parameters that were not used in producing its output (the return value of forward). You can enable unused parameter detection by passing the keyword argument find_unused_parameters=True to torch.nn.parallel.DistributedDataParallel. If you already have this argument set, then the distributed data parallel module wasn't able to locate the output tensors in the return value of your module's forward function. Please include the structure of the return value of forward of your module when reporting this issue (e.g. list, dict, iterable)
fix:
https://github.com/open-mmlab/mmaction/blob/c7e3b7c11fb94131be9b48a8e3d510589addc3ce/mmaction/apis/train.py#L73
from: find_unused_parameters = cfg.get('find_unused_parameters', False)
to: find_unused_parameters = cfg.get('find_unused_parameters', True)

This one works for me!

The logs:
RuntimeError: Expected to have finished reduction in the prior iteration before starting a new one. This error indicates that your module has parameters that were not used in producing its output (the return value of forward). You can enable unused parameter detection by passing the keyword argument find_unused_parameters=True to torch.nn.parallel.DistributedDataParallel. If you already have this argument set, then the distributed data parallel module wasn't able to locate the output tensors in the return value of your module's forward function. Please include the structure of the return value of forward of your module when reporting this issue (e.g. list, dict, iterable)
fix:
https://github.com/open-mmlab/mmaction/blob/c7e3b7c11fb94131be9b48a8e3d510589addc3ce/mmaction/apis/train.py#L73
from: find_unused_parameters = cfg.get('find_unused_parameters', False)
to: find_unused_parameters = cfg.get('find_unused_parameters', True)

This one works for me!

日志：
RuntimeError：预计在开始新的迭代之前已经完成了先前迭代的缩减。此错误表明您的模块具有未用于生成其输出的参数（forward 的返回值）。您可以通过将关键字参数_find_unused_parameters=True_传递给 torch.nn.parallel.DistributedDataParallel 来启用未使用参数检测。如果您已经设置了此参数，则分布式数据并行模块无法在模块的 forward 函数的返回值中定位输出张量。报告此问题时请包括您的模块前向返回值的结构（例如列表、字典、可迭代）
使固定：
https://github.com/open-mmlab/mmaction/blob/c7e3b7c11fb94131be9b48a8e3d510589addc3ce/mmaction/apis/train.py#L73
从：find_unused_parameters = cfg.get('find_unused_parameters', False) 到：find_unused_parameters = cfg.get('find_unused_parameters', True)

good thanks


'SecResponseBodyAccess Off' does not prevent response body from being logged (SecRuleEngine == DetectionOnly)

Hi!
Setting  SecResponseBodyAccess to Off does not prevent response body from being logged ( SecAuditLogParts ABIJDEFHZ)
Does it mean that with SecResponseBodyAccess Off ResponseBody is still being intercepted?
Expected behaviour - according  to https://github.com/SpiderLabs/ModSecurity/wiki/Reference-Manual-(v2.x)#SecAuditLogParts "E: Intermediary response body (present only if ModSecurity is configured to intercept response bodies... Intercepting response bodies requires SecResponseBodyAccess to be enabled"
Server:

ModSecurity version: ModSecurity v3.0.4
WebServer: nginx-1.18.0
OS (and distro): Ubuntu 16.04

Rule Set :
https://github.com/coreruleset/coreruleset/archive/v3.3.0.tar.gz
Additional context
SecRuleEngine  == DetectionOnly and SecAuditLogType == Concurrent
Removing E from  SecAuditLogParts do prevent ResponseBody  from being logged.
If I set SecRuleEngine  to On and turn on DEBUG logging - logs do confirm that SecResponseBodyAccess is not ignored (https://github.com/SpiderLabs/ModSecurity/commit/42a472adbda21e6ecc4711fd37d704cdb3d98fbb):
Response body is disabled, returning... 1

Hi @amorozkin ,
I'll first confirm your observation:  in ModSecurity v3, part E logging can indeed occur even when SecResponseBodyAccess is Off.
The message that you are seeing ('Response body is disabled, returning...') indicates that any rules inspecting the response body will be skipped.  The message does not mean that part E logging will be omitted (if E is one of the parts that has been requested).
I agree also that this is a departure from what the v2.x Reference Manual states.  Whether this change in behaviour is an intentional one from v2.9 or a bug may be ambiguous.
In the immediate period, if you find the part E output bothersome, I would suggest simply removing part E from SecAuditLogParts whenever you set SecResponseBodyAccess to Off.

Thanks, @martinhsv
Another way to skip E part under these circumstances is by setting SecResponseBodyMimeType  to something like text/doesnotexist - which seems to me more preferable one as it allows to save some RAM by skipping copy response body in Transaction::appendResponseBody function.
BTW - may be adding if (m_rules->m_secResponseBodyAccess != RulesSetProperties::TrueConfigBoolean) check at the top of Transaction::appendResponseBody() function the same way it is done at Transaction::processResponseBody()  would make sense - this way it would restore v2.9 logic relating SecAuditLogParts (if departure was not intentional)


Fixing typo

ometimes, -> Sometimes,

@last72 would you be interested in joining the spotDL developer team?

@last72 would you be interested in joining the spotDL developer team?

Sure. I have changed the last line as well.

Sure. I have changed the last line as well.

Why was the PR closed?

Why was the PR closed?

I have added you to the organisation. You should receive an email from GitHub, or you can accept it at https://github.com/spotDL.
Please also join the spotDL discord @ https://discord.gg/xCa23pwJWY

I have added you to the organisation. You should receive an email from GitHub, or you can accept it at https://github.com/spotDL.
Please also join the spotDL discord @ https://discord.gg/xCa23pwJWY


fix(multiselect): items can be removed from modelValue [MA-2104]

Add code path to handle the case where items are removed from modelValue rather than added.
Add test.

Summary

PR Checklist

[ ] Conventional Commits all commits follow the conventional commit standards outlined in the main README.
[ ] Tests coverage: test coverage was added for new features and bug fixes
[ ] Docs: includes a technically accurate README

@adorack this will need to target the alpha branch

This is targeting main rather than alpha because it's a bug fix.  Let's say that you have a Multiselect that looks roughly like this:
<Multiselect :items="items" v-model="selectedItems" />

selectedItems is an array of string values.  If you add an item value to the array (or set an array with more item values, or whatever), the multiselect will update its DOM to reflect the change.  However, if you remove an item from the array, the multiselect will not update its DOM to reflect the change.  This is because the code path for detecting missing items wasn't present.  This PR adds that path.


ToDo: Add Update Functionality

Objective
Most of the CRUD operations are currently featured within this project except for Updating notes.
Task
Create logic that allows users to update existing notes

Logic For Update Notes Implemented Server-Side on:

lib/notes.js/
routes/api/routes/notesRoutes.js

This has been tested on insomnia and is currently working as expected. The client-side still requires adjustment to handle the saves.

This function has been implemented


How to skip maxPercentIncrease for certain PR

Is your feature request related to a problem? Please describe.
Sometimes, binary size increases are expected. How do we bypass the maxPercentIncrease limit for the PR that introduces such change?
Describe the solution you'd like
Specify a keyword in the PR description or a label to skip the failing bundlemon test.
Describe alternatives you've considered
Use maxSize instead since we can update the bundlemon settings in the same PR. But maxPercentIncrease shouldn't change
Additional context

See here: https://github.com/LironEr/bundlemon/issues/115#issuecomment-1273363609
Forgot to document this ability, but let me know if this solves the issue for you

Yes. It's amazing!
Quick question about the behavior, if I set the limit by max increase percent and I approve the change that increases the binary size, will the next PR fail the bundlemon CI again since the baseline didn't change and the binary size will stay high?

yes, each approval is for a specific commit, the baseline will always be the target branch, once you merge the baseline will change.

I found two issues when trying this feature:

After I approve the change, the bundlemon CI job still fails. I was expecting it to pass.
There is a race condition in the UI between when the review button checks auth state and the auth state changes. If auth state changes after the review button checks the auth status, it will show me log in is required even if I have logged in.

The approval applies to a specific commit that goes to GitHub and changes the current commit status from failed to success, re-running or a new commit will cause it to fail again.
We can think of a solution that checks the PR labels and auto-approves new jobs.
For now, you can do that bundlemon wont fail the pipeline
- name: Run BundleMon
   uses: lironer/bundlemon-action@v1
   with:
     fail-on-error: false

Then your job will always be successful, and if you want to verify the bundlemon status have a "required" status of the bundlemon commit status

Yeah, having a way to auto approve the jobs with a PR labels can be a great idea.
Do you mind elaborating what you meant by "have a "required" status of the bundlemon commit status"
Thanks for the reply!

Setup bundlemon github-integration, then on every commit in a PR bundlemon will add a commit status

In branch protection rules, make BundleMon a required status check

When one of the defined limits exceeds BundleMon will create a failed commit status, You can decide if you want to approve it via the BundleMon website. If you approve it will change the commit status to success


Support Python 3.12

Is your feature request related to a problem? Please describe.
Python 3.12 has been out of beta for several months yet minerva still doesn't support it.
Describe the solution you'd like
Add official support for Python 3.12 by adjusting the requirements and adding 3.12 workflows for CI/CD.
Describe alternatives you've considered
Not supporting Python 3.12 and sticking with 3.11 as the most up-to-date Python supported.
Additional context
Currently blocked by onnx having a Python 3.12 wheel.

Blocker is now tensorflow. Latest stable build does not support Python 3,12. Using latest tf-nightly as shown in a3597ce does not rectify this.

tensorflow has been updated and onnx has a onnx-weekly version that supports 3.12. The only blocker now is onnx2torch which will only use onnx not onnx-weekly. Also, torch.compile is still unsupported in 3,12 so has to be bypassed.


Aug 10 22:28:44 ss220-host-2 taskset[407149]: [2022.08.10-19.28.09:359][ 34]LogRCONServer: 3846912:CloseConnection(): Closing socket for <IP_ADDRESS>:41478
Aug 10 22:28:44 ss220-host-2 taskset[407149]: [2022.08.10-19.28.09:359][ 34]LogRCONServer: 3846912:Run(): Client <IP_ADDRESS>:41478 has closed its connection...

Aug 10 22:28:44 ss220-host-2 taskset[407149]: [2022.08.10-19.28.39:570][540]LogRCONServer: 3846985:CloseConnection(): Closing socket for <IP_ADDRESS>:41640
Aug 10 22:28:44 ss220-host-2 taskset[407149]: [2022.08.10-19.28.39:570][540]LogRCONServer: 3846985:Run(): Client <IP_ADDRESS>:41640 has closed its connection...
Aug 10 22:28:44 ss220-host-2 taskset[407149]: [2022.08.10-19.28.44:764][799]LogOnlineSession: Warning: STEAM: Empty session setting CurrentModLoadedHash2  : OnlineService of type String```

I think it's an error with the Squad server itself when multiple RCON clients try to connect, try increasing the number of possible connections in the RCON configuration file.

Hi, that would be cool. I faced the issue because sqcp when it generate multiple connections and SquadJS does not connect to server rcon.

I got the same issue.
Are there any updates?
Rcon limit is at 50 in my rcon config.
The panel gets the squad names, but not the users.
Exacly the same error messsages.

I had the same problem.There are no users because squad v7.0 update, you need to fix regex for users.

@milutinke

I had the same problem.There are no users because squad v7.0 update, you need to fix regex for users.

Sorry for the late reply, I am not at home.
Can you post an output of the PlayerList command so I can change the regex.
Thanks.
@Maximator98
I'll put the time to finish the C# implementation soon, so this will not be an issue in the new version, currently it's a bug with the Squad Server Rcon implementation, it happens after some time when there has been a lot of connections, my guess is that they don't release some resources and then run out of the available slots.
Since Squad JS keeps a single connection open, this should not be an issue.

I had the same problem.There are no users because squad v7.0 update, you need to fix regex for users.

@anklav24 Sorry for the late reply, I am not at home. Can you post an output of the PlayerList command so I can change the regex. Thanks.
@Maximator98 I'll put the time to finish the C# implementation soon, so this will not be an issue in the new version, currently it's a bug with the Squad Server Rcon implementation, it happens after some time when there has been a lot of connections, my guess is that they don't release some resources and then run out of the available slots. Since Squad JS keeps a single connection open, this should not be an issue.

I've added pull request, this has been working for me fine for about a month #17

How to update?

How to update?

Stop the SQCP.
Download the source code, and then replace backend/Rcon/SquadRcon.js file with the one from the new source code.
Start it again.


Bugfix: element.target is undefined

Thanks for taking the time to file a pull request with us.
Please take a moment to provide the following details:

Description

Motivation and context
If you have some item you are observing, and you add & remove it from Dom *on button click for eg), reobserve get called, but this.target is sometimes null as the ref isn't present yet. So, even though we do findDOMNode(this.target)  in  IntersectionObserver.observe() , this.target is still null. changed it to do findDomNode(this), if this.target is not found.
This is the stacktrace of the error that this prevents:
Uncaught TypeError: Failed to execute 'observe' on 'IntersectionObserver': parameter 1 is not of type 'Element'.
    at Function.observe (IntersectionObserverContainer.js:91)
    at IntersectionObserver.observe (VM161176 IntersectionObserver.js:86)
    at IntersectionObserver.reobserve (VM161176 IntersectionObserver.js:96)
    at IntersectionObserver.componentDidUpdate (VM161176 IntersectionObserver.js:116)
    at measureLifeCyclePerf (vendor.js:43253)
    at vendor.js:43906
    at CallbackQueue.notifyAll (vendor.js:14619)
    at ReactReconcileTransaction.close (vendor.js:45399)
    at ReactReconcileTransaction.closeAll (vendor.js:8969)
    at ReactReconcileTransaction.perform (vendor.js:8916)

How has this been tested?
Tested by adding & removing the parent (+ observed children) from the Dom
Screenshots (if relevant):
Type of change

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

Checklist:

[x] My code follows the code style of this project
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[ ] I have read the CONTRIBUTING document
[ ] I have added tests to cover my changes
[ ] All new and existing tests passed

Hi, thanks for the PR.
In order to reproduce this issue, could you please create an example online to help us verify this behavior? For example using codepen, codesandbox or jsfiddle.

I was able to create something here, using recompose (which is what I'm using in my project too)
Click on the toggle button to see the error. all i do is pass the first div (which says testing) through compose, and put it in intersection observer. You can try it without the condition, it fails on first load too
https://codesandbox.io/s/10z0ox8o4l

Thanks for the concrete example, this helps loads.
After the button onClick event there's a React warning stating that MyDiv is a functional component. Functional components can't be assigned refs so <Observer /> won't be able to use the ref in IntersectionObserver.observe() call.
Warning: Stateless function components cannot be given refs. Attempts to access this ref will fail.

Check the render method of `IntersectionObserver`.
    in MyDiv (created by IntersectionObserver)
    in IntersectionObserver (created by ExampleComponent)
    in div (created by ExampleComponent)
    in ExampleComponent (created by App)
    in div (created by App)
    in App

This isn't a shortcoming of react-intersection-observer, but imposed by React as a requirement for refs to work. In fact, I added this warning manually for React 15 (React 16 implemented it internally).
Turning <MyDiv /> into a class solves the issue: https://codesandbox.io/s/mjr97q07qj
Let me know if it works as expected for you as well.

I'm closing this pull request due to the lack of inactivity. Feel free to re-open the issue if you have further info.


Hash function improvement

Currently, our hash function is quite naive, and it's very easy to cause very bad collisions in the hash index.
We should replace it. DuckDB seems to be using a very simple but efficient hash function, which seems to be a good choice based on some cursory reading online.
We should also change the name since we aren't actually using murmurhash.

Changing the integer hash is straightforward, but there are some major remaining questions:

What do we do about our CombineHash function? DuckDB uses a simple XOR, but this means ab and ba have the same hash, which is generally undesirable, or so I've been told.
Semi-related, our string hash algorithm is std::hash. Should we change this? DuckDB and Postgres are both using something super specialized.

Semi-related, our string hash algorithm is std::hash. Should we change this? DuckDB and Postgres are both using something super specialized.

Simple answer is yes: #2952.
DuckDB is using a string function from here, which is actually just MurmurHash2. I did a quick benchmark and the performance is almost identical to the rolling hash implementation I wrote for #2952; though I don't know if it produces better results. Postgres seems to be using a hash function from here; the author says that murmurhash is faster, but weaker, than his hash function (the postgres version has an update from 2006, but I think we'd be better off looking at newer hash function comparisons like smhasher if we want to find something better).
Also, I think this issue was already more or less implemented in 359c09d5ad, so maybe this could be closed after #2952 is merged.

Yes, integer hashing was changed by me. String hashing, seems like you have a PR for. I think the other hash functions are defined in terms of those two.


feat(n-dropdown): customizable optionPadding

This function solves the problem (这个功能解决的问题)
the padding of an option is not adjustable via configprovider
Expected API (期望的 API)
const override: GlobalThemeOverrides = {
    Dropdown: {
        optionPadding: `8px 8px`
    }
}

You can use prefix width & suffix width

and vertical padding? 😅

and vertical padding? 😅
edit: i guess height?

Yep.

Height is more consistent for implementation.


Document repo layout

seperate spots for small and large mol

#12 took care of most of it but we can keep this issue open until we are done with the layout, we might need more stubbed out


Syntax for default message in custom template

What is the correct syntax to change the default message in a custom template ?
This one doesn't work:

Your Custom Message

Have you tried using any of the language settings in your options object?
Can you post a fuller snippet?

It works fine with language settings in my options but I need to change the CSS of this message, that's why I'm trying to put it in my template.
    methods: {
        template: function() {
            return `

===CUSTOM MESSAGE===

`;
        },
}

The div with dz-message class is not displaying anything, other divs work fine.

@devondahon Did you resolved your issue?

@rowanwins he means this message (default message / dictDefaultMessage) :

disappears when a custom template is used (previewTemplate) and looks like this:

briefly,  if we use "previewTemplate" the default message is not rendering.
I have the same problem btw!

+1 . Anyone able to solve this?

@boka18 u need its works for me.
you need to define dz-default dz-message in the html of the form, not in the template
<form action="/file-upload" class="dropzone dz-clickable" id="uploadImageForm">
Drop files here to uploassd
</form>

Anyone have any info on this? @boka18 @jomedinna @rowanwins @devondahon
I am running into an issue, while using useCustomSlot and previewTemplate().
I have to set useCustomSlot to false after adding the file to fix it, but I'm wondering if this is the correct way to use these together?   I am only testing this stuff out without a live endpoint now.  I will want to be able to add progress, and messaging along the way for the user, but it is feeling hard to manipulate the previewTemplate at this point.

Setting the default message with a custom template is actually in the official docs, but took me a while to find it: https://rowanwins.github.io/vue-dropzone/docs/dist/#/custom-html
Hope that helps someone else, or future me.

@johncarter-  bro you save me :D ty


Upload images wrong path

My environment is;

Python 3.6.6 in venv
Django 2.1.1

The package upload images to wrong path at project folder if install instruction followed exactly. Even settings.py has;
MEDIA_URL = '/media/'
MEDIA_ROOT = os.path.join(BASE_DIR, 'media/')

it still uploads file to BASE_DIR + django_summernote instead of BASEDIR + media/django_summernote.
The Attachment model has gets it at admin panel. But the image link for an instance contains /media/ in the beginning so it returns 404 (because of that image uploaded to wrong path)

I'm at the development server and my statics loading setting are OK (even for media folder). Like this;
urlpatterns = [
    # ...
    path('summernote/', include('django_summernote.urls')),
] + static(settings.MEDIA_URL, document_root=settings.MEDIA_ROOT)

I'm not sure if it's a Python problem or JS

Problem solved after refreshing migrations and venv. Issue can be closed.


Update pomodoro extension

Description
This pull request adds slack support to update slack status when using pomodoro intervals.
2 new commands were added to not modify existing pomodoro commands.
Screencast
https://github.com/raycast/extensions/assets/4130008/6467d10f-894f-45fb-9590-0161b5e928e5
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Thanks for adding this. I think it's adding a lot of value, and I do think that you're right about having it as separate commands works.
Could you check this, @KELiON?

@KELiON just pinging

Thanks for the correction, let's merge it 🙂


WIP: progress indicators

Current coverage is 78.57% (diff: 83.87%)

Merging #41 into master will increase coverage by 0.20%

@@             master        #41   diff @@
==========================================
  Files             8          8
  Lines           356        378    +22
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            279        297    +18
- Misses           77         81     +4
  Partials          0          0

Powered by Codecov. Last update a1c5e40...4b64312


📌 nexus 3.41.1 📌

What is this PR About?
update to latest nexus supported 3.14.1 ubi image
How do we test this?
helm install
cc: @redhat-cop/day-in-the-life

Lgtm 🙌🏼🍁🍂


[AOT][Testing] Print output values on test failure

This commit enhances the AOT test harness to print the "actual" and "reference" values when there is a mismatch. This helps when debugging a failing test. Sample output:
Actual, Reference
8.502946, 8.887751
9.810405, 9.108611
8.563767, 9.041000
10.019511, 9.190888
....

Thanks @lhutton1, would you mind adding a test for this change? There is currently no path that exercises this option.


ｅｖａｌ error

2019-03-31 16:47:03.634158: W tensorflow/core/framework/op_kernel.cc:1192] Invalid argument: ValueError: attempt to get argmin of an empty sequence
[[Node: PyFuncStateless = PyFuncStateless[Tin=[DT_STRING, DT_STRING], Tout=[DT_STRING], token="pyfunc_0", _device="/job:localhost/replica:0/task:0/device:CPU:0"](prefetch_queue_Dequeue:4, strided_slice_5)]]
INFO:root:Skipping image

hello,did you find the solution ,i have met the same problem,thanks a lot! @zhuliqian

I have met the same problem, how to solve it? Thanks!

If you are working on your own dataset, the problem may be caused by evaluating with the lexicon(not available). You can modify the eval_config in 'experiments/demo/config/trainval.prototxt' to not use the lexicon.
eval_with_lexicon: false
It works for me.


Update win32 metadata

Pick up 17.0.5 and improve support for string escaping, needed for some of the new string literals in this update.

This version of the win32 metadata has a few too many bugs - will try again soon.


Reformatting only changed code does not work in 1.4 and latest Intellij

Describe the bug
https://github.com/dubreuia/intellij-plugin-save-actions/issues/217 mentions reformatting only changed code option should work. However, it doesn't. In latest version of intellij it still modifies the entire file.
To Reproduce

Open an existing file with VCS enabled (e.g. git) that requires re-formatting in various places.
Change a line that triggers save action to fix it (e.g. requiring re-formatting)
Save the file
The entire file is modified

Expected behavior
Only that line should be reformatted.
Screenshots (optional)

Versions
IntelliJ IDEA 2018.3.5 (Ultimate Edition)
Build #IU-183.5912.21, built on February 26, 2019
JRE: 1.8.0_152-release-1343-b28 x86_64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
macOS 10.14.2
Plugin version: 1.4
Attachments
<?xml version="1.0" encoding="UTF-8"?>
<project version="4">
  <component name="SaveActionSettings">
    <option name="actions">
      <set>
        <option value="activate" />
        <option value="noActionIfCompileErrors" />
        <option value="organizeImports" />
        <option value="reformatChangedCode" />
        <option value="fieldCanBeFinal" />
        <option value="localCanBeFinal" />
        <option value="missingOverrideAnnotation" />
        <option value="useBlocks" />
      </set>
    </option>
    <option name="configurationPath" value="" />
  </component>
</project>

+1, I am also having this issue

Hey @awelm and @xunnanxu can you define step number 3 in "To Reproduce"? How do you trigger save-actions?

Frame deactivation (window loses focus)
Shortcut
...

@dubreuia frame deactivation/save all action

+1
I go into a file and change a single line and the whole file gets reformatted. I have to disable this plugin for now.
Worked well but stopped all of a sudden. I updated IntelliJ and save actions with no luck.
Anyone was able to find a fix?
IntelliJ IDEA 2018.3.6 (Community Edition)
Build #IC-183.6156.11, built on March 25, 2019
JRE: 1.8.0_152-release-1343-b28 X86_64
JVM: OpenJDK 64-Bit Server VM by JetBrains s.r.o

Save Actions: v1.5.0+2018.3

macOS Mojave 10.14.3

Settings.xml:
<?xml version="1.0" encoding="UTF-8"?>
<project version="4">
  <component name="SaveActionSettings">
    <option name="actions">
      <set>
        <option value="activate" />
        <option value="organizeImports" />
        <option value="reformatChangedCode" />
        <option value="fieldCanBeFinal" />
        <option value="localCanBeFinal" />
        <option value="missingOverrideAnnotation" />
        <option value="useBlocks" />
        <option value="unnecessarySemicolon" />
      </set>
    </option>
    <option name="configurationPath" value="" />
  </component>
</project>

It is really hard to fix since I can't reproduce. I'm pretty sure it has to do with the recent refactoring, but I have no idea why that happens.

Same thing happens to me after updating to 2018.3.6

I'm also experiencing the same bug on 2019.1.3 (Ultimate Edition)

If I am not mistaken, I believe underlying issue is related to how Java Fixes operate when using the option to reformat only changed changed code. I would assume that the java fixes are only applied to changed code when the option is selected. This does not appear to be what is happening. To demonstrate the issue, here is some code that has many formatting problems.
package issues;

import java.util.Set;
import java.util.List;
import java.util.Collection;

public class SaveAction230 {
	public static void main(String[] args) {
		int x = 0;;

        System.out.println(x);

		Set<String> s;
	    Collection<String> c;
	}
}

I ran a few experiments by adding int y = 0; on line 10. Then I saved the file.

Test reformat only changed code

<?xml version="1.0" encoding="UTF-8"?>
<project version="4">
  <component name="SaveActionSettings">
    <option name="actions">
      <set>
        <option value="activate" />
        <option value="reformatChangedCode" />
      </set>
    </option>
    <option name="configurationPath" value="" />
  </component>
</project>

diff --git a/src/issues/SaveAction230.java b/src/issues/SaveAction230.java
index 0182199..d611f91 100644
--- a/src/issues/SaveAction230.java
+++ b/src/issues/SaveAction230.java
@@ -7,6 +7,7 @@ import java.util.Collection;
 public class SaveAction230 {
 	public static void main(String[] args) {
 		int x = 0;;
+		int y = 0;

         System.out.println(x);

This appears to be working as expected.

Verify that reformat all lines works as expected to make sure that other issues are detected.

<?xml version="1.0" encoding="UTF-8"?>
<project version="4">
  <component name="SaveActionSettings">
    <option name="actions">
      <set>
        <option value="activate" />
        <option value="reformat" />
      </set>
    </option>
    <option name="configurationPath" value="" />
  </component>
</project>

diff --git a/src/issues/SaveAction230.java b/src/issues/SaveAction230.java
index 0182199..1211ff9 100644
--- a/src/issues/SaveAction230.java
+++ b/src/issues/SaveAction230.java
@@ -6,11 +6,13 @@ import java.util.Collection;

 public class SaveAction230 {
 	public static void main(String[] args) {
-		int x = 0;;
+		int x = 0;
+		;
+		int y = 0;

-        System.out.println(x);
+		System.out.println(x);

 		Set<String> s;
-	    Collection<String> c;
+		Collection<String> c;
 	}
 }

This appears to work as well.

Test reformat only changed code in combination with remove unnecessary semicolons.

<?xml version="1.0" encoding="UTF-8"?>
<project version="4">
  <component name="SaveActionSettings">
    <option name="actions">
      <set>
        <option value="activate" />
        <option value="reformatChangedCode" />
        <option value="unnecessarySemicolon" />
      </set>
    </option>
    <option name="configurationPath" value="" />
  </component>
</project>

diff --git a/src/issues/SaveAction230.java b/src/issues/SaveAction230.java
index 0182199..6aa1311 100644
--- a/src/issues/SaveAction230.java
+++ b/src/issues/SaveAction230.java
@@ -6,7 +6,8 @@ import java.util.Collection;

 public class SaveAction230 {
 	public static void main(String[] args) {
-		int x = 0;;
+		int x = 0;
+		int y = 0;

         System.out.println(x);

This does not work as I would expect as it fixes line 9 which was not changed by this edit. I also tried making the change below the print statement to make sure that I was not accidentally editing line 9. However, line 9 was still reformatted to drop the unnecessary semicolon.
On a side note, it appears that when Optimize Imports is enabled it will also optimize imports even when reformat only changed code is enabled. I assume that this is by design though. But could also be a reason a file changes when you did not directly edit that portion.
Versions:
IntelliJ IDEA 2019.2.4 (Ultimate Edition)
Build #IU-192.7142.36, built on October 28, 2019
Runtime version: 11.0.4+10-b304.77 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 10 10.0
Save Actions: 1.6.0+2018.3

Hello @DragonAssassin, thank you for the debugging! You can confusing two different things: "reformating" and "quick fixes". Reformatting has an option to work on changed line only, but no other action has that functionality. Quick fixes always apply to all the lines.
I thought of implementing the "Action on changed code", but it is a lot of work, and honestly, I'm not sure applying the modification to only changed lines is a good practice, but kept it because people kept asking for it.


text-classification-db-pedia Error while text-classification-ag-news OK

Describe the bug
thread 'main' panicked at 'called Result::unwrap() on an Err value: Error("missing field title", line: 1, column: 263)', /home/mycat/projects/burn/burn-dataset/src/dataset/in_memory.rs:40:69
To Reproduce
cargo run --example text-classification-db-pedia --release
Desktop (please complete the following information):

OS: ubuntu 22.04

Additional context
cargo run --example text-classification-ag-news --release
Successfully generate model.bin

Thanks for the bug report, since both text-classification-db-pedia and text-generation downloads db-pedia dataset, but requires different fields, there was a clash where the cache of the dataset version used for text-generation was also used for text-classification.


Add travis-ci account to test build

We should link account to travis-ci (for example, see scipy) so that passing builds can be confirmed and status shown on README.md page of site.

Resolved with Merge request #13 .


[WIP] Fall back to compile flags when serialized debugging options are missing

lldb already has access to the compilation flags from the DWARF,
we should make a best effort to fall back to those flags if the module
was built without serialized debugging options enabled.

@swift-ci Please test

Hmmm, I'm trying to base this test on TestResilience, but that one is disabled on swiftpr, and fails when I run it locally.

Adrian is working to re-enable that test.

Awesome, I'll watch out for that.

WIP while I migrate the test to lit.

@swift-ci please test

@swift-ci please test

@swift-ci please test

Verified that this passes ninja check-lldb-lit locally on macOS, since the CI bots don't run the lit suite afaik.

@swift-ci please test

@swift-ci please test

Please don't do this. We've been trying to kill the flags out of DWARF for years now and there keeps being one more client using them. We've already identified that they cause more problems than they solve.

I'm not sure that there's a good alternative. Swift debugging relies heavily on compilation info, and -serialize-debugging-options can cause build failures in subtle cases (which means that in the cases we care about, it's unusable). This doesn't even really add more uses of the compile flags, it just slightly widens the case where existing code to use the flags is used.

As an aside - added the test as lit test instead of python due to lower overhead (aka smaller diff) and higher reliability (the python version was failing for infrastructure-related reasons).

I'm not sure that there's a good alternative. Swift debugging relies heavily on compilation info, and -serialize-debugging-options can cause build failures in subtle cases (which means that in the cases we care about, it's unusable). This doesn't even really add more uses of the compile flags, it just slightly widens the case where existing code to use the flags is used.

The kinds of things that are in the raw options are things like header maps, which are also the things that cause the most issues.
Swift debugging relies heavily on search paths. If -serialize-debugging-options is causing problems, that means the debugger will have problems regardless. The DWARF copy of the flags won't be better.

Thanks for you effort, but I agree with Jordan that this is not the right approach. In my experience the primary reason for failing Clang imports is when we have too many (conflicting) header search options, piling more options from various sources on top won't make this better. What's your motivating example for this patch?

The issue we have is that we build some libraries using a certain path to a set of module files. We then export those libraries for use in applications. When the applications are built, they use the same set of module files but at a different path (for reasons that don't relate to Swift, but which we can't change). For developers who work on both the initial set of exported libraries and the applications, both paths exist on the system, and debugging (or building with serialized options) will fail due to duplicate definitions.
I know that our situation with different module paths for different libraries isn't necessarily expected, so we may be stuck carrying a local patch for this (although would be great to have an upstream solution).

This particular patch would run into the exact same problem, then.

Yeah, this patch is only intended to resolve the lack of serialized debugging options, it isn't intended to (and doesn't) solve the entirety of our particular use case. I'm okay with closing this and carrying locally if needed, but I'm still not sure that's the right decision. The existing code already has logic for missing serialized debugging options (if (!got_serialized_options), etc), and this patch just fixes/improves that logic. If we don't want to support that, I understand, but presumably we should just delete that code then, rather than keeping it in a broken state? Keeping it around and also not fixing it doesn't seem like the right call to me.


y coordinate becomes NaN

Describe the bug
The y coordinate in location.href becomes NaN when using useGeographic(), map.addInteraction(new Link()) and
const geolocation = new Geolocation({
	trackingOptions: {
		enableHighAccuracy: true,
	},
	tracking: true,
	projection: view.getProjection(),
});
geolocation.once('change:position', ()=>{
	view.setCenter(geolocation.getPosition());
	view.setResolution(2.388657133911758);
});

Or when start tracking by button click
let startTracking = false;
document.querySelector('.location').addEventListener('click', ()=>{
        startTracking = !startTracking;
	geolocation.setTracking(startTracking);
})

You should not use projection: view.getProjection(), if you are using geographic coordinates as geographic coordinates will be returned if the option is omitted.  In a more general situation where you might have user coordinates you could handle any scenario with projection: getUserProjection() || map.getView().getProjection(),

Thanks for the clarification.
In the “Geographic Coordinates” example, this point is not clarified


[DOCUMENTATION] Video Guides

Feature/Improvement Description
Some people wonder hey does AGiXT have a youtube channel or something well look no further if we have a channel people can learn how to use AGiXT with the awesome guides we create.
Proposed Solution
Making videos can help people better understand how to use AGiXT. BTW Full credit to this request from @Kiro from the discord server.
Acknowledgements

[X] I have searched the existing issues to make sure this feature has not been requested yet.
[X] I have provided enough information for everyone to understand why this feature request is needed in AGiXT.

Sounds like a good job for @ven-ven-ven !  My video editing experience is a lot less than thrilling.  I did take a bunch of screenshots earlier though.
https://josh-xt.github.io/AGiXT/1-Getting started/Screenshots.html

Sounds like a good job for @ven-ven-ven ! My video editing experience is a lot less than thrilling. I did take a bunch of screenshots earlier though.
https://josh-xt.github.io/AGiXT/1-Getting started/Screenshots.html

If nobody can do it I can if it comes down to that.

This is very necessary and important in my opinion.  I understand the logic behind the Josh-XT project.  But I don't understand how to properly configure the project to work with LLM, for example with koboldcpp.  Yes, the documentation https://josh-xt.github.io/AGiXT/ is nice and structured.  But it's not detailed enough in my opinion.  I was unable to set up the interaction between AGiXT and koboldcpp (https://github.com/LostRuins/koboldcpp) using this documentation.  I tried many times.  And I don't understand what I'm doing wrong.  Or it just doesn't work as intended.  It would be interesting to learn as much as possible about the examples of setting up the AGiXT project for working with local large language models.  In video or text format.  It would be great if there would be a detailed FAQ on the project.

https://www.youtube.com/watch?v=XbjjPdYRM_k