App crash when try to open certain e-mails.

Expected behavior
Show mail content
Actual behavior
App crash
Steps to reproduce

Open "buggy" mail.
If doesnt work open second time same mail.
App crash. Done

Environment
K-9 Mail version:
5.704
Android version:
9.0
Account type :
IMAP
k9-log.txt
📑 Wybrane z tygodnia (aktualizacja).txt
Changed from eml to txt.

The crash doesn't seem to involve app code. What device are you using? Can you share a message that triggers the crash?

Nokia 2.1
Mail in attachmemt
Dnia 10 stycznia 2020 21:27:33 CET, cketti<EMAIL_ADDRESS>napisał(a):

The crash doesn't seem to involve app code. What device are you using? Can you share a message that triggers the crash?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/k9mail/k-9/issues/4437#issuecomment-573192858
-- Wysłane za pomocą K-9 Mail.

Mail in attachmemt

That doesn't work. You have to use GitHub's web UI to upload attachments.

Mail in attachmemt

That doesn't work. You have to use GitHub's web UI to upload attachments.

Mail in attachmemt

That doesn't work. You have to use GitHub's web UI to upload attachments.

Uploaded and changed file ext from eml to txt.

I'm not able to reproduce the crash. The message doesn't look particularly special either.

I will check on other phone. I'm guessing it's some kind of buffer overflow crash. Sometimes app crash when "show full message"
Dnia 10 stycznia 2020 22:27:06 CET, cketti<EMAIL_ADDRESS>napisał(a):

I'm not able to reproduce the crash. The message doesn't look particularly special either.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/k9mail/k-9/issues/4437#issuecomment-573212012
-- Wysłane za pomocą K-9 Mail.

Tested on Sony Xperia L1 with android 7. No bug at all.


Tests: Table

What kind of change does this PR introduce?
Added tests for Table and TableWrapper.
Does this PR introduce a breaking change?
No
#271

Codecov Report

Merging #299 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #299   +/-   ##
=======================================
  Coverage   70.41%   70.41%
=======================================
  Files          85       85
  Lines         649      649
  Branches      149      149
=======================================
  Hits          457      457
  Misses        192      192

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 b214aa0...7315832. Read the comment docs.


Create signup1.html(Updated the Signup Page)

I've updated the Signup Page with "Username" and "Confirm Password".Kindly check
Related Issue
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]
Fixes:  #(issue no.) #3127 solved
Description
[Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.]

Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes. Make sure to attach before & after screenshots in your PR.]
Checklist:

[x] I have made this change from my own.
[ ] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

@sailaja-adapa Also I've used inline CSS for this one


Make IRQ wrappers

Write the interrupt request line wrappers.
Resources:

OSDev Wiki
BrokenThorn
Bkerndev
Wikipedia

Moved to medium priority because #12 is more important.

I can send this to you tomorrow if you want

Closed in 4b22f19e7992fd7e3b1cf12f78de9739dc04361d


Docker postgres image 13.6 is stuck when trying to add pgloader to it

Hello Team,
I was trying to have a image based on 13.6 to add a pgloader using apt-get command and it is getting stuck
When I try the same using the latest postgres version it is not giving issue
I am attaching both the dockerfiles I used for build
test : using 13.6 tag for base image
FROM postgres:13.6
RUN  set -ex
RUN  apt-get upgrade
RUN  apt-get update
RUN  apt-get -y install pgloader
test1: using latest for base image
FROM postgres
RUN  set -ex
RUN  apt-get upgrade
RUN  apt-get update
RUN  apt-get -y install pgloader
Thanks and Regards
Haiderali Lakhani

