Can't change display resolution in Virtualbox

Describe the bug
when i change the display resolution in KDE in Virtualbox, the window changes for 0.5 sec. and changes back to 800x600
it makes no difference if it is the live image or installed
https://peertube.social/videos/watch/31518aaa-5363-4513-822e-e3e583819d1b
it is not a problem with pantheon desktop
To Reproduce
Steps to reproduce the behavior:

...
...
...

Expected behavior
be able to changes display resolution
Notify maintainers

Metadata

system: "x86_64-linux"
host os: Linux 5.4.50, NixOS, 20.03.2425.9a3e34ae601 (Markhor)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.6
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

I'm getting the same thing on a macOS host. I think it may be from using VBoxVGA instead of VMSVGA for the .ova.

It looks like it's from this bug in KDE:
https://bugs.kde.org/show_bug.cgi?id=407058

I no longer have this issue with 20.09. @davidak can you still repro?


Add ? x Rap Cham Thoi (Offest Fix) by GDVL MASHUP 2024

Song Name
? x Rap Cham Thoi (Offest Fix)
Artist Name
GDVL MASHUP 2024
Source
Youtube
Youtube Link
YjQXddtIjlJli-md
Direct File Link
No response
Song ID
727020
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 62556400

reject
invalid YouTube link


Onnx models are not available (onnx examples)

Server returned HTTP response code: 403 for URL: https://kotlindl.s3.amazonaws.com/models/onnx/objectdetection/yolov4.onnx

Hi @kokorins could I ask you: how do you get this error message? There is no official example with the YOLOv4 model.
All models are public on S3 (I made a double-check for that) but I got sometimes the same error message on macOS then I tried to download model from S3 firstly (I don't know why)

in onnx example replace sff with yolov4

Could you open this URL in browser (it should suggest to load model)? What OS and browser do you use?

@kokorins we made a double-check on our side from a few machines with a different OS, looks like your request to S3 is forbidden for some reason, so let's try to find the reason for it.

Thanks, it's working now!

Thanks for the signal @kokorins the problem is that I do nothing and it worries me.


[SPARK-35397][SQL] Replace sys.err usage with explicit exception type

What changes were proposed in this pull request?

This patch replaces sys.err usages with explicit exception types.
Why are the changes needed?

Motivated by the previous comment https://github.com/apache/spark/pull/32519#discussion_r630787080, it sounds better to replace sys.err usages with explicit exception type.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Existing tests.

Test build #138503 has started for PR 32535 at commit 122d853.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43023/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43023/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43023/

Test build #138503 has finished for PR 32535 at commit 122d853.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
    new RuntimeException(s\"Failed to convert value $value (class of $cls) \" +

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138503/

Test build #138517 has started for PR 32535 at commit 98b2183.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43038/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43038/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43038/

Merged to master for Apache Spark 3.2.0.
Thank you, @viirya and all!

Thank you @dongjoon-hyun @maropu @srowen

Test build #138517 has finished for PR 32535 at commit 98b2183.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138517/


Typechecking of datatypes with several occurences of same var in clause patterns

Consider following "second index is a successor of first index" predicate.
\static \data Next Nat Nat
    | Next n (suc n) => next

\static \function
next-test : Next (suc zero) (suc zero) => next

Obviously, no term with type Next 1 1 exists but typechecker accepts current definition as correct.
For a more complex example you can check this "list contains element" predicate which also typechecks without errors: here successfully fills the goal with type ListIn A x (x' :: xs').
\open Data.List
\close Data.List (concat)

\static \data ListIn (A : \Type0) (x : A) (xs : List A)
    | ListIn A x ((::) x xs) => here
    | ListIn A x ((::) y xs) => there (y : A) (p : ListIn A x xs)

\open Logic
\open Data.Either

\static \function
list-in-concat (A : \Type0) (xs ys : List A) (x : A) (in : ListIn A x (xs ++ ys))
               : (ListIn A x xs) || (ListIn A x ys) <= \elim xs
    | nil => inr in
    | (::) x' xs' => inl here

This is not a bug. You can't define these data types in this way. Two occurrences of variable n in patterns are different. One of them shaddows the other; so data type Next is equivalent to
\static \data Next Nat Nat
    | Next _ (suc n) => next

If you want to define data types like Next you should do something like this:
\static \data Next Nat Nat
    | Next zero (suc zero) => next-zero
    | Next (suc n) (suc m) => next-suc (Next n m)

or this:
\static \data Next Nat Nat
    | Next n (suc m) => next (n = m)


Support archive_unstable_call

While all the archive_unstable_ were quite straightforward so far, I'd like to support another call we're using in Multix archive_unstable_call which we use to call the metadata_metaddata at a specific block hash:
client
  ._request<{
    result: HexString
  }>('archive_unstable_call', [blockHash, 'Metadata_metadata', ''])
  .then((x) => opaqueMetadata(x.result)[1])

Here's the implementation: https://github.com/paritytech/polkadot-sdk/pull/1582

Any pointer for this one?

looks like just an alias of state_call?


Teslamate, Grafana & Database Exited with code 139

Hello -
I'm hoping somebody can help me out, after install when I run 'compose up' I receive the same error over and over:
pi_teslamate_1 exited with code 139
pi_grafana_1 exited with code 139
pi_database_1 exited with code 139
This is running on a raspberry pi w zero. I am not sure what details people would need, but I am happy to provide additional details upon request i

The raspberry pi w zero is not supported  as it uses the ARMv6 instruction set architecture. But I suspect you wouldn't be happy with that hardware anyway. I would recommend a rpi 3 or 4 or, if possible, something even more performant.

Thanks so much for the quick response. Awesome project BTW.


Rename sprint label to in-progress (issue #701)

Description
Resolves issue #701 by renaming sprint label to in-progress and removing reference to sprint in description. Also includes change to readme to update the board names to remove reference to Sprint board and include Support and Kanban project boards.

Tasks:

[x] Workflow updated (global.yml)
[x] ReadMe.md updated
[x] HOW_WE_USE_GITHUB.md updated
[ ]  Kanban project board name updated to In Progress

There are also some remaining references to "sprint" in the HOW_WE_USE_GITHUB.md document, can you take a glance there and update as needed?

Updated. The name of the Kanban board will need to be updated to be in line with the updated document, but I don't have permissions.

There are also some remaining references to "sprint" in the HOW_WE_USE_GITHUB.md document, can you take a glance there and update as needed?

Updated. The name of the Kanban board will need to be updated to be in line with the updated document, but I don't have permissions.

Done ✅


Cannot write to a binary file (LocalStorage)

Trying to implement a msgpack target (maybe not right approach...)
class MessagepackTarget(luigi.file.LocalTarget):
    def __init__(self, filename):
        super(MessagepackTarget, self).__init__(path=filename)

    def dump(self, d):
        with self.open('wb') as f:
            msgpack.pack(d, f)

This fails wether using mode w or wb:
  File "... python3.5/site-packages/msgpack/__init__.py", line 38, in pack
    stream.write(packer.pack(o))
TypeError: write() argument must be str, not bytes

I also tried with different format super(MessagepackTarget, self).__init__(path=filename, format=GzipFormat) but that fails as well:
  File "... python3.5/site-packages/luigi/file.py", line 126, in open:
    return self.format.pipe_writer(atomic_file(self.path))
TypeError: pipe_writer() missing 1 required positional argument: 'output_pipe'

What am I doing wrong?

sounds like a python 3 thing.
can you figure out what's the underlying issue and send a pull request? much appreciated

I tried, including various work arounds, but I couldn't get it to work... I think maybe it's due to somehow opening the file in 'w' mode, here https://github.com/spotify/luigi/blob/master/luigi/target.py#L266
I will isolate an example demonstrating the issue!

I think Python 3 has both TextIO and BytesIO. We should probably pick one of them based on whether "b" is in the mode. Not sure

O_O ... I am not sure either, other than that I'm not enjoying this Python 3 transition ⛅
https://docs.python.org/3/library/io.html#binary-i-o <= on first sight it looks like you are right, picking BytesIO on b might do the trick.

OTOH, FileIO should be able to write(bytes) as well... (as it implements RawIOBase) 😕 https://docs.python.org/3/library/io.html#io.RawIOBase.write

# test.py
import luigi

class Test(luigi.Task):
    def run(self):
        self.output().open('wb').write(b'1234')

    def output(self):
        return luigi.file.LocalTarget(path='/tmp/zzzz')

# python -m luigi --local-scheduler --module test Test
# => TypeError: write() argument must be str, not bytes

^ minimal example demonstrating the problem
import io
# fails
#br = io.BufferedWriter(io.FileIO('/tmp/xzz', 'w'))
#br.write(b'134')

# works
br = io.BufferedWriter(io.FileIO('/tmp/also_works', 'wb'))
br.write(b'134')

Hmm,
seems the problem is indeed in format, passing NopFormat works...
import luigi
from luigi.format import NopFormat

class Bullshit(luigi.Task):
    def run(self):
        self.output().open('wb').write(b'1234')

    def output(self):
        return luigi.file.LocalTarget(path='/tmp/zzzz', format=NopFormat)

if luigi.build([Bullshit()], workers=1, local_scheduler=True):
    print("success")

so let's just fix it so that we pass the flags to io.FileIO.
can you give it a shot and send a pull request? thanks!

Yeah, I fixed it here locally finally. I'll give PR a try, we can discuss the implementation details further there

Hi, sorry, got interrupted; WIP branch is here https://github.com/wires/luigi/tree/issue-1647 will PR when done (test, etc).
I'm wondering about this strange thing, the following code doesn't work:
return self.format.pipe_writer(atomic_file(self.path))

  File ".../python3.5/site-packages/luigi/file.py", line 126, in open
    return self.format.pipe_writer(atomic_file(self.path))
TypeError: pipe_writer() missing 1 required positional argument: 'output_pipe'

But this code does:
af = atomic_file(self.path)
return self.format.pipe_writer(self.format, output_pipe=af)

Any thoughts?

The line you note as not working is what currently exists in Luigi and has tests which open('w') (thus executing that line of code).
Where is the location of this self.format.pipe_writer(...)?

Ah, it's luigi/file.py:126, sorry I meant no longer working after patching it https://github.com/wires/luigi/commit/280c5e0446090f36f90db376b6628e673c4fa863

I made a PR https://github.com/spotify/luigi/pull/1648 please don't merge yet ; this should make it easier to discuss it.

Just started experimenting with luigi, trying to use it in a workflow that downloads a binary .zip file and writes it do disk.
I hit this issue immediately on Python 3.
FWIW, my expectation was that self.output().open('wb') would behave as expected. I would support an implementation that makes this happen.

I hit the same issue trying to save task state using pickle objects. My current workaround is to use
with open(self.output().path, 'wb') as out_file:
instead of
with self.output().open('wb') as out_file:

It turns out that setting format=luigi.format.Nop on the target also does the trick.
def target(self):
    return LocalTarget(..., format=luigi.format.Nop)

Is there any update to this?
It's February 2017. I just tried out Luigi and my first task hit this bug because of pickled files.

@cmmp does the comment above your not work for you? (setting format=luigi.format.Nop)

@dlstadther it does work, but it's sad to write code using workarounds :(
I was really excited to use Luigi (coming from a Gearman background), but this made me a little sad. I mean, I get this is open source, but this bug is from April last year and there is a potential workaround from the same time https://github.com/spotify/luigi/pull/1648 that seems abandoned. I'm using Python 3 for everything. I'm now worried to start using it if support is not good atm.

@cmmp I wanted to change this PR to become a documentation update which
explains that format option is needed for binary files, but didn't free
time yet. It was more confusion on how Luigi works than a "real bug".
And FWIW, we are still using Luigi, where most data is stored using
msgpack (as pickle doesn't perform except for the smallest datafiles).
Our processing code derives from a base class that (amongst others) sets up
msgpack writing. This has been running in production without too many
issues.
But (and I hope I don't offend anyone), I must say Luigi is somewhat
incidentally complex and brittle and in retrospect maybe not the best fit
for our use-case. Still, with some effort Luigi gets the job done and
otherwise stays out of your way.
There are good alternatives though, for instance, if you use pandas, you
might like dask which is a part
of the blaze ecosystem.
HTH

@wires That is an incredibly helpful answer. Thank you very much. I will look into dask and blaze.

So the source of that open method is:

    def open(self, mode='r'):
        rwmode = mode.replace('b', '').replace('t', '')
        if rwmode == 'w':
            self.makedirs()
            return self.format.pipe_writer(atomic_file(self.path))

        elif rwmode == 'r':
            fileobj = FileWrapper(io.BufferedReader(io.FileIO(self.path, mode)))
            return self.format.pipe_reader(fileobj)

        else:
            raise Exception("mode must be 'r' or 'w' (got: %s)" % mode)

It intentionally clobbers the 'b' from the format string. But why is that necessary?

@honnibal I agree. If someone is putting "b" in the string, it implies they don't understand luigi and don't know about the format constructor argument for LocalFile. I propose that the presence of a 'b' should throw an appropriate exception with the text "Use format=.....", rather than swallowing the "b"

I've used luigi for several years, and just ran into this while upgrading to Python 3.
As a user, it's very confusing to find that I can't write a binary target file by opening it with a 'wb' mode.  Even if format.Nop were documented, it would be confusing for luigi target file semantics to be almost-identical to the python standard.  Standard fs usage is to pass the binary flag to mode, not to set a separate format kwarg.
Is there support from the code owners for an implementation that supports mode=wb for Targets?  I think several people in this thread would be happy to offer code, but need some direction on acceptable solutions first.

@dcheck I'm not intimately familiar with the implementation of Targets and why they were written the way they were. But that said, I'm open to improving them for binary writes.
Feel free to submit a PR, easing this behavior! Thanks!


Is Canada supported?

SmartHQ with GE profile loaded in the app. Works fine.
Integration loaded via HACs custom repository and integration.
When asked the username and password always error out even though it is the correct password.
Tried troubleshooting steps bur SmartHQ forces a special character in the password.
Could the issue be that my region is Canada?

I am in Canada. I have been successful in adding this integration to Home Assistant for a Cafe Dishwasher.

Canada is supported.


New config: allows to set of options per-browser

Options that can not be set per-browser are grouped
into system section. Additional changes:

capabilities option is renamed to desriedCapabilties to match
the spec.
"--noCalibrate" flag is gone, calibration now can be disabled with
"calibrate: no" option.
http section is removed, only timeout can now be set ("httpTimeout"
option).

:ok:


Null Pointer Exception When Upgrading Past Version 7.11

When we attempt to run recipes using versions later than 7.11 the Run method inside the Recipe class gets executed, it throws a null pointer exception that we cannot track down.
Rolling back to 7.11 or earlier solves the issue.

@chrismhess can you provide more details? Which recipe are you trying to run? What's the full stack trace? You can also join our slack and message us directly to share more details.

@chrismhess Thanks for opening an issue! I suspect this was related to the addition of Javadoc elements in the AST, and many of the NPEs have been fixed now.
Closing the issues due to inactivity. Please let us know if you have any other problems with a stack trace and we'll track it down


Should packages to install be optional in papermill run notebooks kfp component

Describe the issue
Should packages to install be optional in papermill run notebooks kfp component.
The question we should ask is, do we always require a package to be installed in order to run a notebook?

also see https://github.com/elyra-ai/elyra/issues/1956, which would set a default of [] instead of making this property optional

Closing due to duplicate of #1956


tslab instructions - Update README.md

I was very familiar with Jupyter notebooks but hadn't used tslab before. Took me a minute to figure out how to run the examples. This may not be the best approach, but got me up and running and thought it might help someone else.

Fixes # (issue)

Thanks for this! yarn build makes sense but I'm surprised you needed to add that tsconfig.json file 🤔


[lineageos] Add 20.0

See https://lineageos.org/Changelog-27/

Should this set 18.1 as unsupported? (since only the latest two get official builds).


OAuth2 middleware with appHttpPipeline leads to 302 error

I am currently using Dapr for a project I am working on and have taken a closer look at the OAuth2 middleware (AzureAD).
If I use httpPipeline in the Dapr configuration, I can call the sidecar directly (Ext --> Sidecar --> App2). Btw if I am already logged in a browser window, I often get this error message "Too big request header". However, in a fresh browser window or an incognito tab, the middleware always works when I call, for example, http://localhost:3500/v1.0/invoke/app2/method/app2route.
However, I want the middleware to be called on service to service invocation calls (using Dotnet Sdk). The call chain would look like this App1 --> Dapr1 --> Dapr2 --> App2 .  As soon as I use appHttpPipeline in my Dapr configuration I get the following error message in every sidecar:
ERRO[0003] app returned http status code 302 from subscription endpoint
I also use PubSub in my project. App2 in this case would be a publisher and App1 a subscriber.  I have configured my subscription endpoint as described here.
If I ignore that message and try to call app2 from app1 I am getting the following error message in the App1 logs:
[2023-07-12T16:21:36.396Z] Error: Dapr.Client.InvocationException: An exception occurred while invoking method: 'app2route' on app-id: 'app2'
 ---> System.Text.Json.JsonException: '<' is an invalid start of a value. Path: $ | LineNumber: 2 | BytePositionInLine: 0.
 ---> System.Text.Json.JsonReaderException: '<' is an invalid start of a value. LineNumber: 2 | BytePositionInLine: 0.
   at System.Text.Json.ThrowHelper.ThrowJsonReaderException(Utf8JsonReader& json, ExceptionResource resource, Byte nextByte, ReadOnlySpan`1 bytes)
   at System.Text.Json.Utf8JsonReader.ConsumeValue(Byte marker)
   at System.Text.Json.Utf8JsonReader.ReadFirstToken(Byte first)
   at System.Text.Json.Utf8JsonReader.ReadSingleSegment()
   at System.Text.Json.Utf8JsonReader.Read()
   at System.Text.Json.Serialization.JsonConverter`1.ReadCore(Utf8JsonReader& reader, JsonSerializerOptions options, ReadStack& state)

I suspect that the OAuth middleware is now also used for pubsub calls, which is very inconvenient in this case.
If this is the case, would it perhaps be possible to apply the middleware only to certain building blocks (in my case, service-to-service invocation)?
Expected Behavior
Oauth middleware kicks in when I am calling Dapr sidecars from application code (appHttpPipeline configured). If there is no valid token for the user, it will redirect to login the user. After login was successfull it redirects to the initial made request route.
Actual Behavior
See above.
Steps to Reproduce the Problem
Setup two dotnet projects with Dapr sidecars & configure a Dapr pubsub component.
Configure the OAuth2 middleware component (Azure AD) with the appHttpPipeline configuration.
Make App2 publishing messages and App1 subscribing to them.
Try to call an App2 endpoint from App1 with Daprs dotnet service invocation.

I will move this to dapr/dapr since it's an issue with how the runtime treats 302 responses

Dapr load OAuth2, daprClient. InvokeMethodAsync method is redirected to the login page to return to 302 code. However, a json parsing error occurred when the result was received. How can I have dapr automatically redirect the client to the login page or determine if the client has permission

I will move this to dapr/dapr since it's an issue with how the runtime treats 302 responses

Dapr load OAuth2, daprClient. InvokeMethodAsync method is redirected to the login page to return to 302 code. However, a json parsing error occurred when the result was received. How can I have dapr automatically redirect the client to the login page or determine if the client has permission


AzureRm? Really?

Please update to no-longer-new 'Az' cmdlets

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 4a3d35c1-9e1b-c3ae-06f1-af0402ac24e2
Version Independent ID: a3006cac-6a67-c14e-125f-c816d38df6ed
Content: Create a Windows Server VM by using PowerShell in Azure Stack Hub - Azure Stack Hub
Content Source: azure-stack/user/azure-stack-quick-create-vm-windows-powershell.md
Service: azure-stack
GitHub Login: @mattbriggs
Microsoft Alias: mabrigg

@jtuliani Azure Stack Hub hasn't release the Az cmdlets yet.
#please-close


Crud e2e

This introduces an e2e test that:

Signs in
Creates an observation without evidence
Uploads the observation
Adds a comment to the observation
Deletes the observation

It is currently not checking if the comment appears because of this:  https://github.com/inaturalist/iNaturalistReactNative/issues/695

Not always, but sometimes the test fails because of this behaviour: https://github.com/inaturalist/iNaturalistReactNative/issues/694

Closes #694
Closes #683


[Breaking] AnimController time in state fix

The AnimController class doesn't factor in the speed of the current state when tracking the total time that state has been active. This means transitions with an exit time parameter will not trigger as designed (based on the normalised duration of the active state) unless that source state has a speed of 1.
This PR addresses this bug, but in doing so will change the behaviour of all transition's that use an exit time to determine when to move from a state with a modified speed. It also affects the values returned by the AnimComponentLayer#activeStateCurrentTime and AnimComponentLayer#activeStateProgress functions. When the active state has a modified speed value, these functions now return values which account for that speed.
Fixes #3837
I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

I'm struggling to see how much disruption this breaking change will cause existing customers. Do you have any concerns if we roll this out? Do we need to carefully communicate this change in the next engine release? Anything else we could/should do to mitigate any disruption?

I don't think it would be a huge impact otherwise I would have thought I see more activity in the forums and ticket about animation speed

Most state graphs will be fine so I think we just have to be clear in the release notes which specific usages of these properties may require adjustment.


Add event to track node expansion errors

This will help us diagnose how often these issues happen and help determine if changes impact node expansion.
To start with this is just tracking timeout since there isn't an easy way to tell the full set of errors that may occur. I plan on following up with looking at adding an optional errorType property to the expand response but want to think about that a bit more so getting this in now so we at least have something.
Note that this is only for the MSSQL provider since we have full control over that. Later on we can add for all providers if needed, but we'll need to do some extra work to ensure we're not sending any personal information coming from a provider (the folder names)
Had to refactor the OE service a bit to get the node type down to the place where the expansion error is seen, no functional changes.

@alanrenmsft This look good to you?


Medtronic Max Bolus warning

It would appear that the Max Bolus directive on Medtronic pumps (Tested on 754 and 722) does not accept values higher than 25 over the remote controller OR a bug in Trio prevents it from being sent. (Anything over 25 seemingly demands you manually change it on the pump, where a difference is that you must acknowledge a warning that the value you're attempting to set is "unusual".)
Trio, however, pretends the value gets set when you choose "Save on pump" in the Pump Settings menu. It does show it didn't actually get set when you go out and back in to Pump Settings.
I think it may be beneficial to add a warning (or flat-out refuse anything higher than 25 with an explanation why) whenever someone attempts to save a Max Bolus value higher than 25 on Medtronic pumps.
-edit-
This leads to another problem: even after having set the value to 35 on the pump manually, Trio still thinks its 25. Attempting to save 35 in Trio simply doesn't stick. So the pump is now on 35, but Trio on 25 - which limits the maximum bolus that can be administered.
-edit2-
Edited the json file for pump settings to fix the issue in the edit above this edit. This edit to the config does seem to stick. (So manually edit on pump, then edit in config json)

Interesting. Do you know if AAPS or Loop or iAPS handles this better?
Looks like Max Bolus on x54 can be manually set as high as 75U, but I think all other Medtronics tap out at 25U?

@MikePlante1 Not sure. I never tried to set it higher then 25 before as I thought it was a viable safety limiter. (Turns out it isn't enough anymore -_-) Doubt iAPS has a fix, never heard anyone complain about this before.
I'm not sure about the x22 versions. As those contain smaller vials, it wouldn't surprise me if they can't go as high. Will look around :)

AAPS docs just says

The maximum value permitted by the pump is 25.

@MikePlante1 For the 7xx varieties? Both 754's I have here allow up to 75U to be set. It just gives a warning that it's "unusual". :P

@LiroyvH Could you give Loop a try to see if it handles things differently? I only have a 722 myself for testing.

@MikePlante1 The 722 only does 25?
I can test it once my other 754's keypad has dried up. :( It went swimming yesterday. In a watertight bag. Which turned out wasn't watertight. Can probably test this somewhere in the weekend. (Because if it still shorts the keyboard then, I'll have to open it up and fix it :P)

I can test this tonight with both 715 and 754.

Tested on Loop 3.3.0 - the picker doesn’t allow selection of values greater than 25U.
iAPS (2.3.x dev Dan’s branch) and Trio both allow choosing a value higher than 25U, but that value is not set on the pump, presumably because the pump requires confirmation of the value as Liroy has identified. The value isn’t retained after you exit the pump settings screen.
if you set a value over 25 on the pump, and edit the setting in Trio to a different value still over 25, no change is made to the value set on the pump. Once you exit the pump settings screen, no value over 25 for max bolus seems to be stored, the UI retains the last saved value below 25. This is true even if you set the max bolus in Trio to the same value you have manually programmed in the pump.
If you manually program a value over 25 in the pump, then save a value under 25 in Trio, the correct value is also saved on the pump.
When you set a maxbolus of 25 or below, log.txt includes both “setmaxbolus” and “readsettings”, which returns the values saved on the pump. However when you try to set a maxbolus over 25, nothing seems to be added to log.txt.
I don’t know how many people use a max of greater than 25, it seems likely a bit of an edge case given that neither Loop nor iAPS 2.3.3 appear to allow you the set a value this high?
Is setting an upper limit on max bolus of 25U a good enough solution?

Sounds like we need to enforce a max 25 U bolus for the relevant pump models?

Interestingly, the Omnipod DASH and Eros manuals say max bolus can be as high as 30U.

Exactly, so we probably don’t want to limit pods as well.

Loop uses the per-pump settings to determine maxBolus. See links below.
For pods, it is 30 U (600 pulses).
For Medtronic, it could be adjusted by pump model (as basal rates are), but all pump models are limited to 25 U.
Looks like DanaKit is also 30 U.

https://github.com/LoopKit/OmniBLE/blob/dev/OmniBLE/PumpManager/OmniBLEPumpManager.swift#L1572-L1576
https://github.com/LoopKit/OmniKit/blob/main/OmniKit/PumpManager/OmnipodPumpManager.swift#L1529-L1533
https://github.com/LoopKit/MinimedKit/blob/main/MinimedKit/Models/PumpModel.swift#L106-L108

Worth keeping in mind that the bolus rate of these pumps is only 1.5U/min. For 25U bolus, that's 16 min 40s or more than three full loop cycles. How often is someone going to want to ignore three or more updated CGM readings and just keep bolusing away? I don't recall ever seeing a complaint on @Looped or @LoopandLearn fb groups from someone wanting to set a higher maxbolus, so limiting it to 25 is probably fine. Updating the docs to say "If you wish to deliver a bolus greater than 25U, you can trigger this on the pump" might be nice and low-effort.

Why would you hard limit this when there are clear usecases when people need more...? That there aren't many complaints (or maybe some people simply haven't noticed because the app pretends it actually get's saved.) is imho not a very strong reason to make the app unusable or unnecessarily hard/annoying to use for the users whom do need far bigger boluses than 25U.
Not sure why you should care about missing loop cycles either. You know the pump will be busy for a while. :)
I find it very odd that we'd lazily put a cap on this to make it easier to manage rather than doing it properly for the pump models that support higher boluses. I honestly don't find it a good solution to put a hard cap on 25U. With the same reasoning we'd re-introduce the COB calculation caps and denying users with an I:C lower than 1:3 access to a working app. Surely that's not a desired direction to go in.

Why would you hard limit this when there are clear usecases where people need more...? That there aren't many complaints (or maybe some people simply haven't noticed because the app pretends it actually get's saved.) is imho not a very strong reason to make the app unusable or unnecessarily hard/annoying to use for the users whom do need far bigger boluses than 25U. Not sure why you should care about missing loop cycles either. You know the pump will be busy for a while. :)
I find it very odd that we'd "low-effort" put a cap on this to make it easier to manage rather than doing it properly for the pump models that do support higher boluses. I honestly don't find it a good solution to put a hard cap on 25U, sorry. With the same reasoning we could re-introduce the COB calculation caps and denying users with an I:C lower than 1:3 access to a working app; it was way simpler not to support it at the time. Surely that's not a desired direction to go in. Keep in mind there's also T2D's getting more interested in loop solutions with access to CGM's and pumps.
I'd strongly prefer an outcome where if its possible to support higher boluses because the pump supports it: it's supported. Limit it to the maximum of the pump model, don't put a 25U cap on it "just because that's easier". ;) And that's not only because I so happen to be one of those users who really doesn't want to have to bolus from his pump every time I need more than 25U in one-go; which can occur multiple times a day. I find it a matter of principle as well. Patients (T1/T2/LADA/MODY and all sorts in between) come with all kinds of requirements of their own, one patient isn't the other. Whilst you reasonably cannot always cater to all wishes/needs: something relatively simple such as this is the kind of low-hanging fruit that ought to be addressed rather than be dismissed imho. There's multiple users with I:C < 2 that'll very easily run in to the 25U cap and would prefer not having to grab their pump instead of conveniently bolusing from their phone which, in addition, gives the proper recommendation.

Just to update/clarify what I find the ideal solution:

Most importantly: do not impose a master limit of 25U, impose the limit based on the pump model.
On supported Medtronic models, when the user attempts to set a value higher than 25U: warn them this cannot be set through the app, but that it can be set directly on the pump and that they should do so. Ask them to confirm they've set it or to cancel. (If they confirm: update MaxBolus to the desired amount. (If the user lied and didn't set it: pump will error out when attempting to bolus and Trio will throw error that Bolus failed - so that's not a problem.).)
And of course if the user has a Medtronic model that allows only up to 25U: then that's the upper limit, simple as that.
If, just as suggested in #266, there's a method to ask the pump what the value is currently set to: this can be used as an extra verification. If the user in the suggested flow above taps "confirm" that they've set the new MaxBolus in their pump, pumpManager will query the pump to verify if it's true. If it isn't (eg: Max Bolus is higher/lower than what the user gave as input): error out. If it is: splendid, save the value as new MaxBolus-limit.
(- Bonus: I wonder if it's ideal to query the pump for such settings from time to time and warn if there are mismatches.).

And for Omnipod: simply allow the user to set up to 30U if that is its limit. Higher than 30U shall not be allowed as Omnipod doesn't support it.
And for Dana: I suppose a warning should be added that the MaxBolus value cannot be changed through the app. (But tbh, #266 is still unclear to me so I still don't understand what the check on the pump is for if it always throws other values than what is set.)

tbh, https://github.com/nightscout/Trio/pull/266 is still unclear to me so I still don't understand what the check on the pump is for if it always throws other values than what is set.)

PR 266 restores the Pump Setting page to the original value if the save to pump fails, otherwise it reports what was saved to the pump.
It is better than not having it, but more work is needed in the future.
Without PR 266 - if you try to save an invalid value to the pump, it pretends to save and page is left with the requested value; but if you exit the page and go back into Pump Settings you see that no change was made.
With PR 266 - after it comes back from the save tap, you can see on the page itself that the original value was restored.
Different pumps:

Omnipod - no concept of maxBolus, but 30 U is the max bolus the pod can support
Medtronic - I'm guessing the 25 U was set as the one and only maxBolus early on. Now that there are additional pumps supported with higher allowed maxBolus, this could be modified to handle a maxBolus per pump model. This would require work added to LoopKit/MinimedKit and a volunteer to do the work.
DanaKit - this is a read-only function for this pump. Moving forward, adding a pump without remote option to set maxBolus or maxBasal may trigger some higher level redesign to handle it.

I can understand why DanaKit provides selectable rate options for how fast to deliver a bolus with a maxBolus of 80 U.

Why would you hard limit this when there are clear usecases where people need more...? That there aren't many complaints (or maybe some people simply haven't noticed because the app pretends it actually get's saved.) is imho not a very strong reason to make the app unusable or unnecessarily hard/annoying to use for the users whom do need far bigger boluses than 25U. Not sure why you should care about missing loop cycles either. You know the pump will be busy for a while. :)
I find it very odd that we'd "low-effort" put a cap on this to make it easier to manage rather than doing it properly for the pump models that do support higher boluses. I honestly don't find it a good solution to put a hard cap on 25U, sorry. With the same reasoning we could re-introduce the COB calculation caps and denying users with an I:C lower than 1:3 access to a working app; it was way simpler not to support it at the time. Surely that's not a desired direction to go in. Keep in mind there's also T2D's getting more interested in loop solutions with access to CGM's and pumps.
I'd strongly prefer an outcome where if its possible to support higher boluses because the pump supports it: it's supported. Limit it to the maximum of the pump model, don't put a 25U cap on it "just because that's easier". ;) And that's not only because I so happen to be one of those users who really doesn't want to have to bolus from his pump every time I need more than 25U in one-go; which can occur multiple times a day. I find it a matter of principle as well. Patients (T1/T2/LADA/MODY and all sorts in between) come with all kinds of requirements of their own, one patient isn't the other. Whilst you reasonably cannot always cater to all wishes/needs: something relatively simple such as this is the kind of low-hanging fruit that ought to be addressed rather than be dismissed imho. There's multiple users with I:C < 2 that'll very easily run in to the 25U cap and would prefer not having to grab their pump instead of conveniently bolusing from their phone which, in addition, gives the proper recommendation.

I don't disagree with your ideal outcome, I just don't think it's likely to be a high priority because it comes up so infrequently. I've done a search on Looped FB group and I only found one post (https://www.facebook.com/groups/TheLoopedGroup/permalink/2975389479344365) from someone wanting to change this in the last 3 years. There are probably posts I missed, and more people who would like to but don't know it's possible, but the subset of loopers with I:C < 2 who also own one of the pumps that can do this must be small, and getting smaller all the time.

Let's assume that's true, though I don't think it is as the Dana pumps support values (muuuuch) higher than 25U as well (And Omni does 30) - so if the group has been getting smaller due to declination of Medtronic use: it'll likely grow again with the addition of modern pumps (Dana, Medtrum), then there's still a very big difference between adding proper error handling and adding a hard-cap of 25U which causes the people whom do need the larger boluses (and can at least manually work-around it at the moment) to get in to major trouble operating the app in a satisfactory way.

I think the bottom line is that this Issue

should be renamed to Allow more general pump dependent settings for maximum bolus and basal rates
should be completely decoupled from PR #266
if necessary, write a new Issue to explain why PR #266 is needed

Interesting. Do you know if AAPS or Loop or iAPS handles this better?

AAPS has limits set per pump, and an overall hard limits for certain things.
For Medtrum Development it will even be different per Medtrum Model (Nano, 300U and future 400U have different bolus and TBR allowance) Ultimately my preference is that we would specify limits in the pump plugins
Reference:
https://github.com/nightscout/AndroidAPS/blob/3df83535f9f8f3246308121900c2785c50915fa8/core/interfaces/src/main/kotlin/app/aaps/core/interfaces/pump/defs/PumpType.kt#L414
https://github.com/nightscout/AndroidAPS/blob/3df83535f9f8f3246308121900c2785c50915fa8/core/interfaces/src/main/kotlin/app/aaps/core/interfaces/pump/defs/PumpCapability.kt#L30
Reading the code againI realize the max bolus on AAPS could also improve but different topic


Set allowJs and checkJs to false

Ticket
resolves #20
Changes
Simple change to change two tsconfig.json settings, namely allowJs and checkJs, to false.
Context for reviewers
Context in the linked issue, but in summary, allowJs isn't necessary for a template since the template is likely to be used to build something from scratch without any legacy js code.
Testing
Tested by running npm run test, npm run build, and npm run dev, and they all worked as expected.

I tried testing this by creating a .js file and importing it into _app.tsx. When I run the type checks (npm run ts:check) I'm not getting any type errors:

@sawyerh I just tried with the same test that you mentioned, but mine does get an error:
➜  app git:(jimmy/disallowJs) ✗ npm run ts:check

><EMAIL_ADDRESS>ts:check
> tsc --noEmit

src/pages/_app.tsx:6:18 - error TS7016: Could not find a declaration file for module '../components/test'. '/Users/jimmyfagan/git/navapbc/template-application-nextjs/app/src/components/test.js' implicitly has an 'any' type.

6 import test from "../components/test";
                   ~~~~~~~~~~~~~~~~~~~~

Found 1 error in src/pages/_app.tsx:6

When I reverted to allowJs and checkJs true, it runs cleanly. Any other environmental factors that might be coming into play? Down for a quick huddle to compare each other's setup to get to the bottom of this, just let me know.

@jimmyfagan ah right, I got that too and temporarily set the strict field to false but now I'm realizing that is the error for allowJs. I was expecting an error more like "You can't import .js files" instead of "Could not find a declaration file". My bad, sounds like this is working as expected.

All good, I appreciate you testing things to make sure. I got that same error when building, but didn't think to use ts:check, so it was a useful exercise double checking some of that.


Upgrade bootstrap-sass 3.3.7

Coverage remained the same at 100.0% when pulling 2021095a364d9d21db8bfa2522798357ad949782 on feature/bundle-update-bootstrap-sass into 6241030a9651de98087de1656c6cf802e9af7adf on master.


BLIP-2 / Q-Former / Benchmarks

Hi,
the ViT and the Q-Former are frozen. What Q-Former Model is being used by TinyGPT-V? BLIP-2 has ha big zoo of models.

https://huggingface.co/Salesforce/blip2-opt-2.7b
https://huggingface.co/Salesforce/blip2-opt-6.7b
…

Furthermore what models were used to benchmark against blip-2?
Is there a source for the benchmark harness?
Thank you

Hello and thank you for your question.
In the results section of the paper, we cite results from other models in the MiniGPT report (including BLiP-2), and our Vision Backbone and MiniGPT-4 frameworks remain consistent.
For more detailed evaluation setups, you will need to consult the MiniGPT-4 or v2 papers in https://github.com/Vision-CAIR/MiniGPT-4.

Thanks. I found it in the code. blip2_pretrained_flant5xxl.pth is currently being used.
https://github.com/DLYuanGod/TinyGPT-V/blob/f433ff914ca457ec94be8aa0346f794b32be95a9/minigpt4/models/minigpt_v2.py#L70


feat: typography update

Descrizione
This pull request simplifies the typography sizes calculation.
What's changed

reset base font size at 16px on body
replaced em units with rem
updated font-size values to match the new rem unit calculation
removed obsolete and unused variables from _variables.scss partial
updated components specific variables to match UI kit v3.0

Checklist

[x] Le modifiche sono conformi alle linee guida di design.
[x] Il codice è coerente con le indicazioni di progetto.
[ ] Le modifiche sono state verificate sui Browser supportati e per diverse risoluzioni dello schermo.
[x] Sono stati effettuati test di accessibilità in ottemperanza a quanto descritto nell'area "Accessibilità" delle linee guida di design.
[ ] La documentazione è stata aggiornata.

@Fupete @astagi PR is ready to merge, please check the changes for approval or let me know if I could close on my own

@Fupete @astagi PR is ready to merge, please check the changes for approval or let me know if I could close on my own

@zetareticoli this is a huge PR with too many possible implications of 1) visual rendering (see effect already on documentation) and 2) backward compatibility to do separate reviews without talking about it first in my opinion. I would say let's have an ad hoc meeting us three (cc @astagi) to start the review together, please.

@Fupete @astagi PR is ready to merge, please check the changes for approval or let me know if I could close on my own

@zetareticoli this is a huge PR with too many possible implications of 1) visual rendering (see effect already on documentation) and 2) backward compatibility to do separate reviews without talking about it first in my opinion. I would say let's have an ad hoc meeting us three (cc @astagi) to start the review together, please.

Sure, let'go meeting
But consider that BS docs has its own CSS files and classes which overrides the default typography declarations.
Example:
.bd-lead {
  font-size: 1.125rem;
  font-weight: 300;
}

I would say consider removing all special bd- classes on typography for consistency. Let's talk about it live.

@Fupete this PR waits your final approval review :)

@zetareticoli To me it looks like the typography part is fine. There are many changes in this PR, so let's be ready to quickly fix any inconsistencies after the merge.
@astagi maybe you need to remove Gemfile.lock from the equation 🤔 before merging?

I think so. Let me check it locally. Thanks @zetareticoli


DCOS-43777: provide default volume type for DSS volumes

↩  This PR back-ports the fix introduced with #3920 to release/1.13 .

Please see https://github.com/dcos/dcos-ui/pull/3920 for instructions on how to test the resp. changes.

Restarted CI.

1 review required for backports, someone with admin privileges can merge it.

:tada: This PR is included in version 2.82.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Can't use x.x.1 schemaVersion accepted by schema that is x.x.1 codified

Environment

Application Services Version: 3.36.1
BIG-IP Version: 14.1.4

Summary
Can't use x.x.1 schemaVersion accepted by schema that is x.x.1 codified
Environments that have pipelines that are fed (choose?) the schemas that have a x.x.1 notation can be schema checked properly but can't be used as a consistent value for pipelines like:

Drop in the RPM or Monitor the latest AS3 release
Monitor device /info for matching schemaVersion or update the device
Assert $schema in json declarations
Use schemaVersion as fed/chosen in declaration which is accepted by Schema.

Difficult to stay programmatically up to date without codifying additional logic to handle mismatched schema tolerance.
Steps To Reproduce
Steps to reproduce the behavior:

Submit the following declaration:

{
  "$schema": "https://raw.githubusercontent.com/F5Networks/f5-appsvcs-extension/master/schema/3.36.1/as3-schema.json",
  "class": "AS3",
  "action": "deploy",
  "declaration": {
    "class": "ADC",
    "schemaVersion": "3.36.1",
    "Common": {
      "class": "Tenant",
      "remark": "Shared Objects",
      "Shared": {
        "class": "Application",
        "template": "shared",
        "site_maintenance": {
          "class": "iFile",
          "iFile": {
            "base64": "U29ycnkhIFVuYXZhaWxhYmxlIGR1ZSB0byBtYWludGVuYW5jZS4gIFBsZWFzZSB0cnkgeW91ciByZXF1ZXN0IGxhdGVyLg=="
          }
        }
      }
    }
  }
}

Observe the following error response:

{
  "code": 422,
  "errors": [
    "/schemaVersion: should be equal to one of the allowed values [\"3.0.0\",\"3.1.0\",\"3.2.0\",\"3.3.0\",\"3.4.0\",\"3.5.0\",\"3.6.0\",\"3.7.0\",\"3.8.0\",\"3.9.0\",\"3.10.0\",\"3.11.0\",\"3.12.0\",\"3.13.0\",\"3.14.0\",\"3.15.0\",\"3.16.0\",\"3.17.0\",\"3.18.0\",\"3.19.0\",\"3.20.0\",\"3.21.0\",\"3.22.0\",\"3.23.0\",\"3.24.0\",\"3.25.0\",\"3.26.0\",\"3.27.0\",\"3.28.0\",\"3.29.0\",\"3.30.0\",\"3.31.0\",\"3.32.0\",\"3.33.0\",\"3.34.0\",\"3.35.0\",\"3.36.0\"]"
  ],
  "declarationFullId": "",
  "message": "declaration is invalid"
}

Expected Behavior
Using a schema version that is accepted by the schema, AS3 engine should permit schemaVersion accepted by schema
Actual Behavior
Error as noted. Declaration is not processed.
Fix Idea
Adjust the schema tolerance in engine?
Don't use x.x.? version model?

Resolved in AS3.49.0. Please review the release notes and upgrade. Thank you!


Fix interop with bundled CJS dependencies

We bundle several dependencies into the dist/index.mjs file. Previously, the build was CJS and everything "just worked" but now that it's ESM-only some top-level constants like __filename and __dirname are not defined and esbuild doesn't handle that for us :( Additionally, require.cache isn't passed through to createRequire(…).cache so we do that as well.
I think in an ideal world world we'd somehow split all bundled CJS dependencies into a separate .cjs file to allow everything to be handled by node itself. I'm not sure what that looks like at the moment though.
Fix for #198 (I hope 😅)

This PR probably fixes multiple issues:

https://github.com/tailwindlabs/prettier-plugin-tailwindcss/issues/201
https://github.com/tailwindlabs/prettier-plugin-tailwindcss/issues/198

@thecrypticace It would be great for this package to bundle both cjs and esm to allow everybody to use it.

@ohkimur This is not possible because of our need to use top-level await to handle plugin compatibility. Prettier itself does not expose async APIs in all the necessary spots unfortunately.


Can't reopen theme picker once you close it

When on https://syntax.fm/system/theme, if you close the theme by clicking the X or clicking the Theme link again, once the theme picker is closed you have to leave the route and go back to show it again. The keyboard shortcut still works though.

fixed!


disappearing entries to stderr and stdout on image public.ecr.aws/lambda/go:1

I'm using "public.ecr.aws/lambda/go:1" image for my golang lambda function. When I write to stderr log.Println("...") and to stdout fmt.Println("...") nothing appears in CloudWatch.
I only see these three entries in CloudWatch:
START RequestId: 1e2ef1a9-6a82-48fa-af65-5e27cde1889b Version: $LATEST
END RequestId: 1e2ef1a9-6a82-48fa-af65-5e27cde1889b
REPORT RequestId: 1e2ef1a9-6a82-48fa-af65-5e27cde1889b	Duration: 1.66 ms	Billed Duration: 158 ms	Memory Size: 128 MB	Max Memory Used: 30 MB	Init Duration: 155.81 ms

I used my image and can write to stderr and stdout again.


using the md-button keeps the remains of unknown hover effect

I had a problem while studying Matelial 2.
using the md-button keeps the remains of unknown hover effect
Using the buttonsA(md-button), the alert(md-card) was executed.
The md-button implemented in the executed alert has a gray hover effect when no action is taken.
Also,When this alert is closed, The gray hover overlaps with the button A.
The generated hover disappears when you click anywhere
It was thought that the cause of the hover was related to the ripple.
But setting up ripple did not solve the problem.
I am curious as to how to solve this problem.
I using macOS(sierra), Angular2 material

can you provide a plunk demonstrating your problem?

Closing this issue as it does not follow the issue template. Please file a new issue that follows the template, which gives the team the information needed to investigate.


global: fix import error

Same as:

pytest-invenio #42
invenio-oauth2server #196

https://travis-ci.org/inveniosoftware/invenio-oauth2server/jobs/656679343
Warning, treated as error:
autodoc: failed to import module u'ext' from module u'invenio_oauth2server'; the following exception was raised:
cannot import name cached_property
The command "./run-tests.sh" exited with 2.

Fix also deprecation warning in Travis config:
https://travis-ci.org/inveniosoftware/invenio-oauth2server/jobs/656679343/config

currently cannot upgrade flash_oauthlib to support werkzeug, as the module is deprecated


Add multi-lingual markdown component

I think we should drop the develop branch, so change this to merge to master instead

I think we should drop the develop branch, so change this to merge to master instead

It didn't really work out. Azure Static Sites are still in a little to much beta. So develop stays for a while


sleep: move from getopts to clap #1735

Fixes #1735.
This request replaces getopts with clap for the sleep binary.
It does not introduce tests.

Excellent, thanks :)


Making a diagram - ethics decision tree

this diagram will help to visualize when an ethical application is needed

We tried out Mermaid, but it turned out a bit ugly...

Since we already have a tree on Miro, we decided to use the Miro version for now. I'm working on a PR.


x/build/cmd/gomote: cannot push VERSION file

When I gomote push from a tree that has a VERSION file, the VERSION file does not end up on the gomote.
% gomote push $vm
2022/12/03 15:41:27 installing go1.4
2022/12/03 15:41:41 Remote doesn't have "src/go/internal/gcimporter/exportdata.go"
2022/12/03 15:41:41 Remote doesn't have "src/runtime/testdata/testprog/sleep.go"
2022/12/03 15:41:41 Remote doesn't have "src/runtime/testdata/testprog/syscalls_none.go"
2022/12/03 15:41:41 Remote doesn't have "src/runtime/trace/trace_stack_test.go"
2022/12/03 15:41:41 Remote doesn't have "src/syscall/exec_aix_test.go"
2022/12/03 15:41:41 Remote doesn't have 12036 files (only showed 5).
2022/12/03 15:41:41 Remote lacks a VERSION file; sending a fake one
2022/12/03 15:41:48 Uploading 12037 new/changed files; 79017606 byte .tar.gz
...

From the messages, you'd think that the fake VERSION is sent before the upload, meaning the upload should overwrite it. But 'gomote ssh' to the VM and printing the VERSION file shows the devel gomote.XXXXX content and not the one I have locally (which says go1.99).
Maybe there's an ordering issue, I thought, so I ran gomote push again. That definitely sends the VERSION file:
% gomote push $vm
...
2022/12/03 15:43:39 Remote's VERSION digest is "06e08a5ba98f3aa9496d990cc86f4a96fde9687d"; want "f51724a3e09407fd330585a259fd89c12e40e159"
...
% echo -n 'devel gomote.XXXXX' | openssl sha1
06e08a5ba98f3aa9496d990cc86f4a96fde9687d
% echo go1.99 | openssl sha1
f51724a3e09407fd330585a259fd89c12e40e159
%

And yet, it remains unchanged on the server (confirmed with 'type VERSION' in ssh). It is as though the buildlet discards uploaded VERSION files. This is on the windows-amd64-2016 gomote, in case the specific buildlet matters.
I have been working on reproducible builds, which are only possible in non-devel trees, so I need to send a VERSION file. I have resorted to running a batch file that echoes the right content into the VERSION file before running the command I want to run. So I'm not blocked, but it was a surprising thing to debug in the first place.

It is as though the buildlet discards uploaded VERSION files.

I think that may happen here?
https://cs.opensource.google/go/x/build/+/master:cmd/gomote/push.go;l=472-481;drc=3e9a6d306718ba1bf48b2b1ca07a8194f41d54d7


Start client with windows logon

Option to turn on auto start on or off for the Desktop agent.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/28049855-start-client-with-windows-logon?utm_campaign=plugin&utm_content=tracker%2F19476668&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F19476668&utm_medium=issues&utm_source=github).

Closed via #116


Feature: Driver - duckdb

Driver for duckdb
Document:
https://github.com/Canner/vulcan/tree/feature/driver-duckdb/packages/extension-driver-duckdb
I deleted the extension for lab env because it can use this driver instead.

Codecov Report
Base: 92.59% // Head: 92.64% // Increases project coverage by +0.05% :tada:

Coverage data is based on head (75842ab) compared to base (59d9e37).
Patch coverage: 95.91% of modified lines in pull request are covered.

:exclamation: Current head 75842ab differs from pull request most recent head 21d7293. Consider uploading reports for the commit 21d7293 to get more accurate results

Additional details and impacted files
@@                   Coverage Diff                    @@
##           fix/response-streams      #43      +/-   ##
========================================================
+ Coverage                 92.59%   92.64%   +0.05%
========================================================
  Files                       222      224       +2
  Lines                      3051     3099      +48
  Branches                    351      359       +8
========================================================
+ Hits                       2825     2871      +46
- Misses                      165      167       +2
  Partials                     61       61

Flag
Coverage Δ

build
94.87% <ø> (ø)

cli
91.90% <ø> (ø)

core
93.38% <50.00%> (-0.12%)
:arrow_down:

extension-dbt
97.43% <ø> (ø)

extension-debug-tools
98.11% <ø> (ø)

extension-driver-duckdb
100.00% <100.00%> (?)

integration-testing
95.00% <ø> (ø)

serve
88.78% <ø> (ø)

test-utility
∅ <ø> (∅)

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

Impacted Files
Coverage Δ

packages/core/src/lib/utils/logger.ts
56.09% <0.00%> (-1.41%)
:arrow_down:

packages/core/src/models/extensions/base.ts
86.66% <50.00%> (-5.65%)
:arrow_down:

packages/core/src/models/extensions/dataSource.ts
100.00% <100.00%> (ø)

packages/extension-driver-duckdb/src/index.ts
100.00% <100.00%> (ø)

...xtension-driver-duckdb/src/lib/duckdbDataSource.ts
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Hi @kokokuo , all issues have been fixed.


[BUG]Alphaj becomes negative, giving error in Wilson bootstrap routine

This was observed by Jon in #213, and leads to error 76 in bootstrap_fraction_wilson. This is because iprofile=1 and qstar has dropped below q0 in routine culcur; so alphaj becomes negative, eventually leading to aj being NaN. I suggest that alphaj is prevented from becoming negative at the point at which it is set in routine culcur:
        if iprofile == 1:
            # Ensure current profile consistency, if required
            # This is as described in Hartmann and Zohm only if icurr = 4 as well...

            # Tokamaks 4th Edition, Wesson, page 116
            alphaj = qstar / q0 - 1.0

Note that the Wilson bootstrap routine is always run, even if it is not selected in the input file:
https://github.com/ukaea/PROCESS/blob/7e15d396f3990df10e8745882e2c76ee04ac0af2/process/physics.py#L921
James claimed to have fixed this problem but it seems to have come back.
Steps to reproduce
Pirizad IN.DAT.txt
Expected behaviour
alphaj should remain positive somehow.
Evidence

This was something I cam across doing #3263. The definition for alphaj above is from Wesson and assumes a circular cross-section plasma using $q_{cyl}$ or qstar in this case. Which is hardly a case used for normal PROCESS runs.


Add 'graph teams user remove' command capability solving #757

Add 'graph teams user remove' command capability solving #757

@waldekmastykarz As requested; the call now retrieves all owners. If the passed user is an owner we execute the appropriate remove call, if the user is not an owner we assume its a member and execute that call. Let me know what you think

Will have a look. Thanks! 👏

Merged manually. Thank you! 👏


remove dependency on mint for HKDF

mint exposes a HKDF extraction and expansion methods, which we used in our crypto package. Tris doesn't expose those methods, and it's just around 40 LoC (which can easily be tested using the cases provided in the HKDF RFC, so it makes sense to copy those to quic-go.

Codecov Report

Merging #1522 into master will decrease coverage by 0.09%.
The diff coverage is 55.56%.

@@            Coverage Diff             @@
##           master    #1522      +/-   ##
==========================================
- Coverage   89.22%   89.12%   -0.09%
==========================================
  Files         122      123       +1
  Lines        8616     8624       +8
==========================================
- Hits         7687     7686       -1
- Misses        629      635       +6
- Partials      300      303       +3

Impacted Files
Coverage Δ

internal/crypto/null_aead_aesgcm.go
100% <100%> (ø)
:arrow_up:

internal/crypto/hkdf.go
50% <50%> (ø)

session.go
71.76% <0%> (-0.68%)
:arrow_down:

Continue to review full report at Codecov.

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

Is there a reason why we can't use https://godoc.org/golang.org/x/crypto/hkdf ?

The standard library hkdf package doesn't expose an extract and expand function.


Why not use the libllvm shipped with rustc?

On Linux for example you can find it at $(rustc --print sysroot)/lib/rustlib/x86_64-unknown-linux-gnu/lib/libLLVM.so. This would save users the effort and time of compiling llvm themselves.

There are a few reasons. Rust uses its own fork of LLVM, and typically lags behind mainline (they only recently started upgrading to LLVM 9, not sure when that will make it into nightly). The larger issue is that the libLLVM.so shipped with Rust isn't necessarily the same as any other libLLVM.so, as that shared object contains whatever components Rust was built with, with the configuration they used to build it. We want control over what components we ship with (which does in fact include some things that Rust does not ship), as well as how those components are built, and not be tied to whatever version and patches that Rust requires.
When the compiler backend gets merged, we can review and see if it can be used as a fallback option, but since LLVM is only required to build the compiler backend (and any components that depend on it, which will ultimately only be the lumen executable), I don't consider it to be unreasonable to have contributors to those components install LLVM. Especially since with proper set up, subsequent builds after the initial first-time build can be very fast. I also have a helper script to make installing/updating extremely easy to do for those that aren't familiar with LLVM and don't otherwise wish to manage it themselves.
For those that aren't working on components that directly depend on LLVM, they shouldn't need to install it at all for working on patches/running tests, only for building the full compiler. This is where we could potentially support the Rust LLVM as a fallback if a "proper" LLVM install is not present - but I suspect that isn't going to be an option due to the lack of components we depend on (namely the linker bits).
I'm going to close this for now, but I will keep it in mind as the codegen bits get wrapped up, and if it is possible, I will make it a fallback option for easier project set up :)


[MNOE-474] Refresh reviews frontend

Rebase on top of https://github.com/maestrano/mno-enterprise/pull/349 - Replacing https://github.com/maestrano/mno-enterprise/pull/327
@adamaziz15  FYI

@x4d3 Could you rebase?
You probably rebased #327 after pushing this one

nvm I did it

I've reverted the merge.
Some of these changes need to go in v3.3


test: activate tests on contrib/raftexample

adds contrib/raftexample to test and fixes a race found by --race

ok everything is passing PTAL /cc @xiang90

LGTM!


Global Secrets Management

Description/Context
secret-operations is a .... disaster. We need a managed space for our secrets that are shared between applications / stacks / whatever.
Plan/Design

Create a substructure stack that makes a kv2 vault mount and populates it with various keys / data structures that will be shared between applications.
Pick a few small apps / stacks and migrate them

That means tracking down the existing values in secret-operations and ensuring they get populated into the new mount via pulumi code.
Update the application stacks to retrieve secrets from the new mount and not the old mount.
Make sure to update the .hcl files for the stacks so they have permission to access the new mount.

Questions

What, if anything should be exported from the substructure stack? Maybe paths to the root keys / groups
Root keys / groups. Come up with a sane orgnaizsation system for the root keys /objects /paths in the new vault mount and make sure it is sane with the rest of the team.

@blarghmatey Now that the mount and a sample app (bootcamps) is complete, should we go ahead and close this issue and open separate issue to update pillar for the individual apps?

Yes, I think it makes sense to have an issue per app so that we can pick it off opportunistically.


Indexed Slices Support

Hi,
I am wondering what the reasoning behind IndexedSlices is and whether they are actually necessary. My understanding is that they are only constructed for the gradient of the gather op. Is there any other place in the Python API that IndexedSlices are being constructed? Now, even though they are only constructed there, there are special cases throughout the Python codebase for dealing with indexed slices. Is the performance benefit so important to justify all this special treatment code? I am asking because I am wondering if it is useful to implement that functionality in an API for a different language.
And if I am to rephrase this question, should one put effort into adding support to the C++ API for indexed slices, or is their use a remnant of a design choice that is not that useful looking in retrospect?
Thank you!

This question is better asked on  StackOverflow since it is not a  bug or feature request. There is also a larger community that reads questions there. Thanks!


Default behavior of Models when IdName is not set and no id field in table

Say I have two models User and Comment, where the Comment model is set to be a child model of the User model.
Now, I queried a User object user and from it, I got comment, the first comment associated with user.
I then made some changes to comment's content field, then finally save the changes.
However the primary key of the Comment table is myId and I did not specify this in the Comment Model class.
Is it really the case that the contents of all comments will be updated silently, including those not linked to the given user ? If so, why allow this default behavior?
I understand that it was my mistake to not specify the custom id field. I know maybe using the default id name will lessen my problems.
However, I think it will be much helpful and hurt less if an exception is thrown to notify that something is a miss. This can prevent something really bad and unexpected (at least in my case)  to happen.

apparently you did not provide any information to the model what the ID is for the COMMENT table. If you have your ID as myId, what prevents you from specifying it? I personally do not see this as a bug.

Just to set the record straight before I close the issue. It was not a bug. It was my fault. But let me share my experience. I was using the library to create a tool for a db of some legacy system, whose structure I cannot modify. As it turns out one of the table has an unusual primary key field name. I created a model for it and forgot to specify the id name in the model. I used the model to query a record and made changes but some other records got updated as well. I checked all my queries but they turned out to be fine. I realized that I forgot to specify the id field and after that everything went well. But still the question remains, why will a change in a single record through a model affect all other records? So that is why I created this issue.
Now I tried to replicate the scenario with a table I created on my own but I cannot replicate the scenario. As it turns out, if your db has a different primary key field and you did not specify it in the model and you save the changes to an existing record an Insert opertion is invoked instead of an Update and some exception will occur related to duplication of primary key. So why is this happening in my table and not on the table of the legacy db?
Well as it happens, the table has an id field which I somehow did not see at first, since it is not important to what I am doing. The id field is not the primary key nor a unique key, so multiple records have the same value. And this is why I have this issue.
So there you go. It was my fault and I learned my lesson. I firmly believe that following a good db naming convention will prevent things like this in the first place, but I also think it will be helpful if activejdbc can check first if the id field it uses by default is primary or uniqe and log some warning in case it isn't. Well I guess this is one of those corner cases I hope others will not encounter in production code. Anyway, cheers to the team and thank you @ipolevoy for your help.

@rlvillacarlos, thanks for a great explanation! Now this is making more sense. This page: http://javalite.io/surrogate_primary_keys explains:

The special meaning of the id attribute
The logic of insert vs update
How to use composite primary keys.

So, if your table has an an unusual primary key field name, you can use the annotation @IdName to tell the model what it is.
In any case, this was educational for me too, thanks for writing.


add feature to search with Ctrl + k

PR Fixes:

Added Ctrl + K to the search box

Before:
https://github.com/user-attachments/assets/52422b91-9434-496a-89b9-a3f5bcc7af95
After:
https://github.com/user-attachments/assets/b185088e-abed-4e27-9b30-2673c3062d99
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there is no similar/duplicate pull request regarding same issue

@hkirat
@manuarora700 asked for this feature in your resume roasting video
So here it is...

@siinghd @devsargam

Can u implement this beautiful library? https://cmdk.paco.me/

Other than that pr is good

Shadcn commandk internally uses cmdk package. Might as well use that
On Sun, 15 Sep 2024 at 12:08 AM, Sargam @.***> wrote:

Other than that pr is good
—
Reply to this email directly, view it on GitHub
https://github.com/code100x/cms/pull/1205#issuecomment-2351098588, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFRSXFKMBQURMXVJZJSHQ2TZWR7B3AVCNFSM6AAAAABODBC7IOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJRGA4TQNJYHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@devsargam added shadcn commandk component

@devsargam any other feedback?

Let me check this pr by eod

Let me check this pr by eod

lol sorry sir

can u flix conflicts @AuenKr?

@devsargam resolves the conflict.
But seems like #1270 is also adding this with other features also.

@devsargam, any other change required?

Let me rereview it

@devsargam ??

yeah yeah it's all good https://github.com/code100x/cms/pull/1270 just want to see what's this got

@devsargam
Do we need to change the user password from cms directly?
I made a PR earlier, but I don't understand why it was not continued.
#749
Shall I add this feature to the current codebase?

Thank u


Addition of more letters in Quick Accent feature.

Description of the new feature / enhancement
There exists more letters in the world that need accents other than provided. Some examples include:
c -> ċ
g -> ġ
h -> ħ
z -> ż
Could there be a configuration screen where these can be set?
Scenario when this would be used?
In the not so common cases where I would need to use my language in emails or texts, this would be very helpful instead of installing a new keyboard mapping or use a native keyboard.
Supporting information
No response

@p5yc0runn3r these characters have already been added, please update to the latest version and close the issue


Add warning if closing a terminal with a running process

I'm doing machine learning research, which often involves long running processes. If I run these in a bash shell (Ubuntu 18.04) and try and close the terminal I get a warning that I'm trying to close a terminal which contains a running process.
I don't get the same warning when I run a process in the vscode terminal, and there's been a number of times when I've accidentally closed either a terminal, or the whole vscode app and lost work. It would be nice if a (perhaps enabled with an option) warning to prevent this.

Related: https://github.com/microsoft/vscode/issues/23808

Not really related to that quoted issue. There's a setting for this:

When that setting is enabled, it asks confirmation. The issue you referenced @Tyriar is that it also asks for confirmation even if the terminal is doing nothing (a defect if you ask me).

@ericbf it's very much related because #23808 is about detecting if it's running a process and only warning then, this is about detecting a process is running and warning when closing a terminal (not the application).

Ah, I kind of skimmed over the "either a terminal" phrase. Yup, makes sense


Added test to playground to verify that issue #79 is fixed.

@chris-schuetz thx for your pull request.
Do you know if #79 is fixed? Or does this change only help to verify whether it is fixed or not?

#79 was already fixed, when I did take a look at it. I had created this commit to take a look at the issue but saw that I was unable to recreate the issue. Just wanted to share this, as a verification that the issue is fixed.


Add Melon to Third Party Projects page, remove projects without proof

Adds info about adding a project to this list
Also removes images, which looked out of place
Removed That Planning Suite since I don't think this is the right place to put a list of Aragon apps

@Smokyish  the protocol is called Melon, Melonport is a company that won't exist anymore very soon
https://medium.com/melonport-blog/introduction-to-the-melon-governance-system-f6ff73c70eb0


Please add the Airfilering option for the new generation lg airco

Please add de air filtering and air purifier option for the latest generation airco

What model is this? Could you provide a product URL to LG's website or link to a manual?

What model is this? Could you provide a product URL to LG's website or link to a manual?

Hello
Model type is AP12RT  Dualcool
https://www.lg.com/nl/air-conditioning/lg-ap12rt
Verstuurd vanaf mijn iPhone

Op 2 jan. 2021 om 23:34 heeft Stuart Olivera<EMAIL_ADDRESS>het volgende geschreven:

What model is this? Could you provide a product URL to LG's website or link to a manual?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hello
Model type is AP12RT  Dualcool
https://www.lg.com/nl/air-conditioning/lg-ap12rt
Verstuurd vanaf mijn iPhone

Op 2 jan. 2021 om 23:34 heeft Stuart Olivera<EMAIL_ADDRESS>het volgende geschreven:

What model is this? Could you provide a product URL to LG's website or link to a manual?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hello!
Same here. Please, add support for this model. Ask me if you need some help.
Some logs:
[16/03/2021, 18:46:06] [LgThinqAirConditioner] TargetHeaterCoolerStateCharacteristic: Warning: Your model may support a "drying" or "dehumidification" mode. This is NOT natively supported by Homekit, and using it may show errors in Homebridge or cause temporary instability. [16/03/2021, 18:46:06] [LgThinqAirConditioner] Starting refresh interval (set to 1 minutes) [16/03/2021, 18:46:07] [LgThinqAirConditioner] SwingModeCharacteristic: Error parsing state Error: Unsupported API value: 6 [16/03/2021, 18:46:07] [LgThinqAirConditioner] TargetHeaterCoolerStateCharacteristic: Error parsing state Error: Unsupported API value: 5

Interesting!
Could you provide a copy of the snapshot for your device when this happens?

Turn on "debug" mode for Homebridge (start Homebridge with -D or in the Homebridge UI, go to "Homebridge Settings" and turn on "Homebridge Debug Mode")
Restart Homebridge
Follow the logs and wait for ~60 seconds
Find the line that starts with device response.result.snapshot and copy & paste its contents


Update logback-classic, logback-core to 1.4.11

About this PR
📦 Updates

ch.qos.logback:logback-classic
ch.qos.logback:logback-core

from 1.4.8 to 1.4.11
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.qos.logback" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "ch.qos.logback" }
}]

labels: test-library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #140.


How does this compare to other sources?

Back when this was started, there seemed to be a dearth of data about layoffs. Today, there are several websites with layoff data, some automated with "AI," some based on WARN, some specializing in specific industries, some collected from media and user reports.
They include:

https://layoffs.fyi
https://layoffdata.com/
http://videogamelayoffs.com
etc.

Presumably some small number of them are based on BLN data or scrapers, but I have yet to see any credit.
It'd be interesting to pull a subset of these and do a comparison, possibly on a docs page, to see what we can learn and find out what we may be missing.

I would love that
Sent from my iPhone
On Apr 3, 2024, at 5:46 AM, Chris Zubak-Skees @.***> wrote:

Back when this was started, there seemed to be a dearth of data about layoffs. Today, there are several websites with layoff data, some automated with "AI," some based on WARN, some specializing in specific industries, some collected from media and user reports.
They include:

https://layoffs.fyi
https://layoffdata.com/
http://videogamelayoffs.com
etc.

Presumably some small number of them are based on BLN data or scrapers, but I have yet to see any credit.
It'd be interesting to pull a subset of these and do a comparison, possibly on a docs page.
—
Reply to this email directly, view it on GitHubhttps://github.com/biglocalnews/warn-scraper/issues/646, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAEFU3XYIMFRJQB6EZWUHF3Y3P2ZRAVCNFSM6AAAAABFVH462OVHI2DSMVQWIX3LMV43ASLTON2WKOZSGIZDEOBUGE4DCMQ.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


Suggestion: Is it possible to offer the tag number for a type in the exception message ?

I found it is hard to track out the type when exception thrown during serialization/deserialization.
usually a message like below shows, but gives no infomation as which type is wrong (usually due to type registration mismatch between the serializer and the deserializer).

Current object's property count is 5 but binary's header maked as 17, can't deserialize about versioning.

I think it would be helpful if the exception message include the pre-registered tag number of the types.

Since the Type on the serialize side does not exist in the data, only the Type on the deserialize side is showable.
Including the Type in the Exception would be nice, however, it would not increase the amount of information itself much, since it would not differ from what can currently be seen from StackTrace.

StackTrace does not reveal which type registered with MemoryPack is causing the problem( e.g. a type being registered with a different tag number with the deserilizer, i.e. failed version control), especially when deserilizing a complicate type which includes references to multiple other types.

v1.9.8 added type name


Error calling npm to fetch template

error is showing like
"Error calling npm to fetch template. Please ensure that node and npm are installed and available on your system. Full error message: Command failed with ENOENT: npm root -g --prefix C:\Users\azureuser\AppData\Roaming\BotFrameworkComposer.yo-repository --loglevel error spawnSync cmd.exe ENOENT"

Closing as duplicate.


OpacityEffect in the new effects engine

Description
The OpacityEffect is a bit tricky to compose since the opacity only is stored in 8 bits, so it can get pretty big numeric discrepancies, so we have to accumulate the rounding errors.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors and are passing (See Contributor Guide).
[x] My PR does not decrease the code coverage, or I have a very special case and explained on the PR description why this PR decreases the coverage.
[x] I updated/added relevant documentation (doc comments with ///) and updated/added examples in doc/examples.
[x] I have formatted my code with flutter format and the flutter analyze does not report any problems.
[x] I read and followed the Flame Style Guide.
[x] I have added a description of the change under [next] in CHANGELOG.md.
[x] I removed the Draft status, by clicking on the Ready for review button in this PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flame users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md).
[x] No, this is not a breaking change.

Related Issues
Replace this paragraph with a list of issues related to this PR from the issue database. Indicate, which of these issues are resolved or fixed by this PR.

@st-pasha any idea how we could handle rounding errors like the ones we will get in here when the affected storage is only 8 bits, so that the rounding errors can become quite big?

any idea how we could handle rounding errors like the ones we will get in here when the affected storage is only 8 bits, so that the rounding errors can become quite big?

We could store double-precision version of a variable within the effect, and then send into the component the rounded value. Also, we'll need to keep track of any external updates to the component's value, and apply it to our double-precision internal value.

We could store double-precision version of a variable within the effect, and then send into the component the rounded value. Also, we'll need to keep track of any external updates to the component's value, and apply it to our double-precision internal value.

If we just take the delta of the internal value each tick it should be fine without keeping track of other updates, right?

If we just take the delta of the internal value each tick it should be fine without keeping track of other updates, right?

There is always a possibility that another effects updates alpha, so need to keep track of that.

There is always a possibility that another effects updates alpha, so need to keep track of that.

So you mean the internal double wouldn't be on the effect, but somewhere else? I was thinking that out would be on the effect, and that we only update with the delta from the last internal double. Then they mix their updates, but maybe it wouldn't be any more acccurate than now, since rounding errors would still have to be taken into consideration when converting it to alpha. Maybe the current solution is good enough?

We can merge it as-is, if it works; and then think about possible improvements later.


Report not in chronological order

Describe the bug
As a Program Admin, I want to view the number of applications my program received each month. The Report tabs includes this data. However it is not in chronological order, by month.
To Reproduce
Steps to reproduce the behavior:

Login as a program admin
View a program with applications received over multiple months
See that it is not in chronological order

Expected behavior
Months should be sorted from oldest to most recent
Screenshots

@elisekalstad is this also an issue with the CSV or just in the web view?

@elisekalstad is this also an issue with the CSV or just in the web view?

@bion -- Good question! CSV is impacted too

@elisekalstad got it, I think the PR mentioned above should fix it

@elisekalstad got it, I think the PR mentioned above should fix it

That's great! Thank you!!!


Record cassettes in memory

Is there a built-in way to avoid saving the cassette YAML file?
There's an integration test I'd like to inspect the responses for but I don't need to record them forever.
Currently, I just rewrite them.

The docs have this covered: https://vcrpy.readthedocs.io/en/latest/advanced.html#register-your-own-cassette-persister


tweak spacing

tweak spacing so array keys are indented the same.

Thanks very much on the tweak and your first contribution 🎉

Happy to contribute, and want to learn more.  Cool project.


Use S3 to track export status jobs

Overview
PR introduces S3 usage for Export statuses.
Checklist

[x] Test

Demo
create_export_definition:

wait_for_status:

Testing Instructions

Run export job (for example create new export through UI)

Closes #1966
Closes #1964

Wicked, this should be what I need.

I'm working on something that will partially address your issue, @notthatbreezy .

@notthatbreezy thx, already know about it 🎉

Updated!


[TASK] Add kudos to @ziegenberg to the changelog

Maybe also add them to the contributors section in the readme?

Maybe also add them to the contributors section in the readme?

Thanks, good point! Done.


QDL Flash in a VM?

Hey,
I'm trying to get this worked in a Proxmox-VM setup.
I'm running a turnkeycore image, but the same happened on the latest ubuntu server.
Linux core 6.1.0-18-amd64 #1 SMP PREEMPT_DYNAMIC Debian 6.1.76-1 (2024-02-01) x86_64 GNU/Linux
It seems it is unable to communicate with the device -> Connection timed out

I tried to increase the timeout in sahara.c - qdl_read but it just hangs.
I tried to blacklist qcserial kernel module on both Proxmox host and the VM.
It works flashing from host directly (and other regular conditions)
I have hotplug enabled on the VM, tried to directly assign the port and the Vendor/Device IDs.
I tried giving full access by adding a udev rule
SUBSYSTEM=="usb", ATTRS{idVendor}=="XXXX", ATTRS{idProduct}=="XXXX", MODE="0666"

Strace gave me this:
write(2, "Waiting for EDL device\n", 23Waiting for EDL device
) = 23
pselect6(4, [3], NULL, NULL, NULL, NULL) = 1 (in [3])
recvfrom(3, NULL, 0, MSG_PEEK|MSG_TRUNC, NULL, NULL) = 1224
recvmsg(3, {msg_name={sa_family=AF_NETLINK, nl_pid=280, nl_groups=0x000002}, msg_namelen=128 => 12, msg_iov=[{iov_base=[{prefix="libudev", magic=htonl(0xfeedcafe), header_size=40, properties_off=40, properties_len=1184, filter_subsystem_hash=htonl(0x577c5e5), filter_devtype_hash=htonl(0x27f8f50c), filter_tag_bloom_hi=htonl(0x2080000), filter_tag_bloom_lo=htonl(0x400001)}, "UDEV_DATABASE_VERSION=1\0ACTION=a"...], iov_len=1224}], msg_iovlen=1, msg_control=[{cmsg_len=28, cmsg_level=SOL_SOCKET, cmsg_type=SCM_CREDENTIALS, cmsg_data={pid=2521, uid=0, gid=0}}], msg_controllen=32, msg_flags=0}, 0) = 1224
getrandom("\x28\x5d\xa9\xdb\x16\x26\x4c\x56\x4a\xd4\x66\x5e\x19\x0d\xe9\x60", 16, GRND_INSECURE) = 16
getrandom("\x55\xd4\x33\x5b\x53\x69\x1d\x70\xee\xe5\x9f\x47\xcf\xc7\x57\x69", 16, GRND_INSECURE) = 16
getrandom("\x66\x14\x81\x2f\x6a\xad\x79\x61\x40\xa6\x0c\xff\x87\x93\xf8\x20", 16, GRND_INSECURE) = 16
getrandom("\x4a\x8f\x0e\xdb\x5b\x42\xd0\xcf\x0c\x18\xbb\xab\xb7\x17\x90\x06", 16, GRND_INSECURE) = 16
openat(AT_FDCWD, "/dev/bus/usb/002/005", O_RDWR) = 4
read(4, "\22\1\0\2\0\0\0@\306\5\10\220\0\0\1\2\0\1\t\2 \0\1\1\0\200\1\t\4\0\0\2"..., 1024) = 50
close(3)                                = 0
ioctl(4, USBDEVFS_IOCTL, 0x7ffd7df19830) = -1 ENODATA (No data available)
ioctl(4, USBDEVFS_CLAIMINTERFACE, 0x7ffd7df1980c) = 0
openat(AT_FDCWD, "prog_firehose_ddr.elf", O_RDONLY) = 3
ioctl(4, USBDEVFS_BULK, 0x7ffd7df18710) = -1 ETIMEDOUT (Connection timed out)
close(3)                                = 0
exit_group(1)                           = ?
+++ exited with 1 +++

Do you have any suggestions on how to investigate further? 😄
Thanks!

Oh, I stumbled on my own issue, while I was digging deeper for this 🙂


Add, style, and use DictionaryCard

This PR fixes...
This PR addresses #121
Note: The DictionaryCard doesn't look exactly like the Figma, but it supports some more cases that the Figma didn't cover. Will have @julietafb take a look and we can perhaps modify in a future PR.
What I did...

Added a DictionaryCard component
Styled DictionaryCard component ~ Figma

How to test...

Navigate to /dictionary
See that dictionary entries each appear with correct content
Check that links to add new perspective and add a definition lead to working Github files

I learned...
I added the last variant to the Tailwind config file - here's how. Keep this in mind if a Tailwind property is not working for you (this is what happened to me!). Read the docs to see if it must be added as a variant first.

Just took a peak at this and want to confirm a few observations:

overall the scale looks very large for me relative to the rest of the page, but not sure if this is the intended design (knowing that other elements on the page are unfinished). Hard to tell by looking at the figma because the page width looks different.. Would it be helpful to see design mockups for a variety of screen sizes? If Julieth is happy with this as is, i'm also happy to keep it! Just pointing it out.

The "add a definition" and "add your perspective" links go to the corresponding github pages, which don't offer much instruction on what to do from that point. This might be okay if we are linking to the instructions elsewhere on the dictionary page? (I think we have something typed up in github already but I can't find it right now 😅 )

Lastly, a design comment that doesn't block this PR from being merged but curious what @julietafb thinks, I would love to be able to click on the whole dictionary entry "card" to view the entry page, rather than just clicking on the word. From the design as is, it's not obvious to me that I can click on it.


TrustGraph CLI (in Rust)

TrustGraph CLI Design notes

other output formats

other JSON-LD formats

"extra" fields for all the things?

"flavors" of TrustAtoms, including Journalistic flavor: https://github.com/blueskyCommunity/OpenTrustClaims/blob/gvelez17-harlan-trustgraph-compatible/open_trust_claim.yaml

Yes!  committed it here - https://github.com/blueskyCommunity/OpenTrustClaims/blob/main/open_trust_atom.yaml

would love to contribute a way to support these fields


Fixed localization in case Harpy linked as Framework (using cocoapods)

Current version doesn't find Harpy bundle in case it is linked as Framework.

Thanks - will look into it tonight.

Looks good. I did this in Siren, but forgot to do this in Harpy. Thanks!

v3.4.8 is now available via all package managers.


⚠ Bofrost.lu has degraded performance

In 130924f, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1279 ms

Resolved: Bofrost.lu performance has improved in ad3e015 after 31 minutes.


Update bonding pools

This PR updates the hardcoded bonding pools to include Rizzolver and Furucombo/Portus. The pools can be found on the CMS under solver-bonding pools

Note these are reduced subpools within the CoW DAO bonding pool, and so far we haven't added any other subpool that has been created in this query. The main query used for the service fee is this: https://dune.com/queries/4017925
and it has a colocated_solvers hardcoded table. So I don't really agree with this PR.

Note these are reduced subpools within the CoW DAO bonding pool, and so far we haven't added any other subpool that has been created in this query. The main query used for the service fee is this: https://dune.com/queries/4017925 and it has a colocated_solvers hardcoded table. So I don't really agree with this PR.

Sorry, looks like I updated the wrong query. I'll redo the PR to update cowprotocol/accounting/rewards/mainnet/service_fee_query_4017925.sql.

Oh my bad, I should have checked more carefully.

Sorry, looks like I updated the wrong query.

Can you document the process in Notion please so that it's easy to follow for future solvers


marker_annotated.xlsx

Very much enjoying the paper!!  I downloaded the fragment files and I'm working through the code but I was wondering if I missed the "marker_annotated.xlsx" somewhere? Could you point me to it if so?

Can you please let me know the codes that refer to the "marker_annotated.xlsx"? It will be easier for me to locate the file. Thank you,

It's in the 01_Cell_Clustering.rmd line 114:
Annotate the cell clusters in excel and add the cell type information to the ArchR object
sig_genes <- openxlsx::read.xlsx("E12_14/results/marker_annotated.xlsx", sheet = 2)
tab <- sig_genes[!duplicated(sig_genes$cluster),]
clusters <- tab$cluster

Thanks again!

I just added the excel file. Please let me know if you have other questions

Thank you!


Update README.md with last versions

Bump from v2 to  GitHub actions/checkout@v3
Bump from v1 to  limitusus/json-syntax-check@v2

Awesome work!

dup #20

Sorry for duplication, good work BTW!


Hierarchical Clustering: Counting the number of clusters from dendrogram

GSSoC 21':
I would like to add simple explanation with images (created by me) of Hierarchical Clustering, that'll explain how to "count" the number of clusters from dendrogram. Kindly assign me the issue :)
Thank you!

Assigning you with the same @AkhileshThite

Thank you :)

@geekquad please assign me this issue I will send the pr in 1 day

@subhangi2731 go ahead.

@geekquad please assign me this issue I will send the pr in 1 day

Hey! My PR for this issue is already merged #145
you can make changes in that if you want to :)

@AkhileshThite I didn't know that your PR is already merged.

@AkhileshThite I didn't know that your PR is already merged.

It's okay.