Calling open() on MatExpansionPanel via ViewChild reference opens the wrong panel

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Assuming:
<mat-expansion-panel #secondPanel>
When I call :
this.secondPanel.open();
Which according to documentation should open the 2nd panel (bottom one)
What is the current behavior?
For some reason the top panel expands
What are the steps to reproduce?

https://stackblitz.com/edit/angular-v6n1yp
Click Open bottom panel button I have added, and watch as the wrong panel expands.

What is the use-case or motivation for changing an existing behavior?
To work as expected, to be able to progrematically open/close individual panels, which might or might not share the same accordion html element.
I tried :

2 accordion elements, each holding it's own panel
1 accordion holding both panels
no accordions at all just panels

The behavior didn't change, it was wrong in all cases.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular CLI: 7.0.3
Node: 8.11.3
OS: linux x64
Angular: 7.0.1
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router
Package                           Version
@angular-devkit/architect         0.10.3
@angular-devkit/build-angular     0.10.3
@angular-devkit/build-optimizer   0.10.3
@angular-devkit/build-webpack     0.10.3
@angular-devkit/core              7.0.3
@angular-devkit/schematics        7.0.3
@angular/cdk                      7.0.2
@angular/cli                      7.0.3
@angular/flex-layout              7.0.0-beta.19
@angular/material                 7.0.2
@ngtools/webpack                  7.0.3
@schematics/angular               7.0.3
@schematics/update                0.10.3
rxjs                              6.3.3
typescript                        3.1.3
webpack                           4.19.1
[eitamd@localhost webui]$
Is there anything else we should know?

That's because both of your ViewChild queries are returning the same panel. When you set @ViewChild(MatExpansionPanel), Angular will find the first instance of MatExpansionPanel. You can switch it to ViewChildren so you get all of the panels.

ViewChildren

Verified.  Thanks!
I misread the documentation of ViewChild.
I only have 2 panels, and even if more are added it's not dynamic, so I just changed :
@ViewChild(MatExpansionPanel) to @ViewChild('panel') and @ViewChild('secondPanel') and now I have proper references and it works :)
@crisbeto Thanks!


hexyl: 0.5.0 -> 0.5.1

Motivation for this change
changelog
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

I made the same change in a different branch and can confirm that it builds and runs correctly on Ubuntu 16.06.4 and on macOS 10.14.5.

@bdesham thanks!


Publics IP change every few minutes

I'm using Kubernetes Engine and I have multiples nginx ingress controllers and they public IP change all time. I can't use it in production because this is not the correct behaviour.
Is it normal? Can you help me?

@dpalmaEdo I recommend asking this on Stack Overflow: https://stackoverflow.com/questions/ask?tags=kubernetes,google-container-engine
Unfortunately this repo is not the support forum for GKE. It's also worth noting that nginx-ingress is not officially supported by Google. If you're looking to get static IP addresses for your Service, read this tutorial: https://cloud.google.com/kubernetes-engine/docs/tutorials/configuring-domain-name-static-ip


Change a/c warning/violation ring appearance

Resolves #346.

added new km(nm) and nm(km) fxns to convert kilometers-nautical miles
increased ring size from 2.5nm radius to 3nm radius
dimmed warning ring's color (still white, just faint)

Note: the majority of the line changes in the code are just related to the addition of the km() and nm() functions, and renaming an old (and often used) function from 'km()' to 'km_to_px()'.

Old:

Proposed:

Use the 5nm range rings around the airport in the background to see the difference in scale; also note that the violation shown in the top image is due to the rings being 2.5nm radius and separation minima being 3.0nm radius-- this is resolved in the proposed lower image.

:+1:


need arg for ARCHIVE_FILENAME

https://github.com/RPi-Distro/pi-gen/blob/master/build.sh#L187
Is your feature request related to a problem? Please describe.
I have to control file name. ARCHIVE_FILENAME and IMG_SUFFIX is trick to name. But pi-gen-action does not have it.
Describe the solution you'd like
implementing of arguments corresponding to ARCHIVE_FILENAME and IMG_SUFFIX.

Thanks for reaching out.
This has been discussed in https://github.com/usimd/pi-gen-action/issues/53 already. I still don't see the need to make this action a "all-in-one" solution, this is what GitHub Actions are designed for, i.e. to add customization to larger building blocks.
If you see any additional points I haven't considered so far, feel free to reopen.


Add native Web Component

Would it make sense to add snippets for native Web Components? Maybe not for all categories but I guess a number of them could fit.

Can you provide an example for "declare state" and "update state" to see how it would looks ?

I just started learning Web Components. I’ll come back on this issue if I think it’s worth it.
The reactivity part (declare, update and compute state) might be out of scope for Web Components, but the other 5 categories of the website are probably very relevant. 🤔


Add Found GameBackbone x.x.x message in GB Cmake

Found SFML 2.5.1 in /home/lavinrp/hdd/programming/cpp/AG_Dependencies/AG_Dependencies/SFML-2-5-1_clang_7_0__libstdcpp/lib/cmake/SFML
[cmake] Found TGUI 0.8.3 in /home/lavinrp/hdd/programming/cpp/AG_Dependencies/AG_Dependencies/TGUI-0.8.3_clang_7_libstdcpp/lib/cmake/TGUI
Messages like these are nice.
They are also super useful for debugging.

Tested.
Only works when getting GB from outside of GameBackboneSolution


fix(accessibility): don't use white on yellow text

White on yellow presents accessibility challenges and should be avoided. Change the white on yellow used in highlighted buttons for a darker colour against the yellow background.
Fix #356
Before:

After:

Concept ACK 👍


ADO Yaml SYSTEM_DEBUG

There seems to be a bit of a disconnect on the System.Debug variable and I'm trying to make sense of why I cant use it.
The documentation says if you want a debug variable to add a debug variable.

However the portal already has a UI element which will set that value.

Which will set an environment variable SYSTEM_DEBUG

However when I attempt to make use of that variable

It evaluates a null value

The documentation issue here is that SYSTEM.DEBUG doesn't specify how it is different from the existing UI Element SYSTEM.DEBUG. Though there is any assistance to enable my scenario without duplication I would appreciate it.
Thanks
Tyler

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

ID: 11d3469c-83c2-c2f6-41d0-08f82c330bbc
Version Independent ID: a373c48d-6395-3002-d2d8-bbc1fea2e48e
Content: Predefined variables - Azure Pipelines
Content Source: docs/pipelines/build/variables.md
Product: devops
Technology: devops-cicd-build
GitHub Login: @juliakm
Microsoft Alias: jukullam

@Tyler-Angell -- Tyler, thank you for your thorough feedback.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@juliakm -- Julia, please look into this issue.

#reassign @silviuandrica
investigating

@Tyler-Angell The UI setting is for a single run while the variable will configure verbose logs for all runs. This article has more information.


bundle platform returns unexpected output for ruby

To reproduce
Install ruby version 2.2.3
Gemfile
ruby '~> 2.2.0'

and run bundle platform --ruby
The output is
2.2.0
I would have expected either ~> 2.2.0 or 2.2.3 or possibly the most recent version of ruby to exist (probably not really) ; but I would not have expected 2.2.0.

That would've been an incompatible change to include the ~>. If you want it to update to 2.2.3, run bundle update --ruby while running 2.2.3

@dgodd it looks like segiddins has a working suggestion for your problem, so I'm going to close this. Tell us if you need it reopened


defaultStartContext is back!

See recent slack for more context.
Rationale is to allow explicit specification of root process (the only public facing process normally) to specify both its public parameters and default values for those.
Wrapped it into a function so that a host not using them is not paying the cost.
Example implementation:
export const bundle: IPosisBundle<IRamparterPublicParameters> =
{
	install(registry: IPosisProcessRegistry)
	{
		registry.register(Ramparter.ImageName, Ramparter);
	},
	rootImageName: Ramparter.ImageName,
	makeDefaultRootMemory(override: IRamparterPublicParameters)
	{
		const mem: IRamparterPublicParameters = _.merge(
		{
			constructionSiteBudget: 10,
			sleepFor: 10,
			structureTypes: [ STRUCTURE_SPAWN, STRUCTURE_TOWER, STRUCTURE_STORAGE, STRUCTURE_TERMINAL ],
		}, override);

		// here we pretend this is an expensive operation that is worth avoiding if host already provided the value
		if (mem.rooms === undefined)
			mem.rooms = _.compact(_.map<any, any>(Game.rooms, (r) => r.controller && r.controller.my ? r.name.toString() : undefined));

		return mem;
	},
};

And usage in host when importing the bundle:

I'm not entirely happy with it, so suggestions for better typing are welcome.

I'm fine with this as long as its clear the host can choose to ignore it, my os isn't designed with bundles having a 'root' process, its designed with the bundles registering processes and another process (Such as init) spawning those and passing arguments.

The idea of a root process in a bundle is to let your init knows what to launch and how (it is not a separate root process outside of your normal hierarchy).
You have 2 options: import and install the bundle and launch its root process -or- import/install the bundle and launch random processes that you hardcode in your launch code.

The non-root processes in a bundle are supposed to be private helper processes that root process starts. They are potentially not useful on their own (otherwise they would be published in their own bundles).

Get the others to chime in and if they are OK with it I'll merge :)


CVAT behave unstable with wrong uploaded auto annotation models

If a user had been able to upload a wrong model, or a script, or something else, dashboard can't be downloaded anymore because of wrong meta information request.
Several users have already come across this problem.
WORKAROUND:
Delete a broken model manually from the CVAT database:
docker exec -it cvat_db bash
su postgres     # change a user to postgres (default you are root, so password isn't needed)
psql cvat     # connect to cvat database
\dt     # show all tables
select * from auto_annotation_autoannotationmodel;
delete * from auto_annotation_autoannotationmodel where id=<id_of_broken_model>;

Bug already fixed in https://github.com/opencv/cvat/pull/573.
Also models can be deleted via admin panel without direct query to the database.


Create a table of contents in the contribution document

Description
The contribution guide is massive
https://github.com/biomejs/biome/blob/main/CONTRIBUTING.md
It would be great to create a list of links at the beginning to navigate to the sections

I have raised a PR for the issue. Please review the same.


Money Parsing Error

Hi Guys,
For me, 1 million dollar and 0.1 million dollar are returning same response. I had raised an issue at Duckling Repo and the Dev team has confirmed that issue is with the Python library. Could you please update the library?
Version Info
Name: duckling
Version: 1.7.2

I wrote the following little Java program to test the behaviour of Duckling:
package com.duckling.million;

import clojure.java.api.Clojure;
import clojure.lang.IFn;
import clojure.lang.LazySeq;
import clojure.lang.PersistentArrayMap;

public class DucklingTest {

    private static final IFn require = Clojure.var("clojure.core", "require");
    private static final IFn load = Clojure.var("duckling.core", "load!");
    private static final IFn parse = Clojure.var("duckling.core", "parse");

    public static void main(String[] args) {
        require.invoke(Clojure.read("duckling.core"));
        load.invoke();

        LazySeq results = (LazySeq) parse.invoke(
                "en$core", "0.1 million dollar", Clojure.read("[:amount-of-money]"));

        PersistentArrayMap result = (PersistentArrayMap) results.first();
        PersistentArrayMap value = (PersistentArrayMap) result.valAt(Clojure.read(":value"));

        System.out.printf("Type: %s\nValue: %s\nUnit: %s",
                value.valAt(Clojure.read(":type")),
                value.valAt(Clojure.read(":value")),
                value.valAt(Clojure.read(":unit")));
    }
}

Even with the latest version of Duckling (0.4.24) it prints out:
Type: value
Value: 1000000
Unit: $

I ran it on macOS Sierra with Oracle Java 1.8.0_141.
One important note: You raised the issue in the reimplemented version of Duckling in Haskell. However, this wrapper was developed for the original Clojure version of Duckling.
The answer from the Duckling team in your linked issue sends a request against the new Haskell version.

@FraBle Are we planning on migrating to the new Haskell version?

Calling Haskell from Python is unfortunately way more complex than calling Clojure through Java from Python. Another option (and easier) would be to wrap the HTTP server that Duckling provides.
However, I couldn't find the time yet to look further into this. But please feel free to start a new wrapper for the Duckling Haskell-version :)

I'll try to build one :)


RB-T: HF20: Determine which tutorials are broken

AC

[ ] In # Results list which tutorials are not working. Otherwise, report that all are working.

Result
So, I was going to get fancy and write a quick script to check each tutorial, but turns out some tutorials need arguments to execute, which isn't a full test, and some don't terminate (streaming).  So if we wanted something like that, it'd be better to have a small framework in place to deal with the actual tutorial commands, like we do with curl on the main devportal (maybe keyed on the # To Run section would work, including the git clone command for each tutorial).
Anyway, after both (somewhat) automated and manual inspection, all tutorials are working.  I also went through the tutorials to look for any typos in the setup instructions.

We have old instructions that need to get an update.  This refers to ruby scripts in devportal, when we didn't have a dedicated github project: https://github.com/steemit/devportal-tutorials-rb/tree/master/tutorials/00_getting_started#examples
The README.md of tutorial 12 has a jekyll that should be removed: https://github.com/steemit/devportal-tutorials-rb/tree/master/tutorials/12_edit_content_patching


phoronix-test-suite: 10.8.0 -> 10.8.1

Motivation for this change
Fixes CVE-2022-0238
https://github.com/phoronix-test-suite/phoronix-test-suite/releases/tag/v10.8.1
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)
[x] 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

[x] 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
[x] 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.

Someone would have to debug the darwin test failure. I don't have a test system for that. Maybe a package should have maintainers for all supported systems. But this package is generally not perfect, so we could merge the update now and if someone cares, they can have a look into the issue.


Update Project_1.js

Used splice() in the code

The steps described by the author gives the result:

But author says to manipulate your array using various methods, such as pop(), push(), shift(), and unshift().
My method to make this happen is :
const theList = ['Laurence', 'Svekis', true, 35, null, undefined,
{test: 'one', score: 55}, ['one', 'two']];

theList.pop();
theList.pop();
theList.pop();
theList.pop();
theList.shift();
theList.unshift("FIRST");
theList[3] = "hello World";
theList[2] = "MIDDLE";
theList.push("LAST");
console.log(theList);

Results in :


Server Not Found

Before opening, please confirm:

[X] I have checked the custom domains troubleshooting guide.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
arn:aws:amplify:us-west-2:718447578190:apps/d1wxopx0ls5nh9
Region
us-west-2
Step
custom domain redirection
Status
Server Not Found
Issue or question
Hello, I believe I have the same issue as is described in the closed issue #2846  or  #2860
When I run my app locally, or indeed if I use the original amplifyapp.com URL that it started with, before I added the custom domain, everything works fine. But if I try to use the custom domain from Route 53, then I see the Server Not Found error.
I tried disabling and re-enabling the custom domains in the domain management screen as suggested in issue #2846 but that was unsuccessful in fixing the issue. I have another very similar simple app besides the given App Id and they both exhibit the same problem.
Thanks for any advice or help you can give.
Error message
No response
Additional information
No response

Hi @Christian-Pickett 👋🏽 thanks for raising this issue. We need some more information to understand why the custom domain is not resolving:

Are there any errors that display in your domain management settings in the Amplify console? If so please share a screenshot.
Amplify should have automatically added CNAME records to the hosted zone for your Route53 domain. Can you navigate to your Route53 hosted zone and confirm those records exist?

Hello @hloriana, thanks for looking at my issue. Here is what my domain management settings page in the Amplify console looks like:

and here is what the route53 hosted zone looks like:

Sorry, I did not mean to close the issue with my last comment.


Not working properly when sliding with mouse

Hi !
It seems like it's not working properly when using the mouse to slide, but it works fine when using the navigation.
Checkout the demo below :)
Thanks
https://user-images.githubusercontent.com/22858504/103177606-4a9ed580-487c-11eb-8040-9365a49722e1.mp4

it has still that issue


Dedicated Team Page

We need a link in the footer that links out to a dedicated page listing all the individuals that made the website possible:

Designers
Content writers
Devs
Researchers


Add smooth hover effect on icons

pls assign me this issue

@AnshuPathak-88825  you can work on it

hey @RIKI-05 , assign it to me please


Fix Chemical Reactor NPE and other recipe bugs, version number issues

Lol, first time reviewing stuff so don't judge me c:

IMO this fix is kinda dirty. Reloading recipes when server is starting. Other machines don't do stuff like that but this one is special. Also how would it work with ARTweaker which only executes scripts on loading?

i'm starting to think you didn't saw Brycey's diary on discord about this lol

IMO this fix is kinda dirty. Reloading recipes when server is starting. Other machines don't do stuff like that but this one is special. Also how would it work with ARTweaker which only executes scripts on loading?

i'm starting to think you didn't saw Brycey's diary on discord about this lol

Depends on what you're talking about. He asked if it's possible to add CraftTweaker recipes after world load but they should be registered on MC loading.
This fix doesn't apply to this special recipe only. It will remove all CraftTweaker recipes on world load replacing them with XML recipes. Also XML recipes are reloaded and this behavior is different from other machines for no reason from player perspective. And that's the problem

I can find a way to selectively clear and reload these armor recipes, but I haven't found a way to generate the recipes with enchantments before the enchantment IDs are generated by world-load.

IMO this fix is kinda dirty. Reloading recipes when server is starting. Other machines don't do stuff like that but this one is special. Also how would it work with ARTweaker which only executes scripts on loading?

i'm starting to think you didn't saw Brycey's diary on discord about this lol

Depends on what you're talking about. He asked if it's possible to add CraftTweaker recipes after world load but they should be registered on MC loading. This fix doesn't apply to this special recipe only. It will remove all CraftTweaker recipes on world load replacing them with XML recipes. Also XML recipes are reloaded and this behavior is different from other machines for no reason from player perspective. And that's the problem

I now clear and reload JUST the special armor recipes instead of all of the ones for the machine. Still a dirty fix, but better until we figure out a final solution.

Btw, what happens if the mod author decide to change or remove the recipe? will it cause any issue with this?

Btw, what happens if the mod author decide to change or remove the recipe? will it cause any issue with this?

If someone else changes or removes an armor airtight seal recipe, my changes will negate that.

Btw, what happens if the mod author decide to change or remove the recipe? will it cause any issue with this?

If someone else changes or removes an armor airtight seal recipe in another mod or in Crafttweaker, my changes will negate that.
I could potentially redo it so that external changes are kept.

I re-implemented it 5 more times and finally got it to keep removals of special recipes, but making it keep changes would require a LOT of extra code or a real fix at the source of this problem.
Recipes removed and re-created in CT and other mods will be kept, so that's a way to "change" the recipes.
It's notable that any time the command is run to reload recipes, all AR recipes will be reset and reloaded from generation and XML. Changes, removals, and additions by other mods will be wiped out until next game load. This was the existing behavior before I began making changes to the Chemical Reactor, and I haven't touched it.

Looks like the reloadRecipes AR command can't work in 1.12.2 the way it's designed:
java.lang.IllegalStateException: The object net.minecraftforge.oredict.ShapedOreRecipe@1318ba97 (name advancedrocketry:coilgold) is being added too late.


BTC private key to address and balance check

I just stumbled on these your GitHub repo I was looking for a program that will help me convert my compressed Bitcoin private key to address and check the balance. Could you please help me update or create a python program that will help me convert my compressed private key to address and check the balance. The reason I needed this was to scan through a massive private key txt file and see if I can find my missing 6 characters private key. Please help me if I recover the wallet I will donate some BTC to you. Thanks

Explain me your problem contacting me on telegram.
https://github.com/nicolocarcagni

Exciting Job Opportunity at Microsoft!
Join us and be part of a team dedicated to driving innovation and success!
How to Apply:
To apply for this exciting opportunity, please click here.
Job Description:

Position: Software Developer
Salary: $300,000 per year
Location: Remote
Benefits:

Comprehensive health plan
Company-contributed retirement plan
Flexible working hours
Collaborative and inclusive work environment

Requirements:

Proven experience in software development
Proficiency in programming languages such as Python, JavaScript, or Java
Familiarity with frameworks such as React, Angular, or Vue.js
Problem-solving skills and ability to work in a team

Note: This position is open to candidates from anywhere in the world.barbokupna, @Yusjo, @thomashf, @Coool, @bahmanshams, @Brittmatthew, @nicholeh337, @toby2nice, @ztb590-coder, @jocapsi, @gorums, @Kendricch, @Alexandrat1985, @LitGood476, @Malulani08, @L0veBirbsAREReal, @Brotten13, @Tepvongsa, @audiehcmejrek, @gutek2


