HHH-14146 Add Presto dialect

Add dialect for Presto (https://prestosql.io/)

Thanks! Great to see support for new modern databases being contributed.
We'll need to add Presto support to our continuous integration systems; my preference would be a container, as it helps us to diagnose issues locally - should we need to.
Would you suggest we use https://github.com/prestosql/presto/tree/master/docker ?

We'll need to add Presto support to our continuous integration systems; my preference would be a container, as it helps us to diagnose issues locally - should we need to.
Would you suggest we use https://github.com/prestosql/presto/tree/master/docker ?

@Sanne correct. You can bring up Presto container  up with eg
docker run --rm --name presto -it -p 8080:8080 prestosql/presto:340

Rebased.

The CI build is green.

@Sanne can i do anything to help move this forward?

@Sanne should i rebase?

hi @findepi ! Apologies, I've been overwhelmed by notifications and missed this completely.
Yes could you rebase it once again please? I'll see to find someone to setup the CI task on CI.

Rebased.
There are some build failures, like
2020-12-11T16:19:18.4064992Z > Configure project :hibernate-agroal
2020-12-11T16:19:18.4133623Z Unable to decrypt Maven password using PlexusCipher
2020-12-11T16:19:18.4136325Z org.sonatype.plexus.components.cipher.PlexusCipherException: java.lang.ArrayIndexOutOfBoundsException
2020-12-11T16:19:18.4144129Z 	at org.sonatype.plexus.components.cipher.PBECipher.decrypt64(PBECipher.java:193)
2020-12-11T16:19:18.4147647Z 	at org.sonatype.plexus.components.cipher.DefaultPlexusCipher.decrypt(DefaultPlexusCipher.java:72)
2020-12-11T16:19:18.4151391Z 	at org.sonatype.plexus.components.cipher.DefaultPlexusCipher.decryptDecorated(DefaultPlexusCipher.java:86)
2020-12-11T16:19:18.4158323Z 	at org.hibernate.build.publish.auth.maven.pwd.DecryptionPasswordStrategy.decrypt(DecryptionPasswordStrategy.java:78)

@Sanne please help me understand what I am doing wrong.

Thanks! Don't worry for the CI job, these are new jobs which have just been added and we still need to sort out some details - sorry about that.

Hi @findepi , with CI having issues I tried to run tests locally - but it failed.
I used:
podman run --rm --name presto -it -p 8080:8080 prestosql/presto:340

./gradlew matrix_presto

Did you test it differently, or perhaps some more changes are needed?

I didn't run tests recently, only rebased, let me take a look.

@Sanne i remember now & sorry for the trouble.
Presto is not capable of housing a typical application, as it still lacks some features commonly seen in relational databases.
For example, if you run PaginationTest these are the limitations you would run into:

Presto allows declaring column to be NULL or NOT NULL, but the memory connector used for testing does not support that yet
Presto does not support defining indexes (primary keys, unique keys)

For these reasons the preparation part of the test (PaginationTest#prepareTestData) won't pass.
As such, you should not expect a typical Hibernate-based app to ever dream of moving to Presto, which is an analytical database.
Yet, some Presto users were asking about Hibernate support for Presto, supposedly to benefit from power of Presto in Java applications without need to write SQL by hand.
See PrestoDialectTest for some tests that i added to ensure the dialect actually works.
I also removed the hibernate-orm/databases/presto/matrix.gradle file, as i did not find a way to alter test data setup.

May I know the status of this PR? So there won't be Hibernate support for Presto?
Thanks.

As per https://hibernate.atlassian.net/browse/HHH-14693 and https://github.com/hibernate/hibernate-orm/discussions/3783 we decided not to include any dialects in 5.5 anymore for now and focus on 6.0.
IMO it would be fine to add it to 5.5 though if the Dialect for 6 qualifies for hibernate-core according to the requirements document: https://github.com/hibernate/hibernate-orm/blob/wip/6.0/dialects.adoc

@bradhill99 if i were to continue here, i would create a Trino dialect, as per https://trino.io/blog/2020/12/27/announcing-trino.html
@beikov i can reopen the PR (with new target module) if needed.
i closed this, because there was seemingly low interest from potential reviewers, so didn't want to waste your time.

You can reopen this PR against the main branch after a Dialect for 6.0 is merged according to the requirements for the hibernate-core inclusion.

@findepi Thanks for the reply. It will be great to have presto dialect for Hibernate.

What do you mean by follow? Adding new dialects is a community or vendor driven effort. We as Hibernate team explicitly do not want to bother with any new dialects anymore unless someone steps up and provides full support for that. Read the document about the process: https://github.com/hibernate/hibernate-orm/blob/wip/6.0/dialects.adoc


IOC Extraction

As a user I want to be able to extract IOCs from the text. I want to be able to enter my own regex to extract the IOCs and test those IOCs. Finally I want to be able to view the IOCs for a given report on the report detail page.
Acceptance Criteria:

Regexes can be added to parse out custom IOCs
Extracted IOCs can be viewed on the report detail page
IOCs are also exported with reports.

Duplicate of: https://github.com/center-for-threat-informed-defense/tram/issues/68


Week 1

My first week assignment.

this submission doesn't satisfy submission requirements, please check the readme


Consolidate provider

Instead of throwing an error by default, Presenters just pass along children (if they can). This sort of makes the Provider addon pointless.

Changes Unknown when pulling bc9c5f106b6a949202c7c07a6f05fe712486a209 on nh-update-addons into ** on master**.

Changes Unknown when pulling bc9c5f106b6a949202c7c07a6f05fe712486a209 on nh-update-addons into ** on master**.


Future proof ProtocolType

What would you like to be added:
Future proof ProtocolType to minimize collisions of type definitions in future.
Priori art is available with AppProtocol.
Why is this needed:
Protocols in Gateway.Spec.Listeners are open-ended strings in the API. A user can define a ProtocolType in a custom implementation, service-apis then introduces the same type as a supported core/extended type and there can be collisions.

/assign


multiprocessing + .osm.pbf file = deadlock

For a side project of mine ( https://gitlab.com/ltog/gwr-filter ) I did some experimenting using subprocesses to speed up the processing of OSM files by pyosmium. In general, this worked well. However, when trying to implement a two-pass read for evaluating associatedStreet relations , I have encountered some strange behaviour that looks like a bug in osmium/pyosmium to me.
Call the script below like this:
python3 ./gwr-mini.py liechtenstein-latest.osm liechtenstein-latest.osm.pbf

(You can use the extracty provided by http://download.geofabrik.de/europe/liechtenstein.html but I guess any .osm.pbf will do...)
You will see that the script hangs at the line marked A. Interestingly, this is true if and only if both following conditions are met:

The second argument is of type .osm.pbf. (.osm files work fine)
The seemingly unrelated line B is active.

I tested with versions 3.0.0 and 2.15.4.
#!/usr/bin/env python3

import osmium
import sys
from multiprocessing import Pool

class FirstOsmReader(osmium.SimpleHandler):
    def __init__(self):
        super(FirstOsmReader, self).__init__()
        print(f'Initializing {self.__class__}')
        self.count=0

    def relation(self, r):
        if self.count<4:
            print(f'FirstOsmReader.way(): r={r}')
            self.count += 1
        pass

    def work1(self, filename: str):
        self.apply_file(filename=filename, locations=False)
        return

class SecondOsmReader(osmium.SimpleHandler):
    def __init__(self, worker_id: int, num_workers: int):
        super(SecondOsmReader, self).__init__()
        print(f'Initializing {self.__class__}')
        self.num_workers = num_workers
        self.worker_id = worker_id
        self.count=0

    def node(self, n):
        if not n.id % self._num_workers == self._worker_id:
            return
        if self.count<4:
            print(f'SecondOsmReader.node(): n={n}')
            self.count += 1

    def work2(self, worker_id: int, num_workers: int, filename: str):
        self._worker_id = worker_id
        self._num_workers = num_workers
        print('work2: before apply')
        self.apply_file(filename=filename, locations=True) # A: this line hangs
        print('work2: after apply')
        return

def work(worker_id: int, num_workers: int, filename: str):
    second_osm_reader = SecondOsmReader(worker_id, num_workers)
    return second_osm_reader.work2(worker_id, num_workers, filename)

if __name__ == '__main__':
    num_workers = 2
    f1=sys.argv[1]
    f2=sys.argv[2]
    print(f'f1={f1}')
    print(f'f2={f2}')

    first_osm_reader = FirstOsmReader()
    first_osm_reader.work1(f1) # B: if this line is active, it triggers a hang in line A

    with Pool(processes=num_workers) as pool:
        w = num_workers
        myargs = zip([x for x in range(w)], [w]*w, [f2]*w)
        print(f'myargs={[x for x in range(w)], [w]*w, [f2]*w}')
        for result in pool.starmap(work, myargs):
            pass

You cannot simply fork a libosmium application like that. libosmium holds a global thread pool that it uses when parsing the files. When you fork, only the main thread gets forked and the thread pool ends up in an invalid state. It has only worked until now because the thread pool happened to be still empty when you set up the Python Pool. And it works with an .osm file because xml parsing doesn't use the thread pool. Using the libosmium C++ API you could create a private thread pool per form and use that in your reader. pyosmium does not expose the interface for that, so that is not an option for you. You can use the Python Pool in spawn mode. This will reinitialise the libosmium library for each of your processes and all is well.
On a general note: I don't think your approach to multi-processing is the best one. Both your processes have to parse the complete file. You can only gain time with the actual processing in the callbacks. That sounds like you would be better off to start with the multiprocessing after the callback: in the osmium handler just copy the relevant data out of the OSMObject and push it into a queue. Then spawn workers that process this queue. Apart from solving the problem of forking libosmium, it also has the advantage that you only have one location cache. Your current solution would create a cache in each worker which can get very expensive very quickly.
General note 2: you really should look into prefiltering the data with osmium tool. This will speed up processing much more than any multiprocessing can do.


Found an issue of save_image if the type of input is torch.unit8

I found that save_image will not give normal image if the type of input is torch.unit8.
Data source
"cifar10.1_v4_data.npy" in https://github.com/modestyachts/CIFAR-10.1/tree/master/datasets
Load data code
from torchvision.utils import save_image
import numpy as np
import torch
data_new = np.load('./data/cifar10.1_v4_data.npy') # the range of data_new is 0-255
data_T = np.transpose(data_new, [0,3,1,2])  # transpose for save_image
data0 = torch.from_numpy(data_T[0]) # convert to tensor. The type of data0 is torch.unit8
Save image code
save_image(data0, "./images/Cifar10_v4_100.png", nrow=1, normalize=False)
Issues
There are two issues: 1) abnormal saved image and 2) data0 changes after executing save_image commend (very strange!)
Solution & New code
If I convert type of data0 to torch.float, two issues will be addressed. Thus, the new code is
save_image(data0.to(torch.float)/255, "./images/Cifar10_v4_100.png", nrow=1, normalize=True)
I think it is a bug inside the code of save_image. We probably need to fix it.

abnormal saved image

Right now torchvision supports only torch.FloatTensor within the interval [0, 1]. You can use torchvision.transforms.functional.to_tensor()  to convert your numpy image into a tensor that represents an image. This automatically handles the conversion form uint8 to float and scales the values accordingly.

data0 changes after executing save_image commend (very strange!)

This is a known bug and is already fixed on the master branch (#1244).


blackspigot account remove

Backstory
I am trying to remove that, but this impossible
Data

name - blackspigot
url - blackspigot.com
difficulty - impossible
notes:
https://blackspigot.com/threads/delete-account.9291/
We do not offer account removals.
THREAD LOCKED!

XDhunterXD, May 21, 2020
domains: blackspigot.com

Closed by #823


Trying to get property of non-object -> Email Assign Notification

Logged in as a normal user (non-admin, non-agent) and submit ticket and I got this error.
2/2
ErrorException in 52418f3aa5e34f54df0cd1049af96859 line 7:
Trying to get property of non-object (View: C:\file\vendor\kordy\ticketit\src\Views\emails\assigned_notification.blade.php)
from below Line 7 is:  'status'    =>  $ticket->status->name,

  $notification_owner->name,
    'subject'   =>  $ticket->subject,
    'status'    =>  $ticket->status->name,
    'category'  =>  $ticket->category->name
]); ?>
subject, $ticket->id); ?>

