Online version finds the right antecedents, but actual version does not

The online version is working fine on the following text:
"I know that Barbara and Sandy are here. I see Barbara watching TV. I hear Sandy breathing."
https://huggingface.co/coref/?text=I know that Barbara and Sandy are here. I see Barbara watching TV. I hear Sandy breathing.
But the actual version doesn't find ennough, just Barbara. It outputs by running:
clusters = coref.one_shot_coref(utterances=u"I know that Barbara and Sandy are here. I see Barbara watching TV. I hear Sandy breathing.")
print(clusters)
print (coref.get_most_representative())
mentions = coref.get_mentions()
print(mentions)

Loading spacy model
Info about model en_core_web_sm

lang               en
pipeline           ['tagger', 'parser', 'ner']
accuracy           {'token_acc': 99.8698372794, 'ents_p': 84.9664503965, 'ents_r': 85.6312524451, 'uas': 91.7237657538, 'tags_acc': 97.0403350292, 'ents_f': 85.2975560875, 'las': 89.800872413}
name               core_web_sm
license            CC BY-SA 3.0
author             Explosion AI
url                https://explosion.ai
vectors            {'keys': 0, 'width': 0, 'vectors': 0}
sources            ['OntoNotes 5', 'Common Crawl']
version            2.0.0
spacy_version      >=2.0.0a18
parent_package     spacy
speed              {'gpu': None, 'nwords': 291344, 'cpu': 5122.3040471407}
email<EMAIL_ADDRESS>description        English multi-task CNN trained on OntoNotes, with GloVe vectors trained on Common Crawl. Assigns word vectors, context-specific token vectors, POS tags, dependency parse and named entities.
source             /usr/local/lib/python3.6/dist-packages/en_core_web_sm

loading model from /usr/local/lib/python3.6/dist-packages/neuralcoref/weights/
{3: [3, 0]}
{}
[Barbara, Barbara and Sandy, Sandy, Barbara, TV, Sandy, Sandy breathing]

I'd be interested in resolving this issue too.  I got the same result as the previous commenter and here are the underlying scores:
{u'pair_scores': {0: {}, 1: {0: -1.8137825597308108}, 2: {0: -1.738390801732288, 1: -1.6511597972712726}, 3: {0: 6.5473994047601911, 1: -0.57869067045464151, 2: -1.6598056098030169}, 4: {0: -1.8103805461400377, 1: -1.5256500224140488, 2: -1.5399936662599227, 3: -1.6966305608918302}, 5: {0: -2.2999057893775179, 1: -1.7149788666508408, 2: 0.68513795195160965, 3: -2.0966374729906301, 4: -1.8540071211764726}, 6: {0: -1.9504528157206593, 1: -1.8210641784028945, 2: -1.8300293314203429, 3: -1.8767248759882404, 4: -1.6296482123249305, 5: -1.9901970079817037}}, u'single_scores': {0: None, 1: 1.5870258214636452, 2: 1.6899656734067761, 3: 1.5896249109319895, 4: 1.8004470287030618, 5: 1.5748515581318938, 6: 1.6261232857954271}}

Hi @bea-alex, @kleinias, after trying the new version it is still seems to be the case.
My guess is that it is related to a different spacy model used in our production setup (we selected a large spacy 1 model) with a higher parsing accuracy.
I will investigate further and keep up updated.

Ok more investigation indicated it's indeed an issue with the accuracy of the spacy model you use.
Parsing the sentence I hear Sandy breathing. with the default spacy 2 en_core_web_sm model incorrectly label Sandy as an ADJ.
The most simple solution is to use the larger model en_core_web_lg which label this example correctly.

We are now on release v3.0 so I am closing this old issue.
Feel free to open it again (or a new one) if you are experiencing some issues with the new release.

Hi,
I am using the en_core_web_lg model, but am still having the issue:
with this sentence:
Once upon a time a lion lived in a forest. One day after a heavy meal it was sleeping.  After a while, a mouse came and it started to play. Suddenly the lion got up with anger and looked for those who disturbed it's nice sleep. Then it saw a small mouse standing trembling with fear. The lion jumped on it. The mouse begged the lion to forgive it. The lion felt pity and left. The mouse ran away. On another day, the lion was caught in a net.
One day after a heavy meal IT was sleeping, is replaced by
One day after a heavy meal ONE DAY AFTER A HEAVY MEAL  was sleeping
and it is replaced correctly in the web version.
Do you have any idea on how to solve this issue?
thank you very much for open-sourcing this project!
https://huggingface.co/coref/my_story


("pack", UrwidImage(...))  causes the UrwidImage to not render

I have an urwid.Columns widget in which I am placing text in some columns, an UrwidImage in another. ("pack", urwid.Text("SomeTextHere"))  works fine.  ("pack",BoxAdapter(UrwidImage(...)),1) causes the UrwidImage to disappear on render.  I think it may be calculating zero-width.  Thoughts?

Yes, UrwidImage isn't a fixed widget.
The reason being, I could think of any way to compute a valid image size without maxcols or maxrows i.e an empty size tuple, and to be fair, it doesn't make much sense anyways.
Use Columns([..., UrwidImage(...), ...]) instead i.e as a box/flow widget with weighted size within the Columns widget.

With Columns(<list of UrwidImage widgets>):

This is totally intended behavior. See:
https://github.com/AnonymouX47/term-image/blob/10ba5add52d33f4f21d9462804b4a2cb9333cbd1/src/term_image/widget/urwid.py#L160
Though, I guess i need to change "packed" to "fixed". That was a mistake.

I'm really sorry, I should've asked first. Any ideas on how to compute the size for a fixed image widget?


redirects vs child routes and tabs vs pages

the way we have tabs implemented is as follows (after the refactor):
{
    path: `/staking`,
    name: `staking`,
    component: staking(`Staking`),
    redirect: `/staking/my-delegations/`,
    children: [
      {
        path: `my-delegations`,
        name: `My Delegations`,
        component: require(`./components/staking/TabMyDelegations`).default
      },
      {
        path: `Validators`,
        name: `Validators`,
        component: require(`./components/staking/TabValidators`).default
      }
    ]
  },

you'll notice that we're using a redirect to move users from /staking to /staking/my-delegations - i'd like to propose that we use the <router-view> element in the template and remove the need for a redirect as documented in the vue-router documentation.
additionally, i'd propose that we drop the naming convention of TabMyDelegations and instead stick with PageMyDelegations or take this one step further and drop the concept of Page altogether ...  just name the files MyDelegations and ValidatorIndex respectively.

you'll notice that we're using a redirect to move users from /staking to /staking/my-delegations - i'd like to propose that we use the  element in the template and remove the need for a redirect as documented in the vue-router documentation.

👍

additionally, i'd propose that we drop the naming convention of TabMyDelegations and instead stick with PageMyDelegations or take this one step further and drop the concept of Page altogether ... just name the files MyDelegations and ValidatorIndex respectively.

I would stick with a prefix. I think this helps to easily get that the component is a page. The extra letters do not hurt imo. I am also not 100% sure to drop the Tab prefix. Pages have headers, Tabs do not have headers as far as I know.

Do you mean you want /staking to represent the My Delegations tab and staking/Validators to represent the Validators tab?

URLs can include uppercase letters and spaces: https://tools.ietf.org/html/rfc1738
I'd prefer not to have an extra layer of translation, e.g., using My Delegations as the URL for My Delegations is more clear to me than my-delegations.

will rename this issue and use #1382 for addressing the tabs


Crypt not encrypting...

macOS 10.14.1, crypt 3.2.1 (updated from 3.2.0)
/var/log/crypt.log
WARNING: fdesetup usingrecoverykey failed to run correctly
Running fdesetup usingrecoverykey via terminal yields:
This command is not supported on APFS volumes.

That is a WARNING message, not an error message. fdesetup usingrecoverykey does not work on apfs drives thanks to Apple but it doesn't stop Crypt from working. Please provide more details about your setup. If encryption is not being turned on it is not because of the above message.

Closing due to inactivity.


2nd-order rrules

This PR is to explore how to implement second-order rrules.
Most urgently, this will allow us to train nonlinear ACE potentials on forces. In the future, we may also use it for training any ACE type model on something that is differentiates (forces, sensitivies...)

Along the way it would be good to also finalize the pullback interface. At the moment, I tentatively suggest,
evaluate(basis, X)
_pb_evaluate(basis, Δ, X)
_pb_pb_evaluate(basis, Δ2, Δ, X)

this seems ready to merge as soon as #50 and #51  are finished.


Improved handling of mailgun errors on bulk send

A proposal to get help message on Mailgun errors and understand how to fix the mailgun integration.
https://github.com/TryGhost/Ghost/issues/15437

Hey @ltoinel thanks for this PR it was really helpful that you deciphered the mailgun error format. I've tweaked what you did to fit with how our error properties are meant to be used (context is for the details, help is for a doc link) and merge this. It'll go out in the next release, which is usually Fridays.


"Light" virtual device is not created

Alexa works fine on the EduKit, however the Light virtual device is not created as described in the workshop and also not recognized by Alexa. Very likely related to issue: https://github.com/espressif/esp-va-sdk/issues/130

Hi Omerio, you are right and there is an issue where the friendlyName parameter seems to not not being used. I believe I've identified the issue in the endpointId parameter, however the code for this is in the proprietary binary supplied by Espressif. We've been working with Espressif and the Alexa team on this issue and they are testing some of the proposed changes. Should see updates on their side and ours if that goes well.
Please stay tuned.

As a temporary workaround, you can rename the light device to "Light" in the Alexa app and it will work from that point.


[question] how to keep last-modified same in multi machines

I'm using express.static() middleware, and utilize these:

max-age = 0
cache by Last-Modified & ETag header
It worked well with first 200 OK response, & next all 304 Not Modified response.

But After I add one more machine for the site(using nginx for reverse proxy). Things get messed.
Here shows 4 requests for a single file moment.js for 4 times:

and theirs request & response header
1.
req
If-Modified-Since: Wed, 09 Sep 2015 12:59:44 GMT
If-None-Match: W/"18e12-14fb230c30c"
res
ETag: W/"18e12-151b325865f"
Last-Modified: Fri, 18 Dec 2015 03:32:39 GMT

2.
req
If-Modified-Since: Fri, 18 Dec 2015 03:32:39 GMT
If-None-Match: W/"18e12-151b325865f"
res
ETag: W/"18e12-14fb230c30c"
Last-Modified: Wed, 09 Sep 2015 12:59:44 GMT

3.
req
If-Modified-Since: Wed, 09 Sep 2015 12:59:44 GMT
If-None-Match: W/"18e12-14fb230c30c"
res
ETag: W/"18e12-151b325865f"
Last-Modified: Fri, 18 Dec 2015 03:32:39 GMT

4
req
If-Modified-Since: Fri, 18 Dec 2015 03:32:39 GMT
If-None-Match: W/"18e12-151b325865f"
res
ETag: W/"18e12-151b325865f"
Last-Modified: Fri, 18 Dec 2015 03:32:39 GMT

