exim: 4.97.1 -> 4.98

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/exim/versions.
meta.description for exim is: Mail transfer agent (MTA)
meta.homepage for exim is: https://exim.org/
meta.changelog for exim is: https://github.com/Exim/exim/blob/exim-4.98/doc/doc-txt/ChangeLog
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.98 with grep in /nix/store/4cwj9ql32sdqhikkfwzgfpwbxq9n381i-exim-4.98
found 4.98 in filename of file in /nix/store/4cwj9ql32sdqhikkfwzgfpwbxq9n381i-exim-4.98

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

exim

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/4cwj9ql32sdqhikkfwzgfpwbxq9n381i-exim-4.98 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A exim https://github.com/r-ryantm/nixpkgs/archive/adcc0e3c8fe9592daf264223fe61546bf1377ab5.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/adcc0e3c8fe9592daf264223fe61546bf1377ab5#exim

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/4cwj9ql32sdqhikkfwzgfpwbxq9n381i-exim-4.98
ls -la /nix/store/4cwj9ql32sdqhikkfwzgfpwbxq9n381i-exim-4.98/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

exim

Maintainer pings
cc @4z3 @dasJ @Conni2461 for testing.

Includes the fix for CVE-2024-39929 (https://bugs.exim.org/show_bug.cgi?id=3099)


:sparkles: Implement Grain filter effect

Description
Implement the Grain filter effect.
The shader is referenced from CBVisualEffectBuiltins - Grain2 which was originally implemented in C style and has now been converted into a Metal shader.

@Justin-Ruan, @Vivienne0206
One question: Do we need to add the isSRGB?
The example project doesn't seem to check the color space.


fix(tier4_localization_rviz_plugin): fix unusedFunction

Description
This is a fix based on cppcheck unusedFunction warnings.
Preparation for future CI changes.
I have tested and confirmed this, but would like to know if it is being used.
common/tier4_localization_rviz_plugin/src/pose_history_footprint/display.cpp:98:0: style: The function 'update_visualization' is never used. [unusedFunction]
void PoseHistoryFootprint::update_visualization()
^

Related links
Parent Issue:

Link

How was this PR tested?

Confirm that the planning simulation works.
Confirm that print debugging is not used.
Confirm that the build passes.

Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

@TakaHoribe Please review this PR :bow:

@TakaHoribe Hi. Please!

@TakaHoribe
Please confirm the PR. Thank you very much.


feat: Adding support for databoost

Enable databoost feature for all customers and removing the beta api annotation.

Can we re-enable integration tests through this PR and merge it once those pass (most likely on 26 Jun)?

Can we re-enable integration tests through this PR and merge it once those pass (most likely on 26 Jun)?

Yes we can and lets wait till 26  as feature is already available in beta mode.


Add integration tests for JMX.

Description of changes
Adds integration tests to check if enabling annotations adds expected JMX environmental variables for app_signals and without.
Testing
make test

See passing tests in Checks.
Addressed @Paramadon's comments from https://github.com/aws/amazon-cloudwatch-agent-operator/pull/245/.

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

The. description lists the output of the unit tests, could you paste a successful run of these new tests ?


[jvm-packages] bring back camel case variants of parameters

This PR brings back the camel case variants of the parameters and keep compatible with the existing user experience.
For example,
    val xgbParams: Map[String, Any] = Map(
      "max_depth" -> 5,
      "featuresCol" -> "abc",
      "num_workers" -> 2,
      "numRound" -> 11
    )
    val estimator = new XGBoostClassifier(xgbParams)

The jvm specific parameters are allowed in the map when constructing the estimators. And we support both the lower underscore naming and camel case naming.

Yes, Please merge it.


Average Stats always empty

Im trying to get the average stats per 10 minutes, aka the average_stats field in the output, but whatever battletag im using, this is the only field thats always empty (its empty in every region too, not only the one that the player plays in)

I have been able to get non-empty average_stats fields. See the following:
import asyncio

from overwatch_api.core import AsyncOWAPI
from overwatch_api.constants import *

async def testing(loop):
    # Instantiating the api
    client = AsyncOWAPI()

    data = {}

    print('Testing......[get_hero_stats]')
    print((await client.get_hero_stats("Danielfrogs#2552", platform=PC))
          ['eu']['stats']['quickplay']['symmetra']['average_stats'])

if __name__ == "__main__":
    loop = asyncio.get_event_loop()
    loop.run_until_complete(testing(loop))

When I run it, I get
Testing......[get_hero_stats]
{'teleporter_uptime_average': 0.16}

I don't know why almost all heroes have empty average_stats fields (Only Zarya and Symmetra have any data for Danielfrogs#2552), but that field is at least not always empty.

this still isnt really getting the per 10 minutes stat, just some generals and that only for 2 heros

Hmm 🤔, maybe OWAPI isn't responding with updated data.

25 feb. 2018 kl. 23:39 skrev bschlegel-tgm<EMAIL_ADDRESS>this still isnt really getting the per 10 minutes stat, just some generals and that only for 2 heros
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Any idea what i could do to get those stats tho?

No, I unfortunately don't know how or where to get those stats 😕.

26 feb. 2018 kl. 10:45 skrev bschlegel-tgm<EMAIL_ADDRESS>Any idea what i could do to get those stats tho?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Any way i could reach out to the developer so he could fix it?

Once you find even more examples of battletags lacking average stats, you could just shoot the developer(s?) of OWAPI an email about it. But make sure to really confirm this problem, I don’t want to bother the dev(s) without a reason to 👍.

26 feb. 2018 kl. 21:19 skrev bschlegel-tgm<EMAIL_ADDRESS>Any way i could reach out to the developer so he could fix it?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Make MCMC builder function

To make it easier on users, we should implement a builder function for the classes in mcmc.py, similar to what we have done for velocity models, intensity maps, etc.

Closed by #13


Top navigation bar's blur effect doesn't work on Safari

Hi,
Noticed something with the header blur effect, as you can guess from the title. It works on Chromium-based browsers but on Safari, it just shows a dark overlay. Figured it's worth noting.

So embarrassed I didn't see this issue until now. Sorry for the late reply.
Actually, we fixed a bug in Kobweb a few weeks back to address this issue. I just rebuilt the site and deployed a new version, so the blur should be working on Safari now!


Update for the GA of Top Actions

Category

[x] Content fix
[ ] New article

What's in this Pull Request?
Updating the Top Actions to be GA'd as part of the 1.17 release - and updating tutorial to match the GA features.

Thx @AJIXuMuK - we will delay publishing of this until the GA version. Thanks for the required edits.


Segfault when executing a cached lexsort implementation

Reporting a bug

[x] I am using the latest released version of Numba (most recent is visible in
the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).

$ pip freeze
llvmlite==0.32.1
numba==0.49.1
numpy==1.18.4

[x] I have included below a minimal working reproducer (if you are unsure how
to write one see http://matthewrocklin.com/blog/work/2018/02/28/minimal-bug-reports).

numba.literal_unroll seems like it would make a lexsort implementation easy. I had a go and it mostly worked! So thanks for numba.literal_unroll!
However, when I try to put cache=True into the generated_jit decorator, the lexsort function is successful on the initial run, but segfaults on subsequent runs:
from numba import njit, generated_jit, literal_unroll, gdb
import numba.core.types as types
import numba
import numpy as np

@njit
def cmp_fn(l, r, *arrays):
    for a in literal_unroll(arrays):
        if a[l] < a[r]:
            return -1  # less than
        elif a[l] > a[r]:
            return 1   # greater than

    return 0  # equal

@njit
def quicksort(index, L, R, *arrays):
    l, r = L, R
    pivot = index[(l + r) // 2]

    while True:
        while l < R and cmp_fn(index[l], pivot, *arrays) == -1:
            l += 1
        while r >= L and cmp_fn(pivot, index[r], *arrays) == -1:
            r -= 1

        if l >= r:
            break

        index[l], index[r] = index[r], index[l]
        l += 1
        r -= 1

        if L < r:
            quicksort(index, L, r, *arrays)

        if l < R:
            quicksort(index, l, R, *arrays)

@njit(nogil=True, cache=True, debug=True)
def lexsort(arrays):
    print("starting lexsort")

    if len(arrays) == 0:
        return np.empty((), dtype=np.intp)

    if len(arrays) == 1:
        return np.argsort(arrays[0])

    for a in literal_unroll(arrays[1:]):
        if a.shape != arrays[0].shape:
            raise ValueError("lexsort array shapes don't match")

    n = arrays[0].shape[0]
    index = np.arange(n)

    quicksort(index, 0, n - 1, *arrays)

    print("ending lexsort")
    return index

a = np.array([2, 1, 4, 3, 0], dtype=np.int32)
b = np.array([1, 2, 3, 4, 5], dtype=np.float64)

print("before lexsort")
print(lexsort((a, b)))
print("after lexsort")

I considered out of bounds access in my quicksort implementation, but that's never reached. I tried running this through gdb, and "starting quicksort" isn't printed.
$ gdb -ex r --args python test_literal_unroll.py
GNU gdb (Ubuntu 8.1-0ubuntu3.2) <IP_ADDRESS>80409-git
Copyright (C) 2018 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "x86_64-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
<http://www.gnu.org/software/gdb/bugs/>.
Find the GDB manual and other documentation resources online at:
<http://www.gnu.org/software/gdb/documentation/>.
For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from python...Reading symbols from /usr/lib/debug/.build-id/5f/4de7b7974f514b4d5baf54bc956904a450c144.debug...done.
done.
Starting program: /home/sperkins/venv/numba/bin/python test_literal_unroll.py
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
[New Thread 0x7fffee395700 (LWP 20316)]
[New Thread 0x7fffebb94700 (LWP 20317)]
[New Thread 0x7fffe9393700 (LWP 20318)]
[New Thread 0x7fffe8b92700 (LWP 20319)]
[New Thread 0x7fffe4391700 (LWP 20320)]
[New Thread 0x7fffe1b90700 (LWP 20321)]
[New Thread 0x7fffe138f700 (LWP 20322)]
[Thread 0x7fffe138f700 (LWP 20322) exited]
[Thread 0x7fffe1b90700 (LWP 20321) exited]
[Thread 0x7fffe4391700 (LWP 20320) exited]
[Thread 0x7fffe8b92700 (LWP 20319) exited]
[Thread 0x7fffe9393700 (LWP 20318) exited]
[Thread 0x7fffebb94700 (LWP 20317) exited]
[Thread 0x7fffee395700 (LWP 20316) exited]
before lexsort
starting lexsort
ending lexsort
[4 1 0 3 2]
after lexsort
[Inferior 1 (process 20309) exited normally]
(gdb) r
Starting program: /home/sperkins/venv/numba/bin/python test_literal_unroll.py
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
[New Thread 0x7fffee395700 (LWP 20327)]
[New Thread 0x7fffebb94700 (LWP 20328)]
[New Thread 0x7fffeb393700 (LWP 20329)]
[New Thread 0x7fffe8b92700 (LWP 20330)]
[New Thread 0x7fffe4391700 (LWP 20332)]
[New Thread 0x7fffe1b90700 (LWP 20333)]
[New Thread 0x7fffdf38f700 (LWP 20334)]
[Thread 0x7fffdf38f700 (LWP 20334) exited]
[Thread 0x7fffe1b90700 (LWP 20333) exited]
[Thread 0x7fffe4391700 (LWP 20332) exited]
[Thread 0x7fffe8b92700 (LWP 20330) exited]
[Thread 0x7fffeb393700 (LWP 20329) exited]
[Thread 0x7fffebb94700 (LWP 20328) exited]
[Thread 0x7fffee395700 (LWP 20327) exited]
before lexsort

Thread 1 "python" received signal SIGSEGV, Segmentation fault.
0x0000000000000000 in ?? ()
(gdb) bt
#0  0x0000000000000000 in ?? ()
#1  0x00007ffff7fe8715 in cpython::__main__::lexsort$241(Tuple<Array<int, 1, C, mutable, aligned>, Array<double, 1, C, mutable, aligned> >) ()
#2  0x00007fffeb98d5d9 in call_cfunc (self=self@entry=0x7fffda0f0660, cfunc=cfunc@entry=0x7fffd9d80ee8, args=args@entry=0x7ffff7e66f28,
    kws=kws@entry=0x0, locals=locals@entry=0x0) at numba/_dispatcher.c:353
#3  0x00007fffeb98d910 in compile_and_invoke (self=self@entry=0x7fffda0f0660, args=args@entry=0x7ffff7e66f28, kws=kws@entry=0x0, locals=locals@entry=0x0)
    at numba/_dispatcher.c:376
#4  0x00007fffeb98dace in Dispatcher_call (self=0x7fffda0f0660, args=0x7ffff7e66f28, kws=<optimised out>) at numba/_dispatcher.c:591
#5  0x00000000005a9cbc in _PyObject_FastCallDict (kwargs=<optimised out>, nargs=1, args=0xae0410, func=0x7fffda0f0660) at ../Objects/tupleobject.c:131
#6  _PyObject_FastCallKeywords () at ../Objects/abstract.c:2496
#7  0x000000000050a5c3 in call_function.lto_priv () at ../Python/ceval.c:4875
#8  0x000000000050bfb4 in _PyEval_EvalFrameDefault () at ../Python/ceval.c:3335
#9  0x0000000000507d64 in PyEval_EvalFrameEx (throwflag=0, f=0xae0288) at ../Python/ceval.c:754
#10 _PyEval_EvalCodeWithName.lto_priv.1820 () at ../Python/ceval.c:4166
#11 0x000000000050ae13 in PyEval_EvalCodeEx (closure=0x0, kwdefs=0x0, defcount=0, defs=0x0, kwcount=0, kws=0x0, argcount=0, args=0x0,
    locals=<optimised out>, globals=<optimised out>, _co=<optimised out>) at ../Python/ceval.c:4187
#12 PyEval_EvalCode (co=<optimised out>, globals=<optimised out>, locals=<optimised out>) at ../Python/ceval.c:731
#13 0x0000000000634c82 in run_mod () at ../Python/pythonrun.c:1025
#14 0x0000000000634d37 in PyRun_FileExFlags () at ../Python/pythonrun.c:978
#15 0x00000000006384ef in PyRun_SimpleFileExFlags () at ../Python/pythonrun.c:419
#16 0x00000000006386c5 in PyRun_AnyFileExFlags () at ../Python/pythonrun.c:81
#17 0x0000000000639091 in run_file (p_cf=0x7fffffffd70c, filename=<optimised out>, fp=<optimised out>) at ../Modules/main.c:340
---Type <return> to continue, or q <return> to quit---```

I think this has the potential to be a numba lexsort implementation. I note that there are quicksort factories in numba.misc.quicksort.py but they'd need updating to support the lexical tuple comparison.

xref https://github.com/numba/numba/issues/3689

Thanks for the report, I can reproduce. Seems like it segfaults on calling .compile() in the dispatcher once the cache has been loaded.

I'm trying to understand whether this is a problem within the reproducer or a problem with the compiler.
Is the fact that there's a numba.literal_unroll in the reproducer a contributing factor?

I think this has the potential to be a numba lexsort implementation. I note that there are quicksort factories in numba.misc.quicksort.py but they'd need updating to support the lexical tuple comparison.
Seems works fine in Python 3.7.9 (default, Aug 31 2020, 17:10:11) [MSC v.1916 64 bit (AMD64)] :: Anaconda, Inc. on win32
numba: 0.51.2
numpy:  1.19.1

This should be re-written, otherwise easily results in stackoverflow...

Sorry about reopening an old conversation, but is this something on the roadmap? Can I help in some way since I would be interested is using lexsort.

+1 for lexsort


Update repository

Hi i like this platform platform very nice, i have struggle in update repository. I try to update values from one repository to another repository how can i do (I try to update bulk operation particular record with some condition using store procedure).

Put your stored procedure function call in your xyzrepository and then from your dialog call that.
There is an example in the forum look at this issue and follow the information posted - or search forum for Stored Procedure.
2593 Stored Procedure


Remove hard-coded test directory #42

Description and Context

The hard-coded TEST_DIRECTORY has been removed from the GitHub workflow. It is replaced with $(pwd). The rum command is also adjusted accordingly.
Related Issues and Pull Requests

Closes #42
Blocks
Is blocked by
Follows
Precedes
Related to
Part of
Composed of

How Has This Been Tested?

The workflow runs without issues in this branch.
Checklist

[x] My commit messages mention the appropriate GitHub issue numbers and are longer than one line where appropriate.
[x] I have added/updated documentation where necessary.

Additional Information

Interested Parties / Possible Reviewers

@mayrmt @eulovi

In #42 Nora had mentioned that trying $(pwd) did not work. I am not sure what she meant, but it works fine with the way I did it now. If this is not the way I am supposed to do it, then the other option would be to do the following
run: echo "TEST_DIRECTORY=$(pwd)" >> $GITHUB_ENV

I think the current option with github.workspace is even better since it takes directly the working directory used by the GitHub runner as pointed out in the documentation. Therefore, I think this can be merged.


Stupid question time

Hi Scott,
I have installed hsmm-pi on three P-pi2's. They can  all see each other on the network list.
How do I manage the nodes via the network. Do I have to connect to each on via the Ethernet port and then connect to the wen interface or can I connect to one node then remotely administer the others?
The second question is how the services work.
What I am trying to set up is a network that can ne set up in an emergency response centre. There will be some nodes configured as specialist providers such as an internet gateway and the router will provide the firewall, another a viop server. Some will be simple repeaters to filling the dark spaces in a building. The others will be the interface between the lan and the mesh.
I want to be able that a pc on say node 2 can ping and see the c$ chare on a pc on node 3 and so on. The plan was that the node2 would have the ip <IP_ADDRESS> and would give out a dhcp scope 11-19. Node 3 and ip of <IP_ADDRESS> and have a dhcp soap of 21 - 29 do you get the idea. When I do this I can’t ping.
 On the surface this build id simple to build onto the R-pi. The headwear is the right cost. you’re not tied to specific hardware. in the UK Linksys WRG54's are hard to come by and the ubiquity kit even harder.
Help, Please? There is a group of UK Hams wanting to use your software and were all as stuck as each other.

Do I have to connect to each on via the Ethernet port and then connect to the wen interface or can I connect to one node then remotely administer the others?

You can connect to one and remotely administer the others via the web interface.

I want to be able that a pc on say node 2 can ping and see the c$ chare on a pc on node 3 and so on.

At the moment, HSMM-Pi operates in NAT mode, which means that by default any machine connected to the Ethernet interface of the Pi can only make outbound connections. To receive incoming connections from the rest of the mesh, you need to configure port forwarding in the "Services" settings page.
Eventually I'd like to add a "direct" mode so that port forwarding is unnecessary: https://github.com/urlgrey/hsmm-pi/issues/79


Replace default storage DBMS

To simplify the installation of bamboo on local environment I would recommend to use SQLite instead of MySQL (like https://github.com/symfony/symfony-demo does).
This could also reduce the requirements on development.

Ping @mmoreram you already know I'm :+1: but didn't remember why you didn't want

Well, SQLite is even more difficult to install than mysql :/
What do U think?

Same problem as before, we can't add SQLite as a requirement, as SQLite is not a requirement xD.

hehe that's right @mickaelandrieu, but I think that installing the SQLite PDO and changing the configuration would be enough

@EmanueleMinotto Doctrine migrations doesn't work with sqlite :( So, for me, it's a :-1:


problem with replacing with history value

VSCode Version:
Version: 1.27.2 (user setup)
Commit: f46c4c469d6e6d8c46f268d1553c5dc4b475840f
Date: 2018-09-12T16:17:45.060Z
Electron: 2.0.7
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
Architecture: x64
OS Version:
windows10 and macOS seirra 10.12.6

Steps to Reproduce:

open a file with content:

aaa
bbb
ccc
ddd

using search and replace view search 'aaa' and input 'a0a' as replacement, then click replace button => ok: 'aaa' was replaced with 'a0a'
search 'bbb' and input 'b1b' as replacement and click replace button => ok: 'bbb' was replaced with 'b1b'
search 'ccc' and use arrow key to change replacement to 'a0a'(The replace history) then click replace button => problem: 'ccc' was replace with 'b1b' rather than 'a0a'
redo step 4 but do something extra: after using arrow key to switch replacement to 'a0a', changing 'a0a' => 'a0' => 'a0a' then click replace button ok: 'ccc' was replaced with 'a0a'

conclusion: It seems like arrow key operation only changed the UI displayed replacement and forgot to change the real replacement.

I can't repro this, can you record a gif or video?

@rebornix Let me know if it is something to do with History.

I can take this if I can get some pointers on where to start

I can no longer reproduce this issue.


Re-instated 'maximumImagesCount' functionality for GMImagePicker

Setting 'maximumImagesCount' option param doesn't work anymore after the ImagePicker library replacement. I needed this functionality for my app so I added a couple quick changes to make it come back to live.

Added back 'maximumImagesCount' param
When 'maximumImagesCount == 1', ImagePicker will return immediately after the first image selection. (And don't show 'Done' button)
When 'maximumImagesCount' is reached, show a popup letting user knows that they can't select more images.

Thanks @lamson! I'll try to make some time and look at the PR. Assuming there are no problems with it, I'll pull it in.


Replace the call to /hosts so we can get the exact device for ID

Description
The following API call: /v1/devices/45 doesn't return error 404 as expected (since there is no device with uuid 45). This is also true for uuid = 4  as  well as for other numbers. There are numbers that truly return 404 like uuid = 1.
The response has only the image-info attribute while all the other attributes are empty it seems like the image-info is taken from the latest device.
Fixes THEEDGE-1617
Type of change
What is it?

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update
[ ] Refactor

Checklist:

[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I run go fmt ./... to check that my code is properly formatted
[x] I run go vet ./... to check that my code is free of common Go style mistakes

/retest

/retest

@biancarosa I really need your eyes on it since I don't know how this will take affect on others

/retest

Codecov Report

Merging #582 (afcdec6) into main (d85806e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #582   +/-   ##
=======================================
  Coverage   30.00%   30.00%
=======================================
  Files          33       33
  Lines        3513     3513
=======================================
  Hits         1054     1054
  Misses       2315     2315
  Partials      144      144

Continue to review full report at Codecov.

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


add info in doc on how to facet with cartopy

This change explains

how to pass projection arguments to faceted subplots
how to access the Axes of the created subplots (not relevant only to cartopy), relevant to issue #1202

Thanks @vnoel !
Unfortunately this won't render well in the docs since we don't import cartopy (yet) (the example above yours isn't really compiling so it's not importing cartopy). I'll merge this tomorrow and have a look at how to render this properly in a separate PR
On a related note, I agree that we should make a plot gallery (https://github.com/pydata/xarray/issues/1061). I've tried it out with salem and it looks nice (http://salem.readthedocs.io/en/latest/auto_examples/index.html). I'll get back to this when I'll find some time...

@vnoel I'm trying to plot coastline using facet.  The subplot_kws is passed down to plt.subplots (https://matplotlib.org/api/_as_gen/matplotlib.pyplot.subplots.html).
You should be able to do:
g = ds_v_test.plot(...)
g.map(plt.gca().coastlines())

@dcherian Thanks a lot for providing an example with another approach


[question] Envoy to fail requests if ext_authz is unavailable?

By doing some investigation, looks like this is not supported, but wanted to confirm.
Is it possible to configure Envoy to fail http requests in case ext_authz service is unavailable? This could be useful when one runs cluster of envoys behind L7 LB and LB performs http health checks towards envoy instances. Does envoy support health checking towards ext_authz cluster? Could those maybe be propagated to the envoys health check response?

ext_authz should be a cluster like any other cluster, meaning that when Envoy goes to send a request to the cluster in the filter, it should handle a no healthy hosts failure condition and fail the request.

Thanks @mattklein123!


Implement Image Captioning

💥 Proposal
The goal of this project is to generate accurate and semantically meaningful captions for different types of images.
I would like to request you that please assign it to me under the tag/label GSSoC'23.

@SAPNILPATEL - a similar issue has already been assigned (#40). Hence, we cannot assign you this issue.

ok no problem.

Can I collaborate with that person and work on the same issue?

PROPOSAL
I would like to work on this issue to generate a model that can classify the images under certain meaningful actions .
Please assign me this issue under GSSoC'23 contributor.


A little reminder about copyright and unfair cloning of services

Greetings.
Could you please change the design of the service a bit? Your current design "slightly" copies the design and features of the existing service "Perplexity.AI".
Your OpenSource version is really good, but simply cloning an existing commercial product looks at least unfair to Perplexity developers.
Comparison of your service and Perplexity AI:

Your service
Perplexity AI

Hey Serena, thanks for the feedback. The project is intended to be a clone of Perplexity.  I’m not commercializing the product at all or looking to compete with Perplexity. My intention is for people to see how Perplexity might be built, and provide a self-hosted alternative.
I’ll consider a redesign, but for now I updated the README to make this more clear.

Also, showing information with images and other links has been used by other search engines even before Perplexity.


Fix docGen and update configuration/metrics

What changes are proposed in this pull request?
Fix docGen and update configuration/metrics
Why are the changes needed?
Please clarify why the changes are needed. For instance,

If you propose a new API, clarify the use case for a new API.
If you fix a bug, describe the bug.

Does this PR introduce any user facing changes?
Please list the user-facing changes introduced by your change, including

change in user-facing APIs
addition or removal of property keys
webui

Codecov Report

Merging #14629 (833b061) into master (e0bd13e) will decrease coverage by 6.24%.
The diff coverage is 39.53%.

:exclamation: Current head 833b061 differs from pull request most recent head 62cb51e. Consider uploading reports for the commit 62cb51e to get more accurate results

@@             Coverage Diff              @@
##             master   #14629      +/-   ##
============================================
- Coverage     41.94%   35.69%   -6.25%
+ Complexity     9363     7732    -1631
============================================
  Files          1497     1498       +1
  Lines         88125    88223      +98
  Branches      10482    10487       +5
============================================
- Hits          36960    31489    -5471
- Misses        48176    54113    +5937
+ Partials       2989     2621     -368

Impacted Files
Coverage Δ

...io/master/audit/AsyncUserAccessAuditLogWriter.java
0.00% <0.00%> (ø)

...a/alluxio/master/file/DefaultFileSystemMaster.java
15.78% <0.00%> (-48.24%)
:arrow_down:

...java/alluxio/master/job/JobMasterAuditContext.java
36.66% <36.66%> (ø)

...er/src/main/java/alluxio/master/job/JobMaster.java
39.67% <42.10%> (-0.78%)
:arrow_down:

.../main/java/alluxio/master/file/meta/InodeView.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...main/java/alluxio/master/metastore/BlockStore.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ain/java/alluxio/master/table/CatalogProperty.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ain/java/alluxio/master/table/PartitionScheme.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...java/alluxio/master/file/BlockDeletionContext.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../alluxio/master/file/meta/LockedInodePathList.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 129 more

Continue to review full report at Codecov.

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

alluxio-bot, merge this please


Improve All checkbox behavior and instrument list

Change metrics tag All option to go from intermediate checked to checked. This is consistent with how most checkboxes in windows and the web work.
Change instrument list on metrics page to use data grid.
Before:

After:

Microsoft Reviewers: Open in CodeFlow

Could someone take a look 🙏 I'd like to merge before conflicts start being created in the resource files.

/azp run


Fatal error during dictionary creation

We encounter this issue with current master code. Our previous build is in April which doesn't have this issue.
Seems like it's related to dictionary creation for large string columns.
Some ref from hs_err_pid40175.log file.

Stack: [0x00007fc5984f5000,0x00007fc5985f6000],  sp=0x00007fc5985f44e0,  free space=1021k
Native frames: (J=compiled Java code, j=interpreted, Vv=VM code, C=native code)
J 14055 C2 com.linkedin.pinot.core.io.util.FixedByteValueReaderWriter.writeUnpaddedString(II[B)V (84 bytes) @ 0x00007fd36b6a1208 [0x00007fd36b6a0ee0+0x328]
J 14068% C2 com.linkedin.pinot.core.segment.creator.impl.SegmentDictionaryCreator.build()V (2352 bytes) @ 0x00007fd36b6e2364 [0x00007fd36b6e21c0+0x1a4]
j  com.linkedin.pinot.core.segment.creator.impl.SegmentColumnarIndexCreator.init(Lcom/linkedin/pinot/core/indexsegment/generator/SegmentGeneratorConfig;Lcom/linkedin/pinot/core/segment/creator/SegmentIndexCreationInfo;Ljava/util/Map;Lcom/linkedin/pinot/common/data/Schema;Ljava/io/File;)V+344
j  com.linkedin.pinot.core.segment.creator.impl.SegmentIndexCreationDriverImpl.buildRaw()V+65
j  com.linkedin.pinot.core.segment.creator.impl.SegmentIndexCreationDriverImpl.build()V+15
j  com.linkedin.pinot.core.realtime.converter.RealtimeSegmentConverter.build(Lcom/linkedin/pinot/core/indexsegment/generator/SegmentVersion;Lcom/linkedin/pinot/common/metrics/ServerMetrics;)V+367
J 13651% C1 com.linkedin.pinot.core.data.manager.realtime.HLRealtimeSegmentDataManager$2.run()V (1199 bytes) @ 0x00007fd36b4cb81c [0x00007fd36b4c74e0+0x433c]
j  java.lang.Thread.run()V+11
v  ~StubRoutines::call_stub
V  [libjvm.so+0x68c453]
V  [libjvm.so+0x689c24]
V  [libjvm.so+0x68a1c9]
V  [libjvm.so+0x6cc52d]
V  [libjvm.so+0xa3fc12]
V  [libjvm.so+0x8e3b62]
C  [libpthread.so.0+0x8064]  start_thread+0xc4

Interesting, we hit it yesterday as well.
The long term fix for this is to have a dictionary that does not pad byes, but in the short term we should limit the number of characters in the string dictionary value and truncate it. The value can be configurable in stream config (not sure if we should make it configurable on a per column basis) but the default can be .... say 1024 chars? Or even 512 chars?

In our case it was not production table, so we got the table running by updating the zk segment metadata to set the start offset to the last consumed offset for the segment (essentially skipping that particular segment) and restarting the servers.

Sounds good, I think per table based config should be good enough.
We have many tables on same tenant, so it's hard to find which table really hit this issue.

it's HLC, just our log is pretty messy and it's crash during bootstrap, which many tables are persisting also.
I think it's good to put at schema however it may impact existing users if they just slightly exceeds the new threshold.

If we allow users to set the limit in the schema, (bit have the default at, say 512 chars) we should be ok, right? Yes. existing users who exceed 512 chars (and use the string in the query) may see a problem. They will need to update their schema to a value that they can live with. Of course, the higher this value, the lower the cardinality needs to be so that the dictionary does not explode. This is until we have variable length dictionary implemented.
@fx19880617  would you be able to code this up? thanks

Had an offline discussion with @Jackie-Jiang about where to put this: dictionary persistence/message decoder. I still have some data consistency concern/backward-compatibility for existing use cases, e.g. how to figure out the max entry length for each column for every table.
Meanwhile I'm playing with non-dictionary encoding, but also seeing other issue https://github.com/linkedin/pinot/issues/3373.


Wrong docker compose command

docker-compose command in the Readme file is missing a - (dash) between docker and compose resulting in a wrong command error.
e.g docker compose pull should be docker-compose pull

Thank for you creating an issue. As explained in #87, this is intentional and works when using the latest versions of Docker for macOS and Docker for Windows, or if using the v2 CLI tool.


chess rules

that script is still far from referring general chessboard games since it is based on standart chess.
it accepts convenient rules satisfactory. it is very complex define new variation cause of not available parameters defining new rules but rollover algorithms. i have just ask a request to handle a new rule to capture my own pieces which cause be mated. he can't even imagine it.

how would that possibly be done?


Special character when used in variants key are not being applied

Bug report
Describe the bug
When we use "100%" as a variant key, its css properties are not applied
To Reproduce
https://codesandbox.io/s/lingering-moon-42j9k?file=/src/App.tsx
Expected behavior
Special character used as variant key should behave exactly like any other key value

Related to #495

Also spaces and colons too. See #564 and https://codesandbox.io/s/stitches-variant-key-bug-5z92g?file=/src/App.tsx

Closing in favour of https://github.com/modulz/stitches/issues/495


Add support to control USB mux connected to target

Build verified offline; overriding apparently-stuck build check.


Bifacial rear irradiance reporting error - rear POA is multiplied by bifaciality coeff.

Hi everyone.  I'd like to report an error in how SAM is reporting rear irradiance which might cause some confusion.  This doesn't cause any errors in the final output of system power, but intermediate parameters are being incorrectly reported.
At the moment, it appears that the value returned in the SAM Data Tables 'Subarray 1 POA rear total irradiance after reflection (IAM) (W/m2)'  is reported as the rear irradiance value times the bifaciality coefficient.  This will cause significant under-reporting of the rear irradiance, for the case of people trying to compare measured back POA irradiance with modeled expected values.
It does appear that the 'Subarray 1 POA total irradiance' is being calculated more or less correctly by adding POA_front + POA_rear * bifaciality coeff.  However, please change the intermediate value reported for POA_rear to not include this bifaciality coefficient.  THANKS!
This may be related to #191, not sure.
-Chris

Possibly we need to update the inputs to the module model to capture this change, which no longer includes bifaciality in poaRear:

@caseyzak24


Clarify log file location

The documentation says the locatio is ~/.cache/nvim/codeium.log but at least on my mac the logfile is at ~/.cache/nvim/codeium/codeium.log.log (note the double .log.log).

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Superseded by https://github.com/Exafunction/codeium.nvim/pull/230 (I think this was caused by https://github.com/nvim-lua/plenary.nvim/commit/55d9fe89e33efd26f532ef20223e5f9430c8b0c0)


analyzer error while typing in a package name

Atom popped up a dialog with this stack trace while I was typing an (incorrect) package URL:
Error from the analysis server: Could not resolve URI (package:s) relative to source (/Users/skybrian/dart/work/exerciser/bin/type.dart) Invalid argument (packageUri): Package URIs must start with the package name followed by a '/': Instance of 'Uri'
#0      checkValidPackageUri (package:package_config/src/util.dart:72)
#1      PackagesBase.resolve (package:package_config/src/packages_impl.dart:37)
#2      SourceFactory._internalResolveUri (package:analyzer/src/generated/source.dart:824)
#3      SourceFactory.resolveUri (package:analyzer/src/generated/source.dart:741)
#4      ParseDartTask.resolveDirective (package:analyzer/src/task/dart.dart:3366)
#5      ParseDartTask.internalPerform (package:analyzer/src/task/dart.dart:3269)
#6      AnalysisTask._safelyPerform (package:analyzer/task/model.dart:310)
#7      AnalysisTask.perform (package:analyzer/task/model.dart:210)
#8      AnalysisDriver.performWorkItem (package:analyzer/src/task/driver.dart:271)
#9      AnalysisDriver.performAnalysisTask (package:analyzer/src/task/driver.dart:246)
#10     AnalysisContextImpl.performAnalysisTask. (package:analyzer/src/context/context.dart:1098)
#11     _PerformanceTagImpl.makeCurrentWhile (package:analyzer/src/generated/utilities_general.dart:170)
#12     AnalysisContextImpl.performAnalysisTask (package:analyzer/src/context/context.dart:1096)
#13     PerformAnalysisOperation.perform (package:analysis_server/src/operation/operation_analysis.dart:367)
#14     AnalysisServer.performOperation (package:analysis_server/src/analysis_server.dart:798)
It seems like this should be reported as an error in the code, without a stack trace.
$ dart --version
Dart VM version: 1.13.0 (Wed Nov 18 12:15:29 2015) on "macos_x64"

Yes, that should produce an error, not an exception.

This bug seems to be Mac-specific. (I wasn't able to reproduce it on Linux.)

It only happens when a .packages file exists.


Date Range Widget

New Date Range Widget:
Purpose: To provide a simple UI for users to select a date range to alter content shown on an atlas. Changes to the date range is reported to the dispatcher through it's own 'dateRangeWidgetUpdate' event, which provides details on the selected date range.
Use Case: The couchDbView could listen for this dispatch event and update the model's view accordingly based on the selected date range provided by the event.
External Libraries Used: jQueryUI's date picker, and Luxon are both used by this widget. Both libraries are already included in Nunaliit.

Date Range Widget Required Changes Following Group Discussion:

[ ] Date range widget needs to support a source model (from filter)
[ ] Source model would auto-update date range widget range
[ ] Widget would be paired with a filter.
[ ] Possibly be able to use a generic couch date view by default. Will need to investigate existing built-in views to see if anything already exists and if we could use it.
[ ] New filter code should use existing functionality used by time line widget
[ ] Add documentation for new date range update dispatch event.


Unimplemented Syscall 20

Syscall 20, also known as getpid.
The source code is here: https://cgit.freebsd.org/src/tree/sys/kern/kern_prot.c#n101

Can I maybe ask that #302 gets completed before this one? Asking as a high majority... (6/8) of my games are stuck on that issue.

Okay. I'll work on that one first.


Gateway pod crashing on Kubernetes test

Test in question: https://github.com/Qiskit-Extensions/quantum-serverless/blob/main/.github/workflows/kubernetes-install.yaml
The gateway pod keeps crashing, i.e.
--- default pods ---
NAME                                READY   STATUS    RESTARTS      AGE
gateway-59b44bcf6d-gl74z            0/1     Running   4 (23s ago)   3m37s
keycloak-0                          1/1     Running   0             3m37s
kuberay-operator-6c498d996f-wfbw2   1/1     Running   0             3m37s
postgresql-0                        1/1     Running   0             3m37s
prepuller-r2wsk                     1/1     Running   0             3m37s
scheduler-55dc447bdd-tct6r          1/1     Running   0             3m37s

source: https://github.com/Qiskit-Extensions/quantum-serverless/actions/runs/6301898339/job/17107951490?pr=974
Seems to be a GHA issue, as the issue doesn't happen when running locally:
$ k get po
NAME                                READY   STATUS    RESTARTS   AGE
gateway-5d95cd6cd5-9s49x            1/1     Running   0          111s
keycloak-0                          1/1     Running   0          111s
kuberay-operator-77b656d4f7-xgkmj   1/1     Running   0          111s
postgresql-0                        1/1     Running   0          111s
prepuller-jfpkz                     1/1     Running   0          111s
scheduler-6b57c9dfbd-4k4x7          1/1     Running   0          111s

Poking and probing in #974 to see if I can get this working (note that it did work ok in #970, not sure what happened since then...)

We got rid of this test, so this is moot


Using recovery link causes 500 error and the backend can be accessed without entering user/password then

Hi,
I just installed your plugin on the current Contao version (4.11.5) and found a big issue.
When I clicked on the provided link in the password reset email I face the following issue

Argument 1 passed to Symfony\Component\EventDispatcher\Debug\TraceableEventDispatcher::dispatch() must be an object, string given, called in /usr/src/app/vendor/markocupic/backend-password-recovery-bundle/src/InteractiveLogin/InteractiveBackendLogin.php on line 113

The whole stacktrace
TypeError:
Argument 1 passed to Symfony\Component\EventDispatcher\Debug\TraceableEventDispatcher::dispatch() must be an object, string given, called in /usr/src/app/vendor/markocupic/backend-password-recovery-bundle/src/InteractiveLogin/InteractiveBackendLogin.php on line 113

  at vendor/symfony/event-dispatcher/Debug/TraceableEventDispatcher.php:131
  at Symfony\Component\EventDispatcher\Debug\TraceableEventDispatcher->dispatch('security.interactive_login', object(InteractiveLoginEvent))
     (vendor/markocupic/backend-password-recovery-bundle/src/InteractiveLogin/InteractiveBackendLogin.php:113)
  at Markocupic\BackendPasswordRecoveryBundle\InteractiveLogin\InteractiveBackendLogin->login('<STRIPPED_USERNAME>')
     (vendor/markocupic/backend-password-recovery-bundle/src/Controller/RenewPasswordController.php:114)
  at Markocupic\BackendPasswordRecoveryBundle\Controller\RenewPasswordController->renewpasswordAction('622f53124ca3e8b684928a801bdbbac3')
     (vendor/symfony/http-kernel/HttpKernel.php:157)
  at Symfony\Component\HttpKernel\HttpKernel->handleRaw(object(Request), 1)
     (vendor/symfony/http-kernel/HttpKernel.php:79)
  at Symfony\Component\HttpKernel\HttpKernel->handle(object(Request), 1, true)
     (vendor/symfony/http-kernel/Kernel.php:195)
  at Symfony\Component\HttpKernel\Kernel->handle(object(Request))
     (web/index.php:31)

When I then try to access the Contao backend again I'm automatically logged as that user the password reset was requested for in instead of getting to the form for setting a new password.
I assume this is not a wanted behavior.
I'm running on a PHP 7.4.19 environment.
If you need any further information please let me know.
Thanks,
Dennis

Hi Dennis
No, this is of course not a wanted behaviour. Actually I can not reproduce the error. But we should fix this issue as soon as possible.
Could you send me your composer.json?
Regards
Marko

Hi Dennis
I found out that the error only occurs if you run contao under symfony > 5.0.
In Symfony 5 the function argument order for event dispatchers has switched. The event object has to be passed as the first argument to the event dispatcher method. And the second argument (the event name) is optional now.
Symfony 5:
https://github.com/symfony/event-dispatcher/blob/3dcbd3c8e4290c4e7ce362c91d9db5f52fe0b2a7/Debug/TraceableEventDispatcher.php#L131
Symfony 4:
https://github.com/symfony/event-dispatcher/blob/f5741bb072896d8cbf102f7f724cff2b45a6e710/Debug/TraceableEventDispatcher.php#L136
Please update markocupic/backend-password-recovery-bundle to the current version or at least 1.3.19
Regards
Marko

Hello Marko,
thanks for your quick response and also the quick fix. It worked so far. 👍
I also took a quick look in the source code but had no time to create a PR.
But one question kept left for me: is the usage of the variable $providerKey in the InteractiveBackendLogin.php (Line 108) an old code fragment? Because this variable is not defined anywhere (but I'm not too deep into the Contao plugin development to be honest).
Kind Regards,
Dennis


Fix annotations on frozen variables and applications

Here's a preliminary patch that fixes #15. At this point all the examples work, although I admit I'm not entirely sure what I implemented from a theoretical standpoint, hence a PR that we can discuss before merging. The key logic of this patch is in SolverLo, lines 290-297. Everything else is fallout and examples.
Examples
Examples are well-typed, unless noted otherwise

let (f : ∀ a. a → a) = λx.x in ...
let (f : ∀ a. a → a) = id in ...
let (f : ∀ a. a → a) = ~id in ...
let (f : ∀ a. ∀ b. b → b) = λx.x in ...
let (f : ∀ a. ∀ b. b → b) = id in ...
let (f : ∀ a. ∀ b. b → b) = ~id in ... ill-typed

Solution
From @frank-emrich's explanation here my understanding is that getting this right requires tracking whether we are dealing with a GVal or not. So I'm recording that information inside CLet and using it when checking inferred scheme against quantifiers. In my patch applications and frozen variables are considered non-generalizable and everything else is generalizable. There are two things that we need to get right when dealing with annotations:

Ensure that annotation is an instance of the inferred type
Determine the set of generalizable variables - these will be Λ-abstracted in the System F term. Note that quantifiers are not synonymous with generalizable variables. In example (3) above the quantifiers are [a] and generalizable variables are empty set.

First check is performed by unifying bodies of annotation and inferred type and is not affected by quantifiers in any way. The second check is now performed in following way:

If we are dealing with a generalizable expression then use the quantifiers from the type annotation. This ensures correct treatment of redundant quantifiers, which we must Λ-abstract.
If we are dealing with a non-generalizable expression we ensure that the inferred and annotated quantifiers are identical  - ensuring this allows to reject example (6) above. If that is the case we Λ-abstract generalizable variables discovered by generalization engine. Note that when we add the binder to the environment we retain it's original signature - quantifiers and generalizable variables are distinct.

When dealing with let bindings without signatures we keep the set of generalizable variables, regardless of whether we're dealing with GVal or not.
Now I get to the point where I am a bit puzzled. The above approach is not synonymous with implementing value restriction, even though we are relying on distinction between GVal and non-GVal expressions. This feels bizarre and I'm not yet sure how it affects theoretical presentation of the system and its equivalence with PLDI system. I'm pondering whether it's worth to bite the bullet and implement the value restriction. Would that simplify theoretical presentation of the paper if we wanted to show equivalence with the PLDI system? Thoughts welcome.
PS. I know naming the branch with this patch value_restriction is confusing, but I thought I'll end up implementing a proper value restriction when I started.

I am merging this PR since I now have two branches that branch off of it and I'm assuming it will get merged anyway. But the questions posed here remain. For now at least, since I'm close to implementing a proper value restriction.


Adding latest devices to the list

Adding latest devices to list of known ios devices

Thanks for that @pravinarr!


Demarcate extension by theme and with // MARK

So as to structure extension files more coherently

Chose to specify theme using the swift extension file naming format: "Class+Theme.swift"


Rework CSS files and organization into something maintainable.

Patchfox CSS is all over the place thanks to multiple rewrites and legacy CSS from past versions being reused. This makes it really hard to contribute. Ideally spectre.css should be coming from NPM and a theme file should be present.

cue %zuoCNmcSnP1uR12SuitqqCNUZZMV00LEWOf7u/5eA5w=.sha256

it is better now even if not perfect. CSS is almost all contained in:

themes/ folder.
each view Svelve component.

The missing part is the augmented CSS which is shoehorned in because no one can stop me.


base reconciler reconciles status

What
Common status reconciler. The signature
func(ctx context.Context, objKey client.ObjectKey, obj client.Object, mutator StatusMutator)

Then, it can be used as follows:
        newStatus := r.calculateStatus(rlp, specErr)
         r.ReconcileResourceStatus(
              ctx,
              client.ObjectKeyFromObject(rlp),
             &kuadrantv1beta2.RateLimitPolicy{},
             kuadrantv1beta2.RateLimitPolicyStatusMutator(newStatus),
       )

And the mutator looks like
  func RateLimitPolicyStatusMutator(desiredStatus *RateLimitPolicyStatus) reconcilers.StatusMutatorFunc {
        return func(obj client.Object) (bool, error) {
              existingRLP, ok := obj.(*RateLimitPolicy)
              if !ok {
                   return false, fmt.Errorf("unsupported object type %T", obj)
              }

             if cmp.Equal(*desiredStatus, existingRLP.Status, opts) {
                  return false, nil
              }

              existingRLP.Status = *desiredStatus

              return true, nil
}

Work being done as part of https://github.com/Kuadrant/kuadrant-operator/issues/530
This is part of a series of PR's that will end with a new controller that will only reconcile limitador limits configuration.

@Boomatang
I have tried a different approach. Iteration #2.
I do not like the panic approach because in production you kill the controller entirely, while returning an error and doing nothing only disables one feature.
This is not entirely clean approach either and there is still a chance to mistakenly mix types by the developer, so the status updater tries again and again returning error. But it is also true that it is the same mutator approach as the controllers use to reconcile resource specs (not status). It is being used by all the controllers and never hit any issue like that. So I think it should be good enough.
You might be interested in looking at the envoygateway controller implementation of the generic status updater: https://github.com/envoyproxy/gateway/blob/main/internal/provider/kubernetes/status_updater.go#L72-L121

There was the one documentation issue, but other than that every thing is working as I expected. This newer method seems far nicer than before.

good catch. Addressed


Order of header parameters

Noticed that the order of the parameters had changed at some point, nowadays the header parameters are added to the end of the parameter array.
That results in possible headers are located at the end of the list in Swagger UI.
What do you think of always having the possible header parameters at the beginning of the list?

I don't see why not. I would spec it as well.

Added test and changelog entry.

👍 … after addressing comment


[Catalog] Generate Models for Emoji pack

Extending Meshery Models
Meshery as an extensible platform allows users to import, generate and export models that are either annotation (SVGs) or infrastructure configuration (CRDs).
Create a new model with the following specifications:
Model:
Model Display Name:
Model Resources:
Then list the model on  by raising a PR.
Contributor Resources

The meshery.io website uses Jekyll and GitHub Pages. Site content is found under the master branch.
See the Contributing to Meshery.io Website section of the readme.md and other contributing instructions, too.
See Meshery site designs in this Figma designfile. Fill-in the Layer5 Community Member Form and join the Layer5 Community Slack for access.
See how to import, generate and export models.
Link to CSV template

@Jougan-0  I want to work on this issue.

@Jougan-0 I would like to work on this.

Here you go @shinigami-777

@Jougan-0 Can you specify the Model Resources and on how to list the model in the PR.


Fixing buffer locking/unlocking code so that it does not leak memory …

…and handles.

Before I was unable to open SteamVR with DisplayServer running, but with these changes its able to startup. Seeing performance issues here during SteamVR demos, looks like a very low framerate, PC barely keeping up. SteamVR (VRServer) crashes during a demo (Senza Peso), and hangs until the process is killed. I don't know if any of that was introduced with these changes, I suspect not, as I had never tested this path before.


Remove reference to BangBang

Amend the warning in the docs to avoid suggesting that the convention is exactly the same as BangBang.jl's

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

Project coverage is 90.49%. Comparing base (11233ce) to head (2ab41f0).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #167      +/-   ##
==========================================
- Coverage   94.10%   90.49%   -3.62%
==========================================
  Files          61       15      -46
  Lines        2156      873    -1283
==========================================
- Hits         2029      790    -1239
+ Misses        127       83      -44

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

Note however that BangBang.jl itself does not guarantee that the input will be mutated whenever possible:
julia> using BangBang, StaticArrays

julia> src = [2.0, 3.0];

julia> dst_before = zero(src);

julia> dst_after = broadcast!!(abs2, dst_before, src)
2-element Vector{Float64}:
 4.0
 9.0

julia> dst_before  # it was mutated
2-element Vector{Float64}:
 4.0
 9.0

julia> src_static = MVector(2.0, 3.0);

julia> dst_before_static = zero(src_static);

julia> dst_after_static = broadcast!!(abs2, dst_before_static, src_static)
2-element MVector{2, Float64} with indices SOneTo(2):
 4.0
 9.0

julia> dst_before_static. # it wasn't mutated but could have been
2-element MVector{2, Float64} with indices SOneTo(2):
 0.0
 0.0


Allensu distribution/kilo

add a pymysql error handle when locked is released
@lloydwuhb @HuanMingLiang

Please double check the commits needed.


Setup Integration - amazon.com works somewhat but amazon.de won't at all

Under construction
I figured I was missing the alexapy debug log - adding it a the moment.
Describe the bug
I recently did an overhaul of my HASS instance and in the progress I needed to recreate the alexa_media integration.
Previously I was using a combination of configuration entry and integration, which was working fine.
With the new HASS instance I am only using the integration as the wiki suggests, but the setup fails with Unknown error, please report log info (logs at the end).
While troubleshooting i did create the integration with amazon.com by accident, but it worked fetched a captcha and 2FA and even found my devices. But devices found with the amazon.com integration are crippled, not showing media and read only, so I am still facing an issue here.

Additional Info
I did delete my pickle while redoing everything deleted the new (amazon.com) one multiple times while troubleshooting.
Actually I am quite sure the error I get now is not the same as a few days before when I was trying. The previous error was an ContentTypeError where the text\html did not match the json that was needed. Likely due to my update 2.8.6 -> 2.8.7?

Solved
I missed the change where the pickle was moved to the .storage dir.
There was a pickle with the wrog credentials stored. With the alexapy debug log I was able to figure that out since there was an error regarding credentials and that it was using stored credentials.
Feature Request
It did not matter what credentials I was using since the config flow was using the stored pickle.
Why using the pickle at all in the config flow? At least the user should be notified, that stored credentials were used and the ones he entered in the setup are not used.

The pickle file is ignored if it doesn't successfully log in so it shouldn't matter if it's there. It allows you to save your credentials across HA installations.

Since I deleted the faulty pickle file and it worked since then I can't provide the logs anymore.
But I can guarantee that at least one of the 25+ times I tried to add the integration should have worked, if the faulty pickle was not preventing it. I am using an password manager to fill the form, so typos are unlikely.
Happy my problem is solved tho.
Keep up the good work.
Btw. - do you have way so I can buy a beer/coffee to express my gratitude?

It's possible the pickle is providing session information that puts it in a weird state. I'll double check that. https://www.buymeacoffee.com/alandtse if you'd like to buy me a coffee. Thanks!


3D Workflow Issues

There has been a ton of great work on the 3D side of Godot.  There are still a few things that really limit the speed of the workflow.

Accidentally selecting the collisionshape or mesh of a rigidbody or kinematic body instead of the parent.  I don't know what the solution would be though.  There might be instances where you would want to be able to select the child (perhaps the player is holding a weapon or item).  This is one of the areas where the node based structure seems to slow down workflow.
Inability to scale a parent of a collision shape, or scale a rigidbody.  Having to change the extends of a collision shape and scale the mesh separately is tedious.  Being able to scale one item and scale all of it's children is intuitive and user friendly.
I would love to have an editor option to put the transform at the top of the inspector.  I reference it more than any other item in the inspector.
The ability to snap to vertices of other meshes.  This is a huge time saver when laying out a level.  Again, the node based system might make it a challenge since the mesh we are moving isn't always the parent node, and so we might snap the mesh to the correct location, but leave the parent and collision shape behind.

I love the coding experience in Godot and the UI system works great, but the workflow for 3D just feels like I am fighting against it rather that it working with me.

Can you talk in detail about?:

I would love to have an editor option to put the transform at the top of the inspector. I reference it more than any other item in the inspector.

I understand that the items in the inspector are organized by node inheritance, but having the transform so far down and collapsed by default makes getting to it cumbersome.  I realize that it isn't perfect for every workflow, but I would like to have an option to move (or pin?) the transform to the top.  This is especially important since selecting items in the viewport often selects a node you didn't intend to click.
On a side note, I still think the transform takes up more space than it needs to:

Would it make sense to add another panel (similar to the node inspector) that only shows certain properties if they're available? Such as translation, rotation_degrees, scale, etc.
Something like this:

In far most cases, you don't want to use transform as-is, and just use the gizmos and stuff, so having it all the time visible does not make much sense. If you really want this, you could make an add-on very easily and put it in the asset library.
What I thought for a while that could be done is either a button or an option in the menu to "flip" inheritance, so base classes are first, in case you are editing a lot the base class and not the derived ones.

The problem with using the Gizmos in 3D is how easy it is to select the wrong node of an item.  I would prefer to use the gizmos, but it's not always user-friendly, and without snapping to meshes, it's sometimes hard to be as precise as you want.

@turtletooth reduz mentioned doing the inheritance in reverse order is quite easy. Do you think trying a build of it with that would help?
Reduz also wants more evidence it is a problem.

https://github.com/godotengine/godot/blob/a20235aeb02c0c9e5ce58c0236f88a19865d571c/editor/editor_inspector.cpp#L1456
Change:
"object->get_property_list(&plist, true);" to "object->get_property_list(&plist, false);"
Note the node scripts are in the wrong place, but it's a cool proof of concept.

@fire Thanks for looking into this.  I'm not sure this is actually any better since there is still so much noise around the transform, such as the Node settings and the Matrix in the transform.
I think that all of this is just struggles with the Node based system of Godot compared to a component based system such as Unity.  In a component based system, you never have to worry about clicking the wrong node in the viewport - each object is all one thing.  Whereas in Godot, unless you sub-scene everything, it's very easy to get mixed up in the child nodes.  The same goes with the inspector - in a component based system, it's not bound by an inheritance based structure that adds noise and complexity to the interface.
There is so much I love about Godot- I just think some user experience needs tweaking.  I get that it will never be a component based engine, but surely there are things that can be done to make the workflow smoother.

Whereas in Godot, unless you sub-scene everything, it's very easy to get mixed up in the child nodes.

This might be related to the inability to lock nodes (and make their children non-selectable) in 3D, like it's already possible in 2D.

@Calinou: Isn't there a PR for locking in 3D?

Just want to throw my hat in the ring on this, I'm for having the transform at the top too. One of my biggest issues with the inspector is the lack of consistency in placement of the classes. The order will always be the same, but the placement would be different. So, some nodes, the transform would be closer to the top (because the inheritance tree) and other times, it'll be way down, or sometimes in the middle. This creates a fairly jarring experience for me personally. Like @turtletooth, I use the Transform the most so far (I actually just stayed up all night from (9:30 PM to the current time of 8:05 AM. This is GMT - 5 by the way) exclusively working in the 3D section.
@fire & @reduz I for one would appreciate an option to at least invert it. As it appears that I interact with base classes more frequently.

@Duroxxigar can you contact me about making a proposal for inverting the properties?

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Add protobuf TxBody demo

Closes #211
Here's the basic flow. From the documentation I'm not clear on how type_url or the interface registry are supposed to work, so will have to discuss that with @webmaster128 next week.

From the documentation I'm not clear on how type_url or the interface registry are supposed to work, so will have to discuss that with @webmaster128 next week.

If I understand ADR-023 correctly, the usage of fake URLs are planned:

The adoption of google.protobuf.Any as the recommended approach for encoding interface types (as opposed to oneof) makes package naming a central part of the encoding as fully-qualified message names now appear in encoded messages.

Here is how Google does the translation:

The pack methods provided by protobuf library will by default use 'type.googleapis.com/full.type.name' as the type URL and the unpack methods only use the fully qualified type name after the last '/' in the type URL, for example "foo.bar.com/x/y.z" will yield type name "y.z".

https://github.com/protocolbuffers/protobuf/blob/master/src/google/protobuf/any.proto#L87-L91
I raised that question in the Cosmos chat. Let's see.

The default values of the class seem to be an issue. Default values are not mentioned in the documentation here: https://github.com/protobufjs/protobuf.js#using-decorators. With the following diff the tests pass:
diff --git a/packages/demo-protobuf/src/decorator.spec.ts b/packages/demo-protobuf/src/decorator.spec.ts
index bc61e47..93d4966 100644
--- a/packages/demo-protobuf/src/decorator.spec.ts
+++ b/packages/demo-protobuf/src/decorator.spec.ts
@@ -17,7 +17,8 @@ describe("decorator demo", () => {
       public readonly booleanDemo: boolean = true;

       @CosmosField.String(2)
-      public readonly stringDemo: string = "example text";
+      // @ts-ignore
+      public readonly stringDemo: string;

       @CosmosField.Bytes(3)
       public readonly bytesDemo: Uint8Array = Uint8Array.from([1, 2, 3, 4, 5, 6, 7, 8]);
@@ -29,14 +30,21 @@ describe("decorator demo", () => {
       public readonly uint64Demo: number = 123;

       @CosmosField.Repeated(6)
-      public readonly listDemo: readonly string[] = ["this", "is", "a", "list"];
+      // @ts-ignore
+      public readonly listDemo: readonly string[];
     }

     const MsgDemoT = myRegistry.lookupType(typeUrl)!;
     const TxBody = myRegistry.lookupType("/cosmos.tx.TxBody")!;
     const Any = myRegistry.lookupType("/google.protobuf.Any")!;

-    const msgDemo = MsgDemoT.create();
+    console.log(MsgDemoT.create);
+
+    const msgDemo = MsgDemoT.create({
+      int64Demo: 20, // value overridden by `public readonly uint64Demo: number = 123;`
+      stringDemo: "foo",
+      listDemo: ["a", "b", "c"]
+    });
     const msgDemoBytes = MsgDemoT.encode(msgDemo).finish();
     const msgDemoWrapped = Any.create({
       type_url: typeUrl,


Add deployment guides

Let's add a collection of deployment guides for common, popular platforms! We can start by matching the Bolt for JavaScript guides for Heroku and AWS Lambda. Ideally, we'd also like to add a guide for a container platform.
The page URLs

https://slack.dev/bolt-python/deployments/

Progress

[ ] AWS Lambda deployment guide
[ ] Heroku deployment guide

Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

Cool 👍  I'm happy to be an early reviewer. Feel free to make a draft PR for views.

Cool 👍  I'm happy to be an early reviewer. Feel free to make a draft PR for views.

@mwbrooks As you have a work-in-progress branch for this, can you create a draft pull request for now? No rush about finishing them. https://github.com/slackapi/bolt-python/compare/main...mwbrooks:190-heroku-deployment-guide

We have not been working on this task for a while. I think that we can close this issue if other maintainers are fine with it. Developers can refer to https://slack.dev/bolt-js/deployments/heroku for Heroku-specific speps.

Personally, I think we can stop work on the Heroku guide and instead ensure that the Heroku example app included in this repository is up-to-date and helpful. So far, I'm seeing that developers are getting more value out of short example apps than long-form guides.
@srajiang Do you have any opinions, since you've taken the time to update the Getting Started and Heroku guides for Bolt JS?

As I mentioned in my last comment above, I agree that we can stop working on this for the reason @mwbrooks mentioned. If we go with the direction, we can close https://github.com/slackapi/bolt-python/pull/441 too.

@mwbrooks @srajiang Let me close this issue and the PR for now. If we come up with some idea for better documentation, we can revisit at anytime!


Adding rateLimit middleware

Hey, My changes are all about rate-limiting for PHP and slim.
The middleWare was also available on https://github.com/amireshoon/RateLimitMicroFrameWork .

Warning: @amireshoon, your code doesn't support concurrent updates to data file (by dafault ../tmp/rate_limit/rate.limit), so it is sensitive to data corruption and/or race conditions.

I'm sorry but this is not something that we will be adding to the skeleton. Rate limiting middleware isn't all that common and is quite a complex problem to solve. Starter projects wouldn't be needing this.
Thank you for an attempt at contributing though.


Fontstack

Why include Montserrat from Google Fonts and then use Source Sans Pro?
Wouldnt it be better using a native fontstack like below?
$font-family:
  -apple-system,
  BlinkMacSystemFont,
  "Segoe UI",
  "Roboto",
  "Helvetica Neue", Arial, sans-serif, !default;

This is taken from Bootstrap but without the Emoji stuff.
https://getbootstrap.com/docs/4.1/content/reboot/#native-font-stack

Oh, it seems like I've choosed Montserrat and then switched and forgot about the including. Thanks!
Using the native fontstack would be definitely better, I will change it!

Okay, I've fixed it in 6faac0934e7f5ffd8e19d183a8177d278b9f80ba :)


Is it possible to get unclustered annotations on current screen?

I'd like to know if it's possible to get all annotations on current screen for certain zoom level? Thanks.

Thinking about the problem, the best option may be to use a very large cell size (possibly the full screen size) then unpack the grouped clusters that are returned.


GPU: detect GPU index

Hi, we have forked your project and made some modifications as gpustack needed. We noticed that your project already supports most of the features we require. We would love to switch back to using the upstream fastfetch, so that we can contribute directly to the project with any future issues or improvements.
In cases where a machine has multiple discrete GPUs from the same vendor and brand, the GPU names are often identical. GPU vendors provide a GPU index to help users quickly identify each GPU like below. Would it be possible for fastfetch to support retrieving this vendor-provided GPU index during detection, allowing users to easily match the detected GPUs with their respective indices?

Output of Json for this PR
(used configc.json to simplify the output)
[
  {
    "type": "OS",
    "result": {
      "buildID": "",
      "codename": "",
      "id": "Windows 11",
      "idLike": "Windows",
      "name": "Windows",
      "prettyName": "Windows",
      "variant": "Pro",
      "variantID": "",
      "version": "11",
      "versionID": ""
    }
  },
  {
    "type": "Kernel",
    "result": {
      "architecture": "x86_64",
      "name": "WIN32_NT",
      "release": "10.0.22631.4037",
      "version": "22621.1.amd64fre.ni_release.220506-1250",
      "displayVersion": "23H2",
      "pageSize": 4096
    }
  },
  {
    "type": "GPU",
    "result": [
      {
        "index": 1,
        "coreCount": 10240,
        "coreUsage": 0.0,
        "memory": {
          "dedicated": {
            "total":<PHONE_NUMBER>2,
            "used": 64249856
          },
          "shared": {
            "total":<PHONE_NUMBER>2,
            "used": null
          }
        },
        "driver": "<IP_ADDRESS>78",
        "name": "NVIDIA GeForce RTX 4080 SUPER",
        "temperature": 32.0,
        "type": "Discrete",
        "vendor": "NVIDIA",
        "platformApi": "Direct3D 12.2",
        "frequency": 3105,
        "deviceId": 73365
      },
      {
        "index": 0,
        "coreCount": 10240,
        "coreUsage": 0.0,
        "memory": {
          "dedicated": {
            "total":<PHONE_NUMBER>2,
            "used": 0
          },
          "shared": {
            "total":<PHONE_NUMBER>2,
            "used": null
          }
        },
        "driver": "<IP_ADDRESS>78",
        "name": "NVIDIA GeForce RTX 4080 SUPER",
        "temperature": 34.0,
        "type": "Discrete",
        "vendor": "NVIDIA",
        "platformApi": "Direct3D 12.2",
        "frequency": 3105,
        "deviceId": 68246
      },
      {
        "index": null,
        "coreCount": 32,
        "coreUsage": null,
        "memory": {
          "dedicated": {
            "total": 134217728,
            "used": null
          },
          "shared": {
            "total":<PHONE_NUMBER>2,
            "used": null
          }
        },
        "driver": "<IP_ADDRESS>37",
        "name": "Intel(R) UHD Graphics 770",
        "temperature": null,
        "type": "Integrated",
        "vendor": "Intel",
        "platformApi": "Direct3D 12.1",
        "frequency": 1550,
        "deviceId": 72580
      }
    ]
  }
]

You should use deviceId. It's a unique, vendor independent PCI device identifier that only changes when you change the PCIe slot of your GPU.

Please also don't change src/3rdparty/yyjson/yyjson.h before you make it in the official repo of yyjson.

You should use deviceId.

On *nix, it's a unique, vendor independent PCI device identifier that only changes when you change the PCIe slot of your GPU.
On Windows, it's AdapterLuid

Thanks for explaining. Many applications use the GPU index to locate the GPU in use. For example, in ray's documentation, you can see that a variety of GPUs use the index to set the GPU visible to the application. If fastfetch can detect the GPU index, it will be very easy to integrate with these applications.


konflux(prod): add toolchain host and member OLM configuration

Move the OLM configuration for host and member operator one level up , so that it get's deployed to both Stage and Prod via ArgoCD.
Jira: https://issues.redhat.com/browse/ASC-496

/retest

/test appstudio-e2e-tests

/test appstudio-e2e-tests

/retest

/test appstudio-e2e-tests

Not sure I understand what exactly failed in the e2e tests: https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/test-platform-results/pr-logs/pull/redhat-appstudio_infra-deployments/3342/pull-ci-redhat-appstudio-infra-deployments-main-appstudio-e2e-tests/1764412885297008640/build-log.txt
Seems like registration service deployment get's deleted? 😕

@hugares @dperaza4dustbit I had to rebase , could you PTAL and /lgtm if looks ok ?

/lgtm

/retest


Added field for additional args for streamlink in settings

Added a field to setting to allow users to pass additional argumants to streamlink.
This is usefull for passing flags that are currently not supported along with future flags that me be added by stream link without the need to update LiveStreamDVR

Have you tested this with arguments that contain spaces?


[FEATURE] Optimize column checking codes in each function with decorator

Description
Decorator pattern is used to check columns in Python evalutor (https://github.com/Microsoft/Recommenders/blob/master/reco_utils/evaluation/python_evaluation.py). It would be good to have it used across the repo where column checking is a common task for properly using a function.
Expected behavior with the suggested feature
The code for functions that may need to check columns can be optimized from
def func_old(df, col_user, col_item, col_rating):
    columns = df.columns
    if col_user not in columns:
        raise ValueError("Column {} is not in the input data".format(col_user))
    ...

to
@check_column
def function_new(df, col_user, col_item, col_rating):
    ...

Other Comments

what can we do if the required columns are missing? catching it and raising an error seems the same as not checking at all and raising the error where it occurs in the function

@gramhagen Yea it should check the existence of columns as well. If it occurs in the function, the error might be different. See #543 for example. The input datasets to the evaluator may be different, and the difference in dtypes may result in some unexpected behaviors.

yeah, you're right. I am working on some changes to the python evaluation code, and will push a PR for review of that approach. Changes to address type checking can be put there.


CSV reporting MODEUR-15 MODEUR-18

This adds three service:

/eusage-reports/stored-reports/use-over-time/csv
/eusage-reports/stored-reports/reqs-by-date-of-use/csv
/eusage-reports/stored-reports/reqs-by-pub-year/csv

YYYY-MM ? or Mon-YYYY (mind you that Mon is language specific).

YYYY-MM for months sounds good since it't language independent


Add a page with the MediaElement

See https://learn.microsoft.com/en-us/dotnet/communitytoolkit/maui/views/mediaelement

This will be released in v1.3


Hotreload and debugging do not work with docker dev environment

we could also use a docker command that run tests


Polish Azure.ResourceManager.ProviderHub for first stable

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.ProviderHub


Increase maximum font size

Currently I'm working on a project which requires a visible font heigth of at least 40 milli meters. The display I'm using has a resolution of 320x480 pixels, while 1 pixel is 0.153mm in height. With a maximum font size of 255 pixels the maximum possible font height would be 39.015 mm. So I'm not able to meet the projects requirements.
A way to set font size above 255 pixels would be great. Including a way in GuiX studio to increase the size. Currently it is not even allowed to set a size bigger than 255 pixels.
I already removed glyphs by defining characters in a string which are really used in the project. That just gives me 30mm, tops.
Converting the needed characters into images, e.g. png, with a size of 40mm is not an option, because then I won't be able to set the font color.

@IvanThor Recommend converting the needed characters into PNG images. In the GUIX Studio pixelmap edit dialog, set the image as the alphamap format, retaining only alpha information. Assign an alphamap to a widget, and set the widget as transparent, the widget fill color will work as the alphamap drawing color.

Alright, it has to suffice. Thank you very much.


Add config for Stale app

This adds a config for Stale, which I'll enable for this repo if/when this PR is merged. I've used the default config, the only change being that I've replaced "wontfix" with "stale" and made a label for it. After 60 days the bot will comment, warning that the issue will soon be closed, and unless there's activity within 7 days the issue will close. I'm happy to change any of the config options, feel free to push to the branch.

@christianbundy go ahead!

(i figured I just had to hit merge, not release a new version so I did)


rune exec succussfully however docker exec failed

kubectl exec also ok with inclavare containers.

kubectl exec also ok with inclavare containers.

Fixed by @haosanzi  in #1190


buildReferences fails - Proabably due to Singularity configuration

Hi,
I am in the  testing phase to see if my installation of Sarek, nextflow(0.27.3) and singularity (v 2.4.5) work together as expected.
So, I started with the test.sh script.
It aborts while creating the references; specifically, it dies in the BuildVCFIndex (b37_cosmic_v74.noCHR.sort.4.1.sma ll.vcf) process.
In summary, the run of test.sh is able to  finish the singularity pull cmds successfully,
but I suspect it is unable to set the variable $IGVTOOLS_HOME appropriately.
I am attaching the nextflow.log file.
nextflow.log
Your help would be greatly appreciated!

Hi,
Thank you for submitting this issue.
I'll look into it as soon as possible.
I see that the error in your log is:
env: singularity: No such file or directory

And I notice that you're using PBS as an executor.
Is singularity available within this executor?
Have you tried the local executor only?That should work without any problem on the small tests.

Thanks @jordeu for the help.
Did you manage to get more help with your sys-admins @grendon ?

We are still troubleshooting with the sys-admins. Thanks for asking.

This is still not a Sarek issue.
Given the Singularity error, I would say it's still probably a Singularity issue.

Hi Max,
This is the progress report for today:
Yes singularity seems to work with  docker containers.
The problem I see today is new. test,sh aborts and complains about the cd command.
The whole battery of tests I performed today is attached.
scriptlog.test.20180411.txt

OK, that's good progress.
I'm looking at all the logs, trying to understand.
I'll keep you up to date.

So Singularity appears to be working, you have no problem pulling images from DockerHub as well.
I think the problem now is possibly coming from Nextflow.
I'll try to find out more.
All the best,
Maxime

@grendon It might still be a Singularity problem in fact.
Can you try this command to verify it mounts the host paths (cf doc)
singularity exec <pathToAnyImage> ls -la /something

Closing due to inactivity and moving to nf-core/sarek


Pieter Cuijpers - first definition of Quantales

First definition of Quantales - a non-commutative generalization of Locales/Frames. There are still some points of discussion, which would require changes elsewhere in Mathlib, but this definition should be workeable as a starting point (I think).
It's my first PR, but I've tried to stay close to what I've seen in CompleteLattices.

Yes, generally, although it is also possible to request review.
I see that you now also consider semi groups, which has led to a proliferation of classes. I think there is yet another possibility to consider here: make the algebraic structure an argument to the class rather than extending it.
So you would write
class Quantale (\a : Type*) [Semigroup \a] extends CompleteLattice \a where ...`

The downside is that you have to write two things to get a quantale, but the upside is that you can easily change the underlying algebraic structure without defining all new classes.
So, to get a bare bones quantale, you just write:  but to get a commutative unital one, you simply change this to: [CommMonoid \a] [Quantale \a].
After that, you could potentially make IsIdem or IsIntegral mixin classes. It depends on how often they come up, and whether they come up in conjunction with each other. This is hard for me to say with my limited knowledge of quantales.

So you would write
class Quantale (\a : Type*) [Semigroup \a] extends CompleteLattice \a where ...`

The downside is that you have to write two things to get a quantale, but the upside is that you can easily change the underlying algebraic structure without defining all new classes.

That does not seem the best way to go to me. I think the definition I have now for quantales fits best.
As I explain in the preceding comments, the term is not completely consistent within literature, with some authors
using it for a semigroup and others using it for a monoid, so we have to decide on which stance Mathlib will take on it.
But in many papers I actually see it acknowledged that the original is without assuming the unit element, to subsequently
say: but in this paper we use the term to mean a unital quantale. Therefore, from a Mathlib point of view, I guess it is okay
to drag along the OneQuantale whenever a paper you are formalizing says Quantale.
The construct you are proposing would make sense if the general way of looking at it is that "you turn a semigroup into a quantale", but (just like with a ring consisting of two semigroups) that is not the way quantales are being used imo.
Thanks for thinking along though! It really helps to have someone giving feedback :-)

I think you are getting hung up on interpretations. You don't have to think of it as "turning a monoid into a quantale". Instead, you can think of it as: "a semigroup is the minimal algebraic structure necessary for a quantale".
Just for comparison, Module R M takes two type class arguments: AddCommMonoid M and Semiring R. So, to get a vector space over a field, you have to write [AddCommGroup V] [Field K] [Module K V].
I would encourage you to throw out preconceptions about how things "should" look based on how you think of it before formalization. Sometimes the requirements of formalization make things different.

Thanks... easier said than done though ;-)
It turns out now that the decision to go for additive versions of Quantales causes a clash with Semirings.
The add_idem has been defined there already, and again I have to let go of some preconceptions...

In fact, there may be multiple ways to turn a semiring into a sup-semilattice. Indeed, when it is an idempotent semiring, choosing the order to match $$x \leq y$$ iff $x + y = y$$ is a natural option, but the reverse is always an option as well.

I don't think the formalization of it all forces one direction or another. But I do understand that in some branches of mathematics people have come to think of the structure in one way (as an algebraic property of the +), and in other branches they think of the same structure in another way (as an order-theoretic property of the +). In this case, the latter group of people got to developing that part of the library first, so the first group of people will have to adapt a bit.

I don't agree with this assessment at all. Sure, there are many ways to put a sup-semilattice on a semiring, it's just that the structures won't be related (and you can do that in Lean). But in an idempotent semiring, there's a canonical way to do it so that they are. And there's no need to think of one structure generating the other. They both just exist, and the axioms force them to play nice. So there's not necessarily anything here about one group getting more say about how things develop (although that sort of thing surely does happen sometimes).

I've tried out your suggestion of making the definition dependent on the Semigroup.
I like the way it makes the definition more independent.
The problem I encountered with add_idem clashing in Kleene now disappears for the time being, because there is no need to have separate definitions for properties that are actually only targetting the Semigroup. Those should go elsewhere. So only the IsIntegral property stays and is not a mix-in class.
Now I bump into being a novice on Git. I need to figure out how to undo my changes to Kleene and IsIdem to get a clean PR... Any suggestions? (Concretely, I need to restore an old version of the Kleene file and remove the IsIdem file for the PR, preferably in such a way that I can easily put them back afterwards when I continue to actually proof theorems about idempotent quantales - because that is when I need them.)

I don't think the formalization of it all forces one direction or another. But I do understand that in some branches of mathematics people have come to think of the structure in one way (as an algebraic property of the +), and in other branches they think of the same structure in another way (as an order-theoretic property of the +). In this case, the latter group of people got to developing that part of the library first, so the first group of people will have to adapt a bit.

I don't agree with this assessment at all. Sure, there are many ways to put a sup-semilattice on a semiring, it's just that the structures won't be related (and you can do that in Lean). But in an idempotent semiring, there's a canonical way to do it so that they are. And there's no need to think of one structure generating the other. They both just exist, and the axioms force them to play nice. So there's not necessarily anything here about one group getting more say about how things develop (although that sort of thing surely does happen sometimes).

I agree. I may not be expressing myself clearly where my concern really is.
What I ran into during an earlier small project, is that I thought I had defined an arbitrary order structure to work on in combination with an algebraic structure. I had no intention of using the canonical order, which apparently also exists. But only when I got into the proofs I found out Lean interpreted me defining an order structure as well as an algebraic structure as "oh he means the canonical one". So all of a sudden I was provided with axioms that I didn't actually want to have and that invalidated my theorem. Luckily, actually, because the same could happen the other way around. If you don't notice that under the hood there is an order structure defined, you might accidentally use that one while the definitions really seem to suggest something else. Such confusions make me quite uncertain when using Lean, so I try to keep control over what is defined and what is not.
Do you have similar experiences?

You still need to merge master. Do you know how to do that?

You still need to merge master. Do you know how to do that?

Did not realize it was a separate step, next to running the CI, but I found this command: git merge origin master.
It seems to do what you suggest. Had to resolve one merge conflict on the bibliography.
Should be okay now.

thanks!
maintainer merge

Thank you!
Is there anything I still need to do now?
And what about future changes?
After this has been merged, I assume I can simply start a new one?
(For example to implement j-loreaux idea of using a variable alias.)

I've decided to remove the variable_alias for the time being, awaiting the PR by Julian on VectorSpaces and having the discussion on how to use it there.
This lead to a small change again regarding namespaces in the Quantale file.
I would like the main namespaces to be Quantale and AddQuantale, not IsQuantale and IsAddQuantale.
When I introduced the variable alias for AddQuantale, this also introduced its namespace, so the definition of addLeftResidual could be done through to_additive. Now, however, the AddQuantale namespace is not implicitly introduced anymore, which meant I had to pull the introduction of those definitions forward.
Do you agree that using the namespace Quantale is better than IsQuantale?
Especially with an eye on the future introduction of the variable_alias, I think it is more natural, but at the moment it has a small cost involved in not being able to use to_additive for the introduction of leftAddResidual.

I would personally stick to the IsQuantale and IsAddQuantale namespaces, with a view towards renaming if ever Quantale and AddQuantale become a thing

What I think is happening, is that to_additive needs to have made the connection between the Quantale namespace and the AddQuantale namespace, before it can transfer between the two. But there is no way to put the @[to_additive] before a namespace declaration, you can only do that in front of a structure or class definition. So only after you've defined a class or structure with the name of the namespace, to_additive can do its job.


[jax2tf] Fixed bug with TF silently using 64-bit operations in 32-bit mode.

We already cast the NumPy arrays to 32-bit, but this was not happening
if the input was tf.Variable or tf.constant.
In the process I have added some invariant checks, if core.skip_checks is
False, which it always is in testing.
Added also some sanity checking or arguments.

This broke some TPU tests; I disabled them here: https://github.com/google/jax/pull/3476


Fix for CARBON-15776

Reviewed, Please merge.

Tests for the fix https://github.com/wso2/carbon-kernel/pull/829


gateway: Enable cln plugin reloads

Update cln_plugin dependency to include https://github.com/ElementsProject/lightning/pull/5397
Add a script to automatically re-build gateway plugin and tell cln to reload it. Without this PR, testing any change to the gateway via CLI is annoying multi-step process.

Would that be a separate PR?

This PR allows us to do 2 things without having to restart lightningd: (1) have core-lighting switch the build of the ln_gateway and (2) change the fedimint config folder.
So it doesn't quite allow, for example, the rust tests to test the gateway with different federations sizes etc. But if we had a way to pass the JSON config itself (instead of the config directory) as a parameter here then we would be able to switch out federation configs during the tests. In the rust integration context we would just make the same call over JSON-RPC.