feat: escape pod startup customized animation

This is a cool idea, but I think it should remain its own project. It could maybe be included in wire-prod-pod


Add a testing lang

We're developing increasingly complex tests with intricate setups. It makes sense to start thinking about a testing language, similar to the one in cbc-casper.
@DavidKnott suggested that a test vector in this language could look somewhat like:
Deposit1[Owner1, Amount1] Transfer1[Deposit1, Owner2, Amount1] Withdraw[Transfer1, Owner2]
This basically means that we could design a single set of test vectors to be parsed/tested by each implementation.

Closed by #92


How to evaluate accuracy of LiDAR localizer ? Is there any way to generate ground truth .

Hello all,
I have successfully tested the localization module in the autoware using our own custom map and bag file. From visualization, it is clear that ndt_matching working well, But I am not sure the accuracy of the matching algorithm.  So how can I evaluate the accuracy of the ndt_matching? How can I consider the ground truth results?
Could anyone please suggest me?
Kind regards,
Ajay

Hello，have you solved your problem?

Thanks for using Autoware and for your question. However the issue tracker is for posting confirmed bugs and feature requests.
We ask that you please ask questions on the Autoware Discourse category following our support guidelines: https://github.com/CPFL/Autoware/wiki/Support-guidelines. Please pay particular attention to the information we ask you to provide.


DMN plugins support both cloud and platform

This will be followed up with a PR in the plugin helpers repo.
Closes #2854

Follow-up PR: https://github.com/camunda/camunda-modeler-plugin-helpers/pull/24


[BUG] [B2bInputList] On "disabled", content still can be cleared

Describe the bug
After an input list has been set to disabled, it is still possible to remove the content using the X. Is there a way to deactivate this as well, or are there reasons why this is not currently the case?
Expected behavior
After disabling the input list, also the X should be disabled.
Screenshots
https://github.com/otto-de/b2b-design-system/assets/150906584/0655bb0b-6846-4c35-9eb9-04e3b108b9d3
Framework Used

local development

Package

Version 1.6.1
@otto-de/b2b-react-components'

Hi @thonit,
thanks for your submission! This issue will be fixed in the next release. Check our teams channel to stay updated on upcoming features!


Data parameter from Operator class does not show properly in API reference

Environment

Qiskit Terra version: N/A
Python version: N/A
Operating system: N/A

What is happening?
The part of the Operator class API reference page from qiskit.quantum_info in which the parameters are listed does not show properly. In particular, where the data argument is supposed to be the page reads:

Parameters
or (data (QuantumCircuit) – Instruction or BaseOperator or matrix): data to initialize operator.

Instead of

Parameters
data (QuantumCircuit or Instruction or BaseOperator or matrix): data to initialize operator.

However I don't see any problem with the source code:
def __init__(self, data, input_dims=None, output_dims=None):
        """Initialize an operator object.

        Args:
            data (QuantumCircuit or
                  Instruction or
                  BaseOperator or
                  matrix): data to initialize operator.
            input_dims (tuple): the input subsystem dimensions.
                                [Default: None]
            output_dims (tuple): the output subsystem dimensions.
                                 [Default: None]

How can we reproduce the issue?

Open the Operator class API reference page.
Check the parameters section

What should happen?
You should see the aforementioned error.
Any suggestions?
No response

Thank you for finding the bug!

As a follow up, could there be an issue with how the docstring is formatted? I am not very familiar with how the docstring is parsed using Napoleon (a Sphinx extension), but maybe the colon should be on the same line as data!? Maybe something like this:
Args:
      data (QuantumCircuit or Operation or BaseOperator or matrix):
                          data to initialize operator.
      input_dims (tuple): the input subsystem dimensions.
                          [Default: None]
      output_dims (tuple): the output subsystem dimensions.
                           [Default: None]

Note also that due to some recent update in the operator.py file, the correct line that should appear in the API reference page is

data (QuantumCircuit or Operation or BaseOperator or matrix): data to initialize operator.

That is, Instruction has been replaced by Operation.

@dalin27: yeah, I would expect that Sphinx's parser is probably choking because it's not finding the colon.  Putting it on one line should help out.
The online documentation still says Instruction because the online documentation is for Terra 0.21.2, and the Operation interface is only on the main branch, and planned to be released in Terra 0.22.  We're not planning to release a 0.21.3 patch (unless something super critical happens very soon), so any documentation fix would go out with 0.22.


Use custom headers for http requests

Provide us a use case of the feature
Sometimes, when I use a lot instaloader with the same account, my account is locked until I type a code sent via email or until I change my passeword. I think this is because the headers sent in the http requests by instaloader are different than the headers I send with my browser. Instagram detects instaloader as a Linux os, which is not commun and triggers the security protocol.
Describe the solution you'd like
Currently, we instance the loader like this L = instaloader.Instaloader()
One should be able to precise custom headers like this
 DefaultHeaders = {'Accept-Encoding': 'gzip, deflate',                   'Accept-Language': 'en-US,en;q=0.8',                   'Connection': 'keep-alive',                   'Content-Length': '0',                   'Host': 'www.instagram.com',                   'Origin': 'https://www.instagram.com',                   'Referer': 'https://www.instagram.com/',                   'User-Agent': 'Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 ' \            '(KHTML, like Gecko) Chrome/92.0.4515.131 Safari/537.36',                   'X-Instagram-AJAX': '1',                   'X-Requested-With': 'XMLHttpRequest'} L = instaloader.Instaloader(headers = DefaultHeaders)
where headers is an optional argument.
What do you guys think about it?

I think this is because the headers sent in the http requests by instaloader are different than the headers I send with my browser. Instagram detects instaloader as a Linux os, which is not commun and triggers the security protocol.

There is an option --user-agent. Are there any other headers different from what your browser sends?


West US datacenter not responding for latency test

Selecting West US as a data center has no impact on the graph and does not show a latency result.

Thanks for raising this up. This is a regression introduced from recent subscription move. Issue has been fixed.

Awesome, thanks a lot.


Aapt shared library

Hello!
Do you have plans to build a library into a .so  file?

Can you share the error that you got? Is exit code 132 that came from aapt or was this something else?

New version Aide not used .so

example : Runtime.getRuntime().exec(new String[] {Context#getApplicationInfo().nativeLibraryDir, "--help"});

I tried running libaapt2.so (arm64-v8a) using System.load(/data/data/packageName/files/x86/lib/libaapt2.so) on x86 API 30 AVD Emulator. I get the following Error in logs:
java.lang.UnsatisfiedLinkError: dlopen failed: library "libprotobuf-cpp-lite.so" not found: needed by /data/data/packageName/files/x86/lib/libaapt2.so in namespace classloader-namespace
        at java.lang.Runtime.load0(Runtime.java:939)
        at java.lang.System.load(System.java:1628)

Maybe this is related to it. Any idea on how can I run it? @JonForShort @TimScriptov
Thanks


No saved changes?

I ask for help, I can’t understand why updating records stopped working in V5, in version 4 everything worked fine. In V5, the code is executed. Update signals that changes have been made and re-selection returns the object with the change, but after the connection is destroyed, changes are not made to the database?
Try usage V5 5.0.0 nuget
Try usage V4 4.1.4 nuget
void Main()
{
	Run();
	Run();
}

private void Run()
{
	using (var db = new LiteDB.LiteDatabase("FileName=test.db;update=true"))
	{
		var cols = db.GetCollection<User>();
		if (cols.Count() == 0)
			cols.Insert(new User() { Name = "Ivan" });

		var user1 = cols.FindOne(x => x.Name == "Ivan");
		Console.WriteLine($"User1.Age: {user1.Age}");
		user1.Age++;
		cols.Update(user1);
		var user2 = cols.FindOne(x => x.Name == "Ivan");
		Console.WriteLine($"User2.Age: {user2.Age}");
		Console.WriteLine($"-----");
	}
}

public class User
{
	public LiteDB.ObjectId Id { get; set; }
	public string Name { get; set; }
	public double Age { get; set; }
}

V5 Return:

User1.Age: 0
User2.Age: 1
User1.Age: 0
User2.Age: 1

V4 Return:

User1.Age: 0
User2.Age: 1
User1.Age: 1
User2.Age: 2

@DevShip Could you test it with v5.0.1?

@DevShip Could you test it with v5.0.1?

Excellent! Thanks for the responsiveness, the errors were fixed in v5.0.1 and the error was corrected in mode = Share on Netcore 3.1, which led to blocking when .Update () now I was able to return the script with one LiteDabase without reopening the LiteDabase.


Include the plus (+) character in objcImported

This character is commonly used when naming Objective-C category header
files (e.g. NSArray+Additions.h).

I attempted to contact the listed maintainer regarding this change, but I have yet to receive a response. Because this is a small and hopefully uncontroversial change, I was hoping it could be included in time for version 8.2.

Codecov Report

Merging #5321 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5321      +/-   ##
==========================================
+ Coverage   82.23%   82.24%   +<.01%
==========================================
  Files         134      134
  Lines      147582   147582
==========================================
+ Hits       121366   121378      +12
+ Misses      26216    26204      -12

Impacted Files
Coverage Δ

src/gui_gtk_x11.c
58.11% <0%> (-0.3%)
:arrow_down:

src/drawline.c
84.64% <0%> (-0.14%)
:arrow_down:

src/gui.c
63.78% <0%> (-0.06%)
:arrow_down:

src/window.c
88.3% <0%> (ø)
:arrow_up:

src/os_unix.c
63.68% <0%> (+0.22%)
:arrow_up:

src/profiler.c
94.67% <0%> (+0.56%)
:arrow_up:

src/ui.c
72.4% <0%> (+1.19%)
: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 c95e8d6...c95954b. Read the comment docs.

@nuko8

@nuko8 @brammool what do you think about including this small improvement?

Kazunobu has been responsive, lets wait a bit.

@nuko8 @brammool wanted to follow up on this now that a few months have passed.

Looks like Kazunobu has disappeared.  I'll include this now.


Added the tutorial "Find in buffer" (1979)

Added the tutorial suggested here

This is awesome @muream - exactly what I had in mind for it 👍 Super cool and a great example for other people who want to write tutorials. Thank you for putting this together!

Just given the tutorial a test, works well!
Going to be great to have a big suite of tutorials to learn from!

I'll bring this in since I'm wrapping up things for the 0.3.2 release - want to make sure this makes it in! 👍

Thanks for the PR and work on this @muream - great job! I always forget to use ? so maybe this will help me practice.


brew search doesn't use HOMEBREW_GITHUB_API_TOKEN when searching casks

When using simply brew search something, brew automatically also searches available casks. The problem is that sometimes an error occurs when it gets to the searching casks part:
$ brew search something
No formula found for "something".
==> Searching pull requests...
Error: GitHub API Error: API rate limit exceeded for XXX.XXX.XXX.XXX. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)
Try again in 55 minutes 9 seconds, or create a personal access token:
  https://github.com/settings/tokens/new?scopes=&description=Homebrew
and then set the token as: export HOMEBREW_GITHUB_API_TOKEN="your_new_token"

This happens even though the HOMEBREW_GITHUB_API_TOKEN variable is set.
Using brew cask search something has no problem searching through the available casks.

Should be reported to homebrew, not us


Remove identity operators from oneDNN optimized graph

Description
This property removes dropout and copy operators from optimized graph for oneDNN - it doesn't affect performance by a lot, but allows to fuse other patterns which wouldn't be fused if there was identity operator between.

@mxnet-bot run ci [unix-gpu, centos-gpu]

@mxnet-bot run ci [unix-gpu, centos-gpu]

@mxnet-bot run ci [unix-cpu, unix-gpu, windows-gpu]

@mxnet-bot run ci [unix-gpu, centos-gpu, windows-gpu]

@mxnet-bot run ci [centos-gpu]

@mxnet-bot run ci [centos-gpu]


[21.05] mill: 0.9.6 -> 0.9.9

Motivation for this change
Backport #130823
cc @scalavision
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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 nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase 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

[x] Fits CONTRIBUTING.md.

@scalavision approved the previous PR, and this is a compatible bump

Thanks! Sorry I didn't see this earlier, probably mixed it up with the previous one.


how to start the kube api server with https?

Hi , I'm trying to connect the kube-apiserver by https, this is the info when I start the api server：
$ ./kube-apiserver --insecure_port=8080  --secure-port=8081 --etcd_servers=http://<IP_ADDRESS>:4001 --logtostderr=true --service-cluster-ip-range=<IP_ADDRESS>/24 --client-ca-file=/home/vcap/certs/ca.crt --tls-cert-file=/home/vcap/certs/server.crt --tls-private-key-file=/home/vcap/certs/server.key

when I use the same ca.cert file to connect the api server in another machine I got the following mistake:
$ curl --cacert ca.crt https://k8master:8081/api/v1beta3/namespaces/default/pods -v
* Hostname was NOT found in DNS cache
*   Trying <IP_ADDRESS>...
* Connected to k8master (<IP_ADDRESS>) port 8081 (#0)
* TLS 1.2 connection using TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA
* Server certificate: k8master
* Server certificate: k8master
> GET /api/v1beta3/namespaces/default/pods HTTP/1.1
> User-Agent: curl/7.37.1
> Host: k8master:8081
> Accept: */*
>
< HTTP/1.1 401 Unauthorized
< Content-Type: text/plain; charset=utf-8
< Date: Sun, 21 Jun 2015 02:52:22 GMT
< Content-Length: 13
<
Unauthorized
* Connection #0 to host k8master left intact

Could any one give me some suggestions? Than you!

The --cacert option tells curl to use the provided certificate authority to verify the server cert that the apiserver present.
The HTTP/1.1 401 Unauthorized you get is because the secure endpoint on the apiserver requires that the client present some form of authentication. That can be done through client-certificate, bearer token, or HTTP basic auth.You can read more about those three options in authentication.md.
Personally, I think the easiest way is using a bearer token.
You can create a file tokens.csv containing a token for your client to use:
abcdTOKEN1234,wangzhezhe,wangzhezhe

The token can be anything. On gce, we generate random tokens for clients.
Start your apiserver with --token_auth_file=tokens.csv.
Then add -H "Authorization: Bearer abcdTOKEN1234" to your curl command, and the apiserver should correctly authenticate you as user wangzhezhe.

Thank you for your suggestions! there are thee questions:

how should I config the port of api server when I try to use the token to make an authentication? Actually I try to start the api server like this :

./kube-apiserver  --insecure-bind-address=<IP_ADDRESS> --insecure-port=8080 --bind-address=<IP_ADDRESS> --secure-port=8081 --etcd_servers=http://<IP_ADDRESS>:4001 --logtostderr=true --serce-cluster-ip-range=<IP_ADDRESS>/24  --token_auth_file=/home/vcap/certs/tokens.csv

than I connect the server :

You still need to specify --cacert with your curl command. Otherwise, curl will use your OS's default CA bundle (which the apiserver's cert will not trace back to).
Your full command should look like:
curl --cacert ca.crt -H "Authorization: Bearer abcdTOKEN1234" https://k8master:8081/api/v1beta3/namespaces/default/pods -v

Thanks for your advice!
It seems ok to add cacert and -H config in curl command at the same time .
This is my info to start the api-server
sudo ./kube-apiserver  --insecure-bind-address=10.10.10350 --insecure-port=8080 --bind-address=<IP_ADDRESS> --secure-port=8081 --etcd_servers=http://<IP_ADDRESS>:4001 --logtostderr=true --service-cluster-ip-range=<IP_ADDRESS>/24  --token_auth_file=/home/vcap/certs/tokens.csv --client_ca_file=/home/vcap/certs/ca.crt --tls-private-key-file=/home/vcap/certs/server.key --tls-cert-file=/home/vcap/certs/server.crt

In my case, I userIf I want to use the cert created by system in /var/run/kubernetes ,  what ca.crt file should I use ?
Thank you !

Oops I didn't see part 3 of your question.
With the self-signed cert, the apiserver cert and the certificate authority are the same, so you would just set --cacert /var/run/kubernetes/apiserver.crt (or wherever you copy that cert to).

sorry , some spell mistakes in curl command , Its  Bearer but not Beare
curl --cacert ca.crt -H "Authorization:Bearer abcdTOKEN1234" https://k8master:8081/api/v1beta3/namespaces/default/pods -v

@wangzhezhe   Can you teach me  how to create .crt and key?  what should '-subj' be ??

@fzu-huang maybe this will help you: http://localhost:4000/blog/2015/08/05/httpsandgolang/

@wangzhezhe 谢谢！
so  what i need to do is  just edit a conf and load it when i create key and crt  by openssl~?
i'll try it !


test against xspec failing

Describe the bug
The test against XSPEC is failing due to the newest XSPEC having a different file structure.
This should be fixed with https://github.com/threeML/astromodels/pull/167
And then it will work in 3ML again

fixed


Fix documentation error with zkUrl credential format

@timcharper should we also backport this?

@alenkacz I'm inclined to not, since it has been wrong for multiple years :)


Google Map conflict when Google and PTC have same username

Expected Behavior
2 bots will be shown in map uniquely
Actual Behavior
Only 1 bot shown and it will feed from both account location
Steps to Reproduce
1.Create PTC account with the same username of your google account.
2.enter username in config/userdata.js
Other Information
OS: Ubuntu Xenial
Git Commit: 6ba5609c6151507b5b832a74e471b6b7b1a182c9
Python Version: 2.7.12

current solution by using full email address for google account to make both account unique

Could you please share your config file ?

var userInfo = {
        users: ["myGoogleAccountTheSameWithPTC"],
        userZoom: true,
        zoom: 16,
        userFollow: true,
        gMapsAPIKey: "AIzaSyCgkeW_uop1YaDXmWUMlH13reiyr8vkEKQ",
        botPath: true,
        actionsEnabled: false
};

Your current solution of using the full email address results in correct functionality. Not clear what you're requesting.


Module system improvements

This branch change mainly 2 things, the way composite types are processed, and the way internal processing of modules is done.

Property evaluation within composite types: This change is a highly demanded feature, which tend to confuse everybody who used properties with submodules in the past.  Composite types such as types.attrsOf, types.loaOf and types.listOf are now able to have properties within their bodies.  The most interesting use-case involve submodule, such as jobs or file systems options:
{config, lib, ...}:

with lib;

{
  options = { … };

  config = {
    jobs.ircSession = mkIf config.….enable {
      …
    };
  };
}

This example used to evaluate a submodule with an empty attribute set for unique submodule.  With this change to the evaluation of types, which evaluate properties while merging the types, this problem no longer exists as it is equivalent to having no value to be merged.  This implies that no submodule would be evaluated, and no empty job would be created.

Argument unification between modules and submodules: This change is a long standing sanity issue which mostly prevented one person to use modules as submodules.  Arguments are now processed by an internal option _module.args, which is given to each module.  This implies that any set of modules which can be build in a standalone manner (such as maybe NixUP one day) can also be included as a submodule even if it relies on extra arguments, as long as these arguments are used for the computation of the definitions only.
This open the door for a feature which I have not yet heard many people mention, but that I wanted to add for a while, which is to share library functions to build the content of option definitions.  For example, the activation script logic can move to the activation script module instead of being part of nixpkgs.
{config, lib, ...}:

{
  _module.args.activationLib = {
    mkActivationWithDep = deps: str: …;
  };
}

In addition, to this change, the computation of the pkgs argument of NixOS modules moved from being computed out-side the module system to be computed within the module system.  This implies that nixos/lib/eval-config.nix no longer has to execute the module system twice.  The first one with a dummy pkgs argument, and the second time with the one computed from the first iteration.  This approach is better as it avoid this work-around.

Sadly this last change is coming with a few non-backward compatible changes, which are either not well known, or easy to fix:

mkOption, mkIf:  These functions are part of the engine of the module system.  As the pkgs argument is no longer a special argument, using pkgs.lib.mkOption will result in an infinite loop during the evaluation.  Thus module should be written as { config, pkgs, lib, ... }: instead of { config, pkgs, ... }:, pkgs can still be kept for finding packages, but functions should use the lib argument.  Thus pkgs.lib.mk… should be replaced by lib.mk…, and identically with pkgs.lib; should be replaced by with lib;.

Deprecate the modulePath argument: This attribute is not well known, nor used.  It was introduced before the addition of the NIX_PATH environment variable as far as I can recall.  From what I can tell, the only purpose of it was to refer to nixos modules which are not included by default within your own the configuration files.  This can be done as follow:
{
   imports = [ <nixos/modules/profiles/headless.nix> ];
}

Explicit argument of modules: Due to the strictness of the deconstruction of attribute set, modules arguments are computed in a way which is similar to the callPackage function.  This implies that one cannot write a module which expects any argument custom argument — other than config, lib and options — which is not explicitly named.
{ ... }@args:

{
  foo.description = "${args.name} thing";
}

This example lead to the fact that name is not found on the attribute set args.  On the other hand, if the attribute is made explicit, then it would be visible.  This is counter intuitive, but hopefully this is not a common pattern.
{ name, ... }@args:

{
  foo.description = "${args.name} thing";
}

This work is based on the work made by @shlevy in pull request #2563.  Reviewed carefully, rebased, and fixed.

:+1:

Wow, this looks pretty nice! :+1:

+1
Did i read correctly, that with this patch you can use modules as submodules? Can you please give an example config for that?

@offlinehacker , Indeed I have not tried it yet, but I guess I could write something like:
{ lib, ... }:

with lib;

{
  options.machines = mkOption {
    type = types.submodules {
      imports = import <nixos/modules/module_list.nix>;
      options = {};
      config = {
         _module.args.system = mkDefault builtins.currentSystem;
      };
    };
  };
}

Even if this is one step in the right direction, this set of patches lacks one feature to make the services option an alias of the service abstraction layer, the same problem exists for the multi-instances issues.

The module system test cases can be tested by using the following command:
$ nix-build ./pkgs/top-level/release.nix -A lib.tests --no-out-link

which prints the following:
these derivations will be built:
  /nix/store/1cy0mxcnriwi6mam565v8k8j6d62lfmh-nixpkgs-lib-tests.drv
building path(s) ‘/nix/store/vm40zm9mb8019y18ggrly32lfrg2rlsw-nixpkgs-lib-tests’
====== module tests ======
31 Pass
0 Fail
/nix/store/vm40zm9mb8019y18ggrly32lfrg2rlsw-nixpkgs-lib-tests

I'll try to have a look at this later this week.

@nbp nice, thanks :)

Does this break modules that accept arguments with default values?
My configuration.nix starts with:
{ config, pkgs, lib
, rg_path ? "/root/src/rg"
, ... }:

let
  rgpkg = import "${rg_path}/default.nix" {};
in

After this PR was merged (I think it was this one, but I could be wrong), I get this error with nixos-rebuild dry-build:
building the system configuration...
error: while evaluating the attribute ‘config.system.build.toplevel’ at "/nixdev/nixpkgs/nixos/modules/system/activation/top-level.nix":228:5:
(...)
while evaluating the attribute ‘value’ at "/nixdev/nixpkgs/lib/types.nix":119:38:
while evaluating the attribute ‘_module.args."${name}"’ at "/nixdev/nixpkgs/lib/attrsets.nix":135:44:
attribute ‘_module.args.rg_path’ missing, at "/nixdev/nixpkgs/lib/modules.nix":138:17

@nbp Could you add a note to the NixOS 15.05 release notes about the pkgs.lib to lib change (and the "Infinite recursion" symptom)?

@nbp will you add these changes to the changelog?

I think I already made a branch for adding these to the release notes … I will try to find it again.

Indeed I had a branch, but I might have forgot to merge it:
https://github.com/NixOS/nixpkgs/compare/master...nbp:doc-6794
I will open a pull request and cc you.

Thanks (would be awesome if you can do that now, I want to release today)
On Wed, Sep 30, 2015 at 9:07 PM, Nicolas B. Pierron <
<EMAIL_ADDRESS>wrote:

Indeed I had a branch, but I might have forgot to merge it:
master...nbp:doc-6794
https://github.com/NixOS/nixpkgs/compare/master...nbp:doc-6794
I will open a pull request and cc you.
—
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/pull/6794#issuecomment-144509322.

This makes (listOf (attrsOf str)).check [ "foo" ] evaluate to true, was that intended? This is a problem because then check-meta.nix fails to fail when meta.maintainers is not in the right format (see https://github.com/NixOS/nixos-search/issues/391 for discussion).
Would it be too much of a performance impact to reintroduce the correct checks for listOf and attrsOf? In this case we might simply make check-meta.nix aware of this by performing deeper checks.
cc @sternenseemann

In this case we might simply make check-meta.nix aware of this by performing deeper checks.

I think doing the check manually (and correctly) in check-meta.nix is the way to go. Reverting this change 7 years later is not a good idea: I know from experience that properly type checking in Nix incurs a significant performance penalty, so I'd expect a revert to cause a very noticeable performance regression in the already slow module system. The typechecking limitation should probably be communicated better, but having parametrized types makes sense for documentation purposes.


Weapon Scanning does not scan the first row and it cannot be changed later in GO (they all get a dull sword)

This is mostly on GO's end. If GO imports a character without an assigned weapon it defaults to Dull Blade regardless of character weapon type. A partial fix for this is coming soon to Kamera.

This issue been fixed by the latest release or prerelease so I am going to close it. Please reopen this issue if the problem persists.


Camera setTint does not seem to do anything

First, I want to say Phaser is really awesome, you guys have a lot of excellent features, I love going thorough the docs trying to find new tricks and features.
Version

Phaser Version: v3.24.1
Operating system: Mac OS
Browser: Chrome 85

Description
I saw camera had setTint, was expecting to use it to give a night time effect. But it does not seem to do anything.
Example Test Code
        this.input.keyboard.on('keydown-' + 'N', () => {
            this.cameras.main.setTint(0x0000aa);
            this.time.delayedCall(3000, () => {
                 this.cameras.main.clearTint();
            });
        });

Additional Information
Seems to pull in the Tint component here:
https://github.com/photonstorm/phaser/blob/v3.22.0/src/cameras/2d/Camera.js#L61
But if it has no real effect on the camera, I wonder if it makes sense to even mix in.

Hmm, no it doesn't use the Tint component at all. It needs removing.

Just looked at this further and tint is used when the Camera is rendering to a texture. So for this reason, I will leave it in.


Need clarification for Screenshot configuration

@jan-molak
Need a clarification, is there any screenshot configuration in Serenity-js (V1) like Serenity (Java version)
serenity.take.screenshots.for.tasks=after_each_step
serenity.take.screenshots.for.interactions=for_failure
serenity.take.screenshots.for.questions=after_each_step
Side Note: I am using :
"serenity-cli": "^0.1.4",
"serenity-js": "1.10.13",

Hey @abhinaba1080! Yes, Serenity/JS V1 has a mechanism similar to Serenity BDD.
Here are the specs, here's an example protractor.conf.js, and here's how it works:
// protractor.conf.js

const crew = require('serenity-js/lib/stage_crew');

exports.config = {

    framework: 'custom',
    frameworkPath: require.resolve('serenity-js'),

    serenity: {
        crew: [
            crew.Photographer.who(_ => _
                .takesPhotosOf(_.Tasks_and_Interactions)  // or _.Failures
                .takesPhotosWhen(_.Activity_Finishes)     // or _.Activity_Starts_and_Finishes, or _.Activity_Starts
            ),
            // ...
        ]
    },

    // other config omitted for brevity
}

Hope this helps!
Jan

Hey @abhinaba1080! Yes, Serenity/JS V1 has a mechanism similar to Serenity BDD.
Here are the specs, here's an example protractor.conf.js, and here's how it works:
// protractor.conf.js

const crew = require('serenity-js/lib/stage_crew');

exports.config = {

    framework: 'custom',
    frameworkPath: require.resolve('serenity-js'),

    serenity: {
        crew: [
            crew.Photographer.who(_ => _
                .takesPhotosOf(_.Tasks_and_Interactions)  // or _.Failures
                .takesPhotosWhen(_.Activity_Finishes)     // or _.Activity_Starts_and_Finishes, or _.Activity_Starts
            ),
            // ...
        ]
    },

    // other config omitted for brevity
}

Hope this helps!
Jan

Really helped. Thanks a ton!

@jan-molak : can you suggest the same for Serenity-js 2:
I have recently migrated to serenity-js 2 . It works for me but there is no screenshot in the report.
How to modify the below to take screenshots in different scenario:
serenity: {
crew: [new ArtifactArchiver(new FileSystem(new Path('./target/site/serenity'))), new SerenityBDDReporter(), new ConsoleReporter()]
}

@abhinaba1080 - sure thing!
const { Photographer, TakePhotosOfInteractions } = require('@serenity-js/protractor');

// ...
serenity: {
  crew: [
    ArtifactArchiver.storingArtifactsAt('./target/site/serenity')),       // <- there's a factory method for this one now
    new SerenityBDDReporter(),
    Photographer.whoWill(TakePhotosOfInteractions), // or TakePhotosBeforeAndAfterInteractions, or TakePhotosOfFailures,
  ]
}

Does this help?

 :)

It really helps.. Just a quick check: do we have something like @types/serenity-js or something like that for typescript support? i want to migrate all require statements into imports.

Photographer.whoWill(TakePhotosOfInteractions)

Getting error.
[12:57:31] E/configParser - Error code: 105
[12:57:31] E/configParser - Error message: failed loading configuration file transform/config/browser/chrome.config.js
[12:57:31] E/configParser - TypeError: Photographer.whoWill is not a function
    at Object.<anonymous> (C:\Dev Room\Projects\ArcherBDDUITests\transform\config\browser\chrome.config.js:22:26)
    at Module._compile (internal/modules/cjs/loader.js:701:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:712:10)
    at Module.load (internal/modules/cjs/loader.js:600:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:539:12)
    at Function.Module._load (internal/modules/cjs/loader.js:531:3)
    at Module.require (internal/modules/cjs/loader.js:637:17)
    at require (internal/modules/cjs/helpers.js:22:18)
    at ConfigParser.addFileConfig (C:\Dev Room\Projects\ArcherBDDUITests\node_modules\protractor\built\configParser.js:135:26)
    at Object.initFn [as init] (C:\Dev Room\Projects\ArcherBDDUITests\node_modules\protractor\built\launcher.js:93:22)

Photographer.whoWill(TakePhotosOfInteractions)

@jan-molak Now i have two problems:

Getting error for above

[12:57:31] E/configParser - Error code: 105
[12:57:31] E/configParser - Error message: failed loading configuration file transform/config/browser/chrome.config.js
[12:57:31] E/configParser - TypeError: Photographer.whoWill is not a function
    at Object.<anonymous> (C:\Dev Room\Projects\ArcherBDDUITests\transform\config\browser\chrome.config.js:22:26)
    at Module._compile (internal/modules/cjs/loader.js:701:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:712:10)
    at Module.load (internal/modules/cjs/loader.js:600:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:539:12)
    at Function.Module._load (internal/modules/cjs/loader.js:531:3)
    at Module.require (internal/modules/cjs/loader.js:637:17)
    at require (internal/modules/cjs/helpers.js:22:18)
    at ConfigParser.addFileConfig (C:\Dev Room\Projects\ArcherBDDUITests\node_modules\protractor\built\configParser.js:135:26)
    at Object.initFn [as init] (C:\Dev Room\Projects\ArcherBDDUITests\node_modules\protractor\built\launcher.js:93:22)

my dependencies are:

    "@serenity-js/core": "next",
    "@serenity-js/protractor": "next",
    "@serenity-js/cucumber": "next",
    "cucumber": "^5.1.0",
     "protractor": "5.4.2",

It shows npm warning like:

npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.

Side note: my config looks like below:
import { browser, Config } from 'protractor';
import { ArtifactArchiver, ConsoleReporter, SerenityBDDReporter } from '@serenity-js/core';
import { Photographer, TakePhotosOfInteractions } from '@serenity-js/protractor';

export const config: Config = {
  // seleniumAddress: "http://INENGHOSHA1L1C:4444/wd/hub",
  // SELENIUM_PROMISE_MANAGER: false,

  framework: 'custom',
  frameworkPath: require.resolve('@serenity-js/protractor/adapter'),

  baseUrl: 'http://localhost:3000',

  allScriptsTimeout: 110000,

  specs: [rootPath + '/test/e2e/features/**/*.feature'],
  serenity: {
    crew: [
      ArtifactArchiver.storingArtifactsAt('./target/site/serenity'),
      new SerenityBDDReporter(),
      Photographer.whoWill(TakePhotosOfInteractions),
      new ConsoleReporter()
    ]
  },
  cucumberOpts: {
    require: [rootPath + '/transform/**/*.js'],
    format: 'pretty',
    compiler: 'ts:ts-node/register'
  },

  capabilities: {
    browserName: 'chrome',
    chromeOptions: {
      useAutomationExtension: false,
      args: ['--disable-gpu', '--disable-extensions']
    }

    // execute tests using n number of browsers running in parallel
    // shardTestFiles: true,
    // maxInstances: 2
  },

  onPrepare: () => {
    require('ts-node/register');
    require('tsconfig-paths/register');
    browser.ignoreSynchronization = true;
    browser.waitForAngularEnabled(false);
  }

  // restartBrowserBetweenTests: true
};

@jan-molak : can you help here please :)

Hi Jan,
Can you please look into the issue. I am totally stuck for the same. I know
, you are busy with quite a lot of work. Sorry for disturbing you.
I just love Serenity Js. And i want my organization also loves the
potential serenity-js provides.
Again sorry for any disturbance I made.
On Wed 7 Aug, 2019, 03:47 Jan Molak<EMAIL_ADDRESS>wrote:

@abhinaba1080 https://github.com/abhinaba1080 - sure thing!
const { Photographer, TakePhotosOfInteractions } = require('@serenity-js/protractor');
// ...
serenity: {
crew: [
ArtifactArchiver.storingArtifactsAt('./target/site/serenity')),       // <- there's a factory method for this one now
new SerenityBDDReporter(),
Photographer.whoWill(TakePhotosOfInteractions), // or TakePhotosBeforeAndAfterInteractions, or TakePhotosOfFailures,
]
}
Does this help?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jan-molak/serenity-js/issues/335?email_source=notifications&email_token=AFQR3UU3YENANM4WLMUZBWTQDHZ6JA5CNFSM4IIP22K2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3WT52A#issuecomment-518864616,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFQR3URZWGELWRJD2MSPXKLQDHZ6JANCNFSM4IIP22KQ
.

Hey @abhinaba1080 would you be able to create a sample repo on Github that demonstrates the problem so that I can try to reproduce it? Thanks.

Hi @jan-molak , here it is : https://github.com/abhinaba1080/Protractor-Serenity-Cucumber-BDD-BoilerPlate

@abhinaba1080 - apologies, when I suggested to use the Photographer.whoWill(TakePhotosOfInteractions) factory method I've committed the feature but haven't pushed the code 🤦♂ Sorry, schoolboy error.
It's available in the latest alpha (2.0.1-alpha.77). Please have a look and let me know what you think!

Also, the warnings you mention will only show up for the alpha versions. That's because @serenity-js/* modules have a 2.x peer dependency on other @serenity-js/* modules, and 2.x-alpha doesn't meet the 2.x requirement. It's only temporary and will go away when a "proper" release is out.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.

Hello sir,
First of all thank you for your work!
For some time now i have been trying to add screenshots to my serenity-js reports with no success.
I have followed steps written in this issue.
My protractor.conf.js is as:
const { Photographer, TakePhotosOfInteractions } = require('@serenity-js/protractor');

serenity: {
    dialect: 'cucumber',
    crew: [
        ArtifactArchiver.storingArtifactsAt('./target/site/serenity'),
        new SerenityBDDReporter(),
        new ConsoleReporter(),
        Photographer.whoWill(TakePhotosOfInteractions),
    ]
},

My steps run without any errors or issues and the final report does not have needed screenshots.
I have spent some time digging through the documentation and cant wrap my head around the actors. At this moment the project I am working on has none:
for example some feature steps:
Feature: Categories
As regular user
I want to use dance

Scenario: test
    When on Party, user clicks on "Dance" item

and implementation:
When(/^on Party, user clicks on "([^"]*)" item$/, async (value) => {
  var el = element(by.id( -- some ID goes here -- ));
  await el.click();
});

As you can see there are no actors in my project and I would like to avoid having to change implementation as it is a rather big project. The need is to have a screenshot in the report for every single step of every scenario.
Is there some part I am missing? Is it possible?
Thank you for your time and I apologize if this is a rather off-topic question.

@Cariffo is it a great question, I have the same problem, reports are generated but without screenshots. @jan-molak would you be so kind to explain us if it is possible to get reports with screenshots attached without implementing Screenplay Pattern?

@magdalenaBartkowiakJowsa ,
what's the serenity-js version you are using? Are you using protractor as your test runner or any other tool?

Hey @Cariffo, @magdalenaBartkowiakJowsa and @abhinaba1080 - I've raised #362 to remember to add this functionality. I've also suggested a workaround that you can use until that's done.


Show or log unmatched lines

It would be useful when trying to set up a new configuration to find out which files are not being matched and either display them to stdout or log them to a file. This would make it much easier to determine if any important log entries are being missed or not parsed properly.
This is probably not something that everyone would want to run in production, so I assume adding a new command-line option to enable this sort of feature would be needed.

Thanks for the suggestion. Could you create an example how the console output should look like from your perspective?

Personally I would be happy if the unmatched log lines were just printed to stdout verbatim. So far I haven't seen grok_exporter provide any output - so there doesn't seem to be any existing output that would be confused for log entries from a log file.
It might be desirable to add some sort of prefix when the log line is output, but some lines (ie. exim) tend to be pretty lengthy already, so adding prefixes might things even more unwieldy.


chore(*) support dev container

Summary

Add devcontainer.json to support Visual Studio Code Remote - Containers and GitHub Codespaces.
ref: https://code.visualstudio.com/docs/remote/containers
Tested locally.
To test: https://code.visualstudio.com/docs/remote/containers#_quick-start-try-a-development-container

Running well on my Mac M1 with VSCode containers. 🎉

tested - works.
Are the whitespace changes in the DEVELOPER.md file intended? If so this should maybe go into a separate PR(or commit)

Okay, the whitespace changes can be placed in a new PR.


SET-UP KANBAN BOARD

Dear Reviewer(s)

Here is a link to our KANBAN BOARD.
We are a group of Two @chimwemwe007 and @peterboro

Hi @peterboro @chimwemwe007 👋,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights

All information provided in issue comment 👍
Link to Kanban board working perfectly 👍

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Thank you.

Thank you.


Add release procedure

What this PR does / why we need it:
This PR, adds release procedure for cloud provider openstack repo.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

The content is mainly authored by @dims
https://gist.github.com/dims/cefbdcfbacf867c315c119d7a2491c05

cc @dims @hogepodge @lingxiankong @jichenjc @ramineni reviews and suggestions are welcome :)

/approve
/lgtm
Let's merge and iterate :)

given the discussion here:
https://etherpad.openstack.org/p/k8s-sig-ptg-train
https://etherpad.openstack.org/p/k8s-sig-ptg-train-repository-refactor
this release thing might be a little bit different?


Add minimum height touch targets to meny flyout items when in mobile view

What

Add increasing touch targets to the api of applicable buttons

Wasn't sure if we should expose a boolean (way it is currently) or open the styles up more broadly - open to feedback on this

Update composites to make use of this and increase the touch targets when mobileView is enabled

Why
For mobile users - needs to be < 48px
How Tested
Best is to see the UI updates when generated by PR. Should be nothing more desktop, but increases for mobile.
Locally:

Options button (before)
Options button (after)

People flyout (before)
People flyout (after)

To simplify the naming can we just say imply Mobile means they are also using  a touchscreen and we would want to set the minHeight?
I am not sure if we want to expose the implementation details of what mobile means

In terms of what we want to expose between Composite and Component layers.
Composite :

isMobile flag - used in composite api
designTokens - custom styles that contoso can set, terminology is geared towards end user usage and not css (potentially could be equivalent to multiple css changes)

Components

isMobile flag - each component will still use isMobile flag as well

styles - uses FluentUI style styling for components
For the context of this change, we should just be using an isMobile flag and then using mobile friendly css classes based on this prop.


Strip ANSI escape codes to compute cursor position

An attempt to fix #135
I added a module crate::ansi that parses ANSI escape codes according to this description, which is then used to skip them when calculating the cursor position on frame_finish();
Note that there are some rendering inconsistencies for ANSI codes between terminals, so it's impossible to make this work for every escape sequence in every terminal, but for non-pathological escape codes I think this implementation should be fine.
Pictured: inconsistencies between Kitty, Wezterm and Alacritty.

I've also confirmed this fixes #135 :)

Cross-platform compatibility doesn't seem like a concern, from what I've seen. Terminal emulators like Alacritty, Wezterm and Kitty apparently use the same code to parse ANSI escape sequences, no matter the OS.
Cross-terminal compatibility, however, is impossible to achieve, so I'm not sure which standard to implement... Both Wezterm and Alacritty point to this state machine for parsing escape and control sequences, but they're somehow inconsistent?? (this might be a bug in the "vte" crate)

I did overlook one thing in my implementation though: only CSI (the sequences that begin with \x1b[ or \9b in wezterm) are parsed currently. I should implement OSCs and DSCs too, in case someone wants to set their terminal's title in an inquire prompt I guess :)

Should be mergeable now 👀


Add files via upload

Contributing my first custom step called MyCustomStep01. I know this name doesn't describe the functionality, it's just a demo :-)

Please include answers to these questions as part of your pull request
For each question simply place an X  in the square brackets, [X], that represents your answer

Q1: Confirm that you have the right to submit the code that is being contributed. Please consider the origin of your code and confirm you have the appropriate rights to make the submission subject to the Apache 2.0 license that applies to everything in this repository of custom steps. If so, follow the instructions for the Contributor Agreement (which is based on the industry-standard Developer Certificate of Origin (DCO)).

[X] Yes, I have the right to submit code and agree to releasing the code under the Apache 2.0 license
[ ] No

Q2: Confirm that your contribution does not include any personally identifiable information (PII), for example, in any examples used in your README file.

[X] My contribution does NOT include PII data
[ ] My contribution includes PII data

Q3: Confirm your contribution does not include any encryption or other export-controlled technology.

[X] My contribution does NOT contain encryption or other export-controlled technology
[ ] My contribution includes encryption or other export-controlled technology

Could you please add a screenshot of the user-defined tabs you defined in your custom step, add them in your img folder, and show them in your README.md file?
Once that's been done, it looks like its ready for merging into the public repo.

Have switched on Branch Protection Rules: Require conversation resolution before Merge. Not clear yet about what conversation resolution entails. So add this comment in the conversation stream and then click the Comment button. Let's see what happens ..

Perhaps this option only comes into affect for  Pull Requests that have been created after this Branch Protection Rule has been defined. Let's test.


Error when account is expired.

Hi,
When Freshbooks account was is expired, API return :
{
  "error" : "Your Account is Expired.",
  "code" : "10060",
  "status" : "fail"
}

How catch this error on strategy omniauth ?

@maxime-d did you figure this out? I'm starting to connect to FreshBooks, so this will be useful. Thanks!


Return 0 bytes read at End of File instead of -1 for safe functions.

This is for (JCIFS, JCIFSNG, or SMBJ)ProxyFileCallbackSafe. The none safe onRead() functions return 0 bytes read at EOF. But I noticed SMBJ and JCIFS-NG return -1 to indicate EOF.
This causes lots of problems. The app trying to read a file might not expect to see -1 (wrapped to 4GB if 32bit) to be read.and complains that the file size is not expected and too big.
Without this fix, an exception would have been thrown, a very long exception with stack trace error log. CIFS Document Provider CRASHES, and then afterwards, the notification of "Opening file" shows up and there is no file opened. I had to force close CIFS Document Provider then.
I am not sure about JCIFS (SMBv1), since my TrueNAS server doesn't support SMBv1, I can't test it. But I assume it would be the same, since JAVA usually returns -1 for end of file. I tested SMBJ and JCIFS-NG, which both had the same issue.
The program I used for testing, AetherSX2 loading BIOS file SCPH10000.bin or `SCPH39001.bin" would trigger the issue. Which only happens on Safe transfer enabled.

This is before the modification:

And this is after the fix:

sorry, I missed up my git commands, closing this and recreating it.

sorry, I messed up my git commands, closing this and recreating it.


[travis] Updated an expected value in a service test of Travis PHP version badge

Currently one service test of travis fails:
1) PHP version from .travis.yml
       gets the package version of yii

	[ GET http://localhost:1111/travis/php-v/yiisoft/yii.json ]:

      AssertionError: expected { name: 'PHP', value: '>= 5.3, HHVM' } to deeply equal { Object (name, value) }
      + expected - actual

       {
         "name": "PHP"
      -  "value": ">= 5.3, HHVM"
      +  "value": "5.3 - 7.1, HHVM"
       }

Version 7.2 was added to https://github.com/yiisoft/yii/blob/master/.travis.yml on Jan 25, 2018. This version is currently the newest version of PHP https://github.com/php/php-src/releases.

Messages

:book:

:sparkles: Thanks for your contribution to Shields, @platan!

  Generated by :no_entry_sign: dangerJS

Thanks for fixing this,
Changes look good to me,
Do you think it would be worth creating a regex for this to avoid future errors?
/^(\d*(\.\d*)?\s)?(>=|-)\s\d*(\.\d*)?,\sHHVM$/

note: im not sure on the possible values that would be returned

@RedSparr0w Good idea. We already have such validator :-) https://github.com/badges/shields/blob/581546d58e2a5683db78071ba92c64afab1278c3/service-tests/helpers/validators.js#L43

Nice, Didn't realise we had that validator already.
Merged! :smile:


Retool add wallet mutation

This PR adds a private authenticator which rotates the nonce for the user and then adds the wallet (but does not verify anything e.g. a signature), but otherwise goes through the same validation a user typically would.  The mutation is decorated with the retool directive so that it can only be called by retool, and the method itself performs does the same authorization check so that it can't be called inadvertently.
It also fixes a graphQL serialization bug I noticed during testing where it couldn't marshal the GnosisSafe enum type correctly.

@radazen mind taking another pass? I moved the publicapi/admin.go into a package named adminapi. Realized that the adminapi also uses the validation helpers in publicapi so I moved those helpers into to a new package called inputcheck (open to other names 😄  )

👍 👍 👍
Discussion thoughts:

At the point where we're moving validation methods to a separate package, would it make sense to move them into our validate package? I'm on the fence; it feels like it should go there, but validate has a lot going on already and one of the nice things about having a private validateFields method in publicapi was that it was obvious what method you should use to validate API input parameters.
I was initially wondering if AdminAPI should still be part of the PublicAPI struct, or if it should be its own thing now since it's in its own package (with its own adminapi.For(ctx) etc). But after thinking about it for a while, imho it does still belong in the PublicAPI struct, because it's part of our public API surface and accessible to anyone who hits our GraphQL query endpoint. TL;DR: looks great, I think it's correct the way it is, just thinking out loud!

had pretty much the same line of thinking: originally I just wanted to exports the validation methods in publicapi, but that would've led to an import cycle. Although AdminAPI having its own struct would've resolve it, I felt that it didn't quite merit its own api yet. I was a bit back and forth on moving it to validate for the reason you mentioned.  But now that I think about it if something should really belong in the package, it probably should just go in there even if theres a lot in the namespace already. And I also don't know what else we would use inputcheck for besides a bridge to fix the imports since everything in validate are all technically input checks lol


feat: Implementation to Google Chrome

It will be awesome to have Aura Theme into Chrome!!

@r3nanp
Done! 🥳
https://chrome.google.com/webstore/detail/ddipnaombfnagpagnpdkdinoekfhfjoh


chore: update maintainers file

Summary of Changes
Proposed changes to the project maintainers.
Fixes #620
Related Issues
Please reference here any issue #'s that are relevant to this PR, or simply enter "N/A" if this PR does not relate to any existing issues.
Pull Request Checklist
Tick all boxes below to demonstrate that you have completed the respective task. If the item does not apply to your this PR check it anyway to make it apparent that there's nothing to do.

[x] All commits contain a DCO Signed-off-by line (we use the DCO GitHub app to enforce this);
[x] Updated LICENSE-3RD-PARTY.md for any added dependencies or vendored components;
[x] Updated documentation as needed for changed code and new or modified features;
[x] Added sufficient tests so that overall code coverage is not reduced.

If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!
Pro Tip 🤓

Read our contribution guide at least once; it will save you a few review cycles!
Your PR will likely not be reviewed until all the above boxes are checked and all automated tests have passed.

PR template adapted from the Python attrs project.

Reviewed on Bifold UG call. Will merge once a fix is in for Ryan's Github ID.


Enable safeHTMLAttr

Hi together,
I'm using Hugo shortcodes to generate HTML forms from JSON. I like to set HTML attributes dynamically, but was not able to do so because the attributes were replaced with "ZgotmplZ". The template functions safeHTML, etc. did not work.
After some research I found the template function "safeHTMLAttr", which was already implemented, but disabled, because the developer has seen no use case for it. See https://github.com/spf13/hugo/issues/347#issuecomment-70612848
I have now enabled the function again, because it would be really useful for me. If you like to merge it, I could also write a test and improve the documentation.
Best regards,
Marco

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

I like this PR.  We live in the Go Template world with it's features and quirks.  We should give users the tools they need to build whatever they want.

I like it, too.

Ditto.  :-)  Looking back at #347, it seems silly to me that I would have been so overly cautious back then.  So, let's merge it!  It's long overdue.
Merged as c21e2b3 after amending the commit message per the contribution guideline.
Thank you for your contribution, @webguerilla!

@anthonyfok Thank you for merging, however I have not contributed much, just uncommented your work :smiley:

@webguerilla Thank you for your compliments!  But the credit actually goes to @tatsushid who contributed this feature in the first place.


Claude sometimes returns no JSON

This happens with images that it's previously had no problem parsing.
When you inspect the returned message, instead of the JSON specified by the tool, you get something like this
"image": "https://s3.amazonaws.com/openai-assets/d9c2c3e4f0fb1c15f05d982f30622844"
Following that URL  sends you to the following XML error:
<Error>


Fixer.io alternative

Hey guys,
I believe that fixer.io was purchased by currency layer or another third party at some point. Looks like the original developer of fixer.io still has a free alternative up:
https://frankfurter.app/
Enjoy :)

wow had no idea. We should make a bit coin API and sell that too lol


Firebase.initializeApp throwing default Firebase app already exists errors in Next.js apps.

Describe your environment

Operating System version: macOS 12.6.3
Firebase SDK version: "firebase-admin": "^11.5.0"
Firebase Product: Admin SDK Firebase.initializeApp
Node.js version: v18.14.2
NPM version: 9.5.0

Describe the problem
Using Next.js SSR with Firebase consequently calls methods that call Firebase.initializeApp multiple times. This causes the following error to be thrown:
error - FirebaseAppError: The default Firebase app already exists. This means you called initializeApp() more than once without providing an app name as the second argument. In most cases you only need to call initializeApp() once. But if you do want to initialize multiple apps, pass a second argument to initializeApp() to give each app a unique name.

This issue is related to https://github.com/firebase/firebase-admin-node/issues/571#issue-457192854, but the resolution is not helpful and the supplied link (https://firebase.google.com/docs/reference/admin/node/admin.app.App.html#name) results in a 404.
Steps to reproduce:
Create a Next.js app using Server Side Rendering (SSR)
Workaround
I found a workaround by first checking if I've already initialized any apps before calling initializeApp() again:
if (!getApps().length) initializeApp();

I meet the exact same issue. Just use nextjs and it will trigger this issue

I had the same issue with Next 13, in order to fix it I did this workaround:
import { getApps, initializeApp } from "firebase-admin/app";
import { getAuth } from "firebase-admin/auth";

const alreadyCreatedAps = getApps();
const yourFirebaseAdminConfig= {}

const App =
  alreadyCreatedAps.length === 0
    ? initializeApp({yourFirebaseAdminConfig}, "app name")
    : alreadyCreatedAps[0];

Describe your environment:

Operating System:  Windows 11
Node.js version: v18.16.1
NPM version: 9.5.1
"firebase-admin": "^11.10.0"
"next": "13.4.8"

Thank you for this clever sollution

thanks for this...Next.js 13 has a lot of issues

Thanks to @MauricioBorawski 's solution, I was able to remove the error while HMR.
However, when i run pnpm dev or pnpm build, FirebaseAppError: The default Firebase app does not exist. happened.
With some workaround, I found following code snippet works
const yourFirebaseAdminConfig= {};

if (admin.apps.length === 0) {
  admin.initializeApp({ yourFirebaseAdminConfig });
}

Describe your environment:
Operating System: MacOS 14.1.1
Node.js version: v18.18.2
PNPM version: 8.11.0
"firebase-admin": "^12.0.0"
"next": "14.0.4"

I'm not sure this solution works, as far as I can tell. It will work on the dev environment, but once deployed Firebase will complain that the app needs to be initialized first. I think that conditional causes problems. By extension, I've had to make code changes to deploy the app.

I'm not sure this solution works, as far as I can tell. It will work on the dev environment, but once deployed Firebase will complain that the app needs to be initialized first. I think that conditional causes problems. By extension, I've had to make code changes to deploy the app.

How did you do for production? i am getting this error after deployment : FirebaseAppError: The default Firebase app does not exist. Make sure you call initializeApp() before using any of the Firebase services.

On Dec I have to use the conditional to check if app exists (!admin.app.length) to make sure admin.initializeApp doesn't run again, but I have to comment out the conditional before deploying as Firebase seems to fail if (I assume) the first process run isn't admin initialization.
This is the best I can figure. It works, but I means you have to make changes to the code before deploying and then back again to get dev working.

Olá, estou enfrentando o mesmo problema.
Estou usando o cron job da Vercel, eu uso o firestore do firebase e a terefa do cron é deletar dados em determinado horário.
Nisso eu preciso usar o admin SDK do google cloud para conseguir ter acesso ao firestore e excluir os dados.
Porém, em produção quando tento chamar minha função cron job exemplo: https://sitename/api/cron,    nada acontece.
ja em desenvolvimento funciona perfeitamente.

What's your logs for the https://sitename/api/cron call ?
Regards

one solution that seemed to work is calling initializeApp with the config passed to it from a different file, like an entry file that loads right after the app loads, (e.g. layout file of nextjs), I know this might not be the optimal solution but it works.

Here's a one-liner that works for me:
import { getApp, initializeApp, } from 'firebase-admin/app'
export const app = getApp() || initializeApp({})

Here's a one-liner that works for me:
import { getApps, initializeApp, } from 'firebase-admin/app'
export const app = getApps()[0] || initializeApp({})

Where do you put this one-liner?

Where do you put this one-liner?

I just make a file called firebase.ts, for instance when running on the server:
//firebase.ts
import { getApps, initializeApp } from 'firebase-admin/app'
import { getFirestore } from 'firebase-admin/firestore'

// do emulator checks and stuff here

export const serverFirebaseApp = getApps()?.[0] || initializeApp({})

export const serverFirestore = getFirestore(serverFirebaseApp)

And only ever reference these exports when using firebase:
//elsewhere.ts

import {serverFirebaseApp, serverFirestore} from './firebase'


Backend: index/mirror/cache NFT data

This issue is to generally handle the case of the frontend asset loading being slow.
The first part is that the preview images are slow (https://github.com/webaverse/app/issues/937).
The larger issue is that hitting the backend will currently read the blockchain. This is far too slow, because it usually requires multiple web3 requests to resolve to find NFT data.
Instead we should mirror all of the NFTs in a local dynamodb.
Table:
sidechain-cache
Primary Key
id: String

My suggestion that we are pushing forward is to modify the _getChainNft() to pre-check the dynamodb and return that, and send from the chain and update to db otherwise.
https://github.com/webaverse/api-backend/blob/9710d1fe01a680cdecfb930ab9175fe3f856f5ad/index.js#L1509
We'll still need to handle cache invalidation -- whenever the metadata is changed we'll need to update the db copy.
This will probably need to happen around here:
https://github.com/webaverse/api-backend/blob/6c185c73771b0e3b6f8a4b48b1ea69fc9bc5ff55/events-manager.js#L37
on these events:
A.${WebaverseNFT.slice(2)}.WebaverseNFT.Withdraw,
A.${WebaverseNFT.slice(2)}.WebaverseNFT.Deposit,
A.${WebaverseNFT.slice(2)}.WebaverseNFT.MetadataChanged,

getPastEvents is only picking up test mints like https://webaverse.com/assets/70, and they are missing most properties such as name, hash, ext and thus can't be cached.
Visiting https://tokens.webaverse.com/70 seems to resolve properly however, and missing items are cached at this point.
Otherwise most things are in place. Just stuck on cache invalidation because I can't seem to pick up the right events... Still looking. Thought I might have to use a specific contract, but mainnet, mainnetsidechain and rinkeby all give me the same results and rinkebysidechain gives an error. Not sure what the problem is yet.

Ok. Found the right combination of contracts. I think it's all working.

What's the best way to test events locally? I want to know that events.allEvents is working correctly.

You can listen to the sidechain ethereum nodes with S3, and then mint something. If you get the event, you know it's working.


How to build a branded client?

I want to build a client with a theme using the kde craft system. I know how to build a themed client using cmake with the OEM_THEME_DIR cmake parameter, but what would be the corresponding option for using ownbuild.py?
Thanks in advance!

Figured it out, closing this issue.


Distinguish between temporary versions

They're currently being aggregated under the same open TR (or LOCAL).
They should either be distinguished in the aggregated TRs or a new versions aggregated table should be added. Probably the latter.

Hum... I think I will disregard this. The temporary versions are code that is part of an unreleased TR (or belonging to a local package) after all. So I don't really see how these could be aggregated in any other way besides the TR.


Documentation: Update documentation for Infineon boards

Documentation: Update documentation for Infineon boards
-Update formatting and contents of index.rst for cy8ckit_062s4
-Update formatting and contents of index.rst for cy8ckit_064s0s2_4343w
-Update formatting and contents of index.rst for cy8cproto_062_4343w
-Update formatting and contents of index.rst for cy8cproto_063_ble
-Update formatting and contents of index.rst for xmc45_relax_kit
-Update formatting and contents of index.rst for xmc47_relax_kit
-Change all instances of "PSoC" to "PSOC" for infineon platforms

I would suggest not touching the past release notes, if that's ok with you. Unless really necessary I think it's best to try and keep these files untouched as they're by definition meant to be reflecting how things were at release time

I would suggest not touching the past release notes, if that's ok with you. Unless really necessary I think it's best to try and keep these files untouched as they're by definition meant to be reflecting how things were at release time. This is actually why these files are not compliant from a .rst perspective as I didn't want to "touch" them just for the sake of making them compliant when I introduced the ReST linter

@kartben, that makes total sense. I have changed the PR to reflect this. Thank you for your comment.

Please rebase and push again to restart CI

Please rebase and push again to restart CI

No need to rebase since there aren't any conflicts. I triggered a rerun of the failed CI Checks manually.

@mcatee-infineon , can you help address the CI failures.

Ping @mcatee-infineon , can you take a look at the CI failures

@mcatee-infineon, @ifyall, @npal-cy, @talih0 as the creator of the PR and maintainer/collaborators for Infineon, can one of you please fix up this PR for the 4.0 final release happening at the end of the week?

let me check this...


Fixes issue #9

This commit adds new function defs to clean_up_files as a stand alone command , delete_files as a option to the stream command enabled by a boolean flag to be able to delete files on exit.
The new auto-delete functionality is set to check if the modified date is more that a week old.
Additions are to set the mtime for each file when it is downloaded, and to check that the mtime of each file stored locally is less than a week old against the current time.

Looks good; I could possibly see wanting the ability to leave all files in place even after a week, which could be another click option. Also might want to make that '604800' user-configurable or else a constant.


Package integration_test tests timing out

Example https://github.com/flutter/packages/pull/7226 (this isn't related to Xcode beta, we're seeing it elsewhere like https://github.com/flutter/packages/pull/7422/checks?check_run_id=28830793413).
Some, but not all, integration_test  tests are timing out.  Example: google_maps_flutter_ios timing out running google_maps_test.dart:
11:58 +0: loading /Volumes/Work/s/w/ir/x/w/packages/packages/google_maps_flutter/google_maps_flutter_ios/example/ios14/integration_test/google_maps_test.dart
11:59 +0: loading /Volumes/Work/s/w/ir/x/w/packages/packages/google_maps_flutter/google_maps_flutter_ios/example/ios14/integration_test/google_maps_test.dart
12:00 +0: loading /Volumes/Work/s/w/ir/x/w/packages/packages/google_maps_flutter/google_maps_flutter_ios/example/ios14/integration_test/google_maps_test.dart
12:00 +0 -1: loading /Volumes/Work/s/w/ir/x/w/packages/packages/google_maps_flutter/google_maps_flutter_ios/example/ios14/integration_test/google_maps_test.dart [E]
   TimeoutException after 0:12:00.000000: Test timed out after 12 minutes.
   package:test_api/src/backend/invoker.dart 338:28  Invoker._handleError.<fn>
   dart:async/zone.dart 1391:47                      _rootRun
   dart:async/zone.dart 1301:19                      _CustomZone.run
   package:test_api/src/backend/invoker.dart 336:10  Invoker._handleError
   package:test_api/src/backend/invoker.dart 291:9   Invoker.heartbeat.<fn>.<fn>
   dart:async/zone.dart 1399:13                      _rootRun
   dart:async/zone.dart 1301:19                      _CustomZone.run
   package:test_api/src/backend/invoker.dart 290:38  Invoker.heartbeat.<fn>
   dart:async-patch/timer_patch.dart 18:15           Timer._createTimer.<fn>
   dart:isolate-patch/timer_impl.dart 398:19         _Timer._runTimers
   dart:isolate-patch/timer_impl.dart 429:5          _Timer._handleMessage
   dart:isolate-patch/isolate_patch.dart 184:12      _RawReceivePort._handleMessage

 To run this test again: /Volumes/Work/s/w/ir/x/w/flutter/bin/cache/dart-sdk/bin/dart test /Volumes/Work/s/w/ir/x/w/packages/packages/google_maps_flutter/google_maps_flutter_ios/example/ios14/integration_test/google_maps_test.dart -p vm --plain-name 'loading /Volumes/Work/s/w/ir/x/w/packages/packages/google_maps_flutter/google_maps_flutter_ios/example/ios14/integration_test/google_maps_test.dart'

12:00 +0 -1: Some tests failed.

https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8739766869905812641/+/u/Run_package_tests/drive_examples/stdout
I wasn't able to reproduce these timeouts locally on the same version of Xcode:
00:04 +0: ... /Users/m/Projects/packages/packages/google_maps_flutter/google_maps_flutter_ios/example/ios14/integration_test/google_maps_test.dart         R00:05 +0: ... /Users/m/Projects/packages/packages/google_maps_flutter/google_maps_flutter_ios/example/ios14/integration_test/google_maps_test.dart  1,076ms
00:22 +0: ... /Users/m/Projects/packages/packages/google_maps_flutter/google_maps_flutter_ios/example/ios14/integration_test/google_maps_test.dart
00:28 +0: ... /Users/m/Projects/packages/packages/google_maps_flutter/google_maps_flutter_ios/example/ios14/integration_test/google_maps_test.dart     6.7s
Xcode build done.                                           23.5s
00:45 +31 ~5: All tests passed!

You can see in the same run there are other drive tests that pass on the same simulator after the failure, like image_picker_ios (so it's not like the simulator isn't ever launching, for example).
Another example:
https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8739766869749591089/+/u/Run_package_tests/drive_examples/stdout
I'm guessing the app isn't launching?  Hard to tell what's going on without screenshots.

I wonder if this is https://github.com/flutter/flutter/issues/153433 ...

After merging https://github.com/flutter/flutter/pull/154970, hopefully these would stop timing out and the test would fail immediately surfacing the actual exception.

This is the roll: https://github.com/flutter/packages/pull/7656

I think this is it:
00:39 +16 ~1 -1: loading /Volumes/Work/s/w/ir/x/w/packages/packages/webview_flutter/webview_flutter_wkwebview/example/integration_test/webview_flutter_test.dart [E]
  ext.flutter.integrationTest: (-32000) Service connection disposed
  package:vm_service/src/vm_service.dart 261:34                      new _OutstandingRequest
  package:vm_service/src/vm_service.dart 1778:25                     VmService._call.<fn>
  package:vm_service/src/vm_service.dart 1790:8                      VmService._call
  package:vm_service/src/vm_service.dart 1729:14                     VmService.callServiceExtension
  package:flutter_tools/src/test/integration_test_device.dart 98:25  IntegrationTestTestDevice.start.<fn>
  dart:async/zone.dart 1422:47                                       _rootRunUnary
  dart:async/zone.dart 1324:19                                       _CustomZone.runUnary
  dart:async/zone.dart 1233:7                                        _CustomZone.runUnaryGuarded
  dart:async/stream_impl.dart 366:11                                 _BufferingStreamSubscription._sendData
  dart:async/stream_impl.dart 297:7                                  _BufferingStreamSubscription._add
  dart:async/stream_controller.dart 777:19                           _SyncStreamControllerDispatch._sendData
  dart:async/stream_controller.dart 651:7                            _StreamController._add
  dart:async/stream_controller.dart 606:5                            _StreamController.add
  dart:async/zone.dart 1422:47                                       _rootRunUnary
  dart:async/zone.dart 1324:19                                       _CustomZone.runUnary
  dart:async/zone.dart 1233:7                                        _CustomZone.runUnaryGuarded
  dart:async/stream_impl.dart 366:11                                 _BufferingStreamSubscription._sendData
  dart:async/stream_impl.dart 542:14                                 _DelayedData.perform
  dart:async/stream_impl.dart 647:11                                 _PendingEvents.handleNext
  dart:async/stream_impl.dart 618:7                                  _PendingEvents.schedule.<fn>
  dart:async/zone.dart 1406:47                                       _rootRun
  dart:async/zone.dart 1317:19                                       _CustomZone.run
  dart:async/zone.dart 1225:7                                        _CustomZone.runGuarded
  dart:async/zone.dart 1265:23                                       _CustomZone.bindCallbackGuarded.<fn>
  dart:async/zone.dart 1414:13                                       _rootRun
  dart:async/zone.dart 1317:19                                       _CustomZone.run
  dart:async/zone.dart 1225:7                                        _CustomZone.runGuarded
  dart:async/zone.dart 1265:23                                       _CustomZone.bindCallbackGuarded.<fn>
  dart:async/schedule_microtask.dart 40:21                           _microtaskLoop
  dart:async/schedule_microtask.dart 49:5                            _startMicrotaskLoop
  dart:isolate-patch/isolate_patch.dart 118:13                       _runPendingImmediateCallback
  dart:isolate-patch/timer_impl.dart 405:11                          _Timer._runTimers
  dart:isolate-patch/timer_impl.dart 429:5                           _Timer._handleMessage
  dart:isolate-patch/isolate_patch.dart 184:12                       _RawReceivePort._handleMessage

https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8736798859005867345/+/u/Run_package_tests/drive_examples/stdout
I will file a new issue

Thanks for finding a way of narrowing down this mystery problem! (Plus, fast failures rather than 12min failures per test will be a huge improvement for flutter/packages quality of life until the root cause can be fixed.)

Thanks for finding a way of narrowing down this mystery problem! (Plus, fast failures rather than 12min failures per test will be a huge improvement for flutter/packages quality of life until the root cause can be fixed.)

No problem! For the record, here is the tracking issue for specifically the service connection disposed RPC error: https://github.com/flutter/flutter/issues/155475

From my initial investigation it's not clear what's going on here exactly.
The similar failure in the task runner (tracked in https://github.com/flutter/flutter/issues/155475) was clearly a case of the target process shutting down after the task completed before the VM service isolate could send the service extension response. However, in this case, invoking the ext.flutter.integrationTest service extension shouldn't result in the isolate shutting down immediately (it seems to echo back the request details on the service's Extension stream).
Obviously the target process is starting if we're able to make a VM service connection in the first place, so that seems to imply that either the process is crashing or it's exiting before the service isolate can send the response. Would we expect there to be any indication in the logs that the target process crashed?

Would we expect there to be any indication in the logs that the target process crashed?

I don't know for certain, but my guess would be no.

I captured a screenshot of the test while it's timing out and it looks like the app is running a white screen.  The simulator isn't on a boot screen or home screen, so I don't think the simulator or app crashed...


FCM.getFCMToken() throws error on FirebaseApp.initializeApp(Context)

Please include following information for better support
What verion of RN and react-native-fcm are you running?
RN 0.33 , fcm 2.3.2
What device are you using? (e.g iOS9 emulator, Android 6 device)?
Android 6 in Genymotion
Is your app running in foreground, background or not running?
Foreground
Advanced:
Have you tried adding break point in native handlers to see if message gets delivered?
can share a project with issue?

Error: Default FirebaseApp is not initialised in this process com.royalmobile. Make sure to call FirebaseApp.initializeApp(Context) first.
If i add in
import firebase_config from '../../config.js';
const firebaseApp = Firebase.initializeApp(firebase_config);
in the file, error becomes
"Firebase App named '[DEFAULT]' already exists.

So I tweaked around the build.gradle file in app folder with the 'com.google.gms.google-services' and it managed to get rid of the failure. However, there is no token returned.

Got it to work after a clean and remake


[api] Rspecs for Tag subcollections for non-Vm collections

[api] Rspecs for Tag subcollections for non-Vm collections
Implementing tags subcollection rspecs for
- /api/providers
- /api/hosts
- /api/data_stores
- /api/resource_pools
- /api/clusters

@Fryguy rebased and repushed, removed wip, also fixed the alignment in categorization as you wanted in #1433.  Thanks.


Can't cache python requirements due to updated last modification time

as the title suggests, python requirements aren't being cached due to updated last modification time.
serverless-python-requirements can cache python requirements with lambda layer.
but in some cases, it can't.
I've investigated this issue for several days:

serverless says the hash of new pythonRequirements.zip is different with the hash of one in S3, so it should be uploaded again
however, when I compare all the files in local pythonRequirements.zip and S3 pythonRequirements.zip, their contents are all the same
finally found that, the last modification time of files is different..

this issue is mentioned before though, the problem is still exist.
In code, I think below snippet should fix this issue, but it couldn't its job.
https://github.com/serverless/serverless-python-requirements/blob/ebd12cb14ea352fb08c0957f213bda7dcce800df/lib/zipTree.js#L74-L75
and I also found that it seems below snippet broke caching
https://github.com/serverless/serverless-python-requirements/blob/ebd12cb14ea352fb08c0957f213bda7dcce800df/lib/pip.js#L664-L665
I don't know what the FIFO cache is in this context... but after I comment these lines out, the cache works like a charm.
please any comment on this. thanks

nevermind. I was using 5.1.1 and I updated to 5.4.0 and the issue is gone. sorry for inappropriate issue.


Nuxt 2.7.0  raises ERROR  The "path" argument must be of type string. Received type undefined

Off-topic
I cannot access https://bug.nuxtjs.org/
It does not work for me at all.

That's why I post it as-is.
On topic
I try to upgrade to the latest Nuxt, I use https://github.com/wemake-services/wemake-vue-template
» npm run dev

><EMAIL_ADDRESS>dev /Users/sobolev/Documents/github/timeless_is
> cross-env NODE_ENV=development nuxt

   ╭─────────────────────────────────────────────╮
   │                                             │
   │   Nuxt.js v2.7.0                            │
   │   Running in development mode (universal)   │
   │   TypeScript support is enabled             │
   │                                             │
   │   Listening on: http://localhost:3000/      │
   │                                             │
   ╰─────────────────────────────────────────────╯

ℹ Preparing project for development                                        17:45:15
ℹ Initial build may take a while                                           17:45:15
✔ Builder initialized                                                      17:45:15

 ERROR  The "path" argument must be of type string. Received type undefined

  at validateString (internal/validators.js:125:11)
  at Object.extname (path.js:1365:5)
  at uniqBy_1.options.plugins.map (node_modules/@nuxt/builder/dist/builder.js:5684:58)
  at Array.map (<anonymous>)
  at Builder.normalizePlugins (node_modules/@nuxt/builder/dist/builder.js:5680:28)
  at Builder.generateRoutesAndFiles (node_modules/@nuxt/builder/dist/builder.js:5653:36)
  at Builder.build (node_modules/@nuxt/builder/dist/builder.js:5574:16)

No ideas where to look.

This question is available on Nuxt community (#c9212)

Resolved! Turned out there was a cache from something. Uninstalled all packages, locks, .nuxt/ folder.
Clean build is working. Thanks for the serverside logs! Awesome feature.

@sobolevn For double investigation can you please share nuxt.config and what files are inside plugins/ dir?

The trace is referring to: https://github.com/nuxt/nuxt.js/blob/v2.7.0/packages/builder/src/builder.js#L265 (/cc @clarkdo)

@pi0 sure!
client/plugins
├── element-ui.ts
├── vee-validate.ts
├── vue-googlemaps.ts
└── vue-observe-visibility.ts

And:
// This is Nuxt configuration file
// See: https://nuxtjs.org/guide/configuration/

// We also use commonjs modules because of vue-styleguidist import system

const path = require('path')
const pkg = require('./package.json')

const envPath = path.resolve(__dirname, 'config', '.env')
require('dotenv').config({ 'path': envPath })

module.exports = {
  /**
   * Headers of the page
   */
  'head': {
    'htmlAttrs': { 'lang': 'en' },
    'title': pkg.name,
    'meta': [
      { 'charset': 'utf-8' },
      { 'name': 'viewport', 'content': 'width=device-width, initial-scale=1' },
      {
        'hid': 'description',
        'name': 'description',
        'content': pkg.description,
      },
    ],
    'link': [
      {
        'rel': 'icon',
        'type': 'image/x-icon',
        'href': '/favicon.ico',
      }, {
        'rel': 'icon',
        'type': 'image/png',
        'sizes': '32x32',
        'href': '/favicon-32x32.png',
      }, {
        'rel': 'icon',
        'type': 'image/png',
        'sizes': '96x96',
        'href': '/favicon-96x96.png',
      },
    ],
  },

  /**
   * Environment variables
   */
  'env': {
    'GOOGLE_MAPS_KEY': process.env.GOOGLE_MAPS_KEY,
  },

  /**
   * Global router settings
   */
  'router': {
    // All pages require auth,
    // see: https://auth.nuxtjs.org/getting-started/middleware
    'middleware': ['auth'],
  },

  /**
   * Specify Nuxt source directory
   */
  'srcDir': 'client',
  'rootDir': path.resolve(__dirname),

  /**
   * Extra Nuxt modules
   */
  'modules': [
    '@nuxtjs/axios',
    '@nuxtjs/proxy',
    '@nuxtjs/auth',
    'nuxt-imagemin',
    'nuxt-svg',
  ],

  /**
   * Axios settings
   */
  'axios': {
    // See https://axios.nuxtjs.org/options.html
    'debug': process.env.NODE_ENV === 'development',
    'https': process.env.NODE_ENV !== 'development',
    'proxyHeadersIgnore': ['accept', 'accept-encoding', 'host'],
    'progress': true,
    'proxy': false,
    'retry': true,
  },

  /**
   * Proxy module for Nuxt and Axios
   */
  'proxy': {},

  /**
   * Auth module for Nuxt,
   * see: https://github.com/nuxt-community/auth-module
   */
  'auth': {
    'redirect': {
      'login': '/',
    },
    'resetOnError': true,
    'strategies': {
      'local': {
        'endpoints': {
          'login': {
            'url': '/api/auth/login',
            'method': 'post',
            'propertyName': 'token',
          },
          // TODO: enable logout
          'logout': false,
          // TODO: enable user profile
          // https://github.com/nuxt-community/auth-module/issues/286
          'user': false,
        },
      },
    },
  },

  /**
   * Custom plugins
   */
  'plugins': [
    '~/plugins/element-ui.ts',
    '~/plugins/vue-googlemaps.ts',
    '~/plugins/vee-validate.ts',

    // Client-side only:
    { 'src': '~/plugins/vue-observe-visibility.ts', 'ssr': false },
  ],

  /**
   * Server side functions that we use for SSR:
   * https://nuxtjs.org/api/configuration-servermiddleware/
   */
  'serverMiddleware': [
    '~/mock-api/server.js',
  ],

  /**
   * Global CSS
   */
  'css': [
    'normalize.css/normalize.css',
    '~/scss/global-styles.scss',
  ],

  /**
   * postcss configuration
   */
  'postcss': {},

  /**
   * Build configuration
   */
  'build': {
    'quiet': false,

    extend (config, { isDev, isClient }): void {
      // This line allows us to use `@import "~/scss/..."` in our app:
      config.resolve.alias['/scss'] = path.resolve(__dirname, 'client', 'scss')

      if (isDev && isClient) {
        // Enabling eslint:
        config.module.rules.push({
          'enforce': 'pre',
          'test': /\.(js|ts|vue)$/u,
          'loader': 'eslint-loader',
          'exclude': /(node_modules)/u,
        })

        // Enabling stylelint:
        config.plugins.push(require('stylelint-webpack-plugin')({
          'files': 'client/**/*.{vue,scss,css}',
        }))
      }
    },
  },
}

@pi0 Do you think if it’s related to ts-node memory  cache?

@clarkdo yes, typescript can be also a reason.

It might be something from my side. I cannot reproduce it anymore.

@sobolevn ok. thanks for the clarification. closing it for now. please tell us it happened again :)
also /cc @kevinmarrec

@pi0 @sobolevn It's internal validators of path and it isn't related to TypeScript :

So it we follow the trace back and look for nuxt builder code
It means p.src is undefined, so it either mean :
plugins: [undefined]

or
plugins: [{ src: undefined }]

Issue is reproducible with such statements in nuxt.config.js.

@sobolevn  Dunno how it became undefined in your case though 😁

The undefined could be like :
User started dev server and removed one plugin, but the plugin item is deleted instead whole plugins array recreated

Also had this issue, but in my case, the problem was that I was missing an environment variable in my .env file that was added during dev process and I just didn't know that it was. So, after I added it everything returned to normal.

Also had the problem but for me it occurred after moving my whole local git repo to another directory.
Deleting .nuxt, node_modules and package-lock.jsonand starting with a clean slate helped.

One way to reproduce this bug is having a typo in plugins section of the nuxt.config.js:
export default {
  plugins: [
    [ 'see/double/brackets/here.js']
  ]
}

This issue happened to me because I imported a plugin as a module.

@pi0  I had the same issue, any updates about this? I upgraded nuxt to the latest version, removing yarn.lock and yarn install again, I checked also my nuxt.config file and nothing works till now, any solution ?


Is this library still maintained?

I love the react hook form and Vue3 , so I find this similar library for Vue3. But it had not been updated since January 2021.If this library will be maintained, I want to contribute my effort. @beizhedenglong

Hi @shlroland, Thanks for your interest. This project will be maintained, but I'm very busy with work recently. If you want to contribute to this project, PR is welcome. I will review your PR in time.

Hi @shlroland, Thanks for your interest. This project will be maintained, but I'm very busy with work recently. If you want to contribute to this project, PR is welcome. I will review your PR in time.

could I get your wechat ID to have a discuss about roadmap?

My WeChat: Problem_of_eval


double free or corruption (!prev) Aborted (core dumped)

Hello,
I would like to say thanks for the library, it is nice seeing someone else with interest in those things.
I am maintaining a repo of logger benchmarks. I have tried to add your logger there.
Your logger is performing very well, if you would like to see how it does compared to other loggers you can have a look at the results here
It runs fine when 1 thread is logging but at the end it terminates and a exception in thrown.
I was unable to benchmark 4  threads pushing logs at the same time as the logger crashed for the same reason
double free or corruption (!prev)

double free or corruption (out)

I am compiling on ubuntu with gcc 11.2 in release mode.
To reproduce you can try to run the benchmark here

sorry please ignore this, I was calling poll on 2 different threads by mistake, forgot to remove my previous line of code starting the pooling thread. Seems to be running okay now!


IE8 coverflow fails

IE8 coverflow fails, s.wrapper.style is undefined so perspectiveOrigin cannot be set.
s.wrapper is an array of elements so s.wrapper[0].style seems to work.
//Set correct perspective for IE10
if (window.navigator.pointerEnabled || window.navigator.msPointerEnabled) {
      var ws = s.wrapper.style;
      ws.perspectiveOrigin = center + 'px 50%';
}

Sorry, IE 8 and IE 9 are not supported


Switched the StreamLayerClientImpl::Flush to use PublishDataTask

Switched the StreamLayerClientImpl::Flush method to use new sync PublishDataTask API.
Also, fixed the integration tests to reorder them according to new sync API design.
Relates-to: OLPEDGE-1184
Signed-off-by: Diachenko Mykahilo<EMAIL_ADDRESS>Signed-off-by: Bohdan Kurylovych<EMAIL_ADDRESS>

The code coverage would be increased after resolving task OLPEDGE-1185


Add Heading and Text components

Summary
Added <Heading/> and <Text/> components
@cerner/terra
Deployment Links
Headings: https://terra-core-deployed-pr-470.herokuapp.com/static/#/site/heading
Text: https://terra-core-deployed-pr-470.herokuapp.com/static/#/site/text

@mmalaker @neilpfeiffer This is now ready for functional / UX review.


Make EventPolicy type in contract more generic

Adding matchers for Exact and Prefix, so we can adjust it later to not only match on the "sub" claim.

/cc @pierDipi

/test channel-reconciler-tests-ssl

/test upgrade-tests

/test upgrade-tests

/test upgrade-tests

Rebased. @pierDipi could you recheck?

/retest

/retest

/test upgrade-tests

/test reconciler-tests-namespaced-broker


fix: Keep unix style line breaks when checking out on windows.

When checking out on windows line breaks can be changed to CRLF, depending on Git configuration, and this leads to this errors, just by running npm run serve:

This git local config ensures that when checking out .css and .js files they keep the LF line breaks, so neither eslint nor minification are triggered by spurious file changes.

There is a linter rule to adjust:
"linebreak-style": 0


dragEnd being fired multiple times.

Hello, thanks for this wonderful product. Saved the day. However, my dragEvent listener is getting fired multiple times.

Thanks for the kind words! Can you provide a reduced test case of your issue? Here is a CodePen you can fork: http://codepen.io/desandro/pen/NPBXgJ


Clash between feature names and fixed keys (e.g. 'order')

In ExtractorResult.to_df(), if feature names are the same as fixed keys (e.g. 'order', 'onset', 'duration'), pandas will raise an error.
In my case, Clarifai returned a key named "order" (which was put into a df by: df = self.extractor._to_df(self, **to_df_kwargs)). Then, when inserting the column "order" into the df, pandas raised an error: https://github.com/tyarkoni/pliers/blob/772d704cb219235f5daea8a1d0a919bee908d0c9/pliers/extractors/base.py#L153

In chat, the suggestion for the fix was to append _ to all protected keys


feature detect checkValiditiy on enclosing form

Feature detect and prefer checkValidity() on the enclosing form. Fallback behavior for IE9.

Looks great, thanks!


Come up with Name

Decided to name it ScreenTracker
Track what's on your screen and organise your watching library - movies and TV shows.

Web App Name: ScreenTracker


Remove pip check

Checklist

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

In https://github.com/conda-forge/bw2calc-feedstock/pull/1 I added pypardiso as a requirement. Downstream mkl does not tell pip its version number so pip check fail there and in any downstream package of bw2calc.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Yes, IMO this is the fastest way forward.

I agree, see also https://github.com/conda-forge/pypardiso-feedstock/commit/a2e8509a09c6ec511e457dd02c84d844509ff1dc


chore(ios): update Google-Mobile-Ads-SDK pod to version 9.14.0

Close https://github.com/capacitor-community/admob/issues/216
Ref: https://github.com/capawesome-team/capacitor-firebase/issues/301#issuecomment-1387145619

tested it in my apps and can verify that it resolves the compatibility issue 👏

Thank you, I will marge. But, the issue is how to manage the Pod version of Google-Mobile-Ads-SDK in the future.


Monocular cues importance

Hi,
Thanks for the great project!    Like https://github.com/baegwangbin/IronDepth ?
Or are the omnidata ones the best?
--
Best, Dmytro

Hi, we have a simple comparison of using different monocular depth and normal methods on one of the Replica scene. Please check table 9 in the paper https://arxiv.org/pdf/2206.00665.pdf. Omnidata model is the best among the models we tested but of course other models might performs better on different scenes.

Thank you!
For those who stumble upon the issue, here is that Table:

Hi @niujinshuchong, I see that depth from monocular_cues.py follows some scale, what is that scale?
Moroever, if I want to try let's say Depth-anything method for monocular cues, what scale should I apply for a custom scene?  how should "--pipeline.model.mono-depth-loss-mult" change for different scales?