Performance issues with engine

Expected Behavior
Tried to run apollo bags on ~12G of siva files (a subset of PGA), on local-gpu cluster. Expected the job to run in under two hour, given that on another subset of ~6G it ran in about 40min before failing at a much further point in the job.
Current Behavior
The job hangs on the last task of the first stage (what would amount to the loading phase). It's been around 5 hours now, and it has still not gone to the next stage. As you can see on this (truncated) snapshot of the spark UI, the last task has been computing for much longer then the other tasks, as if the partition it was considering was completely disproportional in terms of size compared to the others. Although its only the last one that takes forever, the duration of tasks varies a lot. Only 12 out of the 83 tasks took more then 3 mins to finish, but among those 12 tasks 5 of them took between 3 and 5 minutes, and the other took between 10 and 35mins (and the last over 5 hours).

Possible Solution
Modifying the way the data is inputed, perhaps by tuning the way partitions are made ? I specified the value for spark.default.parallelism as 360, but I got only 83 partitions. On the other substet of PGA, I specified the same value, but got 90 partitions, even though the total size was half as big. Tried to play around with that value on my data, and although it seems correlated, the relation is not clear (follows a log curb):
Default Parallelism: 200 | 360 |  1200 | 3600 | 10000
Number of partitions on launch: 46 | 83 | 155 | 238 | 253

There are a little over 1000 siva files in this subset so i was expecting to at least be able to have as much partitions. Some of the siva files are quite big, it might also be related ?
Steps to Reproduce (for bugs)

Log on to science 3
Run docker run  -it --rm -v /storage/romain/:/data -v /home/romain/io:/io --link bblfshd --link scylla-egor-1 srcd/apollo_egor apollo bags -r /data --bow /io/bags/bow.asdf --docfreq /io/bags/docfreq.asdf --persist MEMORY_AND_DISK -f graphlet id uast2seq -l Java Python -s 'local[20]' --min-docfreq 5 --bblfsh bblfshd --cassandra scylla-egor-1 -m 40G,80G,6G   --config spark.default.parallelism=360
Press c then Enter

At first I tried to run it on k8, then copied the siva files and ran it on science3, the behaviours was the same. An error of the form below should pop up, but i don't think it's related.
18/04/02 14:53:06 WARN RefWithCommitIterator: incorrect object found
org.eclipse.jgit.errors.IncorrectObjectTypeException: Object c23029f58a8a74e12e99eead288413af578759e5 is not a commit.

Context
Not considering this issue this will block any attempt to use engine on large amount of data.
Your Environment (for bugs)
Tried it on science3 with Egor's apollo image (he modified it to fix a couple things in ml), engine version is 0.5.1, as well as local-gpu cluster with a custom pod, engine version is 0.5.5.

Could someone guide us to what stage is it executing when this problem arises? A link to relevant apollo source code lines would help.

Sure, so when you use apollo bags here you are actually using the ml function repos2bow_entry_template. The execute call is this one, and the trandformers pipe appearing on the terminal at this point is this: Ignition -> DzhigurdaFiles -> UastExtractor -> Moder -> Cacher -> MetadataSaver
(The cache_hook is MetadataSaver and select is DzhigurdaFiles, both classes are in the same file in apollo I linked to above).
After looking at the Spark DAG below, I think this problem arises when we are in the Ignition phase:

Shall we close this?

As you want, but the problem remains. There are workarounds, like converting siva files to parquet, but yeah, as is there is close to no parallelization when loading siva files with the engine, on all tasks Ive seen, and this phase represents usually most of the computing time on a given job. Hopefully this will change whenever gitbase is ready.

No, if it remains then we should leave it open. I thought after what Antonio told you to try it was fixed for you.

No, I edited my answer, sorry will repost next time

There are workarounds, like converting siva files to parquet,

Not really, the problem remains, but it becomes smaller.
I really hope that we can fix it on our side and make repartition after reading Siva files.
https://github.com/src-d/ml/pull/228
So the real problem is after reading siva files we have really skewed partitions.

As you want, but the problem remains. There are workarounds, like converting siva files to parquet, but yeah, as is there is close to no parallelization when loading siva files with the engine, on all tasks Ive seen, and this phase represents usually most of the computing time on a given job. Hopefully this will change whenever gitbase is ready.

Regarding the .parquet: any ideas how to do that reasonably fast?

@sakalouski if you need to use spark you can use session.read.parquet(parquet_path) if you do not, there is a bunch of libraries to do that like pyarrow or fastparquet.

@zurk Hi, Konstantin! I meant fast transformation from .siva to .parquet.

@sakalouski, oh, ok. You can try to use a sourced-ml python library for that: https://github.com/src-d/ml. There is still a lack of documentation, but what you need is python3 -m sourced.ml preprocrepos entry point. It does exactly what you asked: convert .siva files to .parquet. We use it before all machine learning pipeline. Do not forget to specify which columns you need to save. default list is ("blob_id", "repository_id", "content", "path", "commit_hash", "uast", "lang") for HEADs only.

Thanks a lot - sounds simple! Trying it soon and letting You know about the
chosen approach.
At this point I thought of creating an index and accessing particular .siva
files instead of filtering them.
On Tue, Sep 18, 2018 at 10:32 AM konstantin<EMAIL_ADDRESS>wrote:

@sakalouski https://github.com/sakalouski, oh, ok. You can try to use a
sourced-ml python library for that: https://github.com/src-d/ml. There is
still a lack of documentation, but what you need is python3 -m sourced.ml
preprocrepos entry point. It does exactly what you asked: convert .siva
files to .parquet. We use it before all machine learning pipeline. Do not
forget to specify which columns you need to save. default list is ("blob_id",
"repository_id", "content", "path", "commit_hash", "uast", "lang") for
HEADs only.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/src-d/jgit-spark-connector/issues/351#issuecomment-422324452,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfOlL6Nk9AMM8G3fvlG3NyxZ-cROhituks5ucL2-gaJpZM4TDpOB
.

@zurk for some reason seems not to work for me

Any ideas, how to fix it?

@sakalouski let's move to another issue with it.
Can you please create a new one here https://github.com/src-d/ml/issues and describe exactly what you did, so I can reproduce it and help you? The screenshot is not enough. ;)

@zurk Hi, I am trying to specify the features to store in parquet, but it does not work (the default version works fine).
The command is the following:
srcml preprocrepos -r temp -o test_defined_feats --bblfsh x.x.x.x --dep-zip -l Python JavaScript Java -x blob_id repository_id content path commit_hash uast lang &> logfile.txt
The output is the following:
srcml preprocrepos: error: argument -x/--mode: invalid choice: 'blob_id' (choose from 'file', 'func', 'repo')

@sakalouski Please, do not use this issue for general support. File a new one at https://github.com/src-d/ml/issues

@smola this should be closed, no ? I don't think know if it's still an issue, but if that's the case another one should be raised, this is completely outdated


Correct RCTBridgeModule import priority to allow Expo core to be installed

In our app, we needed to install Expo core and we ran into iOS build issues related to the import of RCTBridgeModule.h in this Emarsys Wrapper.
To resolve this issue we have adjusted your wrapper to first try to import <React/RCTBridgeModule.h> before falling back to "RCTBridgeModule.h" (which allows backwards compatibility).
#if __has_include(<React/RCTBridgeModule.h>)
#import <React/RCTBridgeModule.h>
#else
#import "RCTBridgeModule.h"
#endif

instead of
#if __has_include("RCTBridgeModule.h")
#import "RCTBridgeModule.h"
#else
#import <React/RCTBridgeModule.h>
#endif

Note: To get this fix to be applied, we also needed to get CocoaPods to install the pod files from the local, patched, files, rather than the remote git URL specified in the podspec. To do this we added the following to our Podfile
pod 'RNEmarsysWrapper', :path => '../node_modules/react-native-emarsys-wrapper/RNEmarsysWrapper.podspec'
If you want to test out its PR branch in an example project, then you may also need to temporarily adjust your Podfile in the same way, so that CocoaPods use the local files.
Also, depending on what version of React Native you need to be backwards compatible to, you may just be able to use #import <React/RCTBridgeModule.h> by itself. The imports were changed in React Native 0.40.0, which was released in December 2016.
It would be great if you could merge this PR, or adjust/update the imports, and it would save us (and potential future customers) from having to manually patch it or fork it etc.

I see you just merged a PR that fixes this 🎉
Thanks!

Perfect timing 😉
It'll be available in the next wrapper release. Thank you for reporting 😊


ESP-IDF build instructions

Hi,
The instructions for how to build rs-matter for esp-idf were removed in pull request #129 citing that support was upstreamed. Unfortunately it is now unclear from looking at the repository whether using matter-rs on an ESP32 is supported, and if so what networking stack is supported (wifi, threads... etc).
Some clarification with minimal build instructions on how to build rs-matter for ESP-IDF would be amazing.

Small update on the topic:

BLE support is in the works
For ESP IDF specifically, I would suggest to follow the development of this crate


Updated GTMSessionFetcher package version

The GTMSessionFetcher package version now matches the one in Firebase-iOS-SDK. #1111

Thanks!

@morganchen12 #1111  can be closed now. (didn't link the issue correctly)


Unable to open Kap on newer versions.

On newer versions of Mac OS you can't open kap anymore since it can't be scanned to malicious malware.
macOS version: 10.15.1
Kap version: 3.0.0
Steps to reproduce
Install kap and click to open
Current behaviour
Unable to open
Expected behaviour
App will open.
Workaround

Thank you, this answer will provide a workaround for now: https://github.com/wulkano/kap/issues/770#issuecomment-559761584

There's a new update out that fixes this: https://github.com/wulkano/kap/releases/tag/v3.0.1


Update bootstrap to 4.4.1

Updates org.webjars:bootstrap from 4.3.1 to 4.4.1.
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.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.webjars", artifactId = "bootstrap" } ]

labels: semver-minor

Superseded by #207.

Superseded by #207.


Changes to reflect NDSL import format

Description
Modified import methods for inter and intra-package methods to reflect format adopted for NDSL
How Has This Been Tested?
Tested via lint and currently implemented translate tests
OS: RHEL 8.9 Ootpa
Built conda environment using workflow in github action configurations
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules
[ ] New check tests, if applicable, are included
[ ] Targeted model, if this change was triggered by a model need/shortcoming

Looks good. Needs to move the ci to the right ndsl


Fix bug with running broad jump and update tooltips

Fix bug with running broad jump and update tooltips

No bug, but updated tooltip text.


Capturing only part of a dollars/cents expression (DE)

When a dollars/cents currency expression is written out, only the dollar part is captured:
•	"25 Dollar 50" => 25 "Dollar"

Merged with #2479.


Warning in registration-api/generated-values.yaml of Resource Registration component

W1111 14:54:41.537247 1082435 warnings.go:70] annotation "kubernetes.io/ingress.class" is deprecated, please use 'spec.ingressClassName' instead

The same in registration-harvester/generated-values.yaml

Sound like problems with the BB helm charts. Check this and then reassign to the relevant BBs.


Embedding webchat on serviceNow

Hi All,
i am trying to embed latest  JavaScript or react webchat on Service Now. i am facing one unresolved issue in JavaScript webchat. please see below screenshot.

FYI
i am following this link properly :- https://github.com/microsoft/BotFramework-WebChat/tree/master/samples/01.getting-started/a.full-bundle
Similarly for React WebChat also i am getting issue which you can see on screenshot.
issue 1:-

