setParameter is not implemented but documented in README

I tried out the below and looked the source code it seems that setParameter is not implemented but it's present in the README.md . Is there a way to bind values to prepared statements ?
    string sql = "INSERT INTO testing(data) VALUES (':value');";
    auto stmt = db.prepare(sql);
    stmt.setParameter("value", value);

Empty implementation : https://github.com/huntlabs/database/blob/74ddf46ee42722d2e7967986b0d5fff1f468601c/source/database/statement.d#L42
Thanks

In the latest version 0.0.16, we have implemented this function

@viile Thanks for the fix. I tested the new version with dub and PostgreSQL. It seems that it's dropping the bind values. I removed the assert in the setParameter function since the param_value variable was empty.
import std.conv;
import std.stdio;
import database;

void main()
{
  auto db = new Database("postgresql://root:root@localhost:5432/shouter");

  foreach(number; 1..3) {
    auto value = "Test " ~ to!string(number);
    Statement stmt = db.prepare("SELECT * FROM testing where data = :value");
    stmt.setParameter(":value", value);
    writeln(stmt.sql);
  }

  foreach(row; db.query("SELECT * FROM testing"))
    {
      writeln(row["data"]);
    }
}

Sample output :
➜  db-tut dub run
Performing "debug" build using dmd for x86_64.
database 0.0.16: target for configuration "postgresql" is up to date.
db-tut ~master: building configuration "default"...
Linking...
To force a rebuild of up-to-date targets, run again with --force.
Running ./db-tut
SELECT * FROM testing where data =
SELECT * FROM testing where data =
2017-12-22T10:20:05.387:connection.d:query:35 SELECT * FROM

already fixed

I think there are also some other issues as below. The parameter is not quoted with single quote while being sent to PostgreSQL and hence it generates below SQL
SELECT * FROM testing where data = Test 1
  void setParameter(T = string)(string key, T value)
  {
    param_value[key] = "'" ~ value.to!string ~ "'";
  }

The above change generates the correct SQL but it might be prone to SQL injection.
SELECT * FROM testing where data = 'Test 1'

@viile Added a unit test and formatted the code. It's a workaround and I will try for a proper solution over the weekend looking into how other libraries do this.
Unit test : https://github.com/tirkarthi/database/commit/f9d1a44940748b98be7eddc413786a67a5859e8e#diff-acd2545e2b9c057905921381db98b299R225

