1.800 GB disk full after syncing only 850 GB

Reproduce / what I did

setup ocis beta 7 according to https://owncloud.dev/ocis/deployment/basic-remote-setup/ (on my Odroid HC4 NAS, Armbian 22.08 Jammy, Kernel 5.10 )
Config ocis data path to a dedicated HDD (EXT4)
create user "tb", set quota to 2TB
(the only users are "admin" and "tb"; admin has not uploaded a single file)
Setup sync via Desktopclient 2.11.1 (build 8438) with external HDD (exFAT, no VFS)
Synced ~850 GB with the server, but disk usage is at ~1.800 GB

Expected behaviour

Disk usage should be ~850 GB

Current behaviour

Disk usage is ~1.800 GB

unsure if this a bug or layer 8 issue.

@micbar @dragotin do you have any idea what could cause this problem? It blocks me from using ocis for my family-cloud

it says:

More infos for debugging:
Blocksize of the is 4k:
odroidhc4:users:# tune2fs -l /dev/sda2
tune2fs 1.46.2 (28-Feb-2021)
Filesystem volume name:   OCDATA
Last mounted on:          /media/ocisdata
Filesystem UUID:          30c0abd2-01fe-4aad-b941-6b4b59ae0d23
Filesystem magic number:  0xEF53
Filesystem revision #:    1 (dynamic)
Filesystem features:      has_journal ext_attr resize_inode dir_index filetype needs_recovery extent flex_bg sparse_super large_file huge_file uninit_bg dir_nlink extra_isize
Filesystem flags:         signed_directory_hash
Default mount options:    user_xattr acl
Filesystem state:         clean
Errors behavior:          Continue
Filesystem OS type:       Linux
Inode count:              122093568
Block count:              488345366
Reserved block count:     60
Free blocks:              13172139
Free inodes:              122069680
First block:              0
Block size:               4096
Fragment size:            4096
Reserved GDT blocks:      907
Blocks per group:         32768
Fragments per group:      32768
Inodes per group:         8192
Inode blocks per group:   512
RAID stride:              32716
Flex block group size:    16
Filesystem created:       Mon Nov 26 08:23:39 2018
Last mount time:          Mon Sep  5 22:53:16 2022
Last write time:          Mon Sep  5 22:53:16 2022
Mount count:              74
Maximum mount count:      -1
Last checked:             Mon Feb 28 05:37:21 2022
Check interval:           0 (<none>)
Lifetime writes:          6252 GB
Reserved blocks uid:      0 (user root)
Reserved blocks gid:      0 (group root)
First inode:              11
Inode size:           256
Required extra isize:     28
Desired extra isize:      28
Journal inode:            8
Default directory hash:   half_md4
Directory Hash Seed:      0b30a227-754e-41fb-bfb2-bfe6547a3758
Journal backup:           inode blocks

Metadata size seems ok
odroidhc4:users:# cd /media/ocisdata/storage
odroidhc4:storage:# du -sh *
3.1M metadata
1.8T users
odroidhc4:storage:#

upload directory seems to big:
odroidhc4:storage:# cd users
odroidhc4:users:# du -sh *
40K indexes
1.2T spaces
632G uploads
odroidhc4:users:#

users contains 15k directories:
odroidhc4:users:# find /media/ocisdata/storage/users -type d | wc -l
15613

The data I store on the HDD are movies with filesizes 5-10 GB per Movie.

Tried that with 2.4 GB from my local harddisk:
[kf:~] $ du -sh Downloads/
2,4G    Downloads/
[kf:~] $

which results in the harddisk usage in .ocis:
[kf:~/.ocis/storage] $ du -sh *
88K     metadata
2,6G    users

ok, thx! will retry.
closing as not reproducible.

Well, isn't it worth to dig deeper into the humongous upload folder? 632G uploads certainly feels wrong.

Tested with @tbsbdr again today (Hardware ODroid)
We set up vanilla oCIS beta8 and tried to upload a movie collection of 33 GB via sync client. For that, we created a new VFS folder on windows and moved the files into that.
Uploading started ok'ish, just extremely slow. The server was bored most of the time, maybe because moving and VFS required so much client power?
At some point, the client reported bugs. On the server, most of the files (all?) are still in the upload folder. The nodes seem to be created.
Uploading a large single file via tus works fine. The problem only seems to happen with uploading many files via tus.
@TheOneRing  @butonic pls try to reproduce.

Tus in 2.11 is broken.
Please test with 3.0
Ocis should block legacy clients https://github.com/owncloud/client/issues/9612

What? That would mean we loose the ability to run oCIS with clients earlier than 3.0. That is not ok. We did a lot of compromises in the backend to be able to "satisfy" older clients - loosing that now on the last meters is not doable.
@micbar @michaelstingl pls look into that and find a compromise.