@Bradrajkumar, how are you embedding the page? I don't know too much about Service Now, but I know one method of embedding is to configure an iFrame component. Is this how you are attempting this?

@corinagum Is this ticket going to be in R12?

@Virtual-Josh no, it is not scheduled as an action item. Since it is stale, I am closing this issue.


mephistypsteles:0.1.0 (hotfix)

I am submitting

[x] a new package
[ ] an update for a package

This is a hotfix and only changes the example PNG to also have the @preview import instead of a local file path.
Sorry for the inconvenience, I would've force-pushed to the original PR again, but it was merged too quickly by your amazing team!

Thank you, and sorry I didn't catch that in the previous PR.


Meson build command not recognizing command line option

Disclaimer: This is my first post and so i am sorry if it is a little unclear or not well put together.
when using the meson build command i keep getting
meson.build:97:0: ERROR: Pkg-config binary for machine MachineChoice.HOST not found. Giving up.
so I pulled up the meson-log.txt to which I got a whole list of errors:

None of 'PKG_CONFIG_PATH' are defined in the environment, not changing global flags.
None of 'PKG_CONFIG_PATH' are defined in the environment, not changing global flags.
Project name: wlroots
Project version: 0.10.0
None of 'CC' are defined in the environment, not changing global flags.
None of 'CFLAGS' are defined in the environment, not changing global flags.
None of 'LDFLAGS' are defined in the environment, not changing global flags.
None of 'CPPFLAGS' are defined in the environment, not changing global flags.
None of 'CC_LD' are defined in the environment, not changing global flags.
Sanity testing C compiler: cc
Is cross compiler: False.

I originally thought that it was a cmake problem but i am running the most current version.
I am running:
Operating System: KDE neon 5.18
KDE Plasma Version: 5.18.5
KDE Frameworks Version: 5.70.0
Qt Version: 5.14.2
Kernel Version: 5.3.0-51-generic
OS Type: 64-bit
Processors: 12 × Intel® Core™ i7-9750H CPU @ 2.60GHz
Memory: 15.3 GiB of RAM

This error doesn't seem related to wlroots. Please ask the Meson devs for help.


Build failure in Visual Studio 2015.

I got following error:

error MSB4041: The default XML namespace of the project must be the MSBuild XML namespace. If the project is authored in the MSBuild 2003 format, please add xmlns="http://schemas.microsoft.com/developer/msbuild/2003" to the  element. If the project has been authored in the old 1.0 or 1.2 format, please convert it to MSBuild 2003 format.

sorry, VS2015 is not supported - please use the nuget package or update to VS2017


feat: Load manual fixes for invented primitives

Since the automated translation does not work well and many translations fail because invented primitives are not translated correctly, we provide this workaround: Translations are stored in a JSON file which can be edited and loaded to overwrite the automated invented primitive translations.

392e13c


Divide events into features.

That way people can conditionally load only the events that they are interested in.
@LegNeato this PR might conflict with #53. If you merge this one first, 53 needs to be updated to add features flags to tests. If you merge that one first, this one needs to be update to incorporate the tests in 53. Can you merge one of them and I'll update the other one accordingly?
Fixes #45
Signed-off-by: David Calavera<EMAIL_ADDRESS>

Awesome, great! Apologies, I had to buy a new computer due to giving my work one back and was offline for a bit 😄

oh no worries. @LegNeato if you're looking for a job, I know the aws lambda team is hiring a lot of engineers and managers.


Make it possible to change title bar text

Is it possible to change the title bar? Now i have 3 tabs with the name Dashboard that is totally useless for me. Instead i would like to have the host name first. Before this was changed it was showing Webmin, but now when you changed it, it would have been nice if i could choose my own text instead or at least have a drop down menu with different alternatives to choose from like you do when you choose date format and style.

Sopor, hi!
Sounds like a good idea. I added it.
Give it a try. There is new option in Theme Configuration/Global Options:

The initial title and the way it looks can be tweaked under Webmin Configuration/User Interface section:

Unfortunately it is impossible to remove the Webmin 1.900 so i still can't get the host name first or the custom name i select in the User interface :(

What about Webmin option to prepend hostname to page title?

What is the expect output, by the way?

To get the hostname first so it is easy to see it in the title bar (tabs) and the Other name... in the User interface also could be first to have a custom name in the title bar. When many tabs are open they get really small and only show the first word as you could see in the picture i posted before.

Okay, I am reworking this part and adding more options to format titles, and be shown in one place in UI.

How about these kind of formats?
Module
Module — Webmin verson (OS type)
Webmin verson (OS type) — Module
Webmin verson (OS type)
hostname - Webmin verson (OS type) — Module
username at hostname - Webmin verson (OS type) — Module

I was a little hasty in the afternoon. I added even more. I expect this will satisfy you and other users?

Please give it a try now.

By the way, now you can run theme update by clicking hotkey - Ctrl + Alt + Meta + ⇧ + U. :smile:

By the way, now you can run theme update by clicking hotkey - Ctrl + Alt + Meta + ⇧ + U. 😄

I don't even know where to find those keys 😕

Okay, if you have an idea of other title formats, please share - I might add it.
It's all modifier keys pressed together + U to update to the latest beta, same mod keys + R to clear all kind of theme caches and reload.

Meta key is Cmd on Mac, Alt is Option.
One day I'll translate those keys for different platforms.

So these are some kind of hidden hotkeys?

Not anymore.
If you hover on Clear Cache and Force Update buttons in Theme Configuration, then you would see the explanation tooltip.

For the record, when using the Authentic Theme, the option is moved to the theme configurator


umpf: add new --override option to override topic commit-ish

In one project, umpf tags are pushed immediately, but branches are only pushed upstream once they are reviewed and cleared for merge into BSP. It's thus often the case, that developers want to use the upstream branches, except for that one branch they are currently working on.
This is not an unusual case and the tips-and-tricks lists some workarounds to achieve this. Provide a better way by allowing overriding single topic branches on the command-line.

@michaelolbrich I am thinking of whether we might want to increase --override's scope a bit: If the topic doesn't already exist it will be appended to the list of topics, perhaps with a rename to --with, --inject or w/e. Example:
umpf tag utag --with existing-topic --with new-topic

This would make it more useful to cover two common cases I have: Someone umpf'd a release and I want to either override a single topic branch that exists or to add a single branch into the umpf. The alternative for the latter is currently cumbersome:
umpf show utag >series
git rev-parse new-topic
vi series # add topic and hashinfo
umpf tag utag --identical

I like the idea in general but I think we should have two separate options for this. --override and maybe --append.

I had changed the syntax from --override topic_orig:topic_override a few revisions ago to  --override topic_orig=topic_override, because I found it clearer: When doing a git push, you do git push topic_override:topic_orig so I found that = would be an improvement.
Anyway, I forgot about that and tried to use umpf --override today and found that I broke error handling: git rev-parse returns text even if it fails, so we need to always check the exit code. I learnt that the bash syntax of
if ! rev="$(${GIT} rev-parse "${OVERRIDES[$topic]}^{}" 2> /dev/null)"; then

used elsewhere in umpf, does just that, so I force pushed this single change.


Bug fix cc0091 not working when used with delegates

When a Method, that should be made static is used somewhere like new  X().MethodToBecomeStatic();. I fix it to new X(); X.MethodToBecomeStatic() to ensure that the program logic is not broken.

If it is a static method, it should not be called on the type, not the instance. It should be changed to
T.MethodToBecomeStatic() where T is the type of X.

Yes, this is done, but the constructor call is preserved.
What I mean with this code:
using System;
namespace ConsoleApplication1
{
    class Type1
    {
        public static bool Initialized = false;
        private int i;
        public Type1()
        {
            i = 0;
            Initialized = true;
        }
        public bool Foo()
        {
            return Initialized;
        }
    }
    class Type2
    {
        private int i = 0;
        public void Bar() {
            i++;
            var a = new Type1().Foo();  // <- original, should be refactored
            Console.WriteLine(a ? ""Initialized"" : ""Not Initialized""); //  <- this prints "Initialized"
        }
    }
}

my fix would add a new Type1(); as a new statement and replace the original new Type1() with Type1 so the code refactors to this:
using System;
namespace ConsoleApplication1
{
    class Type1
    {
        public static bool Initialized = false;
        private int i;
        public Type1()
        {
            i = 0;
            Initialized = true;
        }
        public static bool Foo()
        {
            return Initialized;
        }
    }
    class Type2
    {
        private int i = 0;
        public void Bar() {
            i++;
            new Type1();  // <- this is added
            var a = Type1.Foo();  // <- new Type1() is replaced with Type1
            Console.WriteLine(a ? "Initialized" : "Not Initialized");  //  <- this prints "Initialized"
        }
    }
}

instead of just replacing the new Type1() with Type1 :
using System;
namespace ConsoleApplication1
{
    class Type1
    {
        public static bool Initialized = false;
        private int i;
        public Type1()
        {
            i = 0;
            Initialized = true;
        }
        public static bool Foo()
        {
            return Initialized;
        }
    }
    class Type2
    {
        private int i = 0;
        public void Bar() {
            i++;
            var a = Type1.Foo();  // <- only new Type1() is replaced with Type1
            Console.WriteLine(a ? "Initialized" : "Not Initialized");  //  <- this prints "Not Initialized"
        }
    }
}

In this example you can see that the original code would print "Initialized".
The first fix, with the new Type1(); inserted would also print "Initialized".
The second fix without the new Type1(); would print "Not Initialized".
I think that fixes shouldn't change the behavior of the code, so I would prefer the first fix.
But if you want to have the second fix I can also change the code so that no constructor call is added.

PR #691 is fixing this, along with several other problems on CC0091. I was working on it from issue #680. It conflict with this one.
@martinzimmermann I also noticed you commented on #680. When you want to start working on an issue, please check if there isn't somebody already working on it. The first one assigned will have priority. This is explaining on the contributing doc, so we don't have 2 or more people working on the same issue.
Also, I am not sure we should keep the constructor call. #691 is not keeping it. I understand the collateral effects of a constructor call, but there shouldn't be any, besides the object construction itself, it is bad design. I believe that the most common use case would be to not keep it. But I understand some people might want to keep it.
We could offer to code actions for this scenario. But maybe this is too much. @code-cracker/owners what do you think?

@giggio: I’m very sorry for the inconvenience. I was initially fixing #677 this fix also accidentally has fixed #680, much like PR #691 also fixes #677.
Anyway thanks for the feedback, next time I will be more careful.


Fix connectivity check between agent and server

CATTLE_URL var is http://rancher-server:8080/v1/scripts/A9FB52823FF03A706E90:1480636800000:KQM8CcOkslALKB9esPGVsGLNbzk and it's not valid URL (HTTP 404), which is why cURL exits with non-zero status. I believe url var was meant to be used instead.

Before fix:
ERROR: http://rancher-server:8080/v1 is not accessible

After fix:
INFO: http://rancher-server:8080/v1 is accessible
...
INFO: Launched Rancher Agent: 4ac6d5327c7f6078bff737d2a09af5133eadb3a4f37b854c0a52ecde99fe3045

Alright. It seems I got token wrong or it expired and that caused http 404. After re-doing everything from scratch and at more slow pace I couldn't reproduce the problem. Apologize for false alarm and wasted PR ID :)


2.1.0 fails to upload with error in process_user_mode.py

