jaxws-maven-plugin with xauthFile fails because of space in path

Using the following configuration fails when the project is located in a path containing a space.
<plugin>
	<groupId>com.sun.xml.ws</groupId>
	<artifactId>jaxws-maven-plugin</artifactId>
	<version>2.3.2</version>
	<executions>
		<execution>
			<goals>
				<goal>wsimport</goal>
			</goals>
			<configuration>
				<wsdlUrls>***</wsdlUrls>
				<xauthFile>WSDL_login.txt</xauthFile>
			</configuration>
		</execution>
	</executions>
</plugin>

Maven information log indicates that the plugin tries to run: jaxws:wsimport args: [-keep, -s, '<myProjectPath>/target/generated-sources/wsimport', -d, '<myProjectPath>/target/classes', -encoding, UTF-8, -Xnocompile, -Xauthfile, <myProjectPath>/WSDL_login.txt, "<myUrl>"]. (I have redacted my project path and url.)
Note the lack of quotes to protect the argument to -Xauthfile. The command thus fails because it can’t locate the file.

fixed by e9d3f48


add an option to set path to mpv and mediainfo(gui)

for people like me who already have both installed

I'll add this to my to do list

I wanted to update, I'll be starting work on this sometime this evening/tomorrow. I'll add support to choose the path to most of the included apps

Posting back, features has been added in v3.1
https://github.com/jlw4049/FFMPEG-Audio-Encoder/releases/tag/3.1


Enable constants

I'm creatig a main configuration file at Cue where I define all fields and also set the values by default. Then, the user will have to use an user configuration file to custom the configuration by default.
The great advantage is safety, since the main configuration have safe values. In addition, I avoid that the user can change any value.
At this context, it would be very useful that a constant defined into a file, can not be changed later.
A good case for constants is the name and version of an application, when you need to create/access to files whose names have any of those data (i.e. into a log file).

Technically FOO: "bar" is the constant "FOO" with value "bar".
While official constant support might be nice I'm inclined to think it wouldn't mesh well, given lattice construct of the language.

@tredoe All values are already final in CUE. Can you give an example of what you would like to achieve with constants in addition to this? I’m not sure I understand your use case.

@mpvl An example would be: to define version, description and program name, which could be used into a CLI, to show such information. So, you have a main file with that information and other one to be edited by a third-developer/user where he could set all information excep constants (version, description, program name, etc).

If another developer tried to also set the constants, they would introduce an error (version would be ⊥), and currently its up to the configuration-consuming program to deal with this error. Hopefully the 3rd party will try out their changes before handing them down :)
"if one sees a concrete value for a field, it is guaranteed that this will be the final result" (https://cuelang.org/docs/concepts/logic/#relation-to-inheritance).

@tredoe With an example I meant an example in CUE what is not working for you and what you wish to accomplish. As other point out, values are always constant, so I'm not sure what you mean. For instance:
version: "foo"  // this cannot be changed

a user cannot override this.
The only thing one could do is provide a default value
version: *"foo" | string

if a user should be able to override it, but without this being explicit, a value is final/constant.

I was using string | *"foo" to indicate the type; by that reason, it could be overrided.
So it was working as it's supposed to be. Sorry!

Np, it’s a new concept and takes some getting used to. You could write string & “foo” to accomplish this, although it is not necessary as the type is already implied by ”foo”.

This issue has been migrated to https://github.com/cue-lang/cue/issues/110.
For more details about CUE's migration to a new home, please see https://github.com/cue-lang/cue/issues/1078.


Option to change Supabase URL for Supabase Studio

Feature request
Is your feature request related to a problem? Please describe.
I want to setup an ephemeral development environment with Gitpod to speed up our workflow. With Gitpod, everything runs in a cloud instance with Docker access. Ports are made available through subdomains: localhost:54321➡https://54321-WORKSPACE.gitpod.io. Accessing Supabase Studio gets stuck after selecting the Default Project because it is configured to search on localhost. The browser console shows failed requests to http://localhost:54321/rest/v1/.
Describe the solution you'd like
I would love to configure the Supabase URL for my local Supabase Studio. An option for the supabase start command would be great. Maybe `supabase start --supabase-url=https://custom-domain.com
Describe alternatives you've considered
You can make this work when using Gitpods VS-Code extension. This way, all the ports are forwarded through localhost and the assumptions of the CLI still stand. This adds some friction to the process, especially when using Gitpod for reviewing PRs or onboarding people.

I believe this is the same issue as https://github.com/supabase/cli/issues/270. Will close this as dupe.


Train a tensorflow model using 1 Terabyte swap file

Hi there,
i want to train a tensorflow model that is huge and require alot of memory.
So i increased the swap file size and swapiness, but in training it also uses the ram so it fails.
Is there a way to train tensorflow with using swap file as memory?

@deepseek
This question is better asked on StackOverflow since it is not a bug or feature request. There is also a larger community that reads questions there and provide better and faster support for such issues. Thanks!

@ravikyram i already did, but it is still a very important question to answer here, especially when dealing with a huge model and data that requires Terabytes of ram, using an m.2 storage which is very fast, dealing with large scale data requires using swap file only without ram in training.
What if i want Tensorflow to use only the swap file as main source of memory?

@ravikyram  i already did,
But this is a very serious question to answer, when dealing which large scale data, and i mean real big data that requires Terabytes of RAM. Corporations trying to build a huge model at that scale, it's vital to shift training from using the actual RAM, and instead use the swap. Note that my storage is fast M.2
Answering this question is a serious matter

There are several tricks you can try, including decreasing batch size and looking into mixed precision, but the community on Stack Overflow is better prepared to answer and advise. If there are specific bugs or feature requests that come out of this, please open a new Issue.

@karmel it's not about mixed precision, it's about using the system storage as memory instead of ram.


Parse mdtest results

The mdtest results should be included with the parser to give a more complete picture of the results from all tests that are run.
Closes #15
Signed-Off-By: Robert Clark<EMAIL_ADDRESS>

Given how extensive the mdtest output is, I only include it with the textual output from the parser which is visible while passing the --verbose flag. Here's an example from a DGX Station:
Systems tested: 1
Aggregate Read Bandwidth: 1.595  GB/s
Aggregate Write Bandwidth: 1.232  GB/s
Aggregate Read IOPS: 136.5 k IOPS
Aggregate Write IOPS: 135.0 k IOPS
NCCL Max Bus Bandwidth: 79.865 at 512.0 MB
Mdtest
    Directory creation: 71406.29550000001 ops
    Directory stat: 2698234.1525 ops
    Directory removal: 16016.5275 ops
    File creation: 137218.586 ops
    File stat: 2705405.084 ops
    File read: 2230275.9365 ops
    File removal:<PHONE_NUMBER> ops
    Tree creation: 1546.792 ops
    Tree removal: 5878.747 ops

DALI Standard 800x600
    Min Speed: 2509.35 images/second (0.727 GB/s)
    Avg Speed: 2694.595 images/second (0.78 GB/s)
DALI Standard 3840x2160
    Min Speed: 344.078 images/second (1.712 GB/s)
    Avg Speed: 430.854 images/second (2.144 GB/s)
DALI TFRecord 800x600
    Min Speed: 2508.069 images/second (0.726 GB/s)
    Avg Speed: 2665.653 images/second (0.772 GB/s)
DALI TFRecord 3840x2160
    Min Speed: 317.276 images/second (1.579 GB/s)
    Avg Speed: 376.862 images/second (1.875 GB/s)

It's good enough for now, definitely. We can expand as the data is more closely assessed over time - might be able to distill down to a minimal criteria for what we look at from mdtest.


is JQuery UI a dependency for JSONedit?

I want to use JSONedit, but I don't have JQuery UI in my project and I don't want to use it,  is JQuery UI a necessity for JSONedit?

Yes, as you can see in the README, it's a dependency. This is due to angular-ui-sortable which depends on jQuery UI.


Featured content container in .NET Home page doesn't render properly

In the .NET Home page, the FEATURED CONTENT section renders outside the page:

 Featured Content

Get started with .NET Core

Get started with ASP.NET Core

Removing width: 100%; rule from the container class solves the issue!!
The issue occurs in Firefox 48 and Chromium 52.0.2743.116 (Developer Build). OS: Ubuntu 16.04 (64-bit)

I could repro that.
@DuncanmaMSFT @adkinn can you please take a look? I'll be filing a bug internally to track this.

@badersur can you test this again? The engineering team resolved our internal bug as fixed and we also have a new home page since last month. Thanks!

@mairaw Your new home page is awesome! Thanks for letting me know about it :)

You're welcome! I'm glad you like it. I'm closing this issue then.


Ubuntu 14.04 LTS 'python setup.py install' fails

mike@mike-test2:~/Ice$ python setup.py install
Traceback (most recent call last):
  File "setup.py", line 72, in <module>
    requirements = [str(ir.req) for ir in parse_requirements('requirements.txt')]
  File "/usr/local/lib/python2.7/dist-packages/pip/req/req_file.py", line 19, in parse_requirements
    "parse_requirements() missing 1 required keyword argument: "
TypeError: parse_requirements() missing 1 required keyword argument: 'session'
mike@mike-test2:~/Ice$

Contents of requirements:
psutil==2.1.1
appdirs==1.2.0
pysteam>=0.1.3

Results of sudo easy_install <target>
Searching for psutil
Best match: psutil 2.2.0
Processing psutil-2.2.0-py2.7-linux-x86_64.egg
psutil 2.2.0 is already the active version in easy-install.pth

Searching for appdirs
Best match: appdirs 1.4.0
Processing appdirs-1.4.0-py2.7.egg
appdirs 1.4.0 is already the active version in easy-install.pth

Processing pysteam
Writing /home/mike/Ice/pysteam/setup.cfg
Running setup.py -q bdist_egg --dist-dir /home/mike/Ice/pysteam/egg-dist-tmp-mRs8JM
removing '/usr/local/lib/python2.7/dist-packages/pysteam-0.1.3-py2.7.egg' (and everything under it)
creating /usr/local/lib/python2.7/dist-packages/pysteam-0.1.3-py2.7.egg
Extracting pysteam-0.1.3-py2.7.egg to /usr/local/lib/python2.7/dist-packages
pysteam 0.1.3 is already the active version in easy-install.pth

Ice runs after I reran pysteam through these methods, but this is mighty confusing to folks who have no experience with this. Although the script noted it added my 2 SNES ROMs, there are not to be found in Steam. Steam was not running at the time.
INFO	 =========== Starting Ice ===========
INFO	 Detected Emulator: mednafen
WARNING	 [DISABLED] No emulator provided for console `Nintendo Entertainment System`
INFO	 Detected Console: Super Nintendo => mednafen
WARNING	 [DISABLED] No emulator provided for console `Nintendo 64`
WARNING	 [DISABLED] No emulator provided for console `Nintendo Gamecube`
WARNING	 [DISABLED] No emulator provided for console `Nintendo Wii`
WARNING	 [DISABLED] No emulator provided for console `Playstation 1`
WARNING	 [DISABLED] No emulator provided for console `Playstation 2`
WARNING	 [DISABLED] No emulator provided for console `Sega Genesis`
WARNING	 [DISABLED] No emulator provided for console `Sega Dreamcast`
WARNING	 [DISABLED] No emulator provided for console `Nintendo Gameboy`
WARNING	 [DISABLED] No emulator provided for console `Gameboy Advance`
WARNING	 [DISABLED] No emulator provided for console `Nintendo DS`
INFO	 =========== User: 21885827 ===========
INFO	 Adding ROM: `Kirby's Dream Land 3`
INFO	 Adding ROM: `Gokujou Parodius (Japan)`
INFO	 No image found for `Kirby's Dream Land 3`
INFO	 No image found for `Gokujou Parodius (Japan)`

Close the window, or hit enter to exit...

Hey ProfessorKaos64. I think I fixed the saving issue yesterday, try pulling the most recent version of Ice.
The install issue is pretty troubling though. You are the second person who has had that issue in recent memory, and I have absolutely no idea why. Its interesting that it worked when you just easy_installd though. I'll look into this.

Welp, google is my friend.
http://stackoverflow.com/questions/27869152/heroku-typeerror-parse-requirements-missing-1-required-keyword-argument-ses
Looks like a newer version of pip requires adding this keyword. Would you mind running pip --version and posting your output here to confirm?
I'll put a fix up soon.

Ok, I'll check on this when I get home or did you commit what was needed anyway?

I committed anyway, but would you mind checking to confirm? Environment issues are always tricky, and the only environment I can test on is OS X so it would be super useful to see if my fix worked for you as well.

Hi @scottrice, things work fine now. python setup.py install requires sudo access it seems. Is that documented? The instructions for this project are in need of some work, but I should be good now. Thank you.

That's definitely true. Keeping the readme up to date and wih good information is hard. Maybe I should use the wiki more.
With regards to sudo access - is that something you need to do 'pip install' normally or is that specific to Ice? Ice shouldn't need anything more than pip does, so if it's the later that's worrying.

I had to use sudo for the python setup.py install command or write access was denied for the installs. Ice itself was fine.

Sorry, I should clarify. Do you need sudo access to do 'pip install appdirs'? Or for 'pip install psutil'? Or does it only require sudo access when you do it through 'python setup.py install'

I did not use pip, I used easy_install per the readme suggestion. I used sudo just for python setup.py install


Multi-server (slash) commands

During the day I'm logged in into multiple mattermost servers. This means that when I want to change my status I have to click on each server and set my status to the desired state. Doing this every start and end of a meeting is not fun.
It would be great if it was possible to change my status in multiple servers at once. Either via a slash command or a UI widget in the application wrapper.
Maybe this is already possible with some javascript in the application wrapper console?

Closing as the slash commands are currently controlled by each individual Mattermost server, so we can't actually synchronize these unless the servers are linked.
We could look into something like this in the future when the Desktop App no longer relies on using the webapp within a BrowserView


Feedback on chapter 13 - RISC-V N-Trace (Nexus-based Trace) Specification

None of the sections here seem to qualify as normative specifications. These could be removed or moved to a non-normative appendix. If this is intended to help the TG in future these could be moved into a separate TODO document in the Github.
Section 13.1 is very sparse. Either add more details or remove this. This section is not adding any normative specification value.
Section 13.2 This could be removed, or maybe add as non-normative comment in I-CNT section
Section 13.3 - Table 27 - This table does not have any normative specification. This could be removed.

I am closing all N-Trace PDF related issues with same comments as all issues were handled via comments/discussions in SINGLE Google Docs. Relevant links are as follows:
Notes to N-Trace PDF:   https://docs.google.com/document/d/1h__c0Kc7TQAWMh5bw9cNC9bl_IGqyY_ylPV14uc2xj0
N-Trace PDF rc20:       https://github.com/riscv-non-isa/tg-nexus-trace/commit/221f6b1cef94c5f503554e0cb0deb4046dd49686
N-Trace for ARC review: https://github.com/riscv-non-isa/tg-nexus-trace/commit/1de77dcff1a75197232f72b8fe6863892da0cc74


Exposed working directory doesn't get written to from built container

With normal docker/docker-compose if I copy a working directory and then write to that location from my running container, those writes are reflected in my local directory.
This doesn't seem to happen w/ Tilt which is shame b/c it helps with development quite a bit.
One simple example is having a Golang app that uses modules. Normally I could use a (blank) go.mod file and have it be written to with all the app's dependencies once the app and container has been built; the same with go.sum etc...
Is there a way we can have this same behavior w/ Tilt?
Example Dockerfile:
FROM golang:1.12.0

ENV GOWORKDIR /go/src/com/example/app

ENV GO111MODULE=on

COPY ./ $GOWORKDIR
WORKDIR $GOWORKDIR

ENTRYPOINT app

In my local directory that I copy from I have a go.mod that doesn't get written to.

Thanks for the issue @ozburo. I assume you're talking about Volumes here right?
I recall we do something off-label with DC Volumes, because of how Tilt's internals work, but I don't remember exactly what. I'll mark this issue as a feature request, and go dig a bit into our existing behavior to find out where we're at. More soon!

Thanks for taking a look at this @maiamcc :)

I assume you're talking about Volumes here right?
No, not in this particular instance. However, I haven't looked much into using Volumes yet so can't comment on whether that's related or a workaround etc.

...Oh, then I'm confused about what Docker Compose behavior you're talking about. Can you paste your docker-compose.yml?

You know what, I think I was mistaken! Yes I am was using volumes in docker-compose -- I thought you meant using volumes in Kubernetes.
Here's my sample docker-compose.yaml:
version: '3'
services:
  app:
    build:
      context: ./
    container_name: my-app
    volumes:
      - .:/go/src/com/myapp
    ports:
      - 80:80

You know what, this could totally be user error on my part and not understanding Tilt/K8s properly.
For clarification, the above docker-compose.yaml was how I used to develop before moving to Tilt, I am not using Tilt's docker-compose functionality (https://docs.tilt.dev/docker_compose.html) with that docker-compose file posted above but rather starting from scratch with a fresh Tiltfile that builds my  Dockerfile (the one in my first post).
So maybe my issue is that I haven't created an exposed volume like we do w/ docker-compose? Or perhaps I should use a docker-compose file w/ Tilt to get this behavior instead?
Sorry for my confusion!

...oh! Yeah, that would do it :P
So Tilt has no way to mount volumes like Docker Compose does, i.e. you can make a change in the container and it propagates back to your local filesystem -- but other people have definitely requested it!
I'd recommend you give Tilt a shot with Docker Compose mode and see if that works for your use case -- and if not, give us a ping/file another issue and we'll look into adding something similar for k8s.

@ozburo I'm going to go ahead and close this ticket, but please reach out if you hit any trouble with Docker Compose, or if it doesn't suit your use case.
In the meantime, I opened #1956 for "support for volumes via k8s" -- definitely chime in over there if this is behavior you need on k8s.


Add more lorem ipsum sub-libraries

a la: https://github.com/stympy/faker
maybe like hipster ipsum or something

Of course, thank you very much for the collaboration.
You can see them in the new versions


Add --show-layout flag

This flag displays the current keyboard layout next to the time.
Intends to close https://github.com/pavanjadhaw/betterlockscreen/issues/107
Very small changes in order to display the keyboard layout on the lockscreen as cleanly as possible. The perfect way to do it would be to change the size of the rectangle on update(), but that is way too complex and unnecessary for an option like this in my opinion.
Another solution would be to use setxkbmap -query in order to always get two-letters code (us, fr, cn, ru, ...) and not i3lock's --keylayout option that displays (us, french, chinese, ...). Let me know what you think.

I think that i3lock's --keylayout option is not live, meaning that it will not update the label if the layout is changed while the screen is locked

I think that i3lock's --keylayout option is not live, meaning that it will not update the label if the layout is changed while the screen is locked

In https://github.com/Raymo111/i3lock-color it should be dynamic, see https://github.com/Raymo111/i3lock-color/pull/167

Thanks for your contribution, this will be resolved with the merge of the multi-monitor branch from multilockscreen-fork :)

Well, turns out I completely missed the PR approval notification :man_facepalming:
Thanks for the heads-up @SebTM and keep up the good work! I am still using betterlockscreen daily

No Problem, thanks for your support 👍🏻


Heyy! So I am getting this error whenever I load minecraft, and beacause of it, the texture for my item "ruby" is not coming

[13:23:27] [Worker-Main-13/WARN] (Minecraft) Unable to load model: 'copyoftutorial:ruby#inventory' referenced from: copyoftutorial:ruby#inventory: java.io.FileNotFoundException: copyoftutorial:models/item/ruby.json

You need to create assets/models/item/ruby.json.

You need to create assets/models/item/ruby.json.

Thanks! You're series has been really helpfull. I hope there are more
advanced episodes to come.
Oh I was wondering.. How do I compile my fabric project into one .jar file.
On Tue 2 Mar, 2021, 2:33 AM Thomas<EMAIL_ADDRESS>wrote:

Closed #1
https://github.com/TechnoVisionDev/Fabric-Modding-Tutorial-1.16.4/issues/1
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TechnoVisionDev/Fabric-Modding-Tutorial-1.16.4/issues/1#event-4391948652,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASK7YMZOOD6BELSA7XG56IDTBP6JLANCNFSM4VSU6W6A
.


Given it might not be used at all, we might consider lazy loading it.

@Mrazator Thank you for your input regarding the fallback fonts like Xiaolai, which exceed 100 kB. I have created a pull request to implement lazy loading for these font face definitions, ensuring they are only loaded when necessary.
Could you please take a look and provide your feedback?

Actually, it might be non-trivial, as ideally we would separate only fallback fonts (Xiaolai as of now) into separate chunk/s, register them lazily, and just afterward trigger the manual font face loading + onLoaded to re-render affected text elements.
Why separate only fallback fonts to be lazy-loadable? Ideally, font registration for default does not rely on fetching other chunks, as it might prolong the time of displaying the default fonts incorrectly. For fallback fonts we IMO could afford it, though it might take a bit longer to load CJK fonts, so still not ideal.

Understood, separating fallback fonts into separate chunks and loading them lazily after default fonts sounds reasonable, especially to prevent delays in default font rendering. I see the challenge with CJK fonts potentially taking longer to load as well.

@Mrazator , Just wanted to know that considering that Xiaolai is self-hosted and relying on DOM APIs could break server-side use cases, would it make sense to fully leverage the FontFace API for lazy loading the fonts? I can shift the logic to use the FontFace API to ensure compatibility with both SSR and client-side environments.
Should I explore this approach further, or is there another method you'd prefer for implementing lazy loading?