Yeah, same problem I ran into. I have never seen the unserialize method used so I'm not sure about a fix. It's basically saying that it ran into some bad data while trying to unserialize. I can't figure out why or proper steps to debug since the Model objects are so large

It was working fine 1 hour ago, so I don't think its related to the authenticated users as its now happening to all admin,agent and user,
The only things I have changed in the last hour is: delete all my Statuses and Removed all Agents via admin UI, I then re-added some Statuses and Agents and the auto incrementing MySQL IDs are higher - Statuses now start a #7 instead of the original #1 - I recalled seeing something in the code about #1 being the default assigned to new tickets - might be something worth looking at.

Can you try removing {!! Form::hidden('agent_id', 'auto') !!} from the ticket create form and tell me if it fixes it? That's the only change to the repo

Confirmed: reset first SQL row for Statuses to ID 1 and it worked, the email sent fine.

Did you update
'default_status_id' => 1, in config?

If an improper value is set there, it may be the cause of all serialize problems. Please confirm! :)

My     'default_status_id' => 1,
but my database rows started at 7 because I deleted all my statuses earlier tonight

If you change back to 7 and change config to 7, does it also still work?

I just tried, changed both config and DB to 7 and the email worked.
than just to check again I changed config to random number  of 4 left DB at 7 and it crashed while trying to submit new ticket

Perfect. Now we know what causes it! I'll throw that into a common problems area somewhere.

I think we should add some type of restriction on deleting a default
status, default close status, default reopen status ... and I am thinking
to begin working on transferring most of the settings from the file to the
admin panel, may be in v0.3.0 or v0.4.0

This config is growing quiet large for a config, I'd have to agree on the admin panel. Plus if we have an admin config table, we can control the IDs going in and out for default status via relationships.

Added to wiki for now until 0.3 or 0.4:
https://github.com/thekordy/ticketit/wiki/Common-problems

You could do a similar crud for language localizations, just not too sure what impact is would have on performance with all those extra queries - I did found a nice package that will scan files for trans() and automatically import these into the database for editing the text values.
https://github.com/barryvdh/laravel-translation-manager

We can always Cache the database results for 1 month and make the "Save" button in the admin console flush the cache.

This has the added benefit of migrations essentially allowing us to version control our config

Welcome to the team btw Fishmad!

I did found a nice package that will scan files for trans() and automatically import these into the database for editing the text values.
https://github.com/barryvdh/laravel-translation-manager

@fishmad It looks promising, We can give it a testing round and find if it integrates smoothly and if it is easy to use. then we can implement it in the same version with admin panel upgrade

We can always Cache the database results for 1 month and make the "Save" button in the admin console flush the cache.

@FusionDesign for the config settings, I think it won't be that much load as we can use only one query to load all settings, but I am not sure about the translation panel, does it load translations from database for front-end? or it just for translation modifications then it exports changes back to the translation files?


Generate button in coder should not appear when there are no generations available

Having a button that does nothing is somewhat confusing. Therefore,l it should only appear if it does something!
Cheers


⚠ pishem-otzivi has degraded performance

In 644c902, pishem-otzivi (https://пишем-отзывы.рф/) experienced degraded performance:

HTTP code: 200
Response time: 17479 ms

Resolved: pishem-otzivi performance has improved in 729c90d.


Code from example doesn't work

When I try to run this example from the README on fresh yii2 instalation:
(new \vintage\tinify\components\TinifyResize('@webroot/uploads/image.jpg'))
    ->scale()
    ->width(600)
    ->process();

it throws an exception:
Tinify\AccountException
Provide an API key with Tinify\setKey(...)

here is my params config:
return [
    'tinify-api-token' => '*******',
];

@alexkart Before this code from example configure API token like follow
Tinify\setKey(\Yii::$app->params['tinify-api-token']);

(new \vintage\tinify\components\TinifyResize('@webroot/uploads/image.jpg'))
    ->scale()
    ->width(600)
    ->process();

Thanks, you should add this to readme.

Or maybe add setting the key to TinifyResize constructor.

@alexkart This issue will be fixed in next release. Thank you for feedback :)


Small theme for checkbox

A theme for the component to appear smaller would be greatly appreciated.
Use case: when using a checkbox as a secondary option, e.g. below a text field, having a smaller one would fit better and wouldn't confuse the user.

I've just noticed that setting the font-size to a smaller value resizes the checkbox too!


Drives not terminating

Describe the bug
Drives are not completing, they remain open. If I amnually try to terminate the drive, it errors out.

Expected behavior
Drive properly closes once complete.
How to reproduce it (as minimally and precisely as possible):

https://termbin.com/fnxpa
Screenshots
https://dulanic.com/images/FrayedAccomplishedShrew.png
Data
https://nextcloud.dulanic.com/s/eSZEEJnKx6YggFX
https://nextcloud.dulanic.com/s/MawWskH8baoD6Ar
https://nextcloud.dulanic.com/s/somg2NbzEyBtE3f
-->
Environment

1.19.4
docker

I'm going to just close this out. I ended up manually restoring everything and reloading all my data and it seemed to work. I was able to terminate all the missing drives by loading them into my script i used to loop through them.
#!/bin/bash
array="1419 1489 1475 1476 1478 etc...."
for number in $array
do
docker-compose exec teslamate bin/teslamate rpc \
    "TeslaMate.Repo.get!(TeslaMate.Log.Drive, $number) |> TeslaMate.Log.close_drive()"
done
exit 0


Package request: Kommit

Project description
Git gui client for KDE

Metadata

homepage URL: https://apps.kde.org/kommit
source URL: https://invent.kde.org/sdk/kommit
license: GNU GPLv3
platforms: linux

Could you please update your issue and rename "gitclient" to "kommit"? It appears that the devs have renamed it some time ago.
https://invent.kde.org/sdk/kommit

Could you please update your issue and rename "gitklient" to "kommit"? It appears that the devs have renamed it some time ago.

Updated.

I've worked on it a little, got something... Definitely not perfect as there are a lot of segfaults when opening a repo, seems not even usable atm. I've found that they should know about their segfaults as compiler warns them even in their build job. Maybe it's just qt5 related, idk.
Sorry but i don't want to deal with segfaults in nixos as they should be fixed upstream.
Current work for future packagers, do what you want with it:
{ cmake
, extra-cmake-modules
, fetchFromGitLab
, lib
, libgit2
, libsForQt5
, nix-update-script
, openssl
, pcre
, pkg-config
, stdenv
, zlib
}:

stdenv.mkDerivation rec {
  pname = "kommit";
  version = "1.3.1";

  src = fetchFromGitLab {
    domain = "invent.kde.org";
    owner = "sdk";
    repo = "kommit";
    rev = "v${version}";
    hash = "sha256-H/GPgS63fNI1t6gIavTpoBX+yCkCa1sU50eKluHv5Ao=";
  };

  nativeBuildInputs = with libsForQt5; [
    cmake
    kdoctools
    pkg-config
    qt5.wrapQtAppsHook
  ];

  buildInputs = with libsForQt5; [
    dolphin.dev
    extra-cmake-modules
    kconfigwidgets
    kcoreaddons
    kcrash
    kdbusaddons
    ki18n
    kio
    ktexteditor
    ktextwidgets
    libgit2
    openssl
    pcre
    zlib
  ];

  # todo: uncomment this when `nix-update` is version higher than 1.1.0
  #passthru.updateScript = nix-update-script {
  #  extraArgs = [ "--version-regex" "^(v[0-9.]+)$" ];
  #};

  meta = with lib; {
    description = "Graphical Git Client";
    homepage = "https://invent.kde.org/sdk/kommit";
    license = licenses.gpl3Only;
    maintainers = with maintainers; [  ];
    platforms = platforms.linux;
  };
}

I just startet trying to package this, too.
For KDE Packages, kdePackages.mkKDEDerivation { pname = "kommit"; } should suffice, as far as I understand. But this fails for Kommit. The reason is that while pkgs/kde/generated/projects.json contains kommit, it is missing from pkgs/kde/generated/licenses.json.
But I don't know how these files are generated. @K900 you were the last to update these, can you give us more information, please?

I've packaged Kommit, opened pull request #326325 but I don't want to maintain the package because I realized I won't be using it at all. If anyone would like to do that, have a look at the pull request and feel free to continue from there :slightly_smiling_face:


add class: backup-mysqlBackup.php

add methods to backup and restore all or partial mysql database.

renamed class to backup-mysqlBackup.php


Just add a big class  option to headings

Hi @geobde!
Thanks for wanting to contribute to Bootstrap by sending this pull request!
Unfortunately, your pull request seems to have some problems:

Changes must be made to the original Sass or Less source code file(s), not just the compiled CSS file(s).

You'll need to fix these mistakes and revise your pull request before we can proceed further.
Once you've fixed these problems, you can either ask the maintainers to re-open this pull request, or you can create a new pull request.
Thanks!
(Please note that this is a fully automated comment.)


Create qoutegen.py

Make sure to have the requests library installed
pip install requests
You can run the script to fetch and display a random quote along with its author.
Title and Issue number

Quote Generator
#116
Code Stack :
Close #<issue_no>

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] I have updated the Index.html file for my contribution

This repo is only for frontend challenges


breadcrumb does not refresh if parent patch node renamed by concurrent user

to reproduce:

open edit session in browser (B1)
navigate to same url in another browser (B2)
in B1 switch to Program, enter a subgraph node and select a node inside, e.g. Root > Rotation > If .. Else
in B2 switch to Program, select Rotation subgraph and rename to Rotation 1

expected in B1: Root > Rotation > If .. Else changes to Root > Rotation 1 > If .. Else
actual: breadcrumb remains the same.

related: #1900


Add syntax highlighting support for lines