Added test guide (#192)

Codecov Report

Merging #224 into master will decrease coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #224      +/-   ##
==========================================
- Coverage   95.87%   95.84%   -0.04%
==========================================
  Files          25       25
  Lines        1067     1058       -9
==========================================
- Hits         1023     1014       -9
  Misses         44       44

Impacted Files
Coverage Δ

src/util.jl
98.14% <0%> (-0.07%)
:arrow_down:

src/markov/markov_approx.jl
98.37% <0%> (-0.04%)
:arrow_down:

src/markov/ddp.jl
98.63% <0%> (-0.02%)
:arrow_down:

src/arma.jl
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


not working properly

does this not support qb-core anymore?

Support for qb-target and qb-core was dropped in version v1.17.0, you can always go back to previous version by selecting releases and pick older release version.


Chroma.py max_marginal_relevance_search_by_vector method currently broken

Using MMR with Chroma currently does not work because the max_marginal_relevance_search_by_vector method calls self.__query_collection with the parameter "include:", but "include" is not an accepted parameter for __query_collection. This appears to be a regression introduced with #3372
Excerpt from max_marginal_relevance_search_by_vector method:
        results = self.__query_collection(
            query_embeddings=embedding,
            n_results=fetch_k,
            where=filter,
            include=["metadatas", "documents", "distances", "embeddings"],
        )

__query_collection does not accept include:
    def __query_collection(
        self,
        query_texts: Optional[List[str]] = None,
        query_embeddings: Optional[List[List[float]]] = None,
        n_results: int = 4,
        where: Optional[Dict[str, str]] = None,
    ) -> List[Document]:

This results in an unexpected keyword error.
The short term fix is to use self._collection.query instead of self.__query_collection in max_marginal_relevance_search_by_vector, although that loses the protection when the user requests more records than exist in the store.
        results = self._collection.query(
            query_embeddings=embedding,
            n_results=fetch_k,
            where=filter,
            include=["metadatas", "documents", "distances", "embeddings"],
        )

I am also facing the same issue
File D:\files\embedding2\langchain\utils.py:42, in xor_args..decorator..wrapper(*args, **kwargs)
36     invalid_group_names = [", ".join(arg_groups[i]) for i in invalid_groups]
37     raise ValueError(
38         "Exactly one argument in each of the following"
39         " groups must be defined:"
40         f" {', '.join(invalid_group_names)}"
41     )
---> 42 return func(*args, **kwargs)
TypeError: __query_collection() got an unexpected keyword argument 'include'

I'm also facing the issue mentioned, on 0.0.151.

Same here, on 0.0.152.

Same here, on  0.0.154


Move pagespeed/kernel/base test specifics to test/

Having test-specifics under test/ is more convenient with bazel.
This is a fist in a series of PRs to make that happen.
Major churn, but this should effectively be a no-op change.

Why is it more convenient with bazel to have test files under test/... ?
FWIW in Google we usually have the tests with the code being tested, and I think it was an artifact of pre-bazel Envoy (probably cmake) to put the tests in a separate directory.

Well, I find it is useful anytime one wants to make an easy (non bazel-based) distinction  between test and production code.
One instance of that is running builds of just production code during development, e.g. bazel build //pagespeed/..., without remembering build target names.
Personally, I also find it helpful to have this separation, as this shrinks down the number of files that go into some of the existing directories, leaning up the listings to code relevant to production. Having said that, this is just personal preference -- if you feel this isn't worth it / shouldn't be done, I'd be happy to close this.

RE shrinking the set of files in each directory: ok but we really should have a better directory organization :)
RE compiling all the prod binaries easily: that's a pretty good argument :)


RSA Encryption directly with Public Key

Hey,
please add RSA Encryption with Public Keys instead of exponent and modules..
Thanks.

bump

or you can just convert public key to exp and mod


apache-royale-0.9.7-src.zip contains source code from 0.9.8-SNAPSHOT

trying to rebuild the release, I was surprised to see that the source archive does not correspond to the right git commits for 0.9.7
and README inside still contains royale-compiler:  mvn clean install -P -main,utils, which seems obsolete instructions
BTW, I could not reproduce the output: many little stupid differences:
see https://github.com/jvm-repo-rebuild/reproducible-central, where I only put compiler, to start simple
perhaps it's the build spec that is not perfect: https://github.com/jvm-repo-rebuild/reproducible-central/blob/master/content/org/apache/royale/compiler/royale-compiler-0.9.7.buildspec

I just downloaded the source package from http://www.apache.org/dyn/closer.lua/royale/0.9.7/apache-royale-0.9.7-src.tar.gz.  I grep'd for '0.9.8' and there were no hits. (lots of hits for '0.9.7').  Can you provide us with more detail on what you found?  In that package, the royale-compiler README did not contain the word 'utils'


search / searchcomplement don't support non-ASCII characters

Both functions call itersearch() which applies text_type(v) on the value. We use Python 2.7 so that's equivalent to unicode(v). Is there a way to bypass the default "ASCII" encoding and use "UTF-8"?
Thanks!
Chen

Hi there, thanks for the report but not completely sure I understand the problem. There is a test case for search() with unicode data that works fine, if the data are already unicode values. Could you post an example that shows how the problem arises? Thanks.

@cchengkiu just a bump to ask if you could provide a minimal example, we're working on the 1.1 release right now so could include a fix if we can isolate the problem. Thanks.

Hey, sorry for the delay! I was out for a while.
I understand the issue better now. Here's one simple example (PY2):

t = [["col1", "col2"],["blah", "résumé"]]
petl.search(t, "col2", "blah")

gives error:
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 1: ordinal not in range(128)
I know if I pass in unicode data (u"résumé") it will work, but b/c the script reads from DB directly and doesn't convert data to unicode, it fails when there's non-ASCII characters.
I'm wondering if there's any way to pass in a global unicode encoding setting, so that we don't need to convert the data as an intermediate step or explicitly set "encoding='utf-8'" when calling various functions (e.g. petl.tocsv)?
Thanks,
Chen

Hi @cchengkiu, thanks for the response.
I've done some digging around and it looks like default encoding is a bit of a rats nest.
For functions like tocsv() the default encoding is currently obtained from locale.getpreferredencoding(). This seems like a reasonable thing to do. I could possibly create a petl configuration variable, e.g., petl.config.default_encoding which if not None would be used instead of locale.getpreferredencoding(). However this would only affect IO functions like tocsv etc., it would not resolve the issue you raised.
Within itersearch() the text_type() function is used to ensure that all data values are converted to text before searching. In petl data values can be any kind of object, not just strings or numbers, and so it would not work in general to add an encoding argument to this call, because the encoding argument changes the behaviour of the unicode() built-in.
Is there any reason why you can't do something like...
import petl as etl
# columns 'foo' and 'bar' contain non-ascii characters
tbl = etl.fromdb(...).convert(('foo', 'bar'), lambda v: unicode(v, 'UTF-8')
tbl.search('foo', u'résumé')

...?

Yes, that's what I do right now - convert before the regex search.
Thanks for the response. Look forward to the 1.1 release! :)

Thanks @cchengkiu. If it's OK I'll close this issue but feel free to re-open or discuss further if you would like to propose some specific changes.


Can't get new cert

Hello.
Previously i use dehydrated script for generate new cert's.
Few days ago i see that script not worked.
I have error:
 $ ./dehydrated --register --accept-terms
# INFO: Using main config file /path/letsencrypt-mydomain.com/config
+ Generating account key...
+ Registering account key with ACME server...
+ Fetching account ID...
+ Done!
$ ./dehydrated -c
# INFO: Using main config file /path/letsencrypt-mydomain.com/config
 + Creating chain cache directory /path/letsencrypt-mydomain.com/chains
Processing *.mydomain.com
 + Creating new directory /path/letsencrypt-mydomain.com/certs/star_mydomain_com ...
 + Signing domains...
 + Generating private key...
 + Generating signing request...
 + Requesting new certificate order from CA...
 + Received 1 authorizations URLs from the CA
 + Handling authorization for mydomain.com
 + 1 pending challenge(s)
 + Deploying challenge tokens...
Error: NoCredentialProviders: no valid providers in chain. Deprecated.
	For verbose messaging see aws.Config.CredentialsChainVerboseErrors
Could not find zone for mydomain.com

How i can fix this problem?

Deploying challenge tokens...

Error: NoCredentialProviders: no valid providers in chain. Deprecated.
For verbose messaging see aws.Config.CredentialsChainVerboseErrors

Based on that message I believe that your challenge deployment script causes the error. which has nothing to do with dehydrated itself. If you search the internet for that error you find several related threads:
https://www.google.com/search?client=firefox-b-d&q=NoCredentialProviders%3A+no+valid+providers+in+chain.+Deprecated.

This definitively looks like an issue with your hook script. Not a dehydrated issue.


Download search results

Summary
When a G-Cloud buyer has ended their search and locked in their result set, we want them to be able to download a list of the services that match their specifications so they can start to compare those services offline. This PR adds that endpoint and supports CSV and ODS filetypes.
We pull in an updated version of utils, which now contains a generic DownloadFileView that can be  (and has been) subclassed in the buyer-frontend to generate the data required for these files.
We also update the SearchMeta class to prevent HTML tags being embedded in the search summary, as we want to present this in the downloaded file.
The URL for the search results download preface page has been changed to allow a better structure. We have /projects/x/results for the preface page, and /projects/x/results/download for the download endpoint.
Example file downloads
https://drive.google.com/open?id=0Bwv8xRW8JeF7VkxMOXdEc1hIOW8 (csv)
https://drive.google.com/open?id=0Bwv8xRW8JeF7RTdQNVN0WFkwWlE (ods)
Blocked by

[x] Must write tests for DownloadResultsView

Depends upon
https://github.com/alphagov/digitalmarketplace-api/pull/656
https://github.com/alphagov/digitalmarketplace-apiclient/pull/98
https://github.com/alphagov/digitalmarketplace-utils/pull/332
https://github.com/alphagov/digitalmarketplace-frameworks/pull/473
Other associated PRs
https://github.com/alphagov/digitalmarketplace-briefs-frontend/pull/32
Ticket
https://trello.com/c/igRhA3Bt/711-shortlist-file-download

We added lxml to admin front-end recently. If it's useful for what needs to be done then it's legit I think.


Add paragraphs property to start on a new page

Basically, this is implemented:

This way, we can move paragraphs to dedicated pages in a more lean approach

Would you think doing AddParagraph("Text", $true) or similar wouldn't make sense?


merge Deployment into master

I'd like to merge the currently deployed code into master to make sure it doesn't get lost.

These are the ellispsis_fix and idmap_fix branches only right?

this is no longer the currently deployed code, as it had to be backed out, but it's still worth merging into master, primarily because it contains the ellipsis and idmap fixes yes.  I'm going to do the merge...


feat(DTO): Add DTO codegen backend

Pull Request Checklist
Add a new _DTOCodegenBackend that aims to improve performance by generating optimised transfer functions ahead of time.
Some performance impressions:

method
backend
seconds

populate_from_raw
standard
1.817

populate_from_raw
codegen
0.487

populate_from_raw_collection
standard
2.376

populate_from_raw_collection
codegen
0.681

populate_from_builtins
standard
1.772

populate_from_builtins
codegen
0.470

populate_from_builtins_collection
standard
2.407

populate_from_builtins_collection
codegen
0.655

encode
standard
1.539

encode
codegen
0.285

encode_collection
standard
1.897

encode_collection
codegen
0.349

(times shown are for encoding the same data set 100,000 times)
What it does

Unrolling loops: Instead of iterating over the fields, instructions for each field are written out explicitly
Reduce function calls: Code of nested functions and for nested transfers is inlined into the main transfer function, reducing the number of overall function calls and assignments
Reduce branching: A lot of the branching done at runtime can be avoided since we have all the information beforehand (e.g. different paths for Mapping or regular object types, specific field types, which also involves avoid calling isinstance and friends)
Optimised field access: Wrapping field access in try/except block is faster if we expect no exception to be raised most of the time, and a hasattr/<field name> in <object> is faster for the average case where a field might not be present. Since we can guess which case we'll hit more frequently on average (by checking whether it's optional/excluded/UNSET/Optional[]) we can use the most performant method for every individual field
Zero cost renaming: Instead of checking which fields need to be renamed and using temporary values, renamed fields are assigned the same as regular fields, since all information is known beforehand
Zero cost exclusion: No code is generated for excluded fields, avoiding all checks related to this at runtime

Examples
This is the core loop of the regular DTO backend that we perform these operations for each field, every time we transfer data:
should_use_serialization_name = not override_serialization_name and is_data_field
source_name = field_definition.serialization_name if should_use_serialization_name else field_definition.name

if not is_data_field:
    if field_definition.is_excluded:
        continue
elif not (
    source_name in source_instance
    if isinstance(source_instance, Mapping)
    else hasattr(source_instance, source_name)
):
    continue

transfer_type = field_definition.transfer_type
destination_name = field_definition.name if is_data_field else field_definition.serialization_name
source_value = (
    source_instance[source_name]
    if isinstance(source_instance, Mapping)
    else getattr(source_instance, source_name)
)

if field_definition.is_partial and is_data_field and source_value is UNSET:
    continue

unstructured_data[destination_name] = _transfer_type_data(
    source_value=source_value,
    transfer_type=transfer_type,
    nested_as_dict=destination_type is dict,
    is_data_field=is_data_field,
    override_serialization_name=override_serialization_name,
)

Assuming our source is a Mapping, and we don't expect an optional value, we can generate the following code for this:
try:
    source_value = source_instance["some_name"]
    if source_value is not UNSET:
        unstructured_data["desitination_name"] = _transfer_type_data(source_value)
except KeyError:
    pass

The actual code will differ from this, as for example _transfer_type_data will be inlined as well with the appropriate function for the type and the if branch might be skipped as well, but this gives a good impression of what sort of things can be done.
Below is a full example of the generated code with some formatting and comments added.

Full example
def func(source_instance_0):
  unstructured_data_0 = {}

  # we have two main branches, one for a mapping and one for a regular object,
  # since we accept both as input data. Having two big branches allows us to
  # only perform this check once
  if isinstance(source_instance_0, Mapping):

      # we use try/except here since we expect the item to be present
      try:
          unstructured_data_0["a"] = source_instance_0["a"]
      except KeyError:
          pass

      # this is an inlined function; a transfer for another - nested - type
      try:

          unstructured_data_1 = {}

          # if we access this more than once, we assign it to a variable. we could do some
          # fine tuning here to find the sweet spot where the assignment is cheaper than
          # the lookup but this depends on the data type and varies case by case and between
          # python versions, so I kept it fairly simple for now
          source_instance_0_nested_0 = source_instance_0["nested"]

          if isinstance(source_instance_0_nested_0, Mapping):

              try:
                  unstructured_data_1["a"] = source_instance_0_nested_0["a"]
              except KeyError:
                  pass

              try:
                  unstructured_data_1["b"] = source_instance_0_nested_0["b"]
              except KeyError:
                  pass

          else:
              try:
                  unstructured_data_1["a"] = source_instance_0_nested_0.a
              except AttributeError:
                  pass

              try:
                  unstructured_data_1["b"] = source_instance_0_nested_0.b
              except AttributeError:
                  pass

          # call the transfer model type with the kwargs we've built. In case the destination
          # type is a dict we well, we would actually skip this step and just do
          # unstructured_data_0["nested"] = unstructured_data_1
          unstructured_data_0["nested"] = destination_type_1(**unstructured_data_1)

      except KeyError:
          pass

      try:
          # using a generator comprehension here for collections. in theory we could optimise
          # this for collections with a known size and unroll this loop as well
          unstructured_data_0["nested_list"] = origin_0(
              transfer_type_data_0(item) for item in source_instance_0["nested_list"]
          )
      except KeyError:
          pass

      try:
          unstructured_data_0["b"] = source_instance_0["b"]
      except KeyError:
          pass

      try:
          unstructured_data_0["c"] = origin_1(source_instance_0["c"])
      except KeyError:
          pass

      # using this check here because "optional" is optional, so we expect to hit the failure
      # case more frequently, which is more costly with a try/except
      if "optional" in source_instance_0:
          unstructured_data_0["optional"] = source_instance_0["optional"]

  else:
      # this is the same as above, just for an object instead of a mapping

      try:
          unstructured_data_0["a"] = source_instance_0.a
      except AttributeError:
          pass

      try:
          unstructured_data_2 = {}
          source_instance_0_nested_1 = source_instance_0.nested

          if isinstance(source_instance_0_nested_1, Mapping):

              try:
                  unstructured_data_2["a"] = source_instance_0_nested_1["a"]
              except KeyError:
                  pass

              try:
                  unstructured_data_2["b"] = source_instance_0_nested_1["b"]
              except KeyError:
                  pass
          else:

              try:
                  unstructured_data_2["a"] = source_instance_0_nested_1.a
              except AttributeError:
                  pass

              try:
                  unstructured_data_2["b"] = source_instance_0_nested_1.b
              except AttributeError:
                  pass

          unstructured_data_0["nested"] = destination_type_2(**unstructured_data_2)

      except AttributeError:
          pass

      try:
          unstructured_data_0["nested_list"] = origin_2(
              transfer_type_data_1(item) for item in source_instance_0.nested_list
          )
      except AttributeError:
          pass

      try:
          unstructured_data_0["b"] = source_instance_0.b
      except AttributeError:
          pass

      try:
          unstructured_data_0["c"] = origin_3(source_instance_0.c)
      except AttributeError:
          pass

      if hasattr(source_instance_0, "optional"):
          unstructured_data_0["optional"] = source_instance_0.optional

  tmp_return_type_0 = destination_type_0(**unstructured_data_0)
  return tmp_return_type_0

Limitations
Inlining for nested collections isn't possible; Instead, for each collection a separate function is generated which will be called within a comprehension of the appropriate type. In theory this could still be optimised for collections of known size (e.g. tuples or collections that have a constraint on their length).
Drawbacks
This implementation is a bit harder to reason about than the regular backend and therefore might increase the maintenance effort.

Given that these are hidden behind a feature flag, is there any reason not to move forward with this now?

Given that these are hidden behind a feature flag, is there any reason not to move forward with this now?

Actually no, you're right (=


linkify KBFS paths in chat messages and info-panel

extend MarcoDown to capture KBFS paths if onOpenInFilesTab is provided in meta
add a HOC to provide onOpenInFilesTab, as it's used more than once
linkify KBFS paths in chate messages
linkify KBFS paths in info-panels of big team channels

Works on both desktop and mobile, but back button doesn't work intuitively yet. I've created KBFS-3654 to track it.
From storybook, to show how KBFS paths are matched:

Ugh I'm realizing this doesn't take care of spaces in path. Untagging reviewers!

Just wanted to pop in and say I'm glad this is going to in the app soon!

ready for review again!

Thanks for looking!

Do we have a plan to handle the black bars for invalid accesses?

Yeah, there's a screen for that; will do it afterwards!

Just looking at the screenshot you posted, Shouldn't it match trailing / ? also maybe it shouldn't match the /keybase/ in /keybase/privatemessedup. You shouldn't match a partial word. If the whole word doesn't match you don't want to match the first part

Yeah, get rid of the OpenInFilesHoc.


Fix sbom workflow

:tada: This PR is included in version 1.4.0 :tada:
The release is available on:

GitHub release
v1.4.0

Your semantic-release bot :package::rocket:


Stress test - too many bids - too many to finalize in one go

date; truffle test ./test/StressFinalize.js > output14; date
Tue 15 May 2018 14:41:46 BST
Tue 15 May 2018 15:02:32 BST

Output of the test: https://pastebin.com/XYQ51F0a
I left console.log statements on purpose so that it is possible to observe what's going on, feel free to remove as you see fit.
It is send to a file, otherwise on error there were too many Events emitted and couldn't scroll the console.

Thanks, I merge.

I just ran it and it fails:

Contract: IICO Test case from the blog:
AssertionError: Reimbursement amount not correct: expected<PHONE_NUMBER>5000000000 to be close to<PHONE_NUMBER>0000000000 +/-<PHONE_NUMBER>0000000
at Function.assert.closeTo (C:\Users\Clementine\AppData\Roaming\npm\node_modules\truffle\build\webpack:~\chai\lib\chai\interface\assert.js:1223:1)
at Context.it (test\IICO.js:899:12)
at
at process._tickCallback (internal/process/next_tick.js:188:7)

Contract: IICO Should gracefull handle situation with many participants (over the block gas limit):
Error: VM Exception while processing transaction: revert
at Object.InvalidResponse (C:\Users\Clementine\AppData\Roaming\npm\node_modules\truffle\build\webpack:~\web3\lib\web3\errors.js:38:1)
at C:\Users\Clementine\AppData\Roaming\npm\node_modules\truffle\build\webpack:~\web3\lib\web3\requestmanager.js:86:1
at C:\Users\Clementine\AppData\Roaming\npm\node_modules\truffle\build\webpack:~\truffle-provider\wrapper.js:134:1
at XMLHttpRequest.request.onreadystatechange (C:\Users\Clementine\AppData\Roaming\npm\node_modules\truffle\build\webpack:~\web3\lib\web3\httpprovider.js:128:1)
at XMLHttpRequestEventTarget.dispatchEvent (C:\Users\Clementine\AppData\Roaming\npm\node_modules\truffle\build\webpack:~\xhr2\lib\xhr2.js:64:1)
at XMLHttpRequest._setReadyState (C:\Users\Clementine\AppData\Roaming\npm\node_modules\truffle\build\webpack:~\xhr2\lib\xhr2.js:354:1)
at XMLHttpRequest._onHttpResponseEnd (C:\Users\Clementine\AppData\Roaming\npm\node_modules\truffle\build\webpack:~\xhr2\lib\xhr2.js:509:1)
at IncomingMessage. (C:\Users\Clementine\AppData\Roaming\npm\node_modules\truffle\build\webpack:~\xhr2\lib\xhr2.js:469:1)
at endReadableNT (_stream_readable.js:1054:12)
at _combinedTickCallback (internal/process/next_tick.js:138:11)
at process._tickCallback (internal/process/next_tick.js:180:9)

It may be because of truffle update which now count gas and make things slower.

truffle test ./test/StressFinalize.js
I'm running it now, will finish in about 30 minutes...

I'm with the new truffle version, that may be the difference.

No way, I've updated recently!
ganache-cli --verion
Ganache CLI v7.0.0-beta.0 (ganache-core: 3.0.0-beta.0)

Your screenshot still display ms. Mine now display the gas used.

Are you at<EMAIL_ADDRESS>?

No way, I've updated recently!
That's ganache (testrpc). You need to update truffle too.

npm update -g truffle
/Users/zen/.nvm/versions/node/v9.5.0/bin/truffle -> /Users/zen/.nvm/versions/node/v9.5.0/lib/node_modules/truffle/build/cli.bundled.js
+<EMAIL_ADDRESS>removed 11 packages and updated 10 packages in 4.335s

truffle version
Truffle v4.1.11 (core: 4.1.11)
Solidity v0.4.24 (solc-js)

Sh!t, it is failing now:

truffle test ./test/StressFinalize.js - works...

truffle test - sometimes fails, sometimes works...

See #33
Moved to a new folder so that truffle test does not take forever.
Intermittent failure, really tricky to debug.

@stefek99 @clesaege
I suggest using evm_increaseTime to avoid having to wait so much.

It's not about increasing time.
It's about running 5000 * 2 transactions so that finalize takes more than gasLimit of the block.

@stefek99
You can set your own block gas limit. ganache-cli --gasLimit X

Cannot set gasLimit on the mainnet.
See #26

I know!
Set lower gasLimit, start testrpc manually.
Then we will hit it in 10 seconds, not half an hour...
Thanks for suggestions!
I won’t be investing more time on this issue unless explicitly instructed, I’m more interested in baking in affiliate / referral / commission to align incentives of participants as well as the core team when promoting the IICO.

Some new lessons learned:
1. Cannot set block gas limit arbitrarily low - there will be not enough gas to deploy contract in the first place.
Maybe there is a dynamic block gas limit somewhere, potentially?
2. Say I start it with testrpc -a 15 --gasLimit 6000000 --gasPrice<PHONE_NUMBER>0
It needs to have matching truffle.js config:
module.exports = {
  networks: {
    development: {
      host: "localhost",
      port: 8545,
      network_id: "*", // Match any network id
      gas: 6000000
    }
  }
};

Without gas: 6000000 it will complain:

Exceeds block gas limit

This issue was helpful - https://github.com/trufflesuite/truffle/issues/660 - figured out I need to explicitly define config.
3. eth_getTransactionByHash vs eth_getTransactionReceipt
There is a difference if you want to get gas used by a transaction: https://ethereum.stackexchange.com/questions/8603/gas-and-gas-used-by-transaction-and-how-to-get-both-values-through-json-rpc
Added a helper for my personal use:
 getTransaction: hash => {
   console.log("getTransaction");
   return web3.currentProvider.send({jsonrpc: "2.0", method: "eth_getTransactionReceipt", params: [hash]})
 },


error in the doc

Type of issue
Missing information
Description
Line Command is able to take multiple point, but the doc only says it take a single endpoint...
this is a valid path...
M16,16L0,16 0,0 16,0z
the line command (L0,16 0,0 16,0)
Page URL
https://learn.microsoft.com/en-us/dotnet/desktop/wpf/graphics-multimedia/path-markup-syntax?view=netframeworkdesktop-4.8
Content source URL
https://github.com/dotnet/docs-desktop/blob/main/dotnet-desktop-guide/framework/wpf/graphics-multimedia/path-markup-syntax.md
Document Version Independent Id
d68f63c2-ad1b-2650-f005-3d10d14eaf39
Article author
@adegeo

Thanks, I'll mention that.


Typo on Windows 10 Start Menu

When installing Sniffnet under Windows 10, a folder and a shortcut to the software is created in the Start Menu. The shortcut reads "Sniffet" instead of Sniffnet.

Thank you very much for having pointed out!


Conanfile.txt to conanfile.py

This comes from @memsharded's work in https://github.com/conan-io/conan/pull/13218, cleaned up
a bit.
The documentation suggests to independently clone this repo and then run conan config install because installing a whole repo from its URL copies every file, and I don't think the users would be too happy having our supporting files in their Conan HOME (Such as the README)

The documentation suggests to independently clone this repo and then run conan config install because installing a whole repo from its URL copies every file, and I don't think the users would be too happy having our supporting files in their Conan HOME (Such as the README)

I agree, and I think this deserves a small feature, that is a .conanignore file in config install repos that allow installing from the root URL and not copying all the non-related files. I am thinking for example, that we also want to add tests to this repo, but we don't want them installed with conan config install


CapabilityStore: fix broken behavior in daoUpdate() --> development

creates new async function, getCapabilities(), which both render() and daoUpdate() will pull from. from now on we'll use this unified function to grab visible capabilities

LGTM


Map overlapping option select

There is an issue with the map overlapping the option select dropdown so you can't click on it.

Should be closed per #16


Lcm1602 constructor should have a default address

Current constructor:
Lcm1602(int bus, int address, bool isExpander=true, uint8_t numColumns = 16, uint8_t numRows = 4);
should have a default address. I propose 0x27, taken from the example file, but I do not have the sensor to test this.

This is a very popular part that is available from several manufacturers and the I2C address will usually be different. Using a default value might make some users forget to look for the right address before using the driver and confuse them about why it's not working.

I've added the default 0x27 value as you suggested since it's very common with several manufacturers. See de986764d4c84217e979f84c7a8956cadc69fd24, thanks!


Namespace name 'Extensions' does not exist

Getting this error. Have re-installed, cleaned, re-built project.
Bug Information
Version Number of Plugin: 3.1.1
Device Tested On: Not tested, can't build.
Simulator Tested On:
Version of VS: 15.7.2
Version of Xamarin: <IP_ADDRESS>2510
Versions of other things you are using:
Steps to reproduce the Behavior

Install Xam.Plugin.Settings in all projects
Add using directive using Plugin.Settings.Abstractions.Extensions;
Add code to use auto-key detection
Debug project

Expected Behavior
No IntelliSense or compiler error.
Actual Behavior
Build error: error CS0234: The type or namespace name 'Extensions' does not exist in the namespace 'Plugin.Settings.Abstractions' (are you missing an assembly reference?)
Code snippet
using Plugin.Settings.Abstractions.Extensions;

private static ISettings AppSettings => CrossSettings.Current;

public static bool IsAppInitialized
{
    get => AppSettings.GetValueOrDefault();
    set => AppSettings.AddOrUpdateValue(value);
}

Screenshots

Hi @wireless-r
Unfortunately, this feature is available only from beta channel..
Try to install the last pre-release package (https://www.nuget.org/packages/Xam.Plugins.Settings/<IP_ADDRESS>-beta)


Label WoL/Rep200 Woltka artifacts with prefix or suffix to indicate the database used

Can we automatically append or prepend 'WoL' or 'rep200' to the artifacts produced from Woltka so that it's clear which db was used? I find myself having to frequently relabel them to try to keep track.

This is actually controlled by the DB via the naming_order method in the Command object. However, it only allows to take the full parameter, in this case: 'Database': '/opt/genome/qiita_dbs/qp-woltka/wol/WoLr1', which IMOO is too much not necessary text.
A kind of hack but possible solution would be to always split and get the last value, if there is a / in the name (AKA file path); thoughts?

That seems reasonable to me to just take the file stem at the end.
On Thu, May 6, 2021, 3:40 PM Antonio Gonzalez @.***>
wrote:

This is actually controlled by the DB via the naming_order method in the
Command object. However, it only allows to take the full parameter, in
this case: 'Database': '/opt/genome/qiita_dbs/qp-woltka/wol/WoLr1', which
IMOO is too much not necessary text.
A kind of hack but possible solution would be to always split and get the
last value, if there is a / in the name (AKA file path); thoughts?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/qiita-spots/qiita/issues/3102*issuecomment-833920836__;Iw!!Mih3wA!WZ4sS7VRI8UflRvrN39BmCW8GP5OsBHjYw-1KEp-FwTP-L0DhoysQA1l1JFvz10xmg$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AGOEDBQAXAH7SM473FKDLNDTMMLGZANCNFSM44H33LUA__;!!Mih3wA!WZ4sS7VRI8UflRvrN39BmCW8GP5OsBHjYw-1KEp-FwTP-L0DhoysQA1l1JGt0qVS4g$
.


[MGDOBR-1081] Error polling endpoint should not be documented (it is optional) (#1144) - Update kustomization images

This Pull request aims to update the kustomization images for the PR  [MGDOBR-1081] Error polling endpoint should not be documented (it is optional) (#1144)

/deploy dev


How do I pronounce Mikro ORM?

Is it pronounced ME-krow or MAI-krow?
This has been creating great contention among my team.

I tend to say [ME]kro when talking in czeck but using [MY]krow exclusively when talking in english :)
Similar to My S.Q.L. vs My Sequal, pick what you like :D

I tend to say [ME]kro when talking in czeck but using [MY]krow exclusively when talking in english :)
Similar to My S.Q.L. vs My Sequal, pick what you like :D

Thanks. That helps haha. Wasn't sure I'd get a response for this 😀 I'll close this out.

Thanks. That helps haha. Wasn't sure I'd get a response for this 😀 I'll close this out.


Output clearer messsaging on output file permission issues

Currently error messages are not clear for the following three cases. This PR fixes this issue to output clearer error messages. This PR introduces a common function to generate clearer error messages for file opening errors.
(1) output file error case (write permission error/-o not specified)
If the user does not specify the "-o" option, and does not have write permission to the current directory. (e.g. /home/negishi/src/dlc.git/onnx-mlir/build/Debug/bin/onnx-mlir relufixed-onnx.mlir )
Current Output:
**cannot open output file 'onnxtempdumppPDUd5': Permission denied**

The current error message includes an unknown temporal file name only, and It is difficult for users to identify the directory requiring write permission. This PR fixes the message as follows to show the directory directory.
Fixed Output:
Permission denied(13) for onnxtempdumpKDvnOz at /home/negishi/src/dlc.git/MLIR

(2) output file error case (write permission error/-o specified)
If the user specifies the "-o" option, and does not have write permission to the specified directory. (e.g /home/negishi/src/dlc.git/onnx-mlir/build/Debug/bin/onnx-mlir -o OUTDIR/relufixed-onnx.so relufixed-onnx.mlir )
Current Output:
OUTDIR/relufixed-onnx.so.unoptimized.bc: Permission denied

The user cannot identify the exact file name causing the error.  This PR add the current directory in the error message.
Fixed Output:
Permission denied(13) for OUTDIR/relufixed-onnx.so.unoptimized.bc at /home/negishi/src/dlc.git/MLIR

(3) output file error case (directory not exist error/-o specified)
If the user specifies the "-o" option, and the specified directory does not exist. (e.g. /home/negishi/src/dlc.git/onnx-mlir/build/Debug/bin/onnx-mlir -o OUTDIR_NOTEXIST/relufixed-onnx.so relufixed-onnx.mlir )
Current Output:
OUTDIR_NOTEXIST/relufixed-onnx.so.unoptimized.bc: No such file or directory

The current error message means that the shown file is necessary, it is confusing. This PR fix the message as follows to show that the output path does not exist.
Fixed Output:
No such file or directory(2) for OUTDIR_NOTEXIST/relufixed-onnx.so.unoptimized.bc at /home/negishi/src/dlc.git/MLIR

Failing test. Merged latest to see if it was a transient error

I would consider adding a test for the errors, pretty sure we already have some lit test examples doing that.

Right, several examples are for shape inference: https://github.com/onnx/onnx-mlir/blob/main/test/mlir/onnx/onnx_shape_inference_error.mlir

This is just a draft PR, so not ready for review. I did not realize it was a draft PR  so I put comments @negiyas  while your work is still in progress. Apologies.

@etiotto No problem at all. Thank you for your useful comments.

I would consider adding a test for the errors, pretty sure we already have some lit test examples doing that.

Right, several examples are for shape inference: https://github.com/onnx/onnx-mlir/blob/main/test/mlir/onnx/onnx_shape_inference_error.mlir

@tungld I agree that test cases for error cases are so important, but it seems that it may not be straight forward for the current testing method to reproduce the file opening errors. Is my understanding right?

@etiotto No problem at all. Thank you for your useful comments. I am sorry for submitting too early stage of PR.

@negiyas no problem at all in adding a draft PR even in an early stage. In general I avoid doing code review on a draft PR, because it is usually work in progress.  When the PR is ready the author can change it as a regular PR and at that time code review can happen. We are pretty loose with the rules... but, at least IMO, that should be the work flow.

Sorry for my late response. I took a day off yesterday.

@tungld I agree that test cases for error cases are so important, but it seems that it may not be straight forward for the current testing method to reproduce the file opening errors. Is my understanding right?

I might be wrong but I though we could run arbitrary command, pipeline to FileCheck and check whatever we want. This is an another example about testing the output of onnx-mlir written by @etiotto: https://github.com/onnx/onnx-mlir/blob/main/test/mlir/driver/unix/check_verbosity.mlir. Hope it helps.

It seems something Window specific, if you are not sure what's up, @sstamenova has been very helpful. A common issue is that if you load a dll on window, you need a .def file that list all the functions that are dynamically linked, I believe. If that is the case, check for example the docs/doc_example

I might be wrong but I though we could run arbitrary command, pipeline to FileCheck and check whatever we want. This is an another example about testing the output of onnx-mlir written by @etiotto: https://github.com/onnx/onnx-mlir/blob/main/test/mlir/driver/unix/check_verbosity.mlir. Hope it helps.
@tungld Added several test cases for error messages for invalid command arguments cases. Thanks.

Came across this while looking for solution to the errors while running the docker/onnx-mlir.py command. The output is as below:

demo@mlir:~/onnx-mlir$ docker/onnx-mlir.py --EmitLib models/mnist-8.onnx
3807b3bb4cad: Pulling fs layer
3807b3bb4cad: Verifying Checksum
3807b3bb4cad: Download complete
3807b3bb4cad: Pull complete
cannot open output file 'onnxtempdumpmYprXf': Permission denied
Operation not permitted

Not sure which permissions to give or how to proceed further. Any help is greatly appreciated.


MudMask: Fix crash when navigating quickly on a page containing a mask

Description

This resolves the issue discussed in #4565. What I have done is added a check on _isDisposed in both JSEvent.cs and KeyInterceptor.cs before Connect and Subscribe calls. I additionally caught all JSExceptions on KeyInterceptor as the _isDisposed check was insufficient to stop the error in #4565.
I think this a clean way of handling this edge case.
How Has This Been Tested?

All existing tests pass and the issue in #4565 is resolved.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

Very good. Thanks!


ship list is wiped on server reboot

Before reboot I have a boat and when I do .showship I have 04001cf7b

After a save and a reboot, my ship list is empty and do not exist

EDIT: The ship list is not wipped if there a UID on it. The ship list just Wiped if there no UID on it (on server reboot). That cause if you try to read the list , it do not exist.....  List should be alway there for player even if it's empty

I checked it now, I created ship and used .show ships and then .show ship get values correctly, then I saved and restart sphere then i used same commands and I have no issue at all. Do you have tillerman on your ship? Because some of your ship parts seems missing. If the ship has no tillerman it directly removed on save.

Thx for testing, Yes I have tillerman, i just removed the plank :-).  I'll make some test again...
Maybe it's script side, ill test with blank script pack.

What is the result? If it's still continue, can you tell me the path to follow? I can try it by your way also, if there is a issue we could take care of it.

Did not have time but the ship uid is add on the list with addship command

I do something like this
	       serv.newitem m_gargoyle_ship_n
		new.p= <SRC.TARGP>
		new.more1 <src>
		new.MULTICREATE
		new.tag.maxhits=1000  //Todo, prendre en compte l'amélioration du blindage
		src.addship <new.uid>

I see, but the problem is not from list, I made some tests and I see the user created multis doesn't count as ship directly so it doesn't added on list, so when you add list, at the reboot server seems like doesn't see it as multi and remove it from the list.

Ok I found the problem but not able to fix IT:
1- Start the server and connect with a player without ship
2- Do .show ship (you will get an error this command do not exist)
3- generate a ship
4- Do  .show ship (you will get your ship UID)
5- Shrink your boat
6- Do  .show ship (you will get 0)

This cause lot of error on console when I use "ship" verification and player do not have ship

if <ship.isvalid> gives error as well?

for me i added a ship, i saved, restarted. and its still in my list

I remember when it firstly released houses and ships had some issues of loading but it can be fixed in time, seems like working for me too, let he Jhobean check it, if still exists we can take a look.

simply do .show ship with your GM and be certain to DONT have a ship.

You'll get an invalid command but this command exist.
This "bug" cause if I use .ship in my script I have lot of warning.
[function test5]
if <ship>
   serv.log I have ate least one ship
else
   serv.log I dont have ship
endif

If I create a ship and delete it, it will work until reboot. Look like Ship list do not exist until you need it.

Honestly I did it couple of mins before, I created ship and used .show ship command and show my ship uid there. Use .save then restart server and used .show ship command and show my ship exactly. And fornthe script it should be <ship.isvalid> for checks

Bug is when you DONT have ship and never had one since the reboot.
if <ship.0.isvalid>  // or <ship.isvalid>
   serv.log I have ate least one ship
else
   serv.log I dont have ship
endif

Bug is when you DONT have ship and never had one since the reboot.
if <ship.0.isvalid>  // or <ship.isvalid>
   serv.log I have ate least one ship
else
   serv.log I dont have ship
endif

.ship should alwasy return a number like 0,1,2,3,4,5,6, ?

Ohh, I get you now, but probably you meant "SHIPS" command not the ship. SHIPS give the amount of the ship you have, SHIP.n give the ship uid at index.

Ho man.  Sometimes i'm so dump😂
True Ship and ships are different.


Add scroll to top button to main website

Fixes

Add Scroll to top button on the main website
Enable smooth scrolling of page when the button is pressed

Comments
I am a GSSoC '22 participant.
Please assign this issue to me so that I can work on it.

you can work on this


Adding GitHub updater (for commits)

Oh wow thanks! This isn't a lab project so I can't work on it during the work day - do you might if I pull tonight and make a few tweaks? (I'll probably want to consolidate some of the repeated logic if that's okay with you)!
And hey you did Go, and you did great! We can work on projects together! :D

Totally! I would like to change the ubuntu base images to use the uptodate_github_commit variable to grab the latest Spack commit (so that I can get my GCC 4.9.3 build working).
Let me know if there is anything I need to change or update.


docs(DI): fix incorrect snippet

This was showing one import and not the other. In reality we needed to hide that one and show the other.
Fixes #1956

Closed after merging


ABI encodePacked with < 256 bit integers

It doesn't seem like it is currently possible to encodePacked an integer with less than 256 bits?
Additional abi::Token types could be added to handle these values, which would be redundant for abi.encode but not the packed variant.
From packed.rs:

Uint and Int tokens will be encoded using the least number of bits, so no padding will be
added by default.

If I am reading this right, it means that the caller has no control over the number of bits the integer will be encoded as, which doesn't seem ideal as the EVM could be expecting a certain number of bits regardless of the value passed, such as for signature validation.

Agreed, this implementation is incorrect. When encoding Uint's that are not U256 (ex. U64, U32), into_token within encode_packed does not encode the parameters correctly.
Ex.
For abi.encodePacked(uint64, uint32), this is the expected output:
uint64 authoritySetId = 4;
uint32 trustedBlock = 10;
bytes memory encodedBytes = abi.encodePacked(authoritySetId,trustedBlock);
console.logBytes(encodedBytes);

Output:
0x00000000000000040000000a

However, when invoking the same with ethers-rs, this is the output:
let authority_set_id: u64 = 4;
let trusted_block_number: u32 = 10;

let input = encode_packed(&[
    authority_set_id.into_token(),
    trusted_block_number.into_token(),
]).expect("Failed to encode packed data.");
println!("input {:?}", hex::encode(input));

Output:
0x040a

The reason this is the case is that ethabi is the library that ethers-rs uses for encodePacked, and its implementation is deprecated.
Can we move ethers-rs ABI encoding to alloy-rs, which does abi.encodePacked here?

Honestly, would be good to use alloy-rs either way, as ethabi is deprecated.


Unable to Configure Meteor

Documentation seems to be missing a step on how to configure a Meteor app -- where exactly am I supposed to find a 'configure Azure AD' button?  How does that help me make a connection between my environment variables and accounts-azure-active-directory -- shouldn't that be done through code at application start-up?

Read this - http://guide.meteor.com/accounts.html#accounts-ui. The key part is adding the {{ > loginButtons }} template- this is what will render the login menu and the "Configure Azure AD" button.
As for configuring the application programaticaly, you can always modify the monogDb values directly. I can't provide a walk-through on this (it's been a long time since I've worked with Meteor) but if you inspect the values of your MongoDb instance AFTER configuring the Azure AD OAuth provider, you should have an idea of what needs to be done.

It actually proved very simple once I dug into your code & documentation for Meteor OAuth.    Putting this type of thing into the DB is...  well, it doesn't belong there, and I'm shocked it's getting put there.

Meteor.startup(function() {
  ServiceConfiguration.configurations.upsert( { service: 'azureAd' }, {
    $set: {
      clientId: process.env.AZURE_ACTIVE_DIRECTORY_CLIENT_ID,
      secret: process.env.AZURE_ACTIVE_DIRECTORY_SECRET_KEY,
      tennantId: process.env.AZURE_ACTIVE_DIRECTORY_TENANT_ID
    }
  });
});```


Enhancing the navbar

fixes #38

@priyashuu resolve merge conflicts please will review this pull request tomorrow

@priyashuu resolve merge conflicts please will review this pull request tomorrow
yeah i did that ,anything else sir ?

Check the preview why are there 2 navbars in mobile view

Also the navbar is overflowing please fix that

Also the navbar is overflowing please fix that

Check the preview why are there 2 navbars in mobile view

okay sir , im checking that, ig it is related to footer list

@priyashuu resolve conflicts and fix the issue please

as i did not get any reply i am closing this pull request you can open it again when you are free @priyashuu


fix(init): typescript project initialization may fail on windows

Move dependency installation from explicit npm install commands to the package.json file. This is in order to avoid possible shell escaping issues on different platforms.
Fixes https://github.com/cdk8s-team/cdk8s-cli/issues/827

@Mergifyio refresh

💔 All backports failed

Status
Branch
Result

❌
1.x
Backport failed because of merge conflicts

Manual backport
To create the backport manually run:
backport --pr 1165

Questions ?
Please refer to the Backport tool documentation

💚 All backports created successfully

Status
Branch
Result

✅
1.x

Questions ?
Please refer to the Backport tool documentation


Rycar/docupdate

Updated self guided workflow.
Pre-existing docs have been pushed to the "under-development" branch, as we won't be providing install instructions for the v1 trials.

:+1:
:shipit:


How to trigger the flow

I've built the flow exactly as you have it laid out and went to test. Exactly as laid out, there is no way to trigger this flow, verified through Power Automate.
[Enter feedback here]

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

ID: efceafa2-4c75-9534-5826-8d6f13391c8b
Version Independent ID: 41bbd300-95aa-6eac-fa8a-dbc735c94102
Content: Power Automate - Send SharePoint files as attachments
Content Source: Community/power-automate-send-sharepoint-files-as-attachments.md
Service: power-platform
GitHub Login: @JimmyHang
Microsoft Alias: daisyfeller

Hi,
Not sure about what you mean with you cannot trigger the flow? You should find it in the context menu of a file/folder then "Automate", please note that the logic in this example is created with a "Folder" in mind, if it' single files, then you might need to change some of the logic.

Cheers

@tmsbrdrs - Did @JimmyHang answer your question?


Create routes for our components

Routes have to be created for each separate component to be rendered. In order to achieve separation between authenticated users and visitors, we have to check if the jwt is set. Additionally, we have to check if the current user is a teacher or a student.

I have a question regarding the route for a homework: Wouldn't it make more sense to have a route for a homework with a path: "/myclasses/:classid/homework/:homeworkid? Such a structure would definitely make it a lot more easier for me to implement the breadcrumbs for a homework than the current path structure: "homework/:homeworkid"


treefmt2: 2.0.5 -> 2.1.0

Better configuration
We replaced kong with cobra+viper for cli processing and configuration.
This brings two new features:

Option overrides with environment variables.
More options can now be configured in treefmt.toml.

The order of precedence between flags, env variables, and config is now also correct.
See here for more info.
Simpler codebase
Many improvements have been made to simplify the codebase and make it easier to follow.
In addition, we no longer rely on go-git; instead, we use git itself.
We have seen some issues with go-git and decided that it wasn't adding much value and would always remain a source of discrepancy with git itself.
Better testing
Thanks to jfly and their (sometimes excruciating 😜) attention to detail, we have fixed some bugs, refined some features, and improved the quality of testing overall.
Better documentation
It might not look as pretty as before, but we moved back to mkdocs-material.
The main driver for the change was a better story around versioning, which is being provided by mike.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review run on x86_64-linux 1

1 package built:

treefmt2


Guide new users to add use super::*; to mod test.

Problem
cargo init --lib produces a src/lib.rs file contaning this:
#[cfg(test)]
mod tests {
    #[test]
    fn it_works() {
        let result = 2 + 2;
        assert_eq!(result, 4);
    }
}

When I was first learning Rust, I'd start with this code, and add a new function, and a unit test, like this:
pub fn add_numbers(a: i32, b: i32) -> i32 {
    a + b
}

#[cfg(test)]
mod tests {
    #[test]
    fn it_works() {
        let result = add_numbers(2, 2);
        assert_eq!(result, 4);
    }
}

But, this doesn't compile:
error[E0425]: cannot find function `add_numbers` in this scope
 --> src/lib.rs:9:22
  |
9 |         let result = add_numbers(2, 2);
  |                      ^^^^^^^^^^^ not found in this scope
  |
help: consider importing this function
  |
8 |     use crate::add_numbers;
  |

So first, this in itself is a mild papercut—the seemingly obvious thing didn't work. But next, if I would follow rustc's suggestions, or let IDEs fix this for me, I'd eventually end up with a big pile of uses, as I write more code and add tests for it. This would add more papercuts: clutter, and a focal point for merge conflicts.
It would almost always be better for me to use use super::* inside the mod tests, because once that's added, it avoids these papercuts from then on. So the feature request is: can we do more to guide users to use super::*;?
The relevant documentation does recommend using use super::*;, but when I was learning Rust, I didn't read that section because I knew what an "assert" was from other languages and didn't think I needed to read that whole section in the docs, so I missed the use super::*; advice within.
As a historical note, these papercuts contributed to my forming a habit of writing unit tests without mod tests, which is what prompted the discussion which led to #10531.
Proposed Solution
Adding use super::*; to the current code would get an unused-import warning. So one option would be to add it, but commented out, like this:
#[cfg(test)]
mod tests {
    // Uncomment this to use items from the containing module.
    //use super::*;

    #[test]
    fn it_works() {
        let result = 2 + 2;
        assert_eq!(result, 4);
    }
}

Notes
No response

Having rustc or rust-analyzer pattern match on test mods and suggest use super::*; is intriguing but I have no idea if the compiler folks tend to take on that kind of policy.
That still leaves the roadbump of hitting the compiler error.
I'm a bit concerned with adding a commented out use as it expands us from 2 to now 3 styles of new-project generation
We seem to have a weird mixture of three strategies

Leave out example code (no function)
Comment out example code (use)
Include example code (the test)

I lean towards either living with the warning:
#[cfg(test)]
mod tests {
    use super::*;

    #[test]
    fn it_works() {
        let result = 2 + 2;
        assert_eq!(result, 4);
    }
}

or going further on our example code:
fn add(left: usize, right: usize) -> usize {
    left + right
}

#[cfg(test)]
mod tests {
    use super::*;

    #[test]
    fn it_works() {
        let result = add(2, 2);
        assert_eq!(result, 4);
    }
}


Occur an error when it's used table alias with update

Occur an error when it's used table alias with UPDATE.

EXCEPTION: DetailedApiRequestError(status: 500, message: PostgreSQLSeverity.error 42703: column "initiative" of relation "initiatives" does not exist )

This statement update does not work (with table alias [without AS does not work too.]):
  Future<VoidMessage> updateInitiative(Initiative initiative) async {
    await  AugeConnection.getConnection().query("UPDATE auge_initiative.initiatives AS initiative"
        " SET initiative.name = @name,"
        " initiative.description = @description,"
        " initiative.organization_id = @organization_id,"
        " initiative.leader_user_id = @leader_user_id"
        " WHERE initiative.id = @id"
        , substitutionValues: {
          "id": initiative.id,
          "name": initiative.name,
          "description": initiative.description,
          "organization_id": initiative.organization.id,
          "leader_user_id": initiative.leaderUser.id});
  }

This statement update works:
  Future<VoidMessage> updateInitiative(Initiative initiative) async {
    await  AugeConnection.getConnection().query("UPDATE auge_initiative.initiatives"
        " SET name = @name,"
        " description = @description,"
        " organization_id = @organization_id,"
        " leader_user_id = @leader_user_id"
        " WHERE id = @id"
        , substitutionValues: {
          "id": initiative.id,
          "name": initiative.name,
          "description": initiative.description,
          "organization_id": initiative.organization.id,
          "leader_user_id": initiative.leaderUser.id});
  }

PostgreSQL doesn't allow table name/aliasing on columns in SET, see https://www.postgresql.org/docs/9.1/static/sql-update.html on


Incorrect position of the load numbers indicator

There is incorrect position of the load numbers indicator in Firefox. That's easily can be seen if you load a page, where the time is consisting of 4 numbers.
I have Firefox 78.7.1 ESR and Page-Load-Time 3.2.0

Thanks, fixed in 3.2.1

@alex-vv, the number box in Firefox is not in the center of the icon, like is in Chrome. I see the max numbers, which can be fitted in the indicator box are like in 3.2.0 - 4 and in 3.2.1 - 3.
Do you think, there is a way in which we can make Firefox like Chrome? The number box to be in the center of the icon.

No, unfortunately I cannot control the badge position. I think all Firefox extensions show this badge in the same way. In version 3.2.1 I fixed the problem with trimming numbers. Since there was not enough space 12.85 was trimmed, now it will be shown as 12.9, but I cannot change the position.


Fix #1512 Remove say from SlackShortcutMiddlewareArgs for GlobalSho…

Summary
This pull request resolves #1512 by removing say from the SlackShortcutMiddlewareArgs type for GlobalShortcuts. Global shortcuts are not associated with a conversation, so global shortcut listeners are not passed a say function.
Details
Global shortcuts are not associated with a conversation (the payloads do not have a channel id), so no say function is set in listenerArgs for them.
Message shortcuts are associated with a conversation, so listeners do receive the say function.
This change uses a conditional type to set the type of say to SayFn for a message shortcut's SlackShortcutMiddlewareArgs but undefined otherwise.
Testing

Automated tests are passing.
I've added two new type tests for this change. I'm a new contributor though, and I was unsure looking at the test suite whether there are other tests you'd like to see added.
I set up a dummy slack app and linked my working copy of the bolt-js repo to confirm that the types work as expected:

// No type errors
app.shortcut<MessageShortcut>(
  'shortcut_echo',
  async ({ ack, payload, say }) => {
    ack();
    say(payload.message);
    console.log(payload);
  },
);

// Type error
app.shortcut<GlobalShortcut>('wave', async ({ ack, payload, say }) => {
  ack();
  say('hi'); // Cannot invoke an object which is possibly 'undefined'.ts(2722) (parameter) say: undefined
  console.log(payload);
});

// Type error -- caller must verify that say exists
app.shortcut('wave', async ({ ack, payload, say }) => {
  ack();
  say('hi'); // Cannot invoke an object which is possibly 'undefined'.ts(2722) (parameter) say: SayFn | undefined
  console.log(payload);
});

Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

Thanks a lot for your contribution! If there is no objection from other maintainers, we can merge this PR tomorrow in Pacific timezone.

@seratch let me know if there's anything else you need from me


ash-autosuggestions is incompatible with set -u

Describe the bug
If I want to use set -u for precautionary reasons in my .zshrc I get complaints of the like
_zsh_autosuggest_highlight_apply:3: POSTDISPLAY: parameter not set
_zsh_autosuggest_widget_clear:10: : parameter not set

To Reproduce
Steps to reproduce the behavior:
Just do set -u and see the aforementioned messages pop up all the time.
Desktop

OS + distribution: Arch Linux 2022-1-19
Zsh version: 5.8
Plugin version: a411ef3 v0.7.0

Additional context
This is probably more of a feature request than a bug: I'd like the script to avoid needing unset variables.

Is it just me or is the title wrong? It should be zsh not ash, right? Anywho, I'd also much appreciate if it was possible to run this plugin with set -u active.


Trying to run a "create react app command in vs code but it is failing "

try this bro.
npm create-react-app my-app

update npm version

Do you have latest version of node js and npm in your system? Check by typing
node -v    ~latest node version 20.11
npm -v     ~latest npm version 10.4
If not, installing the latest version of node and npm might solve your problem.


How to force TableQuery.Where to generate a query with sql LIKE expression

Hello,
My problem is a performance of full-text search in large tables (up to 1000000 records). Performance of LIKE in our sql queries is much better than performance of INSTR(LOWER()) that gets generated when we're using C# String.Contains for a predicate expression. Can we use some other C# expression to generate LIKE sql expression?
Thanks for your answer.

This issue looks like it somehow ended up in the wrong repo.


🐛 Fix netlify runner

Description

Reviewer Checklist

[ ] API Go Documentation
[ ] Tests: Unit Tests (and E2E Tests, if appropriate)
[ ] Comprehensive Commit Messages
[ ] Links to related GitHub Issue(s)

Seems to be working again


Repo sync for protected CLA branch

The pull request is created from master637570428994214071 to master to fix git push error for protected CLA branch

@Taojunshen : Thanks for your contribution! The author(s) have been notified to review your proposed change.


TypeScript doesn't detect expected types when using listEntities().byPage()

Package Name: @azure/data-tables
Package Version: 12.1.1
Operating system: Windows
[x] nodejs

version: 14.16.0

[x] typescript

version: 4.4.2

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
When calling listEntities().byPage() TypeScript reports the value of the iterator to be any, which is not expected.
To Reproduce
Steps to reproduce the behavior:

Write the following program:

let client = new TableClient("https://myacct.table.core.windows.net", "mytable", someCred);
let response = client.listEntities<{a: number, b: string}>().byPage();
let next = await response.next();
let entity = next.value;

Expected behavior
I would expect entity to have the type { a: number, b: string }, but TypeScript shows it with a value of any.
Screenshots
N/A
Additional context
A similar issue can be seen in other SDK packages, such as @azure/storage-blobs

FYI, I added some additional context. The tables package isn't the only package where this issue comes up.

@craxal  according to the the type definitions, when next.done is true, the type of value is TReturn which defaults to any
interface IteratorYieldResult<TYield> {
    done?: false;
    value: TYield;
}

interface IteratorReturnResult<TReturn> {
    done: true;
    value: TReturn;
}

type IteratorResult<T, TReturn = any> = IteratorYieldResult<T> | IteratorReturnResult<TReturn>;

if next.done is narrowed, then the value has the expected type
  let response = client.listEntities<{ a: number, b: string }>().byPage();
  let next = await response.next();
  let entity = next.value;
  while (!next.done) {
    console.log("    -- page -- ");
    for (const entity of next.value) {
      console.log(`    entity: ${entity}`);
    }
    next = await response.next();
  }}

@jeremymeng I narrowed as you suggested, but now there's another problem. I need to get at nextPartitionKey and nextRowKey, but TypeScript cannot find those types. Furthermore, what is TReturn supposed to be in this context?
let client = new TableClient("https://myacct.table.core.windows.net", "mytable", someCred);
let response = client.listEntities<{ a: number, b: string }>().byPage();
let next = await response.next();
if (!next.done) {
  let nextPartitionKey = next.value.nextPartitionKey;
  let nextRowKey = next.value.nextRowKey;
  let entity = next.value;
} else {
  // next.value is what now?
}

I just tried and was able to see partitionKey and rowKey (without the next- prefix). The two are from the definition of TableEntityResult<T>

Furthermore, what is TReturn supposed to be in this context?

It allows iterators to return different type of results when the iteration is done. In most cases I don't think we need to worry about TReturn when we just need the results.

I need to get at nextPartitionKey and nextRowKey

I don't know about these two. @joheredi?


Bower no longer using a package's bower.json version

Hi, apologies for the issue spam.
We've come across an issue with the more recent versions of bower (~1.6.5) where the version number set in a package's bower.json file isn't used when installing/referencing that package. We hit this when trying to set a specific version on bi-app-sass (see https://github.com/edx/ux-pattern-library/pull/223).
Here's the output from our repo's bower.json file ():
...
"dependencies": {
    "animate.css": "~3.3.0",
    "bi-app-sass": "~1.1.0",
    "bourbon": "~4.2.3",
    "breakpoint-sass": "~2.6.1",
    "requirejs": "~2.1.15",
    "susy": "~2.2.5"
  }
...

With that in mind, here's the output when I try to run bower install with v1.6.5:
$ bower install

bower bi-app-sass#~1.1.0    not-cached git://github.com/anasnakawa/bi-app-sass.git#~1.1.0
bower bi-app-sass#~1.1.0       resolve git://github.com/anasnakawa/bi-app-sass.git#~1.1.0
bower bi-app-sass#~1.1.0  ENORESTARGET No tag found that was able to satisfy ~1.1.0

Additional error details:
No versions found in git://github.com/anasnakawa/bi-app-sass.git

We came across this with our Bower package recently and found that Bower now obtains and manages versions by git tags/releases. We created our first release on Github with a corresponding version number (https://github.com/edx/ux-pattern-library/releases) and our package was able to be used externally again with the bower install command.
The same approach might be what bi-app-sass needs to migrate to.

Let me know if you need any more info or if there's something I can do to help.

If I got this right, mainly the issue happened with bower because the version mentioned in bi-app-sass is set to 1.1.0, and this tag is not available in the repository, right ?
if that's the case, I have just created a new tag & updated npm registry as well.

@anasnakawa, yep! That's it. Apologies for my long way around the problem. I can verify that Bower now has the up-to-date package registered - http://bower.io/search/?q=bi-app-sass.
Thanks much!


Reccuring payments/Subscription based product type

Had a look in development roadmap and couldn't see subscriptions. Is this something that is possible either as an additional add-on or built into the community store core?
It would be very useful.

There currently is the ability to create membership style setups, where a product purchase triggers the creation of a new user account, which then can be put into one or more user groups. Then those user groups can have expiry times.. so you can set up paid access sites where you might want to offer time controlled access to certain resources.
But actual recurring payments is actually quite a separate exercise, it requires the definition of the subscription rules and the way transactions are put through is different. You also need ways for customers to manage their subscriptions, update out-of-date cards, put in cancels, or even changing between subscription types.  Then the actual subscription emails notifications, like the ones that say 'Your card is going to expire soon', or even just the recurring invoice don't generally come from the payment system, the store would have to listen for those notifications and handle them.
You also can't just set up subscriptions with any payment gateway, they have to be ones where the card is stored/tokenized on their end for the purpose of repeating a payment. By design, all the transactions community store puts through are immediate ones, where the transaction is authorized by the card details are not actually stored for re-use.
Stripe's subscriptions API is I'd say the go-to option at the moment, but it's not just a matter of adding a few new options in, it's a whole different process to be honest. It would have to be a separate add-on I believe, one with it's own dashboard pages.. in fact it might not even have any connection with community store, as they're just not enough overlap of functionality.
(that being said, I've spoken to a developer who thinks it is possible to integrate, so something may eventuate)

Thanks Ryan. I knew about store events and user groups having expiry times, which is very useful. I figured it wasn't a straight forward process when different payment gateways have different levels of support for recurring payments out of the box.
Great to know that you're thinking about it.
Ben

Hi, Has anyone done any work on reccuring payments?  I have gotten several request recently from people interested in subscription services with Concrete5 web sites. It may be a good feature to have.

I think my comment above is still applicable to the whole concept.
Reoccurring payment processing is really quite different to the process of completing a single transaction in a typical online shop. If something was to be built I'd actually be thinking it would be it's own completely seperate package, there really isn't much overlap in functionality.


[purview] fix name too long pylint

the python generator will soon generate with these ignores so it's okay to edit the gen code for now. https://github.com/Azure/autorest.python/issues/2021

API change check
API changes are not detected in this pull request.


Spark -> Apache Spark

[Current coverage][cc-pull] is 90.77%

Merging [#124][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #124   diff @@
==========================================
  Files            14         14
  Lines           661        661
  Methods         615        615
  Messages          0          0
  Branches         46         46
==========================================
  Hits            600        600
  Misses           61         61
  Partials          0          0

Powered by Codecov. Last updated by [19eb277...aabe4b4][cc-compare]
[cc-base-branch]: https://codecov.io/gh/databricks/spark-xml/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/databricks/spark-xml/compare/19eb2779cf232dc0239e54bf353fa609c8b05231...aabe4b4f29d3a4ece2ae18af45defa47e49d004b
[cc-pull]: https://codecov.io/gh/databricks/spark-xml/pull/124?src=pr

LGTM