visual studio code application is completly transparent.

Type: Feature Request
By Openning visual studio code application is completly transparent. After opening application i can see desktop screen.
VS Code version: Code 1.85.1 (0ee08df0cf4527e40edc9aa28f4b5bd38bbff2b2, 2023-12-13T09:49:37.021Z)
OS version: Windows_NT x64 10.0.22631
Modes:

Can you share a screenshot?

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Add const keyword, rename aliases to constants

WIP. coming soon. :vertical_traffic_light: :factory_worker:
Closes #250

Awesome! Thinking we should change the alias keyword in the core language to const as well, and update the ASTs/identifiers/docs to suit.

Awesome! Thinking we should change the alias keyword in the core language to const as well, and update the ASTs/identifiers/docs to suit.

Just to clarify, it looks to me like the core language didn't have an alias keyword – i've introduced the const keyword, though.


Fix config file permissions

Pre-review checklist

[x] I've confirmed that instructions included in README.md are still correct after my changes in the codebase.
[ ] I've added or updated automated unit tests to verify correctness of my new code.
[x] I've added or updated integration tests to verify correctness of my new code.
[x] I've confirmed that my changes are working by executing CLI's commands manually.
[ ] I've confirmed that my changes are up-to-date with the target branch.
[x] I've described my changes in the release notes.
[x] I've described my changes in the section below.

Changes description
Set proper permissions for automatically created config.toml

Marking as DRAFT, as

for some reason permissions are still incorrect on Windows
This is a duplicate of part of a larger task: https://snowflakecomputing.atlassian.net/browse/SNOW-1038305

Closed as covered by other PR


perf: Make PublicKey decoding lazy inside WASM

Context
Fixes #5038
Solution
Make PublicKey decoding lazy inside WASM, since PublicKey in WASM is used mostly for comparison operations (==), and comparison can be performed without decoding (see #5038 for details). This givess approximatelly 400tps increase for single peer (from ~2450 to ~2850).
Review notes (optional)
Checklist

[ ] I've read CONTRIBUTING.md.
[ ] (optional) I've written unit tests for the code changes.
[ ] All review comments have been resolved.
[ ] All CI checks pass.

Looks like this PR would fail I2::Dev::Wasm workflow. Please review latest commit with the fix


KAFKA-17783: Adding listeners to remove share partition on partition changes

The PR adds listeners to Partition class which might invoke the clean up of share partitions on Partition changes.
I considered introducing new Listener class which defines methods per SharePartitionKey but avoided as that would have required wiring up at the calls at different places in ReplicaManager. Additionally replica manager needed handling to store the new listener, etc. I found the current PR approach better as SharePartition is abstracted Shared view of Partition hence attaching the change events on Partition is helpful.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Thanks @AndrewJSchofield for review and good points.

Can the partition become a leader after it has previously become a follower? I just wonder whether the state machine is more complicated than this PR implies?

Yes, that can always happen. Once it does then next share fetch request should load the share partition. The scenario is not very different then partition being shuffled across brokers. The broker who is the leader of the partition should be able to load share partition.

The listener is called under a lock in the Partition. The listener implementation needs to be very careful not to do anything brave under that lock. I expect there is potential for a deadlock here.

The lock is for each Partition, but I do get the point. I can make the call outside lock as well, I ll wait for @junrao's comments as he can help.

@apoorvmittal10 what is the purpose of this listener? Is is so that SPM can clean up its in-memory state when it is no longer the leader for a partition?
If that's the main use case, I think we should consider tying into the metadata system directly rather than coupling ourselves to Partition.
@dajac, since its looks like you added it, maybe you can comment on the intended usage of PartitionListener?

@apoorvmittal10 what is the purpose of this listener? Is is so that SPM can clean up its in-memory state when it is no longer the leader for a partition?

Yes, that's the purpose.

If that's the main use case, I think we should consider tying into the metadata system directly rather than coupling ourselves to Partition.

So as far as I see the code there exists 1 approach taken by coordinators: in replica manager where we have a method becomeLeaderOrFollower where we pass onLeadershipChange method. The onLeadershipChange method triggers groupCoordinator and txnCordinators, onElection and onResignation methods. The 2) approach is by invoking the listeners on partition change.
I am not sure if we do get leadership change notifications from metadata system. Not sure if we could have some kind of leaderEpochChange listener to see the partition epoch bump and do some actions. But I found the listeners approach much sensible.

Thanks @mumrah, works well with me. I was more thinking to tie it up with Partition listener initially but can work either way. @dajac @junrao What do you think?

@apoorvmittal10 : Thanks for the PR. Left a comment.
Regarding whether to use a listener at the Partition level or from BrokerMetadataPublisher, it's probably better to use a Partition level listener. It's true that the source of truth of the metadata is BrokerMetadataPublisher. However, the share partition requests only interact with metadata at the Partition level. So, it's more natural to take an action to share partitions after the metadata has been reflected at the Partition level. Otherwise, what could happen is that we remove a partition after the metadata changes in BrokerMetadataPublisher, but not in Partition yet. When we try to create the new SharePartition, it's still created with the outdated leaderEpoch.

Thanks a lot for confirming. Then I will go ahead with this approach and ll add tests to the PR.
cc: @mumrah @dajac

@AndrewJSchofield @mumrah @junrao I have updated the code with tests. Also removed listener call from locked markFollower method, @AndrewJSchofield mentioned, rather moved to replicaManager itself. Please review.

@junrao @chia7712 Thanks for reviewing the changes, I have addressed the comments.

@apoorvmittal10 The listeners utilize a CopyOnWriteArrayList, meaning that both "add" and "remove" operations can trigger array copies. I am concerned that this could lead to performance issues when a partition becomes a follower, as it may result in n * (n + 1) / 2 copies when listeners are removed one by one.

@apoorvmittal10 The listeners utilize a CopyOnWriteArrayList, meaning that both "add" and "remove" operations can trigger array copies. I am concerned that this could lead to performance issues when a partition becomes a follower, as it may result in n * (n + 1) / 2 copies when listeners are removed one by one.

@chia7712 Yes, you are right it's CopyOnWriteArrayList. I considered that initially and my thought process was that listeners (CopyOnWriteArrayList) is per Partition. Hence a partition listener can contain listeners at most to the number of share groups on a broker. Yes, if the number of groups in really huge then it might cause a bit delay. Do you think that is a concern?
If yes, then I can have a single SharePartitionListener which keeps a mapping of topicPartitions to set of sharePartitionKeys.

Yes, if the number of groups in really huge then it might cause a bit delay. Do you think that is a concern?

In fact, I don't have strong evidence about the performance issue. I may run a benchmark for the scenario, but before conducting the experiment, I agree with @junrao's comment that the size is limited by the groups.
By the way, this patch is LGTM :)


WIP: Add DB-specific tests for result::affected_rows

Covered for number of DBMS, should serve as documentation of differences in returned value between database/drive implementations.

I'm surprised affected_rows test causes SQLite to segfault
https://travis-ci.org/lexicalunit/nanodbc/builds/129878504
/home/travis/build/lexicalunit/nanodbc/test/sqlite_test.cpp:42: FAILED:
due to a fatal error condition:
  SIGSEGV - Segmentation violation signal

Interestingly, only two of four SQLite build job are failing.

It is unclear where the SIGSEGV - Segmentation violation signal comes from, but it looks like the message itself is generated by Catch.
Second, the sqlite_test passes on Ubuntu 14.04 without any problems.
On Travis CI's Ubuntu 12.04, it might be failing due some critical issue in that version of the SQLite ODBC Driver.
Or, it might be a bug in Catch.
Fairly annoying.


Cannot regroup with user after reset in chat

I tried to chat with someone in Keybase app, to find out they lost all access, and they had to rekey their account.
Now we are both getting this error:
ERROR CODE 205 - uid XXXX has never been a member of this team. in method chat.1.local.addTeamMemberAfterReset

Welp?

Can you send keybase log send?

cc @mmaxim  and @zapu

Please see the following logs: c658d8cbdc64a299151e441c and 64ada39dcaf88b7e9707c21c.

Yeah this is the same problem as #10520

Thanks for the analysis, @mmaxim, I appreciate that.


Fetcher Slowmode

The fetcher's slow mode is too fast, hard to control. It's speed can be roughly half as fast. - Boan

fixed with commit: https://github.com/FTC7393/FtcRobotController/commit/a2785c48433251a6b36f8df2e9280b80089418bf


Curators can request an access token to make use of the Curation APIs

Story
Curators need to be able to use their API key to generate short-lived access tokens in order to use the Curator API resource endpoints.
 It should be Access ALL public Collections and ONLY AUTHORIZED ~THEIR OWN~ private Collections because curators with lead curator authorization can access all private collections.

@brianraymor noted!


Update dependencies

Guava 23.6 -> 24.0

checker-compat-qual 2.3.1 -> 2.3.2
Backport the changes since we forked MediaType

Revert the cosmetic changes which makes backporting take more time

Nety 4.1.20 -> 4.1.21

Fix the compilation errors due to a breaking change in their
unstable API

/cc @imasahiro @kojilin

Thanks, reviewers!


gnu-efi: Use their crosscompilation support correctly

IIUC, previously, the cross-compilation support was done in a somewhat
hacky way and was, basically, special-cased for ARM.
Now we use the cross-compilation support intergrated into their own
build system.
Test:

nix-build --arg crossSystem '(import <nixpkgs/lib>).systems.examples.musl64' '' -A gnu-efi

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Dropping ARCH=arm broke the native build on armv6l. The build system is buggy and incorrectly sets ARCH=armv6l is it is not manually specified, breaking the build. The bug is easy to fix, so I submitted a patch upstream: https://sourceforge.net/p/gnu-efi/patches/71/

See #72819


unify package imports

Purpose
closes #670
See issue for convention

I also wonder if we could make the convention more visible? E.g. in the docs or in our wiki.

I also wonder if we could make the convention more visible? E.g. in the docs or in our wiki.

Good point! I updated the Coupler Convention part of the repo: https://github.com/CliMA/ClimaCoupler.jl/wiki/Coupler-Conventions#package-import-convention


Checking that meter response data is deleted after it has been retrieved - SLIM-663

All is well.

All is well.


Customize webpack paths (src/build)

tl;dr; Are there any plans (or is there any demand) in making some paths customlizable? Like appIndexJs, appHtml or appBuild.

Context: The reason why I am asking this is that I am developing an app and wanted to use create-react-app to ease my way into working with React. Since the backend is a node server I want to share some code between client and server. Currently I have the folder setup like this:
project/
  src/
    client/
    server/
    shared/

I know about this example: https://github.com/fullstackreact/food-lookup-demo
The problem that I have with this setup is that webpack watchers would not automatically update/refresh when I change code in the shared folder. I know that I just can eject, but I want to profit from all the great updates! :)
So, I would like to change appIndexJs and appHtml to src/client/index.js/src/client/index.html.