As you see, the file's response header are different for the two machaines, the first last modify time is 09 Sep 2015 12:59:44 GMT the other is 18 Dec 2015 03:32:39 GMT, and I found the generation logic code:

last-modified https://github.com/pillarjs/send/blob/master/index.js#L755-L765
etag https://github.com/jshttp/etag/blob/master/index.js#L128

they are about the file's last modify time, and they were deployed via git. Any way to get out of this?

Hi! This is an issue with any file serving system, including nginx and apache. The solution is that you need to use software that will sync file timestamps across systems. The git system does not provide this out of the box. I would suggest using rsync or another similar system.

Another solution is that you can turn off both the last modified feature and theetag feature of static (see documentation for the flags) and calculate your own ETags instead using some other algorithm.


SqlSugarScope vs SqlSugarClient

I have a question
SqlSugarScope vs SqlSugarClient
Scope vs Client
What is the best configuration to use?
Why are there 2 types of these configurations?
Many thanks

SqlSugarScope singleton mode, thread safe
SqlSugarClient=SqlConnection


[ENG-4224] Collection moderation integration

Ticket: ENG-4224
Feature flag: n/a

Purpose

Make FE work with BE code

Summary of Changes

Combine moderator_remove and admin_remove to just remove to match BE
Change how submissions are removed to use a collection-submission-action with trigger: "remove"

Screenshot(s)

NA

Side Effects

QA Notes

We are we removing fidelity between a moderator and project admin?

As per scrum today, we will be removing the specificity of project-admin vs collection-moderator removals


Deployment error with trying to use Fly.io

I am curious if you have tried or know anyone that has deployed an app using TMI on fly.io?
I am experiencing this error when it trying to authenticate the user. It appears it connecting to IRC fine but when trying to auth it breaks down from there. I had double-checked the OAuth creds to make sure they are good. My only thought is that it's a networking issue with how fly.io forwards connections. Any ideas you might have will be appreciated.

Looks like maybe password in config is not coming through:
:erlang.iolist_to_iovec([['PASS ', :error], '\r\n'])

Through the power of trying random stuff till things work, I got it deployed. Did a couple of changes to how the configs are loaded and looks like it worked. Nothing to see here.


Adjust disclaimers on the new wallets

Chore
I think we can make the mainnet disclaimer applicable to both networks and versions, and make sure there’s a small additional notice on the canary build about it being a pre-release version and the risks of that, especially on mainnet.
Tasks

[ ] Update main wallet
[ ] Update canary wallet

Notes

Dave confirmed he is happy to just the same disclaimer on both wallets.


How to run cpp after installing package in atom

How to run and debug simple Hello World.cpp files after installing cpp14 package? Do I need to setup other dependencies?
#include <iostream>

using namespace std;

int main(int argc, char* argv[])
{
    cout <<"Hey, you, I'am alive! Ohn and Hello World!\n";
    cin >> ;;
    return 0;
}

How to run it in atom? I don't see Make or Run button anywhere? Many thank.

Hi,
language-cpp14 is a package for C++ syntax highlighting. It can not currently build or run programs. However, there are probably other packages available to do this.
Thanks!


#127 Wrapped classes to be data classes, to ensure value equality

This fixes #127 , ensuring all the wrapped classes behave as expected when performing equality comparison.

Awesome. Thanks!


Update ioctls in guest applications

Some applications fail to compile when using the newer kernel and header files from sev-snp-v10 branch of the AMDESE/linux repository, which is also listed as the current stable guest kernel here .
Both the sev-guest-get-report and sev-guest-kdf binaries fail to build because they use a different (outdated) ioctl interface to communicate with the sev-guest driver.
This pull request updates both programs to use the ioctl interface described in the sev-snp-v10 branch. As a side note, I could not get the sev-host-set-cert-chain binaries to build as the SEV_SET_EXT_CONFIG command is not implemented in sev-snp-v10.

Thanks for the fix, MatthiasGr!


ethd config doesn't work properly when grafana option is set to no

Here's log
ubuntu@ip-xxx:~/ssv-node$ ./ethd config
Your configuration file template is: default.env
You chose to run on goerli testnet
Your deployment choice is: blox
Your consensus client file is: lighthouse-cl-only.yml
Your execution client file is: geth.yml
Your checkpoint consensus client is: https://goerli.beaconstate.info
Your MEV relay(s): https://0xafa4c6985aa049fb79dd37010438cfebeb0f2bd42b115b89dd678dab0670c1de38da0c4e9138c9290a398ecd9a0b3110@boost-relay-goerli.flashbots.net,https://0x821f2a65afb70e7f2e820a925a9b4c80a159620582c1766b1b09729fec178b11ea22abb3a51f07b288be815a1a2ff516@bloxroute.max-profit.builder.goerli.blxrbdn.com,https://0x8f7b17a74569b7a57e9bdafd2e159380759f5dc3ccbd4bf600414147e8c4e1dc6ebada83c0139ac15850eb6c975e82d0@builder-relay-goerli.blocknative.com,https://0xaa1488eae4b06a1fff840a2b6db167afc520758dc2c8af0dfb57037954df3431b747e2f900fe8805f05d635e9a29717b@relay-goerli.edennetwork.io,https://0xb1559beef7b5ba3127485bbbb090362d9f497ba64e177ee2c8e7db74746306efad687f2cf8574e38d70067d40ef136dc@relay-stag.ultrasound.money,https://0xab78bf8c781c58078c3beb5710c57940874dd96aef2835e7742c866b4c7c0406754376c2c8285a36c630346aa5c5f833<EMAIL_ADDRESS>Your rewards address is: 0x634b31F29C3724F39A0FC67e5ddc536E04454136
./ethd: line 1858: GRAFANA_CLIENT: unbound variable

Apologies! That was caused by me getting too aggressive with a scripting cleanup / linting pass. Fixed!


[Docs] Overview: Broken markdown tables in overview page

Current State:
Overview page has broken markdown tables.
Screenshot:

Contributor Resources

Meshery documentation site
Meshery documentation source
Instructions for contributing to documentation

This is a strange thing, VSCode shows it proper 👀

Yes actually, it's shows table is properly configured, how this issue arrised! 🤔
Can I take the charge & lookout over docs. If I can do something?

@leecalcote Sir, it's fixed now & able to merge..
Pl verify! 😃


Capture Marital Status

Describe the bug
Add capability to find and convert the following patient information

Patient.maritalStatus / PID-16

NOTE: this was moved from #94

To Reproduce
Create a sample HL7V2.6 patient PID segment with aforementioned patient information.
Expected behavior
Patient information should be detected and added to corresponding fields in HL7 FHIR resources.
Screenshots
N/A
Desktop (please complete the following information):

OS:  Mac OS
Version Big Sur

Additional context
Information about HL7V2.6 codes
Information about HL7 FHIR codes
Tasks:
Create an addition to Patient.yml
Create a Utility in Hl7RelatedGeneralUtils.java to support the changes to Patient.yml.  getAddressUse is a good pattern to follow.
Create a unit test for the utility you created. test_getAddressUse is a good pattern to follow.
Create or enhance an existing unit test with a patient in src/test/java/io/github/linuxforhealth/hl7/segments/Hl7PatientFHIRConversionTest.java

In Release 1.0.9


Run all tests under Miri.

Currently, some tests have #[cfg_attr(miri, ignore)] because they call into ring which calls into some assembly.
The best way to deal with this is probably to mock out all of the ring components to do nothing when #[cfg(miri)] holds.

Hi @mcy , I take this one.

Assigned! GH failed to notify me of your comment >_>


add requestAnimationFrame function and mouseup and mousedown events f…

increase canvas rendering performance by using request animation frame and add two event listeners mouseUp and mouseDown to track the dragging, increase the performace and seo of website  from 84 to 93 and 90 Thanks for you corporation.Go through the code i have added comments as well.

Please mention your issue no.
Also resolve the merge conflicts.


Can't open the alvr dashboard

I try and try but can't get it to work

Please do not open up multiple issues these are for bugs not support


Incremental writes and hotspotting

Mteigers writes:

I think S3 still has the recommendation against writing atomically incrementing files as they are served by the same compute cluster and at scale can cause hotspotting. Maybe they no longer have that advice, my S3 knowledge is a little outdated.
But an alternate would be to provide a fast hash of your counter as a prefix and use like the first 3 characters and use them as “folders”. Something like Meow Hash is supposed to be fast at this so you end up with something like:
meow(000001) = DDB147 (example) And then you write DDB/000001

https://old.reddit.com/r/golang/comments/1h48ujv/building_a_distributed_log_using_s3_under_150/lzwmjzw/

Çağdaş Yelen writes:

Storing objects with this sort of naming convention is not great in S3. Because S3 employs trie-like prefix tree, it puts all of your objects (starts with “0”) in the same partition, which will impact your high qps reads.

https://x.com/cagdasyelen/status/1863303444288635085

ChurroLoco
https://docs.aws.amazon.com/AmazonS3/latest/userguide/optimizing-performance.html
https://old.reddit.com/r/golang/comments/1h48ujv/building_a_distributed_log_using_s3_under_150/lzx3g0o/

I believe prefix randomization is no longer necessary. Also from the same guide pasted above,

This guidance supersedes any previous guidance on optimizing performance for Amazon S3. For example, previously Amazon S3 performance guidelines recommended randomizing prefix naming with hashed characters to optimize performance for frequent data retrievals. You no longer have to randomize prefix naming for performance, and can use sequential date-based naming for your prefixes.

As for the original post for incremental writes, S3 recently added support for Conditional Writes : https://aws.amazon.com/about-aws/whats-new/2024/08/amazon-s3-conditional-writes/

Using conditional writes, you can simplify how distributed applications with multiple clients concurrently update data in parallel across shared datasets. Each client can conditionally write objects, making sure that it does not overwrite any objects already written by another client. [...] To use conditional writes, you can add the HTTP if-none-match conditional header along with PutObject and CompleteMultipartUpload API requests.

I'd imagine a system where the client tries to write an object with some sequential-id and if it already exists, waits for a some bounded backoff interval (with jitter) and tries again with a +1 value (noting ofc backoff interval might not be suitable for all use-cases :) )
Also obligatory hn thread with discussions for use-cases: https://news.ycombinator.com/item?id=42240969

that settles it, then! So current scheme is pretty much okay. Thank you @shiredude95


feat: ability to set no records , no filter results & filter input title text in UiTable

feat: added “title” attribute to UiFilter input
feat: ability to set “noRecordsText” (text to display when zero records where passed to UiTable) in UiTable
feat: ability to set “noFiterResultsText” (text to display when zero records where found while filtering table ) in UiTable
feat: ability to set “filterTitle” (text to display user hover over filter input field ) in UiTable
feat: added UiTable tests

It looks like this PR covers several new features. Could you update the title and add a description outline what all has been done?

@skhanna10 It looks like this PR covers several new features. Could you update the title and add a description outline what all has been done?

@mdeanjones : I have updated title & description , Please let me know if looks good now ?


Add some environment parameters and some new view types

Add environment parameters:

white_plane: if True, use a white ground instead of the default white/blue grid
black_workspace: if True, add a black area covering the workspace
trans_plane: if True, set the ground to be transparent
trans_robot: only useful for kuka robot, if True, set robot to be transparent (but gripper will still be visible)

Add some side view view types

Little worried about bloat in terms of the white/black/normal workspace changes here. Do you think this is all we would ever add and what is the difference between the white/black, i.e. why generate a black box on top of the workspace?


Using zplug for loading *.zsh files from fzf only works once

Describe the bug
Using zplug for loading *.zsh files from fzf only works once.
To Reproduce
Steps to reproduce the behavior:

Add in .zshrc:

zplug "junegunn/fzf", from:github, as:command, rename-to:fzf, hook-build:"./install --all"
zplug "junegunn/fzf", from:github, as:plugin, use:"shell/*.zsh"

Delete the source instruction that fzf installation may have put in the .zshrc.

Expected behavior
All .zsh scripts under shell/ directory are loaded by zplug every time I open a shell.
Env (please complete the following information):

zplug version: [zplug] version: no such command
zsh --version: zsh 5.6.2 (x86_64-debian-linux-gnu)
uname -a: Linux foobar 4.18.0-2-amd64 #1 SMP Debian 4.18.10-2 (2018-11-02) x86_64 GNU/Linux

Minimal zshrc (with less than 30 lines)
export XDG_CONFIG_HOME=${XDG_CONFIG_HOME:=${HOME}/.config}
export XDG_CACHE_HOME=${XDG_CACHE_HOME:=${HOME}/.cache}
export XDG_DATA_HOME=${XDG_DATA_HOME:=${HOME}/.local/share}
export ZDOTDIR=${ZDOTDIR:=${XDG_CONFIG_HOME}/zsh}
ZPLUG_HOME="$XDG_DATA_HOME/zplug"
ZPLUG_LOADFILE="$XDG_CONFIG_HOME/zsh/plugins.zsh"
ZPLUG_BIN='/usr/local/bin'
ZPLUG_USE_CACHE=true
ZPLUG_CACHE_DIR="$XDG_CACHE_HOME/zplug"

source "$ZPLUG_HOME/init.zsh"

zplug "junegunn/fzf", from:github, as:command, rename-to:fzf, hook-build:"./install --all"
zplug "junegunn/fzf", from:github, as:plugin, use:"shell/*.zsh"

# I do not use the following because it increases the startup time.
#if zplug check || zplug install; then
#  zplug load --verbose
#fi

Additional context
$ zplug --version
2.4.2

I load fzf like this:
zplug "junegunn/fzf-bin", \
    from:gh-r, \
    as:command, \
    rename-to:fzf, \
    use:"*linux*amd64*"
zplug "junegunn/fzf", use:"shell/*.zsh", defer:2

I don't know why, but you have to defer the loading. The key-binding is working for me.

Since version 0.24 (see #561) the bin is installable as:
zplug "junegunn/fzf", from:gh-r, as:command, ...

]Now the problem is how to source the zplug "junegunn/fzf", use:"shell/*.zsh", defer:2
It doesn't load the repo probably because it is in the first declaration as command.
How should we source that?

The OP version with the defer works! Thank you @soullivaneuh

I do not see how defer could help with inability to load as:command and as:plugn from the same repo.
The closest to making this work I got with
zplug "junegunn/fzf", use:"shell/*.zsh",defer:2
zplug "junegunn/fzf", as:command, dir:"${ZPLUG_REPOS}/fzf-bin", from:gh-r, use:"*linux_arm64*"

I could install these one at a time. In the end I had
\.zplug/repos/
             fzf-bin/
                    fzf
             junegun/fzf/
                        install
                        uninstall
                        shell/
                             completion.zsh
                             key-bindings.zsh

This, unfortunately, wasn’t enough, as zplug --load gave:
 Link "~/.zplug/repos/junegunn/fzf/install" (junegunn/fzf@)
 Link "~/.zplug/repos/junegunn/fzf/uninstall" (junegunn/fzf@)
[zplug] Run compinit
 Load "~/.zplug/repos/junegunn/fzf/shell/completion.zsh" (junegunn/fzf)
 Load "~/.zplug/repos/junegunn/fzf/shell/key-bindings.zsh" (junegunn/fzf)

I.e. it failed to install the fzf command from fzf-bin directory, and linked (in|unin)stall executables from junegunn/fzf instead.
It seems that zplug can’t handle two plugins with the same repo name. Is there a way to perhaps rename a plugin?

I load fzf like this:
zplug "junegunn/fzf-bin", \
    from:gh-r, \
    as:command, \
    rename-to:fzf, \
    use:"*linux*amd64*"
zplug "junegunn/fzf", use:"shell/*.zsh", defer:2

I don't know why, but you have to defer the loading. The key-binding is working for me.

junegunn/fzf-bin has been deprecated, all functionality is in junegunn/fzf now.
Apparently it cannot be installed with zplug anymore.


Bring back coalesce with predicate

Need the coalesce with predicate functionality back from Rekall 0.0.0 - found more use cases where it's necessary (and having the user reimplement it as a fold is a pain)

Turns out this wasn't necessary - but it would be nice to have coalesce with a general merge op, instead of having to split it into bounds merge op and payload merge op.

This is necessary again!
coalesce needs a predicate argument. The predicate argument should take an Interval that is currently being coalesced and a new Interval, and return True if the new Interval should be merged in. Otherwise, the new Interval should not be merged in.
This means that while the coalesce is happening, you need to keep track of multiple Intervals/tracks that are currently being merged in.
See https://github.com/scanner-research/rekall/blob/c0b3ea903367bca135437bd80b8dcf8c6cdcaaec/rekall/interval_list.py#L212 for the old implementation of how this used to work.


No Nox folder in roaming.

Just as the title says i have no nox folder in roaming i tried manually i tried automatically is still doesnt work. Tried restarting nox nothing at all.

@SpiritEnvy this is to copy the adb executable and the required dll right?

Yes sir, i cant copy it to there and if i run the install it cant install there so I'm not to sure what to do.

Where is Nox installed? You can get a clue from right-clicking the shortcut icon on the desktop or windows bar file location.

Sorry for being a pain but thanks for working with me ill just move on. Your a great person for trying to help though sir. :D


Deploy with TravisCI

Two private elements have to be configured on https://travis-ci.org/karma-runner/karma-browserstack-launcher/settings:

NPM_USER_EMAIL with an email
NPM_API_KEY with a token taken from https://www.npmjs.com/settings/~/tokens, associated to the above npm user email account

I signed it!


Addition of a GetAllVisible Actors function?

Hello,
I have been using CARLA for some graduate related research in the field of Pedestrian detection. I am trying to find a way to get all visible actors (in my case it would be pedestrians) from a camera view using the CARLA API. Unfortunately, the best method I have been able to use involves checking the positions of the actors relative to the camera and using this to determine if the actor is in view or not. This has many issues, most revolving around pedestrians being occluded by buildings, trees, cars, or even other pedestrians but still being recorded using their bounding box.
I was wondering if a function could be implemented into the CARLA API that would return a list of all visible actors from a given RGB camera view?
Thank you

Hey, I've done something similar by using the semantic lidar sensor. Query each point to see if it corresponds to a pedestrian.
https://carla.readthedocs.io/en/latest/ref_sensors/#semantic-lidar-sensor

Hello @And1210,
Thank you for the sugestion, we will study if we can add this functionality in the future.
As @FiodarKazhamiaka suggest, one workaround that is possible with the current tools of carla can be using the Semantic Lidar.

You can set a semantic lidar with a similar FOV of the camera
Check all the actors detected by the semantic lidar
Transform the actor's points to camera space to check if the are visible to the camera. You can find an example of this in the lidar_to_camera.py example script.
Of course this can be a bit costly but it provides the solution to what you need.


[1.19.2] [Crash] Embedded Polymer library crashes EMI.

While loading a world with Eldritch Mobs installed it crashes, this does not happen with the mod removed, seems to be the out of date Polymer library or something; adding the up to date Polymer version causes the issue to poof.
[05:14:30] [Thread-38/INFO]: [EMI] Loading plugin from polymer
[05:14:30] [Thread-38/ERROR]: [EMI] Exception loading plugin provided by polymer
[05:14:30] [Thread-38/INFO]: [STDERR]: java.lang.NoClassDefFoundError: dev/emi/emi/EmiStackList

Which version of polymer did you have to add in order to fix the issue?

I just added the latest at the time, 0.2.28+1.19.2

Fixed in latest release

Nice c:


Secure connection with SSL

I would love to leave pgweb running for quick access any-time I need it. However, I don't see a way to secure the connection to the web interface using HTTPS. In my case I have a wildcard cert that will work perfect if I could just tell pgweb to use it somehow.
Is there any current way to secure the connection to pgweb with HTTPS?
Any possibility of this being added in the future?

Hi. You could use nginx as a proxy in front of pgweb, configuration in this case would super simple
if you already have your ssl certificate. The reason why ssl is not supported out of the box is because
of the same lack of ssl support in gin web framework used by pgweb.

ah, nginx proxy! I was looking for some way to handle it with my current stack, forgot it does proxying. Thanks for the recommendation.
This issue may help others too searching at least. Will close up since that handles what I need.

For anyone else wanting this, I setup an nginx site with this config:

server {
    listen [::]443 ssl spdy;
    listen 443 ssl spdy;

    servername sub.site.com;

    include h5bp/directive-only/ssl.conf; # From the H5BP Nginx Server Configs, very handy stuff.

    location / {
        proxy_pass http://localhost:9999; # set port/domain to whatever you run pgweb bound to.
        proxy_set_header Host $host;
        proxy_set_header X-Real-Ip $remote_addr;  # Not even sure if these last two are needed, just pulled from the nginx wiki example.
    }
}

Boom, nginx proxying to pgweb just fine with full SSL!

Thanks! I will add that to wiki

Any idea how to do it using apache2?

@kurotsukikaitou Just search for Apache reverse proxy instructions. That's all my nginx config is doing.

@kurotsukikaitou also keep in mind that running pgweb on your production servers is not recommended for security reasons.

Well.. I planning just running it only when I need to access it. Otherwise, I'll kill the process.

There are a couple of options to consider if you need to connect to your production database:

Setup pgweb username and password (there's no default), and perhaps a read-only database user.
If you're using ssh keys to connect to the server you can use pgweb's built-in ssh tunnel feature or using native ssh client (https://github.com/sosedoff/pgweb/wiki/SSH-Gateway)

Protect it with a password (htaccess) and it should be fine unless it gets brute forced. Otherwise I don't see any other potential hole from it in this context as to any other part of your server getting attacked.


Update command names and grouping

Changes include:

Updating below command names:
Build -> Build Bicep File
Deploy... -> Deploy Bicep file...
Open Visualizer -> Open Bicep Visualizer
Open Visualizer to the Side -> Open Bicep Visualizer to the Side
All our commands are now group together

Note: For explorer and editor context, navigation group would always show up first. That's by design. Please see https://code.visualstudio.com/api/references/contribution-points#contributes.menus for more info. For editor title, our commands would show up first

@ucheNkadiCode, could you please take a look at the screenshots and +1 if it looks good

@bhsubra looks good to me!


Should we be using premultipled alpha for UI images?

We have some threads around fringing of UI assets:

https://forum.playcanvas.com/t/artifacts-on-our-2d-assets/14834
https://forum.playcanvas.com/t/why-do-my-9-slice-sprites-look-terrible/21081

Should we be using premultipled alpha as this article suggests? https://forum.playcanvas.com/t/why-do-my-9-slice-sprites-look-terrible/21081
Or is there performance issues that should be considered? Maybe we can give users an option

Premultiplied alpha should have no performance impact. We probably should make this a user setting.

So?)


Security

Do you plan develop some security framework for Takes? For treat CSRF atack, user autentication and authorization, SSL, etc...
Congratulations, I love the way Takes was designed, it's clean and well done. Thanks!

@yegor256 please, pay attention to this issue

@eljames we have SSL already. As for other things, feel free to contribute or submit separate tickets, we'll try to fix, if we have resources available. The best way, of course, is to contribute with a pull request :) Thanks for using Takes!


NPE when selecting a MenuItem with no title defined

My menus are defined in xml. I get an NPE when I select one that doesn't have a title defined.

Duplicate of #184.


fix: daredata careers link

ping @marmelo

@afonsojramos maybe you can approve your own PR due to lack of response :D
¯\(ツ)/¯

@afonsojramos maybe you can approve your own PR due to lack of response :D
¯\(ツ)/¯

Not possible on GitHub 😢


[Web] Support minified index.html or minify it automatically when building

Use case / Problem
There are features which are breaks if the index.html is minifed by the developer manually.
For example, the serviceWorkerVersion javascript variable does not update automatically when the web app is built so the users do not see the fresh app after deploy. The problem is that the value of serviceWorkerVersion is always null in the build. It is because in the minified index.html the affected part looks like this:
var serviceWorkerVersion=null
while the web.dart file is looking for this to change:
var serviceWorkerVersion = null
The version changer part in the flutter code:
https://github.com/flutter/flutter/blob/895beb04bb7ae9bed508b27a0f59c5328c7ea7c9/packages/flutter_tools/lib/src/build_system/targets/web.dart#L379-L383
There may be other places / features involved in the problem.
Proposal
The best would be if flutter would minify the index.html (while also removing unnecessary comments) before doing anything with it. The built version should be minified anyway.

You should be able to post-process your flutter web app after it's been built inside build/web, I think?
(We've also made the index.html much smaller in the current stable, and split it into a flutter.js file and a small index.html file, so this might not be as much of an issue anymore?). See the new index.html.


Basic metrics

closes #107, closes #106, closes #61, closes #67
In this PR, I covered the remaining basic tests outlined in the Test Catalogue that don't require file IO (.gbk or similar).
As for issue #67, this PR implements the necessary functions to identify which reactions are unconstrained, irreversible, zero-constrained (lb = ub =0) and non-zero-constrained ( 0 < lb | ub < 1000), but doesn't include any tests that require a certain number/ distribution. While this information is useful when working with a model, I don't think that we need to require any fixed values here. The discussion about whether there should be tests for this or merely a listing in the report can be found here #67.

Codecov Report

Merging #108 into develop will decrease coverage by 0.5%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##           develop    #108      +/-   ##
==========================================
- Coverage       83%   82.5%   -0.51%
==========================================
  Files            8       8
  Lines          306     320      +14
  Branches        62      68       +6
==========================================
+ Hits           254     264      +10
- Misses          47      51       +4
  Partials         5       5

Impacted Files
Coverage Δ

memote/support/basic.py
80.95% <71.42%> (-19.05%)
:arrow_down:

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 e3c199a...52e3eda. Read the comment docs.


Creating new Categories takes a long time for the form to load

When I try to create a new Categorie via the Frontend or the App, it shows "Loading Category..." on the top and it seems like nothing happens. I could not spot anything in the Docker logs so far and no "error" in the Browser Debugging Console. But I left it open while I left the computer for an hour and after I came back, the form to create a new categorie was loaded. The Browser Debugging session looks like this (the 50+ minute entry is still counting upwards):

I have seen in the documentation that the category management is WIP, so I am not sure if this is expected in the current version?

A bug. Will be fixed in next release.

Fixed in release v.0.6.1

Wow, that was quick! I downloaded the latest images / apk, tested and it works for me. Thank you very much 😃


Mi wireless switch (WXKG01LM) problem after update to v1.36.0

Description of the issue
After update zigbee2mqtt Addon to v1.36.0, the button Mi wireless switch (WXKG01LM) started to interpret a single click as a long press. Automations triggered by a single click no longer work.
The situation occurs on all Mi wireless switch (WXKG01LM)
Addon version
v1.36.0
Platform
Core 2024.2.4
Supervisor 2024.02.1
Operating System 12.0
Logs of the issue (if applicable)
In Z2M log "single click" looks like this (MQTT publish two different action topics)  :
info 2024-03-02 19:01:29MQTT publish: topic 'zigbee2mqtt/switch_1', payload '{"action":"single","battery":100,"click":null,"linkquality":54,"power_outage_count":95,"voltage":3022}'
info 2024-03-02 19:01:29MQTT publish: topic 'zigbee2mqtt/switch_1', payload '{"action":"","battery":100,"click":null,"linkquality":54,"power_outage_count":95,"voltage":3022}'
info 2024-03-02 19:01:30MQTT publish: topic 'zigbee2mqtt/switch_1/action', payload 'single'
info 2024-03-02 19:01:30MQTT publish: topic 'zigbee2mqtt/switch_1', payload '{"action":null,"battery":100,"click":"long","linkquality":54,"power_outage_count":95,"voltage":3022}'
info 2024-03-02 19:01:30MQTT publish: topic 'zigbee2mqtt/switch_1', payload '{"action":null,"battery":100,"click":"","linkquality":54,"power_outage_count":95,"voltage":3022}'
info 2024-03-02 19:01:30MQTT publish: topic 'zigbee2mqtt/switch_1/click', payload 'long'

I reported the same issue in the Z2M official repo: https://github.com/Koenkk/zigbee2mqtt/issues/21661 .
Hope can be useful.

+1

will be fixed in the 1 April release.


Update docs

Added documentation for scope.hpp
Updated logo and favicon
Linked prerequisites to the quickstart page
Minor fixes of the Public API page

Please note that the link to the cpp reference for the scope header file looks empty (https://en.cppreference.com/w/cpp/header/experimental/scope). I believe this is the correct link though since they they list it under the scope_* classes (i.e. https://en.cppreference.com/w/cpp/experimental/scope_exit). It just seems that they just haven't added any text (yet?).

Please note that the link to the cpp reference for the scope header file looks empty (https://en.cppreference.com/w/cpp/header/experimental/scope). I believe this is the correct link though since they list it under the scope_* classes (i.e. https://en.cppreference.com/w/cpp/experimental/scope_exit). It just seems that they haven't added any text (yet?).

Yes, they didn't add the docs for that header yet. The link is correct.


Migrate to pydantic v2

Description:

Closes #187
Update to use pydantic v2:

[x] Upgrade dependencies accordingly.

oteapi-core >=0.6.0
otelib >=0.4.0

[x] Use new pydantic v2 model methods throughout.
[x] Use typing.Annotated throughout as this is recommended.
[x] Upgrade to use Python 3.9+ format typing throughout (this can also be achieved by running pyupgrade --py39-plus.
[x] Update tests accordingly.
[x] Ensure all pydantic v2 warnings are taken care of.
[ ] Ensure tests succeed for all environments in the CI.

Type of change:

[ ] Bug fix.
[x] New feature.
[ ] Documentation update.

Checklist for the reviewer:

This checklist should be used as a help for the reviewer.

[ ] Is the change limited to one issue?
[ ] Does this PR close the issue?
[ ] Is the code easy to read and understand, including clearly named variables?
[ ] Do all new feature have an accompanying new test?
[ ] Has the documentation been updated as necessary?

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (1042fe3) 87.29% compared to head (59e7810) 88.43%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #192      +/-   ##
==========================================
+ Coverage   87.29%   88.43%   +1.13%
==========================================
  Files          15       15
  Lines         425      415      -10
==========================================
- Hits          371      367       -4
+ Misses         54       48       -6

Flag
Coverage Δ

linux
88.43% <97.29%> (+1.13%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

oteapi_dlite/models/session.py
100.00% <100.00%> (ø)

oteapi_dlite/strategies/convert.py
94.00% <100.00%> (+1.84%)
:arrow_up:

oteapi_dlite/strategies/filter.py
86.95% <100.00%> (+1.77%)
:arrow_up:

oteapi_dlite/strategies/generate.py
79.59% <100.00%> (+1.16%)
:arrow_up:

oteapi_dlite/strategies/mapping.py
100.00% <100.00%> (ø)

oteapi_dlite/strategies/parse.py
90.90% <100.00%> (+1.77%)
:arrow_up:

oteapi_dlite/strategies/serialise.py
90.62% <100.00%> (+0.30%)
:arrow_up:

oteapi_dlite/utils/nputils.py
84.61% <100.00%> (+1.28%)
:arrow_up:

oteapi_dlite/utils/utils.py
70.58% <100.00%> (+0.77%)
:arrow_up:

oteapi_dlite/strategies/parse_excel.py
91.17% <94.44%> (+1.32%)
:arrow_up:

... and 1 more

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

This is momentarily put on hold/delayed until its impact on current project deliverables depending on this plugin have been determined.

So I've tested this branch with the TEM demo and found that no changes were really necessary, other than ensuring all dependencies support Pydantic v2. In conclusion, I think there is no issue with merging this PR, other than letting any users know that they should ensure they have the following minimum dependencies:

oteapi-dlite>0.1.6 (i.e., the next version to be released after this PR has been merged)
oteapi-core>=0.6
otelib>=0.4

@francescalb Gentle reminder that this has been texted externally and poses no apparent issues - other than assuring the latest versions are used for all OTEAPI-related packages after merging and updating the version. And possibly upgrade plugins as otherwise expected to support pydantic v2 if the depend on oteapi-dlite.


Adds cloud_services table

This table will be used for CloudService model.
EDIT (@blomquisg): Pulled from https://github.com/ManageIQ/manageiq/pull/7996

maybe I would call it host_name and zone_name, but it can be like it is now. Looks good to me now :+1:

@petrblaho I think I asked this in another PR (not sure which)...
What's the difference between a cloud_service and a system_service?
In other words, could the fields being added to this new cloud_service table, just be added to system_service and we use that instead?
I'm not sure I understand the distinction here.

What's the difference between a cloud_service and a system_service?

Answering my own question...
Just talked to @Ladas about this.  He said that cloud_service details actually come from the OSP Cloud REST API.  While the system_service details come from the the OSP Infra SSA.
We would have a CloudService available on the cloud side regardless of whether the there's a corresponding OSP Infra connected to provide the SystemService.

@blomquisg I think answers for your inline questions can be found in discussion at https://github.com/ManageIQ/manageiq/pull/7996 - @Fryguy already asked similar questions.
And as I can see you already has the answer for CloudService vs. SystemService.

@blomquisg zonename, hostname and binary are coming from Fog Compute Service model - so I need to store that to match CloudService with AvailabilityZone, Host and SystemService from MIQ models.

Everything else looks fine, though I do question the binary column name.  I get that it matches fog's interface, but when I hear "binary" in a database I think of the binary column type...I'm not thinking about an executable on the command line.  Maybe executable or cli might be a better name choice.  @ladas @blomquisg thoughts?

@Fryguy executable_name - what do you think?
And I should probably rename zonename to availability_zonename...

@Fryguy executable_name - what do you think?

I like this better.

And I should probably rename zonename to availability_zonename...

availability_zone_name , maybe?  ... I know it's inconsistent with hostname ...

yeah exectuable_name sounds great...very clear.

Summary of a conversation from https://gitter.im/ManageIQ/manageiq/providers

Keep hostname because the user wants to see what host the service is on, and there might not be an underlying OSP Infra layer to dereference cloud_service.host.hostname.
Remove availability_zone_name because this is a cloud object, and should definitely have a relationship in cloud_service, so cloud_service.availability_zone.name should always work.
Rename binary to executable_name for clarity.

Thanks @petrblaho !!

Looks great @petrblaho ...if you could align the symbols for style, then this is good to go.

LGTM :+1: will merge when green.


Advanced search needs front end to be implemented

The back end for the advanced search already exists, there should be a view for using this.

Possible implementation is like how http://www.greatschools.org/ works with showing list view and map view.


[BUG] Memory Leak

Hi, thank you for making this very useful project, it's really awesome.
I'm experiencing a suspected memory leak during my use.
When downloading a large batch of songs, I can see the memory usage slowly growing.
I'd like to ask you if there are any areas I can look at to help locate a fix for this memory leak?
My environment is Ubuntu 20 and python 3.8.
sample code:

import sys

from librespot.core import Session
from librespot.metadata import TrackId
from librespot.audio.decoders import AudioQuality, VorbisOnlyAudioQuality

account_user = ''
account_passwd = ''
session = Session.Builder().user_pass(account_user, account_passwd).create()

def download_from_spotify(track_id):
    spotify_track_id = TrackId.from_uri('spotify:track:' + track_id)
    stream = session.content_feeder().load(spotify_track_id, VorbisOnlyAudioQuality(AudioQuality.NORMAL), False, None)
    audio_buffer = stream.input_stream.stream().read()
    with open(f'{track_id}.ogg', 'wb') as f:
        f.write(audio_buffer)

def main():
    target = sys.argv[1]
    with open(target, 'r') as f:
        for line in f:
            track_id = line.strip()
            download_from_spotify(track_id)

if __name__ == '__main__':
    main()

Side note:

We DO NOT encourage piracy and DO NOT support any form of downloader/recorder designed with the help of this repository and in general anything that goes against the Spotify ToS.

Okay.

Hope you can notice this issue.
I've located the memory leak.
Python's GC doesn't seem to recycle the used buffer, so every time a song is loaded it stays in memory permanently.
Temporary solution
Add a method to clear the buffer in the InternalStream class
  class InternalStream(AbsChunkedInputStream):
          streamer: CdnManager.Streamer
+        def clear_buffer(self):
+            del self.streamer.buffer

Call the clear_buffer() method in the close method of the AbsChunkedInputStream class
 def close(self) -> None:
+    self.closed = True
     self.clear_buffer()
     with self.wait_lock:
         self.wait_lock.notify_all()

Then after each read(), call close() method.
def load_data_from_spotify(track_id):
    spotify_track_id = TrackId.from_uri('spotify:track:' + track_id)
    stream = session.content_feeder().load(spotify_track_id, VorbisOnlyAudioQuality(AudioQuality.NORMAL), False, None)
    audio_buffer = stream.input_stream.stream().read()
    # dosomething
    audio_buffer.close() # must add this, to clear buffer

Hope you can fix this problem in main branch code.

Yes, that is because the buffer has not been released.

Thanks for your work!

@kokarare1212 When will these changes be released?

OK, I have published.
https://github.com/kokarare1212/librespot-python/releases/tag/v0.0.6

@kokarare1212 Thank you.


Cannot provide Background steps for more than one Rule

Current behavior
Adding more than one Background steps  within one feature file, even when each Background is within different Rule section resolves in error
expected: #EOF, #TableRow, #DocStringSeparator, #StepLine, #TagLine, #ScenarioLine, #ScenarioOutlineLine, #Comment, #Empty, got 'Rule: ***'
(32:9): expected: #EOF, #TableRow, #DocStringSeparator, #StepLine, #TagLine, #ScenarioLine, #ScenarioOutlineLine, #Comment, #Empty, got 'Background:'

Desired behavior
Different Background steps should be supported within different Rule section.
Cucumber documentation: https://cucumber.io/docs/gherkin/reference/#background

You can only have one set of Background steps per Feature or Rule. If you need different Background steps for different scenarios, consider breaking up your set of scenarios into more Rules or more Features.

Test code to reproduce
Feature: Test Multiple Backgrounds
      Rule: First Section
           Background: First Background
               Given: Do something for the first scenario

           Scenario: Test first section
                 When I do something
                  ...

      Rule: Second Section
           Background: Second Background
               Given: Do something for the second scenario

           Scenario: Test second section
               When I do something
                ...

Versions

Cypress version: 8.5.0
Preprocessor version: 4.0.0
Node version: 12.7.8

It seems the real reason is that Rule: is not supported at all. See #143

@dpresiana You can probably try Cytorus which supports multiple rules. Check this for an example

Due to personal reasons, the previous maintainers of this package are stepping down and handing the reigns over to me, a long-time contributor to the project and a user of it myself. This is a responsibility I'm very excited about. Furthermore, I'd like to thank @lgandecki ++ for all the work that they've done so far.
Read more about the transfer of ownership here.
The repository has however moved and all outstanding issues are being closed. This is not a reflection of the perceived importance of your reported issue. However, if after upgrading to the new version, you still find there to be an issue, feel free to open up another ticket or comment below. Please make sure to read CONTRIBUTING.md before doing so.


error: torch 1.1.0 is installed but torch==1.3.1 is required by {'torchvision'}

When I install, this error appeared.
Must I change torch to 1.3.1?

You can also change the version of torchvision.

I have installed torchvision 0.2.2. But when I install mmdet, It will download torchvision 0.4.2 automatically.
here is the command:
Processing dependencies for mmdet==1.0rc0+b17f88c
Searching for torchvision
Reading https://pypi.org/simple/torchvision/
Downloading https://files.pythonhosted.org/packages/74/ee/d872c12ea508f9cca4cf0d7b91fc5a5e476cec77628a66795ec5a585a67f/torchvision-0.4.2-cp37-cp37m-manylinux1_x86_64.whl#sha256=276a385f2f5fe484bf08467b5d081d9144b97eb458ba5b4a11e4640389e53149
Best match: torchvision 0.4.2
Processing torchvision-0.4.2-cp37-cp37m-manylinux1_x86_64.whl
Installing torchvision-0.4.2-cp37-cp37m-manylinux1_x86_64.whl to /mnt/lustre/fangweizhen/anaconda3/envs/mmlab/lib/python3.7/site-packages
Adding torchvision 0.4.2 to easy-install.pth file
Installed /mnt/lustre/fangweizhen/anaconda3/envs/mmlab/lib/python3.7/site-packages/torchvision-0.4.2-py3.7-linux-x86_64.egg
error: torch 1.1.0 is installed but torch==1.3.1 is required by {'torchvision'}
Is that mean I must use torch 1.3.1?

You can also change the version of torchvision.
Since this does not relate to the mmdet, this issue is closed, feel free to reopen it if you have questions related to the codebase.

I have installed torchvision 0.2.2. But when I install mmdet, It will download torchvision 0.4.2 automatically.
here is the command:
Processing dependencies for mmdet==1.0rc0+b17f88c
Searching for torchvision
Reading https://pypi.org/simple/torchvision/
Downloading https://files.pythonhosted.org/packages/74/ee/d872c12ea508f9cca4cf0d7b91fc5a5e476cec77628a66795ec5a585a67f/torchvision-0.4.2-cp37-cp37m-manylinux1_x86_64.whl#sha256=276a385f2f5fe484bf08467b5d081d9144b97eb458ba5b4a11e4640389e53149
Best match: torchvision 0.4.2
Processing torchvision-0.4.2-cp37-cp37m-manylinux1_x86_64.whl
Installing torchvision-0.4.2-cp37-cp37m-manylinux1_x86_64.whl to /mnt/lustre/fangweizhen/anaconda3/envs/mmlab/lib/python3.7/site-packages
Adding torchvision 0.4.2 to easy-install.pth file
Installed /mnt/lustre/fangweizhen/anaconda3/envs/mmlab/lib/python3.7/site-packages/torchvision-0.4.2-py3.7-linux-x86_64.egg
error: torch 1.1.0 is installed but torch==1.3.1 is required by {'torchvision'}
Is that mean I must use torch 1.3.1?

No, but how do you installed your pytorch? Usually we use the command conda install pytorch torchvision -c pytorch following the guideline, this already install both pytorch and torchvision, so during compilation it will not install torchvision anymore.

PyTorch 1.1 works with torchvision 0.3.0 and PyTorch 1.3.1 works with torchvision 0.4.2. You need to install compatible versions of pytorch and torchvision. If you are not confident about the installation, just follow the official guide mentioned above.


Support runtime commands without az login

Is your feature request related to a problem? Please describe.
Investigate ways to support acr login, acr repository, and acr helm commands without logging to Azure. The only possible scenario is when both --username and --password are specified (service principal or admin user credentials).
There are currently a few blockers:

Login server is not consistent across different clouds or SKU (including legacy registries).
The request route is different depending on SKU (Classic vs. Managed).
It's preferred that access to Azure is validated.

Alternative approach when using SP is to first login to Azure using the SP, then use all acr commands without specifying --username or --password.
az login --service-principal -u http://azure-cli-2016-08-05-14-31-15 -p VerySecret --tenant contoso.onmicrosoft.com
az acr login -n MyRegistry

Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Given that ACR is in the process of deprecating Classic registries, we provided a workaround in https://github.com/Azure/azure-cli/pull/8455 that will be available in the next release.


Optimize Django Admin GUI

In order to facilitate the usage of the admin area, tools like https://github.com/alesdotio/django-admin-shortcuts or similar should be implemented.

Two ideads for login and dashboard :)


Increase receive timeout

Description
Increase the timeout for receiving in WS polling. This should hopefully reduce the amount of times it goes unavailable in Home Assistant.
Checklist

[ ] Tests covering the new functionality have been added
[x] Documentation has been updated OR the change is too minor to be documented
[x] Changes are listed in the CHANGELOG.md OR changes are insignificant

Summary by Sourcery
Increase the receive timeout for WebSocket polling from 60 seconds to 300 seconds to address unavailability issues in Home Assistant.
Bug Fixes:

Increased the receive timeout for WebSocket polling to reduce unavailability issues in Home Assistant.

Documentation:

Updated the CHANGELOG.md to reflect the increase in receive timeout for WebSocket polling.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (1ed794f) to head (5b9159b).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #224   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           15        15
  Lines         1174      1174
=========================================
  Hits          1174      1174

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


Add example HAProxy with logging

What's the status of this?  It's been here for a while, so we should either finalize or close it for now...


Save page title (extracted domain) along with credentials

Task/Issue URL: https://app.asana.com/0/0/1202630343540910/f
Description
When saving a credential, the title will be populated alongside the usual URL, username and password.
Steps to test this PR

[x] Visit a login page and attempt to sign in (e.g., https://trello.com/login)
[x] Choose to save the credential
[ ] Visit the credential management page and verify the title has been pre-populated with just the domain (e.g., trello.com)
[ ] Manually edit the title; verify it updates successfully

Current dependencies on/for this PR:

develop

PR #2136   👈

This comment was auto-generated by Graphite.

It seems like the list item in list view is not aligned with spec https://www.figma.com/file/e2sV7oVkDQX9brDIlRj52v/Autofill-(Android)?node-id=0%3A1. The title should be the website title and the subtitle should be the username. Atm, it's the other way around: the title is the username and the subtitle is the website title.


FEAT-#5936: support pandas 2.0.0

What do these changes do?

[x] first commit message and PR title follow format outlined here

NOTE:  If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

[x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] signed commit with git commit -s
[x] Resolves #5936
[ ] tests added and passing
[x] module layout described at docs/development/architecture.rst is up-to-date

Do not forget to add a check for the equivalence of parameters for groupby objects.

Do not forget to add a check for the equivalence of parameters for methods of str, and dt accessor.

It will be possible to close https://github.com/modin-project/modin/issues/3142.

I guess we can go straight to support 2.0.1

Current status:

test_api.py - works
test_concat.py - works
test_expanding.py - doesn't work
test_general.py - doesn't work
test_groupby.py - doesn't work
test_io.py - works
test_reshape.py - works

One of the remaining problems is converting a pyarrow table to a pandas dataframe while saving timestamp resolution.
@AndreyPavlenko can you suggest a solution?
Desired behaviour:
timestamp[s] -> datetime64[s]
timestamp[ms] -> datetime64[ms]
timestamp[us] -> datetime64[us]
timestamp[ns] -> datetime64[ns]

Current behaviour:
timestamp[s] -> datetime64[ns]
timestamp[ms] -> datetime64[ns]
timestamp[us] -> datetime64[ns]
timestamp[ns] -> datetime64[ns]

I guess we can start the review, as the pull request is large and it is very easy to miss something.

can you suggest a solution?

I don't know an efficient solution. Moreover, pandas forcibly converts all datetime64 values to datetime64[ns]:
array = pa.array([1678966200,<PHONE_NUMBER>,<PHONE_NUMBER>], type=pa.timestamp('s'))
at = pa.Table.from_arrays([array], names=['A'])
data = {f._name: f.to_numpy() for f in at.columns}
print(data["A"].dtype)
print(pandas.DataFrame(data).dtypes["A"])

datetime64[s]
datetime64[ns]

Moreover, pandas forcibly converts all datetime64 values to datetime64[ns]:

@AndreyPavlenko since pandas 2.0 the behavior is changing.
datetime64[s]
datetime64[s]

@YarShev @dchigarev ready for review

How to enable this option on workers https://pandas.pydata.org/docs/whatsnew/v2.0.0.html#copy-on-write-improvements?
Maybe to use ray.worker.global_worker.run_function_on_all_workers? https://github.com/modin-project/modin/pull/4603/files

Disallow passing non-keyword arguments:

reindex +
sort_values +
replace +
bfill +
ffill +
clip +
rename_axis +
set_axis +
where +
to_stata +
mask +
read_* +
pivot +
concat +
any +
interpolate +
sort_index +
drop_duplicates +
dropna +
reset_index +
set_index +
split/rsplit +
fillna +
drop +

Maybe to use ray.worker.global_worker.run_function_on_all_workers

This doesn't exist 🙃 but you could modify the modin-autoimport-pandas.pth so that it would do what you want, like
import os; os.environ.get("__MODIN_AUTOIMPORT_PANDAS__", None) and __import__("pandas").set_option("mode.copy_on_write", True)

@vnlitvinov ready for review

@anmyachev there are a few open questions left from previous review, but most of them are minor.
I think the only major blocker for me is my last comment, about updating QC in-place.

@dchigarev ready for review

@vnlitvinov, any comments?

@vnlitvinov, any comments?

@YarShev @dchigarev I see that @vnlitvinov comment about _update_inplace is still open
https://github.com/modin-project/modin/pull/5995/files/7edd9bdd8bcaa16466f666949d5314d460f610ea..6d7e8d260bf08a7bb304c258e902c237736aa043#r1219040039

@vnlitvinov, any comments?

@YarShev @dchigarev I see that @vnlitvinov comment about _update_inplace is still open
https://github.com/modin-project/modin/pull/5995/files/7edd9bdd8bcaa16466f666949d5314d460f610ea..6d7e8d260bf08a7bb304c258e902c237736aa043#r1219040039

@mvashishtha I fixed it but forgot to reply to the comment

@anmyachev thanks for the tremendous amount of effort into making this PR by the way!


Need steps for newly added labs 6 & 7

@Vchintal: we need guidance on content for labs 6 & 7: SSO with spring boot & JS apps.

Guidance provided and created two new issues


potential race condition with large upload tasks

There seems to be a concurrency related issue with large uploads.
I have a large upload task of ~60k files and 100GiB total size. I have done >10 of attempts to upload this, and it has always failed at some random point during the upload. The upload stops to error skicka: …filename…: googleapi: Error 404: File not found: …id…, notFound or couldn't run Google Drive query: googleapi: Error 400: Invalid Value, invalid. I've always started a new upload (ie. to a new, non-existing Drive path), since getting the list of files of an interrupted download is very slow. Also, the creation of all paths (the first step in upload) succeeds, the error happens when actually uploading the file contents.
After suspecting this, I have tried with both higher and lower nWorkers values. With higher values it seemed to die earlier, and with value 1 the transfer succeeded in first attempt:
106.30 GB / 106.30 GB [=====================================] 100.00 % 71h15m39s
skicka: preparation time 0s, sync time 71h 15m 55s
skicka: updated 58121 Drive files, 0 local files
skicka: 0 B read from disk, 0 B written to disk
skicka: 106.30 GiB uploaded (434.44 kiB/s), 0 B downloaded (0 B/s)
skicka: 129.20 MiB peak memory used

Unforunately this is hard to reproduce exactly. The skicka version was ac702d253ea3389ec71c41a2f85f218b1397e8d5 .. this version reproduced both error cases above and the final success (with nWorkers := 1).

It is possible that this is related to issue #12, rather than being a race condition (though it may be a race as well!).
Are any of the files you're uploading large (>1GB or so), and is it failing on large files? What can happen with large files is that, after an hour the authorization token automatically expires, and the regular Google Drive API upload call doesn't allow updating the token in the middle of an upload. What could be happening is that with 4 workers, then they are all fighting for bandwidth and the big files don't finish in an hour, but with 1 worker, the big files can get through.
There's an experimental skicka branch (chunked-uploads) that uses Google Drive's "chunked" upload API, which also allows token refresh in the middle of uploads.  I'll test that a bit today and see if I can get it merged, to see if that helps, pending more information about the above.  If the failures you're seeing are on smaller files, then this will take some more digging. :-)

Ok, this sounds possible. I don't know which file is failing, but I have 2 files ~2GB. Rest of the largest files are under 1GB. I have some of the old (failed) transfer attempts still stored in the Drive. Searching for the 10 largest files' names shows that they are only present in the successful upload target path. However, I don't know it it's just that the largest files are transferred last?
OTOH, I have quite decent uplink speed, so the 2GB file takes about 2.5 minutes to upload. Just tested with the largest file I have in that set:
skicka: Getting list of files to upload... Done. Starting upload.
1.93 GB / 1.93 GB [============================================================================================================================] 100.00 % 2m24s
skicka: preparation time 0s, sync time 2m 24s
skicka: updated 1 Drive files, 0 local files
skicka: 0 B read from disk, 0 B written to disk
skicka: 1.93 GiB uploaded (13.63 MiB/s), 0 B downloaded (0 B/s)
skicka: 2.39 MiB peak memory used

Nice uplink bandwidth!
I've reviewed the code for possible races and done a bunch of tests, including some with the go race detector (and have done so with a fast uplink), and haven't been able to reproduce this.  The fact that you can upload 2GB in a few minutes suggests that the issue isn't that the authorization token is expiring.
In the next day or so, I'll be landing the changes to skicka that use the resumable upload API for large files; my hope is that this will fix this issue for you.  (Though I'd still like to figure out the root cause.)
If you still see the issue with those changes, then we can see if we can get some tracing data that can give some more insight...

As of e8f2736003288da377c8192e63f476c940bb0f3e, resumable uploads for large files are checked in; if you have a chance to try it, I'd be curious to know if this fixes the issues you're seeing.

Great, thanks! I've just started a new upload test with that version. I'll report back asap; The directory creation takes 8..9 hours (btw, it's great to have a separate progress indicator for that!), before we get to the upload stage..

Ok, I had two transfers running at the same time, with same source folder: one was running ac702d253ea3389ec71c41a2f85f218b1397e8d5 .. modified so that nWorkers was set to 1 (as described above). This worked:
skicka: Getting list of files to upload... Done. Starting upload.
106.43 GB / 106.43 GB [=====================================] 100.00 % 66h39m47s
skicka: preparation time 0s, sync time 66h 40m 22s
skicka: updated 58611 Drive files, 0 local files
skicka: 0 B read from disk, 0 B written to disk
skicka: 106.43 GiB uploaded (464.95 kiB/s), 0 B downloaded (0 B/s)
skicka: 125.04 MiB peak memory used

The other was running e8f2736003288da377c8192e63f476c940bb0f3e (not modified). This had several issues during the "files"-stage (ie. when uploading, not during directory creation) and finally died to an API error.
During the transfer I got some of these:
skicka: …local file path…: googleapi: Error 404: File not found: …drive id…, notFound
And dozens of these - all these failures seemed to be happening to large files:
skicka: …local file path…: giving up after 6 retries: rate limit exceeded
However, there was no other error output, and the transfer continued, until it finally died to this:
Files: 58.53 GB / 95.70 GB [=====================>-------------] 61.16 % 4h36m0s
2015/01/18 04:35:47 couldn't run Google Drive query: googleapi: Error 400: Invalid Value, invalid

Ugh, that's pretty bad! Just to confirm, you were uploading to different destination folders, right?
Running two transfers at once is likely to cause Google Drive to issue more errors due to rate limits, but on the other hand, skicka should be as robust as possible to that case--that definitely doesn't excuse the errors.
I have a few ideas for changes that may help with this; I'll try to test them out in the next few days.
Thanks for your patience with this.

Yes, I was uploading to separate destination folders on My Drive.

Is there any chance you could retry this with the most recent version of skicka?  I still have yet to find any races, but the upload code has been improved substantially, so I'm hopeful/curious to know if things works well now.
If it still fails, if you could do a run like "skicka -debug upload /src/path /dst/path" and send the debugging output, that'd be a huge help.
Thanks!

Closing this out, under the thinking that the upload code has changed a lot since this was originally reported, and no similar issues have been reported.


Support override of SSL target name

I am receiving this error when trying to connect to my grpc server.
Error [ERR_TLS_CERT_ALTNAME_INVALID]: Hostname/IP does not match certificate's altnames ...
This is probably because my service certificate is using a different domain than the one I am connecting from. The official grpc client allows me to bypass this passing in the appropriate target using grpc.ssl_target_name_override. Is there a way to do this using connectrpc?

Hey! You can use the nodeOptions when creating a Transport and do this:
createGrpcTransport({
  baseUrl: "...",
  httpVersion: "2",
  nodeOptions: {
    checkServerIdentity() {
      return undefined;
    },
  }
})

More on checkServerIdentity: https://nodejs.org/api/tls.html#tlscheckserveridentityhostname-cert

This works, thanks :)


no logs

i don't get any logs with
docker-compose logs
if i run
without docker - i get logs

Hi, it seems there was a typo in a starting print parameter.
Should be fixed by: https://github.com/alexbers/mtprotoproxy/commit/00f87d955bbd83b0ad92811b5af25c86a4701e8f
The bug should be fixed. Could you recheck, please

hmm, on Mac i get logs, on Ubuntu - don't
user@server:~/mtprotoproxy/mtprotoproxy$ docker-compose logs
Attaching to mtprotoproxy_mtprotoproxy_1

did you launch docker-compose up --build ?

my fault, after rebuild it works


Letter bar index at the top of the dictionary page.

Adds a bar with all of the letters of the words to the top on the dictionary. When you click on a letter it skips to that portion of the dictionary.
To review this branch, please visit: https://rawgit.com/sande570/Moro-database/page_headers/index.html#/dict

@sande570 If this is what you were looking for, then go ahead and merge this PR and delete the branch.


Patch/#35 codacy issues

A: Patched the kind of obvious PMD issues and left out some that we have to discuss further and code marked as duplicate by codacy.
This is mostly:

switch (default, break)
package private
define fields/variables before the rest
unnecessary constructors

B: Added .codacy.yml to exclude ptolemy plot and .js and .py files.

Coverage decreased (-0.08%) to 17.356% when pulling 56c7cdd946469ff7981a50038b5fe64265f3b59f on patch/#35-codacy-issues into b6c946578870c664987e9496921f316dfc83337a on master.


Cursor actions are off-center

Use a cursor item like girder or teleporter, once you start moving far away from the center the location becomes offset.

Fixed with https://github.com/apetavern/sbox-grubs/commit/88175f074ad874cf08e96adcff583864288305d1


The IntelliJ project tree is sometimes hilariously wrong

Description of the bug:
Sometimes, in some folders, the project view is very wrong. Here's an example of something it's generating today:

Note that it's consistent per commit, that is, the state of the source tree at a given commit hash generates the same set of glitches.
What's the simplest, easiest way to reproduce this bug? Please provide a minimal example if possible.
I don't have a good repro for arbitrary repositories, but in our repository:

Clone the repository
Import the project
Click around, you'll find something busted

Which Intellij IDE are you using? Please provide the specific version.
IntelliJ IDEA Ultimate 2022.3.1 (Build 223.8214.52)
What programming languages and tools are you using? Please provide specific versions.
Java (coretto-1.8)
What Bazel plugin version are you using?
2<IP_ADDRESS>.1-api-version-223
Have you found anything relevant by searching the web?
I have not been able to find anything. :(
Any other information, logs, or outputs that you want to share?
No response

Hi @DeCarabas, Thanks for creating the issue. Could you please help us to reproduce the issue by sharing the sample repo? Thanks!

@Pavank1992 We don't have a sample repo at the moment. It is our organisation's codebase where we are seeing this issue.
I would suggest to reproduce issue in some large codebase which has multiple modules and those modules have src/main/java and src/test/java folder structure.
Once you do bazel sync the complete project you should be seeing this issue.

I was not able to reproduce this, I will keep it open if someone faced this and detected a clear reproducible pattern.


Create stream admin on stream creation.

[X] Add stream admin flag to model.
[X] Automatic make stream creator an stream admin.
[X] Allow stream admin to for basic stream management in backend.
[ ] Show stream admin in stream settings page.
[X] Allow stream admin for basic stream management in frontend.

This is ready to fresh review. Tests are failing because of non coverage. This implement suggested solution in #8101 and #3783

@rishig It will be great if you can give first fresh review to this. I will be adding frontend for this next, but the backend part is the one for actual permissions to stream admins, you can review whether stream basic management permission model is right or required any changes.
Issue #3783

Closing, since our plan for this area now does not involve a "stream admin" flag of this form.
See https://github.com/zulip/zulip/issues/19525 for details.
Thanks for all your work prototyping this @YJDave!


legacyPackages.x86_64-linux.pkgsCross.raspberryPi.tailscale fails to build (go-1.18.2 fails, even though it builds in other situations?)

Steps To Reproduce
This is from nixos-unstable current. I don't really understand what's up. Go builds, but not when used by tailscale:
nix build .\#legacyPackages.x86_64-linux.pkgsCross.raspberryPi.go_1_18
=> success

nix build .\#legacyPackages.x86_64-linux.pkgsCross.raspberryPi.tailscale
=> starts building go-1.18.2 (again??)
=> fails

So, I'm curious what piece I'm missing, and why it seems like Go does cross-build, but just not the way the tailscale package uses it?
full log: https://gist.github.com/colemickens/151301e1a39b315d4930d2e34cce5173
snippet:
warning: Git tree '/home/cole/code/nixpkgs/cmpkgs' is dirty
building '/nix/store/bmhdnyvac8w8xpd2dfmqpzfms1m36f07-go-1.18.2.drv'...
error: builder for '/nix/store/bmhdnyvac8w8xpd2dfmqpzfms1m36f07-go-1.18.2.drv' failed with exit code 2;
       last 10 log lines:
       > Building Go toolchain1 using /nix/store/xmyr6x9bd3brnn01am6caw8m3m3js8c9-go-bootstrap/share/go.
       > Building Go bootstrap cmd/go (go_bootstrap) using Go toolchain1.
       > Building Go toolchain2 using go_bootstrap and Go toolchain1.
       > Building Go toolchain3 using go_bootstrap and Go toolchain2.
       > Building packages and commands for host, linux/amd64.
       > Building packages and commands for target, linux/arm.
       > # runtime/cgo
       > /nix/store/bvy2z17rzlvkx2sj7fy99ajm853yv898-glibc-2.34-210/lib/libpthread.so: file not recognized: file format not recognized
       > collect2: error: ld returned 1 exit status
       > go tool dist: FAILED: /build/go/pkg/tool/linux_amd64/go_bootstrap install -gcflags=all= -ldflags=all= std cmd: exit status 2
       For full logs, run 'nix log /nix/store/bmhdnyvac8w8xpd2dfmqpzfms1m36f07-go-1.18.2.drv'.
error: 1 dependencies of derivation '/nix/store/2g3dccqpny7fjnfgxg7wg9x0d3i6rn45-tailscale-armv6l-unknown-linux-gnueabihf-1.24.2.drv' failed to build

the go failing is build -> host
(pkgsCross.raspberryPi.pkgsBuildHost.go_1_18)
while pkgsCross.raspberryPi.go_1_18 is host -> target
(pkgsCross.raspberryPi.pkgsHostTarget.go_1_18)
https://nixos.org/manual/nixpkgs/stable/#ssec-cross-dependency-implementation

https://github.com/NixOS/nixpkgs/pull/174612


Multiple "load image batch" doesn't change the indexes.

This might be a weird interaction with another extension im using, but I dont see how.
Basically I have two folders, "top half" and "bottom half".
I'm trying to put one image from "top half" and one image from "bottom half" through an img2img on the same workflow then combine at the end. So after the first two are done, the "load image batch" would load the next two images and those next two would go through the img2img process and combine at the end. But this doesn't happen.
Instead it only loads index image 1 from the "top half" and index image 1 from the "bottom half" for each job on the batch.
also edit: there's no error to go along with it.

If you notice, you're using the same label for both batches, which means the data from both is overriding the others. The labels are used to differentiate the batches.

Ah that makes sense.  Thanks.


Fix keystore format (WIP)

The (possible) reason Android wallet and MetaMask don't import
Web3swift uses aes-128-cbc mode as default. The above wallets don't seem to support this format.
More info
aes-128-cbc is the default mode for Keystore v2 format. Since v3 aes-128-ctr is the minimal requirement; but a good wallet should support both.
Solution
Just to make things simple, we'll change the default mode to aes-128-ctr.
TODO

See if it's possible to change default scrypt params to:

r: 8 (Web3swift default to 6)
p: 1
n: > 4096 (Web3swift default to 4096). A strong n would e as big as 262144 but that might be slow to generate thus need release mode benchmark.

Web3swift doesn't offer public interface to set default N 🤔


[CT-1850] [Bug] Using combo schema names in generate_schema_name() prevents adapter.get_columns_in_relation from working

Describe the bug
If I try to combine two values to define my schema using the generate_schema_name() macro, using union_relations produces SQL with a trailing comma in the select due to omitting columns
Steps to reproduce
Use this for generate_schema_name.sql:
{% macro generate_schema_name(custom_schema_name, node) -%}

    {%- set default_schema = target.schema -%}
    {%- if custom_schema_name is none -%}

        {{ default_schema }}

    {%- else -%}

        {{ custom_schema_name ~ '__' ~ default_schema }}

    {%- endif -%}

{%- endmacro %}

I've tried a few different things in the else. If I just use custom_schema_name OR default_schema, then it works fine. But if I try to combine them, I get the error. I've also tried combining them as the dbt docs recommend:
{{ default_schema }}_{{ custom_schema_name | trim }}
Expected results
I'd expect dbt_utils.union_relations to produce valid SQL with functioning unions.
Actual results
Compiled SQL in target/run:
create or replace transient table DATABASE_NAME.SCHEMA_NAME.TABLE_NAME copy grants as
(

(
    select
        cast('DATABASE_NAME.SCHEMA_NAME.TABLE_NAME' as TEXT) as _dbt_source_relation,

    from DATABASE_NAME.SCHEMA_NAME.TABLE_NAME

)

union all

(
    select
        cast('DATABASE_NAME.SCHEMA_NAME.TABLE_NAME' as TEXT) as _dbt_source_relation,

    from DATABASE_NAME.SCHEMA_NAME.TABLE_NAME

)

);

You can see the comma at the end of the first column of each select. This causes the model to fail. If I only use one value in the else block of the macro, then the columns that should be there appear in the compiled SQL and the model builds fine.
Screenshots and log output
Terminal output when I try to run the model:
001003 (42000): SQL compilation error:
syntax error line 14 at position 12 unexpected 'from'.

System information
The contents of your packages.yml file:
packages:
  - package: calogica/dbt_date
    version: 0.7.0

  - package: dbt-labs/dbt_utils
    version: 1.0.0

  # - package: dbt-labs/dbt_project_evaluator
  #   version: 0.2.0

  # - package: calogica/dbt_expectations
  #   version: 0.7.0

  - package: dbt-labs/codegen
    version: 0.9.0

Which database are you using dbt with?

[ ] postgres
[ ] redshift
[ ] bigquery
[X] snowflake
[ ] other (specify: ____________)

The output of dbt --version:
Core:
  - installed: 1.3.0
  - latest:    1.3.1 - Update available!

  Your version of dbt-core is out of date!
  You can find instructions for upgrading here:
  https://docs.getdbt.com/docs/installation

Plugins:
  - snowflake: 1.3.0 - Up to date!

Additional context

Are you interested in contributing the fix?

I'm not a dbt Labs employee so take this as an unofficial response, but this there a chance the models you're unioning haven't been rebuilt after you change the generate_schema_name() macro but before you run the model that unions things?
I'd expect this behavior if that were the case, since they'd exist in the original schema only and so retrieving the columns would come back empty.

Thanks for reaching out @jaredx435k2d0 !
Could you try out the suggestion from @jaypeedevlin? The most simple way to make sure everything everything is built is to do:
dbt build

(If necessary you could choose more targeted ways to make sure each of your objects exist within the expected database & schema.)
Example that worked for me
See below for example code and commands that worked for me.
Project files
packages.yml
packages:
  - package: calogica/dbt_date
    version: 0.7.0

  - package: dbt-labs/dbt_utils
    version: 1.0.0

  - package: dbt-labs/codegen
    version: 0.9.0

seeds/colors_primary.csv
category,color
primary,red
primary,yellow
primary,blue

seeds/colors_secondary.csv
category,color
secondary,orange
secondary,green
secondary,violet

models/colors.sql
{{
  config(
    materialized='table',
    copy_grants=true,
  )
}}

{{ dbt_utils.union_relations(
    relations=[ref('colors_primary'), ref('colors_secondary')],
) }}

macros/generate_schema_name.sql
{% macro generate_schema_name(custom_schema_name, node) -%}

    {%- set default_schema = target.schema -%}
    {%- if custom_schema_name is none -%}

        {{ default_schema }}

    {%- else -%}

        {{ custom_schema_name ~ '__' ~ default_schema }}

    {%- endif -%}

{%- endmacro %}

Run
This worked for me:
dbt deps
dbt seed
dbt run -s colors

But if I change the generate_schema_name macro and then run only the following, then I get an error similar to you:
dbt run -s colors

Looking at the logs
If you look in your logs where it says:
create or replace transient table DATABASE_NAME.SCHEMA_NAME.TABLE_NAME copy grants

Just a few lines above, you'll see something like:
describe table DATABASE_NAME.SCHEMA_NAME.TABLE_NAME

If you also see a line like the following, it will give a hint about the cause:
Table 'DATABASE_NAME.SCHEMA_NAME.TABLE_NAME' does not exist or not authorized

This makes sense. I really appreciate such a detailed response!
I just started trying to test this quickly before signing off for the day with our actual models but, due to the fact that we're midway through our dbt migration, it gets messy quickly.
I'll spend time testing thoroughly next week. Looking forward to the results!

I think this takes care of it!
I really appreciate your thorough reply and help here :)

Awesome to hear @jaredx435k2d0 !


Topic name hashed/encoded

Hi,
When using the turbo_stream_listen(<topic_name>) function, it would be nice if the topic was hashed/encoded.
The same applies to:
$hub->publish(new Update(
    <topic_name>,
    $this->renderView('chat/message.stream.html.twig', ['message' => $data['message']])
));

The same applies to:
// src/Entity/Book.php
namespace App\Entity;

use Symfony\UX\Turbo\Attribute\Broadcast;

#[Broadcast(topics: ['@="<topic_name>", 'books'], template: 'book_list.stream.html.twig', private: true)]
class Book
{
    // ...
}

For example, I used the following code:

This generated:
<div id="songs" data-controller="symfony--ux-turbo-mercure--turbo-stream" data-symfony--ux-turbo-mercure--turbo-stream-topic-value="songs_by_artist_21" data-symfony--ux-turbo-mercure--turbo-stream-hub-value="http://<IP_ADDRESS>:59632/.well-known/mercure">

I find it a shame to have songs_by_artist_21 and not a string for instance : ZmViZDE3OWE3Zjk4MDM2NTBmZDI1N2FiYWUwMjFmYmY4OTAzNzJkNjA5NTA1OGQ3NWI2NGRlYTkzNjYwODliYg==
I would like your feedback :)

Hi!
This may be something that is needed in some cases... where your channel names need to be obscured for some reason. But I think it's outside the scope if the package. Is there any spot where creating and using your own hashed topic name is not currently feasible due to some limitations in the library?
Cheers!

Is there any spot where creating and using your own hashed topic name is not currently feasible due to some limitations in the library?

Yes, for instance:
#[Broadcast(topics: ['@=service("App\\Service\\Generator").generate("songs_by_artist_" ~ entity.getId())'])]

Using a service does not work. Finally I got the error:
The function "service" does not exist around position 1 for expression container('App\\Service\\Generator').generate().

I suppose we could allow services to be fetched here - could you open a dedicated issue or PR?

Close for #446


Pages don't show up on "pages" in admin-tool

Hey there! I'm very new to orionjs. I try to create an easy website which is not too complex to begin.
My problem is, after I created a new page, it doesn't show up on the site, i can't manage them.
I tried it with this code

All Pages

{{# each pages }}

    {{ title }}

{{/ each }}

and my pages show up, but I think this behaviour isn't intended? It looks really ugly and isn't at the right place.
To add a template, i used this:
orion.pages.addTemplate({
    layout: 'layout',
    template: 'pagesSimple',
    name: 'Simple',
    description: 'Simple template'
}, {
    content: orion.attribute('froala', {
      label: 'Content'
    })
})

What did i miss?
Thx for help guys :)

I found mi mistake, I did not add the meteor package "session". It would be helpful if someone could write a note or something in the docs. Or even better, create some dependencies for the page-package, 'cause without session-package it won't work.


Persistent notification not disappearing even after leaving the app with the back button

I have to remove the app from recents. I don't think this is good behavior.
Why is a notification needed in the first place? I didnt even try to download anything

Because this is a cross-platform project, and the tooling isn't there for everything, this is a compromise that needs to be made for now. The notification is there because a foreground Service is used to manage the downloads on Android (to allow background downloading). Foreground Services require notifications.
In the future, I might revisit how and when the Service is started, but it's just going to be this way for now. You can always disable the notification.


using shap_values matrix in ordet to get value impact on target variable

Hi,
I have been using shap in order to analyze feature importance on a XGB classifier model that predicts CTR rate.
most of my predicting variables are categorical. and I was wondering if I could use the shap_values matrix in order to find how every category of some variable impact the CTR.
my code is:

params2 = {'n_estimators': 100,
               'learning_rate': 0.5,
               'seed': 0,
               'subsample': 0.8,
               'n_jobs': 50,
               'colsample_bytree': 0.8,
               'objective': 'binary:logistic',
               'max_depth': 10,
               'min_child_weight': 300,
               'gamma': 2,
               'max_delta_step': 6
               }

estimator = xgb.XGBClassifier(**params2)
mapper = DataFrameMapper([(i, None) if j != 'object' and j != 'bool' else (i,
                          [CategoricalDomain(
                           missing_value_treatment="as_value",
                           invalid_value_treatment="as_missing",
                           missing_value_replacement=
                           train_x[i].value_counts().idxmax(),
						   invalid_value_replacement=
						   train_x[i].value_counts().idxmax()),
                           LabelEncoder()])
                           for i, j in zip(train_x.columns.values, train_x.dtypes.values)],
						   input_df=True, df_out=True)

pmml_pipeline = PMMLPipeline([("mapper", mapper),("classifier", estimator)])
pmml_pipeline.fit(train_x, train_y.as_matrix())
train_x_encode = pmml_pipeline.named_steps['mapper'].fit_transform(train_x)
explainer = shap.TreeExplainer(pmml_pipeline.named_steps['classifier'])
shap_values = explainer.shap_values(train_x_encode)

shap_values_1 = pd.DataFrame(shap_values)
shap_values_1.columns = train_x.columns

shap_values_1.columns = ['shap_' + str(col) for col in shap_values_1.columns]
train_x.columns = ['vals_' + str(col) for col in train_x.columns]
all_data  = pd.concat([shap_values_1, train_x], axis=1)

domains = pd.DataFrame(all_data.groupby(['vals_domain']).agg({'shap_domain':'mean'})).reset_index()

when I'm expecting that higher value for each domain means higher impact on CTR .
Does that make sense ?
Thanks

SHAP values tell you the difference in the tree output (log odds for XGB classification) between when you know that feature vs. you don't know its value for the current prediction. So a large positive value means learning the value of that feature increases the model output log odds, while a large negative value means it reduces the model output log odds. If you want to find important features for each click I would sort by the absolute value of the SHAP values.


gui resource: example using HsQML for notes app

The talk gives a really good overview on how to build user interfaces with HsQML in Haskell.

Thanks!  This is great


Optimization: add a space after punctuation marks when there is none

This is an artefact from the solution graphs, which occurs rarely but would still be nice to fix.
Example of current output: "On ne peut pas ?Si, on peut."

Closing this as the cost outweighs the benefits, especially given how rare it is.


Any plans to expand with multilingual support?

Do you plan to expand to other languages? Especially the numbers in words to numerals depend largely on the language. Morphologically complex languages where word compounding is common, but there are inconsistencies in splitting (like ice cream ->  icecream example in paper), this tool would largely avoid unncessarily poor WER.

Dear @kavyamanohar, yes, enabling other languages is definitely necessary for wider adoption. Especially for things like numbers (as you mentioned). To the best of my knowledge, language-specific normalisation is not yet common. The Whisper and JiWER normalisers only apply language-specific normalisation to English (all other languages get the same basic normalisation).
This implementation was built for the evaluation in the paper and for the interactive demo in the browser (so I had to use typescript). I think the first step would be to rewrite the library in Python and to separate general normalisations from English specific ones and provide a structure for other languages.
I am also thinking about using an NLP parser like spaCy, which would add NER to every word and extend the possibilities to rank different errors (e.g. proper nouns).

@KorbinianKuhn, Thanks for the reply.  And congratulations on the paper. I must add that, the demo is so well designed.
Porting to Python would be a great idea. Ready to join hands on such a project.
The normalizer in Whisper is known to have some serious issues when applied to Indic languages.
Regarding multilingual normalizers, I have started working on such a project. Currently focusing on implementing rules specific for Malayalam language.
https://github.com/libindic/normalizer

Great, I appreciate that. As soon as I find some time I'll start with the basic python lib on a new branch and let you know.