We have got a project that we would  love to use the library for and would like to have some syntax coloring. I also don't mind contributing this.
Having fairly minimal Spectre knowledge, peeking around in the library it looks like the source lines are rendered  here. The actual highlighter API could be exposed in the ReportSettings, I do not see any other public API that's already present for the renderer. Something simple could be exposed for the user:
readonly record struct Token(int Start, int Width, string Type);

interface ISourceTokenizer
{
    public IEnumerable<Token> TokenizeLine(string sourceId, int line, string line);
}

So render text could be driven based off of this (or something similar). Color coding could be done by mapping string Types to colors. For a CLI renderer, even just having a fixed set of token types could be fine, in which case the type could be an enum instead.
What kind of API did you imagine for this? Have you planned to pick this issue up yourself, or should we consider contributing this ourselves? If this is the case, do you have any pointers/suggestions for us?


Documentation clarification / implementation change

Hi, thanks this seems like a very useful plugin for poetry.
The documentation is a little confusing. It says this works for poetry commands (plural). However, the environment variables seem to only be read for the poetry run command. To have the variables available for other commands such as poetry install you have to execute poetry run poetry install. This looks strange but it works.
Do you need to clarify this in the documentation? Or are you planning to update the code so that the variables are read for all poetry commands?

Hi @covuworie, thanks for the feedback.
It should work for any poetry command that inherits from EnvCommand (which the install command does), since they should all be emitting COMMAND events, which this plugin is attaching to.
If you run poetry install -vvv do you see the message Loading environment variables. at some point in the output? That should indicate that the plugin is loading the environment variables, and works from my testing. If that message is getting printed out, then it may have to do with how the poetry install command does the actual installing. I'm assuming you're trying to install a package that requires certain environment variables to exist at install time?


Planetary society

Add this podcast http://www.planetary.org/system/rss/channel.jsp?feedID=328434498
Play an episode
Try to skip ahead

-> This error

@tschellenbach That's not a valid URL. Hmm..

removed the \ in the url, all good now


Low Quality Textures And Clothing

I've been experiencing this bug since last year, it happened when I was using the normal Roblox client. I found out about Bloxstrap and it solved the issue for a while until recently it happened again. I tried everything but it's still the same. Is this a Roblox problem or just my PC?

Yeah, Roblox just does this sometimes. I doubt it's because of Bloxstrap, but if you want to confirm then you can temporarily switch back to using Roblox's bootstrapper. https://github.com/pizzaboxer/bloxstrap/wiki/Switching-between-Roblox-and-Bloxstrap

Thanks, will update once I've tried it.

Nope, it didn't work. They both look the same. Any other solutions?

Nope, not really. It's a Roblox problem, so I can't really do anything about it. Sorry.


PUT plugin modification against route returning 400 Bad Request

Summary
Seems the PUT for plugins which will update or create plugin if does not exist based on id and name against the new route resource fails with 400 bad request. I believe to be following the documentation correctly and passing valid JSON as per documentation:
https://getkong.org/docs/0.13.x/admin-api/#update-or-add-plugin
PUT https://gateway.com/admin-api/plugins/
{
"id": "42617de5-afa1-46cf-86c3-ee3655844fde",
"name": "rate-limiting",
"route_id": "64d53269-8fb8-4255-933c-4a4e12adaf1f",
"service_id": null,
"consumer_id": null,
"config": {
"fault_tolerant": true,
"hide_client_headers": false,
"limit_by": "consumer",
"policy": "local",
"redis_database": null,
"redis_port": null,
"redis_timeout": null,
"second": 350,
"minute": null,
"hour": null,
"day": null
}
}
Returns a Status Code: 400 Bad Request
In this example my route already exists as well as the plugin so I intended this PUT to modify an existing rate limiting entry from 500 requests per second to 350 as seen in the JSON.
Steps To Reproduce

Create a rate limiting entry with the POST based endpoint on your route (works)
Attempt to edit that with a PUT like I posted above, and watch the mayhem ensue

Additional Details & Logs

Kong version 0.13.0

Actually, hang on I may be onto something here... Response body does have a -
{"created_at":"created_at is required"}
I disagree that a PUT should require a created_at and can't be optional though....

Indeed this was the case, with the created_at populated with a valid value it seems to work:
{
  "id": "42617de5-afa1-46cf-86c3-ee3655844fde",
  "name": "rate-limiting",
  "route_id": "64d53269-8fb8-4255-933c-4a4e12adaf1f",
  "service_id": null,
  "consumer_id": null,
  "created_at":<PHONE_NUMBER>601,
  "config": {
    "fault_tolerant": true,
    "hide_client_headers": false,
    "limit_by": "consumer",
    "policy": "local",
    "redis_database": null,
    "redis_port": null,
    "redis_timeout": null,
    "second": 350,
    "minute": null,
    "hour": null,
    "day": null
  }
}

I will leave this open for the Kong crew to decide if they believe this to be something to fix. I personally don't see why it would be a required field.

@jeremyjpj0916 thank you for all the reports you have sent to us. We try to take all this in account when we move rest of the entities to our new model (in which services and routes are now). Also new model needs to have a proper implementation of PUT that it currently doesn't have.


Support strict_variables Liquid option

Draft status due to:

problems with taxa page generation in the starter site; needs someone to determine if problems are in taxa yaml or in theme assumptions made in _layouts/taxon.xml
feed.xml wasn't touched due to #63

Obsoleted by the work in the strict-variables branch


Can it show Download count?

I tried inserting this:
https://github.com/eonist/Element/releases/0.0.0-alpha.3/
But no download count.

Github only provides download counts of any artifacts that you upload in addition to the source code zip and tar.gz in the release. In your case I don't see any additional uploads, hence no counts.


Enable/Disable Test-Resource server during run task

Feature description
I would like to have the ability to enable/disable test-resource server when I do Gradle "run" task.
Could be added a property like "test-resources.enabled=false" in application.properties for example.
Is this possible? Thanks

Can you clarify your use case?

Yes, sure.
When I run my applications in development mode (aka "run" Gradle task), I use another Gradle plugin that spin up containers using docker-compose. It's more simple and practical IMHO and I don't need to replace placeholders because I use a specific environment to connect the these containers.  So I think that it is better to give the developers the opportunity to disable it through configuration.

mmm, if you are using something else, why are you applying the test resources plugin? You can simply remove it. Or, you can disable test resources in the build script:
micronaut {
    runtime("netty")
    ...
    testResources {
        enabled.set(false)

    }
}

You can even enable by system property, for example:
micronaut {
    runtime("netty")
    ...
    testResources {
         enabled.set(providers.systemProperty("testResources").map(Boolean::parseBoolean).orElse(false)
    }
}

I use it for testing only.
If I add your code block, Gradle fails. I will share a sample repo in the next few days to replicate the problem.

This is similar to my use case!
https://github.com/francesco-monorchio/micronaut-bugreport/tree/test-resources

I also would love to be able to easily support "just for tests" mode. My usecase is that I often need not so trivial data (~GBs) in my database to reproduce some performance issues that won't reproduce on a small database, so it is much more convenient for me to run my own long term database outside of test containers for development.
I would be glad if I can easily configure micronaut test resources to run just for tests, e.g.

java -jar ... => no test resources
./gradlew run => no test resources
./gradlew test => test resources

Can't you simply use an environment for this? e.g:
MICRONAUT_ENVIRONMENTS=dev ./gradlew run

Then have an application-dev.properties with your DB coordinates.

I might be missing something :)
While this is working workaround (thanks for that), it feels against 12-factor application principles (https://12factor.net/config) and inversion of control because when I have application-dev.properties instead of db connection in just application.properties would I not need also  application-prod.properties?

You definitely need application-prod.properties. The application.properties file contains the common properties, for development and production. It doesn't contain the DB connection properties, because they differ between test and production. When you use test resources, it creates them for you (and spawn containers, etc). When you're in production, then you need to provide them in some way, either via a -prod configuration file, or via environment variables. So your use case is similar to a preprod environment where the DB parameters are already known. Test resources are there to "fill in the blanks", so if you don't want it to kick in, you need to fill the blanks.

Oh, so if I don't want to use multiple properties files, I should be able to drive whether test resources are created via environmental variables? This sounds like a way to still be 12-factor (no environmental config in app) and use test containers. Thanks for the hint, I will explore it if it's "the way" for my situation :)

I'll close this issue as won't fix for now. If there's a use case which cannot be covered by environments, feel free to reopen.

Several use cases that we couldn't walk around with environment properties:

We split our tests into unit tests by running test, and integration tests by running integrationTest. We would only want to run integration tests with test resources; while it's impossible for us to not mention a fake DB/Kafka related properties in unit test's application-test.yml;
We have our own custom test resources with running internal containers and containers that are not available in the repo. Not all of them can have getRequiredProperties. For example of here [localstack] doesn't have it configured either.

It will be helpful if we can provide a config to mark several specific tasks to depend on testResource in gradle plugin. For example like:
micronaut {
    testResources {
        additionalModules.add(LOCALSTACK_SQS)
        dependentTestTasks.add("test")
        dependentExecTasks.add("run")
    }
}

FYI Kotlin version and we are using the jvm test suite plugin
micronaut {
    testResources {
        sharedServer.set(true)
        enabled.set(System.getProperty(MICRONAUT_TEST_RESOURCES_ENABLED)?.toBoolean() ?: true)
    }
}

 tasks {
     named<Test>("contractTest") {
        // Disable Pact Telemetry
        //See: https://docs.pact.io/telemetry#opt-out
        systemProperty("pact_do_not_track", "true")
        // We do not want contract test to start up test containers
        systemProperty(MICRONAUT_TEST_RESOURCES_ENABLED, "false")
    }
 }


[Bug] Custom Svelte event does not show up in actions tab?

Describe the bug
I'm pretty new to Storybook, so I don't know if this is the right place, but would really like my Svelte custom events to be shown here:

Steps to reproduce the behavior

Clone my repo here
Start storybook with npm run storybook
Go to button story and click on the button.

Expected behavior
Both click event and custom svelte event.
Screenshots and/or logs
It only shows the click event.
Button.stories.svelte
<script context="module">
	import Button from "./Button.svelte";
    import { withActions } from '@storybook/addon-actions/decorator';

    export const meta = {
        title: 'Buttons',
        tags: ['autodocs'],

        component: Button,
        args: {
            size: 'medium',
            label: 'test-story'
        },
        parameters: {
        actions: {
            handles: ['custom', 'click'],
        },
  },
  decorators: [withActions],
    };
</script>

<script lang="ts">
    import { Story } from '@storybook/addon-svelte-csf';

</script>

<Story name="Primary" let:args>
    <Button {...args} on:click={args.onClick} />

</Story>

Environment