Getting the following error after upgrading to 2.1.0
tconnectsync     | Traceback (most recent call last):
tconnectsync     |   File "/home/appuser/main.py", line 5, in <module>
tconnectsync     |     main()
tconnectsync     |   File "/home/appuser/tconnectsync/__init__.py", line 120, in main
tconnectsync     |     sys.exit(u.process(tconnect, nightscout, time_start, time_end, args.pretend, features=args.features))
tconnectsync     |   File "/home/appuser/tconnectsync/sync/tandemsource/autoupdate.py", line 53, in process
tconnectsync     |     added, event_id = ProcessTimeRange(tconnect, nightscout, tconnectDevice, pretend, features=features).process(time_start, time_end)
tconnectsync     |   File "/home/appuser/tconnectsync/sync/tandemsource/process.py", line 84, in process
tconnectsync     |     ns_entries = c.process(events, events_first_time, events_last_time)
tconnectsync     |   File "/home/appuser/tconnectsync/sync/tandemsource/process_user_mode.py", line 118, in process
tconnectsync     |     ns_entries.append(self.exercise_to_nsentry(start=items[0], stop=items[1], time_end=time_end))
tconnectsync     |   File "/home/appuser/tconnectsync/sync/tandemsource/process_user_mode.py", line 201, in exercise_to_nsentry
tconnectsync     |     duration_mins = (time_end.eventTimestamp - start.eventTimestamp).seconds / 60
tconnectsync     |   File "/base/.venv/lib/python3.9/site-packages/arrow/arrow.py", line 820, in __getattr__
tconnectsync     |     return cast(int, object.__getattribute__(self, name))
tconnectsync     | AttributeError: 'Arrow' object has no attribute 'eventTimestamp'

https://github.com/jwoglom/tconnectsync/blob/master/tconnectsync/sync/tandemsource/process_user_mode.py#L201
Not sure if there is more information I can provide.  Thanks again for everything you guys do!

Should be fixed in v2.1.1. Good catch!

Thanks for the fast response.  Looks like the error migrated on us:
tconnectsync     | Traceback (most recent call last):
tconnectsync     |   File "/home/appuser/main.py", line 5, in <module>
tconnectsync     |     main()
tconnectsync     |   File "/home/appuser/tconnectsync/__init__.py", line 120, in main
tconnectsync     |     sys.exit(u.process(tconnect, nightscout, time_start, time_end, args.pretend, features=args.features))
tconnectsync     |   File "/home/appuser/tconnectsync/sync/tandemsource/autoupdate.py", line 53, in process
tconnectsync     |     added, event_id = ProcessTimeRange(tconnect, nightscout, tconnectDevice, pretend, features=features).process(time_start, time_end)
tconnectsync     |   File "/home/appuser/tconnectsync/sync/tandemsource/process.py", line 84, in process
tconnectsync     |     ns_entries = c.process(events, events_first_time, events_last_time)
tconnectsync     |   File "/home/appuser/tconnectsync/sync/tandemsource/process_user_mode.py", line 118, in process
tconnectsync     |     ns_entries.append(self.exercise_to_nsentry(start=items[0], stop=items[1], time_end=time_end))
tconnectsync     |   File "/home/appuser/tconnectsync/sync/tandemsource/process_user_mode.py", line 207, in exercise_to_nsentry
tconnectsync     |     pump_event_id = "%s,%s" % (start.eventId, stop.eventId)
tconnectsync     | AttributeError: 'NoneType' object has no attribute 'eventId'

https://github.com/jwoglom/tconnectsync/blob/master/tconnectsync/sync/tandemsource/process_user_mode.py#L207
Sorry, I'm not a python guy or I'd try to jump in and help.  I'd end up doing more harm than good.

D'oh, some real brain farts in this section of the code. (why would we read from the stop variable in the "no stop time" section...)
Before rewriting this all to work with Tandem Source I had a good amount of unit tests which would typically catch stuff like this. Hoping to get around to adding those back, just had to rush the implementation of the last bits of this to get over the finish line before the EOL.
Pushed a commit/version to fix this one.

Looks good from my end!

D'oh, some real brain farts in this section of the code. (why would we read from the stop variable in the "no stop time" section...)

Ha, we've all been there.  You're killing it, thanks for the fast respones.


automatically retry examples for specific exceptions typical on Travis

I'm getting tired of hitting the re-run button on Travis when we have a IOError or Net::ReadTimeout. I'm not certain this actually fixes the problem, though, since each "it" statement isn't starting a new session; a dead session might be a dead session.

Pull Request Test Coverage Report for Build 1769

0 of 0   changed or added relevant lines in 0 files are covered.
226 unchanged lines in 12 files lost coverage.
Overall coverage decreased (-0.05%) to 96.718%

Files with Coverage Reduction
New Missed Lines
%

lib/watir/elements/select.rb
1
91.21%

spec/watirspec/elements/element_spec.rb
1
96.99%

lib/watir/browser.rb
2
94.38%

lib/watir/elements/element.rb
3
96.58%

lib/watir/wait.rb
3
89.29%

spec/watirspec/window_switching_spec.rb
4
96.76%

spec/watirspec/alert_spec.rb
7
100.0%

spec/watirspec/relaxed_locate_spec.rb
15
100.0%

lib/watir/legacy_wait.rb
18
77.27%

lib/watir/capabilities.rb
20
82.93%

Totals

Change from base Build 1764:
-0.05%

Covered Lines:
10432

Relevant Lines:
10786

💛  - Coveralls

🤷♂


Link to child route prepends parent route on transition

Hello vuejs!
I have a project I am building using vue at the moment, and have started to read into the vue-router documentation for nested routes.
I have created a view for a parent component with links to children view within an array of objects in my data.
<template>

        <router-link :to="section.to">{{ section.text }}</router-link>

    <router-view></router-view>

</template>

When I navigate to ../refugees, the home path loads fine. When I click on the first link (local, for example), it routes properly to ../refugees/local. When I click a second link (online, for example), it routes to ../refugees/refugees/online.
Is this due to the logic I am using to iterate over the sections/links in the data structure, or am I doing something else completely wrong?
The pull request I have everything in can be found here: https://github.com/awitherow/openrefuge/pull/4/files
Thank you!

Hello, it is working as expected: you're using relatives links that depend on the current route.
You should use named routes instead, they're explicit and absolut 🙂
Please, next time, ask questions on the forum or StackOverflow. Issues are for bug reports and feature requests only

Thanks for the help @posva but it is not clear how a child route should be named according to the documentation. Only a single tier route is shown.
Would I name the route, for example, refugees/home?
Perhaps the documentation could be updated to reflect the correct implementation?

@posva ah I see, it does not matter, as long as it links to that name, which serves similar to an 'id'. Got it! Thanks.


[x] I checked Firebase Crashlytics and verified that this release does not introduce any new crashes.

cc @Expensify/applauseleads

Please keep this unlocked for a bit

@roryabraham tag us when it's good to lock it. It's already late, we will start regression in the morning

@Expensify/applauseleads You can lock this whenever you're ready 🙇

Verified that https://github.com/Expensify/App/issues/9786 was resolved by https://github.com/Expensify/App/pull/9785

Checked off https://github.com/Expensify/App/pull/9732 because it's not a blocker, but it appears electron-log maybe hasn't been working since March? That's the most recent log I'm seeing in ~/Library/Logs/new.expensify/main.log

Also there's a bunch of PRs that were already deployed to production included in this checklist. Going to check them off and hope that it's a one-off error resulting from the deploy woes earlier today / me having to manually create this checklist.

Checking off https://github.com/Expensify/App/pull/9750 because it was fixed by https://github.com/Expensify/App/pull/9783. You can verify that it's working on staging by going to https://staging.new.expensify.com/docs/index.html

:rocket: All staging deploys are complete, @Expensify/applauseleads please begin QA on version https://github.com/Expensify/App/releases/tag/1.1.82-5 :rocket:

@AndrewGable
PR #9385 one tester was able to repro https://github.com/Expensify/App/issues/8641#issuecomment-1179566811. Since its existing issue, checking off.
PR https://github.com/Expensify/App/pull/9501 is failing https://github.com/Expensify/App/issues/9805. Repro in PROD checking off
PR #9627 & #9650 is failing https://github.com/Expensify/App/issues/9802. Potential blocker. Pls double check
PR https://github.com/Expensify/App/pull/9649 - @arosiclair @Justicea83 Can you check this off?
PR #9665 is pending clarification https://github.com/Expensify/App/pull/9665#issuecomment-1179141605
Issues logged
https://github.com/Expensify/App/issues/9742
https://github.com/Expensify/App/issues/9741
https://github.com/Expensify/App/issues/9747
https://github.com/Expensify/App/issues/9766
https://github.com/Expensify/App/issues/9802
https://github.com/Expensify/App/issues/9805
https://github.com/Expensify/App/issues/9807
https://github.com/Expensify/App/issues/9808
https://github.com/Expensify/App/issues/9809
https://github.com/Expensify/App/issues/9816
https://github.com/Expensify/App/issues/9817
https://github.com/Expensify/App/issues/9827

No, I am working on getting all the internal QA items checked off

checking off https://github.com/Expensify/App/pull/9665 https://expensify.slack.com/archives/C9YU7BX5M/p1657649041410549

@AndrewGable Yes, The only issue is https://github.com/Expensify/App/issues/9802. I think we can check it off as thats expected

Deploy did not work, we are looking into it

Closing this again

New checklist is here


behind a proxy getAlternateLinksHeaderValue dont play well

Description
when yo uare behind a proxy using the alternate links will thorw the wrong urls
Mandatory reproduction URL (CodeSandbox or GitHub repository)
no response
Reproduction description
using behind a proxy
Expected behaviour
url to match the real hostnames

I hope you understand that this bug report is incomplete, misses the mandatory reproduction and doesn't provide any relevant details about what exactly the issue is.
I'll close this for the time being, as it's not actionable. Feel free to open a new issue, but make sure to include all relevant details (e.g. a curl example, showing which response headers you receive, what the expected response is, etc.).


False positive for no-empty-attrs

We have a local variable named attr and the no-empty-attrs rule is being triggered. Renaming the variable works but should not raise this.
someArrayOfStrings.filter(function(attr) {
  return attr.underscore().endsWith('_id');
});

Yeah, makes sense. We should probably more strictly check this and scope the check to only model's properties.

I'll look into this soon 👍  For now you can disable this rule in this file or for this specific line:
/* eslint-disable-next-line ember/no-empty-attrs */
 return attr.underscore().endsWith('_id');

https://github.com/netguru/eslint-plugin-ember/releases/tag/v2.2.1


Order of -tags isn't tested.

Maybe this was done intentionally to give learning programmers more leeway. If that's not the case though, it's a bug.
Describe your problem and how to reproduce it:
In the Introduction to HTML5 Elements-challenge, the request clearly is to add the new tag AFTER the previous p-tag.
However, the following code also gets accepted as a correct solution.

  CatPhotoApp

  Purr jump eat the grass rip the couch scratched sunbathe, shed everywhere rip the couch sleep in the sink fluffy fur catnip scratched.

  Kitty ipsum dolor sit amet, shed everywhere shed everywhere stretching attack your ankles chase the red dot, hairball run catnip eat the grass sniff.

Add a Link to the page with the problem:
https://www.freecodecamp.org/learn/responsive-web-design/basic-html-and-html5/introduction-to-html5-elements
Tell us about your browser and operating system:

Browser Name: Firefox
Browser Version: 87.0 (64-bit)
Operating System: Linux (Linux Mint)

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

