TextToImage Queries

1.) How to pass various parameters like a seed, output resolution, etc?
2.) Caching of responses, is this being worked upon?
3.) Changing the endpoint to https://api-inference.huggingface.co/models/stabilityai/stable-diffusion-2-1 and hitting generate produces the same result for the same string always.

I am encountering this as well... I'm trying to go through the api code to see if the seed is locked ... the seed is not provided in the example files.  I did convert the 2d example to 3d however :) put this script on a 3d object you want with a text to image texture and build an overlay ui to call the functions.
`using System;
using UnityEngine;
using System.Collections;
using HuggingFace.API;// Replace with the actual namespace for the HuggingFace API
public class ai_cube : MonoBehaviour
{
public string textToConvert = "a cat in a hat"; // The text you want to convert to an image
void Start()
{
    Debug.Log("starting");
}

public void setText(string Text)
{
    textToConvert = Text;
}

public void onButtonClick()
{
    HuggingFaceAPI.TextToImage(textToConvert, result =>
    {
        // Do something with the result, which in this case is a Texture2D
        Debug.Log(result);

        // Get the Material from the MeshRenderer attached to the same GameObject
        MeshRenderer meshRenderer = GetComponent<MeshRenderer>();

        // Ensure we have a valid MeshRenderer component
        if (meshRenderer != null)
        {
            // Create a new material instance and assign the generated texture to its main texture slot
            Material newMaterial = new Material(meshRenderer.sharedMaterial);
            newMaterial.mainTexture = result;

            // Assign the new material to the MeshRenderer
            meshRenderer.material = newMaterial;
        }
        else
        {
            Debug.LogWarning("MeshRenderer component not found on the GameObject.");
        }
    }, error =>
    {
        // Handle errors
        Debug.LogError(error);
    });
}

}
`
pair with a button and a text box with the ui system and trigger the 2 functions :)


Keyboard shortcuts

Using the arrow keys to navigate the options list doesn't seem to work, and I can't find a setting for it. Am I missing something? I'm happy to open a PR if this is something that's missing.

It should've been working only that it did not scrolled the dropdown div. That got fixed in 2.2.0. If you still have an issue please create an example and/or specify browser and OS.


Split codes to remove circluar import

Description
For removing the circular import, I moved types and GroupCheckboxContext to other files.
Checklist
Before you create this PR confirms that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] Run yarn test or yarn test -u if you need to update snapshot.
[x] Run yarn lint
[x] I am willing to follow-up on review comments in a timely manner.

@JongtaekChoi How does this remove circular import?

@hyochan
In the previous commit, GroupCheckboxContext was imported from CheckBox.tsx file, and CheckBox Component was imported from CheckBoxGroup.tsx File, so circular import occurred.
I moved GroupCheckboxContext into a new file (GroupCheckboxContext.ts) and set to import it from CheckBox.tsx and CheckBoxGroup.tsx files respectively.
And because the context required some types in CheckboxGroup.tsx file so I added another new file (types.tsx) and moved them to the file.


Monte Carlo based noise simulation

Describe the feature you'd like
A new simulator which implements Monte Carlo sampling to perform trajectory-based noise simulation. This would be shots>0 only but would allow qubit counts up to the maximum the state vector simulator supports.
How would this feature be used? Please describe.
Allows simulation of noisy circuits with >= 16 qubits.
Describe alternatives you've considered
We have already implemented the "exact" form of noise simulation, density matrix based noise simulation. This allows users to trade accuracy for higher qubit counts.
Additional context
There is another opportunity here to use threaded parallelism but this will require care when interacting with the random number generator for the Monte Carlo moves.

Hi, are you looking for a full simulator implementation, or is a wrapper over an existing library (such as QuantumOptics) enough?

You could also use the existing StateVectorSimulator in this package and extend it to use MC sampling.

is this issue still open ?


[Proposal] Contributing scratch re-implementation of the C++ SDK

Motivation
The current SDK C++ is faulty and not usable for a variety of reasons, both in its current mantainance state and its design as a whole. As a primer, the SDK is not up to date with the latest plugin API version 3.0.0, thus producing plugins that are not loadable by the latest Falco. Second, the design relies on virtual inheritance interfaces, which bring the following concerns (just to name a few):

Plugin capabilities are composable and implemented implicitly, which can only implemented by going through the diamond OOP problem, which is not handy nor optimal
The base interfaces rely on virtual function methods to define "traits" to be implemented, which is inherently a slow option. Virtual function are implemented by compilers through vtables, and invoking a virtual function can be ~10x-20x slower than an inlined function call depending on the situation. Now that plugins support the syscall event stream, this overhead caused by the SDK itself is not acceptable.
The current design hides some internal components that make use of dynamic casting at init time to check that the user-defined plugin type adheres to each of the capabilities interfaces. This is costly and error prone.

Proposal
I worked on re-implementing the SDK from scratch with a whole new design based on mixins, implicit interfaces based on type traits, RAII, and intensive function inlining. Although I hate to re-propose a design for the third time, I think these changes would be worth the effort. Plus, the current SDK is non usable, had no traction in the community, and is still a sandbox-level project in our organization, meaning that we consider it still in its early stages of development.
The progress of the big refactor can be checked on a branch of my fork: https://github.com/jasondellaluce/plugin-sdk-cpp/tree/refactor/performance-design-2
I invite anyone interested to take a look, as that will heavily help on understanding the technical approach.
Apprach
Goals

Plugins to be defined as simple structs/classes respecting type traits, one for each capability
Plugin capabilities can be implemented one-by-one in composition
All the machinery and the C type definitions of the falcosecurity/libs plugin API definitions should be hidden, everything should be covered by a zero-cost C++ abstraction (issues #12, #13)
When a type trait is not respected, the compiler is meant to provide guidance on the issue and the expected function signatures
The cost of the SDK should be close to zero, meaning that the end binary should almost be equivalent as implementing each plugin extern "C" symbol manually

Goal 1,2
Defining a plugin will look like:
#include <falcosecurity/sdk.h>

struct my_plugin
{
    virtual ~my_plugin() = default;
    std::string get_name() { return "cpptest"; }
    std::string get_version() { return "0.1.0"; }
    std::string get_description() { return "some description"; }
    std::string get_contact() { return "some contact"; }
    ...
    bool init(falcosecurity::init_input& i) { return true; }
    ...
};

FALCOSECURITY_PLUGIN(my_plugin);
FALCOSECURITY_PLUGIN_FIELD_EXTRACTION(my_plugin);
... // (other capabilities)

As visible, no virtual function or override marker is present. The SDK makes the heavy lift of figuring out if the type complies with the expected traits or not.
Goal 3
Users should not deal with the complexity of the C API and the event definitions of falcosecurity/libs. For example, here's how an event can be produced for the event sourcing capability:
falcosecurity::result_code next_event(falcosecurity::event_writer& evt)
{
    falcosecurity::events::pluginevent_e_encoder enc;
    enc.set_data((void*)"hello world", strlen("hello world") + 1);
    enc.encode(evt);
    return falcosecurity::result_code::SS_PLUGIN_SUCCESS;
}

Goal 4
If a type trait is not respected, and a function is either missing or has the wrong signature, the compiler is capable of detecting it through the static assertions of the SDK and will provide errors such as:
struct my_plugin
{
    ...
    const char* get_name() { return "cpptest"; }
    ...
}

# g++ -std=c++0x -fPIC -I../../include/ -shared -o libtest.so test.cpp
In file included from ../../include/falcosecurity/internal/plugin_mixin.h:21,
                 from ../../include/falcosecurity/internal/symbols_async.h:20,
                 from ../../include/falcosecurity/sdk.h:20,
                 from test.cpp:1:
...
test.cpp:250:1:   required from here
../../include/falcosecurity/internal/plugin_mixin_common.h:71:66: error: static assertion failed: expected signature: std::string get_name()
   71 |                                    decltype(&Plugin::get_name)>::value,
      |                                                                  ^~~~~
make: *** [Makefile:30: libtest.so] Error 1

Note how the SDK prompts an  error and suggests the right/missing function signature :
error: static assertion failed: expected signature: std::string get_name()
Goal 5
The new design makes use of widespread inlining wherever possible, and some hacks involing templates and macros to avoid any performance penalty.
For example, we want the following two examples to produce almost the same assembly output, to prove that the SDK adds no extra overhead:
Without the SDK
...
extern "C" uint32_t plugin_get_id() { return 999; };
...

With the SDK
struct my_plugin
{
    ...
    uint32_t get_id() { return 999; };
    ...
}
...

Compilation output with the SDK (x86_64)

0000000000005733 <plugin_get_id>:
    5733:	f3 0f 1e fa          	endbr64
    5737:	55                   	push   %rbp
    5738:	48 89 e5             	mov    %rsp,%rbp
    573b:	48 8d 05 7e 0b 04 00 	lea    0x40b7e(%rip),%rax        # 462c0 <_ZN13falcosecurity9_internalL17s_plugin_sourcingE>
    5742:	48 8d 15 77 0b 04 00 	lea    0x40b77(%rip),%rdx        # 462c0 <_ZN13falcosecurity9_internalL17s_plugin_sourcingE>
    5749:	48 89 55 e8          	mov    %rdx,-0x18(%rbp)
    574d:	48 89 45 f0          	mov    %rax,-0x10(%rbp)
    5751:	48 8b 45 f0          	mov    -0x10(%rbp),%rax
    5755:	48 89 45 f8          	mov    %rax,-0x8(%rbp)
    5759:	b8 e7 03 00 00       	mov    $0x3e7,%eax
    575e:	90                   	nop
    575f:	90                   	nop
    5760:	5d                   	pop    %rbp
    5761:	c3                   	ret

As visible, no virtual function call is involved and the function is as minimal as it gets, thus confirming that the compiler is capable of squashing most of the SDK abstractions. Of course, with more complex function symbols extra SDK work will be required, but this gives the idea.

cc @falcosecurity/core-maintainers, @falcosecurity/plugin-sdk-cpp-maintainers
What's everyone's feeling?

Big +1 for example if we want to follow through with approaches like https://github.com/falcosecurity/falco/pull/2655 and expose more sophisticated functionality over the plugins framework we urgently need this. Exposing syscalls hot path to plugins at least for me just marks the beginning.

+1 from me! Also, big thank you for doing this @jasondellaluce , our Plugins Master!

big +1 from me! :heart_eyes:


please increase docker rate limits

ERROR: 429 Too Many Requests - Server message: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit
https://www.docker.com/increase-rate-limits/
To increase your pull rate limits you can upgrade your account to a Docker Pro or Team subscription.   please tell me you subscribed :yum:

I can't communicate with docker websocket
I can't give permission to user to write file on server

This is fixed now


html should show word in tooltip when hovering over a dot with a mouse

Is it possible for you to post a screengrab of what's happening, let me know the browser/OS you're using?  The tooltips work for me in Chrome, Safari and Firefox, although it looks like Firefox has some slight word-list positioning issues.

I'm using latest Chrom on Mac. My guess is that I'm using python 2.7

the tooltips on browser are working on https://jasonkessler.github.io/Conventions-Visualization.html

There are stil some issues with Python 2.x-- the easiest thing may be to use python 3. If you'd still like to use 2, make sure you upload to the latest version of scattertext (<IP_ADDRESS>.2).

I tired re-running my code on py3 and continue to have same problems

That's odd.  The code is in the showTooltip function of main.js.  Could you please post (or email me) a copy of the HTML file that's being generated along with how your calling Scattertext?
I'm having trouble replicating the bug.

on python 3.6 I managed to get the tooltips when using produce_scattertext_explorer but not when using produce_scattertext_html
In python 2.7 I get an exception when calling produce_scattertext_explorer:
/Users/udi/anaconda2/lib/python2.7/site-packages/scattertext/init.pyc in produce_scattertext_explorer(corpus, category, category_name, not_category_name, protocol, pmi_filter_thresold, minimum_term_frequency, max_terms, filter_unigrams, height_in_pixels, width_in_pixels, max_snippets, max_docs_per_category, metadata, scores, singleScoreMode, sort_by_dist, reverse_sort_scores_for_not_category, use_full_doc, transform, jitter, grey_zero_scores, term_ranker, chinese_mode, use_non_text_features, show_characteristic, word_vec_use_p_vals, max_p_val, term_significance, save_svg_button)
226                                          max_p_val=max_p_val,
227 	                                 save_svg_button=save_svg_button)
--> 228                 .to_html(protocol=protocol)
229
230
/Users/udi/anaconda2/lib/python2.7/site-packages/scattertext/viz/HTMLVisualizationAssembly.pyc in to_html(self, protocol)
80 		javascript_to_insert = '\n'.join([
81                         self._full_content_of_javascript_files(),
---> 82                         self._visualization_data.to_javascript(),
83                         self._call_build_visualization_in_javascript()
84 		])
/Users/udi/anaconda2/lib/python2.7/site-packages/scattertext/viz/VizDataAdapter.pyc in to_javascript(self)
35         def to_javascript(self):
36                 starter = 'function getDataAndInfo() { return'
---> 37                 word_dict_json = json.dumps(self.word_dict)
38                 ender = '; }'
39                 return starter + word_dict_json + ender
/Users/udi/anaconda2/lib/python2.7/json/init.pyc in dumps(obj, skipkeys, ensure_ascii, check_circular, allow_nan, cls, indent, separators, encoding, default, sort_keys, **kw)
242         cls is None and indent is None and separators is None and
243         encoding == 'utf-8' and default is None and not sort_keys and not kw):
--> 244         return _default_encoder.encode(obj)
245     if cls is None:
246         cls = JSONEncoder
/Users/udi/anaconda2/lib/python2.7/json/encoder.pyc in encode(self, o)
205         # exceptions aren't as detailed.  The list call should be roughly
206         # equivalent to the PySequence_Fast that ''.join() would do.
--> 207         chunks = self.iterencode(o, _one_shot=True)
208         if not isinstance(chunks, (list, tuple)):
209             chunks = list(chunks)
/Users/udi/anaconda2/lib/python2.7/json/encoder.pyc in iterencode(self, o, _one_shot)
268                 self.key_separator, self.item_separator, self.sort_keys,
269                 self.skipkeys, _one_shot)
--> 270         return _iterencode(o, 0)
271
272 def _make_iterencode(markers, _default, _encoder, _indent, _floatstr,
/Users/udi/anaconda2/lib/python2.7/json/encoder.pyc in default(self, o)
182
183         """
--> 184         raise TypeError(repr(o) + " is not JSON serializable")
185
186     def encode(self, o):
TypeError: MY_UNICODE_TEXT is not JSON serializable

Thanks for finding the issue with produce_scattertext_html.  I'll push out a fix in the next version.
Unfortunately, I can't replicate the exception you're seeing.  If you can post how you called the function and the data you passed into it, then I'll be able to address the issue.

I re-run your notebook after changing the python engine from 3 to 2
and got the same exception on the imdb corpus

I reran the notebook in Python 3.6, and I can't replicate your error.
Which cell are you seeing the error on?  What is your environment?  Have you made any changes to the notebook?

Sorry for not being clear enough on last post. I took your original notebook and modified it to work on python 2.7 and then run it on python 2.7 and then on the cell in which you do
html = st.produce_scattertext_explorer(... for the first time I got the same error:
TypeError: smart and alert , thirteen conversations about one thing is a small gem . is not JSON serializable

Hmm.  It looks like Pandas' JSON writer is having trouble decoding string format.
I really don't envision spending much time supporting Python 2.  If you'd like, feel free to take a look at this in the code and submit a PR.

Since the discussion has drifted to a separate issue and the original bug was fixed, I'm going to close this issue.

I was able to replicate this in Python 2 last night and figured out the issue.  There's a fix coming in the next few minutes, but again, there may be a lot of Python 2 functionality that's broken.


Mandrill subaccounts

Codecov Report

Merging #33 into master will increase coverage by 0.39%.
The diff coverage is 94.87%.

@@            Coverage Diff             @@
##           master      #33      +/-   ##
==========================================
+ Coverage   84.31%   84.71%   +0.39%
==========================================
  Files          11       11
  Lines        1167     1191      +24
  Branches      111      115       +4
==========================================
+ Hits          984     1009      +25
+ Misses        154      153       -1
  Partials       29       29


SSH Git clone

Add option to choose between SSH or HTTPS when cloning a repository.
When I clone a repository in vscode, it clones using HTTPS.
So when I try to push the repository using the terminal it asks for my GitHub credentials because it doesn't get my SSH key.

@sandenbergmelo, could you please try the following setting - github.gitProtocol.

It worked! Thanks.


Migrate to multiple packages (workspace)

Fixes #734

PR Checklist

[x] Tests
[ ] Documentation
[x] Added entry to CHANGELOG.md
[ ] Tried the feature on a public network

[ ] The temporary Universal Sierra Compiler installation in the workflow should be removed before merging.
[ ] Depends on the snforge-migration branch.
[ ] Documentation is still missing.


Rebooting the bare-metal this is deployed on results in bizarre routing

Error description:
I'm using docker-compose in conjunction with this reverse proxy application to automate the deployment, routing, and encryption of a micro-service deployment. When I set this up all goes well. However, if I reboot the bare-metal server this is running on, the sub-domains will route to random containers. This error has haunted me for months and has made it so that I am unable to deploy this anywhere outside of a testing environment.
To fix this, I use docker-compose stop on this repository as well as on my own stack. Then I remove the data directory which this repository utilizes. Then, if I start the reverse proxy, wait a few seconds, and boot my own stack everything works fine again.
Steps to reproduce

Deploy this reverse proxy
Deploy your own stack using docker-compose
Reboot the server these are running on

Expected result

db.mydomain.com -> Routes to MySQL
pma.mydomain.com -> Routes to PHPMyAdmin
mydomain.com -> Routes to the main web container

etc.
Error result

db.mydomain.com -> Routes to one of the containers in the stack. (ie. maybe this now routes to the main web container)
pma.mydomain.com -> Routes to one of the containers in the stack. (ie. maybe this now routes to the MySQL DB)
mydomain.com -> Routes to one of the containers in the stack. (ie. maybe this now routes to the PHPMyAdmin)

Comments

Here are the modifications I made to the reverse proxy container: https://github.com/No-Patience/Reverse-Proxy
Here is the current deployment we use: https://github.com/No-Patience/No-Patience

I also noticed that when I reboot the server, the nginx-gen container does not come back up.

I've noticed sometimes not all the primary containers (nginx, docker-gen, and letsencrypt) start up as well. @ShawnClake, can you try using Docker's depends_on statement to see if it makes a difference for you?
A more advanced option is to wait for another container to be ready before starting another container.

Hi @j7an, which order should they be starting in? Is the order of the startup the cause of the issue I'm experiencing?

Thanks, I should have a few hours this weekend I can spend tinkering with this. I'll reply back once I know more.

Update: I re-deployed the app fresh and with that came an update to the nginx-web and nginx-gen containers. I've yet to reproduce this issue on that fresh deployment. I'm not sure whether that means this is resolved now or that I just haven't gotten unlucky yet.

@chrisckc
So, we better keep "always", right?

yes, but maybe add something in the docs about "unless-stopped" potentially not working as expected.
Always best to test it first to make sure that the containers will still start after a reboot if changing this setting.


HDDS-3612. Document details of bucket mount design

What changes were proposed in this pull request?
Add proposed implementation details to Ozone S3 volume mapping design doc.  The primary goal is to be able to discuss it before going forward with big code changes.
https://issues.apache.org/jira/browse/HDDS-3612
How was this patch tested?
Checked preview on GitHub.

Thanks @arp7 for reviewing this doc.  I have updated it with some clarifications.  Also changed to use "symbolic link" terminology instead of "mount" for consistency.  This changes the text of the original proposal very slightly, but keeps its spirit/intentions, I think.  Eg. propose to call the command ozone sh bucket link instead of ozone sh mount.

Thanks @arp7 and @elek for the review.


Updated issue link

Many to many is hands down the top issue on the EF Core sorted by most commented and most reacted.

Most commented
Most upvotes

Thanks @weitzhandler for your contribution.


adonis -h too slow

It's too slow to display the result everytime
I waited 5sec above for every command
Package version
adonis 4.1.0
Node.js and npm version
Node.js v10.8.0
npm 6.4.1
Sample Code (to reproduce the issue)
just type: adonis -h
adonis
adonis route:list
BONUS (a sample repo to reproduce the issue)
MAC air 13-inch 2015 early
RAM: 4GB
SSD: 128G

it would be nice if you add more context, because I had reproduce those three commands and I only wait 2 secs for each one
So if you have a group of routes or anything else please add them

1 sec on windows 10 and newest hardware (i5 8250U | 16gm ram)

Hey guys 👋
I'm currently working on the CLI and Ace to make it faster. 🚀

excuse me @RomainLanz do you need help? how I can contribute?

There's no need for help for the moment. Thanks for asking 👍

someone look at #1058


Changes for Hibernate Reactive

Fix HHH-15761
Make it possible for Hibernate Reactive to override, reuse some methods without copying them.
It also add a null check to avoid a NullPointerException (not sure if it's the right solution)

Fixed the checkstyle error, rebased and merged in main


can't ssh via bastion

uname -a: Darwin Davids-Work-MacBook-Pro.local 21.2.0 Darwin Kernel Version 21.2.0: Sun Nov 28 20:28:54 PST 2021; root:xnu-8019.61.5~1/RELEASE_X86_64 x86_64
az version
{
"azure-cli": "2.32.0",
"azure-cli-core": "2.32.0",
"azure-cli-telemetry": "1.0.6",
"extensions": {}
}
az network  bastion ssh --name xxx --resource-group es-connectivity-xxx --target-resource-id nic-dts-dev-kigx-vm-vmLinux-0 --auth-type ssh-key --username vmadmin --ssh-key /Users/xxx/Downloads/vmadmin_id_rsa
Command group 'network bastion' is in preview and under development. Reference and support levels: https://aka.ms/CLI_refstatus
The command failed with an unexpected error. Here is the traceback:
The extension ssh is not installed.
Traceback (most recent call last):
  File "/usr/local/Cellar/azure-cli/2.32.0/libexec/lib/python3.10/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/usr/local/Cellar/azure-cli/2.32.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 658, in execute
    raise ex
  File "/usr/local/Cellar/azure-cli/2.32.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 721, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/usr/local/Cellar/azure-cli/2.32.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 692, in _run_job
    result = cmd_copy(params)
  File "/usr/local/Cellar/azure-cli/2.32.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/usr/local/Cellar/azure-cli/2.32.0/libexec/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/usr/local/Cellar/azure-cli/2.32.0/libexec/lib/python3.10/site-packages/azure/cli/command_modules/network/custom.py", line 7995, in ssh_bastion_host
    _test_extension(SSH_EXTENSION_NAME)
  File "/usr/local/Cellar/azure-cli/2.32.0/libexec/lib/python3.10/site-packages/azure/cli/command_modules/network/custom.py", line 7928, in _test_extension
    ext = get_extension(extension_name)
  File "/usr/local/Cellar/azure-cli/2.32.0/libexec/lib/python3.10/site-packages/azure/cli/core/extension/__init__.py", line 340, in get_extension
    raise ExtensionNotInstalledException(ext_name)
azure.cli.core.extension.ExtensionNotInstalledException: The extension ssh is not installed.

route to service team

If you do a az extension add --name ssh it should install the extension for you

If you do a az extension add --name ssh it should install the extension for you

it is only supported on windows :( Mac support for a start would be great.

On Mac M1 running macOS 12.5.1 it looks good now:
% az version
{
  "azure-cli": "2.39.0",
  "azure-cli-core": "2.39.0",
  "azure-cli-telemetry": "1.0.6",
  "extensions": {
    "account": "0.2.3",
    "spring": "1.1.5",
    "ssh": "1.1.2"
  }
}

@yonzhan looks like it is supported. Can we close this?

It has worked for quite some time and can be closed

Reopening because the error message should be actionable i.e. it should provide the command to install the ssh extension. We can't expect customers to know what how to resolve "extension ssh is not installed"  @yonzhan


python310Packages.losant-rest: 1.16.1 -> 1.16.2

Description of changes
Update to latest upstream release 1.16.2

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 172854 run on x86_64-linux 1

2 packages built:

python310Packages.losant-rest
python39Packages.losant-rest


add support for vagrant snapshot command

in addition to sahara vagrant 1.8 now have built-in snapshot handling capabilities!

done in https://github.com/bitrise-tools/bitrise-machine/pull/3 - will be released in v0.9.9

we decided to replace sahara instead of adding this though - the rollback cleanup mode now uses vagrant snapshot pop --no-delete


[stable/traefik] Fix a typo in RBAC (secret -> secrets)

This PR follows https://github.com/kubernetes/charts/pull/1485 and fixes a tiny typo we missed.

@krancour @prydonius could you please have a look at this pr since it should be superquick? It changes just one character following #1485 so it'd be great if it was merged it before a rebase is needed :–)

@kachkaev, it looks like you were aware of this issue before #1485 was merged? I know you were eager for that to be merged, but I wish this had been addressed then and there so that we weren't knowingly introducing a bug.
At any rate, this LGTM and we should get this merged ASAP.
cc @prydonius @viglesiasce @lachie83

Hi @krancour! Unfortunately I only spotted the typo after the PR was merged and I updated my charts locally  – you can see this from the timestamp of my comment. The presence of secret instead of secrets does not break anything, it just does not fix the RBAC issue that's been around for a long while.
However, I agree with you that it'd be great if this tiny PR was merged ASAP.

Unfortunately I only spotted the typo when the PR was merged – you can see this from the timestamp of my comment.

Got it. I saw this mentioned in that thread, but failed to notice it was mentioned after the merge. Very sorry!
Thanks for the fix!

/ok-to-test

/ok-to-test

/lgtm


Don't stop propagation of click event if dnd is disabled

Don't stop propagation of click event if dnd is disabled. Fixes 131
Don't know how you typically generate the min file, but I've updated the non-minified file.

Thanks for contributing, will merge with next release

One problem I see: With nested lists, the dnd-selected callback would be fired on the disabled element, as well as any parent dnd-draggable element. I'm now thinking it might be better to only propagate the event if the callback returns true, for example.

Is there a way to limit dnd-selected callback to only that specific draggable item without silencing click events?  Maybe by looking for the click target's closest dnd-item?

Good idea actually, we can just set a property on the event object so that other handler along the way will know that we handled it already.


Reducer as alias fix

Ft.AGGREGATE fails when using reducer with alias name

Thank you for the proposed changes. Please correct all checks.

Done @mrniko please review

Thanks for contribution!


Error and attempted 404 after sending swagger spec

If you use a swagger_raw pipe to send out the swagger spec when mediated by connect_middleware, the request processing continues after the response has been sent.  This eventually leads to express's default 404 handler kicking in, which in turn produces an error when it attempts to set the status code header for a response that has already been sent.
Specifically, the commit ac6dad9fb5071bddd7a95c32cc38e9cc15c57228 needs to be reverted.  It's not clear what problem it's attempting to solve, so I can't say what the impact is, but whatever it's trying to do, this manner of doing it breaks express's contact.
Stack trace:
Error: Can't set headers after they are sent.
  at ServerResponse.OutgoingMessage.setHeader (http.js:690:11)
  at ServerResponse.res.setHeader (/development-source/web/node_modules/express/node_modules/connect/lib/patch.js:63:22)
  at next (/development-source/web/node_modules/express/node_modules/connect/lib/proto.js:156:13)
  at /development-source/web/node_modules/express/lib/application.js:121:9
  at next (/development-source/web/node_modules/express/node_modules/connect/lib/proto.js:129:23)
  at /development-source/web/node_modules/express/lib/application.js:121:9
  at next (/development-source/web/node_modules/express/node_modules/connect/lib/proto.js:129:23)
  at [object Object].finishConnect (/development-source/web/node_modules/swagger-node-runner/lib/connect_middleware.js:94:13)
<snip...>

Could you take a look, @osher ?

I should probably clarify that, from the client side, this may cause a connection reset.  We detected this when our test for our api spec endpoint returned ECONNRESET.  It's possible a smaller document might have gotten out before the server crashed (I'm not super familiar the the internal details of the express and http modules' buffering schemes).  For reference, our spec was 36k as yaml, 51k as json (endpoint serves json).

@chrisegner  - if you say that with a shorter yaml it works while with a heavier yaml it doesnt - it's a score for the buffer theory. I know that my end-to-end tests pass - however, they do work with a rather short yaml.
I'll be in the office tomorrow (I'm in GMT+2), and will be able to run some tests. I'll keep you updated with my findings as soon as I have any

@chrisegner  - man, I need help to reproduce.
I've constructed an e2e test with a huge doc (over 200K), and did not encounter the bug.
can you please help provide a reproduction scenario?
If it helps, you can see what I've done here:
https://github.com/theganyo/swagger-node-runner/pull/82
which is basically, raise a connect server with a document I inflated with a #definition/veryVeryBigCompoundType as a type with 200 string fields with a lorem-ipsum description
I could retrieve the doc both as yaml and as json

@chrisegner  - can you help me reproduce?

@chrisegner - what framework are you using? could it have any relation to Accept-Encoding: gzip? I may have found something, I need a little help from you to determine if what I found is relevant to your case or on another area

could also have to do with https://github.com/theganyo/swagger-node-runner/issues/87 since he is using  connect middleware

I think the intention of that commit should be to put a return statement  in  the if and remove else but the first didn't happen going to do a pull request for this. I hope it's getting released soon


Allowing DIBSTER to process donations.

I'd be able to process donations when Sarah and Dan are unavailable, but I can only do it with ticket proof.
If you disagree kindly comment it.

agreed.

would be cool if i could , as i have not much to do atm , same concept as dib mentioned.

I don't fully agree on this, yes, it always takes some time before Dan or Sarah check DBH's PayPal, but it is kinda easy to fake a payment if you don't have access to DBH's PayPal. Just wanted to point that out.

I don't fully agree on this, yes, it always takes some time before Dan or Sarah check DBH's PayPal, but it is kinda easy to fake a payment if you don't have access to DBH's PayPal. Just wanted to point that out.

I completely agree with this.

I don't fully agree on this, yes, it always takes some time before Dan or Sarah check DBH's PayPal, but it is kinda easy to fake a payment if you don't have access to DBH's PayPal. Just wanted to point that out.

I completely agree with this.

i agree with this that darker agrees with that that jeweetzelf doesn't agree fully.

I don't fully agree on this, yes, it always takes some time before Dan or Sarah check DBH's PayPal, but it is kinda easy to fake a payment if you don't have access to DBH's PayPal. Just wanted to point that out.

I completely agree with this.

i agree with this that darker agrees with that that jeweetzelf doesn't agree fully.

Nice.

I don't fully agree on this, yes, it always takes some time before Dan or Sarah check DBH's PayPal, but it is kinda easy to fake a payment if you don't have access to DBH's PayPal. Just wanted to point that out.

Fair response which is understandable, it seems some of the fellow staff members agree with this, I will close edit it out and close my PR in a couple hours.

I disagree, it’s not fair just to give one staff member a permission mods don’t even have.

I disagree, it’s not fair just to give one staff member a permission mods don’t even have.

Yeah, I'm editing out my commits and closing the PR.


Include a legend for crime type colors

The inclusion of the legend might clutter too much information on to the dashboard. The existence of the tooltip serves a similar purpose as the legend.


Unable to run Hive with a domain containing .local

I've been trying to self-host Hive on a network where the domain name has to contain .local.
Whilst it starts, I am unable to login/register as I get an error that the apiDomain doesn't have a correct value.
I believe it is related to this issue in Supertokens.

looks like it was resolved https://github.com/supertokens/supertokens-node/issues/823#issuecomment-2076659219


Add variable for jinja step related to "optional" feature

The #26 and #27 add optional feature on Deployment repo and Testspace Link.
The initialized template should have often section related to these features (in the Readme, in the workflows, ...).
To can remove easily such related variable must be sent to the jinja step to can keep or not such section.

thanks to the jinja2 v1.1.0 action, the new variable on jinja can be simple variable, and not env variable only.


Better docs to distinguish from TestEquality

See https://github.com/haskell/core-libraries-committee/issues/21. The important thing about GEq is that it is doing term and type equality. Likewise GCompare is doing value ordering.
This will be easier after https://github.com/haskellari/some/pull/21, as the quantified constraints bring the plain H98 classes "in scope", so we can less awkwardly write laws clarifying the situation.
(That PR is in turn blocked on https://github.com/haskell/core-libraries-committee/issues/10, but the current plain H98 class instances for Sum and Product are too pointlessly restrictive for this to work.)

I wouldn't hold my breath on #21 going in. Please, hold your horses.

Oh I know

If GEq f is doing term and type equality, then shouldn't TestEquality f, which is just type quality, be a superclass?

@bb010g if they would do different thing, I think it might be a good idea to keep them separate. Having types implement the function with a complicated type (suggesting they do a thing).

Typically, only singleton types should inhabit this class. In that case type argument equality coincides with term equality:

I'd follow that suggestion and only write TestEquality for singleton types. If GEq had TestEquality as a constraint,
that would force to write TestEquality instance for non-singleton types too, something people may not want to do.
TL;DR I'd keep the classes separate. I have to adjust the docs now as the @Ericson2314 change in base is released, so I can refer to it.

@phadej Right after that suggestion, it says:

Singleton types are not required, however, and so the latter two would-be laws are not in fact valid in general.

haskell/core-libraries-committee#21 explicitly accepted the "Weaker" option of ghc/ghc#7333, which means
instance TestEquality Tag where
    testEquality TagInt1 TagInt1 = Just Refl
    testEquality TagInt1 TagInt2 = Just Refl
    testEquality TagInt2 TagInt1 = Just Refl
    testEquality TagInt2 TagInt2 = Just Refl

is explicitly a legal instance. If people want a typeclass where type argument equality coincides with term equality, then they should use a typeclass where that is explicitly the case (i.e. not TestEquality).
I don't see why GEq should act as if the "Strongest" option for TestEquality was chosen by the committee when that isn't the case.

is explicitly a legal instance.

It's also the only legal instance for Tag.
That however is probably not the GEq instance you want. (GEq can act the same way, but then the == and defaultEq should probably agree as well).

I agree that you don't want that instance for GEq. I'd imagine GEq would still have a geq :: f a -> f b -> Maybe (a :~: b) that doesn't default to geq = testEquality? You would want TestEquality as a superclass constraint, though, as geq x y != None should entail geq x y == testEquality x y, and testEquality x y == None should entail testEquality x y == geq x y. geq differs from testEquality in that for some values x and y where testEquality x y == Just Refl, geq x y == None, whenever x and y are not equal at the term level.

@bb010g your analysis of TestEquality and GEq relationship seems correct, but I still disagree that either class should imply other.
As I said, it's very much the purpose of GEq to have instances for types like GEq, but forcing users to write phishy instance TestEquality Tag is IMO not nice. Yes, the laws of TestEquality allow to write TestEquality Tag, but I argue that it's not good idea. And base docs say Typically, only singleton types should inhabit this class.. I repeat, I don't want to force users of GEq to go against that recommendation.
I also think that if relation could be there, it should be GEq f => TestEquality f.
I won't add the dependency, and close this issue when I get to update the docs. There is no practical motivation to have GEq or TestEquality imply each other.
(3rd option is to add geq method to TestEquality, as I don't think there is a type which can implement only GEq or TestEquality but not the other - I might be wrong though, as I didn't think too hard about that).

@phadej I think you could implement TestEquality DynamicF where data DynamicF f where { DynamicF :: forall a. TypeRep (f a) -> f a -> DynamicF f; }, but not GEq DynamicF, as you don't have a witness of value equality for f a.

I repeat, I don't want to force users of GEq to go against that recommendation.

I'm curious what @Ericson2314's thoughts are here. This doesn't feel right to me; if this wording is making you feel unsure about what laws the class has, I would like to see it removed and, if necessary, TypeEquality's laws changed again to reflect how the class is going to be used. I personally don't see the point in having a typeclass like this in base that's going to be inconsistently implemented. Maybe there really just needs to be a typeclass in base where

Just Refl means the type params are equal, and the values are equal, and the class assume if the type params are equal the values must also be equal. In other words, the type is a singleton type when the type parameter is a closed term.

alongside a typeclass with weaker laws, so people know their options and can choose what's necessary without assuming TestEquality has stronger laws than it does.

There is no practical motivation to have GEq or TestEquality imply each other.

By that reasoning, is there a practical reason for Ord to imply Eq? Both compare and <= don't rely on ==.

I'm curious what @Ericson2314's thoughts are here.

It's irrelevant. CLC didn't wanted to restrict TestEquality to singletons only (which was an option). Then TestEquality couldn't be a super-class of GEq, and this case would been closed for good.

Ord and Eq

Ord explicitly mentions Eq in laws.

Antisymmetry
if x <= y && y <= x = True, then x == y = True

One can argue that GEq should have laws tying it to testEquality, as we discussed. But I disagree. These classes are different enough, so I consider better to make people think which one they need. I cannot think of example where people need both at the same time.

data Foo a where
  MkFoo :: (Nat -> Nat) -> Foo ()

is a simple example of a data type for which TestEquality Foo exists (and is trivial), and GEq Foo does not exist.
TestEquality is indeed logically a super class, but I am in no rush to add the superclass relationship because it would be a big breaking change and I think the "stylistic" argument that the purpose are rather different does hold water. But I could go either way on this.
Regardless:
geqFromTestEquality a b = do
  prf @ Refl <- testEquality
  guard $ a == b
  Just prf

is now a valid function and we should probably add at least that.

geqFromTestEquality a b = do
  prf @ Refl <- testEquality
  guard $ a == b
  Just prf

is "good", but OTOH we don't require that Eq and GEq are compatible. We probably should (re #21), but I want to wait until 9.6 hits the masses and we learn a bit more about QuantifiedConstraints in the wild.

I come up with an example, where one can have GEq but cannot have TestEquality.
Well, rather one can have GCoerce (#46 ) but not TestCoercion:
We can have GCoerce IORef as if IORefs are equal the types should be coercible, but we cannot test whether the types are coercible in TestCoercion sense:
If we have x :: IORef a and y :: IORef b, we cannot satisfy TestCoercion (x :: IORef a) (y :: IORef b) = Just Coercion ⟺ Coercible a b
Similarly if one makes IORef-like nominally represented type, it can have GEq, but no TestEquality.

Though TestCoercion doesn't have docs clarified as TestEquality, i think it's just an oversight to be fixed soon, what you think @Ericson2314?

Another, maybe even less contrived example is the instance for Sum
instance (GEq a, GEq b) => GEq (Sum a b) where
    geq (InL x) (InL y) = geq x y
    geq (InR x) (InR y) = geq x y
    geq _ _ = Nothing

But one cannot have
instance (TestEquality a, TestEquality b) => TestEquality (Sum a b) where

as e.g. testEquality (InL TagInt1) (InR TagInt2) should result in Just Refl, but it seems impossible to do that in general.

Frankly, it sounds like TestEquality might have made a mistake going with "if and only if"!

If and only if is fine for singleton types, in fact I makes TestEquality more useful in that case.


⚠ Bofrost.at has degraded performance

In 890d112, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1144 ms

Resolved: Bofrost.at performance has improved in fd1cbb4.


How to call openai's api in vanna when using vpn

Hello, I am deploying vanna locally, but I get a "Connection Error" error. Most likely, I am unable to call the openai api. I saw online that I can change the api_requestor.py file of the openai library, but I don't see this file in my files（My instructions are ‘ pip install 'vanna[chromadb,openai,postgres]'’）
How can I add an agent in Vanna? (Just like in the picture.)Or how can I connect to openai’s api?thank you for your reply

hi，are you solve it?


Setting default flavour to caasp

Since the openSUSE flavour does not currently work, it makes more sense
for the default in this repo to be the working, supported platform.

Took the 1.0 out

I thought about that, but since it doesn't work right now, I didn't want to suggest it as an option.

While it doesn't work right now I'd say we still want to document it. The idea is that it should be possible to use opensuse, despite we aren't centering our efforts on it right now...

not sure when we decided this, but i disagree with making caasp the main flavor. when i read @cyphar's email he was suggesting to get into the habit of treating openSUSE as upstream to our project and not the other way around.
this defies the whole purpose of making it open source. besides https://github.com/kubic-project/terraform/issues/42 being fixed, what else is needed to make openSUSE work?

IMHO it's not about making openSUSE work, it's about having to maintain different scenarios. When the Beta 2 release was in place lots of things changed because we were developing with opensuse and then we changed stuff to work on caasp. After this opensuse was broken for a week (and it still is).
Don't get me wrong, I want to be able to support opensuse (of course, why not?). Given the humanpower we got we need to center our efforts on something specific, and given that it is a good idea to develop as closer as possible to production we thought we could center our development efforts on caasp, but keeping an eye on opensuse. This basically means: develop on a daily basis with caasp (close to production), and test that opensuse X times a week/whatever. I guess that once that we have been able to remove lots of divergencies between opensuse and caasp it will make sense to revisit this issue... Until then I think it's just too much to try to develop on opensuse without expecting surprises when everything gets baked for caasp on further releases.

We have migrated public/private scheme to production manifests.
We will remove manifests from velum [I am working on this ATM], and use the production ones with some substitutions.
OpenSUSE will use docker hub containers (opensuse based), while caasp uses sles12 docker images (delivered by container-feeder now, by our own registry in the future).
https://github.com/kubic-project/terraform/issues/42 as you said
Salt states are OS agnostic (pkg.installed will just do nothing on caasp if the package is already installed, will install it on opensuse)
Probably lots of other small details

Just my 2 cents. It makes sense for now, and nothing makes me more sad that remove the open source default; but I think it makes sense until we can revisit this and we have been able to remove some divergences between caasp and opensuse.

This wasn't my favorite change to make either, but it aligns with our desire to do MicroOS first, openSUSE next for the short-term. I was attempting to lower the barrier for us to run and contribute.
I'd love to leave openSUSE the default, but we need to put the effort into making it viable.

We had a meeting & discussion about this and we stated that our main priority is CaaSP. It would be good to support openSUSE but we will not spend time doing that: if the community wants to do all the changes necessary for running this on openSUSE it will be fantastic and it will have our full support, but it is a task that we cannot do by ourselves.


Azure Bot Framework Web Chat in Nodejs: Not able to recognize prompts while converting text to speech

Below is my code for one of the waterfall steps in webchat code in Nodejs. I am able to enable voice option in webchat but while it converts from text to speech it is not able to recognize prompts, though it is able to recognise simple text. Please help me in enabling speech option for prompts as well.
Also added code for html file where speech option is enabled.
Solution needed : In text to speech it should be able to recognize prompts also, currently it is able to recognise the text messages.
Waterfall Prompt:
async financeFirstStep(step) {
    console.log("Inside financeFirstStep::");
    return await step.prompt(CHOICE_PROMPT, {
        prompt: 'Please select a option to which your query is related to-',
        size:'Medium',
        choices: ChoiceFactory.toChoices(['Account Statement', 'Credit Limit','Credit Term','Status of Claim','Home']),
        style: ListStyle.heroCard,
        retryPrompt: 'Invalid value entered. \r\nPlease select from the available options-'
    });
}

HTML Code:
<!DOCTYPE html>

Currently, we do not limit to recognize only texts in a specific range of texts (i.e. in the prompts field).
Generally, bot should be designed to adapt to prompts that is outside of a specific list of model answers. You can consider using the Language Understanding module to provide better adaptation.

Closing due to inactivity.


Pinch to Zoom stops rendering

I'm using the Hammer.js library to provide touch events to the PDF.js viewer.  However when the zoom on mobile devices reaches greater than 110% percent any text in PDF is removed.  All that remains are white pages and certain elements (i.e a link that was highlighted still has the blue border around it)

I have worked with Hammer.js too to try to implement pinch to zoom, but it would transition waty too fast, causing a lot of re-rendering and performance issues. The pages can become white if re-rendering fails, which is a problem I also encountered. It seems like Hammer.js is just not the right tool for the job. Closing as answered, otherwise try the mailing list.


Rename the environment variable to DATABASE

Fixes #15

Good work @geektimus
I will wait for another review before merging this in. 😄
cc @icyflame @themousepotato @Ayushk4 @thealphadollar

@icyflame The testing (primarily handler tests) is rather naive and preliminary at the moment.
I'm still reading about the best practices for testing APIs with DB interactions.
Having said that, I agree with you that the variables shouldn't be hardcoded for testing.
We will most definitely use envconfig in tests as well. 😅


VSCode: Bad / Better intellisense

Hi.
I'm having a bit of a problem with bad hinting in Vue.js files.
For example, I write a property, and I want it to hint a list of all available datatypes (I know roughly which ones they are, but it doesn't hint anything, or shows a nonsensical list)
Video: http://remotefiles.rjwebdesign.cz/f/20230926-161339.mp4
Does it not do that or is something wrong with me?
It's related, but all hints, I have to help him a lot and write almost all of it for him (he could do this and detect it no?

Do you mean: only provide valid vue datatypes, such as String, Boolean?

Because technically we can pass any value to it, but vue will warn in your runtime code.

ok thx


Refactor remote.Resolver interface and implementation

Changes
The existing implementation has been refactored to better integrate with
the existing logic in the Reconcilers. (This work was started in
https://github.com/tektoncd/pipeline/pull/2233 and pulled out to here).
Included in this is a move for the interface to implement GetTask
which is what gets passed into the TaskRun and Pipeline reconciler and
require less changes in the embedding of the task itself into the pod
spec. A couple of test helpers have been modified and a new remote test
helper is added to make pushing to images in a test easier. Tests are
included for both.
Signed-off-by: Sunil Thaha<EMAIL_ADDRESS>Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Includes tests (if functionality changed/added)
[🙅♂  ] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
Describe any user facing changes here, or delete this block.

Examples of user facing changes:
- API changes
- Bug fixes
- Any changes in behavior

/close
Closing in favor of #2395

oh :sweat:
/close


invoke:

A script with;
  run-tests:
    - invoke: run-test
        WITH:
          REPO: myRepo

state variables are not being populated in run-test script ${{REPO}} is null

Didn't see this open issue until now.
I added invoke_with_state_script_name
it should test invoke: scriptName and script: ${{FOO}} the problem may have been that I commented out the invoke from the waml parser because invoke and script are both the same ScriptCmd implementation. I added invoke back to the parser but if we do not think of how they should differ I vote we eliminate one


color and PMTiles utils

and first version of maplibre express
closes #95

@crazycapivara We're enjoying using this from the branch but would love to see this on pypi.  Really appreciate the fantastic work in this cool package.

@cboettig Yep. You are right. Need to merge. I will do it in the next days :-)

:tada: :tada: Thank you very much, this is fantastic!


Ambiguous match when it shouldn't be.

Hi,
I've got a feature that creates a table:
Given the standard business:
      | field_type_of_business | Customer      |
      | field_billable         | YES           |

And the following two (relevant) steps
/**
   *
   * Creates a :business_type business node.
   * Convenience step which simply adds the value of $business_type to
   * the field field_type_of_business.
   *
   * @param string $business_type
   *   Any valid machine name value for the field 'field_type_of_business'
   *   on the business content type.
   *
   * @param  TableNode|null $table
   *   An optional table node containing data.  Assigns defaults set in
   *   BusinessOptions if not provided.
   *
   * @return object
   *   The drupal node of type business.
   *
   * Steps:
   * @Given a/the :business_type business:
   *  | field_billable         | YES           |
   */
  public function theTypeBusiness($business_type, TableNode $table=NULL){
    $values = self::convertTableNodeToArray($table);
    $values['field_type_of_business'] = $business_type;
    $node = $this->theBusiness($values);
    return $node;
  }
/**
   * Creates a standard business node.
   *
   * @param  TableNode|null $table
   *   An optional table node containing data.  Assigns defaults set in
   *   BusinessOptions if not provided.
   *
   * @return object
   *   The drupal node of type business.
   *
   * Steps:
   * @Given a/the standard business:
   *    | field_billable         | YES           |
   */
  public function theStandardBusiness(TableNode $table=NULL){
    $values = self::convertTableNodeToArray($table);
    $node = $this->theBusiness($values);
    return $node;
  }

When I run the scenario, though, I get an ambiguous match:
        Ambiguous match of "the standard business:":
        to `a/the :business_type business:` from Drupal\z_contacts\Tests\BusinessContext::theTypeBusiness()
        to `a/the standard business:` from Drupal\z_contacts\Tests\BusinessContext::theStandardBusiness()

I would have thought this was not ambiguous at all - one accepts a variable, one does not, and they are distinguished in the feature by "standard" being in quotes or not.  This seems like a bug, unless I'm overlooking something obvious.

Never mind, I see upon further experimentation that there is less distinction between the two types than I thought. I leave my solution in case anyone else is interested.
I was able to consolidate my tests using the regex matching step:
/**
* @Given /(?:a|the) ([^\s]+) business:?/
*/
public function theTypeBusiness($business_type, TableNode $table=NULL){
}

The syntax of which obviates the need for quoted arguments.


Add Zmanim (Jewish Prayer Times)

Add Zmanim App
This app displays Jewish prayer times (zmanim) based on user location:

Customizable by ZIP code
Shows all important daily prayer times
Clear display with highlighted times
Vertical scrolling display
Data from Chabad.org API

Tested locally and verified working with multiple ZIP codes.

I have read the CLA Document and I hereby sign the CLA

review

Hey. Please remove the .star file in the root of the repo. All app files need to live under apps/.

@matslina done


Support for output data schema

What would you like to be added:
Currently when defining a new workflow one can specify dataInputSchema which we can use to validate the input. Yet a user starting a workflow instance does not know what the output of this workflow is. What is the expected to get when the workflow finishes? This is primarily useful for HTTP requests.
I propose in the same we have dataInputSchema to have a dataOutputSchema which accepts a JSON schema.
Why is this needed:
This will make it easier to document the outputs of workflow for example in an OpenAPI document as the response from the workflow runtime. This makes it especially useful in a microservice architecture where someone can look at the OpenAPI and know what to expect.
I'm not sure how this fits in with output CloudEvents? Ideally, this output doesn't have to be a CloudEvent.

@EnriqueL8 This is a good suggestion IMO. However, we should decide if that's for informational purpose only or not.
As a matter of fact, such a schema could be used to verify that the output of a workflow instance matches what's expected just before a successfull completion.

Good point, I hadn't thought of it as a validation step but now that you say it it does feel natural for it to be.

I vote for same functionality than dataInputSchema, thats it, by default the output is validated againts the output shema if present

Yet a user starting a workflow instance does not know what the output of this workflow is. What is the expected to get when the workflow finishes?

A JSON result (state data output of last workflow state) or a failure (thats dependent on the runtime).

This is primarily useful for HTTP requests.

If you are exposing workflows from rest endpoint, think you could describe your workflow using OpenAPI which does include response schema. Similarly for event-triggered workflow you can use AsyncApi.
Trying to figure out what's the purpose of this request as if we add this it can go pretty deep into what all can be added that other frameworks have already.

@tsurdilo I agree with your concerns, but for these use cases, you just avoid specifying an output schema. I mean, the same than inputSchema, the new field will be optional.

I'm gonna have a spec binge next week. Should I work on that PR too guys? @tsurdilo @ricardozanini @fjtirado
I'm a big fan of the idea, personally speaking!


compile the code snippets in tutorial/api-type.(md/lhs)

Still working on converting all the files. The testing machinery is ready for review.

I realize that in the tutorial there's lots of code quoted from the servant source. That of course clashes with the things imported from servant. Not sure how to deal with that nicely. For now I just keep it as unchecked code blobs in there.

I'm wondering whether all imports in the tutorial should have explicit import lists. Any opinions?
(@jkarni, @alpmestan, @fizruk)

I'd say so, but we can do that as part of the 0.5 rewrite (or separately altogether) rather than as part of this.

I am not strongly inclined to having explicit imports.
I also agree than we can clean this up later.

+1, let's worry about that later.

LGTM


RuntimeError: "addmm_impl_cpu_" not implemented for 'Half'

When I try to run the model I have:
RuntimeError: "addmm_impl_cpu_" not implemented for 'Half'
which should mean that the model is on cpu and thus it doesn't support half precision.
However, I have cuda and the device is cuda at least for the model loaded with LlamaForCausalLM, but the one loaded with PeftModel is in cpu, not sure if this is related the issue. (I added offload_folder because it was required):
model = LlamaForCausalLM.from_pretrained(
            base_model,
            load_in_8bit=load_8bit,
            torch_dtype=torch.float16,
            device_map="auto",
            offload_folder=offload_folder
        )
print(model.device) # model is in cuda

model = PeftModel.from_pretrained(
            model,
            lora_weights,
            torch_dtype=torch.float16,
            offload_folder=offload_folder
        )
print(model.device) # model is in cpu

any idea to fix the issue?
with float32 it works but it's super slow.
I don't wont to use 8 bits, I also had issues with bitsandbytes for gpu, I remove it for the moment. (but this is another story)
full error:
Traceback (most recent call last):
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/gradio/routes.py", line 393, in run_predict
    output = await app.get_blocks().process_api(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/gradio/blocks.py", line 1108, in process_api
    result = await self.call_function(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/gradio/blocks.py", line 929, in call_function
    prediction = await anyio.to_thread.run_sync(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/anyio/to_thread.py", line 31, in run_sync
    return await get_asynclib().run_sync_in_worker_thread(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 937, in run_sync_in_worker_thread
    return await future
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 867, in run
    result = context.run(func, *args)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/gradio/utils.py", line 490, in async_iteration
    return next(iterator)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/gradio/interface.py", line 621, in fn
    for output in self.fn(*args):
  File "/home/davide/code/alpaca-lora/generate.py", line 159, in evaluate
    generation_output = model.generate(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/peft/peft_model.py", line 716, in generate
    outputs = self.base_model.generate(**kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/transformers/generation/utils.py", line 1524, in generate
    return self.beam_search(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/transformers/generation/utils.py", line 2810, in beam_search
    outputs = self(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
    output = old_forward(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 687, in forward
    outputs = self.model(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
    output = old_forward(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 577, in forward
    layer_outputs = decoder_layer(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
    output = old_forward(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 292, in forward
    hidden_states, self_attn_weights, present_key_value = self.self_attn(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
    output = old_forward(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 196, in forward
    query_states = self.q_proj(hidden_states).view(bsz, q_len, self.num_heads, self.head_dim).transpose(1, 2)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
    output = old_forward(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/peft/tuners/lora.py", line 466, in forward
    result = F.linear(x, transpose(self.weight, self.fan_in_fan_out), bias=self.bias)
RuntimeError: "addmm_impl_cpu_" not implemented for 'Half'

Try putting with torch.autocast("cuda"): at the start of your evaluate function
def generate_response(
instruction,
inputs=None,
temperature=0.7,
top_p=0.75,
top_k=40,
num_beams=4,
max_new_tokens=128,
**kwargs,
):
prompt = prompter.generate_prompt(history, context)
with torch.autocast("cuda"): #Useful if you receive the scalar error.
    inputs = tokenizer(prompt, return_tensors="pt")
    input_ids = inputs["input_ids"].to(device)
    generation_config = GenerationConfig(
        temperature=temperature,
        top_p=top_p,
        top_k=top_k,
        num_beams=num_beams,
        length_penalty=length_penalty,
        **kwargs,
    )
    with torch.no_grad():
        generation_output = model.generate(
            input_ids=input_ids,
            generation_config=generation_config,
            return_dict_in_generate=True,
            output_scores=True,
            max_new_tokens=max_new_tokens,
    )

Hmm, I'm not sure then peft_model is just a wrapper for LoRA though, it should use the same mechanism for both

@ElleLeonne can you tell me the version of the packages used? like torch or other relevant?

@ElleLeonne I am trying to run generate.py code in cpu but i am getting same error
I change float16 to float32
Can you please help

is PeftModel required? without it I have a cuda out of memory but at least I don't have that bug

@davidenitti could you upload that code without peftmodel that is working for you

@davidenitti could you upload that code without peftmodel that is working for you

it was only a test because the peftmodel is necessary to load the alpaca-lora model, however I managed to fix it in someway upgrading transformers and peft.
first uninstall both of them then use
pip install git+https://github.com/huggingface/peft.git
pip install git+https://github.com/huggingface/transformers.git

for the code I use generate.py changing this part below to make sure that it doesn't go cuda out of memory (you can change it to float32 to make sure it works (change the max memory if you have more or less GPU memory):
dtype = torch.float16
    if device == "cuda":
        model = LlamaForCausalLM.from_pretrained(
            base_model,
            load_in_8bit=load_8bit,
            torch_dtype=dtype,
            device_map="auto",
            offload_folder=offload_folder,
            max_memory={0: "5GiB"}
        )
        model = PeftModel.from_pretrained(
            model,
            lora_weights,
            torch_dtype=dtype,
            device_map="auto",
            offload_folder=offload_folder,
            max_memory={0: "5GiB"}
        )

I ran into the same error message when launching.  In my case, I commented out this section in generate.py since I'm purely on a CPU.
# if not load_8bit:
        # model.half()  # seems to fix bugs for some users.

I ran into the same error message when launching. In my case, I commented out this section in generate.py since I'm purely on a CPU.
# if not load_8bit:
        # model.half()  # seems to fix bugs for some users.

If you're using the latest version of PEFT, this line of code is actually completely redundant now anyways

@davidenitti could you upload that code without peftmodel that is working for you

it was only a test because the peftmodel is necessary to load the alpaca-lora model, however I managed to fix it in someway upgrading transformers and peft. first uninstall both of them then use
pip install git+https://github.com/huggingface/peft.git
pip install git+https://github.com/huggingface/transformers.git

for the code I use generate.py changing this part below to make sure that it doesn't go cuda out of memory (you can change it to float32 to make sure it works (change the max memory if you have more or less GPU memory):
dtype = torch.float16
    if device == "cuda":
        model = LlamaForCausalLM.from_pretrained(
            base_model,
            load_in_8bit=load_8bit,
            torch_dtype=dtype,
            device_map="auto",
            offload_folder=offload_folder,
            max_memory={0: "5GiB"}
        )
        model = PeftModel.from_pretrained(
            model,
            lora_weights,
            torch_dtype=dtype,
            device_map="auto",
            offload_folder=offload_folder,
            max_memory={0: "5GiB"}
        )

Thanks ,it works for me.

alguien pudo solucionarlo

Cuando intento ejecutar el modelo tengo: RuntimeError: "addmm_impl_cpu_" no implementado para 'Half'
lo que debería significar que el modelo está en la CPU y, por lo tanto, no admite la mitad de precisión. Sin embargo, tengo cuda y el dispositivo es cuda al menos para el modelo cargado con LlamaForCausalLM, pero el que está cargado con PeftModel está en la CPU, no estoy seguro si esto está relacionado con el problema. (Agregué offload_folder porque era necesario):
model = LlamaForCausalLM.from_pretrained(
            base_model,
            load_in_8bit=load_8bit,
            torch_dtype=torch.float16,
            device_map="auto",
            offload_folder=offload_folder
        )
print(model.device) # model is in cuda

model = PeftModel.from_pretrained(
            model,
            lora_weights,
            torch_dtype=torch.float16,
            offload_folder=offload_folder
        )
print(model.device) # model is in cpu

¿Alguna idea para solucionar el problema? con float32 funciona pero es súper lento. No suelo usar 8 bits, también tuve problemas con bitsandbytes para gpu, lo elimino por el momento. (Pero esta es otra historia)
error completo:
Traceback (most recent call last):
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/gradio/routes.py", line 393, in run_predict
    output = await app.get_blocks().process_api(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/gradio/blocks.py", line 1108, in process_api
    result = await self.call_function(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/gradio/blocks.py", line 929, in call_function
    prediction = await anyio.to_thread.run_sync(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/anyio/to_thread.py", line 31, in run_sync
    return await get_asynclib().run_sync_in_worker_thread(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 937, in run_sync_in_worker_thread
    return await future
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/anyio/_backends/_asyncio.py", line 867, in run
    result = context.run(func, *args)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/gradio/utils.py", line 490, in async_iteration
    return next(iterator)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/gradio/interface.py", line 621, in fn
    for output in self.fn(*args):
  File "/home/davide/code/alpaca-lora/generate.py", line 159, in evaluate
    generation_output = model.generate(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/peft/peft_model.py", line 716, in generate
    outputs = self.base_model.generate(**kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/transformers/generation/utils.py", line 1524, in generate
    return self.beam_search(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/transformers/generation/utils.py", line 2810, in beam_search
    outputs = self(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
    output = old_forward(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 687, in forward
    outputs = self.model(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
    output = old_forward(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 577, in forward
    layer_outputs = decoder_layer(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
    output = old_forward(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 292, in forward
    hidden_states, self_attn_weights, present_key_value = self.self_attn(
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
    output = old_forward(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/transformers/models/llama/modeling_llama.py", line 196, in forward
    query_states = self.q_proj(hidden_states).view(bsz, q_len, self.num_heads, self.head_dim).transpose(1, 2)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/accelerate/hooks.py", line 165, in new_forward
    output = old_forward(*args, **kwargs)
  File "/home/davide/code/alpaca-lora/venv/lib/python3.10/site-packages/peft/tuners/lora.py", line 466, in forward
    result = F.linear(x, transpose(self.weight, self.fan_in_fan_out), bias=self.bias)
RuntimeError: "addmm_impl_cpu_" not implemented for 'Half'

RESOLVISTE TU PROBLEMA?, can you share your solution ?

i have the same issue,RuntimeError: "addmm_impl_cpu_" not implemented for 'Half',what is finally solution, need your advise.

I have successfully solved this problem by way of
Start the webUI with the following Mac OS Terminal command.
cd stable-diffusion-webui ./webui.sh --precision full --no-half
Special Thank >> https://stable-diffusion-art.com/install-mac/comment-page-1/
https://stackoverflow.com/questions/75641074/i-run-stable-diffusion-its-wrong-runtimeerror-layernormkernelimpl-not-implem/75859574#75859574?newreg=4188d469fd6b411b9837743e51b57a78


added commit by Raunakk02

added instruction for entering the array elements in ascending order only.

Thanks, @Raunakk02  for the contribution


Goals

High level goals for the library
### Goals
- [ ] https://github.com/gulfofmaine/dataset_catalog/issues/3
- [ ] https://github.com/gulfofmaine/dataset_catalog/issues/14
- [ ] Use Pandas dataframes and Xarray datasets to template ERDDAP dataset fragments
- [ ] Generate full datasets.xml and support round triping including persisting comments?
- [ ] Support linting of individual datasets/full catalogs (integrate with IOOS QC)
- [ ] Enable similar usage with THREDDS catalogs
- [ ] Support translation between catalog types (maybe including Intake and STAC?)

My personal goals behind this aren't just for ERDDAP datasets.xml wrangling, but I'd like to build an Xpublish plugin that can load datasets from an ERDDAP catalog, thus making it really useful to run Xpublish alongside an existing ERDDAP server.
Also to come up with a better name as dataset_catalog is overly generic.


AnimateLeft and AnimateTop are both false

https://github.com/nolimits4web/dom7/blob/28b0db30d77fa765ccae9497a2f0e3dfbc2cf55f/src/scroll.js#L76
Hello,
We have seen an issue where us the animateTop and animateLeft are both false and the done is never set to true, this causes the render loop to run forever and stops the app/freezes
Could you add this at the end to prevent this
  if (!animateLeft && !animateTop) {
    done = true;
  }

Have same problem caused by card.close() on framework7


Update printers.md

the inner printer of Sunmi v1s not found in a list

Have you tried this package for Sunmi printers ?
https://github.com/andrey-ushakov/flutter_sunmi_printer

Have you tried this package for Sunmi printers ?
https://github.com/andrey-ushakov/flutter_sunmi_printer

Thank you very much. I have tried it. it worked for sunmi v1s !!. :)
appreciate !


Update command parsing to detect package version to be installed

Description
Each Linux package manager specifies how to install a particular version of a package differently. For example, when installing a package on a Debian/Ubuntu system, you would run sudo apt install package=version.release and if you were using Alpine you would similarly run apk add packagename=1.2.3-suffix. If you were using a Photon/CentOS/Fedora package manager like tdnf or rpm you would use rpm -i package-version.release.
This parsing works towards the Dockerfile "freeze" functionality. The reason we want to be able to detect package versions that are specified is so we can automatically report the package version in the "frozen" Dockerfile without having to do the work of looking for it in the built container.
To Do

Do some research to compare the python shlex and getopt library to see if there is a good way to use these for this bug.
Add a section under each package manager in tern/command_lib/base.yml that denotes how the package manger specifies package versions for install (i.e. - or =)
Update the parse_command function in tern/utils/general to recognize and record package versions when they are present after an install command.

Super Issues
#454

This should also fix #2 - a really old bug :)

Fixed in a4c107f447621defecfc227d31d043e2c3a91326


Add utility to list JSX event handlers

Provides a list of common JSX event handlers as a flat list and grouped into types.

I guess there are eslint issues in the build, but otherwise LGTM. Can you also update README? thanks!

Coverage increased (+0.03%) to 98.788% when pulling 564833fb6b7d9bda1a9aadd5e6dfd5d66db776a2 on jessebeach:synthetic-events into f617f70967c07f3d99952b9f70cee63a61925db9 on evcohen:master.

Coverage increased (+0.02%) to 98.78% when pulling 6086bc1ffd09fa04203120e40355187b0cc170d7 on jessebeach:synthetic-events into f617f70967c07f3d99952b9f70cee63a61925db9 on evcohen:master.

@evcohen gtg. I don't have merge privileges on this repo, so I defer to you :)

I'll PR an update in eslint-plugin-jsx-a11y if you publish a point release.

v1.4.0 published


Feature Request: Flagging proofreading questions on ELU

In English.SE, we get a few questions with the following style:

Block of text, with very little if no paragraph formatting
Generally capitals in the first sentence
Contains either IELTS or TOEFL in the title
Says something like 'Hi everyone. I start to learn TOFFEL writing and I have lots of mistake. Could you find out and fix them for me?' in the title
Tags like: [grammar], [punctuation], [spelling]

A few examples: I think a few might have been deleted, but I'm a sub-10k user.

https://english.stackexchange.com/questions/404087/hi-everyone-i-start-to-learn-toffel-writing-and-i-have-lots-of-mistake-could-y
https://english.stackexchange.com/questions/403750/im-praticing-the-ielts-task-1-and-id-like-you-have-a-look-at-my-writing-thank

Is there something SmokeDetector could do to flag these types of questions?

SmokeDetector only deals with spam and abusive posts, so if the posts you're trying to catch/flag are simply off-topic, than we're not really the right people to be talking to sorry :)
Of course, all our code is open source and you're free to build your own bot based on it. If you need help we're always happy to answer questions. Additionally, the folks over at SOBotics have build a bunch of bots which conduct a variety of tasks on SO, so if you drop in there maybe someone can help you out.

I adapted Smokey to do something similar to this on ELU not long ago, so it's certainly possible to use the bot to do this kind of thing. "The" SmokeDetector, however, only focuses on spam, like Angus said.

Okay, thanks for the advice @angussidney. I'll check SOBotics and the source code out...


Refactor Delete Instance Wizard in Tentacle Manager

Move some :cheese: in the Tentacle Manager's "Delete Instance" wizard.
Background
We want to add some automated tests to Tentacle Manager as part of the Tentacle Manager on .NET project.
We first need to make the Tentacle Manager project testable by automated tests (e.g. NUnit) by separating the UI logic from the business logic. This will allow the automated tests to exercise most of the code (i.e. everything not in a UI component) without the pain of dealing with UI threads and Dispatchers etc.
[sc-70325]
Results
There should be zero visible changes from an end-user's perspective. The intent is that the functionality is completely unchanged, but the code base is in a better shape for automated tests.
Summary of the main changes within:
Remove ShellView factory method from container
UI components must be created on the UI thread, which gets real messy when the container is involved. It's best to just not do it, and only use the container for resolving models and other business logic components.
DeleteWizardModel and TentacleManagerModel now derive from ShellViewModel
Existing logic means that multiple models must be created when opening a new Tentacle Manager window - one for the ShellView, and one for the ShellView's UI content.
ShellViewModel only contains properties that are common to all features being hosted in the ShellView, so it makes sense to simplify window creation by allowing ShellView to use the same view model as its content.
Remove DeleteWizardLauncher
This class' only responsibility was to create the models and views required for the "Delete Instance" wizard. Creating models will now be done via the container, and creating the views will be done manually within the UI logic, so the UI creation functionality has been moved to TentacleManagerView where it is used.
TentacleManagerModel creates the DeleteWizardModel
It should not be the responsibility of a UI component to know how to create models for a child UI component. DeleteWizardModel is now registered in the container, and TentacleManagerModel depends on a factory function to create them on demand i.e. whenever the Tentacle Manager view wants to open a new instance of the "Delete Instance" wizard.
TentacleManagerView depends on the container
While the end goal is to have views relatively ignorant of how to create models, iteratively implementing this means we need to let TentacleManagerView create/resolve components it probably shouldn't be... at least until we've converted all the views/components.
Add exclusion for local dev builds to the IsEarlyAccessProgram helper method
By removing some of the usages of the container to resolve ShellViewModel, a side effect was introduced. This line, which has been removed, added a handler action for the container's OnActivating event when resolving a ShellViewModel. This action modified the resolved instance by disabling the EAP message. While it is highly unlikely we will be releasing any EAP versions of Tentacle Manager in the future, rather than delete this functionality it has been updated so that the EAP message gets disabled if the build is a local dev build i.e. Major version starts with 0, such as 0.0.0-local.
How to review this PR
Quality :heavy_check_mark:
As there are no automated tests, the only way to verify this PR is to manually test the Tentacle Manager. There's not much functionality in here, so testing the following should cover everything:

Install new instance
Reinstall existing instance
Restart service
Start service
Stop service
Update proxy settings
Delete instance

Pre-requisites

[x] I have read How we use GitHub Issues for help deciding when and where it's appropriate to make an issue.
[x] I have considered informing or consulting the right people, according to the ownership map.
[x] I have considered appropriate testing for my change.

When I was trying this out on a machine with many tentacles installed I got an error starting the service:
Object reference not set to an instance of an object.
System.NullReferenceException
   at Octopus.Manager.Tentacle.TentacleConfiguration.TentacleManager.TentacleManagerView.StartServiceClicked(Object sender, EventArgs e) in TentacleManagerView.xaml.cs:line 109
   at System.Windows.EventRoute.InvokeHandlersImpl(Object source, RoutedEventArgs args, Boolean reRaised)
   at System.Windows.UIElement.RaiseEventImpl(DependencyObject sender, RoutedEventArgs args)
   at System.Windows.Controls.Primitives.ButtonBase.OnClick()
   at System.Windows.Controls.Button.OnClick()
   at System.Windows.Controls.Primitives.ButtonBase.OnMouseLeftButtonUp(MouseButtonEventArgs e)
   at System.Windows.RoutedEventArgs.InvokeHandler(Delegate handler, Object target)
   at System.Windows.RoutedEventHandlerInfo.InvokeHandler(Object target, RoutedEventArgs routedEventArgs)
   at System.Windows.EventRoute.InvokeHandlersImpl(Object source, RoutedEventArgs args, Boolean reRaised)
   at System.Windows.UIElement.ReRaiseEventAs(DependencyObject sender, RoutedEventArgs args, RoutedEvent newEvent)
   at System.Windows.UIElement.OnMouseUpThunk(Object sender, MouseButtonEventArgs e)
   at System.Windows.RoutedEventArgs.InvokeHandler(Delegate handler, Object target)
   at System.Windows.RoutedEventHandlerInfo.InvokeHandler(Object target, RoutedEventArgs routedEventArgs)
   at System.Windows.EventRoute.InvokeHandlersImpl(Object source, RoutedEventArgs args, Boolean reRaised)
   at System.Windows.UIElement.RaiseEventImpl(DependencyObject sender, RoutedEventArgs args)
   at System.Windows.UIElement.RaiseTrustedEvent(RoutedEventArgs args)
   at System.Windows.Input.InputManager.ProcessStagingArea()
   at System.Windows.Input.InputManager.ProcessInput(InputEventArgs input)
   at System.Windows.Input.InputProviderSite.ReportInput(InputReport inputReport)
   at System.Windows.Interop.HwndMouseInputProvider.ReportInput(IntPtr hwnd, InputMode mode, Int32 timestamp, RawMouseActions actions, Int32 x, Int32 y, Int32 wheel)
   at System.Windows.Interop.HwndMouseInputProvider.FilterMessage(IntPtr hwnd, WindowMessage msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at System.Windows.Interop.HwndSource.InputFilterMessage(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)


Bump Polly to 8.4.1

Bump Polly to 8.4.1 to pick up bug fixes:

https://github.com/App-vNext/Polly/pull/2164
https://github.com/App-vNext/Polly/pull/2166

Group Polly dependabot updates to avoid multiple PRs such as:

#4167
#4200

Microsoft Reviewers: Open in CodeFlow

I think the packages need mirroring.

I thought I did.. I'll try again

I had mirrored Polly but not Polly.Extensions.
We really need an the internal mirror process to be easier, it makes me type in every package and version number, each one is a pipeline run unless I just want latest .. maybe @joperezr already has a script...

for this reason, I'm inclined to mostly ignore random updates eg AWS patch versions and then catch up for our releases. whereas this Polly update, it's clear it brings improvements we need.

I had mirrored Polly but not Polly.Extensions.
We really need an the internal mirror process to be easier, it makes me type in every package and version number, each one is a pipeline run unless I just want latest .. maybe @joperezr already has a script...

Technically speaking, you can likely get away with a single run most of the time if you keep version as "latest". Given the whole point of these updates is to mirror latest on NuGet, having "latest" on the pipeline means "latest for each package available on NuGet" which is most of the time what you want with these updates. That said, I think it is great feedback, and we should ask @dotnet/dnceng to see if they can allow passing in key-value pairs instead.

cc @radical fyi ^^

Looks like it takes 46 mins to do the whole lot.


Child actions code generation fails on 8.0.0

Version 7.5.11 works fine, but I have to override dependencies to use it.
Minimal reproducible example:
abstract class HomeActions implements ReduxActions {
  HomeActions._();
  factory HomeActions() = _$HomeActions;
}

abstract class AppActions extends ReduxActions {
  AppActions._();
  factory AppActions() = _$AppActions;

  HomeActions get home;
}

causes:

[INFO] built_redux:built_redux on lib/domain/actions/app_actions.dart:Generating action classes for AppActions
[SEVERE] built_redux:built_redux on lib/domain/actions/app_actions.dart:
An error FormatterException occurred while formatting the generated source for
package:***/domain/actions/app_actions.dart
which was output to
lib/domain/actions/app_actions.built_redux.g.part.
This may indicate an issue in the generator, the input source code, or in the
source formatter.
Could not format because the source could not be parsed:
line 21, column 27 of .: Expected an identifier.
╷
21 │ final home = HomeActions*();
│                           ^
╵

Note: My project doesn't use NNBD.

Probably, there should be an option to generate code without null safety. For now, I have to pass "withNullability: false" to the getDisplayString method.


Question: Define description columns using Custom Criteria

In the Data tab of a MySQL table I opened the Value viewer panel. In this panel it shows me a dictionary if I select a foreign key value in the Data tab. By clicking Define Description I open the Dictionary structure dialog.
I see there is a Custom criteria text box available. What can I type in there? for example if I type in select 'foo' then the output in the Value viewer becomes completely blank. I can type in two or more columns from the existing table like col1, col2 and that seems to work - but that's just the same as selecting those columns using the checkboxes provided above.
So the question is : what can I use the custom criteria for? if there's already a section in the user guide - I'd be happy to get pointed to that as well

System information:

Operating system (distribution) and version
DBeaver version
Java version
Additional extensions

Connection specification:

Database name and version
Driver name
Do you use tunnels or proxies (SSH, SOCKS, etc)?

Describe the problem you're observing:
Steps to reproduce, if exist:
Include any warning/errors/backtraces from the logs

We need to clarify the meaning of this obscure Custom criteria field.

I'm also would be happy to understand how this Custom criteria field works.


SQL Lab DO not support other languages

do againt a query using some column alias not in english, for example:
select word as 'おはようございます' from test;
select word as '你好' from test;
select word as 'أهلا و سهلا'from test;
error msg:
'ascii' codec can't encode characters in position 21-24: ordinal not in range(128)

This has already been reported (just search for ascii), unfortunately nobody with the issue provided a patch fixing the issue.


CB-10594 geolocation.spec.10 crashes Windows Phone 8.1

Jira issue

@dblotsky, @sarangan12, I've updated the PR, please take a look.

@daserge I am not sure if removing the event listeners is a good idea. From the user perspective, there is no reason for him to have only one event listener. All of this boils down to one issue, reusing the 'loc' variable. With the restriction that you cannot modify the properties when there are event listeners on it, I do not see a good point in reusing it (in ensurelocator function) I have created another PR to address that issue: https://github.com/apache/cordova-plugin-geolocation/pull/70

@sarangan12, listeners are removed and then subsribed again.

@daserge this change will change an existing listener, which is not expected behaviour. It is expected that when addWatch is called a second time, a new watch is created and the previous watch is not modified.

Please review #70. It looks like it specifically fixes the behaviour to match what's expected.


Add learning rate schedulers in Adversary

What does this PR do?
This PR adds support to learning rate scheduling in Adversary, borrowing the optimization utils from LitModular.
This PR also adds ReduceLROnPlateau as an example.
Type of change
Please check all relevant options.

[ ] Improvement (non-breaking)
[ ] Bug fix (non-breaking)
[x] New feature (non-breaking)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Testing
Please describe the tests that you ran to verify your changes. Consider listing any relevant details of your test configuration.

[x] pytest
[ ] CUDA_VISIBLE_DEVICES=0 python -m mart experiment=CIFAR10_CNN_Adv trainer=gpu trainer.precision=16 reports 70% (21 sec/epoch).
[ ] CUDA_VISIBLE_DEVICES=0,1 python -m mart experiment=CIFAR10_CNN_Adv trainer=ddp trainer.precision=16 trainer.devices=2 model.optimizer.lr=0.2 trainer.max_steps=2925 datamodule.ims_per_batch=256 datamodule.world_size=2 reports 70% (14 sec/epoch).

Before submitting

[x] The title is self-explanatory and the description concisely explains the PR
[x] My PR does only one thing, instead of bundling different changes together
[ ] I list all the breaking changes introduced by this pull request
[x] I have commented my code
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have run pre-commit hooks with pre-commit run -a command without errors

Did you have fun?
Make sure you had fun coding 🙃

I will revisit the optimizer issue later.