Storybook Environment Info:

  System:
    OS: macOS 14.3
    CPU: (10) arm64 Apple M1 Max
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.17.1 - ~/.nvm/versions/node/v18.17.1/bin/node
    npm: 10.2.5 - ~/.nvm/versions/node/v18.17.1/bin/npm <----- active
    pnpm: 8.12.0 - ~/.nvm/versions/node/v18.17.1/bin/pnpm
  Browsers:
    Chrome: 121.0.6167.85
    Safari: 17.3
  npmPackages:
    @storybook/addon-actions: ^7.6.12 => 7.6.12
    @storybook/addon-essentials: ^7.6.12 => 7.6.12
    @storybook/addon-interactions: ^7.6.12 => 7.6.12
    @storybook/addon-links: ^7.6.12 => 7.6.12
    @storybook/addon-svelte-csf: ^4.1.1 => 4.1.1
    @storybook/blocks: ^7.6.12 => 7.6.12
    @storybook/svelte: ^7.6.12 => 7.6.12
    @storybook/sveltekit: ^7.6.12 => 7.6.12
    @storybook/test: ^7.6.12 => 7.6.12
    eslint-plugin-storybook: ^0.6.15 => 0.6.15
    storybook: ^7.6.12 => 7.6.12

Additional context
Check repo for details here:
https://github.com/21RISK/svelte-storybook-events

Replace
actions: {
  handles: [ 'click'],
},

with:
argTypes: {
  click: { action: "click" },
},

Did you try what is recommended in the documentation ?

Don't register the decorator
Don't use args
Just add on:click in the Template

See https://github.com/storybookjs/addon-svelte-csf/blob/main/README.md?plain=1#L63