mysql,pgsql,sqlite have different open and close quote, I have implemented this  in  dialect (https://github.com/huntlabs/database/blob/master/source/database/driver/dialect.d)


Range control auto add px on using arrow keys

Hello, I am using range control and it was working perfect before update but when i theme and plugin i get the issue of changing number by arrow keys. It add px automaticly to numbers and it also don't validate the max value. If i use slider then it works fine.
'list_divider_weight' => array(
'label' => esc_html__('Weight', 'bpd-booster-pack-divi'),
'type' => 'range',
'default' => '1',
'range_settings' =>array(
'min' => '0',
'max' => '20',
'step' => '1',
),
'option_category' => 'basic_option',
'description' => esc_html__('Divider Weight.', 'bpd-booster-pack-divi'),
'tab_slug' => 'advanced',
'toggle_slug' => 'list_style',
'show_if'         => array(
'list_divider' => 'on',
),
),
Kindly update me regarding the issue.

QA Checklist
Reporter

[ ] Helpful title used

Helpful title format: Scope(s) :: One Sentence To Describe The Issue
Example: VB :: Floating Toolbar :: Jumps To Top Of Page

[ ] Detailed description

Description should usually be 150+ characters.

[ ] Steps to reproduce included
[ ] Support Ticket URL included
[ ] Image and/or gif

Developer

[ ] Steps to reproduce proven solved
[ ] Test included and coverage acceptable and all Tests passing?
[ ] PR description has layman's description of problem and solution and "Areas Affected" listed/explained?
[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Estimated points on issue
[ ] Proper code documentation and conventions?
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Peer Review

[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Steps to reproduce proven solved
[ ] Affected areas are tested and ok
[ ] Proper code documentation and conventions
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Quality Assurance

[ ] All tests passed on CircleCI.
[ ] Steps to reproduce passed, the fix works.
[ ] No errors in console or output
[ ] Affected areas are tested and ok
[ ] The fix does not change a customer's website in any surprising or unintended way

[ ] If Divi's design is adjusted, settings are migrated gracefully and invisibly

Design Review

[ ] UI is to spec
[ ] UX is to spec
[ ] Design is to spec
[ ] No surprising visual changes to a customer's website will be experienced after updating

[ ] Any/all visual changes are migrated gracefully

Dev Review

[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?
[ ] i18n used if applicable
[ ] Implementation acceptable, DRY, Scaleable, future safe
[ ] Best practices used, cleverness and grokage, ES6 use, etc.

There are a few options which lets you further define what the valid range is:
'unitless'        => true,
'validate_unit'   => true,
'fixed_unit'      => 'deg',
'fixed_range'     => true,

Especially the unitless, possibly in combination with validate_unit, could prevent px or any other unit to show up, even if leaving the range or when changing values using arrow keys


extracting Feature points by using superpoint model is not good

hello,thank you for your amazing work.
when i use the model superpoint_v1.pth to extracting Feature points,i found the result is not good,some Feature points in the sky and ground seems not right.as shown in the figure below:

the param is:
'superpoint_aachen': {
'output': 'feats-superpoint-n4096-r1024',
'model': {
'name': 'superpoint',
'nms_radius': 3,
'max_keypoints': 4096,
},
'preprocessing': {
'grayscale': True,
'resize_max':1920,
},
},
but when i use the tensorflow-superpoint,it seems right:

Is it my configuration error or something else？thx

What input size are you using for the Tensorflow implementation? Could you compare with the exact input sizes? 'resize_max':1920, resizes the image to 1920px at most, which is probably too large. You should:

drop this to 1200-1600 pixels if you care more about very accurate keypoints and are ok with some noisy detections in textureless areas
drop this to 720-1024 if you prefer to have fewer, more repeatable but less accurate keypoints

Thank you for your reply.

The param  'resize_max':1920 not used and the first image above(pytorch implementation ) use the input sizes : 540X960(HXW)
The second image above(Tensorflow implementation) use the input sizes:480X640(HXW)

In the Tensorflow implementation,I changed the input sizes to 540X960(HXW), as shown in the figure below and seems good:

There is no significant difference in the results. The Tensorflow model might have been trained with more noise so it has fewer detections in textureless areas. I don't think that this has a large impact on the results.

OK.Thank you very much.


InjectionNext doesn't allow app to build on device

I integrated InjectionNext yesterday after transitioning from InjectionIII.
So far I loved it when building on the simulator.
Unfortunately it doesn't allow the project to build on a real device.
App Structure

App

InjectionNext
Inject
CustomFramework1

Inject
InjectionNext

What I tried

InjectionNext app on/off
Enable Devices on/off
Open Xcode manually

This is the build error.

Please @johnno1962 help me solve it, because this is blocking me from submitting to the App Store.
Thanks.

Here is the error log as well.
Error-InjectionNext.txt

This is Xcode 16.1 right? Can you try turning off the "Undefined behaviour Sanitizer" in your scheme?

I turned it off. Now it works. Thanks John.
The app runs fine on the device.

It's a strange problem. I have a project which by chance replicates what you've seem in Xcode 15 as well when you build for device with "Undefined behaviour Sanitizer" but, when I tried to prepare a replicating project for Apple I couldn't get it to fail and can't see what the difference would be. ¯\_(ツ)_/¯. Any other strange problems with InjectionNext let me know!

I am working on Xcode 16.1. Maybe having frameworks that also have InjectionNext and Inject as dependencies might trigger it?

I tried adding a framework but it doesn't seem to be that. Seems to be a very odd edge case in the build system related to the InjectionLite or DLKit framework which I have a smallish replicating project which doesn't build for simulator or device, so I raised FB15653175.


[FLINK-9572] Extend InternalAppendingState with internal stored state access

What is the purpose of the change
Extend InternalAppendingState with get and update methods for internal stored state.
Implement them in concrete states in backends.
Brief change log

InternalAppendingState has now methods getInternal and updateInternal
Heap and Rocksdb merging states implement the methods

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
It should be covered by existing tests.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (yes)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

cc @StefanRRichter

Thanks for review, @sihuazhou! I added fixing commit to address your comments.

I think when all comments are addressed this is good to merge 👍 . Will merge it tomorrow once we have a green build.

+1 LGTM


unknown option 'winbar '

Describe the bug
I just updated the new version, and then there was an error. After my investigation, it should be caused by this push https://github.com/glepnir/dashboard-nvim/commit/8f5b3996ec99e53af8c78a8a238a571604a95d96
Screenshots

version
NVIM v0.7.2
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by nixbld

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "
/nix/store/8svf52rgypxxjcwq5l1sjcykmibb7a61-neovim-unwrapped-0.7.2/share/nvim
"

oh ..because the winbar is 0.8 option. it will release a few later

oh ..because the winbar is 0.8 option. it will release a few later

em... the neovim in nixos's unstable pkgs is 0.7.2
Maybe I need to live with this error for a while


Finish moving HTML generation to Blaze

At the edge of HTML generation, we're using Blaze, but a lot of the parts are still String.  Instead they should all be Blaze for speeed.

Closed with #111


lisp-modules: use lib.makeSearchPath in shell.nix

The code is more readable this way
@7c6f434c

And of course this is not checkable by ofBorg but whatever.


feat:User auth,Jwt and permissions

Added User Model
Added spring security impl
Basic Auth and authentication over routes
Implemented Jwt token for both admin and user
Left most routes accessible to all (will be fixed before completion and final testing)

Follow up could take time since now work on frontend will start


RenderGif panic during load config

During load the renderung lutex does not work.
Panic is usual with large body sets

mutex lock misplaced


Mass Unit Converter working

Describe the bug

Mass can never be negative but the current calculator works over negative values too.

Possible solution

Checking for negative input and prompting the user obout the incorrect input given

Screenshots

Kindly assign this issue to me @sairish2001

Go ahead
Deadline 7 April,2021 11:59 PM IST


Connect button

Describe the solution you'd like
Can you add a "Disconnect from Ollama" button ? (may be useful in case there is an Ollama update to be done)
Great work, BTW
Thanks

Great idea. I can implement that for the next update.

Request will be implemented into next release.

Thank you!

Disconnect button is now implemented.

Great! 😀


Use ReplaySpecializer to fix DomainBase replays

DomainBase currently has a number of ancillary objects that require a
cascading delete that doesn't get propagated.  Implement beforeSqlDelete() in
DomainContent to delete these child entities.

This change is

Note that this change depends on  #989


Crash on client errors

Our thrift clients accumulate processes when there is an error on the
server, This PR should help alleviate that by closing the clients if
they encounter an unexpected error.
@jparise
@stemmler

@jparise, The underlying problem I'm trying to resolve is a bug in the thrift client that's fixed in 0.10 (whenever that comes out).
This solves it at the wrong layer, though it's helpful if you're making a client for Elixir under 0.9.x. I'm not going to merge this unless people demand it.


Minor requested clarifications in JOSS paper

Hi @lorenaacuna,
as a part of https://github.com/openjournals/joss-reviews/issues/7288, I wanted to drop in here two minor requests for the paper. It is overall very well written and concise, and addresses all the requirements, so both points I bring up here are completely minor.
The first point is that I think it might help if there is some clarification in the part of the paper about the package itself, that describes the parallelization on 8 cores. In the parentheses that follow, it is written:
[...] (four at 3.49 GHz and four at 2.42 GHz processors) [...]

I wanted to confirm that it is indeed supposed to say "four" for both processor frequency cases, and I also wanted to suggest to clarify four of what exactly you mean.
The second point are the references. All references of Acuña et al. include the author's first name, while none of the other references do. Is this on purpose, and what is the reasoning behind it? From what I know about this citation style, the goal is to be consistent throughout, even for own works of the author. I could be wrong though, so if there is a specific reason for this, please elaborate. Otherwise, I would ask you to change this in the paper and use one consistent reference style.

@ivalaginja thanks for the comments. Yes, it's "four": four cores with 3.49 GHz processors and four cores with 2.42 GHz processors. We have rephrased the parentheses text in the .md doc to be more clear as: "(4 cores operating at 3.49 GHz and the other 4 cores operating at 2.42 GHz)".
For the second point, I think the references that show the first author name are because they were taken from arxiv. I replaced them with the NASA bibtex entry and it was solved.


Will not run with the only mod I need it for, "The Graveyard"

1.17 worked just fine for me, but 1.18 refuses to work. Tells me I need to update Gekolib3, which I have. I'm still told it is out of date. It's the version that was literally updated 6 hours ago. 3.0.3 (Fabric)
The exact message states;
"ModResoulutionException: Mod resolution encountered an incompatible mod set!
A potential solution has been determined:
      Install geckolib3, version 3.0.20 or later."

Everything is updated as of 12/22/21. I'm not sure exactly where the problem is so I'm posting this here and on The Graveyard page.
If I've done something wrong in the installation step, please tell me, I'm not super tech saavy. Thank you, merry winter's veil!

You will need resolve this with the mod dev of that mod, not this.


SCons: Fix Ninja compilation with MSVC

Incorporates a handful of fixes brought up in the initial Ninja PR^1, allowing MSVC builds to successfully compile.

This breaks GCC/Clang:
gcc -o thirdparty/mbedtls/library/aes.linuxbsd.editor.x86_64.o -c -std=gnu11 -pipe -O2 -Wall -Wextra -Wwrite-strings -Wno-unused-parameter -Wshadow -Wno-misleading-indentation -Wno-type-limits -Walloc-zero -Wduplicated-branches -Wduplicated-cond -Wstringop-overflow=4 -Wattribute-alias=2 -Werror -w -isystem thirdparty/glad -DTOOLS_ENABLED -DDEBUG_ENABLED -DNDEBUG -DNO_EDITOR_SPLASH -DSOWRAP_ENABLED -DTOUCH_ENABLED -DFONTCONFIG_ENABLED -DALSA_ENABLED -DALSAMIDI_ENABLED -DPULSEAUDIO_ENABLED -D_REENTRANT -DDBUS_ENABLED -DSPEECHD_ENABLED -DXKB_ENABLED -DJOYDEV_ENABLED -DUDEV_ENABLED -DLINUXBSD_ENABLED -DUNIX_ENABLED -D_FILE_OFFSET_BITS=64 -DX11_ENABLED -DLIBDECOR_ENABLED -DWAYLAND_ENABLED -DVULKAN_ENABLED -DRD_ENABLED -DGLES3_ENABLED -DCRASH_HANDLER_ENABLED -DMINIZIP_ENABLED -DBROTLI_ENABLED -DTHREADS_ENABLED -DCLIPPER2_ENABLED -DZSTD_STATIC_LINKING_ONLY -DUSE_VOLK -DVK_USE_PLATFORM_XLIB_KHR -DVK_USE_PLATFORM_WAYLAND_KHR -DGLAD_ENABLED -DEGL_ENABLED -DGODOT_MODULE -DMBEDTLS_CONFIG_FILE=<thirdparty/mbedtls/include/godot_module_mbedtls_config.h> -Ithirdparty/mbedtls/include -Ithirdparty/libpng -Ithirdparty/volk -Ithirdparty/vulkan -Ithirdparty/vulkan/include -Ithirdparty/zstd -Ithirdparty/zlib -Ithirdparty/clipper2/include -Ithirdparty/brotli/include -Iplatform/linuxbsd -Ithirdparty/linuxbsd_headers/wayland -Ithirdparty/linuxbsd_headers -Iplatform/linuxbsd -I. thirdparty/mbedtls/library/aes.c
sh: 1: cannot create -Ithirdparty/mbedtls/include: Directory nonexistent

The angle brackets < and > are interpreted by the shell to read/write from streams.

Thanks!


Migrations are not up to date

I've installed version 0.12.2, and ran makemigrations after I had created a custom content type for the FeinCMS page model. Django saw changes and created a migration (see contents below), and worse, added the new migrations for forms builder as a dependency of my own app's migration. Because of this, I got a "migration does not exist" error running the migrations on production.
The following migration is created:
operations = [
    migrations.AlterField(
        model_name='form',
        name='email_from',
        field=models.EmailField(verbose_name='From address', help_text='The address the email will be sent from', max_length=254, blank=True),
    ),
    migrations.AlterField(
        model_name='form',
        name='sites',
        field=models.ManyToManyField(default=[1], editable=False, to='sites.Site', related_name='forms_form_forms'),
    ),
]

Duplicate #167

Well, it's partially a duplicate. Using the version from master fixed the issue with the sites field, but the issue with the email_from field's max_length change is still there. But that is an issue for Django 1.8, and should be in a "Django 1.8 support" ticket I guess.

Please note that this is still an issue, which I unfortunately fell for again on anther project. Django made a new migration for forms, and that again was referenced in one of my project apps. This of course failed when deploying.

You're right, the email field automatically changed length in Django 1.8 so this wasn't resolved.
It's fixed in a18862031b61cc8ab4cae74b2bace47ec6c6bd3b
Thanks


Use Native::Resouces instead of LibraryMake

I recalled that I had had the same issue Type check failed in binding %RESOURCES and had gave up using Native::Resources :)
In fact, https://github.com/skaji/perl6-librarymake-example does not use Native::Resources.
So you have 2 choices:

raise an issue to Native::Resources, or even pull request
give up using Native::Resources and use sub library:diff --git lib/Algorithm/KdTree.pm6 lib/Algorithm/KdTree.pm6
index aa23162..efa9bf8 100644
--- lib/Algorithm/KdTree.pm6
+++ lib/Algorithm/KdTree.pm6
@@ -2,16 +2,21 @@ use v6;
 unit class Algorithm::KdTree is export is repr('CStruct');

 use NativeCall;
-use Native::Resources;
 use NativeHelpers::Array;
+use LibraryMake;
 use Algorithm::KdTree::Response;

-my sub kd_insert(Algorithm::KdTree, CArray[num64], Pointer) returns int32 is native(resource-lib('kdtree', :%?RESOURCES)) { * }
-my sub kd_nearest(Algorithm::KdTree, CArray[num64]) returns Algorithm::KdTree::Response is native(resource-lib('kdtree', :%?RESOURCES)) { * }
-my sub kd_nearest_range(Algorithm::KdTree, CArray[num64], num64) returns Algorithm::KdTree::Response is native(resource-lib('kdtree', :%?RESOURCES)) { * }
-my sub kd_create(int32) returns Algorithm::KdTree is native(resource-lib('kdtree', :%?RESOURCES)) { * }
-my sub kd_free(Algorithm::KdTree) is native(resource-lib('kdtree', :%?RESOURCES)) { * }
-my sub kd_clear(Algorithm::KdTree) is native(resource-lib('kdtree', :%?RESOURCES)) { * }
+my sub library {
+    my $so = get-vars('')<SO>;
+    return ~%?RESOURCES{"lib/libkdtree$so"};
+}
+
+my sub kd_insert(Algorithm::KdTree, CArray[num64], Pointer) returns int32 is native(&library) { * }
+my sub kd_nearest(Algorithm::KdTree, CArray[num64]) returns Algorithm::KdTree::Response is native(&library) { * }
+my sub kd_nearest_range(Algorithm::KdTree, CArray[num64], num64) returns Algorithm::KdTree::Response is native(&library) { * }
+my sub kd_create(int32) returns Algorithm::KdTree is native(&library) { * }
+my sub kd_free(Algorithm::KdTree) is native(&library) { * }
+my sub kd_clear(Algorithm::KdTree) is native(&library) { * }
 my int32 $c-dimension;

 method insert(@array) returns Int {

@skaji Thank you for sharing your experience !
It looks that using sub library is the rational way to solve this problem.


Can move calculateBudget to the only line that uses it

https://github.com/programmablewealth/gotchiverse-crowdfarming-contracts/blob/6487d7dca7cb93cd1448b315369956f2fff5203a/contracts/facets/CrowdFundingFacet.sol#L57

implemented 👍


Add ETA to transfer informations

It would be nice if was shown the ETA of the transfer. Maybe it could be placed near the progress bar.
Thanks!

Possible, but potentially useless:
https://www.youtube.com/watch?v=iZnLZFRylbs

It would be great if anyone would like to tackle that, as most likely I won't have time to work on that.
I do not know how consistent the transfer speed is and thus how accurate and useful it would be.

Like the linked video shows, accurate progress bars are kind of an unsolved problem, but there are solutions that are good enough.
There have to be libraries that do the hard part, so it should actually be quiet easy to add this feature (for a JS developer).
It would be very useful to have, especially when sending big files which take hours.


Doesn't work with Java 21

Describe the bug
The current release, 2.2.0, doesn't work with Java 21.
Additional context
I think other plugins have fixed the same issue already:
https://github.com/palantir/palantir-java-format/pull/909

Is someone going to contribute to update to work with Java version 21?

Will 2.21 be published to Maven Central, too?
Thank you very much,
Stefan

Yes, it's already there: https://mvnrepository.com/artifact/com.spotify.fmt/fmt-maven-plugin/2.21.1


Update _index.md

Added  Arvancloud to list of supported webhooks
Signed-off-by: Hamed Bahadorzadeh<EMAIL_ADDRESS>

/ok-to-test
Do we have guidelines that must be followed to be able to have an external issuer listed?
Related: #501

Oooops sorry, I mistakenly closed the issue! 😇

@maelvls Not for webhooks :smile:
/ok-to-test
/approve
/lgtm


Enable use with SWADE system

Savage Worlds Adventure Edition (SWADE) uses "tokens" to measure progress with subsystem rules such as Dramatic Tasks and Social Conflicts. Enabling this for SWADE would allow GMs and players to track those tokens easily.

@kristianserrano Done. :-) v1.3.0 supports every system now, included SWADE.


Release fails: Cannot read property 'length' of null

Hey everyone,
I tried to release a Docker image but I'm getting this stack trace:
Remote addons: bonsai (1)
Local addons:  (0)
Missing addons:  (0)
Creating local slug...
web uses an image, skipping
/home/user/.local/share/heroku/plugins/node_modules/heroku-container-tools/commands/release.js:121
        var imageId = tokens[tokens.length - 1];
                                   ^

TypeError: Cannot read property 'length' of null
    at ChildProcess.onBuildExit (/home/user/.local/share/heroku/plugins/node_modules/heroku-container-tools/commands/release.js:121:36)
    at emitTwo (events.js:106:13)
    at ChildProcess.emit (events.js:191:7)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:204:12)

Did someone of you ran into the same error? It also happened without the bonsai addon

Willi,
This plugin is deprecated.
Could you try using our container registry and runtime instead?

Of course!
Thanks and sorry for the inconvenience


Change to Cache for Loadbalance

Pull Request Test Coverage Report for Build 1202

0 of 7 (0.0%)  changed or added relevant lines in 1 file are covered.
25 unchanged lines in 9 files lost coverage.
Overall coverage decreased (-0.2%) to 62.95%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

control/archaius/transfer.go
0
7
0.0%

Files with Coverage Reduction
New Missed Lines
%

core/registry/healthz.go
1
47.01%

core/loadbalancer/filter.go
1
76.6%

core/config/sr_config.go
2
0.0%

core/options.go
2
85.96%

core/router/cse/router_manager.go
2
72.73%

core/handler/router_handler.go
3
0.0%

core/registry/util.go
4
65.63%

client/rest/rest_client.go
5
77.45%

core/lager/logrotate.go
5
33.94%

Totals

Change from base Build 1200:
-0.2%

Covered Lines:
5002

Relevant Lines:
7946

💛  - Coveralls


Form field for users, spawn rate, and run time still visible in UI although CustomShape defined without use_common_options.

Prerequisites

[X] I am using the latest version of Locust
[X] I am reporting a bug, not asking a question

Description
in documenttions it mention

When using shapes, the the Users, Spawn Rate and Run Time options will be hidden from the UI, and if you specify them on command line Locust will log a warning

But I am still able to see those fields in UI.

Command line
locust -f scripts/websocket_load_test.py --logfile halo.txt
Locustfile contents
# locustfile.py
import json
import logging
import os
import time

import websocket
from locust import HttpUser
from locust import LoadTestShape
from locust import TaskSet
from locust import between
from locust import task
from locust_plugins.users.socketio import SocketIOUser

DUMMY_COURSE_SLUG = os.getenv("DUMMY_COURSE_SLUG", default="xx")
DUMMY_CHAPTER_SLUG = os.getenv("DUMMY_CHAPTER_SLUG", default="yy")
DUMMY_EXERCISE_SLUG = os.getenv("DUMMY_EXERCISE_SLUG", default="zz")
BASE_URL = os.getenv("BASE_URL", default="ws://localhost:3001")
URL = "ws://localhost:3001/"

class StepLoadShape(LoadTestShape):  # Define a new LoadTestShape class
    stages = []
    for minute in range(5):
        duration = minute * 60
        users = minute * 2
        stages.append(
            {
                "duration": duration,
                "users": users,
                "spawn_rate": 1,
            },
        )

    def tick(self):
        run_time = self.get_run_time()

        for stage in self.stages:
            if run_time < stage["duration"]:
                tick_data = (
                    stage["users"],
                    stage["spawn_rate"],
                )
                return tick_data

        return None

class MySocketIOUser(SocketIOUser):
    wait_time = between(20, 20)

    @task
    def my_task(self):
        logging.info("task")

Python version
python 3.12
Locust version
2.31.8
Operating system
Mac os

@andrewbaldwin44 Maybe you can take a look at this when you're back?


Powershell now supports dual path to single path migration

Can I migrate VMs that were encrypted with an Azure AD app to encryption without an Azure AD app?
Currently, there isn't a direct migration path for machines that were encrypted with an Azure AD app to encryption without an Azure AD app. Additionally, there isn't a direct path from encryption without an Azure AD app to encryption with an AD app.
QA mentioned above it doesn't support migration but following commands support dual path to single path migration, the document should be updated.
Set-AzVMDiskEncryptionExtension -ResourceGroupName  -VMName  -Migrate

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

ID: e1db5eef-75ae-aeb0-5562-595d7a72a047
Version Independent ID: c2b5ff71-4a5c-ee59-4e8a-bf8501b86db8
Content: FAQ - Azure Disk Encryption for Windows VMs - Azure Virtual Machines
Content Source: articles/virtual-machines/windows/disk-encryption-faq.yml
Service: virtual-machines
Sub-service: disks
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@gengyujie
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@msmbaldwin
Can you please check and add your comments on this doc update request as applicable.

I would suggest asking this on Microsoft Q&A where we have experts who can answer these kinds of questions. If a doc update is needed post answer, please just let me know.
https://learn.microsoft.com/en-us/answers/questions/
#please-close


Using sudo pip install is a bad idea

Using sudo pip install ... is a bad idea as managing the packages of the system python is the job of system's package manager. it should instead be pip install --user ....
This would be a change for both the README and command substitution behaviour.
See pypa/pip#1668

Thanks, fixed.


Example Livebooks have deps that error

Pretty much all the Livebooks from the Axon guides have deps that don't work out of the box.  It's 4/4 so far.
To rectify this I've been deleting them and adding them in manually which grabs the latest.
The error I'm getting:
Unchecked dependencies for environment dev:
* nx (https://github.com/elixir-nx/nx.git)
  the dependency does not match the requirement "~> 0.3.0", got "0.5.1"

** (Mix.Error) Can't continue due to errors on dependencies
    (mix 1.14.2) lib/mix.ex:513: Mix.raise/2
    (mix 1.14.2) lib/mix/tasks/deps.loadpaths.ex:43: Mix.Tasks.Deps.Loadpaths.run/1
    (mix 1.14.2) lib/mix/task.ex:421: anonymous fn/3 in Mix.Task.run_task/4
    (mix 1.14.2) lib/mix.ex:782: anonymous fn/5 in Mix.install/2
    (elixir 1.14.2) lib/file.ex:1607: File.cd!/2
    (mix 1.14.2) lib/mix.ex:749: Mix.install/2
    (stdlib 4.2) erl_eval.erl:744: :erl_eval.do_apply/7
    (elixir 1.14.2) src/elixir.erl:294: :elixir.eval_forms/4

I think you should be able to remove the Nx dependency. Instead, only list Axon and potentially EXLA. PRs are welcome!

Fixed in #500


Jupyter notebook autocomplete suggestion "%%!"

Issue Type: Bug
Starting docstring (''') or a function definition causes autocomplete to suggest "%%!". It means I need to ESC away from it constantly, or it chucks "%%!" in my code.
Someone else reported this issue here: https://stackoverflow.com/questions/69134016/vscode-jupyter-strange-autocomplete
Extension version: 2021.9.1001312534
VS Code version: Code 1.61.0 (ee8c7def80afc00dd6e593ef12f37756d8f504ea, 2021-10-07T18:13:09.652Z)
OS version: Windows_NT x64 10.0.19041
Restricted Mode: No

System Info

Item
Value

CPUs
AMD Ryzen 5 3600 6-Core Processor               (12 x 4229)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.93GB (25.78GB free)

Process Argv
--crash-reporter-id 7693d6ac-337e-4f4a-a5f3-9b5534313163

Screen Reader
no

VM
0%

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
pythonvspyt639:30300192
pythontb:30283811
pythonvspyt551:30345470
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30358481
pythondataviewer:30285071
pythonvsuse255:30340121
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
vsccppwtct:30378365
pythonvssor306:30344512
bridge0708:30335490
pygetstartedt2:30371810
dockerwalkthru:30377721
bridge0723:30353136
pythonrunftest32:30373476
pythonf5test824:30373475
javagetstartedt:30364666
pythonvspyt187:30373474
pydsgsc2:30361791
vsqsis200:30374795
vsaa593cf:30376535
vssld246:30379911

I'm facing the same issue.
Jupyter Extension
Version: 2021.9.1101343141
VS Code
Version: 1.61.0 (user setup)
Commit: ee8c7def80afc00dd6e593ef12f37756d8f504ea
Date: 2021-10-07T18:13:09.652Z
OS: Windows_NT x64 10.0.19042

Same here, quite annoying.
This seems to have been introduced at v2021.8.2031187289.
Rolling mine back to v2021.8.1236758218 solved it for now.

Indeed, it really breaks up your typing workflow, having to Escape everytime you enter a new line in a notebook.
Can't there be a filter for "%%!"? I can't imagine it every being useful.

I also downgraded to v2021.8.2031187289 to avoid this problem for now, but I hope you'll do something about it.


Limit experience

Purpose
Allow option to limit experience cards shown on page to a given number. The page shows the requested number of cards and prints a link "Show more". If clicked, the rest of experience cards are shown, and "Show more" link turns to "Show less". If clicked on "Show less", additional cards collapse and the original number of cards is shown.
Fixes #2455
Screenshots
Collapsed view

Expanded view

Documentation
Requires specification of the desired number in the frontmatter of the experience.md file via option count. For example,
count: 5

The Experience widget is not intended for this more advanced use case.
Consider sharing a custom widget instead: http://localhost/docs/widget/custom/


svgrrc json schema

Is there a json schema available for .svgrrc files?

No, but I am open to contributions!

I've once used ts-json-schema-generator package to turn a typescript interface definition to a json schema and been successful. I think it might be possible to do something like that here.


ntp serial gps page is confusing

When Generic or Default is selected you can change the list for 'NMEA sentences' but upon save it get's reset to 'ALL' I only seem to be able to change that one if I select Custom.
This needs to be checked if this is the desired behavior, if it is, the list should be disabled when selecting something other than Custom.

#85 now also covers this, closing


Issue#426 Add stream() and reverseStream() methods that provide Int/LongStream() for streaming the index of set bits in Roaring(64)Bitmaps. (#427)

This is a copy of  @JervenBolleman PR.
Fixes https://github.com/RoaringBitmap/RoaringBitmap/issues/426

Ah. Now we are getting the tests.

@JervenBolleman  You like really long lines!!! :-)
We have style checkers in place and we forbid overly long lines.

Now we are good. Merging.

@lemire thanks for fixing the long lines. I only saw the line length requirements after making the pull request and did not have time yet to fix it myself. I really depend on the autoformatter of eclipse in my day job to avoid this, but used netbeans to do this patch.


Can we get a new release?

It seems that the last release links to serde 0.7 while serde 0.8 is already released. Thank you!

Yes. I'll try to do a new release soon.

Yes. I'll try to do a new release soon.

Closed by: 12d49e8

Thank you!


Not compile with gflags

RocksDB cmake default to build with gflags but we actually don't depend on it. We thought ldb would depend on gflags but it actually not. Disabling it.
Tested by build tikv with the change. Tried tikv-ctl --db=xxx scan --from=xxx --to=xxx and it works.
Signed-off-by: Yi Wu<EMAIL_ADDRESS>

cc @GiltterIsMe

failures:
---- perf_context::test::test_perf_context stdout ----
thread 'main' panicked at 'assertion failed: (left == right)
left: 2574,
right: 0', src/perf_context.rs:431:9
Is this test unstable?

@zhangjinpeng1987 That is being fixed by #321. Let me rebase and see.


Timeout was reached

Newsboat version (copy the output of newsboat -v or the first line of git show):
newsboat 2.20.0
System: Linux 5.7.4-arch1-1 (x86_64)
Compiler: g++ 10.1.0
ncurses: ncurses 6.2.20200212 (compiled with 6.2)
libcurl: libcurl/7.70.0 OpenSSL/1.1.1g zlib/1.2.11 libidn2/2.3.0 libpsl/0.21.0 (+libidn2/2.2.0) libssh2/1.9.0 nghttp2/1.41.0 (compiled with 7.70.0)
SQLite: 3.32.3 (compiled with 3.32.3)
libxml2: compiled with 2.9.10

Config file (copy from ~/.newsboat/config or ~/.config/newsboat/config):
user-agent  "Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.13; ko; rv:1.9.1b2) Gecko/20081201 Firefox/60.0"
use-proxy   yes
proxy-type  socks5
proxy   <IP_ADDRESS>:6666
color background          white   default
color listnormal          white   default
color listfocus           color36 black
color listnormal_unread   color133 default
color listfocus_unread    color36 black
color info                white  default  bold
color article             white   black
color end-of-text-marker    white black
browser "/usr/bin/firefox-developer-edition %u"

Related logs:
[2020-06-23 03:37:26] INFO: FeedListFormAction: reloading feed at position `39'
[2020-06-23 03:37:26] DEBUG: Reloader::reload: pos = 39 max = 0
[2020-06-23 03:37:26] DEBUG: Reloader::reload: created parser
[2020-06-23 03:37:26] DEBUG: RssParser::download_http: user-agent = Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.13; ko; rv:1.9.1b2) Gecko/20081201 Firefox/60.0
[2020-06-23 03:37:26] DEBUG: running query: SELECT lastmodified, etag FROM rss_feed WHERE rssurl = 'https://blog.torproject.org/rss.xml';
[2020-06-23 03:37:26] DEBUG: Cache::fetch_lastmodified: t = 0 etag =
[2020-06-23 03:37:56] DEBUG: rsspp::Parser::parse_url: ret = 28 (Timeout was reached)
[2020-06-23 03:37:56] ERROR: rsspp::Parser::parse_url: curl_easy_perform returned err 28: Timeout was reached
[2020-06-23 03:37:56] USERERROR: Error while retrieving https://blog.torproject.org/rss.xml: Timeout was reached
[2020-06-23 03:38:10] INFO: ReloadThread: starting reload

Steps to reproduce the issue:

Only need to update these two urls:

https://blog.torproject.org/rss.xml
https://intrusiontruth.wordpress.com/feed/

Other info you think is relevant:
Why does updating the following two fail (timeout)? I can open it normally in the browser,Other URLs are normal,my proxy is also working fine,I can't find the problem.

I can't reproduce, which doesn't surprise me since I don't have your proxy. I only have one idea regarding how this issue can happen: since you're using socks5 not socks5h, the domains are resolved to IPs that are very far from your proxy's exit point, thus leading to delays. But the default timeout in Newsboat is 30 seconds, which should be sufficient even for the slowest of connections.
My suggestions, in the order of decreasing priority:

change proxy type to socks5h, so DNS requests are proxied as well, and see if it helps;
bump download-timeout;
try with an earlier Newsboat version to see if it's a regression.

Please report about the results.

I can't reproduce, which doesn't surprise me since I don't have your proxy. I only have one idea regarding how this issue can happen: since you're using socks5 not socks5h, the domains are resolved to IPs that are very far from your proxy's exit point, thus leading to delays. But the default timeout in Newsboat is 30 seconds, which should be sufficient even for the slowest of connections.
My suggestions, in the order of decreasing priority:
1. change proxy type to `socks5h`, so DNS requests are proxied as well, and see if it helps;

2. bump [`download-timeout`](https://newsboat.org/releases/2.20/docs/newsboat.html#download-timeout);

3. try with an earlier Newsboat version to see if it's a regression.

Please report about the results.

thank you very much! Now the problem is solved: just change the proxy-type from socks5 to socks5h.
I get a timeout because curl does not proxy my DNS request, and the URL is polluted by DNS in my country.Everything works now.


How do you do a restore

I would like to perform an automatic restore when on start the backup source directory is empty.
Or more in general how can perform a restore with conplicity.

@Vad1mo you should use duplicity to restore a backup.
As this is a on-demand thing for us, we didn't implement any automated thing to do it, but if you have any workflow requiring an automated restore, we'd gladly merge a PR allowing to do it.

Actually since Bivac v2 we should be able to add restore feature. I re-open this issue.

Restore is in alpha stage in the following pull request.
https://github.com/camptocamp/bivac/pull/273

The PR has been merged, this feature will be available in the next release (v2.1). The documentation is available here : https://github.com/camptocamp/bivac/wiki/Usage#restore-a-volume


Add CXXII Liceum Ogólnokształcące Warsaw

School domain:
https://domeyko.edu.pl/
Student's school emails follow<EMAIL_ADDRESS>format
https://domeyko.edu.pl/kontakt page has contact email. By writing to this email you can hopefully verify that student's emails follow the format given above.

@pam-param-pam First stage of verification passed, official domain is domeyko.edu.pl as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://domeyko.edu.pl/wp-content/uploads/2022/09/specjalisci.pdf

@pam-param-pam Pull request merged. Thank you.

Thanks

@philipto Sorry to mention you but i still cannot apply.

Have i done something wrong?


Update StackExchange.Redis

I'm working on the SO Redis upgrade and I want to make sure we are using the latest SE.Redis version, it probably won't have an impact but just in case. I'm updating this project cause I see it's used on Calculon. I've build the project locally and run the unit tests.

Not needed anymore


Created "DescriptivePool" utility class

This PR created the "DescriptivePool" utility class which is used for calling methods onObtain and onFree whenever an object is freed/obtained from a DescriptivePool instance. Provides extension interface of Poolable named DescriptivePoolable.
Any comments, concerns, suggestions, etcetera would be appreciated if there is anything keeping this from being accepted.
p.s.: thanks @Darkyenus for the tip. x2

@Darkyenus Looks like, once again, you had a great idea and it ended up prettier :)

Is it just me or is "DescriptivePool" not a very descriptive name for this? What makes it descriptive?

@MobiDevelop I chose it because it gives more description to the Poolable object on whats happening (Free / Obtain)

@MobiDevelop Yeah, you're definitely right now that I think about it some more. I've come up with two other possible names: CallbackPool and EventPool. How do you feel about those? As well, if you have any suggestions personally, I'm up to hear!

We already have Poolable which has reset, called when an object is freed. That is identical to this PR's onFree. The other thing this PR adds is onObtain. Do objects need to know when they were obtained? What would be a use case? Is it common enough to warrant a specialized type of pool? Does this make more sense as application code?

@NathanSweet Poolable#reset is a protected method that can be overriden and thus its behavior cannot be reliably predicted. The main use case I found for this is for an object to know if it is active or inactive, which most definitely is useful.

@Ktar5 DescriptivePoolable onFree is public and can be overridden (it's an interface after all). I don't follow how that is any different from Poolable reset, except that it is public rather than protected.
It's very common to need to clean up state when an object is no longer used, hence Poolable reset. It's also very common to need to configure an object obtained from a pool for a particular use. However, such configuration usually always requires data from outside the object, eg:
Thing thing = pool.obtain();
thing.setPosition(x, y);
thing.setColor(...);
// Other application specific configuration for this particular thing instance.

For your use case, it is trivial to set the state, eg thing.setActive(true);. Even if the object wanted to do something like resource loading, it's easy enough to do thing.loadResources(); after obtain.
I have a feeling that onObtain is not useful enough in most cases and is easily done with application code like the above rather than adding a new type of pool to libgdx. It might have made sense to add it to the Poolable interface, but doing that now would require adding the new method to a lot of existing code and isn't worth the trouble.

@NathanSweet Oops, I meant to say Pool#reset() is protected and can be overriden. Because that is what's called, as ooposed to direct Poolable#reset().
But I definitely see where you're coming from. It is quite an edge case. But I do feel a portion of people would find it useful.
The problem with writing it in application code is doing thing.setActive(false); currently I can't think of any way to do that without the DescriptivePool class.

@Ktar5 I don't see the need for this either, why can't you just implement Poolable#reset() and call setActive(false) there?

@obigu Because Pool#reset() (which calls Poolable#reset() ) is protected. So it could be randomly called by a subclass, or not call Poolable#reset() at all-- thus producing unexpected behavior.

Pool subclasses will mostly always implement the only abstract method on Pool which is newObject. True, you might use a subclass you don't know the implementation details about and not have reset called if that implementation docs doesn't specify it. But following that logic you could say the same for all the Pool public methods which can be overriden aswell and cause unexpected behaviour. I'm sorry but is this a purely academic discussion or you have a real issue/use case that can't currenlty be implemented?

If someone wants to do something bad like call reset at odd times or never at all, that can't be avoided. It's not worth worry about, there are so many other terrible things someone can do, they deserve the consequences.
The rest I think can be implemented in application code, so I think we can close this issue.


Terraform to ignore changes to reply uri's

As architect I would not like terraform to override any manual entries that we put in the reply URI's used for app registrations
Originally posted by @qcaas-nhs-sjt in #52

This has been completed


docs: example

although there is a docker-compose file that can be used to bring everything up, we should document how that works under the hood and how one could make use of it.

done @ 55659b7af23b9b72ad174eabc2390d82301b86d8


new dataset format with librispeech and commonvoice

Introducing a new dataset format based on generators, along with two new datasets: LibriSpeech and CommonVoice.
Related to

pytorch/pytorch#24915
keunwoochoi/torchaudio-contrib#67
notebook for experiment

For best efficiency (if that's a goal here), it might make some sense to format the audio such that utterances are shuffled and packed into one or more tars (or something like it) such that sequential i/o can be used. Especially for these larger datasets, dealing with thousands of files and accessing them randomly over network can become a significant challenge.

@ryanleary - I can't agree more that pipelining and chunking are necessary optimizations. For now we're aiming for decreased complexity and coverage for these new datasets without breaking backwards compatibility. After that we'll work on performance.

I've defined the interface close to what was existing, threw a few deprecation warnings, and removed the "legacy" folder. @cpuhrsch thoughts?


Custom Endpoint on Collection Breaks Values in Admin UI

Bug Report
When creating a custom endpoint on a collection, if only a /:parameter is passed as the path value, a record in this collection values disappears in the Admin Panel.
Steps to Reproduce

Create a custom endpoint on a collection
For the endpoints path value, use a path that is just a parameter, such as { path: "/:id", ... }

For example, if you had a collection of Stores, you would call http://localhost/api/stores/13

Navigate to the Admin Panel and select a record in the collection and notice its values are missing

Other Details

If you put additional pathing before or after the parameter, the bug goes away, for example:

{ path: "/grocery/:id", ... } or { path: "/:id/grocery", ... }
... will not cause this bug

If constructed properly, the endpoint will still work even with the UI bug present.

This is expected. What you are doing by adding that endpoint is replacing the existing endpoints mounted on /api/:collection/:id - like PATCH for update, GET for findByID, etc.
Does that make sense?


Syslog Integration

Environment

OS: Any
Erlang/OTP: R18+
EMQ: 2.0.5

Description
Integration with Syslog

The feature has been implemented in 2.1 release.


Minor changes to function names and small amount of code reorganization.

Should be a non-functional change.

Coverage decreased (-0.08%) to 83.23% when pulling e0824048d2c002285b97cff284e4f328038c5704 on thongsav-usgs:master into 31efca62303dc1ebb0c051dab568833127e03401 on USGS-R:master.


StageButtons

How can you hide the previous button and show the next button dependent on the stage index?
I tried:
{context.state.currentStage !== 1 &&
    <Button transparent large onPress={context.prev}>
        <Icon name="ios-arrow-dropleft-circle" />
    </Button>
}
{context.state.stages.length !== context.currentStage() &&
    <Button transparent rounded large onPress={context.next}>
        <Icon name="ios-arrow-dropright-circle" />
    </Button>
}

you have a couple of options. I usually prefer to delegate the current control to an outside state, in the parent of Stager and check from there. you can use the functions in context like context.has('prev') or context.has('next') to determinate if you can show / hide the control buttons.
the docs are lackluster regarding the API, but I'll complement it soon

OK I’ll give that a shot. Thanks.

On Sep 11, 2017, at 8:45 PM, Paulo Cesar<EMAIL_ADDRESS>wrote:
you have a couple of options. I usually prefer to delegate the current control to an outside state, in the parent of Stager and check from there. you can use the functions in context like context.has('prev') or context.has('next') to determinate if you can show / hide the control buttons.
the docs are lackluster regarding the API, but I'll complement it soon
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/pocesar/react-native-stager/issues/3#issuecomment-328707095, or mute the thread https://github.com/notifications/unsubscribe-auth/AKZgMHNLWaoYlRojMsjQNBzh4YuhRwyzks5sheJUgaJpZM4PT2_h.

Ok that works. Thanks!
   <StageButtons>
          {({ context }) => (
            <View style={{ flexDirection: 'row' }}>
              {context.has('prev') &&
                <Button transparent large onPress={context.prev}>
                  <Icon name="ios-arrow-dropleft-circle" />
                </Button>
              }
              {context.has('next') &&
                <Button transparent rounded large onPress={context.next}>
                  <Icon name="ios-arrow-dropright-circle" />
                </Button>
              }
            </View>
          )}
        </StageButtons>

How would you go about conditionally showing the next only on some stages?

On Sep 11, 2017, at 8:45 PM, Paulo Cesar<EMAIL_ADDRESS>wrote:
context.has('prev')

How would you go about conditionally showing the next only on some stages?
There are some situations where the next button should be disabled until the user enters the correct information.
Example: next buttons should be disabled until the pin has 4 digits...

Nvm. I just saw this.
<Button title="can continue" onPress={() => {
          this.setState({
            canContinue: true
          }, instance.refresh)
        }} />

Next question is, in the above example the step progress bar does not stretch across the view. How do that?

the
<View style={styles.progressPad} />

in https://github.com/pocesar/react-native-stager/blob/master/src/index.tsx#L436 is to force using 30% of the available screen width with an empty element. you can remove that and set the flex of the styles.progressFlex with a flex: 1 to use all available space


Navigate between different pages containing tabs, wrong tab is shown

Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x] 3.x
[ ] 4.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
Please do not submit support requests or "How to" questions here. Instead, please use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
I've two different lazy loaded pages, both containing ion-tabs with two tabs inside it.
Page X:
<ion-tabs tabsPlacement="top">
  <ion-tab [root]="'TabAPage'" tabTitle="Tab A"></ion-tab>
  <ion-tab [root]="'TabBPage'" tabTitle="Tab B"></ion-tab>
</ion-tabs>

Page Y:
<ion-tabs tabsPlacement="top">
  <ion-tab [root]="'TabCPage'" tabTitle="TabC"></ion-tab>
  <ion-tab [root]="'TabDPage'" tabTitle="TabD"></ion-tab>
</ion-tabs>

I start navigating to Page X, okay, everything normal. Switch between tabs works fine.
But, when I navigate to Page Y, that should only show tabs C and D, starts with tab A (or B, the latest shown on page X).
When I switch between tabs on page Y, instead of switching between C and D, I switch between A and D.
If I start the process navigating from Page Y, the result is the same, it works initially until I navigate to another page containing tabs.
Expected behavior:
It should not mix up contents! Makes no sense!!!
Steps to reproduce:
Explained above
Related code:
I'm unable to create an sample code now, but I'll do ASAP!

Other information:
This looks like a problem with NavCtrl, since I need to do this.nav.parent.setRoot() to switch between pages, it replaces the parent view, but doesn't reset the child view.
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
cli packages: (/home/esoares/.npm-global/lib/node_modules)

    @ionic/cli-utils  : 1.19.1
    ionic (Ionic CLI) : 3.19.1

global packages:

    cordova (Cordova CLI) : 7.0.1

local packages:

    @ionic/app-scripts : 3.1.8
    Cordova Platforms  : android 6.2.3
    Ionic Framework    : ionic-angular 3.9.2

System:

    Android SDK Tools : 26.0.2
    Node              : v8.9.4
    npm               : 5.6.0
    OS                : Linux 4.13

Environment Variables:

    ANDROID_HOME : /home/esoares/android-sdk/

Misc:

    backend : legacy

This work around fixes my issue:
https://github.com/ionic-team/ionic/issues/12625#issuecomment-370205019

almost same problem but didn't resolve it


IGDM issue when I type 3 dots "..."

Not sure if its a bug, or a secret thing. I tend to use three dots, in place of punctuation. Yeah its a habit...but, when I do I get random pics pop up in IGDM. LAtely its a 2017 picture of Jeffrey Epstein.

this issue is now fixed in the latest version

I've just downloaded the latest version using git clone https://github.com/ifedapoolarewaju/igdm.git and it is still occurring.


Feature: Smart pointer that does not support usage across multiple th…

…reads

Thanks for adding this one! I've been meaning to get around to it but never have the time. It'll very much come in handy. I did change the implementation a bit and hopefully simplified a few parts.
Later on I'd love to have make_shared for it, ensuring both the pointer and the counter can be allocated in a single allocation, instead of needing two separate ones.
Anyway, merged.


docs: load lightdom css on demo pages

What I did

account for -lightdom-shim.css in the 11ty build

Testing Instructions

check dp /elements/call-to-action/demo/no-javascript-customization/

Notes to Reviewers

@marionnegp i think it's a problem with the cta lightdom shim approach, and one of the reasons we should prefer ssr. I may be mistaken here, @zeroedin can perhaps provide clarity.

That demo appears to be not using <rh-surface> but <rh-context-provider> will update.


feat(logging): Add prometheus metrics & structure logs

Relative Issues: Resolve #42
Describe the pull request

Add a prometheus endpoint to provide metrics about application and structure the logs given by the application
TODO

[x] Add prometheus /metrics endpoint
[x] Add possibility to enable/disable the prometheus endpoint via configuration
[x] Add a logging middleware to provide more logs when application is running

Checklist

[x] I have linked the relative issue to this pull request
[x] I have made the modifications or added tests related to my PR
[x] I have added/updated the documentation for my RP
[x] I put my PR in Ready for Review only when all the checklist is checked

Breaking changes ?
no
Additional context
Documentation here : https://github.com/42Atomys/webhooked/wiki/Observability

@rgaiffe takes a look 👀


how can i export admin bread configurations made on my local machine to remote machine

Laravel Version: #.#.#
Voyager Version: #.#.#
PHP Version:
Database Driver & Version:

Description:
i have followed all the steps in the documentation and have successfully created breads for the neccessary tables but after pushing my project to the remote machine, i cant see those configurations anymore.
Steps To Reproduce:

mazeeblanke I think you make a valid point even if you do not understand why your configuration and bread no longer show up.
I see this was marked as Invalid and I'm sorta surprised as to why.  One of the amazing features built into Laravel is migrations.  I can work work locally, with a team, and push to production all while keeping the DB intact.  Why oh why does Voyager throw this functionality out the door?

This was marked invalid because there was no question or issue clearly identified, the version information was not provided, and it was closed the same day by the same user who opened it

If you stumble on this issue, there is a package to create seeders from the database : https://github.com/orangehill/iseed
Hope it helps


Support multiple font files in commands

I was dumping metrics on a few dozen font files so added support for font <command> *.<ext> without having to resort to shell loops or xargs. When multiple files are passed I used ==> filename <== header that head and tail use.

Thanks!


Dropping support for JDK 7 in upcoming releases

We are about to drop support for JDK 7.
This seems uncontroversial at this point.

AWS supports java 8: https://aws.amazon.com/sdk-for-java/
So does Google: https://cloud.google.com/appengine/docs/java/
Apache Spark requires Java 8.
Druid dropped support for java 7 it seems: https://github.com/druid-io/druid/blob/master/.travis.yml
Kylin is on java 8 : https://github.com/apache/kylin/blob/master/.travis.yml

It seems that Pinot still tests on Java 7. So there might be some pain there when upgrading.
https://github.com/linkedin/pinot
Please chime in if you have concerns.
Ping @gssiyankai @blasd @Kineolyan @ppiotrow  @kishoreg @richardstartin ...

@lemire having Java 8 would open new possibilities for solving #42 (parallelization). Java 8 introduces CompletableFuture which might be helpful.

Pinot depends on Hadoop to for batch data ingestion and it's not clear to me that all Hadoop users have moved on to Java 8+. We would like to follow the major Apache projects such as Hadoop, Hive, Kafka etc. Do we know if they have dropped support for Java 7

@kishoreg
Hadoop requires Java 8 or better
Apache Hive no longer seems to support Java 7:

HIVE-16281: Upgrade master branch to JDK8

Lucene/Solr trunk moved to Java8.
Kafka 3.0.0 supports Java 8. It does not support Java 7.
So, yeah, it looks like the ecosystem has moved to Java 8 and it is now safe to move to Java 7, but I'd prefer to have your ok since you are a major use case for Roaring.
In all cases, we will continue to support (as far as reported bugs go) older JDK7-only releases. So we won't force people to upgrade, but we would like future versions of Roaring to be able to assume JDK8, and that's where the new features will go.

That's great.  I will check with Uber team as well since they were on JDK 7 some time back. We will make the changes to Pinot to drop support for java 7.

Ok. I think we are safe enough.


Reginamaria - Missing "Targetare.ro" link

ENVIROMENT: production
URL: https://scrapers.peviitor.ro/src/reginamaria/index.html
Browser: Chrome
Device: laptop
OS: Windows 10
STEPS TO REPRODUCE:
1.Open URL in browser
2. Go to company logo
3. Check for Targetare.ro link
EXPECTED RESULTS:
Targetare.ro link is visible and opens
ACTUAL RESULTS:
Targetare.ro link for Reginamaria company does not exist
ATTACHMENTS:

Retested, Targetare.ro link is visible and opens


Integrate GitHub Actions for CI

Integrate Github actions for the repo to:

Run the tests on every pull request raised
Run on each push, merge commit on main branch
Remove the existing .travis.yml file

I'd like to work on this issue. Please assign me.


Creating a Custom Data Transformer using Scikit-Learn

Article checklist
ATTENTION: In your PR - add the ISSUE (topic form) # that this PR corresponds to. You can do this by also adding a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
If you have not already please go over our Resources Page for more tips and tricks on how to improve our overall technical writing.
Preliminary Checks - Formatting and Structure

[x] Does your article follow any of the suggested structure formats? - see example formats
[x] Is your article properly formatted in Markdown? - see Github Markdown guide
[x] Have you used the correct folder and file structure? - see Contribution Guidelines
[x] Is your article over 750 words? - Tool: Wordcounter.net
[x] Have you used our preferred heading structure? - H3 (###) and up
[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.
[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.
[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images
[x] In your PR - add a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid -  Grammarly
[x] Please place your article through a 3rd party plagiarism checker? We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide

Contribution guidelines
For first-time contributors and for more details, see Contributing Guidelines
This fixes #2374

@srishilesh Please let someone else review this article - I believe you have more than 2-3 articles in reivew 👍
Thank you

@srishilesh Please let someone else review this article - I believe you have more than 2-3 articles in reivew 👍
Thank you

Sure.

plagiarism check done

@lalith1403 the link to the Google Colab has been added.

@lalith1403 I fixed some changes. check

@lalith1403

I can't access your Google Colab notebook.
It is asking for a password, please change the setting so that anyone with the link can access it.
Will merge when you do that. @Agusioma

I can't access your Google Colab notebook.
It is asking for a password, please change the settings so that anyone with the link can access it.
Will merge when you do that. @Agusioma


Integration tests are failing (usually just one)

Maybe cut down on parallel nature again? It used to work fine, though. Not sure what has changed.

Cut down from 4 to 2 in parallel tests. So far, everything passes. Will close if 10 successful runs on both appveyor and drone.


[hotfix][optimizer] remove duplicated code

What is the purpose of the change
remove duplicated code
Verifying this change
This change is already covered by existing tests, such as CoGroupGlobalPropertiesCompatibilityTest.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)

Hey @JingGe , sorry I found the CI fails suddenly. And the checkstyle is broken.

@1996fanrui  No worries. Thanks for the review. There is a follow-up PR for checkstyle fix: #23803


feat: add parsing email addresses, phones, links from cv

Closes #57.

[ ] I have reviewed my code in "Files changed" tab.
[ ] I have re-read the issue and this PR fully resolves it.

did you try on this file?

Yep, tried and it worked


*.github.com domains should have a HTTPS scheme, not HTTP

Perhaps the repo can have an HTTPS flag, which would look to the CNAME, but could be later expanded to look for other conditions.
To note, github_repo? is necessary, but not sufficient, as for example, choosealicense.com is not HTTPS.

@benbalter fixed in #35.


Issues when running as an Android app on Chromebook

This plugin has been working great for my app on both iOS and Android devices.  However, I installed the Android version of the app onto a chromebook, and it seems to freeze when calling the SpeechRecognition.startListening() function.  Has anyone else had similar issues?  If so, is there something I can do to get this working?

Josh, when I use SpeechRecognition.startListening() on a Chromebook Acer 10 (which has "Hey Google" recognition - so I know it's in there) - I get BIND_VOICE_INTERACTION is denied errors when I look at logcat. I've repeatedly tried to get permissions for this (android:permission="android.permission.BIND_VOICE_INTERACTION"), but I have not succeeded yet - like Google has proprietary permissions on Chromebooks for this kind of voice recognition. Hopefully this isn't so because education apps that use voice recognition is one of the points to having a Chromebook...  (I should mention, this is not a Cordova plugin specific issue, but an overall JNI issue as it applies to Chromebooks, I think.)


⚠ Fortnite has degraded performance

In ce27904, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1525 ms

Resolved: Fortnite performance has improved in e5670ae.


Multi-dimensional arrays are converted wrong in .NET Native

When using the .NET Native toolchain, the "as" operator can convert a 1-dimensional array to a multi-dimensional array.
Steps to reproduce:

Create a new UWP blank application.
Set the properties to "Compile with .NET Native toolchain"
Add a button, and paste the following code in the event handler:

       private void Button_Click(object sender, RoutedEventArgs e)
        {
            object[] test = new object[2];
            object test1 = test;
            object[,] test2 = test1 as object[,];
            if (test2 != null) throw new Exception("Test2 is not a multidimensional array");

        }

This code correctly sets test2 to null in all .NET platforms except .NET native. Note that you can't even assign those types directly: If you tried the code:
       private void Button_Click(object sender, RoutedEventArgs e)
        {
            object[] test = new object[2];
            object[,] test2 = test as object[,];
        }

This wouldn't compile with the message: Cannot convert type 'object[]' to 'object[,]' via a reference conversion, boxing conversion, unboxing conversion, wrapping conversion, or null type conversion
This is of course a very stripped down example, and in this case it could be fixed by checking for test2.Rank == 2. But in a huge codebase of existing code where we expect the "as" cast to return null, it causes significant issues.

Thanks for reporting this! Hmmm... this was fixed 2 years ago in #1558. Let me dig out why the fix is not there.

This was some unfortunate timing. I fixed the issue for the open source compiler in July 2016. At that time, only the open source compiler was generating multidimensional arrays this way (the UWP compiler was using a different representation). The UWP compiler switched to using the open source representation in September 2016 (90698bcde54ddc0eb7c1c452cef6641f8b73bbac).
Unfortunately, the closed source compiler forked to a stabilization branch at the end of July (missing my fix). The September 2016 was selectively ported to the stabilization branch, but my fix fell through the cracks.
To work around, you'll either need to downgrade to version 5.2.4 of the Microsoft.NETCore.UniversalWindowsPlatform NuGet package, or upgrade to 6.0.0+ (this means you'll have to target the Fall Creators Update of Windows 10 as well).

@MichalStrehovsky do you mean setting the Min version to Fall Creators Update? I have tried using Microsoft.NETCore.UniversalWindowsPlatform 6.0.8 with UWP MinVersion 10586 and TargetVersion 16299 with no luck.

Microsoft.NETCore.UniversalWindowsPlatform >= 6.0.0 has a dependency on new APIs introduced in the fall creators update to support NetStandard 2.0.
You need to set the MinVersion, otherwise it silently falls back to the 1.7 version of the compiler/runtime that has the bug.

@MichalStrehovsky Updating the MinVersion is not an option for us right now because a large part of our commercial users still have pre-FCU Windows 10 versions installed. I reckon it will take at least a few months or a year until most of our users will have the update. Are there any plans to fix this bug in the 1.7 version in the mean time? Or are there any downsides to using the NuGet version 5.2.4?


do you have specific example

HI
do you have any specific example if i need to use scss how i can use this package

I do not, but the format is the exact same as the sample config in there. Just add your files, and filter and it will process SCSS files.

Wasi - here are some excerpts from mine....
            'resources' => [
            'styles/scss/main.scss',

            'output' => 'app.css'

As Matt said it is no different. In my case, main.scss is my central include file for all my other scss files like variables, modules, includes, mixin's etc.
I do a lot of Sass/Compass front end work, so I have a fork of Matt's excellent repo you might like at https://github.com/mattyglover/laravel-potion
Specifically it allows for output path modifiers within potions which can be handy :D

I should say though that the example I included above works out of the box with this repo..

@mattyglover thanks alot it works for me :)


C++17 string_view

https://github.com/eidheim/Simple-Web-Server/blob/1056bd2e70c6abdd6a334d57247f1ea18bab45ef/client_http.hpp#L19
I changed this line to:
using string_view = std::string_view;

and life became wonderful. My application keeps data in char* format (with a known size_t). I was pushing these char* buffers into streams, and passing them to request. I realized that this results in 2 copies of the data (one to create my stream, and another request uses << to push that stream into ASIO).
string_view solves this wonderfully, as it simply wraps the pointer and length, and results in one less copy (some of my transfers are over 100mb, making copies painful and memory intensive).
I realize your original code comment might have to do with C++17 feature detection, but I think this can be cured with the following:
#ifdef __has_include
  #if __has_include(<string_view>)
    #include<string_view>
    using string_view = std::string_view;
    #define __has_string_view 1
  #endif
#endif
#ifndef __has_string_view
  using string_view = const std::string &;
#endif

Super-ultra-nice would be versions of request that took a char* and a size_t, I could see this being a super common use case.

Brilliant! Would you mind creating a pull request so that you get credited for this:)?

Lol, sure, I'll do it for the fame and fortune!

@eidheim Can't seem to push my PR. Getting permission denied.

@SethHamilton You need to fork this repository on your account, then you push your changes there. Then you can create a Pull Request to eidheim's repo.

I've made the PR here: https://github.com/eidheim/Simple-Web-Server/pull/214
This is failing CI, but I think it's because it's not set to the C++17 standard for compilation.

@pboettch @eidheim Ok, so, there is a (stupid) flaw with __has_include, it checks for the presence of the include, not whether it will compile or not. When not compiling with C++17 flags it will still pass. I'm now thinking a simple USE_CPP17_STRING_VIEW define might be the best way of selecting this.
The alternative would be to use the __cplusplus flag and check for C++17, but compilers like MSVC are refusing to switch this flag to the standard date for C++17 until they are fully compliant (gcc and clang don't seem to make this distinction). So, the option there would be to test for the right __cplusplus version if it's not  MSVC and perhaps another test for a specific MSVC define (if I can find one).

like perhaps something like this:
#if defined(__has_include) && __has_include(<string_view>) && (__cplusplus >> 201402L || (defined(_MSC_VER) && _MSC_VER >= 1912))
#include <string_view>
namespace SimpleWeb { using string_view = std::string_view; }
#elif !defined(USE_STANDALONE_ASIO)
#include <boost/utility/string_ref.hpp>
namespace SimpleWeb { using string_view = boost::string_ref; }
#else
namespace SimpleWeb { using string_view = const std::string &; }
#endif

I'm starting to lean toward something like USE_CPP17_STRING_VIEW, but I also see that potentially limiting going forward, C++17 features like string_view are going to be normal eventually.

https://github.com/eidheim/Simple-Web-Server/pull/214


🔨 added a function to the notebook

These functions were used to grab ALL outputs from the tasks, and add metadata to them after export

Sorry to intrude but I noticed this PR contains Jupyter notebook files that are hard to review on GitHub due to the JSON diffs.
You might want to checkout ReviewNB, code review tool for notebooks. Provides visual diffs & commenting for Jupyter notebooks on GitHub. Completely free for open source repositories.
Disclaimer: I built ReviewNB


Add github action for running the tests

Thanks @lukasz-gryzbon for starting this piece of work - after a couple small tweaks we now have a working github action for the builds 👍


chore(#40): Internals: ProxyAccessor optimizations

Field .ctors should depend on interfaces, not on classes. It is needed for mocking Registry and ObjectGraphTypeConfigurator in ProxyAccessor 's unit tests
Cache methods info, which are used in reflection calls, in static fields
Querying nested loaders/IQueryable via key and querying root key cause duplication of the field for the root query:
Properties for dependent expressions should have human-readable names
Get rid of Proxy<>.GetOriginal() method, replace it by dependence on identity expression

Codecov Report

Merging #42 (b642985) into develop (074d427) will increase coverage by 0.75%.
The diff coverage is 84.38%.

@@             Coverage Diff             @@
##           develop      #42      +/-   ##
===========================================
+ Coverage    77.60%   78.36%   +0.75%
===========================================
  Files          247      241       -6
  Lines        15678    15523     -155
  Branches      1341     1342       +1
===========================================
- Hits         12167    12164       -3
+ Misses        3030     2879     -151
+ Partials       481      480       -1

Impacted Files
Coverage Δ

...Configuration/Implementations/FieldDependencies.cs
100.00% <ø> (ø)

...mplementations/FieldResolvers/AsyncFuncResolver.cs
85.71% <ø> (+10.71%)
:arrow_up:

...lementations/FieldResolvers/ProxiedFuncResolver.cs
100.00% <ø> (+27.27%)
:arrow_up:

...ieldResolvers/ProxiedQueryableAsyncFuncResolver.cs
85.00% <ø> (+3.07%)
:arrow_up:

...ations/Fields/ChildFields/ConnectionLoaderField.cs
100.00% <ø> (ø)

...ns/Fields/ChildFields/ConnectionLoaderFieldBase.cs
95.08% <ø> (-0.16%)
:arrow_down:

...ons/Fields/ChildFields/ConnectionQueryableField.cs
72.50% <ø> (+1.07%)
:arrow_up:

...lementations/Fields/ChildFields/EnumerableField.cs
95.65% <ø> (+64.88%)
:arrow_up:

...ntations/Fields/ChildFields/EnumerableFieldBase.cs
92.85% <ø> (-0.25%)
:arrow_down:

...plementations/Fields/ChildFields/QueryableField.cs
95.74% <ø> (-0.34%)
:arrow_down:

... and 64 more

Continue to review full report at Codecov.

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


0 data collected

What happened?
After the last update, no more data is collected.
This night he therefore did not irrigate despite the high temperatures. And this morning there are still 0 data collected

How to reproduce
Currently in progress
config_entry-smart_irrigation-73ee0f4bb3c39b6f9c7c010022a9f909-4.json
Relevant log output
No response
Which version are you running?
v2024.7.4
Diagnostics file

[X] I have attached a diagnostics file

Additional information
No response

Please install 2024.7.5 first and try again.

Hi,
Just updated to 2024.7.5. Got the same issue. I needed to enter OW api key once again but OpenWeather stopped working. In sensors groups OpenWeather is crossed out.
it seems that latest update completely broke this integration.
How can I install 2024.4.3?

Clear your browser cache and restart HA. Mostly likely that's the solution. If doesn't help please open your own issue. Saying 'same issue' doesn't help.

I have upgraded to  v2024.7.5.
I activated the new continuous data collection function, Now it would seem to collect data correctly but even tonight the watering time was 0 despite the great heat.
config_entry-smart_irrigation-73ee0f4bb3c39b6f9c7c010022a9f909-5.json

I have upgraded to v2024.7.5. I activated the new continuous data collection function, Now it would seem to collect data correctly but even tonight the watering time was 0 despite the great heat. config_entry-smart_irrigation-73ee0f4bb3c39b6f9c7c010022a9f909-5.json

Try this:
https://github.com/jeroenterheerdt/HAsmartirrigation/issues/435#issuecomment-2227478512

I have everything selected correctly. I only use entities from the weather station

@cappa88 it looks like you are hitting a bug with the continuous data collection feature. can you enable debug logging (for a short period) and post your logs?

That's what I suspected.
I disabled the new feature and everything was working again.
I was also having trouble opening the integration interface.
I will do the test you asked me and I will provide you with the results.

.

It's can't parse the home assistant logs but it does collect the info from the home assistant sensors. So I am not sure what you mean exactly. Thanks for providing the files.

It can't parse the home assistant logs but it does collect the info from the home assistant sensors. So I am not sure what you mean exactly. Thanks for providing the files.

Did you receive the file? I thought I couldn't load it because it was larger than 2,5GB.

Sorry no, I am not seeing any files.

Here is the link where you can download the file.
unfortunately it always generates me a file larger than 2GB even if I do a debug of a few seconds
https://drive.google.com/file/d/1rM0QZxKeCFJl5HHt28dVnHMuN-H7FdBn/view?usp=sharing
I hope it's helpful
Now (11:00 p.m. local time) I launch another debug which I will stop tomorrow morning.
The calculation should take place at 2:50

https://drive.google.com/file/d/1EX_cFAm18hBHojyjmqV-YCe_PE_5EK7i/view?usp=drive_link
this is tonight's debug.
This time there were no crashes and the calculation was done even though the result seems a bit high to me
config_entry-smart_irrigation-73ee0f4bb3c39b6f9c7c010022a9f909.json
Another thing to report is that during the turn, while collecting data, I would see the bucket change even far from the time of calculation,
I hope my feedback is helpful to you

After working one day, it returned a time of 0 s last night.
As can be seen from the diagniostics (attached), a bucket of -6.8 was calculated and a time of 1268s for one zone and 441s for the other.
How is it then that the remained at 0?
Plus the interface seems to crash

config_entry-smart_irrigation-73ee0f4bb3c39b6f9c7c010022a9f909.json

After working one day, it returned a time of 0 s last night. As can be seen from the diagniostics (attached), a bucket of -6.8 was calculated and a time of 1268s for one zone and 441s for the other. How is it then that the remained at 0? Plus the interface seems to crash  config_entry-smart_irrigation-73ee0f4bb3c39b6f9c7c010022a9f909.json

so you still have continuous updates enabled - any chance you can post debug logs?

In these last few days I have disabled continuous collection and everything works fine.
However, I did not try again with the latest updates.
As soon as I try again I'll let you know

I still can't use continuous data collection.
The result of the calculation is always 0.
error_log-1.txt
config_entry-smart_irrigation-73ee0f4bb3c39b6f9c7c010022a9f909-8.json

thanks, I noticed this in the error log:
 File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/usr/src/homeassistant/homeassistant/__main__.py", line 223, in <module>
    sys.exit(main())
  File "/usr/src/homeassistant/homeassistant/__main__.py", line 209, in main
    exit_code = runner.run(runtime_conf)
  File "/usr/src/homeassistant/homeassistant/runner.py", line 190, in run
    return loop.run_until_complete(setup_and_run_hass(runtime_config))
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 674, in run_until_complete
    self.run_forever()
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 641, in run_forever
    self._run_once()
  File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1990, in _run_once
    handle._run()
  File "/usr/local/lib/python3.12/asyncio/events.py", line 88, in _run
    self._context.run(self._callback, *self._args)
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2122, in async_forward_entry_setup
    _report_non_awaited_platform_forwards(entry, "async_forward_entry_setup")
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 1175, in _report_non_awaited_platform_forwards
    report(
  File "/usr/src/homeassistant/homeassistant/helpers/frame.py", line 151, in report
    _LOGGER.warning(msg, stack_info=True)
2024-08-03 22:57:53.705 WARNING (MainThread) [homeassistant.helpers.template] Template variable warning: 'dict object' has no attribute 'battery' when rendering '{{ value_json.battery }}'
2024-08-03 22:57:53.706 WARNING (MainThread) [homeassistant.helpers.template] Template variable warning: 'dict object' has no attribute 'device_temperature' when rendering '{{ value_json.device_temperature }}'
2024-08-03 22:57:59.469 ERROR (MainThread) [homeassistant.helpers.event] Error while processing template: Template<template=({{states('sensor.velocita_vento')|float()*(4.87/log((67.8*3.3)-5.42))}}) renders=2>
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 2258, in forgiving_float_filter
    return float(value)
           ^^^^^^^^^^^^
ValueError: could not convert string to float: 'unavailable'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 604, in async_render
    render_result = _render_with_context(self.template, compiled, **kwargs)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 2639, in _render_with_context
    return template.render(**kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/jinja2/environment.py", line 1304, in render
    self.environment.handle_exception()
  File "/usr/local/lib/python3.12/site-packages/jinja2/environment.py", line 939, in handle_exception
    raise rewrite_traceback_stack(source=source)
  File "<template>", line 1, in top-level template code
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 2261, in forgiving_float_filter
    raise_no_default("float", value)
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 1872, in raise_no_default
    raise ValueError(
ValueError: Template error: float got invalid input 'unavailable' when rendering template '{{states('sensor.velocita_vento')|float()*(4.87/log((67.8*3.3)-5.42))}}' but no default was specified

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 716, in async_render_to_info
    render_info._result = self.async_render(  # noqa: SLF001
                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/template.py", line 606, in async_render
    raise TemplateError(err) from err
homeassistant.exceptions.TemplateError: ValueError: Template error: float got invalid input 'unavailable' when rendering template '{{states('sensor.velocita_vento')|float()*(4.87/log((67.8*3.3)-5.42))}}' but no default was specified
2024-08-03 22:57:59.476 ERROR (MainThread) [homeassistant.components.template.template_entity] TemplateError('ValueError: Template error: float got invalid input 'unavailable' when rendering template '{{states('sensor.velocita_vento')|float()*(4.87/log((67.8*3.3)-5.42))}}' but no default was specified') while processing template 'Template<template=({{states('sensor.velocita_vento')|float()*(4.87/log((67.8*3.3)-5.42))}}) renders=4>' for attribute '_attr_native_value' in entity 'sensor.vento_a_2m'

Seems like a template to convert the windspeed to 2m high is not working?
Also, it looks like your PyETO module has forecast days > 0 so any zones using that module will not be included in the continuous updates, as is shown in the log:
2024-08-03 23:01:42.549 INFO (MainThread) [custom_components.smart_irrigation] checked config for PyETO module on zone 3, forecast_days>0, skipping to avoid API calls that can incur costs.

Can you check that forecast days is indeed > 0?

If the conversion creates problems I can directly enter the entity of wind, it shouldn't change much

No, the issue is the forecast days, that needs to be 0.

It still doesn't work. Now the forecast days are at 0.
I also replaced (to be sure) the entity of the wind with the actual one of the weather station, without the cast at 2 meters.
The problem now is that the calculation is not done at the set time 2:50 AM
https://drive.google.com/file/d/1Jf4kDxvJELIANlf0YD9WFpbBtFTrH5lh/view?usp=drivesdk

It still doesn't work. Now the forecast days are at 0. I also replaced (to be sure) the entity of the wind with the actual one of the weather station, without the cast at 2 meters. The problem now is that the calculation is not done at the set time 2:50 AM https://drive.google.com/file/d/1Jf4kDxvJELIANlf0YD9WFpbBtFTrH5lh/view?usp=drivesdk

Remind me please, do you have continuous updates enabled now? If so, then of course the zones that can be continuously updated and calculated will not be included in the automatic calculation schedule. So if that's what your seeing that is expected.

Now I'm confused.
Yes, I'm trying to use continuous updates. I thought he still had to make the calculation at the appointed time.
How should it work then? The bucket always remains at 0 if the calculation is not done

continuous updates should update and calculate in real time.

Ok then my problem is that the bucket always remains at 0 despite the data being collected.

And you are still using continuous updates, right?

Yes. The supplement is always the continuous update, But the bucket never updates, It always remains at 0.
Now I have momentarily disabled the automation otherwise it would never irrigate. I'm using an automation with set times.

Here is my whole configuration

Some other users have had success with recreating the zone and sensor group. Not sure why that matters, but can you try to see if that helps at all?

I did this test. I also tried uninstalling the integration.
But nothing: the bucket never updates automatically.
Could I solve this by calling the "Calculate all zones" service at the beginning of the automation?

That should not be necessary so yes, that will be a solution but not what should happen. Please bear with me as I try to find the bug.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: cappa88 @.>
Sent: Saturday, August 10, 2024 10:18:52 AM
To: jeroenterheerdt/HAsmartirrigation @.>
Cc: Comment @.>; Assign @.>; Subscribed @.***>
Subject: Re: [jeroenterheerdt/HAsmartirrigation] 0 data collected (Issue #439)
I did this test. I also tried uninstalling the integration.
But nothing: the bucket never updates automatically.
Could I solve this by calling the "Calculate all zones" service at the beginning of the automation?
—
Reply to this email directly, view it on GitHubhttps://github.com/jeroenterheerdt/HAsmartirrigation/issues/439#issuecomment-2282215377 or unsubscribehttps://github.com/notifications/unsubscribe-auth/AB6PIPV63FJ64P27IQVNUF3ZQZDP3BFKMF2HI4TJMJ2XIZLTSWBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLAVFOZQWY5LFVI3TCOBXGU2TONRUGOSG4YLNMWUWQYLTL5WGCYTFNSBKK5TBNR2WLKRXGIZDGNJUGQ3DKMVENZQW2ZNJNBQXGX3MMFRGK3FMON2WE2TFMN2F65DZOBS2YSLTON2WKQ3PNVWWK3TUUZ2G64DJMNZZJAVEOR4XAZNKOJSXA33TNF2G64TZUV3GC3DVMWUTENRTGE3TSMJXG2BKI5DZOBS2K2LTON2WLJLWMFWHKZNKGI2DANRXGQ4DGMZUQKSHI6LQMWSWYYLCMVWKK5TBNR2WLKRXGE4DONJVG43DIM4CUR2HS4DFUVWGCYTFNSSXMYLMOVS2UNZSGIZTKNBUGY2TFJ3UOJUWOZ3FOKTGG4TFMF2GK.
You are receiving this email because you commented on the thread.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Ok. Thank you for the effort you put in.
So in the meantime I disable continuous updates, so I can use the integration

Since there have been a few updates I have tried re-enabling the continuous updates, but still nothing... it does not update the bucket in real time.
I was going to try uninstalling the integration to reset everything from the beginning, but I can't completely delete the data.
Once I reinstall from HACS I already find all the old configuration.
How can I do to reinstall the integration as if for the first time?

I attach debug and diagnostic files. I have also tried creating new test zones and new modules and sensor groups
config_entry-smart_irrigation-01J6HCVJYKBM47NG5NEBYYXGEG.json
https://drive.google.com/file/d/1va-rF4WJJBE1LbayoEOO1Rteei1jS6dW/view?usp=sharing


GEODE-1557: Not logging BucketNotFoundException

Not logging BucketNotFoundException as an error in LuceneEventLister and added
a test shouldIgnoreBucketNotFoundException to verify it

I was just curious as to know why manager was asked to throw a BucketNotFoundException but in the verification we are checking that logger was never called with an error method containing any exception parameter. Why is it not checking just for BucketNotFoundException but rather all Exceptions.
verify(log, never()).error(anyString(), any(Exception.class));
@upthewaterspout

+1


[kitchen] Check Win uninstall doesn't remove any system files

What does this PR do?
Adds a cookbook recipe to list system files before and after an
install/uninstall cycle, plus a spec test that compares both file lists.
Motivation
Sanity check to prevent damaging the system.
Additional Notes

I'm not really knowledgeable about Kitchen, Chef nor RSpec, so I might have placed my code in the wrong places. Maybe I'm not even testing what I think I am. Please review with extra care.

This test might be flaky if the test VM happens to be installing updates, removing temporary files, etc. during the Agent install. I've added an exclusion list for directories that can cause this sort of problems, but we might need to extend it.

It should be trivial to extend this to Linux. We can do so if this proves to work on Windows.

Got a first run in the pipeline. It failed with some files in c:/windows/SoftwareDistribution/DataStore/Logs/ and in some other places. This seems to be a Windows Update running. I can exclude the Logs but not the new files that it creates...


Login/logout

Modified the routes to use custom routes instead of the default ones provided by the resources keyword and added logic for logging in to the members controller. Also got rid of the session controller.

looks good, go ahead and merge


Implement naive per-project component search

This pull request implements a rudimentary component search service. The scope is fairly limited; the search only looks through the provided project and makes several assumptions (documented in code) about what it's looking for. However, our goal is to provide this service with as few false positives as possible rather than catch every case.

Hey all, I've made the request changes and added tests, but I'm not really sure if these tests are sufficiently comprehensive/can be more comprehensive. Any thoughts?


Feature request: Allow user to create/delete folders for OSF addons that support this (GitHub, Amazon S3, etc.)

See #259, #258 and #388

Folders were added with this pull request:
https://github.com/CenterForOpenScience/osf.io/pull/2327


Cache queries made in dashboard and VEO

writing to the cache

[ ] On dashboard page, when query is issued for cell, query results should be cached in redux store. cache is keyed by externs and query text.
[ ] On dashboard page, when VEO is visible, when query is issued in VEO, query results should be cached in redux store.  queries cached in VEO should 'expire' after a few minutes.

invalidating cache:

[ ] When user navigates away from a dashboard, the query cache should be invalidated/cleared so that when user navigates to the same dashboard again, cells are re-queried.
[ ] When a dashboard loads for the first time, all queries must query the database directly and not the cache
[ ] On dashboard Page, when viewing cells, the reload button should invalidate the cache and re-query for all cells.

reading from cache:

[ ] On dashboard page, when user clicks on configure to edit cell query, the VEO should load and display query results from the cache. (this is already done)
[ ] On dashboard page, when user clicks 'cancel' on VEO, background dashboard cells should attempt to query, but results should resolve from the cache, user should see dashboard as it was before they launched VEO even if they had selected a different time range or variable value in the VEO.
[ ] On dashboard page, when user clicks 'save' on VEO without having ever submitting the query in the VEO (by clicking submit or changing variable value/time range), background dashboard cells should attempt to query, but results should resolve from the cache, user should see dashboard as it was before they launched VEO.
[ ] On dashboard page, when VEO is visible, background dashboard cells should not attempt to query, even when time windows/variable values change.  (in PR)
[ ] On dashboard page, when user clicks 'save' on VEO, if user had resubmitted query in VEO, either by clicking submit, or changing query/variables/timeRange- all cells on dashboard should re-query. The edited cell should receive results from cache if it has not expired.

Here's the first issue I figured we could tackle:
#18929

Need to reset query's based on IDs when a query is submitted in the VEO

Need to invalidate the cached data after 5s

this should be easy to change.

Need to invalidate the cached data after 5s

this should be easy to change.

Famous last words 😃
Those were just mental notes that are already addressed in the PR linked to this:
#19010


Plane segmentation filters

Adds VTK filters and a python test.  Here is output of the test script on DRC trials terrain: