🐛 Fix the cloud-init in the centos template

This PRs fixes an issue in the Centos cluster template. This issue leads to a networking problem in the Centos target nodes. |The ptoblem is that the ironicendpoint bridge doesn't have IP address although it should have it.

/test-integration
/assign @fmuyassarov
/cc @kashifest

Does it mean that we have the same issue in metal3-dev-env, because we have the same configuration there: https://github.com/metal3-io/metal3-dev-env/blob/4e53921a232ed02a1cbbaf8c46da961dfaf01744/vm-setup/roles/v1aX_integration_test/templates/cluster-template-workers-kubeadm-config-centos.yaml#L44
I see that CentOS job passes in CI though.

No, we don't have the issue in m3-dev-env because the template is processed by Ansible before it is sent  KCP/machinedeployment. Ansbile should removes the first "{{ }}". However, in the e2e test, the template is not used by Ansible, so having two layers of brackets leads to the issue that the variable is not substituted correctly.

/test-e2e-v1a5
/test-centos-e2e-v1a5

/test-v1a5-e2e
/test-v1a5-centos-e2e

/lgtm


Kubernetes

It appears that there needs to be a slight modification in the helm install command on https://cert-manager.io/docs/installation/kubernetes/
Currently it reads:
$ helm install
--name cert-manager
--namespace cert-manager
--version v0.13.0
jetstack/cert-manager
Turns out that Helm doesn't want the --name flag as this returns
Error: unknown flag: --name
This does work:
$ helm install
cert-manager
--namespace cert-manager
--version v0.13.0
jetstack/cert-manager

Closing as a duplicate of #92


RYA-11 initial implementation of free text for mongo backend

Initial implementation.  Mongo backed free text query does not match Accumulo, but I don't have the cycles to finish that.  Updating this so that it matches would be a good future task for someone to pick up.

Added fixes for Rya-34 to this pull.  Sorry for multiple bugs on the same PR, had to build on fixes from the original pull request

If you fix the conflicts below, I'll merge your PR:
CONFLICT (content): Merge conflict in extras/indexingExample/src/main/java/MongoRyaDirectExample.java
CONFLICT (content): Merge conflict in extras/indexing/src/test/java/mvm/rya/indexing/external/PrecompJoinOptimizerIntegrationTest.java
CONFLICT (modify/delete): extras/indexing/src/test/java/mvm/rya/indexing/external/AccumuloIndexSetTest2.java deleted in 13de8a5645391b65ba0a6753eedbb51bff4d7bee and modified

Hey Aaron, the merge conflict is fixed now, but I've been having trouble building.  maven can't resolve the updated geomesa dependencies.  Either way, I think it should be good to build, but I was going to ask you or David to check that it built (since you probably have the dependencies from an earlier build and they must have disappeared recently).

committed.


Add developer guide

Is your feature request related to a problem? Please describe.
Add a guide for new developers for NGINX Kubernetes Gateway (NKG)
Describe the solution you'd like

How to setup a dev environment
How to build and test NKG
How to implement a feature
Information or links to the architecture/design of the project, etc.

Describe alternatives you've considered
Additional context
Aha! Link: https://nginx.aha.io/features/NKG-23

Need to add style guide to this story


Make JSON the configuration of LogStreamer (drop XML)

I've cleared this with @linkdata.

@linkdata
When doing this, I'm dropping the feature that the log streamer creates a default configuration as a file on disk when either no configuration is given, or a vale is given and that is invalid. Especially that last case I think is wrong, or at least overkill.
Personally, I don't feel we need that feature at all. And if you disagree, I suggest we implement it as a "tooling thing", keeping them separate. As an example, supporting some way to instruct the log streamer to generate a default config, like logstreamer new config.

Also removing that feature that if you provide an upstream URI on the command line, and the configuration is file based, the file based configuration is updated. Touching FPS in my book. :smile:
You are to kind, @linkdata. Just tell users what to pass, and make them pass that. And if the don't, punish them. :wink:

@linkdata
You're not using the partials and wwwroot stuff anymore, right? The HTML provision is now solely based on what is provided by the streamer entrypoint class, correct?
If so, I won't bother to adapt those, but instead just drop them while I'm at it.

@linkdata
I'm about to wrap this one up soon, based on decisions on what I like (you gave me that freedom, right? :smile:). It's messing quite a bit with some central pieces - e.g. project file and Program.cs - so please don't do any major changes to those until I've pushed it.

Råger, Vilko. And on the wwwroot stuff, nuke it from orbit. It's the only way to be sure.

So what is now the easiest way to make persistent configuration changes to a LogStreamer running on a certain database?

Shared to the world in https://github.com/StarcounterPrefabs/LogStreamer/commit/974511a37acd567b1de952e197c1998bc1769bfd.
Before you pull, here's a writeup of all stuff that has happened:

XML configuration no longer supported (duh).
Everything file-related is now opaque to the logstreamer - if you pass it a config URI that is file based, it will read that file as a JSON config. No more, no less.
LogStreamerConfiguration now provision (static) configuration defaults + a Load method that
returns an abstraction, not an implementation.
You can pass the log streamer a config URI and a upstreamURI, just as before. See how they are treated below.
The root page now show when there are whitelisted tables, and you can click the set to see a page with the different sets.
Older viewmodels and views are dropped
New handler exposed: /logstreamer/default.config.json. Does what the URI implies.
The Program.Whitelist is removed in favor of new handler ´/logstreamer/sample.config.json. Invoke that one to see samples in JSON format. It's randomized, just like Program.Whitelist`, so hit F5 in the browser a few times to see different variations.

Q: I want to start with a default configuration from file?
A: Browse to /logstreamer/default.config.json. Copy the content, save it to a file with config.json ending and pass the path along to the log streamer upon next start.
Q: What happens if I don't pass any configuration URI?
A: The log streamer will internally fetch defaults from [7].
Q: What happens if I pass a config URI and an upstream URI?
A: The log streamer will override the UpstreamUri resolved from config with the explicit one you give.
Q: What happens if I pass only an upstream URI?
A: Q1 + Q2.

So what is now the easiest way to make persistent configuration changes to a LogStreamer running on a certain database?

I knew I should have written the above before I pushed it - you're always so het på gröten dammit!

So what is now the easiest way to make persistent configuration changes to a LogStreamer running on a certain database?

Hm, I realized that particular question was not answered. The answer is: you don't. We gather configuration at startup, you need to change that - go edit and restart. You think thats awful, I can support a REST PUT (or POST) to get it done.

Slow kid is slow.
What?  No, the best way to find out how high a cliff is, is to jump off and count the seconds loudly before you smack face first into the ground.
I will tweak it a bit to look for a ..\Starcounter\Personal\DBNAME.logstreamer.config.json before grabbing the /logstreamer/default.config.json result body.

I will tweak it a bit to look for a ..\Starcounter\Personal\DBNAME.logstreamer.config.json before grabbing the /logstreamer/default.config.json result body.

Yeah, sure, do. Sounds good.
Closing this one as "done". Open up a new issue in case you think something is missing, wrecked, w/e.


Remove callback_url and callback_secret_key from Event

This feature is not in use and has, as far as I can find, has only ever been used for 1 event (erroneously). The event in question is pandora ending drink of 2018.

It seems unnecessary to remove a feature that, while not actively used, is causing no problems and as far as I know still works. I don't see why it should be removed


[Documentation] best-practices / ui

I'm submitting a...

[+] Bug report

Current behavior
After visiting this link  , I saw "not found" page
Expected behavior
To see page about separation between ui and model.
Minimal reproduction of the problem with instructions
Go to the link https://datorama.github.io/akita/docs/best-practices/ui
What is the motivation / use case for changing the behavior?
More expressive documentation

https://datorama.github.io/akita/docs/ui


mms/mmk differences - olb dependencies

$ mms/id
%MMS-I-IDENT, MMS V3.3-4 � Digital Equipment Corporation 1989, 1998. All rights
reserved. � Electronic Data Systems Limited 1995, 1998
$ mmk/id
%MMK-I-IDENT, this is the MMK Make Utility V5.0
-MMK-I-COPYRIGHT, Copyright (c) 2008, Matthew Madison.
    Copyright (c) 2013, Endless Software Solutions.
  See LICENSE.TXT in distribution kit for license information.
$ type descrip.mms
x.olb(a) : a.h
$ dir a

Directory DISK$USER:[USER]

A.C;1               A.H;1

Total of 2 files.
$
$ mms/noaction

CC /NOLIST/OBJECT=A.OBJ A.C
If "''F$Search("X.OLB")'" .EQS. "" Then LIBRARY/Create X.OLB
LIBRARY/REPLACE X.OLB A.OBJ
$ mmk/noaction
%MMK-F-NOACTION, actions to build X.OLB(A=A.OBJ) unknown
$

Apologies for the late response. This looks like a short coming in target/source determination.  I'll investigate further to determine the appropriate action.


The part about network equivariance that I don't understand.

I have a question that I don't quite understand. In the MultiHeadAttentionLayer module, when calculating the distance gate, you need to normalize full_g.ndata['coors'] first and then use DGL's u_sub_v to get the difference between src and dst. However, the difference vector obtained this way is not the actual distance in the spatial coordinates, meaning it will vary according to the coordinate system, thus not satisfying equivariance. If I don't add self.coor_norm, would that be correct?

I don't understand why mean(-2) is needed here. My understanding is that the tensor returned from self.atom_encoder has dimensions (Batch, Nodes, Channels), and adding mean(-2) transforms it into (Batch, Channels). However, the network can still run normally. I don't understand why it works this way.
h = g.ndata['x']
h = self.atom_encoder(h.long()).mean(-2)

However, the difference vector obtained this way is not the actual distance in the spatial coordinates, meaning it will vary according to the coordinate system, thus not satisfying equivariance.

I am not clear understand the meaning of the sentence "However, the difference vector obtained this way is not the actual distance in the spatial coordinates, meaning it will vary according to the coordinate system, thus not satisfying equivariance." , can you explain it?


doesn't work in an image gallery while using with framer motion

so i'm using it in an image gallery but it doesn't work for some reason. i can't see the zoom-in cursor & it doesn't zoom in on click either.
how do i fix this?
demo with code → https://stackblitz.com/edit/framer-motion-image-gallery-nextjs?file=pages%2Findex.tsx

Hello.
I know nothing about Framer Motion. My library works at the lowest, general-purpose level, e.g. HTML elements.
You may try to inspect the actual DOM elements generated by Framer, their hierarchy and events. Firefox may be useful, as for every element it can display all the event handlers attached to it.

cool, no problem :)


Add TCP Keepalives for column encoding tool

This is a pull request for the column encoding tool.  This would help with the issue in  https://github.com/awslabs/amazon-redshift-utils/issues/224
In the ideal case pg8000 will support TCP keepalives however since for this tool it's always beneficial to have keepalives on we can just manually do it by operating on the socket.
[ec2-user@ip-172-31-7-182 ~]$ netstat -top | grep "`ps -ef | grep analyze | grep -v grep | awk '{ print $2}'`"
(Not all processes could be identified, non-owned process info
 will not be shown, you would have to be root to see it all.)
tcp        0      0 ip-XXX-XX-XX-XXX.eu-we:43566 ip-XXX-XX-XX-XXX.eu-we:5439 ESTABLISHED 3021/python         keepalive (5.75/0/0)

Awesome - never managed to spot the usock property. Thanks!


Referral system imports and function calls update

Project organization

Related trello card
Dependant on new liteflow-js version with the PR

Description
Accepting referral doesn't need signer to be passed.
Splitting hook into two fixes the issue.
TODO

[ ] Update with the new release for @liteflow/react package

Checklist

[x] Base branch of the PR is dev

Can be reopened later on.


User Profiles

Hi all,
I'll leave this here. Don't know where else I should go with it. I'm working on a website for my extended families shared vacation home. It's intended to host a bunch of basic administrative infos, as well as periodic updates and a collection of email addresses and phone numbers of all parties involved.
As of now we have a general site-wide password in place. This works ok, but security is admittedly very low. Is there a simple FOSS way of implementing user profiles for a case like this? Imo this would make it quite a bit more secure. Not too sure where I should start. There are some payed wordpress-like options which I would very much like to avoid.
Any opinions?
All the best.

If you are willing to do a little coding yourself, I already have CSRF, users, and sessions done on my minimal Go web framework. I think you could adapt what I made fairly easily to support a staff role and protect pages based on if the current user is staff or not.


Is There a Way to Handle process.stdout.write?

Starting from Next 14.2.0, Next will output request logs in dev mode (https://github.com/vercel/next.js/pull/62946), which is always enabled and uses process.stdout.write to output the logs.
I tried to create a new patch for process.stdout.write but realized that Pino also uses the same method underneath by default (I believed it is this code). Even though during the testing this recursive calling loop seems to work, but the output is not as expected, i.e. the whole JSON format message is wrapped in the msg field, instead of itself being the output.
One potential hacky workaround is to check the caller and decide whether we need to patch the call or not, but Function.prototype.caller is deprecated due to security risks, so this doesn't seem to be a viable path.
I wonder if there are any other solutions, or it just can't be solved.

Patch-package next itself would probably work. Maybe a postinstall that does a sort-of search and replace - Biome.js uses a nifty library for doing search-and-replace type commands.
Although it's probably just as easy to get a PR merged into core, which allows for doing these logging things.
Lastly, sentry has a fine logging solution for next, so they will certainly have a solution on their sdk.
Good luck.

We really need first-class support for JSON logging in Next.js itself. 😅

We really need first-class support for JSON logging in Next.js itself. 😅

There is a discussion on the Next.js side: https://github.com/vercel/next.js/discussions/63787


Address post-merge comments from 2106

#2106

What's the reason to add a QueryHelpers instead of implementing them in the store?

What's the reason to add a QueryHelpers instead of implementing them in the store?

mainly because it would require making AuthorityState::database public which I don't feel good about. It also didn't feel appropriate to be checking API errors (e.g. MAX_TX_RANGE_SIZE) in authority_store.rs

So for future reference, what kind of functions should go to QueryHelpers?

So for future reference, what kind of functions should go to QueryHelpers?

I think anything that needs to be shared between GatewayState and AuthorityState. But since the plan is to phase out the gateway (at least the read-only parts), that part of GatewayState should actually go away entirely and the code can be moved back into AuthorityState, and QueryHelpers deleted. Sound good? I can leave a comment in the code to that effect if you agree.


'Thread' object has no attribute 'member_ids'

Summary
Errors out randomly with that error
Reproduction Steps
No idea
Code
No response
Expected Results
Nothing
Actual Results
Error
System Information

Python v3.8.12-final
discord.py-self v1.9.1-final
aiohttp v3.7.4.post0
system info: Linux 5.11.0-1026-gcp #29~20.04.1-Ubuntu SMP Fri Jan 7 12:24:31 UTC 2022

Checklist

[X] I have searched the open issues for duplicates.
[x] I have shared the entire traceback.
[X] I am using a user token (and it isn't visible in the code).

Additional Information
No response

face desk
How the hell did I not run into this during testing.

object._member_ids is declared and mentioned at the following places -

1
2
3
On the rebase branch

Also, there isn't a threads.py on the 1.x branch, no mention of threads in the message.py file either, yet the version number says 1.9.1. Could you tell me where the thread class is mentioned Dolfies?

oh whoops, i think i called the wrong discord - its actually the rebase branch.

The error happens on line 1053 of discord/state.py

Although for me the error says member_ids not _member_ids

Seems that error was fixed 4 days ago but poetry just doesn't like to update the git rev. Thanks!

Why is it that line 1053 of discord/state.py says thread.member_ids in GitHub but when I click edit it says thread._member_ids and yet my local file when I pip install also has thread.member_ids and still errors out

This was indeed a bug in a seperate place in state.py. Fixed in latest commit.


a way to observe the locale change for the utility function

Simplified example:
anObserver: function() {
  this.set('something', this.t('item1.item2'));
}.on('init')

When the application locale changes, template helpers get updated, but these are static. I'm having this issue with bound select drop downs. I don't know a way to observe a change on the container.lookup('application:main'). Perhaps it would make sense to inject another property along with the utility function so that you could do something like this:
}.observes('locale').on('init')

Then you could rebind a select drop down or similar more easily.

I created the addon ember-cli-i18n-observer to solve my problem. It injects i18n.locale so you can }.observes('i18n.locale') then use the shortcut this.get('i18n.locale') to get it.
Is this something you would be interested in for the main repo?

I created the addon ember-cli-i18n-observer to solve my problem. It injects i18n.locale so you can }.observes('i18n.locale') then use the shortcut this.get('i18n.locale') to get it.
Is this something you would be interested in for the main repo?

Exactly the problem I was facing today.
Yes, I would definitely like to see this as part of the main repo!

@kellyselden
Your observer works excelent! I created this mixin to easily create dynamic fields that listen to locale changes, since I don't want to write an observer over and over again. Feel free to use or add to your project.
import Ember from 'ember';

export default Ember.Mixin.create({

  localeValues: {},

  setLocaleValue: function(fieldName, value, params) {
    this.localeValues[fieldName] = {
      value: value,
      params: params
    };
    this.updateLocaleField(fieldName);
  },

  updateLocaleField: function(fieldName) {
    var settings = this.localeValues[fieldName];
    if (!settings.params || settings.params.length === 0) {
      this.set(fieldName, this.t(settings.value));
    } else {
      eval('this.set(fieldName, this.t(settings.value, "' + settings.params.join('","') + '"));');
    }
  },

  updaleLocale: function() {
    for (var fieldName in this.localeValues) {
      this.updateLocaleField(fieldName);
    }
  }.observes('i18n.locale')

});

// Locale file
export default {
  headerTitle: "This is a header title",
  presentMyself: "My first name is %@, and my family name is<EMAIL_ADDRESS>}

// In any object using the mixin
updateTitle: function() {
  this.setLocaleValue('title', 'headerTitle');
  this.setLocaleValue('present', 'presentMyself', ['Bauke', 'Regnerus']);
}.on('init'),

// Template

{{title}}

{{present}}


Support configTcps with parameter port number and wallet password

The configTcps today can be executed to run TCPS support at listener level. Please extend it to support with optional parameter of port number and wallet password.

@aosingh please raise a PR for this. @aditya059 please help review the PR

@aditya059 let me know if you have any questions. I have raised a PR


contactUs section update

fixes #110

Description:  I've made the contact-us component to render on home page only.
Checklist

[x] I've followed the Contributing guidelines provided in the repository.
[x] I've made the changes which were demanded in the linked issue.
[x] I've tested my code on a Chromium based browser.
[x] I've tested my code on Mozilla Firefox.
[x] My code gave a clean console on debugging. (no warnings/errors)

Testimonial

NA

In the mobile view, the footer seems a bit more compact as the icons are being superposed over the footer-background, a bit of padding, and margin can be added on the icons for the mobile view.
I tried a bit of experiment see if it looks good

In the mobile view, the footer seems a bit more compact as the icons are being superposed over the footer-background, a bit of padding, and margin can be added on the icons for the mobile view.
I tried a bit of experiment see if it looks good

PS: Have added margin across the icons for resolution less than 767px.

Also, we should be adding the LinkedIn handle of hG as well! 😄

In the mobile view, the footer seems a bit more compact as the icons are being superposed over the footer-background, a bit of padding, and margin can be added on the icons for the mobile view.
I tried a bit of experiment see if it looks good

PS: Have added margin across the icons for resolution less than 767px.

This looks good , I think we can create another issue for that and can also add the Linked-in icon in it.. For the community section , I've found a fix let me make a commit for it .


[Backport release-23.05] armcord: 3.2.1 -> 3.2.3

(cherry picked from commit 810ec794359b1aa7439b337796a8d364b92b9b4e)
Backport to release-23.05 based on PR #249094
Changelog: https://github.com/ArmCord/ArmCord/releases/tag/v3.2.3

[ ] Before merging, ensure that this backport complies with the Criteria for Backporting.

Even as a non-commiter, if you find that it does not comply, leave a comment.

I appreciate this so much! 🙏


use ablc command

I decided to refactor the adblock_dns/compiler rust codes, into a separate project:

github project: https://github.com/ragibkl/adblock-list-compiler
rust package: https://crates.io/crates/adblock-list-compiler

It comes with the following difference:

rewritten from scratch
better use of types for code correctness
single threaded processing, no parallelism yet
lower memory usage. see: https://github.com/ragibkl/adblock-dns-server/issues/162
slower because files are fetched one at a time

Everything else is the same.
This PR is to integrate the new ablc compile command into our current adblock_dns server. I also had to make some small changes to the bind part, because some configs are no longer used in newer bind servers.
I'm going to run this on sg-dns1.bancuh.com for a few days.
Expected outcome:

Everything should work the same for the user.
Adblocklist compilation might take slightly longer to finish, but that's only happening on server side.
Less memory usage by the compiler

Failed condition:

some sources not loaded as usual
some overrides such as google safesearch or bing safesearch not working
ads not blocked
adblock list not updating as usual interval

sg-dns1 is up. For now, looks like the size of the blacklist.zone file is almost identical with sg-dns2.
# sg-dns1
$ ls -lh /etc/bind/blacklist.zone
-rw-r--r--    1 root     root       93.8M Dec 18 14:33 /etc/bind/blacklist.zone

$ wc /etc/bind/blacklist.zone
  2004753   6014263  98379672 /etc/bind/blacklist.zone

# sg-dns2
$ ls -lh /etc/bind/blacklist.zone
-rw-r--r--    1 root     root       93.8M Dec 18 13:44 /etc/bind/blacklist.zone

$ wc /etc/bind/blacklist.zone
  2004431   6013300  98365552 /etc/bind/blacklist.zone

sg-dns1 seems to use much less memory: 1.265 GiB vs 2.659 GiB. I might even be able to reduce the server RAM from 4GiB down to just 2GiB!!!
# sg-dns1
$ docker stats
CONTAINER           CPU %               MEM USAGE / LIMIT     MEM %               NET I/O             BLOCK I/O           PIDS
476d24237b0b        0.30%               1.265GiB / 3.859GiB   32.77%              1.69GB / 162MB      4.1kB / 1.84GB      8
9062c6e9bf94        0.68%               39.31MiB / 3.859GiB   0.99%               0B / 0B             6.37MB / 53.3MB     47
0fbf3260b172        0.00%               30.94MiB / 3.859GiB   0.78%               0B / 0B             124MB / 0B          11
1a2fdad08a0f        0.00%               3.535MiB / 3.859GiB   0.09%               258kB / 0B          37.9MB / 0B         9

# sg-dns2
$ docker stats
CONTAINER           CPU %               MEM USAGE / LIMIT     MEM %               NET I/O             BLOCK I/O           PIDS
2b2a32a2f5fb        0.03%               2.659GiB / 3.859GiB   68.91%              146GB / 3.88GB      9.89GB / 86.8GB     10
2f7cfbb84725        0.37%               35.82MiB / 3.859GiB   0.91%               0B / 0B             30.8GB / 34.7MB     47
3eeb974e9b78        0.00%               24.39MiB / 3.859GiB   0.62%               0B / 0B             9.97GB / 0B         11
6ffae0cc613f        0.00%               3.543MiB / 3.859GiB   0.09%               1.67kB / 0B         3.07GB / 0B         10

I tested memory usage when running this locally:

ablc: ~220MB
compiler: ~600MB

I think will just use ablc moving forward.


Use token like GitHub token as sole config

Currently, kubesync expects REPOCREDS to be in the format username:credentials, where credentials can be one of:

password
other credentials (e.g. API key)

We should be able to use the API key without requiring the username:credentials format for git. We already sort-of can with curl using
CURL_OPTIONS="--header 'Authorization: token $GIT_TOKEN  --header 'Accept: application/vnd.github.v3.raw'"

where $GIT_TOKEN is the credentials part of $REPOCREDS.
That is a bit hacky, though. We should be able to just make REPOCREDS work as just credentials.

Apparently GitHub supports it via setting the username to <token> and no password, or at most setting the password to x-oauth-basic. See here .


POT Generation not detecting exported variables

Tested versions

Reproductible in 4.2.stable

System information
Windows 10 - Godot v4.2.1.stable Forward+
Issue description
Generating a POT file from Project > Project Settings > Localization > POT Generation creates a template file which only includes text written for built-in nodes.
I would expect that any node which has an exported text property to be considered for POT Generation.
Steps to reproduce

Create a Label and set its text to 'EXAMPLE'
Create a node2D which has a script that exports a String variable: @export var text: String = "". Instantiate that node and set its exported variable text to 'OTHER_EXAMPLE'
Go to Project > Project Settings > Localization > POT Generation. Add the main scene and click Generate POT
The generated template does not contain 'OTHER_EXAMPLE':

# LANGUAGE translation for LocalizationMRP for the following files:
# res://main.tscn
#
# FIRST AUTHOR <EMAIL@ADDRESS>, YEAR.
#
#, fuzzy
msgid ""
msgstr ""
"Project-Id-Version: LocalizationMRP\n"
"MIME-Version: 1.0\n"
"Content-Type: text/plain; charset=UTF-8\n"
"Content-Transfer-Encoding: 8-bit\n"

#: main.tscn
msgid "EXAMPLE"
msgstr ""

Minimal reproduction project (MRP)
LocalizationMRP.zip

Reproductible in 4.2.stable

Can't reproduce in 4.3 dev (current master). Probably fixed by #89286.

template.pot
# LANGUAGE translation for LocalizationMRP for the following files:
# res://main.tscn
# res://Node2D.gd
#
# FIRST AUTHOR <EMAIL@ADDRESS>, YEAR.
#
#, fuzzy
msgid ""
msgstr ""
"Project-Id-Version: LocalizationMRP\n"
"MIME-Version: 1.0\n"
"Content-Type: text/plain; charset=UTF-8\n"
"Content-Transfer-Encoding: 8-bit\n"

#: main.tscn
msgid "EXAMPLE"
msgstr ""

#: main.tscn
msgid "OTHER_EXAMPLE"
msgstr ""

I think this was fixed when the node type check was removed by #87530. Before it would only include Control or Window nodes.
My PR fixed parsing of instanced scenes, and it made the included properties more dynamic. It will try to include all properties with a name that ends with _text.
https://github.com/godotengine/godot/blob/29b3d9e9e538f0aa8effc8ad8bf19a2915292a89/editor/plugins/packed_scene_translation_parser_plugin.cpp#L196-L207
This issue shouldn't be closed yet. Included and excluded node types and properties should be configurable to the user so they can decide themselves what should be included. I already have wanted to implement this, as i said in my PR, but im missing the motivation to figure out how to add project settings and start compiling Godot again (it takes like 3 hours).

Included and excluded node types and properties should be configurable to the user so they can decide themselves what should be included.

This sounds like something that should be a proposal, it's far outside the scope of this report

Closing as fixed, please open a proposal for any additional features


Update BLS signature module

What was wrong?
#1565
How was it fixed?

Follow the logic from the BLS signature verification

Replace BN128 with BLS-12-381 curve.
Replace the current blake hashing function with keccak-256.

Note

Please ignore the eth/beacon/aggregation.py for now. It's supposed to be updated in other PRs.
This PR is opened for reviewing the reference implementation. The current CI error is due to other type hinting issues. @carver, feel free to solve that and merge #1576 before it, I'll resolve the conflicts. :)

Cute Animal Picture

@pipermerriam yes, I'll investigate it.

@carver, feel free to solve that and merge #1576 before it, I'll resolve the conflicts. :)

Done. Thanks for doing the type cleanups! :)

@djrtwo merging it now! We can optimize the BLS APIs while moving it to py-ecc later (#1592). :)


Remove dark mode on download apps example page

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
The dark mode was removed for the newest releases.
Because of that, in the dowload apps example page (https://boosted.orange.com/docs/5.3/examples/download-app/), we have at the same time images from the light AND dark mode:

It's working but it looks confusing.
Reduced test cases
Go to https://boosted.orange.com/docs/5.3/examples/download-app/
What operating system(s) are you seeing the problem on?
Windows
What browser(s) are you seeing the problem on?
Chrome
What version of Boosted are you using?
v5.3.1

Fixed via https://github.com/Orange-OpenSource/Orange-Boosted-Bootstrap/commit/f9fcefeba442d2da7c3dd7e7fb6aeecc612de28a


Remote end closed connection without response

after successful connection is made and our python script is running, Is seems like at times if a PA server is or becomes busy? We get: “Remote end closed connection without response” and obviously the connection is dropped and our script raises the error and cannot continue.

Hi @JamesDMiller1969,
are you working with TM1 in the IBM cloud?
If yes, the TM1Service needs to be created slightly differently to work around the hard 60s timeout on the IBM cloud infrastructure.
https://code.cubewise.com/tm1py-help-content/connecting-to-the-ibm-cloud-remotely-with-tm1py

yes we are using the IBM cloud. was missing the "verify=True" so will give that a try. Will requesting the timeout be increase resolve this?

Actually, the key parameter is the async_requests_mode =True.
In this mode, TM1py doesn't wait for REST API Operations to complete, but it asks the server to do an operation and then checks every 1 sec if it completed.

Do you have a piece of code that you can post, that reproduces the error?

I understand that not all operations fail. Like the initial connection (TM1Service(...)) is established successfully and then further down the road some operation fails. is that correct?

Not all operations fail, the initial connection (TM1Service(...)) is established successfully and then further down the road some operation fails.
Other times it "seems" that if a PA server is busy then the connection attempt fails

It doesn't look like a TM1py issue to me. I think you may need to bring this up with the IBM cloud support.
This kind of occasional failure of REST operations looks more like an issue on the server/cloud side.
One last thing you can try though is to upgrade to the most recent version of TM1py. We introduced the async_requests_mode about 2 years ago. Perhaps your TM1py installation predates that change.

so we found that playing with the "timeout" number makes a difference:
with TM1Service(**config[tm1_target_server], timeout=4800) as tm1:

Hi @JamesDMiller1969,
I'm glad it works. I'm surprised the timeout argument makes the difference though.
I will close the ticket for now. Feel free to re-open.


Global Concat Pooling

❓ Questions & Help
Hi! First of all, many thanks for such an amazing tool!
I want to ask what would be the best way to do a global concatenation pooling over a batch of graphs. Basically, I wanted to concat the features of all the nodes in a certain graph for some downstream graph classification.
Given that there's already a global_add_pool, global_mean_pool, etc, I don't know whether this should be a feature request for something like global_concat_pool, but it's definitely a request for help :p

Do you really want to do that? Nodes in your graph aren‘t handled permutation invariant that way, and need to be filled with zeros to take varying node sizes into account. However, this is possible by using torch_geometric.utils.to_dense_batch and reshaping the resulting tensor.

In addition, the sort pooling achieves something similar.

Hi, thanks for the prompt reply! Regarding to_dense_batch: when concatenation is performed for each graph, the order of the nodes is the same of the node's labellings for each graph? I mean, if I identify my nodes as [N_0, N_1, N_2, ....] (ids used in edge indexes), will the concatenation for each graph also be in the same order?: [F_0 || F_1 || F_2 || ...]
I tried to look to the sort code to understand whether this happens, but I admit there are a few parts which I don't understand yet.

Yes :)

Might be worth to add a sentence in the function's documentation just be clearer.
Thanks a lot, that's all I needed! :)


Perper fabric container stops after some time

When running perper-fabric from docker-compose, the container stops after some time.
In the container logs we can find the following message:
5321884 [client-connector-#73] [ERROR] ROOT - JVM will be halted immediately due to the failure: [failureCtx=FailureContext [type=CRITICAL_ERROR, err=class
o.a.i.i.mem.IgniteOutOfMemoryException: Out of memory in data region [name=default, initSize=256.0 MiB, maxSize=397.2 MiB, persistenceEnabled=false] Try the following:
  ^-- Increase maximum off-heap memory size (DataRegionConfiguration.maxSize)
  ^-- Enable Ignite persistence (DataRegionConfiguration.persistenceEnabled)
  ^-- Enable eviction or expiration policies]]

For Perper 0.7, try passing --max-data-region-size <some number more than 397> in services["obecto.perper.fabric"].command in docker-compose.yml.

ok, fixed;
change will be a part of different commit


fixing typo for pointsWrittenOK

Very basic typo fix for httpd stats in _internal

+1, thanks @sebito91 -- CLA confirmed signed.
I agree this is more line with Go style and metrics are still considered experimental.

Cosmetic change only.


Unit test failing due to NoClassDefFoundError

Describe the bug
Every test that involves classes defined in a different package than the test is defined in fails due to a NoClassDefFoundError when loading the classes defined outside the package the test is defined in.
To Reproduce
Steps to reproduce the behavior:

Define an Arquillian test
Define a class outside the Arquillian test's package
Create a new instance of the class from the Arquillian test
A java.lang.NoClassDefFoundError exception is thrown

Expected behavior
The test should run as expected and either fail or pass.
Desktop:

OS: Linux Mint 19.1
Version 2.3.0.Final

Stacktrace
Exception
Additional context

Code of the Arquillian test
Code of the fixture
Code of the weather station class
Code of the DAO factory interface
Code of the PostgreSQL DAO factory

As this relates to v2, could you create an issue in JIRA: https://issues.jboss.org/browse/THORN

The test uses @DefaultDeployment, so this is actually by design. We can discuss if this design is wrong, but currently, this is the intended behavior. You can construct your own @Deployment manually, and you have full control over which classes and resources are included.

Okay, I'll try to create my own @Deployment. It would be nice if you could include creating an own @Deployment in the guide.


Invalid prompt schema. (type=value_error)

Thx for sharing this repo!
I met a problem when I am trying to costumize few-shot prompt template:
I define the prompt set as follow:
_prompt_tabletop_ui = [
    {
        "goal": "put the yellow block on the yellow bowl",
        "plan":
        """
        objects = ['yellow block', 'green block', 'yellow bowl', 'blue block', 'blue bowl', 'green bowl']
        # put the yellow block on the yellow bowl.
        say('Ok - putting the yellow block on the yellow bowl')
        put_first_on_second('yellow block', 'yellow bowl')
        """
    }
]

and use the following code to construct the prompt
PROMPT = PromptTemplate(input_variables=["goal", "plan"], template="{goal}\n{plan}")
    # feed examples and formatter to few-shot prompt template
prompt = FewShotPromptTemplate(
        examples=_prompt_tabletop_ui,
        example_prompt=PROMPT,
        suffix="#",
        input_variables=["goal"]
    )

then I met an error
pydantic.error_wrappers.ValidationError: 1 validation error for FewShotPromptTemplate
__root__
  Invalid prompt schema. (type=value_error)

anyone knows how to fix this problem?

I find the reason

Hi, I am also getting the same error. Please share the resolution. Thank you.

Please share the resolution. Thank you.

Please share the resolution. Thank you.

I find solution

guys...maybe talk about your resolutions

@zhouliang-yu Can you reveal the mistery?

zhouliang-yu Well, why did you get the error ?


Refactor fetch_all feature

Currently, the list of available methods for the fetch_all feature is hardcoded.
It's a bad approach since we need to maintain this list and there is a risk to forget adding some new methods.
We need to refactor this feature and get rid of these hardcoded values. Also, it would be great to somehow validate the passed method (if it supports pagination), or add some graceful exception handling for that.

This would be interesting to work on. All the method names beginning with list_  would support  fetch_all feature? Or is there a further classification in the supported methods? @andrii-bodnar

@sabarish98 we can rely on the list_ part of the method name, but the main characteristic of these methods is the ability to be paginated (presence of the limit and offset parameters)

@andrii-bodnar : Yes, I understood the pagination aspect, but even if those optional params aren't supplied, we are setting a default value right?
Are you suggesting we use both the method name prefix as well as the presence of these optional params?

@sabarish98 you're right, the optional limit and offset parameters could be missing in the request
Probably, it would be enough to check only the name of the method. I hope that all the list methods have the list_ part 🙂

@andrii-bodnar : Yeah, probably, the method name alone should be enough. We can also check if all the methods that support pagination has the prefix.
I believe we can also raise the same error which is already being raised if an unsupported method is given.
I would love to work on this, but probably after doing the docs first

@sabarish98 sure, thank you! So I'll assign this issue to you

Hey @andrii-bodnar, when's the plan to release it? I will let you know if this can be done before that or not

@sabarish98 there's no plan with a strict deadline, but probably the next week would be good for release.
Anyway, I'm not forcing you to finish this issue 🙂
If you need more time for this then I will just release a new version and then another version will be released.

@andrii-bodnar : That's fine, I was just juggling a lot of stuff and this got pushed back. Open a PR for this now.
Please check #35 and let me know if anything needs to be changed


I cannot see "Help" option, Graphing mode: how to specify function domain (say, 1 < x < 10)?

I cannot see "Help" option, Graphing mode: how to specify function domain (say, 1 < x < 10)?

Usually people use this syntax:

However, Graphing mode gives:

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.


Pressing the enter key while typing in the number input box reloads the page

On the demonstration page (https://mathcookie17.github.io/Eternal-Notations), pressing the enter key while having the number input box focused reloads the page. This is caused by the input box being placed inside a <form> element and the behavior resulting from that interaction.

I was aware of this behavior but didn't know it was actually fixable, but a quick search revealed it is. Fixed.


http://<IP_ADDRESS>:5001

Why did you decide to use 5001 instead of 5000?

I actually sometimes run both quokka (the full application), with these mini-quokka 'quokka-server.py' flask applications. Since flask uses port 5000 by default, that's what the full quokka does; so I just decided to use 5001, for the times when I am running both at the same time.
I might change it back, since I don't see myself running them both at the same time, like I did for one of the lessons.


Update os-upgrade-hana-large-instance.md

Information that customer must include on their os upgrade ticket.

@mottach : Thanks for your contribution! The author, @saghorpa, has been notified to review your proposed change.

@mottach

@ktoliver : Thanks for your contribution! The author, @saghorpa, has been notified to review your proposed change.

Merging per email from @arob98


How can I recompile shmetis?

I see that the project provides shmetis for MACOS compilation, but my installation of metis in Linux does not contain shmetis, could the author please explain what shmetis is or give a way to recompile it.

Hi, thanks for the interest in our project! The official release of hmetis contains shmetis which could be found in this link: http://glaros.dtc.umn.edu/gkhome/metis/hmetis/overview

Thanks for the pointers, I have successfully run shmetis, what determines the number of hypergraphs, on the first epoch of Res18 training I have divided 40 hypergraphs, when will it end?

Sorry for the late reply.
The number of hypergraphs is the same as the number used in [1]. We did not modify this part.
The process can be time-consuming, but it will end (less than 10 minutes).
Please let me know if you have any further question. Thanks!
[1] Accelerating Sparse CNN Inference on GPUs with Performance-Aware Weight Pruning


Continous control agent in plane with obstacles

In this wonderful paper by Huebotter et al., they demonstrated an agent in a 2D planar environment that navigates to a setpoint, which, upon the agent reaching it, jumps to a new location. It's a bit like the old mobile game, Snake, but then in a continuous state-space with a continuous action-space.
I propose we build a similar agent but also add obstacles to this plane. We can use chance constraints (see also #68) to indicate that the part of state-space the obstacle occupies, should not be traversed. The agent should find a trajectory around it.
The next step would be to learn the parameters of the chance constraints through interactions, i.e., the agent bumps into the obstacle and incurs a large prediction error.

Chengfeng implemented a ship control agent operating on a 2d plane, see folder ship.
Implementing obstacles, through for instance chance constraints, is still a todo.

Hey @wmkouw @Chengfeng-Jia! Any developments on this one? I see the ETA is on May 18.

Considering that Bart has already utilized NUV prior to making control decisions to avoid obstacles, I suggest perhaps closing this issue.


Fix the use of custom isotopics combined with a material class

What is the change?
This fixes some problems that were found with #1745. Those issues are described in #1818 .
Why is the change being made?
We are fixing a bug that made it into the codebase.

Checklist

[x] This PR has only one purpose or idea.
[x] Tests have been added/updated to verify any new/changed code.

[x] The code style follows good practices.
[x] The commit message(s) follow good practices.

[ ] The release notes have been updated if necessary.
[x] The documentation is still up-to-date in the doc folder.
[x] The dependencies are still up-to-date in pyproject.toml.

@jakehader   Just a reminder of the ARMI PR policy:
https://github.com/terrapower/armi/blob/bd0b43adadf6e9c0063e27e490353f4a16e1b623/doc/developer/tooling.rst?plain=1#L45-L49
We are all extremely busy, and if a PR gets opened before it is ready I will get an email:

every single time you commit, and the CI passes
for each and every time a single CI job fails

I love the help, and the interesting in fixing the feature. But in the future, please wait until the feature is ready for final review before opening a PR.

@jakehader Just a reminder of the ARMI PR policy:
https://github.com/terrapower/armi/blob/bd0b43adadf6e9c0063e27e490353f4a16e1b623/doc/developer/tooling.rst?plain=1#L45-L49
We are all extremely busy, and if a PR gets opened before it is ready I will get an email:

every single time you commit, and the CI passes
for each and every time a single CI job fails

I love the help, and the interesting in fixing the feature. But in the future, please wait until the feature is ready for final review before opening a PR.

Thanks for the reminder @john-science - I put the PR here but forgot to make it a draft. This was just intended to provide some ideas for how to resolve the ticket and to put something out there to be stress tested by @keckler and others.
There are failing unit tests because I changed the construct arguments so that needs to be resolved still (probably a poor design to require case settings to build a component). Also, we will need to add more unit tests around custom Isotopics (same comment on the ticket).

I ran some tests on my FFTF model to see if this branch works. A couple points:

The keff in my model increased by nearly 5000 pcm to a value that is very incorrect.
The predicted isothermal temperature reactivity coefficient looks correct.
The fuel density when the model is initialized to Thot is way too high (~10%).
The change in density between different Thot values appears to be roughly correct, and the number of atoms is being correctly conserved.

So it seems like this change is half working. Somehow the initialized model is getting the densities very incorrect, though.

Was this fixed elsewhere @keckler?

Nope, still broken

Running my FFTF tests, I now believe that this PR is working correctly.

@john-science I believe that this is ready for your eyes.
In addition to the updated unit tests, I did quite a bit of offline testing with my FFTF model to convince myself that this is working properly.

Random thought: I think this PR is big enough for a release note in doc/release/0.5.rst.


graph load/save into DOT file corrections for py3

Just a small corrections for save / load .DOT files

Fixes #67 presumably.

The problem still happens with galactic. It would be great if this PR is merged to other branches.

@mergifyio backport ros2 humble-devel galactic-devel

I guess mergify isn't set up on this organization.

I'll enable it.

@Mergifyio backport ros2 humble-devel galactic-devel

Thank you !!


[Bug] Periodic CPU spike when there are too many connections

🐛 Bug Report

I'm trying to bump the max nodes count to 100 on my operator node and found that there are strange CPU spikes. I've failed to observe a fixed interval for it.
I've also changed the thread count in main.rs to make it always uses all available cores (basically using sync node settings), except the blocking threads are the same as core count to make profiling data easier to read.
I'm not using the official codebase but I think it's still relevant.
Steps to Reproduce

Connect to like 100 peers
Observe that sometimes snarkOS uses 100% of CPU

Expected Behavior

Ideally peering shouldn't be that resource intensive.
Your Environment

snarkOS 2.0.1, but on my branch (HarukaMa/custom-pool-staging)
rust 1.58
Debian sid

I'm running a profiler to see what's actually using all CPUs.

It looks like almost all CPU time is used to verify PoSW proofs. From the known function calls, they most probably come from deserializing block headers.
I have tried to print some logs around snarkvm_dpc::block::block::Block<N>::is_valid and snarkvm_dpc::block::header::BlockHeader<N>::is_valid, and I've found that it's still related to snarkos::network::ledger::Ledger<N, E>::update_peer: There are 64 linear block headers inside each block locator which means there will be 64 block headers to deserialize. During each deserialization, the PoSW proof inside the block header will be verified. Say each header verification takes about 35 ms (which is what I'm reading from my node), it would take about 2.24 seconds to verify all of the headers. For 100 peers, that will be 224 seconds. Say the machine has 32 threads, that'll be 7 seconds in total. (In my case my test machine only has 12 threads so it takes almost 20 seconds to verify all of them.)
I'm not sure if this is intended or connecting to more peers is actively discouraged, but IMO this process should not take that much CPU resources.

Thanks for the report, @HarukaMa! I'm well-aware of this issue, and now that testnet3 is coming soon, I will propose a mitigation for this :+1:.

Not applicable to testnet3 (at least not in this shape).


Reduce + compose accumulator not resetting

See this codepen:
https://codepen.io/anon/pen/KyNebP?editors=1111
Between the calls i found the accumulator keeps the values while should re-initialize itself.
Expected output is both the calls to return 3 elements the second one instead has 6

Hi @Slumber86,
This is one of the perils of writing point free code in JS.
Once a curried function becomes partially applied, it doesn't re-evaluate or clone its applied parameters.  This is fine for primitive values, because they are immutable, however objects are not. So once you begin assigning properties to the object, they will stay there, even across function calls because it was defined in the outer scope, and is not scoped to the function.
See here.
There  are two ways you could fix the situation.
1, Update your reducing function so that it doesn't mutate the initial accumulator.  Here. Once you remove the mutation by property assignment, the issue goes away.

Wrap the composition in a function. Here. By wrapping the composition, you are creating a new scope, which will in essence reset the accumulator each time the function is called.

Thanks @Bradcomp!
I ended using   reduce((acc, cat) => set(lensProp(cat.category), cat, acc), {}),
This doesn't mutate the acc object so all keep working normally


[#159402131] Use graphql descriptions

https://www.pivotaltracker.com/story/show/159402131
What
Use GraphQL description as content type and field description
For fields we originally used the @config/description directive argument but we decided that GraphQL documentation descriptions [1] are more readable and already part of the GraphQL schema language.
[1] http://facebook.github.io/graphql/draft/#sec-Descriptions

It looks like there was a hiccup during the merge because the last commit 1ea5d8ba6c49abd42e84b55aea2697bee3f72446 on master is the merge of this PR, but it's still open.


$ionicModal show/hide causes event handler leak

In ModalView.show() an anonymous resize event handler is registered.
Later on, in ModalView.hide(), the resize event is removed, using null as the callback argument to ionic.off().
This does not remove the event handler, causing the resize handler to be called a growing amount of times as the modal is shown/hidden.
The correct way of doing this is to use a named function as the callback argument.

Greetings @hravnx!
We are now requiring all issues to be submitted through our custom issue form so that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
This issue will be closed if not resubmitted through the form. Please fill out the rest of the form so that we may assist you.
Thank you for allowing me to assist you.

Greetings @hravnx!
I've closed this issue because my sensors indicated it was old and inactive, and may have already been fixed in recent versions of Ionic. However, if you are still experiencing this issue, please feel free to reopen this issue by creating a new one, and include any examples and other necessary information, so that we can look into it further.
Thank you for allowing me to assist you.


FAILURE: tasks.build_artifact.python_macos_x64_python3.8

https://source.cloud.google.com/results/invocations/b0fe2678-102b-4f76-bcfa-d207f578239e/targets/github%2Fgrpc/tests
Installing collected packages: distlib, filelock, six, appdirs, virtualenv
  WARNING: The script virtualenv is installed in '/Library/Frameworks/Python.framework/Versions/3.8/bin' which is not on PATH.
  Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
Successfully installed appdirs-1.4.3 distlib-0.3.0 filelock-3.0.12 six-1.14.0 virtualenv-20.0.2
WARNING: You are using pip version 19.2.3, however version 20.0.2 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
+ python3.8 -m virtualenv venv
OSError: [Errno 18] Cross-device link: '/Volumes/BuildData/tmpfs/src/github/grpc/workspace_python_macos_x64_python3.8/venv/bin/python' -> '/Volumes/BuildData/tmpfs/src/github/grpc/workspace_python_macos_x64_python3.8/venv/bin/python3'

I've seen this a few times recently.

Likely caused by the recent change in behavior in virtualenv that caused https://github.com/grpc/grpc/issues/21966. Looking into it.


Add maximum roughness cutoff to SSR to improve performance

master version of https://github.com/godotengine/godot/pull/56815.
In a test scene with mixed rough and non-rough materials, this saves upwards of 0.15 ms of GPU time with very little visual artifacting (GTX 1080, 2560×1440). This improves performance the most in scenes that have mostly rough materials. The optimization works regardless of the SSR roughness quality selected in the Project Settings.
The roughness cutoff was empirically chosen – it seems to strike a good balance between improving performance and avoiding obvious visual artifacts. The roughness cutoff could be exposed to a project setting if needed, but the Params push constant is already full (128 bytes).
Testing project: test_ssr_performance.zip
Preview
Some more comparisons: Trying to find a good roughness cutoff, High-quality settings
SSR Roughness Low quality (default)
Before

After

SSR Roughness Disabled
Before

After

I made a pr in the same area of roughness https://github.com/godotengine/godot/pull/56822.

Benchmark with the new roughness cutoff of 0.6 in 3840×2160 on a AMD Radeon RX 6900 XT (Fedora 36, RADV):

Before
After (this PR)

479 FPS (2.08 mspf)
551 FPS (1.81 mspf)

Thanks!

Despite the comment in the shader, I noticed the roughness cutoff in master doesn't actually match https://github.com/godotengine/godot/pull/69828, since it's 0.6 instead of 0.7:
https://github.com/godotengine/godot/blob/950f4c3b41acef0403618dd6169be90530dea5da/servers/rendering/renderer_rd/shaders/effects/screen_space_reflection.glsl#L71-L72
https://github.com/godotengine/godot/blob/950f4c3b41acef0403618dd6169be90530dea5da/servers/rendering/renderer_rd/shaders/effects/screen_space_reflection.glsl#L226-L228
In practice, this appears to have no visual impact, but restoring the "correct" cutoff value of 0.7 makes performance slightly worse (+0.03 mspf in 3840×2160 on a GeForce RTX 4090):
0.6 (master)

0.7 (proposed change)

What should we do? For reference, I've made https://github.com/godotengine/godot/pull/56815 use the "correct" cutoff value of 0.7, but maybe we should make it use 0.6.


Closing to test auto-upgrade job.


Classname hashing functionality to be implemented possibly in styleUtils

This may be unnecessary with the idea of making every single DOM element wrapped by Styled-Component.

Closed for now


use std::lazy::SyncLazy instead of lazy_static

Can we use the nightly feature once_cell?

Thanks for your first PR!

Can we use the nightly feature once_cell?

Speaking for myself, I don't see why we couldn't use it. I'm personally not too familiar with the once_cell feature. It would be great if you could elaborate on the advantages to using this over lazy_static!. Is the idea just to replace a 3rd party dependency?

Remove a 3rd party dependency
Macro free
When the feature is stabled, we can just switch over it
Performance improvement according to this https://github.com/async-rs/async-std/issues/406

This is the once_cell tracing issue: https://github.com/rust-lang/rust/issues/74465

That all sounds awesome! Thanks for including those links. I'll be sure to give them a read 😄

Thanks for the PR!
We've historically stayed away from any gated features (obviously excluding rustc_private which we need for consuming compiler internals). However, I think we're kidding ourselves if we continue that refrain in the hopes of being able to build with a stable compiler some day.
That being said, I'm still on the fence about this one as I don't think it buys us all that much just yet, given that lazy_static is still in our build graph due to various transitive dependencies and we'd technically be pulling in once_cell too.
I concur that this is the right move strategically, but not quite convinced yet that it's all that beneficial today. Will leave it parked for a bit in case I change my mind.

This one is still on hold, but I wanted to mention that the latest tracking issue for this is https://github.com/rust-lang/rust/issues/109736.

lazy_static was removed in #6154


system/audio: check if channels are valid

check ac_channels upper four bits which means min channels.
Change-Id: Ie7f7265437a9186a1708a6cb911a08bcf41fd3ca
Summary
Impact
Testing

@jinxiuxu please fix "Remove Gerrit Change-ID's before submitting upstream".


Emacs package: Quote command arguments.

This allows using the Emacs package with file names containing shell
metacharacters.
Fixes #83

LGTM

Merged as dea1301.


Show different ways to volunteer

There are many people who are willing to volunteer but might not see how. Currently I can't find any tab or page that shows the different ways someone who's interested can get involved, other than the "nominate a candidate" or "contribute" buttons

Just updated this -- there's a "Work with Us" tab at the top now!  Closing this issue.


Feature request

Hey this is more of an  half request. Some accounts in lost kingdom can get locked out there and won't be able to be viewed by tool or manually but their power will still be in top 300. Would be really sweet if could add them just power instead of hopping over those.
Also any chance for formula for Excel to calculate t4/t5 gains , deads.

Hello!
Excel calculation formula will be a feature to consider. About the request you made,  I think it's not how the game works. If your city cannot be located on the map, it's not measured in the total power(stats) calculation of each kingdom. If we can verify the opposite via support, I will consider adding that, but for now, it's not under consideration.
Thank you for your support. I will keep this open as an improvement.

Yeah i found fork of your tool that does look at those accs so i guess it's optional. Thanks for this tool !


libimagequant: 2.13.1 -> 2.14.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/libimagequant/versions.
meta.description for libimagequant is: "Image quantization library"
meta.homepage for libimagequant is: "https://pngquant.org/lib/"
meta.changelog for libimagequant is: ""
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 2.14.0 with grep in /nix/store/lmf94cq9vzp7qsbfdvhclccxrzgcschi-libimagequant-2.14.0

directory tree listing: https://gist.github.com/a61cc2a1e0c44bd8c596848454452646

du listing: https://gist.github.com/aca67809dd636a63e6411f499d3586ec

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

4299 total rebuild path(s)

1335 package rebuild(s)

1219 x86_64-linux rebuild(s)
1022 i686-linux rebuild(s)
832 x86_64-darwin rebuild(s)
1226 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
R
abcl
actdiag
adapta-gtk-theme
adoptopenjdk-icedtea-web
alloy
alloy4
alloy5
almonds
anki
ankisyncd
antlr
antlr3
antlr3_4
antlr3_5
antlr4
antlr4_8
apache-airflow
apache-jena
apache-jena-fuseki
apacheKafka
apacheKafka_2_4
apacheKafka_2_5
ape
apeClex
apktool
areca
asciidoc-full
asciidoc-full-with-plugins
avian
aws-google-auth
axoloti
azureus
bazel
bazel-remote
bazel-watcher
bazel_0
bazel_0_26
bazel_0_29
bazel_1
bazel_3
belle-sip
bibata-cursors
bibata-extra-cursors
blastem
blockdiag
btrbk
bubblemail
buck
buku

Instructions to test this update (click to expand)

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

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

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

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @furrycatherder @marsam for testing.

@GrahamcOfBorg build pythonPackages.pillow python3Packages.pillow

@GrahamcOfBorg build pythonPackages.pillow python3Packages.pillow


about testing

Thanks for your work. I have a question that after training how to test the model on my own data sets?

You can set the mode flag to infer and provide the test images directory path like so:
python main.py --mode=infer --infer_data=path/to/infer/images --output_folder=path/to/where/you/want/the/predicted/images/to/be/saved


Website missing and incorrect links

Maybe useful.
Incorrect links on pages:
API Documentation page link is on the page Learn
Home page link on the page Download
API Documentation  page link on the page Download
API Documentation doesn't have any links

Hello
Thanks a lot i want to rewrite the webpage using maybe angular since it has a lot of repeated HTML stuff it would be better to use some framework to clean things up.
I have been quite busy, unfortunately.  So has been quite hart to find some proper time to work on the project.
Thanks a lot!

If you need help with it (html coding is very easy) I could monitor that part when I can. Your project is very important, because many designers stay away from WebGL and ThreeJS specifically because you have to learn things from scratch.


M3-1311 Attach Multiple Tickets

Purpose
Attach multiple tickets in the file explorer at once on support tickets
Notes

We are not limiting the user to 5 tickets per instance because there's no <input /> attribute to handle this and if we did it programmatically in the onChange, we'd need to pop-up an alert to the user that they can't upload more than 5 tickets at once (which isn't true in some cases).

Coverage decreased (-0.005%) to 26.388% when pulling 66766b53a89c8e3d67ecd7995e5acca23fc8879f on martinmckenna:M3-1311-attach-many-tickets into 44e131c7393dc86211fee31608df42431c0f8c71 on linode:develop.


plugin not found after add

Last login: Fri Jan 14 10:37:18 on ttys000 [oh-my-zsh] plugin 'pnpm' not found
my config

restart my item2, but still not fount @ntnyq

rename folder omz-plugin-pnpm to pnpm and run source  ~/.zshrc

it's right way, thks @sarasate


How to use custom datasets to test the performance of a model?

How to use custom datasets to test the performance of a model? I did not see the test module in the code.I want to do trajectory prediction on my own dataset to test the performance of the model, but I couldn't find the testing module in the code. How to use your own dataset for testing and prediction?

The testing module loads the dataset here. You need to modify the dataset to your own format.

Thank you for your reply. If I want to modify the map information that matches the training set and the validation set, how to do so and how to modify it.

Map information is converted into vectors during training and inference. You need to modify map processing to support your own map format.

Thank you for your reply, which effectively solved my above problems. I have the following questions

If I want to apply this code for trajectory prediction in a certain area, there are already vehicle trajectory data sets in this area, but the editing of the map becomes difficult. If it's a 20m square of empty space, how do we build this simple map and apply it to our algorithm?
What if I only want to use your own map data during testing?
What if I want to change the density of the target point?
This model is really excellent, but I did encounter the above problems in application, thank you for your patience to answer


Enable leader election and 2 replicas

This was addressed in https://github.com/castai/kvisor/pull/188


Stop syntax highlighting of curl.sh after 3,000 characters

Steps to reproduce
Not read.
Expected behaviour
Not read.
Version of Vim
Not read.
Environment
Not read.
Logs and stack traces
No response

The Super User answer 1202897 solves this issue by adding set synmaxcol=5000 to /etc/vim/vimrc for instance.


Use Github Actions via sbt-github-actions

This looks good to me, thanks! @jonas, do you have an opinion here? The plan would be that this PR kicks off GitHub Actions once it's merged, and then we remove Travis CI in a follow-up PR once Actions is confirmed to be working.


Vb.Net

Do you have vb.Net sample how to use this? How to download new records.

Hi,
Currently we don't provide a VB.Net sample, however code is very simple to translate. This should get you started:
Imports Anviz.SDK

Public Class Form1
    Private Async Sub Form1_Load(sender As Object, e As EventArgs) Handles MyBase.Load
        Dim manager As New AnvizManager(0)
        Using device = Await manager.Connect("<IP_ADDRESS>")
            Dim employees As New Dictionary(Of ULong, String)
            For Each emp In Await device.GetEmployeesData()
                employees.Add(emp.Id, emp.Name)
            Next
            For Each stamp In Await device.DownloadRecords(False) 'True to get only new records
                Dim employeeName = employees(stamp.UserCode)
                Dim recordDate = stamp.DateTime
                MsgBox(employeeName & " at " & recordDate)
            Next
        End Using
    End Sub
End Class

Thanks, working already but only the download new record. Download all record does not work. Im using W1.
Now working in EP300.

Does the included c# sample work for you? Can you try cloning the latest development branch and see if it works? Thanks

Yes tried cloning it, only the download new record is working. Download all does not work.

If you can download new records, you should be able to download all records. Can you write the returned data here?

Hot to do it?
When i checked the records variable it return null value.

Connected to device W1 ID 11 SN<PHONE_NUMBER>200116
Current device time is 8/22/2019 6:23 AM
Device IP is <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> 08-00-0F-06-37-27 mode is Server
Employee 1 ->  pwd  card  fp RightThumb
-> RightThumb GxkVVcEbIlehIAwUYTahGME8GCzBUyBcgVSxXwFiHCJBdihjoRwbrGIrnVYiLSfDIjylw6JFG5jCR4XRokqxhkJLKZyiT6CD4lQF5aJXGesCXpilQmWN5IJlk6SCZpZPAmmZZGJzCw4icxcOwgAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAABsUmVWhIR8WgSEjQAE0pZiBUTWfYVUk3KFdnuLhYoWOgWUfoqF1q+OhF5/sQhelluIdj9NiJ6FWgiksg2I4nmxCOqrDgkUKEeJFoJjiSa3cYlClA+JRCmbCWR5AQmWRZQJnGk5Capzkom8OzwIAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAtdg=
Employee 2 ->  pwd  card  fp RightThumb, RightIndex
-> RightThumb ECGmLCE9r9shQxRlgVMkoyFWHCQBdx9hwSSs2IIskeeiMKeWIjmvw4JDoZMiTiYeYlUo4GJlrSCCZyghom8tS4IAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA8TnewBGqSZgSIfGMFFrR9BbZpLwS4lg+Ixo5vCNhbTwkaQJCJGp98iR5ygwkmriOJZoCDiWxuiImWgiqJvLUuCAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAtdg=
-> RightIndex FhATFKEbk6pBHibDISORKMElm1hBJaWbYS0hA4ExE5OBSw0OIVwo4ME8C2WiPBUUAjwiHaI8mVuCSJTkYkmcB4JMoOCiV50iAmQL4kJwkguCcY9KAniTDqIAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAABYVFuqBGSiCoR8fGAEfKNuBJqSDQUMQTqFEGWIhSaSgYVkroYFhlc2hZpONwSqV0oI1DqXCNZhUAjYc26I2pZ5iQ59H4kWdIQJRoCLiXI5iImuRi2J1IeOCAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAtdg=
Employee 4 ->  pwd  card  fp
Employee 222 ->  pwd  card  fp

Try the new released version. The sample program outputs at this line
Console.WriteLine($"TotalUsers {stats.UserAmount} TotalRecords {stats.AllRecordAmount}");

It should have the correct values

I'm closing this issue, feel free to reopen it if your problem is not solved

What Features should i use if i want to disconnect? and how do i know if i was connected to device?

AnvizDevice implementa IDisposable, so you need to dispose the object to disconnect. You cannot know if you are still connected or not, you can only sent commands and if they succeed then you are still connected

AnvizDevice implementa IDisposable, so you need to dispose the object to disconnect. You cannot know if you are still connected or not, you can only sent commands and if they succeed then you are still connected

Does the Download Record in AnvizDevice is getting all records of attendance?
I was trying to create it in VB.net
Dim records = Await device.DownloadRecords(True)
For Each rec In records
Dim uid As String
Dim datetotime As Date
Dim recordtype As String
Dim worktype As String

uid = dict(rec.UserCode)
datetotime = rec.DateTime.ToShortDateString() & " " & rec.DateTime.ToShortTimeString()
recordtype = rec.RecordType
worktype = rec.WorkType

                Dim newitem As New ListViewItem()

                newitem.Text = uid
                newitem.SubItems.Add(datetotime)
                newitem.SubItems.Add(recordtype)
                newitem.SubItems.Add(worktype)

                LvAll.Items.Add(newitem)
 Next

And what command should i use to go in AnvizStream to get the Dispose


bug: performing action automatically changes mode to Insert mode

Neovim version (nvim -v)
0.10.1
Operating system/version
14.4
Describe the bug
When doing quick actions like restart or watch it constantly changing the mode to insert mode even if I'm not in insert mode
What is the severity of this bug?
minor (annoyance)
Steps To Reproduce
:OverseerQuickAction restart
Expected Behavior
Don't change the vim mode
Minimal example file
No response
Minimal init.lua
{
		"stevearc/overseer.nvim",
		keys = {
			{ "<leader>cc", "<cmd>OverseerRunCmd<cr>", desc = "run cmd" },
			{ "<leader>ct", "<cmd>OverseerToggle<cr>", desc = "overseer" },
		},
		opts = {
			dap = false,
			task_list = {
				bindings = {
					["<C-u>"] = "ScrollOutputUp",
					["<C-d>"] = "ScrollOutputDown",
					["<C-q>"] = "q<cmd>OverseerQuickAction open output in quickfix<cr>",
					["r"] = "<cmd>OverseerQuickAction restart<cr><esc>",
					["x"] = "<cmd>OverseerQuickAction dispose<cr>",
					["w"] = "<cmd>OverseerQuickAction watch<cr>",
				},
			},
		},
	},

Additional context
No response

Yep it would be nice to avoid that. My guess is that it's because of the TermEnter autocmd that does this

Please use a minimal init.lua file that is a complete config, as referenced in the bug report template.

Well I just did here (reproduction) but it doesn't do it in isolation 😅 ...
I'll try to find where it comes from and report here if I ever found out

Not sure if it is related to the OP, but in my config I have, which was the root of the issue, because overseer opens a lot of terminal windows without actually focusing them. Like OverseerToggle or any actions which open terminal buffers.
vim.api.nvim_create_autocmd({ "BufWinEnter", "WinEnter" }, {
   pattern = "term://*",
   command = "startinsert"
})

Not sure if it is possible to fix from plugin side. Maybe checking the mode it is in and if it is in insert mode change it to normal? And for me :startinsert seems like a hack and enters insert mode even in non-modifiable buffers.
But from user side there are some solutions to narrow such autocmd to only specific terminal buffers, etc.

I'll try your autocmd in my repro I might have a similar one I overlooked in my config, but the one there (termopen) doesn't interfere with overseer

Yuo the problem was caused by an autocmd configuration. Thanks for pointing it out, and it’s resolved now!


feature: Be able to identify which Space the object resides in (from the object)

Feature Description
KCP objects have special annotation("kcp.io/cluster") that holds a reference to the cluster. From that reference is is also possible to obtain the  workspace. This annotation is set by KCP and will not be available in other providers.
For example, when we use a watch or List over an APIExport using kube-bind, we get a list of objects and such annotation is the only way to know which cluster the object belongs to.
Proposed Solution
Be able to get the cluster (or space) from the  object (e.g., through the NS name, annotation, other way)
Want to contribute?

[ ] I would like to work on this issue.

Additional Context
No response

Can I have some more information? If my understanding is correct, the KS platform objects (EdgePlacements, for example) are the ones that kube-bind is responsible to sync between the KS space and other spaces. OTOH, the objects that representing users' workloads are not. Am I right?

I guess we have a solution for this right? And, since that solution is documented and tracked by #1208, can we close this (duplicated) one?

Included in https://github.com/kubestellar/kubestellar/issues/1208   second section


⚠ Bofrost.de has degraded performance

In 3b31f69, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1665 ms

Resolved: Bofrost.de performance has improved in fff4242 after 9 minutes.


Manual Alarm doesn't restore previous state after reboot

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.57.x
0.58.x
Python release (python3 --version):
3.4.2 (Raspberry Pi 3)
Component/platform:
Manual Alarm Control Panel
Description of problem:
The Manual Alarm Control Panel does not restore its previous "armed away" state after rebooting homeassistant.
Expected:
Restore the state after reboot.
Problem-relevant configuration.yaml entries and steps to reproduce:
alarm_control_panel:
  platform: manual
  code: 7777

Traceback (if applicable):

Additional info:
It seems like the issue has been introduced with some update since 0.57.x

i have the same issue on 0.60.0

Same here in 0.60.0. Would it possible to fix this behavior?

According to #11552, this is still a thing.

Still happens in 0.63.4

This custom sensor solves the problem, but would be nice if it wasn't necessary: https://github.com/gazoscalvertos/HASS-Persistence

Still an issue on 0.67.0. My alarm state goes back to "Armed Away" after every restart.

Still this issue on 0.69.1

Yeah and the persistence sensor i linked to above hasn't been updated to fix it's issues.  So i'm unaware of any workaround/working solution to this problem.

Seriously!  Still not fixed! Is the alarm component not a "security" element of HA? We had power outage for a couple hours today and was reminded of this issue when all my automation since have not worked.  UPS did its job but no one was around to gracefully turn some things off and back on when power was restored.  I am however happy my HA installation on Raspberry PI started back up without issue!
I originally commented on #11552.  Being aware of this issue I know to reset the alarm when I do upgrades but have forgotten at times.
Why is this issue not being prioritize?

Bump.

Same here

Why is this still an issue!?  It's a security component!

Just adding my concern for this issue to be resolved.  I am also using the recorder and still the state is reset to disarmed after a restart.

I don't think those numbers are for watching this issue, it means how many are watching the entire git for HASS.
I've written an AppDaemon script that restores the state of the alarm.  Here's my code:
`import appdaemon.plugins.hass.hassapi as hass
import shelve
import time
import json

App to reset alarm_control_panel to previous state after HA restart

Args:

file - file in which to save state

class AlarmReset(hass.Hass):
def initialize(self):
self.alarm_handler = self.listen_event(self.store_alarm_state, "call_service", domain = "alarm_control_panel")
self.listen_event(self.restore_alarm_state, "plugin_started")
def store_alarm_state(self, event_name, data, kwargs):
with shelve.open(self.args["file"]) as db:
db["alarm setting"] = json.dumps({'state': data['service'], 'entity': data['service_data']['entity_id'], 'code': data['service_data']['code'] })
self.log("Alarm state changed to {}".format(data['service']))
def restore_alarm_state(self, event_name, data, kwargs):
self.cancel_listen_event(self.alarm_handler) # to avoid a potential loop, not so bad though
with shelve.open(self.args["file"]) as db:
alarm_settings = json.loads(db["alarm setting"])
self.log("Restoring state to {}".format(alarm_settings['state']))
state = alarm_settings['state']
entity_id = alarm_settings['entity']
code = alarm_settings['code']
self.call_service("alarm_control_panel/{}".format(state), entity_id = entity_id, code = code)
self.alarm_handler = self.listen_event(self.store_alarm_state, "call_service", domain = "alarm_control_panel")`
And the config portion is this:
Alarm_Reset:
class: AlarmReset
module: alarmreset
Choose where to store the states across restarts
file: /home/homeassistant/.homeassistant/alarm

@FuzzyMistborn I did edit those numbers out, I didn't realize it was for the entire project.  I'm not entirely sure how to incorporate your fix.  I am curious, why the recorder wouldn't be able to save the state though just like it does for other entities.  It certainly keeps a history of the alarm state over time, so reason would have it that it could keep the state of the alarm just as it does boolean_datetime states for instance.

I agree it's something the recorder should pick up on and restore, but I'm not expert enough to make that fix.
You need to install AppDaemon to run my "fix" (it's not a fix, it's more of a work around).  Google/look around the HASS forums for more info on how to get AppDaemon set up.

Just adding to the thread to count me as another who is impacted by this issue.  I'm not liking that I would have to go the Appdaemon or custom component route as a workaround just to restore the alarm state after a power outage.

This issue's been bugging me for a while... Decided to have a crack at solving it myself.

I will PayPal you some cash if you fix it. I have no workaround as I can’t seem to get the HASS persistence script to run. It’s annoying when my alarm ends up armed after every restart.

Glad to see this is being fixed after almost a year!

@murphyslaw05 The component initializes to disarmed state by default. I didn't see anything that would cause it to start up armed. My issue was certainly with it restarting and disarming the alarm.
Either way, the fix i submitted gets the last state from DB at startup.

Mine ALWAYS starts in an armed state. No clue why as I have specified it that way. Great to hear that your submitted fix pulls the last state. That’ll solve both our problems.

@murphyslaw05 Are you definitely using the manual alarm control panel, not the mqtt one?

It would seem to me, the easiest fix would be for both the Manual Alarm and the Manual Alarm with MQTT to be added to the recorder for state history tracking.  Although, upon further inspection, I do see a history for my alarm state, doesn't that mean the recorder is already saving the state over time?  And if so, why are these two components not just retrieving their last state like most other components from the recorder?  If this is an undesirable affect, then the alarm components could have a simple configuration flag added for retain_state. In my honest opinon, that is!

@liaanvdm would your fix also work for the Manual Alarm Control Panel with MQTT Support?  manual_mqtt:

william,  in referring to the in-progress fix PR (#17521) , liaanvdm specifically states "This fix will restore the last state that was logged by the logger component whenever hass restarts."
I did not see anywhere a mention of this fix also applying to the manual alarm with MQTT, so I can't comment that it would also apply.

The fix wont apply to the mqtt component. I haven't used that component but
can't you use retained messages feature of mqtt to restore the state? If
not that would be a nice feature to add as well. But I could probably apply
the same fix to the mqtt alarm component, I'll have a look tomorrow.
On Sun, Oct 21, 2018, 8:33 PM GaryOkie<EMAIL_ADDRESS>wrote:

william, in referring to the in-progress fix PR (#17521
https://github.com/home-assistant/home-assistant/pull/17521) , liaanvdm
specifically states "This fix will restore the last state that was
logged by the logger component whenever hass restarts."
I did not see anywhere a mention of this fix also applying to the manual
alarm with MQTT, so I can't comment that it would also apply.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/10793#issuecomment-431692536,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ApPKt50DcPKgC8MVdMN_Y-HA0Z3beDJVks5unL4RgaJpZM4Qqb-6
.

@liaanvdm as with the Manual Alarm, I do not believe that this component restores from any state.  Now, it would be fine with me if it restored it from the MQTT value, but no at this time it does not allow for retained messages to set its state after a reload either.
Also, I'd like to point out that the MQTT Support isn't mean for MQTT to be the alarm panel but only to update and read its state from the alarm panel within Home Assistant.  Therefor, I would think it would be more proper to do it using the recorder, same as the manual alarm.

@thewilliambond I've had a look and applying the same fix to restore the alarm state using states logged by the recorder component is not as straightforward as I'd hoped. The problem is the start up trigger is already used in the code to subscribe to the mqtt topics and I cant get the asynchronous db restore state methods to play well with this.
What I did see is that state changes are published to mqtt with the retain flag enabled so it looks to me like, as long as your mqtt keypad is also publishing with a retained state, the hass component should recover the state on startup.

Bumpint this thread as this is still an issue with 0.82.0.
Any hope for a solution? I would really want my house to stay protected even if there has been a power failure.

And are you sure you're using the manual alarm and not manual alarm with mqtt?  The patch does NOT fix the mqtt version

And the fix will only work if you have the recorder component enabled and
it's logging the manual alarm control panel's state.
On Wed, Nov 14, 2018, 5:54 PM Geoff<EMAIL_ADDRESS>wrote:

And are you sure you're using the manual alarm and not manual alarm with
mqtt? The patch does NOT fix the mqtt version
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/10793#issuecomment-438712050,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ApPKt_5z1S8HkTD_sydYyUaGga8FX4Drks5uvDyrgaJpZM4Qqb-6
.

Just upgraded to 82.0.    Manual alarm (no mqtt) properly restored to alarmed state after HA restart.
Also tested rebooting Hassio, and that also restores the state properly.
Well done liaanvdm!    Thanks so much for fixing this.

Sorry everyone. I was too quick. I have the Ine with mqtt.

There is a bug. If the status is Pending and I reboot Hassio then after the reboot the status remains in pending forever.

Yikes, I never tested that case. Thanks for reporting
this, I will look into it and report back soon.


fix: add flush to tokio file writer in write_ml_model tests

This PR is intended to fix a random error, where the tokio file writer sometimes hasn't actually finished, before the test reads the relevant content.

 Bug: CI failed zufällig, wenn async io im write_ml_model test durcheinander kommt

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 2 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.01%) to 87.823%

Files with Coverage Reduction
New Missed Lines
%

services/src/pro/handlers/datasets.rs
1
93.7%

services/src/pro/handlers/drone_mapping.rs
3
94.91%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
86022

Relevant Lines:
97949

💛  - Coveralls

bors merge

bors retry


Credential bastion

credentials bastion to provision an EKS cluster
credentials bastion to create EFS filesystem
adds a conditional flag (default false) for the privileged IAM policy to be enabled on the bastion

supports reusability of the module in different capacities without over-permissioning those resources

/test all

closes #248

Is this PR meant to address the issues with the <cluster>-aws-efs-csi-driver-sa-irsa policy that seems to be blocking the ability to create EFS access points?

Is this PR meant to address the issues with the <cluster>-aws-efs-csi-driver-sa-irsa policy that seems to be blocking the ability to create EFS access points?

yes... thank you for reminding me. Updated PR description and issue to capture that as well


mic input very poor on bluetooth

I just started to use mumble, on iOS 12.2 on iPhone 6S plus and iPhone 6, any bluetooth earbuds; the mic input is very choppy. I tried all settings under VOX, very little changes. With wired earbuds, everything is great.
Under Plumble on Android 4.2.1 on Nexus 4, both wired and bluetooth work at default settings.
Under mumblefy (on iOS 12.2), both wired and bluetooth work at default settings.

Hi, I did some digging into this and I believe I've identified the root of this issue! It appears to be a sample rate problem.
I found another project called Audio Kit where people were experiencing similar symptoms, and it turned out that a system call was giving them an incorrect sample rate for the microphone only, causing an improperly sized buffer to be generated and creating the choppy audio also experienced by Mumble for iOS users.
I pulled down the Mumble for iOS source code and started experimenting. I found that the problem is resolved if you enter the MumbleKit source and modify line 13 of MKAudio.h to use a lower sample rate, e.g.:
#define SAMPLE_RATE 16000
Now, I don't know that much about this codebase, or iOS programming at all for that matter. But I noticed that in MKAudio.m there is a method that sets the preferred hardware sample rate at line 203 that includes a comment:

The AudioSession can reject this, in which case we need to be able to handle whatever input sampling rate is chosen for us.

My guess as to the actual root cause of the problem is that the Bluetooth headsets experiencing this choppy audio problem are headsets where the maximum input sample rate is lower than the maximum output sample rate. Looking through other areas of the source, MumbleKit seems to always assume they are the same. By setting a lower "preferred" sample rate that both input and output are compatible with, the issue is resolved.
The long term solution is to update the entire codebase to appropriately handle disparate input/output sample rates.
As a short term solution, a switch could be added to the Preferences/Advanced menu called "Bluetooth compatibility mode", which would force a common lower sample rate like 16000.
@mkrautz : I would do either of these myself and submit a Pull Request, but I have very little experience with Objective C and iOS, so I'm not sure my code would be up to snuff and it would take me a very long time. Is this something you might have the bandwidth for, or should I try my best and submit a PR?


fix stdlib

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

git@github.com:jornbr/gotm-feedstock.git

@conda-forge-admin, please rerender


Check out test version of API (in email)

@James-Sangalli reassign to me if you'd like.

I will need to produce some design document for that. Will commit the design documents and ask for your comment. This issue doesn't belong to ios so I close it.


logTarget cannot be blank error occurs when each service starts up

🐞 Bug Report
Affected Services
The issue is located in: each service in edgex-go repository
Is this a regression?
Yes, the previous version in which this bug was not present was: Edinburgh (v1.0.1)
Description and Minimal Reproduction
Use the docker-compose file in developer-script and start EdgeX by
docker-compose up -d
Then, see the logs in each service:
docker logs edgex-core-data
docker logs edgex-core-metadata
docker logs edgex-support-logging

🔥 Exception or Error

You will see the following error logs:
level=ERROR ts=2019-10-24T05:44:35.969925173Z app=edgex-core-metadata source=logger.go:105 msg="logTarget cannot be blank, using stdout only"

🌍  Your Environment
Deployment Environment:
Ubuntu 18.04
EdgeX Version:
fuji (master branch)
Anything else relevant?

Known issue.
Bootstrap creates logger with empty logTarget here.  This happens before configuration is loaded and is done to provide a consistent logging interface.  The error is harmless but should not be logged.  Error occurs here.
Fix is to add a second factory function in go-mod-core-contracts that does not take a logTarget. It should execute all of the same logic (preferably without duplicating it) that the existing NewClient method executes except for the if..then that logs the error when logTarget == "".
Once this change is merged, edgex-go should be modified to use the updated go-mod-core-contracts and the FactoryToStdout function in the bootstrap should be modified to call it.


handling missing values

AlgebraOfGraphics don't seem to handle missing values the same as Makie. Consider

Using dropmissing on the full DataFrame is not viable, as the missings are likely to be in columns you don't use. A workaround is
data(dropmissing(allison, [:BodyWt, :Gestation])) *
	mapping(:BodyWt, :Gestation) |>
        draw

but I believe you would expect AoG to handle missing like Makie (and it would be more user friendly - the current behaviour seems like a bug.)

I would rather get an error and pass skipmissing=true where needed rather than have incomplete and misleading results by default. Maybe there could even be a functional "preprocessing" API similar to the Analysis API which would allow transforming the data prior to plotting.
data(d) * preprocess(dropmissing) * ...


Do not add multiple appenders

Prevent an appender from being added to log4j each time the scheduler is included

👍


Kami creature type missing

The list of Kami creatures is pretty short, but it still is apparent the category is missing, when they're considered unknown.
on a technical level, they are spirits, so depending how strict you're using the categorization, they could go under there.
Half the Kami has both the Kami and Spirit trait, example: Toshigami.

That is a solved issue 😄
Kami isn't an official creature type in the system, but you can easily add anything you want as a bookmark. There's a setting for selecting any defined Creature Traits to be used for additional bookmarks. And since you can add more Creature Traits in the Homebrew section of the system itself, anything goes.
For the Kami usecase, you'd just need to go into the Bestiary menu and add Kami to the list since it's already a creature trait.


only check cache for "previously stuck to master" once per context

Hi TaskRabbit people, I'd like to propose this change to help reduce usage of the context cache.
We use makara in a master-slave, multi-app server rails environment.  We use redis for centralized caching and for Rails.cache.  We noticed that over 90% of the commands processed by our redis master are for makara contexts.
Looking through makara code, and debugging, I found that in _appropriate_pool in proxy.rb, if the current context is NOT master, then every DB request during the processing of a single web request calls previously_stuck_to_master, which queries redis (in our case).  A lot of our web requests are page loads that do a lot (say, 40+) DB requests, so it also hits redis 40+ times.
However, during handling a web request, previously_stuck_to_master is either true or false as of the first use of _appropriate_pool, and it won't (and can't) change during the handling of that web request.  So it would be much more efficient to just check previously_stuck_to_master once.  If previously_stuck_to_master returns true, then the current context is switched to master, and any subsequent _appropriate_pool calls will use master because current == master (this is existing behaviour).
It's a bit convoluted to try to explain but hopefully that makes sense.
The fix I made just uses a thread local var to track whether the previous context has been checked yet or not.  It is set to false when Makara::Context.set_previous is called (it's called via middleware.rb before handling a web request), and then it is set to true in _appropriate_pool after previously_stuck_to_master is called.
If you want to change the method names or signatures I added, feel free, or let me know and I'll change them.
Thanks!
Greg Patrick -<EMAIL_ADDRESS>

Thanks for the PR @vt-gpatrick
What would you think about storing the boolean value of !!Makara::Cache.read("makara::#{Makara::Context.get_previous}-#{@id}") in the thread instead of if it checked?
Basically, we can memoize that Cache.read in previously_stuck_to_master? and keep calling previously_stuck_to_master? from above. I'm not fully clear on the nuances at play here and how they all get called especially with @sticky but that would seem to have the same effect with slightly less risk.

Hi @bleonard, yes, that's a good idea.  I didn't think of that but it definitely makes sense.  I'll make some changes.  Thanks!

Hi @bleonard I updated my PR with the changes you suggested.  I like it better now.  Added a couple more unit tests too.

Thanks! Looks great.


Put start/end in URL hash so they can be linked to

I thought this was really neat, except it was missing one thing... the ability to link interesting paths/scales to other people. So I added that.
Clicking 'Go' now puts the start and end values in the location hash.
When the page is opened, the values are pulled out of the hash and the inputs are set to the given values. If both start and end have values in the hash, then it waits 1 second and starts trying to find a path.
+ in the hash are correctly converted back into spaces.

This looks great thank you, it is cool that you found this project and contributed.
However this PR made me realize that I hadn't actually released ratewithscience under any sort of license. @davepagurek and I just updated it to be released under the MIT license (see #3), do you consent to your code being released under the MIT license?
If so I plan on merging this great PR.

Yeah, the MIT license is totally fine. :+1:

Great thanks again, I'll try and get around to actually deploying this to my server some time soon, perhaps tomorrow.
May I ask how you found Rate With Science? Most people who know about it heard about it through word of mouth at UWaterloo from the hackathon, which I assume isn't how you heard about it.

I was linked to it by a friend, but they don't remember where they found it. Alas.