I think this can go either way, but I'm leaning towards changing the directions, and removing the after existing p element to give the more leeway for newcomers, to somewhat like:

Create a second p element with the following kitty ipsum text: ...

This is ofcourse just my opinion, I would love for others to pitch in...

I like the change.  The less explicit version seems to better describe how the page is already behaving.

I agree that this sounds like a good change  :)

As we do not want any confusion about what the main element should wrap. I suggest the following:

Create a second p element with the following kitty ipsum text: ...
Then, create a main element to nest only the two p elements.

I am opening this up to first timers
An issue with the help wanted or first timers only label is open for contribution. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors.
If you would like to contribute, and have not read the contributors docs, please do so here: https://contribute.freecodecamp.org/#/
If you have any issues with contributing, be sure to join us on the contributors channel, or on the contributors sub-forum

Sure, I'll do my best!

See #41859.
I wrote this. I changed the second line to improve it with "only", while still keeping it similar to the previous description.
Create a second `p` element with the following kitty ipsum text: `Purr jump eat the grass rip the couch scratched sunbathe, shed everywhere rip the couch sleep in the sink fluffy fur catnip scratched.`

Then, create a `main` element and nest only the two `p` elements inside the `main` element.


Subtraction not working properly in constraints

It seems like the minus sign can get lost in constraints.  With this LP file:
Minimize
    obj: 3x1 + 5x2 + 6x3 + 9x4 + 10x5 + 10x6
Subject To
c003:      -5 x1 - 1 x2 + 4 x3 - 2 x4 + 2 x5 - 1 x6 >=  3
Binaries
    x1 x2 x3 x4 x5 x6
End

I get the following dump of the parser. Note that the c003 constraint coefficient for x1 (-5) is properly captured, but the "- 1 x2" just captures "1.0" for the coefficient.
LPProblem {
    problem_name: "",
    problem_sense: Minimize,
    variables: {
        "x5": Binary,
        "x6": Binary,
        "x4": Binary,
        "x1": Binary,
        "x2": Binary,
        "x3": Binary,
    },
    objectives: [
        Objective {
            name: "obj",
            coefficients: [
                Coefficient {
                    var_name: "x1",
                    coefficient: 3.0,
                },
                Coefficient {
                    var_name: "x2",
                    coefficient: 5.0,
                },
                Coefficient {
                    var_name: "x3",
                    coefficient: 6.0,
                },
                Coefficient {
                    var_name: "x4",
                    coefficient: 9.0,
                },
                Coefficient {
                    var_name: "x5",
                    coefficient: 10.0,
                },
                Coefficient {
                    var_name: "x6",
                    coefficient: 10.0,
                },
            ],
        },
    ],
    constraints: {
        "c003": Standard {
            name: "c003",
            coefficients: [
                Coefficient {
                    var_name: "x1",
                    coefficient: -5.0,
                },
                Coefficient {
                    var_name: "",
                    coefficient: 1.0,
                },
                Coefficient {
                    var_name: "x2",
                    coefficient: 1.0,
                },
                Coefficient {
                    var_name: "",
                    coefficient: 1.0,
                },
                Coefficient {
                    var_name: "x3",
                    coefficient: 4.0,
                },
                Coefficient {
                    var_name: "",
                    coefficient: 1.0,
                },
                Coefficient {
                    var_name: "x4",
                    coefficient: 2.0,
                },
                Coefficient {
                    var_name: "",
                    coefficient: 1.0,
                },
                Coefficient {
                    var_name: "x5",
                    coefficient: 2.0,
                },
                Coefficient {
                    var_name: "",
                    coefficient: 1.0,
                },
                Coefficient {
                    var_name: "x6",
                    coefficient: 1.0,
                },
            ],
            sense: ">=",
            rhs: 3.0,
        },
    },
}

Thanks. The assertions could definately be more stringent.
I've invited you as a contributor - feel free to raise and fix issues.

That's very kind of you. Will do!