Proposal: Add some configuration options to package.json to further configure create-react-app. This is done by other software already, e.g. ava.
This would allow (beside the "no-config" usage) to have some "advanced" configuration without ejecting. If this is something other people than me want, I would love to make a PR :)

@sebald Exactly my thoughts , Having a default yet configurable webpack would be really helpful. I am currently working on implementing this feature in my own repo (Hopefully will be done in a few hours.)
Gatsby (https://github.com/gatsbyjs/gatsby) provides this feature by leveraging Webpack configurator (https://github.com/jakubbabiak/webpack-configurator). Though i am preferring to use webpack-config(https://github.com/mdreizin/webpack-config) as it is more active.

For reference this the from gatsby documentation
https://github.com/gatsbyjs/gatsby#how-to-use-your-own-webpack-loaders

@shrynx I think this is not part of the core-ideas of create-react-app. I guess my proposal isn't either, but I wanted to ask anyway :)

Oh yes definitely, That's why i am preferring on creating my own repo for this.
But there is thread regarding plugin system too , so maybe this could be useful to.
https://github.com/facebookincubator/create-react-app/issues/670

Thanks for the link, that very interesting.
Just for reference: This is what I suggested https://github.com/sebald/create-react-app/commit/7407b19b7ec03410e8ee260101add60aa4ad0e05
Super minimal, but makes the package configurable/extensible.

Hi, thanks for the suggestion. I expect that we might look into the problem of sharing code as part of work on #741.
However we won't be adding configuration options for source and build output folder. If configuring them is important to you, we recommend that you use Webpack (or other tools) directly to have full control over your setup.


about search reference

Search reference on search reference result. + Allow to decompile it
Short description of the feature
Allow searching reference on reference result and beyond.
Allow decompiling not just bytecode. on reference when you right click it

Extra details:
no more just read verbose..

More verbose details of features

because when we search the menu context at right click is so limited. we cant perform more reference .. example if I search a reference.. then reference show. I wanted to right-click that and search for reference.. instead of finding the class and manually click the search reference there.. also allow to decompile not just assembly.

So to clarify, you want to be able to see the references to the references from a search?
This sounds a lot like how dnSpy does their searching:

As for the decompilation instead of the bytecode do you mean like this?

Search for a reference to XYZ
Shows result from ABC
Right clicking result decompiles ABC

yes yes.. like dnspy :) I use that too.  I also look for similar dnspy like but in java.. and I found yours... 👍

Currently I'm not looking to add to the 1.X branch but for when 2.X is out I'll look to add this :+1:

Added in: 6671f78b1bf2ac836202189d71527b403c4f3a33

While it's not literally the same as DnSpy, the same functionality is now supported in 2.X. I may add reference search to specific instruction types in the search window as well (class/field/method references)
I chose not to copy DnSpy's layout exactly because it didn't align well with the architecture of how the UI was designed in Recaf. I'm open to PR's if anyone's up to the challenge though.


Support Environment Variables in the docker_service library

It would be nice to be able to add Environment variables to the docker_service.
The use case here is for the aws cloudwatch logging driver, the docker daemon needs to be able to have access to the aws credentials.

Can the misc_opts do what you need?
https://github.com/chef-cookbooks/docker/blob/master/libraries/docker_service_base.rb#L69-L70

I'm not sure how docker daemon opts would help.
The solution I ended up going with was to mount a volume and use the awslogs service on the host.
I could still see adding environment variables to the service that start the docker daemon useful.

pending rebase on #865

released in v2.15.24

This should probably have been a new issue instead of being tagged onto a question about docker_service.
Try this pattern:
docker_container 'blah' do
action :run
env ['PORT=8001', 'NODE_REJECT_TLS_UNAUTHORIZED=0', 'VAULT_AUTH_DEFAULT=TOKEN']
end


Dropping / Overwriting existing labels in Prometheus Operator's objects

Hi,
We are using Argocd for deployments of our apps.
Some apps are creating Prometheus ServiceMonitors.
When the VM operator is deployed, it's creating VM-specific files, based on existing Prom Operator objects.
Problem:

VM objects have labels, which identify them as part of ArgoCD Apps deployment.
VM objects are not defined within ArgoCD Apps, so marked as OutOfSync (requires pruning)
At the end, as automated sync is on, those objects are deleted and then recreated by the operator, which leads to higher resources usage (ArgoCD + VM Operator)  as process is in a closed-loop

Example labels:
labels:
  app.kubernetes.io/component: application-controller
  app.kubernetes.io/instance: argo
  app.kubernetes.io/managed-by: Helm
  app.kubernetes.io/name: argocd-application-controller
  app.kubernetes.io/part-of: argocd
  helm.sh/chart: argo-cd-3.29.5

I've tried to set in operator's deployment
env:
    - name: VM_LABELS
      value: app.kubernetes.io/instance=-

but it didn't give any effect.
How should one proceed in such a case?

I think, that it possible to implement filter logic for labels deny list.
Something like: VM_PROMETHEUSCONVERTERLABELPREFIXES=app.kubernetes.io/,helm.sh/

New env variables for operator:
VM_FILTERPROMETHEUSCONVERTERLABELPREFIXES=filter-1,filter-2
VM_FILTERPROMETHEUSCONVERTERANNOTATIONPREFIXES=filter-1,filter-2

Allows to drop matched prefixes for labels and annotations, when object is converted by operator.

@f41gh7
using next helm chart values:
image:
  tag: 446-merge

env:
  - name: VM_FILTERPROMETHEUSCONVERTERLABELPREFIXES
    value: app.kubernetes.io/
  - name: VM_FILTERPROMETHEUSCONVERTERANNOTATIONPREFIXES
    value: meta.helm.sh/

VMRule object before :
apiVersion: operator.victoriametrics.com/v1beta1
kind: VMRule
metadata:
  annotations:
    meta.helm.sh/release-name: monitoring
    meta.helm.sh/release-namespace: monitoring
    prometheus-operator-validated: "true"
  labels:
    app: monitoring
    app.kubernetes.io/instance: monitoring
    app.kubernetes.io/managed-by: Helm
    app.kubernetes.io/part-of: monitoring
    app.kubernetes.io/version: 16.6.0
    chart: monitoring-16.6.0
    heritage: Helm
    release: monitoring
  name: monitoring-alertmanager.rules
  namespace: monitoring

After :
apiVersion: operator.victoriametrics.com/v1beta1
kind: VMRule
metadata:
  annotations:
    prometheus-operator-validated: "true"
  labels:
    app: monitoring
    chart: monitoring-16.6.0
    heritage: Helm
    release: monitoring
  name: monitoring-alertmanager.rules
  namespace: monitoring

Closing issue and thanks for quick solution.


Look into when to use React node, React Element or JSX Element as types

Describe the desired behavior
Need a unified rule for when to use React node, React Element or JSX Element as return types & propTypes
Describe the current behavior
Nearly everything is JSX Element
https://github.com/pxblue/react-native-component-library/issues/15

A ReactElement is an object with a type and props.
A ReactNode is a ReactElement | ReactFragment | string | number | boolean | ReactNode[] | undefined | null
A JSX.Element is a ReactElement that uses <any,any> for the props and type.

https://stackoverflow.com/a/58123882

Further, when we use React.FC<props> or React.FunctionComponent<props>, the interface for that is:

ReactElement vs. JSX.Element
Use ReactElement if you need to specify a specific props configuration for a return type
Use JSX.Element if you don't need to specify a specific prop configuration (or you don't care)
ReactNode
Use ReactNode if you want to support the additional types (besides a component), such as boolean, string, null, etc.
Only use this if you actually intend to support all of the types. If you just need null or undefined, you'd be better served by using JSX.Element | null | undefined

Do we need to update any react/react-native components to standardize this?

Do we need to update any react/react-native components to standardize this?

I just made a PR with minor updates for React. I’ll be going through RN tomorrow.


Metrics test Mismatch in SAT data source

In SAT datasource Heat chart Indicator school level there is mismatch found in marks column.

@PavitraIR Could you please the status of the ticket and target completion date

sure i will update @RNarmatha

@Shoaib15khan please do the retest and update the comment

@murali003 sat mapreport overall files are not generated  please check .

@Shoaib15khan the queries is not updated in the sat_queries.json, so @PavitraIR  please look into this issue.

Task :
 1.To processing sat_learning_p1_indicator_cluster query to calculate the school_id wise marks of students.

Expected Result:
 1. All students marks should be updated in the table and nifi queries as well.

@PavitraIR , @murali003  and @Shoaib15khan , Please update the status of this ticket and mentioned the target completion date.

Assigning to @Shoaib15khan for retesting.

Need to discuss with Jalaja - Shoaib

Issue with the sat marks column has been resolved

Completed


Finish the elixir builders

The elixir builders are in place but we need to review that they work and enable latest.ex

Future todo

fix https://github.com/esl/packages/pull/51/files#diff-f391dd7b318105c1b4038704eab059e08c5f1b5b3a83495835a392743237a0f3R153
fix the ubuntu/debian builder
enable latest workflow

otp_version used to build should be appended to the package name!!

review GPG signing!


diff between Check Text content and PDF Should Contain String

Hi, whats the difference between Check Text content and PDF Should Contain String? Documentation says the same thing for both and both work the same way.

You're absolutely right, that's a duplicate which is there due to ...history ;)
I will set one of those keywords to deprecated to clear it up in the next days.
I will keep the PDF Should Contain Strings


flow: Enable SYN/FIN/RST capturing

Capturing of SYN/FIN/RST timestamps
Additional detailes that are captured:
1. Calculate packet length according to SEQ on TCP flows.
2. Add support for elastic search scheme in date format for ABSynStart
3. Use accurate clock from the packet info (and not from flow-table)
4. Make extended capturing configurable via agent parameters in .yml
5. Change tests to reflect changes in the tables that we create
Related issue: https://github.com/skydive-project/skydive/issues/323

ok to test

add to whitelist

rerun tests

Hi, a first feed back about the general approach before digging deeply into the code.

I think it would be better to not add specific tcp code for latency calculation. I would prefer to add timestamp for the first and second packet seen at each layer so that we can calculate latency for different kind of protocol. FlowLayer could be modified accordingly.

The retransmission part is really interesting but I would see that as a counter of the transport layer or a specific field.

I think we should not use the config in order to use or not such feature we should use the capture api so that we can active it on-demand and for specific traffic (bpf, etc). You can have a look at the PR #299 I added TableOpts that can be use for that.

For the synData I think that the PR #299 could be use too.

What do you think ?


setting MemoryManager = false causes the came to CTD on startup

I have disabled all mods that depend on SSE Engine Fixes, so it's not caused by something else. I'm running 1.6.640. This reminds me of when the LE version would run out of memory on launch and crash.

Fixed, just set ScaleformAllocator = false


Variables named name are replaced with the result of calling their .toString method

related to #1070 and ds300/derivablejs#47
Demo: http://jsbin.com/migaxacuwe/edit?js,console
Screenshot for posterity:

This is actually a quirk to do with the name variable.
name already exists in scope, though you're able (I think) to redefine it using const (this is where my understanding gets a bit hazy). But to see the exact same behaviour, try running your code (once converted to ES5) on a page like google.com (in the browser devtools), you'll see the same result.
console.clear();
var name = {
    toString: function () {
        console.log("uh oh, I called name.toString()");
    },
};
alert(JSON.stringify(name))
console.log("name === 'undefined': " + (name === 'undefined'));

I also wonder if the TypeScript transpile of const is using var instead of const which is masking the issue. If you run the above code, but change var for const it does actually work.
Never the less, this isn't actually a JS Bin issue, but certainly an interesting edge case.
Bottom line: don't use name…which is a bit random really (but I've seen other people run into this before too).

Thanks for the detailed response. Just one more of JS's many many esoteric delights 🌈


10 picture to train character

Hello, in your project, I trained the characters(a girl) with 3 pictures and 10 pictures respectively. I found that in the results of training with 10 pictures, the faces of the characters in the pictures were processed worse. Why？

There could be multiple reasons for this. In the original textual inversion paper, we showed that adding additional images may not necessarily help in the training process. For example, if the images you add contain very different (or similar) backgrounds or if the differences between the images representing the subject are very large, this could also harm performance.
Are the new images you added relatively similar or are there significant differences between them?


Mpu9250 issue with Navio2 Firmware

Instead of buying a Navio2 board..I download Navio2 Firmware to Raspberry Pi ..
Px4 installed correctly !!
Then I integerated GPS Neo 6M with Navio2 Firmware on RPI board..
That's also worked perfectly as I changed in Px4 config of GPS to uart as default gps in navio connects thrgh spi protocol
But Mpu9520 when connected , devices is not detected in Navio2 firmware I tried i2c config ..device not detected
Is there any diff bw external mpu9520 and on board imu used in Navio2 board ??

Some of the IMU drivers are still in flux because we have newer rewritten drivers that transfer all raw data (8 kHz gyro), but only support SPI. For now you could try using the old mpu9250 on i2c by changing the mpu9250 in the board config.
https://github.com/PX4/Firmware/blob/c15efea429faa9b5a3af802bbf0008eca956b762/boards/emlid/navio2/default.cmake#L24

old mpu9250 is in src/drivers/imu/mpu9250.
new mpu9250 is in src/drivers/imu/invensense/mpu9250.

Alternatively you could try this branch. https://github.com/PX4/Firmware/pull/15421

Some of the IMU drivers are still in flux because we have newer rewritten drivers that transfer all raw data (8 kHz gyro), but only support SPI. For now you could try using the old mpu9250 on i2c by changing the mpu9250 in the board config.
https://github.com/PX4/Firmware/blob/c15efea429faa9b5a3af802bbf0008eca956b762/boards/emlid/navio2/default.cmake#L24

old mpu9250 is in src/drivers/imu/mpu9250.
new mpu9250 is in src/drivers/imu/invensense/mpu9250.

Alternatively you could try this branch. #15421

Will try and check

Some of the IMU drivers are still in flux because we have newer rewritten drivers that transfer all raw data (8 kHz gyro), but only support SPI. For now you could try using the old mpu9250 on i2c by changing the mpu9250 in the board config.
https://github.com/PX4/Firmware/blob/c15efea429faa9b5a3af802bbf0008eca956b762/boards/emlid/navio2/default.cmake#L24

old mpu9250 is in src/drivers/imu/mpu9250.
new mpu9250 is in src/drivers/imu/invensense/mpu9250.

Alternatively you could try this branch. #15421

Inboard mpu9250 files in navio supports both i2c and spi?

Some of the IMU drivers are still in flux because we have newer rewritten drivers that transfer all raw data (8 kHz gyro), but only support SPI. For now you could try using the old mpu9250 on i2c by changing the mpu9250 in the board config.
https://github.com/PX4/Firmware/blob/c15efea429faa9b5a3af802bbf0008eca956b762/boards/emlid/navio2/default.cmake#L24

old mpu9250 is in src/drivers/imu/mpu9250.
new mpu9250 is in src/drivers/imu/invensense/mpu9250.

Alternatively you could try this branch. #15421

Inboard mpu9250 files in navio supports both i2c and spi?

old mpu9250 is in src/drivers/imu/mpu9250 (supports both i2c/spi, but only low rate data and sensor side filtering)
new mpu9250 is in src/drivers/imu/invensense/mpu9250. (currently SPI only, but there's an I2C version in https://github.com/PX4/Firmware/pull/15421)


Update Azure Service Bus components to track2 SDK

Description
Move the Azure Service Bus components (binding and pubsub) to the new "track2" SDKs. Thanks a lot to @halspang for the initial work in #1648!
Thanks to the update and some refactoring work, two metadata keys for Azure Service Bus Pub/Sub are not needed anymore:

prefetchCount
maxActiveMessagesRecoveryInSec

Notes:

Because the Service Bus SDKs depended on the newer version of azcore and azidentity, I had to update the other Azure SDKs that were already in "track2". Some of the updates contained breaking changes in the APIs, which I've fixed.
This requires Go 1.18 so it depends on #1697 (which depends on #1685 itself)

Issue reference
Please reference the issue this PR will:
closes #1532
closes #1531
closes #1701

/ok-to-test

@ItalyPaleAle please fix linter issues. Also use /ok-to-test to trigger actions for conformance and certification tests (separately from this PR) to run the Service Bus Queues binding certification test.

At the moment this breaks EventHubs Binding, PubSub and Service Bus Queues Binding.
Please take a look by running the certification and conformance tests via /ok-to-test

/ok-to-test

@berndverst This is ready for review IMHO.

All tests are passing (waiting for the conformance tests to complete)
I have confirmed that both the binding and pubsub are able to recover successfully in case of failures (both pubsub and input/output bindings)

Once this is merged, I'll need to update the docs because of metadata changes, as well as our E2E tests if needed.

Eventhubs Binding conformance test is failing
Eventhubs Binding certification test is failing
Service Bus Queues Binding certification test is failing
(you can see in the results of your /ok-to-test run)

/ok-to-test

I'll look into the Service Bus binding failure tomorrow.
For Event Hubs, that's very odd because I did not touch that module besides for a minimal change that was required. However, it looked like the SDK got upgraded (which was un-intentional). I've reverted the change to the SDK and let's see if that fixes the tests.

/ok-to-test

/ok-to-test

@berndverst  I have fixed all tests that were failing because of changes in my code.
After fixing dependencies, there were 2 tests that fail.

Service Bus Queue binding: when a message fails to be processed by a handler, it's inserted back into the queue. This means that it's re-delivered later, so it may not come in order. However the conf tests were expecting messages to be in order. I think this was a problem with the tests as the re-enqueueing of the message is arguably the desired behavior
Event Hubs binding: this is still failing and it looks like it was failing before 100% of the times when running on master. See: https://github.com/dapr/components-contrib/actions/workflows/certification.yml The error in this PR is the same. The test is broken and needs to be fixed, but it should not block this PR.

(On why the Event Hubs test is failing: the test TestAzureServiceBusQueueRetriesOnError is creating 2 Event Hubs listeners with the same consumer group, which seems to be disallowed)

/ok-to-test


Adding a rocm_compute_capacbility field to the DeviceDescription class

Currently, in the DeviceDeescription class,  we store the

rocm_amdgpu_isa_version as an int member
rocm_amdgpu_gcn_arch_name as a string member

The rocm_amdgpu_isa_version int member usage, which stores integer suffix in the gfx arch (i.e 900 from "gfx900") is already flawed with the introduction of "gfx90a", and needs to dropped
The rocm_amdgpu_gcn_arch_name string usages needs to be updated, to make it similar to what is done on the CUDA side.
This PR

introduces a class RocmComputeCapability, that wraps the gcn_arch_name routine and provides helpers functions to do all the gcn_arch_name based checks, that are currently dispersed in the TF code.
adds an instance of the RocmCompuateCapability class, rocm_compute_capability, as a member of DeviceDescription
removes the rocm_amdgpu_isa_version and rocm_amdgpu_gcn_arch_name members from the DeviceDescription class and replaces their usage with rocm_compute_capability instead
replaces references to amdgpu_arch string (in XLA code), with references to rocm_compute_capability instead

This look good! Thanks @deven-amd !

https://github.com/ROCmSoftwarePlatform/frameworks-internal/issues/620


[Docs]:  Provider missing in Documentation

Object type(s)
No response
Documentation Link
https://registry.terraform.io/providers/Snowflake-Labs/snowflake/latest/docs/resources/
Description
When following the instructions at the documentation above I got this error:
Error: Invalid provider configuration
Provider "registry.terraform.io/snowflake-labs/snowflake" requires explicit configuration. Add a provider block to the root module and configure the provider's required arguments as described in the provider documentation.

In order to fix it I had to include provider, like this:
resource "snowflake_grant_privileges_to_account_role" "example" {

  provider = <provider_role>
  account_role_name = snowflake_role.db_role.name
  on_account        = true
  all_privileges    = true
  with_grant_option = true
}

References
No response
Would you like to implement a fix?

[ ] Yeah, I'll take it 😎

Hey @natashamathur. Thanks for reaching out to us.
This is the terraform behavior (in general) when you specify aliases. You can check more in https://developer.hashicorp.com/terraform/language/providers/configuration#default-provider-configurations.


Import map PDF layers

Some PDFs have multiple layers which show different features. It would be nice if Trail Sense could separately import each layer and allow them to be toggled on or off.
The current architecture does not support this, and I may run into memory constraints if I load them as stacked layers.
Maybe this can be an option at the time of import? So that the saved image contains the layers the user wants to see.

I'm not going to add this because I leverage the Android PDF renderer which does not support this and the frequency of this being needed is low.


Run build.sh as a normal user

The docker container runs as root so mounting registry and git in the container results in pollution of a users .cargo folder with root-owned artifacts. Trying to do things like cargo update will then break.
Instead run the container as a normal user.
Requires pulling the cargo install out of /root. Also the yum install step won't work when running as a normal user. When run in the test suite the container will recompile the project even though it doesn't have to (I haven't figured out why, when I docker run by hand in the test folder it works as expected (finishes compilation in less than a second because its already done).
Note: this will still write ~/.cargo/registry and ~/.cargo/git as
owned by root if they don't exist when the container runs. May be
due to -v mounts ignoring -u when the mountpoint doesn't exist.

When run in the test suite the container will recompile the project even though it doesn't have to (I haven't figured out why, when I docker run by hand in the test folder it works as expected (finishes compilation in less than a second because its already done).

Oh this is nasty, the old behaviour (tests not doing a full recompile on the second release build) is actually a bug due to the target folder being owned by root. It's supposed to clean up the target folder before every build but since the tests didn't have permissions the old target folder remained.

Thanks for spotting this. I'll take a closer look soon


Bluetooth: Adds preprocessor ifdef to reconfigure() in L2CAP tester application

Bluetooth: Adds preprocessor ifdefs around function call in l2cap tester
application depending on the KConfig option CONFIG_BT_L2CAP_ECRED
Fixes issue #40456
Signed-off-by: Johan Lundin<EMAIL_ADDRESS>

@jhedberg Fixed the two missing empty lines.

Hi,
I've miss this initially but wanted to note that idea was to have tester app always enabled all features supported. We added build time configuration only for features that otherwise cannot be tested or tune in runtime (eg LE SC only mode or EATT disable for some of L2CAP tests)...


Support to pass custom data/methods

It would be nice to pass custom methods from my component to social sharing components. Let's say I have a custom click condition or something that needs to be trigged on the network tag.
I could do something like this:
<social-sharing url="example.com" :customMethods="myCustomMethods" inline-template>
   <network @click="customMethods.doSomethingCustom()">
   [....]

im a vue newbie so maybe that's not the best approach

I think a better way of doing this is to use the event fired by the library:
this.$root.$on('social_shares_click', function (network, url) {
  // Do your thing here
});

Hello @nicolasbeauvais this doesn't really work for me.
I am trying to trigger a collapsable element outside the social-sharing component without luck.
I cannot leave the network element without a network= (otherwise the element just doesn't show up), and when I do define the network it just trigger the action of the network (in my case the email) I see the data-link property on it, but it doesn't trigger my custom method.
I do have something like this:
<template>

    <social-sharing url="example.com" inline-template>
       <network @click="$parent.doSomethingCustom()"></network>
    </social-sharing>
    <collapse :open.sync="isOpen">

My Content to hide/show

    </collapse>

</template>

and inside of the methods I defined the doSomethingCustom() to toggle the collapsable component.
Could you please let me understand what should be done in order to trigger a collapsable component outside the social-sharing component, with a click on a network element of the social-sharing?
Thanks in advance

@nicolasbeauvais never mind, I did solve the issue:)
I just used the local call, and triggered it with that event.
BTW Thank you for the nice component


[STAT] Record per query CPU utilization

We already have the latency(real time/wall clock time) stat per query, we should also have CPU time (the time CPU spends on processing instructions) stat via getrusage() per thread.
Actual CPU utilization per query = CPU time / wall clock time

Perhaps something like (thread user time + thread system time)/(wall clock
time/hardware threads)
On Tuesday, October 4, 2016, Haibin Lin, Eric<EMAIL_ADDRESS>wrote:

We already have the latency(real time/wall clock time) stat per query, we
should also have CPU time (the time CPU spends on processing instructions)
stat per query via getrusage().
Actual CPU utilization per query = CPU time / wall clock time
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cmu-db/peloton/issues/295, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABS8HClA8M2cwaEI7Q_Nv67CJpd0mjSwks5qwvQwgaJpZM4KOUII
.

--
Anthony Tomasic
Language Technologies Institute
Carnegie Mellon University
http://www.tiramisutransit.com
http://mcds.cs.cmu.edu http://mcds.cs.cmu.edu
http://www.cs.cmu.edu/~tomasic

Yes, getrusage returns both thread user time and thread system time


Improve mindreader/merger behavior when syncing a large chain from block 1

Context
Currently, the mindreader creates "one-block-files" and the merger collects them to create "merged (100) blocks files".
The reason for this two-steps architecture is for having a few mindreader processes running on HEAD and getting different versions of same block numbers when microforks are created, we want to make sure that any single block (forked out or not)  that has gone through our system (mindreader->relayer->...) gets written to a merged blocks file so it can be retrieved afterwards.
This process is NOT MEANT for going over millions of blocks in an attempt to sync/bootstrap a large chain, it is meant for providing a stronger guarantee to the clients accessing the service that any block served close to the HEAD, in real-time, will be able to be served later, even if it was forked out, so the cursors are always valid. When default settings are used to sync linearly a large chain like eos-mainnet, something is bound to fail (usually an out-of-memory error happens at some point in time and crashes the mindreader's nodeos instance into a corrupted state)
This has been discussed in #26 and on multiple other occasions.
However, we keep seeing that users try to sync the whole chain from the start directly (ex: #80) . We talked to some of our users who prefer this approach without going into the operational steps of gathering nodeos snapshots and running them in parallel, even if it takes weeks of sync time. (throwing "time" at the problem instead of operational complexity.).
Proposed behavior

Make the mindreader able to decide to produce merged blocks-files depending when it is "catching up"  (the timestamp of blocks being processed could be enough to determine that)
Make the mindreader able to flush to disk a "partially-merged" blocks-file and append to it on restart.
add a dfuseeos sync command that launches only the syncing components:

mindreader with auto-merge mode
trxdb-loader
search-indexer

Note that search-indexer uses small shards (200-blocks) by default, this will be ridiculously inefficient if it covers 128 million blocks, but does not work well with large shards close to the head. However, search-archive does not support serving different sizes of shards directly, so it cannot be run in the same single-process afterwards. A solution will have to be proposed to the user.

About point 3 of Proposed behaviors:
I'd prefer we do not push to users phases of procesing, additional config files, and templates that force them to understand ALL the complexities when they barely are getting started.
I'd prefer the thing just WORKS, and the default behavior is you have to WAIT.
We can have the programs self adjust when they know their status, wait before starting, etc.. based on catch-up mode.
--
Regarding huge indexes, the search-archive, defaults could be a moving window of 1 week or so, so search-archive, knowing its not near live, would just do nothing.
Eventually, you could start from a snapshot, and get a partial sync.. but we'll slowly get there.

Point (1) on its own without any of the other features is very useful.

flag could be like this: mergeBlockFiles=[always, auto, none]
also add a reference to a "blockmeta" service that may (or may not) be able to
state is [catching up] if:

the blocks are at least 5 minutes late, and the destination merged file already exists
the blocks are at least 5 minutes late, and a connection to a "blockmeta" service tells us that we are processing blocks that are passed the LIB
the blocks are at least 12 hours late

changed a bit the requirements for simplicity:
state is 'catching up' if:

the destination merged-block exist (ex: you have a few mindreaders and a merger, one mindreader crashed and restarted from snapshot...)
the blocks being processed are at least [x hours] old  (12h by default, it is risky to cover 'live' blocks like this, because you could end up with the wrong chain in case there is a huge reorg and the LIB does not advance in an EOS chain)

Modified the requirements again, the "destination merged-block exists" was too risky close to the head..
so now, it will produce merged block files automatically if either of these is true when the mindreader stars:

the blocks being processed are at least [x hours] old (12h by default, it is risky to cover 'live' blocks like this, because you could end up with the wrong chain in case there is a huge reorg and the LIB does not advance in an EOS chain)
The mindreader can connect to 'blockmeta' and determine the LIB. The current block num + 100 is behind the LIB (so no risk of fork)

When those two conditions become false (when close to HEAD), it will stop producing merged-blocks and go back to producing one-block-files, being careful to change its operation mode only on boundary blocks (modulo 100)

Note that the merger, if it is waiting to produce merged blocks [200,201...] it looks at the merged-blocks-storage and if it sees that file appear, it skips to the next, so it should operate well with the new mindreader behavior.
This removes the need for: a) a "catchup phase" with a specific mindreader flag and b) going through hoops to join the batch-produced blocks with the one produced from the merger.

When it skips, are there no chances of missing forks? How does it know what was in that file in order to skip to the right place, what if the merger had a 99b unknown to others?

When it skips, are there no chances of missing forks? How does it know what was in that file in order to skip to the right place, what if the merger had a 99b unknown to others?

it only skips when the merger cannot find enough one-block-files to complete the merge before a mindreader uploads the merged file in its place. A mindreader will only upload that merged file if it is "catching up" (ex: restarted with >100 blocks behind the lib or with these blocks older than 12h...)
so unless you stop the merger for a few minutes, then restart a mindreader from an earlier point in time, let it catch up, then start the merger again, you won't lose any fork. In that particular chain of event, you would just end up losing a block that got forked out, because the correct fork will be in the merged file.
The risk AND impact is minimal and gives you the valid forks.


[Azure DevOps] View Azure Repos Pull Requests from Entity

Feature Suggestion
As a user of Azure Repos I'd like to be able to see Pull Requests when I'm looking at an Entity in the Catalog
Possible Implementation
Add a table to view Azure Repos Pull Requests, this could look a lot like the Git Hub Pull Requests plugin
Context
Being able to see Pull Requests will help out teams stay on top of things and surface this information to the boarder user base

The backend changes to support this were done in #7532 and the front end changes are waiting to be reviewed, approved and merged in PR #7852

@abhith and @jvilimek I think you were both interested in this feature based on your 👍 in #598, if that's right then this will help you track progress on this

With #7852 now merged in and part of the last release - release-2021-11-18 - the ability to view Azure Repos Pull Requests from an Entity is now available. Instruction for setting this up can be found in the Azure DevOps plugin README


video_player - Support device rotation and full screen

Do we have any option to enable the  auto rotation when turning the device.

I am also looking for same option

Any hope to have fullscreen & device rotation for video_player ?


[ Bug ] Text widgets are placed at the top, in the Hub node

The text widgets are not placed correctly in the Hub node.

If we zoom in a little, the text widget goes way.
But it's still not displayed in the Hub, in the correct area.

Hm weird, I also updated my comfy instance and this does not shows correctly.

Hm weird, I also updated my comfy instance and this does shows correctly.
Maybe a conflict with other node pack.

Nvm I know why now. The current Hub did not take account into new menu UI.

I disabled all my other custom nodes, but I still have this issue.
I tried in Chrome and Edge.
I'm on Windows 10.

I also tried the alternative DOM implementation.

It's a little better, but it also has issues.
The text boxes are still not shown in the nodes.
The text boxes are placed correctly in the Hub, but they are not usable.
When I click on the text box in the Hub, it disappears, and it shows up in the node instead !

Yeah past version of comfyui did make text box editable in Hub. For some reason the current comfyui tried to refocus and shift the textbox widget to original node. Honestly this is really hard to fix so it's unlikely I'll touch it soon until I can find out what's keep shifting the textbox back to original location.
Will leave this issue open and see if I can figure out the root cause.


Only getting a purple screen

After deploying flame using this docker-compose file
version: '3'
services:
    flame:
        image: pawelmalak/flame:latest
        restart: unless-stopped
        volumes:
            - ./data:/app/data
        networks:
            - caddy_net

networks:
  caddy_net:
    external: true

I only get a purple screen when visiting the URL (I use Caddy2 as a reverse proxy). When I refresh the page I can see
Thursday 10, June 2021
Good Evening

Applications

Bookmarks

for a fraction of a second, before only the background color is shown again.
I tried Firefox and vivaldi and both show the same behaviour

Are you getting any errors in the console?

I'm seeing the same thing when trying to access flame from behind an nginx reverse proxy. It works fine if I hit the site at the docker host:port itself, so I am guessing I'm missing some config or something. I can post my nginx config if that would help.

I have never used reverse proxy before so I will try to setup one, check if I can reproduce this problem and then work from there. I suspect it all comes from this ws:// protocol.

Indeed, it works when not using a reverse proxy

So I setup reverse proxy using NginxProxyManager and got the same error on all major browsers.
As expected, the error was caused by use of ws:// protocol with https://.
I've added simple check on WeatherWidget which will connect to either ws or wss based on document.location.protocol so it can be used both in development and on production.
I don't know how it works with other reverse proxy services but here I had to allow for websocket support to get it fully working.

@maeries Do you have reverse proxy working on Caddy v2?  For me it works on mobile (chrome & safari) but not on desktop (chrome).  I just get the purple screen.  If so, could you share your Caddyfile?


Handle status flags in description (! and *)

Hi,
When entering a transaction "Description", is it possible to input a pending transaction and have autocomplete work?
Autocomplete works:

Autocomplete does not work:

Not yet, unfortunately. The current workaround is to input (part of) the description without the status flag, press Tab to autocomplete, press Ctrl+a to move to the beginning and then input the status (in this case !). Completion won't reliably for the remaining fields, though.

Ah, ok. By the way, thank you for creating and sharing this project - I've been using it for a few weeks and it's already saved me loads of time.


Adding a new controller - View expected string but got xml node

When following along with the quick start and adding a new controller using tool, I get the following compile error:
`D:\Projects\ToRead\src\ToRead\Books\BooksController.fs(16,40): error FS0001: This expression was expected to have type    'string'    but here has type    'Giraffe.GiraffeViewEngine.XmlNode' [D:\Projects\ToRead\src\ToRead\ToRead.fsproj]
D:\Projects\ToRead\src\ToRead\Books\BooksController.fs(16,40): error FS0001: This expression was expected to have type    'string'    but here has type    'Giraffe.GiraffeViewEngine.XmlNode' [D:\Projects\ToRead\src\ToRead\ToRead.fsproj]
D:\Projects\ToRead\src\ToRead\Books\BooksController.fs(27,40): error FS0001: This expression was expected to have type    'string'    but here has type    'Giraffe.GiraffeViewEngine.XmlNode' [D:\Projects\ToRead\src\ToRead\ToRead.fsproj]
D:\Projects\ToRead\src\ToRead\Books\BooksController.fs(27,40): error FS0001: This expression was expected to have type    'string'    but here has type    'Giraffe.GiraffeViewEngine.XmlNode' [D:\Projects\ToRead\src\ToRead\ToRead.fsproj]
D:\Projects\ToRead\src\ToRead\Books\BooksController.fs(29,39): error FS0001: This expression was expected to have type    'string'    but here has type    'Giraffe.GiraffeViewEngine.XmlNode' [D:\Projects\ToRead\src\ToRead\ToRead.fsproj]
D:\Projects\ToRead\src\ToRead\Books\BooksController.fs(35,28): error FS0001: This expression was expected to have type    'string'    but here has type    'Giraffe.GiraffeViewEngine.XmlNode' [D:\Projects\ToRead\src\ToRead\ToRead.fsproj]
D:\Projects\ToRead\src\ToRead\Books\BooksController.fs(35,28): error FS0001: This expression was expected to have type    'string'    but here has type    'Giraffe.GiraffeViewEngine.XmlNode' [D:\Projects\ToRead\src\ToRead\ToRead.fsproj]
D:\Projects\ToRead\src\ToRead\Books\BooksController.fs(43,40): error FS0001: This expression was expected to have type    'string'    but here has type    'Giraffe.GiraffeViewEngine.XmlNode' [D:\Projects\ToRead\src\ToRead\ToRead.fsproj]
D:\Projects\ToRead\src\ToRead\Books\BooksController.fs(43,40): error FS0001: This expression was expected to have type    'string'    but here has type    'Giraffe.GiraffeViewEngine.XmlNode' [D:\Projects\ToRead\src\ToRead\ToRead.fsproj]
D:\Projects\ToRead\src\ToRead\Books\BooksController.fs(45,39): error FS0001: This expression was expected to have type    'string'    but here has type    'Giraffe.GiraffeViewEngine.XmlNode' [D:\Projects\ToRead\src\ToRead\ToRead.fsproj]
D:\Projects\ToRead\src\ToRead\Books\BooksController.fs(64,40): error FS0001: This expression was expected to have type    'string'    but here has type    'Giraffe.GiraffeViewEngine.XmlNode' [D:\Projects\ToRead\src\ToRead\ToRead.fsproj]
D:\Projects\ToRead\src\ToRead\Books\BooksController.fs(64,40): error FS0001: This expression was expected to have type    'string'    but here has type    'Giraffe.GiraffeViewEngine.XmlNode' [D:\Projects\ToRead\src\ToRead\ToRead.fsproj]
D:\Projects\ToRead\src\ToRead\Books\BooksController.fs(80,40): error FS0001: This expression was expected to have type    'string'    but here has type    'Giraffe.GiraffeViewEngine.XmlNode' [D:\Projects\ToRead\src\ToRead\ToRead.fsproj]
D:\Projects\ToRead\src\ToRead\Books\BooksController.fs(80,40): error FS0001: This expression was expected to have type    'string'    but here has type    'Giraffe.GiraffeViewEngine.XmlNode' [D:\Projects\ToRead\src\ToRead\ToRead.fsproj]
Build FAILED.`

I had the same issue and worked around it by adding a new function
  let asString (xml:GiraffeViewEngine.XmlNode) =
                  Giraffe.GiraffeViewEngine.renderXmlNode xml

and then adding it into the generated code:
return! Controller.render ctx (asString (Views.index ctx (List.ofSeq result)))

Ideally I think it should be added to ControllerHelpers.fs
    ///Returns to the client rendered template.
    let renderXml (ctx: HttpContext) template =
      ctx.WriteHtmlStringAsync (Giraffe.GiraffeViewEngine.renderXmlNode template)

And then update the template to call it:
return! Controller.renderXml ctx  (Views.index ctx (List.ofSeq result))

Fixed in latest release


Delete bundle command

Fixes #63 , subject to testing with a Duffle build that fixes the -r bug.

I was wondering about 'delete all versions' but skipped it for this drop. We can certainly add it, but I'll make that a separate PR as it involves some broader refactoring...


Added a microbenchmark for Timeline events.

Minimizing the overhead for performance tracking tools should be a goal.  I see there might be some room for optimization here.
Test exempt: This is a test.
cc @iskakaushik
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Looks like the checks are pretty unhappy.

Link to the Dart Timeline speedup code: https://dart-review.googlesource.com/c/sdk/+/233652

Looks like the checks are pretty unhappy.

Do you know what is going on with these tests?  I assumed the tree was red, but it looks like things should be fine.  The tests that are failing should be unrelated and they are saying things like "Could not find Java" which I didn't change.

Not sure what's up with them. Maybe ask on the infra chat?


securing access to ignition configs

Is there a good way to prevent an attacker from downloading ignition configs from my matchbox server?
I'm running a homelab where all my servers are booted via PXE. I don't install CoreOS on any disks - some of these servers don't even have disks. I load all the configuration I need via matchbox, every time a server boots up. This configuration includes SSH keys, kubernetes keys, tinc keys, and a bunch of other secret information. Of course, my servers are on a relatively secure network, and the matchbox server isn't exposed to the internet, but if someone managed to connect to my LAN (or even ran curl from any computer on the network), there's nothing to prevent them from downloading my ignition configs via the HTTP API.
Since servers have no information on them prior to provisioning, I can't imagine a way of adding authentication to the matchbox API, but maybe there's some technique that I just don't know about?

You should never store sensitive information in Matchbox. By definition, its one of the earliest configs a bare-metal machine loads. That's why you'll see Typhoon and Tectonic copy sensitive information onto the node after bring-up. That's why you see the examples do it this way and the reason for the recommendation. Place your SSH public key on the box and copy secrets.
You may wonder, as many do, "why not just add authentication"? To which the answer is that the client must authenticate with_something_. How did that something get there? Matchbox is the earliest service. Perhaps you could pick an even earlier service to add an auth token/cert/key into, but you can ask the same question - how did the "fresh from the manufacturer" key get on the box. Eventually, you'll end up with something like relying on the TPM and collecting keys upfront or other forms of strong attestation.
Perhaps more than you wanted to know. But bootstrapping from zero state to trust is not an easy problem. Don't store secrets in Matchbox.

Thanks for the reply, that all makes sense. I closed the issue since this is just a question.

fetch them with units during startup, from some attesting store.

That would ultimately depend on me manually putting some sort of a key on the machine, right? Like keeping a key on a small USB drive attached to an otherwise diskless system, and using that key to pull down all my secrets from Vault.
Would it be sane to write a custom metadata provider to handle reading a TLS cert from a USB key, logging into Vault via curl, and then writing all the secrets I need to /run/metadata/coreos?

Eventually, you'll end up using the TPM and collecting keys upfront or other forms of strong attestation for physical machine identity.

Yeah, I'd definitely like to avoid depending upon TPM.


Improve performances on checkout setup and opening

Description
This Feature will improve the way the Checkout is setting up and improve load time. We are going to decide if an order refresh is needed or not, based on a the following parameters:

order's autorefresh attribute
order's guest attribute
kind of token

In particular we can reference this flowchart:
flowchart LR
    A[Start] --> B{order.autorefresh?}
    B -- true --> C{Token type?}
    C -- Sales channel --> G
    C -- Customer token --> D{order.guest?}
    D -- false --> G[No refresh]
    D -- true --> F
    B -- false ----> F[Refresh sync]

:tada: This issue has been resolved in version 4.0.15 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update and packaged install failed. version 1.8.0 Windows 7 64bit

Prerequisites

[x] Can you reproduce the problem in safe mode?
[x] Are you running the latest version of Atom?
[x] Did you check the debugging guide?
[ ] Did you check the FAQs on Discuss?
[x] Are you reporting to the correct repository?
[ ] Did you perform a cursory search to see if your bug or enhancement is already reported?

For more information on how to write a good bug report or enhancement request, see the CONTRIBUTING guide.
Description
[Description of the bug or feature]
Steps to Reproduce

[First Step]
[Second Step]
[and so on...]

Expected behavior: [What you expected to happen]
Actual behavior: [What actually happened]
Versions
You can get this information from executing atom --version and apm --version at the command line. Also, please include the OS and what version of the OS you're running.

I tried many ways but could not update Atom or install packages that I need.

Can you click 'Show output' so we can get some detail on what's going wrong?

Here is the output details:
Fetching outdated packages and themes failed.Hide output…
Request for package information failed: connect ETIMEDOUT
Fetching outdated packages and themes failed.Hide output…
Request for package information failed: connect ETIMEDOUT

Are you behind a proxy @beachd80?

I don't think so, I have tried with different connections and it appears the same.

i got the some problem
Atom    : 1.9.8
Electron: 0.37.8
Chrome  : 49.0.2623.75
Node    : 5.10.0
Atom    : 1.9.8
Electron: 0.37.8
Chrome  : 49.0.2623.75
Node    : 5.10.0
PS C:\Users\Mars> apm --version
apm  1.10.0
npm  2.13.3
node 0.10.40
python
git 2.9.2.windows.1
visual studio

Are you on a corporate network?  Do you know if your machine normally uses a proxy server?

I've got the same issue using portable. Windows 10. Atom Portable version 1.9.9 (just downloaded from github so I could use without installing to my C: ssd)
I'm not behind a proxy and this is not a corporate server. Same error repeated twice as shown in @beachd80 's box above.

any word on this? Just checking.

This solved it for me:  Can't get Atom pass through a firewall #10357

@beachd80 Can you confirm if you still have the problem?  If so are you running a firewall? If so does permitting node.exe to access the network resolve the problem?

No response.


blog: GSoD 2020 post improvements

Preview @ https://dvc-landing-gsod-mkn8cutqbcgst.herokuapp.com/blog/gsod-ideas-2020

Hi @casperdcl do you know why the link checked script wants https://dvc.org/uploads/images/2020-04-30/tooltip.png to exist in order to pass? 🙂 The link is /uploads/images/2020-04-30/tooltip.png in the code. Please see https://circleci.com/gh/iterative/dvc.org/3296

@jorgeorpinel disregard the check if image actually exists

Not sure about the question @jorgeorpinel - do you mean the /uploads/images/... code actually refers to a base url other than https://dvc.org?

Ah I see the new file is part lf this PR, so yes it'll fail until it's actually merged.
@shcheklein What do you think about running a local webserver in the CI and checking against base URL http://localhost?

Seems like the easiest way is to ignore this. Running a local server sounds like overkill (build, serve, etc). Any way it can know when there's a preview deployment? In this case it's https://dvc-landing-gsod-mkn8cutqbcgst.herokuapp.com/ but not every PR will have one.

@casperdcl running it will complicate the check significantly - it takes minutes to build the app.
could we somehow run/re-run this check after preview is done and get the preview URL from the Github itself?

Seems like the easiest way is to ignore this.

simple, but breaks the flow - merging only ✅ ... which easily leads to merging it with some major issues.

Merged per agreement to iterate quickly on these improvements. I think it only needs one more PR, not as big as this one. Will send ASAP


Run sync task in its own subprocess

Currently, a synchronous task runs in its own thread (since v2.13.0 / PR #1160). As discussed in #1156, we should evaluate whether we want to run a synchronous task in its own subprocess.
Advantages:

Good for CPU-intensive work (for I/O intensive work, we already have async tasks)
Shutdown and timeout handling is easier as subprocesses can be killed
Tasks are more isolated (no threading issues)

Disadvantages:

Complex implementation

Communication between job and worker (signals)
Calling the job_manager (database context, connector, ....)

Problematic Windows support (no forking of processes on Windows)

I have to admit I have limited multiprocessing experience (it just happens to never have been on my radar).
From what I know, Pipes and Queues might be what Python gives us to communicate between processes. Object put in queues are pickled. pipes let us transfer text payloads.
Another complex point (but linked to the JobManager point) is the psycopg pool: does multiprocesing imply that each process will open its own pool ? That might be a little overkill, though I don't know how it will play. Especially: we might not need a connection except if we use task.defer from within the task. We could hack something to use a special connector in the task process that sends postgres queries in the pipe, to be handled by the parent.

From what I know, Pipes and Queues might be what Python gives us to communicate between processes. Object put in queues are pickled. pipes let us transfer text payloads.

Yes, and events (for something like the abort request).

Another complex point (but linked to the JobManager point) is the psycopg pool: does multiprocesing imply that each process will open its own pool ? That might be a little overkill, though I don't know how it will play. Especially: we might not need a connection except if we use task.defer from within the task. We could hack something to use a special connector in the task process that sends postgres queries in the pipe, to be handled by the parent.

Yes, if the database is queried directly, each process would use its own connection pool. I find it difficult to judge whether this could really be a problem in a real-life application or just a theoretical problem. Having a special connector and doing something like RPC between the processes sounds like a cool idea. Unfortunately, the same problem exists for database connections besides Procrastinate, when, for example, users do Django model queries. Those connections would take place in the subprocess anyway.

I'm a bit wary of changing the model just like this.
I wonder if we should maybe add options (we don't have to pick them all):

Legacy runner that runs async tasks & sync tasks via async to sync (breaking change could be that you need to set your worker to legacy explicitly to get the previous behaviour, or legacy could be the default when nothing is specified with a DeprecationWarning)
Subprocess runner

As usual the annoying part is to try and guess how it's going to be like for folks who use just async, just sync, a mix of both, with or without Django etc.

Or, make it configurable, as Huey does with worker types. We could have a sync_type option on the worker (or the task itself).
But there is so much stuff already in the v3 release that we should postpone this feature for a later release. We can add it as an experimental feature with a minor release (and keep the default sync tasks threaded), and when we are sure it's stable, we can still switch to subprocesses as the default.

We can add it as an experimental feature

Yes that was my point: I'm comfortable with adding it, I'm comfortable with it having better support over some features (such as aborting) than the standard way but I'm not (yet) comfortable with removing the way we do things now.


docs: wip contributing guidelines

We can later add code of conduct like this as well at a later stage: https://github.com/paritytech/substrate/blob/master/docs/CODE_OF_CONDUCT.md

I'm merging this in now—we might want to add stuff as and when contributions come in, but this is a good starting point and the PR's been hanging around for a while now :)


ImageBuilder May preview for Python

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-js

Can one of the admins verify this patch?

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

Automation for azure-sdk-for-java


Start in the background

When the system start up and the Launch on Startup option is enabled, the app window pop up automatically. Sometimes it's a bit annoying and every time you should close it (I use notifications).
It would be awesome if there were an options which start the app in the background, so only "small icon" will be appeared on the system tray, whithout the window on the task bar.

This is fixed in the beta.
Please see https://github.com/Aluxian/Facebook-Messenger-Desktop/issues/458


Device Zigbee - does not have registered zigbee2mqtt device - On any reboot

Hi,
I have installed domoticz on raspberry
It runs:
Version: 4.11590
Build Hash: d7cb21393
Compile Date: 2019-12-21 12:03:24
dzVents Version: 2.5.3
Python Version: 3.5.3 (default, Sep 27 2018, 17:25:39) [GCC 6.3.0 20170516]
Every time i reboot raspberry i have an error on startup:
Device Zigbee - does not have registered zigbee2mqtt device
and none of zigbee devices work.
If i restart domotiz service manually it solves the problem resulting to everything works like a charm!
Or
If i disable zigbee2mqtt python plugin and re-enable it, it works again.
Any help is going to be appreciated!!

To avoid error you need to start your services in such order:

MQTT Server
zigbee2mqtt
Domoticz

Thank you for your help.
I try the above settings but i have the same error.
Both services startup with systemd
My config:

/etc/systemd/system/mosquitto.service
[Unit]
Description=Mosquitto MQTT Broker daemon
After=network.target
Requires=network.target
Before=zigbee2mqtt.service domoticz.service

/etc/systemd/system/zigbee2mqtt.service
[Unit]
Description=zigbee2mqtt
Requires=mosquitto.service
After=network.target mosquitto.service
Before=domoticz.service

/etc/systemd/system/domoticz.service
[Unit]
Description=domoticz_service
Requires=mosquitto.service zigbee2mqtt.service
After=network.target mosquitto.service zigbee2mqtt.service

Any other hint i would appreciate.......

Could you please explain where do you have this error:
Device Zigbee - does not have registered zigbee2mqtt device

Domoticz log - filtered by zigbee
**First reboot
2020-01-04 16:10:31.277 Status: (Zigbee) Started.
2020-01-04 16:10:32.875 (Zigbee) Connected to MQTT Server: <IP_ADDRESS>:1883
2020-01-04 16:10:32.980 (Zigbee) Zigbee2mqtt bridge is offline
2020-01-04 16:10:32.982 (Zigbee) Joining new devices is disabled on the zigbee bridge
2020-01-04 16:10:32.807 Status: (Zigbee) Entering work loop.
2020-01-04 16:10:32.809 Status: (Zigbee) Initialized version 0.0.20
2020-01-04 16:10:59.812 (Zigbee) Zigbee2mqtt bridge is online
2020-01-04 16:11:00.167 (Zigbee) Joining new devices is disabled on the zigbee bridge
2020-01-04 16:11:04.078 (Zigbee) Device SonoffBasicZBR301 does not have registered zigbee2mqtt device
2020-01-04 16:11:05.632 (Zigbee) Device SonoffBasicZBR301 does not have registered zigbee2mqtt device

Device BASICZBR3 not work

manual restart sevice
sudo systemctl restart domoticz.service
2020-01-04 16:14:45.956 Status: (Zigbee) Started.
2020-01-04 16:14:46.778 Status: (Zigbee) Entering work loop.
2020-01-04 16:14:46.779 Status: (Zigbee) Initialized version 0.0.20
2020-01-04 16:14:47.038 (Zigbee) Connected to MQTT Server: <IP_ADDRESS>:1883
2020-01-04 16:14:47.141 (Zigbee) Zigbee2mqtt bridge is online
2020-01-04 16:14:47.142 (Zigbee) Joining new devices is disabled on the zigbee bridge
2020-01-04 16:14:47.195 (Zigbee) Received available devices list from bridge
2020-01-04 16:14:47.195 (Zigbee) Device BASICZBR3 0x00124b001e73154f (SonoffBasicZBR301)
2020-01-04 16:14:47.377 (Zigbee) Joining new devices is disabled on the zigbee bridge

Device BASICZBR3 work

solution
create a .timer systemd unit file to control the execution of your .service unit file.
sudo nano /etc/systemd/system/domoticz.timer
[Unit]
Description=Run domoticz after 1 min  on boot
[Timer]
OnBootSec=1min
[Install]
WantedBy=timers.target
sudo systemctl disable domoticz.service
sudo systemctl enable  domoticz.timer
sudo systemctl daemon-reload
sudo reboot


allow user to switch parameter order

display action description on save

any parameters in action description need to be generated on save, not before then. so added two params actionDesc and genActionDesc which will be calculated on save.

making a local copy of parameter list actionParams = action.params on load, on save, it is copied back and sent to the database.

BUG: types dont show up anymore on load, only after changing order