Using this Dockerfile it builds fine
$ docker build --no-cache -t postgres:pgloader - << EOF
FROM postgres:13.6
RUN apt-get update && apt-get install -y pgloader; rm -rf /var/lib/apt/lists/*
EOF

Sending build context to Docker daemon  2.048kB
Step 1/2 : FROM postgres:13.6
 ---> a0b9c5d9ee48
Step 2/2 : RUN apt-get update && apt-get install -y pgloader; rm -rf /var/lib/apt/lists/*
 ---> Running in 9704d80cce54
Get:1 http://security.debian.org/debian-security bullseye-security InRelease [48.4 kB]
Get:2 http://deb.debian.org/debian bullseye InRelease [116 kB]
Get:3 http://deb.debian.org/debian bullseye-updates InRelease [44.1 kB]
Get:4 http://security.debian.org/debian-security bullseye-security/main amd64 Packages [228 kB]
Get:5 http://deb.debian.org/debian bullseye/main amd64 Packages [8,183 kB]
Get:6 http://apt.postgresql.org/pub/repos/apt bullseye-pgdg InRelease [91.7 kB]
Get:7 http://deb.debian.org/debian bullseye-updates/main amd64 Packages [14.6 kB]
Get:8 http://apt.postgresql.org/pub/repos/apt bullseye-pgdg/main amd64 Packages [268 kB]
Get:9 http://apt.postgresql.org/pub/repos/apt bullseye-pgdg/13 amd64 Packages [2,564 B]
Fetched 8,996 kB in 2s (4,900 kB/s)
Reading package lists...
Reading package lists...
Building dependency tree...
Reading state information...
The following additional packages will be installed:
  freetds-common freetds-dev libct4 libsybdb5
The following NEW packages will be installed:
  freetds-common freetds-dev libct4 libsybdb5 pgloader
0 upgraded, 5 newly installed, 0 to remove and 42 not upgraded.
Need to get 27.6 MB of archives.
After this operation, 33.7 MB of additional disk space will be used.
Get:1 http://deb.debian.org/debian bullseye/main amd64 freetds-common all 1.2.3-1 [32.5 kB]
Get:2 http://deb.debian.org/debian bullseye/main amd64 libct4 amd64 1.2.3-1 [164 kB]
Get:3 http://deb.debian.org/debian bullseye/main amd64 libsybdb5 amd64 1.2.3-1 [191 kB]
Get:4 http://deb.debian.org/debian bullseye/main amd64 freetds-dev amd64 1.2.3-1 [282 kB]
Get:5 http://apt.postgresql.org/pub/repos/apt bullseye-pgdg/main amd64 pgloader amd64 3.6.9-1.pgdg110+1 [26.9 MB]
debconf: delaying package configuration, since apt-utils is not installed
Fetched 27.6 MB in 3s (8,641 kB/s)
Selecting previously unselected package freetds-common.
(Reading database ... 12041 files and directories currently installed.)
Preparing to unpack .../freetds-common_1.2.3-1_all.deb ...
Unpacking freetds-common (1.2.3-1) ...
Selecting previously unselected package libct4:amd64.
Preparing to unpack .../libct4_1.2.3-1_amd64.deb ...
Unpacking libct4:amd64 (1.2.3-1) ...
Selecting previously unselected package libsybdb5:amd64.
Preparing to unpack .../libsybdb5_1.2.3-1_amd64.deb ...
Unpacking libsybdb5:amd64 (1.2.3-1) ...
Selecting previously unselected package freetds-dev.
Preparing to unpack .../freetds-dev_1.2.3-1_amd64.deb ...
Unpacking freetds-dev (1.2.3-1) ...
Selecting previously unselected package pgloader.
Preparing to unpack .../pgloader_3.6.9-1.pgdg110+1_amd64.deb ...
Unpacking pgloader (3.6.9-1.pgdg110+1) ...
Setting up freetds-common (1.2.3-1) ...
Setting up libsybdb5:amd64 (1.2.3-1) ...
Setting up libct4:amd64 (1.2.3-1) ...
Setting up freetds-dev (1.2.3-1) ...
Setting up pgloader (3.6.9-1.pgdg110+1) ...
Processing triggers for libc-bin (2.31-13+deb11u3) ...
Removing intermediate container 9704d80cce54
 ---> 13c3905854a7
Successfully built 13c3905854a7
Successfully tagged postgres:pgloader

You could also try asking over at the Docker Community Forums, Docker Community Slack, or Stack Overflow. Since these repos aren't really a user-help forum


Typo in 3.07

// TODO (5) Implement OnClickListener in the ForecastAdapterViewHolder class
should be
// TODO (5) Implement OnClickListener in the NumberViewHolder  class

// TODO (6) Override onClick, passing the clicked item's position (getAdapterPosition()) to mClickHandler via its onListItemClick method
should be
// TODO (6) Override onClick, passing the clicked item's position (getAdapterPosition()) to mOnClickListener via its onListItemClick method

I second this.

The typo is also present in the solution part.

Thanks! Fixed in https://github.com/udacity/ud851-Exercises/commit/0d988fb0047e08fd73fe0ecd4e3b92a91096b53e


.Net Core support

Please add .Net Core  / NetStandard support.

As of version 8.0.0 we support .Net Standard v2.


[CI] [Docs] Run docs build for PRs CI

Is your feature request related to a problem? Please describe.
Some type issues will cause type doc to fail on master to build the docs, thus fail to build
Describe the solution you'd like
Tweak CI to run doc building on PRs CI
Describe alternatives you've considered
Split docs / webb.js publishing
Additional context
Webb.js publishing don't work since this PR https://github.com/webb-tools/webb.js/pull/232

docs and webb.js publishing are split, and new builds of webb.js are able to be published. This issue is still useful to track the failure of docs generation. Perhaps as a workaround we can try not to include the @webb-tools/types package for docs generation.

@nepoche @AhmedKorim the docs will pass if add // @ts-nocheck to packages/types/src/interfaces/types-lookup.ts. I had done this in the past but because they are auto generated I guess it had removed it?


Only trigger preprocess if ingest if there are new sequences or new annotations

Context
From watching behavior of GISAID fetch and ingest action and from reading through the script ingest-gisaid: https://github.com/nextstrain/ncov-ingest/blob/master/bin/ingest-gisaid#L166, I'm pretty sure that preprocess is always triggered.
This means that we can't have ingest-gisaid running every ~6 hours to better catch updates to the endpoint by GISAID. (Unless we want a whole lot of wasted processing, and even then it's going to overwrite manual updates)
Description
Running ingest-gisaid should compare metadata.tsv.gz produced during the script's run to the file metadata.tsv.gz on S3, along with a similar comparison of sequences.fasta.xz. Only if hashes of either metadata or sequences are different should preprocess be triggered.
cc @tsibley @joverlee521

Should we push this diff even further upstream and only run ingest when there's new/updated GISAID data?
For ingest-gisaid --fetch, we can diff the gisaid.ndjson against the gisaid.ndjson.xz on S3 and only run ingest + preprocess if there are updates.

Is it fair to say this issue has been resolved by https://github.com/nextstrain/ncov-ingest/pull/237 speeding up the ingest workflow? Or is this still an issue we need to resolve by as described above?

@huddlej I think that's fair, esp. given this comment from @trvrb on Slack:

I think rerunning phylogenetics workflow is not too big of a deal. Especially as we expect to continue to make refinements to ncov that would be good to be live (like https://github.com/nextstrain/ncov/commit/662ce24345454170cc914d8d872bdeff94752f8a).

Plus, it seems like things have been ~fine without this for the past ~6 months.


Adding a LG commercial TV

I have 2 LG commercial TVs and I am trying to connect them but am unable to access a way to generate the keycode that the module needs to add it. It would be nice if there is another way to be able to add them.
Thanks

Does not the LG industrial or commercial documentation qualify how to use the remote network control?  Key would be in the printed documentation? PDF?  or instructions with TV to generate keycode as needed?


perf: remove @[inline] from NameMap.find?

This @[inline] causes Lean to respecialize RBMap.find? to NameMap at each call site of NameMap.find?, creating lots of unnecessary duplicate IR.

@Kha, do you have any idea why interactive test 1403 would be failing with a mutex error:
Fatal glibc error: pthread_mutex_lock.c:94 (___pthread_mutex_lock): assertion failed: mutex->__data.__owner == 0

This seems like it could just be some generally flaky issue with the server. I can't imagine how this change could cause this.

!bench


Bug 1503233 - Add liveness and readiness checks to ASB dc

This PR adds liveness and readiness checks to the ASB template on the ASB pod.
Changes proposed in this pull request

Adds liveness and readiness check on /healthz

ACK


Make sure the side bar works with Valo

But do this after #95

New side bar implementation especially designed for Valo has been added to master.


R47 credential holder definition

R47 states See the W3C Verifiable Credential Standard [[W3C VC]](https://github.com/eea-oasis/baseline-standard/blob/main/core/baseline-core-v1.0-psd01.md#w3c-vc) for a definition of credential holder.
Upon searching the document linked, no definition of credential holder appears (the term "credential holder" is not present in the document ?!).
The specification linked defines the term "holder". I'd be interested to compare the term to the definition of principal owner used elsewhere.

will link term directly to W3C definitionv-> in PR

Closing. Addressed with merged PR https://github.com/eea-oasis/baseline-standard/pull/173


Add bulk deletion support to ark restore delete

Modify ark restore to bulk delete restores, so that a user could do bulk deletes using --all/-l or  --selector. This would be in line with kubectl.
Examples:

delete restores named "restore-1" and "restore-2"
ark restore delete restore-1 restore-2

delete all restores with label foo: bar
ark restore delete --selector foo=bar

delete all restores
ark restore delete --all

This is related to this issue: https://github.com/heptio/ark/issues/295

Need to resolve
https://github.com/heptio/ark/issues/748#issuecomment-411474132

Edited the description, please check!

I'd like to take a stab at this!

@carlisia To clarify: is -l an alias for --all or --selector?
Kubectl behavior uses -l an alias for --selector and I think we should match that.
  -l, --selector='': Selector (label query) to filter on, not including uninitialized ones.

Hi @shubheksha!  Thanks for volunteering to tackle this and #748.  You'll find that they're both likely similar to https://github.com/heptio/ark/pull/745.
Please read over https://github.com/heptio/ark/blob/master/CONTRIBUTING.md and you're welcome to join us in the #ark-dr channel on the Kubernetes slack server.

@rosskukulinski -l is short for --selector. I'll adjust the description.


Custom element doesn't exist: bar-card.

Hi,
I installed the bar-card using HACS and restarted HA. I tried it with a simple card:
type: 'custom:bar-card'
entity: sensor.climate_buro_rssi_device
title: Severity

It does not work. I get the error: Custom element doesn't exist: bar-card.
The resource is integrated in the Lovelace Configuration....
I use the following versions:
HASSIO: Home Assistant OS 5.9
HA: 2020.12.7
HACS: 1.9.0 (Frontend:<PHONE_NUMBER>2459)

Same exact Issue as gmarjano1975

Version
2020.12.0 <--Apparently there is an update! :)

Installation Type
Home Assistant Container

Installed HACS yesterday.
Thank you!

Same exact Issue as gmarjano1975

Version
2020.12.0 <--Apparently there is an update! :)

Installation Type
Home Assistant Container

Installed HACS yesterday.
Thank you!

My issue fixed itself.  I changed nothing and today it works! Thanks you.

My issue fixed itself.  I changed nothing and today it works! Thanks you.

Works again....

Works again....


git_submodule-update shouldn't overwrite submodule remotes sometimes...

git_submodule-update runs git submodule sync which wasn't an issue, until we started having multiple remotes.
Now it doesn't just overwrite origin, but it overwrites the remote that the current branch is tracking. Both behaviors can be troublesome.
Need to come up with a better methodology and manually sync. Maybe our own default remote name, say original?
@drewgilliam @scott-vsi @NoahRJohnson

Hopefully I can find a way to create a specific remote just for this feature, and default to using it instead of the "active tracking" using a -c override?


(PUP-6235) Change PMT tests to examples in build test

The PMT skeleton has been changed to name the 'tests' directory
to 'examples' in commit 420cb9f6bb. This caused the
'build_should_not_allow_symlinks' test to fail because the test
was using the 'tests' directory.
This commit renames the 'tests' directory to 'examples' in the
'build_should_not_allow_symlinks' test.
[skip-ci]

ugh.  😞


Logout after password change

If you change your password and reload the page, you have to relogin.

When sending the password change request to /rest/users/set_password everything is ok.
The next request (for instance clicking on a project button) will be rejected with 403 - forbidden. The interesting thing is the OpenSlidesSessionID-Cookie: It is included in the second request, but the response header is:
Set-Cookie: OpenSlidesSessionID=""; expires=Thu, 01-Jan-1970 00:00:00 GMT; Max-Age=0; Path=/

So the session handling on the server is broken. I think the session id changes with the password change, so the password is part of the session id creation.
@ostcar Do you have an idea with the sessions?

Ok, I looked in the django implementation. See this line. The session key is generated with the password. If the password changes, the session id changes too. I think we have three possibilities:

Override the session key generation not to use the password
when processing the set_password request, change the sessionid from the user
force a reload on client and the user has to relogin

Another question: Why do we have a custom AuthenticationMiddleware if it is not enabled in the global settings?

Our AuthenticationMiddleware was removed from @normanjaeckel in this commit: https://github.com/OpenSlides/OpenSlides/commit/d7936c53e3124b9c6d2dc0e56b8a5191c341ca31#diff-a8f854a332fc77a1ed965fc94d7925df
Maybe he can tell us why he did it. It is strange, that the code works at the moment. Now, request.user is always a DjangoAnonymousUser and not our AnonymousUser. If anything works fine with the DjangoAnonymousUser, then maybe we can remove our Sessions backend, AnonymousUsers and any function that we added for it?
For this issue I would prefer the solution 2. I don't like to change a django security feature and I also don't like a forced logout.

Why not just use update_session_auth_hash from django.contrib.auth on server side in the password change view?
I do not know why I changed the middleware settings. Maybe I thought that we do not have any problems with the anonymous user any more ... Just remove the old code.

With option two I thought exactly on @normanjaeckel proposal.
But why does the DjangoAnonymousUser has permissions when the check in has_perm() should return false?
I make a debug print giving me the permission string, user type and the result has_perm. When using the anonymous user (type: DjangoAnonymousUser) this function always return false but in the client the anonymous user have the permission of group 1. Something is broken and does not care about the result of has_perm().
I tested a little bit:

Anonymous enabled; one window as anonymous and one with the admin.
The admin sees 15 agenda items; the anonymous none (he should see all).
The admin toggles 'internal entry'. Now the anonymous gets an autoupdate and the item is in his list (always with correct internal entry state)

The permissions works somehow (Why??):

The anonymous has no permissions for users. The list is empty (ok). Admin changes a user; No autoupdate for anonymous.
The anonymous gains the users.can_see_names permission. Reload of client. Now the list is still empty, but with an autoupdate from admin, the list gets the changes entry (same as the agenda item)

Here, the permissions are broken:

Changed the access_permissions to always return false. So nobody has the permission to see motions. Anonymous as well as the admin have access.

@ostcar @emanuelschuetze Can you recreate all these things? If yes, there have to be something to fix...

Please retest with current master after #2909 is merged.


Fix the issue of incorrect macro definition IFCALLRESULT and fix show black window when connect transparent app in windows server 2012r2

Fix the issue of incorrect macro definition IFCALLRESULT:
Correct code that looks similar to this:
if (!IFCALLRESULT(TRUE, pointer->New, context, pointer))
goto out_fail;
this will always run "goto out_fail".
fix show black window when connect transparent app in windows server 2012r2:
when specify RDPGFX_CAPS_FLAG_THINCLIENT and RDPGFX_CAPS_FLAG_SMALL_CACHE, windows server 2012 r2 will not send
surface command with codec id RDPGFX_CODECID_ALPHA.
my command line is: wfreerdp.exe /v:xx /u:xx /p:xx /app:program:xx /gfx:rfx /f

Please amend the second commit message to not extend over 72 characters. Use the body of the commit message for further details (also don't exceed the 72 char limit per line there too, just do new lines).
The title of the commit message must not exceed 72 characters. Second line must be empty. The body of the commit message starts at the third line.

when specify RDPGFX_CAPS_FLAG_THINCLIENT and RDPGFX_CAPS_FLAG_SMALL_CACHE, windows server 2012 r2 will not send
surface command with codec id RDPGFX_CODECID_ALPHA.

Hmm.. do we actively do this somewhere? The RDPGFX_CAPS_FLAG_THINCLIENT and the RDPGFX_CAPS_FLAG_SMALL_CACHE flag should not be combined together (this is afaik also mentioned in the docs). So, if we currently do that, that would indeed be wrong.
Using the RDPGFX_CAPS_FLAG_THINCLIENT flag btw implicitly includes the RDPGFX_CAPS_FLAG_SMALL_CACHE effect.
I.e. both flags should not be used at the same time, but the small cache part is also part of the thinclient flag.

@2fly2 the second change, you have an issue with rails there? might be a good idea to split this into two commits

@freerdp-bot test

@2fly2 sorry, did accidentally close that, created a new pr with your changes (but clang-formatted)


bug: Commenting throws an error in lua files

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.9.0-dev-1115+g446c353a5
Operating system/version
Arch Linux
Describe the bug
Commenting lines is throwing errors.
E5108: Error executing lua ...ig/nvim/.repro/plugins/mini.comment/lua/mini/comment.lua:155: nvim_exec(): Vim(lua):E5108: Error executing lua ...ig/nvim/.repro/
plugins/mini.comment/lua/mini/comment.lua:387: attempt to call field 'get_filetypes' (a nil value)
stack traceback:
        ...ig/nvim/.repro/plugins/mini.comment/lua/mini/comment.lua:387: in function 'traverse'
        ...ig/nvim/.repro/plugins/mini.comment/lua/mini/comment.lua:398: in function 'get_commentstring'
        ...ig/nvim/.repro/plugins/mini.comment/lua/mini/comment.lua:340: in function 'make_comment_parts'
        ...ig/nvim/.repro/plugins/mini.comment/lua/mini/comment.lua:210: in function 'toggle_lines'
        [string ":lua"]:1: in main chunk
        [C]: in function 'cmd'
        ...ig/nvim/.repro/plugins/mini.comment/lua/mini/comment.lua:155: in function <...ig/nvim/.repro/plugins/mini.comment/lua/mini/comment.lua:123>
stack traceback:
        [C]: in function 'cmd'
        ...ig/nvim/.repro/plugins/mini.comment/lua/mini/comment.lua:155: in function <...ig/nvim/.repro/plugins/mini.comment/lua/mini/comment.lua:123>

Steps To Reproduce

Press gcc in normal mode

Expected Behavior
Line commented out
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  "folke/LazyVim",
  "echasnovski/mini.comment",
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

require("mini.comment").setup()

vim.cmd.colorscheme("tokyonight")
-- add anything else here

Works fine for me in lua files. Since this is not a LazyVim issue, please file an issue at mini.nvim.

I see you're using an older nightly build. Pretty sure that's the issue. LazyVim and most plugins are compatible with either the stable releases or the latest nighly builds. not with old nightly builds.


Remove MIG layout test menu

Fixes #1012

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

Curiously, would it be an idea like in the linked issue to just hide it somewhere out of the way if somebody is tweaking with NUI later and wants to see a before-after impact on all the elements at once?
Maybe we should start thinking about TeraEd in that sense. If you launch with TeraEd, leave the test layout menu active. If not, leave it disabled.

Alright - I left the code in engine, but remove the button from the main menu. You can still open the screen with the new showScreen command:

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

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

Excellent, thanks! I went ahead and added a little more help text so we won't forget the name of the test screen :-)

Great!


[SR-3082] Throwing constructors with a method immediately called on them result in warning

Previous ID
SR-3082

Radar
None

Original Reporter
@khanlou

Type
Bug

Environment
Swift 3, Xcode 8.1

Additional Detail from JIRA

Votes
0

Component/s

Labels
Bug

Assignee
None

Priority
Medium

md5: 2b4e024ed59312e8b6a1f66407e28d1e

relates to:

SR-1681 spurious "unused result" warning in Swift 3

Issue Description:
A throwing constructor that has a method with a `discardableResult` immediately called on it results in a warning:
        struct ThrowingConstructor {
            init() throws {

            }

            @discardableResult
            func method() -> String {
                return "test"
            }
        }

        try? ThrowingConstructor().method()  // Warning: Result of 'try?' is unused

This issue can't be reproduced in a playground, I think because playgrounds don't have unused result errors turned on.
The warning can be silenced by using `_ =`. It can also be silenced with parentheses: `(try? ThrowingConstructor())?.method()`

Hopefully, this can be fixed soon


create IStartup and IShutdown interfaces for device

to be able to inject for example locale update and return for each device automatically

obsolete, we found much better way to inject any functionality


added docker version

As in the installation page I could not find any packages for ubuntu
I tried to create a basic docker image using archlinux as the base image, so now I can run it in ubuntu and anywhere else that has docker installed.

Hi, Yazi provides a snap package for Ubuntu users, does it work for you?

hey
thanks for the reply
on this url https://yazi-rs.github.io/docs/installation/ I couldn't find anything for ubuntu
still I tried to dockerize it and it's working just fine
thanks for a really handy tool, great job

on this url https://yazi-rs.github.io/docs/installation/ I couldn't find anything for ubuntu

I think Ubuntu users will use Official binaries or build Yazi from source code, or install via Cargo.

still I tried to dockerize it and it's working just fine

Thank you for adding it, but I have some concerns about Dockerizing it:
I want to know if the system clipboard (copy file path by cc) and image preview (especially when used with ueberzugpp) work after Dockerizing Yazi? - do they both need to call commands installed on the host machine. If so, I will accept this PR ;)

hey
can u name the exact packages needed to be installed on host machine?
apparently I couldn't also edit files because vi was not installed
I could get this solved by adding vi in Dockerfile
if you can name the system commands used in yazi I can also add it to the Dockerfile

wl-clipboard for Wayland system clipboard support
xclip / xsel for X11 system clipboard support
pbcopy / pbpaste for macOS system clipboard support, comes with macOS by default
ueberzugpp for image preview under X11/Wayland

hey
it was new year in here and i couldn't spend much time on this
but wl-clipboard and xclip / xsel are fine to be installed in the arch linux as the base image but the other ones are somehow problematic and needs some r&d, and to be honest im not sure about the end result that it would work or not

Hi
No, what I mean is not that it can be installed in Docker, but that it can work in Docker - that is, when the user presses c => c to copy the path, Yazi will invoke the clipboard command listed above, but it runs in Docker, and I'm not sure if the copied content can be somehow passed to the host machine, as I'm not familiar with how it works.
If this doesn't work, then I can't accept this PR because it would cause these features to not work.

I'm going to close this because it still needs some work and hasn't seen any activity recently. @sinawic thanks for the effort!


Refactor: make state struct use named fields

In the future we may want to have different structs for each phase, but for now this at least allows us to avoid many de-structuring calls as well as the nasty state.6 thing.


ISP DNS can leak with default setup

I think it would be a good idea to add to the documentation for users to do a dnsleaktest. This is because people will plug in various ISP configed / locked down devices. For example I plugged in a router / modem from British Telecom that is locked down so I can't change the dns settings or put it in bridge mode. My dns leaked to the ISP dns server 😱
It's a simple fix by changing the "Use DNS servers advertised by peer setting" on the WAN interfaces.
Note: some streaming services e.g. Amazon are using DNS leaks to detect VPN's.

Using DNS forwarding next release, for older version users example:

Next version will forward to inactive ADGhome by default.

Fixed in latest release. (no resolvconf)


Main window won't open after update to 0.58.0

Microsoft PowerToys version
0.58.0
Running as admin

[ ] Yes

Area(s) with issue?
General
Steps to reproduce
I had been using powertoys 0.57.0 from a normal user account.
Today, it prompted me to update it. After the update I removed .NET 3.1 runtime because the release notes said it is no more required.
(I didn't try opening the main window then)
Now,
The Powertoys and Powertoys Awake icon show in the system tray.
Exiting Powertoys from the tray and starting again "Powertoys (Preview)" [From search] leads it to show the icon again in the tray.
All the utilities are working fine. Even the window for FancyZones opens as normal.
UPDATE: MouseHighlighter and Shortcut remaps aren't also working (while Key remaps are)
✔ Expected Behavior
When PowerToys icon is showing in the system tray,
Starting PowerToys from search, from PowerToys Run or from the system tray (settings) should open the main window.
❌ Actual Behavior
But the main window doesn't open and the cursor just shows that loading symbol for few secs and nothing happens after that.
I tried rebooting and also tried running it as administrator but nothing helps.
Other Software
No response

I am also having an issue where I cannot open the settings window after updating.

Runner logs when trying to launch "PowerToys (Preview)" from Windows Search:
[2022-05-06 09:59:33.920382] [p-2176] [t-544] [info] runner logger is initialized
[2022-05-06 09:59:33.920426] [p-2176] [t-544] [info] Running powertoys with cmd args:
[2022-05-06 09:59:33.920520] [p-2176] [t-544] [error] Can't convert string value= to ESettingsWindowNames

/dup #18015


Issue with PyCaret and pdf generation

Parallel processes are not optimized. Multiple people using the app at the same time leads to the app throwing an error.
Try to fix by cleaning up all variables and files from disk ---

Workaround: run the app locally. :)

Pycaret issue related to https://github.com/pycaret/pycaret/issues/895


Add vLLM support

https://www.reddit.com/r/LocalLLaMA/comments/14em713/just_released_vllm_inference_library_that/?utm_source=share&utm_medium=ios_app&utm_name=ioscss&utm_content=2&utm_term=1
Just released - vLLM inference library that accelerates HF Transformers by 24x
Posted in r/LocalLLaMA by u/yanjb • 48 points and 8 comments
LLM is an open-source LLM inference and serving library that accelerates HuggingFace Transformers by 24x and powers Vicuna and Chatbot Arena.
Github: https://github.com/vllm-project/vllmBlog post: https://vllm.ai/
Edit - it wasn't "just released" apparently it's live for several days
https://preview.redd.it/nzceocfbg87b1.png?width=1532&format=png&auto=webp&v=enabled&s=30bcee178214cb8c88e05c37db1b2215c0a1bf73

Faster than TGI by 3-4x, does it have continuous batching?

Please recheck the installation guide at https://github.com/h2oai/h2ogpt/blob/main/docs/README_InferenceServers.md
In the local install I think we missed out:
pip install vllm
Due to which when one runs the inference server;  error is given that Module vllm not found.

@rohitnanda1443 The block for installing it is there, but it's a bit old and setup for helping Mixtral work when it wasn't in vLLM yet.  I cleaned it up.


Provide a means to add labels and annotations on stunnerd pods

Currently there is no configurable way to add user-defined labels and annotations to the stunnerd pods launched by STUNner to run the data plane for a Gateway. This would be useful, for instance, to enable/disable automatic sidecar proxy injection (e.g., for linkerd) or specify the sidecar proxy parameters.
One possible workflow to support this feature would be to create the Gateway from a separate Dataplane CRD and then simply copy all labels and annotations from the Dataplane into the pods created for the Gateway. If you want different labels/annotations for different Gateways, launch them from a different Dataplane via the GatewayConfig.
This issue is to initiate the discussion on this feature and track implementation.

Eventually the decision was made to not copy labels/annotations from the Dataplane to the stunnerd Deployments. First, the same Dataplane is often shared across multiple Gateways and it seems wrong to copy the same labels/annotations to every Deployment spawned from a Dataplane. Instead, copying from the Gateway lets users customize stunnerd Deployment labels/annotations on a per-Gateway basis. Second, changing this may break existing deployments that rely on this behavior.
Instead, we cleaned up the way labels and annotations are copied between Gateways and Deployments, see 9a6729d.
Rules for the labels/annotations on the stunnerd Deployment:

mandatory labels/annotations are enforced (see details in internal/renderer/dataplane_util.go)
labels/annotations copied verbatim from the parent Gateway unless in conflict with the mandatory labels/annotations
labels/annotations manually added to the deployment are retained unless in conflict with the above

Rules for the labels/annotations on the pod template for the stunnerd Deployment:

pod template labels are fixed and there is no way to manually add labels to the pods: this is needed to make sure that the operator reliably finds the pods it manages
mandatory pod template annotations are enforced (see details in internal/renderer/dataplane_util.go)
pod template annotations copied verbatim from the parent Gateway unless in conflict with the mandatory annotations
pod template annotations are copied from the manually added pod template annotations unless in conflict with the above

After repeated requests from users dissatisfied with the current state-of-the-art that there is no way to customize stunnerd  pod labels/annotations, we again changed the corresponding behavior of the operator.
Previously we have ruled out copying labels/annotations from the Dataplane on the basis that "the same Dataplane is often shared across multiple Gateways and it seems wrong to copy the same labels/annotations to every Deployment spawned from a Dataplane." In addition, we found that Dataplane resources often get, from our perspective completely random labels/annotations in operation (e.g., Helm and ArgoCD all add their specific annotations) and we didn't want these random events to induce a full dataplane restart (which would happen if we were to copy these to the pod template).
Eventually, we decided to still adopt this (suboptimal) approach with a little twist: instead of copying the labels/annotations from the Dataplane we rather added to fields two the Dataplane spec and made custom pod labels/annotations explicit. This rules out random pod restarts. At the same time, adding Gateway-specific labels/annotations is still difficult (you have to create a separate Dataplane per each Gateway), but it still seems like a tolerable compromise.
The new rules for the stunnerd pod labels/annotations are as follows:

Pod labels can be set from the labels field in the Dataplane spec. Mandatory labels override custom pod labels set via the Dataplane spec on conflict, and labels manually added to stunnerd pods are removed on the next update.
Pod annotations can be set from the annotations field in the Dataplane spec. On conflict mandatory annotations override custom pod annotations set via the Dataplane spec, which in turn override annotations manually added to the pod. Otherwise, manually added pod annotations are preserved.


Hyprland: Default Invalid Session Entry

Describe the bug
2 session entries are generated on login manager.

Hyprland (systemd)
Hyprland

Hyprland (systemd) is selected by default and it is buggy.

No way to launch kitty or any terminal.
exec-once is not working.
Keybindings are not working.

Just Hyprland entry is working as expected.
Steps To Reproduce
Steps to reproduce the behavior:

programs.hyprland.enable = true;
reboot
Select Hyprland (systemd) session entry on boot. (Selected by default on my system).

Expected behavior

Regular Hyprland experience.

Additional context
Happened after https://github.com/NixOS/nixpkgs/pull/354900 and related to uwsm session manager.
Notify maintainers
@fufexan
Add a :+1: reaction to issues you find important.

The breaking changes listed on the release page also do not mention the introduction of uwsm.

Because uwsm was introduced after the release.
I'm not sure why the systemd session file appears, as it's not explicitly defined in passthru.providedSessions. I can add a patch that removes it until the next release though.
The fact that it's selected by default is most likely because of the the display manager's ordering.

I had the same issue, had to fix the default entry by enabling
services.displayManager.sddm.settings = {
 General = {
   DefaultSession = "hyprland.desktop";
 };
};

for anyone else experiencing this issue:
programs.hyprland.withUWSM  = true;
and in home manager, make sure you disable systemd
wayland.windowManager.hyprland.systemd.enable = false;
my hyprland launched, but was super buggy, graphics were broken, and all sorts of other errors.
setting these two things fixed it.
hope it saves somebody an hour or two.


Samples/go build fails: error: missing binary operator before token "("

Describe the bug
I'm running into trouble when trying to compile the code using go:
rasmus@LAPTOP-MN1Q0MH0:~/projects/wallet-core/samples/go$ go build -o main
# tw/types
In file included from /home/rasmus/projects/wallet-core/include/TrustWalletCore/TWData.h:9,
                 from types/twdata.go:5:
/home/rasmus/projects/wallet-core/include/TrustWalletCore/TWBase.h:47:18: error: missing binary operator before token "("
   47 | #if __has_feature(assume_nonnull)
      |                  ^
/home/rasmus/projects/wallet-core/include/TrustWalletCore/TWBase.h:56:19: error: missing binary operator before token "("
   56 | #if !__has_feature(nullability)
      |                   ^

To Reproduce
Steps to reproduce the behavior:
Follow the instructions as described here
Relavant Information
I'm not sure what information is relevant but I have the following setup:
OS: Ubuntu 20.04
gcc version:  9.3.0 (not sure If this is the compiler used by go)
go version: go1.13.8 linux/amd64 (tried a newer version but didn't made any difference)
This is my go env:
rasmus@LAPTOP-MN1Q0MH0:~/projects/wallet-core/samples/go$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/rasmus/.cache/go-build"
GOENV="/home/rasmus/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/rasmus/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/lib/go-1.13"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go-1.13/pkg/tool/linux_amd64"
GCCGO="/usr/bin/gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/rasmus/projects/wallet-core/samples/go/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build356540807=/tmp/go-build -gno-record-gcc-switches"

I also tried compiling with clang instead, but then I get another error:
rasmus@LAPTOP-MN1Q0MH0:~/projects/wallet-core/samples/go$ CC=clang CXX=clang++ go build -o main
# tw/types
/usr/bin/ld: cannot find -lTrustWalletCore
/usr/bin/ld: cannot find -lprotobuf
/usr/bin/ld: cannot find -lTrezorCrypto
/usr/bin/ld: cannot find -lc++
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Have you run bootstrap.sh successfully? it looks like the lib is not built yet

seems broken with clang-11, can you try the Docker image now?

./bootstrap.sh build seems to fail
...
Found a struct Mnemonic
Found a class FIOAccount
Found a class GroestlcoinAddress
Found an enum Blockchain
Found an enum StellarVersionByte
Found an enum SS58AddressType
Found an enum HDVersion

WARNING: Skipped generating Xcode project because the xcodegen tool is not installed.
#### Building... ####
CMake Error at CMakeLists.txt:19 (message):
  You should use clang compiler

-- Configuring incomplete, errors occurred!
See also "/home/rasmus/projects/wallet-core/build/CMakeFiles/CMakeOutput.log".

CMakeOutput.log:
https://gist.github.com/RasmusN/0d4aec819d8a0af96a82604597a21289
Not sure if this xcodegen i relevant? It seems like it isn't using clang compiler?

It looks to me that gcc, not clang is used.

Building with clang gives me the following error:
rasmus@LAPTOP-MN1Q0MH0:~/projects/wallet-core$ cmake -H. -Bbuild -DCMAKE_BUILD_TYPE=Debug -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++
-- The CXX compiler identification is Clang 10.0.0
-- Check for working CXX compiler: /usr/bin/clang++
-- Check for working CXX compiler: /usr/bin/clang++ -- broken
CMake Error at /usr/share/cmake-3.16/Modules/CMakeTestCXXCompiler.cmake:53 (message):
  The C++ compiler

    "/usr/bin/clang++"

  is not able to compile a simple test program.

  It fails with the following output:

    Change Dir: /home/rasmus/projects/wallet-core/build/CMakeFiles/CMakeTmp

    Run Build Command(s):/usr/bin/make cmTC_0286b/fast && /usr/bin/make -f CMakeFiles/cmTC_0286b.dir/build.make CMakeFiles/cmTC_0286b.dir/build
    make[1]: Entering directory '/home/rasmus/projects/wallet-core/build/CMakeFiles/CMakeTmp'
    Building CXX object CMakeFiles/cmTC_0286b.dir/testCXXCompiler.cxx.o
    /usr/bin/clang++     -o CMakeFiles/cmTC_0286b.dir/testCXXCompiler.cxx.o -c /home/rasmus/projects/wallet-core/build/CMakeFiles/CMakeTmp/testCXXCompiler.cxx
    Linking CXX executable cmTC_0286b
    /usr/bin/cmake -E cmake_link_script CMakeFiles/cmTC_0286b.dir/link.txt --verbose=1
    /usr/bin/clang++       CMakeFiles/cmTC_0286b.dir/testCXXCompiler.cxx.o  -o cmTC_0286b
    /usr/bin/ld: cannot find -lstdc++
    clang: error: linker command failed with exit code 1 (use -v to see invocation)
    make[1]: *** [CMakeFiles/cmTC_0286b.dir/build.make:87: cmTC_0286b] Error 1
    make[1]: Leaving directory '/home/rasmus/projects/wallet-core/build/CMakeFiles/CMakeTmp'
    make: *** [Makefile:121: cmTC_0286b/fast] Error 2

  CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
  CMakeLists.txt:3 (project)

-- Configuring incomplete, errors occurred!
See also "/home/rasmus/projects/wallet-core/build/CMakeFiles/CMakeOutput.log".
See also "/home/rasmus/projects/wallet-core/build/CMakeFiles/CMakeError.log".

seems broken with clang-11, can you try the Docker image now?

I'm not sure how to do this. I have tried:

docker run -it trustwallet/wallet-core

And inside the docker:
2.
cd wallet-core
./tools/generate-files
cmake -H. -Bbuild -DCMAKE_BUILD_TYPE=Debug
make -Cbuild

which seems to succeed. Still inside docker:
3. apt-get update && apt-get install golang

cd wallet-core/samples/go

root@9034ce903301:/wallet-core/samples/go# go build -o main
main.go:18:2: cannot find package "github.com/golang/protobuf/proto" in any of:
        /usr/lib/go-1.10/src/github.com/golang/protobuf/proto (from $GOROOT)
        /root/go/src/github.com/golang/protobuf/proto (from $GOPATH)
main.go:15:2: cannot find package "tw/protos/bitcoin" in any of:
        /usr/lib/go-1.10/src/tw/protos/bitcoin (from $GOROOT)
        /root/go/src/tw/protos/bitcoin (from $GOPATH)
main.go:16:2: cannot find package "tw/types" in any of:
        /usr/lib/go-1.10/src/tw/types (from $GOROOT)
        /root/go/src/tw/types (from $GOPATH)

The problem was an old version of golang inside the docker container.
Fixed by adding this:
add-apt-repository ppa:longsleep/golang-backports
apt-get update
apt-get install goalng

Okay I need@ravidsrk

``

@hewigovens  rjl493456442

✅
Zoey Willam - Chat @ Spike	[1a7agy]

Managed to build outside docker now. The issue was a missing libc++-dev and libc++abi-dev. Solved by:
sudo apt-get update && apt-get install libc++-dev libc++abi-dev -y


Order listings by column

I've scoured the documentation and cannot find if this feature is supported or not, so I apologize if I missed it.
Basically, in the balance sheets, journals, etc. provide a method for ordering by a given column (based on the type). For instance, in particular being able to swap whether the default date ordering is ascending or descending. For other columns, order by alphabetical or numerical depending on the type.
Ideally this would be integrated into the column header such that clicking it would change the order.

This is currently not supported/implemented, so you haven't missed it ;-)
For basic tables (Statistics, Options, etc.) this is already implemented Client-side with JavaScript, as these are real
s. Advanced tables like Balance Sheet, Journal, etc. are not real
s, so this has to be done Server-side in Python.
I will keep this issue open as an enhancement and will have a look into it as how hard/easy it is to implement this.
Thanks for reporting!

Thanks!
I've been browsing the source code a bit but haven't yet put all the pieces together. If I get some more time in the coming weeks I will give my attempt to it and submit a pull request.

Great! If you have any questions regarding the source code or other parts, feel free to open an issue!

Even though we're not using
s, I think this can and should be done in Javascript. Reordering the Journal in Python would lead to all kinds of inconsistencies.


A little bug in trainer.py

Hello, wkentaro. Do you find that the training loss will suddenly become larger when a new epoch starts? I've noticed that and think it is because of the incorrect use of the training and evaluating mode of the model.
In train_epoch(), line 171, when you finish validation, I think you should change the model back into training mode.

Thank you I will check it.

My pull request resolves this issue. #56

I was getting unpredictable results and I just realized that the reason was this issue.
Came here to post it and saw it here!

Sorry, currently I have no GPU left to try this. Almost all GPUs at my lab is used for other experiments.
Maybe it takes more a few days to solve this.

@wkentaro Thanks. I believe one only needs to set self.model.train() again right after self.validate(), as @WeiQijie remarked.

@nolsigan @monaj07 thanks! d10dd3062824195fe5cc9d196b4af4081247bb3f should fix this.


Inject secrets from provider:"secret" tags

This PR add explicit secret injection to infer. This is necessary for explicit providers in YAML, which does not correctly apply schema based secrets to provider config. This is what the bridge does.
To guard against misbehaved or partially implemented SDKs, I have also added explicit secret injection to resources.
Fixes #251

also many thanks for all the explanations/answers to my n00b questions 🙇


History in command menu

I searched for sometime but can't figure out how to access to command history in command mode?
Any hint or something coding work is required?

Duplicate of #708


Exclude floating windows

Thank you! Looks great!
There must be a standard option to exclude floating windows from managing.

Might be able to exclude windows by subrole. There's some definitions here. Anecdotally, AXSystemFloatingWindow seems to avoid the window filter and is not tiled for me. What's the subrole of your offending window?

There are too many different of them to add a subrole every time. Like iStat Menus are treated as windows, not menus.
As for AXSystemFloatingWindow, could you please explain how should i use it? I'm just starting into all of it, don't know much.
Also would be great to add a rule for window instances. Like if you have an opened window for Finder, each new window will be opened horizontally in the same row. Then this wm will be near perfect!

Hey, to find the subrole of offending windows:

open the window (eg: open "HelpViewer")
in the HammerSpoon Console, enter hs.window.find("macOS User Guide"):subrole(), replacing the title string with the title of your window
note the subrole (eg: "AXSystemFloatingWindow" for HelpViewer app)

Any missed subroles can be added to the default PaperWM.spoon window filter, instead of handled on an app-by-app basis.
For custom window tiling and placement behavior, you'll be better off forking this project and hardcoding that behavior. The goal of this project is to mirror the behavior of the PaperWM Gnome extension and there's no methods to add custom behavior.
See the addWindow() method. Instead of inserting a new window based on it's horizontal location on screen, you can search for another instance of the same app in the same space. Then insert the new window in a column to the right.

Thank you! I'l check it.

This wm is so cool, too bad I am too bad at coding to implement all what I want on your base. Got back to yabai. But want badly this branch done to final usablity. I think that there is no need for trying making it 1x1 as on gnome, because it isn't possible with macos and too much work. Better to focus on what can be done to make it flawless with macos standard behaviours. And I see it as a merge with usual tiling managers with your paperwm.


fix(server): server wait group

Description
Introduces the ability to set a wait-group to the go-api/server to ensure in-flight requests are gracefully handled upon shutdown.
Upon Stop(), Wait()ing after the exit channel has been pushed onto, ensures the listener stops accepting any new requests while it awaits in-flight requests to complete.

Closing in favour of https://github.com/divisionone/go-api/pull/4


chore(checkbox): add accessibility attributes

Checkbox
TASK: 39
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Only
[ ] Refactor

Description, Motivation and Context

Screenshots - Animations
Adds a11y properties to Checkbox and CheckboxGroup components and updated all tests to make sure the new attributes are working.
Reference: WAI-ARIA Checkbox

Codecov Report

Merging #67 (e8fb4b0) into main (b6343f4) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #67   +/-   ##
=======================================
  Coverage   86.73%   86.74%
=======================================
  Files          93       93
  Lines        1546     1547    +1
  Branches      525      526    +1
=======================================
+ Hits         1341     1342    +1
  Misses        200      200
  Partials        5        5

Impacted Files
Coverage Δ

packages/nextui/src/checkbox/checkbox-group.tsx
100.00% <ø> (ø)

packages/nextui/src/checkbox/checkbox.tsx
100.00% <100.00%> (ø)

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 b6343f4...e8fb4b0. Read the comment docs.


Fix tracking consent label display, which wasn't updated after the change in the dialog

What does this PR do?
This change fixes refresh of the tracking consent label: it wasn't updated, because it is a part of hosting activity layout, so onResume is not called after fragments switch, because same activity is active all the time. Probably it would be cleaner to register some listener in the object responsible for preferences, but it would be more code, because each time Preferences#defaultPreferences is called new object is created.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #635 (d859352) into master (d6a559c) will decrease coverage by 0.16%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #635      +/-   ##
============================================
- Coverage     88.45%   88.28%   -0.16%
+ Complexity     1622     1617       -5
============================================
  Files           181      181
  Lines          5488     5488
  Branches        673      673
============================================
- Hits           4854     4845       -9
- Misses          394      402       +8
- Partials        240      241       +1

Impacted Files
Coverage Δ

...android/rum/internal/ndk/DatadogNdkCrashHandler.kt
75.48% <0.00%> (-6.45%)
:arrow_down:

...g/internal/domain/event/DdSpanToSpanEventMapper.kt
87.76% <0.00%> (-4.08%)
:arrow_down:

...droid/rum/tracking/ActivityViewTrackingStrategy.kt
84.91% <0.00%> (-3.77%)
:arrow_down:

...droid/rum/internal/domain/scope/RumSessionScope.kt
96.34% <0.00%> (-1.22%)
:arrow_down:

...oid/src/main/kotlin/com/datadog/android/Datadog.kt
78.79% <0.00%> (-1.01%)
:arrow_down:

...c/main/kotlin/com/datadog/android/DatadogConfig.kt
93.03% <0.00%> (-0.50%)
:arrow_down:

...ernal/instrumentation/gestures/GesturesListener.kt
89.74% <0.00%> (ø)

...l/net/info/BroadcastReceiverNetworkInfoProvider.kt
79.17% <0.00%> (ø)

...g/android/rum/internal/domain/scope/RumEventExt.kt
82.73% <0.00%> (+0.91%)
:arrow_up:

.../kotlin/com/datadog/android/rum/RumResourceKind.kt
86.36% <0.00%> (+4.55%)
:arrow_up:

... and 3 more


Dimension changes does not work in 1.16

This specifically involves boss fights not playing directly when a boss appears

The issue is that the bossmusic seem to start before the actual biome music. This is problematic because biome trigger methods do not respect bossmusic


Not Working in 0.116

Checklist:

[X ] I updated to the latest version available
[ X] I cleared the cache of my browser

Release with the issue:
0.116
Last working release (if known):
0.115.6
Browser and Operating System:
FireFox 81.0.1 on Windows 10
 HomeAssistant Server: Debian Buster VENV Install

Description of problem:
text-divider-row does not show anything in LoveLace web page.

text-divider-row does not show anything in LoveLace web page.  The page is displayed as if there is no reference to text-divider-row in the .yaml file.
Javascript errors shown in the web inspector (if applicable):

Additional information:

Do you see TEXT-DIVIDER-ROW was printed in the browser console?

I brought up my devcontainer and am unable to replicate. resource is loading just fine

Hi;
No, I do not see that.
Thank You,John
On Tuesday, October 20, 2020, 7:06:23 AM PDT, Ian Richardson<EMAIL_ADDRESS>wrote:

Do you see TEXT-DIVIDER-ROW was printed in the browser console?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi;
Hmmmm.  This has been working for many, many months.  The only things I did were update HomeAssistant and when it did not show up, I updated all of my resources listed in configuration.yaml.  Nothing changed.  Then, since I was using vertical-stack-in-card, which was throwing an error covered in the blog on one of my pages and it appears to not be maintained, I switched to stack-in-card and updated my lovelace yaml files for this change.
I was stumped and had found someone reference the same issue, though I forget where.  Nobody gave an actual response to the person, other than assuming they were a noob and providing basic information I had already used.  I noticed text-divider-row had not been updated in a few months and thought the issue might be the Custom Lovelace Cards Breaking Change listed in the release.  It was then I created a bug report, which is something I rarely do.
It would be nice if the log showed me some error.  I do not even see an error when I inspect and debug the web page from the browser.  I see no mention of anything to do with text-divider row.

Thank You,John Zable
On Tuesday, October 20, 2020, 1:59:41 PM PDT, Ian Richardson<EMAIL_ADDRESS>wrote:

I brought up my devcontainer and am unable to replicate. resource is loading just fine
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi;
So, for some reason I had to add "javascript_version: latest" under the "frontend" section of configuration.yaml.  I had never had that there, since the default "auto" always worked before.  I am bewildered that I now have to add a deprecated configuration variable into the system after an upgrade, but oh well.  At least I got it working.  (I do love having the text divider separating groups of entities again without having to use a new card.)
Thank You,John Zable
On Tuesday, October 20, 2020, 1:59:41 PM PDT, Ian Richardson<EMAIL_ADDRESS>wrote:

I brought up my devcontainer and am unable to replicate. resource is loading just fine
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I am not sure if the new frontend fix mentioned will fix this, but since this is related to the frontend and appears to not be specific to text-divider-row, I am closing this issue.
Thank You,
John Zable


⚠ ClamAV has degraded performance

In bab7385, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 84 ms

Resolved: ClamAV performance has improved in bcdacb4.


add MessageEncoder and make createPublisher etc. polymorphic

Hey,
I've made some changes to allow for binary messages. The approach is basically the same as for EnvelopeDecoder, a MessageEncoder[F, A] is just a Kleisli[F, A, AmqpMessage[Array[Byte]]]. It doesn't have as many nice compositional properties as the EnvelopeDecoder case, but I can't think of anything else that does.
Let me know what you think.

I've now fixed the tests. I moved the decoding logic from AmqpClientStream to ConsumerProgram, because that way it can be shared between the in-memory client and the regular client. A nice side effect is that this eliminates the last remaining use of asInstanceOf.

Ah, tut of course. I should have known…

Codecov Report

Merging #137 into master will increase coverage by 0.14%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #137      +/-   ##
==========================================
+ Coverage   96.55%   96.69%   +0.14%
==========================================
  Files          12       12
  Lines         116      121       +5
==========================================
+ Hits          112      117       +5
  Misses          4        4

Impacted Files
Coverage Δ

...ithub/gvolpe/fs2rabbit/interpreter/Fs2Rabbit.scala
90% <ø> (ø)
:arrow_up:

...hub/gvolpe/fs2rabbit/program/ConsumerProgram.scala
90.9% <100%> (+3.4%)
:arrow_up:

...b/gvolpe/fs2rabbit/program/PublishingProgram.scala
100% <100%> (ø)
:arrow_up:

...main/scala/com/github/gvolpe/fs2rabbit/model.scala
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

I believe I've addressed all the remaining issues with this PR; let me know if there's anything I missed.
Regarding AmqpMessage's type parameter, I think it's orthogonal to this PR and should thus be addressed separately.


Something go wrong in submission website.

Every time I open the 3D detection submission website, the submission button disappears and it shows "working" all the time, as the attached image shows.

Could you try this again?  It should be working now.  Thank you for flagging this.

Thanks, it works now.


Kogan 1500W Panel Heater not working

Hi there. Thanks for making this add-on. I was excited to stumble upon your repo after recently purchasing one of these heaters.
Quick question though. I've managed to extract the UUID and localKey from the Kogan SmarterHome app (Kogan SmarterHome_v2.0.1_apkpure.com.apk) using Bluestacks and BSTweaker but these do not seem to work in your add-on. Do I need to configure the heater in the official Tuya app for this to work?
With the UUID, localKey and IP configured, the add-ons seems to kill with WiFi on the heather as I get about 4 pings then it goes offline for a bit and the cycle continues. I've removed the integration and rebooted HA and now I get a consistent ping and the heater can again be reliably controlled from the Kogan app again.

If it is a new purchase, it may be a newer model with different dps assignments.  Does anything appear in the Home Assistant log?
You may need to set the log level to at least INFO for the tuya-local integration in order to see useful info with something like this in your configuration.yaml:
logger:
  default: warning
  logs:
    custom_components.tuya_local: info

Thanks. So it should work with the Kogan Smart Home app? I'll enable the logging and report back after the long weekend.

The integration will not work with the app.  You need to use one or the other.  Tuya devices only allow one connection at a time, and can become quite unstable if you try to control them with two at the same time.
Maybe that is what you are seeing with the device going offline after a few pings.

I realise my last comment may be misleading.  What I mean is that any app needs to be exited for Home Assistant to be able to connect to the device.   The Device ID and Local Key should work which ever app they come from, since you are communicating locally.
For the official Tuya integration, which uses the cloud, you need to be using an app that creates accounts in the same domain as the HA Tuya is using (I think it can be configured to use either us.tuya.com or eu.tuya.com, maybe also cn.tuya.com but some manufacturers use their own domain - Goldair I think does this, but I don't think Kogan does).
If you switch between apps and re-pair your device in the process, the local key will be changed, and you'll have to extract it again.

Thanks for the clarification. I didn't have the Kogan app open when trying to connect so immediately thought I must need to be registered with the actual Tuya app. I'll have another go with the app force closed and logging enabled and report back.

There is another thread on issue #20 where another user seems to have a similar problem. In their case they previously had the heater running, and have two others that still run, but one has stopped working.  The software version is different than the other two heaters, and it is starting to look like a software update has changed the protocol somehow.

0.8.5 adds some Warning messages when setting up a new device that does not perfectly match any known config.  Previous log messages with useful info were at INFO level or below, which is not enabled by default in latest HA releases.
This message may provide some clues on what has changed in the new firmware so we can get it working again.

The current "main" branch has added a new Kogan Heater with different DPS layout.
Your heater may have also been using the newer layout, so you might want to test the "main" branch.  You can do this by clicking the menu button on the Tuya Local integration in HACS, and selecting "Reinstall", and choosing "main" as the version from the bottom of the list.

As another similar issue has been closed by the new config, I've released it, so no special steps to install it, just update to 0.9.1 and restart.

Thanks! Going to test tonight.

Working now under 0.9.1. Thank you!


[FEATURE] Banner

Describe the feature you want

Banner: In some circumstances it is required to notify users of something because it would be nice to provide a banner. That banner should show up at the top of the topbar as shown in mkdocs-material:

but it should also have a button to close the banner and not show up again like in StackOverflow.

The logic should be the same as the cookie consent.

I doubt how many people will use it, but here is an example that might help.
https://codepen.io/matswainson/pen/PGrqGG


Increase default timeout for docker-job from 1 hour to 2 hours

run_tests_c++_windows_dbg_native_default_default_buildonly has been flaky due to timeout. It spent 55 minutes or so recently. Because we already have timeout for test suite (by kokoro configuration), it can be considered okay to increase the default timeout for docker job.
Ref: Failure case link

The error message from the invocation log is
+ python workspace_c++_windows_dbg_native_x64_cmake_vs2017_buildonly/tools/run_tests/run_tests.py -t -j 2 -x ../run_tests/cpp_windows_dbg_native_x64_cmake_vs2017_buildonly/sponge_log.xml --report_suite_name cpp_windows_dbg_native_x64_cmake_vs2017_buildonly -l c++ -c dbg --iomgr_platform native --arch x64 --compiler cmake_vs2017 --build_only --build_only
2019-11-01 08:02:59,405 START: tools\run_tests\helper_scripts\pre_build_cmake.bat
2019-11-01 08:05:28,944 PASSED: tools\run_tests\helper_scripts\pre_build_cmake.bat [time=149.5sec, retries=0:0]
2019-11-01 08:05:28,944 START: cmake

2019-11-01 09:01:55,881 TIMEOUT: run_tests_c++_windows_dbg_native_x64_cmake_vs2017_buildonly [pid=1384, time=3600.1sec]

Agreed that that's cause by the run_tests_matrix.py job timeout.

Is this happening only in grpc/core/master/windows/grpc_portability_build_only job or also elsewhere?
From the log it looks like we're building multiple things in parallel (in different sudirectories), but the overall duration is still too much.

I updated the all portability tests for C and C++ to use _CPP_RUNTESTS_TIMEOUT which is for all basictests for C and C++, too.

Still need to answer these questions:

How common is the timeout?
For which jobs does it happen?

If it's only very rare, I'm against increasing the timeout.
If it actually causes failures on regular basis, let's increase.

Timeout
I think timeout is pretty common so that I saw the timeout on my PR 3 times this week. (If you have a good dashboard on this, please let me know) Today, test fusion has a timeout on grpc_portability_build_only.
Average duration
The job which spends most is run_tests_c++_windows_dbg_native_x64_cmake_vs2017_buildonly and time took for the recent 3 tests: 3376.7sec, 3376.7sec, and 3170.6sec which is pretty close to 1 hour.


[FLINK-16658][FLINK-16660] Introduce the ApplicationDispatcherBootstrap and wire it to StandaloneJobEntrypoint

What is the purpose of the change
This PR introduces the ApplicationDispatcherBootstrap which is responsible for bootstrapping the Dispatcher when running in application mode, as described in FLIP-85. In these settings, the user's main() runs on the Dispatcher. From now on, the entrypoint will launch a "session cluster" only for the jobs in the application that it starts with, and the cluster will be shutdown if:

all the jobs in the application fail
at least one of the jobs fails
if a job gets cancelled

So far we allow only a single job application to be executed, for reasons described in the FLIP.
In addition, we wire the ApplicationDispatcherBootstrap to the StandaloneJobClusterEntrypoint so that we do not hijack the execute() method anymore. With this PR, the StandaloneJobClusterEntrypoint runs the user code in "Application Mode", i.e. the main() runs on the cluster and not on the client, but we do not rename the class, for backwards compatibility.
Verifying this change
We added:

the ApplicationDispatcherBootstrapTest,
the JobStatusPollingUtilsTest,
added tests in the StandaloneJobClusterConfigurationParserFactoryTest
and there are already the end-to-end tests for the StandaloneJobClusterEntrypoint that cover the changes.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented yet)

@flinkbot run azure @flinkbot run travis

@flinkbot run azure

@flinkbot run azure


UX Feedback

Datasets page

the fact that its a preview needs to be more visible

Datasets page:
What can I do with this data?
Cross-datasets widgets
Pandas-style data metrics
Variable length previews / load full datasets
Per council district pages (with sonar subscribe?)
Resources / Vs Data header
Overview section / preview

https://www.kaggle.com/kingburrito666/better-donald-trump-tweets -- good example of UX patterns for dataset


CrowdsaleTokenExt Errors

If you are reporting a problem with Token Wizard, please include the following information:
Which network did you use? (Mainnet, Kovan, Rinkeby, etc.)
Your answer
Mainnet
If you were able to create it, what is the URL of your crowdsale?
Your answer
https://wizard.poa.network/crowdsale?addr=0x5a4f22fB8fdf877ca3e607C46FeB09be1017Ee65&networkID=1
Do you have screenshots showing the problem?
Your answer
Yes

Do you see errors in the dev console? If yes, please include a screenshot
To open the dev console in Google Chrome, press F12, or go to View -> Developer -> Developer Tools, and open the Console tab
If you see errors, please right click on them and "Save as..". Zip saved file and attach it to the Issue.
Your answer
I have been able to verifry the rest of the files except the 'CrowdsaleTokenExt'
I also used eth to buy the token to my metascan eth address and it was successful however when you check Konero (XKO) on etherscan you  see 2 Konero xko name appear and you get this view

and this

My total supply is 10 million i dont know why its showing 100 only. I used the same Current account to purchase the 100 token.
Finaly how do i transfer all the token to my seserved account should the crowdsale be unsuccessful in getting to target or hardcap ?
Thank you.

My total supply is 10 million i dont know why its showing 100 only. I used the same Current account to purchase the 100 token.

Token form Token Wizard is mintable. You see the amount of sold tokens.

Finaly how do i transfer all the token to my seserved account should the crowdsale be unsuccessful in getting to target or hardcap ?

There is no refund strategy for Token Wizard yet
Regarding to verification issue:

attach zip summary files for crowdsale
attach a screenshot of all filled fields for verification of Token contract before you click Verify button

Please examine the website https://www.konero.co   i was able to get source code through etherscan and uploaded it to the website  but i believe marketing has been one of my problems in getting reesult in the crowdsale. Can i extend the date of the crowdsale to another month so i can focus more on marketing and advertising it?

Can i extend the date of the crowdsale to another month so i can focus more on marketing and advertising it?

https://github.com/poanetwork/token-wizard/wiki#managing-the-crowdsale


Rename Chapter: devon4node -> NodeJS

Rename chapters in docs.
devon4node -> NodeJS

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.dolvie seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Support Custom Field Mappings between Engagement Plan Task and Task

Critical Changes
Changes

Community Sprint Driven Idea to Map Matching Custom Fields between Engagement Plan Task And Task when creating the Engagement Plan from the Template

Issues Closed
New Metadata
Deleted Metadata

@RadSFDeveloper - Hi Radmilla, I have not added unit tests yet for the UTIL_SimpleTestMapper class, but wanted to give you an opportunity to provide feedback while I am doing that.  Hence, I am tagging you for a preliminary round of feedback.
@force2b - FYI, I went a little further with it today, and generalized the part that determines what fields are matches into a general shared class we could reuse if we want/need to to do this with other objects.  It follows the <>.getInstanceOf(...).with<<ConfigParam(...)...doSomething() that we are using in many of our newer code implementations.
I will work on the test class later this weekend, or some evening this week.

**lurch: attach W-017870

Tracking W-017870


Skip ldflags_dirs step for Rubies where it's not necessary

ldflags_dirs was added for Ruby 2.1.0-dev in 50bf60f9c2510fbee7f111046e97fe8bf9e66472.
Since then, in releases of Ruby 2.1.x the LDFLAGS checker seems to have gotten more permissive and this step isn't necessary anymore. However, Ruby 2.1.2 and 2.2.0-dev seem to need it still.
This removes ldflags_dirs from at least those build definitions where it's not necessary in order to clean them up and avoid cargo-culting.
@hsbt: Why do you think LDFLAGS checker has gotten less strict since 2.1.0 release but has become more strict again in 2.1.2 and onwards?

@mislav Sorry, too late response. I confirmed this.


[Feature Request] - Add 'Send to Controlnet's Mask' feature

Hi, Since Controlnet Allow mask upload, can you add a button to send those mosaic and mask inputs directly to Controlnet input image and mask please?
Thank you.

Hello, please try out the dev branch~

(btw, due to how ControlNet works, the source image may not show up. Simple click the Clear mask button to refresh it)

thank you. I gave it a try and it does work. However due to the nature of CN itself there's clear seam around the edge. I will test it on XL soon.

Funnily, all I've got were results like this:
https://github.com/Mikubill/sd-webui-controlnet/issues/2625
Glad that at least it works better for you.
Also, iirc there isn't an Inpaint ControlNet model for SDXL yet...


Point2Grid: Add a new MET tool to process point observations into a gridded field.

Here's an email exchange describing the functionality that was requested by the DTC - Regional Ensembles group as well as the NOAA - HWT group... and NOAA - HMT group

Need to think of a good name for this.  Maybe "grid_point_obs"... but that's confusing.  Could be "process_point_obs"?

FROM JOHN:

Do you agree that the practically perfect algorithm would require a new tool which...

(1) Reads point observations... specifically one or more NetCDF output files from ascii2nc, madis2nc, pb2nc, or lidar2nc.

(2) Read a configuration file which defines the group(s) of point obs to be processed (i.e. ADPSFC surface temperatures or 6-hour rain gauge obs) and defines the desired output grid (e.g. the 80km grid you referenced).

(3) For each "field" (i.e. group of point observations), apply the gaussian filter to define a practically perfect forecast for the requested output domain.

Some questions to consider are:

(1) Are there any additional configuration options required for the practically perfect algorithm?

(2) How do you name the output varables?

(3) Should the tool process multiple output times in a single run or just one?  How do you define those time windows... and therefore, the timestamp of the output.  Since these are observations... from the perspective of timing meta-data this should be considered an analysis.

Supporting the practically perfect algorithm would be the first step, but we could potentially add support for additional "gridding" algorithms in the future.

Do all those details sound right?  And if so, what would we name such a tool?  We currently have "plot_point_obs"... would this be "grid_point_obs"?

FROM BURKLEY TWIEST:

(1) For the configuration of the practically perfect algorithm, having the ability to vary the sigma used in the Gaussian smoother is critical, particularly so that this tool can be applicable across a variety of space and time scales. Also, (and I think you already have this in the description of the tool) the ability to have the practically perfect fields generated on and output to grids other than the 80-km grid is necessary. Finally, being able to indicate a neighborhood around the report that would indicate a "hit" would be useful. I think this would need to go into the new tool being outlined here. For example, I've recently done some regridding of practically perfect fields, and needed to light up every point within a 40-km radius of the observation prior to applying the smoother in order to get the practically perfect values we needed on a fine-scale grid.

(2) Good question! We use "practically perfect" when dealing with the observations (and that's pretty widespread in the literature) and "surrogate severe" when dealing with the model fields. However, when looking at things like precip, you might have two gridded fields of precip exceedance that you're dealing with. Perhaps indicating the output fields as "Gaussian" or "Gaussian smoothed"+ would be a good compromise that could apply equally to the observations and model field datasets?

(3) I think our main applications for these would be to 24-h time periods and 1-h time periods, although there are a couple projects in the works here that look at time periods between 1-h and 24-h, and there could be some potential with Warn-on Forecast to look at even shorter time frames. It would be really nice to be able to process multiple times without having to call the tool multiple times for shorter increments, especially if we start applying this to warn-on forecast output. Would it be possible to be able to designate different chunks of time, one smaller than the other? So the tool could accumulate over a 24-h period, but also spit out hourly fields simultaneously? This would be a really useful functionality, but I'm not sure if it's too large of an ask!

Hopefully these responses help in your development, and feel free to poke me with any continued questions. We really appreciate the implementation of this functionality into MET, and think it'll be very useful to the community as a whole.
[MET-1078] created by johnhg

Done for the initial release.
Not implemented:

supporting multiple probabilities
python support


Import errors and Modules missing

Hello,
I've been trying to replicate the environment and I noticed that a pip install openai, roboflow is missing in order to have the code imports properly working (It seems to be ommited in the requirements.txt). Moreover, autodistill renamed one of the models that the script (samgpt.py) imports, so that, what was called CustomDetectionModel is now called ComposedDetectionModel as you can check in here -> autodistill/core/composed_detection_model.py
I don't know the reason of this changes in Autodistill and it seems that @capjamesg (creator of the sam-gpt4v repo) was the one making those changes recently according to the logs. Clarification on this would be more than welcomed. I suppose that just by changing the naming of the imports will be enough to make it work.
the change will be needed here:
from autodistill.core.composed_detection_model import ComposedDetectionModel
and here:
SAMGPT = ComposedDetectionModel(
    detection_model=GroundedSAM(
        CaptionOntology({"car": "car"})
    ),
    classification_model=GPT4V(
        CaptionOntology({k: k for k in classes})
    )
)

Thanks in advance!

Hello @jaimecorton! I had some local autodistill changes from a PR branch that were not available via PyPi while I was working on this project. I should have included instructions to pull from the branch and install from source while the module was uploaded. I have updated the project to use the new API name. The new autodistill release should also fix the roboflow import issue. Run pip install --upgrade autodistill to get the latest version.

Hello again and thanks for the answer!
It seems to be solved then. Have a nice week!
I will close the issue.


Extra border in search bar when egui is launched with light theme

Uh-oh, yet another 3 tiny things to report ;)

When egui is launched in light mode, switch to regex mode, and you will notice that the bottom border of the search box changes its width. To my eye, it looks proper in regex mode, but wrong in the default mode. Interestingly, this visual inconsistency does not seem to be visible in the dark mode.

I'm not sure whether fixing it in light mode will break it in dark, or it's just egui lib thing. But anyway, letting you know my observations.

Apart from that, when you are bored, you might want to look into inconsistent spacing between items and separators:

tiny ugly gap between the first normal item and search box separator (and between the first favorite item and search box separator)
bigger gap between the last favorite item and favorite box separator (and between the first normal item and the favorite box separator)

Dunno whether it should be big or tiny gaps or no gaps at all, but whatever the choice, it should be consistent :D

Also, due to the change of the font size when switching to regex mode and back, the search box changes its height which looks a bit jerky.

I'm away from my computer for the next month and a half 😅 so won't be able to fix this anytime soon, but this is great attention to detail. Hopefully I'll get a chance to tweak it in October. :)

Should all be fixed now!

Released as 0.6.2, let me know what else you find. :)


Call out "history-retaining merkle tree" more prominently, pls

Hi @paul-hammant, thanks for opening this issue!
Would you kindly provide more context to this ask? (I guess it's related to the documentation or our system internal but will appreciate more details)

Elsewhere in the repo's issues/commits etc there is talk of merkle-trees as an important design factor for your tech. You've not called it out on the README and perhaps should do.
https://aws.amazon.com/qldb/ implements merkle-tree aspects. Though as I check now, the blurb they have no longer lists that as a selling point :(

Thanks @paul-hammant!
Did you get a chance to read through https://docs.lakefs.io/understand/data-model.html? in particular through https://docs.lakefs.io/understand/data-model.html#constructing-a-consistent-view-of-the-keyspace-ie-a-commit?
Let me know if this is what you were expecting to see.
It is an (important) implementation detail, therefore I believe that adding it to the README may be a bit distracting, but our docs should definitely include this.

Hi @paul-hammant !
Thanks for the suggestion!  I'm going to ramble on for a bit to explain my motivations, then ask a concrete question (I promise!).
AFAICT we primary refer to Merkle trees on our Data Model page, "Constructing a consistent view of the keyspace".  And this is of course a core implementation detail.  It appears in @ozkatz 's  design doc that transmogrified into the "Data Model" page above, and less formally I've blogged about it (twice).
As an open-source project, our documentation has to reach two distinct audiences:

Our users are  primarily interested in the git-level abstraction that lakeFS provides for managing their data.
Like any other Git user, they care mostly about porcelain, and less about the plumbing.  Ideally these abstractions should be enough for them to be able to ignore entirely the implementation, even headlines such as "Merkle tree".  But I believe that we should aim to allow them to be happily oblivious of the design.  This is similar to how Git should be usable without reading the Git Internals chapter, at least not beyond "plumbing and porcelain.
Our developers (and our "tinkerers" and our "curious users") may need to understand the design at the lower levels.  Even then, we try to keep the "Graveler" abstraction wall to allow developers (ourselves included) to ignore its details when working outside of it.
We keep the data model documentation and informal descriptions in order to help our developers (ourselves included) in the loop.  Currently we do not have a "lakeFS Internals" chapter in our documentation.  Indeed, we try to stand on the shoulders of giants and are happy to use terms such as "Merkle", "git"[-like], "SSTable", and "B+Tree".

I would categorize this issue that you have opened as increasing the level of documentation for our developers.  Do you agree?  Where do you imagine seeing this added description, and at what level -- more annotation on its current mention, a new section, a references / further reading chapter in the data model, somewhere better perhaps?
Thanks!

I am happy that y'all have duly considered it and designated the fact that the tech uses Merkle trees to be an internal detail. I guess git does too: There's no explicit git verify-merkle-tree operation over a clone to a FS. Git does that silently and would blow up if something in a clone didn't add up.


Android crashes on sync for certain situations

When an account syncs, in some cases it crashes entirely for no reason.

perhaps handled in #1008 because these only happen on initial sync


Cards not updating with dynamic content

Hi,
I´m trying to display dynamic content loaded from a firebase database into one of the cards. However the are showing empty strings where the content of the database should appear.
global variable
String phone = "";
then one of my infoCards
MaterialAboutCard.Builder infoCard = new MaterialAboutCard.Builder();
        infoCard.title(info_title);

        infoCard.addItem(ConvenienceBuilder.createPhoneItem(context,
                new IconicsDrawable(context)
                        .icon(GoogleMaterial.Icon.gmd_phone)
                        .color(ContextCompat.getColor(context, colorIcon))
                        .sizeDp(18),
                info_phone,
                true,
                phone));//here I set the string taken from the database

        Log.e("IN-BUILDER","PHONE in materialaboutbuilder:"+phone);

My onCreate
public void onCreate(@Nullable Bundle savedInstanceState) {
        //Get Firebase auth instance
        auth = FirebaseAuth.getInstance();
        user = auth.getCurrentUser();
        //get database reference
        database = FirebaseDatabase.getInstance().getReference();
        if (user!=null && database!=null) {
            readUserStuff();
        }

        super.onCreate(savedInstanceState);
    }

and the methode readUserStuff
private void readUserStuff(){
        String UID = auth.getCurrentUser().getUid();

        database.child("ServiceUsers").addChildEventListener(new ChildEventListener() {
            @Override
            public void onChildAdded(DataSnapshot dataSnapshot, String previousKey) {
                mUser = dataSnapshot.getValue(User.class);
                phone = mUser.getPhone();
                Log.e("READ","PHONE "+mUser.getPhone());
                refreshMaterialAboutList();
            }
//....
}

So the output of the LOGs is as follows.
First the Log with in the getMaterialAboutList:

"IN-BUILDER", ""

= therefore the cards are empty (expected behaviour)
then the methode readUserStuff() has the Log output

"READ" ,<PHONE_NUMBER>6(correct number)

then refreshMaterialAboutList() is called and the Log in the getMaterialAboutList says:

"IN-BUILDER",<PHONE_NUMBER>6(correct number)

so the refresh gets called
all Strings are updating correctly from the database

BUT the cards remain empty no matter how often I call refresh. Also recreating the activity has the same result.
Any Idea?

Solved it by passing the User Object from the activity to the fragment and therfore the cards build with the correct strings. Seems that the refresh is not working as expected.

Hi! Thanks for all the detail. Updating global primitive variables will not update the reference in the Material About List, therefore you need to store the ActionItem that you're putting in the list as a global variable and when you get the updated phone number from the database you need to reference the phoneActionItem and call .setSubText(phone) and .setOnClickAction(ConvenienceBuilder.createPhoneOnClickAction(getActivity(), phone) and then refreshMaterialAboutList(). This is a limitation of Java!

Solved it by passing the object from the activity to the fragment. In this case the builder can already use all of the loaded strings. So no need to refresh the List

There are two examples of how to update the items here: https://github.com/daniel-stoneuk/material-about-library/blob/master/app/src/main/java/com/danielstone/materialaboutlibrarydemo/ExampleMaterialAboutFragment.java.
Yeah, sorry I didn't see that message at first. Just wanted to clarify how to use dynamic items if you do need to in the future :)

Thanks for your help. Great library by the way. Exactly what I wanted and everything is working ;)


Added Mindanao State University - General Santos City

Mindanao State University - General Santos
Official Domain

@Ren-Logronio First stage of verification passed, official domain is msugensan.edu.ph as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@Ren-Logronio Please provide us with an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course. Thank you!

https://msugensan.edu.ph/colleges/natural-sciences-and-mathematics/
You can get in touch with the head of the IT Physics Department regarding the matter

@Ren-Logronio Thank you for the clarification and your patience while waiting for a review and processing your request. Pull request merged.


MIOpen Error: /MIOpen/src/tensor.cpp:67: Invalid length. Length must be greater than 0

Hi,
While running an LSTM-based model using rocm I get the following error, while with CUDA on NVIDIA GPU, it works fine. I checked the size of the tensor is not 0 (i.e., the size is torch.Size([2000, 1536, 128]).
Do we not have support for it in ROCm at the moment?
MIOpen Error: /MIOpen/src/tensor.cpp:67: Invalid length. Length must be greater than 0.
Exception in thread Thread-3:
Traceback (most recent call last):
File "/usr/lib/python3.8/threading.py", line 932, in _bootstrap_inner
self.run()
File " ~/tools/ont/bonito/multiprocessing.py", line 110, in run
for item in self.iterator:
File " ~/tools/ont/bonito/crf/basecall.py", line 69, in
(read, compute_scores(model, batch, reverse=reverse)) for read, batch in batches
File " ~/tools/ont/bonito/crf/basecall.py", line 34, in compute_scores
scores = model(batch.to(dtype).to(device))
File "/home/gagandee/env/rubicon/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File " ~/tools/ont/bonito/crf/model.py", line 179, in forward
return self.encoder(x)
File "/home/gagandee/env/rubicon/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/home/gagandee/env/rubicon/lib/python3.8/site-packages/torch/nn/modules/container.py", line 204, in forward
input = module(input)
File "/home/gagandee/env/rubicon/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File " ~/tools/ont/bonito/nn.py", line 247, in forward
y, h = self.rnn(x)
File "/home/gagandee/env/rubicon/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1194, in _call_impl
return forward_call(*input, **kwargs)
File "/home/gagandee/env/rubicon/lib/python3.8/site-packages/torch/nn/modules/rnn.py", line 774, in forward
result = _VF.lstm(input, hx, self._flat_weights, self.bias, self.num_layers,
RuntimeError: miopenStatusUnknownError

@singagan Please provide clear reproduction instructions of your problem. ROCm version, base OS version, GPU type, which software needs to be installed etc. Optimal way is providing a docker image so our engineers are able to quickly reproduce the issue without affecting their test systems.
Without the above we will be unable to help you.
/CC @junliume @JehandadKhan @shurale-nkn

@atamazov Thanks for the quick reply. I will try to share a docker. Meanwhile, I am attaching the log I generated using env MIOPEN_ENABLE_LOGGING
run.log
If I reduce the batch size from 1536 to 768, then I do not see this error. There seems to be some issue with the buffer sizing
HIP version: 5.4.22804-474e8620
AMD clang version 15.0.0
OS: Ubuntu 20.04.3 LTS (Focal Fossa)
GPU: Mi210

@shurale-nkn  could you also take a look at the log above?

@singagan Thanks for providing the log and for additional information about batch size, but I am afraid this is not enough for us to understand what happens. If you can provide logs with more info, then we can try again (but docker + repro instructions is be better of course). Recommended env settings for generating log:
export MIOPEN_ENABLE_LOGGING=1 ;\
export MIOPEN_ENABLE_LOGGING_CMD=1 ;\
export MIOPEN_LOG_LEVEL=7 ;\
export MIOPEN_ENABLE_LOGGING_MPMT=1

Note: log will be huge.

@shurale-nkn could you also take a look at the log above?

A month ago I discussed with singagan in chat about this problem. After the requested logs with MIOPEN_ENABLE_LOGGING=1 were passed to me, I offered two options for how Gagandeep can quickly workaround this problem: decrease batch_size or seq_length, and run this operation twice, Because in this configuration MIOpen have to work with buffer size larger than INT type and our implementation can't work with it.

@singagan Has this issue been resolved for you? If so, please close ticket. Thanks!


VsTest - How to get fully qualified test names

I have a standard set of tests for each entity e.g. Add/Create etc, which means if something fails VSTS just tells me that I have a failed test "Add", which is not ideal when it could be one of 150 other Add tests.
According to #1732 I can get fully qualified name by passing some additional parameters to the task e.g. /ListFullyQualifiedTests /ListTestsTargetPath:abc.txt but the task won't let me do this unless I supply a testsettings file which I don't normally use.
I'm also not sure if this is affected by the fact I'm using the NUnit3TestAdapter to make the NUnit tests available.

@phatcher, The options /ListFullyQualifiedTests /ListTestsTargetPath:abc.txt are only for discovery and not execution. i.e. it will only list the tests available and not execute them. While the console logs may not give enough information, the tests are grouped under their class names in the test window. That will help you nail down which test method is actually failing.

@phatcher this doesn't belong in this repo. Hence I am closing it.
And it's already tracked here: https://github.com/Microsoft/vstest/issues/1732


Random duration (3-15s) on upload failure, instead of fixed 3 seconds

Having used assets-uploader as part of go-release-action, with a matrix of multiple builds, upload seems to fail arbitrarily.
I am hoping that using a random time might reduce the likelihood of such uploads failing one after the other.
My guess being that the connection gets reset because multiple jobs are uploading at the same time.

This could also be reserved as an override feature (flag) like e.g. retry.
Which would mean:

use 3 seconds if there is no retry_time_max
if retry_time_max exists, generate a random time between 3 and retry_time_max for each loop

What do you think?

tag https://github.com/wangyoucao577/assets-uploader/releases/tag/v0.10.0 is ready for this feature, thanks very much!


Initial MAD PVP Support

Uses Ohbem to calculate pvp ranks for each request
Configurable leagues
Configurable level caps
Requires config updates!

Little cup does not work.

:eyes: Why you make little cup no work?

Little cup does not work.

👀 Why you make little cup no work?

Cuz heaven forbids this map has a separate master file.

Little cup does not work.

👀 Why you make little cup no work?

Cuz heaven forbids this map has a separate master file.

😭 Maybe some day.

Little cup has been fixed.