From: Dan @.>
Sent: Wednesday, January 3, 2024 1:32:08 PM
To: dandxy89/lp_parser_rs @.>
Cc: Andrew Henshaw @.>; Author @.>
Subject: Re: [dandxy89/lp_parser_rs] Subtraction not working properly in constraints (Issue #32)
Thanks. The assertions could definately be more stringent.
I've invited you as a contributor - feel free to raise and fix issues.
—
Reply to this email directly, view it on GitHubhttps://github.com/dandxy89/lp_parser_rs/issues/32#issuecomment-1875794692, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAOZY2XLHO5FDBACKWU6FULYMWP2RAVCNFSM6AAAAABBLT4QAGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZVG44TINRZGI.
You are receiving this because you authored the thread.Message ID: @.***>

I've pushed some fixes this evening before I headed out. Hopefully that unblocks you for the time being.

Negatives work perfectly, now.  Thanks!

I'll mark this is closed - will raise a seperate issue to add more thorough testing.


Is Temurin affected by CVE-2022-40433?

Is Temurin affected by CVE-2022-40433?
Following vulnerability posted in August https://cve.org/CVERecord?id=CVE-2022-40433, is Temurin affected?
Best regards,
Olivier
Context
Java version: 8
Your operating system and platform:
All

The latest versions are not affected - bug was patched

Hey,
I doubt that. In all official links I found only patches for >11 are done, see the list of prs in https://bugs.openjdk.org/browse/JDK-8283441
Ich checked the source-code in jdk8-392-b01 in which the patch was not applied:

So please double-check this for us - thanks!

My apologies - it does indeed require a backport to Java 8.  If you have a support contract with a commercial support vendor (https://adoptium.net/en-GB/temurin/commercial-support/) then you can contact them to expedite a fix. Else this will be a case of waiting for an OpenJDK committer to volunteer a backport

@karianna thank you - no problem, but good to know it will be adressed in Java 8, too 👍

Hi Martin & Leander,
I read on other commercial built support forum based on OpenJDK that Windows builds are not affected.
https://access.redhat.com/security/cve/cve-2022-40433
Could you confirm that please? Sorry I am not well knowledgeable to find out about this myself.
Best regards,
Olivier
De : Martijn Verburg @.>
Envoyé : lundi, 11 septembre 2023 00:18
À : adoptium/adoptium-support @.>
Cc : SCHMITT Olivier @.>; Author @.>
Objet : Re: [adoptium/adoptium-support] Is Temurin affected by CVE-2022-40433? (Issue #889)
 Attention email externe

My apologies - it does indeed require a backport to Java 8. If you have a support contract with a commercial support vendor (https://adoptium.net/en-GB/temurin/commercial-support/) then you can contact them to expedite a fix. Else this will be a case of waiting for an OpenJDK committer to volunteer a backport
—
Reply to this email directly, view it on GitHubhttps://github.com/adoptium/adoptium-support/issues/889#issuecomment-1712955355, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AO7435ZC3TLHLAC5LVRV373XZY4ADANCNFSM6AAAAAA4HPMBCU.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

@dsa-ols I don't thinks so. Otherwhise a backport of the patch to java 17 and 11 wouldn't have made any sense.

Hi Martin & Leander, I read on other commercial built support forum based on OpenJDK that Windows builds are not affected. https://access.redhat.com/security/cve/cve-2022-40433 Could you confirm that please? Sorry I am not well knowledgeable to find out about this myself. Best regards, Olivier De : Martijn Verburg @.> Envoyé : lundi, 11 septembre 2023 00:18 À : adoptium/adoptium-support @.> Cc : SCHMITT Olivier @.>; Author @.> Objet : Re: [adoptium/adoptium-support] Is Temurin affected by CVE-2022-40433? (Issue #889) Attention email externe My apologies - it does indeed require a backport to Java 8. If you have a support contract with a commercial support vendor (https://adoptium.net/en-GB/temurin/commercial-support/) then you can contact them to expedite a fix. Else this will be a case of waiting for an OpenJDK committer to volunteer a backport — Reply to this email directly, view it on GitHub<#889 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AO7435ZC3TLHLAC5LVRV373XZY4ADANCNFSM6AAAAAA4HPMBCU. You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
That code is in a shared block in HotSpot so I personally can't confirm that Windows escapes this.


Add higher-order abstractions on Lock to remove boilerplate

A pretty typical pattern is:
l.lock();
do something
do something else
l.unlock();

Observe especially how this pattern omits try...finally wrappers, which it probably should have to guarantee unlocking in the case of break, continue, return, or thrown exceptions.  For the sake of convenience, and because JS likes this types of abstraction, we might introduce:
l.perform(function() {
  do something
  do something else
})

Similar ideas are possible for Cond, but the applicability is not really obvious.

One reason I haven't just written this code (it's trivial) is that I need to investigate whether a try...finally structure materially affects performance of common cases, notably, of a critical region that does not do something that will throw an exception.  It would be good to test that in more than one browser, too.

Incorrectly closed by wrong bug reference in commit msg.

Sadly, in the SpiderMonkey shell, this (and its obvious variations):
try {
  this.lock()
  return thunk()
} finally {
  this.unlock()
}

is about 4x slower than this:
this.lock()
let r = thunk()
this.unlock()
return r

while this equivalent form is "only" 2.5x slower:
let r
try {
  this.lock()
  r = thunk()
  this.unlock()
} catch (e) {
  this.unlock()
  throw e
}
return r

How about having two variants of perform: safe and unsafe?
The safe variant would use try...finally, while the unsafe variant wouldn't do any exception handling. It would still be convenient than remembering to call unlock manually.

One could do that, but I really would prefer if try..finally wasn't that expensive.  I consider that a bug.  It's also a bug that simulating finally with try..catch is much faster than just using finally.  (And for all I know those are only problems in Firefox, I haven't investigated yet.)

And for all I know those are only problems in Firefox, I haven't investigated yet.

FWIW, I have seen the same with v8 in Chromium and Node-js, and in the profiler, the reason for de-optimising is shown as "try block".

Filip Pizlo's strawman threaded JS proposal uses the syntax lock.hold(thunk) for what's called perform above.


How to dynamically open Marker Info Window by clicking on Item in Sidebar.

Hi Sebastian,
I have a quick question regarding AGM and the SnazzyInfoWindow.
I have successfully implemented the AGM and loop through my object to position Markers on the map. I also have a sidebar where the object Name & Image will be presented.
By clicking on the Object I would like to dynamically open the marker window and zoom to it.
I know that I need to fiddle around with isOpen. I would like to use isOpen by using the index of the sidebar Items and pass it to the AGM to open the "correct" snazzyWindow.
I am a bit stuck on how to proceed.
Thanks for the help!

Hi Sebastian! I have the same question

@kirchhoffandre & @IraErshova here is my solution for the normal InfoWindow, but it should work with SnazzyWindow as well:
.html
<agm-map>
  <agm-marker *ngFor="let marker of markers"
    (markerClick)="openWindow(marker.id)"
    [latitude]="marker.lat"
    [longitude]="marker.lng">
    <agm-info-window
      [isOpen]="isInfoWindowOpen(marker.id)"
      [latitude]="marker.lat"
      [longitude]="marker.lng">InfoWindow #{{marker.id}}</agm-info-window>
  </agm-marker>
</agm-map>

<button type="button" (click)="openWindow(5)">Open InfoWindow 5</button>

.ts
openedWindow : number = 0; // alternative: array of numbers

openWindow(id) {
    this.openedWindow = id; // alternative: push to array of numbers
}

isInfoWindowOpen(id) {
    return this.openedWindow == id; // alternative: check if id is in array
}

Make sure you added the latitude and longitude attributes to <agm-marker> and <agm-info-window>. Otherwise the <agm-info-window> position would differ from the marker, when you open it by the button.
If you want to have multiple info windows open at the same time, have a look at the comments above, starting with alternative:.
You may need to add more attributes to make it work. The code is limited to explain the basic functionality.
Inspiration: https://github.com/SebastianM/angular-google-maps/issues/378#issuecomment-225368565

Hi JimmyTheNerd,
Is there same functionality in while hovering on a button ?

Hi @JimmyTheNerd
Is there same functionality in while hovering on a button ?

Try using (mouseover) instead of (click).

@JimmyTheNerd  You save my 24 hours time.. Thank you so much brother. and also thanks @majorsaaam for suggestion (mouseover).

Thanks @JimmyTheNerd


App stuck after being granted access

After logging in and granting access the app doesn't do anything. It just sits on the access granted page saying I can now close the window. Nothing happens even after 10+ minutes waiting. Force closing the app and opening it again goes back through login process and gets stuck at the same point. Also hitting back after access granted page just gives a token error page instead.

I had multiple issues about a similar behaviour.
I just published a new version which should hit the google play store tomorrow? (i hope)
In this new version i changed the login flow to no longer work with this authorization page.
I now uses the username + password to create an app passsword as promoted by the nextcloud api
https://docs.nextcloud.com/server/latest/developer_manual/client_apis/LoginFlow/index.html#converting-to-app-passwords
Please let me know if this change fixed your problem!


Chapter Move does not load book list Bookstack is accessed remotely (bookstack.mysite.com) but works when accessed locally (<IP_ADDRESS>:4444)

Issue
Chapter Move page does not load the list of books when bookstack is accessed remotely using bookstack.mysite.com. When I access bookstack from the local network via <IP_ADDRESS>:4444, the feature works as expected.
comment creation and template insertion are also affected (more details below).
Steps To Reproduce

Screenshot below of Chapter Move working locally while not working remotely illustrates the issue. similar issue occurs when I try to create comments remotely.
Reproduced issue with Chrome and Firefox. Cleared browser data in both cases to rule out browser issue.
During testing, i discovered that comment creation & template insertion will not work remotely either.
3a.Templates will not insert into a page whether you select template, append before or after, or drag and drop.
3b. comments can be typed but when the comment is submitted, the loading symbol shows endlessly as demonstrated in the screenshot and the comment is never added.

Expected behavior
See left side of attached image for expected behaviour. Note that bookstack is being accessed locally and works correctly. Right side of screenshot shows the (consistant) behaviour when bookstack is being accessed remotely.
Screenshots

Configuration

Exact BookStack Version: BookStack v0.29.3
PHP Version:  Unsure, using Linuxserver/bookstack:latest docker image
Hosting Method: Docker

Additional context
Unsure if this is relevant but I use Hera (https://github.com/aschzero/hera) to make bookstack remotely accessible through cloudflare. Could this setup be causing the issue with these specific features? do these features of bookstack have anything in common that could help me trace a problem with my setup?
Thank you,

Hi @gdmac1,
BookStack does fundamentally not support being hosted via multiple hostnames/ports. You are likely seeing this issue due to BookStack making AJAX calls over the original hostname/port. If you have an APP_URL configured, that is the url that will be used.
You'll generally encounter more issues than just AJAX requests as times goes on.

Hi @ssddanbrown,
Thank you for your reply. I had left APP_URL unset because, when I set APP_URL to bookstack.mysite.com, bookstack would not load correctly. It would break the bookstack UI, as if styling were not loading properly. I had tried other variations with https:// etc, which all caused the same styling issue.
In the end the problem was a trailing slash. https://bookstack.mysite.com/ was what I needed to set APP_URL to, to get it working.
Thank you for pointing me in the right direction! Everything is now working great.
Thank you


Fix cosmetic bugs from RC tests

Make sure inline elements like Badge and Code that rely on padding for their height don't stretch vertically in Flex/Grid layouts, similarly to how fixed-height elements work
Fix a reset regression when links would get an unintended inline-block style by default
Rework Text Field and Text Area

Make sure that the font weight is not inherited, e.g. from a wrapping <label> element
Add styles for inputs of different types, like date and search
Got rid of the internal Chrome part given good support for :has, which simplifies greatly our implementation and allows the style prop to work more predictably.
Older browser that don't support :has will render the disabled and autofilled styles as regular state; otherwise these components are still going to be fully functional.

bigger changes in width for the ones at the bottom right

Fixed, I missed how those inputs were breaking out of the parent containers thinking that I actually had fixed them shrinking too much 😅
https://github.com/radix-ui/themes/pull/291/commits/3e6208464341cd5bf7beb1d46cb2982efd01f759

There's still a slight difference when comparing to before, we used to have a 1px transparent border on the input which made the whole thing a little bigger overall. Not sure if that was a mistake before and we fixed it now by removing it or what.

Yeah it's because I re-allocated that 1px space onto the root part. I think we shouldn't worry about it, it's visible only in edge case when you force the intrinsic size of the input (e.g. via width: min-content on the root or flex column with align-items: start; and in this case it's different between browsers anyway).
Also just committed an update to fix missing submenu icon color and this regression
https://github.com/radix-ui/themes/pull/291/commits/9856253ccb4f4aca030678320fb45b629f471016
Could you check that too and let me know if you have any nits?


Espresso icon

Summary of the Pull Request
What is this about:
What is include in the PR:
How does someone test / validate:
Quality Checklist

[X] Linked issue: #11146
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[ ] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

Love it!


How to help Uberspace?

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

From the website:

We would also like to heartily thank our main website hoster Uberspace who is currently being sued in Germany for hosting our essentially business card website and who have already spent thousands of Euros in their legal defense.

How can we donate or help either the youtube-dl project or Uberspace during this lawsuit?

For Uberspace: Instead of donating you could simply become a paying customer (of course at a sustainable level, not their near-to free tier). No idea regarding youtube-dl, though...


CMakelists.txt is not configured properly for catkin

I'm getting lots of missing packages and dependencies in CMakeLists.txt.
e.g.
-- uavf: 15 messages, 0 services
CMake Error at /opt/ros/noetic/share/catkin/cmake/catkin_package.cmake:196 (message):
  catkin_package() the catkin package 'roscpp' has been find_package()-ed but
  is not listed as a build dependency in the package.xml
Call Stack (most recent call first):
  /opt/ros/noetic/share/catkin/cmake/catkin_package.cmake:102 (_catkin_package)
  uavf/CMakeLists.txt:129 (catkin_package)

-- Configuring incomplete, errors occurred!
See also "/home/ms/catkin_ws/build/CMakeFiles/CMakeOutput.log".
See also "/home/ms/catkin_ws/build/CMakeFiles/CMakeError.log".
make: *** [Makefile:320: cmake_check_build_system] Error 1
Invoking "make cmake_check_build_system" failed

https://github.com/uci-uav-forge/uavf/commit/414961cb88fff30b1844c812e2e47a68753411fb fixed


Improvements to delete videos by long press

Description
LEARNER-1288

SnackBar's visibility time increased to 5000 milliseconds
Allow whole row long tap if its downloaded i.e. make downloaded state
icon non-clickable
Fix crash where SnackBar's dismiss callback gets called twice when it
is about to dismiss and the activity finishes

Codecov Report

Merging #990 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master     #990      +/-   ##
============================================
- Coverage     16.78%   16.77%   -0.01%
  Complexity     1023     1023
============================================
  Files           477      477
  Lines         20476    20482       +6
  Branches       2359     2361       +2
============================================
  Hits           3436     3436
- Misses        16646    16652       +6
  Partials        394      394

Impacted Files
Coverage Δ
Complexity Δ

...ava/org/edx/mobile/view/CourseOutlineFragment.java
43.11% <0%> (-1.06%)
17 <0> (ø)

...edx/mobile/view/adapters/CourseOutlineAdapter.java
17.4% <0%> (-0.15%)
14 <0> (ø)

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 61ddacb...37fa6df. Read the comment docs.


add the ability to add nix applications to recipe.yml

There should be a variable like this. it can either be false or true. this variable configures if nix should be installed automatically or not.
nix: true

then
nixapps:
- nixpkgs#htop
- nixpkgs#btop
- nixpkgs#virt-manager
- nixpkgs#distrobox
- nixpkgs#toolbox

Nix is not installed by default (flatpak is), and cannot be installed in the image AFAIK so the installation should be handled after first boot, and IMO that's something for the user to do rather than us
We already had a debate about just update updating Nix applications and came to the conclusion that it's futile, since there's many different ways to install Nix stuff.
Yafti is not built to install Nix applications so this would have to be a custom screen.
Fleek is closely related to uBlue (being a part of the org), and it uses home manager. If you want some Nix packages to be available ootb you could ship a fleek.yml or home manager config in your own image and instruct users (or yourself) to use that.

You can always add such a feature to your own image, though, but with the diversity of the Nix ecosystem I doubt such a feature should ever be added to this template. Even the Nix install script is kind of on shaky ground.

I'm gonna do this myself


EC minutes for 28 May 2024

EC minutes for 28 May 2024

As a best practice I would propose that the EC establish time in our weekly meetings to approve the minutes from the previous meeting. This is a standard practice in many orgs - committee approval public minutes. The bar of having one other person approve before committing is not high enough.

I agree we should make approval more official.
For last week, I was assuming that us working on the minutes together in the google doc was enough of a "EC approval", and then the review here was to check my transcription, so not as high of a bar. This week I think I reworked some of the text in the document after the meeting, so I requested more reviews. But even then, I agree we should make the approval more explicit.
Do you think approval via the PR review mechanism here is sufficient for this week? Or should we take up the topic next week in our actual meeting?

I agree with last week being ok as is because there were not changes or additions post the meeting that I'm aware of. I do think baking it into the process of creating the minutes does two things: 1. ensures it happens in a consistent/timely manner 2. Allows for quick clarification/discussion/changes. I think it's fine to review asynch and that could speed up the vote. However I would say that we don't merge the minutes until the weekly meeting. If a majority reviews before the meeting in an  asynchronous manner that will speed things up.

So what I'm hearing is the following process:

(Ideally) Create the public meeting minutes together in the last few minutes of our meeting, summarizing our discussion and internal meeting minutes.
Open a PR with the meeting minutes (which may have some post-meeting editing), ask the rest of the EC for a PR review.
(Ideally) EC members approve the GitHub PR before the next EC meeting.
Officially approve the minutes at the next EC meeting (ideally this is quick if everyone has already approved them on GitHub)
Merge the meeting minutes PR.

Is that what you had in mind?

I agree except where 1 says ideally. I would simply say that is the required process. Any exceptions should be infrequent and explained.

Thank you @jasongrout and @Ruv7 for fleshing out a concrete process for minute keeping. 🚀 I love where y'all landed.

Minutes approved in the 4 June 2024 meeting


Next versions will be implemented as grails-data-mapping-plugin

With GORM v3.0 we can write native GORM implementation for orientdb, future work will be posted here: grails-data-mapping

now repository moved: https://github.com/eugene-kamenev/gorm-orientdb
because of new gorm project structure style


chore(actions): add matrix

Applied mkpr on .github/workflows/update_package_lock.yml
cat .github/workflows/update_package_lock.yml

:tada: This PR is included in version 1.11.0 :tada:
The release is available on:

npm package (@latest dist-tag)
v1.11.0
GitHub release

Your semantic-release bot :package::rocket:


This code looks leftover from a debugging session

See this line of code:
https://github.com/OpenVnmrJ/OpenVnmrJ/blob/0ad7471b90726f53a32eef7d3aa53085f84d9bf1/src/nvpsg/solidmpseqs.h#L461
See that it is setting n90m etc to hard coded values. Look a few lines above and notice that they were already set in a standard way.
Is the existing code leftover from a debugging session?

This code is in a function that is never used.


blind fix on cludo styles

I am not seeing these Cludo styles i'm overwriting locally, not sure why yet, so created these a bit blind, hopefully this catches them all.

Looks good!


⚠ ku-s has degraded performance

In 482829c, ku-s ($KU_API_STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 15473 ms

Resolved: ku-s performance has improved in 90dd314.


Google assistant storage of connected agents

Description:
This adds support for remembering which user agent have connected to your home assistance instance and gives support for disconnect calls as well as separate listed entities for different users.
Currently this a draft pull to keep track of any test breaking during development. Comments welcome, but code may be in flux.
Example entry for configuration.yaml (if applicable):

Checklist:

[ ] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.
[ ] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[ ] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Nice, this pull is starting to look quite clean now.
Do we need an upgrade function? The store will be empty for upgrading users. A user that have already connected would need to relink, issue a sync by refreshing in google home, explicitly do a sync request from HA. I think that is enough to mention in release notes.


Add support for find in notebook editor

For #9470

Codecov Report

Merging #9471 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #9471      +/-   ##
==========================================
- Coverage   60.91%   60.89%   -0.02%
==========================================
  Files         530      530
  Lines       28650    28650
  Branches     4344     4344
==========================================
- Hits        17452    17447       -5
- Misses      10245    10248       +3
- Partials      953      955       +2

Impacted Files
Coverage Δ

...rc/client/common/application/webPanels/webPanel.ts
13.63% <ø> (ø)
:arrow_up:

src/client/common/utils/platform.ts
64.7% <0%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.36% <0%> (-2.28%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.7% <0%> (-0.74%)
: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 bbd5152...58b3a4f. Read the comment docs.


disallow-host-ports-range: simplify CEL expressions

See: https://github.com/kyverno/policies/issues/1058

picking up this one

/assign

Policy is updated and tested. Ready to submit pull request

PR submitted-> https://github.com/kyverno/policies/pull/1129


dev: try making the benchmark script multi-threaded

Describe the enhancement request

currently the benchmark script uses a loop and a sleep mechanism to fire new txs
we can instead use Promise.all to try and connect to the RPC simultaneously
this is meant to help @greged93 debug hive tests that apparently cannot run in parallel
Describe Preferred Solution

slightly modify the benchmark typescript script to simultaneously try to fire txs or do lots of calls at the same time
share the results with @greged93

Test output:
-- http/TransactionInBlock (kakarot)
>>  {"jsonrpc":"2.0","id":1,"method":"eth_sendRawTransaction","params":["0xf8a50180830124f894000000000000000000000000000000000000031580b844ef2769ca000000000000000000000000f51d99bb140a3ed1b643df76d10cd95826eef5700000000000000000000000000000000000000000000000000de0b6b3a764000032a07f3350bad40422ff13ffb02fda09d6bab5835fe982bb18109e56a7f4b1b44d2ea014517de2ce95436dc9a4048abca4df6079546dcb7ed8761bd97087ca5193797c"]}
unable to send funding transaction: Post "http://<IP_ADDRESS>:8545/": read tcp <IP_ADDRESS>:60922-><IP_ADDRESS>:8545: read: connection reset by peer

-- http/CanonicalChain (kakarot)
>>  {"jsonrpc":"2.0","id":1,"method":"eth_getBlockByNumber","params":["latest",true]}
Unable to fetch latest block

-- http/TransactionCount (kakarot)
>>  {"jsonrpc":"2.0","id":1,"method":"eth_sendRawTransaction","params":["0xf8a50280830124f894000000000000000000000000000000000000031580b844ef2769ca00000000000000000000000023353d2e6126451c7d77f1350d1921f0f9065a960000000000000000000000000000000000000000000000000de0b6b3a764000032a092df333ed03495ebd6cb1507b7c740a59b637144936a823d1384bfd78309d7a5a0752cd1258c1dbda8ad81090d8016d1c6402cd6eaf32f9e5b81c797445a389945"]}
unable to send funding transaction: Post "http://<IP_ADDRESS>:8545/": read tcp <IP_ADDRESS>:60878-><IP_ADDRESS>:8545: read: connection reset by peer

-- http/SyncProgress (kakarot)
>>  {"jsonrpc":"2.0","id":1,"method":"eth_syncing"}
Unable to determine sync progress: Post "http://<IP_ADDRESS>:8545/": EOF

-- http/ABICall (kakarot)
>>  {"jsonrpc":"2.0","id":1,"method":"eth_call","params":[{"data":"0xa223e05d","from":"0x0000000000000000000000000000000000000000","to":"0x0000000000000000000000000000000000000314"},"pending"]}
Unable to fetch `ui` variable: Post "http://<IP_ADDRESS>:8545/": read tcp <IP_ADDRESS>:60862-><IP_ADDRESS>:8545: read: connection reset by peer

-- http/GenesisBlockByNumber (kakarot)
>>  {"jsonrpc":"2.0","id":1,"method":"eth_getBlockByNumber","params":["0x0",true]}
Unable to fetch genesis block: Post "http://<IP_ADDRESS>:8545/": dial tcp <IP_ADDRESS>:8545: connect: connection refused

-- http/CodeAt (kakarot)
>>  {"jsonrpc":"2.0","id":1,"method":"eth_getCode","params":["0x0000000000000000000000000000000000000314","0x0"]}
Could not fetch code for predeployed contract: Post "http://<IP_ADDRESS>:8545/": read tcp <IP_ADDRESS>:60910-><IP_ADDRESS>:8545: read: connection reset by peer

-- http/ContractDeploymentOutOfGas (kakarot)
>>  {"jsonrpc":"2.0","id":1,"method":"eth_sendRawTransaction","params":["0xf8a58080830124f894000000000000000000000000000000000000031580b844ef2769ca0000000000000000000000006b9accbf190ca3b5c590d214448058b0558a53e20000000000000000000000000000000000000000000000000de0b6b3a764000032a0558656b147f4f1a373d48e857cd1670124f818fbc18306a6314afc7ab7980884a029f0a0d1b2e3c4e6b1efbee7e3818a4531b30b535d95499360d417b9e2400915"]}
unable to send funding transaction: Post "http://<IP_ADDRESS>:8545/": read tcp <IP_ADDRESS>:60898-><IP_ADDRESS>:8545: read: connection reset by peer

-- http/TransactionReceipt (kakarot)
>>  {"jsonrpc":"2.0","id":1,"method":"eth_sendRawTransaction","params":["0xf8a50380830124f894000000000000000000000000000000000000031580b844ef2769ca000000000000000000000000c91e190d0ee3a191e2e8a9574216692dcb7509000000000000000000000000000000000000000000000000000de0b6b3a764000031a02848134af96c8777832bedc8bc45f835373eab00f6ba226f52b8616019b443f0a0485c2af74d282aa76bfa7b42ea4b298b06ea3fa3541e16ec94d8faf3c7004aa5"]}
unable to send funding transaction: Post "http://<IP_ADDRESS>:8545/": dial tcp <IP_ADDRESS>:8545: connect: connection refused

-- http/GenesisHeaderByNumber (kakarot)
>>  {"jsonrpc":"2.0","id":1,"method":"eth_getBlockByNumber","params":["0x0",false]}
Unable to fetch genesis block: Post "http://<IP_ADDRESS>:8545/": dial tcp <IP_ADDRESS>:8545: connect: connection refused

-- http/Receipt (kakarot)
>>  {"jsonrpc":"2.0","id":1,"method":"eth_sendRawTransaction","params":["0xf8a50480830124f894000000000000000000000000000000000000031580b844ef2769ca000000000000000000000000758033ff43af9eba9dbe45830e07a25e36fda9a70000000000000000000000000000000000000000000000000de0b6b3a764000031a0fe23bf0ff6b6d017d74a433b39ccb9e11091f0d934484a9aa6137f4af3ceca92a07208d75c7bc4d63fa05a9b009fbb40571446f620ff28298fdae63aaf69b2dc72"]}
unable to send funding transaction: Post "http://<IP_ADDRESS>:8545/": dial tcp <IP_ADDRESS>:8545: connect: connection refused

-- http/GenesisBlockByHash (kakarot)
>>  {"jsonrpc":"2.0","id":1,"method":"eth_getBlockByHash","params":["0xf24584aea2f75656cf2064042fc1660108f7438a8bc585553c32611be6155f10",true]}
Unable to fetch block f24584aea2f75656cf2064042fc1660108f7438a8bc585553c32611be6155f10: Post "http://<IP_ADDRESS>:8545/": dial tcp <IP_ADDRESS>:8545: connect: connection refused

-- http/ABITransact (kakarot)
>>  {"jsonrpc":"2.0","id":1,"method":"eth_sendRawTransaction","params":["0xf8a50880830124f894000000000000000000000000000000000000031580b844ef2769ca00000000000000000000000016efbdd6116cae96de5b8757640832f243865bd60000000000000000000000000000000000000000000000000de0b6b3a764000032a0e0f16e638928cebe3cee12950f4f5a8fcdbc1dcfbe4cf5dab50c1e9dab9e2c18a049ef0b07c96b1ad6e5fa0b5833e1178af50ca6656564c3869fdce91e6931737a"]}
unable to send funding transaction: Post "http://<IP_ADDRESS>:8545/": dial tcp <IP_ADDRESS>:8545: connect: connection refused

-- http/ContractDeployment (kakarot)
>>  {"jsonrpc":"2.0","id":1,"method":"eth_sendRawTransaction","params":["0xf8a50780830124f894000000000000000000000000000000000000031580b844ef2769ca0000000000000000000000006abf049d0cab726a34028bfe43aeb5cb0912f3330000000000000000000000000000000000000000000000000de0b6b3a764000032a05fc8015d576bc81af4951ced0f175ead33fadd0af6765616d565a59e5d3008c8a05d42efcea7a2dc5de4c2844ce9de0ba7247e51af4fdd00269ba937b89ab7f10c"]}
unable to send funding transaction: Post "http://<IP_ADDRESS>:8545/": read tcp <IP_ADDRESS>:60888-><IP_ADDRESS>:8545: read: connection reset by peer

-- http/GenesisHeaderByHash (kakarot)
>>  {"jsonrpc":"2.0","id":1,"method":"eth_getBlockByHash","params":["0xf24584aea2f75656cf2064042fc1660108f7438a8bc585553c32611be6155f10",false]}
Unable to fetch block f24584aea2f75656cf2064042fc1660108f7438a8bc585553c32611be6155f10: Post "http://<IP_ADDRESS>:8545/": dial tcp <IP_ADDRESS>:8545: connect: connection refused

-- http/EstimateGas (kakarot)
>>  {"jsonrpc":"2.0","id":1,"method":"eth_sendRawTransaction","params":["0xf8a50680830124f894000000000000000000000000000000000000031580b844ef2769ca0000000000000000000000002f354c9a50b1bc76fd59f694340570ae0b4707550000000000000000000000000000000000000000000000000de0b6b3a764000032a03b6130df23a8597cb13d34df659d3ef18a16cf0767391fec7c690bd2f874302ca0542ed2a6321e572430cf343a733d085bbd90d47f342adb4581213b36c79042dd"]}
unable to send funding transaction: Post "http://<IP_ADDRESS>:8545/": dial tcp <IP_ADDRESS>:8545: connect: connection refused

-- http/BalanceAndNonceAt (kakarot)
>>  {"jsonrpc":"2.0","id":1,"method":"eth_sendRawTransaction","params":["0xf8a50580830124f894000000000000000000000000000000000000031580b844ef2769ca0000000000000000000000003f00a24ed122b06ad990c63e4c083765f1b96ec80000000000000000000000000000000000000000000000000de0b6b3a764000031a0998312e7bae4261a9ae9ce47013bb7febbd508199393b1c690e157bac061d025a00f223a04d64892b6cd319db97af878eae2fcafc3cacb61b1cc2e9f04dd9cb9bd"]}
unable to send funding transaction: Post "http://<IP_ADDRESS>:8545/": dial tcp <IP_ADDRESS>:8545: connect: connection refused

Dockerfile output:
Creating the genesis file...
Launching Katana...
Waiting for Katana to start...

██╗  ██╗ █████╗ ████████╗ █████╗ ███╗   ██╗ █████╗
██║ ██╔╝██╔══██╗╚══██╔══╝██╔══██╗████╗  ██║██╔══██╗
█████╔╝ ███████║   ██║   ███████║██╔██╗ ██║███████║
██╔═██╗ ██╔══██║   ██║   ██╔══██║██║╚██╗██║██╔══██║
██║  ██╗██║  ██║   ██║   ██║  ██║██║ ╚████║██║  ██║
╚═╝  ╚═╝╚═╝  ╚═╝   ╚═╝   ╚═╝  ╚═╝╚═╝  ╚═══╝╚═╝  ╚═╝

PREDEPLOYED CONTRACTS
==================

| Contract        | Fee Token
| Address         | 0x49d36570d4e46f48e99674bd3fcc84644ddd6b96f7c741b1562b82f9e004dc7
| Class Hash      | 0x02a8846878b6ad1f54f6ba46f5f40e11cee755c677f130b2c4b60566c9003f1f

| Contract        | Account Contract
| Class Hash      | 0x05400e90f7e0ae78bd02c77cd75527280470e2fe19c54970dd79dc37a9d3645c

PREFUNDED ACCOUNTS
==================

| Account address |  0x326d3da43d2bf335905ea48e98e8630f589d4e028b42284e417e95fa0fd337
| Public key      |  0x22796e1e0b20cd19185398001252dbbced3066054dbbab226c1d020a7e51fad

| Account address |  0x195fa917a3daf077145b9aece2f315ba3808543d03e595be52fcb7bed9add2a
| Public key      |  0x2e5225082a276453856402ad3ed1921fd32a5b5f7ff0d723fb5f01963fdd7cf

| Account address |  0x1b44e32c8edbcada35fa7ae6f1f44bbc16657f442b7b19286812e1976c34d83
| Public key      |  0x209b4f5c516c51edc98ba1ff716c61582a36bd9f1ec77d74fd194f45abc18e4

| Account address |  0x3d03dc52e165489c910530c6e882d374b42591bb89c9b9e4a63f28fa9212ec4
| Public key      |  0x42397031114e40d63d644b2bfb08b960218900d9ae8e33e7a9e0e3a5f8b9a3c

| Account address |  0x3fff6b2ab6ec17013e5effc4de0914bd6f8b69b7e84db4d5a23f2446b3b6e4d
| Public key      |  0x1501c0282d931e940cb3efce8df72c92216feadfac0b9163cc14261f80fa3a4

| Account address |  0x426e385f30d292b6b9b563cb02f5136e8e92340d9a3d2818e9611c51cd30e85
| Public key      |  0x4eeeccfe0f96035b2576bdd3480ec340eb3150eca79dde7cfe574c573bb4be8

| Account address |  0x4a22bef67856eb9a55058aa1e66274455b23565a35379582db0821d292179d5
| Public key      |  0x3e642788efd4974adc7a73d0c0da0088ec55afeda0578ac185d60c2c0a8c243

| Account address |  0x53dfd956d5d5204506269795f00a8bc1e63910b02e3ebd31320707f25328afd
| Public key      |  0x2f99396aad3919789352397b13bf620e00a30d72e364743043651d1f9dc81a2

| Account address |  0x6a960e151a08165a04ad9fc5ebea03d117de2b134834b167a84e6c8cf4b755e
| Public key      |  0x2be7210355a6a885bc5e3a0a8d0c6668d861f72e0b80ce42a648927ac9fde8f

| Account address |  0x6f5b000cdc092f70db0c783a2c0f08b2416e269cdc3074502e002a326ce312a
| Public key      |  0x78e6e3e4a50285be0f6e8d0b8a61044033e24023df6eb95979ae4073f159ae6

ACCOUNTS SEED
=============
0

🚀 JSON-RPC server started: http://<IP_ADDRESS>:5050

{"jsonrpc":"2.0","result":0,"id":1}Launching mongo...
Launching DNA...
Launching indexer...
Launching Kakarot RPC...
{"t":{"$date":"2024-02-20T16:16:09.178+00:00"},"s":"I",  "c":"NETWORK",  "id":4915701, "ctx":"-","msg":"Initialized wire specification","attr":{"spec":{"incomingExternalClient":{"minWireVersion":0,"maxWireVersion":17},"incomingInternalClient":{"minWireVersion":0,"maxWireVersion":17},"outgoing":{"minWireVersion":6,"maxWireVersion":17},"isInternalClient":true}}}
{"t":{"$date":"2024-02-20T16:16:09.179+00:00"},"s":"I",  "c":"CONTROL",  "id":23285,   "ctx":"main","msg":"Automatically disabling TLS 1.0, to force-enable TLS 1.0 specify --sslDisabledProtocols 'none'"}
{"t":{"$date":"2024-02-20T16:16:09.179+00:00"},"s":"I",  "c":"NETWORK",  "id":4648601, "ctx":"main","msg":"Implicit TCP FastOpen unavailable. If TCP FastOpen is required, set tcpFastOpenServer, tcpFastOpenClient, and tcpFastOpenQueueSize."}
{"t":{"$date":"2024-02-20T16:16:09.179+00:00"},"s":"I",  "c":"REPL",     "id":5123008, "ctx":"main","msg":"Successfully registered PrimaryOnlyService","attr":{"service":"TenantMigrationDonorService","namespace":"config.tenantMigrationDonors"}}
{"t":{"$date":"2024-02-20T16:16:09.179+00:00"},"s":"I",  "c":"REPL",     "id":5123008, "ctx":"main","msg":"Successfully registered PrimaryOnlyService","attr":{"service":"TenantMigrationRecipientService","namespace":"config.tenantMigrationRecipients"}}
{"t":{"$date":"2024-02-20T16:16:09.179+00:00"},"s":"I",  "c":"REPL",     "id":5123008, "ctx":"main","msg":"Successfully registered PrimaryOnlyService","attr":{"service":"ShardSplitDonorService","namespace":"config.tenantSplitDonors"}}
{"t":{"$date":"2024-02-20T16:16:09.179+00:00"},"s":"I",  "c":"CONTROL",  "id":5945603, "ctx":"main","msg":"Multi threading initialized"}
{"t":{"$date":"2024-02-20T16:16:09.179+00:00"},"s":"I",  "c":"CONTROL",  "id":4615611, "ctx":"initandlisten","msg":"MongoDB starting","attr":{"pid":65,"port":27017,"dbPath":"/data/db","architecture":"64-bit","host":"5d09c171b1c2"}}
{"t":{"$date":"2024-02-20T16:16:09.179+00:00"},"s":"I",  "c":"CONTROL",  "id":23403,   "ctx":"initandlisten","msg":"Build Info","attr":{"buildInfo":{"version":"6.0.8","gitVersion":"3d84c0dd4e5d99be0d69003652313e7eaf4cdd74","openSSLVersion":"OpenSSL 3.0.11 19 Sep 2023","modules":[],"allocator":"tcmalloc","environment":{"distmod":"ubuntu2204","distarch":"aarch64","target_arch":"aarch64"}}}}
{"t":{"$date":"2024-02-20T16:16:09.179+00:00"},"s":"I",  "c":"CONTROL",  "id":51765,   "ctx":"initandlisten","msg":"Operating System","attr":{"os":{"name":"PRETTY_NAME=\"Debian GNU/Linux 12 (bookworm)\"","version":"Kernel 6.6.12-linuxkit"}}}
{"t":{"$date":"2024-02-20T16:16:09.179+00:00"},"s":"I",  "c":"CONTROL",  "id":21951,   "ctx":"initandlisten","msg":"Options set by command line","attr":{"options":{"net":{"bindIp":"<IP_ADDRESS>"},"security":{"authorization":"disabled"}}}}
{"t":{"$date":"2024-02-20T16:16:09.180+00:00"},"s":"I",  "c":"STORAGE",  "id":22315,   "ctx":"initandlisten","msg":"Opening WiredTiger","attr":{"config":"create,cache_size=1449M,session_max=33000,eviction=(threads_min=4,threads_max=4),config_base=false,statistics=(fast),log=(enabled=true,remove=true,path=journal,compressor=snappy),builtin_extension_config=(zstd=(compression_level=6)),file_manager=(close_idle_time=600,close_scan_interval=10,close_handle_minimum=2000),statistics_log=(wait=0),json_output=(error,message),verbose=[recovery_progress:1,checkpoint_progress:1,compact_progress:1,backup:0,checkpoint:0,compact:0,evict:0,history_store:0,recovery:0,rts:0,salvage:0,tiered:0,timestamp:0,transaction:0,verify:0,log:0],"}}
RPC Server running on http://<IP_ADDRESS>:8545...

thread 'tokio-runtime-worker' has overflowed its stack
fatal runtime error: stack overflow
Aborted

As far as I know the benchmark aren't used in the CI. If we truly want them to have any value, better to add them to CI, otherwise I don't really see the need to improve them.

As far as I know the benchmark aren't used in the CI. If we truly want them to have any value, better to add them to CI, otherwise I don't really see the need to improve them.

Could you create a continuous improvement EPIC to add benchmarks to the CI?

but do we want this? The benchmarks were for benchmarking Katana/Madara no? I'm not even sure the benchmarks use Kakarot.

but do we want this? The benchmarks were for benchmarking Katana/Madara no?

true! this is useless on Starknet Sepolia
we might want to add benchmarks later but not sure how


Try reconnecting with IgnoreFailureOnStart=true (with exponential backoff?)

Describe the bug
When the bot starts with IgnoreFailureOnStart=true and fails to connect somewhere, it won't attempt reconnecting there at all and the admin will have to notice or get notified about some protocol being down and restart the bot in hopes that it will connect this time.
To Reproduce
Encounter a connection problem with IgnoreFailureOnStart=true.
Expected behavior
I wish matterbridge tried reconnecting again after maybe a few seconds and if it failed connecting, it would exponentially increase the time to wait before reconnecting. I don't know what would be a good maximum time.
Screenshots/debug logs
I am not sure it's applicable.
Environment (please complete the following information):

OS: Ubuntu 18.04.4 LTS
Matterbridge version: 1.17.3 4f406b2c (I am aware of it not being the latest version, but I understood from the support chat that that doesn't matter in this case)

Additional context
I don't think config file is applicable.

GItHub suggests https://github.com/42wim/matterbridge/issues/267 as a similar issue and this could be a duplicate of it, but my problem inspiring me to open this was with Mattermost timing out rather than IRC and I can see this issue happening with any protocol.

No sorry, I'm not in favor of implementing this.
There are two options now: continue on failure or fail on failure.
If you want to do retries on failure, you can use systemd for that which will retry (also no exponential backoff: https://github.com/systemd/systemd/issues/6129) until it succeeds.
You can also make an own script that does the exponential backoff.
That said, if someone makes a PR, it's still open for discussion.


debouncer issue?

I see this was literally added yesterday. :)
Just installing vault:
Must install debouncer gem to use hiera-vault backend
so I:
puppetserver gem install debouncer (gets 0.2.2):
Internal Server Error: org.jruby.exceptions.RaiseException: (SyntaxError) /opt/puppetlabs/server/data/puppetserver/jruby-gems/gems/debouncer-0.2.2/lib/debouncer.rb:86: syntax error, unexpected tLABEL
def flush(id = EMPTY, and_join: false)
Dropping to debouncer 0.1.0:
Evaluation Error: Error while evaluating a Function Call, Function Load Error for function 'hiera_vault': undefined method `shutdown' for #Vault::Client:0x180f2b89
Not sure if this is a WIP, I'm doing something wrong, or ? :)
Thanks!

Getting the same issue

I believe the debouncer 0.2.2 problem is realted to which version of puppetserver you're using - mine is still bundled with ruby 1.9 and I get the same error.
dropping down to 0.1.0 will work, if you do one of two things:

point your puppetfile's reference to this module to commit SHA
7bf6219b3afbcb4da8d21ad160bb546eb058cdf6

which is the commit before the call to the vault gem's shutdown function was added.

you can build your own version of the vault gem from https://github.com/hashicorp/vault-ruby/commit/5e5688a0947c1c60464b7b17ff59bb623c4085ea ( or later ) since that commit/function hasn't made it into an official release yet.

fwiw, we got around this by switching Puppet to JRuby 9k.

Yep, the solution is either: Upgrade to Puppetserver 6 where JRuby has been changed to JRUBY9K by default, or change it.
Here's some example code:
  ini_setting { "Change jruby to 9k":
    ensure  => present,
    setting => 'JRUBY_JAR',
    path    => "/etc/sysconfig/puppetserver",
    key_val_separator => '=',
    section => '',
    value   => '"/opt/puppetlabs/server/apps/puppetserver/jruby-9k.jar"',
    show_diff => true,
    notify  => Class['puppetserver::service']
  }
  ->
  package { 'vault-puppetserver-gem':
    ensure   => 'present',
    name     => 'vault',
    provider => 'puppetserver_gem',
  }
  ->
  package { 'vault-puppetpath-gem':
    ensure   => 'present',
    name     => 'vault',
    provider => 'puppet_gem',
  }

Note added to Readme (8b54fbe735d6bb9862c2f2eca6853e84b3518947)


Remove pt duplicate translation

We had duplicate translation files for pt and pt-BR. We're using pt-BR, so remove pt.
(See #71)

@JohnRDOrazio I believe this is the best way to clean things up. Do we need to do anything on the Weblate side before or after I merge this?

Yes, I believe this should be done from weblate, since weblate all had it's own internal database with the translation information. I'm not sure what works happen if you just do it from here: would weblate recreate it again?
The sure way is to remove the translation from weblate. And that will be reflected in the next PR from weblate.

I suppose we can test to see what happens: go ahead and pull this, and we'll see how weblate responds. If it re-creates the pt translation, then we know that we just have to do it from weblate.

This actually seems to have worked, the pt translation is gone from weblate.


[Webpack] Chunk for worker file is not created

When using webpack, the chunk that should be populated for the worker.js file does not exist and the dynamic load of the worker fails when pdfjs is used.
Also, when webpack finishes I have a chunk that contains the following:
webpackJsonp([1],{

/***/ 92:
/***/ function(module, exports) {

	/* (ignored) */

/***/ }

});

I have no idea, what that means.
The version 1.4.x has this problem, version 1.3.x works ok.
Note: we are using pdfjs-dist npm package.

regression from #7150, probably webpack is looking into "browser" section as well (?)

Looks like the complexity of the webpack solution does not let us to introduce browserify support and also complicated. We may need to redo the webpack the same way we support browserify and system.js.

Blocked by https://github.com/defunctzombie/package-browser-field-spec/issues/3

I am maybe lost in the code of pdfjs but how does browser field relate to this bug? (Out of curiosity)

I also ran to this issue and able to workaround by pinning the version to 1.4.149.

@redhead right, now we are trying to support all available packagers (without any preference). Since there is no good support for our use case from any of frameworks vendors, I'm trying to add a workarounds. So far:

webpack requires to add "entry?.." loader and "node-ensure"
browserify does not support workers and barfs on "entry?name=[hash]-worker.js!./pdf.worker.js"
almond has its own quicks, see #7185

So far webpack solution looks most ugliest.
When we will find from defunctzombie/package-browser-field-spec#3 what is wrong browserify or webpack, we will make a decision to revert #7150 or make obsolete webpack support because its complexity. Please provide more information on  the subject so we will make a right decision. Also find me on IRC irc.mozilla.org/pdfjs

Well, I merely want to use pdf.js to render pdfs in browser and we use webpack for bundling our application. Webpack was very helpful at not loading the pdf.js script when PDF rendering was not yet needed in our app. Just when the user wants to view a PDF, we dynamically require (using require.ensure) the 'pdf-dist' and then do stuff to render the file.
For sure it is because of the browser field, that you mentioned according to my tests.
I do not know how browserify works, but I doubt it should be resolved by the browser field. Because that is supposed to be working across bundlers (so if it disables loading for one, it disables loading for others).

I even tried to bundle and load the worker.js file my own way as:
require.ensure([], function () {
	var pdfjsLib = require('pdfjs-dist');
	pdfjsLib.PDFJS.workerSrc = require('file!pdfjs-dist/build/pdf.worker.js')
	pdfjsLib.getDocument(...);
            ...
})

Which should be good, but again it fails because of that browser field (if I remove it, it works). I cannot think of any other way of how to bundle the worker and load it using webpack, so it works.

I just removed completely webpack support from pdf.js and left it manual (setting workerSrc as above) and removed the lines from browser field, this is the most clean version of bundling I've yet got from webpack (in previous build, even on 1.3.x, it bundled the worker file twice for some reason, one of the files was never being loaded by any script).

it bundled the worker file twice for some reason, one of the files was never being loaded by any script).

Ideally it shall be bundled twice: one for use with worker and other one for PDFJS.disableWorker support (e.g. for IE9); or use the same bundle from two places. But as you said, the goal is the pdf.js shall not be loaded until it's needed and it is wrong if a packager includes pdf.worker code in the main bundle.

Oh I see the point with the 2 files (in case of IE9, pdfjs inserts a <script> with the worker file to the page?).
Webpack however does not bundle pdfjs (nor the worker) to the main bundle file but it creates two extra chunks (?) with the same worker code. One is loaded as a web worker, the other never I guess (I don't know how the IE9 support works if it uses the second file).
Anyway, in 1.3.x it worked as expected loading the files on-demand when needed. The current browser field disables that now though.

in case of IE9, pdfjs inserts a

Pin the version to 1.4.149 can workaround this issue.

I am having the same issue here: worker file is ignored (worse than that, worker is an empty object). I thought it misusing webpack.

The worker is not bundled automatically after #7189. Please add workerSrc setting and bundling procedure for pdf.worker.entry.js. See https://github.com/mozilla/pdf.js/blob/master/examples/webpack/main.js#L11 and https://github.com/mozilla/pdf.js/blob/master/examples/webpack/webpack.config.js#L8 .

@redhead Do you have async loading functional again after this change? I have pretty much the same configuration as you but can't get it to pull in the worker correctly.

@andrewgleave Didn't try yet, but I think it should work since workerSrc is just a URL to the actual file which is loaded by pdfjs as worker dynamically. You just need to ensure webpack does not bundle the file into your bundle by using the file loader, for example, to get the file path only (and thus create a chunk out of that worker file).

Yeah, I got it working. Thanks.


Add installation steps to Anaconda users

Document installation steps for https://anaconda.org/anaconda/cupy

https://github.com/cupy/cupy/pull/3052


How to deprecate / un-promote package that was promoted to a view for Maven?

Is it not possible? I am only allowed to promote but not unpromote / deprecate.
See screenshots:

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

ID: 31a8d31b-229b-7383-1cc0-f3d7c65eca84
Version Independent ID: 70ba7acf-adae-e1b7-a42d-3f1344c5293c
Content: Delete and recover packages - Azure Artifacts
Content Source: docs/artifacts/how-to/delete-and-recover-packages.md
Product: devops
Technology: devops-artifacts
GitHub Login: @chasewilson
Microsoft Alias: phwilson

@soccerjoshj07 Thank you for the feedback. Package demotion is not supported. I added a note for more clarification. The updated article will go live shortly.


Select word edge case on Windows, Linux

I did some rough investigation and, from the code, I suspect that the edge case I encountered in https://github.com/knausj85/knausj_talon/issues/499 is also present on Windows and Linux.
Linux:

Windows:

The above PR addresses this, closing.


NodeUpdateTransaction fails with INVALID_GOSSIP_CA_CERTIFICATE

When creating a NodeUpdateTransaction that does not include a GossipCaCertificate, an error is returned from the network. This happens only for the multi signer flow. If the gossip cert is not set, it is assumed null. But when the transaction is deserialized from bytes to a transaction (which happens twice during the multi signer flow), the cert is now set as an empty array instead of null. The network will throw an error if the cert is just an empty array.
In a single signer flow, the cert is set as null, but no additional deserialization occurs. The transaction bytes are sent to the network and the cert appears to still be set as null.
There appears to be a difference in the deserialization processes.

fixed in #552 along with an SDK fix


Minor improvements needed on downloads page

The feature added to the download page with PR #29  has a bug. Download sections can be toggle from anywhere in the list, but the list content does not toggle even if the +/- icon changes its current state as a result of interaction.
In addition, text selection is active for +/- icon.
It might be better to disable it.
Tried on Google Chrome, Firefox, and Safari.
And just Safari has this problem.

Microsoft Edge seems to work properly.

Website is rewritten, new version is not has this issue.

Here is the rewrite commit: 923095bfe27a3f171a34c29f00162c0f5bfa488a


Set up a continuous integration build

Do we want to use Travis, or something else?
We should decide what matrix of versions we want to support. I suppose if we're aiming to have this included in psycopg.extras we'll need to at least cover the same matrix of Python and Postgres versions that psycopg supports?
We may also want a third axis for different psycopg versions while this is a separate library?

Travis build is running green for Python 2.7 and 3.6 at https://travis-ci.org/asqui/psycopg-nestedtransactions.
Matrix needs to be expanded further...


Changed fake user passwords to their hash

Q
A

Is bugfix?
Yes

New feature?
No

Fixed issues

Review requested !


Improving calendar modal form

Hi @davish, first of all thank you for creating this plugin and for your work :clap:
I think I should have opened an issue before making this pull request, but I thought it was a bit daring to ask you for something more than what you've already done. Here I leave you some changes that I made to the modal, reorganizing the UI a bit and adding some style.
Before

After

I make the pull request in case you like how it looks, if not or you plan to make other changes, no problem. :)
Regards

Hahaha I just saw the https://github.com/davish/obsidian-full-calendar/pull/113, and I like much more that this version.