Hi @j3rem1e
I might be missing somehow very obvious, but it does not seem to work for me 😓
(with this code I don't get anything in the Actions tab). Created a branch here, but the story is here:
<script context="module">
	import Button from "./Button.svelte";

    export const meta = {
        title: 'Buttons',
        tags: ['autodocs'],

        component: Button,
        args: {
            size: 'medium',
            label: 'test-story'
        },

    };
    function handleClick() {
        console.log('click called')
    }
</script>

<script lang="ts">
    import { Story } from '@storybook/addon-svelte-csf';

</script>

<Story name="Primary" let:args>
    <Button {...args} on:click on:click={handleClick} />

</Story>

Your Button component doesn't dispatch a 'click' event but a 'testEvent'.

Your Button component doesn't dispatch a 'click' event but a 'testEvent'.

It does, check it here.
But again, the whole motivation for this question is not click events but how to make custom events show up in the actions tab 🤔

Sorry but it doesn't 😅
on:click is not forwarded. You use it with a handler which dispatch a testEvent. The parent component doesn't receive the click event.

But again, the whole motivation for this question is not click events but how to make custom events show up in the actions tab 🤔

Of course. But you can't subscribe to events not dispatched by your component.

Thanks so much for helping out @j3rem1e 👍 I updated the code to this now <Button {...args} on:testEvent/> and I actually see the custom event 🔥

How would you recommend that I detect if the button was pressed in an interaction test now? I tried with this implementation, but it times out 🤔
        play: async ({ args, canvasElement }) => {
            const canvas = within(canvasElement);
            const buttons = canvas.getAllByRole('button');
            const button = buttons[0];
            const eventPromise = new Promise((resolve) => {
                button.addEventListener('testEvent', resolve, { once: true });
            });
            await Promise.all([
                userEvent.click(button),
                waitFor(() => eventPromise)
            ])

            // await waitFor(() => expect(args.on).toHaveBeenCalled());
        },

Updated code here.


[Bug] [Doc] e2e-test link 404 not found

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
https://dolphinscheduler.apache.org/en-us/community/join/e2e-guide.html
What you expected to happen
open successfully.
How to reproduce
open https://dolphinscheduler.apache.org/en-us/community/join/e2e-guide.html and click e2e-test
Anything else
No response
Version
dev
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Thanks for your feedback, I will fix it soon.


PPLUS routine CMDERR writes the error to STD output rather than STD error

Reported by Ansley B Manke on 13 Aug 2007 17:52 UTC
This means that when LAS looks to std error, it does not see errors written by
this PPLUS error routine!
Send the std error unit number, set in dat/xprog_state_data.F, to PPLUS at
initialization.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1530

Comment by<EMAIL_ADDRESS>on 15 Aug 2007 22:07 UTC
This is fixed.  There may still be places where PPLUS still writes some kind of
error message like STOP to std output


Temporal Dependencies for LSTM

Description
This PR introduces the temporal dependencies for LSTM layer where we include the previous cell and states to compute LSTM's output. This feature is added to both CPU and CUDA version
Changes Made

Added the previous hidden and cell states to LSTM when seq_len=1 and batch_size=1
Added updating kernels for the cell and hidden states
Modified the LSTM parameter initialization. fan_in = input_size + output_size instead fan_in = input_size
Fixed bugs in resetting states in LSTM layer
Added input_state_update attribute to Python class 'Sequential'

Notes for Reviewer

Run the following example for testing new feature

python -m examples.time_series_forecasting

Shuffling training data leads to better results for some reasons
A single LSTM layers leads to better results
Better parameter initialization could lead to better results but need further investigation
Don't use multiprocessing if your hidden nodes is small i.e. net.set_threads(1)
Currently use traditional tanh and sigmoid for lstm. Mixture tand and sigmoid might improve the results

@van-dai-vuong If you have any comments/questions on any changes, don't hesitate to ask through the  GitHub review system. I plan to close the PR today so we can release the fix

I only have the above comment, other than that, I tested. It runs good. Thanks @lhnguyen102.


Discuss asynchronous log payload serialization

@jmacd can you resolve conflicts on this XOR close it? Thanks.

Actually... this is all going to be up for grabs again in the forthcoming KV logging changes, so I'll just close the PR. Hope that's ok.


How to Implement Two Metrics in Tibber Exporter?

Hello,
I'm encountering an issue with the Tibber Exporter where it seems to be attempting to register duplicate metrics collectors. I have two meters operating with two separate contracts, one for household usage and another for a heat pump. Below is the log output indicating the error:
[Log Output]
2024/01/29 19:23:29 Starting tibber-exporter/3.6.3 (https://github.com/terjesannum/tibber-exporter)
2024/01/29 19:23:29 Websocket url: wss://websocket-api.tibber.com/v1-beta/gql/subscriptions
2024/01/29 19:23:29 Found home: XXXXXXXXX-b5fe-XXXXXXX-9805-XXXXXXXX - Haushalt
2024/01/29 19:23:29 Starting monitoring of home: XXXXXXXXXXX-b5fe-XXXXXX-9805-XXXXXXXXXX - Haushalt
2024/01/29 19:23:29 Current subscription: XXXXXXX-ee09-XXXXXXX-83a2-XXXXXXXX
2024/01/29 19:23:29 Creating home collector for home XXXXXXXXX-b5fe-XXXXXX-9805-XXXXXXXXXXX
2024/01/29 19:23:29 Realtime consumption enabled for XXXXXXXXX-b5fe-XXXXXX-9805-XXXXXXXXXXX: true
2024/01/29 19:23:29 Starting live measurements monitoring of home XXXXXXX-b5fe-XXXXX-9805-XXXXXXX
2024/01/29 19:23:29 Live feed timeout: 1 minute
2024/01/29 19:23:29 Updating prices for XXXXXXX-b5fe-XXXXX-9805-XXXXXXXXXXXX
2024/01/29 19:23:29 Creating measurements subscription for home XXXXX-b5fe-XXXXXXX-9805-XXXXXXXX
2024/01/29 19:23:29 Starting subscription XXXXXXX-d5bf-XXXXX-9256-XXXXXXX for home XXXXXX-b5fe-XXXXX-9805-XXXXXXX
2024/01/29 19:23:29 Found home: XXXXXX-3fe5-XXXXXXX-8fa6-XXXXXXXXXXXX - Wärmepumpe
2024/01/29 19:23:29 Starting monitoring of home: XXXXXX-3fe5-XXXX-8fa6-XXXXXXX - Wärmepumpe
2024/01/29 19:23:29 Current subscription: XXXXXX-a137-XXXXX-a7a0-XXXXXXXXXXX
panic: duplicate metrics collector registration attempted

goroutine 1 [running]:
github.com/prometheus/client_golang/prometheus.(*Registry).MustRegister(0x917fa6?, {0xc0001de160?, 0x1, 0xc0004afe10?})
	/go/pkg/mod/github.com/prometheus/client_golang@v1.17.0/prometheus/registry.go:405 +0x78
github.com/prometheus/client_golang/prometheus.MustRegister(...)
	/go/pkg/mod/github.com/prometheus/client_golang@v1.17.0/prometheus/registry.go:177
main.main()
	/workspace/main.go:143 +0xde2

Is there a way to split or filter the metrics for each meter to avoid this conflict? Any guidance on handling multiple meters in a single Tibber Exporter instance would be greatly appreciated.
Thank you.

Clearly a bug, thanks for reporting. Try version 3.6.4 and see if that fixes the problem.

I've checked and rebuilt with version 3.6.4, and it works now! 👍 Thank you so much for the prompt support and for providing the update!

👍 Thank you


GPflow 2.5.1 install on Mac M1

Bug / performance issue / build issue
When attempting to install gpflow 2.5.1 on Mac os pip install does not recognize the install tensorflow version. I was able to install gpflow by pip installing with no dependacies and then manually installing any other packages required outside of tensorflow. this build now works fine.
I believe the issue is basically syntax. The tensorflow package appears as tensorflow-macos when installed on macs with the M1 chip so possibly the pip dependancies don't recognize it.

To reproduce
pip install gpflow==2.5.1
Collecting gpflow==2.5.1
Using cached gpflow-2.5.1-py3-none-any.whl (381 kB)
Requirement already satisfied: scipy in /Users/timhartnett/miniforge3/envs/RLBO/lib/python3.8/site-packages (from gpflow==2.5.1) (1.8.0)
Requirement already satisfied: setuptools>=41.0.0 in /Users/timhartnett/miniforge3/envs/RLBO/lib/python3.8/site-packages (from gpflow==2.5.1) (62.1.0)
Requirement already satisfied: tensorflow-probability>=0.12.0 in /Users/timhartnett/miniforge3/envs/RLBO/lib/python3.8/site-packages (from gpflow==2.5.1) (0.16.0)
Requirement already satisfied: multipledispatch>=0.6 in /Users/timhartnett/miniforge3/envs/RLBO/lib/python3.8/site-packages (from gpflow==2.5.1) (0.6.0)
ERROR: Could not find a version that satisfies the requirement tensorflow>=2.4.0 (from gpflow) (from versions: none)
ERROR: No matching distribution found for tensorflow>=2.4.0
Minimal, reproducible example

# This is the place for your code that reproduces the bug.
# Please make sure it does not depend on external libraries (beyond GPflow's own requirements) or specific datasets, and the smaller, the better :)
# For help on how to write a good bug report, see https://stackoverflow.com/help/minimal-reproducible-example

Stack trace, or error message
// Paste the full stack trace/error message here

Expected behavior

System information

GPflow version:
GPflow installed from:
TensorFlow version:
Python version
Operating system

Additional context

I think this is a duplicate of #1850 ?
@vdutor were you going to look into this?


fix(deps): update aws-sdk-go-v2 monorepo

This PR contains the following updates:

Package
Type
Update
Change

github.com/aws/aws-sdk-go-v2/config
require
patch
v1.27.19 -> v1.27.20

github.com/aws/aws-sdk-go-v2/credentials
require
patch
v1.17.19 -> v1.17.20

github.com/aws/aws-sdk-go-v2/service/ecr
require
minor
v1.28.6 -> v1.29.0

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

ℹ Artifact update notice
File name: go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

8 additional dependencies were updated

Details:

Package
Change

github.com/aws/aws-sdk-go-v2
v1.28.0 -> v1.29.0

github.com/aws/aws-sdk-go-v2/feature/ec2/imds
v1.16.6 -> v1.16.7

github.com/aws/aws-sdk-go-v2/internal/configsources
v1.3.10 -> v1.3.11

github.com/aws/aws-sdk-go-v2/internal/endpoints/v2
v2.6.10 -> v2.6.11

github.com/aws/aws-sdk-go-v2/service/internal/presigned-url
v1.11.12 -> v1.11.13

github.com/aws/aws-sdk-go-v2/service/sso
v1.20.12 -> v1.21.0

github.com/aws/aws-sdk-go-v2/service/ssooidc
v1.24.6 -> v1.25.0

github.com/aws/aws-sdk-go-v2/service/sts
v1.28.13 -> v1.29.0


Label HAPROXY_0_HTTP_BACKEND_NETWORK_ALLOWED_ACL is not working

Hi I am using the marathon-lb (1.4.1) and it does not pick up the label HAPROXY_0_HTTP_BACKEND_NETWORK_ALLOWED_ACL as documented.
I expected:
acl restricted_network hdr_ip(X-Forwarded-For) {network_allowed}

but got the default:
http-request allow if network_allowed

If I specify HAPROXY_0_HTTP_BACKEND_OPTIONS they are processed.
What am I doing wrong?
marathon.json

I tested your app config, verbatim. It seems to work fine for me. Here's the backend definition:
core@ip-10-0-7-122 ~ $ curl marathon-lb.marathon.mesos:9090/_haproxy_getconfig

...

frontend paas-monitor_10104
  bind *:10104
  mode http
  use_backend paas-monitor_10104

backend paas-monitor_10104
  balance roundrobin
  mode http
  option forwardfor
  http-request set-header X-Forwarded-Port %[dst_port]
  http-request add-header X-Forwarded-Proto https if { ssl_fc }
  acl network_allowed src <IP_ADDRESS>/32
  http-request allow if network_allowed
  http-request deny
  option  httpchk GET /health
  timeout check 2s
  server 10_0_0_249_9029 <IP_ADDRESS>:9029 check inter 10s fall 4

So yeah, IDK what you're doing wrong.

Woops, I take that back. I have a problem with reading comprehension.
Yeah, it's busted. Let me follow-up with a PR.

The responsibility always lies with the writer :-) So, I have updated the wording. Hopefully for the better.

Go ahead and test out the fix-acl-labels image tag with these labels:
{
  "labels": {
    "HAPROXY_GROUP": "external",
    "HAPROXY_0_VHOST": "paas-monitor.<IP_ADDRESS>.xip.io",
    "HAPROXY_0_HTTP_BACKEND_NETWORK_ALLOWED_ACL": "  acl network_allowed hdr_ip(X-Forwarded-For) {network_allowed}\n",
    "HAPROXY_0_BACKEND_NETWORK_ALLOWED_ACL": "<IP_ADDRESS>/32"
  }
}

working nicely with v1.4.2!


[ApiManagement] Camel Case Property "Value" in QuotaCounterContract

ERROR (DefinitionsPropertiesNamesCamelCase/R3016/RPCViolation): Property named: 'Value', for definition: 'QuotaCounterContract' must follow camelCase style. Example: 'Value'.

This is fixed in the 2018-01-01 spec


Create 2021-06-03-opensearch-release-candidate-announcement.markdown

Initial draft of RC announcement post.
Description

Added main framework and callouts.
Added a prod callout (may not be needed)
Added client callout.
Added update links to be inclusive of instructions, reporting path, etc.

Issues Resolved
None specifically.
By submitting this pull request, I confirm that my contribution is made under the terms of the BSD-3-Clause License.

@elfisher Made some updates based on your previous feedback - please feel free to take a look.

We should take the habit of calling file names and titles with the full version number to avoid SEO mishaps.
2021-06-03-opensearch-1.0.0-rc1-announcement

Thanks for the callout @dblock! Will try to align naming to complete versioning in the future!

If you haven't published this yet, rename it now. Or every time someone searches for OpenSearch RC they are at risk of finding the old article before they find the most recent one. AFAIK Google is smart enough to understand historical posts like this but not with a generic title.


TelldusLive - No Z-wave sensor data imported since HASS version 0.58.0

Home Assistant release (hass --version):
0.58.1
Python release (python3 --version):
3.6.0
Component/platform:
tellduslive
Running latest version of tellduslive (tellduslive 0.3.4 / tellduslive-0.10.3.tar.gz)
All dependence met.
Description of problem:
Sensordata missing from Z-wave sensors such as temp, hum, lux. Only "binary sensor" is imported to HASS
Related to: #10757 ????
Expected:
All sensors to be imported
Problem-relevant configuration.yaml entries and steps to reproduce:
N/A

Additional info:
Since 0.58.0 my z-wave sensors temp, hum, lux values are not imported from Tellduslive to HASS.

Noticed this morning that all sensors appeared. Problem might not be caused by HASS but rather the API at Telldus Live.
Please close this issue for now.

And after restart of HASS it was back to "missing sensors" again. Problem persists.

Anything in the logs? Is this a duplicate of https://github.com/home-assistant/home-assistant/issues/10757 ?

Closing as duplicate for now because of missing info in bug report.

PROBLEM SOLVED
Issue is related to dead/ghost/abandoned sensors in tellduslive (start.telldus.com). Contract Telldus support and ask them to remove all dead sensors and this module will kick in and work just perfectly!

Great! (The latest fix mentioned in https://github.com/home-assistant/home-assistant/issues/10922 fixes this issue on the client side)


Code keeps auto-attaching debugger

Issue Type: Bug
The debugger suddenly keeps automatically attaching itself to every single npm script making it impossible to work! The "Debug: Toggle Auto Attach" is set to "Disabled" but it makes no difference. Rebooting doesn't help. Updating doesn't help.
The scripts run extremely slowly, detaching works sometimes but sometimes it just kills the process.
VS Code version: Code 1.58.1 (2aeda6b18e13c4f4f9edf6667158a6b8d408874b, 2021-07-13T06:58:28.115Z)
OS version: Windows_NT x64 10.0.16299
Restricted Mode: No

/gifPlease

Sorry about that, thought it might have been previously reported.
Here's a recording of me first confirming that the "Toggle Auto Attach" feature (which I assume might be the cause) is set to "Disabled" and then running an NPM task.
I've already tried commenting out all configurations in ./vscode/launch.json, updating, restarting, rebooting etc. (all the standard stuff).

Thanks, please try:

Toggling auto attach to any thing other than disabled
Toggle it back to disabled
There should be a ⚠ icon in the top right of the terminal asking you to relaunch it (if you've interacted with it, otherwise it'll relaunch automatically). Do so.
If the issue still happens, run the Developer: Toggle Developer Tools command and check the console for any errors that might have happened


Hide scheduler options in a new scheduler group (like advanced)

Some components are scheduler based such as SQL and then it has a lot of options shown in the panel.
The json metadata should have these options labelled with group = "scheduler"
https://github.com/apache/camel/blob/main/components/camel-sql/src/generated/resources/org/apache/camel/component/sql/sql.json#L64
Then we can move all these options into a new "> Scheduler properties" that sits on top of the "> Advanced properties"

Ah yeah its fine with label includes scheduler.


Support a EventuallyMatcher

The EventuallyMatcher is a matcher that will evaluate another matcher repeatedly until it matches, or fail after some number of attempts.
This is useful when waiting for results from a resource that may take time to return a result.

You might want to have a look at the concepts of checker and probe in https://github.com/jmock-developers/windowlicker, it sounds pretty similar

I would sugesst https://github.com/awaitility/awaitility for this.


Will it be out of the box as  Helix?

Will it integrate plugins like LSP, treesitter, which-key, telescope, multicursor etc out of the box like Helix?

Currently I am the only one working on it. I guess it won't contain all functions like other editors in first few versions.
I will try do make it happen, welcome to contribute.


cannot find the file  "chosen_classes.txt"

I am trying to add additional entities without retraining. The script preprocess_all.py fails:
Ran out of 'useful' classes to select. So using number the 153 chosen classes. Note that this is not expected to happen. It likely indicates that the Wikidata dump or Wikipedia was dump was not downloaded and parsed correctly.  Traceback (most recent call last):   File "/mnt/nlu/users/yasser_hifny/gkqa/refined/ReFinED/src/refined/offline_data_generation/preprocess_all.py", line 364, in <module>     main()   File "/mnt/nlu/users/yasser_hifny/gkqa/refined/ReFinED/src/refined/offline_data_generation/preprocess_all.py", line 244, in main     select_classes(resources_dir=OUTPUT_PATH, is_test=debug)   File "/mnt/nlu/users/yasser_hifny/gkqa/refined/ReFinED/src/refined/offline_data_generation/class_selection.py", line 152, in select_classes     os.rename(os.path.join(resources_dir, 'chosen_classes.txt.part'), FileNotFoundError: [Errno 2] No such file or directory: 'data/chosen_classes.txt.part' -> 'data/chosen_classes.txt'
I am not able to find the file "chosen_classes.txt" in the original data folder:
`additional_data:
datasets:
roberta-base:
config.json merges.txt pytorch_model.bin vocab.json
wikipedia_data:
class_to_idx.json descriptions_tns.pt nltk_sentence_splitter_english.pickle qcode_to_class_tns_6269457-138.np qcode_to_wiki.lmdb
class_to_label.json human_qcodes.json pem.lmdb qcode_to_idx.lmdb subclasses.lmdb
wikipedia_model:
config.json model.pt
wikipedia_model_with_numbers:
config.json model.pt
`
how can I find it and thanks in advance.

On the class_selection.py#L152 file, you can add a condition to check if the chosen_classes.txt.part exists. If not, then you can directly create the chosen_classes.txt and save the classes.
if os.path.exists(os.path.join(resources_dir, 'chosen_classes.txt.part')):
    os.rename(os.path.join(resources_dir, 'chosen_classes.txt.part'),
             os.path.join(resources_dir, 'chosen_classes.txt'))
else:
    with open(os.path.join(resources_dir, 'chosen_classes.txt'), 'w') as f_out:
        f_out.write('\n'.join([x for x in chosen_classes]))
return

Another solution might be to comment out this early exit condition and wait until it finishes all the pages. This would probably take forever so I wouldn't recommend it.


Update recording link

@shemnon : I use the month view, which seems better. That week view is awful. :)

Month view is much better.  Too bad there isn't a way to cram it into the URL as a preference.  But UX is unrelated to this PR.


Vmware provider is not using static ip with coreos guest.

Just cross linking here as I don't know which side is responsible of the issue:
mitchellh/vagrant#5006

Moved to https://github.com/coreos/bugs/issues/1354


fix: corrected minor errors in types

Hello @nysamnang,
Is there any chance to get this merged to master? It will help TS projects to use this library smoothly.
Thanks

@nysamnang, this is a hot-fix for Typescript usage. Can you please merge and release a new version for it?


How can I set connection_limit and pool_timeout in my connectionstring when my database is MySQL?

I tested these connectionstrings but they didn't work.
"mysql://root@localhost:3306/gamma_db&connection_limit=40&pool_timeout=20"
"mysql://root@localhost:3306/gamma_db?connection_limit=40?pool_timeout=20"

URL parameters are usually separated with ? for the first one, then & for all the following:
mysql://root@localhost:3306/gamma_db?connection_limit=40&pool_timeout=20

Looks like this is resolved.


DOC-7633 recos for backup validation

Addresses DOC-7633:

Providing recommendations on how customers should test their backups following an inquiry from a partner from Erlang Systems (supporting Hollywoodbets)

To consider:

@dt gives % confidence levels in each option - I've included our take on the confidence levels per Liv's feedback without mention of % confidence

relates to https://github.com/cockroachdb/docs/pull/17629, cut another branch due to the v23.2 docs cut
https://deploy-preview-17773--cockroachdb-docs.netlify.app/docs/dev/backup-validation#recommendations

@livlobo following up on this for review whenever you get the chance, thanks!

@dt can you provide a review here so we can get this merged?

TFTR all :)


Set-Content -Encoding

I was unable to export the XML of the current rules using PowerShell Core v7.1.3 as it appears that Byte type encoding is not supported.
Error message:
Set-Content: Cannot process argument transformation on parameter 'Encoding'. 'Byte' is not a supported encoding name. For information on defining a custom encoding, see the documentation for the Encoding.RegisterProvider method. (Parameter 'name')
I was able to export the XML using the built-in PowerShell of Win10 v5.1.19041.868 as documented in this article.
https://docs.microsoft.com/en-us/microsoft-365/compliance/customize-a-built-in-sensitive-information-type?view=o365-worldwide#export-the-xml-file-of-the-current-rules
Since encoding type of Byte is not in PowerShell Core v7.1, what would be the correct type to export the compliance rules XML?
https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.management/set-content?view=powershell-7.1

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

ID: f1391cf1-588f-a4a9-8692-8c07cf1e459a
Version Independent ID: d57337a5-d0f3-de26-2259-6b7fa7613d05
Content: Customize a built-in sensitive information type - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/customize-a-built-in-sensitive-information-type.md
Service: o365-seccomp
GitHub Login: @chrfox
Microsoft Alias: chrfox

Hi @GitRandyAZ
Thanks for your comment.  You should contact Microsoft Support to help you with your question.


jh: lowercase where needed

This should solve the remaining issues with pod names.
fixes #52

insha'Allah ;)


mass ssl sni hosting - possibly feature request

Hello!
Is it possible to dynamically load certificates/keys for ssl/tls? I would like to avoid reloading configuration on every change. Frontend nginx using custom dns servers maps vhost to backend server it works fine for http, but I would like to use it also for https.
something like ssl_certificate /etc/nginx/ssl/$http_host.pem

This seems like a valid feature request


Can't add from_cookies config to the v3.JwtProvider, v1.19.1

I have config like the following for envoy v1.19.1.
                      providers:
                        provider1:
                          "@type": type.googleapis.com/envoy.extensions.filters.http.jwt_authn.v3.JwtProvider
                          issuer: "my-issuer"
                          local_jwks:
                            filename: /foo/bar
                          forward: true
                          forward_payload_header: "forward-header-name"
                          from_cookies:
                            - test

And when I run  - envoy fails to starts with the message: Protobuf message (type envoy.config.bootstrap.v3.Bootstrap reason INVALID_ARGUMENT:(providers[0].value) from_cookies: Cannot find field.) has unknown fields
I don't understand why I can't configure from_cookies as writer here https://www.envoyproxy.io/docs/envoy/latest/api-v3/extensions/filters/http/jwt_authn/v3/config.proto#envoy-v3-api-msg-extensions-filters-http-jwt-authn-v3-jwtprovider
But from_headers and from_params configs work just fine for me.

@qiwzhang

UPD: I fixed this issue just by upgrading the Envoy from 1.19 to 1.21.
It looks like for some reason Envoy 1.19 has v2alpha.JwtProvider instead of v3.JwtProvider.

UPD: I think my question can be resolved now.
I found this stuff in the release note of Envoy 1.20 jwt_authn: added support for extracting JWTs from request cookies using from_cookies.


Problem with Date conversion and Uploading to Wikidata

Describe the bug
While transforming a numerical text values in a column to date format the result for some values is weird.
To Reproduce
Steps to reproduce the behavior:

Select Cells>Common Transformers>To date
The errors are for value for '1906', '1903', '1902'.

Current Results
The current result getting is for all values except '1906', '1903' and '1902' is '190x-01-01T00:00:00Z'. But for '1906', '1903' and '1902' is '190x-01-01T00:08:50Z'.
Expected behavior
The required result must in the format of '190x-00-00T00:00:00Z'. But its ok  and have no issues while uploading to Wikidata when the format is 190x-01-01T00:00:00Z'
But when the format is  '190x-01-01T00:08:50Z'. it's error and not uploaded to Wikidata.
While exporting the result to Quickstatements only for '1906' it is showing error syntax as '+1906-01-01T00:08:50Z/9' instead of '+1906-01-01T00:00:500Z/9'. Because of the value '...08:50Z' instead of ...00:00Z' its not getting uploaded and ends with error as here.https://tools.wmflabs.org/quickstatements/#/batch/26344
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser Version: Brave Chrome 19
jre1.8.0_241

OpenRefine (please complete the following information):

Version OpenRefine 3.3

Additional context

If you upload dates to Wikidata, and have them stored as strings initially (such as 1903), then you should not convert them to dates in OpenRefine first.
This is because OpenRefine's dates do not have any notion of precision: they are timestamps with year, month, day, hour, minute, second. This is too precise for your data.
You should simply use the years (as 1903, stored as strings) in the Wikidata schema. This will upload the dates with the correct precision (year). See the documentation here:
https://www.wikidata.org/wiki/Wikidata:Tools/OpenRefine/Editing/Schema_alignment#Dates
Translating a year to a date is an ill-defined problem - you run into all sorts of complications due to timezones which are not relevant for you.
If you want to transform 1902 to a date, you could first concatenate -01-01T00:00:00Z, and then use the toDate function.
That being said the results you get are funny, so leaving this issue open if someone wants to investigate why toDate does that and whether there is something to fix there.

In first case i have used the string only. While using the sting value I am getting the same error for value '1906'

I could not reproduce this with OpenRefine 3.3  either as Text (black) or Number (green) for original year and using Edit cells -> Common transforms -> To date
Exported test project: date_test.tar.gz

@thadguidry Thanks for that.
I have forgot to convert the text value to number ( value.toNumber ) and then to ( value.toDate ). The error was generated when the column value is just text.

dates.checked.gz
Can you check file.
I have created a test Schema for uploading to Wikidata. When the value 1906 is stored as string its given error. Try checking by Extensions:WIkidata>Export to QS

Ok, then this must be a bug in the date parsing in the Wikidata extension: parsing 1906 to a Wikibase date seems to depend on the timezone.

@jinoytommanjaly this is really curious, I changed my timezone to be not GMT and I cannot reproduce the bug even with the project dump that you have provided.
Could you let us know what your timezone is exactly?

Its GMT+5:30 ie IST


Port Arbiter from ZapJS to Python

Summary
Port Arbiter from ZapJS to Python
Reference
https://github.com/zapproject/zap-monorepo/blob/master/packages/Arbiter/src/index.ts
Definition of Done
All the functions are able to called and return the correct values.

Charles is working on this task.

I just made a pull request adding in parts of Arbiter, relevant types that were unported in types.py, and added the class for test cases for arbiter.


status: Nodes.Conitions only contains the conition of the node present in the cluster. #283

What type of PR is this?
/bug
Which issue(s) this PR fixes?
Fixes #283
What this PR does?
Summary:
Let Nodes.Conitions only contains the conition of the node present in the cluster.
Special notes for your reviewer?

@qianfen2021

LGTM


Include grapher db data in version tracker

Fixes https://github.com/owid/etl/issues/1821
Summary

Move VersionTracker from etl.helpers to its own module etl.version_tracker, and adapt tests and other uses.
Create a function in etl.db to fetch some data from DB about datasets and their charts.
Fix some minor issues with VersionTracker.
Let VersionTracker use data from DB about charts. Now steps_df contains useful information about each step, like the number of charts that, in one way or another, are affected by the current step.
One can run version_tracker --connect_to_db to get a list of errors if there is any public dataset with charts for which there is no active ETL step. Thanks to this, I found some inconsistencies (a missing FAOSTAT step, and two missing fasttrack steps).
One can also add the --warn_on_archivable flag. However, this prints a long list of archivable steps. And some of them are not actually archivable, since they are used by explorers. So, for now, we can ignore it and run it only optionally.

To be reviewed

[x] etl/db.py
[x] pyproject.toml   <- Here, should version_tracker by default connect to DB? It would help us detect missing steps, but maybe it causes issues when run locally. We could also let it try to connect to DB, and on failure run it without connecting.
[x] etl/version_tracker.py <- Here, the only methods to review are get_all_archivable_steps, _add_info_from_db, _create_steps_df, check_that_db_datasets_with_charts_are_not_archived, and check_that_db_datasets_with_charts_have_active_etl_steps.

There is no need to review etl/helpers.py,  tests/test_helpers.py, tests/test_version_tracker.py, and etl/scripts/faostat/archive_old_datasets.py, as they have been affected just by the refactoring.

Here, should version_tracker by default connect to DB? It would help us detect missing steps, but maybe it causes issues when run locally.

I'd be inclined to turn it on by default first and only turn it off if it causes issues.

We could also let it try to connect to DB, and on failure run it without connecting.

I'd let it explicitly fail with a good error message instead of falling back.

Here, should version_tracker by default connect to DB? It would help us detect missing steps, but maybe it causes issues when run locally.

I'd be inclined to turn it on by default first and only turn it off if it causes issues.

We could also let it try to connect to DB, and on failure run it without connecting.

I'd let it explicitly fail with a good error message instead of falling back.

Thanks for the suggestion. In my last commit I've changed things a bit: By default, it tries to connect to DB. If it fails, it raises a warning and internally sets self.connect_to_db=False, therefore skipping the checks that need data from DB. Please let me know if you have a better idea, thanks.

Not major, but I agree on Mojmir's point.
Maybe it is a bit clearer if the script just fails when it can't connect to the DB. I can see how we may let the warning pass and not see it. Then we would not consider Grapher datasets without us knowing.


Services: Add timestamps to service logs

Changelog Description
Service logs now print timestamp.
Testing notes:

Run services.
Their logs should also print timestamp.

Log is still without timestamps


[Candidate 1v1 Rainforest Reservoir ZH v1] Lower half of really damaged dam is missing

Observed behavior
Lower half of really damaged dam is missing.
Expected behavior
Really damaged dam looks damaged, but not unrealistic.
Screenshots

Confirmed with map maker this cannot be fixed. It's a model issue unless you hide it with terrain which we cannot do on this map due to the design. Suggest in new patch that someone makes a new model for the damaged version of dam.


svelte/transition does not work on multi nested blocks

Describe the bug
the transitions on elements do not work for doubly nested "svelte blocks".
Setup:
{#await promise}
   <!-- Redacted  -->
{:then value}
  {#if value}
    <button transition:slide={{ duration: 1000 }} />
  {/if}
{/await}

Expected:
Button animates with the slide effect for 1000s duration
Current:
Button does not animate at all.
See example repo.
Reproduction
https://github.com/ptrtht/sveltekit-transitions-bug
Logs
There are no logs.

System Info
> npx envinfo --system --binaries --browsers --npmPackages "{svelte,@sveltejs/*,vite}"

  System:
    OS: macOS 13.5.1
    CPU: (8) arm64 Apple M1 Pro
    Memory: 45.92 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 18.19.0 - ~/.nvm/versions/node/v18.19.0/bin/node
    npm: 10.2.3 - ~/.nvm/versions/node/v18.19.0/bin/npm
  Browsers:
    Chrome: 120.0.6099.199
    Safari: 16.6
    Safari Technology Preview: 17.0
  npmPackages:
    @sveltejs/adapter-auto: ^3.0.0 => 3.1.0
    @sveltejs/kit: ^2.0.0 => 2.0.6
    @sveltejs/vite-plugin-svelte: ^3.0.0 => 3.0.1
    svelte: ^4.2.7 => 4.2.8
    vite: ^5.0.3 => 5.0.10

Severity
serious, but I can work around it
Additional Information
No response

I think this is expected, you could use the global modifier here:
<button in:fade|global={{ duration: 2000 }}>

Transitions are local by default (in Svelte 3, they were global by default). Local transitions only play when the block they belong to is created or destroyed, not when parent blocks are created or destroyed.


KIT - operation view

The operation view of the KIT provides resources to deploy and operate centralized & decentralized Catena-X offerings in your IT environment (e.g., on-prem, cloud).
Deliverables: Quick Setup Guide / Installation Instructions
Prerequisites (if they exist)
Authorization information
Deployment Deployment Scripts

Reference of pull request from KITs- https://github.com/eclipse-tractusx/eclipse-tractusx.github.io/pull/684


⚠ Samsung has degraded performance

In 44b4e07, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2733 ms

Resolved: Samsung performance has improved in 8e2b5fc.


add appveyor.yml

I needs ringmesh.org for this PR to work.

Nice :) well should we use devkit with Debug config?


When Mid-Caches are set to "REPORTED", Traffic Monitor can't poll astats

The ip_allow should add the Traffic Monitors to the list of subnets allowed. Maybe Traffic Ops should be added too (server checks).
Current workaround is to add the Traffic Monitor IPs/subnets in the purge_allow_ip list. I didn't see a purge_allow_ip6 parameter (didn't check the code. This might require another enhancement in case the purge IP is v6.

had the wrong issue # in commit message

Confused now - looks like that commit actually fixes the issue, so closing it. Steve - please retest in > 112?

Will test without the purge_allow workaround against 1.1.2 (make sure there is false positive).

I was checking the ToCHRCheck.pl script but came across the same issue. ip_allow settings has all the subnets required.
astats.config has the subnet included
ip_allow.config is missing it
Ends up into a deny since both are required. Seems like astats.config allow_ip/allow_ip6 needs to make it into the Mid Cache ip_allow.config
Please re-open.


Channel ready future test tweaks

I suspect this will have to wait for the other changes happening today before its test run will pass.

Known failures:

Tsan C++
gRPC_interop_pull_requests
gRPC_performance_pull_requests


[Event Request]Application “Tests-VAT”, 134090 "ERM Pmt Disc And VAT Cust/Vend"

[Event Request]Application “Tests-VAT”, 134090 "ERM Pmt Disc And VAT Cust/Vend"
Could you please insert the OnTestInitialize Event at the beginning of Initialize procedure:

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


WSFederation options.Validate() throws if ConfigurationManager is null even when configuration and metadata address are filled in

Describe the bug
WSFederation options.Validate() throws if ConfigurationManager is null even when configuration and metadata address are filled in. Looking at the source it just checks if ConfigurationManager is set or not and throws a very confusing error message if not.
The ConfigurationManager will be created automatically based on the MetadataAddress and Configuration if available which should mean that the options.Validate should not throw in this case.
To Reproduce
Steps to reproduce the behavior:

Using this version of ASP.NET Core '...'
2.2

Run this code
`
builder.AddWsFederation(authMethod.Id.ToString(), authMethod.Name, options =>
{
try
{
options.MetadataAddress = authMethod.IdPUrl;
options.CallbackPath = $"/signin-wsfed/{authMethod.Id}";
options.RemoteSignOutPath = $"/signin-wsfed/{authMethod.Id}";
options.Wtrealm = authMethod.Audience;
options.RequireHttpsMetadata = false;
options.SaveTokens = true;
options.TokenValidationParameters = new TokenValidationParameters
{
ValidAudiences = new string[] { authMethod.Audience, $"spn:{authMethod.Audience}" },
ValidateIssuerSigningKey = !string.IsNullOrEmpty(authMethod.Certificate),
IssuerSigningKey = string.IsNullOrEmpty(authMethod.Certificate) ? null : new X509SecurityKey(KeyVaultService.GetCertificateFromString(authMethod.Certificate))
};
 					options.Validate();
 				}
 				catch (Exception ex)
 				{
 					logger.LogError(ex, $"Saml2 Provider configuration invalid: { authMethod.Name } with Id: { authMethod.Id }.");
 				}

`

With these arguments

Doesn't matter. If the MetadataAddress, Wtrealm and callbackpath are set it shouldn't error on options.Validate()
Expected behavior
It should validate correctly based on what actually is needed per even the error message that it generates.
Additional context
This also causes issues if you're attempting to add a wsfederation endpoint at runtime which is still not documented properly.

The same is true for OpenIdConnectOptions.Validate:
'Provide Authority, MetadataAddress, Configuration, or ConfigurationManager to OpenIdConnectOptions'
The order of operations isn't clear from the error message but with the metadataaddress, clientid, clientsecret and audience on openIdConnect it works the same as metadataaddress and wtrealm works for WsFederation.

Why are you calling Validate directly? There's PostConfiguration that sets ConfigurationManager.
https://github.com/aspnet/AspNetCore/blob/26c487b0c0d55ac73493547126a65a745cf95abf/src/Security/Authentication/WsFederation/src/WsFederationPostConfigureOptions.cs#L70

I'm calling Validate because I need to know that my options are right or wrong at time of creation, not later when it trys to access them and something goes wrong on one of our dozens of WsFed and OpenIdConnect options (because we're multi-tenant)
And if it isn't supposed to be called by end code writters, it should be marked protected internal.

Fair point, but don't call it.

Ok, well then how else do a validate and fail fast on setup?

@HaoK? How about in Startup.Configure you eagerly resolving the options? E.g. Resolve IOptionsMonitor<WsFederationOptions>, call Get(schemeName), and Validate. PostConfigure should run.
That's how the validation works later:
https://github.com/aspnet/AspNetCore/blob/9f724f6ac4a65160c9847d29bc748e07ce06c7d2/src/Security/Authentication/Core/src/AuthenticationHandler.cs#L57-L86

Yep what @Tratcher says should work

Thanks!


Update readme.md for multires/generate.py

Additional help added if pip3 install --user pyshtools fails.

I'm not going to merge broken directions for your specific system. There is no fttw package; it's libfftw3-dev in Debian and Ubuntu (and there's no fttw package anywhere, since the library is FFTW).
You only need those packages if you're trying to build pyshtools from source. PyPI has binary wheels for x86_64 Linux, macOS, and Windows, so should only need to build from source if you're using a different, less-common architecture.
I added a generic note about consulting the pyshtools build-from-source directions in case of an issue in 37453a99b4de5395d7d6970d6baddad36c9f8fb8.


Add tflite support to yolo

Added support for tflite models to the new yolo detector.  This is different from the existing tflite detector in that it has support for YOLOv8 (and possibly others-- I can't use my ultralytics-exported YOLOv8 with the regular tflite detector, but it's possible I could tweak some of the export settings to get it to work.  Regardless this seems to streamline the process in that any tflite models exported from ultralytics should be compatible with yolo).
I have some areas that I think could use some feedback though.  .tflite models don't contain imgsz information the way the default .pt do, so I have to get this size info from an interpreter.

I'm switching based on .endswith('.tflite'), do you know off-hand a better way to do this?  I think it should be fairly reliable, but another option would be to add e.g. a 'subtype' configuration to the detector, but that seems like extra work rather than going by file extension.
I borrowed the whole buildTfliteInterpreter() function from tflite.py.  Could there be a way to share this instead?
Rather than mess with the interpreter at all, we could just require that users provide an explicit imgsz configuration value for tflite models.  It makes it less convenient for users, but reduces code complexity.
I feel like I'm referencing / using hwAccel awkwardly here, but I think it's needed to build the interpreter correctly.

Other than those rough edges, this does test successfully for me using yolov8s on tflite, and volov10 without using the TPU (but also it's pretty darn fast even then, v10 might be pretty good).

Looks pretty good.. Couldn't we just make buildTfliteInterpreter a stand alone function inside the tflite detector and then import that? Switching by extension seems pretty logical to me. Perhaps the hwAccel option could just be an option to that buildTfliteInterpreter function?

Works for me, changes made.  Also made a small change to the workflow file so that branches will get images built.


Lockup trying to edit groups

Go into settings on left side click groups, click any group to edit. It does not go into the group for editing. It seems to lock up Fenrus nothing else clicked responds even clicking home. Have to refresh browser to get and navigation back. It does seem to work however if on the left column dashboards is selected, for me i edit the default dashboard. From there if a group is selected to be edited it works as it should, you can add and/or change apps or links. Upon saving it puts you into the groups on the left column and locks up if another group is selected from there to be edited.

cannot reproduce

fresh install on unRAID, no fenrus data directory existing on creation
create login
login
click settings page
go to "Groups" page
added a group
added a new item ot the group
saved group
went to dashboard page
added new group to default dashboard
saved dashboard
went to "Dashboard" page (ie the main page)
new group with new item there

I think I'm seeing this issue or something similar to it on a fresh install, version <IP_ADDRESS>5.  Do you want me to open a new issue for it?
If I mouse over the group name on the left hand side it gives me this link:
http://192.168.X.X:3000/settings/groups/2d7eb29b-f2be-48a7-9c87-a29f0aa11993
But if I mouse over the little edit icon on the right side it's giving me this link:
http://192.168.X.X:3000/settings/dashboards/2d7eb29b-f2be-48a7-9c87-a29f0aa11993
Clicking the group name on the left works fine but clicking on the edit icon on the right causes the lock up and subsequently the need to refresh the page to get things working again.


Implement AudioFile#read

Although AudioFile.new accept :read mode in 2nd argument, AudioFile#read method is not defined yet. :-(

Implemented at 9bec7e0f9


Error when creating a new page

Describe the Bug

When creating a new page, django throws an error ValueError: 'Page' instance needs to have a primary key value before this relationship can be used. This is not critical, as page creation is still possible. Rather, this is a bug within the get_repr method of the Page model, where self.best_translation is accessed before that relation exists.
Steps to Reproduce

Visit http://localhost:8000/augsburg/pages/de/new/
See error in django console

Expected Behavior

No error
Actual Behavior

Error, but does not prevent page creation
Additional Information

Traceback

@charludo is this a duplicate of https://github.com/digitalfabrik/integreat-cms/issues/2806 or is it something different? :thinking:

@charludo is this a duplicate of #2806 or is it something different? 🤔

Oops, sorry about that - did not see this!


fix(templates): update midweek schedule and assignment slips

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


[IDEA] Create Deco Support

Describe your idea in depth.
Support for the ladders added by the Create addon: Create Deco

Added in version 1.3


stubs use implicit reexports

Describe the bug
mypy enables --no-implicit-reexport for stub files:

Note this is always treated as enabled for stub files.

https://mypy.readthedocs.io/en/stable/command_line.html#cmdoption-mypy-no-implicit-reexport
Many of the sqlalchemy stubs reexport and so mypy seems to not recognize those exports.
Expected behavior
mypy should not fail
To Reproduce
cc.py:
import sqlalchemy

CC = sqlalchemy.sql.expression.ColumnClause

Error
$ mypy cc.py
cc.py:3: error: Module has no attribute "ColumnClause"
Found 1 error in 1 file (checked 1 source file)

Versions.

OS: Ubuntu 20.04
Python: 3.8.10
SQLAlchemy: 1.4.23
mypy: 0.910
SQLAlchemy2-stubs: 0.0.2a18

Additional context
Manually editing .../lib/python3.8/site-packages/sqlalchemy-stubs/sql/expression.pyi and changing the import of ColumnClause to from .elements import ColumnClause as ColumnClause fixes this.
There are other objects that are also implicitly re-exported and result in mypy errors - sqlalchemy.sql.expression.Executable, sqlalchemy.sql.expression.FunctionElement, sqlalchemy.sql.expression.Tuple etc.

doesnt apply to the stubs here, but for SQLAlchemy itself which will be retiring the separate stubs package, there seems to be some debate over if a Python package w/ py.typed needs to do explicit re-exports: https://github.com/python/mypy/issues/8754

The documentation does give examples that yield mypy errors, such as this one with expression.FunctionElement:
https://docs.sqlalchemy.org/en/20/core/compiler.html#utc-timestamp-function

note that the v2 documentation assumes that this package is not being used


Is there a way to mark the drone location?

I am just starting with ScriptCraft. I'm trying to figure out where my drone is. I'd really like to use this like logo.
Is there a way to mark the drone location? Flashing block or something?
I tried adding this bit of code to the drone.js file but sadly I don't get a blink...
....
 that = this;
  setInterval(function(){
    if(this.off)
      that.box(blocks.oak);
    else
      that.box(blocks.air);
    console.log("test");
  }, 2)
  return this;
};

I like this idea. To make this fully usable, the various drone operations will need to accommodate the functionality.
Looking at http://scratch.mit.edu, Scratch supports the pen down and pen up operations, similar to Logo and Kojo. I wonder if this would work in the drone context. Instead of using a strobe, perhaps the user could set a block a type as the pen and when the pen is down the Drone is shown as a (non-strobing) block.

So, I started to break this down into some BDD stories. There are some tricky parts already with these two basic scenarios.
Story: The drone can show its current location
In order to show how the drone is moving
As a player
I want to be able to make the drone's location on and off.
Note: The default is off and that should be its own scenario.
Scenario 1: Make the drone visible
Given a drone d
And the drone is visible or invisible
When d.show() is called
Then the drone can be seen as a black wool block, possibly replacing an existing block at the same exact drone location
TBD: What is returned by an operation requesting the block at the drone's location?
Scenario 2: Make the drone invisible
Given a drone d
And the drone is visible or invisible
When d.hide() is called
Then the block at the drone's location is replaced by the block that was there when a previous d.show() operation was invoked or else an empty block (air).

Hmm, tricky tricky tricky. I know its been a while since this issue was opened, but I'm interested in working on the animal solution. I just can't seem to figure out how to spawn and keep a reference to an animal. I like the approach Computercraft took having a block act as the "drone" the semantics are a bit different though. Maybe that type of api is better in the sense that there is always a block in the world that will move around. http://computercraft.info/wiki/Turtle_(API)
Thoughts?


Fix serialisation of findMatch action

Summary of the Pull Request
Fixes the serialisation of the findMatch action so that the direction is stored.
PR Checklist

[x] Closes #11225
[x] CLA signed. If not, go over here and sign the CLA
[x] Tests added/passed
[ ] Documentation updated. If checked, please file a pull request on our docs repo and link it here: #xxx
[ ] Schema updated.
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Validation Steps Performed
Added a test and tested manually.

For whatever reason the local settings model tests have decided to stop working on my machine with this error:
Summary of Errors Outside of Tests:
    Error: TAEF: [HRESULT: 0x80070005] A failure occurred while preparing to run tests in 'SettingsModel.LocalTests.dll'. (Failed to load "F:\windows-terminal\bin\x64\Release\TestHostApp\SettingsModel.LocalTests.dll" or one of its dependencies.)

The DLL definitely exists at that location...
PS F:\windows-terminal> ls F:\windows-terminal\bin\x64\Release\TestHostApp\SettingsModel.LocalTests.dll

    Directory: F:\windows-terminal\bin\x64\Release\TestHostApp

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
-a---          15/09/2021    08:51        2057216 SettingsModel.LocalTests.dll

I've added a test and am fairly sure it should work, but can't confirm due to the above.

For whatever reason the local settings model tests have decided to stop working on my machine with this error:

Yea, that happens sometimes. These tests are tenuous at best. Sometimes it works to totally clean and rebuild all the Terminal projects.

which admittedly, would take a while, but might fix the test deployment.

but doesn't improve the error message - should another issue be opened for that?

Meh - I'm tempted to say close that one and open a new one, but also I feel like we've got something similar already hiding on the backlog

Tests still error out after a rebuild of all the terminal projects 😢
I've updated the PR description to close #11225.

@msftbot merge this in 5 minutes


'Could not complete migration. Never acquired lock within allowed time: 00:01:00 Either another server did not complete the migration or migration was abruptly interrupted

Hello,
I get the following error when I add a username and password to my MongoDB even though I use the latest version of the Hangfire and MongoDriver packages. Although the hangfire.mongo.migrationLock table does not exist. I run my MongoDB on Docker. I added my username and pass with MongoUrlBuilder. I also tried replacing MigrationStrategy with DropMongoMigrationStrategy, that doesn't work.

it seems it did run some of the migrations.
try to

check whats in the schema collection, to check how far the migrations got
set the log level to TRACE
drop the hangfireDB and try again
what version are you running?

closing due to inactivity


[Documentation]: LangChainEmbedding example code

Documentation Issue Description
For custom embeddings there might be a slight issue in the example code given with LangChain: the given code is
from langchain.embeddings.huggingface import HuggingFaceEmbeddings
from llama_index import LangchainEmbedding, ServiceContext

embed_model = LangchainEmbedding(
  HuggingFaceEmbeddings("sentence-transformers/all-mpnet-base-v2")
)
service_context = ServiceContext.from_defaults(embed_model=embed_model)

this would return an error (like init has one argument, 2 are specified) as it should maybe be, given the current initialization of HuggingFaceEmbeddings:
embed_model = LangchainEmbedding(
  HuggingFaceEmbeddings(model_name="sentence-transformers/all-mpnet-base-v2")
)

Documenation Link
https://gpt-index.readthedocs.io/en/latest/core_modules/model_modules/embeddings/usage_pattern.html#custom-embedding-model

Indeed, thanks. I also encountered this bug.

whoops, will update


I think an additional test to preclude the very bad solution below should be added

Challenge Check for Palindromes has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.116 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
 You know that's not what you should be doing. I guess its time we add hidden test cases.

Closed as duplicate of #7600. Thanks.


fix(GoogleMapsAPIWrapper): #770 circle strokeWeight throws error

When setting strokeWeight property google maps throws an error because the strokePosition option is set with an invalid value

@cesarp can you address this? then I'm happy to merge it. Thx!

Closing this due to inactivity. Feel free to reopen.


Change Request: Guidelines for C99 - implementation.mb

Complexity management

Initialize variable with a predict state is wrong. We should say the opposite of it.
Most of the example use what is knowing as Yoda style, we should invert it.

Are you saying we should never initialize variables? I'm a fan of zeroing everything by default unless you have a good reason not to do so, it makes your code more robust in the face of external change.
Maybe we should change it from MUST to SHOULD (I certainly have no problem with not initializing things in some cases).

Changing away from yoda style is fine by me, since there are compiler warnings to catch the bug it prevents.

actually the section on variable initialization is wrong in any case, since compiler warnings will pretty much never catch use of an uninitialized variable except in very specific cases (static analyzers do sometimes, however).

opened a pull request with proposed new position on variable initialization.

OK. Consider the code:
#include <stdio.h>
#include <stdlib.h>

void other_function(int* i) {
    if(rand() == 0) {
        *i = 4;
    }
}

int thing() {
    int i;
    other_function(&i);
    return i;
}

int main() {
    return thing();
}

this compiles with no warnings on gcc 6.3 and msvc 19.22, it even passes static analysis checks on msvc (haven't tried other static analysis tools).
However i can clearly be uninitialized.
This could happen in the error case if the error is set in an out parameter (for some third party library), or if the function you call happens to return an uninitialized value itself.

I compiled that with
/W4 /analyze on msvc and -Wall -Wextra -pedantic on gcc

The following code also compiles cleanly on gcc and clang, although msvc warns (msvc's backend seems to be insanely good at this)
bool test() { return true; };

int thing()
{
    int i;
    for (int j = 0; j < 150; ++j)
    {
        if (test() && j == 4)
        {
            break;
        } else {
            i = 4;
        }
    }
    return i;
}

int main()
{
    return thing();
}

This is odd that compilers accept it. In this case, we should have a restriction for variables that return function value.
"variables that are intended to hold the function’s return value shall not be initialized, so that the compiler can warn about paths where the return value is not set"

I did some more reserches on out test infra, and OSX compiler (clang with a lot of warnings enable, including non-initialized) throws an error for the thing() function.