Nested Composition

Hybrid visualizations in which visual components are nested within other visual components should be trivial to assemble. This requirement may fundamentally shape how the library is constructed.
Examples:

Facet (small multiples in X and/or Y)
Scatter plot of pie charts
Treemap of line charts

Nested composition is a fundamental aspect. This could be done by adopting the "rubber stamp" idea from Towards Reusable Charts. Each visual component could be constructed in such a way that it exposes a "nesting" point, where another visual component could pick up the task of rendering marks. Maybe this could be called mark
As an example, a "Scatter Plot" could be a nested composition of two visual components "Scatter" and "Circle". The code that uses the components might look something like this:
var RV = require("reactive-vis");
var circle = RV.Circle();
var scatter = RV.Scatter()
  .mark(circle);

A prototype of this is implemented in Composable Visualization Test.
One implication of this is that the parameters for the "Scatter Plot" would actually be split up between the "Scatter" and "Circle" instances, like this:
var circle = RV.Circle();
var scatter = RV.Scatter()
  .mark(circle);

circle
  .colorScale(...)
  .colorColumn(...);

scatter
  .xColumn(...)
  .yColumn(...)
  .sizeColumn(...);

Part of https://github.com/datavis-tech/reactive-vis/issues/12

Tried an implementation that looks like this:
var circle = ReactiveVis.Circle();

var scatter = ReactiveVis.Scatter()
  .svg(createSVG())
  .data(exampleData);

scatter
  .xColumn("sepal_length")
  .yColumn("sepal_width")
  .sizeColumn("petal_length");

//ReactiveVis.nest(scatter, circle);
scatter(function (marks){
  marks.each(function (d){
    circle
      .data([d])
      .svg(this);

    // Depends on nested digests
    //
    //ReactiveVis.digest();
  });
  console.log(marks);
}, "marks");

ReactiveVis.digest();

It doesn't work because a digest within a digest is not currently supported.
Depends on https://github.com/datavis-tech/reactive-function/issues/18

This is what nested composition setup looks like now:
var svg = createSVG();
var circle = ReactiveVis.Circle();

var scatter = ReactiveVis.Scatter()
  .svg(svg)
  .data(exampleData);

scatter
  .xColumn("sepal_length")
  .yColumn("sepal_width")
  .sizeColumn("petal_length");

scatter(function (marks){
  marks.each(function (d){
    var size = marks.sizeLocal.get(this);
    circle
      .data([d])
      .width(size)
      .height(size)
      .svg(this)
      .digest();
  });
}, "marks");

ReactiveVis.digest();


Release 2 on Alfajores

Expected Behavior
Please describe the behavior you are expecting
Current Behavior
What is the current behavior?

Core Contracts Release 2 has been deployed on the Alfajores network.
ProposalID: 8
Txhash of proposal execution:  0x81aba7dba0424a2fdc182ebb1de2285ff631407648ce623f9830906366affe26

Core Contracts Release 2 has been deployed on the Alfajores network.
ProposalID: 8
Txhash of proposal execution:  0x81aba7dba0424a2fdc182ebb1de2285ff631407648ce623f9830906366affe26

env-tests is still in progress.

env-tests is still in progress.

Tests and their status run on alfajores for release 2:
env-tests:

Transfer cUSD: PASS
Exchange cUSD for CELO: PASS
Oracle: Report a Rate: FAIL
Reserve: move funds from the Reserve to a custodian and back: FAIL
Attestation: receive attestations for a phone number: PASS
Attestation: receives attestations using security code flow: PASS

Manual Tests:

Transfer CELO: PASS
Register Account: PASS
Show Account: PASS
Lock gold: PASS
Show locked gold: PASS
List election validator groups: PASS
Vote for a validator group: PASS
Run mock election: PASS

Per @nambrot - The failed tests from the env-tests is expected, since the oracle/reserve spenders are not added.


no-shadow: Already declared in the upper scope

$ cat foo.js
export default class Foo{}

$ eslint foo.js
foo.js
  1:21  error  Foo is already declared in the upper scope  no-shadow

✖ 1 problem (1 error, 0 warnings)

This seems to be the same as #99? Are you using the latest eslint - 0.21.2?
ESLint issue: https://github.com/eslint/eslint/issues/2545
https://github.com/eslint/eslint/releases/tag/v0.21.2

Yes, it seems to be the same issue. I'm closing it now as it seems to be an issue with eslint.

Does the latest eslint fix it for you? Maybe try the master branch.

Yes. The latest eslint (0.21.2) fixed the issue for me. Thanks!


Removed duplicate variable declaration

The "path" variable declaration and value assignment in serve.js was done twice.

Thanks!


[FIXED JENKINS-27627] Color fixes

When running a job it adds a HTML class without removing the previous ones. This causes glitches where succesful build are still marked as failed on the EzWall screen

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests


Create a quick list with everyone's GitHub usernames

Using the "introduce yourself" discussion create a quick team.md file listing all the contributor's GitHub usernames, each one starting with a @ and separated by a comma and a space. eg:
@madhatter, @whiterabbit, @redqueen,
This will help us maintainers efficiently message everyone.
Thanks,
Liz

Hey @GingerKiwi
Shall I take this up ?

@GingerKiwi
Would you mind if I take this issue up ?

Here's the file that you requested, I hope it meets your requirements:
team.md


Adding BiCGSTAB solver for sparse systems

Signed-off-by: Gil Tabak<EMAIL_ADDRESS>Description
This pull request will add a custom CPU op for solving sparse linear systems using the BiCGSTAB method. This is based on using the Eigen library.
Fixes #2387
Type of change
I'd like to start a new subpackage specifically for sparse linear algebra operations not supported (and without plans of being supported) by core Tensorflow. This includes implementation of sparse solvers using iterative methods as custom ops.

[ ] Bug fix
[ ] New Tutorial
[ ] Updated or additional documentation
[ ] Additional Testing
[ ] New Activation and the changes conform to the activation contribution guidelines
[ ] New Callback and the changes conform to the callback contribution guidelines
[ ] New Image addition and the changes conform to the image op contribution guidelines
[ ] New Layer and the changes conform to the layer contribution guidelines
[ ] New Loss and the changes conform to the loss contribution guidelines
[ ] New Metric and the changes conform to the metric contribution guidelines
[ ] New Optimizer and the changes conform to the optimizer contribution guidelines
[ ] New RNN Cell and the changes conform to the rnn contribution guidelines
[ ] New Seq2seq addition and the changes conform to the seq2seq contribution guidelines
[ ] New Text addition and the changes conform to the text op contribution guidelines

Checklist:

[x] I've properly formatted my code according to the guidelines

[ ] By running Black + Flake8
[x] By running pre-commit hooks

[x] This PR addresses an already submitted issue for TensorFlow Addons
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] This PR contains modifications to C++ custom-ops

How Has This Been Tested?
I used pytest locally, and I'm not familiar enough with the CI workflows here yet to make sure my tests are run - I'm going to open the pull request and integrate tests after I can see how it works.

Hi maintainers! I was able to finally get the _sparse_ops.so library to build locally, and it came down to carefully going through the steps to install and build (and honestly I'm not entirely sure what the correct sequence of steps is, but it magically appeared at one point). Now I'm having the same issue with the file missing here across the tests. Could you point me in the right direction to what might be missing or needs to be tweaked to get this working? Thank you!

Looks like only windows and macos fail. @bhack Can you help this one? I'm busy with other things 😭

Looks like only windows and macos fail. @bhack Can you help this one? I'm busy with other things sob

I suppose it was still on ecosystem review and not feature-approved-for-pr. There was a similar issue in TF core.

Looks like only windows and macos fail. @bhack Can you help this one? I'm busy with other things sob

I suppose it was still on ecosystem review and not feature-approved-for-pr. There was a similar issue in TF core.

What is the process for ecosystem review? In an internal discussion it was suggested to me to proceed adding this to TF Addons. Sorry, I'm not familiar with this process.

Ecosystem review label it is mainly this https://github.com/tensorflow/addons/blob/master/CONTRIBUTING.md#requirements-for-new-contributions-to-the-repository
@tomerk Will periodically review ecosystem reviews labels with TF core, TF models, Keras and the other ecosystem projects/SIGs about potential conflicts/duplicates with the internals, often "not-public", roadmaps.
We also asked to @tomerk to evaluate if an ecosystem resource could review the PR in another TF ecosystem repository or if there is a resource that could help to review the feature here  e.g. like https://github.com/tensorflow/addons/issues/2388#issuecomment-778422760.

Thanks! I'll reach out to @tomerk with the internal thread I mentioned.

Ecosystem review notes: Our internal discussion concluded addons would be a better home for this than tf core. It also does not conflict with plans for core tf.

@tomerk Do you have also an interested reviewer for this?

Not off the top of my head but I imagine @tabakg can find one of the internal reviewers to do so.

Thank you! My suggestion would be @penpornk (Penporn Koanantakool) and @bloops (Anudhyan Boral). I'll email them to make sure I found their correct Github accounts, and to let them know.

@tabakg Let us know if you have any feedback.

Thank you for your contribution. We sincerely apologize for any delay in reviewing, but TensorFlow Addons is transitioning to a minimal maintenance and release mode. New features will not be added to this repository. For more information, please see our public messaging on this decision:
TensorFlow Addons Wind Down
Please consider sending feature requests / contributions to other repositories in the TF community with a similar charters to TFA:
Keras
Keras-CV
Keras-NLP


Running "ionic serve" succeeded, but the browser could not browse

Ionic version:  (check one with "x")
(For Ionic 3.x issues, please use https://github.com/ionic-team/ionic-v3)
[ ] 2.x
[ ] 3.x
[ ] 4.x
I'm submitting a ...  (check one with "x")
[ ] bug report
[ ] feature request
Please do not submit support requests or "How to" questions here. Instead, please use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

Expected behavior:

Steps to reproduce:

Related code:

insert short code snippets here

Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
insert the output from ionic info here

`
C:\Users\Bill\cutePuppyPics>ionic serve
Starting app-scripts server: --address <IP_ADDRESS> --port 8100 --livereload-port 35729 --dev-logger-port 53703 --nobrowser -
Ctrl+C to cancel
[14:13:17]  watch started ...
[14:13:17]  build dev started ...
[14:13:17]  clean started ...
[14:13:17]  clean finished in 5 ms
[14:13:17]  copy started ...
[14:13:17]  deeplinks started ...
[14:13:17]  deeplinks finished in 28 ms
[14:13:17]  transpile started ...
[14:13:21]  transpile finished in 3.53 s
[14:13:21]  preprocess started ...
[14:13:21]  preprocess finished in 1 ms
[14:13:21]  webpack started ...
[14:13:21]  copy finished in 3.76 s
[14:13:28]  webpack finished in 7.91 s
[14:13:28]  sass started ...
[14:13:30]  sass finished in 1.11 s
[14:13:30]  postprocess started ...
[14:13:30]  postprocess finished in 8 ms
[14:13:30]  lint started ...
[14:13:30]  build dev finished in 12.69 s
[14:13:30]  watch ready in 12.80 s
[14:13:30]  dev server running: http://localhost:8100/
[OK] Development server running!
Local: http://localhost:8100
External: http://<IP_ADDRESS>:8100
DevApp: cutePuppyPics@8100 on Bill-PC
C:\Users\Bill\cutePuppyPics>
`
Then I Google visit "http //: localhost: 8100", the browser can not be accessed

Hello! Thank you for opening an issue with us!
The Ionic Framework issues list is designated for the reporting of bugs with the Ionic Framework or for requesting new features. As this seems like more of a support question I am going to close this issue.
I urge that you ask this question on our forum or on our slack channel.
I did verify that I am able to generate and serve an Ionic application on my Windows machine and it worked. What browser are you using to load this? It should automatically start your default browser.
Thanks for using Ionic!


Main package should be compiled for node env

I had an issue with the axios library when I was using axios in the main package, the code was resolving to the XHR adapter. It was happening due to this field in package.json of axios:
  "browser": {
    "./lib/adapters/http.js": "./lib/adapters/xhr.js"
  },

It means that for browser env ./lib/adapters/http.js file should be replaced with ./lib/adapters/xhr.js, but main package is running under node environment, and it was causing the error. Here is the more detailed issue https://github.com/vitejs/vite/issues/6382 .
So, I fixed it with ssr: true in the build section of config.vite.js. What do you think about this? I can provide PR if you want 🙂

Oops, the electron app will not work after compilation when ssr: true is provided; it fails to resolve dependencies(cause ssr: true doesn't bundle all deps to one file). So I am not sure how we could fix it 🤔 But we need to let vite know that it is running in node env for the main package

Also related issue https://github.com/vitejs/vite/issues/4405

Hmmm...
Try add axios to external packages. That axios should resolve in runtime in electron environment

Wow, thanks for the fast response, I will check 🙂 Also related https://github.com/axios/axios/issues/456

@cawa-93 this error happens when I running the compiled version

The yarn watch works like a charm

Btw I am getting the same error for ssr: true for compiled version, while yarn watch works correctly

🤔 I just tried run compilation with ssr: true and doesn't saw problems

It seems you can import axios from relative path:
import axios from '../../../node_modules/axios';

That force Vite does't resolve browser mapping and resolve only as node. I'm not sure how safe it is 🤷♂

Oops, I used yarn workspaces, and that what the reason why some node packages were not loaded. Btw, I still think that the main package should use ssr: true, cause I suppose you will fail to use any native package(that uses gyp). As I understand later Vite will introduce this flag:
{
  resolve : {
    environment: 'node'
  }
}

Btw, I still think that the main package should use ssr: true,

You may be right. But I need more tests and research for that change

It seems build in ssr: true works fine for main and preload. But I can't dare use it because SSR support is still experimental.

Should be resolve in https://github.com/cawa-93/vite-electron-builder/commit/d002d6f2267418407d173e538e3792e012a67415


Add alias to .bashrc for more succinct command

We should be able to do this through a predefined bash command. We should also come up with the best possible name. I have been just using ui as an alias for nyg nyg-ui-generator.

It would be possible to set some alias as part of a postinstall script, including maybe some for quickly creating modules of a certain type.

Yes! That is exactly what I was getting at. Definitely an alias for ui-generator and module-generator. Let me know what else you have in mind.

those two sound good. i'll try and think of some more aliases.


DPO: RuntimeError: element 0 of tensors does not require grad and does not have a grad_fn

Hi,
When running script dpo_llama2.py from the TRL examples I get this error. Issue #801 describes the same problem and recommends a workaround  but it might be more appropriate to verify it that is an appropriate fix and if so, make it part of a new release. Thanks.

Thanks for reporting the issue, I reopened the original one and am closing this one so we only track it once.

Thank you @lvwerra


When pulling up data from a varchar column it appears to clear certain types of whitespace chars.

Issue Type: Bug
I'm not sure how to replicate, but I have a varchar (max) cell with "d  WHERE" contained in the value, but SQL Operations Studio is pulling it up as "dWHERE".
I've tested SSMS and it's working fine.
SQL Operations Studio version: sqlops 0.30.6 (df7e3ec7d52a82febee5b597a1019cabf021b883, 2018-06-19T21:45:43.589Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6600U CPU @ 2.60GHz (4 x 2808)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwarevideo_decode: enabledvideo_encode: enabledvpx_decode: unavailable_softwarewebgl: enabledwebgl2: enabled

Memory (System)
15.93GB (4.07GB free)

Process Argv
C:\Program Files\SQL Operations Studio\sqlops.exe

Screen Reader
no

VM
0%

Extensions: none

@efraimbart please let us know if you're able to provide more repro details.  Returning incorrect results is a serious bug that we'd like to diagnosis and fix.

@kburtram I've been trying to reproduce but have only been successful in the one preexisting cell/row where I bumped into this issue originally, when I create a new row in my local DB with the same data everything works fine.
Maybe it has to do with the fact that I'm attempting to replicate on a different instance of SQL Server? Local vs remote? SQL Server 2016 vs Azure? Regular DB vs replication/slave DB? Or maybe it has to do with the actual characters, maybe they're not getting copied properly and therefore not replicating the issue?

It originally appeared to be two whitespace characters, I just noticed that it got stripped in the submission as well, I'll try to paste here:
d  WHERE

About the theory of being unable to replicate due to a difference in the SQL Server instance, I just found a similar row of data in my local instance and was able to replicate.

@efraimbart is there any chance the whitespace character is actually a newline character? This would match the behavior you're describing, since SSMS and SQL Ops Studio both strip newlines by default when copying.
To verify, you could open the user settings (right click on the gear icon in the bottom left of SQL Ops Studio and click Settings), search for "sql.copyRemoveNewLine" and set it to true, save your settings, and then try copying the result again

@MattIrv I will try that when I get a minute, but I should point out that it is not being stripped in SSMS.

@MattIrv I tried with "sql.copyRemoveNewLine" both set to true and false, I even restarted the application in between; no change.

I just spent some time debugging an issue with disappearing whitespace that turned out to be ADS collapsing several space characters into one in the view. I assume this is related to its HTML internals. It's not completely the same issue as this one, but I wouldn't be surprised if it's related and certain whitespace characters are ignored in HTML.
Fun fact: The VS Code variable inspector also collapsed the same whitespace characters. Nearly drove me to insanity.

in edit data, the editor cells collapse all whitespace into single space, this causes difficulty when pasting multiline text into nvarchar columns, it will be good if we can have a pop up editor for text cells so that there is better visibility as well as retention of whitespace.
for eg:  if I paste text
A
B
C
D
into an editor cell, it becomes

The new lines have been collapsed into space


Move TODO aliases to aliases.nix

Motivation for this change
Things done

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

[ ] 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/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

In principle I +1 the removal of aliases in all-packages.nix, so looks good to me from that point. We should probably CC the maintainers of the ~9 packages which had aliases moved.

pining the maintainers of the packages which aliases where moved.
gtest:
@Zoomulator @ivan-tkatchev (no github name)
iproute:
@primeos @eelco @fpletz @globin
jack2:
@goibhniu
libkrb5:
has no maintainers
xz:
@svanderburg
mariadb:
@thoughtpolice
bison:
has no maintainers

Since iproute wasn't replaced in all modules, nixpkgs cannot be evaluated without aliases right now.

#118805


minor optimizations

These are all on the order of a percent or two on darwin/arm64,
so leaving it to you to decide whether they're worth it
(including checking impact on amd64). My 2c:

avoid extra data copies

Clearly generates better code. Almost no readability impact.

avoid extra zeroing

Probably a good idea, despite being weird.

reduce nested loops

Probably not worth it? But maybe interesting?

unroll ntt inner loop

Possibly the highest impact of the optimizations. Won't cherry-pick cleanly without "reduce nested loops", but the idea is simple and it is very easy to recreate.

How can head = append(nil, make([]byte, n)...) be significantly faster than head = make([]byte, n); copy(head, nil)

Speaking in generalities, make+copy does:

alloc array
zero array
copy data to array

whereas the append form skips the middle zeroing step.
It really shouldn't be slower; dunno what's happening there. I totally believe that in this case it doesn't move the needle perceptibly, though, in which case the clearer/simpler code is definitely better.


[knowrob_map_tools] add conversion script from URDF to knowrob:SemanticEnvironmentMap

I'm not sure it's a right place to put these codes.
This pull request includes script to convert URDF model file into knowrob:SemanticEnvironmentMap.

@asilx Thanks for merging!


[Map] DM-Anihilation

Hash: 35d4e43e30a43bafbaa3542d22ea9355a96a3e10
Current name: DM-Anihilation
The author of this map is "Dangerus" Dave O.
[from the readme - not sure if the . after O is part of the name or not]

Updated


Should we officially retire this tool now that Microsoft have finally delivered their own tool

See: https://devblogs.microsoft.com/dotnet/introducing-the-net-upgrade-assistant-preview/
It looks like it covers everything and perhaps even takes inspiration from some features like the great walkthrough mode that @andrew-boyarshin implemented. It also tries to remove transitive dependencies which is something we struggled to do.
We could point people towards it in the Readme and in any open issues
@hvanbakel @andrew-boyarshin

I think that's fair. Have you actually tried the tool?
I would want to give it a try first, but official tooling is always better.
On Thu, Mar 4, 2021, 11:57 Mark Adamson<EMAIL_ADDRESS>wrote:

See:
https://devblogs.microsoft.com/dotnet/introducing-the-net-upgrade-assistant-preview/
It looks like it covers everything and perhaps even takes inspiration from
some features like the great walkthrough mode that @andrew-boyarshin
https://github.com/andrew-boyarshin implemented. It also tries to
remove transitive dependencies which is something we struggled to do.
We could point people towards it in the Readme and in any open issues
@hvanbakel https://github.com/hvanbakel @andrew-boyarshin
https://github.com/andrew-boyarshin
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hvanbakel/CsprojToVs2017/issues/286, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7QKPDWN4DERMDSGLXNOLLTB7Q4FANCNFSM4YT4PBPA
.

Does this tool work with old asp.mvc and ef projects?

I haven't tried it actually and I probably won't get around to with so many
things going on at the moment. It would be nice if they recognised projects
like yours that have given people a decent option all these years.
It would be cool if anyone watching can give it a go and see how it compares
On Thu, 4 Mar 2021, 21:42 Hans van Bakel<EMAIL_ADDRESS>wrote:

I think that's fair. Have you actually tried the tool?
I would want to give it a try first, but official tooling is always better.
On Thu, Mar 4, 2021, 11:57 Mark Adamson<EMAIL_ADDRESS>wrote:

See:

https://devblogs.microsoft.com/dotnet/introducing-the-net-upgrade-assistant-preview/

It looks like it covers everything and perhaps even takes inspiration
from
some features like the great walkthrough mode that @andrew-boyarshin
https://github.com/andrew-boyarshin implemented. It also tries to
remove transitive dependencies which is something we struggled to do.
We could point people towards it in the Readme and in any open issues
@hvanbakel https://github.com/hvanbakel @andrew-boyarshin
https://github.com/andrew-boyarshin
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hvanbakel/CsprojToVs2017/issues/286, or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AA7QKPDWN4DERMDSGLXNOLLTB7Q4FANCNFSM4YT4PBPA
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hvanbakel/CsprojToVs2017/issues/286#issuecomment-790965351,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYCFSYSPHJKOERW4RLVNPDTB75CPANCNFSM4YT4PBPA
.

I used try-convert on a couple projects compared to dotnet-migrate-2019, and try-convert does a better job on detecting and adding UseWpf and UseWindowsForms.
I prefer the way that migrate-2019 does the AssemblyInfo transform, as try-convert uses GenerateAssemblyInfo=false.
migrate-2019 also removes a couple VS2013-isms like Reference/RequiredTargetFramework and BootstrapperPackage that try-convert keeps.
In my opinion, migrate-2019 still has value, and would hate to see it retired.

I started this literally because I thought it was a problem that Microsoft
didn't have a tool for such a repetitive and, in large solutions, time
consuming task.
I suggest we just mention the other tool. If you have one project to
convert then tooling wasn't that useful anyway. If you have a lot of them
then just see which one you prefer.
I'll see if I can put something in the readme this weekend, just mentioning
it.
On Fri, Mar 5, 2021, 16:27 superstrom<EMAIL_ADDRESS>wrote:

I used try-convert on a couple projects compared to dotnet-migrate-2019,
and try-convert does a better job on detecting and adding UseWpf and
UseWindowsForms.
I prefer the way that migrate-2019 does the AssemblyInfo transform, as
try-convert uses GenerateAssemblyInfo=false.
migrate-2019 also removes a couple VS2013-isms like
Reference/RequiredTargetFramework and BootstrapperPackage that try-convert
keeps.
In my opinion, migrate-2019 still has value, and would hate to see it
retired.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hvanbakel/CsprojToVs2017/issues/286#issuecomment-791810717,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7QKPG7PR7M53YMK3L55ETTCFZH5ANCNFSM4YT4PBPA
.

The blog post seems to suggest that try-convert will re-target the projects to .NET 5, which is not what I want. At least until there's support for viewing and exporting SSRS reports in .NET 5, I have to stay on .NET 4.8.
I'll be sticking with migrate-2019 for now.


Change 134. Single Number II to 137 Single Number II

Change 134. Single Number II to 137 Single Number II -
What will change - Rename the code 134. Single Number II to 137 Single Number II in python folder
Type of Issue - Edit a file
Please add/delete options that are not relevant.

[x] Improving Code
[x] Improving Documentation

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

Please assign this issue to me,
Thanks :)


Opening file fails when icon is inside a table

Describe the bug
Obsidian fails to open the file with the error: Cannot read properties of null: (reading 'tagName').
This happens only when the file contains an icon inside a table cell.
To Reproduce
Obsidian will be able to open a file containing this snippet:
:LiCheck:

| A | B |
| --- | --- |
| 1 | Yes |

However, add a new row with an icon:
:LiCheck:

| A | B |
| ---- | ---- |
| 1 | Yes |
| 2 | :LiCheck: |

Obsidian fails to open this file with the above error.
Additional context
On debugging, it seems to crash on this line:
const tagName = element.firstElementChild.tagName.toLowerCase();
element.firstElementChild is null since element is
:LiCheck:
. Not entirely sure how to fix this.
Community Plugins installed: Only obsidian-iconize:v2.8.1
Obsidian version: 1.5.3
Settings enabled:

'Icons Background Check'
'Toggle Icons while editing notes'

Thank you for reporting!
Unfortunately, I was not able to reproduce this issue. I am not sure where the class table-cell-wrapper comes from? Maybe a theme or something?
For me, the table nodes do not have any classes at all.

The default theme has the same issue. I have not enabled any custom CSS rules or any other community plugin.
Fwiw, it broke when I updated to Obsidian 1.5.3 (released 4 days ago).
If we still can't reproduce it, we can archive this issue. I don't mind changing my :LiCheck: to "Yes". We can wait to see if more folks face the same problem before trying to solve this 'bug'.

The default theme has the same issue. I have not enabled any custom CSS rules or any other community plugin. Fwiw, it broke when I updated to Obsidian 1.5.3 (released 4 days ago).
If we still can't reproduce it, we can archive this issue. I don't mind changing my :LiCheck: to "Yes". We can wait to see if more folks face the same problem before trying to solve this 'bug'.

Oh yep nvm. I can reproduce it now. I was still on version 1.4 :D

Pretty sure this is related. If I have an icon within a table, it then freezes the ability to edit within the table, and only displays a "loading bar" as if trying to load the table and everything else below the table. Took a while to figure out which plugin was causing the issue. Iconize is 100% the culprit. I have a video recording of it but github only supports 10MB. I can link it if needed.


Challenge 2 wagmi migration

Challenge 2 migration
Please check if vercel works for you. I have error on deployment but it seems something wrong on their side

Challenge 2 migration
Please check if vercel works for you. I have error on deployment but it seems something wrong on their side

Great job @rin-st !!
It's working great, just added a couple of minor issues to review, one was yarn vercel. Will check tomorrow a bit more comparing old deploy vs new deploy, but I don't think I'll find any other issue 🙌

Tysm all, merging this 🙌 Just updated useScaffoldEventHistory hook hint to use n in README

Tysm all, merging this 🙌 Just updated useScaffoldEventHistory hook hint to use n in README

Thanks!


Currency with conjunctions in PT

"15 dollars and 15 cents" is a single currency entity in EN, but the PT and ES counterparts are captured as a pair of currency entities.
EN: 15 dollars and 15 cents  Returns 15.15 Dollar
PT: 15 dólares e 15 centavos  Returns 15 Dólar, 15 Centavo
ES: 15 dólares y 15 centavos  Returns 15 Dólar, 15 Centavo

Merged with #2479.


Debian Bullseye 8u322

cf. https://hub.docker.com/layers/openjdk/library/openjdk/8u322-jdk-bullseye/images/sha256-1c1628356d0a13aae0a347f0781736de70eb0b3b53433b0d07421458db8d13df?context=explore

Thanks
We currently have a problem with publishing our images so this might take some time, see #194


Trying to update listen_addresses triggers ISE: address already in use

EdgeDB Version: 1.0-beta.2+ga7130d5c7.cv202104290000
OS Version: Ubuntu 20.04

Steps to Reproduce:

create server instance
edgedb -I SERVER_NAME configure set listen_addresses <IP_ADDRESS> ::1 <IP_ADDRESS>

edgedb error: ERROR: InternalServerError: [Errno 98] error while attempting to bind on address ('<IP_ADDRESS>', 10700): address already in use
  Hint: This is most likely a bug in EdgeDB. Please consider opening an issue ticket at https://github.com/edgedb/edgedb/issues/new?template=bug_report.md
  Server traceback:
      Traceback (most recent call last):
        File "edb/server/protocol/binary.pyx", line 1694, in edb.server.protocol.binary.EdgeConnection.main
        File "edb/server/protocol/binary.pyx", line 950, in simple_query
        File "edb/server/protocol/binary.pyx", line 1045, in _simple_query
        File "edb/server/protocol/binary.pyx", line 1002, in edb.server.protocol.binary.EdgeConnection._simple_query
        File "edb/server/protocol/binary.pyx", line 1405, in _execute_system_config
        File "edb/server/dbview/dbview.pyx", line 515, in apply_config_ops
        File "edb/server/dbview/dbview.pyx", line 649, in apply_system_config_op
        File "/usr/lib/x86_64-linux-gnu/edgedb-server-1-beta2/lib/python3.9/site-packages/edb/server/server.py", line 629, in _on_system_config_set
          await self._restart_servers_new_addr(value, self._listen_port)
        File "/usr/lib/x86_64-linux-gnu/edgedb-server-1-beta2/lib/python3.9/site-packages/edb/server/server.py", line 573, in _restart_servers_new_addr
          self._servers, _ = await self._start_servers(nethost, netport)
        File "/usr/lib/x86_64-linux-gnu/edgedb-server-1-beta2/lib/python3.9/site-packages/edb/server/server.py", line 786, in _start_servers
          tcp_srv = await self._loop.create_server(
        File "uvloop/loop.pyx", line 1768, in create_server
      OSError: [Errno 98] error while attempting to bind on address ('<IP_ADDRESS>', 10700): address already in use

Despite the error, value is changed in config:
edgedb> SELECT cfg::DatabaseConfig.listen_addresses;
{'<IP_ADDRESS>', '<IP_ADDRESS>', '::1'}

It seems to me like we should explicitly check for <IP_ADDRESS> and either reject or ignore the other entries in the listen_addresses set.

It's a bit over-complex to reject the other entries if <IP_ADDRESS> is present before actually updating the system config into the database, and rejecting when starting the server is just as bad as the error reported in this issue. So I think I'll create a PR to simply ignore the other entries and print a warning log entry about the misconfiguration.

So I think I'll create a PR to simply ignore the other entries and print a warning log entry about the misconfiguration.

That works

Oh seems like this is not only an issue of <IP_ADDRESS> - the current _restart_servers_new_addr() method is simply binding to each new hosts without checking if we're already listening on that host. I'll also compute a diff, bind to newly-added hosts/ports and unbind the old ones that is no longer in the listen_addresses.

Ahhh asyncio Server doesn't support binding to new hosts or unbinding existing hosts on the fly ... so we'll likely have to customize asyncio server (maybe in uvloop) if we want to smoothly support "add a new host to the current listen_addresses list".

Found another bug: when using --port auto with multiple hosts, the bound port is not consistent on different hosts. I'll fix this in the same PR as the approach is the same.

I'm not sure this is fixable though. port=auto essentially means binding to port 0, and it's expected that the port would be different for different hosts, or am I missing something?

You're very much on the same page. I'm thinking about binding to those hosts sequentially, or at least bind to the first host first, get the port, and use that port for the remaining hosts. But the problem would then be that, binding to the same port on other hosts may fail due to conflict.
On the other hand, the issue for not fixing the --port=auto issue on multiple hosts is that, it's likely that only the first host works with the logged port - other hosts are having different ports (+1 incremental on at least macOS). Neither does --emit-server-status report all the ports.

But the problem would then be that, binding to the same port on other hosts may fail due to conflict.

Exactly.

Neither does --emit-server-status report all the ports.

I think we should fix this -- it should emit pairs of host/port. When port=auto, ports can be different for every host and that's fine. @elprans does this sound OK to you?

Also the admin UNIX socket file is named after a port - we should pick one port, or use PID instead 🤔

I'm not sure I like the idea of picking one port for multiple hosts. We've been there before, see the git log. I really like the comfort of the OS selecting an available port for us in a non-racing fashion.

Perhaps we shouldn't create admin sockets for --port=auto since that's used only in CI/testing.

ports can be different for every host and that's fine.

I feel like this is too complicated for no good reason.  Is it possible to use the result of binding on the first address for the remaining sockets?  This will be fallible, but I think this failure mode is quite marginal, as, like is mentioned elsewhere, --port=auto is mostly used for testing and there multiple listen interfaces are unlikely.

Is it possible to use the result of binding on the first address for the remaining sockets?

It is possible, but it is racy. I also don't view it as super complicated; the recommendation will be "don't configure the server to use multiple hosts with port=auto". On the server side of things, outputting pairs of host/port in emit-server-status is more future-proof anyways.


docs: Update kustomize examples to use non-deprecated resources key (English)

Kustomize has deprecated the use of bases in kustomization.yaml files. This PR updates all references to bases in the docs to use the newer resources, which is the suggested alternative.

/lgtm
/remove-language zh
/remove-language id
/remove-language ko

Thanks @highb
/approve


Add environment variable flag to disable the context GC for tests

Also re-enable both contexts in the drone config files, and disable the GC in florence's indexer pipeline.

@bkontur @sebastiencs ok, new version reads environment variable directly from the InMem storage constructor, which is hacky, but the patch is much cleaner and will be easier to revert once history modes have been implemented. It doesn't require an updated libtezos-ffi either now.


Add documentation page title to header data

Adds documentation page title as a prefix to <title> tag in site header as proposed in https://github.com/facebook/draft-js/issues/965. For example, "Custom Block Components | Draft.js | Rich Text Editor Framework for React" would be the title for the "Custom Block Components" page.
Implements pageTitle prop on the Site component to accomplish this. If titlePrefix is not available, it will fall back to "Draft.js | Rich Text Editor Framework for React".

So awesome! ^_^ Thanks for doing this.


set transfer keeper for pfm

Description and Motivation
add PFM SetTransferKeeper
Related Issues

Checklist:

[ ] I have read Migaloo's contribution guidelines.
[ ] My pull request has a sound title and description (not something vague like Update index.md)
[ ] All existing and new tests are passing.
[ ] I updated/added relevant documentation.
[ ] The code is formatted properly golangci-lint run ./... --fix.

Thank you!


[Feature]: Added 5-day forecasting and animated svg

The changes will add a 5-day forecast to your weather app. The forecast will be displayed below the current weather information and will update whenever a new location is searched or when the geolocation is used.

Assigning this issue to @yazdanhaider


UofT-DSI | Python - Assignment 1

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Adding code according to the assignment.
What did you learn from the changes you have made?
I learned how to create a function and use conditional statements.
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
The changes were tested by running the code in the VS Code on my computer.
A reference to a related issue in your repository (if applicable)
Checklist

[x] I can confirm that my changes are working as intended

You did a great job here!
Please go ahead and close this pull request now. Thanks!


Allow the creation of engine when the record type don't have a parameter less constructor

Today we throw BadUsageException in the engine creation but maybe the user only want to write records, so there is no problem if there is no constructor.
Throw the BadUsageException only when trying to read objects
That will also speed up a bit the initial setup process because don't need to use reflection to take the constructor pointer on engine creation, that will be delayed to first read :)

It would be even better if one could create the record in the engine.BeforeReadRecord Event. This way you get the freedom to use different factories for record creation without parameter less constructors  and gain performance in the case of SkipThisRecord = true. Additionally you get the possibility to use different inherited types or classes implementing an interface. For backwards compatibility this behaviour is controlled with a boolean flag 'engine.DelayRecordCreation'.


Compare can be implemented for custom TypedValues

Before this PR, compare could only be implemented on a TypedValue if the TypedValue trait happened to expose its underlying representation in a useful way. e.g. string could implement compare using the to_str function.
For custom types implemented outside of this crate, this means that compare cannot be meaningfully implemented (other than maybe by parsing out the to_str implementation, or similar).
This series of commits (which are useful to review separately) makes it possible to actually implement compare for custom types, and adds a test showing that this works.

Hi Daniel, glad to get contribution from you :)
The PR looks good to me. Indeed it is not needed for Value to implemented TypedValue.


import 'url-change-event' gives error while importing library in angular

While importing this library in angular using import statement, it gives the following error:
'constructor', which lacks return-type annotation, implicitly has an 'any' return type.
constructor(option: IOption)
Following is the PR for solving this issue, please review
#16

constructor return void is better than any, i fixed this bug in 0.1.3, thanks for the report


IE11: Zoom-in and Zoom-out cursors are not supported

SEVERITY:  Major
MODULE: Bubble Chart
PLATFORM: Windows 7 / Internet Explorer 11.0
STEPS TO REPRODUCE:

Navigate to the URL: http://tools-stage.gapminderdev.org/tools/bubbles
Click on options and click on zoom
Click on zoom in or zoom out icon
Hover mouse on chart

ACTUAL RESULTS:
When user selects the zoom in or zoom out and hover the chart the zoom in or zoom out pointer (magnifier) did not appear instead the simple pointer is visible over the chart
EXPECTED RESULTS:
The magnifier icon of zoom in and zoom out should be visible
SCREENSHOT:
http://g.recordit.co/dOGWmSShEs.gif
http://g.recordit.co/SP4mfE5Co9.gif

Tested, works, cursor is a bit... win95'ish?


fix(ast)!: add missing AssignmentTargetProperty::computed

#8097  👈 (View in Graphite)
main

How to use the Graphite Merge Queue
Add either label to this PR to merge it via the merge queue:

0-merge - adds this PR to the back of the merge queue
hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.


Fix Resume for 1024 Token VAE

@lucidrains is there a way to determine what type the VAE is from the existing dalle params alone?  I feel like that's a bit more resilient than hard-coding each new pretrained VAE (assuming there are more).

@afiaka87 very nice! hmm, let's cross that bridge once we get there :)


Plotting for the 'TESS, Kepler, K2' dataset might be broken

There is a comment in this section of the plotting function saying that it is no longer in use.  I had to edit it to make the new Gaia speedup function work (go figure).
Since we need to touch the functions which retrieve these lightcurves using lightKurve in the next month, I leave this issue here as a placeholder to remind us (me) to check the plotting at the same time.

There were a couple of bugs that caused these bands to not be plotted in many cases. They are now fixed, and I've removed that comment from the plotting function. I tested a sample of 130 objects and found both TESS and K2 bands, but not Kepler. I assume there was just no Kepler data for those objects (but obviously can't be sure). I think we can close this issue now?

thanks for fixing, good to close.


Token from theme object not assignable to util

Bug report
When trying to use utils in combination with tokens from theme I get a ts error.
Describe the bug
import { styled, theme } from '../../../styles/stitches.config'

const Component = styled('span', {
  mv: theme.space.xs, // TypeError
})

// stitches.config:

export const {
  styled,
  theme,
} = createStitches({
  utils: {
    mv: (value: PropertyValue<'margin'>) => ({ marginTop: value, marginBottom: value }),
  },
})

Due to the lack of information on this topic in docs, along with inability to solve it with neither PropertyValue nor ScaleValue and missing export of appropriate type, I assume this might be a bug.
System information

Version of Stitches: ^1.2.8
Version of Node.js: 16.17.0
ts: 4.7.4

I'm not really sure what you want to do, but you don't need to import the theme object from the config to use your theme tokens.
By prefixing them with the scale name, you can pick a token from any of your available theme scales.
import { styled} from '../../../styles/stitches.config'

const Component = styled('span', {
  mv: "$space$xs"
})

You can read further in the docs here

@Code-Victor I know about the other way, but I want/need to import tokens from theme for better type control. The 'string token magic' turned out to be pretty unreliable with refactorings.

Then I think the best this is the best solution.
import { styled, theme } from '../../../styles/stitches.config'

const Component = styled('span', {
  mv: theme.space.xs.computedValue, // TypeError
})

It generates the serialized CSS var() representing the token(so in this case, var(--space-xs)).

Cool, thanks. It's actually a nice workaround.

Also, if you're working with multiple themes or want the exact value.
you can try this
import { styled, config} from '../../../styles/stitches.config'

const Component = styled('span', {
  mv: config.space.xs, // returns the exact value
})


Pop up to be pressed several times to continue navigation

Using a Database with about 50 elements it happens that when pressing on the Enities they are not always shown with all their properties and in case a new Risk Session is created (pressing new risk session button) nothing is displayed. Passing to Browse menu it appears the pop-up and pressing several times OK the page is displayed and the Risk Session is found

We need more detail about this issue:

What are you testing?
The entities (50?), where are not displayed?
What are you refering to with "50 elements", 50 entities?
-Are you running risk sessions (menu Run/Multi-layer analysis)? and you don't see all the entities in the  left tree?
Are you browing risk sessions (menu Browse/Risk analysis sessions)?
-What message appears in the pop-up?


Update install_standalone-docker-compose.md

hello, i have tried default command
"sudo docker compose up -d"
but it is error for docker compose file to run in a VM we required to run  "sudo docker-compose up -d"
only then it will run

hello please check this. "sudo docker-compose up -d" will only work


Add INSTALL with build instructions

That looks generic enough to apply to any platform in one sense, and specific enough to be Homebrew only (and a particular configuration thereof) rather than general to OS X...
Also, it should probably create an INSTALL file rather than bundling in README.

@luke-jr Fair enough. Updated PR as requested.

@luke-jr I've made those changes, but added a troubleshooting section:
./autogen.sh didn't actually work for me on OS X, but autoreconf --install did.
Similarly, libgcrypt wasn't picked up until I specified the env var.

What is the problem you had with autogen.sh?
dist_doc_DATA in Makefile.am needs updating.

@luke-jr Wasn't producing a Makefile with buildable targets after ./configure. Since the obvious alternative worked I didn't investigate too closely. Have updated my build env since hitting that issue, and just tested that ./autogen.sh now works. Perhaps outdated autotools.

Looks good. Maybe squash?

@luke-jr Squashed.


[DOCFIX] Inline documentation code snippets in MapR-FS.md

Partial substitution to a part of a certain profile.

alluxio-bot, check this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2650/
Test PASSed.


⚠ Secret Website 3 has degraded performance

In 44b10a8, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 8588 ms

Resolved: Secret Website 3 performance has improved in b65350b.


Create sh-checker.yml

The download of source code should not require root privileges.

It appears to work.


Send Slack message if the CI fails on main branch

Related to #940

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.7%) to 88.614%

Totals

Change from base Build<PHONE_NUMBER>:
-0.7%

Covered Lines:
2654

Relevant Lines:
2995

💛  - Coveralls


Export subcommand ignores encoding-fmt arg

snphost requires, but ignores encoding format argument on export and exports certificates as-is. Probably it should convert certificates to the specified encoding or encoding-fmt may be removed at all.
$ ls certs
ark.der  ask.der  vcek.der

$ file -i certs/*
certs/ark.der:  application/octet-stream; charset=binary
certs/ask.der:  application/octet-stream; charset=binary
certs/vcek.der: application/octet-stream; charset=binary

$ ./snphost/target/debug/snphost import certs

$ mkdir out

$ ./snphost/target/debug/snphost export pem out

$ file -i out/*
out/ark.pem:  application/octet-stream; charset=binary
out/ask.pem:  application/octet-stream; charset=binary
out/vcek.pem: application/octet-stream; charset=binary

$ diff certs/ark.der out/ark.pem

$ diff certs/ask.der out/ask.pem

$ diff certs/vcek.der out/vcek.pem

@sza-1 Thanks for pointing this out, at the moment we rely on the user to use properly, but a proper checking that the certs are of the correct type and conversion if necessary is desired.


TabView (Selected item)

Hi, could you add event handler property that invokes when, i tap on tap?

It's a useful enhancement, I'll add it to the milestone.
Until implementing it you can use the binding method to handle it.
It'll invoke the setter of the property that you used for binding. You can handle the changed from there.

Thanks, as I thought


Remove unnecessary node dependencies

consul is still part of package.json; I think that can be safely removed.
moment is used only at two places only to fetch current time, it can be replaced wiht inbuild Date api
Ideally, we shouldn't be having any dependency on external modules other than nats, node-nats-streaming and bunyan (where bunyan is really debateable, since it's dev only dependency imo)

is node-nats-streaming not needed anymore?

Everything seemed to work. We are not using the streaming part of nats.


add default process.env.LAMBDA_TASK_ROOT as resolve path to require l…

…ogic

Pull Request Test Coverage Report for Build 925

0 of 0   changed or added relevant lines in 0 files are covered.
47 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.005%) to 68.947%

Files with Coverage Reduction
New Missed Lines
%

dist/plugins/integrations/AWSIntegration.js
47
70.58%

Totals

Change from base Build 921:
-0.005%

Covered Lines:
2652

Relevant Lines:
3595

💛  - Coveralls


Fix config and log file permissions to work with latest Bosh stemcells (3541.x)

What

Chown the datadog.conf to vcap:vcap after copying

In newer Bosh stemcells (3541.x) the /var/vcap/* permissions are hardened and the datadog.conf file is not readable by the vcap user.

Always create forwarder.log file

Make sure the forwarder.log file exists otherwise it will be created with owner root:root when ddagent.py is started the first time. The output redirection is not affected by chpst.

Always create dogstatsd.log file

Make sure the dogstatsd.log file exists otherwise it will be created with owner root:root when dogstatsd.py is started the first time. The output redirection is not affected by chpst.

Thank you for the updates! I really appreciate the extra time to keep this up to date.


UAT | Pending Task Localization

Localization of Case name not appearing in Pending Taks

Hi @Taherabharmal  Can you please provide more context for this?
The case name is made with this name only it seems.
cc:- @Ramu-kandimalla

this is not an issue


ec_host_cmd: uart: do not send response twice

There is a possibility, that a command handler that calls ec_host_cmd_send_response function returns anyway (which is a mistake). It would cause sending a response twice, which isn't allowed.
Make sure that a response is sent only in the UART_HOST_CMD_PROCESSING state.

Sorry for disturbing, I've changed me mind and I'll upload checking HC state in the general code, for all backends.


Develop -> Master

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Using Phonegap Build cli 8.0.0, getting this error: "Error: ENOENT: no such file or directory, open '/AndroidManifest.xml'."

Using Phonegap Build, I tried updating to cli 8.0.0, as advised, to get cordova-plugin-firebase working properly. However, I am now getting this build error: "Error: ENOENT: no such file or directory, open '/AndroidManifest.xml'."
Any ideas on how to fix this issue in Phonegap Build? Or, since my main objective is to get any admob and firebase push notifications working together properly, whether in Phonegap Build cli 8.0.0 or cli 7.1.0, alternate suggestions for that would be appreciated.
Thanks for any help.
Here is the error in the build log:
Android Studio project detected
"plugman-151 install --platform android --project  --plugin cordova-plugin-firebase --nohooks=.": Fetching plugin "cordova-plugin-firebase" via npm
Installing "cordova-plugin-firebase" at "1.0.5" for android
Failed to install 'cordova-plugin-firebase': Error: ENOENT: no such file or directory, open '/AndroidManifest.xml'
    at Error (native)
    at Object.fs.openSync (fs.js:640:18)
    at Object.fs.readFileSync (fs.js:508:33)
    at Object.parseElementtreeSync (/cordova/node_modules/cordova-common/src/util/xml-helpers.js:180:27)
    at new AndroidManifest (/cordova/lib/AndroidManifest.js:29:20)
    at AndroidProject.getPackageName (/cordova/lib/AndroidProject.js:99:12)
    at Api.addPlugin (/cordova/Api.js:223:57)
    at handleInstall (/usr/local/lib/npm/lib/node_modules/pgb-plugman-151/node_modules/pgb-cordova-lib/src/plugman/install.js:594:10)
    at /usr/local/lib/npm/lib/node_modules/pgb-plugman-151/node_modules/pgb-cordova-lib/src/plugman/install.js:364:24
    at _fulfilled (/usr/local/lib/npm/lib/node_modules/pgb-plugman-151/node_modules/q/q.js:787:54)
ENOENT: no such file or directory, open '/AndroidManifest.xml'

same error , any solution ?

Android 7.0+ changed the AndroidManifest location. it seems most plugins don't know how to look for it in its new location
I tried symlinking it:
ln -s platforms/android/app/src/main/AndroidManifest.xml platforms/android/AndroidManifest.xml
Ended up working, but then the next error is any plugin adds/deletes triggers a .gradle invalid error

@jamminjames is this still an issue with the latest version?

@soumak77 I don't know, I resorted to using Pushwoosh.

@soumak77 I can confirm it is an issue with the latest version.


Fix: add navigation support to history

Makes sure the history.back() / go() ... methods are supported.
Replaces replaceHistory with pushHistory.
Rewrites a few specs

Codecov Report

Merging #25 into master will increase coverage by 0.78%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #25      +/-   ##
==========================================
+ Coverage   95.27%   96.06%   +0.78%
==========================================
  Files           5        5
  Lines         127      127
  Branches       26       26
==========================================
+ Hits          121      122       +1
+ Misses          5        4       -1
  Partials        1        1

Impacted Files
Coverage Δ

history.js
100% <100%> (ø)
:arrow_up:

index.js
94.54% <100%> (+1.81%)
: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 410a79d...65ad697. Read the comment docs.


When using RadosGW, if an operation is specified on a bucket that does not exist the error string is empty

I've tracked this issue down to the bucket location request 404 response not containing a Message field. I think the right answer is to have the code that falls back to HTTP errors also do so if the Message field is empty, like diff below.
For reference here is the XML I received:
<?xml version="1.0" encoding="UTF-8"?><Error>NoSuchBucket<BucketName>foo</BucketName><RequestId>tx00000000000000000be1f-00584b26f6-100d-default</RequestId><HostId>100d-default-default</HostId></Error>
diff --git a/vendor/github.com/minio/minio-go/api-error-response.go b/vendor/github.com/minio/minio-go/api-error-response.go
index bcfad37..65cb8b3 100644
--- a/vendor/github.com/minio/minio-go/api-error-response.go
+++ b/vendor/github.com/minio/minio-go/api-error-response.go
@@ -93,7 +93,7 @@ func httpRespToErrorResponse(resp *http.Response, bucketName, objectName string)
        var errResp ErrorResponse
        err := xmlDecoder(resp.Body, &errResp)
        // Xml decoding failed with no body, fall back to HTTP headers.
-       if err != nil {
+       if err != nil || errResp.Message == "" {
                switch resp.StatusCode {
                case http.StatusNotFound:
                        if objectName == "" {

This not the right approach this means S3 compatibility issues @tjenkins0 - hand crafting hacks like will make bugs go silent - we can't do this across all our libraries.
http://docs.aws.amazon.com/AmazonS3/latest/API/ErrorResponses.html are listed properly - so it should be addressed on the server.

Ok, thanks for the feedback. I'll take it up with Ceph.

Ok, thanks for the feedback. I'll take it up with Ceph.

Thanks @tjenkins0 let us know if you have any other issues with minio-go happy to help. Closing this for now.


selected network controller: redirect domains to default endpoint

Explanation
Changes how the SelectedNetworkController responds to state updates from the network controller.  Now when the default RPC endpoint changes for a network, it will redirect domains that were referencing a network client id on that network to the new default endpoint for that network.  Instead of redirecting them to the globally selected network, as happened before.
Accompanies https://github.com/MetaMask/metamask-extension/pull/26433

References

Changelog

@metamask/selected-network-controller

CHANGED: When the default RPC endpoint changes for a network, domains that were referencing a network client id on that network are redirected to the new default RPC endpoint.

Checklist

[ ] I've updated the test suite for new or updated code as appropriate
[ ] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
[ ] I've highlighted breaking changes using the "BREAKING" category above as appropriate

I will also update the migration in https://github.com/MetaMask/metamask-extension/pull/26433 to point domains to their default endpoint so its correct initially.


Readthedocs build is failing because of the empty pyproject.toml file

Since you created the pyproject.toml file, the docs build is broken because it complains about missing requirements.
See https://readthedocs.org/projects/python-scalpel/builds/18041070/

PR is merged now.


Resolve the error that appears when trying creating the same service principal n times

Resolve the error that appears when trying creating the same service principal n times

To solve this issue I change the client secret gen and store it in correct way across script execution


wave section for the contactUS page.

Description of the enhancement
A part for the section is behaving differently in the ContactUS page.
Screenshots
Screenshots of the proposed changes compared with the current version of the project

@aashimgarg close this now! it has been solved


"Add a tag to your device" may need updating or clarification

Currently the instructions in the section "Add a tag to your device" instruct the user to edit the device twin JSON and set a tag there. Unfortunately I had no luck doing that; the portal ditched my edits upon saving every time I tried it. Editing device tags directly on the device page seemed to produce the intended result, however.
Perhaps this part of the tutorial should be updated.
If editing the JSON data should still be the preferred method, perhaps the sample JSON should contain some of the surrounding JSON document to make it more obvious which part of the document the addition should be made in?

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

ID: fae57922-c3b2-5fb6-230a-9be79c40c774
Version Independent ID: 5650812e-bdcf-8f72-dfec-36474795f474
Content: Device Update for Azure IoT Hub tutorial using the Ubuntu Server 18.04 x64 package agent
Content Source: articles/iot-hub-device-update/device-update-ubuntu-agent.md
Service: iot-hub-device-update
GitHub Login: @vimeht
Microsoft Alias: vimeht

Hi @patheikkinen Thank you for your feedback. We will review it and update the document as appropriate.

Hi @patheikkinen Thank you for reporting this issue to us. We have created a PR to add more details to the step per your suggestion. The document will be updated with additional details after the PR has been merged. The updates include details pointing out the section of device twin where the tag needs to be included and an image supporting the same.
Hope this addresses your concerns.
We will now proceed with closing the issue. Please do not hesitate to tag me in the comments below if you need any further assistance with regards to this issue.
Best regards,
Leela Rajesh.


Typed CBORObject.FromInt etc.

Currently to convert a primitive to CBOR, CBORObject.FromObject is used which has a lot of overloads. This gives a lack of safety, as overload resolution is affected by op_Implicit, and also has an obj overload which makes it extra unsafe.
Instead there should be added FromInt, FromInt64, FromChar, FromArray, FromGuid etc..

This is causing NativeAOT incompatibilities now:
ILC : Trim analysis warning IL2067: PeterO.Cbor.PropertyMap.TypeToObject(CBORObject,Type,CBORTypeMapper,PODOptions,Int32): 'type' argument does not satisfy 'DynamicallyAccessedMemberTypes.PublicParameterlessConstructor' in call to 'System.Activator.CreateInstance(Type)'. The parameter '#1' of method 'PeterO.Cbor.PropertyMap.TypeToObject(CBORObject,Type,CBORTypeMapper,PODOptions,Int32)' does not have matching annotations. The source value must declare at least the same requirements as those declared on the target location it is assigned to.
ILC : Trim analysis warning IL2067: PeterO.Cbor.PropertyMap.TypeToObject(CBORObject,Type,CBORTypeMapper,PODOptions,Int32): 'type' argument does not satisfy 'DynamicallyAccessedMemberTypes.PublicConstructors' in call to 'System.Activator.CreateInstance(Type,Object[])'. The parameter '#1' of method 'PeterO.Cbor.PropertyMap.TypeToObject(CBORObject,Type,CBORTypeMapper,PODOptions,Int32)' does not have matching annotations. The source value must declare at least the same requirements as those declared on the target location it is assigned to.
ILC : AOT analysis warning IL3050: PeterO.Cbor.PropertyMap.TypeToObject(CBORObject,Type,CBORTypeMapper,PODOptions,Int32): Using member 'System.Array.CreateInstance(Type,Int32[])' which has 'RequiresDynamicCodeAttribute' can break functionality when AOT compiling. The code for an array of the specified type might not be available.
ILC : AOT analysis warning IL3050: PeterO.Cbor.PropertyMap.TypeToObject(CBORObject,Type,CBORTypeMapper,PODOptions,Int32): Using member 'System.Type.MakeGenericType(Type[])' which has 'RequiresDynamicCodeAttribute' can break functionality when AOT compiling. The native code for this instantiation might not be available at runtime.
ILC : AOT analysis warning IL3050: PeterO.Cbor.PropertyMap.TypeToObject(CBORObject,Type,CBORTypeMapper,PODOptions,Int32): Using member 'System.Type.MakeGenericType(Type[])' which has 'RequiresDynamicCodeAttribute' can break functionality when AOT compiling. The native code for this instantiation might not be available at runtime.
ILC : Trim analysis warning IL2067: PeterO.Cbor.PropertyMap.TypeToObject(CBORObject,Type,CBORTypeMapper,PODOptions,Int32): 'type' argument does not satisfy 'DynamicallyAccessedMemberTypes.PublicParameterlessConstructor' in call to 'System.Activator.CreateInstance(Type)'. The parameter '#1' of method 'PeterO.Cbor.PropertyMap.TypeToObject(CBORObject,Type,CBORTypeMapper,PODOptions,Int32)' does not have matching annotations. The source value must declare at least the same requirements as those declared on the target location it is assigned to.
ILC : AOT analysis warning IL3050: PeterO.Cbor.PropertyMap.TypeToObject(CBORObject,Type,CBORTypeMapper,PODOptions,Int32): Using member 'System.Type.MakeGenericType(Type[])' which has 'RequiresDynamicCodeAttribute' can break functionality when AOT compiling. The native code for this instantiation might not be available at runtime.
ILC : AOT analysis warning IL3050: PeterO.Cbor.PropertyMap.TypeToObject(CBORObject,Type,CBORTypeMapper,PODOptions,Int32): Using member 'System.Type.MakeGenericType(Type[])' which has 'RequiresDynamicCodeAttribute' can break functionality when AOT compiling. The native code for this instantiation might not be available at runtime.
ILC : AOT analysis warning IL3050: PeterO.Cbor.PropertyMap.TypeToObject(CBORObject,Type,CBORTypeMapper,PODOptions,Int32): Using member 'System.Type.MakeGenericType(Type[])' which has 'RequiresDynamicCodeAttribute' can break functionality when AOT compiling. The native code for this instantiation might not be available at runtime.

I don't see how to address the warnings you mention and I don't see what to do to show those warnings, even assuming I add methods like CBORObject.FromInt32, etc.

I've made a draft PR with the scaffolding to reproduce this: https://github.com/peteroupc/CBOR/pull/69

I did some initial work on this.
Effectively there needs to be a division between a type-safe part of the library and a reflection-based part. The reflection-based parts should be marked as such, and the type-safe parts shouldn't call them. The best such line is between things that take one normal type to another, which are safe, and things that start with obj and do type testing, or things that take a Type argument, which are not.


Module not found: Error:

Bug Report
Hi Team React-Virtualized,
I installed this "react-virtualized": "^9.21.2", and after this when ran my project then facing below errors.

ERROR in ../node_modules/react-virtualized/dist/es/Collection/CollectionView.js
Module not found: Error: Can't resolve 'dom-helpers/scrollbarSize' in

ERROR in ../node_modules/react-virtualized/dist/es/Grid/Grid.js
Module not found: Error: Can't resolve 'dom-helpers/scrollbarSize'

Issue 01)
Path not correct Inside the CollectionView.js for import getScrollbarSize from 'dom-helpers/scrollbarSize';
Solution 01) It's working fine when I used this path  import getScrollbarSize from '../../../node_modules/dom-helpers/scrollbarSize';
Issue 02)
Path not correct Inside the Grid.js for import scrollbarSize from 'dom-helpers/scrollbarSize';
Solution 02) It's working fine when I used this path  import scrollbarSize from '../../../node_modules/dom-helpers/scrollbarSize';
Please see attached images.

Which versions of React and react-virtualized, and which browser / OS are affected by this issue? Did this work in previous versions of react-virtualized?

Browser
Version 84.0.4147.105 (Official Build) (64-bit)

OS
Windows 10

React
"react": "^16.4.0",

React DOM
"react-dom": "^16.4.0",

react-virtualized
"react-virtualized": "^9.21.2"

Thanks,
BSR

@rathorebs I managed to get around this issue with this in package.json:
"resolutions": {
    "dom-helpers": "^5.1.3"
}

I'm guessing that you (or one of your dependencies) are using an older version of dom-helpers, which did not yet have scrollbarSize.

@mbonaci I have added above solution but still facing same error.

I faced this issue today and wasted an hour trying to fix it by playing with the versions in the resolutions in package.json and I used npm-force-resolutions to force the correct version in all my dependencies.
unfortunately, I couldn't figure out a solution BUT I have a workaround if someone is using webpack
as you can see from the error message somewhere in your dependencies the code tries to import from dom-helpers/scrollbarSize but in dom-helpers the scrollbarSize is inside dom-helpers/uitl/scrollbarSize
so as a solution I added an alias for dom-helpers/scrollbarSize to point to the correct place
webpack.config.js
module.exports = {
  entry: {
     .....
  },
  output: {
      ...
  },
  module: {
    rules: [....]
  },
  resolve: {
    modules: [
      path.resolve('./node_modules')
    ],
    alias: {
      'dom-helpers/scrollbarSize': path.resolve(__dirname, 'node_modules', 'dom-helpers', 'util', 'scrollbarSize'),
      .....
    }
  },
  plugins: [...]
};

just a temp solution I guess

@mbonaci I'm reopening as still same error and not fixed.

What happens if you downgrade to "react-virtualized": "9.21.0"?
FYI I'm not affiliated with this project in any way.


opt Update test_hive_query_cache.groovy (#29694)

set longer timeout to avoid case fail
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Add carbon community to footer

Description
Added a link in the footer for joining the carbon community.

As of right now, we are going to to leave it on our README alone until there is more interest. If people start engaging more, and we have access to moderate the channels, then we can reconsider adding it to our lander.

we have access to moderate the channels

I can transfer the community to you, I have no issues. 😀

That would be great!

@mfix22 The carbon community has been transferred to you. Now you can merge this pull request.


Eichrecht : Signed Data is not correctly transmited with some systems

Signed Data is only taken from MeterData where it should be taken from StopTransaction as well.

@jerome-benoit draft PR coming tommorow at the latest

@jerome-benoit draft PR coming tomorrow at the latest

@WillyLellouch We are tomorrow since yesterday. Could you please give us a status?

@jerome-benoit I was a bit too optimistic with the timing. I've encountered a bug in my implementation that I'm resolving today. Also, should I include my unit tests in the PR or make a separate one ?

You can put everything in the same PR. Thanks.

@WillyLellouch: 5 more days since without any status update on a customer critical issue. What the hell is happening?

@jerome-benoit Sorry for the lack of communication. I've finished to implement my solution but I have some trouble with the unit tests.
As in I can't run them and I don't know why that is.
The solution I implemented is the simplest I could do. the reason it took longer than expected is that I originally went in a completely different direction that required me to rewrite large sections of code and I got lost in the weeds. My new code only changes 1 file.
Since I could not do the Unit tests properly, I'll do the PR anyway and you can see if this is satisfactory. Who must I ask for indications as to why my Unit Tests are not operative ?

The checks at PRs do a unit tests run.
For developers support, we  have a teams and a mailing list to ask for it.
Thanks.

The fix is non functional with ChargeX.
I'm investigating to see what went wrong in testing

@LucasBrazi06 The tests were a success. There were no additional issues to report

@LucasBrazi06 The tests were a success. There were no additional issues to report

Ok, perfect!
Thanks!


Play MP3 on cardputer

Is there a way to play MP3 from SDcard or SPIFF on cardputer?
I see that the speaker it's not yet compatible, is this planned ?

I found a way 😜 closing the issue


[FEATURE REQUEST] Run scripts in the server's command

Discussed in https://github.com/spiral/roadrunner/discussions/796

Originally posted by rustatian September 13, 2021
Hey, RR community. I want to ask you about one feature.
How likely do you guys need to have the ability to execute scripts in the server plugin, like this:
Script sample:
#!/usr/bin/env bash
setenv, generate config, etc etc etc
php ../roadrunner/tests/jobs_ok.php <- run PHP or bin

rr.yaml sample:
server:
    command: "./start.sh" <-- script
    relay: "pipes"
    relay_timeout: "20s"

At the moment we have a limitation around such configuration because we check for the process PID and worker's PID to properly attach relay and they are mismatches. However, for such configuration, the child process will inherit process descriptors from the parent and we may safely send data to the script fd without checking the worker's FD.

I use RR as entrypoint in docker so I really would love to have some way of moving "boot" script to it. I need to warmup symfony, check some services etc. And only when worker is waiting for request and is actually ready I send requests to it. I have hard dependency for docker health check.
My usual config looks like: server: "php --ini /service/docker/php.ini /service/server". server is entry point for rr and console applications. Depends on RR_MODE and php_sapi_name().

Hey @OO00O0O . Thanks for your feedback.
on_boot feature looks natural to me. This is a good proposal. Could you please describe your on_boot case in a separate ticket and I'll discuss this with our PHP team?

#57 - done.
Back on topic: if RR can accept any script, then it makes no difference who communicates with it. I mean RR could run ANY worker in ANY language that can work with protobuf and/or json(for fallback).


Club trading update

Justin's club trading updates.

Closing this as I updated stats in 94e881a2f6cdc7b1fc75cce5195a1ed9173e3b93. I've done a quick check and things seem accurate and up to date but feel free to point out any issues you see.


No more wildcard imports and general import cleanup

[x] I have followed the instructions in the PR template

Not sure if wildcard imports are wanted or not, but this removes them and also changes it to warnings.
In addition, there are a number of fixes where the "correct" import location is used for emath and Color32. That is, the re-exports are not used. Again, I do not know the stance of this, but to keep things consistent in the code base and possibly simplify deprecating the re-exports etc now that there are multiple crates.
Edit: Seems only half-finished, but any input if I should finish it is appreciated.

Closing and adding parts in separate PRs. Will cause too many conflicts and probably wildcard imports are wanted every now and then...


fix for $http.get().success removal in angular 1.6

Sorry for the white spaces changes, have no idea when it happened. Real changes are from line 552 and 724.Hope possible to merge it without whitespaces.
http://stackoverflow.com/questions/9776527/merging-without-whitespace-conflicts

Coverage decreased (-0.08%) to 97.706% when pulling cd33dc8c23ccff17576835a529e337cdca3b7600 on p1lare:master into b62815ab23d40683da993ca170a326cabd8799f6 on Zizzamia:master.

I agree this is a blocking issue for all projects using ngTasty and wanting to upgrade to 1.6. The tests have to be fixed though.


feat(STONEINTG-1009): e2e tests for group snapshot

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Summary:
Adding a multirepo and single repo(multicomponent) scenarios for creation of group snapshot.
Upcoming changes post mortem
Reduce the triplication of component A, B and C into one for loop to reduce code amount.
Include gitLab repo for scenario of multiple repos having a group snapshot.
Issue ticket number and link
Type of change

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:

[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
[ ] I have updated labels (if needed)

/retest

/retest

/retest

/retest

/retest

@jsztuka , great job Kudos !

/lgtm


[iOS] AutofillGroup only fills the selected field

I'm facing the problem described in this issue: https://github.com/flutter/flutter/issues/99356. Thread has been closed, but the problem is still relevant to me.
In my application, there are three input fields that are grouped together with AutofillGroup. autofillHints value is specified for each field. But autocomplete only works for the current field that has focus.
Device: iPhoneSE (1st gen), iOS 15.5
https://user-images.githubusercontent.com/41895601/213684942-c956a337-9b6d-4570-a258-6a1eb1a678d7.mp4
Steps to Reproduce

Run code sample on ios device
Try to populate the First name field with suggested autocomplete value

Expected results: all form fields are filled in automatically
Actual results: only the firs name field is filled. Autocomplete only works for the field that has focus

Code sample
import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  static const String _title = 'Flutter Code Sample';

  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: _title,
      home: Scaffold(
        appBar: AppBar(title: const Text(_title)),
        body: const MyStatefulWidget(),
      ),
    );
  }
}

class MyStatefulWidget extends StatefulWidget {
  const MyStatefulWidget({Key? key}) : super(key: key);

  @override
  State<MyStatefulWidget> createState() => _MyStatefulWidgetState();
}

class _MyStatefulWidgetState extends State<MyStatefulWidget> {
  final TextEditingController _firstNameController = TextEditingController();
  final TextEditingController _secondNameController = TextEditingController();
  final TextEditingController _emailController = TextEditingController();

  @override
  Widget build(BuildContext context) {
    return ListView(
      padding: const EdgeInsets.symmetric(horizontal: 16),
      children: [
        AutofillGroup(
          child: Column(
            children: [
              TextFormField(
                controller: _firstNameController,
                autofillHints: const [AutofillHints.givenName],
                decoration: const InputDecoration(labelText: 'First name'),
                textInputAction: TextInputAction.next,
                keyboardType: TextInputType.name,
                autocorrect: false,
              ),
              TextFormField(
                controller: _secondNameController,
                autofillHints: const [AutofillHints.familyName],
                decoration: const InputDecoration(labelText: 'Second name'),
                textInputAction: TextInputAction.next,
                keyboardType: TextInputType.name,
                autocorrect: false,
              ),
              TextFormField(
                controller: _emailController,
                autofillHints: const [AutofillHints.email],
                decoration: const InputDecoration(labelText: 'Email'),
                textInputAction: TextInputAction.done,
                keyboardType: TextInputType.emailAddress,
                autocorrect: false,
              ),
            ],
          ),
        ),
      ],
    );
  }

  @override
  void dispose() {
    _firstNameController.dispose();
    _secondNameController.dispose();
    _emailController.dispose();

    super.dispose();
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 3.3.10, on macOS 12.6 21G115 darwin-arm, locale en-AU)
    • Flutter version 3.3.10 on channel stable at /Users/ssheromova/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 135454af32 (5 weeks ago), 2022-12-15 07:36:55 -0800
    • Engine revision 3316dd8728
    • Dart version 2.18.6
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/ssheromova/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • ANDROID_HOME = /Users/ssheromova/Library/Android/sdk
    • Java binary at: /Users/ssheromova/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/212.5712.43.2112.8815526/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14A400
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Users/ssheromova/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/212.5712.43.2112.8815526/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] VS Code (version 1.74.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.56.0

[✓] Connected device (3 available)
    • iPhone SE (1st generation) (mobile) • e4fc1b9fae0ff96c53ec310ab3326ad492784934 • ios            • iOS 15.5 19F77
    • macOS (desktop)                     • macos                                    • darwin-arm64   • macOS 12.6 21G115 darwin-arm
    • Chrome (web)                        • chrome                                   • web-javascript • Google Chrome 109.0.5414.87

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Thanks for the report @SofiSheromova
Per flutter team member's comment in the original issue:
If I remember correctly ios only autofills multiple fields for username + password? If autofilling first name + last name isn't supported by the platform's autofill service then we won't be able to do that in flutter either.
So may it works as intended currently since from your video, you have first name, last name and email fields. Can you try using username and password fields instead and see if focusing on username also populates password too ? If so, then the comment holds true.

@darshankawar thanks for your answer!
I did not really trust the quoted comment, as I did not find other reliable sources. For me, autocomplete works fine in combination with email + password. It is very sad that other groupings are not available by the platform(

Thanks for the update. Is there a way for you to verify this on a native iOS app and see how it works ? If it properly works with fields other than email and password and not in Flutter, then it would make sense for Flutter to have that functionality as well, but in native iOS too, if it doesn't work as expected, then Flutter also can't make it work.

I'm sorry, I seem to have mislead you. When I said email/password combination, I meant that it works well in my Flutter app :)
I think the issue can be closed. You're right that iOS doesn't support other groupings, I was convinced of this by checking native applications.


Minor docs improvements

Document __bool__.
Document in __add__ and __mul__ that the reverse operator (vector-like op vector) also exists.
Use the imperative form in short-form descriptions (i.e. the docstring's first line).
Use asserts in examples where relevant, rather than showing True or implicitly expecting users to notice an equality.
Other minor changes.

bors retry


Adds support for distribution and distributor to ISO 19115-2 reader

Related to:

https://github.com/GSA/data.gov/issues/4894
https://github.com/GSA/data.gov/issues/4862

@rshewitt I updated tests and addressed your feedback.


Setup Redux Store

Description
Trello: https://trello.com/c/gZ8v4Zjl/12-feature-redux-setup
What I did:
I setup a redux store for state management of this project. I redid this pull request to fix the package-lock.json issues I was having.
Pair-programmed with Durell Jules
Driver & Navigator
Type of change

[x] Feature

Change Status

[x] Complete, tested, ready to review and merge

Has This Been Tested

[x] Yes

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My code has been reviewed by at least one peer
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] There are no merge conflicts

https://user-images.githubusercontent.com/10622915/150256867-2f1e0e26-c161-4552-9062-9bfd2b94fe5d.mp4

 Feature/Redux-setup

Really great video and explanation, especially on a topic that's rather complex. Code looks good :)


Not Working

Hello
I think your service not works,
when i use createInvoice

Hi there,
We announced over 3 months ago that v1 will be deprecated starting april. My guess is that you are still using v1?
Please review the NPM docs to migrate to v2, that should fix your issues.
Kind regards


How to update an IdentityUser within CosmosDB?

Discussed in https://github.com/MoonriseSoftwareCalifornia/AspNetCore.Identity.CosmosDb/discussions/23

Originally posted by etriebe February  2, 2024
I'm trying to update objects within CosmosDB just like I've done with other objects but I get an exception when I do so. I'm only updating username but I get this error.
Microsoft.Azure.Cosmos.CosmosException
  HResult=0x80131500
  Message=Response status code does not indicate success: BadRequest (400); Substatus: 0; ActivityId: 2aaf5d6d-4433-4488-8f52-905849797594; Reason: (Message: {"Errors":["One of the specified inputs is invalid"]}
ActivityId: 2aaf5d6d-4433-4488-8f52-905849797594, Request URI: /apps/8ce365e5-ba15-438a-a471-07d6b564394d/services/22cbecbc-8764-4e5b-b75b-76fa9b95216b/partitions/fd5ad3bd-b31c-455f-868a-f9c2ca0c97c2/replicas/133265169497594395p/, RequestStats: Microsoft.Azure.Cosmos.Tracing.TraceData.ClientSideRequestStatisticsTraceDatum, SDK: Windows/10.0.22631 cosmos-netstandard-sdk/3.31.4);
  Source=Microsoft.Azure.Cosmos.Client
  StackTrace:
   at Microsoft.Azure.Documents.StoreResult.ToResponse(RequestChargeTracker requestChargeTracker)
   at Microsoft.Azure.Documents.ConsistencyWriter.WritePrivateAsync(DocumentServiceRequest request, TimeoutHelper timeout, Boolean forceRefresh)
   at Microsoft.Azure.Documents.BackoffRetryUtility`1.ExecuteRetryAsync[TParam,TPolicy](Func`1 callbackMethod, Func`3 callbackMethodWithParam, Func`2 callbackMethodWithPolicy, TParam param, IRetryPolicy retryPolicy, IRetryPolicy`1 retryPolicyWithArg, Func`1 inBackoffAlternateCallbackMethod, Func`2 inBackoffAlternateCallbackMethodWithPolicy, TimeSpan minBackoffForInBackoffCallback, CancellationToken cancellationToken, Action`1 preRetryCallback)
   at Microsoft.Azure.Documents.ShouldRetryResult.ThrowIfDoneTrying(ExceptionDispatchInfo capturedException)
   at Microsoft.Azure.Documents.BackoffRetryUtility`1.ExecuteRetryAsync[TParam,TPolicy](Func`1 callbackMethod, Func`3 callbackMethodWithParam, Func`2 callbackMethodWithPolicy, TParam param, IRetryPolicy retryPolicy, IRetryPolicy`1 retryPolicyWithArg, Func`1 inBackoffAlternateCallbackMethod, Func`2 inBackoffAlternateCallbackMethodWithPolicy, TimeSpan minBackoffForInBackoffCallback, CancellationToken cancellationToken, Action`1 preRetryCallback)
   at Microsoft.Azure.Documents.ConsistencyWriter.WriteAsync(DocumentServiceRequest entity, TimeoutHelper timeout, Boolean forceRefresh, CancellationToken cancellationToken)
   at Microsoft.Azure.Documents.ReplicatedResourceClient.<>c__DisplayClass31_0.<<InvokeAsync>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at Microsoft.Azure.Documents.RequestRetryUtility.ProcessRequestAsync[TRequest,IRetriableResponse](Func`1 executeAsync, Func`1 prepareRequest, IRequestRetryPolicy`2 policy, CancellationToken cancellationToken, Func`1 inBackoffAlternateCallbackMethod, Nullable`1 minBackoffForInBackoffCallback)
   at Microsoft.Azure.Documents.ShouldRetryResult.ThrowIfDoneTrying(ExceptionDispatchInfo capturedException)
   at Microsoft.Azure.Documents.RequestRetryUtility.ProcessRequestAsync[TRequest,IRetriableResponse](Func`1 executeAsync, Func`1 prepareRequest, IRequestRetryPolicy`2 policy, CancellationToken cancellationToken, Func`1 inBackoffAlternateCallbackMethod, Nullable`1 minBackoffForInBackoffCallback)
   at Microsoft.Azure.Documents.StoreClient.ProcessMessageAsync(DocumentServiceRequest request, CancellationToken cancellationToken, IRetryPolicy retryPolicy)
   at Microsoft.Azure.Cosmos.Handlers.TransportHandler.ProcessMessageAsync(RequestMessage request, CancellationToken cancellationToken)
   at Microsoft.Azure.Cosmos.Handlers.TransportHandler.SendAsync(RequestMessage request, CancellationToken cancellationToken)

  This exception was originally thrown at this call stack:
    [External Code]
    Pickem.Shared.Utilities.Data.IdentityDBUtils.UpdateUserAsync(Microsoft.AspNetCore.Identity.IdentityUser) in IdentityDBUtils.cs
```</div>

Hello etriebe!
Sorry for the late reply. Give me a day or two to research this and get back to you.
Thanks!
-Eric

Meanwhile, this might help. I have a test for setting the username. This may not be what you want to do since it is not exactly the same as what you are trying to do.
See below to see if this might help:
// This method is tested with SetUserNameAsync().
[TestMethod()]
public async Task SetNormalizedUserNameAsyncTest()
{
// Arrange
using var userStore = _testUtilities.GetUserStore();
var user = await GetMockRandomUserAsync(userStore);
var newEmail = $"A{GetNextRandomNumber(111<EMAIL_ADDRESS>        // Act
        await userStore.SetNormalizedUserNameAsync(user, newEmail.ToUpper());

        // Assert
        var user2 = await userStore.FindByIdAsync(user.Id);
        Assert.AreEqual(newEmail.ToUpper(), user2.NormalizedUserName);
    }

I had been using Cosmos DB functions to do an UpsertItemAsync.
ItemResponse<IdentityUser> udpatedUser = await this.container.UpsertItemAsync<IdentityUser>(user, new PartitionKey(user.Id));

What is that _testUtilities object type?

TestUtilities  is a class used to setup unit tests in the project AspNetCore.Identity.CosmosDb.Tests.csproj.  It does things like gets configuration from secrets, has test user IDs and email addresses, gets database options, mocks up role and user store and more.  The unit tests run through various functions to make sure things are working right. Often it is handy to look at a unit test to see how something is done.

I'm thinking back to your question above about changing user names.
With the UserManager object there is a method called SetUserNameAsync.  The benefit of using this is it saves the new user name, plus the "normalized" value.
This code snipped from the unit tests might be more to what you need:
 [TestMethod]
 public async Task SetUserNameAsyncTest()
 {
     // Arrange
     using var userManager = GetTestUserManager(_testUtilities.GetUserStore());
     var user = await GetTestUser(userManager);
     var userName = "Az" + user.UserName;

     // Act
     await userManager.SetUserNameAsync(user, userName);

     // Assert
     user = await userManager.FindByIdAsync(user.Id);
     Assert.IsTrue(user.UserName == userName);
 }

The "GetTestUserManager" method above is in the TestUtilities class. It mocks a test user.

Gotcha, yeah, I'm trying out using UserManager now. Guessing that'll fix
things for me! Thank you!
On Thu, Feb 15, 2024 at 9:29 AM Eric Dean Kauffman @.***>
wrote:

I'm thinking back to your question above about changing user names.
With the UserManager object there is a method called SetUserNameAsync. The
benefit of using this is it saves the new user name, plus the
"normalized" value.
This code snipped from the unit tests might be more to what you need:
[TestMethod]
public async Task SetUserNameAsyncTest()
{
// Arrange
using var userManager = GetTestUserManager(_testUtilities.GetUserStore());
var user = await GetTestUser(userManager);
var userName = "Az" + user.UserName;
 // Act
 await userManager.SetUserNameAsync(user, userName);

 // Assert
 user = await userManager.FindByIdAsync(user.Id);
 Assert.IsTrue(user.UserName == userName);

}
The "GetTestUserManager" method above is in the TestUtilities class. It
mocks a test user.
—
Reply to this email directly, view it on GitHub
https://github.com/MoonriseSoftwareCalifornia/AspNetCore.Identity.CosmosDb/issues/24#issuecomment-1946686679,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZH2OFFUXRLGKANADWBOWDYTZAWRAVCNFSM6AAAAABC5EDOIWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBWGY4DMNRXHE
.
You are receiving this because you were mentioned.Message ID:
<MoonriseSoftwareCalifornia/AspNetCore.Identity.
@.***>

That worked! Thanks so much!


"Misplaced &" in the document

There are some typos and undefined variables in the documents.

In the document SoftBoundsReferenceDevice, there is a Misplaced & error in the equation.

There is incorrect alpha in SoftBoundsReferenceDevice.subtract_symmetry_point.

The variables $\alpha_+$ and $\alpha_-$ seem to be undefined in the documents.

There is an unrendered LaTeX formulation $W - R$

BTW, I wonder how to specify a variable for  $\alpha_+$ and $\alpha_-$?

We are adding the description of the alphas in the next release of the documentation.
The alpha+ and alpha- are the scaling factors that determine the magnitude of positive and negative weight updates.
For more details about how this work, please refer to the journal paper of the aihwkit: https://watermark.silverchair.com/041102_1_5.0168089.pdf?token=AQECAHi208BE49Ooan9kkhW_Er[…]KdKlId4X5KsiuUUaFqYGe2cZmJoSYJou-gpM92u156ezqbliMoProaFw

Thanks for your attention on that. But Could you check the link of the paper @kaoutar55 ? I got an error Token was not provided. Please go back to the article page and click the PDF link again. when I tried to open the link. Is it this paper?
Besides, I wonder whether there is any parameter in the toolkit for this alpha+ and alpha-?