Problem still exists with Desktop client 3.0 (though upload is much faster and stable - feels almost illegal fast 😼 )
"uploads" folder gets huge:
tg@odroidhc4:~$ du -sh /data/ocisdata/storage/users/*
28K /data/ocisdata/storage/users/indexes
104G /data/ocisdata/storage/users/spaces
164G /data/ocisdata/storage/users/uploads

Whenever an upload is finished, regardless which protocol (TUS or plain PUT) is used, finishUpload() and consecusively discardChunk() are called. It wipes the info and the bin file corresponding to the upload.
@tbsbdr To check what is going on could you post an ls -lh /data/ocisdata/storage/users/uploads. And maybe cat one of the json files in there?

AFAICT we currently have no cleanup job that removes unfinished uploads. The problem here is that the client started several uploads in parallel and they could not finish because collectively they filled the disk. Leading to a situation where no upload could finish.

We agreed that we can prevent this by checking the "uploads which are in progress" during "InitiateUpload".

If df shows 100% usage of ocisdata:

❌ Delete file via Web UI -> not possible (see above https://github.com/owncloud/ocis/issues/4516#issuecomment-1258053676))
✅ Delete smalish file via cli, then: delete file via Web UI -> OK
✅ Empty trashbin, then: delete file via Web UI -> OK
✅ ocis storage-users uploads list:

Incomplete uploads:
- 4fc553f2-5bef-4530-9e39-324192c11732 (Ad Astra - Zu den Sternen (2019) 2160p AAC.mkv, Size:<PHONE_NUMBER>, Expires: 2022-09-29T12:02:45+02:00)
- 643b98e5-aca9-45b0-91c8-c2f7ae47b9ef (Alien - Covenant (2017) 1080p AC3.mkv, Size:<PHONE_NUMBER>, Expires: 2022-09-29T12:02:27+02:00)
- 9fdb7c60-e3f5-46b0-b9be-e862f70ad963 (American Beauty (1999) 480p AAC.mkv, Size:<PHONE_NUMBER>, Expires: 2022-09-29T12:10:09+02:00)
- a94ea515-0cd7-4af2-b9a3-745874f0e641 (ubuntu-22.04.1-desktop-amd64.iso, Size:<PHONE_NUMBER>, Expires: 2022-09-29T11:54:39+02:00)
- da8134df-e136-40fb-8301-8c3da5df84cd (A Quiet Place 2 (2021) 1080p EAC3 Atmos.mkv, Size:<PHONE_NUMBER>, Expires: 2022-09-29T12:05:23+02:00)
- eebf3f37-a452-4d51-ba9e-21e8f786bebb (A Cure for Wellness (2017) 1080p DTS.mkv, Size:<PHONE_NUMBER>, Expires: 2022-09-29T12:05:35+02:00)

❌ ocis storage-users uploads clean:

Cleaned uploads:
tg@odroidhc4:~/ocis$

uploads is not cleaned:
tg@odroidhc4:~/ocis$ ll /data/ocisdata/storage/users/uploads/
total 14139120
drwx------ 2 tg tg       4096 Sep 28 12:23 ./
drwx------ 5 tg tg       4096 Sep 26 14:48 ../
-rw-rw-r-- 1 tg tg<PHONE_NUMBER> Sep 28 12:04 4fc553f2-5bef-4530-9e39-324192c11732
-rw-rw-r-- 1 tg tg        810 Sep 28 12:04 4fc553f2-5bef-4530-9e39-324192c11732.info
-rw-rw-r-- 1 tg tg<PHONE_NUMBER> Sep 28 12:04 643b98e5-aca9-45b0-91c8-c2f7ae47b9ef
-rw-rw-r-- 1 tg tg        783 Sep 28 12:04 643b98e5-aca9-45b0-91c8-c2f7ae47b9ef.info
-rw-rw-r-- 1 tg tg<PHONE_NUMBER> Sep 28 12:10 9fdb7c60-e3f5-46b0-b9be-e862f70ad963
-rw-rw-r-- 1 tg tg        778 Sep 28 12:10 9fdb7c60-e3f5-46b0-b9be-e862f70ad963.info
-rw-rw-r-- 1 tg tg  253411840 Sep 28 11:54 a94ea515-0cd7-4af2-b9a3-745874f0e641
-rw-rw-r-- 1 tg tg        742 Sep 28 11:54 a94ea515-0cd7-4af2-b9a3-745874f0e641.info
-rw-rw-r-- 1 tg tg<PHONE_NUMBER> Sep 28 12:07 da8134df-e136-40fb-8301-8c3da5df84cd
-rw-rw-r-- 1 tg tg        830 Sep 28 12:07 da8134df-e136-40fb-8301-8c3da5df84cd.info
-rw-rw-r-- 1 tg tg<PHONE_NUMBER> Sep 28 12:07 eebf3f37-a452-4d51-ba9e-21e8f786bebb
-rw-rw-r-- 1 tg tg        792 Sep 28 12:07 eebf3f37-a452-4d51-ba9e-21e8f786bebb.info

@tbsbdr @dragotin
All possible options here were discussed with the team.
Outcome

The uploads in the upload folder are valid and could be resumed by the client any time
We should not "just" delete incomplete uploads
We should not "reserve" disk space at the beginning of each upload because

This makes denial-of service attacks easier
We cannot optimize for the worst case, in many cases (S3NG) the upload folder space gets more free space when another upload finishes successfully

It is very hard to prevent some free space on the disk because

no quota limit is hit
the different filesystems could provide not exact values about the remaining space (e.g. butterFS)

We need to assume that the admin always provides enough memory
We should always make it a best practise to put the data folder on a different partition than the OS

This needs better documentation, follow up in https://github.com/owncloud/docs-ocis/issues/248

The uploads in the upload folder are valid and could be resumed by the client any time
We should not "just" delete incomplete uploads

oC10 cleans unfinished uploads/chunks after 24h


vcenter event driver crashes while making request to sidecar

Bug Report
The vcenter event driver crashes intermittently:
time="2018-05-17T00:06:41Z" level=debug msg="No dedicated handler"
time="2018-05-17T00:06:45Z" level=debug msg="got event of type DvsPortConnectedEvent"
time="2018-05-17T00:06:45Z" level=debug msg="No dedicated handler"
Error: Post http://localhost:8080/: net/http: request canceled (Client.Timeout exceeded while awaiting headers)
Usage:
  event-driver vcenter [flags]

Flags:
  -h, --help                help for vcenter
      --vcenterurl string   URL to vCenter instance (default "https://vcenter.corp.local:443")

Global Flags:
      --config string   config file (default is $HOME/.event-driver)

It appears to crash due to a timeout posting to the sidecar.
Expected behavior

Current behavior

Steps to reproduce

Impact

[ ] Low - Annoyance, but does not impact business or functionality
[x] Medium - Issue can be worked around, but is causing pain
[ ] High - Blocker

Possible solution

Your Environment

Dispatch CLI version (or git commit):
Dispatch Chart version (or image[s] tag):
Operating System and version:
Kubernetes version (and distribution):
Etc (any other useful environmental information):

I haven't seen this in a while... closing


Fix authors field format

This PR change authors field in pyproject.toml to the new style metadata format.

I'd just as soon still leave out the email, especially for the default of "yaq Developers"...
You could do something like default to empty string and omit the , email= with an if len(email)


Variable names such as l, o violate Flake8

Hello,
Just a minor issue to bring to your attention in using variable names, while running pre-commit checks Flake8 complaint about 'E741 ambiguous variable name 'l'' coming from two instances in the model definition : https://github.com/KietzmannLab/BLT-pytorch-CCN23/blob/7e801e0cd7d9c78f05413aa03a18acb0eabdd9e4/BLT_train/models/BLT_net.py#L118
It violates PEP naming conventions.
Cheers,
Varun

interesting. This is the first time I am hearing about PEP. Also, this seems to be a cosmetic issue no? In any case, when we write the code next time, I'll use better variable names anyway. maybe 'iter' is always an option in such cases. Thanks for noticing this.


OMJulia api docu: sample code should work right away

The introduction to OMJulia is nice but still not straight forward with stumbling blocks for new users.
For instance the example code:
using OMJulia
mod = OMJulia.OMCSession()
ModelicaSystem(mod,"BouncingBall.mo","BouncingBall")

doesn't work unless you have the mo file in the proper place.
A new user should be able to run the code just like that.
I'd suggest to provide the required models for download or use a model from the MSL and document all prerequisites, like path to the MSL and so on, such that everybody is able to reproduce the results without being confronted with error messages after executing the code examples and having to trouble shoot all kind of things (e.g. figure out the path to the MSL) that could be easily documented or integrated in the code.

We could switch from the Sphinx documentation from OpenModelica to Documenter.jl and only refer to that documentation from the OpenModelica User's Guide.
It's possible to make all examples inside the documentation test, so we can ensure they are always working as intended and also test them on different OS (Windows and Linux).
Adding it is not too much work, but someone would need to do it.
Regarding your issue: The Readme.md provides an example using the MSL.
We could also use the shipped version of BouncingBall.mo, but it makes the example very hard to understand:
using OMJulia
mod = OMJulia.OMCSession()
sendExpression(mod, "loadFile(getInstallationDirectoryPath() + \"/share/doc/omc/testmodels/BouncingBall.mo\")")
ModelicaSystem(mod,"BouncingBall.mo","BouncingBall")

It's probably best, to just add the source code of bouncingBall.mo into the documentation.
model BouncingBall
  parameter Real e=0.7 "coefficient of restitution";
  parameter Real g=9.81 "gravity acceleration";
  Real h(fixed=true, start=1) "height of ball";
  Real v(fixed=true) "velocity of ball";
  Boolean flying(fixed=true, start=true) "true, if ball is flying";
  Boolean impact;
  Real v_new(fixed=true);
  Integer foo;

equation
  impact = h <= 0.0;
  foo = if impact then 1 else 2;
  der(v) = if flying then -g else 0;
  der(h) = v;

  when {h <= 0.0 and v <= 0.0,impact} then
    v_new = if edge(impact) then -e*pre(v) else 0;
    flying = v_new > 0;
    reinit(v, v_new);
  end when;

end BouncingBall;

The parameter sensitivity is using the lottka voltera example, which is very compact.

We decided in today's dev-meeting to switch to Documenter.jl. I'll create a PR with the initial setup and CI to build and upload the documentation to a GitHub page.
If that is up and running we can add more and more stuff to it.
Some help for what to put into the documentation is greatly appreciated. What is missing in your opinion / what sections are unclear and so on.

@olivleno is https://openmodelica.github.io/OMJulia.jl/dev/quickstart/#ModelicaSystem a better documentation or is there any information missing?

@AnHeuermann thanks for your efforts.
I like the enhancement of the example code showing how to manage result file. Very nice.
Also the note where to find the BouncingBall.mo is very helpful.
But still is my point from above: "doesn't work unless you have the mo file in the proper place." not fully covered.
I'd suggest to extend the note by saying where this file should be copied to such the code works or modify the code such that it directly references the mentioned location.
All this can be considered very very basic stuff that everybody should know, but the path handling can always be a stumbling block if you are using a new environment without knowing how path information is handled, e.g. , the sendExpression(omc, "loadModel(Modelica)") command works out of the box without providing the path to the MSL. If the test models are also shipped with OpenModelica, then one could expect that this location is also known by the environment.

I tested the Scripting API example code in a julia script.
The line:
OMJulia.quit(omc)
throws the error:
ERROR: UndefVarError: quit not defined suggestion: To exit Julia, use Ctrl-D, or type exit() and press enter.

You'll need to update OMJulia to v0.2.2


ceph-osd-device-activation:  a systemd service to trigger underlying LVM device activation

Description of changes
This addresses point #2 of https://github.com/NixOS/nixpkgs/issues/242152, by creating a service to activate the LVM devices underlying the OSDs.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I cannot review this PR, as I'm not using the NixOS Ceph service; we use our own NixOS service which does not use ceph-volume but writes systemd units explicitly (I dumped that as open source somewhere but want to make it available to NixOS in a more cleaned-up way eventually).

I cannot review this PR, as I'm not using the NixOS Ceph service; we use our own NixOS service which does not use ceph-volume but writes systemd units explicitly (I dumped that as open source somewhere but want to make it available to NixOS in a more cleaned-up way eventually).

@nh2, would you mind sharing that in the imperfect state, in the release-early-release-often paradigm, though? : -)

@deepfire Our custom NixOS Ceph service modules: https://gist.github.com/nh2/13425a1f18b4c1ce82edb63c10b163c9

@flokli
From: https://docs.ceph.com/en/mimic/ceph-volume/lvm/activate/#summary

tldr; ceph-volume tries to activate the LVMs themselves if they are not activated already, and... if it was only that, you're right, we wouldn't need it. But, activation step 3 and 4 prepares "stuff" in /var which needs to be present before that start of the OSD's will succeed.
--no-systemd just tells ceph-volume to skip step 2, which would fail hard on NixOS anyway.

@deepfire Our custom NixOS Ceph service modules: https://gist.github.com/nh2/13425a1f18b4c1ce82edb63c10b163c9

Looks very interesting, thank you @nh2!
I'd need to get my long-planned playground Ceph cluster with nixops-libvirtd, before I get to hack on that, though..

@SuperSandro2000, done, thanks!
@flokli, what are your considerations now, after @johanot 's clarification?

I tried to dig deeper into this, but apparently ceph is broken on nixos-unstable and master.
The first point in #242152 also mentions:

Use template service files, particularly for the OSD daemons.

I'm wondering how this plays together with this approach, just want to make sure we don't put ourselves into one corner where it's harder to navigate out of.
Spelunking around in (a successful) build, there's ceph-volume-systemd, which normally seems to get be called by a ceph-volume@… template unit (which we don't ship in NixOS yet).
It seems it takes care of all the necessary activation bits mentioned under the hood as well, so moving to template units should would the necessity for the unit file introduced here alltogether.

@flokli, oh, so we have a better startup point -- so that's great news!
I'd like to play with that, once I get a staging setup with nixops-libvirtd.
So, I guess, it'd be wise to deprecate this PR in favor of the better future implementation.


postSearchHandler returns incorrect value for totalPosts

In the returned PostSearchData, .totalPosts is capped at the value of .limit.
Query with a limit of 5:
curl -s "http://localhost:8081/api/v3/forum/post/search?start=0&limit=5&byself=true" -H "Authorization: Bearer gUtTxxxxxxxxxxxxxxxxxx==" | jq
{
  "posts": [
    ...
  ],
  "start": 0,
  "queryString": "start=0&limit=5&byself=true",
  "limit": 5,
  "totalPosts": 5
}

Query with a limit of 33:
curl -s "http://localhost:8081/api/v3/forum/post/search?start=0&limit=33&byself=true" -H "Authorization: Bearer gUtTxxxxxxxxxxxxxxxxxx==" | jq
{
  "queryString": "start=0&limit=33&byself=true",
  "start": 0,
  "posts": [
    ...
  ],
  "limit": 33,
  "totalPosts": 33
}

They only seem to differ if the limit is larger than the number of results:
curl -s "http://localhost:8081/api/v3/forum/post/search?start=0&limit=200&search=Lorem%20ipsum" -H "Authorization: Bearer gUtTxxxxxxxxxxxxxxxxxx==" | jq
{
  "queryString": "start=0&limit=200&search=Lorem%20ipsum",
  "totalPosts": 195,
  "posts": [
    ...
  ],
  "limit": 200,
  "start": 0
}

Should do the right thing now. I updated the docs on PostSearchData to indicate that totalPosts might not return accurate results due to post-query filtering. Of course, for any paged result set you can't rely on totalPosts being perfectly accurate anyway (as the db could get changed between calls).


Backend Doesn't work with unifiednlp with GAPPS

Currently not1working with the newly unifiednlp (which doesn't require Gapps deletion)

Can you get me the version numbers of the software you are using and a logcat? Thanks!
(Has the new gapps compatible version of UnifiedNLP shown up on F-Droid yet? That is where I get mine from and I am showing that component as up to date on my phone.)

Oh, I think I understand why it's Not working. I live in france so I have to change mcc filter to 208 (number for france ). Am I right ?

Yes, according to http://en.wikipedia.org/wiki/Mobile_country_code#F the mobile country code for France is 208 so if that is where you are that is the code you should use.

Fixed, thanks


Server sometimes leaves a child process running after crashing

When I start the server/worker using run_combined.sh it starts two processes which are running dpxdt/tools/run_server.py - one a child of the other. When one of them dies as in #163 the other remains running.

Only an issue using the development deployment instructions so I'll close this.


small problems with menclose

Issue Summary
There are some issues with positioning and length of 'strikes' arrows in menclose.
Also, RTL direction is not being handled (an issue only if RTL is meant to be handled).
Description:
Position and Length
Overall, there are some small rendering errors with the positioning of the strikes and arrow notation values. There are a few that are larger. You can see the contrast in the following. In the first example below, the arrows are close to correct -- the lines should go through the center of the '+' and through the division line as those are the centerpoint of the box. Within a subpixel, these are correct (shown zoomed at 200% to make it easier to see, but it is off at 100% also):

A problem with the above rendering is that the diagonal arrows are a little short and don't reach the box around the contents; the horizontal and vertical lines do touch the box.
If only left or right is given (as opposed to box which is both left and right), the position is worse:

I believe the problem occurs because the border padding is only added on one side. The positioning of the vertical line/arrow should not be affected by other decorations (as per the spec).
A similar problem happens with the horizontal positioning and top and bottom:

RTL
I didn't see anything in the MathJax docs that said MathJax supports right-to-left rendering. If it does, then you missed support of it on menclose. Here are three cases that should switch according to the spec:

Technical details:
I have put a large suite of test examples in codepen. It is using MathJax (see the JS section for details)

MathJax Version: 3.1.2 (CHTML Output Jax)
Happens with Chrome and Firefox.

Kudos:
I discovered these problems when working on a polyfill for MathML Core. I used the clever ideas for drawing the pieces with CSS borders in my polyfill. A big round of applause to whomever came up the CSS for drawing the arrowheads! I wouldn't have thought you could do that if I hadn't seen it written down.

Thanks for the report.  You are correct that the misplacement is due to the change in size on one side only.  I will look into how to resolve that.
For 19 and 20, do you mean that the arrow should be on the fraction line (not the center of the plus)?
We don't currently support RTL, but would like to in the future.
Thanks also for the kudos.  The arrows are my doing.  I was pleased that I was able to do all the enclosures with CSS, and didn't have to resort to SVG as I did in v2.

You are correct that the the text for 19 and 20 is incorrect -- copy/paste that wasn't updated. I'll fix up the codepen.


Add proto3 encoding

We should eventually map out a proto3 encoding even if this is just a direct translation from the json. I thought we had an issue, but I don't see it.
In the mean time, we should at least make it possible in the java library to do this. Minimally, this requires adding zipkin2.codec.Encoding.PROTO3 to permit this format when using the async reporter.

Looking at docs and generated code in proto3, I think the following are true for our key functions. Note that I'm assuming zipkin will use a field number less than 128 for the span data (StackDriver is field number 1, so yeah works for them too).

int listSizeInBytes(int encodedSizeInBytes)
The overhead is the field number then the size
uint(field_number) + uint(encodedSizeInBytes)

int listSizeInBytes(List<byte[]> values)
The overhead is repeating above for each element (as messages can't be packed)

@basvanbeek this seem to gel with your understanding of proto3?
It looks like what the generated code does:
    size = 0;
    for (int i = 0; i < traces_.size(); i++) {
      size += com.google.protobuf.CodedOutputStream
        .computeMessageSize(1, traces_.get(i));
    }

https://developers.google.com/protocol-buffers/docs/encoding

FYI @saturnism I will try to get this in by zipkin 2.6 which should land sometime in the next week or so depending on the UI-related changes that are still in flight. After that, it should be possible to make the zipkin-gcp sender using stackdriver-specific proto3 encoding even if we haven't defined a proto3 zipkin fomat yet

will this eventually lead to zipkin starting as a grpc server ? or it's really just the protobuf encoding of the traces at this point ?

will this eventually lead to zipkin starting as a grpc server ? or it's
really just the protobuf encoding of the traces at this point ?

no. this is just data format. similar to how we handled thrift, to the
degree we have core lib support for grpc, we'd do it in a zero dependency
way. gRPC is a somewhat toxic dependency already, for example, if we
packaged anything we'd have a wrestling match with zipkin-grpc for the
version of boringssl + grpc etc. This is more about making the data format
possible, and later defined.

s/to the degree we have core lib support for grpc/to the degree we have
core lib support for proto3/
sorry!


Add CI job to check code formatting

Summary
Run CI job to check formatting (what would black do?) and order of imports (what would isort do?).
Details
Provide immediate feedback on PRs whether the code meets our coding style standards.

We now have GitHub Workflows: https://github.com/harrystech/arthur-redshift-etl/tree/next/.github/workflows


suggestion: quotes exclusion

Firstly, thank you so much for this underrated tool.
I think there should be a distinction between an article's own tone of storytelling, and a quote containing sensational words being mentioned; correctly attributed quotes should be exempted from the evaluation because it affects the author's bias when it is factual reporting.
Thank you again and keep up the great work.

Thank you for your positive feedback!  You're right; this version is a basic demo. We're developing future versions to clearly distinguish between quotes and the author's text. Thank you again for taking the time to write your suggestion 😊


Allow static headers for OTLP exporter

A new parameter is added which can be used to specify static headers to be included in the HTTP requests
Defaults to an empty map

@brendanmaguire Should we have added the same parameter to io.janstenpickle.trace4cats.opentelemetry.otlp.OpenTelemetryOtlpHttpSpanExporter?

@brendanmaguire I think it should've been List[(CIString, String)] since duplicate header names are allowed

Yep. Make sense to me.


Lack of function for computing the inverse of a matrix from its Cholesky factor

Summary:
We do not have a function that inputs a Cholesky factor L and outputs (L * L')^(-1).
Description:
Such a function (for upper triangular Cholesky factors) exists in LAPACK and R under the name chol2inv.
Expected Output:
(L')^(-1) * L^(-1)
Additional Information:
I have written such a function.
Current Version:
v2.17.0

@bgoodri, it's gonna be pretty exhausting if you want to call things "stupid" all the time. There are a few things you can do:

[ ] (easy) remove the section when you create the issue
[ ] (better) use the section to describe actual tests
[ ] (best) be constructive and suggest a change to the template that's better than what we have

I'm sure there are other possible things you can do.
If you didn't know, the pull request template is in the code base here: https://github.com/stan-dev/math/blob/develop/.github/ISSUE_TEMPLATE.md
We are using GitHub to the best of our ability. There is currently no ability to conditionally serve a template based on the label. Whatever process we have is fluid and we can change it.
Thanks. (I couldn't tell if that was in jest -- it's really hard to read emotion through an issue.)

I agree with @syclik here.  Please try to not insult people on GitHub or our forum.
When someone calls something you've done "stupid", it feels like they're calling you "stupid" and it hurts.

I am fine with removing sections, which I had assumed were obligatory. I should have said "Having to fill out a section for Reproducible Steps for a missing feature is stupid."

I understood that.  I'm pretty thick-skinned myself, but I've worked with a lot of people who take "stupid" as the worst possible insult, so I think we should just avoid the term (and synonyms).   Having said that, I have a hard time following that advice myself, being particularly prone to overuse "idiotic" in relation to what I think are really bad ideas.

I didn't read the original remark as an insult, but as frustration directed at the process / template. I'd rather just find a way to make the process better than have it be a source of frustration. Anyway, feel free to remove when you create an issue when it's not appropriate or suggest something more appropriate to replace it with.


retry afterEach

Thanks for the plugin - a great step forward - it seems overall to work for us, except.. we have some assertions in afterEach which cause failures.. and we want to retry.
Are there any plans to add support for this? Or any ideas to always run an assertion on every test other than calling a custom command in every it() ?

@lukeapage I have only ever used an AfterEach for teardown code, but I understand why you would want an assertion there. I'll give this some thought.
You could try proxying the it method like so:
      describe.only('some suite', () => {

        const itWithTest = (fn) => (...args) => {
          it(args[0], () => {
            args[1]()
            fn()
          })
        }

        const it_ = itWithTest(() => {
          expect('foo').ok
        })

        it_('can test', () => {
          expect(true).ok

        })

        it_('can test 2', () => {
          expect(true).ok
        })

      })

but then you'll lose calls to .only/.skip unless you add more complexity to the itWithTest function.

Thanks for the work-around!
I can apply this patching to the global it since thats where I previously had the afterEach..
function afterEachCode() {
    // custom test code
    expect('foo').ok
}

const _it = it;

const makeItFn = (it) => (name, fn) => {
    it(name, () => {
        fn();
        afterEachCode();
    });
};

// eslint-disable-next-line no-global-assign
it = makeItFn(_it);
it.only = makeItFn(_it.only);
it.skip = makeItFn(_it.skip);

@lukeapage sounds like a good solution. I would also recommend putting a Cypress.log in there like so, to quickly identify where a failure is.
fn()
Cypress.log({type:'afterEach'})`
afterEachCode()

I can apply this patching to the global it since thats where I previously had the afterEach..

Would love to get more info on how to implement this?

afterEach hooks are retried as part of test retries added in Cypress 5.0. https://docs.cypress.io/guides/references/migration-guide.html#Tests-retries


tabindex chain is broken when selecting a value with keyboard

Hello,
The following problem occurs with 3.4.6, it was OK with 3.4.5. Maybe is it linked to "#2176 Fix Focus war with modals" ?
I have a form with some input, and a select in the middle. No tabindex attribute are specified on the elements. By using the "Enter" or "tab" key, I select a value in the select element. Then it seems that tabindex is reset, because the following element in the form doesn't get the focus : the tabindex chain is broken. This also occurs when setting tabindex attributes on the form elements.
With 3.4.5, it was fine : when  the value was selected in the select with the Enter key (or tab), then taping tab set the focus to the next form element.
Thanks for your answers,
Sébastien.

I am still having same issue :(


onAttach and onRender are not executing

As-salamu-alikum
Zazakallah khair for sowing interest to my issue.I am very new to marionette.Please pardon me if you get angry with my issue.
My issue is when I attach a view via a component i.e
var region = myApp.getRegion();
myComponent.showIn(region);
var MyComponent = Marionette.Toolkit.Component.extend({
  ViewClass: MyViewClass,

not by
myApp.showView(new myView())
only myComponent's
onBeforeShow ,
onShow,
onBeforeRenderView,
onRenderView

fire but myView's onAttach and onRender do not fire.Is there any way to execute onAttach , onRender and other callbacks within the framework through the execution of myComponent.showIn() function.i.e I do not want to do anything for the callbacks triggering in  myComponent defination except the ViewClass defination?
I am very weak in English as well as in programming, so please pardon me again if I have done any mistake.
Zazakallah khair again for reading my post with irritating english by spending your valuable time with passions
Allah-hafiz
Nadvi

I'm not 100% sure I follow, but the component does not get the same events as the view.
You should get:
var MyComponent = Marionette.Toolkit.Component.extend({
  onViewBeforeRender() { }
  onViewRender() { }
  onViewBeforeAttach() { }
  onViewAttach() { }
});

Does that answer the question?

Zazakallah khair for answering my issue by spending your valuable time.
Sorry for my weakness in english as well as in english may I do not understand your responce and also might not convey my issue.My issue is that, is it possible to trigger attach, before:attach, render, before:render on the view from the 2 function below in Component portion of Marionette.Toolkit.Component
 show: function show(viewOptions) {
      var region = this.getRegion();

      if (this._isShown) {
        throw new Marionette.Error({
          name: 'ComponentShowError',
          message: 'Component has already been shown in a region.'
        });
      }

      if (!region) {
        throw new Marionette.Error({
          name: 'ComponentRegionError',
          message: 'Component has no defined region.'
        });
      }

      this.triggerMethod('before:show');

      this.renderView(viewOptions);
      this._isShown = true;

      this.triggerMethod('show');

      // Destroy the component if the region is emptied because
      // it destroys the view
      this.listenTo(region, 'empty', this._destroy);

      return this;
    },
renderView: function renderView(options) {
      var ViewClass = this._getViewClass(options);

      var viewOptions = this.mixinOptions(options);

      var view = this.buildView(ViewClass, viewOptions);

      // Attach current built view to component
      this.currentView = view;

      this._proxyViewEvents(view);

      this.triggerMethod('before:render:view', view);

      // _shouldDestroy is flag that prevents the Component from being
      // destroyed if the region is emptied by Component itself.
      this._shouldDestroy = false;

      // Show the view in the region
      this.getRegion().show(view);

      this._shouldDestroy = true;

      this.triggerMethod('render:view', view);

      return this;
    },

I am very weak in English as well as in programming, so please pardon me again if I have done any mistake.
Zazakallah khair again for reading and answering my post with irritating english by spending your valuable time with passions.May Allah bless you for your effort
Allah-hafiz
Nadvi

Yes. The view should get both render and attach events.
A code sample may help to talk further:
https://jsfiddle.net/msqbLq9k/

I would also recommend asking on https://gitter.im/marionettejs/backbone.marionette if you want/need faster answers.  sometimes they are very fast in gitter.


Unlike not possible for Contexts

Example: you can Like and Unlike Photos or Discussions, but you can only Like Groups or Organizations and cannot Unlike them.

By default Content modules use Reactions based Voting while Context modules - Like based one.
By default Unlike is disabled in Like based Voting. It can be enabled via DB.

Reactions will be integrated in Contexts (Profiles/Groups based modules) in the next major UNA version: Ticket #3873.

The possibility to do Unlike is controlled with IsUndo field in sys_objects_vote table.


python310Packages.bthome-ble: 2.7.0 -> 2.8.0

Changelog: https://github.com/bluetooth-devices/bthome-ble/blob/v2.8.0/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 219739 run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.bthome
python310Packages.bthome-ble
python311Packages.bthome-ble


Header elements are not centered

Hi,
I've just updated to beta 5 and notice that when you change the header height, the header elements are not vertically aligned (because it ignores the height of the status bar).

My StyleSheet is basic:
const styles = StyleSheet.create({
  background: {
    backgroundColor: colors.primary,
    height: 80,
  },
  title: {
    fontSize: 16,
    fontWeight: '400',
    color: 'white',
    backgroundColor: 'yellow',
  },
  button: {
    backgroundColor: colors.transparent,
    color: 'white',
    fontSize: 16,
    backgroundColor: 'yellow',
  },
})

I believe it's a duplicate, anyways, my pending PR changes the way layout is calculated and centered (no absolute positioning) and I believe that should fix the issue.

As I suspected, it's no longer an issue.

static navigationOptions = {
    ...
    header: ({ state, setParams }) => ({
      style: { height: 40, backgroundColor: '#0099ff', justifyContent: 'center' },
      titleStyle: {  },
      tintColor: '#fff'
    })
  }


Nordic: support for nRF21540 Devkit in Zephyr

nRF21540 Front End Module Development Kit board definition
nRF52840 MCU definitions based on nRF52840 Devkit files

Signed-off-by: Karol Lasończyk<EMAIL_ADDRESS>

@kl-cruz thanks for the PR. Shouldn't this board enable the following options in the BLE Controller by default?

https://github.com/zephyrproject-rtos/zephyr/blob/master/subsys/bluetooth/controller/Kconfig.ll_sw_split#L486
https://github.com/zephyrproject-rtos/zephyr/blob/master/subsys/bluetooth/controller/Kconfig.ll_sw_split#L517

I'm sure @cvinayak can help you test this if you have questions.

Please note that we plan to add nRF21540 pin configuration to dts describing this board. You can find WIP draft of this idea here: https://github.com/nrfconnect/sdk-nrf/pull/2594.
It seems that it is in conflict with Kconfig-based configuration used for BLE Controller:

https://github.com/zephyrproject-rtos/zephyr/blob/07b4df1945b59d37139433ec97cb3cae0e1f33a2/subsys/bluetooth/controller/Kconfig.ll_sw_split#L496
https://github.com/zephyrproject-rtos/zephyr/blob/07b4df1945b59d37139433ec97cb3cae0e1f33a2/subsys/bluetooth/controller/Kconfig.ll_sw_split#L527

I think we should unify these approaches.

@carlescufi @hubertmis Thank you for comments!
I see no direct option in Zephyr to configure and setup NRF21540 FEM. These options that you pointed looks promising, but in my opinion it needs to be refactored. I think it will be another task than this one. It is only board support one.
As first approach I'd like to add board to compilation process and it works. My starting point was nRF9160 DK. There are two independent SoCs, but connected together.

@kl-cruz , I definitely agree. My intention was to point out that sooner or later we shall find a way to add FEMs to device tree and align all Nordic radio protocols with such configuration. But I want to be clear that such activity is out of scope of this PR. I think we should keep this PR as simple as it is now and work on FEMs modeling in dts separately.

Added support for the nRF21540 DK in the ADC test

I think we should unify these approaches.

I completely agree. I think this should be discussed together with @anangl to come up with a plan and remove the Kconfig options that I linked.


Allow to pass keyword arguments to itensor mps estimator for fast computation.

Description
This PR allows users to pass keyword arguments to jl.apply in the auxiliary function _estimate. Note that jl.apply Julia function ITensors.apply. By passing maxdim, it enables us to calculate the expectation value at high speed by obtaining the Truncated SVD in ITensors.factorize and svd. These functions are called from the ITensors.apply.

[x] I have added tests that verify the behavior of the changes I made.

I have read the CLA Document and I hereby sign the CLA

I think the updates here can also be applied to Sampler. Can you do the same for it?

Sure, leave it to me!

Sure, leave it to me!

Done. Ready for review 🙏


Apply latest Docker CI login on Ops-Eng team repo's GH Actions

Docker have changed the way that GH Actions should log into Docker and push and pull images.
This Task is to update the GH Actions to use this method.
It has been implemented already here:
https://github.com/ministryofjustice/tech-docs-github-pages-publisher/blob/9add8364020c830c7c2fe1925885fe6a526fcfc5/.github/workflows/docker-hub.yml#L15

Applies to these repo's:
https://github.com/ministryofjustice/cloud-platform-maintenance-pages/pull/25


Incomplete CDS response cause cluster_not_found error during xds update

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
During CDS update, istiod might return incomplete CDS response, thus causing cluster_not_found error for local cluster inbound|80||. However, after <1 min, the istiod seems to start to return correct CDS response, as the cluster_not_found error is gone.
envoy proxy log:
CDS: update / add cluster inbound|1080||, and remove inbound|80||

cluster_not_found error started to happen shortly

istiod log:

Version
data plane version: 1.20.0
istiod version: 1.20.0

Additional Information
No response

Can you please share istiod log and proxy log?

cc @hzxuzhonghu

https://github.com/istio/istio/pull/49507 Was backported, but not 1.20.0. Can you upgrade to have a test

Thanks for responses, I'll report it back once we try out the new version.

The fix was not backported to 1.20


This PR replaces hardcoded home folder path with ~

for better portablility

@unxed , I'll apply your change but not via this pull request as I want another change in this function.
Just to avoid too many commits. Thanks.

No, it does not work. Probably ~ is good for the terminal command line but not for fopen().

maybe this solution will help?
https://stackoverflow.com/questions/35165680/writing-a-file-to-users-home-directory-in-c

Yes, this solution must be OK.
This Log function is in fact a temporary solution.
Under Windows there is OutputDebugString function which is very useful. I use it all the time.


[Bug]"Require approvals" unchecking instruction required in Step 4.4 for consistency

Describe the bug
When following the current instruction of Step 4, learners are later forced to do
Merge without waiting for requirements to be met (bypass branch protections) in Step 5 to proceed
and this does not meet with the purpose of setting up branch protection (somewhat contradicting)
To Reproduce
Steps to reproduce the behavior:

start the course of continuous-integration
complete until Step 3
in Step 4.4, check Require a pull request before merging as suggested in #8
doing so will automatically insert check into Require approvals and set required number of approvals to 1 (Screenshot 1)
since the learners are likely to have only him/herself in the learning repository, he/she cannot approve the pull request on the own in Step 5 and end up doing Merge without waiting for requirements to be met (bypass branch protections) to proceed (Screenshot 2)
this does not make sense since learners are forced to bypass branch protection they just configured

Expected behavior
learners must be instructed to uncheck Require approvals in Step 4.4
Screenshots

Screenshot 1

Screenshot 2

Device information

in any environment

Additional context

None

I strongly welcome pull requests if folks have ideas on how to fix up this course :) We struggled with this one for sure figuring out how to approach it, so we welcome new ideas.


Incorrect Color Reproduction For Specific Color Spaces

Original PDF -
Original_Test_Doc.pdf
PDF Converted to Adobe RGB (1998) -
Original_Test_Doc_RGB1998.pdf
Configuration:

Web browser and its version:
- Firefox 61.0.1 (64 bit)
- Chrome 67.0.3396.99 (64 bit)
Operating system and its version: Mac OS High Sierra 10.13.4
PDF.js version: Latest
Is a browser extension: No

Steps to reproduce the problem:

Open https://mozilla.github.io/pdf.js/web/viewer.html
Load PDF

What is the expected behavior? (add screenshot)

What went wrong? (add screenshot)

Using the Original_Test_Doc.pdf file above in the viewer produces an incorrect color. After inspecting the attached pdf file, I believe the color type for the image that is discolored is of type "Spot Color." From what I can find online, it seems that converting this color type into sRGB (The format I think PDF.JS is using) is not very straight forward, so it is possible PDF.JS does not support it.
I tried converting the original PDF into the Adobe RGB (1998) color profile using Adobe Acrobat and the results seem to be accurate to the original PDF (Converted file attached at top).
Does PDF.JS support PDF's that contain images from these specific color spaces? Is there a list of acceptable color spaces somewhere in the wiki that I may have missed? If PDF.JS does not support this color space, what would be the recommended course of action to take in order to convert the PDF into something more PDF.JS friendly other than running each document through Adobe Acrobat?

I think this is what you're looking for:
https://github.com/mozilla/pdf.js/blob/master/src/core/colorspace.js#L212-L255
Those are the colorspaces that we parse/support. If the colorspace is not supported, you should see an error in the console.

I am not seeing an error in the console using the above PDF. Does this mean it is supported?

I looked into this PDF file a bit more and it's using the CalRGB colorspace, which is supported, though it looks like something is going wrong in the conversion steps here. It looks like this is a bug in the colorspace conversion; thank you for reporting this.

@timvandermeij I started to debug and may have found what is causing the issue.
colorspaces.CalRGBCS() line 939 ... this.MXB = matrix[3];
For my pdf, this value is being set to a negative number (-0.00002). This causes the check at line 964:
if (this.MXA < 0 || this.MYA < 0 || this.MZA < 0 || this.MXB < 0 || this.MYB < 0 || this.MZB < 0 || this.MXC < 0 || this.MYC < 0 || this.MZC < 0)
to fail and reset the values for: this.MXA, this.MYB, this.MZC, this.MXB, this.MYA, this.MZA, this.MXC, this.MYC, and this.MZB.
I noticed that by modifying the negative matrix value to a positive value and forcing the test to pass at run time, the resulting pdf's color is MUCH closer to the correct result.
I am trying to track back to where this value is first set. The furthest i can find is in the method "loadResources" in document.js , which eventually creates the value xobj (which i believe contains Stream/Dict/xRef ... /map/matrix) in Evaluator.js. Is this value set by PDF.JS or is it generated from the contents of my pdf?
I am hoping that if I can fully understand where/how exactly this matrix variable is being set, that I will be able to determine what is generating this negative value and causing my issues.
Sorry for any confusions with the above text, this is my first time working with JS. Please ask for clarification if any of the above does not make sense.

This seems to correct my issues. Are there any unseen consequences of removing this check (I have yet to run the lint tests)? Also, for future reference, what specs are you referring to?
If there is no reason for this check to be in the code, would you like me to contribute my changes/generate a PR?

I don't see any issues with removing that check on the matrix values. It's normal to have negative values in a transformation matrix.
I'm referring to the PDF spec portion on CalRGB https://www.adobe.com/content/dam/acom/en/devnet/pdf/pdfs/PDF32000_2008.pdf#G7.1850396
A PR would be great! If you can, please also add this file as an "eq" ref test file so we don't regress this issue. For an example on doing this, see the changes to test/test_manifest.json, test/pdfs/.gitignore in https://github.com/mozilla/pdf.js/pull/9827/files


fix the height error it is not working

I use the code from the npm I have installed all of the packages but it is saying style property 'height' is not supported by native animated module

Hello, I have the same problem

hi @abhishekv-gloify  nope but I use another package that works https://www.npmjs.com/package/@gorhom/bottom-sheet

Hi @Beki78 , i am also getting same error. have you found any solution.

hi @abhishekv-gloify nope but I use another package that works https://github.com/nysamnang/react-native-raw-bottom-sheet/issues/190#issuecomment-2273571385

Did anyone found a solution? caus I'm still getting an error


mpg, remove lro workaround, use getLroMetadata

Seems getLroMetadata is working in typespec-azure-core 0.37.2.

I assume we had an LRO in arm.tsp?

I assume we had an LRO in arm.tsp?

Yeah: https://github.com/Azure/autorest.java/blob/main/typespec-tests/tsp/arm.tsp#L82
https://github.com/Azure/autorest.java/blob/main/typespec-tests/src/main/java/com/cadl/armresourceprovider/implementation/TopLevelArmResourceInterfacesClientImpl.java#L351


Doorman docker + nginx configuration (502 Bad Gateway)

Hi there,
I'm having trouble trying to make Doorman work with Docker and Nginx to be able to access it remotely. I'm new to docker so I might be doing something quite dumb. I hope it's not the case...
I run the Doorman docker container as:
# docker run -d -e DOORMAN_ENROLL_SECRET=<random_uuid_from_osquery_client> -e DOORMAN_SECRET_KEY=<invented_uuid> -e POSTGRES_USER=doorman -e POSTGRES_PASSWORD=<invented_pwd> -e POSTGRES_PORT=<IP_ADDRESS>:5432 -e POSTGRES_ENV_POSTGRES_PORT=<host_public_ip> -p <IP_ADDRESS>:12345:5000 doorman
ef11e83e774fc39c778d1a67ee8132c640b8ef853c03259f872922d2d6b0f053

Container listed as:
# docker container ls
CONTAINER ID        IMAGE               COMMAND                  CREATED             STATUS              PORTS                       NAMES
ef11e83e774f        doorman             "runsvdir /etc/ser..."   13 seconds ago      Up 10 seconds       <IP_ADDRESS>:12345->5000/tcp   elegant_mcclintock

Nginx configuration:
# cat doorman
server {
	listen 4433;
	ssl on;
	ssl_certificate /opt/doorman_nginx/certs/osquery_certificate.crt;
	ssl_certificate_key /opt/doorman_nginx/certs/osquery_private.key;

	location / {
		proxy_pass https://<IP_ADDRESS>:12345;
		proxy_redirect off;
		proxy_set_header Host $host;
		proxy_set_header X-Real-IP $remote_addr;
		proxy_set_header X-Forwarded-For $remote_addr;
		proxy_set_header X-Forwarded-Proto $scheme;
	}
}

When I access Doorman from the browser as https://ip:4433 I get a 502.
There are no errors in the error.log from nginx.
Just want to mention that when I run the container with the previous command, it's when I input for the first time both postgres password and doorman secret key. Is that all right?
Netstat shows me how port 5000 and 12345 are listening:
tcp        0      0 <IP_ADDRESS>:4433            <IP_ADDRESS>:*               LISTEN      off (0.00/0/0)
tcp        0      0 <IP_ADDRESS>:12345         <IP_ADDRESS>:*               LISTEN      off (0.00/0/0)

It looks like the server spend some time processing the request, but eventually it fails.
As I see it, nginx listens at 4433. It passes the request to the host port 12345, where docker is listening and where it sends it back to the docker container port 5000, where doorman should handle it. Am I mistaken?
Any ideas?

It looks I cannot even access doorman from inside the container.
# docker exec -it 84b8 bash
bash-4.3# ps axu
PID   USER     TIME   COMMAND
    1 root       0:00 runsvdir /etc/service
    6 root       0:00 runsv api
    7 root       0:00 runsv redis
    8 root       0:00 runsv celery
    9 doorman    0:10 {celery} /usr/bin/python2 /usr/bin/celery worker -A doorman.worker:celery -l INFO
   10 redis      0:01 redis-server /etc/redis.conf
   11 doorman    0:02 {gunicorn} /usr/bin/python2 /usr/bin/gunicorn --workers 4 --bind <IP_ADDRESS>:5000 manage:app
   18 doorman    0:03 {gunicorn} /usr/bin/python2 /usr/bin/gunicorn --workers 4 --bind <IP_ADDRESS>:5000 manage:app
   21 doorman    0:03 {gunicorn} /usr/bin/python2 /usr/bin/gunicorn --workers 4 --bind <IP_ADDRESS>:5000 manage:app
   22 doorman    0:00 {celery} /usr/bin/python2 /usr/bin/celery worker -A doorman.worker:celery -l INFO
   23 doorman    0:00 {celery} /usr/bin/python2 /usr/bin/celery worker -A doorman.worker:celery -l INFO
   24 doorman    0:00 {celery} /usr/bin/python2 /usr/bin/celery worker -A doorman.worker:celery -l INFO
   25 doorman    0:00 {celery} /usr/bin/python2 /usr/bin/celery worker -A doorman.worker:celery -l INFO
   26 doorman    0:02 {gunicorn} /usr/bin/python2 /usr/bin/gunicorn --workers 4 --bind <IP_ADDRESS>:5000 manage:app
   27 doorman    0:02 {gunicorn} /usr/bin/python2 /usr/bin/gunicorn --workers 4 --bind <IP_ADDRESS>:5000 manage:app
   97 root       0:00 bash
  103 root       0:00 ps axu
bash-4.3# wget https://<IP_ADDRESS>:5000
Connecting to <IP_ADDRESS>:5000 (<IP_ADDRESS>:5000)
wget: error getting response: Connection reset by peer

Interesting enough, I cannot see postgresql anywhere in the process list...


Removed unneeded hyphens in "counterintuitive".

Follow-up to 65ad4ade74dc9208b9d686a451cd6045df0c9c3a which added counterintuitive to the wordlist. Removes unneeded (antiquated) hyphenated usages.
See e.g. https://www.merriam-webster.com/dictionary/counterintuitive

I was surprised that I needed to add this word, surly it has been used before in the docs I thought, but that explains it.


Portfolio: preserve data in the browser

Changes we (@Tadesse-Alemayehu💻 and @Juanse7793💻) made.

Create a localStorage and JavaScript object for storing form data.
Manipulate JSON to store and use data inside local storage.
Update the form element value by fetching data from local storage.
Handle exceptions ❌ that might be raised in creating a local storage object.

@prolajumokeoni Thank you for the review.


[Bug]: not able to download file on MacOs - add folder access request

Before you file a bug report

[X] I have checked the issue tracker and have not found an issue that matches the one I'm filing.
[X] This issue is not a troubleshooting question. Troubleshooting questions go here: https://forum.mattermost.com/c/trouble-shoot/16.
[X] This issue is not a feature request. You can request features and make product suggestions here: https://mattermost.com/suggestions/.
[X] This issue doesn't reproduce on web browsers (such as in Chrome). If it does, issue reports go to the Mattermost Server repository.
[X] This issue reproduces on the most recent stable version, or the most recent prerelease version of the Mattermost Desktop App.
[X] I have read the contribution guidelines.

Mattermost Desktop Version
5.3.1
Operating System
macOS Ventura 13.3.1 Apple Silicon
Mattermost Server Version
No response
Steps to reproduce
When downloading a file, it says "cancelled"
Expected behavior
File must be downloaded to the Downloads folder
Observed behavior
File cannot be download (I have to copy the public link and download it through my browser).
It says cancelled and when I click on it, I receive the following Mac notification: The application “Mattermost” does not have permission to open “Downloads”.
Log Output
No usable log output in the logs

Additional Information
No response

This has been fixed here: https://github.com/mattermost/desktop/pull/2699
It will be released in v5.4. You can preview the release here: https://github.com/mattermost/desktop/releases/tag/v5.4.0-rc.1


Open Menu

What's the Problem?
So when you try to upload an image or something, or import something to an application it still uses the old explorer despite setting files as default in the experimental settings
Solution/Idea
If it's possible to make it so files shows up instead of the old explorer, that'd be very nice
Alternatives

Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
No response
Windows Version
No response
Comments
No response

@BragonGod this isn't posible right now but it's something that we're exploring.


Progress

As a learner, I have some sense of how many tasks I've completed and how many more I have to complete.

We can measure progress using the code challenges inside every lesson, it is much simpler than tracking every time that a user clicks next after reading all the introductory/lesson part (but if we have to, we'll do it).
What Exercism seems to do is that every-time a challenge is passed(tests are working) they generate a solution, to thats how they can respond whenever a user asks: ¿Whats my current progress, how many and which challenges have i completed?

Overall progress will be visible on the Chapters page (see here in Figma). When you're in a chapter, the top bar indicates your progress.
We deal with a linear sequence of chapters and challenges, so from a technical side, maybe it's enough to just track which chapter & challenge the user is on at the moment?
Any way to update the title of this issue to a more narrow scope?

#86 partially addresses this.

Going to close this meta issue for now given we have more discrete tasks to focus on.


pandoc: migrate to Homebrew

Deleting pandoc from cask as part of deduplication process . See context here.
Please note that this PR is created to continue case-by-case discussions from thread linked above and if decision is made not to remove the cask, PR will be closed.

Your pull request was merged directly in 8fd4313edc8d93877c3d7f76400572124017d678 to keep commit history cleaner. It needed some changes which were made in 8fd4313edc8d93877c3d7f76400572124017d678. Thank you for your contribution!


Make sure we don't run integration cli test if compilation fails

Otherwise, while using test-integration-shell, it runs the tests using the previously compiled test binary.
To verify this, on master, run :
$ make BIND_DIR=. shell
# […]
$ ./hack/make.sh binary test-integration-shell
# […]
$ bundle_test_integration_cli
# […]
# Should run the entire suite
# Now edit a file so that it doesn't compile
# and re run the tests
$ bundle_test_integration_cli
# […]
# There should be a compilation error
# BUT it will run the entires suite again, using the old binary

/cc @tianon @dnephin @cpuguy83
🐸

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "integration-test-shell-dont-run-if-compilation-failure" git@github.com:vdemeester/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.


Session 05

Exceptions Lab and Comprehensions Lab. Please take a look at everything except Trigrams. Maria has already provided feedback for Trigrams.

More good work Paul.


Support for latest ytmusicapi

ytmusicapi was released as 0.24.0 a couple of days ago with some breaking changes.
A quick search in the source showed that get_playlist_suggestions and get_watch_playlist_shuffle are not used.
Currently there is a constraint present ytmusicapi>=0.22.0,<0.23.0 which will break packages shipped by distributions when they update ytmusicapi.
Any chance to get support for 0.24.0?
Thanks.

Already happened for Manjaro (which is 2 weeks behind Arch so it's safe to say it's broken for Arch as well):
Disabled extension ytmusic: ytmusicapi<0.23.0,>=0.22.0 required, but found ytmusicapi 0.24.0 at /usr/lib/python3.10/site-packages

I've widened the version requirements for ytmusicapi, so this should be fixed with 0.3.8.

Works fine with 0.24.0 and 0.25.0 (current - just updated). This can be closed.

At this point with 1.0 and up released there's new authentication workflow as well (it can do proper OAuth now!). So, what will it take to get updated to that?

I've got a basic PR in that makes ytmusicapi > 1.0.0 work, but nothing about supporting OAuth yet. I'm not supre well versed in python so I didn't dive much into getting that part working, just wanted to fox it on my Arch system

I guess #71 should be marked as duplicate.

It could really be nice that this project lives again, since oAuth is officially supported.
The expiring auth credentials were so annoying I had given up on Mopidy-YTMusic.


feat: auto-generate overloads for Client.wait_for

Summary
Part 1/? of event QoL work, adds just a few overloads (106, to be exact) to Client.wait_for, one for each event.
This allows for correct typing of the check parameter and the return type, particularly when used with the Event enum (see below).
These are being auto-generated through yet another custom libcst module. While certainly not ideal, there isn't a better way to do this with Python's current typing system due to several reasons.
We may want to consider merging overloads with the same return types, I'm not sure if that affects pyright performance in any significant way.
That one big lookup table is a separate module as part of the library, since it'll be reused and expanded in future PRs with runtime usage.
Some other things:
Fallback for str
One of the overloads takes an unspecific str event. This is meant to be the fallback overload for anyone using the existing event infrastructure in the lib for custom events, since doing that would otherwise run into typing issues:
https://github.com/DisnakeDev/disnake/blob/a52790d008139633723d128c6de9a5d71c367ba4/disnake/client.py#L2742-L2751
This, however, means that incorrectly typing a library event handler also falls back to this, since it's the most generic one - i.e. passing a wrong check type for .wait_for("message") will not show any errors. This is not the case with Event.message, which will correctly show a typing error if necessary.
But what about the other event methods?
No :)  Well, maybe.
I partially implemented this, but decided to remove it again; it may or may not affect the performance of pyright(/mypy), and the library typing is already complex enough.
Reason for choosing wait_for in particular is that it's usually used together with check=lambda x, y, z: ..., and you can't add type annotations to inline lambdas.
Somewhat depends on #1016 due to tests.
Checklist

[x] If code changes were made, then they have been tested

[ ] I have updated the documentation to reflect the changes
[x] I have formatted the code properly by running pdm lint
[x] I have type-checked the code by running pdm pyright

[ ] This PR fixes an issue
[x] This PR adds something new (e.g. new method or parameters)
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

@shiftinv would you please solve conflicts?

@shiftinv would you please solve conflicts?

done, rebased away the commits from the base PR and updated to master :+1:

I hate the concept of putting a ton of overloads in the main client.py file. It makes it way more bloated than it needs to be. Can't wait_for and its overloads be put in a mixin in a separate file?

I hate the concept of putting a ton of overloads in the main client.py file. It makes it way more bloated than it needs to be. Can't wait_for and its overloads be put in a mixin in a separate file?

It's not ideal either way, yeah. This still seems to be the least awful best solution, given we unfortunately can't just have a wait_for<T extends keyof typeof Event>(event: T, check: (...args: EventData[T]) => boolean) or something else that would avoid all these overloads in the first place :/
I'd like to avoid mixins as much as possible due to them being mostly incompatible with type checking, moving wait_for into a separate mixin/file would require additional # type: ignores.

I'd like to avoid mixins as much as possible due to them being mostly incompatible with type checking, moving wait_for into a separate mixin/file would require additional # type: ignores.

How come?

I hate the concept of putting a ton of overloads in the main client.py file. It makes it way more bloated than it needs to be. Can't wait_for and its overloads be put in a mixin in a separate file?

Instead of mixins maybe .pyi files could be used to achieve this? I don't know much about those though

I hate the concept of putting a ton of overloads in the main client.py file. It makes it way more bloated than it needs to be. Can't wait_for and its overloads be put in a mixin in a separate file?

Instead of mixins maybe .pyi files could be used to achieve this? I don't know much about those though

Not an expert on the topic either, but i heard that .pyi files are preferred over source code by type checkers, which means if we're going to supply a .pyi file we'll have to type hint the entirety of client.py there (since for TCs can't consume both stubs and source hints).
(mypy reference: https://mypy.readthedocs.io/en/stable/stubs.html#creating-a-stub - "If a directory contains both a .py and a .pyi file for the same module, the .pyi file takes precedence.")

which means if we're going to supply a .pyi file we'll have to type hint the entirety of client.py there

Yea, not something we want to do. Creating "partial" stubs like you mentioned would be cool, but that mechanism only applies at the file/module level and not the class/func level, which would be needed here.


deep objects in effect state

It seems like some unexpected behaviors can happen when using deep objects in an effect state. I have an array of nested objects in an effect definition. Once the effect is activated, it starts editing / rearranging though nested objects.
If multiple copies of the effect are created later, the state of the deep objects that are edited is copied over to the new effects. So it seems to me that the weird part is actually that the definition of the effect is being rewritten.
I ended up working around it for the moment by creating a deep copy of the original nested objected before working on them within the effect.
this.state.messageList = JSON.parse(JSON.stringify(this.state.messageListOriginal));
But thought I'd post about it to hear any thoughts from everyone.

I addressed this in #121.


Add support for AWS ECR Public in ECRAuthorizationToken generator

Is your feature request related to a problem? Please describe.
The ECRAuthorizationToken generator kind seems to only be able to get a private ECR authorization token. This token is only valid for private registries within the AWS account. If you would like to use the AWS ECR Public Gallery, there is a separate API and IAM permission associated with it that needs to be used.
The AWS CLI command is as follows for logging in to such a registry:
aws ecr-public get-login-password --region us-east-1
Note the use of ecr-public vs ecr, and the region I believe has to be us-east-1 (global).
Describe the solution you'd like
The ability to manage AWS ECR Public authorization tokens via external-secrets. This would greatly simplify credential management for OCI repositories which are hosted in the AWS public gallery. Right now its a mess of cronjobs to try and fudge it.
Describe alternatives you've considered
One solution might be the implementation of an ECRPublicAuthorizationToken kind. I don't really know enough about the implementation to know whether or not adding a scope: <private|public> to the spec might be possible while still using an ECRAuthorizationToken.
Additional context
The ecrpublic service appears to have dedicated support in the AWS SDK for it:
https://github.com/aws/aws-sdk-go/tree/main/service/ecrpublic
I would have a go at trying to implement something, but fear the result would be somewhat crude.

Hey, thanks for writing this up. I' prefer a scope field rattert than having a separate CRD/Generator for that.


Integration with Rasa Core

Add additional support to model domain and stories for rasa core from rasa-ui. This enhancement will create new tables in Postgres and all the data related to core models will be persisted to it.

@paschmann  Did you get a chance to look at the core-integration branch? Since rasa-core doesn't offer a train endpoint, I might build a workaround using a wrapper. For now, I will just provide a way to download these files from UI.
Any thoughts?

+1

Thanks for the integration @pradeepmvn !!! Awesome! PS. I just pushed another change I have been working on to simplify installation/node packages.

@pradeepmvn - I think we can close this issue for now and open any issues needed for the new integration


update modules / security

related: #416
Please update web3 module to ensure vulnerable dependency isn't installed.
Expected Behavior
Shouldn't install vulnerable dependency.
Possible Solution
update web3 to 1.0.0-beta.53
Steps to Reproduce (for bugs)
yarn add
warning ganache-core > web3 > web3-bzz > swarm-js ><EMAIL_ADDRESS>⚠  WARNING ⚠ tar.gz module has been deprecated and your application is vulnerable. Please use tar module instead: https://npmjs.com/tar

Closing as<EMAIL_ADDRESS>will be released as the canonical v1 (web3@latest) in the near future. We also plan to remove web3 as a dependency in Ganache soon.


allow passing variables into a query refetch

refetch(variables: any) is now possible on a query handle.
 Nice.


Add simple NTU model

In addition to the moving boundary NTU heat exchanger, a simple one is good for simplified studies.

@sebastianostlender as discussed, let's focus on the aspect of a single heat transfer coefficient for all phases instead of a single-boundary NTU and implement the changes in the new lmtd model in #4


Document QueryState

Objective

QueryState is lacking documentation.

Fixes #2090
Solution

Provide documentation that mirrors Query (as suggested in #2090) and modify as needed.

Hopefully this is to standard :)

@Frizi since you were the primary author for the iter_combinations stuff, I'd like your input for those documented functions :)

Hopefully the latest revisions are an improvement.


Can it be used only with Gyro data

I am writing a software for a model rocket attitude control.
Because of excessive acceleration I can not use an accelerometer and must rely almost exclusively to the Gyroscope data.
I looked at your 9DoF samples but it doesn't seem to be easy to convert to Gyro only.
Any pointers you could share.
Regards,

I think you can just update the quaternion directly instead of using the 9DOF lib.
_q *= Quaternion::from_euler_rotation_approx(gyrox, gyroy, gyroz);
_q.normalize();


List blocking processes with pid

This does a very small change to give both process and pid when the gvfs installer cannot run.
This will allow users to better see the git processes causing issues.
If we feel necessary we could go a step further and list the full command line, but that isn't on Process so we'd have to call lower level  windows/linux specific commands so I'm punting unless we fine we need it.
This
ERROR: Blocking processes are running.
Run `gvfs upgrade --confirm` again after quitting these processes - git

Becomes
ERROR: Blocking processes are running.
Run `gvfs upgrade --confirm` again after quitting these processes - git pid:9116, git pid:16396

Ideally we'd want to kill background processes when we unmount.
Speaking with @dscho that unfortunately is non-trival.
What we have in code today sends process.Kill (Terminate) to the running git process.
This isn't sufficient however, since git may have spawned other processes.
Here are some options:

Walk the processes tree calling Kill.  This is the easiest and also a cross-platform solution.  The issue here could be that we leave .lock files interfering with future runs causing user frustration.
This would be something to double check with @derrickstolee , but @dscho believes we'll almost certainly have this issue.

Create a Windows and Unix specific solution

On Windows

Ensure VFS / Git have the same bit version
Enumerate the process tree filtering by original pid as (possibly transitive) parent, then send all of them a remote thread with ExitProcess()

On Linux/Mac

kill(2)

Leverage Git For Windows for the Windows specific solution

On Windows

Expose an existing git feature for killing here

For now I think it's best to see how big of an issue this is before planning reaction work.
#704

As far as the .lock files are concerned: the biggest one to worry about is index.lock. We already clean up the commit-graph.lock and multi-pack-index.lock files before calling the processes that rewrite them. We may also strand some tmp_XXXX files when writing .idx files or pack files.
I would focus on the commands we run in the background, as we can expect a user to help their own processes before running upgrade. Other tools may run read-only processes in the background (like status or rev-list) that will not be problematic to halt.

The bigger questions\concerns in my mind are how users will react when they see a message like this:
Run gvfs upgrade --confirm again after quitting these processes - git pid:9116, git pid:16396

Agree.  But of course this is an improvement from:
Run gvfs upgrade --confirm again after quitting these processes - git

Do we want users to kill these processes (assuming they are BG processes), or should they wait?
If users do kill the BG git processes, can they restart again properly (given there might be stale lock files)?

When I previously discussed with @derrickstolee we said it was fine to force kill any background processes.  As part of the steps we clean up several lock files.  I want to double check ALL lock files from commands run from background maintenance jobs are cleaned up by maintenance steps.  If so the user could kill, but we should definitely just kill ourselves on unmount.

Will asking users to "quit" processes they did not start themselves cause confusion?

I bet it would :-)

I think listing the process PID could be helpful to let the user know exactly which process is blocking the upgrade, and stands by itself.
I agree with @derrickstolee - I think the main issue we should focus on is with background git processes that VFSForGit kicks off. As we initiated those processes, I would expect VFSForGit to coordinate running upgrade with those processes and ensure that VFSForGit itself is not blocking upgrade. I would be less surprised if VFSForGit refused to upgrade because an outside git process is running.
Given this scope of focusing on process VFSForGit initiates, does that make the problem easier (can we possibly terminate our process that we know are safe to terminate, and clean up left behind files)?
Is there anything we can log in the unmount code path to indicate that the repository was unmounted, but it failed to terminate background git processes? Are there issues with the background git processes still running after the repository is unmounted?

@jamill @wilbaker @derrickstolee we're all on the same page that scope here is background jobs.  My understanding is that all of the jobs we run in the background are safe to stop and that was what I was hoping to double check.  Speaking with @dscho he's concerned about option 1 (probably because folks have been burned several times by terminating jobs they shouldn't) so it's worth getting the details out there and verifying.

Having a few questions...

After a forced kill, for vfsforgit processes to recover later (after the upgrade), would the user have to take further actions - like run gvfs repair or in the worst case, email support. Can the repair & recovery happen without user action?
If the recovery cannot happen automatically - should VFS4Git warn the user before it decides to kill its child processes. May be provide a --force switch in upgrade?
About the messaging (Run gvfs upgrade --confirm again after quitting these processes - git pid:9116, git pid:16396).
3.1 Agree that we don’t want the user to quit processes launched by VFS4Git. Should probably rephrase the text to something like “Could not unmount all VFSForGit repositories. Manually unmount and re-run upgrade.” And don’t show VFS4Git children in the list (if needed, we can always capture detailed process listing in the upgrade logs.)
3.2 Also I’m not sure I understand the problem we are solving by showing pid numbers in the message. (If there is only one git instance running, Windows Task Manager would show just the one. If there are many git instances, the user would have to order the process listing by name and terminate all. In both cases pid is not being useful.)

@alameenshah

After a forced kill, for vfsforgit processes to recover later (after the upgrade), would the user have to take further actions - like run gvfs repair or in the worst case, email support. Can the repair & recovery happen without user action?
No need to run repair, these processes can be safely killed

If the recovery cannot happen automatically - should VFS4Git warn the user before it decides to kill its child processes. May be even provide a --force switch in upgrade?
Since they can be safely killed we're ok here.

About the messaging (Run gvfs upgrade --confirm again after quitting these processes - git pid:9116, git pid:16396).
3.1 Agree that we don’t want the user to quit processes launched by VFS4Git. When VFS4Git children are the ones blocking upgrade, may be rephrase the text to something like “Could not unmount all
I've upgraded the latest PR to kill the background jobs so the problem won't exist.

VFSForGit repositories. Manually unmount and re-run upgrade”.

3.2 Also I’m not sure I understand the problem we are solving by showing pid numbers in the message. (If there is only one git instance running, Windows Task Manager would show just the one. If there are many git instances, the user would have to order the process listing by name and terminate all. In both cases pid is not being useful.)
We want to let the user know what exactly is blockly.  pid becomes important in case the processes disappears or in general to know how many instances are blocking.


consolidate to one CDK crate

Having the two crates adds complexity without benefit since features can be used instead.
Credit: Thought about this after seeing Yuki's fork

@yukibtc  @lorenzolfm @davidcaseria thoughts?

I agree with this approach. Seems much simpler :)


error creating statement 'insert or replace into realisations" broke nix installation on update

Upgraded things on Friday, ran into this issue, and filed https://github.com/NixOS/nix/issues/4705.
A recent comment said it was related to ca-derivations, I went and ripgrep'd devos and saw it was enabled now.
Thought it would be good to create an issue here in case others run into it.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Yeah we should probably just disable ca-derivations for now, I think I got a little too excited when I enabled it :laughing:

Locally for now I'm just removing ca-derivations and guessing that won't break anything, but I'm not sure. We'll see since this machine can't be updated right now anyway :stuck_out_tongue_winking_eye:

Huh, it didn't like that lol:
error (ignored): error: cannot unlink '/nix/store/trash/mw3sn78f15rkzs17aa17kqqxnq71jhl3-emacs-gcc-20210407.0.drv.chroot/nix/store/3l6sdyzgn1hylmlgmrvnj1kcnz4plra2-emacs-gcc-20210407.0/share/emacs/28.0.50/lisp': Directory not empty
error: builder for '/nix/store/mw3sn78f15rkzs17aa17kqqxnq71jhl3-emacs-gcc-20210407.0.drv' failed with exit code 2;
       last 10 log lines:
       > gzip: ./windmove.el: No such file or directory
       > gzip: ./font-lock.el: No such file or directory
       > gzip: ./frame.el: No such file or directory
       > gzip: ./pixel-scroll.el: No such file or directory
       > gzip: ./proced.el: No such file or directory
       > gzip: ../etc/publicsuffix.txt: No such file or directory
       > make: *** [Makefile:581: install-arch-indep] Error 1
       > /nix/store/bmnhfb10m2s3whl6478dmqhcrkjwk77y-stdenv-linux/setup: line 95: /dev/null: No such file or directory
       > /nix/store/bmnhfb10m2s3whl6478dmqhcrkjwk77y-stdenv-linux/setup: line 76: /dev/null: No such file or directory
       > /nix/store/bmnhfb10m2s3whl6478dmqhcrkjwk77y-stdenv-linux/setup: line 78: /dev/null: No such file or directory
       For full logs, run 'nix log /nix/store/mw3sn78f15rkzs17aa17kqqxnq71jhl3-emacs-gcc-20210407.0.drv'.

Well... I don't know how to fix this:
❯ sudo nix-collect-garbage -d
[sudo] password for cody:
removing old generations of profile /nix/var/nix/profiles/system
removing old generations of profile /nix/var/nix/profiles/per-user/root/channels
removing old generations of profile /nix/var/nix/profiles/per-user/nixos/profile
removing old generations of profile /nix/var/nix/profiles/per-user/nixos/home-manager
removing old generations of profile /nix/var/nix/profiles/per-user/cody/channels
removing old generations of profile /nix/var/nix/profiles/per-user/cody/profile
removing old generations of profile /nix/var/nix/profiles/per-user/cody/home-manager
removing old generations of profile /nix/var/nix/profiles/default
removing old generations of profile /nix/var/nix/profiles/per-container/flake-test/system
finding garbage collector roots...
deleting garbage...
deleting '/nix/store/trash'
deleting unused links...
note: currently hard linking saves 20665.42 MiB
0 store paths deleted, 0.00 MiB freed

devos2  cody 「↪ ✘ 𝚫 📁 」  ⎔ took 18s
❯ sudo nix shell ".#nixosConfigurations.razer.config.system.build.toplevel" -c switch-to-configuration switch
warning: Git tree '/home/cody/devos2' is dirty
error (ignored): error: cannot unlink '/nix/store/trash/mw3sn78f15rkzs17aa17kqqxnq71jhl3-emacs-gcc-20210407.0.drv.chroot/nix/store/3l6sdyzgn1hylmlgmrvnj1kcnz4plra2-emacs-gcc-20210407.0/share/emacs/28.0.50/lisp/obsolete': Directory not empty
error: builder for '/nix/store/mw3sn78f15rkzs17aa17kqqxnq71jhl3-emacs-gcc-20210407.0.drv' failed with exit code 2;
       last 10 log lines:
       > gzip: ./rot13.el: No such file or directory
       > gzip: ./windmove.el: No such file or directory
       > gzip: ./font-lock.el: No such file or directory
       > gzip: ./frame.el: No such file or directory
       > gzip: ./pixel-scroll.el: No such file or directory
       > gzip: ../etc/publicsuffix.txt: No such file or directory
       > make: *** [Makefile:581: install-arch-indep] Error 1
       > /nix/store/bmnhfb10m2s3whl6478dmqhcrkjwk77y-stdenv-linux/setup: line 95: /dev/null: No such file or directory
       > /nix/store/bmnhfb10m2s3whl6478dmqhcrkjwk77y-stdenv-linux/setup: line 76: /dev/null: No such file or directory
       > /nix/store/bmnhfb10m2s3whl6478dmqhcrkjwk77y-stdenv-linux/setup: line 78: /dev/null: No such file or directory
       For full logs, run 'nix log /nix/store/mw3sn78f15rkzs17aa17kqqxnq71jhl3-emacs-gcc-20210407.0.drv'.
error: 1 dependencies of derivation '/nix/store/j1l9nyaq0psx7alr9wb7vd3qpwil50h9-emacs-gcc-with-packages-20210407.0.drv' failed to build
error: 1 dependencies of derivation '/nix/store/73fs7j47y9mdyrk0v85rn9fcnw6wbn3m-system-path.drv' failed to build
error: 1 dependencies of derivation '/nix/store/amprw6wk3wbfvyvgiqh1jkwnvf69m8az-nixos-system-razer-21.05.20210410.a73020b.drv' failed to build

devos2  cody 「↪ ✘ 𝚫 📁 」  ⎔ took 25m47s

But I guess emacs-overlay updates every day so I'll just update my flake and... that fixes it? :mag_right:

Well... I removed the ca-references and ca-derivations stuff and my laptop is working.
I then pulled updates to my PC and... now it's broken :laughing:

My bad, I never should've enabled it in the first place. I'll remember to update this in the next release, which is hopefully coming on the weekend.

I just run into this for the second time. We really should drop ca-derivations by default.

This is now completely fixed in core. I would say to leave this open for a bit longer. In the case that others run into it, they can read this issue to learn how to fix it.

Its been a couple weeks now since this has been fixed. Going to close, thanks for the report.


Chrome: Uncaught TypeError: navigator.mediaDevices.getUserMedia is not a function

Getting this in our app but also on all your demos. See screencap below. Chrome on Mac OS Version 47.0.2526.73 (64-bit)
http://d.pr/i/14Zk2

Yikes, thank you for letting me know.  They did something weird in Chrome 47.  I'll get this fixed tonight.

Thank you for very fast reply. We are using webcam.js in our intranet and it is very critical for us.

Thanks for you effort jhuckaby. Chrome retired support for "getUserMedia" :(.

Seems like navigator.webkitGetUserMedia() still works fine.

Hey guys,
So ironically, I already fixed this in the latest commit, but I failed to update my demo site.  That's all fixed now, but there is a new problem.  Chrome 47 requires that the site be HTTPS in order for user media / webcam to work!  This is a brand new requirement that I didn't see coming...

Chromium Docs on the topic:
https://www.chromium.org/Home/chromium-security/prefer-secure-origins-for-powerful-new-features
This means that all my hosted demos will fail, because I don't have HTTPS on my server yet.  I'll have to buy a SSL cert and update the docs and all my links.  But if you grab the latest commit from GitHub, the library should work fine in Chrome 47, as long as you are using it on <IP_ADDRESS>, or over HTTPS.
I'll tag v1.0.6 with this commit too, as soon as I get my HTTPS situation resolved.
Thanks for reporting this!

I tested the library with https and it works fine. I have reviewed in different sites and this problem is in all websites. Sincerely, I don't understand this decision from Google.

We have to buy now a certificate for local(!) Server.

You don't need HTTPS for localhost.  Check the Chromium docs for the whitelist:
'''
(https, *, *)
(wss, *, )
(, localhost, )
(, *.localhost, )
(, 127/8, )
(, ::1/128, *)
(file, *, —)
(chrome-extension, *, —)
'''

for local Network, not for same PC.

This free SSL cert service might be worth a shot. http://www.theregister.co.uk/2015/12/03/letsencrypt_public_beta/?mt=1449248133223
Actually... looks like you got https working on your site.
Another solution would be to push demos to gh-pages branch, and have them hosted on github (which supports https).

Yup, got it working late last night.  I just have to update all the links to the demos, put a huge disclaimer in the docs, and push out a v1.0.6 tag.

Tagged v1.0.6 with support for navigator.mediaDevices in Firefox and Chrome, and updated all the demo links to be HTTPS.  Updated docs with very loud disclaimer about Chrome.
https://github.com/jhuckaby/webcamjs/releases/tag/v1.0.6

Thanks @jhuckaby! 1.0.6 works with SSL on the server and just works locally as you said. Our production server already has SSL but if anyone is looking for free ssl for your development or staging servers you should try https://letsencrypt.org/.


Setting generateId option not working anymore

I pass the following to fortune:
fortune.create({
    adapter: {
        type: MongodbAdapter,
        options: {
            url: 'some_url',
            generateId: function(){
                return crypto.randomBytes(16).toString('hex');
            }
        }
    }
})

But the function is not correctly overwritten as it still generates the randomBytes when I create a document instead of using my function.
Fortune beta 38
Fortune-mongodb 1.0.1 and 1.0.2

Can you verify if this is a problem in Fortune or in the MongoDB adapter?


rs3->dis/svg conversion can't handle Szeryng example text

feng-hirst-2014-result.rs3.txt
curl -XPOST localhost:9150/convert/rs3/dis -F<EMAIL_ADDRESS>{"error":"<class 'discoursegraphs.readwrite.rst.rs3.rs3tree.RSTTree'> can't handle input file 'feng-hirst-2014-result.rs3.txt'.

The "Le Duc" was the instrument on which he performed and recorded mostly, while the latter ("King David" Strad) was donated to the State of Israel.

feng-hirst-2014 output:
ParseTree('Elaboration[N][S]', [ParseTree('Temporal[N][S]', ['Szeryng subsequently focused on teaching', 'before resuming his concert career in 1954 .']), ParseTree('Elaboration[N][S]', ['The " Le Duc " was the instrument', ParseTree('Temporal[N][N]', ['on which he performed and recorded mostly ,', ParseTree('same-unit[N][N]', [ParseTree('same-unit[N][N]', [ParseTree('Elaboration[N][S]', ['while the latter', '( " King David "']), 'Strad )']), 'was donated to the State of Israel .'])])])])

feng-hirst-2014-result-minimal.rs3.txt

If we convert the original feng-hirst-2014 parser output to a tree,
$ curl -X POST -F<EMAIL_ADDRESS>http://localhost:5000/convert/hilda/tree.prettyprint
                                                               Elaboration
                     _______________________________________________|________________
                    |                                                                S
                    |                                                                |
                    |                                                           Elaboration
                    |                              __________________________________|____________________________
                    |                             |                                                               S
                    |                             |                                                               |
                    |                             |                                                            Temporal
                    |                             |                  _____________________________________________|_____________
                    |                             |                 |                                                           N
                    |                             |                 |                                                           |
                    |                             |                 |                                                       same-unit
                    |                             |                 |                                              _____________|____________________
                    |                             |                 |                                             N                                  |
                    |                             |                 |                                             |                                  |
                    |                             |                 |                                         same-unit                              |
                    |                             |                 |                                _____________|______________________            |
                    N                             |                 |                               N                                    |           |
                    |                             |                 |                               |                                    |           |
                 Temporal                         |                 |                          Elaboration                               |           |
        ____________|____________                 |                 |                 ______________|_____________                       |           |
       N                         S                N                 N                N                            S                      N           N
       |                         |                |                 |                |                            |                      |           |
Szeryng subseque          before resuming  The " Le Duc "      on which he    while the latter             ( " King David "           Strad ) was donated to
ntly focused on             his concert    was the instrume   performed and                                                                    the State of
    teaching              career in 1954 .        nt        recorded mostly ,                                                                     Israel .

we see that it has two Temporal relations with different nuclearity:
The first one is (N: ... focused on teaching, S: before resuming ...).
The second one is (N: on which he performed ... mostly, N: while the latter ... was donated).
The while should not be interpreted in a temporal sense, but that's probably not the issue here.

The rs3 files has some odd things, like relation names in different casing/spelling in the <relations> section,
while in the  section, we only find Temporal (and not temporal)
    <relations>
      <rel name="Manner-Means" type="rst"/>
      <rel name="Mannermeans" type="rst"/>
      ...
      <rel name="Same-Unit" type="multinuc"/>
      <rel name="Same-unit" type="multinuc"/>
      <rel name="Temporal" type="multinuc"/>
      <rel name="same-unit" type="multinuc"/>
      <rel name="same_unit" type="multinuc"/>
      <rel name="temporal" type="rst"/>
    </relations>
...
  <body>
    <segment id="5" parent="3" relname="Temporal">Szeryng subsequently focused on teaching</segment>
    <segment id="7" parent="3" relname="Temporal">before resuming his concert career in 1954 .</segment>
    <segment id="15" parent="13" relname="Temporal">on which he performed and recorded mostly ,</segment>
    <group id="17" type="multinuc" parent="13" relname="Temporal"/>
  </body>


Swiper Vue: Uncaught TypeError when using SwiperSlide as a wrapper

Check that this is really a bug

[X] I confirm

Reproduction link
https://codesandbox.io/s/swiper-navigation-vue-forked-t0r5bc?file=/src/App.vue
Bug description
Check the browser console. You get the warning and error:

Unhandled error during execution of mounted hook

Uncaught` TypeError: Cannot read properties of undefined (reading 'value')
at VM54 swiper_vue.js:1229:22

This only happens, if I use my own Slide as a wrapper to add some extra content.
Expected Behavior
No error and warning. You should use the <swiper-slide> in a wrapper as well.
Actual Behavior
Swiper is not initialized.
Swiper version
^8.0.0
Platform/Target and Browser Versions
mac
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

+1

have the same issue :/

Same issue with Vue 2.7.x :(

Same issue using Vue 2.7.x

I gave up with this plugin......

Same issue here with 2.7.x vue version

Same issue using Vue 2.6.x

Also having issues Vue 3 (Nuxt 3 rc.9)
Error comes from swiperRef
    onMounted(() => {
      if (!swiperRef.value) return;
      swiperRef.value.on('_slideClass', updateClasses);
      eventAttached = true;
    });
```

Anyone figured a solution for this one ?? I'm stuck and cannot use a wrapper component

Seems like adding the below will work.

Any thoughts on it ? This is inside the getChildren function in the get-children.js file.
Basically on top of check if the node is SwiperSlider or an AsyncComp check if it's a wrapper that has some SwiperSlider.

Issue is closed because of outdated/irrelevant/not actual/needed more information/inactivity.
If this issue is still actual and reproducible for latest version of Swiper, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


bug: Claim button looks disabled, fee calculation is incorrect

What is happening?

Section description
Provide as much context as you can. Give as much context as you can to make it easier for the developers to figure what is happening.

Two things are happening.

The fee amount does not have the correct decimals/denom. In the example below it shows as 0.2351, but it should be 0.002351 according to the transaction.
User was able to claim and the transaction went through, even though the button looks disabled

Loom and screenshot from testing
Loom: https://www.loom.com/share/f334f5aa2e7e490fbb72cae9a0a0c86d?highlightComment=11261124&t=113

Transaction details

Link to explorer
Calculation of balance

Closing for now.